Add non-panic way of registering constraints

Also, fixed bug in ValidatorFor not copying defaulted fields for v8n constraint building

Codecov Report

Merging #22 (cb7d9c0) into master (b5e5231) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #22   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           10        10
  Lines         1412      1427   +15
=========================================
+ Hits          1412      1427   +15

Impacted Files
Coverage Δ

constraints_registry.go
100.00% <100.00%> (ø)

tag_parsing.go
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Requests only matching once?

I am running this on Node.JS....I have configured my js as follows using a JSON schema matcher -
var mockserver = require('mockserver-node');
var mockServerClient = require('mockserver-client').mockServerClient;
var mockserver_port = 1080;

mockserver
  .start_mockserver({serverPort: mockserver_port, verbose: true})
  .then(
    function () {
      mockServerClient("localhost", 1080).mockAnyResponse({
        "httpRequest": {
          'path': '/test/JSONSCHEMA',
          "body": {
            "jsonSchema": JSON.stringify({
              "$id": "http://example.com/example.json",
              "type": "object",
              "definitions": {},
              "$schema": "http://json-schema.org/draft-06/schema#",
              "properties": {
                "Name": {
                  "$id": "/properties/Name",
                  "type": "string"
                },
                "Number": {
                  "$id": "/properties/Number",
                  "type": "integer"
                }
              }
            })
          }
        },
        "httpResponse": {
          'body': JSON.stringify({name: 'You Sent JSON'}),
        }
      }).then(
        function () {
          console.log("created \"/JSON SCHEMA\" expectation");
        },
        function (error) {
          console.log(error);
        }
      );

    },
    function (error) {
      console.log(JSON.stringify(error));
    }
  );

When I run the server and post the following json via Postman -
POST /test/JSONSCHEMA HTTP/1.1
Host: localhost:1080
Content-Type: application/json
Cache-Control: no-cache
Postman-Token: 41050455-0962-ff30-b517-8ef40fc9b481

{
  "Name": "customer",
  "Number": 408401890
}

The first response is a 200 with
{     "name": "You Sent JSON" }
2018-01-25 14:31:06,581 INFO o.m.m.HttpStateHandler request:

	{
	  "method" : "POST",
	  "path" : "/test/JSONSCHEMA",
	  "body" : {
	    "type" : "STRING",
	    "string" : "{\n  \"Name\": \"customer\",\n  \"Number\": 408401890\n}",
	    "contentType" : "text/plain; charset=utf-8"
	  },
	  "cookies" : {
	    "sessionId" : "2By8LOhBmaW5nZXJwcmludCIlMDAzMW"
	  },
	  "headers" : {
	    "Origin" : [ "chrome-extension://fhbjgbiflinjbdggehcddcbncdddomop" ],
	    "Cookie" : [ "sessionId=2By8LOhBmaW5nZXJwcmludCIlMDAzMW" ],
	    "Cache-Control" : [ "no-cache" ],
	    "Accept" : [ "*/*" ],
	    "Connection" : [ "keep-alive" ],
	    "User-Agent" : [ "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/63.0.3239.132 Safari/537.36" ],
	    "Host" : [ "localhost:1080" ],
	    "Postman-Token" : [ "2d726b23-c7af-0eb5-b2a8-48cb81fb9522" ],
	    "Accept-Encoding" : [ "gzip, deflate, br" ],
	    "Accept-Language" : [ "en-US,en;q=0.9" ],
	    "Content-Length" : [ "47" ],
	    "Content-Type" : [ "application/json" ]
	  },
	  "keepAlive" : true,
	  "secure" : false
	}

 matched expectation:

	{
	  "path" : "/test/JSONSCHEMA",
	  "body" : {
	    "type" : "JSON_SCHEMA",
	    "jsonSchema" : "{\"$id\":\"http://example.com/example.json\",\"type\":\"object\",\"definitions\":{},\"$schema\":\"http://json-schema.org/draft-06/schema#\",\"properties\":{\"Name\":{\"$id\":\"/properties/Name\",\"type\":\"string\"},\"Number\":{\"$id\":\"/properties/Number\",\"type\":\"integer\"}}}"
	  }
	}

2018-01-25 14:31:06,586 INFO o.m.m.HttpStateHandler returning response:

	{
	  "headers" : {
	    "Content-Type" : [ "application/json; charset=utf-8" ],
	    "Cache-Control" : [ "no-cache, no-store" ],
	    "connection" : [ "keep-alive" ]
	  },
	  "body" : "{\"name\":\"You Sent JSON\"}"
	}

 for request:

	{
	  "method" : "POST",
	  "path" : "/test/JSONSCHEMA",
	  "body" : {
	    "type" : "STRING",
	    "string" : "{\n  \"Name\": \"customer\",\n  \"Number\": 408401890\n}",
	    "contentType" : "text/plain; charset=utf-8"
	  },
	  "cookies" : {
	    "sessionId" : "2By8LOhBmaW5nZXJwcmludCIlMDAzMW"
	  },
	  "headers" : {
	    "Origin" : [ "chrome-extension://fhbjgbiflinjbdggehcddcbncdddomop" ],
	    "Cookie" : [ "sessionId=2By8LOhBmaW5nZXJwcmludCIlMDAzMW" ],
	    "Cache-Control" : [ "no-cache" ],
	    "Accept" : [ "*/*" ],
	    "Connection" : [ "keep-alive" ],
	    "User-Agent" : [ "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/63.0.3239.132 Safari/537.36" ],
	    "Host" : [ "localhost:1080" ],
	    "Postman-Token" : [ "2d726b23-c7af-0eb5-b2a8-48cb81fb9522" ],
	    "Accept-Encoding" : [ "gzip, deflate, br" ],
	    "Accept-Language" : [ "en-US,en;q=0.9" ],
	    "Content-Length" : [ "47" ],
	    "Content-Type" : [ "application/json" ]
	  },
	  "keepAlive" : true,
	  "secure" : false
	}

 for response action:

	{
	  "headers" : {
	    "Content-Type" : [ "application/json; charset=utf-8" ],
	    "Cache-Control" : [ "no-cache, no-store" ]
	  },
	  "body" : "{\"name\":\"You Sent JSON\"}"
	}

As expected, but any posts i do after the first one all come back as 404s and do not work, but nothing has changed??
2018-01-25 14:31:42,546 INFO o.m.m.HttpStateHandler no active expectations when receiving request:

	{
	  "method" : "POST",
	  "path" : "/test/JSONSCHEMA",
	  "body" : {
	    "type" : "STRING",
	    "string" : "{\n  \"Name\": \"customer\",\n  \"Number\": 408401890\n}",
	    "contentType" : "text/plain; charset=utf-8"
	  },
	  "cookies" : {
	    "sessionId" : "2By8LOhBmaW5nZXJwcmludCIlMDAzMW"
	  },
	  "headers" : {
	    "Origin" : [ "chrome-extension://fhbjgbiflinjbdggehcddcbncdddomop" ],
	    "Cookie" : [ "sessionId=2By8LOhBmaW5nZXJwcmludCIlMDAzMW" ],
	    "Cache-Control" : [ "no-cache" ],
	    "Accept" : [ "*/*" ],
	    "Connection" : [ "keep-alive" ],
	    "User-Agent" : [ "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/63.0.3239.132 Safari/537.36" ],
	    "Host" : [ "localhost:1080" ],
	    "Postman-Token" : [ "f518fa26-227d-2375-57cf-e60de88ddbb8" ],
	    "Accept-Encoding" : [ "gzip, deflate, br" ],
	    "Accept-Language" : [ "en-US,en;q=0.9" ],
	    "Content-Length" : [ "47" ],
	    "Content-Type" : [ "application/json" ]
	  },
	  "keepAlive" : true,
	  "secure" : false
	}

2018-01-25 14:31:42,550 INFO o.m.m.HttpStateHandler no matching expectation - returning:

	{
	  "statusCode" : 404,
	  "reasonPhrase" : "Not Found"
	}

 for request:

	{
	  "method" : "POST",
	  "path" : "/test/JSONSCHEMA",
	  "body" : {
	    "type" : "STRING",
	    "string" : "{\n  \"Name\": \"customer\",\n  \"Number\": 408401890\n}",
	    "contentType" : "text/plain; charset=utf-8"
	  },
	  "cookies" : {
	    "sessionId" : "2By8LOhBmaW5nZXJwcmludCIlMDAzMW"
	  },
	  "headers" : {
	    "Origin" : [ "chrome-extension://fhbjgbiflinjbdggehcddcbncdddomop" ],
	    "Cookie" : [ "sessionId=2By8LOhBmaW5nZXJwcmludCIlMDAzMW" ],
	    "Cache-Control" : [ "no-cache" ],
	    "Accept" : [ "*/*" ],
	    "Connection" : [ "keep-alive" ],
	    "User-Agent" : [ "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/63.0.3239.132 Safari/537.36" ],
	    "Host" : [ "localhost:1080" ],
	    "Postman-Token" : [ "f518fa26-227d-2375-57cf-e60de88ddbb8" ],
	    "Accept-Encoding" : [ "gzip, deflate, br" ],
	    "Accept-Language" : [ "en-US,en;q=0.9" ],
	    "Content-Length" : [ "47" ],
	    "Content-Type" : [ "application/json" ]
	  },
	  "keepAlive" : true,
	  "secure" : false
	}

Any help would be appreciated, kinda pulling my hair out..
UPDATE, I have just done some GET requests tests, and this does the same, the first request works, all the subsequent (repeated) calls all get 404?

Sorry @jimkohl I didn't see the update in the status


dependency failed to start: container supabase-analytics is unhealthy

