Migration to standalone components

References

See: https://wiki.lyrasis.org/display/DSPACE/DSpace+8+Angular+%3A+library-based+architecture+proposal
Fixes: #2370

Description
The goal of this document is to migrate the DSpace Angular application to a standalone components approach.
Instructions for Reviewers
From a code review perspective the best approach to review this PR is to refer to the dedicated wiki page mentioned below.
From a functional perspective the best approach it to review this PR by running the application (against sandbox rest server) to verify everything works properly as the current sandbox environment.
List of changes in this PR:
See wiki page https://wiki.lyrasis.org/display/DSPACE/Migration+to+Standalone+Components
Checklist

[x] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & specs/tests), or I have provided reasons as to why that's not possible.
[x] My PR passes ESLint validation using yarn lint
[x] My PR doesn't introduce circular dependencies (verified via yarn check-circ-deps)
[x] My PR includes TypeDoc comments for all new (or modified) public methods and classes. It also includes TypeDoc for large or complex private methods.
[x] My PR passes all specs/tests and includes new/updated specs or tests based on the Code Testing Guide.
[x] If my PR includes new libraries/dependencies (in package.json), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.
[x] If my PR includes new features or configurations, I've provided basic technical documentation in the PR itself.
[x] If my PR fixes an issue ticket, I've linked them together.

@atarix83 : I've just merged the new ESLint rules PR (#2343).  So, if you (or someone at 4Science) can get this PR updated / cleaned up quickly, I'd love to test this early next week (either Monday or Tuesday).  The goal is to get this PR merged quickly to free up the work on upgrading to Angular 16 in #2752

