ClientSideAssets

Another more recent article seems to suggest that "clientsideassets" is also a default public origin.
It is not mentioned in this article.  Reference -  https://docs.microsoft.com/en-us/sharepoint/dev/spfx/web-parts/get-started/hosting-webpart-from-office-365-cdn

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

ID: 2836d55d-6874-f366-60c2-e3043dd5547e
Version Independent ID: 3c84c811-52bc-872e-48cf-54ecc2a9d7de
Content: Use the Office 365 content delivery network with SharePoint Online
Content Source: Enterprise/use-office-365-cdn-with-spo.md
Service: o365-administration
GitHub Login: @KCCross
Microsoft Alias: krowley

@Lady-MCSE Thank you for this feedback
I agree we need to clarify "clientsideassets". I'm going to get this over to the Enterprise team for investigation and clarification.
The team will get this prioritized into a sprint plan. Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.


Save zoom size and position for each map

It would be nice that the zoom level and position will be automatically saved for each map.
Yesterday we jumped between some maps in a short period of time, and we always had to adjust the zooming an position when we did load a map which was loaded and set up before. (we use a tv on a table)

Related to #207

well it's not really a relation i think. At least not in the usecase. In the codebase, yes. I agree.
But for this usecase i do not want to control the player map from the dm screen nor sync the positioning of map in this case (this is another issue).
i  want the client to store it's last zoom level and position for a map on the server with a "unique machine id" (node-machine-id for example). When the client requests the map again, it will also get the last positioning info for it's machine id.
So each client in fact has its own positioning info for the map on the server. If its done like this, the Issue #207 may use this for syncing the client view to the dm view.
An easier method for this usecase would be to implement it only on the client side with some cookies or something like that.
So that the browser stores the mapId + zooming and position.


Deck Manager (editor) - keyboard shortcuts for adding a card to the deck

Motivation:
Often building arena decks, and my workflow is currently:

press ⌘+F
type first few chars of the card until there are only few cards (ideally just one) on the screen so that it is easily spotted in the few
click on the card
goto step 1.

The switch between keyboard for filtering cards and mouse for selecting one of the few cards left after filtering is annoying me :D

**Solution:**
Implemented keyboard shortcuts ⌘-[1 to 9], where the number is the position of the first 9 cards in the grid view (counting from left to right including wrapping around). This allows for the following flow:
 1. press ⌘+F
 2. type first few chars of the card until there are only few cards (ideally just one) on the screen so that it is easily spotted in the few
 3. seeing the card I desire is now the 3rd card listed, press ⌘+3
 4. goto step 1.
Sometimes even the first few chars of a card's name are unique enough to only show one card, in which case this is still a time saver as the ⌘+1 adds that one (understandably) :)
Implementation relies on "characters pressed" rather than "keycodes pressed", which should cover both numbers on the numpad and above the numbers above the letters on the keyboard.
Needs testing on a numpad, I don't have a full-sized keyboard around, tenkeyless for life! :D

**Other comments**:
Diff in github looks a bit shite since I moved around one whole block of original code to make logic for introducing my set of commands simpler. Fear not!
Also beware, this is my first time seeing/coding Swift :)

Hi @martin-podlubny looks good.
Could you refactor to add a initKeyboardShortcut and move all the code for addLocalMonitorForEventsMatchingMask in it ?

windowDidLoad() now calling initKeyboardShortcuts() 👍

Awesome, thanks @martin-podlubny


Fixed the menu image profile and notification, private list UI issue

Please Verified and merged with the master.

Verified and merged with the master.


Incorrect value for X-Forwarded-Proto

Environment
Irrelevant but Nitro 2.8.1 on Node 20.5
Reproduction
Issue is visible in:
https://github.com/unjs/nitro/blob/06323b736f0d8adaa609d5a0348fa6b9e0db75e7/src/dev/server.ts#L203-L205
Describe the bug
The X-Forwarded-Proto header should take a value in [http, https], and instead takes a value in [IPv4, IPv6]
With the current code, accessing a Nitro server with plaintext results in X-Forwarded-Proto: IPv4 (for example)
This is not correct, and should instead give X-Forwarded-Proto: http
Additional context
Since it is not a standardised header, the best source of documentation is your pick between:
a. The Mozilla docs about the de-facto way it is meant to work: https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/X-Forwarded-Proto#syntax
b. The spec for the Forwarded header which is based on it (and other X-Forwarded-* friends): https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Forwarded#proto
c. Nearly every (widespread) webserver in existence (CF: https://developers.cloudflare.com/fundamentals/reference/http-request-headers/#x-forwarded-proto, AWS: https://docs.aws.amazon.com/elasticloadbalancing/latest/application/x-forwarded-headers.html#x-forwarded-proto, HAProxy: https://docs.haproxy.org/2.8/configuration.html#7.3.1-iif, ...)
Logs
No response

This is causing me a problem implementing tus-node-server in nuxt

Seems reasonable IMO. PR Welcome 👍🏻


Move settings out of Context

To help keep Context small, we ought to have fewer things in it, and settings don't change over time. We would need some other lookup mechanism for rules, though.

[ ] Add settings to the generated World struct.
[ ] Reference the settings as world properties in generated code.
[ ] Remove the settings from Context.


Powershell Hashtables no longer work when using SDK

I have a build task written in Powershell that uses the Hashtable feature from PS to pass data between functions. All the data is within the task itself. If I don't pull in the SDK but stick with PS proper then everything works. When I bring I the SDK then at runtime PS reports an error saying "Cannot convert the "System.Object[]" value of type "System.Object[]" to type "System.Collections.Hashtable".
The only thing that differs between the working and non-working versions is the inclusion of the SDK. I don't understand how the SDK is getting in the way.  The relevant pieces of the script.
`function GenerateAttributes
{
$attributes = @{
"Key1" = "value1";
"Key2" = "value2";
}
return $attributes;
}
function UpdateStuff
{
param(
[hashtable] $attributes
)
}
$attributes = GenerateAttributes
UpdateStuff $attributes
`

I am unable to repro. I'm on PSVersion 5.1.14393.693
Here is my task script:
function GenerateAttributes {
    $attributes = @{
        "Key1" = "value1";
        "Key2" = "value2";
    }
    return $attributes;
}

function UpdateStuff {
    param([hashtable] $attributes)
    Write-Host ("attributes.GetType().Name: " + $attributes.GetType().Name)
    Write-Host ("attributes.Keys.Count: " + $attributes.Keys.Count)
    Write-Host ("attributes['Key1']: " + $attributes['Key1'])
    Write-Host ("attributes['Key2']: " + $attributes['Key2'])
}

$attributes = GenerateAttributes
UpdateStuff $attributes

Here is the output:
attributes.GetType().Name: Hashtable
attributes.Keys.Count: 2
attributes['Key1']: value1
attributes['Key2']: value2

Can you clarify whether you are using the "PowerShell3" handler? Also whether you are importing the VstsTaskSdk (should not do that, handler does it before calling your script)?

Yes I'm using Powershell3 handler. I don't have an implicit import for VstsTaskSdk. I replicated this issue on 2 different machines using the same script. In my case I was enumerating the hashtable using a foreach but I believe the call was failing at entry to the function. Michael Taylor - Microsoft MVP, MCAD
http://www.michaeltaylorp3.net
On Sunday, February 5, 2017 9:19 AM, ericsciple<EMAIL_ADDRESS>wrote:

I am unable to repro. I'm on PSVersion 5.1.14393.693Here is my task script:function GenerateAttributes {
$attributes = @{
"Key1" = "value1";
"Key2" = "value2";
}
return $attributes;
}
function UpdateStuff {
param([hashtable] $attributes)
Write-Host ("attributes.GetType().Name: " + $attributes.GetType().Name)
Write-Host ("attributes.Keys.Count: " + $attributes.Keys.Count)
Write-Host ("attributes['Key1']: " + $attributes['Key1'])
Write-Host ("attributes['Key2']: " + $attributes['Key2'])
}
$attributes = GenerateAttributes
UpdateStuff $attributesHere is the output:attributes.GetType().Name: Hashtable
attributes.Keys.Count: 2
attributes['Key1']: value1
attributes['Key2']: value2
Can you clarify whether you are using the "PowerShell3" handler? Also whether you are importing the VstsTaskSdk (should not do that, handler does it before calling your script)?—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

What version of PowerShell?

I replicated this issue on 2 different machines using the same script. In my case I was enumerating the hashtable using a foreach but I believe the call was failing at entry to the function.

I'm confused, are you saying you used the exact script I pasted? Or some other other script involving a foreach? Note, the script I pasted was my entry script that task.json points to. If you can send the smallest possible repro to ersciple would be great (at microsoft com).

..forgot to mention. if you can zip the task folder and send. or might have better luck pasting here rather than fighting outlook attachment filters.

I'm using PS 5.1. I went back to the original extension that I was trying to use hashtables with and switched back to them along with using the newest VSTS SDK. The extension is working now. I'm not sure why it was having issues with the hashtable before. The only thing I changed originally to cause it to fail was start using the VSTS SDK.
I'll assume my problem is resolved until I can replicate the issue again.
Michael Taylor - Microsoft MVP, MCAD
http://www.michaeltaylorp3.net
On Sunday, February 5, 2017 9:55 PM, ericsciple<EMAIL_ADDRESS>wrote:

What version of PowerShell?
I replicated this issue on 2 different machines using the same script. In my case I was enumerating the hashtable using a foreach but I believe the call was failing at entry to the function.
I'm confused, are you saying you used the exact script I pasted? Or some other other script involving a foreach? Note, the script I pasted was my entry script that task.json points to. If you can send the smallest possible repro to ersciple would be great (at microsoft com).—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


How to move a note to trash bin?

Sometimes I need to restore some notes.

There is currently not a trash bin feature.  When notes are deleted, they are deleted permanently.


US first day of the week

Us week starting on Sunday, not Monday
When selecting:
en_US
America/Los_Angeles
Week changing on Monday.

Unfortunately, all processing of dates happens via the moment.js library. So if there are issues, it is with that library I'm afraid. The moment node really needs a full rewrite to use a more currently library but I don't have time to do that right now.


Change the icon and text state of the copy _inline_content in Send coins back.

Currently, the icon and text remain the same even after copying the key.
Want to change the text from "copy" to "copied'  and present  icon with a check icon and then reset to the default text and icon  for few seconds.
for example:
https://user-images.githubusercontent.com/73611313/223705720-5ccaaf04-6c53-46d1-aa79-8a007bdfe1bb.mp4

