Slide SlideOnRemove references on create conflict

Fix issue with inconsistent reference position on conflict between position create and remove range.
Update reference position create APIs in preparation for additional SlideOnRemove fixes.
@anthony-murphy @Abe27342 Please review API changes - if they look good I'll add documentation and API tests.

This is a bit different than I was expecting. Is there a reason we are starting with this case, rather than fixing the existing remove logic to slide on ack, rather than slide immediately?

This is a bit different than I was expecting. Is there a reason we are starting with this case, rather than fixing the existing remove logic to slide on ack, rather than slide immediately?

Since this case wasn't handled at all before it was more obviously broken and is the more significant change (adding missing functionality instead of changing existing functionality).
This is the change that requires the larger API changes between IntervalCollection and merge-tree.
As a result, this seemed like the higher risk part that was more likely to uncover issues.

I can add the remove logic, either to this change or as a separate PR. I do think it will be more obvious how all of it fits together once that is done, but I was also trying to keep the change small.

This is a bit different than I was expecting. Is there a reason we are starting with this case, rather than fixing the existing remove logic to slide on ack, rather than slide immediately?

Since this case wasn't handled at all before it was more obviously broken and is the more significant change (adding missing functionality instead of changing existing functionality).
This is the change that requires the larger API changes between IntervalCollection and merge-tree.
As a result, this seemed like the higher risk part that was more likely to uncover issues.

I can add the remove logic, either to this change or as a separate PR. I do think it will be more obvious how all of it fits together once that is done, but I was also trying to keep the change small.

it can be useful to have a bigger combined PR as a draft that shows how it all fits together, then break it down piecemeal to go it. I think there are enough moving parts here that being able to walk though it holistically would be useful

This is a bit different than I was expecting. Is there a reason we are starting with this case, rather than fixing the existing remove logic to slide on ack, rather than slide immediately?

Since this case wasn't handled at all before it was more obviously broken and is the more significant change (adding missing functionality instead of changing existing functionality).
This is the change that requires the larger API changes between IntervalCollection and merge-tree.
As a result, this seemed like the higher risk part that was more likely to uncover issues.

I can add the remove logic, either to this change or as a separate PR. I do think it will be more obvious how all of it fits together once that is done, but I was also trying to keep the change small.

it can be useful to have a bigger combined PR as a draft that shows how it all fits together, then break it down piecemeal to go it. I think there are enough moving parts here that being able to walk though it holistically would be useful

Agree. I'll mark this draft for now. Was useful to get the feedback - thank you.

Looked at tests, not too many comments there. Obviously there are still some TODOs, but I'm ok with addressing them in follow-ups.

What's the rationale for the explicit update of reference offsets on segment removal? I don't think we can come up with an eventually consistent scheme while doing that. If it's just to make sure the observed offset matches the segment's position in the SharedString, there is already logic which handles returning 0 in getOffset() when the segment is removed.
Here's an explicit fuzz test case (minimized, so pretty readable) which demonstrates why this strategy loses information:
[
    {
        "type": "addText",
        "index": 0,
        "content": "ABCDEFGHIJ",
        "stringId": "B"
    },
    {
        "type": "synchronize"
    },
    {
        "type": "addInterval",
        "start": 6,
        "end": 8,
        "collectionName": "comments",
        "stringId": "B",
        "id": "7a5d683d-4b3e-44b3-8f91-e8f9d664de5a"
    },
    {
        "type": "addText",
        "index": 7,
        "content": "XYZ",
        "stringId": "B"
    },
    {
        "type": "removeRange",
        "start": 2,
        "end": 10,
        "stringId": "A"
    },
    {
        "type": "synchronize"
    }
]

A needs to be able to slide the references added by B to the start and end of XYZ, respectively, but it won't know how to do this if it blindly sets both offsets to 0 upon acking them.
(side note: I do think we will probably need to either expose a method which gets the offset ignoring segment removals, or push reference comparison + "split" knowledge somehow to the reference itself. There are a couple maintenance things which we do need to make sure we preserve that information)


KeyError due to missing element in multiplier_dict

Hello,
I am developing a script to read neware nda files into my database.
I have encountered the following error due a missing key/value in the multiplier dict.
Stacktrace:
2023-12-06T15:27:08.923267 [INFO] NewareNDA.read_nda: NDA version: 29
2023-12-06T15:27:08.923267 [INFO] NewareNDA.read_nda: Server: BTSServer(R3)-<IP_ADDRESS>1 (2018.07.30)
2023-12-06T15:27:08.923267 [INFO] NewareNDA.read_nda: Client: BTS Client <IP_ADDRESS>4(2020.12.22)
Traceback (most recent call last):
  File "C:\Users\User\projects\project\src\prototype\main.py", line 51, in <module>
    main()
  File "C:\Users\User\projects\project\src\prototype\main.py", line 43, in main
    df = NewareNDA.read(temp_file)
         ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\User\projects\project\venv\Lib\site-packages\NewareNDA\NewareNDA.py", line 30, in read
    return read_nda(file, software_cycle_number)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\User\projects\project\venv\Lib\site-packages\NewareNDA\NewareNDA.py", line 94, in read_nda
    output.append(_bytes_to_list(bytes))
                  ^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\User\projects\project\venv\Lib\site-packages\NewareNDA\NewareNDA.py", line 152, in _bytes_to_list
    multiplier = multiplier_dict[Range]
                 ~~~~~~~~~~~~~~~^^^^^^^
KeyError: 1

Possible Solutions:

Add element to for key 1 with value 1e-4
This appears to be the correct multiplier for my example based off my own testing.
Unfortunately, I am not able to share the data files as it is confidential.

Set multiplier via keyword argument
df = NewareNDA.read('my_file.nda', multiplier={1:1e-4})

Please advise on next steps to move this forward.
Regards,
Steven Gallagher

Hi,
I also found a missing entry in the multiplier_dict that is used by the CT-4008Q series:
-100000000: 10
That is the only missing range I found when importing multiple ndax files from BTS server 8.0. That entry could be added as well if you are already onto it.
Thanks for the great work on this project, helps a lot!
BR
Stefan

I just added these ranges to the development branch. Thanks for your help! You can install the development branch with this pip command:
pip install git+https://github.com/Solid-Energy-Systems/NewareNDA.git@development

@d-cogswell Great, thanks!

@d-cogswell thankyou for a fast response!
I have tested the development branch and it looks like it solves my problem.


Datasets: Write tests for Dataset.load() method

Codecov Report
Base: 60.47% // Head: 66.90% // Increases project coverage by +6.43% :tada:

Coverage data is based on head (10eb931) compared to base (8fe9daa).
Patch coverage: 92.85% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #201      +/-   ##
==========================================
+ Coverage   60.47%   66.90%   +6.43%
==========================================
  Files          21       21
  Lines         802      834      +32
  Branches      188      197       +9
==========================================
+ Hits          485      558      +73
+ Misses        306      251      -55
- Partials       11       25      +14

Impacted Files
Coverage Δ

src/pymovements/datasets/gazebase.py
94.44% <83.33%> (-5.56%)
:arrow_down:

src/pymovements/datasets/judo1000.py
94.44% <85.71%> (-5.56%)
:arrow_down:

src/pymovements/datasets/dataset.py
45.16% <93.33%> (+20.04%)
:arrow_up:

src/pymovements/datasets/public_dataset.py
59.61% <100.00%> (+12.11%)
:arrow_up:

src/pymovements/utils/paths.py
54.54% <0.00%> (+31.81%)
:arrow_up:

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Inventory grid activates on just a click instead of only a drag

Describe the bug
Just doing a single click without dragging enables the grid.
To Reproduce
Steps to reproduce the behavior:

Turn on the inventory grid plugin
Click on any item in the inventory without dragging

Expected behavior
Wait until the mouse drags before showing the highlight.
Environment (please complete the following information):

OS: Arch Linux
RuneLite version: 1.5.26-SNAPSHOT

I'm partially making this ticket because I'm not sure if it should be fixed in the plugin via a mouse listener, or in internal (even though just holding down an item initiates the drag effect in-game).

If it should be fixed within the plugin, here is a fix based on the original PR.

Seems like it got mostly fixed in internal. (Tapping on a touchpad seems to still activate it for me, so the delay should probably be larger, but I'll just let this issue play out at this point.)


Generate the correct link for the content in the readme

Search terms

readme
Expected Behavior

During the readme rendering process, generate correct HTML for the referenced local markdown file link
Actual Behavior

Link error in readme
Steps to reproduce the bug
E.g. README.md
[简体中文](./README.md) | English
Environment

Typedoc version: 0.20.34
TypeScript version: 4.0.3
Node.js version: 14.4.0
OS: macOS 11.2.3

If it is confirmed that this is a bug, I will fix this problem through PR later

I'm somewhat worried about trying to rewrite links - if the readme linked to ./other.file.md and we rewrote that link, it would lead to users expecting that file to be included in the generated site, which it currently won't be (and including arbitrary files is definitely something I don't want to handle! The folder to include, yes, eventually, but arbitrary files from anywhere? Nope)
I think my advice for now would be to create a second readme specifically for your docs, and point TypeDoc to it with the --readme option

I'm somewhat worried about trying to rewrite links - if the readme linked to ./other.file.md and we rewrote that link, it would lead to users expecting that file to be included in the generated site, which it currently won't be (and including arbitrary files is definitely something I don't want to handle! The folder to include, yes, eventually, but arbitrary files from anywhere? Nope)
I think my advice for now would be to create a second readme specifically for your docs, and point TypeDoc to it with the --readme option

@Gerrit0 First of all, thank you for your reply. I understand your worry. It is necessary to ensure that the referenced documents are in line with the expectations.  To solve the problem, I think of a modification method:
1.The readme option supports the form of array types to clearly define the file you need to reference (or add a new options). It supports passing in a group of parameters, and its configuration items are similar:
{
    "readme": [
      "./README-en-GB.md",
      "./README.md"
    ]
}

2.If any path specified in the readme parameters is found during the rendering of the readme file, it will be modified to html  hyperlink instead of the current md file link
At the same time, I'm not sure whether this change should be put into the framework. At present, this ability can be realized in the form of plug-ins provided by typedoc.

I've reviewed this a couple of times, and I don't think this is something I want to live in TypeDoc proper, it's a bit too magical. This might change once TypeDoc has support for guides, but for now it doesn't really fit with the architecture well.


Application deletion not deleting services associated with it on windows 3.11 cluster

/kind bug

What versions of software are you using?

Running make test-cmd-service on windows 3.11 cluster. Also ran the steps manually.
Manual steps:
> odo project create appdlt
 V  Project 'appdlt' is ready for use
 V  New project created and now using project: appdlt

> odo service create --app app -w dh-postgresql-apb --project appdlt --plan dev -p postgresql_user=luke -p postgresql_password=secret -p postgresql_database=my_data -p postgresql_version=9.6
Deploying service dh-postgresql-apb of type: dh-postgresql-apb
 V  Waiting for service to come up [3m]
 V  Service 'dh-postgresql-apb' is ready for use
Optionally, link dh-postgresql-apb to your component by running: 'odo link <component-name>'

> oc get serviceinstance -o name -n appdlt
serviceinstance.servicecatalog.k8s.io/dh-postgresql-apb

> odo app delete app --project appdlt -f
This application has following service(s) that will be deleted
service named dh-postgresql-apb of type dh-postgresql-apb
Deleted application: app from project: appdlt

> oc get serviceinstances -n appdlt
NAME                CLASS                                   PLAN      STATUS           AGE
dh-postgresql-apb   ClusterServiceClass/dh-postgresql-apb   dev       Deprovisioning   5m

make test-cmd-service
> make test-cmd-service
Running Suite: Servicecatalog Suite
===================================
Random Seed:<PHONE_NUMBER>
Will run 1 of 23 specs

SSSSSSSSSSSSSSSSSSSSSSCreated dir: C:\Users\Admin\AppData\Local\Temp\369834843
Creating a new project: rhyxmmyxpn
Running odo.exe with args [odo project create rhyxmmyxpn -w -v4]
[odo]  -  Waiting for project to come up  ...
[odo] I0706 10:18:20.399182   11920 occlient.go:543] Status of creation of project rhyxmmyxpn is Active
[odo] I0706 10:18:20.473971   11920 occlient.go:548] Project rhyxmmyxpn now exists
[odo] I0706 10:18:20.486589   11920 occlient.go:583] Status of creation of service account &ServiceAccount{ObjectMeta:{default  rhyxmmyxpn /api/v1/namespaces/rhyxmmyxpn/serviceaccounts/default 014e115c-bf72-11ea-a319-00155d69a004 1738662 0 2020-07-06 10:18:15 +0000 GMT <nil> <nil> map[] map[] [] []  []},Secrets:[]ObjectReference{ObjectReference{Kind:,Namespace:,Name:default-token-7tdws,UID:,APIVersion:,ResourceVersion:,FieldPath:,},ObjectReference{Kind:,Namespace:,Name:default-dockercfg-lgqf9,UID:,APIVersion:,ResourceVersion:,FieldPath:,},},ImagePullSecrets:[]LocalObjectReference{LocalObjectReference{Name:default-dockercfg-lgqf9,},},AutomountServiceAccountToken:nil,} is ready
 V  Waiting for project to come up [6s]
