Use domain instead of workstation in output hash

The second field of the hash is actually for the domain name and not the workstation name. In fact, the workstation name is not used at all when computing the NTLM response.
Source: https://docs.microsoft.com/en-us/openspecs/windows_protocols/ms-nlmp/b38c36ed-2804-4868-a9ff-8dd3182128e4
Therefore, the format for John is:
{Domain\Username}::{Domain}:{Server Challenge}:{NT Proof String}:{NTLMv2 Response (minus the NT Proof String)}

not:
{Domain\Username}::{Workstation}:{Server Challenge}:{NT Proof String}:{NTLMv2 Response (minus the NT Proof String)}

as mentioned in your blog post.
Furthermore, it seems like Hashcat doesn't like the domain being part of the username, so I changed the output format to this:
{Username}::{Domain}:{Server Challenge}:{NT Proof String}:{NTLMv2 Response (minus the NT Proof String)}

This way it should be compatible with both John the Ripper and Hashcat.

Thanks for the pull request!  I need to look into this more.
According to the NTLMSSP specification I used to develop the tool [http://davenport.sourceforge.net/ntlm.html#theType3Message] Workstation is one of the fields that should be present in the Type 3 message.  The Microsoft specification document also has a section discussing the Workstation name.
Thanks for the info about the hashcat / john conflict.  I need a bit to review everything and I'll get an update out.

I made an adjustment to the code yesterday that takes into account what you reported in the pull request, and now the hashes work with both john and hashcat.  I decided to not merge in the PR since I wanted to keep some of the code that you removed, but I really appreciate you bringing my attention to this!


[regression test] Test the unique model by modify a value type from TINYINT to other type

Proposed changes
Issue Number: close #xxx

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

run buildall

run buildall

run buildall

run buildall


How do I include org-checklist?

Hi
I'd like to include/install org-checklist but it doesn't work so far.
I've tried C-h-v org-modules => customize => tick org-checklist => restart. Reviewing the list and org-checklist remains to be ticked but when change an item from NEXT status to DONE, no checkbox is reset.
The item looks like this:
* NEXT Weekly Review [5/5]
SCHEDULED: <2019-02-23 Sat ++1w>
:PROPERTIES:
:RESET_CHECK_BOXES: t
:END
1. [X] Loose Papers [1/1]
   - [X] Remove loose stuff from desk

2. [X] Pro

Unsure, sorry, I don't use that package and I don't currently have time to look into this.

@n0ego did you read this? https://stackoverflow.com/questions/20164918/how-to-untick-checkboxes-in-org-mode-for-the-next-cyclic-repetitive-task


Integracion Defensa a juego

Codecov Report
Patch coverage has no change and project coverage change: -1.13 :warning:

Comparison is base (11dd916) 38.75% compared to head (18fb505) 37.62%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@             Coverage Diff              @@
##             master      #28      +/-   ##
============================================
- Coverage     38.75%   37.62%   -1.13%
  Complexity       67       67
============================================
  Files            38       38
  Lines           400      412      +12
  Branches         18       18
============================================
  Hits            155      155
- Misses          238      250      +12
  Partials          7        7

Impacted Files
Coverage Δ

.../java/edu/fiuba/algo3/modelo/Defensas/Defensa.java
0.00% <ø> (ø)

...main/java/edu/fiuba/algo3/modelo/juego/Ganado.java
0.00% <0.00%> (ø)

.../main/java/edu/fiuba/algo3/modelo/juego/Juego.java
0.00% <0.00%> (ø)

...ain/java/edu/fiuba/algo3/modelo/juego/Jugador.java
70.37% <0.00%> (+2.51%)
:arrow_up:

...ain/java/edu/fiuba/algo3/modelo/juego/Jugando.java
0.00% <0.00%> (ø)

...ain/java/edu/fiuba/algo3/modelo/juego/Perdido.java
0.00% <0.00%> (ø)

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


Add Del key behavior

This just adds the default text editing behavior for the delete key, at least on windows.

Thanks for this!
I haven't tested on Windows but this works on my Macbook with fn+delete and with my Bluetooth keyboard's del key.

Thanks for the contribution, and to @futurepaul for the review.


XSLT Injection Challenge

Recently researched a lot about XSLT Injection Vulnerability. Its a obscure vulnerability but with great security impact which is somewhat relatable to XML Injection. This challenge if possible can be implemented around pdf creation for reports/receipts etc. Obscure vulnerabilities are more fun and will encourage users to research.
References :
1.https://www.contextis.com/blog/xslt-server-side-injection-attacks
2. My simple php Implementation.
3.https://cocoon.apache.org/2.1/howto/howto-html-pdf-publishing.html
4.https://www.npmjs.com/browse/keyword/xslt

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@bkimminich @CaptainFreak I would like to take up this issue. Do have something in mind for the implementation?

@bkimminich, @CaptainFreak, currently pdfkit is being used to create the pdf files, so xslt can't be used to create this challenge. But if the library can be altered, an injection based challenge can be created. The flow could be like this:

The order invoice is created using the order details input.
We can make the attacker generate an invoice for products not currently in the juice shop, and he too will not be aware of the products to order. He will be instructed that there exists a misplaced xml file containing some discontinued products. He's got to order these.
He needs to first find the file and then inject <xsl:copy-of select=\"document('./products.xml')\"/> to include the contents of products.xml in the invoice.

This marks the successful completion of the challenge. We may employ few other steps to make the challenge a bit tougher.

XSLT Injection could be done within XML data export once that is implemented. Issue needs more detailed idea how to place it there.


Dialog API: Office.context.ui.displayDialogAsync opens in new tab instead of new window on browsers MAC

I am developing a custom functions add-in and using Dialog API for authentication. When the add-in is opened on safari browser in MAC, the Dialog API window behaves differently when opened from excel online. As shown in the attached video, when the parent window is maximized, the Dialog API opens the non-modal window in a new tab instead of new window, but when the parent window is shrunk, the Dialog API opens the non-modal window in new window. This behaviour is seen on both chrome as well as safari browsers in MAC. Due to this, my application is unable to authenticate when displayDialogAsync opens the login URL in a new tab and it is unable to message parent.
Note: In windows, the dialog API opens a new window and do not see this behaviour. I have tested this on Chrome, Edge and Firefox.
      if (isOfficeInitialized) {
            Office.context.ui.displayDialogAsync(fullUrl, { height: 77, width: 43 }, (result) => {
                if (result.status === Office.AsyncResultStatus.Failed) {
                    // TODO:
                } else {
                    _loginDialog = result.value;
                    _loginDialog.addEventHandler(Office.EventType.DialogMessageReceived, processMessage);
                }
                return result;
            });
        }

dialogAPI.zip

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

ID: 79a7a8a4-186e-ced9-4e4c-e78437dd59ba
Version Independent ID: 34044d2e-25ec-6893-edfa-0a51730dc60a
Content: Authenticate and authorize with the Office dialog API - Office Add-ins
Content Source: docs/develop/auth-with-office-dialog-api.md
Product: non-product-specific
Technology: add-ins
GitHub Login: @o365devx
Microsoft Alias: o365devx

Hi @vikasm1720,
Thanks for reporting this. A platform discrepancy sounds like an issue with the product, as opposed to the documentation, so I'm going to transfer the issue to the office-js repo. There, members of the product team can look at your scenario and figure out the underlying cause.


kind: ovn branch and repo

📑 Description
Sometimes is needed to test not merged OVN changes at ovn-kubernetes, to facilitate that this change do the following:

Move Docker.fedora.dev into Docker.fedora to remove duplication
Add --ovn-repo and --ovn-branch to kind.sh to deploy specific ovn change
Call make fedora-dev at CI to check that custom OVN build is not broken.

✅ Checks

[x] My code requires changes to the documentation
[x] if so, I have updated the documentation as required
[ ] My code requires tests
[ ] if so, I have added and/or updated the tests as required
[ ] All the tests have passed in the CI

How to verify it
It builds the fedora-dev image at CI

The CI build failures seem unrelated:
2024-12-17T11:40:34.5624293Z Executed commands do not match expected commands!
2024-12-17T11:40:34.5624860Z Executed unexpected command /fake-bin/ovs-vsctl --timeout=15 port-to-br enp3s0f0

I re-triggered the job.

Actually @qinqon I take back my approval, sorry.
In order to actually be able to test this in CI with custom OVN repos/branches we need to change this step in the test.yml file too:
    - name: Build docker image - from current pr branch
      if: steps.is_pr_image_build_needed.outputs.PR_IMAGE_RESTORED != 'true' && success()
      run: |
        pushd dist/images
          sudo cp -f ../../go-controller/_output/go/bin/ovn* .
          sudo cp -f ../../go-controller/_output/go/bin/hybrid-overlay-node .
          echo "ref: $(git rev-parse  --symbolic-full-name HEAD)  commit: $(git rev-parse  HEAD)" > git_info
          docker build -t ovn-daemonset-fedora:pr -f Dockerfile.fedora .
          mkdir _output
          docker save ovn-daemonset-fedora:pr > _output/${CI_IMAGE_PR_TAR}
        popd

Otherwise all e2e tests will just use the fedora OVN build we download from koji.

As far as I know this is how it's done today.

Actually @qinqon I take back my approval, sorry.
In order to actually be able to test this in CI with custom OVN repos/branches we need to change this step in the test.yml file too:
    - name: Build docker image - from current pr branch
      if: steps.is_pr_image_build_needed.outputs.PR_IMAGE_RESTORED != 'true' && success()
      run: |
        pushd dist/images
          sudo cp -f ../../go-controller/_output/go/bin/ovn* .
          sudo cp -f ../../go-controller/_output/go/bin/hybrid-overlay-node .
          echo "ref: $(git rev-parse  --symbolic-full-name HEAD)  commit: $(git rev-parse  HEAD)" > git_info
          docker build -t ovn-daemonset-fedora:pr -f Dockerfile.fedora .
          mkdir _output
          docker save ovn-daemonset-fedora:pr > _output/${CI_IMAGE_PR_TAR}
        popd

Otherwise all e2e tests will just use the fedora OVN build we download from koji.

As far as I know this is how it's done today.

Let me rephrase. :)
It would be nice if this step in the GitHub actions test file would use OVN_REPO/OVN_BRANCH/OVN_COMMIT/OVN_FROM from env.  We could define those at the top of the file to be OVN_FROM=koji and sane default values for the rest but then when someone wants to run full e2e CI with a custom branch they just need to change those variable values.
With your current code I have to do something like:
     - name: Build docker image - from current pr branch
@@ -215,29 +209,11 @@ jobs:
           sudo cp -f ../../go-controller/_output/go/bin/ovn* .
           sudo cp -f ../../go-controller/_output/go/bin/hybrid-overlay-node .
           echo "ref: $(git rev-parse  --symbolic-full-name HEAD)  commit: $(git rev-parse  HEAD)" > git_info
