text:
chore: remove obsolete editorconfig setting

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@75d2e5d). Click here to learn what that means.
Patch has no changes to coverable lines.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff            @@
##             main      #151   +/-   ##
========================================
  Coverage        ?   100.00%
========================================
  Files           ?        20
  Lines           ?       163
  Branches        ?        35
========================================
  Hits            ?       163
  Misses          ?         0
  Partials        ?         0

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

authors: codecov-commenter json-derulo

repo: json-derulo/angular-ecmascript-intl


text:
Update Composer dependencies (2024-03-05-00-05)

Visual regression test failed!

source: gharchive/pull-request

license: MIT

license_type: permissive

license_source: github-api


text:
Update Composer dependencies (2024-03-27-00-06)

Visual regression test failed!


text:
chunk/store: simplify upload code of cached store

Codecov Report

Merging #1695 (d47fbf3) into main (5d63b63) will increase coverage by 0.26%.
The diff coverage is 68.51%.

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

@@            Coverage Diff             @@
##             main    #1695      +/-   ##
==========================================
+ Coverage   61.38%   61.65%   +0.26%
==========================================
  Files         134      134
  Lines       22651    22605      -46
==========================================
+ Hits        13904    13936      +32
+ Misses       7172     7105      -67
+ Partials     1575     1564      -11

Impacted Files
Coverage Δ

pkg/chunk/cached_store.go
78.53% <68.51%> (+3.40%)
:arrow_up:

pkg/chunk/utils_unix.go
73.33% <0.00%> (-20.00%)
:arrow_down:

pkg/vfs/writer.go
78.64% <0.00%> (-1.02%)
:arrow_down:

pkg/meta/sql.go
67.28% <0.00%> (+0.20%)
:arrow_up:

pkg/chunk/disk_cache.go
80.33% <0.00%> (+0.48%)
:arrow_up:

pkg/meta/base.go
72.41% <0.00%> (+0.50%)
:arrow_up:

pkg/vfs/reader.go
84.67% <0.00%> (+7.48%)
:arrow_up:

pkg/fuse/context.go
67.56% <0.00%> (+10.81%)
:arrow_up:

Continue to review full report at Codecov.

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

authors: SandyXSD codecov-commenter

repo: juicedata/juicefs


text:
Gromov TextFileAnalyzer

Your tests fail
TextFileAnalyzer> test (suite: text-file-analyzer-test)

### Failure in: 0:0:Test Successful File Read
test/FileReaderSpec.hs:19
sample.txt not found. Create test/testfiles/sample.txt with some content.
### Failure in: 1:0:Test Statistics Computation
test/StatisticsSpec.hs:21
Word count should be 5
expected: 5
 but got: 6
Cases: 18  Tried: 3  Errors: 0  Failures: 2+++ OK, passed 100 tests.
Cases: 18  Tried: 6  Errors: 0  Failures: 2+++ OK, passed 100 tests.
### Failure in: 3:2:Test Repeated Phrases
test/NGramAnalysisSpec.hs:31
Most frequent n-gram
expected: ("i love jetrbains",3)
 but got: ("i love jetbrains",2)
Cases: 18  Tried: 18  Errors: 0  Failures: 3

What is it in the first line?
Ekaterina.Verbitskaya@NVC00653 TextFileAnalyzer % stack run src/TextUtils.hs

Text File Analyzer Menu:
1. View Basic Statistics
2. Find Most Frequent Words
3. Perform N-Gram Analysis
4. Generate Word Cloud
5. Exit
Enter your choice: 4

Word Cloud:
**********
***** tokenizewords
*** cleanword
*** isalphanum
*** map
*** string
*** tolower
*** where
* datachar
* filter
* import
* module
* textutils
* words

Otherwise, very well done!

Good 👍

authors: giothun kajigor


text:
Agregar API do VirusTotal ao projeto

O desenvolvimento da integração da API do Virus Total foi seguindo a documentação oficial do próprio Virus Total no link: https://docs.virustotal.com/reference/overview

Alguns problemas e limitações devem ser considerados.

A API do Virus Total tem um limite de usuários simultâneos.
A API do Virus Total tem um limite por usuário.
Algumas características são disponibilizadas apenas para versão paga da API.

Devido a grande quantidade de dados e as várias alternativas possiveis de retorno, podem haver algumas inconsistencias de dados.

source: gharchive/issue

repo: kakanetwork/Kyrios


text:
Use GraphJSON to track organization's progress

Closing. @kamilkisiela feel free to reopen/prioritize if needed.

authors: dotansimha kamilkisiela

repo: kamilkisiela/graphql-hive


text:
fbx

mesh, material, bone, animationくらいができればよさそう

mesh クリア


text:
parse note

#41 support note

repo: kcwl/elastic


text:
Add Int

@rultor merge

@rultor merge

@kerelape The pull request is closed already, so I can't merge it

@rultor merge

@kerelape I'm sorry, I don't understand you :( Check this page and try again please

authors: kerelape rultor

repo: kerelape/memoria


text:
Added Armstrong Number Python Program

not acceptable only projects considered


text:
Will  this support OLTP Trace over HTTP?

可以通过 WithSdkTracerProvider 配置成需要的 exporter

https://github.com/cloudwego/biz-demo/blob/b0b848231865af4c43f2ed0b8d5967eaf2502d9d/gomall/app/user/main.go#L80
https://github.com/cloudwego/biz-demo/blob/b0b848231865af4c43f2ed0b8d5967eaf2502d9d/gomall/app/user/infra/mtl/tracing.go#L32
类似这种吧，使用 otlptracehttp 配置下就好了，不需要复制一份代码

repo: kitex-contrib/obs-opentelemetry


text:
LIVREP is incorrectly sql

because I make a 1D rod node

it now says it is 1D and then returns no code

repo: kjappelbaum/moffragmentor


text:
feat: RUST_LOG is now the recommended way to set the log level

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

repo: kristof-mattei/endless-ssh-rs


text:
Automate version tagging

Pull Request Test Coverage Report for Build 10204765284
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 0.0%

Totals

Change from base Build 8441771801:
0.0%

Covered Lines:
0

Relevant Lines:
0

💛  - Coveralls

authors: coveralls fogfish

repo: kshard/embeddings


text:
Fix or change Contact page so it works

Icons to email, LinkedIn, Github function. Included email address as well


text:
confirm error with collabfold and check looping over seeds

closing this - should be fixed

repo: labdao/plex


text:
added a method to scale by x or y axis

@letsar  I noticed there was no method to scale by x or y. I am currently experimenting with this package and the need arose so I thought to contribute that. Is my PR alright or could it be better implemented.

@letsar

Hi @CoderNamedHendrick!
Thanks for your PR but I think I will find another way to create a non uniform scale effect, because I would prefer having only one method for scaling.
I think I will remove the scale parameter and use x,y,z parameters instead, for more flexibility. It will be a breaking change though.

Hi @CoderNamedHendrick!
Thanks for your PR but I think I will find another way to create a non uniform scale effect, because I would prefer having only one method for scaling.
I think I will remove the scale parameter and use x,y,z parameters instead, for more flexibility. It will be a breaking change though.

I was going to do thats so it matches the Transform.scale parameters but wanted to get your feedback on it. I can fix my pr to do that instead if you don't mind

Oh I took the liberty to do it myself since it was not a lot of work, I hope you don't mind.
I've just published a 0.0.4 version of this package with these parameters.
I will close this PR since it's the code is there now.
Thanks again for your PR and don't hesitate to share your feedbacks ;-)

authors: CoderNamedHendrick letsar


text:
Add some test vectors to check other implementations.

Thanks for the speedy review!

repo: lightsparkdev/lightspark-crypto-uniffi


text:
Add support for .nu domain

Hello @linkdata
Thanks for your PR.

authors: likexian linkdata

repo: likexian/whois-parser


text:
[DRAFT][vcr-2.0] Fix cloud-metrics

Codecov Report
Attention: Patch coverage is 0% with 63 lines in your changes are missing coverage. Please review.

Project coverage is 18.21%. Comparing base (21cb075) to head (ae88413).
Report is 2 commits behind head on master.

Files
Patch %
Lines

...ava/com/github/ambry/cloud/azure/AzureMetrics.java
0.00%
31 Missing :warning:

...b/ambry/cloud/azure/AzureCloudDestinationSync.java
0.00%
20 Missing :warning:

.../com/github/ambry/cloud/VcrReplicationManager.java
0.00%
6 Missing :warning:

.../java/com/github/ambry/cloud/VcrReplicaThread.java
0.00%
3 Missing :warning:

...n/java/com/github/ambry/cloud/RecoveryMetrics.java
0.00%
1 Missing :warning:

.../com/github/ambry/cloud/RecoveryNetworkClient.java
0.00%
1 Missing :warning:

...ithub/ambry/cloud/azure/AzureCloudDestination.java
0.00%
1 Missing :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             master    #2719       +/-   ##
=============================================
- Coverage     52.90%   18.21%   -34.70%
+ Complexity     7935     2809     -5126
=============================================
  Files           831      831
  Lines         70771    70790       +19
  Branches       8510     8519        +9
=============================================
- Hits          37444    12893    -24551
- Misses        30329    56697    +26368
+ Partials       2998     1200     -1798

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

authors: codecov-commenter snalli

repo: linkedin/ambry

license_source: bigquery


text:
docs(docs): :memo: just test deploy

:tada: This PR is included in version 1.0.3 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


text:
请问SmartShadow中用到的数据集可以公开分享吗？

"Data from rendering engine. We use non-photorealistic rendering (NPR) techniques to obtain line art and shadow
pairs. To be specific, we search the key word “free” in Unity 3D Assets Store and download 471 random 3D prefabs. We
import them to the rendering engine Blender [13] and write a NPR script to generate 25,413 line art and shadow pairs at
random shadow directions (Fig. 4-(b))."
想使用一下文中提到的这个数据集。


text:
add missing country guinea

Thanks so much @issa2580 :)


text:
Update 31_card_docs_update.sql doc about a style added

Hi. I think it was done here #355

I see! Can you make the change in the other pull request, then? Closing this one.

Hi, please take a look at #365. I hope it is ok now.

authors: amrutadotorg lovasoa

repo: lovasoa/SQLpage


text:
Some fixes to get the demo up and running on Hugo themes site

Thanks, PR!

authors: bep lubang

repo: lubang/hugo-hello-programmer-theme

license: mit


text:
Develop

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 95.75%. Comparing base (93af3dd) to head (321dceb).
Report is 10 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #116   +/-   ##
=======================================
  Coverage   95.75%   95.75%
=======================================
  Files          10       10
  Lines         942      942
=======================================
  Hits          902      902
  Misses         40       40

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

authors: codecov-commenter lucasimi

repo: lucasimi/tda-mapper-python


text:
fix params in train_decoder

👍🙏

authors: lucidrains nousr


text:
Fix use statement in example in README.

Thanks! I ended up removing the need for the inner module, so this got cleaned up further by removing this line completely.
Cheers


text:
docs(sbb-header): add readme section about scroll-padding-top

Codecov Report

Merging #1714 (852f24f) into master (ab917b0) will decrease coverage by 2.64%.
The diff coverage is 53.79%.

: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
@@            Coverage Diff             @@
##           master    #1714      +/-   ##
==========================================
- Coverage   54.85%   52.21%   -2.64%
==========================================
  Files          49      138      +89
  Lines        1659     5039    +3380
  Branches      406     1382     +976
