text:
Accessing ValidationFailure from ValidationContext

Currently ValidationFailure are internal in ValidationContext in IValidationContext.cs. Is there a way to access the Failures. I would like to check the severity of the validations.
Any suggestions will be appreciated .

Hi @farshidnz, this library doesn't expose the ValidationContext and IValidationContext types, those are part of the FluentValidation library. Perhaps post your question in the FluentValidation repo?

authors: farshidnz mvdgun


text:
Making the paging SEO friendly

I have partially created an update so the paging will be more seo friendly. I have modified the code to create urls in the format:
http://www.domainname.com/blog/page/2
http://www.domainname.com/blog/categories/CategoryName/page/2
instead of
http://www.domainname.com/blog?p=2
but I cannot see how to modify the routing within articulate. For now I have created a URL rewrite rule.
Modifications are in the attached images.

+1 :)

could you explain why this would be more seo friendly?

+10 static urls are better than dynamic ones from an seo point of view.
http://searchengineland.com/seo-friendly-url-syntax-practices-134218
http://www.wordtracker.com/academy/learn-seo/technical-guides/seo-clean-urls

yes, clean URLs are great, and Articulate/Umbraco follow these rules, but there is a reason for query strings = for dynamically displayed content. Query strings for paging or searching isn't poor SEO because the content displayed for these pages is dynamic based on the query strings. That said it would be better SEO to have the query string parameter say 'page' instead of 'p'
It wouldn't be hard to change the paging and searching URLs to use paths instead but I actually think this is worse for SEO since this URL is less meaningful:
http://www.domainname.com/blog/page/2
than this:
http://www.domainname.com/blog?page=2

Any chance to make it optional?

Yeah, i'll keep this issue open and review when i get some time but happy for anyone else to implement this if they have time

I don't see any SEO advantages in using paths instead of query strings - on the contrary i prefer query strings on my sites.
However i suggest optimizing the paging url's - please see PR #99:
[...] Avoid duplicate content by redirect ?page=1 links. Don't generate links with ?page=1. [...]

+1

I'm marking this as won't fix as I also agree that query strings are the correct approach. If necessary you can use rewrite rules if you don't agree. Thanks all.

source: gharchive/issue

authors: Shazwazza benediktvaldez biapar meixger shoecake vincentparrett

repo: Shazwazza/Articulate

license: mit

license_type: permissive

license_source: bigquery


text:
error when open generated xls file in exel 2016 or exel online

But xlsx file correct  (only change bookType "xls" => "xlsx")
Attach files - https://drive.google.com/open?id=1cLfXMfskb8eYzIBG2xcrntObzk6va-wH

How was the xls file generated?  If you used bookType "xls" the generated file would have been an XLS file.  This looks like a mal-formed XLSX file.
To verify the file type, look at the first few bytes:
$ xxd test.xlsx | head -n 2
00000000: 504b 0304 0a00 0000 0000 bc82 334d d692  PK..........3M..
00000010: 7c11 5a01 0000 5a01 0000 1100 0000 646f  |.Z...Z.......do

$ xxd test.xls | head -n 2
00000000: 504b 0304 0a00 0000 0000 e781 334d d692  PK..........3M..
00000010: 7c11 5a01 0000 5a01 0000 1100 0000 646f  |.Z...Z.......do

$ xlsx -8 test.xlsx # generate an XLS file using the library
$ xxd test.xlsx.xls | head -n 2
00000000: d0cf 11e0 a1b1 1ae1 0000 0000 0000 0000  ................
00000010: 0000 0000 0000 0000 3e00 0300 feff 0900  ........>.......

XLS files generated by the library will start with 0xD0 0xCF 0x11 0xE0, while XLSX files start with the ZIP marker 0x50 0x4b 0x03 0x04

Sorry, but why Node.js XLSX module doesn't supported biff8 type?
Quote from source model code (typical path to file: "./node_modules/xlsx/xlsx.js", line number: 12542):
function writeSync(wb, opts) { 	var o = opts||{}; 	switch(o.bookType || 'xlsx') { 		case 'xml': return write_string_type(write_xlml(wb, o), o); 		case 'csv': return write_string_type(write_csv_str(wb, o), o); 		case 'fods': return write_string_type(write_ods(wb, o), o); 		case 'biff2': return write_binary_type(write_biff_buf(wb, o), o); 		case 'xlsx': 		case 'xlsm': 		case 'xlsb': 		case 'ods': return write_zip_type(wb, o); 		default: throw new Error ("Unrecognized bookType |" + o.bookType + "|"); 	} }

repo: SheetJS/js-xlsx

license_source: github-api


text:
Will not import value if numeric format = "Rs."#,###.00

For some reason, if number formatted "Rs."#,###.00, the numeric value will not be properly be converted.   See attached.   -Rs.51,968,287.00  imports as "Rs.00"    .. it should be 51,968,287.00
testrs.xlsx

@dougschiller The parser was tripped up by the presence of the . in the text.  That was resolved in SSF 0.9.1 and will be pushed here as XLSX version 0.9.13.

authors: SheetJSDev dougschiller


text:
vue 项目中 解析带有拆分的xlsx文件 数据行数缺失问题

环境：
node -v : v14.19.0
vue @2.6.14
如果xlsx文件中存在拆分，通过 antd vue 中的文件上传组件上传改文件，通过XLSX.utils.sheet_to_json 获取数据是缺少数据
观察WorkBook对象sheet对象的!ref值不对
请问怎么解决？

Check the value of ws["!ref"] when you read the file.  It may not be including all of the cells.
If the range is incorrect, try copying and using update_sheet_range from https://github.com/SheetJS/sheetjs/wiki/General-Utility-Functions#updating-worksheet-range
If the range is correct, please share a file and we will take a closer look

Share a file and we can reopen the issue.

repo: SheetJS/sheetjs


text:
ENH: Untrack .c files generated by Cython

It is not necessary to store these files in Github. They should be generated locally after a clone.
Tracking them leads to unnecessary commits, such as the last one in #719.
Can also mask issues in underlying .pyx files. I assume this is how the issues fixed in #697 must have made it into the devel branch. Otherwise tests could not have passed.
.c files added to .gitignore
cythonised c files untracked
Any future C files which are not generated by Cython can be added using
git add -f

The two C files that fit that description are not untracked by this PR.

This is great, thanks for improving the code sanitization 🙏
Problem is it's not passing CI (the build expects these .c files).
Once that's fixed, this LGTM, good engineering.

Crap, ok, so upon reflection - this is a great upkeep PR and I missed merging it and now it's out of date. If you update it I'd say let's just get it in. Otherwise this PR will hang here for a moment longer.

The only thing blocking this (otherwise correct) PR is to update the CI so it builds - and therefore doesn't require - the .c files you've removed. It's the right move, just incomplete.

Closing due to no activity & merge conflicts

source: gharchive/pull-request

authors: ajgpitch ekalosak

repo: SheffieldML/GPy


