chore: rename process to propose for clarity

Changes the naming of the process function on the rollup to become propose since it is a proposal for a block.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#8265  👈
#8262
#8148
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @LHerskind and the rest of your teammates on  Graphite

Benchmark results
Metrics with a significant change:

protocol_circuit_simulation_time_in_ms (private-kernel-inner): 236 (+38%)
protocol_circuit_simulation_time_in_ms (private-kernel-tail-to-public): 760 (-45%)
avm_simulation_time_ms (FeeJuice:set_portal): 19.6 (+96%)
avm_simulation_time_ms (Token:mint_public): 79.8 (+77%)
avm_simulation_time_ms (FPC:prepare_fee): 276 (+19%)
avm_simulation_time_ms (Token:transfer_public): 36.3 (+77%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Proof generation
Each column represents the number of threads used in proof generation.

Metric
1 threads
4 threads
16 threads
32 threads
64 threads

proof_construction_time_sha256_ms
5,770
1,568
715
762
786 (+2%)

proof_construction_time_sha256_30_ms
11,448 (-1%)
3,081
1,391 (+1%)
1,434 (+2%)
1,463

proof_construction_time_sha256_100_ms
44,455 (+1%)
11,926 (+1%)
5,471
5,654 (+3%)
5,707 (+1%)

proof_construction_time_poseidon_hash_ms
78.0
34.0
34.0
58.0 (-3%)
88.0 (-1%)

proof_construction_time_poseidon_hash_30_ms
1,531
441 (+5%)
204
230 (+5%)
269

proof_construction_time_poseidon_hash_100_ms
5,668
1,520 (+1%)
674 (-3%)
725
749 (+1%)

L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
4 txs
8 txs
16 txs

l1_rollup_calldata_size_in_bytes
4,356
7,876
14,884

l1_rollup_calldata_gas
50,244
93,022
178,108

l1_rollup_execution_gas
845,578
1,579,252
3,364,394

l2_block_processing_time_in_ms
262
445 (-2%)
833

l2_block_building_time_in_ms
11,400
22,237
44,714 (+1%)

l2_block_rollup_simulation_time_in_ms
11,400
22,237
44,714 (+1%)

l2_block_public_tx_process_time_in_ms
9,674
20,434
42,873 (+1%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 8 txs.

Metric
3 blocks
5 blocks

node_history_sync_time_in_ms
2,999 (-3%)
3,903

node_database_size_in_bytes
12,599,376
16,719,952

pxe_database_size_in_bytes
16,254
26,813

Circuits stats
Stats on running time and I/O sizes collected for every kernel circuit run across all benchmarks.

Circuit
simulation_time_in_ms
witness_generation_time_in_ms
input_size_in_bytes
output_size_in_bytes
proving_time_in_ms

private-kernel-init
97.9 (+7%)
393
21,735
44,860
N/A

private-kernel-inner
:warning: 236 (+38%)
673 (-4%)
72,544
45,007
N/A

private-kernel-reset-tiny
309
688 (-4%)
65,593
44,846
N/A

private-kernel-tail
166
131 (-6%)
50,644
52,257
N/A

base-parity
5.54 (-1%)
N/A
160
96.0
N/A

root-parity
35.4 (-1%)
N/A
73,948
96.0
N/A

base-rollup
3,458 (+1%)
N/A
189,136
664
N/A

block-root-rollup
41.0 (-1%)
N/A
58,205
2,448
N/A

public-kernel-setup
85.8 (+2%)
N/A
105,085
71,222
N/A

public-kernel-app-logic
97.1 (+1%)
N/A
104,911
71,222
N/A

public-kernel-tail
856
N/A
390,582
16,414
N/A

private-kernel-reset-small
308 (-2%)
N/A
66,341
45,629
N/A

private-kernel-tail-to-public
:warning: 760 (-45%)
586 (-5%)
455,400
1,825
N/A

public-kernel-teardown
83.6 (-1%)
N/A
105,349
71,222
N/A

merge-rollup
20.3 (+1%)
N/A
38,174
664
N/A

undefined
N/A
N/A
N/A
N/A
83,886 (+6%)

Stats on running time collected for app circuits

Function
input_size_in_bytes
output_size_in_bytes
witness_generation_time_in_ms

ContractClassRegisterer:register
1,344
11,731
343

ContractInstanceDeployer:deploy
1,408
11,731
18.1 (-1%)

MultiCallEntrypoint:entrypoint
1,920
11,731
403 (-1%)

FeeJuice:deploy
1,376
11,731
385 (-2%)

SchnorrAccount:constructor
1,312
11,731
72.9 (-1%)

SchnorrAccount:entrypoint
2,304
11,731
394

Token:privately_mint_private_note
1,280
11,731
103 (+1%)

FPC:fee_entrypoint_public
1,344
11,731
27.3 (-9%)

Token:transfer
1,312
11,731
224 (-1%)

Benchmarking:create_note
1,344
11,731
84.3 (-2%)

SchnorrAccount:verify_private_authwit
1,280
11,731
27.9 (+1%)

Token:unshield
1,376
11,731
522 (-1%)

FPC:fee_entrypoint_private
1,376
11,731
700 (-1%)

AVM Simulation
Time to simulate various public functions in the AVM.

Function
time_ms
bytecode_size_in_bytes

FeeJuice:_increase_public_balance
46.4 (-29%)
7,101

FeeJuice:set_portal
:warning: 19.6 (+96%)
2,128

Token:constructor
89.7 (+11%)
25,285

FPC:constructor
53.8 (-2%)
17,853

FeeJuice:mint_public
39.3 (-2%)
5,415

Token:mint_public
:warning: 79.8 (+77%)
10,101

Token:assert_minter_and_mint
51.4 (-22%)
6,844

AuthRegistry:set_authorized
48.3 (+18%)
3,969

FPC:prepare_fee
:warning: 276 (+19%)
6,747

Token:transfer_public
:warning: 36.3 (+77%)
38,126

FPC:pay_refund
53.2 (+7%)
9,398

Benchmarking:increment_balance
1,202 (+1%)
6,179

Token:_increase_public_balance
41.7 (+4%)
7,705

FPC:pay_refund_with_shielded_rebate
62.0 (-4%)
9,881

Public DB Access
Time to access various public DBs.

Function
time_ms

get-nullifier-index
0.168 (+5%)

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
256 leaves
512 leaves
1024 leaves

batch_insert_into_append_only_tree_16_depth_ms
2.16 (-2%)
3.94 (-1%)
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.8
31.7
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.112 (-3%)
0.111 (-1%)
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
11.3 (-1%)
17.6 (-3%)
31.7 (-2%)
59.3
113 (-3%)

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
95.9
159
287
543
1,055

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.107 (-2%)
0.102 (-2%)
0.103 (-1%)
0.102
0.101 (-2%)

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
14.2 (-2%)
25.4 (-3%)
45.3
82.0 (-1%)
163 (-1%)

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
109
207
355
691
1,363

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.107 (-2%)
0.102 (-2%)
0.110 (+1%)
0.101 (-1%)
0.103 (-1%)

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
16.5 (-1%)
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
132
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
0.106 (-1%)
N/A
N/A
N/A
N/A

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes
1 registered classes

tx_size_in_bytes
64,779
668,997

Transaction size based on fee payment method
| Metric |  |
| - |  |


code have bug

in code:
public subscript (position: JSON.Index) -> JSON.Generator.Element {
switch self.type {
case .Array:
return (String(position.arrayIndex), JSON(self.rawArray[position.arrayIndex!]))
case .Dictionary:
let (key, value) = self.rawDictionary[position.dictionaryIndex!]
return (key, JSON(value))
default:
return ("", JSON.null)
}
}
String(position.arrayIndex)  "Optional(0)" not change : let index = Int(index)//nil
String(position.arrayIndex!) Optional(0) can change : let index = Int(index)// 0
please edit String(position.arrayIndex)  ->  String(position.arrayIndex!)

There hasn't been much activity on this issue for a while so I'm gonna go ahead and consider it fixed. Let me know if you'd like to reopen it.


Remove clear and cursor move in min reporter

Relates to #2312
Fixes #2761
Description of the Change
Affecting the entire view of the console is not minimal and it prevents this from being used in conjunction with other watch scripts running in parallel in the same process, such as linters and nodemon, docker, etc.
Its not appropriate to do this in a reporter and a separate --clearscreen feature should be added if anyone wants that feature specifically.
Alternate Designs

Adding a new reporter called even-more-min which was identical to min but just didn't clear screen or move the cursor.
Adding a --clearscreen false option which has been rejected by mocha devs as being too complex in an already complicated feature.

Why should this be in core?
It is a reduction in features. It removes functionality already in core that negatively impacts some scenarios, especially running in docker or in conjunction with other watch scripts.
Benefits
It will allow the min reporter to be able to be used in the same process as other "watch" scripts. This is really important for being able to be run in docker, for example. When running in docker many processes are emitting output and being displayed into the console of a single shell, if any of those processes does a "clear screen" or a move of a cursor it can wipeout the output of other processes. It's not polite to do this and should be an opt-in only feature.
Possible Drawbacks
The draw back is that upon success during --watch -R min the minimal reporter will append the epilogue rather than controlling the entire console window. Over time you will accumulate messages such as 300 passing (400ms).
This is necessary in a multi-child process terminal but may not be as pretty in 5 minute demos.
Applicable issues
This is technically a breaking change but I don't think it actually warrants a major version bump. I would recommend a minor version bump.

Generally, I also like the idea of the screen not being cleared by the min reporter.
@boneskull is there a reason why we might want to consider leaving it in, other than the scenario presented in "Possible Drawbacks"?
Otherwise LGTM 👍

I believe the reason why is just so you can get that experience of doing:
mocha -r min

And then you get that "minimalist" output of just seeing your failing tests as you develop. Look here at the documentation, I think it explains the reasoning:
https://mochajs.org/#min

The “min” reporter displays the summary only, while still outputting errors on failure. This reporter works great with --watch as it clears the terminal in order to keep your test summary at the top.

And now that I mention it, I will update the documentation to remove the comment about clearing the terminal also. But I believe that was the entire reasoning.


[cosmos-db] incorrect type definition in SqlRoleDefinitionResource

Description
According to the definition of SqlRoleDefinitionResource, the type property should be a string.
The corresponding example also verify that. See the type property in both request and response.
However, in reality:

the type value in request (e.g. when used in SqlRoleDefinitionCreateUpdateParameters)  is a string.

{
  "properties": {
    "roleName": "xxxxxxx",
    "type": "CustomRole", // This is a string aligned with definition
    "assignableScopes": [
      "/subscriptions/xxxxx/resourceGroups/xxxx/providers/Microsoft.DocumentDB/databaseAccounts/dbaccount-xxxxx/dbs/sql-db-xxxx"
    ],
    "permissions": [
      {
        "dataActions": [
          "Microsoft.DocumentDB/databaseAccounts/sqlDatabases/containers/items/create"
        ]
      }
    ]
  }
}

the type value in response (e.g. when used in SqlRoleDefinitionGetResults) is an integer

{
  "id": "/subscriptions/xxxxx/resourceGroups/xxx/providers/Microsoft.DocumentDB/databaseAccounts/dbaccount-xxxx/sqlRoleDefinitions/xxxx",
  "name": "xxxxx",
  "properties": {
    "roleName": "xxxx",
    "type": 1, // Now it becomes an integer
    "assignableScopes": [
      "/subscriptionsxxxx/resourceGroups/xxxxx/providers/Microsoft.DocumentDB/databaseAccounts/dbaccount-xxxx/dbs/sql-db-xxxx"
    ],
    "permissions": [
      {
        "dataActions": [
          "Microsoft.DocumentDB/databaseAccounts/sqlDatabases/containers/items/create"
        ],
        "notDataActions": []
      }
    ]
  },
  "type": "Microsoft.DocumentDB/databaseAccounts/sqlRoleDefinitions"
}

Looks like the server side returns a response with wrong type.

Hi @msftbot, @wmengmsft & @MehaKaushik
Do you perhaps have an update or estimated date to resolve issue #16560 ?
It is currently blocking #13907 , which we are dependent on.
Thank you very much in advance!

Any update on when this will get resolved? It's blocking https://github.com/hashicorp/terraform-provider-azurerm/issues/13907, which is pretty important for CosmosDB RBAC access. Thank you.

Hi, any update?

The fix has been deployed and is live in Production.


Zeit micro support

This adds support for Zeit micro to allow GraphQL to be served as a function.  Fixes issue #324
TODO:

[X] Update CHANGELOG.md with your change (include reference to issue & this PR)
[X] Make sure all of the significant new logic is covered by tests
[X] Rebase your changes on master so that they can be merged easily
[X] Make sure all tests and linter rules pass

I think we might need an example on the README and update the list of integrations.

That's a good suggestion
On Sun, Apr 2, 2017 at 9:55 AM Reyad Attiyat<EMAIL_ADDRESS>wrote:

I think we might need an example on the README and update the list of
integrations.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/apollographql/graphql-server/pull/347#issuecomment-290998826,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA-x3K7-Z7HJm2J2ghQeGXT7De1XeHmLks5rr9MGgaJpZM4Mws50
.


Extra data: line 1 column 189067 (char 189066)

Discussed in https://github.com/asreview/asreview/discussions/1188

Originally posted by MireyK July 20, 2022
Hello
While working on a project on AS reviews I got this error message :
Extra data: line 1 column 189067 (char 189066)
now i can not continue my screening
Please  help
Mirey

Hi @mireyK, what version of ASReview are you using?
Can you still export your ASReview file? If so, can you send it to me<EMAIL_ADDRESS>

I received the file from @mireyK (Thanks). It turned out to be an issue with the converter, caused by an old issue https://github.com/asreview/asreview/issues/792. This issue is resolved in V1.
Fix:

Open pool.json in your project in the .asreview folder.
Remove the last few characters as described in https://github.com/asreview/asreview/issues/792.
You can convert the file now.


Get Current Location

How to get the current location?
subscribeToSignificantLocationUpdates will only fire if there is a change in location, what if I just want the current location.
I didn't see any method for that in the documentation.

When you call RNLocation.subscribeToLocationUpdates it will usually call your callback right away with the last known location of the device, however, this is not guaranteed to happen as the device might not have a known location (for example, it has been in Airplane mode since being turned on).
You can subscribe to the events, and then unsubscribe when you receive your first callback. For example:
const unsubscribe = RNLocation.subscribeToLocationUpdates(locations => {
  // Do something with the locations

  if (unsubscribe) {
    unsubscribe();
  }
});

It would be possible for the library to provide a method which essentially implements this for. I will leave this issue open as a feature request for that and let you know when an update in released.

This is now included in v2.1.0 which is released on npm.

Where is the function unsubscribe from the watch?

@roysG From the code above, calling the RNLocation.subscribeToLocationUpdates method returns a function which, when called, will unsubscribe from location updates.
For example:
const unsubscribe = RNLocation.subscribeToLocationUpdates(locations => {
  // Do something with the locations
});

// Later on, unsubscribe
unsubscribe();

@matt-oakes , thanks it works.
I have to say, the "subscribeToLocationUpdates" function works amazing in compare to "getCurrentPostion".
I made some tests to check the accurate of the function "getCurrentPostion" and every check the update location is happen immediately just after some 10 meters.
Maybe i miss something? Is there any best practice configuration for this method?

@roysG If you need to check location every few seconds then you should use subscribeToLocationUpdates to subscribe to updates. You can configure that with your desired accuracy and minimum displacement to get the best balance between battery use and accuracy.
The best resources for best practice are the official Apple and Android documentation:
Apple: https://developer.apple.com/documentation/corelocation?language=objc
Android: https://developer.android.com/guide/topics/location/battery

I read the documentation and i little bit feel confusing.
Lets say you define the next config:
RNLocation.configure({
distanceFilter: 100, // Meters
desiredAccuracy: {
android: "highAccuracy"
}
})
RNLocation.requestPermission({
android: {
detail: "coarse"
}
}).then(granted => {
if (granted) {
this.locationSubscription = RNLocation.subscribeToLocationUpdates(locations => {
})
}
})
what priority will win?
(As you know coarse is just working with tower and wifi and "highAccuracy" works with gps.)

The permission levels and desired accuracy are two different things but to get highly accurate locations you will need to requestion "fine" permissions on Android.


[bug] Connecting with Email or Social opens a window for connection and wallet is not connected

Link to minimal reproducible example
https://github.com/0xonerb/next-reown-appkit-email-social
Summary
I disconnected Gmail login, connected with MetaMask, then disconnected MetaMask and tried connected with Gmail again and it got stuck here (see the images below). When I refreshed the tab I could connect with Gmail again.
https://github.com/user-attachments/assets/29227f65-ee26-4d81-9491-3a658b70ef14

List of related npm package versions
"dependencies": {
  "@reown/appkit": "^1.0.4",
  "@reown/appkit-adapter-wagmi": "^1.0.4",
  "@tanstack/react-query": "^5.56.2",
  "@wagmi/core": "^2.13.7",
  "next": "14.2.13",
  "react": "^18",
  "react-dom": "^18",
  "viem": "^2.21.15",
  "wagmi": "^2.12.14"
},
"devDependencies": {
  "@types/node": "^20",
  "@types/react": "^18",
  "@types/react-dom": "^18",
  "eslint": "^8",
  "eslint-config-next": "14.2.13",
  "postcss": "^8",
  "tailwindcss": "^3.4.1",
  "typescript": "^5"
}

why is this not being fixed? social logging is not working for me either, even with v1.3.0

I'm not having problems to connect after this errors.

Internal -> https://linear.app/reown/issue/APKT-1736/investigate-common-errors-with-google-social-login


Export to google cloud storage bucket fails w/ ee.ee_exception.EEException: Unknown configuration options: {'assetId': 'test'}.

Thank you for this excellent tool! I was playing around with the new export function/types and ran into a minor issue with export to google cloud bucket. When using export() with ExportType "cloud" and filename e.g. "test" I get:
ee.ee_exception.EEException: Unknown configuration options: {'assetId': 'test'}

It appears that assetId is not an argument of Export.image.toCloudStorage(). I think it should work to set just description here:
https://github.com/dugalh/geedim/blob/7d8ecfcb2e4505192a3e19364fc0afcfda7452ae/geedim/download.py#L805-L808

Hi Andrew.  Thanks for picking this up.  It should be fixed now with v1.6.1.  Can you confirm it is working for you?

@dugalh Thanks Dugal! I confirm it is working w/ v1.6.1

Great - thank you!


QueryClient preserves only last API call if cacheTime is 0 on server [PrefetchQuery]

Describe the bug
I am working on an application, where I prefetch the queries on the server and send them to the client in dehydratedState, I am setting cacheTime and staleTime as 0 since I don't want any queries to be cached on server. If more than one query call is made on the server and sent to the client in dehydratedState, only the last query call is preserved in querClient
Your minimal, reproducible example
https://stackblitz.com/edit/stackblitz-starters-a8fjed?file=pages%2F_app.js
Steps to reproduce
In the above example, in pages/index  two API calls are made on the server fetchPosts and fetchArticles
However in the browser, if you go to the site and check in the console with window.__NEXT_DATA__, only the last API call, i.e articles is present

Expected behavior
Both fetchPosts and fetchArticles API calls should be present in dehydratedState
How often does this bug happen?
Every time
Screenshots or Videos
No response
Platform
OS: macOS
Browser: Chrome
Tanstack Query Version: 4.29.19
Tanstack Query adapter
react-query
TanStack Query version
4.29.19
TypeScript version
No response
Additional context
No response

your problem is that a cacheTime of 0 will implicitly garbage collect the posts query by the time you deyhdrate it, because it has no active observer.
There is no need to change cacheTime and staleTime on the server. staleTime is 0 anyways, and cacheTime defaults to Infinity on the server to avoid high memory consumption.
a new request will create a new QueryClient anyways, so there is really no point in setting up cacheTime like that.


Changing app name can leave app / artisan in a non-working state

Laravel Version: 5.3.x
PHP Version: 7.0.x

Description:
If you delete the database/ folder from a Laravel app created by using composer create-project laravel/laravel (for any reason, but lets assume it's because your app has no DB), later using artisan app:name MyApp can't update the database factory namespaces. This causes the command to fail, which means the composer calls to regenerate autoloaders never get run, thus leaving your app in a non-working state. Manually running composer can be used to fix this issue, but it seems reasonable that this should fail more gracefully.

What do you suggest?


HDF5 views for Dataset and ReactionDataset, supporting list_values and get_values

Description
This WIP PR adds client-side HDF5 views for Dataset and ReactionDataset, supporting the list_values and get_values methods. There is not a public interface for attaching views to {Reaction_,}Dataset since views cannot yet replace the server.
In addition, this PR:

introduces a h5py dependency
parameterizes collection test fixtures to run with and without a view
adds request counting to client for testing purposes

Todos
Notable points that this PR has either accomplished or will accomplish.

[x] Rename _get_values_from_records to _get_native_values
[x] Clean up/refactor duplicated code
[x] Ensure tests pass on Travis
[x] This PR should not be merged until after the 0.11 release.

Questions

[ ] How to handle h5py dependency

Status

[ ] Changelog updated
[ ] Ready to go

@doaa-altarawy @dgasmith I think this is ready for review.
@Lnaden could you check that I added the h5py dependency correctly?

Thanks for taking a look, @Lnaden.


Allow to replace link text in link checker

Motivation

In many cases, links are not only contained in the href attribute of the specific link:
Readable link text

But also in the link text itself:
https://some-link.tld

At the moment, it is only possible to replace the link in the href attribute, and not the link text. This is mostly due to the simpler implementation (we can just make use of lxml.html.rewrite_links() instead of some complex regex) and the fact that the URL is identical for all replaced link objects, but the link texts might differ.
Proposed Solution

At least in the case where all link texts are identical, I suggest to go the extra mile and allow the users to replace the link text as well. This would be a little bit more complicated, but can probably also be done with lxml (e.g. iterlinks()), so hopefully no regex matching.
Optionally, we could also think about allowing to "override" the link text even if it differs in the other sources.
Alternatives

Leave it up to the content creators to go through all affected pages and replace the link texts manually. However, since only the first link text is shown in the link checker, it's practically impossible to find the other occurrences of the link.
Probably we could have to at least provide a simpler method of finding all occurrences of a specific link.
User Story

As a region manager or editor, I want to replace not only the links but also the link texts (in case they are URLs) to make sure that the app users see the correct link text for the new link target.
Additional Context

As mentioned in #2307, making the link text editable via the form field led to several problems.
Thus, I suggest to choose a more black-box feature instead:

Try to determine whether the link text should be replaced as well
Automatically replace the link text if it's identical or similar to the URL (e.g. mailto:, tel: links or shortened URL domains etc)
Optionally show a warning if the "magic" method detects an inconsistency between URL and link text that it cannot fix automatically


chore+feat: script to compare versions

High Level Overview of Change

Adds a shell script to compare the Docker, NPM and Git tag versions.
If there is a mismatch, this script errors and exits.
If they all match, it prints out the current version.
Adds this shell script as an NPM script.
Context of Change

Currently all of our versioning is manual.
We need to keep 3 versions, and these should stay synced.
Currently, they are not in sync.
To start automating this process, I wrote a script that can compare these
versions.
I looked for something comparable for our use case, and could not find it, so
decided to roll my own.
Type of Change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Refactor (non-breaking change that only restructures code)
[ ] Tests (You added tests for code that already exists, or your new feature included in this PR)
[ ] Documentation Updates
[ ] Release

Before / After

Adds a shell script & npm script
Test Plan

Tested locally

Question: How will this script fare on Windows 10?


Register fails. 0: cURL error 7: Failed to connect to localhost port 80:

I found  this question with same error like in my case but you just close this issue without solving the problem so I open new one.
When I register page redirect's me to the Home page and set the token in localstorage null.
I'll get this error in console
 http://localhost:8000/api/v1/auth/login 401 (Unauthorized)
After if I change page again I redirecting to the login page.And then if I sign in I get this error bottom of the input.
0: cURL error 7: Failed to connect to localhost port 80: Connection refused (see http://curl.haxx.se/libcurl/c/libcurl-errors.html)
my APP_URL
APP_URL=http://localhost:8000

pull latest code and let me know if it works for you

@moeen-basra  Right now it's not even register or login I click the login or register button the request is going to server but without any respond from server. =(

I updated the readme and also added the docker you can try either way.

@moeen-basra  Yup that's working thank's a lot.


Unclear how to subclass a built-in type to add custom validation

Checks

[x] I added a descriptive title to this issue
[x] I have searched (google, github) for similar issues and couldn't find anything
[x] I have read and followed the docs and couldn't find an answer
[x] After submitting this, I commit to one of:

Look through open issues and helped at least one other person
Hit the "watch" button on this repo to receive notifications and I commit to help at least 2 people that ask questions in the future
Implement a Pull Request for a confirmed bug

Question
Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
             pydantic version: 1.7.3
            pydantic compiled: False
                 install path: ../.venv/lib/python3.9/site-packages/pydantic
               python version: 3.9.0 (default, Dec  6 2020, 19:27:10)  [Clang 12.0.0 (clang-1<IP_ADDRESS>)]
                     platform: macOS-11.0.1-x86_64-i386-64bit
     optional deps. installed: ['typing-extensions']

from datetime import timedelta

from pydantic import BaseModel

class MillisecondTimedelta(timedelta):
    __origin__ = timedelta

    @classmethod
    def __get_validators__(cls):
        # timedelta expects seconds
        yield lambda v: v / 1000
        yield cls

class MyModel(BaseModel):
    td: MillisecondTimedelta

    class Config:
        arbitrary_types_allowed = True
data = {
    "td": 7598040,
}

print(MyModel(**data))

This just results in td=datetime.timedelta(days=87, seconds=81240).. the custom validation isn't happening.
Also, I was surprised that I needed to specify arbitrary_types_allowed as I was hoping my custom type would just masquerade as timedelta which is normally allowed.
Am I missing something? Is there a more appropriate way to preprocess all fields of a certain type without having to explicitly add a validator for each field?

Investigating further I've found that I can get this working by omitting __origin__, and instantiating the timedelta in the validator.
class MillisecondTimedelta(timedelta):
    @classmethod
    def __get_validators__(cls):
        # timedelta expects seconds
        yield lambda v: cls(milliseconds=v)

Is it not possible to subclass a type and just add some validation before the usual validation?
 How do I get the custom validation to happen if I specify __origin__?

Investigating further I've found that I can get this working by omitting __origin__, and instantiating the timedelta in the validator.
class MillisecondTimedelta(timedelta):
    @classmethod
    def __get_validators__(cls):
        # timedelta expects seconds
        yield lambda v: cls(milliseconds=v)

Is it not possible to subclass a type and just add some validation before the usual validation?
 How do I get the custom validation to happen if I specify __origin__?

Hi @johtso
How about
from datetime import timedelta

from pydantic import BaseModel

class MillisecondTimedelta:
    @classmethod
    def __get_validators__(cls):
        yield cls.validate

    @classmethod
    def validate(cls, v: int) -> timedelta:
        # timedelta expects seconds
        return timedelta(microseconds=v)

class MyModel(BaseModel):
    td: MillisecondTimedelta

data = {
    "td": 7598040,
}

m = MyModel(**data)
assert repr(m) == 'MyModel(td=datetime.timedelta(seconds=7, microseconds=598040))'
assert isinstance(m.td, timedelta)

Why do you want to set __origin__ ?

Ah, oops! I was shooting myself in the foot. Obviously my validator wasn't returning a timedelta, it was returning an instance of my subclass.
Awesome. Thanks!
In this case this approach works totally fine. But how about if I wanted to keep the default validation and just add some pre-validation?


fix: WEBXR_SUPPORTED is always true

Details
This fixes the issue that WEBXR_SUPPORTED value is always set to true.

Coverage remained the same at 89.932% when pulling d4d35d4f27adc7301fa3dc692214073ef6c6ad3c on WoodNeck:fix-xr-supported into 02c0270ea4d185ec3b5107fc9e8017ba9c834b0c on naver:master.


update 0-stor dependency

Cause we need this commit in: https://github.com/zero-os/0-stor/commit/1bb4a2d96ceb1d85a869044079065145db6a336a

This PR has been replaced by PR #518, due to weirdness in the CI...


Making a few chagnes to generate a pull request

Adding a comment to PR #2.

Writing a comment......


Fix ffmpeg building. Add acc support.

I made version management of the libs more strict as to avoid build failures caused by breaking changes in them. Additionally, I added fdk-aac support.

@binoculars How can I force CI to build the docker image? I want to make sure that the Dockerfile works.

Not enabling non-free as it would violate the copyright


Update link.

@jinlccs,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Get quoted message object

Hi, is there a way to get the quoted message object, instead of only the quoted_stanza_id, wich references the id of the quoted message. The thing I saw which maybe could affect is this property.
But turning the property to True, and executing the make commands again I only obtained the quoted message id.
Is there a way to get the full object body?

I think you should load message using quoted_stanza_id:
qouted_msg = await self._driver.messages.get_item_by_id(msg.quoted_stanza_id)

The property you mention is just to tell frontend whether message should show the forwarded tag.

Thank you, I had to do a little trick to work, only the quotedStanzaId, could not find the element. So I did this:
if message.quotedStanzaID is not None:
    quoted_id = message.id[:message.id.index('@c.us_')] +<EMAIL_ADDRESS>+ message.quotedStanzaID
    quoted_message = await self._driver.messages.get_item_by_id(quoted_id)

When the item id was not found, the exception throwing in js had an error, I fixed that too.
Thank you very much!!

Actually, the quoted id is different, depending if the quoted is sent by me or other. So I did this:
if message.quotedStanzaID is not None:
    quoted_id = message.id[:message.id.index('@c.us_')] +<EMAIL_ADDRESS>+ message.quotedStanzaID \
        if message.quotedParticipant == message.chat.id else \
        'true_' + message.chat.id + '_' + message.quotedStanzaID

    quoted_message = await self._driver.messages.get_item_by_id(quoted_id)

So, this will work just fine.


private registries scan

It's technically possible to scan private registries if we have a way to pass
TRIVY_INSECURE
TRIVY_USERNAME
TRIVY_PASSWORD
variables to the trivy scanner.
We could set up TRIVY_INSECURE parameter via env.
And we could get TRIVY_USERNAME&TRIVY_PASSWORD from k8s secret with registry pull rights (or any other way).

@3rgfbrgf Trivy-operator knows to find references to image pull secrets (direct references and via service account).
and create the temporary secret with basic credentials for each container of the scanned workload.
create the scan job that references the temporary secret. The secret has the ownerReference property set to point to the job.

@chen-keinan ok, what about the TRIVY_INSECURE flag? It's needed for scanning private registries with trivy CLI.
Hot to set it up for the operator?

@chen-keinan ok, what about the TRIVY_INSECURE flag? It's needed for scanning private registries with trivy CLI.

It is not always needed when using private registries. If you private registry has a public signed cert, you don't need it.

@3rgfbrgf see above

@chen-keinan I don't see this parameter in the latest pulled helm chart (0.0.5)

@chen-keinan I don't see this parameter in the latest pulled helm chart (0.0.5)

I have a task to work out the documentation for it, in the mean, while take a look at trivy scanning config. you'll need to update the trivy-operator-trivy-config configmap  with this param


detailed commands in background

Prerequisites

[X] This is not a support issue or a question. For support, questions, or help, see support links.
[X] I performed a cursory search of the issue tracker on APK Tool GUI to avoid opening a duplicate issue
[X] I have read the FAQ

Describe the feature you'd like
Hi,
in the past I used apktool, zipalign and  jarsigner for Linux.
Then I tested your tool, it is easier to use, although the zip and sign happens automatically after recompiling.
Would it be possible to show in the logfile which commands and which flags will be executed during APGToolGUI will be used?
Thanks.

Well hidden in the settings.
Thanks. :-)

I'll probably make a Wiki about all its features


Added cloudflare-php binding to the other SDKs

Thanks for the approvals; any chance someone could merge for me (I don't have write access to this repo).

Done!


quick-comment-edit not longer available in discussions

Description
I'm no longer seeing the quick-comment-edit button in discussions

Console log shows me
refined-github.js:5437 Uncaught TypeError: Cannot read properties of null (reading 'before')
    at addQuickEditButton (refined-github.js:5437:155)
    at refined-github.js:5082:7
addQuickEditButton @ refined-github.js:5437

How to replicate the issue

create discussion
try to edit it

Extension version
22.9.25
Browser(s) used
Chrome Version 105.0.5195.125

@ConstantinoSchillebeeckx URL to an example discussion needed, as requested by the issue template.


Please consider fixing issues detected by repository checker

Notification from ioBroker Check and Service Bot
Dear adapter developer,
I'm the ioBroker Check and Service Bot. I'm an automated tool processing routine tasks for the ioBroker infrastructure. I have recently checked the repository for your adapter govee-local for common errors and appropiate suggestions to keep this adapter up to date.
Please see the result of the check below.
ioBroker.govee-local
   -

:thumbsup: No errors found
WARNINGS:

[x] :eyes: [W040] "keywords" within package.json should contain "ioBroker"
[x] :eyes: [W184] "common.main" is deprecated and ignored. Please remove from io-package.json. Use "main" at package.json instead.

Please review issues reported and consider fixing them as soon as appropiate.
Warnings reported by repository checker should be reviewed. While some warnings can be considered as a suggestion and be ignored due to good reasons or a dedicated decision of the developer, most warnings should be fixed as soon as appropiate.
You may start a new check at any time by adding the following comment to this issue:
@iobroker-bot recheck
Please note that I (and the server at GitHub) have always plenty of work to do. So it may last up to 30 minutes until you see a reaction. I will drop a comment here as soon as I start processing.
Feel free to contact me (@iobroker-bot) if you have any questions or feel that an issue is incorrectly flagged.
And THANKS A LOT for maintaining this adapter from me and all users.
Let's work together for the best user experience.
your
ioBroker Check and Service Bot
Note: This issue replaces issue #114
@mcm1957 for evidence

All issues reported earlier seem to be fixed now.
THANKS for your support.
This issue can be closed.
your
ioBroker Check and Service Bot


Onboard Intel HDA not recognized properly (ALC892)

I installed this plugin on unraid 6.12-rc5, driver loads and detects my nvidia hdmi audio but not the integrated intel one. Logs from alsa-info.sh:
alsa-info-1.txt

Sorry, I somehow missed that issue completely...
I can't do anything about that since this will require a complete rebuild from Unraid because some built in modules are missing and most of the drivers need to be recompiled including I have to provide a custom Kernel that I would have to provide through the plugin which I won't/can't do.
This plugins is mostly meant for the usage with USB Sound devices, if your built in hardware won't work I can't do anything about it.


Body is not sent as request.

The Body parameter is not sent in the request, and seems to serve no purpose.

Added Body, thanks

thanks! I'll use latest version now :)


does TRACER pretrained model used for training?

Hi,
I downloaded pre-trained models of TRACER as you mentioned in README, but I'm confused about the step below
Change the model name as 'best_model.pth' and put the weights to the path 'results/DUTS/TEx_0/best_model.pth'
question is, this path is model save path, not pretrained model load path, and I could not find you load "best_model.pth" at start of training anywhere in your code, did I miss something?

Hi,
Thank you for your interest in our research.
If you want to train TRACER with your own dataset or benchmark datasets (e.g., DUT-TR), I recommend training TRACER with EfficientNet based pre-trained model.
In this case, you don't need to employ the pre-trained TRACER (just build the scripts without best_model.pth).
In other cases, such as testing your own dataset, put the pre-trained weight best_model.pth to the path results/DUTS/TEx_0/best_model.pth corresponding to backbone scale.
For example, if you determine EfficientNet-b2 as a backbone, the path is results/DUTS/TE2_0/best_model.pth (here, best_model.pth is pre-trained weight of TE2).
Afterward, please run main.py using test mode (i.e., python main.py test --save_map True).
If the above answers are not enough, please leave the issue with what you plan to do.

Thanks for responding me so fast.
Your "Pre-trained models of TRACER" is actually for testing, there is no need to use it as pretrained model for finetuning with new dataset
Have I got that right？

Indeed, when we initialized the network with pre-trained TRACER on other dataset, not the benchmark dataset, employing EfficienctNet as a backbone showed better generalization performance compared to fine tuning with TRACER.
I'm not sure training with pre-trained TRACER show better performance on your dataset, but I recommend you initializing the network with EfficientNet if you want to train your own dataset.


[resources] Add support to show custom columns

It is now possible to customize the columns which should be shown for a
resource via the new "columns" field in the resource panel options.
To show a set of custom columns for a resource, a user can set a lost of
columns. For each column the "title", "resource" and "jsonPath" must be
provided. The "title" is used as header for the column. The "resource"
field is used to apply the custom style only for the specifed resource
in a list of resources. The "jsonPath" field is used to select the value
for the column from the resource manifest file. The optional "type"
field can be used to apply a special formatter for the field value
(currently only "date" is supported as special formatter).

[x] I added a CHANGELOG entry for this change.
[x] I adjusted the corresponding documentation for this change.
[ ] I adjusted the values.yaml file and the documentation for all values.

Codecov Report

Merging #288 (71d3419) into main (7a032e5) will decrease coverage by 0.17%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #288      +/-   ##
==========================================
- Coverage   44.58%   44.40%   -0.18%
==========================================
  Files          96       96
  Lines        8100     8100
==========================================
- Hits         3611     3597      -14
- Misses       4457     4469      +12
- Partials       32       34       +2

Impacted Files
Coverage Δ

plugins/applications/applications.go
93.39% <0.00%> (-6.61%)
:arrow_down:

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 7a032e5...71d3419. Read the comment docs.


support single file embedded assemblies

runtime compilation in .net 6 app published as a single file (<PublishSingleFile>true</PublishSingleFile>) works only with extraction to filesystem (<IncludeAllContentForSelfExtract>true</IncludeAllContentForSelfExtract>).
without extraction, it throws:
      System.ArgumentException: The value cannot be an empty string. (Parameter 'path')
         at System.ArgumentException.ThrowNullOrEmptyException(String argument, String paramName)
         at System.IO.Strategies.FileStreamHelpers.ValidateArguments(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options, Int64 preallocationSize)
         at System.IO.File.OpenRead(String path)
         at Microsoft.AspNetCore.Mvc.Razor.RuntimeCompilation.RazorReferenceManager.CreateMetadataReference(String path)
         at System.Linq.Enumerable.SelectListIterator`2.ToList()
         at Microsoft.AspNetCore.Mvc.Razor.RuntimeCompilation.RazorReferenceManager.GetCompilationReferences()
         at System.Threading.LazyInitializer.EnsureInitializedCore[T](T& target, Boolean& initialized, Object& syncLock, Func`1 valueFactory)
         at Microsoft.AspNetCore.Mvc.Razor.RuntimeCompilation.RazorReferenceManager.get_CompilationReferences()
         at Microsoft.AspNetCore.Mvc.Razor.RuntimeCompilation.LazyMetadataReferenceFeature.get_References()
         at Microsoft.CodeAnalysis.Razor.CompilationTagHelperFeature.GetDescriptors()
         at Microsoft.AspNetCore.Razor.Language.DefaultRazorTagHelperBinderPhase.ExecuteCore(RazorCodeDocument codeDocument)
         at Microsoft.AspNetCore.Razor.Language.RazorEnginePhaseBase.Execute(RazorCodeDocument codeDocument)
         at Microsoft.AspNetCore.Razor.Language.DefaultRazorEngine.Process(RazorCodeDocument document)
         at Microsoft.AspNetCore.Razor.Language.DefaultRazorProjectEngine.ProcessCore(RazorCodeDocument codeDocument)
         at Microsoft.AspNetCore.Razor.Language.RazorProjectEngine.Process(RazorProjectItem projectItem)
         at Microsoft.AspNetCore.Mvc.Razor.RuntimeCompilation.RuntimeViewCompiler.CompileAndEmit(String relativePath)
         at Microsoft.AspNetCore.Mvc.Razor.RuntimeCompilation.RuntimeViewCompiler.OnCacheMiss(String normalizedPath)
      --- End of stack trace from previous location ---

https://learn.microsoft.com/en-us/dotnet/core/deploying/single-file/overview?tabs=cli#api-incompatibility
if file-on-disk requirement is the only thing limiting user to access razor runtime compilation in PublishSingleFile apps, please consider adding support of in-memory assemblies without going through file i/o.
aside: in .net 7, this problem becomes real with <PublishAot>true</PublishAot>. there is no workaround as IncludeAllContentForSelfExtract because nativeaot does not use apphost.

Moving to aspnetcore. In general it feels like these are two incompatible asks: to be ahead of time compiled, but also to do runtime compilation. Agree that the single file story could be improved though.


release 0.13.0 does not run on Linux Subsystem for Windows

release 0.13.0 does not run on Ubuntu bash for Windows.
Steps to reproduce:
Follow setup guide that verifies the PATH modification in ~/.profile.
Enter this command in bash (Windows): dotnet script
You should get:
No executable found matching command "dotnet-script"
Segmentation fault (core dumped)

Enter this command in bash (Windows): dotnet script.cmd
You should get nothing (but this actually works on full-blown Ubuntu Desktop: dotnet script.sh).
Enter this command in bash (Windows): dotnet exec /mnt/e/your/path/to/dotnet-script/dotne t-script.dll.
You should get dotnet-script command-line args/help.
Enter this command in bash (Windows):  dotnet exec /mnt/e/your/path/to/dotnet-script/dotne t-script.dll helloworld.csx.
When you are running the above command from the same folder where helloworld.csx is, you should get: The command dotnet nuget locals global-packages -l failed to execute or it might actually work. Looking into nuget issues on my machine---also there might be issues with dotnet trying to do initial-run metrics setup but the segmentation fault stuff looks bigger than these possibilities.
I am sure with chocolatey and PowerShell it works fine.

@BryanWilhite A lot of water under the bridge since 0.13.0. Appreciate if you could see if this works as expected with the latest version 👍

Okay, on Windows in bash when I run this:
dotnet tool install -g dotnet-script

I am getting this:
No executable found matching command "dotnet-tool"

Please note that you need the .Net Core 2.1 SDK to use global tools

Ah, I see I need to install dotnet-sdk-2.1.300-preview2-008533. I have updated my setup guide.
Update: I installed the preview according to the guidance of this repo and am getting the same error.

you should install rc1 not preview2 - they changed the installation process
in rc1
Bryan Wilhite<EMAIL_ADDRESS>schrieb am Mo., 21. Mai 2018,
20:41:

Ah, I see I need to install dotnet-sdk-2.1.300-preview2-008533. I have
updated my setup guide
https://github.com/BryanWilhite/dotnet-core/tree/master/dotnet-script-minimal
.
Update: I installed the preview according to the guidance of this repo
https://github.com/filipw/dotnet-script#net-core-21-global-tool and am
getting the same error.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/filipw/dotnet-script/issues/133#issuecomment-390744674,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABoZISlQta7fBA2ZQJftZn1dzt6Fs9weks5t0wpDgaJpZM4PjUu4
.

Also, make sure you are really using the latest .NET Core 2.1 SDK by running dotnet --info. It should print Version:   2.1.300-rc1-008673

I followed the install instruction in “Install .NET Core SDK 2.1.300-rc1 on Linux Ubuntu 16.04.”
Here is my dotnet --info output:
.NET Core SDK (reflecting any global.json):
 Version:   2.1.300-rc1-008673
 Commit:    f5e3ddbe73

Runtime Environment:
 OS Name:     ubuntu
 OS Version:  16.04
 OS Platform: Linux
 RID:         ubuntu.16.04-x64
 Base Path:   /usr/share/dotnet/sdk/2.1.300-rc1-008673/

Host (useful for support):
  Version: 2.1.0-rc1
  Commit:  eb9bc92051

.NET Core SDKs installed:
  1.0.4 [/usr/share/dotnet/sdk]
  2.0.0 [/usr/share/dotnet/sdk]
  2.1.200 [/usr/share/dotnet/sdk]
  2.1.300-preview2-008533 [/usr/share/dotnet/sdk]
  2.1.300-rc1-008673 [/usr/share/dotnet/sdk]

.NET Core runtimes installed:
  Microsoft.AspNetCore.All 2.1.0-preview2-final [/usr/share/dotnet/shared/Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.1.0-rc1-final [/usr/share/dotnet/shared/Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.App 2.1.0-preview2-final [/usr/share/dotnet/shared/Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.1.0-rc1-final [/usr/share/dotnet/shared/Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 1.0.5 [/usr/share/dotnet/shared/Microsoft.NETCore.App]
  Microsoft.NETCore.App 1.1.2 [/usr/share/dotnet/shared/Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.0.0 [/usr/share/dotnet/shared/Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.0.7 [/usr/share/dotnet/shared/Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.1.0-preview2-26406-04 [/usr/share/dotnet/shared/Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.1.0-rc1 [/usr/share/dotnet/shared/Microsoft.NETCore.App]

For dotnet install -g dotnet-script, I am getting the same error as aforementioned. Did I miss a step?

BTW: works just fine on full Ubuntu (for dotnet tool install -g dotnet-script):
Tools directory '/home/user/.dotnet/tools' is not currently on the PATH environment variable.
If you are using bash, you can add it to your profile by running the following command:

cat << \EOF >> ~/.bash_profile
# Add .NET Core SDK tools
export PATH="$PATH:/home/rasx/.dotnet/tools"
EOF

You can add it to the current session by running the following command:

export PATH="$PATH:/home/user/.dotnet/tools"

You can invoke the tool using the following command: dotnet-script
Tool 'dotnet-script' (version '0.24.0') was successfully installed.


Is there anyway that I can find complete demo version of low code engine

I want to make Website Builder in english using Alibaba low code engine but it is completely in chinese and it is hard to translate it into english. Is there any easy ways that I can translate the whole low code engine into english?

sorry for the inconvenience, for now you can use chrome`s translation, right click on the webpage and you can see ’translate to english‘，which works for both demo and document site.


Provide descriptions of objects in schema for tools like graphiql

This is just an enhancement idea but I think it would be awesome to parse the comments above each type declaration (and field) and make them available to tools like graphiql so that we can have self documenting apis.
	# A humanoid creature from the Star Wars universe
	type Human implements Character {
		# The ID of the human
		id: ID!
		# What this human calls themselves
		name: String!
		# Height in the preferred unit, default is meters
		height(unit: LengthUnit = METER): Float!
		# Mass in kilograms, or null if unknown
		mass: Float
		# This human's friends, or an empty list if they have none
		friends: [Character]
		# The friends of the human exposed as a connection with edges
		friendsConnection(first: Int, after: ID): FriendsConnection!
		# The movies this human appears in
		appearsIn: [Episode!]!
		# A list of starships this person has piloted, or an empty list if none
		starships: [Starship]
	}

Wanted to do that for a while, did it now. ;-)


Debt #80 fix duplicated code

Hi  @dgarnitz
Addresses issue#80.
Please let me know if you have any feedback or suggested changes 😊

Hey @dgarnitz
Apologies for all the commits, transparently I'm rather new to contributing to OS projects and have had a hard time getting VS code settings in place to not affect existing code, leading to unintended merge conflicts/changes.
I have now addressed the recent code base changes and fixed unintended auto-refactor changes I was having.
If you have any feedback, it would be greatly appreciated 😊

@Aleksandir  This looks great! Thanks for completing this. Before it can merge tho, you need to test it using the Docker compose to ensure that the end to end flow still works. This can be done using the setup.sh script and the standard_upload_client.py in clients
Also please re-run all unit tests in API and worker to show they still pass.
Please post screen shots to prove everything works as anticipated.

Closing PR due to various errors and missing internal modules. have confirmed all required external modules have been installed. Unfortunately its outside of my ability to resolve these errors.

@Aleksandir can you post the errors you were getting please? I will try to take a look at what is going wrong.


Add "Generated by ..." comment to top of generated files

Thought I'd ask your thoughts on this. Might prevent others from editing those files.

Yeah, but the idea is you would only generated a model, for example, once. Then it would become a part of your app. You're not meant to keep re generating it.

A better solution would be a y/a/q dialog in the generate if the file already exists.


AuthorizerConfig - Secret or ConfigMap keeps erroring

What steps will reproduce the bug?
Nifi is running with the default authorizors.xml and there is no issue with it.
Trying to utilise ReadOnlyConfig.AuthorizorConfig, both Secret or ConfigMap is not replacing and coming up with errors no matter the content.
authorizers.xml is a working copy from conf/authorizers.xml with a slightly changed authorizer identifier.
Identifier is changed from managed-authorizer to managed-authorizer-test

Created a secret or config map with below commands:
kubectl create secret generic nifi-file-secret -n nifi --from-file=authorizers.xml --from-file=nifi.properties
kubectl create configmap nifi-config-map -n nifi --from-file=authorizers.xml
Nifikop starts a nifi-cluster node

What is the expected behavior?
The content in nifi-config-map.data."authorizers.xml" should be over-writing conf/authorizers.xml file.
I am expecting to see the conf/authorizers.xml file to have managed-authorizer-test.
What do you see instead?
I still see the default settings for the authorizers.xml.
Additional errors I am seeing. The first error is correctly showing where I deleted the config map for testing purposes. It correctly showed the error with the missing configmap.
{"level":"error","time":"2023-04-14T05:18:39.893Z","logger":"controllers.NifiCluster","caller":"nifi/secretconfig.go:468","msg":"error occurred during getting authorizer readonly configmap","clusterName":"nifi-cluster","nodeId":1,"error":"configmap not ready: ConfigMap "nifi-config-map" not found","stacktrace":"github.com/konpyutaika/nifikop/pkg/resources/nifi.(*Reconciler).getAuthorizersConfigString\n\t/workspace/pkg/resources/nifi/secretconfig.go:468\ngithub.com/konpyutaika/nifikop/pkg/resources/nifi.(*Reconciler).secretConfig\n\t/workspace/pkg/resources/nifi/secretconfig.go:53\ngithub.com/konpyutaika/nifikop/pkg/resources/nifi.(*Reconciler).Reconcile\n\t/workspace/pkg/resources/nifi/nifi.go:156\ngithub.com/konpyutaika/nifikop/controllers.(*NifiClusterReconciler).Reconcile\n\t/workspace/controllers/nificluster_controller.go:133\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Reconcile\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.13.0/pkg/internal/controller/controller.go:121\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.13.0/pkg/internal/controller/controller.go:320\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.13.0/pkg/internal/controller/controller.go:273\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.13.0/pkg/internal/controller/controller.go:234"}
With the config map created it always shows a stacktrace with no obvious error.
{"level":"error","time":"2023-04-14T05:20:52.106Z","logger":"controllers.NifiCluster","caller":"nifi/secretconfig.go:468","msg":"error occurred during getting authorizer readonly configmap","clusterName":"nifi-cluster","nodeId":1,"stacktrace":"github.com/konpyutaika/nifikop/pkg/resources/nifi.(*Reconciler).getAuthorizersConfigString\n\t/workspace/pkg/resources/nifi/secretconfig.go:468\ngithub.com/konpyutaika/nifikop/pkg/resources/nifi.(*Reconciler).secretConfig\n\t/workspace/pkg/resources/nifi/secretconfig.go:53\ngithub.com/konpyutaika/nifikop/pkg/resources/nifi.(*Reconciler).Reconcile\n\t/workspace/pkg/resources/nifi/nifi.go:156\ngithub.com/konpyutaika/nifikop/controllers.(*NifiClusterReconciler).Reconcile\n\t/workspace/controllers/nificluster_controller.go:133\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Reconcile\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.13.0/pkg/internal/controller/controller.go:121\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.13.0/pkg/internal/controller/controller.go:320\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.13.0/pkg/internal/controller/controller.go:273\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.13.0/pkg/internal/controller/controller.go:234"}
Possible solution
The stacktrace is from pkg/resources/nifi/secretconfig.go:468
However I'm not sure where to go from here.
Is it possible that err is always populated for some reason even though there's no error?
Or perhaps there's an issue with my configuration but I do not see what I'm doing wrong there.
No matter what content I have I still get the same stacktrace - whether it's a working authorizers.xml or just a string in the configmap.
NiFiKop version
v.1.1.0-release
Golang version
??
Kubernetes version
1.25.1
NiFi version
1.21.0
Additional context
No response

@juldrixx I think this necessitates a v1.1.0 release including the custom authorizers fix.

There's a lot of context here in this nifikop slack thread: https://konpytika.slack.com/archives/C035X6KP684/p1679324260885219
TLDR is there's a fix staged but it hasn't been released yet

Released 1.1.1, should be good with it.


Invalid Calls To logout API Endpoint While Using Ansible Check Point Collection (Check_Point.Mgmt)

Ansible generates dummy calls to the target Check Point platform, these calls are getting rejected every time with response 400, but it consumes resources.

check_point.mgmt collection version 6.1.0

Hi @gabhishek627,
Which calls are they? Can you share the logs ($MDS_FWDIR/log/api.elg) and your playbook?
Regards


Subtle differences between <> and {}

The problem
The documentation for the < >  postcircumfix says:

Technically, not a real operator; it's syntactic sugar that's turned into the { } postcircumfix operator at compile-time.

However, it's not really true. <> puts the index object in a string context.
> Set.new(1,2,3).Mix{3}
1
> Set.new(1,2,3).Mix<3>
0
> Set.new(1,2,<3>).Mix<3>
1

In the last case, <3> creates an allomorph, which is correctly interpreted by the string context created by the <> circumflex
(this also applies if you use Mix or Bag directly, it's just the example where I found it.)
Suggestions
Certainly change that in the description of the indexing operator, maybe document it as a trap.

 is subject to val processing.  That's really all there is to it?

Sorry, I don't quite follow you...

https://docs.raku.org/routine/val

Given a Str that may be parsable as a numeric value, it will attempt to construct the appropriate allomorph, returning one of IntStr, NumStr, RatStr or ComplexStr or a plain Str if a numeric value cannot be parsed.

Still don't. Maybe because the "is subject" lacks, well, a subject.

the postcircumfix < > is really syntax sugar for the postcircumfix { }. That doesn't mean that it's only a search-and-replace operation; what would be the point of that anyway.

Shortcut for postcircumfix { } that quotes its argument using the same rules as the quote-words operator of the same name.

This description seems fine to me as it is.


Adminer 4.7.9 left hand side menu not displaying correctly

menu should display as "select table"
but it's displaying as
select
table
select
table

Hey @tjhunkin!
Thanks for your report, it is very important for us. I'm sorry for the unanswered waiting time.
For the maintenance of this project, we are waiting for the Dracula UI stable version, where we will have more suitable components for interfaces.
But while it, feel free to send a pull request.
Thank so much!

I'm also having this issue. I worked work back on it to get it usable.
It requires commenting out a line, and then adding a code block to the end of the file.

Adding padding-bottom for the lines in the table:

#logins li, #tables li {
  list-style: none;
  padding-bottom: 20px;
}

and commenting out of the display: block on line 398:

Then it'll look like this:

I'm also having this issue. I worked work back on it to get it usable.
It requires commenting out a line, and then adding a code block to the end of the file.

Adding padding-bottom for the lines in the table:
#logins li, #tables li {
  list-style: none;
  padding-bottom: 20px;
}

and commenting out of the display: block on line 398:
Then it'll look like this:

Can you post your update please?

This is what I get....

I'm getting closer.... don't think I need your code.... just need to figure out how to clean up the left menu box and language selection.


Type Time and NaiveTime differently

Hey everyone, just wanted to start a discussion. Since Gleam has a really nice type system and as a language heavily encourages you to use types to reduce the possibility of bugs or crashes, I thought it would really help reduce time related bugs if Time and NaiveTime were different types. I know a big downside is that it would add a lot more code to this package and probably bump birl up a major version, but it will really make sure that when people do comparisons and conversions between times, they know how time offsets will be handled. For example, if I wanted to compare two time values, say 2019-01-02T14:00:00.000 and 2019-01-02T14:00:00.000-04:00, it is not obvious to me how the compare function will behave in this case. Maybe only the naive times are compared, maybe they are both converted to UTC and compared, I cannot know until I run some tests. If instead I was forced to convert them both to the same time type myself, then I could be sure how they are being compared: I could give the first an offset and compare it to the second with the compare function. Or I could make the second naive by calling to_naive (and rename the current func to to_naive_string) and compare them with a compare_naive function. This gives me, the application implementer, explicit control over how I deal with time offsets and zones, instead of assuming the birl package will handle them the way I want every time.

For another example, without running tests or knowing the implementation, can anyone guess what this will return:
  let assert Ok(t) = birl.parse("2019-01-01T14:00:00.003")
  birl.get_offset(t)
  |> io.debug

The result is "-01:00". I wouldn't have guessed that myself, but maybe we shouldn't be able to pass a NaiveTime into a function like get_offset. Making them two types would prevent this. Love the package, just want it to be the best it can be!

As has been mentioned in #29, this is a bug in parse, it should return an error for that input. But your suggestion about having a separate type for naive values makes sense, I think Eilxir's standard library has that.
I'll consider this for version 2.
Thanks

Hey! This issue and the want for easy decision-making around the current time vs a target time are what prompted me to create tempo. The standard time library is being worked on as you know, so that might change what the future of Birl looks like. I think it'll always make sense to have external time libraries, so I'm sure Birl will still serve a purpose in the long run. But I think my original wants for this issue have been fulfilled externally, so I am inclined to close it unless you object.
Also, I have to say Birl has provided a great resource to the community being the first time package by far, and the name is fantastic!

Thanks for the kind words @jrstrunk
But to be honest, given the weird state of my life, I don't think I can keep up with the gleam community anymore which is sad. For some time I was hoping at some point I would regain my enthusiasm and motivation towards my occasional gleam contributions or packages, but it hasn't happened yet and I'm not sure if it's going to happen or not at this point. We'll see.


Added new OPLS types to XML force field file

Alright, all of the atom types that were added to the OPLS force field XML file should now be included in small molecule atom typing tests.
The formic acid test appears to be failing now, which is a bit confusing to me, as it is finding two possible atom types where it should only be left with one... This does, however, bring up a good point about which rule should be chosen if an atom is a part of multiple functional groups (in the case of formic acid should aldehyde or carboxylic acid rules be used?). There are a lot of combinations of functional groups to consider, but eventually some additional overrides will need to be included for those cases.


Swap variables using destructuring assignment

This construct was stabilised in Rust 1.59.
See https://stackoverflow.com/questions/31798737/how-to-swap-two-variables.
Since Rust 1.59 was released in February 2022, before this repository existed, you may care to apply this change. If you would prefer to support earlier versions of Rust, don't hesitate to close this PR.

@drtconway I wanted to thank you for merging my PRs. You may be interested in this blog post which talks about your code: https://underlap.org/open-development-experiment-ordering-function-definition


Complete gitflow for release v.0.9.2

Completes gitflow for release v.0.9.2 by merging current state of release-0.9 into develop, as with master.

LGTM


feat: use tsv file to archive ortholog pairs

Description of the issue:

As discussed in #14, a better of storing ortholog pairs would be in tsv, instead of raw json file that is super large and hard to track

Expected feature/value/output:

Replace json with tsv file while updating ortholog pairs between human and mouse

I hereby confirm that I have:

[X] Done this analysis in the main branch of the repository
[X] Checked that a similar issue does not exist already

That sounds very useful. I'm just wondering what is the standard way to do this, if there is one.

I'm just wondering what is the standard way to do this, if there is one.

not heard of

here I close this issue that has been resolved PRs #482 in Human-GEM and 81890bd6569ae3b6a3158f30cdba787d3ca97d8b in Mouse-GEM.


Clean up brittle precommit script

A couple changes to stop the precommit script from breaking unnecessarily

pip install the flake8 linter before running lint checks. makefile assumes it is already installed
remove the very brittle unchecked_tests flag which was using multiple greps on the test files to verify if every run_cli call has a corresponding assertResult call immediately following it. I don't think using greps in this way is the right approach as it is very brittle. It was already failing on valid test cases that had been committed to the repo which means that the precommit script was being ignored. Rather than using grep hacks, I think this type of check should be handled by an actual syntax parsing linter or a developer doing a thorough code review of new test cases.
reorganize make lint so that the linting for tests is located in tests/Makefile. This is more consistent with the organization of other make targets
change make target precommit_test to precommit in tests/Makefile, making it more consistent with other make targets

Looks good to me, @mikestack-tuono you were in this codebase recently, look reasonable?


[e2e]: go schedule main config-ldflags-main-dir slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/5807743723
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.schedule.main.config-ldflags-main-dir.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.go.schedule.main.config-ldflags-main-dir.slsa3.yml
Trigger: schedule
Branch: main
Date: Wed Aug  9 10:19:44 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/5819800513
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.schedule.main.config-ldflags-main-dir.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.go.schedule.main.config-ldflags-main-dir.slsa3.yml
Trigger: schedule
Branch: main
Date: Thu Aug 10 10:19:46 UTC 2023

the usual flake slsa-verifier problem.

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/5831410719
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.schedule.main.config-ldflags-main-dir.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.go.schedule.main.config-ldflags-main-dir.slsa3.yml
Trigger: schedule
Branch: main
Date: Fri Aug 11 10:33:33 UTC 2023
Tests are passing now. Closing this issue.


Rename 'cartodb3' to 'builder' 2

Another renaming done from master to avoid several conflicts seen on https://github.com/CartoDB/cartodb/pull/13573

✨


[UVerticalNavigation] Type issue: 'exactHash' does not exist in type 'VerticalNavigationLink'

Environment

Operating System: Darwin
Node Version:     v20.12.2
Nuxt Version:     3.11.2
CLI Version:      3.11.1
Nitro Version:    2.9.6
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      colorMode, devtools, modules, runtimeConfig, ui
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Version
v2.15.2
Reproduction
Description should be enough.
Description
Apparently exactHash is missing on the Link interface.
https://github.com/nuxt/ui/blob/dev/src/runtime/types/link.d.ts
Additional context
No response
Logs
No response

Thanks for reporting this! Didn't notice that before 😊

Thank you (and all others) for the awesome job with Nuxt UI @benjamincanac!


fix(deps): fix broken dependencies on react types

Resolved in https://github.com/SocialGouv/archifiltre/pull/142


chore(contrib/ec2): rename ec2 to aws

We were incorrectly using "AWS" and "EC2" interchangeably, when in
fact EC2 is one of the services that compose AWS. Deis clusters on
AWS make use of EC2 and CloudFormation and other AWS services,
so calling it EC2 is incorrect, and using both terms is inconsistent.

:+1: haven't reviewed this but the language change makes sense.

Oops... looks liek we can't use test-ec2 for this since it won't find the old script. I could symlink it to test this PR, but not sure how to go about testing it. Deferring to @mboersma

We were incorrectly using "AWS" and "EC2" interchangeably

Old habits die hard. You're right--this is better.
I'll see if I can rig the Jenkins job to find either test script location.

Test failed with:
test.test-775fc7b80e.deis.works
ok
sleeping for 60 seconds until certs are generated...
ok
SIGQUIT: quit
PC=0x43ec61

Seems like a transient error?

Yes, looks transient. Let's re-run it once more to make sure but this should be good if it passes.

Code LGTM.

code LGTM!


RLlib post-training print full command to visualize checkpoint

Choose an option
This is a:

[ ] Question
[x] Feature request
[ ] Bug report

Describe the issue
After training using RLLib, it would be nice to have a command to visualize the results trained outputted/ pretty-printed right after. For instance, "run python flow/..../visualizer_rllib.py <path in ray_results> 1 to visualize agent training", where the path to ray_results is the correct path to the results just trained. This may help users if they can just directly copy/paste the command to visualize.
Additional context
Not sure if internal flow architecture can allow for this change. This also seems like a good first issue / open coding project.

This should not be impossible. I will look into this.

Visualize the learning curve might be a good option.

@CYBruce Do you know that you can visualize the learning curves using tensorboard --logdir ~/ray_results?


Docker build fails

~/eos/Docker# docker build . -t eosio/eos
Sending build context to Docker daemon 16.38 kB
Step 1 : FROM ubuntu:16.04 as builder
Error parsing reference: "ubuntu:16.04 as builder" is not a valid repository/tag

Hi EOSers (golly that doesnt look right!)
Docker build fails..... details below.
Docker Version: 17.12.0-ce-mac49 (21995) [ABOVE 17.05!]
Log:
Step 19/26 : COPY --from=builder /opt/eosio/bin /opt/eosio/bin
COPY failed: stat /var/lib/docker/overlay2/5f632b46e8ae8f05b134be25fe2600906c5330eb95b7260f467360781b6b8e9f/merged/opt/eosio/bin: no such file or directory


[local-volume] Check that discovered directories are mountpoints before creating local PV.

This PR checks whether discovered directories are actual mountpoints before creating local PVs.
This PR is part of a fix for https://github.com/kubernetes-incubator/external-storage/issues/370 & https://github.com/kubernetes-incubator/external-storage/issues/482. In a subsequent PR, I intend to provide instructions for enabling mount propagation in Kubernetes 1.8+ to fix these two issues.

/lgtm

/assign @wongma7


What is DoF(Depth of Field) of D435 camera?

Required Info

Camera Model
D435

I would like to know what if Depth of Field (DoF) of the color camera of D435. (NOT Field of View and Degree of Freedom)
I couldn't find any information of range of camera focus in the document.
Would you let me know it?

Hi @JANGSOONMYUN   I could not find any data for the Depth of Field of the D435 (or any RealSense camera model) either.  Depth of Field is something that can be calculated for a camera using a formula though:
https://en.wikipedia.org/wiki/Depth_of_field#Factors_affecting_depth_of_field
If you have access to an Android device, there is a DOF Calculator app on the Google Play Store.
https://play.google.com/store/apps/details?id=com.aimenrg.dof

It seems to be related to camera focus, as well as camera intrinsic calibration.
@agrunnet - perhaps you know?
@JANGSOONMYUN - in what context are you using this number?

@JANGSOONMYUN  I think you want to know what the focal length of the lens is and the F# so you can calculate all this.
The D415 and D435 share the same RGB sensor which is the OV2740 (1920x1080). The lens is a custom lens with FOV of ~70x43deg. The focal length is 1.92mm and the F# is 2.0. Assuming an allowable circle of confusion of two pixels (which are 1.4um each), you get a hyper focal distance of 66cm.  I think this is all the information you need. In general we are in focus from half the hyper focal distance (~33cm) to infinity.  The focus is fixed.

@dorodnic Thank you. I need it to make sure working distance of the camera.
@agrunnet Yes that is what I need. I couldn't find information of CoC and the hyper focal distance. Thank you.

Thanks very much for the update @JANGSOONMYUN  - I'm pleased that we could be of help to you.


Randomly Pauses while listening to music, No reason at all. Just randomly pauses.

OS:Windows 10
Issue Descriptions:While listening to music. it will just randomly pause and wont play at all. And when attempting to click the play button, its grey.
Steps to Reproduce:Play a track.
https://gyazo.com/ef72ae6825eab91982c9ed244b2277cc

This can happen for numerous reasons. Sometimes its due to audio device configuration changes, a bug with windows notifications, or an unknown bug. You didn't mention which version of GPMDP you are using. Are you on latest?

Just want to add, this happened to me today, but was only momentary while the song was buffering.

@Jacobaw Is this still an issue?

Putting this down to network issues, if it is consistently reproducible on multiple networks (and different devices) please comment below with specifics 👍


Public Kafka problems

Gareth
I am with Michael Fulton at NAM, and we can't get anything from the public kafka (errors below). I'll send you the code in email, it should just print the list of topics.
Roy
$ python3 consumer.py
%4|1657624602.344|FAIL|rdkafka#consumer-1| [thrd:kafka.lsst.ac.uk:9092/bootstrap]: kafka.lsst.ac.uk:9092/bootstrap: Connection setup timed out in state CONNECT (after 30084ms in state CONNECT)
%4|1657624632.436|FAIL|rdkafka#consumer-1| [thrd:kafka.lsst.ac.uk:9092/bootstrap]: kafka.lsst.ac.uk:9092/bootstrap: Connection setup timed out in state CONNECT (after 30092ms in state CONNECT, 1 identical error(s) suppressed)
%4|1657624662.511|FAIL|rdkafka#consumer-1| [thrd:kafka.lsst.ac.uk:9092/bootstrap]: kafka.lsst.ac.uk:9092/bootstrap: Connection setup timed out in state CONNECT (after 30075ms in state CONNECT, 1 identical error(s) suppressed)

Tried it just now, works fine! Hate intermittent problems!

Has the been any recurrence of this? If not, it may have been an issue with the old firewall in which case there's no reason to keep it open.

Closing this as irreproducible. Feel free to reopen if that's not the case.


createHmac (from node:crypto) is not implemented for nodeless targets

Environment
nodejs20
"nuxt": "^3.12.3",
wrangler:
compatibility_date = "2024-07-01"
compatibility_flags = ["nodejs_compat"]

Reproduction
Just enable nodejs-compat in wrangler and using cloudlfare_module preset, you will get the same error:
[unenv] crypto.createHmac is not implemented yet!
Describe the bug
Cannot use createHmac in cloudflare worker with cloudflare_module preset,
got error: [unenv] crypto.createHmac is not implemented yet!
but createHmac is supported in cloudflare worker: https://developers.cloudflare.com/workers/runtime-apis/nodejs/crypto/
Additional context
No response
Logs
[unenv] crypto.createHmac is not implemented yet!

would love to see this implemented, the only thing stopping me from using lucia auth in cloudflare workers

@Flowko i would suggest to contact with lucia auth maintainers. unenv is a compatibility layer, even if at some point we implement a compatible export, libraries should only depend on web APIs if they officially want to target workers.

@vicb

unenv support is not limited to cloudflare
even for cloudflare we cannot upgrade to unenv v2 because of the blockers I shared with you

This issue is not solved at least for original reported problem.

Thanks for the clarification @pi0!

unenv support is not limited to cloudflare

Yep, that's why I asked about closing this PR.

even for cloudflare we cannot upgrade to unenv v2 for Nitr/Nuxt because of the blockers I shared with you

I thought the reported issue was solved. I didn't consider v1 vs v2.

any updates on this issue? would also love to see this implemented. thank you!


Missing call to loop()

BaseFootprintBroadcaster::loop() is never called. Where do we start loopin' ?

Yes this is true, the loop is currently not called and therefore the node does not work. This was done due to the change in https://github.com/ros-sports/humanoid_base_footprint/commit/d2a482fab0705849213f332e335445a4d28f1e41 where the node was changed to be a component. The main() was removed which called the loop before.
Actually, I don't know how the correct procedure is with component nodes as I have never used them before. There seem to be multiple methods on how to use them https://docs.ros.org/en/rolling/Concepts/About-Composition.html#using-components
All of them look a bit like additional overhead, but maybe I am missing something?

Oops, that'll be a bug introduced by me.
The documentation on compositions are somewhat lacking, but line 41 from CMakeLists.txt creates an executable called base_footprint, along with the rclcpp component, which means you can just call ros2 run humanoid_base_footprint base_footprint. This means you're not forced to use components, but can run the node as an executable with no overhead (if there is any overhead in running it as a compoent, I'm not sure).
Relying on calling the loop method seems unnatural for a ROS2 node (and hence completely flew over me), and will catch out others trying to use base_footprint as a library.
The issue of loop not getting called, will be resolved by the changes in https://github.com/ros-sports/humanoid_base_footprint/issues/32

This Japanese blog post covers reasons for using components fairly well, you just have to translate it to English using google chrome's translation.

Thanks for the clarification, the documentation is indeed lacking a bit on this topic.
I agree, that this issue will be resolved by #32 so I guess we don't need to take additional actions for now.

As #32 was closed without a solution, I am going to investigate fixing this bug.


disable useless heightMapInNormalMap if both normal and relief mapping are disabled, also fix #376

Disable useless heightMapInNormalMap if both normal and relief mapping are disabled, fix #376.
Before:

After:

This bug is known to affect Nvidia Curie (Geforce 6000/7000 series) with driver Nvidia 304.
Root cause of #376 is fixed in #385. This commit is hardening the fix anyway.

Implementation is obvious.


ci: cache selene install

This speeds up the Selene job in the lint action significantly when the cache is valid. Selene's last update was 4 months ago so there's really no need to be installing it on every run. Much faster to use a cache.
Take a look at this run to see the speed up. It only took the Selene job ~9 seconds to complete with the cache.

Love it - this was on my TODO, so, glad you beat me to it 😍


re3 stuck in third person with no arm movement

I do not have any vr ability other than when i put on the headset im in game and can see all around me but the character body is still in front of me (third person) and i do not have any arm movement.
using valve index

Turn it on under FirstPerson. It's not on by default.
You can find this in the menu with the "insert" key or the in-game VR menu. You can also change it in the re2_fw_config.txt in your game directory.  https://cursey.github.io/reframework-book/troubleshooting/VR-Troubleshooting.html


ElasticsearchParseException[Failed to derive xcontent]

Hi,
Using this plugin in version 2.1.1 + ES 1.7.1 I face an error following this installation guide: http://developer.couchbase.com/documentation/server/4.0/connectors/elasticsearch-2.1/install-intro.html
curl -X PUT http://localhost:9200/_template/couchbase -d /elasticsearch/plugins/transport-couchbase/couchbase_template.json

=>

  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   145  100    79  100    66   1809   1511 --:--:-- --:--:-- --:--:--  1837
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

=>
{"error":"ElasticsearchParseException[Failed to derive xcontent]","status":400}

any idea ?

It's works fine in my machine with plugin in version 2.1.1 + ES 1.7.1 I.
you can change the log config to debug, and check the elasticsearch.log.
the log config:
<ES_HOME>/config/logging.yml
for example, change
es.logger.level: INFO
to
es.logger.level: DEBUG
the log should be:
<ES_HOME>/logs/elasticsearch.log

Thanks for your answer,
I setup log level to DEBUG
but there is not any log in ES logs... just the exact same error:
curl -X PUT http://localhost:9200/_template/couchbase -d /elasticsearch/plugins/transport-couchbase/couchbase_template.json
{"error":"ElasticsearchParseException[Failed to derive xcontent]","status":400}

with ES 1.5.2 here is the error:
{"error":"ElasticsearchParseException[Failed to derive xcontent from org.elasticsearch.common.bytes.ChannelBufferBytesReference@550c7219]","status":400}

the ES cluster have no index before you define template?

Seems like this '@' character is useful ^^

@clakech What does the '@' character do to make it work for you?

@naisanza I don't know, with the @ it works, without the @ it doesn't work... found this tip somewhere on the dark web ^^ (in a deep forum)

@ actually says that what follows is a file name, and content of that file must be sent as payload of PUT request. Without the @, it is just a plain verbatim string that goes as payload.

@clakech i try your way to resolve this, but shtill report
{"error":{"root_cause":[{"type":"parse_exception","reason":"Failed to derive xcontent"}],"type":"parse_exception","reason":"Failed to derive xcontent"},"status":400}
so ,i want to consult you that es has other config?

@zhouyu94 your json have syntax errors ?
switch to debug in:
[ES_HOME]/config/logging.yml
change:
es.logger.level: INFO
to
es.logger.level: DEBUG


⚠ Bofrost.fr has degraded performance

In 3d452a2, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1625 ms

Resolved: Bofrost.fr performance has improved in 8320328 after 7 minutes.


env_file variables not loading into slack notification properly

Hello,
There seems to be an issue with loading variables from env_file into the slack notification. I had some help from @eedwards-sk through concourse need-help room and he independently verified that it was not working as expected. Currently envsubst works properly but does not get loaded into the notification.
Full discussion can be found on Concourse - #need-help June 7th (~3pm EST till EOD)

Maybe it has something to do with fact that the image at the Docker hub was updated a year ago?
I am currently trying to figure out why I cannot use values from my env_file in the text parameter. I hijacked the resources container of my slack notification and looked at the tests. There is no test specified that covers env_file, contrary to the tests I ran on my local machine after cloning this repo.

I can indeed confirm that his is due to the outdated image on the Docker hub. I have build the resource myself and uploaded it to my private Docker registry, configured my slack-notification-resource to use my image and voilá! The environment variables are substitued as intended.

Here SchemaService and staging (before "swarm") were replaced from this template:
      text: |
        Deploy of service *`$SERVICE_NAME`* to $SWARM_NAME swarm succeeded.

Do we just need to build and push a new image from master?

This is what I did.

1.5.0 is out: https://github.com/cloudfoundry-community/slack-notification-resource/releases/tag/v1.5.0 (and tagged as :latest)

This looks like it is working as expected now. Thank you so much for the help!!


[SPARK-21281][SQL] Throw AnalysisException if array and map have no argument

What changes were proposed in this pull request?
This pr modified code to throw AnalysisException if array and map in functions have no argument. This error handling is the same with coalesce in functions.
How was this patch tested?
Added tests in DataFrameFunctionsSuite.

Test build #79103 has started for PR 18516 at commit 1e29eab.

Test build #79103 has finished for PR 18516 at commit 1e29eab.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79103/
Test FAILed.

The hive-related tests failed, so I checked the hive behaviour;
hive> CREATE TEMPORARY TABLE t1 AS SELECT map();
hive> DESCRIBE t1;
_c0   map<string,string>

hive> CREATE TEMPORARY TABLE t2 AS SELECT array();
hive> DESCRIBE t2;
_c0   array<string>

hive> CREATE TEMPORARY TABLE t3 AS SELECT coalesce();
FAILED: SemanticException [Error 10305]: CREATE-TABLE-AS-SELECT creates a VOID type, please use CAST to specify the type, near field:  _c0

hive> CREATE TEMPORARY TABLE t4 AS SELECT struct();
hive> DESCRIBE t4;
_c0                     struct<>

hive> CREATE TEMPORARY TABLE t5 AS SELECT greatest();
FAILED: SemanticException [Error 10015]: Line 1:37 Arguments length mismatch 'greatest': greatest requires at least 2 arguments, got 0

hive> CREATE TEMPORARY TABLE t6 AS SELECT least();
FAILED: SemanticException [Error 10015]: Line 1:37 Arguments length mismatch 'least': least requires at least 2 arguments, got 0

In hive, array and map with no argument use string types by default. Spark fails when using struct with no argument though, it seems hive return an empty struct. If you get time, could you check this? @gatorsmile Thanks!

Test build #79138 has started for PR 18516 at commit 1636f97.

Test build #79138 has finished for PR 18516 at commit 1636f97.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79138/
Test FAILed.

Test build #79145 has started for PR 18516 at commit 0b492fd.

Test build #79145 has finished for PR 18516 at commit 0b492fd.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79145/
Test FAILed.

Retest this please.

Test build #79161 has started for PR 18516 at commit 0b492fd.

Test build #79161 has finished for PR 18516 at commit 0b492fd.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79161/
Test FAILed.

Test build #79174 has started for PR 18516 at commit 24834fc.

Test build #79175 has started for PR 18516 at commit 28d060f.

Test build #79174 has finished for PR 18516 at commit 24834fc.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79174/
Test FAILed.

Merged build finished. Test FAILed.

Test build #79175 has finished for PR 18516 at commit 28d060f.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79175/
Test FAILed.

Test build #79180 has started for PR 18516 at commit d9c05a6.

Test build #79180 has finished for PR 18516 at commit d9c05a6.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79180/
Test FAILed.

Test build #79193 has started for PR 18516 at commit 04cbf78.

Test build #79193 has finished for PR 18516 at commit 04cbf78.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79193/
Test FAILed.

Merged build finished. Test FAILed.

Jenkins, retest this please.

Test build #79209 has started for PR 18516 at commit 04cbf78.

Test build #79209 has finished for PR 18516 at commit 04cbf78.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79209/
Test PASSed.

Test build #79253 has started for PR 18516 at commit f71dc27.

Test build #79253 has finished for PR 18516 at commit f71dc27.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79253/
Test PASSed.

Merged build finished. Test PASSed.

retest this please

Test build #79353 has started for PR 18516 at commit f71dc27.

Test build #79353 has finished for PR 18516 at commit f71dc27.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79353/
Test PASSed.

Thanks! Merging to master.