Can not enable Dark Mode in Ferdi v5.4.4-beta.1 on Windows 10

complete new w10 and ferdi installation and i can not enable dark mode.. the button is just disabled..
i can however toggle the universal dark mode though that doesnt seem to do anything
Your Environment

Ferdi Version used: 5.4.4 B1
Operating System and version: W10

Regression introduced via https://github.com/getferdi/ferdi/commit/eefda6c7b3e9f0e9b0cfc056efbf59d04a54be85 at https://github.com/getferdi/ferdi/blob/dcc05d2e9e066ce0a64425b9b2c7772bf2add3aa/src/components/settings/settings/EditSettingsForm.js#L453

Removed the disabled={isAdaptableDarkModeEnabled} via 374b5219 which should fix this issue but not change the behaviour on macOS.

Shipped in v5.4.4-beta.2.


IT Failure: ITNotebookServiceClientTest failing with resource not found

July 29, 2023
July 30, 2023
July 31, 2023
The failure occurs before the native image compilation step. Snippet of stacktrace:
08:16:51:343 [INFO] -------------------------------------------------------
08:16:51:343 [INFO]  T E S T S
08:16:51:343 [INFO] -------------------------------------------------------
08:16:53:051 [INFO] Running com.google.cloud.notebooks.v1beta1.it.ITNotebookServiceClientTest
08:17:05:445 [ERROR] Tests run: 1, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 12.378 s <<< FAILURE! - in com.google.cloud.notebooks.v1beta1.it.ITNotebookServiceClientTest
08:17:05:446 [ERROR] com.google.cloud.notebooks.v1beta1.it.ITNotebookServiceClientTest  Time elapsed: 12.378 s  <<< FAILURE!
org.opentest4j.MultipleFailuresError:
Multiple Failures (2 failures)
	java.util.concurrent.ExecutionException: com.google.api.gax.rpc.InvalidArgumentException: io.grpc.StatusRuntimeException: INVALID_ARGUMENT: List constraint enforcement queries must contain a non-empty value to check.
	java.util.concurrent.ExecutionException: com.google.api.gax.rpc.NotFoundException: io.grpc.StatusRuntimeException: NOT_FOUND: Resource 'projects/cloud-java-ci-test/locations/us-central1-a/instances/it-test-notebook-instance-id-0328a4e9' was not found

It doesn't exist anymore.


Upload artifact on debug or tmpdir

If the tmpdir is set or debugging is enabled, the temporary folder will be added to the summary as an artifact.

Testing passed successfully.


v5.1.0-rc2 $id contains v5.0

https://github.com/CVEProject/cve-schema/blob/2aa608b6733cc2730a43901472ef0e706d0ef2b5/schema/v5.0/CVE_JSON_5.1_schema.json#L3-L5
Including "v5.0" as part of the $id field may be confusing to some people because version 5.1.0 doesn't have any direct relationship to "v5.0" material. There had been a proposal in which 5.1.0 would be considered part of a "5.0 family" of schemas (in which dataVersion could be "5.0") but this proposal was already rejected, and there is no other advertised context in which 5.1.0 is considered one of the 5.0 schemas.
The term "CVE JSON" is no longer used by the CVE Program, and thus replacing "CVE_JSON_bundled" with something like "CVE_Record_Format_bundled" may be consistent with the Program's direction. Simlarly, the description (line 5 above) would ideally state "specifies the CVE Record Format" and "This CVE Record Format."
The filename "CVE_JSON_5.1_schema.json" may be confusing because no schema is being shipped with the number "5.1" nor is "5.1" an intended dataVersion value. A filename of "CVE_JSON_5.1.0_schema.json" may be more clear.
The $id URL of https://cveproject.github.io/cve-schema/schema/v5.0/docs/CVE_JSON_bundled.json currently results in a 404 error, presumably implying that nobody is relying on that yet. It thus might be safer to make any needed changes now, rather than after that CVE_JSON_bundled.json file exists in the cve-schema master branch.

We later moved the schema dataversion to 5.1. We will move the dataversion to 5.1.1 in the near future to match the current version of the CVE Record Format schema. The $id url is now "https://cveproject.github.io/cve-schema/schema/CVE_Record_Format.json" and it resolves with the new name. Closing this issue since it's referring to the older version issues and is resolved.


dio does not automatically convert the model to json

Package
dio
Version
5.0.1
Output of flutter doctor -v
❯ flutter doctor -v
[✓] Flutter (Channel stable, 3.7.3, on macOS 12.6.3 21G419 darwin-x64, locale zh-Hans-CN)
    • Flutter version 3.7.3 on channel stable at /usr/local/Caskroom/flutter/3.7.0/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision<PHONE_NUMBER> (4 weeks ago), 2023-02-08 15:46:04 -0800
    • Engine revision 248290d6d5
    • Dart version 2.19.2
    • DevTools version 2.20.1
    • Pub download mirror https://pub.flutter-io.cn
    • Flutter download mirror https://storage.flutter-io.cn

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
    • Android SDK at /Users/ken/Library/Android/sdk
    • Platform android-33, build-tools 33.0.1
    • ANDROID_HOME = /Users/ken/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14C18
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)

[✓] VS Code (version 1.76.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.60.0

[✓] Connected device (3 available)
    • Redmi 5 Plus (mobile) • 7ade323c0804 • android-arm64  • Android 8.1.0 (API 27)
    • macOS (desktop)       • macos        • darwin-x64     • macOS 12.6.3 21G419 darwin-x64
    • Chrome (web)          • chrome       • web-javascript • Google Chrome 110.0.5481.177

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

Dart Version
2.19.2
Steps to Reproduce
@freezed
class Model with _$Model {
 factory Model({
  required String name,
  required String address,
 }) = _Model;
}

final model = Model(name: 'jack', address: 'xxxx');
dio.post('http://api.myhost.com', data: model)

Expected Result
dio can post data as json format
Actual Result
dio throw a error

We cannot parse classes from unknown types, and this is not a bug but rather a feature request against freezed. Please use explicit toJson when parsing your models.

Hi @AlexV525
the post success if I add header manually
dio.post(xxx, options: { headers: { 'Content-Type': 'application/json' }}

the defaultTransform automatically calls toJson, The documentation also says that data can be any data
So is there a change in 5.0 here?

And you'll need a toJson to your model, most possibly with json_serializable, which I didn't find in your model.

The default content type is no longer implied since 5.0 but it'll be inferred from the type of your data. It'll inform you if your data type is not recognizable.

Hi @AlexV525
All right, so let's change it to an explicit toJson() function call
Best regards


⚠ Secret Website 3 has degraded performance

In 2111b94, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 3688 ms

Resolved: Secret Website 3 performance has improved in c8215e2.


[Flake] api-tests: OrgRepository ListOrgs when we filter for guids org1 and org3 [It] returns just those

The test is creating three orgs with their names prefixed by org1, org2, org3. Then the test is listing orgs providing the names of the first and the last orgs. However, when verifying the results, the test would assert on the object themselves, rather than on org name. In the example below, the expectation does not match the actual by a second mismatch in CreatedAt and UpdatedAt
• [FAILED] [0.162 seconds]
OrgRepository ListOrgs when we filter for guids org1 and org3 [It] returns just those
/tmp/build/e45b3164/root/api/repositories/org_repository_test.go:279

  Expected
      <[]repositories.OrgRecord | len:2, cap:2>: [
          {
              Name: "org2-207d7e80",
              GUID: "7a274e74-2d09-493b-a088-b236a1674c0e",
              Suspended: false,
              Labels: nil,
              Annotations: nil,
              CreatedAt: "2022-08-30T10:02:18Z",
              UpdatedAt: "2022-08-30T10:02:18Z",
          },
          {
              Name: "org1-1db40854",
              GUID: "b6ec7dc5-3e5e-43c6-9ca8-ac09d98e4b70",
              Suspended: false,
              Labels: nil,
              Annotations: nil,
              CreatedAt: "2022-08-30T10:02:17Z",
              UpdatedAt: "2022-08-30T10:02:18Z",
          },
      ]
  to consist of
      <[]repositories.OrgRecord | len:2, cap:2>: [
          {
              Name: "org1-1db40854",
              GUID: "b6ec7dc5-3e5e-43c6-9ca8-ac09d98e4b70",
              Suspended: false,
              Labels: nil,
              Annotations: nil,
              CreatedAt: "2022-08-30T10:02:17Z",
              UpdatedAt: "2022-08-30T10:02:17Z",
          },
          {
              Name: "org2-207d7e80",
              GUID: "7a274e74-2d09-493b-a088-b236a1674c0e",
              Suspended: false,
              Labels: nil,
              Annotations: nil,
              CreatedAt: "2022-08-30T10:02:18Z",
              UpdatedAt: "2022-08-30T10:02:18Z",
          },
      ]
  the missing elements were
      <[]repositories.OrgRecord | len:1, cap:1>: [
          {
              Name: "org1-1db40854",
              GUID: "b6ec7dc5-3e5e-43c6-9ca8-ac09d98e4b70",
              Suspended: false,
              Labels: nil,
              Annotations: nil,
              CreatedAt: "2022-08-30T10:02:17Z",
              UpdatedAt: "2022-08-30T10:02:17Z",
          },
      ]
  the extra elements were
      <[]repositories.OrgRecord | len:1, cap:1>: [
          {
              Name: "org1-1db40854",
              GUID: "b6ec7dc5-3e5e-43c6-9ca8-ac09d98e4b70",
              Suspended: false,
              Labels: nil,
              Annotations: nil,
              CreatedAt: "2022-08-30T10:02:17Z",
              UpdatedAt: "2022-08-30T10:02:18Z",
          },
      ]
  In [It] at: /tmp/build/e45b3164/root/api/repositories/org_repository_test.go:283

❯ flake-hunter "when we filter for guids org1 and org3"
+-------+----------------------------------+-----------------------------------------------------
| Ended | Job                              | Url
+-------+----------------------------------+-----------------------------------------------------
| 14m   | main/run-tests-main              | https://ci.korifi.cf-app.com//teams/main/pipelines/main/jobs/run-tests-main/builds/273
| 2d    | main/run-tests-periodic          | https://ci.korifi.cf-app.com//teams/main/pipelines/main/jobs/run-tests-periodic/builds/1957
| 2d    | main/run-tests-periodic          | https://ci.korifi.cf-app.com//teams/main/pipelines/main/jobs/run-tests-periodic/builds/1937
| 5d    | main/run-tests-main              | https://ci.korifi.cf-app.com//teams/main/pipelines/main/jobs/run-tests-main/builds/265
| 5d    | main/run-tests-main              | https://ci.korifi.cf-app.com//teams/main/pipelines/main/jobs/run-tests-main/builds/264
+-------+----------------------------------+-----------------------------------------------------

I ran into this independently in one of my PRs. I think this commit fixes it:
https://github.com/cloudfoundry/korifi/commit/70cd68b8c005f49aa964c19538995e16272ad491
After that change I ran ginkgo with --until-it-fails and didn't see it fail again after 25+ attempts.


Make email field updatable

Updating email should mark the account as unverified subsequently require verification.

can you send here a link of the code here
i wanna help you on this project
if you can tell me from where can I start to help you

@stevensamy27, thanks for your desire to help. To get started working on the issue, you can override update(...) method in ProfileSerializer. Then before an actual update is done, you can compare the value of the email retrieved from validated_data to value in the (existing) instance. If they are not the same then you could update the user.is_verified field to false when doing an update.
Relevant docs: https://www.django-rest-framework.org/api-guide/serializers/#saving-instances


[TASK] Improve readme a little

Just renders the compore require line as fixed-with text

Thanks :)


feat: Add andFinally functionality and tests

This PR is related to the earlier #536 which proposed an implementation for an andFinally method that will be called regardless of whether an earlier step contained an error or not. This would allow the caller to avoid awkward constructs which pass the same or similar callback to both map/mapErr or andThen/orElse.
The earlier implementation had some problems, and I believe the design still needed some considerations. Primarily, the earlier version did not allow for asynchronous cleanup logic which is a common use case for resources such as database connections, login sessions, etc.
The primary use case considered is one similar to the following. More broadly, the feature is designed to cover use cases that would be normally covered by the finally block in either synchronous or asynchronous code.

function createConnection(): ResultAsync<Connection, NetworkError | AuthenticationError> {
}
function doStuffWithConnection(connection: Connection): ResultAsync<void, NetworkError | BusinessLogicError> {
}
function terminateConnection(connection: Connection): ResultAsync<void, NetworkError> {
}

function main() {
    return createConnection().andThen(connection =>
        doStuffWithConnection(connection)
            // this will be called regardless of whether doStuffWithConnection errors or not
            .andFinally(connection => terminateConnection(connection))
    );
}

 Design decisions
This feature was designed explicitly to handle scenarios which would normally be covered by the finally block in code that synchronously throws errors or uses async/await. This allows similar thought patterns easily apply to this library, and such code to more easily be updated to use neverthrow.
Key decisions are outlined below with further context.

 1. The andFinally callback does not receive a parameter.

In the finally block, I'm not aware of any languages that allow the programmer to directly check if an error has occurred or not. So, I elected not to pass any information about the earlier result to the callback. If it's absolutely necessary, the user can always use mutable state declared in an above scope just like they can with finally. (I rarely see this in practice).
Additionally, if the user wants to add separate logic for ok/error cases, I argue that we already have many features to do so andThen, orElse, etc.
I consider this decision to be low-risk since we can always add support for a result parameter to the callback without being a breaking change.

2. ResultAsync.andFinally must return a Result/ResultAsync and `ok` values are ignored.

In a normal finally block, it is possible for errors to be thrown. This is especially true in cases where a resource such as a database connection/transaction is used and the cleanup logic is asynchronous.
The most straightforward way to handle this is to require andFinally to return a Result. It also allows re-use of other functions that return Result/ResultAsync during cleanup. Other approaches require some kind of "sentinel" value to indicate that no error occurred.
OK values are ignored. This does diverge from languages which allow a finally block to contain a return statement, which will override any return value from elsewhere in the function. However, I do not recall seeing this used in practice.

3. When andFinally returns an error, that error will propagate (and overwrite a previous error).

I believe this is the most intuitive way to handle errors from andFinally. Firstly, it maps exactly to how throw is handled inside finally.
If the user wants to ignore errors inside the callback, they are still free to explicitly ignore them, for example:
createConnection().andThen(connection =>
    connection.doStuff().andFinally(() =>
        connection.terminate().orElse((e) => {
            console.warn("Error terminating connection", e);
            return ok(undefined);
        })
    )
)

4. Result.andFinally can return only a synchronous Result and not a ResultAsync.

This decision was made partly to simplify the implementation and avoid the need for any overloads which can make it more difficult to ensure type-safe code.
In practice I would also expect that if a particular piece of cleanup logic is asynchronous, it is likely that the earlier steps' logic is also asynchronous. This would be the case for most use cases I can think of (database connections and other resource-based cleanup).
Low-risk since we can always add an overload that broadens the accepted types without breaking existing code.
If we feel it's important to continue adding support for ResultAsync in Result's methods, I propose that a better path forwards is to add a .asAsync() method to both Ok and Err that simply lifts the result into a ResultAsync. It's also trivial for the user to do the wrapping themselves.

5. ResultAsync.andFinally will call the cleanup function even if the internal promise rejects

Normally, we can make the assumption that ResultAsync's internal promise should only ever reject if someone is misusing the library (e.g. calling fromSafePromise or new ResultAsync with something that's not safe). I decided that the cleanup function should still be called in this case, but we still propagate the rejection.
My justification:

If an erroneous rejection occurs inside ResultAsync, it is indicative of a bug and if we don't run the cleanup function we are more likely to leave the whole application in a globally-broken state from the callers' perspective. (Resource leak, hanging database transaction, etc.). I believe the caller is more likely to want the cleanup to run.
We can well-define the contract of andFinally to include this feature and the caller will know it will happen.
If there are multiple andFinally blocks in the same chain, they will still all be called which is logically consistent.
The rejection is not silently absorbed but will instead propagate through the ResultAsync returned from andFinally, and the bug causing it to happen can still be discovered.

It is a little awkward that we can't do the same for synchronous Results since if an error is thrown during an earlier step, the result whose andFinally method we are calling will never exist. A problem I can see is that the contract for Result/ResultAsync differs in this one aspect.

@mattpocock @supermacro @m-shaka
This functionality has been discussed at a few points in the past, I really appreciate any comments on this implementation. Or if it simply isn't a priority for the library, please let me know! Eager to hear any feedback.
I'm ready to use this in my own applications, but I'd prefer to settle on a particular interface before doing that (instead of e.g. forking off).


Make "getRecordedLogs" and "accesses" view

I think that these two cheatcodes should be of mutability view, based on the logic in the comments at the top of the file.

As a data point I did not make them view because they do change internal state -- they enable calling their respective log/access record functions without causing an error. So strictly speaking, they do change the context! In practice it may be worth having them be view though.

Let's leave these as mutable for the rationale @adhusson gave above + convo in https://github.com/foundry-rs/forge-std/pull/250. This rationale is explained in Vm.sol in the comment:

If you can influence a later call's observable behaviour, you're neither view nor pure (you are modifying some state be it the EVM, interpreter, filesystem, etc),

Calling e.g. getRecordedLogs clears the recorded logs, so calling it twice without a recordLogs in between results in different return data from the two calls, which is why it's not view
Def open to rephrasing for more clarity if needed

In this case, renaming this cheatcode to something like clearAndGetRecordedLogs might be helpful.


Add tests

Need to figure out how Hubitat driver code can be tested automatically.

See https://github.com/biocomp/hubitat_ci


No sense to restrict menu Json output

There is no sense to restrict json input to menu name "admin" only

While I don't necessarily disagree, could you please describe the reasoning behind this change?

I needed to create new admin menus (with different names off course) and wanted them as JSON output, my local solution was to override Voyager Menu Model.
But the question should be, what is the reason behind the restriction of JSON output only to menu named "admin"?
Regards


Improve CNCF Kubernetes Test Coverage

Coverage should be more than 90 % for operator/trigger/hook
Make sure no test is skipped.

since we deprecated it. it should not require now


IPL Dataset

Added ipl dataset of year 2017 and 2018.

Hey can you check my pull request?

Hey can you check my pull request?

Cool.. we'll merge it, tho' matches.md needs to be added in to the dataset-defination branch i.e. https://github.com/kishanpython/IPL-Analysis/tree/dataset-defination
Let's know if there's any problem that you face.

@all-contributors please add @kishanpython for data

@all-contributors please add @kishanpython  for data


task fails with 'docker is not a docker command'

Hi - I just tried a very simple test task using the plugin (downloaded the release jar) and while it does load, and the option appears in the task menu, the job once started just hangs indefinitely. I'm tailing the logs on both server and agent - nothing. Any hints on how to debug further?The docker service itself is running on the agent (Centos 7), the 'go' user is in the right group to run docker commands, I use a local repository from which I did test pull/build/push.

Fiddled with the settings until I did get some output in the logs, namely
2015-09-08 04:18:09,194 [loopThread] ERROR go.domain.builder.Builder:110 - Interaction with plugin with id 'docker-task' implementing 'task' extension failed while requesting for 'execute'. Reason: [null]
...
Caused by: java.lang.RuntimeException        at com.thoughtworks.go.plugin.infra.FelixGoPluginOSGiFramework.executeActionOnTheService(FelixGoPluginOSGiFramework.java:317)
...
Caused by: java.lang.StackOverflowError
at com.google.gson.stream.JsonWriter.string(JsonWriter.java:534)
...
And the task output in Go is
04:18:09.120 [go] Start to execute task: Plugin with ID: docker-task.
04:18:09.147 docker: 'docker' is not a docker command.
^^^ this is odd
04:18:09.148 See 'docker --help'.
04:18:09.171 dial unix /var/run/docker.sock: permission denied. Are you trying to connect to a TLS-enabled daemon without TLS?
04:18:09.171 Error: failed to remove images: [docker-task:8]
04:18:09.194 Error: Interaction with plugin with id 'docker-task' implementing 'task' extension failed while requesting for 'execute'. Reason: [null]
04:18:09.218 [go] Current job status: failed.

also about /var/run/docker.sock: permission denied - the user 'go' does have privs to run/build images, needed a service restart. the error I have now is
08:37:41.370 docker: 'docker' is not a docker command.
08:37:41.371 See 'docker --help'.
08:37:41.402 Error response from daemon: No such image: docker-task:10
08:37:41.403 Error: failed to remove images: [docker-task:10]
08:37:41.442 Error: Interaction with plugin with id 'docker-task' implementing 'task' extension failed while requesting for 'execute'. Reason: [null]

@rgcosma - thanks for the detailed analysis. There was a silly bug that introduced the command as docker docker. That was fixed in this - https://github.com/manojlds/gocd-docker/commit/b9ee3b46fdd33e3cdb6a7e02e26793a5f05034ca
I haven't looked at this in a while, I will run some sanity tests and upload a new build.

Hi, i have the same mistake,  do you know when your next release will be uploaded ?

any news on this ???

Hi All,
Do we know of any updates to this app? Is it supported at all?
Please let me know.
bf.

Hi @bf16574403  - I am the author of this plugin, and I haven't been looking at it recently. But will look into any issues you might have, and will also release a new version soon. Thanks!

