Make PSReadline render correct portion of prompt on Unix when it's a multi-line string

Issue Summary
My prompt function returns a multi-line string. On Linux, when typing any input, PSReadline tries to render the whole prompt string again, instead of just the portion that is shown on the current line, which results in a bad prompt:

Fix
Check whether the prompt is multi-line or not. If it is, then only use the part that is shown in the input line.
Additional Info
I am using VT100 escape sequences to colorize my prompt string, and I found that PSReadline doesn't work with VT100 on Unix, because it's calling Console.Write(Char) instead of Console.Write(String) (see code here). I tried to fix that along with this PR, but it turns out to be not trivial, so I will postpone that fix to another PR.
Since PSReadline doesn't work with VT100 on Unix yet, you need to make sure that the part of the prompt string shown on the input line does not contain any escape sequences. Otherwise, you will see random characters like in the above screenshot.
Prompt function I use on Unix
I think it would be useful to share the prompt function I'm using on Unix. It works fine with this fix.
function Prompt
{
    $id = 1
    $historyItem = Get-History -Count 1
    if ($historyItem)
    {
        $id = $historyItem.Id + 1
    }

    if ($host.UI.SupportsVirtualTerminal)
    {
        $CSI = [char]0x1b + '['
        "${CSI}00m${CSI}01;30m[$($executionContext.SessionState.Path.CurrentLocation)]${CSI}00m`nPS:${id}$('>' * ($nestedPromptLevel + 1)) "
    }
    else
    {

        Write-Host -ForegroundColor DarkGray "[$($executionContext.SessionState.Path.CurrentLocation)]"
        "PS:$id$('>' * ($nestedPromptLevel + 1)) "
    }
}

Yay! ref to the PSReadLine issue https://github.com/lzybkr/PSReadLine/issues/470


Consider keywords after matching identifiers

@tenderlove, here you have the PR.
I have little experience with Ruby, so any suggestion will be very welcome. Thanks.

I tried combining this with a perfect hash in 28117d062a2252f51a2f927ff54a15c2aeaf48bb, but this still seems slightly faster. I'm not sure why though. It seems I could extract fewer bytes from the string (thus reducing method calls), but my patch seemed to slow things down.
  I'll try to look in to it more later, but suffice it to say we should stick with this PR for now. 😄