[odo]  V  Project 'rhyxmmyxpn' is ready for use
[odo]  V  New project created and now using project: rhyxmmyxpn
[odo] I0706 10:18:20.517207   11920 odo.go:72] Could not get the latest release information in time. Never mind, exiting gracefully :)
Created dir: C:\Users\Admin\AppData\Local\Temp\693413118
Created dir: C:\Users\Admin\AppData\Local\Temp\116853829
Current working dir: C:\Users\Admin\go\src\github.com\openshift\odo\tests\integration\servicecatalog
Running odo.exe with args [odo service create --app omgdte -w dh-postgresql-apb --project rhyxmmyxpn --plan dev -p postgresql_user=luke -p postgresql_password=secret -p postgresql_database=my_data -p postgresql_version=9.6]
[odo] Deploying service dh-postgresql-apb of type: dh-postgresql-apb
 V  Waiting for service to come up [3m]
[odo]  V  Service 'dh-postgresql-apb' is ready for use
[odo] Optionally, link dh-postgresql-apb to your component by running: 'odo link <component-name>'
Running oc.exe with args [oc get serviceinstance -o name -n rhyxmmyxpn]
[oc] serviceinstance.servicecatalog.k8s.io/dh-postgresql-apb
Running odo.exe with args [odo app delete omgdte --project rhyxmmyxpn -f]
[odo] This application has following service(s) that will be deleted
[odo] service named dh-postgresql-apb of type dh-postgresql-apb
[odo] Deleted application: omgdte from project: rhyxmmyxpn
Running oc.exe with args [oc get serviceinstances -n rhyxmmyxpn]
[oc] NAME                CLASS                                   PLAN      STATUS                                    AGE
[oc] dh-postgresql-apb   ClusterServiceClass/dh-postgresql-apb   dev       DeprovisionBlockedByExistingCredentials   3m
Running oc.exe with args [oc get serviceinstances -n rhyxmmyxpn]
[oc] NAME                CLASS                                   PLAN      STATUS                                    AGE
[oc] dh-postgresql-apb   ClusterServiceClass/dh-postgresql-apb   dev       DeprovisionBlockedByExistingCredentials   3m
Running oc.exe with args [oc get serviceinstances -n rhyxmmyxpn]
[oc] NAME                CLASS                                   PLAN      STATUS                       AGE
[oc] dh-postgresql-apb   ClusterServiceClass/dh-postgresql-apb   dev       DeprovisionRequestInFlight   3m
Running oc.exe with args [oc get serviceinstances -n rhyxmmyxpn]
[oc] NAME                CLASS                                   PLAN      STATUS                       AGE
[oc] dh-postgresql-apb   ClusterServiceClass/dh-postgresql-apb   dev       DeprovisionRequestInFlight   3m
Running oc.exe with args [oc get serviceinstances -n rhyxmmyxpn]
[oc] NAME                CLASS                                   PLAN      STATUS           AGE
[oc] dh-postgresql-apb   ClusterServiceClass/dh-postgresql-apb   dev       Deprovisioning   3m
Running oc.exe with args [oc get serviceinstances -n rhyxmmyxpn]
[oc] NAME                CLASS                                   PLAN      STATUS           AGE
[oc] dh-postgresql-apb   ClusterServiceClass/dh-postgresql-apb   dev       Deprovisioning   3m
Running oc.exe with args [oc get serviceinstances -n rhyxmmyxpn]
[oc] NAME                CLASS                                   PLAN      STATUS           AGE
[oc] dh-postgresql-apb   ClusterServiceClass/dh-postgresql-apb   dev       Deprovisioning   3m
Running oc.exe with args [oc get serviceinstances -n rhyxmmyxpn]
[oc] NAME                CLASS                                   PLAN      STATUS           AGE
[...]

------------------------------
+ Failure [252.584 seconds]
odo service command tests
C:/Users/Admin/go/src/github.com/openshift/odo/tests/integration/servicecatalog/cmd_service_test.go:15
  When the application is deleted
    should delete the service(s) in the application as well [It]
    C:/Users/Admin/go/src/github.com/openshift/odo/tests/integration/servicecatalog/cmd_service_test.go:434

    Timeout out after 1 minutes

    C:/Users/Admin/go/src/github.com/openshift/odo/tests/helper/helper_generic.go:41
------------------------------

Summarizing 1 Failure:
C:/Users/Admin/go/src/github.com/openshift/odo/tests/helper/helper_generic.go:41

Ran 1 of 23 Specs in 260.833 seconds
--- FAIL: TestServicecatalog (261.50s)
FAIL

Actual behavior
odo app delete app --project appdlt -f does not delete the services associated with the application.
Expected behavior
It should delete the services as per the behavior.
Any logs, error output, etc?
Ran make test-cmd-service on windows 3.11 cluster. Also verified step by step.

[oc] NAME                CLASS                                   PLAN      STATUS           AGE
[oc] dh-postgresql-apb   ClusterServiceClass/dh-postgresql-apb   dev       Deprovisioning   3m

This output suggests that odo correctly deleted the service, but is taking a long time for the cluster to completely deprovision it.
This seems to be more a cluster issue than odo issue.
Can you check output of oc get ClusterServiceClass/dh-postgresql-apb -o yaml? I suspect that you will see errors in a status indicating why the service is not getting deleted

Hits the same failure on travis CI.
Created dir: /tmp/065313953
Creating a new project: psanqfkaap
Running odo with args [odo project create psanqfkaap -w -v4]
[odo] I0820 20:01:38.373152   16537 util.go:734] HTTPGetRequest: https://raw.githubusercontent.com/openshift/odo/master/build/VERSION
[odo] I0820 20:01:38.373231   16537 util.go:755] Response will be cached in /tmp/odohttpcache for 1h0m0s
[odo] I0820 20:01:38.373416   16537 util.go:768] Cached response used.
[...]
Current working dir: /home/travis/gopath/src/github.com/openshift/odo/tests/integration/servicecatalog
Setting KUBECONFIG=/tmp/065313953/config
Created dir: /tmp/883359116
Created dir: /tmp/256647035
Running odo with args [odo service create --app vfhiah -w dh-postgresql-apb --project psanqfkaap --plan dev -p postgresql_user=luke -p postgresql_password=secret -p postgresql_database=my_data -p postgresql_version=9.6]
[odo] I0820 20:01:39.800358   16750 preference.go:209] The path for preference file is /tmp/065313953/config.yaml
[odo] I0820 20:01:39.817819   16750 preference.go:209] The path for preference file is /tmp/065313953/config.yaml
[odo] I0820 20:01:39.817850   16750 preference.go:209] The path for preference file is /tmp/065313953/config.yaml
[odo] I0820 20:01:39.817857   16750 preference.go:209] The path for preference file is /tmp/065313953/config.yaml
[...]
[odo]  ✓  Service 'dh-postgresql-apb' is ready for use
[odo] Optionally, link dh-postgresql-apb to your component by running: 'odo link <component-name>'
[odo] I0820 20:02:18.187779   16750 occlient.go:1976] Secret dh-postgresql-apb now exists
[odo] I0820 20:02:18.187892   16750 odo.go:72] Could not get the latest release information in time. Never mind, exiting gracefully :)
Running oc with args [oc get serviceinstance -o name -n psanqfkaap]
[oc] serviceinstance.servicecatalog.k8s.io/dh-postgresql-apb
Running odo with args [odo app delete vfhiah --project psanqfkaap -f]
[odo] I0820 20:02:20.383034   20698 application.go:75] Deleting application vfhiah
[odo] This application has following service(s) that will be deleted
[odo] service named dh-postgresql-apb of type dh-postgresql-apb
[odo] I0820 20:02:20.402484   20698 occlient.go:2131] Deleting Service Instance
[odo] I0820 20:02:20.402505   20698 occlient.go:2135] Selectors used for deletion: app.kubernetes.io/part-of=vfhiah,app.kubernetes.io/instance=dh-postgresql-apb
[odo] I0820 20:02:20.861349   20698 occlient.go:2049] Selectors used for deletion: app.kubernetes.io/part-of=vfhiah
[odo] I0820 20:02:20.861370   20698 occlient.go:2059] Deleting DeploymentConfigs
[odo] I0820 20:02:20.932991   20698 occlient.go:2065] Deleting BuildConfigs
[odo] I0820 20:02:21.096661   20698 occlient.go:2071] Deleting ImageStreams
[odo] Deleted application: vfhiah from project: psanqfkaap
[odo] I0820 20:02:21.123694   20698 odo.go:72] Could not get the latest release information in time. Never mind, exiting gracefully :)
Running oc with args [oc get serviceinstances -n psanqfkaap]
[oc] NAME                CLASS                                   PLAN      STATUS           AGE
[oc] dh-postgresql-apb   ClusterServiceClass/dh-postgresql-apb   dev       Deprovisioning   44s
Running oc with args [oc get serviceinstances -n psanqfkaap]
[oc] NAME                CLASS                                   PLAN      STATUS           AGE
[oc] dh-postgresql-apb   ClusterServiceClass/dh-postgresql-apb   dev       Deprovisioning   45s
Running oc with args [oc get serviceinstances -n psanqfkaap]
[...]
Running oc with args [oc get serviceinstances -n psanqfkaap]
[oc] NAME                CLASS                                   PLAN      STATUS           AGE
[oc] dh-postgresql-apb   ClusterServiceClass/dh-postgresql-apb   dev       Deprovisioning   1m
Deleting project: psanqfkaap
Running odo with args [odo project delete psanqfkaap -f]
[odo] I0820 20:03:21.555075   24155 util.go:768] Cached response used.
[odo]  ⚠  Warning! Projects are deleted from the cluster asynchronously. Odo does its best to delete the project. Due to multi-tenant clusters, the project may still exist on a different node.
[odo] I0820 20:03:21.779798   24155 odo.go:72] Could not get the latest release information in time. Never mind, exiting gracefully :)
[odo]  ✓  Deleted project : psanqfkaap
Setting current dir to: /home/travis/gopath/src/github.com/openshift/odo/tests/integration/servicecatalog
Deleting dir: /tmp/065313953
Deleting dir: /tmp/883359116
Deleting dir: /tmp/256647035
• Failure [103.620 seconds]
odo service command tests
/home/travis/gopath/src/github.com/openshift/odo/tests/integration/servicecatalog/cmd_service_test.go:13
  When the application is deleted
  /home/travis/gopath/src/github.com/openshift/odo/tests/integration/servicecatalog/cmd_service_test.go:358
    should delete the service(s) in the application as well [It]
    /home/travis/gopath/src/github.com/openshift/odo/tests/integration/servicecatalog/cmd_service_test.go:362
    Timeout after 1 minutes


