🐞 types.Map not supported at modules

What is the issue?
Currently, zenith do not support map types, resulting in compilation errors.
Log output
dagger call workflow string
✘ load call ERROR [3.73s]
├ [3.73s] loading module
┃ Error: failed to get loaded module ID: input:1: host.directory.asModule.s
┃ ve failed to load module types: failed to call module "gale" to get funct
┃ ns: failed to get function output directory: process "/usr/local/bin/code
┃ n --module . --propagate-logs=true --introspection-json-path /schema.json
┃ did not complete successfully: exit code: 1
✘ serve ERROR [0.49s]
✘ exec /usr/local/bin/codegen --module . --propagate-logs=true --introspection-json-path /schema.json ERROR [0.47s]
┃ Error: generate code: template: module:59:3: executing "module" at <Modul
┃ ainSrc>: error calling ModuleMainSrc: failed to convert field type: unsup
┃ rted type *types.Map
✘ generating go module: gale ERROR [0.15s]
├ [0.03s] go mod tidy
┃ writing dagger.gen.go
┃ writing go.mod
┃ writing go.sum
┃ needs another pass...
• Cloud URL: https://dagger.cloud/runs/fbd1131a-6339-4d26-b2d1-98e5c6d533fd
• Engine: 4c1ca39d411b (version devel ())
⧗ 3.89s ✔ 170 ∅ 8 ✘ 4

Steps to reproduce
Use map[string]string in the zenith module with golang SDK like following example
// Step holds the configuration of a step.
type Step struct {
	StepID string
	Name   string
	Env    map[string]string
	With   map[string]string
	Uses   string
}

and execute the module
SDK version
Go SDK v0.9.3
OS version
macOS 14.0

I don't think we can support this because GraphQL doesn't have maps. The only way is to have it "map" to a sequence of key/value pairs underneath. We should at least give a user friendly error here, explaining this.

In Go there's the additional issue of maps not preserving order, so that can affect caching.

I don't think we can support this because GraphQL doesn't have maps. We should at least give a user friendly error here, explaining this though.
The only way is to have it "map" to a sequence of key/value pairs underneath, like EnvVariable.

I think having the translation is the right call long term - not supporting maps is kinda frustrating, and I've definitely reimplemented the proposed solution myself several times when I've wanted kv pairs.

In Go there's the additional issue of maps not preserving order, so that can affect caching.

Potentially, we could normalize and sort input. I don't think this is a concern for go (in the engine or SDK) since marshalling order of JSON is lexographically sorted.

Potentially, we could normalize and sort input. I don't think this is a concern for go (in the engine or SDK) since marshalling maps order of JSON is lexographically sorted.

Yes, but I'm thinking about what you do with it inside the function. However you do have to power to use it in a way that doesn’t affect caching.

The only way is to have it "map" to a sequence of key/value pairs underneath, like EnvVariable.

I think having the translation is the right call long term - not supporting maps is kinda frustrating, and I've definitely reimplemented the proposed solution myself several times when I've wanted kv pairs.

To avoid having an input type for the argument and another for the output, we can basically make a general replacement for EnvVariable:
https://github.com/dagger/dagger/blob/aec81d10a55db52937a5e67e5ad8870b29fe9a35/docs/docs-graphql/schema.graphqls#L1128-L1138
That’s only for string values though. We’d need multiple versions of this to support other types.
So a function defined like this:
func (m *Test) Sorted(env map[string]string) map[string]string {
    // sort env
    return env
}

Would produce:
type Test {
    sorted(env: [KeyValueID!]!): [KeyValue!]!
}

Assuming:
scalar KeyValueID

type KeyValue {
    id: KeyValueID!
    name: String!
    value: String!
}

type Query {
    keyValue: KeyValue!
}

And in client code, used like this:
dag.Test().Sorted(
    []*KeyValue{
        dag.KeyValue("name1", "value1"),
        dag.KeyValue("name2", "value2"),
    },
)

We could also try to have codegen handle this transparently so you never have to use KeyValue directly, but sometimes order does matter and automatically sorting may not produce the intended result. I guess you’d not use a map[string]string in this case.
r, err := dag.Test.Sorted(ctx, map[string]string{
    "name1": "value1",
    "name2": "value2",
})
// r = map[name1:value1 name2:value2]

Or more simply, using a KeyValue scalar that uses the name=value format, so it could be used in both the input and output and when used in a list, translate it to a map in codegen.
Reminder that we had a similar request for using a map with a WithEnvVariables in:

https://github.com/dagger/dagger/issues/4835


Fix spelling of retrieval

Changed retrival to retrieval.

This PR fixes #14489

Attempt to login to SQL via AAD - MFA with an expired token.
Get the error message.

Pull Request Test Coverage Report for Build 612123158

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 43.155%

Totals

Change from base Build 612095270:
0.0%

Covered Lines:
25081

Relevant Lines:
53186

💛  - Coveralls


feature: ASGI compatibility

Currently, Lona uses aiohttp as a server. Aiohttp server is slower than most ASGI libraries and doesn't use a standardized interface.
What are your thoughts on eventually making this ASGI compatible, so it can run in common ASGI servers like uvicorn?
Im thinking Starlette would be a good option as a server framework, as it's used in FastAPI which is decently well established and proven to be stable and performant.

Laundmo just introduced me to this project and it looks like something I'd definitely be interested in using, but with it using aiohttp as a backend rather than having it as an ASGI compatible app I can run with Uvicorn it's hard for me to justify using it. (This is largely due to a lot of the huge number of libraries that do work with Starlette/ASGI apps, such as starlette-discord for Discord OAuth login, for example)

@laundmo, @vcokltfre sounds reasonable. Can you provide a PR that adds ASGI compatibility?

Could you tell us which are the main points where the serverside code interfaces with the library itself? in other words: what classes would we have to replace to fully change out aiophttpo as a server?

@laundmo Sure! The only parts that interface with aiohttp directly are lona/server.py and lona/command_line/run_server.py

@laundmo, @vcokltfre: Do you guys have code to look at?

@fscherf im sorry to say: no. i've been quite busy and didn't have any time.

@laundmo don't be sorry! :) I just wanted to make sure you are not stuck on something

@laundmo: this issue is open pretty long now. Can i transfer it to the ideas section?

Sure, i doubt I'll be working on it anytime soon.
On Mon, 9 May 2022, 19:43 Florian Scherf, @.***> wrote:

@laundmo https://github.com/laundmo: this issue is open pretty long
now. Can i transfer it to the ideas section?
—
Reply to this email directly, view it on GitHub
https://github.com/lona-web-org/lona/issues/176#issuecomment-1121392664,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF5ULDI7MAYK5Y64VPVWYN3VJFFERANCNFSM5HHYBUBQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Rename ReorderRecord to Take

The new name reflects what the function does, we are not ordering anything we are just taking specific rows.

I almost made a mistake to recommend duplicate implementation on ticket #672 because it was impossible to discover.


error[E0599]: no method named set_attribute found for struct Mesh in the current scope

Hey, I get following error when I try to use the latest version 762ff4de1327a9e1f5abade018824e3b1072780c. And I can't use 0.4 because of Path-Geometry error.
Reported on windows 11, rust nightly.
error[E0599]: no method named `set_attribute` found for struct `Mesh` in the current scope
   --> C:\Users\tomek\.cargo\git\checkouts\bevy_prototype_lyon-ae009892befac619\762ff4d\src\plugin.rs:133:10
    |