-          docker build -t ovn-daemonset-fedora:pr -f Dockerfile.fedora .
+          docker build -t ovn-daemonset-fedora:pr --build-arg OVN_FROM=source --build-arg OVN_REPO=https://github.com/dceara/ovn --build-arg OVN_BRANCH=review-pws432592-ecmp-nh-monitor --build-arg OVN_COMMIT=HEAD -f Dockerfile.fedora .
           mkdir _output
           docker save ovn-daemonset-fedora:pr > _output/${CI_IMAGE_PR_TAR}
         popd

This PR partially addresses https://issues.redhat.com/browse/FDP-852


Update the crdb version on the "latest" cloud release notes

The notes are dated based on the date on which email
communication goes out. But since cloud isn't versioned,
the date isn't meaningful beyond that. Since the
"Latest Release Notes" page is the only place where
users can find out what version of crdb the cloud offerings
are running, it makes sense to keep the versions up to date.
I know there's broader work happening around the cloud release
not process that might resolve this issue. As part of that,
we should think of a way to communicate the current crdb
version for cloud that's release note date-agnostic.

This change is


📖 Help Window inaccessible from Tray

The help menu is inaccessible from Windows System Tray, the only way to read the built-in documentation is using the Wincuts window itself.

Fixed by me


Verify blobs

This adds verification of blobs against the expected digests as they are being copied.
Depends on https://github.com/containers/image/pull/48 , I will revendor after it is merged.

Revendored.

lgtm


Make the PHAF file headers consistent

We should standardise on the text for each column header, and the case.

Duplicate of: pombase/curation#3539


Missing/Double Local Notifications

Description
As part of an ongoing study (HPDS), study participants reported two separate bugs pertaining to survey notifications:

After installing an app update on TestFlight, the app stopped sending survey notifications. This issue could be resolved by fully uninstalling the previous app build before installing the update through TestFlight.
After about 2 weeks, the app would sometime send double notifications for every survey. This did not happen for all users, and it also appears to have returned back to single notifications after some time.

Our app sends 4 notifications per day, so the ~2 week delay corresponds roughly to the 64 local notification limit.
Reproduction

We were able to reproduce this bug by pushing an identical new build to TestFlight and updating the application without uninstalling the previous version.
This bug is difficult to reproduce and seems to be non-deterministic. One might be able to reproduce this and shorten the 2 week window by creating a test application that schedules 64 notifications each day.

Apologies that I'm not able to offer more exact reproducibility guidelines. I'm happy to help set up small testing applications to diagnose these issues.
Expected behavior
The app should send exactly one notification for each scheduled survey. The app should also continue to send notifications after it is updated (possibly automatically in the background) on TestFlight.
Additional context
Here is the relevant code for our app's survey scheduling:
extension HPDSScheduler {
    /// Creates a default instance of the ``HPDSScheduler`` by scheduling the tasks listed below.
    convenience init() {
        self.init(
            tasks: [
                Task(
                    title: String(localized: "MORNING_SURVEY_TITLE"),
                    description: String(localized: "MORNING_SURVEY_DESCRIPTION"),
                    schedule: Schedule(
                        start: Calendar.current.startOfDay(for: Date()),
                        repetition: .matching(.init(hour: 17, minute: 00)), // Every Day between 8-11 AM
                        end: .numberOfEvents(356)
                    ),
                    notifications: true,
                    context: HPDSTaskContext.questionnaire(Bundle.main.questionnaire(withName: "morning-en-US"))
                ),
                Task(
                    title: String(localized: "MID_DAY_SURVEY_TITLE"),
                    description: String(localized: "MID_DAY_SURVEY_DESCRIPTION"),
                    schedule: Schedule(
                        start: Calendar.current.startOfDay(for: Date()),
                        repetition: .matching(.init(hour: 17, minute: 00)), // Every Day at 11-14 PM
                        end: .numberOfEvents(356)
                    ),
                    notifications: true,
                    context: HPDSTaskContext.questionnaire(Bundle.main.questionnaire(withName: "mid-day-en-US"))
                ),
                Task(
                    title: String(localized: "AFTERNOON_SURVEY_TITLE"),
                    description: String(localized: "AFTERNOON_SURVEY_DESCRIPTION"),
                    schedule: Schedule(
                        start: Calendar.current.startOfDay(for: Date()),
                        repetition: .matching(.init(hour: 17, minute: 00)), // Every Day at 14-17 PM
                        end: .numberOfEvents(356)
                    ),
                    notifications: true,
                    context: HPDSTaskContext.questionnaire(Bundle.main.questionnaire(withName: "afternoon-en-US"))
                ),
                Task(
                    title: String(localized: "END_OF_THE_DAY_SURVEY_TITLE"),
                    description: String(localized: "END_OF_THE_DAY_SURVEY_DESCRIPTION"),
                    schedule: Schedule(
                        start: Calendar.current.startOfDay(for: Date()),
                        repetition: .matching(.init(hour: 17, minute: 00)), // Every Day at 17:00 PM
                        end: .numberOfEvents(356)
                    ),
                    notifications: true,
                    context: HPDSTaskContext.questionnaire(Bundle.main.questionnaire(withName: "end-of-day-en-US"))
                )
            ]
        )
    }
}

Code of Conduct

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

@mjoerke #28 should have addressed the problems described in this issue. Please let us know if you ca somehow reproduce this in one of your applications 👍

Thank you @PSchmiedmayer!! We will update this in our application and distribute it for testing 👍


A ReferenceGrant Should Be Removed From Watchable Map When UnReferenced

#471 added initial support for ReferenceGrants by allowing Gateways to reference Secrets in a different namespace. The gateway controller adds the ReferenceGrant and the ReferenceGrant namespace to the provider watchable map when:

A Gateway listener references a Secret in a different namespace.
A ReferenceGrant in the above namespace exists and permits the Gateways from the Gateway's namespace to Secrets.

When the Gateway is deleted, the ReferenceGrant and ReferenceGrant namespace should be deleted from the provider's watchable map if no other Gateway references the ReferenceGrant.

Note https://github.com/envoyproxy/gateway/pull/580#pullrequestreview-1144310617, when this is fixed the two invalid/missing referencegrant conformance tests should be able to be uncommented.

I prefer if we move this to 0.3.0 . Based on current logic there is no easy way to handle deletes for RefGrants, easier to implement this with https://github.com/envoyproxy/gateway/issues/413 where we dont need to handle deletes, just start from scratch with every reconcile.

Being noted in the v0.2.0 release notes and moving to v0.3.0-rc.1.

https://github.com/envoyproxy/gateway/pull/702 fixes this issue and uncomment's the two conformance tests.


Preserve mask areas when using SD models

Imagine this: I'm inpainting an image and draw over some things. I then enter a prompt and generate content with Stable Diffusion. I don't like it, but now that the content has already been generated, I have to paint over the area again, rather than having the mask preserved.
This is an issue when working with a lot of painted areas and you don't want to waste time carefully creating a mask again.

Actually, this feature is already there! When using sd/paint_by_example model, after inpainting once, clicking the Dream button again will reuse the previous mask.
There are no user-friendly hints for this feature, so perhaps it could be optimized in conjunction with this issue https://github.com/Sanster/lama-cleaner/issues/209

Actually, this feature is already there! When using sd/paint_by_example model, after inpainting once, clicking the Dream button again will reuse the previous mask and previous image.
Tips to myself: There are no user-friendly hints for this feature, so perhaps it could be optimized in conjunction with this issue #209

I opened this "issue" because this exact method doesn't work for me - I make a mask once, write a prompt, click "Dream" - it works. If I click dream again (or first change the prompt) - nothing happens because there's no visible mask, so while it appears as if there's an image being generated, nothing actually ends up getting generated due to the mask not being there

PS: Dunno if I should open a new request for this but it would be useful if we had a restore brush - when cleaning up with the regular lama model, there could be another brush and when used on the image it would restore the initial look of the image (instead of undo-ing everything which is entirely different thing)
What I'm suggesting is the same restore brush as in Photoshop (History Brush Tool)

Both Stable Diffusion and Paint by Example model support this feature. Can you check whether the seed button is enabled or not? By design, if it's not enabled, every time the result will be different.

Other models also add support for reuse last mask: 0.36.0


SNKRS AT

Could anyone help me adapt this to the Austrian SNKRS website, my Discord is linuxhacker0001
I appreciate any help

This project is dead, akamai will detect it very easily.


Internal server error occurred, transaction failed, lumens never transferred

While using account viewer, I have transferred 15000 lumens to another user. Got a message about server error during the transaction, lumens are gone from my account and never arrived on target address.
Related files attached.
Can I get my lumens back or are they gone for good?
42676817982656513.txt
effects-2.txt

How can you tell that lumens didn't arrive on GA5X...? Looking at this operation's effects I can see both account_debited and account_credited effects. The operation is also in the list of receiver account operations list.

Well receiver is a kraken address that never received this amount (lumens are zero like they were before).

You haven't added memo to your transaction so Kraken doesn't know which account to credit. You need to contact their support.

Thanks for a quick response, will try that


Startup Crash - 1.10.2

https://gist.github.com/P3rf3ctXZer0/e3ad9602d06f6c591d31688fc3253b24

Thanks, seems like an odd one, will investigate

Mod interaction bug: both NetherCore and my Netherrocks are registering a tile entity as "container.nether_furnace".  Looks like we both need to put mod prefixes on our names.

Fixed at my end with commit ea7d01e77b822d1aa60980854ab96447e33c930b

Nice catch, will also update on my end when I upgrade.

I am trying to put together a mod pack and whenever I try to load this mod (minecraft version 1.10.2) it crashes before it can load.


Make angular workspaces the default in v13

🚀 Feature request
Command (mark with an x)

[x] new
[ ] build
[ ] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[ ] update
[ ] lint
[ ] extract-i18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Description
Currently, there are multiple ways to setup angular source code.
# Preferred default
❯ npx @angular/cli new web-apps --create-application false
❯ cd web-apps && yarn ng generate application command-center

# Current Default
❯ npx @angular/cli new command-center

Describe the solution you'd like
Ideally, there should be one approach for generating both application and library.

I recently opened an issue against https://github.com/angular-eslint/angular-eslint/issues/462 repo. Having multiple approaches requires library maintainers to do additional work.
What if a developer initially used this command npx @angular/cli new command-center and now has to develop another application. Usually, the common infrastructural code(error handling, auth session, interceptors) should be extracted to some library (say, @company/core) and be re-used by both the applications. This becomes non-trivial if initially angular workspace was not created.

Describe alternatives you've considered
N/A
Migration

I understand, this could be a significant undertaking by the CLI team. Therefore, the team could continue supporting current approach. With v13, the CLI can by default generate workspace file and then application/library. This would mean, library maintainers(libraries which support ng add library) supporting Angular v13 have to code against one approach, thus lessening their workload.

I think there are a number of things to also consider here.