@AndreaBarbasso : Please make sure to also resolve the merge conflicts in this PR (as I see you've been making updates).  It appears GitHub cannot run any of the automated tests/checks until the merge conflicts are resolved.  Thanks!

@AndreaBarbasso : Please make sure to also resolve the merge conflicts in this PR (as I see you've been making updates). It appears GitHub cannot run any of the automated tests/checks until the merge conflicts are resolved. Thanks!

@tdonohue, sure thing, we should be able to merge the main branch and resolve conflicts by Tuesday.
We would recommend, however, to start checking and testing the PR now, as the migration to the standalone components pattern is already in place and extensive testing is needed!

@atarix83 and @AndreaBarbasso :  Unfortunately, this missed our "soft deadline" of today (Tuesday). As mentioned in the DevMtg, I'll be out of the office for the remainder of the week.  I'll have to rely on your team and/or @artlowel to fully test this as much as possible this week.  When I return on Monday, I hope it can be merged quickly.
In the meantime, please also see if you can start #2752 , as I'm worried that may become more difficult to achieve in time for 8.0 as this PR is running behind.

thanks @tdonohue for your feedback we'll take a look

@tdonohue
point 1, 2, 4 and 6 should be fixed, still working on point 3 and 5


Updated Azure docs links

Replaced MSDN links
Made absolute links relative
Fixes #456

OPS Build status updates of commit 9075aa5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/rest/compare-graph.md
:white_check_mark:Succeeded

docs/rest/dotnet-tutorial.md
:white_check_mark:Succeeded

docs/rest/java-tutorial.md
:white_check_mark:Succeeded

docs/rest/node-tutorial.md
:white_check_mark:Succeeded

docs/rest/php-tutorial.md
:white_check_mark:Succeeded

docs/rest/python-tutorial.md
:white_check_mark:Succeeded

docs/rest/ruby-tutorial.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Activities. Renaming file with file-id gives move activity instead of rename

Describe the bug

Create a file: lorem.txt
Rename a file with file-id

curl -XMOVE 'https://host.docker.internal:9200/dav/spaces/{file-id}' \
-H 'Destination: https://host.docker.internal:9200/dav/spaces/{space-id}/lorem-renamed.txt' \
-uadmin:admin -vk

{
  "message": "{user} moved {resource} to {space}",
}

WIth file path:
curl -XMOVE 'https://host.docker.internal:9200/dav/spaces/{space-id}/lorem.txt' \
-H 'Destination: https://host.docker.internal:9200/dav/spaces/{space-id}/lorem-renamed.txt' \
-uadmin:admin -vk

{
  "message": "{user} renamed {oldResource} to {resource}",
}

For the ocis server there is no difference between move and rename. In all manners this is a move to the server.
For user facing notifications (activities/notifications/...) we change the naming so the user isn't confused. We use a simple logic everywhere. If the parent folder stays the same, this is a rename. Everything else is considered a move. So:

Sometimes the file goes to another folder, and with a different name - that is a "move and rename"?

The server would consider this a move, not a rename.
This is also the reason for this bug. Since we address the items by itemID, the path of these items will be empty, hence the server will consider this a rename.
The fix is probably somehow related to https://github.com/owncloud/ocis/issues/9712. Our events do not contain enough data to reliably reconstruct information from them.


Update composer package

I installed this package with composer and got a version with an outdated composer.json (didn't include changes from #10).
Pretty please update?

For now, I managed to get the latest version by changing my composer.json file to:
{
    "require": {
        "dilab/resumable.php": "dev-master"
    }
}


Modify Tests to Run on Consumer Hardware

This PR modifies the FFCV tests to work on consumer hardware by adding cpu_count checks and lowering the batch and size in test_cuda_nonblocking.py.
The test_cuda_nonblocking.py changes could be moved to a pytest config file with a pytest extension like pytest-testconfig, assuming pytest doesn't this built in and I am missing it.
The dataset writer tests currently hang on my machine, I believe I fixed this issue by merging in the fix-198 and FFCV 1.0 branches.

This looks like a great and easy fix, thank you!


optimize: when in TCC mode there is no need to delete global locks

Signed-off-by: slievrly<EMAIL_ADDRESS>

[x] I have registered the PR changes.

Ⅰ. Describe what this PR did
optimize: when in TCC mode there is no need to delete global locks
fixes #4233
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #4307 (8eb803a) into develop (11d3478) will decrease coverage by 0.17%.
The diff coverage is 50.00%.

@@              Coverage Diff              @@
##             develop    #4307      +/-   ##
=============================================
- Coverage      48.54%   48.37%   -0.18%
+ Complexity      3793     3790       -3
=============================================
  Files            724      724
  Lines          24240    24239       -1
  Branches        2971     2939      -32
=============================================
- Hits           11768    11725      -43
- Misses         11230    11251      +21
- Partials        1242     1263      +21

Impacted Files
Coverage Δ

...in/java/io/seata/server/session/GlobalSession.java
79.74% <50.00%> (+0.86%)
:arrow_up:

...er/src/main/java/io/seata/server/ServerRunner.java
50.00% <0.00%> (-15.39%)
:arrow_down:

...obuf/convertor/BranchRegisterRequestConvertor.java
90.47% <0.00%> (-9.53%)
:arrow_down:

...otobuf/convertor/GlobalBeginResponseConvertor.java
92.59% <0.00%> (-7.41%)
:arrow_down:

...n/src/main/java/io/seata/common/util/IdWorker.java
77.08% <0.00%> (-6.25%)
:arrow_down:

...otobuf/convertor/GlobalCommitRequestConvertor.java
93.75% <0.00%> (-6.25%)
:arrow_down:

...otobuf/convertor/GlobalStatusRequestConvertor.java
93.75% <0.00%> (-6.25%)
:arrow_down:

...obuf/convertor/GlobalRollbackRequestConvertor.java
93.75% <0.00%> (-6.25%)
:arrow_down:

...o/seata/server/coordinator/DefaultCoordinator.java
43.01% <0.00%> (-5.92%)
:arrow_down:

...protobuf/convertor/RegisterTMRequestConvertor.java
94.73% <0.00%> (-5.27%)
:arrow_down:

... and 19 more


Is this project still actively developed?

Is anyone is still working on this / using this or if it should be considered deprecated / abandoned / defunct? Looks like there has been very little activity since 2014.
According to this thread on the mailing list AppJS is no longer active, but I guess that didn't make it clear whether or not "deskshell" was part of AppJS or just a fork/off-shoot.

This project is no longer actively developed. The developers suggest that you use the electron framework from github: http://electron.atom.io/


TBI - Add utility function to project-access module to detect mta folder path

Description (include screenshots)
Managed MTA (multi-target application) has the mta configuration in an app router folder. This folder can contain multiple Fiori apps, and allow bundling and deploy of multiple Fiori apps together.
Value
It is useful to provide a utility function in project-access, which can look up the parent folder chain to find the app router folder path from a given Fiori app.
Architecture Elaboration
Does it requires architecture elaboration?
Discussed with @Klaus-Keller. Please comment on this.
Tasks

[x] Migrate the utility function
[x] Unit tests

@slin-sap is this completed with https://github.com/SAP/open-ux-tools/pull/1130?

Yes, functionality is in main branch, closing this ticket.


Tidied up some docs. Fixed some non-idiomatic usage of return in scal…

…a code.

Wow, that was quick! :-)
Thanks for the PR and the update.
Looks good to merge, IMO

+1 for merging. Thanks for your contribution @sksamuel :-)

Will merge this PR.


Update concentratord runtime stats_interval with the configured gateway stats_interval from a GatewayConfiguration ChirpStack message

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Summary
ChirpStack already relays the configured gateway stats_interval in a GatewayConfiguration message sent to concentratord. However, concentratord just seem to silently ignore the stats_interval and simply proceeds to channel reconfiguration, if required.
What is the use-case?

For gateways with cellular back-haul, having the possibility to remotely finetune the heartbeat message interval would be incredibly useful
There are situations in which reducing the heartbeat message interval temporarily would be useful (gateway installation, gateway servicing) as the heartbeat message also exposes dynamic metadatas/commands outputs that can carry insight about the gateway operational status

Implementation description
Looks like updating config.concentratord.stats_interval with the received new_config.stats_interval in function update_configuration of files src/handler/config.rs would do the trick.
I don't know why it is not already done in the first place.
Can you implement this by yourself and make a pull request?
Yes. However, my rust knowledge is rusty (aka non-existent) and it would without a doubt be cleaner if done by knowledged rust developer

Thanks for reporting, the above commit fixes this.


Humidity indicator source as attribute

My humidity is an attribute of my climate entity. I've currently setup a template sensor for it to show up, but it would be great to just set the source to an attribute of an entity.

If I understand what you need correctly, then here is an example:
- type: custom:mini-climate
  entity: climate.daikin
  indicators:
    humidity:
      icon: mdi:water
      unit: '%'
      source:
        attribute: ac_humidity


Validation pipeline fails on PDB-Dev 8 and 66

Parsing of the PDB-Dev entries 8 and 66 fails with the following trace:
Traceback (most recent call last):
  File "/IHMValidation/example/Execute.py", line 203, in <module>
    report = WriteReport(args.f)
  File "/IHMValidation/example/../master/pyext/src/validation/Report.py", line 26, in __init__
    self.input = GetInputInformation(self.mmcif_file)
  File "/IHMValidation/example/../master/pyext/src/validation/__init__.py", line 32, in __init__
    self.system, = ihm.reader.read(fh, model_class=self.model)
  File "/root/miniforge/lib/python3.9/site-packages/ihm/reader.py", line 3260, in read
    more_data = r.read_file()
  File "/root/miniforge/lib/python3.9/site-packages/ihm/format.py", line 589, in read_file
    return self._read_file_c()
  File "/root/miniforge/lib/python3.9/site-packages/ihm/format.py", line 640, in _read_file_c
    eof, more_data = _format.ihm_read_file(self._c_format)
  File "/root/miniforge/lib/python3.9/site-packages/ihm/reader.py", line 1247, in __call__
    a.append(self.sysr.ranges.get(obj, entity_poly_segment_id))
  File "/root/miniforge/lib/python3.9/site-packages/ihm/reader.py", line 191, in get
    return asym_or_entity(*self._id_map[range_id])
  File "/root/miniforge/lib/python3.9/site-packages/ihm/__init__.py", line 1289, in __call__
    return AsymUnitRange(self, seq_id_begin, seq_id_end)
  File "/root/miniforge/lib/python3.9/site-packages/ihm/__init__.py", line 1198, in __init__
    raise TypeError("Can only create ranges for polymeric entities")
TypeError: Can only create ranges for polymeric entities

The parsing code seems to be quite generic:
https://github.com/salilab/IHMValidation/blob/9d767d1ad6fc669661ee39a56db9212ae5be751d/master/pyext/src/validation/__init__.py#L29-L35
So I presume the problem is indeed in the cif files. PDB-Dev 8 has this in the header:
#
loop_
_entity.id
_entity.type
_entity.src_method
_entity.pdbx_description
_entity.formula_weight
_entity.pdbx_number_of_molecules
_entity.details
1 polymer man chr2L_60-161 ? 1 ?
#
<...>
loop_
_struct_asym.id
_struct_asym.entity_id
_struct_asym.details
A 1 chr2L_60-161
#

Which I guess causes failure at the asym.entity check from the python-ihm https://github.com/ihmwg/python-ihm/blob/0989b68412c01359e9f51aaf8413325532306737/ihm/__init__.py#L1196-L1201
@benmwebb I guess I need your advice on this: is this an actual artifact in the cif or it should be handled in the code?

Ah, I bet this is caused by ihmwg/python-ihm#70 - easy to check for by downgrading python-ihm to 0.27 to see if the problem goes away. PDB-Dev 8 has always been problematic because it does not provide a primary sequence. The proper fix in python-ihm would be to allow the user to override whether an entity is a polymer or not (usually it guesses based on the length of the sequence). In this case you know it's a polymer even though length==1 because the entity table in the input file says so.
Same deal with PDB-Dev 66 where entity 4 is a single GLY residue. This would normally be considered a nonpolymer but they have marked it as a polymer. This seems like a mistake to me but we should handle it better in python-ihm.

@aozalevsky, did this commit to python-ihm fix this issue for you? @brindakv will look at PDB-Dev 66 to try to properly fix entity 4, but python-ihm should work with both 8 and 66 now. If it does, I'll make a new python-ihm release shortly so that you can just pull the latest version into your docker/podman/singularity container.

yep, now both structures can be parsed, and reports seem fine. I guess the issue could be closed. Or better wait for python-ihm release to link in the issue?

Fix released in python-ihm-0.30


docs: update example links to opentelemetry-go-contrib repository

Go examples has been moved to opentelemetry-go-contrib repository in https://github.com/open-telemetry/opentelemetry-go/issues/5801.
This PR updates docs to point to the new location of the go examples.

@open-telemetry/go-approvers does this change also impact the "roll the dice demo" that we source from the go repo right now? If so we need to change that submodule to -contrib- as well.

Yes, the dice demo is indeed moved to the contrib repository.
https://github.com/open-telemetry/opentelemetry-go-contrib/tree/main/examples/dice

Yes, the dice demo is indeed moved to the contrib repository. open-telemetry/opentelemetry-go-contrib@main/examples/dice

thanks, let's fix that in a separate PR!


Add test for @EndpointExposed annotated class

Add at least one test to ensure that classes annotated with @EndpointExposed are correctly handled: not treated as an Endpoint but handled in subclasses.

This is now covered by the new React based integration test modules. They use ListRepositoryService and CrudRepositoryService that are annotated with @EndpointExposed and extended to handle a specific entity type


Fix more remote node reordering issues

My previous fix in https://github.com/Shopify/remote-ui/pull/160 only covered a subset of the reordering bugs in remote-ui. There were two other issues that caused problems in other cases:

We were not sending the correct index to appendChildBefore() in cases where the child was moving to a larger index in children in the same parent as it was already attached
We were not replicating the "append starts by removing from an existing node" behavior in the RemoteRoot object.

This PR fixes both of those issues. The code is a little gnarly and I feel like I could do the whole "map remote root to remote receiver" much more cleanly if I started from scratch, but it appears to fix the reordering bugs that the last PR did not.

@alxclark Could you give a brief description of the issues you were hitting? I'm curious if some of the bugs we're seeing at Stripe are related.


Support XXHash

Update: There is now a xxhash3 dub package that may be of interest.

Tried it, range violation on xxhash3-64 and xxhash3-128 using OOP API. Posted issue. Waiting for reply.


Clarify doc WRT SI unit system precedence

quantity.rst:
"The default unit system used when performing quantity operations is SI. This
ensures consistency between Quantity objects using different unit systems, such
as CGS or BT."
I have commented here: https://github.com/ansys/pyansys-units/pull/172/files#r1413859386
I guess the main things are:

doc exactly what the observable behaviour is that we're trying to specify here
avoid mentioning anything about implementation details (especially as this is rst)

Yeah, there were two issues about the documentation of the quantity class.  What's important is to document the behavior:
>>> from ansys.units import Quantity
>>> f = Quantity(value=10, units="ft")
>>> m = Quantity(value=2, units="m")
>>> f + m
Quantity (12.0, "ft")
>>> m + f
Quantity (12.0, "m")
>>>

but you might notice that this has a bug, I would have expected this output
>>> f + m
Quantity (16.56167979002625, "ft")
>>> m + f
Quantity (5.048, "m")
>>>

Looks like current behavior is adding together the raw values?


Allows uncommons-math tests to run on Travis-CI.

Updates Cobertura; replaces Mac-only Diehard binary with /usr/bin/dieharder. Splits Dieharder suite into 2 ant targets per PRNG class (necessary due to Travis's 50-minute timeout, unless you have a paid account with them). This is a newly-split partition of https://github.com/dwdyer/uncommons-maths/pull/16.

Closing, since this PR wasn't properly isolated on its current branch. Will reopen on a branch where everything else is reverted.


[BUG] Issue on redirect url after user is logged out on jwt refresh token expiration.

Describe the bug
I am using the JWT authentication on my project and it works fine when I log in and save the access token in my local or session storage. But whenever my refresh token is expired and the server responds with 401 status the user gets logged out which is fine. When the user is logged out the redirect URL must be the resource that the user is trying to access instead the redirect URL will be /login.
Steps To Reproduce
https://stackblitz.com/edit/refinedev-refine-voibwj?file=src/App.tsx
The steps to reproduce the issues is on the above link.
Expected behavior
For example:
Let us say I am trying to access the users resource from the menu but my refresh token is expired then I will be redirected to the login page with the URL as <domain>/login?to=/login instead of <domain>/login?to=/users
Screenshot

Desktop
No response
Mobile
No response
Additional Context
No response

Hi @dipbazz, Than you for reporting the issue 🙏
We started to working on this 👊

Hi @dipbazz thank you for the issue, we opened PR.
But, there are also some problems with the code you reproduced.
You need to reject or resolve with failedRequest.
const refreshAuthLogic = async (failedRequest: AxiosError) => {
     ...

    return Promise.reject(failedRequest)
}

You need to change status field to error.response.status.
checkError: (error) => {
      if (error.response.status === 401) {
        return Promise.reject(new Error('Your login sessoin expired. Please login again'))
      }
      return Promise.resolve()
}

If you have any more questions, please don't hesitate to ask. 🙏


Add Content for Developer-Workflow.md

Document Name:
Developer-Workflow.md
Task:
The Developer Workflow document should describe the development processes to follow within the project. It should explain our Git workflow.

Close as completed.


AJV always throws error "should be an object"

Hi There,
Bit of a noob in the realm of JSON schemas, NodeJS and ajv so please excuse me if there is something glaringly obvious I have overlooked here.
I have posted the different aspects of what I am trying to achieve below.
I have tested the same JSON schema and data sample using this online tool:
https://jsonschema.net/#/
and it passes validation fine using that tool.
However whenever I try to pass it using ajv running from within node I always get the following validation error raised:
[{
"keyword": "type",
"dataPath": "",
"schemaPath": "#/type",
"params": {
"type": "object"
},
"message": "should be object"
}]
I am assuming ajv thinks that the root object of the data passed in is not a valid json object ... ?
The only thing I am doing that is not really covered in the docs is requiring my JSON schema file from a relative path like so:
const assessChangesDtoSchema = require('./schemas/changeEventDto');
any help would be greatly appreciated.
Thanks greatly :)

What version of Ajv are you using? Does the issue happen if you use the latest version?
v 6.1.0
Ajv options object
No ajv initializing options are used

JSON Schema

{
"type": "object",
"definitions": {},
"$schema": "http://json-schema.org/draft-04/schema#",
"properties": {
"dateStamp": {
"$id": "/properties/dateStamp",
"type": "string",
"title": "The Datestamp Schema",
"description": "Date object in UTC string format.",
"default": "",
"examples": [
"2018-01-29T08:26:12.911Z"
]
},
"isReversal": {
"$id": "/properties/isReversal",
"type": "boolean",
"title": "The Isreversal Schema",
"description": "A boolean value indicating whether this change event is a reversal. Reversals are only possible if the change being reversed was the last change saved",
"default": false,
"examples": [
false
]
},
"editingSessionId": {
"$id": "/properties/editingSessionId",
"type": "string",
"title": "The Editingsessionid Schema",
"description": "The assessment editing session id.",
"default": "",
"examples": [
"a3c65ff7-ee67-44be-846a-7dc4441d96ca"
]
},
"changeItems": {
"$id": "/properties/changeItems",
"type": "array",
"items": {
"$id": "/properties/changeItems/items",
"type": "object",
"properties": {
"changeType": {
"$id": "/properties/changeItems/items/properties/changeType",
"type": "integer",
"enum": [0, 1, 2, 3, 4],
"title": "The Changetype Schema",
"description": "ChangeType enum { ClassFrequency = 0,  ClassNessecity = 1,  Recatorisation = 2,  NAFlag = 3,  Note = 4 }.",
"default": 0,
"examples": [
1
]
},
"txnId": {
"$id": "/properties/changeItems/items/properties/txnId",
"type": "integer",
"title": "The Txnid Schema",
"description": "The insights core API transaction id reference.",
"default": 0,
"examples": [
402128
]
},
"changedFrom": {
"$id": "/properties/changeItems/items/properties/changedFrom",
"type": ["integer", "boolean"],
"title": "The Changedfrom Schema",
"description": "In case of a ClassNess change its the ClassNess enum value { Mandatory = 1, Discretionary = 2 }. In case of a ClassFreq change its the ClassFreq enum value { Recurring = 1, OnceOff = 2 }. In case of a Category change its the Sub Category ID. In case of a NAFlag change its a boolean value.",
"default": 0,
"examples": [
2
]
},
"changedTo": {
"$id": "/properties/changeItems/items/properties/changedTo",
"type": ["integer", "boolean"],
"title": "The Changedto Schema",
"description": "In case of a ClassNess change its the ClassNess enum value { Mandatory = 1, Discretionary = 2 }. In case of a ClassFreq change its the ClassFreq enum value { Recurring = 1, OnceOff = 2 }. In case of a Category change its the Sub Category ID. In case of a NAFlag change its a boolean value.",
"default": 0,
"examples": [
1
]
},
"text": {
"$id": "/properties/changeItems/items/properties/changedTo",
"type": "string",
"title": "The Changedto Schema",
"description": "Only applicable to change of type 'Note'.",
"default": 0,
"examples": [
1
]
}
}
}
}
}
}
Sample data

{
"dateStamp": "2018-01-30T06:07:32.912Z",
"isReversal": false,
"editingSessionId": "32ca71b3-f110-4586-9fd8-f875d7f367b3",
"changeItems": [
{
"changeType": 1,
"txnId": 402128,
"changedFrom": 2,
"changedTo": 1
},
{
"changeType": 0,
"txnId": 402128,
"changedFrom": 2,
"changedTo": 1
}
}
Your code

const Ajv = require('ajv');
const ajv = new Ajv({schemaId: 'id'});
ajv.addMetaSchema(require('ajv/lib/refs/json-schema-draft-04.json'));
const assessChangesDtoSchema = require('./schemas/changeEventDto');
var validate = ajv.compile(assessChangesDtoSchema);
var valid = validate(assessmentChangesDto);
Validation result, data AFTER validation, error messages
[{
"keyword": "type",
"dataPath": "",
"schemaPath": "#/type",
"params": {
"type": "object"
},
"message": "should be object"
}]
What results did you expect?
I expected it to pass validation successfully.
I have tested the same schema and data within this online tool, and it passes successfully.
https://jsonschema.net/#/
Are you going to resolve the issue?

@BrettStrongEH How did you manage to fix the error?

Yes, I am having the same issue. please say how you fixed it.

you may be using ajv API incorrectly.

Hi Guys,
Sorry for late reply. To be honest I can't remember what I was doing wrong.
In an effort to help you, here the code I am using now ... hopefully you can spot the difference that makes the difference :) ...
const Ajv = require('ajv');
const ajv = new Ajv(); // options can be passed, e.g. {allErrors: true}
// const ajv = new Ajv({schemaId: 'id'});
ajv.addMetaSchema(require('ajv/lib/refs/json-schema-draft-06.json'));
// ajv.addMetaSchema(require('ajv/lib/refs/json-schema-draft-04.json'));
const assessChangesDtoSchema = require('./schemas/changeEventDto');
// Validate the incoming assessmentChange DTO json schema ...
var validate = ajv.compile(assessChangesDtoSchema);
var valid = validate(assessmentChangesDto);
//Where assessmentChangesDto is the posted json object to validate
The json schema file i have create an am using is as below ...
{
"type": "object",
"definitions": {},
"$schema": "http://json-schema.org/draft-06/schema#",
"required": [ "dateStamp", "isReversal", "editingSessionId" ],
"properties": {
"dateStamp": {
"$id": "/properties/dateStamp",
"type": "string",
"title": "The Datestamp Schema",
"description": "Date object in UTC string format.",
"default": "",
"examples": [
"2018-01-29T08:26:12.911Z"
]
},
"isReversal": {
"$id": "/properties/isReversal",
"type": "boolean",
"title": "The Isreversal Schema",
"description": "A boolean value indicating whether this change event is a reversal. Reversals are only possible if the change being reversed was the last change saved",
"default": false,
"examples": [
false
]
},
"editingSessionId": {
"$id": "/properties/editingSessionId",
"type": "string",
"title": "The Editingsessionid Schema",
"description": "The assessment editing session id.",
"default": "",
"examples": [
"a3c65ff7-ee67-44be-846a-7dc4441d96ca"
]
},
"changeItems": {
"$id": "/properties/changeItems",
"type": "array",
"items": {
"$id": "/properties/changeItems/items",
"type": "object",
"required": ["changeType", "txnId"],
"properties": {
"changeType": {
"$id": "/properties/changeItems/items/properties/changeType",
"type": "integer",
"enum": [0, 1, 2, 3, 4],
"title": "The Changetype Schema",
"description": "ChangeType enum { ClassFrequency = 0,  ClassNessecity = 1,  Recatorisation = 2,  NAFlag = 3,  Note = 4 }.",
"default": 0,
"examples": [
1
]
},
"txnId": {
"$id": "/properties/changeItems/items/properties/txnId",
"type": ["integer", "string"],
"title": "The Txnid Schema",
"description": "The insights core API transaction id reference.",
"default": 0,
"examples": [
402128
]
},
"existingOverrideId": {
"$id": "/properties/existingOverrideId",
"type": ["string", "null"],
"title": "The existingOverrideId Schema",
"description": "The existing overide record id in the database. If ",
"default": "",
"examples": [
"a3c65ff7-ee67-44be-846a-7dc4441d96ca"
]
},
"applicantId": {
"$id": "/properties/applicantId",
"type": ["string"],
"title": "The applicantId Schema",
"description": "The applicantId id in the database.",
"default": "",
"examples": [
"a3c65ff7-ee67-44be-846a-7dc4441d96ca"
]
},
"changedFrom": {
"$id": "/properties/changeItems/items/properties/changedFrom",
"type": ["integer", "string", "boolean"],
"title": "The Changedfrom Schema",
"description": "In case of a ClassNess change its the ClassNess enum value { Mandatory = 1, Discretionary = 2 }. In case of a ClassFreq change its the ClassFreq enum value { Recurring = 1, OnceOff = 2 }. In case of a Category change its the Sub Category ID. In case of a NAFlag change its a boolean value.",
"default": 0,
"examples": [
2
]
},
"changedTo": {
"$id": "/properties/changeItems/items/properties/changedTo",
"type": ["integer", "string", "boolean"],
"title": "The Changedto Schema",
"description": "In case of a ClassNess change its the ClassNess enum value { Mandatory = 1, Discretionary = 2 }. In case of a ClassFreq change its the ClassFreq enum value { Recurring = 1, OnceOff = 2 }. In case of a Category change its the Sub Category ID. In case of a NAFlag change its a boolean value.",
"default": 0,
"examples": [
1
]
},
"text": {
"$id": "/properties/changeItems/items/properties/changedTo",
"type": "string",
"title": "The Changedto Schema",
"description": "Only applicable to change of type 'Note'.",
"default": 0,
"examples": [
1
]
}
}
}
}
}
}

Hey guys,
I figured out what was happening, in my case.
I was sending the data for validation, in String format, instead of Object, this is why it complained, that the schema expected an Object type ("type": "object"), but in reality it was not an Object, it was a String.
To verify the type of your data (sent for validation), you could use typeof(data), to see if you are indeed sending an Object.

I'm getting this now too. However, unlike @rdiacenco  I am not passing any string property through that the schema is expecting to be an object. I've been going through my simple JSON data and schema line-by-line for an hour+ and there is no invalid data anywhere.

@locohost you can post a code sample that fails to runkit.com and post the link here.

I just hit this with fastify and took me a while to get to the bottom of it.
My problem was missing the response type:
response: mySchema,

Fix
response: {
   200: mySchema
}


Update the header editor to work in the 2.0 version of GraphiQL

Surprisingly, I think this mostly works. I originally intended to just try this because I wanted to mess with the 2.0 editor, but this actually seems to work surprisingly well.
I'm not sure if we wanted the headers editor as a plugin for 2.0, or something else?
I didn't spend that long on this, so if this isn't the route we want to go down that's fine by me. Like I said, I didn't originally intend for this to end up becoming anything.

hey, i'm terribly sorry but this was supposed to be a plugin. we need plugin API first!

2.0.0 alpha still has massive restructuring/rewriting to do before we can start introducing more features. this will be implemented as a plugin using the plugin API. See the pinned issue https://github.com/graphql/graphiql/issues/983

Fine by me, like I said I just did this on a whim :)

