text:
filter property access issue?

how would I filter on the pet names below?
people = [
  {
  name: 'Bob',
  favoriteColor: 'Blue',
  pets:
    [{name: fido}, {name: spike}]
  },
  {
  name: 'Steve',
  favoriteColor: 'Red',
  pets:
    [{name: fluffy}]
  }
]

@RettBehrens This will solve the problem! I have submitted a PR but it's pending to the get merged!


text:
Adding total cases count to sidepanel

Can you make this a little more aesthetic? Something like this with fewer frills, perhaps.

@awadheshv are you still working on this?

No, been busy with other stuff lately.

source: gharchive/pull-request

authors: awadheshv sibeshkar someshkar

license: MIT

license_type: permissive

license_source: github-api


text:
ollama（llama2模型）渠道测试请求一直返回404（gpt-3.5-turbo）

例行检查

我已确认目前没有类似 issue
我已确认我已升级到最新版本
我已完整查看过项目 README，尤其是常见问题部分
我理解并愿意跟进此 issue，协助测试和提供反馈
我理解并认可上述内容，并理解项目维护者精力有限，不遵循规则的 issue 可能会被无视或直接关闭

问题描述
渠道配置的llama2 模型，但是测试请求一直请求gpt-3.5-turbo报404

复现步骤
预期结果
相关截图
如果没有的话，请删除此节。

本地请求是正常的
curl http://localhost:11434/v1/chat/completions
-H "Content-Type: application/json"
-d '{
"model": "llama2",
"messages": [
{
"role": "system",
"content": "You are a helpful assistant."
},
{
"role": "user",
"content": "Hello!"
}
]
}'

本地请求是正常的 curl http://localhost:11434/v1/chat/completions -H "Content-Type: application/json" -d '{ "model": "llama2", "messages": [ { "role": "system", "content": "You are a helpful assistant." }, { "role": "user", "content": "Hello!" } ] }'

同遇到

是最新版本么

我这边配了模型重定向 {"gpt-3.5-turbo": "qwen:4b"} 后，只有点页面“测试”的时候报"status code 404: model 'gpt-3.5-turbo' not found, try pulling it first"，但在NextChat中可以正常使用

我这边只有点页面“测试”的时候报"status code 404: model 'gpt-3.5-turbo' not found, try pulling it first"，但在NextChat中可以正常使用
另外请教下，Base URL我只有配成 http://host.docker.internal:11434 才可以，docker怎么设置才能配成 http://localhost:11434 呢？不调整ollama可以吗？以下是我的docker命令:
docker run --name one-api-mysql -d --restart always -p 3000:3000 --add-host="host.docker.internal:host-gateway" -e SQL_DSN="root:123456@tcp(host.docker.internal:3306)/oneapi" -e TZ=Asia/Shanghai -v /Users/user/data/one-api-mysql:/data justsong/one-api

大佬，ollama没有密钥。这个密钥怎么填写？

我这边只有点页面“测试”的时候报"status code 404: model 'gpt-3.5-turbo' not found, try pulling it first"，但在NextChat中可以正常使用
另外请教下，Base URL我只有配成 http://host.docker.internal:11434 才可以，docker怎么设置才能配成 http://localhost:11434 呢？不调整ollama可以吗？以下是我的docker命令:
docker run --name one-api-mysql -d --restart always -p 3000:3000 --add-host="host.docker.internal:host-gateway" -e SQL_DSN="root:123456@tcp(host.docker.internal:3306)/oneapi" -e TZ=Asia/Shanghai -v /Users/user/data/one-api-mysql:/data justsong/one-api

大佬，ollama没有密钥。这个密钥怎么填写？

不是大佬，随便填个空格或者"ollama"都可以的

好，非常感谢，我也和你一样了，curl能弄通oneapi配不了，我还是昨晚更新了最新版的，我今天在折腾下试试
y2k
@.***

------------------ 原始邮件 ------------------
发件人: @.>;
发送时间: 2024年3月16日(星期六) 上午8:29
收件人: @.>;
抄送: @.>; @.>;
主题: Re: [songquanpeng/one-api] ollama（llama2模型）渠道测试请求一直返回404（gpt-3.5-turbo） (Issue #1076)
我这边只有点页面“测试”的时候报"status code 404: model 'gpt-3.5-turbo' not found, try pulling it first"，但在NextChat中可以正常使用
另外请教下，Base URL我只有配成 http://host.docker.internal:11434 才可以，docker怎么设置才能配成 http://localhost:11434 呢？不调整ollama可以吗？以下是我的docker命令:
docker run --name one-api-mysql -d --restart always -p 3000:3000 --add-host="host.docker.internal:host-gateway" -e @.***(host.docker.internal:3306)/oneapi" -e TZ=Asia/Shanghai -v /Users/user/data/one-api-mysql:/data justsong/one-api
大佬，ollama没有密钥。这个密钥怎么填写？
不是大佬，随便填个空格或者"ollama"都可以的
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

source: gharchive/issue

authors: KrabsWong colinWaq katfionn qw457812 songquanpeng

repo: songquanpeng/one-api


text:
plans to update to sdk 2.4.1

Any plans to update to sdk 2.4.1?  Specifically would like to see Android update from 2.3.1 since it triggers a warning from Google Play that it is statically linking to a vulnerable openssl.  (The fix is in 2.4.0)

iOS has been updated to use 2.4.1 but not Android. We should update Android too. I believe that 2.5 is due to come out soon, it might make sense to just update both to use 2.5 when that comes out.

awesome, looking forward to the update!

Yep, 2.5.0 is out now. https://tokbox.com/opentok/libraries/client/

:) any estimate on when the plugin will be updated?

