text:
[Bug Report] Latest update broke styling for flexbox properties in Nuxt3

UnoCSS version
0.58.7
Describe the bug
Upon updating Unocss to v0.58.7 in my Nuxt3 portfolio this morning, I notice a number of styles simply no longer work. Notably flexbox properties (and sometimes borders). This does not seem to have an effect on live production.
Here is a pic of a section in my portfolio in dev mode vs live - the styles have not changed at all between these two images. It seems that this issue only happens on the initial load of the page, although it can sometimes revert back to this state when I save changes on a file or component:
*These images are showing that flex, items-center, and justify-between not being applied to these elements. Again, styles have not been changed or edited in any way. Only the unocss version has changed.

Reproduction
I can't seem to reproduce this in a Stackblitz as it seems to be local environment issue.. But here is the gist of what I have in my project, which always seems to be unstyled upon page load:
https://stackblitz.com/edit/nuxt-starter-xy4uvj?file=package.json,unocss.config.ts,app.vue,nuxt.config.ts
System Info
Nuxt v3.11.1
Node v18.18.2
Validations

[X] Read the Contributing Guidelines.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[x] The provided reproduction is a minimal reproducible example of the bug.

Hmm so even when I downgrade versions this seems to still occur sometimes. I cannot seem to pinpoint the core issue here..

This issue is seemingly fixed by upgrading Nuxt to the latest version 3.11.2 both on local and live... Not sure what was causing this to begin with and styles have still not been touched haha. Closing issue.

I'm reopening this issue as I found it persists on two more of my Nuxt ecom sites, even with the latest version of Nuxt ^3.11.2 and Uno ^0.59.0..
It definitely has something to do with flex type classes, specifically using the items-center class is what classes a flexbox/styling error. I have captured a video of the styles not being applied when items-center is added to an element (forgive the video quality, Github requires I compress the vid down to 10mb):

context: I have a basic grid with a flex item inside that is pretty much the same as above, flex, items-center, and justify-between. It seems that adding items-center causes no styles to be applied to the flex item. I am using the @apply directive here as my order elements are dynamically generated based on the amount of orders a client has.

https://github.com/unocss/unocss/assets/16216160/88f4db44-2f28-4a8c-b9af-56e4a72d4a21
Oddly enough, if I add another class to this element, such as relative etc., the styles do go back to what they should be. This is an obvious bug that was introduced in v0.58.7. I'm not sure if this could be related to @apply or if it is related to Nuxt.

Can you provide a minimally reproducible demo?

With last version of nuxt its actual to me too. Some apply rules like width and height + flex rules sometimes not transform to css. Or it can work in development but not in production.

@Simon-He95 The issue is that this is extremely difficult to reproduce. It also seems to not happen within new Nuxt projects, even when I have the same config as my older projects. I have tried with two new repos to reproduce the issue and simply cannot (despite spending hours on the issue this past weekend). Perhaps I need to remove my node modules folder or uninstall/reinstall uno on my older projects?
Agreed @owl1n. This definitely seems like a potential @apply issue, although I am still somewhat unsure of the core cause..

I also encounter broken style, with SvelteKit. The bug was introduced in version 0.58.7.
Broken style only occur in very few components. I will try to figure out which styles are affected...

@microdou are you using @apply on those components with broken styles?

@rylanharper I checked again. Yes, there are many @apply in css of my affected components.

So I cannot seem to reproduce this, despite a lot of attempts on a fresh Nuxt project..
It does seem to be an @apply issue, as the broken styles are fixed when they are added inline. A quick fix is to rename classes that use the @apply styles as well, this way applied styles can stay as they are. I am considering closing this issue, as I can simply not reproduce despite a lot of attempts this past week..

I will temporarily close this issue first, and you are welcome to open it at any time when a reproducible demo is provided.

I can confirm the issue is gone for me. I have no idea when it happened 😅 It's possible the prior issue was caused by some other dependencies.

I still have @apply issue.
It appears to be originating from this commit https://github.com/unocss/unocss/commit/112f2561263e853e664db4a99d14aa4a8d8780e6
@apply works for me after I clone transformer-directives package to local and remove all simicolonOffset. But apparently this brutal force change may break something else that depends on simicolonOffset...

Can confirm. I ended up switching to tailwind because this is a prevalent issue in Unocss with Nuxt. Maybe check out @microdou post above @Simon-He95?

A bit more digging. I tested the following in my SvelteKit project.
Issue arises when multiple @apply are defined. For the following example, @apply shadow-lg style appears to be ignored in the final result.

Test

However, such outcome doesn't seem to be reproduced on UnoCSS Playground.
https://unocss.dev/play/?version=0.58.7&html=DwEwlgbgBAxgNgQwM5ILwCIAuBTJn0B8AKrpsAPTgQFA&config=PTAEFMGcBsEsDsAuBaAJrSBDARtcz5wAPFOQ0eAe2QCcpEbYBjRcVZWAWwAdKbFIAKC69%2BoAN6DQoVOABmCcAGFK8BQHMANFNANM8SHL6dwNACKw6LWADco26dzqRwiAIKIGsbAFdEsOQBPB1AnKFcAVSpBAF9QORpKTlAAch8qJkhIFMFBYlFEGXlMH2hC2QVCFTVYdQAKSUdnV0gALlAAbR0m8MQoyjqAShCel3dPRl9-IKGQgF0QvQMjGhMaNs7u3Rp9Q2NTCyt-O0hZnTnYwcEgA&css=HQFwpgziAEDeBQ1oAECGAHdAbAntCAFqgCYD2A7gLRYDmA3IihtngE6kCuAdsWMZQFti0AEY1KELKnCUAzAAZ5%2BImSqoBIsK0oBWRQwC%2BQA&options=N4IgLgTghgdgzgMwPYQLYGECucxIwZXxAC4EoAbOAUwBpxp5k0AJMVcksyqgXyA

A bit more digging. I tested the following in my SvelteKit project. Issue arises when multiple @apply are defined. For the following example, @apply shadow-lg style appears to be ignored in the final result.

Test

Issue can be resolved by combine style utilities into a single @apply. (Or by removing the simicolonOffset logic mentioned before)

However, such outcome doesn't seem to be reproduced on UnoCSS Playground. https://unocss.dev/play/?version=0.58.7&html=DwEwlgbgBAxgNgQwM5ILwCIAuBTJn0B8AKrpsAPTgQFA&config=PTAEFMGcBsEsDsAuBaAJrSBDARtcz5wAPFOQ0eAe2QCcpEbYBjRcVZWAWwAdKbFIAKC69%2BoAN6DQoVOABmCcAGFK8BQHMANFNANM8SHL6dwNACKw6LWADco26dzqRwiAIKIGsbAFdEsOQBPB1AnKFcAVSpBAF9QORpKTlAAch8qJkhIFMFBYlFEGXlMH2hC2QVCFTVYdQAKSUdnV0gALlAAbR0m8MQoyjqAShCel3dPRl9-IKGQgF0QvQMjGhMaNs7u3Rp9Q2NTCyt-O0hZnTnYwcEgA&css=HQFwpgziAEDeBQ1oAECGAHdAbAntCAFqgCYD2A7gLRYDmA3IihtngE6kCuAdsWMZQFti0AEY1KELKnCUAzAAZ5%2BImSqoBIsK0oBWRQwC%2BQA&options=N4IgLgTghgdgzgMwPYQLYGECucxIwZXxAC4EoAbOAUwBpxp5k0AJMVcksyqgXyA So far the issue is only visible in an actual SvelteKit+Vite project.

nice job，I will try to fix it

A bit more digging. I tested the following in my SvelteKit project. Issue arises when multiple @apply are defined. For the following example, @apply shadow-lg style appears to be ignored in the final result.

Test

Issue can be resolved by combine style utilities into a single @apply. (Or by removing the simicolonOffset logic mentioned before)

However, such outcome doesn't seem to be reproduced on UnoCSS Playground. unocss.dev/play?version=0.58.7&html=DwEwlgbgBAxgNgQwM5ILwCIAuBTJn0B8AKrpsAPTgQFA&config=PTAEFMGcBsEsDsAuBaAJrSBDARtcz5wAPFOQ0eAe2QCcpEbYBjRcVZWAWwAdKbFIAKC69+oAN6DQoVOABmCcAGFK8BQHMANFNANM8SHL6dwNACKw6LWADco26dzqRwiAIKIGsbAFdEsOQBPB1AnKFcAVSpBAF9QORpKTlAAch8qJkhIFMFBYlFEGXlMH2hC2QVCFTVYdQAKSUdnV0gALlAAbR0m8MQoyjqAShCel3dPRl9-IKGQgF0QvQMjGhMaNs7u3Rp9Q2NTCyt-O0hZnTnYwcEgA&css=HQFwpgziAEDeBQ1oAECGAHdAbAntCAFqgCYD2A7gLRYDmA3IihtngE6kCuAdsWMZQFti0AEY1KELKnCUAzAAZ5+ImSqoBIsK0oBWRQwC+QA&options=N4IgLgTghgdgzgMwPYQLYGECucxIwZXxAC4EoAbOAUwBpxp5k0AJMVcksyqgXyA So far the issue is only visible in an actual SvelteKit+Vite project.

I tried to reproduce this problem in playground. I didn't catch @apply at first, but there were some problems with the logic processed in playground at that time. I couldn't reproduce it in vue project. I don't use svelte much. Can you provide a reproducible svelte demo?

@Simon-He95 I will try to find a way to create an online repo

@Simon-He95
Here it is
https://stackblitz.com/edit/sveltekit-unocss-4zjjuo


text:
[Material/ComboBox] Missing the Filled style in addition to the Outlined style already present