re-opened for reference, because I'm going to be borrowing most of this for the plugin!
the only difference is the imports really:
import { jsx } from 'theme-ui';

import React from 'react';

import { EditorWrapper } from '@graphiql/sdk';

import { useEditorsContext } from '@graphiql/sdk';
import { useSessionContext } from '@graphiql/sdk';

import type { EditorOptions } from '@graphiql/sdk';

and a plugin definition


Certain states should be excluded from the list of things to take action on

 Issue by jcoyne
Friday Sep 23, 2016 at 15:21 GMT
Originally opened as https://github.com/projecthydra/curation_concerns/issues/1034

For example, you may have the ability to "flag" everything, but we don't want that ability on the list. Potentially put this as a field in the config (e.g. shows_on_admin_page: false)

 Comment by mjgiarlo
Tuesday Jan 10, 2017 at 17:31 GMT

@jcoyne is this actionable now, or should we close it?

 Comment by jcoyne
Tuesday Jan 10, 2017 at 19:11 GMT

I believe this is actionable now

@vantuyls Do you want to slot this into MD milestone 2 or 3, or leave in its current milestone?

I'm not sure this is still needed with our current set of states/actions. Closing.


Total quantity cache column for Itemizables

Resolves #2416
Description
This effectively solves the N+1 query described in the issue, but also introduces a new practice to the project: counter caches.
The implementation is done using the gem counter_culture, which in my experience is performant, its well documented, and up to date. This gem allow us to add counter caches with more flexibility than the native Rails implementation, for example in this PR a polymorphic quantity total.
For this initial implementation, I'm replacing the instances of line_items.total with line_items_total that corresponds to the itemizable polymorphic parent of the entity using a new column with the same name. This change will require new resources using the Itemizable concern to add the column.
I thought about adding a conditional, but in my opinion, it makes sense to have the cache column if we have line_items.
Regarding performance, you can see the logs in this gist: "/diaper_bank/donations logs, old vs new". I'm using a cached environment since it's more realistic for production. But in summary, here is the important part:
Old
Completed 200 OK in 279ms (Views: 114.6ms | ActiveRecord: 33.7ms | Allocations: 191508)