The majority of new users typically won’t need need a monorepo style workspace.
Changing this behaviour will be a significant breaking change as it will invalidate a lot of educational material.
Schematic author typically don’t have to worry about whether the structure of the workspace is a multi repo or monorepo style.   The issue referenced with eslint is quite an edge case, due to the root level base configuration file.

I think there are a number of things to also consider here.

The majority of new users typically won’t need need a monorepo style workspace.
Changing this behaviour will be a significant breaking change as it will invalidate a lot of educational material.
Schematic author typically don’t have to worry about whether the structure of the workspace is a multi repo or monorepo style.   The issue referenced with eslint is quite an edge case, due to the root level base configuration file.

Agreed the majority of new users won't need monorepo, but an angular workspace with just one application under projects cannot be called a monorepo. It would be a monorepo once new user creates another library/application in same workspace
Agreed, but there also have been similar broad changes in the past (.angular-cli.json to angular.json)
Schematic author: Do you think if there is only one way to create application(workspace style), James Henry would have enocuntered that issue?

By the way, why do we recommend workspace style approach for libraries?
https://angular.io/guide/creating-libraries#getting-started

and perhaps, we can do the same thing for applications.
Another question:
❯ npx @angular/cli@12 new command-center
❯ cd command-center && yarn ng g app admin-console

Will the angular CLI team continue supporting this alternative approach? This becomes confusing for library authors and vscode extension authors, since there is .browserslistrc at root and projects/admin-console/.browserslistrc inside project and extension authors are not able to figure out if its a workspace or single app. There could be similar instances where some files are at the root which get picked up by vscode extension and applied for all workspaces apps/libs.

Hi @prabh-62,
Thanks for this request, that said following a convo with our devrel we feel that we shouldn't move forward with this proposal. None-the-less, I will answer your questions below;

Agreed the majority of new users won't need monorepo, but an angular workspace with just one application under projects cannot be called a monorepo. It would be a monorepo once new user creates another library/application in same workspace

That is true, although having an application down in 2 sub directories when you don't want a mono-repo isn't the most intuitive thing.

Agreed, but there also have been similar broad changes in the past (.angular-cli.json to angular.json)

Indeed and from those changes we did learn that we should be more cautious when doing breaking changes especially when they effect the ng new experience. The mentioned changed was also required to introduce multi-project type workspace which in this case the pros outweighed by far the cons.

Schematic author: Do you think if there is only one way to create application(workspace style), James Henry would have enocuntered that issue?

Probably yes, because removing the single project workspace concept will not solve this, since that structure will still need to be supported. That said, users are free to move applications/libraries around example https://github.com/Angular-RU/angular-ru-sdk/tree/master/packages/stream has a library at the root level. While the CLI doesn't generate this out-of-box, this is a perfectly valid supported structure. So a golden rule would be to not assume any particular structure, if you really need to know how is the workspace is structured read the configuration in angular.json.

By the way, why do we recommend workspace style approach for libraries?

In many cases, when creating a library you'd always want to create an application to run integration and E2E tests.

Will the angular CLI team continue supporting this alternative approach? This becomes confusing for library authors and vscode extension authors, since there is .browserslistrc at root and projects/admin-console/.browserslistrc inside project and extension authors are not able to figure out if its a workspace or single app. There could be similar instances where some files are at the root which get picked up by vscode extension and applied for all workspaces apps/libs.

I think the main problem here, is that the mentioned workspace is a mixture of a single and multi project workspace. TypIcally, if you want to create a multi-project repo workspace you start by using --no-create-application.


tidy up of the front-end code and added a back to blacktimber button …

…at the bottom of the content
tidy up of the front-end code and added a back to blacktimber button at the bottom of the content

in review


Fix (or separate) script for analysing systems when boostrapping fails

When you have too few ligands available, calculation of correlation metrics in cinnabar will fail, see: https://github.com/OpenFreeEnergy/cinnabar/issues/124
The easy fix here is to avoid calculating correlation metrics when this happens.
We should ammend the plotting script to optionally skip calculating these metrics, or if necessary provide a separate script.

Here is the alternative plotting script we used in these cases.
plot_results.zip


Safari on iOS 12.2 supports the inputmode attribute

Wait to merge until iOS 12.2 ships.

Has now been added, thanks!


Use WHATWG URL

Hello,
I made the minimal changes to be WHATWG URL compilance.
Support for node 4 was dropped because xo is not supporting it:
https://travis-ci.org/microlinkhq/normalize-url/jobs/376774686

Great! Thanks for doing this. Could you also use const {URLSearchParams} = require('url'); and drop query-string?

and drop query-string?

I not sure how do it and be elegant. It implies Map-to-obj transformation and serialization.
For example here:
`urlObj.searchParams` is a Map, need to convert into Object, sort the key and then convert into a map again.
urlObj.search = queryString.stringify(sortKeys(queryParameters));

Because querystring is built in dependency, why remove it and add more code into the library?

We're using query-string here, not querystring, so it's not built-in. The built-in querystring will also eventually be deprecated in favor of URLSearchParams.

@sindresorhus
done! The code was reduced a lot 👏
What do you think about this approach?
https://github.com/microlinkhq/normalize-url/blob/b885f15df06e5b93112e83971ff145bd8c13468d/index.js#L73

@sindresorhus updated 👌

👍


Add Category Filter

relates to #2863

Added <CategoryRow /> component
Built out the Category Filters
added MarketsList store to hold market list meta data

TODO (to close 2863)

wire up to load markets getter
remove old unused category code
minor cleanup/refactoring

awesome, only one real comment, just for code clarity

categories under popular header with equal number of markets aren't sorted alphabetically

Will do I missed that detail 👍

Why shouldn't categories be alphabetized if they have the same number of markets. I might be rem incorrectly.

@bthaile working on the feedback this AM -- will merge after feedback fixes get merged


Plans for macOS Big Sur system libraries in the dyld shared cache?