Hi manojlds  , good to hear from you !!
I am keen to get in touch with you really need this app updated and support for a private registry.
Let me know how to get in touch.
bf.

It took a while, but I finally got around to testing the plugin and creating a new release - https://github.com/manojlds/gocd-docker/releases/tag/0.1.27


Updated refresher spec and yml file for resources in sponsored account.

Since we cannot transfer resources across an AD in Azure, we had to recreate the resources ourselves, which is at least partially done now. That means we have to update the refresher spec since the resources and resource counts have changed.
This is the start of that update. However I need help from @bzwei and @bronaghs to update the template and orchestration stack pieces, which I'm not that familiar with.
Also, I am seeing the OrchestrationStackResource.count changing between test runs, so there a possible bug there.
Folks running these tests should also add a subscription_id to their secrets.yml file.

Update: because of the subscription changes we made, the cloud manager specs needed to be updated as well. I took the opportunity to refactor them a bit, using the config/secrets approach there as well.
@bronaghs @bzwei @blomquisg Please review.

looks good to me :+1:

@djberg96 - LGTM , What was the fix for the constantly changing stack counts?

The resource count is temporarily removed. We will put it back after how we extract resources is stabilized.

@bronaghs the "fix" was to remove it. To paraphrase @bzwei, there is no clear way to get resources generated by the template. We are relying on the operations to conclude the list of resources, which is not reliable. The same resource can appear multiple times in the operations, and the resource may not be a newly created one.
So, we'll revisit this later once we have a more robust way of checking the count.

@djberg96, @bzwei  - it is a shortfall of the ARM API?

@bronaghs The list of resources is available in other cloud providers. ARM API does not directly provide such list.

@bzwei @djberg96 - thanks for the explanations.


Fix/test endpoint

:tada: This PR is included in version 9.0.4-rc.1 :tada:
The release is available on:

GitHub release
npm package (@next dist-tag)

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 9.0.4 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Simplify ProblemsListener structure

By having ConfigurationCacheProblems directly handle onProblem notifications and decoupling problem "providers" from the concrete implementation of ProblemsListener.

@bot-gradle test this

OK, I've already triggered ReadyForMerge build for you.


Defibrillator with no access tag do not appear on mobile

This is the same bug than https://github.com/openstreetmap-polska/openaedmap-frontend/issues/74.
It was closed, but the bug still remains when accessing openaedmap on a mobile phone

defib with access=no are grey on mobile instead of red. Maybe it's the cause of the bug

defib with access=no are grey on mobile instead of red. Maybe it's the cause of the bug

Hey, please verify. It should be fixed with commit https://github.com/openstreetmap-polska/openaedmap-frontend/commit/fa836807b56e68d2542fc0a8a6eb37a6ca5312dd
For me it works well (icons are red) on mobile:

Indeed, it works on the dev platform. Thank you for your work !


Feature/crud dynamodb

deploy dynamodb table with serverless
implement dynamodb functionality for sneaker management endpoints
update openapi docs
convert httpApi to rest api inside serverless
split each endpoint into its own lambda and split cloudwatch logs


mongo or any db

plz add to project db, mongo or any sql db

Maybe I will develop the project, but there is no time yet. Sorry.
If there is a need, then create P&R and I will accept it with joy


log os and arch

Log os and arch for better info in tickets

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 6 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 60.453%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/sync/sync.go
0
6
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.2%

Covered Lines:
801

Relevant Lines:
1325

💛  - Coveralls


request translations for autofill updates

Task/Issue URL: https://app.asana.com/0/1203822806345703/1208634215033039/f
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#5199  👈
#5198
develop

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

Join @cmonfortep and the rest of your teammates on  Graphite


Feature/zach event handlers milestone4

I had forgotten to open this branch sooner and tried pushing to the milestone 4 branch. Everything should be good though. This is mostly css, I did add an event handler to the dashboard so that searching would work using the search button that we have. It isn't working for me at this moment though, it just isn't connecting to the backend for me. I think that I just need to get it up and running but not sure how to do that right now.

Made the same mistake again and tried merging to master instead of milestone 4, closing this pr and opening another...


Allow to maintain the Item state on tab change

🙋 Feature Request
I would like to be able to keep the tab content state between tab changes.
Given:
A component with a RadioGroup as a tab content
Set the defaultValue of the RadioGroup  from the context (useContext)
Change the selected radio option in the group above
Change tab
Change tab back to the radio group one
🤔 Expected Behavior
RadioGroup state is maintained, and the default option is not selected
😯 Current Behavior
RadioGroup state is NOT maintained, and the default option IS selected
💁 Possible Solution
Add an option the keep the state of a tab item even when it is not displayed
🔦 Context
We are developing a UI component that can be opened with a context defined by the consumer.
The UI component state falls back to the context values on tab changes, whereas from UX point of view, UI state must be maintained.
💻 Examples
🧢 Your Company/Team
Adobe
🎁 Tracking Ticket (optional)

Seems like a good use case, but we can't change the current default behavior. We could look at adding a new optional prop that determines this behavior.
Maybe something like this:
tabMountBehavior?: 'active' | 'immediate' | 'lazy'

active (default): Mounts a tab when it is selected, then unmounts it when another tab gets selected. State inside a tab will not persist when switching tabs.

immediate: Immediately mounts all tabs. State inside a tab will persist when switching tabs.

lazy: Mounts a tab when it is selected, and doesn't unmount it when another tab gets selected. State inside a tab will persist when switching tabs.

I think the option should be given to the developer. Hoisting the state of a form with a lot of fields in a tab is a lot of work and we are not using a state manager unfortunately.

Hoisting shouldn't be that difficult if you use React context. And anyway, "a lot of work" is not a good enough reason for us to support this. There are many considerations and side effects of such a change as well, such as accessibility, keyboard focus behavior, memory usage, performance, etc. The future React Offscreen feature might help at some point.

Closing as not doing for React Spectrum v3. If you're using RAC, you should be able to use https://react-spectrum.adobe.com/react-aria/Tabs.html#tabpanel


Allow setting custom primary keys

@antonmarin Did you have time to try out this PR. Please let me know your thoughts. :)

Ping @antonmarin.

sorry, not able to test it. is there some kind of doc to understand how to use it?


feat: add form message UI

Addresses https://sendbird.atlassian.net/browse/AC-459

Design Guide

Happy path

https://github.com/sendbird/chat-ai-widget/assets/10060731/25ffc3ab-339a-42ad-9b32-14ad067b9ccf

When any of required fields is missing

https://github.com/sendbird/chat-ai-widget/assets/10060731/e2ef7c4d-762b-4676-b86f-1d3f174d9ee8

@liamcho I think I went through all your comments and addressed them if possible. Please check again and let me know if I missed something.

I'll merge this after Chuseok.


Text Extractor OCR mistake

Microsoft PowerToys version
0.63.0
Running as admin

[ ] Yes

Area(s) with issue?
TextExtractor
Steps to reproduce
Used the text extractor on this snippet:

✔ Expected Behavior
Clipboard with text
KRI /V. P. Chechev, N. K. Kuzmenko
LNE - LNHB/CEA - Table de Radionucléides
❌ Actual Behavior
Clipboard with text
KRI /V. P. Chechev, N. K. Kuzmenko
LIVE - LNHB/CEA - Table de Radionucléides
Problem: Poor detection of clear given character "N", mistaken as "IV".
Other Software
No response

Indeed, the recognition isn't perfect. This is partly expected, that's why it's always needed to double-check.

Related: #20899


NetworkManagementClient not in azure.mgmt.network.network_management_client

The azure_rm.py script on the ansible project is broken with the same message I reproduced in the following lines.
I've installed the network package like this: pip install azure-mgmt-network==0.20.0, then using ipython when I do:
In [1]: from azure.mgmt.network.network_management_client import NetworkManagementClient
---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
<ipython-input-1-cd3844722fc4> in <module>()
----> 1 from azure.mgmt.network.network_management_client import NetworkManagementClient

ImportError: No module named network_management_client

So I went on the source code looking for that network_management_client and found out:
$ git clone https://github.com/Azure/azure-sdk-for-python
$ cd azure-sdk-for-python/azure-mgmt-network/azure
$ ipython
[...]
In [1]: from mgmt.network.v2017_03_01.network_management_client import NetworkManagementClient

and I was able to load it.
Not sure how the build/release procedure is here, but that NetworkManagementClient seems to be in the repo and somehow being removed when the package is released.
Cf. also: https://github.com/ansible/ansible/issues/24936
$ python --version
Python 2.7.13
$ pip --version
pip 9.0.1 from /usr/lib/python2.7/dist-packages (python 2.7)
$ cat /etc/debian_version # using the test repository for deb packages
8.7

Hi @tappoz
The documentation of Ansible is asking for azure 2.0.0rc5, meaning network 0.30.0rc5. We have an open PR at the Ansible level to support network 1.x.
The correct import for both 0.30.x version and 1.x version is:
from azure.mgmt.network import NetworkManagementClient

Look to at network tutorials using this import, for instance https://github.com/Azure-Samples/network-python-manage-application-gateway
Thank you,


Dataset.T property creates conflict with data named 'T'

I frequently find myself working with datasets that include a variable named 'T' (for temperature!)
The problem with this is that, unlike any other variable, mydataset.T does not return the variable, because there is a built-in property .T which is just a map to the mydataset.transpose() method.
This is mostly just a minor annoyance, because of course I can access the data as mydataset['T'], but it necessitates an odd asymmetry in any code that accesses T as well as other data. And, maybe more importantly, mydataset.T does NOT immediately raise an error in code that is attempting to access the temperature data.
I have not been able to simply remove the property interactively from the dataset objects, e.g.
del mydataset.T
raises an error
AttributeError: can't delete attribute
Since the transpose functionality is already easily accessible through mydataset.transpose() (which is more readable, in my opinion), perhaps the T property should just be removed?

Removing .T would break so much existing code, I don't think that's going to get much support. It's also consistent with numpy.
If you use IPython, tab-complete will work, so if your column is called temperature, you can access it almost as quickly...

Agreed, I don't think we want to get rid of the transpose method T.  A better solution for you would probably be to rename your temperature variable using the rename method.  I often do this when comparing multiple datasets that use different variable naming conventions.

Brilliant, somehow I missed the rename method in the documentation. Also, I missed the fact that .T is consistent with numpy. I will just make a habit of renaming all my variables called T from now on. Thanks for these comments.


Lumira Designer 2.2 SDK Simpledate

Hi
We are running Lumira Designer 2.2. and we have some problem using simpledate like we used in design Studio. The report are stucked in prompt screen when adding the simpledate function. Do you have any idea why?
I see also the Application. getInfo(); isn't a possible choice in the scriping screen.
Please let me know if this will be fixed or do you have any other solutions to use the flexibility of the date- function.
Kind regards
Per Christian Naess