New
Completed 200 OK in 160ms (Views: 67.6ms | ActiveRecord: 15.2ms | Allocations: 152202)

I see a lot of areas we can improve using counter caches, this will without a doubt make the website snappier and consume less resources.
A note about the last commit, I'm adding a migration in order to run the command LineItem.counter_culture_fix_counts that will update all the parents with the new total in their corresponding column. I'm not sure if this is something you usually do, I for instance, prefer to run the script in production directly or use data_migrations. Please let me know.
Type of change

New feature (non-breaking change which adds functionality)
This change requires a documentation update

How Has This Been Tested?
I included the corresponding tests and modified the ones that were failing.
In order to test this, you'll need to pull the changes, migrate, and you're good to go.
Any line_item create, update, destroy action should update the column of their parent.
Selfie

Also nice addition with the selfie!

@edwinthinks I just pushed a fix for the failing spec.
Regarding counter caches: in my experience if implemented well, and not doing something out of the Rails scope, they work like a charm. Counters are updated inside the transaction of the entity that is being touched. In the case of the test, FactoryBot was triggering them twice from memory.
That being said, if you execute raw SQL, counters are not going to be updated, but neither are callbacks. If you use update_column, for example, or delete.
In my opinion, it's worth the trouble, and also for the size of this application and the complexity, it shouldn't cause issues.
I'm all in with this solution and I'm willing to put more time in it if necessary since the 50% performance increase is worth it, but I will respect any decision that the RFG team needs to make.

Hey, @kinduff I appreciate the added context including your experience working in this! I agree with you that if it's done well it works like a charm!  But for a few reasons related to the nature of the project and "business" needs, I'd going to say we should avoid using the counter cache mechanism.

Regarding counter caches: in my experience if implemented well, and not doing something out of the Rails scope, they work like a charm. Counters are updated inside the transaction of the entity that is being touched. In the case of the test, FactoryBot was triggering them twice from memory.

I believe you 100% that it is implemented well that it works amazingly! However, this project is open source and will hopefully live longer than myself or you being the contributor. I can imagine the chances of overlooking and using update_column or even << to trigger an extra callback are pretty great IMO. The mindset that I'am trying to harbor is reducing the number of things newer contributors need to keep in mind and help them avoid chances of grievous issues that could put a negative impact on our users (in this case the line item count being wrong would be potentially devastating).

In my opinion, it's worth the trouble, and also for the size of this application and the complexity, it shouldn't cause issues.

I also agree with you that at its current application state and team make-up that it may be worth it. However, as mentioned above I want to be cognizant of newer developers that will inherit this project and won't have this context.

I'm all in with this solution and I'm willing to put more time in it if necessary since the 50% performance increase is worth it, but I will respect any decision that the RFG team needs to make.

In this case, I'd like to voice my disagreement on this. The performance increase is secondary (even tertiary) to the accuracy of the values we are giving as per the "business" needs. For context, our users don't care that it is slow even or have even noticed it. However, they will be very vocal when they notice that the numbers are off and potentially they will make some inaccurate decisions based on them.
I'd like to thank you again for working on this! I really enjoyed learning about counter_culture and possibly use it on a project at work + your selfie haha. Please let me know if you want to discuss any of the points above, happy to give more context and have a dialogue about it!

I agree with you in some things, in others I do not. My final stand is: if I were the maintainer of this project, I would push the change, but because I'm not, I'm okay with your decision. It's a request anyways.
Hope you can trust this implementation in the future, I'm sure it's going to come around again, counter caches always do, since it's one of the primary performance issues that are fixable this way to avoid N+1 queries.

@kinduff I appreciate your understanding and your flexibility to disagree and commit!

I agree with you in some things, in others I do not.

Love to talk with you sometime about this and discuss our different viewpoints. I certainly don't have the same experience with the library and its application to projects! I'am sure I can learn a ton from you!
Just to be clear are you interested in working on this issue #2444 still but using a different means?


spot_market_making alert: "xxxxx floating point exception  ./spot_market_making"

Describe the bug
After finishing all steps and config, when I enter "./spot_market_making", it shows "xxxxx floating point exception  ./spot_market_making".

Let me know whether this can solve your problem or not: https://github.com/crypto-chassis/ccapi/issues/270.

Floating-point exceptions also happen when the environment variables are not set.


Add performance metric monitoring for API in GCP

As a developer I want to see how the API is behaving so I can better diagnose performance issues.
Background
The API runs in the JVM, in a Docker container, on a GCP VM. Performance monitoring and metrics in the JVM are provided by JMX. Logging and monitoring is done using the GCP Monitoring web page. Currently we can only view the metrics for the VM hosting the Docker instance, but don't know what is going on inside the container. I want to be able to look at a dashboard which tells me the memory characteristics of the JVM.
Tasks

[x] Update the API's Dockerfile to configure JMX
[ ] Update Terraform deployment to:

[ ] Include configuration for JMX for Google's Stackdriver logging agent (see docs here and here)
[ ] Update API module to expose JMX port on Docker container to local machine

This article can be used for inspiration on configuring the Docker side of things.
Acceptance tests

When I go the Google Monitoring page of the project in which the API is deployed, I can see jvm metrics.

@mbdebian Is this done?

@mbdebian Is this done?

Not yet, I need to review the ticket alongside this year's roadmap

We are taking a platform-wide strategy for telemetry, which goes in a different direction.
I can see the changes in the Dockerfile, so the JMX interface is enabled, running on port 31238
Having a look at one of the development machines for platform, I can see the port is not exposed
mbernal@mbdevpf-api-0-api-57gg ~ $ docker ps
CONTAINER ID   IMAGE                                            COMMAND                  CREATED       STATUS       PORTS                                       NAMES
69c7f63f60af   quay.io/opentargets/platform-api:dir-of-effect   "/bin/sh -c 'ot-plat…"   7 hours ago   Up 7 hours   <IP_ADDRESS>:8080->8080/tcp, :::8080->8080/tcp   festive_matsumoto

If we look into the container, we can see the JVM listening for JMX connections
mbernal@mbdevpf-api-0-api-57gg ~ $ docker exec -it festive_matsumoto /bin/bash
root@69c7f63f60af:/srv/app# lsof -i -P -n | grep LISTEN
java      7 root  117u  IPv4  23705      0t0  TCP *:34653 (LISTEN)
java      7 root  118u  IPv4  23706      0t0  TCP *:31238 (LISTEN)
java      7 root  119u  IPv4  23709      0t0  TCP *:37281 (LISTEN)
java      7 root  123u  IPv4  17246      0t0  TCP *:8080 (LISTEN)