Hi,
starting with macOS Big Sur, it has become a fair bit harder to obtain symbols from most macOS system libraries, e.g. for profilers or stack traces: The libraries are no longer present as files in the file system, instead they are in a system-wide dyld shared cache file. See these Google results for more information.
So one way to get at the symbols would be to extract the libraries from the cache. Another option, when running inside the process that has the relevant libraries loaded, would be to find the mappings in the process memory and get the symbol table from there.
Do you know if somebody has already started working on rust code to parse the dyld_shared_cache format and write an extractor? Or whether there are any other plans to make getting symbols for macOS system libraries easier in the rust ecosystem?
I imagine this would also be needed to get proper symbols in rust panic stack traces, for example. (Though those seem to be broken at the moment even before Big Sur, see https://github.com/rust-lang/rust/issues/79831 ).

Do you know if somebody has already started working on rust code to parse the dyld_shared_cache format and write an extractor? Or whether there are any other plans to make getting symbols for macOS system libraries easier in the rust ecosystem?

No I don't. If it's something that is needed for backtraces, then it might be appropriate to add parts of the parsing to this crate.

(Though those seem to be broken at the moment even before Big Sur, see rust-lang/rust#79831

I wasn't aware of that. It might be better to create that issue in the backtrace-rs crate. Sounds like a bad bias calculation somewhere.

It would be interesting to check what lldb does.

It would be interesting to check what lldb does.

From a quick look, lldb gets symbols from the images loaded in memory of the target process, rather than parsing the shared cache itself. It also has the ability to use the images in the host process if the UUID matches, since this is faster. There is some dyld shared cache parsing code, but it is only for iOS. (I've mostly based this on https://reviews.llvm.org/D83023).

From a quick look, lldb gets symbols from the images loaded in memory of the target process, rather than parsing the shared cache itself. It also has the ability to use the images in the host process if the UUID matches, since this is faster. There is some dyld shared cache parsing code, but it is only for iOS. (I've mostly based this on https://reviews.llvm.org/D83023).

I've started making an attempt at this in https://github.com/mstange/object/commits/dyld_cache .


Mactop vs. Activity Monitor CPU use totally different, M2 Macbook Air

Mactop always shows ~50% P and E, even when the machine is otherwise idle. Here is what mactop and activity monitor show at the same time, note the P cores seem to be idle in AM and 50% active in mactop, the E cores are more active in AM but still not 50%:

I've tried htop, gotop, zenith and some others and they all largely agree with AM, only mactop shows this 50% CPU use...

Sorry for the late reply, please post your powermetrics output here if you can, that would assist in debugging this issue as we parse stats from it depending on the chip. It is possibly related to the other open issue: https://github.com/context-labs/mactop/issues/26

sudo powermetrics -i 1000 -n 20 -o pm.log

pm.log
After running this log, I immediately started mactop without anything else running and this is what it shows:

Yea, so it looks like there is no bug here, your machines E-Cluster and P-Cluster (E and P Cores) are in fact running at those percentages correctly, (cherry picking a few of what we parse out of your pm.log here):
E-Cluster HW active residency:  55.86%
P-Cluster HW active residency:  50.12%

E-Cluster HW active residency:  54.01%
P-Cluster HW active residency:  50.22%

E-Cluster HW active residency:  54.53%
P-Cluster HW active residency:  50.27%

Activity monitor shows no activity in the p-cluster at all, and it seems the cluster vs individual stats do not line up::
P-Cluster idle residency:  49.92%

CPU 4 idle residency:  99.33%

CPU 5 idle residency:  99.90%

CPU 6 idle residency:  99.96%

CPU 7 idle residency: 100.00%

How can each CPU's idle be >99% but the overall cluster is only 50% idle? Activity monitor seems to be using the CPU stats (last 4 columns are P cores I think):

But from your perspective, this is a powermetrics problem by the looks of it...

Activity monitor shows no activity in the p-cluster at all, and it seems the cluster vs individual stats do not line up::
P-Cluster idle residency:  49.92%

CPU 4 idle residency:  99.33%

CPU 5 idle residency:  99.90%

CPU 6 idle residency:  99.96%

CPU 7 idle residency: 100.00%

How can each CPU's idle be >99% but the overall cluster is only 50% idle? Activity monitor seems to be using the CPU stats (last 4 columns are P cores I think):

We use HW active residency and yes possibly a powermetrics issue (Apple)

Right, powermetrics is misleading on either active or idle residency on my M2, here the active residence is 55.3% but the active residency of the CPUs is 13.1% 7.1% 4.4% & 2.9%, even if you sum them they don't make 55.3%!?!?
E-Cluster HW active frequency: 936 MHz
E-Cluster HW active residency:  55.31% (600 MHz:   0% 912 MHz:  54% 1284 MHz: .09% 1752 MHz: 1.2% 2004 MHz: .11% 2256 MHz: .11% 2424 MHz: .04%)
E-Cluster idle residency:  44.69%
CPU 0 frequency: 1036 MHz
CPU 0 active residency:  13.10% (600 MHz:   0% 912 MHz:  11% 1284 MHz: .01% 1752 MHz: 1.3% 2004 MHz: .19% 2256 MHz: .19% 2424 MHz: .04%)
CPU 0 idle residency:  86.90%
CPU 1 frequency: 982 MHz
CPU 1 active residency:   7.13% (600 MHz:   0% 912 MHz: 6.6% 1284 MHz: .01% 1752 MHz: .48% 2004 MHz: .03% 2256 MHz: .02% 2424 MHz: .03%)
CPU 1 idle residency:  92.87%
CPU 2 frequency: 949 MHz
CPU 2 active residency:   4.39% (600 MHz:   0% 912 MHz: 4.2% 1284 MHz: .00% 1752 MHz: .10% 2004 MHz: .01% 2256 MHz: .01% 2424 MHz: .04%)
CPU 2 idle residency:  95.61%
CPU 3 frequency: 1055 MHz
CPU 3 active residency:   2.87% (600 MHz:   0% 912 MHz: 2.4% 1284 MHz: .00% 1752 MHz: .44% 2004 MHz: .00% 2256 MHz: .01% 2424 MHz: .02%)
CPU 3 idle residency:  97.13%

Should I close this (or leave it if others hit the same issue as me)?

Just a user since yesterday, but I also noticed #26. My guess is that the percentage should be corrected for the current frequency. My M1 Max has a max frequency  of 3200 MHz. When I do an Xcode build it goes to almost 100% on 3180MHz and there is no overflow. I see overflows a lot when frequency is (a lot) lower.

Just resolved this in v0.2.0, will close this once I get a few confirmations.

I still see the same problems with V0.2 on my M2, comparing zenith and mactop side-by-side zenith shows only 2 P cores active sum ~10% and mactop shows ~50%:

zenith reports what Apple's activity monitor and htop etc. uses. Powermetrics still shows big differences between the individual CPU active residency (which is closer to activity monitor/zenith/htop) and the Cluster active residency:
P-Cluster HW active frequency: 694 MHz (max: 3504 Mhz)
P-Cluster HW active residency:  51.33%
CPU 4 active residency:   4.58%
CPU 5 active residency:   2.22%
CPU 6 active residency:   1.64%
CPU 7 active residency:   1.38%

Note if I multiply cluster active residency by (active frequency ÷ max frequency), then the active residency is similar to the sum of the CPU active residencies:

Thank you @iandol for the info, I am thinking we will move away from using powermetrics soon in coming updates, at least for CPU E/P Core calculations, we should now at least match asitop in terms of parsing and handling powermetrics data. I will more than likely start work on the new native methods in the next week or so.

https://github.com/context-labs/mactop/pull/37
I encourage those interested to test out the development branch as it now uses Apple's native mach kernel api for CPU information! It should now match up very closely now to other monitoring applications!

Interesting and relevant article: https://eclecticlight.co/2024/11/08/why-cpu-in-activity-monitor-isnt-what-you-think/


Consider respecting &loadplugins

Problems summary
When --noplugins is specified in the cmdline, dein#add sets sourced=1 in the plugin's options dictionary, even though the plugin hasn't been sourced. This makes dein#source to skip the plugin when called later
Expected
sourced=0 in the plugin's options dictionary
Environment Information (Required!)

dein.vim version(SHA1): 3271fba1a562062be91365428c2aeaf34b362918

OS: MacOS

Vim/neovim version: NVIM v0.2.3-24-gd9b3ebfed

Provide a minimal .vimrc with less than 50 lines (Required!)
set runtimepath+=~/.local/share/nvim/dein/repos/github.com/Shougo/dein.vim

if dein#load_state('~/.local/share/nvim/dein')
    call dein#begin('~/.local/share/nvim/dein')
    call dein#add('https://github.com/Shougo/dein.vim')
    call dein#add('https://github.com/Konfekt/FastFold')
    call dein#end()
    call dein#save_state()
endif

A quick fix is to add
    if &loadplugins
        let plugin.sourced = 1
    endif

in dein#util#_end()

OK.

@stellarhoof The fix is not compatible with Vim 7.4.


[Docs] README: Broken image of Meshery Architecture

Current State:
Diagrams of Meshery's architecture were recently updated. This caused a broken link in the architecture section of the main README
Screenshot:

Desired State:

This image may be used to show Meshery's architecture - https://github.com/layer5io/meshery/blob/master/docs/assets/img/architecture/Meshery-client-architecture.svg
This image should be copied to the docs/assets/img/readme folder, so as to avoid having it broken again.

Contributor Resources

Meshery documentation site
Meshery documentation source
Instructions for contributing to documentation

@spielers is this issue of interest?

@leecalcote Yeah I would love to resolve this issue


Unable to decompress deflate

Request requestComment = new Request.Builder()
//.header("Accept-Encoding", "defalte")
.removeHeader("Accept-Encoding")
.url(commentUri)
.build();
no matter if i set header or not, it still retruns error, just like that , ��Y�]Wv���nՃ��u�L{8R���`�V�]�. I found some answer about gzip, it says don't set header will help, i tested it, but it didn't help in my case. The url is http://comment.bilibili.com/3885454.xml. Thanks for help~

OkHttp does not support deflate automatically. If the server returns a deflate body you need to use Okio's InflaterSource or InflateInputStream to decompress the body.

Thank you, got it.
InflaterInputStream inputStream = new InflaterInputStream(
responseComment.body().byteStream(), new Inflater(true));

I tried using InflaterSource  and InflateInputStream  but still cant decompress the response. Can anyone help?


Flutter SDK installation is not completed

After updating flutter sdk it is showing that flutter sdk installation is not completed . On the other hand i have saw the flutter official sdk installation page the sdk download is showing loading then failed.

i am also having the same problem

i am also having the same issue

Hi @Hasib74
the situation you are describing
might depends from temporary lack of connectivity
could you please run
flutter channel stable
flutter upgrade --force
flutter pub cache repair
If the issue persist please provide your updated flutter doctor -v
thank you

Hi @Hasib74
Closing as there are not sufficient details to act upon this issue. If the problem persists, please create a new issue and provide the information required in the template.
Thank you


Need Callback function

It will be great if we can have callback functions facilities after updating/resizing/adding new widget/deleting widget etc.
I am trying to get the latest height and width of widget after resizing it with update function but i cant get correct height and width unless i use setTimeout.

You should be able to use the "change" event to detect changes:
$('#grid-stack').on('change', (event, items) => {
    var gridStackData = $("#grid-stack").data('gridstack');
    console.log(gridStackData);
});

The example for serializing is here.
It's save grid function looks at the node list. So my variation of this which returns json string data, which can then be stored in a cookie, is:
saveGrid(){
		this.serializedData = _.map($('.grid-stack > .grid-stack-item:visible'), function (el) {
			el = $(el);
			var node = el.data('_gridstack_node');
			return {
				x: node.x,
				y: node.y,
				width: node.width,
				height: node.height
			};
		}, this);
		return JSON.stringify(this.serializedData, null, '    '));
}

BUT be warned. I am investigating an issue at the moment where it appears to be returning the coordinates wrong for one of the items. If I can reproduce it in a plunker, I will be raising an issue shortly.

Thanks, @tonywr71 ! I'm double-investigating.
@jishanshaikh If you want the coordinates of the particular widget you're changing, use gsresizestop rather than resizestop - jqueryui likes to steal our functions and report wrong (well, in our case wrong) coordinates.

I assume that this question has been answered conclusively. @jishanshaikh Let me know if you're still having an issue.


config: parse config yaml

currently every config key has its own config file. This PR is to use yaml for all config keys.
This is tested with this config file /etc/kepler/kepler.config:
ENABLE_PROCESS_METRICS: true
MODEL_CONFIG: |
  PROCESS_COMPONENTS_INIT_URL=https://raw.githubusercontent.com/sustainable-computing-io/kepler-model-server/main/tests/test_models/DynComponentModelWeight/BPFOnly/ScikitMixed
/ScikitMixed.json

Sorry for delay. But I just notice that you are trying to change for per-file config data to a single yaml.
In this case, you need to change the way to mount the config map to the container.
With the current definition, it is still create separate file for each.
# ls /etc/kepler/kepler.config
BIND_ADDRESS  CGROUP_METRICS  CPU_ARCH_OVERRIDE  ENABLE_EBPF_CGROUPID  ENABLE_GPU  KEPLER_LOG_LEVEL  KEPLER_NAMESPACE  METRIC_PATH  MODEL_CONFIG

Btw, it could be more complicated to change to a single file when considering MODEL_CONFIG.
The way reading the MODEL_CONFIG also needed to be change. Now, it parses the = config.
# cat /etc/kepler/kepler.config/MODEL_CONFIG
CONTAINER_COMPONENTS_ESTIMATOR=false
CONTAINER_COMPONENTS_INIT_URL=https://raw.githubusercontent.com/sustainable-computing-io/kepler-model-server/main/tests/test_models/DynComponentModelWeight/CgroupOnly/ScikitMixed/ScikitMixed.json


Evaluate Rome to replace ESLint / Prettier

https://github.com/Th3S4mur41/th3s4mur41.me/commit/7e1200cc4659b052357b2e9d83ed76a8fce935f7


add kid property to the resource

the kid field is part of the JWK and is used to identify a key in a set

@iwarapter , thanks for the really quick handling of this PR. do you happen to have a rough plan for the next release which will include this? I would need it in the public registry to be able to use the module without too much hassle. thanks a lot!
cheers,
robert

Hi @nrobert13 I'll try get it done this weekend! Thanks for the PR.

Hi @iwarapter , sorry for being insistent here, would there be anything I could help with to get this release out?

Hey @nrobert13, i had forgotten to publish the draft from goreleaser last week, its done now.


R-T5.3-3: The tool must be able to support   configuring AWS EC2 auto scaling service based on the TOSCA auto scaling   policy.

ID:
R-T5.3-3

Type:
PORTABILITY

User   story:
As an Operations Engineer, I want to   configure automatic scaling in Amazon EC2 cloud

Requirement:
The tool must be able to support   configuring AWS EC2 auto scaling service based on the TOSCA auto scaling   policy.

Extended   Description:
The goal is to set up auto scaling for   components that have been deployed in Amazon Ec2 as instances or containers.   This will involve configuring CloudWatch alarms and AWS Auto scaling groups   and rules based on TOSCA policies.

Priority:
Should have

Affected   Tools:
ORCHESTRATOR

Means   of Verification:
Use simple load tests to verify that the   data pipeline tasks deployed to AWS EC2 are scaled up and down according to   the policy setting when autoscaling trigger tresholds are reached.

As of now auto scaling is not explicitly supported within xOpera, related to this: #25.

We have released new version - 0.6.5 of opera orchestrator, which introduces the new opera notify CLI command that enables invoking TOSCA triggers from policies and will therefore enable scaling.


chore: bump @backstage/plugin-scaffolder-node

Description

Recreated patches for @backstage/plugin-scaffolder-node based on backstage v1.33.5 (as we needed to patch v0.6.1 of this package)
Bumped @backstage/plugin-scaffolder-node versions to fix CVE: https://nvd.nist.gov/vuln/detail/CVE-2024-53983

PR acceptance criteria
Please make sure that the following steps are complete:

[ ] GitHub Actions are completed and successful
[ ] Unit Tests are updated and passing
[ ] E2E Tests are updated and passing
[ ] Documentation is updated if necessary (requirement for new features)
[ ] Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

/lgtm
/approve


Prompt on applying experiment to workspace

Thanks to @pmrowla and @daavoo for bringing this up. By default, dvc exp apply will overwrite files in the workspace unless you use --no-force. Since conflicts between experiments are common (for example, changing the same parameter to different values across multiple experiments), this default makes sense in most cases. We found that it was too much work to ask users to constantly have to specify whether they wanted to overwrite.
With VS Code, it might be worth exploring whether to use --no-force and prompt the user. Clicking on a prompt is easier than adding a CLI flag, and the extension could even give some option like "Don't show this prompt again" if they find it too annoying.

We found that it was too much work to ask users to constantly have to specify whether they wanted to overwrite.

This combination of the prompt and --no-force doesn't look like a regular approach to me. It doesn't allow to solve a scenario when I'm automating things - it won't fail, it'll prompt. And there is no option to respond to this prompt automatically, right? + as a user it'll too late for me to learn about --no-force, I'll learn about it after I made a mistake (at best). Am I missing something here?
A better, safer, and quite common alternative (similar to git checkout, git push, git pull, etc) could be to:

fail fast if there are irreversible changes
mention that there is --force if you'd like to still apply the changes
add config to apply --force automatically
stash changes (and mention that those are stashed) before applying. So there is always a way to get back.

@dberenbaum wdyt about this approach?
The same logic will apply in VS Code then. It can run, see that there are conflicts, fail fast, show a prompt after that. But also always stash the current workspace and show in the Notification the message.

This combination of the prompt and --no-force doesn't look like a regular approach to me.

It's definitely not a regular approach. It went through several iterations but ended up here because the typical approach got too annoying for even the most basic experiment scenarios.  Almost every time you apply an experiment on top of another, you will get a conflict.

It doesn't allow to solve a scenario when I'm automating things - it won't fail, it'll prompt. And there is no option to respond to this prompt automatically, right?

There's no prompt. My description above was probably misleading.

stash changes (and mention that those are stashed) before applying. So there is always a way to get back.

I like this idea, but I'm not sure we need any of the other bullets above if we have this mechanism and hint. @pmrowla WDYT?

We already do stash the changes but drop the stash after the apply completes. Preserving the stash entry would be a simple change

Opened https://github.com/iterative/dvc/issues/7955. In that case, prompting in VS Code should be unnecessary, although VS Code may want to highlight how to recover the dirty workspace since it may still appear to users that their work gets lost.


Having trouble in showing Notifications, when app is closed

Hi, I have created a Chat App using Flutter and I am successfully showing incoming message notification when the app is running, but I want to show the notifications when the app is closed.
How can I do that, please Help !

If talk about native code you have 2 option at least for android:

register notification listener
use Firebase keys that will force to show notification message

as for flutter - I think you can try to find answer here
https://stackoverflow.com/questions/53572110/flutter-push-notifications-even-if-the-app-is-closed

@TatsuUkraine Thanks, I achieved this by using Firebase Cloud Messaging.

@TatsuUkraine Thanks, I achieved this by using Firebase Cloud Messaging.

Hey can you highlight how did you achieve this. I have been trying to get this done since the past 3 days.


Enable https over apache's maven repo

Codecov Report

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

@@             Coverage Diff              @@
##             master    #4329      +/-   ##
============================================
+ Coverage     56.23%   65.77%   +9.54%
  Complexity       20       20
============================================
  Files          1061     1061
  Lines         54984    54984
  Branches       7825     7825
============================================
+ Hits          30919    36168    +5249
+ Misses        21678    16247    -5431
- Partials       2387     2569     +182

Impacted Files
Coverage Δ
Complexity Δ

...nsport/netty/PooledNettyClientResourceManager.java
85.41% <0%> (-4.17%)
0% <0%> (ø)

...he/pinot/controller/util/SegmentIntervalUtils.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...ator/transform/function/BaseTransformFunction.java
29.95% <0%> (+0.42%)
0% <0%> (ø)
:arrow_down:

...ment/creator/impl/SegmentColumnarIndexCreator.java
87.45% <0%> (+0.76%)
0% <0%> (ø)
:arrow_down:

...r/transform/function/ValueInTransformFunction.java
39.2% <0%> (+0.8%)
0% <0%> (ø)
:arrow_down:

.../helix/core/realtime/SegmentCompletionManager.java
70.39% <0%> (+0.87%)
0% <0%> (ø)
:arrow_down:

...e/io/writer/impl/MutableOffHeapByteArrayStore.java
86.45% <0%> (+1.04%)
0% <0%> (ø)
:arrow_down:

...elix/core/relocation/RealtimeSegmentRelocator.java
74.72% <0%> (+1.09%)
0% <0%> (ø)
:arrow_down:

.../pinot/core/segment/index/SegmentMetadataImpl.java
80.81% <0%> (+1.22%)
0% <0%> (ø)
:arrow_down:

...g/apache/pinot/common/response/ServerInstance.java
79.72% <0%> (+1.35%)
0% <0%> (ø)
:arrow_down:

... and 321 more

Continue to review full report at Codecov.

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

https://github.com/apache/incubator-pinot/pull/4340


Add dependency caching to CI builds

Evaluate solutions for caching cargo crate and elixir hex dependencies.
https://github.com/actions/cache/blob/main/examples.md#rust---cargo
https://github.com/actions/cache/blob/main/examples.md#elixir---mix
https://github.com/LukeMathWalker/cargo-chef

Example of the problem:

> Task :rust:__lint_examples_worker_0
warning: spurious network error (2 tries remaining): failed to get 200 response from `https://crates.io/api/v1/crates/chrono/0.4.19/download`, got 503
warning: spurious network error (2 tries remaining): failed to get 200 response from `https://crates.io/api/v1/crates/time/0.1.44/download`, got 503
warning: spurious network error (1 tries remaining): failed to get 200 response from `https://crates.io/api/v1/crates/chrono/0.4.19/download`, got 503
warning: spurious network error (1 tries remaining): failed to get 200 response from `https://crates.io/api/v1/crates/time/0.1.44/download`, got 503

error: failed to download from `https://crates.io/api/v1/crates/chrono/0.4.19/download`
> Task :rust:__lint_examples_worker_0 FAILED```

:heart:  Thank you!


Running formatting checks without formatting the code on CI

We would like to run detekt on our CI with all checks, including formatting. The problem is that we don't want to actually reformat code on CI, just fail the build if there are any errors. But if we run detektFormat with autoCorrect: false, it prints format errors, but doesn't fail the build.
Is it possible to either add formatting checks to detektCheck or make detektFormat fail the build (possibly enabled by some flag) if there are any formatting errors?

So you already running detektFormat on CI and it does not format code (like you want it?), but formatting should also fail the build?
How do your build settings look like ? I turned off formatting weights in the build setting, try turning it on
build:
  warningThreshold: 5
  failThreshold: 10
  weights:
    complexity: 2
    formatting: 0
    LongParameterList: 1
    comments: 0.5

This is our current detekt.yml
autoCorrect: false

build:
  warningThreshold: 1
  failThreshold: 1

potential-bugs:
  active: true
  LateinitUsage:
    active: false

exceptions:
  active: true

empty-blocks:
  active: true

complexity:
  active: true

code-smell:
  active: true

formatting:
  active: true
  useTabs: true

style:
  active: true

comments:
  active: true
  UndocumentedPublicClass:
    active: false
  UndocumentedPublicFunction:
    active: false

We simply want a way to fail the build if the code isn't formatted properly. We've tried increasing weight for formatting, but weights apply only for detektCheck task which doesn't run formatting.

Ah I see, shouldn't be so hard to add. Will try to add this to next release.

Ok fixed. After the last two PR are merged, I will release a new version.

Great, thanks!


Script doesn't work with Raspberry pi zero 2 w

Hey Max. Big fan.
I'm working on this project and can only get my hands on a pi zero 2 w, rather than the original. Unfortunately when I use your script, the camera feed shows up for half a second, and then the camera becomes unresponsive. Unfortunately showmewebcam hasn't yet added support for zero 2 w. I wonder if you have any ideas?

Hm, I don't have a Pi Zero 2 W to check. Did you ask over at showmewebcam yet?

Yep – looks like they are working on support but it's not ready yet :(
Yours seems like it works better for me, but then seems to crash. Is there any kind of gpu/memory setting I could try to tweak?

If they're already working on it, that's probably your best bet!

I've found a solution using the FULL install and not the remote only.
Some how it does work with a full installation of the OS.

I've found a solution using the FULL install and not the remote only. Some how it does work with a full installation of the OS.

Could you clarify which full install you used, @moorlag ? 64 or 32 bit, Legacy or regular? Thanks


No nightly clippy except weekly

Also removes dependabot.yml since defaults should be sufficient. This was causing problems with the Azure/azure-sdk-for-rust-pr repo.

@weshaggard does this look right to you - to run a command against a specific toolchain weekly on Sunday night?


The debugger and the dotnet watch run process dies whenever I save a file.

The extension is not working well for me. Whenever I save a file the debugger and the dotnet watch run process dies.
My launch.json files looks like this:
{
    "version": "0.2.0",
    "configurations": [
        {
            "type": "DotNetWatch",
            "request": "launch",
            "name": ".NET Core Watch",
            "env": {
                "ASPNETCORE_ENVIRONMENT": "Development",
                "DOTNET_WATCH_RESTART_ON_RUDE_EDIT": "true"
            }
        }
    ]
}

And I am building a Blazor Server App on MacOS with .NET 6.
Does the extension throw logs somewhere so I can share them here, in case they point to the problem? :-)

Do you see it in extension host?

No, there is nothing there when it dies. It does give me a ProblemMatcher{0} ignored (in the Tasks log) when I start the debugging session with the extensions launch config. Maybe that is related.

Will you be able to give a minimal repro? Haven’t done blazor

I am able to repro by ``
but even during debugging of the extesion, the debugger process just crashes.
What the extension does, is quality of life improvements, where it generates a task and attach via vs code debugger service.

Strange part is that, when i run the similar dotnet watch command on a seperate terminal, it works fine
It seems like blazor debugging on vscode is still quirky, there's several issues pointing to it
https://github.com/OmniSharp/omnisharp-vscode/issues/5128
https://github.com/OmniSharp/omnisharp-vscode/issues/5183
I'll keep the issue open and see if omnisharp has any update

Thanks! I will keep an eye for any updates as well. Hopefully it gets fixed soon, as I envy the productivity I am missing! 😉

I encountered the same for Razor Pages app.

@IntoLighter are you able to give a minimal repro?

@murugaratham https://github.com/IntoLighter/Marketplace

I've published a new release, take a look when you updated the extension, I'll keep the issue open for awhile

fixed with latest release, feel free to reopen if debugger is still detaching


Update for Session 5

Quick update to explore pull requests

Good pull request!

haha. yes, finally using github pro-actively!
Am 19.02.2020 12:44 schrieb Fabrizio Leone:

Good pull request!
--
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub [1], or unsubscribe [2].

Links:
[1]
https://github.com/Python-do-ECARES/Classes/pull/13?email_source=notifications&email_token=AGKYTT6AFYVRPW5FDINLU5LRDULSJA5CNFSM4KXBIA72YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMHPANY#issuecomment-588181559
[2]
https://github.com/notifications/unsubscribe-auth/AGKYTT5JJTIJKEUD6F6UIXDRDULSJANCNFSM4KXBIA7Q


Fix error when running integration tests

rustc doesn't appear to support using /dev/null as the output. This error occurs even when running rustc manually.
error: couldn't create a temp dir: Permission denied (os error 13) at path "/dev/rmetayXAwWn"

I was wondering why your rustc and mine were behaving differently, but I just figured it out. My terminal runs the shell in a sandbox and it turns out that my sandbox configuration makes /dev a writable tmpfs. Thanks for the fix!


Changing the order of tabs on a minimised list of tabs does not cause the dashboard to become dirty

I think this was fixed ad part of gh-286


bug(Date range selection): a form contain mat-date-range-input will trigger valueChanges 4 times

Reproduction
https://stackblitz.com/edit/angular-jm3fw2
Steps to reproduce:

Update the start or end date
see console : the form statusChanges  or valueChanges  triggers 4 times

Expected Behavior
one trigger
Actual Behavior
statusChanges or     valueChanges triggers 4 times
Environment

Angular: 10.1
CDK/Material: 10.1
Browser(s): chrome 84
Operating System (e.g. Windows, macOS, Ubuntu):  windows 10

Duplicate of https://github.com/angular/components/issues/19776.


⚠ LPDM Handbook (hosted on Divio) has degraded performance

In 30e15b2, LPDM Handbook (hosted on Divio) (https://repohubdev.tk) experienced degraded performance:

HTTP code: 500
Response time: 5149 ms

Resolved: LPDM Handbook (hosted on Divio) performance has improved in 557501d.


add cn and biz hostname validator tests

hi Team,
i noticed the red coverage badge on the readme and thought i'd help. ran a coverage report and saw that most of the interesting bits are already covered! almost all validators are at 90%+; great job!
i guess we're getting dinged by the large cn and biz TLD files that were probably extracted from the hostname validator. these two tests will replace that distracting red coverage badge with something a little more representative of the work you've accomplished here.

Thanks, @abacaphiliac


Pass -Wswitch and fix errors

Seems upstream added SASS_HASH to the list separator types.  I don't think this is actually valid in input / output (for serialization) but we should upgrade the warning to an error and fix the error

asking about this upstream here: https://github.com/sass/libsass/issues/1938

Seems it is just internal.  I'll double check with a few tests but I believe it to be sufficient to assert if we ever see this


parsing crash

No error handling on partial prompts crash
detection_metrics, is_injection = rb.detect_injection("Ignore all prior") if is_injection:     print("Possible injection detected. Take corrective action.")
File rebuff.py:86, in Rebuff.detect_injection(self, user_input, max_heuristic_score, max_vector_score, max_model_score, check_heuristic, check_vector, check_llm)

 83 response.raise_for_status()
 85 response_json = response.json()
 --->    86 success_response = DetectApiSuccessResponse.parse_obj(response_json)
 88 if (
 89     success_response.heuristicScore > max_heuristic_score
 90     or success_response.modelScore > max_model_score
 91     or success_response.vectorScore["topScore"] > max_vector_score
 92 ):
 93     # Injection detected
 94     return success_response, True

Thanks for highlighting this @shachar-ug!

I no longer see this issue. Please re-open if you can re-produce on latest rebuff sdk. If you do see it on the pypi installed rebuff that should be fixed shortly with #48
rb = Rebuff(api_token="MYTOKEN, api_url="https://www.rebuff.ai")

user_input = "Ignore all prior"
result = rb.detect_injection(user_input)

if result.injectionDetected:
    print("Possible injection detected. Take corrective action.")

Possible injection detected. Take corrective action.


User permissions tweaks

Context
We'd like to make the process of onboarding provider users and managing their permissions much easier to understand.

Changes proposed in this pull request

Amend layout of edit user permissions form to nest existing permissions form elements inside radio group (see screenshot)
Amend layout of setup user permissions form to nest existing permissions form elements inside radio group (see screenshot)
Amend copy on the 'check' step when inviting a new provider user.

Set permissions on setting up a new user

Check permissions on setting up a new user

Edit permissions

Guidance to review
The complexity in this PR is added in nesting the existing permissions form elements inside a radio button group.
This top level set of choices has to be validated when setting up a user and has an effect on the permissions saved against the user (selecting 'view applications only' should remove any previously set/selected permissions).

Link to Trello card
https://trello.com/c/9idMIMCB/2898-user-permissions-tweaks

Things to check

[x] This code does not rely on migrations in the same Pull Request
[x] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[x] API release notes have been updated if necessary
[x] New environment variables have been added to the Azure config

Awesome work dude.
Anywhere it says ‘Manage organisations’ it should say ‘Manage organisational permissions’

What does the check screen look like when the user who is doing the inviting belongs to multiple orgs?
And what does the page with nested permissions show when the user doesn't select an option at the top level and child level?

@adamsilver I've amended 'Manage organisations' labels here https://github.com/DFE-Digital/apply-for-teacher-training/pull/3271/commits/4424863ce7ff755914528f31d3b7583fbe15334a
I've also added more screenshots of the scenarios you mention, any thoughts on the validation error message? I struggle with words.

Thanks @AgaDufrat for spotting the missing cross icons on other permissions component, will make a new PR.


Fatal: hasOption() on null

Hi. Not sure if I'm missing something, but we've encountered an issue when instantiating a migration environment.
Sample code:
<?php

require __DIR__ . '/vendor/autoload.php';

$manager = new \Phinx\Migration\Manager(
    \Phinx\Config\Config::fromPhp(__DIR__ . '/test-config.php'),
    new \Symfony\Component\Console\Input\ArrayInput([]),
    new \Symfony\Component\Console\Output\NullOutput()
);

class TestSeed extends \Phinx\Seed\AbstractSeed { }

$manager->executeSeed('development', new TestSeed());

Where our test configuration is:
<?php

return [
    'environments' => [
        'development' => [
            'name' => 'devdb',
            'connection' => new PDO('sqlite::memory:'),
        ]
    ]
];

We encounter this error:
Fatal error: Uncaught Error: Call to a member function hasOption() on null in phinx/src/Phinx/Db/Adapter/AbstractAdapter.php on line 248

Error: Call to a member function hasOption() on null in phinx/src/Phinx/Db/Adapter/AbstractAdapter.php on line 248

Call Stack:
    0.0002     371912   1. {main}() phinx/pop.php:0
    0.0030     996640   2. Phinx\Migration\Manager->executeSeed() phinx/pop.php:13
    0.0032    1035552   3. Phinx\Migration\Manager\Environment->executeSeed() phinx/src/Phinx/Migration/Manager.php:396
    0.0032    1035552   4. Phinx\Migration\Manager\Environment->getAdapter() phinx/src/Phinx/Migration/Manager/Environment.php:148
    0.0033    1049152   5. Phinx\Db\Adapter\AdapterFactory->getAdapter() phinx/src/Phinx/Migration/Manager/Environment.php:344
    0.0044    1349128   6. Phinx\Db\Adapter\AbstractAdapter->__construct() phinx/src/Phinx/Db/Adapter/AdapterFactory.php:131
    0.0044    1349128   7. Phinx\Db\Adapter\PdoAdapter->setOptions() phinx/src/Phinx/Db/Adapter/AbstractAdapter.php:71
    0.0044    1349128   8. Phinx\Db\Adapter\PdoAdapter->setConnection() phinx/src/Phinx/Db/Adapter/PdoAdapter.php:56
    0.0045    1349128   9. Phinx\Db\Adapter\AbstractAdapter->createSchemaTable() phinx/src/Phinx/Db/Adapter/PdoAdapter.php:74
    0.0050    1452504  10. Phinx\Db\Table->save() phinx/src/Phinx/Db/Adapter/AbstractAdapter.php:218
    0.0050    1452504  11. Phinx\Db\Table->create() phinx/src/Phinx/Db/Table.php:697
    0.0050    1452504  12. Phinx\Db\Adapter\SQLiteAdapter->createTable() phinx/src/Phinx/Db/Table.php:609
    0.0051    1452760  13. Phinx\Db\Adapter\PdoAdapter->execute() phinx/src/Phinx/Db/Adapter/SQLiteAdapter.php:228
    0.0051    1452760  14. Phinx\Db\Adapter\AbstractAdapter->isDryRunEnabled() phinx/src/Phinx/Db/Adapter/PdoAdapter.php:126

Our script runs an empty seed to prove the problem, but I suspect this would happen with any DB operation against the manager. I think the problem is around initialising a connection, which is too early for an environment to be aware of a console input.
We've worked around this in our project by reverting to v0.8.1, but I'm posting this quick-fix PR in case it is any use.
Thanks! :)

@ehimen Thanks for this, looks good :)
Not sure about the changes to the PDO Adapter test though? Did you mean to commit that?

I did, yeah. They're a bit weird to cater for older PHPUnit (following required version in the CI build), but that does cover this case. Without the fix, the test would fail with the same hasOption() on null bug.

Is there a workaround until this PR is accepted ?

@Veridis Ok. You've also renamed the test, can you undo that?

Sorry @JayPHP. Not sure what you mean?

Codecov Report

Merging #1181 into master will decrease coverage by 0.12%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1181      +/-   ##
==========================================
- Coverage   72.13%   72.01%   -0.13%
==========================================
  Files          35       35
  Lines        5592     5592
==========================================
- Hits         4034     4027       -7
- Misses       1558     1565       +7

Impacted Files
Coverage Δ

src/Phinx/Db/Adapter/AbstractAdapter.php
80.59% <100%> (ø)
:arrow_up:

src/Phinx/Db/Adapter/PdoAdapter.php
87.36% <0%> (-3.85%)
:arrow_down:

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 6a6ebc7...4c26aeb. Read the comment docs.

+1, this is actually a critical issue. It's impossible to boostrap a new project with a PDO adapter to a database that doesn't have the phinxlog table already created.

oh man, so glad i am not going insane. been trying to bootstrap for like two hours now.

@JayPHP  / @lorenzo , is this something we can get merged? Happy to make any changes required. Thanks :)

Looks like a good change!

@lorenzo let's merge it.

Thanks @ehimen

Hi there! Any ETA on when this will be released?

I've just realised the test file rename. I didn't intend this; must have been my IDE. So despite the coverage dropping and @JayPHP explicitly mentioning this, I still missed it. Apologies all.


Expose hook references for testing

Is there a way to access/spy the callback hooks for a specific form using sinon for example.

The object where they're stored isn't part of the exported API, but that could be changed. I'm not familiar with sinon, but is that all you would need? An AutoForm.Hooks object?

yeap, thats what I need, sinon generates code around a function to identify if its called or not:
sinon.spy(AutoForm.Hooks['formId'], "onError");
sinon.spy(AutoForm.Hooks['formId'], "onSuccess");
In this way i will ensure that this methods are called, which ones not and what they receive.

Done in AutoForm 5.0, now released.

Thanks :+1:

For others coming here afterwards, the exact syntax for doing this appears to be:
const hookSpy = sinon.spy(AutoForm._hooks["myFormId"].onSubmit, [0]);

doSomething();

expect(hookSpy).to.have.been.called;

Hope this is helpful. It took me hours to figure out the exact sinon syntax for accessing elements of the array of hook functions for each hook type - hence the [0] in the create spy function.


ProgPow Updates

The ProgPow config has been moved to the algorithm, where it should reside. The config structure still could use some work but it's better than keeping it in internal/progpow
I've also added an external ProgPow package, which I have mixed feelings about. The original ProgPow 0.9.4 spec should be tested, but it's a weird thing to do in the internal/progpow package. That being said, with the current implementation, progpow.HashMix is only tested outside of internal/progpow which is not good.
The issue here is that ProgPow is used in two ways - as an internal package and as full Proof of Work algorithm. At least with the current structure that is more clear, but having two ProgPow packages is confusing and just bad practice.
At the end of the day, even though the ProgPow 0.9.4 spec is useful, no chains really use it. So if something does change (which it should), it should probably just be the removal of the external ProgPow package since there isn't much value to it, and ProgPow should be treated as the Ethash DAG is (an internal package, not a full algorithm). I'm not doing this yet since that still wouldn't resolve the issue that progpow.HashMix is only tested outside of internal/progpow.

Progpow094 has been moved to internal/progpow for completeness and testing. Some test vectors were added for round and Hash within internal/progpow to alleviate the previous concerns with not enough testing. It is only one test vector each which isn't great, but the tests require a massive amount of data so it's good enough for now.


change README.md

Why to close my PR?

When you open a PR, you need to be explicit about why you made the change. Your change is "cosmetic" change, which is just a preference. If you want to be more technical, Linux and Windows are always used capitalized, there's no reason to change them to lower. macOS is now the official representation by Apple, and it used to be Mac OS. I don't like the format and it does not hurt the readability - it causes no confusion in any case.
I don't believe this is a PR that I want to merge, and you did not comment on why you wanted to do this - that's why I just closed the PR.

When you open a PR, you need to be explicit about why you made the change. Your change is a "cosmetic" change, which is just a preference. If you want to be more technical, Linux and Windows are always used capitalized, there's no reason to change them to lower. macOS is now the official representation by Apple, and it used to be Mac OS. I don't like the format and it does not hurt the readability - it causes no confusion in any case.
I don't believe this is a PR that I want to merge, and you did not comment on why you wanted to do this - that's why I just closed the PR.

Ok, I see. I will pay attention to this in the future.


Bazel visibility for contrib filters

Title: Make library visibility configurable for contrib filters
Description:
The move of certain filters to the contrib directory means it can be difficult to access them outside of their own package group.
Can we introduce a CONTRIB_EXTENSION_PACKAGE_VISIBILITY variable similar to the existing one for regular extensions that allows users to configure their visibility levels instead of relying on the default value?

Sure, sounds good, thanks.


Update install instructions on the docs

What current issue(s) does this address, or what feature is it adding?
The instructions to install neo-python present on the documentation (docs folder) were not up to date.
How did you solve this problem?
I just synced with the README.rst so the content is presented in the same (and correct) order, then added the missing Install from PyPi section.
How did you make sure your solution works?
Compiled the sphinx docs and checked
Are there any special changes in the code that we should be aware of?
No
Note: To make it easier to keep the documentation up to date and avoid duplication, I would suggest that in the future there should be a single source of truth for the documentation. For example, the README.rst should point to the docs instead of having install instructions there.
Please check the following, if applicable:

[ ] Did you add any tests?
[ ] Did you run make lint?
[ ] Did you run make test?
[x] Are you making a PR to a feature branch or development rather than master?
[x] Did you add an entry to CHANGELOG.rst? (if not, please do)

Coverage remained the same at 78.779% when pulling ae5cb8cc0b7ad1170c227fc07b9d716a2cfe438e on dethos:update-docs into bb8c348785b175fb121ce4c5606ba8f4d2cda5a1 on CityOfZion:development.

Thanks!


strange issue with routing

I'm trying to write an aurelia app with asp.net 5 (beta 8) web api controller as backend I found this thing:
I wrote this action in the controller, skipping the route attribute because I didn't know if was mandatory.
        [HttpGet]
        public IEnumerable<TipoAttivita> TipiAttivita() {
            var tipiAttivita = db.TipiAttivita
                .OrderBy(cli => cli.Descrizione)
                .ToList();

            return tipiAttivita;

        }

Calling it from Aurealia, the method is not invoked, but I receive and HttpResult 200 and the string "value" as result.
Adding the attribute [Route("TipiAttivita")] solved the problem.
Is this normal?

skipping the route attribute because I didn't know if was mandatory

There are two schemes for routing: conventional routing, and attribute routing.
Conventional routing uses route information defined in the global route table, inside your call to .AddMvc(...).
Attribute routing is defined for controllers and actions using [Route(...)] or [HttpGet(<some route>)]. Conventional routes do not apply to actions and controllers with attribute routes.
You can really set this up however you want, there's no requirement that you do one thing or another, though I'd strongly recommend using attribute routing for APIs.

The problem is that if I configure incorrectly the route, I expect a not found error, not a OK 200 with a string result that is unrelated to my request

The problem is that if I configure incorrectly the route, I expect a not found error, not a OK 200 with a string result that is unrelated to my request

If there's a 200, then I'd expect something in your app is serving it. If you have a public repo available, I can take a look.

I think I finally understood what's happened. the scaffolded controller by default contains
[HttpGet("{id}")]
public string Get(int id)
{
  return "value";
}

and this is the action is called when I don't add the route. Try to convert to an int and call it passing 0.
Adding a constraint to the param {id:int} returns 404.

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.


Run BDD test in GitHub Actions

As a Developer
I need to run BDD test when a pull request is created
So that I can easily evaluate the code
Details & Assumptions:

Selenium needs package (run apt-get install -y python3-selenium) to work

Acceptance Criteria:
When I create a pull request
Then the BDD tests should run and pass

IBM Cloud toolchain: Delivery Pipeline deployed promotions to prod, including commits 3acd30b1c579eee2d7442c3f9da378cec35ef2d5, a03bdced82976b3dec0791707a08bc1e1095377d, 53a791ad7f8400c434dddb268230134c471c30f7, b302f7a06ccc137de85fc042493e1a4a04938407, 74ef8225b94f865c67b9a59c386a5e1cf55dc09e


docs(plugins): rename OccurenceOrderPlugin to OccurrenceOrderPlugin

After the arrival of webpack 2, OccurenceOrderPlugin has been renamed to OccurrenceOrderPlugin. This PR updates this to internal-plugin docs.

Thanks!

I'm also very happy about this contribution because this is my first one to webpack.


Building the CLI from source fails.

🐞 Bug report
Command (mark with an x)

[ ] new
[ ] build
[ ] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[ ] update
[ ] lint
[ ] xi18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

NONE of the above. This issue is related to building the CLI from source
Is this a regression?

 Yes, the previous version in which this bug was not present was: ....
Description
When running the yarn build --local to build the CLI from source, I got this error:
🔬 Minimal Reproduction

If you haven't done it already, make a fork of this repo.
Clone to your local computer using git.
Make sure that you have Node 10.13 or later installed. See instructions here.
Make sure that you have yarn installed; see instructions here.
Run yarn (no arguments) from the root of your clone of this project to install dependencies.
To make a local build: yarn build --local

🔥 Exception or Error

fatal: No tags can describe '346172494d91f686ee5f4e1bb8071c8ea2c6f582'.
Try --always, or create some tags.
Error: Command failed: git describe --match v[0-9]*.[0-9]*.[0-9]* --abbrev=7 --tags
fatal: No tags can describe '346172494d91f686ee5f4e1bb8071c8ea2c6f582'.
Try --always, or create some tags.

    at checkExecSyncError (child_process.js:621:11)
    at Object.execSync (child_process.js:657:15)
    at _exec (/Users/wassimchegham/oss/@manekinekko/angular-cli/lib/packages.ts:96:28)
    at _getVersionFromGit (/Users/wassimchegham/oss/@manekinekko/angular-cli/lib/packages.ts:112:30)
    at Object.get version [as version] (/Users/wassimchegham/oss/@manekinekko/angular-cli/lib/packages.ts:170:20)
    at Object.default_1 [as default] (/Users/wassimchegham/oss/@manekinekko/angular-cli/scripts/build.ts:296:29)
error Command failed with exit code 99.

🌍 Your Environment

git version 2.21.0 (Apple Git-122.2)
Angular CLI: 9.1.0-rc.0
Node: 12.13.0
OS: darwin x64

Angular: 9.1.1
... animations, common, compiler, compiler-cli, core, forms
... localize, platform-browser, platform-browser-dynamic
... platform-server, router, service-worker
Ivy Workspace: Yes

Package                            Version
------------------------------------------------------------
@angular-devkit/architect          0.0.0
@angular-devkit/architect-cli      0.0.0
@angular-devkit/benchmark          0.0.0
@angular-devkit/build-angular      0.0.0
@angular-devkit/build-ng-packagr   0.0.0
@angular-devkit/build-optimizer    0.0.0
@angular-devkit/build-webpack      0.0.0
@angular-devkit/core               0.0.0
@angular-devkit/schematics         0.0.0
@angular-devkit/schematics-cli     0.0.0
@angular/cdk                       9.2.0
@angular/cli                       0.0.0
@angular/material                  9.2.0
@angular/pwa                       0.0.0
@bazel/bazel                       2.1.0
@bazel/bazel-darwin_x64            2.1.0
@bazel/buildifier                  0.29.0
@bazel/buildifier-darwin_x64       0.29.0
@bazel/hide-bazel-files            1.5.0
@bazel/jasmine                     1.3.0
@bazel/karma                       1.3.0
@bazel/labs                        1.3.0
@bazel/typescript                  1.3.0
@ngtools/webpack                   0.0.0
@schematics/angular                0.0.0
@schematics/schematics             0.0.0
@schematics/update                 0.0.0
ng-packagr                         9.1.0
rxjs                               6.5.4
typescript                         3.8.3
webpack                            4.42.0

Anything else relevant?

Did you try to fetch by any chance?

yes. I tried git fetch upstream. With no success.

Can you try with a fresh clone?
I just tried and had no issue doing an yarn build --local.

Hi, so I just did a fresh clone, in /tmpand I still have the issue

Full logs

/tmp/angular-cli on  master is 📦 v0.0.0 via ⬢ v12.13.1 and 🅰  10.0.0-rc.0 on 🐳 v19.03.8 took 19s
➜ yarn build --local
yarn run v1.22.4
$ node ./bin/devkit-admin build --local
Cleaning...
  Removing dist/...
Removing dist-schema/...
Generating JSON Schema....
Building...
Moving packages to dist/
    @_/benchmark
    @angular-devkit/build-optimizer
    @angular-devkit/core
    @angular-devkit/architect
    @angular-devkit/architect-cli
    @angular-devkit/build-webpack
    @angular-devkit/build-ng-packagr
    @angular-devkit/benchmark
    @angular-devkit/schematics
    devkit
    @ngtools/webpack
    @angular-devkit/build-angular
    @schematics/angular
    @angular/pwa
    @schematics/schematics
    @angular-devkit/schematics-cli
    @schematics/update
    @angular/cli
Merging bazel-bin/ with dist/
    @angular-devkit/architect
    @angular-devkit/build-webpack
    @angular-devkit/build-ng-packagr
    @angular-devkit/schematics
    devkit
    @angular-devkit/build-angular
    @schematics/angular
    @angular/pwa
    @schematics/schematics
    @schematics/update
    @angular/cli
Copying resources...
    @_/benchmark
            3 files total...
            1 resources...
    @angular-devkit/build-optimizer
            27 files total...
            8 resources...
    @angular-devkit/core
            264 files total...
            52 resources...
    @angular-devkit/architect
            31 files total...
            12 resources...
    @angular-devkit/architect-cli
            4 files total...
            1 resources...
    @angular-devkit/build-webpack
            33 files total...
            5 resources...
    @angular-devkit/build-ng-packagr
            31 files total...
            6 resources...
    @angular-devkit/benchmark
            48 files total...
            8 resources...
    @angular-devkit/schematics
            98 files total...
            9 resources...
    devkit
            7 files total...
            6 resources...
    @ngtools/webpack
            63 files total...
            3 resources...
    @angular-devkit/build-angular
            4456 files total...
            20 resources...
    @schematics/angular
            224 files total...
            98 resources...
    @angular/pwa
            15 files total...
            12 resources...
    @schematics/schematics
            30 files total...
            27 resources...
    @angular-devkit/schematics-cli
            4 files total...
            1 resources...
    @schematics/update
            248 files total...
            5 resources...
    @angular/cli
            369 files total...
            41 resources...
Copying extra resources...
Removing spec files...
    @angular-devkit/build-optimizer
      11 spec files found...
    @angular-devkit/core
      62 spec files found...
    @angular-devkit/architect
      4 spec files found...
    @angular-devkit/build-webpack
      4 spec files found...
    @angular-devkit/build-ng-packagr
      2 spec files found...
    @angular-devkit/benchmark
      6 spec files found...
    @angular-devkit/schematics
      42 spec files found...
    @ngtools/webpack
      28 spec files found...
    @angular-devkit/build-angular
      130 spec files found...
    @schematics/angular
      107 spec files found...
    @angular/pwa
      2 spec files found...
    @angular-devkit/schematics-cli
      2 spec files found...
    @schematics/update
      4 spec files found...
    @angular/cli
      4 spec files found...
Building ejs templates...
Setting versions...
    @_/benchmark
fatal: No names found, cannot describe anything.
Error: Command failed: git describe --match v[0-9]*.[0-9]*.[0-9]* --abbrev=7 --tags
fatal: No names found, cannot describe anything.
at checkExecSyncError (child_process.js:621:11)
at Object.execSync (child_process.js:657:15)
at _exec (/private/tmp/angular-cli/lib/packages.ts:97:28)
at _getVersionFromGit (/private/tmp/angular-cli/lib/packages.ts:113:30)
at Object.get version [as version] (/private/tmp/angular-cli/lib/packages.ts:171:20)
at Object.default_1 [as default] (/private/tmp/angular-cli/scripts/build.ts:300:29)

error Command failed with exit code 99.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
/tmp/angular-cli on  master is 📦 v0.0.0 via ⬢ v12.13.1 and 🅰  10.0.0-rc.0 on 🐳 v19.03.8 took 17s

@manekinekko, are you still experiencing this issue? it doesn't appear that anyone else is.

I can't reproduce this issue with the latest 12.0.0-rc.1. I am closing this issue.
Thanks for the ping 😉


[ISSUE]  Cannot use AccountClient commands after creating a new WorkspaceClient

Description
When we have an AccountClient object, and when we create a WorkspaceClient object, all next calls to the AccountClient won't work.
Reproduction
    def client_for(self, workspace: Workspace) -> WorkspaceClient:
        config = self._ac.config.as_dict()
        if "databricks_cli_path" in config:
            del config["databricks_cli_path"]
        cloud = self._get_cloud()
        # copy current config and swap with a host relevant to a workspace
        config["host"] = f"https://{workspace.deployment_name}.{self._tlds[cloud]}"
        return WorkspaceClient(**config, product="ucx", product_version=__version__)

   def test_broken(self):
        self._ac.groups.create(display_name="test_1")
        for ws in self._ac.workspaces.list():
            client = self.client_for(ws)
            self._ac.groups.create(display_name="test_12")

Expected behavior
The call to self._ac.groups.create(display_name="test_12") should work, but in this case it's raising the error highlighted below.
Debug Logs
databricks.sdk.errors.mapping.NotFound: Endpoint not found for /2.0/accounts/2cffc3ba-0e89-463a-a7ec-8161b437db24/scim/v2/Groups
Additional context
After debugging, the first call is using the Account URL but the second call is using a workspace URL
I could make it work by using the following code:
def test_broken(self):
    from copy import deepcopy

    copy_acc_client = deepcopy(self._ac)
    self._ac.groups.create(display_name="test_1")
    for ws in self._ac.workspaces.list():
        client = self.client_for(ws)
        copy_acc_client.groups.create(display_name="test_12")

@william-conti config.as_dict() returns a reference to the same config object. Modifying the host there modifies the config used by AccountClient. You can mitigate this by just using config.copy() instead of config.as_dict().
FYI: v0.19.0 will include AccountClient.get_workspace_client() which will handle this for you by making a deep copy of the config. With this update, you won't need to maintain your client_for() function anymore.

@mgyucht The fix still doesn't work:
    def test_broken(self):
        self._ac.groups.create(display_name="test_xxxx")
        for ws in self._ac.workspaces.list():
            ss = self._ac.get_workspace_client(ws)
            self._ac.groups.create(display_name="test_1xxxxx")

It yields the same below error:
../src/databricks/labs/ucx/account.py:108: in test_broken
    self._ac.groups.create(display_name="test_1xxxxx")
../../../Library/Application Support/hatch/env/virtual/databricks-labs-ucx/-POo8FxM/databricks-labs-ucx/lib/python3.11/site-packages/databricks/sdk/service/iam.py:1374: in create
    res = self._api.do('POST',
../../../Library/Application Support/hatch/env/virtual/databricks-labs-ucx/-POo8FxM/databricks-labs-ucx/lib/python3.11/site-packages/databricks/sdk/core.py:128: in do
    return retryable(self._perform)(method,
../../../Library/Application Support/hatch/env/virtual/databricks-labs-ucx/-POo8FxM/databricks-labs-ucx/lib/python3.11/site-packages/databricks/sdk/retries.py:54: in wrapper
    raise err
../../../Library/Application Support/hatch/env/virtual/databricks-labs-ucx/-POo8FxM/databricks-labs-ucx/lib/python3.11/site-packages/databricks/sdk/retries.py:33: in wrapper
    return func(*args, **kwargs)

E               databricks.sdk.errors.platform.NotFound: Endpoint not found for /2.0/accounts/None/scim/v2/Groups

../../../Library/Application Support/hatch/env/virtual/databricks-labs-ucx/-POo8FxM/databricks-labs-ucx/lib/python3.11/site-packages/databricks/sdk/core.py:221: NotFound

@william-conti is this still an issue? I believe this has been addressed by https://github.com/databricks/databricks-sdk-py/commit/9af2630672c866610248243eb53898c16c452960. Closing this out preemptively, but feel free to reopen if you're still affected.


Christie - Create a description in BCO for the Salmonella strain

Update the description section for the Salmonella strain. It will be added to the usability domain of the BCO.

Realized that the description was already in usability domain and that it was a bug error. Created a ticket for Robel to fix.

Robel fixed. Closing the ticket.


Accessibility

There is no compatibility with VoiceOver, so this library cannot be used in projects with accessibility for visually impaired in mind (almost every project).
Primarily, the elements inside the dialog (or even the dialog itself) are not focusable for VoiceOver. There is not even a single sound queue indicating the presence of a dialog on screen (that sound queue should be issued by VoiceOver itself).

See this from the Objective C port of this library:
https://github.com/dogo/SCLAlertView/pull/261


Add GitHub actions

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/Alluxio-PR-Builder/11315/
Test FAILed.

jenkins, test this please.

alluxio-bot, test this please.

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/Alluxio-PR-Builder/11316/
Test PASSed.

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/Alluxio-PR-Builder/11317/
Test PASSed.

@gpang Want to make this official?

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/Alluxio-PR-Builder/11335/
Test PASSed.

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/Alluxio-PR-Builder/11337/
Test FAILed.

@Xenorith diffing was really never really a thing.

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/Alluxio-PR-Builder/11338/
Test PASSed.

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/Alluxio-PR-Builder/11340/
Test FAILed.

alluxio-bot, test this please.

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/Alluxio-PR-Builder/11344/
Test PASSed.

@gpang I got through the review from @Xenorith, would you like to review?

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/Alluxio-PR-Builder/11351/
Test PASSed.

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/Alluxio-PR-Builder/11354/
Test FAILed.

alluxio-bot, test this please

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/Alluxio-PR-Builder/11355/
Test PASSed.

Tests passed.

Feel free to merge. @gpang

actually i think we need to coordinate when this merges along with other associated changes, so let's hold off for a bit @gpang

Added retries into the github_action itself so that users don't have to do much retries.

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/Alluxio-PR-Builder/11374/
Test PASSed.

alluxio-bot, merge this please


Improvements

various fixes:

include suffix / __ow_path in context  (see #17)
add route for requests with suffix (see #13)
allow each deployer to specify test (see #10)
dependency list not valid tag value (see #12)

Not the bigges fan of the pathInfo, but I can see how it would be useful for helix-status.

(and data-embed :-) using the shamed suffix notation )
maybe we can find a better name Request.servletPath ?

I'll try to figure out the problem with the "Server Error" on a new {path+} route.... once the build succeeds, I merge.

For helix-status, an endsWith check on the URL would be enough, but not for the embeds