Aaron, I was also surprised of the good performance of KW_FP3 as a normal map. Nevertheless, there seem to be some possible explanations.
If I understood it correctly, the goal of KW_LUT was to recognize the matched keyword without extracting it (to avoid the allocation) and without checking against a map (that it's slow).
The purpose of KW_FP3 is a bit different. Its main goal is to avoid checking against KW_RE as much as possible because that regex is pretty expensive. To do that, we need to compute a value in a much bigger space than the KW_LUT. KW_LUT is an array of 32 positions, with 17 filled with keywords. The probability of a proper identifier to be recognized as a possible keyword by KW_LUT is 17/32. Therefore, more than half of the proper identifiers will be checked against KW_RE. On the other hand, the fingerprint for KW_FP3 computes a value in the range [0,43264], therefore, the probability now is only 17/43264 (this math is approximate and optimistic, because neither the fingerprint nor the data are uniform).
Consider that, in your original code, all the access to KW_LUT are successful, but most of the access to KW_FP3 are failures. The more failures we get for KW_FP3, the better.
I tried a larger fingerprint using 256 instead of 13. With 256, collisions on the first bytes are avoided. But the performance is a bit worse. I don't know why; maybe 256*256 is too big and yjit generates worse code. On the other side, for the benchmarking data, when scaling by 13, all the failures are caused by differences out of the first 4 bytes (maybe it is an overoptimization for the benchmarking data?).
I also considered to use a bitmap instead of a normal map. That would require around 5.5Kb for the bitmap and something like the perfect hash to find out the matched keyword. But I didn't find a bitmap in the Ruby std lib. I found some native implementations but I was afraid of the problem you describe in the post: too many jumps between C and Ruby. I also consider to make a pure Ruby bitmap, but I don't know enough Ruby to choose proper int sizes to avoid boxing. Maybe it can be implemented on a (binary) string.

The purpose of KW_FP3 is a bit different. Its main goal is to avoid checking against KW_RE as much as possible because that regex is pretty expensive. To do that, we need to compute a value in a much bigger space than the KW_LUT. KW_LUT is an array of 32 positions, with 17 filled with keywords. The probability of a proper identifier to be recognized as a possible keyword by KW_LUT is 17/32. Therefore, more than half of the proper identifiers will be checked against KW_RE. On the other hand, the fingerprint for KW_FP3 computes a value in the range [0,43264], therefore, the probability now is only 17/43264 (this math is approximate and optimistic, because neither the fingerprint nor the data are uniform

ahhh yes. This totally makes sense. Thank you for spelling it out. Great optimization, and thank you so much for the PR!


fix(xhr): finalize the response on "load"

This change moves request interception finalization to occur earlier than the current loadend. Request interception finalization involves, for example, surfacing the underlying xhr.status. This is notable because unaware code may have onload handlers that read xhr.status to determine whether the request succeeded, but loadend explicitly emits after load, so in the current state of things, the xhr.status will not be initialized and so will be 0, leading unaware client code to believe that every intercepted request failed.
The approach I took here was to run finalization during readystatechange when it is DONE instead. I made sure finalization only runs once since I saw another open issue https://github.com/mswjs/interceptors/issues/103 where it gets triggered twice for DONE.
This completely fixed the issue I was observing, for whatever that's worth.
Closes #321

@kettanaito no rush just in case you didn't notice the update here. Sorry for the back and forth!

@kettanaito happy new year! Is there anything I can help with to move this along?
To recap: you were right! I adapted the PR with the simpler approach that you mentioned.
Thanks again for your continued work on this project!

If you would like to team up with me again, I could use some help with #324. This improves the browser operability of this library by moving away from the StrictEventEmitter and using EventTarget since it's present in both browser and Node.js. The change itself is simple and done but it causes a number of tests to fail seemingly for no reason. We could investigate that, if it sounds interesting to you.

Thank you so much, I look forward to the release when you get a chance!
I'll take a look at #324 when I get a chance.

Released: v0.19.3 🎉
This has been released in v0.19.3!

📄 Release notes
📦 npm package

Make sure to always update to the latest version (npm i @mswjs/interceptors@latest) to get the newest features and bug fixes.

Predictable release automation by @ossjs/release.


Too much height on Chapter Intro page

Describe the bug
Not sure if this is a bug or if its intended (@GBKS) but on the intro page for a chapter there is this extra height which causes a scrollbar to appear. It seems to be the exact height of the Topbar, so we should probably just subtract that from the content height.
To Reproduce
Steps to reproduce the behavior:

Go to Chapter 1 Intro page.
Observe the scrollbar caused by extra height.

Expected behavior
I expect the scrollbar to only be visible if the content exceeds the viewport height.
Screenshots and/or logs

@benalleng will take this on.

Reason for this is that the img tag has no height set (it's not related to the nav bar). It should be set to 100%.

One solution @afterburn suggested on the call tuesday was to have a fixed pixel height for the top bar. Then use that, subtract it from total, and save it as a variable. Then use tailwind class to define min height for content. So instead of modifying image, force min height on content.
In this method, min. height will enabled scrolling.

That's not necessary. The CSS is already super well set up to allow this to be dynamic, we really just need this one extra CSS attribute on the image.
I generally would not try to use fix heights. With responsive design, accessibility settings, and browser scaling, elements can be dynamically resized in many ways for very legit reasons. So it's always better to try to make a dynamic layout work.

Yeah the min-height approach was just a suggestion if it couldn't be easily fixed otherwise. @benalleng has tackled this issue, though it was a little more involved than setting the image height to be 100% judging by his code changes.


Idea: most popular browser as plot

The most-popular browser data are hard to read, because each days are not super-well delimited. Maybe a plot would help? Associated with longer-duration data (see #103), it would be helpful in giving an idea of the browser-usage trend.

This is a good idea, but I am pretty bad at frontend development involving JavaScript. I will be happy to accept a pull request with the corresponding plot.


chore: Update Graalvm version in GitHub workflows

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@0b39358). Click here to learn what that means.
The diff coverage is n/a.

: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.
@@           Coverage Diff           @@
##             main       #3   +/-   ##
=======================================
  Coverage        ?   79.94%
  Complexity      ?      159
=======================================
  Files           ?       53
  Lines           ?     2793
  Branches        ?      457
=======================================
  Hits            ?     2233
  Misses          ?      374
  Partials        ?      186

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Numbers in templates should be parsed using the InvariantCulture

Given the following template:
{{ myVal = 1.0  }}
myVal:  {{myVal}}
{{ if myVal > 2 }}
   Parsed as ten
{{ else }}
   Parsed as one
{{ end }}`

The end result will depend on the culture. If I use a culture where the decimal separator is ",", this will print "parsed as ten".

Han, I thought that the parser was using proper invariant culture for terminals, but it turns out not, that's a mistake. I will fix it later today.

Not sure I will have time to look at this in the coming days, a bit too busy. If you feel to do a PR (it should be just a few fixes in Parser.Terminal.cs  where there are xxx.TryParse methods used.


Edge Console Refused to connect because it violates the Document Security Policy

Edge Version: 116.0.1938.54 (Compilación oficial) (64 bits)
The following script in the console give the message:
"Refused to connect to '' because it violates the following Content Security Policy directive: "default-src 'none'". Note that 'connect-src' was not explicitly set, so 'default-src' is used as a fallback."
"Refused to connect to 'http://localhost:8080/api/v1/product/' because it violates the document's Content Security Policy.
`const response = await fetch('http://localhost:8080/api/v1/product/', {
method: 'GET'
})

console.log(await response.json())`
In chrome works ok. I also try to add the headers: Content-Security-Policy': default-src 'self' but it doesn't work.
In the network console there is no request to the server.

Hi, thank you for reaching out.
This repository is for technical documentation only. If I'm understanding your issue correctly, you don't need help with documentation, but with DevTools, or with Edge itself. Which means, here is not the right place for this issue.
I'm not exactly sure what the issue is, but I can offer two suggestions:

If you think your code is correct, and it works in Chrome, but not in Edge, and you suspect a web compatibility issue, then you should report this bug to Edge directly. Here's how:

Click Settings and more (the ... button in the top-right corner of the browser).
Click Help and feedback.
Click Send feedback.

If, instead, you need help with this specific error message in the Console, and you think the Console could do a better job at explaining it, then I suggest sending this to the DevTools team. Here's how:

Go to https://github.com/MicrosoftEdge/DevTools/issues/
Open a new issue.


Fix PowerShell version

Fix PowerShell 6.0.1 to 6.0.2.

@alexandair : Thanks for your contribution! The author, @jluk, has been notified to review your proposed change.


Filter fairy ring favorites

As is, fairy ring favorites aren't searchable and are completely hidden when a search criteria is entered.
Would also be a precursor for https://github.com/runelite/runelite/pull/16168 to allow searching tags on all entries.

#16168 now includes these changes in it


Indicator boxShadow is clipped

In this example i want the indicator to have a boxShadow without the entire toggle having a shadow. Right now this boxShadow is clipped. The plugin should expose an option to prevent clipping.

I'll take a look at it today :) Thanks for reporting this issue.

Wonderful, this works! 🚀

Thanks :) The new version is now available on pub.dev.


Replace deprecated uses of <<< with send

<<< operator has been deprecated in TSC, as it mostly duplicates string interpolation, and the latter should be used instead in most cases where <<< was used previously. Additionally, <<< can't be introduced to scope with a qualified import (which is the case for any operator).

@swift-ci test

@swift-ci test windows

@swift-ci test windows


typo (space between link and text)

Could you please modify the source files instead of the output HTML? :)

Done. My bad, I didn't add the jade file


Python files should be snake_case.py

| - SharedCode
| | - myFirstHelperFunction.py
| | - mySecondHelperFunction.py
Should be
| - shared_code
| | - my_first_helper_function.py
| | - my_second_helper_function.py

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

ID: d955d169-8fb8-c511-8911-3e49f95c5ce6
Version Independent ID: 8a3d927c-51ce-d7d4-e5d1-acd928826db2
Content: Python developer reference for Azure Functions
Content Source: articles/azure-functions/functions-reference-python.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

Hi @robertlagrant Thanks for your feedback. We will review and update as appropriate.

Same goes for folder names, by the way. SharedCode looks very C#.

Thanks @robertlagrant an excellent point! We want to try to be a Python-y as possible.

Agree, let's make the folder name snake_cased. 👍


Star counts on template page are wrong

E.g. the fourth template is SvelteKit i18n Template. The site says it has 2096 stars, but when I go to https://github.com/ivanhofer/typesafe-i18n-demo-sveltekit it has only 123

It has the correct data in the .json file: https://github.com/svelte-society/sveltesociety.dev/blob/eb6ace2f096a6b21fe81466b188d6a1fe61eed0b/src/lib/data/github.json#L172
I guess that somehow it must be displaying the data for https://github.com/ivanhofer/typesafe-i18n instead of https://github.com/ivanhofer/typesafe-i18n-demo-sveltekit

Probably because this is doing .includes instead of ===:
https://github.com/svelte-society/sveltesociety.dev/blob/eb6ace2f096a6b21fe81466b188d6a1fe61eed0b/src/lib/utils/injectData.ts#L13C4-L13C46


Sonoff TX shown but unable to turn off or on

I'm able to show my new TX wall switch. It appears to be able to get the state of both switches however, it fails to turn off or on from HA. Unsure how to fix this since it is now working from ewelink apps but not working from HA.

Anyone able to help?

Show your config

my sonoff config
sonoff:
  username: !secret sonoff_user
  password: !secret sonoff_pass
  mode: local
  #reload: always

Lovelace
cards:
  - type: vertical-stack
    cards:
      - cards:
          - type: horizontal-stack
            title: Bathroom
            cards:
              - type: 'custom:button-card'
                color_type: blank-card
              - type: 'custom:button-card'
                color_type: icon
                icon: 'mdi:light-switch'
                name: Light
                entity: switch.sonoff_1000cbcad9_2
                tap_action:
                  action: toggle
              - type: 'custom:button-card'
                color_type: icon
                icon: 'mdi:fan'
                name: Fan
                entity: switch.sonoff_1000cbcad9_1
                tap_action:
                  action: toggle
              - type: 'custom:button-card'
                color_type: blank-card
        type: vertical-stack
type: vertical-stack

Remove mode local


Drop Puppet 3 support

With the latest versions of the concat submodule, the ensure attribute is forbidden on the fragment resource:
https://github.com/puppetlabs/puppetlabs-concat/blob/master/lib/puppet/type/concat_fragment.rb
To support this we need to remove this line:
https://github.com/bartavelle/language-puppet/blob/master/src/Puppet/Language/NativeTypes/Concat.hs#L40
This change would completely break a puppet 3 compatible version that requires an older version of the concat module.
Maybe it is time to create a release 1.4.x that would stop supporting puppet version < 4 ?
What do you think ?

Sounds good to me. However it will probably require a bit more work before
release.
Le 2 mars 2018 11:12 AM, "Pi3r"<EMAIL_ADDRESS>a écrit :

With the latest versions of the concat submodule, the ensure attribute is
forbidden on the fragment resource:
https://github.com/puppetlabs/puppetlabs-concat/blob/master/
lib/puppet/type/concat_fragment.rb
To support this we need to remove this line:
https://github.com/bartavelle/language-puppet/blob/master/
src/Puppet/Language/NativeTypes/Concat.hs#L40
This change would completely break a puppet 3 compatible version that
requires an older version of the concat module.
Maybe it is time to create a release 1.4.x that would stop supporting
puppet version < 4 ?
What do you think ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bartavelle/language-puppet/issues/240, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ABIQ73rhxx4CRZ4L5-b8B1Cfu5fEjTjtks5taRsbgaJpZM4SZrMG
.


consul health endpoint required by promtheus v2.17.0

The following commit to prometheus
https://github.com/prometheus/prometheus/commit/9b9c58aea8e0043bb944961fb776b48ac602c7f9
looks like prometheus is now querying service health endpoint (/v1/health/service/...) which isn't implemented by eureka-consul-adapter.
V2.16.0 seems to be the last prometheus release that is working as of now.

Thanks for the report. I will look into it.


firma debug para testapp

//Description
Firma debug y configuración para release de testapp
//Issue related
Closes #
Requirements

[ ] Coverage > 90%

Note:

[ ] Android Lint check

Note:

[ ] Code Style checked

Note:

[ ] Tested with "Don't keep Activities" configuration.

Note:
If impacts documentation (new components, inputs or outputs?)

[ ] PR to develop-docs

Note:

@jsotuyod me lo mergeaste justo al otro, le agregué la firma de debug y config así cuando testean release pueden testear con proguard. sino cada uno va a tener que estar agregando la configuración

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@ef846e4). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop     #849   +/-   ##
==========================================
  Coverage           ?   31.84%
==========================================
  Files              ?      324
  Lines              ?    17298
  Branches           ?     1552
==========================================
  Hits               ?     5508
  Misses             ?    11127
  Partials           ?      663

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 ef846e4...ae33c1c. Read the comment docs.


unknown high-level configuration parameter: "dbNames"

User reports:
$ ./perkeepd
2018/04/29 19:15:43 Starting perkeepd version 2018-04-28-b0c1faccfe; Go go1.10.1 (linux/amd64)
Error loading config file: unknown high-level configuration parameter: "dbNames" in file "/home/ubuntu/.config/perkeep/server-config.json"

@mpl, did we remove that but not change the default server config that we write?

@NorthDecoder I've just removed my $HOME/.config/perkeep , then I started ./perkeepd , and I could not reproduce. The generated config file looks ok to me. Could you give me the steps to reproduce the issue please?

Retried install in the same workspace by re-cloning code, now go run make.go getting a new error, which I did not get before: Out of Memory.  This workspace only has arbitrary limit of 512Mb, I will switch to server with more available memory, then continue testing.

pushing the milestone since 0.10 is out

I switched from the Cloud9 virtual machine with limited memory, to testing on a desktop and the install works okay.
$ uname -a
Linux machinename 4.13.16-100.fc25.i686+PAE #1 SMP Mon Nov 27 20:26:05 UTC 2017 i686 i686 i386 GNU/Linux
$ free -ht
              total        used        free      shared  buff/cache   available
Mem:           2.0G        1.1G         57M        317M        801M        601M
Swap:          2.1G         17M        2.1G
Total:         4.0G        1.1G        2.1G

I guess some kind of prerequisite README notes would advise against attempts to do the impossible, plus some quick check during the first part of the go run install to see if there will be enough memory to actually do the install.
Thanks for looking over my shoulder at my difficulty with the install during linuxfestnw; if you think we are done with this please feel to close it.

Thanks for the update. Will close.


Ability to show file upload related backend errors to the user during submission.

Is your feature request related to a problem? Please describe.
We have implemented a file validation procedure for uploaded files in our fork of invenio-files-rest package to enhance user security and improve sustainability of the uploaded data in our local instance, Aperta. In short, we make sure that the files are in the formats we like, there are no nasty surprises inside, and files are intact and openable.
However, when a user encounters an error, or rejected by one of these filters, we can't notify them about what happened.
This feature is a carryover from our old Aperta installation, which was a fork of Zenodo, and we were able to show these notifications on that version.
Describe the solution you'd like
We want to be able to notify the user about problems in their file(s), and what failed. So they can fix the problem and send the correct version/alternative of the file instead.
Describe alternatives you've considered
Since there's no way to communicate these errors and float them towards the UI and the user, we were unable to find any other alternatives.
Additional context
The things we do are as follows:

The process starts with file extension verification, which is performed through magic byte inspection, utilizing the open-source filetype library to infer file types.
We also perform health checks on tar (tar, tar.gz, tar.bz2, tar.xz) and zip compressed files to detect any issues during extraction. Rar and 7z formats are not supported; rar is a proprietary format, and reliable health checks for both formats would require third-party libraries, which we aim to avoid.
After health check, we verify the total uncompressed size not exceed 10 GiB. This limit is in place to protect against unusually large zip bombs, further ensuring end-user security. Password protected compressed files rejected as well due security and policy reasons.

If files pass all these checks, they're accepted as valid download, otherwise we reject them with an error. For a detailed explanation of what we do, please see the flowchart added below.
The file validation checks are executed once the file has fully uploaded, allowing us to perform integrity checks on compressed formats. If any validation fails, the file is immediately deleted from storage, and a FileCheckError —a custom exception inheriting from StorageError— is raised to indicate the failure.
This is where the problem surfaces. Unfortunately there is no support in InvenioRDM for showing this kind custom upload errors in UI. To be able to inform the user about the problem in a detailed manner, we want these errors to be visible in the UI, and want to introduce this feature into the file uploader(s) InvenioRDM has. There're talks about a new file uploader, and if that one will be the default, we'd love to add the support for it. Otherwise we want to introduce the feature to the old one and hopefully backport it to the newer one in due course.
Any help and guidance is greatly appreciated.
Flowchart:
flowchart TD
        flow_start@{ shape: circle, label: "Start" }
        flow_end@{ shape: circle, label: "End" }
        flow_end_end@{ shape: circle, label: "End" }
        upload@{ shape: lean-r, label: "Upload file" }
        is_not_allowed@{ shape: diamond, label: "if file_ext in [\"7z\", \"rar\"]" }
        file_ext@{ shape: rectangle, label: "file_ext = filename.split('.')[-1]" }
        file_check_error@{ shape: rectangle, label: "raise FileCheckError" }
        file_ext_guess@{ shape: rectangle, label: "guessed_file_ext = file_ext_guess(fp)" }
        is_file_ext_matched@{ shape: diamond, label: "if guessed_file_ext == file_ext" }
        is_zip@{ shape: diamond, label: "file_ext == \"zip\"" }
        is_tar@{ shape: diamond, label: "if file_ext in [\"tar\", \"tar.gz\", \"tar.bz2\", \"tar.xz\"]" }
        zip_integrity_test@{ shape: rectangle, label: "test_zip_file(fp)" }
        tar_integrity_test@{ shape: rectangle, label: "test_tar_file(fp)" }
        get_zip_uncompressed_size@{ shape: rectangle, label: "uncompressed_size = get_zip_uncompressed_size(fp)" }
        get_tar_uncompressed_size@{ shape: rectangle, label: "uncompressed_size = get_tar_uncompressed_size(fp)" }
        is_large_zip@{ shape: diamond, label: "uncompressed_size > 10 GB" }
        is_large_tar@{ shape: diamond, label: "uncompressed_size > 10 GB" }

        flow_start --> upload
        upload --> file_ext
        subgraph check_file["check_file(fp, filename)"]
        file_ext --> is_not_allowed
        is_not_allowed --True--> file_check_error
        is_not_allowed --False--> file_ext_guess
        file_ext_guess --> is_file_ext_matched
        is_file_ext_matched --True-->is_zip
        is_file_ext_matched --False-->file_check_error
        is_zip --True--> zip_integrity_test
        is_zip --False-->is_tar
        is_tar--True--> tar_integrity_test
        is_tar--False--> flow_end
        subgraph check_zip_file["check_zip_file(fp)"]
        zip_integrity_test --> get_zip_uncompressed_size
        get_zip_uncompressed_size --> is_large_zip
        end
        is_large_zip --True--> file_check_error
        is_large_zip --False--> flow_end
        subgraph check_tar_file["check_tar_file(fp)"]
        tar_integrity_test --> get_tar_uncompressed_size
        get_tar_uncompressed_size --> is_large_tar
        end
        is_large_tar --True--> file_check_error
        is_large_tar --False--> flow_end
        end
        check_file --> flow_end_end

P.S.: You can ping me and @geekdinazor about this issue. As I aforementioned, we want to implement and contribute this, if accepted. We may need some help from you.

Unfortunately there is no support in InvenioRDM for showing this kind custom upload errors in UI

What is the exact issue here?
Are you running this validation asynchronously? Are you expecting to respond with an error to the upload request?

It is still not 100% clear to me how you would like to solve this. Let's take an example: I create a new upload, add a file, and a few seconds later, publish. The backend files checks happen afterwards, asynchronously.
How would like to notify the user? Maybe via e-mail? Or any other way?
BTW, I am moving this to the Product's discussions, it is more relevant there as a global feature.


0.4.1

@lirantal We should probably bump the version on the package.json to match the latest release. This should be included in the 0.4.1 release tag if possible.

Yep. I was scanning the files to see where we have 0.4.0, probably also on the gh-pages area: https://github.com/meanjs/mean/pull/877

LGTM


Add test case for soul-common SoulThreadFactoryTest

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #658 (821afa2) into master (ac87820) will increase coverage by 0.26%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #658      +/-   ##
============================================
+ Coverage     48.55%   48.82%   +0.26%
- Complexity      840      845       +5
============================================
  Files           224      224
  Lines          5346     5346
  Branches        591      591
============================================
+ Hits           2596     2610      +14
+ Misses         2511     2497      -14
  Partials        239      239

Impacted Files
Coverage Δ
Complexity Δ

...mara/soul/common/concurrent/SoulThreadFactory.java
100.00% <0.00%> (+100.00%)
5.00% <0.00%> (+5.00%)

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 ac87820...821afa2. Read the comment docs.


Missing config files

The instruction: ” For reference, two example config files are provided in the example_config folder.”
Cant find the files...

Indeed, by a misconfiguration in de gitignore they were not committed ... fixed!


#145186309 Fix accounts tests

What does this PR do?

Complete missing asserts on existing test cases for unit testing accounts module
Add new test cases to test the accounts module

Description of Task to be completed?
-Go through each test file and follow the instructions to fix the tests. The instructions are distributed throughout each file and the instruction lines begin with ##
How should this be manually tested?

Clone the repository
cd into the repository
run the command ./manage.py test hc.accounts

Any background context you want to provide?

There are five test files in the accounts app folder that had already created test cases with pssing tests, this ticket was to complete some missing asserts on existing test cases and add some new test cases as instructed in the respective test files preceded by ###

What are the relevant pivotal tracker stories?

#145186309

Screenshots (if appropriate)

Coverage increased (+0.3%) to 80.851% when pulling dd65380b5c38b5a30bcea6ebc86a6ee433c138b9 on ft-accounts-tests-fixes-145186309 into 121c195aa368288ef4d2011eff1a9a890894296c on develop.


How to convert the predicted bounding boxes to actual predicted text file.

Hi @YoungminBaek @ClovaAIAdmin,
While testing the code on the image I've got the heatmap, bounding box.png and the locations of bounding boxes but I want the text file which consists of extracted text. Is there any way to do that?
Thanks in advance!!

same question

@SanatMoharir it can't because this CRAFT model is not for optical character recognition (OCR) but for text detection. If you want extract text, you can use some OCR model like https://github.com/clovaai/deep-text-recognition-benchmark with belong to @clovaai too or other model.


Pub sub with NATS

I use nats with htmx.
fir could too .
then all endpoints are virtualised and so are the handlers.

nats can be embedded also, so can make it easier for deployment

Hi @gedw99 . Generally looks like an interesting idea but right now focused on getting the event -> template change api right. Next up is https://github.com/livefir/fir/issues/42

Unsure how NATS fits into Fir. I ran some benchmarks and the combination of Redis + Websocket is good enough for now. Will revisit this when scale becomes a problem.


Drop support for Python 3.3 and 3.4, and fix CICD

Codecov Report

Merging #7 into master will decrease coverage by 1.12%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master       #7      +/-   ##
==========================================
- Coverage   69.87%   68.75%   -1.13%
==========================================
  Files           3        2       -1
  Lines          83       80       -3
  Branches       11       11
==========================================
- Hits           58       55       -3
  Misses         17       17
  Partials        8        8

Impacted Files
Coverage Δ

django_ssl_auth/__init__.py

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 d141f83...3c02c20. Read the comment docs.


add pug engine

adding pug engine

Thanks for you contribution @esmiralha , can you make sure the tests are passing?

Can I update the deno runtime used by the CI GH action to 1.29.1?

@gjuoun Are we good to merge? Thank you!

@gjuoun Are we good to merge? Thank you!

Done.


is nesting the only way to use a query range with this module?

<MediaQuery query='(min-width: 401px)'>
     <MediaQuery query='(max-width:800px)'>
             ...
    </MediaQuery>
</MediaQuery>

No, it's just normal media queries. Documentation here: https://developer.mozilla.org/en-US/docs/Web/CSS/Media_Queries/Using_media_queries (You're looking for the and keyword)
You can also use the props to do the same thing <MediaQuery minWidth={401} maxWidth={800}>

This might be a good thing to stick in the examples on README.md.
I had to stumble upon it in here, because I figured it existed, sometimes nesting is not always the prettiest solution. Great work on the component I'm implementing it at Rebelhorizons.com !

@csprance Want to send a quick PR to add more to the docs?


Waymo Road Map lane_to_left asserts

BUG REPORT
High Level Description
Waymo road map is unable to find a lane left of an existing lane for the purposes of generating the lane dividers.
SMARTS version
NeurIPS competition branch comp-1
58a61aa
Previous associated issues
N/A
Steps to reproduce the bug
This issue shows up when generating waymo scenarios:
+ validation.tfrecord-00045-of-00150
+ eb0f5d3f1760206e

Resulting and expected behaviour
Should generate the scenario without issues but runs into a missing lane.
Error logs and screenshots
cli/studio.py, line 79, in _build_single_scenario_proc
   build_single_scenario(clean, allow_offset_map, scenario, seed, click.echo)
smarts/sstudio/build_scenario.py, line 75, in build_single_scenario
   road_map.to_glb(os.path.join(scenario_root, "map.glb"))
smarts/core/waymo_map.py, line 994, in to_glb
   glb = self._make_glb_from_polys()
smarts/core/waymo_map.py, line 1019, in _make_glb_from_polys
   lane_dividers = self._compute_traffic_dividers()
smarts/core/waymo_map.py, line 1037, in _compute_traffic_dividers
   lane_to_left, _ = lane.lane_to_left
smarts/core/waymo_map.py, line 1255, in lane_to_left
   return self._get_side_lane("left")
smarts/core/waymo_map.py, line 1251, in _get_side_lane
   return self._map.lane_by_id(side_lane_id), same_dir
smarts/core/waymo_map.py, line 1564, in lane_by_id
   assert lane, f"WaymoMap got request for unknown lane_id: '{lane_id}'"
AssertionError: WaymoMap got request for unknown lane_id: '151_3'

There are 3 speculated reasons why this is happening:

lane_id=="151_3" should exist but was not generated.
lane index is inappropriately counted resulting in incorrect lane_id.
composite_roads somehow has a mismatched number of lanes (number of lanes on all roads in the composite should be the same)

I believe this was fixed, though I can't find the PR.


Finished the documentation for CommonGenerators

Much of this is simply about filling in the boilerplate for the Generators that are just external references.
But I also decided that this list is long enough to warrant using Scaladoc's @group mechanism. (Note the tweaked to the build file to turn that on.) The order of the sections is arbitrary, and be be easily rearranged by changing the values on the @groupprio entries in the top-level Scaladoc.

The groupings look great on CommonGenerators.


Overriding NAVI_FZF_OVERRIDES_VAR --header doesn't work with spaces

Describe the bug
I'm trying to override the --header function of fzf for the variable selection so I can print help text about distinguishing selecting the query or the selection like so (in fish shell):
$ begin; set -lx NAVI_FZF_OVERRIDES_VAR '--header "Press Enter or Tab to choose selection or query."'; navi --print; end

I get 'unknown option: Enter' and choosing the variable is skipped. It seems to me that parsing the spaces fails? Maybe this is related to #695 ? I also tried it with various escaping (escaping the spaces, using different quotes " and ', escaping the quotes " and ') but couldn't get it to work. Setting it in the config.yaml failed, too.
fzf does support spaces in its header, so this works:
fzf --header "Press Enter"

To Reproduce
Steps to reproduce the behavior:

Use fish shell.
Use the example cheatsheet:

% git, code

# Change branch
git checkout <branch>

$ branch: git branch | awk '{print $NF}'

in a git repo with at least 2 branches.

This works:

begin; set -lx NAVI_FZF_OVERRIDES_VAR '--header "Press_Enter_Or_Tab"'; navi --print; end

but this doesn't:
begin; set -lx NAVI_FZF_OVERRIDES_VAR '--header "Press Enter Or Tab"'; navi --print; end

Selection of the variable is skipped, and the output is

unknown option: Enter
git checkout

Expected behavior
The variable selection should work, and the header should be correctly displayed as "Press Enter Or Tab".
Versions:

OS: kubuntu 22.04 LTS
fish, version 3.5.0
navi 2.20.1 (downloaded latest navi binary)

The problem is in https://github.com/denisidoro/navi/blob/8725654a746fa8c999fbd5437559fdfcf1f7fb53/src/finder/mod.rs#L154
Instead of splitting by spaces, navi should honor bash's conventions for "word" splitting.
It should be trivial to fix it by calling the shellwords library, which already is a dependency.


⚠ T:Riza Community Forum has degraded performance

In 8e0c866, T:Riza Community Forum (https://community.triza.dev) experienced degraded performance:

HTTP code: 200
Response time: 1437 ms

Resolved: T:Riza Community Forum performance has improved in 43914ff.


Don't declare registry in .npmrc

Hey, I just made a Pull Request!
I am not sure what the reason to include the public npm registry in .npmrc is, but it seems unnecessary, and it also prevents npm install from working by default when in an enterprise environment with no access to public registry.
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

Ironically, I think this is in place because corporate npm mirrors kept sneaking into yarn.lock in pull requests 😁  Does setting the YARN_REGISTRY environment variable override this?

😆
we use npm, and tried npm config set registry, but that didn't override it.

We actually do not support npm - we use yarn.

Yarn has equivalent config commands but will also respect the .npmrc file config. From my understanding a local version of this file overrides the global settings. I suspect (but could be wrong) the registry is set locally to ensure the yarn.lock file uses that npm registry and not the yarnpkg registry proxy.

@andrewthauer is right - this file is there to make sure that any commits to the project are made with a sane registry, and any installs too. @tnguyen14  if you're trying to override this for inside your working installation of backstage, I recommend either using npx @backstage/create-app to create your own backstage installation which is not opinionated about this, or using environment variables to override, which I think take precedence over the config instead.

Hey! Since we want to keep this, and have a couple of suggestions of ways forward going, I'll close the PR. Feel free to provide further feedback if you feel that this is in error.


Replace more-dropdown with extend-repo-tabs?

What do you think about dropping "more dropdown" in favor of just adding links to the tab bar? The new links could be faded or just be icons so that they're not too noisy.
Example text-only

Example text-only dimmed

I personally prefer hiding them, but I don't feel strongly about it.

I personally prefer hiding them, but I don't feel strongly about it.

Same, but I've come to realize that I never actually use the dropdown because I just forget what links it has

Same, but I've come to realize that I never actually use the dropdown because I just forget what links it has


BAN-2160 Show captions for images and embed objects

This PR makes the necessary adjustments so that captions show in AppleNews articles for images and embed objects.
The article_json gem was outputting JSON with which Apple News should be showing captions in articles. However these captions were not showing in any form. This PR corrects the building of Caption objects.
https://mydevex.atlassian.net/browse/BAN-2160

I'm confused, weren't the image captions already added here?! The commit is identical, I think... :thinking:

@dsager
This PR has 2 commits, the first one was already merged in PR #186 .
The second commit however, seems to be new.

The first commit is the exact same commit.
Not newly added, it was just sitting in the branch.
I used the same branch. When I have done that (using the same branch for a second PR on top of an already merged PR on the same branch) on Grape the first does not appear, but here it does.
So now, what should I do here?

Maybe we can use GitHub's "Squash and merge" feature, see the green button below.

I think a git pull origin master --rebase && git push -f mine should make the commit disappear from here...

Maybe we can use GitHub's "Squash and merge" feature, see the green button below.

In this repo we usually use the "Rebase & Merge" feature, which might already be enough. I'm pretty sure that squashing the two commits into one would cause a conflict, because git would no longer be able to know the same commit is already on the main branch...
Let's try the local rebase, if that doesn't change anything I'll try the rebase & merge...

I think a git pull origin master --rebase && git push -f mine should make the commit disappear from here...

This did it. Thank you.
This single commit PR is now open for review.

Thanks!

Thanks!

No, thank you!!


Remove EMBER_TRY_SCENARIO=ember-release from allowed failures

Once Ember CLI 0.2.8 is released and this repo is upgraded to Ember 1.13

From allowed failures or being ran at all

Have decided to leave this configuration as is for now.


fix mixed precision

Description
Brief Description of the PR:
Fixes #2398
Type of change

[ ] Bug fix
[ ] New Tutorial
[ ] Updated or additional documentation
[ ] Additional Testing
[ ] New Activation and the changes conform to the activation contribution guidelines
[ ] New Callback and the changes conform to the callback contribution guidelines
[ ] New Image addition and the changes conform to the image op contribution guidelines
[ ] New Layer and the changes conform to the layer contribution guidelines
[ ] New Loss and the changes conform to the loss contribution guidelines
[ ] New Metric and the changes conform to the metric contribution guidelines
[ ] New Optimizer and the changes conform to the optimizer contribution guidelines
[ ] New RNN Cell and the changes conform to the rnn contribution guidelines
[ ] New Seq2seq addition and the changes conform to the seq2seq contribution guidelines
[ ] New Text addition and the changes conform to the text op contribution guidelines

Checklist:

[ ] I've properly formatted my code according to the guidelines

[ ] By running Black + Flake8
[ ] By running pre-commit hooks

[ ] This PR addresses an already submitted issue for TensorFlow Addons
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] This PR contains modifications to C++ custom-ops

How Has This Been Tested?
If you're adding a bugfix or new feature please describe the tests that you ran to verify your changes:
*

@charlielito
You are owner of some files modified in this pull request.
Would you kindly review the changes whenever you have the time to?
Thank you very much.


TEST-#4562: Keep trying to start test-windows ray cluster until we succeed.

Results from comparing infinite tries on baseline on  this branch versus  base:

base: 2/38 runs had the time out error. First round was 1/8 (it was this one out of all the commits between Test only test-windows[ray] in push.yml. and Empty commit 7 to trigger CI.). Later i ran another series of 30 tests from this one through this one and just this commit had this error.
with infinite retries: 0/50 runs had this error. used this branch. Ran 50 times between this commit and this one. 0/50 tests had this failure.

striked off because i ran more times and gave updated numbers above: I tried this change with several test-windows runs on this branch. Before the change to push.yml, 1/8 runs had the time out error (it was this one out of all the commits between Test only test-windows[ray] in push.yml. and Empty commit 7 to trigger CI.). (one out of 8 failed due to unrelated error #4905.) After this change, 0/20 runs had the time out error (1/20 had a mysterious other failure).
Signed-off-by: mvashishtha<EMAIL_ADDRESS>
What do these changes do?

[x] commit message follows format outlined here
[x] passes flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] passes black --check modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] signed commit with git commit -s
[x] Resolves #4562
[x] tests added and passing
[x] module layout described at docs/development/architecture.rst is up-to-date

@mvashishtha should we get this moving forward?

@vnlitvinov I think someone should continue this, but I'm not prioritizing it right now. I posted results in the first comment and they look good: https://github.com/modin-project/modin/pull/5101#issue-1398915504
I think we should github.com/marketplace/actions/retry-step as you suggested.
I'll make this a draft for now.


AnyRPG AI/Third Party AI Support

Hello Again!  I am wondering if we are going to be able to integrate third party systems such as Emerald AI.  I would rather a complex and easy solution be within AnyRPG but third party support would be awesome.

Multi Phase AI with changing background music, pet summons, ranged abilities, melee abilities, and buff maintenance is now working.
Check out the fight scenes in https://www.youtube.com/watch?v=9_3dAxb4F04 to get an idea of how it all fits together.
I still have to add class roles to the AI (like healer and tank), but that is a little way down the road.
Are there any other types of AI features (like running away low health etc) that you would find to be a higher priority?


fix queryservice integration tests related to tableacl

Deny any table access by default if tableacl does not find a ACL entry.
Fix queryservice integration test config file.
Log tableacl error only when 'queryserver-config-strict-mode' flag
is turned on (default: off)

That's not gonna break production, right?

Fixed the comment. Production won't break, because the "queryserver-config-strict-table-acl" flag is set false by default (https://github.com/youtube/vitess/blob/master/go/vt/tabletserver/queryctl.go#L157).
In query_executor.go, we only log and return error if that flag is turned on; Otherwise, we always return nil.
https://github.com/youtube/vitess/pull/937/files#diff-dd62190fdeaddd82bdf2bc8928da965aR235

LGTM


[USERSNAP] Paste in Cardano address

Taken at: https://gov.tools/
Fields:

Name of the idea or new feature: Paste in Cardano address
Please summarize your idea or new feature: I don't want to connect my ledger to your website. I don't necessarily understand what is secure and what isn't, so I'm being overly cautious while trust is being built. Please allow me to paste in my wallet address or stake address so that you can monitor my stakepool for me and alert me when any stake pool changes are made.

Browser: Chrome 131 (Windows 10)
Screen size: 5120x1440
Open #96 in Usersnap
Powered by Usersnap.

Hello @kickloop  A support ticket has been opened for you. Kindly Talk to Support on the Live support page so that your issue can be looked into. CLICK HERE


[api-management] Check in missing changes

Packages impacted by this PR
@azure/api-management-custom-widgets-tools
Describe the problem that is addressed by this PR
When building from main, these files showed as modified, just getting them checked in to avoid having a dirty tree

API change check
API changes are not detected in this pull request.


Add DMIT-CSD-Eugeniu-Ceban's file

Until now I accomplished all 6 points from the instruction.

f

f


The application may be doing too much work on its main thread?

The code it's very simple, and every time i click on a day the log says "Skipped x frames!  The application may be doing too much work on its main thread.". "x" is about 40 frames every time.
The code it's also in a separate thread that runs every time a user click on a new day. This is the code:
final Calendar mostra = Calendar.getInstance();
![cattura](https://cloud.githubusercontent.com/assets/12747695/14104388/40ae5962-f5a5-11e5-841b-fe0a2ca56b04.PNG)

m_handler = new Handler();
                            m_handlerAgenda = new Runnable() {
                                @Override
                                public void run() {
                                    if (updateAgenda) {
                                        if (Off_Agenda != null) {
                                            CVDataList.clear();
                                            compitiDatas.clear();
                                            compactCalendarView.removeAllEvents();
                                            Document docNote;
                                            if (WP_Agenda != null) docNote = Jsoup.parse(WP_Agenda);
                                            else {
                                                swipeRefreshLayout.post(new Runnable() {
                                                    @Override
                                                    public void run() {
                                                        swipeRefreshLayout.setRefreshing(true);
                                                    }
                                                });
                                                docNote = Jsoup.parse(Off_Agenda);
                                            }
                                            Elements metaElemsAgenda = docNote.select("Row");
                                            String dataInizio;
                                            for (int i = 1; i < metaElemsAgenda.size(); i++) {
                                                Elements AgendaDati = metaElemsAgenda.get(i).select("Data");
                                                SimpleDateFormat sdf = new SimpleDateFormat("yyyy-MM-dd", Locale.ITALIAN);

                                                Log.v("metaElemsAgenda", "Size: " + String.valueOf(metaElemsAgenda.size()));
                                                dataInizio = AgendaDati.get(1).text();

                                                Calendar c2 = Calendar.getInstance();
                                                try {
                                                    c2.setTime(sdf.parse(dataInizio));
                                                    compactCalendarView.addEvent(new CalendarDayEvent(c2.getTimeInMillis(), Color.argb(200, 244, 67, 54)), false);
                                                } catch (ParseException e) {
                                                    e.printStackTrace();
                                                }
                                            }

                                            if (WP_Agenda != null) {
                                                updateAgenda = false;
                                                swipeRefreshLayout.setRefreshing(false);
                                            }
                                            adapter.notifyDataSetChanged();

                                        } else {
                                            CVDataList.clear();
                                            CVDataList.add(new CVData("Aggiorno", "Aggiorno i dati...", "", null));
                                            adapter.notifyDataSetChanged();
                                        }
                                        m_handler.postDelayed(m_handlerAgenda, 5000);
                                    }
                                }
                            };
                            m_handlerAgenda.run();

                            compactCalendarView.setListener(new CompactCalendarView.CompactCalendarViewListener() {
                                @Override
                                public void onDayClick(Date dateClicked) {
                                    mostra.setTime(dateClicked);
                                    updateAgenda = true;
                                    m_handlerAgenda.run();
                                    Log.v("NewDate", String.valueOf(dateClicked));
                                }

                                @Override
                                public void onMonthScroll(Date firstDayOfNewMonth) {
                                    txMese.setText(new SimpleDateFormat("MMMM yyyy", Locale.ITALIAN).format(firstDayOfNewMonth.getTime()));
                                }
                            });

Really i don't get the point...why it says that the app is doing too much work on its main thread? It's a problem with the compact calendar view?
This is the log:

I solved a bit by avoid doing the "Jsoup.parse()" every time i click a day, now the message in the log doesn't appear any more, but the calendar it's still a bit slow.

Hi,
Heres a good developer guide on performance profiling. http://developer.android.com/tools/performance/index.html
Usually it will tell you what bit of code is taking the longest so you can dig deeper. If it indeed is the calendar, then we can further have a look. I particularly like systrace and trace view because they can tell you the percentage of time taken in a particular method.
Try refactoring your code as well to be a little cleaner, then you might be able to tell where performance problems might be. Try refactoring code to generate events onto a seperate thread class which returns a list of CalendarDayEvent. Remember, you should only need to add one event per a day to get it to highlight. Once you have a list, post back to the main thread by sending a message using the handler to the main thread where you can do a removeAll and then addEvents. Unfortunaly, I haven't used Handler class in a very long time so I cannot offer adive on its usage.
You can also try RxJava with android. I makes this kind of thing very easy to work with.
Other thoughts:

You do not need to create SimpleDateFormat in the foor loop. It can be taken outside.
Calendar.getInstance() can also be taken outside. It is mutable, so you can just change the date and reuse it.
Does your code actually run in a thread? m_handlerAgenda.run(); It just looks like your calling the run method directly rather than submitting to a ExecutorService or creating a Thread directlty and calling start(). So it will all run on the main thread blocking execution.

Thanks for your tips! I will try ;)

I'm going to close this ticket. Feel free to re-open if you are still having problems and think it's directly related to the calendar. I will likely try and do some benchmarking at some point as well.


client not exposed or accessible for Provider, doesn't allow for extension of extra queries.

If I wanted to implement extra custom queries into the Provider I wouldn't be able to as client isn't accessible. Source
/// Encapsulates common client operations in the SDK.
/// Note that you may also use `client`, which is an instance
/// of `FuelClient`, directly, which provides a broader API.
#[derive(Debug, Clone)]
pub struct Provider {
    client: RetryableClient,
    consensus_parameters: ConsensusParameters,
    #[cfg(feature = "coin-cache")]
    cache: Arc<Mutex<CoinsCache>>,
}

A function to return client so I can use it externally in custom implementation would be helpful for example:
#[async_trait::async_trait]
impl ClientExt for Provider {
    async fn full_blocks(
        &self,
        request: PaginationRequest<String>,
    ) -> std::io::Result<PaginatedResult<FullBlock, String>> {
        let query = FullBlocksQuery::build(request.into());
        let blocks = self.client().query(query).await?.blocks.into();
        Ok(blocks)
    }
}

Or adding query to the Provider would also do the trick:
    /// Send the GraphQL query to the client.
    pub async fn query<ResponseData, Vars>(
        &self,
        q: Operation<ResponseData, Vars>,
    ) -> io::Result<ResponseData>
    where
        Vars: serde::Serialize,
        ResponseData: serde::de::DeserializeOwned + 'static,
    {
        let response = self
            .client
            .post(self.url.clone())
            .run_graphql(q)
            .await
            .map_err(|e| io::Error::new(io::ErrorKind::Other, e))?;

        Self::decode_response(response)
    }
    ```


Investigate possible issues with partial views

As evidenced by #242 and #243, middle_outer_views is not tested enough, and edge cases may happen in numerous algorithms. It's necessary to write tests for many functions involving these partial views, to reveal the presence of possible bugs.

What are the requirements of data and indices? Do they have to be the same length, or is it enough for them to be as long as indptr requires? Does indices outside of the range given by indptr have to be convertible to usize?

Minor optimisation: to_owned of a view could adjust indptr to start at 0, and contain the minimum amount of entries in indices and data

What are the requirements of data and indices? Do they have to be the same length, or is it enough for them to be as long as indptr requires? Does indices outside of the range given by indptr have to be convertible to usize?

In most algorithms, the only requirement is that for all i < indptr.len() - 1, indices[indptr[i]..indptr[i + 1] is not out of bounds, same for data. This imposes that data and indices have the same length, but does not impose anything outside that range. However, in the current state, the structure checks of CsMat are much more restrictive, and it's very probable some implementations depend on these more precise details.
I see two possible lines of action:

commit to the possibility to have zero-copy partial views of a matrix, relax structure checks and increase test coverage
take a shortcut and allocate when trying to slice a matrix. This seems to be scipy's approach.

I'm lean towards the first approach, as I don't like the performance penalty of the second one.

Minor optimisation: to_owned of a view could adjust indptr to start at 0, and contain the minimum amount of entries in indices and data

Yes that looks like a good idea.

A drawback of the zero-copy approach is that equality between two sparse matrices is no longer trivial, because the non-indexed elements are arbitrary. Pretty sure the current implementation exposes this bug.

There is a way to have a zero copy approach without having unused/invalid elements. The idea is to use another type for the indptr storage, that would store a slice and an offset to apply to the values. To make this fit into the API there would need to be a trait defining everything that can be done with an indptr, and all algorithms would be implemented using this trait. This way the type system would help to prove correct usage.
That would technically be a breaking change, but not one that would affect much code in practice I guess.

Equality as defined is given as structural equality, so don't think this is a big problem. The approx trait could be used for semantic equality

Scipy ensures indptr always starts at 0, what does suitesparse do?

Looks like SuiteSparse enforces the indptr to start at 0.

There is a way to have a zero copy approach without having unused/invalid elements. The idea is to use another type for the indptr storage, that would store a slice and an offset to apply to the values. To make this fit into the API there would need to be a trait defining everything that can be done with an indptr, and all algorithms would be implemented using this trait. This way the type system would help to prove correct usage.
That would technically be a breaking change, but not one that would affect much code in practice I guess.

I've started prototyping a trait around that idea, any feedback is appreciated!

How about this approach: https://gist.github.com/mulimoen/c476ee6a2daf5ba9e341d87dfbc46e14? data and indices should always be a part of the matrix, indptr[0] is just a remapping down to zero.

That's a nice insight, no need for a trait with your approach!

Another alternative: indptr could return a Cow

I really like the concrete type approach with indptr[0] used as an offset, that makes the type a guarantee of valid data, and it can take its own share of the structure checks. I've started implementing it, it looks like this for now: https://github.com/vbarrielle/sprs/blob/99663dcad21e34e3260b64e37b6e5fa33d14ff5a/src/sparse/indptr.rs

Looking great! I have a couple of ideas below

I'm missing a form of canonical indptr, which can be used for FFI, maybe expressed as a Cow for an improper indptr?
ndarray calls the slice operation as_slice, should we follow the same naming?
new consumes storage on failure, could we return this on failure?
Internal API idea: a method ranges which returns an iterator of std::ops::Range<usize> which can be directly consumed

* I'm missing a form of canonical `indptr`, which can be used for FFI, maybe expressed as a `Cow` for an improper indptr?

Yes this looks like a good idea.

* `ndarray` calls the `slice` operation `as_slice`, should we follow the same naming?

I'll change that for coherence. But it's fallible here, so maybe try_as_slice?

* `new` consumes `storage` on failure, could we return this on failure?

I'll see if it can be done.

* Internal API idea: a method `ranges` which returns an iterator of `std::ops::Range<usize>` which can be directly consumed

That's interesting indeed, maybe it should be what's returned by iter_outer? There's one downside to that, as Rangeis not Copy AFAIR, but other than that I like this idea even for a public API.

* A lot of the `map`s can be removed, and we could use `unwrap`, we should always have a `first` and `last` element

I don't think we will always have a first and last element, for instance I don't think we should disallow creating a view around 0 rows. I know in dense linear algebra this kind of edge case can be very useful.

ndarrays as_slice returns an Option, try_as_slice seems a bit verbose.
A size one indptr has two elements, it would be "natural" to have a zero sized indptr with one element. Although a length zero would mean we could implement Default

@mulimoen was that the kind of Cow-returning you had in mind?

That was exactly what I had in mind, we can now use this in to_owned on CsMatbase

Hello, just an update to inform that my work on this is going to be a bit slow, my second son is born this week and I'm having much less free time. I still hope to be able to keep working on this though.

My current state is shown is #245, it's not without issues but I think most of the work is there now.


remove unreachable code in visit method

This PR remove unreachable code in hashstructure.go.
$ go vet .
hashstructure.go:328: unreachable code
exit status 1

Looks good. Thanks!


Check out meshio effort

This looks interesting
https://github.com/nschloe/meshio

Closing for housekeeping.


ui: submit form with false boolean params

Description
Skip only empty strings while submitting UI form.
Fixes #5223
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[x] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

@blueorangutan ui

@shwstppr a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/5224 (SL-JID-407)


tasks and projects don't show up in interface

The web interface works but none of my data is shown (pending:0) enven though the cli shows data.  Oddly I could add a task with the web interface and it showed up via cli, but not the web interface.
The log shows this over and over
Configuration override rc.xterm.title:no
Configuration override rc.color:off
Configuration override rc.verbose:no
Configuration override rc.confirmation:no
No matches.

this was built with ruby-2.1.8 (via rvm)

+1
Having the same issue.

+1
I am having this problem as well.  Also, I am getting an error at task-web startup regarding accessing the console:
[2016-12-02 11:58:49 +0100] Starting 'taskwarrior-web'...
[2016-12-02 11:58:49 +0100] trying port 5678...
**Couldn't get a file descriptor referring to the console**

Similiar to @mattmartini and @aravindhsampath, $HOME/.vegas/taskwarrior_web/taskwarrior_web.log is showing the following after each request:
Configuration override rc.xterm.title:no
Configuration override rc.color:off
Configuration override rc.verbose:no
Configuration override rc.confirmation:no
No matches.

I installed today taskwarrior-web today (2016-12-02) on Ubuntu 15.10.
Thanks!

BTW, I am running TW 2.5.1.  So, I am wondering if this is a duplicate of or somehow related to issue #84?
@mattmartini and @aravindhsampath: Are you also running version 2.5.1 of TaskWarrior?

"Couldn't get a file descriptor referring to the console" apparently has to do with the "open" command trying to open the next virtual terminal.  I think the fix is to call xdg-open instead (as starting task-web does not open it in the browser as indicated).  Not really an issue though until items not showing up is resolved.

@morsedl  Yes, I am running TW 2.5.1

I too faced this issue with TW 2.5.0 on ubuntu 16.04 but running with sudo worked fine.

Couldn't get a file descriptor referring to the console

Hi,
I was facing the same issue, but I'm not sure that it was for the same reason.
Anyway I share with you my workaround.
My problem was that the docker image hadn't the /usr/share/doc/task/rc directory.
So I've just added this line :
- /usr/share/doc/task/rc:/usr/share/doc/task/rc:ro in the docker-compose.yml file.
And now it's working fine !

@takuyozora can you provide your Dockerfile and docker-compose.yml?

No change on the Dockerfile.
My complete docker-compose.yml :
version: '2'

services:
  app:
    build:
      context: .
      dockerfile: docker/Dockerfile
    ports:
    - 5678:5678
    volumes:
    - /home/olivier/.task:/root/.task
    - /home/olivier/.taskrc:/root/.taskrc:ro
    - /usr/share/doc/task/rc:/usr/share/doc/task/rc:ro

(Change /home/olivier with your home directory)


Release GIL in more Numba methods

This commit was extracted from #1158 as it is independent but touches a number of files. The original commit message is:
It turns out that tiling slows down the threaded case. This
change releases the GIL for more parts of the LiberTEM internals.  The
float32 case uses a very large single tile for the partition and shows
the difference.
Rebased from @uellue commit d0037924
Contributor Checklist:

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

Thanks for extracting the commit, really makes sense as separate PR. LGTM!


Customizable speed dial with a tooltip

Description
Create the component as shown in the video attached below.
Animation Preview

Screenshot/Video:

https://github.com/user-attachments/assets/66bff353-ff78-420f-95d4-01bd4588d272

[!NOTE]
Please refer to the Animata contributing guidelines available here for rules on how to contribute to the project. Let us know in the comments if you’re working on this issue, and feel free to ask any questions!

Requirements

Create a new animated component that matches the provided design.
Add example usage of the component in the documentation or storybook.

 Guidelines & Best Practices

Write reusable and clean code following the project’s style guide.
The component should be fully customizable. The user should be able to modify props such as duration, easing, color, and size where applicable.
Follow the Contributing Guidelines before submitting your pull request.

[!IMPORTANT]
To ensure more contributors have the opportunity to participate, we kindly request that:

Each contributor submits only one pull request related to this issue.
If you're interested in working on this issue, please comment below. We will assign the issue on a first-come, first-served basis.

[!Tip]
Join our Discord community if you need help or have any questions. We're here to help you get started and guide you through the process. Join now

The video preview is the solution itself, if this component doesn't exist or isn't planned, I would love to contribute this component to animata.

@masabinhok Sure, please add references as well (if any). Thanks

Added.
Preview is not working and i am getting some errors, i couln't resolve.


Add ability to customize fonts

The json files for the fonts get overwritten by updates, users would want a way to customize them with a setting option (path to json to use? need for css?) which persists across upgrades.

How would one add a custom language font? I'm trying to add Thai letters to PolyGlot.


[community] Update submariner-operator to v0.6.1

Signed-off-by: Steve Mattar<EMAIL_ADDRESS>Thanks submitting your Operator. Please check below list before you create your Pull Request.

Flat operator directory structure is obsolete from 23-rd of October 2019, only nested directory structure will be accepted.

New Submissions

[x] Does your operator have nested directory structure?
[x] Have you selected the Project Community Operator Submissions in your PR on the right-hand menu bar?
[x] Are you familiar with our contribution guidelines?
[x] Have you packaged and deployed your Operator for Operator Framework?

[x] Is your submission signed?

Updates to existing Operators

[x] Is your new CSV pointing to the previous version with the replaces property?
[x] Is your new CSV referenced in the appropriate channel defined in the package.yaml ?
[x] Have you tested an update to your Operator when deployed via OLM?
[x] Is your submission signed?

Your submission should not

[x] Modify more than one operator
[x] Modify an Operator you don't own
[x] Rename an operator - please remove and add with a different name instead
[x] Submit operators to both upstream-community-operators and community-operators at once
[x] Modify any files outside the above mentioned folders
[x] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[x] Description about the managed Application and where to find more information
[x] Features and capabilities of your Operator and how to use it
[x] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[x] Human readable name and 1-liner description about your Operator
[x] Valid category name1
[x] One of the pre-defined capability levels2
[x] Links to the maintainer, source code and documentation
[x] Example templates for all Custom Resource Definitions intended to be used
[x] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

/test CVP

@SteveMattar Threre is problem with replaces.
submariner.v0.6.1 specifies replacement that couldn't be found"
exit status 1

Please don't modify 2 versions in on PR. I would rename 0.5.0 version in one PR and then update to version 0.6.1.

@skitt could you please sign?

Acked-by: Stephen Kitt<EMAIL_ADDRESS>


Loading more than one scene additively with the ZenjectSceneLoader doesn't work

I ran into a problem recently. I'm trying to create a dynamic content system so I can load and unload different sections of a level without loading screens.
The catch though, seems to be that I need to load more than one scene at a time. And using the ZenjectSceneLoader the second loaded scene doesn't inherit the dependencies installed on the main scene context.
I'm already using the LoadSceneContainerMode.Child
Here's an example of the problem: https://github.com/halzate93/additive-loading-test
Any ideas on what could be going wrong?

I load scenes using the ZenjectSceneLoader,
I want the additional scenes to keep the installers I configured on the main scene, my current setup is this one:
![setup] (https://s14.postimg.org/dpjictd01/Screen_Shot_2016_10_22_at_10_21_52_AM.png)
The idea is that the Greeters all use the Logger created on the test installer. However, the Greeter on Additional Scene 2
fails to resolve the Logger created on the Main Scene.
![error] (https://s14.postimg.org/d3bgxvp9t/Screen_Shot_2016_10_22_at_10_28_48_AM.png)
The code I use to load the scenes is this one:
using UnityEngine;
using Zenject;

public class AdditionalSceneLoader : MonoBehaviour
{
	[SerializeField]
	private string[] additionalScenes;

	[Inject]
	private ZenjectSceneLoader loader;

	private void Awake ()
	{
		foreach (string sceneName in additionalScenes)
		{
			loader.LoadSceneAdditive (sceneName, LoadSceneContainerMode.Child);
		}
	}
}

Any help will be appreciated!

Thanks for the minimal reproducible example  :)
I can see what the issue is.  When you load scenes this way, Unity delays actually starting the scene until the next frame.  And the ZenjectSceneLoader uses a static field to temporarily store the arguments to the new scene.  When you load multiple scenes in sequence like this, this static field gets overridden.
You have two options to fix it:

You can either load each scene using ZenjectSceneLoader.LoadSceneAsync from within a coroutine.
You can use a new undocumented feature called "scene contract names".  The way this works is that you would fill in the "contract name" field on the scene context in your main scene (for eg. you could use "Main") and then you would set the "parent contract name" to this same value on all the scenes you want to parent to it, then you could just load the scene normally without even using ZenjectSceneLoader

Both of these are only available on the develop branch, however it is pretty stable, so you should be fine to use it.  I'm planning on documenting this stuff this weekend and then merging it back into master as version 4.6, so you could also just wait for that.

The other nice thing about option 2 is that you can use Unity's edit-time multi-scene support.  ie. You could drag all your scenes into the scene heirarchy then hit play, without the need to load them dynamically

Thanks for the quick answer! I'll play around with the contract names, and if I can't figure it out I will go with the coroutine solution. The scene contract names sounds way nicer though.


AC: moving on IECore API

moving from deprecated IEPlugin API to IECore API
clean up launcher tests (removed part for working with adapter because adapter is independent entry now)
TO DO:

[x]  device versions logging
[ ] multi device infer requests assignment
[ ] use metrics for optimal async requests number
[x] clean up unused API

Jenkins please retry a build


DD 1.6.3 install failing

Hi, the fix provided for the DD (fix tyogi cloudflare ddos issue) is failing to install on Android Kodi 16.1 and 17.6, is it the phython compatibility issue? Will mygica picks this one..Pls suggest.. thanks for the help.

Cloudflare fix was already used by others such as mhdtv in the addon and is not new. Can you enable debug log and then try to install dd1.6.3 and then upload the log using Kodi log uploader and give me the link?
I tested it on Android and windows 17.6 and works fine
Sent from ProtonMail mobile
-------- Original Message --------
On 23 Jun 2018 1:03 pm, SunFloWeR15 wrote:

Hi, the fix provided for the DD (fix tyogi cloudflare ddos issue) is failing to install on Android Kodi 16.1 and 17.6, is it the phython compatibility issue? Will mygica picks this one..Pls suggest.. thanks for the help.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Please close the issue, I reinstalled again now and it went through..not sure what was causing this.. thank you!


tikv_alloc: remove unwrap and refine README.md

What have you changed?
Remove unwrap and refine readme.
What is the type of the changes?

Engineering (engineering change which doesn't change any feature or fix any issue)

How is the PR tested?

Unit test

Does this PR affect documentation (docs) or should it be mentioned in the release notes?

Does this PR affect tidb-ansible?

Refer to a related PR or issue link (optional)
#6717
Benchmark result if necessary (optional)
Any examples? (optional)

@zhangjinpeng1987  PTAL~


add five other companies.

That includes Angellist, Flexport, Mark 43, rubric and stitch fix.

Thanks for contributing even more companies! :)


Update Java examples readme (/examples/java).

Update Java examples README (/examples/java).
Readme contains instructions on getting started with API in Java, but final step incorrect.
After building, final run command leads to a FileNotFound exception.
Change Run command $ java ReferenceData ../examples/ReferenceData/IBMHolders.json change to
$ java ReferenceData ../json/ReferenceData/IBMHolders.json

lgtm - thanks for the contribution!


Add some more consistency checks in tls_decrypt_ticket

This adds some additional consistency checks in tls_decrypt_ticket.
The PR works for master.  If you think it is a good idea, I can
back-port that to 1.1.0 & 1.0.2 as well.

There's no need to make separate PRs for 1.1.0 and 1.0.2, this change can be cherry picked as is (I just tried).

I spoke too fast, never mind my previous comment

Is this still useful since #2704 was merged to all branches?

Opps.  I wanted to use the define TICKET_NO_DECRYPT in master.

Is this "fixup" ok, or do you need a new PR?

Fine as is.  Ping @levitte or @snhenson :)

Ping.

Ping?
Note, that this is a kind of obvious fix because TICKET_NO_DECRYPT != 2
and it is only necessary because of a cherry-pick-glitch...

ping @levitte :)

Ping @richsalz

reconfirm

Pushed.


Improve Mixpanel metrics

Currently we don't identify individual members and see their behaviour. People's tab in Mixpanel should be activated to do this.

Fixed in: fabcd0062ad5d021c5b9156d73153037188c1cb2


Create contact.cpp

Create contact and #4784

Close #4784

Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md

please merge this @MohitGupta121


Alternate caps.shs

I'd like to run a room on an alternate SSB network. I have the room running via docker-compose, and I can see my secret in the ssb-go-room-secrets directory. I tried adding adding a config file there with my caps dictionary, but it doesn't seem to be picked up by the room. Is there another way to pass caps keys to the room?

Hey @mplorentz i'm unfamiliar with this feature but I server cli options available:
  -shscap string
    	secret-handshake app-key (or capability) (default "1KHLiKZvAvjbY1ziZEHMXawbCEIM6qwjCDm3VYRan/s=")

Perhaps a config option is not supported? Unsure.

Ah thanks @decentral1se that looks like what I want but it doesn't seem to be working when I set it. My client (go-ssb/Planetary) says:
level=error ts="2022-07-26 18:38:37.3088830 (UTC)" where=ssbConnectPeer err="connecting to \"net:<IP_ADDRESS>:8008~shs:bGo2yHoSjTFVYmZxPD9+AZM2LYnp6A/5WxHzezfDLls=\" failed: node/connect: error dialing: error wrapping connection: secrethandshake: failed during data transfer of receiving challenge: EOF"

@boreq I think you fixed a bug related to alternate caps in cryptoscope/ssb a few months ago. Do you remember the details of that and do you think that go-ssb-room would have the same issue? I think they share many of the underlying Go modules.

The problem which you are probably referring to that I fixed was related to not being able to redeem invites on the test network. This was caused by the fact that the code always used the default network key. The linked commit allows configuring an alternate network key when redeeming invites.
https://github.com/planetary-social/ssb/commit/fc36febcd484bdfe6327ee421354bb4becd0834e

@mplorentz and to clarify, you can connect when not using the -shscap option on the CLI? You should be able to confirm that go-sbot is processing your expected value by inspecting the .ssb-go/running-config.json also.

I am very confused as to how the shscap work. In the code it says "appkey" so I supposed it should be a unique key. But my setup never works if I put any other 32 bytes in that field than the default ones. Do they need to be the fixed 1KHLiKZvAvjbY1ziZEHMXawbCEIM6qwjCDm3VYRan/s=?

I am very confused as to how the shscap work. In the code it says "appkey" so I supposed it should be a unique key. But my setup never works if I put any other 32 bytes in that field than the default ones. Do they need to be the fixed 1KHLiKZvAvjbY1ziZEHMXawbCEIM6qwjCDm3VYRan/s=?

Yes, the main network uses a predefined network key. It is sometimes referred to as the "app key" but the protocol guide consistently calls it a "network key". I find the name "app key" confusing.
https://ssbc.github.io/scuttlebutt-protocol-guide/#handshake


/dev/sdX doesn't exist

I am using pico-pi-imx8mq board and want to interface it with a 5.5 inch amoled hdmi display by waveshare.
1st I installed the pre-built image of android pie. It is giving lot of distortions on the screen, not working properly.
So I started building the image from git. Have been facing issues such as
"This script is validated with uuu 1.2.92 version, please align with this version" --while executing ./uuu_imx_android_flash.sh -c 13 -f imx8mq -e -D.
After spending time, I started regenerating files, build u-boot in the same directory. I got following error:
$ ./install_uboot_imx8.sh -b imx8mq-pico-pi -d /dev/sdX
Before run this script, please build u-boot first!
And
/dev/sdX doesn't exist !!! -- at the end of the line.
Can anyone help me understand this?
I am new to using u-boot tool.
log file1.txt

I don't this its the right place to this bug.
Your problem its not related with meta-webkit layer. It looks something much more lower level related with your BSP or with the one that provided you with that install_uboot_imx8.sh script.
Please fill this bug there.


IE 11 renders side label content inappropriate

The class gantt-row-label has defined an overflow:hidden which causes IE 11 to misplace the side elements.
When setting the overflow attribute to overflow:inherit it works.

I don't use IE, PR are welcome.

I will look into it

I tried to reproduce this issue with the master demo but the side panel is displayed correctly in IE11.
I tested with IE 11.0.9600.17498 and Windows 8.1. Tomorrow I will quickly check with my work computer as this is running Windows 7. Maybe there is a small difference on how the IE renders things.
@kschwidder: Could you attach a screenshot of the misplacement and if somehow possible with the master demo as it will be easier to fix & re-test.

I have used IE 11 on Windows 7. I fixed the problem with the following CSS style override:
/* Fix for IE 11 by deleting overflow:hidden */
.gantt-row-label {
overflow: inherit;
white-space: nowrap;
}

And it doesn't cause problem when resizing the side to a very small width ? smaller than the labels ?

It does not impact the resize.
This fix works for angular material which I am using heavily.


Avoid storing metadata rules in the in-memory cache

This commit avoids storing the component metadata rules when
we store component metadata in the in-memory cache. While this
shouldn't be necessary in practice because the in-memory cache
only lives for a build, it's possible that the instance is
actually only freed once we reach enough memory pressure, which
means we can unnecessarily keep project references in memory.
It's more problematic if the rule is a @CacheableRule but even in
this case, we retain weak references to the cached metadata.
This commit gives more chances to release the memory of projects
earlier, which may be important for larger projects.
Fixes #13497

@bot-gradle test this

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

@bot-gradle test this

OK, I've already cancelled the old build and triggered a new ReadyForMerge build for you.

@bot-gradle test this

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


fix(worker): fix breaking script when checking ORIGIN_HOSTNAME

Description
The last change to make sure that ORIGIN_HOSTNAME is well-formatted actually breaks the worker.
[wrangler:err] TypeError: url.match is not a function
    at Object.handleRequest [as fetch] (file:///src/index.mjs:40:12)
    at fetchDispatcher (file:///.wrangler/tmp/bundle-rIuIXO/middleware-loader.entry.ts:56:17)

In the script, url is an Url type and not a string. A property of url should be checked instead.
Related Issue
https://github.com/adobe/helix-cloudflare-prod-worker-template/issues/6
Motivation and Context
Fixes a bug.
How Has This Been Tested?
Checked with local tests (wrangler dev) and on published workers.
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

@trieloff

Thanks for reporting this issue and providing a PR!


[Feature][Config UI] Make selecting all repos in an org for GitHub more intuitive

Search before asking

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

Use case
Several users have raised the problem of not being able to select all repos in a org by one click in GitHub. This action is actually possible in the current Config UI, but not intuitive enough that the users can figure out by themselves.
Right now:
The user needs to click on the org in order to load part of its repos, scroll down to the bottom until all repos are loaded, and then select the check box of the org.
New:
The user is able to select all repos under an org (the checkbox will not be disabled). When the user click on the org nmae or check the checkbox of the org, the miller column will load all repos. There will be a spinning wheel to indicate when they are still loading, because it might take take a while.

The checkbox in the above screenshot will not be disabled.
Prototype will be coming soon.
Description
No response
Related issues
https://github.com/apache/incubator-devlake/issues/4742
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

Updated - we will allow loading all data scope when users set up Connections.

After clicking on Load All Repositories:

This feature would be great for the Gitlab integration as well. Right now I have to click through several layers of our projects and force the GUI to fetch all repos by scrolling to the bottom before I'm able to select all repos under a particular project.

@yumengwang03 Any ETA on when this feature will be available? Trying to setup an org with 12K+ repos and it could come in very handy! @stepmr

@foxted Hi, we have been discussing this problem for a while, unfortunately, we don't have a universal solution so far since different data sources need to be handled differently.