@thunderbiscuit ,if you found the issue relevant, then please assign it. I would love to contribute.

Yeah, I think this makes sense

Yeah, I think this makes sense

Okay , please assign me this issue.


Deliverables (15%)

User Story
AS A BC Student
I WANT to confirm my code meets the highest level of deliverability
SO THAT I can get the best grade possible
Acceptance Criteria
GIVEN a LOGO generator assignment
WHEN I've completed my code
THEN I've confirmed it meets the following specifications:

At least one sample SVG file generated using the application must be submitted.
Your GitHub repository containing your application code.

Repository: https://github.com/bldambtn/logoAgogo
SVG File Attached


discontinued?

are you still working on the version or is it discontinued?

I will continue in this month, sadly (or not) I had too much task on my work, I had very little free time, right now I'm waiting a cable for my microphone and then I will  continue the stream :)


Guidance for support squad: where to leave docs feedback

This PR adds a section to the Support Squad guide describing how to give docs feedback they collect while helping people in support threads

LGTM


cdn code snippets/samples added

@woodp,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/best-practices/cdn.md
:white_check_mark:Succeeded
View

docs/best-practices/images/cdn/rules-engine.png
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@woodp Could you please rebase and resolve conflicts?

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/best-practices/cdn.md
:white_check_mark:Succeeded
View

docs/best-practices/images/cdn/rules-engine.png
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/best-practices/cdn.md
:white_check_mark:Succeeded
View

docs/best-practices/images/cdn/rules-engine.png
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Some errors Seems about filepath on Windows10.

Seems some errors about filepath on Windows10.
D:\home  λ gh inspire
jq: error: Could not open file C:\Users\scillidan\AppData\Local\GitHub: No such file or directory
jq: error: Could not open file CLI\extensions\gh-inspire\gh-inspire/quotes.min.json: No such file or directory
shuf: invalid input range: ‘’

Only GitHub CLI in AppData/local,
and quotes.min.json is here:
C:\Users\scillidan\AppData\Local\GitHub CLI\extensions\gh-inspire\quotes.min.json

@scillidan thank you for reporting this, unfornately i dont have windows to try it. a PR is welcome.

@lakuapik OK, I'll give it a try. Because I'm inexperienced🤡.

Seems to be because there is a space in the path, between github cli.
I don't know if it works, but I try change the line https://github.com/lakuapik/gh-inspire/blob/b427d37ae20838bfaea921870f5a88db9004e428/gh-inspire#L9
to:
    if ! type -p "$1 >/dev/null"; then

But said:
D:\home  λ where fold
C:\Users\scillidan\scoop\apps\git\current\usr\bin\fold.exe
C:\Users\scillidan\scoop\shims\fold.exe

D:\home  λ gh inspire2

ERROR: fold not found on the system
please install and try again

I get gh.exe from releases. Then I found that it works well on Windows 10.
The shuf, fold, bash.exe be installed with install Git:
scoop install git jq

git clone https://github.com/lakuapik/gh-inspire
cd gh-inspire
bash.exe gh-inspire

Example return:
DRY - Don't Repeat Yourself - Every piece of knowledge must have a single,
unambiguous, athoritative representation within a system.

~ Andy Hunt & Dave Thomas


Sreq

Why do we need sessions? I'd prefer the server to be completely stateless, and if need be, persist any state out of process.

Can this PR be closed?


Cleaning up unused MNM events

This is like #2108 but daggy-fix based on @henri-tremblay 's commit (https://github.com/ehcache/ehcache3/pull/2102/commits) and merged onto master.

replaced by #2132


Pass error/meta to case reducers, support non-FSAs

I would like to suggest that we pass error/meta to case reducers:
function myCaseReducer(state, payload, { error, meta }) {
    // ...
}

It might be useful.
Also I've added support to non-FSA.
Building is now failing, I don't know how to add support for rest spread in Rollup. Alternatively I could change the code to use omit():
const {type, payload} = action;
const rest = omit(action, ["type", "payload"])

Nice, I like that we're using the FSA package for consistency with other tooling.

I would like to suggest that we pass error/meta to case reducers:

Does this change actually cause the FSA package to pass error and meta to the reducer or is this something the user would set up manually in action creators?

Building is now failing, I don't know how to add support for rest spread in Rollup.

Rest spread is supported in Node 8.6.0 (active LTS), so I'd recommend updating Node and not worrying about it. We could tweak babel-preset-env to compile against older versions of Node when we need to, or you can do it if you can't update Node for some reason.

@nickmccurdy the FSA package is just used for checking if the action is a FSA.
I'm passing the error and meta to the case reducer so you can use that info to transform state inside the case reducer:
const addTodoSuccess = todo => ({ type: ADD_TODO, payload: todo })

const addTodoFailure = error => ({ type: ADD_TODO, payload: error, error: true })

// here we don't need to read error or meta, just payload
const addTodoCaseReducer = (state, todo) => state.push({ ...todo, completed : false })

const myErrorManagerCaseReducer = (state, payload, { error }) => error ? state.push(payload) : state

The reason is that when error is true, payload could be anything, new Error() is just a convention, but might not be followed. Also the user might want to read meta.

That makes sense, would you mind documentation that this convention can be optionally used in the readme?

@nickmccurdy I've updated the createReducer section in README. Please review it, you might want to improve the wording.

I like your wording and the code examples, but I find the difference between the first two examples a little confusing. They both use FSAs now so maybe they should be merged since there's not much different between them. We should also stay consistent and use immer-style mutating reducers (unlike the FSA docs) because that's unrelated to how we implement reducers with or without FSAs.

@nickmccurdy I've merged first two examples. I removed the returns, I'm not familiar with immer, so you might want to review.

Eh, I'm inclined to say we just switch back to a standard (state, action) signature rather than (state, payload).  I guess if people want to grab just the payload in their reducer, they can always destructure it like function myReducer(state, {payload})

I'll close this PR, but please go ahead and submit one that changes createReducer to just pass the whole action instead.


ci(test.yml): enable test execution for node v20

Enable node v20 checks
See https://github.com/SAP/ui5-fs/pull/512 for details
JIRA: CPOUI5FOUNDATION-612

LGTM. I can not approve because I am the creator of the PR


feat: add a plugin bitbucket

PreChecklist
firstly you should create a app_password for your account
https://bitbucket.org/account/settings/app-passwords/
bitbucket.go run configuration example
go run bitbucket.go -c 1 -o thenicetgp -r lake
Summary
add bitbucket connection repo pr issue issue_comments pr_comments
add e2e tests: issue_test, pr_test, comment_test, account_test, repo_test, etc.
Does this close any open issues?
#2100
Screenshots
table "repos"

table "accounts"

table "issues"

table "pull_requests"

table "pull_request_comments"

table "issue_comments"

Other information
Thanks to @mindlesscloud  for his guidance on my work.

Actually, you can convert a pr to a draft pr in the right pannel. 😁

Good work. Make sure you add e2e tests as well.

Good work. Make sure you add e2e tests as well.

thanks,I will add

Hi @thenicetgp, could you give an overall status update on this PR? Are there more changes you are working on?

Hi @thenicetgp, could you give an overall status update on this PR? Are there more changes you are working on?

Hi @keon94 , thank for your comment,  this PR just add a plugin bitbucket, it can currently collect issues,pullrequests,accounts,repos,issue_comments and pr_comments. Domainlayer table accounts,repos,issue,pull_requests,issue_comments and pull_request_comments are filled. These tables have basically all the data they should have.
This PR makes no changes to the any other plugin.


Change value of lookup field enters linked old entity

When try to change the value of a lookup the field is clicked a second time to check if the focus is already on field. If the field already has a focus this leads in entering the linked entity and therfore leaving the current page.

Ok, we'll test this scenerio and provide an update.  Which version are you working with?

We use Microsoft Dynamics 365 Version 1612 (<IP_ADDRESS>) on-premises

@WinniB   Can you please provide exact steps or elaborate on the issue? Is this consistent or intermittent behavior? Unable to repro this issue.

This always happens when I try to change the value of a lookup field which is already filled with data.
Example steps could be:
xrmBrowser.Entity.SetValue("my_lookup", "TextA"); //Fill for first time
xrmBrowser.Entity.SetValue("my_lookup", "TextB"); //Fill for second time / change value

@WinniB Please try using this when changing the value of a lookup field. The code you referenced above is used to change a text field. Let me know if this works for you.
xrmBrowser.Entity.SetValue(new Lookup { Name = "primarycontactid", Value = "Rene Valdes (sample)" });

I tried this before, but if the value I want to set does not equal one in the dialog items (but is an valid value for the lookup) I get an invalid operation exception : "List does not have XYZ."


Not search specific letter

Here when search using "PdfTextSearcher.startTextSearch" it will search overall word and highlight word but i want to highlight only single letter

At least currently, Web version could not do character level selection. pdf.js sample actually does it but the pdf.js API does not provide any APIs to do that; it seems it uses SVG or such to do that.

The issue has been discussed on #4 but should we need a separate issue?

No


docs: Fix example as Terraform performs shallow merge of nested maps

Description
Fix the example with a custom CloudWatch log group name because log_configuration.options override during the merge operation as Terraform doesn't perform a deep merge of nested maps.
Fixes https://github.com/terraform-aws-modules/terraform-aws-ecs/issues/173
How Has This Been Tested?

