text:
Keep re-creating resources as long as instance exists

What would you like to be added:
Keep all the components up like statefulset, svc, cm etc. when the instance is still exist.
Why is this needed:
When any of the components was deleted by mal-operation prevention, it's necessary for an operator to get the event and re-create it.

This should be rolled up into a drift detection feature. May be fixed by another issue.

This issue is for drift detection and is Important long term

authors: abmybgx gerred kensipe


text:
Fix markup

What type of PR is this?
/kind documentation
What this PR does / why we need it:
Apparently hugo does not like asterisked bullet points leading to broken formatting.
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

BTW, the comments in the PR description template are outdated:
For reference on required PR/issue labels, read here:
https://github.com/kudobuilder/kudo/blob/master/RELEASE.md

That file does not say anything about labels...

Thanks @porridge

source: gharchive/pull-request

repo: kudobuilder/kudo

license_type: permissive

license_source: github-api


text:
Update compiler name

Summary
Update documentation
Use typescript instead of ttypescript.

It will allow the end-user to copy and paste the options easily.
It will avoid confusion for people who has not too much experience with the JS/TS ecosystem.

Test plan

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

Hi, ttypescript is a custom TypeScript compiler which ts-jest supports, this is their page https://github.com/cevek/ttypescript :) The documentation is correct.

I didn't know about this option, thanks for the clarification.
I'll close the PR.

authors: MDOR ahnpnl

repo: kulshekhar/ts-jest

license: MIT


text:
Interne Issues auf öffentliches Board überführen

Als PV möchte ich, dass alle Issues auf dem Board sichtbar sind.

[ ] Issues übertragen
[ ] Size vergeben
[ ] Prio vergeben
[ ] Epics vergeben

Please review: https://git.dataport.de/groups/kulturfinder/-/boards

https://git.dataport.de/kulturfinder/backend/-/issues/34

source: gharchive/issue

repo: kultursphaere-sh/kulturfinder-redaktion


text:
Add support for a DropboxFile from stream

Sometimes we need to upload a file that isn't on the filesystem but is already a file stream.
This commit adds a new static function CreateByStream that allows us to create a new DropboxFile instance using a stream.
You can now instantiate a new DropboxFile using these static functions:
// create new DropboxFile from stream
$dropboxFile = DropboxFile::createByStream($fileName, $fileStream, DropboxFile::MODE_READ);

// create new DropboxFile from path (same as the constructor but just added in order to match the createByPath function)
$dropboxFile = DropboxFile::createByPath($filePath ,DropboxFile::MODE_READ);

@jnstr Thanks for the contribution :)

license: mit

license_source: bigquery


text:
🎮 Game Request
I want to add color-crush-game using HTML CSS javascript.
Can you please assign me this task under GSSOC'23 contribution @ku
Point down the features
Game points
Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

/assign

Hey @Kalivarapubindusree !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

authors: Kalivarapubindusree kunjgit

repo: kunjgit/GameZone


text:
[New Game]: Not_morethan5 #550

PR Description 📜
Adding a new Game.
!morethan5  is a simple game word guessing game where the has to guess the word by filling the blanks.User has  to guess one letter at a time to fill in all the blanks and user can make only 5 wrong attempts.
Fixes # <550>

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @SusanStorm ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

@kunjgit I have made the changes that you have requested. Please Review this PR

authors: SusanStorm kunjgit


text:
use nocookie domain

Instead of using the normal embed URL, we could use https://www.youtube-nocookie.com/embed/
Perhaps it is only a small privacy enhancement, but it will not hurt to use this domain

This could be offered as an option?
Also, not sure why this ticket was closed the day it was open. @lioman ?

authors: axelsimon lioman


text:
add source information to plots

I was wondering if it would make sense to add some sort of source information to the plots. The pride example shows a PRIDE spetra on top and the Prosit spectra on the bottom. Would be nice to have this information added to the plot. Once you export/save it to disc its not obvious anymore which plots displays what.

Mathias requested that as well. not sure how easy it is but I will figure it out

can be closed; done

authors: tkschmidt tobiasko


text:
[docs] add: docs directory

Add English document lists

Thanks for your contribution!

authors: ifTNT wcwutw

repo: kuwaai/genai-os


text:
Proper support of user-defined constants

And also of rename:

Issue was actually matching constants without a relying on a \b (end of word)

repo: kuzyn/chatscript-tmlanguage


text:
[optim] Bundle JS and CSS files

What does this PR do?
Bundle JS and CSS files
How should this be manually tested?

 Snippets tests reports   SDK Version Tests overview      v6        v5                                   :heavy_minus_sign:                    v1        v1        v1

authors: Njuelle kuzzle


text:
Planning to support datasets with background?

Hi, thanks for open sourcing this amazing work! I was wondering if there is any plan to support more realistic datasets without object masks (e.g. the plain and simple T&T instead of the preprocessed from NSVF)? I guess so because supporting custom datasets (with COLMAP?) is on the TODO list, but asking just in case :) thank you again!

Yes, it's my next step!


text:
Modify request headers

Hi,
I would like to modify request headers in one of my middleware steps. Example:
  const newHeaders = new Headers(request.headers);
  newHeaders.set("X-Super-header", isSuper);

  const modifiedRequest = new Request(request.url, {
    body: request.body,
    headers: request.headers,
    method: request.method,
    redirect: request.redirect
  })

  request = modifiedRequest

I know if I make a return it has to be Response but when reassigning the var request, it's not being changed.

Hey @js0cha - you're basically trying to do something illegal in JavaScript. You can't reassigned a locally scoped variable and affect anything other than the local scope. For example:
function noMutation(a) {
  a = "replaced" // this does nothing aside from replace the locally-scoped reference to a

  console.log(a) // "replaced"... so it worked, right?
}

// let's create a variable
let m = "foo"

// now let's send it to a function like the request in itty
noMutation(m)

console.log(m) // still "foo", because that statement above only affected the local function scope

If you want to modify headers via middleware, you need to mutate the request, not replace it. Example:
const middleware = request => {
  request.headers.set('X-Super-header', true) // we just mutate a property on the existing request
}

Hope this helps!

Yes, i get it. But Cloudflare do not allow us to modify request headers like you proposed = TypeError: can't modify immutable headers

Ah yes gotcha... so in that case, you were so close!
If you need to pass a reformed request onto downstream handlers, just re-embed the new request INTO the existing one (anywhere):
const middleware = request => {
  const newRequest = new Request() // etc etc
  request.newRequest = newRequest
}

const downstream = ({ newRequest }) => {
  // tada!
}

Otherwise, in the scenario you want to just exit with the Request itself (itty has no requirements for WHAT you return, making it universally applicable), just return that new Request directly.  Just make sure (with CF) that something in your request/response handlers you DO return something or you'll crash obv.
const middleware = request => {
  const newRequest = new Request() // etc etc
  return newRequest
}

// assuming your routes are in place

const maybeARequest = await router.handle(request) // might now return your Request

repo: kwhitley/itty-router


text:
Channel label mismatch between Cluster View and Vaveform View

I am encountering a strange issue where the channel label in Cluster and Similarity views do not match the channel labels in the Waveform view (screenshot). For this particular case, the data was generated by KiloSort (test/benchmark data) but I encountered the same issue with the test data set data from the TemplateGUI documentation:
https://codeload.github.com/kwikteam/phy-data/zip/master
Does anyone know why this is happening?

From a comment in #746 it seems that this has been fixed in the development version I will try that.

repo: kwikteam/phy


text:
Extra LeafletInteractiveObservableLayersDirective created when using <lflt-interactive-layers #main="observable">

Each time the following template is used, an unnecessary LeafletInteractiveObservableLayersDirective is created when only LeafletInteractiveObserverLayersDirective should be created.
<lflt-interactive-layers #main="observable">
  ...
<lflt-interactive-layers>

Fixed by b33fd01592d2dececc9c55072854eb3584547990

repo: kygoh/lflt-core


text:
is-svg high priority vulnerability

Hi I see there is a PR to upgrade is-svg
Could this get merged ASAP?
Thanks

would be great it's a high severity vulnerability

Yes, this is needed, please accept PR!

2022, needed, please merge that PR

@penfold45 @Benraay @joetex Feel free to do npm install git@github.com:GabrielBB/multer-s3.git in the meantime. I created a fork with the fix: https://github.com/GabrielBB/multer-s3

