text:
how do I install the missing dependency and after deleting the heroku file how do i redo the "brew link"

==> Downloading https://homebrew.bintray.com/bottles/heroku-7.12.1.high_sierra.bottle.tar.gz
==> Pouring heroku--7.12.1.high_sierra.bottle.tar.gz
Error: The brew link step did not complete successfully
The formula built, but is not symlinked into /usr/local
Could not symlink bin/heroku
Target /usr/local/bin/heroku
already exists. You may want to remove it:
rm '/usr/local/bin/heroku'
To force the link and overwrite all conflicting files:
brew link --overwrite heroku
To list all files that would be deleted:
brew link --overwrite --dry-run heroku
Possible conflicting files are:
/usr/local/bin/heroku -> /usr/local/lib/heroku/bin/heroku
==> Summary
🍺  /usr/local/Cellar/heroku/7.12.1: 18,332 files, 53.9MB
Installing heroku has failed!
Installing parity
Installing hub
Installing imagemagick
Installing qt@5.5
Installing libyaml
Installing coreutils
Using yarn
Password:
Installing gpg-suite
Installing postgres
Installing redis
Homebrew Bundle failed! 1 Brewfile dependency failed to install.
failed
brew link --overwrite --dry-run heroku
Possible conflicting files are:
/usr/local/bin/heroku -> /usr/local/lib/heroku/bin/heroku
==> Summary
🍺  /usr/local/Cellar/heroku/7.12.1: 18,332 files, 53.9MB
Installing heroku has failed!
Installing parity
Installing hub
Installing imagemagick
Installing qt@5.5
Installing libyaml
Installing coreutils
Using yarn
Installing gpg-suite
Installing postgres
Installing redis
Homebrew Bundle failed! 1 Brewfile dependency failed to install.
failed

I think this is like issue #547, which was resolved by #549. I'll close for now, but feel free to reopen if you see this issue again. Thanks!

authors: composerinteralia dacasanovat

repo: thoughtbot/laptop


text:
Is it necessary to include clear:both to an outer-container?

I have a section underneath my header and I am noticing that when viewing the section in Chrome's inspector, touches the top of the viewport.
When I add clear:both; to the section, the section goes properly underneath the header.

Inside your outer-container you should also have en element with span-columns, even if it's span-columns(12 of 12) or another full-width grid column. The outer-container already contains a clearfix, and that will give height to the container equal to the size of the floated columns within.
@christiantype does that help?

Closing issue, reopen if you still need help.

source: gharchive/issue

authors: christiantype wardpenney whmii

repo: thoughtbot/neat

license: mit

license_type: permissive

license_source: bigquery


text:
Is there a specific version of JQuery required?

Additionally, if there is a specific version of JQuery can you please add it to the README? Thanks.

@ravenwilde did you run into trouble with a specific version of jQuery? Our intent is that Refills should always work with the latest version, or at least the version used on the site!

No problems, I'm just currently working on backwards compatibility for IE 8 and contemplating my options. Any advice?

@ravenwilde You should use 1.x of jQuery, since 2.x intentionally doesn't support IE8. http://jquery.com/browser-support/

yeah that's about what I'd determined... thanks for the confirmation though :)

authors: Magnus-G cllns ravenwilde

repo: thoughtbot/refills


text:
configure: put the "advanced mode" in a new tab instead of a link

Suggestion from Lance. On the patch selection page, instead of having just the list of languages with a small and scary "advanced mode..." link, have two tabs: the default tab would be "Languages", with the current default UI, and the other tab could be "Other patches", "Gameplay patches", "All patches", or something else that tells right away that non-language patches exist, and that could invite a curious user to try some of them. Inside this tab, we would have the same advanced UI, just with a more enticing name.
The aim of this change is to improve discoverability of non-translation patches.
Because, with this change, switching between the two tabs feels more natural, the simple view (with only the languages) should reflect better any change that the advanced view would have done. For example, below the radio buttons, write the list of patches added from advanced mode, and have no radio button checked if the user removed the language from the advanced view.
If we want to try it, we need to implement it in a branch and to do some user testing before merging it:

How does a casual user tasked with installing a translation patch react? Ideally, they should either ignore the new tab, or go to the new tab and quickly go back to the original one (if they go to the new tab and use the advanced mode successfully and without struggle, they're probably too advanced for this test).
How does an user without any set goal act? Do they want to explore that part? Do they ignore it? Should probably be tested on several users of various skill level.
We tell the test user "you can install the patch xxx by xxx using thcrap" (that's the kind of scenario that a Touhou fan seeing a patch creator advertising their patch on Twitter would encounter), and let them try to install it without any further help.

Scenario 3 can be done right after scenario 1 by the same user - they did something simple, we ask for a more advanced task, their prior knowledge won't help. Scenario 2 should be done by a different user / group of users, because both 1 and 2 need a user who never saw the new UI with tabs (or even someone who never used thcrap configure v3 at all).
The part about making sure the simple tab shows the changes in the advanced tab don't need to exist for the user testing, I'm mostly concerned on how they will react to the tabbed UI.

See https://github.com/tudi20/thcrap/tree/page2-tabbed

repo: thpatch/thcrap

license: unlicense


text:
ETCD reachability is only checked if there is a single endpoint

If more than one endpoint is set, there is no check to see if the cluster is reachable. Since we rely on library behavior for this, it might be best to somehow add a check of our own for this

Closing as wontfix, ETCD is deprecated with the introduction of #46, and support for it will be dropped in the next full release

license_source: github-api


text:
Support for hex based TF Chain secrets

Many farms have been created with the TF Connect App, where the farmer only receives a hex based secret for the TF Chain account controlling the twin and the farm. It would be nice to support these farms in the farmerbot.

[x] rmb-rs - already supports this
[x] grid3_client_ts - feature request created here

With support in both, it would just be a matter of adding some logic to detect which form is provided and invoke the services appropriately.

It actually already works, just by subbing the hex secret into the MNEMONIC field in the .env file. All that's left, in that case is to add a note in the documentation that a "mnemonic or hex secret" is accepted.

Ok. I will update the documentation. I'll rename the MNEMONIC field to SECRET, I think it will be more appropriate.

authors: brandonpille scottyeager

repo: threefoldtech/farmerbot


text:
nginx configurations shouldn't use StoredFactory

it needs to keep getting regenerated from the packages, always. we hit this issue after migration all the packages to a single bottle server, and all of the existing old, preconfigured locations still exist on the nginx locations object, meaning it will gets regenerated every time -with the old locations too-. Suggestion is make sure to delete the locations in 3bot start -as a method for an upgrade- and not use a StoredFactory for nginx SAL.

verifying this

Verified
Branch
development
Commit ID
73616bac6fad9857c0f915ecba2e6a7f54307412

authors: RafyAmgadBenjamin xmonader

repo: threefoldtech/js-sdk


text:
init stellar gui

Description
Advanced desktop wallet for stellar for the sdk framework

entrypoint: poetry run stellargui
jsng 'j.tools.stellargui.run()'

Codecov Report

Merging #2055 (ce0d280) into development (57fb5fe) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@               Coverage Diff               @@
##           development    #2055      +/-   ##
===============================================
- Coverage        32.47%   32.45%   -0.02%
===============================================
  Files              136      136
  Lines            13192    13198       +6
===============================================
  Hits              4284     4284
- Misses            8908     8914       +6

Impacted Files
Coverage Δ

jumpscale/packages/tfgrid_solutions/chats/flist.py
33.57% <0.00%> (ø)

...umpscale/packages/tfgrid_solutions/chats/ubuntu.py
40.20% <0.00%> (ø)

jumpscale/sals/marketplace/apps_chatflow.py
13.00% <0.00%> (-0.17%)
:arrow_down:

jumpscale/sals/marketplace/deployer.py
12.35% <0.00%> (-0.04%)
:arrow_down:

Continue to review full report at Codecov.

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

source: gharchive/pull-request

authors: codecov-io xmonader


text:
Builders : create sandbox

create sandbox for the following  builders :
BuilderBitcoin
BuilderEthereum
BuilderRipple
BuilderMinio
BuilderRestic
BuilderSyncthing
BuilderOpenResty

@Dinaamagdy , please update this card

finished and can find code there
8fe86d202d74fe4e38e614a1924ae4f6ae3ce89c

authors: Dinaamagdy rkhamis

repo: threefoldtech/jumpscaleX


text:
Big fail to upload properly

see: https://docs.grid.tf/threefold/proj_bancadati/issues/90

Ok so this is actually not a bug but just a limitation of the archive with the current config it has. Let me explain, when uploading a file, we do process it through a pipeline that will split the files and then erasure code it over multiple shards. All of this generates metadata so we can later on reconstruct the file from the metadata information.
What happens here, is we reach a metadata size that is bigger then what 0-db can accept. 0-DB has a limit of around 8MiB of data per write call. So here, when all the file is uploaded, we try to write the metadata to the tlog, but 0-db refuse cause this metadata block is too big. Since the write to the tlog file, minio signal the write of the file as failed too.
An easy way to solve this is to change the configuration of the minio itself. If you need to store bigger file, then you can for example set the BlockSize of the minio configuration to a Higher value. With a bigger BlockSize, minio will generate less metadata and thus you can store bigger files.
I've create a small sheet where you can play with the different configuration to see what fits your needs.
https://docs.google.com/spreadsheets/d/1M8lTpN00yFul4NH2el3gJ0bN-JJC4-o5HT721GWUYV4/edit?usp=sharing On this sheet, you can edit the blue cells and it will compute the size of the metadata generated for the config and file size you specified. If the result is highlighted in orange, that means you config don't support the file size and you need to tweak the config a bit.

repo: threefoldtech/minio


text:
Incorrect resource packages for multiple solutions

Description
The resource packages for multiple solutions are incorrectly configured.
Caprover
according to the docs the packages should be:
Minimum: {cpu: 1, memory: 1024, diskSize: 50 }
Standard: {cpu: 2, memory: 1024 * 2, diskSize: 100 }
Recommended: {cpu: 4, memory: 1024 * 4, diskSize: 250 }

and currently they are

Funkwhale
according to the docs the packages should be:
Minimum: {cpu: 2, memory: 1024, diskSize: 50 }
Standard: {cpu: 2, memory: 1024 * 2, diskSize: 100 }
Recommended: {cpu: 4, memory: 1024 * 4, diskSize: 250 }

and currently they are

Mattermost
according to the docs the packages should be:
Minimum: {cpu: 1, memory: 1024 * 2, diskSize: 10 }
Standard: {cpu: 2, memory: 1024 * 4, diskSize: 50 }
Recommended: {cpu: 4, memory: 1024 * 4, diskSize: 100 }

and currently they are

Discourse
according to the docs the packages should be:
Minimum: {cpu: 1, memory: 1024 * 2, diskSize: 10 }
Standard: {cpu: 2, memory: 1024 * 2, diskSize: 50 }
Recommended: {cpu: 4, memory: 1024 * 4, diskSize: 100 }

and currently they are

Taiga
according to the docs the packages should be:
Minimum: { cpu: 2, memory: 1024 * 2, diskSize: 100 }
Standard: { cpu: 2, memory: 1024 * 4, diskSize: 150 }
Recommended: { cpu: 4, memory: 1024 * 4, diskSize: 250 }

and currently they are

ownCloud
according to the docs the packages should be:
Minimum: { cpu: 2, memory: 1024 * 16, diskSize: 250 }
Standard: { cpu: 2, memory: 1024 * 16, diskSize: 500 }
Recommended: { cpu: 4, memory: 1024 * 16, diskSize: 1000 }

and currently they are

Subsquid
according to the docs the packages should be:
Minimum: { cpu: 1, memory: 1024 , diskSize: 50 }
Standard: { cpu: 2, memory: 1024 * 2, diskSize: 100 }
Recommended: { cpu: 4, memory: 1024 * 4, diskSize: 250 }

and currently they are

Casperlabs
according to the docs the packages should be:
Minimum: {cpu: 1, memory: 1024 * 4, diskSize: 100 }
Standard: {cpu: 2, memory: 1024 * 16, diskSize: 500 }
Recommended: {cpu: 4, memory: 1024 * 32, diskSize: 1000 }

and currently they are

Wordpress
according to the docs the packages should be:
Minimum: { cpu: 1, memory: 2048 , diskSize: 10 }
Standard: { cpu: 2, memory: 2048 , diskSize: 50 }
Recommended: { cpu: 4, memory: 4096 , diskSize: 100 }

and currently they are

Umbrel
according to the docs the packages should be:
Minimum: { cpu: 2, memory: 2048 , diskSize: 10 }
Standard: { cpu: 2, memory: 4096 , diskSize: 50 }
Recommended: { cpu: 4, memory: 4096 , diskSize: 100 }

and currently they are

Status: pr #351 ready for review

repo: threefoldtech/tfgrid-sdk-ts


text:
alternatives to 'pretty' classnames

because we use the data attribute name purely as an indexing scheme and don't 'accept' names in our function calls, we don't generate 'pretty' classnames, which might make debugging hard. keeping this issue open for alternate solutions.

first attempt - a special prop label which gets added as the data-* attribute's value. not too bad!

have something better now, with tests. closing!

repo: threepointone/react-css

license: MIT


text:
Reward model training implementation fails

Hello. The following is my training script for the reward model, which follows the setting in the paper（also 8 A100 GPUs).  By the way, I used at most 20 min to finish training, but you report 1.5 h, which is weird.
My average statistics for MN, NR, CD& FD are 0.59, 0.54, 0.57, 0.74 , it seems to be a large gap to your result.
Would you mind pointing out the different parts or unveiling your training setting, thx.

### model
model_name_or_path: meta-llama/Llama-3.1-8B-Instruct
trust_remote_code: true

### method
stage: sft
do_train: true
finetuning_type: full
deepspeed: examples/deepspeed/ds_z3_config.json  # choices: [ds_z0_config.json, ds_z2_config.json, ds_z3_config.json]

### dataset
dataset_dir: /dataset/reward_data/
dataset: trainset_reward_llama
template: llama3
cutoff_len: 2048
max_samples: 2000
overwrite_cache: true
preprocessing_num_workers: 16

### output
output_dir: saves/llama3-8b_sft/full/sft
logging_steps: 10
save_steps: 28
plot_loss: true
overwrite_output_dir: true

### train
per_device_train_batch_size: 8
gradient_accumulation_steps: 1
learning_rate: 1.0e-5
num_train_epochs: 3
lr_scheduler_type: cosine
warmup_ratio: 0.01
bf16: true
ddp_timeout: 180000000
save_only_model: true

There are several problems with your configuration:

The cutoff_len should be set to 8196 to avoid cut-off (Though some data may exceed the 8196 limitation).
Try to enlarge the max_samples if the trainset contains more than 2000 samples.
If you run out of memory, try to set per_device_train_batch_size to 1.
Set the num_train_epochs to a bigger one like 5.

Besides, would you report what framework (like vLLM, SGLang) is used to host reward models?

Thanks for your reply.
I have made changes based on your advice. But have little progress.
I follow all settings you mentioned in the paper by llama_factory.
The result after 3 epochs are:
{'MN': 0.48, 'CR': 0.6333333333333334, 'CD': 0.5933333333333333, 'FA': 0.7333333333333334}
after 5 epochs are:
{'MN': 0.6249999999999999, 'CR': 0.42500000000000004, 'CD': 0.7166666666666666, 'FA': 0.8}
I thought the key point is the cutoff_len, however it does not work even I have set it as 8196. I also investigate performance in different inference temperatures, it shows that temperature has little influence.
As I mentioned that you use 1.5h for training, however I only cost 20 min, so I wonder whether there is any technique gap between us, which could cause the difference.
Here is the manipulated file.
### model
model_name_or_path: meta-llama/Llama-3.1-8B-Instruct
trust_remote_code: true

### method
stage: sft
do_train: true
finetuning_type: full
deepspeed: examples/deepspeed/ds_z3_config.json  # choices: [ds_z0_config.json, ds_z2_config.json, ds_z3_config.json]

### dataset
dataset_dir: /dataset/reward_data/
dataset: trainset_reward_llama
template: llama3
cutoff_len: 8096
max_samples: 5000
overwrite_cache: true
preprocessing_num_workers: 16

### output
output_dir: saves/llama3_1-8b_bsft/full/sft
logging_steps: 10
save_steps: 84
plot_loss: true
overwrite_output_dir: true

### train
per_device_train_batch_size: 4
gradient_accumulation_steps: 1
learning_rate: 1.0e-5
num_train_epochs: 5
lr_scheduler_type: cosine
warmup_ratio: 0.01
bf16: true
ddp_timeout: 180000000
save_only_model: true

I would appreciate it if you could help.

Happy New Year First!
Here are the settings we used to train our reward model.
### model
model_name_or_path: meta-llama/Meta-Llama-3.1-8B-Instruct/

### method
stage: sft
do_train: true
finetuning_type: full
deepspeed: examples/deepspeed/ds_z3_config.json

### dataset
dataset: active_rmllama
template: llama3
cutoff_len: 8192
max_samples: 100000
overwrite_cache: true
preprocessing_num_workers: 16

## output
output_dir: saves/llama3-8b-re/full/sft
logging_steps: 10
save_steps: 500
plot_loss: true
overwrite_output_dir: true

### train
per_device_train_batch_size: 1
gradient_accumulation_steps: 2
learning_rate: 1.0e-5
num_train_epochs: 5.0
lr_scheduler_type: cosine
warmup_ratio: 0.01
bf16: true
ddp_timeout: 180000000

We rerun the training and get following scores:
Final result:
Missed-Need (MN) 0.8
Correct-Rejection (CR) 0.9333333333333333
Correct-Detection (CD) 0.9666666666666667
False-Alarm (FA) 1.0
Average: 0.925
Confusion Matrix:
{'TP': 57, 'FP': 6, 'TN': 54, 'FN': 3}
Accuracy: 0.925
Precision: 0.9047619047619048
Recall: 0.95
F1: 0.926829268292683

Besides, we are uploading the reward model to the huggingface.
As for the training time, the 1.5 Hour is the worst estimation with CPU Offloading.

Thanks for your reply. I have made changes based on your advice. But have little progress. I follow all settings you mentioned in the paper by llama_factory.
The result after 3 epochs are: {'MN': 0.48, 'CR': 0.6333333333333334, 'CD': 0.5933333333333333, 'FA': 0.7333333333333334} after 5 epochs are: {'MN': 0.6249999999999999, 'CR': 0.42500000000000004, 'CD': 0.7166666666666666, 'FA': 0.8}
I thought the key point is the cutoff_len, however it does not work even I have set it as 8196. I also investigate performance in different inference temperatures, it shows that temperature has little influence.
As I mentioned that you use 1.5h for training, however I only cost 20 min, so I wonder whether there is any technique gap between us, which could cause the difference.
Here is the manipulated file.
### model
model_name_or_path: meta-llama/Llama-3.1-8B-Instruct
trust_remote_code: true

### method
stage: sft
do_train: true
finetuning_type: full
deepspeed: examples/deepspeed/ds_z3_config.json  # choices: [ds_z0_config.json, ds_z2_config.json, ds_z3_config.json]

### dataset
dataset_dir: /dataset/reward_data/
dataset: trainset_reward_llama
template: llama3
cutoff_len: 8096
max_samples: 5000
overwrite_cache: true
preprocessing_num_workers: 16

### output
output_dir: saves/llama3_1-8b_bsft/full/sft
logging_steps: 10
save_steps: 84
plot_loss: true
overwrite_output_dir: true

### train
per_device_train_batch_size: 4
gradient_accumulation_steps: 1
learning_rate: 1.0e-5
num_train_epochs: 5
lr_scheduler_type: cosine
warmup_ratio: 0.01
bf16: true
ddp_timeout: 180000000
save_only_model: true

I would appreciate it if you could help.

We rerun the experiment with your settings, it seems that the major problem is the big batch size. As the training set is relatively small, to better optimize the model, we recommend using a smaller batch size and having more optimization steps. We will fix the parameters in our paper. Thanks very much for helping us find the mismatched content.

Happy new year and thanks for your detailed explanation. :)
Sorry to bother you in the first day of the year. Please enjoy your first day of 2025 first!
Change of batch size does work, but can not reach the experimental result as you display.
This takes me to the reward evaluation difference. For this part, my code is manipulated based on eval/reward_model_scoring.py, which is only available for the evaluation of the close-sourced LLM.
For the prompt template, I simply use the "system: {SYSTEM}\n user: {user_prompt}\n assistant:" format. May I ask whether there is any difference for the evaluation part?
I would be appreciate it if you were willing to share detailed code for the reward_scoring script for open-source llm!

By the way, I have found a tip:
You mentioned the prompt is
< Task >
Evaluate the task proposed by the proactive assistant as the user.
</ Task >
< Rule >
0. Analyze the current observation to understand your current situation
and requirements.
1. If the proposed task is ‘ null ‘ ( indicating no task is proposed under
the current observation ), follow these steps :
- Accept the ‘ null ‘ task if you believe there is no need for a task.
- Reject the ‘ null ‘ task if you believe a task is needed .
2. Minimize interruptions from the assistant by only accepting tasks that
are valuable.
3. Evaluate the current observation and make a judgment on the proposed
task accordingly.
</ Rule >
< Format >
You should answer with the following JSON format :
{
" thought ": " Give your thoughts first , then provide the judgment of the
task ." ,
" judgment ": " accepted or rejected "
}
</ Format >

However, the llm sometimes miss-understands the meaning of accepted and rejected as need proactive suggestions or not. So I change the prompt by simply adding
- Accept the ‘ null ‘ task if you believe there is no need for a task.
- Reject the ‘ null ‘ task if you believe a task is needed .

at the end of the " judgment ": " accepted or rejected ". The performance increases, this is quite helpful for un-tuned open source llm.

Happy new year and thanks for your detailed explanation. :) Sorry to bother you in the first day of the year. Please enjoy your first day of 2025 first!
Change of batch size does work, but can not reach the experimental result as you display. This takes me to the reward evaluation difference. For this part, my code is manipulated based on eval/reward_model_scoring.py, which is only available for the evaluation of the close-sourced LLM.
For the prompt template, I simply use the "system: {SYSTEM}\n user: {user_prompt}\n assistant:" format. May I ask whether there is any difference for the evaluation part?
I would be appreciate it if you were willing to share detailed code for the reward_scoring script for open-source llm!

The script eval/reward_model_scoring.py should work for both closed-sourced and open-sourced LLM. We organize the context in the format of multi-turn conversations. You can host OpenAI Compatible Server for open source models using frameworks like vLLM or SGLang. See https://docs.vllm.ai/en/latest/serving/openai_compatible_server.html

authors: Elysia-afk luyaxi

repo: thunlp/ProactiveAgent


text:
代码问题

您好，我这边在Windows跑您的代码，Starting training using file E:\C++Code\Word2Vector\datasets\Sougo-T(sample).txt
Vocab size: 462667
Words in train file: 2655924606
462667
1983
success load data
success InitNet
success InitUnigramTable
start train
在训练的时候，// BP
g /= total;
total存在等于0的情况，参数是readme 设置的。麻烦帮忙解答一下，谢谢。

