ReferenceError: localStorage is not defined

Newest version.
"vite-plugin-vue-devtools": "^7.4.6",
npm run dev

><EMAIL_ADDRESS>dev
> vite --host <IP_ADDRESS>

failed to load config from /Users/dennis/Projects/nq-client/vite.config.js
error when starting dev server:
ReferenceError: localStorage is not defined
    at getTimelineLayersStateFromStorage (file:///Users/dennis/Projects/nq-client/node_modules/@vue/devtools-kit/dist/index.js:2266:17)
    at initStateFactory (file:///Users/dennis/Projects/nq-client/node_modules/@vue/devtools-kit/dist/index.js:2546:26)
    at file:///Users/dennis/Projects/nq-client/node_modules/@vue/devtools-kit/dist/index.js:2550:68
    at ModuleJob.run (node:internal/modules/esm/module_job:262:25)
    at async onImport.tracePromise.__proto__ (node:internal/modules/esm/loader:483:26)
    at async loadConfigFromBundledFile (file:///Users/dennis/Projects/nq-client/node_modules/vite/dist/node/chunks/dep-Cyk9bIUq.js:66680:15)
    at async loadConfigFromFile (file:///Users/dennis/Projects/nq-client/node_modules/vite/dist/node/chunks/dep-Cyk9bIUq.js:66521:24)
    at async resolveConfig (file:///Users/dennis/Projects/nq-client/node_modules/vite/dist/node/chunks/dep-Cyk9bIUq.js:66129:24)
    at async _createServer (file:///Users/dennis/Projects/nq-client/node_modules/vite/dist/node/chunks/dep-Cyk9bIUq.js:62747:18)
    at async CAC.<anonymous> (file:///Users/dennis/Projects/nq-client/node_modules/vite/dist/node/cli.js:735:20)

I've try to fix it in v7.5.2. But I am so curious why this error occurs in the browser environment, can you try to provide minimal repo or share more info? thank you

getting the same issue on vite build :
><EMAIL_ADDRESS>build
> vite build

failed to load config from /home/runner/work/website/website/vite.config.js
error during build:
ReferenceError: localStorage is not defined
    at getTimelineLayersStateFromStorage (file:///home/runner/work/website/website/node_modules/@vue/devtools-kit/dist/index.js:22[6](https://github.com/EDM115/website/actions/runs/11366951788/job/31618497963?pr=150#step:5:7)6:17)
    at initStateFactory (file:///home/runner/work/website/website/node_modules/@vue/devtools-kit/dist/index.js:2546:26)
    at file:///home/runner/work/website/website/node_modules/@vue/devtools-kit/dist/index.js:2550:6[8](https://github.com/EDM115/website/actions/runs/11366951788/job/31618497963?pr=150#step:5:9)
    at ModuleJob.run (node:internal/modules/esm/module_job:262:25)
    at async onImport.tracePromise.__proto__ (node:internal/modules/esm/loader:483:26)
    at async loadConfigFromBundledFile (file:///home/runner/work/website/website/node_modules/vite/dist/node/chunks/dep-Cyk[9](https://github.com/EDM115/website/actions/runs/11366951788/job/31618497963?pr=150#step:5:10)bIUq.js:66680:15)
    at async loadConfigFromFile (file:///home/runner/work/website/website/node_modules/vite/dist/node/chunks/dep-Cyk9bIUq.js:66521:24)
    at async resolveConfig (file:///home/runner/work/website/website/node_modules/vite/dist/node/chunks/dep-Cyk9bIUq.js:66[12](https://github.com/EDM115/website/actions/runs/11366951788/job/31618497963?pr=150#step:5:13)9:24)
    at async build (file:///home/runner/work/website/website/node_modules/vite/dist/node/chunks/dep-Cyk9bIUq.js:65226:18)
    at async CAC.<anonymous> (file:///home/runner/work/website/website/node_modules/vite/dist/node/cli.js:828:5)
Error: Process completed with exit code 1.

getting the same issue on vite build :
><EMAIL_ADDRESS>build
> vite build

failed to load config from /home/runner/work/website/website/vite.config.js
error during build:
ReferenceError: localStorage is not defined
    at getTimelineLayersStateFromStorage (file:///home/runner/work/website/website/node_modules/@vue/devtools-kit/dist/index.js:22[6](https://github.com/EDM115/website/actions/runs/11366951788/job/31618497963?pr=150#step:5:7)6:17)
    at initStateFactory (file:///home/runner/work/website/website/node_modules/@vue/devtools-kit/dist/index.js:2546:26)
    at file:///home/runner/work/website/website/node_modules/@vue/devtools-kit/dist/index.js:2550:6[8](https://github.com/EDM115/website/actions/runs/11366951788/job/31618497963?pr=150#step:5:9)
    at ModuleJob.run (node:internal/modules/esm/module_job:262:25)
    at async onImport.tracePromise.__proto__ (node:internal/modules/esm/loader:483:26)
    at async loadConfigFromBundledFile (file:///home/runner/work/website/website/node_modules/vite/dist/node/chunks/dep-Cyk[9](https://github.com/EDM115/website/actions/runs/11366951788/job/31618497963?pr=150#step:5:10)bIUq.js:66680:15)
    at async loadConfigFromFile (file:///home/runner/work/website/website/node_modules/vite/dist/node/chunks/dep-Cyk9bIUq.js:66521:24)
    at async resolveConfig (file:///home/runner/work/website/website/node_modules/vite/dist/node/chunks/dep-Cyk9bIUq.js:66[12](https://github.com/EDM115/website/actions/runs/11366951788/job/31618497963?pr=150#step:5:13)9:24)
    at async build (file:///home/runner/work/website/website/node_modules/vite/dist/node/chunks/dep-Cyk9bIUq.js:65226:18)
    at async CAC.<anonymous> (file:///home/runner/work/website/website/node_modules/vite/dist/node/cli.js:828:5)
Error: Process completed with exit code 1.

The v7.5.2 still meeting the issue?

Can confirm it's been fixed. Cheers!

@webfansplz  Can we re-open this issue? The newly added undefined check doesn't work for WebViews who have localStorage disabled. When localStorage is disabled, then the value of localStorage is null, not undefined.
This breaks any application using vee-validate npm package since devtools for some reason is always embedded in the production build.
Or do i need to open a new issue?

@webfansplz Can we re-open this issue? The newly added undefined check doesn't work for WebViews who have localStorage disabled. When localStorage is disabled, then the value of localStorage is null, not undefined.
This breaks any application using vee-validate npm package since devtools for some reason is always embedded in the production build.
Or do i need to open a new issue?

Thanks for the feedback, try to fix it in the v7.5.4 (vite plugin) and browser extension v7.0.0-beta.12 (pending review). Try it again.

@webfansplz Wow you're fast 🚄 ! Faster than our review and QA pipeline for getting a rollback out.
I've updated the latest version and can confirm that it not works when WebViews on Android has their localStorage disabled. Awesome!
Virtual highfive from me 👍


add terraform test cases about volume and keypair

Changes

UPDATE terraform_install.sh to fix download latest version bug
UPDATE .gitignore to exclude all folders in terraform_test_artifacts/
NEW configure option terraform-provider-harvester to configure version of Harvester Terraform Provider
NEW binary tool json2hcl to convert json format into hcl (terraform) format
NEW fixture fixtures/terraform.py for terraform related test cases
NEW Test cases

test_create_ssh_key for legacy test_create_keypairs_using_terraform
test_delete_ssh_key
test_create_volume for legacy test_create_volume_using_terraform
test_delete_volume

Tested on harvester-runtests#407
The filename test_z is necessary, we would need to let terraform related test cases be executed in the last.


Fix npm link usage on Node.js 6

Fixes #814, but causes all sorts of test breakage.
I can't really figure out why it breaks the tests, it seems to work fine IRL. My only guess is that being wrapped by tap is somehow causing the problem.

Tests fixed

What if we rewrite the require('ava') call in the workers to be a real path?
I'm not sure about how often Node.js releases occur, do we really need to land this? I doubt we'd want to maintain this for the early Node 6 releases. On the other hand it'll help us during development, so maybe that's OK.

My thought is to merge and track Node 6 progress. Once the old behavior is restored and stable, revert this out.
The code I've copied here out of child_process has remained pretty stable, so I don't think we're in danger of missing out on any updates in the core module.
Still, I agree, this isn't something I want to maintain long term, hence all the extra comments letting us know what and when to revert.
I don't want to wait for Node to sort this out before I can use npm link again.

👍

When running this fix on our test suite, nyc drops reported coverage from 35.95% (on v0.14.0) to 1.82% (running HEAD from master).
Not sure if you were aware of this but you may want to delay publishing to NPM anytime soon.

@dcousineau - Good Catch!
#827 should fix your issue. NYC still will not work if you are using npm link ava on Node 6. But it will always work on all earlier versions of Node, and will also work on Node 6 as long as you haven't npm linked AVA.

FYI: The NYC issue is because spawn-wrap does not support the -e flag.
I have opened https://github.com/tapjs/spawn-wrap/issues/22, requesting that support for -e be added, but #827 works to solve this issue.

Makes sense. I'm not using npm link anywhere becuase I keep forgetting its a thing :P


Using JS SDK to get Managed Identity token doesn't work in Azure Container Apps environment

We are moving our nodejs project from AKS to ACA. The same container can work in AKS and get a token using DefaultAzureCredential successfully and then get a secret from keyvault. But in ACA, it just doesn't work. Here is the error log in console:
CredentialUnavailableError: EnvironmentCredential is unavailable. No underlying credential could be used. To troubleshoot, visit https://aka.ms/azsdk/js/identity/environmentcredential/troubleshoot.
CredentialUnavailableError: Ye: The managed identity endpoint is indicating there's no available identity. Message: undefined Status code: 400
More details:
undefined
CredentialUnavailableError: Azure CLI could not be found. Please visit https://aka.ms/azure-cli for installation instructions and then, once installed, authenticate to your Azure account using 'az login'.
CredentialUnavailableError: Error: Unable to execute PowerShell. Ensure that it is installed in your system. To troubleshoot, visit https://aka.ms/azsdk/js/identity/powershellcredential/troubleshoot.
And we are using the latest packages:
"@azure/identity": "^3.1.3",
"@azure/keyvault-secrets": "^4.6.0",
Any idea why it doesn't work in Azure Container Apps? Is the SDK still not updated to support ACA or it's ACA's issue?

OK, setting environment variable AZURE_CLIENT_ID will make it work. This was also not set in AKS environment though.

@KarishmaGhiya I'm not familiar with the default configuration of AKS vs ACA, is there anything we should be doing differently?

@arisewanggithub For ACA, you can check what env variable is present. I don't think we have explicitly written a code in managed identity to support ACA, but if the Managed Identity works, it might be picking one of the following environments -

Azure Kubernetes
Azure VM, or
IMDS endpoint
But if the client id is not present then, you can either pass it as an argument to Managed Identity constructor OR set the env variable yourself.

To verify what managed identity is really being used underneath, you could enable logging in your code and we can check.

@KarishmaGhiya
Here are the env variables after I add AZURE_CLIENT_ID. I've changed sensitive data to xxx:
KUBERNETES_SERVICE_PORT_HTTPS=443
KUBERNETES_SERVICE_PORT=443
RAZZLE_PUBLIC_DIR=/srv/build
HOSTNAME=xxx
SERVER_ROOT=/srv
MSI_SECRET=xxx
PWD=/var/srv
PORT=80
NODE_ENV=production
HOME=/root
KUBERNETES_PORT_443_TCP=tcp://<IP_ADDRESS>:443
IDENTITY_HEADER=xxx
IDENTITY_ENDPOINT=http://localhost:42356/msi/token
AZURE_LOG_LEVEL=info
TERM=xterm
NPM_CONFIG_LOGLEVEL=notice
CONTAINER_APP_HOSTNAME=xxx
CONTAINER_APP_NAME=xxx
SHLVL=1
KUBERNETES_PORT_443_TCP_PROTO=tcp
AZURE_CLIENT_ID=xxx
KUBERNETES_PORT_443_TCP_ADDR=<IP_ADDRESS>
Environment=int
KUBERNETES_SERVICE_HOST=<IP_ADDRESS>
KUBERNETES_PORT=tcp://<IP_ADDRESS>:443
CONTAINER_APP_ENV_DNS_SUFFIX=xxx
KUBERNETES_PORT_443_TCP_PORT=443
CONTAINER_APP_PORT=80
PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
CONTAINER_APP_REVISION=xxx
MSI_ENDPOINT=http://localhost:42356/msi/token
_=/usr/bin/env
And when I add AZURE_CLIENT_ID env variable, it starts working. By setting AZURE_LOG_LEVEL to info, I can see the following log:
azure:identity:info ManagedIdentityCredential => SetAppTokenProvider invoked with parameters- {"correlationId":"noCorrelationId","tenantId":"common","scopes":["https://vault.azure.net/.default"]}
azure:identity:info ManagedIdentityCredential - Azure Arc MSI => ManagedIdentityCredential - Azure Arc MSI: The environment variables needed are: IMDS_ENDPOINT and IDENTITY_ENDPOINT
azure:identity:info ManagedIdentityCredential - Fabric MSI => ManagedIdentityCredential - Fabric MSI: Unavailable. The environment variables needed are: IDENTITY_ENDPOINT, IDENTITY_HEADER and IDENTITY_SERVER_THUMBPRINT
azure:identity:info ManagedIdentityCredential - AppServiceMSI 2019 => ManagedIdentityCredential - AppServiceMSI 2019: Using the endpoint and the secret coming form the environment variables: IDENTITY_ENDPOINT=http://localhost:42356/msi/token and IDENTITY_HEADER=[REDACTED].
azure:identity:info IdentityClient: sending token request to [http://localhost:42356/msi/token?resource=https%3A%2F%2Fvault.azure.net&api-version=2019-08-01&client_id=xxx]
azure:core-rest-pipeline retryPolicy:info Retry 0: Attempting to send request 45b5bc0f-cdf6-4695-843e-68321d2c60ba
azure:core-rest-pipeline:info Request: {
"url": "http://localhost:42356/msi/token?resource=REDACTED&api-version=2019-08-01&client_id=REDACTED",
"headers": {
"accept": "application/json",
"x-identity-header": "REDACTED",
"accept-encoding": "gzip,deflate",
"user-agent": "azsdk-js-identity/3.1.2 core-rest-pipeline/1.5.0 Node/v16.18.1 OS/(x64-Linux-5.4.0-1103-azure)",
"x-ms-client-request-id": "45b5bc0f-cdf6-4695-843e-68321d2c60ba"
},
"method": "GET",
"timeout": 0,
"disableKeepAlive": false,
"withCredentials": false,
"requestId": "45b5bc0f-cdf6-4695-843e-68321d2c60ba",
"allowInsecureConnection": true,
"enableBrowserStreams": false
}
azure:identity:info IdentityClient: [http://localhost:42356/msi/token?resource=https%3A%2F%2Fvault.azure.net&api-version=2019-08-01&client_id=xxx] token acquired, expires on<PHONE_NUMBER>000

@arisewanggithub  That looks good. We also have the parametrized constructor overload for DefaultAzureCredential (and ManagedIdentity) where you can pass in the clientId value through the constructor parameters as well, since it's not automatically set up through the ACA. Apart from that, did you have any other concerns?


Per-partition information in dask.dataframe.DataFrame.memory_usage

We currently provide a per-column information in a pandas.Series when calling dask.dataframe.DataFrame.memory_usage Series. It would also be nice to to have that information on a per-partition-per-column basis. This would allow easier analysis in the case of out-of-memory errors.
I could open a PR that returns this information via a parameter but this would then also change the return-type to a pandas.DataFrame. Thus I'm unsure of the right API to implement:

dd.DataFrame.memory_usage(per_partition=True) -> pandas.DataFrame
dd.DataFrame.memory_usage_detailed() -> pandas.DataFrame.

Any opinions before I start a PR?

My initial preference is for a separate method (memory_usage_detailed or partitionwise_memory_usage sound fine), but I think a keyword is also reasonable.

I wonder if it is possible to just use df.map_partitions(pd.DataFrame.memory_usage) here.  If this doesn't work, why doesn't it work, and is there some way that we can improve it?

df.map_partitions(pd.DataFrame.memory_usage) seems to work for determining per-partition memory usage:
In [1]: import pandas as pd

In [2]: import dask.dataframe as dd

In [3]: df = pd.DataFrame({"a": [1, 2, 3, 4], "b": [5, 6, 7, 8]})

In [4]: ddf = dd.from_pandas(df, npartitions=3)

In [5]: ddf.map_partitions(pd.DataFrame.memory_usage)
Out[5]:
Dask Series Structure:
npartitions=2
0    int64
2      ...
3      ...
dtype: int64
Dask Name: memory_usage, 4 tasks

In [6]: ddf.map_partitions(pd.DataFrame.memory_usage).compute()
Out[6]:
Index    128
a         16
b         16
Index    132
a         16
b         16
dtype: int64

@xhochy does that contain the information you're after?

I want to have information in there that identifies the partition, so we can also group by partition to get the total partition sizes.

Gotcha, I think an additional map_partitions(pd.Series.sum) will result in the total memory for each partition
import pandas as pd
import dask.dataframe as dd

df = pd.DataFrame({"a": [1, 2, 3, 4], "b": [5, 6, 7, 8]})
ddf = dd.from_pandas(df, npartitions=2)
mem_usage = ddf.map_partitions(pd.DataFrame.memory_usage)
mem_per_partition = mem_usage.map_partitions(pd.Series.sum)
print("mem_per_partition.compute():")
print(mem_per_partition.compute())

outputs
mem_per_partition.compute():
0    160
1    164
dtype: int64

Yes, that's the expected result. I would really like to have this as a single API call as a I often get problems reports from colleagues where their large dask graph fails with KilledWorker due to uneven partition sizes. This would be a good routine for check this.

fwiw, this comes up fairly frequently in debugging for problems I come across

Echoing Tom's previous comment, I think a separate memory_usage_per_partition or memory_usage_partitionwise method would be a nice addition


Auto complete for field is incorrect

When you try and auto complete an attribute of a field, you get:
relationship.flow.data.id[RESOURCE_ID:flows]
It should be
relationship.flow.data.id

Lol it should be relationships for both the flow.data.id and flow.data.type.


Heap Usage

Excuse me, what are the meanings of Heap Usage and non-heap Usage in inspector?What does it stand for?

@tonfay
Please, place your mouse over the question mark at top right of each graph

@tonfay
Please, place your mouse over the question mark at top right of each graph

hello!
Can I understand the Heap Usage as the Usage of the Heap (young/old) and the non-heap Usage as perm?

@tonfay
Sometimes it can be very easy if you look at the source code :)
Heap is from
https://docs.oracle.com/javase/7/docs/api/java/lang/management/MemoryMXBean.html#getHeapMemoryUsage()
Non-Heap is from
https://docs.oracle.com/javase/7/docs/api/java/lang/management/MemoryMXBean.html#getNonHeapMemoryUsage()

@tonfay
Sometimes it can be very easy if you look at the source code :)
Heap is from
https://docs.oracle.com/javase/7/docs/api/java/lang/management/MemoryMXBean.html#getHeapMemoryUsage()
Non-Heap is from
https://docs.oracle.com/javase/7/docs/api/java/lang/management/MemoryMXBean.html#getNonHeapMemoryUsage()

tks very much!


Using BackSlashSolver as default coarsest grid solver in serial computations is dangerous

The GMG preconditioner constructor uses Gridap.Algebra.BackslashSolver() as the default solver for the coarsest grid problem. In particular, in the following line:
https://github.com/gridap/GridapSolvers.jl/blob/47955a82d55958cc0585780325c005db3755efe2/src/LinearSolvers/GMGLinearSolvers.jl#L18
I think this is dangerous from the performance point of view as Gridap.Algebra.BackslashSolver() is implemented such that for each call to solve!(...) it computes the full LU factorization.
I would use LUSolver() instead, which performs the factorization once, and uses backward/forward substitution within each call to solve!(...). However, when leveraging LUSolver() in a single process, I have realized that the following line:
https://github.com/gridap/GridapSolvers.jl/blob/47955a82d55958cc0585780325c005db3755efe2/src/LinearSolvers/GMGLinearSolvers.jl#L189
triggers an error, as xh and rh are of type SubVector, and solve!(...) is not overriden for that particular combination of types. The work-around around this is to define an additional method specialization:
function LinearAlgebra.ldiv!(Y::SubArray,
                             A::LinearAlgebra.LU,
                             B::SubArray)
  LinearAlgebra.ldiv!(collect(Y),A,collect(B))
end

However, I am not sure if this is the best way to go. In particular, I wonder whether there is a more efficient way of dealing with the transformation of SubArray to Array. Two possible ideas: (1) completely avoid the copy if the SubArray  is consecutive. (2) if it is not consecutive, perhaps to have arrays in a cache such that we reduce overhead associated to dynamic memory allocation.
On the other hand, another consideration is that BackSlashSolver() is implemented in GridapDistributed such that it is able to handle distributed data objects for the case in which the coarsest grid problem is distributed among processes. I guess this is not the case for LUSolver(). I did not check it.

Mhm, I think we can get around SubVector:
Right now, the SubVector typing appears because of
map_parts(xh.owned_values,rh.owned_values) do xh, rh
       solve!(xh,caches,rh)
end

but I believe that we can actually use
map_parts(xh.values,rh.values) do xh, rh
       solve!(xh,caches,rh)
end

since there are no ghost values sequential. That should get rid of the weird typing.
What do you think @amartinhuertas ?

What do you think @amartinhuertas ?

Yes, I think that would solve one of the two issues. I will do it.
The second issue I guess should be easy to be solved (if it is not solved already, I have to check, perhaps it is not an issue at all)


pcfdev-v0.447.0: rror: failed to provision VM: Process exited with: 1. Reason was:  ()

Hi,
While starting pcfdev using cf dev start -s all, throwing below error.
'cf dev start' this commands starts without issues. I want to test spring cloud config services for that I started with cf dev start -s all, its failing. I also tested with cf dev start -s scs, same results (error).
My system details: Windows 10, 16GB RAM
PS C:> cf dev start -s all
Using existing image.
Allocating 8132 MB out of 16264 MB total system memory (13841 MB free).
Importing VM...
Starting VM...
Provisioning VM...
Waiting for services to start...
8 out of 57 running
8 out of 57 running
8 out of 57 running
35 out of 57 running
57 out of 57 running
FAILED
Error: failed to provision VM: Process exited with: 1. Reason was:  ().

PCF Dev: pcfdev-v0.447.0
CF CLI: cf-cli-installer 6.23.1
Virtual box : VirtualBox-5.1.18-114002-Win
Thanks a lot for your help.
Regards,
Giri

I'm closing this issue, after setting 12GB its working. I think this need to updated some where in docs, min ram need to be set as 12gb otherwise it will not work.

Thanks for the report. Many factors play into whether your system is performant enough to start PCF Dev with SCS successfully. Usually 8GB of RAM is sufficient, but I'll consider have PCF Dev allocate up to 10GB when more than 8GB is available. Apologies for the poor logging when starting the VM fails in this manner.

Stephen, first I would like to thank you. I saw your suggestions on many issues then I went on checking provision.log, first time I set 8GB it failed in loading 1 buildpack, then I increased to 10GB I saw that buildpack loaded next it failed on 2nd buildpack, increased to 12gb then all buildpacks loaded and issue resolved.
I have one question, for my requirement I dont want all the buildpacks for now. To improve is there a way  I can disable for buildpack while PROVISIONING VM? I know command to delete buildpack, but this is after starting of cf, and its provisioned CF with 12gb.

We don't have support for that functionality, and I wouldn't expect the availability of the buildpacks to make a noticeable difference in performance. Are you able to replicate these failures?
We will definitely review the performance requirements of PCF Dev running SCS before the next release.


Eeror in runing sample project after add darragable panel as lib project

i am runing the code in eclipse and add all library needed but this error keep crashing my app , in Read Me itis supposed to build with gradle would you please help me how to run this app in gradle may be this is my problem :(
java.lang.RuntimeException: Unable to create application com.github.pedrovgs.sample.DraggablePanelApplication: java.lang.IllegalStateException: Module adapter for class com.github.pedrovgs.sample.di.MainModule could not be loaded. Please ensure that code generation was run for this module.

You have the library artifact uploaded to maven central. http://search.maven.org/#artifactdetails|com.github.pedrovgs|draggablepanel|1.7|
I'm not going to support eclipse for this project. Sorry.

./gradlew build


Measure push performance

The bottleneck of push is nix copy, https://github.com/NixOS/nix/issues/2559.


@tailwind

@tailwind is an unknown "at" rule.
@tailwind base;
@tailwind components;
@tailwind utilities;
All have error code as "unknown 'at' (@) rule".
!How can Visual Studio Code not recognize "@tailwind" but has a working CSS Tailwind IntelliSense??
A lot of people are online with the same problem. My question is how does Adam Wathan's tutorial on setting up Tailwind CSS NOT have the same problem? He types it in on his video and works great. Hearing from anyone would be great. I really don't think it's just me. Thanks to all!

@tailwind is an unknown "at" rule.
@tailwind base;
@tailwind components;
@tailwind utilities;
All have error code as "unknown 'at' (@) rule".
!How can Visual Studio Code not recognize "@tailwind" but has a working CSS Tailwind IntelliSense??
A lot of people are online with the same problem. My question is how does Adam Wathan's tutorial on setting up Tailwind CSS NOT have the same problem? He types it in on his video and works great. Hearing from anyone would be great. I really don't think it's just me. Thanks to all!

i had the same issue. Check this link out.
https://stackoverflow.com/questions/47607602/how-to-add-a-tailwind-css-rule-to-css-checker


get_weights_topo on fully conencted layer after convolutional layer returns the wrong space

For example, if the input_space to the layer is 8(filters) x 4 (w) x 4 (h) and the output space is 128, get_weights_topo should return 128 x 4 x 4 x 8 (given the default axes b, 0, 1, c). However, it returns 128 x 4 x 8 x 4. This is because the code internally already transposes the weight to adhere to b, 0, 1, c, but then again transposes them assuming it to be b, c, 0, 1. The wrong code can be found in mlp.py, in the Linear-class.

Thanks for the report!
That is right, SoftmaxPool and Linear reshape W.T assuming the input of the dot product is b01c (which it currently is, since that is the default axes of Conv2DSpace), but then transposes it assuming the axes were the input space.
Softmax however calls desired_space.format_as(W.T, input_space) instead of reshaping, then transposes the axes to b01c, which gives the right result.
Given that get_weight_topo should return a tensor with axes b01c, I think the more future-proof approach would be to follow the implementation of Softmax.
A simpler implementation would be to make sure that the default axes of Conv2DSpace are still b01c, keep the explicit reshape, and drop the existing transpose.


Incorrect cost quoted when creating a Safe

Title/Description
Incorrect cost quoted for Safe creation
Environment

Browser: N/A
Safe: Gnosis Safe Multisig
Wallet: Ledger Nano S
App Version: 2.11.1
Environment: Win10

production (mainnet)

Steps to reproduce

Create a Safe (Fill in name, owners)
Review

Issue exists in other environments, too (e.g. mainnet, web)
Expected result
Estimated creation costs is correct.

Note 1: If this cannot be fixed, then let's just remove that sentence ("The creation will cost ...") since it's more confusing than anything.
Note 2: We have a similar sentence/estimation when confirming/executing txs. We should check if the same issue exists there and apply the same fix.

Obtained result
The on-screen text says
You're about to create a new Safe and will have to confirm a transaction with your currently connected wallet. The creation will cost approximately < 0.001 ETH. The exact amount will be determined by your wallet.
that is an incorrect cost (at ~US$0.35).  The cost the Ledger reported was closer to US$11, which is what I would expect.
Updates

Oct 19, Tobi: Add expected result and steps

As fas as I understood when implementing this issue, the second part (estimation when confirming/executing) it's not possible to display the exact amount on metamask/ledger, because we are paying some gas to execute that transaction to a proxy and that it's what metamask will see, but the safe will need to execute another contract like multisend and will use some extra gas that's different from what metamask can see. Correct me @fernandomg if I understood something else

Mhh... the issue here is about the estimated gas that is displayed in the Safe's modal.
That's something that may have been fixed in #1523, for all networks?
@Agupane, have you tried to track the estimated tx cost? (I mean, by looking at the estimate in the safe and the actual cost in the explorer once the tx has been mined?)
Another thing, that value that we're displaying is based on the estimated safeTxGas only. Probably we should add an extra amount of gas for the cost of the executeTransaction call. I'm not sure about the latter.

This seems to be fixed now.

If this ticket refers just to the estimation in the message before creating the safe then yes, that issue is fixed and the MM popUp will have the correct amount of tokens as well.
If it refers to the actual cost and the estimation not being that accurate then yes, chances are the cost is going to be lower at the end of tx execution, but I don't think we can lower that estimation more without having a good chance of failing the tx.


Explorer tree after update disappears

Preflight Checklist

[X] I have installed the latest version of Storage Explorer.
[X] I have checked existing resources, including the troubleshooting guide and the release notes.
[X] I have searched for similar issues.

Storage Explorer Version
1.27.0
Regression From
No response
Architecture
x64
Storage Explorer Build Number
20221201.1
Platform
Windows
OS Version
Windows 10
Bug Description
after i opened tool after new update all my connections are gone
Steps to Reproduce
Launch
.. no connections are present  although I had a ton before
Actual Experience
Connections disappeared
Expected Experience
Connections would still be there
Additional Context
No response

@vmachacek

What exactly do you experience when you say your connections have disappeared? Can you share any screenshots of the issue?
Have you tried opening Storage Explorer a second time to see if they are still absent?
Can you follow the steps up to step 3 as shown in the troubleshooting guide for connection issues? Can you share the data you see there?
Do your connections come back when reverting to the previous version of Storage Explorer?

Possibly related to #6355.

Sorry for misleading title, what happened was when I click the first tab on the left menu which has a icon of tree structure

the panel right next to it was empty - which is listing storage accounts? I called that connections previously.
When that happened I just added the ones I needed for the current task
When I opened the Storage Explorer second time the state was same as when I exited last time

@vmachacek Sorry, I think it's still a little unclear what problem you're experiencing.
You can click the explorer tree button to toggle the panel on and off:

Are you saying that the entire panel doesn't show when it's toggled on? Or is the panel visible, but there's nothing in it? Or do you see the search bar and links, but there are no tree nodes it it?
Also, when you say you "added the ones I needed for the current task", what do you mean? Are you saying that you attached to a storage account? A specific resource?

the panel shows fine, the problem it was empty and I had to manually add the storage accounts

@vmachacek Do you normally sign in with your Azure account? If so, did you try signing in again? Or do you normally attach to individual accounts (manually add accounts, as you say)?

@vmachacek We understand. Our follow-up questions are just us trying to clarify the details, get more information, and see if they are related to other issues we are aware of. Did all of your accounts disappear? Or just some of them?
Just so you know, we are aware of two issues which may both be the same thing you're experiencing. We are releasing a new version very soon that should fix these issues (see #6360, #6364).

sweet, yes, all of the accounts disappeared

@vmachacek We have released 1.27.1. Can you install that version and see whether your issue is resolved?

@vmachacek Did you install 1.27.1. Does your issue persist?

I did, and nothing bad happened 👍


Movie details page and search broken

I was looking for inspiration and I came back to the site and the search and movie details page are now broken.
Search loop infinitely:

Movie page doesn't load:

Same here, it can't open any of the movie pages, and they error out on page refresh.

I have same problem!
console :
TypeError: p is not a function at setup (id.fe0da9c9.js:1:8129)

This is a regression on Nitro. We are working on it

Same issue

Same issue at my end!

Update:
Search still broken.
Really hoping this can be fixed.

same issue

this can be closed

Search seems to work but the details page does not (at least for me). Opening the website results in a 404 error that can only be bypassed by pressing the button to go back to the Home page.


GROOVY-7474: Difference between junit3 and junit 4 shouldFail {...} not described in docs

added - thanks!


[WIP] Allow custom panic messages in assert_* macros

This is an attempt to resolve #44.

Would love to get this in, was just looking for this feature! Anything we're waiting on? @brendanzab?


Add feed for the Showcase

This would allow to post new showcase entries to Twitter or to a Slack channel :)

@budparr would be nice to have a Netlify's preview on PRs for this repo 👼

view-source:https://showcase-feed--the-new-dynamic.netlify.com/showcase/index.xml

Hey, @DirtyF I don't know how to turn the previews on. Seems like that's always been automatic. Do you know? At any rate, I'll merge this.

@budparr For deploy preview notifications you need to use a GitHub Token and add a webhook as explained here: https://www.netlify.com/docs/webhooks/#github-commit-statuses

Thank you. Should be set now. I have no idea why it wasn't set for this repo, as I think it is for all the others.


fix: get only fastq.gz not all gzipped

CI is broken because this is from a fork and the fork doesn't have access to secrets. Can you redo the PR with your write collaborator role?


Update soaper-dl.sh

Removes excess parameters to --data-raw which causes $_HOST to be removed, then readding it by overwriting $el to add $_HOST to its overall string.
Testing seems to only have this happen with movies and not shows, will PR again if I find evidence otherwise.
EDIT: I know this is a very sloppy and overall lazy way to fix it, but I'd rather someone not have to do it themselves. Maybe you could switch it to a cleaner approach.

Hey @blackopsoperative, thanks for this pull request. I don't see any benefit of causing host name removed in the response and then add it back. If the parameter can return host name, I think they are useful and not excess parameters :)

Hey @blackopsoperative, thanks for the details. I see, there must be some server side changes. Let me take a look.

Thanks @blackopsoperative! This pull request has been merged.


Develop

Please remember to follow the Lasair Pull Request checklist when issuing & reviewing this pull request:
https://lsst-uk.atlassian.net/wiki/spaces/LUSC/pages/2612264961/How+to+do+Pull+Request#STEP-3%3A-Issue-a-Pull-Request-on-Github
CHANGES

Incorporate latest changes for release 4.2

What I have done is mostly the documentation, and a bit of fink. OK with me to merge.

Lots of little fixes in the last week. Good to go. Merging now.


Hmb1 master2

HI Joe,
have merged in  lizaclark mods and fixed tests so all working now
Please merge in when you get a chance
...Henry

I'm unclear what's holding up this PR. Are the Travis failures fixable? If so, @hmb1 please fix them. If not, @jhamman would you please merge this?


:sparkles: Add settings file option to openrewrite

Maven settings.xml file can be now specified as parameter for kantra transform openrewrite command (that uses maven for its execution).
Fixes: https://github.com/konveyor/kantra/issues/19

@eemcmullan Updated usage in readme, ready for review.


Manual Testing

[x] Sample data > 25
[ ] Converted .adoc file

Coverage remained the same at 91.712% when pulling c693dc983125e9b91067abc2e59cfd30e64903e5 on ZY-Ang:ManualTesting into 1bf52aa305b69b6937ef1bc6f1fa2b5a4e5419c3 on CS2103AUG2017-W11-B2:master.

Coverage remained the same at 91.712% when pulling 177396f5ff4cc1e5271ebe3bdf1123d4afcb5c3a on ZY-Ang:ManualTesting into 1bf52aa305b69b6937ef1bc6f1fa2b5a4e5419c3 on CS2103AUG2017-W11-B2:master.

Coverage remained the same at 91.712% when pulling f0af5888d6da86731c6b1674e18dd7ea84f4b86e on ZY-Ang:ManualTesting into 1bf52aa305b69b6937ef1bc6f1fa2b5a4e5419c3 on CS2103AUG2017-W11-B2:master.

Coverage remained the same at 91.895% when pulling 39fa01f2a976d123037faa94ae84ac9afec458ad on ZY-Ang:ManualTesting into e8fca4b30d0f81adde60aac2c249a3bcb38a4b48 on CS2103AUG2017-W11-B2:master.


Optimization linq sql for post with extensions

Hi,
With my website on v<IP_ADDRESS>, I had created extensions on my post but I don't use permalink to call my contents.
On view which call my post data, the linq sql query is not optimize because I call all posts before I can filter the post I want. My view don't use postmodel. Is there an other solution to call only the post ID which I filter with an extension field.
First I filter my posts with template name and after extension field. I get all posts before I can filter the post. When my data increase the SQL query is slower.
This is my use code on post view
 using (var db = new Piranha.DataContext())     { events = db.Posts.Include(p => p.Categories).Include(p => p.Extensions).Where(p => **p.Template.Name == "spectacle"** && p.Categories.Where(c => c.Parent.Name == categorie_principale.ToUpper()).Count() > 0).ToList().Where(x => **x.Extensions.ByInternalId("Extension_Manifestations").GetBody<Extension_Manifestations>().ID_Catalog == Id_Catalog)**.OrderBy(x => x.Extensions.ByInternalId("Extension_Manifestations").GetBody<Extension_Manifestations>().Seances1.Min(s => s.SeanceDate)).ToList(); }
Thanks a lot for your help. This CMS is very good for my use. 👍

Like you say your query fetches all posts to memory and then filters them on the content of their deserialized extension values which of course isn't optimal. If you need to filter large sets of posts on extension values I would do it like this:

Create (if you don't already have) a custom DbContext.
Add a PostFilter entity where you store your "indexed" values and the post id.
Add a hook to Manager.PostEditModelAfterSave where you store the values and post id.

It could look something like this:
public MyContext : DbContext
{
    public DbSet<PostFilter> PostFilters { get; set; }
}

public class PostFilter
{
    public string Key { get; set; }
    public string Value { get; set; }
    public Guid PostId { get; set; }
}

// This should go in your Global.asax.cs where you init the Piranha app
Piranha.Hooks.Manager.PostEditModelAfterSave => (model)
{
    using (var db = new MyContext())
    {
        // Store any values from the model you need to search in the PostFilters
    }
}

With this approach you can do a fast query on your filter table and get the Id's of the matching posts. After that you can get all of the post matching the result of your query.
Please note that I haven't tested the code above, so please see it as suggestion for how it could be implemented.
Regards

Hi,
Thanks for these informations. But I’m not sure to understand when you said in step 3 to store any values from the model you need to search in the PostFilters. I have capture my post form for my custom ID (ID AparetWeb) which I want to filter.
// This should go in your Global.asax.cs where you init the Piranha app
Piranha.Hooks.Manager.PostEditModelAfterSave => (model)
{
using (var db = new MyContext())

{

    // Store any values from the model you need to search in the PostFilters

}

}
Thank you so much for your help.
<EMAIL_ADDRESS>[go re]http://www.monticket.re/[go mu]http://www.monticket.mu/
Laurent FONTAINE
Directeur d’exploitation
Co-Fondateur
[fice]<file:///D:/web/MONTICKET/html/signature-mail/signature-model/signature-SA-model-04.htm>0692 88 23 24
[ail]mailto:sebastien<EMAIL_ADDRESS>[bsite]http://www.monticket.re/[cebook]https://www.facebook.com/MonTicket.re/[itter]https://twitter.com/monticketre[ogle+]https://plus.google.com/+MonticketRe/posts
[nticketkdo]http://www.monticket.re/monticketkdo-le-cheque-cadeau
De : Håkan Edling<EMAIL_ADDRESS>Répondre à : PiranhaCMS/Piranha<EMAIL_ADDRESS>Date : jeudi 22 mars 2018 à 01:05
À : PiranhaCMS/Piranha<EMAIL_ADDRESS>Cc : Laurent FONTAINE<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Objet : Re: [PiranhaCMS/Piranha] Optimization linq sql for post with extensions (#566)
Like you say your query fetches all posts to memory and then filters them on the content of their deserialized extension values which of course isn't optimal. If you need to filter large sets of posts on extension values I would do it like this:

Create (if you don't already have) a custom DbContext.
Add a PostFilter entity where you store your "indexed" values and the post id.
Add a hook to Manager.PostEditModelAfterSave where you store the values and post id.

It could look something like this:
public MyContext : DbContext
{
public DbSet<PostFilter> PostFilters { get; set; }

}
public class PostFilter
{
public string Key { get; set; }

public string Value { get; set; }

public Guid PostId { get; set; }

}
// This should go in your Global.asax.cs where you init the Piranha app
Piranha.Hooks.Manager.PostEditModelAfterSave => (model)
{
using (var db = new MyContext())

{

    // Store any values from the model you need to search in the PostFilters

}

}
With this approach you can do a fast query on your filter table and get the Id's of the matching posts. After that you can get all of the post matching the result of your query.
Please note that I haven't tested the code above, so please see it as suggestion for how it could be implemented.
Regards
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/PiranhaCMS/Piranha/issues/566#issuecomment-375095229, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AHnoHRtT0LMWMTCCNbMgrBciG-whtFnGks5tgsCBgaJpZM4Su5_k.

Hi Hakan,
Could you help me about a cache problem on Piranha v2 ? In my home page for https://www.monticket.re I put this :
Response.Cache.SetExpires(DateTime.Now.AddMinutes(120.0));
Response.Cache.SetCacheability(HttpCacheability.ServerAndPrivate);
Response.Cache.SetMaxAge(new TimeSpan(0,120,0));
Response.Cache.SetValidUntilExpires(true);

But the page is keep in navigator cache for more long time, if I’m not force refresh in navigator. It’s only for my HomePage.
Have you an idea ?
Thanks a lot !
[signature_792767066]
[go re]http://www.monticket.re/[go mu]http://www.monticket.mu/
Laurent FONTAINE
Directeur d’exploitation
Co-Fondateur
[fice]<file:///D:/web/MONTICKET/html/signature-mail/signature-model/signature-SA-model-04.htm>0692 88 23 24
[ail]mailto:sebastien<EMAIL_ADDRESS>[bsite]http://www.monticket.re/[cebook]https://www.facebook.com/MonTicket.re/[itter]https://twitter.com/monticketre[ogle+]https://plus.google.com/+MonticketRe/posts
[nticketkdo]http://www.monticket.re/monticketkdo-le-cheque-cadeau

Hi there! If client cache is enabled for pages & posts it is set by Piranha here:
https://github.com/PiranhaCMS/Piranha/blob/master/Core/Piranha/Web/ClientCache.cs#L63 https://github.com/PiranhaCMS/Piranha/blob/master/Core/Piranha/Web/ClientCache.cs#L63
What values have you entered for the other pages/posts in the site, and is the caching correct for them?
Regards
Håkan Edling / @tidyui https://twitter.com/tidyui, piranhacms.org http://piranhacms.org/

25 feb. 2019 kl. 14:51 skrev lfontaine974<EMAIL_ADDRESS>Hi Hakan,
Could you help me about a cache problem on Piranha v2 ? In my home page for https://www.monticket.re I put this :
Response.Cache.SetExpires(DateTime.Now.AddMinutes(120.0));
Response.Cache.SetCacheability(HttpCacheability.ServerAndPrivate);
Response.Cache.SetMaxAge(new TimeSpan(0,120,0));
Response.Cache.SetValidUntilExpires(true);
But the page is keep in navigator cache for more long time, if I’m not force refresh in navigator. It’s only for my HomePage.
Have you an idea ?
Thanks a lot !
[signature_792767066]
[go re]http://www.monticket.re/[go mu]http://www.monticket.mu/
Laurent FONTAINE
Directeur d’exploitation
Co-Fondateur
[fice]<file:///D:/web/MONTICKET/html/signature-mail/signature-model/signature-SA-model-04.htm>0692 88 23 24
[ail]mailto:sebastien<EMAIL_ADDRESS>[bsite]http://www.monticket.re/[cebook]https://www.facebook.com/MonTicket.re/[itter]https://twitter.com/monticketre[ogle+]https://plus.google.com/+MonticketRe/posts
[nticketkdo]http://www.monticket.re/monticketkdo-le-cheque-cadeau
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/PiranhaCMS/Piranha/issues/566#issuecomment-467018114, or mute the thread https://github.com/notifications/unsubscribe-auth/AAvr0o3GcyfQc_QbdxY3ujK_EZz9hCdSks5vQ-pWgaJpZM4Su5_k.

Oh sorry I have forget this settings !!! ☹
What is type of cache by default for Piranha ?
HttpCacheability.ServerAndPrivate or HttpCacheability.Server
If I put HttpCacheability.ServerAndPrivate, website is faster. Ca I use this HttpCacheability functionality for define cache type ?
Thanks for theses informations.
P.S : Piranha is a very good CMS !!! I’m having a lot of fun with it !!!
[signature_1122125573]
[go re]http://www.monticket.re/[go mu]http://www.monticket.mu/
Laurent FONTAINE
Directeur d’exploitation
Co-Fondateur
[fice]<file:///D:/web/MONTICKET/html/signature-mail/signature-model/signature-SA-model-04.htm>0692 88 23 24
[ail]mailto:sebastien<EMAIL_ADDRESS>[bsite]http://www.monticket.re/[cebook]https://www.facebook.com/MonTicket.re/[itter]https://twitter.com/monticketre[ogle+]https://plus.google.com/+MonticketRe/posts
[nticketkdo]http://www.monticket.re/monticketkdo-le-cheque-cadeau
De : Håkan Edling<EMAIL_ADDRESS>Répondre à : PiranhaCMS/Piranha<EMAIL_ADDRESS>Date : lundi 25 février 2019 à 18:04
À : PiranhaCMS/Piranha<EMAIL_ADDRESS>Cc : Laurent Fontaine<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Objet : Re: [PiranhaCMS/Piranha] Optimization linq sql for post with extensions (#566)
Hi there! If client cache is enabled for pages & posts it is set by Piranha here:
https://github.com/PiranhaCMS/Piranha/blob/master/Core/Piranha/Web/ClientCache.cs#L63 https://github.com/PiranhaCMS/Piranha/blob/master/Core/Piranha/Web/ClientCache.cs#L63
What values have you entered for the other pages/posts in the site, and is the caching correct for them?
Regards
Håkan Edling / @tidyui https://twitter.com/tidyui, piranhacms.org http://piranhacms.org/

25 feb. 2019 kl. 14:51 skrev lfontaine974<EMAIL_ADDRESS>Hi Hakan,
Could you help me about a cache problem on Piranha v2 ? In my home page for https://www.monticket.re I put this :
Response.Cache.SetExpires(DateTime.Now.AddMinutes(120.0));
Response.Cache.SetCacheability(HttpCacheability.ServerAndPrivate);
Response.Cache.SetMaxAge(new TimeSpan(0,120,0));
Response.Cache.SetValidUntilExpires(true);
But the page is keep in navigator cache for more long time, if I’m not force refresh in navigator. It’s only for my HomePage.
Have you an idea ?
Thanks a lot !
[signature_792767066]
[go re]http://www.monticket.re/[go mu]http://www.monticket.mu/
Laurent FONTAINE
Directeur d’exploitation
Co-Fondateur
[fice]<file:///D:/web/MONTICKET/html/signature-mail/signature-model/signature-SA-model-04.htm>0692 88 23 24
[ail]mailto:sebastien<EMAIL_ADDRESS>[bsite]http://www.monticket.re/[cebook]https://www.facebook.com/MonTicket.re/[itter]https://twitter.com/monticketre[ogle+]https://plus.google.com/+MonticketRe/posts
[nticketkdo]http://www.monticket.re/monticketkdo-le-cheque-cadeau
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/PiranhaCMS/Piranha/issues/566#issuecomment-467018114, or mute the thread https://github.com/notifications/unsubscribe-auth/AAvr0o3GcyfQc_QbdxY3ujK_EZz9hCdSks5vQ-pWgaJpZM4Su5_k.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/PiranhaCMS/Piranha/issues/566#issuecomment-467022618, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AHnoHYPBvW64jXjkhmrV4SzcE0deo5xPks5vQ-1wgaJpZM4Su5_k.

Ok thanks For link page on clientcache. I am understanding my code is not necessary 🤪
Obtenez Outlook pour iOShttps://aka.ms/o0ukef

De : Laurent FONTAINE<EMAIL_ADDRESS>Envoyé : lundi, février 25, 2019 8:37 PM
À : PiranhaCMS/Piranha; PiranhaCMS/Piranha
Cc : Author
Objet : Re: [PiranhaCMS/Piranha] Optimization linq sql for post with extensions (#566)
Oh sorry I have forget this settings !!! ☹
What is type of cache by default for Piranha ?
HttpCacheability.ServerAndPrivate or HttpCacheability.Server
If I put HttpCacheability.ServerAndPrivate, website is faster. Ca I use this HttpCacheability functionality for define cache type ?
Thanks for theses informations.
P.S : Piranha is a very good CMS !!! I’m having a lot of fun with it !!!
[signature_1122125573]
[go re]http://www.monticket.re/[go mu]http://www.monticket.mu/
Laurent FONTAINE
Directeur d’exploitation
Co-Fondateur
[fice]<file:///D:/web/MONTICKET/html/signature-mail/signature-model/signature-SA-model-04.htm>0692 88 23 24
[ail]mailto:sebastien<EMAIL_ADDRESS>[bsite]http://www.monticket.re/[cebook]https://www.facebook.com/MonTicket.re/[itter]https://twitter.com/monticketre[ogle+]https://plus.google.com/+MonticketRe/posts
[nticketkdo]http://www.monticket.re/monticketkdo-le-cheque-cadeau
De : Håkan Edling<EMAIL_ADDRESS>Répondre à : PiranhaCMS/Piranha<EMAIL_ADDRESS>Date : lundi 25 février 2019 à 18:04
À : PiranhaCMS/Piranha<EMAIL_ADDRESS>Cc : Laurent Fontaine<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Objet : Re: [PiranhaCMS/Piranha] Optimization linq sql for post with extensions (#566)
Hi there! If client cache is enabled for pages & posts it is set by Piranha here:
https://github.com/PiranhaCMS/Piranha/blob/master/Core/Piranha/Web/ClientCache.cs#L63 https://github.com/PiranhaCMS/Piranha/blob/master/Core/Piranha/Web/ClientCache.cs#L63
What values have you entered for the other pages/posts in the site, and is the caching correct for them?
Regards
Håkan Edling / @tidyui https://twitter.com/tidyui, piranhacms.org http://piranhacms.org/

25 feb. 2019 kl. 14:51 skrev lfontaine974<EMAIL_ADDRESS>Hi Hakan,
Could you help me about a cache problem on Piranha v2 ? In my home page for https://www.monticket.re I put this :
Response.Cache.SetExpires(DateTime.Now.AddMinutes(120.0));
Response.Cache.SetCacheability(HttpCacheability.ServerAndPrivate);
Response.Cache.SetMaxAge(new TimeSpan(0,120,0));
Response.Cache.SetValidUntilExpires(true);
But the page is keep in navigator cache for more long time, if I’m not force refresh in navigator. It’s only for my HomePage.
Have you an idea ?
Thanks a lot !
[signature_792767066]
[go re]http://www.monticket.re/[go mu]http://www.monticket.mu/
Laurent FONTAINE
Directeur d’exploitation
Co-Fondateur
[fice]<file:///D:/web/MONTICKET/html/signature-mail/signature-model/signature-SA-model-04.htm>0692 88 23 24
[ail]mailto:sebastien<EMAIL_ADDRESS>[bsite]http://www.monticket.re/[cebook]https://www.facebook.com/MonTicket.re/[itter]https://twitter.com/monticketre[ogle+]https://plus.google.com/+MonticketRe/posts
[nticketkdo]http://www.monticket.re/monticketkdo-le-cheque-cadeau
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/PiranhaCMS/Piranha/issues/566#issuecomment-467018114, or mute the thread https://github.com/notifications/unsubscribe-auth/AAvr0o3GcyfQc_QbdxY3ujK_EZz9hCdSks5vQ-pWgaJpZM4Su5_k.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/PiranhaCMS/Piranha/issues/566#issuecomment-467022618, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AHnoHYPBvW64jXjkhmrV4SzcE0deo5xPks5vQ-1wgaJpZM4Su5_k.

Hi Hakan,
For information with Response.Cache.SetCacheability(HttpCacheability.ServerAndPrivate) in my post view, the opening is faster than without.
I think the cacheclient class doesn’t put correctly this synthax.
If you can test it but without it’s slower.
[signature_1480227227]
[go re]http://www.monticket.re/[go mu]http://www.monticket.mu/
Laurent FONTAINE
Directeur d’exploitation
Co-Fondateur
[fice]<file:///D:/web/MONTICKET/html/signature-mail/signature-model/signature-SA-model-04.htm>0692 88 23 24
[ail]mailto:sebastien<EMAIL_ADDRESS>[bsite]http://www.monticket.re/[cebook]https://www.facebook.com/MonTicket.re/[itter]https://twitter.com/monticketre[ogle+]https://plus.google.com/+MonticketRe/posts
[nticketkdo]http://www.monticket.re/monticketkdo-le-cheque-cadeau
De : Håkan Edling<EMAIL_ADDRESS>Répondre à : PiranhaCMS/Piranha<EMAIL_ADDRESS>Date : lundi 25 février 2019 à 18:04
À : PiranhaCMS/Piranha<EMAIL_ADDRESS>Cc : Laurent Fontaine<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Objet : Re: [PiranhaCMS/Piranha] Optimization linq sql for post with extensions (#566)
Hi there! If client cache is enabled for pages & posts it is set by Piranha here:
https://github.com/PiranhaCMS/Piranha/blob/master/Core/Piranha/Web/ClientCache.cs#L63 https://github.com/PiranhaCMS/Piranha/blob/master/Core/Piranha/Web/ClientCache.cs#L63
What values have you entered for the other pages/posts in the site, and is the caching correct for them?
Regards
Håkan Edling / @tidyui https://twitter.com/tidyui, piranhacms.org http://piranhacms.org/

25 feb. 2019 kl. 14:51 skrev lfontaine974<EMAIL_ADDRESS>Hi Hakan,
Could you help me about a cache problem on Piranha v2 ? In my home page for https://www.monticket.re I put this :
Response.Cache.SetExpires(DateTime.Now.AddMinutes(120.0));
Response.Cache.SetCacheability(HttpCacheability.ServerAndPrivate);
Response.Cache.SetMaxAge(new TimeSpan(0,120,0));
Response.Cache.SetValidUntilExpires(true);
But the page is keep in navigator cache for more long time, if I’m not force refresh in navigator. It’s only for my HomePage.
Have you an idea ?
Thanks a lot !
[signature_792767066]
[go re]http://www.monticket.re/[go mu]http://www.monticket.mu/
Laurent FONTAINE
Directeur d’exploitation
Co-Fondateur
[fice]<file:///D:/web/MONTICKET/html/signature-mail/signature-model/signature-SA-model-04.htm>0692 88 23 24
[ail]mailto:sebastien<EMAIL_ADDRESS>[bsite]http://www.monticket.re/[cebook]https://www.facebook.com/MonTicket.re/[itter]https://twitter.com/monticketre[ogle+]https://plus.google.com/+MonticketRe/posts
[nticketkdo]http://www.monticket.re/monticketkdo-le-cheque-cadeau
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/PiranhaCMS/Piranha/issues/566#issuecomment-467018114, or mute the thread https://github.com/notifications/unsubscribe-auth/AAvr0o3GcyfQc_QbdxY3ujK_EZz9hCdSks5vQ-pWgaJpZM4Su5_k.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/PiranhaCMS/Piranha/issues/566#issuecomment-467022618, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AHnoHYPBvW64jXjkhmrV4SzcE0deo5xPks5vQ-1wgaJpZM4Su5_k.

Hi
I can’t insert this synthax in global.asax. It’s not correct for params. Have you a complete synthax in sample ?
Thanks.
Hooks.Manager.PostEditModelAfterSave += (model) =>
        {

                using (var db = new MyContext())

                {

                    // Store any values from the model you need to search in the PostFilters

                }

            }
        }

De : Håkan Edling<EMAIL_ADDRESS>Répondre à : PiranhaCMS/Piranha<EMAIL_ADDRESS>Date : jeudi 22 mars 2018 à 01:05
À : PiranhaCMS/Piranha<EMAIL_ADDRESS>Cc : Laurent Fontaine<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Objet : Re: [PiranhaCMS/Piranha] Optimization linq sql for post with extensions (#566)
Like you say your query fetches all posts to memory and then filters them on the content of their deserialized extension values which of course isn't optimal. If you need to filter large sets of posts on extension values I would do it like this:

Create (if you don't already have) a custom DbContext.
Add a PostFilter entity where you store your "indexed" values and the post id.
Add a hook to Manager.PostEditModelAfterSave where you store the values and post id.

It could look something like this:
public MyContext : DbContext
{
public DbSet<PostFilter> PostFilters { get; set; }

}
public class PostFilter
{
public string Key { get; set; }

public string Value { get; set; }

public Guid PostId { get; set; }

}
// This should go in your Global.asax.cs where you init the Piranha app
Piranha.Hooks.Manager.PostEditModelAfterSave => (model)
{
using (var db = new MyContext())

{

    // Store any values from the model you need to search in the PostFilters

}

}
With this approach you can do a fast query on your filter table and get the Id's of the matching posts. After that you can get all of the post matching the result of your query.
Please note that I haven't tested the code above, so please see it as suggestion for how it could be implemented.
Regards
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/PiranhaCMS/Piranha/issues/566#issuecomment-375095229, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AHnoHRtT0LMWMTCCNbMgrBciG-whtFnGks5tgsCBgaJpZM4Su5_k.
Laurent FONTAINE
Directeur d'exploitation & Co-Fondateur
+262 692 88 23 24
<EMAIL_ADDRESS>www.monticket.re

Hi! Here are the docs for the available hooks for version 2.x
http://legacy.piranhacms.org/docs/api-reference/hooks
Regards

Btw, we are looking at closing issues for this repository as it's over 2 1/2 years since we released the latest version of the legacy version and we're a rather small team.


Review handling of versions in task collection

While playing around with fractal-web, I noticed that the task-collection endpoint apparently ignores the task-package version, see for instance
Create task colleciton action
{
  package: 'fractal-tasks-core',
  version: '0.10.0a1',
  python_version: '3.9',
  package_extras: 'fractal-tasks'
}
Task collection created {
  data: {
    status: 'pending',
    package: 'fractal-tasks-core',
    venv_path: '.fractal/fractal-tasks-core0.9.4',
    task_list: [],
    log: null,
    info: 'Collecting tasks in the background. GET /task/collect/1 to query collection status'
  },
  timestamp: '2023-06-13T06:52:53.843404',
  id: 1
}

The issue is simply that we renamed TaskCollectPip.version to TaskCollectPip.package_version, and fractal-web needs to be updated.

This now works as expected (as of deab7b9), that is, we can correctly collect task packages with different versions.
The only issue which is left is that there is still an "Unspecified" version in the table:

My bad, fixed with 3d08d30.


cosmic-panel: unstable-2023-09-22 -> unstable-2023-10-30

Description of changes
Updating, formatting, moved cosmic-panel to new by-name package layout.

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Are there actual user-facing changes in this update? We don't necessarily want to be updating to every new git rev of packages with committed Cargo.lock files, because they're very big and end up making the git history bigger forever.

Theming support, so yes

Can you either not reformat the file, not move the file, or have two separate commits?  As it is, the file has changed too much for git to realise it's been moved.

Can you either not reformat the file, not move the file, or have two separate commits? As it is, the file has changed too much for git to realise it's been moved.

done

#266539 (review)

done


Update ROCm to 3.7.0

Motivation for this change
Tested with clinfo and Darktable.
The good news is that this release removes the need for rocm-runtime-ext, since image support is added to rocm-runtime. Also restructured dependencies a bit, since rocm-runtime now requires clang and rocm-device-libs.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@acowley @Flakebi this is now ready for review.

Wow, you made this upgrade fast, and made overall improvements while doing it! The integration of image support isn't even mentioned in the ROCm 3.7 release notes, which is sort of shocking given how long it's been sparking Issues on their repositories (an example that remains open).
Everything looks good to me, but I haven't yet ported the diffs to the overlay to push it through CI there.


Update readme.md

Fixing a small typo/spacing issue.

@ibarra-michelle Pinging here as the Reviewers tab is not populating a dropdown.

Good catch, thanks @Scr1ptcat !


Keys for mvit model and its pretrained weights are not matched.

Fail to load pretrained mvit model:
import torch

model = torch.hub.load('facebookresearch/pytorchvideo', model='mvit_base_16x4', pretrained=True)

it works with my previous copy of ~/.cache/torch/hub/facebookresearch_pytorchvideo_main/
the issue happens when I use the latest pytorchvideo

Try this: https://github.com/facebookresearch/pytorchvideo/pull/164

#164

Still the same error

Some workaround: replace the pytorchvideo in torch cache with some previous version.
I tried with tag v0.1.3 and it works. However, better it can be fixed in latest code as well.

Okay, I think that they forgot to update the code. Try with an older commit of the repo too.


fix(deps): move to @types/eslint v9.x

Flat config type deprecated: eslint 9 only support flat config

Are the types exported breaking in any way? It used to export a FlatConfig and it still does, right?

Are the types exported breaking in any way? It used to export a FlatConfig and it still does, right?

Yes, but I'm not sure the type content has not changed in @types/eslint major revision. stylistic/eslint-plugin has not made a major version bump as switching to it ...

I think we should be fine


Forestry: add newly supported SSG

Forestry supports whatever SSG storing content in Markdown, YAML, JSON, TOML.

You can just put - All like we do for Netlify CMS


[Potential BUG] Latency differs for different runs

Describe the bug
Hi, I'm benchmarking some mlx functions, but find that the same mlx function can be slow in the 1st run. Could you help me understand the reason?
Another question is that, do we need to be careful about caching when we run benchmarks? For example, when we benchmark with for loops, would the subsequent mx.eval reuse the previous results?
If yes, is mx.metal.clear_cache() the right way to avoid the reuse of cache?
To Reproduce
import time

import mlx.core as mx

def benchmark_array_creation(N: int):
    mx.metal.clear_cache()
    toi = time.perf_counter()
    mx.eval(mx.ones((1, N)))
    toc = time.perf_counter()
    tpi = 1e3 * (toc - toi)
    print(f"array creation takes {tpi:.3f} ms")

if __name__ == "__main__":
    print(mx.default_device())
    for i in range(10):
        print(f"benchmark array creation, run {i}")
        benchmark_array_creation(1024)

Output
Device(gpu, 0)
benchmark array creation, run 0
array creation takes 10.567 ms
benchmark array creation, run 1
array creation takes 0.397 ms
benchmark array creation, run 2
array creation takes 0.267 ms
benchmark array creation, run 3
array creation takes 0.234 ms
benchmark array creation, run 4
array creation takes 0.216 ms
benchmark array creation, run 5
array creation takes 0.200 ms
benchmark array creation, run 6
array creation takes 0.208 ms
benchmark array creation, run 7
array creation takes 0.244 ms
benchmark array creation, run 8
array creation takes 0.205 ms
benchmark array creation, run 9
array creation takes 0.221 ms

Expected behavior
The first run should be as fast as the subsequent runs.
Desktop (please complete the following information):

OS Version: MacOS 14.5
Version 0.14.1

I have the same question. I made each run handles different tensors (even though mx.metal.clear_cache() is called for each run). I also add another benchmark of matmul.
import time

import mlx.core as mx

def benchmark_array_creation(N: int):
    mx.metal.clear_cache()
    toi = time.perf_counter()
    mx.eval(mx.ones((1, N)))
    toc = time.perf_counter()
    tpi = 1e3 * (toc - toi)
    print(f"array creation takes {tpi:.3f} ms")

def benchmark_matmul(N: int):
    mx.metal.clear_cache()
    toi = time.perf_counter()
    mx.eval(mx.ones((N, N)) * mx.ones((N, N)) * N)
    toc = time.perf_counter()
    tpi = 1e3 * (toc - toi)
    print(f"array creation takes {tpi:.3f} ms")

if __name__ == "__main__":
    print(mx.default_device())
    for i in range(10):
        print(f"array creation run {i}", end=" ")
        benchmark_array_creation(1024 + i)

    for i in range(10):
        print(f"matmul run {i}", end=" ")
        benchmark_matmul(1024 + i)

The result is as follows:
array creation run 0 array creation takes 10.809 ms
array creation run 1 array creation takes 0.424 ms
array creation run 2 array creation takes 0.843 ms
array creation run 3 array creation takes 0.301 ms
array creation run 4 array creation takes 0.418 ms
array creation run 5 array creation takes 0.272 ms
array creation run 6 array creation takes 0.262 ms
array creation run 7 array creation takes 0.258 ms
array creation run 8 array creation takes 0.269 ms
array creation run 9 array creation takes 0.258 ms
matmul run 0 array creation takes 0.947 ms
matmul run 1 array creation takes 0.798 ms
matmul run 2 array creation takes 0.764 ms
matmul run 3 array creation takes 0.430 ms
matmul run 4 array creation takes 0.727 ms
matmul run 5 array creation takes 0.384 ms
matmul run 6 array creation takes 0.703 ms
matmul run 7 array creation takes 0.418 ms
matmul run 8 array creation takes 0.766 ms
matmul run 9 array creation takes 0.399 ms

So in general when you run stuff on the device you have to setup a bunch of state. This includes loading the GPU device itself, loading the metal library, setting up the allocator, possibly some other stuff. All of that takes time. And it usually happens the first time you evaluate an operation in MLX.
Its good practice in general to time with a warmup like so:
# warmup
for _ in range(5):
  myfun()

tic = time.perf_counter()
for _ in range(10):
  myfun()
toc = time.perf_counter()
time_per_it = (toc - tic) / 10

Regarding the cache:

would the subsequent mx.eval reuse the previous results?

No that never happens

If yes, is mx.metal.clear_cache() the right way to avoid the reuse of cache?

The cache is a memory buffer cache but we don't reuse results from it, just data buffers. I would not clear it unless you are concerned about memory use because clearing it will slow things down.


ksonnet changes objects even in dry-run mode

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:

/kind bug

/kind feature

What happened:
I ran ks apply prod --dry-run. Despite the --dry-run flag, ksonnet added labels and annotations to some cluster objects:
app.kubernetes.io/deploy-manager=ksonnet
ksonnet.io/managed={"pristine":"H4sIAAAAAAAA/3yPP0sEMRDFez/Gq+OtZyWpLS2sbOSK2eywBvNnyIwHuuS7SxQXRbgyvLzfb94GkvjETWMt8CARnc7HmY1u4fAaywKPe5ZU3zMXg0Nmo4
WM4DcUygyPudbMjcqK7qDCYWSNJcVACn90UE4crLYRZLLw8kAz...

What you expected to happen:
I expected no changes whatsoever to be made to the cluster in dry-run mode.
How to reproduce it (as minimally and precisely as possible):
Do a ks apply ENV --dry-run to objects which have not been managed by ksonnet before.
Anything else we need to know?:
In:
func (a *Apply) handleObject(co clientOpts, obj *unstructured.Unstructured) (string, error) {
	if err := tagManaged(obj); err != nil {
		return "", errors.Wrap(err, "tagging ksonnet managed object")
	}

it looks as if the call to tagManaged() should be wrapped with if !a.DryRun { ... }.
Environment:

ksonnet version (use ks version):

ksonnet version: 0.11.0
jsonnet version: v0.10.0
client-go version:

Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.1", GitCommit:"d4ab47518836c750f9949b9e0d387f20fb92260b", GitTreeState:"clean", BuildDate:"2018-04-13T22:28:25Z", GoVersion:"go1.9.5", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"10+", GitVersion:"v1.10.2-gke.1", GitCommit:"75d2af854b1df023c7ce10a8795b85d3dd1f8d37", GitTreeState:"clean", BuildDate:"2018-05-10T17:23:18Z", GoVersion:"go1.9.3b4", Compiler:"gc", Platform:"linux/amd64"}

I just ran into this issue also.
ksonnet version: 0.13.1
jsonnet version: v0.11.2
client-go version: kubernetes-1.10.4

$ ks apply default --dry-run

level=info msg="tagging ksonnet managed object (dry-run)"
level=info msg="upserting object (dry-run)"

Says it's a dry-run, however the changes are actually applied.
@bitfield Has PR #699 fixed it for you?

I couldn't tell you. Because the unexpected resource update took down an important production service, ksonnet was summarily fired from the organization where I was trialling its use.

@bitfield Wow, that's pretty bad.
I've just configured our dry-run CI job with a read-only Kubernetes token via RBAC to protect against these kinds of bugs.
Now the dry run fails because it wants to patch, but it can't:
$ ks apply default --dry-run

level=info msg="tagging ksonnet managed object (dry-run)"
level=error msg="handle object: patching object from cluster: merging object with existing state: ingresses.extensions \"guestbook-ui\" is forbidden: User \"system:serviceaccount:gitlab:gitlab-ci-read-only\" cannot get ingresses.extensions in the namespace \"default\""

@shomron This is a pretty serious bug.

Thanks for reporting! I’ll look into this.


Navbar improvements

Move the UISegmentedControl into the navbar
Provide a scroll-up behavior that hides most of the navbar

Closing issue and migrating to new repo.


APEXCORE-745 Buffer server may stop processing tuples when backpressure is enabled

@PramodSSImmaneni, @sandeshh Please review

test this please

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Apex_Core_PR/850/

test this please

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Apex_Core_PR/852/

test this please

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Apex_Core_PR/853/

test this please

test this please

test this please

Can you rebase

Done.


Implement panic stack traces

The exception type needs support for stack traces.
When an exception is thrown, the N topmost tiro function names should be represented in the trace.
Exact positions within those functions (e.g. line numbers) are currently out of scope because the compiler does not yet emit those information.

If panic stack traces are enabled (a flag during vm construction, off by default), a very simple stack trace is attached to every exception.
The stack trace reflects function names only at the moment.
Example:
import std;

export func test() {
    nested(2);
}

func nested(n) {
    if n == 0 {
        std.panic("help!");
    }
    nested(n - 1);
}

Calling test produces the following trace (most nested function on top):
Coroutine-1:
  - nested
  - nested
  - nested
  - test

Note: module names etc. are currently not part of the function name. Modules are not fleshed out well, yet.


3 new Unknown attribute messages

I have noticed 3 new Unknown attribute messages being printed in the console every time weConnect.update() is run. Not sure exactly what they mean but would be great if they could be added.
/vehicles/***: Unknown attribute brandCode with value V
/vehicles/***/domains/charging/chargingSettings: Unknown attribute autoUnlockPlugWhenChargedAC with value off
/vehicles/***/domains/charging/plugStatus: Unknown attribute ledColor with value none

I added those for 0.42.0, but do you have seen anything else than "none" for ledColor yet? The problem is to guess the possible values.

No, I have not seen any other value except none.

Released with 0.42.0


Compilation

Building C# + Razor as one assembly at build time

@rynowak Closing as we think this is done. If there's still more work to track in this area, please file a specific tracking issue.


Rename 'conversion' Variables to something meaningful

What was I thinking that night?

conversion = inputs['integer']

What does that even mean?
Print the index of the inputs list data structure and think of something that makes more sense.

Decided to just name it arg for argument
b6eae9eb62738e7b6dd44db28f8fdf25a9e7806d


Fix invoice payment state not updating when given order is paid

Change conditions settings Invoice Payment state.
$order->getPaymentState() is returning a string from OrderPaymentStates, therefore Invoice status is never set to 'completed'

Thanks, Gregoire! :1st_place_medal:


Stability issues

Hi @kylemin !
I am working myself on an implementation of HEX graphs in Python using PyTorch and I'm having some stability issues : when the scores are very negative and very positive before the HEX layer, the computations diverge too much inside the HEX layer due to the passage to exponentials and I get some NaNs or approximations (some numbers can shrinked to 0 because they are too small to be taken into account given the precision of the system) which make predictions wrong.
Did you have similar numerical stability issues yourself ? And if yes, how did you deal with them ?
Best regards,
Arthur

Hi Arthur,
I didn't have stability issues myself, but it totally makes sense that you have those issues. Over/under-flow of exp is a well-known issue so I think I once tried subtracting the maximum element as shown here: link. Maybe you could try this?
Thank you,
Kyle

Hi @kylemin, thank you very much for the quick answer !
The problem is that I have strongly diverging score vectors (the vector you get out of the neural network), some are very positive and others very positive.
If tried subtracting the max like you proposed but then the negative ones get even more negative which results in underflow.
Do you have any idea of what the output scores of your neural network (before they get processed by the HEX layer) looked like after training ? That could help me a lot understand my problem ! (I know it has been a long time since you worked on this project so I understand if you don't remember anything of the sort)
Best regards,
Arthur

Hi Arthur,
I am not sure how to address your problem and I don't recall how the output scores changed after training. I'll try to log in to my previous school server to see if the information is still there. Thank you!


Analyzer: Cascade with null-aware selector seems to spuriously make type nullable

Consider the following program:
class C<X> {
  X x;
  C(this.x);
}

void main() {
  C c = C<String?>('Hello!')..x?.length;
}

With a fresh dartanalyzer from commit 208ab625c159ccefedea58709f9a15ea5e441b19, we get the following response:
> dartanalyzer --enable-experiment=non-nullable n019.dart
ERROR|STATIC_TYPE_WARNING|INVALID_ASSIGNMENT|/usr/local/google/home/eernst/lang/dart/scratch/202004/n019.dart|7|9|31|A value of type 'C<String?>?' can't be assigned to a variable of type 'C<dynamic>'.

However, the expression C<String?>('Hello!')..x?.length evaluates to the newly created instance of C<String?>, and there is no way it could be null. The fact that the cascade section uses a null-aware selector ?.length seems to spuriously make the type of the cascade as a whole nullable.
In support of that assumption, the following variant has no error:
class C<X> {
  X x;
  C(this.x);
}

void main() {
  C c = C<String?>('Hello!')..x.toString;
}

I thought this could be a duplicate and searched, but did not find an existing issue on the same topic.

https://dart-review.googlesource.com/c/sdk/+/144672


Fix poll notify

Summary
Fix a race condition in poll_notify, revert seemingly unnecessary change in serial driver.
Originally, the race condition was found with an older nuttx version inside the poll_notify, and the same race condition exists also in the current upstream version.
This PR has the first patch, which protects from the RMW issue. Second patch is an optimization to the poll_notify, but also fixes another race which may cause callback being called twice.
For the serial driver, the issue has been circumvented in #13914, but the race condition should be fixed in poll_notify, as it exists for all the poll interfaces, and not just serial driver. I believe that the serial driver fix is unnecessary, so third patch is reverting that - it is not nice to add sched_locks around the drivers unnecessarily.
Impact
Impacts the implementation of poll / poll_notify, fixing a read-modify-write race condition.
Testing
The code has been inspected by 2 engineers, and desk-tested with real application using the poll interface. However, I don't have a possibility to run extensive testing which would show the exact issue which was fixed in #13914, so I'd appreciate if the solution is reviewed & tested by e.g. @CV-Bowen who fixed the race condition for serial driver.

[Experimental Bot, please feedback here]
Yes, this PR mostly meets the NuttX requirements, although some clarifications and additions are needed.
Missing/Needs Improvement:

Summary: While the summary explains the what, it lacks detail on how the changes fix the race condition.  Describe the specific mechanism used (e.g., mutex, atomic operation, etc.) in each patch. Briefly mention why the serial driver change in #13914 is believed to be unnecessary.
Impact:

User Impact:  While it fixes a bug, does this introduce any behavioral changes the user might observe? If the answer is truly NO, explicitly state "NO". If there are subtle timing changes that could affect user code (even if unlikely), mention that.
Build Impact: NO (Explicitly state)
Hardware Impact:  NO (Explicitly state)
Documentation Impact:  NO (Explicitly state, or YES if documentation needs updating).
Security Impact:  This fixes a race condition.  This likely has positive security implications.  Describe this briefly. E.g., "YES - Improves robustness and potentially mitigates denial-of-service possibilities due to the race condition."
Compatibility Impact: NO (Explicitly state, unless there's a subtle edge case)

Testing: The testing section is weak. Desk testing and code inspection are good, but insufficient for a race condition fix.  While you may not be able to reproduce the original issue, you should attempt to create a test case that stresses the poll_notify mechanism and demonstrates that the race condition is no longer present.  Even a simple test case is better than none.  Explain why you can't fully reproduce the original issue if possible.  Reaching out to @CV-Bowen for review and testing is good practice.

Recommendations:

Elaborate on the "how" in the Summary. Provide a concise, technical description of the fix for each patch.
Complete all Impact sections explicitly. Even if the answer is "NO", state it clearly.
Strengthen the Testing section.  Create at least one test case, even if it's a simple one, to demonstrate the absence of the race condition after your changes.  Document the limitations of your testing environment and explain why reproducing the original issue is difficult.

By addressing these points, you will significantly improve the quality of your PR and make it easier for reviewers to understand and accept your changes.

re-based to current  master


Add the "arena" category to the crate for crates.io

I think because it's not categorized as an arena crate, it shows up lower in the rankings despite being the most popular arena crate by far.

Would happily merge a PR that added this! :)


Uncaught TypeError: Cannot use 'in' operator to search for 'length'

I'm getting the error "Uncaught TypeError: Cannot use 'in' operator to search for 'length' in isArraylike @ jquery.self-".
It happens in these lines:
var length = "length" in obj && obj.length,
      type = jQuery.type( obj );

    if ( type === "function" || jQuery.isWindow( obj ) ) {
        return false;
    }

A stackoverflow question states that "The versions 1.11.3 and 2.1.4 have this issue."
Jquery-rails versions, say:
module Jquery

    module Rails

      VERSION = "4.0.5"

      JQUERY_VERSION = "1.11.3"

      JQUERY_2_VERSION = "2.1.4"

      JQUERY_UJS_VERSION = "1.1.0"

    end

 end

How can I fix my issue?

As a temporary workaround, you can lock the jquery-rails gem to 4.0.4. That works in my environment where I was getting the same issues with datatables.

As a temporary workaround, you can lock the jquery-rails gem to 4.0.4. That works in my environment where I was getting the same issues with datatables.

This has been fixed in datatables https://github.com/DataTables/DataTablesSrc/commit/485b259e5c02ae7 can it be integrated and a new gem cut?  For my project I need jquery-rails 4.0.5, another gem requires that version.

:+1:
Looks like the same issue as #193.

Yes please! @ricardodovalle @rweng Any chance you could give us a timeline for integrating https://github.com/DataTables/DataTablesSrc/commit/485b259e5c02ae7 into a new version of the jquery-datatables-rails gem? Thanks for all your work maintaining this.

Confirmed locking jquery-rails gem to 4.0.4 solves it as stated by @nightsurge . Hoping this would be fixed soon

As @nightsurge said, locking the gem fixes the issue:
gem 'jquery-rails', '=4.0.4'
(Had to do this to update the gem)
$ bundle update jquery-rails
+1 for a new version with the fix

:heavy_plus_sign::one:

:+1:

@nightsurge thx for the workaround! drove me nuts.

I believe this has been resolved with jquery-datatables-rails (3.4.0)

Can confirm that I'm no longer seeing this issue with v. 3.4.0.

Thanks!


IPv6 IP Options extension headers

In bmv2 we are currently not supporting (able to parse/process) extensions.  IPv6 has multiple headers.  In v4 and v6 we are handling only vanilla headers.  Supporting extension headers in bmv2.

Saad has completed IPv4, would like to know how to test.  May also reach out to Chris if needed.

hi @saad-mzhr - going through PRs, is there an update to this one?
@chrispsommers too

Saad has completed IPv4, would like to know how to test. May also reach out to Chris if needed.

Is there a PR for these enhancements? This seems like several months old so the P4 code will need to be synced to main. I recommend adding test cases to accompany the enhancements, using existing test cases as a starting point. In the PR, include a link to this issue so they will be cross-referenced. Thanks.

@KrisNey-MSFT
These comments are old. I have already tested/discussed/approved and merged ipv4 related code into master. During discussion, it was told that these ipv4/ipv6 extensions were not actually needed at the moment. So I did not proceed with ipv6 part of this ticket.

Thanks, is it ok if we close it out?  Os keep it open to remind us later of needed extensions? Let me know what you think… Thank you

From: saad-mzhr @.>
Sent: Friday, November 4, 2022 6:35 AM
To: Azure/DASH @.>
Cc: Kristina Moore @.>; Mention @.>
Subject: Re: [Azure/DASH] IPv6 IP Options extension headers (Issue #145)
@KrisNey-MSFThttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FKrisNey-MSFT&data=05|01|kristina.moore%40microsoft.com|7aeac888e3e0458070b508dabe6980cc|72f988bf86f141af91ab2d7cd011db47|1|0|638031657590211312|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=pctsWzFo4T3M%2FHcxecmVyL4s7k1CndBPqk%2B%2BsVx8UHs%3D&reserved=0
These comments are old. I have already tested/discussed/approved and merged ipv4 related code into master. During discussion, it was told that these ipv4/ipv6 extensions were not actually needed at the moment. So I did not proceed with ipv6 part of this ticket.
—
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2FDASH%2Fissues%2F145%23issuecomment-1303548872&data=05|01|kristina.moore%40microsoft.com|7aeac888e3e0458070b508dabe6980cc|72f988bf86f141af91ab2d7cd011db47|1|0|638031657590211312|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=AfkOnQeWTwQfIf2n6h%2FyaeKnOdFDdCm%2B4H1GM7fwfUQ%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAFJSI6H4ICYWJ3NO7TANII3WGUGLRANCNFSM53TC6TBA&data=05|01|kristina.moore%40microsoft.com|7aeac888e3e0458070b508dabe6980cc|72f988bf86f141af91ab2d7cd011db47|1|0|638031657590211312|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=3m5zCuJRjQfYqBFVIgPxUTrQtrembcM6jaVxYAOSLMU%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>

Closing per comments above


Can you add a feature, for panel location, add float panel.  for now , Its already have bottom, left,  right,

/duplicate of #193267

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Added: http request context handling

see: https://github.com/pivotal-cf/brokerapi/issues/29
#132019309

Will also upload alternative impl.

Closed in favour of the implementation in #32


Add templates to release archive

Related to discord discussion https://discord.com/channels/860500297297821756/865526691350839296/978938211080495154.

Thanks @andresterba


add +grad to md mark

The md mark didn't have a +grad, so it crashed if you ever changed an md file.
It also lacked a trailing newline.

Longstanding complaint of yours truly: #175


HHH-10185 In nonstrict-read-write mode the remove may be not applied

@galderz Forgot to issue this PR some time ago... Applicable to 5.0 branch, too.

Looking...

I've integrated it into master but could not apply the commit cleanly to 5.0 branch. Can you please send a PR for 5.0?

@galderz OK, I'll check it...

@galderz I've applied the commit on Friday and it went well, besides the testsuite; in 5.0 I had failures running hibernate-infinispan testsuite even without the update applied. I'll try later this week if it got fixed.

Created #1146


Night CI 2022-12-06: Failed

action url: https://github.com/spidernet-io/spiderdoctor/actions/runs/3633154672

codecov issue


Move to Numba 0.52

Motivation: keep up to date with latest Numba.

/AzurePipelines run

As an exceptional case agreed to merge and include in the next release despite internal CI fails with python3.6.


Feedback Opportunity: Improving the sidebar layout

Updated (1/24/2021): Now include pre-requisite work

[x] Pre-requisite work in the code behind
[ ] Add section to access favorite items quickly
[ ] Make the headers collapsible

Original sidebar concepts:

@xPoppyx That's an interesting take-having the drives go under This device.
Originally, I was proposing something like this, with three sections:

(Ignore the new tab page stuff for now)

It looks great, maybe with an option to choose the order? for example first "pinned items", then "This device" and "Drives"

I don't know how hard it would be but ideally it should be possible to rearrange the items in the sidebar. Additionally home page shouldn't prevent improving the sidebar.

@mdtauk There are native apps for iCloud and Dropbox. We can pin their folders if the native client is installed on the device.

@duke7553 I took the liberty to modify your idea to propose a slightly different version.

Idea

Proposed
The sidebar looks streamlined and the blue marking is not floating far from the left edge. (important for quickly identifying which folder the user is currently in)
I feel having an icon in the sections is redundant because, for example, the drives section doesn't need a drive icon when all its children have the drive icon.
Also, the spacing between sections improves the grouping a bit better; the extra space can be removed when the sections are minimized (as can be seen in the dummy section). Maybe the actual spacing can be tweaked to make it a little denser to fit more.

Option 2

I like the idea of having a bit more separation like the previous concept, with either padded nesting for each header or space between headers. I think it makes a potential long list cleaner/easier to navigate.

Regardless, we'll be using the default hierarchical nav view design with little/no customizations to the padding or item indentations.

Regardless, we'll be using the default hierarchical nav view design with little/no customizations to the padding or item indentations.

Is that the design shown in "Option 2"?

Will there be an optional to disable Quick Access?

I think so

The goal of the sidebar control is to access the most important directories on your device. The built in Files Explorer in Windows was designed a little differently and it blurs the lines between the navigation menu and the list of items. The new design we are looking at implementing in Files no longer pins all the library locations by default, instead the new design will only have four items pinned by default that will go under the favorites section.
Underneath the favorites section, there will be a collapsible section for your cloud storage, this section will be hidden if Files does not detect a cloud storage provider on your device.
The next section will be for devices, this section will only show if you connect removable drives. This section will be customizable and will have the option to show internal drives in case the user prefers to see their drives there.
As seen in the concepts, there will be a section for Network drives, and we can add a section for WSL drives as well once we add support for them.

I am planning on starting work on some of the pre-requisite tasks for enhancements to the sidebar.
In the meantime, we'd like to know:

If you had to pick, what set of 3-4 locations do you need quick access to the most? For example, we know that the Desktop folder is far less important for immediate access than the Downloads folder in the collaborative world we live in. This is because the desktop is already visible by default in Windows providing users ample opportunity to access it. Conversely, we want to accommodate the users who would want the Desktop location easily accessible from the Sidebar.
How many sidebar items do you typically have? This count should include the default items. Please let us know if this can be improved.
How satisfied are you with the height of the sidebar items on both desktop and touch?
If you could add one feature to the sidebar to improve your workflow, what would it be?

I am planning on starting work on some of the pre-requisite tasks for enhancements to the sidebar.
In the meantime, we'd like to know:

If you had to pick, what set of 3-4 locations do you need quick access to the most? For example, we know that the Desktop folder is far less important for immediate access than the Downloads folder in the collaborative world we live in. This is because the desktop is already visible by default in Windows providing users ample opportunity to access it. Conversely, we want to accommodate the users who would want the Desktop location easily accessible from the Sidebar.
How many sidebar items do you typically have? This count should include the default items. Please let us know if this can be improved.
How satisfied are you with the height of the sidebar items on both desktop and touch?
If you could add one feature to the sidebar to improve your workflow, what would it be?

I hide my desktop icons, so when I dump something on the desktop, I access it via file explorer window

I hide my desktop icons, so when I dump something on the desktop, I access it via file explorer window

If you had to pick, what set of 3-4 locations do you need quick access to the most?
Desktop and Downloads in the "quick access" location, as well as I guess the "Home" link, which in explorer I would equate to the "This PC" link. Though I assume that's not going anywhere haha.
How many sidebar items do you typically have?

I don't typically use the recycling bin, but I leave it there for occasional testing.
How satisfied are you with the height of the sidebar items on both desktop and touch?
I'd like it (and the rest of the UI, eventually) to be more compact. Not necessarily to the same level as explorer, but just a bit more. You can see in the screenshots above the difference in space the same number of links takes up.
If you could add one feature to the sidebar to improve your workflow, what would it be?
Dragging to reorder/place items, though I know that's being worked or planned on. I'd love to see coloured icons if that ever becomes possible.

If you had to pick, what set of 3-4 locations do you need quick access to the most?
Desktop and Downloads in the "quick access" location, as well as I guess the "Home" link, which in explorer I would equate to the "This PC" link. Though I assume that's not going anywhere haha.
How many sidebar items do you typically have?

I don't typically use the recycling bin, but I leave it there for occasional testing.
How satisfied are you with the height of the sidebar items on both desktop and touch?
I'd like it (and the rest of the UI, eventually) to be more compact. Not necessarily to the same level as explorer, but just a bit more. You can see in the screenshots above the difference in space the same number of links takes up.
If you could add one feature to the sidebar to improve your workflow, what would it be?
Dragging to reorder/place items, though I know that's being worked or planned on. I'd love to see coloured icons if that ever becomes possible.

If you could add one feature to the sidebar to improve your workflow, what would it be?

Just discovered Files so I'm probably wrong, but when reading this thread it seems to me that you are not planning on having an unlimited number of sections in the sidebar?
I would love to be able to create (and rename) as many sections as I like, and then freely move my pinned folders from one section to the other. Did I just missed it?
Of course I don't mind at all the automatically generated sections such as Drives, Devices and so on. And maybe we shouldn't be able to move a pinned folder in those sections, but I don't know how much more complexity it would add?
P.s: Since Slack rework of their sidebar I've seen a huge boost on how efficiently I can retrieve my pinned items and I can't wait to have the same for my PC folders :)

If you could add one feature to the sidebar to improve your workflow, what would it be?

Just discovered Files so I'm probably wrong, but when reading this thread it seems to me that you are not planning on having an unlimited number of sections in the sidebar?
I would love to be able to create (and rename) as many sections as I like, and then freely move my pinned folders from one section to the other. Did I just missed it?
Of course I don't mind at all the automatically generated sections such as Drives, Devices and so on. And maybe we shouldn't be able to move a pinned folder in those sections, but I don't know how much more complexity it would add?
P.s: Since Slack rework of their sidebar I've seen a huge boost on how efficiently I can retrieve my pinned items and I can't wait to have the same for my PC folders :)

I can't use this explorer yet as we're missing the favorites part. Probably something very personal, or maybe not, who knows! But I'll be expecting this feature with eager!

@XPoppyX This issue should be resolved with the latest in main.


Add coredns to add-on list

Add Service Discovery section and CoreDNS to add on list.

This change is

@miekg @chrisohaver fyi


bootstrap-terraform-state fails on minio due to no enpoint-url

This is a fix for the above error, it has been fixed and ran the tests.

@mcb we've merged your pr; please let us know if you have any additional issues by either re-opening this pr or creating a new issue. thanks!


Filtering

Hi is there a way to alter the threshold for filtering from < 40 to a different value?

Amazing! thank you