If we use a JMX command line friendly tool, we can establish a connection
root@69c7f63f60af:/srv/app# wget https://github.com/jiaqi/jmxterm/releases/download/v1.0.1/jmxterm-1.0.1-uber.jar
--2024-01-04 17:25:51--  https://github.com/jiaqi/jmxterm/releases/download/v1.0.1/jmxterm-1.0.1-uber.jar
Resolving github.com (github.com)... <IP_ADDRESS>
Connecting to github.com (github.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 302 Found
Location: https://objects.githubusercontent.com/github-production-release-asset-2e65be/1848246/2d26ed80-6bfb-11e9-9918-abd0675fa683?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAVCODYLSA53PQK4ZA%2F20240104%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20240104T172551Z&X-Amz-Expires=300&X-Amz-Signature=35aaccfcec40e7a32251c418c1ddfbcc28e42d02b2ec83715c15391e29824cd9&X-Amz-SignedHeaders=host&actor_id=0&key_id=0&repo_id=1848246&response-content-disposition=attachment%3B%20filename%3Djmxterm-1.0.1-uber.jar&response-content-type=application%2Foctet-stream [following]
--2024-01-04 17:25:51--  https://objects.githubusercontent.com/github-production-release-asset-2e65be/1848246/2d26ed80-6bfb-11e9-9918-abd0675fa683?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAVCODYLSA53PQK4ZA%2F20240104%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20240104T172551Z&X-Amz-Expires=300&X-Amz-Signature=35aaccfcec40e7a32251c418c1ddfbcc28e42d02b2ec83715c15391e29824cd9&X-Amz-SignedHeaders=host&actor_id=0&key_id=0&repo_id=1848246&response-content-disposition=attachment%3B%20filename%3Djmxterm-1.0.1-uber.jar&response-content-type=application%2Foctet-stream
Resolving objects.githubusercontent.com (objects.githubusercontent.com)... <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, ...
Connecting to objects.githubusercontent.com (objects.githubusercontent.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 7086101 (6.8M) [application/octet-stream]
Saving to: ‘jmxterm-1.0.1-uber.jar’

jmxterm-1.0.1-uber.jar                         100%[=================================================================================================>]   6.76M  --.-KB/s    in 0.04s

2024-01-04 17:25:51 (159 MB/s) - ‘jmxterm-1.0.1-uber.jar’ saved [7086101/7086101]

root@69c7f63f60af:/srv/app# java -jar jmxterm-1.0.1-uber.jar
Welcome to JMX terminal. Type "help" for available commands.
$>open localhost:31238
#Connection to localhost:31238 is opened
$>beans
#domain = JMImplementation:
JMImplementation:type=MBeanServerDelegate
#domain = com.sun.management:
com.sun.management:type=DiagnosticCommand
com.sun.management:type=HotSpotDiagnostic
#domain = java.lang:
java.lang:name=CodeCacheManager,type=MemoryManager
java.lang:name=CodeHeap 'non-nmethods',type=MemoryPool
java.lang:name=CodeHeap 'non-profiled nmethods',type=MemoryPool
java.lang:name=CodeHeap 'profiled nmethods',type=MemoryPool
java.lang:name=Compressed Class Space,type=MemoryPool
java.lang:name=G1 Eden Space,type=MemoryPool
java.lang:name=G1 Old Gen,type=MemoryPool
java.lang:name=G1 Old Generation,type=GarbageCollector
java.lang:name=G1 Survivor Space,type=MemoryPool
java.lang:name=G1 Young Generation,type=GarbageCollector
java.lang:name=Metaspace Manager,type=MemoryManager
java.lang:name=Metaspace,type=MemoryPool
java.lang:type=ClassLoading
java.lang:type=Compilation
java.lang:type=Memory
java.lang:type=OperatingSystem
java.lang:type=Runtime
java.lang:type=Threading
#domain = java.nio:
java.nio:name=direct,type=BufferPool
java.nio:name=mapped,type=BufferPool
#domain = java.util.logging:
java.util.logging:type=Logging
#domain = jdk.management.jfr:
jdk.management.jfr:type=FlightRecorder

To complete this, it would need either an easy way of inspecting the metrics in real time when we are working on a release, and / or to complete the connection with the Stackdriver subsystem on GCP.
As we are going on a different, platform-wide, direction on telemetry, completing the works on this ticket doesn't really apply anymore.
@remo87 , would you mind verifying the findings on the metrics collection side of the ticket?. Also, unless you find an easy and quick way of getting CPU, memory and I/O usage in real time for development in a release cycle on a deployed API, I would probably save the overhead of computing these metrics and put the energy on the strategy we're following for telemetry.


Consider collecting aws-appmesh.log

The recent App Mesh CNI integration with proxyConfiguration logs to /var/log/ecs/aws-appmesh.log.yyy-mm-dd-hh
Raising this to create awareness and determine if this is useful to collect in the ecs-logs-collector

We collect all the files stored in the folder /var/log/ecs.
https://github.com/awslabs/ecs-logs-collector/blob/master/ecs-logs-collector.sh#L373


Unexpected behavior when callback throws an error

Consider the following small modification of the UrlShortener example in the README:
urlshortener.url.get(params, (err, res) => {
  if (err) return console.error('The API returned an error: ' + err);
  console.log(`Long url is ${res.data.longUrl}`);
  throw new Error('This error is not from the API');
});

Expected behavior
The final error is bubbled up the stack and logged in the console.
Actual behavior
The final error is funneled back into my callback and treated as an API error:
Long url is https://www.google.com/
The API returned an error: Error: This error is not from the API

This seems to be a side effect of setting the callback as the catch() handler for the promise. The catch executes not only for errors resulting from the API call but also for any errors thrown in the then() handler for a successful call.
I'm not sure what Promise interface Axios supports, but I think the solution would be to set the callback as the error handler for a specific then(), instead of as a catch():
createAPIRequestAsync<T>(parameters)
        .then(r => callback(null, r),
              e => callback(e));

Thanks for calling this out!  Also double thanks for including the fix :) I submitted a PR over in #1161, so hopefully this clears things up!

No problem, thanks for the quick PR!


Console fix up with mutual TLS

This PR is based on https://github.com/redpanda-data/helm-charts/pull/1386 to solve early return problem in go template.
Client certificates does not relay on secret reference, that's why checks are removed.
Add integration test that setup mTLS for all listeners (Kafka, Admin API, HTTP Proxy, Schema Registry, RPC).
Verification in rpk client is not always done using rpk. curl should be replaced with port-forward and simple go based http client Get/Post/Delete implementation.
Fix ClientAuthRequired check as Dig function should start from listeners object.
Expose, in Redpanda values, Console config and chunk of Connectors values.
Mount all client certificates (ca, certificate and key) in Console deployment.
Move Admin API configuration from env vars to ConfigMap.
Reference
https://docs.redpanda.com/current/develop/http-proxy/
https://docs.redpanda.com/current/manage/schema-reg/schema-reg-api/

I added new CI execution that using kind cluster is performing new mTLS test cases
https://github.com/redpanda-data/helm-charts/actions/runs/9864729172/job/27240338010?pr=1402#step:8:1111
https://github.com/redpanda-data/helm-charts/actions/runs/9864729172/job/27240338010?pr=1402#step:8:1116
P.S. I don't understand why REDPANDA_LICENSE is not found, so that tiered-storage-secrets is skipped.
https://github.com/redpanda-data/helm-charts/actions/runs/9864729172/job/27240338010?pr=1402#step:8:1039


fix/allow absolute or relative path

Could you provide some use cases?

I test it. It breaks my workflow:

I have a bunch of data in dataset directory. I save the config.json in each collection directory.


Fix to easyClose function

This look right?

Looks great to me! I'll merge when I have a few minutes to tag a new release and publish. Thanks!


Update get-networkcontrollerloadbalancerconfiguration.md

Added example and descriptions

@AnirbanPaul,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


single-spa errors being swallowed

I'm putting together an example and observed that single-spa errors are being swallowed when using single-spa-layout.
Steps to reproduce

clone repo at this commit https://github.com/filoxo/single-spa-example-rxjs-shared-state/commit/f7242c7253f825d7d2fac706baf23e594c4f3f9b

follow the setup instructions in README to start running locally

navigate to http://localhost:9000/ and inspect the console. There are no errors.

edit root-config/src/example-root-config.js and comment out the single-spa-layout code and replace with registerApplication equivalents (below)
registerApplication({
  name: "@example/navbar",
  app: () =>
    System.import("@example/navbar"),
  activeWhen: ["/"],
});

registerApplication({
  name: "@example/home",
  app: () =>
    System.import("@example/home"),
  activeWhen: ["/home"],
})

registerApplication({
  name: "@example/login",
  app: () =>
    System.import("@example/login"),
  activeWhen: ["/login"],
})

go back to page and inspect the console. single-spa errors are being shown

Current behavior
What's happening is that the home and login applications are dying with a status of LOAD ERROR (visible in the single-spa inspector) but there are no console errors. I switched to using the registerApplication API and then started seeing the error message that explained what is happening (systemjs cannot resolve the bare specifier that these applications require).

I understand what I need to do to fix the error itself, but it was very confusing/frustrating that I wasn't seeing those error messages when using single-spa-layout.
Expected behavior
I would expect that if a single-spa error occurs that that error would be surfaced the same way as if using the registerApplication API.

This sounds like something that should be fixed.
The installation didn't work for me, though. 1I'm using<EMAIL_ADDRESS>and I suspect the project may require yarn 2? I was able to get it working by running yarn install in each folder, though.
Once I ran the projects, the home application mounted successfully both when using single-spa-layout and when not using single-spa-layout.

If I forcibly cause the home app to not load, by changing its url to one that doesn't exist, then I'm able to reproduce the problem.

I think the reason that I wasn't able to reproduce it at first was that I didn't check out the proper commit.

Fixed in https://github.com/single-spa/single-spa-layout/releases/tag/v1.1.4. Thanks for reporting and providing a demonstration - that made it much easier.

👏  nice!


.github: set lvh image tags with -latest

Since lvh image changed their tags to include -latest we need to update them manually so that renovate bot can pick them up automatically.

/test-backport-1.12

closing as explained in https://github.com/cilium/cilium/pull/27415


Could you recapture all the data using --json switch for smartctl?

Consider to use --json=o  to include original output as JSON array smartctl.output[].

@chrfranke, I feel like we'll have to write a parser for smartctl output... BTW, what is your project/research?

I agree that linuxhw/SMART and the similar project bsdhw/SMART should use JSON output if smartctl >= 7.0 is available. With option --json=o, the traditional output could be easily extracted later (for example with: jq -r '.smartctl.output[]' ).
BTW, my project is smartmontools itself :-)

Do we need traditional output? Can't it be recreated from JSON?

JSON output does still not include all info from traditional output. ATA/SATA is fairly complete (see ticket 767), but SCSI/SAS and NVMe are not. Try --json=u to see affected lines as strings named smartctl_NNNN_u.
No, there is no easy way to recreate traditional output from JSON output.

Recapturing of existing probes is not possible because people usually don't create more than one probe of the computer.
We can capture this json output in addition to default one in the next version of the hw-probe utility.

Is it not possible to generate such json output from default plain format by a script?

Is it not possible to generate such json output from default plain format by a script?

Should be possible, but it is damn a lot of work - to create a convertor for a plain-text format and then debug and test it.

