Fix/8209 Taxonomies Terms

fixes #8209
The way FullWeight used to be computed resulted in sometimes wrong values.
The FullWeight was not computed the same way in different places in the code.
The FullWeight ordering was not consistent with the ordering from 1.10.x.
The FullWeight ordering did not account for the Name property of TermParts.
This code is based off and targeted to the dev branch.
(I also took the chance to clean up and refactor some code that bothered me)

The upgrade code is from 1.10.x to dev?
@BenedekFarkas can you get it tested?

It should work to upgrade from 1.10 to dev as well as to upgrade dev to what is in this pr

Any news on this?
We've been running this on 40+ tenants for a few days, and so far we haven't had any issue with it. It's been boring to run the upgrade step on all of those, but apparently nothing broke yet.

Sorry @MatteoPiovanelli-Laser, reviewing/testing now!

I will look into the proposed changes.
Let me know if there is any way I can also improve the comments around the code I added to make what's being done more clear.
Regarding examples:
The best way to test this, I think is:

Start from a 1.10.x tenant. Create a taxonomy there, whose terms are organized in a few levels, and with different weights. Some terms on a same level in the hierarchy should have the same weight to be ordered alphabetically.
"move" the tenancy to the current dev branch here. Run the upgrade process there to upgrade the FullWeight. In most cases the terms will not be ordered correctly. IN some cases something may actually break (I don't remember the exact conditions to break the current code off the top of my head see the issue #8209 for a bit more context).
merge the code form this PR in, run the upgrade again. The terms should be correctly ordered again.

As a side, the code as is in this PR gives a significant performance boost in the cases where a taxonomy is very large. We had a taxonomy with several hundred terms, and every time we tried to look at it opening the back office page would take several minutes. The FullWeight as computed in this PR allows proper ordering and pagination, so we don't have to do it in memory.

I updated the PR. I think that answers the points raised in the review so far.
I also cleaned up the code in the upgrade module and separated this upgrade for the terms from the old one for Orchard 1.7.

Waiting for @BenedekFarkas to approve the changes, thanks.

Yes, sorry, I'll take care of it hopefully this week.

@BenedekFarkas

Sorry it took this long - tested with Matteo's instructions and looks good to me!


Bring module up to speed with CP 2.22

Since CP v2.22.0, okta application is named internally as okta.<IDP_APP_NAME>, where IDP_APP_NAME is an alias for the okta integration. This module should then make sure the URLs are created correctly using the proper IDP_APP_NAME as defined in our current documentation::
This name is derived from the App name you specified in Step 5 above.
To generate the IDP_APP_NAME name, Cyral removes all spaces and symbols
from the Okta App name, converts all letters to lower case, and prefixes
the name with okta. (note the trailing dot). For example, an Okta app
called "Main Okta-NA" will have the IDP_APP_NAME, "okta.mainoktana".

Fixed by #7


The client name string pattern does not support '.' (dots)

Within client names dots should be allowed for backward compatibility reasons.
Currently client names containg dots are failing because the client name pattern: [a-z0-9-_:]+ does not match:
[http-nio-8080-exec-4] WARN  i.r.s.w.a.GlobalDefaultExceptionHandler - ErrorResponse (status: 500, IllegalArgumentException: Client name must match pattern: [a-z0-9-_:]+)

The pattern in should be adhusted accordingly: [a-z0-9-_:.]+
io/redlink/smarti/services/ClientService.java#27

changed the pattern to [a-z0-9-_\.:]+


Allow including view_id in output to disambiguate multiple GA properties

See some more discussion here: https://github.com/meltano/meltano/issues/7165#issuecomment-1397105636
I am trying to set things up to use tap-google-analytics to load 49 GA properties into Redshift. As things stand today there's no information in the records from this tap that associates it with a specific GA property. This PR is an attempt to provide the ability to disambiguate them.
I'm not sure this is the best way, so feedback is very welcome!

@pnadolny13 I'm curious wdyt of this?

@menzenski @edgarrmondragon I cant think of any reason why we shouldnt support this, especially since its backwards compatible.
I think this PR is missing logic to add the new field to the schema though, see https://github.com/MeltanoLabs/tap-google-analytics/blob/3348b2d1e4c36cb4a69e4aeefe042c828bfc8f80/tap_google_analytics/client.py#L393. If I had to guess this would work but the view attribute would get skipped because its not selected in the schema. Does that make sense?

@pnadolny13 updated!


Expose torch.compile arguments as compile options

🚀 Feature
Motivation
The torch.compile call is internal and has fullgraph=True hardcoded. It would be useful to allow customizing it, especially after #140 lands.
https://github.com/Lightning-AI/lightning-thunder/blob/3a2b27e9b38e639133913c42c03acdf0e136ec4f/thunder/executors/torch_compile.py#L84
Pitch
from thunder.executors.torch_compile import torch_compile_ex

... = thunder.jit(..., executors=[torch_compile_ex], torch_compile_fullgraph=True, torch_compile_backend="reduce-overhead")

Use get_compile_option to allow customizing fullgraph= and backend=
https://github.com/Lightning-AI/lightning-thunder/blob/3a2b27e9b38e639133913c42c03acdf0e136ec4f/thunder/core/compile_data.py#L57
cc @apaz-cli

Hey @carmocca! I am trying to work on this issue and want to understand how things exactly work.
Once I updated the code, is there a way to verify that this works as expected? I am not sure how I can test this out as the following code works with and without modification:

Hi @carmocca thanks for such suggestions. However, I am still unable to test this out would appreciate if you can provide me with a detailed explanation. Let me ask some specific questions. Please bare with me as I may ask trivial questions. I am trying to understand how compile works in more detail.

When will get_compile_option be invoked when we jit? I am going through each step and cannot figure out when get_compile_option comes into play
When do we use make_compiled in executors/torch_compile.py? I am trying to mock what we have in line 84 as you have mentioned, however, it is unclear how this is linked to jit.

Would appreciate your help :)

get_compile_option doesn't get invoked automatically. You need to add it in your PR. If I'm not mistaken, right before torch.compile() gets called
make_compiled will get called on the first call of the model/function. After it has been thunder.jitted.
Don't hesitate to open a draft PR! It's probably easier to comment and help directly there


accounts problems on discovery and wrong password problem.

From few days ago I am getting error over all my servers like:
 { error: 'Wrong password', error_code: 12, success: false }
When I add new user and grant permissions this part of driver:
if (options.user === undefined) {
      this.user = r._user;
    } else {
      this.user = options.user;
    }

is completely ignore and self.user is always admin (but i put options.user = "myusername").
If discovery is set to true and user is different from "admin" i am getting of course error regarding server_status.
So as far as I can see it some huge problems and no solution!

What's the stacktrace? Code snippet?
Discovery is using the server_status table. Does your user have access to that table?

@neumino only admin user has access to server_status. But I give up on your driver since it causing port flooding.


Support expiration for tooz

What this PR does / why we need it:
When we using tooz for distributed lock, we need to set expiration to avoid dead lock, but now tooz has no configuration about it.
This pr add expiration configuration for tooz
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

Test case
|-------------------------------------------------------------------------------------|
| Test case                                    | Expected behavior                    |
|-------------------------------------------------------------------------------------|
| Expiration is not configured in dolphin.conf | Default expiration should be 100s    |
| Expiration is configured to 50s              | Expiration should be 50s             |
| Expiration is configured to 150s             | Expiration should be 150s            |
|-------------------------------------------------------------------------------------|

Codecov Report

Merging #80 into master will decrease coverage by 0.61%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #80      +/-   ##
==========================================
- Coverage   30.53%   29.92%   -0.62%
==========================================
  Files          57       57
  Lines        3684     3633      -51
  Branches      412      412
==========================================
- Hits         1125     1087      -38
+ Misses       2542     2529      -13
  Partials       17       17

Impacted Files
Coverage Δ

dolphin/coordination.py
49.25% <ø> (-0.75%)
:arrow_down:

dolphin/drivers/driver.py
56.52% <0.00%> (-11.23%)
:arrow_down:

dolphin/wsgi/common.py
53.57% <0.00%> (-4.50%)
:arrow_down:

dolphin/api/validation/validators.py
21.60% <0.00%> (-2.44%)
:arrow_down:

dolphin/api/validation/__init__.py
72.72% <0.00%> (-2.28%)
:arrow_down:

dolphin/api/v1/storages.py
40.35% <0.00%> (-2.03%)
:arrow_down:

dolphin/api/common/wsgi.py
22.60% <0.00%> (-1.09%)
:arrow_down:

dolphin/api/common/__init__.py
28.35% <0.00%> (-1.06%)
:arrow_down:

dolphin/db/sqlalchemy/api.py
22.22% <0.00%> (-0.95%)
:arrow_down:

dolphin/context.py
57.14% <0.00%> (-0.86%)
:arrow_down:

... and 9 more

Codecov Report

Merging #80 into master will decrease coverage by 0.60%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #80      +/-   ##
==========================================
- Coverage   30.53%   29.93%   -0.61%
==========================================
  Files          57       57
  Lines        3684     3615      -69
  Branches      412      411       -1
==========================================
- Hits         1125     1082      -43
+ Misses       2542     2516      -26
  Partials       17       17

Impacted Files
Coverage Δ

dolphin/coordination.py
49.25% <ø> (-0.75%)
:arrow_down:

dolphin/utils.py
25.79% <ø> (-0.37%)
:arrow_down:

dolphin/drivers/driver.py
56.52% <0.00%> (-11.23%)
:arrow_down:

dolphin/wsgi/common.py
53.57% <0.00%> (-4.50%)
:arrow_down:

dolphin/api/validation/validators.py
21.60% <0.00%> (-2.44%)
:arrow_down:

dolphin/api/validation/__init__.py
72.72% <0.00%> (-2.28%)
:arrow_down:

dolphin/api/v1/storages.py
40.35% <0.00%> (-2.03%)
:arrow_down:

dolphin/api/common/wsgi.py
22.60% <0.00%> (-1.09%)
:arrow_down:

dolphin/api/common/__init__.py
28.35% <0.00%> (-1.06%)
:arrow_down:

dolphin/db/sqlalchemy/api.py
22.22% <0.00%> (-0.95%)
:arrow_down:

... and 10 more


OSIF: change the heap management

Use sys_heap instead of shared_multie_heap for managing the RTK heap, as the logic of shared_multie_heap is incompatible with RTK heap management.

Are there situations that multiple threads access a heap simultaneously? The sys_heap doesn't implement a locking mechanism.😃

https://github.com/rtkconnectivity/hal_realtek/pull/47 Update a version using k_heap. Close this PR.


VSCode Extension v0.12.1 and above - ARM template build adds API version for AutomationAccount

Bicep version
v0.12.1 and above - Issue Noted
v0.11.1 and below - working as expected
Describe the bug
Bug in VSCode Bicep v0.12.1 and above.

Related to the nested deployments for Logic Apps in which a webhook is created with an automation account
Symptoms are that the Logic App will not have the URI populated for the HTTP action as it is returned as null
Deployment via BICEP file works fine in later versions but the ARM template created has the API version when it should not.
"Microsoft.Logic/workflows" properties for the URI reference the webhook and include the API version which is considered out of scope and thus a NULL value