I'm trying to run the local development environment with docker, following the steps in the readme, but the "supabase-analytics" container doesn't start.
[+] Building 826.7s (13/13) FINISHED                                                                                docker:default
 => [superflows internal] load build definition from Dockerfile                                                                         0.0s
 => => transferring dockerfile: 545B                                                                                0.0s
 => WARN: FromPlatformFlagConstDisallowed: FROM --platform flag should not use constant value "linux/amd64" (line 4)                    0.0s
 => [superflows internal] load metadata for docker.io/library/node:alpine                                                               2.3s
 => [superflows internal] load .dockerignore                                                                                0.0s
 => => transferring context: 58B                                                                                0.0s
 => [superflows internal] load build context                                                                                0.3s
 => => transferring context: 68.36MB                                                                                0.3s
 => [superflows 1/7] FROM docker.io/library/node:alpine@sha256:1467ea23cce893347696b155b9e00e7f0ac7d21555eb6f27236f1328212e045e        17.9s
 => => resolve docker.io/library/node:alpine@sha256:1467ea23cce893347696b155b9e00e7f0ac7d21555eb6f27236f1328212e045e                    0.0s
 => => sha256:7a291b8a7b575b411b08e82b066a02d5d73db2377f324d619e07141f3169937c 1.39MB / 1.39MB                                          4.1s
 => => sha256:1467ea23cce893347696b155b9e00e7f0ac7d21555eb6f27236f1328212e045e 6.41kB / 6.41kB                                          0.0s
 => => sha256:db7d4c706c493243e556c19cf3b80556f8dc91d69fcb750b16909624bd5cbfa1 1.72kB / 1.72kB                                          0.0s
 => => sha256:bd8023897f3875f1759c7b6704a1c7ee793e5c6fbc35b3b680279476ed84bdc8 6.19kB / 6.19kB                                          0.0s
 => => sha256:43c4264eed91be63b206e17d93e75256a6097070ce643c5e8f0379998b44f170 3.62MB / 3.62MB                                          3.9s
 => => sha256:453ca8c35a6710c3a236b7f3e68f3688ef04449b59a2b80ad92abc69b7c4c718 50.85MB / 50.85MB                                       16.5s
 => => extracting sha256:43c4264eed91be63b206e17d93e75256a6097070ce643c5e8f0379998b44f170                                               0.0s
 => => sha256:15929fe50b42a226c1ffcbf061b9446ceefb6fc7ae3201e53aaaf6109b719d46 445B / 445B                                              4.2s
 => => extracting sha256:453ca8c35a6710c3a236b7f3e68f3688ef04449b59a2b80ad92abc69b7c4c718                                               1.0s
 => => extracting sha256:7a291b8a7b575b411b08e82b066a02d5d73db2377f324d619e07141f3169937c                                               0.0s
 => => extracting sha256:15929fe50b42a226c1ffcbf061b9446ceefb6fc7ae3201e53aaaf6109b719d46                                               0.0s
 => [superflows 2/7] RUN mkdir -p /app                                                                                0.7s
 => [superflows 3/7] WORKDIR /app                                                                                0.1s
 => [superflows 4/7] COPY package.json ./                                                                                0.0s
 => [superflows 5/7] RUN npm i && npm cache clean --force                                                                             632.5s
 => [superflows 6/7] COPY . /app                                                                                2.4s
 => [superflows 7/7] RUN rm -f .env                                                                                0.3s
 => [superflows] exporting to image                                                                                170.2s
 => => exporting layers                                                                                170.1s
 => => writing image sha256:ea9ce2862a36de2660873b9d9eb4cdde255d0f0c4906760979653e17c3d30844                                            0.0s
 => => naming to docker.io/library/development-superflows                                                                               0.0s
 => [superflows] resolving provenance for metadata file                                                                                0.0s
[+] Running 15/15
 ✔ Network development_default               Created                                                                                0.2s
 ✔ Container supabase-imgproxy               Started                                                                                207.8s
 ✔ Container supabase-vector                 Healthy                                                                                213.3s
 ✔ Container supabase-db                     Healthy                                                                                219.9s
 ✘ Container supabase-analytics              Error                                                                                253.1s
 ✔ Container supabase-meta                   Created                                                                                0.1s
 ✔ Container realtime-dev.supabase-realtime  Created                                                                                0.1s
 ✔ Container supabase-kong                   Created                                                                                0.1s
 ✔ Container supabase-rest                   Created                                                                                0.1s
 ✔ Container supabase-auth                   Created                                                                                0.1s
 ✔ Container supabase-edge-functions         Created                                                                                0.1s
 ✔ Container supabase-pooler                 Created                                                                                0.1s
 ✔ Container supabase-studio                 Created                                                                                0.1s
 ✔ Container superflows                      Created                                                                                207.0s
 ✔ Container supabase-storage                Created                                                                                0.0s
dependency failed to start: container supabase-analytics is unhealthy

Log
sudo docker logs supabase-analytics
[sudo] senha para asaas:
LOGFLARE_NODE_HOST is: <IP_ADDRESS>

14:12:18.452 [info] Starting migration