==========================================
+ Hits          910     2631    +1721
- Misses        671     2232    +1561
- Partials       78      176      +98

Impacted Files
Coverage Δ

...ts/sbb-accordion-item/sbb-accordion-item.events.ts
100.00% <ø> (ø)

...mponents/sbb-accordion-item/sbb-accordion-item.tsx
0.00% <0.00%> (-37.21%)
:arrow_down:

src/components/sbb-checkbox/sbb-checkbox.events.ts
0.00% <ø> (ø)

src/components/sbb-header/sbb-header.tsx
27.63% <ø> (ø)

src/components/sbb-icon/sbb-icon-request.ts
70.00% <ø> (ø)

src/components/sbb-icon/sbb-icon-validate.ts
45.16% <ø> (ø)

src/components/sbb-icon/sbb-icon.tsx
92.30% <ø> (ø)

src/components/sbb-image/sbb-image.helper.tsx
0.00% <ø> (-64.52%)
:arrow_down:

src/components/sbb-image/sbb-image.tsx
0.00% <ø> (-62.97%)
:arrow_down:

...mponents/sbb-journey-header/sbb-journey-header.tsx
85.71% <ø> (-1.25%)
:arrow_down:

... and 23 more

... and 115 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

authors: codecov-commenter jeripeierSBB

repo: lyne-design-system/lyne-components


text:
feat: Update vale version

Heya, thank you for your PR! I have actually transferred the vale COPR repo to David O. at https://github.com/daobrien/vale-spec - that is the main repository for the COPR builds. You can reach David internally on gchat or on that repo :) I'll update the main readme.

repo: m-czernek/vale-spec


text:
updated db uri fmt

#35  👈 (View in Graphite)
master

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

repo: mAmineChniti/Gordian


text:
Create UnionOfTwo.cpp

Now I have added comments and also describe the question . Kindly check thanks.


text:
Update release.yml

/release


text:
Changed minSdk to 21

Ktlint упал

Task :ktlint
/home/runner/work/Fragula/Fragula/app/src/main/kotlin/com/fragula2/sample/MainActivity.kt:83:2: Redundant newline (\n) at the end of file (final-newline)

Локально можешь запустить ./gradlew ktlintFormat, он поправит кодстайл

Эх, терминал не видит JAVA_HOME, не получается выполнить команду, завтра попробую разобраться, извини за неудобства  😞

Все таки была причина оставить API 23. Но ничего, сделаю разные конфиги для приложения и либы

authors: MrRuslanYT massivemadness

repo: massivemadness/Fragula


text:
feat(assembly): Add line number while token parsing

Closing since parsing will be affected by issue #298.

repo: maticnetwork/miden


text:
Desktop app v5.3 changelog

Just fyi, we might have to make changes to this if Claudio isn't lucky in getting the Electron team to fix a bug we found in this Chromium version.

authors: amyblais cpoile

repo: mattermost/docs


text:
V9.3 documentation

@cwarnermm Thank you, I will likely not merge this until next week, but we can have this ready just in case it's needed earlier.


text:
add so cal csv file

Has been added in #112

repo: mdenolle/NoisePy


text:
xsv search full

That's al

repo: medialab/xsv

license: Unlicense


text:
deploy

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.ajaykkm seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

authors: CLA-Mercedes-Benz ajayKrishnankm

repo: mercedes-benz/DnA


text:
use slog instead of zerolog

Codecov Report

Merging #14 (406ac8d) into main (f36595f) will increase coverage by 0.25%.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main      #14      +/-   ##
==========================================
+ Coverage   90.11%   90.37%   +0.25%
==========================================
  Files           6        6
  Lines         334      343       +9
==========================================
+ Hits          301      310       +9
  Misses         27       27
  Partials        6        6

Files Changed
Coverage Δ

logger/logger.go
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

authors: codecov-commenter michimani

repo: michimani/evidently-local


text:
Awaited -> released version hyper link updated

Thanks!

authors: JeevanMahesha andrewbranch

repo: microsoft/TypeScript-Website


text:
WSL 2.0: networkingMode=mirrored，How to Set Up a Host Agent in a Docker Container

The above issues are the same as mine, but they fail to provide valid answers in these issues.

repo: microsoft/WSL


text:
Delta on Row Level Security Policy

Fixed in 0.15.0.

repo: microsoft/delta-kusto


text:
Add Prototype Pseudocode

Can this go in its own file? The qeo_ndis header is meant to contain NDIS definitions for QEO.

It could, but it's going to get deleted eventually anyways, so why does it matter?

This seems like it belongs in markdown; the point of this header file is to take the code snippets in the QEO markdown file and produce something an actual compiler will accept. Why would we prefer putting non-compiling, commented-out pseudo code into the compiler-readable header file rather than a human-readable markdown file, which also provides the opportunity for description between code blocks?

I don't want to be super nit-picky, but why is the markdown file in the include directory?
I guess I'm just a bit lost: what is the intent of this pseudo code? Is this meant to be a non-normative extension of the QEO spec, sample code, or something driver developers actually include in their projects?

authors: mtfriesen nibanks

repo: microsoft/net-offloads


text:
Bump numpy to 1.22.0

Looks like this bump may leave behind Python 3.7 support... let's hold on this.

Per discussion this morning we're going to drop support for 3.7 as it EOLs in June anyway, so I will update this PR to do that as well.

Oh man, this had the very unfortunate side effect of breaking Onnx builds on PyTorch. (It uses this script to install Onnx, and still supports python 3.7)
https://github.com/pytorch/pytorch/actions/runs/3878248792/jobs/6614365795

Should we pin a commit in PyTorch for now?

authors: ZainRizvi abock justinchuby

repo: microsoft/onnx-script


text:
Colab Implementation of RatSQL and SPIDER dataset

Hey @jaivanti
Here is one implementation of the paper using colab. This colab notebook uses the files from this gitrepo, but commands to run them are in the colab. hope this helps
RatSQL.zip
If there are any queries. do feel free to ask

No description provided.

repo: microsoft/rat-sql


text:
updated release plan

Codecov Report

Merging #11021 (82a4be1) into main (a0f2385) will decrease coverage by 0%.
The diff coverage is n/a.

@@          Coverage Diff           @@
##            main   #11021   +/-   ##
======================================
- Coverage     63%      63%   -1%
======================================
  Files        482      482
  Lines      33829    33829
  Branches    5512     5512
======================================
- Hits       21403    21384   -19
- Misses     10398    10399    +1
- Partials    2028     2046   +18

Impacted Files
Coverage Δ

src/notebooks/debugger/debugger.ts
66% <0%> (-12%)
:arrow_down:

src/kernels/execution/notebookUpdater.ts
88% <0%> (-8%)
:arrow_down:

src/notebooks/outputs/tracebackFormatter.ts
93% <0%> (-7%)
:arrow_down:

src/notebooks/debugger/debuggingManagerBase.ts
69% <0%> (-6%)
:arrow_down:

...kernels/jupyter/preferredRemoteKernelIdProvider.ts
92% <0%> (-5%)
:arrow_down:

src/kernels/installer/productInstaller.node.ts
74% <0%> (-5%)
:arrow_down:

src/platform/common/cancellation.ts
68% <0%> (-4%)
:arrow_down:

src/notebooks/debugger/kernelDebugAdapterBase.ts
76% <0%> (-3%)
:arrow_down:

src/kernels/kernelFinder.base.ts
82% <0%> (-2%)
:arrow_down:

src/notebooks/debugger/debuggerVariables.ts
67% <0%> (-1%)
:arrow_down:

... and 6 more

authors: amunger codecov-commenter

repo: microsoft/vscode-jupyter


text:
Update: jely2002.youtube-dl-gui version 1.8.6

#42616

repo: microsoft/winget-pkgs


text:
mobileScanningFeature

Ready

repo: minimal-ui-kit/material-kit-react


text:
Update build.yml

Windows build fails but that seems unavoidable right now.
I've created a branch to handle a windows build with the fixes necessary but haven't had time to get it running yet

repo: mintlayer/core


text:
chore: moving presigned URL examples to incubating

i'm a tiny bit surprised that the solutions file is not modified in this PR? or is this project just already omitted from the solutions file?

I was surprised too, but the project wasn't referenced in the solution at all.

repo: momentohq/client-sdk-dotnet


text:
fix: use new accessor syntax & reinterpret

Pull Request Test Coverage Report for Build 3011
Details

22 of 27 (81.48%)  changed or added relevant lines in 12 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 83.709%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

builtin/array_block.mbt
0
1
0.0%

hashmap/hashmap.mbt
2
3
66.67%

hashset/hashset.mbt
2
3
66.67%

builtin/autoloc.mbt
0
2
0.0%

Totals

Change from base Build 3010:
0.0%

Covered Lines:
3895

Relevant Lines:
4653

💛  - Coveralls

authors: coveralls peter-jerry-ye

repo: moonbitlang/core


text:
Ft feedback front

Merged

authors: Felix-Barosio morehwachege

repo: morehwachege/kodomasuta


text:
Update Composer dependencies (2022-02-24-00-03)

Visual regression test failed!

repo: mostafizpantheon/phpdotenv


text:
ci: Test on oldest OS runners, too

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 95.03%. Comparing base (c732bd2) to head (e94c1fe).
Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main       #7      +/-   ##
==========================================
+ Coverage   93.43%   95.03%   +1.60%
==========================================
  Files           1        1
  Lines         198      141      -57
==========================================
- Hits          185      134      -51
+ Misses         13        7       -6

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

authors: codecov-commenter larseggert

repo: mozilla/mtu


text:
#43 - use Command Line instead of filepath.WalkDir

@HungTP-Play
Please review my code.


text:
es6语法不支持，希望可以更新

应该已经可以了

authors: ecustspace mumuy

repo: mumuy/browser


text:
Improve Readme

gj

authors: mindoodoo nLatt


text:
move to buildkit 0.15.1

Towards NSL-3888

repo: namespacelabs/foundation


text:
fix(start-command): replace npm run by yarn

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

authors: napse-investment tomjeannesson

repo: napse-invest/Napse


text:
Add tinywl+

Thank you!

authors: keshto natpen

repo: natpen/awesome-wayland


text:
Added a Simple Calculator

I have added a Simple Calculator in your Project. Please merge this PR #38


text:
Lagde en mocket versjon av NOM for testing i dev

Alternativet er dessverre ikke så mye bedre. Det å lage et mocket graphql API i en ny applikasjon er SUPER vanskelig å gjøre riktig. Hvis man skal bare lage noe som returnerer en dum respons tilbake så er det greit, men hvis man skal gjøre flere typer spørringer til NOM på forskjellige ting så blir det fort kaotisk.

Har dog snakket litt med NOM om Z-identer, mulig de legger til støtte for det i fremtiden

repo: navikt/amt-tiltak


text:
Fix division by zero panic in determine_offloader.

Well, close reading of these 5-10 nearby lines should say this is impossible. I mean, we could add a check, but then if it fails it would be reasonable to do assert/panic to attract attention.

Well, close reading of these 5-10 nearby lines should say this is impossible.

Yes, but complete verification is a bit harder than reading 5-10 nearby lines. To verify that panic is impossible, you should also look into Lsn type definition, whether it's signed or not, check if Lsn(0) <= anything is always true, look into what checked_sub does, and check that actual types of all used variables (like conf.max_offloader_lag_bytes and p.commit_lsn) match your expectations.
Actually, I'm more worried about future changes (for example, additional logic to filtering) that can cause unexpected behavior in the future. I'd rely less on nearby code and add more checks, because it's easy to overlook a change in a single line at a 5-10 line distance.