These snippets are from my AVD Alerts GitHub Repository:
https://github.com/JCoreMS/AVDAlerts
\deploySubscription\solution.json (Current version is v0.11.1 and working)

around line 1828

v0.12.1 and above (Not Working)
"uri": "[reference(resourceId('Microsoft.Automation/automationAccounts/webhooks', parameters('AutomationAccountName'), format('{0}_{1}', parameters('RunbookNameGetStorage'), dateTimeAdd(parameters('Timestamp'), 'PT0H', 'yyyyMMddhhmmss'))), '2018-06-30').uri]",
v0.11.1 (Working)
"uri": "[reference(resourceId('Microsoft.Automation/automationAccounts/webhooks', parameters('AutomationAccountName'), format('{0}_{1}', parameters('RunbookNameGetStorage'), dateTimeAdd(parameters('Timestamp'), 'PT0H', 'yyyyMMddhhmmss')))).uri]",
To Reproduce
Steps to reproduce the behavior:
Compile BICEP from AVD Alerts or with nested deployments for an Automation Account with VS Code's BICEP extension version 0.12.1 or above incorporating a webhook. When deploying an automation account and runbook with HTTP and using a webhook, the webhook URI isn't captured and thus empty for the URI for the HTTP action in the workflow.
Additional context
Add any other context about the problem here.

Tested v0.13.1 VS Code extension as well and still has the same problem.

@JCoreMS This is a deliberate change we decided to make, and we are not planning to switch back to the old behavior. The root cause of this issue is that the PUT response and GET response for automationAccounts resource returned from the Automation RP is inconsistent. If a reference function contains the API version parameter, ARM will always send a GET request to evaluate the reference function, whereas without the API version, ARM might use the PUT response instead to do evaluation depending on whether the resource type supports asynchronous PUT.
To fix the issue, the Automation RP will need to update their service to return the uri property. For now, you may use the reference function with API version in Bicep as a workaround, for example:
-automationAccount.properties.uri
+reference(automationAccount.name, '2018-06-30').uri

@shenglol I tried the above approach but got a template validation failure. Any idea when the Automation RP will update their service?  It seems like this should have been a coordinated change to ensure we don't impact functionality of existing deployments that have been developed.

Could you let me know what the template validation failure is?
I don't know if there's an ETA for the Automation RP to fix the behavior. @alex-frankel do we have any plan to sync with the Automation RP to discuss the issue?

It would be interesting to test whether the symbolicNameCodegen experimental feature fixes this: https://github.com/Azure/bicep/blob/main/docs/experimental-features.md#symbolicnamecodegen

It would be interesting to test whether the symbolicNameCodegen experimental feature fixes this: https://github.com/Azure/bicep/blob/main/docs/experimental-features.md#symbolicnamecodegen

Hello Everyone, just to add here that I have tested the experimental feature and I was able to fetch the URI on my test using bicep api version 0.25.53

It would be interesting to test whether the symbolicNameCodegen experimental feature fixes this: https://github.com/Azure/bicep/blob/main/docs/experimental-features.md#symbolicnamecodegen

This worked for me

Related issue: #13396

It would be interesting to test whether the symbolicNameCodegen experimental feature fixes this: https://github.com/Azure/bicep/blob/main/docs/experimental-features.md#symbolicnamecodegen

This worked for me

Hello @anthony-c-martin, Could you share an example using symbolicNameCodegen please ?

You don't need to change anything in your .bicep files, you just need to turn the feature on - see https://learn.microsoft.com/en-us/azure/azure-resource-manager/bicep/bicep-config#enable-experimental-features


1.19.4 support

Is this planned? I'm still using this in 1.19.3 because useful mods like this have not updated to 1.19.4
While 1.19.4 seems to be improving alot in game performance so I would like to see this mod supporting 1.19.4
If you can't then I can understand, thanks.

The new release has been published at both Github Release and Modrinth.
Thanks for requesting. I've almost forget this project, as I'm no longer playing vanilla at all. lol
https://modrinth.com/mod/minequery/version/1.7.4

I still like this mod's purpose so yeah, glad you remembered :D


Detect if screen is locked with xautolock before trying to lock it again

I'm not sure if the problem is exactly as I wrote on the title.
Steps to reproduce:

Install the<EMAIL_ADDRESS>and enable it.
Run xautolock with betterlockscreen.
Either run xautolock -locknow or wait the time you have specified in step 2.
Suspend and wake the machine.

After step 4, the machine is locked, no problems on this, but two messages (locking..., then failed to lock) will appear on top of the circle that detects keyboard presses, which just looks ugly.

Ah, big blunder on my part. Just found out that this can be easily solved by changing

ExecStart=/usr/bin/betterlockscreen --lock

to

ExecStart=/usr/bin/xautolock -locknow

on the<EMAIL_ADDRESS>file. Though perhaps this could be mentioned somewhere.

not sure what you are trying to achieve here, just installing the service file will automatically lock your screen every time you suspend, no need to use xautolock as well

The purpose of xautolock is to suspend after some time in case of inactivity.

ok, but what you are doing there is executing xautolock -locknow every time you suspend, which is in turn just calling betterlockscreen --lock, so that change shouldn't do anything.
Try using the original service file, and then just suspend without doing anything else, it should lock correctly without error messages.
xautolock just executes a specified program, it doesn't suspend the system

Sorry if I wasn't clear, before. The original service file works just fine, if you're not using xautolock. But say, for example, that you set xautolock to lock the screen after 10 minutes of inactivity. These minutes pass and the screen is locked. Now, with the screen locked, you suspend the system (i.e. by closing the lid of your laptop). This makes betterlockscreen attempt to lock the screen again, even though it is already locked, and the messages I mentioned appear.

oh ok yeah i understand


add bleu script

