Directly opening Harlequin keybindings results in "too many redirects"

If I click the link in the readme, or go directly to https://harlequin.sh/docs/bindings, Chrome shows me a:

However, if I navigate to https://harlequin.sh and navigate to the bindings, no issue.
I haven't repro'd on other devices, so if you can't repro, feel free to close 👍

Getting the same. Thanks. Super weird...


Hard Lumbridge Diary - Rename 'Duel Arena' to 'PvP Arena'

Description for the Smite Altar task needs to be updated, it is still using the old name:
Rename the Duel Arena to PvP Arena

The task diary refers to it as the Emir Arena in-game, though the map, world names and Grouping interface all call it the PvP Arena. Newsposts also refer to it as the PvP Arena more commonly than Emir's Arena, though the wiki's chosen name for it is Emir's Arena. Sorry for the tangent - I was weighing up if PvP Arena is the name to use, and I think it is.


Making sure an image stays centered when zoomed out.

Is there a way to make sure the image stays centered when zoomed out.

zoom out by the zoom method, not touch or wheel.

Thanks! Could you please clarify how I could zoom out w/ the zoom method?


feat(firestore): Query explain and explain analyze

Description
Adding Firestore query explain and explain analyze samples.
The build will pass once 1.17.0 has been released. I will merge this PR once https://github.com/googleapis/google-cloud-go/pull/10597 has been merged.
This has been tested locally by pointing cloud.google.com/go/firestore to local directory
Checklist

[x] I have followed Contributing Guidelines from CONTRIBUTING.MD
[x] Tests pass:   go test -v ./.. (see Testing)
[x] Code formatted:   gofmt (see Formatting)
[x] Vetting pass:   go vet (see Formatting)
[ ] These samples need a new API enabled in testing projects to pass (let us know which ones)
[ ] These samples need a new/updated env vars in testing projects set to pass (let us know which ones)
[ ] This sample adds a new sample directory, and I updated the CODEOWNERS file with the codeowners for this sample
[ ] This sample adds a new Product API, and I updated the Blunderbuss issue/PR auto-assigner with the codeowners for this sample
[x] Please merge this PR for me once it is approved

Please be sure to address the errors from vet.


prepare 0.32.0 release

Coverage remained the same at 99.213% when pulling ac8044a68c77e9a71f22868336496c3f13a38f1d on LeFnord:master into c08c6f46ca7e5ec79aa0bf56ca25df257eef47ca on ruby-grape:master.


usernames with spaces

new users can currently set up accounts with spaces in the username.
We either need to make this impossible or adapt the client code to pass spaces appropriately.

I think the account creation process can catch this so we dont have to
deal with this in BRIDGES (yeah, they could always put in some junk with
spaces -- but wouldnt that trigger an unknown user id then when the JSON
is sent to the server with that?).
I think the above is reasonable, since none of user accounts (on
computer systems do now allow spaces)..
- krs
On 10/11/18 8:07 PM, David Burlinson wrote:

new users can currently set up accounts with spaces in the username.
We either need to make this impossible or adapt the client code to
pass spaces appropriately.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/squeetus/bridgesAPI/issues/23, or mute the thread
https://github.com/notifications/unsubscribe-auth/AFZSOOGSHTiIetjbw7WjLW3sGNAQJrhJks5uj91AgaJpZM4XYkrp.

--
Kalpathi Subramanian                                 Ph:<PHONE_NUMBER>
Associate Professor                                    Email<EMAIL_ADDRESS>Dept of Computer Science                          Web:http://webpages.uncc.edu/krs
The University of North Carolina
Charlotte, NC 28202-0001

Spaces are addressed in the Java client in the following commit https://github.com/squeetus/bridges/commit/43925282aef9e3cdbdabbeafd66e1f74f5fdb7c0
We need to reevaluate the symbols we allow in usernames in general (Erik mentions people who create accounts with plus signs).

Added username validation in this commit https://github.com/squeetus/bridgesAPI/commit/4e22afcc8d7036f860e04e78254304d57c43628b.
Usernames will now be restricted to alphanumeric characters, underscores, and dashes.


CRASH: index 0 beyond bounds for empty array

*** Terminating app due to uncaught exception 'NSRangeException', reason: '*** -[__NSArrayM objectAtIndex:]: index 0 beyond bounds for empty array'
HTTPConnection.m:1477
DDRange range = [[ranges objectAtIndex:rangeIndex] ddrangeValue];

Some years later, I have also hit the same hurdle.
Thanks for the explanation.


[BUG] Changing variable expiration_days in yml file is ignored

Describe the bug
Changing this variable have no effect, after deployment certificate expiration date will be 365 days
How to reproduce
Steps to reproduce the behavior:

execute epicli init ... (with params)
edit config file, change expiration_days to any value different than 365
execute epicli apply ...
check expiration date of k8s certificate

Expected behavior
k8s certificate expiration date set to value of expiration_days variable
Config files
n/a
Environment

Cloud provider: Azure
OS: Ubuntu 18.04.5

epicli version: 0.10.0
Additional context
n/a

DoD checklist

[ ] Changelog updated (if affected version was released)
[ ] COMPONENTS.md updated / doesn't need to be updated
[ ] Automated tests passed (QA pipelines)

[ ] apply
[ ] upgrade

[ ] Case covered by automated test (if possible)
[ ] Idempotency tested
[ ] Documentation updated / doesn't need to be updated
[ ] All conversations in PR resolved
[ ] Backport tasks created / doesn't need to be backported

Hello @wkalbar, please attach used configuration and verification commands. Did you set renew: true according to the doc?

 Of course I didn't set it, as of course I didn't RTFM...


Add note about execveat

/cc @brauner

Updated to fix typo.

