text:
可以上传一个workflow.json吗，版本好像有很多冲突

hallo2是不是有很多版本以来的冲突，torchaudio我已经重新装了，huggingface hub也降低了版本。
workflow跑到HalloUpscale的时候说我模型或者torch版本有问题，可以帮忙看看吗
:
Traceback (most recent call last):
File "/workspace/lawks/ComfyUI/execution.py", line 328, in execute
output_data, output_ui, has_subgraph = get_output_data(obj, input_data_all, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)
File "/workspace/lawks/ComfyUI/execution.py", line 203, in get_output_data
return_values = _map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)
File "/workspace/lawks/ComfyUI/execution.py", line 174, in _map_node_over_list
process_inputs(input_dict, i)
File "/workspace/lawks/ComfyUI/execution.py", line 163, in process_inputs
results.append(getattr(obj, func)(**inputs))
File "/workspace/lawks/ComfyUI/custom_nodes/ComfyUI_Hallo2/Hallo_node.py", line 618, in Upscale_main
net,face_upsampler,bg_upsampler,face_helper=pre_u_loader(bg_upsampler, model_path, bg_tile, upscale, face_upsample, device, hallo_model_path,face_detection_model,parse_model,has_aligned)
File "/workspace/lawks/ComfyUI/custom_nodes/ComfyUI_Hallo2/hallo/video_sr.py", line 95, in pre_u_loader
raise "un support torch version or checkpoints"
TypeError: exceptions must derive from BaseException

我看了一下这行代码的报错，是net_g.pth 这个模型的问题，你重新下载一个试试，而且这个的确跟torch版本有关系，代码是涵盖了不同版本的torch加载的