请问还有问题吗？抱歉之前没有收到邮件提醒，只收到了关闭问题的提醒。

repo: thunlp/SE-WRL


text:
🛑 Nyasama is down

In 852260a, Nyasama (https://bbs.nyasama.com/) was down:

HTTP code: 403
Response time: 83 ms

Resolved: Nyasama is back up in a3f390d after 1 day, 2 hours, 33 minutes.

repo: thwiki/status


text:
Search-TssSecret error

Verified issue does not already exist?
Yes
What error did you receive
Search-TssSecret -TssSession $session -SecretName 'synapsevd'
Cannot convert value "@{id=111452; name=AH-Atriuishealth.net\synapsevd; secretTemplateId=6191; secretTemplateName=AH-AD_Service_Accounts; folderId=5290; folderPath=\AH\IT\Management Accounts\Service Accounts;
siteId=46; active=True; checkedOut=False; isRestricted=False; isOutOfSync=False; outOfSyncReason=; lastHeartBeatStatus=Success; lastPasswordChangeAttempt=0001-01-01T00:00:00; responseCodes=; lastAccessed=;
extendedFields=; checkOutEnabled=False; autoChangeEnabled=False; doubleLockEnabled=False; requiresApproval=False; requiresComment=False; inheritsPermissions=False; hidePassword=False;
createDate=2022-08-08T17:46:23; daysUntilExpiration=106; hasLauncher=True; checkOutUserId=-1; checkOutUserName=}" to type "Thycotic.PowerShell.Secrets.Summary". Error: "Cannot convert the "@{id=111452;
name=AH-Atriuishealth.net\synapsevd; secretTemplateId=6191; secretTemplateName=AH-AD_Service_Accounts; folderId=5290; folderPath=\AH\IT\Management Accounts\Service Accounts; siteId=46; active=True;
checkedOut=False; isRestricted=False; isOutOfSync=False; outOfSyncReason=; lastHeartBeatStatus=Success; lastPasswordChangeAttempt=0001-01-01T00:00:00; responseCodes=; lastAccessed=; extendedFields=;
checkOutEnabled=False; autoChangeEnabled=False; doubleLockEnabled=False; requiresApproval=False; requiresComment=False; inheritsPermissions=False; hidePassword=False; createDate=2022-08-08T17:46:23;
daysUntilExpiration=106; hasLauncher=True; checkOutUserId=-1; checkOutUserName=}" value of type "System.Management.Automation.PSCustomObject" to type "Thycotic.PowerShell.Secrets.Summary"."
At C:\Program Files\WindowsPowerShell\Modules\Thycotic.SecretServer\0.61.0\functions\secrets\Search-TssSecret.ps1:274 char:17

...             [Thycotic.PowerShell.Secrets.Summary[]]$restResponse.reco ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidArgument: (:) [], RuntimeException
FullyQualifiedErrorId : InvalidCastConstructorException

Please run the command using -Verbose
VERBOSE: Command invocation: Search-TssSecret -TssSession:TssSessionObject -SearchText:synapsevd -Verbose:True
VERBOSE: Filters: filter.searchText=synapsevd
VERBOSE: Performing the operation GET https://oct.secretservercloud.com/api/v1/secrets?sortBy[0].direction=asc&sortBy[0].name=Name&take=2147483647&filter.includeRestricted=true&filter.searchText=synapsevd
Cannot convert value "@{id=111452; name=AH-Atriuishealth.net\synapsevd; secretTemplateId=6191; secretTemplateName=AH-AD_Service_Accounts; folderId=5290; folderPath=\AH\IT\Management Accounts\Service Accounts;
siteId=46; active=True; checkedOut=False; isRestricted=False; isOutOfSync=False; outOfSyncReason=; lastHeartBeatStatus=Success; lastPasswordChangeAttempt=0001-01-01T00:00:00; responseCodes=; lastAccessed=;
extendedFields=; checkOutEnabled=False; autoChangeEnabled=False; doubleLockEnabled=False; requiresApproval=False; requiresComment=False; inheritsPermissions=False; hidePassword=False;
createDate=2022-08-08T17:46:23; daysUntilExpiration=105; hasLauncher=True; checkOutUserId=-1; checkOutUserName=}" to type "Thycotic.PowerShell.Secrets.Summary". Error: "Cannot convert the "@{id=111452;
name=AH-Atriuishealth.net\synapsevd; secretTemplateId=6191; secretTemplateName=AH-AD_Service_Accounts; folderId=5290; folderPath=\AH\IT\Management Accounts\Service Accounts; siteId=46; active=True;
checkedOut=False; isRestricted=False; isOutOfSync=False; outOfSyncReason=; lastHeartBeatStatus=Success; lastPasswordChangeAttempt=0001-01-01T00:00:00; responseCodes=; lastAccessed=; extendedFields=;
checkOutEnabled=False; autoChangeEnabled=False; doubleLockEnabled=False; requiresApproval=False; requiresComment=False; inheritsPermissions=False; hidePassword=False; createDate=2022-08-08T17:46:23;
daysUntilExpiration=105; hasLauncher=True; checkOutUserId=-1; checkOutUserName=}" value of type "System.Management.Automation.PSCustomObject" to type "Thycotic.PowerShell.Secrets.Summary"."
At C:\Program Files\WindowsPowerShell\Modules\Thycotic.SecretServer\0.61.0\functions\secrets\Search-TssSecret.ps1:274 char:17

...             [Thycotic.PowerShell.Secrets.Summary[]]$restResponse.reco ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidArgument: (:) [], RuntimeException
FullyQualifiedErrorId : InvalidCastConstructorException

Provide a test case or steps to reproduce
Trying to run a Search on a secret that we have access to and is a valid secret.
Expected behavior
To return details on the secret searched for.
What Edition of Secret Server?
Cloud Subscription
What version of Secret Server
Secret Server Cloud
What PowerShell host was used when producing this error
Windows PowerShell ISE (powershell_ise)
PowerShell Host Version
Name                           Value

PSVersion                      5.1.14393.6343
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.14393.6343
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           1.1.0.1

Cannot validate error using module 0.61.8 and SSC US

Sadly this is happening with our version of the module. We are also stuck with using this version for the time being too as it is tied to our PS version. Happy to provide what I can on this end if you think any additional information will help.

We had the same issue with version 0.60.6 of the module, upgrading to 0.61.8 fixed it for us.
$PSversiontable

Name                           Value
----                           -----
PSVersion                      7.4.4
PSEdition                      Core
GitCommitId                    7.4.4
OS                             Microsoft Windows 10.0.17763
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           1.1.0.1
WSManStackVersion              3.0

We're using on prem with version 11.7.000002

Can confirm - I had to update to PS7 and module version 0.61.3 to resolve, but that is going to be painful in some places.  It would be really great if someone found a workaround for older modules.

authors: enphyniti jagger jmackxiii kjetils-labs


text:
Low VRAM after running

My 2GB GPU is capable of running Stable Diffusion.
This script's GPU support also works just fine after generating an image.
After using the script, however, subsequent Stable Diffusion runs fail due to VRAM unavailability. Is the model persisting in VRAM after use? If so, can it be freed between uses?

Hi, thanks for reporting. I'm still pretty new at python and was assuming everything would be garbage collected when it goes out of scope, including the pytorch model in vram.
Some googling seems to confirm this. (https://pytorch.org/docs/stable/notes/cuda.html#cuda-memory-management)
I'm looking into it further ..

I will specifically delete the model before returning from the script in the next version, so we don't need to wait for the GC to kick in.

In v0.1.7 the model is deleted (freed) at the end of the script now, did that fix the problem ?

I had a similar issue while using your script in txt2img on a 8gb vram card yesterday (just out of memory error and couldnt generate any images anymore). Just tested it with v1.7 and it does not seem to happen anymore.

I'm closing this issue as the problem seems to be solved.


text:
feat: tiup: not marking required checks but wait for all to success

Close https://github.com/ti-community-infra/tichi/issues/393
As some of PR in TiUP repo may not run all of the CI jobs, just remove required mark for them and let bot wait to all checks to pass before commit merge.

/ok-to-test

repo: ti-community-infra/configs


text:
config: rename pingcap/ticdc to pingcap/tiflow

This is part job of step-4 in https://github.com/pingcap/ticdc/issues/3749
The change is triggered by:
sed -i "s/pingcap\/ticdc/pingcap\/tiflow/g" $(find . -type f)

/ok-to-test

/ok-to-test

/merge


text:
OpenAPI deepObject serialization for query parameters

First check

[x] I added a very descriptive title to this issue.
[x] I used the GitHub search to find a similar issue and didn't find it.
[x] I searched the FastAPI documentation, with the integrated search.
[x] I already searched in Google "How to X in FastAPI" and didn't find any information.
[x] I already read and followed all the tutorial in the docs and didn't find an answer.
[x] I already checked if it is not related to FastAPI but to Pydantic.
[x] I already checked if it is not related to FastAPI but to Swagger UI.
[x] I already checked if it is not related to FastAPI but to ReDoc.
[x] After submitting this, I commit to:

Implement a Pull Request for a confirmed bug - or implement the feature described in this issue :)

Description
Allow usage of deepObject serialization from OpenAPI 3.
What it could look like
from fastapi import FastAPI

app = FastAPI()

class UserQuery(BaseModel):
    role: str
    firstName: str

@app.get("/")
def read_root(id: Optional[UserQuery]):
    return {"Hello": "World"}

Query:
/users?id[role]=admin&id[firstName]=Alex

Related

https://github.com/tiangolo/fastapi/issues/245#issuecomment-762729917

Links to a good example: https://stackoverflow.com/questions/48491688/how-to-define-parameters-with-square-brackets-in-openapi-swagger

https://github.com/tiangolo/fastapi/issues/203

See also #283 for broader support of encoding styles.
This is the sort of issue where it's not entirely clear if the responsibility of supporting this should be on FastAPI or Pydantic. On one hand, it has to do with value parsing and encoding, which is something FastAPI tends to fully delegate to Pydantic, but on the other hand it's not part of JSON Schema, it's an OpenAPI extension that only applies to route parameter objects, and most of these encoding styles don't make sense in a JSON context (which is what pydantic is generally meant for).
I would like to see these supported as well.

I'm keen to pursue this issue as I think it is a fairly common use-case (for example what to sort by or filter by)
See links on the different supported options below:

OpenAPI
Swagger

The current solution is to create your own parser if you want to support it. However, the only way I have found is to use a regex pattern to display on the OpenAPI spec. See below link for an example of a rough implementation of custom parsing for sortby and filterby
https://gist.github.com/ghandic/21c27470f6797dd856208a2c68f3e43a
Ideally this could look more as follows
from fastapi import QueryMixIn
from pydantic import BaseModel

...

class FilterBy(BaseModel, QueryMixIn):
    name: str
    q: str

    class Config:
        query_style = "deepObject"

class SortBy(BaseModel, QueryMixIn):
    name: str
    by: str

    class Config:
        query_style = "deepObject"

@router.get("/search")
async def search(
    sort: Optional[SortBy] = Query(None, ),
    filter: List[FilterBy] = = Query(None, ),
    limit: int = Query(10, ge=1, le=50),
    page: int = Query(1, ge=1),
) -> List[Record]:
    ...

Then the user could standardize their Query style across their API using inheritance
from fastapi import QueryMixIn
from pydantic import BaseModel

class QueryModel(BaseModel, QueryMixIn):
    class Config:
        query_style = "deepObject"

from custom import QueryModel

...

class FilterBy(QueryModel):
    name: str
    q: str

class SortBy(QueryModel):
    name: str
    by: str

@router.get("/search")
async def search(
    sort: Optional[SortBy] = Query(None, ),
    filter: List[FilterBy] = = Query(None, ),
    limit: int = Query(10, ge=1, le=50),
    page: int = Query(1, ge=1),
) -> List[Record]:
    ...