Needs a rebase.
To cut things short I took the liberty to incorporating this (and updating it a bit) into #22. Closing this one.
(@keszybz please review/merge #22!)


Bookstore  add reducers and actions

Bookstore: add reducers and actions
In this Milestone, I implemented Redux Toolkit inside of a React application.
To accomplish that, I :

Created a directory that will contain all  Redux logic (/src/redux)
Setup a Redux store (/src/redux/store.js)
Create a slice to save state and reducers

Hello @danielochuba Kindly remove the live demo section from your README Microversse announced that you should delete the live demo section since you didn't deploy the project
but you can add once you deploy it
it is not required to deploy it
but for now just I am asking to delete the live demo section
once you deploy it, you can add it :star-struck:


added surface area setter and getter setter checks

added surface area setter and getter setter checks
Description

see summary
Motivation and Context

There was no surface area setter for the sphere

Types of Changes

[ ] Documentation update
[ ] Bug fix
[x] New feature
[ ] Breaking change1

1The change breaks (or has the potential to break) existing functionality.
Checklist:

[x] I am familiar with the Contributing Guidelines.
[x] I agree with the terms of the Contributor Agreement.
[x] The changes introduced by this pull request are covered by existing or newly introduced tests.
[x] I have updated the changelog and the credits.

I've created a separate issue #114 to address @bdice's request across the package.


ToolNode output is broken for non-english

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangGraph/LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangGraph/LangChain rather than my code.
[X] I am sure this is better as an issue rather than a GitHub discussion, since this is a LangGraph bug and not a design question.

Example Code
langgraph.prebuilt.tool_node.py

def str_output(output: Any) -> str:
    if isinstance(output, str):
        return output
    else:
        try:
            return json.dumps(output)
        except Exception:
            return str(output)

Error Message and Stack Trace (if applicable)
No response
Description
I found the tool node output is encoded.
And when I check _run_one and _arun_one function, they use str_output function.
And I checked the arg, the input is alreadt encoded before return str or json.dumps.
I don't know about that well...
this is not completely suitable code but I can solve my problem temporarily
def str_output(output: Any) -> str:
    return json.dumps(json.loads(output), ensure_ascii=False)

And I think the origin problem presents here: langchain_core.tools.base
def _stringify(content: Any) -> str:
    try:
        return json.dumps(content)
    except Exception:
        return str(content)

they just got a dumps without ensure_ascii=False.
Thank you for reading and I am forwarding to fixing this quickly
System Info
Just simple error

Test Case
from enum import Enum

from flask.cli import load_dotenv
from langchain.pydantic_v1 import BaseModel
from langchain_core.tools import tool
from langchain_openai import ChatOpenAI
from langgraph.graph import MessagesState, StateGraph
from langgraph.prebuilt import ToolNode

load_dotenv()

class Day(str, Enum):
    MONDAY = "星期一"
    TUESDAY = "星期二"
    WEDNESDAY = "星期三"
    THURSDAY = "星期四"
    FRIDAY = "星期五"
    SATURDAY = "星期六"
    SUNDAY = "星期日"

class Week(BaseModel):
    """choose days"""

    days: list[Day]

@tool(args_schema=Week)
def get_day_list(days: list[Day]) -> list[Day]:
    return days

tools = [get_day_list]
model = ChatOpenAI(model="gpt-4o-mini").bind_tools(tools)

workflow = StateGraph(MessagesState)
workflow.add_node("enter", lambda state: {"messages": model.invoke(state["messages"])})
workflow.add_node("tool", ToolNode(tools))

workflow.set_entry_point("enter")
workflow.add_edge("enter", "tool")
workflow.set_finish_point("tool")
app = workflow.compile()
outputs = app.invoke(
    {"messages": "我星期一和 Friday and 水曜日 和 목요일 have free time"}
)
print(outputs["messages"][-1].content)

Output
["\u661f\u671f\u4e00", "\u661f\u671f\u4e94", "\u661f\u671f\u4e09", "\u661f\u671f\u56db"]

Will merge a fix in core, but users should be able to return strings directly from the tool to avoid relying on any auto-conversion of Enum values to strings (or anything more complex).


Reference input randomSeeds by idx rather than batchSlot

Steps to reproduce:

Compile a model with max_batch_size 8 or higher (this will make the steps more easily reproducible). I used 128
Perform an extremely high temperature request (will be random output)

curl -Z -s --parallel-max 10 -d  '{"text_input": "Why did the chicken cross the", "max_tokens": 1, "bad_words": [], "stop_words":[],"stream":true, "temperature":100.0,"random_seed":30,"top_k":0,"top_p":1.0,"return_log_probs":true}' 'http://localhost:8000/v2/models/tensorrt_llm_bls/generate_stream?[1-4]' | grep text_output

Perform the same request with different randomSeed:

curl -Z -s --parallel-max 10 -d  '{"text_input": "Why did the chicken cross the", "max_tokens": 1, "bad_words": [], "stop_words":[],"stream":true, "temperature":100.0,"random_seed":31,"top_k":0,"top_p":1.0,"return_log_probs":true}' 'http://localhost:8000/v2/models/tensorrt_llm_bls/generate_stream?[1-4]' | grep text_output

Finally, perform a single request with randomSeed 0:

curl -Z -s --parallel-max 10 -d  '{"text_input": "Why did the chicken cross the", "max_tokens": 1, "bad_words": [], "stop_words":[],"stream":true, "temperature":100.0,"random_seed":0,"top_k":0,"top_p":1.0,"return_log_probs":true}' 'http://localhost:8000/v2/models/tensorrt_llm_bls/generate_stream' | grep text_output

The output of the steps I get in my build of phi-4-mini-4k is
2.
"output_log_probs":-10.639476776123047,"text_output":"ram"
"output_log_probs":-10.670726776123047,"text_output":"jpeg"
"output_log_probs":-10.670726776123047,"text_output":"jpeg"
"output_log_probs":-10.670726776123047,"text_output":"jpeg"
3.
"output_log_probs":-10.946898460388184,"text_output":"implementation"
"output_log_probs":-10.670726776123047,"text_output":"jpeg"
"output_log_probs":-10.670726776123047,"text_output":"jpeg"
"output_log_probs":-10.670726776123047,"text_output":"jpeg"
4.
"output_log_probs":-10.670726776123047,"text_output":"jpeg"

The same bug occurs in all models. you will get different tokens output, but the point is that the incorrect responses in 2. and 3. will match the randomSeed 0 from step 4.
A single request goes through curandInitialize which is correct, rather than curandBatchInitialize, so we can trust the output of performing a single request at once.
Note also that when performing a batch, both curl and trtllm are greedy and start processing it not as a batch, before processing the other 3. This is why you see one result in the batch with the correct answer before the batch produces the incorrect answer.
You will see that in steps 2 and 3, you see one different output which respects the randomSeed, followed by 3 outputs which matches the randomSeed 0 one.
Debugging steps:
Thread 447 "tritonserver" hit Breakpoint 15, tensorrt_llm::kernels::invokeCurandBatchInitialize (states=0x3414e6600, batchSlots=0x7fffc5c00200, batchSize=3, randomSeeds=0x3414e7e00, stream=0x7ff8d0dcb380) at /app/tensorrt_llm/cpp/tensorrt_llm/kernels/decodingCommon.cu:63
63          curandBatchInitialize<<<grid, block, 0, stream>>>(states, batchSlots, batchSize, randomSeeds);
(gdb) print *(int(*)[128])batchSlots
$1 = {35, 36, 37, 0, 0, 0, 0, ... (all 0)}
(gdb) print batchSize
$2 = 3
(gdb) init-if-undefined $tensor_scratch_buf = malloc(1024*1024*256)
(gdb) print (int)cudaMemcpy($tensor_scratch_buf, randomSeeds, 128 * 8, 4)
$3 = 0
(gdb) x/128gd $tensor_scratch_buf
0x7ffbfbe00010: 123     123
0x7ffbfbe00020: 123     0
0x7ffbfbe00030: 0       0
0x7ffbfbe00040: 0       0
... (all 0) ...

Thus it appears to be incorrect to read from randomSeeds[batchSlots[0]] and correct to read from randomSeeds[0] instead.

@pathorn thanks for submitting the fix, our engineer is working to cherry-pick this fix into our internal repo and after the cherry-pick gets done it will be pushed to the github main branch in the subsequent Tuesday. It is possible that this MR will get landed into the TensorRT-LLM main branch in the next week or the week after it.
Thanks
June

This fix was merged in. Thanks!


Issues 26 uncaught type error

Hi, please accept this PR. It's fix issues #26, #29 and pehraps #30.
I use API documentation for onDidAddTextEditor.
https://atom.io/docs/api/v1.0.9/Workspace#instance-onDidAddTextEditor
getEditor method doesn't exist for event object, so it is undefined.
And thank you for your package.
Toff9

Yes, this appears to fix that issue as well. Thanks! @jacogr any issues with merging this?
-s

It'd be really great to see this merged in.  @jacogr is there anything any of us could to do help to make this happen?

Merged & published at 0.10.5

Thanks so much!


Migration code generation

This issue has been moved from a ticket on Developer Community.

I am using migrations to create my tables in my database.  I ran a migration yesterday to create a table.  Today I decided to rename the table, add a few columns and delete a few columns.  The Up() method that was generated gave me the following code along with other code to implement my changes:
RenameTable(name: "StandardSilo.StandardSiloPartArchived", newName: "StandardSiloPartLogs");
DropForeignKey("StandardSilo.StandardSiloPartArchived", "SegmentNumberId", "dbo. SegmentNumbers");
This caused an error because you can't rename  a table, and then try to drop a foreignkey using the old name.  It was a simple fix to simply switch the lines, but took a bit of time to realize what was going on.

Original Comments
Visual Studio Feedback System on 9/11/2019, 04:05 AM:
We have directed your feedback to the appropriate engineering team for further evaluation. The team will review the feedback and notify you about the next steps.

Original Solutions
(no solutions)

EF Team Triage: This issue is not something that our team is planning to address in the EF6.x code base.  This does not mean that we would not consider a community contribution to address this issue.
Moving forwards, our team will be fixing bugs, implementing small improvements, and accepting community contributions to the EF6.x code base. Larger feature work and innovation will happen in the EF Core code base (https://github.com/aspnet/EntityFramework).
Closing an issue in the EF6.x project does not exclude us addressing it in EF Core. In fact, a number of popular feature requests for EF have already been implemented in EF Core (alternate keys, batching in SaveChanges, etc.).
BTW this is a canned response and may have info or details that do not directly apply to this particular issue. While we'd like to spend the time to uniquely address every incoming issue, we get a lot traffic on the EF projects and that is not practical. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we use canned responses for common triage decisions.


Parameterize the command that install dependencies

Basically I want to be able to pass the --ignore-scripts flag. Normally just parameterizing npm flags would be better but one might want to use npm ci instead of npm install. One possibility is to replace npm install with npm ci, hard code the latter and take the flags with a parameter like npmFlags.
I also left --nodedir=${nodeSource nodejs} hardcoded since it is not possible to pass it otherwise since nodeSource is an internal function.

It seems npm ci doesn't run hook scripts, so it is not an option to make it replace npm install without resolving https://github.com/nix-community/npmlock2nix/issues/110.

Actually I just realized even with the proposed changes the interface is not expressive enough for me. Ideally there should be a parameter named installCommands that defaults to [ "npm install --offline --nodedir=${nodeSource nodejs}" ], similar to buildCommands in npmlock2nix.build. But the problem is again, one can't reuse the --nodedir=${nodeSource nodejs}" part if they want to define the parameter themselves.

I just realized npmlock2nix has internal attribute, so it is actually possible to reuse nodeSource function. With this in mind, I think introducing installCommands is the better solution.


"Getting the most recent point with a new asset" endless loop

Environment
I'm using official cardano-graphql:5.1.0 and cardano-graphql-hasura:5.1.0 docker images.
{
  os: {
    arch: 'x64',
    cpus: { count: 4, model: 'AMD EPYC 7282 16-Core Processor' },
    release: '5.4.0-80-generic',
    totalMem:<PHONE_NUMBER>,
    type: 'Linux'
  },
  version: '5.1.0'
}

Platform

[x] Linux / Ubuntu
[ ] Linux / Other
[ ] macOS
[ ] Windows

Platform version:
Runtime

[ ] Node.js
[x] Docker

Runtime version:

Node:

$ node -v
v14.17.4

Docker (well containerd from Kubernetes):

OS-IMAGE                        KERNEL-VERSION     CONTAINER-RUNTIME
Ubuntu 20.04.1 LTS 31403879ea   5.4.0-80-generic   containerd://1.5.4

Steps to reproduce the bug
It goes into this loop while booting and never brings the API up.
What is the expected behavior?
That it boots :)
Additional details
After enabling debug output, I found that it goes into an endless loop with this:
{"name":"cardano-graphql","hostname":"cardano-graphql-75f8d6f79f-tjlnf","pid":1,"level":20,"msg":"Getting the most recent point with a new asset: Attempt 1 of 1001, retrying...","time":"2021-09-13T21:54:49.162Z","v":0}

I've checked the query that it's performed for that task and realised it returns empty results in this deployment:
root@cardano-node-0:/# curl 'http://hasura-headless:8080/v1/graphql' -H 'Accept-Encoding: gzip, deflate, br' -H 'Content-Type: application/json' -H 'Accept: application/json' -H 'Connection: keep-alive' -H 'DNT: 1' - --data-binary '{"query":"query {\n            assets (\n                limit: 1\n                offset: 1\n                order_by: { firstAppearedInBlock: { slotNo: desc }}\n            ) {\n                firstAppearedInBlock {\n                    hash\n                    slotNo\n                }\n            }\n        }"}' --compressed

{"data":{"assets":[{"firstAppearedInBlock":null}]}}

While it works in other ones I have:
root@cardano-node-0:/# curl 'http://hasura-headless:8080/v1/graphql' -H 'Accept-Encoding: gzip, deflate, br' -H 'Content-Type: application/json' -H 'Accept: application/json' -H 'Connection: keep-alive' -H 'DNT: 1' --data-binary '{"query":"query {\n            assets (\n                limit: 1\n                offset: 1\n                order_by: { firstAppearedInBlock: { slotNo: desc }}\n            ) {\n                firstAppearedInBlock {\n                    hash\n                    slotNo\n                }\n            }\n        }"}' --compressed

{"data":{"assets":[{"firstAppearedInBlock":{"hash":"\\x7d101f5d5c73e1e0827a1a1283086def2e58e1d5c3beb1e7ddaa27dd756dd353","slotNo":37251447}}]}}

I've tried restarting db-sync, hasura and ogmios also, just to see if there was some problem with any schema/table/view that would recover on restart, but no luck :(
It seems db might have been corrupted because of some graphql abrupt restart (I had some), but wanted to report the issue just in case.

It seems db-sync went a bit nuts, after restarting everything again it worked, so I'm assuming there was some other problem not related to graphql service itself. Closing!


Incorrect function reference?

"Some of these Run methods, such as the Task.Run(Func<Task>) overload, exist as shorthand for the TaskFactory.StartNew method. Other overloads, such as Task.Run(Func<Task>),"
The same method is referenced twice here. I'm not sure that makes sense.

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

ID: cabcc3ba-9033-c65f-8969-482a436d510b
Version Independent ID: a74714e2-d059-e33f-a8c6-591d988ef2a4
Content: Consuming the Task-based Asynchronous Pattern
Content Source: docs/standard/asynchronous-programming-patterns/consuming-the-task-based-asynchronous-pattern.md
Product: dotnet
Technology: dotnet-standard
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thanks for writing this @martingbrown
I've added it to our backlog to address when we next update this article.


fix: process geth steps iteratively

ref https://github.com/paradigmxyz/reth/issues/2974
This rewrites the recursive function iteratively, the recursive version is susceptible to stack overflows with very deep calls.
This adds a stack of steps that maintains the order in which they appear in the transactions, so the recursive call is replaced by pushing its items onto the stack

Codecov Report

Merging #2980 (9b8a750) into main (5e01f9d) will decrease coverage by 57.51%.
The diff coverage is 0.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@             Coverage Diff             @@
##             main    #2980       +/-   ##
===========================================
- Coverage   70.97%   13.46%   -57.51%
===========================================
  Files         520      520
  Lines       67151    67174       +23
===========================================
- Hits        47662     9048    -38614
- Misses      19489    58126    +38637

Flag
Coverage Δ

integration-tests
13.46% <0.00%> (-3.84%)
:arrow_down:

unit-tests
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...s/revm/revm-inspectors/src/tracing/builder/geth.rs
0.00% <0.00%> (ø)

crates/revm/revm-inspectors/src/tracing/types.rs
0.00% <0.00%> (ø)

... and 398 files with indirect coverage changes


add schema for team data

We need to be able to query for an individual team.
Requirements
Input → team ID
Fields:

Full Team Name (ex: Los Angeles Dodgers)
Stadium Name
Venue ID
League Name
League ID
Division Name
Division ID

References

https://statsapi.mlb.com/api/v1/teams/119/

Closed by #3


Adding Istanbul Atatürk Airport (LTBA)

This airport sounds very interesting especially since there are plans for a new parallel runway (currently 3 runways). Also one of the heaviest traffic movements in the world. I would also love to pull request and add but I'm new to this.

Also I created THY, and NAX files also I just created LTBA where am I able to pull request cause I'm not able to.

To open a pull request, you have to fork this repository (the Fork button in the top right), then push to your copy as usual. From your repository, you can then open a pull request back to this one.

Got it thanks!


How can I change the Cursor Color of the Textfield independent of the Primary color

Steps to Reproduce
The cursor seems to default to the primary color as well as the drag handles of a textfield.
The primary color is sometimes very hard to see above a white background and I would like to change it for certain themes. I could not find a theme setting to do so.
Flutter Doctor
C:\Pojects\lambDev\lookatmybaby_flutter>flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (on Microsoft Windows [Version 10.0.16299.214], locale en-US, channe
l dev)
[√] Android toolchain - develop for Android devices (Android SDK 27.0.2)
[√] Android Studio (version 3.0)
[√] IntelliJ IDEA Community Edition (version 2017.2)
[√] VS Code (version 1.20.0)
[√] Connected devices

• No issues found!

This solved the problem for me (see #10063): add textSelectionColor: Colors.black to the ThemeData when calling the constructor of MaterialApp at the very "top" of the app.

@ride4sun Does @dsteinkopf's solution work for you?

@Hixie thanks that's working.
#15571 is related.

They have already provided cursorColor property in ThemeData

Does not seems to work on my iOS device, but is working on my Android.
Should it be working ?
Flutter version stable 1.2.1

@g-balas
This should work well.
MaterialApp(
  title: "Rate your Colleagues",
  theme: ThemeData(
    cupertinoOverrideTheme: CupertinoThemeData(
      primaryColor: Colors.red,
    ),
    cursorColor: Colors.red,
    home: SplashScreen(),
  );

TextFormField(
cursorColor: Colors.yellow,
    )

Step By Step Guide to flutter textfield background color on focus

Step By Step Guide to flutter textfield background color on focus

return MaterialApp(
theme: ThemeData(
textSelectionTheme: TextSelectionThemeData(
cursorColor: Colors.orange
)
),


struct "IstioComponentSetSpec" miss field citadel

Bug description
I want to  use istio-operator(tag 1.6.3) to install istio.I also need to add some code to implement my own function.So i meet the problem.

First,I dont know what version of istio/api i should import.So i imported the default version,maybe it's the latest

Second,here is the bug

Expected behavior
When I create an IstioOperator, the independent operator can install istio successfully.
Steps to reproduce the bug

clone code
execut: git checkout 1.6.3
copy the operator directory as a new project
execute : go mod init github.com/myproject/istio-operator
use ide to replace all istio.io/istio with github.com/myproject/istio-operator
execut: go mod tidy && go mod vendor
start project
execute:

kubectl apply -f - <<EOF
apiVersion: install.istio.io/v1alpha1
kind: IstioOperator
metadata:
  namespace: istio-system
  name: example-istiocontrolplane
spec:
  profile: demo
EOF
istiooperator.install.istio.io/example-istiocontrolplane created

check out the output log. The result as fllows:

Version (include the output of istioctl version --remote and kubectl version and helm version if you used Helm)
tag 1.6.3
Environment where bug was observed (cloud vendor, OS, etc)
MacOs

Citadel is removed and no longer present in Istio 1.6+

Citadel is removed and no longer present in Istio 1.6+
If i want to do some change on operator,I should quote code from this?

https://github.com/istio/istio/tree/1.6.3/operator

Yes

Yes

https://github.com/istio/istio/blob/1.6.3/operator/pkg/vfs/assets.gen.go#L397

Take a look at this. this is default.yaml,but it contains citadel.And that's what leads to installing failed.
How can I fix it? Would it be ok if I delete all config values about citadel in assets.gen.go?

And another problem is:
https://github.com/istio/istio/blob/1.6.3/operator/pkg/vfs/assets.gen.go

some configurations in this file are missing even if i delete values about citadel.
So I wonder how you managed to build project。
If you show can show me how to build istio-opearot , that would be great。


VNodes can't have both text content and children properties

In the current implementation, it seems that having .text and .children are mutually exclusive. The current code is:
if (is.array(children)) {
    ...
} else if (is.primitive(vnode.text)) {
    api.appendChild(elm, api.createTextNode(vnode.text));
}

Since text content is technically the first node (a text node) child of an element, I think it should instead be more like:
if (is.primitive(vnode.text)) {
    api.appendChild(elm, api.createTextNode(vnode.text));
}
if (is.array(children)) {
    ...
}

Will PR if approved.

textNode uniformity has significant performance implications.
parentNode.firstChild.nodeValue = ...
is ~15% faster than
parentNode.textContent = ...
which is > 100% faster than
parentNode.appendChild(textNode)
I know this because I went through the same process in domvm[1]: "Why not make the internal impl uniform". Needless to say, that experimental branch did not survive the perf regressions.
[1] https://github.com/leeoniya/domvm

So how about an additional conditional between .nodeValue = ... and .textContent = ...? I'd say API uniformity is worth far more than the extra cost of evaluating a single brahc.

right. the external api should be uniform. conditionals are cheap.

@leeoniya So if I understand you correctly the way Snabbdom currently does it is not the fastest way? We should replace node.textContent with node.firstChild.nodeValue?
@chrisdotcode
The text property is used if you create a vnode like this h('a', {...}, 'Click me'). I.e. in the case where the entire content is text. They should never be set at the same time. If you want text and other child elements you can do h('div', ['Some text', aVnode, 'more text']).

@paldepind yep.
if node contains only text set node.textContent during initial creation and node.firstChild.nodeValue during patching.

They should never be set at the same time.

@paldepind Yes, that's what I'm asking about. In my nicety to not make them mutually exclusive: that is, why can't we make it possible such that .text and .children can be set at the time time? As noted above, it's just a change in the conditional I posted above, and another conditional using node.textContent if there is text content during node diffing.

btw, domvm does use createTextNode when there is no other option, like when there are sibling nodes. but the patching is still done using nodeValue. aside from some conditionals in handling some special cases for optimization (textContent/nodeValue) , text nodes are no differe than normal nodes. i dunno about snabbdom's internals, but it shouldnt be particularly difficcult to implement.

@paldepind Yes, that's what I'm asking about. It would be nice if they were not mutually exclusive: that is, why can't we make it possible such that .text and .children can be set at the time time? As noted above by me and @leeoniya it's just a change to one or two conditionals.

Because it is unnecessary.  If you want an element with both text nodes and elements as children you can do something like this: h('div', ['Some text', aVnode, 'more text']). The text property handles the special case where an element has only a single text node as a child.

I think this would be a welcome change, but you've already decided.
Thanks anyway.

Cain you explain what your use case is?

Well, as of now, the API is inconsistent: if it's children or text, then the two should maybe occupy the same parameter like so:
VNode('div', 'textContent')
or
VNode('div', [ ... children ...])
```js

As of now, it's a gotcha that you can *specify* both text and children, like this:

```js
VNode('div', 'textContent', [ ... children ...])

It makes it seem as if both will be utilized, when text is ignored if there are children present. My solution was that both could be present, but if not, how about the above solution: in that either text or children is specified in the same parameter?


Update UG, DG

Closes #238 #258

Codecov Report

Merging #259 (194fad8) into master (3779371) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #259   +/-   ##
=========================================
  Coverage     71.57%   71.57%
  Complexity      956      956
=========================================
  Files           120      120
  Lines          3244     3244
  Branches        441      441
=========================================
  Hits           2322     2322
  Misses          802      802
  Partials        120      120

Continue to review full report at Codecov.

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


[BUG]

-32603: Internal Error - [-32000] Server error: DB Not Found Error: BLOCK HEIGHT: 86451935 Cause: Unknown. Please report a bug at https://github.com/aurora-is-near/aurora-relayer/issues

hi @dprylypko . There is a fix landed on testnet. Can you please try and validate if this issue is still occuring?

Hi
Everything working

That is great to hear. Thank you. Please feel free to reopen this issue if it will occur again


WIP: feature/consoleDockerCompose

I don't really know what to check in the assertDockerCompose

@maybeec Only the linux pipeline succeeds, but windows always fails, i tried multiple projects
. Locally on my windows pc it works fine. Seems like its a problem with the installed docker. See failed jobs for logs.

@maybeec Only the linux pipeline succeeds, but windows always fails, i tried multiple projects
. Locally on my windows pc it works fine. Seems like its a problem with the installed docker. See failed jobs for logs.

@MarcelDiessner, @maybeec I think for most projects the command will not work.
I can reproduce the error thrown in the build for the my-thai-star project when switching to windows containers in docker. In the project, 'bridge' is set as the network driver for docker. This driver is not supported on windows. When switching back to linux containers, the docker-compose command works as expected.
The problem is that linux containers are not supported on Windows Server 2019, as explained here: https://github.com/actions/virtual-environments/issues/1143#issuecomment-651178811

@MarcelDiessner, @maybeec I think for most projects the command will not work.
I can reproduce the error thrown in the build for the my-thai-star project when switching to windows containers in docker. In the project, 'bridge' is set as the network driver for docker. This driver is not supported on windows. When switching back to linux containers, the docker-compose command works as expected.
The problem is that linux containers are not supported on Windows Server 2019, as explained here: https://github.com/actions/virtual-environments/issues/1143#issuecomment-651178811

I think with my latest changes i should be able to make it run on windows pipeline, but the path of the DockerCli.exe is wrong. Is there a way to get the path of this file.

I think with my latest changes i should be able to make it run on windows pipeline, but the path of the DockerCli.exe is wrong. Is there a way to get the path of this file.

I think with my latest changes i should be able to make it run on windows pipeline, but the path of the DockerCli.exe is wrong. Is there a way to get the path of this file.

Maybe you can extract the docker installation directory from the environment variables.

I think with my latest changes i should be able to make it run on windows pipeline, but the path of the DockerCli.exe is wrong. Is there a way to get the path of this file.

Maybe you can extract the docker installation directory from the environment variables.


Revised first steps in docs

Pull request type:

[ ] Feature
[ ] Bug fix
[ x] Documentation
[ ] Breaking change

I have:

[ ] Tested the code
[ ] Wrote unit tests for the code
[ ] Documented changes:

[ ] Guide(s)
[ ] API reference
[ ] Changelog

Changes:

Lgtm, I just removed the poetry.lock file


Migrate progress

updated the info and relocated it in an issue, as its much easier to edit.
@tungld @chentong319 @doru1004 @tjingrant
Please update the "additional functionality" column (last one) as I think many are not accurate. I believe we don't support broadcast, but many are labeled as "M" (indicating Multicast is implemented). If that is the case, great. If it is not the case, then label it as "noM," meaning it needs multicast but it is not yet implemented.
Thanks

It is correct that we fully support bidirectional/unidirectional broadcasting as described in ONNX.

@tung How about matmul and mul? Currently listed as unsupported?

They support broadcasting also. BTW, I would like to move ReduceL1, ReduceL2, ReduceLogSum, ReduceLogSumExp, and ReduceSumSquare to the supported operation table since they were implemented but I have no permission to edit the issue’s content.

In general, all binary elementwise ops will support bidirectional broadcasting because they use the same lowering template.

oh, if you cannot edit, then I should put it back to a GitHub md... or is there a way to make anyone edit a specific issue entry

ok, migrated the data back to the SharedWork.md to allow editing by anyone


Suggestion: Export weather data to ENV variables

Great tool, thank you!
Please add export of weather data to ENV variable and an optional „no display“ mode. Users could then integrate weather data via 3.2+ WorkbenchPrefs/ConfigureTitlebar using placeholders for these environment variables instead of drawing a window over the screen titlebar.
This has no side effects. We added the feature for this very purpose (displaying temperature sensor data in my case) ;-)

Thank you. It's a good idea to keep the variables related to this in ENV:. I'll try it for next version.

Hi again,
Added environment variables (CurrentTemperature, Location and WeatherDescription) for other applications to use.


Update copyright for 2018

LICENSE file should have a copyright date in 2018 since last release was in February.

Coverage remained the same at 96.032% when pulling 3e13f514ce5185a057a686221b61d9a9d9e86889 on Crayon2000:patch-1 into aed855e6920923898b94a1b922fbace27a34ddf2 on akheron:master.

Merged, thanks!


How to generate a picture verification code in .net Core?

I can't find any samples about picture verification code in current .net Core RC2.

@HaoK @blowdart @divega

That's because it is not a feature we offer. There are extension points where you could add it yourself though.

can you recommand some extensions about build verification code?

why not use recaptcha? that is what most people use and it is very easy to integrate

@joeaudette I'm in China. can't  access the google's services. Is there anything else do you recommand ?

http://www.yinxiangma.com/ is recommended in http://stackoverflow.com/a/24000429

@blowdart   even js can easyly generate  a picture with text .
Is .net Core really powerless ?


Update README.md

@satya164 the project is public so we have to add this so people don't assume that this package is ready


Refinements on mutually recursive sum types

I would like to write something like to following :
type list1 (a:Type) =
  | Nil1 : list1 a
  | Cons1: hd:a -> tl:list2 a -> list1 a
and list2 (a:Type) =
  | List: t:uint8{t=0uy \/ t=1uy} ->
           v:list a{ (t=0uy <==> is_Nil1 v)
                     /\ (t=1uy <==> is_Cons1 v) } ->
           list2 a

However the identifiers is_Nil1 and is_Cons1 are unknown to F* at this point. Is it possible to make F* aware of those constructors ?
Is there another way to write that ?

I think the following type is isomorphic:
type list1 (a:Type) =
  | Nil1 : list1 a
  | Cons1: hd:a -> tl:list2 a -> list1 a
and list2 (a:Type) =
  | Nil2: list2 a
  | Cons2: hd:a -> tl:list1 a -> list2 a

What exactly are you trying to do?

I want to make some type transformations in F* to make some boxing / unboxing explicit and add tags to sum types to experiment with going to C.
Said otherwise, I would like to know if I can write something like
type ptr (#a:Type) = | Ptr: v:a -> ptr a
type list1 (a:Type) =
  | Nil1 : list1 a
  | Cons1: hd:a -> tl:list2 a -> list1 a
and list2 (a:Type) =
  | List: t:uint8{t=0uy \/ t=1uy} ->
           v:ptr (list1 a){ (t=0uy <==> is_Nil1 (Ptr.v v))
                     /\ (t=1uy <==> is_Cons1 (Ptr.v v)) } ->
           list2 a

It is supposed to mimic C pointers and structures, and I want to say that the knowledge of the tag value is enough to know which constructor it points to, I don't need to dereference the pointer.
In any case, I would be interested in knowing if the above typeis acceptable with regards to the type system and if so, if we could get F* to accept it.

Allowing such a type would be shady from a metatheory point of view. Coq also rejects such types for instance:
Inductive list1 (a:Type) : Type :=
  | Nil1 : list1 a
  | Cons1: forall hd:a, forall tl:list2 a, list1 a
with list2 (a:Type) : Type :=
  | List: forall t:bool,
            { v:list a | (t=false <-> match v with
                                        | Nil1 => True
                                        | _    => False
                                      end)
                         /\ (t=true <-> match v with
                                          | Cons1 _ _ => True
                                          | _         => False
                                        end) } ->
            list2 a.

produces:
Toplevel input, characters 462-467:
Error: Unknown constructor: Cons1.

Just discussed this a bit with Nik, and he thinks you can add additional indices to identify each constructor of an inductive type. Here is how this would look for your type (using bool instead of uint8 here):
type list1 (a:Type) : bool -> Type =
  | Nil1 : list1 a false
  | Cons1: hd:a -> tl:list2 a -> list1 a true
and list2 (a:Type) =
  | List: #b:bool ->
           v:list1 a b ->
           list2 a

OK I will experiment with that.
Thanks.

Note that Agda accepts this kind of inductive-inductive definition:
module Shady where

open import Data.Bool
open import Data.Product

mutual
  data List1 (A : Set) : Set where
    nil1  : List1 A
    cons1 : (hd : A) -> (tl : List2 A) -> List1 A

  data List2 (A : Set) : Set where
    list2 : (t : Bool) ->
            (v : Σ (List1 A)
                   (λ x -> T (if t then is_nil1 x else is_cons1 x))) ->
            List2 A

  is_nil1 : ∀ {A : Set} -> List1 A -> Bool
  is_nil1 nil1         = true
  is_nil1 (cons1 _ _)  = false

  is_cons1 : ∀ {A : Set} -> List1 A -> Bool
  is_cons1 nil1        = false
  is_cons1 (cons1 _ _) = true

Now I'm not saying that the theory behind this is entirely clear, but the idiom is useful enough to raise the question of whether you want F* to support it. Coq is quite conservative in this regard...and Agda is, well, on the opposite end.

Yes, Catalin and I were discussing that Agda supports it. Induction-recursion is pretty non-trivial though. I don't think we'll be adding this to F* any time soon.

Some things to add to me reading list :)
https://en.wikipedia.org/wiki/Induction-recursion_(type_theory)

I think the original question is more about inductive-inductive definitions than inductive-recursive ones. Add this one to the reading list too :)
http://www.cs.swan.ac.uk/~csetzer/articles/forsbergSetzerInductionInductionCsl2010.pdf

+@adl I too am interested in supporting induction recursion in F*

This is not in the roadmap. @ad-l, reopen this if you think this is necessary/relevant.


Simple example throws exception when parsing GitHub JSON response.

Consider the following code:
  /* Creates a GitHub Client */
  var github = createGitHubClient();

  github.repositories.getRepository(new RepositorySlug("DirectMyFile", "github.dart")).then((Repository repo)
  {
    print(repo.description);
  });

When executing this code the following exception is thrown:
Uncaught Unhandled exception:
FormatException: Unexpected character (at line 2, character 1)
//# sourceURL=https://api.github.com/repos/DirectMyFile/github.dart
^

#0      _ChunkedJsonParser.fail (dart:convert-patch/convert_patch.dart:1326)
#1      _ChunkedJsonParser.parse (dart:convert-patch/convert_patch.dart:898)
#2      _parseJson (dart:convert-patch/convert_patch.dart:19)
#3      JsonDecoder.convert (dart:convert/json.dart:501)
#4      JsonCodec.decode (dart:convert/json.dart:129)
#5      GitHub.getJSON.<getJSON_async_body> (package:github/src/common/github.dart:251:21)
#6      _RootZone.runUnary (dart:async/zone.dart:1137)
...

If I set a break point just prior to the crash, I observe this input passed to decode:
{
    "id":22344823,"name":"github.dart","full_name":"DirectMyFile/github.dart",
    ... a lot of reasonable looking JSON ...
}
//# sourceURL=https://api.github.com/repos/DirectMyFile/github.dart

If I inspect the web traffic between my browser and the internet, I see the following HTTP response:
{
    "id":22344823,"name":"github.dart","full_name":"DirectMyFile/github.dart",
    ... a lot of reasonable looking JSON ...
}

Note the lack of the final //.

The behavior is only observed when running the code in debug mode.


Can't open VSCode anymore with UNC path mapped user

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

VS Code Version:

1.81.1
6c3e3dba23e8fadc360aed75ce363ba185c49794
x64

OS Version: Windows Server 2019

Steps to Reproduce:

Use mapped roaming profile with unc path
Open VSCode

Recently everything worked, after upgrading to the latest version, VSCode can't be started anymore.
NODE_UNC_HOST_ALLOWLIST is set accordingly.

This needs more exact steps for how to reproduce, from reading the description I have no clue how I would reproduce this on Windows. For example:

is this a mapped network drive
if not, why not? does it work?
do you start with any command line arguments such as --user-data-dir
is the folder writeable?
can you try with the previous version wheather this really regressed?

We have a command to test out previous builds to find the day of breakage. It will start previous releases of VS Code insiders to see where the regression was introduced:

npx --yes vscode-bisect@latest
follow the instructions until you found the offending build
report back the commit range in this issue

Thanks!


Ensure brand image shows up properly and fix typo

This, with https://github.com/JuliaComputing/MultiDocumenter.jl/pull/26, will ensure that the brand image shows up properly and redirects users to a valid link when clicked.

The assets would need to be doubled, otherwise it would break the gh-pages docs

The assets would need to be doubled, otherwise it would break the gh-pages docs

Hmm. Maybe we should simply copy the directory before running MultiDocumenter.make then. I'll fix that up.


Values for request parameters are invalid: properties.encryption.identity

Is there an existing issue for this?

[X] I have searched the existing issues

Greenfield/Brownfield provisioning
greenfield
Terraform Version
1.7.2
Module Version
0.1.1
AzureRM Provider Version
3.93.0
Affected Resource(s)/Data Source(s)
azurerm_storage_account_customer_managed_key
Terraform Configuration Files
Identity resources:

main.tf:

resource "azurerm_user_assigned_identity" "uami" {
  for_each = local.identities

  resource_group_name = each.value.resource_group_name
  location            = each.value.location
  name                = each.value.name
}

locals.tf:

locals {
  identities = {
    staraw = {
      resource_group_name = var.resource_group_name
      location            = var.location
      name                = var.staraw_identity_name
    },
    staecu = {
      resource_group_name = var.resource_group_name
      location            = var.location
      name                = var.staecu_identity_name
    },
    stawks = {
      resource_group_name = var.resource_group_name
      location            = var.location
      name                = var.stawks_identity_name
    }
  }
}

outputs.tf:

output "identity_info" {
  description = "The resource IDs, names, and principal IDs of the user-assigned identities"
  value       = { for i, identity in azurerm_user_assigned_identity.uami : "identity${i}" => { id = identity.id, name = identity.name, principal_id = identity.principal_id } }
}

AVM-RES storage account:

Module in main.tf:

module "avm-res-storage-storageaccount_int" {
  source   = "Azure/avm-res-storage-storageaccount/azurerm"
  for_each = local.int_storage_accounts
  version  = "0.1.1"

  account_kind              = each.value.account_kind
  account_replication_type  = each.value.account_replication_type
  account_tier              = each.value.account_tier
  allowed_copy_scope        = each.value.allowed_copy_scope
  blob_properties           = each.value.blob_properties
  customer_managed_key      = each.value.customer_managed_key
  enable_telemetry          = each.value.enable_telemetry
  is_hns_enabled            = each.value.is_hns_enabled
  name                      = each.key
  resource_group_name       = module.cog-res-resource-groups.data_resource_group_names["${module.naming_style.resource_group.name}-storage"]
  shared_access_key_enabled = each.value.shared_access_key_enabled

  depends_on = [module.cog-res-resource-groups, module.avm-res-keyvault-vault, module.cog-res-identity]
}

locals.tf:

locals {
  int_storage_accounts = {
    "sta${join("", var.company_prefix)}raw" = {
      account_kind             = "StorageV2"
      account_replication_type = "LRS"
      account_tier             = "Standard"
      allowed_copy_scope       = "AAD"
      blob_properties = {
        delete_retention_policy = {
          days = 7
        }
        last_access_time_enabled = false
      }
      customer_managed_key = {
        key_vault_resource_id              = module.avm-res-keyvault-vault[0].resource.id
        key_name                           = "${module.naming_style.key_vault_key.name}-raw"
        user_assigned_identity_resource_id = module.cog-res-identity.identity_info["identitystaraw"].id
      }
      enable_telemetry          = var.enable_telemetry
      is_hns_enabled            = true
      shared_access_key_enabled = false
    },
    "sta${join("", var.company_prefix)}ecu" = {
      account_kind             = "StorageV2"
      account_replication_type = "LRS"
      account_tier             = "Standard"
      allowed_copy_scope       = "AAD"
      blob_properties = {
        delete_retention_policy = {
          days = 7
        }
        last_access_time_enabled = false
      }
      customer_managed_key = {
        key_vault_resource_id              = module.avm-res-keyvault-vault[0].resource.id
        key_name                           = "${module.naming_style.key_vault_key.name}-ecu"
        user_assigned_identity_resource_id = module.cog-res-identity.identity_info["identitystaecu"].id
      }
      enable_telemetry          = var.enable_telemetry
      is_hns_enabled            = true
      shared_access_key_enabled = false
    },
    "sta${join("", var.company_prefix)}wks" = {
      account_kind             = "StorageV2"
      account_replication_type = "LRS"
      account_tier             = "Standard"
      allowed_copy_scope       = "AAD"
      blob_properties = {
        delete_retention_policy = {
          days = 7
        }
        last_access_time_enabled = false
      }
      customer_managed_key = {
        key_vault_resource_id              = module.avm-res-keyvault-vault[0].resource.id
        key_name                           = "${module.naming_style.key_vault_key.name}-wks"
        user_assigned_identity_resource_id = module.cog-res-identity.identity_info["identitystawks"].id
      }
      enable_telemetry          = var.enable_telemetry
      is_hns_enabled            = true
      shared_access_key_enabled = false
    }
  }
}

tfvars variables values
N/A

Debug Output/Panic Output
Expected Behaviour
Storage accounts encrypted using managed identities. Terraform plan shows the correct user-assigned managed identities being associated but fails with terraform apply.
TF plan:
module.avm-res-storage-storageaccount_int["stacogdevuksecu"].azurerm_storage_account_customer_managed_key.this[0] will be created

resource "azurerm_storage_account_customer_managed_key" "this" {

id                        = (known after apply)
key_name                  = "kvk-cog-dev-uks-ecu"
key_vault_id              = "/subscriptions/xxx-xxx-xxx-xxx/resourceGroups/rg-cog-dev-uks-keyvault/providers/Microsoft.KeyVault/vaults/kv-cog-dev-uks"
key_vault_uri             = (known after apply)
storage_account_id        = "/subscriptions/xxx-xxx-xxx-xxx/resourceGroups/rg-cog-dev-uks-storage/providers/Microsoft.Storage/storageAccounts/stacogdevuksecu"
user_assigned_identity_id = "/subscriptions/xxx-xxx-xxx-xxx/resourceGroups/rg-cog-dev-uks-storage/providers/Microsoft.ManagedIdentity/userAssignedIdentities/uai-cog-dev-uks-staecu"
}

Actual Behaviour
Error: updating Customer Managed Key for Storage Account (Subscription: "xxx-xxx-xxx-xxx"
│ Resource Group Name: "rg-cog-dev-uks-storage"
│ Storage Account Name: "stacogdevukswks"): storage.AccountsClient#Update: Failure responding to request: StatusCode=400 -- Original Error: autorest/azure: Service returned an error. Status=400 Code="InvalidValuesForRequestParameters" Message="Values for request parameters are invalid: properties.encryption.identity."
│
│   with module.avm-res-storage-storageaccount_int["stacogdevukswks"].azurerm_storage_account_customer_managed_key.this[0],
│   on .terraform/modules/avm-res-storage-storageaccount_int/main.tf line 314, in resource "azurerm_storage_account_customer_managed_key" "this":
│  314: resource "azurerm_storage_account_customer_managed_key" "this" {
Steps to Reproduce

terraform plan
terraform apply

Important Factoids
No response
References
No response

@Tommy-Ten thanks for reaching out. Please can you upload the variables files and other modules configuration files to re-produce the issue.

Thanks for the reply @chinthakaru. Apologies, I missed the variable configs but please find them below.
variables.tf for identity module:
variable "location" {
type = string
}
variable "resource_group_name" {
type = string
}
variable "staecu_identity_name" {
description = "The name of the staecu identity"
type        = string
}
variable "staraw_identity_name" {
description = "The name of the staraw identity"
type        = string
}
variable "stawks_identity_name" {
description = "The name of the stawks identity"
type        = string
}
avm-res-storageaccount does not use any variables in the root dir variables.tf.
Hope that helps?

Hi @chinthakaru,
Have you been able to reproduce the issue based on the configuration I provided, please?

@Tommy-Ten Thanks for the sample configuration code. managed_identities block is missing in your local.tf for each storage account. Please can you update the main.tf and local.tf files with folowing code blocks and let us know ?
local.tf update on each storage account
  managed_identities = {         system_assigned            = true         user_assigned_resource_ids = [module.cog-res-identity.identity_info["identitystawks"].id]       }
main.tf
 managed_identities        = each.value.managed_identities

hi @chinthakaru ,
Ah, that was it, can't believe that I missed it. Got it working now, thanks very much for your time looking into this!

Solved


It crashes as soon as I select it (coin) in the inventory

game version:1.19.1
Fabric version:0.14.9
Mod version:0.2.3
log:-1

Please provide a crash report and your game log of when this crash occured. Otherwise we cant tell what is crashing

The game crashes without any reports but shows -1

This is why I asked for your game log, or rather your latest.log. It contains a lot more useful information than a non-existing crash report.
See this for more information: https://fabricmc.net/wiki/player:tutorials:logs_ml:windows

The game crashed whilst unexpected error
Error: java.lang.IllegalArgumentException: Unknown TooltipComponent
結束代碼：-1

end code: -1

There is no way for me to reproduce this without the log. Please post it, or I am unable to help you

As this issue has now been stale for over a month, I'll go ahead and close it. If you can actually provide the log to us, please re-open it so we can figure out what's going on
Cheers


Variable named "v"

See:
https://github.com/UCSD-PL/RefScript/blob/master/tests/todo/v-name.ts
Easy workaround: don't name things "v".

See:
https://github.com/UCSD-PL/RefScript/blob/master/tests/todo/v-name-01.ts

Seems to be working at 15ccd4ff8fc4d6447eb842bf6d5f1b283a74decf


HDDS-4818. Add command to get status of Finalize in OM & SCM

What changes were proposed in this pull request?
Add command to get status of Finalize in OM & SCM
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-4818
How was this patch tested?
a new ut
manual test

Hi @avijayanhwx , I added a subcommand to om cli first, maybe you would like to have a look at it first and then, I'll do some changes and then go with SCM.

cc @errose28 for review.

Thanks for working on this @guihecheng. The new OM command looks good, I think you can the PR out of draft mode.

Thanks @errose28 , I'll push an update this week with a similar scm command and tests.

@avijayanhwx @errose28 updated for review, thx~

Hi @guihecheng,
thank you for working on this, it seems to be a good improvement, I did throw the idea when working on the first implementation, as we thought about finalization as a not-too-long process, which will be monitored by an admin, but I can see the usefulness of this or the ease of understanding from the user's point of view compared to takeover.
I think there is a critical flaw remained in the implementation, I have added a detailed comment about it inline.
Besides that the changes looking good, however I have a linguistic suggestion, in the JIRA you have also mentione finalization-status, in the code we use finalizeStatus. I would go with finalizationStatus as the term on the CLI, and in the code as well, as we use the word finalization in other parts of the code. If you are up to changing this, I would be glad, though this is probably just a preference on my side.

Sure, I'll use finalizationStatus to rename the CLI. (At first I checked the finalizeupgrade CLI, so I picked that name.)

Thank you!

Hi @guihecheng,
thank you for your dedication on working through this. I might misunderstand some parts of the suggestion. My false assumption was that the idea is to direct the query to a different method on the server side only, I did not expected a whole new proto request-response pair.
My first impression was that this might be an overkill, thinking it through, it is cleaner to have a new request-response and handle it, than having a flag in a message, so I am fine with this approach as well, though I leave it up for others to decide if adding a new request-response into the proto layer is something we can do anytime, as I am not sure if it has any negative consequences to have many message pair for similar purposes. If it does not, and no one raises a concern, then the patch looks good to me, after the junit failure is fixed. I left a comment on the problematic line.

@fapifta thanks for the detailed reviews, I also think that it is a cleaner way to have a separate proto pair, but a bit complex indeed.
Maybe @avijayanhwx could give some suggestions ? And I'll fix the problems in next push :)

@guihecheng / @fapifta There are 3 status requests that we are trying to support here.

Client who started finalization asks for status and messages
Client who did not start finalization asks for status & messages with takeover = true
Client just wants to know the general finalization status, irrespective of whether a finalization was started / completed or in progress.

I believe a single proto request with multiple arguments (or better a 'FinalizeUpgradeProgressRequestArgs' parameter in the proto request) is sufficient to handle the above cases. The UpgradeFinalizer interface can have a new getStatus API that just returns the status of finalization, along with the layout version (MLV, SLV).  Based on the args passed in, the request handler can call the correct API of the finalizer.
Maybe something like this?

message FinalizeUpgradeProgressRequestArgs {
optional string upgradeClientId = 1;
optional bool takeover = 2;
optional bool readOnly = 3;
}

@guihecheng / @fapifta There are 3 status requests that we are trying to support here.

Client who started finalization asks for status and messages
Client who did not start finalization asks for status & messages with takeover = true
Client just wants to know the general finalization status, irrespective of whether a finalization was started / completed or in progress.

I believe a single proto request with multiple arguments (or better a 'FinalizeUpgradeProgressRequestArgs' parameter in the proto request) is sufficient to handle the above cases. The UpgradeFinalizer interface can have a new getStatus API that just returns the status of finalization, along with the layout version (MLV, SLV). Based on the args passed in, the request handler can call the correct API of the finalizer.
Maybe something like this?

message FinalizeUpgradeProgressRequestArgs {
optional string upgradeClientId = 1;
optional bool takeover = 2;
optional bool readOnly = 3;
}

So, there is an agreement that we shall not have new proto request & response pairs.
Instead, we add a flag into the existing request and change the "required" upgradeClientId to "optional".
And we shall put a new API "getStatus" only in the UpgradeFinalizer, with SLV & MLV (may be in a new class like StatusAndLayoutVersions).
Got it, thanks @avijayanhwx @fapifta

I think both solution can be fine, but what we are essentially talking about is the following:

having one method with a couple of boolean flags as parameters, then some if conditions inside the method
or having two different methods that serves their own purpose.

In this case this is a bit more complicated though, as we need to define proto messages and types in order to have different methods that serves their own purposes, so we have a lot of boilerplate here and there and with that the line of code overhead is way more bigger then just changing one proto message and add a few boolean flags, still I think it is cleaner, and I like it more.
I don't have a strong opinion though, and I can accept the more flags version as well, if the internal API remains clear, and based on the flags we map to the desired methods that are living only in the server side.
Even though, I don't have a strong opinion, we create precedent here, and my heart pulls me to the having different methods version, unless it has drawback I am ignorant of in the proto layer or regarding compatibility/perf or any other way more important stuff than code structure.

After CI is green, +1 from my side as well.

@guihecheng Can we check if the CI failure is related?

@avijayanhwx checked the failed case, but did not spot any related failures, it seems a compose suite ozonesecure-mr is unable to exit saftemode, Shall we try retrigger the CI ?

@guihecheng Yes, please retrigger CI.

@guihecheng Yes, please retrigger CI.

@avijayanhwx @fapifta updated, thanks~

Thank you for working on this patiently @guihecheng, and for the reviews @fapifta & @errose28. Merging this PR.


Different Taghelpers purpose

@SteveSandersonMS  can you give some idea or refer me some Api docs where the SpaServices taghelpers are briefly described. TIA :)

The only relevant ones are asp-prerender-module and asp-prerender-data, which are described at https://github.com/aspnet/JavaScriptServices/tree/dev/src/Microsoft.AspNetCore.SpaServices#server-side-prerendering


WarningLoggintrait call to drupal_error_handler cause too few arguments  fatal error

Hello there,
I use https://github.com/HDDen/Webp-Drupal-7 for webp image conversion  in a drupal site, conversion works fine though every time drupal catch an error it goes through vendor/rosell-dk/webp-convert/src/Convert/Converters/BaseTraits/WarningLoggerTrait.php and at some point
return call_user_func($this->previousErrorHandler, $errno, $errstr, $errfile, $errline); use drupal_error_handler which expect 5 arguments but this call_user_func only sends 4 which cause a too few arguments fatal error, any leads on how to prevent this?
for reference :
_drupal_error_handler_real($error_level, $message, $filename, $line, $context)

The $context parameter has been deprecated. Excerpt from PHP docs:
Warning: This parameter has been DEPRECATED as of PHP 7.2.0, and REMOVED as of PHP 8.0.0. If your function defines this parameter without a default, an error of "too few arguments" will be raised when it is called.
What drupal does it exactly that: It defines an error handler with that deprecated parameter, without supplying a default value. The drupal error handler should provide a default value for the parameter.
Being that the parameter is deprecated, it seems it would be a bad idea to implement it. However, it would be preferable if webp-convert can avoid these kind of problems. And well, actually it can. There is no such thing as supplying too many parameters to PHP functions, so simply supplying the extra parameter should solve the problem :)


Manually override IDF_VER value

Now adds the version.txt file to IDF_PATH with the value of the rev argument, so that the IDF_VER macro gets a value. It doesn't perferctly replicate the expected behaviour, since I don't feel like simulating git describe perfectly, but the behaviour is at least correct when rev is set to a tag, like v5.3.
This would partly fix https://github.com/mirrexagon/nixpkgs-esp-dev/issues/5. It of course doesn't fix any warnings generated during the build process due to the missing .git.

Thanks!


Revert "git please work"

Remove injecting include paths for no alloc builds

Motivation

Future Work

Current dependencies on/for this PR:

main

PR #392

PR #393

PR #394

PR #395   👈

This comment was auto-generated by Graphite.


tensorflow_addons tutorial

System information

macOS 11.5.1 (Big Sur) Macbook Air M1 2020

Hi! Would you help with a tutorial for installing Tensorflow and tensorflow_addons in this system?

The newest tfa-nightly release ships with prebuilt Python 3.8 and 3.9 wheels for M1 Macs. So the next stable release should have full support. @bhack Do you think it would be good to trigger a 0.13.1 release? For 0.14.1 I guess it makes sense to wait until tensorflow-macos 2.6.0 is released.

Ok /cc @seanpmorgan what do you think?

Generally in support of 0.13.1 release on tensorflow-macos 2.5 but will hold off a day or two to see if we can settle our strategy on this:
https://github.com/tensorflow/addons/pull/2559#issuecomment-918738096

Sorry, but I still didn't quite understand how to install tensorflow and tensorflow-addons on M1 macos big sur 11+.

TensorFlow Addons is transitioning to a minimal maintenance and release mode. New features will not be added to this repository. For more information, please see our public messaging on this decision:
TensorFlow Addons Wind Down
Please consider sending feature requests / contributions to other repositories in the TF community with a similar charters to TFA:
Keras
Keras-CV
Keras-NLP


How to stringify a record?

Hello, I'm trying to rewrite my old project in reason. And I feel great. However, when I need to print some records for debugging, I didn't find any function work. May I ask is there any functions that can convert any record to string? For example:
type person = {
  age: int,
  name: string,
};

let someone = {age: 9, name: "jerry"};

// convert a record to string
let someone_str = record_to_string(someone); // "{age: 9, name: \"jerry\"}"

Thank you! :D

Currently there is no simple way. The best option is to use bs-json to convert it to json.

@cullophid Thank you! I'll try. 😄


Add suppport for Location IQ

http://locationiq.org/ is a new service which offers a generous 30k limit per day on their free plan.
Seems a good provider to support.

Hi, we welcome you to add a pull-request for this :)
Please look at this PR as an example: https://github.com/alexreisner/geocoder/pull/997