133 |     mesh.set_attribute(
    |          ^^^^^^^^^^^^^ help: there is an associated function with a similar name: `insert_attribute`

error[E0599]: no method named `set_attribute` found for struct `Mesh` in the current scope
   --> C:\Users\tomek\.cargo\git\checkouts\bevy_prototype_lyon-ae009892befac619\762ff4d\src\plugin.rs:141:10
    |
141 |     mesh.set_attribute(
    |          ^^^^^^^^^^^^^ help: there is an associated function with a similar name: `insert_attribute`

Of you use bevy 0.6 with the latest release of bevy_prototype_lyon you shouldn't get that error. This error is caused by a breaking change by Bevy after the release.


Adding docs to repo for publication.

Doc .md files added.
Repo synced to gitbook.
Screenshots added.
Structure validated against other plugins'.

Codecov Report

Merging #23 (4bab53c) into master (1bf0c50) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #23   +/-   ##
=======================================
  Coverage   21.63%   21.63%
=======================================
  Files          10       10
  Lines         772      772
=======================================
  Hits          167      167
  Misses        598      598
  Partials        7        7

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 1bf0c50...4bab53c. Read the comment docs.

@justinegeffen Can this PR be merged?

Yep! They can be. :)


Rundeck seems to frequently  be rebuilding its inventory from Ansible

I may have a hard time documenting my issue as my Rundeck server is not connected to the Internet and am unable to upload any output. Apologies.
Rundeck version:  <IP_ADDRESS>90927
Ansible version: 2.6.1
Python Version 2.7.5
Our Ansible inventory is fairly complex, approx 1200 hosts, with many host groups (built dynamically)  and ini files for vars definitions etc.
I have a number of projects, each project uses the same Ansible inventory file, and filters are used within Ansible-Resource-Model to limit hosts within the project.
By what I have read Rundeck keeps its own inventory that is built from the supplied Ansible inventory. When filters are used within Ansible-Resource-Model it seems that Rundeck builds its inventory very inefficiently.
As an example say I have 1000 hosts, with a naming convention of A001 > A100, B001 > B100, C001 > C100  all the way through to J100...  10 X 100 = 1000. If I select only the "B" hosts in the  Ansible-Resource-Model  filter (B*), When I look at the output of service.log it is rapidly spewing data and  I see the list seems to be being built by searching my entire inventory of 1000 hosts one host at a time, finding the first instance of a B host eg: B001, it then searches the 1000 hosts again and finds the next instance, B002 and appends this to its list, it then searches the 1000 hosts again, and appends B003 and so on. So to build my complete list of B hosts the inventory is searched 100 times.
This takes up a huge amount of CPU while doing this as dozens and dozens of "ansible-playbook gather-hosts.yml" processes are running. This would not be a huge deal if it only ran after editing the  Ansible-Resource-Model however every time you click in to the project nodes page it starts off again. If you are working within a number of projects then in a very short time Rundeck becomes unresponsive.
An example of a process is:
/opt/anisble/bin/python /usr/local/bin/ansible-playbook gather-hosts.yml  --inventory-file=/var/lib/rundeck/ansible/environment/prod/inventory/1_hosts/all-hosts.ini -l B*<EMAIL_ADDRESS>I have seen over 60 of these running at one time while the inventory is built.
Is there a way to stop Rundeck constantly rebuilding its inventory every time you enter a project nodes page  or click on Edit Nodes page? Ideally it would only rebuild if a nodes config file has been updated.
Is there a way to have Rundeck filter more efficiently?
Help would be appreciated... Stay safe..
Austin

Hi!,
I have the same problem. We have an inventory with 500 hosts and it is refreshing the node list everytime. This action overloads a lot the pod (in our case we are running rundeck in kubernetes) and origins other problems.
Also, when a node can't execute the ansible script to check its status (scheduled every time):
/usr/bin/python /home/<user>/.ansible-<user>/tmp/ansible-tmp-1589815139.1577857-18710571914723/AnsiballZ_setup.py
the process stucks and, when the process is going to be executed again, rundeck jobs defined into the project are not executed. This is because rundeck is waiting until the first process finishes, so it is stucked forever...
Have you discovered the problem? Do you also have this problem?
Daniel.

@dfradejas I finally made it a little better by setting the "Cache Delay" on "Edit Nodes" Configuration tab to 28800 seconds (8 hours). It can still be an issue but an improvement is better that none. Still not a very efficient way of building a node list. I would like to understand why it was done this way.

Was any way ever determined to improve this? I've set it to 86400 and it's still rebuilding the entire list whenever I click on anything even remotely related to nodes.

I have dynamic inventory with almost 1000 nodes and "Ansible Resource Model Source" is practically unusable.

Same here...

me too


GitHub repo keeps growing due to frequently modified test data

The test data in https://github.com/CWorthy-ocean/roms-tools/tree/main/roms_tools/tests/test_setup/test_data is used in the validation tests:
https://github.com/CWorthy-ocean/roms-tools/blob/10c0f6dc6e21a87f8cda9cc6acbb5c32e6766bc2/roms_tools/tests/test_setup/test_validation.py#L49-L72
and is

really useful for code refactors because they check whether the results have changed
really annoying if we change the filling or regridding method because then we have to overwrite the test data

Because 2. happens quite a lot, the frequent overwriting of the test data leads to a growing repo size. Currently, we are at
cd roms-tools
du -hs .
47M	.

of which
4.1M	roms_tools/tests/test_setup/test_data   # current test data
30M	        .git                            # due to changed git history

I really like having the validation tests for 1., but don't want the repo size to get out of hand. Thoughts?

We discussed a possible solution today of moving the regression test data into a cloud bucket. The developer experience would be basically the same (i.e. you explicitly regenerate the new regression test data when you know that a change to roms-tools will affect the output), but that regenerated data would be uploaded to a bucket instead of committed locally and uploaded to this repo.
Because we're talking about tracking versions of zarr data, this is a sensible use case for icechunk. We don't have to use icechunk - we could just upload each new zarr store to a different bucket prefix, but using icechunk would make the downloading and uploading of the zarr data into an xarray one-liner. It also might give us more powerful introspection capabilities in the future (e.g. something like git bisect would be useful - https://github.com/earth-mover/icechunk/issues/377).
Once we moved the data into a bucket we would want to clean out the history of this repo.


Extract associated-token-address crate from associated-token-account crate

Problem
People often import spl-associated-token-account just to derive an associated token address. This makes the ecosystem dependency hell and bloat problem much worse.
Changes

Make a new spl-associated-token-address that contains just the get_associated_token_address* functions.
Re-export these functions in spl-associated-token-account with a deprecation notice

This branches off #6989 so that should be merged first

@kevinheavey looks like this one is also going to be closed, as per the solution proposed in #6989? Fwiw, we are close to said clients.

This basically is the solution proposed in #6989 no?

Oops, I don't think I was following the bouncing ball over there.
This is looking pretty good, but I believe @joncinque had some additional thoughts so I'll let him weigh in on this one.

@kevinheavey I worked this a bit to be a more generic client crate that exposes the instruction creators and the address derivation functions. This way, we remove a dependency from token-client to the ata program, which means that I don't have to bump and publish the ata program every time there's a new version of token-2022, which always felt wrong.
Let me know what you think!

@joncinque lgtm, full send


Soft deleted Integration key can not be added again

Seems like there is a bug here caused by the fact that we have added SoftDelete to the integrations model but not updated the unique constraint. So, since there are entities in the SegmentConfiguration model table still (although they've been deleted), it is not possible to add any new ones. We need to update the unique constraint to only check for entities that do not have a value for deleted_at.

@matthewelwell can we close this?


api calls

for #17
Tested each route with postman and things seem to be good. Only caveat is that the response after an edit operation is the old document and not the one with changes applied (at least on postman) so to get the updated document after an edit, you need to GET the document again.
some comments:

Changed the pinnedPosts type to String for id, I was getting an error about ObjectID not being defined (which I think is because it should be mongoose.Types.ObjectID), but I think String is easier to work with and I'm not sure if there's an added benefit to using ObjectID (if there is let me know and I'll change things to work with it!)
There might be some overlap with the user API calls between Rodrigo's PR, will have to discuss and see how we want to structure the endpoint methods
Created a routes/ folder for storing all the different router files, so maybe we can get rid of api.js if it's not being used? I think putting all the APIs inside api.js will make it too big + conflicts might arise when working on APIs so maybe we can get rid of it.
For posts I limited it to just changing the tags and content, maybe we can also add methods for editing the title/type/publicity? Would be very easy to do.

some more comments:

should we add implementations for post limiting, sorting (by timestamp), and randomizing?
do we need some way of validating unique posts, or should we trust the user to not submit the exact same thing multiple times?
should we clean everything from the database and readd things just to be sure we don't have extras/test documents in there?

to get all posts:
GET api/posts/
nothing in req body or query needed
returns an array of Post objects
to get a post by id:
GET api/posts/:id
nothing in req body or query needed
returns a Post object
to add a post:
POST api/posts/
with a Post object in the request body
will return the id of the new Post
to edit a post:
POST api/posts/:id
with the fields you want to change in the request body
eg. want to change post tags and make post private
request body would be:
{
"tags": [
"dashboard",
"amy-lei",
"testing",
"testing2"
],
"isPublic": false
}
will return a post object (should be the new document but currently bugged, if you GET the document the returned document has the new values)
to create a post object from a youtube/github URL:
POST api/posts
with the request body containing an object with url and creator fields
will return a post object that the user can verify
to add it to the database, make another POST api/posts/ with the Post object

to get a user:
GET api/users/:username
nothing in req body or query
will return a User object -> can get pins from User object
to update a users pins:
POST api/users/:username/pins
with a new pinnedPosts array in the request body
will return the User object (again, should be an updated user object but returns the old one)
to update a user object:
POST api/sers/:username
with the fields you want to change in the request body
eg. you want to update the discord username, request body would be:
{
"discord": "NEW DISCORD NAME"
}
will return the User object


[$1000] Web – Contact Method – Dashes are not changing to red upon entering invalid magic code.

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

Action Performed:

Open your profile.
Click on "Contact methods".
Click on "New contact method".
Add new contact and then proceed to next step.
Write an invalid magic code so that error appear.
Notice dashes or input fields are not changed to red due to error.

Expected Result:
Dashes should change to red upon entering invalid magic code.
Actual Result:
Dashes are not changing to red upon entering invalid magic code.
Workaround:
Unknown
Platforms:

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

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.36-2
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/8a613c23-788f-4f21-9ec3-a5437c0ed13c
https://github.com/Expensify/App/assets/93399543/f6915264-62df-4665-ac89-20c56c8d7599
Expensify/Expensify Issue URL:
Issue reported by: @usmantariq96
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1688460879570129
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01327a7e34d0845a95
Upwork Job ID:<PHONE_NUMBER>843287552
Last Price Increase: 2023-07-17

@aimane-chnaif - how are the proposals here?

I believe I have already fixed this issue. Check below. Very simple fix.

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~0165e33e93f86ebbd4

Duplicate like #22190 :(

 Here is a proposed fix that appears to be working.

Ah, this is a dupe of https://github.com/Expensify/App/issues/20416. Thanks for pointing it out @ygshbht.
@jbeoris, @bernhardoj and @samh-nl - feel free to move these proposals over there!

Okay, if @aimane-chnaif agrees, I think we can go ahead. I'd still be intrigued to know whether we should HOLD until after https://github.com/Expensify/App/issues/18244#issuecomment-1634506185 is deployed to avoid conflicts, or if it's really negligible.
Either way, I'd be happy to assign you. If it's going to be HELD though, I think we'd prefer to hold on assigning as well, to give you the opportunity to hit the 3 day turnaround after we unhold. I would just remove the Help Wanted in that instance and we'd assign you once we were ready.

Glad to see this progressing! If it makes any difference, regarding the error, i had mentioned the solution for this {here)[https://github.com/Expensify/App/issues/22190#issuecomment-1621047883]
 in #22190

This should've been merged earlier, the only reason is I was ooo. As such, I would recommend this is eligible for the bonus @twisterdotcom

Payment summary:

@aimane-chnaif paid $1500 here: https://www.upwork.com/nx/wm/workroom/34252840/overview
@bernhardoj paid $1500 here: https://www.upwork.com/nx/wm/offer/new/32195282
@usmantariq96 paid $250 here: https://www.upwork.com/nx/wm/pre-hire/c/8577561/offer/25905224

@aimane-chnaif - Still need the Checklist completing here please.

Bump @aimane-chnaif.

This one can't be reassigned really @aimane-chnaif so please prioritise it today.


[DO NOT MERGE] Upgrade Rails

Still WIP as it needs a rebase on master, but all tests are passing locally. Needs the govuk_content_models PR to be merged, and the gem to be bumped, for tests to pass here.
https://trello.com/c/H3gARFuk/56-upgrade-publisher-to-rails-5

Can we squash 7dd8899 and b063e36 together?

We certainly can!

There are the correct number of users now!

Synk is unhappy - I think it's so far an unfixable Nokogiri gem issue.

Yep, we can proceed even if Snyk's unhappy for this.  We have more upgrades to come 😱


Opening ion-select makes parent view going blank (scroll)

Short description of the problem:
When I click on an ion-select the view goes blank for half a second, making up for an impression of the app shaking. This only happens when there's is scrolling and I was only able to reproduce it on iOS
Video
What behavior are you expecting?
The view not to go blank.
Which Ionic Version? 2.x
Plunker that shows an example of your issue
I've setup a repo that reproduces the bug : Here
Run ionic info from terminal/cmd prompt: (paste output below)
Cordova CLI: 6.1.1
Gulp version:  CLI version 3.9.1
Gulp local:   Local version 3.9.1
Ionic Framework Version: 2.0.0-beta.8
Ionic CLI Version: 2.0.0-beta.25
Ionic App Lib Version: 2.0.0-beta.15
ios-deploy version: 1.8.6
ios-sim version: 5.0.6
OS: Mac OS X El Capitan
Node Version: v5.0.0
Xcode version: Xcode 7.3.1 Build version 7D1014

Hello, thanks for using Ionic.This has since been fixed.


Use ubuntu base image

When the binary is surveysvc binary is built on travis and a docker
image based on alpine is used starting the surveysvc container fails to
start. The error suggests the binary is invalid.
standard_init_linux.go:185: exec user process caused "no such file or
directory".

If the same process is followed on a osx by running make and
docker build . -t sdcplatform/surveysvc the docker container will run
with the alpine docker base image.
When changing the docker base image to ubuntu it fixes the problem. This
is a path of least resistance fix. This impact is that the image is
slightly bigger but as we are only using it for local development it
shouldn't matter.

@benjefferies - have you checked the line endings on the Dockerfile? This is a common issue when for Alpine when a file has been edited in Windows at some point.

I checked that first in vim as it normally shows ^M for crlf and seemed ok and also ran dos2unix on it to make sure and no difference


[feat] Accordion component should support single selectable section behavior by default

Does your feature request relate to a specific USWDS feature?
This is an enhancement to the existing Accordion component https://designsystem.digital.gov/components/accordion/.  The current implementation defaults to multiselectable behavior where multiple sections can be open at once.  However in the USWDS library the default behavior is expanding one section will collapse all others and multiselectable is opt in.
Is your feature request related to a problem? Please describe.
No it is just another component feature of the current spec that someone may want.
Describe the solution you'd like
To align with the behavior of USWDS make multiselectable an attribute/prop set to false by default.
Additional context
From a Slack conversation, the original implementation was made for a project that only needed the multiselectable behavior

^^ I accidentally updated the title here and not in my PR


Handle and skip structs with "func" fields.

Describe the PR
Added code for parsing and skipping structure fields with func type.
Relation issue
fix #615
Additional context

@DexterHD Thanks for your contribution.  👍


AWS feedback

Why is this PR needed?
This PR addresses the feedback raised on this PR https://github.com/SUSE/skuba/pull/637#issuecomment-535254583 by @rssfed23.
What does this PR do?
Be more secure
All the worker nodes are now moved inside of a private subnet. The subnet was already there, we weren't simply using it (dunno why).
This design is more secure and reflects a common pattern on AWS.
TBH, we could move also the master nodes into the secure network, but then we would need to have a bastion host. This is definitely something for later, once we collect more feedback from users.
Fix IAM role creation for worker node
@rssfed23 faced the issue. Terraform wasn't creating a IAM role for the worker nodes due to a leftover inside of the state file.
Anything else a reviewer needs to know?
This is not yet going to be exposed to our customers, right now our packages don't have any AWS terraform stuff in them.
Info for QA
Nothing special to do: just ensure all the worker nodes are inside of the private subnet and the cluster can still be provisioned as expected via skuba.
Related info
Nothing.
Status BEFORE applying the patch
All the worker nodes had a public IP.
Status AFTER applying the patch
Only master nodes have a public IP. Worker nodes are reachable from nodes located inside of the VPC created by terraform.
Docs
Docs need to be updated, I'm going to file a PR. The contents about AWS are already dated.

I've fixed the linting errors. @ereslibre could you approve it again?

Just tried it out again on my machine and can confirm all the feedback I raised has been addressed and it's working awesomely and more secure than it was before - LGTM :)


Process args in "extra" key of recorded message

This PR aims to fix the problem #836 (extra key of the recorded message is completely ignored) as quickly as possible not to loose data from logs

I agree, reading from $context['extra'] should have never been implemented. If you want to "capture" additional data by extracting it from the Monolog payload besides the standard information like the channel name or the message then the best thing is to write your own handler or processor and use configureScope or withScope. Closing this as this kind of feature is something we don't want to support anymore out-of-the-box

@ste93cry but how handling $record['context']['extra'] differs from $record['extra']? Both of them are handled as an array, with no data type checking of the values. How can You be sure that $record['context']['extra'] won't contain anything "bad"? My code was also using withScope so I don't understand the reasons why this PR got closed.

Quoting from the Monolog documentation:

At first glance context and extra look very similar, and they are in the sense that they both carry arbitrary data that is related to the log message somehow. The main difference is that context can be supplied in user land (it is the 3rd parameter to Logger::addRecord()) whereas extra is internal only and can be filled by processors. The reason processors write to extra and not to context is to prevent overriding any user provided data in context.

The main problem is that there is no guarantee that the data added to this key is data that you want to log in Sentry, that the "shape" of the data is compatible with what Sentry expects or that you want to blindly override what has been set by the user with the data coming from the processors if the share the same key

Both of them are handled as an array, with no data type checking of the values. How can You be sure that $record['context']['extra'] won't contain anything "bad"?

We can't of course, and this is one of the reasons why we want to avoid in the future blindly pulling information from the log data. I would say that we can be safer in thinking that the data coming from $record['context']['extra'] is already in a format compatible with Sentry because it's supplied from the user, but honestly I wouldn't bet on it. All these use-cases that wants to pull data from the log imho are too projects-dependant.

reading from $context['extra'] should have never been implemented

To clarify, pulling data from $record['context']['tags'] and $record['context']['extra'] (which is supported out-of-the-box right now) was an error, but we cannot remove such features or we will break BC. This doesn't prevent us to not add more support for similar things to avoid the pitfalls described above :wink:

Oh, support for $record['context']['extra'] was a mistake in the first place, now I get it.
What about introducing a processor that moves $record['context'] and $record['extra'] to the scope directly in Your repo? Should You handle this in a (more or less) generic way (may not fit to every project) should I totally implement it on my side?

Should You handle this in a (more or less) generic way (may not fit to every project) should I totally implement it on my side?

For the moment yes, you should implement in on your side. This way you can decide whether to blindly move everything or only certain things to the scope

Hello everyone,
I was looking to do exactly this (sending $record['context'] to Sentry) but I totally understand the reasoning behind why this PR was closed.
That being said, I don't see how to implement this on my side ...
@prgTW How did you do it ?
Thanks!

You should use composition to wrap the handler inside your custom one and then call withScope to add the data you need to the scope. Something like this should work:
use Monolog\Handler\AbstractProcessingHandler;
use Sentry\Scope;
use function Sentry\withScope;

final class CustomMonologHandler extends AbstractProcessingHandler
{
    private $innerHandler;

    public function __construct(HandlerInterface $handler)
    {
        $this->innerHandler = $handler;
    }

    public function handle(array $record)
    {
        withScope(function (Scope $scope): void {
            $scope->setExtra('...', '...');

            $this->innerHandler->handle($record);
        });
    }
}

I coded this example on the fly so don't expect it to work as is, but it should give you an hint on how I would do it

Thanks @ste93cry ! That makes sense 👍

Could also easily be handled in Monolog :)
$log->pushProcessor(function ($record) {
            if (isset($record['context']) && is_array($record['context'])) {
                foreach ($record['context'] as $key => $value) {
                    if ($key === 'tags') {
                        //Handled natively by Sentry monolog handler
                        continue;
                    }
                    $record['context']['extra'][$key] = is_string($value) && json_decode($value) !== null ? json_decode($value) : $value;
                }
            }

            return $record;
        });

To add extra information along the simple textual message, I also use the logging context to pass an array of data along the record. For example:
$logger->info('Adding a new user', array('username' => 'Seldaek'));

It was really important for me to get the context in Sentry issues.
Thank you very much @ste93cry and @Spriz. I have created a custom handler with a mix of your two answers. Tested successfully with Symfony 3.4. I decorate the Sentry\Monolog\Handler service.
config.yml:
monolog:
    handlers:
        sentry:
            type: service
            id: Acme\Monolog\SentryHandler

services.yml:
services:
    Acme\Monolog\SentryHandler:
        decorates: Sentry\Monolog\Handler
        arguments<EMAIL_ADDRESS>
src/Acme/Monolog/SentryHandler.php:
<?php

namespace Acme\Monolog;

use Monolog\Handler\AbstractProcessingHandler;
use Monolog\Logger;
use Sentry\Monolog\Handler as InnerHandler;
use Sentry\Severity;
use Sentry\State\HubInterface;
use Sentry\State\Scope;
use function Sentry\withScope;

/**
 * This Monolog handler logs every message to a Sentry's server using the given
 * hub instance.
 *
 * If extra information are added to the simple textual message (through an array
 * of data along the record), this custom Sentry handler iterate the array to set
 * data in the extra Sentry context.
 *
 * @author Stefano Arlandini<EMAIL_ADDRESS> */
final class SentryHandler extends AbstractProcessingHandler
{
    /**
     * @var HubInterface
     */
    private $hub;

    /**
     * @var InnerHandler
     */
    private $handler;

    /**
     * SentryHandler constructor.
     *
     * @param InnerHandler $handler
     * @param HubInterface $hub    The hub to which errors are reported
     * @param int          $level  The minimum logging level at which this
     *                             handler will be triggered
     * @param bool         $bubble Whether the messages that are handled can
     *                             bubble up the stack or not
     */
    public function __construct(InnerHandler $handler, HubInterface $hub, $level = Logger::DEBUG, bool $bubble = true)
    {
        $this->handler = $handler;
        $this->hub = $hub;

        parent::__construct($level, $bubble);
    }

    public function handle(array $record)
    {
        withScope(function (Scope $scope) use ($record): void {
            if (isset($record['context']) && is_array($record['context'])) {
                foreach ($record['context'] as $key => $value) {
                    if ($key === 'tags') {
                        // Handled natively by Sentry monolog handler
                        continue;
                    }
                    $scope->setExtra($key, is_string($value) && json_decode($value) !== null ? json_decode($value) : $value);
                }
            }

            $this->handler->handle($record);
        });
    }

    /**
     * {@inheritdoc}
     */
    protected function write(array $record): void
    {
        $payload = [
            'level' => $this->getSeverityFromLevel($record['level']),
            'message' => $record['message'],
            'logger' => 'monolog.' . $record['channel'],
        ];

        if (isset($record['context']['exception']) && $record['context']['exception'] instanceof \Throwable) {
            $payload['exception'] = $record['context']['exception'];
        }

        $this->hub->withScope(function (Scope $scope) use ($record, $payload): void {
            $scope->setExtra('monolog.channel', $record['channel']);
            $scope->setExtra('monolog.level', $record['level_name']);

            if (isset($record['context']['extra']) && \is_array($record['context']['extra'])) {
                foreach ($record['context']['extra'] as $key => $value) {
                    $scope->setExtra((string) $key, $value);
                }
            }

            if (isset($record['context']['tags']) && \is_array($record['context']['tags'])) {
                foreach ($record['context']['tags'] as $key => $value) {
                    $scope->setTag($key, $value);
                }
            }

            $this->hub->captureEvent($payload);
        });
    }

    /**
     * Translates the Monolog level into the Sentry severity.
     *
     * @param int $level The Monolog log level
     * @return Severity
     */
    private function getSeverityFromLevel(int $level): Severity
    {
        switch ($level) {
            case Logger::DEBUG:
                return Severity::debug();
            case Logger::INFO:
            case Logger::NOTICE:
                return Severity::info();
            case Logger::WARNING:
                return Severity::warning();
            case Logger::ERROR:
                return Severity::error();
            case Logger::CRITICAL:
            case Logger::ALERT:
            case Logger::EMERGENCY:
                return Severity::fatal();
            default:
                return Severity::info();
        }
    }
}

The write method is abstract and protected, so I had to implement this method and I could not use the decorator $this->handler->write(). Moreover, this method also calls the private method getSeverityFromLevel. So, lot of copy and paste from the original Sentry\Monolog\Handler handler... Is there a better way?

That's not really how I would expect a decorator to be written. Isn't this sufficient?
class TestHandler implements HandlerInterface
{
    /**
     * @var HandlerInterface
     */
    private $decoratedHandler;

    public function __construct(HandlerInterface $decoratedHandler)
    {
        $this->decoratedHandler = $decoratedHandler;
    }

    public function isHandling(array $record): bool
    {
        return $this->decoratedHandler->isHandling($record);
    }

    public function handle(array $record): bool
    {
        $result = false;

        withScope(function (Scope $scope) use ($record, &$result): void {
            $scope->setExtra(...);

            $result = $this->decoratedHandler->handle($record);
        });

        return $result;
    }

    public function handleBatch(array $records): void
    {
        $this->decoratedHandler->handleBatch($records);
    }

    public function close(): void
    {
        $this->decoratedHandler->close();
    }
}

Sorry, I'm not a Symfony developer. I have tested your code, and implementing HandlerInterface is much better! With my version of Monolog I had to implement also pushProcessor, popProcessor, setFormatter and getFormatter.
So the final code, tested successfully is:
<?php

namespace Acme\Monolog;

use Monolog\Formatter\FormatterInterface;
use Monolog\Handler\HandlerInterface;
use Sentry\State\Scope;
use function Sentry\withScope;

/**
 * This Monolog handler logs every message to a Sentry's server using the given
 * hub instance.
 *
 * If extra information are added to the simple textual message (through an array
 * of data along the record), this custom Sentry handler iterate the array to set
 * data in the extra Sentry context.
 *
 * @author Stefano Arlandini<EMAIL_ADDRESS> */
class SentryHandler implements HandlerInterface
{
    /**
     * @var HandlerInterface
     */
    private $decoratedHandler;

    /**
     * SentryHandler constructor.
     *
     * @param HandlerInterface $decoratedHandler
     */
    public function __construct(HandlerInterface $decoratedHandler)
    {
        $this->decoratedHandler = $decoratedHandler;
    }

    /**
     * @inheritDoc
     */
    public function isHandling(array $record): bool
    {
        return $this->decoratedHandler->isHandling($record);
    }

    /**
     * @inheritDoc
     */
    public function handle(array $record): bool
    {
        $result = false;

        withScope(function (Scope $scope) use ($record, &$result): void {
            if (isset($record['context']) && is_array($record['context'])) {
                foreach ($record['context'] as $key => $value) {
                    if ($key === 'tags') {
                        // Handled natively by Sentry monolog handler
                        continue;
                    }
                    $scope->setExtra($key, is_string($value) && json_decode($value) !== null ? json_decode($value) : $value);
                }
            }

            $result = $this->decoratedHandler->handle($record);
        });

        return $result;
    }

    /**
     * @inheritDoc
     */
    public function handleBatch(array $records): void
    {
        $this->decoratedHandler->handleBatch($records);
    }

    /**
     * @inheritDoc
     */
    public function close(): void
    {
        $this->decoratedHandler->close();
    }

    /**
     * @inheritDoc
     */
    public function pushProcessor($callback)
    {
        $this->decoratedHandler->pushProcessor($callback);

        return $this;
    }

    /**
     * @inheritDoc
     */
    public function popProcessor()
    {
        $this->decoratedHandler->popProcessor();
    }

    /**
     * @inheritDoc
     */
    public function setFormatter(FormatterInterface $formatter)
    {
        $this->decoratedHandler->setFormatter($formatter);

        return $this;
    }

    /**
     * @inheritDoc
     */
    public function getFormatter()
    {
        return $this->decoratedHandler->getFormatter();
    }
}

Thank you @ste93cry :)

Hi there! I have an issue using "withScope". I've decorated Sentry\Monolog\Handler, but "withScope" pushes modified scope in stack, so when i call write(), i got another scope, than modified in decorator. What do i do wrong?

Scopes work with a stack, so you can push & pop as needed.
withScope pushes a new scope on the stack temporarily, just for the time of execution of the callback: https://github.com/getsentry/sentry-php/blob/d9fdc40728dc58936f59e9b2e1104163fdcafe24/src/State/HubInterface.php#L52-L58
You're probably gonna need configureScope, with a pushScope just before if needed.

`public function handle(array $record): bool
{
$result = false;
    withScope(function (Scope $scope) use ($record, &$result): void {
        if (isset($record['context']) && is_array($record['context']) && count($record['context']) > 0) {
            $scope->setContext('monolog.context', $record['context']);
            $scope->setExtras($record['extra']);
            $scope->setExtra('monolog.formatted', $record['formatted'] ?? '');

        }

        $result = $this->handler->handle($record);
    });

    return $result;
}

`
I mean handle() method in callback has it's own withScope() call, so your changes to scope in decorator has no effect on final event. Also, configureScope has no effect.

It's a symfony + monolog-bundle issue: decorator has different hub instance (via function withScope) than Sentry\Monolog\Handler decorated instance. One from DI container, other from static SentrySdk call.


Provide a way to check if locale is supported before calling paypal.Button.render

There may be cases where it is useful to check if a locale is supported before calling paypal.Button.render.
Real world use case: Braintree got a pull request for adding a Hungarian translation to braintree-web-drop-in: https://github.com/braintree/braintree-web-drop-in/pull/187
This locale is not currently compatible with checkout.js, so the PayPal setup fails. Would be great to have the rest of Drop-in support the locale, and just fall back to the customer's browser settings for the PayPal flow.

Should be able to rely on https://github.com/paypal/paypal-sdk-constants/blob/master/src/locale.js#L448 -- if it's part of this array, it's supported by paypal web sdks.


Get error when using task 'PublishCodeCoverageResults@1' in azure-pipelines.yml.

Situation
It works normal when we using cloud build agent. But It returns error in publishing test report when we use the local build agent. It can't generate the test report after the test step is done.
Version

os : ubuntu 16.04
dotnet : 2.1.810
Error :

/usr/bin/dotnet /home/RCTAzureAutoAgent/azure_agent/_work/_tasks/PublishCodeCoverageResults_2a7ebc54-c13e-490e-81a5-d7561ab7cd97/1.160.4/netcoreapp2.0/ReportGenerator.dll -reports:/home/RCTAzureAutoAgent/azure_agent/_work/1/s/TestResults//coverage.cobertura.xml -targetdir:/home/RCTAzureAutoAgent/azure_agent/_work/_temp/cchtml -reporttypes:HtmlInline_AzurePipelines -sourcedirs:/home/RCTAzureAutoAgent/azure_agent/_work/1/s 2020-09-16T14:41:00: Arguments 2020-09-16T14:41:00: -reports:/home/RCTAzureAutoAgent/azure_agent/_work/1/s/TestResults//coverage.cobertura.xml 2020-09-16T14:41:00: -targetdir:/home/RCTAzureAutoAgent/azure_agent/_work/_temp/cchtml 2020-09-16T14:41:00: -reporttypes:HtmlInline_AzurePipelines 2020-09-16T14:41:00: -sourcedirs:/home/RCTAzureAutoAgent/azure_agent/_work/1/s Assert Failure Expression: [Recursive resource lookup bug] Description: Infinite recursion during resource lookup within System.Private.CoreLib. This may be a bug in System.Private.CoreLib, or potentially in certain extensibility points such as assembly resolve events or CultureInfo names. Resource name: ArgumentNull_Generic
Solution
We already find the solution by updating the file.
After updating 'netcoreapp2.0' to 'netcoreapp2.1', the task works.
Request
We will need to manually edit the file after we add a new build agent in local. This will cause an issue.
Could you help to update the file in Github? Or do you have any better solution?
I also create a help desk ticket

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

ID: c6ac4ff4-9839-6a6d-8d20-a25084d221ef
Version Independent ID: bd09f139-8e69-8ea1-6671-b7bba8e17c23
Content: Publish Code Coverage Results task - Azure Pipelines
Content Source: docs/pipelines/tasks/test/publish-code-coverage-results.md
Product: devops
Technology: devops-cicd-tasks
GitHub Login: @PBoraMSFT
Microsoft Alias: pbora

@ATYUYTA Thank you for the feedback.
For issues with specific files, please move this issue to the azure-pipelines-tasks repo.
For all documentation updates, please reference any changes below and we can assign this to the author for further review.


[Mobile] I can't add the Container->Header widget in new page

Technical
VSC
(by design editor PR #230)
I use WATT Sample for TAU_1.1_Base_Mobile (TAU library is 1.1.0)
Description
I can't add the Container->Header widget in new empty page
Steps to reproduce

Create a new project in Design Editor
New Page -> create Empty page
Drag&Drop the Widget pannel-> Container->Header widget in new empty page

Expected behaviour
Can add the Header widget
Actual behaviour
Can't add the Header widget

It looks like there is an issue with _onSetNewGuide function in guide.js (it seems to have a problem with working properly when a component has to be placed in the current container's parent).
I'm in the middle of debugging it (it's old and large function so it takes time to understand and fix it).

I've partially refactored guide.js during work on this case but this issue isn't still resolved.
As I've mentioned above this issue occurs most probably due to errors in guide.js code.

I've made PR with changes and it works. But I think we need to strong refactor Design Editor Core elements like a guide, attributes etc.

This PR was merged. Can you review it?

It's good working. Thanks


Bug/593/write transaction no sideeffects

Had troubles with passing the tests in the driver as bigchaindb's http server adds assignee as a key to a transaction submitted to be written into the backlog.
This PR simply removes the reference to the passed transaction dict in write_transaction by copying the dict

Current coverage is 92.47% (diff: 100%)

Merging #606 into master will increase coverage by 0.01%

@@             master       #606   diff @@
==========================================
  Files            22         22
  Lines          1207       1209     +2
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           1116       1118     +2
  Misses           91         91
  Partials          0          0

Powered by Codecov. Last update efaf383...dda1605

:+1:

all good :+1:

Dear GitHub, this is related to issue #593 (hint hint).

Haha @ttmc, you have to use closes #593 or fixes #593 for them to pick it up ;).


Redesign the footer with a responsive design

I'd like to modify the footer and make it responsive. If you want me to work on this please assign me under Hacktoberfest topic.

@Shanb98 I am assigning you this issue. Please make sure you start the Repo and Follow me on GitHub (https://github.com/karamraj)

@Shanb98 Join Discord Server for this repo - https://discord.gg/SH6XRrHs8E

Please preview my PR


Unresponsive controller detection not working for secure commands

2023-09-29T02:32:45.919Z DRIVER » [Node 118] [REQ] [SendDataBridge]
                                  │ source node id:   1
                                  │ transmit options: 0x25
                                  │ callback id:      177
                                  └─[Security2CCMessageEncapsulation]
                                    │ sequence number: 236
                                    │ security class:  S2_Authenticated
                                    └─[FirmwareUpdateMetaDataCCReport]
                                        report #: 79
                                        is last:  false
2023-09-29T02:32:45.931Z SERIAL « [ACK]                                                                   (0x06)
2023-09-29T02:32:45.932Z SERIAL « 0x010401a90152                                                       (6 bytes)
2023-09-29T02:32:45.933Z SERIAL » [ACK]                                                                   (0x06)
2023-09-29T02:32:45.933Z DRIVER « [RES] [SendDataBridge]
                                    was sent: true

...

2023-09-29T02:33:50.936Z DRIVER » [Node 118] [REQ] [SendDataBridge]
                                  │ source node id:   1
                                  │ transmit options: 0x05
                                  │ callback id:      178
                                  └─[Security2CCNonceReport]
                                      sequence number:  237
                                      SOS:              true
                                      MOS:              false
                                      receiver entropy: 0x5b03d3f1d2797eef5977b5a363bdc5c0
2023-09-29T02:33:50.936Z DRIVER » [REQ] [SendDataAbort]
...
2023-09-29T02:33:50.944Z DRIVER « [RES] [SendDataBridge]
                                    was sent: true
2023-09-29T02:33:50.951Z DRIVER « [REQ] [SendDataBridge]
                                    callback id:                           178
                                    transmit status:                       Fail, took 0 ms
                                    routing attempts:                      0
                                    protocol & route speed:                Unknown (0x00)
                                    routing scheme:                        Idle
                                    ACK RSSI:                              0 dBm
                                    ACK channel no.:                       0
                                    TX channel no.:                        0
                                    TX power:                              0 dBm
                                    measured noise floor:                  0 dBm
                                    ACK TX power by destination:           0 dBm
                                    measured RSSI of ACK from destination: 0 dBm
                                    measured noise floor by destination:   0 dBm
2023-09-29T02:33:50.951Z CNTRLR   The controller is jammed

The above example is using the immediate queue I think, but https://github.com/zwave-js/node-zwave-js/issues/6260#issuecomment-1738310463 reports an example where the next command on the normal queue is sent immediately after the abort.


Support for some sort of collect()/extend() into LinkedList<Vec<T>>

From what I can tell, building a LinkedList<Vec<T>> is an efficient (one of the more efficient?) ways of collecting a parallel iterator, which is why ParallelExtend for Vec uses it. However, if you don't need specifically a Vec the process of concatenating all the vecs might be unnecessary overhead. It'd be nice if ListVecConsumer was exposed in some way, so that users could collect like that. (Yes, you could just .fold(Vec::new, |vec, item| vec.push(item)), but that's an element-wise push that skips specialization of Extend for Vec)
Directly implementing FromParallelIterator<T> for LinkedList<Vec<T>> might break type inference, but there could be a collect_vec_list() method on ParallelIterator, maybe.

From what I can tell, building a LinkedList<Vec<T>> is an efficient (one of the more efficient?) ways of collecting a parallel iterator, which is why ParallelExtend for Vec uses it.

It's the most efficient way that we know, at least for unindexed iterators. For IndexedParallelIterator, we have a bit of pseudo-specialization (via opt_len) that lets it collect directly into the final pre-allocated Vec. We could still do that in your scenario as well, just slightly wastefully allocating a single LinkedList node to hold the entire Vec.

there could be a collect_vec_list() method on ParallelIterator, maybe.

I think that sounds reasonable!
I don't think we need an extend flavor though, because anyone can cheaply append a collection to an existing list.


Rewrite Entity Extraction recipe to utilize larger context

Update model used in this recipe:
https://github.com/ibm-granite-community/granite-snack-cookbook/blob/main/recipes/Entity-Extraction/entity_extraction.ipynb
And usefully extend context.

will addd @gh-madhu1 when he is in the org

I have updated the recipe to granite 3.1. Not sure if there is anything to do for the longer context length, so leaving this open.


chef-server and chef-client 's version must match?

for example ,if my chef-server version is 12.11.1, and my chef-client is  11.12.8. they will work ok ?

Can someone help me?  thank you very much。because the debian 8 (cpu is powerpc)internal 's  chef-client is 11.12.8.

Thanks for filing a GitHub Issue.
At Chef, we use GitHub Issues to track bugs and feature requests. Questions like "How do I .... with Chef?" are better answered by our awesome community on the Chef mailing lists, or on StackOverflow. (Please pick only one place to ask your question, however, as it's considered impolite to post the same question to multiple forums.)
You can find information about joining and asking questions on our mailing lists here. On StackOverflow, please tag your question with the "chef" label so that it shows up in this list.
We will be closing your issue. Please get in touch with the community via the above mechanisms in order to find a solution to your problem.


More logs

create a json file when training a model with all the parameters used
add a flag to stable_txt2img to create a json file describing the prompt and all parameter used.

tested locally and working fine

@GpuLover would love to merge this in! could you resolve that conflict?

now using opt.dict so it automaticallty updates if argument are added to the parsers in the future
both commit are tested and working

@JoePenna Done

Added in the Simple colab script


When degrading, goog.ui.LabelInput shouldn't remove placeholder text when focused and empty

This issue was imported from Closure Library's previous home at http://closure-library.googlecode.com
The link to the original issue is:
https://code.google.com/p/closure-library/issues/detail?id=412

This issue is being closed as a preparation step before archiving the repository.  See the README for more detail.


error when running build.tcl install - > zmq.tcl:1948: error: 'clientdata' undeclared (first use in this function)

Hi
I'm trying to install tclzmq but keep getting an error. I have the following installed:
tclsh8.6
zmq4.0.1
critcl 3.0.7 (critcl.kit download)
tclkit 8.6.3 (tclkit-8.6.3-rhel5-x86_64 download)
Machine used:
RHEL 5.0
Kernel Version .........:  2.6.18-194.el5
Can someone please help here?

Amit

ERROR:
FAILED   zmq.tcl: In function 'cset_socket_option_as_tcl_obj':
FAILED   zmq.tcl:816: warning: passing argument 3 of 'tclStubsPtr->tcl_GetWideIntFromObj' from incompatible pointer type
FAILED   zmq.tcl:834: warning: passing argument 3 of 'tclStubsPtr->tcl_GetWideIntFromObj' from incompatible pointer type
FAILED   zmq.tcl: In function 'tcl__zmq_version':
FAILED   zmq.tcl:1948: error: 'clientdata' undeclared (first use in this function)
FAILED   zmq.tcl:1948: error: (Each undeclared identifier is reported only once
FAILED   zmq.tcl:1948: error: for each function it appears in.)
FAILED   zmq.tcl:1948: error: expected ';' before 'zmqClientDataInitVar'
FAILED   zmq.tcl: In function 'tcl__zmq_strerror':
FAILED   zmq.tcl:1966: error: 'clientdata' undeclared (first use in this function)
FAILED   zmq.tcl:1966: error: expected ';' before 'zmqClientDataInitVar'
FAILED   zmq.tcl: In function 'tcl__zmq_max_block_time':
FAILED   zmq.tcl:1981: error: 'clientdata' undeclared (first use in this function)
FAILED   zmq.tcl:1981: error: expected ';' before 'zmqClientDataInitVar'
FAILED   zmq.tcl: In function 'tcl__zmq_context':
FAILED   zmq.tcl:2060: error: 'clientdata' undeclared (first use in this function)
FAILED   zmq.tcl:2060: error: expected ';' before 'zmqClientDataInitVar'
FAILED   zmq.tcl: In function 'tcl__zmq_socket':
FAILED   zmq.tcl:2128: error: 'clientdata' undeclared (first use in this function)
FAILED   zmq.tcl:2128: error: expected ';' before 'zmqClientDataInitVar'
FAILED   zmq.tcl: In function 'tcl__zmq_message':
FAILED   zmq.tcl:2212: error: 'clientdata' undeclared (first use in this function)
FAILED   zmq.tcl:2212: error: expected ';' before 'zmqClientDataInitVar'
FAILED   zmq.tcl: In function 'tcl__zmq_poll':
FAILED   zmq.tcl:2299: error: 'clientdata' undeclared (first use in this function)
FAILED   zmq.tcl:2299: error: expected ';' before 'zmqClientDataInitVar'
FAILED   zmq.tcl: In function 'tcl__zmq_device':
FAILED   zmq.tcl:2329: error: 'clientdata' undeclared (first use in this function)
FAILED   zmq.tcl:2329: error: expected ';' before 'zmqClientDataInitVar'
FAILED   zmq.tcl: In function 'tcl__zmq_proxy':
FAILED   zmq.tcl:2353: error: 'clientdata' undeclared (first use in this function)
FAILED   zmq.tcl:2353: error: expected ';' before 'zmqClientDataInitVar'
FAILED   ERROR while compiling code in /icd/common/thirdparty/ZMQ/tclzmq/download/tclzmq-4.0.1/zmq.tcl:
FAILED   child process exited abnormally
FAILED
child process exited abnormally
while executing
"exec 2>@ stderr >@ stdout /icd/common/thirdparty/ZMQ/critcl/download/critcl3.kit -with-mode static -cache /icd/common/thirdparty/ZMQ/tclzmq/download/t..."
("eval" body line 1)
invoked from within
"eval $cmd"
(procedure "RunCritcl" line 39)
invoked from within
"RunCritcl {*}$rcargs"
(procedure "_install" line 27)
invoked from within
"_install /icd/common/thirdparty/ZMQ/tclzmq/install -zmq /icd/common/thirdparty/ZMQ/zmq/install -static"
("eval" body line 1)
invoked from within
"eval _$argv"

The error caused by the author's decision to add [UUID.serial $file] to the name of a critcl::ccommand https://github.com/andreas-kupries/critcl/issues/37
Fixed in critcl 3.1.7 http://andreas-kupries.github.io/critcl/#download


⚠ Bofrost.be has degraded performance

In 6402af4, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1772 ms

Resolved: Bofrost.be performance has improved in c234171.


[SMALLFIX] Increase accuracy of timeout message

@tedyu

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/561/
Test PASSed.


Added contracts to PowerNotifications.idl

Adding contract to the IDL to facilitate adding new APIs and more!

/azp run

/azp run


Use of implicit union return type causes  "'boolean' is not assignable to type 'false'"

TypeScript Version:  2.6.0-dev.20171015
This relates to the render method of React components, specifically the use of false in the union return type:
render(): JSX.Element | null | false

I've create a stand-alone example that demonstrates the issue. See the code comments for instructions.
Code
declare class Base {
    render(): Date | false;
}

class TestImplicitReturnType extends Base {

    render() /* ': false' is required here as explicit return type to prevent a compile error */ {
       return false;
    }

}

class TestControlFlow extends Base {

    render() {
        if (new Date().getTime() % 2 === 0) {
            return new Date(); /* removing this return statement causes the false value below to be rejected  */
        }
        return false;
    }

}

Expected behavior:
For TestImplicitReturn I should not be required to manually add : false as return type.
For TestControlFlow I should not be required to also return a Date to satisfy the type checker.
Actual behavior:
For TestImplicitReturn the compiler outputs:
Class 'TestImplicitReturnType' incorrectly extends base class 'Base'.
  Types of property 'render' are incompatible.
    Type '() => boolean' is not assignable to type '() => false | Date'.
      Type 'boolean' is not assignable to type 'false | Date'.

For TestImplicitReturnType the compiler outputs the following when the date return statement is removed:
Class 'TestControlFlow' incorrectly extends base class 'Base'.
  Types of property 'render' are incompatible.
    Type '() => boolean' is not assignable to type '() => false | Date'.
      Type 'boolean' is not assignable to type 'false | Date'.

The issue here is that a method in a derived class is not contextually typed by the type of the base's method. This has been discussed in by https://github.com/Microsoft/TypeScript/issues/1373  (and related to https://github.com/Microsoft/TypeScript/issues/3667, https://github.com/Microsoft/TypeScript/pull/6118 and https://github.com/Microsoft/TypeScript/pull/10610).

Duplicate of  https://github.com/Microsoft/TypeScript/issues/1373

Thanks for the detailed reply.
I still find it odd (counter-intuitive) that false produces a compilation error when boolean doesn't:
TypeScript playground using boolean
TypeScript playground using false
Does this narrow the scope of a potential fix? As it stands we're seeing the unwanted compilation error when using @graphql.

As the linked issue mentions, we don't currently use information about the base class to inform the type of a base class method. So understand the below in that context (i.e. we think the best fix involves using that information)
There are rules around widening that attempt to figure out what behavior you're trying expose in a method. If you wrote something like
    someMethod() {
       if (...) return 3; else return 6;
    }

we think that you want the signature someMethod(): number, not someMethod(): 3 | 6 -- basically, TS shouldn't immediately expose the internals of the function without some kind of hint to do so.
We do infer Date | false as a return type when the return expressions are heterogeneous.
All these rules came from experimental data produced when we added literal types to the type system (in other words, we messed with them until we got the "best" set of breaking changes) and are based on our "best guess" about what the inferred return type should be. We already tweaked these rules to the best of our ability and any change here would move away from the local maximum.

Automatically closing this issue for housekeeping purposes. The issue labels indicate that it is unactionable at the moment or has already been addressed.


fixes #12771: Added Greenplum Ingestion Connector

Describe your changes:
fixes #12771  : Added Greenplum Ingestion Connector (based on Postgres connector from OpenMetadata 1.1 but queries updated to support Greenplum).
Tested on Greenplum v6

What changes did I make?
Added Greenplum connector in Ingestion, Test Connection Service, UI, JsonSchema
Why did you make them?
Connector was needed to support Greenplum in my project
How did you test your changes?
Using Metadata CLI and Airflow on production Greenplum in my company

Type of change:

[x] Improvement

Checklist:

[x] I have read the CONTRIBUTING document.
[x] My PR title is Fixes <issue-number>: <short explanation>
[x] I have commented on my code, particularly in hard-to-understand areas.
[x] For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

[ ] I have added tests around the new logic.
[ ] For connector/ingestion changes: I updated the documentation.

@ulixius9 Thanks for your comments. Yes, my connector is based entirely on Postgres connector, key changes are only in SQL queries to get metadata. I understand that my implementation makes it harder to maintain the connector, that is a right point.
But, I don't believe implementing it as connection schema (e.g. pgspider implementation) is going to give good user experience. Greenplum is based on Postgre but it has it's own established brand and recognition - hence I would like to see it having its own databaseService enum entry and icon in interface. AFAIK you can't have those advantages if I am going to implement it in your suggested way.
There are actually a lot of Postgre based databases - see this link, for example Redshift is also postgre derived, and has a standalone connector in your code base.
Also my Greenplum connector does not support some advanced information gathering such as tags, lineage extraction, usage (I tried to find a way to implement them as well, but my Greenplum installation supported none of those things)
IMO the ideal solution is to create some basic postgre utilities that can be shared by multiple databaseServices - PostgreSQL, Greenplum, PGSpider, RedShift - etc.
What do you think?

+1 for having a separate connector. However can we centralize the code in a common class that both Postgres and green plum can inherit.  @T01iK

@chirag-madlani @karanh37 Could you please also review and give an approve for this pull request? We are really looking forward to the support of Greenplum in OpenMetadata 🙏

@evmuravev @T01iK Can you resolve the conflicts here?

resolved conflicts and retested the connector locally

@chirag-madlani @harshach


docs: Don't suggest --audit-fts-indexes for non-3.0 upgrades.

Only Zulip 3.0 and above support the --audit-fts-indexes option to
upgrade-zulip-stage-2; saying "same as Bionic to Focal" on other
other steps, which are for Zulip 2.1 or 2.0, will result in errors.
Provide the full text of the updated upgrade-zulip-stage-2 call in
step 5 for all non-3.0 upgrades.  For Trusty to Xenial and Stretch to
Buster, we do not say "Same as Xenial to Bionic" , because it is
likely that readers do not notice that step does not read "Same as
Bionic to Focal."

Tacked on a commit to suggest upgrading all of the way.

Aha, this is also #15877, which I'd half-forgotten about.

Took another stab at this.  The steps are getting more involved, which is unfortunate.

I think this is the best we can do.  Merged, thanks @alexmv!  I'll also backport to 3.x, which updates the /stable/ ReadTheDocs tree.


Project still maintained?

I see on nicedit.com that the last release was in 2012.
There's also a bogus 1969 date.

I'd suggest ckeditor or redactor at this point.

I've been evaluating 40+ editors over the weekend. Redactor can't paste images.

I moved NicEdit over to GitHub a few years ago because I was using it for a project and figured since it wasn't on GitHub I'd create a project for it. I have no use for it anymore, so I haven't really been maintaining it much.


Support setting identify properties in gateway

Support users setting the properties to send during identify via the new ClusterBuilder::identify_properties and ShardBuilder::identify_properties methods. When not set a default will be used.
Closes #1119.

Addressed comments.


[interop][SwiftToCxx] disable -Wnon-modular-include-in-framework-modu…

…le diagnostic before including ptrauth

@swift-ci please test


[tx] Implement :db/tx, :db/txInstant in transactor

I just didn't get to tracking transaction IDs at all in #170.  This might require #184, depending on how it's implemented, but there are simpler ways to achieve it.
Each transaction gets an extra assertion of the form:
[[:db/add :db/tx :db/txInstant INSTANT]]

where :db/tx is the next sequential transaction ID (in the :db.part/tx partition), and INSTANT is the transactors local time.  (We'll ignore clock drift for now -- at our own peril.)

Is this done, @ncalexan?

Is this done, @ncalexan?

Yep.  Worth linking to https://github.com/mozilla/mentat/issues/225, which tracks allowing the consumer to use :db/tx and :db/txInstant in the transaction.

OK. Closing this one!


Onboarding UI

Describe the bug

To Reproduce
Steps to reproduce the behavior:

Sign up new account

Expected behavior

Screenshots

Details

Platform: Android
Platform version: Android 10
App version: 1.70.0

Additional context

Just merged in a fix for this. Thank you!
https://github.com/bluesky-social/social-app/pull/3176


materia.json: annotate with more information

Addresses part of #42 .

Hmmm... I'm not sure I understand the test failures. Let me know if you understand what needs to be changed.

Do you mind squashing these commits into on?  I like to use "rebase and merge" to keep a clean history.

Superceded by #46


Support for Github Environments

Prerequisites:

Is the functionality available in the GitHub UI? If so, please provide a link to information about the feature.
https://github.com/{org}/{repo}/settings/environments

Is the functionality available through the GitHub API? If the functionality is available, please provide links to the
API documentation (https://developer.github.com/v3/) as well as the Octokit documentation (https://octokit.github.io/).

https://docs.github.com/en/rest/deployments/environments

If the functionality is not yet available in the API, it would be helpful if you
contacted support (https://support.github.com/) or posted in the Community Forum (https://github.community/). Please
include a link to the forum post if you create one or a copy of the response from support.

New Feature
Please describe the desired new functionality:
Add support to manage Github environments with settings.yml

@gr2m wdyt?

@gr2m do you have any thoughts on this issue? thx


Run Blazor Web App - The app does not launch in the browser automatically

URL
https://dotnet.microsoft.com/en-us/learn/aspnet/blazor-tutorial/run
Operating system
windows
More information about your system
Windows 11
Visual Studio Microsoft Visual Studio Professional 2022 (64-bit) -  Version 17.10.0 Preview 3.0
Chrome and Edge are available
Description
I follow the Web App Blazor tutorial, at the step Run your app. the app does not open in any browser. All I have is a console window like the below screenshot:

dotnet --info
No response

Update:
I have to manually open a chrome tab with url: http://localhost:5000/ , then the app is shown:

And what is the option you have selected in your debug menu?
It should look similar to this

@danroth27 any other suggestions?

And what is the option you have selected in your debug menu? It should look similar to this
@danroth27 any other suggestions?

It now can launch well on both Chrome and Edge. Thank you.

Great to hear! ❤


Any way to extract GPU info using atom-shell?

Node.js has an OS module and it provides some basic CPU and memory info.
Wondering if we have some similar way to get data about GPU?
For what I know now:
You can type chrome://gpu in google chrome to check all the gpu information you want.
But how to fill the gap between this "chrome://gpu" and atom-shell?

Hi, @zcbenz
mind telling me how to get this gpu info, I mean which module which method?
I did a search on release page, but didn't manage to find it out.

It can be got by loading chrome://gpu url in browser window, it hasn't been released yet.

@zcbenz  Thank you for the info~

Hi, @zcbenz
after updated to 22.3, I can load chrome://gpu with a browser window, but I wonder if we can read meaningful data from it into atom-shell?
Looks like it's not that fancy to grab data using dom operations.
Just wonder if we have some method like what nw did?
Here is a link of it, https://github.com/nwjs/nw.js/wiki/Extract-info-about-gpu.
Any suggestions?


Product quantity for stock management

hi , I was checking  add product feature in admin dashboard,  I noticed there is no quantity field and I thought It would be nicer if  we manage stock and if product  is out of stock , we can put a label on product. so that user can know before hand.  I can do some PR if needed.

Added in 3ab76684bb371470cbd09bf7cc28390d0807d65b.

I still don't see quantity when creating a new product.

never mind found it on settings.json


Calculation of hydrogen chain energy using the MDF method

Hello, I am using HF to calculate the energy of one-dimensional hydrogen chain. The energy by using AFTDT method is close to the literature, but the results by using DF and MDF methods are far from the actual value. I tried to use the result of the AFTDT method as the initial guess for the MDF method, but it still failed. Are there anything obviously wrong in my input? Thanks.
import numpy
from pyscf import scf
from pyscf.pbc import df as pdf
from pyscf.pbc import scf as pbchf
from pyscf.pbc import gto as pbcgto
from pyscf.pbc import tools
e = []
L = 1.8
cell = pbcgto.Cell()
cell.build(unit = 'A',
           a = [[L,0,0],[0,100,0],[0,0,100]],
           mesh = [40,20,20],
           atom = 'H 0 0 0; H 0.9 0 0',
           dimension=1,
           verbose = 4,
           basis = 'gth-szv',
           pseudo = 'gth-pbe')
mf = pbchf.KRHF(cell)
mf.with_df = pdf.AFTDF(cell)
mf.kpts = cell.make_kpts([4,1,1])
e.append(mf.kernel())
dm_init_guess = mf.make_rdm1()

mf = pbchf.KRHF(cell).density_fit(auxbasis='weigend')
mf.kpts = cell.make_kpts([4,1,1])
e.append(mf.kernel())

mf = pbchf.KRHF(cell).mix_density_fit(auxbasis='weigend')
mf.init_guess = dm_init_guess
mf.kpts = cell.make_kpts([4,1,1])
e.append(mf.kernel())

# mol = tools.super_cell(cell, [4,1,1]).to_mol()
# mf = scf.RHF(mol)
# e.append(mf.kernel()/4)

print('1D: AFT      DF       MDF')
print(e)

Results:
AFT: -1.0123035727096745
DF: -11516.569579106246
MDF: -11516.566757778026]

Please use dimension=2 with one large vacuum in y-axis for now. In DF and MDF, there requires a cancellation between integrals computed with two different methods. Some improper parameter estimations probably cause one integral method singular. It is not easy to fix this problem in near future.

Thank you very much! I will try it.

This issue should have been resolved in pyscf-2.6


Add SendGrid Template substitutions to SMTP API Header

As per https://sendgrid.com/docs/API_Reference/Web_API_v3/Transactional_Templates/smtpapi.html, the SMTP API allows the use of template substitutions.
Example usage:
    from django.core.mail import EmailMultiAlternatives
    subject = "My cool subject"
    from_email =<EMAIL_ADDRESS>    to_email =<EMAIL_ADDRESS>
    mail = EmailMultiAlternatives(
        subject=subject,
        from_email=from_email,
        to=[to_email],
        body="",
        headers={
            "Reply-To": from_email,
            "sub": {
                ":sendername": [  # ':sendername' is a custom template replacement defined in SendGrid
                    "John Doe"  # Number of array entries == number of recipients in the EmailMultiAlternatives 'to' param
                ],
            },
            "Filters": {
                "templates": {
                    "settings": {
                        "enable": 1,
                        "template_id": "xxxxxxxx-yyyy-xxxx-yyyy-xxxxxxxxxxxx"
                    }
                }
            }
        }
    )
    mail.attach_alternative("some text that will get replaced anyway", "text/html")
    mail.send()

Added #19


Don't Merge! PR real time test for source branch full build

Docs Build status updates of commit f4cbf43:
:clock10: Pending: waiting for processors (27 builds ahead of you)
:warning: Docs Build is busy, currently there are 27 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit f4cbf43:
:clock10: Full build: the average full build time is 5 min(s) 47 sec(s), this is based on the last 30 build(s) for this repository.

Docs Build status updates of commit f4cbf43:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docfx.json
:white_check_mark:Succeeded

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


AURO MIGRATION: Update node to v22

Resolves AlaskaAirlines/auro-templates#6
Summary by Sourcery
Update Auro Hyperlink to use Node.js v22 and update supporting documentation and workflows.
Build:

Update Node.js to v22.

CI:

Update CI workflows to test on Node.js v20 and v22.

Documentation:

Update documentation to reflect Node.js v22 support and conventional commit guidelines.

Tests:

Update test matrix to include Node.js v22.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Deploy from source code

As an administrator,
I can deploy new version of the Editor software from source code to Production

[ ] Introduction to Docker
[ ] Get Production server from Hezner => Jarkko
[ ] SSH keys to Jarkko
[ ] Sudo access to all in the Hezner (Production) => Jarkko
[ ] Github Actions template in use: build Docker image & deploy that
[ ] Editor software accessible from public Internet (Write documentation what was done)


Export Styler settings

It would be super useful if:

the Theme and Layout settings in the Settings widget could be exported as CSS for use in another app.

all the settings could be saved locally and the read automatically on app restart.

Yip, I've got some prototypes for this that export the JSON and load it. I'm kinda waiting however because I might just put that info in a webmap.

You can use the share tool now in https://github.com/Esri/calcite-maps-styler-template to get the json for your current config. Not quite export but it is there.


LIB-170 data views support

3 Parts of this task here:

PleaseForView in SCDataObject
SCDataObject "isView"
Parsing references (expand options in view)

Started reviewing, will finish tomorrow, so please don't merge before. But looks good so far.

I like the idea of adding new pleaseForView methods, and being able to use views with known convention of SCPlease :+1: Good job!
After you answer my question about the relationsCache we will be able to merge - everything else looks good!

I refreshed this branch with offline support and instead of this "relationsCache" I created RelationsForClass dict in SCRegisterManager. Please review this and let's go with this merge :)

lgtm


Fused image black everywhere except for the first image

By following the steps in 'notebooks/stitching_bigstitcher_grid_2d.ipynb', the code correctly finds correct registration parameters and shows correct results in napari viewer. After following with the fusion part of the code, even though everything runs smoothly and without any errors, the resulting image is incorrect. The output shape of the stitched image is correct, but the values are everywhere 0 except for the most top left image (index 0).
Here is the final image from the fusion:

Hi @ashkan-mokarian, thanks for posting this issue!
There had been a problem with the fusion in a previous release, which should however be fixed. Which version of multiview-stitcher are you using? When running the notebook with 0.1.5 (released just now) the fusion looks good. Please don't hesitate to write back here if the problem persists with 0.1.5.

Thanks for your quick reply.
After updating the package to 0.1.5, the problem was resolved.


Update params in RLOS

Updates params in RLOS from erroneous 'param_3" to correct "param_7"

This changes the param_3s that should be param_7s back.
Description
fixes blunder
Related Issue
https://github.com/Esri/military-tools-geoprocessing-toolbox/issues/334
https://github.com/Esri/military-tools-geoprocessing-toolbox/issues/331
Motivation and Context
Tool did not have correct parameters, failed.
How Has This Been Tested?
Tested locally. Tool has radius of observer parameter and runs, rather than failing.

Types of changes

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

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.

@dfoll - this fixes the RLOS issue I found while doing the cucumber test for RLOS.

@kgonzago - found another case of this problem, fixed in this PR.

Oops, sorry, I usually remember to exclude the GDB changes.

Closing - change made in #335


Wiki

I might be just extremely stupid and its on the 'readme.md' of the git repo but skimmed over it or it's on the app itself, but is there an actual wiki for all the various functions that can be done on 'xremap'?
So far from all of my searching for a Linux alternative for XMBC (X-mouse button control), this seems the best alternative that covers most of the probable use case that XMBC offers. my only issue is I looking for any documentation if there are options for XMBC 'Repeatedly while the button is down', 'Sticky (Repeatedly until button is pressed again)', and etc. with repeat delay and stuff.

In fact I can't figure out how to run xremap on Ubuntu24.04 because I use wayland and downloaded the xremap extension, and after switching on the xremap in extensions app I don't know where to put the config.yml.
Basically I don't know the relationship between the extension and the rust crate.
Do we need both? If not, where to put the config.yml and load it with xremap gnome shell extension?


Renew github-ranking data importer

I'll do this in another implementation.

I'll do this in another implementation.


[PART 1 ]Feature/video 2 [DONT MERGE]

What does this PR?
Where should the reviewer start?
Steps to reproduce
Issue number
Screenshots/Video

will close this for now, we might integrate video uploads in future!


Feat: Automate greeting when PR is made using Github bot 🤖

Describe the feature
As the contributor count rises on the repo, it becomes increasingly challenging for maintainers to personally greet and encourage each contributor for their valuable input. Equally important is the reminder for them to review the project's contribution guidelines.
I would like to add a github bot , which would greet the user whenever there is a PR request
For PR
" Great job, @ username 🎉 , Thank you for submitting your pull request. Your contribution is valuable and we appreciate your efforts to improve our project. Our team will review your changes shortly."
Add ScreenShots

Record

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

Go ahead @shubhammjha22

@nikhil25803 I have made the PR. Kindly review it and  i would request you to add the apt level and labels

@nikhil25803  i have add the demo video , kindly review it


[Feature Request]: Allow models to be updated for inference

What would you like to happen?
Probably through a slowly updating side input
Issue Priority
Priority: 2
Issue Component
Component: sdk-py-core

This has been implemented


New-AzSqlDatabaseImport should support -Tags parameter

Is your feature request related to a problem? Please describe.
Billing is base on TAGs and now when I create new DB from .bacpac file, import time (hours) is not included because TAGs are set after import.
Describe the solution you'd like
New-AzSqlDatabaseImport powershell command should support -Tags parameter, so newly created DB have right tags, right from the start
Describe alternatives you've considered
none
Additional context
none

Thank you for the feedback! Because this issue is not related to docs.microsoft.com, we cannot address it in this forum.


freeze on iPhone 8

This module is working fine on iPhone 6 but the live photo view freezes on iPhone 8 (although it works if I blindly capture). Any clue?

hmm ok you might need to provide a bit more details on this.
If I understand correctly, the camera preview freezes? Is it always frozen or only after you try to capture an image? Does this lead to a crash? And a blind capture would mean capturing while the preview is frozen?
Helpful note: I have and test this package on an iPhone 8 (not the plus model). Is it the plus model? The dual cameras may be affecting this.

Thank you for your answer. I know it's hard to understand what's going on since there is no hard crash (no stacktrace).
The camera preview starts correcly, and after about one second, it freezes, without having to try to capture. No crash. And yes a blind capture means If I capture when it's frozen, it captures a live photo (and not the one shown on the screen that is frozen). So it's just a visual issue (but a major one).
Here is the rendered JSX:
<SafeAreaView style={styles.container}>
  <RectangleScanner
    ref={scanner}
    style={styles.scanner}
    onPictureProcessed={handlePicture}
    onRectangleDetected={handleRectangle}
  />
  <RectangleOverlay
    detectedRectangle={rect}
    allowDetection
    onDetectedCapture={handleCapture}
    backgroundColor="rgba(255,165,0,0.2)"
    borderColor="rgb(255,165,0)"
    borderWidth={4}
    detectedBackgroundColor="rgba(0,175,25,0.2)"
    detectedBorderColor="rgb(0,175,25)"
    detectedBorderWidth={6}
  />
  <TouchableOpacity onPress={handleCapture} style={styles.button}>
    <View style={styles.innerButton} />
  </TouchableOpacity>
  <View style={styles.bottom}>
    <Text style={styles.disclaimer}>
      Fixez le document pour le scanner automatiquement.
      En cas d’échec, vous pouvez prendre la photo manuellement.
    </Text>
  </View>
</SafeAreaView>

And handleRectangle is just a wrapper for useState to update rect.
All of this works on iPhone 6 (both dev and release versions) and on Android.
Dependencies:
    "@babel/runtime": "^7.7.7",
    "@react-native-community/async-storage": "^1.7.1",
    "@react-native-community/datetimepicker": "^2.1.0",
    "bugsnag-react-native": "^2.23.4",
    "date-fns": "^2.8.1",
    "final-form": "^4.18.6",
    "react": "16.9.0",
    "react-final-form": "^6.3.3",
    "react-native": "^0.61.5",
    "react-native-gesture-handler": "^1.5.3",
    "react-native-image-picker": "^2.2.0",
    "react-native-image-resizer": "^1.1.0",
    "react-native-keyboard-aware-scroll-view": "^0.9.1",
    "react-native-reanimated": "^1.7.0",
    "react-native-rectangle-scanner": "^1.0.8",
    "react-native-screens": "^2.0.0-alpha.22",
    "react-native-signature-capture": "^0.4.10",
    "react-native-svg": "^11.0.1",
    "react-native-svg-charts": "^5.3.0",
    "react-native-view-pdf": "^0.10.1",
    "react-native-webview": "^8.0.5",
    "react-navigation": "^4.0.10",
    "react-navigation-stack": "^1.10.3",
    "react-navigation-tabs": "^2.7.0",
    "rn-fetch-blob": "^0.12.0",
    "standard-http-error": "^2.0.1"

@antoinerousseau thanks for the details. Is this an iPhone 8 Plus? Also what version of iOS are you running?

Sorry yes iPhone 8+, iOS 13.3.1

Thanks @antoinerousseau I'll see if I can find a way to reproduce this and report what I find.

@humphreyja thank you! :)

@antoinerousseau I was facing the same issue on an iPhone SE and iPhone XS, both on iOS 13.3.1. I dug around a bit and it looks like starting the camera session blocks the main UI and freezes the preview. I opened PR #25 to fix this issue, hopefully that solves the problem for you.

@kev-chen I can confirm that your PR fixes the issue, thank you!

Please publish this fix

@jayan2019 Thanks for reminding me! I'll have this out in a few minutes!


Grammar fault in appsignal-docs / source / guides / filter-data / index.html.md

"Sometimes it sends collects too much (or too little) app specific data by default."
sends AND collects?

Technically yes 😅  But that's not what it should say. Good catch!


Allow a manually specified collection via querystring

In the collection page, we can allow users specify an ephemeral collection by a comma-separated list of project slugs in the query string.

This is effectively complete, now that you can go to a collection slug as a page


OPL fails to compile using latest PS2SDK due to dns_getserver return value

Issue checklist
Note: these are not necessarily requirements

[x] I have checked existing OPL issues for duplicates and found none @ http://psx-scene.com/forums/official-open-ps2-loader-forum/ & https://github.com/ifcaro/Open-PS2-Loader/issues

Open PS2 loader version
0.9.3+.1220-Beta-2c95dc1
Open PS2 loader mode (USB, SMB, HDD)
Not applicable
Open PS2 type (Normal, Childproof, VMC, GSM, PS2RD)
IGS=1 PADEMU=1 HIRES=1
Issue description - Describe the problem you are having
Trying to build the latest commit on master (2c95dc1be6da8e747e179a2440190be000529799) using latest commit of PS2SDK (9cf086dfb013f196cc378f9da90089d9d5df2133).
I've successfully compiled PS2SDK and installed other dependencies using ps2toolchain and the compilation guides posted on the forums (to install a few ports dependencies)
However, every time I try to compile OPL I get this error:
ubuntu-linux-1604 :: ps2dev/src/Open-PS2-Loader ‹git master› » make IGS=1 PADEMU=1 HIRES=1
Building Open PS2 Loader 0.9.3+.1220-Beta-2c95dc1...
-Interface
src/ethsupport.c: In function `ethApplyIPConfig':
src/ethsupport.c:821: incompatible types in assignment
Makefile:759: recipe for target 'obj/ethsupport.o' failed
make[1]: *** [obj/ethsupport.o] Error 1
Makefile:182: recipe for target 'all' failed
make: *** [all] Error 2

The issue is with the return type of dns_getserver. I doubled checked the definition in the PS2SDK headers and it looks like while the ethApplyIPConfig function in src/ethsupport.c expects an ip4_addr struct to be returned from dns_getserver, the definition of that function in PS2SDK returns a ip_addr_t*.
I tried looking through the PS2SDK git history but I didn't see any changes to the return value of dns_getserver so I'm not really sure how it ever worked. This is my first time compiling OPL though.
Expected result
It should compile.
Actual result
It doesn't compile.
How to reproduce the issue

Clone and build the latest PS2SDK
Install ps2toolchain
Install OPL dependencies from ps2sdk-ports
Compile OPL using either make or make IGS=1 PADEMU=1 HIRES=1 (it doesn't make a difference)

This is fixed in #163
Can be closed

Awesome, thanks for the quick response! I'm looking forward to diving in and starting to contribute once I get my bearings.

Just pulled the latest and can confirm it resolves the compilation issue.

Hi
i have still this issue with the latest ps2sdk (4c9a644b9b3e987b1f76b763c7f1b528159efcdc)
└───╼  make
Building Open PS2 Loader 0.9.3+.1008-Beta-b0abf55...
-Interface
src/ethsupport.c: En la función `ethApplyIPConfig':
src/ethsupport.c:854: tipos incompatibles en asignación
make[1]: *** [Makefile:762: obj/ethsupport.o] Error 1
make: *** [Makefile:225: all] Error 2

greetings

@sl1pkn07 -- seems that you are trying to compile a very old version of OPL.  Do a git pull to get the latest revision/commit version and try again.

what the h.....!!


Truncate role session name to 64 chars

Normalize role session names such that it doesn't exceed the 64 char limit.
Fix #16

Pull Request Test Coverage Report for Build 90

21 of 21 (100.0%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.3%) to 67.968%

Files with Coverage Reduction
New Missed Lines
%

credentials_getter.go
2
75.81%

Totals

Change from base Build 87:
0.3%

Covered Lines:
505

Relevant Lines:
743

💛  - Coveralls


SRS Review #4

Introduction:
@karolserkis ,
Please do not take this as a review. But just to help you. I found that you haven't cited the articles but typed it. I found this in your introduction which says "Smith and Lai(2005); Smith et al. (2007) [8]",
I can see you have put in lot of effort to numbering your literature and referencing them . But in latex there is a command ~\cite{}, which you can use as shown below:
~\cite{SmithAndLai2005} and ~\cite{SmithEtAl2007} and it will take care of your referencing.

@Malavika-Srinivasan @smiths
Thanks! I figured out from other examples and reinstalling TeX Live


Fix markdown blocks

Changes/adds metadata to code blocks so we have correct colours

Remove me from your system
On Friday, 11 December 2020, 14:26:37 GMT, Antony Bishop<EMAIL_ADDRESS>wrote:
@AntonyBishop approved this pull request.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Download video posted in facebook comment.

Checklist

[x] I'm reporting a site feature request
[x] I've verified that I'm running youtube-dl version 2021.01.16
[x] I've searched the bugtracker for similar site feature requests including closed ones

Description
I’m trying to download a video from a comment on a public Facebook post (https://www.facebook.com/stoneyfid/posts/10159665750956988?comment_id=10159665837386988). This results in the following output:
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['https://www.facebook.com/stoneyfid/posts/10159665750956988?comment_id=10159665837386988', '--verbose']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.01.16
[debug] Python version 3.8.6 (CPython) -<EMAIL_ADDRESS>[debug] exe versions: ffmpeg 4.3.1, ffprobe 4.3.1
[debug] Proxy map: {}
[facebook]<PHONE_NUMBER>0956988: Downloading webpage
[facebook]<PHONE_NUMBER>0956988: Downloading webpage
ERROR: Cannot parse data; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/home/roman/.venv/lib/python3.8/site-packages/youtube_dl/YoutubeDL.py", line 805, in wrapper
    return func(self, *args, **kwargs)
  File "/home/roman/.venv/lib/python3.8/site-packages/youtube_dl/YoutubeDL.py", line 826, in __extract_info
    ie_result = ie.extract(url)
  File "/home/roman/.venv/lib/python3.8/site-packages/youtube_dl/extractor/common.py", line 532, in extract
    ie_result = self._real_extract(url)
  File "/home/roman/.venv/lib/python3.8/site-packages/youtube_dl/extractor/facebook.py", line 677, in _real_extract
    return self._extract_from_url(real_url, video_id)
  File "/home/roman/.venv/lib/python3.8/site-packages/youtube_dl/extractor/facebook.py", line 591, in _extract_from_url
    raise ExtractorError('Cannot parse data')
youtube_dl.utils.ExtractorError: Cannot parse data; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

for this link - https://www.facebook.com/stoneyfid/posts/10159665750956988?comment_id=10159665837386988  just change the  www to mbasic  (mobile version of facebook)
press enter and the page will reload, right click on the  video and select save video as
to download the video

for this link - https://www.facebook.com/stoneyfid/posts/10159665750956988?comment_id=10159665837386988  just change the  www to mbasic  (mobile version of facebook)
press enter and the page will reload, right click on the  video and select save video as
to download the video


0.0.3 artifacts use filenames are for 0.0.2

Can't pull that via script because it's not like https://github.com/KonstantinosPetrakis/open-league-displays/releases/download/${version}/open-league-displays_${version}_amd64.deb

Hello.
It is resolved.


Added missing header to Mixed Reality Remote Rendering CHANGELOG.

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