text:
add pr description job to workflow

Self Checklist

[x] I wrote a PR title in English and added an appropriate label to the PR.
[x] I wrote the commit message in English and to follow the Conventional Commits specification.
[x] I added the changeset about the changes that needed to be released. (or didn't have to)
[x] I wrote or updated documentation related to the changes. (or didn't have to)
[x] I wrote or updated tests related to the changes. (or didn't have to)
[x] I tested the changes in various browsers. (or didn't have to)

Windows: Chrome, Edge, (Optional) Firefox
macOS: Chrome, Edge, Safari, (Optional) Firefox

Related Issue

https://github.com/channel-io/bezier-react/issues/863

Summary

피그마 플러그인을 사용해서 아이콘 업데이트 pr 을 날리면 업데이트 내역을 보고 적절히 description 을 추가하는 job 을 만들었습니다. (실행결과)

Details

description 을 추가하기 위해 job 이 실행되기 직전 커밋 내역으로부터 아이콘 업데이트 내역을 가져옵니다(git log -1 --name-status --pretty="format:"). 이걸로 적절히 description 을 생성한다음 Github REST API 를 통해 pr 을 업데이트 하면 됩니다. pr 업데이트 할 때 클라이언트는 octokit 을 써도 되지만  디펜던시를 설치하게 되면 1분 정도 시간이 소요되기 때문에 일부러 fetch 를 사용했습니다.

추가되는 descripiton 양식은 아래와 같습니다.

Icon update is ready to be merged! 🎉
23 icon(s) Deleted 🗑

bank.svg
breaktime.svg
cart-abandoned.svg

2 icon(s) Added 🎨

all.svg
chevron-right.svg

1 icon(s) Modified 🖊

check.svg

물론 테이블이라던지 다른 양식도 가능하기 때문에 제안 주시면 고려해보겠습니다.

Breaking change? (Yes/No)

No

References

pr 업데이트 할 때 클라이언트는 octokit 을 써도 되지만 디펜던시를 설치하게 되면 1분 정도 시간이 소요되기 때문에 일부러 fetch 를 사용했습니다.

👍

물론 테이블이라던지 다른 양식도 가능하기 때문에 제안 주시면 고려해보겠습니다.

지금 양식도 너무 좋구요, 아래 테이블과 같은 양식도 적용해볼 수 있을 거 같아요 (GPT의 도움..)
Summary
Icon update is ready to be merged! 🎉

🆕 2 Icon(s) Added
✏ 1 Icon(s) Modified
🗑 23 Icon(s) Deleted

Name	Status
all.svg	🆕
chevron-right.svg	🆕
check.svg	✏
bank.svg	🗑
breaktime.svg	🗑
cart-abandoned.svg	🗑

오 이거 너무 좋네요! 적용해볼게요

머지해도 좋을 거 같아요~!

authors: sungik-choi yangwooseong

repo: channel-io/bezier-react


text:
Always log if there's an exception occurred.

Hi,
I'm trying to run sanic in one of my server. My problem is that when there's an exception, I cannot see what happened because there's no log to see. The solution to this is to enable debug mode, the problem with that is the user can see the exception logs as well.
I'm not sure this is the best approach but I propose we always log the exception even though we are not running in debug mode.

I agree with this as well. Thought it was a bit weird when I found out sanic was swallowing all of the exceptions unless in debug.

Isn't it good to shallow those exceptions for security reasons, maybe you could trow these into a log if not in debug mode and then check the log directly on the server?

I see no noticeable degradation from these changes:
After with no exception
$ wrk -t50 -c100 -d30 http://localhost:8000
Running 30s test @ http://localhost:8000
  50 threads and 100 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency     1.45ms  510.97us  15.91ms   82.72%
    Req/Sec     1.39k   272.20     9.94k    73.65%
  2080812 requests in 30.10s, 255.99MB read
Requests/sec:  69129.72
Transfer/sec:      8.50MB

Before with no exception
$ wrk -t50 -c100 -d30 http://localhost:8000
Running 30s test @ http://localhost:8000
  50 threads and 100 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency     1.48ms  519.05us  28.79ms   84.82%
    Req/Sec     1.37k   240.10    16.64k    77.67%
  2049662 requests in 30.10s, 252.16MB read
Requests/sec:  68094.88
Transfer/sec:      8.38MB

After with exception
$ wrk -t50 -c100 -d30 http://localhost:8000
Running 30s test @ http://localhost:8000
  50 threads and 100 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency     1.65ms  618.78us  26.86ms   77.83%
    Req/Sec     1.22k   316.54    14.46k    74.39%
  1827891 requests in 30.10s, 338.18MB read
  Non-2xx or 3xx responses: 1827891
Requests/sec:  60734.48
Transfer/sec:     11.24MB

Before with exception
$ wrk -t50 -c100 -d30 http://localhost:8000
Running 30s test @ http://localhost:8000
  50 threads and 100 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency     1.64ms  509.15us  11.68ms   74.98%
    Req/Sec     1.23k   218.65     2.02k    69.49%
  1832898 requests in 30.02s, 339.11MB read
  Non-2xx or 3xx responses: 1832898
Requests/sec:  61046.20
Transfer/sec:     11.29MB

@zkanda is it possible to write a unit test to make sure it's doing what you expect, or is that too difficult? I had some troubles when testing with logging, since there are some oddities with the pytest logging treatment.

I ask because I implemented your change and did not see an exception logged in non debug mode when I used an invalid route:
from sanic.response import text
from sanic import Sanic
import asyncio
import time

app = Sanic()

@app.route('/')
async def hello(request):
    x == 3
    return text("OK")

if __name__ == '__main__':
    app.run(host="0.0.0.0",
            port=8000,
            workers=4)

@zkanda that's ok if that's a different bug, but I would need to see an example where this does something before approving it. Can you provide one?

This would bring it into line with similar programs - errors are generally at the default level of output

Seems like something is wrong with the test..
@r0fls how are you running the test? I'm using tox but I would prefer to use py.test directly.

Here's how I did it, install all requirements-dev.txt
Run pytest

I get a lot of failed test. 90 in total
e.g.
tests/test_views.py:194:
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _

app = <sanic.sanic.Sanic object at 0x7f18d426a828>, method = 'get', uri = '/', gather_request = True, loop = None, debug = False, server_kwargs = {}, request_args = (), request_kwargs = {}
_collect_request = <function sanic_endpoint_test.<locals>._collect_request at 0x7f18d6713950>, _collect_response = <function sanic_endpoint_test.<locals>._collect_response at 0x7f18d42da048>

    def sanic_endpoint_test(app, method='get', uri='/', gather_request=True,
                            loop=None, debug=False, server_kwargs={},
                            *request_args, **request_kwargs):
        results = []
        exceptions = []

        if gather_request:
            def _collect_request(request):
                results.append(request)
            app.request_middleware.appendleft(_collect_request)

        async def _collect_response(sanic, loop):
            try:
                response = await local_request(method, uri, *request_args,
                                               **request_kwargs)
                results.append(response)
            except Exception as e:
                exceptions.append(e)
            app.stop()

        app.run(host=HOST, debug=debug, port=PORT,
                after_start=_collect_response, loop=loop, **server_kwargs)

        if exceptions:
            raise ValueError("Exception during request: {}".format(exceptions))

        if gather_request:
            try:
                request, response = results
                return request, response
            except:
                raise ValueError(
                    "Request and response object expected, got ({})".format(
>                       results))
E               ValueError: Request and response object expected, got ([])

../../../.virtualenvs/sanic/lib/python3.6/site-packages/Sanic-0.2.0-py3.6.egg/sanic/utils.py:51: ValueError
================================================================================== 90 failed, 8 passed, 2 skipped in 6.87 seconds ===================================================================================

@zkanda I'm able to run the tests with python3 -m pytest tests from within the directory. Is that the command you're using? Anyway I'm fine merging this as is, because I agree it's the right behavior.

hmmm the reason why it's failing on test is because the request object is None when we call request.url. I tried debugging why it's not passing that object properly on testing but I'm not so familiar with the code base yet(and currently at work).
With that said, if we just log without the request then obviously the test would pass but there is redundancy in logging if we are in debug mode.

@zkanda let's start here, if you want to improve the functionality further just open another PR. This is an improvement already since if there is an exception, it should probably be logged.

source: gharchive/pull-request

authors: GenericError awiddersheim odarbelaeze r0fls zkanda

repo: channelcat/sanic

license: mit

license_type: permissive

license_source: bigquery


text:
Add splicing information

@pablo-gar commented on Fri Mar 24 2023
Original contributor Zach DeBruine @zdebruine
Description
To have splicing information/status in the Cell Census. Specifically, for a given gene (or transcript?) to be able to separate spliced counts vs unspliced counts
Context
Splicing information is useful for modeling biological behavior and in particular trajectory analysis, e.g. pseudotime
Relevance
It currently blocks certain analysis as the aforementioned trajectory analysis.
See also comment below by @zdebruine
Ideal behavior
See comment below by @zdebruine

@pablo-gar commented on Fri Mar 24 2023
Zach, can you please add any other information as comment for the Context, Relevance, an Ideal behavior sections?

@zdebruine commented on Fri Mar 24 2023
Certainly! It would be great to include unspliced transcript expression alongside spliced transcript expression, as it can provide valuable information in single-cell sequencing assays.
Context and Relevance:  Here are some reasons why unspliced transcript expression is important (and I'm not an expert per se on this):

Unspliced transcript expression can be used to map cellular differentiation trajectories in real-time, rather than relying on pseudotime (as seen in scVelo/velocyto).
Including unspliced transcript expression in dimension reduction analysis can provide better sensitivity and more powerful results, as it effectively doubles the amount of information available. I would be happy to provide an example of this in a post if desired.
Excluding unspliced transcript expression means throwing away up to ~40% of the mapped sequencing reads (based on data I have analyzed from developing mouse embryos), which contains useful information.
The ratio of unspliced to spliced transcript expression is a measure of cellular transcriptional activity, and can be used to distinguish between active and quiescent cells.
The inclusion of unspliced transcript expression can support research that may be underpowered due to small sample sizes, as there are currently no large datasets available (to my knowledge) that provide both spliced and unspliced transcript count matrices.

Ideal Behavior:  Simply add rows to the matrix for unspliced features.  You may add feature metadata so that it is easy to filter just spliced, if desired, or simply append "_unspliced" to the feature name, however you like.

@pablo-gar commented on Fri Mar 24 2023
Thanks @zdebruine this is great!

@brianraymor commented on Wed May 24 2023
@pablo-gar to review with @BAevermann and Lattice.

@BAevermann commented on Wed May 24 2023
I am confused, are we proposing:

Adding support for differentially spliced transcripts
Adding support for Unspliced transcripts only, i.e., whole gene (including all introns)
Thanks!

@zdebruine commented on Wed May 24 2023
@BAevermann The second item.  There is already some support for the first point, and further support would be challenging given the experimental/unstandardized software in this domain and limited sequencing depth in most assays.  But with advances in long-read deep sequencing, it may be worth revisiting differentially spliced transcripts again in time.

@BAevermann commented on Fri May 26 2023
Thanks for the clarification @zdebruine!
This may be a daft question, but how does the resulting expression matrices look different between "unspliced" counts to the current "gene" counts (CELLxGENE schema requires Ensemble gene IDs). I am just curious how we can determine if the gene concept used contains all introns/exons or omits portions.
Thanks!
b.

@zdebruine commented on Fri May 26 2023
@BAevermann I am not sure.  This may be more of a question for the authors of velocyto or scVelo who have more experience with various builds.  I have only used processed data from other authors and found it to significantly improve the power of dimension reduction, hence the request.

@BAevermann commented on Mon May 29 2023
Thanks @zdebruine! I will look into velocyto and scVelo to get a better understanding about how they invoke annotation in their data workflow...

10x CellRanger now defaults to including intronic reads: https://support.10xgenomics.com/docs/intron-mode-rec. So, I am assuming most of the newly aligned data does include unspliced transcripts. Most of the single nuclei data that goes through Lattice include intronic reads, and we capture this with a boolean field intronic_reads_counted, defined as: “True if reads mapped to intronic regions were used for gene quantifications.”
We do have one dataset where contributors ran VeloCyto for their gene quantification, but they merged the unspliced and spliced counts together as their raw counts. If contributors were interested, though, they could provide the either counts as additional layers in adata.layers.

@jychien You are correct, but I am not sure why 10x does not provide intronic and exonic reads separately.  After all, they provide different information.
I gather that the ability to support this within CZI resources would require contributors to provide these counts matrices separately.  As you say, perhaps contributors could at least be encouraged to provide the spliced and unspliced counts separately in adata.layers?

Hey @zdebruine!
Sorry about the delay. The summer is always chaotic with work and personal travel.
You are correct that this data contributors would need to provide these layers. Do you have an example matrix or could you point me to a publication that used splicing in this way?
Thanks!
Brian A

Got a similar request from Luca Pinello to support this Data Insights Project
A next-best option may be to make it easier for CxG users to identify raw sequence reads that correspond to CxG obs, so that they can realign.

It would be very exciting to have easier access to raw sequence reads.  Just linking this to soma_joinid would already be very helpful, and somehow integrating into the CellCensus API would make it very easy to gather data for these large transformer models.

Per November 27 triage, there was consensus to close this issue. @jahilton may open another more general issue related to his suggestion:

A next-best option may be to make it easier for CxG users to identify raw sequence reads that correspond to CxG obs, so that they can realign.

source: gharchive/issue

authors: BAevermann brianraymor jahilton jychien zdebruine

repo: chanzuckerberg/single-cell-curation

license: MIT

license_source: github-api


text:
[Question] about sorbet dependency

I love this gem. Very useful.
I'm wondering whether it's better to move sorbet dependency to development only?
The static analysis is only useful  in development mode. In production, the sorbet-runtime is the must have gem IMO.
I'm not sure how sorbet-coerce works, but IMO in runtime the sorbet-coerce  would only need the sorbet-runtime dependency

Hi, you're right! Will move sorbet to dev-dependency. Thanks for catching this!

Great, thanks for the response~

repo: chanzuckerberg/sorbet-coerce


text:
Use schema in show pipes

use SCHEMA instead of DATABASE in show for stages and pipes.
Addresses https://github.com/chanzuckerberg/terraform-provider-snowflake/issues/290

Test Plan

Current tests pass...
Have used in our current production environment with two workspaces where resources share names but are in different schemas.

References

https://docs.snowflake.com/en/sql-reference/sql/show-pipes.html (note use of SCHEMA or DATABASE)

This looks good! Can you pull in recent changes?

authors: alldoami elliottohara


text:
🛑 Silky.Network - us-ca is down

In e868477, Silky.Network - us-ca (https://us-ca-mirror.chaotic.cx/no-failover/chaotic-aur/lastupdate) was down:

HTTP code: 503
Response time: 63 ms

Resolved: Silky.Network - us-ca is back up in c90c98f.

repo: chaotic-aur/chaotic-uptimes


text:
🛑 Silky.Network - jp-2 is down

In d9706e0, Silky.Network - jp-2 (https://jp-2-mirror.chaotic.cx/no-failover/chaotic-aur/lastupdate) was down:

HTTP code: 503
Response time: 120 ms

Resolved: Silky.Network - jp-2 is back up in 8a2abe1.


text:
[BUG] Daily Poll Failing to complete (Solution Found)

For some reason the Daily Poll was failing to execute correctly. To make it work again, change line 343 to
browser.find_element(By.XPATH, '//*[@id="daily-sets"]/mee-card-group[1]/div/mee-card[' + str(cardNumber) + ']/div/card-content/mee-rewards-daily-set-item-content/div/a/div['+ str(cardNumber)+']/span').click()

Also cahnge line 347 to
browser.find_element(By.XPATH, '//*[@id="btoption' + str(random.randint(0, 1)) + '"]/div[2]/div[2]' ).click()

These lines of code ensure that the script can find and select the correct buttons on the page.

I forked this project and uploaded my current working version that will complete all searches and punch cards. Also made the README even more simple to understand.
https://github.com/AlexyGalaxy/Microsoft-Rewards-Farmer

Fixed with v3.0 #188

authors: AlexyGalaxy charlesbel

repo: charlesbel/Microsoft-Rewards-Farmer


text:
Select All - leaving 30% of items unselected.

Hi, so right now regardless of how I do selections such as invert or select all, a significant number of items are not selected.

Any chance you can record your screen doing this? I've noticed this happen when trying to select a lot of items and the ones that are unselected are too small in the viewport or obscured by other objects. I've always determined it to be a Blender issue, not a NMS Plugin issue. Perhaps there's a setting that would help it.

I went thru some of the steps here.
BlenderSelectIssue.zip

OK, not sure but it has something to do with visibility being ghosted. Although I went back to normal viz, the items could not be selected or moved. When I ghosted the view then put it back to normal it let me select everything. Will keep you posted.

OK so it looks like its a good idea to always cycle visibility in the plug in before you do mass select work. If not it will always be iffy. It seems that once you have returned to normal viz then you can select anything as usual.

Cool, yea I think that's the intended behaviour. I've made it so items set to "ghosted" will be transparent and unelectable. This is so you can manage items inside rooms a bit easier.

Just to be clear, I get that and it works. However, I am finding that even if the view is "Normal" there are times that it will not select everything, so cycling thru from Normal and all the way back to Normal seems to be the way to go.

authors: charliebanks ukmerlin

repo: charliebanks/nms-base-builder


text:
checksum mismatch

Describe the bug
I'm trying to build the example code for charmbracelet/huh. When running go run main.go I get this error from lipgloss:
❯ go get github.com/charmbracelet/lipgloss@v0.9.1
verifying github.com/charmbracelet/lipgloss@v0.9.1/go.mod: checksum mismatch
        downloaded: h1:G7mAYYxgmS0lVkHyy2hEOLQCFB0DlQFTMLWggykrydY=
        go.sum:     h1:1mPmG4cxScwUQALAAnacHaigiiHB9Pmr+v1VEawJl6I=

SECURITY ERROR
This download does NOT match an earlier download recorded in go.sum.
The bits may have been replaced on the origin server, or an attacker may
have intercepted the download attempt.

For more information, see 'go help module-auth'.

I have attached go.mod and go.sum in the gist
Setup
Please complete the following information along with version numbers, if applicable.

OS NixOS
Shell zsh
Terminal Emulator foot
Terminal Multiplexer zellij
Locale en_US.UTF-8

To Reproduce
Steps to reproduce the behavior:

Save main.go, go.mod and go.sum liked above to a folder
run go run main.go
See error

Source Code
gist
Expected behavior
example runs

This looks like it may be an issue with your local cache or Go proxy. I'd recommend go mod tidy — and barring that regenerating your go.sum. If that doesn't help I'd suggest rebuilding the contents of your GOPATH.
We're not able to reproduce this internally.

Confirmed using those files that it requires a go mod tidy.
This issue should be closed as this is clearly a tooling error and has nothing to do with Lipgloss.

Okay something borked in my side then, sorry to bother.

authors: meowgorithm mikelorant pinpox

repo: charmbracelet/lipgloss


text:
Logarithmic chart loosing points

as from the screenshot below and the attached example, the chart is messing up with initial an final points when there are many zeroes and the scale is logarithmic, when reset to linear everything works as expected (tried with v2.13 and v2.14)

logarithmic.zip

@grigoni the issue here is that log(0) is undefined mathematically.

So is this an enhancement that is being worked on?  If so, what will be the proper behavior?
I'm using the logarithmic scale because some points may be so high that the other points are completely dwarfed and basically look like zero.
The best workaround I've found so far is to make it so at least the other data points render, is to set the min of ticks to something small, like 0.0001.  But this has the undesired side effect of the point being graphed as infinity (see screenshot).  The other bad thing is that I have to specify the min on the scale, and not allow it to responsively adapt to the data.  All these 0 data points are showing as really tall bars.  Ideally they'd show as 0, even though log(0) is not zero.

This has been resolved now that the logarithmic axis now handles 0 in a somewhat clean manner. See https://jsfiddle.net/jzpcjtrs/

authors: etimberg grigoni lbjones


text:
titleFontSize does not work for pie charts

Expected Behavior
expected the following tooltip option to increase label font-size:
options: {
  tooltips: {
    titleFontSize: 20
  }
}

Current Behavior
current behaviour is the tooltip title label font-size stays the same
Possible Solution

Steps to Reproduce (for bugs)

Add options for titleFontSize
Hover over to show label

Environment

Chart.js version: 2.4.0
Browser name and version: Chrome Version 55.0.2883.95 (64-bit)
Link to your project: https://jsfiddle.net/ufhm763h/1/

Extra
This seems to work on bar and line charts. I will investigate this and see if it affects other chart types.

@Jareechang the reason that it doesn't apply is that the default title callback returns nothing https://github.com/chartjs/Chart.js/blob/master/src/controllers/controller.doughnut.js#L101-L103
Not sure if this is a bug or not. thoughts @chartjs/maintainers ?

@etimberg ah ok. I just noticed it while looking at a fiddle.
I just thought it would make the API more consistent. As this works on the other charts.

Closing as not a bug since this is currently as designed

authors: Jareechang etimberg


text:
Hide just one y-axes out of 3 in the grouped bar chart

I have a grouped bar chart, 3 columns for each group.
Can I hide only one y-axe column out of 3 but still show the hidden column's data in the tooltip?
This is my code:
`$.ajax({
type: 'GET',
url: "data/recoverability.json",
dataType: "json",
success: function( response ) {
var datascource = response;
BarChartGroupedLoad(datascource);
},
error: function (textStatus, errorThrown) {
console.log("Error with json data");
}
});
function BarChartGroupedLoad(jsonData) {
var chartjsValues = [];
var chartjsLabels = [];
var chartjsMonths = [];

chartjsMonths = jsonData.months;

for (var i = 0; i < jsonData.dataset.length; i++) {
	var d = {};

	d.data = jsonData.dataset[i].data;
	d.label = jsonData.dataset[i].label
	d.backgroundColor = jsonData.dataset[i].backgroundColor

	chartjsValues.push(d);
}

var barChart = $("#barChart-grouped");

var myChart = new Chart(barChart, {
	type: 'bar',
	data: {
		labels: chartjsMonths,
		datasets: chartjsValues
	},
	options: {
		scales: {
			yAxes: [{
				ticks: {
					beginAtZero:true
				}
			}]
		},
		tooltips: {
			mode: 'index'
		}
	}
});

}`
This is json:
{   "months": ["April 2017","May 2017","June 2017","July 2017","August 2017","September 2017"],   "dataset": [     {      "label":"worked",       "data": [1020,495,875,325,225,3120], 	  "backgroundColor": "#8cd1f3"     },     { 	  "label":"net-revenue",       "data": [1100,0,875,325,225,3050], 	  "backgroundColor": "#d6dade"     },     { 	  "label":"difference",       "data": [80,-495,0,0,0,-70], 	  "backgroundColor": "#f4f4f4"     }   ] }

@sashkanz so you want to hide one of the bars, but still show it in the tooltip?

@etimberg that's correct, hide 1 out of 3 in each group but show all of them in the tooltip.

@sashkanz I don't think there is a good way to do that unfortunately. The tooltip never receives the items that are from hidden datasets. You could modify https://github.com/chartjs/Chart.js/blob/master/src/core/core.interaction.js#L32 to not check if the dataset is visible or not

Closing as there is currently no way to avoid this and for the majority of charts it doesn't make sense to include hidden datasets. Modifying the code above is the best way to achieve this

authors: etimberg sashkanz


text:
Change Point Dot color

How to, it's not in documents.

I need to change Point dot color too.I've seen on line 276 (https://github.com/jtblin/angular-chart.js/blob/master/dist/angular-chart.js):
function getColour (colour) {       return {         fillColor: rgba(colour, 0.2),         strokeColor: rgba(colour, 1),         pointColor: rgba(colour, 1),         pointStrokeColor: '#fff',         pointHighlightFill: '#fff',         pointHighlightStroke: rgba(colour, 0.8)       };     }
I need to change pointColor to always be white on some cases, not the same color of stroke.  Any advice here?

Going further on this, I've found that on chart.js v1, you can set a full 'data' object, with labels and datasets, and pass it when you create a new ChartJs. But angular-chart forces me to set separated configs in each canvas attribute (chart-labels, chart-data, chart-series, ....).
There is a way to pass the full data object directly using angular-chart? If not, do you think to allow this as an option (you can use both ways, not at the same time, of course)? Ty.

authors: newtonwagner thalesfsp


text:
Uncaught TypeError: Cannot read properties of undefined (reading 'defaultFontFamily')

I'm try to use your plugin but get this error just after load chartjs-plugin-annotation.js. I tried some examples in the end I tried your sample code https://github.com/chartjs/chartjs-plugin-annotation/blob/master/old_samples/box.html but result was the same.  Sorry may be it's stupid question and I lost some info how to use it
chartjs-plugin-annotation.js:485 Uncaught TypeError: Cannot read properties of undefined (reading 'defaultFontFamily')
at Object.6../annotation.js (chartjs-plugin-annotation.js:485)
at s (chartjs-plugin-annotation.js:10)
at e (chartjs-plugin-annotation.js:10)
at chartjs-plugin-annotation.js:10

What version of the plugins are you trying to use and with what version of Chart.js?

Version: 0.5.7

You need to use annotation v1+ with chart.js v3+

Thx

repo: chartjs/chartjs-plugin-annotation


text:
[BUG] 第一次提问后报错 /  utils.py[line:24] - ERROR: object of type 'NoneType' has no len()

问题描述 / Problem Description
当我第一次提问 时，报错 utils.py[line:24] - ERROR: object of type 'NoneType' has no len()
环境信息 / Environment Information

langchain-ChatGLM 版本/commit 号：v2.0.8
是否使用 Docker 部署（是/否）：否
使用的模型（ChatGLM2-6B / Qwen-7B 等）：Qwen-1_8B-Chat
使用的 Embedding 模型（moka-ai/m3e-base 等）：m3e-base
操作系统及版本 / Operating system and version: windows
Python 版本 / Python version: 3.10.10

我也是这个问题，用的qwen-api，同样用的openai api没这个问题。

我也是这个问题，centos下用的qwen-api，同样用的openai api没这个问题。

您好 请问有测试哪个离线的模型可以使用吗？

我也是这个问题，centos下用的qwen-api，同样用的openai api没这个问题。

您好 请问有测试哪个离线的模型可以使用吗？

没试过

同问题，用Qwen-14B-Chat启动正常无报错，但一提问就报错。

应该是配置问题，这两个模型我这边是跑过的

换成  bge-large-zh  和 chatglm2-6b  后 ，正常了

有朋友们解决了这个问题吗，我用的bge-large-zh和baichuan2-13B也是这个报错

baichuan模型要降级transomers

authors: Gavin-China fxsome kaixuanjames sleepyddl zRzRzRzRzRzRzR

repo: chatchat-space/Langchain-Chatchat


text:
Is puid cookie need to refresh? if yes, how?

Is puid cookie need to refresh? if yes, how?

puid need to be refreshed manual.

repo: chatgp/chatgpt-go


text:
Add daemon specific options for windows

Hello,
i would like to have the possibility to set an attribute similar ( or use the same)  to node['chef_client']['daemon_options'] for windows tasks.
Thanks

Created a branch for testing by OP https://github.com/drrk/chef-client/tree/add_windows_task_daemon_options

authors: Roviluca drrk

repo: chef-cookbooks/chef-client


text:
Error alerts styling & behavior in trial license modal

Describe the bug
There are two main issues with these errors:
1. The errors in the trial license modal do not have the expected styles.

They should look like this:

It looks like the chef-error in the modal is not properly inheriting the .chef-error class.
chef-error element rendered on license-lockout modal
<chef-error _ngcontent-qgm-c1="">
           Valid email required.
</chef-error>

chef-error element rendered on create-user modal
<chef-error _ngcontent-iqu-c28="" class="chef-error ng-star-inserted">
            Display name is required.
</chef-error>

2. The errors do not pop up if you tab past them
I can tab through all of the form fields without getting any error feedback.
The create-user modal component on https://a2-dev.test/settings/users demonstrates the behavior we expect: if a user leaves an input without entering a value, a Field is required error should pop up.
To Reproduce
1. styling issues

start up Automate + UI
run chef-automate dev psql chef_license_control_service -- -c "UPDATE licenses SET active=NULL"
open the UI or refresh to see the trial license modal
type a single character in the email field to see the error pop up

2. tabbing

after removing the UI, open the UI or refresh to see the trial license modal
tab through the input fields

Versions (please complete the following information):

Browser: chrome
Automate Build Number: 20200129212217

this work has been deprioritized, so closing.

authors: bcmdarroch susanev

repo: chef/automate


text:
Add new option update-strategy for policy updates

Description
This option has two possible values for now:

'relaxed' (default value). This is the behavior before this patch.
chef update Policyfile.rb apache2 will upgrade apache2 and all its
dependencies
'strict'. This behavior is very strict: it updates only the cookbook(s)
mentionned in the command line.

The intent of this patch is to pave a possible way to have a third
strategy who would upgrade cookbooks mentionned on the command line to
their maximum existing version (possibly by upgrading other cookbooks if
necessary).
This patch already gives very fine control to users who want to update a
single cookbook (or a few cookbooks only).
Check List

[x] New functionality includes tests
[x] All tests pass
[x] RELEASE_NOTES.md has been updated if required (not required for bugfixes, required for API changes)
[x] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/master/CONTRIBUTING.md#developer-certification-of-origin-dco

I'll write tests in the next few days

@tas50 would you have early feedback on the intent?

@thommay would you have feedback on that patch?

@chef/client-maintainers would you have any feedback regarding this patch (or even the intent)?

Are you basically building out bundle update --conservative <gems> and bundle update --strict <gems> and should we just use that API?

Looks good to me!
I am wondering if it would be useful to be able to set the update strategy for a specified cookbook dependency in the policyfile itself.
Eg:
cookbook 'foo', update_strategy: 'strict'

It's definitely something which can be revisited later though after people start using the feature.

Thanks for all the feedbacks.
@rhass, you're right that could be something that is set per cookbook at some point. Let's see how users want to play the option at first.
@lamont-granquist , thanks for the link!
It seems that what bundler and I call strict/conservative is a bit different.
Using example commands:

bundle update X Y
chef update X Y

bundler
my proposal for chef update

strict
Update only gem explicitely mentioned  (X & Y) with respect to --patch, --minor, --major flag
Update only cookbook explicitely mentioned (X & Y) to the latest version possible.

conservative
Freeze all shared dependencies
Update dependencies only it allows X and Y to be resolved to a higher version. If X and Y can be resolved to the highest version without changing their dependencies, nothing else happen

default (no option)
all dependencies are updated as well
all dependencies are updated as well

From my experience with chef in Criteo (we don't publish patch versions of older cookbook versions, users always need to update to latest version), bundler's strict is not necessary (for us, it might different for others).
It seems my proposal of strict for chef update is actually equivalent to bundler's conservative.
The third option I mentioned in my PR would be called indeed conservative but would have a different meaning that bundler's.
It could be a bit confusing for users used to bundler.
What do you think @lamont-granquist ?

@lamont-granquist would you have more feedback?

@chef/client-maintainers would you have more feedback?

@danielsdeleo Any thoughts on this?

Thanks for the feedback. I'll push a commit to move from enum style option to a flag --exclude-deps.
This is a separate commit to allow easy revert to a enum option if a third option is implemented.

All done, thanks for the catch

:tada:

authors: kamaradclimber lamont-granquist rhass tas50

repo: chef/chef-dk


text:
Don't clobber options[:prefix] if set

If machine_options['transport_options']['username'] is not 'root', then machine_options['transport_options']['options']['prefix'] gets unconditionally set to 'sudo ', even if the user specified some other value in the recipe. Let's fix things so as to not clobber it.

I see that this has some overlap with #18.

Hey Glenn,
You're right, #18 covers this and the key vs key_data part as well.
As such I am going to go with that one, assuming testing goes smooth.
Gonna leave this open for now and close on final resolution.
Thx for the PR!

Merged alongside of PR #18 in 756c186. Thanks, @glennmatthews!

authors: adamleff double-z glennmatthews

repo: chef/chef-provisioning-ssh


text:
data-collector rewrite based on the path of root_url attribute

After reading this I'm proposing a different solution for fix: https://github.com/chef/chef-server/pull/1010

Do we need this to go out in 12.11?

@stevendanna nope, we don't. This change makes something configurable that was hardcoded before, but the hardcoded version is ok for super-near-future.

We probably should make sure the compliance profile is changed to match; it was modeled after the data collector.
Also, the release notes may need updating to fit with this change.

authors: alexpop markan srenatus stevendanna

repo: chef/chef-server


text:
dsc_script Fails to Pass Along timeout Parameter

As a customer, I would like a dsc_script resource like this to modify the powershell default timeout of 600s. Many meaningful runs of the dsc_script resource may take longer than 600s.
dsc_script "thing" do
    timeout 800
end

Versions

Chef Client 11.16.4
Windows Server 2012 R2

Process
Added a debugging command output statement at https://github.com/chef/chef/blob/11.16.4/lib/chef/util/powershell/cmdlet.rb#L64 of
Chef::Log.info("Found this info about the timeout setting ------ #{command.inspect}")

Example Run and test-dsc-script-script.rb: https://gist.github.com/sean-horn/3925be080780736b0f5b
Problem
As shown in the Example Run above, the timeout attribute of the dsc_script is not making it all the way down to the cmdlet invocation. Thus, the default timeout of 600s is always in effect in the chef-client run of the resource, no matter the setting in the resource.
ZenDesk 2655

This is fixed in the next release via #2885. This can currently be tested in the 12.1.0.rc.0 release. Instructions for downloading that can be found here: http://lists.opscode.com/sympa/arc/chef/2015-02/msg00352.html
Otherwise we expect 12.1.0 to go out this week.

authors: btm sean-horn

repo: chef/chef


text:
Chef config always in C:\chef when using windows MSI

Hello,
When we set the chef folder in MSI (as default c:\opscode\chef), the config folder is forced to be C:\chef. This is an inconveniant when using MSI because we have to make the changes after install.
The config folder could be moved from c:\Chef to <MSI_install_folder>\chef\config (as default C:\opscode\chef\config) for exemple?
the interest is to have only one folder defined for chef-client in all case.
It is pretty much the same as #3952 with the choice of target folder.
Best Regards,
lbn team,

I'm closing this due to age. We should keep configuration and the application installation folders separate so that the application can be upgraded separately.  That is the status-quo today.

authors: lbn-chef-team stuartpreston


text:
knife cookbook metadata from is a lie

Description
knife help shows two different some commands: knife cookbook metadata and knife cookbook metadata from... but the later doesn't seem to really exist, using it what you get is knife cookbook metadata looking for a cookbook named from.
Chef Version
Chef Workstation version: 0.7.4
Chef Infra Client version: 15.1.36
Chef InSpec version: 4.7.3
Test Kitchen version: 2.2.5
Foodcritic version: 16.1.1
Cookstyle version: 5.0.0

Platform Version
Ubuntu 18.04
Replication Case
knife cookbook metadata from path/to/metadata.rb

yeah that is some horribly ancient pre-berkshelf / pre-stove API

Fix is provided in #8812

authors: Vasu1105 jaymzh lamont-granquist


text:
Remove duplicate Gemfile gems + update ohai to 16.7.13

Remove gemfile gems that are also in the gemspec.
Signed-off-by: Tim Smith tsmith@chef.io

It should really be in the gemspec so knife works in a gem install


text:
rspec 3.4.0 broke master

pinning to 3.3.x as a temp fix

:person_frowning: :+1:

Can you paste whatever the error is in here, so people might be able to find it via Google?

dammit travis...  looks unrelated..

:+1: on this change, would be nice to add some more context either in here or as a comment in the gemspec change, per @coderanger's suggestion

chef-client
  when the chef repo has a cookbook with only an audit recipe
    should exit with a non-zero code when there is an audit failure (FAILED - 1)

Failures:

  1) chef-client when the chef repo has a cookbook with only an audit recipe should exit with a non-zero code when there is an audit failure
     Failure/Error: expect(result.stdout).to include("Failure/Error: expect(2 - 2).to eq(1)")

       expected "[2015-11-16T11:45:11-08:00] INFO: Started chef-zero at chefzero://localhost:8889 with repository at /tmp/chef_repo20151116-61897-1wkrjt6\n  One version per cookbook\n\n[2015-11-16T11:45:11-08:00] INFO: Forking chef instance to converge...\n[2015-11-16T11:45:11-08:00] INFO: *** Chef 12.5.1 ***\n[2015-11-16T11:45:11-08:00] INFO: Chef-client pid: 61951\n[2015-11-16T11:45:12-08:00] INFO: HTTP Request Returned 404 Not Found: Object not found: chefzero://localhost:8889/nodes/coredump.scriptkiddie.org\n[2015-11-16T11:45:12-08:00] WARN: Run List override has been provided.\n[2015-11-16T11:45:12-08:00] WARN: Original Run List: []\n[2015-11-16T11:45:12-08:00] WARN: Overridden Run List: [recipe[audit_test::fail]]\n[2015-11-16T11:45:12-08:00] INFO: Run List is [recipe[audit_test::fail]]\n[2015-11-16T11:45:12-08:00] INFO: Run List expands to [audit_test::fail]\n[2015-11-16T11:45:12-08:00] INFO: Starting Chef Run for coredump.scriptkiddie.org\n[2015-11-16T11:45:12-08:00] INFO: Running start handlers\n[2015-11-16T11:45:12-08:00] INFO: Start handlers complete.\n[2015-11-16T11:45:12-08:00] INFO: HTTP Request Returned 404 Not Found: Object not found: \n[2015-11-16T11:45:12-08:00] INFO: Loading cookbooks [audit_test@0.0.0]\n[2015-11-16T11:45:12-08:00] INFO: Skipping removal of obsoleted cookbooks from the cache\n[2015-11-16T11:45:12-08:00] INFO: Storing updated cookbooks/audit_test/recipes/fail.rb in the cache.\n[2015-11-16T11:45:12-08:00] WARN: Skipping final node save because override_runlist was given\n[2015-11-16T11:45:12-08:00] INFO: Starting audit phase\n[2015-11-16T11:45:12-08:00] INFO: \n[2015-11-16T11:45:12-08:00] INFO: control group without top level control\n[2015-11-16T11:45:12-08:00] INFO:   should fail (FAILED - 1)\n[2015-11-16T11:45:12-08:00] INFO: Successfully executed all `control_group` blocks and contained examples\n[2015-11-16T11:45:12-08:00] INFO: \nFailures:\n\n  1) control group without top level control should fail\n     Failure/Error: DEFAULT_FAILURE_NOTIFIER = lambda { |failure, _opts| raise failure }\n\n       expected: 1\n            got: 0\n\n       (compared using ==)\n     # /tmp/chef_repo20151116-61897-1wkrjt6/config/local-mode-cache/cache/cookbooks/audit_test/recipes/fail.rb:3:in `block (2 levels) in from_file'\n\n[2015-11-16T11:45:12-08:00] INFO: \nFinished in 0.00168 seconds (files took 0.15028 seconds to load)\n1 example, 1 failure\n\nFailed examples:\n\nrspec  # control group without top level control should fail\n\n[2015-11-16T11:45:12-08:00] INFO: Chef Run complete in 0.180314964 seconds\n[2015-11-16T11:45:12-08:00] INFO: Skipping removal of unused files from the cache\n[2015-11-16T11:45:12-08:00] INFO: Running report handlers\n[2015-11-16T11:45:12-08:00] INFO: Report handlers complete\n[2015-11-16T11:45:12-08:00] INFO: HTTP Request Returned 404 Not Found: Object not found: \n[2015-11-16T11:45:12-08:00] FATAL: Stacktrace dumped to /tmp/chef_repo20151116-61897-1wkrjt6/config/local-mode-cache/cache/chef-stacktrace.out\n[2015-11-16T11:45:12-08:00] ERROR: Found 1 errors, they are stored in the backtrace\n[2015-11-16T11:45:13-08:00] FATAL: Chef::Exceptions::ChildConvergeError: Chef run process exited unsuccessfully (exit code 1)\n" to include "Failure/Error: expect(2 - 2).to eq(1)"
       Diff:
       @@ -1,2 +1,54 @@
       -Failure/Error: expect(2 - 2).to eq(1)
       +[2015-11-16T11:45:11-08:00] INFO: Started chef-zero at chefzero://localhost:8889 with repository at /tmp/chef_repo20151116-61897-1wkrjt6
       +  One version per cookbook
       +
       +[2015-11-16T11:45:11-08:00] INFO: Forking chef instance to converge...
       +[2015-11-16T11:45:11-08:00] INFO: *** Chef 12.5.1 ***
       +[2015-11-16T11:45:11-08:00] INFO: Chef-client pid: 61951
       +[2015-11-16T11:45:12-08:00] INFO: HTTP Request Returned 404 Not Found: Object not found: chefzero://localhost:8889/nodes/coredump.scriptkiddie.org
       +[2015-11-16T11:45:12-08:00] WARN: Run List override has been provided.
       +[2015-11-16T11:45:12-08:00] WARN: Original Run List: []
       +[2015-11-16T11:45:12-08:00] WARN: Overridden Run List: [recipe[audit_test::fail]]
       +[2015-11-16T11:45:12-08:00] INFO: Run List is [recipe[audit_test::fail]]
       +[2015-11-16T11:45:12-08:00] INFO: Run List expands to [audit_test::fail]
       +[2015-11-16T11:45:12-08:00] INFO: Starting Chef Run for coredump.scriptkiddie.org
       +[2015-11-16T11:45:12-08:00] INFO: Running start handlers
       +[2015-11-16T11:45:12-08:00] INFO: Start handlers complete.
       +[2015-11-16T11:45:12-08:00] INFO: HTTP Request Returned 404 Not Found: Object not found:
       +[2015-11-16T11:45:12-08:00] INFO: Loading cookbooks [audit_test@0.0.0]
       +[2015-11-16T11:45:12-08:00] INFO: Skipping removal of obsoleted cookbooks from the cache
       +[2015-11-16T11:45:12-08:00] INFO: Storing updated cookbooks/audit_test/recipes/fail.rb in the cache.
       +[2015-11-16T11:45:12-08:00] WARN: Skipping final node save because override_runlist was given
       +[2015-11-16T11:45:12-08:00] INFO: Starting audit phase
       +[2015-11-16T11:45:12-08:00] INFO:
       +[2015-11-16T11:45:12-08:00] INFO: control group without top level control
       +[2015-11-16T11:45:12-08:00] INFO:   should fail (FAILED - 1)
       +[2015-11-16T11:45:12-08:00] INFO: Successfully executed all `control_group` blocks and contained examples
       +[2015-11-16T11:45:12-08:00] INFO:
       +Failures:
       +
       +  1) control group without top level control should fail
       +     Failure/Error: DEFAULT_FAILURE_NOTIFIER = lambda { |failure, _opts| raise failure }
       +
       +       expected: 1
       +            got: 0
       +
       +       (compared using ==)
       +     # /tmp/chef_repo20151116-61897-1wkrjt6/config/local-mode-cache/cache/cookbooks/audit_test/recipes/fail.rb:3:in `block (2 levels) in from_file'
       +
       +[2015-11-16T11:45:12-08:00] INFO:
       +Finished in 0.00168 seconds (files took 0.15028 seconds to load)
       +1 example, 1 failure
       +
       +Failed examples:
       +
       +rspec  # control group without top level control should fail
       +
       +[2015-11-16T11:45:12-08:00] INFO: Chef Run complete in 0.180314964 seconds
       +[2015-11-16T11:45:12-08:00] INFO: Skipping removal of unused files from the cache
       +[2015-11-16T11:45:12-08:00] INFO: Running report handlers
       +[2015-11-16T11:45:12-08:00] INFO: Report handlers complete
       +[2015-11-16T11:45:12-08:00] INFO: HTTP Request Returned 404 Not Found: Object not found:
       +[2015-11-16T11:45:12-08:00] FATAL: Stacktrace dumped to /tmp/chef_repo20151116-61897-1wkrjt6/config/local-mode-cache/cache/chef-stacktrace.out
       +[2015-11-16T11:45:12-08:00] ERROR: Found 1 errors, they are stored in the backtrace
       +[2015-11-16T11:45:13-08:00] FATAL: Chef::Exceptions::ChildConvergeError: Chef run process exited unsuccessfully (exit code 1)

     # ./spec/integration/client/client_spec.rb:429:in `block (3 levels) in <top (required)>'

Finished in 2.75 seconds (files took 0.96659 seconds to load)
1 example, 1 failure

Failed examples:

rspec ./spec/integration/client/client_spec.rb:418 # chef-client when the chef repo has a cookbook with only an audit recipe should exit with a non-zero code when there is an audit failure

authors: coderanger danielsdeleo lamont-granquist thommay


text:
Change the allow_downgrade pseudo-default in the package provider superclass to true

package providers that don't know anything about allow_downgrade should
get true by default.
Signed-off-by: Lamont Granquist lamont@scriptkiddie.org

Strangely this got added in c196fa94b5353402916d96b6b92ddb6a1830ab25 almost 4 years ago.


text:
usb/ip code?

Hi,
I found your post here: https://esp32.com/viewtopic.php?p=94610#p94610
I am very interested in this. I am trying to build a reliable software platform for Raspberry Pis and they are a small nightmare. Currently, I'm trying to figure out a way to fully automate testing of new firmware. I have looked for a USB muxer or a SD muxer and they're either hard-to-impossible to find, or are prohibitively expensive.
However, an ESP32-S2 could be a really brilliant solution, combined with some software to do usb/ip.
I could have my automation server serve up a new firmware image, serve it over gadgetfs, and then use usb/ip to get over to the esp32-s2 connected to the Raspberry Pi USB otg port.
This is assuming that OTG + gadgetfs play nice, but it sounds like you were having some successes with it anyway.
I'd love any details, any code, thought about my described use case. Thanks!

Hi,
its hard to say if it will works with RPi as gadgetfs, also this code is not open source yet, sorry.
Im not fully understand your needs, but maybe second RPi with https://virtualhere.com/ may be helpful.
It is also USBIP implemented on many platforms with own client/server.

Thanks for the tip, I am currently using Raspberry Pi Zeros for this, but it's a bit pricey.
CherryUSB claims to have USB/IP working, though I can't really understand it all due to me only knowing English.

authors: chegewara colemickens

repo: chegewara/EspTinyUSB


text:
Pair-programming review

Fantastic job is done so far! kindly follow the code reviewer's comment to implement the needful because it will help you in the long run

It was implemented.
Thank you

authors: Ellyboi chelobotix

repo: chelobotix/Portafolio-Website


text:
Time Requirements of Sampling Watermarked Dataset

Thanks for the great work and resources!
I'm currently following your work. I wonder how much time is needed to generate these watermarked datasets (e.g. this one)?

On one A100 80GB, it should take somewhere around 24-30 hours to generate the full dataset.

authors: chenchenygu skpig

repo: chenchenygu/watermark-learnability


text:
onRest improvements?

Possible to get the final styles from param in onRest function?
Codepen link
This codepen example, there is no real way to know if the slider ended on the left side or right side if you press the button continuously.
How about implementing into TransitionMotion and StaggeredMotion?
...
<TransitionMotion
  willLeave={this.willLeave}
  styles={this.state.items.map(item => ({
    key: item.key,
    style: {width: item.size, height: item.size},
  }))}
  onRest={
    (styleOnRestAll) => { console.log('all the animations are done, heres the final styles: ', styleOnRestAll) }
  }
  onRests={this.state.items.map(item => ({
    key: item.key,
    cb: (styleOnRest, styleOnRestAll) => { console.log(`do something after this ${item.key} is complete`, styleOnRest, styleOnRestAll); }
  }))}>
  {interpolatedStyles =>
    // first render: a, b, c. Second: still a, b, c! Only last one's a, b.

      {interpolatedStyles.map(config => {
        return

      })}

  }
</TransitionMotion>
...

onRests -> cb would also be able to access styleOnRestAll in case it needs to do some stuff when a certain element animation value hits a sweet spot.
Maybe even add a new prop to the stylesObject, something like "isAnimating" or "isComplete"?

At first blush, I'm not opposed to passing the final styles into onRest.  Shouldn't be too hard.  You'd probably just pass this.state.currentStyle into onRest.
The ergonomics of the proposal for TransitionMotion look gross, but I haven't played with TransitionMotion, so I don't have much input there.

Sorry, I haven't used TransitionMotion.  I no longer use React Motion, and I'm traveling right now, so I don't think I'll be much help.

I have a PR for adding a simple onRest to TransitionMotion #552, which could be a first step towards what you are asking for.

authors: appsforartists beckend ollwenjones

repo: chenglou/react-motion


text:
🛑 tt is down

In fd761f3, tt (https://telebot-app-serverless-i7z8-chenhaonanniubi.vercel.app/checkhealth) was down:

HTTP code: 500
Response time: 2453 ms

Resolved: tt is back up in 605b9ce after 10 minutes.

repo: chenhaonanniubi/upptime


text:
🛑 tt is down

In ee0da9f, tt (https://telebot-app-serverless-i7z8-chenhaonanniubi.vercel.app/checkhealth) was down:

HTTP code: 504
Response time: 10199 ms

Resolved: tt is back up in c9a2dda after 21 minutes.


text:
🛑 tt is down

In b64497b, tt (https://telebot-app-serverless-i7z8-chenhaonanniubi.vercel.app/checkhealth) was down:

HTTP code: 500
Response time: 3223 ms

Resolved: tt is back up in ff9a5ba.


text:
🛑 tt is down

In 9641e76, tt (https://telebot-app-serverless-i7z8-chenhaonanniubi.vercel.app/checkhealth) was down:

HTTP code: 500
Response time: 4064 ms

Resolved: tt is back up in fe61580 after 10 minutes.


text:
🛑 Chequeo Colectivo is down

In 7e6d53b, Chequeo Colectivo (https://chequeado.com/colectivo) was down:

HTTP code: 502
Response time: 9062 ms

Resolved: Chequeo Colectivo is back up in 73238ae.

repo: chequeado/status


text:
feat: add product_exists property to the line item fields

Additions

Add the 'product_exists' property to the line item fields. This is a field that exists only on the response object of order and defines if a product is a temp product/item or not

Documentation
Response payload
more on product_exists

@h2akim

@chewlim

repo: chewlim/shopify-php-sdk


text:
Scraping from website

Added scraping of category, date, publication, blurb, all links in the blurb, and the article link for both bullet-pointed categories and non-bullet-pointed categories. Also added pandas and openpyxl for putting the data into a spreadsheet.

@SuragNuthulapaty looking good! did you want a review of this pull request or should we review it together tonight?

I was thinking we would review it later tonight

The data in the spreadsheet seems good to me. One small comment: is there a built-in python lib we could use instead of importing pandas?

@derekeder i just committed what we were working on yesterday if you want to take a look at it

repo: chihacknight/energy-news-roundup


text:
Njit parallelization

Added parallelization to individual fdmt loops using numba, for a factor of ~2 speed improvement.
Additionally, added a first setup for poetry.

@chrisfandrade16 I suppose we also want to merge pre-commit and the respective hooks?

Nice to see the line profiler, interesting that so much time is in the init step. Agreed that we can definitely speed it up (as you said, DTs is linked to num_dms, which I think in unnecessary), but let's do that as a future to do

authors: larskuenkel ramain

repo: chime-sps/FDMT


text:
fix(ws): internal server upgrade

fixes #335

Coverage increased (+0.01%) to 93.371% when pulling e17622526932e1498536cef66027f83756170f06 on fix-ws-internal-server-upgrade into 616f60fbd7c6493fdca4e59af78d93b58c65a85b on master.

authors: chimurai coveralls

repo: chimurai/http-proxy-middleware


text:
HMAC DRBG with SHA-384 is not FIPS 140-3 compliant

In my understanding Caliptra uses HMAC-SHA-384 DRBG based on the spec.
As of May 2023, for FIPS 140-3 NIST's implementation guidance (attached) states that only SHA-1, SHA-256, or SHA-512 should be used in HMAC_DRBG implementations.
From what I've read in the documentation, it seems like Caliptra is striving to be a FIPS 140-3 compliant implementation.  What can be done to reconcile this problem?

This was identified last month, and an issue was filed for it.
https://github.com/chipsalliance/caliptra-rtl/issues/526
Thank you for bringing this to our attention.

authors: JohnTraverAmd dbids

repo: chipsalliance/Caliptra


text:
Consider updating to Rust 1.74 for fuzzer features

From what I can tell, Rust 1.74 no longer requires using nightly to build our fuzz targets. This should make the CI quite a bit simpler.

I lied. The -Z flag is only available on nightly, so when you run cargo fuzz run ... it fails.

repo: chipsalliance/caliptra-sw


text:
Remove deprecation note from ListLookup

The comment on ListLookup documentation does not reflect current use of the API. As seen, the it's a pattern in use for Decoders on multiple cores like:

DinoCPU: https://github.com/jlpteaching/dinocpu/blob/124cc110410f7daae924301c1e33157f6b02d3d5/src/main/scala/components/control.scala#L50
ChiselV: https://github.com/carlosedp/chiselv/blob/f73d654b2d50546d87e565ca4b152871d572fec8/src/main/scala/Decoder.scala#L29
Chiselwatt: https://github.com/antonblanchard/chiselwatt/blob/c8336341125c146336b2ddfb104382733e08fc19/src/main/scala/Control.scala#L296

And many others.
Contributor Checklist

[x] Did you add Scaladoc to every public function/method?
[x] Did you add at least one test demonstrating the PR?
[x] Did you delete any extraneous printlns/debugging code?
[x] Did you specify the type of improvement?
[x] Did you add appropriate documentation in docs/src?
[x] Did you state the API impact?
[x] Did you specify the code generation impact?
[x] Did you request a desired merge strategy?
[x] Did you add text to be included in the Release Notes for this change?

Type of Improvement

documentation

API Impact
No impact, documentation change.
Backend Code Generation Impact
No impact.
Desired Merge Strategy

Squash: The PR will be squashed and merged (choose this if you have no preference.

Release Notes
Reviewer Checklist (only modified by reviewer)

[ ] Did you add the appropriate labels?
[ ] Did you mark the proper milestone (Bug fix: 3.3.x, [small] API extension: 3.4.x, API modification or big change: 3.5.0)?
[ ] Did you review?
[ ] Did you check whether all relevant Contributor checkboxes have been checked?
[ ] Did you mark as Please Merge?

I'd like to deprecate this API, asking user switching to decoder API. Since this API will introduce a cascade Mux rather than parallel pla, which can be the critical path in most of CPU designs. So I still encourage designers switching to decoder for the this performance issue.

@sequencer  Would you mind making a PR on @carlosedp chiselv design to show how to move to the decoder API? Then we could always point people to your change to show how to migrate.

@ekiwi sure, I'll work on this today.

A deprecation PR #2259 is sent, fell free to open this if @carlosedp has more questions ;p

authors: carlosedp ekiwi sequencer


text:
On Windows, use c++20 mode as correctly needed for designated initial…

…izers.
Designated initializers is technically only supported starting with c++20, and the compiler on Windows points that out with an error.

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

Comparison is base (cfcbb82) 92.84% compared to head (0196432) 92.84%.
Report is 2 commits behind head on master.

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

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2049   +/-   ##
=======================================
  Coverage   92.84%   92.84%
=======================================
  Files         357      357
  Lines       26396    26396
=======================================
  Hits        24507    24507
  Misses       1889     1889

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

authors: codecov-commenter hzeller

repo: chipsalliance/verible


text:
How to render input form ?

I need form fields,  how to render input and radio form and others ?

Just writing original HTML in markdown files.

Closing as out of scope. See #956 and targeted specifications.

authors: hojas joshbruce minwe

repo: chjj/marked


text:
Support arrow2

Since polars switched to arrow2, serde_arrow no longer works with polars.
Questions to sort out:

Support arrow / arrow2 in the same dependecy?

Alternative: require users to include multiple versions of serde_arrow

Initial plan of attack:

Isolate all arrow interaction into separate modules
Investigate how to support different arrow "backends" without code duplication

Branch: https://github.com/chmp/serde_arrow/tree/feature/arrow2

Implemented on the arrow2 branch


text:
Double sigmoid inside RNNCell

I was looking at the implementation of the DCGRUCell, and I've spotted something out of order. If we are using 'fc' for  U and R gates we are going to apply the sigmoid twice (line 121 and  96). Is this how it's intended to work, or is there a bug?

could you share a screenshot?

could you share a screenshot?

@yuqirose Sure. If we look at first image on line 96 the value is sigmoid(fn(...)).

Now if we look at the second picture (line 21) we are applying again sigmoid.

Am I missing something?

@yuqirose Sure. If we look at first image on line 96 the value is sigmoid(fn(...)).

Now if we look at the second picture (line 21) we are applying again sigmoid.

Am I missing something?

@razvanc92  line 121 is applied to Wx, line 96 is applied to f(Wx) + b

@yuqirose Thank you for your replay. I understand what the code does, but if we are doing this we are applying sigmoid twice to Wx, which is different from the original implementation.


text:
(ffmpeg*) descriptions missing codecs/libraries included in each

The packages "ffmpeg" and "ffmpeg-full" descriptions make no reference to the libraries included in each.
They instead refer to "essentials" and "full", neither of which exist in the upstream ffmpeg project at www.ffmpeg.org.
Expected Behavior

Be able to look at package description and see the differences between the packages.
or
Have the package description link to a site that shows the differences

Current Behavior
"ffmpeg" is described as the "essentials ffmpeg version"
"ffmpeg-full" is described as the "full ffmpeg version"
Possible Solution
The packages seem to be using builds from gyan.dev, so a reference to https://www.gyan.dev/ffmpeg/builds/#libraries could suffice, since it lists the included libraries for each there.

@slycordinator that is a good catch that we somehow had missed.
If you are interested, I have marked this issue as Up for grabs if you are willing to provide a PR to get the link containing the libraries added (only the Readme.md file need to be updated, and should be in a ## Notes section in that file).

authors: AdmiringWorm slycordinator


text:
(bleachbit.install) fix installer removal

Description
Changes $toolsPath to $toolsDir, because $toolsPath is not defined in this package.
Motivation and Context
Fixes #1850
How Has this Been Tested?
Installed in the test environment and ensured that the installer was removed.
Screenshot (if appropriate, usually isn't needed):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Migrated package (a package has been migrated from another repository)

Checklist:

[x] My code follows the code style of this repository.
[ ] My change requires a change to documentation (this usually means the notes in the description of a package).
[ ] I have updated the documentation accordingly (this usually means the notes in the description of a package).
[ ] I have updated the package description and it is less than 4000 characters.
[x] All files are up to date with the latest Contributing Guidelines
[x] The added/modified package passed install/uninstall in the chocolatey test environment.
[x] The changes only affect a single package (not including meta package).

:white_check_mark: Package verification completed without issues. PR is now pending human review

@TheCakeIsNaOH awesome work! Thanks for your contribution!

authors: AppVeyorBot TheCakeIsNaOH pauby


text:
ChocolateyGUI Icon

Hello,
ChocolateyGUI icon can't be loaded because of the icon file removed:
https://cdn.rawgit.com/chocolatey/choco/8b6217061a4efe6543622e903996bb8eac012db2/docs/images/Choco_Icon.png

@xmha97 thank you for raising this issue.  This has actually been addressed here:
https://github.com/chocolatey/ChocolateyGUI/issues/558
And it will ship in the next release of Chocolatey GUI.

repo: chocolatey/ChocolateyGUI


text:
Improve performance of search page

The search page now fetches all the name of trending packages and their icons every single time I want to search for a package. While this page changes hardly ever. From the time I remember, the list of these trending packages has not changed for months or years.
Suggestions:

It would be better to let the users search for a package rather than blocking them by loading the trending packages.

Loading the trending page should become asynchronous.

The trending page shall be cached (its icons, names, etc), and only updated once a week. This page changes hardly ever.

Use a faster fetching/downloading method. The chocolatey webpage opens instantly but this GUI app is very slow in downloading. This is very strange since the website uses JavaScript, but the GUI app is written in C#!

This issue is related to https://github.com/chocolatey/ChocolateyGUI/issues/777
Screenshots

@aminya can I please ask that you try the latest pre-release version of Chocolatey GUI 0.18.0.  The instructions are on the readme:
https://github.com/chocolatey/ChocolateyGUI#installation
This includes a couple of new options that I think will help in this regard, specifically:
https://github.com/chocolatey/ChocolateyGUI/issues/769
https://github.com/chocolatey/ChocolateyGUI/issues/706

There is no 0.18.0
❯ choco install chocolateygui --source myget.org/F/chocolateygui --pre
Chocolatey v0.10.15
Installing the following packages:
chocolateygui
By installing you accept licenses for the packages.
chocolateygui v0.17.2 already installed.
 Use --force to reinstall, specify a version to install, or try upgrade.

Chocolatey installed 0/1 packages.
 See the log for details (C:\ProgramData\chocolatey\logs\chocolatey.log).

Warnings:
 - chocolateygui - chocolateygui v0.17.2 already installed.
 Use --force to reinstall, specify a version to install, or try upgrade.

Ah, you will need to use the following:
choco upgrade chocolateygui --source myget.org/F/chocolateygui --pre

Why don't you just release this on chocolatey itself?
Same error?
❯ choco upgrade chocolateygui --source myget.org/F/chocolateygui --pre
Chocolatey v0.10.15
Upgrading the following packages:
chocolateygui
By upgrading you accept licenses for the packages.

chocolateygui was not found with the source(s) listed.
 If you specified a particular version and are receiving this message, it is possible that the package name exists but the version does not.
 Version: ""; Source(s): "myget.org/F/chocolateygui"

Chocolatey upgraded 0/1 packages.
 See the log for details (C:\ProgramData\chocolatey\logs\chocolatey.log).

Warnings:
 - chocolateygui - chocolateygui was not found with the source(s) listed.

The complete command, taken directly from the readme page, would be the following:
choco upgrade chocolateygui --source https://www.myget.org/F/chocolateygui/ --pre

We don't push continuous integration builds of Chocolatey GUI onto the chocolatey.org feed.  They only go to the MyGet feed.

Well, I tried this command. It gives the error in https://github.com/chocolatey/ChocolateyGUI/issues/789#issuecomment-691261960

Yes, 100%.
I have just run this command, and it results in the following:

Ah, the error was because of the "Refined GitHub" browser extension. It shortened the link to myget.org/F/chocolateygui which was not working 🤦♂

I tried 0.18.0. There is no noticeable difference. My suggestions are still applicable.

Just to confirm, you enabled both of the new settings that are available in the new version?

Once enabled, when navigating to the chocolatey source, you should see the following:

i.e. it openly immediately

After doing that it finally works! Thanks!
I don't usually change my settings, and I think this should become the default behavior.

@aminya said...
(or as I said just separate the search bar logic from the loading logic by running them on separate threads or making them async).

This is an open source project.  If you are interested in seeing this change made, we would welcome a contribution.

This is an open source project. If you are interested in seeing this change made,

For sure! That's why I opened this issue.

, we would welcome a contribution.

Unfortunately, I don't speak C# myself.

This is a duplicate of #595.

authors: aminya gep13 pauby


text:
Update Roadmap Page

Roadmap page needs some updates as some of the things have been released such as Dark Mode for Chocolatey GUI. Likely needs a lot of dates moved as well. Also noticing no mention of Intune Integration.
Link to current roadmap page: https://docs.chocolatey.org/en-us/roadmap#mainContent

Need to go through and link to a public GitHub issue for each of the things listed on the roadmap page.

I believe https://docs.chocolatey.org/en-us/roadmap#mainContent requires an update for 2024 and potentially for 2025

repo: chocolatey/docs


text:
Clarify usage of underscores in package IDs on the Community Repository

What New Or Updated Would You Like To See?
Explicit instructions about when underscores (_) are allowed in package IDs for packages uploaded to the Community Repository.
I see two places where this should be updated:
https://github.com/chocolatey/docs/blob/master/input/en-us/community-repository/moderation/package-validator/rules/cpmr0070.md
https://github.com/chocolatey/docs/blob/master/input/en-us/create/create-packages.md#naming-your-package
Why Is It Needed?
There is no documentation about when underscores may be used for package ID's.
Additional Context?
Going by the package validator rule, underscores are acceptable when they are part of a pre-existing package ID.
However, what about for new packages?

Should they never be acceptable?
Should they be acceptable only when the software name has an underscore?
Should they be acceptable only when the package for the software on another package manager has an underscore?
Should they be acceptable if the software name has an underscore and the package on another manager has an underscore?

This should be decided before the documentation is updated. I personally don't care which option is picked, as they all have upsides and downsides.

Underscores, in my opinion, shouldn't be used in new packages. The only exception would be this one:

Should they be acceptable if the software name has an underscore and the package on another manager has an underscore?

I agree with @pauby that understores should not be allowed in new packages, except for when:

Should they be acceptable only when the software name has an underscore?

While there could be a case being made if a different package manager uses underscores for the package id, i would still say in general the only exception is if the software itself uses an underscore for its name.

authors: AdmiringWorm TheCakeIsNaOH pauby


text:
Update devtools miniconda url

Why:
Currently, builds are experiencing an md5 checksum issue.
This command is broken due to some change on either the webpage, or perhaps
curl, leading to an empty md5 sum to check against.
This change addresses the need by:

Updating the url to https.

There might still be some issues related to changes on the OpenEye side of things.

I can't tell if some of the build errors are because of a new bug, or because the pull-request was merged prematurely.

I can't tell if some of the build errors are because of a new bug, or because the pull-request was merged prematurely.

You had just changed http to https, and it succeeded in correctly grabbing miniconda in one branch, so I merged it so we wouldn't be waiting around all day.

The new failures are because ambermini  16.16 is now being pulled:
https://github.com/choderalab/openmoltools/issues/235

authors: bas-rustenburg jchodera

repo: choderalab/openmoltools


text:
We cannot copy.deepcopy Topology object

This is currently done in NullProposalEngine.propose and in PolymerProposalEngine.propose. See https://github.com/choderalab/perses/pull/281 for a detailed explanation of why we cannot do this and what the fix is.
Using deepcopy on Topology objects causes Perses to fail with openmm 7.1.0 builds.
perses will not run correctly on openmm 7.0.1 because of known bugs in CustomIntegrator. We must use 7.1.0.

I also cherry-picked this commit into benchmark in a new branch https://github.com/choderalab/perses/tree/fix-benchmark

repo: choderalab/perses


text:
Use io.StringWriter

Benchmarks against the previous implementation
new-iowriter.txt
new-stringwriter.txt
old.txt
benchstat old=old.txt new=new-stringwriter.txt
goos: darwin
goarch: arm64
pkg: github.com/choonkeat/dom-go
               │     old     │                 new                 │
               │   sec/op    │   sec/op     vs base                │
AttrHTML-8       123.9n ± 2%   157.5n ± 1%  +27.12% (p=0.000 n=10)
NodeHTML-8       520.4n ± 0%   670.9n ± 2%  +28.91% (p=0.000 n=10)
HtmlTemplate-8   3.312µ ± 1%   3.378µ ± 1%   +1.98% (p=0.001 n=10)
geomean          597.8n        709.3n       +18.67%

           │     old     │                 new                 │
           │  AttrHTML   │  AttrHTML    vs base                │
AttrHTML-8   9.479M ± 4%   7.499M ± 7%  -20.89% (p=0.000 n=10)

               │     old      │                  new                   │
               │     B/op     │     B/op      vs base                  │
AttrHTML-8         128.0 ± 0%     160.0 ± 0%  +25.00% (p=0.000 n=10)
NodeHTML-8         456.0 ± 0%     800.0 ± 0%  +75.44% (p=0.000 n=10)
HtmlTemplate-8   1.352Ki ± 0%   1.352Ki ± 0%        ~ (p=1.000 n=10) ¹
geomean            432.3          561.6       +29.92%
¹ all samples are equal

               │    old     │                 new                  │
               │ allocs/op  │ allocs/op   vs base                  │
AttrHTML-8       4.000 ± 0%   5.000 ± 0%  +25.00% (p=0.000 n=10)
NodeHTML-8       11.00 ± 0%   16.00 ± 0%  +45.45% (p=0.000 n=10)
HtmlTemplate-8   53.00 ± 0%   53.00 ± 0%        ~ (p=1.000 n=10) ¹
geomean          13.26        16.19       +22.05%
¹ all samples are equal

           │     old     │                 new                 │
           │  NodeHTML   │  NodeHTML    vs base                │
NodeHTML-8   2.297M ± 1%   1.797M ± 3%  -21.77% (p=0.000 n=10)

               │     old      │              new               │
               │ HTMLTemplate │ HTMLTemplate  vs base          │
HtmlTemplate-8    354.2k ± 0%    351.9k ± 2%  ~ (p=0.631 n=10)

the numbers don't look good, huh :-(
my understanding is that the goal is to get func (e Node) Write(w io.StringWriter) where we can supply our own io.StringWriter
how about we instead expose our internal helper
-func (a Attribute) buildHTML(sb *strings.Builder) *strings.Builder {
+func (a Attribute) HTMLBuilder(sb *strings.Builder) *strings.Builder {

so instead of only being able to call HTML(), now we have the choice to provide our strings.Builder

however with

BenchmarkNodeHTMLBuildMany : doing HTMLBuilder(&sb); HTMLBuilder(&sb); HTMLBuilder(&sb)
BenchmarkNodeHTMLMany: doing HTML() + HTML() + HTML()

HTMLBuilder(sb *strings.Builder) *strings.Builder doesn't look like it outperforms by much
BenchmarkAttrHTML-8            	 9900267	       121.1 ns/op	   9900267 AttrHTML	     128 B/op	       4 allocs/op
BenchmarkNodeHTML-8            	 2326978	       514.4 ns/op	   2326978 NodeHTML	     456 B/op	      11 allocs/op
BenchmarkNodeHTMLBuildMany-8   	  790669	      1490 ns/op	    790669 NodeHTML	    1640 B/op	      25 allocs/op
BenchmarkNodeHTMLMany-8        	  744154	      1591 ns/op	    744154 NodeHTML	    1656 B/op	      34 allocs/op
BenchmarkHtmlTemplate-8        	  357118	      3330 ns/op	    357118 HTMLTemplate	    1384 B/op	      53 allocs/op

then I close this PR? maybe relook at it if we hit any performance issues during load tests?

👌

authors: choonkeat yongchongye

repo: choonkeat/dom-go


text:
Use the block crate and functions that take blocks instead of boxing callbacks.

Hey Christian,
you would be interested in a PR that adds the block crate and replaces calls to MIDIInputPortCreate with a call to MIDIInputPortCreateWithBlock and call to MIDIClientCreate with call to MIDIClientCreateWithBlock . It would make the codebase easier to understand. I just tried a basic proof of concept and it seems to work.

Hi @adamnemecek, sorry for the late response, I completely missed this issue.
I'm wondering if you are still interested in this topic.
If so, would you please explain the motivation for that change and the advantages?
Thanks

It's been a while since I touched this but there's a crate called block which lets you use objective-c blocks from rust.
I think that this should work mostly
https://github.com/adamnemecek/wmidi_coremidi/blob/4c927e9cafd5ca59061a0a5c450827a61e29664f/src/midi_input.rs#L259

@adamnemecek I appreciate your time, thanks 😉
I have a working solution in #36 so I will close this issue.

authors: adamnemecek chris-zen

repo: chris-zen/coremidi


text:
Use of deprecated methods in onSingleTapConfirmed DefaultOnDoubleTapListener

After deprecating the methods getOnPhotoTapListener and getOnViewTapListener, the source code hasn't been modified to accommodate these changes. I can fix this in the future, but I don't have much time over the next few weeks. So if anyone wants to fix this in the meantime, please feel free to, but I just wanted to bring it to your attention.

What do you mean the source code hasn't changed?

As both these methods have now been deprecated (and my assumption is that these will be removed in a build in the future), any use of these methods should be removed from the source code and be modified in order to work as it currently does. In the current state, the source code hasn't been modified to remove all the references to these methods.

Ahh, I see. Yes, this does need to be done. Thanks

I was just about to remove the internal dependency on getOnPhotoTapListener() and getOnViewTapListener() but it appears that you have un-deprecated(?) these methods. Has the consensus on the deprecation of these two methods changed? It is possible just to convert these methods to package local, so the benefits of not exposing this method to the user is gained and another benefit is the lack of use of deprecated methods in the library source.

Yeah, making them package local is a good idea. Go for it. I am thinking that the next release with be a major release so feel free to modify as needed

authors: Jawnnypoo veyndan

repo: chrisbanes/PhotoView


text:
Serve HLS segments from /hls to make relative playlist entries work properly

When initiating the HLS stream off of /media-player/:id/hls/now, the relative segments are requested from '/media-player/:id/hls/segment' -- we should allow segments to be loaded from the /hls prefix in addition to the stream prefix. More context can be found at #334

I'm not sure that this is quite right - with the playlist itself moving up a level, the segments also need to move up a level to be at the same level as the playlist, which they are not currently.
mpv 'http://tunarr:8000/media-player/38f2c3df-3c05-4155-8e23-87762ded456b/hls'
[ffmpeg] Not detecting m3u8/hls with non standard extension and non standard mime type
[ffmpeg] Not detecting m3u8/hls with non standard extension and non standard mime type
Playing: http://tunarr:8000/media-player/38f2c3df-3c05-4155-8e23-87762ded456b/data00000.ts
[ffmpeg] http: HTTP error 404 Not Found
Failed to open http://tunarr:8000/media-player/38f2c3df-3c05-4155-8e23-87762ded456b/data00000.ts

Interesting. This worked well for me in my local tests. I’ll take a look.
On Fri, Apr 19, 2024 at 2:10 PM Jason Dove @.***> wrote:

I'm not sure that this is quite right - with the playlist itself moving up
a level, the segments also need to move up a level to be at the same level
as the playlist, which they are not currently.
mpv 'http://tunarr:8000/media-player/38f2c3df-3c05-4155-8e23-87762ded456b/hls'
[ffmpeg] Not detecting m3u8/hls with non standard extension and non standard mime type
[ffmpeg] Not detecting m3u8/hls with non standard extension and non standard mime type
Playing: http://tunarr:8000/media-player/38f2c3df-3c05-4155-8e23-87762ded456b/data00000.ts
[ffmpeg] http: HTTP error 404 Not Found
Failed to open http://tunarr:8000/media-player/38f2c3df-3c05-4155-8e23-87762ded456b/data00000.ts
—
Reply to this email directly, view it on GitHub
https://github.com/chrisbenincasa/tunarr/issues/339#issuecomment-2067064030,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMQRX3TVDXR6G23FFCEKRTY6FMZ3AVCNFSM6AAAAABGPNZ6CGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRXGA3DIMBTGA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

I think there were just too many paths flying around here... let me know what you think of this: https://github.com/chrisbenincasa/tunarr/pull/346
We should just standardize on a single path and make sure all entrypoints to the stream align with that.

https://github.com/chrisbenincasa/tunarr/pull/346 merged

authors: chrisbenincasa jasongdove

repo: chrisbenincasa/tunarr

license: Zlib


text:
UI Updates

This PR addresses part of #8

[x] with the new typeface, x-axis labels sometimes overrun (see attached image)
[x] charts have overlapping text with 1 or 2 sites (see attached image)
[x] UI approach for lots of presets. Rob has lots of great ideas for presets, and it'd be a cool way of letting users "explore" the data. I think we could reasonably have 30+ presets, so we wouldn't want to show them all at once
[ ] add the origin (0) to the charts

This PR:

split out App.tsx into many components
add styled-components with corresponding theme and global styles
design a better information architecture

“split out App.tsx into many components”
Yay finally!!

finally checked this out locally. i love the look on desktop. i'll dive in to the code soon!

@paulathevalley i love the code organization and layout. And IBM Plex Mono is great—how did i not know about this?
Let's make a feature branch and merge this in there. I'll make that branch now, and add you as a collaborator to this repo.
Grab-bag of feedback...

Mobile - love the mobile view you put together on the current site. I don't want to push to prod yet because I want to have a mobile view. I don't think people will use the site a lot on mobile, but many folks' first impression will be on mobile.
starting a new list is hard (or i think impossible?). when you clear the current list, you don't have a way to add a site.
"My Stuff" - maybe that could be "My Lists", then it'd need less explanation. but maybe you were planning for other type of "stuff" to be saved?
I associate "🔍" with searching. I wonder if we should just drop the emoji altogether and have highlighting simply change the background color of the bar on the chart (like it already does) and do the same thing for the site in the sidebar. I think that parity would tie things together. maybe?

also, are you having fun?

made you a collaborator on the repo! and made a summer-ui-love branch we can merge this into

Thank you for the feedback!! I'll update the PR to "ready for review" once your feedback is addressed, and there is a reasonable mobile view. I may do something really basic for mobile in the meantime while I think through the classic drawer... I don't think a drawer is possible without making it a modal, which has a lot of implications for accessibility and scrolling. Anyway, that's what I've been thinking about it.
hmm, having fun, having fun... let me get back to you on that.

authors: chrisbolin paulathevalley

repo: chrisbolin/perf-land


text:
Can't install on a new Yesod scaffold (postgres)

Error: While constructing the build plan, the following exceptions were encountered:

In the dependencies for yesod-lucid-0.1:
    yesod-core-1.6.6 from stack configuration does not match (>=1.1.5 && <1.5)  (latest matching
                     version is 1.4.37.3)
needed due to circuit-yesod-0.0.0 -> yesod-lucid-0.1

Some different approaches to resolving this:

  * Set 'allow-newer: true' to ignore all version constraints and build anyway.

  * Consider trying 'stack solver', which uses the cabal-install solver to attempt to find some
    working build configuration. This can be convenient when dealing with many complicated
    constraint errors, but results may be unpredictable.

  * Recommended action: try adding the following to your extra-deps
    in /Users/itama/Sites/circuit-yesod/stack.yaml:

- yesod-core-1.4.37.3

Adding - yesod-core-1.4.37.3 resulted with being asked for more versions, but it couldn't build.
and also stack solver didn't work

Until I/ someone else figures it out, here's how I've included it, using the specific commit:
# stack.yaml

# ...
extra-deps:
  # See https://github.com/chrisdone/yesod-lucid/issues/1
- github: chrisdone/yesod-lucid
  commit: de8b455a6efcc02b95d5f6363a29044e230dc75f

repo: chrisdone/yesod-lucid


text:
Bug: visual line mode not recognized

Make sure you have done the following

[X] I have updated to the latest version of the plugin.
[X] I have read the README.

Bug Description
When starting a sub with in visual line mode, the substitution defaults to normal mode instead of restricting to the range.
Relevant Screenshot
https://github.com/chrisgrieser/nvim-rip-substitute/assets/899235/6abcf94f-da69-4e66-af9b-07f0fdda7d0c
Screen recording taken with my setup, same behaviour happens with the minimal conf you provide in the last box of this form
To Reproduce

select a range in visual line mode
run this mapping: vnoremap <leader>s :lua require("rip-substitute").sub()<cr>
box opens with full buffer substitute setup

neovim version
NVIM v0.10.0
Build type: Release
LuaJIT 2.1.1713484068

ripgrep version
❯ rg --pcre2-version --version
ripgrep 14.1.0

features:-simd-accel,+pcre2
simd(compile):+SSE2,-SSSE3,-AVX2
simd(runtime):+SSE2,+SSSE3,+AVX2

PCRE2 10.43 is available (JIT is available)

Minimal reproducible config
local spec = {
	{
		"chrisgrieser/nvim-rip-substitute",
		opts = {}, -- insert config here
		keys = {
			{
				"gs",
				function() require("rip-substitute").sub() end,
				mode = { "n", "x" },
				desc = "rip substitute",
			},
		},
	},
}
--------------------------------------------------------------------------------
vim.env.LAZY_STDPATH = "./nvim-repro"
load(vim.fn.system("curl -s https://raw.githubusercontent.com/folke/lazy.nvim/main/bootstrap.lua"))()
require("lazy.minit").repro { spec = spec }

Well, you were really unlucky with your timing. That's a bug I accidentally introduced an hour ago, and which I fixed a few minutes after you submitted this bug report 🙈
update to the latest version and this should be fixed

Updated to f39e8c82250549431979d2d18eae3e6b97b46f04 but the issue remains.

are you sure? using f39e8c8, I cannot reproduce the issue with the minimal config.

ah, the issue is that you are using :lua require("rip-substitute").sub() in the RHS of your mapping. The : causes nvim to leave visual mode, which is why the rip-substitute is not able to pick up the correct mode anymore.
Use function() require("rip-substitute").sub() end as RHS of your keymap, and the issue should be fixed.

Ah I see. Well I actually used the minimal config, but I was manually calling require("rip-substitute").sub() since I missed the binding provided and fell back to documentation (it was the only interface the README provided so I went with that).
I see you just added :RipSubstitute to the doc which also works.
Maybe function() require("rip-substitute").sub() end should be added to the README as well since the need for a wrapper function is not really clear.
Thanks for your help!

Ah, yes, it's fully written out in the Installation section, but not the Usage section. Added, thanks.

authors: chrisgrieser mriva

repo: chrisgrieser/nvim-rip-substitute


text:
Add to F-Droid

It'd be great to have this on F-Droid, as there is similar app in there as of yet: http://f-droid.org/

Sure add V3 too if you like?

authors: chrisjenx picsi

repo: chrisjenx/Calligraphy


text:
Video with attribute leads to error

$og->video(                 'https://www.youtube.com/embed/123',                 ['type' => 'text/html']             );
leads to
Attempted to call function "starts_with" from namespace "ChrisKonnertz\OpenGraph".

@chriskonnertz would you mind taking a look?

@chriskonnertz ?

repo: chriskonnertz/open-graph