text:
[Bug: shopify cli You have reached the limit for the number of extensions you can create

Please confirm that you have:

[X] Searched existing issues to see if your issue is a duplicate. (If you’ve found a duplicate issue, feel free to add additional information in a comment on it.)
[X] Reproduced the issue in the latest CLI version.

In which of these areas are you experiencing a problem?
App, Extension
Expected behavior

Create a fresh app using npm init @shopify/app@latest
Choose node
After completion, cd to app dir
Run npm run shopify app generate extension
CLI proceeds and allows me to choose the type of extension to create

Actual behavior
Instead of allowing me to choose the type of extension to create, CLI displays error:
You have reached the limit for the number of extensions you can create.

I am using Node.js version 18.18.1 on Ubuntu and I encountered this error as well. However, on my MacBook, which has the same Node.js version 18.18.1 and Shopify CLI version 3.52, everything is working fine.
Verbose output
pnpm shopify app generate extension

> max-1k@ shopify /home/tien/Desktop/javis's works/max-1k
> shopify "app" "generate" "extension"

╭─ info ─────────────────────────────────────╮
│                                            │
│  Using shopify.app.toml:                   │
│                                            │
│    • Org:          home                    │
│    • App:          max-1k                  │
│                                            │
│   You can pass `--reset` to your command   │
│  to reset your app configuration.          │
│                                            │
╰────────────────────────────────────────────╯

╭─ error ────────────────────────────────────╮
│                                            │
│  You have reached the limit for the        │
│  number of extensions you can create.      │
│                                            │
╰────────────────────────────────────────────╯

 ELIFECYCLE  Command failed with exit code 1.

Reproduction steps

Create a fresh app using npm init @shopify/app@latest
Choose node
After completion, cd to app dir
Run npm run shopify app generate extension

Operating System
Ubuntu 20.04
Shopify CLI version (check your project's package.json if you're not sure)
3.52
Shell
No response
Node version (run node -v if you're not sure)
v18.18.1
What language and version are you using in your application?
node

You can only create 1 theme app extension per app.
Like in this issue: https://github.com/Shopify/cli/issues/3069

Hi @isaacroldan could you give me a solution for this issue

repo: Shopify/cli

license: MIT


text:
[Bug]: "Unexpected token 'with'" on Shopify App

Please confirm that you have:

[x] Searched existing issues to see if your issue is a duplicate. (If you’ve found a duplicate issue, feel free to add additional information in a comment on it.)
[X] Reproduced the issue in the latest CLI version.

In which of these areas are you experiencing a problem?
App
Expected behavior
I followed the tutorial on how to initialize a Shopify App using Shopify CLI v3.0 on Node. The app is supposed to start after installation and show the home screen from "app._index.ts".
Actual behavior
An error popped out showing "Unexpected token 'with'"
Verbose output
Prisma schema loaded from prisma\schema.prisma
11:46:05 │ remix    │
11:46:05 │ remix    │ ✔ Generated Prisma Client (v5.14.0) to .\node_modules\@prisma\client in 96ms
11:46:05 │ remix    │
11:46:05 │ remix    │ Start using Prisma Client in Node.js (See: https://pris.ly/d/client)
11:46:05 │ remix    │
11:46:05 │ remix    │ import { PrismaClient } from '@prisma/client'
11:46:05 │ remix    │ const prisma = new PrismaClient()
11:46:05 │ remix    │ ```
11:46:05 │ remix    │ or start using Prisma Client at the edge (See: https://pris.ly/d/accelerate)
11:46:05 │ remix    │ ```
11:46:05 │ remix    │ import { PrismaClient } from '@prisma/client/edge'
11:46:05 │ remix    │ const prisma = new PrismaClient()
11:46:05 │ remix    │ ```
11:46:05 │ remix    │
11:46:05 │ remix    │ See other ways of importing Prisma Client: http://pris.ly/d/importing-client
11:46:05 │ remix    │
11:46:05 │ remix    │ ┌─────────────────────────────────────────────────────────────┐
11:46:05 │ remix    │ │  Deploying your app to serverless or edge functions?        │
11:46:05 │ remix    │ │  Try Prisma Accelerate for connection pooling and caching.  │
11:46:05 │ remix    │ │  https://pris.ly/cli/--accelerate                           │
11:46:05 │ remix    │ └─────────────────────────────────────────────────────────────┘
11:46:05 │ remix    │
11:46:08 │ remix    │ Prisma schema loaded from prisma\schema.prisma
11:46:08 │ remix    │ Datasource "db": SQLite database "dev.sqlite" at "file:dev.sqlite"
11:46:09 │ remix    │
11:46:09 │ remix    │ SQLite database dev.sqlite created at file:dev.sqlite
11:46:09 │ remix    │
11:46:09 │ remix    │ 1 migration found in prisma/migrations
11:46:09 │ remix    │
11:46:09 │ remix    │ Applying migration `20240530213853_create_session_table`
11:46:09 │ remix    │
11:46:09 │ remix    │ The following migration(s) have been applied:
11:46:09 │ remix    │
11:46:09 │ remix    │ migrations/
11:46:09 │ remix    │   └─ 20240530213853_create_session_table/
11:46:09 │ remix    │     └─ migration.sql
11:46:09 │ remix    │
11:46:09 │ remix    │ All migrations have been successfully applied.
11:46:09 │ webhooks │ App isn't responding yet, retrying in 5 seconds
11:46:11 │ remix    │ ✔ Console Ninja extension is connected to Remix, see https://tinyurl.com/2vt8jxzw
11:46:14 │ webhooks │ App isn't responding yet, retrying in 5 seconds
11:46:19 │ webhooks │ App hasn't started in time, giving up
11:46:19 │ webhooks │ APP_UNINSTALLED webhook delivery failed
Error forwarding web request: AggregateError
11:46:46 │ remix    │   ➜  Local:   http://localhost:50417/
11:46:46 │ remix    │   ➜  Network: use --host to expose
11:47:08 │ remix    │ [shopify-api/INFO] version 11.0.0, environment Remix
11:47:35 │ remix    │ 11:47:35 AM [vite] Error when evaluating SSR module /app/routes/app.tsx: failed to import "@shopify/shopify-app-remix/react"
11:47:35 │ remix    │ |- SyntaxError: Unexpected token 'with'
11:47:35 │ remix    │     at ModuleLoader.moduleStrategy (node:internal/modules/esm/translators:118:18)
11:47:35 │ remix    │     at callTranslator (node:internal/modules/esm/loader:273:14)
11:47:35 │ remix    │     at ModuleLoader.moduleProvider (node:internal/modules/esm/loader:278:30)
11:47:35 │ remix    │
11:47:35 │ remix    │ 11:47:35 AM [vite] Error when evaluating SSR module virtual:remix/server-build: failed to import "/app/routes/app.tsx"
11:47:35 │ remix    │ |- SyntaxError: Unexpected token 'with'
11:47:35 │ remix    │     at ModuleLoader.moduleStrategy (node:internal/modules/esm/translators:118:18)
11:47:35 │ remix    │     at callTranslator (node:internal/modules/esm/loader:273:14)
11:47:35 │ remix    │     at ModuleLoader.moduleProvider (node:internal/modules/esm/loader:278:30)
11:47:35 │ remix    │
11:47:35 │ remix    │ 11:47:35 AM [vite] Internal server error: Unexpected token 'with'
11:47:35 │ remix    │       at ModuleLoader.moduleStrategy (node:internal/modules/esm/translators:118:18)
11:47:35 │ remix    │       at callTranslator (node:internal/modules/esm/loader:273:14)
11:47:35 │ remix    │       at ModuleLoader.moduleProvider (node:internal/modules/esm/loader:278:30)
11:55:37 │ remix    │ 11:55:37 AM [vite] Internal server error: Unexpected token 'with'
11:55:37 │ remix    │       at ModuleLoader.moduleStrategy (node:internal/modules/esm/translators:118:18)
11:55:37 │ remix    │       at callTranslator (node:internal/modules/esm/loader:273:14)
11:55:37 │ remix    │       at ModuleLoader.moduleProvider (node:internal/modules/esm/loader:278:30)

Reproduction steps

Create a shopify app using shopify app init
Choose "Install with Remix" and "Typescript" for language.
Upon finish installation, start the app using shopify app dev
Preview the app by pressing "p".
Install the app on your partner store.
Error pop out.

Operating System
Windows 11
Shopify CLI version (check your project's package.json if you're not sure)
3.60.1
Shell
Powershell
Node version (run node -v if you're not sure)
v20.9.0
What language and version are you using in your application?
Typescript on Node v20.9.0

It was an issue server-side. It should work now no matter the version, sorry for the inconvenience! We'll try to ensure it doesn't happen again.

Hi @gonzaloriestra ! This issue is still happening to me, should it simply work again or do I need to do something in my App's code? I just generated a new app and am trying to run it

This is happening again with:
Ubuntu 24.04
Node 22.6.0
Shopify CLI 3.65.3
Same error message:
remix │ 6:38:29 PM [vite] Error when evaluating SSR module /app/routes/app.jsx: failed to import "@shopify/shopify-app-remix/react"
remix │ |- SyntaxError: Unexpected token 'with'
remix │     at ModuleLoader.moduleStrategy (node:internal/modules/esm/translators:152:18)
remix │     at ModuleLoader.moduleProvider (node:internal/modules/esm/loader:298:14)
remix │
remix │ 6:38:29 PM [vite] Error when evaluating SSR module virtual:remix/server-build: failed to import "@shopify/shopify-app-remix/react"
remix │ |- SyntaxError: Unexpected token 'with'
remix │     at ModuleLoader.moduleStrategy (node:internal/modules/esm/translators:152:18)
remix │     at ModuleLoader.moduleProvider (node:internal/modules/esm/loader:298:14)
remix │
remix │ 6:38:29 PM [vite] Internal server error: Unexpected token 'with'
remix │       at ModuleLoader.moduleStrategy (node:internal/modules/esm/translators:152:18)
remix │       at ModuleLoader.moduleProvider (node:internal/modules/esm/loader:298:14)

I tried it again this morning on my windows 10 machine, running node 22.3.0 and shopify CLI v3.65.3, and it worked fine.
The errors I had encountered before were running on an Ubuntu 24.04 VM on the remix template with default selections prompted by shopify app init.
I'm torn between a few projects at work right now, but I plan to investigate and submit a full ticket about the issue running on the VM when I have some time.
Thank you for your help, and for all the work you do to keep up this project!

I’m encountering the same "Unexpected token 'with'" error as mentioned in the discussion when trying to run my Shopify app. Here are the details of my setup:

Operating System: macOS 13.6.3
Shopify CLI Version: 3.66.1
Node.js Version: 22.7.0

Steps to Reproduce:

Created a new Shopify app using shopify app init.
Chose "Install with Remix" and "JavaScript" for the language.
After finishing the installation, started the app using shopify app dev.
Pressed "p" to preview the app and installed it on my development store.
Encountered the "Unexpected token 'with'" error in the console logs.

Error Details:
Here are some excerpts from my error logs:

[vite] Error when evaluating SSR module /app/routes/app.jsx: failed to import "@shopify/shopify-app-remix/react"
|- SyntaxError: Unexpected token 'with'
    at ESMLoader.moduleStrategy (node:internal/modules/esm/translators:117:18)
    at ESMLoader.moduleProvider (node:internal/modules/esm/loader:218:14)

I’ve tried updating my Node.js version and ensuring all dependencies are up to date, but the error persists.
It seems similar to the issues others have reported with Node versions 20.0.0 - 20.9.0, but I’m currently using 22.7.0.
Any insights or suggestions would be greatly appreciated!

authors: Hassan-Bin-Tariq Saverton gonzaloriestra imterryng pedro108


text:
Improve environment variable docs

Description
Adding more examples.

Before submitting the PR, please make sure you do the following:

[x] Read the Contributing Guidelines
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123)
[x] Update docs in this repository according to your change
[ ] Run yarn changeset add if this PR cause a version bump based on Keep a Changelog and adheres to Semantic Versioning

Alright, thanks @mcvinci !

repo: Shopify/hydrogen


text:
Add mochaContextToGlobalReferences codemod

This PR adds a new transform to handle the renaming of some things that are injected into the test context (spy/ stub/ clock etc) to use a global defined for the tests instead. The codemod allows you to define these as either replacing the whole member expression (this.clock -> testClock) or to just replace the object in the member expression with the test global (this.spy -> sinon.spy).
cc/ @goodforonefare

One concern about teardowns, but this looks really good.  The config bikeshedding isn't really a blocker.

🚢

authors: GoodForOneFare lemonmade

repo: Shopify/javascript


text:
Question about idiomaticness of "Codecs"

Why were the codecs defined as x = 1, y = 2, z = n... rather than  x = iota, y = iota, z = iota...? As per go's "iota" keyword?

When you use iota, you basically say: I don't really care what the exact value is as long as all listed values are different. But we do care. They should have the value as defined by the Kafka protocol.

I see, thank you@
On Wed, Jun 10, 2015 at 3:56 AM, Willem van Bergen <notifications@github.com

wrote:

When you use iota, you basically say: I don't really care what the exact
value is as long as all listed values are different. But we do care. They
should have the value as defined by the Kafka protocol.
—
Reply to this email directly or view it on GitHub
https://github.com/Shopify/sarama/issues/471#issuecomment-110699610.

authors: jaytaylor wvanbergen

repo: Shopify/sarama


text:
Address inputSelectors options

Feel like this may be more of a question than issue so please let me know if this is the wrong spot.
As I understand the import { AddressForm } from '@shopify/theme-addresses'; helper it seems as though the inputSelector allow us to define the necessary selectors by a different element (data attribute, etc) but does not allow us to update the div structure/order of our forms?
If that is correct, what's the point/intention of allowing a different name for inputSelectors? Not sure I entirely understand the purpose of the options object.

@t-kelly any word here?

repo: Shopify/theme-scripts


text:
Help : How do I prevent whitespace character completion

Warning: I will close the issue without the minimal init.vim and the
reproduction instructions.
Problems summary
How do I prevent whitespace character completion? I set  'minAutoCompleteLength': 0, but I don't want <space> ,<cr> and other whitespace character to trigger completion. What should I do ?
Expected
Environment Information

ddc.vim version (SHA1):
latest
denops.vim version (SHA1):
latest
deno version(deno -V output):
deno 1.14.1
OS:
windows wsl ubuntu 20.04
neovim/Vim :version output:
NVIM v0.5.0
:checkhealth or :CheckHealth result(neovim only):

health#denops#check
========================================================================
  - INFO: Supported Deno version: `1.14.0`
  - INFO: Detected Deno version: `1.14.1`
  - OK: Deno version check: passed
  - INFO: Supported Neovim version: `0.5.0`
  - INFO: Detected Neovim version: `0.5.0`
  - OK: Neovim version check: passed
  - INFO: Denops status: `running`
  - OK: Denops status check: passed

health#lspconfig#check
========================================================================
 Checking language server protocol configuration
  - INFO: clangd: configuration checked.

health#nvim#check
========================================================================
Configuration
  - OK: no issues found

Performance
  - OK: Build type: RelWithDebInfo

 Remote Plugins
  - OK: Up to date

 terminal
  - INFO: key_backspace (kbs) terminfo entry: key_backspace=\177
  - INFO: key_dc (kdch1) terminfo entry: key_dc=\E[3~

health#provider#check

Provide a minimal init.vim/vimrc with less than 50 lines (Required!)
set encoding=utf-8
"dein Scripts-----------------------------
if &compatible
    set nocompatible               " Be iMproved
endif
set runtimepath+=~/.cache/nvim/dein/repos/github.com/Shougo/dein.vim
let s:dein_path = $HOME . '/.cache/nvim/dein'
let g:dein#auto_recache = 1
call dein#begin(s:dein_path)
call dein#add(s:dein_path)
call dein#add('neovim/nvim-lspconfig')
call dein#add('vim-denops/denops.vim')
call dein#add('Shougo/ddc.vim')
call dein#add('Shougo/ddc-matcher_head')
call dein#add('Shougo/ddc-sorter_rank')
call dein#add('Shougo/ddc-nvim-lsp')
call dein#add('LumaKernel/ddc-file')
call dein#add('Shougo/ddc-converter_remove_overlap')
call dein#end()
call dein#call_hook('source')
call dein#call_hook('post_source')
" Required:
filetype plugin indent on
syntax enable
set background=dark

call ddc#custom#patch_global('sources', [ 'file', 'nvimlsp'])
call ddc#custom#patch_global('sources',
    \ ['around', 'file', 'nvimlsp'],
    \ )
call ddc#custom#patch_global('sourceOptions', {
    \ '_': {
    \   'ignoreCase': v:true,
    \   'matchers': ['matcher_head'],
    \   'sorters': ['sorter_rank'],
    \   'converters': ['converter_remove_overlap'],
    \   'keywordPattern': '[a-zA-Z_]\w*',
    \   'minAutoCompleteLength': 0,
    \ },
    \ 'around': {
    \   'mark': 'A',
    \   'matchers': ['matcher_head'],
    \ },
    \ 'nvimlsp': {
    \   'mark': 'lsp',
    \   'isVolatile': v:true,
    \   'minAutoCompleteLength': 0,
    \   'forceCompletionPattern': "\\.|:\\s*|->"
    \ },
    \ 'file': {
    \   'mark': 'F',
    \   'isVolatile': v:true,
    \ },
    \ })
call ddc#custom#patch_global('sourceParams', {
      \ 'nvimlsp': {'useIcon': v:true},
      \ })
" Use ddc.
call ddc#enable()

lua require'lspconfig'.clangd.setup{}

How to reproduce the problem from neovim/Vim startup (Required!)

nvim -u minimal.vim a.c
in normal mode , type gg
type o

Screenshot (if possible)

I think you should use ddc-source-option-forceCompletionPattern instead.
	call ddc#custom#patch_global('sourceOptions', {
	      \ '_': {
	      \   'forceCompletionPattern': `\S+`,
	      \ })

Thanks . I think I misunderstood the ddc-source-option-forceCompletionPattern option

Thanks . I think I misunderstood the ddc-source-option-forceCompletionPattern option

authors: Shougo krisfans


text:
[WIP] Remove the vimproc dependency

It removes the vimproc dependency.
Please test it.
~Current problem:~
~I cannot get the command output from job_system() in Vim8.~
~Vim8 job API behavior is hard to understand...~

Fixed the current problem.
Thank you for @lambdalisue !
The library is very useful for me.

Hm.
vim-vimlint does not support new style function().
We should use vint instead?

I need Windows testers for the branch.
In Linux, Vim8/neovim works for me.

Windows 10 (14986), Vim 8 (p109), vimproc.vim is not installed.
master branch
Works fine.
vimproc branch
Some operations that use XCOPY are slow (20-30s, but is still successfully), while Vim just freezes and "not responding".  An example of such a command:
xcopy  /EXCLUDE:C:\Users\Alex\AppData\Local\Temp\VIB2DF7.tmp  "C:\Apps\Vim\vimfiles\dein\repos\github.com\othree\jspc.vim\"* "C:\Apps\Vim\vimfiles\dein\.cache\_vimrc\.dein" /E /H /I /R /Y

This command is perform instantly when you manually run from the console (cmd.exe).

Thank you for the testing.
I don't know why xcopy is slow.

I have added sleep.  Please test the latest version.

@Shougo as well.
I changed the command handler to system() and everything works well and quickly.

I can change it easily.  But I don't want to use system() if possible.

It looks like a problem with the job implementation. (?)
btw.
neomake and job runner from vim-quickrun also does not work properly on Windows.
Instead, I use vim-quickrun-job.

Really?  vim-quickrun-job uses timer feature.  I cannot use it in the case.

@AlexMasterov
let result = dein#install#_system([temp])

Does not fix the problem?

@Shougo vim-quickrun-job has no problems with the output from the console.
Need to add /Q here.

Did.
It fixes the problem?

For 77b191cc92 — yes.
I think you understand what the problem was.

After 77b191cc92 I got other errors. But /Q is really working.
ile processing function <SNR>28_timer_handler[1]..<SNR>28_install_async[5]..<SNR>28_check_loop[15]..<SNR>28_check_output[52]..<SNR>28_get_updated_log_message[9]..dein#instal
l#_system[10]..13[4]..dein#job#start:
line   24:
E474: Invalid argument
Error detected while processing function <SNR>28_timer_handler[1]..<SNR>28_install_async[5]..<SNR>28_check_loop[15]..<SNR>28_check_output:
line   52:
E171: Missing :endif
Error detected while processing function <SNR>28_timer_handler[1]..<SNR>28_install_async[5]..<SNR>28_check_loop:

@AlexMasterov Please test the latest version.

7b4b178 works fine, thanks.
And faster than system() 👍

Nice!

@Shougo If the output is not used, maybe you should also add -q to rsync?
rsync documentation:

-q, --quiet
This option decreases the amount of information you are given during the transfer, notably suppressing information messages from the remote server. This option is useful when invoking rsync from cron.

It is nice.  I will do it later.

Force pushed.

After building call dein#build(['deoplete-go']), I get
Error detected while processing function dein#build[1]..dein#install#_build[6]..dein#install#_status:
line    1:
E716: Key not present in Dictionary: status : v:shell_error
E15: Invalid expression: dein#install#_has_job() ? s:job_system.status : v:shell_error

Fixed.

It's very slow again when updating and has('vim_starting'). https://asciinema.org/a/4t1ybnh8qo9xddf3yu9uqbhmp

Here is a comparison between vim-plug, dein, and dein when installing at startup:
vim-plug: https://asciinema.org/a/3rfumjf5yebdiurimph17zvhj
dein: https://asciinema.org/a/21hb8cor76cwtyes2i9ebz6av
startup-dein: https://asciinema.org/a/3c8f9alq064m879ewgmarvcd5
Both dein and vim-plug are very fast, but statup-dein is more than 2x slower than vim-plug and dein!!!

I have implemented asynchronous installation in vim_starting.

I will merge the feature soon.
Please test it more.

@Shougo b90b19b works OK.
@nhooyr also try to use: (--depth 1, processes * n)
let g:dein#types#git#clone_depth = 1
let g:dein#install_max_processes =
    \ exists('$NUMBER_OF_PROCESSORS') ? str2nr($NUMBER_OF_PROCESSORS) * 4 : 8

Merged.
Thank you for testing!

authors: AlexMasterov Shougo nhooyr


text:
Drop non-major IRRs

Update: Removed non-major IRRs.
$ backup/                                                                                internet_route_verification/data/irrs   main
$ ll                                                                                internet_route_verification/data/irrs/backup   main
Permissions Size User      Date Modified Git Name
.rw-------@ 9.6M sichanghe 15 Jun  2023   -I  altdb.db
.rw-------@  40M sichanghe 15 Jun  2023   -I  arin.db
.rw-------@ 551k sichanghe 15 Jun  2023   -I  bboi.db
.rw-------@ 538k sichanghe 15 Jun  2023   -I  canarie.db
.rw-------@ 4.3M sichanghe 15 Jun  2023   -I  jpirr.db
.rw-------@ 2.3k sichanghe 15 Jun  2023   -I  nestegg.db
.rw-------@ 194M sichanghe 15 Jun  2023   -I  nttcom.db
.rw-------@  11k sichanghe 15 Jun  2023   -I  panix.db
$ rm bboi.db canarie.db nestegg.db panix.db                                                                     internet_route_verification/data/irrs/backup   main 67ms
$ ll                                                                                internet_route_verification/data/irrs/backup   main
Permissions Size User      Date Modified Git Name
.rw-------@ 9.6M sichanghe 15 Jun  2023   -I  altdb.db
.rw-------@  40M sichanghe 15 Jun  2023   -I  arin.db
.rw-------@ 4.3M sichanghe 15 Jun  2023   -I  jpirr.db
.rw-------@ 194M sichanghe 15 Jun  2023   -I  nttcom.db
$ ../priority/                                                                                internet_route_verification/data/irrs/backup   main
$ ll                                                                                internet_route_verification/data/irrs/priority   main
Permissions Size User      Date Modified Git Name
.rw-------@ 135M sichanghe 15 Jun  2023   -I  afrinic.db
.rw-------@ 9.4M sichanghe 23 Jun  2023   -I  altdb.db
.rw-------@ 127k sichanghe 15 Jun  2023   -I  apnic.db.as-block
.rw-------@ 1.8M sichanghe 15 Jun  2023   -I  apnic.db.as-set
.rw-------@  11M sichanghe 15 Jun  2023   -I  apnic.db.aut-num
.rw-------@  90M sichanghe 15 Jun  2023   -I  apnic.db.domain
.rw-------@ 5.1k sichanghe 15 Jun  2023   -I  apnic.db.filter-set
.rw-------@ 6.8k sichanghe 15 Jun  2023   -I  apnic.db.inet-rtr
.rw-------@  50M sichanghe 15 Jun  2023   -I  apnic.db.inet6num
.rw-------@ 504M sichanghe 15 Jun  2023   -I  apnic.db.inetnum
.rw-------@  10M sichanghe 15 Jun  2023   -I  apnic.db.irt
.rw-------@ 741k sichanghe 15 Jun  2023   -I  apnic.db.key-cert
.rw-------@ 1.9k sichanghe 15 Jun  2023   -I  apnic.db.limerick
.rw-------@  13M sichanghe 15 Jun  2023   -I  apnic.db.mntner
.rw-------@ 5.6M sichanghe 15 Jun  2023   -I  apnic.db.organisation
.rw-------@  12k sichanghe 15 Jun  2023   -I  apnic.db.peering-set
.rw-------@  15M sichanghe 15 Jun  2023   -I  apnic.db.role
.rw-------@ 189M sichanghe 15 Jun  2023   -I  apnic.db.route
.rw-------@ 177k sichanghe 15 Jun  2023   -I  apnic.db.route-set
.rw-------@  85M sichanghe 15 Jun  2023   -I  apnic.db.route6
.rw-------@ 1.6k sichanghe 15 Jun  2023   -I  apnic.db.rtr-set
.rw-------@  35M sichanghe 23 Jun  2023   -I  arin.db
.rw-------@ 507k sichanghe 23 Jun  2023   -I  bboi.db
.rw-------@ 4.2M sichanghe 15 Jun  2023   -I  bell.db
.rw-r--r--@ 535k sichanghe 23 Jun  2023   -I  canarie.db
.rw-------@  895 sichanghe 15 Jun  2023   -I  host.db
.rw-------@ 4.9M sichanghe 15 Jun  2023   -I  idnic.db
.rw-------@ 3.9M sichanghe 23 Jun  2023   -I  jpirr.db
.rw-------@  12M sichanghe 15 Jun  2023   -I  lacnic.db
.rw-------@  70M sichanghe 15 Jun  2023   -I  level3.db
.rw-------@ 2.3k sichanghe 23 Jun  2023   -I  nestegg.db
.rw-------@ 185M sichanghe 23 Jun  2023   -I  nttcom.db
.rw-------@ 9.0k sichanghe 15 Jun  2023   -I  openface.db
.rw-------@  11k sichanghe 23 Jun  2023   -I  panix.db
.rw-------@ 511M sichanghe 15 Jun  2023   -I  radb.db
.rw-------@  16M sichanghe 15 Jun  2023   -I  reach.db
.rw-------@  14k sichanghe 15 Jun  2023   -I  rgnet.db
.rw-------@ 5.4G sichanghe 15 Jun  2023   -I  ripe.db
.rw-------@  17M sichanghe 15 Jun  2023   -I  tc.db
$ rm bboi.db bell.db canarie.db host.db nestegg.db openface.db panix.db rgnet.db                                   internet_route_verification/data/irrs/priority   main

Full log:
0102parse_log.txt

Originally posted by @SichangHe in https://github.com/SichangHe/internet_route_verification/issues/57#issuecomment-1874055268
This caused these issues to be out-dated:

[ ] #99
[ ] #60 (should be very close)
[ ] #103
[ ] #64

Changed again in https://github.com/SichangHe/internet_route_verification/issues/57#issuecomment-1882399155.


text:
🛑 SmartDisplayPi Backend is down

In 4de6f38, SmartDisplayPi Backend (https://backend-sdp.plios.tech) was down:

HTTP code: 530
Response time: 3054 ms

Resolved: SmartDisplayPi Backend is back up in 38bfab2 after 21 minutes.


text:
视频时长

config.maximumSelectedVideoDuration = 15
config.minimumSelectedVideoDuration = 5
我这样限制时间，然后可剪辑时长却只有10s
有人遇到同样的问题吗？

maximumSelectedVideoDuration不管这里设置多长时间，下面都只显示10s

在视频编辑配置里面

authors: SilenceLove huhaharui

repo: SilenceLove/HXPHPicker


text:
Suggested Edits

All the suggested edits from https://github.com/Microsoft/MixedRealityToolkit-Unity/pull/1920

Merging recommendations, will discuss more changes later

authors: SimonDarksideJ StephenHodgson


text:
selectionAreaContainer is not working correctly?

Do you want to request a feature or report a bug?
This is a bug report
What is the current behavior?
Perhaps I mis-understand how this option should work, but I try to ensure that the drag-select behavior cannot happen outside certain element by specifying a dom node for this option
If the current behavior is a bug, please provide the steps to reproduce.
What is the expected behavior?
I expected that I cannot drag-select outside of the provided selectionAreaContainer dom node
Your environment:
Selection-version: latest from https://github.com/Simonwep/selection/tree/master/dist at the time this is posted
Browser-version:  Chrome Version 75.0.3770.100 (Official Build) (64-bit)
Operating-system:  Windows 10

Ah, I believe you mean boundaries which is designed to specify the boundaries. selectionAreaContainer is just the parent of where the selection-area (which is an element) should be appended :)
If you got confused by boundaries vs selectionAreaContainer I'll update the readme since it seems like it isn't that obvious :D

Thank you. One quick request if I may: Sometimes we want to use specified dom node(s) as boundaries, is it possible to pass them in as well. I think in the code it just have to check if the values are dom nodes or strings.

I believe all selector/query related stuff get's resolved by this function. So you can mix queries / dom-nodes whatever you want :)

Perfect, thank you @Simonwep. 1 quick question I would like to ask: is there a way to require lets say holding shift key to select? The reason is that when selection is on, other behaviors such as selecting a text to copy is disabled which is quite annoying.

You can use the validateStart event to check if the user pressed the shift key:
{
    validateStart: e => e.shiftKey // Allow selection only if user presses the shift key
    ...yourAwesomeConfig
}

awesome, thank you so much

repo: Simonwep/selection


text:
Removed unneeded TokenUser.ne method.

This method is not needed since Python 3 is the minimum requirement.

#241 should take care of it.

authors: Andrew-Chen-Wang atombrella


text:
global scss throw error

When i use global scss  https://github.com/SimulatedGREG/electron-vue/blob/master/docs/en/using_pre-processors.md.

But it still throw error

Node version: 10.15.1
NPM version: 6.4.1
vue-cli version: 2.9.6
*node-loader version: 0.6.0
node-sass version: 4.9.2
*sass-loader version: 7.0.3
*style-loader version : 0.21.0
*vue-loader version: 15.2.4
Operating System: win10

In addition:   When I give up using the global scss variable to work properly, I comment  vue-loader option config, it actually runs scss as well

me too

  {
    test: /\.scss$/,
    use: ['vue-style-loader', 'css-loader', 'sass-loader?data=@import "./src/renderer/assets/style/globals";']
  },

我也遇到了这个问题，大佬解决了么？

我也遇到了这个问题，大佬解决了么？

No 🙃

{       loader: 'sass-loader',       options: {         data: '@import "./src/renderer/globals";'       }     }
你那个sass-loader用这个替换试试

  {
    test: /\.scss$/,
    use: ['vue-style-loader', 'css-loader', 'sass-loader?data=@import "./src/renderer/assets/style/globals";']
  },

这个可以

I tried every recommanded fix, doesn't work either for me.
Might be a windows issue I guess...

  {
    test: /\.scss$/,
    use: ['vue-style-loader', 'css-loader', 'sass-loader?data=@import "./src/renderer/assets/style/globals";']
  },

well down🤘

repo: SimulatedGREG/electron-vue


text:
Electron-builder autoupdate

Found an issue or bug with electron-vue? Tell me all about it!
Although referenced in the documentation and issue #502, electron-builder auto-updater is not working. I am not using code signing, but I am curious what about the boilerplate is keeping it from working since I have tested generic electron apps (such as auto update example) that work without code signing. I thought code signing was only required with mac.
Describe the issue / bug.
When using the default code in main/index.js, as well as modified code from auto updater example repository, auto update is not working, and I cannot even get electron-log to create a log.
How can I reproduce this problem?
Uncomment the code in main/index.js, set-up package.json file and create repository/git and link and publish and update.
If visual, provide a screenshot.
#
Tell me about your development environment.

Node version: 10.15.1
NPM version: 6.9.0
vue-cli version: (if necessary)
Operating System: windows 10

Update. Auto update does work without keys for windows only. My main issue was that I was trying to auto update a private repository. My workaround was to have two repositories. One for the source code which is private, and one for the releases, which is public. I have attached the most basic SimulatedGreg repository with auto update working https://github.com/wstefan20/autoupdate


text:
Beneath the Wetlands won't boot w/ sinytra

Mod Name
Beneath the Wetlands
Mod Homepage
https://modrinth.com/mod/beneath-the-wetlands
Describe the bug
incompatibility
Steps to reproduce
No response
Logs
https://gist.github.com/Its-Just-Luci/ec251ae851dc82f8a1e3e6bc37d6a4c8
Additional context
No response

Confirmed it's working. I had geckolib for forge present in my other install, but it seems that the translation layer only works one way. Switched to the fabric version and all is well. Closing the issue with this.

repo: Sinytra/Connector


text:
Jukebox plays Music Disc again on the "off" end

Describe the bug
When a Music Disc ends playing, it start playing again (with redstone emitted from the Jukebox and everything) if you have a Hopper below it. It extract the Disc but still plays the music fully
Steps to reproduce
1.Play a Music Disc onto a Jukebox
2.Extract it via Hopper (when Disc ends, since Hopper is locked by Jukebox)
3.Music Disc start playing again (locking the Hopper, which already has the Music Disc)
Logs
No response
Additional context
I used Mellohi as a test because it's the shorter one (01:36) but can confirm it happens with Far (02:54) and Wait (03:58), so my guess is this is extended to all the Discs.
MODS:
Connector-1.0.0-beta.32+1.20.1
ConnectorExtras-1.8.0+1.20.1
fabric-api-0.90.7+1.10.3+1.20.1

I was able to replicate this issue. The music starts again only once, but the jukebox seems to be in an invalid state afterwards, as it is not possible to interact with it anymore (it also constantly outputs a redstone signal, even after the music stops).
Because Connector itself does not have any mixins (as far as I know) that modify the Jukebox or Hopper behavior I currently suspect this to be a bug in sinytra/forgifiedfabricapi. I will investigate further and report back.

Jeah this is definitely a problem with the forgified fabric transfer api. Maybe @Su5eD can move this issue while I try to figure out why this does not happen with the normal fabric transfer api and how we can fix this?

Sure

authors: ItoKyoshiro Su5eD niklaswimmer


text:
experience editor e2e tests for delivery edge

Description / Motivation
Test automation with selenium webdriverIO for experience editor e2e.
wdio is setup with teamcity reporter to output test results to teamcity 'tests' tab.
wdio is setup with rerun on failures to help with any flaky tests.
wdio is setup with outputting screenshots on failures to help with debugging and is pushed as artifacts in ci job and can access these artifacts in outputs/ release/Tests/integration/results/screenshots/.
Testing Details
I tested my changes locally and in CI and tests are all green.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation update (non-breaking change; modified files are limited to the /docs directory)

Obsolete work

repo: Sitecore/jss


text:
ReferenceError: URL is not defined

I have just started using this module. I have created an elm-git.json file, and have tried to run the command, getting:
$ elm-git-install
/Users/me/.nvm/versions/node/v9.3.0/lib/node_modules/elm-git-install/index.js:98
  const url = new URL(giturl);
              ^

ReferenceError: URL is not defined
    at pathify (/Users/me/.nvm/versions/node/v9.3.0/lib/node_modules/elm-git-install/index.js:98:15)
    at buildUpdateChain (/Users/me/.nvm/versions/node/v9.3.0/lib/node_modules/elm-git-install/index.js:70:21)
    at ensureDependencies (/Users/me/.nvm/versions/node/v9.3.0/lib/node_modules/elm-git-install/index.js:49:16)
    at Object.<anonymous> (/Users/me/.nvm/versions/node/v9.3.0/lib/node_modules/elm-git-install/index.js:17:3)
    at Module._compile (module.js:660:30)
    at Object.Module._extensions..js (module.js:671:10)
    at Module.load (module.js:573:32)
    at tryModuleLoad (module.js:513:12)
    at Function.Module._load (module.js:505:3)
    at Function.Module.runMain (module.js:701:10)

URL isn't available globally until Node 10, try updating your Node version.
@Skinney Might be worth specifying the engine version in the package.json? Although that just produces a warning when installing the package.

That's the thing.  +1 for adding the node version in the README.

Reopening this so I don't forget.

authors: SidneyNemzer Skinney brendangibsondrip

repo: Skinney/elm-git-install


text:
source broken ( sushiscan)

hi is it possible to refresh the sources sushiscan for the french community
it s one of the sources which were really good. and for the moment the source dont work at all
thx a lot

#654

repo: Skittyblock/aidoku-community-sources


text:
Label Request: Language Labels

It would be helpful to introduce language labels (e.g., English, Korean, Mandarin(Chinese)) in issues to allow for better filtering and categorisation of issues based on the language.

Need this

I made some and added them to a few of the issues (im kinda too lazy to go through all of em though) -- ill try and make sure to add them in the future for new issues

authors: NeKoOuO Skittyblock aphronyx


text:
adjustment suggestion

synchronize the tags of post game teams with the endgame one, so you don't need to edit them on two separate pages, but do it on one page, or in the end game, change acronyms for the team logo and separate the victory/defeat of the teams so you can customize better, for example in my layout I can't move just the abbreviation to the space I made in the layout without moving the victory or defeat

already in the picks and bans, for the next update or I don't know the adjustment of the layouts to put the choice and banning of the teams the little arrow changing sides with support for the score of how many victories/defeats there are during the course of the championship, below there is a model drawn of how would it look with the adjustments, if possible add the spells after the end of the choices and banishments

I was forgetting, the whole system has a bug when changing fonts, I tried to change different forms of fonts and not one of them works.   "/..assets/css/font/fonts <//> /..assets/css/font/fontLOL"

As for synchronizing team names, I'll look into making a sync button. The aim is also to remain independent and to be able to change names on each page if desired.
As for the history of victories and defeats, I didn't understand the problem, everything is automated, and updates itself, you just have to move the team names as needed.
As for the choice of ban, I can only retrieve the ban that has been carried out, not the current one. Is that the question?
As for the arrow, I'll make a note of it.
For summoner spells, the problem is that if players choose them after picking all the champions, I can't retrieve the last summoner spells chosen.
That's noted for the font, I'll take a look at where it's stuck.

and finally the mvp screen only without the image of the players with just a shadow is it feasible to do it by api?

authors: Sky-CSC overkido

repo: Sky-CSC/OSL


text:
showStarRateDialog doesn't open Google Play Store page

Describe the bug
showStarRateDialog doesn't open Google Play Store page. On the other hand showRateDialog opens Google Play Store page as expected on the same configuration.
UPD: Looks like the problem is in actionsBuilder. When I add custom actionsBuilder to showRateDialog, it stops open Google Play Store page too.
actionsBuilder: (context) {
return [
FlatButton(
child: Text(Localization.of(context).later),
onPressed: () async {
rateMyApp.callEvent(RateMyAppEventType.laterButtonPressed);
Navigator.pop(
context, RateMyAppDialogButton.rate);
},
),
FlatButton(
child: Text(Localization.of(context).rate),
onPressed: () async {
await rateMyApp.callEvent(RateMyAppEventType.rateButtonPressed);
Navigator.pop(
context, RateMyAppDialogButton.rate);
},
),
];
},
To Reproduce
Follow instructions to add rate_my_app library to flutter project.
Call showStarRateDialog.
Dialog is closed without showing Google Play Store page.
Expected behavior
Google Play Store page should be displayed.
Smartphone (please complete the following information):

Device: Google Pixel 3a, Xioami Mi5
OS: Android 11, Android 9

@stepanov-standy You have to manually call rateMyApp.launchStore() if you want to open the store page in a custom action widget.

Thanks! Should we add this line of code to this section on pub.dev?

I don't think so, as it is only an example, and it only that is only there to suggest you what to do, nothing more.

authors: Skyost stepanov-standy

repo: Skyost/RateMyApp


text:
req

req bro find sticker vhtear https://api.vhtear.com/wasticker?query=lucu&apikey=apikey

besok ya, mau istirahat

besok ya, mau istirahat

selamat beristirahat bg

authors: SlavyanDesu rafaelmoris-hash

repo: SlavyanDesu/BocchiBot


text:
[WIP Tracker] Update to 1.19

Description
Slowly working on porting to 1.19. Will not build let alone run yet.
Waiting for us to be able to run on latest forge with registrate before merge.
Fixes #59
Todo

[x] Buildable
[x] Runnable
[ ] Fix issues
[ ] Tidy the new code
[ ] Review old practice, make sure we're following standards.
[x] Wait for PR's to be merged
[x] Fix merge
[x] Upgrade to latest forge
[x] Wait for registrate
[ ] Wait for registrate to update past fluid rework
[x] New codesplitting technique

Issues:

[x] Energy and progress tooltips don't display numbers
[ ] Fix door datagen MinecraftForge/MinecraftForge#8687
[x] Check datagen is fixed in EIODecor once we bump past forge 41.0.41

Note: this PR got big, so some of the 1.19 porting stuff will continue in future PRs now. Just waiting for the Registrate PR then we'll merge.
Checklist:

[x] My code follows the style guidelines of this project (.editorconfig, most IDEs will use this for you)
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation

Gonna go for merging this soon, waiting for the build to hopefully pass first

repo: SleepyTrousers/EnderIO-Rewrite

license: Unlicense


text:
Push / Pull feature sometimes broken

Issue Description:
Note: If this bug occurs in a modpack, please report this to the modpack author. I will of course be reporting this to FTB, but wanted to post here as well. FTB Infinity for minecraft 1.7.10, modpack version 2.60.
What happens:
After server restarts the push / pull features on at least sag mills and alloy smelters ceases to function until you clear all machine configs, break the machine, place it back down, and reconfigure the side you want to push / pull. This happens with ANY connected inventory, it doesn't matter weather it be chests or ME interfaces.
What you expected to happen:
Items from connected inventory get pulled into machine, processed, and returned to inventory.
Steps to reproduce:
1.Place Machine
2.Connect it to an adjacent inventory and configure a side to push / pull or both at the same time.
3.Restart server.
...

Affected Versions (Do not use "latest"):

EnderIO:
EnderCore:
Minecraft: 1.7.10
Forge:

Your most recent log file where the issue was present:
[pastebin/gist/etc link here]

Why didn't you fill out the version numbers?

Sorry i didn't notice that part.
Ender IO: 1.7.10-2.3.0.428_beta
EnderCore: 1.7.10-0.2.0.36_beta
Minecraft: 1.7.10
Forge: 7.10.99.99

authors: KillaKlown MatthiasMann

repo: SleepyTrousers/EnderIO


text:
disconnect on joining vanilla server with enderio installed on client

Issue Description:
Enderio 5.0.36
ATM3Remix
Using sponge but reproduced without
What happens:
Connect to fallback server via BungeeCord (using Waterfall) when ATM Remix server goes down. Players can't connect to lobby server as "EnderIO Failed mod's custom NetworkCheckHandler"
What you expected to happen:
Players join vanilla server and it ignores mods
Steps to reproduce:

Install EnderIO 5.0.36 on client
Connect to vanilla server (with bungeecord?)
either time out or get ERROR
...

Affected Versions (Do not use "latest"):

EnderIO: 5.0.36
EnderCore: 0.5.41
Minecraft: 1.12.2
Forge: 1.12.2--2768
SpongeForge? yes
Optifine? no
Server?

Your most recent log file where the issue was present:
[pastebin/gist/etc link here]

Players join vanilla server and it ignores mods

modding doesn't work that way...

Alright. So if I installed EnderIO on the receiving server, everyone connecting to that would need to have it installed too? I've connected to other servers with lobbies running the same pack. Is there no way around this at all?

error when enderio is on server too. Works without enderio/endercore/endergy installed. error

authors: HenryLoenwind krisjenner


text:
Refactored TrackerPosition

- Deleted WindowsSteamVRPipeInputBridge as its no longer used
- Removed `None` variant of `TrackerPosition`
* Changed code to use `Optional<TrackerPosition>` or `TrackerPosition`
  depending on whether it is valid for the position to not exist
- Removed `id` field of `TrackerPosition`
* Fixed improper association of TrackerPosition.WAIST to
  TrackerRole.WAIST

Didn’t mean to press the review button thing oops. Curious what that circle did is all, don’t mind me :p

Reviewing is good! Of course only members with write access can do reviews that satisfy merge requirements, but non-write access ppl leaving reviews is still helpful as a signal that the code looks good

Alos i'm still not sold on Optional. null exists.
It's about two things:
*Forcing the person to address the fact that it can be null
*Documenting that null is an intended value and shouldn't just be checked from habit but rather, actively

Allows the meaning of a lack of a TrackerPositon to be expressed in the signature of functions or classes

rebased on skeleton-data

@Eirenliel did you rebase on something weird? I see commits I did not intend to be in this PR

repo: SlimeVR/SlimeVR-Server


text:
wip: gallery functionality

I will appreciate a wave of constructive criticism :)

How to test:
src/renderer/images: here goes test.png thumbnail of artwork
src/renderer/public/details here goes:

images images of ui icons + sngskin folder
test/tiles
test/test.xml
test/sng.css


text:
How to debug this?

A breakpoint I set a in the Developer Tools do not work, code just runs through. How do I debug this?

Hey @monotv, I think there is no problem on putting a breakpoint to debug the system. I tried it now on Chrome developer tools and It works. can you describe more detailed, please?

@Sly777 forgot about this issue, got it working in VSCode now in the mean time :)
I can not remember the steps but I basically used this:
https://github.com/Microsoft/vscode-recipes/tree/master/Next-js

perfect! :) So I closed this for now.


text:
🛑 My blog(github.io) is down

In 5e12772, My blog(github.io) (https://smartbrave.github.io/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: My blog(github.io) is back up in b3a6854.

repo: SmartBrave/uptime


text:
🛑 My blog(github.io) is down

In 447c279, My blog(github.io) (https://smartbrave.github.io/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: My blog(github.io) is back up in ea3b090.


text:
fix:修复nvue下旋转屏幕下拉刷新宽度异常的问题

fix:修复nvue下旋转屏幕下拉刷新宽度异常的问题

谢谢！！

这是来自QQ邮箱的假期自动回复邮件。您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。

repo: SmileZXLee/uni-z-paging


text:
Allow removal of handlers from event queue/loops

cc #134

Coverage decreased (-0.5%) to 65.537% when pulling b0a0bee2f7f173087d922a6e3deba19c9d6f443b on vberger:drop_handler into 58592bfe787949f0c715cefb69959c858fd06595 on Smithay:master.

Coverage decreased (-0.08%) to 65.961% when pulling b0a0bee2f7f173087d922a6e3deba19c9d6f443b on vberger:drop_handler into 58592bfe787949f0c715cefb69959c858fd06595 on Smithay:master.

authors: coveralls vberger

repo: Smithay/wayland-rs


text:
🛑 MPC Homepage is down

In 82a17ef, MPC Homepage (https://www.minorplanetcenter.net) was down:

HTTP code: 0
Response time: 0 ms

Resolved: MPC Homepage is back up in 3da8998 after 3 minutes.

repo: Smithsonian/upptime


text:
🛑 NEOCP is down

In be36ea1, NEOCP (https://minorplanetcenter.net/iau/NEO/toconfirm_tabular.html) was down:

HTTP code: 0
Response time: 0 ms

Resolved: NEOCP is back up in 97bfd08 after 1 minute.


text:
🛑 NEOCP is down

In 79d5565, NEOCP (https://minorplanetcenter.net/iau/NEO/toconfirm_tabular.html) was down:

HTTP code: 0
Response time: 0 ms

Resolved: NEOCP is back up in ec6a733 after .


text:
🛑 MPC Homepage is down

In 0c0614f, MPC Homepage (https://www.minorplanetcenter.net) was down:

HTTP code: 0
Response time: 0 ms

Resolved: MPC Homepage is back up in 231ec4e after 1 minute.


text:
🛑 MPC Homepage is down

In 1bae99d, MPC Homepage (https://www.minorplanetcenter.net) was down:

HTTP code: 0
Response time: 0 ms

Resolved: MPC Homepage is back up in 5dbd057 after 2 minutes.


text:
profiles documentation

Hi snakemakers! I don't think this is the right place to ask, but I am not sure where else to go. Is there an introductory explanation for beginners about how to set up a profile and specify resource requirements such as memory and walltime for each rule in a snakefile? Thank you!

Hi, I'm also just recently starting to convert all my cluster config to profiles and it has been quite a headache through trial and errors.
Documentation is the only place I've seen and it is very limited so not too helpful.
There are also some example profiles that one can learn by combing through codes. But a more detailed documentation and/or tutorial would certainly be helpful!

authors: SichongP concatenize

repo: Snakemake-Profiles/doc


text:
How to replace a constraint?

I setup constraint for messageInputContainerView before:
        messageInputContainerView.snp.makeConstraints { (maker) in
            maker.height.equalTo(48)
            maker.bottom.equalTo(view.safeAreaLayoutGuide.snp.bottom).offset(0)
            maker.leading.equalToSuperview().offset(0)
            maker.trailing.equalToSuperview().offset(0)
        }

And after, I want to replace maker.bottom.equalTo(view.safeAreaLayoutGuide.snp.bottom).offset(0)
by maker.bottom.equalToSuperview().offset(padding)
How to do it with SnapKit? Replace or delete and add a new constraint
Thank you

snp.removeConstraints() will remove all constraints. Unless you want to remove all and add all them back again. You can use snp.remake but you have to include all 4 constraint you need not just bottom.
What you want is the mechanism to update a single constraint. Check the the

Hold on for dear life

documentation in SnapKit for that.

@phongle6893 I think that the most clear approach is to create both constraints in maleConstraints block, save them as a variables and activate and deactivate them as needed.
var safeAreaBottomConstraint: Constraint?
var superviewBottomConstraint: Constraint?

func setupLayout() {
  view.snp.makeConstraints { make in
     safeAreaBottomConstraint = make.bottom.equalTo(view.safeAreaLayoutGuide.snp.bottom).constraint
     superviewBottomConstraint = make.bottom.equalToSuperview().constraint
  }
  safeAreaBottomConstraint?.activate()
  superviewBottomConstraint?.deactivate()
}

func updateState() {
  let something: Bool = ...
  if something  {
    safeAreaBottomConstraint?.activate()
    superviewBottomConstraint?.deactivate()
  } else {
    safeAreaBottomConstraint?.deactivate()
    superviewBottomConstraint?.activate()
  }

  view.layoutIfNeeded()
}

@danshevluk  that's can work,but  if write  superviewBottomConstraint before the  safeAreaBottomConstraint, the console will print warning or error

@phongle6893 Your code should look like this.
var bottomSafeArea: Constraint?
var bottomSuperView: Constraint?

func setupLayout() {
messageInputContainerView.snp.makeConstraints { (maker) in
    maker.height.equalTo(48)
    bottomSafeArea = maker.bottom.equalTo(view.safeAreaLayoutGuide.snp.bottom).offset(0).constraint
    maker.leading.equalToSuperview().offset(0)
    maker.trailing.equalToSuperview().offset(0)
}

bottomSafeArea?.deactivate()

messageInputContainerView.snp.makeConstraints { (maker) in
    bottomSuperView = maker.bottom.equalToSuperview().offset(padding)
}

}

After that bottomSuperView is active and no console error.
You can now simply.
bottomSuperView.deactivate()
bottomSafeArea.activate()

repo: SnapKit/SnapKit


text:
centerX, centerY offset/inset are the same

New Issue Checklist
🚫 If this template is not filled out your issue will be closed with no comment. 🚫

[ X] I have looked at the Documentation
[ X] I have read the F.A.Q.
[ X] I have filled out this issue template.

Issue Info

Info
Value

Platform
iOS

Platform Version
10.0

SnapKit Version
4.0.0

Integration Method
cocoapods

Issue Description
It appears that centerY.offset(value) does the same thing that centerY.inset(value) does.
In order to circumvent, one must provide a negative value.

yes it does seem that way.  offset and inset are the same.   this is very confusing and unhelpful

Fixed by #569

authors: AlexanderBollbach Cyberbeni sh3244


text:
Datasource snowflake_database not returning values

Provider Version
v0.62.0
Terraform Version
1.4.4
Description
We are seeing an issue where the database level retention policy is being set to a default 1. The provider is querying Snowflake to get the database details correctly however it is then unable to parse those results into the required struct. Snowflake now returns an additional field, kind. The datasource does not return a value so the value just defaults to 1
e.g.
data "snowflake_database" "this" {
  name = var.database_name
}

locals {
retention_time = var.data_retention_days != null ? var.data_retention_days : data.snowflake_database.this.retention_time
}

resource "snowflake_schema" "schema" {
  database            = var.database_name
  name                = local.schema_name
  comment             = "Terraform-managed Schema"
  is_transient        = false
  is_managed          = var.is_managed
  data_retention_days = local.retention_time
}

Unfortunately the provider doesn't error and carries on without logging an error, although if we set
export TF_LOG=DEBUG
we can see
2023-06-14T21:06:38.909+0100 [INFO]  provider.terraform-provider-snowflake: 2023/06/14 21:06:38 [DEBUG] sql-conn-query [query SHOW DATABASES LIKE 'DEV_DB' err <nil> duration 106.320233ms args {}]: timestamp=2023-06-14T21:06:38.908+0100
2023-06-14T21:06:38.909+0100 [INFO]  provider.terraform-provider-snowflake: 2023/06/14 21:06:38 [DEBUG] no databases found: timestamp=2023-06-14T21:06:38.909+0100
2023-06-14T21:06:38.909+0100 [INFO]  provider.terraform-provider-snowflake: 2023/06/14 21:06:38 [DEBUG] list database failed to decode: timestamp=2023-06-14T21:06:38.909+0100

We can see a successful SHOW DATABASES LIKE 'DEV_DB' in the Query History
A couple of potential fixes

Add kind into the various definitions and functions
Use Unsafe() to allow any new fields to be ignore until explicitly added in future versions of the provider

This looks to have been introduced by the change in Snowflake Behaviour documented Behaviour Change in 2023_03

Probably outside of the scope for this MR, but I'd argue that provider code should use Unsafe() across the board (or change to another row mapping approach) so that the provider is not tightly coupled with the current snowflake release

should be fixed by #1872

This is working as of 0.67.0
Closing

authors: WobblyRobbly malduarte mlorek


text:
[Feature]: Add support for new Account parameters ADMIN_USER_TYPE etc

Use Cases or Problem Statement
Snowflake is introducing new parameters for account creation, specifically relevant for use cases where accounts are created programmatically.
As part of our continued effort to improve our customer security, we recently announced enforcement of MFA by default for new Snowflake accounts. This change will be introduced as part of BCR Bundle 2024_08 and your account will be impacted by this change. This bundle is expected to be deployed for testing during the week of October 7th, 2024 in release 8.38.  Please note dates are subject to change.
Details of the changes are below:
What you need to do
We recommend following the steps in this whitepaper and this video. More specifically:
When creating new accounts programmatically, use the newly introduced ADMIN_USER_TYPE property for the admin as part of account creation
CREATE ACCOUNT  [ ADMIN_USER_TYPE = PERSON | SERVICE | LEGACY_SERVICE | NULL ]
For human users, set ADMIN_USER_TYPE=PERSON. PERSON users will be subject to MFA policies.
For service users, set ADMIN_USER_TYPE=SERVICE. SERVICE users will not be subject to MFA policies.
Service users will not be allowed to use passwords by policy and the caller should specify ADMIN_RSA_PUBLIC_KEY instead of admin password as part of account creation.
If your service users cannot use key pair authentication, set ADMIN_USER_TYPE=LEGACY_SERVICE during account creation and continue setting up passwords. LEGACY_SERVICE users will not be subject to MFA policies.
LEGACY_SERVICE  users are not allowed to log in via UI and  cannot have a first name / last name. See this link for more details.
LEGACY_SERVICE is a temporary solution and we highly recommend that you fix your tooling.
Read more about user types and their limitations in the create user documentation.
For users created after a new account is bootstrapped
Make sure to mark the TYPE for all users
For human users TYPE=PERSON, follow the DUO enrollment process and start using MFA
For service users TYPE=SERVICE, make sure to either: 1) use external OAuth (preferred), or 2) use key pair authentication combined with network policy.
If a service user cannot leverage a key pair, mark them as TYPE= LEGACY_SERVICE. This is a temporary solution and we highly recommend that you fix your deployment based on the above.
This feature request would be to update the Account resource to support the newly introduced arguments/parameters, and also update the User resource to support the Snowflake TYPE setting (e.g. TYPE = "SERVICE" or TYPE = "PERSON".
Having had a look, I can see some sort of awareness of this as a //TODO comment in: https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/97fa9b4485cf26c5bf93dedfe3b88f688a71b3e6/pkg/resources/user.go#L206C1-L206C14
[SNOW-1645348]: revisit with service user work
Category
category:resource
Object type(s)
resource:account
Proposal
Update the Account resource
How much impact is this issue causing?
Low
Additional Information
No response
Would you like to implement a fix?

[ ] Yeah, I'll take it 😎

Hey @sean-stage. Thanks for reaching out to us.
We have it on our radar. We will check if the new syntax breaks the current account resource implementation this or early next week and adjust it if needed. I will ;eave a comment in this issue after these tests.
The new parameters will be handled no sooner than with the account resource rework (https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/main/v1-preparations/ESSENTIAL_GA_OBJECTS.MD) that is incoming. Before that you can use https://registry.terraform.io/providers/Snowflake-Labs/snowflake/latest/docs/resources/unsafe_execute to change the type.

Thanks for the response Artur @sfc-gh-asawicki

Hi @sfc-gh-asawicki Artur, any progress on this front? Snowflake Trust Center is now flagging accounts without MFA around this, and it'll be nice to ensure programatically created service style accounts are set to the correct 'type' via Terraform.

Hey, yes: https://github.com/Snowflake-Labs/terraform-provider-snowflake/pull/3236/files#diff-cbd81edd0019b2c801bdab54fa0e2697ee800d9f381edbebe4899eb20fa83bedR52.

Hello 👋
Recently, we released v0.100.0 and v1.0.0 that contains reworked account resource. The the admin_user_type is supported there, please migrate with the use of migration guide and let us know if the issue can be closed. Thank You :)


text:
Added detailed documentation on how to contribute to Pumpkin

I added detailed steps on how to use Github Desktop or git to create a fork, edit said fork, merge code, and create a pull request.

Ignore this, I didn't commit my code before the pull request.

repo: Snowiiii/Pumpkin


text:
"cdk" npm package name

Hi @Snugug, my name is Elad and I am working on the AWS Cloud Development Kit. We have a command line tool called cdk and we'd like users to be able to run it through npx. This requires that the package name will have the same name as the executable.
I saw that you have claimed this npm package name, but haven't seen any activity on this code base for a few years. I am wondering if you'd be willing to give up the package name so our users will have a little less typing to do.

Just talked to Sam on Twitter.
He mentioned that he's swamped with work currently, but will think about our request when he is able to. He also mentioned that he created this repo while working fo a previous company, so they might need to agree to the name transfer as well.

repo: Snugug/cdk


text:
NullReference Exception on Subscribe

Every other start of my service fabric application i get errors on subscribing to messages via the broker.
The subscription is made from the OnOpenAsync method, ad the brokerClient initialized in the constructor.
Is there any issue with the following code or is this maybe a bug in the PubSub implementation?
I also sometimes had a concurrency exception which was also thrown in OnSubscribedAsync Of the DefaultBrokerEventsManager.
Code
        protected override async Task OnOpenAsync(ReplicaOpenMode openMode, CancellationToken cancellationToken)
        {
            await _brokerClient.SubscribeAsync<CamAdded>(this, Handle);
        }

    internal sealed class CamDeviceSvc: StatefulService, ICamDeviceSvc
    {
        private IBrokerClient _brokerClient;

        public XovisDeviceSvc(StatefulServiceContext context)
            : base(context)
        {
            _brokerClient = new BrokerClient();
        }

    public interface IXovisDeviceSvc : ISubscriberService
Stacktrace

I assume ICamDeviceSvc extends ISubscriberService (you mentioned IXovisDeviceSvc).
I assume also that you have implemented ReceiveMessageAsync and CreateServiceInstanceListeners as mentioned in the documentation.  Since you are not using the Base class, you have to implement this boilerplate yourself.  If it's not exactly right, messages won't be delivered.  The base class also has some retry logic that might be helpful.
I assume that Handle has also been implemented.
With these assumptions, everything looks correct.
From the stack trace, it seems to be blowing up in DefaultBrokerEventsManager.OnSubscribeAsync() when it tries to add a new element to the _stats dictionary.  I was able to reproduce the exception in a unit test by setting the ReferenceWrapper to null, but I don't know how this could happen.
Can you provide more details on how to reproduce the error?  Maybe set up a temporary Github project I could clone and play with myself.  How are you starting the Application?  Just "Deploy Solution" in Visual Studio?

@danadesrosiers  A null reference exception on a dictionary is usually caused by concurrent write access to the dictionary.
This is further supported by TS remark about a ConcurrencyException.
So I think the _stats object should probably be a ConcurrentDictionary.

Hi, thanks for the replies.
@danadesrosiers Your assumptions are correct. And i start the app out of VS just by starting the SF App with F5 in Debug mode.
@loekd That would explain why sometimes its a concurrency exception and sometimes its the null when both can be concurrency issues.
I could not find where the problems originate from because there is no further interaction on Startup with the broker client than the subscribe call from my code.

v9.1.1 is now available.  Please let us know if you continue to experience errors.

authors: BoeseB danadesrosiers loekd

repo: SoCreate/service-fabric-pub-sub


text:
🛑 SocialGenius SAAS is down

In 1b2e7e5, SocialGenius SAAS (https://app.socialgenius.net/login) was down:

HTTP code: 0
Response time: 0 ms

Resolved: SocialGenius SAAS is back up in 1370d51 after 22 minutes.

repo: SocialGenius/status


text:
ApplyImpulseAt() called to props upon weapon shots and grenade impact

current caveat as of PR:

Network GameObjects assigned as "Network Snapshot" will not enact any of both functions for clients. Meaning maps that contain pre-allocated props which upon loading are normally assigned as snapshot objects will not register physics impact from clients, unless if its a spawned object from either the host or the client themselves, as spawned objects are fully networked and real.

Sweet, this looks great.

repo: Softsplit/sandbox-classic


text:
Mismatch between parameter counts in sage.py wrapper in ccbench and sage.sh script

Thank you for your work! I'm trying to test sage in ccBench but encountered some issues. In ccBench's https://github.com/Soheil-ab/ccBench/blob/main/pantheon-modified/src/wrappers/sage.py, there's:
cmd = [send_src, args.port, ' 50',' 150',' 10','0', "pure", rl_fld , args.comment,args.num_flows,args.bw,args.basetime_fld,args.bw2,args.trace_period]

which explicitly passes 13 variables, but in send_src = path.join(cc_repo1, 'sage.sh'), sage.sh requires 17 variables as shown in:
if [ $# != 17 ]
then
    echo -e "usage:$0 [latency] [port] [trace downlink] [trace uplink] [iteration ID] [qsize] [Link capacity]"
    echo -e "\t \t[bw2: in case of pulse traces, this indicates BW of the 2nd part, otherwise it's equal the <link capacity>]"
    echo -e "\t \t[trace_period: default 7] [training:1 , evaluation:0] [log prefix] [time] [actor id] [duration: steps] [number of flows] [save] [analyze]"
    echo "you gave $# inputs, but we need 17 inputs!"
    echo "your inputs were $@"
    exit
fi

I cannot match these parameters, as obviously the first input to sage.sh in wrappers/sage.py is not latency. I've been struggling with this for a while, and would greatly appreciate if you could tell me how to modify this so I can test sage in ccBench.

Specifically I don't understand the significance of the parameters ' 50',' 150',' 10','0', I can't relate them to any parameter. Can you please help me to understand the meaning of these parameters?

repo: Soheil-ab/sage


text:
[기능] 해외 주식 조회 및 거래 기능 추가 요청

기능 설명
안녕하세요, python-kis 개발자님,
먼저, 프로젝트를 만들어 주셔서 감사합니다. python-kis가 큰 도움이 되고 있습니다.
프로젝트의 기능 중 하나를 개선하여, 해외 주식 조회 및 거래 기능이 추가 되었으면 하는 마음에서 글을 올립니다.
감사합니다.

기다려주셔서 감사합니다.
현재 라이브러리를 개편하면서, 국내 및 해외 구분 없이 동일한 인터페이스로 쿼리할 수 있는 기능이 개발되었습니다.
pip install python-kis --upgrade

Tutorial#2. 종목 시세 및 차트 조회 문서를 참고하여 원하시는 기능을 구현할 수 있습니다.
from pykis import PyKis

kis = PyKis("secret.json", keep_token=True)
from pykis import KisStock

# 국내 주식
hynix: KisStock = kis.stock("000660")  # SK하이닉스 (코스피)
ecopro: KisStock = kis.stock("247540")  # 에코프로비엠 (코스닥)

# 해외 주식 (미국)
nvida: KisStock = kis.stock("NVDA")  # 엔비디아 (나스닥)
coupang: KisStock = kis.stock("CPNG")  # 쿠팡 (뉴욕)

tokyo_electric: KisStock = kis.stock("9501", market="TYO")  # 도쿄 전력 (도쿄)
# 또는 국가코드를 사용할 수 있습니다.
tokyo_electric: KisStock = kis.stock("9501", market="JP")  # 도쿄 전력 (도쿄)

현재 구현된 모든 조회는 국내외 구분 없이 동일한 코드로 데이터를 얻을 수 있습니다.
from datetime import date
from pykis import KisOrderProfits

profits: KisOrderProfits =account.profits(start=date(2023, 8, 1))

print(repr(profits)) # repr을 통해 객체의 주요 내용을 확인할 수 있습니다.

KisIntegrationOrderProfits(
    account_number=KisAccountNumber('00000000-01'),
    buy_amount=8456747.364,
    sell_amount=8458122.90431,
    profit=1375.54031,
    orders=[
        KisDomesticOrderProfit(time_kst='2024-07-11T00:00:00+09:00', market='KRX', symbol='462870', name='시프트업', buy_price=60000, sell_price=85000, qty=1, profit=25000, profit_rate=41.66666666666666666666666667),
        KisForeignOrderProfit(time_kst='2024-06-18T00:00:00+09:00', market='NASDAQ', symbol='ARM', name='에이알엠 홀딩스(ADR)', buy_price=135.39, sell_price=161.14, qty=7, profit=180.25, profit_rate=19.01912992096905236723539405),
        KisForeignOrderProfit(time_kst='2024-05-23T00:00:00+09:00', market='NYSE', symbol='CPNG', name='쿠팡', buy_price=17.55, sell_price=22.3601, qty=1, profit=4.8101, profit_rate=27.40797720797720797720797721),
        KisDomesticOrderProfit(time_kst='2024-04-01T00:00:00+09:00', market='KRX', symbol='005930', name='삼성전자', buy_price=81700, sell_price=83200, qty=7, profit=10500, profit_rate=1.835985312117503059975520196),
        KisForeignOrderProfit(time_kst='2024-02-29T00:00:00+09:00', market='NASDAQ', symbol='MARA', name='매러선 디지털 홀딩스', buy_price=32.8718, sell_price=29.41, qty=37, profit=-128.09, profit_rate=-10.53146531169322348839886209),
        KisForeignOrderProfit(time_kst='2024-02-28T00:00:00+09:00', market='NASDAQ', symbol='NVDA', name='엔비디아', buy_price=738, sell_price=787.2, qty=1, profit=49.2, profit_rate=6.666666666666666666666666667),
        KisForeignOrderProfit(time_kst='2024-02-28T00:00:00+09:00', market='NASDAQ', symbol='SOUN', name='사운드하운드 AI', buy_price=7.028, sell_price=6.72, qty=45, profit=-13.86, profit_rate=-4.382470119521912350597609562),
        KisForeignOrderProfit(time_kst='2024-02-16T00:00:00+09:00', market='NASDAQ', symbol='AAPL', name='애플', buy_price=194, sell_price=184.46, qty=11, profit=-104.94, profit_rate=-4.917525773195876288659793814),
        KisForeignOrderProfit(time_kst='2024-02-16T00:00:00+09:00', market='NYSE', symbol='CPNG', name='쿠팡', buy_price=17.71, sell_price=15.76, qty=8, profit=-15.6, profit_rate=-11.01072840203274985883681536),
        KisForeignOrderProfit(time_kst='2023-12-13T00:00:00+09:00', market='NYSE', symbol='WMT', name='월마트', buy_price=159.09, sell_price=151.36, qty=1, profit=-7.73, profit_rate=-4.858884907913759507197183984),
        KisDomesticOrderProfit(time_kst='2023-09-04T00:00:00+09:00', market='KRX', symbol='389500', name='에스비비테크', buy_price=55400, sell_price=59200, qty=1, profit=3800, profit_rate=6.859205776173285198555956679),
        KisDomesticOrderProfit(time_kst='2023-08-09T00:00:00+09:00', market='KRX', symbol='228760', name='지노믹트리', buy_price=22200, sell_price=22250, qty=4, profit=200, profit_rate=0.2252252252252252252252252252),
        KisDomesticOrderProfit(time_kst='2023-08-09T00:00:00+09:00', market='KRX', symbol='008930', name='한미사이언스', buy_price=39600, sell_price=40200, qty=2, profit=1200, profit_rate=1.515151515151515151515151515)
    ]
)


text:
Fix linux compile instructions and add steam_appid.txt creation

Include building prebuilt-libs from source repository.
fix typos and versions
add steam_appid.txt creation

@Shoozza could you have a look into this, please?

authors: furai jopadan

repo: Soldat/soldat


text:
why args.dt is not set to 0 at first tick?

I resolved issue by myself. sorry.

Hey, okey, that's great! I saw your posting initally, but didn't have the time to answer before now.
dt and args.dt which you referred to in the are the same because args = UpdateArgs { dt }, so you can use either.
Also, dt specifies how much into the future you want your behavior tree to advance. Typically, you will set the dt time to be the same time as the 1 / rate_of_you_program e.g 1 / 100Hz = 0.01s. The different processes/nodes in your behavior tree will consume of the value dt, so this value should be strictly decreasing as time goes.
e.g when a node returns, it should return whatever status the node resulted in and the input dt time minus the time it took to process the node (Success, dt-process_time) (see concecpts page  for more info, specifially the Events section)
If you have any more questions, just let me know :)

Thanks for answering even though I closed the issue.
After looking at your answer, I have a question.
Here's my simple example code.
#[derive(serde::Deserialize, serde::Serialize, Clone, Debug, PartialEq)]
enum Npc {
    Walk,
    Run,
}

fn game_tick(dt: f64, bt: &mut BT<Npc, String, serde_json::Value>) {
    let e: Event = UpdateArgs { dt }.into();

    bt.state.tick(&e, &mut |mut args: bonsai_bt::ActionArgs<Event, Npc>| {
        match *args.action {
            Npc::Walk => {
                println!("walk! dt: {}", args.dt);
                (bonsai_bt::Success, args.dt)
            },
            Npc::Run => {
                println!("run! dt: {}", args.dt);
                (bonsai_bt::Success, args.dt)
            }
        }
    });
}

fn main() {
    let blackboard = HashMap::new();

    let walk = Action(Walk);
    let run = Action(Run);
    let seq = Sequence(vec![walk, run, Wait(200.0)]);
    let behavior = While(Box::new(WaitForever), vec![seq]);

    let mut bt = BT::new(behavior, blackboard);

    loop {
        game_tick(70.0, &mut bt);
        sleep(Duration::from_secs(1));
        println!("1 sec");
    }

}

Here's the output of the code.
walk! dt: 70
run! dt: 70
1 sec
1 sec
walk! dt: 10
run! dt: 10
1 sec
1 sec
1 sec
walk! dt: 20
run! dt: 20
1 sec
1 sec
1 sec
walk! dt: 30
run! dt: 30
1 sec
1 sec
1 sec
...

My expectation of args.dt:
tick1: Wait 200 - 70 = 130
tick2: Wait 130 - 70 = 60
tick3: Wait 60 - 70 = -10
tick4: now behavior tree is finished so args.dt would be set to 80(remaining dt 10 + UpdateArgs{dt} )
But tick4 sets args.dt to 10. So args.dt is not updated whenever behavior tree is finished.
UpdateArgs{dt}.into works after args.dt is set to 0?

If you BT runs at intervals of 70, and you wait for 200, then you will be overdue of (200 - 210) = -10 as you say. However, I don't portray overdue time as a negative value because that messes with the timing in the following nodes. so it is basically abs(Wait 60 - 70) = Abs (Wait -10 ) = 10
see implementation here: https://github.com/Sollimann/bonsai/blob/42081d26c7344723d6aa0fd4f1e5a8db57bc591d/bonsai/src/state.rs#L144

Here is another question related to time as well, in case that gives you some more clarification: https://github.com/Sollimann/bonsai/issues/10

authors: PudgeKim Sollimann

repo: Sollimann/bonsai


text:
Exporting with no collision material gives an exception error

both embedded and ybn
and when the col has a shader material

When the col has a shader material / without shader material /embbeded or unembedded :
Error exporting: E:\Documents\gta5\Mes projets\SOZ\TESTING\Assets_Socialite\Sollumz\contourporte.ydr.xml
 Traceback (most recent call last):
  File "C:\Users\rigon\AppData\Roaming\Blender Foundation\Blender\4.1\scripts\addons\Sollumz\sollumz_operators.py", line 208, in execute_timed
    success = export_ydr(obj, filepath)
              ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\rigon\AppData\Roaming\Blender Foundation\Blender\4.1\scripts\addons\Sollumz\ydr\ydrexport.py", line 68, in export_ydr
    drawable_xml = create_drawable_xml(drawable_obj, apply_transforms=export_settings.apply_transforms)
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\rigon\AppData\Roaming\Blender Foundation\Blender\4.1\scripts\addons\Sollumz\ydr\ydrexport.py", line 116, in create_drawable_xml
    create_embedded_collision_xmls(drawable_obj, drawable_xml)
  File "C:\Users\rigon\AppData\Roaming\Blender Foundation\Blender\4.1\scripts\addons\Sollumz\ydr\ydrexport.py", line 898, in create_embedded_collision_xmls
    bound_xml = create_composite_xml(bound_obj)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\rigon\AppData\Roaming\Blender Foundation\Blender\4.1\scripts\addons\Sollumz\ybn\ybnexport.py", line 80, in create_composite_xml
    cg /= volume
ZeroDivisionError: Vector division: divide by zero error

repo: Sollumz/Sollumz


text:
Pass in coupler directly

instead of:
gustav.addCoupler(new GustavRedis());
just pass the coupler directly as a source:
let red = new GustavRedis();

gustav.createWorkflow('asdf')
  .source(red, 'some-channel')
  // ...

Now it's easier to work with different types couplers, and has fewer magic strings from default names.

+1, I'm leaning further away from magic strings and globals hidden in Gustav.

repo: SomeKittens/gustav


text:
Sound doesn't work in Gnome Web

Hey. There's no sound for me when playing in the browser Gnome Web 40.0 in Ubuntu 20.10. Sound works on other pages.
Hope you can look into this. Thanks! :)

Hi, thanks for the issue, I’ll give it a try to see if there’s an easy fix. I’ve had a problem with a lot of mobile browsers because they don’t let you play sound without a user interacting with the page first. So, it will play sounds every now and then, but it can’t play them all. I’m not sure if a good way to fix it, but maybe this is unrelated!

I was able to confirm this bug, but I do not know why it happens.
Here is the error that Gnome Web reports when royalur.net tries to play sounds:

Although I never got any notification asking to play sound, and I had interacted with the page before trying to play the sound. All in all, this is a weird bug...

authors: Sothatsit forteller

repo: Sothatsit/RoyalUrClient


text:
🛑 Marcel Web is down

In cf5ca7a, Marcel Web ($MARCEL_WEB) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Marcel Web is back up in 269348d.

repo: Source-Graphics-GmbH/upptime


text:
🛑 Marcel Web is down

In c84dada, Marcel Web ($MARCEL_WEB) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Marcel Web is back up in c11117f after 1 hour, 31 minutes.


text:
🛑 Marcel Web is down

In 67d9c16, Marcel Web ($MARCEL_WEB) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Marcel Web is back up in c43762d after 1 day, 3 hours, 22 minutes.


text:
Create status page comparison.md

Signed-off-by: Benjamin 2255421+master-caster@users.noreply.github.com

very cool - thanks for your work @master-caster!
I would suggest that we don't place it in Contributor-Docs, since that folder is for documents regarding contributions.
Instead I suggest a new folder 'Decisions' in which we place a folder 'Status-Page'. There we can place your comparions (please no spaces in the filename).  Based on your document, we can then proceed with the decision making and document that in an ADR that references the comparison.

authors: fkr master-caster

repo: SovereignCloudStack/Docs


text:
Read and create references during import

References should be read, converted into CSL and created as Reference entities regardless where they originally are coming from.
ACEF has distinct references that should be read and created. Links to references should be extracted from:

Common Names.ReferenceID linking a single vernacular name to a reference
NameReferecesLinks table should then be read to link the reference to a taxon via the ID property which refers to either an AcceptedTaxonID or a Synonym.ID. Links of type ComRefType should be ignored. type=TaxAccRef is a regular taxon reference, type=NomRef might be the namePublishedIn and should end up as the Description act. Needs to be confirmed. It means distributions have no reference

DWCA
TODO

Correct, and this basically closes #62

fixed handling of taxon bibliographies: https://github.com/Sp2000/colplus-backend/commit/f20fe94cfb8831ceca0aed4576ed34d2038f4ae4

authors: aycoholleman mdoering


text:
Pull RF_LATCHUP_DIS pin low on all HW tests

Fixes #298

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

Project coverage is 88.34%. Comparing base (ab9c466) to head (2e4b1ab).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #300   +/-   ##
=======================================
  Coverage   88.34%   88.34%
=======================================
  Files          27       27
  Lines         695      695
  Branches        8        8
=======================================
  Hits          614      614
  Misses         81       81

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

authors: codecov-commenter elenajochum


text:
chore(ffe-form-react): fix typo, Checkbox should have lower case b

Changes does not affect anything for the consumer. The typo was just in some documentation and internally in the js-file. The export is a default export, and the exported name from index.js is correct with lower case b.

Coverage remained the same at 81.719% when pulling 3195cfa5fda0f9c0eb0527641e3472b4b25e90b1 on fix-typo into 49c2e484fd8a495fcfa0ee00b8d176e828184d3d on develop.

authors: coveralls henrikhermansen


text:
Better support for OIDC4VP

I'm submitting a ...
[ ] bug report
[x] feature request
[ ] question about the decisions made in the repository
[ ] question about how to use this project

Summary

Hi!
In the OIDC4VP specs, the presentation_submission object is either in the id_token or in the vp_token itself. If I want to evaluate a VP using @sphereon/pex, I have to preprocess the verifiablePresentation param of the evaluatePresentation function. IMHO, it would be better to provide 2 separate params: 1 for presentation_submission, the other for verifiablePresentation. But this not my main request.
Right now, the evaluatePresentation requires the second param (verifiablePresentation) to be an IPresentation. How am I supposed to pass a JWT VP then? In my case, I would have a presentation_submission like:
        "presentation_submission": {
            "id": "Selective disclosure example presentation",
            "definition_id": "Selective disclosure example",
            "descriptor_map": [
                {
                    "id": "Ontario Health Insurance Plan",
                    "format": "jwt_vp",
                    "path": "$.presentation",
                    "path_nested": {
                        "format": "jwt_vc",
                        "path": "$.presentation.vp.verifiableCredential[0]"
                    }
                }
            ]
        }

And pass a verifiablePresentation object like this:
{
    "presentation":
        "eyJhbGciOiJSUzI1NiIsInR5cCI6IkpXVCIsImtpZCI6ImRpZDpleGFtcGxlOmFiZmUxM2Y3MTIxMjA0
        MzFjMjc2ZTEyZWNhYiNrZXlzLTEifQ.eyJzdWIiOiJkaWQ6ZXhhbXBsZTplYmZlYjFmNzEyZWJjNmYxY
        zI3NmUxMmVjMjEiLCJqdGkiOiJodHRwOi8vZXhhbXBsZS5lZHUvY3JlZGVudGlhbHMvMzczMiIsImlzc
        yI6Imh0dHBzOi8vZXhhbXBsZS5jb20va2V5cy9mb28uandrIiwibmJmIjoxNTQxNDkzNzI0LCJpYXQiO
        jE1NDE0OTM3MjQsImV4cCI6MTU3MzAyOTcyMywibm9uY2UiOiI2NjAhNjM0NUZTZXIiLCJ2YyI6eyJAY
        29udGV4dCI6WyJodHRwczovL3d3dy53My5vcmcvMjAxOC9jcmVkZW50aWFscy92MSIsImh0dHBzOi8vd
        3d3LnczLm9yZy8yMDE4L2NyZWRlbnRpYWxzL2V4YW1wbGVzL3YxIl0sInR5cGUiOlsiVmVyaWZpYWJsZ
        UNyZWRlbnRpYWwiLCJVbml2ZXJzaXR5RGVncmVlQ3JlZGVudGlhbCJdLCJjcmVkZW50aWFsU3ViamVjd
        CI6eyJkZWdyZWUiOnsidHlwZSI6IkJhY2hlbG9yRGVncmVlIiwibmFtZSI6IjxzcGFuIGxhbmc9J2ZyL
        UNBJz5CYWNjYWxhdXLDqWF0IGVuIG11c2lxdWVzIG51bcOpcmlxdWVzPC9zcGFuPiJ9fX19.KLJo5GAy
        BND3LDTn9H7FQokEsUEi8jKwXhGvoN3JtRa51xrNDgXDb0cq1UTYB-rK4Ft9YVmR1NI_ZOF8oGc_7wAp
        8PHbF2HaWodQIoOBxxT-4WNqAxft7ET6lkH-4S6Ux3rSGAmczMohEEf8eCeN-jC8WekdPl6zKZQj0YPB
        1rx6X0-xlFBs7cl6Wt8rfBP_tZ9YgVWrQmUWypSioc0MUyiphmyEbLZagTyPlUyflGlEdqrZAv6eSe6R
        txJy6M1-lD7a5HTzanYTWBPAUHDZGyGKXdJw-W_x0IWChBzI8t3kpG253fg6V3tPgHeKXE94fz_QpYfg
        --7kLsyBAfQGbg"
}

In other terms, the evaluatePresentation should not make any assumption about the shape of the verfiablePresentation. It should only process it based on the information given by the new presentationSubmission param.
Or maybe I've missed some features of this lib. If it can already process JWT VP properly, please show me how. :)
Thanks!

Hi Yannick and thanks for contacting us.
We will look into it and implement the support for it as soon as possible. In the mean time you can decode the JWT which should be acceptable as an ICredential.

Hi @sksadjad, thanks for your feedback.
I also realise that the the lib doesn't support VC JWT either. So I have to fully decode both the JWT VP and each of the JWT VCs. I was hoping that @sphereon/pex would do it out of the box.

Hi Yannick,
That is correct. Unfortunately we are lacking a bit on the JWT side of things. It is something we will be fixing soon and should be part of the next minor release (1.1.x).
I also agree with the separation of the VP and the submission BTW

Thanks @nklomp, I'll keep an eye on the future releases :)

@yhuard V2 is just released, which was long overdue. It has better support for OID4VP.
You can now have separate presentation submissions, but also supply Formats and DID methods to filter against, outside of what is in the definition. Handy for when a RP lists these in it's registration/discovery metadata.

Excellent, thank you @nklomp 👍 I will check it on Tuesday 😃

authors: nklomp sksadjad yhuard

repo: Sphereon-Opensource/PEX


text:
Logging and Testing in Modsecurity-nginx + Libmodsecurity

Hi,
I have compiled libmodsecurity with ModSecurity-Nginx connector over Nginx-1.10.3 correctly without any issues. Now im looking at usage example at this. How do i add logging and do actual tests if the modsecurity is working or not in this setup ? Because im not seeing any difference in requests so far, so any info on how to add logs for this, May be an usage example would be great.

Hi @vishaalgc ,
try look here.
As far as I understood the question you need to use SecAudit* directives.

Hi @vishaalgc,
There are different ways to test if ModSecurity is really working, the most "informative" one is by enabling the DebugLogs - https://github.com/SpiderLabs/ModSecurity/wiki/Reference-Manual#SecDebugLog

authors: Airis777 vishaalgc zimmerle

repo: SpiderLabs/ModSecurity


text:
reload memory usage

Hello,
We are running ModSecurity@1edd3570e11e9bb2b6d86b249232b24917a4b0ac and ModSecurity-nginx@abbf2c47f6f3205484a1a9db618e067dce213b89 with nginx 1.13.1.
I'm attaching the rule set + modsecurity configuration as well.
When reloading nginx, memory usage will sometimes grow exponentially and easily overwhelm the resources on the server (~100GB of ram). This does not happen 100% of the time, but, is trivial to reproduce just by doing while true;do nginx -s reload;done, nginx will cause the system to OOM reliably after a few minutes.
rules.zip

Testing with cc1d220b408fe73a4e1950b71848772d505d6ce0. Not able to reproduce. Do you mind to shared your configuration summary?

Hi @zimmerle,
Could I share offline with you the full configuration, or just give you access to an environment where I can demonstrate it?
-Adam

Hi @AdamJacobMuller,
Sure it won't be a problem. But please, before do that, it will be very good if you can upgrade to the latest version and check if everything is working as expected [or not]. If not, that will be good if you can narrow down the problem to the exactly configuration that is leading to the problem.

Hello @zimmerle.
While running continuous reloads with libmodsec loaded as a dynamic module, stock modsecurity.conf and CRS3.0.2 loaded, I can see growing RES footprint. It doesn't happen with the same Nginx binary without ngx_http_modsecurity_module.so loaded.
ps aux | grep nginx | grep -v grep | awk '{print $6}'
15344
16220
16220
for i in {1..100}; do nginx -s reload; done
ps aux | grep nginx | grep -v grep | awk '{print $6}'
33092
31156
31156
ModSecurity@04f7009980d6715749de697710a640fd54a32789, ModSecurity-nginx@a2a5858d249222938c2f5e48087a922c63d7f9d8
Configure options for libmodsecurity:
./configure --build=x86_64-redhat-linux-gnu --host=x86_64-redhat-linux-gnu --program-prefix= --disable-dependency-tracking --prefix=/usr --exec-prefix=/usr --bindir=/usr/bin --sbindir=/usr/sbin --sysconfdir=/etc --datadir=/usr/share --includedir=/usr/include --libdir=/
usr/lib64 --libexecdir=/usr/libexec --localstatedir=/var --sharedstatedir=/var/lib --mandir=/usr/share/man --infodir=/usr/share/info --with-pcre=/usr/bin/pcre-config --with-curl=/usr/bin/curl-config
Default CentOS 7 CXXFLAGS: '-O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector-strong --param=ssp-buffer-size=4 -grecord-gcc-switches   -m64 -mtune=generic'
ModSecurity - v2.9.0-912-g04f7009 for Linux
Mandatory dependencies

libInjection                                  ....v2.9.0-912-g04f7009
SecLang tests                                 ....04f7009

Optional dependencies

GeoIP                                         ....found v1.5.0
-lGeoIP  , -I/usr/include/
LibCURL                                       ....found v7.29.0
-lcurl  ,  -DWITH_CURL
YAJL                                          ....found v2.0.4
-lyajl  , -DWITH_YAJL
LMDB                                          ....not found
LibXML2                                       ....found v2.9.1
-lxml2 -lz -lm -ldl, -I/usr/include/libxml2 -DWITH_LIBXML2

Other Options

Test Utilities                                ....enabled
SecDebugLog                                   ....enabled
afl fuzzer                                    ....disabled
library examples                              ....enabled
Building parser                               ....disabled

Nginx version is 1.11.10, configure options are pretty much the same as in Fedora package (https://src.fedoraproject.org/rpms/nginx/blob/master/f/nginx.spec), give or take an optional module.
Let me know if you need more information.

Thanks for the info @alexdavydov.
I was able to reproduce it here. I ran some tests with Valgrind and is also flagging some memory leaks when ngx_http_modsecurity is loaded:
==34396== 41,879 (88 direct, 41,791 indirect) bytes in 1 blocks are definitely lost in loss record 704 of 711
==34396== 3,184,513 (6,952 direct, 3,177,561 indirect) bytes in 79 blocks are definitely lost in loss record 711 of 711
==34396== LEAK SUMMARY:
==34396==    definitely lost: 7,040 bytes in 80 blocks
==34396==    indirectly lost: 3,219,352 bytes in 863 blocks
==34396==      possibly lost: 115,576 bytes in 15 blocks
==34396==    still reachable: 199,722 bytes in 3,462 blocks
==34396==         suppressed: 0 bytes in 0 blocks
As of now I'm inclined to believe that the issue might be on the connector rather than libModSecurity. Will investigate a bit more.

Please check 1ad95254cd8caec4a0af83d01844fc3cc65f3ce7. There was a fixed at #1563 which is likely to fix this behavior as well.

Please check 1ad9525. There was a fixed at #1563 which is likely to fix this behavior as well.

This problem persists. We have the same kind of memory leak during reloads. We have an environment for you to replicate the same issue.

Hi @ozermetin,
Thanks for the report. Re-opening this issue.

@ozermetin @alexdavydov @AdamJacobMuller there is the PR that addresses some real leaks in nginx connector: https://github.com/SpiderLabs/ModSecurity-nginx/pull/80
Could you please apply the proposed patch and check whether it helps in your cases?

Considering the fact that this issue appears only during reload and having in mind  that the initial reporter @AdamJacobMuller is having 3k+ rules this might be a duplicate  of an issue I also reported in https://github.com/SpiderLabs/ModSecurity/issues/1663 and tracked down. Please check if it is ok after applying https://github.com/SpiderLabs/ModSecurity/pull/1667/files

@AdamJacobMuller
Do you mind letting us know if you notice improvements taking
https://github.com/SpiderLabs/ModSecurity/issues/1663#issuecomment-379856117 into consideration?
Thanks.

I'm afraid I'm facing this problem.
With only 1 vhost on a testing server with no load, for each reload, nginx's memory grows by 12392KB. On other server with 8 vhost each reload add more than 100MB. After weeks, nginx use huge amounts of ram.
To avoid OOM errors I have to restart nginx after a few reloads.
Server:
ModSecurity - v3.0.4 (7e0bc2691727b8c75f74638cdc4d1c45a689a7b6) with ModSecurity-nginx v1.0.1 (https://github.com/SpiderLabs/ModSecurity-nginx/commit/63feb81785140d5a5dca564befb5c87ff22282de) and Modsecurity-crs 3.3/dev (https://github.com/SpiderLabs/owasp-modsecurity-crs/commit/6ed306e1b9aa8784d5e9c3e19da54cc7e20eb113)
nginx-1.17.4
Ubuntu 18.04.1

Hi @averges, I am reopinig the issue to do a further investigation.

Hi all, I'm hitting the same issue. On every nginx reload the memory usage is increasing. Do you have any update on this matter?

@cocooma we are working on the branch v3/dev/phases. The configuration issues will be tackled there. I will let you know as soon as we manage to have it ok to be tested.

no longer an issue in v3.1-experimental.

no longer an issue in v3.1-experimental.

authors: AdamJacobMuller alexdavydov averges cocooma defanator ozermetin vaLski victorhora zimmerle