Hi
What are you doing with the simpledate component? I have lumira 2.2 and have added it to an app. The app loads but i am not utilising it.
I also have application.getinfo() available for use in the script editor.


Bug: Files installation failed

Description
I tried to install Files through the classic installer. I visited this GitHub Repo README, and simple clicked on the "Classic Installer" button, which lead me to download Files.stable.appinstaller. When attempting to install Files, the installation fails, and gives me the following output:
Files_<IP_ADDRESS>_x64__1y0xx7n9077q4 because this package depends on a framework that could not be found. Provide the framework "Microsoft.WindowsAppRuntime.1.5" published by "CN=Microsoft Corporation, O=Microsoft Corporation, L=Redmond, S=Washington, C=US", with neutral or x64 processor architecture and minimum version 5001.178.1908.0, along with this package to install. The frameworks with name "Microsoft.WindowsAppRuntime.1.5" currently installed are: {Microsoft.WindowsAppRuntime.1.5_5001.275.500.0_x86__8wekyb3d8bbwe Microsoft.WindowsAppRuntime.1.5_5001.275.500.0_x64__8wekyb3d8bbwe} (0x80073cf3)

Steps To Reproduce

Download the Files app directly from the GitHub Repo by clicking the "Classic Installer" button on the GitHub README.
Opened the file, and attempt to install it.
It fails and I get an error message.

Requirements
I expected the installation to happen successfully but it didn't work.
Files Version
Unsure, <IP_ADDRESS>
Windows Version
Edition	Windows 11 Pro Version	21H2 Installed on	15/10/2024 OS build	22000.2538 Experience	Windows Feature Experience Pack 1000.22001.1000.0
User ID
No response
Log File
Unable to access the log files as I cannot even open the Files app, the installation fails. The error message is stated above.

Please try removing existing installations of WinAppSdk from the Settings app. It's possible that one of them is in a bad state.

Please try removing existing installations of WinAppSdk from the Settings app. It's possible that one of them is in a bad state.

How should I do this

If you have already tried the PowerShell method I said above. Open Windows Settings > Apps > Installed apps > Search Windows app runtime > Click ... > Uninstall for all them

If you have already tried the PowerShell method I said above. Open Windows Settings > Apps > Installed apps > Search Windows app runtime > Click ... > Uninstall for all them

ok thanks. I tried that and the files installation succeeded. Thanks for the help.


Fixed rollup sourcemap warnings

Coverage: 93.713%. Remained the same when pulling f67180b040a31d492eb57e01eb4316f56239695a on cody-sourcemap-warnings into a97e9f1aa83676354b9d1f4b77be1b66eff91d83 on main.


WIP: bos-tasks-k38b6: Update translations from Fedora Zanata

Update translations from Fedora Zanata

[ ] po-refresh

po-refresh in progress on bos-tasks-k38b6.
Log: http://fedorapeople.org/groups/cockpit/logs/po-refresh-451-20181126-073636/


What should I do if the advertising size exceeds 31 bytes ?

If I use 128-bit service class UUIDs, the payload size will be large and there will be little room to store other items.
Now when the payload exceeds 31 bytes, it will abort as follows:
CRIT NimBLEAdvertising: "error setting advertisement data; rc=4, "

If I use uuid128, specify Appearance, the local name can only use up to 4 characters.
What should I do?

You'll need to move the excess data to the scan response data. Keep in mind each individual data type has 2 extra bytes added for the type and length indicators.

In NimBLEAdvertising::start, it calculates the payload length and decide whether to rsp before calling ble_gap_adv_rsp_set_fields, but I feel that it is because I only consider the size of the address book and local name.
I'm trying to fix NimBLEAdvertising::start to take into account the length of other items, is this approach correct?

Using the NimBLEAdvertising::addServiceUUID parameter was only intended for basic uses, when the more "advanced" uses are needed the NimBLEAdvertisementData class should be used with the corresponding NimBLEAdvertising::setAdvertisementData and NimBLEAdvertising::setScanResponseData calls.
If I was to write this library without basing it on the original I would not have implemented the addserviceUUID method (it was a pain to get working) and just had the NimBLEAdvertisementData class. It was left in to make things easier for transition.

I understood the situation.
However, if a user uses uuid128, they will most likely end up in a situation where they must use the RSP.
So my suggestion is that it's a pain to find the payload for all items.
As a temporary measure, why not always advertise the device name in RSP when m_scanResp == true ?
It doesn't matter if m_scanResp == true and uuid128.

It should do that already. If there isn't room in the ADV packet for the UUID and the device name NimBLEAdvertising::start moves the name into the RSP data (or at least it should). If that is not working then there is a bug I have not seen. The only time that would not be the case is when using custom data.
Can you share a code snippet to reproduce this?

#include <NimBLEDevice.h>

void setup() {
  BLEDevice::init("12345");
  /*
   * Conditions to abort
   * OK "1234"      // RSP not required
   * NG "12345"     // have to use RSP but not trying to use it
   * NG "123456"    // have to use RSP but not trying to use it
   * NG "1234567"   // have to use RSP but not trying to use it
   * NG "12345678"  // have to use RSP but not trying to use it
   * OK "123456789" // use RSP
   */

  auto pServer = BLEDevice::createServer();
  auto pService = pServer->createService("550e8400-e29b-41d4-a716-446655440000"); // dummy
  pService->start();

  auto pAdvertising = pServer->getAdvertising();
  pAdvertising->addServiceUUID(pService->getUUID());
  pAdvertising->setAppearance(0x00);
  pAdvertising->setScanResponse(true);
  pAdvertising->start();
}

void loop() {}

After setting the priority to be used, how about storing it in m_advData as follows ?

flags
appearance
tx_pwr_lvl
BLE_UUID (s)
name

So what about interrupting when it exceeds 31 bytes and moving only the name to the m_advData when it exceeds?

Good catch with the appearance bytes not being calculated in the payload, thanks!
I left a comment on your PR and I will note it here as well. If we prioritize tx_pwr_lvl over the name, we will be limiting the name length by another 3 bytes which may cause issues for those not using scan data and already near the edge of full advertisement payload.
If we were to change it in this way there should be an option to disable tx_pwr_lvl advertising.


when error, cannot print success logs

Signed-off-by: gy95<EMAIL_ADDRESS>when update secrets failed, can't print success related logs: "success update [%s] addr %v to secret", constants.ServerAddrName, t.Host.Addrs()
So need return the function

/lgtm

/assign @fisherxu


Netstandard Version

Any chance you could make a netstandard nuget. If ti was netstandard 2.0 it should basically support everything and run on 4.6.1 and dotnet core. It may also work on a lesser version of netstandard as well i am not sure.

it seems someone had done that in fork jack-burgess/TokenBucket and i don't think functionality was lost like the other persons attempt. It would be great to see an published net standard version so it can be used in net standard projects.

Better late than never, as this repo appears unsupported I have forked/published the library to support .NET Standard 2.0 and .NET 5.0


Simulate transaction feature similar to invokeContract for all types of transactions

Task description
It would be good to have a simulate transaction/feature in the webSDK, so a front-end can first call to simulate a specific transaction to see if it would revert (e.g. because insufficient funds in the wallet or because of a contract revert) and only if the simulation was successful, allow the user the execution of the real transaction on the front end. It would be similar to the invokeContract call (which can simulate Transaction.Update transactions). This request is to expand the simulation to any type of transaction e.g. deploy/initilize transactions as well.
Add label for component and priority.

This would need node support to implement properly.
We would add an API call like invoke_instance we have now that would try to make a block with a (list of) transactions, and report on the results.


Extrusion Multiplier filament data field

Describe the solution you'd like
I would really like to see a data field for filament specs that allows inclusion of the extrusion multiplier. Since the extrusion multiplier is a function of ONLY the filament specification it makes sense to add.
Where as pressure advance / linear advance is a function of both the extruder and the filament, that would not make as much sense.

I have now added the ability to make custom extra fields. Please test it out in the beta for me before I make a release. See #283 for more information.


Jasmina_Wwise bg+sfx

YarnMerge, CatPurr when it gets a yarn, Pause Music(same as ingame only without melody), Gameover

Still fixing branch, will need to update this to use new In-Game level

Fast forward changes from develop. All sound changes seem to be working correctly


Tests should not be installed

Python module should not install tests in site-packages/tests. This patch fixes the problem:
--- setup.py.orig	2023-08-22 07:19:18.000000000 +0000
+++ setup.py
@@ -45,7 +45,7 @@ def _get_version():
 _VERSION = _get_version()

 # Library packages.
-_PACKAGES = find_packages()
+_PACKAGES = find_packages(exclude=['tests*'])

 # User readme.
 _README = _read("README.rst")

@moubctez Thank you for highlighting this.  Just pushed your suggested patch.  Probably moving to poetry fairly soon so things will change a little anyway, but regardless thx once again.


Citations by count

Hej,
There is the scenario of reducing the word count by removing little used references. I use the tool quite often to count references for this purpose, and then I have to look in the JSON for items cited once or twice.
I suggest including an option to sort references by cite count into the bibliography to make this easier and accessible for my non tech-savvy colleagues.
In this fork, the citation count is added in front of each item in the APA bibliography and it is sorted ascendingly, so that little used items are on top of the list.
Technically, I transferred the count into APA by moving it from the notes to the title, then once the bibliography is rendered to the front. To not destroy the usability of the CSL JSON, this all happens in the convertOutput-function by parsing and then re-stringifying the JSON.
It works well locally on two examples. It would only fail if the tick box for counting is not pressed, or when an item has no title.
I hope you can consider it for merging and including in the website.
With best wishes,
Malte

@mbroedl, thanks, interesting idea. Maybe it would be handier to provide a CSV or TSV output format option for more robust sorting/analysis? This could be as simple as a two-column file with the citation count in one column and the formatted citation in the other.

@rmzelle Excellent idea! I've implemented and troubleshooted this now. The text field (and export) is now a two-column tsv with cite_count and reference as the column headers.

Hi @mbroedl, I adjusted your code a little and removed the "Store cite counts" toggle in favor of more output options, which I think makes the tool simpler to use. I think it works but I'd welcome any testing and/or code review.

Hej @rmzelle — It looks good and feels much more intuitive now!
Made two small changes:

fixing a typo so that the output file for the TSV is actually tsv.
removing empty note fields so that they don't pollute bibtex, RIS, and no-cite-count json

Looks good to me now!

Thanks so much for the PR, and having patience :).

Thank you for being so kind and responsive. :)


Docker image cannot be pulled down

Docker image cannot be pulled down：
Hello, I encountered an issue with the dropdown image while using your warehouse：

Docker image cannot be pulled down： Hello, I encountered an issue with the dropdown image while using your warehouse：