Karolina's Page Design Coded

Created homepage.html
Added Background image and logo
Converted /login and /register into Dialog Window

would be greatly appreciated if you move your css to Sass (scss)

This is great first role, we can continue building upon this. Meanwhile, we can add a coming soon page where external users can subscribe with their emails.

@saviaga your fork is from an unknown repository, looks like the changes are not reflected in your repo?

closing - changes from unknown repo and it doesn't work.


Default value in a $prompt

I have a system that I want to allow a user to rename a title. I $prompt them for a rename, but I'd love to prefill the title so if they're fixing a typo for a long title, they don't have to retype the whole title again. Let me know your thoughts.

Good idea.

Awesome! Thank you so much for this patch!


Fix document section reference

Remove redundant ' %} in note:
Skip to the How Pulsar uses ZooKeeper and BookKeeper section below for a more schematic explanation of the role of these two systems in Pulsar. ' %}

And fix Configuration Store broken reference, which refer to #deploying-configuration-store but we only have a section Deploying the configuration store {#configuration-store}, which should be Deploying configuration store like other sections.

run cpp tests
run integration tests


Added Rate Us

Hello Project Admin,
I Added a Rate Us Option - Solved #266
Please Kindly go through it
https://github.com/VanshKing30/FoodiesWeb/assets/161804629/61ac214e-0503-4c62-859e-b4ba2b131198

@aayushchugh i will change it Sir, No Problem

@sailaja-adapa Thanks and best of luck

@aayushchugh  Thank you sir


Fixed filesystem choice

Filesystem should be exFAT based on https://www.youtube.com/watch?v=qPAOMczcuZw at 6:40
Or am I missing something?

My bad, I misunderstood some steps. I'm sorry for that


FYPO:0000343 change