This PR introduces a BLEU measuring script, which exploits sacrebleu (https://github.com/mjpost/sacrebleu) to measure BLEU scores for Speech Translation tasks. It is therefore necessary to install sacrebleu for the script to work:
pip install sacrebleu

main - Average batchsize: 7.9523809523809526 -> Wouldn't this information better be in the log.log only ?

LGTM Thanks @mhn226 !


Big notification when process is finished

For Labs, adding an optional big notification would allow better visual control of the status of the process.

You could for example have an optional preference in the SplashBuddy.plist called labEnvironment that when set to true would hide the sidebar and direct the webview to a complete.html page in the presentation.bundle once all of the application installations were completed.

that should have been closed ages ago. It was implemented in 1.2

With labMode set to true, SplashBuddy will automatically display complete.html from presentation.bundle when all software is done installing (successfully or not) -- thank you @howardgmac


Python server 'Unavailable' in many cloud_to_cloud:*:python_server interop tests

Might be an infrastructure flake.
a recent batch of interop tests with python server failed today (4/24 at 3pm), with about all clients appearing to get connect failed and unexpected Unavailable status codes.
log in https://grpc-testing.appspot.com/job/gRPC_interop_master/12528/#showFailuresLink

Whatever it was it must have been something in the core, something infrastructural, or both - gRPC Python itself never sends UNAVAILABLE.
Looks like the status details are "Connect Failed".


Add rancher/rke2-upgrade images to Rancher release artifact rancher-images-sources.txt

What kind of request is this (question/bug/enhancement/feature request):
enhancement
In the same way that rancher/k3s-upgrade images are included in Rancher release artifact rancher-images-sources.txt, the rancher/rke2-upgrade images should be included as well.

Dupe of https://github.com/rancher/rancher/issues/32051.


[TD-968]: Multiple version support.

Add multiple version support to  releng templates based on a policy key named RelengVersion

Demo today?

@alephnull Yes, will do.


resolve complex types and workaround promise reject types

2 things here, let me know and I can split them.

Complex types like a union that had a generic, or a generic that had a union could not be resolved, ex: Promise<Array.<*>|Object|number|string>

We like to document our promise resolve and rejection types separated with a comma, ex: Promise<ResolveType, RejectType>.  There doesn't seem to be consensus on exactly how do to this based on this still open feature request: https://github.com/jsdoc3/jsdoc/issues/509.  Typescript does not define the RejectType of a promise it is always 'any', see: https://github.com/Microsoft/TypeScript/issues/7588.  Due to this I've added a workaround that removes the second type when parsing a Promise.

Thanks btw,

Gord

Updates based on your review and squashed them, hope that's okay.  Thanks for the review.
I might try to clean up the code a bit more in the future, but I'll do a new PR for that.  I think I can remove the 2nd recursive pass and just do it in one.


Add support for an ODBC backend

This pull request adds support for an ODBC backend.  At the moment the support is protected by the 'odbc' feature, and is not enabled by default.  A subsequent pull request will enable this feature in an alternate shared library (so that we don't unnecessarily introduce a dependency on the UnixODBC shared library).
This pull request also contains a fix for an issue which was recently introduced in the JavaScript API.  The 'descending' parameter is no longer required in the scan and fetchAll JavaScript API's.
Although I am an experienced C/C++ developer, this is my first attempt at developing in Rust - so I hope that I have not done anything too silly.
As a part of the testing of this change the store.test.ts script was successfully executed against a DB2 server.

Sorry about the delay on this but we've been migrating things from Hyperledger to OpenWallet Foundation, and as part of that migration the JS wrapper is being split into a separate repo (https://github.com/openwallet-foundation/askar-wrapper-js-rn).
There are conflicts at the moment (probably mostly in Cargo.lock due to dependency updates) which I'm not able to resolve. I think the JS updates can be applied here, although the wrapper will be removed from here the changes can then be cherry-picked to the other repo.
One minor issue I see, you're using the lazy_static crate while once_cell is already in the dependency tree and can accomplish the same thing.

@andrewwhitehead Thanks for the review.  It looks like the performance is pretty terrible with the ODBC driver approach and so in the end we went a different way (sorry that I didn't let you know about this prior to you spending time on the review).  I'm not sure what you want to do with this pull request?  It won't be built by default and so I am happy for it to be merged, in case someone wants this capability in the future, or I am happy for the pull request to be closed.

Ah okay, given that nobody is currently using it (or able to maintain it) I'll have to close the PR for now. It's there if somebody wants to revive the support. Thanks!


Do we support use storage size equal or greater than 1024Gi?

What happened?
Hi experts,
We are continously seeing this kind of error with storage size set to 1024Gi when deploying CassandraDatacenter.
"admission webhook "vcassandradatacenter.kb.io" denied the request: CassandraDatacenter write rejected, attempted to change storageConfig"

storageConfig:
cassandraDataVolumeClaimSpec:
reclaimPolicy: Retain
  storageClassName: managed-csi-premium
  accessModes:
    - ReadWriteOnce
  resources:
    requests:
      storage: 1024Gi

But, once we reduced the size to a smaller one, say 500Gi, then services come up as expected with all required pvc created successfully.
Would you kindly suggest if there is any limation to the storage size?
Install command: helm install cass-operator k8ssandra/cass-operator -n cass-operator--version=0.40.0
Cluster is running on AKS+managed-csi-premium
What did you expect to happen?
Support storage size equal or greater than 1024Gi.
How can we reproduce it (as minimally and precisely as possible)?
Cluster running on AKS and deploy CassandraDatacenter with storage size 1024Gi.
sample configuration:
apiVersion: cassandra.datastax.com/v1beta1
kind: CassandraDatacenter
metadata:
name: cassandra
namespace: cass-operator
spec:
clusterName: xxxx
serverType: cassandra
serverVersion: "4.0.7"
managementApiAuth:
insecure: {}
size: 3
allowMultipleNodesPerWorker: true
resources:
requests:
cpu: 500m
memory: 24Gi
limits:
cpu: 2
memory: 24Gi
storageConfig:
cassandraDataVolumeClaimSpec:
reclaimPolicy: Retain
  storageClassName: managed-csi-premium
  accessModes:
    - ReadWriteOnce
  resources:
    requests:
      storage: 500Gi

cass-operator version
0.40.0
Kubernetes version
v1.25
Method of installation
helm
Anything else we need to know?
No response

The size does not seem to be the issue in other ways than Kubernetes trying to update the Spec. So, from what I can see in the logs is that after you've created 1024Gi sized resourceQuantity, the Kubernetes will do an update to 1Ti. Thus, we detect that as a modification (it's an update to the PVC size which StS does not allow) and then reject it.
If you wish to overcome this issue, simply write the size as 1Ti and it works fine.

Here's the relevant part from my debug logs:
2023-11-13T12:41:08.259Z        INFO    api     Validating webhook called for update
2023-11-13T12:41:08.259Z        INFO    api     differences in storageConfigs: &PersistentVolumeClaimSpec{AccessModes:[ReadWriteOnce],Resources:ResourceRequirements{Limits:ResourceList{},Requests:ResourceList{storage: {{1099511627776 0} {<nil>}  BinarySI},},Claims:[]ResourceClaim{},},VolumeName:,Selector:nil,StorageClassName:*standard,VolumeMode:nil,DataSource:nil,DataSourceRef:nil,} vs &PersistentVolumeClaimSpec{AccessModes:[ReadWriteOnce],Resources:ResourceRequirements{Limits:ResourceList{},Requests:ResourceList{storage: {{1099511627776 0} {<nil>} 1Ti BinarySI},},Claims:[]ResourceClaim{},},VolumeName:,Selector:nil,StorageClassName:*standard,VolumeMode:nil,DataSource:nil,DataSourceRef:nil,}

The 1024Gi is originally output to the server as<PHONE_NUMBER>776 with unit BinarySI and then something (Kubernetes / some controller etc) calls an update to it to replace it with 1Ti instead.
This is probably something we could detect, but for now, setting it directly to Ti works as a workaround.

Thank you Michael~ that's really helps. Will find a maintenance Windows to do this.
Also, that means we can't not update the storage size after deployment?

Hey, that's sadly a limitation of StatefulSets (and some StorageClasses), see https://github.com/k8ssandra/cass-operator/issues/263 for more details.

Michael, got it. Thanks for the explanation. Just one more question. If we want to specify 1.5T, what kind of value is suggested. 1.5Ti or 1536Gi?

Sadly there I think the Gi is the correct method..
➜  cass-operator git:(master) ✗ kubectl get pvc
NAME                                STATUS   VOLUME                                     CAPACITY   ACCESS MODES   STORAGECLASS   AGE
server-data-cluster2-dc2-r1-sts-0   Bound    pvc-9d5f342a-1898-42cb-8b0f-748aa9d554a8   1536Gi     RWO            standard       6s
➜  cass-operator git:(master) ✗


TimeoutError: Knex: Timeout acquiring a connection. error using Pool.afterCreate property

Environment
Knex version:0.15.2
Database + version:Sqlite3
OS:Linux Mint Sarah 18
Select applicable tempalate from below.
If issue is about oracledb support tag @ atiertant. For MSSql tag @ smorey2 .
Rest of dialects doesn't need tags.
Bug
By Adding Pool Property , i am getting Connection Timeout error. Even there is just one client, even if i specify max and min pool i get error
TimeoutError: Knex: Timeout acquiring a connection. The pool is probably full. Are you missing a .transacting(trx) call?

Explain what kind of behaviour you are getting and how you think it should do
Timeout error shouldnt appear by adding pool.afterCreate property

Error message
TimeoutError: Knex: Timeout acquiring a connection.

Reduced test code, for example in https://npm.runkit.com/knex or if it needs real
database connection to mysql or postgresql, then single file example which initializes
needed data and demonstrates the problem.

var knex = require("knex")({
  client: "sqlite3",
  connection: {
    filename: "./db/chinook.sqlite3"
  },
  pool: {
    afterCreate: (conn, done) => {

    }
  })

Looks like you are not calling done() callback to tell that afterCreate  hook is ready.
Try:
afterCreate: (conn, done) => {
  done();
}

Thank you , I apologize for bothering you .
It seems i missed this part in docs


plugins/crates: init

Add support for crates.nvim.
I am not convinced on whether I should explicitly declare some options or none...
Fixes #1149

@khaneliman if you happen to want to pick this up, go on :)

I could look at making changes to this pr as there is two version of crates both cmp and non cmp.

I could look at making changes to this pr as there is two version of crates both cmp and non cmp, to help out if needed.

Feel free if you're interested. I can jump in if needed,

Is there any way of good testing?

Is there any way of good testing?

You can use nix flake lock --override-input in order to temporarily update a nixvim config's flake.lock to use a different input url.
For example:
# From a nixvim _config_ repo

# Use a local repo
nix flake lock --override-input nixvim ../nixvim-fork

# Use a PR branch
nix flake lock --override-input nixvim github:GaetanLepage/nixvim/crates

# Undo override
git restore flake.lock

You can also run nixvim's tests using the tests command while in a nixvim devshell:
# From a nixvim repo

# Enter devshell
nix develop

# Run tests for this plugin (interactive filter)
tests -i
> crates

I will resolve the conflict and everything on this weekend.

I'm going to close this for now. We have plugins.crates since #2717, and this PR is so old the conflicts will be a nightmare.
If there's anything of value still in this PR, then new PRs can be created to improve or expand the plugins.crates module.


(Beta) Autofill needs to (re)click LED/Output for select pins dropdown

Describe the bug
If you jump over to display everything autofills but you have to select LED/Output from the list for it to work correctly

Steps To Reproduce
Steps to reproduce the behavior:

I made a configuration that uses LEDs
they autofilled output device, module, and "use type of",
The devices didn't show what device it was affecting until I selected LED/Output from the list

Selecting already selected LED from list shows this ^^
Expected behavior
Select pins shows automatically
Actual behavior
Select pins is hidden until you select LED/Output
Details

Beta  <IP_ADDRESS>

Additional context
Sorry for doing this in GitHub - I think you prefer discord for beta bugs?

this is fixed in #971


Update Shapeless version to release candidate

Current coverage is 79.04%

Merging #202 into master will decrease coverage by -0.48% as of 07a4df8

Powered by Codecov. Updated on successful CI builds.


Digit grouping for large numbers - PowerToys Run's Calculator and Unit Converter plugins

Description of the new feature / enhancement
Currently, when you use the Calculator or Unit Converter plugins for PowerToys Run, if the result is a large number, it will display all the digits of that number in one string without any spaces.
For example, try typing "2 megatonne in kg" in PowerToys Run and you'll get<PHONE_NUMBER> kg. This is a number that is very annoying to parse for a human since there are no spaces or commas every three digit, like there are when using Window's calculator app.
It would be great if the Calculator/Unit Converter plugins had an option to show results with the digits grouped in groups of 3. Ideally, this option would also let us choose what our separating character is (a space, a comma, or maybe whatever we want).
Another important factor is that some people might want the result displayed with a separator character but want the result added to the clipboard without said character, as some other programs that can use that result might not accept a string with the separator character (for example, Google Sheets sometimes won't accept numbers that are comma-separated)
Scenario when this would be used?
I always use PowerToys Run to make calculations/unit conversions instead of using the calculator as it's just MUCH faster to alt+space and type the equation than going to fetch the calculator app.
But parsing the results when there are lots of digits can waste that productivity improvement and ends up making using the Calculator/Unit Converter plugins a "trap" since the time it took me to count the number of 0s ended up making the use of PowerToys Run not worth it
Supporting information
See how Window's own Calculator app shows results in a much more readable way

@jaimecbernardo
I tried to implement this yesterday. But it stopped implementation because the official way of setting the parameter in decimal.ToString() will change the decimal digits behavior.
So this has to be discussed.

To what details did you get? I think some code in there ends up trying to use the defined number culture setting, right?

+1 for this idea, working with large numbers on PowerToys Calculator is really hard and separators on every 3 digits would improve the UX a lot

Ideally, this option would also let us choose what our separating character is (a space, a comma, or maybe whatever we want).

...

Note
Numbers consisting of long sequences of digits can be made more readable by separating them into groups, preferably groups of three, separated by a small space. For this reason, ISO 31-0 specifies that such groups of digits should never be separated by a comma or point, as these are reserved for use as the decimal sign.

https://en.wikipedia.org/wiki/ISO_31-0#Numbers or
https://www.bipm.org/en/committees/cg/cgpm/22-2003/resolution-10
@jaimecbernardo @crutkas Maybe we should follow this.
Note that, currently, a space is interpreted as an implicit multiplication.

The thin space is the answer
https://en.m.wikipedia.org/wiki/Thin_space

Could this issue be prioritized? Seems like a low hanging improvement?

@crutkas @jaimecbernardo
Can you guys please prioritize this? It’s been 2 and a half years since it’s stuck.


Add NetStd2.1 for HTTP polyfills

The HttpContent and HttpClient polyfills add with #16 should work on both .NET Standard 2.0 and 2.1.
The APIs they're substituting were added in .NET 5.0, and are not part of .NET Standard.

awesome. thanks

deployed


Implementation of Delayed Access

DIG 2024 approved the approach which is describe in Annex 3 of the DIG report https://doi.org/10.17895/ices.pub.26969044.v1
Actions:

Communicate out to working group (JNCC)
adding an additional field to the Cetaceans data standard https://www.ices.dk/data/Documents/Cetaceans/Cetaceans_data_template.zip and to be used in conjunction with the ‘Restricted’ value of the [DataAccess] field
Draft criteria for acceptable conditions of delay (JNCC/ICES) based on Annex 3: time delay considerations and DIG discussion
Review of: data standard document, data submission guide and user guide to ensure description follows new logic for delayed access (JNCC with support from ICES)

@NikkiTaylorJNCC to send a message to the working group.
@MariaMakri and @pcrjoana to make the changes in the template and format.
@NikkiTaylorJNCC  and @neil-ices-dk to draft criteria for acceptance
@NikkiTaylorJNCC  with the support of @MariaMakri and @pcrjoana to review data standard, submission and user guides.

@neil-ices-dk First draft of the criteria doc Criteria for submission of delayed access data V0.1.docx. Assumption is this will be adapted to slot into the existing JCDP published data access/use doc.

@neil-ices-dk First draft of the criteria doc Criteria for submission of delayed access data V0.1.docx. Assumption is this will be adapted to slot into the existing JCDP published data access/use doc.

thanks Nikki, will get back to you as soon as i can :)

@NikkiTaylorJNCC thanks for draft, have added/edited and made comments/questions - we might need a quick call - let me know
https://community.ices.dk/ExpertGroups/WGJCDP/2024 Meeting Documents/06. Other working documents/Criteria.for.submission.of.delayed.access.data.V0.1.docx

@NikkiTaylorJNCC thanks for draft, have added/edited and made comments/questions - we might need a quick call - let me know
https://community.ices.dk/ExpertGroups/WGJCDP/2024 Meeting Documents/06. Other working documents/Criteria.for.submission.of.delayed.access.data.V0.1.docx

@NikkiTaylorJNCC just following up, anything outstanding from my side - let me know 😄

@NikkiTaylorJNCC thanks for draft, have added/edited and made comments/questions - we might need a quick call - let me know
https://community.ices.dk/ExpertGroups/WGJCDP/2024 Meeting Documents/06. Other working documents/Criteria.for.submission.of.delayed.access.data.V0.1.docx

@NikkiTaylorJNCC just following up, anything outstanding from my side - let me know 😄

@neil-ices-dk I've been through the doc, thanks for your comments! Still a few things outstanding, might be worth a quick call to try and bottom out. Do you have anytime on Monday (16th)?

14:00 UK time (i only have that slot) does that work?

@neil-ices-dk my changes didn't appear to update - version 2 uploaded hopefully:
Criteria.for.submission.of.delayed.access.data.V0.2.docx

@NikkiTaylorJNCC to finalize document based on check in discussion 16/12
@neil-ices-dk to create form based on criteria document and add to website

Document updated and ready for report form link in the new year

@NikkiTaylorJNCC to finalize document based on check in discussion 16/12 @neil-ices-dk to create form based on criteria document and add to website

Criteria.for.submission.of.delayed.access.data.V0.3.docx


No go: UcscTrack() with start="chromStart"

Getting the cpgIslandExt track from UcscTrack() has stopped working with following error:
Error in .new_IRanges_from_start_end(start, end) :
'start' and 'end' must be numeric vectors

It seems "chromStart" and "chromEnd" are no longer valid for IRanges? It needs to numerics now?
A command from the vignette will be enough to reproduce
cpgIslands <- UcscTrack(genome = "mm9", chromosome = "chrX",
                   track = "cpgIslandExt", from = from, to = to,
                   trackType = "AnnotationTrack",
                   start = "chromStart", end = "chromEnd",
                   id = "name", shape = "box", fill = "#006400",
                   name = "CpG Islands")

Ooops sorry forgot to mention versions
R version 4.3.0 (2023-04-21)
Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Debian GNU/Linux 12 (bookworm)

Matrix products: default
BLAS/LAPACK: /usr/lib/x86_64-linux-gnu/openblas-pthread/libopenblasp-r0.3.21.so;  LAPACK version 3.11.0

locale:
 [1] LC_CTYPE=en_IE.UTF-8       LC_NUMERIC=C               LC_TIME=en_IE.UTF-8        LC_COLLATE=en_IE.UTF-8     LC_MONETARY=en_IE.UTF-8    LC_MESSAGES=en_IE.UTF-8    LC_PAPER=en_IE.UTF-8       LC_NAME=C                  LC_ADDRESS=C               LC_TELEPHONE=C
[11] LC_MEASUREMENT=en_IE.UTF-8 LC_IDENTIFICATION=C

time zone: Europe/Dublin
tzcode source: system (glibc)

attached base packages:
[1] grid      stats4    stats     graphics  grDevices utils     datasets  methods   base

other attached packages:
[1] Cairo_1.6-0          Gviz_1.44.0          GenomicRanges_1.52.0 GenomeInfoDb_1.36.0  IRanges_2.34.0       S4Vectors_0.38.1     BiocGenerics_0.46.0

loaded via a namespace (and not attached):
  [1] DBI_1.1.3                   bitops_1.0-7                deldir_1.0-9                gridExtra_2.3               biomaRt_2.56.0              rlang_1.1.1                 magrittr_2.0.3              biovizBase_1.48.0           matrixStats_0.63.0
 [10] compiler_4.3.0              RSQLite_2.3.1               GenomicFeatures_1.52.0      png_0.1-8                   vctrs_0.6.2                 ProtGenerics_1.32.0         stringr_1.5.0               pkgconfig_2.0.3             crayon_1.5.2
 [19] fastmap_1.1.1               backports_1.4.1             dbplyr_2.3.2                XVector_0.40.0              utf8_1.2.3                  Rsamtools_2.16.0            rmarkdown_2.21              bit_4.0.5                   xfun_0.39
 [28] zlibbioc_1.46.0             cachem_1.0.8                progress_1.2.2              blob_1.2.4                  DelayedArray_0.26.3         BiocParallel_1.34.2         jpeg_0.1-10                 parallel_4.3.0              prettyunits_1.1.1
 [37] cluster_2.1.4               VariantAnnotation_1.46.0    R6_2.5.1                    stringi_1.7.12              RColorBrewer_1.1-3          rtracklayer_1.60.0          rpart_4.1.19                Rcpp_1.0.10                 SummarizedExperiment_1.30.1
 [46] knitr_1.42                  base64enc_0.1-3             Matrix_1.5-4.1              nnet_7.3-19                 tidyselect_1.2.0            rstudioapi_0.14             dichromat_2.0-0.1           yaml_2.3.7                  codetools_0.2-19
 [55] curl_5.0.0                  lattice_0.21-8              tibble_3.2.1                Biobase_2.60.0              KEGGREST_1.40.0             evaluate_0.21               foreign_0.8-84              BiocFileCache_2.8.0         xml2_1.3.4
 [64] Biostrings_2.68.1           pillar_1.9.0                filelock_1.0.2              MatrixGenerics_1.12.0       checkmate_2.2.0             generics_0.1.3              RCurl_1.98-1.12             ensembldb_2.24.0            hms_1.1.3
 [73] ggplot2_3.4.2               munsell_0.5.0               scales_1.2.1                glue_1.6.2                  Hmisc_5.1-0                 lazyeval_0.2.2              tools_4.3.0                 interp_1.1-4                BiocIO_1.10.0
 [82] data.table_1.14.8           BSgenome_1.68.0             GenomicAlignments_1.36.0    XML_3.99-0.14               latticeExtra_0.6-30         AnnotationDbi_1.62.1        colorspace_2.1-0            GenomeInfoDbData_1.2.10     htmlTable_2.4.1
 [91] restfulr_0.0.15             Formula_1.2-5               cli_3.6.1                   rappdirs_0.3.3              fansi_1.0.4                 S4Arrays_1.0.4              dplyr_1.1.2                 AnnotationFilter_1.24.0     gtable_0.3.3
[100] digest_0.6.31               rjson_0.2.21                htmlwidgets_1.6.2           memoise_2.0.1               htmltools_0.5.5             lifecycle_1.0.3             httr_1.4.6                  bit64_4.0.5

Hi @ivanek,
OK, yes, I hadn't defined from and to. That example does work for me now.
But not in my production code, though my call is almost identical.
Even still, with from and to undefined, the error text is not helpful as it talks about start and end, and yet it was the from and to that I got wrong. Still doing some tests ...

I am afraid, that without reproducible example I cannot help.

Yes, I understand, sorry, I should have thanked you before .. this could easily be a problem outside of Gviz .. I'm trying again this weekend.

No feedback, closing.


Document that calling tooltip/popover .show()/.hide() qualifies as manual triggering

According to http://getbootstrap.com/javascript/#popovers there doesn't appear to be a way to auto-hide a popover, that is manually shown/triggered, after some interval.
As such, I end up doing this in my application:
button.popover('show');

setTimeout(function() {
  button.popover('hide');
}, button.attr('data-delay-hide') || 2000);

Looking at the documentation on the data-delay attribute for popovers, I see "does not apply to manual trigger type".  I conjecture my button.popover('show') falls into this manual trigger category and as a result I'm responsible for hiding the popover myself?
Somewhat expected, I tried <button data-delay='{"hide":3000}'> without the JS above and this failed miserably.
Shouldn't a popover I trigger manually be hidden after the hide delay I provide in the data-delay attribute?  If not, I'd like to suggest an update the documentation at http://getbootstrap.com/javascript/#popovers to be a bit clearer with a red callout highlighting this limitation.

Thanks for the clarifications.

Philosophically speaking, honoring delay in the manual case makes things less manual, and as you've shown, the behavior you desire is still easy to implement even without this feature.

Yep, understandable.
To be honest, I think the way it behaves now is fine.  At first, I was a bit "ugh, really?" when I realized that I had to setTimeout all over the place to hide manually triggered popovers, but given the design and some further reflection I think this makes sense.

We're trying to cut down on callouts actually (it's a "if everything is top-priority, then nothing is" problem). Given that we've had few, if any, other complaints about this particular issue, a callout doesn't seem warranted yet.

Sure — a callout is overkill.
Just a quick note in the docs somewhere in the section covering popovers would suffice, I think.

 Is my understanding of this issue incorrect?

Hi @addremove!
You appear to have posted a live example (http://fiddle.jshell.net/on6czwz2/2/show/light/), which is always a good first step. However, according to Bootlint, your example has some Bootstrap usage errors, which might potentially be causing your issue:

line 51, column 3: W007: Found one or more <button>s missing a type attribute.

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!
(Please note that this is a fully automated comment.)

 Is my understanding of this issue incorrect?

Your $(this).tooltip('show') counts as manually triggering a show.
In contrast, the hiding when the mouse hovers away is automatic; indeed, your example code contains no manual .tooltip('hide'). You could disable the automatic hover event handler by setting the trigger option to an appropriate value.
In other words, the manual-ness only applies to the immediate action and doesn't infect other actions (like the subsequent hide).

In other words, the manual-ness only applies to the immediate action and doesn't infect other actions (like the subsequent hide).

Thanks for clarifying. This wasn't apparent from the docs.


When will you upload the program as a module to PyPi?

When will you upload the program as a module to PyPi? The version on PyPi is quite old. I'm writing a rsocket client connecting to a java rsocket server. It would be much easier to use if I could introduce rsocket-py as a module.

@wtj0906 currently attempting to get the pypi credentials as the maintainer of the official package cannot be contacted.
A ticket was opened with pypi. It may take time.

https://pypi.org/project/rsocket/0.3.dev1/ available for testing


[BUG] Address field with Google Api Key doesn't work on modal

Environment
Please provide as many details as you can:
I have a resource named 'A' with a component of type Address with a Google Api Key. It works well, I have the autocompletion.
But I have also an another form named 'B' with a Select component with Data Source Type as 'Resource' pluged to resource 'A' and the field 'Add resource' checked.
Adding a file for my form 'B', when I click on 'Add resource', I have a modal with the components of my resource 'A'. Everything works well but the component of type Address doesn't work.

Hosting type

[] Form.io
[x] Local deployment

Version:

Formio.js version: 4.X.X
Frontend framework: Ionic
Browser: Chrome
Browser version: latest

Steps to Reproduce

Having two forms : 'A' with a component of type address and google api key and another form 'B' with a Select component, data source type 'Resource' plugged to the form 'A' and 'Add resource' checked.
Create a new file for my form 'B', click on 'Add resource', fill in the form 'A' displayed in the modal, the field Address doesn't work

Expected behavior
Field Address working on modal, with autocompletion
Observed behavior
Field Address doesn't autocomplete on modal

Thanks for reaching out!
We are aware of this issue and have tagged you on the ticket (T4X-409).
We will let you know once this issue with the Address Field not working when the Google API is included.
Thanks,

Can I also get tagged on this ticket?

You have been tagged.

Closing this thread as it is outdated. Please re-open if it is still relevant. Thank you for your contribution!


Fixes #803 - surface all index properties

Persistence API does not surface all index properties

/gcbrun

/gcbrun

I pushed the changes for the DSE part.
The integration tests pass locally for both 3.11 and DSE. For 4.0 it looks a bit more complicated; I don't want to delay this PR so I disabled the tests and created this follow-up ticket: #815.

/gcbrun

@olim7t I am sorry, but I think my latest changes messed up with DSE module again. Would you mind to take a look and re-run the CI tests, please?

/gcbrun

/gcbrun


bug on some fields history models + missing SLA variables

Hi ! Fivetran customer, currently testing your great dbt packages on Jira and Intercom 😄.
1°)
It seems that there is the same bug in two of your models:

stg_intercom__company_history
stg_intercom__contact_history

If we take stg_intercom__company_history as an example, we have this error:
Database Error in model stg_intercom__company_history (models/stg_intercom__company_history.sql)   column reference "plan_name" is ambiguous   compiled SQL at target/run/intercom_source/models/stg_intercom__company_history.sql
Indeed, get_company_history_columns refers to multiple fields:
   {"name": "_fivetran_deleted", "datatype": "boolean"},     {"name": "_fivetran_synced", "datatype": dbt_utils.type_timestamp()},     {"name": "company_id", "datatype": dbt_utils.type_string()},     {"name": "created_at", "datatype": dbt_utils.type_timestamp()},     {"name": "id", "datatype": dbt_utils.type_string()},     {"name": "industry", "datatype": dbt_utils.type_int()},     {"name": "last_request_at", "datatype": dbt_utils.type_timestamp()},     {"name": "monthly_spend", "datatype": dbt_utils.type_int()},     {"name": "name", "datatype": dbt_utils.type_string()},     {"name": "plan_id", "datatype": dbt_utils.type_int()},     {"name": "plan_name", "datatype": dbt_utils.type_int()},     {"name": "remote_created_at", "datatype": dbt_utils.type_timestamp()},     {"name": "session_count", "datatype": dbt_utils.type_int()},     {"name": "size", "datatype": dbt_utils.type_int()},     {"name": "updated_at", "datatype": dbt_utils.type_timestamp()},     {"name": "user_count", "datatype": dbt_utils.type_int()},     {"name": "website", "datatype": dbt_utils.type_int()}
The problem is that in your subquery named final, you only select some of the fields above. For instance, plan_id and plan_name are not in this SELECT. This is the same problem for other variables above not in the final subquery's SELECT.
Let's say I want to include these fields. I'd try to add them into the company_history_pass_through_columns array. However, this is triggering an error when running stg_intercom__company_history model, because in the fields subquery, you'd get 2 times plan_name and plan_id: one time because of the fivetran_utils.fill_staging_columns (through get_company_history_columns), and one more time because of company_history_pass_through_columns --> dbt error.
This is the same issue with stg_intercom__contact_history because of the get_contact_history_columns vs. final subquery.
So I think either:

the concerned variables (plan_id, plan_name and some other ones) must be deleted from get_company_history_columns and you keep the final subquery like this
or
you don't edit the get_company_history_columns and you add the concerned variables in the final subquery.

Hope it's really a bug and not some misunderstanding from me 😆
2°) It's not a bug, but it seems that sla_name and sla_status are not taken from conversation_history in your models. It'd be awesome to include these two variables, not only in dbt_intercom_source, but also in dbt_intercom, as they are definitely key metrics for conversations.
Thank you in advance and sorry for this loooong message!
Antoine