I think it‘s network issue. There's a copy on aliyun:
docker pull crpi-7p1lmijmzs6f2s62.cn-hangzhou.personal.cr.aliyuncs.com/clearhanhui/ubuntu-libtorch-{cpu/cuda}
docker run -it  crpi-7p1lmijmzs6f2s62.cn-hangzhou.personal.cr.aliyuncs.com/clearhanhui/ubuntu-libtorch-{cpu/cuda}:latest


Fix duplicated text in docstring

I noticed this is resulting in a duplicated description on the PyPI listing at https://pypi.org/project/hathi/

Thanks Simon!


[SPARK-12133][STREAMING] Streaming dynamic allocation

What changes were proposed in this pull request?
Added a new Executor Allocation Manager for the Streaming scheduler for doing Streaming Dynamic Allocation.
How was this patch tested
Unit tests, and cluster tests.

@andrewor14

12133

Test build #54893 has started for PR 12154 at commit 81ad1dd.

Test build #54905 has started for PR 12154 at commit 0c6d94b.

Test build #54893 has finished for PR 12154 at commit 81ad1dd.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54893/
Test FAILed.

Merged build finished. Test FAILed.

Test build #54905 has finished for PR 12154 at commit 0c6d94b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54905/
Test PASSed.

Test build #2750 has started for PR 12154 at commit 0c6d94b.

Test build #2750 has finished for PR 12154 at commit 0c6d94b.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

@tdas Looks great. I think you could add more comments in the code but the rest is pretty good.

@andrewor14 Updated. Please take a look.

Test build #2759 has started for PR 12154 at commit 3b501a0.

Test build #55136 has started for PR 12154 at commit 0598c85.

LGTM

Test build #55140 has started for PR 12154 at commit ce36c76.

Test build #55136 has finished for PR 12154 at commit 0598c85.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/55136/
Test PASSed.

Test build #55140 has finished for PR 12154 at commit ce36c76.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/55140/
Test FAILed.

Merged into master thanks.

@tdas or @andrewor14 does this depend on any 2.0 APIs, I would like to backport this to 1.5 or 1.6 if possible.
need to run multiple concurrent streaming jobs on mesos

@jayv big new features like this are never backported into older branches.

I understand that, but I want to port this feature to our internal custom 1.6 build, if it's not too much trouble.

I see. I don't believe this depends on new APIs. You may have some difficulty just backporting into 1.6 in general for big patches, however.

Is there a way to specify the Initial executors?

@tdas - Why we cannot see this in the documentation and I am not sure if AWS EMR supports this feature?


Added a simple handshake example.

Since it recently took me some time to figure out how to use rust-bitcoin, and the contribution of example code is explicitly mentioned in the README, I thought I'd start with a basic handshake example: the supplied code connects to a Bitcoin node, complete the initial version/verack handshake, before it disconnects again.
Let me know if I should change something. If it is desired, I'd consider adding more small examples, if I stumble upon something worth highlighting.

Fwiw, I've been working on https://github.com/stevenroose/rust-bitcoin-p2p/ for a while a few months back. I've been quite busy with work since, but I certainly intend to pick that back up!

Fwiw, I've been working on https://github.com/stevenroose/rust-bitcoin-p2p/ for a while a few months back. I've been quite busy with work since, but I certainly intend to pick that back up!

That's an interesting project! How mature is it?

I cleaned up the example a bit further. It also builds now via cargo build --examples.

Can you please rebase this on master to remove the unrelated commits?

Can you please rebase this on master to remove the unrelated commits?

Done!

Tested. looks awesome :)
cargo run --example handshake <IP_ADDRESS>:8333
Sent version message
Received version message: Version(VersionMessage { version: 70015, services: ServiceFlags(1037), timestamp:<PHONE_NUMBER>, receiver: Address {services: ServiceFlags(NONE), address: <IP_ADDRESS>, port: 36006}, sender: Address {services: ServiceFlags(NETWORK|BLOOM|WITNESS|NETWORK_LIMITED), address: <IP_ADDRESS>, port: 0}, nonce:<PHONE_NUMBER>945067093, user_agent: "/Satoshi:0.18.0/", start_height: 619975, relay: true })
Sent verack message
Received verack message: Verack

Last thing from me, can you please fix this for rust 1.22.0?
This diff seems to be enough:
diff --git a/examples/handshake.rs b/examples/handshake.rs
index e04470a..23022e1 100644
--- a/examples/handshake.rs
+++ b/examples/handshake.rs
@@ -2,11 +2,14 @@ extern crate bitcoin;

 use std::net::{IpAddr, Ipv4Addr, Shutdown, SocketAddr, TcpStream};
 use std::time::{SystemTime, UNIX_EPOCH};
-use std::{env, io::Write, process};
+use std::{env, process};
+use std::io::Write;

 use bitcoin::consensus::encode;
-use bitcoin::network::{address, constants, message, message_network, stream_reader::StreamReader};
+use bitcoin::network::{address, constants, message, message_network};
+use bitcoin::network::stream_reader::StreamReader;
 use bitcoin::secp256k1::rand::Rng;
+use bitcoin::secp256k1;

 fn main() {
     // This example establishes a connection to a Bitcoin node, sends the intial

Thanks! Updated, now builds under nightly and 1.22.0 environments.

Codecov Report

Merging #411 into master will increase coverage by 0.23%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #411      +/-   ##
==========================================
+ Coverage   85.81%   86.05%   +0.23%
==========================================
  Files          40       40
  Lines        7461     7715     +254
==========================================
+ Hits         6403     6639     +236
- Misses       1058     1076      +18

Impacted Files
Coverage Δ

src/util/base58.rs
84.1% <0%> (+0.32%)
:arrow_up:

src/consensus/encode.rs
90.46% <0%> (+0.46%)
:arrow_up:

src/util/amount.rs
89.74% <0%> (+0.75%)
:arrow_up:

Continue to review full report at Codecov.

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

Nice :) thanks!

What do you mean by 'recreate'?
FTR I get an EOF error when I try and run this example against a regtest node
cargo  run --example handshake --features=rand-std <IP_ADDRESS>:18444                                                                              ✘ 101
    Finished dev [unoptimized + debuginfo] target(s) in 0.02s
     Running `/home/tobin/.cache/cargo/debug/examples/handshake '<IP_ADDRESS>:18444'`
Sent version message
thread 'main' panicked at bitcoin/examples/handshake.rs:44:90:
called `Result::unwrap()` on an `Err` value: Io(Error { kind: UnexpectedEof, error: None })
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

I guess we should have some tests for examples too.


Mistake in function PCD_CommunicateWithPICC ?

In function PCD_CommunicateWithPICC(..) there is a check of the register ErrorReg
byte errorRegValue = PCD_ReadRegister(ErrorReg); // ErrorReg[7..0] bits are: WrErr TempErr reserved BufferOvfl CollErr CRCErr ParityErr ProtocolErr
 if (errorRegValue & 0x13) {	 // BufferOvfl ParityErr ProtocolErr
	return STATUS_ERROR;
}

But the documentation describes that the ProtocolErr bit is valid only for the command MFAuthent.

set to logic 1 if the SOF is incorrect
automatically cleared during receiver start-up phase
bit is only valid for 106 kBd
during the MFAuthent command, the ProtocolErr bit is set to logic 1 if > the number of bytes received in one data stream is incorrect

It may be necessary to fix the code on
byte mask = 0x12;
if (command == PCD_MFAuthent)
{
	mask |= 0x01;
}
if (errorRegValue & mask)
{

edit: highlightning

I think the documentation means, that the bit ProtocolErr is used for SOF error and MFAuthent error


metrics-exporter-prometheus: split off tokio and hyper functionality to another (default) feature

I threw this up relatively quickly, I can dive into it more tonight and improve documentation + ci (adding different features to actions).
What this PR does it make the Tokio and hyper functionality apart of a feature. It is enabled by default, but allows for users to replace disable it if they wish. The only major change is introducing another build function on the PrometheusBuilder that only returns the Recorder rather than both the Recorder and Exporter. This function is only enabled when the tokio-exporter feature is disabled.

@tobz Pushed up an implementation of the handle function to return a PrometheusHandle which is simply a type alias around Arc<Inner>. Curious on your thoughts on this way of doing it.
Also I can't promise my documentation comments are the best, but happy to have them ripped apart haha.

Okay I think I fixed all those!

Looks good to me!
I'll merge now and release a new alpha later tonight.

Released<EMAIL_ADDRESS>Thanks again for your contribution!


Developer should be able to validate and display Pyrsia details about an image

Suggestion details
We should have the CLI that takes the image tag or digest and validates it against Pyrsia returning the details that Pyrsia knows about the image.
Possilble CLI call:
pyrsia validate helloworld:1.0
helloworld:1.0 would be an image that the developer built that uses the official DockerHub python:3 base image.  The expected output would be data showing that python:3 layers were valid, signed etc from Pyrsia.  The other layers, developer ones, would show that they are not valid, signed etc.  Eventually, the developer layers would show valid and signed when we start building and pushing docker images in Pyrsia.
Additional details
No response

More details in the docker use case

Can we set a priority on the ticket when we should start working or may be milestone?


Update README language about beta version requirements

Requirements:

Xcode 9 Beta 5 or higher
An iDevice with an A9 (or greater) processor running iOS 11 Beta 5 or higher

Now that Xcode 9 and iOS 11 have been released, we just need those, right? Originally when I was evaluating whether to use this vs other libraries I was thrown off by the beta requirements and discounted it because it seemed incompatible with my timelines.

This repo was created before Xcode 9 and iOS 11 were released. I updated the readme in https://github.com/mapbox/mapbox-arkit-ios/commit/a73ff0cd84e43b501530740898fd3a7cf5400e5f. Please note the warning at the top of the readme:

Also, thanks for the report!


MTIImageView always displays images using scaleAspectFit

I'm displaying an image using a MTIImageView using the below configuration but it always shows using scaleAspectFit:
    private lazy var backgroundView: MTIImageView = {
        let view = MTIImageView()
        view.translatesAutoresizingMaskIntoConstraints = false
        view.isUserInteractionEnabled = false
        view.contentMode = .scaleAspectFill
        view.backgroundColor = .clear
        view.isOpaque = false
        view.clearColor = .init(red: 0, green: 0, blue: 0, alpha: 0)
        return view
    }()

Note that i am using iPadOS 17.0 beta so it could be a beta issue but i don't have a way of verifying this.
Checklist

[x] I've read the README
[x] If possible, I've reproduced the issue using the master branch of this repo
[x] I've searched for existing GitHub issues

Environment

Info
Value

MetalPetal Version
1.25.2

Integration Method
SwiftPM

Platform & Version
iPadOS 17.0 beta 3

Device
iPad Pro 2018 11"

Steps to Reproduce
Use above MTIImageView config.
Add constraints to pin the view to the superview.
Assign .image with MTIImage.
Expected behavior
For the MTIImage to fill the entire view.
Actual behavior
MTIImage scales to fit the view.

Is there a workaround for this at all? Converting my MTIImage to UIImage for use in an UIImageView that scales properly is not ideal.


feat(gitbrowse): add custom range formatter

Description
This PR defines new option
  ---@type table<string, fun(line_start: string, line_end: string): string?>
  range_formatters

that allows specifying a custom line range formatter for services that use a specific range format, e.g.
        range_formatters = {
          ["another%.git%.service"] = function(range_start, range_end)
            return range_end > range_start and range_start .. ("&lines-count=" .. range_end - range_start + 1) or ""
          end,
        },

I changed the code to expose line_start and line_end directly, which should cover your use-case.

@folke thank you,
I cannot figure out how interpolation would work here:
I'm trying to specify it as follows
file = "/files/{branch}/{file}?&line={line_start}"
  .. "{line_end} > {line_start} and ('&lines-count=' .. {line_end} - {line_start} + 1) or ''}",

but it doesn't work

Just added line_count, so you can use that directly


Linuxbrew does not install the software into /usr/local, behaviours not equals the document

Linuxbrew does not install the software into /usr/local, behaviours not equals the document
The real path it will installed software was $HOME/.linuxbrew/bin
And the document said:
The standard script installs Homebrew to /usr/local so that you don’t need sudo when you brew install. It is a careful script, it can be run even if you have stuff installed to /usr/local already. It tells you exactly what it will do before it does it too. And you have to confirm everything it will do before it starts.

 Which document gave you that?

https://github.com/Homebrew/linuxbrew/blob/master/share/doc/homebrew/Installation.md

Looks like that part of the manpage hasn't been updated to be Linuxbrew-specific.  Feel free to open a PR!

I am not sure if it can even be updated as it seems to be Homebrew directly and if I understood correctly, Linuxbrew is supposed to be merged to Homebrew sometimes.

I think at this point the merges are mostly one-way, with Shaun pulling in changes from main Homebrew.

Yes, that document is inaccurate for Linuxbrew. It could be patched. The Linuxbrew installation instructions are at
http://brew.sh/linuxbrew/


How to remove OK and cancel buttons on Datepicker?

There is no way. And we will not add such functionality. If you need so you can import directly <DatePicker /> component and make your own modal. (/DatePicker/DatePicker)

app.css :
.MuiDialogActions-root.css-knqc4i-MuiDialogActions-root {
display: none;
}


feat: cookbookbat cooks

Cookbookbat offers 5 free cooks per day.
Not too sure as to whether the ConcoctionDatabase changes are right: I don't understand what adventureSmithingLimit and the like do, and whether we need a corresponding one for cooking. Also everything that refers to Jewelrycrafting taking adventures can presumably be removed?

This probably wants tests for crafting order

I think this probably wants tests all over the place ;). It's just a lot of work to set up for the first time.
Plus ConcoctionDatabase wants a bit of cleanup after tests are in place.