14:12:19.005 [error] Postgrex.Protocol (#PID<0.163.0>) failed to connect: ** (Postgrex.Error) FATAL 28P01 (invalid_password) password authentication failed for user "supabase_admin"

14:12:19.005 [error] Postgrex.Protocol (#PID<0.162.0>) failed to connect: ** (Postgrex.Error) FATAL 28P01 (invalid_password) password authentication failed for user "supabase_admin"

14:12:20.837 [error] Postgrex.Protocol (#PID<0.163.0>) failed to connect: ** (Postgrex.Error) FATAL 28P01 (invalid_password) password authentication failed for user "supabase_admin"

14:12:20.890 [error] Postgrex.Protocol (#PID<0.162.0>) failed to connect: ** (Postgrex.Error) FATAL 28P01 (invalid_password) password authentication failed for user "supabase_admin"

14:12:23.192 [error] Postgrex.Protocol (#PID<0.162.0>) failed to connect: ** (Postgrex.Error) FATAL 28P01 (invalid_password) password authentication failed for user "supabase_admin"

14:12:24.514 [error] Postgrex.Protocol (#PID<0.163.0>) failed to connect: ** (Postgrex.Error) FATAL 28P01 (invalid_password) password authentication failed for user "supabase_admin"

14:12:27.029 [error] Postgrex.Protocol (#PID<0.162.0>) failed to connect: ** (Postgrex.Error) FATAL 28P01 (invalid_password) password authentication failed for user "supabase_admin"

14:12:29.900 [error] Could not create schema migrations table. This error usually happens due to the following:

  * The database does not exist
  * The "schema_migrations" table, which Ecto uses for managing
    migrations, was defined by another library
  * There is a deadlock while migrating (such as using concurrent
    indexes with a migration_lock)

To fix the first issue, run "mix ecto.create" for the desired MIX_ENV.

To address the second, you can run "mix ecto.drop" followed by
"mix ecto.create", both for the desired MIX_ENV. Alternatively you may
configure Ecto to use another table and/or repository for managing
migrations:

    config :logflare, Logflare.Repo,
      migration_source: "some_other_table_for_schema_migrations",
      migration_repo: AnotherRepoForSchemaMigrations

The full error report is shown below.

** (DBConnection.ConnectionError) connection not available and request was dropped from queue after 10950ms. This means requests are coming in and your connection pool cannot serve them fast enough. You can address this by:

  1. Ensuring your database is available and that you can connect to it
  2. Tracking down slow queries and making sure they are running fast enough
  3. Increasing the pool_size (although this increases resource consumption)
  4. Allowing requests to wait longer by increasing :queue_target and :queue_interval

See DBConnection.start_link/2 for more information

    (ecto_sql 3.10.1) lib/ecto/adapters/sql.ex:913: Ecto.Adapters.SQL.raise_sql_call_error/1
    (elixir 1.14.4) lib/enum.ex:1658: Enum."-map/2-lists^map/1-0-"/2
    (ecto_sql 3.10.1) lib/ecto/adapters/sql.ex:1005: Ecto.Adapters.SQL.execute_ddl/4
    (ecto_sql 3.10.1) lib/ecto/migrator.ex:738: Ecto.Migrator.verbose_schema_migration/3
    (ecto_sql 3.10.1) lib/ecto/migrator.ex:552: Ecto.Migrator.lock_for_migrations/4
    (ecto_sql 3.10.1) lib/ecto/migrator.ex:428: Ecto.Migrator.run/4
    (ecto_sql 3.10.1) lib/ecto/migrator.ex:170: Ecto.Migrator.with_repo/3
    nofile:1: (file)

14:12:33.501 [notice]     :alarm_handler: {:set, {:system_memory_high_watermark, []}}

https://github.com/orgs/supabase/discussions/20211

I would recommend not to use the docker build method and instead the npx supabase method

@rituraj-yactraq Would you have a script of the steps, as per your suggestion, to run in development mode?

@GlauberF
npm i
npm install supabase --save-dev
npx supabase start
This will create supabase local instance and start superflows dashboard. You should join the superflows slack channel, common installation issues solution you will find there.

@rituraj-yactraq Thanks

@rituraj-yactraq When trying to run the npm i command, I get the error below.


stats command shows that the subscription is enabled, but no LSNs for some time

So, something possibly odd has happened and I'm not sure why.  Usually I see multiple slots on my subscription server that appear to be created by the subscription created with pg_easy_replicate, and they often show a [null] value. However, there's one slot that is actualy named for the subscription that is in 'streaming' mode and DOES show changing LSNs.  I am not sure why, but this stream slot has disappeared.  I tried executing a postgres command to disable and re-enable the subscription (I notice pg_easy_replicate's stop_sync actually completely deletes the publisher and subscriber configurations), and that didn't seem to resolve the issue.
I see that the slots are all executing COPY commands, but it doesn't seem like any data is actually travelling over. I did bump up my max_wal_senders on the target from the default of 4 to 10, increased my max_logical_replication_slots on the target to 20, and max_logical_subscriber_workers to 10, also on the target.  While this seems to have increased the number of slots that pg_easy_replicate is returning, I still don't see any LSNs on them.

Just FYI, basically stopping/starting sync resolves the issue, but that's not entirely ideal since I don't know if pg_easy_replicate will resume replication from where it stopped, or if it will start over from scratch.

Thats odd, going to block some time to look into this proper in the next week and get back.

Well dropping and creating the publisher/subscriber "fixed" it, but still no idea why the streaming slot would just plain disappear.  One thing that might be a reason is that someone else edited the publication settings to remove a couple tables from it, but didn't refresh the subscriber side.  Even with refreshing it, it still didn't resume. So, it could be that however pg_easy_replicate actually creates the publication and subscription, any settings viewed by editing the publication or subscription should not be changed via SQL command.

pg_easy_replicate will resume replication from where it stopped,

No, pg_easy_replicate, which uses subscription/publication under the hood from Postgres, won't resume from where it left off. It will try to perform an initial copy first and then replicate changes. So in case of issues, it's best to start over. It would be nice to have a resume functionality, which is a bit tricky to do inside pg_easy_replicate, but is something that pg_flo supports.
That said - if there were no LSNs, I have typically found something going wrong with the publication creation state or that the initial copy itself could be failing on the target data, and usually Postgres logs would have more info. Potential reasons could be:

Inserting data is running into potential constraints and failing
Replication workers or the copy worker unexpectedly died
Connection issues between the source <> target database during the replication

In the last two cases, starting again could typically fix the issue, so I think that's likely what you're seeing.

So, it could be that however pg_easy_replicate actually creates the publication and subscription, any settings viewed by editing the publication or subscription should not be changed via SQL command.

Yes, that is correct - it's better to manage it via pg_easy_replicate. It supports --exclude-tables FWIW. If you also update/stop/remove the publication outside of pg_easy_replicate, there could be partial state left over in the groups table, so that could cause issues for the same group-name in the future, when starting bootstrap, start_sync, or something else.
If you have any logs from start_sync or other commands, I'm happy to help debug.


OPENCODE-14 Added details about Open-Source in the readme

Not sure why the PR doesn't get updated with recent commit, I'm going to close this PR and re-open a new one on the same branch.


No Scrollbar inside Storybook

Describe the bug
i'm not getting scrollbar on the render side of the storybook
To Reproduce
Steps to reproduce the behavior:
做澳洲文凭/办理澳洲硕士文凭【WX：jywenping】办理澳大利亚国立大学毕业证成绩单,制作ANU本科毕业证学位证,买硕士文凭,澳洲留信认证
Load Storybook with content greater than the layout
Scrollbar won't show
Expected behavior
Scrollbar should appear if the content overflows the container

Spam.


Fix decimals and insufficiente balance on swap

This PR solves #388  and #389
User can not input more than the token decimals (or default 7)
Insufficient balance on XLM now is compared with the available balance instead of the full xlm balance
https://github.com/soroswap/frontend/assets/56497105/c502291f-9bbc-4a82-a57b-af657a1ca063

me sucedió esto:


Add SIA numerical modeling section

It will be in the ice flow part.
It will need to introduce finite difference.
Then solve the SIA equation with finite difference.
Then do some experiments with the model.
(Note that the numerical methods we will use will be intentionally simple to help with understanding.)
We can also include some extensions: e.g., variable bed topography, debris cover, GIA (maybe).

some of this can come straight from the ice-sheet theory seminar: https://github.com/ldeo-glaciology/ice-sheet-seminar-2022

#9 made a start on this.
But lots still needs to be done on that chapter

#26 added a new page building on the SIA equation.
#19 will add a derivation of the SSA and the Blatter-Pattyn models.


FEAT: Add Hangman game #36

Added hangman game
Please check and close issue #36
Thank you

Looks great,please resolve conflicts and make PR again @Sneha6003 !


Use start_active_span instead of start_span in ServerMiddleware

start_active_span changes the active span of the tracer, which allows
Sidekiq worker to access the server span.

cc @iaintshine

cc @iaintshine

cc @iaintshine


Offline Tools pages

Introduces a page in the "Essentials" section of each SDK describing the API surface concerning resilience to connectivity loss.
How should this be manually tested?
Go to SDK,  , Essenials, Offline Tools

 Snippets tests reports   SDK Version Tests overview      v6        v5                           :heavy_check_mark:                            v1        v1        v1


Adding ValueTuple and ITuple to corlib in CoreRT

ValueTuple types are supporting the C#7.0 tuples feature. They have been implemented as a standalone CoreFx package, but should be moved down into the various corlibs (mscorlib/desktop and mscorlib/mono are already done) to be consistent with System.Tuple and can be used by other APIs.
This PR does not contain ITuple yet, but I will add it shortly. It is an interface in CompilerServices namespace used for dynamic pattern matching (in a future version of C#). It applies to System.Tuple and System.ValueTuple types. It was implemented in mscorlib/desktop, and I'm in the process of implementing it in other corlibs.
The main purpose for opening this PR at this point is to raise two questions:

where should test code for ValueTuple go?
where is the existing test code for System.Tuple? I could not find it.

FYI @tarekgh @VSadov

Hi @jcouv, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

looks EditorBrowsableAttribute is not exist in corert yet.

cc @jkotas

looks EditorBrowsableAttribute is not exist in corert yet.

Some related discussion on this here: https://github.com/dotnet/corefx/issues/13746#issuecomment-265792463. It looks like it's being planned to move the attribute down.

where should test code for ValueTuple go?
where is the existing test code for System.Tuple?

CoreRT is going to use tests in CoreFX. (.NET Native has some old tests in TFS as well, but they are going away.)

looks EditorBrowsableAttribute is not exist in corert yet.

If you would like to add/move the EditorBrowsableAttribute to CoreLib as part of this, you are welcomed to. It needs to be added in both CoreCLR and CoreRT CoreLibs.

Same comments as for the CoreCLR change - the .cs files should be exactly same between CoreCLR and CoreRT.

@jkotas Regarding CoreCLR and CoreRT files, the new files I added do match now. But Tuple.cs had some differences, which I preserved.

ok

@dotnet-bot test Windows_NT Release  please
// reason: remoting error

@jcouv I have moved the EditorBrowsableAttribute to CoreLib. You can uncomment it.

@dotnet-bot test Windows_NT Debug + CoreCLR tests please


Duplicate entry for data Text in autocomplete

The autocomplete dialog shows two entries for data Text.  It seems that haddock puts it in the hoogle-format output twice.  We lost the deduping in #795 by @nixorn, causing this bug to reappear.  Should fix it by readding deduping; but it will have to be done in a different way, since the order of lines is now important.

This was fixed by #799


AssertionError occured in a customized iterator based on PrefetchingIter

I'm working on assigning a pre-defined weight for each training instance in weighted Softmax loss. The image data and labels are loaded by using ImageRecordIter, and the weights are loaded by using CSVIter. To merge the two iters into one, I created a customized iterator based on PrefetchingIter (see the code here). Compared with the original PrefetchingIter code, in my customized iterator I modified self.provide_data, self.provide_label and self.current_batch as follows:
## In Line 26
# set provide_data and provide_label
self.provide_data = [('data', iters[0].provide_data[0][1]), ('weight', iters[1].provide_data[0][1])]
self.provide_label = iters[0].provide_label

...

## In Line 88
self.current_batch = mx.io.DataBatch(
    # two parts: data and weight
    self.next_batch[0].data + self.next_batch[1].data,
    # only the first iter has label
    self.next_batch[0].label,
    self.next_batch[0].pad,
    self.next_batch[0].index
)

...

For the weighted Softmax loss, I created a new based on CustomOp, which can be found in test_instance_weighted_softmax.py. This file also provides a runnable test by using the MNIST dataset. When training the model, the code throws an error (which I have no clue on what's going on there):
AssertionError: Number of entry mismatches between iterators

I put everything in this repo. If you have installed MXNet, you can directly run test_instance_weighted_softmax.py to reproduce the error.
Can someone please advise on how to solve that? And what causes that error?
Thanks.

the iterators you are combining has different number of entries per epoch. i.e. when one iter has finished the other is still going. They need to have the same length

@piiswrong Thanks for the reply. Actually both image data and weights contain 60000 entries (or rows). And for both the iters, I set the batch size as the same. So I don't think it's due to the different length issue.

try write a simple loop to check for data/label consistency

I discovered that it was due to a strange behavior of iter.reset() for the CSVIter. Basically, in my machine, it does not reset the CSVIter to the beginning. Anyway, I'll have some more debug to figure out. Thanks


[Feature] Getting the current block style

Maybe it should be document well kr implemented. I need a functionality to receive the current block style as well as formatting like bold, italic etc.
If there is already a solution I would highly appreciate an example.

In milkdown, something like bold, italic are prosemirror marks. You can get current node's marks through:
const currentNode = state.selection.$from.node();
const marks = currentNode.marks;

That's part of prosemirror ability, you can check the reference manual of it.


Support RESTIC_REST_PASSWORD_FILE

Output of restic version
restic 0.17.3 compiled with go1.23.2 on linux/amd64
What should restic do differently? Which functionality do you think we should add?
Just like RESTIC_REST_PASSWORD contains the cleartext password for the REST backend, RESTIC_REST_PASSWORD_FILE would specify the file path from where to read the password.
What are you trying to do? What problem would this solve?
A file containing a password is especially useful in Docker and systemd, where sensible data like passwords (aka "secrets") are better stored in files. See Docker's secrets and sytemd's credentials.
In particular, my use case is in systemd to automate my backups. Credentials are encrypted, but get unencrypted into files when the unit starts, and they are only available for that unit.
Did restic help you today? Did it make you happy in any way?
Oh yes! I wish the CLI were less convoluted, still it works, reliably and fast.

Oh and I want to add that I already have a working implementation of this feature, paired with a unit test. I'd be happy to file a PR.

You can use:
RESTIC_PASSWORD_COMMAND="cat /path/file_with_password"

I'm talking about the password for the REST user, not the repository's password.

There is already RESTIC_REPOSITORY_FILE which for the REST backend already reads the password and URL from a file. Won't that also work for you?


I get an error `Could not find a declaration file for module

var require: NodeRequire
(id: string) => any
Could not find a declaration file for module '@savvywombat/tailwindcss-grid-areas'.
<EMAIL_ADDRESS>implicitly has an 'any' type.
Try `npm i --save-dev @types/savvywombat__tailwindcss-grid-areas`
if it exists or add a new declaration (.d.ts) file containing `declare module '@savvywombat/tailwindcss-grid-areas';`
ts(7016)

this error occurs from the code below
// in tailwind.config.js
plugins: [require('@savvywombat/tailwindcss-grid-areas')]

Hi, I haven't done any work with TypeScript before, and the plugin is vanilla JavaScript. In the meantime, you can do as suggested in the above message:

add a new declaration (.d.ts) file containing declare module '@savvywombat/tailwindcss-grid-areas';

Hi again,
I've tried to make a simple failing project to reproduce the problem and develop a fix, but I'm not getting the error. If you're still having trouble, would you be able to provide a simple failing project, or a link to a repo/branch?

I'm going to close this issue, as I can't reproduce the problem and there's been no further details/follow-up.


Touch to Search not working

The Bug
Touch to search doesn't work, nothing happens when I touch on a word.
It's enabled in settings but it says it is off.

How to reproduce the bug?

Enable Touch to search in Google Services
Touch any word

Expected behavior
It should display definitions, pictures, search results, and other details when I touch a word.
Device Information

Device: Vivo Y20
Android Version: Android 11
Kiwi Browser Version: 99.0.4844.86

All works. Xiaomi Mi 9 Lite android 11.

Looks like only the displayed checkbox on / off in options is not working properly

Looks like only the displayed checkbox on / off in options is not working properly

Yes, it's on but says off. And also, it REALLY IS not working on my phone. How did you made the gifs? I want to show you mine too.

https://ibb.co/HDRPRgY][img]https://i.ibb.co/HDRPRgY/20220328-222630.gif

[img]https://ibb.co/HDRPRgY][img]https://i.ibb.co/HDRPRgY/20220328-222630.gif[/img]

Looks like only the displayed checkbox on / off in options is not working properly

Yes, it's on but displays it's off.
And also, it REALLY IS not working on my phone. 👇

What app do you use to create gifs?


Envisalink pgm

Description:
This release will allow users of the DSC panels to leverage custom functions on their panels to control things like garage door openers.  This has been requested several times.  In addition, the "code" parameter is now optional for people that wish to not include the code, and enter it in at runtime.  Also resolved issue 18422.
Related issue (if applicable): fixes #18422
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#7919
Example entry for configuration.yaml (if applicable):

Checklist:

[X] The code change is tested and works locally.
[X] Local tests pass with tox. Your PR cannot be merged unless tests pass
[X] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[X] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[X] New dependencies have been added to the REQUIREMENTS variable (example).
[X] New dependencies are only imported inside functions that use them (example).
[X] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Wondering if you had any thoughts on this, @MartinHjelmare , I'm not sure what I can do to fix this since it appears the travis build failure is unrelated to my change.

Okay great thank you!!

Hi @Cinntax, just wanted to thank you for the fix for #18422. This will help us greatly with the component stability. I can't test it in dev branch unfortunately (given I don't have a test panel to work with) but can certainly try it out in the next release as the issues are reproducable on my side.

Okay sure no problem- it's my first time doing a custom service so I missed the requirement too.  I'll get that done today.

In the meantime I'll at least have people test the fix for 18422.


CmpService.ShowCmpForExistingUser() - method doesn't exist

MAX Module Version
6.2.1
React Native Version
0.72.6
Device/Platform Info
All
Current Behavior
According to the documentation, this is the method signature for showing the cmp prompt to existing users.
https://dash.applovin.com/documentation/mediation/react-native/getting-started/terms-and-privacy-policy-flow#showing-gdpr-flow-to-existing-users
Expected Behavior
No response
How to Reproduce
CmpService is undefined
Additional Info
No response

Possibly related to #296

@GLSinthu, thanks for raising the issue.  We are currently working on it.

We've fixed the issue and released it with 6.2.2.   Please check our docs.


⚠ Samsung has degraded performance

In 6041b01, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2771 ms

Resolved: Samsung performance has improved in 3c3b40c.


More keywords for "pattern-matching product name"

I think I've resurrected the "pattern-matching product name" reason. See this example.
Since it's been left alone for a long time without maintenance, we can probably utilize this reason by updating the keywords list so it catches more recent spam. I have added and changed a few keywords according to the latest "blacklisted website in title" posts. I'd like to know what else we can do about this reason.

Coverage remained the same at 66.005% when pulling 84630093ea020f1973e8507aa0c7ac3782db3d3d on iBug:update-pattern into 077d32f71616635fc642fa5d1aa9ce62eed1f54e on Charcoal-SE:master.

!!/approve
Sent with GitHawk


Expose number of tickets for Event resource

API returns number_of_tickets in response body but does not expose it to client.

Thanks for the change


Confusing instructions in truncate a string

Challenge Name

Truncate a string
Issue Description

Inspired by https://github.com/FreeCodeCamp/freeCodeCamp/issues/12418
Let's talk about why I think this is confusing. Here are the instructions:

Truncate a string (first argument) if it is longer than the given maximum string length (second argument). Return the truncated string with a ... ending.
Note that inserting the three dots to the end will add to the string length.
However, if the given maximum string length num is less than or equal to 3, then the addition of the three dots does not add to the string length in determining the truncated string.

If the string is longer than three characters, the string should be truncated at num - 3
If it's three characters or shorted, it should be truncated at num

This doesn't really make sense to me.
Now, I don't want to talk about real world cases too much, but I'll do it a for a little bit.
I think the point of a truncate function is to get an output of a predictable length. With this function, regardless of the input, the output should be num or num + 3.
I suggest we simplify this challenge to always shorten the string to the amount of characters passed in as the second argument (num), making the output string always num + 3.
I thought a bit about it being more complicated intentionally to make the camper dissect the requirements and use if statements, but I think it's a bad idea to intentionally make things more complicated than they would be for the sake of having them practice something.

@systimotic Yes, I agree we should make the objective of this challenge simple. We should update the instructions and the test cases to reflect num + 3.

I agree with you, the instruction are confusing. and The problem seems very simple


fix(versions): update Activiti/activiti-cloud-example-chart versions

UpdateBot pushed helm dependency: application to: 0.0.161

UpdateBot commands:
updatebot push-version --kind helm application 0.0.161


Generics with classes and inheritance possibly broken?

Not sure if this is a bug or I just don't understand how to use generics. Here is a distilled down repro:
https://flowtype.org/try/#0PQKgBAAgZgNg9gdzCYAodBjGBDAzrsAWQE8BhOOGUnfAHgBUA+MAb1TDCgrAF4wAKAEbYATgC4w9AJS9mbDguDAwAEzhgALgAtsGzVoCWAOwDmAOnZgAvqhuoseAgHUdGgKYA3NyLBuAHu5GKgQk5JTUjrS4GiLGJnJWANyYcEbRYH68YEZuSC66nt78UsmoSmAAInBuuEYA5HoIcCIA1mbtqH5mXHD8dfQAogDK9HUlQA
class MyCoolClass<T> {
  foo = (bar: T) => {
     // do that thing.
  }
}
class Whatever extends MyCoolClass<string> {};

const x = new Whatever();

// Doesn't work...
x.foo('TEST');

For my real use-case, I essentially have a set of classes that have the same business logic but only differ in input and output types.  So I made a base class and used generics, yet the "generic" portion doesn't seem to be working as in the above example.

Interestingly, getting rid of the variable binding  and just using a function directly seems to work:
class MyCoolClass<T> {
  foo(bar: T) {
     // do that thing.
  }
}

https://flowtype.org/try/#0PQKgBAAgZgNg9gdzCYAodBjGBDAzrsAWQE8BhOOGUnfAHgBUA+MAb1TDCgoAoAjbAE4AuMPQCUrdhw7BgYACZwwAFwAW2ZStUBLAHYBzAHRSAvqjOoseAgHV1ygKYA3BwLAOAHo93yCJcpTU1rS4ygJ6+swsJgDcmHC6oWAeYAC8YLoOSHYazq7cYnGosmAAInAOuLoA5JoIcAIA1oYtqB6GXHDc1fQAogDK9NWFQA
Which makes me think this is a bug 🐞

class MyCoolClass<T> {
  foo: (t: T) => void = (bar) => {
     // do that thing.
  }
}

Ah, thanks! So it appears this is working as intended and I just didn't understand how to use generics + class instance fields, correct? If so I will close.


BZ1297383: Add style property with height and width to PDF created by…

… "Share Process PDF" menu option

@jlindop same comment for this as in https://github.com/droolsjbpm/jbpm-designer/pull/191

@tsurdilo  Modified according to your comment on  #191. Thanks

+1

6.4.x: 48219691cb2ec3742fd0c915c75718fda799c3b6


Option to change pokemon to candy, if higher level already

hello,
no problem, just a question or a request.
is it already possible, that the bot doesnt carry 2 or more of the same kind but trades the lower ones?
if not... would be great!

This feature is already implemented.
Please upgrade to the latest dev branch by running this command:
git reset --hard
git pull -f

Then run this command:
pip install --upgrade -r requirements.txt
Closing this for now.
/closed


Detect if tab is open when rolling

Via document.hidden I can detect if the tab is open right now. If a tab is closed, I can ignore the roll. This should be an optional setting though

Fixed in 1010fed


If Paypal.me account is deleted while offline then show it as pending until back online

Details
The logic for deleting a Paypal.me payment method was not following our standard for Pattern B Offline First (Optimistic UI With Feedback Pattern) and instead of showing the option crossed when deleting it in offline mode it was simply removing the option as if the request was successfull.
This P.R. fixes that so now if the user is offline and tries to delete the PayPal.me account it will show that payment option grayed out and "strikethrough'ed" to indicate the deletion is pending.
Fixed Issues
$ https://github.com/Expensify/App/issues/17795
PROPOSAL: GH_LINK_ISSUE(COMMENT)
Tests

Sign into NewDot
Go to your Settings -> Payments and add a Paypal.me account
Delete that Paypal.me account and make sure no errors happen
Refresh the page or exit the screen and return then make sure that deleted Paypal.me account doesn't re-appear

[ ] Verify that no errors appear in the JS console

Offline tests

Sign into NewDot
Go to your Settings -> Payments and add a Paypal.me account
Go offline
Delete that Paypal.me account
Make sure that Paypal.me account still shows up but as grayed out and "strikethrough'ed"
Navigate in the app then go back to the Payment screen and make sure the option is still there
Go back online and make sure the Paypal.me account disappears

QA Steps

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

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

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://user-images.githubusercontent.com/6564265/234985788-f9dcd571-3ea4-4bf5-890b-8bcffba2309a.mov

Mobile Web - Chrome
https://user-images.githubusercontent.com/6564265/234985777-e7fa54ac-4ad0-45af-bad6-bc4abb8a860e.mov

Mobile Web - Safari
Can't test offline functionality in simulator so only tested if adding and removing Paypal.me still works.
https://user-images.githubusercontent.com/6564265/234079922-6391ddf8-e26f-46e3-b817-56e1d81ab0ef.mov

Desktop
https://user-images.githubusercontent.com/6564265/234985984-afc0210c-d020-4eb9-be48-28b6ff1014f7.mov

iOS
Can't test offline functionality in simulator so only tested if adding and removing Paypal.me still works.
https://user-images.githubusercontent.com/6564265/234080198-5823fd2d-0178-4848-b78a-b60aceb52202.mov

Android
https://user-images.githubusercontent.com/6564265/234983354-c9eb1ec1-0998-43fb-b6b1-8d5ad4510c85.mov

Screenshots/Videos

Web
https://user-images.githubusercontent.com/25876548/234219242-0825c28c-2fc8-477b-86aa-e212fc17d09c.mov

Can we get the checklists fixed up on this one? Ready to do the final review whenever that is sorted out.

Can we get the checklists fixed up on this one? Ready to do the final review whenever that is sorted out.

Doing that right now @marcaaron! Latest update here

Ready for re-reviews @marcaaron @sobitneupane

Bump @marcaaron : )

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/marcaaron in version: 1.3.10-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/roryabraham in version: 1.3.12-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

Offline testing was not missed here! We didn't realize that both texts should be striked-through'ed but werent for native iOS and Android I think (as can be seen in the description of https://github.com/Expensify/App/pull/18524)


AWS mailer user credentials creation

💡 Summary
What is the work, as a high-level summary?
Motivation and context
Why does this work belong in this project?
This would be useful because...
Implementation notes
Please provide details for implementation, such as:

an example for how this would be used
what this would look like
how this would act
any related work, including links to related issues

Acceptance criteria
How do we know when this work is done?

[ ] Criterion

Blocked state waiting for further guidance


Error when trying to deploy (ERROR running force:source:convert:)

Unhandled Exception : java.lang.RuntimeException: ERROR running force:source:convert: This command is required to run from within an SFDX project.

my issue

Hey @ericrsmith35 , can you please share how you resolved this issue?

I'm sorry, I don't remember.


Mobile storage with saveToPhotos: true

Hi :)
I'm using image-picker to save visit reports photos (out side of cities) on phone. Then, when user could get connection with internet, they send it to an API. So, to store an image until send, i'm saving the image with the prop saveToPhotos: true.
My question is: It keeps the photos tooked at the gallery, but if I delete it, they still persists at reports visualization. So i'm not sure that this photo showed at gallery is really my img, or just a "shortcut".
When using saveToPhotos: true, the lib save image on another place too? And it will be there forever (even deleting it from gallery)?

All photos will be saved in internal temporary cache, I don't know when android will clear it. When saveToPhotos: true is provided that copy of the image will also be saved in photos.

How can access a saved image? can we access save image use response?

@sathiralamal
If you use propr saveToPhotos: true, the images will persist. So, this way you can save the path to the image, and then to access this  you just use this path as uri.
Example:
Saving img:
launchCamera(this.PHOTO_OPTIONS, (response) => {
const source = response.uri
this.setState({ image: source });
}

Acessing img:
<Image
                style={styles.Image}
                source={{ uri: source }}
 />

but uri return temporary location and fileName is different for a saved image in Photoes ?

Yes, thats a thing that I dont understood very well.
I'm saving this URI, because I register some reports during the day, and at night I convert it and send to my API.
But I dont know for how much time it keeps the URI in cache.
If you discover a proper way to do it, please share with me :)

When we taking an image using launchCamera() method, the image is saved in a temporary location so the Image can be deleted at any time, You need to move or copy that Image into Storage. You can use  react-native-fs package to copy or move that image from a temporary location to Storage.   launchCamera() return temporary location uri, you can use that uri for performing move or copy function.
This method allows you to take multiple images and upload them after taking images.

@sathiralamal Hello, I just had time to implement this now, but I'm facing some problems.
My code is:
const hasPermissions = await Util.checkPermissions();
    if (hasPermissions) {
      launchCamera(this.PHOTO_OPTIONS, (response) => {
        if (response.didCancel) {
          return;
        }
        if (response.error) {
          Alert.alert('ImagePicker error: ', `${response.error}`, [
            { text: 'OK', onPress: () => { } },
          ]);
        }

        const destiny = RNFS.DocumentDirectoryPath;

        RNFS.moveFile(response.uri, destiny)
          .then((success) => {
            console.log('file moved!' + success);
          })
          .catch((err) => {
            console.log("Error: " + err.message);
          });
      }

But it keeps giving a error:
Error: EISDIR: illegal operation on a directory, read '/data/user/0/com.fieldreportig/files'
Did you face it?

@gabrielporcher found any solution?
const cameraOptions = {
mediaType: "video",
cameraType: 'back',
quality: 1,
videoQuality: "high",
saveToPhotos: true
};
I am getting corrupted uri file is saved on directory.

@gabrielporcher after move the photo to new location, where is the new path? Is the same called destiny?


Get() returns interface{} in case of no row found

According to the Get function description:

Returns a pointer to a struct that matches or nil if no row is found.

DbMap.Get(i interface{}, keys ...interface{}) should return nil when no row found, but returns empty interface{}.

Feel free to send a PR with either a fix for the documentation or a code change with tests.


_reactNativeAndroidLocationServicesDialogBox2

ReactNativeJS: undefined is not an object (evaluating '_reactNativeAndroidLocationServicesDialogBox2.default.checkLocationServicesIsEnabled')

Be sure to do the correct installation and restart the js server


🚀 Feature: Default --mode if in an existing repo

Bug Report Checklist

[X] I have tried restarting my IDE and the issue persists.
[X] I have pulled the latest main branch of the repository.
[X] I have searched for related issues and found none that matched my issue.

Overview
If you run npx create-typescript-app in an existing repository's directory, it's very unlikely you want the --mode create. You probably want to either --mode initialize if it's a repo based on the template, or --mode migrate it. So let's do that automatically if we detect that the current repository is a Git one.
Investigation: how reliably can we determine which one to use?
Additional Info
Users should still be able to specify --mode if they really want.
If it's not possible to determine which of the two likely modes to use, we should at least move create to last.

This was resolved by #919.


Add gilbertkomin keys

@myf9000 the key name should be same as GitHub username!!!

File name should be the same as github username and placed in files/active/[USERNAME].pub location.

The name should be myf9000.pub instead of gilbertkomin.pub


Add download data for April 2018

Down very slightly from March, with 555,734 downloads from 18,494 unique IPs, and Scala 2.10 downloads for circe-core actually up a bit, to 4.8%.

Codecov Report

Merging #896 into master will increase coverage by 0.12%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #896      +/-   ##
==========================================
+ Coverage   85.56%   85.68%   +0.12%
==========================================
  Files          76       76
  Lines        2459     2459
  Branches      166      157       -9
==========================================
+ Hits         2104     2107       +3
+ Misses        355      352       -3

Impacted Files
Coverage Δ

...rc/main/scala/io/circe/numbers/BiggerDecimal.scala
91.25% <0%> (ø)
:arrow_up:

...d/src/main/scala/io/circe/NonEmptySeqDecoder.scala
100% <0%> (+8.33%)
:arrow_up:

...rc/main/scala/io/circe/shapes/SizedInstances.scala
92.85% <0%> (+14.28%)
:arrow_up:

Continue to review full report at Codecov.

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


[6497] Updating the placements kills the trainee page

Context
Updating the placement details kills the trainee page
Changes proposed in this pull request
Fixed translation for when the placements exceed 6+ placements
Fixed timeline whereby it expects school or school name to change only
Guidance to review
Create 6 or more placements
Change school urn or postcode
View timeline
Important business
N/A

[ ] Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
[ ] Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?
[ ] Do we need to send any updates to DQT as part of the work in this PR?
[ ] Does this PR need an ADR?

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

 'Update record' kills the hesa record after changing the placement details


Check if ETHEREUM_4337_BUNDLER_URL uses the same chainId as ETHEREUM_NODE_URL

What is needed?
It would be great to check if ETHEREUM_4337_BUNDLER_URL uses the same chainId as ETHEREUM_NODE_URL so that we can detect mistakes of using different chain_ids in the same service
In case that chainId is different, an option could be:

Show an error log and do not start

Depends on https://github.com/safe-global/safe-eth-py/pull/899

https://github.com/safe-global/safe-eth-py/pull/899 has been completed 🙌
So this can be scheduled to be implemented 🚀

@luarx PR is ready to review https://github.com/safe-global/safe-transaction-service/pull/2013


Numbers in transactions details don't make sense

This is a transaction for OCNZ:

Is there another number that's supposed to come before +? If not why is it there?
Shouldn't it say US $2,055.42?
It was actually US $2,000.

= -NZ$3,167.56 (net amount for Open Collective Inc)
Doesn't make sense to me... why is this there?

@kewitz is this part of your current work?

@znarf I'll review this in the redesign.


Feature Request: Configurable allow-list of consistency levels

Some of our clusters have many datacenters, which makes QUORUM and EACH_QUORUM inefficient for low latency use-cases.
Being able to configure stargate to only enable certain consistency levels would be a nice safety feature for us, so that our users don't accidentally use a high consistency level at the expense of their availability.
Our use-case is primarily on the GraphQL API, but I suspect this would also be useful to folks in the other APIs.

Duplicate of #729


LANTERN-805: Added a BundleQuerierParser for WRS Health

Pull requests into this repository require the following checks to be completed by the submitter and reviewers.
Submitter:

[x] Administrative tasks are complete.

This pull request describes why these changes were made.
The JIRA ticket links to this PR.
The JIRA ticket for this PR is: LANTERN-805

[x] Tests are complete.

Tests are included and test edge cases.
Tests have been run locally and pass.

[x] Code review has been performed.

Code diff has been done and checked to ensure only expected code is being committed.
A linter for the language has been run.

Primary Reviewer:

[ ] Code works.

Code has been run locally and the basic functionality affected by the PR has been checked.
Edge cases have been checked to ensure behavior is as expected.

[ ] Code review has been performed.

Code is maintainable, reusable, efficient, and correct.
Code accomplishes the tasks purpose.
Code follows style guidance appropriate for the language, including passing linter checks.
Code is well commented (which does not mean verbosely commented).

[ ] Tests are complete.

Tests are included and test edge cases.
Tests have been run locally and pass.

Merger:
Prior to merging, ensure that all checks have been completed. Delete the branch after the merge
is complete.

Approved based on email from Ashley sent on 10-Dec-2024


Unable to Generate valid Client Key/Cert on Ubuntu 18.04

What happened:
I followed the instructions https://falco.org/docs/grpc/#certificates in Ubuntu 18.04 where falco is installed.
I was able to Generate valid Server Key/Cert
I then tried to Generate valid Client Key/Cert on another Ubuntu18.04 client
What you expected to happen:
Signature ok
subject=C = SP, ST = Italy, L = Ornavasso, O = Test, OU = Client, CN = localhost
Getting CA Private Key
How to reproduce it (as minimally and precisely as possible):
On Ubuntu 18.04 try the steps documented for Generate valid Client Key/Cert
Anything else we need to know?:
As you can see in the terminal output we do run into an error when generating the client key, however when you  comment out RANDFILE = $ENV::HOME/.rnd line in /etc/ssl/openssl.cnf. It works. However, the ca.crt key does not get genarated.
Generating RSA private key, 4096 bit long modulus (2 primes)
........................................................................................................++++
............................................................................................++++
e is 65537 (0x010001)
root@client:/tmp# openssl req -passin pass:1234 -new -key client.key -out client.csr -subj  "/C=SP/ST=Italy/L=Ornavasso/O=Test/OU=Client/CN=localhost"
Can't load /root/.rnd into RNG
139812046795200:error:2406F079:random number generator:RAND_load_file:Cannot open file:../crypto/rand/randfile.c:88:Filename=/root/.rnd
root@client:/tmp# nano /etc/ssl/openssl.cnf
root@client:/tmp# openssl req -passin pass:1234 -new -key client.key -out client.csr -subj  "/C=SP/ST=Italy/L=Ornavasso/O=Test/OU=Client/CN=localhost"
root@client:/tmp# openssl x509 -passin pass:1234 -req -days 365 -in client.csr -CA ca.crt -CAkey ca.key -set_serial 01 -out client.crt
Signature ok
subject=C = SP, ST = Italy, L = Ornavasso, O = Test, OU = Client, CN = localhost
Can't open ca.crt for reading, No such file or directory
140055815455168:error:02001002:system library:fopen:No such file or directory:../crypto/bio/bss_file.c:72:fopen('ca.crt','r')
140055815455168:error:2006D080:BIO routines:BIO_new_file:no such file:../crypto/bio/bss_file.c:79:
unable to load certificate
root@client:/tmp# ```

closing the issue because ca.crt was not copied


Feat/events linkasset

Description
This PR allow the user to connect to two different events that are triggered before and after an asset is linked to a device.
Two events when this action is called, allowing to modify the device before it is linked to the asset:
app.pipe.register('device-manager:device:link-asset:before', async data => {
  app.log.debug('before link-asset trigered');
  return data;
})
app.pipe.register('device-manager:device:link-asset:after', async data => {
  app.log.debug('after link-asset trigered');
  return data;
})

close #122

missing new tests for events

functional tests are coming in the next commit :+1:


Throw an expected exception for EXPLAIN ANALYZE INSERT

Summary of the changes / Why this improves CrateDB
Copy of https://github.com/crate/crate/pull/17162 in order to merge.
Checklist

[x] Added an entry in the latest docs/appendices/release-notes/<x.y.0>.rst for user facing changes
[x] Updated documentation & sql_features table for user facing changes
[x] Touched code is covered by tests
[x] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in the latest docs/appendices/release-notes/<x.y.0>.rst
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

Looks like comments from https://github.com/crate/crate/pull/17162 are not addressed?

@mergifyio requeue

@mergifyio requeue


[dunfell][gatesgarth][hardknott][honister][kirkstone] Add recipe for pymavlink

I would like to add pymavlink to this layer.
Some questions:

Should this recipe be name python3-pymavlink instead?
Is the recipe in the right folder?

Merged to other branches as well:
75e30a250..7399bf64c  dunfell -> dunfell-next
0001c0d1e..bc7efd170  gatesgarth -> gatesgarth
0001c0d1e..bc7efd170  gatesgarth -> gatesgarth-next
bcf254b0b..31e5929ee  hardknott -> hardknott
bcf254b0b..31e5929ee  hardknott -> hardknott-next
b948140d6..53d3b8f59  honister -> honister
b948140d6..53d3b8f59  honister -> honister-next
b948140d6..53d3b8f59  master -> master
b948140d6..53d3b8f59  master -> master-next


[Suggest LOI] Crash

java.lang.IllegalStateException: Null LOI ID but no add LOI task
                 	at com.google.android.ground.domain.usecases.submission.SubmitDataUseCase.addLocationOfInterest(SubmitDataUseCase.kt:67)
                 	at com.google.android.ground.domain.usecases.submission.SubmitDataUseCase.invoke(SubmitDataUseCase.kt:54)
                 	at com.google.android.ground.ui.datacollection.DataCollectionViewModel$saveChanges$1.invokeSuspend(DataCollectionViewModel.kt:188)

The logs suggest this survey had no suggest task, but the suggestion card was still displayed to me at a far enough zoom level.


Will support BorderDet？

The original single-point feature can be directly optimized by using the border features, and the SOTA target detection algorithm BorderDet is proposed based on BorderAlign
Motivation
A clear and concise description of the motivation of the feature.
Ex1. Many sliding-window object detectors(such as FCOS,SSD,RetinaNet) adopte the feature maps on the point of the grid to generate the bounding box predictions, but lack the explicit border information for accurate localization.
Ex2.If those detectors use the border information, the performance of the detector will be improved.
Ex3. There is a recent paper《BorderDet: Border Feature for Dense Object Detection》.This paper gives a detailed explanation of the method of using border information to improve the performance of FCOS(mAP: 38.6 v.s. 41.4) and has achieved good results.
Related resources
official code：https://github.com/Megvii-BaseDetection/BorderDet

You may request new features in #2931.


bzip2: update patch URL

Description of changes
For some reason ftp.suse.com refuses connections via IPv6 on HTTPS, which breaks download from IPv6-only network. This commit changes protocol to FTP, which works fine via IPv6 and which is consistent with other mention of this domain in nixpkgs.
Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[x] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[x] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 272640 run on aarch64-darwin 1

Can we merge this or if we don't like the ftp url, just grab the source and make it a source controlled patch to remove the network call?

You may want to backport this to 24.05 as well, since we're past the master cutoff


how does the texure UV coordinates works?

Hi
I am trying to understand your code, and Met one problem about texture,
I see a lot of calling about function  nvg__vset called, to set the X-Y position and UX coordinates.
But most of them contains 0.0, 0.5 or 1.0.
If I am drawing a circle or rectangle, The uv coordinates should not working in this way.
Or there maybe something I missed. Can you explain how the UV works?
Or tell me where I can find the documents
Thanks a hug!!

The texturing is done in pixel shader:
https://github.com/memononen/nanovg/blob/master/src/nanovg_gl.h#L616
The values in nvg__vset() define the anti-alising alpha.
That is, the texturing is defined as a matrix which takes "world" position and returns UV coordinate.


not able to install the plugin

not able to install the plugin
Searching CF-Community for plugin Diego-Enabler...
Plugin Diego-Enabler not found in repository CF-Community.
Use 'cf repo-plugins -r CF-Community' to list plugins available in the repo.
FAILED

Hi @ajitrishabh unfortunately this plugin is in the attic and it is no longer available to be downloaded.

@abbyachau  Okay. so how to i move my app to Diego? because i am not able to use CF Sync Plugin

Hey, @ajitrishabh, you can still install this plugin by downloading one of the binaries from the releases page (https://github.com/cloudfoundry-attic/Diego-Enabler/releases) to some location on your local workstation and then running cf install-plugin $LOCAL_PATH_TO_PLUGIN.
Best,
Eric, CF Diego PM


fix: LSDV-4572: Sound sometimes delayed to playback visual cursor

PR fulfills these requirements

[x] Commit message(s) and PR title follows the format [fix|feat|ci|chore|doc]: TICKET-ID: Short description of change made ex. fix: DEV-XXXX: Removed inconsistent code usage causing intermittent errors
[ ] Tests for the changes have been added/updated (for bug fixes/features)
[ ] Docs have been added/updated (for bug fixes/features)
[x] Best efforts were made to ensure docs/code are concise and coherent (checked for spelling/grammatical errors, commented out code, debug logs etc.)
[x] Self-reviewed and ran all changes on a local instance (for bug fixes/features)

Change has impacts in these area(s)
(check all that apply)

[ ] Product design
[x] Frontend

Describe the reason for change
When loading larger audio files through an audio media element, the playback sometimes will buffer for a period of time which can offset the visual cursor rendered in the waveform, and the sound playback heard.
What does this fix?
Compensates for the time it could take buffering, resynchronizing the currentTime with the audio playback.
What is the new behavior?
N/A
What is the current behavior?
N/A
What libraries were added/updated?
N/A
Does this change affect performance?
N/A
Does this change affect security?
N/A
What alternative approaches were there?
N/A
What feature flags were used to cover this change?
N/A
Does this PR introduce a breaking change?
(check only one)

[ ] Yes, and covered entirely by feature flag(s)
[ ] Yes, and covered partially by feature flag(s)
[X] No
[ ] Not sure (briefly explain the situation below)

What level of testing was included in the change?
(check all that apply)

[-] e2e (existing tests cover this functionality)
[ ] integration
[ ] unit

Which logical domain(s) does this change affect?
Audio V3

Codecov Report
Base: 63.78% // Head: 63.87% // Increases project coverage by +0.08% :tada:

Coverage data is based on head (4322656) compared to base (2fd78b4).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1212      +/-   ##
==========================================
+ Coverage   63.78%   63.87%   +0.08%
==========================================
  Files         390      390
  Lines       24901    24891      -10
  Branches     6358     6354       -4
==========================================
+ Hits        15882    15898      +16
+ Misses       9019     8993      -26

Impacted Files
Coverage Δ

src/lib/AudioUltra/Controls/Player.ts
88.04% <100.00%> (+13.04%)
:arrow_up:

src/lib/AudioUltra/Media/AudioDecoder.ts
83.47% <100.00%> (ø)

src/lib/AudioUltra/Media/MediaLoader.ts
94.68% <100.00%> (ø)

src/lib/AudioUltra/Media/WaveformAudio.ts
92.10% <100.00%> (+0.97%)
:arrow_up:

src/components/VideoCanvas/VideoCanvas.tsx
69.14% <0.00%> (-0.79%)
:arrow_down:

src/lib/AudioUltra/react/index.ts
97.43% <0.00%> (+2.56%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Training takes too long

Hi!
I am trying to train basset with my own data but I have been running into several issues with the gpus so I decided to try on the normal nodes in our cluster, however it's been running for 4 days now and I have far less sequences than the used amount in the paper so I don't know if there is any chance something went wrong without killing the process and now it's just halted or it really takes that long to run.
All the best,
Laura

Hi Laura,
Unfortunately, you'll only be able to do very small analyses on CPUs. E.g you could score on the order of hundreds of SNPs or computed saturation mutagenesis maps for dozens of sequences. Anything beyond that really needs a GPU to speed things up.
Best,
Dave

Hi Dave,
Thanks for your reply! Then I will try to get it running on the GPUs. Do you have any recommendations on how many GPU/cores/memory I should use for training a model with ~300,000 sequences?
Best,
Laura

One GPU should be fine


Prevent Textfield to show prefix/suffix components only when TextField has focus/content

Description
The actual implementation of TextField only show prefix/suffix when the field has focus or content is not empty. I consither this a bug since this does not follow the Material Design specs.
As we can see here: https://material.io/components/text-fields#filled-text-field the text field shows the suffix even when it is empty. We can see more examples at: https://material.io/components/text-fields#anatomy.
The purpose of this PR is to show the prefix/suffix if they are provided.
Related Issues
https://github.com/flutter/flutter/issues/56543
Tests
To be added.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I signed the [CLA].
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read [Handling breaking changes].

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: https://flutter.dev/go/breaking-changes-template Replace this with a link to a pull request that adds the migration guide to https://flutter.dev/docs/release/breaking-changes

@gspencergoog The following test is failing:
https://github.com/pedromassango/flutter/blob/7657e6f718f65fca97af842e2bda89cc55e8534c/packages/flutter/test/material/input_decorator_test.dart#L4179-L4237
Maybe this is a breaking change because with the proposed implementation in this PR the prefix and the label text are overlapping eachother. The label text is flyig too close to the prefix widget.


Error:Int64 ID insert fail

Sometimes, a class has a ID field that is a Int64 type, and the insertion operation would fail. Said that the duplicate index value is (null). It seems that the error appears after the EnsurceIndex ("ID", true)?only can be inserted once time, even if the ID value is simple as 1, 2, 3.

Hi @suxiaojack, there is not built-in Int64 auto-increment support. Only for Int32. If you want you can create your own. Take a look here how Int32 was implemented:
https://github.com/mbdavid/LiteDB/blob/master/LiteDB/Mapper/BsonMapper.cs#L142

Thanks @mbdavid , sorry, I can't understand，I'm not used the autoid functional,just manually set the id value and try insert.But why, once I commented to remove the line of EnsureIndex("ID",true), it works fine.

Ok, you have no "ID" field (and if you have this property in you class it's converted to _id document field), so, when you create an unique index in this field that not exists, first index key contains "null" and in secound insert try insert another null and this throw duplicate key.

Aha, that's why. I think the help documentation should indicate that we don't need to write EnsureIndex ("ID", true) ,EnsureIndex ("Id", true) ,EnsureIndex ("id", true) ,and so on.Because there is only one _id document field,it's easy to confuse.Thanks @mbdavid !


Fix some HF models exit without tuning

Type of Change

bug fix
None API changed

Description
detail description
JIRA ticket:  ILITV-2749
How has this PR been tested?
Pre-CI
extension test
Dependency Change?
None

/Azurepipeline run Model-Test


extra/readings: Add Computer Systems: A Programmer's Perspective book

This is probably the best introduction to computer systems for programmers. It contains all needed essentials of computer architecture, describes how C language works internally (e.g. how data structures are organized in memory and managed on machine language level) and even describes essentials of operating systems.

This looks great. Thanks!


[SKIP-953] Exclude namespaces using a configmap

This PR attempts to use a ConfigMap called namespace-exclusions in the skiperator-system namespace to avoid reconciling certain namespaces. This is a lot more agile than having to add and redeploy skiperator for every change we want to make to this list.
The commit a5a43d5 was added simply because I started working in that folder before realising that would not work. Can be removed from PR if wanted, but its somewhat of a clean-up so I let it stay.
I am not sure if we want to keep the static list in the code or if we want to exclusively use the config map. In addition, I'm not sure if we should keep an example of this config map in the repo, and where it should stay if that is the case. Awaiting feedback here :)
It would be preferable to use the WithEventFilter function on the manager, as earlier, but you are not allowed to fetch kubernetes resources before the controllers are setup. As such, I'm not sure this approach is possible.

Also worth mentioning: Adding a namespace to either the in-code list or the ConfigMap will not delete resources created by namespace reconciliation. This must be done manually. I'm not entirely sure how we should go about deleting, perhaps someone else might know how.

@esphen @evenh @omaen
Changes according to comments have been made :)


Support HTTP Proxy for WebSocketPeer

Issue description
Affected version: 4.3.stable
Godot currently only supports HTTP proxying for HTTPClient but not WebSocketPeer and WebSocketMultiplayerPeer interfaces that also use HTTP internally. It should expose the same set_http_proxy and set_https_proxy methods that HTTPClient has.
Steps to reproduce
Try to use Godot to connect with Websocket through a proxy.

Feature proposals are tracked here, this is for bugs. Please open a proposal instead.


Add pyproject.toml for Custom Node Registry

We are building a global registry for custom nodes (similar to PyPI). The registry makes using custom nodes more reliable by reserving globally unique names for each custom node, and supporting semantic versioning.
Here’s some more information on the registry.
Action Required:

[x] Go to the registry. Login and create a publisher id. Comment it here and we will update the PR (or just merge it and change it yourself).
[x] Write a short the description.
[x] Merge the separate Github Actions PR and run the workflow.

If you want to publish the node manually, install the cli and run comfy node publish
Please message me on Discord if you have any questions!

https://comfyregistry.org/publishers/eagleshadow
Hopefully I did everything I was supposed to. Sorry I don't have much prior experience with github.


For a 0.2.1 release

#171 #172

Codecov Report

Merging #173 (80afa37) into master (5e5d698) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #173   +/-   ##
=======================================
  Coverage   90.74%   90.74%
=======================================
  Files           8        8
  Lines         216      216
=======================================
  Hits          196      196
  Misses         20       20

Continue to review full report at Codecov.

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


[FEATURE] Add broken link checker and issue bot

Add broken link checker and issue bot to create a GitHub issue when a broken link is seen in a build
Am I willing to work on this?
Not currently. Happy for someone else to take the torch on this one.

@VeckoTheGecko - I was thinking of using node cron and just doing a sweep on a daily basis if you want? Do you have any implementation details?

I was thinking a GH Actions workflow that is cron scheduled calling a script would be a good way to go about it (I'm not sure how node cron works, but I think that might be a less elegant implementation?).
Implementation details:

Check for non 2xx codes (e.g. we're also interested in 301 (domain moved) for example so we can update the links)
Weekly cron (less spammy in case there's a website that temporarily goes down for a few days)
A way of surfacing failing links (easy: Failing CI/CD; hard: automatic creation of GitHub issue)
Python preffered (although that's a soft requirement. If the implementation code is clean then I'm happy for it to be any language (please not golfscript))

Feel free to discuss any of those details. They're not hard and fast.


Stack fails setup when precompiled Prisma engines are not available.

Have you experienced this bug with the latest version of the template?
Yes
Steps to Reproduce
On NixOS or in a Nix Shell, run npx create-remix --template remix-run/indie-stack

View Output
Need to install the following packages:
  create-remix
Ok to proceed? (y) y
? Where would you like to create your app? ./test-stack
? Do you want me to run `npm install`? Yes
npm WARN deprecated<EMAIL_ADDRESS>See https://github.com/lydell/source-map-url#deprecated
npm WARN deprecated<EMAIL_ADDRESS>Please see https://github.com/lydell/urix#deprecated
npm WARN deprecated<EMAIL_ADDRESS>See https://github.com/lydell/source-map-resolve#deprecated
npm WARN deprecated<EMAIL_ADDRESS>https://github.com/lydell/resolve-url#deprecated
npm WARN deprecated<EMAIL_ADDRESS>This package has been deprecated and is no longer maintained. Please use @rollup/plugin-inject.
npm WARN deprecated<EMAIL_ADDRESS>See https://github.com/lydell/source-map-resolve#deprecated

added 1328 packages, and audited 1329 packages in 19s

235 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities
? TypeScript or JavaScript? TypeScript
💿 Running remix.init script

> setup
> prisma migrate dev && prisma db seed

Warning Precompiled engine files are not available for nixos.
Environment variables loaded from .env
Prisma schema loaded from prisma/schema.prisma
Datasource "db": SQLite database "data.db" at "file:./data.db?connection_limit=1"

The following migration(s) have been applied:

migrations/
  └─<PHONE_NUMBER>0657_init/
    └─ migration.sql

Your database is now in sync with your schema.

Running generate... (Use --skip-generate to skip the generators)
Warning Precompiled engine files are not available for nixos.
TypeError: collection is not iterable
    at keyBy (/home/short/work/test-stack/node_modules/@prisma/client/generator-build/index.js:57531:21)
    at Object.getTypeMap (/home/short/work/test-stack/node_modules/@prisma/client/generator-build/index.js:59314:17)
    at new DMMFHelper (/home/short/work/test-stack/node_modules/@prisma/client/generator-build/index.js:59211:25)
    at new TSClient (/home/short/work/test-stack/node_modules/@prisma/client/generator-build/index.js:60494:17)
    at buildClient (/home/short/work/test-stack/node_modules/@prisma/client/generator-build/index.js:60740:18)
    at generateClient (/home/short/work/test-stack/node_modules/@prisma/client/generator-build/index.js:60811:47)
    at async LineStream.<anonymous> (/home/short/work/test-stack/node_modules/@prisma/client/generator-build/index.js:54039:24)

Warning Precompiled engine files are not available for nixos.
Environment variables loaded from .env
Running seed command `ts-node --require tsconfig-paths/register prisma/seed.ts` ...
/home/short/work/test-stack/node_modules/.prisma/client/index.js:3
    throw new Error(
          ^
Error: @prisma/client did not initialize yet. Please run "prisma generate" and try to import it again.
In case this error is unexpected for you, please report it in https://github.com/prisma/prisma/issues
    at new PrismaClient (/home/short/work/test-stack/node_modules/.prisma/client/index.js:3:11)
    at Object.<anonymous> (/home/short/work/test-stack/prisma/seed.ts:4:16)
    at Module._compile (node:internal/modules/cjs/loader:1099:14)
    at Module.m._compile (/home/short/work/test-stack/node_modules/ts-node/src/index.ts:1455:23)
    at Module._extensions..js (node:internal/modules/cjs/loader:1153:10)
    at Object.require.extensions.<computed> [as .ts] (/home/short/work/test-stack/node_modules/ts-node/src/index.ts:1458:12)
    at Module.load (node:internal/modules/cjs/loader:975:32)
    at Function.Module._load (node:internal/modules/cjs/loader:822:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:77:12)
    at phase4 (/home/short/work/test-stack/node_modules/ts-node/src/bin.ts:567:12)

An error occured while running the seed command:
Error: Command failed with exit code 1: ts-node --require tsconfig-paths/register prisma/seed.ts
🚨 Oops, remix.init failed

Command failed: npm run setup

Investigating a bit, it seems that running prisma generate prior to running npm run setup will succeed.

Run prisma generate

View Output
Warning Precompiled binaries are not available for nixos.
Environment variables loaded from .env
Prisma schema loaded from prisma/schema.prisma
Warning Precompiled binaries are not available for nixos.
[Error: ENOENT: no such file or directory, mkdir '/nix/store/vn57rafwfanmrzjndnlcr4q56r55p9k8-prisma-3.12.0/lib/node_modules/prisma/engines'] {
  errno: -2,
  code: 'ENOENT',
  syscall: 'mkdir',
  path: '/nix/store/vn57rafwfanmrzjndnlcr4q56r55p9k8-prisma-3.12.0/lib/node_modules/prisma/engines'
}
Warning Precompiled binaries are not available for nixos.

✔ Generated Prisma Client (3.14.0 | library) to ./node_modules/@prisma/client in 75ms
You can now start using Prisma Client in your code. Reference: https://pris.ly/d/client
```
import { PrismaClient } from '@prisma/client'
const prisma = new PrismaClient()
```

warn Versions of<EMAIL_ADDRESS>and<EMAIL_ADDRESS>don't match.
This might lead to unexpected behavior.
Please make sure they have the same version.

Run npm run setup

View Output
> setup
> prisma migrate dev && prisma db seed

Warning Precompiled engine files are not available for nixos.
Environment variables loaded from .env
Prisma schema loaded from prisma/schema.prisma
Datasource "db": SQLite database "data.db" at "file:./data.db?connection_limit=1"

Already in sync, no schema change or pending migration was found.

Running generate... (Use --skip-generate to skip the generators)
Warning Precompiled engine files are not available for nixos.
TypeError: collection is not iterable
    at keyBy (/home/short/work/test-stack/node_modules/@prisma/client/generator-build/index.js:57531:21)
    at Object.getTypeMap (/home/short/work/test-stack/node_modules/@prisma/client/generator-build/index.js:59314:17)
    at new DMMFHelper (/home/short/work/test-stack/node_modules/@prisma/client/generator-build/index.js:59211:25)
    at new TSClient (/home/short/work/test-stack/node_modules/@prisma/client/generator-build/index.js:60494:17)
    at buildClient (/home/short/work/test-stack/node_modules/@prisma/client/generator-build/index.js:60740:18)
    at generateClient (/home/short/work/test-stack/node_modules/@prisma/client/generator-build/index.js:60811:47)
    at async LineStream.<anonymous> (/home/short/work/test-stack/node_modules/@prisma/client/generator-build/index.js:54039:24)

Warning Precompiled engine files are not available for nixos.
Environment variables loaded from .env
Running seed command `ts-node --require tsconfig-paths/register prisma/seed.ts` ...
Database has been seeded. 🌱

🌱  The seed command has been executed.

Expected Behavior
The stack should not fail due to precompiled Prisma engines not existing. Instead, prisma generate should be run first or the Prisma step should be skipped and the reponsibility of performing Prisma setup handed to the user.
Actual Behavior
When attempting to setup a new project using this Remix Stack, the process fails before completing. Precompiled Prisma engines are not available in a Nix environment due to how packages, linking, and the environment are configured. Instead, it is expected that prisma generate is run to generate a client that uses Nix's version of Prisma since Nix environments are typically created per-project as seen here.

There are a few ways this could be handled, so I wanted to hear your folks' thoughts. From my perspective, we could:
a. Always run prisma generate first.
I believe that Prisma will use precompiled binaries for supported systems in this case, but I could be wrong. If this introduces waiting time for everyone else then it's not worth it. Though if it is a freebie, then this would be the preferred solution.
b. Try npm run setup and retry with prisma generate if an error occurred.
A bit more hacky, but it would be possible to attempt running setup first without prisma generate. Then, if an error occurred, we could generate the Prisma client and retry. The naive implementation here wouldn't know what error occurred, so it's possible we waste some time if there's another problem with Prisma setup.


Pyscaffold and Setuptools < 30.3.0

When installing pyscaffold I get the following:
  File "/tmp/easy_install-c56y92hx/PyScaffold-3.0.2/setup.py", line 76, in bootstrap_cfg
  File "/tmp/easy_install-c56y92hx/PyScaffold-3.0.2/src/pyscaffold/utils.py", line 280, in check_setuptools_version
pyscaffold.exceptions.OldSetuptools: Your setuptools version is too old (<30.3.0). Use `pip install -U setuptools` to upgrade.

Should setuptools >=30.3.0 should be included in the setup.cfg of the package?

Thank you. Not sure if there was a reason I didn't specify this dependency.
Let me check when I am back from vacation. Thanks for your PR!
On Mon, 9 Apr 2018, 11:23 Juan Leni<EMAIL_ADDRESS>wrote:

When installing pyscaffold I get the following:
File "/tmp/easy_install-c56y92hx/PyScaffold-3.0.2/setup.py", line 76, in bootstrap_cfg
File "/tmp/easy_install-c56y92hx/PyScaffold-3.0.2/src/pyscaffold/utils.py", line 280, in check_setuptools_version
pyscaffold.exceptions.OldSetuptools: Your setuptools version is too old (<30.3.0). Use pip install -U setuptools to upgrade.
Should setuptools >=30.3.0 should be included in the setup.cfg of the
package?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/blue-yonder/pyscaffold/issues/155, or mute the thread
https://github.com/notifications/unsubscribe-auth/ADCVyE7AEDetT_DSrFiWsP4GOVstX1_yks5tmyh5gaJpZM4TMNkq
.

Well, it definitely makes sense. It's merged. Thank you!

Is there any chance we could have pyscaffold 3.0.3 released with the fix?


Pull request

Description

[ x ] I have added a folder(Named it as my GitHub user id)
[ x ] I have added a Readme.md file inside the folder
[ x ] I understand the GitHub Forking Concept fully

Thanks for creating this helpful public repo.


Parser: Treat compound literals as a suffix expression

The C grammar treats them as suffix expressions instead of cast expressions
Closes #451

Is there a reason to not make it a primaryExpr?

I guess this does match the official grammar better:
postfix-expression:
    primary-expression
    postfix-expression [ expression ]
    postfix-expression ( argument-expression-listopt )
    postfix-expression . identifier
    postfix-expression -> identifier
    postfix-expression ++
    postfix-expression -
    ( type-name ) { initializer-list }
    ( type-name ) { initializer-list , }

argument-expression-list:
    assignment-expression
    argument-expression-list , assignment-expression

Functionally I think it would be the same if we put it there; the only potential problem would be if primaryExpr is ever called from anywhere else (currently it's only called from one place) and that new calling location doesn't allow the possibility of a compound literal.