[x] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[x] I have tested and validated these changes using one or more of the provided examples/* projects

[x] I have executed pre-commit run -a on my pull request

This PR is included in version 5.9.2 :tada:


swap tokens - rate toggle

as a user, i would like to have the option to toggle the rate of the assets in display.
when i click the rate area (where it says "1 BNT = 0.0023467 ETH", you should display the inverse rate
example:
set the swap window to convert from ETH->BNT, rate will be: "1 ETH =<PHONE_NUMBER>84 BNT"
click the rate area
update the rate to display the inverse rate (as if i am setting the swap window to BNT->ETH): "1 BNT = 0.002039102 ETH"
https://www.figma.com/file/3yED7o7TgoLwoEjV2EkRLL/Bancor-V2?node-id=1328%3A7

@ashachaf can you check staging. The math seems correct. But there is a difference in swapping BNT->ETH and ETH->BNT - I am not sure if that is expected or not.
Please double check.

Checked with Asaf. Done.


fix: hdinsights model validation

Add "body" for RESPONSE_BODY_NOT_IN_EXAMPLE errors

Automation for azure-sdk-for-python

Automation for azure-sdk-for-go

Can one of the admins verify this patch?


mcrypt is deprecated

i use php 7.1.1 and mcrypt is deprecated in my php version.Is there any way around it in other to enable me access the withdraw method.

As you very well pointed out, mcrypt has been deprecated in php7.1. The only solutions to resolve this issues would be to move down to php7.0. If that is an option for you. I will go ahead and close the issue, since solving it is not within this repository control. I will be adding a note in the readme, so thanks for pointing this issue out. Cheers and the best of luck.


RFC: Auto-load User-provided Environment Variables

Readable.

I think we should note that this is a breaking change (an app that expects a provided JAVA_OPTS not to be loaded by default could break).
I'm lukewarm on the key name, but I understand your concern about flags that default to true. My feeling is that secure-env might not convey the intention well enough, especially because the primary use case isn't about security. Is there a better inverse of load-env? Maybe serialize-env?

@sclevine @nebhale are you doing this for most of the CF (non-CNB) buildpacks today? I wonder if this is expected behavior? Is it better to capture this kind of functionality into libbuildpack?

We already have this functionality in libbuildpack -- this RFC is intended to make it easier for buildpack authors to write simple buildpacks in the language of their choosing.
I've seen quite a few users surprised by the current behavior: you set environment variables, but they don't show up in the environment.

@sclevine thoughts on providing a blacklist as a buildpack author?

I think an blacklist would be a nice supplement to this. but the libbuildpack solution prevents "it just works" behavior for env vars, which stinks for new buildpack authors.
Sidenote: I wonder if we should blacklist PATH by defafult.

@dzuelke suggests clear-env

Overall, I'm 👍 this change now. From the spec WG today, it looks like we're in agreement that we'd want to blacklist PATH, since there isn't a good way to deal with appending/prepending and knowing what the user wants and eval'ng the value they insert. Where do we handle this blacklist? I'm in favor of doing this at the lifecycle level. A concern that was raised was disjoint between what exists in platform/env vs what is autoloaded into the env. Would it be possible to have the default PATH blacklist as part of this RFC?
A separate platform level to blacklist env vars, could be a separate RFC and out of scope of this one imo.

I'm in favor of doing this at the lifecycle level.

Does this mean that builds fail immediately if PATH is provided as a build time env var?
Or does it mean that without clear-env = true, PATH doesn't get set automatically? (Maybe the user can still suggest PATH values to the buildpack this way, as a mechanism to provide additional values?)
Should we blacklist the other lifecycle-managed env vars? (See: https://github.com/buildpack/spec/blob/master/buildpack.md#provided-by-the-lifecycle)

Should we blacklist the other lifecycle-managed env vars? (See: https://github.com/buildpack/spec/blob/master/buildpack.md#provided-by-the-lifecycle)

I was wondering that too.

Or does it mean that without clear-env = true, PATH doesn't get set automatically? (Maybe the user can still suggest PATH values to the buildpack this way, as a mechanism to provide additional values?)

I was leaning towards this. The thing that's unclear is how the buildpack would or should use PATH?

How about: for any of those five env vars, we only set them automatically (assuming no clear-env = true) if they are not already set in the real environment by the platform.
Realistically, this just means blacklisting auto-set PATH, but it could prevent unexpected behavior if the platform sets the others.

We could also have a list of "smart" env vars where we support automatic appending or prepending based on a : syntax without the existing variable being referenced.
Meaning that if you set $PATH to /some/thing, you're out of luck, but if you set it to :/some/thing, or /some/thing:, we append or prepend the existing value (and of course you could also do /foo/bar::/bar/baz; any empty segment essentially references the existing value).
That's how I've seen it done in other systems where an env var is used to append or prepend to a built-in path, where no existing env var is available to reference.
That's obviously then a slightly proprietary syntax, but OTOH, it's not like people will constantly shoot themselves in the foot by setting $PATH to a value on an existing high volume production app without prior testing or anything. Hopefully. :)

But generally, I think we worry about the $PATH case too much. Yes, people will set it. But no, not all the time, and they can trivially use a .profile file if they want to customize it. It's a fairly advanced use case.

@hone Updated the RFC with @dzuelke's suggestion. The lifecycle-provided env vars (like PATH) are now prepended when they are user-provided. They always precede layer-provided values and platform-provided values.

@nebhale @ekcasey


Factory with nested input data crashes with "Array to string conversion"

Laravel Version: 5.8.26
PHP Version: 7.2.19-0ubuntu<IP_ADDRESS>
Database Driver & Version: Postgresql (11.4-1.pgdg90+1 via Docker container)

Description:
Utilizing factories with nested array data in its create input causes issues an Array to string conversion exception.
Steps To Reproduce:
Create a factory that is dependent on another factory and pass the data from the original factory create in a test file through to the final factory.
I created a repo for easy reproduction: https://github.com/emilhem/laravel-factory-bug
The only changed files from a default laravel installation are two migrations, two factories, two models and one test case (list in repo).

The values passed to create() are used as the model's attributes, that's why you're getting a nested array:
[
    "name" => "producty",
    "box_id" => 1,
    "box" => [
        "name" => "boxy",
    ],
]

If you want to create boxes with custom names, you'll need to do that outside of the factory and then pass the id to create().

Hmm, that's quite unfortunate.


Fix links

http://validator.w3.org/checklink?uri=http%3A%2F%2F6to5.org%2F&hide_redirects=on&hide_type=all&depth=10&check=Check

Seems to have been fixed.


material-symbols 无法自动导入

material-symbols 图标无法自动导入

同样遇到了这个问题

We temporarily close this due to the lack of enough information.
Please provide a minimal reproduction to reopen the issue.
Thanks.

Why reproduction is required

I ran into this issue as well. I just had to update my @iconify/json and unplugin-icons versions. material-symbols was missing from whatever version I was on.


Fixed cpp-capture SIGSEGV

Thread 1 "cpp-capture" received signal SIGSEGV, Segmentation fault.
0x00007ffff7b0ff77 in rsimpl::uvc::query_devices (context=std::shared_ptr (count 4, weak 0) 0x65ebf0) at src/uvc-v4l2.cpp:815
815 bool bFishEyeDevice = ((busnum == dev->subdevices[3]->busnum) && (devnum == dev->subdevices[3]->devnum));
(gdb) print dev->subdevices
$1 = std::vector of length 3, capacity 4 = {std::unique_ptrrsimpl::uvc::subdevice containing 0x695e10,
std::unique_ptrrsimpl::uvc::subdevice containing 0x68d3c0, std::unique_ptrrsimpl::uvc::subdevice containing 0x68d470}
Signed-off-by: Hong Yang<EMAIL_ADDRESS>

Thanks, please review contributing.md in the root directory


Order table

A naive mistake selecting the forked MRQ repository

@mvquezm hi! didn't you want that merged into mrq ?

Hi, it was a mistake, changes are not ready yet, when will be ready i will create a pull request again, best regards M

El 24 mar 2017, a las 11:40, Sylvain Zimmer<EMAIL_ADDRESS>escribió:
@mvquezm https://github.com/mvquezm hi! didn't you want that merged into mrq ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/pricingassistant/mrq/pull/160#issuecomment-288988831, or mute the thread https://github.com/notifications/unsubscribe-auth/AVuA5CT6V8iJ2gJwL8LBTlM55CVnUaw7ks5ro52JgaJpZM4MnJX5.

Okay well let me know if you want some feedback anyway! For instance I think it would be way better to use an existing javascript widget (for instance http://www.daterangepicker.com/) than to reimplement it :)


Library suggestion: PHPJasper

This library is awesome for generate PDF reports and many more formats, is unique in what it proposes to do and widely accepted by the PHP community.
https://github.com/PHPJasper/phpjasper

@geekcom It'd be good if you could open a proper PR for this, thanks.


Update discord.place icon

discord.place's logo has changed, so I'm changing the icon here too

@MattIPv4


Could you make a proper library out of this and maybe apply the following suggestions?

It would be nice if this would be a proper library that can be used. But then you need to fix and improve some things.

Password has a spelling mistake on the website that gets hosted.
The SSID and Password should not be hard coded.
For security reasons the password should never be printed to serial.
On boards with a crypto chip, it would be good to use that to store the wifi password. (But I am not sure where to save the crypto key maybe it is a hardcoded key or this key is actually stored on the flash memory, not sure.)
It would be good if it is not necessary to know the IP address to connect to the arduino, maybe it could have a fixed web address and uses it's own DNS resolver? Or when connecting to the wifi, it automatically redirects you to the page. So the created wifi does not use a password.
It would be nice, if the website to connect to your wifi, would actually scan for all wifi's in reach, so you can just select one, instead of typing the SSID by hand, which can be annoying to do, if it is a long one, here is an example on how to do that: Example
Using HTTPS for the login page, to make sure the password is not stolen on the way.

I am new to arduino programming, but I am trying to solve some of the things myself, but due to time constraints, I wish you could help me with some parts. The most important thing would be to get to the webpage without knowing the IP address.

It would be nice if this would be a proper library that can be used. But then you need to fix and improve some things.

Password has a spelling mistake on the website that gets hosted.
The SSID and Password should not be hard coded.
For security reasons the password should never be printed to serial.
On boards with a crypto chip, it would be good to use that to store the wifi password. (But I am not sure where to save the crypto key maybe it is a hardcoded key or this key is actually stored on the flash memory, not sure.)
It would be good if it is not necessary to know the IP address to connect to the arduino, maybe it could have a fixed web address and uses it's own DNS resolver? Or when connecting to the wifi, it automatically redirects you to the page. So the created wifi does not use a password.
It would be nice, if the website to connect to your wifi, would actually scan for all wifi's in reach, so you can just select one, instead of typing the SSID by hand, which can be annoying to do, if it is a long one, here is an example on how to do that: Example
Using HTTPS for the login page, to make sure the password is not stolen on the way.

I am new to arduino programming, but I am trying to solve some of the things myself, but due to time constraints, I wish you could help me with some parts. The most important thing would be to get to the webpage without knowing the IP address.

Hi, those are legit and good suggestions, let me work on it

In the meanwhile, since this project is open source, feel free to make contributions to improve it yourself

I also found an older project, but it works on the WifiNina Chip of the Arduino Nano 33 IoT as well: EasyWifi-for-MKR1010
It does a few things worse and a few things better, maybe we can combine some things because there are still some wifi or network-related things I do not fully understand yet.
The best of the other project is, that it creates an Access Point with a Captive Portal, so it forces you onto the website, when connecting via Android for example. So you do not need a password at first and also do not need to know the IP address, which makes it more user-friendly when built into an inaccessible location. It also shows the scanned networks, but the website refreshes in intervals and does not show the strength of the scanned networks. The refreshing gives you only a small time window for entering the wifi password, which is bad, but with copy-paste, doable.
It also scrambles the password a bit before storing it, so it is not directly readable if hacked.
I will look into that project and will see what I can combine.
I am not so familiar with Github itself, so I am not sure how to create a pull-request yet, I also need to setup my project first with github, I will do that a bit later.

A little bit unrelated, but I have the Arduino only for a couple of days yet, so I meanly do learning by doing currently. I have calibrated a distance sensor and I want it to use it to control the lights of my Phillips Hue Bridge. So for this I got the sensor part working, now I work on the easy wifi setup part and then I still need to figure out how to find and connect to the hue bridge and control a light with it, but luckily I found an example for it on the WifiWebServer repo. I only have a few days left to accomplish this. I should also try to get over-the-air updates working, to allow a firmware update from my PC when I need to improve things, but I am not sure how to realize that yet.


Unable to download from lynda.com site

Amans-MacBook-Pro:WordPress Ecommerce Easy Digital Downloads aman$ youtube-dl -u<EMAIL_ADDRESS>www.lynda.com/WordPress-tutorials/WordPress-Ecommerce-Easy-Digital-Downloads/122465-2.html
Type account password and press [Return]:
[lynda:course] Logging in as<EMAIL_ADDRESS>ERROR: Unable to download webpage: HTTP Error 401: Unauthorized (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Same here. What is wired is I downloaded another course a few hours ago and it was OK. I have no problem viewing this course online, but just not able to download.

Post the full output of youtube-dl when run with -v, i.e. add -v flag to your command line, copy the whole output and post it in the issue body wrapped in ``` for better formatting. It should look similar to this:
$ youtube-dl -v <your command line>
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.youtube.com/watch?v=BaW_jenozKcj']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2015.12.06
[debug] Git HEAD: 135392e
[debug] Python version 2.6.6 - Windows-2003Server-5.2.3790-SP2
[debug] exe versions: ffmpeg N-75573-g1d0487f, ffprobe N-75573-g1d0487f, rtmpdump 2.4
[debug] Proxy map: {}
...

Do not post screenshots of verbose log only plain text is acceptable.
The output (including the first lines) contains important debugging information. Issues without the full output are often not reproducible and therefore do not get solved in short order, if ever.

Hi,
Below is what i have got with -v flag
Not sure password is not matching up. But i tried it several time and now it has started working with same password.
Amans-MacBook-Pro:WordPress Ecommerce Easy Digital Downloads aman$ youtube-dl -v -u<EMAIL_ADDRESS>www.lynda.com/WordPress-tutorials/WordPress-Ecommerce-Easy-Digital-Downloads/122465-2.html
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'-u', u'PRIVATE', u'www.lynda.com/WordPress-tutorials/WordPress-Ecommerce-Easy-Digital-Downloads/122465-2.html']
Type account password and press [Return]:
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2016.06.02
[debug] Python version 2.7.10 - Darwin-15.5.0-x86_64-i386-64bit
[debug] exe versions: avconv 11.4, avprobe 11.4, ffmpeg 3.0.1, ffprobe 3.0.1
[debug] Proxy map: {}
WARNING: The url doesn't specify the protocol, trying with http
[lynda:course] Logging in as<EMAIL_ADDRESS>ERROR: Unable to download webpage: HTTP Error 401: Unauthorized (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 388, in _request_webpage
    return self._downloader.urlopen(url_or_request)
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 1946, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 437, in open
    response = meth(req, response)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 550, in http_response
    'http', request, response, code, msg, hdrs)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 475, in error
    return self._call_chain(*args)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 409, in _call_chain
    result = func(*args)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 558, in http_error_default
    raise HTTPError(req.get_full_url(), code, msg, hdrs, fp)

Some trick found to get it working remove https from url and its working
as soon as i put https it stops working and says unauthorised.
Mite can help to debug issue.


Move reference classification to ra_ide_db

Lost some marks along the way :-(
bors r+
🤖

bors r+

Hm, I guess I can/should restore marks tomorrow by making them cross-crate... This should be possible.


Error building Linux Mint - cannot find sqlite3

I'm getting this error below and for searching cannot see a way through it.
sqlite3 is installed and working fine.
I cannot see sqlite3.pc or similar and I'm not familiar with those .pc files then to know an obvious answer.
Curious that majkcramer commented in May on the other issue that he's been successful in Linux.. I tried building only this from git clone too but same error.
I'm using Cargo build; unsure what the rustc equivalent to Cargo.toml is to try rustc but I would expect the same error perhaps will follow from that too.
       Fresh libc v0.1.8
       Fresh winapi v0.2.1
       Fresh pkg-config v0.3.5
   Compiling libsqlite3-sys v0.2.0 (file:///home/davidpbrown/rust/firefox/rusqlite)
     Running `/home/davidpbrown/rust/firefox/rusqlite/target/debug/build/libsqlite3-sys-863cdbd64abd8e99/build-script-build`
       Fresh bitflags v0.1.1
       Fresh winapi-build v0.1.1
       Fresh advapi32-sys v0.1.2
       Fresh rand v0.3.9
       Fresh tempdir v0.3.4
       Fresh kernel32-sys v0.1.3
       Fresh time v0.1.32
failed to run custom build command for `libsqlite3-sys v0.2.0 (file:///home/davidpbrown/rust/firefox/rusqlite)`
Process didn't exit successfully: `/home/davidpbrown/rust/firefox/rusqlite/target/debug/build/libsqlite3-sys-863cdbd64abd8e99/build-script-build` (exit code: 101)
--- stderr
thread '<main>' panicked at 'called `Result::unwrap()` on an `Err` value: "`\"pkg-config\" \"--libs\" \"--cflags\" \"sqlite3\"` did not exit successfully: exit code: 1\n--- stderr\nPackage sqlite3 was not found in the pkg-config search path.\nPerhaps you should add the directory containing `sqlite3.pc\'\nto the PKG_CONFIG_PATH environment variable\nNo package \'sqlite3\' found\n"', ../src/libcore/result.rs:732

sqlite3.pc is the configuration file that should be installed so you can use pkg-config to figure out where the headers / library files are located. I haven't used Linux Mint, but I know on some systems, you have to install a -dev version of the sqlite package to get the headers and .pc file (e.g., libsqlite3-dev on Debian). Does it look like there's an equivalent on Mint?

Yes.. it needed libsqlite3-dev .. thanks!

@jgallagher I have the same problem on osx el.
   Compiling libsqlite3-sys v0.2.0
failed to run custom build command for `libsqlite3-sys v0.2.0`
Process didn't exit successfully: `/Users/doabit/Codes/rust/projects/TencentOAuth/target/debug/build/libsqlite3-sys-2fe684602508b1c3/build-script-build` (exit code: 101)
--- stderr
thread '<main>' panicked at 'called `Result::unwrap()` on an `Err` value: "`\"pkg-config\" \"--libs\" \"--cflags\" \"sqlite3\"` did not exit successfully: exit code: 1\n--- stderr\nPackage sqlite3 was not found in the pkg-config search path.\nPerhaps you should add the directory containing `sqlite3.pc\'\nto the PKG_CONFIG_PATH environment variable\nNo package \'sqlite3\' found\n"', ../src/libcore/result.rs:736

Ok, use  PKG_CONFIG_PATH=echo /usr/local/Cellar/sqlite/*/lib/pkgconfig/ cargo build`` can build success.

