Adding light contrast theme

Issue: #19865
Image:

Query Editor:

Query Plan

Notebook

Dashboards:

Edit data

Dialog

Wizard

Marketplace:

Table Designer:

Azure Pane:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.002%) to 41.945%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/sql/platform/actions/browser/menuEntryActionViewItem.ts
0
1
0.0%

Files with Coverage Reduction
New Missed Lines
%

extensions/sql-bindings/src/common/azureFunctionsUtils.ts
1
80.65%

Totals

Change from base Build<PHONE_NUMBER>:
-0.002%

Covered Lines:
28727

Relevant Lines:
63599

💛  - Coveralls

@aasimkhan30 Thanks for making the fix. could you please add more screenshots that covers all the UI components?

@alanrenmsft, I am still adding css rules. It is a WIP. I will add screenshots after I am done.

Will convert this to a draft PR

Azure pane icons are not visible:

@alanrenmsft, screenshots added.

@aasimkhan30 does it automatically switches to the corresponding high contrast themes in ADS when you change the Windows theme?

is Azure view the only remaining issue now?

@alanrenmsft , it does automatically switch to light contrast theme and azure viewlet is the only issue right now.

please update the azure pane screenshot.

@alanrenmsft added screenshot

@alanrenmsft, azure icons in connections pane are still broken

Created a vscode PR for it:
https://github.com/microsoft/vscode/pull/175925

@aasimkhan30 go ahead.


Power Apps - user needs

Collecting some user needs/requests based on what I have discussed with users in different forums the past weeks:

Model driven app: developer has no ability to change the design. Many are choosing this so they don't have to make any tough design choices. There is a default on the tenant. How can this be customized?