Just wondering, but is anyone working on updating this plugin to 2.5.0?
If not, what is a rough estimate of the amount of effort to do the update?
(note: I've tried the simple replace/rename approach but all sorts of errors popped up)

Currently, I have an app with video chat enabled (with users liking it a lot) with this plugin but Google Play is about to pull it from the app store by July due to the openssl vulnerability issue in 2.3.1.  Thus, I'm hoping to find a solution before that happens.

I just merged in a change that updated the Android SDK to 2.5.0.
I was hoping to update iOS to 2.5.0 as well but haven't had a chance. Anyone else want to volunteer to work on it?

Thank you for the Android SDK 2.5.0 update!
Everything works great and my app is now in Google Play without any violations.

authors: aullman tysjiang

repo: songz/cordova-plugin-opentok

license: mit

license_source: bigquery


text:
Add a CI Workflow

Adds a Gihub Actions workflow definition file that does the following (once for each currently-supported Node.js version):

Checks out the repository
Sets up Node
Runs npm ci
Runs npm run lint (to lint the code)
Runs npm run build (type-checks and then builds the Vue app)
Runs npm run test:unit (unit tests)
Runs npm run test:e2e (Cypress E2E tests)

 Set up build pipeline

repo: sonikblue/milford-menu


text:
Patched GraphQL Cursor Connections Specification support - #14

@sonofmagic sorry, there was a bug in my implementation :-(
Both code and tests updated to catch this problem.
Do you think it would be possible to provide changelog for this library, somehow generated as part of release?

It's okay, there will inevitably be small mistakes in programming.
As for the changelog, I first manually write a copy in the directory, and then I learn how to automatically generate it and make it automatic.

Perfect, thanks a lot

authors: sonofmagic velias

repo: sonofmagic/graphql-depth-limit-ts


text:
100.64.0.0/10 not recognized as private ip space

When using sonyflake on a host with an IP address in the 100.64.0.0/10 space, a nil is returned.
The isPrivateIPv4 function does not recognize IP in 100.64.0.0/10 as private IP addresses.
Found this bug when using sonyflake in an app running on Kubernetes with Calico as an overlay network. Calico uses IP addresses from the 100.64.0.0/10 space for its pods.

isPrivateIPv4 does not handle 100.64.0.0/10 because it is not private address space but shared address space.
You can give custom MachineID to handle 100.64.0.0/10.
https://github.com/sony/sonyflake#usage
Sonyflake's machine ID is 16-bit wide. So please care about duplicate machine ID.

authors: YoshiyukiMineo hugoboos

repo: sony/sonyflake


text:
Tests for entities

Add tests for all entities.

https://github.com/sopra-fs24-group-12/PeekSeek-server/commit/4116016ee61fddadcf336df7c1c904cf0818d18e
Around half of entities are done in this commit, rest is still to do.


text:
Error windows 11 closes ps window

When trying to find an image, a ps window opens but closes instantly
https://user-images.githubusercontent.com/5487950/134695231-9b1eacca-a56d-421b-8e79-68312bc48fe3.mp4

I haven't figured out a way to hide the powershell console when triggerring the powershell script.
However, a window should prompt as soon as the console close.
I haven't tested it on windows 11, I only tried this on a fresh windows 10.

No window or error appears, I don't know if I can help you in any way

Can't reproduce this on Windows 11 as of now.

authors: kaiserdj soruly


text:
Search sugestions

How about an option for adding search suggestion URLs to new search engines?

Available in v3.5.0


text:
Add module params to Jsslang Context creation

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change

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

How to Test
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist
Please delete options that are not relevant.

[ ] I have tested this code
[ ] I have updated the documentation

Pull Request Test Coverage Report for Build 5802

2 of 2 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 27.429%

Totals

Change from base Build 5800:
0.02%

Covered Lines:
2141

Relevant Lines:
7211

💛  - Coveralls

authors: coveralls sciffany

repo: source-academy/cadet-frontend


text:
Catch all uncaught errors thrown from Sagas

Description
Catch all uncaught errors thrown from Sagas.
This prevents the Sagas from terminating.
In development mode, also show the React runtime error screen.
Type of change

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

How to test
Throw an error in a Saga.
Checklist

[x] I have tested this code
[ ] I have updated the documentation

Pull Request Test Coverage Report for Build 6960

7 of 13 (53.85%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 34.204%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/commons/sagas/SafeEffects.ts
6
12
50.0%

Totals

Change from base Build 6955:
0.01%

Covered Lines:
2343

Relevant Lines:
6362

💛  - Coveralls

authors: angelsl coveralls


text:
Change AuthController to fix #53

Fixes #53

Pull Request Test Coverage Report for Build 971

6 of 6 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 950:
0.0%

Covered Lines:
487

Relevant Lines:
487

💛  - Coveralls

All fixed and rebased on master @tuesmiddt

Fixed! @tuesmiddt

authors: coveralls indocomsoft

repo: source-academy/cadet


text:
add: nord theme

Added Nord theme based on Nord palette.

Some "issues" (places where the theme isn't used, probably not the wanted effect). If you need I can open an issue on the app repo.

I've pushed a commit to SourceGit repo: eaf5eba0e7 - ux: using Brush.Popup for menu flyouts.

repo: sourcegit-scm/sourcegit-theme


text:
是否可以支持 某一个范围的 commit 的时候生成patch

现在只能将某一个commit 生成 patch

现在增加了两个另存为补丁的地方：

多个Commit的右键菜单。由于多个commit可能对同一文件进行修改，该方式会为每个commit生成patch文件，并在用户选择的保存目录下创建patches子目录，存放生成的patch文件

可直接使用Revision Compare的结果，另存为单一的一个补丁文件

thanks and FYI
$ git format-patch HEAD^       #生成最近的1次commit的patch
$ git format-patch HEAD^^      #生成最近的2次commit的patch
$ git format-patch HEAD^^^     #生成最近的3次commit的patch
$ git format-patch HEAD^^^^    #生成最近的4次commit的patch
$ git format-patch <r1>..<r2>  #生成两个commit间的修改的patch（生成的patch不包含r1. <r1>和<r2>都是具体的commit号)
$ git format-patch -1 <r1>     #生成单个commit的patch
$ git format-patch <r1>        #生成某commit以来的修改patch（不包含该commit）
$ git format-patch --root <r1> #生成从根到r1提交的所有patch

右键使用的就是git format-path，但是由于用户可以使用Ctrl键选择不连续的提交，所以只能循环对每个选中的提交执行format-path。
Revision Compare由于是diff的结果，所以就直接程序拼接了一个最终的patch文件

理解

如果用户选择不同的分支，或者选择的是不在同一个分支的。那么可以逐个导出
如果用户选择的是同一个分支

如果用户使用 使用 shift 键盘选择，那么就是连续的提交 or 如果用户ctrl 选择多个>=2，刚好也是连续的。

提供逐个导出，
或者连续导出。

如果用户使用 ctrl 选择 = 2 个，且不连续

提供逐个导出选项。
提供以这两个为起点和终点的连续导出选项。

我不明白连续导出指的是什么。git format-patch 如果涉及多个提交，也是一个个生成patch的，唯一的区别是序号有了意义。
选中的提交列表中的元素顺序取决于你先选择了哪个（这是ListBox控件的行为）。这带来一些问题：

如果要根据parent判断是否连续，必需对该列表排序。且排序不会按时间先后，因为有些人本机的时间不正确，导致提交时间是不对的。目前想到的算法是有消耗的，会导致在配置差的机器上，右键菜单不能及时弹出。
同时git format-patch R1..R2对于R1与R2的顺序也有要求 - 如果R1与R2无关，或是R2是R1的祖先，是不会生成patch的。

authors: heartacker love-linger

repo: sourcegit-scm/sourcegit


text:
PRD: [VSC] Enterprise users can run a natural language search across all projects

Description:

For December GA, we released Natural Language Search in Beta for VSC, enabling users to perform a "fuzzy search" across their open, local project. For February, we need to extend this functionality for enterprise users who have purchased both Cody and Search. Those enterprise users should be able to search across all repositories that have been indexed by their admin on their Sourcegraph instance.
Objectives:

Enable enterprise users who have purchased both Cody and Search to perform a natural language search across all repositories that have been indexed by their admin on their Sourcegraph instance.
Acceptance Criteria:

Enterprise users who have purchased both Search and Cody can search across all repos connected to their Sourcegraph instance
Users can navigate to each of the search results

Design Tasks:
### Design Tasks

Engineering Tasks:
### Engineering Tasks

Related Docs:

PRD: Cody Chat and Commands

Closing this for now, because we have some product work to do around Cody + Search integration for both Enterprise and PLG SKUs. This issue is still included in PRD: Cody Chat and Commands, and we can reopen it if appropriate.

repo: sourcegraph/cody


text:
bug: com.intellij.diagnostic.PluginException: ActionUpdateThread.OLD_EDT is deprecated and going to be removed soon. 'com.sourcegraph.cody.statusbar.CodyDisableLanguageForAutocompleteAction' must override getActionUpdateThread and chose EDT or BGT. See ActionUpdateThread javadoc. [Plugin: com.sourcegraph.jetbrains]

Plugin version: 5.2.21709
IDE version: WS-241.10840.2
Exception: com.intellij.diagnostic.PluginException: `ActionUpdateThread.OLD_EDT` is deprecated and going to be removed soon. 'com.sourcegraph.cody.statusbar.CodyDisableLanguageForAutocompleteAction' must override `getActionUpdateThread` and chose EDT or BGT. See ActionUpdateThread javadoc. [Plugin: com.sourcegraph.jetbrains]
Stacktrace:
com.intellij.diagnostic.PluginException: `ActionUpdateThread.OLD_EDT` is deprecated and going to be removed soon. 'com.sourcegraph.cody.statusbar.CodyDisableLanguageForAutocompleteAction' must override `getActionUpdateThread` and chose EDT or BGT. See ActionUpdateThread javadoc. [Plugin: com.sourcegraph.jetbrains]
	at com.intellij.diagnostic.PluginProblemReporterImpl.createPluginExceptionByClass(PluginProblemReporterImpl.java:23)
	at com.intellij.diagnostic.PluginException.createByClass(PluginException.java:90)
	at com.intellij.diagnostic.PluginException.reportDeprecatedUsage(PluginException.java:125)
	at com.intellij.openapi.actionSystem.ActionUpdateThreadAware.getActionUpdateThread(ActionUpdateThreadAware.java:21)
	at com.intellij.openapi.actionSystem.AnAction.getActionUpdateThread(AnAction.java:198)

dupe of https://github.com/sourcegraph/jetbrains/issues/539

authors: lluistar taylorsperry

repo: sourcegraph/jetbrains


text:
Error message when selecting action

Using Sourcegraph.com, when I select the below.

Produced the error message.
Request commands/$executeCommand failed with message: Cannot read property 'showInputBox' of undefined.

This occurs because there is no code file open, and showInputBox is a method on a window (which is only non-null when there's a code file). Kind of defeats the point I mentioned in https://github.com/sourcegraph/sourcegraph/issues/13288, haha. I will make this item show up only when there is a code file for now.

authors: ryan-blunden sqs

repo: sourcegraph/sourcegraph-codecov


text:
Cannot focus terminal when text elsewhere on the page is selected

If I select text somewhere else on the page, I cannot focus the terminal by clicking inside it. It works fine after I clear the selection.

Details

Browser and browser version: Chrome Version 59.0.3071.115 (Official Build) (64-bit)
OS version: macOS Sierra 10.12.6
xterm.js version: 2.8.0 (and current master)

Steps to reproduce

npm install
npm start
Open http://0.0.0.0:3000 in a browser
Select some text on the page outside of the terminal
Try to click inside the terminal

Problem: The terminal cannot be focused.

I guess this happens because we call evt.preventDefault() in a mouse handler somewhere...

Opened #828 to get around with this, before 2.9.0.
This was caused by old code trying to get around with selections on Firefox (#41). Since selection is taken care of by xterm.js now, this code is not needed any more.

Thanks for reporting this @spadgett 😄!

authors: mofux parisk spadgett


text:
Verification process never completes

Summary
The Verification gets to a "request_sent" state but doesn't ever progress past that on my box.
Definition (optional)
See Activity 5 (among others)

Looks like it got to "verified" this time.  Maybe this is situation dependent?  Got to verified after requesting a first_name/last_name verification from my own Company to my own Holder.

I will leave this open for now in case I see the issue again in a different scenario, But it is now a much lower priority to look at.

I submitted a pr yesterday ,#40, that I think corrected this. we were calling the wrong API to get the updated list.

authors: burdettadam lynnbendixsen

repo: sovrin-foundation/aries-toolbox


text:
Improve handling touch events

improve close-on-touch event by triggering the blur manually
blur the input element on focus, so no keyboard will be shown on touch-devices when opening the picker

This normalizes the behavior across more mobile browsers, including the new Safari on iOS 10.

In my tests, input.blur() doesn’t trigger this._events.inputBlur.

indeed this is because of isElementInContainer, we don't want to hide the date picker if we click on a date.
Can you try this:

diff --git a/src/dpicker.js b/src/dpicker.js
index 85e4043..59f3442 100644
--- a/src/dpicker.js
+++ b/src/dpicker.js
@@ -179,8 +179,8 @@ function DPicker(element, options = {}) {
   let input = element.querySelector('input')

   input.addEventListener('blur', this._events.inputBlur)
-  document.addEventListener('touchend', function() {
-    input.blur()
+  document.addEventListener('touchend', (e) => {
+    this._events.inputBlur(e)
   })

   return this

If this doesn't work we can add a flag on line 433 if (isElementInContainer(node.parentNode, this._container)) { for example if (forceBlur !== true && isElementInContainer(node.parentNode, this._container) and set e.forceBlur = true before calling it.

Can you try 3.1.2 and let me know if it works? Thanks!

authors: florianpircher soyuka

repo: soyuka/dpicker


text:
Нужен датасет для вульп

Иваны Сидеренки и Ларисы Петровы на вульпах это просто, извините, пиздецище

ахуенно морти, сделай пожалуйста ещё больше коммитов без пров
https://github.com/space-syndicate/space-station-14/commit/62a32143ae202305ac547f1e9e0dd4515d4ce0c6


text:
Character Loadouts

Описание PR

За основу взят PR: https://github.com/Park-Station/Parkstation/pull/28
Так же реализовано отображения предметов на превью кукле.
Медиа

Проверки

[x] PR полностью завершён и мне не нужна помощь чтобы его закончить.
[x] Я внимательно просмотрел все свои изменения и багов в них не нашёл.
[x] Я запускал локальный сервер со своими изменениями и всё протестировал.
[x] Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.

Изменения

:cl: Morty

add: Добавлено меню выбора стартовой экипировки в настройках персонажа.

https://github.com/space-wizards/space-station-14/pull/17517

Молюсь на это - https://github.com/space-wizards/space-station-14/pull/17517
Однажды примут.


text:
Add new random spawners

Описание PR

Добавление 4х новых спавнеров для маппинга:

3 спавнера случайной контрабанды разной ценности (низкая, средняя и высокая ценность)
спавнер случайного животного

Медиа

Проверки

[x] PR полностью завершён и мне не нужна помощь чтобы его закончить.
[x] Я внимательно просмотрел все свои изменения и багов в них не нашёл.
[x] Я запускал локальный сервер со своими изменениями и всё протестировал.
[x] Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.

Изменения

:cl:

add: Добавлено веселье!
remove: Убрано веселье!
tweak: Изменено веселье!
fix: Исправлено веселье!

оффам :trollface:

оффам :trollface:

Морти не против. У оффов нет такого понятия как контрабанда и потребности у них в этом нет, а ради одного спавнера животных - впадлу.

ты ахуеешь
https://wiki.spacestation14.io/wiki/Space_Law#Restricted_Items_Lists

ты ахуеешь https://wiki.spacestation14.io/wiki/Space_Law#Restricted_Items_Lists

Не думаю что оффам это нужно, они не использую эти предметы на картах вообще + у меня сделано конкретно по нашему вики.

значит может стоит прекратить спавнить контрабанду на самых защищённых станциях нт???

значит может стоит прекратить спавнить контрабанду на самых защищённых станциях нт???

Это для этого и сделано, если уж и делать, то только рандомно


text:
stomach metabolises injected reagents

Description
it shouldnt do that since its not going into the stomach duh
it might need adding a new solution container specifically for reagents that go into the stomach
mouth (none) -> stomach (food+drink) -> temp bloodstream???? (?) -> bloodstream (poison+narcotic+medicine)
Reproduction

inject yourself with water
instead of bubbles in your blood being bad you just stop being thirsty

air embolism meta

yes when smoke worked we did that a lot just smoke some soda and it works

needle too long, goes directly through flesh into stomach

authors: deltanedas notquitehadouken


text:
Need a quick button to turn off sensors on clothes

Description
When you play as a sindicat, it's hard to take off all the clothes from a person when you kill them or put them to sleep, I think you need to add this button.
Screenshots

I think it should be added to this menu.

if this is implemented not just a button to turn off, but all options to make it easier for medics to turn peoples sensors on

If IIRC, a patient's sensor blip will remain active on the crew monitor for about 10 s after the sensors are turned off (the patient's information doesn't update though). So it looks like the system is actually designed to give people watching the monitor a chance of spotting an assassination and cover up. Have to double check this though, could be misremembering

authors: TheKilk chromiumboy deltanedas


text:
Overlapping trees prevent direct attacks

Description
Trees from floral anoms love to grow inside walls so you can't do anything about them without tearing it down, if you want to hit something directly behind tree from your camera POV you will be attacking tree in wall which you can't hit for obvious reasons.
Reproduction
Put 2 trees, one of them in wall, attack one not in wall 1 tile north.

Fixed in #24783


text:
Condiment bottle tweaks

About the PR

Condiment bottles behaved unexpectedly (Not able to refill).
Condiment bottles now inherit from DrinkBase to add better functionality.
Media

Before:
https://github.com/space-wizards/space-station-14/assets/62370007/f2344747-7b71-4cbb-acdb-184decef8db6
After:
https://github.com/space-wizards/space-station-14/assets/62370007/d363293c-1a47-4d64-961e-287f8217494d

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:
no cl?

I could have just added refillable component without the other stuff, oh well.
Should the condiment bottles be breakable like other glass bottles?


text:
Glasses/Hud Art Dump

About the PR

Adds in a bunch of artwork for @PrPleGoo to use in their upcoming PR.
Also changes the wearing texture for beer glasses to resemble it better. (It was previously the sunglasses texture)
(Subnote: Moved all "glasses" to be effected with a hud element over to hud instead of glasses)
Media

Bundle of the new art assets:

Old and New Beer Goggles:

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

N/A

do we even want huds that just let you see everything? it seems really overpowered.

It's mainly a sort of end-game tier item. Not meant to be gotten immediately. Likely to be made with science or the likes

i'm not a huge fan conceptually of "it's all the previous things combined", but that decision is up to a maintainer.

I see your point. Though having a doctor know that the patient is starving or a brigmedic able to look at both medical status and security status is definitely a good thing.

I agree the panopticon might be a bit too strong. But it can't hurt to just put it in and leave it as a badmin item.

@Emisse Merge Pls  👉👈

authors: DerbyX EmoGarbage404 PrPleGoo


text:
Ports a fuckton of vox clothing from Paradise and /vg/station

About the PR
Ports basically all vox clothing from /vg/ and Paradise I could find that fit with the current sprites we have. Also adds a few original sprites (the non-normal gas masks and welding masks) since they were easy enough to make.
Does NOT port gloves, backpacks, or most non-full helmet sprites, since those can probably just be displacement mapped.
Some of these are technically not quite fully fitting since they have tails and stuff built in to the sprite. This should not be an issue since the tail layers over the suit anyways.
This PR was not tested and it is likely that at least a few of these may need code-side changes to actually display. The only thing this PR does is add the sprites.
Why / Balance
I just wanted to get this out of the way. There's no point in spriting new sprites if other codebases already have good ones for a lot of the clothes we have already.
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog
no CL no fun

There are textures in there for features not in the game yet, like the welder mask being up

There are textures in there for features not in the game yet, like the welder mask being up, which is in there twice

I am aware. They exist in the RSI for a reason, though - they're presumably meant to be in the game at some point, we just don't support displaying them yet.

I salute you

authors: Cojoke-dot Errant-4 Flareguy


text:
Cheaper Python and Hristov bundle

About the PR
 just TC price changes + removing ammo from the bundle. (5 tc Hristov bundle + 1 tc ammo = 6 tc). Not doing any stat changes because of the freeze
fixes #27968
Why / Balance

Technical details

Media

[ ] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl: Hmeister

tweak Hristov bundle and python are now cheaper

Not doing any stat changes because of the freeze

it's still TC balance changes and it is still frozen

thats fine, il re-open after freeze. just didn't think it would be impactful enough to be part of it

Re-open please

so, merge? Or maybe some suggestions?

Hristov no touch until weapon attachments which I say to every hristov PR.
I think there's no point touching weapon TCs without attachments.

fair


text:
Update water.yml

https://github.com/space-wizards/space-station-14/assets/125140938/e9b2e08e-d05a-4528-a3f7-4dd00ad1585e
Water should put out the fire. That's why now the water tiles are extinguishing you.

About the PR

Why / Balance
bruh i burn in water
Technical details

Media

[ ] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

fixes #21532

This should use #28168 instead

kinda... ummm... not working. Am I stupid?


text:
ApcNet updating fix

Closes #3056
Moves updating of ApcNets to a grid component, since there is no individual component that guarantees the existence of an ApcNet that I can do an EntityQuery for.
~TODO: Ensure ApcNets always stop getting updated after removal.~

Will take some time to fix the test to be compatible with nullable.

If it's painful to fix the test just out of scope it.

Any reason this is a grid component instead of an entity system? This seems like it'd make creating "proper" new grids with power support more difficult

Fixing nullability on the power test is big enough to be its own PR, undoing that for now.

Any reason this is a grid component instead of an entity system? This seems like it'd make creating "proper" new grids with power support more difficult

@Zumorica
I could make the entity system maintain the set of ApcNets to update, but I'm not sure how I'd efficiently check which ApcNets are paused other than checking if each ApcNet's associated grid is paused every frame. If that's not a problem, I could move it to the entity system though.

Hmm, it is true that making this a grid component elegantly solves the paused grid problem. I guess newly created APCs could ensure that there's a GridPower component on the grid they're on...

Any reason this is a grid component instead of an entity system? This seems like it'd make creating "proper" new grids with power support more difficult

@Zumorica
I could make the entity system maintain the set of ApcNets to update, but I'm not sure how I'd efficiently check which ApcNets are paused other than checking if each ApcNet's associated grid is paused every frame. If that's not a problem, I could move it to the entity system though.

You could check if the APC itself is paused during the uniqueApcNet update though checking it at a grid-level is faster.

Maybe APC nets could store which grid ID they belong to? That'd allow you to check if the grid is paused before updating, I guess.

Maybe APC nets could store which grid ID they belong to? That'd allow you to check if the grid is paused before updating, I guess.

I could have an entity system maintain a set of all ApcNets, and do that. I just wasn't sure if it was efficient to check GridPaused so often, but that might be negilgible?

Maybe APC nets could store which grid ID they belong to? That'd allow you to check if the grid is paused before updating, I guess.

I could have an entity system maintain a set of all ApcNets, and do that. I just wasn't sure if it was efficient to check GridPaused so often, but that might be negilgible?

Sounds good imo.

I've updated this to follow the feedback, the grid component has been replaced with each ApcNet adding itself to an EntitySystem.

authors: Zumorica collinlunn metalgearsloth


text:
Cannot exit go-sync

Description
This cmd, used in the system tests, cannot be killed using CTRL-C. If you run it locally, there is no way to kill it:
> make sync
> build/go-sync
Starting sync
2020-10-30T13:58:08.077-0400    INFO    00000.defaultLogger     Initializing NewSyncApp {"DataDir": ...}
2020-10-30T13:58:08.078-0400    INFO    sync_test       ------------ Start sync test -----------
...
2020-10-30T13:58:12.837-0400    INFO    sync_test       could not get layer from neighbors      {"layer_id": 1, "errmsg": "no peers "}
2020-10-30T13:58:12.877-0400    INFO    sync_test       Node is out of sync setting gossip-synced to false and starting sync
2020-10-30T13:58:12.877-0400    INFO    sync_test       syncing layer   {"current_sync_layer": 1, "last_ticked_layer": 102}
2020-10-30T13:58:12.877-0400    INFO    sync_test.peers         now connected   {"n_peers": 0}
2020-10-30T13:58:12.877-0400    INFO    sync_test       could not get layer from neighbors      {"layer_id": 1, "errmsg": "no peers "}
2020-10-30T13:58:12.994-0400    INFO    sync_test       Node is out of sync setting gossip-synced to false and starting sync
2020-10-30T13:58:12.994-0400    INFO    sync_test       syncing layer   {"current_sync_layer": 1, "last_ticked_layer": 102}
2020-10-30T13:58:12.994-0400    INFO    sync_test.peers         now connected   {"n_peers": 0}
2020-10-30T13:58:12.994-0400    INFO    sync_test       could not get layer from neighbors      {"layer_id": 1, "errmsg": "no peers "}
2020-10-30T13:58:13.114-0400    INFO    sync_test       Node is out of sync setting gossip-synced to false and starting sync
2020-10-30T13:58:13.114-0400    INFO    sync_test       syncing layer   {"current_sync_layer": 1, "last_ticked_layer": 102}
2020-10-30T13:58:13.114-0400    INFO    sync_test.peers         now connected   {"n_peers": 0}
2020-10-30T13:58:13.114-0400    INFO    sync_test       could not get layer from neighbors      {"layer_id": 1, "errmsg": "no peers "}
2020-10-30T13:58:13.234-0400    INFO    sync_test       Node is out of sync setting gossip-synced to false and starting sync
2020-10-30T13:58:13.234-0400    INFO    sync_test       syncing layer   {"current_sync_layer": 1, "last_ticked_layer": 102}
2020-10-30T13:58:13.234-0400    INFO    sync_test.peers         now connected   {"n_peers": 0}
2020-10-30T13:58:13.234-0400    INFO    sync_test       could not get layer from neighbors      {"layer_id": 1, "errmsg": "no peers "}
2020-10-30T13:58:13.358-0400    INFO    sync_test       Node is out of sync setting gossip-synced to false and starting sync
2020-10-30T13:58:13.358-0400    INFO    sync_test       syncing layer   {"current_sync_layer": 1, "last_ticked_layer": 102}
...
^C2020-10-30T13:58:15.250-0400  INFO    00000.defaultLogger     Received an interrupt, stopping services...

2020-10-30T13:58:15.278-0400    INFO    sync_test       Node is out of sync setting gossip-synced to false and starting sync
2020-10-30T13:58:15.279-0400    INFO    sync_test       syncing layer   {"current_sync_layer": 1, "last_ticked_layer": 102}
2020-10-30T13:58:15.279-0400    INFO    sync_test.peers         now connected   {"n_peers": 0}
2020-10-30T13:58:15.279-0400    INFO    sync_test       could not get layer from neighbors      {"layer_id": 1, "errmsg": "no peers "}
2020-10-30T13:58:15.395-0400    INFO    sync_test       Node is out of sync setting gossip-synced to false and starting sync
2020-10-30T13:58:15.395-0400    INFO    sync_test       syncing layer   {"current_sync_layer": 1, "last_ticked_layer": 102}
2020-10-30T13:58:15.395-0400    INFO    sync_test.peers         now connected   {"n_peers": 0}
...

Steps To Reproduce

make sync
build/go-sync
Hit CTRL-C

Expected Behavior
The app should exit cleanly
Environment
Please complete the following information:

Node Version: 356468ca6b9216bcba3e259e05c8f0e90c8336ce

I will work on this issue @lrettig

authors: huyntsgs lrettig

repo: spacemeshos/go-spacemesh


text:
6 code improvements

Added option to save patches as .tif, and removed deletion of coordinate reference system in msg geoprocessor.
Open points in patcher:

add band_names to .nc patches?
do we also save the band_wavelengths in the .tif patches? I don't think we currently do.

Added option to save patches as .tif, and removed deletion of coordinate reference system in msg geoprocessor.
Open points in patcher:

add band_names to .nc patches?

The band names should automatically be in the .nc patches. But their will be differences with the .tif patches. There one needs to manually set the band names (in order of the file) as attributes.

do we also save the band_wavelengths in the .tif patches? I don't think we currently do.

We also don’t do this for the tiff patches. One option could be to do the same as the band names. However, something easier to do is just to make a dictionary with the band names as keys and the wavelengths as values because:

it’s a static variable that doesn’t change
you’re only going to use that for ML training.
This is actually what’s typically done in practice and I think the CloudSAT team already did this for their dataset.

authors: annajungbluth jejjohnson


text:
can't open notebooks TESS/begginers

Hi,
I am not able to run/open the notebooks TESS/begginers. For example:
notebooks/notebooks/MAST/TESS/beginner_astroquery_dv/beginner_astroquery_dv.ipynb
An error occurs in github and when tryin to run it locally.
Cheers,
Sergio

Hi. Can you please provide the traceback? How exactly are you trying to run/open it? What OS and Python/Jupyter versions?

OS linux, python 3.7 , ipython 7.6.1
When visiting github:
https://github.com/spacetelescope/notebooks/blob/master/notebooks/MAST/TESS/beginner_astroquery_dv/beginner_astroquery_dv.ipynb
I got the message "something went wrong"
When running locally (jupyter core 4.5, jupyter-notebook 6) I got:
Unreadable Notebook: beginner_astroquery_dv.ipynb NotJSONError("Notebook does not appear to be JSON: '\n\n\n\n\n\n\n<html lang...")

The "something went wrong" message is an error with GitHub not displaying the contents of a notebook in your browser.  But even if GitHub does fix that, it won't allow you to run the notebook in the browser, just see the contents.
The second error does seem like a problem though, potentially with the notebook itself.  Just to make sure, did you check out the Github repostiory with a "git clone" command, or did you attempt to save the file from the browser to your disk?  If you saved the file, depending on how you did it, there might be HTML in there and not be a real .ipynb file.

Hi,
Github shows me the content of the notebook now! no more "something went wrong"
Regarding the second error,it is very likely that was related on how I downloaded the file.  I will try with the clone command later.  But for the moment I can survive with the content showed by Github.
Thanks!

OK great, I will close this Issue for now, but if you have any questions about how to git clone the repo to run the notebook, don't hesitate to contact us!

@pllim @ceb8 This can be closed as resolved (I don't seem to have the necessary permissions to close it myself).

authors: pllim scfleming sergiohoyer

repo: spacetelescope/notebooks


text:
Dummy release

Test PR of a release for PRDOPSSOC-051 using the NRS workaround described in #255 .
This replaces a mistake in the earlier PR.

Comments from PR #254 that should have been here:
Now this is weird...
Every test passes on my Mac, but test_aperture.py::test_idl_to_tel fails here for NIRISS. I noticed the values are very small- 7e-14. We still get the warnings about 'np.float' being deprecated and that we should switch to 'np.float64' to avoid platform differences. Is that what's going on here?
On my Mac:
`========================================================= test session starts ==========================================================
platform darwin -- Python 3.9.12, pytest-7.1.1, pluggy-1.0.0
rootdir: /Users/dlong/pysiaf, configfile: setup.cfg
collected 32 items
test_aperture.py ...... [ 18%]`
And then on witserv2:
`========================================================= test session starts ==========================================================
platform linux -- Python 3.9.13, pytest-7.1.2, pluggy-1.0.0 -- /user/dlong/linux/miniconda3/envs/pysiaf/bin/python
cachedir: .pytest_cache
rootdir: /home/dlong/pysiaf, configfile: setup.cfg
collected 32 items
test_aperture.py::test_idl_to_tel FAILED [ 3%]`
@tonysohn is it safe to ignore these tests failures here, or is something more serious broken?
@tonysohn
Member
tonysohn commented 3 hours ago
@Witchblade101 Was that the exact error message? It's hard to tell whether it's safe to ignore or not just from that...
@Witchblade101
Member
Witchblade101 commented now
@Witchblade101 Was that the exact error message? It's hard to tell whether it's safe to ignore or not just from that...

No. To see the full error look at the failed checks for PR https://github.com/spacetelescope/pysiaf/pull/261 (which is what these comments refer to. It was a long night, and Mario's NRC work here is completely innocent!)

@tonysohn for completeness here's the full run I just did on my Mac:
`========================================================= test session starts ==========================================================
platform darwin -- Python 3.9.12, pytest-7.1.1, pluggy-1.0.0 -- /Users/dlong/miniconda3/envs/pysiaf/bin/python
cachedir: .pytest_cache
rootdir: /Users/dlong/pysiaf, configfile: setup.cfg
collected 32 items
test_aperture.py::test_idl_to_tel PASSED                                                                                [  3%]
test_aperture.py::test_hst_fgs_idl_to_tel PASSED                                                                                [  6%]
test_aperture.py::test_jwst_aperture_transforms PASSED                                                                           [  9%]
test_aperture.py::test_jwst_aperture_vertices PASSED                                                                             [ 12%]
test_aperture.py::test_raw_transformations PASSED                                                                                [ 15%]
test_aperture.py::test_jwst_sky_transformations PASSED                                                                           [ 18%]
test_hst.py::test_hst_aperture_init PASSED                                                                                [ 21%]
test_hst.py::test_hst_siaf PASSED                                                                                [ 25%]
test_hst.py::test_hst_amudotrep PASSED                                                                                [ 28%]
test_iando.py::test_write_jwst_siaf_xml PASSED                                                                                [ 31%]
test_iando.py::test_write_jwst_siaf_xlsx PASSED                                                                                [ 34%]
test_match_v2v3.py::test_match_v2v3 PASSED                                                                                [ 37%]
test_miri.py::test_against_test_data PASSED                                                                                [ 40%]
test_nirspec.py::test_against_test_data PASSED                                                                                [ 43%]
test_nirspec.py::test_nirspec_aperture_transforms PASSED                                                                         [ 46%]
test_nirspec.py::test_nirspec_slit_transformations PASSED                                                                        [ 50%]
test_nirspec.py::test_sci_det_consistency PASSED                                                                                [ 53%]
test_plotting.py::test_aperture_plotting PASSED                                                                                [ 56%]
test_polynomial.py::test_poly PASSED                                                                                [ 59%]
test_polynomial.py::test_RotateCoeffs PASSED                                                                                [ 62%]
test_polynomial.py::test_two_step PASSED                                                                                [ 65%]
test_polynomial.py::test_invert PASSED                                                                                [ 68%]
test_polynomial.py::test_ShiftCoeffs PASSED                                                                                [ 71%]
test_projection.py::test_tangent_plane_projection_roundtrip PASSED                                                               [ 75%]
test_projection.py::test_project_to_tangent_plane PASSED                                                                         [ 78%]
test_projection.py::test_deproject_from_tangent_plane PASSED                                                                     [ 81%]
test_rotations.py::test_attitude PASSED                                                                                [ 84%]
test_rotations.py::test_attitude_matrix PASSED                                                                                [ 87%]
test_rotations.py::test_sky_to_tel PASSED                                                                                [ 90%]
test_rotations.py::test_axial_rotation PASSED                                                                                [ 93%]
test_rotations.py::test_unit_vector_from_cartesian PASSED                                                                        [ 96%]
test_tools.py::test_jwst_fgs_to_fgs_matrix PASSED                                                                                [100%]
========================================================= 32 passed in 31.71s ==========================================================`

Okay, the fact that this error message is machine dependent (as you implied) tells me that this is not an issue. We can look into this further later, but I'd say we close this for now.

Sounds good. I'll open a new issue for test_idl_to_tel later.

repo: spacetelescope/pysiaf


text:
New Crowdin translations

Enjoy!

Should fix https://github.com/sparanoid/chinese-copywriting-guidelines/pull/92 and https://github.com/sparanoid/chinese-copywriting-guidelines/issues/91


text:
#835 이벤트 종료 후 미션 달성 API 호출 관련 코드 삭제

Summary
It closes #835
여러 기능에서, 미션 달성 관련 로직이 들어간 코드를 삭제했습니다.
Images or Screenshots
Further Work

Do something...

다음 학기 이벤트를 위해, 관련 코드를 삭제하는 것보다는 주석 처리를 하거나, 또는 questComplete 훅에서 axios로 백엔드 호출하는 부분만 비활성화 하는 것도 괜찮은 방법일 것 같습니다.

repo: sparcs-kaist/taxi-front


text:
Pipeline fails on cached images

We need to correct the pipeline's handling of cached images; in long renders (>100 images) I often run into a FileNotFoundError like so:
File "/home/ubuntu/generate_augraphy.py", line 23, in run_pipeline
 data = pipeline.augment(image)
File "/home/ubuntu/.venv/lib/python3.10/site-packages/augraphy/base/augmentationpipeline.py", line 152, in augment
 os.remove(outfilename)
FileNotFoundError: [Errno 2] No such file or directory: '/home/ubuntu/augraphy_cache/image_11.png'

Thanks, i think the evaluation to check if the file exist is missing out here, i will check and correct it.

I pushed an edit here: https://github.com/sparkfish/augraphy/pull/230
So the problem may occur only in parallel execution, where multiple processes are deleting a same file. In this case, i remove the file deletion part because the same file will be get replaced anyway. Please try again if you facing any other issue.

Is there any way to validate that this issue is resolved and can now be closed?

This should be resolved since @proofconstruction was able to generate those images previously. @proofconstruction could you confirm on this?

authors: jboarman kwcckw proofconstruction

repo: sparkfish/augraphy


text:
Error in DirtyRollersAugmentation

Hi,
Facing error with :
    post_phase = AugmentationSequence([

        DirtyRollersAugmentation(
            line_width_range=(8, 12),
            probability=1),

        LightingGradientAugmentation(),
        BrightnessAugmentation('post'),
        SubtleNoiseAugmentation(),
        JpegAugmentation()
    ])

Error:
    File "C:\augraphy\augraphy-dev\src\Augraphy\Augmentations.py", line 236, in __call__
        if (not self.debug and rotate):

    AttributeError: 'DirtyRollersAugmentation' object has no attribute 'debug

There wasn't any "self.debug" so the "self.debug" should be removed?

Same as "self.transform":
  File "C:\augraphy\augraphy-dev\src\Augraphy\Augmentations.py", line 240, in __call__
    mask = self.transform(self.create_scanline_mask, image.shape[1], image.shape[0], line_width)

  AttributeError: 'DirtyRollersAugmentation' object has no attribute 'transform'

Or DirtyRollersAugmentation is still under development?

This was a property that has been removed. I've updated the code to correct this and deployed the fix in 2.0.5
Thanks for pointing this out!

authors: kwcckw tbrettallen


text:
Fix for Issue #3 Missing return value in SerLCD::write()

This pull request adds the missing return statement to the write(uint_8) function.
Currently with no return statement, the returned size_t value is undefined.  It's probably zero, but who can say?
This fix adds an explicit return statement to return 1 for the size_t value for one byte written.
The bug was in my original code harvested by Sparkfun.  Mea Culpa.

Thanks for the PR fourstix. And thanks @gvansickle for reporting.

authors: fourstix nseidle


text:
Add namespace attributes when necessary

ext/java/nokogiri/XmlNode.java (relink_namespace): Add a namespace attribute
to the new adopted node, if the current scope namespace URI or prefix don't
match the new node's namespace.

Fixes #1774

@jvshahid said in https://github.com/sparklemotion/nokogiri/issues/1774#issuecomment-405101219:

I would like someone to review the newly added tests to make sure they are idiomatic and make sense.

Going to rebase this on master to get tests to pass.

@jvshahid +1 on the tests, pretty clear to me. This just went green, so: merging.

authors: flavorjones jvshahid

repo: sparklemotion/nokogiri


text:
UnrealGDK/Setup.bat fail to download some dependencies.

Description
Running the UnrealGDK/Setup.bat return the following errors:
Error 1:
...
Successfully retrieved package type 'worker_sdk' with name 'c-dynamic-x86_64-gcc_libstdcpp-linux' and version '13.8.1' to '...\UnrealGDK\SpatialGDK\Build\core_sdk\worker_sdk\c-dynamic-x86_64-gcc_libstdcpp-linux.zip'
Encountered an error during command execution.        ]  43% [0.7MiB/s]s]
failed to download package.
failed to retrieve worker package: rpc error: context deadline exceeded
Error 2:
...
Finished: Retrieve dependencies
Starting: Unpack dependencies
Expand-Archive : The path '...\UnrealGDK\SpatialGDK\Build\core_sdk\worker_sdk\c-static-fullylinked-arm-clang_libcpp-ios.zip' either does not exist or is not a valid file system path.
At line:1 char:1136

... p" -Force;  Expand-Archive -Path "...\UnrealGDK\SpatialGDK\Build\ ...

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

CategoryInfo          : InvalidArgument: (...\UnrealG..._libcpp-ios.zip:String) [Expand-Archive], InvalidOperationException
FullyQualifiedErrorId : ArchiveCmdletPathNotFound,Expand-Archive

OS : Windows 10
Unreal Engine Version : 4.22-SpatialOSUnrealGDK-release (21/08/19)
UnrealGDK : release (21/08/19)

You can resolve the Error 1 by modifying the Setup.bat with --grpc_client_connection_timeout_s flag
Example:
spatial package retrieve worker_sdk c-dynamic-x86_64-gcc_libstdcpp-linux       %PINNED_CORE_SDK_VERSION%   "%CORE_SDK_DIR%\worker_sdk\c-dynamic-x86_64-gcc_libstdcpp-linux.zip" --grpc_client_connection_timeout_s 5m

Error 1 : I try "--grpc_client_connection_timeout_s 5m" and it is ignored. After 60 second i get the same timeout error.
Error 2 : I guess this error is due to the partial downloaded file that cant be unzipped.
Also in the end the batch file keep saying "UnrealGDK build completed successfully!"
I guess the batch file is not detecting the errors here.

They are both a result of the same issue.
Give the 'c-static-fullylinked-arm-clang_libcpp-ios.zip' more time to download instead, not 'c-dynamic-x86_64-gcc_libstdcpp-linux.zip' which is successfully downloaded.
So:
    spatial package retrieve worker_sdk      c-static-fullylinked-arm-clang_libcpp-ios  %PINNED_CORE_SDK_VERSION%   "%CORE_SDK_DIR%\worker_sdk\c-static-fullylinked-arm-clang_libcpp-ios.zip" --grpc_client_connection_timeout_s 5m
@JohnsonJackblue just misread the error but gave the right solution!

repo: spatialos/UnrealGDK


text:
spatie/laravel-translatable fields broken

Adding the string type of a spatie/laravel-translatable property causes a must be of type ?string, array given, called in /var/www/html/vendor/spatie/laravel-data/src/Resolvers/DataFromArrayResolver.php on line 95 error.
🖥 Versions
Laravel: 11.21.0
Laravel Data: 4.8.1
PHP: 8.3.10

Please follow the issue format when creating issues.

repo: spatie/laravel-data


text:
Add support for invokable rules

This pull request adds support for invokable rules, which were added in Laravel 9.18.

Looking good! Thanks!

authors: aidan-casey rubenvanassche


text:
Drop unused colors

Reduces front.css file size from 231 to 155 kb.

👌

authors: freekmurze sebastiandedeyne


text:
compilation warning

First of all, thanks for your awesome work on this so useful component...
Trying to use it for th first time , with your default example I get the following warning:
  (Emitted value instead of an instance of Error) the "scope" attribute for scoped slots have been
  deprecated and replaced by "slot-scope" since 2.5. The new "slot-scope" attribute can also be
  used on plain elements in addition to <template> to denote scoped slots.

I added into my main.js file
 import TableComponent from 'vue-table-component'
 Vue.use(TableComponent)

and I added the example template  into my UsersPageComponent.vue

found the issue !  you should update your README.md and use slot-scope in the
    <table-column label="" :sortable="false" :filterable="false">
         <template slot-scope="row">
             <a :href="`#${row.firstName}`">Edit</a>
         </template>
     </table-column>

repo: spatie/vue-table-component


text:
Missing python packages for test_all.sh

Describe the bug
The packages  opt_einsum, pytest and mpi4py are missing in  requirements.txt (i.e. test_all.sh fails locally with error messages indicating that the dependency couldn't be found.
To Reproduce
Steps to reproduce the behavior:

install pip packages: i.e. python3 -m pip install -r requirements.txt (local environment)
run all tests: i.e. ./test_all.sh

Desktop (please complete the following information):

OS: Ubuntu 20.04.1 LTS

Solution
add opt_einsum, pytest and mpi4py to requirements.txt

Well yes, for that you need to install the [testing] optional requirement, since it's not required by dace but only the tests.
Please use pip install ".[testing]" or dace[testing]
See https://github.com/spcl/dace/blob/master/setup.py#L69 and https://github.com/spcl/dace/blob/master/.travis.yml
for testing environment setup.

authors: andreaskuster tbennun

repo: spcl/dace


text:
Is there a way to copy it when I click on it?

When I click on a link, I would like to copy the URL to the clipboard instead of going to the link, is there a way to do this?

This should work with right-click -> Copy link.


text:
Uses tag but needs version specific stuff

uses minver again

Rejecting since we figured out how to move forward with GitVersion

authors: AlanRynne adamhathcock

repo: specklesystems/speckle-sharp-connectors


text:
DUI3-71 receive bindings and receive geometry

Description & motivation

Changes:
Connectors / Revit

Added receive binding (pretty much copied from Rhino)
Modified service registrations for new types

Added default traversal function

Registered a run on thread method that uses the Revit Async package

Running on main thread alone is not sufficient

Added simple host object "builder" that just opens a transaction, traverses, receives, and then commits the transaction
Copied transaction manager from dui2 connector

Converter / Revit

added import method next to export method on referencePointConverter (and interface)
Changed name of 'RevitRootToHostConverter' which implements 'IRootToSpeckleConverter' to accurately reflect the implemented interface (aka replaced ToHost with ToSpeckle)
Added toHost scaling service to mirror toSpeckle scaling service

Converter / Revit / ToHost

implemented ITypedConverter interface for several types of geometries
added a modelCurve topLevel converter to be able to actually use the typedConverters

Converter / Revit / ToSpeckle

I moved a bunch of files into this 'toSpeckle' folder to mirror the folder structure of Rhino

To-do before merge:

Screenshots:

Validation of changes:
receiving works for model lines of various types

Checklist:

[x] My pull request follows the guidelines in the Contributing guide?
[x] My pull request does not duplicate any other open Pull Requests for the same update/change?
[x] My commits are related to the pull request and do not amend unrelated code or documentation.
[x] My code follows a similar style to existing code.
[ ] I have added appropriate tests.
[ ] I have updated or added relevant documentation.

References

Oh yeah and the Dispose()/lack of using/disabling the warning

authors: BovineOx connorivy

repo: specklesystems/speckle-sharp


text:
Markup parsing exception

Information

OS: Windows
Version: 0.45
Terminal: any

Describe the bug
With version 0.44 the following worked:
new Markup("[white][[[/][white]]][/]")

With version 0.45 the code from above crashes with:

Unhandled exception. System.InvalidOperationException: Could not find color or style 'white]'.
at Spectre.Console.StyleParser.Parse(String text) in //src/Spectre.Console/StyleParser.cs:line 10
at Spectre.Console.MarkupParser.Parse(String text, Style style) in //src/Spectre.Console/Internal/Text/Markup/MarkupParser.cs:line 22
at Spectre.Console.Markup..ctor(String text, Style style) in /_/src/Spectre.Console/Widgets/Markup.cs:line 42

Expected behavior
The code working with 0.44 should also work with 0.45.

Could be a regression from #911. I'll have a look at it.

Hello, I'm having the exact same issue with this code:
private static void WriteLine(string message)
{
	AnsiConsole.MarkupLine($"[red][[[/]{DateTime.Now:yyyy-MM-dd HH:mm:ss.fff}[red]]][/] {message}");
}

@majorregal What (exact) version of Spectre.Console?

@patriksvensson Sorry, using 0.45.0.

I think @nils-a already fixed this so if you could reference the latest preview of Spectre.Console and verify that it is fixed, it would be great.

@patriksvensson if you're thinking about https://github.com/spectreconsole/spectre.console/pull/1027, that one is not merged. :-)
But that gives me the opportunity to add the above problem to the unit tests.

@patriksvensson You were right, I fixed it (well, in the PR). The problem above is precisely my first test case: "[white][[[/][white]]][/]". I rebased the PR.

authors: WolfBublitz majorregal nils-a patriksvensson


text:
Registrar/Resolver Redesign

I've never really been happy with the current type registrar/resolver model in Spectre.Console. Registration/resolution concerns are scattered across objects in the library with no clear line drawn between setup code and code with dependencies on services. This has been a complaint of mine going pretty far back.
The command interceptor logic introduced recently made the mutual exclusivity of the registrar/resolver design a little less painful, but I still often find myself in catch 22 scenarios simply because Spectre.Console steals away control over my own DI container from me.
I'd like to end this once and for all by completely rethinking the way Spectre.Console approaches DI. Instead of implementing its own nested facades, adopt a more modern approach. Something along the lines of this:
var services = new ServiceCollection();

services.AddSpectreConsole(config =>
{
    config.AddSetting("demo");
});

var provider = services.BuildServiceProvider();

var app = provider.GetRequiredService<ICommandApp>();
return app.Run(args);

There's certainly different approaches here. There's a fluent builder pattern that might be more appropriate here as well, although I will say that when I tried it, it didn't give me that separation between registration & resolution like this approach does.
I'll provide a more comprehensive implementation of this below. Instead of diving into the Spectre.Console code base and going crazy with refactoring, I wanted to start with a small demo to mimick some of the top level objects in the library and show the approach without much concern for implementation detail right off the bat.
This also gives us a chance to discuss the idea and see if the maintainers are interested in an overhaul in this area. I understand backward compatibility will be a big concern here. I honestly don't see a way to keep things backward compatible and still be able to overhaul the aspects of this that desperately need it, IMO.
Let me know how you folks feel about this. I'm happy to help drive the work, I just want to make sure it's not a PR bound to be rejected.
Thanks in advance.
NOTE: In the demo, I'm using the nuget package Microsoft.Extensions.DependencyInjection, but Spectre.Console will actually only use the abstractions (i.e. Microsoft.Extensions.DependencyInjection.Abstractions). I think going forward, the main spectre.console package should only be concerned with the IServiceProvider/IServiceCollection abstractions which allow users to adapt these just as they do today with the registrar/resolver facades. In the future we can add packages like Spectre.Console.DependencyInjection, where we might provide additional facilities like the AddSpectreConsole() method.
I'm open to opinion here but I'm just assuming you'd want to separate DI-specific implementations into their own packages because you might want to support other containers like Autofac separately without impacting the main library.
using System;
using System.Collections.Generic;
using Microsoft.Extensions.DependencyInjection;

namespace ConsoleApp1;

public interface IConfigurator
{
    void AddSetting(string value);
}

public class Configurator : IConfigurator
{
    private readonly List<string> _settings = [];

    public void AddSetting(string value)
    {
        _settings.Add(value);
    }
}

public interface ICommandApp
{
    int Run(IEnumerable<string> args);
}

internal class CommandApp : ICommandApp
{
    public readonly CommandExecutor _executor;
    private readonly IConfigurator _config;

    public CommandApp(IServiceProvider provider, IConfigurator config)
    {
        // CommandExecutor should probably be injected via constructor arg...
        _executor = provider.GetRequiredService<CommandExecutor>();
        _config = config;
    }

    public int Run(IEnumerable<string> args)
    {
        return _executor.Execute(_config, args);
    }
}

internal class CommandExecutor
{
    public int Execute(IConfigurator config, IEnumerable<string> args)
    {
        // do meaningful work
        return 0;
    }
}

public static class SpectreConsoleExtensions
{
    public static void AddSpectreConsole(
        this IServiceCollection services,
        Action<IConfigurator> configure)
    {
        services.AddTransient<ICommandApp, CommandApp>();
        services.AddTransient<CommandExecutor>();
        services.AddTransient<IConfigurator>(_ =>
        {
            var config = new Configurator();
            configure(config);

            // This is where the built-in (hidden) commands get added
            config.AddSetting("default stuff");

            return config;
        });
    }
}

public static class Program
{
    public static int Main(string[] args)
    {
        var services = new ServiceCollection();

        services.AddSpectreConsole(config =>
        {
            config.AddSetting("demo");
        });

        var provider = services.BuildServiceProvider();

        var app = provider.GetRequiredService<ICommandApp>();
        return app.Run(args);
    }
}

@rcdailey I'm not against the idea per se, as long as there is no breaking change in the API. We're about to ship 1.0 of Spectre.Console and Spectre.Console.Cli, and we can't really introduce breaking changes to the API at this point.

@patriksvensson In addition to my previous reply, I have another question:
        // Register the arguments with the container.
        _registrar.RegisterInstance(typeof(CommandTreeParserResult), parsedResult);
        _registrar.RegisterInstance(typeof(IRemainingArguments), parsedResult.Remaining);

How important are these registrations in CommandExecutor.Execute()? I'm not really seeing anywhere they might be injected.

@rcdailey this will force all users to use the "Microsoft DI" and restrict everyone else who likes to use some other DI container, right?

@rcdailey this will force all users to use the "Microsoft DI" and restrict everyone else who likes to use some other DI container, right?

No, that is something I'm deliberately trying to avoid, but I'd like to share my plan below for feedback and also to make sure things really will go as I hope.

Main `Spectre.Console Package
Instead of using our own abstractions for DI concerns, such as ITypeRegistrar and ITypeResolver, consume dependencies directly in constructors. For example, in one instance, the CommandApp constructor would take a CommandExecutor (or abstraction of it) and then using the fluent builder pattern, remove construction concerns from the user (i.e. factory).

New Spectre.Console.DependencyInjection Package
This package would introduce extension methods for IServiceCollection (e.g. IServiceCollection.AddSpectreConsole()) that support transparently registering internal dependencies into the MS DI container, without users having to be aware of the ins & outs of how the registrations work and what types are needed.

The benefits I'm aiming for:

Clearer separation of concerns (DI-related code won't be sprinkled across the Spectre.Console code base)
More flexibility for users. Users have more control over what code is executed between the registration & configure steps of Spectre.Console and actual execution (e.g. RunAsync()) of the CLI application. The command interceptors are supposed to fill this gap today but they add a layer of complexity and are still not entirely flexible enough.
Users no longer have to implement their own DI container adapters. The responsibility of DI container support is shifted to specific extension packages. I don't suspect these to really ever change, so I think the maintainers taking on this responsibility is a good balance for achieving higher usability.

I think achieving these goals would be much easier if we were willing to accept breaking changes. It might be possible to avoid it, but it will result in a significant amount of problem solving and effort that I do not personally believe to be worth it. My principle is always to avoid backward breaking changes, so this isn't an opinion I take lightly. However, if you all feel strongly about backward compatibility, I will do my best to find an iterative approach that allows for it.
I would likely take a two step approach:

First PR would likely be an effort to refactor the existing implementation to shift existing registrations into the CommandApp.Configure() method, and greatly simplify the amount of setup work that is happening in the CommandExecutor class.
Second PR would lay the foundation for passing parameters directly into constructors to make them DI-friendly, and also probably introduce the fluent builder system at this time.
Third PR would probably be the MS DI-specific package. The reason I start with MS DI is because many other DI frameworks (e.g. Autofac) already have adapters for MS DI so I think starting here gets us the best bang for our buck, so to speak, in terms of DI compatibility across the board.

I hope my explanation is helpful. If my plan sounds too ambitious or just impossible let me know. Happy to adjust, after all, I need you all to be on board with this for PR approvals and I really don't want to waste effort if I can avoid it.

I noticed IPairDeconstructor is internal but CommandParameter supports a custom deconstructor via the PairDeconstructor property. However, because it's internal, no clients of the library can effectively implement their own deconstructor due to this logic (taken from CommandValueBinder):
        // Create deconstructor.
        var deconstructorType = parameter.PairDeconstructor?.Type ?? typeof(DefaultPairDeconstructor);
        if (!(resolver.Resolve(deconstructorType) is IPairDeconstructor deconstructor))
        {
            if (!(Activator.CreateInstance(deconstructorType) is IPairDeconstructor activatedDeconstructor))
            {
                throw new InvalidOperationException($"Could not create pair deconstructor.");
            }

            deconstructor = activatedDeconstructor;
        }

This leads me to believe that there are no active users for CommandParameter.PairDeconstructor since that type is required to implement IPairDeconstructor but they do not have access to it.
What is the intent here? Should the configurability of a "pair deconstructor" be removed from the public interface, or should the interface simply be made public?

I've read none of the above @rcdailey, but when my plate is more clear (that shouldn't be too long away), I'm happy to work alongside you with this, if it's still of interest.

Thanks Frank. I'll try to renew interest later, but the slow turnaround makes any motivation or progress difficult. I understand everyone is spending their free time here, so I'm not upset and certainly not pointing fingers. But it is a reality, and unfortunately I barely have enough free time as it is working on my own projects.
I absolutely do want to maintain a helpful relationship so if you ever get some downtime let me know. I mentioned connecting on Discord earlier, if that's an option anyone is comfortable with, that would likely make communication much easier.

I would propose considering making this work tied closely with any AOT work (e.g. #1322, #1155, #955). Both are going to have some overlap on  breaking changes and architecture rework, so both being in the same discussion would probably save some headaches in both directions

@rcdailey  There is a lull in maintainer activity/free time at the moment, except for myself, which is what you are seeing from the outside. Nothing untoward, just the reality of situation. I'm currently clearing down a few PR's and preparing the command line system for version 1 of Spectre.Console. But once that's out of the way, say early Dec, I can stick around and support any development you do, so you know it will be merged. That said, if it's too ambitious or breaking, then I'll become slowed down having to square it off with the other maintainers. But other things I can merge, and/or we could look at other issues if you feel like contributing there instead. Or simply acknowledging this is the situation currently, and waiting a bit. I think you're good in terms of 'maintain a helpful relationship', so don't let that cloud the decision how we proceed here.

Thanks again Frank. You've always been a pleasure to chat with on here so definitely will take advantage of your free time in December.
I think this is a challenging effort, especially if it means not breaking backward compatibility. I've mulled over a few different approaches but it involves a LOT of hacks and shims just to avoid breaking things. And the way the resolver is exposed to users in various structures and callbacks makes it even more challenging.
My inclination is to just break everything to get it the way we want, instead of tip toeing around compatibility. I understand that implies everything you mentioned, but maybe Phil's idea of bundling it with Native AOT work might make it more palatable (although I have no idea what that effort is yet, or if it's a good idea to build a bigger kitchen sink).

I also noticed an issue with the current implementation that should be addressed if a new one is made.
Currently there is two main steps, the creation of the app (new CommandApp()) where the TypeRegistrar is passed and the app  run (app.RunAsync).
From a DI perspective I expect that all dependencies registrations are made in the application building phase and not in the run phase, It is not currently the case.
I use myself SimpleInjector container which lock itself after the first dependency resolution.
So if I want to do something like :

Build container
Build app
Resolve some dependencies
Run app
I can't, because the app try to register some dependencies in the run phase and can't because the container is locked.

I want to understand the situation with Spectre.Console AOT trimming, hence my comment on 1508 here.

From a DI perspective I expect that all dependencies registrations are made in the application building phase and not in the run phase, It is not currently the case.

nb. It's the following line that resolves the registrations, obviously deep in the command execution method:
https://github.com/spectreconsole/spectre.console/blob/aa9e5c48c6b212e75ee9bd25b890359267997579/src/Spectre.Console.Cli/Internal/CommandExecutor.cs#L60
If this was moved somewhere else @kvpt, how should it work?
For example.
new CommandApp(DI container)

(opportunity to continue modifying registrations here)

CommandApp.ResolveDependencies()

(container is locked, internally registered types locked also)

CommandApp.Run()

Is this what you had in mind?

I think there is no need for a new method here, it only need to adhere on this two principes :

All the registrations need to be made in the build phase
new CommandApp(DI container)

All the resolutions need to be done in the run phase
CommandApp.Run()

Currently rule 2 seems ok, but rule 1 is not.
https://github.com/spectreconsole/spectre.console/blob/aa9e5c48c6b212e75ee9bd25b890359267997579/src/Spectre.Console.Cli/Internal/CommandExecutor.cs#L13-L28
For exemple the executor register IConfiguration, IAnsiConsole and CommandModel.
Is it not possible to build and register them in build phase instead of doing it at the execution ?

Yes, I see what you mean @kvpt.
I'm keen to look into whether a non-breaking refactor could separate out these different build/run concerns.
I'll update the thread once I've investigated further.

It's nearly December @rcdailey, my diary is emptying a little - how are you feeling about this issue?

I'm going to take a look at this tonight and see if we can remove the TypeRegistrar/TypeResolver and instead take a dependency on the Microsoft.Extensions.DependencyInjection package.

I'm going to take a look at this tonight and see if we can remove the TypeRegistrar/TypeResolver and instead take a dependency on the Microsoft.Extensions.DependencyInjection package.

@patriksvensson It would be worth keeping in mind @kvpt's comment above, when looking into this, see: https://github.com/spectreconsole/spectre.console/issues/1657#issuecomment-2486250826

I'm going to take a look at this tonight and see if we can remove the TypeRegistrar/TypeResolver and instead take a dependency on the Microsoft.Extensions.DependencyInjection package.

Awesome news. I would see if you can instead get away with a dependency on just Microsoft.Extensions.DependencyInjection.Abstractions for the main package, this would allow users to adapt to their own DI system if they want. Then you can provide an extension package for actual concrete support for specific DI frameworks like Microsoft's.

No, we still used a built-in DI container even if the user doesn't provide one, so would need to have an actual implementation as well.

I guess what I'm saying is that there's a strategy you could adopt where the Spectre.Console library itself uses constructor injection normally with no DI framework dependencies. Imagine using pure DI in this case. There would be nothing special about this, you're just using dependency inversion and factories.
Then via extension libraries, you can extend your configuration builder pattern to facilitate initialization of the library (i.e. provide its dependencies) using specific DI libraries, such as MS.DI, Autofac, etc. Users can also implement their own.
I'm not saying you should do this; it's just an approach. If you want to hard-code to support Microsoft.DI, then I completely understand. Other popular libraries like MassTransit do this, and it seems to work fine (except when you need to support older .NET frameworks).
Thanks again for acknowledging the issue and expressing support for it. If I can help at all please let me know.

I have solved a very similar issue in https://github.com/JKamsker/Spectre.Console.Extensions.Hosting-ng

authors: FrankRay78 JKamsker kvpt nils-a patriksvensson phil-scott-78 rcdailey


text:
update to Verify beta

handle this change https://github.com/VerifyTests/Verify/pull/646

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@SimonCropp Happy to merge this, but would need the checks to pass for it.

@SimonCropp I have some spare time, so I'll do it.

@SimonCropp Actually, I'll wait until 18.0.0 has been released. Want the version number of this project to follow Verify.

yep. no worries. i will re submit when it is stable

authors: CLAassistant SimonCropp patriksvensson


text:
Can we use streams instead of files for transcription?

Thank you for creating this helpful and interesting software.  Writing to and reading from files takes time.  Is there anything like an asr_model.transcribe_stream() function?

Hi, in practice, nothing prevents this appart from the ASR model itself. We do have a stream-ready (I.e. unidirectional model) for the AISHELL recipe.

Would appreciate the stream ready code.

Thank you.  I'm having trouble finding the example code in recipes.  Would you please point me to it?  Sorry for being so dim.

I'm running asr with a GPU and it's much faster. https://www.youtube.com/watch?v=JrCA5Li5bTc&t=5s

Thanks for your email. We're looking more for speech cleanup and background removal and separation, then transcription.   Do you think that's possible in real time if we run it on a GPU?
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Mike Bee @.>
Sent: Sunday, April 10, 2022 8:25:00 AM
To: speechbrain/speechbrain @.>
Cc: InSightVlad @.>; Comment @.>
Subject: Re: [speechbrain/speechbrain] Can we use streams instead of files for transcription? (Issue #1365)
I'm running asr with a GPU and it's much faster. https://www.youtube.com/watch?v=JrCA5Li5bTc&t=5s
—
Reply to this email directly, view it on GitHubhttps://github.com/speechbrain/speechbrain/issues/1365#issuecomment-1094182492, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALNFS35L62DUNWRGGYKAVXLVEJQ2ZANCNFSM5S6K4TNQ.
You are receiving this because you commented.Message ID: @.***>

Hi, we don't have any ready example of online ASR or online speech processing. I just said that we have one mode that supports it. With a GPU, achieving real-time shouldn't be an issue.

How can we begin to collaborate? I can send you a system block diagram.  If you believe the current model will provide improvements in the speech signal quality per BF output, please send me a rough quote for your consultation work.  We already sell in the public safety and drone security markets.  If this experiment with conferencing goes well, we are likely to be spinning off a conferencing venture and will require considerably more consultation and or full-time participation.
Cheers,
Vlad
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Parcollet Titouan @.>
Sent: Sunday, April 10, 2022 10:15:53 AM
To: speechbrain/speechbrain @.>
Cc: InSightVlad @.>; Comment @.>
Subject: Re: [speechbrain/speechbrain] Can we use streams instead of files for transcription? (Issue #1365)
Hi, we don't have any ready example of online ASR or online speech processing. I just said that we have one mode that supports it. With a GPU, achieving real-time shouldn't be an issue.
—
Reply to this email directly, view it on GitHubhttps://github.com/speechbrain/speechbrain/issues/1365#issuecomment-1094202876, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALNFS34TLLBXUYQ3WK7IJE3VEJ52TANCNFSM5S6K4TNQ.
You are receiving this because you commented.Message ID: @.***>

Thanks for your email. We're looking more for speech cleanup and background removal and separation, then transcription. Do you think that's possible in real time if we run it on a GPU? Get Outlook for Androidhttps://aka.ms/AAb9ysg
…
________________________________ From: Mike Bee @.> Sent: Sunday, April 10, 2022 8:25:00 AM To: speechbrain/speechbrain @.> Cc: InSightVlad @.>; Comment @.> Subject: Re: [speechbrain/speechbrain] Can we use streams instead of files for transcription? (Issue #1365) I'm running asr with a GPU and it's much faster. https://www.youtube.com/watch?v=JrCA5Li5bTc&t=5s — Reply to this email directly, view it on GitHub<#1365 (comment)>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALNFS35L62DUNWRGGYKAVXLVEJQ2ZANCNFSM5S6K4TNQ. You are receiving this because you commented.Message ID: @.***>
Cleanup can be done on audio files using sox.  I have a video on that.  Noise removal also takes some time and requires a wav file of the ambient noise.

Thank you for your response.

authors: InSightVlad MikeyBeez TParcollet

repo: speechbrain/speechbrain


text:
Hugo Robust theme no longer coincides with QuickStart

While working through the Hugo QuickStart, Step 8. Customize Robust Theme has you create a file (li.html) in bookshelf/layouts/_default. This file only affects the sidebar items as the main page items are now addressed under grid.html in the theme/layouts folder.

@shekhargulati would it be possible for you to update the quickstart guide?

@digitalcraftsman sure will fix it this weekend.

I ran into this problem working through the quickstart guide too.  I took a look at updating the guide and see that only telling the user to provide a grid.html isn't enough for a good look because the robust theme fixes the height of the grid element in styles.css.
Options I see:
. Find some other simpler change to have the user make in the quickstart guide.  Maybe this is best achieved using a different theme?
. Tell them to override the css as well as grid.html in the guide.  This is a little harder for the user presumably.
. Tell them to just update the grid.html, acknowledge that it doesn't look good and mention that they can dig into the css if they want to fix it.

This issue has been automatically marked as stale because it has not been commented on for at least four months.
The resources of the Hugo team are limited, and so we are asking for your help.
If this is a bug and you can still reproduce this error on the master branch, please reply with all of the information you have about it in order to keep the issue open.
If this is a feature request, and you feel that it is still valuable, please open a proposal at https://discuss.gohugo.io/.
This issue will automatically be closed in four months if no further activity occurs. Thank you for all your contributions.

Note/Update: This issue is marked as stale, and I may have said something earlier about "opening a thread on the discussion forum". Please don't.
If this is a bug and you can still reproduce this error on the latest release or the master branch, please reply with all of the information you have about it in order to keep the issue open.
If this is a feature request, and you feel that it is still relevant and valuable, please tell us why.

authors: bep digitalcraftsman ginabythebay jpreston86 shekhargulati


text:
Update mongodb-driver-core to 3.12.4

Updates org.mongodb:mongodb-driver-core from 3.12.3 to 3.12.4.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.mongodb", artifactId = "mongodb-driver-core" } ]

labels: library-update, semver-patch

https://developer.mongodb.com/community/forums/t/mongodb-java-driver-3-12-4-released/3953
https://jira.mongodb.org/browse/JAVA-3688?jql=project %3D JAVA AND issuetype %3D Bug AND fixVersion %3D 3.12.4 ORDER BY priority DESC
"my" issue is not fixed by this version

authors: scala-steward yanns

repo: sphereio/sphere-scala-libs


text:
Search Cuisine: Drop Down Menu on Homepage Doesn't Include Full List of Options

When you are on the homepage, after you select a cuisine you are unable to change your mind.  In the screenshot attached, you can see that I selected Food Truck.  I decided that I instead wanted a different cuisine before I proceeded with clicking on the "search" button.  But when I clicked back on the drop down arrow, it only keeps the original selected "Food Truck" option there.  So I have to refresh the page to get the full menu back.

Also happens on the search results page.  It'd be nice if we could choose a different cuisine from there instead of having to go back to the home page, re-select a location and then choose a cuisine.

repo: spicygreenbook/greenbook


text:
[Feature Request] Static network option

I noitced for mac and android transfers due to limitations, you have to enter a ssid and password (which can be a bit long).
I was wondering if it would be possible for to set a static network (ssid and password don't change) for specific peers (say mac).

Unfortunately no. The Android device has to host the hotspot since macOS doesn't let you start hotspots programmatically. And Android's startLocalOnlyHotspot method generates the SSID and password for you.

Unfortunately no. The Android device has to host the hotspot since macOS doesn't let you start hotspots programmatically. And Android's startLocalOnlyHotspot method generates the SSID and password for you.

Thanks for the clarification. Would it be possible to add a option to scan the qr code generated on android from Mac (say using Webcam if available)?
Alternatively would it be possible to use Bluetooth to share wireless credentials?

Webcam, maybe? I'm not sure how complicated that is. Bluetooth is a good idea but would be a lot of work. Will keep these in mind and do some research.

Hi. I have just found your app. It is really great as a completely local version of eg Snapdrop. I would really love, though if the connection between the two devices was a little smoother, like suggested above. When that is said, it has bridged the gap for me after changing from iOS to Android (/e/os). Thank you!

I've started looking into Bluetooth and I haven't found solid reason to say it isn't possible yet, but it would be a long-term project. I'd have to write a Bluetooth server and client in:

Rust on Windows with windows-rs
Swift on iOS with CoreBluetooth
Kotlin on Android
Rust on Linux with https://github.com/bluez/bluer
and for macOS, hopefully reuse the Swift/iOS code and wrap with Rust somehow, but more likely write the same thing in Obj-C and write Rust wrappers for that.

Each one of those steps is going to be pretty involved and will be a steep learning curve as I haven't worked with Bluetooth before. After all that, I'd have to figure out how to fall back to QR code or manual entry for devices that don't have Bluetooth (or failing that, cameras). And one device can't really know if the other device has Bluetooth, so there will have to be good UI for the user to know and inform the app of whether Bluetooth is worth trying. I don't want to make Bluetooth a requirement for future versions as I imagine there are people with laptops that have WiFi but not Bluetooth. So this is likely months away if it happens.

And thank you @agdamsbo!

authors: Rihcus agdamsbo spieglt

repo: spieglt/FlyingCarpet


text:
Debugging and documentation improvements

Hello!
This is largely a set of nitpicks if I'm 100% honest. While working in the codebase I noted that quite a bit of the debugging output using fmt.Printf etc is actually useful but would be even more useful if it worked with the logging library already available. While in there I took care of some go vet, golint suggestions as well.

Appreciate the cleanup!

authors: benjaminws robzienert

repo: spinnaker/roer


text:
[FEATURE REQUEST] support for allowlisting uploads for improved security

Is your feature request related to a problem? Please describe.
At this moment you can deny what files are uploaded by the uploads.forbid configuration option. Most of the time applications have a really limited scope of files they want to process. Adding an uploads.allow_only option would improve server security by a lot because the user only needs to know what he wants to support.
Describe the solution you'd like
Have a config option like the following, to only support uploading a limited set:
  # File uploading settings.
  uploads:
    # Allow only files with the following extensions to upload. This will overrule the `forbid` config option when not empty
    #
    # Default: []
    allow_only: [".jpg", ".png", ".doc", ".docx"]

Hey @rjd22 . Thanks for the FR. Agree with you. Will plan this feature for the next after 2.6 release.

@rjd22 Will be in the v2.6.0

repo: spiral/roadrunner-plugins


text:
App is currently not Python 3 compatible

Name of the app
Proofpoint
Describe the bug
App is currently not Python 3 compatible
To Reproduce
Error when installing in SOAR Cloud
Expected behavior
Should be installed without issues
Splunk SOAR Version (please complete the following information):

App Version: 1.2.22

Additional context
New build compatible with Python 3 was created (fork link) but not yet tested. Awaiting for test/customer environment.

I am downloading and will implement in the next few weeks and return with feedback.

The fork linked package fails to install with the error:
'proofpoint.json' failed validation on value 'main_module'

This seems to have been fixed.

authors: bneises iso-rgomez marie-splunk

repo: splunk-soar-connectors/proofpoint


text:
ci: multiple update for the pipeline

See commits.

Codecov Report

Merging #426 (4d754cc) into main (b02a176) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #426   +/-   ##
=======================================
  Coverage   78.95%   78.95%
=======================================
  Files          27       27
  Lines        1730     1730
=======================================
  Hits         1366     1366
  Misses        364      364

Continue to review full report at Codecov.

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

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

v5.10.3
GitHub release

Your semantic-release bot :package::rocket:

repo: splunk/addonfactory-ucc-generator


text:
Fixing namespace inconsistency issue

Fixing

https://github.com/splunk/splunk-connect-for-kubernetes/issues/133
https://github.com/splunk/splunk-connect-for-kubernetes/issues/256
https://github.com/splunk/splunk-connect-for-kubernetes/issues/263

this will resolve inconsistency issue, but it will break existing queries.

so is it hard coding field name to container-name and pod-name ?
but now we are changing it to container_name and pod_name?

As mentioned in the issues , there is inconsistency due to - and _ for name , so have to change it to either one (_). As far as queries are concerned , we can add docs for next release. What are your thoughts on this , should we update docs or should I close PR ?

Okay, it is better to be consistent, but we should be careful for releasing breaking changes. let's hold off on this.

This post is more than year old but I still have same issues. I am using Kubernetes 1.20 and Splunk 9.0.2.
Someone has update on this please ?

repo: splunk/fluent-plugin-kubernetes-metrics


text:
GitHub Enterprise Server Monitor Dashboards appear out of order in Nav Menu

Describe the bug
Within GitHub Enterprise, the Monitoring dashboards appear in a top to bottom order all on one page. However in Splunk, the different segments are split into separate dashboards. With that, they appear in alphabetical order in the navigation menu unlike the order they appear in GitHub Enterprise.
To Reproduce
Steps to reproduce the behavior:

Go to 'GitHub App for Splunk'
Click on 'Enterprise Server Monitor' navigation menu
See error

Expected behavior
It would be expected that they appear in the same order in the navigation as they do on the Monitor page in GitHub Enterprise Server.

This has been fixed and will be included in the next release.


text:
ESCU - Cloud Instance Modified By Previously Unseen User - Rule - Error in 'SearchParser': Mismatched ']'.

ESCU version - https://repo.splunk.com/artifactory/webapp/#/artifacts/browse/tree/General/Solutions/DA/da-ess-contentupdate/builds/escu_mustang/bugfix/0.0.5/DA-ESS-ContentUpdate-3.0.9.tar.gz
Search has mismatched ] which needs to be removed.
Error in 'SearchParser': Mismatched ']'.

This search didn't get updated since it wasn't scoped for Beta 2. Updated the search logic and also generated test dataset for testing. Thank you for filing this issue

repo: splunk/security-content


text:
[TR-2958] AWS IAM Account Unlocking

Pull Request Type
Please check all that apply:

[x] New playbook
[ ] Bugfix
[ ] Feature add
[ ] Code style update (formatting, renaming)
[ ] Documentation
[ ] Other (please describe):

Release Notes
Replace the following list with release notes that describe the high level components of the PR:

Feature - Added new playbook for unlocking accounts w/ AWS IAM connector

Playbook quality checklist
Please check if your PR fulfills the following requirements.
Requirements for Settings

[x] Playbook name is A-Z in Title case with underscores between words. (e.g. MS_Graph_Search_and_Purge)
[x] Category in Title case with spaces between words (e.g. Identifier Reputation Analysis)
[x] Description is free of grammatical errors and describe what the playbook does.
[x] Notes list any setup required on the third-party API as well as intended areas for customization.
[x] Label is set to '*'

Requirements for all playbooks

[x] Playbook block count not greater than 15 (not including Start and End blocks).
[x] No more than 3 branching paths.
[x] If referencing a custom list, Notes document what the expected values are in that custom list.

Requirements for all playbook blocks

[x] All blocks have a custom name no more than 4 words, all lowercase, and separated by space (e.g. close workbook task)
[x] All blocks that support a Notes Tooltip have it filled out. Must be grammatically correct and describes the intended purpose of that block.
[x] Where custom code is used, block notes indicate presence of custom code (e.g. "This block uses custom code")
[x] No block is disabled by custom code
[x] Custom code is documented with notes

Requirements for specific blocks
Action

[x] Use apps available on Splunkbase
[x] Use asset names that are the app name, all lowercase separated by underscores (e.g. Azure AD Graph becomes azure_ad_graph)

Utility

[x] Block is using community version

Playbook

[x] Block is using local version

Requirements for specific playbooks
Input playbooks

[x] Start blocks use ocsf variable names and a minimum of one data type per variable name (e.g. device (type: host name))
[x] Has at least one category tag (e.g. reputation)
[x] Playbook has a tag for each vendor app used

Other considerations (PR type specific)

[x] If new playbook, there is a screenshot ending in .png with the same name as the playbook .json
[x] Playbook major minor version matches repo (e.g. 5.5 != 6.0)
[x] PR contains both .py and .json

Thanks for contributing!

Remove D3-AL tag.
Update description to indicate that enabling user reattaches their login profile which requires setting a new password.
Filter block tag says "disable_success" instead of "enable_success"

Other than that, looks good.

authors: kelby-shelton ljstella


text:
Handling of queue size overflow?

We have noticed that when we have overflow in the queue (queue is full = @code false)
Loggdata just gets thrown away and no information is presented that indicates overflow.
Is this intended behaviour? Or am I missing something?
I would love to have some sort of indication logged that the queue size have been reached.
/**
     * Inserts the specified element into this queue if it is possible to do
     * so immediately without violating capacity restrictions, returning
     * {@code true} upon success and {@code false} if no space is currently
     * available.  When using a capacity-restricted queue, this method is
     * generally preferable to {@link #add}, which can fail to insert an
     * element only by throwing an exception.
     *
     * @param e the element to add
     * @return {@code true} if the element was added to this queue, else
     *         {@code false}
     * @throws ClassCastException if the class of the specified element
     *         prevents it from being added to this queue
     * @throws NullPointerException if the specified element is null
     * @throws IllegalArgumentException if some property of the specified
     *         element prevents it from being added to this queue
     */
    boolean offer(E e);

Hi @blackside,
Thanks for reaching out.
Could you please elaborate on this issue?
It would help us in troubleshooting into more detail.
Also, we are made several releases since this issue posted. We would recommend you to update logging library to newer version 1.11.6 and re-validate your use-case.

Hi @blackside,
We have not heard any update regarding your issue.
We are closing this issue for now.
Please let us know if you have any further issues.
Thank you.

authors: blackside bparmar-splunk

repo: splunk/splunk-library-javalogging


text:
Update init.py

Add support for Splunk authentication tokens in example scripts (search.py, etc.)

@hogoboom , in the latest release we have added support for both the authentication tokens i.e splunkToken(sessionKey) and token(bearer token). please refer updated. Closing this PR as the change is already been added.

authors: ashah-splunk hogoboom

repo: splunk/splunk-sdk-python


text:
CocoaPods specification

Would it be possible to get this library turned into a CocoaPod for use with dependency management?
Thanks!

There is a Cocoapod. pod 'SPGooglePlacesAutocomplete', '1.0.4'

authors: AnthonyMDev hkellaway

repo: spoletto/SPGooglePlacesAutocomplete


text:
Reload Page keyword

Originally submitted to Google Code by @yanne on 29 Apr 2011

useful when you want to e.g. reload page.

Originally submitted to Google Code by @yanne on 29 Apr 2011

This issue was updated by revision 470e146182fb.

repo: spooning/SeleniumLibrary