As I just posted in the hw-probe repo (https://github.com/linuxhw/hw-probe/issues/156) ... Not having the parsed data makes things much more difficult. It's clear that something is parsing the data and generating the Markdown files... But those scripts/programs don't appear to be available?
It would also be useful to understand the methodology/code being used to generate some of the Markdown tables... (Not that I see any particular issues with the methodology ... but not having the code that generates the reports makes independent/peer review next to impossible.)


CI: Set up Buildkite

E2E tests and packaging remain in cloudbuild for now, due to IPFS and docker compose shenanigans. As we're moving away from IPFS, we'll sunset cloudbuild wholesale.
Fixes radicle-dev/cocopoco#7

we're moving away from IPFS

@kim is there more information about this?


Separated NPM production/development Dependabot upgrade rules

Current dependencies on/for this PR:

main

PR #158   👈

This comment was auto-generated by Graphite.


Arnej/queue input to local rpc monitor map

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.
@havardpe please review

looks good (except the abort inside the switch). Not able to review (github issues?)


Tloz BoTW 560p & 360p causing visual glitches while zooming

Whenever i zoom with the Sheikah slate (using the scope)  most of the textures go transparent & the other turn black, & it stays like that after de-zooming too, only fix is to restart the game :/

#65


Fix event dispatcher dep

After composer require symfony/console I get this error:
Fatal error: Class 'Symfony\Component\EventDispatcher\Event' not found in /var/www/phpgraph/vendor/symfony/console/Event/ConsoleEvent.php on line 25

I think this should be by default because it's not optional.

My mistake. Sorry.


add MODERATE_MEMBERS permission

Pull Request Etiquette

[x] I have checked the PRs for upcoming features/bug fixes.
[x] I have read the contributing guidelines.

Changes

[ ] Internal code
[x] Library interface (affecting end-user code)
[ ] Documentation
[ ] Other: _____

Closes Issue: NaN
Description
this PR adds the MODERATE_MEMBERS permission which is used for timing out members (already on canary, not an experiment). in case someone makes a PR for implementing timeouts, I'll close this PR in favor of adding the permission with that PR.

closing in favor of #1902


chore: Catch error for MTU Exchange

docs: add small trouble shooting section
Thanks for this project! Got it connected to my controller this evening after running into the silly error of looking at my controller on my phone while trying to connect!
I found that I was running into an issue with discovering services on the controller that corresponded to this ATT request failed error. I noticed that this error was actually happening earlier in the flow, first at the MTU Exchange, and noticed the error wasn't being caught. I added an error check there.
Very open to feedback on this!
A note: I would love to extend this repo into something that can be used as a go single binary CLI for the controllers. I would think it'd be more appropriate as a separate repo, are you aware of any similar project in Go?

Thanks for this project! Got it connected to my controller this evening after running into the silly error of looking at my controller on my phone while trying to connect!
Glad you're finding it useful. Yes, the controllers apparently only support once paired device at a time, it makes sense to add a note about that to the docs.

I found that I was running into an issue with discovering services on the controller that corresponded to this ATT request failed error. I noticed that this error was actually happening earlier in the flow, first at the MTU Exchange, and noticed the error wasn't being caught. I added an error check there.
Thanks, I agree that it makes sense to log an error here.

A note: I would love to extend this repo into something that can be used as a go single binary CLI for the controllers. I would think it'd be more appropriate as a separate repo, are you aware of any similar project in Go?
I am not aware of a similar project. Having a single binary CLI does seem like it could be useful and I agree that it's out of scope for this repo. If you want such a binary to operate a traditional UNIX-like command which spins up a new process for every command (as opposed to a single process with an interactive session) you will probably want to keep a daemon process in the background to hold the bluetooth connection otherwise latency and error rate are likely to go through the roof. I can think of basically three ways to go about it:

have your client talk to this bridge over an mqtt relay and submit patches to this repo to support more functionality as needed
fork this repo to have the client use a different IPC mechanism with the daemon process
use bluez instead of raw hci sockets, which basically means talking to an existing daemon over D-Bus, I couldn't get it to work reliably in my case but your mileage might vary

I would happily welcome patches if you go with option 1. I would also potentially be open to accepting a patch for an alternate IPC channel to allow equivalent request/responses to be made via MQTT or another local IPC channel for use by a CLI depending on how much complexity that adds.

Thanks for your thoughts on the CLI project. I think I'll end up forking this and doing this base as the long running daemon, and doing method 2.
I actually don't use MQTT or Home Assistant, my curiosity has been mostly around reverse engineering the controller so that eventually I can have my own hardware controller, but have it be compatible with AC Infinity devices. Until I learn everything I will need to learn to be able to do that, I'll be hacking on this to try and get some better behavior around device shut off/turn on around chasing VPD. I don't think the controller software does a very good job at learning how long to run devices/how they approach things like Target VPD.


Track whether actors emit sound and use in hearing detection tests

I think it might be better to default emits sound to false, then flip the default for creature. Because I think vehicles and siege weapons currently emit sound.


Replace references of openjdk-systemtest with aqa-systemtest

Related to : https://github.com/adoptium/aqa-systemtest/issues/424
Signed-off-by<EMAIL_ADDRESS><EMAIL_ADDRESS>

Works fine in non-parallel build. Sanity.jck green: Test_grinder/job/Grinder/14986/


Check in IAM policy for kubeflow-examples.

kubeflow-examples is the project that hosts our example containers.

Grant the google group kubeflow-examples-gcr-writers storage admin permission
on project kubeflow-examples. We will use this group to grant folks
permission to push GCR containers to the examples repo.

This change is

/assign @abhi-g
/assign @IronPan

/lgtm

/lgtm
/approve

/approve


[fix][admin] Passing force parameter to deleteNamespaceBundle when delete namespaces

Motivation
In the current implementation, don't pass the force parameter to deleteNamespaceBundle when deleting namespaces, we should pass the force parameter.
Modifications
Passing force parameter to deleteNamespaceBundle when delete namespaces.
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository: https://github.com/coderzc/pulsar/pull/20

Codecov Report

Merging #18153 (20f2bb6) into master (6c65ca0) will decrease coverage by 2.70%.
The diff coverage is 19.20%.

@@             Coverage Diff              @@
##             master   #18153      +/-   ##
============================================
- Coverage     34.91%   32.21%   -2.71%
+ Complexity     5707     4250    -1457
============================================
  Files           607      393     -214
  Lines         53396    43489    -9907
  Branches       5712     4470    -1242
============================================
- Hits          18644    14008    -4636
+ Misses        32119    27420    -4699
+ Partials       2633     2061     -572

Flag
Coverage Δ

unittests
32.21% <19.20%> (-2.71%)
:arrow_down:

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

Impacted Files
Coverage Δ

.../main/java/org/apache/pulsar/PulsarStandalone.java
0.00% <0.00%> (ø)

.../apache/pulsar/broker/admin/impl/ClustersBase.java
8.57% <0.00%> (-0.48%)
:arrow_down:

.../pulsar/broker/admin/impl/SchemasResourceBase.java
38.38% <0.00%> (-7.08%)
:arrow_down:

.../org/apache/pulsar/broker/admin/v2/Namespaces.java
11.77% <0.00%> (+3.74%)
:arrow_up:

.../pulsar/broker/service/AbstractBaseDispatcher.java
43.55% <0.00%> (-2.31%)
:arrow_down:

...che/pulsar/broker/service/BacklogQuotaManager.java
9.09% <0.00%> (-0.40%)
:arrow_down:

.../pulsar/broker/service/BrokerServiceException.java
27.77% <0.00%> (+2.77%)
:arrow_up:

...ava/org/apache/pulsar/broker/service/Consumer.java
64.36% <0.00%> (+2.35%)
:arrow_up:

...ava/org/apache/pulsar/broker/service/Producer.java
58.28% <0.00%> (-1.36%)
:arrow_down:

...pulsar/broker/service/PulsarCommandSenderImpl.java
63.07% <0.00%> (-3.59%)
:arrow_down:

... and 323 more

/pulsarbot run-failure-checks


Option for no-poll?

The previous version included a --no-poll flag so the script wouldn't wait for operations to complete before returning to the command prompt. Are you planning to add this to CLI v2?

We'll add it. Thanks for pointing that out.

This is particularly helpful when running terminus sites create in a script. In terminus 1.0 psite-create returned the site UUID and the site name before ending the local process. This allows us to do a bunch of other processing related to the site while we are waiting for the spin-up to complete.  It would be great if that functionality could be preserved in this version.

Hey All,
Just wanted to close this thread out. At the moment we aren't ready to restore a --no-poll style option, because what we've seen is that this can easily be misused to create a lot of sites programatically, with less than stellar results.
The "right" answer to this problem is for us to implement proper rate-limiting and quotas on the platform side. Until that's done and we're confident about it, we'll have to hold off on restoring this old functionality.


Update MC to use openassetio from PyPI

What
Update the project to use openassetio from PyPI:

setup.py dependency (+migrate to pyproject.toml)
Docs

Why
Manual build/installation is no longer needed
Notes
This will require #10  to be done first.

After discussion with @feltech , decided we're going to do this task first in the update effort, updating the hand rolled traits to modern openassetio, before embarking on #10 to autogenerate them.


Cannot install FileTimesPropertySheet correctly

Hello,
I was able to make work the FileTimesPropertySheet extension on my developpement machine (64bits) using regasm.
So I tried to install the extension on another machine using Visual Studio 2015 Installer. I followed the instructions on the CodeProject page. I used srm.exe to register the extension as advised.
The install sequence worked and I could see the registry entries with regedit (at the good place as it is a 64 bit machine : HKCR{...}\CLSID...). I checked with ShellExtView that the extension is registered.
Nevertheless, the extension didn't show up by right clicking on a file.
Did I miss something?
Thank you for your advises

Hello,
here is the situation as today:

I checked that that I have followed the recommandations of the post "Troubleshooting SharpShell Servers.md" https://github.com/dwmkerr/sharpshell/blob/master/docs/Troubleshooting SharpShell Servers.md

the server manager can find the COM server in the other machine (not the development machine)
the Install procedure worked on the development machine and the proprty sheet could show up (and not on the other machine)

So I wonder if there is a component missing (I installed Visual C++ redistributables)

Hello,
problem solved.
I started again from scratch:

downloaded the latest version of Sharpshell from Github
set solution configuration with Release and AnyCPU everywhere (COM visible)
regenerate Sharpshell
install Visual C++ 2012 redistributables (x64 AND x86)
tried manually srm on the target machine : OK
then tried the installation package (written with Wix Toolset) 🆗

So Dave's advises in his  troubleshootings post  worked.
As allways, editing a post let me think about it and solve the issue by myself...for this ime.
Thank you for you patience and congratulations @dwmkerr  for this great and usefull tool !
Christophe

Thanks @cboutonnet glad it all worked out!


support tmux 2.4

TMUX has changed the way bind-keys works in version 2.4+ This change
supports both older and newer versions.
In addition, I added tmux-is-at-least bash function to help with this and future changes in the TMUX commands.

Hope to this get merged soon. Can't use tmux-open with tmux 2.4 at all 😭

@bruno- merge please....

@docwhat thanks for the PR.
I've added you was a maintainer to this repo.


Add screenshots of the different prompts

Would be a lot faster for users to pick a prompt if you included a screenshot of each of them in the list.
Example: https://github.com/sindresorhus/quick-look-plugins

It is good for user to understand but we need to set a limit of items because if we add a lot of images, it will take a time to load.

@sotayamashita Yes, I agree with this. The only solution I can think of is create (and maintain) a real single image with a collage of the screenshots. I had thought about that before, but it's a lot of work and to make matters worse, themes sometimes change.

Or just pick the top / best looking 5 prompts to have screenshots for and list the rest normally. That's what I would do.

@sindresorhus Didn't think of that one. Thanks.

Or just pick the top / best looking 5 prompts to have screenshots for and list the rest normally. That's what I would do.

Eh, that sounds like a sure fire way to start no end of squabbles.

Maybe we can do it like this:

https://github.com/jbucaran/awesome-fish/pull/38#issuecomment-302992710

Should be nice to have.


⚠ Bofrost.fr has degraded performance

In 6bc444f, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1620 ms

Resolved: Bofrost.fr performance has improved in 236db4f after 8 minutes.


Come up with a better name

Having a hard time coming up with an imaginative name for this. Suggestions welcome.

thinking of dubbing it "wIRC" for lack of a more creative name.

What about bwic or bwirc, for Better Web IRC Client :) ?

Thanks for the ideas, but this has just been resolved  :-)
I'm combining web-irc with Subway, a similar client started recently. The new repo is here:
https://github.com/thedjpetersen/subway


feat: Changes to gear.json

Made some changes to gear.json:
Added Price and Durability
Updated names to match item.json
Second PR Hopefully its right this time 😅

nice!


Correctly bypass sync calls in UIManager during remote debugging

