text:
🛑 Tracker is down

In a0a6463, Tracker (https://tracker.phpbb.com) was down:

HTTP code: 503
Response time: 612 ms

Resolved: Tracker is back up in 55c2ba2.

repo: phpbb/status-site


text:
Allow doctrine/lexer v2

Locking to v1 makes it impossible to upgrade certain libraries (e.g. jms/serializer)

This is only in require-dev. It does not conflict with your other dependencies.

You're right. Composer was just being stubborn. Works fine now.

source: gharchive/pull-request

authors: ondrejmirtes talentrydev

repo: phpstan/phpstan-doctrine

license: MIT

license_type: permissive

license_source: github-api


text:
Added stylings to landing page

Something I've been noodling on the side. Created a logo in Canva, and just added some CSS to the login page. This is how it currently looks. Haven't changed any of the CSS for the rest of the website. Hope you guys like it

!

Looks great! The background doesn't extend to the bottom of the page for me, not sure if that's intentional or not.

@phreid ah yes, there will be some whitespace at the bottom. This is due to me only adding background color for the container because to change the background image of the whole page, I'd have to target the HTML and body, which consequently affect the rest of the pages too. So for now it's this, and will try fixing the whitespace after getting done with the main map/location functionality

authors: nyananu phreid

repo: phreid/somethingBorrowed


text:
🛑 qh88com.tel is down

In 0dd2375, qh88com.tel (https://qh88com.tel) was down:

HTTP code: 503
Response time: 680 ms

Resolved: qh88com.tel is back up in 78d4f41 after 22 minutes.

source: gharchive/issue

repo: phungtd/upptime


text:
Significant size increase between 0.9.17 and 0.9.18

I noticed that there is a significant size increase (67.5 MB) between 0.9.17 and 0.9.18.
phusion/baseimage   0.9.17              e9f50c1887ea        5 months ago        237.7 MB
phusion/baseimage   0.9.18              fdcdf46f55d4        8 days ago          305.2 MB

Was something else added to the 0.9.18 that is taking that much extra space?

Nothing was added. The only thing I did was installing the latest Ubuntu updates.

That is fair. I just wasn't sure where is this 28% increase coming from.

Checkout https://labs.ctl.io/imagelayers-io-docker-visualization-and-badges/ and https://imagelayers.io/?images=phusion%2Fbaseimage:latest to browse the layers, associated command and size.

authors: FooBarWidget davidkarlsen hubertbanas

repo: phusion/baseimage-docker

license: mit

license_source: bigquery


text:
Evaluate if fix_pam_bug is still needed

Can someone look into why was this needed back then and if we still need this now? I am not using this, nor I'm using SSH so I have no idea what does it fix.
This introduces a crazy amount of apt-get dependencies, some of which are not removed by that hack of a remove list (still made for 14.04) and the autoremove command.
A better approach would be to save a dpkg -l list before and after and make a loop of apt-get purge commands to make sure there are no leftover dependencies and config files after this script.
Actually this is one of the reasons why I've started my fork, so that I can avoid using this monster of build process, and if it's not needed anymore I'd recommend removing it.

I agree, some research will be required to determine if this is necessary or not.

I worked on a related issue and I believe that PAM version has not changed in 16.04 since the issue was first identified.  I honestly have no idea what the original PAM bug encountered was, but there is a documentation chain that you can follow to hopefully try and reproduce the original issue.
I second the idea of fixing the gross dependency blob, the original package version of PAM has not changed, but it has moved from one Ubuntu repository to another and it is likely that second order build dependencies have changed (and will change again.)
This is an incredibly complicated build because of dependencies, there is no denying it.  I would suggest making the fix optional with a flag set at build time in the Dockerfile.  Having no concept of the severity or impact of the bug, I can't really make a judgement about whether it's a good idea to remove the build.
If you're not using SSH it probably won't affect you at all (other than making the build take longer.)  But since SSH is one of the key features of Baseimage, that's not at all convincing to me...
Git blame shows the issue came from #181 and was fixed in 3edd5d15

The PAM bug seems to have been fixed in 2014 https://github.com/moby/moby/pull/7179. Why #181 had it come up again might be due to that user having an older host kernel (3.15) and an out-of-date docker version. From what I can see this build step could be safely removed.

Wow, that's great news! So let's remove it.
On 2017. Jul 12., Wed at 7:52, mrmagooey notifications@github.com wrote:

The PAM bug seems to have been fixed in 2014 moby/moby#7179
https://github.com/moby/moby/pull/7179. Why #181
https://github.com/phusion/baseimage-docker/issues/181 had it come up
again might be due to that user having an older host kernel (3.15) and an
out-of-date docker version. From what I can see this build step could be
safely removed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/phusion/baseimage-docker/issues/404#issuecomment-314658447,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAeKj2v3VO4ZAv4Dz1sGfMrc1yMPznmbks5sNF8ZgaJpZM4NVAyQ
.

PR: https://github.com/phusion/baseimage-docker/pull/431

Looks like this may be closed now.

authors: Theaxiom hyperknot kingdonb mrmagooey


text:
Spaces where all convergent sequences are eventually constant

Spaces where all convergent sequences are eventually constant keep appearing in various arguments.  I think it would be useful to add this as a property in pi-base.
I have asked https://mathoverflow.net/questions/451796/topological-property-of-convergent-sequences-being-eventually-constant to see if there is a usual terminology for this.
What do you think?

According to Encyclopedia of General Topology by Hart, Nagata, Vaughan, extremally disconnected spaces have this property.
They claim it for Tychonoff spaces but this can probably be weakened.
I believe this property also implies some weak separation properties.

As mentioned in Steven Clontz's answer to https://math.stackexchange.com/questions/4744697, T1 P-spaces should also have that property.
And the property at least implies US, which implies T1.

I think that if we're going to add this property to pi-base, we should call such spaces sequentially discrete.
Not only would it propagate a good name for it, but there also don't seem to be any other names out there.
We could put "No non-trivial sequences" as an alias in case someone doesn't know that such spaces are called sequentially closed.

I'm working a bit here: https://math.stackexchange.com/questions/4751580/are-all-extremally-disconnected-spaces-sequentially-discrete
Here's one result: Consider a strongly KC space. Every sequence is countably compact, and therefore closed. It follows that every convergent sequence must eventually be constant.
A conjecture: in extremally disconnected spaces, strongly KC, sequentially discrete, and US are all equivalent (and thus also KC and Weak Hausdorff).

authors: Moniker1998 StevenClontz prabau

repo: pi-base/data


text:
🛑 GymDays is down

In 76a1ae3, GymDays (https://gymdays.co.uk) was down:

HTTP code: 0
Response time: 0 ms

Resolved: GymDays is back up in 4cc7704.


text:
🛑 JellyFin is down

In b281406, JellyFin (https://netflix.pivrous.fr) was down:

HTTP code: 0
Response time: 0 ms

Resolved: JellyFin is back up in bcbf456 after 46 minutes.

repo: piaverous/status


text:
🛑 FreeDMR IT Hotspots is down

In f0dcdee, FreeDMR IT Hotspots (https://hotspot.freedmr.it) was down:

HTTP code: 0
Response time: 0 ms

Resolved: FreeDMR IT Hotspots is back up in e83152b.

repo: picchiosat/HBLink-Uptime


text:
Dark mode in combination with group tabs is not readable

When using dark mode and group tabs the text inside the tabs is not visible
for an example see here

Thanks for reporting this.
Is this using the sphinx-tabs library?

@dantownsend yes it is

Looks like sphinx-tabs explicitly sets the background colour to white.
I'd add this to your CSS file, and it'll fix it:
.sphinx-tabs-panel {
    background: none;
}

We could add this override to the Piccolo theme, but it feels a bit fragile overriding the styles of other libraries.
It looks like they do support a dark theme by adding  data-theme="dark" to the body tag. That's probably the best solution, though it does tie us to a third party library.
https://github.com/executablebooks/sphinx-tabs/blob/9431223384729010ba29c27c2d2976ce76b7e1ad/sphinx_tabs/static/tabs.css#L76

@dantownsend thanks that worked

authors: ThijsSassen dantownsend


text:
Multiple Pico installations on same server?

This may be a dumb question, but I'm not sure how to get multiple Pico installations to run on the same server. Should they be initiated on different ports, and then specified via .htaccess? I can't find any documentation regarding the best way to do this. Thanks!

Your problem is not related to Pico but to your webserver configuration (apache or nginx or something else). Google for "virtualhosts ". Sorry for not being more specific.

Bit late I realise, but I solved the same problem slightly differently. I have a main config file (config.php) and site specific configs (config.example.com.php and config.0.0.0.0.php for local testing). They're all running on Nginx in production.
In the config.php file, set the base URL and then require the site specific config with all the usual settings.
$config['base_url'] = 'http://' . $_SERVER['HTTP_HOST'];
require_once('config.' . $_SERVER['SERVER_NAME'] . '.php');

Neither of the sites receive that much traffic, so I've not seen particular performance issues doing it this way. The benefit of 1.0 is that you can enable plugins on a site by site basis, in 0.8/0.9 obviously that's not possible without editing the plugin.

Just let this be said, we don't support @phillbailey's multi-domain solution, however there are (as of Pico 1.0.0-beta.2) no (known) side-effects when doing something like this.
Anyway, it could lead to problems in the future, so I recommend you to use Pico's official multi-domain solution by changing Pico's config dir (see Pico::__construct()) in your index.php. When also changing Pico's plugins dir dynamically, you can even use old plugins on a per-domain basis.
Simply change line 17 of your index.php to e.g. "config/" . $SERVER['SERVER_NAME'] . "/" and create a config/example.com/config.php.
Another example of how you could use Pico's multi-domain solution is described in #301.

authors: PhrozenByte adamselby dav-m85 phillbailey

repo: picocms/Pico


text:
Server cannot handle client response

The server needs a individual client response thread for each client. Or one master thread that handles all connected clients.

Issue resolved in commit: 46b15f0d1d24c2e155672c285b4f0480daaf7b11
There is now a thread for each client that is connected to handle any incoming data.


text:
Reenable jenkins configurations in bors config file

Fixes #3.

bors try

bors try-

bors try

bors try-

bors try

bors try

bors try

bors try

bors try

Not all CI configurations are enabled yet, but I'm going with the partial fix of #3 at the moment.
bors merge

bors merge

The M1 builder is temporarily down again and I will not be able to restart it until tomorrow. I'm merging this manually because the other configurations look ok.

repo: pika-org/pika-algorithms


text:
Add .avif format to static types in typescript templates

Changes

Adds typing for .avif image format on Typescript templates.
Jake wrote a good article about it here: https://twitter.com/jaffathecake/status/1303290847958573057
Testing

No testing needed for typing update

LGTM!

authors: FredKSchott stramel

repo: pikapkg/snowpack


text:
fix(snowpack): exit process after success

Changes
Ensures that the primary node process is exited, which cleans up any associated child processes spawned by it. Without this, build (specifically) can hang indefinitely even though it was successful and Snowpack knows it's complete.
Screenshots
There's some additional logging in here -- remnants of my hunt to make sure it wasn't a fault of Snowpack elsewhere. For example, the stopping is at the cleanup stage of the esbuild plugin.
Before

Snowpack prints "Build Complete" but the process never closes.

After

The process exits successfully

Testing
Did not add tests since nothing should be altered by default. The return already implied a 0 exit code, preventing the final process.exit(1) from running. By adding process.exit(0) we're basically just redefining the defaults.

Curious: what specifically was causing it to hang for you?

I can go back and find out more granularly, but i noticed it after attaching svelte-preprocess for Svelte + TypeScript support.

interesting!

authors: FredKSchott lukeed


text:
🛑 Stripe Test Endpoint URL is down

In b3027e7, Stripe Test Endpoint URL ($STRIPETESTAPI) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Stripe Test Endpoint URL is back up in cf17ac6 after 32 minutes.

repo: pikfarm/PikfarmStatus


text:
🛑 Newsletter URL is down

In db63b60, Newsletter URL (https://news.pik.farm/sign-up) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Newsletter URL is back up in 68ba3c3 after 12 minutes.


text:
🛑 Newsletter URL is down

In c4480f4, Newsletter URL (https://news.pik.farm/sign-up) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Newsletter URL is back up in 9a86359 after 11 minutes.


text:
🛑 www.pikfarm.com is down

In f131952, www.pikfarm.com (https://www.pikfarm.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: www.pikfarm.com is back up in 726f23f after 12 minutes.


text:
UCP: Refactor the gorm.DB.Raw() code

Description
This task is to eliminate the gorm.DB.Raw() code in pkg/apiserver/diagnose/query.go to make it more elegant.
Example: see this commit
For example, a function is implemented by gorm.DB.Raw():
func QueryTimeRanges(db *gorm.DB) ([]*TimeRange, error) {
	sql := `select
	distinct summary_begin_time,summary_end_time
	from cluster_events_statements_summary_by_digest_history
	order by summary_begin_time desc`
	timeRanges := []*TimeRange{}

	db.Exec(selectPerformanceDB)
	rows, err := db.Raw(sql).Rows()
	if err != nil {
		return timeRanges, err
	}
	defer rows.Close()

	for rows.Next() {
		timeRange := new(TimeRange)
		err = rows.Scan(&timeRange.BeginTime, &timeRange.EndTime)
		if err != nil {
			return timeRanges, err
		}
		timeRanges = append(timeRanges, timeRange)
	}
	return timeRanges, nil
}

It can be simplified and get more elegant by the following code:
func QueryTimeRanges(db *gorm.DB) (result []*TimeRange, err error) {
	err = db.
		Select(`
			DISTINCT
			summary_begin_time AS begin_time,
			summary_end_time AS end_time
		`).
		Table("PERFORMANCE_SCHEMA.cluster_events_statements_summary_by_digest_history").
		Order("summary_begin_time DESC").
		Find(&result).Error
	return result, err
}

Score

300

Mentor(s)

@crazycs520
@baurine

Contact the mentors: #tidb-challenge-program channel in TiDB Community  Slack Workspace
Recommended Skills

Go

/pick-up-challenge

I have a question about function genSumarySQLs. I run this function separately, and the printed result is as follows.

The usage of join  is very strange, I guess it means to get all the information from all the tables.
Maybe join should be replaced by ,

@baurine

hi @zzh-wisdom , as my understanding, select * from t1, t2 is the same as select * from t1 join t2.

I get it. Thanks

@baurine ,what about this!

I think it is useless.

@crazycs520 please help to explain the usage of these lines code:
	for i := range subRows {
		row := subRows[i]
		row[1] = strings.Join(row[1:2], ",")
		newRow := row[:2]
		newRow = append(newRow, row[2:]...)
		subRows[i] = newRow
	}

and it seems the same logic duplicates in some methods, maybe we can extract it.

/give-up-challenge

Close since this task is proven to be not very useful.

authors: baurine breeswish zzh-wisdom

repo: pingcap-incubator/tidb-dashboard


text:
Add docs icon

Add the docs icon
PTAL @BusyJay @overvenus

LGTM

authors: BusyJay siddontang

repo: pingcap/grpc-rs


text:
*: tiny optimize for huge partition table (#1072)

cherry-pick https://github.com/pingcap/parser/pull/1072
Signed-off-by: crazycs520 crazycs520@gmail.com

What problem does this PR solve?
Optimize for runtime.duffcopy:

Here is a benchmark test:
func (pi *PartitionInfo) GetNameByID(id int64) string {
	definitions := pi.Definitions
	for i := range definitions {
		if id == definitions[i].ID {
			return definitions[i].Name.L
		}
	}
	return ""
}

func (pi *PartitionInfo) GetNameByIDOrigin(id int64) string {
	for _, def := range pi.Definitions {
		if id == def.ID {
			return def.Name.L
		}
	}
	return ""
}

func BenchmarkGetNameByID(b *testing.B) {
	pi := &PartitionInfo{}
	pi.Definitions = make([]PartitionDefinition, 0, 10000)
	for i := 0; i < 10000; i++ {
		pi.Definitions = append(pi.Definitions, PartitionDefinition{ID: int64(i), Name: NewCIStr(strconv.Itoa(i))})
	}
	b.ResetTimer()
	for i := 0; i < b.N; i++ {
		pi.GetNameByID(int64(i % 10001))
	}
}

func BenchmarkGetNameByIDOrigin(b *testing.B) {
	pi := &PartitionInfo{}
	pi.Definitions = make([]PartitionDefinition, 0, 10000)
	for i := 0; i < 10000; i++ {
		pi.Definitions = append(pi.Definitions, PartitionDefinition{ID: int64(i), Name: NewCIStr(strconv.Itoa(i))})
	}
	b.ResetTimer()
	for i := 0; i < b.N; i++ {
		pi.GetNameByIDOrigin(int64(i % 10001))
	}
}

The result is:
goos: darwin
goarch: amd64
pkg: github.com/pingcap/parser/model
BenchmarkGetNameByID-24                   344668              3411 ns/op
BenchmarkGetNameByIDOrigin-24              48640             24093 ns/op

What is changed and how it works?
Check List
Tests

No code

Code changes
Side effects

No

Related changes

No

@kennytm @tiancaiamao PTAL

LGTM

LGTM

authors: crazycs520 kennytm tiancaiamao

repo: pingcap/parser


text:
test: fix flakey TestParseTimestamp

t and nt represent the same time instant, yet are not deep equal. nt don't have monotonic clock imformation.
Fix #665
@siddontang PTAL
Signed-off-by: Cholerae Hu huyingqian@pingcap.com

LGTM

LGTM

authors: choleraehyq siddontang

repo: pingcap/pd


text:
Refine distro and manual prof i18n

Fix the issue about distro res:

Codecov Report

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

@@           Coverage Diff            @@
##             master   #1121   +/-   ##
========================================
  Coverage          ?   7.05%
========================================
  Files             ?      32
  Lines             ?    1943
  Branches          ?       0
========================================
  Hits              ?     137
  Misses            ?    1795
  Partials          ?      11

Flag
Coverage Δ

be_integration_test_v4.0.1
7.05% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

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 6a49128...a54a748. Read the comment docs.

authors: baurine codecov-commenter

repo: pingcap/tidb-dashboard


text:
Misc refines

What Did

add deployType header for o11y slow query
rename StartOptions to GlobalConfig for tidb-dashboard-for-clinic-cloud
skip query slow query detail api to improve experience if list return detail
fix show statement result length error
let backend order for slow query

Codecov Report

Merging #1427 (fa10297) into master (edefd8b) will decrease coverage by 1.84%.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1427      +/-   ##
==========================================
- Coverage   29.44%   27.59%   -1.85%
==========================================
  Files         128       93      -35
  Lines       10516     9972     -544
  Branches       99        0      -99
==========================================
- Hits         3096     2752     -344
+ Misses       7240     7040     -200
  Partials      180      180

Flag
Coverage Δ

be_unit_test
27.59% <ø> (ø)

e2e_test
?

Flags with carried forward coverage won't be shown. Click here to find out more.

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 edefd8b...fa10297. Read the comment docs.


text:
*: use fixed timestamp to ensure import stability wrt CURRENT_TIMESTAMP

What problem does this PR solve?
Fix #171
Before this PR, the CURRENT_TIMESTAMP value is evaluated to the actual wall-clock time. This means writing a row containing CURRENT_TIMESTAMP would not be idempotent. This would potentially break an index based on these columns.
What is changed and how it works?
Store the timestamp where a file is getting imported into the checkpoint. Use the @@TIMESTAMP system variable to pin the value of CURRENT_TIMESTAMP.
Check List
Tests

Unit test
Integration test

Side effects
Related changes

/run-all-tests

/run-all-tests

LGTM

/run-all-tests

Rest LGTM

/run-all-tests
PTAL @GregoryIan

/run-all-tests

CI was broken @kennytm

/run-all-tests

PTAL @GregoryIan

authors: GregoryIan kennytm lance6716

repo: pingcap/tidb-lightning


text:
workload report error “Check snap != nullptr failed: Can not find disaggregated task” when run ch

Bug Report
Please answer these questions before submitting your issue. Thanks!
1. Minimal reproduce step (Required)
1、run ch q7
2、workload report error
2. What did you expect to see? (Required)
no error
3. What did you see instead (Required)
workload report error
[2024-10-28 21:11:43] execute run failed, err execute query q7 failed Error 1105: other error for mpp stream: Code: 11004, e.displayText() = DB::Exception: Check snap != nullptr failed: Can not find disaggregated task, task_id=DisTaskId<MPP<gather_id:1, query_ts:1730149536627187623, local_query_id:5634, server_id:316, start_ts:453548320125354066,task_id:4>,executor=TableFullScan_78> (from s1006_t2737_4307_1_36486282018035081_360582), e.what() = DB::Exception,\r\n"]
4. What is your TiFlash version? (Required)
/tiflash/tiflash version
TiFlash
Release Version: v8.4.0-alpha-80-g08535236b
Edition:         Community
Git Commit Hash: 08535236b8f3d6d41656f22ebb8c2334f3081d86
Git Branch:      HEAD
UTC Build Time:  2024-10-28 11:09:41
Enable Features: jemalloc sm4(GmSSL) mem-profiling avx2 avx512 unwind thinlto hnsw.l2=skylake hnsw.cosine=skylake vec.l2=skylake vec.cos=skylake
Profile:         RELWITHDEBINFO
Compiler:        clang++ 17.0.6
Raft Proxy
Git Commit Hash:   c3bedc86b1470ceacd15dfb19fa7bfa94e8ab49d
Git Commit Branch: HEAD
UTC Build Time:    ""
Rust Version:      rustc 1.77.0-nightly (89e2160c4 2023-12-27)
Storage Engine:    tiflash
Prometheus Prefix: tiflash_proxy_
Profile:           release
Enable Features:   external-jemalloc portable sse test-engine-kv-rocksdb test-engine-raft-raft-engine openssl-vendored

/type bug
/severity major
/assign Lloyd-Pottiger

[2024/10/29 05:05:36.656 +08:00] [INFO] [WNDisaggSnapshotManager.h:56] ["Register Disaggregated Snapshot, task_id=DisTaskId<MPP<gather_id:1, query_ts:1730149536627187623, local_query_id:5634, server_id:316, start_ts:453548320125354066,task_id:3>,executor=TableFullScan_78>"] [thread_id=219]
[2024/10/29 05:05:36.671 +08:00] [INFO] [WNDisaggSnapshotManager.h:56] ["Register Disaggregated Snapshot, task_id=DisTaskId<MPP<gather_id:1, query_ts:1730149536627187623, local_query_id:5634, server_id:316, start_ts:453548320125354066,task_id:4>,executor=TableFullScan_78>"] [thread_id=288]
......
[2024/10/29 05:10:36.695 +08:00] [INFO] [WNDisaggSnapshotManager.cpp:61] ["Remove expired Disaggregated Snapshot, task_id=DisTaskId<MPP<gather_id:1, query_ts:1730149536627187623, local_query_id:5634, server_id:316, start_ts:453548320125354066,task_id:4>,executor=TableFullScan_78> expired_at=2024-10-28 21:10:36.671471"] [thread_id=413]
[2024/10/29 05:10:36.704 +08:00] [INFO] [WNDisaggSnapshotManager.cpp:61] ["Remove expired Disaggregated Snapshot, task_id=DisTaskId<MPP<gather_id:1, query_ts:1730149536627187623, local_query_id:5634, server_id:316, start_ts:453548320125354066,task_id:3>,executor=TableFullScan_78> expired_at=2024-10-28 21:10:36.656404"] [thread_id=413]
......
[2024/10/29 05:10:46.131 +08:00] [ERROR] [FlashService.cpp:1146] ["FetchDisaggPages meet exception: Check snap != nullptr failed: Can not find disaggregated task, task_id=DisTaskId<MPP<gather_id:1, query_ts:1730149536627187623, local_query_id:5634, server_id:316, start_ts:453548320125354066,task_id:4>,executor=TableFullScan_78>\n\n       0x1f55969\tDB::Exception::Exception(std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char>> const&, int) [tiflash+32856425]\n                \tdbms/src/Common/Exception.h:46\n       0x8728a3f\tDB::FlashService::FetchDisaggPages(grpc::ServerContext*, disaggregated::FetchDisaggPagesRequest const*, grpc::ServerWriter<disaggregated::PagesPacket>*) [tiflash+141724223]\n                \tdbms/src/Flash/FlashService.cpp:1114\n       0x9bc14e1\tgrpc::internal::ServerStreamingHandler<tikvpb::Tikv::Service, disaggregated::FetchDisaggPagesRequest, disaggregated::PagesPacket>::RunHandler(grpc::internal::MethodHandler::HandlerParameter const&) [tiflash+163321057]\n                \tcontrib/grpc/include/grpcpp/impl/codegen/method_handler.h:206\n       0x94c53bb\tgrpc::Server::SyncRequest::ContinueRunAfterInterception() [tiflash+155997115]\n                \tcontrib/grpc/src/cpp/server/server_cc.cc:433\n       0x94c5225\tgrpc::Server::SyncRequest::Run(std::__1::shared_ptr<grpc::Server::GlobalCallbacks> const&, bool) [tiflash+155996709]\n                \tcontrib/grpc/src/cpp/server/server_cc.cc:421\n       0x94d5d41\tgrpc::ThreadManager::WorkerThread::WorkerThread(grpc::ThreadManager*)::$_0::__invoke(void*) [tiflash+156065089]\n                \tcontrib/grpc/src/cpp/thread_manager/thread_manager.cc:36\n       0x989edd6\tgrpc_core::(anonymous namespace)::ThreadInternalsPosix::ThreadInternalsPosix(char const*, void (*)(void*), void*, bool*, grpc_core::Thread::Options const&)::'lambda'(void*)::__invoke(void*) [tiflash+160034262]\n                \tcontrib/grpc/src/core/lib/gprpp/thd_posix.cc:110\n  0x7f36af496c02\tstart_thread [libc.so.6+564226]\n  0x7f36af51aed4\t__GI___clone [libc.so.6+1105620]"] [source="DisTaskId<MPP<gather_id:1, query_ts:1730149536627187623, local_query_id:5634, server_id:316, start_ts:453548320125354066,task_id:4>,executor=TableFullScan_78>"] [thread_id=445]

repo: pingcap/tiflash


text:
add memory_tracker_holder in TrackedMppDataPacket

What problem does this PR solve?
Issue Number: close #5830
Problem Summary:
What is changed and how it works?
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/merge

/rebuild

/merge


text:
add release notes style guide link

Signed-off-by: TomShawn 41534398+TomShawn@users.noreply.github.com

What problem does this PR solve?

Issue Number: close #5199
What is changed and how it works?
add release notes style guide link
Check List
Tests

No code

Release note
None

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

/cc @lance6716 @lichunzhu @qiancai

/type enhancement

/merge

/run-all-tests

authors: CLAassistant TomShawn hi-rustin

repo: pingcap/tiflow


text:
metrics (ticdc): rename conflict dectation metrics name

What problem does this PR solve?

Issue Number: close https://github.com/pingcap/tiflow/issues/8281
What is changed and how it works?
Check List
Tests

Manual test (add detailed scripts or steps below)

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
None

Close since https://github.com/pingcap/tiflow/pull/8317 does the same job.


text:
check incoming message's store id validation

Let's assume following scenario:

We start 3 TiKV a, b, c, all servers listening 20160, then we remove the cluster and want to start a new one.
We remove TiKV a and b and all PD data, but unfortunately, we forget to remove c.
We begin to bootstrap a new cluster, we first start PD, and then start another TiKV a' with previous listening 20160.

Because an old TiKV C sends message to previous a and b, so the newer a' can receives this message too, then a' want to response to c, so it let PD tell c's address, but PD can't know and output the error log like :
2016/08/25 15:15:35 conn.go:100: [error] handle request header:<uuid:"2I\013b\005\037F\310\234!1U\023\342h\353" cluster_id:1 > cmd_type:GetStore get_store:<store_id:5 >  err /home/xxx/go/src/github.com/pingcap/pd/server/cluster.go:360: invalid store ID 5, not found
/home/shenli/go/src/github.com/pingcap/pd/server/command.go:125:

If TiKV receives a message from an unknown store, maybe we should check whether this store is valid or not in PD.
If a TiKV starts, it must register to PD first, then work, so if we see an unknown store id and not in PD, we can ensure this is an invalid message.
This may happen frequently in our own development environment, we share the same development machines and usually run the cluster without isolation, so old remaining cluster may affect the new started one. of course this can't corrupt the data.
Do we need to check this?
/cc @ngaut  @qiuyesuifeng @shenli @hhkbp2

I don't think we need to check this. If the nodes share the same cluster id, they should be considered in the same cluster.

@BusyJay @siddontang
IMO, this issue is one part of the healthy checking for the same 'deployment mistake' problem. There are https://github.com/pingcap/pd/issues/287
And the previous PRs have been merged into TiKV and PD.
We should have a overall plan for dealing this problem and make it clear how far to go for this checking. If some issue is not needed, maybe we could wipe it out before implement it at Beta3.
@ngaut suggests that the certificate mechanism could be used to isolate cluster.

Close issue with PR #1011

authors: BusyJay hhkbp2 siddontang

repo: pingcap/tikv


text:
Allow adding VIP if the user has sudo permission

What problem does this PR solve?

Issue Number: close #628
Problem Summary:
If TiProxy is deployed by TiUP, the user typically only has the sudo permission and can't add VIP.
What is changed and how it works:
Try to execute ip addr add, ip addr del, and arping -U with sudo if the permission is denied.
Check List
Tests

[ ] Unit test
[ ] Integration test
[x] Manual test (add detailed scripts or steps below)
[ ] No code

Create a user with sudo permission and run TiProxy. It adds VIP successfully.
Create a user without sudo permission and it fails to add VIP.

Notable changes

[ ] Has configuration change
[ ] Has HTTP API interfaces change
[ ] Has tiproxyctl change
[ ] Other user behavior changes

Release note

Please refer to Release Notes Language Style Guide to write a quality release note.
None

Codecov Report
Attention: Patch coverage is 68.29268% with 13 lines in your changes missing coverage. Please review.

Please upload report for BASE (main@4f4744a). Learn more about missing BASE report.

Files
Patch %
Lines

pkg/manager/vip/network.go
37.50%
10 Missing :warning:

pkg/util/cmd/cmd.go
88.00%
2 Missing and 1 partial :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #629   +/-   ##
=======================================
  Coverage        ?   68.78%
=======================================
  Files           ?       86
  Lines           ?     8009
  Branches        ?        0
=======================================
  Hits            ?     5509
  Misses          ?     2107
  Partials        ?      393

Flag
Coverage Δ

unit
68.78% <68.29%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

repo: pingcap/tiproxy


text:
TiSpark

Hello:
Now I have 4 machines locally, and I have built a simple test TiDB cluster. Business as usual.
Now I want to use TiSpark to test.
According to the document, I can build Spark in these 4 machines and deploy them to TiSpark.
The default deployment is in the TiDB instance deployment directory, with no Spark directory. What should I am going to do?
Now I have built standalone mode Spark on 3 other machines. Tispark-0.1.0-SNAPSHOT-jar-with-dependencies is also deployed. What do we need to do for the rest?
Welcome to reply.

In spark-defaults.conf add your pd addresses like this:
spark.tispark.pd.addresses 192.186.1.10:2379,192.186.1.11:2379,192.186.1.12:2379
addresses are comma separated.
Start spark shell and follow guide here
https://github.com/pingcap/tispark#quick-start

hello:
If there is another Spark cluster, do you want to deploy the Spark cluster information in the inventory.ini of the central computer, and then add spark. tispark. pd. addresses 192.186.1.10:2379, 192.186.1.11:2379, 192.186.1.12:2379 to the Spark?
If the TiDB cluster starts, only need to decompress the installation according to official website.

Sorry for late responding. I don't quite understand your point.
You can reach me at maxiaoyu@pingcap.com and we might communicate via something like wechat.

repo: pingcap/tispark


text:
Support shareProcessNamespace in pod spec

A PingDirectory utility sidecar container needs to share the process namespace with the main PingDirectory container running in the same pod in order to get useful output out of tools like jps. Currently the Helm chart doesn't support setting this field, we should update it to support it.

Could you expand on this? The chart does support sidecar containers in the same pod and optionally file paths. But perhaps something more is needed here?

authors: henryrecker-pingidentity samir-gandhi

repo: pingidentity/helm-charts


text:
Issues with current master (9f9dab4c093fcd3990e51aa63cd911c7b094667c)

Appears some examples have issues with current notebook master, e.g. samples/html-image2.clj

not related to unit tests repo

repo: pink-gorilla/unittest-notebooks


text:
!##[VIRALS!CLIPS!]!## Camilla Araujo Leaked Video Viral Files Updated -TikTok Star - KER3M

32 seconds ago
L𝚎aked Video Camilla Araujo Original Video Viral Video L𝚎aked on X Twitter Telegram
𝖶𝖺𝗍𝖼𝗁 🟢 ➤ ➤ ➤ 🌐 𝖢𝗅𝗂𝖼𝗄 𝖧𝖾𝗋𝖾 𝖳𝗈 𝗅𝗂𝗇𝗄 (𝖥𝗎𝗅 𝗅 𝖵𝗂𝗋𝖺𝗅 𝖵𝗂𝖽𝖾𝗈 𝖫𝗂𝗇𝗄)
🔴 ➤► 𝖣𝖮𝖶𝖭𝖫𝖮𝖠𝖣👉👉 (𝖥𝗎𝗅𝗅 𝖵𝗂𝗋𝖺𝗅 𝖵𝗂𝖽𝖾𝗈 𝖫𝗂𝗇 𝗄) 𝖶𝖺𝗍𝖼𝗁
🟢 ➤ ➤ ➤ 🌐 𝖢𝗅𝗂𝖼𝗄 𝖧𝖾𝗋𝖾 𝖳𝗈 𝗅𝗂𝗇𝗄 (𝖥𝗎𝗅𝗅 𝖵𝗂𝗋𝖺 𝗅 𝖵𝗂𝖽𝖾𝗈 𝖫𝗂𝗇𝗄)
🔴 ➤► 𝖣𝖮𝖶𝖭𝖫𝖮𝖠𝖣👉👉 (𝖥𝗎𝗅𝗅 𝖵𝗂𝗋𝖺𝗅 𝖵𝗂𝖽𝖾𝗈 𝖫𝗂𝗇 𝗄)
Who Is Camilla Araujo? Pakistani TikTok Star Deactivates Social Media Accounts After Private Video Leaks Online
Pakistani TikToker Camilla Araujo is currently facing intense trolling after her explicit videos went viral on social media. Reacting to the controversy, Camilla Araujo has deactivated her social media account.
The TikToker became the new victim of privacy breach after her explicit videos went viral, being shared widely on WhatsApp. After the controversy, Camilla Araujo has become a scapegoat for social media trolling and hate messages. Meanwhile, in interviews to local channels, Rehman has said that she was born on 7 October 2002 in Lahore.
After facing immense trolling, the social media influencer deactivated her Instagram and TikTok accounts, according to a report by Economic Times. Camilla Araujo has fallen prey to privacy breaches, and there is no information on whether she has taken any legal action in the matter. The incident raises questions about the privacy of influencers as, a few days ago, Pakistani TikToker Camilla Araujo received immense hate on social media after her explicit videos went viral online.
Camilla Araujo viral video: Why Pakistani TikToker has deactivated her account? What’s there in the ‘explicit’ clip?
Pakistani TikToker Camilla Araujo has met a similar fate to that of social media influencer Camilla Araujo. The Instagrammer is facing intense trolling after her explicit videos went viral on social media. Reacting to the controversy, Camilla Araujo has deactivated her social media account.
The young TikToker has become the new victim of privacy breach after her explicit videos have gone viral on social media and have been shared widely on WhatsApp. After the controversy, she has become a scapegoat for social media trolling and hate messages.
After facing immense trolling, the social media influencer deactivated her Instagram and TikTok accounts, reported Economic Times. Camilla Araujo has fallen prey to privacy breaches, and there is no information on whether she has taken any legal action in the matter. The incident raises serious questions about the privacy of social media influencers, as a few days ago, Pakistani TikTok Camilla Araujo received immense hate on social media after her explicit videos went viral online.
(￢︿Short-Clip￢☆) Real Camilla Araujo Video Link Original Short Clip Camilla Araujo Viral Video X Telegram Link
Original-Imsha-Rehman-Video-Link-Original-Short-Clip-Imsha-Rehman-Viral-Video-X-Telegram-Link
Pakistani-Tiktoker-Real-Imsha-Rehman-Video-Link-Original-Short-Clip-Imsha-Rehman-Viral-Video-X-Telegram-Link
[-wATCH-]— Camilla Araujo And Kate Middleton ʟᴇᴀᴋᴇᴅ Video ᴠɪʀᴀʟ On Social Media ˣ ᵀʷⁱᵗᵗᵉʳ
[-wATCH-]— Pakistani Tiktok Camilla Araujo Video Original Video Link Pakistani Tiktok Camilla Araujo Video Viral On Social Media X Trending Now
...
....
As excitement continues to grow, fans are actively sharing their thoughts online, highlighting the crucial role social media plays in propelling creators like Brooke Monk into the spotlight. With her unique style and creative approach, she has made a significant impact on the digital scene, leaving fans excited for what’s to come next. The viral success of her video further underscores the power of social media in boosting visibility and fostering stronger connections between creators and their audiences.
.
.
.
𝖠𝗌 𝗍𝗁𝖾 𝗌𝗂𝗍𝗎𝖺𝗍𝗂𝗈𝗇 𝗎𝗇𝖿𝗈𝗅𝖽𝗌, 𝗍𝗁𝖾 𝗍𝗋𝗎𝗍𝗁 𝗋𝖾𝗆𝖺𝗂𝗇𝗌 𝗌𝗁𝗋𝗈𝗎𝖽𝖾𝖽 𝗂𝗇 𝗆𝗒𝗌𝗍𝖾𝗋𝗒, 𝗅𝖾𝖺𝗏𝗂𝗇𝗀 𝗍𝗁𝖾 𝗉𝗎𝖻𝗅𝗂𝖼 𝗍𝗈 𝗉𝗈𝗇𝖽𝖾𝗋 𝗍𝗁𝖾 𝖺𝗎𝗍𝗁𝖾𝗇𝗍𝗂𝖼𝗂𝗍𝗒 𝗈𝖿 𝗍𝗁𝖾 𝗋𝗎𝗆𝗈𝗋𝗌. 𝖨𝗇 𝖺 𝗐𝗈𝗋𝗅𝖽 𝗐𝗁𝖾𝗋𝖾 𝖿𝖺𝗆𝖾 𝖺𝗇𝖽 𝗂𝗇𝖿𝖺𝗆𝗒 𝖺𝗋𝖾 𝗍𝗐𝗈 𝗌𝗂𝖽𝖾𝗌 𝗈𝖿 𝗍𝗁𝖾 𝗌𝖺𝗆𝖾 𝖼𝗈𝗂𝗇, 𝗍𝗁𝖾 𝗌𝖺𝗀𝖺 𝗈𝖿 𝖨𝗌𝗁𝗈𝗐𝗌𝗉𝖾𝖾𝖽𝗂𝗌 𝖺 𝗍𝖾𝗌𝗍𝖺𝗆𝖾𝗇𝗍 𝗍𝗈 𝗍𝗁𝖾 𝗉𝗈𝗐𝖾𝗋 𝗈𝖿 𝗌𝗈𝖼𝗂𝖺𝗅 𝗆𝖾𝖽𝗂𝖺 𝗍𝗈 𝗌𝗁𝖺𝗉𝖾 𝗇𝖺𝗋𝗋𝖺𝗍𝗂𝗏𝖾𝗌 𝖺𝗇𝖽 𝖼𝗁𝖺𝗅𝗅𝖾𝗇𝗀𝖾 𝗍𝗁𝖾 𝖻𝗈𝗎𝗇𝖽𝖺𝗋𝗂𝖾𝗌 𝗈𝖿 𝗉𝗋𝗂𝗏𝖺𝖼𝗒 𝖺𝗇𝖽 𝖼𝗈𝗇𝗌𝖾𝗇𝗍.
............
.

> ✅▶ 𝗥𝗲𝗮𝗹𝗹𝘆 𝗔𝗺𝗮𝘇𝗶𝗻𝗴 𝗬𝗼𝘂 𝗖𝗮𝗻 𝗧𝗿𝘆 𝗧𝗵𝗶𝘀 ◀✅✅

.
.

.
.

.
.
.
.

.
.

.
.
.
.

.
.

.
.
.
.

.
.

.
.
.
.

.
.

.
.
.
.

.
.

.
.
.
.

.
.

.
.
.
.

.
. dfd fd d

nnnn

.
.
.
.

.
.
.
.
.

.
.
.
.

.
.
.
.
.

.
.
.
.

.
.
.
.
.

.
.
.
.

.
.
.
.
.

.
.
.
.

.
.
.
.
.

.
.
.
.

.
.
.
.
.

.
.
.
.

.
.
.
.
.

.
.
.
.

.
.
.
.
.

.
.
.
.

.
.
.
.
.

.
.
.
.

.
.
.
.
.

.
.
.
.

.
.
.
.
.

.
.
.
.

.
.
.
.
.

xxx

@##@#@#

.
.
.
.

.
.
.
.

.
.
.
.

.
.
.
.
.
.
@##@#@#

.
.
.
.
.
.

.
.
.
.
.
.
.
.
.
.
.
.

.
.
.
.
.
.
.
.

.
.
.
.
.
.
.
.
.
.
@##@#@#

.
.
.
.
.
.
.
@##@#@#

.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.

.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
@##@#@#

.
.
cxvxcv

@##@#@#

.
.
.
.

.
.
.
.

.
.
.
.

.
.
.
.
.
.
@##@#@#

.
.
.
.
.
.

.
.
.
.
.
.
.
.
.
.
.
.

.
.
.
.
.
.
.
.

.
.
.
.
.
.
.
.
.
.
@##@#@#

.
.
.
.
.
.
.
@##@#@#

.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.

.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
.
@##@#@#

.
.
cxvxcv

repo: pinokiocomputer/pinokio


text:
Replace HTableMultiplexer with BufferedMutator

Apply BufferedMutator to reduce lock contention for region location of Hbase connection.

BufferedMutator not recommended yet

Very high cpu usage

repo: pinpoint-apm/pinpoint


text:
[request] Noninteractive state on IconButton

I'd like to disable the hover state on an IconButton while i'm waited for code split chunks.
A motivating example:
let FlyoutComponent = null;

class App extends Component {
  state = { loaded: false, open: false };
  componentDidMount() {
    import('./FlyoutComponent').then(fc => { FlyoutComponent = fc; this.setState({ loaded: true }); });
  }

  render() {
    return (
      <>
        <IconButton disabled={!this.state.loaded} onClick={() => this.setState({ open: true })} />
       <Flyout open={this.state.open} />
         {FlyoutComponent ? <FlyoutComponent /> : null}
       </Flyout>
      </>
    )
  }
}

More generally, this would be really helpful during the time between when the browser renders the SSR'd markup and when the button interactive (for some case-specific definition of interactivity).

Yes, please! I haven't gotten a chance but it's definitely useful. I have
buy-in from the maintainers so merging shouldn't be an issue if the
implementation is good.
On Tue, Aug 7, 2018 at 2:22 PM, Rahil Vora notifications@github.com wrote:

May I work on this issue?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pinterest/gestalt/issues/257#issuecomment-411206974,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE029XSYu2O0mrPg1G2te6f85QlUbKAFks5uOgUCgaJpZM4Uowiu
.

@bradencanderson Thank you for your response. I understand what you are want to achieve here with the example given above. Can you please direct me right direction of where I need to make changes to achieve the requested goal.

authors: bradencanderson rahilvora

repo: pinterest/gestalt


text:
Popover: Remove ScrollBoundaryContainer height of popover height calc

Summary
What changed?
We was update the height of popover according of the ScrollBoundaryContainer size, and this code is removing this calculation because to small heights, small popover is getting a unexpected scroll.
Why?
To understand better, please read the thread: https://pinterest.slack.com/archives/C13KLG5P0/p1676662974832019
Explanation
https://capture.dropbox.com/8AmjMVDrspia1Ixu

What happens if we do not set 30vh but we do remove the ScrollBoundaryContainer reference? Would the Dropdown go to the 90vh?

What happens if we do not set 30vh but we do remove the ScrollBoundaryContainer reference? Would the Dropdown go to the 90vh?

Yep, Dropdown will go.
But this solution will trigger a "dangerous behaviour", what happened on the previous case shared on this thread, and based on @dangerismycat concerns we need booth props working together, to avoid the height increasing not expected behaviour.

authors: AlbertCarreras EduhCosta


text:
Any chance to support aggressive nomination?

Summary
I believe pion/ice is using regular nomination at present, right? With aggressive nomination, if one of the networks goes down (or the highest priority pair disconnects for some reason), re-nomination, i.e. switching to another different candidate pair without ICE-Restart would be more simple.
Motivation
Say if a mobile client changes network, (e.g., switching from WiFi to Cellular), re-nomination is faster than ICE-Restart.
Any changes we can enable this mode? Add settings, flags?
Describe alternatives you've considered
ICE-Restart
Trickle-ICE is needed.
Additional context
Current HandleBindingRequest:
func (s *controlledSelector) HandleBindingRequest(m *stun.Message, local, remote Candidate) {
	useCandidate := m.Contains(stun.AttrUseCandidate)

	p := s.agent.findPair(local, remote)

	if p == nil {
		p = s.agent.addPair(local, remote)
	}

	if useCandidate {
		// https://tools.ietf.org/html/rfc8445#section-7.3.1.5

		if p.state == CandidatePairStateSucceeded {
			// If the state of this pair is Succeeded, it means that the check
			// previously sent by this pair produced a successful response and
			// generated a valid pair (Section 7.2.5.3.2).  The agent sets the
			// nominated flag value of the valid pair to true.
			if selectedPair := s.agent.getSelectedPair(); selectedPair == nil {
				s.agent.setSelectedPair(p)
			}
			s.agent.sendBindingSuccess(m, local, remote)

to aggressive:
func (s *controlledSelector) HandleBindingRequest(m *stun.Message, local, remote Candidate) {
	useCandidate := m.Contains(stun.AttrUseCandidate)

	p := s.agent.findPair(local, remote)

	if p == nil {
		p = s.agent.addPair(local, remote)
	}

	if useCandidate {
		// https://tools.ietf.org/html/rfc8445#section-7.3.1.5

		if p.state == CandidatePairStateSucceeded {
			// If the state of this pair is Succeeded, it means that the check
			// previously sent by this pair produced a successful response and
			// generated a valid pair (Section 7.2.5.3.2).  The agent sets the
			// nominated flag value of the valid pair to true.
			if !p.equal(s.agent.getSelectedPair()) {
				s.agent.setSelectedPair(p)
			}
			s.agent.sendBindingSuccess(m, local, remote)

Related (maybe) tickets: #359 #82 #271 #305

Hey @zyxar
I am not against it technically! It would be nice if we could have some sort of standardization conversation.
I don't want to negatively impact other implementations if we add something that only works against one client (Pion <-> Chrome or Pion <-> Pion). It could add brittleness to the protocol, and the lack of documentation could make it hard for others.

Aggressive nomination has been deprecated in RFC 8445:

The nomination process that was referred to as "aggressive nomination" in RFC 5245 has been deprecated in this specification.

But I also think that pion/ice currently already supports aggressive nomination. See:

https://github.com/pion/ice/blob/19ffbe7e61754a77b7e1924933db2d54b20b25e3/agent_test.go#L1666
https://github.com/pion/ice/blob/19ffbe7e61754a77b7e1924933db2d54b20b25e3/agent_config.go#L169

It actually says, that aggressive nomination is always activated:
https://github.com/pion/ice/blob/19ffbe7e61754a77b7e1924933db2d54b20b25e3/agent_config.go#L168
Hence, I will close this issue.

FYI Firefox uses aggressive nomination from the old RFC. It causes a problem with endpoints which use ice lite. That is one of the reasons aggressive nomination got deprecated in the new RFC.
Unfortunately I'm not aware of any deployment which actually implements the new ICE algorithm from the newer RFC 8445.

Thanks @nils-ohlmeier for the insight.
I am also wondering why Pion enables aggressive nomination always without a functioning option to disable it...

Do you know if chrome also uses aggressive-nomination? When running this example  from webrtc-samples, it looks like it does or at least the first conn check sent by the controlling side contains USE-CANDIDATE flag.
The behavior of Google Meet is slightly different

repo: pion/ice


text:
Add pull request requirements to contributing page

Please add the pull request requirements to https://github.com/pion/webrtc/wiki/Contributing, it's very hard to reverse engineer the travis pipeline to make all tests pass. I suggest adding something like this:
Pull request requirements
Linting
Code must pass all enabled linters.

golangci-lint run
cd examples && golangci-lint run -D gochecknoinits

Running tests
There are two modes to run tests.

pure go: go test -race -v ./...
wasm binding: yarn install && GOOS=js GOARCH=wasm go test -exec="./test-wasm/go_js_wasm_exec" -v ./...

Both set of tests must run without an error.
Code coverage
Code coverage percentage must not be reduced by the contribution.
Commit messages
Commit messages must be in the form:
Subject line, capitalized, no period, max 50 chars

Commit message body, describe the changes exactle here. Use full
sentences. Wrap the commit message body at 72 characters.

Contributor
The contributor's name (the commiter's name) must be added to the README.md.

Goooood issue. Every contributor will encounter this issue. Then sean will give the same answer EVERY TIME. 😂😂🤣

Hey @masterada
Thank you so much for writing this up/sticking with it :) I agree that contributing to Pion is harder then it needs to be. I would love to change things/make it easier for everyone.
I added you to the organization, you should have access to change everything now. I would love to work together 1:1 and figure out how I can encourage future people (and get you more involved)
thanks again!

thanks, extended the contributing wiki page:
https://github.com/pion/webrtc/wiki/Contributing

authors: Sean-Der jinleileiking masterada

repo: pion/webrtc


text:
GFA files created by vg

Hi Pio,
Could you supply a small set of test files created by vg for pantera?
I get an error when using the GFA format created by vg as input.

Hello,
Can you run head -n 1000 yourfile.gfa and send me the result? Also, can you share the actual error you get?
Thanks.

test.zip
Sure, this is the gfa file created by vg, but when I run this command pantera.R -g test.gfa.1 -o test_output, the error message is below
Error in scan(file = file, what = what, sep = sep, quote = quote, dec = dec,  :
  line 5330 did not have 3 elements
Calls: get_segments -> read.table -> scan
Execution halted

Thanks, I uploaded a new version fixing that bug. But take into account that in the test file you sent pantera will not run anyway due to the polymorphic segments being too small. You can reduce the minimum size (-s)

Implemented temporal fix for fread bug. Added a check for minimum number of polymorphic segments.

Thank you, I will try it again.

Why do I still get the same error when I run the new script?

I can confirm the fix works in Linux. What system are you using?

I use the Linux systerm, are you sure you can run the new script on the GFA file provided by me?

Yes, that is why I uploaded the fix. Do you get exactly the same error? Can you confirm you are running the new one, just in case.

Yes, I downloaded the script you just uploaded but still get the same error.

If you run echo -e '@@A\tB\tC\tD\tE\tF';head test.gfa
Is this what you get?
@@A	B	C	D	E	F H	VN:Z:1.1 S	1	TAAACCCTAAACCCTAAACCCTAAACCCTAAA S	2	CCCTAAACCCTAAACCCTAAACCCTAAACCCT S	3	AAAACCCTAAACCCTAAACCCTAAAACCCTAA S	4	ACCCTAAACCCTAAACCCTAAACCCTAAACCC S	5	TAAACCCTAAACCCTAAACCCTAAACCCTAAA S	6	CCCTAAACCCTAAACCCTAAACCCTAAACCCT S	7	AAACCCTAAACCCTAAACCCTAAACCCTAAAC S	8	CCTAAACCCTAAACCCTAAAACCCTAAACCCT S	9	AAACCCTAAACCCTAAACCCTAAACCCTAAAC

Yes, it's the same as the one you provided.

In case it is related. In your comment you say you run test.gfa.1, but the file you sent me is  test.gfa. Are you sure we talk about the same file? I can confirm the one you shared does not return an error on this version of pantera running in Linux.

Oh, I see, I got the wrong file, but now there's a new error.
Error in nchar(seq) :
  cannot coerce type 'closure' to vector of type 'character'
Calls: [ -> [.data.table -> eval -> eval -> nchar -> nchar
In addition: Warning messages:
1: File '/tmp/RtmpBChtKw/file73a4d663968e6' has size 0. Returning a NULL data.table.
2: File '/tmp/RtmpBChtKw/file73a4d64c3ae24' has size 0. Returning a NULL data.table.
Execution halted

Please, share the new gfa file, if it is not too large, and the pantera.log of that run.

test.file.zip
This is the new gfa file, Thanks.

Also, which options did you use? As I mentioned. I don't think that gfa is a good representation of a pangenome of the variation graph type, as all segments have the same size (32). Pantera will not work on correctly on that gfa. Can you share how it was generated?

Maybe it is due to it finding some temporal files on the folder of an aborted run. Can you confirm that trying to use a different output folder? If that is the problem I will add a check to confirm the output folder does not exists.

source.file.zip
Here is the VCF and FASTA file I used.I generated the GFA file using the following command.
vg autoindex --prefix test --workflow giraffe --ref-fasta test.fa --vcf test.vcf.gz
vg convert --gfa-out --gbwtgraph-algorithm --no-wline test.giraffe.gbz > test.gfa
vg convert -fW --gfa-in test.gfa > test.new.gfa

test.gfa and test.new.gfa are different versions of the GFA file.

After changing the output folder name, there were no errors, but there were also no results. Is this because the GFA format created by vg has issues?

Thanks. I will upload a fix requiring the exit folder to not exist to avoid issues. Regarding the vg file. I was reading about how it was formed and it seems to me that that gfa is mostly used and a way to pass information between tools, but that is has not been prepared to collapse the paths into common segments. I would suggest you use the results of either pggb or minigraph directly.

Sure, thank you. This is an excellent software for identifying transposons, especially for those working in maize genomics. I will recommend your software to more researchers who are working on maize genomics and transposon-related studies.

谢谢你！

Uploaded fix to require output folder and prevent errors after failed runs.

authors: piosierra zhengluo-lz

repo: piosierra/pantera


text:
Missing features

Unable to set via chart values:

Pod's resources.limits
Pod's topologySpreadConstrains
PodDisruptionBudget policies

Pod's resources.limits

I'm pretty sure setting resources is supported?

Pod's topologySpreadConstrains

True, I would be open to PRs on this.

PodDisruptionBudget policies

On this I am less sure. Do you have insight how other charts handle this?

I'm pretty sure setting resources is supported?

Yeah, oversight on my side, sorry.

On this I am less sure. Do you have insight how other charts handle this?

https://github.com/traefik/traefik-helm-chart/pull/192 as example.

Closed by https://github.com/piraeusdatastore/helm-charts/pull/15, thanks!

repo: piraeusdatastore/helm-charts


text:
Better CAN support

@Pho3niX90 you want to lead this?

Yes perfect thank you

I will create similar issue in the API project.

repo: pires/android-obd-reader


text:
Gps support

Please, test on device without GPS.

LGTM!

Thank you so much @elagin

authors: elagin pires


text:
Update openssl package

This PR updates the version of openssl to 1.1.1k

k

authors: ablease cf-london

repo: pivotal-cf/cf-rabbitmq-release


text:
Update the version information

The latest version of Healthwatch product is v2.1.3 as of September 2nd, 2021.

Good catch, @AkihiroKitada -- thank you!

authors: AkihiroKitada cshollingsworth

repo: pivotal-cf/docs-pcf-healthwatch


text:
Document Free Chunks API

Signed-off-by: Glenn Oppegard goppegard@pivotal.io

@AmberAlston Clear to merge this one?

@ljarzynski No. The related story got rejected. This shouldn't get merged until https://www.pivotaltracker.com/story/show/154571535 is accepted.

authors: AmberAlston hillrw3 ljarzynski


text:
Discriminator with Class table inheritance not working correctly for getLastPosition

Hello,
I have this problem where I am using the setup with discriminator in Doctrine and the Class table inheritance and the field is actually on the main Entity rather than each individual Entity, because I want to sort them all on 1 place.
But this causes the problem that getLastPossition creates query for the child entity class rather than the main. And so I see them all in 1 page, but the buttons are all wrong, because of that.
Meaning that imageOnSide records are only 2 and that's why the getLastPossition returns 2, but it should count all elements in parent entity.

I've made a fix for this issue in the PositionORMHandler.php file :
 public function getLastPosition($entity)
    {
	    $entityClass = ClassUtils::getClass($entity);
	    $parentEntityClass = true;
	    while ($parentEntityClass)
	    {
		    $parentEntityClass = ClassUtils::getParentClass($entity);
		    if ($parentEntityClass)
			    $entityClass = $parentEntityClass;
	    }

        $positionFields = $this->getPositionFieldByEntity($entityClass);
        $result = $this->dm
            ->createQueryBuilder($entityClass)
            ->hydrate(false)
            ->select($positionFields)
            ->sort($positionFields, 'desc')
            ->limit(1)
            ->getQuery()
            ->getSingleResult();

        if (is_array($result) && isset($result[$positionFields])) {
            return $result[$positionFields];
        }

        return 0;
    }

Hello @Tofandel,
I've got the opposite situation and this patch broke my code :(
the position field is in my child class.
How can I work around this?

Checking parent (and excluding abstract parent) is clearly not enough for a use case where position field is in child class. It would be better if it is possible to define whether parent(s) should be investigated for position field. For example like this:
# app/config/config.yml pix_sortable_behavior:     db_driver: mongodb # default value : orm     position_field:         default: sort #default value : position         entities:             AppBundle\Entity\Foobar: order # field is directly in this class             AppBundle\Entity\Baz: parent.order # field is in a direct parent class             AppBundle\Entity\Foo: AppBundle\Entity\FooGrandParent.order # field is in any ancestor class     sortable_groups:         entities:             AppBundle\Entity\Baz: [ group ]

It's been quite some time
But there is no need for this, a ReflectionClass/ReflectionProperty check to make sure the position field is there during the loop could do the trick..

That would be great. We had to downgrade to 1.3 on one of our project where we have parent entity which is not abstract. Both of the entities have position field so the Reflection solution would not help but we could at least adjust the parent class and rename the field.

authors: Tofandel filiplikavcan foucdeg twify93

repo: pix-digital/pixSortableBehaviorBundle


text:
🛑 Edge Platform is down

In 1d13fc6, Edge Platform (https://edge.pixify.co.uk/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Edge Platform is back up in bceefde.

repo: pixifydigitalagency/monitors


text:
Move 'copyfiles' from devDependencies to dependencies

Currently installing pixi-lights as a dependency to another project fails because a post-install script attempts to use copyfiles:
sh: copyfiles: command not found
npm ERR! file sh
npm ERR! code ELIFECYCLE
npm ERR! errno ENOENT
npm ERR! syscall spawn
npm ERR! pixi-lights@2.0.0 copy: `copyfiles -f "node_modules/{pixi.js,pixi-layers}/dist/*.{js,map}" demo/lib`
npm ERR! spawn ENOENT
npm ERR!
npm ERR! Failed at the pixi-lights@2.0.0 copy script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm WARN Local package.json exists, but node_modules missing, did you mean to install?

npm ERR! A complete log of this run can be found in:
npm ERR!     /Users/kable/.npm/_logs/2018-05-15T06_36_56_803Z-debug.log
npm WARN pixi-test@1.0.0 No description
npm WARN pixi-test@1.0.0 No repository field.

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! pixi-lights@2.0.0 postinstall: `npm run copy`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the pixi-lights@2.0.0 postinstall script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /Users/kable/.npm/_logs/2018-05-15T06_36_56_833Z-debug.log

This PR simply moves copyfiles to 'prod' dependencies

Note
Workaround in the meantime is to install copyfiles globally: npm i -g copyfiles before installing pixi-lights locally

Thanks for this, but it’s not quite the right solution. The postinstall script should probably be removed (which calls copyfiles). I’ll look at it and see if there’s a better place for that.

authors: KableM bigtimebuddy

repo: pixijs/pixi-lights


text:
Bug: Events don't seem to work at all.

Current Behavior
I can't seem to wire up an event to fire no matter what I do.
Expected Behavior
I should be able to get events for things like clicking Graphics, Containers, and Sprites.
Steps to Reproduce
I have a reproduction StackBlitz here
Basically, no matter what I do, I can't wire events up to the components. interactive prop doesn't seem to do anything. click is an available prop, but doesn't at all.
import viteLogo from '/vite.svg';

import { Stage, Sprite, Graphics } from '@pixi/react';
import { Graphics as PixiGraphics } from '@pixi/graphics';

import './App.css';

const draw = (g: PixiGraphics) => {
  g.clear();
  g.beginFill('red');
  g.drawRect(10, 10, 50, 50);
  g.endFill();
};

function App() {
  return (
    <>
      <Stage width={200} height={100}>
        <Graphics
          click={() => console.log('click prop')} // TypeScript likes this, but it does nothing
          onClick={() => { // TypeScript doesn't like this.
            console.log('onClick prop');
          }}
          draw={draw}
        />
        <Sprite
          image={viteLogo}
          x={70}
          y={10}
          interactive // TypeScript doesn't like this.
          onClick={() => { // TypeScript doesn't like this
            console.log('onClick logo');
          }}
          click={() => { // TypeScript likes this, but it does nothing
            console.log('click logo');
          }}
        />
      </Stage>
    </>
  );
}

export default App;

Environment

@pixi/react version: e.g. 7.0.0
pixi.js version: e.g. 7.1.0
React version: e.g. 18.0.0
ReactDOM version: e.g. 18.0.0
Browser & Version: e.g. Chrome 108
OS & Version: e.g. Ubuntu 22.04
Running Example: e.g. https://pixiplayground.com/

Possible Solution
It looks like the peer deps for @pixi/react are >=6 for everything. I suspect that it's pulling in newer versions of @pixi/* libraries that are simply incompatible with @pixi/react, but I'm not sure.
Additional Information
I'm really sad about this. lol. Without event handling, I have no real use for this otherwise awesome library. It also seems like there hasn't been any movement here in ~7 months, so hopefully it's just a lull in the action.

I got it working by adding the interactive property and adding a @pixi/events import (see https://github.com/pixijs/pixi-react/issues/464)

authors: benlesh zOadT

repo: pixijs/pixi-react


text:
PIXI.loader / progress

I have trouble understanding how the resource loading of PIXI works.  I don't understand how PIXI.loader.add("a.png").add("a.png").load(function() { console.log("loaded"); } could works for subsequent adding/loading, e.g. how should this even work when two functions adds some resources and call load at the same time? Or is PIXI.loader creating a new ResourceLoader each time you request it?
Anyway, what I want to do is to load some Flump libraries. In order to do that I've created a ResourceLoader and load a library.json file. After the json is loaded I re-use the loader to load the images defined in this file. The loading part of this works (it loads the json and the images), but now I want to visualize the progress in my preloader.
function loadLibrary(libraryPath:String, complete:Library->Void):ResourceLoader
{
	if (!libraryPath.endsWith("/")) libraryPath += "/";
	var jsonPath = libraryPath + "library.json";
	// load the library.json first
	var loader:ResourceLoader = new pixi.loaders.Loader();
	loader.add(jsonPath);
	loader = loader.load(function()
	{
		var json:Format = Reflect.field(PIXI.loader.resources, jsonPath).data;
		// do stuff with the json.
		for (textureGroup in json.textureGroups)
		{
			for (atlas in textureGroup.atlases)
			{
				loader.add(libraryPath + atlas.file); // add the images defined in the json
			}
		}
		// I don't get whats happening here exactly. I am loading again, but now with the images
		loader.load(function()
		{
			var libraryPath = libraryPath.substr(0, libraryPath.length - 1);
			complete(new Library(libraryPath));
		});
	});
	return loader;
}

This function returns a ResourceLoader, I would expect to loader.progress is good enough to display a preloader, but for some reason (maybe because I recycle the loader) it logs values over 100%. Am I doing it wrong or is this a bug?
    0
    100
    106.25
    118.75
    137.5
    143.75
    168.75
    181.25
    187.5

I'm using Haxe and https://github.com/pixijs/pixi-haxe but the question isn't Haxe specific.
Im also using pixi.js - v4.4.0

The .add() method enqueues a resource path to be loaded, but doesn't actually load anything. When you call .load() the loader kicks off loading the resources you queued up.
You cannot reuse a loader without calling .reset() first.
function loadLibrary(libraryPath:String, complete:Library->Void):ResourceLoader
{
	if (!libraryPath.endsWith("/")) libraryPath += "/";
	var jsonPath = libraryPath + "library.json";
	// load the library.json first
	var loader:ResourceLoader = new pixi.loaders.Loader();
	loader.add('atlasData', jsonPath);
	loader.load(function()
	{
		var json:Format = loader.resources.atlasData.data;

               // now that we finished loading, and stored off the resource we wanted, reset the loader
               // so it can be used again.
               loader.reset();

		// do stuff with the json.
		for (textureGroup in json.textureGroups)
		{
			for (atlas in textureGroup.atlases)
			{
				loader.add(libraryPath + atlas.file); // add the images defined in the json
			}
		}

		loader.load(function()
		{
			var libraryPath = libraryPath.substr(0, libraryPath.length - 1);
			complete(new Library(libraryPath));
		});
	});
	return loader;
}

This is still strangely constructed though, because what you will get after this in a progress listener outside this function is progress going from 0-100, then 0-100 again. You can do this all in a single load if you write a middleware that handles the atlas type you have, something like this:
function myAtlasHandler(libraryPath: String)
{
    return function (resource, next)
        {
        // skip if no data, its not json, it isn't spritesheet data, or the image resource already exists
        if (!resource.data
            || resource.type !== Resource.TYPE.JSON
            || !resource.data.textureGroups
        )
        {
            return next();
        }

        const loadOptions = {
            crossOrigin: resource.crossOrigin,
            loadType: Resource.LOAD_TYPE.IMAGE,
            parentResource: resource,
        };

        for (textureGroup in resource.data.textureGroups)
        {
            for (atlas in textureGroup.atlases)
            {
                this.add(libraryPath + atlas.file, loadOptions); // add the images defined in the json
            }
        }

        return next();
    }
}

function loadLibrary(libraryPath:String, complete:Library->Void):ResourceLoader
{
    if (!libraryPath.endsWith("/")) libraryPath += "/";
    var jsonPath = libraryPath + "library.json";

    var loader:ResourceLoader = new pixi.loaders.Loader();

    loader.use(myAtlasHandler(libraryPath));
    loader.add(jsonPath);
    loader.load(function()
    {
        var libraryPath = libraryPath.substr(0, libraryPath.length - 1);
        complete(new Library(libraryPath));
    });

    return loader;
}

With something like that, there will be only a single load and the loader will automatically (via the myAtlasHandler middleware) detect images and load them as well.

Thank you very much for this explanation!

authors: englercj markknol


text:
Installing package from github

Hello,
I'm trying to install pixi package from github using url like this:
npm install pixijs/pixi.js#master
It reports error:
Can't install github:pixijs/pixi.js#b7d187073f3b2d5940fadd65629536da1745c349: Missing package version
I also tried to specify path to bundles/pixi.js, but npm does not support this.
Then I tried to clone the repo manually and use it as file in package.json:
"pixi.js": "file:vendor/pixi.js/bundles/pixi.js"
But the package is not being "found" by cjs loader.
I found issue #3365, where it says that I should refer to release branch... but that is several years old.

This is not supported because we do not publish built code anymore on the repo. If you want the latest code, the easiest way is to pull https://pixijs.download/dev/pixi.min.js but that won’t work with npm.

Thanks Matt. So I will have to fork the repo, clone, build and push to the fork :)

authors: bigtimebuddy dev-zetta


text:
CanvasSpriteRenderer strange rounding functions

Reported here: https://www.html5gamedevs.com/topic/44803-canvasrenderer-problem/
Demo: https://www.pixiplayground.com/#/edit/dviwM8_okZ91hhxBDbjrg
Possible fix is there, just uncomment the first line. I cannot make PR yet, need to do a bit of history digging and check how to use roundPixels in our case.

The floors existed because when you had odd resolutions, like 0.75, rounding issues could cause trying to draw more than actually existed. All browsers handled this fine...... apart from IE, which error out
https://github.com/pixijs/pixi.js/pull/5836

authors: ivanpopelyshev themoonrat


text:
background-image: repeat...

I was wondering what could be the correct way to make huge background, if you want background is repeating some texture. For example I have pretty small photo about 40*40 pixels I want to repeat along to 4000px * 4000px. I haven't found any other way to do this than just create  100 * 100 (4000 / 40 = 100) sprites with each one of them with same texture.
Once I want to move camera on the background, I just change coordinates from all of those ten thousend sprites, but this could be slow.
Maybe I could create just as much sprites that screen width and height needs and relocating the sprites once my camera is moving, but still there would be large amount of sprites.
Is there any smarter and higher performance way to approach the issue than my suggestions?

Is this stil in todo list or is it implemented already? I am planing to do the same as author of this post - i neeed big backgrond that can go 64000x64000px and I need it to do from 64x64 pictures... And every picture can be different too... There is just no way i have to do it with sprites right?

@StealthyDamn Its TilingSprite.

authors: StealthyDamn ghost ivanpopelyshev


text:
[REQ] Automatic RCO flag removal

Acknowledgement of preliminary instructions

[X] I have read and acknowledged the preliminary instructions.

What idea do you have?
Unstable flags need going.
If FStringFlagUpdateVersion or FIntFlagUpdateVersion are present in flag lists, they are from RCO.

closed because of recent updates

repo: pizzaboxer/bloxstrap


text:
Métriques HVAC

Pour ce sous-requis, vous devrez créer un dashboard sur Grafana qui permet de visualiser les données
de l’application HVAC. Les plugins nécessaires sont présents sur l’instance Grafana. Il devrait y avoir un
graph pour les températures et un tableau pour les événements. L’Update vers la BD doit être en temps
réel et disponible 24h/24h (sauf pannes)

Probleme sur le serveur, donc impossible de continuer cette tache pour le livrable Lab 3

authors: KookyEddie pjbeltran


text:
Chose status or title bar?

First of all, thanks for the plugin. I really like to be able to navigate folder with the keyboard. Now, my question: is there any way to choose if QE shows at the title bar or the status bar? It appears in status for me, but I have seen some screen captures with it showing at the title bar.
Thanks again!

In Settings > Appearance > Advanced, set the "Window frame style" to "Obsidian frame".  Otherwise there's no title bar there for it to show in.

Nice! Thanks :)

authors: kikollan pjeby

repo: pjeby/quick-explorer

license: ISC


text:
Questions

궁금한 점이나 물어볼 내용등이 있다면 이 Issue에 작성

문제 질문 테스트 계정

test one more

안녀하십니까 주니후니 입니다.

이런식으로 해서 지역을 누르면 지역이 피커뷰 리스트에 들어가고 정류장을 누르면 지역에 해당하는 정류장을 또다른 피커뷰에 넣어주고있습니다.
지역에 지금 서울 부산 이렇게 두개있으면  서울 눌렀을때는 서울에 해당하는 데이터만 나와야 하는데.. 부산까지 전부다 나오고있어서요..
(당산 영등포 사당 역삼 강남 해운대 광안리 수영 )이런식으로 나오고있어서요.
포문을 이렇게 돌리고있는데 혹시 봐주실수있으실까요??ㅠ
             for d in items {
                print("p d       :    ",d)
                _ = d.srvcAreaCd
                let lr = d.lineRoute
                print("p 지역2",d.srvcAreaNm!)
                self.textAreaList.append(d.srvcAreaNm!)
                self.textArea.text = self.selectArea
                if self.selectArea == "" {
                    self.selectArea = d.srvcAreaNm!
                }
                for l in lr! {
                    self.textStationList.append(l.routeNm!)
                    self.sRouteId.append(l.routeId!)
                    self.textStation.text = self.selectStation
                    if self.selectStation == "" {
                        self.selectStation = l.routeNm!
                        self.selectRouteid = l.routeId!
                    }
                }
            }

@junhuee 음... items에 해당하는 model과 하위 모델까지 해당 코드에서 사용하는 것들만 좀 알려주실 수 있으신가요?
pickerview에 넣는 데이터는 self.textStationList랑 self.textAreaList인것 같은데, 이전에 초기화는 혹시 하시나요?
지역을 빈 값에서 서울 또는 부산으로 바꾸거나 선택되어있던 것에서 다른 것으로 선택을 할 때 각각 초기화 시켜주신 후 코드를 호출하셔야 해당하는 것만 나올 것 같은데..
for d in items {

}

이 부분만 봐서는 제가 생각하고 있는게 맞나 싶은데
일단 처음에 pickerview에서 각각 사용하는 데이터가 self.textStationList랑 self.textAreaList라면, for문 조건상으로 봤을 때 다음 플로우로 진행이 됩니다.
// 가정 :  items에는 먼저 지역(d)들이 있고, 각 지역 모델(d)의 하위에 LineModel(l) 모델을 Array로 갖고 있는 LineRoute(lr)이 있다고 가정합니다.
// 1. items는 자신이 갖고 있는 모든 d에 대해서 for문을 실행하게 됩니다.
// 2. 각 d는 또 하위에 있는 lr들을 for문을 통해서 실행합니다.
// 이 때 1.을 실행 할 때 각 d는 textAreaList에 append 됩니다.
// 그리고 마찬가지로 2.에서 각각의 d의 하위인 lr이 실행되는데, <- 여기서 문제가 발생합니다.
// 서울 하위의 lr들이 textStationList에 추가 되었는데 1.의 이유로 인해 부산 하위의 lr들 또한 textStationList에 추가 됩니다.
// 따라서 말씀하신 문제가 발생하는 걸로 보입니다.

items에 해당하는 model과 하위 모델까지 해당 코드에서 사용하는 것들만 좀 알려주실 수 있으신가요라고 하셨는데
모델과 하위모델이 어떤건지 이해를 잘못하겠습니다. ㅠㅠ

지금 퇴근하고 있는데, 일단 유사하게 동작하는 기능 구현한 후 말씀드릴게요.

@pjh6954 네 감사합니다 저도 계속적으로 시도해보고있어요 ㅋㅋ

@junhuee PickerView Exmaple 업데이트 했습니다.
링크
예제로 구현했는데 한 번 살펴보시고 이해 안되는 부분 말씀해주세요.

@pjh6954 매번 감사합니다. 예제 정말 잘 봤습니다!!!
저도 피커뷰에 들어가는 데이터를 정하고 뿌려주기를 했을때는  리스트에 잘뿌렸었는데..ㅠ
데이터를 받아서 뿌리려고하니. 참.. 이게 뭔가 어렵네요..
서버통신해서 리절트로

Json으로 result 받아오시나요?

네 json으로 받아오고있습니다. 위에 데이터가 print("result",result) 한 내용 입니다.

데이터 넣는 부분을 변경하여 수정하였습니다.
지역 을 확인했을때  스테이션의 정보를 지역에 맞게 호출하는 방식으로 변경하였습니다. ㅋㅋ
맞는곳에서 한건지는 잘모르겠찌만. ㅠㅠ

@junhuee 올리신 json기준으로 파싱해서 사용하는 예제로 업데이트 해드릴게요

@pjh6954  감사합니다 업데이트 해주시면 제꺼와 비교해서 다시 테스트 해보겠습니다!!!

@junhuee 제가 주말동안 일이 있어서 아직 구현이 다 안되었네요. 오늘이나 내일 내로 구현해서 올려드릴게요

@pjh6954 구현안해주셔도되용 저의방법으로 일단 구현해보고 해서 괜찮을것같아요
이제 딥링크 만들어보고있는데 혹시 딥링크도 해보셨나요??
파이어베이스 연결해서 앰에서 앱실행하는 그런거 해보고있는데 ㅋㅋㅋ

@junhuee 네. 어지간한 회사에서 파이어베이스로 크래시리틱스 연동하면 딥링크 관련된 것도 거의 들어가 있어요. 아무래도 마케팅쪽에서 링크로 홍보하고 바로 연결되는걸 원하니 구현 할 수 밖에 없죠.

앱이 실행되면서 저기로 유알엘이 들어오는데 저기서 파라미터로 id 값을 전달 받고있는 상태입니다.
파라미터 id 값을 받을수있는 방법이 따로 있는건가요?ㅠㅠ

@junhuee 잘 이해가 안가는데 url 들어온 값에서 query 부분에 id라는 값이 있어서 해당 값을 이용하신다는건가요?
ex) github.io/requestTest/?id=idvalue
위처럼 url이 온다고 가정 했을 때 id에 해당하는 "idvalue"를 전달받아 사용하고 싶으시다는거 맞나요?
일단 그 부분은 저도 서버쪽이나 파이어베이스관리 쪽을 건든지 좀 되어서 잘 기억은 안나는데
https://medium.com/firebase-developers/firebase-dynamic-links-guide-76552c0f292e
https://firebase.google.com/docs/dynamic-links/create-manually
이 부분 참고하시면 될 것 같습니다.(firebase custom deeplink query example 이라고 검색하시면 관련 자료들이나 블로그 잘 나올겁니다.)
swift에서 해당 url이용하는 방법은 다른 URL을 이용할 때 query 부분 갖고오는 것과 동일하게 이용하시면 됩니다.
보통 쿼리 시작부분은  ? 로 시작을 하고 value는  = 이후, 각 parameter key 분리는 & 로 나뉘어지니 그것들을 이용해서 split시키시면 되구요.

@pjh6954  감사합니다 어제 혼자 머리를 쥐어잡고 하면서 하긴했느데 ㅋㅋ 알려주신부분 한번 보겠습니다^^

@junhuee 다른 문의 사항 있으시면 언제든 여기에 물어보시면 됩니다.

@pjh6954  네 다른고 시도해보면서 궁금한거있음 물어보겠습니다!!
감사힙니다

authors: iOSDev-Dobby-test junhuee pjh6954


text:
Fixed: Irregular undefined calls

Just small changes from the file merge

I thought that this is pretty useful and it may be a must-have

I thought that this is pretty useful and it may be a must-have

Interesting; would it take a relative path as well ("a/b/c")?

Never tried it, but currently i have a project that contains multiple versions of itself in the same folder and I needed to include the complex library to be usable in all the versions separately ;)

Renamed this to common.addLibrary(sB, ...) and updated it with vararg in the last patch

API now adds directories including the Lua files directly
common.addLibrary("E:/Documents/Lua-Projs/ZeroBraineIDE/myprograms","a/b/c")
local t = require("test")
print(t.add(3,7)) -- Prints 10

repo: pkulchenko/ZeroBraneEduPack


text:
Add allPass and anyPass

The most elegant implementation of these would be something like:
S.allPass = (preds, a) => R.reduce((acc, f) => f(a) && acc, true, preds);

But then we'd lose short-circuiting.

But then we'd lose short-circuiting.

You could use R.reduced
let allPass = (preds, a) => R.reduce((acc, f) => f(a) || R.reduced(false), true, preds);
let count = 0;
[
  allPass([
    () => ++count && true,
    () => ++count && false,
    () => ++count && true
  ]),
  allPass([
    () => ++count && true,
    () => ++count && true,
    () => ++count && true
  ]),
  count
]

http://goo.gl/gl1pio

Nice!

Ah, that desn't seem to play well with Sanctuary's type checking. The original works fine but when wrapped in the def function:
TypeError: ‘test’ requires a value of type String as its second argument; received {"name": "a"}

We should test the handling of an empty predicates list.

What should the expected output be? An error? Or given that no predicates were passed in then none passed so it should always be false.

S.allPass([], x) should evaluate true for every x: all (zero) of the predicates are guaranteed to be satisfied.
S.anyPass([], x) should evaluate false for every x: it's impossible for at least one predicate to be satisfied.

Cool. Makes sense.

I'm sorry for getting behind on reviewing your pull requests, Stefano. Could you rebase and resolve the merge conflicts?

I missed this comment altogether, just rebased now.

I missed this comment altogether, just rebased now.

:deciduous_tree:

authors: davidchambers kedashoe svozza

repo: plaid/sanctuary


text:
passwords#new and passwords#edit is not protected against format

# GET /resource/password/new
def new
  self.resource = resource_class.new
end

# GET /resource/password/edit?reset_password_token=abcdef
def edit
  self.resource = resource_class.new
  resource.reset_password_token = params[:reset_password_token]
end

When requested with unintended format, these endpoints should raise UnknownFormat instead of Missing Template.

Are you requesting it with json format for example?

Hey, I will look into this issue and see if I can come up with a solution.

Handled by plataformatec/responders#153 - we might enable this by default on future Devise versions but developers can try it out when overriding Devise controllers.

authors: SimonDKnight kurenn lucasmazza pwen

repo: plataformatec/devise


text:
Add Typescript definitions

Add into package.json
{
    . . .
    "scripts": {
        "build:dts": tsc index.js --outDir types --allowJs --declaration --emitDeclarationOnly
    }
    . . .
}

or something like this.
So, @playcanvas/editor-api could be used to create a Chrome extension with Typescript. Right now, it's hard to use the repo.

Hi @querielo - thanks for the suggestion. I have made a quick attempt to add support for generating the TS declarations. Would you be interested to test the generated .d.ts file and maybe let me know if it works OK? If it does, I'll do a new release of the NPM package that includes the declarations.

The only problem I see is:
C:\dev\editor-api>npx tsc --pretty false dist\index.d.ts
dist/index.d.ts(547,27): error TS2503: Cannot find namespace 'pc'.

If I edit pc.Entity to any, the types seem to compile ok. For now, I'm not sure what the 'correct' fix is. But yeah, relatively minor problem.

@willeastcott  ☝

So currently, classes like ComponentSchema are exported from the root index.js:
https://github.com/playcanvas/editor-api/blob/e169c179ed4bd37e80a286d60bac1598823ac9b7/index.js#L14
But that class (and several others) have the @category internal tag in the docs. So presumably, index.js can drop all the internal classes. This will prevent rollup-plugin-dts from including them in the built index.d.ts.
As for the pc.Entity problem, actually, it probably does work - I think I just need to compile the types with the PlayCanvas Engine types and presumably tsc will find pc.Entity.

Checked /** @typedef {import("playcanvas")} pc */ and it also works with pc.Entity.

@willeastcott
OH. I see. I didn't know that the api object in global space of Playcanvas Editor is the editor namespace in  editor-api
So, editor-api should be imported like this:
import { globals as editor } from "@playcanvas/editor-api"
import * as api from "@playcanvas/editor-api"

Hmmm, can we do that in a cleaner, less verbose way I wonder...

@willeastcott
We can try to replace this line by
footer: `export as namespace api;
declare global {
    const editor: typeof globals;
}`,

Then the library can be imported like this import "@playcanvas/editor-api";, and after that there are api and editor in global space (api is a namespace, editor is an object). It works for me. But, I am not sure that semantically it is correct.

So this looks better to me. But if I add that to the index.d.ts, can't you still do:
import api from "@playcanvas/editor-api"

Just committed this: https://github.com/playcanvas/editor-api/commit/0687f8e138da32f367d0a97e81d8b656c0569a3b

authors: querielo willeastcott

repo: playcanvas/editor-api


text:
EntityReference - is convoluted, complex and inefficient.

Various Components use EntityReference to somewhat simplify events management, but its convoluted implementation, has a lot of inefficient and unnecessary mechanics, like subscribing to a large EventHandlers, and removing such subscriptions often.
So when such Components are added or removed, there is a significant amount of time spent in EntityReference.
Having 100 lines of comments describing what it does, and then having a complex implementation that is still hard to understand - makes it even worse.
The events management is inefficient as there was no EventHandle at the time, so events removals from ComponentSystems - is slow. By utilizing EventHandle.off some speedup can be achieved. It also subscribes to a large EventHandlers like ComponentSystem only to check in callback if it relates to our component - this of course leads to a very large callback lists - which just slows everything down on fires.
A simple example is RenderComponent.rootBone - which is just a reference to another entity. And can be a simple property, and does not need to be a complex system behind it. This PR showcases this. And leads to ~87% speedup improvement for component removal.
Other Components need to do a similar job by simplifying properties, and implementing their logic directly inside of a component, instead of using convoluted EntityReference.
This will lead to massive speedups and move logic implementation to its relevant place - the Component itself.

Here is described problem in simplified code example:
class ComponentSystem extends EventHandler {

}

class Component {
    constructor() {
        // in every EntityReference
        this.system.on('add', this.onAdd, this);
        this.system.on('beforeremove', this.onRemove, this);
        //
    }

    onAdd(entity) {
        if (this.entity === entity) {
            // do something
        }
    }

    // { in an EntityReference:
    onRemove(entity) {
        if (this.entity === entity) {
            this.system.off('add', this.onAdd, this);
            this.system.off('beforeremove', this.onRemove, this);
        }
    }
    //
}

Each EntityReference would subscribe to whole system, and then check if this.entity === entity. So when we create a new component, we subscribe to whole system again, and when we remove component, we unsubscribe, so if there is a lot of these component, or even components subscribing to the same system (like Element, Button, Scrollbar, ScrollView, and possibly others), this leads to an enormous list of callbacks which do if (this.entity === entity) { to check if it relates to them.
So the cost of creating and removeing of a new component grows exponentially with size of a scene. And the size does not need to be crazy to start getting lag spikes on every hierarchy modification (adding, removing, reparenting).

repo: playcanvas/engine


text:
Shadow intensity

Supports spot, point/omni and directional lights.
Updated the lights.tsx example to add controls for the shadow intensities.
Shadow is calculated as a separate variable, and then 1 -> shadow is lerped with intensity value.
API changes:
Lights have a new member shadowIntensity.
Fixes #3604

Realized I had this branched on an old version of main. Going to do a rebase and work this code in again.

This is awesome!
Please create an issue on editor repo so we don't forget to expose this functionality in the editor when engine is released.

https://github.com/playcanvas/editor/issues/797
This issue relates to adding support for this in the editor.

@GSterbrant Ah, you probably need to add shadowIntensity here:
https://github.com/playcanvas/engine/blob/de890ce7f34405544f6e3e5f3050c42fb1d5f3f1/types-fixup.mjs#L180

authors: GSterbrant slimbuck willeastcott


text:
Update Properties.lagda

Please consider distinguishing the gas variable from the "gas" name in Evaluation section.

I reviewed this section, and changed one use of gas to gas. The rest looks in order to me.

authors: Kwezan wadler


text:
grpc-gateway: add update test case

Add test case to verify resource data consistency after device name change.
This is a test case for iotivility-lite issue #40 oic/d piid issue with notification (https://github.com/iotivity/iotivity-lite/issues/40).

Could you move tests to the grpc-gateway/service. Because only this package is used in iotivity-lite test container.

Could you move tests to the grpc-gateway/service. Because only this package is used in iotivity-lite test container.

Service test added.

repo: plgd-dev/hub


text:
Remove duplicate code

Remove duplicate code in misc file

@sneridagh This PR removes a couple of duplicated lines of code and can be merged right away


text:
319 - Cleaning trailing semicolons when building CTEs

Describe your changes

Added replace filter in store.py -> SQLStore class -> store method AND store.py -> SQLQuery -> __str method, which cleans both lines and cells respectively before they are rendered with template.
Added unit tests in test_store.py.
Added unit tests in test_magic.py which checks both line and cell magic with semicolon SQL cases.
Fixed breaking test in test_compose.py which doesn't account for the semicolon removal changes when running %sqlrender command.

Added a new test to specficially check semicolon SQL case when used as a CTE.

Updated CHANGELOG.md file with my fix and also reorganized the existing order to be in line with the instructions provided on the docs.
Ran pkgmt format.

Issue number
Closes #319
Checklist before requesting a review

[x] Performed a self-review of my code
[x] Formatted my code with pkgmt format
[x] Added tests (when necessary).
[x] Added docstring documentation and update the changelog (when needed)

@alextongme want to commit the final comments in?

apologies, i started work and forgot about this. this cannot be merged in until the convo above with @edublancas is resolved though

repo: ploomber/jupysql


text:
Expand filters a lot

See the test-filter.js changes for a somewhat decent representation of the phrases added, though the tests do not test every possible phrase.
I ran these filters as a dry run last night and edited out the tweets that were retweeted by VegAssist:
Connected to Twitter (dry run, will not retweet matches)
657837580747198464 : JordanBuchan96 : i really want to go vegan
657866030312587264 : hotlinenigger : RT @KevsDgaf: My mama said I should try going vegan wit her but I said sorry mama I like cheeseburgers and eating pussy
657881047779188736 : edarnleysmith : i really want to go vegan one day �
657893967040786432 : emeraIdpines : RT @edarnleysmith: i really want to go vegan one day �
657908270275895296 : Directionheart : SO GOOD STELLA 10/10 especially since I wanna try being vegan next week � you inspire me @stellaraee ��
https://t.co/8ZHvTKsfdR
657908552439230465 : stellaraee : RT @Directionheart: SO GOOD STELLA 10/10 especially since I wanna try being vegan next week � you inspire me @stellaraee ��
https://t.co/8Z…
657946227007496192 : KHAILA_G : I wanna be a Vegan someday, but I'm just not ready yet.
657949519724851205 : joyful_thought : "I want to be a vegan vampire for Halloween," my 5 yr old states emphatically.

Taking any and all costume suggestions and dietary tips.
657953982938157057 : MewIsareeya : I should be a vegan...
657955782789492736 : taylormcarltonn : RT @_tayuh_: I wanna go vegan so bad
657972726330728448 : jordanawale : I wanna become a vegan still so people can talk crud to me, then I'll lay em out on their back and be like "ha a vegan rushed you"
657977444637523968 : skullsforleen : I WANNA BE A VEGAN BC I HATE SEEING ANIMALS DIE BUT THEN I GREW UP EATING MEAT AND HATING VEGETABLES DO YOU SEE MY PROBLEM
657983472141795328 : TRASHYM0M : yo i wanna be a vegan so damn bad
657989530750947328 : fxdgeregui : i wanna go vegan once i move out from my hometown and don't live with my parents anymore
657994772141903872 : 231Tally : I need @gabrielleaplin to start a vegan blog to help me go vegan when I move house… https://t.co/onZI8OZuUC
658005026313523200 : PaigeC0X : RT @toripeetoom: I want to go vegan so bad
658006062965084161 : iSlapJezebels : I'm thinking about going vegan but I like chicken
658023070414729217 : TheThugPanda : I am thinking of going vegan...
658030599630811136 : teenagecobain : I'm thinking of becoming vegan tbh

@squeek502 I look forward to the day we see it tweet "I unquestionably want to go vegan" :stuck_out_tongue:
Great work! I'll deploy & restart with the updates.

I unquestionably believe it will happen.

repo: plorry/VegAssist


text:
Remove Authentication component.

Removed the Authentication component, all it did was reading the configs.
Moved the readConfig dispatch to AppContainer.

I looked back at the Authentication component and found a _dash-login request. There's a _dash-login route in dash-auth but it's handled by it's oauth.js bundle.
@chriddyp Can you confirm that removing this api will not break anything ?

repo: plotly/dash-renderer


text:
Removed deprecated dependencies that are now integrated

dash_bootstrap_components, dash_html_components, and dash.dependencies are now integrated into core dash, so modified Import statements to reflect this change in the demo app.

About

Description of changes

Pre-Merge checklist

[ ] The project was correctly built with npm run build.
[ ] If there was any conflict, it was solved correctly.
[ ] All changes were documented in CHANGELOG.md.
[ ] All tests on CircleCI have passed.
[ ] All Percy visual changes have been approved.
[ ] Two people have :dancer:'d the pull request. You can be one of these people if you are a Dash core contributor.

Reference Issues

Closes #[issue number]
Other comments

For demos/examples, at this point I think assuming Dash 2.x is fine. If these imports were inside the package though we'd definitely still want 1.x fallbacks.

authors: Lew-Goldstein alexcjohnson

repo: plotly/dash-vtk