Category not auto-added for second request with same Merchant as first.

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v1.4.53-2
Reproducible in staging?: Y
Reproducible in production?: Y
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:
1, Navigate to FAB > Request Money.
2, Enter the amount, select the user, and click "Add Split". Add Merchant and Category.
3, Proceed with splitting.
4, Again, navigate to FAB > Request Money. Enter the amount and select the same user as the first request. Click "Add Split" and add the same Merchant as before, but do not select Category. Split the bill.
5, Go to the IOU for the second request and observe that the Category is not automatically added.
Expected Result:
When requesting money twice for the same user, if the first request includes the Merchant and Category, the second request with the same Merchant should automatically select the Category.
Actual Result:
The Category is not automatically selected for the second request if the first request includes the Merchant and Category, but the second request lacks the Category selection.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[x] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/d8eb14fc-ab84-4abe-a513-0cc015e85a3c

View all open jobs on GitHub

We think this issue might be related to the #vip-vsb.

Proposal
Please re-state the problem that we are trying to solve in this issue.

Category not auto-added for second request with same Merchant as first.

What is the root cause of that problem?

We do not have a logic that auto select the category in the second request.

What changes do you think we should make in order to solve the problem?

When we create a money request, there is a field recentlyUsedCategories which stores all the recently used category. We can implement the feature based on that fact.
First, we need to subscribe the recentlyUsedCategories in here
Then, we can update this to:

    // Auto select the category if there is only one enabled category and it is required
    useEffect(() => {
        const enabledCategories = _.filter(policyCategories, (category) => category.enabled);
        if (iouCategory || !shouldShowCategories || (enabledCategories.length !== 1 && isEmpty(policyRecentlyUsedCategories)) || !isCategoryRequired) {
            return;
        }
        if (enabledCategories.length === 1) {
            IOU.setMoneyRequestCategory(transaction.transactionID, enabledCategories[0].name);
        } else {
            IOU.setMoneyRequestCategory(transaction.transactionID, last(enabledCategories).name);
        }
    }, [iouCategory, shouldShowCategories, policyCategories, transaction, isCategoryRequired, policyRecentlyUsedCategories]);

What alternative solutions did you explore? (Optional)
Reminder: Please use plain English, be brief and avoid jargon. Feel free to use images, charts or pseudo-code if necessary. Do not post large multi-line diffs or write walls of text. Do not create PRs unless you have been hired for this job.

Nice, thank you @gijoe0295. Closing as behaviour is expected.


How to prompt existing chatbot?

The evaluatePrompt does not know my chatbot. It's also not listed in agents.
I can create a chat session for that, but how do I prompt it still 🤔

ahhh predictions api


ISPN-5932 Drop ClusterRegistry and ScopedKey. Make org.infinispan.registry private API

https://issues.jboss.org/browse/ISPN-5932
Good riddance.

Integrated, thanks Tristan!


New option to flip image took by front camera

Added shouldFlipFrontCameraImage instance property on CameraManager, so that developers can decide whether to flip image took by front camera or not.
Default value is false so that default behavior won't change from previous versions of this library.
This resolves #102.

Thanks @gbmksquare 👍


Change: get_complete_position algorithm

in order to use the new UseCompleter and ClassNameCompleter's function, some changes need to be made.

changed candidate object's attribute name
updated get_complete_position method (need to find a better way to do this instead of a bunch of large-overhead-regex)
updated test cases
add some log

The reason for the deplete plugin returning cursor at the beginning of the class name was to make the deoplete fuzzy filter the full class with the namespace. Now It's getting confused, see the screen shot.

The reason for the deplete plugin returning cursor at the beginning of the class name was to make the deoplete fuzzy filter the full class with the namespace. Now It's getting confused, see the screen shot.

Please try this PR: https://github.com/padawan-php/padawan.php/pull/82
It's not only for use completions, but also used for expressions like:
\Path\To\Some\Class::staticMethod();

which is returned by ClassNameCompleter

Please try this PR: https://github.com/padawan-php/padawan.php/pull/82
It's not only for use completions, but also used for expressions like:
\Path\To\Some\Class::staticMethod();

which is returned by ClassNameCompleter

Also, when making candidate list of the root namespace in a large project can be very slow. Adding some prefix can definitely increase the performance.

Also, when making candidate list of the root namespace in a large project can be very slow. Adding some prefix can definitely increase the performance.


[v3] divides? (D updot) doesn't have num, lst and lst, num implemented

A little goofy ngl

Not sure why this didn't get closed when #1895 was merged


Some tests must be improved.

Some unit tests need to be rewritten. They either were written too lazy, use random data or use large data (when it is unnecessary since this is not benchmark). The difficulty of unit testing is thinking about the corner cases.

Tests to improve:
Graphs:

[ ] dijkstra_shortest_paths_test: Add more tests. Also consider to change the dijkstra function interface to match interfaces of other graph algorithms.

Sorting:

[ ] counting_sort_test: Remove <random> usage.

Strings:

[ ] knuth_morris_pratt_test: Remove <random> usage.
[ ] lcp_array_test: Remove <random> usage.
[ ] suffix_array_test: Remove <random> usage.
[ ] z_algorithm_test: Improve test quality in general. Add a test where each element of the returned sequence is checked.

Done.


Command line usage

I'm considering using papaya in a larger project involving image processing, although it's lacking a few key features. One being the ability to open a file provided as an argument, "./papaya ~/myimage.png". And the other is having a reliable way of closing the editor with the keyboard. The shortcut key reads Alt+F4, although this doesn't seem to work for me in Arch Linux. Otherwise Ctrl+W/Ctrl+Q would be prefered as additions.

I am still working on ironing out various bugs in Linux keyboard input. I will have a look at Alt+F4 as well.
That said, I really don't think you should be using Papaya directly in any larger project. It's still very much a work in progress. I will continue to add features, and maybe even a scripting API at some point, but I would heavily discourage using Papaya in any "real" project at the moment.
I will keep this bug open until I fix the Alt+F4 problem on linux.


Fix case for 'Bit/s' to 'bit/s'

The correct letter case for bits-per-second is "bit/s"
https://en.wikipedia.org/wiki/Data-rate_units

Could you also please change the msgid in the po files as well?
So that weblate is satisfied and the translators don't have to translate everything again?

Could you also please change the msgid in the po files as well?
So that weblate is satisfied and the translators don't have to translate everything again?

Could you also please change the msgid in the po files as well?
So that weblate is satisfied and the translators don't have to translate everything again?

To be fair in the translation files it appears both as "MBit/s" and "Mbit/s", in fact I noticed while translating. So it is not additional work for the translators, it is in fact a removal and I understand it should happen automatically by running the xgettext scripts?
If needed I can also add it to the pull request. Should I remove the sentence from templates or also all the translations?

Could you also please change the msgid in the po files as well?
So that weblate is satisfied and the translators don't have to translate everything again?

To be fair in the translation files it appears both as "MBit/s" and "Mbit/s", in fact I noticed while translating. So it is not additional work for the translators, it is in fact a removal and I understand it should happen automatically by running the xgettext scripts?
If needed I can also add it to the pull request. Should I remove the sentence from templates or also all the translations?

The following commit changes the luci-i18n-base-* for all languages
modules/luci-mod-status/htdocs/luci-static/resources/view/status/wireless.js
The following commit changes the luci-18n-statistics-* for all languages
applications/luci-app-statistics/htdocs/luci-static/resources/statistics/rrdtool/definitions/iwinfo.js luci-i18m-statistics
I just wanted to say that if we make this change, then we should also adjust all msgid in the languages bo files. This should be added to this pullrquest. Then not all translations will be invalid and the translators don't have to start all over again.
If there is another place that does not fit, you can make another pullrequest. I am a fan of keeping the changes per pullrequest small. Experience has shown that this is then quickly reviewed and excepted.

The following commit changes the luci-i18n-base-* for all languages
modules/luci-mod-status/htdocs/luci-static/resources/view/status/wireless.js
The following commit changes the luci-18n-statistics-* for all languages
applications/luci-app-statistics/htdocs/luci-static/resources/statistics/rrdtool/definitions/iwinfo.js luci-i18m-statistics
I just wanted to say that if we make this change, then we should also adjust all msgid in the languages bo files. This should be added to this pullrquest. Then not all translations will be invalid and the translators don't have to start all over again.
If there is another place that does not fit, you can make another pullrequest. I am a fan of keeping the changes per pullrequest small. Experience has shown that this is then quickly reviewed and excepted.

So it is not additional work for the translators, it is in fact a removal and I understand it should happen automatically by running the xgettext scripts?

Feckert means that you should correspondingly fix in .po files (and the .pot) the translation English source strings ("msgid"), which serve as the keys for the actual translations ("msgstr").
see example a a recent string change that I did.
https://github.com/openwrt/luci/commit/e2ac2e92b0de9f4be78b090ad7fc62a8df222b37

So it is not additional work for the translators, it is in fact a removal and I understand it should happen automatically by running the xgettext scripts?

Feckert means that you should correspondingly fix in .po files (and the .pot) the translation English source strings ("msgid"), which serve as the keys for the actual translations ("msgstr").
see example a a recent string change that I did.
https://github.com/openwrt/luci/commit/e2ac2e92b0de9f4be78b090ad7fc62a8df222b37

@hnyman I rebased and squashed the change with the language updates. As I tried to say, in the base.pot there were both MBit/s and Mbit/s, so I just unified the two strings.
Also please note that you should run an ./build/mkbasepot.sh && ./build/i18n-update.pl as there is some pending updates to the language lines and translations.

@hnyman I rebased and squashed the change with the language updates. As I tried to say, in the base.pot there were both MBit/s and Mbit/s, so I just unified the two strings.
Also please note that you should run an ./build/mkbasepot.sh && ./build/i18n-update.pl as there is some pending updates to the language lines and translations.

There is a slight conflict as I merged the weblate changes, including a MBit/s Italian translation from you...
I will manually edit your commit to fix that.

There is a slight conflict as I merged the weblate changes, including a MBit/s Italian translation from you...
I will manually edit your commit to fix that.

Applied as dfa82b4494

Applied as dfa82b4494


so many errors!!!


Adds flag for Route Generation

if fabric8.openshift.generateRoute is set to false no route.yaml will be generated and if set true or not set it will generate route.yml
#1209

@piyush1594 LGTM


[kjobctl] Support $SLURM_JOB_NODELIST in Slurm mode.

What type of PR is this?
/kind feature

What this PR does / why we need it:
Support $SLURM_JOB_NODELIST in kjobctl Slurm mode.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
This is an example of variable values:
$ echo $SLURM_JOB_NODELIST
slurm-profile-slurm-84f7w-0.slurm-profile-slurm-84f7w,slurm-profile-slurm-84f7w-1.slurm-profile-slurm-84f7w