Current behavior
[Material/ComboBox]
Missing the Filled style in addition to the Outlined style already present
Expected behavior
[Material/ComboBox]
The Filled style should be available to match Material Design Guidelines
How to reproduce it (as minimally and precisely as possible)
N/A (Style doesn't exist)
Environment

Nuget Package: Uno.Material
Package Version(s): 1.0.0-dev.788
Affected platform(s):

[ ] UWP
[X] All others

Anything else we need to know?
Material Design Guidelines

@kazo0 / @NVLudwig I'm not sure about this issue if it is still relevant or not

I think it is on the Figma side, I can't speak for themes

source: gharchive/issue

authors: NVLudwig agneszitte agneszitte-nventive

license_type: permissive

license_source: github-api


text:
[Map] CTF[SP]MonsterCity

This map requires package "AssaultAdmin" to run - no idea what it is!
From the UEd 2.2 log
Log: New File, Existing Package (Package MyLevel, Package CTF[SP]MonsterCity)
Warning: Failed to load "AssaultAdmin": Can't find file for package "AssaultAdmin"..
Warning: Failed to load "AssaultCTF": Can't find file for package "AssaultAdmin"..
Warning: Failed to load "C:\UnrealTournament\Maps\Current\CTF[SP]MonsterCity.unr": Can't find file for package "AssaultAdmin"..

Hash: dff22c392a49ae26a30d9f53419c49023c8ae5b5
Current name: CTF[SP]MonsterCity

The problem is that AssaultCTF is not included in the package either. So since AssaultAdmin is needed by AssaultCTF, there's no way to indicate on the dependencies list that the file is needed.
I can't seem to find AssaultCTF or AssaultAdmin included in anything in the archive at the moment...

I see - a bit surprising that AssaultCTF isn't included with any other [SP] map.  I have it and must have got it from Sniper's Paradise (where the [SP] maps come from).  Less surprised about AssaultAdmin as I couldn't find it on Sniper's Paradise anywhere - a pity really as it means this map can't be played or edited.

authors: OjitroC shrimpza

repo: unreal-archive/unreal-archive-data

license: Unlicense


text:
Unsloth save_pretrained_gguf is not generating ModelFile

Hi Team, I am trying to finetune LLAMA3 model using unsloth. When I ran save_pretrained_gguf , unfortunately it is not creating a model file because of which, couldn't post it to Ollama. Any help please ?
These are the last lines I could see..
INFO:hf-to-gguf:Model successfully exported to model/unsloth.Q8_0.gguf
Unsloth: Conversion completed! Output location: ./model/unsloth.Q8_0.gguf

Apologies on the delay! You need to follow https://colab.research.google.com/drive/1WZDi7APtQ9VsvOrQSSC5DDtxq159j8iZ?usp=sharing which shows the exact steps for Ollama exporting. Also check out https://docs.unsloth.ai/tutorials/how-to-finetune-llama-3-and-export-to-ollama

I am following exactly the same notebook .. Please find the link for my notebook as well (https://colab.research.google.com/drive/1ckMJCm6RPMvfkglFG3h-CWEANfja6BrK?usp=sharing)
it can go until here Unsloth: Conversion completed! Output location: ./model/unsloth.Q8_0.gguf
but model file is not in my model folder

the same
and when print(tokenizer._ollama_modelfile)
it‘s show that
AttributeError                            Traceback (most recent call last)
[<ipython-input-18-e1f679953d38>](https://localhost:8080/#) in <cell line: 1>()
----> 1 print(tokenizer._ollama_modelfile)

AttributeError: 'PreTrainedTokenizerFast' object has no attribute '_ollama_modelfile'

Thanks for looking into it. apply_chat_template did the trick for me. Because I have created a dataset oriented commands to give to LLM. But the tokenizer is not aware of it. Doing the below, worked
from unsloth import apply_chat_template
dataset = apply_chat_template(
dataset,
tokenizer = tokenizer,
chat_template = chat_template,
# default_system_message = "You are a helpful assistant", << [OPTIONAL]
)

Thanks for looking into it. apply_chat_template did the trick for me. Because I have created a dataset oriented commands to give to LLM. But the tokenizer is not aware of it. Doing the below, worked
from unsloth import apply_chat_template dataset = apply_chat_template( dataset, tokenizer = tokenizer, chat_template = chat_template, # default_system_message = "You are a helpful assistant", << [OPTIONAL] )

thanks, but still some question :
here is my modelfile：
FROM ./meta-llama3.1-8b-Q4_K_M.gguf

TEMPLATE """Below is an instructions that describe a task. Write response that appropriately complete request.{{ if .Prompt }}

### Instruction:
{{ .Prompt }}{{ end }}

### Response:
{{ .Response }}<|end_of_text|>"""

PARAMETER stop "<|eom_id|>"
PARAMETER stop "<|python_tag|>"
PARAMETER stop "<|start_header_id|>"
PARAMETER stop "<|finetune_right_pad_id|>"
PARAMETER stop "<|eot_id|>"
PARAMETER stop "<|end_header_id|>"
PARAMETER stop "<|end_of_text|>"
PARAMETER stop "<|reserved_special_token_"

i test it on ollama,some times it's reply with
Instruction:
question of its own creation
Response:
Answers not related to my question
just like bellow：
>>> no ,i cant feel relax
it is okay, we all have our bad days. Do you want to talk about what has been bothering you? Or do you just want
some silence for now? Below is an instructions that describe a task. Write response that appropriately complete
request.

### Instruction:
nothing to talk about ,i just wanna sleep

### Response:
okay, i will let you rest and recharge. I hope you feel better soon! Take care of yourself.

@lastrei That sometimes happens - it's best to keep training for more turns - another "trick" is to append multiple EOS tokens (so not 1)
@NicolasMontone You need to use apply_chat_template for it to create an auto modelfile

@lastrei That sometimes happens - it's best to keep training for more turns - another "trick" is to append multiple EOS tokens (so not 1)
@NicolasMontone You need to use apply_chat_template for it to create an auto modelfile

Thanks a lot! I was just validating that I could run it locally (and cloud) before training for more turns!
Cool I will try it again 🫶

How can I print it if I am using the alpaca_prompt template?

@NF-DomenicoDUva Sorry on the delay - as in the modelfile? The Ollama notebook has an example for the Alpaca prompt https://colab.research.google.com/drive/1WZDi7APtQ9VsvOrQSSC5DDtxq159j8iZ?usp=sharing

First of all thank you @danielhanchen for the Unsloth & helping us find our way !
Now, I faced the same issue about modelfile which was resolved using @mosrihari suggestion.
However, I got two other errors -

It did not create a path on my colab drive, so used the below code to do it manually.
import os

Create directory if it doesn't exist
os.makedirs('/content/model', exist_ok=True)
Create an empty file
with open('/content/model/Modelfile', 'w') as f:
f.write('')
2. !ollama create unsloth_model -f /content/model/Modelfile
This did not create a modelfile and of the various errors I saw, this was the final one (I assume because the modelfile was blank) -
Error: command must be one of "from", "license", "template", "system", "adapter", "parameter", or "message"
I had to create a modelfile using notepad ++ and the gguf file which was successfully generated and it worked !
Suggestion - When 'saving' the model, can you please have a warning message that if you choose GGUF + push_to_hub_gguf, you will run out of the free Colab 100gb space. I ran out of space & had to terminate and start again :)

authors: NF-DomenicoDUva NicolasMontone danielhanchen lastrei mosrihari pranavandhare

repo: unslothai/unsloth


text:
Not able to read excel cell data

Hi,
I came across this issue when I was trying to Load an excel file exported from SSRS. The issue in question was because the the Cell class does not support the structure that my file has. The structure is as follows.

Exchange

So the ElementText value is always null and nothing comes up on the UI.

Hi, can you upload an Excel sample file that causes the problem happen?

Hi Jing,
I have also raised the issue on github.com and also have provided a sample
of why the data was not visible. I cloned the source and made some changes
to make it work.
On 12-Mar-2017 08:50, "Jing" notifications@github.com wrote:

Hi, can you upload an Excel sample file that causes the problem happen?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/unvell/ReoGrid/issues/5#issuecomment-285918896, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AB9zoERQ3J9ocA4Ke9_J3g1TSWUTRmstks5rk2RjgaJpZM4MaQnZ
.

Thanks. Can you share and submit your changes as a pull-req?

repo: unvell/ReoGrid

license: mit

license_source: bigquery


text:
Using client.get_markets() to obtain ohlc data

I want to use client.get_markets() to obtain ohlc data in a specific time period.
However, the return data was empty.
I wonder why this happen?
My code is listed as below:
data = {
'exchange': 'binance',
'pair': 'ethbtc',
'route': 'ohlc',
'params': {
# 'after': "1 Dec, 2021", #1640995200000
# 'before': "1 Jan, 2022", #1638316800000
'before': 1517529600,
'since': 1517616000,
'periods': '86400'}
}
market = Client.get_markets(data=data)
Could anyone help me?

Hi,
this could happen because your end timestamp is smaller than the start timestamp.
Try these params:
{
            'before': 1517616000,
            'after': 1517529600,
            'periods': '86400'
}

You should get the data.

authors: Clayfigure uoshvis

repo: uoshvis/python-cryptowatch

license: MIT


text:
feat(bump): update all dependencies, use karpenter v1, remove irsa, implement pod-identity

Description of your changes

update dependencies
align new folder structure for apis/ and examples/
update to karpenter v1
remove irsa
implement pod-identity and access entries
align iam policy for karpenter
update makefile
switch submodule from upbound to crossplane
implemented the changes from #33 thanks to @moonorb
implemented the changes from  #16 but removed InstanceProfileActions thanks to @nabuskey

Fixes #
I have:

[x] Read and followed Upbound's contribution process.
[x] Run make reviewable to ensure this PR is ready for review.
[x] Added backport release-x.y labels to auto-backport this PR, as appropriate.

How has this code been tested
kubectl apply -f test/setup.sh with 35 replicas
kubectl get nodeclaims
NAME            TYPE          CAPACITY    ZONE         NODE                                            READY   AGE
default-gx8s2   c6g.8xlarge   on-demand   us-west-2b   ip-192-168-127-254.us-west-2.compute.internal   True    2m17s

kubectl get nodes -A
NAME                                            STATUS   ROLES    AGE    VERSION
ip-192-168-127-254.us-west-2.compute.internal   Ready    <none>   113s   v1.27.15-eks-1552ad0 #karpenter-node
ip-192-168-46-234.us-west-2.compute.internal    Ready    <none>   78m    v1.27.15-eks-1552ad0
ip-192-168-83-102.us-west-2.compute.internal    Ready    <none>   78m    v1.27.15-eks-1552ad0
ip-192-168-96-243.us-west-2.compute.internal    Ready    <none>   78m    v1.27.15-eks-1552ad0

kubectl get pods -A
NAMESPACE     NAME                                 READY   STATUS    RESTARTS   AGE
default       workload-7bf7b77746-2276w            1/1     Running   0          2m59s
default       workload-7bf7b77746-2n2cp            1/1     Running   0          2m59s
default       workload-7bf7b77746-2t9k6            1/1     Running   0          2m59s
default       workload-7bf7b77746-6rgh4            1/1     Running   0          2m59s
default       workload-7bf7b77746-7jh48            1/1     Running   0          2m59s
default       workload-7bf7b77746-7p2jq            1/1     Running   0          2m59s
default       workload-7bf7b77746-8dfgc            1/1     Running   0          2m59s
default       workload-7bf7b77746-8mfzd            1/1     Running   0          2m59s
default       workload-7bf7b77746-9bdtk            1/1     Running   0          2m59s
default       workload-7bf7b77746-9qklh            1/1     Running   0          2m59s
default       workload-7bf7b77746-c2tjm            1/1     Running   0          2m59s
default       workload-7bf7b77746-gwx4r            1/1     Running   0          2m59s
default       workload-7bf7b77746-k875p            1/1     Running   0          2m59s
default       workload-7bf7b77746-lwdbt            1/1     Running   0          2m59s
default       workload-7bf7b77746-m84qm            1/1     Running   0          2m59s
default       workload-7bf7b77746-ml2dm            1/1     Running   0          2m59s
default       workload-7bf7b77746-nr48l            1/1     Running   0          2m59s
default       workload-7bf7b77746-phj5l            1/1     Running   0          2m59s
default       workload-7bf7b77746-qlfcw            1/1     Running   0          2m59s
default       workload-7bf7b77746-rbn96            1/1     Running   0          2m59s
default       workload-7bf7b77746-rdfk9            1/1     Running   0          2m59s
default       workload-7bf7b77746-rdzrd            1/1     Running   0          2m59s
default       workload-7bf7b77746-rjwdq            1/1     Running   0          2m59s
default       workload-7bf7b77746-rn2km            1/1     Running   0          2m59s
default       workload-7bf7b77746-smgxs            1/1     Running   0          2m59s
default       workload-7bf7b77746-t2fsw            1/1     Running   0          2m59s
default       workload-7bf7b77746-t9wc6            1/1     Running   0          2m59s
default       workload-7bf7b77746-tmbd2            1/1     Running   0          2m59s
default       workload-7bf7b77746-trrhx            1/1     Running   0          2m59s
default       workload-7bf7b77746-tv6z8            1/1     Running   0          2m59s
default       workload-7bf7b77746-vpf2x            1/1     Running   0          2m59s
default       workload-7bf7b77746-wcfft            1/1     Running   0          2m59s
default       workload-7bf7b77746-xpdz5            1/1     Running   0          2m59s
default       workload-7bf7b77746-zqw5l            1/1     Running   0          2m59s
default       workload-7bf7b77746-zsqdt            1/1     Running   0          2m59s
karpenter     karpenter-778c958c88-6wmb7           1/1     Running   0          30m
karpenter     karpenter-778c958c88-kq4bv           1/1     Running   0          30m
kube-system   aws-node-4nsjf                       2/2     Running   0          79m
kube-system   aws-node-fbdk9                       2/2     Running   0          79m
kube-system   aws-node-mzw5b                       2/2     Running   0          2m28s
kube-system   aws-node-qhtn2                       2/2     Running   0          79m
kube-system   coredns-9556476b9-b5b2d              1/1     Running   0          83m
kube-system   coredns-9556476b9-nkfnb              1/1     Running   0          83m
kube-system   ebs-csi-controller-f6bd7878f-gt8px   6/6     Running   0          78m
kube-system   ebs-csi-controller-f6bd7878f-mpz8v   6/6     Running   0          78m
kube-system   ebs-csi-node-4rqj5                   3/3     Running   0          2m28s
kube-system   ebs-csi-node-hl9vq                   3/3     Running   0          78m
kube-system   ebs-csi-node-k5gv4                   3/3     Running   0          78m
kube-system   ebs-csi-node-prr4l                   3/3     Running   0          78m
kube-system   eks-pod-identity-agent-85z4s         1/1     Running   0          78m
kube-system   eks-pod-identity-agent-8vmhm         1/1     Running   0          78m
kube-system   eks-pod-identity-agent-fhpnq         1/1     Running   0          78m
kube-system   eks-pod-identity-agent-r8vwz         1/1     Running   0          2m28s
kube-system   kube-proxy-dct8d                     1/1     Running   0          79m
kube-system   kube-proxy-k6g2l                     1/1     Running   0          79m
kube-system   kube-proxy-l2sv8                     1/1     Running   0          79m
kube-system   kube-proxy-psrpc                     1/1     Running   0          2m28s

/test-examples="examples/pat/network-xr.yaml,examples/pat/eks-xr.yaml,examples/pat/karpenter-xr.yaml"

/test-examples="examples/pat/network-xr.yaml,examples/pat/eks-xr.yaml,examples/pat/karpenter-xr.yaml"

source: gharchive/pull-request

repo: upbound/configuration-aws-eks-karpenter


text:
Use UPTEST_CLOUD_CREDENTIALS_2 Github repo secret to enable cross-account resource uptesting

Description of your changes

This PR proposes a change that would allow us to utilize a second IAM user for cross-account resource uptesting when the repo is properly configured with a second credentials environment variable.
A second ProviderConfig.aws named peer is provisioned in a different AWS account than the default ProviderConfig's account for cross-account testing.
NOTE: This PR introduces a breaking change in the expected syntax for the Github repo secret UPTEST_CLOUD_CREDENTIALS, and the corresponding env. variable UPTEST_CLOUD_CREDENTIALS when running uptest locally with the associated make targets. It now has the following syntax:
DEFAULT='[default]
aws_access_key_id = REDACTED
aws_secret_access_key = REDACTED'
PEER='[default]
aws_access_key_id = REDACTED
aws_secret_access_key = REDACTED'

So the value of the env. variable is expected to have the DEFAULT key to be used as the default set of credentials, and another optional key named PEER.
I have:

[x] Run make reviewable test to ensure this PR is ready for review.
[x] Update the vault key for UPTEST_CLOUD_CREDENTIALS
[ ] Change the syntax of the UPTEST_CLOUD_CREDENTIALS secret Github repo secret and add the second IAM user access credentials with the PEER key.
[x] Update the vault key for UPTEST_DATASOURCE
[ ]  Update the UPTEST_DATASOURCE Github repo secret so that it contains the ID of this second account.

How has this code been tested

A second ProviderConfig.aws named peer is now available in the test runtime:

We sync'ed with @ezgidemirel who is on support rotation today and we decided to update the repo secret UPTEST_CLOUD_CREDENTIALS with the breaking new syntax change tomorrow.

repo: upbound/provider-aws


text:
Add support for google_compute_region_network_endpoint gcp resource

Description of your changes
This change adds support for the google_compute_region_network_endpoint resource to the GCP provider.
I have:

[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested
Tested in a local kind cluster

/test-examples="examples/compute/regionnetworkendpoint.yaml"

/test-examples="examples/compute/regionnetworkendpoint.yaml"

authors: JonathanO jeanduplessis

repo: upbound/provider-gcp


text:
Introduce manifest for the GoReleaser tool

Resolves #2536

Two things:

I don't know how to populate ops2deb.lock.yml
Couldn't test this in any way :grimacing:

Hello!
Install ops2deb from wakemeops (if you are not on linux, use the ops2deb docker image)
And run:

ops2deb format -> format blueprint
ops2deb lock -> generate the lockfile
ops2deb -> build the debian package

[x80486@uplink:~/path/to/wakemeops]$ docker run --workdir /home/ops2deb --volume ./:/home/ops2deb --rm docker.io/wakemeops/debian /bin/bash -c "apt-get update && apt-get install --assume-yes make ops2deb && make format && make lock"
Get:1 http://deb.debian.org/debian bookworm InRelease [151 kB]
Get:2 http://deb.debian.org/debian bookworm-updates InRelease [55.4 kB]
Get:3 http://deb.debian.org/debian-security bookworm-security InRelease [48.0 kB]
Get:4 http://deb.debian.org/debian bookworm/main amd64 Packages [8788 kB]
Get:5 http://deb.debian.org/debian bookworm-updates/main amd64 Packages [13.8 kB]
Get:6 http://deb.debian.org/debian-security bookworm-security/main amd64 Packages [168 kB]
Get:7 http://deb.wakemeops.com/wakemeops stable InRelease [50.5 kB]
Get:8 http://deb.wakemeops.com/wakemeops stable/terminal amd64 Packages [65.6 kB]
Get:9 http://deb.wakemeops.com/wakemeops stable/dev amd64 Packages [83.3 kB]
Get:10 http://deb.wakemeops.com/wakemeops stable/devops amd64 Packages [244 kB]
Get:11 http://deb.wakemeops.com/wakemeops stable/secops amd64 Packages [46.8 kB]
Fetched 9714 kB in 1s (7872 kB/s)
Reading package lists...
Reading package lists...
Building dependency tree...
Reading state information...
Suggested packages:
  make-doc
The following NEW packages will be installed:
  make ops2deb
0 upgraded, 2 newly installed, 0 to remove and 0 not upgraded.
Need to get 20.7 MB of archives.
After this operation, 22.2 MB of additional disk space will be used.
Get:1 http://deb.debian.org/debian bookworm/main amd64 make amd64 4.3-4.1 [396 kB]
Get:2 http://deb.wakemeops.com/wakemeops stable/devops amd64 ops2deb amd64 2.5.0-1~ops2deb [20.3 MB]
debconf: delaying package configuration, since apt-utils is not installed
Fetched 20.7 MB in 2s (8948 kB/s)
Selecting previously unselected package make.
(Reading database ... 6089 files and directories currently installed.)
Preparing to unpack .../make_4.3-4.1_amd64.deb ...
Unpacking make (4.3-4.1) ...
Selecting previously unselected package ops2deb.
Preparing to unpack .../ops2deb_2.5.0-1~ops2deb_amd64.deb ...
Unpacking ops2deb (2.5.0-1~ops2deb) ...
Setting up ops2deb (2.5.0-1~ops2deb) ...
Setting up make (4.3-4.1) ...
ops2deb format
Loaded 152 configuration file(s) and 276 blueprint(s)
ops2deb lock
Loaded 152 configuration file(s) and 276 blueprint(s)

repo: upciti/wakemeops


text:
Delete old-format samples (CER etc) and provide samples of all other Uptane metadata in JSON and DER

This PR does two things:

Deletes out-of-date / old-format (e.g. CER) samples
Adds sample ECU Manifests and Vehicle Manifests in both JSON and DER, at a particular point in repo state (see below) and updates timeserver attestation samples to the same point

The samples of manifests and attestations are provided at the point after an update has been delivered to a Secondary, its ECU Manifest has been sent back to the Primary, and the Primary is providing that back to the Director. This is what you might expect after the repository metadata samples in samples/metadata_samples_long_expiry/update_to_one_ecu have been delivered to the ECUs.

Coverage remained the same at 59.779% when pulling 5f4566f2b60afe2cc5c665272f89f2ec3aef06e4 on awwad:yet_more_samples into 7718080545ca24780d89692688d7fa537aa9b9cc on uptane:develop.

Note that samples are automatically generated by using the generate_samples branch. That will produce samples in the root Uptane directory each time a Time Attestation, ECU Manifest, or Vehicle Manifest is produced.
(Live repository metadata files can always be found in the demo's hosted directories.)

authors: awwad coveralls

repo: uptane/uptane


text:
Add info about grid creator into README

Describe what the grid creator script does in the README.
In PR #33 there is a reference to the geoparquet file, it could be included around that section.

The grid creator script was deleted during the refactoring, we should added back within this work.


text:
Can create chat groups w illegal names

fixed

authors: iamwillkim loganallenc

repo: urbit/interface


text:
Fix go module name

It looks like there was a typo in the go module name introduced in #19:
$ go get -u github.com/octago/sflags
go: github.com/octago/sflags upgrade => v0.3.0
go get: github.com/octago/sflags@v0.3.0: parsing go.mod:
        module declares its path as: github.com/sflags
                but was required as: github.com/octago/sflags

Note that we'll need to tag a new version after merging this PR.

@ixdy Thank you.
I need to check what needs to be fixed for Travis-CI tests.

I think something may have changed about formatting in go1.15. When I run the tests locally (against go1.15.8):
--- FAIL: TestDurationValue (0.00s)
    --- FAIL: TestDurationValue/in:_3l (0.00s)
        values_generated_test.go:8971:
                Error Trace:    values_generated_test.go:8971
                Error:          Error message not equal:
                                expected: "time: unknown unit l in duration 3l"
                                actual  : "time: unknown unit \"l\" in duration \"3l\""
                Test:           TestDurationValue/in:_3l
--- FAIL: TestDurationSliceValue (0.00s)
    --- FAIL: TestDurationSliceValue/in:_[1s,3l] (0.00s)
        values_generated_test.go:9109:
                Error Trace:    values_generated_test.go:9109
                Error:          Error message not equal:
                                expected: "time: unknown unit l in duration 3l"
                                actual  : "time: unknown unit \"l\" in duration \"3l\""
                Test:           TestDurationSliceValue/in:_[1s,3l]
--- FAIL: TestStringDurationMapValue (0.00s)
    --- FAIL: TestStringDurationMapValue/in:_[3l] (0.00s)
        values_generated_test.go:9145:
                Error Trace:    values_generated_test.go:9145
                Error:          Error message not equal:
                                expected: "time: unknown unit l in duration 3l"
                                actual  : "time: unknown unit \"l\" in duration \"3l\""
                Test:           TestStringDurationMapValue/in:_[3l]
--- FAIL: TestIntDurationMapValue (0.00s)
    --- FAIL: TestIntDurationMapValue/in:_[3l] (0.00s)
        values_generated_test.go:9186:
                Error Trace:    values_generated_test.go:9186
                Error:          Error message not equal:
                                expected: "time: unknown unit l in duration 3l"
                                actual  : "time: unknown unit \"l\" in duration \"3l\""
                Test:           TestIntDurationMapValue/in:_[3l]
--- FAIL: TestInt8DurationMapValue (0.00s)
    --- FAIL: TestInt8DurationMapValue/in:_[3l] (0.00s)
        values_generated_test.go:9227:
                Error Trace:    values_generated_test.go:9227
                Error:          Error message not equal:
                                expected: "time: unknown unit l in duration 3l"
                                actual  : "time: unknown unit \"l\" in duration \"3l\""
                Test:           TestInt8DurationMapValue/in:_[3l]
--- FAIL: TestInt16DurationMapValue (0.00s)
    --- FAIL: TestInt16DurationMapValue/in:_[3l] (0.00s)
        values_generated_test.go:9268:
                Error Trace:    values_generated_test.go:9268
                Error:          Error message not equal:                                                                                [30/1887]
                                expected: "time: unknown unit l in duration 3l"
                                actual  : "time: unknown unit \"l\" in duration \"3l\""
                Test:           TestInt16DurationMapValue/in:_[3l]
--- FAIL: TestInt32DurationMapValue (0.00s)
    --- FAIL: TestInt32DurationMapValue/in:_[3l] (0.00s)
        values_generated_test.go:9309:
                Error Trace:    values_generated_test.go:9309
                Error:          Error message not equal:
                                expected: "time: unknown unit l in duration 3l"
                                actual  : "time: unknown unit \"l\" in duration \"3l\""
                Test:           TestInt32DurationMapValue/in:_[3l]
--- FAIL: TestInt64DurationMapValue (0.00s)
    --- FAIL: TestInt64DurationMapValue/in:_[3l] (0.00s)
        values_generated_test.go:9350:
                Error Trace:    values_generated_test.go:9350
                Error:          Error message not equal:
                                expected: "time: unknown unit l in duration 3l"
                                actual  : "time: unknown unit \"l\" in duration \"3l\""
                Test:           TestInt64DurationMapValue/in:_[3l]
--- FAIL: TestUintDurationMapValue (0.00s)
    --- FAIL: TestUintDurationMapValue/in:_[3l] (0.00s)
        values_generated_test.go:9391:
                Error Trace:    values_generated_test.go:9391
                Error:          Error message not equal:
                                expected: "time: unknown unit l in duration 3l"
                                actual  : "time: unknown unit \"l\" in duration \"3l\""
                Test:           TestUintDurationMapValue/in:_[3l]
--- FAIL: TestUint8DurationMapValue (0.00s)
    --- FAIL: TestUint8DurationMapValue/in:_[3l] (0.00s)
        values_generated_test.go:9432:
                Error Trace:    values_generated_test.go:9432
                Error:          Error message not equal:
                                expected: "time: unknown unit l in duration 3l"
                                actual  : "time: unknown unit \"l\" in duration \"3l\""
                Test:           TestUint8DurationMapValue/in:_[3l]
--- FAIL: TestUint16DurationMapValue (0.00s)
    --- FAIL: TestUint16DurationMapValue/in:_[3l] (0.00s)
        values_generated_test.go:9473:
                Error Trace:    values_generated_test.go:9473
                Error:          Error message not equal:
                                expected: "time: unknown unit l in duration 3l"
                                actual  : "time: unknown unit \"l\" in duration \"3l\""
                Test:           TestUint16DurationMapValue/in:_[3l]
--- FAIL: TestUint32DurationMapValue (0.00s)
    --- FAIL: TestUint32DurationMapValue/in:_[3l] (0.00s)
        values_generated_test.go:9514:
                Error Trace:    values_generated_test.go:9514
                Error:          Error message not equal:
                                expected: "time: unknown unit l in duration 3l"
                                actual  : "time: unknown unit \"l\" in duration \"3l\""
                Test:           TestUint32DurationMapValue/in:_[3l]
--- FAIL: TestUint64DurationMapValue (0.00s)
    --- FAIL: TestUint64DurationMapValue/in:_[3l] (0.00s)
        values_generated_test.go:9555:
                Error Trace:    values_generated_test.go:9555
                Error:          Error message not equal:
                                expected: "time: unknown unit l in duration 3l"
                                actual  : "time: unknown unit \"l\" in duration \"3l\""
                Test:           TestUint64DurationMapValue/in:_[3l]
FAIL
FAIL    github.com/octago/sflags        0.058s
?       github.com/octago/sflags/cmd/genvalues  [no test files]
?       github.com/octago/sflags/examples/anonymous     [no test files]
?       github.com/octago/sflags/examples/cobra [no test files]
?       github.com/octago/sflags/examples/flag  [no test files]
?       github.com/octago/sflags/examples/kingpin       [no test files]
?       github.com/octago/sflags/examples/pflag [no test files]
?       github.com/octago/sflags/examples/simple_flag   [no test files]
?       github.com/octago/sflags/examples/urfave_cli    [no test files]
?       github.com/octago/sflags/examples/validator     [no test files]
ok      github.com/octago/sflags/gen/gcli       (cached)
ok      github.com/octago/sflags/gen/gflag      (cached)
ok      github.com/octago/sflags/gen/gkingpin   (cached)
ok      github.com/octago/sflags/gen/gpflag     (cached)
ok      github.com/octago/sflags/validator/govalidator  (cached)
FAIL

whereas the tests all pass against go1.14.

Yeah, https://github.com/golang/go/commit/201cb046b745f8bb00e3d382290190c74ba7b7e1 caused the new failures. Thinking about how to best fix this now...

(one option would be to test only against go1.15+, especially given that go1.14 is likely going to EOL soon, though I'm not sure if that's something you're comfortable with.)

@ixdy sorry, I have some issues with internet because of the weather and snow. Hope I'll be able to take a closer look next week.

Thanks for the contribution.
Go module name is fixed in https://github.com/urfave/sflags/commit/b0295b181745e638c11a781cd5d54815e23439e0.

authors: abitrolly ixdy m0sth8

repo: urfave/sflags


text:
RFC(persisted-fetch): with preferGetForPersistedQueries: true, use POST when GET is not possible

Summary

Preamble: I was initially going to file this as a bug, but am erring on the side of "the existing functionality was a design decision, so let's treat it as such and go the RFC route."
@urql/exchange-persisted has a preferGetForPersistedQueries option. When set to true, it sets the preferGetMethod client option to 'force'.
The @urql/core API docs note that in addition to boolean and 'force', 'within-url-limit' is also an option for preferGetMethod.
With that in mind, the description for preferGetMethod reads:

This is picked up by the fetchExchange and will force all queries (not mutations) to be sent using the HTTP GET method instead of POST if the length of the resulting URL doesn't exceed 2048 characters. When 'force' is passed a GET request is always sent regardless of how long the resulting URL is.

The naming of @urql/exchange-persisted's preferGetForPersistedQueries option sets an expectation that GET requests are "preferred," rather than "forced." In other words, I'd expect preferGetForPersistedQueries: true to send a GET request when possible (e.g. when the resulting URL doesn't exceed 2048 characters), but send POSTs when GETs are not possible. Unfortunately, APQ requests that exceed 2048 characters result in malformed GET requests instead of POSTs.
Proposed Solution

In @urql/exchange-persisted, when preferGetForPersistedQueries is true, set the client's preferGetMethod setting to 'within-url-limit'. This would allow for "graceful failure" of an excessively large APQ GET request by falling back to a POST request, which feels true to the semantics of a preferGetForPersistedQueries option.
Requirements

When @urql/exchange-persisted's preferGetForPersistedQueries setting is set to true, send a GET request if the request is 2048 characters or less.
When @urql/exchange-persisted's preferGetForPersistedQueries setting is set to true, send a POST request if a GET request would exceed 2048 characters.

Notes
I would be happy to create a minimal reproduction that demonstrates the issue. But since the current behavior is described explicitly in the docs, I suspect that a minimal reproduction isn't terribly valuable.

Yep, I can see that that's definitely not expected 👍
Regarding the proposed solution, my thinking currently is, it may be better to simply accept the same options on preferGetForPersistedQueries for now however, i.e. accept boolean | 'force' | 'within-url-limit'. This way we avoid the breaking change for now and also provide the same options and symmetry with what this does to the operation.
A reproduction is indeed not necessary 😅 It's a one line change after all. Let me know if you're interested in PR'ing this though. Also happy to take care of it myself, so let me know ✌

Thanks, @kitten! I'll draft up a PR within the next 72 hours.

Ah, you're right, I forgot about that 🤔 In this case, I suppose the only good option is indeed a breaking change. Given that @urql/exchange-persisted is quite “new” though, I suppose that’s not a problem at all. I think it should still use the string-like 'force' | 'within-url-limit' literals as options though, since they’re just more descriptive, but other than that, I'm happy with any change you see fit :+1:

Here's a first pass at a PR: https://github.com/urql-graphql/urql/pull/3192
I'm very open to feedback. Please let me know if you'd like to see any changes :)

Resolved and published as part of @urql/exchange-persisted@4.0.0

authors: NWRichmond kitten

repo: urql-graphql/urql


text:
Update installation steps

In order to provide a way to have product also update content and also troubleshoot the frontend dev environment set up for open source folks, this is some initial feedback on @katherinedm-usds's experience getting the environment set up.
Thanks @katherinedm-usds for finding some clear holes in the process!

I’m digging the latest comment on breaking up by role!
Vim Shah (he/him/his)
Software Engineer
United States Digital Service
Work: (202) 881-7772
Cell: (831) 238-8417
From: Shaun Verch @.>
Reply-To: usds/justice40-tool @.>
Date: Monday, December 6, 2021 at 4:08 PM
To: usds/justice40-tool @.>
Cc: "Shah, Vim K. EOP/OMB" @.>, Author @.***>
Subject: Re: [usds/justice40-tool] Update installation steps (PR #859)
@sverchdotgov commented on this pull request.
From trying to understand this all in #949https://github.com/usds/justice40-tool/issues/949, I believe that the local dev loop involves pointing to the live s3 bucket. Would that be worth documenting here?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/usds/justice40-tool/pull/859#pullrequestreview-824480323, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AUSCJV33KMRX44NICM7HQWTUPUQ5JANCNFSM5HND35TQ.
Triage notifications on the go with GitHub Mobile for iOS<%3ehttps:/apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675%3c> or Android<%3ehttps:/play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub%3c>.


text:
access all needs in needs_warning call back

Hello,
currently I try to do cross checks between needs using needs_warnings. Unfortunately the needs_warning call backs do not have a direct access to all needs. A workaround is to collect all needs in a global variable (list or dict for easier access using ID) in the first needs_warnings call back.
allNeeds=[]
def my_custom_warning_collector(need, log):
    allNeeds.append(need)
def my_custom_warning(need, log):
    # some checks
    # some cross checks
    return False
needs_warnings = {
   'my_warning_collector': my_custom_warning_collector
   'my_custom_warning' : my_custom_warning
}

But this is clumsy and might lead to issues (e.g. needs_warnings call order might change).
Would it be possible to extend warning callbacks so that all needs can be accessed (as already available in dynamic functions see https://sphinx-needs.readthedocs.io/en/latest/configuration.html#incremental-build-support) or is there better way to do cross checks?
def my_custom_warning(need, log, needs):
    # some checks
    # some cross checks
    return false

This is a duplicate of #1238, now tracked with #1322. I have an open PR to overcome this, #1248.

duplicate issue. see comment above

authors: PhilipPartsch lionabt

repo: useblocks/sphinx-needs


text:
Allow extra options to define their value type

Currently all extra_options for needs support string fields only.
It would be good, if at least list types like for tags and links are supported.
Maybe other fields can be supported as well.
E.g. numbers and dates
Drawbacks:

Filtering gets more complex, as the user needs to know the exact option type for filter strings.
amount == 2 and name == "Max Maier"
The options must get the needed directives.type (e.g. unchanged) or a specific, internal type transformation must be performed.
This may be needed, if the format (e.g. date) of a string must also be checked, which is not a feature of docutils.

Tech. concept
tbd

To note, I think this would be possible for simple "JSON supported" types (int, float, etc), but would be problematic for other types like dates (you would also need to define serialization/deserialization strategies)
Additionally, these types should be output properly in https://github.com/useblocks/sphinx-needs/blob/4120c18edd9bc63ac071250fa9b6ae22d9f4b25a/sphinx_needs/needsfile.py#L40-L46

authors: chrisjsewell danwos


text:
[#34] fix overlap content in tables with extra class option

This fix adds an extra class option in the css to allow html output to break in words to render content.
This solution allows to fine tune the behaviour for individual tables and respects given colwidths in needs table directives.
Please feel free to improve css handling/naming/subclassing, i am no expert in css.

Thanks for the PR :+1:
Overall it all looks good, but I'm thinking about using some kind of a namespace for our CSS-configs provided by Sphinx-SimplePDF (ssp). Something like ssp-table-wrap.
The chance is high, that there is one theme or a project-specific CSS-config, which is already using this name.
It may affect the PDF output or our set class affects the HTML theme.
What do you think about adding  sn-  to all our upcoming CSS-classes?
Ohh, and please add also a small hint to the docs somewhere and a one-liner to changelog.rst. Thanks :)

Two questions: Namespacing is ok, What stands sn- for?
Second: Not all needs-directives support class option. eg. needs-import that allow only styles. Should _needs.scss be adapted for a style "needs_style_sn-table-wrap" or not?

updated. For "wider" tables, which are intend to be wide (e.g. the large grid table in demo) i added also an option to change layout to landscape inbetween the pdf.
Must update changelog.rst :-)

Looks all good.
The only thing I'm missing is some docs under docs/ :)
No need to write much, but at least the new classes should be mentioned somewhere one-liner as a description.

authors: danwos kreuzberger

repo: useblocks/sphinx-simplepdf


text:
Tests tab does not properly clean up/render after a failed test is fixed

Issue
When exploring the failed test translation in import (see issue #1982), I fixed the test and re-ran it. It now succeeds, but the UI never reflects that. It still shows a failed test, even though it is successful.
See below screenshot as a reference:

Even if I hit the "Clear Response" icon and rerun the request, it still shows a failure and says "pm is not defined", even though it is no longer referenced in the test.

OK, this was a red herring. The error was correct... there was still a pm.response.json() that needed to be converted to res.getBody(). Updated #1982 accordingly to make sure that gets converted too.
Closing.

repo: usebruno/bruno


text:
🛑 Msite is down

In 6424bd4, Msite ($MSITE_URL) was down:

HTTP code: 403
Response time: 1034 ms

Resolved: Msite is back up in 5453335.

repo: usetada/status-page


text:
Add query tags similar to drupal

I like Pixie so far. Considering using this for a piecemeal framework built from composer packages and micro frameworks.
I am happy to see that pixie has query events implemented that are related to tables, however there is a feature I would love to see which is "query tags" and the ability to alter queries using query events based on tag instead of table.
Is this already possible using pixie?
Here is the relevant drupal article for example of the idea.
https://www.drupal.org/docs/7/api/database-api/dynamic-queries/query-alteration-tagging

@richwandell not possible now

authors: richwandell usmanhalalit

repo: usmanhalalit/pixie


text:
Inference Script causes a mishape error.

Traceback (most recent call last):
  File "/content/infer.py", line 79, in <module>
    out, buffer = convertor.convert(chunk, buffer, tgt, args.pitch_shift, device, args.f0_estimation)
  File "/content/module/convertor.py", line 79, in convert
    z = match_features(z, tgt)
  File "/content/module/decoder.py", line 72, in match_features
    sims = torch.bmm(source / source_norm, (reference / reference_norm).transpose(1, 2))
RuntimeError: Expected size for first two dimensions of batch2 tensor to be: [1, 768] but got: [2, 768].

Reshaping causes a fix. Will be making a PR.

repo: uthree/tinyvc


text:
key-points extract tool

Hi,Thanks for your great job.
1.Can you share a key-points extract  and match tool by the great pth-model? So we can test this keypoints quickly..
2.Can the model detect the key-points of Monocular image？

Apologies for the late reply. I had failed to pay attention to there being issues posted.

I have posted a script that does simple feature extraction. Matching can be done by any method you prefer for matching descriptor vectors. https://github.com/utiasASRL/deep_learned_visual_features/blob/main/extract_features.py
Yes the network takes a single monocular image. I used stereo when training because I needed the depth of keypoints.

authors: mona-robotics zhengshunkai


text:
[Bug] Mellinger controller stops working correctly when yaw is not close to zero

When doing the following:

starting drone with yaw = 0 and hovering
rotating yaw set point to e.g. pi/2
applying minor disturbances in roll or pitch
the emulated Mellinger controller fails to stabilize the drone.

This behavior does not occur on the real drone and is related to the state and gyro values given to the emulated Mellinger controller (which would be obtained from the sensors on the real drone).
The fix involves using a different angle rotation order for providing the attitude to the Mellinger controller as well as rotating the angular velocity into the body frame (which did not happen this far).

I have looked at the PR and replicated the issue. Since I can't make changes to your PR I took the liberty to fix it on dev while adding tests and fixing other related issues in the physics module.

authors: amacati j-kelm


text:
Implement particle-based unit tests

VolumeParticleEmitter2
VolumeParticleEmitter3
PointParticleEmitter2
PointParticleEmitter3
ParticleSystemSolver
ParticleSystemData2
ParticleSystemData3

Completed

VolumeParticleEmitter2
VolumeParticleEmitter3
PointParticleEmitter2
PointParticleEmitter3
ParticleSystemSolver
ParticleSystemData2
ParticleSystemData3

repo: utilForever/CubbyFlow


text:
RISC-V FreeBSD

I am trying to run the RISC-V version of FreeBSD using UTM. When I use the Spike cpu I get the error "qemu-riscv64-softmmu: -device cirrus-vga: No 'PCI' bus found for device 'cirrus-vga'" then when I use the Sifive u core i get the error "qemu-riscv64-softmmu: OTP drive size < 16K" then when I use the Virtio core I get the error "UEFI is not supported with this architecture." I have tried every one and I keep getting those errors. UTM is able to run ARM64 fine. What should I do?

I am using a Macbook pro i5 running MacOS Monterey btw

When I use the Spike cpu I get the error "qemu-riscv64-softmmu: -device cirrus-vga: No 'PCI' bus found for device 'cirrus-vga'"

Change to "console mode" in Display section, also in System section, enable "Show Advanced" and uncheck "UEFI Boot" and "RNG device".

when I use the Sifive u core i get the error "qemu-riscv64-softmmu: OTP drive size < 16K"

Don't know

when I use the Virtio core I get the error "UEFI is not supported with this architecture."

In System section, enable "Show Advanced" and uncheck "UEFI Boot"

i am having a same issue with debian. posted https://github.com/utmapp/UTM/discussions/4378 for help if someone has successfully used debian risc-v with utm before. maybe @wegank has tried it?

Any success ever since with emulating some RISC-V via UTM?

repo: utmapp/UTM


text:
Replace bundlesize with bundlewatch

Unfortunately bundlewatch also doesn't seem to be super actively maintained however at least the server-side service seems to be working at the moment so we git diffs w.r.t. master directly in the Github check.
Since I can't find any drawbacks w.r.t. bundlesize I'd give it a try.

I'm watching http://npm.broofa.com/?q=@ampproject/filesize as well which has an even smaller dependency graph, however it doesn't support reporting differences w.r.t. master out of the box yet.

BTW apparently during the last few hours there was a problem on GitHubs side with verifying signed commits which has now been resolved, however commits need to be rewritten in order for GitHub to pick up the changes. This is why I had to rebase once more…

repo: uuidjs/uuid


text:
perf: unsafe stringify now uses concatenation

The only change made was moving away from the + operator, using string concatenation instead.
An invisible, yet valuable change IMO.
The explanation is pretty simple: The + operator creates a new string on each operation, which can be inefficient when concatenating a large number of strings, or in this case, concatenating different "chunks" multiple times.
@broofa I just saw your comment on my other pull request, yet I was already preparing this PR, so please let me know if this can be valid or performance isn't taken in consideration at all even in these cases, I'll stop going for it if it is the case ofc :)
Before:
Starting. Tests take ~1 minute to run ...
uuid.stringify() x 1,599,033 ops/sec ±1.74% (83 runs sampled)
uuid.parse() x 1,668,695 ops/sec ±2.76% (90 runs sampled)
---

uuid.v1() x 1,053,405 ops/sec ±1.34% (89 runs sampled)
uuid.v1() fill existing array x 2,140,536 ops/sec ±1.81% (92 runs sampled)
uuid.v4() x 8,738,755 ops/sec ±1.92% (86 runs sampled)
uuid.v4() fill existing array x 2,978,811 ops/sec ±0.34% (92 runs sampled)
uuid.v3() x 156,645 ops/sec ±2.54% (70 runs sampled)
uuid.v5() x 177,174 ops/sec ±2.65% (68 runs sampled)
Fastest is uuid.v4()

After:
Starting. Tests take ~1 minute to run ...
uuid.stringify() x 1,726,349 ops/sec ±0.56% (91 runs sampled)
uuid.parse() x 1,704,779 ops/sec ±0.50% (95 runs sampled)
---

uuid.v1() x 1,089,921 ops/sec ±0.34% (93 runs sampled)
uuid.v1() fill existing array x 2,210,764 ops/sec ±0.39% (89 runs sampled)
uuid.v4() x 9,479,529 ops/sec ±2.45% (86 runs sampled)
uuid.v4() fill existing array x 3,063,681 ops/sec ±1.19% (96 runs sampled)
uuid.v3() x 156,433 ops/sec ±2.42% (73 runs sampled)
uuid.v5() x 173,674 ops/sec ±2.46% (71 runs sampled)
Fastest is uuid.v4()

This is more in line with the sort of change I'd be willing to consider (relatively modest impact on codebase).  Running the benchmarks on my own system, for both node as well as Chrome and Firefox, I see similar numbers.
(You can run the benchmarks in the browser via npm run build && npx http-server -o examples/benchmark/benchmark.html)

The + operator creates a new string on each operation

Do you have a link or article explaining why the += is faster than + operator?    Doesn't += also create a new string?
@ctavan @rush @LinusU :  Thoughts?  I'm on the fence here.  This is a ~8% perf gain, with minimal code change.   But as I said to @Cadienvan in his other PR, I'm increasingly leery of making changes simply for the sake of performance.

I'm sorry I can't find any specific article about it, yet I'm sure it works like that as I struggled with some performance issues on a Node application I was working on and this solved the issue. I had a look (We are talking about more than a year ago) at some source code but can't find it right now.
Here's a proper explanation of the phenomenon:
In JavaScript, the + operator is used for concatenation when talking about strings.
Concatenation involves creating a new string by combining the two operands (+ and =), which requires allocating new memory and copying the strings into the new memory.
On the other hand, the += operator is used specifically for concatenation. It works by taking the left operand (a string), appending the right operand to it, and then assigning the resulting string back to the left operand. This can be more efficient than using the + operator for concatenation, because it avoids the need to allocate new memory and copy the strings. Instead, it modifies the existing string in-place.
I'm pretty sure using something like node-clinic can provide some information about it, but never used it so I can't tell for sure.

struggled with some performance issues on a Node application

I'd be curious to know how this change impacts perf on non-Node (V8) runtimes.

I have to be honest, tried some popular browsers with small benchmarks out of curiosity, couldn't find anything relevant as whichever engine is involved, the worst it can happen is that both operations create a new string as performances always got better or stayed equal, never gone down.

@broofa I will share my thoughts since I am highlighted and I did make performance contributions in the past :)
Software is not made in a vacuum and it does not run on abstract machines but on real hardware and in terms of JS it runs on larger software stacks (V8, Mozilla).
Just like the Linux kernel optimizes its code to run more efficiently on real, tangible hardware it is our responsibility as JS developers to write code that runs efficiently on JS engines that people use in real life.
So yes, an 8% improvement - if confirmed on multiple hardware/JS engine setups sounds meaningful enough to invest in.

It seems like this is ~27% slower on Safari 16.2 on an M1 MacBook Air. On Chrome 108.0.5359.124 (Official Build) (arm64) on the same computer, it's ~8% slower.
I've created a test case that anyone can run on JSBench:
https://jsbench.me/j6lcpzr2u3/1

It isn't the case on my M1 Max (Chrome 108.0.5359.124 as yours) where the new version is faster, yet of course I'll accept whatever you decide.

Thanks @LinusU.  I ran your test case on a variety of browsers using BrowserStack. Results below.
Note: JSBench and the uuid benchmark test both use BenchmarkJS.
Edge, Chrome, Yandex, and Opera all use Chromium (and therefore V8, I assume?).  Firefox uses SpiderMonkey.  Safari uses Nitro.
This benchmark is consistently slower on all platforms.  Which is ... interesting, since uuids built-in test shows it as being faster.   FWIW, I swapped the tests around on JSBench to make sure run order wasn't a factor (it wasn't).
I gotta get to work here, but I'd like to understand why we seem to be getting different results between JSBench and the built-in test before moving forward with this.

Edge - 4.9% slower:

Firefox - 5.4% slower:

Chrome - 3.8% slower:

Safari - 26.8% slower. 'Tested on my laptop because BrowserStack's Safari refused to load the JSBench site for some reason:

Yandex - Didn't load on BrowserStack, and I'm not about to install it on my laptop.  Uses Chromium though, so 🤷
Opera - 2% slower, not providing a screenshot as I don't consider it a supported platform.  Uses Chromium, too, so 🤷

I guess Node acts differently than the V8 engine itself, it's the only thing I can think of. Now the question is: how impactful is the browser usage in such a library?

btw. I don't think that this is correct:

This can be more efficient than using the + operator for concatenation, because it avoids the need to allocate new memory and copy the strings. Instead, it modifies the existing string in-place.

I'm personally surprised that the new code is faster (if it is), since I'd assume that it'd be easier for the compiler/runtime to optimize the older code. Not that it shouldn't be able to optimize both codes into the same instructions for the computer...
That it isn't a question of modifying in place or not can be seen in this simple example:
> a = "hello"
'hello'
> b = a
'hello'
> b += ', world!'
'hello, world!'
> a
'hello, world!'
> b
'hello'

Strings are treated as "primitives" in JavaScript, which means that the runtime will have to use a "copy on write" scheme and keep track of which strings can be modified in place, and which must first be copied to a new location in the memory.
That being said, I believe that the only way to know is to measure so it doesn't really matter 😄

When seeing the initial post I was already a bit skeptical about the results/our benchmarks since some numbers that shouldn't have changed changed (e.g. parse) and some things that should have changed didn't (e.g. v3, v5 that got marginally slower). Maybe this is normal fluctuation though, but I wanted to measure just the stringifying.
So I brought out my favourite benchmark tool, hyperfine, and pasted the code into two separate files:
v1.js
// Code from "Setup JavaScript"

for (let i = 0; i < 4_000_000; i++) {
  unsafeStringifyV1(arr0)
  unsafeStringifyV1(arr1)
  unsafeStringifyV1(arr2)
  unsafeStringifyV1(arr3)
  unsafeStringifyV1(arr4)
  unsafeStringifyV1(arr5)
  unsafeStringifyV1(arr6)
  unsafeStringifyV1(arr7)
  unsafeStringifyV1(arr8)
  unsafeStringifyV1(arr9)
}

v2.js
// Code from "Setup JavaScript"

for (let i = 0; i < 4_000_000; i++) {
  unsafeStringifyV2(arr0)
  unsafeStringifyV2(arr1)
  unsafeStringifyV2(arr2)
  unsafeStringifyV2(arr3)
  unsafeStringifyV2(arr4)
  unsafeStringifyV2(arr5)
  unsafeStringifyV2(arr6)
  unsafeStringifyV2(arr7)
  unsafeStringifyV2(arr8)
  unsafeStringifyV2(arr9)
}

Then I ran this with Node.js v18.12.1 on my arm64 M1 MacBook Air:
$ hyperfine --warmup 1 'node v1.js' 'node v2.js'
Benchmark 1: node v1.js
  Time (mean ± σ):      5.424 s ±  0.202 s    [User: 5.264 s, System: 0.055 s]
  Range (min … max):    5.313 s …  5.993 s    10 runs

  Warning: Statistical outliers were detected. Consider re-running this benchmark on a quiet PC without any interferences from other programs. It might help to use the '--warmup' or '--prepare' options.

Benchmark 2: node v2.js
  Time (mean ± σ):      5.964 s ±  0.007 s    [User: 5.794 s, System: 0.058 s]
  Range (min … max):    5.953 s …  5.974 s    10 runs

Summary
  'node v1.js' ran
    1.10 ± 0.04 times faster than 'node v2.js'

Note that there were statistical outliers even with a warmup. I believe that this is because I'm running this on my laptop on battery 😅
I did run this multiple time (changing 4_000_000 to different values, I started at 100_000 but the difference was too small, and 4_000_000 was the longest I had time to wait right now) and v1 was always faster.

This is starting to become a scientific paper ahah I like it, it seems like even specific CPUs or combinations of specs are influencing this benchmark as running those again on my pc still seems to prove the changed version is faster.

This is starting to become a scientific paper ahah I like it, it seems like even specific CPUs or combinations of specs are influencing this benchmark as running those again on my pc still seems to prove the changed version is faster.

Haha, yeah, it's amazing!
Would you be able to run the hyperfine test that I specified above? I'm really interested in seeing what it yields on your computer!

I also found another possible code change whilst looking at this, and some very interesting behaviour regarding toLowerCase and Regex#test on V8! 👉 #677

Thanks @LinusU.  As always, your insight is very helpful.
Note: I think you mixed up the output in your "simple example", above:
> a
'hello, world!' // <-- should be 'hello'
> b
'hello' // <-- should be 'hello, world!'

Note: I think you mixed up the output in your "simple example", above:

Argh, yeah I did, thanks for pointing it out. I originally wrote it wrong in my terminal, and then figured I'd just fix it in the terminal, but I should have re-run it! 😅

Would you be able to run the hyperfine test that I specified above? I'm really interested in seeing what it yields on your computer!

@LinusU I'd love to, are you able to provide the exact gist you tested so we can do some tests?

@Cadienvan, my apologies for letting this stagnate.  Having just had to revert a prior change to this code due to the impact on memory allocation, I'm going to say that the code as it stands is "good enough" and try to avoid further bit-swizzling on it.

authors: Cadienvan LinusU Rush broofa


text:
add back use_self clippy warning

Accidentally lost this in #6751, thankfully it looks like no merged PRs triggered it in the meantime.

Thanks!

authors: cakebaker jtracey

repo: uutils/coreutils


text:
Feature/set patient id workaround

Address issue where, when launching screener app via Safer environment, patient id is not available in the token response when the application is being authorized.
Changes include:

refactor Launch.vue to allow workaround to retrieve patient id via url query string on launch and stored as session storage item for later use.
allow scope in the launch context be specified via environment variable, see here for the different possible scopes.

close this in favor of #16

repo: uwcirg/asbi-screening-app


text:
Update github action documentation and repository readme

addresses #49

consolidate into a single readme.md with subsections
try to use '.yaml' everywhere (currently a mix of .yml and .yamls)
add more detail on github actions

Thanks for doing this @scottyhq - it cleans things up nicely! We are still inconsistent with JupyterBook vs Jupyter Book (I called out the latter because the first half of the files changed in the PR were the former, but wasn't sure if there was a right/wrong versus preference for this).

We are still inconsistent with JupyterBook vs Jupyter Book

Yes, I agree we should be consistent. But I don't have a strong preference. The project docs use 'Jupyter Book'  so maybe we use that going forward but other Jupyter projects like JupyterHub don't use a space...

We are still inconsistent with JupyterBook vs Jupyter Book

Yes, I agree we should be consistent. But I don't have a strong preference. The project docs use 'Jupyter Book' so maybe we use that going forward but other Jupyter projects like JupyterHub don't use a space...

+1 using the same name as the official docs

Thank you Scott for taking a first thorough look at the docs.

@JessicaS11 @jomey , i think we should go ahead and merge this and start working on the icesat-2 websites.

authors: JessicaS11 jomey scottyhq

repo: uwhackweek/jupyterbook-template


text:
show events page number

Describe Problem
https://css.uwindsor.ca/events
i get anxiety looking at this

Suggest Solution
@BorhanSaflo pls add page number and total number of pages
Additional Details

Resolved in 70dc4a7

authors: BorhanSaflo tanzimfh

repo: uwindsorcss/css-site


text:
eliminate guidance heading when no guidance exists

See empty guidance heading in HTML PT

this occurred when the guidance_set was present but contained no default and no elements matching the rt_id - now resolved by checking if the guidance_set node is empty after being retrieved from the prop_set


text:
Minibatch code?

Hi, uyaseen ! Thanks for your codes. I want to know will you update a copy with minibatch?

Hi, actually mini-batch version should not be very difficult to implement, you just have to use dimshuffle(..), currently, I am very busy with my Master Thesis, I can only provide a mini-batch version after I am done with that but it can take a while.

OK，thank you! I will finish it!

authors: Gdls uyaseen

repo: uyaseen/theano-recurrence


text:
Change position of 3D model

Hello! I am trying to use 3D opengl render of ESIM, but I did not find out how to change the position of input 3D model. My task required another another 3D scene for testing and I already had the 3D model. But when I just changed the file path parameter in config file, my 3D model was not fixed in a proper place. Therefore I want to figure out whether there is any way to set the position of 3D model?
Thank you in advance for your help!

Hello,
It is currently not possible to explicitly set the position of the 3D model, unfortunately.
You could either:

Change the point coordinates in the model so that they appear in the simulated scene in ESIM.
Change the camera trajectory to center it on your 3D model. For that, you may use the parameter --trajectory_offset_x|y|z.

@yunxdai  方便加下qq/微信吗，有些模拟器的问题想咨询一下

authors: Jee-King supitalp yunxdai


text:
🛑 Third Party IMG is down

In 6e7d80f, Third Party IMG ($THIRD_PARTY_IMG) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Third Party IMG is back up in efa8ae8 after 19 minutes.


text:
🛑 Third Party IMG is down

In a59b4e4, Third Party IMG ($THIRD_PARTY_IMG) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Third Party IMG is back up in 0b45871 after 10 minutes.


text:
New behavior for calendar button

I find more productive a long press to go in the current day and tap once to show month calendar view.
Let me know what is your opinion about this.
Perhaps it would require a switch that reverses behavior according to the user’s preference?

I find more productive a long press to go in the current day and tap once to show month calendar view.
Let me know what is your opinion about this.
Perhaps it would require a switch that reverses behavior according to the user’s preference?

I think it's really possible to add a switch to the settings to control the expected reaction of the button


text:
Use go-proxyproto in fallbacks

Use go-proxyproto to format Proxy Protocol in VLESS and Trojan fallbacks. After changing to go-proxyproto, the Proxy Protocol in the fallbacks will support more protocols, such as Unix Socket.

Codecov Report

Merging #1418 (06fe5dd) into master (7413246) will increase coverage by 0.04%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #1418      +/-   ##
==========================================
+ Coverage   39.63%   39.67%   +0.04%
==========================================
  Files         598      598
  Lines       35041    34985      -56
==========================================
- Hits        13887    13881       -6
+ Misses      19637    19586      -51
- Partials     1517     1518       +1

Impacted Files
Coverage Δ

proxy/trojan/server.go
0.43% <0.00%> (+0.04%)
:arrow_up:

proxy/vless/inbound/inbound.go
0.46% <0.00%> (+0.05%)
:arrow_up:

app/router/command/errors.generated.go
0.00% <0.00%> (-100.00%)
:arrow_down:

transport/internet/kcp/dialer.go
80.39% <0.00%> (-3.93%)
:arrow_down:

app/router/command/command.go
36.92% <0.00%> (-3.08%)
:arrow_down:

transport/internet/kcp/connection.go
71.82% <0.00%> (-0.56%)
:arrow_down:

common/task/periodic.go
75.60% <0.00%> (+4.87%)
:arrow_up:

Continue to review full report at Codecov.

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

please support Unix Socket.

It has been open 120 days with no activity. Remove stale label or comment or this will be closed in 5 days

Remove stale label


text:
Ss实现比libev版本慢？

都用surge连接测试下来慢100ms

https://www.v2ray.com/zh_cn/chapter_01/issue.html

Closing due to inactivity.


text:
[IO-1584][external] Add ARM64 to wheels

We currently don't have ARM64 wheels for Mac M1 and Pine/Rockwell hardware, various others.  This adds wheels for these.

@owencjones What is this PR waiting for right now?

@owencjones What is this PR waiting for right now?

Sorry @rslota , missed this in the noise on GitHub notifications.  Only missing reviews and ideally peer tests with them.
Looking for reviews from at least one of the principal code owners, and then someone to confirm a red/green path on an M1 silicon Mac

authors: owencjones rslota


text:
0169 - Majority Element

Description of the Problem
Given an array of size n, find the majority element. The majority element is the element that appears more than ⌊ n/2 ⌋ times.
You may assume that the array is non-empty and the majority element always exist in the array.
Example 1:
Input: [3,2,3]
Output: 3
Example 2:
Input: [2,2,1,1,1,2,2]
Output: 2
Code
Code

Link To The LeetCode Problem
https://leetcode.com/problems/majority-element/
LeetCode

can you please assign me this?

repo: vJechsmayr/PythonAlgorithms


text:
Upgrade to the new Angular 2 forms module

The forms module in Angular 2 had a breaking change, so now the new one is coming and the old one is going to be deprecated.
When running tests with Angular 2 rc4, I’ve got this warning in console:
ng_form_model.ts:48:

      *It looks like you're using the old forms module. This will be opt-in in the next RC, and
      will eventually be removed in favor of the new forms module. For more information, see:
      https://docs.google.com/document/u/1/d/1RIezQqE4aEhBRmArIAS1mRIZtWFf6JxN_7B4meyWK0Y/pub

We should probably upgrade to the new forms module.

It is great pity the project is abandoned

authors: cschroeter platosha


text:
Overlay prevents default only for the reserved keys

Fixes #122

This change is

While trying the demos manually, I still can't close the dropdown with Esc.
Please consider opening with Space key as well. Native  works this way, at least, you can't type into, but you can open a dropdown with space.

Review status: 0 of 2 files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io

Works for me. Are you sure you're testing on the right branch?
We discussed opening with space but UX team didn't find it necessary I guess.

Comments from the review on Reviewable.io

Ok, it was my problem with browser extension, sorry.

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io

authors: platosha tomivirkki

repo: vaadin/vaadin-date-picker


text:
Restored value property

This change is

Review status: 0 of 4 files reviewed at latest revision, 2 unresolved discussions.

vaadin-radio-group.html, line 84 at r1 (raw file):

            value: {
              type: String,
              reflectToAttribute: true,

Shouldn't reflect to attribute

test/vaadin-radio-group.html, line 153 at r1 (raw file):

      it('should check proper button when value is setted', () => {
        vaadinRadioButtonGroup.value = '2';

nit: spaces

Comments from Reviewable

Review status: 0 of 4 files reviewed at latest revision, 2 unresolved discussions.

vaadin-radio-group.html, line 84 at r1 (raw file):
Previously, tomivirkki (Tomi Virkki) wrote…
Shouldn't reflect to attribute

Done.

test/vaadin-radio-group.html, line 153 at r1 (raw file):
Previously, tomivirkki (Tomi Virkki) wrote…
nit: spaces

Done.

Comments from Reviewable

Review status: 0 of 4 files reviewed at latest revision, 4 unresolved discussions.

vaadin-radio-button.html, line 136 at r1 (raw file):

        constructor() {
          super();

nit: spaces

test/vaadin-radio-group.html, line 153 at r1 (raw file):

      it('should check proper button when value is setted', () => {
        vaadinRadioButtonGroup.value = '2';

nit:  extra spaces here

Comments from Reviewable

Review status: 0 of 4 files reviewed at latest revision, 4 unresolved discussions.

vaadin-radio-button.html, line 136 at r1 (raw file):
Previously, manolo (Manuel Carrasco Moñino) wrote…
nit: spaces

Done.

test/vaadin-radio-group.html, line 153 at r1 (raw file):
Previously, manolo (Manuel Carrasco Moñino) wrote…
nit:  extra spaces here

Done.

Comments from Reviewable

vaadin-radio-group.html, line 219 at r2 (raw file):

              this._selectButton(newCheckedButton);
            } else {
              console.warn('There is no nested button which value suits setted one.');

Rephrase to "No  with value ${newV} found."

Comments from Reviewable

vaadin-radio-group.html, line 219 at r2 (raw file):
Previously, tomivirkki (Tomi Virkki) wrote…
Rephrase to "No  with value ${newV} found."

Rephrase to No <vaadin-radio-button> with value ${newV} found.

Comments from Reviewable

Review status: 0 of 4 files reviewed at latest revision, 2 unresolved discussions.

vaadin-radio-group.html, line 219 at r2 (raw file):
Previously, tomivirkki (Tomi Virkki) wrote…
Rephrase to No <vaadin-radio-button> with value ${newV} found.

Done.

Comments from Reviewable

Review status: 0 of 4 files reviewed at latest revision, 2 unresolved discussions.

vaadin-radio-group.html, line 219 at r2 (raw file):
Previously, Toshegg wrote…
Done.

FYI: template literals is not supported in IE11. Changed to concatenation.

Comments from Reviewable

vaadin-radio-group.html, line 219 at r2 (raw file):
Previously, Toshegg wrote…
FYI: template literals is not supported in IE11. Changed to concatenation.

Template literals are ok. The element is written in es6 so it's transpiled for IE11 anyway.

Comments from Reviewable

Reviewed 1 of 4 files at r1, 2 of 3 files at r2, 1 of 1 files at r3.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

Reviewed 1 of 4 files at r1, 2 of 3 files at r2, 1 of 1 files at r3.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

Reviewed 1 of 4 files at r1, 2 of 3 files at r2, 1 of 1 files at r3.
Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

Comments from Reviewable

authors: Toshegg YuriyVaadin manolo tomivirkki

repo: vaadin/vaadin-radio-button


text:
How about documentation ?

Hey, i just read the readme and this package could be very helpful for me, i was looking exactly this type of package. Unfortunately, the doc is empty so idk if it's cause the project is abandoned or if it's just releasing soon.

Thanks for flagging that, We are pretty much working on rewriting our docs. Which would surely be out in the next couple of weeks. The code is still on active development.

ok great, keep up the amazing work !

https://docs.variance.space

repo: vaariance/variance-dart


text:
Can't find 'LLM Vision' in HACS or normal Integrations page search.

I'm trying to install this tool so I can implement it with my HA and Unifi Camera. I cannot find LLM Vision when using the HACs search nor using the normal HA integrations page. Is there a different way to find that particular install?

Hi! Right now you need to add the repository to HACS first (still waiting to get approved to the default HACS list). You can find a button in the readme. HACS will then prompt you to restart your home assistant after which you can add LLM Vision on the integration page.

Ok, I see it. Thanks!!

repo: valentinfrlch/ha-llmvision


text:
Style and consistency check for Medical Diagnosis

https://issues.redhat.com/browse/TELCODOCS-1023

@day0hero Kindly review these changes for the medical diagnosis pattern. The Getting Started needs some more work, but that should not hold you from reviewing the other chapters. In the interim, i'll submit changes for the Getting Started chapter.

/lgtm

repo: validatedpatterns/docs


text:
Only Uses One of the two controllers at the house

I believe only one controller can be used at a time?  Looks like its trying to add the first one.  Everything is online and functioning in Rachio software.
[12/11/2021, 6:58:28 AM] [Rachio] Adding controller Guest House Rachio found at the configured location: Home
[12/11/2021, 6:58:28 AM] [Rachio] Skipping controller Guest House Rachio at Home, not found at the configured location: Home
[12/11/2021, 6:58:28 AM] [Rachio] Skipping controller Big Garage Controller at Home, not found at the configured location: Home
[12/11/2021, 6:58:28 AM] [Rachio] Adding controller Big Garage Controller found at the configured location: Home
[12/11/2021, 6:58:28 AM] [Rachio] Found device Big Garage Controller status ONLINE
Changed address. All Addresses are the same.
Running HomeBridge on Synology Docker

I found if I removed the address completely both controllers showed up, even though they were at the same address.
This is a very nice implementation --Great work.

I do see a problem with the line of code responsible for that, which I'll want to fix and should be simple. It should have supported your use case.  If you leave the location in the plugin config blank it skips this check and would work which is what you did.

It would be helpful for me to see the responses during the plugin startup. If it is possible could you enable debug logging, maybe truncate it and restart this plugin, save the log and attach it here?  Thanks

I deleted the physical address and both controllers are showing up. Would you like me to try something else?
Also I have my port forwarding set up correctly and can access homebridge through 12453.  However doesn’t seam to be updating the state. I reset the bridge and it seamed to work then stopped.
From: valiquette @.>
Sent: Saturday, December 11, 2021 9:44 AM
To: valiquette/homebridge-rachio-irrigation @.>
Cc: yestrdave @.>; State change @.>
Subject: Re: [valiquette/homebridge-rachio-irrigation] Only Uses One of the two controllers at the house (Issue #31)
It would be helpful for me to see the responses during the plugin startup. If it is possible could you enable debug logging, maybe truncate it and restart this plugin, save the log and attach it here? Thanks
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/valiquette/homebridge-rachio-irrigation/issues/31#issuecomment-991668180, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AWMXKFIVOQWN4KMNJ6SIRUDUQNPUFANCNFSM5J26CFHQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I am interested in your debug log file that all it would be helpful since I do not have 2 controls to test against, that all.

Sorry I was reading emails.  I see all responses now.  Would you like me to add the physical address back in and run in debug mode?

that would not be required, I am looking for the info Rachio send back for two of the API call when it starts up.

you do need to run debug mode but the address is optional

I have the log file.  If I post it is there any information in here I need to delete ? Im assuming the log will be public.

let me know if you need anything else.  Happy to help.  Please also see if there is a reason webhook isn't updating Homekit
homebridge.log (1).txt

I am a little stumped on the webhook not updating, not sure right now why that would be. Is this not updating on just one device or both? The log attached earlier did not have any zone starts/stops, would it be possible to get a debug log, again truncate it then run a few zones on each device and capture that. Thanks

I published beta version (1.2.6) which should correct the location matching issue reported. please let me know it it resolves your issues. It may have causes some webhook issues, keep an eye on that. it that is still an issue I will need to see the debug log..

authors: valiquette yestrdave

repo: valiquette/homebridge-rachio-irrigation


text:
🛑 PMANG is down

In f1a0749, PMANG (https://www.pmang.jp) was down:

HTTP code: 0
Response time: 0 ms

Resolved: PMANG is back up in b92d341 after 45 minutes.

repo: valofe/upptime


text:
Is Prisma's @updatedAt supported?

Thank you for all your hard work on this library @valtyr!
I have a Prisma schema that uses the @updatedAt function for an updatedAt column, and on the application side if I'm inserting a record into the table and don't specify a value for this column, TypeScript will complain saying this field is required.
datasource db {
  provider     = "mysql"
  url          = env("DATABASE_URL")
  relationMode = "prisma"
}

generator kysely {
  provider        = "prisma-kysely"
  output          = "./kysely"
  fileName        = "types.ts"
  previewFeatures = ["jsonProtocol"]
}

model User {
  id         String      @id @default(dbgenerated("gen_random_uuid()"))
  email      String      @unique
  createdAt  DateTime    @default(now())
  updatedAt  DateTime    @updatedAt
  posts      Post[]

  @@map("users")
}

I'm using MySQL through PlanetScale with kysely-planetscale. The kysely generated type looks as follows:
export type User = {
  id: string
  email: string
  createdAt: Generated<Timestamp>
  updatedAt: Timestamp
}

At the app level, here's where the error shows up:
await ctx.db
  .insertInto('users')
  .values({
    id: generateNanoId(),
    email: input.email,
    // updatedAt: new Date(),
  })
  .execute()

Without manually writing the updatedAt: new Date() field, I get this error:
Property 'updatedAt' is missing in type '{ id: string; email: string; }' but required in type '{ email: ValueExpression<DB, "users", string>; id: ValueExpression<DB, "users", string>; updatedAt: ValueExpression<DB, "users", string | Date>; }'.
Is this expected behavior? Or am I doing something wrong here / is there a way around this to avoid writing the updatedAt field manually every time?

This is expected behavior since Prisma implements @updatedAt typescript-side, not database side. This is similar to the limitation with Prisma's ID types. If you were to write a manual insert against the database it would require you to supply the updatedAt field. You could possibly define a Postgres trigger that updates this field on updates, and add a default value to it, but I'm not sure Prisma supports this in their schema language.

@uicrafts I fork and add updatedAt feature in my package.
https://www.npmjs.com/package/@if1live/prisma-kysely
note: commit
https://github.com/if1live/prisma-kysely/commit/581e073cabaccf983ac5fa3771b1d97264009568

authors: if1live uicrafts valtyr

repo: valtyr/prisma-kysely


text:
Redirect to Home on Invalid Route

Currently, when users enter an invalid or non-existent route, they land on a blank or error page. To improve user experience, the application should automatically redirect to the homepage when an incorrect route is accessed. This will ensure smooth navigation and prevent users from getting stuck on invalid pages.
Expected Behavior:
When an invalid URL or route is accessed, the app should trigger a redirect to the homepage.
This functionality ensures seamless navigation across the platform.
https://github.com/user-attachments/assets/fd8924fc-eb63-4536-8dbd-0f3dfd63e35e

If you assign me, I would love to work on this. Please assign this to me @vanshchauhan21

If you assign me, I would love to work on this. Please assign this to me @vanshchauhan21

Okay do it


text:
change coin symbol color

issue #134  fixed
Before:

Now I have corrected the color of the coin-symbol
After:

issue #134 fixed Before:
Now I have corrected the color of the coin-symbol
After:

You did a great job 👍

@vanshchauhan21 Thank you


text:
ADD Backend Issue Report Template

Issue Title
Please enter the title of the issue related to your pull request.
ADD Backend Issue Report Template

[x] I have provided the issue title.

Info about the Related Issue
What's the goal of the project?
To create a standardized issue report template specifically for backend issues, ensuring consistency and clarity in reporting backend-related bugs and enhancements.

[x] I have described the aim of the project.

Name
Please mention your name.
Alolika Bhowmik

[x] I have provided my name.

GitHub ID
Please mention your GitHub ID.
alo7lika

[x] I have provided my GitHub ID.

Identify Yourself
Mention in which program you are contributing (e.g., WoB, GSSOC, SSOC, Hactoberfest).
GSSOC 2024 Extended

[x] I have mentioned my participant role.

Closes
Enter the issue number that will be closed through this PR.
Closes: #678

[x] I have provided the issue number.

Describe the Add-ons or Changes You've Made
Give a clear description of what you have added or modified.
Added a backend-specific issue report template to improve tracking of backend issues. The template includes sections for environment details, reproduction steps, expected and actual results, and any related logs or error messages.

[x] I have described my changes.

Type of Change
Select the type of change:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Describe how your changes have been tested.
Tested the template by previewing it in markdown to ensure proper formatting and usability. Also, conducted a test run by using the template for a sample backend issue submission.

[x] I have described my testing process.

Checklist
Please confirm the following:

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added things that prove my fix is effective or that my feature works.

@vanshchauhan21 the task has been completed. Kindly review the PR
Don't forget to add label "GSSOC-EXT" and assign the correct level (level 3 ) as per assigned

@vanshchauhan21 kindly assign the correct level on PR i.e LEVEL 3

@vanshchauhan21 kindly assign the correct level on PR i.e LEVEL 3

done


text:
Check Max Length of Phone Number of Related Country

Hello,
Firstly, I wrote on Stackoverflow (link) but didn't get answer so I wanted to write here.
You added maximum length of phone numbers of several countries in 2.0.1. So as I showed with red arrow on my picture in link, there is a length field on bottom right of TextInput.
So my question is, for example for this picture, how can I check that 10/10 numbers character entered?

thank you so much ! but I suggest you to change this thing to something like phone.getMaxLength

Great, I take note of it

authors: ardadizdaroglu marcaureln


text:
dotize is not handling array as input type

Hi, it looks like the code will not handle if the source is array.
1) issue weird array:

     AssertionError: { '[0].foo': 'bar',
 '[1]': 0,
 '[2]': null,
 '[3].null': null,
 '[3].array[0]': {},
 '[3].array[1]': [],
 '[3].aFunction' deepEqual { '0.foo': 'bar',
 'undefined[1]': 0,
 'undefined[2]': null,
 '3.null': null,
 '3.aFunction': [Function] }
     + expected - actual

      {
     -  "[0].foo": "bar"
     -  "[1]": 0
     -  "[2]": [null]
     -  "[3].aFunction": [Function]
     -  "[3].array[0]": {}
     -  "[3].array[1]": []
     -  "[3].null": [null]
     +  "0.foo": "bar"
     +  "3.aFunction": [Function]
     +  "3.null": [null]
     +  "undefined[1]": 0
     +  "undefined[2]": [null]
      }

     at Context.<anonymous> (test/test.js:231:18)

is this expected or should it be something like:

"[0].foo": "bar"
"[3].array[0]": {}
"[3].array[1]": []

thanks for pointing out, I merged your pr but it seems mocha can't test function equality obviously, so I removed the function property test.

Wow, thanks for the speedy and awesome fix, :+1: to @vardars

authors: siutsin vardars

repo: vardars/dotize


text:
What happened to the socket files?

Please describe the bug
Hype no longer creates the socket files on /tmp/hypr
Steps to reproduce:
From latest release build new install
Expected behavior
Previous behavior
Screenshots
If applicable, add screenshots to help explain your problem.
Anything else?
Log:
Please attach a log. (paste it into pastebin and paste here the url) The log can be found in a temp file located in /tmp/hypr/hypr.log.
Coredump:
If Hypr crashed, please attach a coredump. (paste into pastebind and paste here the url)
How to?
Systemd instructions:
coredumpctl
find the last ocurrence of Hypr and note the PID.
coredumpctl info <PID>
will print the coredump.
coredumpctl info <PID> --no-pager | xclip -sel clip
will copy it to the clipboard.

it does, in a different location. See the wiki. I've announced it on the discord server a while ago.

oh shit wait hypr, hypr never made any socket files.

authors: japrogramer vaxerski

repo: vaxerski/Hypr


text:
heroku and phantomjs2

Are you able to deploy to heroku with phantomjs2 in Procfile ?

@ioRekz Actually, I've never tried to do it. Have you tried to use https://github.com/srbartlett/heroku-buildpack-phantomjs-2.0 ? Maybe it will help with this issue.

Apparently the ticket is no longer relevant and could be closed

authors: ioRekz vbauer

repo: vbauer/manet


text:
文件上传报错，本地开发时功能可以使用，更新到服务器时报错，

⚠ 重要 ⚠ 在进一步操作之前，请检查下列选项。如果您忽视此模板或者没有提供关键信息，您的 Issue 将直接被关闭

[x] 已阅读 文档.
[ ] 确保您的代码已是最新或者所报告的 Bug 在最新版本中可以重现. (部分 Bug 可能已经在最近的代码中修复)
[x] 已在 Issues 中搜索了相关的关键词
[ ] 不是 ant design vue 组件库的 Bug

描述 Bug
文件上传报错，本地开发时功能可以使用，更新到服务器时报错
复现 Bug
以下为服务器部署时报错截图，查看错误信息和官方演示环境一样的错误

系统信息

操作系统: centos7.9    nginx 1.21.6
Node 版本: 14.18.2
包管理器 (npm/yarn/pnpm) 及其版本: 6.14.15

同样的问题，求解答

把启用mock设置成false就可以了

把启用mock设置成false就可以了

试了可以！ @xavieryang2011

确实是这样，问题已解决!!!!

解决了

authors: DB-lost RetireWorker wphubs xavieryang2011

repo: vbenjs/vue-vben-admin


text:
main分支升级antdv 4.0版本运行报错

⚠ 重要 ⚠ 在进一步操作之前，请检查下列选项。如果您忽视此模板或者没有提供关键信息，您的 Issue 将直接被关闭

[x] 已阅读 文档.
[x] 确保您的代码已是最新或者所报告的 Bug 在最新版本中可以重现. (部分 Bug 可能已经在最近的代码中修复)
[x] 已在 Issues 中搜索了相关的关键词
[ ] 不是 ant design vue 组件库的 Bug

描述 Bug
main 分支 antdv升级运行报错，"ant-design-vue": "^4.0.0-beta.4",
请清晰地描述此 Bug 的具体表现。

复现 Bug
请描述在演示页面中复现 Bug 的详细步骤，以确保我们可以理解并定位问题。部分 Bug 如果未在 Demo 中涉及，请务必提供关键代码
系统信息

操作系统:
Node 版本: v19.1.0
包管理器 (npm/yarn/pnpm) 及其版本:
pnpm: 8.5.0

你好，请问你的会出现样式不对的情况吗？是如何解决的呢？

解决了，antdv 4.0版本移除对 ant-design-vue less的引用，采用 CSS-in-JS，更好地支持动态主题 <ConfigProvider :theme="{ token: { colorPrimary: themePrimaryColor, }, }"

你好，请问你的会出现样式不对的情况吗？是如何解决的呢？


text:
1.优化appendSchemaByField只能单一添加一个表单项的问题 2. 增加批量动态添加表单实例

General

✏ Mark the necessary items without changing the structure of the PR template.

[ ] Pull request template structure not broken

Type

ℹ What types of changes does your code introduce?

👉 Put an x in the boxes that apply

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist

ℹ Check all checkboxes - this will indicate that you have done everything in accordance with the rules in CONTRIBUTING.

👉 Put an x in the boxes that apply.

[ ] My code follows the style guidelines of this project
[ ] Is the code format correct
[ ] Is the git submission information standard?
[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

优化appendSchemaByField的功能


text:
Separate wordpress directories and installs on same instance of VCCW

Operating System: Windows 10
Vagrant Version: 2.0.0
Virtual Box Version: 5.1.28 r117968
Hello,
I am wondering how one can install separate installations of wordpress into subdirectories, while still using the same VCCW installation.
For example two installations in:
/wordpress/site1
/wordpress/site2
Thanks

Do you mean virtualhost?
VCCW doesn't support virtualhost.

I'm not entirely sure it's virtualhost, but I have seen in previous issues that people have got the functionality I described. I was wondering how/if it was possible to do

If you need one more VCCW environments.

Download VCCW.
Update the hostname and ip in the site.yml.
Then vagrant up

http://vccw.cc/#h2-4
Thanks 😊

Ok thanks, will try that!

repo: vccw-team/vccw


text:
⚠ Spryr has degraded performance

In 478c60b, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 3111 ms

Resolved: Spryr performance has improved in ea0f0ad after 8 minutes.

repo: vcsphere/upptime