Not having any luck making these tests. In ConcoctionTest:
@Test
public void smithingTakesAdventures() {
  var cleanups = new Cleanups(withItem(ItemPool.TENDER_HAMMER), withMeat(1000), withAdventuresLeft(5));

  ConcoctionDatabase.refreshConcoctions();

  var con = ConcoctionPool.get(ItemPool.MACE_OF_THE_TORTOISE);
  assertThat(con.getAdventuresNeeded(1), is(1));
}

This fails, and debugging says the character can't smith. I don't understand how or why.

Not having any luck making these tests. In ConcoctionTest:
var con = ConcoctionPool.get(ItemPool.MACE_OF_THE_TORTOISE);

This fails, and debugging says the character can't smith. I don't understand how or why.

Mace of the Tortoise is not creatable; it is given to you by the ghost of The Unknown Turtle Tamer.


IFD of Mask data handling in COG Data

(This issue comes from DGIWG Imagery and Gridded Data Panel review of COG Draft Standard - September 2022)
The "Requirement 3" clauses ("/req/req-class-geotiff-overviews/overviews") seems to imply that an IFD that is related to a full resolution data image SHALL point to a next IFD of a reduced resolution image (if there is any) in the COG File otherwise it should point to nothing.
If this is really the case (if we have well understood), if a COG file is also shipping a transparency mask image data along with the image data itself, how can the IFD related to the mask image data be reached? In other words where can the pointer to this IFD be placed?

Actually, you are right. The requirement was involuntarily written an a way that prevented more than image (or a mask) in the TIFF file. The language of requirement 3 is particularly complicated due to several possible alternatives. This has been fixed and a clarification note has been added.
Thanks for spotting this

No one complained for a while so I'm closing this issue

Thank you @joanma747 the proposed rewritting seems to solve this issue that was raised.


Request: A319-100 Pack livery

hey, after a350-1000, i made the a319 pack:
HiFly

ChAir

Sundair

Emirates Executive

AirSerbia (blank livery)

United Kingdom Royal Air Force

OneWorld (10 Years Livery)

Sorry, i made some liveries that maked (ChAir & Emirates) you can add as a alt livery, but i want the Embraer E195-E2 & ERJ-170 was added to multiliveries (+ some planes is not added in the maps: Concorde, BN2, B737-200, DC3 and the Dash 8).

I will look after the Embraers, but I'm not sure, that they are ls compatible. The maps will be created at some point, I will have more time to work on this in the summer :)


set client connection and receive timeout via property

Hi @arolfes - thanks so much for the PR.  We've had this topic of config properties for connect and receive timeouts come up in the past.  We thought it best to not include it as part of this extension, due to reasons including:

There are 25 or so properties that can be configured in HTTPClientPolicy.  Accounting for only two of them is not ideal, and sets a precedent for the rest of them to also be configurable through the extension properties.
Similarly, there are numerous configurations for HTTConduit.   Merging and/or overriding custom application specific HTTPConduit configurations with what we set via this extension can become challenging.
Given the points above, and as we have simple and documented ways to set the HTTConduit and HTTPClientPolicy on the CXF bus - we thought it best to leave connection related advanced configurations to the application developers.

I am still of the sentiment that we should not include these properties as part of the extension, but maybe let's keep this open for now to gauge community interest and get any other community feedback.

Hi @shumonsharif ,
I understand your points and I agree with it.
I close this PR.
Regards


Automate CLI release

The CLI is currently released manually. It would be better to release it automatically whenever a tag is pushed, probably using Cloud Build.

💯 for automating releases. But, let's use GitHub Actions or Travis CI -- much simpler.


Emulator data