The tough problem I see would be validation, but since we're using pydantic could the user not use their validators on their custom QueryModel inherited class
Thoughts?

Please see example implementation here: https://github.com/ghandic/FastAPI-deepObject

Ideally, I'd like to not define the types twice, would need to understand more of how the routes get registered for that implementation.

Here is a example of what the OpenAPI spec should look like:
openapi: 3.0.2
info:
  title: FastAPI
  version: 0.1.0
paths:
  /:
    post:
      summary: Demo
      operationId: demo__post
      requestBody:
        content:
          application/json:
            schema:
              $ref: '#/components/schemas/Sort'
        required: true
      responses:
        '200':
          description: Successful Response
          content:
            application/json:
              schema: {}
        '422':
          description: Validation Error
          content:
            application/json:
              schema:
                $ref: '#/components/schemas/HTTPValidationError'
    get:
      summary: Demo
      operationId: demo__get
      parameters:
      - in: query
        name: sort
        schema:
          $ref: '#/components/schemas/Sort'
        style: deepObject.   # ENABLE THIS
        explode: true   # ENABLE THIS
      responses:
        '200':
          description: Successful Response
          content:
            application/json:
              schema: {}
        '422':
          description: Validation Error
          content:
            application/json:
              schema:
                $ref: '#/components/schemas/HTTPValidationError'
components:
  schemas:
    Direction:
      title: Direction
      enum:
        - asc
        - desc
      type: string
      description: An enumeration.
    HTTPValidationError:
      title: HTTPValidationError
      type: object
      properties:
        detail:
          title: Detail
          type: array
          items:
            $ref: '#/components/schemas/ValidationError'
    Sort:
      title: Sort
      required:
        - by
        - direction
      type: object
      properties:
        by:
          title: By
          pattern: ^[a-z]+$
          type: string
          example: name
        direction:
          $ref: '#/components/schemas/Direction'
    ValidationError:
      title: ValidationError
      required:
        - loc
        - msg
        - type
      type: object
      properties:
        loc:
          title: Location
          type: array
          items:
            type: string
        msg:
          title: Message
          type: string
        type:
          title: Error Type
          type: string

For some demo app (proposed solution to be worked on)
from enum import auto

from fastapi import FastAPI, Query
from pydantic import BaseModel, Field
from fastapi_utils.enums import StrEnum

app = FastAPI()

class Direction(StrEnum):
    asc = auto()
    desc = auto()

class Sort(BaseModel):
    by: str = Field(..., example="name", regex="^[a-z]+$")
    direction: Direction

@app.post("/")
def demo(sort: Sort):
    return {}

# NOTE: Currently says:
# AssertionError: Param: sort can only be a request body, using Body(...)
# .... /lib/python3.8/site-packages/fastapi/dependencies/utils.py", line 331, in get_dependant
# This is where we should allow using deepObject
@app.get("/")
def demo(sort: Sort = Query(...)):
    return {}

@tiangolo would you be supportive of this if the effort was put in to build the PR? It would take a good amount of effort and even more testing.

For what it's worth I was able to support the full range of options in Xpresso, maybe that can serve as inspiration for FastAPI: https://xpresso-api.dev/0.42.3/tutorial/query_params/#customizing-deserialization

How about just Query(style="deepObject")?

Rejected in #9768

How about just Query(style="deepObject")?

Rejected in #9768

I don't think Sebastián closed the issue actually, I'll reply there

authors: FlorianLudwig adriangb commonism ghandic sm-Fifteen

repo: tiangolo/fastapi


text:
[Question] File downloading

I'd like the user to be able to download a file I have within the /app directory via an API call. Which is the best way to do this, given that this file can be considerably big?

you can use https://www.starlette.io/responses/#fileresponse
On Wed, Sep 4, 2019 at 3:37 PM driribarne notifications@github.com wrote:

I'd like the user to be able to download a file I have within the /app
directory via an API call. Which is the best way to do this, given that
this file can be considerably big?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tiangolo/full-stack-fastapi-postgresql/issues/58?email_source=notifications&email_token=AAINSPRJK4FFE3WA5KSK2SLQH622FA5CNFSM4ITSA2X2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HJICUMA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAINSPVJ4N4NXPKO4D5TIEDQH622FANCNFSM4ITSA2XQ
.

--
benoit barthelet
http://pgp.mit.edu/pks/lookup?op=get&search=0xF150E01A72F6D2EE

Yes, I had been trying with this but I got stuck... I do not get where to use this async def or if I even have to use it. If someone have done this and would be glad to share a bit more of detail about the implementation I'd be very appreciated :)

Thanks for the help here @euri10 ! :clap: :bow:
Thanks for reporting back and closing the issue @driribarne  :+1:

authors: driribarne euri10 tiangolo

repo: tiangolo/full-stack-fastapi-postgresql


text:
RuntimeError: Error(s) in loading state_dict for CondScoreModel:

I followed the readme:

Training the GraspGF.
Pretrain the pointnet for RL. Then, I got the pt.pt and score.pt. And I put pt.pt and score.pt in the folder: '/home/lq/codes/human-assisting-dex-grasp/Ckpt/gf_lq'.
However, when I ran sh ./rl_train.sh, the error occurred:
Traceback (most recent call last):
File "./Runners/TrainGFPPO.py", line 77, in
runner = GFPPO(vec_env=envs,
File "/home/lq/codes/human-assisting-dex-grasp/./Algorithms/ppo/gf_ppo_update.py", line 178, in init
self.score.load_state_dict(model_dict)
File "/home/lq/miniconda3/envs/dexgrasp-py38-new/lib/python3.8/site-packages/torch/nn/modules/module.py", line 2215, in load_state_dict
raise RuntimeError('Error(s) in loading state_dict for {}:\n\t{}'.format(
RuntimeError: Error(s) in loading state_dict for CondScoreModel:
Missing key(s) in state_dict: "obj_enc.sa1.mlps.0.layer0.conv.weight", "obj_enc.sa1.mlps.0.layer0.bn.bn.weight", "obj_enc.sa1.mlps.0.layer0.bn.bn.bias", "obj_enc.sa1.mlps.0.layer0.bn.bn.running_mean", "obj_enc.sa1.mlps.0.layer0.bn.bn.running_var", "obj_enc.sa1.mlps.0.layer1.conv.weight", "obj_enc.sa1.mlps.0.layer1.bn.bn.weight", "obj_enc.sa1.mlps.0.layer1.bn.bn.bias", "obj_enc.sa1.mlps.0.layer1.bn.bn.running_mean", "obj_enc.sa1.mlps.0.layer1.bn.bn.running_var", "obj_enc.sa1.mlps.0.layer2.conv.weight", "obj_enc.sa1.mlps.0.layer2.bn.bn.weight", "obj_enc.sa1.mlps.0.layer2.bn.bn.bias", "obj_enc.sa1.mlps.0.layer2.bn.bn.running_mean", "obj_enc.sa1.mlps.0.layer2.bn.bn.running_var", "obj_enc.sa2.mlps.0.layer0.conv.weight", "obj_enc.sa2.mlps.0.layer0.bn.bn.weight", "obj_enc.sa2.mlps.0.layer0.bn.bn.bias", "obj_enc.sa2.mlps.0.layer0.bn.bn.running_mean", "obj_enc.sa2.mlps.0.layer0.bn.bn.running_var", "obj_enc.sa2.mlps.0.layer1.conv.weight", "obj_enc.sa2.mlps.0.layer1.bn.bn.weight", "obj_enc.sa2.mlps.0.layer1.bn.bn.bias", "obj_enc.sa2.mlps.0.layer1.bn.bn.running_mean", "obj_enc.sa2.mlps.0.layer1.bn.bn.running_var", "obj_enc.sa2.mlps.0.layer2.conv.weight", "obj_enc.sa2.mlps.0.layer2.bn.bn.weight", "obj_enc.sa2.mlps.0.layer2.bn.bn.bias", "obj_enc.sa2.mlps.0.layer2.bn.bn.running_mean", "obj_enc.sa2.mlps.0.layer2.bn.bn.running_var", "obj_enc.sa3.mlps.0.layer0.conv.weight", "obj_enc.sa3.mlps.0.layer0.bn.bn.weight", "obj_enc.sa3.mlps.0.layer0.bn.bn.bias", "obj_enc.sa3.mlps.0.layer0.bn.bn.running_mean", "obj_enc.sa3.mlps.0.layer0.bn.bn.running_var", "obj_enc.sa3.mlps.0.layer1.conv.weight", "obj_enc.sa3.mlps.0.layer1.bn.bn.weight", "obj_enc.sa3.mlps.0.layer1.bn.bn.bias", "obj_enc.sa3.mlps.0.layer1.bn.bn.running_mean", "obj_enc.sa3.mlps.0.layer1.bn.bn.running_var", "obj_enc.sa3.mlps.0.layer2.conv.weight", "obj_enc.sa3.mlps.0.layer2.bn.bn.weight", "obj_enc.sa3.mlps.0.layer2.bn.bn.bias", "obj_enc.sa3.mlps.0.layer2.bn.bn.running_mean", "obj_enc.sa3.mlps.0.layer2.bn.bn.running_var".
Unexpected key(s) in state_dict: "obj_enc.stn.conv1.weight", "obj_enc.stn.conv1.bias", "obj_enc.stn.conv2.weight", "obj_enc.stn.conv2.bias", "obj_enc.stn.conv3.weight", "obj_enc.stn.conv3.bias", "obj_enc.stn.fc1.weight", "obj_enc.stn.fc1.bias", "obj_enc.stn.fc2.weight", "obj_enc.stn.fc2.bias", "obj_enc.stn.fc3.weight", "obj_enc.stn.fc3.bias", "obj_enc.conv1.weight", "obj_enc.conv1.bias", "obj_enc.conv2.weight", "obj_enc.conv2.bias", "obj_enc.conv3.weight", "obj_enc.conv3.bias", "obj_enc.conv4.weight", "obj_enc.conv4.bias".

这个错误表明在加载 state_dict 时，模型的结构与保存的权重不匹配。具体来说，有一些键在 state_dict 中缺失，而另一些键是多余的。
The command in shell "rl_train.sh" is:
python ./Runners/TrainGFPPO.py
--seed=0
--headless
--num_envs=100
--dataset_type='train'
--score_model_path='/home/lq/codes/human-assisting-dex-grasp/Ckpt/gf_lq'
--t0=0.005
--exp_name="ours"
--run_device_id=0
--constrained \
I can successfully run sh ./rl_train.sh with the pretrained checkpoint (https://drive.google.com/drive/folders/1-_AcEFnnVO9g-CeCPyjleuDYrUdxXtyd) you gave in the readme. And the command in shell "rl_train.sh" is:
python ./Runners/TrainGFPPO.py
--seed=0
--headless
--num_envs=100
--dataset_type='train'
--score_model_path='/home/lq/codes/human-assisting-dex-grasp/Ckpt/gf'
--t0=0.005
--exp_name="ours"
--run_device_id=0
--constrained \

The rl_train process is successful with your pretrained checkpoint:

Can you give some suggestions?

I run it successfully.

repo: tianhaowuhz/human-assisting-dex-grasp


text:
an error when running train.py.

My environment: Python3.6, cuda 10.2, pytorch 1.6.
And when I ran train.py, an ImportError libcudart.so.9.0: No such file or directory.
why it need cuda 9.0？ Is there any problems in my environment？

and the system cuda version is 10.2.

it doesn't need cuda 9.0. when do you get this import error? Import which library?

it doesn't need cuda 9.0. when do you get this import error? Import which library?

in the file det3d/ops/iou3d_nms/init.py,  after this sentence from det3d.ops.iou3d_nms import iou3d_nms_cuda, iou3d_nms_utils, the importerror occured.

ic, seem to be some compilation error for the nms.
export PATH=/usr/local/cuda-10.0/bin:$PATH
export CUDA_PATH=/usr/local/cuda-10.0
export CUDA_HOME=/usr/local/cuda-10.0
export LD_LIBRARY_PATH=/usr/local/cuda-10.0/lib64:$LD_LIBRARY_PATH
bash setup.sh

could you change this path information to your specific Cuda version and then recompile the nms function?

ic, seem to be some compilation error for the nms.
export PATH=/usr/local/cuda-10.0/bin:$PATH
export CUDA_PATH=/usr/local/cuda-10.0
export CUDA_HOME=/usr/local/cuda-10.0
export LD_LIBRARY_PATH=/usr/local/cuda-10.0/lib64:$LD_LIBRARY_PATH
bash setup.sh

could you change this path information to your specific Cuda version and then recompile the nms function?

yes, you are right. I changed the cuda version after compiling cuda extension.
Recompile the cuda extension, it works! Thank you!

repo: tianweiy/CenterPoint


text:
Fixed typo

before before --> before

Thanks.

authors: Cipherwraith treeowl

repo: tibbe/unordered-containers


text:
Added release scripts.

Overview
This change adds a set of scripts to be able to cross compile for different CPU architectures to support changes in #81 . For example, we can compile both MacOS and MacOS M1 or Linux amd64 and Linux arm64 on each platform respectively. These scripts make the assumption that the compilation is happening on the target platform. In an ideal world, we would cross-compile both architecture and platform, but it would require more work to setup our build agents to handle that.
Changes

Added release scripts.

This commit adds release scripts that can cross compile with CGO support for MacOS and Linux for both x86_64 and arm64 based CPU architectures. It assumes that each build is being build on the target platform though. Cross compiling the world on Linux would be ideal, but it would require all of those toolchains to be setup on our build hosts. Doable, but I'll leave it for a future exercise.

Requirements
If you want to release on a Debain based system, you will need to install the ARM compiler aarch64-linux-gnu-gcc, which is included in crossbuild-essential-arm64:
sudo apt install crossbuild-essential-arm64

In order to release on MacOS, you'll need to install gpg to verify signatures:
brew install gnupg

In addition, you will need Xcode 12.2 or later to be able to support the M1 compilation:

Xcode 12.2 and later is a requirement for building universal binaries. Earlier versions of Xcode don’t contain the support needed to build and test universal versions of your macOS code.

Tests
Linux
mark@dev-mark:~/code/src/tidbyt.dev/pixlet|mark/release-scripts ⇒  make release-linux
rm -f pixlet
rm -rf ./build
./scripts/release-linux.sh
Fetching WebP Binaries
Fetched /tmp/libwebp-1.2.2-rc1/linux-x86-64 successfully
Fetched /tmp/libwebp-1.2.2-rc1/linux-arm64 successfully
Building linux_amd64
Built ./build/linux_amd64/pixlet successfully
Building linux_arm64
Built ./build/linux_arm64/pixlet successfully

mark@dev-mark:~/code/src/tidbyt.dev/pixlet|mark/release-scripts ⇒  ./build/linux_amd64/pixlet
Pixlet renders graphics for pixel devices, like Tidbyt

Usage:
  pixlet [command]

Available Commands:
  completion  Generate the autocompletion script for the specified shell
  help        Help about any command
  push        Pushes a webp image to a Tidbyt device
  render      Runs script with provided config parameters.
  serve       Serves a starlark render script over HTTP.

Flags:
  -h, --help   help for pixlet

Use "pixlet [command] --help" for more information about a command.

mark@dev-mark:~/code/src/tidbyt.dev/pixlet|mark/release-scripts ⇒  ./build/linux_arm64/pixlet
zsh: exec format error: ./build/linux_arm64/pixlet

MacOS
mspicer@americano:~/code/src/tidbyt.dev/pixlet|mark/release-scripts ⇒  make release-macos
rm -f pixlet
rm -rf ./build
./scripts/release-macos.sh
Fetching WebP Binaries
Fetched /tmp/libwebp-1.2.2-rc1/mac-arm64 successfully
Fetched /tmp/libwebp-1.2.2-rc1/mac-x86-64 successfully
Building darwin_arm64
Built ./build/darwin_arm64/pixlet successfully
Building darwin_amd64
Built ./build/darwin_amd64/pixlet successfully

mspicer@americano:~/code/src/tidbyt.dev/pixlet|mark/release-scripts ⇒  ./build/darwin_amd64/pixlet
Pixlet renders graphics for pixel devices, like Tidbyt

Usage:
  pixlet [command]

Available Commands:
  completion  Generate the autocompletion script for the specified shell
  help        Help about any command
  push        Pushes a webp image to a Tidbyt device
  render      Runs script with provided config parameters.
  serve       Serves a starlark render script over HTTP.

Flags:
  -h, --help   help for pixlet

Use "pixlet [command] --help" for more information about a command.

mspicer@americano:~/code/src/tidbyt.dev/pixlet|mark/release-scripts ⇒  ./build/darwin_arm64/pixlet
zsh: bad CPU type in executable: ./build/darwin_arm64/pixlet

These are currently dynamically linked. I think we probably want this to be statically linked. Working on that now.

Ok, some updates here.
MacOS
Currently, our released versions of pixlet are dynamically linked for MacOS. We release them through Homebrew which requires webp as a dependency. This will work for both releases on MacOS since the brew formula exist.

I vote we keep the MacOS binaries dynamically linked for now since this appears to be working well.

Linux
For linux, they are dynamically linked with webp being statically linked (-Wl,-Bstatic -lwebp -lwebpdemux -lwebpmux -Wl,-Bdynamic). What's a bit weird is we also release it through Homebrew for Linux that still has the WebP dependency. A Linux ARM version of WebP is not available through Homebrew so this will not work. This doesn't feel like a big deal because who is installing Linux binaries through Homebrew 😄 .
When looking to statically link webp for Linux ARM, I couldn't seem to find a modern version of libwebp statically compiled for Linux arm64 (the one in this PR from Arch Linux is dynamically linked). The one available for Ubuntu is 0.6.1-2.1 whereas the one for Arch is at 1.2.1-2. I then realized this is the version we are currently linking against for Linux builds 😅 .

I vote we stick with the old version of WebP for linux for both CPU architectures so they're at least on the same version. In addition, they both should static link the WebP library so it's portable and can be curled onto a Raspberry Pi V4.

Static everything
One other thing I thought was having a completely static binary. The main gotcha here is that this really isn't supported on MacOS, which came as a surprise to me.

repo: tidbyt/pixlet


text:
[LOOP-4349] updates to Alert to allow muting

https://tidepool.atlassian.net/browse/LOOP-4349

@ps2 At this point, just looking for a initial conceptual review. Formal review will come later.

It seems like isMuted should be a property of the alert delivery system, not an individual alert. The same alert, when the system is muted, would be delivered muted, and in a non-muted context, not be muted. The alert itself doesn't change. What am I missing?

There needs to be a way to get all the muted scheduled alerts and reschedule them as unmuted alerts. So I think there needs to be something in the alert to indicate that it is muted. What alternative are you suggesting?

UNUserNotificationCenter .getPendingNotificationRequests should do that. There's also removePendingNotificationRequests, and removeAllPendingNotificationRequests

Also, there seems to be a need to know how the alerts were delivered (or the system state when the alert was delivered). For example, was critical alerts disabled when a critical alert was delivered. Was Tidepool Loop temp mute enabled when an alert was delivered.

You don't need to know which alerts were muted. You just need to unschedule any scheduled ones, and reschedule with appropriate sound.
As for you question about how the alerts were delivered, could you explain the need there?

You don't need to know which alerts were muted. You just need to unschedule any scheduled ones, and reschedule with appropriate sound.
As for your second question, about how the alerts were delivered, could you explain the need there?

If repeated alerts are blindly updated, this has a negative impact on the repeat interval. Also, this will also unneededly cutter the alert store. It seems better to only reschedule specific alerts.
As for the second, the product/risk needs will be discussed at standup. Let's continue there.

They don't need to be re-issued, cluttering up the alert store, just rescheduled with iOS.

This will still have the same impact of a negative impact on the repeat schedule.
The in-app alert will need to be rescheduled as well. The best way is to use the existing alert system interface. Maybe we can chat more about this in dev sync? I don't think we are on the same page.

Making backgroundContent non mutable is good. Adding mute flag to SettingsStore is good as we discussed, but not sure the type is right. Lastly, making the sound non-optional I think is incorrect. If you want to make it non-optional, we'd need to add a '.default' sound.

The .default sound is .vibrate based on https://tidepool.atlassian.net/browse/LOOP-4229

LGTM. This is some good cleanup. I do wonder about removing "vibrate", as all iOS notifications should vibrate. Or renaming 'vibrate' to default, as having sound == .default to indicate the default sound seems to be more clear than sound == .vibrate, as even all the other sounds should also vibrate.

There is a default critical alert sound and default non-critical alert sound on top of vibrate. So, I see it is possible to use the default sound when not muted and only vibrate when muted.

@ps2 looking for a formal review this time.

repo: tidepool-org/LoopKit


text:
"bootstrap" to UTC for all devices where possible + data model updates

This will be the real PR for bootstrapping to UTC. Supercedes #106, which I will close shortly.

#131 will replace this.

repo: tidepool-org/chrome-uploader


text:
Script to submit release to antivirus whitelist

What it says on the tin. At the moment we'll have to run the script manually after every release. In future we may want to consider a cron job or a webhook to automate this.

As a more general note, it looks like Rollbar has the ability to track deploys, but requires that one hit an API endpoint on release. Since our release mechanism is a GitHub publish, there's no existing script that gets run to piggyback off of easily. If we end up needing two scripts (this one and the Rollbar deploy tracking), I wonder if it'd be worth investigating writing a small script that we host somewhere to accept the GitHub webhook for releases which will automatically execute this and potentially other scripts.

@krystophv I've addressed your comments. Are you happy for me to just merge this, as it contains no user-facing changes and I'm already using it to submit to antivirus whitelist?

@gniezen Yeah, feel free to merge.

authors: gniezen krystophv


text:
Improve getShortName logic for non /TR/ URLs

The getShortname function does not work well with non /TR/ URLs, such as https://webassembly.github.io/spec/js-api/index.html for which it produces spec. IDL filenames are not always good as a result (and it's possible two specs end up with the same filename).

Actually, I don't see any obvious improvement. The Web Assembly spec was the only problematic spec in practice and it has now been published as a FPWD, so Reffy will use the /TR/ shortname from now on (wasm-js-api).
Looking at the URL of the Editor's Draft of that spec, there's not much we can do to extract a meaningful name. Obviously, we could extract webassembly but that's too generic, or js-api but we'd need to hardcode a rule on spec and that would be a bad name as well... In essence, we'd have to hardcode a rule specific for that (and possible other) specs.

Closing per previous comment.

repo: tidoust/reffy


text:
Feature/set circle support

Please do not open a pull request without first filing an issue and/or discussing the feature directly with the project maintainer.
Please ensure you adhere to every item in this list

[ ] This PR was pre-approved by the project maintainer
Raised discussion on Slack: https://tile38.slack.com/archives/C0M6QN8VC/p1718027811980879
[x] I have self-reviewed the code
[x] I have added all necessary tests

[ ] Fix tests, test for concentric circles contains fails.
Depends on: https://github.com/tidwall/geojson/pull/27

Describe your changes
Adds support for SET ... CIRCLE.
127.0.0.1:9851> SET cities sf POINT 37.769092 -122.451505
OK
127.0.0.1:9851> SET cities 10_km_from_sf CIRCLE 37.769092 -122.451505 10000
OK
127.0.0.1:9851> SET cities 100_km_from_sf CIRCLE 37.769092 -122.451505 100000
OK
127.0.0.1:9851> GET cities 10_km_from_sf
"{\"type\":\"Feature\",\"geometry\":{\"type\":\"Point\",\"coordinates\":[-122.451505,37.769092]},\"properties\":{\"type\":\"Circle\",\"radius\":10000,\"radius_units\":\"m\"}}"
127.0.0.1:9851> INTERSECTS cities POINT 37.769092 -122.451505
1) (integer) 0
2) 1) 1) "100_km_from_sf"
      2) "{\"type\":\"Feature\",\"geometry\":{\"type\":\"Point\",\"coordinates\":[-122.451505,37.769092]},\"properties\":{\"type\":\"Circle\",\"radius\":100000,\"radius_units\":\"m\"}}"
   2) 1) "10_km_from_sf"
      2) "{\"type\":\"Feature\",\"geometry\":{\"type\":\"Point\",\"coordinates\":[-122.451505,37.769092]},\"properties\":{\"type\":\"Circle\",\"radius\":10000,\"radius_units\":\"m\"}}"
   3) 1) "sf"
      2) "{\"type\":\"Point\",\"coordinates\":[-122.451505,37.769092]}"