hey @ant0inemtr thanks for bringing this up! We're working on a fix now and will let you know when the changes are ready.

hey @ant0inemtr thanks for bringing this up! We're working on a fix now and will let you know when the changes are ready.

Hey @ant0inemtr we have been able to release a new version of the Intercom packages to the dbt hub with the fixes you mentioned in this issue as well as some other enhancements! You can find the updated packages in the below links:

dbt_intercom_source
dbt_intercom

Thanks again for bringing this bug to our attention, and happy dbt'ing!


Metasploit::Cache::Payload::Stager::Instance

MSP-12426
Pre-verification Steps

[x] Merge #29

Verification Steps

[x] rm Gemfile.lock
[x] bundle install

Feature tests

[ ] open docs/images/metasploit-cache.png
[ ] VERIFY all attribute columns for Metasploit::Cache::Payload::Stager::Instance are implemented
[ ] VERIFY payload_stager_class_id column is implemented as are the payload_stager_class association
[ ] VERIFY inverse payload_stager_instance association on Metasploit::Cache::Payload::Stager::Class.

Test coverage

[ ] rake cucumber spec coverage
[ ] VERIFY no failures
[ ] VERIFY 100% coverage

Documentation

[ ] rake yard
[ ] VERIFY only warnings are:

[warn]: @param tag has unknown parameter name: module_instances
    in file `app/models/metasploit/cache/module/class.rb' near line 147
[warn]: @param tag has unknown parameter name: module_instance
    in file `app/models/metasploit/cache/module/instance.rb' near line 317