I guess no one is maintinaing this or seeing these PR requests any longer :(

authors: Benraay GabrielBB joetex penfold45


text:
Build tags to exclude postgresql engine

What do you want to change?
Would it be acceptable to add a build tag such as disable_postgresql to be able to exclude postgresql code from the build completely?
Currently, it is already stubbed out for windows
https://github.com/kyleconroy/sqlc/blob/58b0b6593fc665600619b0dfc69909c68ff2cda2/internal/engine/postgresql/parse_windows.go#L1
So the change would just be adding something like || disable_postgresl
The postgresl uses heavyweight cgo, resulting in issues like #1956, and likely the reason for the lack of windows support in the first place. Having a build tag would allow avoiding the scary looking warnings when using go run. Hopefully it's ok given the one-line nature of the change (could prefix experimental_ or something if not wanting to support indefinitely).
What database engines need to be changed?
PostgreSQL
What programming language backends need to be changed?
Go

I'd like to keep our build tags tied to operating systems and architectures. In this case, I can also update the PostgreSQL tags to include cgo so that the PostgreSQL engine isn't build when cgo isn't enabled.

Thanks @kyleconroy! I could confirm that adding a cgo check and running with CGO_ENABLED=0 works fine for generating mysql stubs and does not generate log spam for mac os sdk warnings. Sent #2068 with the proposal

authors: anuraaga kyleconroy

repo: kyleconroy/sqlc


text:
mysql: unable to query with reserved words

MySQL column names can be named with reserved words when escaped with backticks. An example table with a MySQL reserved word KEY:
-- file: schema.sql
CREATE TABLE `some_table` (
   -- ...
  `key` varchar NOT NULL
);

sqlc can parse the table and create models for create statements such as the above. But if we want to query some_table with an expression that contains the key column it seems to ignore de backticks. An exemple of what I mean:
-- file: queries.sql
/* name: GetAllSomeTable :many */
SELECT
  *
FROM some_table
WHERE
  `some_table`.`key` = ?;

With these files, we can have a sqlc.yaml file as bellow:
# file: sqlc.yaml
version: "2"

sql:
  - engine: "mysql"
    schema: "./schema.sql"
    queries: "./queries.sql"
    gen:
      go:
        package: "db"
        out: "./db"

sqlc generate, for that example, throws the following output:
// Code generated by sqlc. DO NOT EDIT.
// source: queries.sql

package db

import (

        "context"

)

const getAllSomeTable = `-- name: GetAllSomeTable :many
select id, `key` from some_table where some_table.`key` = ?
`

func (q *Queries) GetAllSomeTable(ctx context.Context, key string) ([]SomeTable, error) {
        rows, err := q.db.QueryContext(ctx, getAllSomeTable, key)
        if err != nil {
                return nil, err
        }
        defer rows.Close()
        var items []SomeTable
        for rows.Next() {
                var i SomeTable
                if err := rows.Scan(&i.ID,&i.Key); err != nil {
                        return nil, err
                }
                items = append(items, i)
        }
        if err := rows.Close(); err != nil {
                return nil, err
        }
        if err := rows.Err(); err != nil {
                return nil, err
        }
        return items, nil
}

# package db
error generating code: source error: 19:13: expected ';', found key (and 3 more errors)

The mysql:beta engine correctly escapes MySQL keywords
// Code generated by sqlc. DO NOT EDIT.
// source: query.sql

package querytest

import (
	"context"
	"database/sql"
)

const getAllSomeTable = `-- name: GetAllSomeTable :many
SELECT
  ` + "`" + `key` + "`" + `
FROM some_table
WHERE
  ` + "`" + `some_table` + "`" + `.` + "`" + `key` + "`" + ` = ?
`

func (q *Queries) GetAllSomeTable(ctx context.Context, key sql.NullString) ([]sql.NullString, error) {
	rows, err := q.db.QueryContext(ctx, getAllSomeTable, key)
	if err != nil {
		return nil, err
	}
	defer rows.Close()
	var items []sql.NullString
	for rows.Next() {
		var key sql.NullString
		if err := rows.Scan(&key); err != nil {
			return nil, err
		}
		items = append(items, key)
	}
	if err := rows.Close(); err != nil {
		return nil, err
	}
	if err := rows.Err(); err != nil {
		return nil, err
	}
	return items, nil
}

Great! Thanks for the good work 😄

authors: gpontesss kyleconroy


text:
Use status

Description
Changes proposed in this pull request:

...
...
...

Related issue(s)

/test all

/test all

/test all

/test all

/test all

/test all

/test all

/test all

/test all

/test all

/test all

repo: kyma-project/api-gateway


text:
Add hint to press Enter when adding labels to SI

Description
Changes proposed in this pull request:

Add hint to press Enter when adding labels to SI

Related issue(s)

See #442

/unhold

repo: kyma-project/btp-manager


text:
Return state suspended for all suspended runtimes

Description
Changes proposed in this pull request:

computing runtimes endpoint Satus.State value

Related issue(s)

/retest

/retest

repo: kyma-project/control-plane


text:
convert independent tools to single binary with subcommands

Description
Changes proposed in this pull request:

create initial subcommands
use kustomize to allow running using ko, or the predefined images

Related issue(s)

/test all

repo: kyma-project/eventing-tools


text:
Share cache between admission controller and signify operator

Description
Warden has a filesystem cache for calls towards notary service.
We should make sure that the cache is shared among the two modes of operation

new pod scheduled interception (admission controller)
normal reconciliation of pods in protected services (operator)
so that verification result for the same image can be reused in the process of pod creation

Reason
New admission controller with be hooked into pod lifecycle in the kubernetes and will increase the times the image signature is checked. We should not increase the number of external calls to signify by introducing admission controller but reuse the cached results.

This proved not to be critical.

repo: kyma-project/warden


text:
Unable to interact with world after changing dimensions via portal

I'm running in a development environment on Minecraft 21w03a, using fabric_version 0.29.5+1.17, and loader_version 0.11.1.
Using a custom portal to travel between dimensions breaks the player's ability to interact with the world (breaking blocks, tilling grass, etc.). Breaking blocks does work but only if the player is in creative mode. This is also the case when using the portal to travel back to the overworld from the custom dimension.
This can be fixed by quitting and re-joining the game, but it is obviously not desirable to do this every time a custom portal is used.

that issues on my works too.

@kyrptonaught I figured out how to fix this issue.
In order to fix the issue you need a Mixin accessor for Entity.unsetRemoved(), and call it in CustomTeleporter after
serverWorld.removePlayer(player, EntityRemovalReason.CHANGED_DIMENSION
I don't know if you accept pull requests from other people, but just let me know if you want me to submit a pull request with the changes I made.

I do accept pull requests! I'll gladly take some help! Can you verify that this works on the most recent snapshot? Assuming you discovered this by checking the vanilla teleportation logic?

Verified that it works on 21w05a as well. Submitted a pull request to fix. Wasn't sure how to link the pull request to this issue since I've never submitted a pull request before :P

Thanks! I'll check over it in a bit.
You can mention the pr just by stating it's number, this works in both directions too. You can mention an issue in a pr. For example: #9

is it fixed or closed issue?

I have not gotten the chance to submit a new build to bintray yet

Thank you for your help. 0.0.1-beta21-1.17 was uploaded to bintray with this fix implemented

authors: RedLime kyrptonaught modderwizard

repo: kyrptonaught/customportalapi


text:
Feat: change command descriptor to attribute

Мне что-то ударило в голову, я решил переделать дескриптор в атрибут. Почти уверен, что это не нужно мёрджить в мастер пока что, просто хочу послушать мнение про эту грязь

Ни на что не намекаю, но в шарп завезли статик методы интерфейсам.

bump @Bibletoon

@Bibletoon

Ладно, ща доделаю...

@Bibletoon только тоже на dev поменяй бейз бранч.

Ни на что не намекаю, но в шарп завезли статик методы интерфейсам.

Кста так и не понял, на что ты намекал

Может @ronimizy хочет посмотреть?

Может @ronimizy хочет посмотреть?

Сегодня после рубежки по физике гляну

authors: Bibletoon FrediKats ronimizy s4xack

repo: kysect/BotFramework


text:
docs: asset perf tweaks

What changed

Minor perf tweaks to improve lighthouse score
Self host assets to prevent links randomly going dead (as a result fixed the two broken assets)
Tweaked descriptions a bit to prevent images being squished
Using lighter GIF (Essentially dropped every a fraction of the frames). Still huge 5MBs, but at least ~50% lighter.

Old lighthouse score

New lighthouse score

Love the improvements!
Wonder if the big 5MB could be streamed as an autoplaying looping video without controls.
I think the video might be too sped up. But I like the idea, the message comes across much quicker.

There's other ways for sure to improve the gif, but this was the lowest effort one 😅

Good changes, except that the GIF is now much much faster and really hard to follow. It needs to go back to the old speed.

Based on some quick tests, converting the GIF to webm or mp4 with high enough quality, results in a ~2MB file. 900KB file still look decent, but the text is a bit soft already.

Alright, should be ready for another review :)

And the new lighthouse score!

authors: fhur igalklebanov koskimas

repo: kysely-org/kysely


text:
typescript: Make the typescript signatures more specific to their module

Made a few changes to the Typescript Indexer so it's easier to index code across different modules:

Module signatures are now the module name instead of the word module.
Anchor signatures now include the file path instead of just having the offsets
Symbol signatures now include their module
Module names are now relative to KYTHE_ROOT_DIRECTORY
Corpus is now set to KYTHE_CORPUS

Anchor signatures now include the file path instead of just having the offsets
This is unnecessary; the anchor VName already includes the file path.  Putting it in the signature is redundant (and potentially confusing to humans).

Cleaning up old PRs, feel free to re-open and address the comments if this is still relevant.

authors: johnedmonds shahms

repo: kythe/kythe


text:
Introduce hacking.rst

I want to do another run through it, but I think it's ready for review.

r++ :)

authors: stasm zbraniecki


text:
Document how to build mops on CPU only despite the presence of a CUDA compiler

This might be as easy as doing CMAKE_MOPS_CUDA=OFF CMAKE_MOPS_TORCH_CUDA=OFF pip install ... (and making sure the torch wheels are CPU-only), but this needs to be documented.

Are you sure that CMAKE_XXX in the env is used to set the XXX cmake option? This happens for native options, but I'm not sure it's the case in general.

authors: Luthaf frostedoyster

repo: lab-cosmo/mops


text:
feat: Integration of pyo3-stub-gen

This PR introduces pyo3-stub-gen into the project to automate the generation of Python interface type hint files (.pyi) from Rust code.
However, during the integration process, several compatibility issues were encountered.
1. Our project currently uses pyo3 0.20.3, (primarily due to the dependency on pyo3-asyncio..?) which is incompatible with pyo3 0.21.0.
This version mismatch leads to compilation failures when trying to run the stub generator, specifically when executing cargo run --bin stub_gen as per the documentation.
2. Errors from Example Code:
The provided example code results in errors when following the documented steps to generate stubs.

Since pyo3-stub-gen requires pyo3 version 0.21 or higher, I think this issue is related to pyo3's version mismatch.
Ref: https://github.com/lablup/raftify/issues/90

authors: jopemachine why-arong

repo: lablup/raftify


text:
docs: add install example for alist && discuz && pingvin-share

install example for alist && discuz && pingvin-share

Seems can't rerun the failed job.  Put them into application store seems a better way.

authors: fanux nightwhite

repo: labring/sealos


text:
[Bug]:  Error creating bridge namespace folder

What happened?
when I run the artisan make:bridge command it is unable to create the base folder for the classes
How to reproduce the bug
run artisan make:bridge command
Package Version
dev
PHP Version
8.2
Laravel Version
11
Which operating systems does with happen with?
macOS
Notes
No response

fixed.

repo: lacasera/elastic-bridge


text:
fix(cli): query run via editor

Relocating cli start/stop progress for lacework query run
ALLY-615

Changed the title just as a suggestion to avoid adding "fix" twice

authors: afiune hazedav

repo: lacework/go-sdk


text:
supertest version 6.3.3 => 6.3.4 is a breaking change

supertest version 6.3.3 => 6.3.4 is a breaking change
supertest version 6.3.3 => 6.3.4 is a breaking change becuse the dependencies for @types/supertest increments the 4x major versions
@types/supertest 2.0.16 => 6.0.2
Node.js version:
v18.19.0
OS version:
macOs sonoma 14.2.1
Description:
(Typescript)
moving from 6.3.3 to 6.3.4 pulls in major type version changes and breaks type assumptions
supertest 6.3.3 => 6.3.4
@types/supertest 2.0.16 => 6.0.2

import supertest, { Response } from 'supertest'
. . .
supertest(url)

error TS2345: Argument of type 'TestAgent<Test>' is not assignable to parameter of type 'SuperTest<Test>'.
  Type 'TestAgent<Test>' is missing the following properties from type 'Stream': pipe, compose, addListener, removeListener, and 11 more.

supertest.SuperTest<supertest.Test>
Actual behavior

Expected behavior
Types should be backward compatible for patch version changes
Better documentation for types usage should be available

Code to reproduce

Checklist

[x] I have searched through GitHub issues for similar issues.
[x] I have completely read through the README and documentation.
[x] I have tested my code with the latest version of Node.js and this package and confirmed it is still not working.

The issue is with @types/supertest@6.X.X

repo: ladjs/supertest