@doabit thanks! PKG_CONFIG_PATH=$(echo /usr/local/Cellar/sqlite/3.9.1/lib/pkgconfig/) cargo run works for me


Issue with k8s.io/docs/tasks/configure-pod-container/pull-image-private-registry/

I created a secret called regcred as mentioned in this document and I also created my-private-reg-pod.yaml and followed the commands. Whenever I run the command kubectl get pod private-reg the pod fails because of a failed pull with the error ImagePullBackOff. When I try to pull the image directly from powershell by logging into docker with the same credentials, it works fine.
This is a...

[ ] Feature Request
[ ] Bug Report
I dont think it is a bug but it must be something I am doing incorrectly. I am seeking help here. Thaks in advance.
Problem:

Proposed Solution:
Page to Update:
https://kubernetes.io/...

@abujji :wave: This issue sounds more like a request for support and less like an issue specifically for docs. You can bring your question to the #kubernetes-users channel in Kubernetes slack. You can also search resources like Stack Overflow for answers to similar questions.


Branch prediction

This implements static branch prediction. It takes an instruction and its PC and determines if
it's a branch or a jump and calculates its target. For jumps it will always predict taken. For
branches it will predict taken if the PC offset is negative.
Improved performance by about 14%.
Coremark ticks for an iteration
W/o  Branch Prediction  --> 13090
With Branch Prediction  -->  11467
Coremark Iterations per second
W/o  Branch Prediction  --> 76
With Branch Prediction  -->  87
Normalized performance score is now 0.872 iterations/Mhz, this is similar to IBex core that has a normalized performance score of about 0.904 Iterations/Mhz.
Comparing with 1.0.0 version (TCC release) that presented 0.670 Iterations/Mhz the RS5 core improved a lot in performance until now (about 30%) with branch prediction mechanism and with one stage forwarding.

with branch prediction

Without

Not a big area overhead but inserted timing problems.
TO DO:
Study a better way to overcome timing issues.

This method presented bad timing and was replaced by a more efficient with new decoder


Core Gui _ENV Error

Error Message
Error: currently-playing/ExpCore/gui.lua:85: attempt to index upvalue '_ENV' (a nil value)
Known Causes
Know causes that will result in error:

Server.interface with a thread
After player death

To Reproduce (please give for each cause outlined above)
Steps to reproduce the behavior:

Server.interface with a thread

Use Server.interface with second param as true
Has caused error once before during testing

After player death

Player died
Error was created
Not every death and not every player

Other information (please complete the following information):

OS: Windows, Linux
Factorio Version: 0.16.43

Additional context
Error does not appear to have a consitant time or event which causes it, functions stored in global table will be the cause but which function is unknown. Any help trying to resolve this is welcome.

Version v4.0 includes a method for sandboxing events, this includes the handling of upvalues. Hopefully this will no longer be a problem.


[Feature Request] Hide checkbox in selection column

Existing Component
Yes
Component Name
el-column
Description
There is a selection column, when some columns is not fit any condition, i want to hide the checkbox instead of unable it.

Duplicate of #11579