127.0.0.1:9851> INTERSECTS cities IDS POINT 37.769092 -122.451505
1) (integer) 0
2) 1) "100_km_from_sf"
   2) "10_km_from_sf"
   3) "sf"

127.0.0.1:9851> INTERSECTS cities IDS POINT 37.554478 -122.020158
1) (integer) 0
2) 1) "100_km_from_sf"

This is based on PR https://github.com/tidwall/tile38/pull/649, but with some differences:

https://github.com/tidwall/tile38/pull/649 no longer compiles
this uses tidwall/geojsons Circle type, while the previous PR stored the polygon representation of the circle.

Issue number and link
Pull request require a prior issue with discussion.
Include the issue number of link here.

Duplicate of #649

The problem with a circle type as a geometry stored in a collection is that there is no GeoJSON compatible way to represent it without losing precision on output. I'm not opposed with the idea of a SET ... CIRCLE ... command but I would like to somehow solve the compatibility issue.

authors: iwpnd tidwall undeadcat


text:
🛑 TDMU Elearning is down

In dc9faea, TDMU Elearning (https://elearning.tdmu.edu.vn) was down:

HTTP code: 0
Response time: 0 ms

Resolved: TDMU Elearning is back up in cad336a.


text:
chore: fix mkdocs url

Signed-off-by: Takagi, Isamu isamu.takagi@tier4.jp
Fix mkdocs URL.

How about using tier4/main instead of tier4/proposal?

authors: isamu-takagi kenji-miyake


text:
feat(andible): update environmental setup

Related Links

https://github.com/tier4/edge-auto/pull/4

Description
Update ansible roles to set up edge.auto_jetson environment.
Review Procedure
Please run setup-dev-env.sh to set up the environment.
Remarks

This PR contains the netplan role, which fixes the IP address for eth1.  If you are using this network interface for ssh connection, running ansilbe-playbook may cause ssh disconnection.
Pre-Review Checklist for the PR Author
PR Author should check the checkboxes below when creating the PR.

[x] Assign PR to reviewer

Checklist for the PR Reviewer
Reviewers should check the checkboxes below before approval.

[ ] Commits are properly organized and messages are according to the guideline
[ ] (Optional) Unit tests have been written for new behavior
[ ] PR title describes the changes

Post-Review Checklist for the PR Author
PR Author should check the checkboxes below before merging.

[ ] All open points are addressed and tracked via issues or tickets

CI Checks

Build and test for PR: Required to pass before the merge.

@manato How about PR status?

@aohsato As well as other PR, I'll open this PR because I guess it's ready.
Please note that I set the default camera configuration to be:

FSYNC trigger mode
fixed exposure time (11ms)

in this PR.

Please note that I set the default camera configuration to be...

@manato Could you add this explain to our tutorials? https://github.com/tier4/edge-auto/pull/3

authors: aohsato manato


text:
Question - How do I install Calico add-on in an EKS cluster?

I am looking to install Calico add-on in an EKS cluster along with AWS VPC CNI as suggested in the official calico documentation which points to the AWS official documentation, which further states that the recommended way of installing Calico on EKS is by using this operator.
I am not sure how to do the same with the operator.
What I have tried :)

look in the certification courses but that too has the old way of installing it

Alternatives

Helm Installation helm install calico projectcalico/tigera-operator --version v3.23.1

Yeah it isn't clear how to do this.
I think you should be able to follow the Calico quickstart directions and just ensure you properly configure the custom-resources.yaml for the AWS CNI plugin (after you install the AWS CNI plugin).

The Amazon EKS documentation has steps for installing Calico listed here: https://docs.aws.amazon.com/eks/latest/userguide/calico.html#calico-install
Looks like it could use an update to the latest version, though.

The Amazon EKS documentation has steps for installing Calico listed here: https://docs.aws.amazon.com/eks/latest/userguide/calico.html#calico-install
Looks like it could use an update to the latest version, though.

Yes, this is the link I have in my query as well. It does not state how to install using the operator.

@vikas027 I see that the doc is not very clear, but the instructions listed under the "Helm" tab on that page are the correct instructions to install the tigera-operator (using helm), very similar to the instructions we have here: https://projectcalico.docs.tigera.io/getting-started/kubernetes/helm
We should clean up the language on the Amazon doc so that it's less confusing, but the content there is correct.

authors: caseydavenport tmjd vikas027

repo: tigera/operator


text:
Enable quota metrics by default

Describe your changes
How best to test these changes
Issue ticket number and link

Codecov Report
Patch coverage: 100.00% and project coverage change: -27.71 :warning:

Comparison is base (c0eb3c0) 57.56% compared to head (b071362) 29.85%.

:exclamation: Current head b071362 differs from pull request most recent head 93343b8. Consider uploading reports for the commit 93343b8 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff             @@
##             main     #882       +/-   ##
===========================================
- Coverage   57.56%   29.85%   -27.71%
===========================================
  Files         175      175
  Lines       19569    19568        -1
===========================================
- Hits        11265     5843     -5422
- Misses       7349    13042     +5693
+ Partials      955      683      -272

Impacted Files
Coverage Δ

server/metrics/metrics.go
83.69% <100.00%> (-7.71%)
:arrow_down:

server/search/factory.go
0.00% <0.00%> (-100.00%)
:arrow_down:

lib/container/priority_queue.go
0.00% <0.00%> (-100.00%)
:arrow_down:

server/quota/namespace.go
0.00% <0.00%> (-98.37%)
:arrow_down:

schema/lang/go.go
0.00% <0.00%> (-90.63%)
:arrow_down:

schema/lang/java.go
0.00% <0.00%> (-90.63%)
:arrow_down:

schema/lang/typescript.go
0.00% <0.00%> (-90.63%)
:arrow_down:

server/search/sorted_hits.go
0.00% <0.00%> (-89.10%)
:arrow_down:

server/search/facets.go
0.00% <0.00%> (-88.60%)
:arrow_down:

schema/inference.go
0.00% <0.00%> (-85.06%)
:arrow_down:

... and 111 more

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

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

authors: codecov-commenter efirs

repo: tigrisdata/tigris


text:
fix: only log unexpected errors in match

Describe your changes
We logging an error that is acceptable. There will be cases when the field we looking for does not exist in the
document or in the json timestamp object when we do a query. In those cases rather check first the field does not exist and return false. Only if it is an unexpected error do we log it.
How best to test these changes
Test should pass
Issue ticket number and link

I verified that this indeed fixes the logging issue.

authors: garrensmith pboros


text:
Objective C example code

Hi
This is not an issue per se, but I could not find a way to communicate this and I thought it'll also help others in similar boat; hence posting here.
Do you have an example of using TPInAppReceipt library in an objective C project? Sample code on how to instantiate the class and call various methods would be great. Or if you have Objective C class documentation, that'd be great too.
I am not familiar as much with Swift, and my project is in Objective C hence the request.
Thank you in advance!

Hi @apatait,
Unfortunately, I don't have an example, but you can use it in objc project.
Here you can find how to install it:
https://github.com/tikhop/TPInAppReceipt/blob/master/Documentation/UseInObjCProject.md
Here is a similar question:
https://github.com/tikhop/TPInAppReceipt/issues/84
Some old example that probably doesn't work anymore.
https://gist.github.com/tikhop/7d1215bbcea7fe8a3be3ce5ef25e15ee
Anyway, it must be pretty easy to use the library in objective c project just by checking swift examples from readme file

authors: apatait tikhop

repo: tikhop/TPInAppReceipt


text:
lock_resolver: let resolve lock timeout properly

fix https://github.com/pingcap/tidb/issues/44822 , after this PR tidb can pass the test.

I think it's better to merge PRs affecting critical path with 2-3 approvals.

authors: cfzjywxk zyguan

repo: tikv/client-go


text:
TableFullScan gets slower resulting in a 5% performance regression in TPCDS

Bug Report

What version of TiKV are you using?

good commit: https://github.com/tikv/tikv/pull/15523
bad commit: https://github.com/tikv/tikv/pull/15522
What operating system and CPU are you using?

Steps to reproduce

deploy a tidb cluster with 1 tidb + 3 tikv with the good and the bad commit
run TPCDS 50g

What did you expect?
no performance regression
What did happened?
Q88 elapsed time: 34s -> 38.9s
Q28 elapsed time: 24.8s->28.3s
exection_plan.zip

/severity critical

It was caused by https://github.com/tikv/tikv/pull/15584

/remove may-affects-5.3
/remove may-affects-5.4
/remove may-affects-6.1
/remove may-affects-6.5
/remove may-affects-7.1

/unlabel may-affects-5.3

/remove-label may-affects-5.3
/remove-label may-affects-5.4
/remove-label may-affects-6.1
/remove-label may-affects-6.5
/remove-label may-affects-7.1

repo: tikv/tikv


text:
coprocessor: not return rows when there is no input for simple aggregation

What is changed and how it works?
Close pingcap/tidb#30923
What's Changed:
For SimpleAggregation does not return row if there is no input rows
Related changes

PR to update pingcap/docs/pingcap/docs-cn:
PR to update pingcap/tidb-ansible:
Need to cherry-pick to the release branch

Check List
Tests

Unit test
Manual test (add detailed scripts or steps below)

Side effects

Performance regression

Consumes more CPU
Consumes more MEM

Breaking backward compatibility

Release note
Fix wrong `any_value` result when there are regions returning empty result

/label needs-cherry-pick-5.3

/label needs-cherry-pick-5.2

/label needs-cherry-pick-5.1

/label needs-cherry-pick-5.0

/label needs-cherry-pick-4.0

/rebuild

pingcap/tidb#30923

TiDB v3.1.2/v3.0.20/v2.1.19, all of them have the same issue.

BTW seems that TiKV is keeping this behavior since v2.0.0, so this might be a bug exists for a lot of versions. Better to check v3.0 and v2.1 as well.

Tested in TiDB v3.1.2/v3.0.20/v2.1.19, all of them have the same issue.

/label needs-cherry-pick-3.1

/label needs-cherry-pick-3.0

/label needs-cherry-pick-2.1

/merge

@breeswish can you help merge it?

/merge

/merge


text:
ci: update owners aliases

What is changed and how it works?

Issue Number: Close #17004

What's Changed:

Related changes