Canvas app: Many design choices that can be made. Need a starting point, best practices and reusable components. Specifically: template to start with, font, buttons, icons (currently using external icon libraries when they can't find what they need), best practices, guidance for common patterns, component library

Using Figma to design Power Apps apps? Fusion team did this for Power BI, and an image was created and imported into Power BI as a background. How can this work with Power Apps? What if a designer that doesn't know Power Apps designs something in Figma that is impossible to implement in Power Apps. Should designers learn Power Apps instead and do design there?

Per Egil Raugstad and Ruben Gjelsten are Power Apps developers  in the BPA team that have raised some of these things.

Copied text into epic. Closing.


[DRAFT] add biswap to uniswap-forks

Messari Subgraphs Explorer URL
Graph URL - Status: Syncing, it's really slow.
Follow ups:

[ ] PR review complete
[ ] Update development status
[ ] Deploy subgraph to messari/biswap-bsc
[ ] Merge PR

Great job on integrating Biswap!

Made the recommended changes.
A few more things todo

lint issues
adding conditionals for old and new fees (based on block when the fees changed)
debugging subgraph metrics (all financial/volume metrics are currently 0)

@steegecs @jaimehgb ready for final review

Looks like everything still works!

@steegecs pushed updates to resolve PR feedback, all good now?

Also, can you update the PR name to use our guidelines in Contributing.md?
https://github.com/messari/subgraphs/blob/master/docs/Contributing.md#:~:text="fix(%23major)%3B compound,without contract call"
I would do something like -- > feat(#major); Biswap; Add Biswap to Uniswap Forks

@steegecs made changes per recommedations


[GHSA-p8p7-x288-28g6] Server-Side Request Forgery in Request

Updates

Affected products

Comments
https://github.com/developers-portal/Developer

Hi @eyu-dev, do you have reference link to support this change?


When delete a vpc-nat-gateway CR related deployment is not deleted and cannot be created when delete the deployment mannually

Expected Behavior
When delete a vpc-nat-gateway cr, releated deployment will be deleted automatically. And recreate a vpc-nat-gateway with same name the pod can run normally.
Actual Behavior
After deleting the vpc-nat-gateway, the deployment is not deleted. If deleted the manually and recreate the gw, the pod can not running
Steps to Reproduce the Problem

create a vpc-nat-gateway CR
delete the cr
delete related deployment
recreate the cr

Additional Info

kube-ovn version:
1.8.2

实际测试，在删除vpc-nat-gw cr之后，经过30s，会删掉vpc-nat-gw的pod

查看代码提交记录，确认该问题在 1.8.2 版本修复


nosnap: allow -> Event dropped - No snapshot or clip available

Hello
I am experiencing a strange issue with regards to the sending of alerts even if no snapshot is available. The relevant portion of the frigate-notify config is as follows:
## Alerting methods
# Any combination of alert destinations can be configured
alerts:
  # General config applies to all alert methods below
  general:
    # Title for any alert messages (Default: Frigate Alert)
    title: Surveillance Alert
    # Optionally modify default time format in notifications
    # Use Golang's reference time format, or see docs for more info
    timeformat:
    # Allow notification of events without a snapshot
    # Set to `drop` to disallow this
    nosnap: allow
    # Note: Following snap_ items only apply when event is in progress
    # Set to `true` to include bounding box on detected object in snapshot
    snap_bbox:
    # Set to `true` to include timestamp on snapshot
    snap_timestamp:
    # Set to `true` to crop snapshot
    snap_crop:

Although nosnap is set to allow, I still get no alerts with the following log entry:
2024/09/06 17:52:40 +0200 INF New event received event_id=1725637954.289952-d8qyed 2024/09/06 17:52:40 +0200 INF Event Detected camera=tapo_cam_c320 event_id=1725637954.289952-d8qyed label=person zones= 2024/09/06 17:52:40 +0200 INF Event dropped - No snapshot or clip available event_id=1725637954.289952-d8qyed 2024/09/06 17:52:41 +0200 INF Event update received event_id=1725637954.289952-d8qyed 2024/09/06 17:52:41 +0200 INF Event Detected camera=tapo_cam_c320 event_id=1725637954.289952-d8qyed label=person zones= 2024/09/06 17:52:41 +0200 INF Event dropped - Already notified on this zone camera=tapo_cam_c320 event_id=1725637954.289952-d8qyed label=person zones=
Any ideas on how to fix this? Maybe configuration error or even a bug?
Thanks a lot - great project!

Hi there! So at the moment there is one circumstance where this might happen - if there is no snapshot and no clip available via Frigate. This was originally implemented because some users have certain filtering configured on the Frigate side, like setting required_zones on a camera. This would result in Frigate still generating an event, but not recording anything yet - which would still trigger a notification via this app. The config setting is mostly there to change notification preference on snapshot availability only.
Now assuming that the detected object eventually enters a spot where there should be a clip recorded, you should be notified. However, there is an open issue (#121) where that isn't always the case.

Hi there! So at the moment there is one circumstance where this might happen - if there is no snapshot and no clip available via Frigate. This was originally implemented because some users have certain filtering configured on the Frigate side, like setting required_zones on a camera. This would result in Frigate still generating an event, but not recording anything yet - which would still trigger a notification via this app. The config setting is mostly there to change notification preference on snapshot availability only.
Now assuming that the detected object eventually enters a spot where there should be a clip recorded, you should be notified. However, there is an open issue (#121) where that isn't always the case.

Thanks a lot for the quick and extensive reply. Will head over to issue #121 and close this one here.


All fields are set to editable when rendered on the details view

When you navigate to any of the details view, all fields are set to editable while the component edit mode is inherited and the form is in readMode
**Steps to reproduce the behavior:

Go to 'https://function-adminportal-test.shesha.dev/dynamic/Shesha/mobileTest-details?id=1b26e518-73e0-47c8-ad3e-01f12a313f4b'
Check all the fields
Actual Results: All fields are in edit mode while the form is in readMode

Expected behavior
If the component are set to inherit, it should inherit the behaviour of the parent form

Related to #1634


Fix FunctionLikeReflectionParser -> ReflectionAstResolver service and NodeRepository method call usage

as removed in https://github.com/rectorphp/rector-src/pull/298

@TomasVotruba could you verify the tests error https://github.com/rectorphp/rector-nette/pull/19/checks?check_run_id=2916696307#step:5:16 :
1) Rector\Nette\Tests\Rector\ArrayDimFetch\AnnotateMagicalControlArrayAccessRector\AnnotateMagicalControlArrayAccessRectorTest::test with data set #5 (Symplify\SmartFileSystem\SmartFileInfo Object (...))
17
Rector\Core\Exception\NotImplementedYetException: someForm
18

19
/home/runner/work/rector-nette/rector-nette/src/Rector/ArrayDimFetch/AnnotateMagicalControlArrayAccessRector.php:150
20
/home/runner/work/rector-nette/rector-nette/src/Rector/ArrayDimFetch/AnnotateMagicalControlArrayAccessRector.php:121

Thank you.

closing in favor of https://github.com/rectorphp/rector-nette/pull/20


Add Recorder class in typings

Missed Recorder class from typings vmsg.d.ts.
Add, please in Exports interface.

@Kagami could u please check https://github.com/Kagami/vmsg/pull/30, which resolves the issue.

Published as 0.3.6.


Recording - Group Channel/ iOS - Update or delete message by operators

Click ‘Create’ to create new group channel
Under ‘Choose member’, add users & select ‘OK’
Click on ‘Channel Name’ box to input channel name
Name channel ‘Birthday Ideas’
Click ‘Create’
Click on ‘Birthday Ideas’ to enter conversation

Drew: Hey guys, do you think we should get roses or pink lilies for Emily’s birthday?
Daniel: Hm they’re both great options
Nadia: Not sure
Nadia: Oh, I remember she loves lilies so I would go with that!

Hold down my message above as Nadia, pop up will appear to ‘Choose action for message’
Click ‘Update’, and update the message to ‘She loves pink lilies so I would go with that!’
Click ‘Confirm’
Hold down my message as Nadia saying ‘Not sure’ and click ‘Delete’

@justinbaldwin
https://drive.google.com/file/d/1HLHetjZTONE15TFIBGXRnWkIUH12wsPm/view?usp=sharing

@petru-gherghel to re-upload new version.

@justinbaldwin
https://drive.google.com/file/d/1yXbPVMATAQe8GT2Eov-A6LBtfIZ1DEqC/view?usp=sharing

@michellewong27 @chrischabot
Ready for review: https://drive.google.com/file/d/1yXbPVMATAQe8GT2Eov-A6LBtfIZ1DEqC/view?usp=sharing
The final taps on "Joyce" are to show that the current user is not an operator in that specific channel, and thusly cannot make changes to Joyce's post.

@justinbaldwin reviewed, good for publishing


Postgress in docker fails

I'm trying to set up the internal docker server according to the documentation. When I do docker compose up it reports the following
docker-postgresql-1  | fixing permissions on existing directory /var/lib/postgresql/data ... ok
docker-postgresql-1  | creating subdirectories ... ok
docker-postgresql-1  | selecting dynamic shared memory implementation ... posix
docker-postgresql-1  | selecting default max_connections ... 100
docker-postgresql-1  | selecting default shared_buffers ... 128MB
docker-postgresql-1  | selecting default time zone ... Etc/UTC
docker-postgresql-1  | creating configuration files ... ok
docker-postgresql-1  | 2022-05-15 19:11:41.856 UTC [39] LOG:  could not open file "pg_wal/000000010000000000000001": No such file or directory
docker-postgresql-1  | 2022-05-15 19:11:41.857 UTC [39] FATAL:  could not open file "pg_wal/000000010000000000000001": No such file or directory
docker-postgresql-1  | child process exited with exit code 1

and later
docker-atuin-1       | Location:
docker-atuin-1       |     /app/atuin-server/src/lib.rs:24:10
docker-atuin-1       | Error: failed to connect to db: postgres://atuin:password@db/atuin
docker-atuin-1       |
docker-atuin-1       | Caused by:
docker-atuin-1       |    0: error communicating with database: failed to lookup address information: Name or service not known
docker-atuin-1       |    1: failed to lookup address information: Name or service not known

After reading this https://stackoverflow.com/questions/70659742/postgres-fatal-could-not-open-log-file-permission-denied
I ran these commands
chown :999 ./database
chmod 770 ./database
chmod g+s ./database

But now I get
docker-postgresql-1  | chmod: changing permissions of '/var/lib/postgresql/data': Operation not permitted
docker-postgresql-1  |
docker-postgresql-1  | PostgreSQL Database directory appears to contain a database; Skipping initialization
docker-postgresql-1  |
docker-postgresql-1  | 2022-05-15 20:21:47.380 UTC [1] FATAL:  data directory "/var/lib/postgresql/data" has wrong ownership
docker-postgresql-1  | 2022-05-15 20:21:47.380 UTC [1] HINT:  The server must be started by the user that owns the data directory.
docker-postgresql-1 exited with code 1

Now I fixed it: I had to change my docker-compose.yml according to https://github.com/docker-library/postgres/issues/435#issuecomment-387598016
    volumes:
      # Don't remove permanent storage for index database files!
      - "database:/var/lib/postgresql/data/"
    environment:
      POSTGRES_USER: $ATUIN_DB_USERNAME
      POSTGRES_PASSWORD: $ATUIN_DB_PASSWORD
      POSTGRES_DB: atuin

volumes:
  database:

Glad you got this sorted :)


unify api prefix usage

[ ] I added an entry to CHANGELOG.md if knowledge of this change could be valuable to users.

Here is a brief summary of what I did:

 External Address und API Prefix

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

8 of 10 (80.0%)  changed or added relevant lines in 6 files are covered.
115 unchanged lines in 4 files lost coverage.
Overall coverage increased (+0.08%) to 89.386%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

services/src/util/config.rs
4
5
80.0%

services/src/util/server.rs
0
1
0.0%

Files with Coverage Reduction
New Missed Lines
%

operators/src/source/ogr_source/mod.rs
1
95.4%

services/src/pro/users/postgres_userdb.rs
2
95.96%

services/src/pro/contexts/postgres.rs
3
96.54%

operators/src/source/gdal_source/mod.rs
109
94.04%

Totals

Change from base Build<PHONE_NUMBER>:
0.08%

Covered Lines:
96328

Relevant Lines:
107766

💛  - Coveralls

bors merge


What is Prerequisites ?(python version,tensorflow version..)

which version of python and tensorflow should i use?
thanks~~

python 2.7


Poll for the state of the current song instead of pausing the player on startup

With the new connect-based player, the player is immediately paused when it starts up to get into a consistent state.
We could instead poll the state of the player, and use that to initialize the bot state.

SetMod can now be configured to sync the remote state of the player with the following configuration option:
[player]
sync_player_interval = "10s"

You probably wan't to keep the interval high, since it will be calling the Spotify API for every check.

There's currently one known bug:
If you switch the song on the Spotify player, it will be noted in SetMod, but immediately afterwards it will reset the song state so that the track shows as not loaded until it has had an oppurtunity to refresh:

Seems like the bug has been fixed!

Done.


Update io.md

I think there where a line break intended. Currently it looks strange on the website.

:+1:
Thanks!


how to carry the cookies split by ; symbol?

here's a example:
if there are at lease two keys need to be carried such as : {"a":"1","b":"2"}
and then if I code like this:
var cookie = request.cookie('a=1;b=2');
We will find out that the second key might be lost at the request
How to resolve it?

I have the same problem, have you resolved it?

cookie = 'a=1;b=2'
 cookie.split(';').forEach((x) => {
      jar.setCookie(request.cookie(x), 'https://vtop.vit.ac.in/student/stud_login_submit.asp')
})

this is how i added multiple cookie for detailed example check this out
(https://github.com/sch00lb0y/Vit-Photo/blob/master/scripts/lib.js)


Allow up part of containers

I'm using remote db server and in this case it would be good don't start db server for each project where I run warden up -d
It would be good to allow configure services at .env file.
For example don't run db and varnish

This would be good. Usually I run the DB on the host instead of out of the container.

Thanks for the suggestion here. It isn't in a ticket here yet, but I have been mulling over the idea of having environment types start with a default set of flags to indicate what services to run and having each service defined in separate files so they can be included individually (or as you mentioned, disabled entirely by overriding those flags in the .env file)
In the case of varnish, there is a BYPASS_VARNISH flag available which will send Magento 2 requests directly to Nginx vs through the Varnish container if desired (typically good enough since varnish uses almost no resources at all, so not a big deal most of the time for the container to just be running). Maybe check that out if you need to not use varnish on a project.
I've got a few projects which don't use Elasticsearch, which is basically what got me thinking of this initially as I ended up resorting to putting the following bit in the project's .warden/warden-env.yml file to just exit the container on env start:
version: "3.5"
services:
  elasticsearch:
    entrypoint: ["echo", "- service is disabled"]   # exit on env start to conserve memory since ES is not used here
    labels:
      - traefik.enable=false

Had not thought about use cases where one might want to run the database on the host machine (or heck, somewhere else on their local network) rather than in a Warden controlled container. Interesting use case, and I can see where it might make sense.
No promise as to when I may get to this, but I'd love to see it happen, and I'm going to tag this for 0.3.0 (which again, has no ETA, but eventually we'll get there).

See notes over here: https://github.com/davidalger/warden/pull/150


Large responses lost

Large responses will result in multiple data events on the socket, the current code attempts to parse the initial prefix as JSON and fails. Should accumulate and parse the entire input.
I wonder if it's worth pulling out a node/socket library or at least building on top of purescript-node-streams.

The problem with purescript-node-streams is that they are missing the onConnect event... They also seemed to be racy. When I tried accumulating the 'data' events in a buffer I got the close event before all the data was collected... One possibility could be to make the socket linebuffered.

OK, I thought the end event might work but as I read up it doesn't sound great, in this case could just wait for the first newline.
(For reference it was a pursuit response that was large, over 4k-ish chars).

fixed in #7


refactor: Reuse assignment logic for type elements for class members

Description:
Related issue:

Closes https://github.com/dudykr/stc/issues/762.

Closing as I messed up


Customer can pay with S-Pankki after order is canceled and order status is set to "Closed".

Describe the bug
If a customer ends the payment transaction just before the final step of paying with "Tunnuslukutaulukko" on S-Pankki and comes back to the transaction with browser back button it is possible to make a successful payment but the order itself is set to "Closed" status.
To Reproduce

Set Stores -> Configuration -> Sales -> Payment Methods -> Automatically cancel orders if the first response from Paytrail API is a failure setting to yes.
Order something with S-Pankki payment method and "Tunnuslukutaulukko" (Password table?) but push "Poistu" or exit just at the last step of payment.
Push back on browser
Finish payment
Order is set to "Closed" and customer receives mail that the payment has gone through.

Expected behavior
Order should be set to "Paid".

@mannnem: Unfortunately we don't have any control over this behaviour, but in general,  S-Pankki shouldn't process the payment after customer has chosen to exit from the payment flow.

paytrail-for-adobe-commerce/Model/ReceiptDataProvider.php
...
if ($paymentVerified === 'ok') {
    if ($this->currentOrder->getStatus() === Order::STATE_CANCELED) {
        ...
    }
    $this->currentOrder->setState($orderState)->setStatus($orderState);
    $this->currentOrder->addCommentToStatusHistory(__('Payment has been completed'));
...

This problem is not restricted to S-Pankki only. You really should investigate this issue more.


Can't build cardano-db-sync and db-tool with nix on buildkite CI machine

I am trying to build cardano-db-sync and cardano-db-tool in my tests https://buildkite.com/input-output-hk/qa-db/builds/352#01882685-80f4-4ff4-a5f7-759104b1a8fd on Buildkite machine ci-equinix-6-ci6-1
using following commands:
nix-build -v .#cardano-db-sync -o db-sync-node
nix-build -v .#cardano-db-tool -o db-sync-tool
and I am seeing this error:

error: getting status of '/var/lib/buildkite-agent-iohk/builds/ci-equinix-6-ci6-1/input-output-hk/qa-db/db_sync_tests/cardano-db-sync/.#cardano-db-sync': No such file or directory

error: getting status of '/var/lib/buildkite-agent-iohk/builds/ci-equinix-6-ci6-1/input-output-hk/qa-db/db_sync_tests/cardano-db-sync/.#cardano-db-tool': No such file or directory

however cardano-node and cardano-cli were succesfully built with those commands:
nix build -v .#cardano-node -o cardano-node-bin
nix build -v .#cardano-cli -o cardano-cli-bin

Could it be similar issue to this ? https://input-output-rnd.slack.com/archives/CC0CLNEH1/p1683593161898199?thread_ts=1682964148.535219&cid=CC0CLNEH1
CC: @jbgi

Should be fixed by 9a4348752ef36e09b5215e174d510dc2ab18b51f: https://buildkite.com/input-output-hk/qa-db/builds/354#018829fa-3b7c-4b95-bff6-72cc65a71153


Mutant config is not preseeded

I just tried to devtool-ify my gem and it worked like a charm, except one small bump:
config/mutant.yml is preconfigured to your_lib/YourLib. Wouldn't it make sense to try to guess that information from the gemspec? e.g. "my-lib" => My::Lib?

There is no devtools init anymore. Closing this issue.


RFC 0009: Add looping functionality in CFN Templates

Language Enhancement Request For Comment
This is a request for comments about a new intrinsic function called Fn::ForEach to support a looping functionality in CloudFormation templates. See the tracking issue (#9) for
additional details.

Licensing
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I have a number of different pieces of feedback.
Naming
In the issue I originally opened (#41), I proposed Fn::Map; I believe it's important for users to understand this as a functional concept, rather than an imperative one. They aren't writing a for loop, they are mapping a JSON template over a set of values. I'd settle for Fn::Foreach, though. I believe there's also value in a Fn::Reduce (for example, to create a list of ARNs from a collection of resources created with this function), which supports Fn::Map as a name.
Syntax
Index/value parameter names
I don't like the need to define the index and value parameter names. They're probably always going to be the same, it's just one more thing to type out and one more thing to go wrong. Could they be made optional, defaulting to some useful names? This brings me to my next point...
Function parameter ordering
There are so many parameters to this function. I understand that all other intrinsic functions use positional parameters, but maybe this is a place to use named parameters. For example:
!Foreach
  Values: !Ref AmiIds
  Template:
    Type: AWS::EC2::Instance
    Properties:
      InstanceType: "m1.small"
      ImageId: !Ref x
  LogicalId: !Sub Instance${i}
  ParameterNames: ["i", "x"]

This would allow for multiple independently-optional parameters, like LogicalId and ParameterNames. Speaking of LogicalId...
Use cases
I think there's actually four use cases here.

Inserting a list as a value
Inserting an object as a value
Merging an object into a parent
Creating a set of CloudFormation resources

Use cases 3 and 4 are both accomplished by the LogicalId parameter. Use case 2 is not supported.
Here's I'd propose:

Instead of LogicalId and its current semantics, allow a Key parameter that, if set, means the function produces an object rather than a list. By default, this object is the value where the function is defined.
Allow a boolean Merge parameter that requires the Key parameter also be set, that when true, merges the object defined by the function with its parent.
Define a separate syntax for resources. Here's why...

Reifying resource mapping
If this intrinsic function simply allows users to create multiple resources with an intrinsic function that shuffles YAML around, we're missing a giant opportunity.
We should be enabling CloudFormation to represent a collection of resources defined by mapping over some values as a first-class concept.
Say that I want subnets in every AZ. That's an architectural intention that should be fully represented in the cloud. It should get a name, it should be able to inspected, etc.
This proposal doesn't serve that purpose. I think it can be a separate RFC, but imagine something like
Resources:
  Subnets:
    Type: CloudFormation::Map
    Values: [...]
    LogicalId: ... # actual logical id for the resource
    ResourceTemplate: ...
    # etc, similar syntax to the function

If we agree that such a thing should exist, it will help focus the syntax of this intrinsic function on use cases 1-3

Fn::Map or at worst Fn::Foreach make a-lot more sense then just Fn::For.

Naming

Issue #9 describes Fn::For as;

Fn::For traverses a single given List of Strings from first to last, and over each iteration, it references each item under a given template snippet to be replicated.

This definition describes and iterator which implies functions like Map, Reduce, Filter, and Zip.
The term  For may confuse developers since this term has a long history as A control-flow-statement.
I agree with @benkehow Fn::Map communicates the intent better than Fn::For.

Really appreciate all the feedback and engagement on this RFC! After some internal discussions, we've determined it needs significant rework. We're going to iterate a bit internally and then post a new revision for more feedback. Stay tuned!

re-posting this as a general comment so it doesn't get lost on an old revision:
from @benbridts

Things I'm wondering:

Would it be clearer to use IndexParameter and ValueParameter for the keys (to make it clear that you're configuring something you will !Ref later)
Would it be clearer to use Item instead of Value? This might be related to the languages I've been exposed to, but I'd use Key-Value or Index-Item, not using Value prevents confusion with the Key parameter

I think "key" and "value" if they are both used should be outputs, currently "fragment" is used for the output value. We've got four parameters here:

input value from the input collection (currently "value")
index of the input value (currently "index")
output value (currently "fragment")
optional output key (currently "key")

Maybe making them more explicit makes sense? InputIndex, InputValue, OutputValue, OutputKey? It would leave open the possibility that in the future the collection could be an object, which would would then involve an InputKey parameter as well.

Re-posting this as a general comment so it doesn't get lost on an old revision:
Both @benbridts and I commented that we think the parameters should be CamelCase to be consistent with CloudFormation conventions

@MalikAtalla-AWS FYI amending the commit and pushing means comments on previous commits are no longer visible in the diff, I'd recommend creating new commits and only squashing when it's ready for merging

I think "key" and "value" if they are both used should be outputs, currently "fragment" is used for the output value. We've got four parameters here:

input value from the input collection (currently "value")
index of the input value (currently "index")
output value (currently "fragment")
optional output key (currently "key")

(and collection, but that one seems to be clear)

Maybe making them more explicit makes sense? InputIndex, InputValue, OutputValue, OutputKey? It would leave open the possibility that in the future the collection could be an object, which would would then involve an InputKey parameter as well.

the current value and index do not feel like inputs to me. They are more meta-inputs (used within the fragment) than inputs (used by the map function).
Since the current default for Value is X, I would propose something like element, item or variable.
I'm not the biggest fan of using "Output" in the names for fragment and key, but if you translates what happens to python, it makes sense to use Key and Value in the names. And even though output is a fitting name, they are also templates. (template already has a specific meaning in CloudFormation, but I don't think this has to be a problem)
def cfn_map(collection: List, key: str, fragment: str):
    return {key.format(i=i, x=x): fragment.format(i=i, x=x) for i, x in enumerate(collection)}

 # or with different names

 def cfn_map(collection: List, key: str, value: str):
    return {key.format(i=i, x=x): value.format(i=i, x=x) for i, x in enumerate(collection)}

This leads me to thinking something like

IndexName (how you will refer to the index)
VariableName, ItemName or ElementName (how you will refer to the variable)
Collection (the actual collection you are mapping over)
KeyTemplate or OutputKey (the key you will output, that can also reference variables)
ValueTemplate or OutputValue (the value/fragment you will output, that can also reference variables)

Update: The AWS CloudFormation team really appreciates the interest and feedback we've received on this RFC! In the past months, our team did a deep dive to better understand specific customer pain points and brainstormed ideas with customers and community representatives. Based on this deep dive, we are updating this RFC with a proposal to add a single Fn::ForEach intrinsic function. Let us know your feedback and thoughts!

Needed in the FAQ, because people will try this immediately:

Can I use an object for the collection instead of a list?
Can I output a list instead of merge into an object?

Are there outstanding concerns preventing the assigned reviewers from approving this?
I'm looking forward to taking advantage of this capability, and am hoping it can soon move towards implementation.

Sorry to be a pest, but if I'm reading the tracking issue #9 correctly, in order to keep things moving, I think someone needs to add an approved label to this PR, and a status/final-comments-period label to the tracking issue?

You're right @rhbecker. Appreciate the reminder. We'll take care of that.

🤔 Maybe instead of pestering you about labels (which still don't seem quite right on this or the tracking issue), I should instead ask whether implementation is fully unblocked.
My fear is that this issue is stuck in some queue, due to a "paperwork" blockage, when it could otherwise be ready for some hungry CFN engineer to start working on.
If it's already unblocked, or perhaps even already in progress, I can sit back, relax, and patiently (🤞)  await completion.

Hey @rhbecker, thanks for following up. This issue is and has been unblocked. We have set the status of the RFC to "implementing" in the main readme of this repo and I (just now) added the approved-label to this PR. (Maybe we should consider removing that line about the approved label from our readme. The info in the main readme and the fact that the PR is merged is probably indication enough about the status.)


Sample code update for regionalization

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[x] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[x] Ensure the tests and linter pass
[x] Code coverage does not decrease (if any source code was changed)
[x] Appropriate docs were updated (if necessary)

Fixes #28 🦕

@wuyuexin Is it ready for merge?

Yes, please proceed.


Image darkening after updating to 0.32.6

Question about an existing feature
I'm seeing image darkening when moving from 0.31.3 to 0.32.6 or 0.33.3 for both jpeg and png.
I'm using gulp-responsive to generate different sizes and sharp as the image processor.
          {
            quality: 80,
            compressionLevel: 7,
            progressive: true,
            withMetadata: false,
            errorOnUnusedConfig: false,
            skipOnEnlargement: true,
            errorOnEnlargement: false,
            silent: true,
          }

Please provide a minimal, standalone code sample, without other dependencies, that demonstrates this question
The darkening is mostly notable less on photo-like images and more for 2-3 color images.

The gulp-responsive package relies on an out-of-date version of sharp from 2019.
https://github.com/mahnunchik/gulp-responsive/blob/8091ebc2fd9d8fe757201600f63c3653446147bd/package.json#L49
Please can you provide the original, input images and sample code using only sharp, without any other dependencies, that allows someone else to reproduce.

@lovell Thanks for the response. When using only sharp I do not see the image darkening effect.
Thanks for the response.

The reason it darkens is because gulp-responsive (and the more up-to-date version @localnerve/gulp-responsive) calls image.normalize(config.normalize) even when config.normalize is false. I imagine the normalize function must have accepted a boolean at some point, and now it doesn't.


Upgrade SDK version

Upgrade SDK version to v4.37.0

@andreeaceachir142 This should probably be upgraded to 4.37.1.

Done ✅ @ammapspeople


not able to unlock car

First of all very good work! i've very appreciate this plugin!
I'm trying to unlock my car using this plugin with no success.
I'm able to lock but not to unlock.
Seems that something is going wrong with PIN. after several tries i had to reset my pin using mercedes me app.
Below you can find an example of the error from log file
thanks
2019-06-07 22:13:23 ERROR (Thread-2) [custom_components.mercedesmeapi.apicontroller] Connection failed with http code 403
NoneType: None
2019-06-07 22:13:23 ERROR (MainThread) [homeassistant.components.websocket_api.http.connection.140373626142672] 'NoneType' object has no attribute 'get'
Traceback (most recent call last):
File "/var/lib/homeassistant/lib/python3.5/site-packages/homeassistant/components/websocket_api/commands.py", line 121, in handle_call_service
connection.context(msg))
File "/var/lib/homeassistant/lib/python3.5/site-packages/homeassistant/core.py", line 1150, in async_call
self._execute_service(handler, service_call))
File "/usr/lib/python3.5/asyncio/futures.py", line 380, in iter
yield self  # This tells Task to wait for completion.
File "/usr/lib/python3.5/asyncio/tasks.py", line 304, in _wakeup
future.result()
File "/usr/lib/python3.5/asyncio/futures.py", line 293, in result
raise self._exception
File "/usr/lib/python3.5/asyncio/tasks.py", line 239, in _step
result = coro.send(None)
File "/var/lib/homeassistant/lib/python3.5/site-packages/homeassistant/core.py", line 1172, in _execute_service
await handler.func(service_call)
File "/var/lib/homeassistant/lib/python3.5/site-packages/homeassistant/helpers/entity_component.py", line 194, in handle_service
required_features
File "/var/lib/homeassistant/lib/python3.5/site-packages/homeassistant/helpers/service.py", line 316, in entity_service_call
future.result()  # pop exception if have
File "/usr/lib/python3.5/asyncio/futures.py", line 293, in result
raise self._exception
File "/usr/lib/python3.5/asyncio/tasks.py", line 241, in _step
result = coro.throw(exc)
File "/var/lib/homeassistant/lib/python3.5/site-packages/homeassistant/helpers/service.py", line 337, in _handle_service_platform_call
await getattr(entity, func)(**data)
File "/usr/lib/python3.5/asyncio/futures.py", line 380, in iter
yield self  # This tells Task to wait for completion.
File "/usr/lib/python3.5/asyncio/tasks.py", line 304, in _wakeup
future.result()
File "/usr/lib/python3.5/asyncio/futures.py", line 293, in result
raise self._exception
File "/usr/lib/python3.5/concurrent/futures/thread.py", line 55, in run
result = self.fn(*self.args, **self.kwargs)
File "/var/lib/homeassistant/.homeassistant/custom_components/mercedesmeapi/lock.py", line 69, in unlock
self._data.unlock(self._vin, "2568")
File "/var/lib/homeassistant/.homeassistant/custom_components/mercedesmeapi/apicontroller.py", line 295, in unlock
return self._execute_car_action(CAR_UNLOCK_URL, car_id, 'car unlock', pin)
File "/var/lib/homeassistant/.homeassistant/custom_components/mercedesmeapi/apicontroller.py", line 337, in _execute_car_action
if result.get("status") == 'PENDING':
AttributeError: 'NoneType' object has no attribute 'get'

Hi,
could you please recheck. I've fixed the PIN issue.
BR
Rene

Works!!! Fantastic!!
Thanks to much!
Il Mar 11 Giu 2019, 11:16 Rene Nulsch<EMAIL_ADDRESS>ha scritto:

Hi,
could you please recheck. I've fixed the PIN issue.
BR
Rene
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ReneNulschDE/mbapipy/issues/1?email_source=notifications&email_token=ALJM7A6LDZSTESTDDYSUGS3PZ5UQJA5CNFSM4HV2ATB2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXMPUBY#issuecomment-500759047,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALJM7A47JYCNKEVJDSO6XILPZ5UQJANCNFSM4HV2ATBQ
.


⚠ Processors has degraded performance

In 8c68bd9, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 53 ms

Resolved: Processors performance has improved in 9e4e431.


Automate New Relic notifying customers of platform outages

Background
As it stands our only methods of communicating platform outages to customers are via the status page and team members responding to zendesk tickets/slack responses.
The pages team currently receives platform notifications strictly via email.
Create an automation in New Relic that upon the Federalist web app (production)  being down longer than x > amount of time, directly notifies customers via one of our public pages slack channels.

[ ] Create an new alert condition to watch our federalist web app in our existing synthetic monitor
[ ] Configure signal behavior
[ ] Define thresholds
[ ] Connect this alert condition to an existing policy or create a new policy (create a new one for testing purposes)

New Relic recommends creating a policy for each separate destination or audience needs to receive a notification.

[ ] Either amend the existing  federalist web  policy workflow or create a new one
[ ] Request New Relic app be installed to TTS slack org
[ ] Add slack channel to alert policy in New Relic

Security Considerations
Do not incorporate any metrics but simply a status message

Pending GSA security/slack admin approval - Self Service Ticket

Blocked until further notice per GSAIT - " The ticket is currently waiting approval from GSA internal software testing team (ISE.Software.Testing queue) and has not been assigned yet. The process normally takes a long time. Nothing can be done to expedite the process which can last from weeks to months long. 'Pending review by the software team' is one of the first steps in the process. Once it is approved it will go to a team who will assign a tech or several techs depending on the scope of work to do the vetting for government use. "

GSA ise-software-testing reached out with follow up questions

"What is the New Relic application, and who is the FISMA System Owner? We will need their written concurrence of this integration."
"What is the intended Slack instance to be integrated?"
"What is the purpose of the integration? Outages of what?"
"Per the link provided, this would appear to support (require?) OAUTH integration? We need more details about this, the what and why as well please."

I relayed the required information back to them via email same day (7/17)


check_es.py gets privilege error when run from my laptop

When I try to run tests/test_fiod.sh from my laptop, I get the error in this pastebin.  The relevant part is
elasticsearch.exceptions.AuthorizationException: AuthorizationException
(403, 'security_exception', 'action [indices:admin/refresh] is unauthorized for user [snafu]')

So there was a call added to tests/check_es.py
    es.indices.refresh(index=index)

That is causing this error (Thx @jtaleric ).   The line was added by @rsevilla87 .   When you remove this line, the test passes.   However, this is not in the equivalent benchmark-wrapper/ci/check_es.py script.   (Why can't snafu CI use the check_es.py script in benchmark-operator, since it clones it?).   Anyway,  the point is: do we really need to refresh the indexes every time we run check_es.py?     Maybe we do, I don't know.   If we do, don't we need to add this priv to the snafu user in the ES server so that anyone can run the CI test?   Not sure why the CI itself doesn't hit this problem.

@bengland2 So a few comments in no particular order:

The benchmar-wrapper/ci/check_es.py script is no longer used. We currently only use the one in benchmark-operator. I can take an AI to remove the old file and function.
The refresh was added because we were constantly getting failures from the check script because the data wasn't available in ES yet. By adding that command it consistently works when the data is indexed.
I'm not sure what the privilege problem is TBH. It has been executing in this fashion for months and we have not seen any issue from it.

@bengland2 you're using the check_es.py with the production ES instance and this script is meant to be used from the CI.

again, it doesn't say in the docs.   We need to tell developers how they are supposed to do this.  I searched a CI log and found:
export ES_SERVER=search-cloud-perf-lqrf3jjtaqo7727m7ynd2xyt4y.us-west-2.es.amazonaws.com

That hardly looks like it's cast in stone.   where does this come from?   So how are people supposed to know?   Are they supposed to set up their own ES server or use this one?   When setting up an ES server, what is the relevant privilege that they need to add?   I didn't find it in kibana.
re: "I'm not sure what the privilege problem is TBH. It has been executing in this fashion for months and we have not seen any issue from it".
The error is highly reproducible with the production server.   Would a default setup for ES cause this problem?
should there be DNS CNAME entries defined that point to the production server and the CI server?   That way, you can change where they really live without disrupting source code, and you can document where these ES servers are using the CNAMEs.

again, it doesn't say in the docs. We need to tell developers how they are supposed to do this. I searched a CI log and found:
export ES_SERVER=search-cloud-perf-lqrf3jjtaqo7727m7ynd2xyt4y.us-west-2.es.amazonaws.com

That hardly looks like it's cast in stone. where does this come from? So how are people supposed to know? Are they supposed to set up their own ES server or use this one? When setting up an ES server, what is the relevant privilege that they need to add? I didn't find it in kibana.

IMHO, yes. Users outside of RH should be deploying and maintaining their own ES server.

re: "I'm not sure what the privilege problem is TBH. It has been executing in this fashion for months and we have not seen any issue from it".
The error is highly reproducible with the production server. Would a default setup for ES cause this problem?

No. A default ES setup would basically let the world write/read/delete/blowup... Hence our production ES server being a bit more picky.

should there be DNS CNAME entries defined that point to the production server and the CI server? That way, you can change where they really live without disrupting source code, and you can document where these ES servers are using the CNAMEs.

meh - maybe... Eventually? We don't really have staff to maintain our resource today. We have replication from our Public Prod ES instance to our internal instance... However, the distinction of prod vs testing isn't clear.

OK, well in that case, this issue will serve to document how the system works today and keep people from getting confused about this error like I was.


Improve parallelogramic basis

Currently, the parallelogramic basis option is poorly behaved, in the sense that as we increase the called-for approximate_num_terms, the actual number of terms in the expansion does not monotonically follow.
Here, we improve the logic for parallelogramic basis, and add tests which confirm the monotonic characteristic. These tests would have failed with the previous implementation. The same test is added also for the circular basis; this passes with the logic as-is.
The uLED regression test is updated, since the new basis logic results in a different expansion used during the test.

we rename primitive_lattice_vectors to reciprocal_vectors

This is a breaking change, so we'll want to version bump.

(@mfschubert if we are following semantic versioning, than a API-breaking during prerelease should technically bump this from 0.3->0.4)

(@mfschubert if we are following semantic versioning, than an API-breaking during prerelease should technically bump this from 0.3->0.4)

Yup! I ended up restoring the original naming convention, so we are good there.


#271- remove oracle dependency

This PR removes ojdbc8 dependency which was causing the build to fail if oracle driver was not installed in local repo. It seems TestCopy.groovy was using this jar, but was  commented out but the pom file was not updated. Instead of testing with oracle driver, it would be appropriate use an im-memory database like h2 for testing.

The project is using HyperSQL http://hsqldb.org/ as the in-memory database for testing. However, the Oracle dependency was added in order to make connections to Oracle databases. It's intended for usage outside of testing and should remain in the project. Please follow the steps in https://github.com/jdereg/n-cube/issues/271 to get the jar added locally for you.

I am closing this PR for now. I'll create a new PR later with some refactoring for your review. I think this project has grown bigger


[Bug]: could not yarn add

Bug expectation
Hi, I tried to run:
yarn add puppeteer

and got this error
error An unexpected error occurred: "ENOENT: no such file or directory, symlink '../../lib/cjs/main-cli.js' -> '/Users/hlau/Library/Caches/Yarn/v5/npm-@puppeteer-browsers-0.3.1-21d43c49f6f8bb5fa905b322d73c11fca9b515bd/node_modules/@puppeteer/browsers/.bin/@puppeteer/browsers'".

any idea why? because i googled around and don't see any results
@OrKoN @Lightning00Blade
does this error message look familiar to you? Because I see you made edits in the last few days related to @puppeteer/browsers
Bug behavior

[ ] Flaky
[ ] PDF

Minimal, reproducible example
yarn add puppeteer -W

Error string
error An unexpected error occurred: "ENOENT: no such file or directory, symlink '../../lib/cjs/main-cli.js' -> '/Users/hlau/Library/Caches/Yarn/v5/npm-@puppeteer-browsers-0.3.1-21d43c49f6f8bb5fa905b322d73c11fca9b515bd/node_modules/@puppeteer/browsers/.bin/@puppeteer/browsers'".
Puppeteer configuration
No response
Puppeteer version
latest
Node version
16
Package manager
yarn
Package manager version
1.19.0
Operating system
macOS

Looks like a bug in yarn? It works for me with yarn 1.22.19.


Only support PHP 7.0 and above

Hi,
This pull requests closes #11 .
Here is a list of things I've changed:

Require PHP version 7.0 and above;
Changed old array() syntax to more modern [];
Updated PSR-0 autoloading to PSR-4;
Updated to PHPUnit version 6.0 and included in dev dependancies;
Made changes to CI (run phpunit from vendor, removed php 5.3 from build matrix).

Let me know what you think about this.
I've not changed any interfaces, but potentially this can be done. Strict typing could be introduced if you think it makes sense.

@martinssipenko I'm happy to merge this as-is without strict typing. Are you happy with it? Or would you like to do something else before doing so?

I think it's really your call wether or not to implement strict typing. As I see it, this would be the perfect time, given new major version is required.

@martinssipenko Sadly, I've not done anything at all with strict typing. So I don't have a lot to go on. That said, if you are capable of adding it, and willing to do so, we might as well.

The only downside is that the interface will change, and users might need to update their code. Given this, if you think it makes sense, I'm happy to add it.

The only thing I'm not sure about is how strict typing would handle the mixed stuff. If that is easily handled/not a problem, then it is probably fine to update the interface for a major version upgrade. Will mixed be a problem?

Not sure, so perhaps lets leave it out for now.

Go ahead and play with 2.0 dev for awhile and make sure it still works for you and then we can cut a tag for 2.0.0 if it all looks good.

All the tests are green, so I'm assuming it should work as before. Also, no changes in logic were introduced. Simply replaced array() with [], and if there would have been a bug introduced tests would have caught it.

Version 2.0.0 should now be available: https://packagist.org/packages/dflydev/dot-access-data

Thank you! 👍


Generate Expires attribute along MaxAge one so IE can honor it, close #1466

Motivation:
Internet Explorer doesn't honor Set-Cookie header Max-Age attribute. It only honors the Expires one.
Modification:
Always generate an Expires attribute along the Max-Age one.
Result:
Internet Explorer compatible expiring cookies. Close #1466.

AUTOMATED MESSAGE for ADMINS:Please verify and accept the pull request.To accept, say @netkins acceptTo build again, say @netkins buildTo whitelist the author, say @netkins whitelist

@netkins whitelist

@netkins accept

@slandelle thanks... Cherry-picked into 4.0 (da4029de00a96a6097ecd8b5b88767a10424a1e4)

Just for my information: why don't you guys never merge PRs but cherry-pick instead?

Keeps a cleaner history

Am 25.01.2015 um 21:02 schrieb Stephane Landelle<EMAIL_ADDRESS>Just for my information: why don't you guys never merge PRs but cherry-pick instead?
—
Reply to this email directly or view it on GitHub.

OK :)


[Bug] [dolphinscheduler-api] Edit a correct MySql datasource, "test connect" failed.

Search before asking

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

What happened

Edit a correct MySql datasource (saved before and passed verification), and then click the "test connect" button, but can't connect the MySql datasource.
See the part of "How to reproduce" for details.

What you expected to happen

Edit a correct MySql datasource, and then click the "test connect" button, return to connect successfully.

How to reproduce

Create a correct MySql datasource, and save.
Restart the dolphinscheduler-api server. (must do this step)
Edit the MySql datasource, and click the "test connect" button.
Connection failed

Anything else

When edit an MySql datasource, the front-end page post a "test connect" request to the dolphinScheduler-api server. The request body that is object of BaseDataSourceParamDTO contains a null password field.
The object of BaseDataSourceParam will be verified before initializing the data source client. If the password is null a default password (123456) will to set.
The verification code is here.

Version
dev
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

You need to re-enter the password when editing the data source

You need to re-enter the password when editing the data source
I see, thanks for your reply.
In my personal opinion, this way will cause some confusion to users.
In another case, if the database does not have a password, how to configure the data source? (If the password is null a default password (123456) will to set.)


update copyright

The copyright of the site is updated. I’m not sure what sense it makes to add them to the files, but it’s good to update it.
As suggested in https://github.com/expressjs/expressjs.com/pull/1698#discussion_r1858643033
cc: @expressjs/express-tc

As I understand it, the copyright would belong to OpenJS. If they recommend adding the following text in the footer:

ref: https://github.com/openjs-foundation/artwork#copyright-notices-for-project-website-footers
and maybe the year should be from 2019 to the present, perhaps it would be good to clarify this in the CPC?


[Input] Fix input shrink issue in Firefox

This PR fixes #9304
From: CSS Flexible Box Layout Module Level 1 - 7.1.1. Basic Values of flex:

By default, flex items won’t shrink below their minimum content size (the length of the longest word or fixed-size element). To change this, set the min-width or min-height property. (See §4.5 Automatic Minimum Size of Flex Items.)

Therefore, if the browser is matching the specification, the input element won't shrink if min-width isn't specified. See https://bugzilla.mozilla.org/show_bug.cgi?id=1108514.
This commit simply adds the min-width attribute.

@t-cst Thank you :)


Removal of serialize and unserialize methods from Client and Storage

Removal of serialize and unserialize methods from Client and Storage, fixes #115
Pull Request description:
Really wasn't too sure about this one. Client was fine to remove the methods from but had implications for ClientBase (removal of $serializeClient), which then had implications for Storage.
I have removed Storage Serialization for the fact that the data can be received from the extending Collection class with the all method, then you can remake the Storage object passing back in the data you stored and with a fresh client object.
My main aim here is to make sure things like Message are still serializable, this was a big thing for me a while ago and I can still see many use cases for it.
Let me know if I should make Storage implement \Serializable again without the unserialize method - I'm not 100% sure on what to do here but thought I'd give it a shot!
I haven't actually tested this, yet, but I will at some point soon (< 48 hr) if you don't get around to it first. Was hoping for some feedback in the meantime.
Why should this Pull Request be merged:
Fixes #115
Semantic Versioning Classification:

[x] This PR includes major breaking changes (such as method changes)
[ ] This PR includes only documentational changes (such as docblocks, README, etc.)

Thank you! Will be merged with the other PR at the same time.


Sharing links doesn't work

The url in the browser window doesn't appear to work for anyone except the person owning the files.
The urls look something like this (but this is a bogus url -- values changed to protect PII):
http://www.driveplayer.com/#fileIds=0BybUoysTQAz6V2RoZjgzT0lDR2c,0BybUoysTPRz6cUcza7h5cUtWelE&userId=104748736614019053950
All I get is a screen full of "This file cannot be read or does not exist".  But the mp3 files in question are shared with "public" with "view only" permissions.  And the normal drive viewer works for playing them from other accounts.

Looks like this is a dup of #120.


DEVC: Delete empty packages

Follow up on #927
Package deletion is not trivial as stated here:
https://github.com/larshp/abapGit/blob/307c1395d2f64ddb5b191363fc674afa94f71d8f/src/objects/zcl_abapgit_object_devc.clas.abap#L211-L218
I think it should at least be possible to delete empty packages.

guess this can be closed


Use STIX 2.1 for objects

The representation of the data objects should conform to a standard such as STIX 2.1 for cyber-observables. This will provide interoperability and compatibility along with semantic meaning.

Thank you for your suggestion, I welcome any help and contributions to improve ATLAS.


Update client to sync with schema

Update client to use schema  5.2.0 for some new features for filters

@tschaffter  I used npm run generate:client ../../rocc-schemas/openapi.yaml:
> npm run remove:src:dirs && openapi-generator-cli generate -g typescript-angular -o projects/rocc-client/src -c projects/rocc-client/config.yaml -i $npm_config_specification "../../rocc-schemas/openapi.yaml"
> rocc-client@ remove:src:dirs /Users/rchai/Clone/rocc/rocc-app/rocc-client-angular
> rm -fr projects/rocc-client/src/api projects/rocc-client/src/model

Download 5.2.0 ...
Downloaded 5.2.0

It seems like it's downloading the schema 5.2.0? I updated the description to specify version of schema 5.2.0.

@rrchai It's OK to use schema 5.2.0. This is actually referring to the schema of the OpenAPI generator configuration.


Adaptive environment check

I think using an adaptive environment check instead of browser check would result in a much better stack processing by unknown environments. There are some traits by stack generation the framework should check. E.g. first frame contains error name and message, last frame is empty, there are empty frames, captureStackTrace is supported, the stack is in a different property e.g. error.stacktrace, and so on... Checking environment traits will result in a much better future environment support as well.

I already thought about this in #69 , but I decided to use the yadda shim approach, which does not suite to this error module.

Closing this in favor of #94 , since I moved the error api to a separate project...


Profile folder for Chrome and OperaGx fixed for windows

Function getProfiles() has been added in browser_windows.go to identify the path of the Chrome profiles.

operaGXProfilePath changed to /AppData/Roaming/Opera Software/Opera GX Stable/.

Before:

After:

LGTM, Thx for your PR.

LGTM, Thx for your PR.

thx!


Using C# to call CBC in Google OR Tools

I noticed that you can solver linear problems using C# with Google OR Tools. But I can't seem to find examples where you can solve integer problems using C# with Google OR Tools. The only example they have is in Python to call the CBC engine. Can you use C# to call CBC or SCIP?
Thanks!
Yong

examples/csharp/csintegerprogramming.cs demonstrates how to call CBC/SCIP
from C#.
Now, to use SCIP, you need to compile or-tools yourself, for licensing
reasons.
--Laurent
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
2016-05-19 12:42 GMT-07:00 yongSolver<EMAIL_ADDRESS>
I noticed that you can solver linear problems using C# with Google OR
Tools. But I can't seem to find examples where you can solve integer
problems using C# with Google OR Tools. The only example they have is in
Python to call the CBC engine. Can you use C# to call CBC or SCIP?
Thanks!
Yong
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/google/or-tools/issues/186

Cool! Thanks very much Laurent.


Scatter example 'unhovered' style not applied first

In scatter plot mark example, the “Adding tooltip and custom hover style” example - the initial drawing has opacity 1, but after a single hover, opacity goes to the unhovered style of opacity 0.5.

This is not a bug. When we do not hover on anything, both hovered_style and unhovered_style should not apply. Just like, when selected is None, which means we are not even trying to select, unselected_style is not applied.

So the opacity should go back to 1 after I stop hovering?

I would have expected it to. But it doesn't as of now. So we'll keep this open till that is fixed.

This is fixed in #425


Edited content for layout. Added new footer, github link.

LGTM.
cc @arthurevans to check the landing page content.


Git Commit Ament caused "Git: connect ENOENT" error

Type: Bug

Work with IDE some period of time.
Try to make Git Commit Amend.
Got "Git: connect ENOENT /var/folders/_x/pjn2dmxx0tl9dgnhgryk54n80000gp/T/vscode-git-299f631f9b.sock".

VS Code version: Code 1.82.1<PHONE_NUMBER>d557a81c9d0b5f8a5a1e9274db5585, 2023-09-08T08:49:22.656Z)
OS version: Darwin arm64 22.6.0
Modes:

System Info

Item
Value

CPUs
Apple M1 Pro (10 x 24)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
3, 3, 3

Memory (System)
32.00GB (0.79GB free)

Process Argv
--crash-reporter-id 85479485-0e20-4bb1-8875-f05e0b4bf5e2

Screen Reader
no

VM
0%

Extensions (31)

Extension
Author (truncated)
Version

markdown-mermaid
bie
1.19.0

java-pmd
cra
0.6.3

xml
Dot
2.5.1

es7-react-js-snippets
dsz
4.4.3

prettier-vscode
esb
10.1.0

go
gol
0.39.1

rest-client
hum
0.25.1

Kotlin
mat
1.7.1

git-graph
mhu
1.30.0

vscode-docker
ms-
1.26.0

black-formatter
ms-
2023.4.1

python
ms-
2023.16.0

remote-containers
ms-
0.309.0

fabric8-analytics
red
0.7.0

java
red
1.22.1

vscode-yaml
red
1.14.0

rust-analyzer
rus
0.3.1657

vscode-checkstyle
she
1.4.2

code-spell-checker
str
3.0.1

code-spell-checker-russian
str
2.1.0

even-better-toml
tam
0.19.2

pdf
tom
1.2.2

intellicode-api-usage-examples
Vis
0.2.8

vscodeintellicode
Vis
1.2.30

vscode-gradle
vsc
3.12.7

vscode-java-debug
vsc
0.54.0

vscode-java-dependency
vsc
0.23.1

vscode-java-pack
vsc
0.25.14

vscode-java-test
vsc
0.39.1

vscode-maven
vsc
0.42.0

volar
Vue
1.8.11

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
vscoreces:30445986
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
pythonvs932:30410667
vsclangdc:30486549
c4g48928:30535728
dsvsc012:30540252
pynewext54:30695312
azure-dev_surveyone:30548225
vscccc:30803845
2e4cg342:30602488
89544117:30613380
showlangstatbar:30737416
962ge761:30835153
03d35959:30757346
pythonfmttext:30731395
fixshowwlkth:30771522
showindicator:30805244
pythongtdpath:30769146
i26e3531:30792625
pythonnosmt12:30797651
pythonidxpt:30805730
pythonnoceb:30805159
dsvsc013:30795093
dsvsc014:30804076
diffeditorv2:30821572
dsvsc015cf:30829746

Same here:
> git -c user.useConfigOnly=true commit --quiet
→ No staged files match any configured task.
Error: connect ENOENT /var/folders/_v/rdq8cmks24jgvtkk5sypbgw00000gn/T/vscode-git-0698a41bdc.sock
    at PipeConnectWrap.afterConnect [as oncomplete] (node:net:1494:16) {
  errno: -2,
  code: 'ENOENT',
  syscall: 'connect',
  address: '/var/folders/_v/rdq8cmks24jgvtkk5sypbgw00000gn/T/vscode-git-0698a41bdc.sock'
}
error: There was a problem with the editor '"/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/git/dist/git-editor.sh"'.
Please supply the message using either -m or -F option.

Same problem in 1.82.2

I've noticed, for me at least, the error only occurs if you leave the commit message blank. If you write a new commit message before pressing amend it works.
So maybe the error happens with the file that opens where it prompts you to edit the previous commit message?

It seems no issue anymore in Version: 1.83.0 according to tested myself and it looks gone

It seems no issue anymore in Version: 1.83.0 according to tested myself and it looks gone

I'm sad to find out that it is not fixed on my side with macOS 14.0.

Oh that's interesting. You are totally same environment with me 😭
> code --version
Version: 1.83.0
Commit: e7e037083ff4455cf320e344325dacb480062c3c
Date: 2023-10-03T16:13:10.518Z (5 days ago)
Electron: 25.8.4
ElectronBuildId: 24154031
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 23.0.0

Same issue on WSL. Can't even do code --version:
code --verson
Ignoring option 'verson': not supported for code.
Unable to connect to VS Code server: Error in request.
Error: connect ENOENT /run/user/1000/vscode-ipc-affbd5aa-60a5-48a6-a26a-1bee68760a6a.sock
    at PipeConnectWrap.afterConnect [as oncomplete] (node:net:1494:16) {
  errno: -2,
  code: 'ENOENT',
  syscall: 'connect',
  address: '/run/user/1000/vscode-ipc-affbd5aa-60a5-48a6-a26a-1bee68760a6a.sock'

@unional In my case quitting and relaunching VSCode helps.

Thanks. I only encounter this once but will keep that in mind and try that next time.

@unional In my case quitting and relaunching VSCode helps.

Just encounter this again. And quitting VScode does not fix it.

Encounter it again. Seems like it happens quite often.
Version: 1.83.1 (user setup)
Commit: f1b07bd25dfad64b0167beb15359ae573aecd2cc
Date: 2023-10-10T23:48:05.904Z
Electron: 25.8.4
ElectronBuildId: 24154031
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621

I had the same issue and the problem on my side was that $XDG_RUNTIME_DIR did not exist.
For some reason the WSL Ubuntu set it to a non-existent /run/user/1000.
I fixed the error by creating the directory like this:
sudo mkdir /run/user/1000/
sudo chown 1000:1000 /run/user/1000/

and issuing a Reload Window.
I think there is some error in some clean-up logic that accidentally removes the folder instead of the socket, which would explain why it breaks intermittently. Just a theory, I didn't have the time to investigate.

Changing permissions on /run/user/1000 fixed error for me

I did not see that problem anymore in 1.86.2.

I did not see that problem anymore in 1.86.2.

Yes, I think this issue may have been fixed.

I also encountered this problem with VS Code 1.90.0 on Debian 12.
There was a different .sock file in /run/user/1000/ than the one VS Code expected. Reloading the VS Code window fixed it for me.

As @Tazmondo mentioned, in my case as well it occurred in correlation with editing last commit with "amend" (it happened once, I haven't used this option since so it's an unconfirmed info). Can we have it fixed to use "amend" safely? Or some workaround?

running below fixes it for me
pkill -f 'code'

Reload Window or Quit Visual Code Studio isn't always sufficient, in my case, i confirmed the /run/user/1001/vscode-ipc-...d61.sock file didn't even exist. so i think actually force killing all vscode processes forces vscode to recreate the socket connection file

Just ran into this. Not making learning git any less challenging:
Version: 1.94.2
Commit: 384ff7382de624fb94dbaf6da11977bba1ecd427
Date: 2024-10-09T16:08:44.566Z
Electron: 30.5.1
ElectronBuildId: 10262041
Chromium: 124.0.6367.243
Node.js: 20.16.0
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 24.0.0

Same here. Getting this error (probably since 1.95.3) a couple of times per week. It worked flawlessly before.


Rag/Web requests have empty user_agent header causing rejection from some web sites

Bug Report
Description
Bug Summary:
Rag/Web requests (#url..) have empty User-Agent header causing some web sites denying the request
Expected Behavior:
Either a fix User-Agent or random User-Agent to imitate a real browser
Note
langchain_community.WebBaseLoader class used to fetch web pages has support for generating fake user agents using fake_useragent package in default use case.  Installing this package is enough to start generating user agents.

Following PR adds necessary package to requirements.txt.  Tested with docker image
#1023


Error

https://mclo.gs/kuklERi
I don't know the circumstances to reproduce.

@matipoirierg

might be fixed by 388b682a886a9d27ec1520469dbfc6d93d7a338e


package: Support customized dump naming

Customize a dump name by renaming the original
default name.
Tested:
Verified that BMC Dump is getting generated
and renamed as follows.
BMCDUMP..<dump_id>.
Change-Id: I5d51693495ef4bf3fc0eff04c03b11cb9e4c225b

Can one of the admins verify this patch?


Restore project_id from analytics.json & improve analytics.json handling

Restore project_id from analytics.json if it is not found in meltano.yml.
Overwrite analytics.json with a new one if it is malformed.
Handle more errors when dealing with analytics.json (e.g. JSONDecodeError).
Add additional debug logging.
Do not update meltano.yml from ProjectContext (n.b. this would cause the project_id key to end up inside of the config of the active environment instead of the top-level config).

CI is failing with an error while setting up the fixtures, but it's passing locally on my computer. Will look into this later today.

CI is failing with an error while setting up the fixtures, but it's passing locally on my computer. Will look into this later today.

👍 fwiw - was gonna ask about getting some explicit tests around state change behavior anyway. Right now we've got the one sort of canary test we added the last time around, but might be nice to get some explicit test cases to verify our behavior for:
(off the top of my head)

When there is an analytics.json and its "valid" (project id and client id match as expected)

Verify behavior when we have a project id in meltano yaml and tracking changes from enabled to disabled and vice versa.
Verify behavior when we lack a project id in meltano yaml and tracking changes from enabled to disabled and vice versa.

When there's not yet an analytics.json

Verify behavior when we have a project id in meltano yaml and tracking changes from enabled to disabled and vice versa.
Verify behavior when we lack a project id in meltano yaml and tracking changes from enabled to disabled and vice versa.

When there's an analytics.json but its contents (client id, project id) don't match the active project as if someone had copied an existing project skeleton.

Verify behavior when we have a project id in meltano yaml and tracking changes from enabled to disabled and vice versa.
Verify behavior when we lack a project id in meltano yaml and tracking changes from enabled to disabled and vice versa.

@aaronsteers in addition to these 3 scenarios, any other iffy/tricky ones we should have explicit tests for?

When there's not yet an analytics.json

Verify behavior when we have a project id in meltano yaml and tracking changes from enabled to disabled and vice versa.
Verify behavior when we lack a project id in meltano yaml and tracking changes from enabled to disabled and vice versa.

@pandemicsyn I don't understand what you mean by "tracking changes" here. With no analytics.json, there is nothing to compare the current telemetry settings to, so we cannot identify a change. As implemented on main and in this PR, in the absence of analytics.json no telemetry state change event is ever fired.

@pandemicsyn I don't understand what you mean by "tracking changes" here. With no analytics.json, there is nothing to compare the current telemetry settings to, so we cannot identify a change. As implemented on main and in this PR, in the absence of analytics.json no telemetry state change event is ever fired.

Sorry, "tracking changes" here means a user actually changes there their tracking opt-in setting, but you're right this would apply to basically any command run after first updating 2.0.  Regardless of whether the opt-in setting changed. The behavior you described is what we should be verifying.   We should test that if there's no analytics.json and a user executes some command that no change events are fired and that an analytics.json gets created with the expected entries.

@pnadolny13 @WillDaSilva this PR still fails test that @pnadolny13 describes in https://github.com/meltano/internal-general/issues/400 because we still fire an event when the project id changes , even if the saved send_anonymous_usage_stats entry is set to false.
So we need a guard statement and matching test that:

If the project id between meltano and analytics.json differs, AND the previous analytics.json entry's send_anonymous_usage_stats is set to False, then we should NOT fire a state change event.

@aaronsteers @pandemicsyn What's up with the tracking_disabled setting? It was added to the new Tracker class, but I don't see it referenced anywhere else. Can it be removed? It's the same as send_anonymous_usage_stats, but negated. It's only respected in one place in the codebase.

What's up with the tracking_disabled setting? It was added to the new Tracker class, but I don't see it referenced anywhere else. Can it be removed? It's the same as send_anonymous_usage_stats, but negated. It's only respected in one place in the codebase.

I've gone ahead and removed it. Let me know if I shouldn't have done that.

@pnadolny13 @pandemicsyn I've added the requested tests and bug fix for https://github.com/meltano/internal-general/issues/400: https://github.com/meltano/meltano/pull/6199/commits/2727df21a9862b0e1bba8967e669004c88b2ce57

There's a test failure caused by some state hanging around from a previous non-tracking test. This might take a few minutes to fix because unless I can identify which test is leaving behind the bad state, I have to re-run the whole test suite each time to check if I've fixed the problem.

@aaronsteers @pandemicsyn What's up with the tracking_disabled setting? It was added to the new Tracker class, but I don't see it referenced anywhere else. Can it be removed? It's the same as send_anonymous_usage_stats, but negated. It's only respected in one place in the codebase.

That looks like it got introduced with the original state change event PR that you and @aaronsteers coauthored: https://github.com/meltano/meltano/blame/f58031c268982c6eed8032c80f7f75c63b89c3a3/src/meltano/core/tracking/tracker.py#L162
Originally, I thought that was actually related to the MELTANO_DISABLE_TRACKING setting but looking around I realize thats just a negated ENV_ALIAS for send_anonymous_usage_stats :https://github.com/meltano/meltano/blob/51881f94e93cb89a1ba69bb3339de037d18d1be1/src/meltano/core/bundle/settings.yml#L6
So removal might be fine but im wondering if @aaronsteers had some other intension behind including that in the original PR?

@aaronsteers @pandemicsyn What's up with the tracking_disabled setting? It was added to the new Tracker class, but I don't see it referenced anywhere else. Can it be removed? It's the same as send_anonymous_usage_stats, but negated. It's only respected in one place in the codebase.

That looks like it got introduced with the original state change event PR that you and @aaronsteers coauthored: https://github.com/meltano/meltano/blame/f58031c268982c6eed8032c80f7f75c63b89c3a3/src/meltano/core/tracking/tracker.py#L162
Originally, I thought that was actually related to the MELTANO_DISABLE_TRACKING setting but looking around I realize thats just a negated ENV_ALIAS for send_anonymous_usage_stats :
https://github.com/meltano/meltano/blob/51881f94e93cb89a1ba69bb3339de037d18d1be1/src/meltano/core/bundle/settings.yml#L6
So removal might be fine but im wondering if @aaronsteers had some other intension behind including that in the original PR?

When we removed env_aliases, this was one that was configured at the Meltano setting level and there was not a way to honor existing users' MELTANO_DISABLE_TRACKING env var while still removing support for env_aliases elsewhere in the product. The decision we landed on was to add a second setting which would emulate the prior behavior without still needing env_aliases.

@aaronsteers I've updated Tracker.send_anonymous_usage_stats to use disable_tracking if send_anonymous_usage_stats is not set. To be clear, it's disable_tracking, and not tracking_disabled, right? The old code had tracking_disabled, but the env alias is for disable_tracking.

To be clear, it's disable_tracking, and not tracking_disabled, right? ... the env alias is for disable_tracking.

@WillDaSilva - Good catch. Yes, it should match the env alias. This misspelling was possibly a bug on my side when I drafted that update.

@WillDaSilva - Per your comment I did some digging and opened a new PR to make sure this is in a good place.

https://github.com/meltano/meltano/pull/6199

You can proceed with merging this and we can take follow-on actions in #6199


Environmental model for collision check

Very impressive work! Previously we were also trying to do collision detection with an environmental model and a virtual robot. We found this is not very easy since the point cloud from Quest cannot be accessed because of privacy problems. The feasible way is to pre-scan the environment with HMD and use that information for collision detection. I am wondering how the prescanned environment information is loaded in your project. There are some packages and which one are you using? Besides, it seems only Quest3 can scan a more precise mesh model. We are using Quest Pro, it seems it does not have an environment scan function and we need to set up the model of the environment manually.

Hi, thanks for your interest! We use scene API to access pre scaned environment from Quest. These links may be helpful:

https://developers.meta.com/horizon/documentation/unity/unity-scene-overview
https://developers.meta.com/horizon/documentation/unity/unity-mr-utility-kit-samples
To check collision between robot and object in the scene, we set each collision to trigger mode and add callback functions to collision: here


Fixes for packages hello eks anywhere docker tests

Signed-off-by: jonahjon<EMAIL_ADDRESS>Issue #, if available:
Description of changes:
Testing (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/hold
Still testing Vsphere


Template Render Error in User Settings

Description
I try to use this template https://github.com/go-gitea/gitea/blob/main/templates/user/settings/navbar.tmpl for my self-hosted instance but Gitea throws this error
Render failed, failed to render template: user/settings/account, error: template error: custom:user/settings/navbar:7:18 : executing "user/settings/navbar" at <$.UserDisabledFeatures.Contains>: wrong number of args for Contains: want 1 got 2
----------------------------------------------------------------------
		{{if not (and ($.UserDisabledFeatures.Contains "manage_credentials" "deletion") (not $.EnableNotifyMail))}}
		                ^
----------------------------------------------------------------------

Gitea Version
1.22.2
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
No response
How are you running Gitea?
I'm running Gitea from Docker
Database
PostgreSQL

You need to use the right template. Your gitea is 1.22.2 but you are using a main branch template.

@lunny Thanks for the feedback, I use the nightly build (1.23-dev)  and it works


Ranked issues

When using k!user it should display Unranked if Ranked Games Total are < 10.
This is intentionally not done in the API for reasons pertaining to React - therefore that's why it's not default'ly set the same way.
This issue also pertains to k!roles where it assigns a Gold rank despite Games Total being < 10.

Will be included in the next update.

In the recent ranked update we made the rank name show Unranked if criteria of games aren't met. The k!rank role could probably just check the rank name and assign based on that (unless it's Unranked, then don't assign any role).


Simplify Link codec

Situation:
Links and Urls are needlessly parsed and serialized. The serialization format is not declaratively stated.
Filters and Links don't fit well in a single type.
Desire:

The relation between Link, State and Url should be stated in a declarative, in-code fashion.
The Link type and the potential Filter type should embody clear use patterns of the Ui state.
The Link encoders as well as the parsers for data and isActive should be much simpler and more declarative.
We want a clearer expressions for links and filters that are just used to draw contingent Ui. We also want to get rid of the \[]->... situation of contingent Link Uis.

Questions:

Can we just use elm-app-url which happens to treat "a=" as "a" but "=" as (""="")? How does it treat "a=b=c" which is relevant for making filters selectors? Is it a good basis for the link encoder and queryer and for the url*state parser?
A: No.
Do we need an Url sanitizer?

Api and Invariants
Link.href
   : Link -> String

hrefs are always minimal, i.e. they represent only the intended transition from the current state to the new one.
Relative operations are encoded in protected assignments. Custom assignments are escaped accordingly.

Link.applyUrl
   : { makeAbsolute : Bool } -> State -> Url -> State

A new path and fragment replaces the old path and fragment. New flags append to the existing ones. New assignments replace the existing ones.
Relative operations are made absolute if makeAbsolute is True:

Toggle becomes turn on;
Bounce { there, here } becomes GoTo there

Imply any flag or path that needs to be present such that the intended path and fragment is rendered.

Link.isActive
   : State -> Link -> Bool

Link.getCurrentData
    : State -> Link -> Maybe Data

When to use links
The Url is a state shared by all interactive elements in a tab. It is very easy to manually edit, to share and to reproduce, but its utility diminishes when the Url gets too long:

The user potentially wants to share the state
The state can be reasonably serialized and edited
The serial form can be kept short
The total number of decisions in a Url is kept within the single digits
The pattern is neither too specific nor too general
The pattern is well-established and/or well-loved in existing applications
The pattern composes well with regions and nested occlusions
The pattern is not parallel to the model, i.e. the model has no variable that mirrors the pattern state, but the model may choose to track it, e.g. to store a per-user list of "recent items" or to (pre)fetch a subset of the model according to such a list.

Sample patterns
Token
token : { flag, label } -> ((String, Ui), (String, Ui))
token { flag, label } =
    let
        inner = (flag, toggle { flag, label })
    in
    ( filter ( flag )
        ( \flags -> if List.member flag then inner else [] )
    , filter ( flag )
        ( \flags -> if List.member flag then[] else inner )
    )

Set
set =
    [ { flag = "a", label = "A" }
    , { flag = "b", label = "B" }
    , { flag = "c", label = "C" }
    ]

(activeTokenList, inactiveTokenList) =
    List.map token set
        |> List.unzip
        |> Tuple.mapBoth (Html.ol []) (Html.ol [])

view =
    activeTokenList ++ Html.singleton [ Html.text "⇋" ] ++ inactiveTokenList

Idea: Make Ui more dumb and reduce module number.
Module Ui - doesn't actually know about occlusions. Perhaps provides a local Ui -> Get region Html function to the layout so it can interpret a wrapper to a simplified Ui.Wrapper. Ui doesn't import Link. Which means:
(1) Total untangle of the Link (and Filter) from the Ui
(2) A more permissive Ui.Wrapper type, which is counter to the "restrictive" in Restrictive...
(3) Adding more Patterns(TM) will be a breeze with such modules as Link and Filter; just need Ui to be equipped to handle the cases.
(4) Will removal and insertion work as intended?

Final steps:

[ ] escape protected assignments
[ ] test coverage with elm-app-test, including all edge cases
[ ] improve comments, along the tests

Question:
After 2 refactorings and lots of little edge cases yet to cover, I wonder which approach is more future-proof:
(a) (Re)Introduce a declarative builder/codec with opaque types. Less general than elm-url-codec and with some features features that elm-app-url is missing. Perhaps such a library exists already.
(b) Use tests, preferably directly in the comments, instead of types, to "describe" the functionality and cover all edge cases


chore(RHINENG-10336): update cypress code-cov package

Description
Associated Jira ticket: # (issue)
Please include a summary of the change, what this fixes/creates/improves.
How to test the PR
Please include steps to test your PR.
Before the change
After the change
Dependent work link
Checklist:

[ ] The commit message has the Jira ticket linked
[ ] PR has a short description
[ ] Screenshots before and after the change are added
[ ] Tests for the changes have been added
[ ] README.md is updated if necessary
[ ] Needs additional dependent work

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

Project coverage is 52.06%. Comparing base (c76cffb) to head (99d2a8f).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #323   +/-   ##
=======================================
  Coverage   52.06%   52.06%
=======================================
  Files          83       83
  Lines        1262     1262
  Branches      266      266
=======================================
  Hits          657      657
  Misses        605      605

Flag
Coverage Δ

?

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

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


fix: use sort posts for rss, fix lastBuildDate

as generateRSS get last build date from first post, so we need to use sorted posts, otherwise the lastBuildDate value is wrong

@timlrx


New package: Unity.Unity.6000.MacMono version 6000.0.1f1

Created by 🥟 Dumplings.
Microsoft Reviewers: Open in CodeFlow

https://github.com/microsoft/winget-pkgs/pull/153347#issuecomment-2108338347

@wingetbot run

https://github.com/microsoft/winget-pkgs/pull/153360#issuecomment-2111491135


JSX Linked Editing not working with Vim extension

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

VS Code Version: 1.79.0
OS Version: Windows 11 Pro 22621.1778

Steps to Reproduce:

Open a file that contains JSX elements
Navigate to an opening tag of JSX, e.g. the  in the example below
Place the Vim cursor over the "d" and press c + w (change word)
The closing tag is automatically deleted ✅
Update the tag to "section" and press Escape
-> The closing tag is not updated ❌

Note: The linked editing feature does appear to work on a single line , but in the case below it fails.
Example source code:
return (
    <div className={containerClasses}>
      <Spinner />
    </div>
  );

Example video showing the issue:
https://github.com/microsoft/vscode/assets/3640638/83b8cfdb-f872-4015-82b0-c4f034731bfa

I have the same bug
VSCode version: 1.79.1
Os Version: x86_64 Linux 6.3.7-arch1-1

The same problem occurs even when using vscode-neovim. I want it fixed as soon as possible

@aeschli
It also works well with vscode-neovim if we use the extension
https://marketplace.visualstudio.com/items?itemName=formulahendry.auto-rename-tag
However, I still want to use it as a built-in function.

This isn't a Vim plugin bug, you can reproduce it without it. Turn off the extension, then backspace the entire tag and start typing. You'll see the same issue.
Maybe a separate issue should be made?

@musjj There's already #185279


⚠ Bofrost.be has degraded performance

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

HTTP code: 200
Response time: 1570 ms

Resolved: Bofrost.be performance has improved in 527ade1.


Get sandcats.io into Public Suffixes List

Luckily Sandstorm mostly doesn't use cookies, but we should still add sandcats.io to the PSL:
https://publicsuffix.org/

Closed, as of https://github.com/publicsuffix/list/pull/17


Upgrade deprecated runtime nodejs6.10

CloudFormation templates in connect-integration-examples have been found to include a deprecated Lambda function runtime (nodejs6.10). The affected templates have been updated to a supported runtime (nodejs10.x).
Please note, this pull request has been generated by a bot; please check the base branch and files changed before merging.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Obsolete


Hide editable PR warning after the PR is merged

A warning was added recently to encourage contributors to enable "Allow edits from maintainers", and it's great! Unfortunately, the warning is shown needlessly after the PR is merged even if the setting was correctly enabled.
Would it be possible to hide it on merged PRs?

Does the checkbox and message stay there even after you refresh the page?

Yes, that screenshot was taken days after the PR was merged and the tab was just opened.

It looks like a bug by GitHub. The checkbox keeps appearing even after the branch is deleted. I'm hiding it in #1894

Thanks for the really quick fix, and for this great extension :tada:


Improve Wordle

Adopt the current wordle-hard-mode (renamed to wordle-solver) as the only version - used in the tests also.

Add in:

Basic instructions
Input validation
Checks for correct answer
Check for no possible answers
Check for valid attempt word

Merged into #520


not able to bootstrap aws windows instance to chef server

when i am trying to bootstrap aws windows instance to chef server. i am getting below error. Any help would be appreciated.
knife bootstrap -o winrm 10.200.xxx.xxx -U Administrator -P 'xxxxxx' -N WindowsNode -VV
INFO: Using configuration from /Users/anadkarni/Documents/src/chef.deploy.rd.elliemae.io/.chef/knife.rb
DEBUG: Checking if we need to accept Chef license to bootstrap node
DEBUG: Reading products and relationships...
DEBUG: Successfully read products and relationships
DEBUG: License acceptance required for chef version: 15
DEBUG: Searching for the following licenses: ["infra-client", "inspec"]
DEBUG: Found license chef_infra_client at /Users/anadkarni/.chef/accepted_licenses/chef_infra_client
DEBUG: Found license inspec at /Users/anadkarni/.chef/accepted_licenses/inspec
DEBUG: Missing licenses remaining: []
DEBUG: All licenses present
Connecting to <IP_ADDRESS>
DEBUG: winrm-fs requested, loading WinRM::FS gem (~> 1.0)
DEBUG: WinRM::FS previously loaded
DEBUG: [WinRM] @<IP_ADDRESS><{:transport=>:negotiate, :disable_sspi=>false, :basic_auth_only=>false, :endpoint=>"http://<IP_ADDRESS>:5985/wsman", :user=>"Administrator", :password=>"", :no_ssl_peer_verification=>false, :realm=>nil, :service=>nil, :ca_trust_file=>nil, :ssl_peer_fingerprint=>nil}> (Write-Host '[WinRM] Established
')
DEBUG: [WinRM] opening remote shell on http://<IP_ADDRESS>:5985/wsman
DEBUG: [WinRM] Endpoint doesn't support config request for MaxEnvelopsizekb
DEBUG: [WinRM] opening remote shell on http://<IP_ADDRESS>:5985/wsman
DEBUG: [WinRM] Endpoint doesn't support config request for MaxEnvelopsizekb

I found there is uppercase and lowercase issue with property MaxEnvelopsizekb.
on windows node, when i try to set  MaxEnvelopsizekb using winrm, it shows below error.
C:\Windows\System32>  winrm set winrm/config '@{MaxEnvelopsizekb="1024"}'
Error: Parameter name does not match any properties on resource: MaxEnvelopsizekb
The property name on winrm is MaxEnvelopeSizekb (S from size is uppercase)
But through knife bootstrap command , it is internally trying to set or use "MaxEnvelopsizekb".
How to resolve this issue? any suggestions ?

This is probably not the root problem, but in your example the case of the parameter probably does not matter, it is likely because there is a typo in the error message and the parameter is MaxEnvelopeSizekb.

Looks like not only does that debug message have a typo but it shouldn't even be rendered. There is a WinRM bug (which I will fix) that puts the debug message in an ensure instead of a rescue.
So the core issue here is that the bootstrapper just cant get through to the node. This is likely due to one or both of the following:

You have not added an inbound rule allowing WinRM-HTTP (port 5985) traffic in aws.
The local Windows firewall on the node is restricting traffic to its local subnet which is the default. Running netsh advfirewall firewall add rule name="WinRM-HTTP" dir=in localport=5985 protocol=TCP action=allow on the node would open that up to outside subnets.


Godot 4 Beta 2 from source mono build crashing unless ran through terminal on windows.

Godot version
v4.0.beta.custom_build.f8745f2f7
System information
Windows 11
Issue description
Editor crashes when running from build executable directly but when ran from terminal with .\godot.windows.tools.x86_64.exe the build runs normally.
Output from terminal:
Godot Engine v4.0.beta.custom_build.f8745f2f7 - https://godotengine.org
ERROR: Condition "!windows.has(p_window)" is true.
at: VulkanContext::window_resize (drivers\vulkan\vulkan_context.cpp:1497)
Vulkan API 1.2.0 - Using Vulkan Device #0: NVIDIA - NVIDIA GeForce RTX 3090 Ti
WARNING: Blend file import is enabled in the project settings, but no Blender path is configured in the editor settings. Blend files will not be imported.
at: _editor_init (modules\gltf\register_types.cpp:75)
WARNING: FBX file import is enabled in the project settings, but no FBX2glTF path is configured in the editor settings. FBX files will not be imported.
at: _editor_init (modules\gltf\register_types.cpp:99)
Steps to reproduce
Build mono executable from source with Net 6.0 release and steps followed from #65936 solution found at the bottom.
run from clicking on the .exe it will crash but if launched from the terminal with .\godot.windows.tools.x86_64.exe it functions fine until restart.
Minimal reproduction project
Not really possible since it's from compiled source code let me know how else I can help.

really weird just realized this is not a mono build and I repeated those steps exactly...

Could the build target be incorrect sometimes?

You're probably running the wrong .exe, the filenames changed recently. It should have editor in its name.

You're probably running the wrong .exe, the filenames changed recently. It should have editor in its name.

oh very interesting I will have a look :) Are there any recommended commands I can use to save time? The current mono process seems a bit convoluted. I would be happy to help with some form of documentation on this topic.

the only difference in the naming convention that show up with those commands are "tools" and "opt"
but from my recall is the opt a template? It has been a while since building in version 4.

Nevermind, I see you're on an older commit. The naming changed a week ago.

Nevermind, I see you're on an older commit. The naming changed a week ago.
Yes I tried to build this from the source commit/dev snapshot of beta 2

Is this still an issue? Also see the updated docs for compiling with .NET: https://docs.godotengine.org/en/stable/contributing/development/compiling/compiling_with_dotnet.html
If yes, please post all the commands you used to build godot and the resulting binaries for investigation.

Closing this due to the lack of response and as a likely user error. Can be reopened if there is more information provided.


Update WWDC video about prototyping in Keynote

Apple seems to have removed the WWDC 2014 video about prototyping in Keynote, but a very similar one exists from 2016. This updates to the 2016 video.
Checklist

[x] I made something good today 💐.
[x] I put links in the correct format: [Tool](link) — description.
[x] My description starts with the lower-case letter and ends with the full stop.
[x] I put the tool in the alphabetical order.

Optional

[x] I added the appropriate labels: free, open-source or mac.
[x] I put the tool only to one category.
[x] I followed Contribution Guidelines.

Hmm, actually I found some hidden links around the web to the original video, and it is quite a bit better for this purpose... Maybe instead of changing videos, I'll leave the original link, but also add the video title for some hope of finding it elsewhere.

thanks a lot for clarification! Indeed, some old videos might have wrong links