text:
Add way to remove freezeheader

It would be good if there was a way to remove the freezeheader after it has been added.

+1

authors: WACMemphis jazmatician

repo: laertejjunior/freezeheader


text:
Error: dopy >= 0.3.2 required for this module

I couldn't get your example playbook to run for me - kept encountering the above error despite having dopy 0.3.3, 0.3.5, or 0.3.7 installed.

turns out it was related to this, sorry: https://github.com/ansible/ansible-modules-core/issues/360#issuecomment-68598504

repo: lafarer/ansible-role-do-droplets


text:
No feedback when the user doesn't select a folder

I'm trying to show the user an alert when they haven't selected a folder to let them know their file won't be downloaded, but in the case that no URI is returned from the document tree in openDocumentTree, nothing is sent back to flutter.
I don't know whether it makes sense to do something like pendingResult.second.success(null) (or even pass the uri regardless of whether it is null) or to return an error case that would need to be handled in flutter, such as an error saying that no folder was chosen.
If you have a preference I'm happy to open a PR against this.
I'm trying to make use of the latest version from master at the moment as I needed access to some of the code as-yet unreleased on pub.dev.
Thanks!

Thanks for the issue! As you described, seems like the native never returns when the user don't pick a folder, right?
I think the solution is to return null when it happens since it's not a error itself but a user choice
If you want, feel free to open the PR! If I could help to answer questions, let me know
Thanks again!

authors: dangilbert lakscastro

repo: lakscastro/shared-storage


text:
Autobattle fights