[runtime][python] Add debug sink to bindings

We don't support custom debug sinks in the Runtime Python bindings. In particular the ability to register a custom callback when tracing tensors.
This change makes it possible to create a HAL module with a Python function as a callback.
A special care is taken to avoid leaking due to circular references where the callback reference the HAL module itself.

This PR plumbs through the Python bindings the trace callback recently added by #18966.

In its current state, I found out that one needs to hold the HAL module python object in order for the debug sink context to not get deleted, which is not what I wanted. I wanted once registered in IREE, the HAL module to continue be a shared owner of the debug sink. I guess the C++ objects for the bindings gets destroyed and created on demand from the underlying IREE runtime objects.
I will need to rethink how to handle this.

I don't know how to reconcile the various desirable aspects. They seem contradictory.
Be able to pass a Python callable for the callback.
Do not burden the user with object lifetime management. Requiring them to hold the debug sink object while the HAL module is used or having them provide information if the callable would introduce cyclic dependencies.
Do not use traversal to detect reference cycles.
I think 1. is the whole point of having the flexibility that the runtime offers and to expose it up the stack. An alternative is to have a preset of implementations that we know don't create the cyclic dependency problem.
If we violate 2. partially, we can have the user specify who is to manage the callback in the API. Whether the bindings should share ownership. Then if the callback is problematic the user should manage it. Thus avoiding the cycle.

I think the balance that is typically used is your option 1 but where the callback explicitly takes the objects it will typically need to close over to function. Then make it safe if the user closes over too much but discourage this as unoptimal. This would let the user (or us as part of utilities) to have a global create_foo_sink(output_path) which doesn't close over anything but the output resource. If the user does it this way, then destruction is completely deterministic. Maintaining that property as an option is what you are after, and then providing some protection for cycles (but the cost will be non deterministic destruction).

You find variants of this in python's asyncio and async executor mechanisms where there is the recommended party of using closure-less callbacks.


Create .Portfolio.json

Coverage remained the same at 83.677% when pulling 5f237b5e986baa036fad72fc9845efbe3696bd4f on MaksSlesarenko:patch-1 into 9c21372b550fcbbe4e18b66d1feaa07eb96bb252 on nixsolutions:master.


JDK 11:  SAAJEnvelopeFactoryParserPoolCleanUp

Hello @mjiderhamn,
since the first PR I've opened for Java 11 compatibility (#87) looks really cluttered, with multiple commits to the same file that sometimes revert each other, I'm splitting the changes into smaller PRs.
To fix the SAAJEnvelopeFactoryParserPoolCleanUp we have to change 2 things. First, we have to use reflection to obtain the parser pool and to create a custom error handler, since the necessary classes are not accessible anymore with Java 11. Second, some neccessary classes have been removed from the JDK with Java 11, so we use a maven dependency if the internal classes can not be found.

Yes, you are right. I added a commit which cleans up both possible classes.
By the way, since this pull now contains 8 commits, would you like me to squash them into 2 or 3 commits and force push them, so you have a clean history, or do you prefer keeping the commits as they are right now?

Nah, this will work. Thanks for the PR!


SanitizeRhinoObjects for Blocks return null material

For e.g. Blocks, the mat is null when reaching this line.
https://github.com/Stykka/glTF-Bin/blob/7446876e116eb94716098f56fd2ce4e9620500ad/glTF-BinExporter/GlTFUtils.cs#L88

Do you have an example file? I can't reproduce this

This is no longer valid after #39


Update yarn.lock

Hey, I just made a Pull Request!
I get this when running yarn install from master.

:heavy_check_mark: Checklist

[ ] All tests are passing yarn test
[ ] Screenshots attached (for UI changes)
[ ] Relevant documentation updated
[ ] Prettier run on changed files
[ ] Tests added for new functionality
[ ] Regression tests added for bug fixes

These don't feel right - they are all downgrades of upgrades that were made recently I think?
https://github.com/spotify/backstage/pull/1979/files


Fix: have cspell check cc files

I fat-fingered it and forgot to reinclude this change.

This change is

@remyabel I'm sorry for this conflict, but I just merged https://github.com/googleapis/google-cloud-cpp/pull/5131 which means you'll need to merge. Should be an easy merge.

@remyabel I'm sorry for this conflict, but I just merged #5131 which means you'll need to merge. Should be an easy merge.

Not an issue. I caught the problem after looking at your PR, actually.


feat: 🎸 added ocsp revocation for did documents

Added revocation of DID documents through an OCSP responder

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

Hi @waynewee,
How would the OCSP revocation work for certificates that share the same certificate ID?
Currently larger institutions might not have unique certificate IDs across all their issuing registrars, as such revoking via certificate ID might cause the other certificates that share the same ID to be revoked as well. Is there a consideration to use something that will be unique across all certificates such as the targetHash of a document?

Hi @skydudie,
It is up to the implementers to decide on what exactly to use for the certificate ID. I believe the targetHash of a document is unique and you can use it as the certificate ID if desired.

Hi @waynewee thanks for the reply,
Just to clarify, that means for implementations where the certificate ID is not unique across they whole organisation, OCSP revocation would not work? Because OA-Verify will always take the certificate's 'id' as the parameter to check against the OSCP revocation list and the certificate id is not a unique value.
Besides, the targetHash is generated after the document is wrapped, so we won't be able to use it as the certificate ID.

Hi @skydudie,
You can give the certificate id any value. If the certificate ID is not unique across the whole organisation, then you have a number of options. One, you can setup two responders for each organisation in which certificate ids are always unique or two, you can generate new ids for all certificates and set that in the certificate id field.


[BUG] rich progress bar display problem

[x] I've checked docs and closed issues for possible solutions.
[x] I can't find my issue in the FAQ.