Uses new json data structure from Shapeshifter repo (I'm still trying out the name 😎).
Replaced UserAgents.ts with user-agent-options.ts from Shapeshifter.
The big downside to this PR is it downgrades http headers to only include Document type.

 Maybe you made it too accurate??!?


AB2D-5727 Data Filtering for Humana

🎫 Ticket
https://jira.cms.gov/browse/AB2D-5727
🛠 Changes
Added extension section in AB2D-Libs/ExplanationOfBenefitTrimmerSTU3
ℹ Context for reviewers
One of our PDP enquired with us about some values that are under extension section in outgoing json for STU3
✅ Acceptance Validation
Tested manually on local environment
🔒 Security Implications

[ ] This PR adds a new software dependency or dependencies.
[ ] This PR modifies or invalidates one or more of our security controls.
[ ] This PR stores or transmits data that was not stored or transmitted before.
[ ] This PR requires additional review of its security implications for other reasons.

If any security implications apply, add Jason Ashbaugh (GitHub username: StewGoin) as a reviewer and do not merge this PR without his approval.

Hey @smirnovaae : Please do me a favor and add comments with ticket number for reference just in case or some info why we commented out

AB2D-5728
Validate Humana AB2D data dictionary for missing elements reported in STU3

Hey @smirnovaae : Please do me a favor and add comments with ticket number for reference just in case or some info why we commented out

done!


Make sure game quits when using game.sh

I noticed when using this for development, my game would not quit using the game.sh script. It seemed the trailing & was the culprit.
Unless it is used for some reason, I'd suggest to remove it. (the game.bat does not seem to have it)

Not sure why it was like that but removing it shouldn't hurt, thanks.

no problem! thanks to you for this awesome mavenized slick library - saved me a lot of trouble!


feat: Add Generator Icons in Android, IOS, and Web

Description
very_good_cli is very helpful for me when creating new Project Apps. But when you want to change the icon it is very complicated, unlike https://pub.dev/packages/flutter_launcher_icons.
Expected Behavior
Create a tool to generate automatic icons similar to https://pub.dev/packages/flutter_launcher_icons.
Maybe you can customize it according to this very_good_cli

This would be awesome, or at least show us how to currently change it. I have tried the https://pub.dev/packages/flutter_launcher_icons package but it does not work properly as I think I need to erase some of the launcher icons VeryGoodCli created before...

How can I change icon in the current version?

@felangel Could you provide some light on the subject?

Can we get a quick comment or guide on how to do this? Changing it manually, the icon Very Good Ventures icon still continues to appear.

@raulmabe the icons are set natively for both android and ios, you can override it by going to the respective folders. For android, if you go inside the AndroidManifest.xml inside your application tag you will find this entry "android:icon="@mipmap/ic_launcher">" try changing this one

@raulmabe the icons are set natively for both android and ios, you can override it by going to the respective folders. For android, if you go inside the AndroidManifest.xml inside your application tag you will find this entry "android:icon="@mipmap/ic_launcher">" try changing this one

What I mean is when you navigate to this part, you will get an idea of how to change the icons and stuffs

Hi everyone! Sorry for the delayed response! As @arbazadam mentioned, you can currently update the icons and you normally would in the ios, android, and web directories respectively. We are planning to work on simplifying this process very soon 👍

Hi everyone! Sorry for the delayed response! As @arbazadam mentioned, you can currently update the icons and you normally would in the ios, android, and web directories respectively. We are planning to work on simplifying this process very soon 👍

Hi @felangel any update on this feature?

Hi everyone! Sorry for the delayed response! As @arbazadam mentioned, you can currently update the icons and you normally would in the ios, android, and web directories respectively. We are planning to work on simplifying this process very soon 👍

will go check it out. much appreciated.

Is there any update on this?

@kranfix we evaluated the use of flutter launcher icons but there are several bugs particularly around using adaptive icons on android that we feel need to be resolved before we make the change (https://github.com/fluttercommunity/flutter_launcher_icons/issues/96).
Currently, the process of updating icons follows the standard process for iOS, Android, and Web and the main drawbacks are it is time-consuming and requires learning how to update the icons on each platform. If we transition to using flutter launcher icons, the process will be simpler but we will face scaling/padding issues reported above. In addition, it doesn't appear like the flutter launcher icons is being actively maintained which is another cause for concern.


#224 Allow for setting custom ID to avoid translation clobbering

Adds a Translation::setId function, with support for the Xliff extractor to call it with the Unit ID.
This permits the library to better conform to xliff's uniqueness requirements (the same source text can be translated several times with different unit IDs) while still supporting PO's uniqueness requirements (the same source text cannot appear twice in the same context).
Note that this means XLIFF files converted to PO files may lose some translations that appear several times with the same source text. (This is necessary to avoid a regression of https://github.com/oscarotero/Gettext/issues/48.)
Note that I did not modify e.g. the PO extractor to use setId, even though it supports round-tripping XLIFF unit IDs. These unit IDs can still be read out from the comments or using \Gettext\Genreators\Xliff::getUnitID. (This felt like the best design choice.)
Note the warnings in Translation::setId and Translations::find: Translations with custom IDs (e.g. XLIFF unit IDs) may not be found by the Translations::find function, as it searches using IDs generated by the Translation::generateId function.
@oscarotero, this may break Translation::find for  XLIFF files that were previously generated by this library, as they will use a md5 sum for the unit ID.

this may break Translation::find for XLIFF files that were previously generated by this library, as they will use a md5 sum for the unit ID.

Yes, I'd add the $translation->setId($unitId); optional. Something like:
$translations = Translations::fromXliffFile('translations.xliff', ['unitid_as_id' => true]);

And by default, set to false.

@oscarotero, thanks, I've added that option and adjusted the tests accordingly. https://github.com/oscarotero/Gettext/pull/225/commits/3384000cbfcb9e4dd35ea039b441647423580d58

Thanks!

Many thanks for your sensible stewardship, @oscarotero!


Run single server from IDEA

This way the server runs "out of the box". User doesn't have to configure anything, just launch AzkabanSingleServer with "Run as Java..".
On the first run test/local-conf-templates/conf/azkaban.properties is copied to local/conf (a git-ignored location).
When running the AzkabanSingleServer, working dir must be the azkaban project root

At least for me this is the default in IntelliJ IDEA, so indeed it works out of the box
So working dir can not be for example azkaban/azkaban-solo-server/ or azkaban/temp/

This is according to the plan that was discussed with @HappyRay in https://github.com/azkaban/azkaban/issues/1130#issuecomment-306006139.
Since that discussion I made one more conf fix:

Set azkaban.jobtype.plugin.dir=azkaban-solo-server/src/main/resources
That folder includes a commonprivate.properties that sets execute.as.user=false
Otherwise ProcessJob jobs would fail with a complaint that azkaban.native.lib must be defined

This is the file tree it now creates when run:

Ready from my perspective. What do you say?

Thanks!


doesn’t work on macos sonoma

the applescript code that changes the terminal profile works when run in script editor, but when run via osascript it fails with this error:
execution error: Terminal got an error: AppleEvent handler failed. (-10000)

/Users/spicicomic/Library/pnpm/global/5/.pnpm/execa@5.1.1/node_modules/execa/lib/error.js:60
		error = new Error(message);
		        ^

Error: Command failed with exit code 1: osascript -e tell application "Terminal"
	set current settings of tabs of windows to settings set "Rose Pine Dawn"
end tell
26:102: execution error: “Terminal”遇到一个错误：AppleEvent处理程序失败。 (-10000)
    at makeError (/Users/spicicomic/Library/pnpm/global/5/.pnpm/execa@5.1.1/node_modules/execa/lib/error.js:60:11)
    at handlePromise (/Users/spicicomic/Library/pnpm/global/5/.pnpm/execa@5.1.1/node_modules/execa/index.js:118:26)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async runAppleScript (file:///Users/spicicomic/Library/pnpm/global/5/.pnpm/run-applescript@6.1.0/node_modules/run-applescript/index.js:11:19)
    at async setTerminalProfile (file:///Users/spicicomic/Library/pnpm/global/5/.pnpm/terminal-profile@2.0.0/node_modules/terminal-profile/index.js:8:2)
    at async Promise.all (index 0)
    at async Command.updateProfile (file:///Users/spicicomic/Library/pnpm/global/5/.pnpm/auto-terminal-profile@2.1.0/node_modules/auto-terminal-profile/actions/update-profile.js:17:2) {
  shortMessage: 'Command failed with exit code 1: osascript -e tell application "Terminal"\n' +
    '\tset current settings of tabs of windows to settings set "Rose Pine Dawn"\n' +
    'end tell ',
  command: 'osascript -e tell application "Terminal"\n' +
    '\tset current settings of tabs of windows to settings set "Rose Pine Dawn"\n' +
    'end tell ',
  escapedCommand: 'osascript -e "tell application \\"Terminal\\"\n' +
    '\tset current settings of tabs of windows to settings set \\"Rose Pine Dawn\\"\n' +
    'end tell" ',
  exitCode: 1,
  signal: undefined,
  signalDescription: undefined,
  stdout: '',
  stderr: '26:102: execution error: “Terminal”遇到一个错误：AppleEvent处理程序失败。 (-10000)',
  failed: true,
  timedOut: false,
  isCanceled: false,
  killed: false
}

I found that the osascript version of the command works as long as you run it from outside Terminal.app - I used Alacritty for this. The first time I did so, I received a UI prompt to allow Alacritty to control Terminal, and then found that permission stored under Privacy and Security -> Automation.
When using the osascript command in Terminal.app, that UI prompt never appears, seemingly Terminal cannot directly control Terminal, which sounds like an OS bug to me.
What you can do however is invoke the other terminal app from Terminal.app, once you've given that other app the permission:
/Applications/Alacritty.app/Contents/MacOS/alacritty -e \
osascript -e "tell application \"Terminal\" to set current settings of tabs of windows to settings set \"PROFILE_NAME\""

So the trick to solving this is to presumably to figure out how to get either Terminal or the node script to request this needed Automation permission.

fixed in f1789d6


Is it possible to pass the SkyDNS/SkyDock hostnames as own hostnames into containers?

First I would like to appreciate for a very useful services that you've developed. I managed to configure name resolution between Docker containers as it was proposed here.
Currently I have an urgent need to make container know its hostname assigned by SkyDNS.
When we launch a container inside Docker environment with SkyDNS+SkyDock enabled, it can be accessed with a fqdn:
isaev@wks-isaev:~$ docker run -d --name=test nginx
b48ff6a2b2718a05d4d307db17ff22f91c31ba8eb8b9c225b2e4bad375607ad3
isaev@wks-isaev:~$ dig @<IP_ADDRESS> test.nginx.dev.skydns.local
...
;; ANSWER SECTION:
test.nginx.dev.skydns.local. 22	IN	A	<IP_ADDRESS>

However, container doesn't know it's own hostname:
isaev@wks-isaev:~$ docker exec test hostname --fqdn
b48ff6a2b271

Can container get its own hostname from dns and use it?

Does the container's /etc/resolv.conf point to <IP_ADDRESS> ?


Fixed TypeError when exporting flows with Hotspot overrides

When exporting a flow with Hotspot overrides (aka hotspots defined in a symbol with targets being defined in overrides) a TypeError (null is not an object) occured.

Thanks for this suggestion, looking into it!

Fixed separately in v0.2.9 (commit 3ad002dcf1c69b1cd9bded7f4923335f6e57694f)

(Please confirm that this fixes things for you! If not, I'll reopen)


Triggering new doc to get added to index

is there a special way to continue feeding the index on the model events for saved and updated? It doesn't appear that the package creates a new doc or updates the doc after model manipulation.
I am not sure if I have something misconfigured or not.

I assume you already have the Laravel Scout Searchable trait on your model. That should be enough. If not then there might be a bug in Explorer. If you don't know how to debug it you could try recreating the issue in the demo app for Explorer (https://github.com/Jeroen-G/explorer-demo) or, if possible, share your repository.

Here is a bit more detail about the project and the implementations so far.

I've implemented my own version of the the ExplorerServiceProvider.php
     `$this->app->bind(ElasticClientFactory::class, function () {
     $client = null;
     if(env('APP_ENV', 'local') != 'local') {
         $client = ClientBuilder::create()
             ->setElasticCloudId(env('ELASTIC_CLOUD_ID', 'xxxxxx'))
             ->setApiKey(env('ELASTIC_API_ID', 'xxxxxxx'), env('ELASTIC_API_KEY', 'xxxxxx'));
     } else {
         .... original client setup
     }`

I've confirmed that the index does not update after the model is stored or searchable is called directly on the eloquent query in the demo app. I can debug further but I am not too sure if it's in the IndexAdapter or if it's not binding correctly. All of the other functions work perfectly, Flushing, importing, create index, and mapping.

I appreciate any guidance as to how to populate the index after updating or creating a new model.

I believe Scout should call a method (don't know which one) on the Explorer Engine to update the index. If everything else works it probably is a bug in Explorer.

I just had an issue with updating as well, in my case it was misconfiguration on my part.  Scout defaults to the null engine, quite annoying and it defaults to queueing the update calls , while I didn't have any queues configured.
Ensure you have your SCOUT_DRIVER env var set to 'elastic' and set SCOUT_QUEUE to false for debugging.
The update process itself is is handled in the Searchable trait's queueMakeSearchable method (vendor/laravel/scout/src/Searchable.php:55). Here $models->first()->searchableUsing() should be the elastic engine.
If it doesn't even hit queueMakeSearchable , look in the ModelObserver's  saved method vendor/laravel/scout/src/ModelObserver.php:91, it is responsable for calling the searchable (and thus queueMakeSearchable)  on the model when it has been saved.
Hope this helps!


Workbook spits json, but sheet_to_json gives empty array

Reports (4).xlsx
Hi, I want to convert this excel file to an array of objects, and for some reason even in sheet.js demos I get only the headers. Can someone advice me on this?
https://oss.sheetjs.com/
https://oss.sheetjs.com/sheetjs/
In both these, the files return me only headers.
I get an empty array when I use the below function.
handleUpload = (evt) => {
	const selectedFile = evt.target.files[0];
	const reader = new FileReader();
	reader.onload = (event)  => {
		const data = event.target.result;
		const workbook = XLSX.read(data, {
			type: 'binary'
		});
		console.log('workbook', workbook)
		workbook.SheetNames.forEach(sheetName => {
			console.log('sheetName', sheetName)

			const xlRowObject = XLSX.utils.sheet_to_json(workbook.Sheets[sheetName]);
			console.log(xlRowObject)
		})
	};
	reader.onerror = (event) => {
		console.error(`File could not be read! Code ${event.target.error.code}`);
	};
	reader.readAsBinaryString(selectedFile);
}

Please advice.

XLSX thinks that "!ref" range in your sheet is just on the first columns ("A1:E1") (IDK why)
https://oss.sheetjs.com/sheetjs/
global_wb.Sheets[global_wb.SheetNames[0]]["!ref"] // "A1:E1"

so all XLSX.utils.sheet_to_* will implicitly honor this range.
I don't know the reason this sheet has this range or how it could happen: saving this document unmodified resets this to "A1:E6" for me. So I think it might be some glitch in export or corruption of file, perhaps (?)
You can manually set that range and subsequent calls will behave as expected
global_wb.Sheets[global_wb.SheetNames[0]]["!ref"] = "A1:E6";
XLSX.utils.sheet_to_csv(global_wb.Sheets[global_wb.SheetNames[0]]);
// gives
"State,FY,Quarter,Numer of Grants,Total Amount Awarded
Alabama,2018,4,1,\"$247,551.00 \"
Alabama,2019,3,5,\"$1,732,820.00 \"
Alaska,2019,2,1,\"$986,123.00 \"
Alaska,2019,3,30,\"$3,066,610.00 \"
Alaska,2019,4,1,\"$345,761.00 \"
"

or for .._to_json you can specify range as a property of second argument XLSX.utils.sheet_to_json(workbook.Sheets[sheetName],{range:"A1:E6"});.

@myfonj summarized the situation well.  This file is most likely an export from a non-Excel utility and cheated with the worksheet dimension.
The file reports the range of the first worksheet as A1:E1.  The reported range shows up in the worksheet XML.  If you go to https://oss.sheetjs.com/cfb-editor/ , drag-drop the file, look at xl/worksheets/sheet1.xml and "view as text" you'll see the reported range in the "dimension" record:

The update_sheet_range function in the wiki shows a way to recalculate the worksheet range.


Server Pagination Sort Only Sorts Loaded Rows - Table

Bug description
Hello,
After enabling "SERVER PAGINATION" in the "Table" chart, I noticed sorting the table will only sort the page that's currently loaded. For example:

We can see the first page is sorted in descending order by the date, and 2:43:53 on September 10 is the latest date shown.
However, when we go to the last page:

we can see a few rows with 21:56:57 on September 10, which is obviously later than the above date. I'm assuming the issue is that only the page that's loaded is sorted (on the client), rather than being done through the database which would be ideal.
How to reproduce the bug

Create a "Table" chart with a significant number of rows (enough so there are two or more pages of server side results). Ensure at least one column, which I'll refer to as test_column, contains only unique values.
Sort the

Screenshots/recordings
See bug description
Superset version
4.0.2
Python version
3.10
Node version
18 or greater
Browser
Firefox
Additional context
I've only tested this with "Raw Records" on the "Table" chart. I doubt this would differ with "Aggregate", but I'll mention that just in case.
Checklist

[X] I have searched Superset docs and Slack and didn't find a solution to my problem.
[X] I have searched the GitHub issue tracker and didn't find a similar bug report.
[X] I have checked Superset's logs for errors and if I found a relevant Python stacktrace, I included it here as text in the "additional context" section.

This also appears to be the same issue when using the search bar on tables with server pagination - it'll only search the currently loaded page.


azure-mgmt-redis: LICENSE missing from PyPi

Package Name: azure-mgmt-redis
Package Version: 13.0.0
Operating System: Linux
Python Version: 3.9.6

Describe the bug
The LICENSE file is missing from azure-mgmt-redis in PyPi.
Additional context
I am making a PR for this now.

same issue : https://github.com/Azure/azure-sdk-for-python/issues/20183


feat: add access tags and utilize in examples/tests

Description
Add access management tags to all resources provisioned by the module
Types of changes in this PR
No release required

[ ] Examples or tests (addition or updates of examples or tests)
[ ] Documentation update
[ ] CI-related update (pipeline, etc.)
[ ] Other changes that don't affect Terraform code

Release required

[ ] Bug fix (patch release (x.x.X): Change that fixes an issue and is compatible with earlier versions)
[x] New feature (minor release (x.X.x): Change that adds functionality and is compatible with earlier versions)
[ ] Breaking change (major release (X.x.x): Change that is likely incompatible with previous versions)

Release notes content
Replace this text with information that users need to know about the bug fixes, features, and breaking changes. This information helps the merger write the commit message that is published in the release notes for the module.

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

Merge actions for mergers

Merge by using "Squash and merge".

Use a relevant conventional commit message that is based on the PR contents and any release notes provided by the PR author.
The commit message determines whether a new version of the module is needed, and if so, which semver increment to use (major, minor, or patch).

@MatthewLemmond Upgrade test is failing:
│ Error: Value for undeclared variable
│
│ A variable named "access_tags" was assigned on the command line, but the
│ root module does not declare a variable of that name. To use this value,
│ add a "variable" block to the configuration.
╵}

Prob needs to be skipped to get the change in that you are making in pr_test.go

:tada: This PR is included in version 7.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Re-organize code layout

I guess we need some structure now. Would be good to split source files, compiler and resources.

Done in v2.0.0


[DOCS] Document Tree & TreeItem

Please make sure to squash the commits together when making the requested changes: http://docs.godotengine.org/en/latest/community/contributing/pr_workflow.html#mastering-the-pr-workflow-the-rebase

Done & squashed. Implemented all suggestions above, hope I haven't missed anything :)

@cbscribe: Done.

Thanks for your efforts - this was a big job.


Add ability to set NODE_ENV for npm_package

Description
Adds the ability to set NODE_ENV for the npm_package resource.
Important to note that setting --production overrides NODE_ENV.
Issues Resolved
#195
Check List

[x] All tests pass.
[x] New functionality includes testing.
[x] New functionality has been documented in the README if applicable

FYI, I'm going to get the GitHub pipeline to run test instances before doing much of anything else.

          1 Error

:no_entry_sign:
Please include a CHANGELOG entry.

  Generated by :no_entry_sign: Danger

Could you add a test to verify that npm_package installed the package as expected?

It took two years, but here we are! 🎉


hanging on NodeJS

Type: Bug
when pressing Ctrl+F5, nodejs won't start running just hanging.
VS Code version: Code - Insiders 1.88.0-insider (12fc36731bd0424dfc53e9f456ac793c1eb1d6d0, 2024-03-11T05:47:58.870Z)
OS version: Windows_NT x64 10.0.22621
Modes:

Please grab https://github.com/microsoft/vscode-js-debug#nightly-extension for the fix

you didn't understand the problem.
in earlier versions this bug wasn't found!

Yes, I understand, however "run without debugging" is still actually handled by the debugger extension, and the nightly version of the extension has the fix you need


CASMPET-7142 tpm-provisioner-client RPM is not istallable on SLES15-SP5

Summary and Scope
tpm-provisioner-client RPM is produced using rolling tag of artifactory.algol60.net/csm-docker/stable/csm-docker-sle:latest container image. This image is currently based on SLES15-SP6. As a result, produced RPM requires glibc 2.34 and is not installable on SLES15-SP5 which comes with glibc 2.31.
Issues and Related PRs

Resolves CASMPET-7142

/backport release/1.5


Change app.window fields to methods for correctness in the case that a user calls window.set_size, etc

Right now if a user calls set_inner_size_pixels the app.window fields will not reflect the new size until the next update. By requiring calling methods instead we can ensure the correct values.


Read aperture/scatterguard hardware on read

Following https://github.com/DiamondLightSource/dodal/pull/769 a side effect of this is that we're no longer reading the state of the hardware whenever we read selected_aperture. This means that the device itself is holding on to the state of the beamline based on what state it last moved to. This means that the state could be wrong if:

The device has just been initialised
The hardware has been moved via EPICS/manually
There was a failure in moving the device

Acceptance Criteria

Hardware is read when the device is read
The device holds no assumptions about the state of the hardware

selected_aperture and radius are both going to be cached in the soft_signal state, so if you're modifying the handling of one should adjust the other too. Mea culpa.

I wonder if we should try and write a general rule in the docs of "devices should avoid holding state and instead read the state of the hardware when read"? Thoughts @callumforrester, @dperl-dls?

Would be nice if there was some utility for making a soft_signal backed by some function that may read other parts of device state


Darkcoin orphans

Hi,
Not sure what the problem is or if there is an update that needs to happen because of recent upgrades to darkcoind, but I'm getting quite a few orphans now as it seems pp doen't always pull the proper block value??
http://www.xpool.ca/blocks/DRK
Anyone else seeing this?

Sorry it took so long, PR is merged in now. Made a few minor tweaks which still need some testing tho


Multicore compilation on roadmap?

Greetings,
Does the Typescript team have any plans for enabling multicore compilation?  At my org, we're developing HTML5 applications with Typescript.  Our test test laptops have 8 cores and the dev machines 64+ cores.  It would be great to speed up builds by spreading compile jobs over multiple CPUs.
If not, I understand.  The priorities of web development are a bit different than applications, and our setup is a bit core-happy.  Just trying to get a sense of whether this is in the team's thoughts.

Which company was that? I'd love to get one of your 64+ core machines.

Does the Typescript team have any plans for enabling multicore compilation? At my org, we're developing HTML5 applications with Typescript. Our test test laptops have 8 cores and the dev machines 64+ cores. It would be great to speed up builds by spreading compile jobs over multiple CPUs.

Really nice to know that there are organizations that still understand the value of this. In so many jobs nowadays a workstation is considered a luxury which is just ridiculous

<mild-rant> As a tag starting </ is an end tag is it not implied that it is the end? also <endrant></endrant> look really odd </mild-rant>

This is not in the scope of the TS compiler; this is rather in the realm of  build orchestration/management tools. We achieve a similar result using MSBuild today in VS, and I am sure you can achieve the same with other build tools.
You can break up your build/test steps down into even single file compiles, and then let your build orchestrator/manager schedule them on whatever available resources.


Improve dropbear key detection and creation

It is not a great idea to copy host keys into an initramfs because an initramfs is often readable by the world (and will always be so when installed on, e.g., an EFI system partition with systemd-boot). Generating keys is good for testing but is not good in production because each new initramfs will have unique keys and trigger warnings or hard failures on the client. The best option is to maintain a second set of host keys only for initramfs images, which can be stored in dropbear format in /etc/dropbear.
To allow this possibility, this PR alters the installation script to simply copy the /etc/dropbear directory of that directory contains at least one recognized key. If not, the old behavior is preserved: converting OpenSSH host keys if possible, generating new ones if not.
While we're refactoring key management, let's add (optional) support for ed25519, ignoring failures in case the version of dropbear used by the hook does not support that type.
This supersedes #13 with a cleaner implementation that doesn't break the hook with older versions of dropbear.

Incorporated in https://github.com/ahesford/mkinitcpio-dropbear/commit/48e496ae61c3d09d98b3fb53ba61b921b7fbe79c