Implements battle simulation. Battle rules can be found here and here, but I'll make an effort to summarize them in this description.
Battle happens between up to 6 units on two teams (we don't validate this in the battle simulation though, it should happen before that). Each has an assigned position which they do not move out of. Units have stats that grow with level and items: ATK used for skill damage, HP that if reaches 0 you die, SHIELD that will block damage (not designed yet). They also have energy, which they gain by casting their basic skill (among others). They do so whenever its cooldown is ready. When units accumuluate 500 energy, they cast their ultimate skill.
Skills are made of effects and a targeting strategy. Effects also have targeting strategies themselves in case we want a skill to target differently for its effects (effect 1: attack 1 enemy, effect 2: heal myself, both part of the same skill).
Battles are simultaneous, meaning that many characters can attack in the same step (as opposed to a turn-based combat where only one character attacks at a time). This is achieved by taking decisions in every step based on the initial state of the battle for that step. For example, if we have 2 units that perform the same attack in a step:

Unit 1 recieves the initial state of the step, decides based on that (hit lowest hp enemy). Let's say target is hit and dies.
Unit 2 recieves the same initial state that unit 2 did. It also decides based on it (hit lowest hp enemy). The target will be the same as Unit 1, since it was the lowest hp. It does not matter that Unit 1 will kill them with their attack, Unit 2 will hit them as well since those actions are taking place at the same time.

Effects have different targetting strategies, stats they can affect, ways they affect the target stat (additive/multiplicative) and ways their amount is interpreted (flat / % based on stat like ATK).
There are many features that are missing in this PR, but it's ready for review & merge because the general outline is finished and future implementations could expand on it. The most important ones are:

Status effects (stun, slow...)
Implementation of other targeting strategies
On-kill effects
On-getting-hit effects (receive energy)

Sample code to run a battle between two muflus:
  user_1 = Champions.Users.register("User1")
  user_2 = Champions.Users.register("User2")
  team1 = Enum.map([1], fn slot -> GameBackend.Units.insert_unit(%{character_id: GameBackend.Units.Characters.get_character_by_name("Muflus").id, unit_level: 1, tier: 1, rank: 1, selected: true, user_id: user_1.id, slot: slot}) |> elem(1) |> GameBackend.Repo.preload(character: [:basic_skill, :ultimate_skill]) end)
  team2 = Enum.map([1], fn slot -> GameBackend.Units.insert_unit(%{character_id: GameBackend.Units.Characters.get_character_by_name("Muflus").id, unit_level: 1, tier: 1, rank: 1, selected: true, user_id: user_2.id, slot: slot}) |> elem(1) |> GameBackend.Repo.preload(character: [:basic_skill, :ultimate_skill]) end)
  Champions.Battle.Simulator.run_battle team1, team2

To test it:
  user_2 = Champions.Users.register("User2")
  team1 = Enum.map([1], fn slot -> GameBackend.Units.insert_unit(%{character_id: GameBackend.Units.Characters.get_character_by_name("Muflus").id, unit_level: 1, tier: 1, rank: 1, selected: true, user_id: user_1.id, slot: slot}) |> elem(1) |> GameBackend.Repo.preload(character: [:basic_skill, :ultimate_skill]) end)
  team2 = Enum.map([1], fn slot -> GameBackend.Units.insert_unit(%{character_id: GameBackend.Units.Characters.get_character_by_name("Muflus").id, unit_level: 1, tier: 1, rank: 1, selected: true, user_id: user_2.id, slot: slot}) |> elem(1) |> GameBackend.Repo.preload(character: [:basic_skill, :ultimate_skill]) end)
  Champions.Battle.Simulator.run_battle team1, team2```

authors: lotuuu manucamejo


text:
Add StorageWriteRequest

Closes #312

Codecov Report

:exclamation: No coverage uploaded for pull request base (0.11@7611b51). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             0.11     #345   +/-   ##
=======================================
  Coverage        ?   95.54%
=======================================
  Files           ?       47
  Lines           ?     8439
  Branches        ?        0
=======================================
  Hits            ?     8063
  Misses          ?      376
  Partials        ?        0

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

authors: codecov-commenter matias-gonz


text:
🛑 trixmall.com is down

In 2fc86e8, trixmall.com (https://www.trixmall.com/) was down:

HTTP code: 567
Response time: 436 ms

Resolved: trixmall.com is back up in 12e859c.

repo: lanen/bs-site


text:
🛑 rexingsports.cn is down

In 4dec178, rexingsports.cn (https://www.rexingsports.cn) was down:

HTTP code: 567
Response time: 5310 ms

Resolved: rexingsports.cn is back up in 66ad5b3 after 18 minutes.


text:
🛑 rexingsports.cn is down

In 616a283, rexingsports.cn (https://www.rexingsports.cn) was down:

HTTP code: 567
Response time: 970 ms

Resolved: rexingsports.cn is back up in f10fb31 after 1 hour.


text:
🛑 rexingsports.cn is down

In ebbb608, rexingsports.cn (https://www.rexingsports.cn) was down:

HTTP code: 567
Response time: 432 ms

Resolved: rexingsports.cn is back up in c0e8734 after 20 minutes.


text:
🛑 Hacker News is down

In dc509ae, Hacker News (https://news.ycombinator.com) was down:

HTTP code: 502
Response time: 285 ms

Resolved: Hacker News is back up in 6991683 after 10 minutes.

repo: langasg/upptime


text:
Issue with callback manager for verbose output

Privileged issue

[X] I am a LangChain maintainer, or was asked directly by a LangChain maintainer to create an issue here.

Issue Content
Following this tutorial: https://python.langchain.com/v0.1/docs/modules/agents/quick_start/#create-the-agent

@eyurtsev I can take this.

identical to: https://github.com/langchain-ai/langchain/issues/26773

repo: langchain-ai/langchain


text:
docs: release date fix

Replaced the overdue release promise.

@baskaryan Bagatur, this check error is out of my control. Could you, please, check it? TNX

Thanks Leonid!

authors: baskaryan leo-gan


text:
Add Document Loader for Google Drive

In reference to this issue in the python repo: https://github.com/langchain-ai/langchain/issues/9783, I would like to create a issue to add document loader for Google Drive in langchainjs. I would like to be assigned and work on this issue.

Hey @saminkhan1 that would be much appreciated! This can be the issue used to track it, so just put up a pull request with the improvement and add the line Fixes #2922

This sounds great!
Note that there was some similar work done as part of the MakerSuite integration that you may wish to leverage.
See the DriveFileReadConnection in experimental/hubs/makersuite/googlemakersuitehub.ts and ping me if you have questions or thoughts.

authors: afirstenberg bracesproul saminkhan1

repo: langchain-ai/langchainjs


text:
"langgraph test" doesn't work with latest langgraph

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangGraph/LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangGraph/LangChain rather than my code.
[X] I am sure this is better as an issue rather than a GitHub discussion, since this is a LangGraph bug and not a design question.

Example Code
uvicorn==0.31.0
python-dotenv==1.0.1
langchain==0.3.2
langchain-core==0.3.9
langchain_openai==0.2.1
langchain-voyageai==0.1.2
langchain_cohere==0.3.0
langchain_experimental==0.3.2
langgraph==0.2.34
langsmith==0.1.131
langchain_pinecone==0.2.0
langgraph-checkpoint-postgres==2.0.0
wikipedia==1.4.0
pinecone-client[grpc]==5.0.1
pinecone-text==0.9.0
psycopg[binary,pool]==3.2.3
psycopg-pool==3.2.3
requests==2.32.3
pytest==8.3.3
setuptools==75.1.0

Error Message and Stack Trace (if applicable)
outer_arc-ai-langgraph/src/requirements.txt" did not complete successfully: exit code: 1
------
 > [3/7] RUN PYTHONDONTWRITEBYTECODE=1 pip install --no-cache-dir -c /api/constraints.txt -r /deps/__outer_arc-ai-langgraph/src/requirements.txt:
2.030
2.030 The conflict is caused by:
2.030     The user requested langchain-core==0.3.9
2.030     The user requested (constraint) langchain-core<0.3.0,>=0.2.27
2.030
2.030 To fix this you could try to:
2.030 1. loosen the range of package versions you've specified
2.030 2. remove package versions to allow pip to attempt to solve the dependency conflict
2.030
2.147 ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts
------
Dockerfile:6
--------------------
   4 |
   5 |     ADD requirements.txt /deps/__outer_arc-ai-langgraph/src/requirements.txt
   6 | >>> RUN PYTHONDONTWRITEBYTECODE=1 pip install --no-cache-dir -c /api/constraints.txt -r /deps/__outer_arc-ai-langgraph/src/requirements.txt
   7 |
   8 |     ADD . /deps/__outer_arc-ai-langgraph/src
--------------------
ERROR: failed to solve: process "/bin/sh -c PYTHONDONTWRITEBYTECODE=1 pip install --no-cache-dir -c /api/constraints.txt -r /deps/__outer_arc-ai-langgraph/src/requirements.txt" did not complete successfully: exit code: 1

Description
"langgraph up" is working however "langgraph test" complains with dependencies, could you update constraints.txt to make it working? thanks.
System Info
windows
python 3.12.5

@junan-trustarc the new recommended way to start a local server is to run langgraph devhttps://langchain-ai.github.io/langgraph/tutorials/langgraph-platform/local-server/
will close this issue, but let me know if you run into any further problems

authors: junan-trustarc vbarda

repo: langchain-ai/langgraph


text:
Support read type hints from the method in add_node

#1950

Add function to read type hints from the __call__ method to resolve issue #1950.

Looks good, thanks for adding the support!


text:
Add six to requirements.txt

Six is used at least in this file:
https://github.com/langchain-ai/langsmith-sdk/blob/main/python/langsmith/_internal/_patch.py#L4
So, the library causes errors, if six is not already installed. Better add it to required libs.

fixed in #1197 instead

authors: Tayamarn efriis

repo: langchain-ai/langsmith-sdk


text:
Add basic ErrorProne config to langchain4j-core.

See ErrorProne

Runs during mvn verify.
fix for two extant issues.
Stricter tests could be enabled later.

This has some issues with the java8 tests:
[INFO] Compiling 86 source files with javac [debug target 8] to target/classes
An exception has occurred in the compiler (1.8.0_392). Please file a bug against the Java compiler via the Java bug reporting page (http://bugreport.java.com/) after checking the Bug Database (http://bugs.java.com/) for duplicates. Include your program and the following diagnostic in your report. Thank you.
java.lang.UnsupportedClassVersionError: com/google/errorprone/ErrorProneJavacPlugin has been compiled by a more recent version of the Java Runtime (class file version 55.0), this version of the Java Runtime only recognizes class file versions up to 52.0

When running in the CI environment; I have to enable <fork>true</fork>; and that seems to have some performance impact for CI which causes the tests to get cancelled (despite producing valid errorprone warnings).
And the version of errorprone isn't compatible with the java8 environment.
I could do some more work to make this work.

@crutcher thanks a lot! Please do not invest much effort in fighting with java 8, I anticipate a move to 11 quite soon

@crutcher I like the overall idea to have some strict checks, I am just not sure why exactly ErrorProne? It seems that there are lots of other more popular tools like Checkstyle, PMD, FindBugs, SonarQube, etc. Is there a reason why ErrorProne was chosen?
Thanks!


text:
chore: update langflow-embedded-chat script URL to v1.0.3

Pull Request Description
This pull request updates the langflow-embedded-chat script URL to version 1.0.3. The update is a crucial step towards ensuring that our application remains up-to-date with the latest features, bug fixes, and performance improvements provided by the langflow-embedded-chat library.

LGTM

repo: langflow-ai/langflow


text:
AgentOps Integration

Implemented AgentOps with better LLM tracking and function monitoring.

Hey @sarath59
How are you?
This integration is a different from what we are doing with tracing integrations. Could you check out the LangWatchTracer and the LangSmightTracer, please?

Sure, will take a look at it. Thanks.
On Sun, Jul 28, 2024 at 9:03 AM Gabriel Luiz Freitas Almeida <
@.***> wrote:

Hey @sarath59 https://github.com/sarath59
How are you?
This integration is a different from what we are doing with tracing
integrations. Could you check out the LangWatchTracer and the
LangSmightTracer, please?
—
Reply to this email directly, view it on GitHub
https://github.com/langflow-ai/langflow/pull/2958#issuecomment-2254530485,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKYFVILDSTYG37DFDK72B6LZOT24BAVCNFSM6AAAAABLO635DOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJUGUZTANBYGU
.
You are receiving this because you were mentioned.Message ID:
@.***>


text:
Broken cargo install

~> cargo --version
cargo 1.69.0 (6e9a83356 2023-04-12)

Log

~> cargo install icemelter
    Updating crates.io index
  Installing icemelter v0.3.1
   Compiling proc-macro2 v1.0.58
   Compiling quote v1.0.27
   Compiling unicode-ident v1.0.9
   Compiling libc v0.2.144
   Compiling io-lifetimes v1.0.11
   Compiling once_cell v1.17.1
   Compiling memchr v2.5.0
   Compiling rustix v0.37.19
   Compiling cc v1.0.79
   Compiling cfg-if v1.0.0
   Compiling linux-raw-sys v0.3.8
   Compiling bitflags v1.3.2
   Compiling regex-syntax v0.7.2
   Compiling log v0.4.17
   Compiling serde v1.0.163
   Compiling tracing-core v0.1.31
   Compiling utf8parse v0.2.1
   Compiling anstyle-parse v0.2.0
   Compiling colorchoice v1.0.0
   Compiling anstyle v1.0.0
   Compiling serde_json v1.0.96
   Compiling anstyle-query v1.0.0
   Compiling clap_lex v0.5.0
   Compiling thiserror v1.0.40
   Compiling heck v0.4.1
   Compiling aho-corasick v1.0.1
   Compiling lazy_static v1.4.0
   Compiling ryu v1.0.13
   Compiling itoa v1.0.6
   Compiling strsim v0.10.0
   Compiling syn v2.0.16
   Compiling pin-project-lite v0.2.9
   Compiling anyhow v1.0.71
   Compiling fastrand v1.9.0
   Compiling overload v0.1.1
   Compiling nu-ansi-term v0.46.0
   Compiling tree-sitter v0.20.10
   Compiling tree-sitter-rust v0.20.3
   Compiling tracing-log v0.1.3
   Compiling sharded-slab v0.1.4
   Compiling regex v1.8.3
   Compiling wait-timeout v0.2.0
   Compiling thread_local v1.1.7
   Compiling smallvec v1.10.0
   Compiling num_cpus v1.15.0
   Compiling is-terminal v0.4.7
   Compiling anstream v0.3.2
   Compiling clap_builder v4.3.0
   Compiling tempfile v3.5.0
   Compiling tree-sitter-traversal v0.1.2
   Compiling tree-sitter-edit v0.3.0
   Compiling serde_derive v1.0.163
   Compiling tracing-attributes v0.1.24
   Compiling clap_derive v4.3.0
   Compiling thiserror-impl v1.0.40
   Compiling tracing v0.1.37
   Compiling clap v4.3.0
   Compiling clap-verbosity-flag v2.0.1
   Compiling tracing-serde v0.1.3
   Compiling treereduce v0.2.2
   Compiling tracing-subscriber v0.3.17
   Compiling icemelter v0.3.1
error[E0063]: missing field `delete_non_optional` in initializer of `Config<_>`
   --> /home/bennet/.cargo/registry/src/github.com-1ecc6299db9ec823/icemelter-0.3.1/src/main.rs:264:25
    |
264 |     let reduce_config = Config {
    |                         ^^^^^^ missing `delete_non_optional`

For more information about this error, try `rustc --explain E0063`.
error: could not compile `icemelter` due to previous error
error: failed to compile `icemelter v0.3.1`, intermediate artifacts can be found at `/tmp/cargo-installv3d3mT`

Thanks for the report! In the short term, you can try installing from the git repo:
git clone https://github.com/langston-barrett/icemelter
cd icemelter
cargo install --path .

Fixed in v0.3.2

authors: Skgland langston-barrett

repo: langston-barrett/icemelter


text:
Use Segoe Fluent Icons for window controls when available

Use Segoe Fluent Icons for window controls on Windows 11 and fallback to  Segoe MDL2 Assets on the others. In addition, change code points to correct values.

Hi, this change is not necessary as the font icons will be replaced with SVGs: https://github.com/lapce/lapce/pull/768

Hi, this change is not necessary as the font icons will be replaced with SVGs: https://github.com/lapce/lapce/pull/768

That's pretty nice. Close this.

authors: panekj pbnoyz

repo: lapce/lapce


text:
Unable to connect to mysql

I am using Laradock on Mac OS with a laravel project.
Issue:
I used laradock containers mysql and nginx. I am able to get the application running on my localhost. But I am unable to connect to mysql container from my host machine (MAC OS).
My docker-compose.yml contains all the configuration for MySQL as shown below:
mysql:
build:
context: ./mysql
args:
- MYSQL_VERSION=${MYSQL_VERSION}
environment:
- MYSQL_DATABASE:homestead
- MYSQL_USER:homestead
- MYSQL_PASSWORD:secret
- MYSQL_ROOT_PASSWORD:secret
- TZ:${WORKSPACE_TIMEZONE}
volumes:
- ${DATA_SAVE_PATH}/mysql:/var/lib/mysql
- ${MYSQL_ENTRYPOINT_INITDB}:/docker-entrypoint-initdb.d
ports:
- "3306:3306"
networks:
- backend
Then I did docker-compose up -d mysql nginx
This created the containers and bought them up and running.
Next I changed the .env file accordingly for DB connection.
APP_DEBUG=true
APP_ENV=local
APP_EXECUTION_TIME_LIMIT=300
APP_KEY=fshhVdCCne5dJTznON0W988vFnDTiDlm
BUGSNAG_API_KEY=
CACHE_DRIVER=file
DB_DATABASE=homestead
DB_HOST=127.0.0.1
DB_PASSWORD=secret
DB_USERNAME=homestead
DB_PORT=3306
SESSION_DRIVER=file
QUEUE_DRIVER=database
After that I did php artisan:migrate. --> This caused the tables to be created and I also seeded them succesfully from the command line.
But when I open my app on localhost, and try to login, I get connection refused for mysql container.
I also tried to change the IP from localhost to the IP of the mysql docker container (172.23.0.2) or something, but then the request times out. And I am unable to ping the machine from my terminal.
The ports are also exposed in laradock.
I am new to laradock and not aware if I am missing something basic to be done here or doing it the wrong way.
I want my application to connect to the database. Can someone please help asap ?

DB_HOST=mysql
and add
127.0.0.1 mysql
in your Mac /etc/hosts

Thanks, I had already done that. I figured out the issue was that, I had to set the port number in the my.cnf file of mysql. Then i was able to connect to MySQL from host machine.

Just for future: had the same issue but no luck with any solution. Turned out I had wrong permissions on my.cnf

try the command:
chown -R mysql:mysql /var/run/mysqld

authors: Narahari-Sundaragopalan exfriend weweliang xushaobo

repo: laradock/laradock


text:
有没有人在workspace的Dockerfile里面加入中科大的ppa源

如标题，在安装php-fpm的时候，中国用户访问ppa.launchpad.net的源永远都是time out，无法成功，看到一篇文章自己试了下非常好用。
有没有人实现源切换功能，然后提个pr上去？
实现方法：https://www.cnblogs.com/cshaptx4869/p/12374234.html

find /etc/apt/sources.list.d/ -type f -name ondrej-ubuntu-php-xenial.list -exec apt-get update -yqq
find: missing argument to `-exec'
ERROR: Service 'workspace' failed to build: The command '/bin/sh -c set -xe;     find /etc/apt/sources.list.d/ -type f -name "ondrej-ubuntu-php-xenial.list" -exec     apt-get update -yqq &&     pecl channel-update pecl.php.net &&     groupadd -g ${PGID} laradock &&     useradd -u ${PUID} -g laradock -m laradock -G docker_env &&     usermod -p "*" laradock -s /bin/bash &&     apt-get install -yqq       apt-utils       libzip-dev zip unzip       php${LARADOCK_PHP_VERSION}-zip       nasm &&       php -m |

Nice!

在workspace加上以后仍然是time out

@CreationLee workspace加上以后仍然是time out ,是怎么解决的

我的添加 也还是 time out

ubuntu 18.04 用这个
find /etc/apt/sources.list.d/ -type f -name "ondrej-ubuntu-php-bionic.list" -exec  sed  -i.bak -r  's#deb(-src)?\s*http(s)?://ppa.launchpad.net#deb\1 https\s://launchpad.proxy.ustclug.org#ig' {} ; && \

如果你本机有代理可以通过环境变量来设置代理

authors: 520 CreationLee Etuloser hysystem kilvn norieli


text:
Idea: Add option to install Google's V8 Javascript Engine PHP extension

Hi,
I came across this ReactJS package I would like to use with Laravel and Laradock, however, it has an indirect dependency on v8js.
I thought it might be useful for some users if there was an option to install the Google's V8 Javascript Engine PHP extension (v8js).
If there is some support for this, I do not mind submitting a pull request. Thanks.

Fixed by #483. Good job @edmundluong 😸

authors: edmundluong philtrep


text:
Add pg client to workspace phpfpm

I completed the 3 steps below:

[] I've read the Contribution Guide.
[] I've updated the documentation. (refer to this for how to do so).
[] I enjoyed my time contributing and making developer's life easier :)

@geraintdong could you please resolve the conflict. It just occurred after merging other PR's.

@Mahmoudz Done. Please review again & merge. Thanks.

@geraintdong Thanks.

authors: Mahmoudz geraintdong


text:
chore: update Italian translations

Update IT translations:

fix some typos
use more precise translations
use the singular form for plural English words when used in Italian

Thank you @eleftrik!

authors: eleftrik luanfreitasdev

repo: laradumps/app


text:
Remove all comments from js file

Laravel Mix Version: 6.0.49
Node Version: v19.4.0
NPM Version: 9.2.0
OS: macOS 13.1

Description:
I can't remove comments from output js file.

Steps To Reproduce:
webpack.mix.js
let mix = require( 'laravel-mix' );

mix.options( {
	processCssUrls: false,
	cssNano: {
		discardComments: {
			removeAll: true,
		},
	},
	manifest: false,
	terser: {
		extractComments: false,
	}
} );

mix.setPublicPath( 'web/app/themes/custom-child/assets/dist' );

mix.js( [ 'web/app/themes/custom-child/assets/src/js/theme.js' ], 'web/app/themes/custom-child/assets/dist' );

When I use extractComments as true then I have *.LICENSE.txt file. I don't want the file and any comments in output theme file.

@sebastianpisula try the following terser config -
mix.options({
    terser: {
	terserOptions: {
	    format: {
	        comments: false, // false => removes all comments
	    },
	},
    },
})

repo: laravel-mix/laravel-mix


text:
[v5.7.28] php artisan package:remove deletes everything from the vendor

Laravel Version: 5.7.28
PHP Version: 7.2.10
Database Driver & Version: N/A

Description:
php artisan package:remove v1 p1 is deleting everything inside v1
Steps To Reproduce:
I have 2 laravel projects my-own-packages and App1

New packages are always created inside my-own-packages using php artisan package:new v1 p1
App1's composer.json contains "repositories": [{ "type": "path", "url": "../my-own-package/packages/*/*"}], so my own packages can be pulled by running composer require v1/p1 and so on.
I do this so that, in time, App2 can re-use v1/p1 via composer
While running a php artisan package:remove v1 p1, artisan deleted several other packages inside my-own-packages/packages/v1 before some permissions issue caused it to throw an exception.
Is this by design? Seems more like a bug to me
If by design, then is there a way to isolate packages from the app during development? i.e. the package sits inside App1/vendor/v1/p1 instead of App1/packages/v1/p1
To workaround, I had to run php artisan package:disable v1 p1 in my-own-packages and then manually delete my-own-packages/packages/v1/p1

Thank you.

You'll need to report this on the repo of the package that is providing these commands.

oh.. got it. Sorry I'm very new to laravel. Thanks!

authors: bilogic devcircus

repo: laravel/framework


text:
not able to add data to database

This repository is only for reporting bugs or issues. If you need support, please use the forums:

https://laracasts.com/discuss
https://laravel.io/forum

Alternatively, you may use Slack (https://larachat.co/), Discord (https://discordapp.com/invite/mPZNm7A), or Stack Overflow (http://stackoverflow.com/questions/tagged/laravel).
Thanks!

Illuminate/Database/Eloquent/MassAssignmentException with message 'Add [name] to fillable property to allow mass assignment on [App/Company].' error coming while using add data via tinker

Looks like you need to add your mass assignment attributes.
As the template stated, this repository issue area is for reporting bugs or issues with the framework itself. Please reach out to the community channels for assistance.

laracasts.com/discuss
laravel.io/forum
Discord


text:
[8.x] pushMiddlewareToGroup not working on Provider::register method

Laravel Version: 8.32.1
PHP Version: 7.2.27
Database Driver & Version: mysql 5.7

Description:
It seems that pushMiddlewareToGroup is not working when inside a provider register method
Steps To Reproduce:
Add a middleware inside register method, it does not work :
Route::pushMiddlewareToGroup('web', MyMiddleware::class);

middleware is not called at all.
add the same code in boot() method and see that it works
it was working fine on v8.25.0
My Use Case
i'm using https://github.com/NoelDeMartin/laravel-dusk-mocking to mock some facades inside my dusk tests, like Socialite,
and since their middleware is added in their provider, it cease to work at some point.
if i edit this file :
https://github.com/NoelDeMartin/laravel-dusk-mocking/blob/master/src/MockingServiceProvider.php
and move pushMiddlewareToGroup calls to the boot method, it works again.
thanks.

It's not meant that you execute services like this in a service provider's register method. The register methods are only there to wire things into Laravel's container. The boot method is the correct place for this.

yes but it was working fine previously and a lot of third party packages were relying on it

@lk77 that wasn't intended to work. Sorry if this broke anything but we don't consider that a BC.


text:
support multiple connections in DatabaseTransactions trait

Altered trait to support multiple database connections and added fix from issue #10619

This still doesn't work surely?

That just starts transactions on any existing connections, so starts no transactions.

Yea you are right, sorry about that. Had a different @before function that was creating the transaction and looked like this logic was working.

authors: GrahamCampbell pulkitjalan


text:
[5.2] Fix eager loading within global scopes

This replaces https://github.com/laravel/framework/pull/11779 and fixes https://github.com/laravel/framework/issues/11764
Changelog:

Builder now applies scopes in get() instead of in getModels() and uses the returned Builder instance to eager load relations, which fixes https://github.com/laravel/framework/issues/11764
I also had to apply the same change on two Relation objects which were using similar pattern.

Compared to https://github.com/laravel/framework/pull/11779, this solution is more performant since it only executes applyScopes() once per query.

OK, managed to get those tests working too.

Can you describe what bug this fixes and what the problem was with our logic before?

If you have a global scope like this one:
public function apply($builder) {
  return $builder->with('posts');
}

It doesn't eager load posts in 5.2 but it did in 5.1
The problem with our logic is:

we clone the builder instance when applyScopes is called
but we didn't use the cloned instance to read the $eagerLoads property: https://github.com/laravel/framework/blob/5.2/src/Illuminate/Database/Eloquent/Builder.php#L516

authors: acasar taylorotwell


text:
[5.4] Fix for ErrorException in FilesystemAdapter.php line 146: fopen(): …

…Filename cannot be empty
If we upload a file under Windows web server, $file->getRealPath() return false. Then we need to use getPathName() to work normaly.

I'm not sure this is the correct fix. We need more explanation.

@pavlov80 @taylorotwell, I was having this issue too. What I did was create a new Temporary folder. (For me it is D:\Temp). Now the D: drive itself needs permissions added for IIS_IUSRS, which are Read, List contents and execute with the option applies to set to only this folder. The Temp folder needs the same permission of the D: drive but then applied to the folder and everything inside it.
Now change the upload_tmp_dir inside your php.ini to this new folder. (Carefull it can come twice where one is commented out)
Now getRealPath() should work

authors: DrowningElysium pavlov80 taylorotwell


text:
[8.x] Adds attempt method to Rate Limiter

What?
You can use the attempt() from the Rate Limiter to execute a callback if available.
use Illuminate\Support\Facades\RateLimiter;

RateLimiter::attempt('something', 5, function () {
    // ...
});

Fourth argument, decaySeconds, is default to 60.

How?
It just simply wraps the tooManyAttempts() and hit() in one call. Syntactic sugar for this:
use Illuminate\Support\Facades\RateLimiter;

if (RateLimiter::tooManyAttempts('something', 5)) {
    return false
}

// Do something...

RateLimiter::hit('something', 60);

return true;

If the shortcut fails, false is returned. Otherwise, it returns result of the callback, or true if the result is null. THis allows the developer to programmatically proceed if the call wasn't executed.
use Illuminate\Support\Facades\RateLimiter;

$executed = RateLimiter::attempt('something', 5, fn() => 'foo', 60 * 60);

if ($executed) {
    return "It returned $executed";
}

Why?

Simplifies effective Rate Limiter usage into one method easy to understand. (Should be documented too!)

Success!


text:
Made console confirmations more explicit and helpful

Hi Taylor,
I think these changes here will help the user to understand whats going to happen before they press y/n on the command line tool.
Cheers,

Please send this to 5.0.

I don't see 5.0 listed in the repo. How do I get to it?

It's the master branch. We'll actually be creating a 5.0 branch soon and making the master 5.1 :)

authors: GrahamCampbell TechyTimo


text:
Update LCE python interpreter docstring with efficiency warning

What do these changes do?
Updates the docstring of the python interpreter wrapper to warn about unoptimized performance for any platform other than ARM v8.
Related issue number
https://github.com/larq/larq/issues/607

@Tombana any idea what's wrong here? This seems unrelated to my changes...

@Tombana any idea what's wrong here? This seems unrelated to my changes...

Looks like a flaky test, I've restarted the tests now.

Looks like a flaky test, I've restarted the tests now.

authors: Tombana jneeven

repo: larq/compute-engine


text:
document link  disabled

Hi,
the link https://cavatica.sbgenomics.com/public/apps#d3b-bixu/app-publisher/mitoscape-wf/ has gone wrong, can you fix it ? Thank you.
Best,
xiucz.

Dear xiucz,
Unfortunately I'm no longer affiliated with the cavatica team. I suggest
you reach out to @.*** for help with this issue.
Thanks for your interest.
Larry.
On Sun, 29 Jan 2023 at 21:23, xiucz @.***> wrote:

Hi,
the link
https://cavatica.sbgenomics.com/public/apps#d3b-bixu/app-publisher/mitoscape-wf/
has gone wrong, can you fix it ? Thank you.
Best,
xiucz.
—
Reply to this email directly, view it on GitHub
https://github.com/larryns/MitoScape/issues/5, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABNKWAVYPUC7PST4UHERBLDWU4Q2VANCNFSM6AAAAAAUKTMMFE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

authors: larryns xiucz

repo: larryns/MitoScape


text:
Missing boundary annotation for parameters like "number of x"

URL Hash
#/sklearn/sklearn.ensemble._forest/RandomForestClassifier/init/n_estimators
Expected Annotation Type
@boundary
Expected Annotation Inputs
Discret: 0 <= x
Suggested Solution (optional)
Entweder an SciKit weiterleiten, oder vielleicht irgendwie ueber das "number of" automatisiert erkennen.
Additional Context (optional)
No response

Can you clarify what you meant by this?

authors: GideonKoenig lars-reimann

repo: lars-reimann/api-editor


text:
ASSERTION_FAILED - ASSERT NOT rv_user IS INITIAL - bug #1071

Introduce coding to check for initial rv_user. Return c_user_unkown if no user could be retrieved.

thanks, merging, welcome to the abapGit project as a contributor 🎆

authors: jaschaDEV larshp

repo: larshp/abapGit


text:
Refactoring on ZCL_ABAPGIT_ENVIRONMENT

Refactoring is needed in order to be able to mock this central abapGIT environment class in unit tests, so that different system states can be tested within unit tests.

constructor should be public, see https://github.com/SAP/styleguides/blob/master/clean-abap/CleanABAP.md#if-your-global-class-is-create-private-leave-the-constructor-public

Hi Lars,
Based on our discussion I shifted the instanciation into the ZCL_ABAPGIT_FACTORY class. Furthermore I use the ZCL_ABAPGIT_INJECTOR class as possibility to mock data during unit tests.

also new method is_restart_required is added, which is not used from anywhere, guess this is SCP relevant. It will show up in static analysis as unused, so suggest adding a comment to make sure its not deleted

for ltcl_test_simple_positive and ltcl_test_simple_negative it looks like none of the code in the class under test is tested? So it tests the factory and the injector, plus the stub implementation? The responsibility for testing the factory and injector can be moved to the factory and injector. And then the stub can be removed?

for ltcl_test_simple_positive and ltcl_test_simple_negative it looks like none of the code in the class under test is tested? So it tests the factory and the injector, plus the stub implementation? The responsibility for testing the factory and injector can be moved to the factory and injector. And then the stub can be removed?

Testing the actual cut is quite hard because this class is just a wrapper to call different APIs to get some system states. Unfortunately,  there are no interfaces in place which whould allowed to mock this real API calls. For that reason I have created this stub including unit tests for reuse purpose in many different places in other unit tests.

also new method is_restart_required is added, which is not used from anywhere, guess this is SCP relevant. It will show up in static analysis as unused, so suggest adding a comment to make sure its not deleted

In the next step I would contribute the usage of those methods in the abapGit core coding. But this will be done in some different pull requests.

the stub is a local class so it cannot be reused
I dont remember exactly the definition of what a "stub" is, but I guess it should not implement the class under test? But instead simulate behavior of something that the class under test depends on

source branch is deleted, @ThomasPloski will open a new PR with the changes
closing

authors: ThomasPloski larshp


text:
Mysqueezebox.com offer a REST call to determine the latest available …

…nightly build. I have incorporated this in to the docker build as the existing link is dead.

Is there also a call to get the amd64 package?

Unfortunately not by the looks of it. I did look for that.
The only way I can see is some nasty SED, but hardly a clean approach:
curl "http://www.mysqueezebox.com/update/?version=7.9.0&revision=1&geturl=1&os=deb" | sed 's/all/amd64/g'

Yes it's possible with sed, but I also thought it's not perfect. If yousing sed, do it at least like this: sed 's/_all\.deb/_amd64\.deb/g'

Worked perfectly for me. With regular updates, what about an option to pull the latest version on start up?

Gets my vote; at least gets latest version upon image build; updating images on docker startup is orthogonal to the way docker works.
This is the best version of the code and should go onto master. Just need a regular automated build to push to docker.io if the image has been mutated.
I can set something up to delta the url and then push to docker.io but to keep to the same naming convention it needs to come from the repo owner really....
I have just built this docker image locally with the latest version http://downloads.slimdevices.com/nightly/7.9/sc/afcfc6d/logitechmediaserver_7.9.0~1480066117_amd64.deb
and it works fine for me.

+1 for merging this PR, for all the reasons stated above. The last commit to Logitech/slimserver was two days ago, so some change is needed at any rate.

+1 for merging this PR. Pulling the lastest version on image build is very sensible.

authors: RichieB2B abufilba jmccoy555 mehstg pgroce thenktor


text:
Update scalatest to 3.0.8

Updates org.scalatest:scalatest from 3.0.7 to 3.0.8.
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 @scala-steward in the comments below.
Have a nice day!

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

labels: semver-patch

Codecov Report

Merging #114 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #114   +/-   ##
=======================================
  Coverage   37.23%   37.23%
=======================================
  Files          33       33
  Lines         948      948
  Branches       10       10
=======================================
  Hits          353      353
  Misses        595      595

Continue to review full report at Codecov.

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

authors: codecov-io scala-steward

repo: laserdisc-io/laserdisc


text:
Update kind-projector to 0.12.0

Updates org.typelevel:kind-projector from 0.11.3 to 0.12.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

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

labels: library-update, semver-minor

Codecov Report

Merging #486 (5ce120a) into master (26ea45c) will increase coverage by 0.34%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #486      +/-   ##
==========================================
+ Coverage   62.37%   62.71%   +0.34%
==========================================
  Files          38       38
  Lines        1289     1274      -15
  Branches        8        7       -1
==========================================
- Hits          804      799       -5
+ Misses        485      475      -10

Impacted Files
Coverage Δ

core/src/main/scala/laserdisc/laserdisc.scala
62.50% <0.00%> (-4.17%)
:arrow_down:

core/src/main/scala/laserdisc/protocol/RESP.scala
91.26% <0.00%> (-0.07%)
:arrow_down:

cli/src/main/scala/laserdisc/cli/CLI.scala
0.00% <0.00%> (ø)

...src/main/scala/laserdisc/protocol/SortedSetP.scala
0.97% <0.00%> (+0.02%)
:arrow_up:

core/src/main/scala/laserdisc/protocol/GeoP.scala
86.61% <0.00%> (+0.60%)
:arrow_up:

...e/src/main/scala/laserdisc/protocol/ClusterP.scala
68.91% <0.00%> (+1.36%)
:arrow_up:

...e/src/main/scala/laserdisc/protocol/Protocol.scala
100.00% <0.00%> (+5.00%)
:arrow_up:

.../scala/laserdisc/protocol/LenientStringCodec.scala
66.66% <0.00%> (+16.66%)
:arrow_up:

Continue to review full report at Codecov.

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

authors: codecov-commenter scala-steward


text:
Update discipline-munit to 1.0.9

Updates org.typelevel:discipline-munit from 1.0.8 to 1.0.9.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

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

labels: test-library-update, semver-patch

Codecov Report

Merging #493 (d2b1ccb) into master (7eab560) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #493   +/-   ##
=======================================
  Coverage   62.37%   62.37%
=======================================
  Files          38       38
  Lines        1289     1289
  Branches        7        8    +1
=======================================
  Hits          804      804
  Misses        485      485

Continue to review full report at Codecov.

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


text:
XMP error in the LaTeX WTPDF examples.

There is a really subtle error in the XMP packet in all of the example PDF 2.0 example documents. You need to verify the XMP in order to find it.
Do so here:
https://www.pdflib.com/pdf-knowledge-base/xmp/free-xmp-validator/
The attached picture says it all.

That is, the  pdfaid  property being added via a pdfaExtension schema is used as 'rev' but declared erroneously as 'year'.
After changing 'year' to 'rev' in one of my own PDFs, (using the pdfx2 package) it then passed validation. Presumably it is also just a 1-word typo in one of the LaTeX source files.

And there are 2 other places that I think are wrong too.
Both <pdfaField:valueType>Bag claim</pdfaField:valueType> and <pdfaField:valueType>Bag declaration</pdfaField:valueType>
should be just <pdfaField:valueType>Bag</pdfaField:valueType>.

That is, the pdfaid property being added via a pdfaExtension schema is used as 'rev' but declared erroneously as 'year'.

Right. Thanks for the report. They changed that in PDF/A-4.

And there are 2 other places that I think are wrong too.
Both <pdfaField:valueType>Bag claim</pdfaField:valueType> and <pdfaField:valueType>Bag declaration</pdfaField:valueType> should be just <pdfaField:valueType>Bag</pdfaField:valueType>.

No, I think they are ok. I got this schema from https://pdfa.org/wp-content/uploads/2023/12/PDF-Declarations.pdf (the pdf has an attachment with an example schema).
There is a resource which defines the type claim:
<pdfaType:type>claim</pdfaType:type> and Bag claim is then a Bag of such claims.
Similar there is a resource defining declaration:
 <pdfaType:type>declaration</pdfaType:type> and this handle then  Bag declaration.
See also https://pdfa.org/wp-content/uploads/2011/09/tn0009_xmp_extension_schemas_in_pdfa-1_2008-03-20.pdf about how new field values can be defined.

That seems an OK explanation (about Bag <whatever>), but doesn't match this result:

when I test the XMP packet from one of the examples.
IfBag claim is changed to just Bag then it is Bag declaration that now is reported as failing.
So there must be more to this, irrespective of what is in the PDF-Declarations.pdf example.
Besides, if you check PDF-Declarations.pdf at the XMP validator site, then you get:

so it shouldn't be any surprise that there are other errors within that document.
And if you save the file  A Extension Schema.xmp to disk, and then test it for XMP validation, you get the same error:

And the TN document has no example of Bag <something> — only what is in the following image:

which may be taken to imply general validity of that usage of Bag, but I'd not trust such a presumption when there is demonstrable failure with some validator.

The 2.0 and 1.7 examples at #72 have been updated  to fix the declaraition of pdfaid:rev and also to improve support for tagging hyphenation at line breaks.

@u-fischer , @davidcarlisle
Thanks for acting on this quickly.
Ulrike, was year ever used with the XMP in any PDF/A version?
I cannot find it in the original ISO standards documents for PDF/A-2 nor PDF/A-3.
These do have pdfaid:amd and pdfaid:corr with the year included after a ;.
There may have been later updates that I do not have.
Also., I cannot find usage of Bag <anything> in the XMP specification documents from
https://github.com/adobe/XMP-Toolkit-SDK/blob/main/docs/
except when <anything> is one of the basic RDF types.
Surely it would at least have to be a pre-defined type?
— not something (self-referentially) declared within the same Resource ?

@ozross  yes there will probably be more adjustement here and as Ulrike says she opened an issue at pdfa asking for clarification, but the initial point you raised was a clear bug so we regenerated all the files just fixing that, and will come back to the other issues.

Ulrike, was year ever used with the XMP in any PDF/A version?

I don't know. I don't own the ISO norms and also do not intend to spend money on it.  Most of the entries we used have been collected from external sources like hyperxmp, pdfx and internet and I tried to validate them with e.g. verapdf.

Also., I cannot find usage of Bag  in the XMP specification

As said I opened an issue and wait for confirmation.

authors: davidcarlisle ozross u-fischer


text:
Hamburger menu is een div, kan beter een button

https://github.com/latoyaln/the-startup-responsive-interactieve-website/blob/c459978244b07918f2bb451e247a073c5f74799a/index.html#L35-L39
Het hamburger menu is een div element. Het zou volgens mij semantisch beter zijn als dit een button element was, omdat het een interactie geeft op de pagina.
Misschien zou het beter zijn om van dit element een button te maken in plaats van een div, omdat dit semantisch (voor screenreaders ook) aangeeft dat het een interactief element is.

Ik heb de issue opgelost door de div te veranderen in een button 7ea64a5


text:
fix for 2024b and update default to 2024a

closes #143

update default to 2024a

Somehow https://data.iana.org/time-zones/tzdata-latest.tar.gz returns 2024a for me. Otherwise I would've updated the default to 2024b.

Seems like there are more problems with 2024b. It fails to identify offsets in some datetimes:
on 2024b:  #DateTime<2015-01-13 07:00:07-12:00 %z Etc/GMT+12>
on 2024a: #DateTime<2015-01-13 07:00:07-12:00 -12 Etc/GMT+12>

Seems like there are more problems with 2024b. With it, tzdata fails to identify zone_abbr in some datetimes:

Thanks for mentioning that. This should be fixed in the recently released tzdata version 1.1.2.

@lau thank you for the fixes!

authors: lau ruslandoga

repo: lau/tzdata


text:
ive made a great fix

sean - please review

terrible - please fix the build in jenkins

repo: laughingbiscuit/maven-jenkins-ci-demo


text:
Support deref types in Postgres arrays

While working with a Postgres text[] type I noticed that sqlx is not very flexible about what it allows as the type to write into a text array like this. Given the types, I think all of these should technically be allowed:

&Vec<String> (this works)
&Vec<&str>
&[String]
&[&str]
&[String; 2]
&[&str; 2]

I'm completely unfamiliar with the sqlx codebase so far, but I came up with the change below which I think would allow (2) and (4) at least. Assuming const generics are allowed in sqlx (MSRV 1.51), perhaps we could also have blanket impls for [String; N] and [&str; N]?

This change makes a lot of sense to me. Since I've been using sqlx, these are types that I expected to have Encode implemented. This implementation does look identical to the previous one except the type constraints, do you think there's any way to reduce that duplication?

Probably! I'm happy to iterate once CI runs and I get some guidance on testing.

Assuming const generics are allowed in sqlx (MSRV 1.51)

That seems fine as we don't track an MSRV: https://github.com/launchbadge/sqlx/blob/master/FAQ.md#what-versions-of-rust-does-sqlx-support-what-is-sqlxs-msrv

@abonander in my previous comment I mentioned I'd be happy to iterate once I got some guidance on testing. That didn't really happen... Anyway, for now this is not a priority for me. I'll file an issue first if I run into this stuff again.

authors: abonander djc loafofpiecrust

repo: launchbadge/sqlx


text:
Unpublished Yarn Berry files

Describe the bug
When using Yarn Zero Installs, you are not intended to publish the dependencies (any of the Yarn files) to the registry.
Accordingly, this package is WAY too large on the NPM registry.

Thanks for reporting this. This is now fixed in v1.0.2.

authors: quinnturner yusinto

repo: launchdarkly/jest-launchdarkly-mock


text:
How I can test TLS Client authentication using est-client-python

I have an EST server which supports TLS authentication. Instead of HTTP basic auth, I need to use TLS certificate for client authentication, how i can test that using est-client-python ?

TLS authentication is not supported.

repo: laurentluce/est-client-python


text:
Certificate renewal breaks if e-mail can't be delivered

Example stack trace:
Mar 31 06:26:02 ca certidude[16379]: Connecing to ldap://dc1.example.com using Kerberos ticket cache from /run/certidude/krb5cc
Mar 31 06:26:02 ca /usr/bin/python[17271]: python /usr/local/bin/certidude serve -l 0.0.0.0 -p 80: GSSAPI client step 1
Mar 31 06:26:02 ca /usr/bin/python[17271]: python /usr/local/bin/certidude serve -l 0.0.0.0 -p 80: GSSAPI client step 1
Mar 31 06:26:02 ca /usr/bin/python[17271]: python /usr/local/bin/certidude serve -l 0.0.0.0 -p 80: GSSAPI client step 1
Mar 31 06:26:02 ca /usr/bin/python[17271]: python /usr/local/bin/certidude serve -l 0.0.0.0 -p 80: GSSAPI client step 2
Mar 31 06:26:02 ca certidude[16379]: Sending e-mail certificate-renewed.md to Lauri Võsandi <lauri@example.com>
Mar 31 06:26:03 ca certidude[16379]: Traceback (most recent call last):
Mar 31 06:26:03 ca certidude[16379]:   File "/usr/lib/python2.7/wsgiref/handlers.py", line 85, in run
Mar 31 06:26:03 ca certidude[16379]:     self.result = application(self.environ, self.start_response)
Mar 31 06:26:03 ca certidude[16379]:   File "/usr/local/lib/python2.7/dist-packages/falcon/api.py", line 209, in __call__
Mar 31 06:26:03 ca certidude[16379]:     responder(req, resp, **params)
Mar 31 06:26:03 ca certidude[16379]:   File "/root/certidude/certidude/auth.py", line 50, in kerberos_authenticate
Mar 31 06:26:03 ca certidude[16379]:     return func(resource, req, resp, *args, **kwargs)
Mar 31 06:26:03 ca certidude[16379]:   File "/root/certidude/certidude/firewall.py", line 24, in wrapped
Mar 31 06:26:03 ca certidude[16379]:     return func(self, req, resp, *args, **kwargs)
Mar 31 06:26:03 ca certidude[16379]:   File "/root/certidude/certidude/firewall.py", line 33, in wrapped
Mar 31 06:26:03 ca certidude[16379]:     return func(self, req, resp, *args, **kwargs)
Mar 31 06:26:03 ca certidude[16379]:   File "/root/certidude/certidude/api/request.py", line 100, in on_post
Mar 31 06:26:03 ca certidude[16379]:     _, resp.body = authority._sign(csr, body, overwrite=True)
Mar 31 06:26:03 ca certidude[16379]:   File "/root/certidude/certidude/authority.py", line 351, in _sign
Mar 31 06:26:03 ca certidude[16379]:     certificate=cert,
Mar 31 06:26:03 ca certidude[16379]:   File "/root/certidude/certidude/mailer.py", line 96, in send
Mar 31 06:26:03 ca certidude[16379]:     conn.sendmail(config.OUTBOX_MAIL, recipients, msg.as_string())
Mar 31 06:26:03 ca certidude[16379]:   File "/usr/lib/python2.7/smtplib.py", line 747, in sendmail
Mar 31 06:26:03 ca certidude[16379]:     raise SMTPRecipientsRefused(senderrs)
Mar 31 06:26:03 ca certidude[16379]: SMTPRecipientsRefused: {u'Lauri V\xf5sandi <lauri@example.com>': (451, '4.3.5 <lauri@example.com>: Recipient address rejected: Server configuration error')}
Mar 31 06:26:03 ca certidude[16379]: 192.168.12.1 - - [31/Mar/2017 06:26:03] "POST /api/request/?autosign=true&wait=30 HTTP/1.1" 500 59

Fixed by submitting to local MTA, Postfix needs to be installed manually

repo: laurivosandi/certidude


text:
CNS-164 fixed typo in polygon spec name

Please check if the PR fulfills these requirements

[ ] Commit message follows the Contribution Guidelines
[ ] Tests ran locally and added/modified if needed
[ ] Docs have been added/updated, if applicable
[ ] If applicable - JIRA ticket ID was added

What kind of change does this PR introduce? (Bug fix, feature, unit tests, docs update, ...)

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?

Please describe what manual tests you ran, if applicable

Other information:

Thanks @Yaroms , this is great.
We'll need to also align the names either to fix lower/upper case, or make everything upper case

authors: SeanZoR Yaroms

repo: lavanet/lava


text:
Launch Error

Uncaught Error: [$injector:modulerr] Failed to instantiate module mrmagnet due to:
Error: [$injector:modulerr] Failed to instantiate module angular-svg-round-progress due to:
Error: [$injector:nomod] Module 'angular-svg-round-progress' is not available! You either misspelled the module name or forgot to load it. If registering a module ensure that you specify the dependencies as the second argument.
http://errors.angularjs.org/1.5.6/$injector/nomod?p0=angular-svg-round-progress
at file:///Users/Vinnie/Downloads/mrmagnet-master/node_modules/angular/angular.js:68:12
at file:///Users/Vinnie/Downloads/mrmagnet-master/node_modules/angular/angular.js:2101:17
at ensure (file:///Users/Vinnie/Downloads/mrmagnet-master/node_modules/angular/angular.js:2025:38)
at module (file:///Users/Vinnie/Downloads/mrmagnet-master/node_modules/angular/angular.js:2099:14)
at file:///Users/Vinnie/Downloads/mrmagnet-master/node_modules/angular/angular.js:4607:22
at forEach (file:///Users/Vinnie/Downloads/mrmagnet-master/node_modules/angular/angular.js:322:20)
at loadModules (file:///Users/Vinnie/Downloads/mrmagnet-master/node_modules/angular/angular.js:4591:5)
at file:///Users/Vinnie/Downloads/mrmagnet-master/node_modules/angular/angular.js:4608:40
at forEach (file:///Users/Vinnie/Downloads/mrmagnet-master/node_modules/angular/angular.js:322:20)
at loadModules (file:///Users/Vinnie/Downloads/mrmagnet-master/node_modules/angular/angular.js:4591:5)
http://errors.angularjs.org/1.5.6/$injector/modulerr?p0=angular-svg-round-p…wnloads%2Fmrmagnet-master%2Fnode_modules%2Fangular%2Fangular.js%3A4591%3A5)
at file:///Users/Vinnie/Downloads/mrmagnet-master/node_modules/angular/angular.js:68:12
at file:///Users/Vinnie/Downloads/mrmagnet-master/node_modules/angular/angular.js:4630:15
at forEach (file:///Users/Vinnie/Downloads/mrmagnet-master/node_modules/angular/angular.js:322:20)
at loadModules (file:///Users/Vinnie/Downloads/mrmagnet-master/node_modules/angular/angular.js:4591:5)
at file:///Users/Vinnie/Downloads/mrmagnet-master/node_modules/angular/angular.js:4608:40
at forEach (file:///Users/Vinnie/Downloads/mrmagnet-master/node_modules/angular/angular.js:322:20)
at loadModules (file:///Users/Vinnie/Downloads/mrmagnet-master/node_modules/angular/angular.js:4591:5)
at createInjector (file:///Users/Vinnie/Downloads/mrmagnet-master/node_modules/angular/angular.js:4513:19)
at doBootstrap (file:///Users/Vinnie/Downloads/mrmagnet-master/node_modules/angular/angular.js:1777:20)
at bootstrap (file:///Users/Vinnie/Downloads/mrmagnet-master/node_modules/angular/angular.js:1798:12)
http://errors.angularjs.org/1.5.6/$injector/modulerr?p0=mrmagnet&p1=Error%3…nloads%2Fmrmagnet-master%2Fnode_modules%2Fangular%2Fangular.js%3A1798%3A12)

I get this too, think I know what's wrong. 👍

authors: HDVinnie PXgamer

repo: laverix/mrmagnet


text:
external navigation links

Adding new icons to denote that a navigation item links to an external website.

Thanks, @arghac14


text:
meshery server cannot connect the traefik-mesh adapter.

Description
The traefik-mesh-adapter is not connected after running the installation command on the lab. The chip does not glow up, and when clicked gives this error
Could not ping adapter.. To configure an adapter, visit Settings
This error persists even after all the components are in a ready state when running mesheryctl system status command
Expected Behavior
The traefik-mesh adapter should be connected after running the installation command
Screenshots

Environment:

OS: macOS
Browser: Brave
Version: [e.g. 22]
Device: Laptop

To Reproduce
Steps to reproduce the behavior:

Go to https://layer5.io/labs
Click on Working with Meshery and Traefik card
Follow the instruction till the installation part
Open the Meshery server tab and we can see the meshery-traefik-mesh chip not activated.

@asubedy Will you reconfirm this issue?

Yes @leecalcote the traffic mesh adapter chip is still not glowing

authors: asubedy leecalcote


text:
[svg] Create RemoveDoneIcon

Open PR  #24

Contributor Guide

🎨 Wireframes and designs for Layer5 site in Figma (open invite)

done


text:
[chore] Update setup prerequisite card css

Notes for Reviewers
This PR updates the SetupPreReq Card component's CSS to have flex: 1 property.

Signed commits

[x] Yes, I signed my commits.

Thank you! Clarification: this means that if only two cards are present, they will split the landscape 50/50, right? And if only one is present, it will consume 100% of the width, right? @dragon-slayer875

Yes sir. Exactly that.


text:
[Docs] README: add weekly meeting details

Current State:
A weekly meeting focuses filters (and related topics) found in this repo.
Desired State:
Ideally, this readme has a description of and invitation to the weekly meeting (details -https://meet.layer5.io)

Contributor Resources

Meshery documentation site
Meshery documentation source
Instructions for contributing to documentation

Current State:
A weekly meeting focuses filters (and related topics) found in this repo.
Desired State:
Ideally, this readme has a description of and invitation to the weekly meeting (details -https://meet.layer5.io)
Contributor Resources

Meshery documentation site
Meshery documentation source
Instructions for contributing to documentation

Do I have to add the link of the weekly meetings,time and topic in readme?

Can I work on this?

That'd be great, @includestdlibrary

authors: includestdlibrary leecalcote


text:
增加合计行展示

增加合计行展示

Coverage decreased (-0.2%) to 29.518% when pulling 6c557188fdfb2306c6b7315498e7157e65c560ee on wenxx1:patch-1 into b7f9a53cd0ddbed8eb8f78b0bb0f524e80455967 on sentsin:master.

相应代码处已进行大幅修改，提交的分支代码存在冲突

authors: coveralls sentsim wenxx1

repo: layui/layui


text:
rails g authentication user --ratelimit

Implement ratelimit, I did it before....
  environment nil, env: "production" do <<~CODE
    config.middleware.use(
      Rack::Ratelimit, name: "General",
      rate:   [1000, 1.hour],
      logger: Rails.logger,
      redis:  Redis.new
    ) { |env| ActionDispatch::Request.new(env).ip }
  CODE

https://github.com/lazaronixon/authentication-zero/releases/tag/v2.6.0

repo: lazaronixon/authentication-zero


text:
Enable SearchGUI integration

Add support for items in tracking issue https://github.com/lazear/sage/issues/30

Support additional enzymes
Enable run-to-run reproducibility/deterministic output

Deterministic output
By switching hash functions during digest & disabling unstable sorting in a couple of key locations, we can achieve deterministic output - ignoring numerical stability of my eigenvector/LDA implementation.
Sage previously broke ties between top hits (e.g. isoleucine/leucine isomers, or random change matches with identical hyperscores) in an unspecified, and random way. This could lead to large swings in the number of PSMs reported across runs.
https://github.com/lazear/sage/commit/23f962965ec3b31b38119de4e9cfff7791e29fe5 changes the behavior to still be unspecified, but it is now consistent, rather than random. We should add in a special case for handling ambiguous PSMs (e.g. isoleucine/leucine isomers), and report both of them. For now, Sage simply reports one of them.
Numerical stability
Test from 24 million MS2 search. Note small differences in eigenvectors. 38 PSMs differential across 24 million MS2 spectra seems acceptable to call it "deterministic" :)
# Run 1
linear model fit with eigenvector: [-0.1896449482748973, 0.12244844766520412, -0.1351561296114254, -0.011337864090857014, -0.001227763460278724, 0.21813249388453376, 0.03475083958119062, -0.057011727725848504, 0.1547336301502748, 0.10312024555812987, -0.06851122010390676, 0.04638871723181479, -0.05516151252271889, -0.00019830037187928187, 0.9133123129501781]
discovered 6,344,685 peptide-spectrum matches at 1% FDR
discovered 94570 peptides at 1% FDR
discovered 8402 proteins at 1% FDR
finished in 911s

# Run 2
linear model fit with eigenvector: [-0.18959070431565897, 0.1224655547660972, -0.13516743497135975, -0.011343024100476055, -0.0012282249470035313, 0.2180549340525521, 0.034749391111184726, -0.05702409797547313, 0.15474626113784573, 0.10308615470299168, -0.06837161046491733, 0.04641101990610453, -0.05517385223806875, -0.0001980905663232505, 0.9133476384171597]
discovered 6,344,723 peptide-spectrum matches at 1% FDR
discovered 94571 peptides at 1% FDR
discovered 8402 proteins at 1% FDR
finished in 888s

Search w/o annotated precursor charge looks good on Harald's test file compared to X!Tandem & MSAmanda. Primary differences are mostly from Sage not yet supporting multiple of the same variable mods on a peptide (e.g. "PEPM[+15]YM[+15]")

repo: lazear/sage


text:
Show smaller NSFW tag if it's my content

Fixes #1581

@kkorte First contribution! Awesome 🙂
Checkout https://lbry.io/faq/tips if you'd like a tip for your contribution :)

authors: kkorte seanyesmunt

repo: lbryio/lbry-app


text:
Subscription downloads are not placed in their separate folders

Subscription downloads are not placed in their separate folders.
Suggested Solutions
Subscription downloads have to be placed in their separate folders.
System Configuration
App | 0.33.2
Daemon (lbrynet) | 0.37.5
Platform | Windows (Windows-10-10.0.17134)

Thanks for the suggestion!
Please see the #ideas-and-feedback channel on Chat.lbry.com to post your suggestions.
This should be possible via the save file api - it supports passing of a path.

authors: bug-re-port-er tzarebczan

repo: lbryio/lbry-desktop


text:
Fix edge case issue where 2 videos players existed on page causing double audio

This is a refactor of the video.js component's lifecycle hooks.
Existing behavior can cause strange behavior and may cause more than 1 video to play at the same time. This is because when any prop is updated for the VideoViewer component, it causes the entire video.js instance to be recreated.
Additionally it relies on a callback from an asynchronous function to begin the creation of a new video.js instance which further leads to unstable behavior. For instance, when the player is switching to a new source URL it begin a fetch to get the content's headers. While waiting for this fetch to return, if we close the player, it will be removed from the DOM by a parent component (FileRender). Our fetch will then finish and execute the callback which now creates a new videojs instance on a dom reference that has been removed. The result is a "zombie" video player that does not exist on the page, has no references to it that are accessible from the outer scope, and has now begun playing a video that we should have closed out.
To solve this, I refactored the video.js component so that it only creates a new instance on mount, and only removes and disposes of the player on unmount by scoping the useEffect hook to an empty array.
from: https://reactjs.org/docs/hooks-effect.html

If you want to run an effect and clean it up only once (on mount and unmount), you can pass an empty array ([]) as a second argument. This tells React that your effect doesn’t depend on any values from props or state, so it never needs to re-run. This isn’t handled as a special case — it follows directly from how the dependencies array always works.

There may still be better ways to approach this, and set values in react, but I'm not an expert in react yet, so open to suggestions and further improvements. However, this behavior should be more predictable and allow for this component to have additional properties set on it without interfering with the video. This also fixes the issues of double audio that I have been able to reproduce.
I dislike that a global window variable is needed to ensure we access the same instance of video js in our fetch callback, but it's just bad practice, not an issue that causes issues with the playback experience.
Overall, this should be more reliable when quickly switching between videos, and also reduce some load time slightly as it no longer needs to recreate the entire video player everytime the source changes.
PR Checklist

[x] I have checked that this PR is not a duplicate of an existing PR (open, closed or merged)
[x] I have checked that this PR does not introduce a breaking change
[ ] This PR introduces breaking changes and I have provided a detailed explanation below

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting)
[x] Refactoring (no functional changes)
[ ] Documentation changes
[ ] Other - Please describe:

Fixes
Issue Number: #5328
What is the current behavior?
duplicated video.js players that cannot be controlled and play without an instance appearing on the page.
What is the new behavior?
Only 1 video player will ever exist at a time on the page, and will only be recreated if unmounted.
Other information
There may potentially be bugs related to the change of autoplay from false to true but I haven't run into any personally yet, and expect autoplay to be more consistent with this value set to true.

With autoplay now set to true this may also resolve #5309

With autoplay now set to true this may also resolve #5309

This seems to work well. Will give it a more thorough code review once it's out of draft.
@btzr-io I think this refactor will fix the double play issue that popped up with your chromecast integration.

This seems to work well. Will give it a more thorough code review once it's out of draft.
@btzr-io I think this refactor will fix the double play issue that popped up with your chromecast integration.

authors: DispatchCommit seanyesmunt


text:
Change internet check to not use google.com

The connected to the internet check fails if run behind a firewall that blocks google.com. For example - in china.
┆Issue is synchronized with this Asana task

➤ Job Evers-Meltzer commented:
Testing unito sync

Testing unito sync the other way

Testing unito sync again again

repo: lbryio/lbry