[ ] PR to update pingcap/docs/pingcap/docs-cn:
[ ] Need to cherry-pick to the release branch

Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[x] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Release note

None

/cc @yudongusa @benmeadowcroft @zhangjinpeng87

/cc @overvenus @LykxSassinator

/ok-to-test

/retest

/retest
flaky unit test

/cc cfzjywxk

authors: overvenus wuhuizuo


text:
test: stabilize lock_manager tests

Signed-off-by: youjiali1995 zlwgx1023@gmail.com

What problem does this PR solve?
Issue Number: close #xxx
Problem Summary:
What is changed and how it works?
Proposal: xxx
What's Changed:
Related changes

PR to update pingcap/docs/pingcap/docs-cn:
PR to update pingcap/tidb-ansible:
Need to cherry-pick to the release branch

Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Side effects

Performance regression

Consumes more CPU
Consumes more MEM

Breaking backward compatibility

Release note

No release note

/test

/test

/run-all-tests

/rebuild

/test

/test

/test

/test

/test

/test

/test

/test


text:
Nuvola does not recognise the current song

Nuvola does not recognise the current song. It shows in the sidebar only (null). So neither the lyrics plugin nor the desktop notifications are working.
I installed the Version 5.1 of the Plugin and Version 3.1.0 of Nuvola via the Arch User Repository.

Hello @lonlazer. Thanks for taking the time to report this issue. I can confirm it.
@SteVwonder, could you take care of it? Thanks!

I wonder if there is still someone taking care of updating this service/app.
Can someone give us some information about it?
@fenryxo @SteVwonder

Hello @das-keyboard. This script is currently unmaintained as the maintainer @SteVwonder seems to have lost interest in it. I might take over it next month if there is any spare time.

I've adopted this abandoned script and released version 5.2 with fixes.
-- Jiri Janousek, Nuvola Player developer • My work on Nuvola Player and user support are funded by Nuvola Patrons. :heart: Make a donation to keep the project going. Thank you.

authors: das-keyboard fenryxo lonlazer

repo: tiliado/nuvola-app-amazon-cloud-player


text:
Compile error from soa-rs-derive: can't leak private type

Using version 0.3.1 of soa-rs I cannot compile anything because a private type is being leaked from soa-rs-derive.
For me, to reproduce the issue it's as simple as
cargo init test_soa_rs;
cd test_soa_rs;
cargo add soa-rs;
cargo check;

I get the following error message
   Compiling soa-rs-derive v0.3.1
error[E0446]: private type `SoaDerive` in public interface
  --> C:\Users\zane\.cargo\registry\src\index.crates.io-6f17d22bba15001f\soa-rs-derive-0.3.1\src\fields.rs:9:1
   |