recast FYPO:0000343 as extracellular sulfide level
remove link to FYPO:0001413
(it would be more strictly correct to make fypo:343 obsolete and add a new term, but no annotations will be adversely affected so I'll take the easy way out)

done
edit file: 04527f41beb7bbeedef0820b9ca0477eb2ffd133
release: 59f83f7a8d4c1a6a1df4741a0527e832f6eb1a31


install kerberos error

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
[INFO] 2023-10-08 14:49:22 com.datasophon.api.master.MasterServiceActor:[75] - Krb5Kdc service role size is 2
[INFO] 2023-10-08 14:49:22 com.datasophon.api.master.MasterServiceActor:[107] - enable ranger plugin is false
[INFO] 2023-10-08 14:49:22 com.datasophon.api.master.MasterServiceActor:[113] - start to install Krb5Kdc in host hadoop1.bigdata.com
[INFO] 2023-10-08 14:49:35 com.datasophon.api.master.MasterServiceActor:[136] - Krb5Kdc install failed in hadoop1.bigdata.com
[INFO] 2023-10-08 14:49:35 com.datasophon.api.master.ActorUtils:[122] - find actor serviceExecuteResultActor
[INFO] 2023-10-08 14:49:35 com.datasophon.api.master.ActorUtils:[122] - find actor submitTaskNodeActor
[INFO] 2023-10-08 14:49:35 com.datasophon.api.master.ServiceExecuteResultActor:[66] - KERBEROS master roles failed , cancel all next node by commandId 98544707c05a4201a1395bdff6c2957a
[INFO] 2023-10-08 14:49:35 com.datasophon.api.utils.ProcessUtils:[196] - command id is 98544707c05a4201a1395bdff6c2957a
[INFO] 2023-10-08 14:49:35 com.datasophon.api.master.ActorUtils:[122] - find actor commandActor
[INFO] 2023-10-08 14:49:35 com.datasophon.api.utils.ProcessUtils:[267] - 安装 Krb5Kdc in hadoop1.bigdata.com failed
[INFO] 2023-10-08 14:49:35 com.datasophon.api.master.ActorUtils:[122] - find actor commandActor
[INFO] 2023-10-08 14:49:35 com.datasophon.api.master.MasterServiceActor:[75] - KAdmin service role size is 2
[INFO] 2023-10-08 14:49:35 com.datasophon.api.utils.ProcessUtils:[205] - 安装 Krb5Kdc host command  set to cancel
[INFO] 2023-10-08 14:49:35 com.datasophon.api.master.MasterServiceActor:[107] - enable ranger plugin is false
[INFO] 2023-10-08 14:49:35 com.datasophon.api.utils.ProcessUtils:[205] - 安装 Krb5Client host command  set to cancel
[INFO] 2023-10-08 14:49:35 com.datasophon.api.master.MasterServiceActor:[113] - start to install KAdmin in host hadoop1.bigdata.com
[INFO] 2023-10-08 14:49:35 com.datasophon.api.utils.ProcessUtils:[205] - 安装 Krb5Client host command  set to cancel
[INFO] 2023-10-08 14:49:35 com.datasophon.api.utils.ProcessUtils:[205] - 安装 KAdmin host command  set to cancel
[INFO] 2023-10-08 14:49:35 com.datasophon.api.utils.ProcessUtils:[205] - 安装 Krb5Client host command  set to cancel
What you expected to happen

How to reproduce
1.add kerberos install
2.

3.

4.install KAdmin

Anything else
No response
Version
v1.1.2
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

hi sir:
       i know .The realm is HADOOP.COM,never modified,
       However，it depends on other issues
       Thank you for responding to my question
------------------ 原始邮件 ------------------
发件人:                                                                                "datavane/datasophon"                                                                                @.>;
发送时间: 2023年10月8日(星期天) 晚上8:09
@.>;
<EMAIL_ADDRESS>主题: Re: [datavane/datasophon] install kerberos error (Issue #427)
The realm must be HADOOP.COM
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Use official recommended host name settings

Installing kerberos requires using yum install, please ensure that the yum source is functioning properly


Operator Group [It] insufficient permissions resolve via RBAC

 Operator Group [It] insufficient permissions resolve via RBAC
/home/runner/work/operator-lifecycle-manager/operator-lifecycle-manager/test/e2e/operator_groups_e2e_test.go:1612

  Timeline >>
  created the operator-group-e2e-smz2l testing namespace
  14:29:40.0142: Creating CRD
  14:29:40.0197: Creating operator group
  14:29:40.0248: Creating CSV
  14:29:40.0395: wait for CSV to fail
  14:29:40.2316:  ():
  waited 10.217669861s for CSV operator-group-e2e-smz2l-vv7fx/another-csv-xtrdv: to be in phases [Failed], in phase  ():
  14:29:40.4316:  ():
  14:29:40.6312:  ():
  14:29:40.8311:  ():
  14:29:41.0321:  ():
  14:29:41.2321: Pending (RequirementsUnknown): requirements not yet checked
  waited 1.000538645s for CSV operator-group-e2e-smz2l-vv7fx/another-csv-xtrdv: to be in phases [Failed], in phase Pending (RequirementsUnknown): requirements not yet checked
  14:29:41.4312: InstallReady (AllRequirementsMet): all requirements found, attempting install
  waited 199.044471ms for CSV operator-group-e2e-smz2l-vv7fx/another-csv-xtrdv: to be in phases [Failed], in phase InstallReady (AllRequirementsMet): all requirements found, attempting install
  14:29:41.6321: Failed (InstallComponentFailedNoRetry): install strategy failed: install strategy failed: deployments.apps "operator-deployment8b7mg" is forbidden: User "system:serviceaccount:operator-group-e2e-smz2l-vv7fx:nginx-sak98h5" cannot get resource "deployments" in API group "apps" in the namespace "operator-group-e2e-smz2l-vv7fx"
  waited 200.980766ms for CSV operator-group-e2e-smz2l-vv7fx/another-csv-xtrdv: to be in phases [Failed], in phase Failed (InstallComponentFailedNoRetry): install strategy failed: install strategy failed: deployments.apps "operator-deployment8b7mg" is forbidden: User "system:serviceaccount:operator-group-e2e-smz2l-vv7fx:nginx-sak98h5" cannot get resource "deployments" in API group "apps" in the namespace "operator-group-e2e-smz2l-vv7fx"
  14:29:41.6377: wait for CSV to succeeed
  14:29:41.8324: Failed (InstallComponentFailedNoRetry): install strategy failed: install strategy failed: deployments.apps "operator-deployment8b7mg" is forbidden: User "system:serviceaccount:operator-group-e2e-smz2l-vv7fx:nginx-sak98h5" cannot get resource "deployments" in API group "apps" in the namespace "operator-group-e2e-smz2l-vv7fx"
  waited 7.277033202s for CSV operator-group-e2e-smz2l-vv7fx/another-csv-xtrdv: to be in phases [Succeeded], in phase Failed (InstallComponentFailedNoRetry): install strategy failed: install strategy failed: deployments.apps "operator-deployment8b7mg" is forbidden: User "system:serviceaccount:operator-group-e2e-smz2l-vv7fx:nginx-sak98h5" cannot get resource "deployments" in API group "apps" in the namespace "operator-group-e2e-smz2l-vv7fx"
  14:29:42.0323: Failed (InstallComponentFailedNoRetry): install strategy failed: install strategy failed: deployments.apps "operator-deployment8b7mg" is forbidden: User "system:serviceaccount:operator-group-e2e-smz2l-vv7fx:nginx-sak98h5" cannot get resource "deployments" in API group "apps" in the namespace "operator-group-e2e-smz2l-vv7fx"

< ... repeated N times ... >

   14:34:42.0321: Failed (InstallComponentFailedNoRetry): install strategy failed: install strategy failed: deployments.apps "operator-deployment8b7mg" is forbidden: User "system:serviceaccount:operator-group-e2e-smz2l-vv7fx:nginx-sak98h5" cannot get resource "deployments" in API group "apps" in the namespace "operator-group-e2e-smz2l-vv7fx"
  [FAILED] in [It] - /home/runner/work/operator-lifecycle-manager/operator-lifecycle-manager/test/e2e/operator_groups_e2e_test.go:1746 @ 11/14/23 14:34:42.147
  cleaning up ephemeral test resources...
  deleting test subscriptions...
  deleting test installplans...
  deleting test catalogsources...
  deleting test crds...
  deleting test csvs...
  test resources deleted
  << Timeline

  [FAILED]
  	Error Trace:	/home/runner/work/operator-lifecycle-manager/operator-lifecycle-manager/test/e2e/operator_groups_e2e_test.go:1746
  	            				/home/runner/work/operator-lifecycle-manager/operator-lifecycle-manager/vendor/github.com/onsi/ginkgo/v2/internal/node.go:463
  	            				/home/runner/work/operator-lifecycle-manager/operator-lifecycle-manager/vendor/github.com/onsi/ginkgo/v2/internal/suite.go:863
  	            				/opt/hostedtoolcache/go/1.20.10/x64/src/runtime/asm_amd64.s:1598
  	Error:      	Received unexpected error:
  	            	timed out waiting for the condition
  	Test:       	Operator Group insufficient permissions resolve via RBAC

  In [It] at: /home/runner/work/operator-lifecycle-manager/operator-lifecycle-manager/test/e2e/operator_groups_e2e_test.go:1746 @ 11/14/23 14:34:42.147

  Full Stack Trace
    github.com/stretchr/testify/require.NoError({0x7f98740e1188, 0xc000df2ab0}, {0x4090400, 0xc0001b8150}, {0x0, 0x0, 0x0})
    	/home/runner/work/operator-lifecycle-manager/operator-lifecycle-manager/vendor/github.com/stretchr/testify/require/require.go:1357 +0x96
    github.com/operator-framework/operator-lifecycle-manager/test/e2e.glob..func20.9()
    	/home/runner/work/operator-lifecycle-manager/operator-lifecycle-manager/test/e2e/operator_groups_e2e_test.go:1746 +0x15b5

Duplicate of #3091


soc/arm/st_stm32/stm32l4:Add power management and shutdown mode

Add support for power management and the shutdown mode for stm32l4.
This now allows boards that have any kind of low power mode to send
it to shutdown mode and wake it up from wakeup source.
Currently wake up source it set as PIN1.
Tested on a STM32L462VE for shutdown and wakeup.
Signed-off-by: Khalid Mustafa<EMAIL_ADDRESS>

@khalidjmi1200, About the wake up pin configuration, please note that adding the sample demonstrating use of wake up pin is not blocking. The only blocking point is that code should not be added in power.c (as long as we don't have proper dts support for it).
The sample is just a nice bonus.


Switch label spacing slightly off

Description

This is the same problem as on the Radio button component label bug. On the the Label for switches on mobile it seems to be two pixels above where the designed label is. On desktop the text is 1 pixel above where it should be.
Steps to reproduce

Go to http://opensource.corp.adobe.com/spectrum-css/
Click on ...
Observe ...

Expected behavior

Screenshots

Desktop

Mobile

Environment

Spectrum CSS version:
Browser(s) and OS(s):

https://reactspectrum.blob.core.windows.net/reactspectrum/20e5131cfc8f65620868f16dc32defaedd215aaf/index.html?path=/story/switch--default
Additional context

@JacobDFrank can you please verify this against http://spectrum-css-archives.s3-website.us-east-2.amazonaws.com/2.15.1/docs/toggle.html ?

the labels are one pixel lower than they should be on desktop and all the labels are two pixels lower


Warnings during full development installation

When doing a full development installation of ESMValTool, the pip install step for ESMValCore raises the following warnings:
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
prospector 1.10.3 requires flake8<6.0.0, but you have flake8 7.0.0 which is incompatible.
prospector 1.10.3 requires pyflakes<3,>=2.2.0, but you have pyflakes 3.2.0 which is incompatible.

See also https://github.com/ESMValGroup/ESMValTool/actions/runs/9342587884/job/25710921454.
@ESMValGroup/technical-lead-development-team
P.S. The installation is "successful", it looks like the ERROR is just treated as a warning.

Let's first see if all other dependencies become available for Python 3.12 and then we can see about dropping prospector. In the long run I would like to switch to ruff, but it would be rather nice if Codacy supported it so we don't lose the webpage with linting results.

methinks this is good to close, Manu? :beer:

I just did a fresh installation and the warnings still appear. I guess they will only disappear when we drop prospector?


Refactor oauth2 error page handling logic

Currently, there error page handling logic in the OAuth2 component is overly complex. It should be simplified.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Appcasts without pubDate

Pinging @caskroom/maintainers (also @Amorymeltzer).
This PR changes the :sha256 of every appcast in the main repo to the wrong one. That seems like a strange thing to do, especially since I just spent days making sure every single one was correct, so naturally there’s an explanation.
There are currently a few (around ten) appcasts that are impossible to use. Reason being they have a <pubDate> tag for the whole feed, that gets rebuilt periodically (sometimes with every request) regardless of it the app was updated or not. That tag seems to be the regular culprit, so instead of removing appcast from those few casks, I instead did curl "{{appcast_url}}" | sed '/<pubDate>/d' | shasum --algorithm 256 on every cask, to get new, reliable appcast shasums.
This might seem overkill just to fix a handful of casks (think of the user confusion!) but in reality it could win us over 150 new appcasts.
So that’s the decision to make. Either we keep :sha256 representing an accurate value, or we get flexibility in a lot more casks. As this PR shows, I’m more inclined for the second option. As long as we’re clear on the documentation how it is we get this value (I would also change the behaviour in cask-repair), we should (I hope) be OK. Auditing would also need to take this into account.
Something else we could do is deprecate :sha256 in favour of a new value. Call it :checkpoint or something like that. That way there’s no way we can get reasonable complaints the value isn’t representative of the true shasum, because it isn’t and we’re making that clear. It’s a checkpoint value we use for ourselves, and users encountering it would need to consult the documentation to understand what it means (and hence learn how to correctly set it).
So in the end the idea ends up being simply about:

Deprecate :sha256 in appcast.
Add :checkpoint (or another name) to appcast.

But I wanted to make this PR as soon as possible for discussion, now that we know every appcast is up to date.

Cancelled the Travis job.
I'd be more comfortable if https://github.com/caskroom/homebrew-cask/issues/16529 was implemented before this change, honestly.

I'd be more comfortable if https://github.com/caskroom/homebrew-cask/issues/16529 was implemented before this change, honestly.

Having the audit won’t change this. This change was made using the system the audit will use, so we can be sure it is correct now, because all appcasts are. That won’t stay like that for long, though (It took me a few days to get them all up to date, and after finishing I still had to red a bunch of them due to updates).
If we agree on this new system, we can merge the change as it is, and then change all occurrences of :sha256 to :checkpoint (the name) and rewire one to the other, and extend audit, all at the same time.

In that case, LGTM.

I am... surprisingly okay with this!

Not wild about :checkpoint.  Maybe :hash or :caskhash?  Not coming up with any other ideas but :checkpoint feels actively misleading, like a last-accessed date.
I'll repeat this in #16949 but I think there must be a simple script to compute this value for a wannabe-contributor

With :hash, :checksum, and variants, I worry users take the hexadecimal value too literally (i.e. they wrongly deduce what it is/how to get it). For that reason I think a name that will make you wonder “what exactly does that mean” might be somewhat beneficial, to force contributors to read the documentation on it.
:value is something that crossed my mind as well. A bit generic, but not too bad. Like you too, I’m also not a huge fan of :checkpoint, but it is specific.
How about :control? As in “control point”, or “control value”. That seems accurate.

@Amorymeltzer The documentation would look something like https://github.com/caskroom/homebrew-cask/issues/16689#issuecomment-170583701
(See also the WIP CONTRIBUTING rewrite and documentation split-up issues)

Yeah, I like vague; :value helps keep that "hmm," as you say.  Would it be crazy to do :version or :update?  Correct message, clear it is connected to the stanzas above, and obvious it needs to be updated.

:identifier? It uniquely identifies a specific appcast at a specific point in time.

:update_id?

I’d be fine with :id. The name is so insignificant/meaningless in this matter, I’d rather not need two words for it. Lets move the name discussion to https://github.com/caskroom/homebrew-cask/issues/16949, though.
Already there are conflicts cropping up, so we just need to decide on if doing this is a good idea at all. We three seem to agree it is. @jawshooah any objection?

I'm fine with this. Not sure about the naming, but as you said we can move that discussion to #16949.

ID is fine with me, fwiw

Will fix conflicts and merge, then.

As a heads-up, the command to get :checkpoint will require a small tweak. Hadn’t noticed it in my initial tests, but caught it now. Automating it and will include the updated values in https://github.com/caskroom/homebrew-cask/issues/16951.
Instead of sed '/<pubDate>/d', it’ll be sed 's|<pubDate>[^<]*</pubDate>||g'. The edge-case (30 casks) are appcasts that are a single-line xml, in which case the previous code would delete the whole line, making the file essentially empty. This way, we’re only deleting the problematic part.


Only archive done task if all its subtasks are done

(Very useful plugin, thanks! :))
Currently, when a main task (e.g. "Revise class") is done and contains subtasks (e.g. "Check out PyTorch"), the "Archive… nested…" command archives it along with its subtasks even if they are not done.
Now, I realize that the main task could written as a bullet (instead of a checkbox), so as to prevent this, but this has some limitations:

When archiving all the subtasks, the main task would remain in the list of tasks. This requires some manual cleaning.
The main task might still be a task: for example, if "Revise class" is not fully done yet (maybe more subtasks should be added before we feel ready for the exam), it is better to have an empty checkbox to indicate this.

Overall, I don't see that one may want to archive any task if its subtasks are not all done, so I would suggest that "Archive… nested…" keep the main task until it is done along with all its subtasks.

Hello, @lebigot,
Thank you for your input! I added the option in 0.21.0.

This worked perfectly, in my test. Thanks a lot for your reactivity!
Now I can sleep well, without fearing that some "Header task" gets deleted or archived before it's done. 😅


Refactor variable renaming in utils.ts

This pull request refactors the variable renaming logic in the utils.ts file. Specifically, it updates the renameVariables function to handle the "inputs" variable separately and renames it to "Prompt Variables". Many users were confused with the naming inputs.

@bekossy can you please review and test this PR. Thank you!


Bump contract version

Current dependencies on/for this PR:

main

PR #119

PR #120

PR #121

PR #122   👈

This comment was auto-generated by Graphite.

Current dependencies on/for this PR:

main

PR #119

PR #120

PR #121

PR #122   👈

This comment was auto-generated by Graphite.


[freeglut] fix debug macro patch https://github.com/microsoft/vcpkg/issues/8315

Describe the pull request

The release versions of freeglut are being linked to the debug libraries.

MSVC does not automatically recognize ifdef NDEBUG macro when it has not specifically been set.

In the current configuration the results of the NDEBUG macro evaluation will always be false for MSVC builds.

Changes to correct the issue for MSVC and maintain compatibility with other vendor compilers are as follows:

freeglut port directory adding file fix-debug-macro.patch

Changes the all occurrences of
ifdef NDEBUG to
if defined(NDEBUG) || !defined(_DEBUG) in the following files:

./include/GL/freeglut_std.h
./src/blackberry/fg_main_blackberry.c

freeglut CONTROL file version updated/incremented from 3.2.1 to 3.2.1-1
freeglut portfile.cmake updated to execute the added fix-debug-macro.patch

What does your PR fix? Fixes https://github.com/microsoft/vcpkg/issues/8315

Which triplets are supported/not supported? Have you updated the CI baseline? No changes made to triplets supported/not supported but will effect static and dynamic builds.

Does your PR follow the maintainer guide? Yes I believe it does.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: timautry sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@JackBoosY This is ready for review and approval at your convenience.

osg:x64-linux regression is not relate to this PR.


t_kjv sqlite3 db has errors

3 John 1:15
[]
There should not be a "3 John 1:15", chapter (and book coincadentally), end after 1:14.
sqlite> select * from t_kjv where b is 64;
64001001|64|1|1|The elder unto the wellbeloved Gaius, whom I love in the truth.
64001002|64|1|2|Beloved, I wish above all things that thou mayest prosper and be in health, even as thy soul prospereth.
64001003|64|1|3|For I rejoiced greatly, when the brethren came and testified of the truth that is in thee, even as thou walkest in the truth.
64001004|64|1|4|I have no greater joy than to hear that my children walk in truth.
64001005|64|1|5|Beloved, thou doest faithfully whatsoever thou doest to the brethren, and to strangers;
64001006|64|1|6|Which have borne witness of thy charity before the church: whom if thou bring forward on their journey after a godly sort, thou shalt do well:
64001007|64|1|7|Because that for his name's sake they went forth, taking nothing of the Gentiles.
64001008|64|1|8|We therefore ought to receive such, that we might be fellowhelpers to the truth.
64001009|64|1|9|I wrote unto the church: but Diotrephes, who loveth to have the preeminence among them, receiveth us not.
64001010|64|1|10|Wherefore, if I come, I will remember his deeds which he doeth, prating against us with malicious words: and not content therewith, neither doth he himself receive the brethren, and forbiddeth them that would, and casteth them out of the church.
64001011|64|1|11|Beloved, follow not that which is evil, but that which is good. He that doeth good is of God: but he that doeth evil hath not seen God.
64001012|64|1|12|Demetrius hath good report of all men, and of the truth itself: yea, and we also bear record; and ye know that our record is true.
64001013|64|1|13|I had many things to write, but I will not with ink and pen write unto thee:
64001014|64|1|14|But I trust I shall shortly see thee, and we shall speak face to face. Peace be to thee. Our friends salute thee. Greet the friends by name.
64001015|64|1|15|[]

I am learning more about the bible as I work on this project, and for that I am grateful. the KJV does indeed not have 15 verses for 3 John. However some translations do, such as the ESV.
Wikipedia states " 3 John has 15 verses in the critical SBL Greek New Testament text,[4] or 14 in the Textus Receptus.[5]" https://en.wikipedia.org/wiki/Third_Epistle_of_John.
blueletterbible states "Summary - Question 8
Why Is the Bible Divided into Chapters and Verses?
In the original text of the various books of the Bible, there are no such things as chapter and verse divisions. They were added later for the sake of convenience. While they are helpful, they are not authoritative in any sense of the term. In fact, they can cause a number of problems.
Chapter and verse divisions give the impression that the Scripture should be read and studied in bits and pieces. This is not what the original authors intended. The entire context must always be considered. Consequently, the chapter and verse divisions should be ignored when one attempts to properly interpret the entire message of Scripture."  https://www.blueletterbible.org/Comm/stewart_don/faq/bible-special/question8-why-is-the-bible-divided-into-chapters-and-verses.cfm.
Thank you for letting me share what I have learned today. The github issue is still correct in that the KJV is split into chapters and verses, and has no verse 15 in 3 John. However I should be careful not to assume this about all translations.

I have opened https://github.com/scrollmapper/bible_databases/pull/60 as a way of discovering these errors sooner. I will have a separate PR to fix these inconsistencies soon.

It appears that https://github.com/scrollmapper/bible_databases/pull/63 fixes this.


Fill in wiki on raw io benchmark utility

I've made page as requested by NCCS staff on how to use the low-level IO benchmarking tools we have in MAPL
I've filled my bits about the checkpoint_simulator.x and restart_simulator.x and will add stuff about how to profile where possible with ExtDataDriver.x at a later date.
@tclune made his own benchmark
https://github.com/GEOS-ESM/MAPL/tree/main/benchmarks/io/raw_bw
There is a section for him to fill about how to use his tool at his leisure:
https://github.com/GEOS-ESM/MAPL/wiki/MAPL-IO-Benchmarks#raw-io-benchmark

Note to @bena-nasa and @tclune, I hope to have MAPL 2.48.0 out soon (see #3049). So the wikis can point to that release instead of develop...hopefully by tomorrow?

I've put a bit into my section, but probably won't add much more without external pressure.


899 available_plugins improvements and tests

This PR adds tests and improves the module to fix found problems with the class
What ticket does this PR close?
Connected to #899 but does not finish it (external_plugins still needs tests)
Where should the reviewer start?
go test -v ./internal/plugin/...
What is the status of the manual tests?
Have you run the following manual tests to verify existing functionality continues to function as expected?

[ ] Manually tested Keychain provider
[ ] Manually run the K8s demo (which requires changing the image the demo uses to the local build of Secretless)
[ ] Manually run Kubernetes-Conjur demo with a local Secretless Broker image build of your branch

If this feature does not have any/sufficent automated tests, have you created/updated a folder in test/manual that includes:

[ ] An updated README with instructions on how to manually test this feature
[ ] Utility start and stop scripts to spin up and tear down the test environments
[ ] A test script to run some basic manual tests (optional; if does not exist, the README should have detailed instructions)

Links to open issues for related automated integration and unit tests
Links to open issues for related documentation (in READMEs, docs, etc)
Screenshots (if appropriate)

Wrong branch target. Closing.


Support TTL for CosmosDbPartitionedStorage in Microsoft.Bot.Builder.Azure

Problem Statement
The current implementation of CosmosDbPartitionedStorage does not support the declaration of a default TTL when creating a new Cosmos DB container. Supporting default TTL requires either creating the container before the code runs or updating the container at a later time to have a default TTL.
Proposed Solution
Add TimeToLive to CosmosDbPartitionedStorageOptions and respect the option when creating a new Cosmos DB container.
Describe alternatives you've considered
Considered creating a callback hook (Action<ContainerBuilder>) that would be called in CreateContainerIfNotExistsAsync before invoking CreateIfNotExistsAsync on the ContainerBuilder. This seemed a little too heavy-handed.
Additional context
If this is accepted, I'm prepared to submit a PR.
[enhancement]

Other requests for this:
https://github.com/microsoft/botbuilder-dotnet/issues/3086
https://github.com/microsoft/botframework-sdk/issues/5890

@WalkingDisaster The SDK team has decided not to implement TTL (or additional Cosmos SDK features) so as not to duplicate the Cosmos SDK, itself.
However, @EricDahlvang provided a GREAT programmatical workaround or you can also set it manually.


Provide a 'feature("featureName", "g:a")' dependency declaration syntax in the spirit of 'testFixtures("g:a")'

Expected Behavior
If I depend on the test fixtures feature of a component, which is a feature called testFixtures, I can do it like this:
dependencies {
   implementation(testFixtures(project(":module-a")))
}

If I depend on another feature variant that I defined and named myself – via java.registereFeature("extraFun") – I would like to be able to address that in a similar matter in a dependency declaration. For example:
dependencies {
   implementation(feature("extraFun", project(":module-a")))
}

Current Behavior (optional)
No response
Context
Currently, you have to use the quite verbose requireCapabilities notation, for which you need to use/know things you have not defined yourself – the group in the capability coordinates and how the "feature name" maps to the name in the coordinates:
dependencies {
   implementation(project(":module-a")) {
     capabilities {
       requireCapabilities("org.example:module-a-extra-fun")
     }
  }
}

But if we think in Java features, the underlying concept of capabilities should not need to shine through on the consumer side. As it is also not required in the registerFeature() on the producer side.
And it seems unintuitive that this is available for testFixtures, a "Gradle-owned feature", but not for custom features.

Thank you for your interest in Gradle!
This feature request is in the backlog of the relevant team, but this area of Gradle is currently not a focus one, so it might take a while before it gets implemented.

Since this issue does not exactly match the feature added (but the new feature solves the use case) we have written a separate issue for the release notes entry https://github.com/gradle/gradle/issues/30685

I am very happy to see a solution for this implemented. Thank you @jvandort for adding the feature!

I just got around to trying this out now. I am happy that there is such a feature now, but why was the original syntax suggestion rejected?
It's now still much too noisy IMO, because you need to add all these curly braces. And you still see the word capabilities (which you don't see on the producer side). 😒
    implementation(testFixtures(projects.app))
    implementation(projects.app) { capabilities { requireFeature("mock-api") } }

I was hoping for something like this:
    implementation(testFixtures(projects.app))
    implementation(feature("mock-api", projects.app))

I agree with @jjohannes. Having to use capabilities feels clunky.

I agree the implemented API vs the API proposed in this issue is clunkier.
Though I question, how would you implement feature("mock-api", projects.app)?
We call methods like this "dependency modifiers". Some common ones are:

The platform / enforcedPlatform modifiers:
platform(projects.app) - This mutates the passed-in dependency and adds an attribute to it. Under the hood, this is a light wrapper around:

implementation(projects.app) {
    attributes {
        attribute(Category.CATEGORY_ATTRIBUTE, objects.named(Category, Category.REGULAR_PLATFORM))
    }
    endorseStrictVersions()
}

The test-fixtures modifier
testFixtures(projects.app) - This mutates the passed-in dependency and adds a capability to it. Before this feature was implemented, it was a light wrapper around:

implementation(projects.app) {
    capabilities {
        requireCapability(new ProjectDerivedCapability(<above_dependency>.dependencyProject, "testFixtures"))
    }
}

In this case, we use the internal ProjectDerivedCapability to create a special capability that at resolution-time resolves to the coordinates of the target project (in an IP unsafe manner).
The core point here is that (without wrapping the dependency in some other wrapper type that later needs to be instanceof'd and deconstructed), we need to have some lower-level construct on the dependency that the modifier can call into.
Without the feature as implemented, I don't see how we would implement the feature as requested in this issue.

I am not sure I fully grasp the implementation details. My thought was, from a naive usability perspective, if there is a
implementation(testFixtures(projects.app))

It should also be possible to have a
implementation(feature("mock-api", projects.app))

I also didn't mean to say that the new technical solution is bad. I think it is great! I am just saying that the user facing API could be improved. Can't you add the original proposed API now on top of the new implementation? I see that testFixtures has been modified to work like this now:
ModuleDependency moduleDependency = (ModuleDependency) testFixturesDependency;
moduleDependency.capabilities(c -> c.requireFeature(TEST_FIXTURES_CAPABILITY_FEATURE_NAME));

I think an API like feature("...", projects.app) could be implemented similarly (or am I missing something?).
ModuleDependency moduleDependency = (ModuleDependency) featureDependency;
moduleDependency.capabilities(c -> c.requireFeature(featureName)); // 'featureName' being the 1st method parameter

I also prefer the notation of @jjohannes so I created a PR to add this overload: https://github.com/gradle/gradle/pull/31358

Reopened since the original fix did not add a convenient API as proposed. Lets fix this for with https://github.com/gradle/gradle/pull/31358


Newsletter 10: beehive, rustsim, safearch, svgface

Part of #139 ("Newsletter 10: Coordination/Tracking")
@Lokathor @sebcrozet @dabreegster @toasteater r?

Merging in a few hours if there're no objections

looks good in the safe_arch portion.

LGTM for svg_face

No objection for the rustsim survey 👍


CE-7621: prepare release

Note: if you click on the API Reference link in the readme, it opens the documentation but the theme of the API pages is removed. I'm not 100% sure but this should be solve by the PR # 21.


Outdated cask: wordmark

Outdated cask: wordmark.
Info:

version: 2.1.7.
appcast url: https://github.com/wordmark/wordmark/releases.atom.

Please do not run find_outdated_appcasts unless you are a maintainer.


deps: make stable_deref_trait an optional dependency

Configures the stable_deref_trait to only be included for arm_llsc and target_arch = "x86" platforms. This is because the dependency is only used for the pool module, and the pool module is only used on the above platforms.
Reduces the dependency graph for all other platforms.

I think this can also be excluded on aarch64: https://github.com/rust-embedded/heapless/blob/9feb6647225316e9022f3ebdd63ab0c2380edeb8/build.rs#L31

I think this can also be excluded on aarch64

Should I also remove the associated constant and compile_probe function?


[inputs] remove multi input realization

Commits to be reviewed in this PR
[inputs] remove multi input realization
This patch removes multiinput realization behavior for now, this is not
used else where so it's fine to remove
Self evaluation:

Build test: [X]Passed [ ]Failed [ ]Skipped
Run test: [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Jihoon Lee<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #1618. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnstreamer.mooo.com/.


Add user inputs for demo website

We need to support inputs for user.
Questions:

[ ] Do we allow user to change Signal and External nullifier in extension?
[ ] Do we allow user to enter random signal value?

https://discordapp.com/channels/943612659163602974/1064538262020042802/1075814823612399728
Answered on discord, but the summary is:
The user shouldn't edit these values, they should be provided by the site that is requesting the proof.


Should .NET Standard be implicitly referenced?

I have a C# wrapper around a native macOS library for Xamarin, and want it to work with .NET 6 as well. It also has a package reference to a .NET Standard library.
However, if I try to consume any enum from that package, I get:
 MyClass.cs(12,12): Error CS0012: The type 'Enum' is defined in an assembly that is not referenced. You must add a reference to assembly 'netstandard, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51'.

For example, with a csproj like:
<Project Sdk="Xamarin.Legacy.Sdk/0.1.2-alpha6">
  <PropertyGroup>
    <TargetFrameworks>xamarinmac2.0;net6.0-macos</TargetFrameworks>
    <Nullable>enable</Nullable>
    <NoNFloatUsing>true</NoNFloatUsing> <!-- https://github.com/xamarin/xamarin-macios/issues/14434 -->
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="System.Text.Json" Version="6.0.3" />
  </ItemGroup>
</Project>

And a MyClass.cs like:
public class MyClass
{
	public MyClass()
	{
		var x = System.IO.FileAccess.Read; // an enum from .NET Standard
		var y = System.Text.Json.JsonTokenType.Comment; // an enum from a package
	}
}

This won't build. Oddly enough, the first line will build, and the second will not.
If I explicitly supply a hint path, it seems to work:
<ItemGroup Condition="'$(TargetFramework)' == 'xamarinmac2.0'">
  <Reference Include="netstandard, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51">
    <HintPath>/Library/Frameworks/Xamarin.Mac.framework/Versions/<IP_ADDRESS>/lib/mono/4.5/Facades/netstandard.dll</HintPath>
  </Reference>
</ItemGroup>

Is this intentional, or should the SDK pull in .NET Standard automatically?

The Mac support here was a contribution, but it sounds like the Facades directory isn't setup appropriately for Xamarin.Mac.
I would probably compare a .binlog from a Xamarin.Mac class library and see if some MSBuild property has a different path we are missing. Probably something needed in this file:
https://github.com/xamarin/Xamarin.Legacy.Sdk/blob/main/src/Xamarin.Legacy.Sdk/Sdk/Xamarin.Legacy.macOS.targets


Can't customize find match color in scrollbar

Version: 1.19.3
OS: Win 10
    "workbench.colorCustomizations":
    {
      "editorOverviewRuler.findMatchForeground": "#ff0000ff",
    }

I presume this is meant to color the find matches under the scrollbar. Doesn't seem to do anything.

Can confirm this.
How can the find widget color be changed in scrollbar?

Looks like this was fixed.


⚠ TiendaEnLinea has degraded performance

In 5be318f, TiendaEnLinea (https://www.fahorro.com/) experienced degraded performance:

HTTP code: 200
Response time: 6372 ms

Resolved: TiendaEnLinea performance has improved in f0ea62d.


Contextual Outlier Detection

Contextual Outlier Detection
--- Isolate outlier detection method from context finding
-- add Unit test cases for contextual outlier detection
-- change Datum to include contextual attributes, and also change data ingest to ingest contextual attributes
-- add description of the parameters in the docs folder
Next steps:
-- implementing general pruning rules
-- implementing algorithm specific pruning rules.

@raininthesun Is this ready for review?

@pbailis As we discussed, I've squashed into one commit, added description in the docs folder, and added test cases. It is ready to be reviewed and merged.

LGTM, with minor comments. Feel free to merge following, and let's discuss gameplan for integrating with the rest of the pipeline.


file_zbc: ZBC disk emulation handler

file_zbc.c implements a ZBC disk emulation using a file as backend
storage. The handler is compiled as handler_file_zbc.
handler_file_zbc can emulate both host aware and host managed disks.
In both cases, the LBA size, zone size and the number of conventional
zones can be configured through options in the configuration string.
The configuration, together with zone information (zone type, zone
condition, write pointer position, etc) are stored as metadata at the
beginning of the backstore file, invisibly to the user. The
configuration and state of the emulated device is thus retained
accross reboots.
The metadata in the backstore file is reinitialized (i.e. the device
configuration and state reset) if the specified backstore file does
not exist, if the metadata in the file is incorrect or if configuration
options are specified in the configuration string. Otherwise, the
metadata is reused as is, restoring the emulated device state from its
last use.
The primary goal of handler_file_zbc is kernel features development
and tests. It is especially useful for people without access to SMR
disks or on systems on which those disks cannot be used
(e.g. a laptop).
The emulated device, used either through the loopback or iscsi fabrics,
successfully passes libzbc conformance tests (using kernel 4.13-rc as
many target patches fixing various problems with sense code and sense
keys reporting are needed). Testing was also performed using f2fs
(which natively supports host managed zoned block devices) and the
dm-zoned device mapper. Additionally, the emulated device is also
currently being used for btrfs zoned block device support development
and can also be very helpful in general to test Linux block and SCSI
stack support of zoned block devices.
The blktets framework would also be another area that can benefit from
ZBC emulation.
Signed-off-by: Damien Le Moal<EMAIL_ADDRESS>

Sorry for the delay. I have been on vacation. I will have comments by next week.

Hi Mike,
Any comment regarding this pull request ?
Is there anything I can help clarify ?
Thank you.

Sorry for the late response.
My only major comment is just about if we can combine this with file_example.c, so we only have the one handler that does IO to a file. I have been going back and forth with myself, and my first thought is that it is not really worth it as the zbc handler will be mostly used for testing and development, so let it be separate so you can do whatever crazy/interesting things you want to do.
I guess one question I had is do you think users will ever use this in production? Will users want to use it in some sort of setup where they do other things like multipating? Will you need stuff like ALUA? Will you need to support persistent reservations?

Mike,
I initially also thought of having a ZBC mode with the file handler, but there was enough differences and changes that making a new handler ended up being easier, both development wise but also for maintenance as any change for the "regular" disk emulation would not necessitate also test the "zbc" side.
As you correctly state, the intent of the ZBC handler is testing and development. I would never ever recommend this for production. That would be nonsensical anyway. The premise for using SMR/ZBC drives is capacity. So emulating that on top of a file on a regular drive would give an application developer all the headaches of dealing with sequential writes while not providing the benefits of higher capacity. Production use makes no sense to me and I will always discourage it.
So features like multipathing/ALUA are not a requirement at all. The kernel does not support that for SMR drives anyway (think of the nightmare that these introduce if one wants to maintain a correct sequential write pattern to the disk sequential zones).
For the test and development use case, there is however a strong need from the kernel folks (at least my team), as well as application developers too (some of our customers are very interested). The biggest headache with SMR drives is the ecosystem. Many BIOSes and HBAs are simply not yet supporting those disks, in particular the host managed flavor. Things are slowly getting better and there are now HBAs supporting ZBC/ZAC. But while a system developer readies a production system for SMR, application and kernel developers can still use tcmu ZBC handler to develop and test their software readiness for SMR. Tcmu-runner with the ZBC handler is really a great tool for that as it can even also give you a bus analyzer level information. Some kernel maintainers have requested such a tool as a condition for accepting ZBC related code (e.g. BtrFS community as we are working on BtrFS support for ZBC).
Last but not the least, standards are evolving and ZBC-2.0 proposals are already being worked on, as well as a dynamic SMR version (where zone types can be changed dynamically to trade capacity vs ease of use). The ZBC handler will also be a great tool for validating standard choices before these choices make it to the final standard (somewhat the same model as was used for NVMe development with the standard and kernel support being developed simultaneously and cross validated).
Best regards.

Sounds good. Just handle the basic review comments and I will merge it up.

Mike,
Just updated the pull according to your comments. Please check.
In particular, the fixes to the README.md file are more extended than you requested. I fixed the markup in different places to improve the display in the top page.
Thank you.
Best regards.

Thanks, and sorry again for the delay.


Feature request: Renaming

There is no way to just change the key for an item. If I wanted to do this manually, I would have to use Delete and then Add, but upon deletion the OnEvicted function would be called, which I don't want.
My use case is a webservice, where the user can create some data entry using an assistant, which consists of multiple web pages. I use go-cache to store the already entered data from past pages until the assistant is finished. I use an XSRF-token (generated with golang.org/x/net/xsrftoken) as the key for the cache item. My issue with this is, that I cannot extend the validity of the token after one step of the assistant has been completed, so I would have to generate a new one, but copying the cache item, so that it can be found with the new token would be expensive (contains large files, which are deleted through OnEvicted).
Ideally I could also set a new duration upon renaming, but maybe this would be better as a separate feature.

I implemented this as Rename() in github.com/arp242/zcache; it doesn't update the expiry, so you'll have to do a second Touch() call to update that if you want it.
Since this hasn't been maintained for a few years, I've been maintaining a fork over there.

Oh, also your PR #153 from last month is implemented as DeleteAll() in zcache. I did that last year already (I made a new function because both calling and not calling onEvicted have valid use cases).

Thanks for the info and thanks for implementing Rename() so quickly. I'll check out your fork.


test: Fix declaration of main method

After reading cabos comment in #44 I wondered if cn-cbor actually compiles with C compilers other than gcc and clang and tested this by compiling it with pcc using the Simple-Makefile.
pcc complained that the function declaration for main in test.c is incorrect and according to section <IP_ADDRESS>.1 of the C99 standard pcc is actually right about this.

Not sure the test.c is particularly portable, as evidenced by
  system("leaks test");

(and the lack of a return...), but this should be an innocuous change.
Further portability fixes are welcome!

(and the lack of a return...)

Section <IP_ADDRESS>.3 of the C99 standard states the following:

[...] reaching the } that terminates the main function returns a value of 0.

So you probably don't need a return statement in that function.


Error with Center Align and Split Text to Size

Hi everyone! I keep getting an error when trying to open a PDF with center align and split text to size on only a specific set of letters!
My code:
doc.text("Smoke Bombs", 0, 392, null, null, 'center' );
I opened the raw PDF and saw this:
(Smoke) Tj 1.7763568394002505e-15 -8 Td (Bombs) Tj ET BT /F1 8 Tf
It is definitely the 1.7763568394002505e-15 value but I don't know how that is calculated or really what that value is for?
I am using 'pt' for units as well.

Seems like on line 1524:
if (align === "center") delta /= 2;
There should be some limit of decimal place?

Yeah. It is definitely an error. But the project owner doesn't merged some pull requests yet to fix those errors...

But the fixing point is not the line you found... Wait

The line is:
text += ") Tj\n" + ((left - prevX) + delta) + " -" + leading +
Change it to
text += ") Tj\n" + f2((left - prevX) + delta) + " -" + f2(leading) +

@arasabbasi made the fix, thanks! Yeah, would hope it makes it to the official build.

var str = '';
var count = 0;
const flattenItems = (items, key) => {
return items.reduce((flattenedItems, item) => {
flattenedItems.push(item);
console.log('id',item.id)
if (Array.isArray(item[key]) && item[key].length > 0 ) {
console.log('id child',item.id)
str += '->';
flattenedItems[flattenedItems.length-1].name = str+flattenedItems[flattenedItems.length-1].name;
flattenedItems = flattenedItems.concat(flattenItems(item[key], key))
}else{
str = '';
}
return flattenedItems
}, [])
}
console.log(flattenItems(items, 'children'));

@arasabbasi Hey, what's the open PR for this?

@MrRio
I already modified everything in the PR #1448 and fully unit tested. But nothing is getting merged. It is just sad...

Fixed by #1433
Will be part of next release.


Practice duel, Faie, empty deck

Summary
I started a Practice Duel against Faie, setting "Difficulty" to 100% and "Random cards" to 40 and she started the game with only the cards in her starter hand and empty deck.
Next try everything was fine, but her deck started at 26/40 (so 10 cards short).
Impacted services

[x] The frontend app
[x] The game servers
[ ] Game builds or automation scripts
[ ] Tests or Github Actions
[ ] Infrastructure (Terraform)

Steps to reproduce
Environment information

Operating system: Windows,
Desktop client, latest

Additional context

How are you measuring the number of cards in the opponent's deck? I'm not able to repro yet with 100% difficulty and random cards enabled.

How are you measuring the number of cards in the opponent's deck? I'm not able to repro yet with 100% difficulty and random cards enabled.

If you enable "Player Detail" in settings it's just shown, it's public info.
Plus in the first case she was taking the "empty deck draw" damage from first turn (otherwise I probably wouldn't have noticed).
It so far happened only once to me (though as I said, another attempt with her gave her less than full deck).


Added log redaction test and docs

Coverage increased (+0.02%) to 99.011% when pulling 6ac8a7771b0ed39696e96ec4a8a54d43b4d1de34 on add-log-redaction into 0c5d6d49e0ca3d07561fe1e173d1a8f2e53a6740 on master.

Coverage increased (+0.02%) to 99.011% when pulling 6ac8a7771b0ed39696e96ec4a8a54d43b4d1de34 on add-log-redaction into 0c5d6d49e0ca3d07561fe1e173d1a8f2e53a6740 on master.


Data xref is not updated when the binary is edited

As can be seen from the screenshot, the line at 0x4018 references data at 0x3fb8. However, the data at 0x4018 is edited and become all zeros. The xref remains, though
Related to: https://github.com/Vector35/binaryninja-api/issues/2323

This is a bit harder to deal with than https://github.com/Vector35/binaryninja-api/issues/2105, since we are only adding/removing data xrefs when data vars are added. There is no consideration for the dynamics that data vars can change.


idle-countdown Directive Not Updating Countdown Value

Copied your demo warning dialog $modal code into an existing project.  Modal dialog template is on a page that is handled by an existing controller using ngRoute.  IdleWarn message fires, but the idleCountdown value is not updated due to the following error:
Error: [$rootScope:inprog] $digest already in progress
http://errors.angularjs.org/1.2.25/$rootScope/inprog?p0=%24digest
at beginPhase (http://localhost:55287/Scripts/dev/lib/angular-scenario.js:22801:9)
at Scope.prototype.$apply (http://localhost:55287/Scripts/dev/lib/angular-scenario.js:22590:11)
at Anonymous function (http://localhost:55287/Scripts/dev/lib/angular-idle.js:310:25)
at Scope.prototype.$broadcast (http://localhost:55287/Scripts/dev/lib/angular-scenario.js:22772:15)
at countdown (http://localhost:55287/Scripts/dev/lib/angular-idle.js:175:25)
at wrappedProgressback (http://localhost:55287/Scripts/dev/lib/angular-scenario.js:21382:15)
at Anonymous function (http://localhost:55287/Scripts/dev/lib/angular-scenario.js:21350:17)
at Scope.prototype.$eval (http://localhost:55287/Scripts/dev/lib/angular-scenario.js:22493:9)
at Scope.prototype.$digest (http://localhost:55287/Scripts/dev/lib/angular-scenario.js:22305:15)
at Anonymous function (http://localhost:55287/Scripts/d
I commented out the $scope.$apply in the IdleWarn message callback function, and the countdown works as expected.  Same error is also raised in IdleTimeout message handler callback function on line 315.  Do we need to be in a $scope.$apply here?

I think this is an issue between AngularJS 1.2 and 1.3. In 1.3 you need the $apply call, but in 1.2 you don't. I need to figure out if a digest is in progress and act accordingly, or figure out how to schedule the action for the next available digest.

Thanks, Mike.  Our project is on 1.2.28.

Rather than use $apply, I switched to $evalSync which should guarantee that the changes made in the expression will happen either during the current digest or cause another digest if one isn't happening. Try a copy from the developer branch to see if it resolves your issue and reopen if it doesn't. I did try it against 1.3.x and 1.2.27 and it looks good in both so far.

I am on Angular v. 1.3.15 and it seems that while the countdown argument in the 'IdleWarn' event is in fact being updated, changes made to $scope within the event still require being wrapped in $apply

+1


NIFI-8698: If using a Data RecordPath and handling schema drift, ensu…

…re that the RecordPath is evaluated against the data before attempting to determine the fields to update. Also refactored to move the handling of schema drift into its own method for the purpose of clarity

Thank you for submitting a contribution to Apache NiFi.
Please provide a short description of the PR here:
Description of PR
Enables X functionality; fixes bug NIFI-YYYY.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically main)?

[ ] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] Have you verified that the full build is successful on JDK 8?
[ ] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

+1 LGTM, thanks for the fix! Merging to main


Added ctor that takes a list element using params

Added a constructor that takes elements with params. It was a bit silly of me not to add it in the original PR (#499) since it makes creation easier.

Thank you for your contribution. It's now live as AutoFixture 3.38.1.


go/tendermint: ABCI interface cleanup

Based on #2244.
Simplifies ABCI transaction execution now that we have ctx.State().

Rebased on master (with timer changes from #2244) so we can merge this early.


Using DynamicObjects in script

I am using V8 script engine and trying to use System.Dynamic.DynamicObject with overridden TryGetMember and TryGetIndex in the script. I am adding it via engine.AddHostObject(). Although I see that GetMetaObject() is called twice - when object is added to engine and when it is accessed from script - this does not seem to work. The overridden methods of DynamicObject get never called when access is attempted from within script yielding [undefined] results. If I use created object from host code in C#, it works as expected.
At the same time, ExpandoObject does work. E.g. if I create object in the host, add some dynamic field to it and then pass it to engine, script can access dynamic field I've created. I guess, there is something about implementation that causes this discrepancy but I could not find what.
Maybe, you could advise?
Thanks!

For its dynamic properties to be accessible from script code, your object must implement GetDynamicMemberNames in addition to TryGetMember.

Ok, thanks! Tried that and it actually started working for "dot" access (obj.field). How would I make it to recognize indexing? Is it translated into member access or do I have to implement some interface (e.g. IDictionary)?

For your dynamic object to support indexing directly, it must expose its valid indexes as property names. That is, GetDynamicMemberNames would have to return a list of string-formatted indexes, and TryGetMember would then have to work with those strings.
A simpler solution might be for your object to have a named property that returns an array, list, or other object that implements IList or IList<T>, which would then automatically support indexing.

Thank you for prompt and quick answers, it works just like you explained!
I guess, it would be useful to add this somehow into the list of examples, just like with ExpandoObject.


get_tree json fixes

null pointer check against container names
use actual JSON null values instead of "null" strings
add "visible" boolean property to all containers
remove "nodes", "floating_nodes" and "unmanaged_nodes" arrays from views as they cannot have children

Thanks!


Avoid checking the version on a Language Server path from manual configuration

I ran the SDL pipeline against this change, but it's still resulting in the same bug detected. Is that bug no longer valid after these changes?

Uhm, I was hoping that it would go away because of the scoping down of the path, but I'll try something else now.


Improve Colorscheme and change Dark/Light Mode Settings

This PR adds more contrast to both light and dark modes and removes the dark mode toggle in the Settings App to rather follow the user's system theme.
Previous

Light Mode
Dark Mode

Update

Light Mode
Dark Mode

@AlexanderG2207 I don't think that the grey header looks that great in light mode. Maybe just use white?
Same for the launch screen

@AlexanderG2207 I don't think that the grey header looks that great in light mode. Maybe just use white? Same for the launch screen

Good point! I changed the navigation bar in both modes and also the launch screen.


HV-1280 (5.4)

Follow up for using the SM correctly.

Merged to master, 5.4, 5.3 and pending PR for 5.2 with some follow-up adjustments. Thanks.


Add Chinese Gaode Map base layer

Gaode Maps is a map service providing detailed and accurate map of China. Google Maps' data is much outdated and OSM is almost blank in some area of China.
This map layer includes loading map tiles from Gaode's server and fixing offset of the GCJ02 coordinate system.

Why did you close it?


Update to version 0.7.0

Codecov Report

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

@@           Coverage Diff           @@
##           master      #84   +/-   ##
=======================================
  Coverage   66.09%   66.09%
=======================================
  Files          34       34
  Lines        1056     1056
  Branches       63       63
=======================================
  Hits          698      698
  Misses        341      341
  Partials       17       17

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 42b9ac9...69111fd. Read the comment docs.


Update sbt to 1.9.8

About this PR
📦 Updates org.scala-sbt:sbt from 1.9.7 to 1.9.8
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-sbt", artifactId = "sbt" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scala-sbt", artifactId = "sbt" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, version-scheme:early-semver, commit-count:1

CI failure seems strange. I'm going to merge this anyway and then if it fails after merge, deal with it

yup, it's fine now (https://github.com/lightbend-labs/scala-fortify-sample/actions/runs/7535735832)


How achieve EoS in kafkajs?

Hello I am working on a project where we are trying to simulate an eshop flow using event driven approach, and in some services we would like to have EoS.
Our flow is one service(A) sends an event to a specific topic1 to service (B), service (B) reacts to event and sends another event to topic2, which is consumed by service (A).
According to my understanding in order to achieve this we have to make an idempotent producer which will use transactions API and a consumer which its isolation level should be read_committed:
My question is: should we use transaction.sendOffsets or just the:
await transaction.send({ topic, messages })
  await transaction.commit()

?

If you are simply doing a transaction within eachMessage, you don't need to use transacton.sendOffsets:
consumer.run({
  eachMessage: async ({ message }) => {
    const transaction = await producer.transaction()

    try {
      await transaction.send( ... )
      await transaction.commit()
    } catch (error) {
      await transaction.abort()
      throw error
    }
  }
})

In this case, if the transaction fails to be committed the consumed offset will not be committed.
However, if you were consuming messages for later processing, you'd need a way to move past that offset without committing it, and later commit only if the transaction was successfully committed. I believe this test shows it the best: https://github.com/tulios/kafkajs/blob/master/src/consumer/tests/consumeMessages.spec.js#L1090-L1203
This was implemented in #232 to resolve #207, both of which add more context.
The key bit to understand is that in order to have a consume-transform-produce loop you need to have a way to commit an offset only if the transaction within which you produce the message in response to the consumed message was successfully committed. So transaction.sendOffsets is a way to commit an offset as part of a transaction from outside of the consumer's loop.


Enhancementcatalog tvf implements user permission checks and hides sensitive information (#41497)

bp #41497
before #21790
Proposed changes
This PR unifies the duplicate parts of catalog tvf and show catalogs, adds permission check when querying catalog tvf, and hides sensitive information.

run buildall

run buildall


⚠ Bofrost.de has degraded performance

In cb69fa6, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 914 ms

Resolved: Bofrost.de performance has improved in b3fd208 after 20 minutes.


XD-3684: Updated job repository configuration to be consistent

This commit causes the database configuration to be consistent across the container and admin nodes.

LGTM

Rebased, regression tested, merged


session disable shared scan in default

Proposed changes
disable shared scan in default

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


[bug] callback case 2

The reduction ocst where printed in the wrong order for the callbackverbose() case 2.

Thanks a lot :)


Not able to connect to Snowflake

System information:

Operating system (distribution) and version: Windows 7
DBeaver version: 5.2.3.
Additional extensions

Connection specification:

Database name and version: snowflake
Driver name: snowflake jdbc
Do you use tunnels or proxies (SSH, SOCKS, etc)?

Describe the problem you're observing: Am getting error to connect to snowflake.Its throwing error SAML error. Please contact administrator

Steps to reproduce, if exist:
Include any warning/errors/backtraces from the logs

@vinayisnaga can you go directly to your snowflake UI and log in successfully?
Was there some change inyour setup that made this stop working?

@vinayisnaga Most likely it is a result of connection misconfiguration.
Please check this: support.snowflake.net/s/question/0D50Z00009T38Z5SAJ/when-i-try-to-connect-to-snowflake-using-snowsql-with-using-user-credentials-getting-error-message-related-to-saml-response-is-this-user-privilege-issue
This error may happen if your user name is incomplete (e.g. domain name is missing).


azure devops 无法注册

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

VS Code Version:
OS Version:

Steps to Reproduce:
1.Get started with Azure DevOps 点击continue后就一直转圈圈，公司和家里电脑，翻墙和不翻墙，都是一直在转圈

Cannot register azure devops.


Not authorized to acces this page

Hey I have read some issues #218 and more but I get a non authorized and a django login screen when I try to login. See picture:

I managed to log in with django.contrib.auth.backends.ModelBackend. But everytime if i log in with my azure account the staff and superuser is not set.

Please see https://django-auth-adfs.readthedocs.io/en/latest/settings_ref.html#group-to-flag-mapping

@Mergpijp that depends on what group you want to use to identify that the user is a staff user. For me it was only security groups, but your AD can be configured differently.

So do I understand correctly you need add group to group_to_flag_mapping?

    "GROUP_TO_FLAG_MAPPING": {"is_staff": ["Django Staff", "Other Django Staff"],
                            "is_superuser": ["Django Admins", "Security groups"]},

Maybe I need Groups and Security?

Maybe I need Groups and Security?

Again, we can't help you there. It's dependent on your set up.

What source attribute do I need to pick?

I don't understand what you mean by source attribute.

I mean in the groups claim edit/creation what to select under customize.

It depends how your Azure is set up. Access kan be sAMAccountName, but you could also use Azure AD roles (recommended). It all depends on your setup and how you want to manage your application registrations.

Did you figure it out? 😊

I parked this but I saw you recommending roles instead of groups. If I remember correctly.

Alright 👍  I’ll close this for now then.

I added a role in app under active directory > app registration > app roles:

I have added my user to the to roll under enterprise applications but I still cannot login into admin.

Wait forget my last comment I am looking at Azure AD roles to group. What role is required for superuser or staff?

I solved this by making a group in django admin with all permissions and put this group name in the group to flag mapping. Note that a superuser must also be staff if you want the rights. Thanks for your help.


this.ownerDocument(...).model is not a function when attempting to add new instance of subdocument

I have a schema called Contact and another Schema called email that is a subdocument of Contact.  The Contact schema can hold an array of emails.
When attempting to add a new instance of email, I get the following error in the AdminBro UI:
this.ownerDocument(...).model is not a function.
The error shows up on the ID field of the email subdocument
How can this be fixed?

can you paste that schemas and admin-bro config?

The admin-bro config is as follows
const AdminBroOptions = {     resources: ['Contact', 'Email', 'Phone', 'URL'],     rootPath: "/admin", };
The schemas are below:
const contactSchema: mongoose.Schema = new mongoose.Schema({     name: { type: String, required: true, match: [/^[a-zA-Z0-9]+$/, "is invalid"], maxlength: 250 },     jobTitle: { type: String, maxlength: 250 },     organization: { type: String},     description: { type: String},     address: { type: String },     category: { type: mongoose.Schema.Types.ObjectId, ref: "ContactCategory" },     group: { type: mongoose.Schema.Types.ObjectId, ref: "ContactGroup" },     order: { type: Number, default: 0},     email: { type: [emailSchema], default: [] },     phone: {type: [phoneSchema], default: [] },     fax: { type: [faxSchema], default: [] },     url: { type: [urlSchema], default: [] },     timestamp: { type: Date, default: Date.now } }, {collection: "contact"});
The email, phone, and fax schemas are defined in a separate file and they just have a string property that contains the appropriate information.
The category and group are working fine as foreign keys, but I'm only having problems with the subdocuments

sorry for a late response - will try to debug that this week and let you know

Hi all. Was this resolved? I have the same issue, any help is welcome. Thanks in advance!

we are making lots of improvements/bug fixes regarding nested arrays/schemas. So this should probably go away in v3.3 (currently in beta)


Replace use key with uses

This pull request the use key with uses, otherwise the following error will occur:
every step must define a uses or run key

Whoops! Thank you.

Perhaps also adding a checkout step may make the documentation easier to understand.
For example:
- uses: mxcl/xcodebuild@v1

Could be:
- uses: actions/checkout@v3
- uses: mxcl/xcodebuild@v1


Fix navbar width overflow

try to resolve #25221

(quote my comment from https://github.com/go-gitea/gitea/issues/25221#issuecomment-1588499035)
It's not related to navbar.
IMO actually is a regression of #21850
You can see I have remove the "navbar" and "footer", only "page-content" there:


Login into shell via kubctl : Release is not in Path

When you login via kubectl ( kubectl exec --tty --stdin pod - /bin/bash ) the release is not in path.

The Reason, why this is not working is, that /etc/profile requires a login shell. So we need to pass an extra argument --login to the /bin/bashcommand


GM: RSOD Cruise Faults when using the Resume button to engage

Describe the bug
I was getting RSOD Cruise Faults if I used the Resume button to engage the cruise control/OpenPilot when the previous set speed was below the current driving speed.
What hardware does this issue affect?
comma two
Which car does this affect?
GMC Acadia 2018
Provide a route where the issue occurs
e38a02cae1ef1e0c|2022-04-17--20-21-26--0
openpilot version
Master 0.8.14 @ 1fa1c44
Additional info
May be related to https://github.com/commaai/openpilot/issues/24223

Here is another short route with several Cruise Faults and a couple CAN Errors mixed in.
e38a02cae1ef1e0c|2022-04-19--17-23-33--0

Does https://github.com/commaai/openpilot/pull/24100 have anything to do with this?

Does https://github.com/commaai/opendbc/pull/591 have anything to do with this issue?

I heard that this is a possible fix.  https://github.com/commaai/openpilot/pull/908