Summary
Remote debugging stopped working (since 0.58, according to #23254). See https://github.com/facebook/react-native/issues/23254#issuecomment-474692753 for repro steps.
The root cause is incorrect checks for Chrome debugging environment in UIManager.js.

In one place where sync function lazilyLoadView should be avoided, we effectively use if (__DEV__ && !global.nativeCallSyncHook) to check remote debugging, which misses ship flavor (i.e. __DEV__ is false).
In another place where we want to pre-populate view managers' constants to avoid calling sync function getConstantsForViewManager, if (__DEV__) is used, also missing ship flavor.

This PR fixes both checks, only using the absense of global.nativeCallSyncHook to determine remote debugging environments.
Changelog
[JavaScript] [Fixed] - Correctly bypass sync calls in UIManager during remote debugging
Test Plan
(Copying repro steps from https://github.com/facebook/react-native/issues/23254#issuecomment-474692753)
On a freshly created project by react-native-cli in android:

Shake phone -> Dev Settings -> Uncheck JS Dev Mode and Use JS Deltas. Then back.
Shake phone -> Turn on Debug JS Remotely. Then back
Reload

Current behavior: Invariant Violation: requireNativeComponent: "RCTView" was not found in the UIManager.
Fixed (expected) behavior: No errors, app executes correctly.

Something is wrong with our Facebook infra, retrying…


BibEdit: Use bibupload quarantine status for document locking

Originally on 2011-02-15
If it makes sense, use bibupload quarantine status for internal document locking.  If it doesn't make sense, then at least respect when documents are marked as quarantined and give the user a helpful error message (using #491) to tell them that the document is in an unsteady state because of another bibupload job (which could be someone else editing with bibedit, but could also be a long-running load).

:+1:


Labelings lose all information about values in original images

Here is a MCVE, on labeling 0.1.12
from labeling.Labeling import Labeling
import numpy as np

data = np.zeros((4, 4), np.int32)
data[1, 1] = 4

print('Original data:\n', data)

img, metadata = Labeling.fromValues(data).get_result()

print('Data from result: \n', img, vars(metadata))

And the printout on my machine, running Ubuntu 20.04:
(napari-imagej-dev) : gselzer@gselzer-OptiPlex-5090 ~/code/imagej/napari-imagej (roi-support*) [napari-imagej-dev] »
python tmp.py                                                                                #  0 {2022-05-05 10:18:57}
Original data:
 [[0 0 0 0]
 [0 4 0 0]
 [0 0 0 0]
 [0 0 0 0]]
Data from result:
 [[0 0 0 0]
 [0 1 0 0]
 [0 0 0 0]
 [0 0 0 0]] {'version': 2, 'numSets': 2, 'numSources': 0, 'indexImg': None, 'labelMapping': {}, 'labelSets': {'0': [], '1': [1]}, 'metadata': None}

As the MCVE shows, looking at the result shows no trace of the value 4.
I do concede it is tricky to retain the values in the result image from the original data given to the Labeling, e.g. when multiple input images have the same non-zero values. But, if this is the behavior, it should be documented somewhere. And, despite the limitations, I'd think we should try our best to ensure that you can get the original values from the result.
@tomburke-rse what do you think? Am I missing something?

You missed nothing.
I can see that adding documentation for that is a good idea. The reasoning behind not retaining old information is that the original label does not hold intrinsic value beyond separating the label from the other labels in the image.
I see no reason to add the complexity of retaining information because I would not know how to handle the very likely case of two images with the same label being added, possibly even at the same spot. Are they different? Are they the same? I cannot tell from the library side.
What I can see is to add the option to add a static mapping or a mapping function similar to the Java library. Then you could use the mapping to reconstruct from result back to data

What I can see is to add the option to add a static mapping or a mapping function similar to the Java library. Then you could use the mapping to reconstruct from result back to data

This sounds great!

@gselzer is this just something you noticed and wanted clarified or does this produce actual problems for anything else? Just so that I can prioritize the issue.

This is not urgent, it just affects what I can assert in my napari-imagej conversions.
See https://github.com/imagej/napari-imagej/commit/6a67a661f9bf4f9de3689ce0acd8d04bdcef2c58#diff-9e1a0edede8c7f299251e523eb4010ed868090b397db9bf7881c18ab3dfdbf18R160

I see, thanks @gselzer! The way you assert is looks absolutely right though. At least for one source image.

Thanks for looking over that function :sweat_smile:. I think the assert covers the use cases for napari-imagej, since we will only ever be adding one source image to the Labeling.


Scene view not working on release

I have applications everything works in debug mode, however when I build release then scene view does not work? Should I add something to proguard file? I use geospatial api and render models using ViewNode and ModelNode

Solved by adding it to proguard file
Keep classes related to Google Play Services Location for ARCore Geospatial API
-keep class com.google.android.gms.location.** { ; }
-keep class com.google.android.gms.common.api.* { ; }
-keep class com.google.android.gms.tasks.* { *; }


Add capability to specify a custom folder when using dapr init --slim command

Describe the proposal
It is possible to install the CLI on a specific folder but not the dapr.exe.
The code uses always the DefaultConfigFilePath() which is using the  homeDir, _ := os.UserHomeDir().
I would like to have a capability to specify another "home" directory. The company where I am working we have not the right to install non signed exe anywhere and we are not admin of our pc. But we have an open door on a specific folder where we can (letting us still developing something 🙄).
Is it something you can imagine?
The work around is to temporarily change the home environment variable before running the dapr init --slim command.
func createSlimConfiguration(wg *sync.WaitGroup, errorChan chan<- error, info initInfo) {
defer wg.Done()
if !info.slimMode {
	return
}

// For --slim we pass empty string so that we do not configure zipkin.
err := createDefaultConfiguration("", DefaultConfigFilePath())
if err != nil {
	errorChan <- fmt.Errorf("error creating default configuration file: %w", err)
	return
}

}
RELEASE NOTE:
ADD custom installation directory for dapr init --slim command.

Another flag could be added to dapr init for specifying custom folder which will be used instead of HomeDir to install dapr binaries as well as config files etc. Something like dapr init --dapr-dir <path_to_local_dir>

The issue here is dapr always looks to a particular path ~/.dapr/bin for the daprd binary. If it is installed in some other location, each time dapr CLI is run, then it would need to know where the binary is installed at.
This needs more discussion.

Hi @mukundansundar, could we imagine to add another environment variable only when this feature --dapr-dir is added. In this case the dapr exe can check if the environment variable exists. If not  it can take the default one ~/.dapr/bin?

Just looked at the code - dapr run command is picking the bin dir from https://github.com/dapr/cli/blob/747dc3f7aeeb525e1f71762e4986bd0e11f54442/pkg/standalone/run.go#L293
https://github.com/dapr/cli/blob/747dc3f7aeeb525e1f71762e4986bd0e11f54442/pkg/standalone/common.go#L30
So, it seems if we provide an alternate install path then dapr run should pick the binary correctly ?


Internal server error" error="[plugin.downstreamError]

Describe the bug
A clear and concise description of what the bug is.
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Network data
If it's related to metric data visualization would be great to get the raw query and response for the network request (check this in browser dev tools network tab, there you can see metric requests, please include the request body and request response)
Software versions

Grafana
Zabbix
Grafana-Zabbix Plugin

9.4.7
5.4.12
4.3.1

logger=context userId=3 orgId=1 uname=vst t=2023-06-07T00:41:04.424940291+07:00 level=error msg="Internal server error" error="[plugin.downstreamError] failed to query data: Failed to query data: rpc error: code = Canceled desc = context canceled" remote_addr=<IP_ADDRESS> traceID=

logger=context userId=3 orgId=1 uname=vst t=2023-06-07T00:41:04.425072697+07:00 level=error msg="Request Completed" method=POST path=/api/ds/query status=500 remote_addr=<IP_ADDRESS> time_ms=17757 duration=17.757257192s size=116 referer="http://<IP_ADDRESS>:3000/d/lsnIOW_Vk/dispetcher?kiosk=&orgId=1&refresh=30s" handler=/api/ds/query

logger=context userId=3 orgId=1 uname=vst t=2023-06-07T00:41:04.425872475+07:00 level=error msg="Internal server error" error="[plugin.downstreamError] failed to query data: Failed to query data: rpc error: code = Canceled desc = context canceled" remote_addr=<IP_ADDRESS> traceID=

To solve the problem, set the value Max data points to 200


Added "Falling Words" game

I added my game, if it's approved i can work on it

Ok i finished the new game mentioned in #5

i've done review your code and check the log. this error is caused by some unused variable. i put the log below.

It checks if the text falling is the same as the text you added, so what you write should be using the Japanese alphabet. I thought that users could learn how to learn the Japanese keyboard

thank you for adding romanised alphabet, but it still error cause react-hook missing dependency "loadGame". here the log below.

Also, the game restarts whenever you change the window width/height, this makes the content more responsive but causes an issue when opening the keyboard on android, should i remove it?

i think its good for now, i will merge your PR, thanks for contributing.


Issue in interactive viewer zooming,when zoom a image in a container some portions of image didn't accessible.

I have wrapped the interactive viewer by container.I zoom one particular position in a view port.I dint zoom and scroll the image
`class MyApp extends StatelessWidget {
static const String _title = 'Flutter Code Sample';

@override
Widget build(BuildContext context) {
return MaterialApp(
title: _title,
home: Scaffold(
appBar: AppBar(title: const Text(_title)),
body: Zoom(),
),
);
}
}
class Zoom extends StatefulWidget
{
@override
MyStatelessWidget createState()=> MyStatelessWidget();
}
/// This is the stateless widget that the main application instantiates.
class MyStatelessWidget extends State {

@override
Widget build(BuildContext context) {
return Center(
child:Container(height: 300,
child: InteractiveViewer(
scaleEnabled: true,maxScale: 4.0,
child: Center(
child:Container( child:Image.asset('Assets/Java-001.jpg',fit: BoxFit.cover,alignment: Alignment.center,)),
 @),
)));
}
}`

How can i solve this issue

@allwinvinosh It seems to be working as intended. The issue may be with your Image.asset, can you send your logs?

import 'package:flutter/material.dart';

final Color darkBlue = Color.fromARGB(255, 18, 32, 47);

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  static const String _title = 'Flutter Code Sample';

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: _title,
      home: Scaffold(
        appBar: AppBar(title: const Text(_title)),
        body: Zoom(),
      ),
    );
  }
}

class Zoom extends StatefulWidget {
  @override
  MyStatelessWidget createState() => MyStatelessWidget();
}

/// This is the stateless widget that the main application instantiates.
class MyStatelessWidget extends State<Zoom> {
  @override
  Widget build(BuildContext context) {
    return Center(
        child: Container(
            height: 300,
            color: Colors.greenAccent,
            child: InteractiveViewer(
              scaleEnabled: true,
              maxScale: 4.0,
              child: Center(
                child: Container(
                  color: Colors.red,
                ),
              ),
            )));
  }
}

Your code without the image, I can't reproduce.