I mean, we could add a check, but then if it fails it would be reasonable to do assert/panic to attract attention.

Attracting attention is good, but assert/panic can cause a lot of damage (which is much worse than the benefits of attracting attention). For example, in the current issue, we had a panic that:

failed the whole wal backup launcher thread (backup is not working for all timelines after that)
created an entry about the panic in the logs (but it didn't attract attention)
wal_backup_launcher_rx was dropped and _tx became closed (and we have code relying on wal_backup_launcher_tx is never closed)
computes couldn't connect to safekeeper, because we send notifications to wal_backup_launcher_tx when compute is connecting
projects didn't work and it was not obvious where the bug is

What we should do IMO:

prefer errors to panics (1)
try to localize the errors, so that only a single job for a single timeline fails, without affecting other timelines (1)
create an alert for new panics in the logs (2)
use something to attract attention in unexpected scenarios (to start, we can just write more logs about unexpected cases) (2)
monitor failures and restart background threads (1)
don't rely on non-essential dependencies (e.g. keep working even if backup channel is closed) (3,4)

True.

create an alert for new panics in the logs
@yeputons was going to add it.

e.g. keep working even if backup channel is closed
that's easy actually, can do that

authors: arssher petuhovskiy

repo: neondatabase/neon


text:
Cosmetic enhancements

LGTM.
(But ci-arm seems to oom, I'm working on it

authors: mohanson xxuejie

repo: nervosnetwork/ckb-vm


text:
feat: add virtualization objects support

:tada: This PR is included in version 0.3.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

authors: leoparente orb-ci

repo: netboxlabs/diode-sdk-python


text:
[Quantization] Add alias mapping for target types (fixes SiLU name change issue)

@dbogunowicz to revert test patches in #2204 to test this PR

repo: neuralmagic/sparseml


text:
Add vyfor/cord.nvim

Thanks!

authors: neurosnap vyfor

repo: neurosnap/neovimcraft


text:
Working pipeline listing page 🎉🎉🎉

@nf-core-bot fix linting


text:
Update Composer dependencies (2020-09-28-00-04)

Visual regression test passed!
View the visual regression test report

repo: nickolas-morales/pantheon-mit-sloan


text:
Create Dockerfile

docker file

repo: niranjantj/hello-github-actions


text:
wire frame project

Saved in Google Drive


text:
Refactor/update enmeshed app

Links to "Mein Bildungsraum" still occur in file webview_constants.dart.

yep :)

repo: nmshd/app


text:
Trying to fix CI false positives

Windows is still broken, that will be fixed in #566

repo: nodegit/nodegit


text:
Localiser les dates

https://gohugo.io/content-management/multilingual/#dates

authors: arnaudlevy sebousan

repo: noesya/osuny-hugo-theme


text:
Xenoarcheology department

This is the main role of the station. There is some crossover with Science, but the progress of this depart will directly affect the story plot. These people may be tasked with away missions to story critical locations.


text:
[DNM] Reproduce some bug

Reproduce in velox-backend-ubuntu2004-test.
Correct result: {7, 7, 7}
Velox result: {7, 7, 7}
After convert RowVector to GlutenBatch: {null, 7, 7} or core

@jinchengchenghh @rui-mo @zhztheplayer @zhouyuan

@Yohahaha Thank you. I will take a look later.

This is caused by the last operator C2R, and toArray uses the memory freed by the iterator. @JkSelf has a fix maybe can resolve the problem.

@Yohahaha   This is a bug for spark 3.2 and 3.3. And we have merged PR#40958 into spark 3.3 to fix this issue. I will port this fix in Gluten later. Thanks.

Good to know this!

authors: JkSelf Ma-Jian1 Yohahaha rui-mo

repo: oap-project/gluten


text:
Give clearer error when instruction_id is not found

@gunther9 good idea 👍

repo: ociswap/scrypto-testenv


text:
Add joaosreis as user of ocaml-ci

Hi @joaosreis thanks for trying out ocaml-ci.
Please install this GH App to get started and as always report any bugs or features as issues. Thanks!

authors: joaosreis tmcgilchrist

repo: ocurrent/ocaml-ci


text:
Namespace argument is needed for the job validate it should work without it as well.

this is already fixed

repo: odpf/optimus


text:
feature request #1

notice how this issue has #5 in URL, discussions are sharing same API


text:
🎨 Update color and text transitions for correct answers in MoodleSentence

@coderabbitai review

repo: olibos/moodle-sentence-order


text:
release 0.0.2

Codecov Report

Merging #463 (cd660f5) into master (1323186) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master     #463   +/-   ##
=======================================
  Coverage   76.09%   76.09%
=======================================
  Files          81       81
  Lines        3037     3037
  Branches      469      469
=======================================
  Hits         2311     2311
  Misses        726      726

Flag
Coverage Δ

batch-submitter
59.51% <ø> (ø)

contracts
86.05% <ø> (ø)

core-utils
68.16% <ø> (ø)

data-transport-layer
37.68% <ø> (ø)

message-relayer
72.22% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

packages/message-relayer/src/relay-tx.ts
71.15% <0.00%> (ø)

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 1323186...cd660f5. Read the comment docs.

authors: InoMurko codecov-commenter

repo: omgnetwork/optimism


text:
Push 2022 01 21

@MarkovaMaria  could you please take a look why Mac OS X build fails?

@MarkovaMaria could you please take a look why Mac OS X build fails?

see that it was fixed in next commit

see that it was fixed in next commit

yes

authors: MarkovaMaria opcm rdementi

repo: opcm/pcm


text:
Capture mailing list activity / engagement - from Force24

Have added data/services/automated-email-campaigns.csv, comprising information about the campaigns that were sent out. Columns are:
date - ISO UTC timestamp of publication
emailSentCount - number of addressees in the list
emailUniqueOpen - number of unique email opens
emailUniquwClick - number of unique clicks
Non-unique versions omitted, as were the bounce rates. I think LDF only do email campaigns, not SMS, although the Force24 report contains both (SMS is all zero).
@slowe I'm feeling bold, so might try and add something to the dashboard ;-)


text:
Ols blog post

@Bisola15 Here you go. 😃


text:
ViewOperation adds Render Pattern

:tada: This PR is included in version 1.0.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:

repo: open-mmlab/labelbee


text:
Hallacy/add embeddings

Hello! This looks like a repeat of a previous commit.  I'm gonna close this PR but please let me know if I'm wrong here.

repo: openai/openai-python


text:
Pin next to the catalog version

You should definitely work on improving testing (there is an issue related to that).
But for now I don't want to have to update the apps (to match the latest Next) while I'm working on the open next integration.

these are also unpinned: vercel-commerce/next-deps

It's pinned (but not to the catalog version).

It's pinned (but not to the catalog version).

yeah sorry that's what I meant 😅

I don't fully agree with this change but I'm not super against it (since we're doing this in the create-next-app example)

It's only temporary and we will improve tests as part of #108
Thanks!

authors: dario-piotrowicz vicb

repo: opennextjs/opennextjs-cloudflare


text:
pull request for e2e test at 01/25/2017 13:45:07

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


text:
pull request for e2e test at 12/10/2016 01:56:03

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


text:
pull request for e2e test at 01/21/2017 02:56:12

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


text:
fix: select seccomp depending on OpenShift version

/lgtm
/approve

authors: lance matejvasek

repo: openshift-knative/kn-plugin-func


text:
add godoc to make govet happy

[merge]

repo: openshift/source-to-image


text:
[ARM plugin] Support native RNNCell operation

@alvoron please review

Failed test - smoke_OVClassHeteroExecutableNetworkGetMetricTest/OVClassHeteroExecutableNetworkGetMetricTest_TARGET_FALLBACK which it don't depend on arm plugin functionality


text:
Dev

need to add the copy right declarations on the code

repo: oracle-devrel/oci-arch-queue-oke-demo


text:
ES module update.

LGTM 👍

authors: haadcode haydenyoung

repo: orbitdb/ipfs-pubsub-peer-monitor


text:
CreateCommand: Simplify setting the source artifact URL

Merged despite unrelated failing tests.

repo: oss-review-toolkit/ort


text:
[full-ci] Bump web to v5.5.0 final

Local API tests are failing due to: https://github.com/owncloud/ocis/issues/3998#issuecomment-1161442467

This PR needs a rebase to get a green CI. The resharing issue has been fixed with #4000

This PR needs a rebase to get a green CI. The resharing issue has been fixed with #4000

Taking care now

We'll most likely provide a v5.6.0 later today, that & changelog update will come later separately

authors: kulmann micbar pascalwengerter saw-jan

repo: owncloud/ocis


text:
refactor(minifier): add NodeUtil trait for accessing symbols on ast nodes

This makes monitor-oxc green again, where I can continue add more top npm packages https://github.com/oxc-project/monitor-oxc/actions/workflows/ci.yml

repo: oxc-project/oxc


text:
refactor(oxc_transformer): use enum to represent Pragma

#7613  👈 (View in Graphite)
main

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

@IWANABETHATGUY Oh I'm sorry. I didn't realize you were working on this.

One question, if you don't mind me asking: Same as in #7553, here you Box the result of the various match arms. What's the advantage of doing that?

One question, if you don't mind me asking: Same as in #7553, here you Box the result of the various match arms. What's the advantage of doing that?

No special reason, I saw your code and I think it is better than me, I just don't realize early return could avoid Boxing a iter trait.

Ah ha now I understand the Box. Thanks for explaining.

authors: IWANABETHATGUY overlookmotel


text:
Put variable to struct, downgrade to 1.18

Codecov Report

Merging #128 (cdc275d) into main (f964ade) will increase coverage by 0.82%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##             main     #128      +/-   ##
==========================================
+ Coverage   42.50%   43.32%   +0.82%
==========================================
  Files          33       33
  Lines        2134     2135       +1
==========================================
+ Hits          907      925      +18
+ Misses       1117     1100      -17
  Partials      110      110

Impacted Files
Coverage Δ

pkg/network/kafka.go
16.07% <40.00%> (+7.14%)
:arrow_up:

pkg/kafsar/kafsar_impl.go
48.31% <100.00%> (+0.93%)
:arrow_up:

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

authors: codecov-commenter goflutterjava


text:
use pulsar codec go

Codecov Report

Merging #136 (3f42f95) into main (d9c166f) will increase coverage by 0.09%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #136      +/-   ##
==========================================
+ Coverage   43.08%   43.17%   +0.09%
==========================================
  Files          33       33
  Lines        2147     2147
==========================================
+ Hits          925      927       +2
+ Misses       1111     1110       -1
+ Partials      111      110       -1

Impacted Files
Coverage Δ

pkg/utils/pulsar_util.go
65.11% <ø> (ø)

pkg/kafsar/group_coordinator_standalone.go
69.85% <0.00%> (+0.46%)
:arrow_up:

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


text:
external + optional interactions are not clear, should go into derive tutorial

Reference: https://github.com/pacak/bpaf/discussions/324#discussioncomment-7880879

authors: pacak polarathene

repo: pacak/bpaf


text:
add preview envs

Closing this PR as obsolete

authors: msteffen seslattery

repo: pachyderm/pachyderm


text:
FROST Feature for Orchard Crate

changes being tracked here
https://github.com/zcash/orchard/issues/430
https://github.com/zcash/librustzcash/pull/1478

repo: pacu/zwcd


text:
Spelling

bot merge

authors: andresilva gavofyork

repo: paritytech/substrate


text:
Create deno.yml

can you clarify what is this and why is it needed?

authors: OKEAMAH paulmillr

repo: paulmillr/scure-btc-signer


text:
Move overview diagram to app instrumentation doc

done

authors: pavolloffay svrnm


text:
replace short with scratch in doc

coverage: 41.267%. remained the same when pulling 6803687a49658f00b5a62df85bf10ecd63b90db3 on aekiss:doc-fix-scratch into d41b0165d545c02c2d8fd34c282b44640b75bfca on payu-org:master.

authors: aekiss coveralls

repo: payu-org/payu


text:
Edit and resend mails

 Make it possible to edit/resend a message already sent

repo: pci-dev/pci


text:
Refactor

@Mergifyio refresh

repo: pen-lang/pen


text:
Fix conflict error on status update

Test name
Status

recreate
passed

scheduled-backup
passed

big-data
passed

upgrade-haproxy
passed

haproxy
passed

storage
passed

scaling
passed

scaling-proxysql
passed

self-healing
passed

init-deploy
passed

restore-to-encrypted-cluster
passed

self-healing-chaos
passed

limits
passed

upgrade-proxysql
passed

security-context
passed

demand-backup
passed

monitoring-2-0
passed

self-healing-advanced
passed

affinity
passed

smart-update
passed

one-pod
passed

self-healing-advanced-chaos
passed

auto-tuning
passed

pitr
passed

proxysql-sidecar-res-limits
passed

operator-self-healing
passed

users
passed

operator-self-healing-chaos
passed

upgrade-consistency
passed

demand-backup-encrypted-with-tls
passed

tls-issue-self
passed

tls-issue-cert-manager
passed

tls-issue-cert-manager-ref
failed

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/991/commits/d82610abd879af84c6568d7fae0394043ecb4618
image: perconalab/percona-xtradb-cluster-operator:PR-991-d82610ab

Test name
Status

upgrade-haproxy
passed

recreate
passed

scaling
passed

restore-to-encrypted-cluster
passed

scaling-proxysql
passed

demand-backup
passed

security-context
passed

pitr
passed

demand-backup-encrypted-with-tls
passed

haproxy
passed

storage
passed

init-deploy
passed

self-healing
passed

limits
passed

self-healing-chaos
passed

monitoring-2-0
passed

self-healing-advanced
passed

affinity
passed

self-healing-advanced-chaos
passed

one-pod
passed

operator-self-healing
passed

auto-tuning
passed

operator-self-healing-chaos
passed

scheduled-backup
passed

proxysql-sidecar-res-limits
passed

big-data
passed

users
passed

tls-issue-self
passed

tls-issue-cert-manager
passed

tls-issue-cert-manager-ref
passed

validation-hook
passed

upgrade-proxysql
passed

smart-update
passed

upgrade-consistency
passed

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/991/commits/d82610abd879af84c6568d7fae0394043ecb4618
image: perconalab/percona-xtradb-cluster-operator:PR-991-d82610ab

authors: JNKPercona egegunes

repo: percona/percona-xtradb-cluster-operator


text:
"Căpălnaș, Ostrov, Virismort" are villages in "Birchiș" common, "ARAD" county

Precondition
Empty the search bar after every step

Steps 1
Acces to the link https://peviitor-ro.github.io/ui.orase/
Expected result
The page opens

Steps 1
Write "Căpălnaș"
Expected result
In the drop down menu you find "Căpălnaș, ARAD(Birchiș)"
Test Data
Căpălnaș

Steps
Write "Ostrov"
Expected result
In the drop down menu you find "Ostrov, ARAD(Birchiș)"
Test Data
Ostrov

Steps 3
Write "Virismort"
Expected result
In the drop down menu you find "Virismort, ARAD(Birchiș)"
Test Data
Virismort


text:
Changelog for Enterprise Server 2022.08.1 release

Fast tracking for Enterprise patch release

repo: pganalyze/pganalyze-docs


text:
Document new severity argument in getIssues API endpoint

I'm gonna merge this so that I can pull this also to https://github.com/pganalyze/pganalyze-web/pull/486


text:
copy/move zip/7z to Android/data

I'm not sure what you're asking here? GBCC can already import zip files containing ROMs and saves, and will extract them to the correct places.

Showing the zip/7zip file in the game list
My Android phone is rooted
All of my 4000+ files are zip, and unzipping them to Android/data would take a lot of time

Android/data/com.philj56.gbcc
Tutorial.gbc
Wario Land 3 (World) (En,Ja).zip
GBCC
Tutorial


text:
set envirornment variable as PRODUCTION if the stage type is DEFAULT

Tested on dev1.
When the stage_type is CONTROL, the output is CONTROL:
Running POST_RESTART script.

deploy-open-sentinel-dev1
test-timeout
CONTROL
OguoTTfvQC6cD-b_KHMYOw
POST_RESTART
DEPLOY
REGULAR
dc7519c6354dfa715b0035a720a7526e4c281453
deploy-open-sentinel
pinterest/teletraan
master
96jzkJADQj2YFy6ReRsQhg_dc7519c
https://deployrepo.pinadmin.com/deploy-open-sentinel/deploy-open-sentinel-dc7519c.tar.gz

When the stage_type is DEFAULT, the output is PRODUCTION:
Running POST_RESTART script.

deploy-open-sentinel-dev1
test-timeout
PRODUCTION
OguoTTfvQC6cD-b_KHMYOw
POST_RESTART
DEPLOY
REGULAR
dc7519c6354dfa715b0035a720a7526e4c281453
deploy-open-sentinel
pinterest/teletraan
master
96jzkJADQj2YFy6ReRsQhg_dc7519c
https://deployrepo.pinadmin.com/deploy-open-sentinel/deploy-open-sentinel-dc7519c.tar.gz

repo: pinterest/teletraan


text:
域名，证书，服务器部署

done，在 #13 顺便用ltsocket重构了ltsig（其实是重写）。
另外在 https://github.com/pjlt/lanthing-pc/pull/51 也有响应修改。

repo: pjlt/lanthing-svr


text:
Version 10.0.0

Due to multiple conflicts and changes, I will be force pushing the dev branch into master after reviewing that all the fixes from master where ported to dev


text:
[WIP] use MySQL instead of MariaDB driver

Is there anything preventing this change from moving forward?

Hi @derekperkins,
what is the current status of supporting vitess from prisma?
I interested in trying out making schema changes to vitess sharded via prisma. is this supported?

Hi @derekperkins ,
any update on support fro vitess using prisma?

@nitins64 We've stopped trying Prisma, so I don't know for sure. Vitess just understands the MySQL protocol, so if this PR gets merged to use a more modern connector, it should work fine.

It doesn't look like this PR is going to fix the issue.
It seems Prisma issue following sql to understand the database schema of the mysql db:

public async listSchemas(): Promise<string[]> {
const res = await this.query(
SELECT           schema_name        FROM           information_schema.schemata        WHERE schema_name NOT LIKE 'information_schema';,
)

Problem is that VtGate hides information_schema so prisma get following error:

_    Could not connect to database. ER_UNKNOWN_ERROR: vtgate: http://nitin-sharma.lan:15001/: no keyspace in database
_    name specified. Supported database name format (items in <> are optional): keyspace<:shard><@type> or
_    keyspace<[range]><@type>

This will be fixed by Prisma 2 where we will use the plain MySQL driver.

authors: derekperkins mavilein nitins64

repo: prisma/prisma


text:
Prepare for Release ver 0.1.0

Now fixlang_minilib is split to submodules, and the version number is assigned for each submodule.
So this issue is closed.


text:
TODO: make interactive mode the default

Released in 0.5.0

repo: ptmcg/logmerger


text:
Cleanup upgrade branch

Codecov Report

:exclamation: No coverage uploaded for pull request base (upgrade-pystiche-1@4ee1ca2). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Current head 7622310 differs from pull request most recent head 4b989cb. Consider uploading reports for the commit 4b989cb to get more accurate results

@@                 Coverage Diff                  @@
##             upgrade-pystiche-1    #272   +/-   ##
====================================================
  Coverage                      ?   97.0%
====================================================
  Files                         ?      39
  Lines                         ?    1639
  Branches                      ?       0
====================================================
  Hits                          ?    1591
  Misses                        ?      48
  Partials                      ?       0

Flag
Coverage Δ

unit
97.0% <0.0%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

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 4ee1ca2...4b989cb. Read the comment docs.

IIRC, the only thing missing here was Ulyanov. We should move forwards with this and fix that later.

authors: codecov-commenter pmeier

repo: pystiche/papers


text:
Manage all AsciiDoc attributes in antora.yml

@gastaldi I find having all AsciiDoc attributes in antora.yaml a bit better because it that way I do not need to include attributes.adoc to every single doc page.
FYI, we also changed the setup for testing the site. Now it is based on Antora TestContainer. It feels much more reality close, easier to maintain and easier to use for visual checks of the Antora output than the previous setup with asciidoctor-maven-plugin.

@ppalaga that's an interesting approach, I wonder if we can reuse some of that stuff to minimize the number of files to be maintained

I wonder if we can reuse some of that stuff to minimize the number of files to be maintained

If you ask me, I'd find it awesome if the code from https://github.com/quarkiverse/quarkus-cxf/blob/main/docs/src/test/java/io/quarkiverse/cxf/doc/it/AntoraTest.java could be packed into a quarkus extension that would serve the generated site on localhost:8080 with live reload included ;)

authors: gastaldi ppalaga

repo: quarkiverse/quarkus-cxf


text:
Fixed configuration reference path

Thanks !

authors: andlinger jtama

repo: quarkiverse/quarkus-minio


text:
2022-05-24

今日の俗世:

3500万円返金
イーロンマスク エルデの王へ

作業 with r7kamura #24 - YouTube

Presenter
Presenterの問題点:

直接モデルにアクセスするか、Presenterを通してアクセスするか、2つの方法が生まれてしまう。2つの方法が生まれるとね3つめもどうせ生まれてしまう
Viewのコードを見ても、値がPresenterなのかただのModelなのか分からない

変数の命名

I strongly recommend you name objects based on their actual class and do not pretend presenters are actually domain objects. For example, use @widget_presenter if the object is a WidgetPresenter, @widget if it’s a Widget.

それ。
ということはつまり、Foo::Barがあるとしたらbarという名前を使うべきという話だよな。ということは、Services::FooではなくServices::FooServiceとしなければならない。
あと個人的に名前について言いたいことがあるとすると、現代はエディタが十分にまともなので、名前は十分に長くても構わないと思う。英単語以上の省略は絶対にするべきではないし (例: eventをeとするなど)、widget_idをidとすべきでもない。

A Philosophy of Software Designは読んでみたいところ。

Validations Are Awesome For User Experience

Validationはデータの整合性を完璧に保つものではないから、UXのために使うべきだよとまとめている。いいまとめ方。ActiveRecordでのvalidationって、フロントエンドでのエラーチェックに似てるところがあるよね。

How to (Barely) Use Callbacks

あんまり使うなと言っている。あるあるですな。自分の話だけど、既にコールバックが多用されてるRailsアプリから、コールバックを引き剥がしたいという要求が開発者から挙がるのを、10社ぐらいで見ている気がする。

Scopes are Often Business Logic and Belong Elsewhere

3回書いたらscope化しろと言っている。ただあまり考察はなされていなくて、「普通にスコープをつくることもできるし、サービスクラスをつくってそこでスコープ的なこともできるよね。まあいろんなところで何度も使われるスコープっていうのはビジネスロジック色は薄くてデータ操作的だと言えるから、これはスコープで良いんじゃないか」と言っている。そういう温度感で良いのか？

Sustainable Web Development with Ruby on Railsを読み終えた。後半はなんかこう言及すべきトピックがあまり無かったのでざーっと読み流した。

同作者のもの:

Brutalist Web Design
ウェブデザインの本質に主眼を置いたデザインガイドライン「ブルータリスト・ウェブ・デザイン」 - GIGAZINE

書いた: 『Sustainable Web Development with Ruby on Rails』を読んだ

書いた: 山崎実業のスポンジホルダー

腹筋ローラー 10回3セット


text:
Use require_relative to avoid modifying $LOAD_PATH in gemspec

coverage: 95.971% (-0.001%) from 95.972%
when pulling 0795f9251ad6f2c63aee7565515390f46c41ed7b on jdufresne:require_relative
into a3eac8c4029bfc7ea6c0f4021dec2633912ee437 on railsadminteam:master.

❤

authors: coveralls jdufresne mshibuya


text:
Fix endpoint cli panic, virtualservices for template

https://github.com/rancher/rio/issues/655

repo: rancher/rio


text:
Demo app for Routes

Tracking: https://github.com/rangle/augury-examples/issues/14

Moving to augury-examples waffle

repo: rangle/augury


text:
NATIVESVTTK

Sorry, I'm still not sure, what's your goal with this. I can only repeat what I've said here: https://github.com/rdbende/Sun-Valley-ttk-theme/pull/132#pullrequestreview-1970155997

Try this
And, BY THE WAY: THIS VERSION USES OUTDATED SETUPTOOLS AND WHY IS YOUR PYPROJECT.TOML EMPTY

I know u hate this

Sorry for not getting to this PR earlier. You may have noticed, that I've been quite busy in the last couple of months, and haven't got to reviewing this PR yet. However I've left pretty clear review comments in #132, and you didn't address most of them.
I indeed don't want this feature in sv-ttk, because it's still just a UI theme wrapped as a Python package to make it easier to install. I don't like the idea of adding some platform-specfic features, that have not much to do with this project, and are otherwise trivial to implement if one wants it. I will probably reject any similar requests in the future.
I also don't want to add irrelevant dependencies like CustomTkinter, as it is an entirely different project with different goals thats has nothing to do with sv-ttk.

And, BY THE WAY: THIS VERSION USES OUTDATED SETUPTOOLS AND WHY IS YOUR PYPROJECT.TOML EMPTY

Feel free to open a pull request to update setuptools. The pyproject.toml file is empty, because I don't actually use it, but some tool I use refused to work without it. You can search in the git history to find out the reason for it.

You also have to understand, that if I merge something, it adds maintenance burden, although I already don't have the capacity to maintain this project.

repo: rdbende/Sun-Valley-ttk-theme


text:
Develop

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: arun-madhavan-013:x: deepthi-psdeepthi-ps seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

authors: CLAassistant deepthi-ps

repo: rdkcentral/vendor-manifest-raspberrypi


text:
fix string format error; fix for missing pods in periodic debug of failed dependencybuilds

/test jvm-build-service-e2e

/test jvm-build-service-in-repo-apicurio-e2e

right now rebuilds is what trigger the local stack deployment creation, whose absence these tests are not choking on
forcing localstack to be created regardless for the purposes of periodic debug

/test jvm-build-service-in-repo-e2e

/test jvm-build-service-in-repo-apicurio-e2e

the test changes mainly did what they were suppose to, but this service registry test suffered from a lot of resource constraint issues (test timeouts do seem to bypass our dump of the controller and cache, may make an update there)
let's try again and see how consistent that is
/test jvm-build-service-in-repo-apicurio-e2e

yep same sort of situation
on the cache / controller logs, they are still available in must gather artifacts so I think I'm just going to punt on the "extra dump" I do in the test.

/test jvm-build-service-in-repo-apicurio-e2e

/retest

with my local testing, I am starting to wonder if watching both the artifactbuilds and dependencybuilds is too expensive and the test program is crashing.  My lap top has seized up a couple of times now and then the test fails when he wakes back up.
I may pivot back to just watching the dependencybuilds for now

/test jvm-build-service-in-repo-apicurio-e2e

known flakes / resources constraints it would appear
testing one at a time just in case
/test jvm-build-service-in-repo-with-pruning-e2e

/test jvm-build-service-e2e

/test jvm-build-service-in-repo-apicurio-e2e

/test jvm-build-service-in-repo-apicurio-e2e

going ahead and merging this;
these test changes did what was expected, but his particular build struggled with various constraints (connection pool shutdowns in dependencybuilds for example), and just did not complete in the timeout period.
No missing logs for failed dependencybuilds and lots of items for @dwalluck @stuartwdouglas and the rest of us to go after.
The stats when the test timed out was
    types.go:25: time: 2022-08-27 18:07:24.318232744 +0000 UTC m=+4997.184348000: dependencybuild created count: 126 complete count: 52, failed count: 39, contaminated count: 2
    types.go:25: time: 2022-08-27 18:07:24.548888241 +0000 UTC m=+4997.415003507: pr 0a6f7f70fe84df5f3bb1b317b6eae3b3-build-2 not done out of 68 items

By comparison, using these change, I ran the service registry build locally, where no CI timeouts were involved (some of which we cannot override) and the final stats I got on Friday were
The final status there were
    types.go:25: time: 2022-08-26 12:12:35.506645676 -0400 EDT m=+5898.349163990: dependencybuild FINAL created count: 125 complete count: 38, failed count: 85, contaminated count: 2

So yeah, since late last week, where we had not merged any jvm-build-service changes, there has been a drop off in dependencies we were able to successfully complete.
Attaching the complete log of that manual run here.
Aug26-periodic.log

repo: redhat-appstudio/jvm-build-service


text:
operator crunchy-postgres-operator (5.5.0)

/pipeline restart operator-hosted-pipeline


text:
updating pdm lock file with latest dependencies

@Allda I have no idea what I'm doing with python here, but I'd like to be able to unblock

#654

@acornett21 We already have that covered in https://github.com/redhat-openshift-ecosystem/operator-pipelines/pull/653 I'll merge your PR with preflight version once the PR with dependencies is reviewed. I assume it is going to be by the end of today.


text:
Test issue

Closing because it's test

repo: regnull/ubikom


text:
docs: fix contributing link

Thank you for your help, but it looks like we've already fixed this.
If I'm wrong, please re-open this issue and we'll get it sorted out. Thanks!

authors: iheyunfei ryanflorence

repo: remix-run/remix


text:
功能：支持识别内容的导出

https://github.com/rerender2021/language-shadow/releases/tag/1.2.0


text:
CI

@arokem would you mind rebasing this when you get a chance? No rush.

rebased!
On Thu, Dec 16, 2021 at 11:36 AM Adam Richie-Halford <
@.***> wrote:

@arokem https://github.com/arokem would you mind rebasing this when you
get a chance? No rush.
—
Reply to this email directly, view it on GitHub
https://github.com/richford/hbn-pod2-paper/pull/4#issuecomment-996120736,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA46NW4FD3YISNTF7DMN6DURI5UFANCNFSM5KG45SBA
.
You are receiving this because you were mentioned.Message ID:
@.***>

authors: arokem richford


text:
fixed typo: cutomer → customer

Thanks for the update

authors: MahmoudFayed pancakevirus

repo: ring-lang/ring


text:
Added requirement for 4B accesses to be atomic if 4B aligned

LGTM

authors: christian-herber-nxp ved-rivos

repo: riscv/riscv-zilsd


text:
implement indirect block addressing

TODO: tests

In addition to the inline comments, I think line 51 should be changed to let su64 = size_of::() as u64;. Do you agree?
Yes, I mostly copied the code from SUFS.

We'll also need to add some large files to the test image to exercise this code. Hopefully we can make the files sparse so the test image doesn't grow too large.
If the block is a 0, that means it's unallocated. In this case the function returns Ok(None),
which is interpreted as a zero block.

In addition to the inline comments, I think line 51 should be changed to let su64 = size_of::() as u64;. Do you agree?

Yes, I mostly copied the code from SUFS.

We'll also need to add some large files to the test image to exercise this code. Hopefully we can make the files sparse so the test image doesn't grow too large.

If the block is a 0, that means it's unallocated. In this case the function returns Ok(None), which is interpreted as a zero block.

Well, it couldn't be completely sparse, of course.  But if it's a huge sparse file with one allocated block at the end, I think that would be a way to exercise indirect addressing.

authors: asomers riscygeek

repo: riscygeek/fuse-ufs


text:
added docker for local development

Thank you

authors: KoNekoD kubk

repo: riverwaysoft/php-converter


text:
Issue 1

Worked on the issue


text:
Implement "native" leader selection using cosmos-sdk staking module

ref: https://github.com/celestiaorg/optimint/issues/18

Duplicate of https://github.com/rollkit/rollkit/issues/815

authors: Manav-Aggarwal liamsi tzdybal

repo: rollkit/rollkit


text:
CI cache

Force merge for test the CI

authors: jolestar yourmoonlight

repo: rooch-network/rooch


text:
Create LICENSE.md

Looks Good.


text:
Use underscore for unused block arguments

Thanks, that's a good suggestion. Let me change it.

repo: ruby/debug


text:
Added asynchronous pwm functionality

Putting it in a separate crate seems fine to me since at least the current version does not share anything with the blocking version other than the error type.
If doing so, I would convert this repository to a workspace and have two folders like sysfs-pwm and sys-pwm-async like we do over at embedded-hal.
If you need help with the restructuring, let me know.

Putting it in a separate crate seems fine to me since at least the current version does not share anything with the blocking version other than the error type.

How would I go about doing that?

How would I go about doing that?

Hey sorry for the delay @lucaVuitton, I had planned to make this change but there's been some things in my personal life that are taking priority. If you want to take a try at it, a good place to start would be looking at the structure of the Cargo.toml files in https://github.com/rust-embedded/embedded-hal. There is a top workspace Cargo.toml, and then each subcrate (sysfs-pwm and sysfs-pwm-async) would get their own subdirectories with Cargo.toml files of their own. If you're okay waiting I'll also get back to this as soon as I can. Thanks!

authors: eldruin lucaVuitton nastevens

repo: rust-embedded/rust-sysfs-pwm


text:
adding getdtablesize on redox

@bors r+

:hourglass: Testing commit 8f3b088454bb906058d1b87ab2bcec52ed8ad522 with merge 02d775b87c84ab2bef89455b2ce06f1ecc95d53b...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13, checks-cirrus-freebsd-14
Approved by: JohnTitor
Pushing 02d775b87c84ab2bef89455b2ce06f1ecc95d53b to master...

authors: JohnTitor bors devnexen

repo: rust-lang/libc


text:
Initial board implementation

Well done.

authors: Serg-Bartosh alena-bartosh

repo: sad-company/sad-tetrispy


text:
Set version 4.10.0

Pull Request Test Coverage Report for Build 3172901627

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 87.841%

Totals

Change from base Build 3172876966:
0.0%

Covered Lines:
5960

Relevant Lines:
6785

💛  - Coveralls

repo: safe-global/safe-transaction-service


text:
Bug: Yeelight UPnP discovery doesn't work on Linux

Resolved:
YeelightAPI looks for available multicast addresses directly on network interfaces. This is completely unreliable on most Linux devices.
Yeelight SSDP discovery is very set up very explicitly and hasn't changed in years, so this sort of scan on the interface isn't necessary. Ripping it out and binding directly to 239.255.255.250 for discovery as per Yeelight's operational specs resolves the issue.

repo: saltybrackets/PsiCAT


text:
Add the ability to customize ITransformOptions in the sandbox

Welcome your PR

authors: AlexRMU samchon

repo: samchon/typia


text:
Create presentation slideshow using presentation template

I'm working on this one


text:
テンプレートを使って実際に八王子市のハザードマップを作る

https://github.com/sankichi92/hachioji-hazardmap
https://sankichi.net/hachioji-hazardmap/


text:
feat: add cli guides

@Favna can you clone and run lint:md, please I'm not on pc

Plwase revert e61fdfe233dc2ac71f2f311624d46391f166f7fb
That's not how CLI works

authors: UndiedGamer enxg

repo: sapphiredev/website


text:
Bump scala-cli to 1.4.0

While bumping Scala CLI v1.4.0, we might want to pass the --skip-cli-updates launcher option in the scala script on platforms other than Windows (more detail on it in https://github.com/VirtusLab/scala-cli/pull/2957)
It wouldn't be necessary on Windows, since --skip-cli-updates is passed implicitly when --cli-version is used.

authors: Gedochao hamzaremmal


text:
chore(pytest): Show flaky error details

Codecov Report

Merging #1501 (69127ad) into master (bdc43e8) will decrease coverage by 0.24%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1501      +/-   ##
==========================================
- Coverage   99.15%   98.91%   -0.25%
==========================================
  Files          91       91
  Lines        6974     6980       +6
  Branches     1230     1198      -32
==========================================
- Hits         6915     6904      -11
- Misses         35       46      +11
- Partials       24       30       +6

Impacted Files
Coverage Δ

src/schemathesis/cli/output/default.py
100.00% <100.00%> (ø)

src/schemathesis/constants.py
100.00% <100.00%> (ø)

src/schemathesis/lazy.py
91.30% <100.00%> (+0.51%)
:arrow_up:

src/schemathesis/specs/openapi/validation.py
76.47% <0.00%> (-11.77%)
:arrow_down:

src/schemathesis/_compat.py
77.77% <0.00%> (-7.41%)
:arrow_down:

src/schemathesis/extra/pytest_plugin.py
95.58% <0.00%> (-4.42%)
:arrow_down:

src/schemathesis/cli/callbacks.py
98.62% <0.00%> (-1.38%)
:arrow_down:

src/schemathesis/cli/cassettes.py
98.87% <0.00%> (-1.13%)
:arrow_down:

src/schemathesis/utils.py
97.78% <0.00%> (-0.89%)
:arrow_down:

src/schemathesis/specs/openapi/loaders.py
98.31% <0.00%> (-0.85%)
:arrow_down:

... and 3 more

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 bdc43e8...69127ad. Read the comment docs.

repo: schemathesis/schemathesis


text:
Refactor existing workflow.

I'll close it since we decided to approach it based on user-stories.

repo: scipp/essimaging


text:
[INFRA] Update CPM Packages

Documentation preview available at https://docs.seqan.de/preview/seqan/chopper/277

repo: seqan/chopper


text:
Feature/faster compilation

Again, this should please @jeanlaroche

repo: serge-sans-paille/pythran


text:
Ajustar documentação do Editor

Toda a documentação foi movida para o repositório do portal.

authors: cv sahrosa

repo: servicosgovbr/editor-de-servicos


text:
Dib dotnet migration

Part of #7

repo: sgrottel/tiny-tools-collection


text:
feat: embedding

Closed by #17


text:
test: Add transmuxer integration tests

Some tests are failing in macOS, but in real it tests works…

Some tests are failing in macOS, but in real it tests works…

Please try increasing the timeout for those steps.  In some cases, playback runs at < 1x on macOS with WebDriver attached.  I've never determined why, and it only seems to affect certain devices.
For 10s of playback, I recommend at least 20s to complete the step.  30s would be best.

@joeyparrish now all tests pass! Thank you

I have thought about using StreamGenerator but... currently it doesn't work for AAC, MP3 (raw formats) for example.
I'm currently working on having our own transmuxer for AAC and MP3, so this is considered a regression test to see that I don't break anything.
If you want I can delete the tests that have streams that do not depend on us, what do you think?

Well, it's probably okay for now, especially with the GitHub Actions environment, which is pretty reliable.  Let's keep it.

authors: avelad joeyparrish

repo: shaka-project/shaka-player


text:
feat: init a chains package containing on-chain helper functions

done by #396

repo: shawakash/payBox


text:
Deploy!

Tentatively deploying to https://callthis.link/, but not quite ready yet.

repo: shazow/callthis


text:
Update GitHub URL in User-Agent

thanks

repo: shhivv/pyston


text:
[modify] json response enhancement

Coverage increased (+0.007%) to 84.882% when pulling 7741fb725ca807fba39c18cd79fac9b7f3342523 on pr-json-resp-enhancement into 1cab9199668d445e17cf6a1de853e208070bdbff on master.

repo: shirasagi/shirasagi


text:
[modify][cms] record cms/member#last_loggedin

Coverage decreased (-32.7%) to 51.426% when pulling d8a3f6d919f793b364006078e009587d4d9170c3 on pr-member-last-loggedin into 7053d37b7a079cdc68dc8d3b0628014fc73bb25e on master.


text:
perf: ⚡ invoke

:tada: This PR is included in version 1.4.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

repo: shunkakinoki/codespaces


text:
Update pubspec.yaml

Merge completed as instructed.

authors: Ayshik smukhiasif

repo: shurjopay-plugins/sp-plugin-flutter


text:
Add webring torus

Kit this animation is super cool omg

This looks super good - I'm not going to merge it in just yet though because I am investigating why the scrollbar is so huge (at least on my device).

Wait I just realized is the 1000em height thing in the css what's causing the large scrollbar?

Also just realized the animation looks alright on mobile but we might need to move around some things to optimize it better


text:
fix: remove isAuthorized from API Gateway authorizer response

#208  👈
main

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

Join @pevisscher and the rest of your teammates on  Graphite

repo: skyleague/event-horizon


text:
refactor: simplify folder structure

Current dependencies on/for this PR:

main

PR #59

PR #60   👈

PR #61

This comment was auto-generated by Graphite.

Graphite Merge Job
Current status: ⏳ Queued to merge
This pull request is currently queued to be merged as part of a stack.
This comment was auto-generated by Graphite.

Job Reference: oPCx7VL6iilScTtJLPDq


text:
Support dummy relayer type

Or if you were just trying to get past the strict unmarshaling, that has been relaxed so that it tolerates unknown networks, so you shouldn't have to do anything.

repo: smartcontractkit/chainlink-common


text:
support adding terra chains

Codecov Report

Merging #196 (8ae5907) into main (ffa9a24) will increase coverage by 0.25%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #196      +/-   ##
==========================================
+ Coverage   19.69%   19.95%   +0.25%
==========================================
  Files          12       12
  Lines        1599     1604       +5
==========================================
+ Hits          315      320       +5
  Misses       1253     1253
  Partials       31       31

Impacted Files
Coverage Δ

client/chainlink_models.go
0.00% <ø> (ø)

client/chainlink.go
61.02% <100.00%> (+0.63%)
:arrow_up:

Continue to review full report at Codecov.

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

authors: RyanRHall codecov-commenter

repo: smartcontractkit/integrations-framework


text:
fix: ffpe event handling now correctly considers the provided purity

This requires Varlociraptor 8.3, see PR https://github.com/snakemake-workflows/dna-seq-varlociraptor/pull/249

repo: snakemake-workflows/dna-seq-mtb


text:
let's set the  to 250Mi so that we can deploy more apps in namespaces with low quota

@rawlingsj Do you need a release including this chance or does OSIO use master branch of the booster ?

We're going to look at using a branch of the booster catalogue https://github.com/openshiftio/booster-catalog/tree/osio if that helps?

As osio branch is equivalent to master branch from the spring boot booster point of view, you can continue to use it. I'm curious to know what is the purpose of this branch osio and why it has been created  @ALRubinger ...

Can we merge this PR?

As osio branch is equivalent to master branch from the spring boot booster point of view, you can continue to use it. I'm curious to know what is the purpose of this branch osio and why it has been created @ALRubinger ...

A few reasons:

not all boosters makes sense on OSIO and there is no good way of filtering (right now we actually have a hack in place in the wizard to only list specific hand-tested boosters)
there are not yet full automatic testing of boosters so that we can block updates to the catalog when things break on OSIO

authors: cmoulliard maxandersen rawlingsj

repo: snowdrop/spring-boot-http-booster


text:
Adding utils to compute entire Pareto curve and plot results

Closed by manual merge

repo: socialfoundations/error-parity


text:
Create MuhammadAli

Kindly accept my application.


text:
After all scores are announced, create a Round that the players can start playing

Done in #55


text:
add functionality for the coloring of preferences (tag? bool? green if all members have matching preferences)

https://github.com/sopra-fs23-group-07/sopra-fs23-group-07-client/commit/1626e935d006f81a97ed2d1b06bdeb0b502125bf

authors: oaboha theraffael


text:
Display the 'stopwatch' on the game screen.

"Tracked by #90" => look for linked commits there


text:
Implement a Endpoint to get the time for synchronizing.

"Tracked by https://github.com/sopra-fs23-group-40/Server/issues/90" => look for linked commits there


text:
Fix broken checks

Rough week... getting sloppy. Replaced the merged commit with fa010883f08068d2db54f1abc099d7a773055437.

repo: spd-mfa/samael


text:
Update roadmap

🚀 Deployed on https://65e67f13eaea39348e680d3e--egressgateway.netlify.app

authors: lou-lan weizhoublue

repo: spidernet-io/egressgateway


text:
iam not able to generate package with the help of maven

I am not sure what you mean but if provide more details and it turns out this is an issue in this project, we can have another look.

authors: siddudev snicoll

repo: spring-projects/spring-petclinic


text:
Automated Package Upgrade

:tada: This PR is included in version 2.0.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

repo: sprucelabsai/spruce-error


text:
Automated Package Upgrade

:tada: This PR is included in version 28.4.15 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

repo: sprucelabsai/spruce-schema


text:
Document firmware update

Firmware and RIOimage (Java Interpreter) are now up to date


text:
strip auto-generated stage name of invalid characters

fixed in 0.0.253

repo: sst/ion


text:
dev: add git commit field and clean up admin layout

This should add git_commit as a field and add it to the admin list view. It should also add a boolean field for if the metadata is null to the list view (make it easier to see if there is a bigger issue with an error)
For is_docker and hostname, this should remain in the database, but it will be removed from the list view (i.e., you'll have to click into a trial to see these fields). You can see in the migration file that these fields haven't been removed from the database.

Perfect, that is what I thought then. I just wanted to make sure I understand it. We can merge then!

authors: AlbertoCasasOrtiz carmichaelong

repo: stanfordnmbl/opencap-api


text:
ka/fix section title

This is relevant, but  I made the same edit in the md instead.
Closed by f3194fb00520359334d49ee38f635d36c58a8592

authors: kantonopoulos percolator

repo: statisticalbiotechnology/dsbook


text:
WHN Docs Update

🚀 Deployed on https://6669f1e87c14411d53c64e2e--cozy-fox-0defec.netlify.app

authors: craig-statsig jasonwzm

repo: statsig-io/docs


text:
Update sidebars.ts

🚀 Deployed on https://671ee59324730621ce86fb91--cozy-fox-0defec.netlify.app

authors: jasonwzm lin-statsig


text:
add redshift/bigquery placeholders to docs

🚀 Deployed on https://62b25cb673529904fc1ba7cd--cozy-fox-0defec.netlify.app

what does this do? Have you tried running this locally to verify it adding these without actual docs won't break?

authors: jasonwzm jkw-statsig vmadhusudanan


text:
Update reactNativeExpoSDK.mdx

🚀 Deployed on https://63220edb0dee892fe89193fe--cozy-fox-0defec.netlify.app

authors: jasonwzm tore-statsig


text:
What's the difference between FieldInput TextInput?

Hello @dzcpy,
The component FieldInput is used in the Field context.
<Field name="password" hint="Description line" error="If set, an error will be displayed">
 <Stack size={1}>
  <FieldLabel>Password</FieldLabel>
  <FieldInput placeholder="Placeholder" value={'Hello world'} disabled={true} onChange={() => {}} />
  <FieldHint />
  <FieldError />
 </Stack>
</Field>

This compound component is used to build and standardize form fields (display label, error, description, etc).
The TextInput component is just a preset that allows to avoid implementing the Field context.
In most cases, you will not need to use the context Field because many existing form components already implement this pattern (TextInput, NumberInput, DatePicker, TimePicker, Checkbox, CarouselInput, Select, TextArea, etc).
The Field pattern is used outside the library only in very rare and specific cases (e.g.: the creation of the Wysiwyg field (rich text) or the JSON field in Strapi).

authors: HichamELBSI dzcpy

repo: strapi/design-system


text:
メスガキオンライン！

🤖 このメッセージはAutoCodeReviewによって行われたコードレビューです 🤖
コードの解説:

ファイル「node.js」
OBSのパスと棒読みちゃんのパスを定義されています。
ストリームの状態を確認する関数「check_stream_status」が作成されています。

コードの改善点:

「commit comment」が入っていないので、必要であればコメントを追加する。
関数名が固有なので、より一般的な名前に変更する。
「console.log ()」を消すことで、デバッグとしてのログをインスペクション手法に変更する。


text:
WITH_OPTIONAL now uses Expression API

@shawkins: can you please have a look? It's pretty much what the title says and its part of the effort of replacing string concatenation with a proper API for code generation.

repo: sundrio/sundrio


text:
feat: Updates header styles

Current dependencies on/for this PR:

main

PR #6

PR #9   👈

This stack of pull requests is managed by Graphite.

repo: sungmanito/mono


text:
Chore: merge master into next

@kangmingtay do you know what's going on with that failing test? It looks like it's fixed in some branches – what should I merge into this to fix it?

repo: supabase/gotrue-js


text:
feat: bump gotrue-js to v2.22

Pull Request Test Coverage Report for Build 4668658340

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 67.513%

Totals

Change from base Build 4665430277:
0.0%

Covered Lines:
87

Relevant Lines:
111

💛  - Coveralls

repo: supabase/supabase-js


text:
not find GridView.builder

Can you be more specific? How is this related to SuperSliverList?

I think more than an issue, is a feature request.
Would be great to have support for GridView and SliverGridView, although the default flutter GridView has other problems also, children width.

I'm not sure what the issues are with the SliverGrid, but the layout there works different IIRC, the items are not sized depending on their contents, so it shouldn't have same performance issues as the SliverList. It's also out of scope for this package.

authors: hectorAguero knopp wanghuasheng


text:
fix: slight enhancements in the project description on README.md

Thanks

authors: caiotava eduardodoria

repo: supernovaengine/supernova


text:
solved issue #398

@Sunil4510 Thank you for your contribution. Looking forward to more such amazing contributions 🥂

@Sunil4510 Thank you for your contribution. Looking forward to more such amazing contributions 🥂

sure 🥂


text:
Fix init

Already fixed, thanks Sid

authors: dnicodemus sid-swirl

repo: swirlai/swirl-search


text:
[SYM-4763] Runtime Connector V2.x

Current dependencies on/for this PR:

main

PR #5   👈

This comment was auto-generated by Graphite.


text:
求训练和微调完整代码？

感谢关注我们的工作。我们正在整理相关代码。如代码公布，我们将会第一时间在GitHub更新

感谢关注我们的工作。我们正在整理相关代码。如代码公布，我们将会第一时间在GitHub更新

您好，请问还有开源的计划吗？

authors: cristianohello litsh yuanheTian

repo: synlp/ChiMed-GPT


text:
those are like trough levels of you

Could you please clarify? I cannot understand if this intended to be English text.

Omg I'm so sorry, I had talon turned on in the background accidentally and it overhead my conversation and somehow opened this issue. Completely accidental and I'll be more careful about having Talon sleep in the future. Sorry!

authors: maxkwallace nriley

repo: talonhub/community


text:
IOS-4258 Adjust seed phrase URL based on user locale

Апрув в надежде, что изменения согласованы, где-то зафиксированы и на Андроиде тоже сделают. Я не нашел обсуждения. Добавь пожалуйста коммент в таску в жире

Да, все согласовано

repo: tangem/tangem-app-ios


text:
Add select from camera

Hi @tanmoy27112000 , I have created this PR to add feature to select image from camera. Can you review and merge it?

Hi @tanmoy27112000 , do get a chance to see my PR ?
I am waiting for your response.

directly use the the package but changing the pubspec ,
simple_form_builder:
git: https://github.com/tanmoy27112000/SimpleFormBuilder.git


text:
ggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggbbbbbbbbbbbbc

definitely an idiot

authors: khaledYS tbaltrushaitis

repo: tdlib/telegram-bot-api


text:
Create recurring process and meeting for reviewing customer requests and bug board

@jliangTT Do we need such a process still in a formal sense?

yes we have the first pass process that we can iterate on. we can close this issue

authors: jliangTT tt-rkim

repo: tenstorrent-metal/tt-metal


text:
Tracy profiler support for n300

adding initial support for profiling host for remote device and enabling dumps from ethernet core: https://github.com/tenstorrent-metal/tt-metal/pull/4422
No changes to post process scripts yet.

Thanks for the merge Allan. Any points to keep in mind coming from this merge?

@mo-tenstorrent There was no functional changes to tensix worker profile dumps. I added an equivalent path for profiling on ethernet cores only. I also added device ids to the runtime scope names, so when I'm dumping multichip it's more clear where each call is coming from.
Also, as mentioned earlier, I didn't make any changes to post process scripts for ethernet yet, so this issue is still open

Great! ETH cores have access to DRAM as well, right? For this ticket to be fully implemented, ETH cores will need to push their profiling results to DRAM through NOC. Considering they are used as dispatch cores, they should be fine right?

Yep, eth cores have DRAM access. I don't think I'm pushing profiled results to DRAM from eth core for now.

Great, No main is all L1 based right now. Migrating to host reading results from DRAM soon.

Ah got it

authors: aliuTT mo-tenstorrent


text:
Fix typo to restore missing example. Fixes #160

Thank you!

authors: elsatch tfeldmann

repo: tfeldmann/organize


text:
Deps

Codecov Report

Merging #157 (50813f2) into develop (5aee303) will decrease coverage by 0.56%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #157      +/-   ##
===========================================
- Coverage    48.43%   47.86%   -0.57%
===========================================
  Files           45       45
  Lines         2269     2321      +52
  Branches       900      952      +52
===========================================
+ Hits          1099     1111      +12
- Misses        1007     1047      +40
  Partials       163      163

Flag
Coverage Δ

unittests
47.86% <ø> (-0.57%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/utils/JSUtils.js
51.19% <0.00%> (-5.39%)
:arrow_down:

src/components/decksidebar/HexPlot.jsx
76.47% <0.00%> (-4.78%)
:arrow_down:

src/components/File.jsx
52.63% <0.00%> (-2.93%)
:arrow_down:

src/components/MultiSelect.jsx
35.29% <0.00%> (-2.21%)
:arrow_down:

src/home/index.js
46.91% <0.00%> (-2.10%)
:arrow_down:

src/components/DataInput.jsx
35.00% <0.00%> (-1.85%)
:arrow_down:

src/components/export/Export.jsx
35.00% <0.00%> (-0.90%)
:arrow_down:

src/components/tooltip/utils.js
56.52% <0.00%> (-0.84%)
:arrow_down:

src/components/decksidebar/Charts.jsx
34.69% <0.00%> (-0.73%)
:arrow_down:

src/components/UI.js
18.75% <0.00%> (-0.61%)
:arrow_down:

... and 5 more

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 5aee303...50813f2. Read the comment docs.

authors: codecov-commenter layik

repo: tgve/tgvejs


text:
Update filter_ratings.ipynb

oke

update ok


text:
Add medical_release_id

The Github interface ignores DCO...

repo: thecardeaproject/cardea


text:
Update publisher

📝 Changed routes:

/bear-market-airdrop

(view full size)

Commit 1cb1a3f9eec93655052e1a57b5833536666851e0 (https://thirdweb-mv4a3l1jm.thirdweb-preview.com).

repo: thirdweb-dev/dashboard


text:
keycloak: fix cors header generation when call not from browser

Thanks!

authors: srose thomasdarimont

repo: thomasdarimont/keycloak-project-example


text:
Release 4.5.0

@herrewig can you bump the chart version?

Wow @Rickyxstar 😍 thanks!!!
All fixed!

authors: Rickyxstar herrewig

repo: thoras-ai/helm-charts


text:
frontend details page UI design

resolve


text:
Migrate to react

Closes #13

repo: timmson/prod-cal-ui


text:
feat: add a test for placeholder escaping

Thanks for pinging me, I forgot to watch the repo after transferring it. 😅
I'll review this on Monday.

If we are using CommonMark, then \\ should become a single backslash.

The page should have \\\\{computer_name}\\{share_name}, which would render out to \\{computer_name}\{share_name}, right?

The backslashes are inside a code block, so they would be treated literally even by a CommonMark parser.
https://spec.commonmark.org/0.31.2/#example-338

Note that backslash escapes do not work in code spans. All backslashes are treated literally

The current client specification doesn't strictly follow CommonMark. Single braces can't be escaped - \{\{ is rendered as {{, whereas \{{ is treated literally. This creates issues, because there are special cases where escaping a single brace would be useful. However, the client spec PR for escaping was merged without accounting for these cases to prevent doubling all backslashes in Windows paths.

I don't think the specifications make sense.

The only other way is to release a specification which states that every brace can be escaped, and all backslashes must be. The problem with that is, most clients will display double backslashes everywhere for a while.

Clients are free to render it as \\{{computer_name}}\{{share_name}} or \\computer_name\share_name, usually if a client does the latter they use colors/styles to signify the args.

https://github.com/tldr-pages/tldr/blob/main/CLIENT-SPECIFICATION.md#page-structure

Clients MAY highlight the placeholders and MUST remove the surrounding curly braces.

If a client removes backlashes and doesn't remove braces that mark the placeholder, the user won't know which braces are part of the command.

The backslashes are inside a code block, so they would be treated literally even by a CommonMark parser.

Ahh, you're right. I forgot they were in inline code. It seems I made the same mistake during the initial discussion for that revision to spec, too. ^-^'

If a client removes backlashes and doesn't remove braces that mark the placeholder, the user won't know which braces are part of the command.

Understood, I misinterpreted the spec, thank you for clarifying this. I would still expect us to use the approach proposed (checking "token" at a time) when checking it to avoid issues with formatting, ANSI codes, etc. However, indeed we should disregard my comments regarding curly braces being optional. In a separate commit, I will also remove the comment I've left in the test file about this.

In that case, this all looks good to me. Thanks for adding the test, and being patient with me and my misguided review. 😅

authors: SethFalco acuteenvy

repo: tldr-pages/tldr-pages-test-harness


text:
bundler 2.4.6でGemfile.lockを再作成

#444, #463


text:
Follow up to #222: fix signatures of luaopen_* functions in hlua

Ah, I forgot to update the main library, which was definitely wrong.
These functions are not documented at all, but looking at their source code they push a table on the stack, so we don't want to just expose them. Instead, they should at least return a PushGuard.

Oh well, I'm just here to silence some compilation issues. To me it'd be fine if these fns continued to not return anything.
Looking at the code, I'm not confident at all in how I should create these PushGuards :) Maybe you can help?

authors: fenollp tomaka

repo: tomaka/hlua


text:
App is showing errors while removing a song from the list

torvalds.dev is analyzing the ticket

torvalds.dev is analyzing the ticket

The task is to understand the issue where the application is showing errors while removing a song from the list and suggest relevant actions. Besides that, we need to identify the files related to the issue.
Plan:

Analyze the provided code to understand the structure and functionalities of the application.
Identify the part of the code responsible for song list management, since the issue stems from there.
Propose potential actions to fix the issue based on the analysis.

Here is the analysis based on the provided files:
1. src/main.ts
This is the main file that initializes the VueMiniPlayerCore and appends a song to the player.
2. src/assets/fonts/iconfont/iconfont.js
This file contains SVG string definitions for icons. This is not directly related to the song list error.
3. VueMiniPlayerCore.ts
This file contains the core logic of the application. It's here where the songs are managed in a list, and it's very likely directly related to the problem at hand.
Observations:
The issue is that the app shows errors when trying to remove a song from the list. In the provided VueMiniPlayerCore.ts file, I see methods for appending a song (AppendSong) and updating the song position in the list (UpdateSongOrderList), but I don't see a method for removing a song from the list.
Suggested Actions:

Create a RemoveSong method in the MusicPlayerCore class in VueMiniPlayerCore.ts file that removes a song from the SongIdList and SongIdMap properties based on the song id.
The method should also handle cases where the song id is invalid (doesn't exist in the list), in which case it would log an error message without breaking the application.
After removing a song, if the removed song was the current song being played, the application should either stop or it should play the next song in the list depending on the desired behavior.
Write tests for the RemoveSong method to ensure it's functioning as expected.

Files Relevant to the Issue:

src/main.ts
VueMiniPlayerCore.ts

authors: SiddaniSravani torvalds-dev

repo: torvalds-dev/vue-mini-player


text:
add ut for rank correction

/lgtm
/approve

repo: towhee-io/towhee


text:
Integrating the chatGPT feature to the UI

Are we done with this feature?

done


text:
remove secret

Remove secret in raw_doc sample, merge by yafang cause encounter validation error in my account.

authors: llalls trend-yafang-qiu


text:
feat: Login

=I agree with Nhi about EOL and comment code

authors: QuocThangPhu tringuyennfq

repo: tringuyennfq/CarForRentSymfony


text:
Create account endpoints

Accounts:

POST: accounts/user/login  -> Login as a generic user
GET: accounts/user/email/verification-link  -> Get a user email verification link
POST: accounts/user/email/verify?link={link}  -> Verify a user email address
GET: accounts/user/phonenumber/verification-code  -> Get a User phone number verification code
POST: accounts/user/phonenumber/verify?code={code}  -> Verify a user's phone number
POST:  accounts/user/password/reset  -> Reset a user password
POST:  accounts/user/password/change  -> Change a user password
POST: accounts/admin/login  -> Login as an Admin
POST: accounts/player/signup  -> Create a new player account
POST: accounts/player/login  -> Login as a player

# TODOs:

Model that connects to the database table.
Serializer that serializes the Model
View that is connected to the serializer
URL that provides the data as an output to the client


text:
Adding parabolic terms for P4estMesh{3}

@apey236 FYI you can queue up all changes by looking at the "Files Changed" tab at the top of this PR.

@ranocha can you please review the changes. I applied your requested changes.

The CI failures are same as https://github.com/trixi-framework/Trixi.jl/pull/1582#issuecomment-1646819161 and are unrelated to this PR. @ranocha can we force merge this?

The CI failures should be fixed with https://github.com/trixi-framework/Trixi.jl/pull/1586

Since we don't have a merge queue and I would like to get a new release just including recent bug fixes, I disabled auto-merge for now. I will take care of this PR and merge it today. Please ping me if I forget to do so.

authors: apey236 jlchan ranocha


text:
Add ibm_iam_role_dashboard

It will be revisited based on future requirements.

repo: turbot/steampipe-mod-ibm-insights


text:
If a file was uploaded but no access level was set, the pills shows an error "Select access level".

default should be NOT to set a default access level -> should be validated by librecat.

@netsensei how did you manage to do this? Field access_level is required for every file entry, and it should be one of a list of fixed values.
See https://github.ugent.be/Universiteitsbibliotheek/librecat/blob/master/config/schemas.yml#L226
See https://github.ugent.be/Universiteitsbibliotheek/librecat/blob/master/config/schemas.yml#L159
So I see no way to update this as such without getting a validation error

authors: netsensei nicolasfranck

repo: ugent-library/biblio-backend


text:
Added a new service that calls CloudMap discover

I have been writing a very AWS specific "Copy Out" service - one that is not appropriate for the general Elsa code base as it only really works with S3.
Anyhow, its kind of like a plugin stack that might be installed/uninstalled.
So we need a dynamic (not cloudformation) mechanism by which functionality like this would be discovered.

repo: umccr/elsa-data


text:
Add president Yakubu Gowon

I need to fix some typographical errors


text:
Topic: Cache

I am interested in this. Please assign this issue to me.

repo: upes-open/first-contributions


text:
Fix the 'test:api' target in server/package.json to make it more reliable

I think the main thing Chris has run into so far is timeouts due to race between server starting up and Mocha test runner; I had an idea in https://usdigitalresponse.slack.com/archives/C0324KDQSCR/p1663712820120909?thread_ts=1663709029.578369&cid=C0324KDQSCR for how to potentially fix that

From @mattbroussard : from some interactive debugging: it's not the TCP socket that's slow; we think it's filesystem operations when registering routes (which imports a bunch of modules to do so). Some reports of slow filesystem operations on proxied folders in WSL https://github.com/microsoft/WSL/issues/4197 -- Chris going to try moving repo checkout to a non-proxied folder inside the WSL environment (which if I understand correctly is basically a VM)

Not a problem that can be fixed in our code.

authors: chrisxkeith mattbroussard

repo: usdigitalresponse/usdr-gost


text:
fix typo

Thanks

authors: howjmay sylvestre

repo: uutils/coreutils


text:
Project Tasks

@Rohith0311
Task1- RE framework
Code verified.
Task 2- RE database
Code verified.
Task -3
Nifty data extraction
Code verified.
Task -4
RE Movefiles
Code verified.
Task -5
RE extract value as uppercase and send mail
Code verified.


text:
fix-casing-block-no-sound: Fixed

a tester avant de le valider

repo: valentinlamine/CreateNuclearFabric


text:
Create PROFILE.md

true


text:
Create selectionSort.cpp

Give better commit messages.

authors: TanishkaKumari vikram-kangotra

repo: vikram-kangotra/DSalgo


text:
Improve camera controls

This has been addressed in an initial pass by Luke M's recent submssions. We can log issues for subsequent issues based on feedback from Luke H and Joel.

authors: cdiggins vim-sroberge

repo: vimaec/vim-webgl-viewer


text:
Add Update_at

Thank you so much
i pull request without token

repo: vinitshahdeo/hacktoberfest-issue-hunt


text:
Use ghcr

Coverage: 47.668%. Remained the same when pulling f7cc9c9f8fd4189daf6fcd06151f80a50992a78e on eiriksm-patch-1 into 5979fc2fad8a5dd79b02ac1f1c335ae54d3d2c02 on main.

authors: coveralls eiriksm

repo: violinist-dev/queue-starter


text:
update ai site

@vitprajzler cant merge

repo: vitprajzler/node-in-english


text:
Docs: Added deployment to Kinsta guide

Seems that Kinsta does not provide free trial / free usage for personal account. We are not going to add the detailed guide in our docs.
Maybe you can post to somewhere else and add a link here like Layer0 did.

Kinsta provides a free $20 trial (it should more than enough for one whole
month).
pt., 9 gru 2022, 07:28 użytkownik Xinyu Liu @.***>
napisał:

Seems that Kinsta does not provide free trial / free usage for personal
account. We are not going to add the detailed guide in our docs.
Maybe you can post to somewhere else and add a link here like Layer0 did.
—
Reply to this email directly, view it on GitHub
https://github.com/vuepress/vuepress-next/pull/1177#issuecomment-1343908632,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAR34OUNY23K4I72J4PVLMLWMLGSHANCNFSM6AAAAAASR7GVIU
.
You are receiving this because you authored the thread.Message ID:
@.***>

Only provider having free plan can be added.

Ok, I will transform it to the Layer0 style, I just thought that if Heroku
is here, there isn't such a limitation.
pt., 9 gru 2022 o 09:19 Mr.Hope @.***> napisał(a):

Only provider having free plan can be added.
—
Reply to this email directly, view it on GitHub
https://github.com/vuepress/vuepress-next/pull/1177#issuecomment-1343993002,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAR34OUS7MH64IGFODKYHITWMLTSHANCNFSM6AAAAAASR7GVIU
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Pozdrawiam
Maciej Palmowski
FreshPixels
ul. Kościuszki 3
90-418 Łódź
www.freshpixels.pl

@Mister-Hope We prepared an article on our end, and I patched the docs to match the Layer0 style, could you reopen the PR?

authors: Mister-Hope meteorlxy palmiak

repo: vuepress/vuepress-next


text:
Merging Lab 2 to Main

Merging


text:
1.5

Codecov Report

Merging #8 (88460a9) into main (0a129c8) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main        #8   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines            5         5
=========================================
  Hits             5         5

Flag
Coverage Δ

unittests
100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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 0a129c8...88460a9. Read the comment docs.

authors: codecov-commenter vulogov

repo: vulogov/nrquery


text:
Advisory fix

Reviewed and Approved

repo: wagov/wasocshared


text:
可以兼容ie吗

很抱歉，目前无法兼容ie

authors: Perferson wchbrad

repo: wchbrad/vue-easy-tree


text:
Clicking the URL of latest 100 LPs in PancakeSwap dashboard cannot work

e.g.:  the navigation cannot work.


text:
WW-250 Simplify binding

@kevinbarfleur on peut avoir un titre plus parlant vis à vis du ticket stp ^^?

Pas besoin de migration du coup?

Pas besoin de migration du coup?

Nope, tout est bon

authors: AurelieV kevinbarfleur

repo: weweb-assets/ww-form-dropdown


text:
Sort alliances by team number on alliance screen

Created sort by number function
https://github.com/wmaltmann/alliance-selector/commit/98cb78a24f8443249fe317556d72473d5c6b5520

repo: wmaltmann/alliance-selector


text:
5.2-2

Migrated from CormenSol