非常感谢您的回答，我的torch相关版本是:
torch                              2.2.2
torchaudio                         2.2.2
torchsde                           0.2.6
torchvision                        0.17.2
重新下载了net_g,发现还是有新的报错，也出现在HallosUpscaleloader node上。报错信息如下：
!!! Exception during processing !!! invalid load key, 'v'.
Traceback (most recent call last):
File "/workspace/lawks/ComfyUI/execution.py", line 328, in execute
output_data, output_ui, has_subgraph = get_output_data(obj, input_data_all, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)
File "/workspace/lawks/ComfyUI/execution.py", line 203, in get_output_data
return_values = _map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)
File "/workspace/lawks/ComfyUI/execution.py", line 174, in _map_node_over_list
process_inputs(input_dict, i)
File "/workspace/lawks/ComfyUI/execution.py", line 163, in process_inputs
results.append(getattr(obj, func)(**inputs))
File "/workspace/lawks/ComfyUI/custom_nodes/ComfyUI_Hallo2/Hallo_node.py", line 618, in Upscale_main
net,face_upsampler,bg_upsampler,face_helper=pre_u_loader(bg_upsampler, model_path, bg_tile, upscale, face_upsample, device, hallo_model_path,face_detection_model,parse_model,has_aligned)
File "/workspace/lawks/ComfyUI/custom_nodes/ComfyUI_Hallo2/hallo/video_sr.py", line 117, in pre_u_loader
face_helper = FaceRestoreHelper(
File "/workspace/lawks/ComfyUI/custom_nodes/ComfyUI_Hallo2/hallo/facelib/utils/face_restoration_helper.py", line 125, in init
self.face_detector = init_detection_model(self.det_model, half=False, device=self.device)
File "/workspace/lawks/ComfyUI/custom_nodes/ComfyUI_Hallo2/hallo/facelib/detection/init.py", line 16, in init_detection_model
model = init_retinaface_model(model_name, half, device)
File "/workspace/lawks/ComfyUI/custom_nodes/ComfyUI_Hallo2/hallo/facelib/detection/init.py", line 37, in init_retinaface_model
load_net = torch.load(model_name, map_location=lambda storage, loc: storage)
File "/workspace/hallo/lib/python3.10/site-packages/torch/serialization.py", line 1040, in load
return _legacy_load(opened_file, map_location, pickle_module, **pickle_load_args)
File "/workspace/hallo/lib/python3.10/site-packages/torch/serialization.py", line 1258, in _legacy_load
magic_number = pickle_module.load(f, **pickle_load_args)
_pickle.UnpicklingError: invalid load key, 'v'.
非常感谢您宝贵的时间

检查一下你的accelerate的版本，官方的要求accelerate==0.28.0

依旧是跑到HallosVideoUpscale有问题，前面的应该都推理完了。warning信息：Pipelines loaded with dtype=torch.float16 cannot run with cpu device. It is not recommended to move them to cpu as running them will fail. Please make sure to use an accelerator to run the pipeline in inference, due to the lack of support forfloat16 operations on this device in PyTorch. Please, remove the torch_dtype=torch.float16 argument, or use another device for inference. 感觉还是某些模型文件或者什么库的版本有问题，我的accelerate就是0.28.0

你是Linux跑的吗? 你把这行 注释掉，这是为了小显存做的模型卸载，可能你那里不支持。

嗯嗯好的非常感谢，我尝试一下，我用的A40 40G的显存

source: gharchive/issue

authors: nothairyyq smthemex

license: MIT

license_type: permissive

license_source: github-api


text:
RFC: Use arrays for media query values

@meyer An idea for a more concise API for media queries:
Current Behavior:
<Block
  mediaQueries={{ sm: '(min-width:700px)',  lg: '(min-width: 1000px)' }}
  color="red"
  smColor="green"
  lgColor="blue"
  margin="2rem"
/>

Desired Functionality:
<Block
  mediaQueries={['(min-width: 400px)', '(min-width: 1000px)' ]}
  color={['red', 'blue', 'green']}
  margin="2rem"     // strings are still the cool
/>

@jaredpalmer I’ve seen this syntax in a few different CSS in JS libraries and I kind of dig it, but I went with the more explicit version for a few reasons:

we’ve got precedent for using prefixes. Doesn’t mean everything needs to fit into the prefix model but it’s a thing to consider when adding new features.
with separate props for each style prop, we get existing prop merging behaviour for freeeee (JSX spread syntax). Single props as arrays are nigh impossible to merge.
This is probably negligible, but I can skip a check if the mediaQueries prop is not present. With the proposed array API I’d either have to do an array check for each prop or I’d have to do something hackier. I might be wrong about this though, as I don’t know all the tricks in the JS book.

More concisely put, the reason I like one prop per style value instead of props with potential array values is that you can keep style “features” somewhere else and then merge them all into one thing. Here’s a contrived example:
import {styles, mediaQueries} from '../common/LayoutConstants';

<Block
  {...styles.button}
  {...styles.redButton}
  {...styles.smFullWidth}
  mediaQueries={mediaQueries}
  props={{type: 'button'}}
>
  Red button that’s full width on small screens
</Block>

with separate props for each style prop, we get existing prop merging behaviour for freeeee (JSX spread syntax). Single props as arrays are nigh impossible to merge.

Okay. You've convinced me.

authors: jaredpalmer meyer

repo: smyte/jsxstyle


text:
add a key argumnet to control whether will async query

https://github.com/sn0wfree/ClickSQL/blob/42906c3e17e0cf5de31ee6c0ad13d89c097fd52b/ClickSQL/clickhouse/ClickHouse.py#L411
add a key argumnet to control whether use async query
 def __execute__(self, sql: (str, list, tuple), convert_to: str = 'dataframe', transfer_sql_format: bool = True,async=False ):

link to this


text:
Snakemake: MissingInputException

Snakefile.txt
Hello,
I am trying to create a simple Snakemake workflow and I am having some issues. My file generates the following errors:
"
MissingInputException in line 29 of /home/agalvez/data/workflow-workshop/test/Snakefile: Missing input files for rule hmm:
hmmsearch --tblout output_tblout_egf --noali -E 99
MissingInputException in line 49 of /home/agalvez/data/workflow-workshop/test/Snakefile: Missing input files for rule create_archive: output/EP00771_Trimastix_marina.out output/EP00759_Prokinetoplastina_sp_PhF-6.out
"
It is the first time I ever try to use Snakemake or anything related to Python so I do not understand why this is failing. Any help would be really appreciated. Thanks in advance!

Can you post your Snakefile content?

You should post these types of questions on Stackoverflow. Anyway, your issue is putting your shell command in the input section. Take a look at the tutorial example to see how to properly define a rule.

Can you post your Snakefile content?

This is how my Snakefile looks:
ARCHIVE_FILE = 'output.tar.gz'
a single output file
OUTPUT_FILE = 'output/{species}.out'
a single input file
INPUT_FILE = 'proteins/{species}.fasta'
Build the list of input files.
INP = glob_wildcards(INPUT_FILE).species
print(INP)
The list of all output files
OUT = expand(OUTPUT_FILE, species=INP)
print(OUT)
pseudo-rule that tries to build everything.
Just add all the final outputs that you want built.
rule all:
input: ARCHIVE_FILE
hmmsearch
rule hmm:
input:
cmd='hmmsearch --tblout output_tblout_egf --noali -E 99',
species=INPUT_FILE ,
hmm='hmm/EGF.hmm'
output: OUTPUT_FILE
shell: '{input.cmd} {input.hmm} {input.species} {output}'
hmmsearch
#rule hmm:
shell: 'hmmsearch --tblout output_tblout_egf --noali -E 99 hmm/EGF.hmm INPUT_FILE OUTPUT_FILE'
create an archive with all results
rule create_archive:
input: OUT
output: ARCHIVE_FILE
shell: 'tar -czvf {output} {input}'

Can you post your Snakefile content?

I am attaching the Snakefile to this reply. Thanks for the feedback.
Snakefile.txt

This question has been asked and answered on stackoverflow. @AlexGalvez I think you should close this issue.

authors: AlexGalvez SichongP cademirch dariober

repo: snakemake/snakemake


text:
Adding a Placeholder

Wondering if there is a way to add a placeholder text like an input element.
Thanks in advance

Just thought I'd drop a comment about monkey patching this:
Lines 408
            // Create the type_zone input using custom class and contenteditable attribute
            self.$type_zone = $( document.createElement( "input" ) )
                             .addClass( self.config[ "type-zone-class" ] )
                             .attr( "contenteditable", true )
                             .attr("placeholder", "New Tag");

I imagine you could do something like:
.attr("placeholder", self.config[ "type-zone-placeholder" ] )
If I find that i need customized placeholders in my project and I end up doing this Ill paste the results/pull request

authors: dishcandanty luchorengo

repo: sniperwolf/taggingJS

license: mit

license_source: bigquery


text:
Do we need stripCmdPath?

@izgzhen @angerman @alpmestan I've removed unused stripCmdPath:
-- ref: mk/config.mk
-- | Command line tool for stripping.
stripCmdPath :: Action FilePath
stripCmdPath = do
    targetPlatform <- setting TargetPlatform
    top <- topDirectory
    case targetPlatform of
        "x86_64-unknown-mingw32" ->
             return (top -/- "inplace/mingw/bin/strip.exe")
        "arm-unknown-linux" ->
             return ":" -- HACK: from the make-based system, see the ref above
        _ -> return "strip"

Is this the right thing to do? There is the following comment in Settings.Builders.GhcCabal:
-- don't strip libraries when cross compiling.
-- XXX we need to set --with-strip= (stripCmdPath :: Action FilePath), and if it's ':' disable
--     stripping as well. As it is now, I believe we might have issues with stripping on
--     windows, as I can't see a consumer of `stripCmdPath`.

stripCmdPath was previously used in install by GhcCabal.

I suggest let's keep this issue open and delete the dead code for now

@izgzhen Thanks, agreed!

authors: izgzhen snowleopard

repo: snowleopard/hadrian


text:
Added example fields to example_event schema and update to README.md

Added description to example_event schema.
Added null on non-required field on example_event schema.
Updated README.md and removed severity level text.

Thank you @joaolcorreia !
These changes have been included in https://github.com/snowplow/iglu-example-schema-registry/pull/11 .
Closing.

source: gharchive/pull-request

authors: adatzer joaolcorreia

repo: snowplow/iglu-example-schema-registry


text:
Make Kafka sink more fault tolerant by allowing retries

See here:
https://github.com/snowplow/snowplow/blob/e977c976151a36894ed7b83e9461c2fea7eec252/2-collectors/scala-stream-collector/src/main/scala/com.snowplowanalytics.snowplow.collectors/scalastream/sinks/KafkaSink.scala#L60
Would be great to have an optional param in config to set the number of retries.
That might bring up duplicates, as the docs says though (https://kafka.apache.org/0100/javadoc/index.html?org/apache/kafka/clients/producer/KafkaProducer.html):
If the request fails, the producer can automatically retry, though since we have specified retries as 0 it won't. Enabling retries also opens up the possibility of duplicates (see the documentation on message delivery semantics for details).

Good idea! Scheduling

authors: alexanderdean simplesteph

repo: snowplow/snowplow


text:
New Feature: Lockable flowchart

Description
Added an update to the flowchart window and editors so flowcharts can be locked, preventing editing.
What is the current behavior?
Currently, when working with flowchart prefabs and defaults, you may want the flowchart to not be editable at least not without needing to. This can also be useful for tracking what flowcharts are done and not.
Issue Number: #992
What is the new behavior?
Clicking on the padlock icon in the flowchart window (or "lock flowchart" in the flowchart inspector) will lock the flowchart. It will not keep the flowchart in the editor window, unlike locking an inspector, but will prevent the blocks, commands, metadata, and variables from being changed. However, you can still scroll through commands to view them and move through the flowchart freely. It does work by greying out the properties, however. Will edit later if needed.

In the future, the flowchart window can be adjusted to also lock on one flowchart, not changing if another is selected. However this will only be useful when multiple flowchart windows can be opened at once; a feature that seems to be missing for now.
Important Notes

My change require modifcations or additions to documentation
My change modifies the workflow/editing of flowcharts/blocks/commands etc.

Possibly. Flowcharts are unlocked by default so it shouldn't change much.

So far, no breaking changes have been found and it should also merge with #996 with how I edited the code. One main issue is that users may see the padlock and think it locks the flowchart in that window when it does not; only locks editing of it. This will be addressed when multiple flowchart windows can be open, at which point an additional behaviour will be added.
Other information
Important: because I did a silly, this PR includes changes from PR #934 which contains an optional boolean to Writer Audio. When merging this branch, 934 can be deleted as this already contains it. (as usual)

Resolved a merge conflict with CommandListAdaptor after my last changes were merged. I saw this coming so it was simple.

Hi @Arylos07 I lef this message on Discord as well, but I'm leaving it here for transparency:
I've been using your lock flowchart feature ever since you PR'ed in, and I only now ran into a bug with it, where it prevents editing the flowchart itself, I mean the actual Flowchart.cs on the gameobject, it seems like it redraws itself constantly if I try to edit it EVEN if the lock is off. I went back to a version right before I added the PR and the bug is not present there. Could you please take a look at this issue sometime? Thank you

Gif added to demonstrate (as soon as I click an editable field or open the inspector window selection thing, it gets redrawn)

Can confirm the fix submitted fixes my issue! Thank you again @Arylos07

repo: snozbot/fungus


text:
app crashed - waiting for file changes before starting

Hello Team, I have an interview scheduled for the 1st of August. I cloned the typescript-recurisve-deps branch and tried to run it locally. I did npm install first and then npm start. But I get this error after doing npm start:

It would be a great help if anyone could help me with it.  I tried to look for the error on StackOverflow but it still persists.
Edit: nodemon is installed locally. I believe this error occurs due to the following line of code in package.json file:

@RaHuLMeghani24 I do not see this error when I try to repeat your steps using npm 6.14.15 on macOS Monterey ...
$ npm --version
6.14.15
$ node --version
v14.17.6

$ npm install

> nodemon@2.0.12 postinstall /Users/georgeharley/code/workspace/snyk-code-review-exercise/node_modules/nodemon
> node bin/postinstall || exit 0

added 709 packages from 418 contributors and audited 710 packages in 12.188s

62 packages are looking for funding
  run `npm fund` for details

found 29 vulnerabilities (2 moderate, 18 high, 9 critical)
  run `npm audit fix` to fix them, or `npm audit` for details
$ npm start

> snyk-code-review-exercise@1.0.0 start /Users/georgeharley/code/workspace/snyk-code-review-exercise
> nodemon --watch src --ext .ts --exec "$(npm bin)/ts-node ./src/main.ts"

[nodemon] 2.0.12
[nodemon] to restart at any time, enter `rs`
[nodemon] watching path(s): src/**/*
[nodemon] watching extensions: ts
[nodemon] starting `/Users/georgeharley/code/workspace/snyk-code-review-exercise/node_modules/.bin/ts-node ./src/main.ts`
Server listening on http://127.0.0.1:3000

. . .

What version of npm are you using?

@georgeharley Thank you for reaching out. I am using npm version-8.1.2 and node version-16.13.2 on windows machine

@RaHuLMeghani24 Yes, looks like the node and npm versions might not be behind this. I just switched to using node version 16.13.2 and npm 8.1.2 (i.e. your original versions) and everything still works on my machine.
I am using the bash shell in my macOS terminal and if I just run echo $(npm bin)/ts-node ./src/main.ts from the root of the repo clone then it also works OK...
$ echo $(npm bin)/ts-node ./src/main.ts
/Users/georgeharley/code/workspace/snyk-code-review-exercise/node_modules/.bin/ts-node ./src/main.ts

@georgeharley Thank you so much for sharing that terminal output. It makes sense cause in linux and MacOS, we use $ notation for variables.
By following the output, I modified my package.json file and added the entire path:

It is working now. Thank you for your help. You can close this issue.

repo: snyk/snyk-code-review-exercise


text:
chore: asset classification

Adding/updating asset classification as per ProdSec standard.

 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.

authors: CLAassistant dragos-cojocari

repo: snyk/snyk-visual-studio-plugin


text:
🛑 Interseed (interseed.app) is down

In b844606, Interseed (interseed.app) (https://interseed.app) was down:

HTTP code: 404
Response time: 128 ms

Resolved: Interseed (interseed.app) is back up in c83703d after 11 minutes.

repo: so-more/proof-monitoring


text:
Dependency Injection?

Is there a way to inject dependencies into a Controller?

@Benjyclay can you give me an example of what you're trying to accomplish?

class ClassOne extends Controller {
    private $classTwo;

    public function __construct(ClassTwo $classTwo) {
        $this->classTwo = $classTwo;
    }

    public function getSomeStuf($store) {
        return $this->classTwo->getSomeStuff();
    }
}

Hm, not sure this is making sense to me. Are you trying to pass a Class in, using __construct?

@darrenjacoby yes - in the example above, an instance of ClassTwo is injected into the constructor of the controller at the moment it is instantiated. Being able to do something like this is really important for clean architecture, as you'd no longer need to hardcode any external dependencies into the controller class.
I recommend looking into Laravel's DI container (which is used by Sage anyway): https://laravel.com/docs/5.5/container - I assume this would be the cleanest way to set it up. If it was possible to use autowiring to inject dependencies, it would make the controller class much more powerful.

What type of class would you want to inject here ?
As i just tested by injecting another Controller, and it works the laravel Way.
use \App\Home; //another controller

class MyController extends Controller {

	private $test;

	public function __construct(Home $home) {
		$this->test = $home;
	}

	public function myMethod() {
		return $this->test->omagad(); //public method from Home Controller
	}
}

+1 would be cool.
Actually, I can achieve the same result as Laravel do, but it's "dirty code" :)
use \App\Home;
/**

Class ConstruisezVotreParcours

@package App
*/
class ConstruisezVotreParcours extends Controller {
private $test;
public function __construct() { //no real injection
add_action( 'wp_ajax_nopriv_getRelatedJobsAsync', [ $this, 'getRelatedJobsAsync' ] );
add_action( 'wp_ajax_getRelatedJobsAsync', [ $this, 'getRelatedJobsAsync' ] );
$this->test = new Home(); //dirty
}
public function ficheMetier() {
$query = new \WP_Query(
[
'post_type'   => 'metier',
'post_status' => 'publish',
'order'       => 'asc',
'orderby'     => 'title'
]
);
 return $this->test->myHomeMethod();

}

}

I'll see what I can do here to try clean this up!

@indrek-k yeah, the autowiring is going to be the best option for dep injection.
I'm assuming something like this should work using the DI container, which Sage just extends—
https://gist.github.com/davejamesmiller/bd857d9b0ac895df7604dd2e63b23afe#basic-usage

Cool, got it working using the Sage container.
So you could do this;
app/Dep/ClassA.php
<?php

namespace App\Dep;

class ClassA
{
    private $classB;

    public function __construct(ClassB $classB)
    {
        $this->classB = $classB;
    }

    public function get()
    {
        return 'Class A and ' . $this->classB->get();
    }
}

app/Controllers/Single.php
<?php

namespace App\Data;

use Sober\Controller\Controller;

use App\Dep\ClassA;

class Single extends Controller
{
    private $classA;

    public function __construct(ClassA $classA)
    {
        $this->classA = $classA;
    }

    public function getClassA()
    {
        return $this->classA->get();
    }
}

I'll push to dev-master soon for further testing.

Is the
public function __construct(ClassA $classA)
{
    $this->classA = $classA;
}

From Single Controller only ran once ? Or multiple times ? As i remember trying to use the __construct to put some add_filters, and it wasn't good cause it was ran multiple times: and so filter added multiple times.

@mtx-z yeah seems to be only running once.

authors: Benjyclay darrenjacoby indrek-k mtx-z

repo: soberwp/controller


text:
example9: Show additional info about last point in the popup binded to the marker

My geojson looks like:
{
  "type": "Feature",
  "geometry": {
    "type": "LineString",
    "coordinates": []
  },
  "properties": {
    "name": "",
    "times": [],
    "accuracy": [],
    "anotherProps": []
  }
}

I want to show time, accuracy, anotherProp of the last point in the popup, but pointToLayer() return me Point with all properties of LineString. What is the correct way to get this info?
Maybe the best way is using coordinateProperties as described here:
https://github.com/mapbox/geojson-coordinate-properties
{
  "properties": {
    "name": "",
    "coordinateProperties": {
      "times": [],
      "accuracy": [],
      "anotherProps": []
    }
  }
}

In this case _getFeatureBetweenDates() can slice coordinateProperties, and pointToLayer() will return only props of the single point.
Any thoughts?

hey @vdemin,
there is a point created: https://github.com/socib/Leaflet.TimeDimension/blob/e698bfd33d3aab15c1cba931b2658ae5dd5c3a29/src/leaflet.timedimension.layer.geojson.js#L82
So just add the following line:
layer.bindPopup(feature.properties.anotherProps);

authors: trillerpfeife vdemin


text:
Can't install this package

Hi,
when trying to installl this package I run into following
15:15 $ npm i socket.io-emmiter --save
npm ERR! Darwin 15.6.0
npm ERR! argv "/usr/local/Cellar/node/6.4.0/bin/node" "/usr/local/bin/npm" "i" "socket.io-emmiter" "--save"
npm ERR! node v6.4.0
npm ERR! npm  v3.10.3
npm ERR! code E404

npm ERR! 404 Registry returned 404 for GET on https://registry.npmjs.org/socket.io-emmiter
npm ERR! 404
npm ERR! 404  'socket.io-emmiter' is not in the npm registry.
npm ERR! 404 You should bug the author to publish it (or use the name yourself!)
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/schovi/work/be/chat-ws/npm-debug.log

For some unknow reason to me, package info in registry is empty https://registry.npmjs.org/socket.io-emmiter

Shouldn't it be socket.io-emitter, instead of socket.io-emmiter?

@darrachequesne oh dear! Can we act, like this never happened?

authors: darrachequesne schovi


text:
Basic GET request errors with "Protocol::HTTP1::InvalidRequest" and "Protocol::HTTP2::StreamError: Stream closed!"

I've been trying to use async-http-faraday and I'm struggling to get it to work, even with the simplest request I can think of: a GET request to example.com.
I made a dedicated reproduction repo here, the rspec test file is here.
You can clone the repo and run the following to see the test results:
bundle install
bundle exec rspec

In summary, it shows that:

Faraday with the default Net::HTTP adapter works fine with webmock or with real requests
Faraday with the async-http adapter fails both with webmock or with real requests

with webmock I get a Protocol::HTTP1::InvalidRequest error
with a real request I get a Protocol::HTTP2::StreamError: Stream closed! error

Using async-http by itself (without Faraday) it works both with a real request or with webmock, although with the real request I get a warning about Closing resource while still in use!

Nevermind, I completely missed that you're supposed to pass the url or domain to Faraday#new, not to Faraday::Connection#get.
Somehow that works for the default adapter but not for the async-http adapter.

After looking at my original application code I noticed that I was not using Faraday::Connection#get but Faraday::Connection#run_request, which takes a url as the second parameter, not a path. So it seems like async-http-faraday is not handling that use case correctly while Faraday's default adapter does (the default adapter also seems to deal well with the usage of get(url), although that does seem to be out of spec)

What do you think we should do here to solve the problem?

@ioquatix
I'm not completely sure yet but I think the issue is that the async-http adapter has no default path, a url like https://example.com leads to it sending "GET  HTTP/1.1" (bad request) while https://example.com/ (difference is the trailing slash) sends a valid "GET / HTTP/1.1" request as expected.
For example:
# BAD
Faraday::Connection.new { |f| f.adapter :async_http }.get('http://example.com')
Faraday::Connection.new { |f| f.adapter :async_http }.run_request(:get,'http://example.com', nil, nil)

# GOOD
Faraday::Connection.new { |f| f.adapter :async_http }.get('http://example.com/')
Faraday::Connection.new { |f| f.adapter :async_http }.run_request(:get,'http://example.com/', nil, nil)

We now normalize the trailing slash in request path, so that it behaves like other adapters.

See https://github.com/socketry/async-http-faraday/commit/645d2458635ec8e796ba96fabfa4146def3ea102 for details.

repo: socketry/async-http-faraday


text:
Implement profiling/EDA metrics

soda-sql offered the following profiling metrics to be pushed into the EDA views of Soda Cloud:

[ ] count

[ ] numeric columns:

[ ] min
[ ] max
[ ] avg
[ ] stddev
[ ] variance
[ ] frequent values
[ ] histograms

[ ] text columns:

[ ] min length
[ ] max length
[ ] avg length
[ ] format derivation

These metrics all need to be re-implemented in soda-core this means:

fit into the new metrics objects
be able to generate the right queries for each supported dialect

More info and links to cloud DTOs can also be found in this closed issue: https://github.com/sodadata/soda-sql-v3/issues/250

In soda cloud, I see that text columns have the "Extreme Values" tab enabled. However, for text columns, I don't really see how that makes sense on text columns.
@tombaeyens @dirkgroenen do you have an idea why that is?

@bastienboutonnet I think this is a duplicate of https://github.com/sodadata/soda-core/issues/1256
Can you close this one if you agree or make a comment if you think otherwise?

Implemented

authors: bastienboutonnet tombaeyens

repo: sodadata/soda-core


text:
SodaCore 3.0.48 is not compatible with Airflow 2.7

Apache Airflow 2.7 has just been released but it looks like it not not possible to install the latest version of Soda Core due to a conflict with opentelemetry-api.
ERROR: Cannot install apache-airflow==2.7.0 and soda-core==3.0.48 because these package versions have conflicting dependencies.

The conflict is caused by:
    apache-airflow 2.7.0 depends on opentelemetry-api==1.15.0
    soda-core 3.0.48 depends on opentelemetry-api~=1.16.0

SODA-1811

Going to close as this is really an issue with Airflow who are looking to correct this in the next version.

authors: jmarien nathadfield


text:
🛑 SamOehlert is down

In 4c20e7e, SamOehlert (https://www.samoehlert.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: SamOehlert is back up in b1d6d2c.

repo: soehlert/upptime


text:
Feature/highlights

Added multiple small weather cards under the main Today's Highlights card.

Checked through the app, and everything looks good. Merging the pull request.


text:
It should be possible to re-provision a VM by doing an OS reload instead of a cancel

It should be possible to re-provision a VM by doing an OS reload instead of a cancel.
i.e. I want something that acts like knife softlayer server destroy ..., except that instead of cancelling the VM through the SL API, it would instead do an OS reload.

@hairyhenderson,
Here's the feature branch if you'd like to take it for an early test drive and give any feedback you've got.
https://github.com/softlayer/knife-softlayer/tree/feature/server_relaunch
Usage is knife softlayer server relaunch <NODE NAME>.  That's going to relaunch the VM and bootstrap it from scratch, same node name, same attributes, etc.
/cc @underscorephil

@hairyhenderson ,
This has been merged and a release has been cut.  Upgrade to v0.3.12 to pick up this functionality.
/cc @underscorephil

@cphrmky - thanks! Sorry I never got around to test-driving it... Haven't had the cycles for this for a few weeks. :(

@hairyhenderson haha, no worries, I took it for a spin myself, of course other people are going to be better I finding the bugs I've written than I will...

authors: cphrmky hairyhenderson

repo: softlayer/knife-softlayer


text:
compatibility with github

Hi,
git-codeowners looks like the right way to quickly test if files will be managed by github.
Is there any way that we could verify that the tool has the same understanding as github in regards to file matching?
For example I think that github would understand: foo/bar @owner to match all the files under foo/bar whereas git-codeowners doesn't.

I'll take a closer look at this weekend. do you have an example repo I could clone an test with?

We're using it over at https://github.com/NixOS/nixpkgs . It's a rather large repo.

Thanks for looking by the way :)

I took a look at this and did find some unhandled cases, Ill try to get a new release out soon with improvements

brew upgrade git-codeowners
This should be fixed now. If you query a file within an owned directory the directory owner should now be resolved
$ grep python .github/CODEOWNERS
/pkgs/top-level/python-packages.nix     @FRidh
/pkgs/development/interpreters/python   @FRidh
/pkgs/development/python-modules        @FRidh
/doc/languages-frameworks/python.md     @FRidh
$ git codeowners  pkgs/development/interpreters/python/setup-hook.py
@FRidh

thanks!

authors: softprops zimbatm

repo: softprops/git-codeowners


text:
Command not defined

I installed the plugin as specified in the README:
addSbtPlugin("me.lessis" % "np" % "0.2.0")

resolvers += "lessis" at "http://repo.lessis.me"

I then created a new folder, cd'ed into it, ran sbt, and then attempted to run np:
> np
[error] Not a valid command: np (similar: <, -, inspect)
[error] Not a valid project ID: np
[error] Not a valid configuration: np
[error] Not a valid key: np (similar: run, name, skip)
[error] np
[error]   ^

Did I do something wrong? This is using sbt 0.11.1.

add the following to your build def
seq(npSettings:_*)

np used to just override settings can get the auto setting mix treatment but there is an issue with that in sbt if you are using global plugins do the the order in which build definitions get evaluated. What I mainly wanted was a way to set a default name and org id for all my projects. I set those at a global level but found that you have a chicken and egg situation if you want to provide a default in the plugin (which I do) and and global override. In my case, the default overrides the global settings because of the order sbt loads settings.
Try explicitly mixing in npSettings and get back to me. I'll leave the issue open until then. In the meantime, I'll update the readme.
For reference, this is what I use locally in a ~/.sbt/np.sbt config
seq(npSettings:_*)

(NpKeys.defaults in (Compile, NpKeys.np)) ~= {
  _.copy(org="me.lessis", version="0.1.0-SNAPSHOT")
}

repo: softprops/np


text:
Ability to not-animate titles on startup, when titleRotateAngle is set to something other than default

Hello! So I set the titleRotateAngle to 90. Is there a feature to cancel the animation that bounces the titles until their final position?

Yes, there is a property for that.
wheel.initTitleRotate = true;
May I use this site in World wide wheel section? :smile:

That'd be awsome! I will deploy to github in upcoming days and let you know.
Thanks for the wheel.initTitleRotate = true; it worked perfectly. I'll close the issue.

authors: ponentesincausa softwaretailoring

repo: softwaretailoring/wheelnav


text:
آیا کدها و برنامه نویسی امکان تاخیر آلزایمر را دارند؟

نام و نام‌خانوادگی شما
userFullName: "امیر شکری"
عنوان محتوا
postTitle: "آیا کدها و برنامه نویسی امکان تاخیر آلزایمر را دارند؟"
خلاصه‌ای حداکثر ۲ الی ۳ خطی درباره مضمون محتوا
postDescription: "در این نوشته در مورد اینکه برنامه نویسی و کدها امکان تاخیر در بیماری آلزایمر را دارند صحبت شده که خیلی جالب میتونه باشه..."
لینک محتوا
postLink: "https://www.vice.com/en/article/qkv5xq/can-learning-to-code-delay-alzheimers"

that was intresting

authors: Foroozani amirshnll

repo: softwaretalks/newsletter


text:
Fix app crash on first launch

On first install, app crashes with not valid mint. Issue #23

This is an identical dupe of PR https://github.com/solana-labs/dapp-scaffold/pull/66

Error message suggests to handle the potential error when deserializeMint is called.
Therefore, the entire contents of the MintParser function, which is where deserializeMint is called, should be wrapped inside a try {} catch {} statement, like this:
export const MintParser = (pubKey: PublicKey, info: AccountInfo) => {
try {
const buffer = Buffer.from(info.data);
const data = deserializeMint(buffer);
const details = {
pubkey: pubKey,
account: {
...info,
},
info: data,
} as ParsedAccountBase;
return details;
} catch(error) {
console.error(error);
}
};
I have submitted a PR here: #103

authors: JoshuaFlood jasonhargrove vsinha

repo: solana-labs/dapp-scaffold


text:
Japanese Castle token

Please note: This repository is being rebuilt to accept the new volume of token additions and modifications. PR merges will be delayed.

I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
This repository is managed using an auto merge action. Please ensure your PR has no deleted lines, and it will be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address:
Token Name:
Token Symbol:
Logo: (logo should be uploaded under assets/mainnet//*.<png/svg>)
Link to the official homepage of token:
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

Auto merge requirements
Your pull request will be automatically merged if the following conditions are met:

Your pull request only adds new tokens to the list. Any modification to existing
tokens will require manual review to prevent unwanted modifications.

Your pull request does not touch unrelated code. In particular, reformatting changes to unrelated
code will cause the auto merge to reject your PR.

Any asset files added correspond to the token address you are adding. Asset files
must be PNG, JPG or SVG files.

Your change is valid JSON and conforms to the schema. If your change failed validation,
read the error message carefully and update your PR accordingly.

No other tokens shares the same name, symbol or address.

For example, this change would be rejected due to unrelated changes:

The bot runs every 60 minutes and bulk-merges all open pull requests to prevent conflicts.
This means that you need to wait up to 60 minutes for your pull request to be merged or reprocessed.

I can help refer this https://github.com/zakimzf/profile/pull/1
PS: if you don't need my help.. please ignore this!

repo: solana-labs/token-list


text:
Add SolKanda INU Token

I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
This repository is managed using an auto merge action. Please ensure your PR has no deleted lines, and it will be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address:
Token Name:
Token Symbol:
Logo: (logo should be uploaded under assets/mainnet//*.<png/svg>)
Link to the official homepage of token:
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

Please follow the token-list README and ensure your token details adhere to the Uniswap Token Specification

authors: SolKandaINU Solana-Token


text:
Person should have a way to define preffered app authorization manager (also an app)

Managing app authorizations in many cases will require UI, preferably person can specify preferred app authorization management application as application with special role to manage authorizations for other applications. Applications which want to request authorizations would need a way to discover it and probably redirect user to approve / disapprove / partially approve.
Use case: discussion boards
I think approach proposed in  Privilege Request Protocol where each RS provides its own authorizations management UI leads to poor UX and may only stay required in some more restrictive deployments where RS admins needs higher level of control.

Right, there is a question on where the actual authorization should take place. Though I think if it's on the Identity Provider, it's fine to have that UI bound to the Identity Provider. If a user doesn't like it they can swap IDPs.

I don't see it necessary tied to OP / IDP in any way, possibly it should not involve OP / IDP at all. Person can store authorizations on RS of one's own choice, possibly something for Data Interop Panel.

This stems off of the idea that the webID should be kept with the IDP as well and basically boils down to "Should your application preferences be bound to your identity or somethine else." Currently it's very much bound to your identity as it's in the WebID (albeit implemented poorly). Not saying that giving the IDP responsibility over this is the right decision, but it's not a very large leap to take.

I agree that discovery will need to start from WebID Profile, just like discovery of OP starts there and follows solid:oidcIssuer predicate. Nowhere we restrict where the WebID Profile stays hosted, if one doesn't want to edit it with solid app it doesn't even have to stay on solid RS.

Similar to my comment on last authentication panel meeting, I think we should document roles and responsibilities just to make sure we all use the same terms in consistent way.

https://solid.github.io/data-interoperability-panel/specification/#authorization-agent
Authorization Agent is associated with the End-user similar to OpenID Provider. It handles authorizing clients (apps) and provides UI.
Early-stage open-source implementation:

https://github.com/janeirodigital/sai-impl-service/
https://github.com/janeirodigital/sai-impl-web/

authors: elf-pavlik jaxoncreed

repo: solid/authorization-panel


text:
Bugfix/fix attachment is parsed as wi link

Supersedes https://github.com/solidify/jira-azuredevops-migrator/pull/566

[x] Integration tests passing

repo: solidify/jira-azuredevops-migrator


text:
export store symbol $NODE

Summary
useful for more advance use cases.

make a prop readonly  / prop guards
extend to support a property with get/set  accessors

for example(requires patching, so wont work in playground)
https://playground.solidjs.com/?hash=-1235554887&version=1.4.1
https://playground.solidjs.com/?hash=657365322&version=1.6.0 (with [thetarnav] extractor)

Looks like I'm gonna need access to those symbols for the debugger, to track stores.
not sure yet though, just experimenting for now
But this is what I did to get to them:
const { $NODE, $NAME } = (() => {
  const obj = {}
  ;(createMutable(obj) as any).v
  let $NODE!: symbol
  let $NAME!: symbol
  for (const s of Object.getOwnPropertySymbols(obj)) {
    if (s.description === 'store-node') $NODE = s
    else if (s.description === 'store-name') $NAME = s
  }
  return { $NODE, $NAME }
})()

Yeah this is too low level I think. Necessary for Dev tools not really desirable as an API. We should focus on the gaps rather than expose this implementation detail at this time.

authors: LiQuidProQuo ryansolid thetarnav

repo: solidjs/solid


text:
Update scala-library to 2.13.12

About this PR
📦 Updates org.scala-lang:scala-library from 2.13.1 to 2.13.12
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
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!

🔍 Files still referring to the old version number
The following files still refer to the old version number (2.13.1).
You might want to review and update them manually.
.gitlab-ci.yml
project/build.scala

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-lang", artifactId = "scala-library" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scala-lang", artifactId = "scala-library" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, old-version-remains, commit-count:1

Superseded by #115.

repo: solidninja/schema-registry-sttp-client


text:
Internationalization?

I tried installing https://github.com/solidusio/solidus_i18n and adjusting the language in the backend, but that did not do anything so far :/

Ah, if I offer only one language I need to set it in the config, the backend setting only works with multiple languages it seems...


text:
Support --password-stdin, similar to other OCI tools

When using repositories like AWS ECR, the authentication expires quite rapidly and the generated passwords tend to be quite long. Other extant OCI style tools, such as the docker utility, support taking in a password from stdin, allowing the developer to skip a step of making a temporary file or copying the password to the clipboard.
Using --password-stdin will match up with the docker and podman utilities, and coincide well with AWS's instructions on how to use their ECR service.
Before:
aws ecr get-login-password --region us-west-2 > .ecr_pw
<cat or copy really_long_pw to clipboard>
wasme login -s <aws_ecr_url> -u AWS -p <paste really_long_pw>

After:
aws ecr get-login-password --region us-west-2 | wasme login -s <aws_ecr_url> -u AWS --password-stdin

Consequently because wasme is using the oras library, the temporary workaround I have here is to log in via the oras command line tool before attempting wasme push or wasme pull commands.
EG:
aws ecr get-login-password --region us-west-2 | oras login -u AWS --password-stdin <aws_ecr_url>
wasme push <aws_ecr_url>/image:<version>

This works because oras writes out login information to  ~/.docker/config.json, which the library also reads from by default.

repo: solo-io/wasm


text:
Is cloc giving lines or files?

in the code cloc is said to return lines but on the plot its files

I can't imagine these actually being confused, even with the log scale graph. but I'll check

These are not mutually exclusive, it counts both. Not an issue

the size of the sphere is lines

As long as we're commenting on dead issues they're circles not spheres

repo: somethingsoftware/git-stats


text:
Design main docs page

Right now it is a bit empty:

Cross posting from the search results page/search page discussion. Proposed layout for the docs landing page:

@ststimac @qzhou1607 is this done?

This the Docs landing page so if that's merged it should be complete.

Confirmed that this has been merged so I will close.
Making a minor style edit in a PR to fix an alignment issue.

authors: molant ststimac


text:
Make minor consistency changes

Remove extra unnecessary call from rule packages.
Use more consistent naming convention for files.

@sarvaje are those changes going to affect the generation of the index page in https://github.com/sonarwhal/sonarwhal.com/pull/406 ?

@sarvaje are those changes going to affect the generation of the index page in sonarwhal/sonarwhal.com#406 ?

At least for rules it shouldn't be a problem. We are using just the documentation files.

authors: alrra molant sarvaje

repo: sonarwhal/sonarwhal


text:
Cargo on crates.io is old and broken

You may have forgotten to push the new version (0.0.6) to crates.io.

Yeah, crates.io haven't been up to date with the repository at all time.
I published 0.0.6 now :+1:

authors: alexchandel sondrele

repo: sondrele/rust-bmp


text:
四人赛无法点击选项

软件版本信息

学习强国版本: [e.g. 2.33.0]

描述BUG
四人赛无法点击选项
详细出BUG的步骤
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

截图和日志报错信息
If applicable, add screenshots to help explain your problem.

还有多的日志不，看不到错误在哪一行


text:
如何绑定[令牌]和[渠道]，以实现客户端相同模型，实际映射成不同模型

例行检查

[x] 我已确认目前没有类似 issue
[x] 我已确认我已升级到最新版本
[x] 我已完整查看过项目 README，已确定现有版本无法满足需求
[x] 我理解并愿意跟进此 issue，协助测试和提供反馈
[x] 我理解并认可上述内容，并理解项目维护者精力有限，不遵循规则的 issue 可能会被无视或直接关闭

*功能描述
不同客户端，输入同一模型，不同令牌，命中与令牌绑定的指定渠道，实际映射到不同模型。
应用场景

客户端1模型处只能填入[gpt-3.5]，简单应用，需要映射使用[qwen-plus]模型
客户端2模型处也只能填入[gpt-3.5]，复杂应用，需要映射使用[qwen-max]模型

one api 配置
【令牌1】->【渠道1】{“gpt3.5”; "qwen-plus"}
【令牌2】->【渠道2】{“gpt3.5”; "qwen-max"}
客户端1填写令牌1，gpt3.5，映射到qwen-plus
客户端2填写令牌2，gpt3.5，映射到qwen-max

【令牌-渠道id】

【令牌-渠道id】

嗯，看到其他issue里面提到过了。 “管理员用户可以用  【令牌key-渠道id] ， 实现令牌和渠道的绑定"。 这里已经可以实现我的诉求了。
好奇问一嘴，那普通用户呢？

普通用户没有渠道可以选啊，渠道本身也是管理员可见的内容

authors: Cool-PKY QAbot-zh

repo: songquanpeng/one-api


text:
fix: 修复高并发下，高额度用户使用低额度令牌没有预扣费而导致令牌大额欠费

修复高并发下，高额度用户使用低额度令牌没有预扣费而导致令牌大额欠费

好的，请先等我重构完成 relay 部分哈，之后的冲突我来解决

authors: Calcium-Ion songquanpeng


text:
How do I reference texture when loading .obj to the decimator?

Hi @songrun, thanks for the work!
I'd like to know how to reference a texture .jpg file as well as .mtl material file when loading in an .obj file to the decimator.
Thanks!

I'm also interested in this

repo: songrun/SeamAwareDecimater


text:
check if mounted before calling setState

https://github.com/sooxie/list_treeview/issues/5
The above issue was still not fixed so I implemented the proposed little change.

Okay, thank you

authors: CaptainArni sooxie


text:
ENFORCE(!symbolTableFrozen) in GlobalState::delete*

Motivation

We should have always had these ENFORCEs, to parallel the enter*
methods.
Test plan

Existing tests

Current dependencies on/for this PR:

master

PR #6602   👈

This comment was auto-generated by Graphite.

repo: sorbet/sorbet


text:
Mostly remove vector references from core

Switch more callsites to using spans, and remove vector references from some structures.
Additionally, add a comment into LexNext::operator< that describes how it differs to std::lexicographic_compare.
Motivation
Using absl::Span in place of vector/InlinedVector references in function arguments allows them to accept anything that converts to a span, and also avoids the possibility of accidentally copying vectors.
Test plan

n/a

and also avoids the possibility of accidentally copying vectors

Isn't this only true for vector references in return types?
In argument types, it's not clear to me that this is actually faster–instead of just being able to have a pointer to a vector, now you have to construct an absl::Span (which might not have been in scope already) and which is going to take two machine words instead of just one machine word, which depending on how many arguments the function takes, might mean that we run out of registers to pass arguments in.
I'm fine to merge this anyways, but I'd like to be sure that we're doing this mostly for aesthetic reasons, not performance.

Isn't this only true for vector references in return types?

Yes, though this also prevents any accidental copying in the body of the function that's given that reference. You're right that we'll take two machine words for span arguments, but for functions that have spare register arguments available anyway, this will be a win. I can go back and try to determine if any functions I've modified will spill over into using the stack as a result of using a span, if you like.
For structures that hold vector references, especially ones whose values that don't outlive the frame they're allocated in, I feel like spans are a pretty unambiguous win because we avoid the cost of indirection through the reference to the vector. DispatchArgs for example will avoid one indirection to access the args by using a span.
The cost of constructing a span will be on par with the first use of the vector in the function that we call, so I'm not too worried about the overhead there: we amortize it with a span instead of relying on CSE later to make it cheap.
Overall, I think the win here is making it harder to accidentally copy vectors, but I do also believe that there are some small perf gains possible as well.

authors: elliottt jez


text:
Squeeze instead of broadcast

seems more "lean" to squeeze than broadcast
the downside is that it's more lines of code

I'm merging this because I need it for the next thing I'm working on, but please speak up if you have any suggestions how to do it better and I can change it

Whoa, that's awesome! I didn't realize that squeezing would be equivalent to broadcasting.
(Sorry that I missed this PR! FWIW, it looked great.)

authors: shawwn sorenlassen

repo: sorenlassen/einsum-experiment


text:
Can there be a way to have pmodload look for modules somewhere other than ZDOTDIR/HOME?

It would be nice to be able to be able to store modules in places other than ~/.zprezto/, or better yet to store the entire .zprezto folder somewhere else, and maybe even give it a different name.
Right now the tree must look like this:
~
|-- .zshrc
|-- .zpreztorc
|-- .zprezto
    |-- modules
        |-- helper
        |-- git

It would be nice if I could configure Prezto to use this instead:
~
|-- .zshrc
|-- .zpreztorc

/usr/local
|-- prezto
    |-- modules
        |-- helper
        |-- git

or even
~
|-- .zshrc
|-- .zpreztorc
|-- .antigen
    |-- https-COLON--SLASH--SLASH-github.com-SLASH-sorin-ionescu-SLASH-prezto.git
        |-- modules
            |-- helper
            |-- git

I think this would be a good feature for a few reasons:

You could store all of the machinery inside /usr/local/prezto or something like that, leaving the home directory solely for configuration files.
It would make it easier for users to write their own modules without forking the entire Prezto project.
It would allow users of programs like Antigen to isolate sections of Prezto. Right now Antigen is getting pmodload to work from inside its own .antigen directory by permanently changing ZDOTDIR. See zsh-users/antigen/issues/137

Duplicate of #761, #271.

authors: gwerbin sorin-ionescu

repo: sorin-ionescu/prezto


text:
UI

shell
commands
tutrial
extensions

polymer で array プロパティの削除
https://www.polymer-project.org/1.0/docs/devguide/model-data#notifysplices

repo: sosuke-k/pandler


text:
Multiple Objects

This is most likely a newbie question, so please forgive me. I'm trying to use this script to make all of my images (".fleximg") fit into their corresponding divs (".block).
var foo = document.querySelector('.block');
var bar = document.querySelector('.fleximg');

fit(bar, foo, {
    hAlign: fit.CENTER,
    vAlign: fit.CENTER,
    cover: true,
    watch: true,
    apply: true
});

When I run the code, it only adjust the first image and ignores the rest. Any idea why this may be happening, I'm sure I'm misunderstanding the code.

In case anyone else is trying to figure this out, I pushed through and found a solution:
window.onload = function() {

	var examples = document.querySelectorAll( '.pic-wrapper' );
	var i, n, el;

	for(var i = 0, n = examples.length; i < n ; i++ ) {

		el = examples[i];

        foo = el.querySelector( '.block' );
        bar = el.querySelector( '.fleximg' );

		fit(bar, foo, {
		    hAlign: fit.CENTER, // or fit.LEFT, fit.RIGHT
		    vAlign: fit.CENTER, // or fit.TOP, fit.BOTTOM
		    cover: true,
		    watch: true,
		    apply: true
		});

	}

};

There may be some minor issues with my code, I'm a javascript lightweight.

Hi @nate8684,
Glad you figured it out. You're right, querySelector would just select the first matching element so using querySelectorAll and applying fit to all matches is the way to go.

license: WTFPL


text:
I have a suggestion about xpath

Hi, how if you make xpath code more flexible, not based on position but by search.
something like this :
"//*[contains(text(), 'Get Reward')]"
instead of this :
/html/body/div[1]/div[2]/main/div/div/div[3]/div[1]/div[2]/div[2]/div[1]/div/div/div[2]/button

i've changed some of your xpath with that method, and it works better. i think, you can try to use some function like parent and child to get where the elements exact location.
It will take sometime to modify, but it will works like charm, even when the website updating itself.
and maybe you want to try to make bot for autodutchycorp? the logic is actually nearly the same with faucetcrypto. but its generating more money.
Thanks for your hardwork, even you say this is your first bot, but i think its a excellent bot!

Hi mate tnx for the info I'm pretty new to selenium and automation so I'm not well versed enough, I'm also not maintaining this bot anymore cause of huge workloads in my workplace this was more of a side project so I don't know if I will update it in the future looking for contributers currently who can improve the bot.

i've changed some of your xpath with that method, and it works better. i think, you can try to use some function like parent and child to get where the elements exact location.
It will take sometime to modify, but it will works like charm, even when the website updating itself.
and maybe you want to try to make bot for autodutchycorp? the logic is actually nearly the same with faucetcrypto. but its generating more money.
Thanks for your hardwork, even you say this is your first bot, but i think its a excellent bot!

Could you share your xpath file @oyi77


text:
Update the FixUp retry instruction text

The retry instruction text has the command attached, which recursively adds to each retry. This PR fixes this issue that in case of any fixup task, only the previous instruction is used/shown in the quick pick.
Test plan

Tested manually.

Before behaviour:
https://github.com/sourcegraph/cody/assets/44617923/70daecc9-bf38-4dd5-bca9-271367f4441c
After:
https://github.com/sourcegraph/cody/assets/44617923/1916df2e-b616-4f1c-96a6-c7144844759d

Thanks! Left a small comment but looks good!

@umpox Made the requested changes. I think it's all good now.

repo: sourcegraph/cody


text:
panics: move PanicCatcher to separate package

Proposal: move conc.PanicCatcher to a subpackage, conc/panics.Catcher. We cannot use panic as a package name without conflicting with the builtin panic().
IMO it is a bit strange for PanicCatcher to be part of the top-level conc package, since:

Most users of conc.WaitGroup, conc/pool, etc will never interact directly with panic catcher
While PanicCatcher can be used multiple times and concurrently, it only ever retains a single panic, so regardless of in a standalone usecase I imagine one might use an instance of PanicCatcher for one goroutine at a time to get more granular RecoveredPanics, so it doesn't really fit in with the theme of conc, and is more of a utility.

Codecov Report

Merging #38 (ea3827f) into main (f821312) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #38   +/-   ##
=======================================
  Coverage   99.44%   99.44%
=======================================
  Files          10       10
  Lines         358      358
=======================================
  Hits          356      356
  Misses          2        2

Impacted Files
Coverage Δ

waitgroup.go
100.00% <ø> (ø)

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

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

pool/context_pool.go
100.00% <0.00%> (ø)

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

authors: bobheadxi codecov-commenter

repo: sourcegraph/conc


text:
Add Engineering Ownership to Principles & Practices

This is based on the "Engineering Ownership" document I shared a few weeks ago.
Feedback on that document has been positive and encouraging, but quite a few of you asked: what's the next step? how can we make sure everybody in the company lives this? I do not know, but I do think that having it in the handbook is a great next step. So here we are.
Some of the Google Docs comments you all left I tried to incorporate, at least where I thought it was possible. There's still more comments in that doc that should be recorded somewhere (i.e. about how to do cross-team collaboration, or how to find out who owns what), but I don't think they fit into this document here (as part of "Principles & Practices").

This stuff is gold, LGTM 🥇

authors: jhchabran mrnugget

repo: sourcegraph/handbook


text:
RFE: team_number does not yet support the optional team argument

Avg response time

Request for Enhancement (RFE)

OpenCoarrays Version: 2.0.0
Fortran Compiler: GNU Fortran (GCC) 8.0.1 20180225 (experimental)
C compiler used for building lib: gcc (GCC) 8.0.1 20180225 (experimental)
Installation method: install.sh
Output of uname -a: Linux 3.13.0-144-generic #193-Ubuntu SMP Thu Mar 15 17:03:53 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
MPI library being used: MPICH 3.2.1
Machine architecture and number of physical cores: laptop computer, 4 cores
Version of CMake: 3.4.0

Observed Behavior
With OpenCoarrays 2.0.0, the Fortran 2018 intrinsic function team_number does not yet support the optional team argument. This should be a known issue because the runtime does give an appropriate runtime error message: Fortran runtime error on image 1: team_number does not yet support the optional team argument. Example program:
program Main
! please compile and run with 15 coarray images
  !
  use,intrinsic :: ISO_FORTRAN_ENV, only: team_type
  implicit none
  !
  integer :: intTeamNumber
  type (team_type) :: MainTeam
  type (team_type) :: SubTeam
  !
!
!************************************************
! MainTeam **************************************
!************************************************
!
! split the 15 images of the initial team into 3 child teams consisting of 5 images each:
  if (this_image() <= 5) then
    intTeamNumber = 1
  else if (this_image() >= 6 .and. this_image() <= 10) then
    intTeamNumber = 2
  else if (this_image() >= 11) then
    intTeamNumber = 3
  end if
  !
form team (intTeamNumber, MainTeam) ! this creates the 3 child teams in MainTeam
!
change team (MainTeam)
!
! important: the following gets executed on the images of all the 3 newly created child teams:
!
!************************************************
! SubTeam ***************************************
!************************************************
!
! split the 5 images of the MainTeam's child teams into two distinct teams resp.:
  if (this_image() < 3) then
    intTeamNumber = 1
  else
    intTeamNumber = 2
  end if
!
form team (intTeamNumber, SubTeam) ! this generates 6 teams (2 new sub teams in the 3 main teams resp.)
                                   ! within the change team (MainTeam) construct
change team (SubTeam)
!
SubTeam_select: select case (team_number())
!
case (1) SubTeam_select
  ! codes for child team 1 of SubTeam:
  write(*,*) 'image number: ', this_image(), ' of team number: ', team_number(), &
             ' in SubTeam, of MainTeam''s child team number:', team_number(MainTeam)
              !Fortran runtime error on image 1: team_number does not yet support the optional team argument
  !
!*******************
case (2) SubTeam_select
  ! codes for child team 2 of SubTeam:
  write(*,*) 'image number: ', this_image(), ' of team number: ', team_number(), &
             ' in SubTeam, of MainTeam''s child team number:', team_number(MainTeam)
              !Fortran runtime error on image 1: team_number does not yet support the optional team argument
!
end select SubTeam_select
!
end team !(SubTeam)
!
!*************
!*************
!*************
!
end team !(MainTeam)
!
end program Main

Expected Behavior
Just to give an indication of the importance of the optional team argument:
As the above example program can already show (it does not fail if compiled with team_number not using the optional team argument), if we execute the same form team statement on all images within a change team construct (that comprises a set of teams), that single form team statement does not only form one set of new teams but instead can form several sets of new teams: For each team in MainTeam a new set of teams is created in SubTeam; Since MainTeam comprises 3 teams, and the form team statement in SubTeam does form 2 new teams, that single form team statement does actually form 6 new teams.
With such nested teams, the team argument should allow to use the team_number intrinsic to get information about the nesting and the belonging of the images of a nested team with respect to their parent team. This could be helpful for the development of new kinds of parallel algorithms later on.

@MichaelSiehl Thanks for submitting this issue and especially for the code example. If you have any interest, the best way to get this feature added is to dive in.  Going forward, OpenCoarrays is going to be mostly a community effort with only small amounts of related project funding.  In case it helps, I have long considered the possibility of replacing some of the C code in OpenCoarrays with Fortran code as a way to increase the likelihood that the Fortran community will contribute code to OpenCoarrays.  As a simple example of what this would look like, take a look at the partial implementation of this_image() in Fortran inside the opencoarrays module:
  ! Return the image number (MPI rank + 1)
  function this_image()  result(image_num)
#ifdef MPI_WORKING_MODULE
    use mpi, only : MPI_Comm_rank
#endif
    integer(c_int) :: image_num,ierr
   !image_num = opencoarrays_this_image(unused)
    call MPI_Comm_rank(CAF_COMM_WORLD,image_num,ierr)
    if (ierr/=0) call error_stop
    image_num = image_num + 1
  end function

The commented line makes a call to the C implementation of this_image().  It's commented because I had originally planned for the above procedure to wrap C, but had trouble getting that to work and so decided that the function is suffiiciently simple that I just wrote it in Fortran rather than wrapping it.  A similar approach could be tried with team_number().  If we go this route and also give the procedure the bind(C) attribute, then some of the C procedures could eventually be replaced with Fortran procedures without any impact on user code.  Would this make it more likely that you would consider implementing the requested feature?

Hi Damian,
the ability to use Fortran for implementing new features sounds great and
yes, it could make it more likely to try implementing features by myself
(without any guaranty of success yet ;-). Still, the team_number intrinsic
function (as well as num_images, which does not support team variables at
compile time yet) is not on top of my wishing list of missing features. (We
can circumvent the missing optional team argument easily by using a local
variable to store team values – since using any of the team-related
statements has no impact to the values of local variables). There are
probably other missing features (e.g. allocatable derived type coarrays at
the team level) that I would consider to be more important yet, but still
NO single missing feature that does prevent from highly sophisticated use
of the Fortran 2018 team features with OpenCoarrays yet.
Further, and considering that upcoming ifort 19 (beta yet) will not already
support Fortran 2018 team features, I do suppose that OpenCoarrays will
continue stay way ahead for quite a long time.
Furthermore, right now I am full of enthusiasm about the working team
features and will prefer to focus on the working features and do further
research using them. I did already succeed with encapsulating F18 team
features into a (yet simple) customized create_team procedure (similar to a
customized synchronization procedure). At this early stage of testing, I
can not fully tell what the opportunities of this will be, but do still
believe that we can easily increase the possibilities of the basic (but
powerful) team features of Fortran 2018 using current OpenCoarrays already.
If things work out well, I will give working example codes in the coming
weeks and month.
My Best Regards
2018-04-25 23:32 GMT+02:00 Damian Rouson notifications@github.com:

@MichaelSiehl https://github.com/MichaelSiehl Thanks for submitting
this issue and especially for the code example. If you have any interest,
the best way to get this feature added is to dive in. Going forward,
OpenCoarrays is going to be mostly a community effort with only small
amounts of related project funding. In case it helps, I have long
considered the possibility of replacing some of the C code in OpenCoarrays
with Fortran code as a way to increase the likelihood that the Fortran
community will contribute code to OpenCoarrays. As a simple example of what
this would look like, take a look at the partial implementation of
this_image()
https://github.com/sourceryinstitute/OpenCoarrays/blob/master/src/extensions/opencoarrays.F90#L760
in Fortran inside the opencoarrays module
https://github.com/sourceryinstitute/OpenCoarrays/blob/master/src/extensions/opencoarrays.F90
:
! Return the image number (MPI rank + 1)
function this_image()  result(image_num)
#ifdef MPI_WORKING_MODULE
use mpi, only : MPI_Comm_rank
#endif
integer(c_int) :: image_num,ierr
!image_num = opencoarrays_this_image(unused)
call MPI_Comm_rank(CAF_COMM_WORLD,image_num,ierr)
if (ierr/=0) call error_stop
image_num = image_num + 1
end function
The commented line makes a call to the C implementation of this_image().
It's commented because I had originally planned for the above procedure to
wrap C, but had trouble getting that to work and so decided that the
function is suffiiciently simple that I just wrote it in Fortran rather
than wrapping it. A similar approach could be tried with team_number().
If we go this route and also give the procedure the bind(C) attribute,
then some of the C procedures could eventually be replaced with Fortran
procedures without any impact on user code. Would this make it more likely
that you would consider implementing the requested feature?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sourceryinstitute/OpenCoarrays/issues/524#issuecomment-384441118,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQUQ_F1cgIoQ0WB4djUXmBxvMqCXBPziks5tsOtjgaJpZM4Tigxr
.

Fixed in #631 and in release 2.5.0. Thanks @nathanweeks for implementing the fix!
And thanks to @MichaelSiehl for reporting the bug!

authors: MichaelSiehl rouson zbeekman

repo: sourceryinstitute/OpenCoarrays


text:
RHEL9 - Unable to load lvm attributes [lvs.yaml] for version [2.03.16(2)].

:speaking_head: Foreword
Thank for taking the time to fill this bug report fully. Without it we may not be able to fix the bug, and the issue may be closed without resolution.
:ghost: Brief Description
Getting this error all RHEL 9 lvm2 version 9:2.03.16-3.el9
Unable to load lvm attributes [lvs.yaml] for version [2.03.16(2)].
:pancakes: Cookbook version
lvm (6.1.2)
:woman_cook:  Chef-Infra Version
Chef Infra Client, version 17.9.46
:tophat: Platform details
Operating system distribution and release version. Cloud provider if running in the cloud.
Red Hat Enterprise Linux release 9.1 (Plow)   Azure
Steps To Reproduce
Steps to reproduce the behavior:

lvm_physical_volume[/dev/sdc] action create[2022-11-22T11:26:48-06:00] INFO: Processing lvm_physical_volume[/dev/sdc] action create (hrb_rh_appdata_cb::add_appdata line 24)
================================================================================
Error executing action create on resource 'lvm_physical_volume[/dev/sdc]'
ArgumentError
Unable to load lvm attributes [lvs.yaml] for version [2.03.16(2)]. The version/object may not be supported or you may need to upgrade the chef-ruby-lvm-attrib gem. Error [No such file or directory @ rb_sysopen - /opt/chef/embedded/lib/ruby/gems/3.0.0/gems/chef-ruby-lvm-attrib-0.3.11/lib/lvm/attributes/2.03.16(2)/lvs.yaml]

:police_car: Expected behavior
create an lvm
:heavy_plus_sign: Additional context
If we downgrade lvm2 version to 9:2.03.14-4.el9 it works.
upgrade to lvm2 latest version ( 9:2.03.16-3.el9 ) it fails

Logged this against the LVM GEM: https://github.com/sous-chefs/chef-ruby-lvm-attrib/issues/44

repo: sous-chefs/lvm


text:
[#465] Create composer version of WaterfallSkillBotDotNet - FileUpload scenario

Addresses # 465
Note: This PR depends on # XXX (SW 47) containing the base implementation of the ComposerSkillBotDotNet.
Description
This PR adds the File Upload scenario to the ComposerSkillBotDotNet bot.
Detailed Changes

Added the FileUploadDialog dialog to implement the FileUpload scenario.
Added the CustomAction DownloadFileDialog to handle the download of the file to the temp folder.
Added DownloadFileDialog as a dependency in ComposerSkillBotDotNet project.

Testing
These images show the WaterfallHosBot connecting with the ComposerSkillBotDotNet and running the FileUpload scenario.

Replaced with #53

repo: southworks/BotFramework-FunctionalTests


text:
Traders restock times does not match Live

General information
Project issue is for:
Server
Expected result:
On Live, each trader has its individual restock time, roughly in the range of 1 to 3,5 hours
Actual result:
All traders have 1hr restock time

Originally written by 5o2

Made some changes to trader restock times: https://dev.sp-tarkov.com/SPT-AKI/Server/commit/f623b1d52e6b7e163a8253da59f68ed3ab08e185
We're gathering restock time data from live, it seems the values vary considerably. There may be further adjustments at a later date.

Originally written by chomp

Something to keep in mind, because of the high variability of trader restock times in live (to stop bots from buying out stocks) making SPT match the times exactly is going to be impossible. We can however ensure the length between restocks is more accurate instead of all being one hour.

Originally written by chomp

I should have a solid set of ranges sometime Monday.

Originally written by Refringe

Based off of the data I've collected so far, here's what I would recommend for the refresh ranges (in seconds):

Jaeger - 2000-5500
Ragman - 6500-15000
Lightkeeper - 3000-7500
BTR Driver - 3000-7500
Therapist - 3000-9500
Skier - 5000-8500
Fence - 3000-9000
Peacekeeper - 2000-14000
Prapor - 7000-13500
Mechanic - 7000-13500

I think this should get us 90% of the way there, but I'll continue to collect times.

Originally written by Refringe

Made changes: https://dev.sp-tarkov.com/SPT-AKI/Server/commit/95f5a49c5c30b1fa4016236d5926f90aeffe0230

Originally written by chomp

I'm going to make the following (small) changes:
Skier - 5000-9000
Peacekeeper - 2000-15000
Mechanic - 6500-13500

With over 600 data points collected, a pretty clear picture has emerged:

After these changes are made we will be close enough to live to close the issue.

Attachments

trinket_plot.png (67 KiB)

Originally written by Refringe

Minor time changes have been made in 662575aef1.

Originally written by Refringe

repo: sp-tarkov/server

license: NCSA


text:
[Maps] Astra NoERP Update

Описание PR
На корваксе секса нет

удалена БДСМ комната из техов
удалена комната для собраний ЛГБТшников
исправлены слепые зоны камер
исправлена труба лифта
изменены техи

Слон

authors: kosticia lastPechkin


text:
You can use welder even when they have no fuel now

Reproduction

Empty a lit welder out of fuel
Click something weldable
It welds

Additional context
@deltanedas a

bruh i thought i checked for it

authors: AJCM-git deltanedas


text:
Should hair be layered above held items?

Potential ability to powergame with massive hair aside

it should change depending on the dir of the person.
South facing? yes
North facing? no
East/West? no


text:
Butcher verb now only visible when wielding a sharp object

About the PR
Butcher verb will now only be visible when wielding a sharp object. This might make it slightly harder for newbies to discover how to do butchering, but personally I like keeping the verb menu as short as possible. Discuss.
Additionally, butchering non-living things dosen't generate such an intrusive message.
Changelog

:cl:

tweak: Butcher verb will now only be visible when wielding a sharp object

i’d prefer if this wasn’t merged.
If not for the ‘butcher’ verb being visible on clothes, i would never have known how to obtain cloth from them.
I wouldn’t have even known to ask other crew about where I could get cloth!

this was intentionally made to be shown at all times. imo more verbs should do this, its way better for discovery

it doesnt make it 'slightly harder to discover', it makes it basically impossible

Yeah, especially since the logic of what is "sharp" isnt always clear. I'd rather options be always shown to the player. To utilize said options may require searching or inventive thinking, but the options shouldn't be hidden.

I suppose discoverability is already a big issue that I shouldn't go exacerbating. Makes sense.

authors: EmoGarbage404 Mervill civilCornball mirrorcult


text:
[New Food] Pelmeni

About the PR

Media

https://user-images.githubusercontent.com/100022571/234249394-84fd734d-1c9c-4ffb-b65d-7bb1a49e2f37.mp4

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

Changelog

:cl: SonicDC

add: Added pelmeni!

whys the video from corvax?

There is a current freeze on new recipes: https://github.com/space-wizards/space-station-14/issues/8524

@SonicHDC you didn't have to close the PR. You can keep it open.

@SonicHDC when the chef rework is done please feel free to reopen this

Just make it dumplings as they're significantly more popular than this.

This pr is closes.
Recreated to https://github.com/space-wizards/space-station-14/pull/16286

authors: SonicHDC keronshb metalgearsloth potato1234x


text:
emagging ame fuel makes it leak antimatter (explode)

About the PR
title
it will leak all its fuel over the span of 10 seconds with a special antimatter explosion
antimatter explosion is:

blue like the jar
cool
does 5 extra burn compared to normal explosion

when it explodes it doesn't do much damage but it will be flung around randomly
it can potentially space tiles and destroy things like the ame if it explodes on it enough
Why / Balance
allows for more syndicate plays and creating some chaos
Technical details
added leak explosion fields to AmeFuelContainerComponent
when its emagged it will start exploding until there is no more fuel, then it gets unemagged
since ame fuel is serverside there is no popup, networking it seems like too much effort for just this small addition
Media
https://github.com/space-wizards/space-station-14/assets/39013340/69450526-3465-4933-97dd-27dd14515fa5

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

Breaking changes
no
Changelog
:cl:

add: AME fuel can now be emagged to break its seal and explode its antimatter.

id kind of rather this was just a normal thing you could do (no emag) but make the explosions not able to space

Making station sabotage exclusive to the emag is probably a bad idea so I’d rather not see that be the case.

id kind of rather this was just a normal thing you could do (no emag) but make the explosions not able to space

if it was available to selfantags people could just leak all the ame fuel roundstart

if it was available to selfantags people could just leak all the ame fuel roundstart
unless you mean like beating it to 100 damage instead of a verb like gas tank

I'd be something deliberate like exploding the AME currently is so

it doesnt need to be as accessible as a verb

it now breaks above 100 damage
still can be emagged and uses emag logic tho

I still don't see the reason for adding EMAG to the equation. either give it valve like gas canisters or burst on sufficient damage.

yeah i dont think emag should be involved in this at all, its extremely unintuitive and its not like AME jars even have any electronics to hack

Well there is such thing as solenoid valve. But those usually have remote control in mind.

@mirrorcult should i have a bool and special destruction behavior to set it to true, or is there a more suitable thing that already exists

also fun fact you can chain c4 and antimatter :trollface:

authors: LankLTE deltanedas drteaspoon420 mirrorcult


text:
CMO Harsuit Change

About the PR

I slightly buffed CMOs Hard Suit
Why / Balance

I feel like CMOs suit is kinda weak looking at the fact that medbay gets attacked pretty frequently so i decided to buff it slightly
Technical details

Just added resistances
Media

[x] 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:

tweak: Made CMO's  hardsuit slighty stronger.

a non-combat hardsuit with some of the best movement in the game does not additionally need to be armor-clad. The CMO can manage just fine by staying out of harm's way. They aren't supposed to be engaging in combat or any other activity that would warrant armor.

If medbay is getting bombed that is SECURITY's job to prevent, not the CMO's.

this isn't really accurate, it's the CMO'S job to prevent medbay bombings by not emergency accessing the front doors and turning medbay into an open-air treatment warehouse :trollface:

CMO doesn't need damage resistances, they're not supposed to be fighting. They already have one of the lowest speed reductions of any hardsuit in the game and probably the best caustic resistance of any armor piece period. If medbay is getting bombed that is SECURITY's job to prevent, not the CMO's.

Not supposed to be fighting but with frequent medbay attacks i belive the few additional resistances would be usefull for defense

authors: EmoGarbage404 GreyMario MrToofters


text:
Picking a ghostrole as an admin will now deadmin you.

About the PR

Picking a ghostrole as an admin will now deadmin said admin.
Why / Balance

to fix #29500
Technical details

When a player picks a ghost role this will check if they are an admin, and if they are it will deadmin them.
Media

https://github.com/space-wizards/space-station-14/assets/22904993/52d06ad4-8721-4359-b7a5-79278d47bda2

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

Breaking changes

I ended up adding a dependency field so that I could use the IsAdmin and DeAdmin methods. I'm still a beginner when it comes to code so please let me know if this will cause issues.
Changelog

:cl: MFMessage

fix: Picking a ghost role as an admin will now deadmin.

Is there a way to implement a check that the admin in question isn't connected to localhost so I can still do admin stuff to debug in my test environment?

Is there a way to implement a check that the admin in question isn't connected to localhost so I can still do admin stuff to debug in my test environment?

The easiest way I can think of Is to just throw a return within a DEBUG block but there might be a better way.

authors: Ghagliiarghii IProduceWidgets MFMessage


text:
Fully Revert Clown Waddling

About the PR

Fully reverts waddling, revival of #29161
Why / Balance

Requested by Sloth here and further approved of here.  A couple other people in the discord, including a game admin, also requested this.

The original PR was closed by the creator deleting their head repository, not by any consensus.

Waddling as implemented is entirely broken, the original creator is no longer working on SS14, and it needs reimplementing according to Sloth's reviews (something entirely above my skill level). I am sad to see it go, but as-is it makes the game look and feel worse as a clown.
Technical details

Reverts  #26338, #26965, #26983, and #27628
Media

Requirements

[x] I have read and am following the Pull Request and Changelog Guidelines.
[x] I have added media to this PR or it does not require an ingame showcase.

Changelog

:cl:

remove: Removed clown waddling until implemented properly

1984

Yeah, this is neccessary, waddling has too many bugs.
Could you mention the removed components and systems in the breaking changes section of your PR?

as said the original author will not work on this, so waddling will be gone for at least a year before anyone cares enough to rework it all

Yeah, this is necessary, waddling has too many bugs. Could you mention the removed components and systems in the breaking changes section of your PR in case forks use them for something else?

should be good now

In case someone wants to work on a proper reimplementation, some of the requirements are mentioned here: #29156

authors: Golinth deltanedas regenpower slarticodefast


text:
Made plasma grindable

About the PR
Made plasma grindable
Screenshots

Changelog

:cl: Seth

add: Added Grindable Plasma

Why is paper grindable seth

yeah the grindable tag should be added to just plasma, adding it to the parent makes paper grindable

Why is paper grindable seth

sure I can change that but why shouldnt paper be grindable?

because it has no solution container

Why is paper grindable seth

sure I can change that but why shouldnt paper be grindable?

A. Because your PR isn't "make paper grindable" it's "make plasma grindable"
B. You added it to the parent. Don't.

it works fine bb (:

You put it on the wrong sheet...

I’m pretty sure I putted it on right sheet dude I tested it and it works with plasma I can see the code

bruh

bruh

Yeah that’s the right one I was told by Tim rod i asked him if it was right because it should probably be plasma ore and not plasma sheet but he told me that no plasma sheet is this plasma I tested it and he was right that’s the right one

Screenshot me where he told you this

Screenshot me where he told you this

I did, because I assumed that plasma ore was the mining material. I apologize if this was incorrect, though I admitted it was mostly just a hunch at the time.

Why have you not put this on the base plasma sheet Seth?

https://hackmd.io/@ss14/yaml

https://hackmd.io/@ss14/yaml
It's beyond stupid that sloth has to baby you through the very basics of yaml in your own PR. Please do not make anymore until you've grasped at least the fundamental concepts of yaml.

How was I suppose to know there was a special thing for plasma ore what is it even for?
I even tested it with all the plasma and I grinder well?

All the plasma on saltern at roundstart is probably SheetPlasma1. This does not mean it is the only plasma spawnable, hence its parent that was pointed out to you 3 times.
This has literally NOTHING to do with plasma ore and at no point have I said it does.
Which part of its parent being the actual base SheetPlasma is causing confusion?

authors: SethLafuente SweptWasTaken TimrodDX metalgearsloth mirrorcult


text:
Adds a UI for gas mixers

About the PR
See #4203
Adds a UI for mixers.
Screenshots

Changelog

:cl:

add: Gas mixers now have a UI
fix: Gas filters and pumps now need to be anchored to open their UIs

CBA to do node directions but I did improve it. It's more immediately clear than using dirs anyways.

The variable names are consistent with the actual nodes in the code being node one & two. Primary/side is just to make it clearer for players.

Linter failure is unrelated to the PR.


text:
feat: add functions for operating on polynomials (PROOF-913)

Rationale for this change
Add functions for operating on polynomials.
These will be used for sumcheck proofs and verifications.
What changes are included in this PR?
Add a new component polynomial_utility with polynomial operations.
Are these changes tested?
Yes

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

authors: SxT-Release rnburn

repo: spaceandtimelabs/blitzar


text:
feat: add example curve for testing (PROOF-642)

Rationale for this change
Add a minimal class that implements the curve concept to support testing.
What changes are included in this PR?
Adds element97 curve class.
Are these changes tested?
Yes

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


text:
feat: add element concept to base field package ( PROOF-748 )

Rationale for this change
The base/field package now has an element concept and a minimal example_element component. The element concept will allow field operations done with elements in Montgomery form to be written generically. These field operations currently exist in the field12 package group and depend on the f12t::element.
What changes are included in this PR?

An element concept is added to the base/field package that is similar to the base/curve/element concept.
A minimal example_element component is added to the base/field package that is similar to the base/curve/example_element component.
Branching is removed from the base/field/subtract_p component.

Are these changes tested?
Yes

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

authors: SxT-Release jacobtrombetta


text:
Apply API config to grpc/http servers

Motivation

Closes https://github.com/spacemeshos/go-spacemesh/issues/1605

Changes
The loaded API config are now being applied to the grpc/http servers.

bors merge

bors merge

repo: spacemeshos/go-spacemesh


text:
public metrics

subset of metrics to record:

connections
version
init size started / completed
post proof time

they will be recorded with 5 first characters from p2p public key, we probably don't want to gather information about smesher key.
example config:
{
    "main": {
        "metrics-push": "https://public-metrics-gateway.spacemesh.dev/",
        "metrics-push-period": "1s",
        "metrics-push-user": "XXX",
        "metrics-push-pass": "XXXXXX",
        "metrics-push-header": {
            "X-Scope-OrgId": "XXXXXXX"
        }
    }
}

bors merge


text:
Add Sentry integration for releases

It closes #882
This PR introduces sentry integration with smapp
As we decided to try Sentry with a whole list of features ( Github / Slack integrations ). We will keep it simple and will know exactly the page and stack trace of an issue with auto-assign in Sentry on our team members.

Alright. I'm merging it now.
In case we can get rid of some excessively specified env vars — we can do it further.

authors: brusherru maparr

repo: spacemeshos/smapp


text:
Canonicalization doesn't works with no root tag in xml document

I'm trying to canonicalize something like this:
<foo></foo><baz></baz>
But, Nokogiri#canonicalize returns only the foo portion: <foo></foo>
When I wrapped this with another tag, this works properly:
<foo><baz></baz></foo>
So, I wonder if this is the right way the nokogiri does canonicalize, or if this is in fact a mistake.

XML with multiple root tags is an invalid document. Nokogiri makes it valid by dropping all but the first root tag.
If you have multiple roots, try parsing it as a fragment, not as a document.

authors: RafaelFerraro flavorjones

repo: sparklemotion/nokogiri


text:
Delete Wallet freezes Sparrow

Sparrow 1.6.4
Ubuntu 20.04
I tried to delete a read-only wallet with transactions

opened sparrow, waited till everything is synced
choose tab with read-only wallet I want to delete
File / Delete Wallet   and confirmed the dialog
The tab was closed
Sparrow was frozen
After a while I killed sparrow and restarted.
Nothing was deleted, I could just reopen the read-only wallet with File/Open

I can't reproduce this on Sparrow v1.6.4 and Ubuntu 20.04.

Did you mean a watch-only wallet?
Is there anything unusual about the storage on this machine?
Is this reproducible?
Can you check the Sparrow log file (See Help menu) to see if any relevant error was recorded there?

yes, watch-only wallet.
no
I removed the walled by hand, and die a new watch-only wallet with an xpub
nothing recorded in log before freezing sparrow

I tried to delete another wallet (multisig), which freezes Sparrow for about 60 seconds. (no mixer running)
Then with a new watch-only wallet just createt it took more than 200 seconds (mixer running on other wallet)

Why does it take so long anyway?

I suspect this was caused by a slow random number generator that is particular to your system. Before it deletes the file, Sparrow overwrites it with random data.
I've greatly reduced the overwrite data entropy in 0f2cf9c to reduce the possibility of this happening.

repo: sparrowwallet/sparrow


text:
Update Kepler.gl view to version 3.x

Version 3.0 of Kepler.gl has just been released: https://github.com/keplergl/kepler.gl/releases/tag/v3.0.0
This ticket collects all changes related to an update of the existing node to use the latest version of the library.

The release of kepler.gl v3.0.0 is not used in keplergl Python package that we are using. However, there are some alpha release of keplergl Python packge which use the alpha release of the Kepler.gl. Howerver, the structure of the saved HTML is different.

authors: koettert wybert


text:
Process hangs when calling urllib.request in a multiprocessing.Process with import of sounddevice package

Hi, I have come accross an issue when trying to use sounddevice together with the multiprocessing and urllib.request modules.
What happens is that when sounddevice is imported and I try to make an http requests in a sub-process, the sub-process hangs.
This happens for me on Mac OS X 10.15.7 with Python 3.7.9 and sounddevice 0.4.1.
I have posted first the issue to bugs.python.org where it was suggested that it may be some interaction between threads and fork, and to also post here.
Here is a minimal example to reproduce the problem. The problem disappear when either line A is commented or line B is uncommented.
import urllib.request
import multiprocessing as mp
import queue
from multiprocessing import Process, Queue
import time

# --> if we comment the following line, it works as expected
import sounddevice  # line A

def get_url(url):
    """ simple function that gets a url """
    content = urllib.request.urlopen(url)
    return f"{url}: ok"

def f_proc(q_in, q_out):
    """ the function that will run in a separate process """
    while True:
        url = q_in.get()
        print(f"Received {url}. Send GET request... ", end="")
        ret = get_url(url)
        print("done.")
        q_out.put(ret)

if __name__ == "__main__":
    # we want to access a number of urls
    urls = ["https://api.github.com", "https://www.google.com"]

    # --> if we uncomment the following line, then it works as expected
    # get_url(urls[0])  # line B

    # We start a process with a queue in and a queue out
    q_in = mp.Queue()
    q_out = mp.Queue()
    p = mp.Process(target=f_proc, args=[q_in, q_out])
    p.start()

    # We feed all the urls to the process via the queue
    for u in urls:
        q_in.put(u)

    # Wait a little bit
    time.sleep(1)

    # Then read the result from the output queue
    while True:
        try:
            r = q_out.get(timeout=0.1)
            print(r)
        except queue.Empty:
            print("Nothing in the queue. Leave.")
            break

    # After that, we try to join, or kill the process
    print("wait for process to terminate...")
    p.join(timeout=0.1)
    p.kill()
    print("finished")

Output with the problem:
> python ./test_mp_req.py
Nothing in the queue. Leave.
wait for process to terminate...
finished

Expected output:
> python ./test_mp_req.py
Received https://api.github.com. Send GET request... done.
Received https://www.google.com. Send GET request... done.
https://api.github.com: ok
https://www.google.com: ok
Nothing in the queue. Leave.
wait for process to terminate...
finished

I could verify that the problem does not occur with python 3.8 and 3.9. I am thus closing the issue.

I could verify that the problem does not occur with python 3.8 and 3.9. I am thus closing the issue.

repo: spatialaudio/python-sounddevice


text:
[UNR-2553][MS] Fix for droppped URL options.

Contributions: We are not currently taking public contributions - see our contributions policy. However, we are accepting issues and we do want your feedback.

Description
Fix for scavengers dropped URL options.

Update.

repo: spatialos/UnrealGDK


text:
[UNR-2417][MS] Fixing some warnings in cloud deployment to do with a "None" component.

Matt Young discovered that the "Nameplate" component on BP_Base_Character had the option "Owner No See" set to true. This meant on owning client the component did not exist but was still being used.
I have simply added checks to see if the "Nameplate" component is valid.

Targeting 0.8.1 instead.

repo: spatialos/UnrealGDKExampleProject