Describe the bug
rich print broken in some terminal, but i don`t know why.

Platform

Click to expand
What platform (Win/Linux/Mac) are you running on? What terminal software are you using?
I may ask you to copy and paste the output of the following commands. It may save some time if you do it now.
If you're using Rich in a terminal:
python -m rich.diagnose
pip freeze | grep rich

If you're using Rich in a Jupyter Notebook, run the following snippet in a cell
and paste the output in your bug report.
from rich.diagnose import report
report()

It's likely an encoding problem. I would be helpful if you could fill in the issue template, or at least let me know on which terminals you are seeing this issue.

Closing, assumed stale. Reopen if you can supply the requested info.


Add MOL space

Yes, It just took time to be available.
Now you can find it moltenfinance.eth space.
Please approve logo. Thanks in advance.


Ensure there is always only a single instance of an Effekt Runner

Sometimes I see that I have like three "Effekt Runner" terminals open. No idea how thay happens, my hypothesis is that VSCode persists them across projects / exits and the plugin just makes a new one on start. Instead, we should perhaps look if there's a terminal called "Effekt Runner" and use it instead?

Hypothesis: this happens when we reload the extension. Perhaps we should also search for a terminals called "Effekt Runner", close em all but the recent one?


Add tests

We should have some tests for this app. I am using cypress on another project. We don't have to use cypress, but it may provide some consistency across projects.

Hi @JesseRWeigel , I can write a few cypress tests for the app.

For now we just need testing set up and make sure the page loads properly. As we add features we will need more tests.

Thanks. I'm on it. Will send out the PR soon.

Hi @JesseRWeigel can this issue be closed now?

Yes, sorry I forgot to close it.


Evaluations

Hi
can someone help me understanding the different evaluations?
There is one<EMAIL_ADDRESS>0.70, 0.70" and one<EMAIL_ADDRESS>0.50, 0.50"
ans also a "Car coco..."
And which of the contents (bbox, bev, 3d, aos) describe the Detection
Precision? I can not find anything about that and the values in KITTI
Benchmark on their website are different.
Thank you!

Hello @traveller59 and @repa1022 , I have the same problem, The evaluation result in Car@0.7,0.7,0.7 and Car@0.7,0.5,0.5 are totally different, i understand that for 0.5 is different from 0.7, but why even for 0.7(easy) is different. Many thanks.

Hello @traveller59 and @repa1022 , I have the same problem, The evaluation result in Car@0.7,0.7,0.7 and Car@0.7,0.5,0.5 are totally different, i understand that for 0.5 is different from 0.7, but why even for 0.7(easy) is different. Many thanks.

Have you understood why they are different? I have the same doubts, can you tell me how to understand this? Thank you

Hi @traveller59  and @repa1022, I know it's an old issue, but I was using this today and I got some results, but I'm not sure how to interpret them.
My first question is what do you mean exactly, when you mention easy, moderate and hard? Is this supposed to describe the size of the object or some kind of occlusion (or lack thereof) and how exactly is it defined?
I also had two results, namely the official evaluation and coco evaluation. What is the difference between them?

Hello @mnik17 , easy, moderate and hard are the difficulty levels defined by KITTI object detection benchmark, you can check the definition of them in the KITTI website and the devkit readme file. Basically it is related to the object's height in the image plane, the occlusion level and truncation levels.
The official evaluation results are the results getting from KITTI evaluation script, COCO is another object detection benchmark, their evaluation metrics are different from KITTI. In my opinion, if you do work mainly on KITTI, you can ignore the COCO results.

Hello @pangsu0613, thank you for the fast reply, so basically if I have a 0.7 threshold for an easy object it will not consider any detections with less overlap than that for this type of objects?

@mnik17, 0.7 is the IoU threshold for car class in KITTI dataset (0.5 is for pedestrian and cyclist). If the IoU between a ground truth and a predicted object is larger than 0.7, it will be treated as a true positive, if it is smaller than 0.7, it will be treated as a false positive.

Ok, thank you very much @pangsu0613

hello @pangsu0613  I have the same problem, The evaluation result in Car@0.7,0.7,0.7 and Car@0.7,0.5,0.5 are totally different, i understand that for 0.5 is different from 0.7, but why even for 0.7(easy) is different. Many thanks.
Now do you understand? Can you help me

Hi @pangsu0613 , my understanding is that the first 0.7 in Car@0.7,0.5,0.5 is a typo, it should be all 0.5. basically, for Car@0.7,0.7,0.7, it shows all the easy, moderate and hard results with IoU thresholding at 0.7, and for Car@0.5,0.5,0.5, it shows all the easy, moderate and hard results with IoU thresholding at 0.5, because 0.7 is larger than 0.5, which means 0.7 is a more strict (harsh) criteria, so the numbers under 0.7 are smaller than the numbers under 0.5 (of course, comparison must be done under ths same difficulty level, 0.7 easy vs 0.5 easy, 0.7 moderate vs 0.5 moderate, etc).


Sats amount should not change for invoices with amount

Describe the bug
When you read a QR (lightning or on-chain) with specific amount the amount in sats sometimes change because price changes and because default currency is usd.
To Reproduce
Steps to reproduce the behavior:

Scan a QR with an invoice or on-chain address with specific amount
Click on Next
Wait for price changes
See sats amount change

Expected behavior
When we read an invoice or an on-chain qr with specific amount the amount in sats should be fixed. This is specially important for on-chain because invoice behavior will be handled by the backend (with the payment request)
Smartphone (please complete the following information):

Device: iPhone Xs
OS:iOS 15.5
Version v2.0.13 build 327

I've been meaning to file this issue as well. I made a video showing the issue
https://user-images.githubusercontent.com/17693119/178311506-d3520529-2298-4a84-815a-df5b2aed5720.mp4

I'm assuming this one has been fixed, but we can open if that's not the case


MINOR: logging improvements

This is a manual cherry-pick of https://github.com/apache/kafka/pull/3769 for 0.11.0

@mjsax @dguy @bbejeck please review.

@guozhangwang looks like you have checkstyle errors

just two minor comments otherwise LGTM.

Merged to 0.11.0 after fixing checkstyle issues (local run passed).


Feature Request: gradio custom component

I initially asked this in another issue but it seems more fitting here maybe.
Description
The gradio lib is a widely used frontend for python developpers. It makes it extremely easy to setup a web gui for some apps. I think that it would be beyond incredible if typesense would create a custom gradio component. It would make it very easy for many machine learning projets to add a webui to interact with typesense!
What do you think?

We prioritize feature requests based on the number of community asks and since this is the first time we're hearing about this, we'd want to wait a bit before evaluating if this is something we want to take on.
That said, if you're down to building an integration, happy to link to it from the documentation.


Add support for branch-specific shields

Some of the shields can be branch specific (like travis, coveralls). I suggest to add a new property to the shields opts called branch, that (if specified) would create the branch-specific shield
shields('travis', {
  repo: 'KenanY/shields',
  branch: 'master'
})

@zkochan Makes sense to me :)


Add print statements

in #43 @ErinWeisbart notes:

Doesn't really fit with the theme of this PR, but since it touches print() statements it made me think, it would be nice to have a print() at the end of every step saying it's done.
0./1. and 0./2. already end with print("All sites complete.") but something like print("Step 0./0. processing complete.") would be nice to have at the end of other steps.

I agree that this would be a nice feature to keep consistent. Perhaps we can bundle this fix with #42

oops! no bug


Add a button to open the first app group folder

"Open first app group folder" open the first app group folder of the selected application.

Output of xcrun simctl listapps <device_id>:
"bundle_id" = {
   [...]
   DataContainer = "file:///Users/xxxx/Library/Developer/CoreSimulator/Devices/4ED1E16E-52A3-4A6F-BFE4-6329C60D2840/data/Containers/Data/Application/2C05DC3C-48B8-44FB-9E22-BE3A5BDF9FF3/";
   GroupContainers = {
      "group_id" = "file:///Users/xxxx/Library/Developer/CoreSimulator/Devices/4ED1E16E-52A3-4A6F-BFE4-6329C60D2840/data/Containers/Shared/AppGroup/4FC3F1B2-76B4-44A4-80F8-92E65B48A41B/";
   };
   [...]
};

Feature requested here: https://github.com/twostraws/ControlRoom/issues/83

Fantastic – thank you very much!


How to add new models into the plugins

Hi,
I would like to check on how to add other models into the ingest-opennlp?
I am using elasticsearch 7.6.0 in docker and I am not sure on how to download the other models into the folder.
[root@5ea95291b711 ingest-opennlp]# ls
en-ner-dates.bin  en-ner-locations.bin  en-ner-persons.bin
[root@5ea95291b711 ingest-opennlp]#

you could call the plugin command in the docker container or volume mount those into your container, if you have created them manually.
Hope that helps!


Update README

Updated README to have more self hosting info and added info about the setup command to the Quick Setup section. Never done a pull request before so please tell me if I messed something up.

Thanks @TheLimifiedLime

No problem


Support Pypy

Description
Add support for pypy 3.8
Unsupported features

memory limited tracking cache (as pympler is not available on pypy)

Closing for now as the effort doesn't seem worth it.


Two versions of Grid LSTM cells

I noticed that there are two distinct versions of Grid RNN and LSTM cells in contrib:
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/contrib/grid_rnn/python/ops/grid_rnn_cell.py
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/contrib/rnn/python/ops/rnn_cell.py
I realize that the RNN interface is still in flux, but I was wondering if these will be harmonized at some point?

This might be a good issue to take up on the discussion board:
https://groups.google.com/a/tensorflow.org/forum/?utm_medium=email&utm_source=footer#!forum/discuss


build: remove unstable build option -Zconfig-profile

What problem does this PR solve?
Remove unstable build option -Zconfig-profile as this option is already stabled by https://github.com/rust-lang/cargo/pull/7823 in 1.43.0
$ make dist_release
error: unknown `-Z` flag specified: config-profile

What is changed and how it works?
What's Changed:
fix build script by remove outdated build option
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Side effects
Release note

/merge


[Add] Makefile

Use Makefile instead of Lakefile.
This Makefile can compile, load, test, and run REPL!

Hello, elderica.
I'd love to accept your patch, but I guess mixing Lakefile and Makefile in one project is confusing. So please wait until I've checked that all functions written in Lake have been transferred well to Make, since as you reported it is recognised that there are environmental problems in cl-waffe2.
Thanks.


LockedModel

How do I use LockedModel?

You need to include a specific phrase in the MOTD of the map to determine which model(s) the player is forced to use.
For example: /map motd -hax +respawn model=humanoid
This makes the map only allow human model.
You can also add more than one allowed model by using commas. For example:
model=humanoid,zombie,skeleton
If you don't have permission to set map options, you can try /os map motd instead.

Why does it not work /map main -hax +respawn model=1


Port more benchmarks to CnC-OCR

Available from other CnC implementations: Blackscholes, Heart Wall Tracking, Denoising (3D), Fluid registration, Smith-Waterman, Matrix Invert, etc.

I'm closing this enhancement request since this project has been rolled into the Habanero CnC Framework project. We can open a duplicate request in the cnc-framework issue tracker if we find that the current set of CnC-OCR examples and apps is insufficient going forward.


Update deepdive.conf documentation

https://github.com/HazyResearch/deepdive/blob/master/doc/doc/basics/configuration.md

[x] Revise Database connection section, which is optional now.
[x] extractors section seem overlapping a lot with the extractors page.

Updated. Please review.


Prevent from adding null callable to route

In Slim\Routable:add($callable), check if $callable is null. If so, throw exception.
This will prevent this kind of errors:

@akrabat What's Slim's policy as far as type checking goes?
I can create a PR for this but don't want to spend time on this if the burden of type checking is on the user and not on Slim.
Note: This type of issue is an argument for employing strict typing in Slim 4.x via PHP 7.1+

@RyanNerd it's on the list of todos once #2555 is merged before we release beta. We are going to type everything.

Thanks!
@RyanNerd You're right, it's an improvement and not a bug.
@l0gicgate Great!
Have a nice day!

@l0gicgate it appears that #2555 is for Slim 4.x -- What about 3.x?
It appears that 3.x has some manual type checking -- for example here:
https://github.com/slimphp/Slim/blob/e97d122a37b854485f39e42b23a72ec245cd276c/Slim/Http/Uri.php#L142
This is why I was asking about the policy for 3.x -- How much manual type checking should there be in 3.x?
In other words should Slim\Routable::add($callable) be manually type checked? If the answer is yes then this begs the question of where else should manual type checking exist? (see #2508)

My bad @RyanNerd I thought this was a 4.x issue. I don’t have the answers unfortunately.

In general, if you do something wrong that results in a runtime error that clearly shows what's wrong, then I think Slim is doing the right thing.
i.e. I wouldn't object to an exception message that explains that you've passed a null in.

@akrabat I was just thinking that assert() could be used but I think that PHP versions prior to 7 have a cost associated with them.
This is a feature that will be included in 4.x so probably best to close this issue as will-not-fix since the next version of Slim is slated to have this feature baked-in.


Failed to build image custom dockerfile

I'm trying to build from a custom dockerfile and im getting error "Failed to build image" and i searched for the error and found that is in line 202 (isImageCached) from generic-container.js.
What i understood is that when i build a imagem from a dockerfile, if the "from" image is not cached it returns and error. Is that a bug? Shouldn't it push automatically when it's not cached? I have to build it first then i can run testcontainers with no errors.

Hi @luismqoliveira

if the "from" image is not cached it returns and error

That is not correct

Shouldn't it push automatically when it's not cached

Push what?
Please share the error you got as well as a reproducible example

Hi @cristianrgreco
I'm sorry if you didn't understand my question, i'll try to explain it better.
I'm trying to create a container from a local dockerfile (which needs to download this library "FROM busybox:1.33.0 AS builder", but this happens with any library). Anytime i try to build this image if i don't have it locally cached it throws an error: "Error: Error: Failed to build image at GenericContainerBuilder. (.../node_modules/testcontainers/dist/generic-container.js:49:23" ). If i first build this image locally with docker build and then try to initialize the container it runs ok without any problem.
Here's my code:
 const buildContextCmpBackend = path.resolve(__dirname, "../../cmp-backend");     const cmpBackendImage = await GenericContainer.fromDockerfile(buildContextCmpBackend, "Dockerfile.qa")         .build();
Here's my Dockerfile.qa:
`FROM registry.airc.pt/library/busybox:1.33.0 AS builder
ARG BUSYBOX_VERSION=1.31.0-i686-uclibc
ADD https://busybox.net/downloads/binaries/$BUSYBOX_VERSION/busybox_WGET /wget
RUN chmod a+x /wget
FROM registry.airc.pt/distroless/java:11
COPY --from=builder /wget /usr/bin/wget
etc...`

Shouldn't it pull the image if it is not cached on my machine

Yes that's right. I've just verified this by removing my local images and running this test:
https://github.com/testcontainers/testcontainers-node/blob/787c63fc39899ecc5986b3748830201af0614ff9/src/generic-container.test.ts#L378-L389
Set DEBUG=testcontainers* as an environment variable and re-try and share the error

ok i found the problem. When i use a private registry like nexus to download the images which are cached there its throws an error.
If i download the image from docker hub its ok. I tried to put DEBUG flag like you said but i get no extra logs from image build. The only thing i get extra with "testcontainers*" are the logs of the started containers. It doesn't show any information about images builds

Great, it should just be a case then of running docker login into your
private registry and trying again?
On Mon, 3 May 2021, 18:31 luismqoliveira, @.***> wrote:

ok i found the problem. When i use a private registry like nexus to
download the images which are cached there its throws an error.
If i download the image from docker hub its ok. I tried to put DEBUG flag
like you said but i get no extra logs from image build. The only thing i
get extra with "testcontainers*" are the logs of the started containers. It
doesn't show any information about images builds
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/testcontainers/testcontainers-node/issues/204#issuecomment-831412422,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACO6R7VCPNDVUESTKNZRX73TL3MVLANCNFSM436WFVPA
.

@cristianrgreco
I'm trying to use my proxy private repo, even thought i have made docker login it seems that testcontainers is not picking up the right credentials... I mean i use testcontainers with java and it works well, but this one is not capable of picking up my private registry. I've compared them both and docker host and socket path are the same. This error is on jenkins, but i did use docker login in that same repo.

[ERROR] 2021-05-04T09:22:35.351Z testcontainers DEBUG Creating new DockerClient
[ERROR] 2021-05-04T09:22:35.351Z testcontainers INFO Using default Docker host: localhost, socket path: /var/run/docker.sock
[ERROR] 2021-05-04T09:22:35.352Z testcontainers DEBUG Creating new Reaper for session: 8eb712662a0b20c12d8cd42ae17994ed
[ERROR] 2021-05-04T09:22:35.379Z testcontainers DEBUG No registry auth locator found for registry: "xpto"
[ERROR] 2021-05-04T09:22:35.380Z testcontainers INFO Pulling image: xpto/cristianrgreco/ryuk:0.4.0
[INFO] [10:22:35] E/launcher - Error: Error: (HTTP code 500) server error - Get https://xpto/v2/cristianrgreco/ryuk/manifests/0.4.0: no basic auth credentials
[INFO]     at /root/workspace/sid/sid-test-n-merge/frontend/node_modules/docker-modem/lib/modem.js:315:17
[INFO]     at IncomingMessage. (/root/workspace/sid/sid-test-n-merge/frontend/node_modules/docker-modem/lib/modem.js:342:9)
[INFO]     at IncomingMessage.emit (events.js:203:15)
[INFO]     at endReadableNT (_stream_readable.js:1129:12)
[INFO]     at process._tickCallback (internal/process/next_tick.js:63:19)

Thanks for the info @luismqoliveira, this seems the problem:
testcontainers DEBUG No registry auth locator found for registry: "xpto"
Could you share the contents of ~/.docker/config.json (make sure to omit any plain text or base 64 encoded creds if you have them)? That will help me see why your registry isn't being found.
Also I know you're substituting the registry name to xpto but that might be masking some issue. Is the real registry a URL?
Please also share what version of testcontainers you are using, as there was a change to how credentials are resolved in 7.8.0

Hello @cristianrgreco
This "xpto" is the URL of a nexus repository. I will try to find out a way to see the content of ~/.docker/config.json during a build on jenkins. I'm not really sure how but i'll try.
Couldn't you improve the log level so that the image creation of a custom docker file is printed as the logs of the containers are in "testcontainers*"? It would be easier to see what is causing the problem if the image creation was logged and it is in "docker build" command on local machine.

I think the issue will be that xpto is represented in a way not currently
supported in your config.json file. If you are reproducing this issue
locally you could also share your local config.json file.
Regarding logs, the issue is clear from your previous message, we're
pulling from a private registry without providing authorisation.
On Tue, 4 May 2021, 18:40 luismqoliveira, @.***> wrote:

Hello @cristianrgreco https://github.com/cristianrgreco
This "xpto" is the URL of a nexus repository. I will try to find out a way
to see the content of ~/.docker/config.json during a build on jenkins. I'm
not really sure how but i'll try.
Couldn't you improve the log level so that the image creation of a custom
docker file is printed as the logs of the containers are in
"testcontainers*"? It would be easier to see what is causing the problem if
the image creation was logged and it is in "docker build" command on local
machine.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/testcontainers/testcontainers-node/issues/204#issuecomment-832121540,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACO6R7UPUX3C7O2DTD2OTCTTMAWQXANCNFSM436WFVPA
.

Here's my config.json file
{
"auths": {
"xpto": {
"auth": "hash"
},
"xpto2": {
"auth": "hash"
}
}
}

I am confused 😄 This test shows that for your auth config, if your registry name is xpto, then it would return true, and you would not see No registry auth locator found for registry: "xpto"
https://github.com/testcontainers/testcontainers-node/blob/f58fe65acbbf1959c61416ad23a8727a7c8ca4e0/src/registry-auth-locator/auths.test.ts#L19-L22
There must be some discrepancy with the information I have provided

Yes I think I know the issue, we are not forwarding auth information when building Docker images, I will raise a PR to implement this as soon as I have a chance. Thanks for raising @luismqoliveira

Thank you @cristianrgreco im happy I could help. Keep your good work mate !

Thanks @luismqoliveira 🙂 I've released the above PR in 7.10.0, could you please update and let me know how it goes?

Hello @cristianrgreco I've upgraded to 7.10.0 and the behavior is strange.
I'm trying to start 5 spring boot containers with custom Dockerfile from classpath. Each one of them have their unique Dockerfile but they are identic (the only diference is the jar name generated). I said the behavior is strange because sometimes i can successfully build two of them without any problem but in the third or other it fails with this error (but the image it tries to pull is from the same registry found one the last ones:

DEBUG No registry auth locator found for registry: "https://index.docker.io/v1/" +47ms
testcontainers INFO Pulling image: 126f7a1c01cd020e2b49e2248dc9db49:e090f45322c638e01c617974c571ccbf +0ms
[11:12:42] E/launcher - Error: Error: (HTTP code 404) unexpected - pull access denied for 126f7a1c01cd020e2b49e2248dc9db49, repository does not exist or may require 'docker login': denied: requested access to the resource is denied
at .../frontend/node_modules/docker-modem/lib/modem.js:315:17

This is the success:

DEBUG Preparing to build Dockerfile: /.../backend/Dockerfile.qa +7ms
testcontainers DEBUG No registry auth locator found for registry: "https://index.docker.io/v1/" +1ms
testcontainers DEBUG Found applicable registry auth locator for registry "xpto.pt": Auths +0ms
testcontainers INFO Building image '126f7a1c01cd020e2b49e2248dc9db49:e090f45322c638e01c617974c571ccbf' with context '/.../backend' +0ms

Hmm, I'll need some more info to be able to reproduce, are you able to share any code which reproduces the issue? My understanding is that you have 5 identical Dockerfiles, all of which pull an image from a private registry, and when you build all of them, only a few of them succeed, is that correct?
There should be more Docker build output as TRACE logs, could you share that as well?

@cristianrgreco it's working. I hidden my comment because it was my mistake.
Thank you very much !

That's great! I will close the issue then, thanks again 🙂


Define duration of pageTransition

Ionic version:
[ ] 1.x (For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[ x ] 3.x
I'm submitting a ...
[ ] bug report
[ x ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
In IonicModule.forRoot, I can set the pageTransition to ios-transition.
Expected behavior:
It would be interesting to be able to set the duration of the transactions too, instead of  set at all push/pop/setRoot animate.
Steps to reproduce:
Like this:
...
pageTransition: 'ios-transition',
duration: 300,
...

Ionic info:
global packages:

    @ionic/cli-utils : 1.4.0
    Cordova CLI      : 7.0.1
    Ionic CLI        : 3.4.0

local packages:

    @ionic/app-scripts              : 1.3.7
    @ionic/cli-plugin-cordova       : 1.4.0
    @ionic/cli-plugin-ionic-angular : 1.3.1
    Cordova Platforms               : android 6.2.3 ios 4.4.0
    Ionic Framework                 : ionic-angular 3.3.0

System:

    Node       : v7.10.0
    OS         : Linux 4.11
    Xcode      : not installed
    ios-deploy : not installed
    ios-sim    : not installed
    npm        : 5.0.3

Hello! Thanks for the feature request. I'm going to move this issue over to our internal list of feature requests for evaluation. We are continually prioritizing all requests that we receive with outstanding issues. We are extremely grateful for your feedback, but it may not always be our next priority. I'll copy the issue back to this repository when we have begun implementation. Thanks!

This issue was moved to ionic-team/ionic-feature-requests#258


[WIP] Some Issues Fixed

[ ] Rename class NotAllowEnterTxHubValidationProvider to ExcludeFromTxHubValidationProvider. #2483

[ ] Add AcsDescriptionHelper.IsAcs(string symbol)  instead of method IsAcs() in plugin #2462

[ ] Add GetAcsSymbol() in plugin. #2461

[ ] Add ProtoTypeConstants.EmptyInstance instead of new Empty. #2439

[ ] Add a Util to get Block Prefix #2438

:x: Build AElf <IP_ADDRESS>63 failed (commit https://github.com/AElfProject/AElf/commit/7686936a37 by @llypb)


Remove support to fetch values of a resolved configuration with oconf binary

As per #14 oconf file a.b.c will be deprecated.
v2:

[x] Add deprecation warning when using oconf file a.b.c and release a new minor version

v3:

[ ] Remove support for oconf file a.b.c
[ ] See if any of the current functionality in oconf file a.b.c needs to put into oconf-extract-value

{
  "foo": "bar"
}

oconf will normally return the JSON.stringified value of an extracted option.
$ oconf file.cjson foo
"bar"

Where oconf-extract-option will return the raw value.
$ oconf-extract-option file.cjson foo
bar

oconf would do the same when passed the --bare option
$ oconf --bare file.cjson foo
bar

oconf-extract-option will keep it's default for compatabilitys sake, and I'll introduce an option --json to get the JSON compatible output.

If you ask for an non existent key, both tools will error out. But oconf had a switch called --allowmissing that would cause it to exit with a status of 0 and no output when a missing key was encountered. This proved useful when using it to configure builds, so I'll add this option to oconf-extract-value too.

Turns out that oconf-extract-option already had the --json switch. Sorry about that - it's really sunday huh? :-)

Landed as 7cc2881b3d33887da1258295db0a8ce15be6c87c and will be released with 3.0.0


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/82659

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/83119

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/96122


Floating action button positioning makes it difficult to create group chat

The FloatingActionButton in UserPickerCard is positioned to be on the bottom right of the list which expands with the number of users instead of being fixed on the bottom right of the screen.
This makes the button appear out of position when the list is small:

But it's much more worse when there is a very large number of users, in which case a user will have to scroll all the way to the end of the UserList to find the button:

This issue also applies to InviteUsersScreen since it uses the same UserPickerCard component.

@zulipbot claim

Thanks @agzuniverse for reporting this! Now fixed. :smile:


feat: Ignore checks in tfvars file

Is your feature request related to a problem? Please describe.

I am using the github_repository resource with for_each to manage resources as they are defined in a tfvars file. For some of these repositories I need to ignore certain checks, for example github-repositories-private and github-repositories-enable_vulnerability_alerts, but for others I do not. To ignore the check I have to place the tfsec:ignore:<check> in the main configuration file, as below, which will ignore for all the repositories in the list.

main.tf

resource "github_repository" "this" {
  for_each    = { for repo in var.repositories : repo.name => repo }
  name        = each.value.name
  description = each.value.description
  #tfsec:ignore:github-repositories-private
  visibility = each.value.visibility
  #tfsec:ignore:github-repositories-enable_vulnerability_alerts
  vulnerability_alerts = try(each.value.vulnerability_alerts, false)
}

terraform.tfvars

repositories = [
  {
    name                      = "Public Repo",
    description              = "Alerts not applicable here because runtime not supported",
    visibility                   = "public",
    vulnerability_alerts = false
  },
  {
    name                      = "Private Repo",
    description              = "Alerts required here",
    visibility                   = "private",
    vulnerability_alerts = true
  },

Describe the solution you'd like

I'd like to be able to place the ignore comment in the tfvars file to enable more granular control over which resources checks are ignored for when using loops.
Describe alternatives you've considered

Use one resource block per repository.
Additional context

tfsec version is v1.26.3

Commenting to prevent closure

@chris3ware, you probably found the solution already, but apparently the ignore annotation needs to go on the resource/module itself, e.g.:
#tfsec:ignore:github-repositories-private
resource "github_repository" "this" {
  # ...
}

@chris3ware, you probably found the solution already, but apparently the ignore annotation needs to go on the resource/module itself, e.g.:

#tfsec:ignore:github-repositories-private

resource "github_repository" "this" {

  # ...

}

Hi, agreed, this is fine for a single resource. However the issue is querying how resources created using for_each can be ignored if required.
If, for example, you create 10 resources from a single resource block with different attributes defined using a variable - you may want to exclude some, but not others because each resource could have different attributes. In the GitHub example, some repos could be private and others public.

We've improved the trivyignore capabilities in the latest release of trivy, you can find more details here https://aquasecurity.github.io/trivy/v0.45/docs/configuration/filtering/#trivyignoreyaml


Autobuy

Hi, bait autobuy not working.

Hello @DomeSs97, have you filled the "BAIT" parameter and defined the "AUTO_BUY_BAITS" to true ? If so, the Scheduler controls the pace at which the commands will be sent, for bait purchase:

the initial delay for sending the first purchase request ranges from 5 to 20 minutes.
subsequent purchase requests (for baits) ranges from 15 to 30 minutes (+240/-120 seconds variation) .
the amount is calculated taking into account these timings

More info at: config guide.


Searching for text in logs returns no results.

Describe the bug
I tried to find some text in the logs but it always returns no results.
To Reproduce
Steps to reproduce the bug:

Go to any pod
Click on Show Logs
Click on loop
Search for any phrase in the text that exists
Search engine will return no results

Environment (please provide info about your environment):
Windows-Winget/In-Cluster

Headlamp Version: 0.26.0

Are you able to fix this issue?
No
Additional Context
Sample application https://github.com/spring-projects/spring-petclinic/tree/main/k8s

This is working for me in the main branch. I think we haven't touched it though, so I think we need to try this application and try to view logs from it. BTW @Andrew1988123123 , what is the Kubernetes flavor you are using (just in case)?

@joaquimrocha
I tested on the following k8s versions. Additionally, I opened console and got an error. Headlamp Version: 0.26.0
v1.29.0+k3s1
v1.29.1+k0s
v1.26.15+vmware.1(Tanzu)


check-sources.sh: move file lists to env variables

to allow alternative implementations to expand them.
With that in mind, make the awk logic used to match filenames a bit more
generic.

/test

/test


[dev] build from main for testing flang-rt

Necessary to build LLVM stack, namely for https://github.com/conda-forge/clangdev-feedstock/pull/326

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.


Add support for xijia1.curtain.x3

Add support for xijia1 curtain x3.

Thanks!

https://github.com/AlexxIT/XiaomiGateway3/releases/tag/v4.0.6


Related Lists doesn't reflect REST API specification.

When creating a new RelatedListOperations object, the SDK requires you to set a layout ID. This is not a requirement of the REST API, where the required argument is the module name. The Standard layout for that module should be used by default.
There is no way to specify the module name at all.

Thanks, @gmcnicol for your contribution. Please refer to this sample code for RelatedList Operations.

@gmcnicol  In both the getRelatedLists and getRelatedList methods, the layout ID is common,  and included as a parameter in the class constructor. Please note that the Layout ID is optional, as demonstrated in the provided sample code. This applies to both versions, v1.0.0 and v2.0.0. Could you please specify where you found that the Layout ID is a mandatory parameter?
We are currently updating the sample codes for all methods, and they will be published soon. However, as you had specific concerns regarding these methods, we have prioritized and updated the sample codes for these methods alone at this time.

Thanks. This was a 100% still issue on my part.
All resolved.


bump: backplane-api client

What type of PR is this?
feature: Bump to latest version of https://github.com/openshift/backplane-api
Which Jira/Github issue(s) does this PR fix?
https://issues.redhat.com/browse/OSD-22636
Pre-checks (if applicable)

[X] Ran unit tests locally
[X] Validated the changes in a cluster
[-] Included documentation changes with PR

/hold cancel

/lgtm
/approve


CI config changes to speed up tests

Updated accumulo 1.5.x version to 1.5.2

Hmm, not sure why this can't be merged automatically - I rebased against master.  Might me something with the formatting cleanup of the Pom.   Since that is hard to read, changes were
*increment default versions of
*geotools
*geoserver
*accumulo
*hadoop
*jacoco
*move hortonworks and cloudera profiles over to their respective versions/builds of accumulo
*travis.yml changes
*make document generation only run on master
*forked coveralls plugin tests first to see if it is in the local maven cache before clone and install
*document generation phase moved to before install so it gets included in the travis repo cache

See my comment in the diff.  I see the changes.  I am ready to purge, but I need to verify the one comment.

See my comment in the diff.
Please verify so I can complete the merge.

Nope, don't need the dash- not sure where it creeped in


GET /roleManagement/directory/roleAssignmentScheduleRequests requires more permissions than necessary

Describe the bug
The endpoint https://graph.microsoft.com/v1.0/roleManagement/directory/roleAssignmentScheduleRequests should be accessible with a token with scope RoleManagement.Read.Directory, but when I try to make a request through the Graph Explorer it fails with a permission error:
{
    "instanceAnnotations": [],
    "message": "Authorization failed due to missing permission scope RoleAssignmentSchedule.ReadWrite.Directory,RoleManagement.ReadWrite.Directory.",
    "errorCode": "PermissionScopeNotGranted"
}

To Reproduce

get an access token with just RoleManagement.Read.Directory app privilege assigned
call https://graph.microsoft.com/v1.0/roleManagement/directory/roleAssignmentScheduleRequests
it errors

Expected behavior
Read permissions should give enough access to read data.
Screenshots

Additional context
This is probably not a graph explorer bug as I can reproduce this also with just calling the API with curl.  I don't know where to report this problem.

Hey @Fuco1, I have been able to reproduce this. I shall look into it

Seeing that it isn't an issue on Graph Explorer and has more to do with the API.
I would suggest you ask the question here: https://learn.microsoft.com/en-us/answers/products/graph


Add version: ransome1.sleek version 1.0.7

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.1 schema?

Microsoft Reviewers: Open in CodeFlow

@wingetbot run


Insert Emoji by Ctrl+Cmd+Space in code break the current line

VSCode Version: Code 1.10.2 (8076a19fdcab7e1fc1707952d652f0bb6c6db331, 2017-03-08T13:56:35.908Z)
OS Version: Darwin x64 16.4.0
Extensions:

Extension
Author
Version

beautify
HookyQR
1.0.2

project-manager
alefragnani
0.13.5

theme-desertex
casualjim
0.0.6

vscode-eslint
dbaeumer
1.2.8

githistory
donjayamanne
0.2.0

json-tools
eriklynd
1.0.2

prettier-vscode
esbenp
0.11.0

flow-for-vscode
flowtype
0.5.0

vscode-icon-theme
jtlowe
1.5.0

Go
lukehoban
0.6.55

addDocComments
stevencl
0.0.8

react-beautify
taichi
0.3.0

Steps to Reproduce:

Go to any line of code, I usually insert Emoji in the HTML code or console.log in Javascript
Press Ctrl+Cmd+Space to open the Dialog to select Emoji
Navigate between Emoji by arrow buttons, press Enter to choose. Sometimes it will break the code, insert more space or comma character, sometimes it just insert exactly that Emoji

Duplicate of #4271


Improve Accuracy

I trained a model using the dnn_metric_learning_on_images_ex.cpp code . The only change I made was to increase the iteration without progress threshold to 10000.
I used a dataset with 63K identites that is a combination of VGG2, Asian Celeb and Clean Microsoft Dataset. I also did some manual cleaning. All identities have at least 80 images.
The result was not good. I downloaded the Face Scrub dataset and generated 5K random genuines pairs and 5K random distractors pairs. EER, FMR100 and FMR1000 were worse on this new model.
I don't know what went wrong since I was excepting a good improvement.
@davisking , do you have any advice on this?

You have to look at the outputs and see what is wrong.  Very likely your dataset is just bad, like the labels are not very accurate.  Run the model on held out data and look at the errors.  Look at pairs of images it gets wrong, and then look at a whole lot more.  The only way to find out what is wrong with your data is to look at the things the model is getting "wrong" according to the data.

Thanks @davisking . I will do some more rounds of data cleaning.
I will let you know once I have new results.

@davisking , can you share the mini batch size you used and also the "iterations without progress threshold"? I performed some cleaning on the dataset using Amazon algorithm, but even after that, the results are far from being as good as with your original model.
Thanks

The http://dlib.net/dnn_face_recognition_ex.cpp.html example program has all these details.  See that for detailed notes.