$ echo $SLURM_JOB_FIRST_NODE
slurm-profile-slurm-84f7w-0.slurm-profile-slurm-84f7w

Does this PR introduce a user-facing change?

NONE

/cc @mimowo


Environment variable name APPLICATIONINSIGHTS_SAMPLING_REQUESTS_PER_SECOND is incorrect.

The correct environment variable name is APPLICATIONINSIGHTS_SAMPLING_LIMIT_PER_SECOND
see https://github.com/microsoft/ApplicationInsights-Java/blob/05e28546c320275ce8093017962dc1168f2ff1ea/agent/agent-tooling/src/main/java/com/microsoft/applicationinsights/agent/internal/configuration/ConfigurationBuilder.java#L71-L72
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: ea43b302-7793-3e32-189e-d2949d399f25
Version Independent ID: 46ac34e0-f0ad-bc1c-a170-c64305e1af9b
Content: Configuration options - Azure Monitor Application Insights for Java - Azure Monitor
Content Source: articles/azure-monitor/app/java-standalone-config.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @AaronMaxwell
Microsoft Alias: aaronmax

@sschu Thanks for your comment. We will review the issue and get back to you shortly.


Add locale-specific action handlers to trigger certain events

Today, the keyboard action handler has some replacement logic, e.g. when tapping a close quotation when there is no opening one. However, there are a bunch of locale-specific replacements that are currently not handled.
For instance, tapping s then ˇ should replace the s with š.
In order to solve this, there should be locale-specific behaviors in place, to make it easy to customize the replacement logic for each locale.

I think diacritics should instead be handled with a new action type, so I'm closing this.

Hi @danielsaidi ,
do I understand correctly that in order to achieve this:

tapping s then ˇ should replace the s with š.

I need to add a custom KeyboardActionHandler where I'd catch these combinations and handle them accordingly to the current Locale.
Following the example: catching this combination and calling actionHandler.handle(.backspace) and inserting "š" instead.
Thanks!

Hi @ondrejkorol
Yes, that's correct :) There's currently such a Kurdish thing in place, but I'd prefer to have localized handlers for this to make each smaller.
However, it would be AMAZING if we could come up with a way to solve it with a new type.
I write a bit about it here: #660


Invalid version error when connecting to a Beta image deployment

It looks like because the release numbering on the Beta images is non-standard, Eclipse trows this error when adding the device:

The image file is: BIGIP-bigip13.1.x-roxy-<IP_ADDRESS>.<IP_ADDRESS>-0.0.23.iso

Repository Archived


Use boa-dev compare instead

Codecov Report

Merging #4 (fc393ee) into main (567e250) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head fc393ee differs from pull request most recent head 2d2ebff. Consider uploading reports for the commit 2d2ebff to get more accurate results

@@           Coverage Diff           @@
##             main       #4   +/-   ##
=======================================
  Coverage   61.81%   61.81%
=======================================
  Files           7        7
  Lines         584      584
=======================================
  Hits          361      361
  Misses        223      223


dos not support redirect url

could you please support redirected url
if you open url http://market.mesbahey.com\imagefile\1DgdX3bTK3 it is redirect to image file and will fail to load image
if you see this lib for android it is have more options and you can Benefits from it . It is Amazing lib for java app
https://github.com/nostra13/Android-Universal-Image-Loader
thanks in advance

Thanks, @oazya500 .
I will support it.

I released  0.12.2

thanks allot


merge from arlac77/template-github-action,arlac77/template-kronos-component

package.json

chore(deps): add ^7.3.1 remove ^7.3.0 (devDependencies.c8)

Codecov Report

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

@@            Coverage Diff            @@
##            master     #1388   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines           39        39
  Branches         1         1
=========================================
  Hits            39        39

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 2ae22e9...0764481. Read the comment docs.


Upgrade adyen api version

TODO: fix failing tests

This PR won't be done as the newest version of Adyen frontend JS there is a different flow. We updated the backend API to the newest version already: https://github.com/commercetools/commercetools-adyen-integration/pull/852


zero byte video file on ios 17

Running the Camera.MAUI.Test test app on iOS 17 on an iPhone 15 Pro the video file seems to be corrupted (it produces a file that is zero bytes long) after recording.  This worked last time I tried it on iOS 16.
I added a couple of lines to debug:
https://github.com/hjam40/Camera.MAUI/compare/master...apoland:Camera.MAUI:debug-video?expand=1#diff-a5e8cccef5d51b0fc414bd64a4e84c580535f101b756ffac87877dfed7a114c1R106-R107
[0:] Start recording result Success
Thread started:  #9
[0:] Stop recording result Success
Thread started:  #10
[0:] Video size: 0
Thread started:  #11
[0:] MediaManager: Error: Cannot Open - This media may be damaged.

i am facing the same problem of zero size video file, is anyone able to fix this?

Running the Camera.MAUI.Test test app on iOS 17 on an iPhone 15 Pro the video file seems to be corrupted (it produces a file that is zero bytes long) after recording. This worked last time I tried it on iOS 16.
I added a couple of lines to debug: https://github.com/hjam40/Camera.MAUI/compare/master...apoland:Camera.MAUI:debug-video?expand=1#diff-a5e8cccef5d51b0fc414bd64a4e84c580535f101b756ffac87877dfed7a114c1R106-R107
[0:] Start recording result Success
Thread started:  #9
[0:] Stop recording result Success
Thread started:  #10
[0:] Video size: 0
Thread started:  #11
[0:] MediaManager: Error: Cannot Open - This media may be damaged.

did you found any solution to it?

I have the same issue as well.

Same

FYI, this issue is fixed on my fork in release 1.4.6.


GetFileList is not async but called with await.

This following is called as async, but is not async.
call :
string[] files = await backupContext.CallActivityAsync<string[]>(
"E2_GetFileList",
rootDirectory);
method:
public static string[] GetFileList(
[ActivityTrigger] string rootDirectory,
ILogger log)

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

ID: 62bbedb6-d4c5-a809-31ad-b8c843ac0f52
Version Independent ID: 07c4822d-0aad-5a8a-7f16-56835f325a42
Content: Fan-out/fan-in scenarios in Durable Functions - Azure
Content Source: articles/azure-functions/durable/durable-functions-cloud-backup.md
Service: azure-functions
GitHub Login: @kashimiz
Microsoft Alias: azfuncdf

@realdanielbyrne Though the function being invoked is not an async function, the method being called in that line, CallActivityAsync is an async function (which internally invokes GetFileList).
Also, though it may seem that you are calling another function directly, in reality (quoting the docs here)

"Orchestrator functions schedule activity functions and receive their responses via internal queue messages"

You can read more about how Durable Functions work here.

@realdanielbyrne Just following up... Hope my comment earlier helps.


Add debugging option

@hugobast

@ciscoriordan  @wenga86 i'm going through old PRs...  feel free to merge this or i will just close and delete the fork in a few days.

I don't have write access to merge, but it LGTM.


Feat/oracle

解决OpenIdConnect在Oracle环境下出现的问题
1、标识符超出30个字符导致无法执行数据库迁移
2、字段必填问题
Checklist
本地测试成功执行迁移

@zhenlei520 @MayueCif  PTAL

@doddgu LGTM

@ihuangxing You should commit merge to main branch, not feat/oracle

A new pr has been submitted #295 , the current pr is closed


Add auto hide the header while scrolling.

Hide the header tab while scrolling down. Facing difficulties while reading the blog.

@sayansur25 can you assign this to me! I would be glad to fix this up.

@sayansur25 Can you please add some screenshots or animated gifs highlighting the issue you are having?  Also, can you let us know if you experience the same behavior on our beta site ?
Thank you.

Even I had the same prblem

I had the same issue

I had the same issue

I had the same issue

Closing as stale.


Error Term in Verlet Integration Page

Hi team,
This is regarding the Verlet Integration page that's here: https://www.algorithm-archive.org/contents/verlet_integration/verlet_integration.html
Similar to how Equation 4 is sum of Eq. 2 and 3, we can see that Eq. 5 is a difference of 2 and 3. When we do that, the error term turns out to be O(del. t^3) and not O(del. t^2) as stated in the page. Did I get something wrong? Or should that page be corrected? Happy to provide more information if this post is not clear.
Best,
Harsha

Right, I was confused as well when reading your issue, and I would have agreed with you.
However, I did the working out, and this is why it's an O(dt^2).
As you noticed, Equation 5 is the difference of equations 2 and 3. As such, the jerk term does not cancel out, because of their different signs.
Thus (using dt instead of delta t): x(t + dt) - x(t-dt) = 2 v(t) dt + 1/3 b(t) dt^3 + O(dt^4).
Solving for v(t), we get: v(t) = [x(t + dt) - x(t - dt)] / (2 dt) - 1/6 b(t) dt^2 + O(dt^3).
However, since we don't use the jerk in the computation, it is lumped in a O(dt^3) at the start.
As such, we have a O(dt^3) in the difference step, and thus only a O(dt^2) term in the velocity.
Hope this helped!

I think this should be explicitly mentioned. I'll do that today.

@Amaras Ah, yes. It's a mistake on my part! That makes perfect sense, thank you!

@leios that sounds great. Maybe I'll leave this open for you to track and close when you're done?


fix: address several issues that instantly fail quality tests

These changes are determined during the process of using this cookiecutter to create a new django app, and getting the output to pass tests.  Changes include:

Replace ugettext_lazy with gettext_lazy (the latter is compatible with Django 4+).
Force Sphinx to ignore decisions/README.rst so that pydocstyle doesn't complain about it not having a header.  This file doesn't appear to be intended for output anyway.
Force isort to skip migration files.  This was already the case for django IDAs, but was missed for django apps.

Merge checklist:

[ ] All reviewers approved
[x] CI build is green
[ ] Changelog record added
[ ] Documentation updated (not only docstrings)
[x] Commits are squashed

Thanks!


🐛 (Extensions API) Shaking input box while repeatedly updating box.title

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

VS Code Version: 1.74.0
OS Version: Ubuntu 20.04.1

Version: 1.74.0
Commit: 5235c6bb189b60b01b1f49062f4ffa42384f8c91
Date: 2022-12-05T16:47:18.476Z
Electron: 19.1.8
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 5.15.0-52-generic
Sandboxed: No

The Problem
When an extension assigns a value to an input box's title property, the box slightly moves to the right or left (See the screen capture below). Note that it's not because of title text length changes or other text rendering details. Even re-assigning the title property with a constant value makes the shaking happen. I've made a repo to help verify this on your side.
As of my observations, the problem was not present in VS Code v1.72. It's now happening in v1.74 (I'm not sure about v1.73 since I've skipped that update).
Steps to Reproduce:

Clone this repo (it's a sample extension):
https://github.com/babakks/vscode-verify-shaking-input-box
Open the cloned directory in VS Code.
Start debugging by pressing F5.
In the opened window, open the command palette, find the "Hello World" command and invoke it.
You should now see a shaking input box, like this:

What the extension does
The extension just creates an input box, and repeatedly updates the title of the input box with a constant value:
const intervalId = setInterval(() => {
    box.title = "A constant string";
}, 500);

I've pushed a fix for that.

This bug has been fixed in the latest release of VS Code Insiders!
@babakks, you can help us out by commenting /verified if things are now working as expected.
If things still don't seem right, please ensure you're on version 270c3b12906a88e9c67fc5ddef87be6aa3b75551 of Insiders (today's or later - you can use Help: About in the command palette to check), and leave a comment letting us know what isn't working as expected.
Happy Coding!

/verified


Greek Translation

Hello,
I add Greek translation for android library, please check and update in master if it is possible.
Thanks,
Ntsounos Paraskevas

Fantastic, thanks!

Oops, I merged this to master.  I've been meaning to remove develop for a while, so I'll sort that out when I bump the version and release this.

@ntsparis @mvorisis This has now been released in 1.5.1. Thanks again for the contribution 🥇


Sync tooling image into ACR

Which issue this PR addresses:
In clusters with no pull secret we can't advise to use oc debug node/ as it is not reachable.
What this PR does / why we need it:
This potentially is the first step to remove need in a pull-secret need for debugging tooling.
Test plan for issue:
N/a
Is there any documentation that needs to be updated for this PR?
potentially adding oc debug node/{node_name} --image acr.name/image somewhere

Is it possible to update the relevant image streams so that when oc debug node is run, then an --image parameter isn't even needed (ditto for the other relevant tooling in this case?)

Is it possible to update the relevant image streams so that when oc debug node is run, then an --image parameter isn't even needed (ditto for the other relevant tooling in this case?)

I think this is not necessary. Up to 4.5 there was a permanent "bug"/different behaviour where it was acting not as expected. It never used ImageStreamTag image and used default. This is now fixed in 4.6+ of binary and all IS are already from mirrored resources (from release payload)


Allow setting the listen address and port for zuul webapp

We can set the listen address and port for the zuul status port. By
default the listen address is <IP_ADDRESS> and we should make that default
<IP_ADDRESS> because we want to hide this route behind apache.
Signed-off-by: Jamie Lennox<EMAIL_ADDRESS>

Starting gate_github jobs.


revert(actions): add input to checkout different branch than default

This reverts commit e443bb44aea63957b34a37474e1ec44dadaa2fc2.
Specifying a branch, or defaulting to master, makes PR tests run on the latest commit on that branch, not on the PR commit.

So I guess what one would do is to set the ref explicitly if one wants to change, but not having it at all if you want the PR branch.
So something like:

  steps:
    - name: Checkout
      if: ${{ inputs.git-branch }} == ""
      uses: actions/checkout@v3
      with:
        # Needed for conventional commit linting.
        fetch-depth: ${{ inputs.fetch-depth }}

    - name: Checkout other branch
      if: ${{ inputs.git-branch }} != ""
      uses: actions/checkout@v3
      with:
        # Needed for conventional commit linting.
        fetch-depth: ${{ inputs.fetch-depth }}
        ref: ${{ inputs.git-branch }}


Update to 1.17

This update was tested on 1.17.1 and appears to be working as intended. I did remove the PageUp/PageDown feature, but that could be added back in relatively easily if it is necessary.

Why'd you close this?

Why'd you close this?

Minecraft is now on 1.18 and I figured instead of trying to PR into a seemingly dead repo multiple times I'll just update my fork of it.


Refactor Dockerfile.parser and directive

Don't expose parser.Directive outside of the parser package
Remove duplication in parser.Parse() and make it easier to support #29161

@yongtang I think this will make #29161 cleaner. No need to pass in stderr, you can just return Warnings as part of Result() and handle them in the builder.

Thanks @dnephin I am on the road but will try to take a look and update #29161 in the next several days. (my flight is with united airline 😢)

@dnephin Hmmm, just seen this. By having processFirstLine separate from processLine, it means adding additional parse directives is very problematic (which is what I'm currently trying to do in another PR).
What was the rationale for splitting the first line out and not just treating every line as "just another line".

Oh, I see. It's the BOM handling. I'll figure it out :)

Ya, I was thinking about that case when I was making these changes, but didn't want to worry about it in case we never added another directive.
I think d.processLine() can be called in both cases, and the Directive() d.processingComplete = true can be set to true on the first non-directive line, instead of after the first line.


Is there a need to release unpooled heap buffers?

I am aware that it is best practice to release() locally created buffers in a finally block.
However, is this strictly necessary for unpooled heap buffers (i.e., buffers obtained via Unpooled.buffer())?
My understanding is that unpooled heap buffers are entirely allocated on the Java heap, and their lifecycle is managed by the garbage collector, but I could be wrong on this.
BTW, this question is similar to #8139 but regarding a different type of buffer.

@ran-su generally speaking no... that said by default we will still report it as leak to make it easier for people to switch between allocators. Even if the GC manage the unpooled buffers we will ensure to give the direct memory back in a fast fashion when calling release. So all in all it is preferred to do so.


Dev versions should be valid PEP 440 versions

Today they have hyphens between parts. My read of 440 is they should have only periods.

Fixed in edb408bcf26b1e3c2562602ebad080a6b33aa9a6, 7400edd31dfe1217039b54e725d1cff70c3db2db.  New scheme is, e.g.,
1.3a20151008.32+g829b37a-dirty

1.3 is the main version number;
a marks as alpha, i.e. development version;
20151008 is the most recent development snapshot date;
32 is the number of commits since the snapshot;
829b37a is (a prefix of) the Git commit id;
-dirty means the working tree was dirty.

We use a now because dev sorts lexicographically after b, and because PEP 440 and Debian disagree on its interpretation.


Add SKU parameters for admin jumpbox and resource processor VMSS

Resolves #2403
What is being addressed
Added environment variables to customize SKUs:
ADMIN_JUMPBOX_VM_SKU="Standard_B2s"
RESOURCE_PROCESSOR_VMSS_SKU="Standard_B2s"

/test

/test-force-approve


comment out code

Another trial run branch - please ignore, ty!

Codecov Report

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

@@          Coverage Diff          @@
##           master   #496   +/-   ##
=====================================
  Coverage      98%    98%
=====================================
  Files          29     29
  Lines         702    702
  Branches      137    137
=====================================
  Hits          688    688
  Misses         12     12
  Partials        2      2

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 d6492c5...a7c6b31. Read the comment docs.


Can't Use By Native Base

Hi , i Use Native Base in My App ,How i Can Use "react-native-loading-placeholder" ?

No , i don't use expo!

Provide a usage example and error stack you are receiving.


altpgen error cds0001

Please include the following with each issue:
1. Describe the bug
We are getting error CDS0001 when authenticating to CDS using altpgen.
2. To Reproduce
The command we are running is:
./altpgen.exe -project:"C:\Users..\Tests\AL Gen2"
-packagecachepath:"C:\Users..\Tests\AL Gen2.alpackages"
-serviceuri:"https://cronus.crm6.dynamics.com"
-entities:contact
-baseid:50101
3. Expected behavior
We were expecting it to generate the AL file
4. Actual behavior
Output is:
Microsoft (R) AL Table Proxy Generator version <IP_ADDRESS>35
Copyright (C) Microsoft Corporation. All rights reserved
Authenticating CDS using ServiceUri=https://cronus.crm6.dynamics.com
error CDS0001:

5. Versions:

AL Language: 7.2.475273
Visual Studio Code: 1.57.0
Business Central: AU Business Central 18.1 (Platform 18.0.24800.26638 + Application 18.1.24822.24906)
List of Visual Studio Code extensions that you have installed: AL Language

Final Checklist
Please remember to do the following:

[X] Search the issue repository to ensure you are reporting a new issue

[x] Reproduce the issue after disabling all extensions except the AL Language extension

[X] Simplify your code around the issue to better isolate the problem

I had the same error, I changed the versión of my AL language extension to 7.1.453917 and thats worked for me.

Thank you - your suggestion worked.  We were using version 7.2.475273 and changed to 7.1.453917 and that solved the problem.

The fix for this issue has been checked in to the master branch. It will be available in the bcinsider.azurecr.io/bcsandbox-master Docker image starting from platform build number 26961 and VS Code Extension Version .
If you don’t have access to these images you need to become part of the Ready2Go program: aka.ms/readytogo
For more details on code branches and docker images please read:
https://blogs.msdn.microsoft.com/nav/2018/05/03/al-developer-previews-multiple-releases-and-github/
https://freddysblog.com/2020/06/25/working-with-artifacts/


Warning on import

Would be nice to fix this one.
WARNING: could not import StructIO.pack into COFF

Bump @Keno Would be nice to fix this to give warning free loading of Gallium.


Add results cache

References

Description

Add MongoDB as results cache to store the query results
Validation performed

Successfully built the package. MongoDB can be started by start-clp.sh and shut down by stop-clp.sh

Resolved conflicts


bug: Version number fails to resolve on Windows app

When running the Windows packaged application, the version number of the app fails to resolve.
How we reproduce the bug

Install the packaged GTFS Validator app on Windows.
Run validation on a feed.
Observe that version info is missing in the output report and in logs.

Expected behaviour
Version info should be present in the report.
Observed behaviour
Current version is missing.
Screenshots:

Logs:
Jun 03, 2022 4:36:29 PM org.mobilitydata.gtfsvalidator.runner.ValidationRunner run
INFO: VersionInfo: Version

Info{currentVersion=Optional.empty, latestReleaseVersion=Optional.empty}

Environment versions

validator version: 3.1.0
Java version: embedded
OS versions: Windows 10

This does not repro for the 3.1.0 CLI jar on Windows or Mac OS.  I've verified that the version number is correct in the jar's META-INF/MANIFEST.MF file.

It also doesn't repro when running the app locally via ./gradlew app:gui:run but does repro when running the packaged app on both Windows and Mac OS.  I've discovered that there's an exception being thrown within the VersionResolver background executor thread:
javax.net.ssl.SSLHandshakeException: Received fatal alert: handshake_failure
        at java.base/sun.security.ssl.Alert.createSSLException(Unknown Source)
        at java.base/sun.security.ssl.Alert.createSSLException(Unknown Source)
        at java.base/sun.security.ssl.TransportContext.fatal(Unknown Source)
        at java.base/sun.security.ssl.Alert$AlertConsumer.consume(Unknown Source)
        at java.base/sun.security.ssl.TransportContext.dispatch(Unknown Source)
        at java.base/sun.security.ssl.SSLTransport.decode(Unknown Source)
...

It unfortunately not caught by the uncaught exception handler because it's on a new thread.
If I had to guess, I'm thinking this must be a Java Module permissions issue?

Adding module dependencies on jdk.crypto.ec and jdk.crypto.cryptoki seems to fix the issue.
For the record, this issue also means that the validator will fail to download feeds from a url that uses https.
Per https://stackoverflow.com/questions/55439599/sslhandshakeexception-with-jlink-created-runtime


ADDON-34289: Added Text, Radio and CheckBox components

ADDON-34289
Added Text, Radio and CheckBox components

:tada: This PR is included in version 5.0.0-develop.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

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


Error running pretrained mobile net

2017-10-13 14:03:26.126089: W tensorflow/core/framework/op_kernel.cc:1192] Not found: Key Openpose/MConv_Stage6_L1_3_pointwise/BatchNorm/moving_mean not found in checkpoint
2017-10-13 14:03:26.126193: W tensorflow/core/framework/op_kernel.cc:1192] Not found: Key Openpose/MConv_Stage6_L1_3_depthwise/depthwise_weights not found in checkpoint
2017-10-13 14:03:26.126316: W tensorflow/core/framework/op_kernel.cc:1192] Not found: Key Openpose/MConv_Stage6_L1_4_depthwise/depthwise_weights not found in checkpoint
2017-10-13 14:03:26.126335: W tensorflow/core/framework/op_kernel.cc:1192] Not found: Key Openpose/MConv_Stage6_L1_3_pointwise/weights not found in checkpoint
2017-10-13 14:03:26.126340: W tensorflow/core/framework/op_kernel.cc:1192] Not found: Key Openpose/MConv_Stage6_L1_4_pointwise/BatchNorm/beta not found in checkpoint
2017-10-13 14:03:26.126343: W tensorflow/core/framework/op_kernel.cc:1192] Not found: Key Openpose/MConv_Stage6_L1_3_pointwise/BatchNorm/beta not found in checkpoint
Traceback (most recent call last):
File "inference.py", line 54, in
loader.restore(sess, '/media/joshua/5d133d05-fcd7-4aa1-b351-a386ed74b6b7/tf-openpose/models/pretrained/mobilenet_v1_0.50_224_2017_06_14/mobilenet_v1_0.50_224.ckpt')#'./models/trained/mobilenet_retry/model-100000')
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 1560, in restore
{self.saver_def.filename_tensor_name: save_path})
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 895, in run
run_metadata_ptr)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 1124, in _run
feed_dict_tensor, options, run_metadata)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 1321, in _do_run
options, run_metadata)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 1340, in _do_call
raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.NotFoundError: Key Openpose/MConv_Stage6_L2_5_pointwise/weights not found in checkpoint
[[Node: save/RestoreV2_358 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/task:0/cpu:0"](_arg_save/Const_0_0, save/RestoreV2_358/tensor_names, save/RestoreV2_358/shape_and_slices)]]
[[Node: save/RestoreV2_35/_665 = _Recvclient_terminated=false, recv_device="/job:localhost/replica:0/task:0/gpu:0", send_device="/job:localhost/replica:0/task:0/cpu:0", send_device_incarnation=1, tensor_name="edge_1386_save/RestoreV2_35", tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/gpu:0"]]
Caused by op u'save/RestoreV2_358', defined at:
File "inference.py", line 53, in
loader = tf.train.Saver()
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 1140, in init
self.build()
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 1172, in build
filename=self._filename)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 688, in build
restore_sequentially, reshape)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 407, in _AddRestoreOps
tensors = self.restore_op(filename_tensor, saveable, preferred_shard)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.py", line 247, in restore_op
[spec.tensor.dtype])[0])
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/gen_io_ops.py", line 663, in restore_v2
dtypes=dtypes, name=name)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/op_def_library.py", line 767, in apply_op
op_def=op_def)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 2630, in create_op
original_op=self._default_original_op, op_def=op_def)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 1204, in init
self._traceback = self._graph._extract_stack()  # pylint: disable=protected-access
NotFoundError (see above for traceback): Key Openpose/MConv_Stage6_L2_5_pointwise/weights not found in checkpoint
[[Node: save/RestoreV2_358 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/task:0/cpu:0"](_arg_save/Const_0_0, save/RestoreV2_358/tensor_names, save/RestoreV2_358/shape_and_slices)]]
[[Node: save/RestoreV2_35/_665 = _Recvclient_terminated=false, recv_device="/job:localhost/replica:0/task:0/gpu:0", send_device="/job:localhost/replica:0/task:0/cpu:0", send_device_incarnation=1, tensor_name="edge_1386_save/RestoreV2_35", tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/gpu:0"]]

I will share it after I trained it with descent performance.

@ildoonet  Has this problem been fixed？ I have also met this error, and don't know how to do. Can you help me?

Close due to inactivity.


firstframe setups broken when mutedAutplay is supported

Exarcerbated incarnation of #50.
Workaround - which arguably could be seen as solution:
If neither splash nor autoplay is configured and mutedAutoplay supported, enforce a splash setup.
Visually no difference; video is not preloaded which is the better way on mobiles in nearly all cases anyway.

Will be fixed in core 7.1.2, workaround confined to 7.1.[01], and to be removed at some point.


Remove warning about PyOpenSSL in setup.py on Windows

Comment on ea11331 should explain most of the reasoning behind this.  I decided to remove the warning after seeing that on Windows this warning is usually shown to a user several times.  Additionally you can't really tell which package the warning came from which is misleading and could lead someone to believe something is wrong when there probably isn't.

@guidow, PTAL

Good call.


Setup repo

The github repo must be setup to ensure src code integrity, force best practices and usability (primarily via docs such as a readme and issue templates)
This includes the below of which sub tasks are to be created for

[x] add read me
[x] python linting / formatting (local and remote repo)
[x] ci - run python checks on commit - Done thanks to python template :smile:
[x] force pull requests
[x] block commits to master
[x] Epic template
[x] sub-task template
[x] issues template

all tasks complete

all tasks complete


feat(app): add a preview for business edition features

Close #4558

@pull-dog up

/azp run

@pull-dog down


Shader uniform blocks fail to populate on Veldrid when one block is unused

This became present while I was working towards fixing sRGB conversion, since I removed toSRGB usage in sh_Blur, which was the only usage of the global uniform block, therefore SPIR-V omitted it and blur rendering broke completely. I've managed to work around it by tricking SPIR-V into thinking that it's still used.
SPIR-V omits any uniform block that's not used by a shader, causing all other uniform blocks in the shader to be re-arranged, breaking our BindUniformBlock logic in VeldridShader from working.
For reference, here's the shader and reflection output of SPIR-V MSL compilation with an unused uniform block:

As you can see, _27_29 is not used by the shader, therefore SPIR-V omitted it and changed the set ID of _139_141 to [[buffer(0)]] meanwhile the reflection points it to be [[buffer(1)]] instead.
We can potentially work around this by modifying the shader output to ensure the parameters of the shader method matches the reflection, but I'm not doing this yet as I'm not even sure if Metal is the only renderer that has this bug.

An alternative direction would be to let every shader also be responsible for including the global uniform data themselves (possibly in a tidied up #include <global_uniforms> format), that way we don't have to worry about working around this ourselves.

#include <global_uniforms>

I wouldn't be against this, but it will need some supportive work. Shaders should probably support #pragma once (only include the file once in sources).

I was about to create an issue in which VeldridUniformBuffer.Data always returns default data and I guess it's related to this one?

That sounds different, this issue only affects what the shader reads, it doesn't affect the buffer content and what not.

I've attempted this by following the approach mentioned above but I've ended up with even more concerning bugs on Metal, due to the sole reason of the metal cross-compiler not following the slot numbers specified in the shader (say a vertex shader uses a global uniform, but a fragment shader doesn't use any and instead uses local uniforms; this will result in the cross-compiler assigning both the global uniform and local uniform blocks to slot 0 because they're the first to appear on both files, which is bad because you can't have different data on the same slot).
I don't think this is resolvable until we move away from veldrid-spirv, and the existing workarounds should do enough for the time being.

This has been alleviated a little bit via https://github.com/ppy/veldrid-spirv/pull/4


4.0.4 healthcheck is failing

In healthcheck.sh you're using the new variable: curl -f http://localhost:${FLATNOTES_PORT}/health || exit 1
If you don't set the variable it uses the docker 'inside' port to query the wrong 'outside' port. In my case 8080 is used by another container. I can obviously resolve this by setting an environment variable but thought you might want to be made aware.

Actually, even with the FLATNOTES_PORT set it still fails the healthcheck. http://ipaddress:port/health says "OK".
docker exec -it flatnotes sh
# curl -f http://localhost:${FLATNOTES_PORT}/health || exit 1
Returns: "OK"
docker ps
CONTAINER ID   IMAGE                                  COMMAND                  CREATED         STATUS                     PORTS                                                                                NAMES
4cc488aba7c4   dullage/flatnotes:latest               "/entrypoint.sh"         6 minutes ago   Up 5 minutes (unhealthy)   8080/tcp, <IP_ADDRESS>:9080->9080/tcp, :::9080->9080/tcp                                              flatnotes
name: flatnotes
services:
    flatnotes:
        container_name: flatnotes
        image: dullage/flatnotes:latest
        environment:
            PUID: 1000
            PGID: 1000
            FLATNOTES_AUTH_TYPE: password
            FLATNOTES_USERNAME: lolusername
            FLATNOTES_PASSWORD: hunter2
            FLATNOTES_SECRET_KEY: hunter2
            FLATNOTES_PORT: 9080
        volumes:
            - /mnt/docker/flatnotes/data:/data
            # Optional. Allows you to save the search index in a different location:
            # - "./index:/data/.flatnotes"
        ports:
            - 9080:9080
        restart: unless-stopped

A default value for this new variable is set in the image so should be ok (unless you're using Synology where weird things seem to be happening).
This said, I accidentally pushed an image early where the healthcheck.sh file had the wrong line endings. Maybe you have this build? Can you try pulling the image again to see if that fixes it?

Confirmed working after re-pull. Thanks so much!


Incorrect Code Generated when field is marked as writeOnly

This is identical to the issue fixed by #813. However, that PR only fixed fields marked as readOnly. We would want the opposite behavior for writeOnly fields. I.e., these fields would be included in the mutator functions but not the functions that retrieve data.
What are the steps to reproduce this issue?
Example Schema:
openapi: 3.0.3
paths:
  /api/vx/cats/:
    get:
      operationId: api_vx_cat_provider_profile_retrieve
      tags:
      - api
      responses:
        '200':
          content:
            application/json:
              schema:
                $ref: '#/components/schemas/Cat'
          description: ''
        '404':
          content:
            application/json:
              schema:
                type: object
                properties:
                  detail:
                    type: string
          description: ''
    post:
      operationId: api_vx_cat_provider_profile_create
      tags:
      - api
      requestBody:
        content:
          application/json:
            schema:
              $ref: '#/components/schemas/Cat'
        required: true
  components:
    schemas:
      Cat:
        type: object
        properties:
          experience:
            type: string
            nullable: true
          title:
            type: string
            nullable: true
            maxLength: 50
          image:
            type: string
            writeOnly: true
          photo:
            allOf:
            - $ref: '#/components/schemas/Image'
            readOnly: true
          years:
            type: integer
            nullable: true
        required:
        - image_url
        - profile_photo

This is company code, so I'm renaming and pasting as little as possible. We are generating react-query code from this schema
What happens?
Relevant generated type:
export interface CatProviderProfile {
  experience?: string | null;
  title?: string | null;
  image: string;
  readonly photo: Image;
  years?: number | null;
}

Relevant generated code
export const apiCatProviderProfileRetrieve = (signal?: AbortSignal) => {
  return HTTP<CatProviderProfile>({
    url: `/api/vx/cats/`,
    method: 'get',
    signal,
  });
};

export const getApiCatProviderProfileRetrieveQueryKey = () =>
  [`/api/vx/cats/`] as const;

export const getApiCatProviderProfileRetrieveQueryOptions = <
  TData = Awaited<ReturnType<typeof apiCatProviderProfileRetrieve>>,
  TError = ErrorType<ApiCatProviderProfileRetrieve404>
>(options?: {
  query?: UseQueryOptions<Awaited<ReturnType<typeof apiCatProviderProfileRetrieve>>, TError, TData>;
}): UseQueryOptions<Awaited<ReturnType<typeof apiCatProviderProfileRetrieve>>, TError, TData> & {
  queryKey: QueryKey;
} => {
  const { query: queryOptions } = options ?? {};

  const queryKey = queryOptions?.queryKey ?? getApiCatProviderProfileRetrieveQueryKey();

  const queryFn: QueryFunction<Awaited<ReturnType<typeof apiCatProviderProfileRetrieve>>> = ({
    signal,
  }) => apiCatProviderProfileRetrieve(signal);

  return { queryKey, queryFn, refetchOnMount: false, refetchOnWindowFocus: false, ...queryOptions };
};

export type ApiCatProviderProfileRetrieveQueryResult = NonNullable<
  Awaited<ReturnType<typeof apiCatProviderProfileRetrieve>>
>;
export type ApiCatProviderProfileRetrieveQueryError = ErrorType<ApiCatProviderProfileRetrieve404>;

export const useApiCatProviderProfileRetrieve = <
  TData = Awaited<ReturnType<typeof apiCatProviderProfileRetrieve>>,
  TError = ErrorType<ApiCatProviderProfileRetrieve404>
>(options?: {
  query?: UseQueryOptions<Awaited<ReturnType<typeof apiCatProviderProfileRetrieve>>, TError, TData>;
}): UseQueryResult<TData, TError> & { queryKey: QueryKey } => {
  const queryOptions = getApiCatProviderProfileRetrieveQueryOptions(options);

  const query = useQuery(queryOptions) as UseQueryResult<TData, TError> & { queryKey: QueryKey };

  query.queryKey = queryOptions.queryKey;

  return query;
};

What were you expecting to happen?
If you were to use the useApiCatProviderProfileRetrieve function, the result is typed to have the image key, when in fact,  that key is not returned on GET request. It is data only expected on PUT|POST requests.
Any other comments?
One solution I could imagine is generating separate types for that include only writeOnly fields and only readOnly fields.
What versions are you using?
Operating System: Running in docker container node:16.20.0-bullseye-slim
Package Version: 6.14.4

Did I open this issue incorrectly?

Definitely created the ticket ok.  Interested in submitting a PR to fix this issue?


Add support for STORAGE_S3_TOKEN

Describe the feature you'd like
Add support for defining STORAGE_S3_TOKEN in addition to STORAGE_S3_KEY and STORAGE_S3_SECRET for S3 storage.
Temporay S3 credentials generated by STS (Security Token Service) need to define credentials['token'] in addition to credentials['key'] and credentials['secret'].
See AWS SDK docs about this: https://docs.aws.amazon.com/sdk-for-php/v3/developer-guide/guide_credentials_temporary.html#providing-temporary-credentials-to-the-sdk-php
Describe the benefits this would bring to existing BookStack users
This permit to use temporary and limited-privilege S3 credentials which increase security and permit some automation process.
Can the goal of this request already be achieved via other means?
No.
Have you searched for an existing open/closed issue?

[X] I have searched for existing issues and none cover my fundamental request

How long have you been using BookStack?
1 to 5 years
Additional context
I only use temporary and auto-generated credentials everywhere i can as a security principle.

Hi @vlaborie, thanks for the request.
Just so I understand, you'd be fetching the token externally then regularly be updating the token value for BookStack?
Just trying to understand the use and scenario, since I really don't want to expand the scope/support of these settings, especially where service specific and/or where not commonly needed or desired.

Hi @ssddanbrown,
Yes that's what i do. This is very usefull because the fetching process is fully automated.
I just need to create an S3 bucket and an S3 policy and the credentials are automaticaly managed.
Should be noted that is not specific to AWS but common to multiple S3 providers (i use it with Minio).


Embedding a Google Sheets preview

Is there a way to embed a preview of a Google Sheets file?
I'm imagining a user adds a link with public permissions and selects a column x row range to display.
Or how would one go about implementing this?

Hi @cgil, been super busy so it's taken me a while to get back to you on this.
I think a reasonable way to go about implementing this is to create a custom Tiptap extension, which Tiptap has a guide for here: https://tiptap.dev/guide/custom-extensions (so be sure to check out the "Create New Extensions" section)
You'll seemingly need to take advantage of Node Views to render your own custom interactive JS-based content (mentioned in the above, and with its own more detailed guide here https://tiptap.dev/guide/node-views). I'm not familiar with Google Sheets' APIs, but presumably you could create a Node View that has interaction for the user to select the Sheets URL column/row range, and use that to render the Sheet in an iframe based on their choice. Hope that's helpful!


⚠ ClamAV has degraded performance

In 161f4e3, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 83 ms

Resolved: ClamAV performance has improved in f6b8187 after 5 minutes.


Update nested-virtualization.md

The name of the new swith  "InternalNATSwitch" is not relevant with the screenshot and the other steps

@gaubert-ms : Thanks for your contribution! The author, @cynthn, has been notified to review your proposed change.

Close/reopen for fresh validation post-migration.

#sign-off Thank you!


newl doesn't work even though it is in the readme

(Line: 7) Error: Command newl does not exist.

I also noticed that in the "commands" dictionary in the code it does not contain the newl command, which may be something to do with it..

Note: a "printl" command with no argument works as a replacement. Example:
var coolvalue
set coolvalue 69
var badvalue
set badvalue 21
switch badvalue coolvalue
printdec badvalue
printl
printdec coolvalue

It was a really long time ago, but I do remember newl being a thing. I probably removed it somewhere along the way. Thanks for the interest and noticing!

Fixed in 43b50fa. Thx!

Fixed in 43b50fa. Thx!

Awesome!


[Bug] Brave does not always use --user-data-dir

Bug Report
Package Name: brave
Current Behavior
Brave does not respect its installation context when set as the default browser. Leading to differing configurations when launched via different contexts (Start Menu/Pinned Icon in taskbar/brave command/Open with...)
Expected Behavior
Brave uses the same User Data directory regardless of its launch context unless otherwise specified. --user-data-dir
Possible Solution

Include a post_install like
New-Item -ItemType Junction -Path "$env:AppData/../Local/BraveSoftware/Brave-Browser/User Data" -Target "$persist_dir/brave/User Data"
Downside: requires elevated prompt or depends: sudo
Include the post_install as an optional extra to be run by the user after install (like some of the context menu .reg files)
Downside: again, requires elevation. But, this is only run if the user wishes.
Get down and dirty with the registry, including a .reg file to fix the file and protocol associations.
Downside: would be a massive PITA.
Hit up the brave developers, see if they would take the reigns on this?

System details
Windows version:
Microsoft Windows 10 IoT Enterprise LTSC 19044
OS architecture:
64
PowerShell version:
7.2.6
Scoop Configuration
{
  "last_update": "2022-10-20T19:02:10.7870260+03:00",
  "scoop_repo": "https://github.com/ScoopInstaller/Scoop",
  "scoop_branch": "master"
}

all browsers have this problem when specifying a profile location.

I see. I'll just close this then.

I did not mean for you to close this issue, it's something worth discusing

I would reopen this issue as well. I ran down a rabbit hole to figure out with "Open with ..." in Brave does not load my profile as well as several other locations.
I figured out, and it is how OP stated, brave.exe needs --user-data-dir. My observation is, if by any means brave is started without it, it spawns a new instance and uses/creates %appdata%/... as profile. This can also be observed in the taskbar, it will have its own new icon in the list of open/pinned applications. The user is now confused, which icon to pin to the taskbar.
We should try to figure out that brave never ever tries to use the profile in AppData. The scoop installer script already takes care of copying the full folder from there if one exists. The installer also writes a start menu entry for Brave in Scoop Apps with the correct --user-data-dir argument. This seems to be the only place where it is defined. If someone opens brave from there or has pulled the icon from there to the taskbar, everything is fine.
One immediate problem occurs if someone tries to make brave as default browser via the brave settings. It will offer itself as selection for Browser in Default Apps, but the entry seems to be missing the --user-data-dir. If someone now opens a link from any external program, it will start brave without this option and thus creating a separate instance. Does somebody know what all exactly happens during this "Make brave as default browser" button in the brave settings?
I was able to fix it with help of filetypesman from the nirsoft bucket (project homepage), but I also needed a file-type helper .html to assign Open with ... with the brave browser entry as it seems I couldn't find the exact place where this Default Apps list for Browsers comes from. The reason of using this tool instead of the Windows Settings integrated one, was, it allowed the command to be edited so I could pass --user-data-dir:

Changing the command was only needed once, as it seems this relation is mapped to User Choice which in my case is called BraveHTML.SASV...:

This seems to also have automatically fixed the Default Apps Browser selection (please can someone try to confirm this).
The problem now is: A fresh scoop brave install wont have any entries for brave in the registry (which generally IMO is kinda a good behaviour). So the installers hands are tied for this. But the installer could try to already create a User Choice, so when someone tries to "Open with ..." something, brave is already there in the list.
The problem goes also a bit further. Once anyone has happened to have such a broken profile in AppData, a complete uninstall and install of brave would copy the broken profile to scoop persist (can someone check if it is overwriting it?). I was happy that I made from both locations a backup before, since at that time I was not sure which one was the one I want to keep.

Hello, I meet this issue as well. Refer to the first of OP's possible solutions:

New-Item -ItemType Junction -Path "$env:AppData/../Local/BraveSoftware/Brave-Browser/User Data" -Target "$persist_dir/brave/User Data"

They don't need admin privilege(sudo)😂,  junction link also used in Scoop via: https://github.com/ScoopInstaller/Scoop/blob/8aee6f99803d0f0aeab252e1d9d4848777185c51/lib/install.ps1#L1254-L1261C2.
Compared to modify regedit or something else, I would like to use junction link to deal with that.

What is scoops strategy in general, does it want to allow an application be installed via scoop and on classic system way (without interference between those)?
If No, i'd support the junction. If Yes, we need to be careful if we really want to do a junction since both would use the same profile.


Remove the client.ByInspecting() callback in responder

Fixes https://github.com/Azure/azure-sdk-for-go/issues/9221

🤖 AutoRest automatic publish job 🤖
success (version: 2.1.146)


Problem with ssl_tunnel when deploying BOSH to  OpenStack using bosh-init

I have an OpenStack cloud installed on VirtualBox virtual machines. I tried to deploy BOSH to  the OpenStack using bosh-init according to the manual.
In the resource_pools section of the deployment manifest, I set the instance_type to m1.large, shown as below,
resource_pools:
- name: vms
  network: private
  stemcell:
    url: https://bosh.io/d/stemcells/bosh-openstack-kvm-ubuntu-trusty-go_agent?v=3197
    sha1: 6bbba614066d77b1fc6d758d33c7e3cc5d2b682a
  cloud_properties:
    instance_type: m1.large

Every thing is OK until bosh-init waiting for the agent on VM to be ready, bosh-init failed with the error message shown as below,
Command 'deploy' failed:
Deploying:
    Creating instance 'bosh/0':
      Waiting until instance is ready:
        Starting SSH tunnel:
          Failed to connect to remote server:
            dial tcp <IP_ADDRESS>:22:getsocketopt: connection refused

But after several minutes the VM is ready, I could ssh into the VM.
I thought it may be caused by timeout of the waiting at first, because I tried several times and found when this problem happened the VM was in different initializing stage.
According to this assumption, I found this post https://github.com/cloudfoundry/bosh-init/issues/67. Fellow up cppforlife's comments, I added  the custom options below to  the deployment manifest,
cloud_provider:
  ssh_tunnel:
    connection_refused_timeout: x
    auth_failure_timeout: x

I don't know whether the x is the letter x or it should be replaced by a number, so I tried both the letter x and the number 36000.
The problem still came after these modifications.
But in these attempts I found the problem show immediately during the VM restarts OpenSSH server, shown as below,
...
Creating SSH2 RSA key; this may take some time ...
Creating SSH2 DSA key; this may take some time ...
Creating SSH2 ECDSA key; this may take some time ...
Creating SSH2 ED25519 key; this may take some time ...
* Stopping OpenSSH server                                            [ OK ]
* Starting OpenSSH server                                            [ OK ]
...

That is to say, when the VM Stopping OpenSSH server, the bosh-init return the error message immediately , even the VM Starting the OpenSSH server again.
I also tried serveral versions of the ubuntu stemcells, same problem still existed. (Tried CentOS stemcells with another error).
After attempts with  different parameter combinations, I found a workaround,  changing the instance_type from m1.large to m1.small can make the deploying process continue sometimes. But the deploying time is extremely long with about 11 hours', and some jobs may failed in running as blow,
...
Compiling package 'director/5a8e3bd5b495695932226d0e5da57e7ef854b7de'... Finished (01:11:31)
Updating instance 'bosh/0'... Finished (00:07:15)
Waiting for instance 'bosh/0' to be running... Failed (00:19:25)
Failed deploying (11:29:35)

Stopping registry... Finished (00:00:00)
Cleaning up rendered CPI jobs... Finished (00:00:00)

Command 'deploy' failed:
Deploying:
  Received non-running job state: 'failing'

Set instance_type to m1.medium was not working neither. It seems that when there are limited VCPUs and memory, sometimes the message of stopping OpenSSH server can be missed by bosh-init.
My question is,  whether I missed some options of the ssh_tunnel or is it a bug of the sshtunnel in order to use the instance with  Flavor be ml.large? Or can bosh-init continue to deploy bosh on a existed VM?

I am sort of running into a similar problem. My bosh-init gets stuck at exactly same point "Waiting for the agent on VM xxx to be ready...".  Enabling bosh-init logging I see this in the log
[sshTunnel] 2016/03/03 21:11:12 DEBUG - Dialing remote server at <IP_ADDRESS>:22
[sshTunnel] 2016/03/03 21:11:12 DEBUG - Making attempt #0
[sshTunnel] 2016/03/03 21:26:59 DEBUG - Attempt failed #0: Dialing remote server: ssh: handshake failed: EOF
[sshTunnel] 2016/03/03 21:27:00 DEBUG - Making attempt #1
...
I could verify that SSH port 22 is open and accessible. Not sure whats the reason.
You mentioned you were able to ssh into the VM. Did you use ssh-key or username/password to ssh into the VM. What key or username/password did you use?

@ajay-aggarwal I used the ssh-key which was created in the "Key Pairs" of the OpenStack project and specified for the instance by the "default_key_name" field and "private_key“ field  in the deployment manifest. The ssh-key is same as the key used by bosh-init.
In my case, the SSH port 22 of the instance is opened, then closed and opened again during its initializing. It seems like the bosh-init aborted when the SSH  port 22 closed. I can ssh into the VM after the port opened again.

@yagweb looks like your env running virtualization inside virtualization and that's resulting in a extremely slow execution times. not much we can do about that except suggesting to switch to an env that is faster. i doubt you'll be able to deploy anything to your existing env if deploying bosh itself is that slow.
@ajay-aggarwal i would recommend opening a different issue https://github.com/cloudfoundry-incubator/bosh-openstack-cpi-release. seems to be just misconfiguration in your openstack env.

@cppforlife I also got the similar situation just like @yagweb.

The Installation logs for Ops Manager Director are as shown:

===== 2016-12-22 13:36:38 UTC Running "bosh-init deploy /var/tempest/workspaces/default/deployments/bosh.yml"
Deployment manifest: '/var/tempest/workspaces/default/deployments/bosh.yml'
Deployment state: '/var/tempest/workspaces/default/deployments/bosh-state.json'

Started validating
  Validating release 'bosh'... Finished (00:01:23)
  Validating release 'bosh-openstack-cpi'... Finished (00:00:04)
  Validating release 'uaa'... Finished (00:00:49)
  Validating cpi release... Finished (00:00:00)
  Validating deployment manifest... Finished (00:00:00)
  Validating stemcell... Finished (00:00:50)
Finished validating (00:03:08)

Started installing CPI
  Compiling package 'ruby_openstack_cpi/9485b5753d4609e92e1491ff991cb28fbde81445'... Finished (01:36:57)
  Compiling package 'bosh_openstack_cpi/dd0bab98dbb820af3ec59b364badfed02ffe3f3b'... Finished (00:00:41)
  Installing packages... Finished (00:00:07)
  Rendering job templates... Finished (00:00:39)
  Installing job 'openstack_cpi'... Finished (00:00:00)
Finished installing CPI (01:38:24)

Starting registry... Finished (00:00:00)
Uploading stemcell 'bosh-openstack-kvm-ubuntu-trusty-go_agent-raw/3312.9'... Finished (00:05:19)

Started deploying
  Creating VM for instance 'bosh/0' from stemcell '7b886ad0-e67c-48e9-8f26-52762210acd9'... Finished (00:01:17)
  Waiting for the agent on VM '3275a1eb-9d32-44e2-936a-fd74652919a4' to be ready... Failed (00:00:00)
Failed deploying (00:01:17)

Stopping registry... Finished (00:00:00)
Cleaning up rendered CPI jobs... Finished (00:00:00)

Command 'deploy' failed:
  Deploying:
    Creating instance 'bosh/0':
      Waiting until instance is ready:
        Starting SSH tunnel:
          Parsing private key file '/tmp/bosh_ec2_private_key.pem':
            asn1: structure error: superfluous leading zeros in length
===== 2016-12-22 15:25:09 UTC Finished "bosh-init deploy /var/tempest/workspaces/default/deployments/bosh.yml"; Duration: 6510s; Exit Status: 1
Exited with 1.

Able to ssh into VM instance with the command ssh -i pcf.pem vcap@VM_IP.

I observe VM instance is not assigned with floating-ip.

FYI, I use Openstack Mitaka. When I run cf-openstack-validator, the validator fails to assign floating-ip to VM. But, I am able to assign floating-ip manually which means that the openstack environment is good enough to deploy the cloudfoundry.
@yagweb If you find any work around to this situation, can you help me with this? Thanks.

@yagweb The solution is described here. I have solved this problem by importing manually created ssh key to OpenStack.


API delegering - font size issues

Go to API delegering pages where you select APIs to be delegated
Expand an API
Here font size of Scopes and Beskrivelse looks a bit bigger than the API name, could be because of the style?
Should this be reduced?

@rtafj @jonkjetiloye

Refinement: new component under construction so won't spend time on this now


Maya: Create Render sets resolution and breaks pixel aspect ratio

Describe the bug
When doing OpenPype > Create... > Render then OpenPype resets the render settings for the current renderer.
If the current render resolution in render resolution mismatches the one in the project manager / database the resolution is reset in render settings but this breaks the Pixel Aspect ratio.
To Reproduce
Steps to reproduce the behavior:

Open an asset that has resolution set to 1920x1080 and pixel aspect to 1.0 (in project manager)
In Maya workfile set your render resolution to 50x50
OpenPype > Create... > Render
Now your render settings pixel aspect ratio is incorrectly set to 0.563

Expected behavior

Create Render should set the settings correctly.
And I'd prefer a studio wide setting to disable Create Render to actually mess with render settings in the scene altogether.

Screenshots

Desktop (please complete the following information):

OS: Windows
Host: Maya 2022
Renderer: Redshift, Arnold, V-Ray (reproducable with all of these)

[cuID:OP-3453]

This needs to be fixed. And

And I'd prefer a studio wide setting to disable Create Render to actually mess with render settings in the scene altogether.

shoud be addressed here https://github.com/pypeclub/OpenPype/pull/3097/files#r906234521

Haven't seen this in AYON - will close it for now.


[DataGrid] Errors occur when using clr-dg-action-overflow and a filter that results in 0 items to display

[X] bug
[ ] feature request
[ ] enhancement

Expected behavior
I should be able to use the clr-dg-action-overflow element and filter my datagrid to something that may not have any matches without it breaking the page.
Actual behavior
Right now if I use a clr-dg-action-overflow element and filter a column with a term that results in 0 matches, the whole page blows up. In my app I am getting a Maximum call stack size exceeded error. In plunker it's a ViewDestroyedError: Attempt to use a destroyed view: detectChanges error. I'm not sure why I'm not seeing the same error in both, but either one fully breaks the page and I have a strong suspicion they are related.
Reproduction of behavior
https://plnkr.co/edit/dnFEBpcGTMFNpEGRxtgQ?p=preview
Environment details

Angular version: 4.0.X

Clarity version: 0.9.14

@youdz
Can you verify if this is a bug or not, especially post EHCAIWC clean up?

I confirm it's a bug on our side, it might be related to #1183 but we need proper investigation to confirm. Thanks for filing this. 😄

@mathisscott @youdz
I have also reproduced this with no filter (just push Delete):
https://plnkr.co/QxqCu2jhqevqO5WG7sxj?p=preview
In my app I get the same error as the one in plunker (ViewDestroyedError: Attempt to use a destroyed view: detectChanges).

This bug can be reproduced on Clarity's document example page for expandable-rows as well, try filter on Creation date column, the error will occur in console if searching hits empty result.

Is there any workaround for this we can try in the mean time?

@youdz
Is this an EHCAIWC issue? Or is this a new issue with the datagrid?

This issue came from the 0.9.14 release. One of the chocolate commits caused this. We had to revert back to 0.9.13 otherwise none of out DataGrids worked.
We encountered 2 scenarios that caused this error to be thrown:
1.) Reducing our dataset which led to less rows in the DataGrid. We use action overflows and checkboxes in our DataGrids and the DataGrid component was trying to access checkboxes and menu items for the rows that were removed.
2.) Server side pagination. We have a tablePageSize of 10 and an apiPageSize of 35. We load 35 rows into the DataGrid and page through them 10 at a time. When reaching the fourth page 5 rows will be visible and we make a call to get the next 35, when those are retrieved we add the next 35 rows to the DataGrid and this was causing the DataGrid to error out with the 0.9.14 release.
Hope this helps.

I tried with all 0.9.x releases and this bug is present in all of them. I am forced to use 0.8.x for the moment.

If possible, let's try to get this fix backported to 0.8 then after the fix is available.
@gnomeontherun

@mathisscott you mean backported to 0.9.x?

I don't know if we would be able to backport this fix -- even to 0.9.x.
Please check with @youdz on that once you have a PR ready for 0.10


runtime for godot engine as it support c++ ?

There has been interest here, and I think with Godot 4 it would be more feasible as Rive requires low level renderer access in order to draw vector graphics, however this is not currently on the roadmap. We have some examples in the low level rendering branch of how to set Rive up with OpenGL.


Feat:Dockerize project

Describe the feature
Create a docker file for project which help to you to package your application and its dependencies into a container, making it highly portable. The Docker image can be run on any system that supports Docker, ensuring consistent behavior across different environments. It allow you define the exact steps needed to build your application environment. By versioning your Dockerfile, you can reproduce the exact same environment at any point in the future. This helps avoid "it works on my machine" issues and facilitates collaboration among team members.
Add ScreenShots
Docker file for project
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'23 contributor
[X] I want to work on this issue

Go Ahead @shraddha761


download_strategy: permit shorter revisions

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

When specifying formulae urls in the revision/tag format, you normally need to specify the full SHA1 hash of the commit. This pull request relaxes this requirement to only require a prefix of the revision.
I'd be open to writing an audit as well to ensure that the prefix of the revision hash is of sufficient length (e.g, 8-10 chars?)

Sorry, I'm afraid I disagree with this. The full revision check is a matter of security and this significantly reduces the space required to find a possible collision.

Fair point. I'll close this one.


Feature Request - allow nhost-js SDK to use Subscription

Apollo is a bit too advanced for me, I'm fine with nhost-js, but it lacks ability to use Subscription

Once we migrate our graphql client to graphql-request (https://github.com/nhost/nhost/issues/1177) we should fix this issue too, because graphql-request has support for subscriptions (https://github.com/prisma-labs/graphql-request/pull/330).

We've decided not to take on this for now. There are a lot of other good graphql libraries that handle subscriptions mentioned here: https://docs.nhost.io/graphql#graphql-clients-for-javascript


cs, validator: use stratumn/go-crypto

This change is

Reviewed 21 of 21 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

jsonhttp/jsonhttp.go, line 88 at r1 (raw file):

		Addr:    config.Address,
		Handler: router,
		// Handler:        &ochttp.Handler{Handler: router},

are you sure about that?

Comments from Reviewable

Reviewed 20 of 21 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

jsonhttp/jsonhttp.go, line 88 at r1 (raw file):
Previously, such (Adrien Montfort) wrote…
are you sure about that?

Please don't commit that, it will be fixed tomorrow ;)

Comments from Reviewable

Reviewed 1 of 21 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

Comments from Reviewable

jsonhttp/jsonhttp.go, line 88 at r1 (raw file):
Previously, t-bast (Bastien Teinturier) wrote…
Please don't commit that, it will be fixed tomorrow ;)

I didn't mean to commit that :)

Comments from Reviewable

Reviewed 4 of 4 files at r2.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable


[runtime] Implement TFSelect and TFStringToNumber at runtime

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.mijiazhi seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Additional property ui is not allowed

First off all great work (=
I'm currently trying to debug the connection from cmake-operator to zookeeper.
Version: 1.4.2
First my values config looked like this:
  21     cmak:
  22       clusters:
  23         - name: "stage"
  24           kafkaVersion: "2.7.0"
  25           curatorConfig:
  26             zkConnect: "stage-zookeeper-client:2181"

For debugging I tried to add some extraArgs to the ui like:
  21     cmak:
  22       clusters:
  23         - name: "stage"
  24           kafkaVersion: "2.7.0"
  25           curatorConfig:
  26             zkConnect: "stage-zookeeper-client:2181"
  27       ui:
  28         extraArgs:
  29           - "-Djavax.net.debug=ssl,handshake"

As I try to update the helm chart I get following message:
Helm upgrade failed: values don't meet the specifications of the schema(s) in the following chart(s):\ncmak-operator:\n- cmak: Additional property ui is not allowed

jfyi, CMAK doesn't support Kafka greater than 2.4.x
your values file has incorrect format, plz double check

https://github.com/eshepelyuk/cmak-operator/blob/master/values.yaml
https://artifacthub.io/packages/helm/cmak-operator/cmak-operator?modal=values-schema&path=ui

Actually HERE pplz are telling that 2.6 is supported, even if it's unavailable in UI.

Closing, since caused by incorrect configuration.

Thanks a lot. Probably starred to long at the same values again.
Oh thank you for that info with the version.

@ohdearaugustin urw !

@ohdearaugustin I'd recommend to take a look at https://akhq.io/
maybe it could be more suitable for your needs

@eshepelyuk  That looks great,. Thank you for the hint (=


Rest diypunks inscription data and diypunk-v2 data

We care about the rest of the diypunks list.
Some guys mint diypunks after 721 , because he found some broken inscriptions.
We want to know range of diypunks collection
I think its fcfs for diypunks holder.

I think its fcfs [first-come / first-serve] for diypunks holder [minter].

yes, the protocol rule is "first-come / first-serve"  BUT the mint inscription MUST be valid to count - to be valid the mint text inscription 1) MUST be valid json  - try here -> https://jsonlint.com - and 2) the g specs / numbers MUST be all in range e.g. 0-59 (60) and 3) no more than one mint in the inscription and 4) the protocol keys must match e.g. p / op / slug . That's it.     Thank you for you patience and undertanding in getting the validation automated so everyone can run / validate (any ORC-721 collection.

Awesome!

@NiceKey8280   now that thinks are quiet in ordgen / orc-721.   the diy punks mint list is here -> https://github.com/ordbase/generative-orc-721/blob/master/diypunks/mint.csv and diy punks v2 mint list is here -> https://github.com/ordbase/generative-orc-721/blob/master/diypunks-v2/mint.csv
does this answer your questions from back in june?


Make sure you run composer install first, before running this scanner

Hello,
Your tool looks great and is just what I need. I am just having an issue running mixed-content-scan. I am using Ubuntu 15.04. I have installed mixed-content-scan using composer
Despite installing using composer I am getting the following error.
I get the error when I run:
php /home/astembjx/Downloads/mixed-content-scan-2.1.1/bin/mixed-content-scan https://www.tdbank.com
I have attempted to run mixed-content-scan  from several different locations
My composer.phar  is installed in /home/astembjx  with vendor dir a sub under that  /home/astembjx/vendor/composer.  I have gone as far as copying the mixed-content-scan dirs into the vendor dir.
I have attempted set the paths several times (as seen below) sometimes leading to PHP commends not being recognized at which point I close and reopen the terminal and I am fine.
export PATH=$PATH:/home/astembjx/Downloads/mixed-content-scan-2.1.1/bin
export PATH=/home/astembjx/Downloads/mixed-content-scan-2.1.1/bin:${PATH}
export PATH= $PATH ~/.composer/vendor/bin
I was hoping you could provide some assistance.
Composer shows the following
astembjx@ubuntu:~$ php composer.phar show | grep mixed-content-scan
bramus/mixed-content-scan
spatie/mixed-content-scanner
Your assistance is much appreciated

Thanks for the response

Hi @astembjx,
You say you have an error when running php /home/astembjx/Downloads/mixed-content-scan-2.1.1/bin/mixed-content-scan https://www.tdbank.com. Could you paste an error/exception log?
Regards,
Bramus.

Think I know what you were doing: you installed mixed-content-scan locally, not globally. It is recommended to install this one globally, using composer global require ... instead of composer require .... Instructions can be found in the readme.


Loading a custom doom theme seems to be switching bg and a base

This seems to be close to a duplicate of the final misbehavior in #2732; aka, my theme is loading, but my background color is only on the current line, and the rest of the background is base...4 (I think? I can find out if needed).
Repro steps:

made a custom theme based off of a random one in emacs-doom-themes
updated the theme name in all the places
mapped the required faces
overrodeline-numbers
saved the theme in .doom.d/themes
added (setq doom-theme 'my-theme) in my .doom.d/config.el

This resulted in the theme not being loaded and what I think is the default emacs dark theme to load? So I added a provide-theme at the bottom of my theme file. Things looked good on load, the doom home screen seemed to have the right theme, background color and all; same with a dired buffer. But when I opened a file I found a buffer with the primary background color being one of the bases. This happens for all text files I've tried to open so far, and is consistent after restarting emacs.
What did you expect to happen?
The color I define for bg, black, in my custom doom-theme to be the primary background when editing a file.
Additional details:
Here's a dired buffer:

And part of my theme file:

I just started playing with doom today coming from vanilla emacs and am loving it so far. Many thanks for all the work done by contributors to this project and thanks for any help in advance!
System information:

SYSTEM  type       darwin
	config     x86_64-apple-darwin19.5.0
	shell      /usr/local/bin/bash
	uname      Darwin 19.6.0 Darwin Kernel Version 19.6.0: Mon Aug 31 22:12:52 PDT 2020; root:xnu-6153.141.2~1/RELEASE_X86_64 x86_64
	path       (/usr/local/Cellar/pyenv-virtualenv/1.1.5/shims ~/.pyenv/shims ~/.pyenv/bin ~/.emacs.d/bin /usr/local/Cellar/pyenv-virtualenv/1.1.5/shims ~/.pyenv/shims ~/.pyenv/bin /usr/local/bin /usr/bin /bin /usr/sbin /sbin<EMAIL_ADDRESS>EMACS   dir        ~/.emacs.d/
	version    27.1
	build      Nov 14, 2020
	buildopts  --disable-dependency-tracking --disable-silent-rules --enable-locallisppath=/usr/local/share/emacs/site-lisp<EMAIL_ADDRESS><EMAIL_ADDRESS>--with-xml2 --with-gnutls --without-dbus --with-imagemagick --with-modules --with-rsvg --with-ns --disable-ns-self-contained
	features   RSVG IMAGEMAGICK GLIB NOTIFY KQUEUE ACL GNUTLS LIBXML2 ZLIB TOOLKIT_SCROLL_BARS NS MODULES THREADS JSON PDUMPER LCMS2 GMP
	traits     (batch server-running envvar-file)
DOOM    dir        ~/.doom.d/
	version    2.0.9
	build      HEAD -> develop, origin/develop, origin/HEAD d88e0795b 2020-12-02 17:58:10 -0500
	elc-files  0
	modules    (:completion company ivy :ui doom doom-dashboard doom-quit hl-todo ophints (popup +defaults) vc-gutter vi-tilde-fringe workspaces :editor (evil +everywhere) file-templates fold snippets :emacs dired electric undo vc :term vterm :checkers syntax spell :tools (eval +overlay) lookup magit :os macos :lang emacs-lisp markdown org python sh yaml :config (default +bindings +smartparens))
	packages   (n/a)
	unpin      (n/a)
	elpa       (swiper rainbow-mode ivy hydra counsel company)

This is not a bug. This is solaire-mode's work. Disable it, or disable the :ui doom module if you're not using a Doom theme.

Ah. That make sense. Thanks again!


Document BOOST_NO_EXCEPTIONS

The overview should explain the use of this macro for Boost and for standalone

There's no "use" of this macro under the Boost configuration - it's automatically set by Boost.Config and automatically respected by boost::throw_exception.
On standalone, the standard feature macro is __cpp_exceptions, I'm not sure how reliable it is. There are compiler-specific macros (from memory, __EXCEPTIONS on GCC, _CPPUNWIND on MSVC.)


cannot load test200 v2.0 in python

Hi, there.
I have followed the instruction. Use the following command to get data from the dataset, but failed.
python test.py pages train.pages.cbor
I get the following errors:
Traceback (most recent call last):
File "test.py", line 29, in
args.func(args)
File "test.py", line 7, in dump_pages
for p in iter_annotations(args.file):
File "E:\PythonProject\TERC-CAR\trec_car\read_data.py", line 232, in iter_annotations
yield Page.from_cbor(cbor.load(file))
File "E:\PythonProject\TERC-CAR\trec_car\read_data.py", line 44, in from_cbor
assert cbor[0] == 0 # tag
AssertionError
Then I try to print out the binary data that I have read from the file, and I got this:
['CAR', [0], [0, [[0, 'enwiki', 'en-US', '20161220', []]], 'trec-car v2.0', [], [[0, 'trec-car-import', '0b10a0342662c908f3198d79100796472918d30a', []]]]]
Is there anything wrong with my operation? Could anyone help me? thank you.

Hi Hongli

It looks like you are using an old version of the code.

Through which way did you obtain the code?  (anaconda, pypi,
  github, ..?)  Maybe I still have an old version out there...

In doubt, use this: https://github.com/TREMA-UNH/trec-car-tools
Best,
  Laura

On 6/12/19 4:44 AM, Hongli Chen wrote:

  Hi, there.
    I have followed the instruction. Use the following command to
    get data from the dataset, but failed.
    python test.py pages train.pages.cbor
  I get the following errors:
  Traceback (most recent call last):
      File "test.py", line 29, in
      args.func(args)
      File "test.py", line 7, in dump_pages
      for p in iter_annotations(args.file):
      File "E:\PythonProject\TERC-CAR\trec_car\read_data.py", line
      232, in iter_annotations
      yield Page.from_cbor(cbor.load(file))
      File "E:\PythonProject\TERC-CAR\trec_car\read_data.py", line
      44, in from_cbor
      assert cbor[0] == 0 # tag
      AssertionError
  Then I try to print out the binary data that I have read from
    the file, and I got this:
  ['CAR', [0], [0, [[0, 'enwiki', 'en-US', '20161220',
      []]], 'trec-car v2.0', [], [[0, 'trec-car-import',
      '0b10a0342662c908f3198d79100796472918d30a', []]]]]
  Is there anything wrong with my operation? Could anyone help
    me? thank you.
  —
    You are receiving this because you are subscribed to this
    thread.
    Reply to this email directly, view it on GitHub, or mute the thread.
  [

{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/TREMA-UNH/trec-car-tools/issues/29?email_source=notifications\u0026email_token=AAGLMNFUQVBR3I5H3GPZP3DP2CZPFA5CNFSM4HXGVK72YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GZABRAA",
"url": "https://github.com/TREMA-UNH/trec-car-tools/issues/29?email_source=notifications\u0026email_token=AAGLMNFUQVBR3I5H3GPZP3DP2CZPFA5CNFSM4HXGVK72YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GZABRAA",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]

Thanks, Laura !
By following your suggestion, I finally read the data.
If you could, please modify the Support Tools Link under the "Data Format Description" section on http://trec-car.cs.unh.edu/.
The URL on the last line points to the v1.5 instead of the latest version.


Extraneous input error as some integer values are treated as octal values

Steps

Run the following code

function main (string[] args) {
    int bill = 008;
    println("Bill num " +bill);
}

Issue
This gives a compilation error : extraneous input '8'.
This error does not throw for values between 001 to 007. But will be thrown for values like, 009, 018,<PHONE_NUMBER>

This is by design. Where, a number which start with 0 is considered an octal value.


Add level_width and level_format params to RichHandler

Type of changes

[ ] Bug fix
[x] New feature
[ ] Documentation / docstrings
[ ] Tests
[ ] Other

Checklist

[x] I've run the latest black with default args on new code.
[x] I've updated CHANGELOG.md and CONTRIBUTORS.md where appropriate.
[x] I've added tests for new code.
[x] I accept that @willmcgugan may be pedantic in the code review.

Description
Add level_width parameter to rich.logging.RichHandler.
Without this parameter it is not possible to reduce level column size properly.
Indeed, if we just override handler._log_render.level_width, output will be
# level_width = 1
           … GET /favicon.ico 404 242                  logging.py:249
[10:14:56] … JSONRPC request                           logging.py:252

# level_width = 4
           WAR… GET /favicon.ico 404 242               logging.py:249
[10:14:56] DEB… JSONRPC request                        logging.py:252

With this new setting, we correctly get level name with valid color rendering and without …:
# level_width = 1
           W GET /favicon.ico 404 242                  logging.py:249
[10:16:27] D JSONRPC request                           logging.py:252

# level_width = 4
           WARN GET /favicon.ico 404 242               logging.py:249
[10:16:27] DEBU JSONRPC request                        logging.py:252

Also adding level_format parameter to allow changing alignment of level column.

Sorry, the logging handler is too complex already. I can't accept new parameters. If you need this for a project I guess you wouldn't have much difficulty extending it in your own code. THanks anyway.

I have follow your advice and make my own LogHandler ...

To give more details on this PR:
I have at first see that parameter  level_width is defined in LogRender but is not forwarded by RichHandler.
This is why I have at first suggest adding it to RichHandler.
Indeed, log handler is a little bit complicated. You can still simplify it by removing level_width unused parameter from LogRender.
This will avoid someone else to ask again for this feature.


for*, loop*, while* generator expressions

@vendethiel's WIP branch aiming to fix #505

The arguments passing thing is missing, the rest should be good. Please take another look :).

Replacing with #1398 (or if @vendethiel wants to make their own PR, I could probably push to the branch)


Remove mount /var/run from host to avoid DNS lookup failure

Proposed Changes
The host's /var/run/nscd leaking into the pod and it caused the DNS lookup failure.
Linked Issues
https://github.com/longhorn/longhorn/issues/2200

Hi @jenting ,
This repo will only be updated once we have a new release. See https://github.com/longhorn/longhorn/wiki/Repositories-for-contribution-guideline for details.
I will close the PR for now.

Hi @jenting ,
This repo will only be updated once we have a new release. See https://github.com/longhorn/longhorn/wiki/Repositories-for-contribution-guideline for details.
I will close the PR for now.


How to detect device platform in Flutter web?

How are you?
I am making a website with Flutter.
When a user opens the website with phone, I want to detect his mobile device platform(android or ios).
If you have a good idea, please share.
Thank you.

import "dart:io" show Platform;
bool isWeb;
    try {
      if (Platform.isIOS || Platform.isAndroid) {
        isWeb = false;
      }
    } catch (e) {
      print("We're web");
      isWeb = true;
    }

@liqiang1995
Please see https://flutter.dev/community for resources, you may also get some help if you post it on Stack Overflow and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you


EventualsGrpcTest.ServerDeathTest fails on Ubuntu

We need to figure out why this test fails on Ubuntu.
You can check it using the link below:
https://github.com/3rdparty/eventuals-grpc/runs/4382126214?check_suite_focus=true

@benh is this fixed by https://github.com/3rdparty/eventuals-grpc/pull/77 ?

Yes thanks! Linked!