text:
Release-testing: v8.3.0 tiflow repo CI trigger

What problem does this PR solve?

Issue Number: close #xxx
What is changed and how it works?
Check List
Tests

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

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
Please refer to [Release Notes Language Style Guide](https://pingcap.github.io/tidb-dev-guide/contribute-to-tidb/release-notes-style-guide.html) to write a quality release note.

If you don't think this PR needs a release note then fill it with `None`.

/test cdc-integration-kafka-test cdc-integration-mysql-test cdc-integration-pulsar-test cdc-integration-storage-test dm-compatibility-test dm-integration-test engine-integration-test
This is an automated do-test comment triggered by a release-testing robot.
Every evening at 11 PM, if there are updates to the PR in the repo, then perform the `/test CI FT` test.
Commented on: 2024-08-13 17:27:02.

/test cdc-integration-kafka-test cdc-integration-mysql-test cdc-integration-pulsar-test cdc-integration-storage-test dm-compatibility-test dm-integration-test engine-integration-test
This is an automated do-test comment triggered by a release-testing robot.
Every evening at 11 PM, if there are updates to the PR in the repo, then perform the `/test CI FT` test.
Commented on: 2024-08-13 17:28:41.

/test cdc-integration-kafka-test cdc-integration-mysql-test cdc-integration-pulsar-test cdc-integration-storage-test dm-compatibility-test dm-integration-test engine-integration-test
This is an automated do-test comment triggered by a release-testing robot.
Every evening at 11 PM, if there are updates to the PR in the repo, then perform the `/test CI FT` test.
Commented on: 2024-08-14 20:09:19.

/test cdc-integration-kafka-test cdc-integration-mysql-test cdc-integration-pulsar-test cdc-integration-storage-test dm-compatibility-test dm-integration-test engine-integration-test
This is an automated do-test comment triggered by a release-testing robot.
Every evening at 11 PM, if there are updates to the PR in the repo, then perform the `/test CI FT` test.
Commented on: 2024-08-15 14:04:18.

/test cdc-integration-kafka-test cdc-integration-mysql-test cdc-integration-pulsar-test cdc-integration-storage-test dm-compatibility-test dm-integration-test engine-integration-test
This is an automated do-test comment triggered by a release-testing robot.
Every evening at 11 PM, if there are updates to the PR in the repo, then perform the `/test CI FT` test.
Commented on: 2024-08-16 22:16:04.

/test cdc-integration-kafka-test cdc-integration-mysql-test cdc-integration-pulsar-test cdc-integration-storage-test dm-compatibility-test dm-integration-test engine-integration-test
This is an automated do-test comment triggered by a release-testing robot.
Every evening at 11 PM, if there are updates to the PR in the repo, then perform the `/test CI FT` test.
Commented on: 2024-08-20 16:30:21.

/test cdc-integration-kafka-test cdc-integration-mysql-test cdc-integration-pulsar-test cdc-integration-storage-test dm-compatibility-test dm-integration-test engine-integration-test
This is an automated do-test comment triggered by a release-testing robot.
Every evening at 11 PM, if there are updates to the PR in the repo, then perform the `/test CI FT` test.
Commented on: 2024-08-20 20:33:50.


text:
capture(cdc): refactor capture run to prevent call close the capture multiple times.

What problem does this PR solve?

Issue Number: close #6339
What is changed and how it works?
Check List
Tests

No code

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

/run-all-tests

/run-integration-test
/run-kafka-integration-test

/run-verify-ci

Codecov Report

Merging #6411 (f331c7d) into master (f14088c) will increase coverage by 0.1336%.
The diff coverage is 70.6250%.

Flag
Coverage Δ

cdc
66.0357% <70.6250%> (+0.1423%)
:arrow_up:

engine
63.5816% <ø> (+0.1185%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
@@               Coverage Diff                @@
##             master      #6411        +/-   ##
================================================
+ Coverage   59.2403%   59.3740%   +0.1336%
================================================
  Files           754        757         +3
  Lines         86755      87033       +278
================================================
+ Hits          51394      51675       +281
+ Misses        30799      30793         -6
- Partials       4562       4565         +3

/run-all-tests

/run-all-tests

/merge

/run-dm-integration-test
/run-kafka-integration-test

/run-dm-integration-test

/run-all-tests

/run-dm-integration-test

/run-dm-integration-test

source: gharchive/pull-request

repo: pingcap/tiflow

license_type: permissive

license_source: github-api


text:
tests:test_server_compact_lock_cf failed

Hi,
test_server_compact_lock_cf  randomly failed on my machine with the following details:
thread 'raftstore::test_compact_lock_cf::test_server_compact_lock_cf' panicked at 'assertion failed: compact_write_bytes > 0', tests/raftstore/test_compact_lock_cf.rs:33
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
stack backtrace:
   0: std::sys::imp::backtrace::tracing::imp::unwind_backtrace
             at /checkout/src/libstd/sys/unix/backtrace/tracing/gcc_s.rs:49
   1: std::sys_common::backtrace::_print
             at /checkout/src/libstd/sys_common/backtrace.rs:71
   2: std::panicking::default_hook::{{closure}}
             at /checkout/src/libstd/sys_common/backtrace.rs:60
             at /checkout/src/libstd/panicking.rs:355
   3: std::panicking::default_hook
             at /checkout/src/libstd/panicking.rs:371
   4: tikv::util::panic_hook::track_hook::{{closure}}
             at src/util/panic_hook.rs:61
   5: <std::thread::local::LocalKey<T>>::with
             at /checkout/src/libstd/thread/local.rs:265
   6: tikv::util::panic_hook::track_hook
             at src/util/panic_hook.rs:55
   7: core::ops::Fn::call
             at /checkout/src/libcore/ops.rs:2588
   8: std::panicking::rust_panic_with_hook
             at /checkout/src/libstd/panicking.rs:550
   9: std::panicking::begin_panic
             at /checkout/src/libstd/panicking.rs:511
  10: tests::raftstore::test_compact_lock_cf::flush_then_check
             at tests/raftstore/test_compact_lock_cf.rs:33
  11: tests::raftstore::test_compact_lock_cf::test_compact_lock_cf
             at tests/raftstore/test_compact_lock_cf.rs:60
  12: tests::raftstore::test_compact_lock_cf::test_server_compact_lock_cf
             at tests/raftstore/test_compact_lock_cf.rs:67
  13: <F as test::FnBox<T>>::call_box
             at /checkout/src/libtest/lib.rs:1451
             at /checkout/src/libcore/ops.rs:2683
             at /checkout/src/libtest/lib.rs:140
  14: __rust_maybe_catch_panic
             at /checkout/src/libpanic_unwind/lib.rs:98
test raftstore::test_compact_lock_cf::test_server_compact_lock_cf ... FAILED

It seems caused by the operation sleep https://github.com/pingcap/tikv/blob/master/tests/raftstore/test_compact_lock_cf.rs#L28.
Is it possible to  use channel instead of the sleep?

Do we fix it now? @zhangjinpeng1987

Yes

But it still fails.

source: gharchive/issue

authors: AndreMouche BusyJay siddontang zhangjinpeng1987

repo: pingcap/tikv


text:
[DNM] pd: bi-streaming heartbeat

This implementation assumes that pd server closes connection on error.

Depend on pingcap/kvproto#152

Now, pingcap/kvproto#152 is up to date.

PTAL

Do we need to handle metrics PD_SEND_MSG_HISTOGRAM in region heartbeat? @siddontang @overvenus

@BusyJay region heartbeat is a bi-stream now, I don't know how to get its duration. Maybe we should only use counter for it.
Btw, please fix conflict and we can go on.
PTAL @overvenus @disksing

How about attach a timer in  Request? Although, additional time will be recorded too, like building futures, waiting for being scheduled.

any update?

There are some changes after merging master, PTAL

@siddontang @overvenus PTAL

LGTM

LGTM

authors: BusyJay disksing overvenus siddontang


text:
Ignore statistics tests

Ignore those tests to fix CI.

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

repo: pingcap/tispark


text:
Support auto increment

In this PR, changes are made to support auto increment column case.

/run-all-tests

/run-all-tests tikv=v2.1.9 tidb=v2.1.9 pd=v2.1.9

/run-all-tests

/run-all-tests tikv=v2.1.9 tidb=v2.1.9 pd=v2.1.9

/run-all-tests

/run-all-tests tikv=v2.1.9 tidb=v2.1.9 pd=v2.1.9

merging since test was already passed.


text:
🌋 E|F ➾ Export all type-aliases.ts functions so we can use it to initialise MMaps

Product Board Link
No response
🚥 Status

[ ] 🔮 Product/Feature Design
[ ] ✍🏻 Architecture/Implementation Planning
[ ] 🏗 Under Construction
[ ] ⛳ Feature Complete
[ ] ⛵ GTM / Release

✅ To Do
From my fronend app, I want to deploy a contract template from a json definition and initialize a storage
I miss the asMap function to create a default MichelsonMap
import { address, asMap, int, timestamp } from "../type-aliases";

  const [contract, setContract] =
    useState<PermissionedSimplePollVotingContract>({
      name: "Enter question here ...",
      from_: new Date().toISOString() as timestamp,
      to: new Date().toISOString() as timestamp,
      options: [],
      registeredVoters: [],
      results: asMap<string, int>([]),
      votes: asMap<address, string>([]),
      owner: userAddress as address,
    });

Please export all utility function from generated types plugin
thank you !!!
📐 Feature Description
see above
super easy to add export keyword on each function
🚩 Problem or Use Case
No response
⚖ Acceptance Criteria
✨ Features:

[ ] ...

📚 Docs:

[ ] ...

🔭 Scope
🏎 In Scope:

...

🙅♂ Out of Scope:

...

🔗  Related Issues and Subtasks
No response
🦄 Sizing
1
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Great suggestion!

authors: mweichert zamrokk

repo: pinnacle-labs/taqueria


text:
🛑 ggmemoryadmin is down

In 5154ec6, ggmemoryadmin (http://mms.hangeul.go.kr:9002/login) was down:

HTTP code: 0
Response time: 0 ms

Resolved: ggmemoryadmin is back up in de2cd52.

repo: pinnode/pinnode

license: MIT


text:
Add option for custom logging format

Using the serializers I can customize what get's logged. However, I do not have control over the logging format. For example, I would like to log like so:
[1574212772844] INFO (35 on db805887c09f): 11.11.111.111 - GET /url/path - Status Code: 200

Essentially it would mean removing the object with res & err here in logger.js:
      log[level]({
        res: this,
        err: err || this.err || new Error('failed with status code ' + this.statusCode),
        responseTime: responseTime
      }, 'request errored')
      return
    }

    log[level]({
      res: this,
      responseTime: responseTime
    }, 'request completed')

...allowing the user to decide how to format the log.

I'm -1 in removing the lines from pino-http for formatting reasons. I'm happy to review a PR that make things more flexible here.

Okay I did some digging and I think what I am looking for is something like this. I think it would be possible by using the stream like you did to manipulate the logging format. Is that true? If so I guess it wouldn't apply to the pino-http module.

I'd like this option as well.
The suggested 11.11.111.111 - GET /url/path - Status Code: 200 format would be preferred in any configuration instead of the default request completed message, however i'd also add an option to remove details of req and res unless I opt into getting them for specific endpoints.
This code looks easy enough to fork though, so I won't wait around too long for it.

I couldn't get pino-http-print to work at all.
Attempt 1:
import express from "express";
import pino from "pino";
import expressPino from "express-pino-logger";
import pinoHttpPrint from "pino-http-print";

const log = pino({
  prettyPrint: {
    translateTime: "SYS:yyyy-mm-dd HH:MM:ss.l",
  },
});

const app = express();

app.use(expressPino(pinoHttpPrint()));

Attempt 2:
app.use(expressPino(pinoHttpPrint({ all: true })));

Attempt 3:
app.use(
  expressPino({
    logger: log,
    stream: pinoHttpPrint({ all: true }),
  }),
);

Anyway, this is probably the wrong place for this comment but instead of starting new issues that I don't really care about I figured I'd just leave it here, fork pino-http and do what I want just for my own projects.
This seems like a really good logger but it's just too bad that I have to compose 3 different packages together just to get some useful Express logging without all of the req/res diarrhea.

I couldn't get pino-http-print to work at all.

An update was released for pino-http-print right after this that added some options, I suggest you check it out again.
The difference is that the pino-http-print module returns a factory instead of a print stream directly. So you have to call it again to get a stream.
My current setup:
const express =  require("express");
const pino = require('pino');
const pinoPrint = require('pino-http-print');
const expressPino = require('express-pino-logger')

const prettyPrintFactory = pinoPrint({
	all: true,
	relativeUrl: true,
	translateTime: 'SYS:standard',
});

const prettyPrint = prettyPrintFactory();

const log = pino(prettyPrint);

const app = express();

app.use(expressPino(prettyPrint));
// OR reuse existing pino instance instead
// app.use(expressPino({logger: log}));

app.get('/test', (req, res) => {
    res.send('OK');
})

app.listen(3000, () => {
    log.info('Listening now...');
});

Output:
[2020-02-16 18:52:07.083 +0530] INFO  (22235 on lg-gram-elementary): Listening now...
[2020-02-16 18:52:11.892 +0530] GET /test 200
[2020-02-16 18:52:12.007 +0530] GET /favicon.ico 404

@waynebloss to be clear, there is a much simpler pattern for this http://getpino.io/#/docs/pretty?id=pretty-printing
you pipe the JSON output to a prettifer, that's it.

you pipe the JSON output to a prettifer, that's it.

pino-http-print also works as a command. Equivalent example would be:
node index.js | pino-http-print -a -r -t 'SYS:standard'
with the below index.js file.
// index.js
const express =  require("express");
const expressPino = require('express-pino-logger')

const app = express();
app.use(expressPino());

Though setting up pino with a prettifier conditionally in dev environment is also pretty convenient.
@davidmarkclements Would a pino-http-print example in the pretty printing section be a good idea for the docs? I know a few people who migrated from morgan to pino/express-pino but had a hard time setting up pretty printing for dev environments for request logs.

absolutely, PR's to pino docs is always welcome
On Sun, Feb 16, 2020 at 6:52 PM Rohit Gohri notifications@github.com
wrote:

you pipe the JSON output to a prettifer, that's it.
pino-http-print also works as a command
https://github.com/pinojs/pino-http-print#cli. Equivalent example would
be:
node index.js | pino-http-print -a -r -t 'SYS:standard'
with the below index.js file.
// index.jsconst express =  require("express");const expressPino = require('express-pino-logger')
const app = express();app.use(expressPino());
Though setting up pino with a prettifier conditionally in dev environment
is also pretty convenient.
@davidmarkclements https://github.com/davidmarkclements Would a
pino-http-print https://github.com/pinojs/pino-http-print example in
the pretty printing section be a good idea for the docs? I know a few
people who migrated from morgan to pino/express-pino but had a hard time
setting up pretty printing for dev environments for request logs.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pinojs/pino-http/issues/73?email_source=notifications&email_token=AAJCWPDSBTAR6GQQAQPRQ7TRDF4PJA5CNFSM4JP3O3CKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEL4NRTI#issuecomment-586733773,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJCWPBDJUVOPVPMWBE25JTRDF4PJANCNFSM4JP3O3CA
.

Might customSuccessMessage be a solution for you?
Similar to:
const requestLogger = require("express-pino-logger")({   logger,   customSuccessMessage: function (res) {     return ${res.req.ip} - ${res.req.originalMethod} ${res.req.originalUrl} - ${res.statusCode};   }, });

I'd like to have that too... The pino-http-print does not work for me...
I'll adopt the @clemenspeters solution but it will be a better if I can have req and res already serialized as parameters there (there is no way to access responseTime in the customSuccessMessage function...).
This solution is actually a workaround and if I want to have only that message I have to put req and res to null in the serializers...

Any updates on this? A meaningful message including the requested URL and reponse time should be a must

authors: clemenspeters davidmarkclements giovanni-bertoncelli mcollina rohit-gohri rstoermer taylorjdawson waynebloss

repo: pinojs/pino-http

license: mit

license_source: bigquery


text:
Open Web UI on Flux WebUI doesn't work

I installed Flux WebUI and it said successfully installed, however, nothing appears even though I press Open Web UI.
ENOENT: no such file or directory, stat 'C:\pinokio\api\flux-webui.git{{input.event[0]}}'

i have the same problem.
app do no opens.
i removed and reinstalled too

Getting same error on MacOS

same with Mac OS, intel version

i have the same problem. app do no opens. i removed and reinstalled too

is it working?

i have the same problem. app do no opens. i removed and reinstalled too

is it working?

Nope

Downloaded today, still same problem

Last update fixed in my case

same issue.

Encountered the same issue with multiple Pinokio applications. Resolved by editing the default directory in the "Save as" dialog to remove the ".git" suffix at the end. For example, with this issue, instead of saving as "flux-webui.git", save as "flux-webui".
(Note: also similar to issue here: https://github.com/pinokiocomputer/pinokio/issues/219)

Encountered the same issue with multiple Pinokio applications. Resolved by editing the default directory in the "Save as" dialog to remove the ".git" suffix at the end. For example, with this issue, instead of saving as "flux-webui.git", save as "flux-webui".
Default for Pinocchio?  Looking for where to change this.

Please give a complete screenshot because I can't find the save as

You have to delete application then reinstall it. When launching the install, you'll have the save as dialog window.

Had the same issue, for me it fixed it after I run manualy the "pip install E:\Pinokio\api\fluxgym.git\requirements.txt" after that it was starting.

authors: Astroburner LunaKrissy Martylei Mattieug dulanyw iboth360 moebiussurfing najamelan saikido screan spiritform

repo: pinokiocomputer/pinokio


text:
Disable no wildcard rule

Bandaid for https://github.com/pinterest/ktlint/issues/48

Please make a release with this change as it is wanted by a lot of developers :) @shashachu

@mortrnholmgaard I will try my best to put out a release this weekend, but it may have to wait until Tuesday (Monday is a holiday in the US).

authors: mortenholmgaard shashachu

repo: pinterest/ktlint


text:
PORT

how exactly each service communicate each other?
and is it okay if i changed pprof or expose port?

SFU, ISLB, AVP and BIZ mostly interact with each other using NATS as a messaging queue and etcd for service registration. ISLB also uses redis internally as a routing cache (I think).
PProf is a profiling helper for development and not necessary in production; you can disable it in the config.toml for each service, and if you do leave it enabled you should ensure the port isn't accessible. Also I think docker-compose is the most common/advised way to run the services, so if you don't expose the pprof ports on the docker containers, nobody can access it.
The only service that needs an outside port for ion is biz:8443 which is a websocket signal interface for sending join() and publish() and subscribe() messages, basically. If you are running ion-app-web also via docker (recommended) as a web UI, you should keep biz:8443 private and expose ion-app-web (it's a Caddy reverse proxy) over port 443 for HTTPS -- Caddy will serve the necessary HTML+JS, and will proxy all requests to /ws to biz:8443 over the internal docker network.

Thanks a lot @leewardbound
But how about listen port for biz?
can i change the port to 80 instead port 8443?
and how about SFU server? i read in WIKI it must expose port 5000-5200/udp,
please your help, thanks a lot

Yes, you can change biz to 80 if you want, if you aren't using ion-app-web. Biz does not really serve a UI though so I don't know why you'd want it to be on 80; up to you.
Yes, sorry I forgot to mention that (large) detail - SFU also has to open UDP ports, 1-or-more per webrtc connection. You can specify the port range in the config; really you should try to make as many ports as possible available, 200 isn't very many for a large conference server. I'm experimenting myself with opening 50,000+ UDP ports but I don't think there's any official documentation around this.

allright thanks a lot @leewardbound
can i create 3 instance of each services ( biz, islb, sfu, avp ) ?

Hi @leewardbound
Is SFU needs an outside ports or just open udp ports internally?

UDP ports need to be open externally to establish webrtc transport

@masatrio-business-account The plan is to support multiple of each service, but currently behavior for multiples of each instances isn't well defined; it is still very WIP and unsupported.

thanks guys @tarrencev @leewardbound
whats avp?
why avp need - "./:/out/" volume in docker compose?

avp is for audio/video processing. the mounted directory is for outputting recordings

is there any extra configuration if i deploy ion-web-app and ion-backend in separate domain?
please help, thanks

can you guys guide me to deploy this service separately?
i need this to deploy it for education startup in indonesia, thanks a lot

Sorry, this use case is not currently supported. It might be possible, but
we aren't completely sure, and we can't really help you through this.
Also, we should close this ticket. This ticking system is primarily for bug
reports and tracking issues, rather than support tickets -- since it seems
like you need a lot of assistance, you might find someone kind enough to
help in #pion on the Golang Gophers slack chat, there's a link on the
project page to join -- but our project is run by a community of
volunteers, so if you wind up getting that working, you should write a
guide for our docs!
On Sun, Jun 14, 2020 at 10:56 PM masatrio-business-account <
notifications@github.com> wrote:

can you guys guide me to deploy this service separately?
i need this to deploy it for education startup in indonesia, thanks a lot
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pion/ion/issues/302#issuecomment-643919047, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AADAOY5S6FX4XMM5JU2R6PLRWWZYZANCNFSM4N3FIL5A
.

repo: pion/ion


text:
v2 Plans

@enobufs, I figured we could start talking here publicly about what we should do in v2.
Right now we know that ultimately we want:

RACK
TLP
Blocking writes

The first two make sense to me and can likely be done without an official v2. Blocking writes however is a breaking change and I was curious if you had some ideas in your head on what a newer architecture would look like? I know you mentioned that you don't love the current state machine.
I figure we can start here and try to outline the general idea of what we should do. I don't think the plan has to be hyper specific but it would be good to be on the same page and be able to define units of work that don't result in just some massive PR to get to v2 (unless that is truly the best way!).

Also, for what it's worth, I don't think we're in any rush here, so no pressure to respond ASAP to this.

That's fine with me. Do you have any vision for what you'd re-architect to achieve blocking writes? I could "just" go down a proof of concept route to achieve and see what changes fall out. What do you think?

I see (or hope) not much re-architing required here. sctp.Stream is already looks like net.Conn. We have two goroutines: writeLoop and readLoop, which wouldn't change. Read operation is already blocking, so it is Stream.WriteSCTP() and underlying call into Association needs to be modified. I'd say we can just use Association's mutex to block the write operation (Association.sendPayloadData()). Challenges I can see from top of my head is because we have many streams competing over the shared Association, we'd need to figure out how to control the deadlines (e.g. SetDeadline, SetWriteDeadline)
This is my quick thought (I may be wrong). I think it is good idea for you to just go ahead try it out ;)

I'll give it a go once some of my paid work lets up :D

Please also add fix for https://github.com/pion/sctp/issues/239. It is useful for apps which sends lots of data via DataChannel, and handle network contentions by closing old data channel and opening new one. Now unsent/unacked chunks piles up in inflight queue, increasing memory and CPU usage what is not desirable.

authors: edaniels enobufs sirzooro

repo: pion/sctp


text:
Build problems

Maybe I'm missing something but if I try to build from master it is complaining the following files are missing:

script.html & google-analytics-async.html in  layouts/_default/baseof.html
style.html in layouts/partials/head.html
date-and-tags.html in  layouts/_default/list.html (when adding content)

@backkem what version of hugo? I am not seeing these errors on travis but I might only be building a subset? https://travis-ci.org/pions/website/builds/500463482
Also hopefully we don't have GA enabled! I think I removed everything, but I definitely don't want to be tracking users.

Thanks latest (0.54 extended) maybe the problem only occurs on Windows, idk. I can build if I remove the references to those files mentioned. I'll just do that locally for now.

Forgot about the sub-modules 🤦♂.

authors: Sean-Der backkem

repo: pions/website


text:
Reinstall command

[x] I have added an entry to docs/changelog.md

Summary of changes
Adds a reinstall command to only reinstall one venv.
Closes #256
Test plan

Tested by running
pipx install pycowsay
pipx reinstall pycowsay

It would be simple to make this take multiple argument for multiple venvs.  I kept it at one venv because that matched the other pipx sub-commands.

Adds a reinstall command to only reinstall one venv.

Any reason why just not make this the behaviour of pipx install -f or pipx install --overwrite?

Any reason why just not make this the behaviour of pipx install -f or pipx install --overwrite?

One difference would be that this completely reinstalls the venv, including re-injecting all of the injected packages, while install is currently only charged with affecting the main package of a venv.
This command also reuses all of the original options for the venv and installed packages like pip_args, venv_args, install_dependencies without forcing you to re-supply them on the command line, like install might imply you need to do.

I also just thought it would make sense to the user as an analog to upgrade / upgrade-all.

authors: gaborbernat itsayellow

repo: pipxproject/pipx


text:
На сервере устаревшая версия

Прод откатился на позапрошлую версию. Я так понял новая версия не сбилдилась.

fixed

Опять не билдится.

repo: piterjs/piterjs

license: Unlicense


text:
Berrycam express install

I have cloned the node js and other packages in the Berrrycamexpress directory but I do not get a response to node -v command What do I need to do to make it work? Thank you for your guidance

Get the latest stable node.js package available from the node-arm site:
pi@raspberry ~ $ wget http://node-arm.herokuapp.com/node_latest_armhf.deb

After downloading the package, install it:
pi@raspberry ~ $ sudo dpkg -i node_latest_armhf.deb

This takes a while so be patient... once it finishes, verify it's installed:
pi@raspberry ~ $ node -v

Taken from here:  http://www.andrewconnell.com/blog/setup-node-js-on-raspberry-pi-2-b

I have a newer version of this app here:
https://github.com/stevies/berrycam-aurelia
It is still work in progress, but it is a lot nicer if you are wanting to actually play around with the code and it uses a much more up-to-date JavaScript framework - Aurelia - http://aurelia.io

Thank you. Can I clone the BerryCam Aurelia on my earlier clone of the BerryCamExpress? any potential problems?

They are individual projects.  You should clone to separate directories / git repositories.

Thank you for your advice. I did get the latest node.js package and now I have node version 4.2.1 and npm version 2.14.7. When I run npm install command I get the following warnings
npm WARN package.json BerryCamExpress@0.0.4 No repository field.
npm WARN package.json BerryCamExpress@0.0.4 No license field.
Although I do have the license sub directory in the BerryCamExpress directory.
What am I doing wrong? Thank you for your assistance. Also the Dist directory did not install Is this due to the warnings? I have attached a copy in MS word of the screen
Copy of screen.docx

Those are warnings, not errors - should not affect the BerryCam installation.  However, I think none of this will have been tested with node v4.x.  If it does not work, go back to using node 0.12.7 or something older like that.

Thank you
I am a noobie How do I go back to node 0.12.7? Do I need also an older version of npm? I have currently npm v
2.14.7
Date: Mon, 23 Nov 2015 07:01:47 -0800
From: notifications@github.com
To: BerryCamExpress@noreply.github.com
CC: ishaqsaleem@hotmail.com
Subject: Re: [BerryCamExpress] Berrycam express install (#6)
Those are warnings, not errors - should not affect the BerryCam installation.  However, I think none of this will have been tested with node v4.x.  If it does not work, go back to using node 0.12.7 or something older like that.
—
Reply to this email directly or view it on GitHub.

For modifying the version of node/npm you have installed, see: https://davidwalsh.name/nvm

Than you again
To run the sever on the berrycamexpress server your  instructions say cd to dist. But I could find the dist directory.  Did it noe install when Iran the npm install command? How do I get the dsit dirctory to install?
Date: Mon, 23 Nov 2015 07:01:47 -0800
From: notifications@github.com
To: BerryCamExpress@noreply.github.com
CC: ishaqsaleem@hotmail.com
Subject: Re: [BerryCamExpress] Berrycam express install (#6)
Those are warnings, not errors - should not affect the BerryCam installation.  However, I think none of this will have been tested with node v4.x.  If it does not work, go back to using node 0.12.7 or something older like that.
—
Reply to this email directly or view it on GitHub.

Did you run:
grunt pi
from inside the BerryCamExpress directory?
That should create the /dist directory.

Sorry for being such a pest. Thank you again for your patience and guidance. Iran grunt pi from the BerryCamExpress directory but I gor an error message The screen is as below
pi@raspberrypi ~ $ cd BerryCamExpress
pi@raspberrypi ~/BerryCamExpress $ grunt pi
-bash: grunt: command not found
pi@raspberrypi ~/BerryCamExpress $ cd grunt
-bash: cd: grunt: No such file or directory
pi@raspberrypi ~/BerryCamExpress $ ls
app         Gruntfile.js  node_modules  README.md TODO.md
bower.json  LICENSE       package.json  test
pi@raspberrypi ~/BerryCamExpress $ grunt
-bash: grunt: command not found
pi@raspberrypi ~/BerryCamExpress $
Date: Mon, 23 Nov 2015 12:50:37 -0800
From: notifications@github.com
To: BerryCamExpress@noreply.github.com
CC: ishaqsaleem@hotmail.com
Subject: Re: [BerryCamExpress] Berrycam express install (#6)
Did you run:
grunt pi
from inside the BerryCamExpress directory?
That should create the /dist directory.
—
Reply to this email directly or view it on GitHub.

No problem.
If you get error "grunt: command not found" - that means you have not installed grunt.  You need to go back and follow all the instructions from the beginning - looks like you have missed some steps.
sudo npm install -g grunt-cli

Thank you so very much. I finally was able to run the berrycam server and take pictures from my PC. However do I need to run "grunt pi" each time I start the server? Is this due to the error message I get which is
pi@raspberrypi ~/BerryCamExpress $ grunt pi
Loading "npm-install.js" tasks...ERROR

Error: Cannot find module 'npmlog'
If so how do I correct the error . Thank you again

Date: Tue, 24 Nov 2015 12:28:57 -0800
From: notifications@github.com
To: BerryCamExpress@noreply.github.com
CC: ishaqsaleem@hotmail.com
Subject: Re: [BerryCamExpress] Berrycam express install (#6)
No problem.
If you get error "grunt: command not found" - that means you have not installed grunt.  You need to go back and follow all the instructions from the beginning - looks like you have missed some steps.
sudo npm install -g grunt-cli
—
Reply to this email directly or view it on GitHub.

You only need to run grunt pi to build the project once.

authors: bersal stevies

repo: pitography/BerryCamExpress


text:
Can't get infinite scroll to work with 4 number of items

As the title clearly states i can't seem to get this to work with 4 items. I even changed the number of items to 4 in the example you provide but it still doesn't work even though these cells are fairly large. Is there anyway around this?

There's a minimum number of cells after which this method doesn't work or at least I don't know how to get it work. At some item size and number of items cells start disappearing.
Because you use constant item size and varying number of items you have to set minimum number of items yourself. Look for the line
#define MIN_NUMBER_OF_ITEMS_REQUIRED 6.

Try to set to 2 and fill your collection view with 2 items then run and scroll looking for disappearing cells. if you see them then set to 3.

So just to follow on from this. Is there anyway of having preview cells on the left and right similar to this.

authors: pivl pushonTunde tundsdev

repo: pivl/InfiniteHorizontalLayout


text:
concourse deployment should support ELB

Concourse web VMs should have an ELB in front of them, but currently there is no ELB information in the cloud config, meaning I have to download the cloud config, manually add ELB, and re-upload it, which means I'm working outside of the bbl tool, and I imagine that my config will be removed next time I run bbl (I haven't confirmed this though).
Also, we will need to configure cert information for the ELB somehow via bbl, otherwise we have to upload the certs out of band.

Already on roadmap:

support LB in Cloud Config: https://www.pivotaltracker.com/story/show/115989445
support specifying certs: https://www.pivotaltracker.com/story/show/115989461

authors: Amit-PivotalLabs robdimsdale

repo: pivotal-cf-experimental/bosh-bootloader


text:
Update release-notes.html.md.erb

Updated for today's alpha 6 release

Thanks.

authors: AmberAlston vikafed

repo: pivotal-cf/docs-pcf-healthwatch


text:
release type breaks version

using version:
- name: pivnet
  type: docker-image
  source:
    repository: pivotalcf/pivnet-resource
    tag: latest-final

with the following configuration:
- name: pas-tile-2.8
  type: pivnet
  source:
    api_token: ((pivnet-api-token))
    product_slug: elastic-runtime
    product_version: 2.8.*
    release_type: Security Release

Expected:
no results should match
Actual:

Context:
PAS 2.8 has not been released yet to the public.
We use a refresh token that can see all PAS Dev Releases.
There are 2.8 versions available but there are no 2.8 versions that are publicly available.

Thank you, Paul.  I will add this bug to our tracker.

Hi @pnikonowicz.
After investigating, it seems that the expression provided for the product_version is not strict enough. Under the hood, the filtering out for a version uses the regexp.MatchStringhttps://github.com/pivotal-cf/pivnet-resource/blob/7269b361507e0dd2b451f4b8e7f967d7f679239b/filter/filter.go#L40 method provided by the Go regexp library. The asterisk at the end made it so that the period before could be present or not which is why the release of version 2.2.8 was returned.
For the example above, we would recommend writing the product version as 2\.8\.\d+. You should be able to see the expected result of cannot find specified release. Hopefully this was helpful.
Thanks again for submitting this issue! We'll be updating the README to be more clear about this.

Seems like there is already a reference to this in the README located here.
Closing this issue.

repo: pivotal-cf/pivnet-resource


text:
Add experimental support for Rust dependencies with Cargo

Closes #252

Hey @ChrisMacNaughton ,
Is there any particular reason you changed the Dockerfile from Ubuntu:trusty to ruby:2.5.1?
We aim to support multiple version of ruby.
Also i believe you will need to rebase on master in order to resolve some of the issues currently seen within CI.
Let us know if you have any questions

I updated the Dockerfile simply to reduce the build time, given that the Dockerfile only installs Ruby 2.5.1 anyways, will remove that change and rebase on Master

I see that, since my change, the RVM install has been updated to support multiple versions of Ruby, whereas it only did the single version when I made this PR

Awesome! Thanks for the Pull-request @ChrisMacNaughton!

authors: ChrisMacNaughton xtreme-lisheng-tai

repo: pivotal-legacy/LicenseFinder


text:
ChromeOS: The Private Key for Client Certificate is Missing or Invalid

I have a .ovpn file for a pivpn user that I created at the command line. I am using a chromebook with the most recent stable build of chromeos.
When I attempt to import the .ovpn file as a client certificate through the manager certificates interface as outlined in chrome support, I get a pop-up error stating: "The Private Key for Client Certificate is Missing or Invalid".

nm, looks like this is being addressed in #27

repo: pivpn/pivpn


text:
museum entrance imagemap (needs illustration)

        scene bg museum_entrance_top # TODO: museum entrance imagemap (needs illustration)

https://github.com/pixequil/oso-game-1/blob/c2468fc001bf8e61eca61ad72bab613699563c5a/game/room003-museum_entrance.rpy#L7

https://github.com/pixequil/oso-game-1/blob/c9953ec3e6480bd1dfa989c34c13f39644383997/game/room003-museum_entrance.rpy#L7

Asset: https://drive.google.com/file/d/1Sdb8GUIs26hnxAe6QsNN6fAvjfT9lbCu/view?usp=sharing
Credit: Martysh12

Arrows:


text:
[SUGGEST] Make the ColorReplace Filter support multi colors.

In the real game , replace multi colors is very useful.
e.g.   it could be used for making multi characters with one template image + some color palettes.
this is a template image:

this is a color palette, in game, the bottom colors in the template image will be replaced with top colors  :

another color palette, could create another character:

I'll create a new filter (maybe named MultiColorReplaceFilter )  for this work.

repo: pixijs/pixi-filters


text:
Bug:  Container Types

Current Behavior
Hi, It seems like there is an issue with the example provided on the docs:
I have just created a project with :
`
import { BlurFilter } from 'pixi.js';
import { Stage, Container, Sprite, Text } from '@pixi/react';
import { useMemo } from 'react';
export const MyComponent = () =>
{
const blurFilter = useMemo(() => new BlurFilter(4), []);
return (

<Sprite
image="https://pixijs.io/pixi-react/img/bunny.png"
x={400}
y={270}
anchor={{ x: 0.5, y: 0.5 }}
/>
  <Container x={400} y={330}>
    <Text text="Hello World" anchor={{ x: 0.5, y: 0.5 }} filters={[blurFilter]} />
  </Container>
</Stage>

);
};
export default MyComponent;
`
And it is having a TypeScript issue with the Container component:
Type '{ children: Element; x: number; y: number; }' is not assignable to type 'IntrinsicAttributes & Partial<Omit<Container<DisplayObject>, P | ReadonlyKeys<Container<DisplayObject>>> & WithPointLike<...>> & InteractionEvents & { ...; }'.   Property 'children' does not exist on type 'IntrinsicAttributes & Partial<Omit<Container<DisplayObject>, P | ReadonlyKeys<Container<DisplayObject>>> & WithPointLike<...>> & InteractionEvents & { ...; }'.ts(2322)
If I do remove the container it works
Expected Behavior
Compile example as it is on the  docs
Steps to Reproduce
Create a new react typescript project and copy and paste the example from the docs
Environment

@pixi/react version: e.g. 7.0.0
pixi.js version: ^7.1.2
React version: e.g. 18.2.0
ReactDOM version: e.g. 18.2.0
Browser & Version: e.g. Chrome 108
OS & Version: e.g. Mac Ventura

Possible Solution
Update documentation
Additional Information
No response

Many thanks for posting the issue, I believe it's a dupe of https://github.com/pixijs/pixi-react/issues/350 - closing in favour of that issue.

authors: baseten josegamboa

repo: pixijs/pixi-react


text:
[Feature Request] Add an option to have a custom loading screen, or old loading screens

title says it all

Old loading screens are possible, they may break things though. Custom loading screens aren't possible.

Not possible, sorry

aw :(

repo: pizzaboxer/bloxstrap


text:
Obrażanie graczy LOLa

Tak jak mówi tytuł.
Proposal:
W momencie gdy dowolna osoba na serwerze rozpoczyna grę w ligę, bot wysyła losową wiadomość na kanał związany z ligą. Dodatkowo można wprowadzić mechanizm RNG gdzie jest możliwość wylosowania wyrzucenia lub zbanowania z serwera.
$$P = \frac{l * \frac{1}{3}}{m * 2137} $$

l = ilość aktywnych graczy lola na serwerze
m = ilość użytkowników na serwerze ogólnie

+1 byczq

authors: SirCypkowskyy kpostekk


text:
Testing Jenkins Build

@<reviewer_id>
What issues does this address?
Fixes #
WIP #
...
What's this change do?
Where should the reviewer start?
Any background context?

Can one of the admins verify this patch?


text:
没检测用户

大佬帮忙看下一直没反应是怎么回事?

进入web看看？

进入web看看？

web页面打不开

已解决: 打开api.py, 打开53行，注释54行

请问你是怎么改的呀，没有api.py啊，截个图呗

请问你是怎么改的呀，没有api.py啊，截个图呗

全局搜索

已解决: 打开api.py, 打开53行，注释54行

老哥 好奇你是怎么debug到这个地方的  谢谢

authors: Umbrella-holder W-Think kongzhongfeima yuanyue


text:
Fix #230 - render in dock

Pull request details

[ ] This PR is a bug fix
[x] This PR implements a new feature or introduces new behavior.

Description of the change
Atom introduced Docks 2 years ago as a good place for some elements including terminals. This PR renders the terminal view within a tab inside the bottom dock.  Fixes #230
Notes
Being in the dock provides consistency in atom UX.

@akonwi, thank you for reaching out with this contribution.
For my two cents here, this should be optional (off or on by default doesnt matter), straight replacing the bottom fixed position and removing the control buttons which some ppl use and are somewhat useful, especially the close button, because when you have many terminals open and want to close one or another you loose that granular control, the bottom close terminals button is all or nothing.
Hopefully you will be receptive to a solution which will accommodate that user-base.
Thanks.

That's good feedback. I'll update this to have it be a setting whether to use the dock or not. I might be able to make it so closing the terminal tab in the dock doesn't kill it like.
Regarding the height, can't you resize the dock? I can add the preferred height setting and see if the dock respects that consistently.

That's good feedback. I'll update this to have it be a setting whether to use the dock or not. I might be able to make it so closing the terminal tab in the dock doesn't kill it like.

Thanks =)

Regarding the height, can't you resize the dock?

Yes, but need to keep resizing after each terminal is open, where as before the default 300px (me at 200px height) it was a set and forget thing.

I can add the preferred height setting and see if the dock respects that consistently.

Thanks, I could workaround using CSS, if the implementation naturally supported similar functions as before so that way no need to go under the hood.
And thanks again for this PR, to see individuals like yourself taking active interest in this project is a good thing for sure =)

Gotcha. That makes sense.
I'm happy to help. I just discovered this package a few months ago and it's very useful.

I've pushed new changes to add a toggle for rendering in the dock.

Thank for this, give me some time to see how it works. In the meantime @ivankravets please review this.

No rush. I'm also dogfooding it to catch stuff. Im going to see if I can get the auto open of terminal per file working correctly too

No rush. I'm also dogfooding it to catch stuff. Im going to see if I can get the auto open of terminal per file working correctly too

:+1: :clinking_glasses: here's to success on all counts ;)

@akonwi finally had some time to actually test this better.
The implementation seems solid, there are a couple of issues I see immediately, hopefully nothing big.

Render in bottom dock.

It only render the terminal in bottom dock, I cant seem to be able to drag it anywhere else to left/top/either side.
While I agree that the most usable space for the terminal is at bottom and probably be an OK initial position, I would suggest making the dock render draggable to wherever the user wants (from its original position), thus making the dock version of the terminal 100% flexible.
The classic terminal (what Im calling it where its not the dock) is IMO OK where it is.

The position of the insert text icon, Im not sure really if its possible or not to move it further up, where it is ATM, feels like its wasting too much vertical space, between top of terminal dock and actual area where prompt is. See screenshot.

If at all possible move this into a position that is equally accessible and doesnt waste space, perhaps make the button smaller also? IDK ;)
Overall it seems to work well so far and seems robust enough.
There is perhaps another 2 issues (and apologies for bringing them up in  this manner) these issues are not related to your implementation of dock render, its up to you if you decide if you want or anything can be done about it in this ticket.

Im referring to the Close all button (see screenshot)

The associated ticket is #541

There is also still no scrollbar in the terminal, we only can scroll with arrows

Associated ticket #9
I think both of those unrelated issues would make the terminal immediately more sane.
Finally and also offtopic please see https://discord.gg/EpdGSfH given recent developments we could discuss them in a more appropriate place, dont feel pressured to accept though.
I hope this feedback is easily understood, if not I will try to explain better.
Thanks and apologies for taking so long to get here.

That's all good feedback. Keeping the thread here works better for me and this where I'd be more responsive aside from the Atom slack workspace.

I thought about making the dock movable and decided not to because I didn't want to deal with the conflicts that might introduce in terms of code compatibility. I can look into it and hopefully it's not too big of a change.

In trying to minimize the changes I didn't bother moving the insert text button. You are right, it's sort of obtrusive in its current position in the dock. I'd prefer to put it in the status bar with the other icons, while using the dock that is. Another option is to keep it in its current position but only show it when the mouse is in the terminal. What do you think?

My suggestions above would fix this

That's very odd. I use multiple terminal tabs with no issue. I'll investigate more.

As far as the other 2 issues. I can help with those too. Let's consider them as bonuses for this changeset and if I don't get to them, I'll create a separate PR after this.

2\. I'd prefer to put it in the status bar with the other icons

That would do it.

4\. That's very odd. I use multiple terminal tabs with no issue. I'll investigate more.

:+1:

As far as the other 2 issues. I can help with those too. Let's consider them as bonuses for this changeset and if I don't get to them, I'll create a separate PR after this.

Thats fine, I brought them up cause of the obvious relation. Lets hope though @ivankravets is paying attention to this.

I'd be more responsive aside from the Atom slack workspace.

The Atom slack channel? Sure I have an account, if youre hanging around there it would be nice to chat.

recent updates to this package caused this PR to break.
this is due to the addition of updateToolbarVisibility function within view.coffee
adding this fixes the problem
  updateToolbarVisibility: =>
    if !@shouldUseDock

@kaleidoc0de please make a new PR and we will archive this repo. No new changes more. Let's move to VSCode.

@akonwi @kaleidoc0de
Please submit any pull requests to https://github.com/bus-stop/terminus
Ive worked on forking this project up to 2.10.0 and now its time to let this repo go into a deep slumber.
The project will live on and for such to happen please help out with code additions/changes/fixes to the new address at https://github.com/bus-stop/terminus.
Thanks you for all your support.

https://atom.io/packages/x-terminal is a good alternative which provides docking feature

https://atom.io/packages/terminus also provides a way to move terminals into docks via DnD, + this PR is also open there.

repo: platformio/platformio-atom-ide-terminal


text:
Instancing fix

Fix a bug where the instancing flag wasn't being set correctly on the mesh instance and users had to implement updateShader instead.

I'd be good to update the manual as well:
https://developer.playcanvas.com/en/user-manual/optimization/hardware-instancing/

authors: mvaligursky slimbuck

repo: playcanvas/engine


text:
Failing to download Pre-trained Imagenet

Hello,
Unfortunately, the access to the pretrained imagenet model is no longer open. I cant download the pretrained model described in "./data/scripts/fetch_imagenet_models.sh". Is it the same one as found on ZOO (VGG16)?

you can use the same script in rbgirshick's py-faster-rcnn repository

authors: evangello knwng

repo: playerkk/face-py-faster-rcnn


text:
🛑 Treatmentbank is down

In 5f1b460, Treatmentbank (https://tb.plazi.org/GgServer/search?fullText.ftQuery=lepus&fullText.matchMode=prefix&taxonomicName.taxonomicName=&taxonomicName.isNomenclature=true&taxonomicName.exactMatch=true&taxonomicName.order=&taxonomicName.family=&taxonomicName.genus=&taxonomicName.species=&BibMetaData.docAuthor=&BibMetaData.docDate=&BibMetaData.docTitle=&BibMetaData.docOrigin=&BibMetaData.part=&BibMetaData.pageNumber=&BibMetaData.extId=&materialsCitation.location=&materialsCitation.country=&materialsCitation.stateProvince=&materialsCitation.typeStatus=All+Types&materialsCitation.collectionCode=&materialsCitation.specimenCode=&materialsCitation.LSID=&materialsCitation.longitude=&materialsCitation.latitude=&materialsCitation.degreeCircle=1&materialsCitation.elevation=&materialsCitation.elevationCircle=100&indexName=0&subIndexName=0&minSubResultSize=0) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Treatmentbank is back up in 840f4e5 after 15 days, 6 hours, 4 minutes.

repo: plazi/monitoring


text:
Support for optional decimal

My product manager wants the ability for the user to not have to enter decimals until they press the period key [.]
Currently if you press the key [4][2], it will show up as $0.42 instead of $42.00. What he wants is that if you press [4][2], it will show as $42.00, and if you press [4][2][.][3][9], it will show as $42.39.

Hello, has this feature been added ?

authors: DanielCauser baohouse

repo: plentz/jquery-maskmoney


text:
Fix an issue which prevents blockwise xfer working with observe

If you observe a resource and the state change is too big for a single
packet, it will be sent blockwise. However, only the first block is
sent with the observe token, it is expected that clients will request
subsequent blocks by issuing a new request. Returning a response to
this new request would fail on the client due to ETags not matching,
despite no ETags being used. The failure error was:
received message doesn't contains ETAG but cached received message contains it([])
This patch ensures that the received message actually contains an
ETag, which fixes this issue.

Thx for the fix. Pls could you create a test for the future ?

Ok. Thx I merge it.

authors: Kegsay jkralik

repo: plgd-dev/go-coap


text:
biber-linux_aarch64: providing a regular contribution

Not strictly an issue, but a proposal: Up until now, there has been no regular version of biber for the linux_aarch64 platform. This platform is becoming more and more relevant for people running Linux on their Raspberry Pis or in virtual machines on their Apple Silicon Macs.
Some time ago, @sbrass provided a first aarch64 build based on the containerized build pipeline that was originally developed for linux-musl. Now @sbrass and myself have agreed to collaborate on providing a regular build for linux_aarch64. The build pipeline is in place and we have produced builds of both the v2.19 tag and the current dev branch.
If you're willing to accept this contribution, how do we proceed?
Our resulting biber-linux.tar.gz file contains the structure required for CTAN, albeit only the aarch64 build. My approach would be to upload an update and let the fine people at CTAN know that this upload supplements the files that are already there.
For uploads at SourceForge, we'd add the aarch64 build to the /current/binaries/Linux directory, assuming that my privileges for the Linux-musl directory also apply to the Linux directory.
We might have to submit some changes to the build script in /dist before the next release, so kindly let us know some days in advance before you tag the v2.20 release.

Does this answer some of the questions?:
https://github.com/plk/biber/blob/dev/BUILDERS.README

It took sometime to get things sorted on our end. I've uploaded the v2.19 build to both the current and development directories on Sourceforge. I've also submitted the build to CTAN and asked them to put it in the same directory as the regular Linux build. Let's see if it turns up there in the next days.

Finally, biber for Linux on aarch64 is deployed both to SourceForge and to CTAN. @plk please take a look at the message from the CTAN folks that received as cc. I took the liberty to follow Petra's suggestion with the biber-linux-aarch64 platform name. I hope this works for the people at TeX Live.

2.20 is now released

authors: krumeich plk

repo: plk/biber


text:
Add three layers of error reporting (Client, Server, Redux middleware…

…) and Sentry integration.

Coverage decreased (-0.4%) to 58.103% when pulling a66399d3a1fa110901341de0fc9238e1d9e89d71 on full-error-reporting into d195e2003193944b1284e1de6b525775ba17bb60 on master.

authors: coveralls sneridagh

repo: plone/plone-react


text:
shortening CI and doc building time

following up on the comment from @neelasha23 in our team brainstorming:
shortening the CI and doc building time is essential since we're blocked while waiting for this to finish, sharing a few thoughts
ci
I recently added a flag to our repos so pytest prints tests that take >5 seconds to finish, this can help us spot slow tests and fix them. there might be cases where we have no choice, but in the majority of cases we should strive to have fast tests; we could go beyond printing logs and enforce this by failing slow tests. if we don't enforce it as a rule, then we'd have to run periodic checks to look for slow tests
another solution would be to find a vendor that can run our tests in more powerful hardware. I remember seeing some of them. This can be a quick win.
Finally, can also try running pytest in parallel, there's an extension that allows doing that; the challenge there is that if our tests are not fully isolated, we might sun into issues that will be hard to debug.
docs
similarly, as with the CI, we could print the running time for each notebook so we spot slow ones. I think jupyter-book has an option to timeout slow notebooks so that's another option.
we can also check if readthedocs has an option to build our docs in faster hardware.
running notebooks in parallel is also another option, but in this case, we'll have to implement it ourselves. an alternative can be a caching layer. readthedocs doesn't cache anything since it starts from an empty container but jupyter-book has caching built-in. so we could build something that fetches the latest successful docs build from master/main so we can benefit from it. But I'm unsure if readthedocs has an API for downloading previous builds

@idomic is working on this one. He found an option to cache conda environments: https://dev.to/epassaro/caching-anaconda-environments-in-github-actions-5hde
some thoughts:
Since our projects are Python packages, they must have a setup.py file. All the package dependencies are declared there and we even define optional dependencies (for example, the ones required for testing/development).
the solution that Ido found relies on conda environment files (env.yaml), instead of (setup.py) files but the caching action looks like a generic caching feature so I think it'll work.
I think that changing the run step here:
      - name: Update environment
        run: mamba env update -n my-env -f environment.yml
        if: steps.cache.outputs.cache-hit != 'true'

for the series of steps that we run to prepare the environment will work
for example, for sk-eval, the run step would be all the pip install commands that we have here: https://github.com/ploomber/sklearn-evaluation/blob/3b3f6aa8f568c0b444d132f369e123026fb1c303/.github/workflows/ci.yml#L42

@idomic  as discussed earlier - in the top comment here I mention a few things about speeding up the doc building process. let's assign someone to take a deep dive on this

did some research on accelerating docs building
parallel builds
tl;dr; we can enable this locally for quick testing; however, it's not possible to enable this on readthedocs as it doesn't offer a way to add parameters to the command that builds the docs.
we can enable parallel notebook executing with the -j X option. This speeds up jupysql docs:
parallel:
python -m sphinx -T -E -W --keep-going -b html -d _build/doctrees -D  -j 4 .   54.40s user 10.51s system 85% cpu 1:16.00 total

serial:
python -m sphinx -T -E -W --keep-going -b html -d _build/doctrees -D  .   52.33s user 9.79s system 39% cpu 2:35.62 total

notes
looks like sphinx offers the -j parameter and plugins can decide to take it ignore it. the notebook execution comes from the myst-nb package (documented here), but it's passed to the sphinx CLI, and then grabbed by myst-nb.
One important thing I noted is that this is a "best-effort" thing, if some steps are not parallelizable, the parameter is ignored. I realized this because the flag didn't work I saw some warnings about some latex thing not being parallelizable, then I tried commenting out all the variables in conf.py that reference to latex; ran again and this time it worked (the logs don't say anything but I realized it was working bc the logs printed all at once at the end as opposed to line by line, which happens when running serially)
caching

No solution yet, we need to implement something with S3

Another option is caching the notebooks. jupyter-book already offers a mechanism to cache previously executed notebooks.
The first thing I tried was to enable readthedocs offline formats so each build would produce a zip with all the HTML contents. However, this didn't work and the command triggered a full doc build again since jupyter-book uses a separate folder to store the notebook's state.
the only solution I can think of is to use readthedocs config to run a command before and after building the docs (this is supported). after a successful build, we can upload the full folder to S3 and before each build we can fetch the folder. this will speed things up a lot as the build will only have to run notebooks that have changed.

Which repo should we focus on for this issue? (I saw @edublancas has fixed for skeval).

let's focus on JupySQL, the tests are pretty fast on ubuntu but slow on windows and mac, although that appears to be a hardware problem.
I think let's focus on implementing the documentation caching (as described in my earlier comment) and see how well it works.

repo: ploomber/contributing


text:
Initial PR, added telemetry, need to fix failing test and fixture.

This PR is adding telemetry to both the imports and the tracking functionality of the engine.
Also added a test to mock the 2 calls.

looks good!

repo: ploomber/ploomber-engine


text:
Add error message/handling for case in which there is no app on Gallery.

This PR proposes to display a helpful error message whenever an app exists in the repository but has not been declared in Dash Enterprise's app manager, or when the entry has been removed prior to redeployment.

Looks 👌 to me as well! 💃

authors: rpkyle shammamah

repo: plotly/dash-sample-apps


text:
Do we need to stop the MATLAB engine in Python?

The Python MATLAB engine is started here - https://github.com/plotly/dash_matlab/blob/main/updatePlotlyGraphPy.py#L4. Do we need to explicitly stop it? What are the implications of it not being stopped?

terminating MATLABPyhost or 2nd MATLAB process would close the background matlab session and give back control to main window as before. If dash app is running, second matlab instance must be on too

authors: Igor-gomonov jackparmer


text:
v1.7.8 on MacOS: Unable to start application, hands on verification from system

Looks awesome, can't wait to try it!
I dragged the application to my Applications folder after unzipping. When I tried to open Falcon, it just hangs at the following:

Thanks!

v1.7.8 isn't the latest version.
Have you tried the latest release, v2.3.2?

Thanks for the quick reply @n-riesco
So I clicked the "Download for Mac" button on the falcon homepage

That links to what appears to be the proper release link on github for 2.3.2
https://github.com/plotly/falcon-sql-client/releases/download/v2.3.2/mac-falcon-sql-client.zip
But when I unzip the archive and open up the version below

it reads: 1.7.8

i have the same issue. how can i get the latest release?

PR #277 is addressing this issue. We can update the installers once it gets merged.
If you are familiar with Node.js development, these are the instructions to build the app:
git clone https://github.com/plotly/falcon-sql-client.git
cd falcon-sql-client
yarn install
yarn build
yarn run rebuild:modules:electron

and to launch the app:
yarn start

repo: plotly/falcon-sql-client


text:
Rebase

Targetting the updated main branch for this work.

@agateaux I'm reopening this one to address the conflicts. Would you mind taking a look? FYI this was already merged in https://github.com/plotly/paketo-buildpacks_poetry-install/pull/2 before the branch fixes for the release flow.

@agateaux I'm reopening this one to address the conflicts. Would you mind taking a look? FYI this was already merged in #2 before the branch fixes for the release flow.

Ok, done

Pr for 5.4 create here: https://github.com/plotly/paketo-buildpacks_poetry-install/pull/5

authors: BRONSOLO agateaux


text:
Release new versions

I see in the Changelog that there are two versions, 0.10.1 and 0.11.0, that have not yet been published to crates.io. I would like to take advantage of some changes made in those versions. Could whomever is maintaining this crate publish the versions for others to use?

Thanks for reporting this. I actually didn't realise that. Will release the missing versions asap .

@LafeWessel , just released the latest version: https://crates.io/crates/plotly

authors: LafeWessel andrei-ng


text:
Change Contours size to be f64 instead of usize

Without this, the intervals between contour lines can only be integers.
I needed a smaller interval for a figure.
To Do:

[x] Update changelog

I added this under a new minor version in CHANGELOG.md, since it is a breaking change. Let me know if this should be changed.

Yes please (although I find it annoying that we need to bump the major number for such small changes, we night make other ppl unhappy 😁 )
Could you fix the tests as well please 🙏.

authors: andrei-ng rmburg


text:
Is it possible to use compact structure?

Hello,
is it possible to use a compact structure in the pivot?
Like the example in the printscreen.

Besides, is it possible to do some kind of conditional formatting?
Thanks.

There are no built-in features for these, but you could build your own renderer component that implements this kind of thing, based on the built-in one!

Thank you for the fast answer. I am really new to JS, thus I wanted to avoid that kind of customization, but I will try.

authors: jmcorreia nicolaskruchten

repo: plotly/react-pivottable


text:
🛑 KCG is down

In 978a905, KCG (https://kunstrijclubgroningen.nl) was down:

HTTP code: 0
Response time: 0 ms

Resolved: KCG is back up in 8cdaa03 after 37 minutes.


text:
🛑 KCG is down

In a37b4d6, KCG (https://kunstrijclubgroningen.nl) was down:

HTTP code: 0
Response time: 0 ms

Resolved: KCG is back up in fa52a0e after 9 minutes.


text:
🛑 KCG is down

In 455d1db, KCG (https://kunstrijclubgroningen.nl) was down:

HTTP code: 0
Response time: 0 ms

Resolved: KCG is back up in 20a68ba after 11 minutes.


text:
outside cluster flag unnecessary

We use outsidecluster as a flag to indicate we are outside the cluster.
Is there any reason we cannot do it the way kubectl does (or used to, last time I looked at its code), or cloud-provider? I think it basically does the following:

looks for KUBECONFIG env var or --kubeconfig cmd-line option, if found, use that
looks for the default in-cluster config (service account)
error out

It's another throw back from testing, it will probably be removed later down the line :)

Fixed in 0.3.0 including documentation.

authors: deitch thebsdbox

repo: plunder-app/kube-vip


text:
NetConnectionProfile is unknown

When I run the unattend-lab I get a series of errors. Running Get-PSAutoLabSetting shows that the netconnectionprofile isn't set to private. Is there a way to change this to private that I'm missing? This is my home PC running Windows 10 Pro. On a workgroup, no domain.
Thanks!

A value of unknown makes me think remoting isn't setup properly. What do you get when running Get-NetConnectionprofile -interfacealias *LabNet* I'd also suggest re-enabling PowerShell remoting in an elevated session: enable-psremoting -force then trying setup again. If setup if failing, the virtual switch may not be created which would account for what you are seeing. If setup still fails after enabling remoting again, post the errors. I think that is what we need to address.

I appreciate the timely response. That was a factor in my lab setup issue. I did enable remoting and reran the however the network profile was set to public. Easy enough to change to private and now I'm cooking with gas.
Thanks so much! Excited to start watching your courses.

Good to hear.

authors: jdhitsolutions jooshyboy

repo: pluralsight/PS-AutoLab-Env


text:
Table color issue in new version

Instructions

Table: Latest table version have color issues

Bug
I have updated the table to latest version 4.0.3 and theme to latest version of 3.0.1. After doing that table colors are not what they should be. Check the screenshot for details. The version which was working 3.0.9
Screenshots/casts are helpful
Steps to repro

Related packages

Table@4.0.3
Theme@3.0.1

Triage: will investigate.
Looks like it's text fg color and maybe border colors that are off?
We don't see this problem on the DS site atm.  Might be something in-app.  This is also getting old.  @nouman91 is this still a problem?

the new theme(and related package) releases should have fix this issue.
@nouman91 please update your packages and verify

authors: danethurber jaketrent nouman91

repo: pluralsight/design-system


text:
image is resized

Too bad, my 2560x1600 image was resized to 800x650 after just flipping it horizontally. Is that always the case? Can this be changed to keep the resolution at least to HD?

Aviary is meant to handle file sizes intended for web display, not original resolutions (per their docs). The default maximum width and height is 800px, as recommended by Aviary. In future versions of Ember Aviary, we can make this customizable, but this would just be to make it smaller.

ok will look into that. I am in contact with aviary team regarding higher res images. It seems to be possible with additional parameters and setup steps, but requires approval. I will create a fork as soon as I know more.

Sounds good. Thanks, @amoHoban!

authors: amoHoban danielleadams

repo: plyfe/ember-aviary


text:
Allow to import *.js files. Files exporting an JS object.

I have a need to import various json files. It is a migration from stylus, where I was able to import those json files and assign each to a variable. I do not have this option with this importer.
It would be helpful to be able to import Files exporting an JS object. Example:
const xl = require('./variables.json')
const md = require('./variables-md.json')
const xs = require('./variables-xs.json')
const bscolor = require('./colors.json')

module.exports = {
    xl,
    md,
    xs,
    bscolor,
}

I have tested it with just one modification on this line in your source and it works:
return /\.js(on5?)?$/.test(url);
https://github.com/pmowrer/node-sass-json-importer/blob/master/src/index.js#L47

I'd be happy to merge if you made a PR for it (please also add something about the new feature to the README). Please make sure the commit message begins with feat: in order to trigger an automatic release on merge.

Closed by https://github.com/pmowrer/node-sass-json-importer/pull/90. Thanks @alfredriesen!

authors: alfredriesen pmowrer

repo: pmowrer/node-sass-json-importer


text:
group-header-remove-column-name has made me sad, please help

Sample (which sample are you having trouble with)
group-header-remove-column-name
What Should Happen
If a view is grouped by two columns, the group header should be removed for both columns that the view is being grouped by.
What Actually Happens
Whilst the group header is being removed for both columns, a most perculiar thing is being experienced. When the second level of grouping is expanded, all the nodes at that level acquire the same name, and this also happens at the first level of grouping. It's very odd. It is like a variable is set and all the nodes reference that variable instead of the raw data. I believe the "txtContent": "@group.fieldData.displayValue" part of the JSON is where the issue lies.
https://github.com/pnp/List-Formatting/assets/36215002/b8c65ec0-4627-4549-9062-1162d47adad2

Thank you for submitting this issue @JoeAyre ! I checked and was able to reproduce the same phenomenon in my environment.
However, I believe this is not a bug in the sample, but in Microsoft Lists. Therefore, I have created the following issue and reported it to Microsoft.
https://github.com/SharePoint/sp-dev-docs/issues/9794

Thanks @tecchan1107 for going to the effort of reproducing the issue and writing up the bug report in the SharePoint area. Hopefully we get a resolution on this soon.

@JoeAyre
This issue seems to have been fixed. Could you please check when you have time?
If this issue has been resolved, please close this issue.

@JoeAyre This issue seems to have been fixed. Could you please check when you have time? If this issue has been resolved, please close this issue.

Hi @tecchan1107 - yes, this fix appears to have reached our tenant and the odd behaviour is no longer occurring. Thanks for your persistence on this one.

Hi @tecchan1107  - yes, this fix appears to have reached our tenant and the odd behaviour is no longer occurring. Thanks for your persistence on this one.

repo: pnp/List-Formatting


text:
Remove stringify from json property for spo applicationcustomizer get

When you look at the command spo applicationcustomizer get, in the result we have a property called Rights which is a JSON object.
Currently, we are stringifying this JSON object when it's returned from the API. There is no need to stringify this property, let's output it as a proper JSON property.
The output should look something like:
{
  "ClientSideComponentId": "7096cded-b83d-4eab-96f0-df477ed7c0bc",
  "ClientSideComponentProperties": "",
  "CommandUIExtension": null,
  "Description": null,
  "Group": null,
  "Id": "14125658-a9bc-4ddf-9c75-1b5767c9a337",
  "ImageUrl": null,
  "Location": "ClientSideExtension.ApplicationCustomizer",
  "Name": "Some customizer",
  "RegistrationId": null,
  "RegistrationType": 0,
  "Rights": {
    "High": 0,
    "Low": 0
  },
  "Scope": "Web",
  "ScriptBlock": null,
  "ScriptSrc": null,
  "Sequence": 0,
  "Title": "Some customizer",
  "Url": null,
  "VersionOfUserCustomAction": "16.0.1.0"
}

Since we have v8 branch we may now start working on it!
Let's get this 🚢
and Let's put CLI to the 🚀🌙

Can I work on it?

authors: Adam-it SmitaNachan milanholemans


text:
New command: planner plan add

Adds new command planner plan add.
Closes #2253

Nice! We'll review it asap 👏

@waldekmastykarz,
I have fixed all the feedback points. Could you please review the same?
Thank you !

Will do! Appreciate the quick turnaround 👍

Hey @sprider, I'm getting merge conflicts when trying to rebase your PR. Could you please sync the latest changes from main to your fork and update your PR? That way we can ensure that we won't miss anything. Thank you!

Hey @sprider, I'm getting merge conflicts when trying to rebase your PR. Could you please sync the latest changes from main to your fork and update your PR? That way we can ensure that we won't miss anything. Thank you!

@waldekmastykarz Sure, Could you please check?

Hey @sprider, if this PR is ready for review, could you please mark it as such? That way it will be included in the next round of reviews. Thanks!

Hey @sprider, if this PR is ready for review, could you please mark it as such? That way it will be included in the next round of reviews. Thanks!

@waldekmastykarz Sorry, I forgot to press the review button.

Hey @sprider, unfortunately, I'm still getting merge conflicts when trying to rebase this PR onto main. Could you please update your main with upstream and then try to rebase your feature branch? That should let you see what I'm seeing. If you find it easier, you could also close this PR and open a fresh one. Appreciate your help 😊

Hey @sprider, unfortunately, I'm still getting merge conflicts when trying to rebase this PR onto main. Could you please update your main with upstream and then try to rebase your feature branch? That should let you see what I'm seeing. If you find it easier, you could also close this PR and open a fresh one. Appreciate your help 😊

@waldekmastykarz, I did the same last time. Not sure what is going wrong.
Anyways I will close and create new pull request. Sorry for the inconvenience

authors: sprider waldekmastykarz


text:
Added support for upgrading SPFx@1.9.0 projects solving #1044

Pull request created as reference of what files has changed. This will be merged manually.

Coverage remained the same at 100.0% when pulling e4bcd14e45e9df4133b5a08a1deadac3e0e67561 on VelinGeorgiev:upgrade-to-190 into c7ad6cf12f5fa947fc2ab60e0257ee1bff9bd789 on pnp:master.

authors: VelinGeorgiev coveralls


text:
adds 'yammer message like set'

implements  #1455

Coverage remained the same at 100.0% when pulling 9d2852256e1ef40af96134bc356c91d6ae606cc0 on plamber:yammer-message-like-set into b847249a22c3ae32a480a00db4a800503204cd96 on pnp:dev.

Thanks @plamber, we'll go through it shortly 👍

Hi,
I just updated the code to reflect what has been defined in #1558.
br,
Patrick

@waldekmastykarz Will check and come back with a fix

Hi @waldekmastykarz,
i revisited the whole command and changed all tests.
If you are wondering why the message-list command is also included, I had to change the "cli consent" text into that file too.
Hope this is not a problem
Thank you for checking

Thanks for the quick turnaround! Update to message-list is fine. We'll go through it asap! Thanks for your help 👏

Merged manually. Thank you! 👏

authors: coveralls plamber waldekmastykarz


text:
Added the 'spo list contenttype list' command solving #595

Added the 'spo list contenttype list' command solving #595

Coverage remained the same at 100.0% when pulling e44fa9b126b9418fac91cb942d97a6b0e13220c3 on phawrylak:feature/595-list-contenttype-list into 51ac522e5b8e473a454a67fec451f216110bde7c on pnp:dev.

Nice! I'll review it shortly! 👏

authors: coveralls phawrylak waldekmastykarz


text:
Sample Update - Hide SharePoint list from Site Contents

What's in the pull request

Updated PnP PowerShell script to hide SharePoint online list from Site Contents
Added CLI for Microsoft 365 script to hide SharePoint online list from Site Contents

Hi @ganesh-sanap, thank for you the new sample and improvements, this will be live shortly 🎉

authors: ganesh-sanap pkbullock

repo: pnp/script-samples


text:
Cannot read properties of null (reading 'value') at DynamicForm.js:480:1

Category
[ ] Bug
Version
Latest version 3.13.0
SPFx - 1.16.0
Expected / Desired Behavior / Question
Should load the new form for configured list
Observed Behavior
Doesn't load new form. Throws error in console as below

Error get field informations TypeError: Cannot read properties of null (reading 'value')
at DynamicForm.js:480:1
at step (DynamicForm.js:57:1)
at Object.next (DynamicForm.js:38:17)
at step (DynamicForm.js:42:1)
at Object.next (DynamicForm.js:38:17)
at fulfilled (DynamicForm.js:29:1)
Steps to Reproduce
Create a list with MMD field pointing to a termset. Configure the list in form customizer config.
Run gulp serve
Thanks!

Please note that the error disappears and form loads as expected when we remove the MMD field.

The below issue is same as mine. Fix has been merged. Hopefully, we see it in next release. But this is a blocker for now.
https://github.com/pnp/sp-dev-fx-controls-react/issues/1422

repo: pnp/sp-dev-fx-controls-react


text:
PNP SPFx FilePicker multi select

Hi,
i'm just beginning with react so I will not be doing pull request any time soon.
I have and feature request for enhancement of PNP SPFx FilePicker - would it be possible to add option for multi choice of files?
Thanks
Category
[x] Enhancement
[ ] Bug
[ ] Question
Expected / Desired Behavior / Question
User is able to check multiple files in library by clicking on the checkbox in front of the filename that is already displayed in latest version of filepicker
Thanks!

Hi @evlo!
Thank you for the proposal.
We'll take a look into it.

I will open new one if I come across that need in the future and find out it would still be nice enhancement.

@cneSPFX0815 fair point, I think you are correct and I misinterpreted it. Thanks for your input!
@MonalisaBaltatescu still available to help? Can I assign this to you?

@joelfmrodrigues  Yes, I still want to contribute :)

included in v3.6.0

authors: AJIXuMuK MonalisaBaltatescu evlo joelfmrodrigues


text:
Support for check-coverage with isparta

isparta which I seem to need to support my es6 stuff doesn't provide the check-coverage function that comes with istanbul and therefore when I add coverage numbers into the config grunt-mocha-istanbul will fail.
Would it be possible to add a separate option to configure the coverage comand or default to istanbul inside of the executeCheck function?

I'll close this for now, if isparta ever implements this, should work out of the box if it follows the same command line

authors: pocesar xjamundx

repo: pocesar/grunt-mocha-istanbul


text:
Haptic Feedback

Describe the feature you'd like to have in Pock
A clear description of what you want to happen.
Integrating haptic feedback with the touch bar, so if we unwillingly touch anything, we are notified by 3d touch or haptic feedback.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Have used a few apps from the internet that have it done, but they're clunky and resource dependent
Additional context
Add any other context or screenshots about the feature request here.
n/a

bumping

repo: pock/pock


text:
Feature Request: “personal data”, its “processing” and “data controllers” under the GDPR

Add an option to automate the GDPR (Europe) or CCPA (California) or IT ACT (India) or PIPL (China) THE APPI (Asia) implementation

I'm not sure that I understand what do you mean. PocketBase by default doesn't use cookies and doesn't collect personal identifiable information. Could you elaborate?

Thank you for your interest. I am suggesting to add an option for PocketBase users collection to automate the process of collecting user consent for data privacy laws such as GDPR, CCPA, IT ACT, PIPL, and APPI, by the moment we have "verified", it would be nice to have one bolean labeled as "data" for data protection acceptance by the user.
This is an interesting suggestion as data privacy laws are becoming increasingly important and businesses must comply with them. By automating the process of collecting user consent with a check, PocketBase users would be able to easily comply with these laws without having to manually create screens and collections for each law.
I am passing this suggestion to the PocketBase development team for their consideration. Thank you for your time!

it would be nice to have one bolean labeled as "data" for data protection acceptance by the user.

users and any other auth collection can have as many other custom fields.

By automating the process of collecting user consent with a check, PocketBase users would be able to easily comply with these laws without having to manually create screens and collections for each law.

But what process?
I'll close the issue for now since I don't think this can be generalized as it will depend what personal data your application is storing.
If you have an idea how this could be approached in a generic way, feel free to provide more details and I'll reconsider it.

Any country must comply with data regulation, because users from Europe in US projects must to comply with law data regulations, so a field of a box of "data accepted" should be a good implementation

so a field of a box of "data accepted" should be a good implementation

For me it doesn't make sense to define for every auth collection if there is no extra business logic binded to it. If you just want a bool check, you can add it on your own using the bool field type:
.

Thanks! Yeah!

authors: ganigeorgiev miguelgargallo

repo: pocketbase/pocketbase


text:
Problems running latest Mac OS Sequoia 15.0

"pocketbase" can't be opened because Apple cannot check it for malicious software.
This software needs to be updated.
Contact the developer for more information.
Chrome downloaded this file today at
12:34 PM.

This warning unfortunately is unavoidable as I don't have mac nor I'm able to sign the binaries.
You can check https://github.com/pocketbase/pocketbase/issues/146 and https://github.com/pocketbase/pocketbase/issues/2819.

authors: ganigeorgiev jeffscottward


text:
App not present in recents when opened from a hyperlink

If the app opens from an intercepted hyperlink, it doesn't show up in Recents.

This is normal behavior, it doesn't synthesize a backstack and some tasks behave like this in recents on android

Is there any particular reason this was done? If I open the app from an outside link, isn't it more natural to treat it as if it was a normal application flow(like most other apps) and give users the ability to switch apps as usual?

I, and most users I think, would disagree. That's fairly standard for handling deep links. Your user was navigated to your app from somewhere else, likely just to glance at something or do a quick response.

Besides not having the ability to switch apps, what excluding this from Recents does is that, the activity doesn't really close once you click on Recents, it just doesn't show up there. So say, if you pressed back (from the top left icon) and then clicked on Recents it would look like the app was killed but if you go back again to the app from a deep link, it would open back the same activity which was never killed, and will not open what the deep link was meant to open.

gif incoming..

it doesn't synthesize a backstack

I am not saying we should create a backstack in this case. That's not really necessary. All I am saying is, don't exclude this from the Recents. Recents exists to list out recently accessed activities and tasks, so it's only natural this shows up there. Having this up there will also avoid bug mentioned in https://github.com/pockethub/PocketHub/issues/993#issuecomment-195201759 because user has the ability to kill the activity.

I can't tell what's wrong with that gif, nor did I understand what you were describing in your hypothetical situation. Pressing the home button does bring you farther back into the app, that seems like normal behavior to me. Pressing the navigation back button would go back to your email.
This is also important for cases where you don't actually need to handle a deeplink with UI, such as our callback for oauth login. I'm sorry but you'll have to accept this for now, I can look at it more when I have time.

I can look at it more when I have time.

I am willing to fix this if you'd be willing to accept the fix. :) But alright, I have explained why this is necessary. Think about it when you can. :)

I need to look at it and see if it still makes sense behaviorally. The fix is just a one-liner, not much work for me :P

The fix is just a one-liner

I thought so too at first. ;)
But the way it's written now, what would happen is, you can end up with multiple activities from the same app in Recents once you remove the condition. Something like this

which is also not very good ux :). So we'd need to fix that as well.

authors: aandis hzsweers

repo: pockethub/PocketHub


text:
Create a .yml file for the project

It would be nice if we use the same environment for different projects.
Not sure who else will be working with this, but I can create one and add information about the installation to the README.

Great idea! Please go ahead and create one that works for your tutorial, and then we'll keep it updated as more notebooks are added.

authors: iuryt torimcd


text:
Couple of observation regarding dbsession

Good day @sunli829
I am still testing with poem, I want to share small observation with dbsession for consideration or further checking.

For postgres, not sure if it is correct or not but it looks like it will fail if session column is not JSONB. If it is then maybe there should be a note.

Cleanup task should probably be configurable. When we have multiple poem servers running, we probably don't want every server to ping database to do clean up. I am not sure what the best way to approach this as some pros and cons of each setup or we just need to accept the duplicate jobs?

Not very high priority, the value of in session in db looks like this {"id": "\"xxx\""}, it looks like the library does serialising twice at some points. It doesn't seem to affect how it works, nevertheless it is kind of nice to have correct json presentation.

For postgres, not sure if it is correct or not but it looks like it will fail if session column is not JSONB. If it is then maybe there should be a note.

Yes, if you use postgres, the session column must be jsonb.
https://github.com/poem-web/poem/blob/84e27e6b83cfd1f5c644dde9691284c8bd513770/poem-dbsession/src/sqlx/postgres.rs#L31

Cleanup task should probably be configurable. When we have multiple poem servers running, we probably don't want every server to ping database to do clean up. I am not sure what the best way to approach this as some pros and cons of each setup or we just need to accept the duplicate jobs?

You are right, so I changed it, and now only provides a cleanup function, users need to call it themselves.

Not very high priority, the value of in session in db looks like this {"id": ""xxx""}, it looks like the library does serialising twice at some points. It doesn't seem to affect how it works, nevertheless it is kind of nice to have correct json presentation.

It does serialize twice, but works fine. 🙂

Not very high priority, the value of in session in db looks like this {"id": ""xxx""}, it looks like the library does serialising twice at some points. It doesn't seem to affect how it works, nevertheless it is kind of nice to have correct json presentation.

It does serialize twice, but works fine. 🙂

Yeah looks like it is due to Session is using entries: BTreeMap<String, String>, coming from other languages I see people keep session in a more flexible JSON format.
https://github.com/gorilla/sessions/blob/master/sessions.go#L36
Not saying it is the standard but just something from prior art to consider.

I thought you didn't care about this. 🙂

Fixed in v.1.2.8

Awesome thx!
I was taking a break so that you can do more productive work elsewhere  jk jk ;)
Btw some shower thoughts this morning, happy if you say the following idea is not worth the effort.. etc..:
Basically in Go, people extract session into a separate library https://github.com/gorilla/sessions so that it can grow by itself, i.e. used for different servers or different stores. Not sure if it is worth doing the same thing in Rust? since poem session looks more or less self contained already.
I am not sure how other servers would use it though, cheers

I am closing this as nothing much to do about it and it is end of the year.
Happy NY to you, have a good time with family and friends! @sunli829

repo: poem-web/poem


text:
Transform errors when extract parameters

Description of the feature
This code snippet:
#[handler]
async fn query(
    Json(query): Json<Query>,
) -> String {
    query.query
}

would be expanded to:
#[allow(non_camel_case_types)]
struct query;
impl poem::Endpoint for query {
    type Output = poem::Response;
    #[allow(unused_mut)]
    async fn call(&self, mut req: poem::Request) -> poem::Result<Self::Output> {
        let (req, mut body) = req.split();
        let p0 = <Json<Query> as poem::FromRequest>::from_request(&req, &mut body)
            .await?;
        async fn query(Json(query): Json<Query>) -> String {
            query.query
        }
        let res = query(p0).await;
        let res = poem::error::IntoResult::into_result(res);
        std::result::Result::map(res, poem::IntoResponse::into_response)
    }
}

I know I can change the return type of my function to customize error in my handling logic, but what if I'd like to decorate the error message when fail to <Json<Query> as poem::FromRequest>::from_request(&req, &mut body).await??.
Code example (if possible)
By default, it may return:
HTTP/1.1 400 Bad Request
Content-Length: 77
Connection: keep-alive
Date: Sun, 12 May 2024 06:30:37 GMT
Keep-Alive: timeout=4
Proxy-Connection: keep-alive

parse error: invalid type: integer `1`, expected a string at line 1 column 12

while I'd prefer to customize the error message, like:
HTTP/1.1 400 Bad Request
Content-Length: 77
Connection: keep-alive
Date: Sun, 12 May 2024 06:30:37 GMT
Keep-Alive: timeout=4
Proxy-Connection: keep-alive

0: fail to parse JSON payoad, at src/lib.rs:4:10
1: parse error: invalid type: integer `1`, expected a string at line 1 column 12

By default, the extractor will return a 400 Bad Request when an error occurs, but sometimes you may want to change this behavior, so you can handle the error yourself.
In the following example, when the Query extractor fails, it will return a 500 Internal Server response and the reason for the error.

Seems like we can use a Result<[Extractor]> extractor?

Result<[Extractor]> seems the solution:
impl<'a, T: FromRequest<'a>> FromRequest<'a> for Result<T> {
    async fn from_request(req: &'a Request, body: &mut RequestBody) -> Result<Self> {
        // FIXME: remove the unnecessary boxed
        // https://github.com/rust-lang/rust/issues/100013
        Ok(T::from_request(req, body).boxed().await)
    }
}


text:
without configs with sailjail disable fails to open

Let's try to move this to sailjailed named org/app

Fixed by @carmenfdezb

repo: poetaster/harbour-spritradar


text:
Multiple bridges

I have my lights split between two bridges. Is it possible to add both bridges in lampe?

I like this as a feature. This means a major rework of most parts. Maybe I find the time in December.

One first remark: Because I have only the first generation of the Bridge. I cannot test how this would work with the newer models, if the API has changed.

My current idea to implement this would mean that there is a possibility to switch between bridges with "B" and then typing a number, supporting max 10 brigdes starting with 0 as the first and default bridge. As soon as someone types "B1" and there is currently no bridge 1, the initial wizard would be triggered to register at a new bridge.
Maybe this can be implemented with less code changes I previously thought.
Having bridge-prefixes for lamps would be a thing for another time. I don't want to change the current behaviour managing the lights at this time.
I will base this this on the existing release-branch, meaning additional improvements from ages ago will finally be coming to the main branch.

Awesome. Thank you poinck. Happy new year!

I have just tested my first version with multi-bridge support here: https://github.com/poinck/lampe/releases/tag/v1.2.0
If you have time to test it, this would be awesome. If you have "lampe" already checked out with git, you can get the newest version with git switch main and git pull. There is no need to install it; you can start it directly from the project-root with ./lampe.

authors: pmtek poinck

repo: poinck/lampe


text:
FileStorage persistence sometimes doesn't save data to disk.

Description
I noticed that some mutations sequence of persisted shared state (file storage) breaks saving to disk. What is interesting that it has to be sequence of fast and little slower mutations. After saving on edit become broken then it stay broken until some other saving trigger will save the file (for example willResignActiveNotification), then it restores save on edit.
Problem occurs on either simulator and device and debug and release mode.
During test sometimes saving was broken immediately after second edit. However most of the time, sequence of edits was necessary to cause the problem
Below is a sample project which illustrates the problem and also screen recording. On screen recording I also catch some weird behaviour when editing file on disk, which caused some weird data values to be "restored"
sample project: SharedStateNotSaved.zip
screen recording: https://nextcloud.pomba.org.pl/s/5QiK77q7AKSF49C
Checklist

[ ] I have determined whether this bug is also reproducible in a vanilla SwiftUI project.
[X] If possible, I've reproduced the issue using the main branch of this package.
[X] This issue hasn't been addressed in an existing GitHub issue or discussion.

Expected behavior
Shared state with file storage persistence should be reliably saved according to 1s eager throttle strategy.
Actual behavior
After some sequence of mutation saving is broken.
Steps to reproduce

Run attached sample project.
Tap increase button and then  Read value from disk button to check if value was successfully saved.
Tap Auto increase button, which fires a sequence of edits. After the edits and additional delay, value is read from disk.
Both count values should be the same, but they are not.

The Composable Architecture version information
1.10.3
Destination operating system
17.14.1
Xcode version information
Version 15.3 (15E204a)
Swift Compiler version information
swift-driver version: 1.90.11.1 Apple Swift version 5.10 (swiftlang-5.10.0.13 clang-1500.3.9.4)
Target: arm64-apple-macosx14.0

@Malauch Good find! Thank you! We will get a release out very soon.

authors: Malauch mbrandonw


text:
Feature request: Add non-changeable alt forms and gender differences

Currently trying to track my shiny living dex, but at the moment the only alternate forms pokedextracker.com tracks are G-max and regional variants. This means that I can't track, for example, the fact that I've got both Toxtricity forms shiny. Naturally, this isn't just the case with shinies, but also all alternate forms.
It makes sense that different forms of some Pokémon like Castform or Deoxys aren't tracked, as it is possible to change between them freely by various means. But it would make sense to separately track forms that can't be switched between, such as Urshifu, Lycanrock, or anything with a gender difference.
I'm sorry in advance if there's something I'm doing wrong with this post, as I literally only created a Github account to suggest this, so I don't really know how the site works.

You're totally fine! Thanks for the feedback. Merging into the main issue we have for this feature request: https://github.com/pokedextracker/pokedextracker.com/issues/256

authors: StrawDeath ami


text:
[WIP] fixing header issues in HMC

Fix header issue caused by bias field correction. Closes #497
Use reference image with bias field instead of after correction when doing motion correction. The reasoning is that we should match the bias field between the reference image and the 4D bold volume (which is not corrected for bias).

Waiting for @feilong to confirm this fix works.

Last commit (91161f7f1a8058c2de31897a26524fd84255303a) gave me correct headers and resolutions for all my subjects.
With the most recent commit (f4bf657361a320514fd7dfd503d2799967b21173) the problems are back again.

Commit 51bab778107fad6f2935c3b90ba5ea96965f8cb7 works well on my data.

That's great news - just to confirm did your test include upgrading niworkflows (which is part of this PR)?

I got some problems when I first tried to use the latest commit, so I upgraded niworkflows, nipype, and pybids to versions shown in requirements.txt

Awesome! Thank you so much for testing this. I feel much better knowing we have resolved this issue!

You are most welcome!  Thanks for fixing this issue!

authors: chrisfilo feilong

repo: poldracklab/fmriprep