@allwinvinosh
Can you elaborate more when you say the portion of image isn't accessible ? Can you provide video / gif of the issue ?
Thanks.

after zoom a image top side image was not accessable,when image excced the container height(300 as mentioned in code)

Hi @allwinvinosh,
I tried to replicate the issue on latest stable (1.20.4) but was able to interact / rotate the image outside of the container height. Not sure if this is what the issue or am I missing anything.
65852.mp4.zip

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


Only switch to node when not dragging it

Normally when you click node in the scene tree, the inspector will switch immediately. I changed it so that it only switches when you release the mouse button, but only if you didn't start dragging the nodes.
Also removed a hack that forcefully switched to script editor when drag started (it was quite brilliant though. I just repurposed its code :v)
Also removed the Alt hack added in #55761
Closes #39539
Fixes #33704
Supersedes #40438
https://user-images.githubusercontent.com/2223172/169104356-e39714e4-659f-4019-b25f-41a240854c81.mp4

Thanks!

Cherry-picked for 3.5.


InstanceTag was renamed to ActiveModelInstanceTag

ActionView::Helpers::InstanceTag was renamed to ActionView::Helpers::ActiveModelInstanceTag
It was renamed in v3.0.9

Coverage decreased (-47.66%) to 48.68% when pulling bad1dd4230c1a068b47c61bf3736e18cffe3fe7f on nathanielram:patch-1 into 55f1697d50143db17a056aca2dc9a13e73d98b52 on johncarney:master.


Admin fails on Firefox

Cannot access admin interface on Firefox : even after the certificate is presented to https://api....., user authentication is rejected by the admin interface. Per @gjuchault, this should only happen on older versions of the browser, but the latest release and even this night's nightly build happen to fail as well.

🦀

Fixed by #250


[Bug]: GraphRAG doesn't get generated - 400 Please use a valid role: user, model.

Is there an existing issue for the same bug?

[X] I have checked the existing issues.

Branch name
main
Commit ID
--
Other environment information
* dev version

KB settings
* chunk method: knowledge graph
* embedding model: Gemini (tried others as well)

Actual behavior
Traceback (most recent call last):
  File "/ragflow/graphrag/graph_extractor.py", line 128, in __call__
    result, token_count = self._process_document(text, prompt_variables)
  File "/ragflow/graphrag/graph_extractor.py", line 177, in _process_document
    if response.find("**ERROR**") >=0: raise Exception(response)
Exception: **ERROR**: 400 Please use a valid role: user, model.
**ERROR**: contents must not be empty```

### Expected behavior

_No response_

### Steps to reproduce

```Markdown
Add a plain text document to the KB and parse.

Additional information
No response

Pull the dev version of docker. It's fixed.

Could you submit an issue for gemini-1.5-flash-lastest?


Link to v2.vuejs.org page in description of repository details

Current
The description of the repository details is linking to the documentation of vue.js 3.
Expected
It should link to the archived v2 documentation instead https://v2.vuejs.org

Nice catch! Fixed with thanks.


Redo Hasura3 config

What is it?
Since we removed all the sqlmesh stuff from Clickhouse, and the Clickhouse configs don't seem to work, let's try recreating this from scratch

https://github.com/opensource-observer/oso/pull/2409

https://github.com/opensource-observer/oso/pull/2413


Complete the ROS observer generator

Related to https://github.com/ipa320/ros-model/pull/100

The observers generator allows only the selection of topics (publisher) to be monitored, it has to be extended for other interfaces: https://github.com/ipa320/ros-model/blob/945594e9370ab7470496cb246498b12323f14018/plugins/de.fraunhofer.ipa.ros.observer.generator/src/de/fraunhofer/ipa/ros/observer/GenerationHandler.java#L182-L203

All the models generated have the same hard coded name "my_observer" https://github.com/ipa320/ros-model/blob/945594e9370ab7470496cb246498b12323f14018/plugins/de.fraunhofer.ipa.ros.observer.generator/src/de/fraunhofer/ipa/ros/observer/GenerationHandler.java#L112 this information should be given by the user

This generator should be extended to completely autogenerate a ROS package (the folder of the package, the package.xml and CMakeLists.txt files) -> the rosgrap_monitor has to be first updated to support this infrastructure

Additionally we could play with parameters to generate observers ready to execute. Once the user select the topic to be subscribed a new dialog menu can show all fields of the message type, where the user can choose one of them. For that field the observer model set threshold values for the OK, WARNING and ERROR diagnostic status using parameters. Having this information the observer can be completely auto-generated.

Once the python code is consolidates we should add code generator Xtext tests -> https://github.com/ipa320/ros-model/blob/master/plugins/de.fraunhofer.ipa.ros.xtext.tests/src/de/fraunhofer/ipa/ros/tests/RosGeneratorTest.xtend

@ipa-hsd feel free to add further ToDos

Currently the observers are pre-defined to publish Dignostic messages. To start adding semantics I suggest the following improvements:

[ ] Define a specific message type for Observers with semantics that capture most use cases for QA/QoS (QualityAttributes or Quality of Service) monitoring. E.f. QAvalue.msg
[ ] For each observer, specify in the model the QA/QoS that the observer monitors, e.g. safety. An option for this could be to specify in the observer.rossystem a specific value for the corresponding field in the messages published. E.g. this would correspond to

QAvalue.msg
============
...
qa_type "safety"
value ...

The observers generator is not working in case the src-gen wasn't previously created by Xtend (and/or the auto build option of eclipse is disable)**.
I could create statically the folder when the generator is called and in case doesn't exists. But I prefer to let Xtend to do that ( it contains a configuration to set the rights and find always the path), I should probably call first Xtend to prepare the creation of the python code.
:exclamation: :exclamation: This issue has a high priority and must to be fixed for the next version.
** @chcorbato @marioney this was the issue I had yesterday during the TelCon, I couldn't sleep well thinking on that :laughing: :laughing:

partly solved https://github.com/ipa320/ros-model/commit/502e834ceac5fa33418e612749162ade3936af23


[Improvement-8192][dolphinschedule_sql] Index query performance improvement

to #8192
BEFORE

AFTER

Codecov Report

Merging #8193 (738fc69) into dev (103ccb0) will increase coverage by 0.02%.
The diff coverage is n/a.

:exclamation: Current head 738fc69 differs from pull request most recent head 775d1fd. Consider uploading reports for the commit 775d1fd to get more accurate results

@@             Coverage Diff              @@
##                dev    #8193      +/-   ##
============================================
+ Coverage     41.29%   41.31%   +0.02%
+ Complexity     3744     3743       -1
============================================
  Files           642      642
  Lines         27401    27401
  Branches       3116     3116
============================================
+ Hits          11315    11321       +6
+ Misses        14998    14988      -10
- Partials       1088     1092       +4

Impacted Files
Coverage Δ

...che/dolphinscheduler/alert/AlertPluginManager.java
90.90% <ø> (ø)

...org/apache/dolphinscheduler/alert/AlertServer.java
59.45% <ø> (ø)

...er/master/dispatch/host/assign/RandomSelector.java
77.77% <0.00%> (-5.56%)
:arrow_down:

...dolphinscheduler/remote/future/ResponseFuture.java
81.96% <0.00%> (-1.64%)
:arrow_down:

...r/plugin/registry/zookeeper/ZookeeperRegistry.java
55.35% <0.00%> (+7.14%)
:arrow_up:

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 103ccb0...775d1fd. Read the comment docs.

@lenboo @JinyLeeChina pls review, tks


[Feature Request] Multi agent Workforce

Required prerequisites

[X] I have searched the Issue Tracker and Discussions that this hasn't already been reported. (+1 or comment there if it has.)
[ ] Consider asking first in a Discussion.

Motivation
Workforce support mulit agent communication, user can add as much agent as they want into this Workforce
Solution
one special agent can be designed to manage the chat and decide which agent be the next speaker
need more study to see if there's better design
Alternatives
No response
Additional context
No response

Workforce(task="Human Task",society=[Agent A, Agent B, RolePlaying …..],manager_agent_config={…},
task_agent_config={…},… workforce = [], description=”xx”)→ ChatResult & Message
Guohao: Workforce(task=””, workforce=[workforce, …])
Agents in Workforce

Manager Agent(optional): track task and can create new agent
Task Agent(optional): Specify, Spilt, Generate, Plan tasks, already supported in CAMEL
Critic Agent(optional): Assists in selecting an option, already supported in CAMEL

Flow in Workforce

Human Task → Task A, Task B, Task C (Message) by Task Agent

Manager Agent initialize the first round task assignment
Task A -> Agent A (model=Model A, tool=[T1, T2],knowledgebase=[KB1, KB2])
Task B -> Agent B (model=Model B, tool=[T1, T3, T5],knowledgebase=[KB2, KB3, KB4])
Task C ....

environment for Agent A = related tasks information + all agents information
environment for Manager Agent = all tasks information + all agents information

If Agent A can solve task A, CAMEL Task A done, back to Manager Agent.
If Agent A can’t solve task A, Agent A will ask Task Agent split Task A, make it into Task A.1, Task A.2…, from Agent A’s environment information, Agent A find another agent can solve Task A.1, send message to that Agent, assign Task A.1, once finished ,back to Manager Agent.
Society pipeline? sequential, hierarchical, star, or other topology..
Task A.1 → Workforce with 3 agents


tidy up commands directory

recently i started an initiative to thin out the commands directory to just the logic to execute commands, and move most of the meat of wrangler's logic into top level directories like upload, deploy and tail. this issue is to track that work.

this is done


[EIC-1134]: lint fix

closes https://github.com/CRJI/EIC/issues/1134
This PR addresses all linting & prettier errors except the "any" typescript/ts-lint errors.
For the any ts-lint errors I've split the work in 2 separate tasks:

https://github.com/CRJI/EIC/issues/1151
https://github.com/CRJI/EIC/issues/1152

from the CI:

Error: Image Optimization using Next.js' default loader is not compatible with next export.

I guess that answers my question :D

I suggest to just disable this rule:
@.***/next/no-img-element": "off"
https://stackoverflow.com/a/68224530
W dniu śr., 11.10.2023 o 17:13 Gabriel Vîjială @.***>
napisał(a):

from the CI:
Error: Image Optimization using Next.js' default loader is not compatible
with next export.
I guess that answers my question :D
—
Reply to this email directly, view it on GitHub
https://github.com/liquidinvestigations/hoover-ui/pull/384#issuecomment-1757921067,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACIBWSO4IOXMLU3T3IAZR7LX62ZSVANCNFSM6AAAAAA54A22IA
.
You are receiving this because your review was requested.Message ID:
@.***>

Thanks for the review, guys!