[warn]: @param tag has unknown parameter name: architecture_abbreviations
    in file `app/models/metasploit/cache/module/instance.rb' near line 336
[warn]: @param tag has unknown parameter name: architectured
    in file `app/models/metasploit/cache/module/instance.rb' near line 353
[warn]: @param tag has unknown parameter name: platforms
    in file `app/models/metasploit/cache/module/instance.rb' near line 367
[warn]: @param tag has unknown parameter name: platform_fully_qualified_names
    in file `app/models/metasploit/cache/module/instance.rb' near line 403
[warn]: @param tag has unknown parameter name: module_target
    in file `app/models/metasploit/cache/module/instance.rb' near line 420
[warn]: @param tag has unknown parameter name: module_instance
    in file `app/models/metasploit/cache/module/instance.rb' near line 434
[warn]: @param tag has unknown parameter name: module_target
    in file `app/models/metasploit/cache/module/instance.rb' near line 492

[ ] VERIFY no undocumented objects

Post-merge Steps
Perform these steps prior to pushing to master or the build will be broken on master.
Version

[ ] Edit lib/metasploit/cache/version.rb
[ ] Remove PRERELEASE and its comment as PRERELEASE is not defined on master.

Gem build

[ ] gem build *.gemspec
[ ] VERIFY the gem has no '.pre' version suffix.

RSpec

[ ] rake spec
[ ] VERIFY version examples pass without failures

Commit & Push

[ ] git commit -a
[ ] git push origin master

handler_id also appears to be missing from the mc_payload_stager_instances table in this PR


Trouble with publishing to Sonatype "gpg: no default secret key: No secret key"

Hello,
I have been using your project for years and everything is wonderful except recently.
For two projects:
https://github.com/leobenkel/umlclassdiagram/runs/5167148289?check_suite_focus=true
and
https://github.com/leobenkel/ZparkIO/runs/4674163615?check_suite_focus=true
I am not able to publish anymore.
With the third one: https://github.com/leobenkel/Soteria/runs/4595934045?check_suite_focus=true it works perfectly well.
All three repos have the exact same values for
PGP_PASSPHRASE
PGP_SECRET
SONATYPE_PASSWORD
SONATYPE_USERNAME

What am I doing wrong ?
When I do gpg -k , I get an expiration date of 2031-12-18 I recently expanded the expiration date. Which made the third of the three repo work again but not the others.

I don't have any answers, but I noticed
[info] gpg: no default secret key: No secret key
[info] gpg: signing failed: No secret key
[error] java.lang.RuntimeException: Failure running 'gpg --batch --pinentry-mode loopback --passphrase *** --detach-sign --armor --use-agent --output /home/runner/work/umlclassdiagram/umlclassdiagram/target/scala-2.12/sbt-1.0/umlclassdiagram-1.0.0.pom.asc /home/runner/work/umlclassdiagram/umlclassdiagram/target/scala-2.12/sbt-1.0/umlclassdiagram-1.0.0.pom'.  Exit code: 2

in the log. There was also a similar report https://github.com/sbt/sbt-ci-release/issues/225.
It is confusing because it does also say
gpg (GnuPG) 2.2.19
gpg: directory '/home/runner/.gnupg' created
gpg: keybox '/home/runner/.gnupg/pubring.kbx' created
gpg: /home/runner/.gnupg/trustdb.gpg: trustdb created
gpg: key 8E544915D44EDDD8: public key "Zparkio Bot<EMAIL_ADDRESS>imported
gpg: key 8E544915D44EDDD8: secret key imported
gpg: Total number processed: 1
gpg:               imported: 1
gpg:       secret keys read: 1
gpg:   secret keys imported: 1

Or is there a different publishing platform that might be easier instead of sonatype ?

When you query your public key on https://keyserver.ubuntu.com/ does the updated expiration date show up?

At least I solved one problem https://issues.sonatype.org/browse/OSSRH-78193

I don't understand the connection here. Are you saying that putting
sonatypeCredentialHost := "oss.sonatype.org"
sonatypeRepository     := "https://oss.sonatype.org/service/local"

somehow resolved your GPG signing of POM files? How is that possible?

oh no, i just resolved an other problem i had where i couldnt connect to sonatype.org.

well... regenerating the key on gpg worked https://github.com/leobenkel/umlclassdiagram/runs/5203363539?check_suite_focus=true 🤷♂


Maven POM resolution leads to StackOverflowError if parent GAVs reference resolved POM GAVs

Opening this issue with reference of JIRA issue: https://issues.gradle.org/browse/GRADLE-3485
I am currently facing same issue as mentioned in https://discuss.gradle.org/t/failure-to-resolve-dependency-but-artifact-exists/17977/5
I am trying to build grails 2.3.11 from git repository to my local machine with additional logging statements. But due to above error I am not able to build grails distribution files.
I guess this issue is been reproduce with all the cases with:

Group: org.springframework.uaa
Artefact: org.springframework.uaa.client
Version: 1.0.1.RELEASE/1.0.2.RELEASE

I am not sure whether this issue is gradle issue or POM issue (I have recently migrated my project to Ant to Gradle, so having little knowledge of Gradle). By reading at gradle discussion forum (link mentioned above), I am posting issue here.

@parthbhagat Could you please add the description of the JIRA ticket here as well?

From grails-bootstrap progject, gradle dependency for spring uaa client:
uaa("org.springframework.uaa:org.springframework.uaa.client:1.0.2.RELEASE") {
	["protobuf-java-lite", "json-simple", "bcpg-jdk15", "bcprov-jdk15"].each { dep ->
		exclude group: "org.springframework.roo.wrapping", module: "org.springframework.roo.wrapping." + dep
	}
}

This results in following dependency resolution error.
Could not resolve org.springframework.uaa:org.springframework.uaa.client:1.0.1.RELEASE.
     Required by:
         org.grails:grails-bootstrap:2.3.11
      > Could not parse POM http://repo.grails.org/grails/core/org/springframework/uaa/org.springframework.uaa.client/1.0.1.RELEASE/org.springframework.uaa.client-1.0.1.RELEASE.pom

This error is been thrown recursively and eventually yield to java.lang.StackOverflowError.


fix(content): do not monetize during prerendering

Context

Closes #477.
Changes proposed in this pull request

Extension builds preview

Name
Link

Latest commit
2f5b72902afa2d398004e85d8b4d63d9b4b1d665

Latest job logs
Run #10262740023

DownloadDownload


Add support for BETWEEN

Current coverage is 81.00% (diff: 91.66%)

Merging #4215 into master will increase coverage by <.01%

@@             master      #4215   diff @@
==========================================
  Files          1300       1300
  Lines         44278      44288    +10
  Methods           0          0
  Messages          0          0
  Branches       7060       7060
==========================================
+ Hits          35868      35877     +9
- Misses         5760       5762     +2
+ Partials       2650       2649     -1

Powered by Codecov. Last update dcc6811...ef10265

@matriv please review

@dobe please take a look as well

We can add symmetric later on I guess - if somebody asks for it.
I mainly did this because I saw that it is a very low hanging fruit. Symmetric would require parser changes as well.


Add hour diff to profile popover

Summary
Adds hour difference instead of timezone in the profile popover
Screenshots

before
after

Release Note
Adds hours ahead / behind in profile popover.

E2E tests not automatically triggered, because PR has no approval yet. Please ask a developer to review and then try again to attach the QA label.

Opened a new PR that wasn't based on a fork to allow the build to complete

@coltoneshaw I'm not seeing the diff on the test server. See screenshot below (note, the local time is 12:56, so the diff should show here). Is there a setting that needs to be enabled on the server or something?

@matthewbirtch it only shows when the two users have a diff in local time, if they have the same local time then it doesn't show. This pulls from the timezone configuration on their profile. I can adjust it to utilize the timezone of the browser if that would be more ideal.

@matthewbirtch it only shows when the two users have a diff in local time, if they have the same local time then it doesn't show. This pulls from the timezone configuration on their profile. I can adjust it to utilize the timezone of the browser if that would be more ideal.

@coltoneshaw I changed the sysadmin user's local time setting in MM to be different from the user-1 and still didn't see the diff on the test server

You don't have permissions to trigger this command.
It's only available for organization members.

/e2e-test

Successfully triggered E2E testing!
GitLab pipeline | Test dashboard

This will be nice @coltoneshaw, a couple of minor suggestions, but the overall approach is solid!
@matthewbirtch @coltoneshaw what would you guys think about keeping the timezone name and putting the time offset/difference in a tooltip?
I and I'm sure at least a minority of people enjoy understanding the timezone—it's helpful (and often familiar) as a rough geo/region indicator.

I totally agree it adds value. Maybe local time includes the hours diff, and add a field for timezone? Ideally we have a city profile attribute, but that's for another day.

I totally agree it adds value. Maybe local time includes the hours diff, and add a field for timezone? Ideally we have a city profile attribute, but that's for another day.

That's a great point. I'm happy to let timezone go away if it might come back as a separate field in some full profile later on, but until then it would be a slight downside to this IMO.

That's a great point. I'm happy to let timezone go away if it might come back as a separate field in some full profile later on, but until then it would be a slight downside to this IMO.

Agreed. Let me play with adding a timezone profile field that's auto generated from the configured time zone.

@matthewbirtch @coltoneshaw what would you guys think about keeping the timezone name and putting the time offset/difference in a tooltip?

One possibility, if we feel strongly about displaying the timezone value, is to show this in brackets on the Local Time title (option 1 below). Or we could do show the hours diff inline with the Local Time title instead. (option 2 below)

@matthewbirtch ooo, I love the first one and it doesn't add unneeded fields to the profile!

/e2e-test

Successfully triggered E2E testing!
GitLab pipeline | Test dashboard

@matthewbirtch - Would you prefer 4.75 hr. behind or 4 hr. 45 min. behind ?

Okay, adjusted this with the above feedback. I'm still on the fence of hours only or minutes plus hours. Below are both options, live.

/e2e-test

Successfully triggered E2E testing!
GitLab pipeline | Test dashboard

@matthewbirtch - Would you prefer 4.75 hr. behind or 4 hr. 45 min. behind ?

I would say let's go with 4.75 hr. - it will get too long otherwise

/e2e-test

Successfully triggered E2E testing!
GitLab pipeline | Test dashboard

@furqanmlk - passed with 97% and I don't see any related. Any changes needed?
ps: I tagged you in the wrong PR a second ago, this is the real one.

@furqanmlk - passed with 97% and I don't see any related. Any changes needed?
ps: I tagged you in the wrong PR a second ago, this is the real one.


New set of viral genomes for testing

This PR fixes #94.
We will now have 3 sets of genomes for testing:
Set 1: viral_example four viral genomes, for which all comparisons return meaningful alignments.
Set 2: bacterial_example for bacterial genomes, needed to test fastani option
Set 3: failure_example four viral genomes, for which some comparisons return empty alignments. This is needed to handle ZeroDivisionError.

There seem to be only two viral genomes in the new example (not four), what are genome_3.fasta and genome_4.fasta and do they need better names?

My apologies the naming of the genomes might not be the best, and I'm happy to change this to something else. There are, however, 4 genomes:

MGV-GENOME-0264574.fna
MGV-GENOME-0266457.fna
genome_3.fasta
genome_4.fasta

I wanted to keep the MGV-GENOME-0264574.fna and MGV-GENOME-0266457.fna as these are the "problematic" genomes provided by Donovan a while ago. The other two genomes (genome_3 and genome_4), I downloaded from NCBI after finding these were very similar to the MGV-GENOMOES.

It may be GitHub misleading me then - it looked like just 3 and 4 in the directory.

Do we need/want the bacterial example anymore? We're currently still using it for fastANI, but only because it is fast enough (otherwise I'd have switched it to the viruses). Probably a Leighton question...

Do we need/want the bacterial example anymore?

You are right. We might not need them any more.
I guess it won't reduce time for testing substantially, but it will make tests directory a wee bit neater, or we could rearrange things a little bit if we decide to keep them. Either way let me know, what you prefer and I will take care of things ;)

Thinking about this, right now we don't need multiple sets of genomes at all - we just need one set for the snakemake tests where we also have the intermediate files under the fixtures. And this should be as small and fast as reasonably possible (so maybe 3 rather than 4 viruses).
Is there anything stopping you using the current virus set for testing ANIm calculation? If so, maybe we should just revert b7522253954e3a8e0a18ae472b619f8053762301 and b2b1c227350c3131653818c249935c86f18f49ac and instead add one of your similar viruses.
Separately, shall I switch the fastANI snakemake tests to use the same virus set (meaning the bacterial set will be unused in the short term)?
Later we will want additional sets of genomes for testing things like reports and plotting - but there I see no reason to record the intermediate files in git for testing.

There are two options here.
Option 1
One set containing four viral genomes, where all comparisons return an alignment. This would help verify that everything is working as expected, including multiway comparisons, calculations, etc. This set is currently in viral_example.
Another set of genomes where not all comparisons return an alignment. This would allow us to ensure that ZeroDivisionError is handled correctly. Treating these tests separately would give us a clear distinction when things go wrong. This set is currently in failure_example.
Option 2
Alternatively, we could revert to one of the previous commits, as you suggested, and keep a single test set with four genomes, where two genomes do not return an alignment.
My only issue with option 2 is that I will not be able to run the older version of pyANI on all four genomes to obtain matrices to verify the accuracy of the calculations. This is because pyANI fails to complete the comparisons due to a ZeroDivisionError. Yes, we can run pyANI v0.3 on two genomes, which will rerun an alignment to obtain matrices for other testing purposes.
One thing that might be worth remembering is that there may be a slight issue with the calculations for dnadiff method. I have run into problems before where my calculated dnadiff values occasionally differ from those returned by dnadiff software, because the content of the .rdiff files can sometimes vary between runs. The tool identifies gaps in the alignment using graphs, I believe, which can sometimes result in different values.

Ah - I was under the impression pyANI v0.3 could handle the non-aligning cases, that does make dealing with badly aligning genomes a little harder to deal with long term as we'd have to compile the expected results "by hand".

Ah - I was under the impression pyANI v0.3 could handle the non-aligning cases.

Unfortunately it does not. Should we go with option 1, and have two different sets for both cases for "working" and "failing" genomes?

Option three, a small set of three viruses which all give alignments - #98 ?
And later test non-alignments explicitly which doesn't have to be part of an all-vs-all example.

Handled via https://github.com/pyani-plus/pyani-plus/pull/98


[RFC] Support for external authentication via Macaroons

Add support for external authentication via macaroons (https://github.com/go-macaroon/macaroon)

jenkins: test this please

jenkins: test this please

Can you also move the po/ update to its own commit? that should make things a bit easier to review on Github by not having my browser get stuck on the massive po diff :)

@stgraber done

jenkins: test this please

jenkins: test this please

jenkins: test this please

What's going on with all the sign-offs on the first commit?

I'm going to need the client commit to be split into:

A commit for the stuff under client/
A commit for the stuff under lxc/config/
A commit for the rest of the lxc/ stuff and the change to the tests
A commit for the lxd-benchmark change

We'll need this branch to update doc/security.md with a description of the macaroon based auth stuff.

Also going to need to figure get tests in place before we can merge this.

@stgraber updated commits and added doc

@stgraber also, added the test auth server and a test using external auth

jenkins: test this please

I can't figure out what's failing in tests on jenkins.
They pass locally and I don't see any obvious error in the log

Problem seems to be a rather big diff in godeps output

==> TEST BEGIN: test_static_analysis
2017/10/10 22:27:24 auth - running at http://<IP_ADDRESS>:19114
ERROR: you added a new dependency to the client or shared; please make sure this is what you want
--- test/godeps.list	2017-10-10 22:19:12.722359971 +0000
+++ -	2017-10-10 22:27:33.316777216 +0000
@@ -1,20 +1,16 @@
 github.com/golang/protobuf
 github.com/gorilla/websocket
-github.com/juju/errors
 github.com/juju/go4
 github.com/juju/httprequest
 github.com/juju/loggo
 github.com/juju/persistent-cookiejar
-github.com/juju/schema
-github.com/juju/utils
 github.com/juju/webbrowser
 github.com/julienschmidt/httprouter
+github.com/lxc/lxd
 github.com/rogpeppe/fastuuid
 golang.org/x/crypto
 golang.org/x/net
-golang.org/x/sys
 gopkg.in/errgo.v1
-gopkg.in/juju/environschema.v1
 gopkg.in/macaroon-bakery.v2-unstable
 gopkg.in/macaroon.v2-unstable
 gopkg.in/retry.v1

This is masked due to your change in that particular part of the test. To get it visible again, you'll need:
diff --git a/test/suites/static_analysis.sh b/test/suites/static_analysis.sh
index 0cba3df7..e0f918cb 100644
--- a/test/suites/static_analysis.sh
+++ b/test/suites/static_analysis.sh
@@ -67,7 +67,7 @@ test_static_analysis() {
     fi

     if which godeps >/dev/null 2>&1; then
-      OUT=$(godeps -T ./client ./lxc/config ./shared/api 2>/dev/null | cut -f1 | diff -u test/godeps.list -)
+      OUT=$(godeps -T ./client ./lxc/config ./shared/api 2>/dev/null | cut -f1 | diff -u test/godeps.list - || true)
       if [ -n "${OUT}" ]; then
         echo "ERROR: you added a new dependency to the client or shared; please make sure this is what you want"
         echo "${OUT}"

I'll have to take a closer look at the dependency chain we're getting for client/ and lxc/config/, see if we can make some clever use of interfaces to try to reduce it a bit. I don't care about it too much for LXD itself but I'd like to minimize the impact on downstream users of our client packages (keeping as much of the new dependencies limited to lxc/ and lxd/).

Plan right now is to merge this on Wednesday after LXD 2.19 is out.

# github.com/lxc/lxd/lxd
lxd/daemon.go:178: too many arguments in call to httpbakery.NewDischargeRequiredError
	have (*bakery.Macaroon, string, *bakery.DischargeRequiredError, *http.Request)
	want (httpbakery.DischargeRequiredErrorParams)


Inference problem

Hi! Good job one the model.
But i have trouble testing it.
Setup RTX4090 + 64gb ram (on loading models im kissing 63.9gb :) )
Tested on Windows - can't launch with default code cause of missing support for FLASH_ATTENTION

Exchanged for EFFICIENT_ATTENTION and i hear initial prompt with "bob how's it going bob" and than silence
Unfortunately same on Linux (no errors setup). Only initial prompt and nothing more. Silence :(
Torch installed with this command:
pip3 install torch torchaudio --index-url https://download.pytorch.org/whl/cu118
Any tips how to go further with this?

Maybe somebody end up here with a problem - on inference_client.py i was missing this to launch it on linux Ubuntu
sudo apt-get install libportaudio2

same problem!!

@wpq3142 did the libportaudio2 fix work for you? I've added it to the readme

Sorry i mixes two things in one issue.
Original issue from first post is not resolved.
libportaudio2  fix was needed on Ubuntu to be able to select audio devices. And it WORKS.

how much vRam memory requires?
I have 16GB 3060 and got CUDA out of memory.

how much vRam memory requires? I have 16GB 3060 and got CUDA out of memory.

with our current bfloat16 implementation, 24GB.

I'm able to run it on (gasp) CPU with 64GB RAM, and while I did see it spoke up to 60GB when loading checkpoints, it seems to hover around the 16~20GB range


[Select] asChild props is not available on Select.Trigger

Hello all,
I've noticed a minor discrepancy in the documentation for Select.Trigger. The documentation currently states that it accepts the asChild props, but in practice, it doesn't.
Upon further investigation, it appears that this prop is not supported, suggesting that there may be a documentation error. 😊

Same with the DropdownMenu. It should be documented that the trigger accepts a button as a child component without the use of asChild

Just ran into this as well.

Seems like the documentation needs to be updated.

We’ll update the docs. What outcome were you expecting when using asChild though? The trigger comes with its own styles, so if asChild were supported, it would have merged its styles with the child element. Is this what you waned to achieve?

We'll update this in the docs for 3.0.0


Static-only objects are included in wildcards expansions

Describe the bug
Currently, expanding a wildcard will include members that are only "statically" present.
More specifically: members that are defined or imported under an if TYPE_CHECKING or if False condition.
This is problematic because this can lead to cyclic aliases.
To Reproduce
Steps to reproduce the behavior:
# package/module_a.py
from typing import TYPE_CHECKING

if TYPE_CHECKING:  # always false
    from package.module_b import CONST

# package/__init__.py
from .module_a import *

At runtime, package would definitely not have CONST in its members.
Expected behavior
These "static-only" objects should not be included in wildcards expansions.
Additional context
https://github.com/mkdocstrings/mkdocstrings/issues/382
Maybe an other way to fix that would be to better handle duplicate members.
But that depends on the order we add expanded wildcard's members...

This is how I check for the TYPE checking blocks: https://github.com/tristanlatr/pydocspec/blob/main/pydocspec/astroidutils.py#L359-L374
Objects have a field is_type_guarded wich is set to True is the context is under a type guard if condition.
Then the wildcard import rely on __all__ or the public names returnde with this function: https://github.com/tristanlatr/pydocspec/blob/main/pydocspec/processor/mod_attr.py#L75-L87

Thanks @tristanlatr! I was indeed thinking about adding a field on objects so that we know they are type-guarded or not.

Released in 0.17.0!


Fixes GraphQL <> EdgeQL typo.

I believe this was a simple copy & paste typo.

I believe this was a simple copy & paste typo.

Indeed it was, thank you!


Generate XML file for vulnerabilities RSS feed

Generates an RSS XML file for package vulnerabilities alongside the vulnerabilities docs page.
This is what the XML file would look like if there were some vulnerabilities:
<?xml version="1.0" encoding="UTF-8" ?>
<rss version="2.0">

<channel>
<title>z/OS Open Tools Vulnerabilities</title>
<link>https://zosopentools.org/#/Vulnerabilities</link>
<description>Vulnerabilities in z/OS Open Tools Packages</description>

<item>
<title>caddy (Build 2215) - (STABLE)</title>
<link>https://zosopentools.org/#/Vulnerabilities?id=caddy</link>
<description>1 medium vulnerability</description>
<guid isPermaLink="false">caddy (Build 2215) - (STABLE)</guid>
</item>

<item>
<title>logrotate (Build 2172) - (STABLE)</title>
<link>https://zosopentools.org/#/Vulnerabilities?id=logrotate</link>
<description>1 medium vulnerability</description>
<guid isPermaLink="false">logrotate (Build 2172) - (STABLE)</guid>
</item>

<item>
<title>grafana (Build 2267) - (STABLE)</title>
<link>https://zosopentools.org/#/Vulnerabilities?id=grafana</link>
<description>1 high vulnerability</description>
<guid isPermaLink="false">grafana (Build 2267) - (STABLE)</guid>
</item>

<item>
<title>grafana (Build 2266) - (STABLE)</title>
<link>https://zosopentools.org/#/Vulnerabilities?id=grafana</link>
<description>2 vulnerabilities (1 critical, 1 high)</description>
<guid isPermaLink="false">grafana (Build 2266) - (STABLE)</guid>
</item>

</channel>

</rss>

Each package release is a separate item and each item has a unique GUID, so I believe users should be notified whenever a new package release with vulnerabilities comes out and shouldn't get repeat notifications when the XML file gets updated.
Also, I'm not sure how users actually subscribe to RSS feeds. Do we need to add it to some feed directories, or can people subscribe to it themselves if we provide the URL to the XML file?

I use a browser plugin like this: https://chromewebstore.google.com/detail/rss-feed-reader/pnjaodmkngahhkoihejjehlcdlnohgmp . I just created a copy and it seems to work: https://raw.githubusercontent.com/IgorTodorovskiIBM/IgorTodorovski/main/feed.xml . Nice job!
I did notice that your branch is a bit oudated. You'll need to rebase or merge with main.
You can do a git fetch first, and then in your branch, you can do git rebase origin/main . And then you can do a git push --force.

An accompanying blog post showing how to add RSS feeds to Outlook desktop app will be handy.
Customers can do that.
Individuals can use a feed reader such as Feedly, Newsblur, etc.
It's not uncommon that browser extension maintainers are enticed by $, as companies want to buy the data they process :)

Looks good, one last thing: can you add a link in the md file that points to the raw RSS xml file? That way users can click on it to get to the RSS feed


Add Hilt Dependency Injection

Change logs

added hilt core functionality
added hilt DI Inside

Activity
Fragments
Repository
Datastore

Updated Datastore to support safe mutations | Read and Write store
Reformatted code for Test support
General code clean up

PR Merged 🚀... Thanks @ch8n 🙌


[BUG] Authentication error during account creation

New delegate RSP authentication error. I'm attempting to create an account following the video on the new delegate checklist. No matter how I login (orcid, github, my institution) I'm getting the following error.

Hi! I just encountered the same issue. Using the link provided by @rra got around the problem.

Thanks for reporting @sarahdraves that's very helpful to know.

We deployed a new version of our authentication software during the patch window today and this problem should hopefully now be properly fixed. Thank you for the report!


clusteradm instructions specify invalid commands

I'm using the latest clusteradm installed from source and the instructions in the Quick Start mention the --wait flag which doesn't exist on both the init and join commands.
Additionally, in the Application Lifecycle addon installation instructions, the clusteradm install command does not exist. I had to follow the directions to install it from source.

For the app addon I created a PR here to fix the clusteradm doc https://github.com/open-cluster-management-io/clusteradm/pull/118 But I forgot to update the website.
The wait command should exists at least with the join command.
$ clusteradm join --help | grep wait
      --wait                             If true, running the cluster registration in foreground.

Can you check? Maybe you have another older clusteradm in the path?

@mikeshng thanks!
This is what I get for output:
~ ❯ clusteradm version
client		version	:0.1.0-alpha.4
server release	version	:v1.21.1
~ ❯ clusteradm join --help | grep wait
      --request-timeout string           The length of time to wait before giving up on a single server request. Non-zero values should contain a corresponding time unit (e.g. 1s, 2m, 3h). A value of zero means don't timeout requests. (default "0")

@mikeshng you were right. I had the incorrect version installed. Thanks!


Make add-on for work Anki 2.1.x

Based on the documentation, this shouldn't be too difficult.

I did some updates here to take care of this.
Thanks a lot for your work. Great plugin!

I forked this and made some small adjustments for compatibility with Windows

@michaelmagistro pull requests welcomed here


A mean to remove reinforcement in a larger area

Opening this here, because its supposed to be implemented here, need discussion first

One idea is a beacon which consumes fuel, and in return it decays the blocks around it? Placing a beacon with pyramid means having control over an area (so an attacker couldn't sneak this into a vault). And this would allow for it to cost fuel to do large areas, and time (perhaps one decay increment every 24 hours or something?)

Maybe it could be a special kind of factory, but instead of a chest at the end it's a beacon, that way you can control the radius and cost through recipes.

Maybe it could be a special kind of factory, but instead of a chest at the end it's a beacon, that way you can control the radius and cost through recipes.

Ooh I like that.
I think another question is how does it work exactly - does it decay/ remove all reinforcements not on the group of the factory within a radius?

Hmm, creating the actual factory should have a gold cost, and the build materials already includes beacons, which is, if I'm not mistaken, already the most expensive block in the game. But we don't want this to be out of reach for newer players who wish to take over an abandoned town, you know? Perhaps it could cost essence? But the most important part is that the aciding should be abundantly obvious, like truly unmissable to anyone around.

So I'm thinking back to the beacon idea. You place a beacon, and fuel it with a stack of gold blocks. It "warms up" similar to acid blocks. After the warmup time period has elapsed, you can /ctacid any block in a radius around the beacon. Doing /ctacid will consume the fuel in the beacon (scaling based on reinforcement type?).
This requires you to play a large piece of infrastructure (beacon + beacon base) and have control of an area to safely place it and protect it. You then need control of the area long enough to warm it up (24 or 48 hours?), and using /ctacid will consume gold blocks (consumption rate TBD) so it's not too OP.
This idea would require a custom beacon, a timer and fuel level, and /ctacid to be linked. I don't think any of those things is too burdensome, and balances the needs of it being very obvious when area-based acid is being used.

Should the beacon be built specifically e.g all gold block pyramid?

Also if either of you suggest making this into a factory any further I'm gonna hunt you down and turn everything you own into a factory


Log exported users

What's this PR do?
:memo: Adds migrated users to the Northstar request log, so we can track if we run into any errors.
:1234: Store the returned Northstar ID to the Drupal profiles of migrated users for future use.
:boom: Fixes a crash I ran into when re-testing this where it was trying to migrate the "anonymous" user and crashing in a big ball of flames.
How should this be manually tested?
Feel free to pull down this branch and test against your local environment.
Any background context you want to provide?
Noooooooope.
What are the relevant tickets?
References #6260.

For review: @angaither

just a minor comment, not needed for merge, but :+1:


[SPARK-37960][SQL][FOLLOWUP] Make the testing CASE WHEN query more reasonable

What changes were proposed in this pull request?
Some testing CASE WHEN queries are not carefully written and do not make sense. In the future, the optimizer may get smarter and get rid of the CASE WHEN completely, and then we loose test coverage.
This PR updates some CASE WHEN queries to make them more reasonable.
Why are the changes needed?
future-proof test coverage.
Does this PR introduce any user-facing change?
'No'.
How was this patch tested?
N/A

ping @huaxingao cc @cloud-fan

thanks, merging to master/3.3!

@cloud-fan @huaxingao Thank you!


Fix Issue 20263 - Wrong value for CustomFloat.min_exp

Just a cast was missing to allow for negative values...

Auto-merge toggled on


Correct element is not being focused in flashlist/flatlist

Description
I am trying to focus to a my set initial element, but it selects entirely different element, on previous version of tvos (npm:react-native-tvos@0.69.8-0), i was kinda able to focus initial element like intended, but only when i pressed dpad navigation button twice. Here's code snippet:
const EventListComponent = () => {
  const firstEventRef = useRef<View | null>(null);
  const liveEventRef = useRef<View | null>(null);
  const flatListRef = useRef<FlashList<IEvent> | null>(null);
  const focusGuideRef = useRef<FocusGuideRef>(null);
  const firstItemIndex = calculateFirstItemIndex();

  useEffect(() => {
    let scroll = true;

    InteractionManager.runAfterInteractions(() => {
      if (!scroll) {
        return;
      }

      let destination;

      if (firstItemIndex !== 0) {
        destination = liveEventRef.current;
        flatListRef.current?.scrollToIndex({
          animated: false,
          index: firstItemIndex,
        });
      } else {
        destination = firstEventRef.current;
      }

      focusGuideRef.current?.setDestinations([destination]);
    });

    return () => {
      scroll = false;
    };
    // selectedChannelId is a dependency because we want to reset the selected
  }, [someDependencyToResetDestination]);

  const renderEventItem: ListRenderItem<IEvent> = ({ index, item: event }) => {
    const isFirstIndex = index === 0;
    const isLive = firstItemIndex === index;
    const isArchive = liveTimestamp > event.end;

    let rightIcon;

    if (isLive) {
      rightIcon = <SVGS.LIVE height="100%" />;
    } else if (isArchive) {
      rightIcon = <SVGS.ARCHIVE height="100%" />;
    }

    return (
      <EventItem
        ref={component => {
          if (firstItemIndex === index) {
            liveEventRef.current = component;
            console.log('index', index);
          }

          if (isFirstIndex) {
            firstEventRef.current = component;
          }
        }}
        event={event}
        onBlur={() => {
          if (isLive) {
            focusGuideRef.current?.setDestinations([]);
          } else if (isFirstIndex) {
            focusGuideRef.current?.setDestinations([]);
          }
        }}
      />
    );
  };

  return (
    <TVFocusGuideView
      key={selectedChannelId}
      ref={focusGuideRef}
      autoFocus
      style={styles.guideContainer}
      trapFocusDown
      trapFocusUp
    >
      <FlashList
        ref={flatListRef}
        data={events}
        disableHorizontalListHeightMeasurement
        estimatedItemSize={EVENT_ITEM_HEIGHT}
        estimatedListSize={{
          height: CONSTANTS.TV_HEIGHT,
          width: EVENT_ITEM_WIDTH,
        }}
        initialScrollIndex={firstItemIndex}
        keyExtractor={keyExtractor}
        renderItem={renderEventItem}
      />
    </TVFocusGuideView>
  );
};

Added video, as you see it focus on some random element instead of live one:
https://user-images.githubusercontent.com/66637451/230934671-7376a3b2-87ae-449a-8959-7624dd071c23.mov
Version
<EMAIL_ADDRESS>Output of npx react-native info
System:
OS: macOS 13.0
CPU: (8) arm64 Apple M1
Memory: 87.84 MB / 16.00 GB
Shell: 5.8.1 - /bin/zsh
Binaries:
Node: 18.15.0 - /usr/local/bin/node
Yarn: 1.22.19 - ~/.yarn/bin/yarn
npm: 9.5.0 - /usr/local/bin/npm
Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
Managers:
CocoaPods: 1.12.0 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 22.2, iOS 16.2, macOS 13.1, tvOS 16.1, watchOS 9.1
Android SDK:
API Levels: 29, 30, 31, 32, 33
Build Tools: 30.0.2, 30.0.3, 31.0.0, 32.0.0, 33.0.0, 34.0.0
System Images: android-29 | Android TV Intel x86 Atom, android-30 | Google APIs Intel x86 Atom, android-31 | Android TV ARM 64 v8a, android-31 | Google Play ARM 64 v8a, android-32 | Google Play ARM 64 v8a, android-33 | Android TV ARM 64 v8a
Android NDK: Not Found
IDEs:
Android Studio: 2021.3 AI-213.7172.25.2113.9123335
Xcode: 14.2/14C18 - /usr/bin/xcodebuild
Languages:
Java: 11.0.13 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 18.0.0 => 18.0.0
react-native: Not Found
react-native-macos: Not Found
react-native-tvos:  0.69.8-1
npmGlobalPackages:
react-native: Not Found
Steps to reproduce

create a big flashlist that doesn't fit into a screen
iniate big flashlist, with initial element somewhere in the list
scroll to initial element
setDestinations to initial element

Snack, code example, screenshot, or link to a repository
No response

Kinda was able to solve my issue by reverting version back 0.69.8-0 and wrapping tvfocusguide in touchableopacity and redirecting focus to tvfocusguide:
  return (
    <TouchableOpacity
      disabled
      onFocus={() => {
        console.log('onFocus');
        focusGuideRef.current?.requestTVFocus();
      }}
    >
      <TVFocusGuideView
        key={selectedChannelId}
        ref={focusGuideRef}
        autoFocus
        style={styles.guideContainer}
        trapFocusDown
        trapFocusUp
      >
        <FlashList
          ref={flatListRef}
          data={events}
          disableHorizontalListHeightMeasurement
          estimatedItemSize={EVENT_ITEM_HEIGHT}
          estimatedListSize={{
            height: CONSTANTS.TV_HEIGHT,
            width: EVENT_ITEM_WIDTH,
          }}
          initialScrollIndex={firstItemIndex}
          keyExtractor={keyExtractor}
          renderItem={renderEventItem}
        />
      </TVFocusGuideView>
    </TouchableOpacity>
  );

@yusufyildirim there is a workaround for this, but thought you should be aware.
@stepas-dev maybe this works better in the new 0.71.10-0 release.


UITextField Scrolling

Good Day,
First off your tutorials and Serial Pro have helped immensely. I am very new to Swift Coding and currently attempting a simpler version of serial that allows me to view six sensors data sent as character. I am using UITextfield to display updated characteristic but I am running into an issue of the text is scrolling much like the serial app. is there a way to keep TextField from scrolling and just updating Text? Any advice is appreciated and much thanks for your time and app!

If the message is of fixed length, you can do something like this:
var bytes: [UInt8] = []

func serialDidReceiveBytes {
  for b in receivedBytes {
    bytes.append(b)
    if bytes.count == EXPECTED_LENGTH {
      textView.text = (set text)
      bytes = []
    }
}

If they are not of fixed length, you'll have to come up with a custom protocol to send the messages with.


Added extra info about some SSHFS issues and security concerns.

Document a couple of issues I ran into whilest trying to get SSHFS Datastores working.

thank you!


Update faq-azure-hybrid-benefit-scope.yml

#sign-off @bandersmsft

@bandersmsft to help

#sign-off

@pri-mittal I signed off on this. It should get merged and republished live later this afternoon or tomorrow morning.


Configuration

Hi! Thanks for this! I was wondering if there is any configuration. I would like to display the file name instead of tracks, and disable the idle feature. Thank you!

These are not configurable, but will happily add them, thanks for asking

Feature Request to display the duration of how long you've been inside Vegas Pro. Similar to how with regular games on discord it would say "Playing for 'x' amount of hours"

Feature Request to display the duration of how long you've been inside Vegas Pro. Similar to how with regular games on discord it would say "Playing for 'x' amount of hours"

I'd love to see that ! Also maybe a translation feature ?

Considering about the translation bit. I need to find an effective way to detect the VEGAS language and choose an appropiate string.

In the coming weeks I'll be implementing a config file to keep track of these settings. Sorry for the wait!

good news guys @infini667 @LoganTann @ItsNifer i have implemented some of these suggestions. they are now in the 1.3 release
thanks for using the extension! and sorry for the late, got other things in my way


Rotation Listener not working

Implemented RotationGestureDetector.RotationListener, but the onRotate is not getting called.
Other challenge is to rotate the compass according to map rotaion..any idea how to do it?

Interesting. Never tried to use it.  I've always used the map events listener then checked the rotation then.  See the heading, compass up sample

Looking at the code, the RotationGestureOverlay sends the "hey someone touched the screen" message to the RotationGestureDetector, which calculates the rotation angle, then sets the map orientation.

For the compass overlay, override drawCompass and figure out how much of a rotation to give the compass. Grab the code from SampleHeadingCompassUp ~line 210 for the calculation

closing due to lack of response, reopen if needed. javadocs updated


finish schema_inferer

This change is

@ilovesoup  only for review. Not ready yet, still working on it.

SchemaInfer's code is too ugly. More refactoring on the way.

Column ref for group by is ok but need expression support for group by in coprocessor


IPROTO-233 Fix MarshallerByteCodeGenerator on JDK 8 and 9+

https://issues.redhat.com/browse/IPROTO-233
Supersedes https://github.com/infinispan/protostream/pull/157
Use mvn clean install -Dorg.infinispan.protostream.skipAnnotationCompilerCheck to build on JDK 8.

Merged! thanks @tristantarrant !
Tested locally with JDK 8, 11 and 17.