9  | / pub fn fields_struct(
10 | |     ident: Ident,
11 | |     vis: Visibility,
12 | |     fields: Punctuated<Field, Comma>,
13 | |     kind: FieldKind,
14 | |     soa_derive: SoaDerive,
15 | | ) -> Result<TokenStream, syn::Error> {
   | |____________________________________^ can't leak private type
   |
  ::: C:\Users\zane\.cargo\registry\src\index.crates.io-6f17d22bba15001f\soa-rs-derive-0.3.1\src\lib.rs:75:1
   |
75 |   struct SoaDerive {
   |   ---------------- `SoaDerive` declared as private

For more information about this error, try `rustc --explain E0446`.
error: could not compile `soa-rs-derive` (lib) due to previous error

I upgraded to Rust 1.76 from 1.74 and I no longer have the issue.

repo: tim-harding/soa-rs


text:
Remove size from image url

movie.imdb.coverUrl.rsplit("@", 1)
lazy loading

Bump

Could you prioritize this?

This is blocking important processes! (My sense of aesthetics)

Real talk: shouldn't this happen in the imdb-resolver?

I actually want to do this in the frontend since the load-times will go down. The removal of the actual size can happen in imdb-resolver.

I don't understand, what do you want to do where? (Keep in mind that I refuse to have a look at the frontend code)

imdb-resolver removes the size from the cover url, but then we'd load the full size image in the frontend which isn't necessary since we'll never display a full size cover.
That's why I want to add the actual dimensions (which are set in the frontend anyways) so we have fewer resources to load -> better loading times/less bandwidth waste/no resizing necessary.

So does imdb-resolver already remove the size from the URL, or do we need to create an issue for that?

https://github.com/tim-hat-die-hand-an-der-maus/imdb-resolver/pull/6

I actually need the image size ratio or original width and height.
Maybe we should convert coverUrl to
cover: {
  "url": "<String>",
  "width": <int>,
  "height": <int>
}

or replace width/height with ratio: <float>
what do you think?

I think ratio is probably the best solution

additionally:
_V1_SY150_CR0,0,101,150_ no idea what CR does, doesn't seem to affect the output (visually at least)
SY150 is the height of the image, S can apparently be replaced with U, I couldn't find any differences in those two.
Other characters seem to just ignore the size and show the given width/height based from the top left.
If Y is replaced by X it takes the width for stretching.

authors: BjoernPetersen torbencarstens


text:
not working on windows

the plugin simply does not work, it is active so I can run the command "Bible reference: verse lookup", but when I write inside the verse there is always no answer, nothing. Also if I write --John1:1 etc I never had any output. Tried also without firewall no difference.
Is there some extra plugin I need to run it?
I tried it also in android phone, same problem, nothing happens...

If on windows, can I try open developer console to see
what logs is there?

this is the toggle for enable the developer console.

If you can get some logs, that will be very helpful. I am very curious why this could happen as well.

Also getting the same issue. Here's what my console produces -

Also getting the same issue. Here's what my console produces -

Hi @SlothUS
Thanks for this.

@fcom000 @SlothUS
Can you try to switch the Bible Version provider by Bolls life? in the settings.

This is a CORS issue, which is a server side issue, I know this will happen one day, just don't know it happens so fast.
CORS issue, which it blocks the 3rd part API consume to consume its API.
This is why it failed.
To fix this, need to host a server by my self.
For now I will suggest you to use bolls life versions.

this should be fixed in 1.9.0 by switching the default version, please re-open this if not.

thanks very much, it works with the Bolls life. I was wondering if it is easy to host a Bible Api offline in my computer so even without internet the plugin works (and there will be no update issues?

authors: SlothUS fcom000 tim-hub

repo: tim-hub/obsidian-bible-reference


text:
wordpress filtered content not available in post object?

I'm not getting the post `content` as expected anymore.
on a single page, i'm using this in the page.php:
`$context = Timber::get_context();
$context['post']= new Timber\Post();'
All my templates that use {{ post.content }} stopped working. When I dump the post object, I don't see the 'content' key as being set in the object.
Expected behavior

{{ post.content }} should output wordpress filtered content (with wpautop, shortcodes, and oembed filters applied.
Actual behavior

Timber\Post Object
(
	[ImageClass] => Timber\Image
	[PostClass] => Timber\Post
	[TermClass] => Timber\Term
	[object_type] => post
	[custom] => Array
		(
			[_edit_lock] => 1563905362:79
			[_edit_last] => 79
			[_wp_page_template] => default
		)

	[_content:protected] =>
	[_permalink:protected] =>
	[_next:protected] => Array
		(
		)

	[_prev:protected] => Array
		(
		)

	[_css_class:protected] =>
	[id] => 15061
	[ID] => 15061
	[post_author] => 79
	[post_content] => this is the content
	[post_date] => 2019-07-23 09:57:18
	[post_excerpt] =>
	[post_parent] => 0
	[post_status] => publish
	[post_title] => test page
	[post_type] => page
	[slug] => test-page
	[__type:protected] =>
	[_edit_lock] => 1563905362:79
	[_edit_last] => 79
	[_wp_page_template] => default
	[post_date_gmt] => 2019-07-23 16:57:18
	[comment_status] => closed
	[ping_status] => closed
	[post_password] =>
	[post_name] => test-page
	[to_ping] =>
	[pinged] =>
	[post_modified] => 2019-07-23 09:57:18
	[post_modified_gmt] => 2019-07-23 16:57:18
	[post_content_filtered] =>
	[guid] => http://localhost:8080/library/?page_id=15061
	[menu_order] => 0
	[post_mime_type] =>
	[comment_count] => 0
	[filter] => raw
	[status] => publish
)

Steps to reproduce behavior

page.php:
	$context = Timber::get_context();
	$context['post']= new Timber\Post();
	Timber::render( array( 'page.twig' ), $context );

page.twig:
{{ post.content }}
or
{{ post | print_r }}
What version of WordPress, PHP and Timber are you using?

PHP 7.1.3, Wordpress 5.2.2, Timber 1.9.2
How did you install Timber? (for example, from GitHub, Composer/Packagist, WP.org?)

Latest from Wordpress plugins

Sorry - discovered some changes to a custom plugin that were breaking this. Not a timber issue.

repo: timber/timber


text:
auth.tesla.cn ?

Hello
One of my users reported  me that he cannot login anymore since I updated my app on the new SSO login.
He also told me that on his side the login page is:
https://auth.tesla.cn/oauth2/v1/authorize?client_id=teslaweb-cn&response_type=code&scope=openid email profile&redirect_uri=https%3A//www.tesla.cn/openid-connect/generic&state=Ht7mUM_AjhcOj5gNaU_xHLUqy-v9TljYQOtnLUghodY&locale=zh-CN
When I asked him to login on Tesla.com he told me that this does not works. On my side, I cannot login to auth.tesla.cn.
So there would be multiple SSO servers not sharing their users databases?
Have you encountered this case?
Last thing I tested on my side, when I try to load: https://auth.tesla.cn/oauth2/v3/authorize?client_id=ownerapi&redirect_uri=https%3A%2F%2Fauth.tesla.cn%2Fvoid%2Fcallback&response_type=code&scope=openid email offline_access&state=xxx
I have an "invalid-redirect-uri" error.

With an account created on tesla.cn I succeed to login using this URL:
https://auth.tesla.cn/oauth2/v3/authorize?client_id=ownerapi&redirect_uri=https%3A%2F%2Fauth.tesla.com%2Fvoid%2Fcallback&response_type=code&scope=openid email offline_access&state=xxxxxx
So the SSO server seems to be on auth.tesla.cn and the redirect_uri seems to be common between all SSO servers (would be logical because the Android app is the same everywhere)
Last thing, the /authorize page seems to be different on auth.tesla.cn and auth.tesla.com , there is one more step on auth.tesla.cn (the password field is hidden when you load the page and you have to click on "Next" to see it).

The first POST to /authorize is sent with the following body:
_csrf: azertyuiopzertyuiop
_phase: identity
transaction_id: azerty
cancel:
identity: user@email

It answer with a 200 and load the /authorize page again, with the password field this time.
Then I post the form with the password and the body contains:
_csrf: qsdfghjklqsdfghjkl
_phase: authenticate
_process: 1
transaction_id: azerty
cancel:
identity: user@email
credential: userPassword

And this time I have the 302 response with the code in the location header.
So it seems there is an additional "identity" phase on the CN login page!

Ok so by adding the extra POST /authorize with the "identity" phase I succeed to obtain an SSO token.
But...
After getting it I cannot obtain an owner API token using the step 4.
The answer is:
Response{protocol=h2, code=404, message=, url=https://owner-api.teslamotors.com/oauth/token} {"response":null,"error":"https://mobilesvc.teslamotors.com:443/accounts/GetUidByFederationId => {:code=>404, :message=>\"Could not get account by federation id\"}","error_description":""}

login_hint query parameter on first GET /authorize seems to be the way to get the good environment, see here:
https://github.com/adriankumpf/teslamate/issues/1297

I'm guessing they distribute a different app build in China, because there are references to the tesla.cn domain in there, but I don't see any code path to actually have them set as the SSO URL or other related variables. So, there may not be that much to glean from the build in the global app stores. Given I'm not a citizen of China, I don't really have much to offer here, unfortunately.

I think the last error I have when trying to exchange my bearer an owner API token is because my account in CN is not associated to any vehicle.
So I sent an alpha version of my app with the login process using the login_hint query parameter, following the redirects on step 1 GET and reuse the Host value of the last redirect.
I will keep you informed.

Ok so I got a feedback from the CN user I sent the alpha version of my app, and this is working for him now!
So I guess the error I got Could not get account by federation id with the CN account I created for test purpose was because I have no vehicle associated with this account.
To summarize, in order to be compliant with all Tesla SSO servers, the GET at step 1 should include a login_hint query parameter with the email as value, and then 2 cases:

you get a 200 response
you get a 303 response redirecting you to another SSO server (for example auth.tesla.cn)
And the rest of the process (step 2 and step 3) should be executed on this SSO server.
So in my case I just followed the redirect if any, then get the request Host, and use this new value as Tesla SSO Environment.

A big thanks to @cliffchen and @adriankumpf for finding a way to handle this!

Documented in #309. Feel free to submit a PR if you want to add anything else.

Thanks! And thanks for this documentation and this repository!!

authors: denouche timdorr

repo: timdorr/tesla-api


text:
Powerwall and energy endpoints

Between this (timdorr/tesla-api) and joeblau/teslaapi.io, there is pretty good coverage of the API. timdorr/tesla-api has a lot more updated information on the auth endpoints, but joeblau/teslaapi.io has much more on the powerwall and energy sites. Can we make these one source of truth?

We've got a start with #580!

authors: insaindesign timdorr


text:
Black out period on department basis

Would be VERY helpful if we could block out a section of the
calendar in a different colour and put a message on that months
team view or a legend of the colours on it so that it says that
people should not book leave on those dates - we have a period in
the year where we don't allow leave between certain dates
(although we need the option to still book it if its approved by
directors) - just a colour indication I think would be good.  So
I would set a date period to be in red, showing that employees
shouldn't book leave on those dates.

+1 as in #283

repo: timeoff-management/application


text:
enhancement: input_format_skip_unknown_fields default to true for external streams

PR checklist:

Did you run ClangFormat ?
Did you separate headers to a different section in existing community code base ?
Did you surround proton: starts/ends for new code in existing community code base ?

Please write user-readable short description of the changes:
Closes #549 .

(Jove Github Bot) added it to the current sprint.

(Jove Github Bot) moved this ticket out of the GitHub project(up to 1200 tickets for one project).

authors: jovezhong zliang-min

repo: timeplus-io/proton


text:
use named port for pgbouncer

The services(s) call for targetPort: pgbouncer but the sts does not currently define that name. This leads to the endpoint not getting computed, and it causes issues with service loadbalancers.

Thanks for the PR, this is part of https://github.com/timescale/timescaledb-kubernetes/pull/218 now (including this commit).
We cannot merge this PR directly, as this commit is now a bugfix on top of 0.7.0, whereas master has diverged a lot already.

authors: drpebcak feikesteenbergen

repo: timescale/timescaledb-kubernetes


text:
Предложение: Autoflake

Описание функциональности: Добавить autoflake как дев зависимость, для обработки импортов в помощь isort
isort сам по себе не может удалять неиспользованные импорты, autoflake может

@timoniq
Эти импорты ведь не нужны? Я помню, что помимо инитов есть bot.py из которого не нужно удалять импорты, эти файлы такие же?

@timoniq
Эти импорты ведь не нужны? Я помню, что помимо инитов есть bot.py из которого не нужно удалять импорты, эти файлы такие же?

Нет, это лишние импорты

authors: dhvcc timoniq

repo: timoniq/vkbottle


text:
added '--defaults' option that gets data with the defaults command

This adds a --defaults option as an alternative to --plist and --dsobject. Basically this will run the defaults read command and create the mobileconfig from that output. That way you do not have to worry where the actual plist is stored and wether cfprefsd has already updated. There is an extra option --currentHost which will enable the -currentHost flag when running the defaults command.
Example:
./mcxToProfile.py --defaults com.apple.screensaver --identifier com.example.screensaver --currentHost --manage Once

I like the idea! However, since we have Python and Foundation at our fingertips, we could do this without shelling out to Python and use CFPreferences natively. I'm also not sure if the output from defaults properly distinguishes between an integer 0 and a Boolean false. I've never tried to throw the defaults output at plistlib directly as you did here.
I don't know if there's a convenient way to do the equivalent of defaults read with a single API call but there is a method to list all keys, and then each value can be retrieved. Then you'd still get things in a nice dict structure that should map to a plist.

You're right, it wasn't that hard at all.

I have also added flags (or a placeholder) to access the 'anyUser' and 'anyApplication' (Global) domain. This is how you get at the different files:

Application
User
Host
File Location

app_id
current
current
~/Library/Preferences/ByHost/app_id.host_id.plist

app_id
current
any
~/Library/Preferences/app_id.plist

app_id
any
current
/Library/Preferences/app_id.plist

app_id
any
any
???

any
current
current
~/Library/Preferences/ByHost/.GlobalPrefences.host_id.plist

any
current
any
~/Library/Preferences/.GlobalPrefences.plist

any
any
current
/Library/Preferences/.GlobalPreferences.plist

any
any
any
???

I wish I knew what the ??? are. I thought it might read forced profiles if present, but that does not seem to work. Setting those parameter always seems to return empty dicts. Maybe with MCX? If there really is no (current) application for this combinations, it might make sense to assume --currentHost when --anyUser is set.
Examples:
./mcxToProfile.py --defaults com.apple.screensaver --identifier ScreenSaver --currentHost

reads ~/Library/Preferences/ByHost/com.apple.screensaver.host_id.plist
./mcxToProfile.py --defaults com.apple.screensaver --identifier ScreenSaver

reads ~/Library/Preferences/com.apple.screensaver.plist
./mcxToProfile.py --defaults com.apple.loginwindow --identifier loginwindow --anyUser --currentHost

reads /Library/Preferences/com.apple.loginwindow.plist
./mcxToProfile.py --defaults ManagedInstalls --identifier MunkiSettings --currentHost --anyUser

reads `/Library/Preferences/ManagedInstalls.plist
./mcxToProfile.py --defaults NSGlobalDomain --identifier GlobalPrefs --currentHost

reads ~/Library/Preferences/ByHost/.GlobalPrefences.host_id.plist
./mcxToProfile.py --defaults NSGlobalDomain --identifier GlobalPrefs

reads ~/Library/Preferences/.GlobalPrefences.plist
./mcxToProfile.py --defaults NSGlobalDomain --identifier GlobalPrefs --anyUser --currentHost

reads /Library/Preferences/.GlobalPreferences.plist

I think the any-host / any-user combination may have been used in the old days of network-mounted storage and user data. I think it's reasonable enough to make these options for --defaults work similarly to how the defaults command works.
Thanks for adding this!

Oh, and this was just merged: https://github.com/timsutton/mcxToProfile/commit/cd1314f31cec9fe8b874b6727c0e998097269153

Thinking again about the behaviour when --currentHost and --anyUser are both set, it doesn't generate the kind of "scope" we want.
Say I want to capture the settings that are effectively stored in /L/P/com.apple.loginwindow:
I need to type this:
./mcxToProfile.py --defaults com.apple.loginwindow --currentHost --anyUser --identifier Loginwindow
But this gives me something like:
			<key>PayloadContent</key>
			<dict>
				<key>com.apple.loginwindow.ByHost</key>
				<dict>
					<key>Forced</key>
					<array>
						<dict>
							<key>mcx_preference_settings</key>
							<dict>
								<key>Hide500Users</key>
								<true/>
							</dict>
						</dict>
					</array>
				</dict>
			</dict>

Notice the ".ByHost" at the end of the domain - except that this is not actually a ByHost pref. I'm starting to wonder if we should instead try and make it behave analogous to defaults, and support the --currentHost flag only. To read from the any-user domain, we could simply take the /L/P/com.whatever.domain syntax that defaults uses, and not put the .ByHost for the domain.

I kind of like the extra behavior, we could also fix the logic to add the "ByHost" to the preference domain:
https://github.com/timsutton/mcxToProfile/pull/12

Thanks, I just wonder what the extra behavior of --anyUser gains us if it only gives expected results when combined with --currentHost. But in practice, it's going to be less common that you want to pull preferences from the any-user domain anyway. We can just document this in an example.

authors: scriptingosx timsutton

repo: timsutton/mcxToProfile


text:
(bot) Update ghcr.io/fluxcd/helm-controller Docker tag to v0.11.1

This PR contains the following updates:

Package
Update
Change

ghcr.io/fluxcd/helm-controller
minor
v0.10.1 -> v0.11.1

Release Notes

fluxcd/helm-controller
v0.11.1
Compare Source
CHANGELOG
v0.11.0
Compare Source
CHANGELOG

Configuration
📅 Schedule: At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box.

This PR has been generated by Renovate Bot.

Renovate Ignore Notification
As this PR has been closed unmerged, Renovate will now ignore this update (v0.11.1). You will still receive a PR once a newer version is released, so if you wish to permanently ignore this dependency, please add it to the ignoreDeps array of your renovate config.
If this PR was closed by mistake or you changed your mind, you can simply rename this PR and you will soon get a fresh replacement PR opened.

repo: timtorChen/pi-cluster


text:
StyledFocusRing has width:100%;height:100% which can break CSS layout

Hey guys,
Awesome job with Tina, couldn't love your project more!!!
Just one issue I noticed. react-tina-inline adds StyledFocusRing to the DOM. It's all fine but by making it width: 100%; height: 100% it can break CSS layout. I'm talking about this line of code:
https://github.com/tinacms/tinacms/blob/a22063fdf2e10f873df9bd401f5d224c73ef6729/packages/react-tinacms-inline/src/styles/focus-ring.tsx#L85-L86
Imagine you have an InlineGroup or InlineBlock with content that has some "intrinsic natural width" which is not 100%. Like a button with width: 200px. Imagine that the parent of this element is display: flex. Flex parent makes a use of natural width of its children during layout (uses width as flex-basis if the latter is missing). However, if you add width: 100% between parent and child (StyledFocusRing) then layout breaks, because flex parent thinks that its child is 100% width. Basically any kind of layout that makes a use of content size of its children will break if we add Tina container in between.
In my opinion StyledFocusRing shouldn't have neither width nor height. Its size should be determined by the content and it should be as non-intrusive as possible. (can't wait for display: contents to have better support btw).
I know this change could break backward compatibility in some projects. Maybe we could add a flag? I'd be happy to do a PR, but curious about rationale behind the current behaviour.

Just with some quick testing it seems that this was done for components that use relative or absolute positioning within their parent, which is probably the minority compared to intrinsic sizing. I think possibly just accepting a class string and passing it to the focus ring, or only setting width/height when some prop is set could solve the issue for now. This component is used by the inline group and inline block components, and they already accept focus ring options which could accommodate this change.

Currently for us, certain types of elements that expect direct children to have certain CSS constraints ends up with incorrect layouts while editing. For example, a FlexGroup that has FlexItems requires FlexItems to have flex properties.
It would be great if we can just pass a className to be appended to the StyledFocusRing elements. This way, if I have an element like FlexGroup and another set called FlexItems, then I can give classNames to StyledFocusRing elements that would make them behave like FlexItems.
In the long term, it would be great to not even have feedback elements injected. For example, notion.so adds separate feedback elements that are positioned absolutely for mouse move and dnd effects.
Also, thanks for an amazing project!

authors: atifsyedali r00dY spbyrne

repo: tinacms/tinacms