Thanks for adding us to the list, @tinderfields ! We don't have any ruby devs on our team to add this PR, though. @TrangPham ours is a much faster version of the OSM Nominatim API, updated daily. The API specs are exactly the same, so perhaps you could just clone the providers?

@gopi-ar That's good info for anyone who wants to give it a try. :)
It might even make sense to subclass the Nominatim provider, the same way as e.g. Dstk subclasses Google: lookups/dstk.rb
If you want to give it a try, Geocoder is in my opinion one of those libraries that should be easy to approach even for a first-time rubyist. ;) And the maintainers are really helpful in guiding you through any quirks you encounter in your PRs.

See also, as a guide: c278f94cb096215283bb35e52901159cb1e311ac

@lime @alexreisner our intern @aleemuddin13 submitted a PR integrating with LocationIQ. He had a lot of difficulty writing tests (because OSM doesn't require an API key and we do), would appreciate any help from you folks to help finish the PR.

Any chance we could get a release cut to include the new LocationIQ lookup? I've been using the master version for a couple weeks now and it's been working great with LocationIQ for my site.

Resolved by #1088


fix(sdk-clients): use http client builder to avoid leaks

Issue #, if available:
Description of changes:
Using .httpClient will create a non-managed SDK HTTP client which means that when the SDK client is closed the HTTP client will NOT close. This is absolutely not what we want in these cases because the SDK client is used and then immediately thrown away. This in turn, causes us to leak HTTP connection managers which are kept alive by the idle connection reaper which will never remove them from tracking because they're never closed.
Using .httpClientBuilder instead, will create a managed HTTP client which is properly closed at the same time the SDK client is closed. We are doubtless making this same mistake in the Nucleus and in our other plugins leading to increased memory usage and ultimately leaks.
Also disabling the idle connection reaper for these because we aren't using pooled HTTP clients, so there's no reason to keep the idle connection alive.
Why is this change necessary:
How was this change tested:
Verified by stepping through with a debugger, that closing the SDK client will close the HTTP client when using httpClientBuilder. @Nelsonochoam verified in his own testing that the heap stays small when using these changes.
Any additional information or context required to review the change:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Add details on how you tested this?

Updated description.

For reference I ran a few manual test
This is before this changes I was creating connections every second and over time the go up.


tsort: show error if directory is specified

GNU tsort shows an error if a directory is specified and returns 1 as exit code:
$ tsort dir
tsort: dir: read error: Is a directory
$ echo $?
1

uutils tsort, on the other hand, doesn't fail:
$ cargo run tsort dir
$ echo $?
0

I cannot replicate this.
$ which tsort
/usr/bin/tsort
$ tsort dir
$ echo $?
0

I cannot replicate this.
$ which tsort
/usr/bin/tsort
$ tsort dir
$ echo $?
0

I can replicate this.
$ tsort $HOME/Downloads
tsort: Downloads: read error: Is a directory

Version
$ tsort --version
tsort (GNU coreutils) 9.4
Copyright (C) 2023 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <https://gnu.org/licenses/gpl.html>.
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

Written by Mark Kettenis.

This seems like a simple fix I can create a pr, path.is_dir() check and early return should fix and give the expected behavior.


[ZEPPELIN-1700] Update docs for ZEPPELIN_INTERPRETER_CONNECT_TIMEOUT

What is this PR for?
Update docs to let devs know about ZEPPELIN_INTERPRETER_CONNECT_TIMEOUT
Some use cases are such that this value would need to be quite a bit higher, e.g., exceed the time required to create the spark executors in a mesos cluster. Developers may not know about this var until after they see the error [ZEPPELIN-1700].
What type of PR is it?
Documentation
Todos

[ ] - Review and merge

What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-1700
How should this be tested?
Review that the documented values reflect code reality https://github.com/apache/zeppelin/blob/5bb38c89ae67f95858547f73d0e833ef91b3d6ee/zeppelin-zengine/src/main/java/org/apache/zeppelin/conf/ZeppelinConfiguration.java#L576
Screenshots (if appropriate)
Questions:

Does the licenses files need update? No.
Is there breaking changes for older versions? No.
Does this needs documentation? Yes; it is documentation.

LGTM


Update howto-authentication-passwordless-deployment.md

FIXING MINOR SPELL!NG MISTAKES!11!11!

@menewol : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Looks good; verified. Thanks @menewol.


feat(FX-3302): Move custom price filter to top and default expand

The type of this PR is: Feature

This PR resolves FX-3302
Description

[x] Move the custom price filter to the top of the list of radio buttons
[x] Ensure that custom price is default expanded so that user can input range right away
[x] Remove the “All” radio button so that a blank custom range implies all by default

Demo
iOS
https://user-images.githubusercontent.com/3513494/133295891-20adebad-3d2e-4be6-8f37-de0972370cf6.mp4

PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have tested my changes on iOS and Android.
[x] I have added tests for my changes, or my changes don't require testing, or I have included a link to a separate Jira ticket covering the tests.
[x] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a changelog entry below or my changes do not require one.

Changelog updates
Changelog updates

Cross-platform user-facing changes

Move custom price filter to top and default expand - dimatretyak

iOS user-facing changes

Android user-facing changes

Dev changes

This PR contains the following changes:

Cross-platform user-facing changes (Move custom price filter to top and default expand - dimatretyak)

  Generated by :no_entry_sign: dangerJS against eff5f87fcf91fde97f7b884d7ee29ba60284a416


error to enable authentication weave-scope-app

What you expected to happen?
What happened?

Hi guys, I am trying to activate the basic authentication, but it is not working, after login/password I see on screen the following message.
"nothing to show. This can have any of these reasons:
. We haven't received any reports from probes recently. Are the probes properly connected?
. Containers view only: you're not running DOcker, or you don't have any containers
but if I remove the basicAuth settings, it works perfectly, but unfortunately without authentication
How to reproduce it?

curl "https://cloud.weave.works/k8s/scope.yaml?k8s-version=$(kubectl version | base64 | tr -d '\n')" > /tmp/scope.yml
kubectl apply -f /tmp/scope.yml
Ref.: https://www.weave.works/docs/scope/latest/installing/#k8s

I am using the following configuration in the /tmp/scope.yml file:
`
scope.yml
    metadata:
      labels:
        name: weave-scope-app
        app: weave-scope
        weave-cloud-component: scope
        weave-scope-component: app
    spec:
      containers:
        - name: app
          args:
            - '--mode=app'
            - '-app.basicAuth'
            - '-app.basicAuth.password=test'
            - '-app.basicAuth.username=test'
            - '-probe.basicAuth'
            - '-probe.basicAuth.password=test'
            - '-probe.basicAuth.username=test'

`
Anything else we need to know?

Versions:

$ kubectl version
Client Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.1",  GitTreeState:"clean", BuildDate:"2019-04-08T17:11:31Z", GoVersion:"go1.12.1", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.11", , GitTreeState:"clean", BuildDate:"2018-11-26T14:25:46Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}

Logs:
$ kubectl get pods -n weave
NAME                                         READY   STATUS    RESTARTS   AGE
weave-scope-agent-5dgqt                      1/1     Running   0          2h
weave-scope-agent-dfnjh                      1/1     Running   0          2h
weave-scope-agent-pbrwt                      1/1     Running   0          2h
weave-scope-agent-pjwnx                      1/1     Running   0          2h
weave-scope-agent-srk8t                      1/1     Running   0          2h
weave-scope-agent-xfk4r                      1/1     Running   0          2h
weave-scope-app-7976f4b755-h85lx             1/1     Running   0          26m
weave-scope-cluster-agent-554cf5ff98-ft28d   1/1     Running   0          30m

$ kubectl logs -f -n weave weave-scope-app-7976f4b755-h85lx
<app> INFO: 2019/05/15 18:49:59.050952 app starting, version 1.10.2, ID 5f5362ca4d6fa412
<app> INFO: 2019/05/15 18:49:59.051064 command line args: --app.basicAuth=true --app.basicAuth.password=teste --app.basicAuth.username=teste --mode=app --probe.basicAuth=true --probe.basicAuth.password=teste --probe.basicAuth.username=teste --probe.no-controls=true
<app> INFO: 2019/05/15 18:49:59.055372 Basic authentication enabled
<app> INFO: 2019/05/15 18:49:59.055634 listening on :4040
<app> WARN: 2019/05/15 18:49:59.086436 Error updating weaveDNS, backing off 20s: Error running weave ps: exit status 1: "Link not found\n". If you are not running Weave Net, you may wish to suppress this warning by launching scope with the `--weave=false` option.
<app> WARN: 2019/05/15 18:50:19.100276 Error updating weaveDNS, backing off 40s: Error running weave ps: exit status 1: "Link not found\n". If you are not running Weave Net, you may wish to suppress this warning by launching scope with the `--weave=false` option.

Hi @gibati,
You have provided wrong configuration in the yaml file. You have provided Basic Auth flags only in app container spec. Basic Auth flags should be provided with both app and probe spec.

"nothing to show. This can have any of these reasons:
. We haven't received any reports from probes recently. Are the probes properly connected?
. Containers view only: you're not running DOcker, or you don't have any containers

Above error is occurring because, you did not specify the basic auth flags in the probe container spec. I was able to reproduce this scenario.
You can refer the following sample yaml. This works for me.

apiVersion: v1
kind: List
items:
  - apiVersion: v1
    kind: Namespace
    metadata:
      name: weave
      annotations:
        cloud.weave.works/version: v1.0.0-27-g45f6e64
  - apiVersion: v1
    kind: ServiceAccount
    metadata:
      name: weave-scope
      annotations:
        cloud.weave.works/launcher-info: |-
          {
            "original-request": {
              "url": "/k8s/v1.8/scope.yaml",
              "date": "Fri Feb 09 2018 08:45:31 GMT+0000 (UTC)"
            },
            "email-address"<EMAIL_ADDRESS>          }
      labels:
        name: weave-scope
      namespace: weave
  - apiVersion: rbac.authorization.k8s.io/v1beta1
    kind: ClusterRole
    metadata:
      name: weave-scope
      annotations:
        cloud.weave.works/launcher-info: |-
          {
            "original-request": {
              "url": "/k8s/v1.8/scope.yaml",
              "date": "Fri Feb 09 2018 08:45:31 GMT+0000 (UTC)"
            },
            "email-address"<EMAIL_ADDRESS>          }
      labels:
        name: weave-scope
      namespace: weave
    rules:
      - apiGroups:
          - '*'
        resources:
          - '*'
        verbs:
          - '*'
      - nonResourceURLs:
          - '*'
        verbs:
          - '*'
  - apiVersion: rbac.authorization.k8s.io/v1beta1
    kind: ClusterRoleBinding
    metadata:
      name: weave-scope
      annotations:
        cloud.weave.works/launcher-info: |-
          {
            "original-request": {
              "url": "/k8s/v1.8/scope.yaml",
              "date": "Fri Feb 09 2018 08:45:31 GMT+0000 (UTC)"
            },
            "email-address"<EMAIL_ADDRESS>          }
      labels:
        name: weave-scope
      namespace: weave
    roleRef:
      kind: ClusterRole
      name: weave-scope
      apiGroup: rbac.authorization.k8s.io
    subjects:
      - kind: ServiceAccount
        name: weave-scope
        namespace: weave
  - apiVersion: apps/v1beta1
    kind: Deployment
    metadata:
      name: weave-scope-app
      annotations:
        cloud.weave.works/launcher-info: |-
          {
            "original-request": {
              "url": "/k8s/v1.8/scope.yaml",
              "date": "Fri Feb 09 2018 08:45:31 GMT+0000 (UTC)"
            },
            "email-address"<EMAIL_ADDRESS>          }
      labels:
        name: weave-scope-app
        app: weave-scope
        weave-cloud-component: scope
        weave-scope-component: app
      namespace: weave
    spec:
      replicas: 1
      revisionHistoryLimit: 2
      template:
        metadata:
          labels:
            name: weave-scope-app
            app: weave-scope
            weave-cloud-component: scope
            weave-scope-component: app
        spec:
          containers:
            - name: app
              args:
                - '--no-probe'
                - '--app.basicAuth=true'
                - '--app.basicAuth.password=test'
                - '--app.basicAuth.username=test'

              image: 'satyamz/scope:jobv1'
              imagePullPolicy: Always
              ports:
                - containerPort: 4040
                  protocol: TCP
  - apiVersion: v1
    kind: Service
    metadata:
      name: weave-scope-app
      annotations:
        cloud.weave.works/launcher-info: |-
          {
            "original-request": {
              "url": "/k8s/v1.8/scope.yaml",
              "date": "Fri Feb 09 2018 08:45:31 GMT+0000 (UTC)"
            },
            "email-address"<EMAIL_ADDRESS>          }
      labels:
        name: weave-scope-app
        app: weave-scope
        weave-cloud-component: scope
        weave-scope-component: app
      namespace: weave
    spec:
      type: NodePort
      ports:
        - name: app
          port: 4040
          protocol: TCP
          nodePort: 32160
      selector:
        name: weave-scope-app
        app: weave-scope
        weave-cloud-component: scope
        weave-scope-component: app
  - apiVersion: extensions/v1beta1
    kind: DaemonSet
    metadata:
      name: weave-scope-agent
      annotations:
        cloud.weave.works/launcher-info: |-
          {
            "original-request": {
              "url": "/k8s/v1.8/scope.yaml",
              "date": "Fri Feb 09 2018 08:45:31 GMT+0000 (UTC)"
            },
            "email-address"<EMAIL_ADDRESS>          }
      labels:
        name: weave-scope-agent
        app: weave-scope
        weave-cloud-component: scope
        weave-scope-component: agent
      namespace: weave
    spec:
      template:
        metadata:
          labels:
            name: weave-scope-agent
            app: weave-scope
            weave-cloud-component: scope
            weave-scope-component: agent
        spec:
          containers:
            - name: scope-agent
              args:
                - '--no-app'
                - '--probe.docker.bridge=docker0'
                - '--probe.docker=true'
                - '--probe.kubernetes=true'
                - '--probe.basicAuth=true'
                - '--probe.basicAuth.password=test'
                - '--probe.basicAuth.username=test'
                - 'weave-scope-app.weave.svc.cluster.local:4040'
              image: 'satyamz/scope:jobv1'
              imagePullPolicy: Always
              securityContext:
                privileged: true
              volumeMounts:
                - name: docker-socket
                  mountPath: /var/run/docker.sock
                - name: scope-plugins
                  mountPath: /var/run/scope/plugins
                - name: sys-kernel-debug
                  mountPath: /sys/kernel/debug
          dnsPolicy: ClusterFirstWithHostNet
          hostNetwork: true
          hostPID: true
          serviceAccountName: weave-scope
          tolerations:
            - effect: NoSchedule
              operator: Exists
          volumes:
            - name: docker-socket
              hostPath:
                path: /var/run/docker.sock
            - name: scope-plugins
              hostPath:
                path: /var/run/scope/plugins
            - name: sys-kernel-debug
              hostPath:
                path: /sys/kernel/debug
      updateStrategy:
        type: RollingUpdate

Closing this issue as this not an actual issue but a deployment mistake.
Thanks @gibati for trying out Scope.

Hi @satyamz its really works, thanks for fix my conf.
see you on next1.

You are welcome @gibati !


Refactor tempto to use com.google.inject

This change:

updates airbase to 140 with Guice 6.0.0
switches injection annotation to be Guice-based
adds minimal support for jakarta.inject annotations

cc @electrum


keypress Event

any1 plz tellme how to capture keypress event on grid edit.

Something is wrong with the solution from https://github.com/tabalinas/jsgrid/issues/76?

thank you very much, its working.


Added convenient method to check if running in any of the simulator

Added test

Nice! Thanks so much!


MIWI - Explicitly load the BoundSASigningKey asset and add it to the install graph

Which issue this PR addresses:
Fixes ARO-4374
What this PR does / why we need it:

[X] Loads the BoundSASigningKey asset and adds it to the install graph if present.

This is an adaptation of #173 to utilize the upstream asset's loading directly, rather than customizing it with an ARO-specific implementation. This implementation will need to be paired with a corresponding change on the ARO RP (https://github.com/Azure/ARO-RP/pull/3877) to provide the BoundSASigningKey as a secret to Hive, in order for Hive to propagate the secret to the expected path in the installer.

Test plan for issue:

None. We'll have to test this change manually end-to-end (RP -> Hive -> Installer) to ensure it behaves as expected.

Is there any documentation that needs to be updated for this PR?
No
How do you know this will function as expected in production?

Functionality will be tested with overall MIWI efforts

/cherrypick release-4.15 release-4.14


Add ExtensionMeta traits

Supersedes #912
Create new marker traits ExtensionMeta as defined below:
pub enum PromotionStatus {
   None,
   PromotedToCore(u32),
   PromotedToExtension(&'static core::ffi::CStr),
}

pub trait ExtensionMeta {
   const NAME: &'static core::ffi::CStr;
   const SPEC_VERSION: u32;
   const PROMOTION_STATUS: PromotionStatus;

   type Device;
   fn new_device(instance: &crate::Instance, device: &crate::Device) -> Self::Device;
   type Instance;
   fn new_instance(entry: &crate::Entry, instance: &crate::Instance) -> Self::Instance;
}

Add marker struct Meta in all extension mod root. ExtensionMeta trait is automatically implemented for all Meta marker structs.
When an extension does not have Instance or Device fp, the corresponding associated types on the trait will be defined as the unit type ().
Why is this useful?
Higher level abstractions may want to offer a way for users to enable extensions during application startup. Instead of asking for a &'static CStr extension name, we could instead ask for an impl ExtensionMeta which is significantly more idiomatic. This also gives us the promotion status of an extension, which allows the framework to behave differently depending on the core Vulkan version.
// During application startup
app.enable_extension<ash::khr::acceleration_structure::Meta>();

Unfortunately, we cannot implement traits for modules. That's why we need the extra Meta type that we added on module roots. These types give us a way to refer to the whole extension as a type.

One additional thing we can do: Exposing the meta types directly on ash::khr::*. So instead of ash::khr::acceleration_structure::Meta you can write ash::khr::AccelerationStructure. And as long as the ExtensionMeta trait is in scope, you can also write ash::khr::AccelerationStructure::Device

@Ralith  @MarijnS95  any changes for this to be adopted?


Warning when dropping keys for saftey in dev mode

Describe the feature
Hey! Based on https://github.com/nuxt/framework/issues/9468, we could give out warnings in dev mode when keys are dropped (constructor/__proto__).
Would that make sense?
Additional information

[X] Would you be willing to help implement this feature?

We could do this only by using conditional exports (since destr cannot depend on process.env) and if people not use proper export conditions, they will get warning also in production.
I'm fine with trying to always show a warning when detecting potential issues if too verbse can revert.


Question: custom MediaControl and custom Playback communication

What I want to do
add a button to MyMediaControl which should trigger a method in MyHtml5Video
Current Way
right now, inside MyMedieControl I can do this by
this.container.playback.myPlaybackMethod()

Question
is it the best practice way ?
should I also add a custom MyPlayer which builds an interface in between instead ?
or should I actually add a listenTo (and trigger) mechanism to MyHtml5Video and MyPlayer
thank you in advance !

@3dd13 you could provide your own media control.
MyMC extends { get overrideSomething(){return 1} }
new Clappr.Player({external: {mediaControl: MyMC}})

Or you could do a plugin like Level selector which includes itself within the existent media control.
@clappr/core We need to provide examples about how to write your own media control.. anyway, do you agree with my advices above?

@3dd13, your call to this.container.playback.myPlaybackMethod() is correct.
In Clappr, data flow is defined this way: components that broadcast data or inform conditions to unknown receivers should trigger events, and components that must interact with other elements within their own scope should call methods. Calling methods for known references to other components is the recommended way.
About the media control, you don't really need to implement a player instance, unless you have to change the player bootstrap logic in some way. If you only need to change the media control and playbacks, you can do it like @leandromoreira mentioned.

thanks guys ~


CRM457-2183: Allow downloading of RFI files

Description of change
We were linking to the supporting evidence download path, but that only worked for documents listed as supporting evidence in the blob.
Link to relevant ticket

Looks to me like the raw file names are being put in, are we confident those will never be malicious?

@elken it's a good point. I think we have to assume they could be malicious. I can't off the top of my head see a way in which a crafted file name could do something we don't want, but that could very well be a failure of imagination on my part. I wonder if there's an exercise to be done to see if we can proof-of-concept something nasty happening with file names.

Might be worth ticketing up a spike for it, but you're probably right that if it got into the system it's safe; I'd just personally sanitise on both sides with file names.


docker configurable index/alias

docker param in spandex-secondary for index/alias name

spandex #360 SUCCESS
This pull request looks good


[BUG] Unkonwn arrays are erroneously added to the constructor, missing package reference to metadata

/cib

Branch 77-_BUG_Unkonwn_arrays_are_erroneously_added_to_the_constructor_missing_package_reference_to_metadata created!


Implement crypto_stream_xchacha20

I'm interested in non-authenticated XChaCha20, for use with SIV.
@timshadel

This issue was extracted from #5, as the remaining tasks of that issue have all been completed with Version 2.0.0.

Hey interested in this (and regular ChaCha20). My use case is pure encryption as I already have Authentication at another layer in the protocol. What would it involve to complete this?


policy/v1beta1 PodSecurityPolicy is deprecated in v1.21+, unavailable in v1.25+

We are maintaining two one for default name space pod_security_policy.yaml  and one for istio pod_security_policy-istio.yaml,so as per the warring , PodSecurityPolicy will not be available in v1.25+
Could you please help us, how to migrate
Note:- we use terraform

https://kubernetes.io/docs/tasks/configure-pod-container/migrate-from-psp/ has good information about migrating
You'll need to determine the permission level required for your pods, and ensure the namespaces containing those pods have at least that permission level. You can set that permission level by labeling the containing namespaces.

/close


[FIX] sonarlint fix: "Refactor this type not to derive from an outdat…

…ed type 'System.ApplicationException'."

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (db99088) to head (2895766).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #110   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           21        21
  Lines          368       368
  Branches        48        48
=========================================
  Hits           368       368

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Slide on Mouse Swipe/Trackpad

Hi there,
Does angular-slick handle swipe events from apple mouse or trackpad swipes?  I added swipe: true and swipeToSlide: true to my sliders and it will not handle swipe movement from mouse swipe or trackpad swipe events.
Is there a way to do this?
Thanks!

I would also like to know if this is possible.


Correct indents of imports

E.g.,
fn foo() {
use Bar;
}

=>
fn foo() {
    use Bar;
}

Best way to do this is convert rewrite_imports to take an indent and width budget, rather than the current crappy way

There doesn't seem to be such a function. There is a rewrite_use_list, but that only applies to use lists..

Yep, sorry, that was the one I meant

I played around a little big with this and I managed to fix the case where the indentation was insufficient, but I cannot seem to reduce indentation. This is difficult because the indentation is not part of the Span..
This is not a problem that is specific to imports, by the way. The following will not be rewritten by rustfmt:
impl Foo {
        fn some_input(&mut self) {
    }
}

Any ideas on how to tackle this?


Update Readme > S3 GetObjectCommand

Hello,
I would like to start by thanking you for creating this helpful library.
Context:
I use aws-sdk-client-mock to mock the use of S3 commands in https://github.com/Cour-de-cassation/juritj.
I encountered some troubles when updating @aws-sdk/client-s3 to 3.369.0 version, and I would like to suggest some updates for the Readme.
First trouble:
The aws-sdk-client-mock readme indicates that S3 GetObjectCommand can be mocked with sdkStreamMixin function located in @aws-sdk/util-stream-node package.
@aws-sdk/util-stream-node package is deprecated (see: (https://docs.aws.amazon.com/AWSJavaScriptSDK/v3/latest/modules/_aws_sdk_util_stream_node.html).
Second trouble:
When updating from @aws-sdk/util-stream-node to @aws-sdk/util-stream, the sdkStreamMixin function was not found. I had to add the package in the devDependencies of package.json.
Proposal:

replace all references of @aws-sdk/util-stream-node by @aws-sdk/util-stream
add a warning : if sdkStreamMixin can't be located, add @aws-sdk/util-stream in devDependencies

Thank you for your time, I hope this helps

Last update : @aws-sdk/util-stream has been deprecated.
We now have to use install "@smithy/util-stream": "2.0.1" instead if we want to use sdkStreamMixin


self is not defined (using react-pixi in a Gatsby project - possible SSR bug)

Description
Hi @inlet . It's me again sorry to post twice in quick succession.
I've installed react-pixi (using --force as we discussed). However now, when I run gatsby develop I get an error (see below).
I think this error is caused by server-side rendering (i.e. something to do with there not being a window object). Do you think this is the case? I don't suppose you can you suggest a work around? Thanks again.
  10 |
  11 | // Support for IE 9 - 11 which does not include Promises
> 12 | if (!self.Promise) {
     | ^
  13 |     self.Promise = Polyfill;
  14 | }
  15 |

  WebpackError: ReferenceError: self is not defined

  - polyfill.js:12
    [moyo-motivations-gatsby-theme]/[@pixi]/polyfill/dist/esm/polyfill.js:12:1

  - bootstrap:18
    moyo-motivations-gatsby-theme/webpack/bootstrap:18:1

  - bootstrap:18
    moyo-motivations-gatsby-theme/webpack/bootstrap:18:1

  - bootstrap:18
    moyo-motivations-gatsby-theme/webpack/bootstrap:18:1

  - module.js:2
    [moyo-motivations-gatsby-theme]/[@inlet]/react-pixi/module.js:2:3

  - bootstrap:18
    moyo-motivations-gatsby-theme/webpack/bootstrap:18:1

  - bootstrap:18
    moyo-motivations-gatsby-theme/webpack/bootstrap:18:1

  - bootstrap:18
    moyo-motivations-gatsby-theme/webpack/bootstrap:18:1

  - gatsby-ssr.js:10
    moyo-motivations-gatsby-theme/gatsby-ssr.js:10:16

  - bootstrap:18
    moyo-motivations-gatsby-theme/webpack/bootstrap:18:1

  - api-runner-ssr.js:19
    moyo-motivations-gatsby-theme/.cache/api-runner-ssr.js:19:15

  - bootstrap:18
    moyo-motivations-gatsby-theme/webpack/bootstrap:18:1

not finished Building development bundle - 6.732s

Steps to reproduce

Run gatsby develop on a gatsby project that imports pixi-react elements

Additional info

@inlet/react-pixi version:  6.3.0
React version:"^17.0.1",
ReactDOM version:"^17.0.1",
PIXI version: "^6.0.0"

Duplicate #233

https://www.gatsbyjs.com/docs/using-client-side-only-packages/

Thanks for the donation 🙏 you rock!