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

Link to reproduction
https://github.com/LorenzoInvernizzi/issue-endsession-undefined-update-access
Describe the Bug
Hi Devs,
I encountered the error in the following situation: edit of a collection item (Timecards collection) with the following access function for create & edit:

Basically i have the following collections:

users: admin or developers, developers must be able to login and create/view/edit their own timecards
clients
projects
timecard: submitted by devs, here i am using the access function and this is the place where i get the error when i try to edit an existing timecard. Actually i am not able to edit the timecard at all, i think it is related to the issue.
i also created another access function (isAdminOrDeveloper) where i was able to implement a workaround but it involves querying the db with the local api

To Reproduce

clone repo, setup .env file with mongodbatlas db or a local mongodb
yarn dev and create admin
create developer user from admin panel
create a client and a project from the admin panel
logout and login with dev account
create timecard with dev account
try to edit timecard with dev account -> get error

Payload Version
^2.1.1
Adapters and Plugins
bundler-vite, db-mongodb

I am running into the same error message for similar cases in several of my collections. After some testing, it seems like it triggers on field access functions if one field has more than a single access function, or if two or more fields have one or more access functions. I'm using the MongoDB adapter. It looks like the transaction is being shut down prematurely.
Here is example code to explain when the error triggers in my app:
const isSuperuser = (args) => args.req.user.superuser;

// No error
{
  fields: [
    { name: 'fieldOne', type: 'text', access: { create: isSuperuser, } },
    { name: 'fieldTwo', type: 'text' }
  ]
}

// Error
{
  fields: [
    { name: 'fieldOne', type: 'text', access: { create: isSuperuser, update: isSuperuser } },
    { name: 'fieldTwo', type: 'text' }
  ]
}

// Error
{
  fields: [
    { name: 'fieldOne', type: 'text', access: { create: isSuperuser } },
    { name: 'fieldTwo', type: 'text', access: { create: isSuperuser } }
  ]
}

Getting this same error with v2.1.1. Reverting to v2.1.0 fixes the issue.

Getting this same error with v2.1.1. Reverting to v2.1.0 fixes the issue.

2.1.0 fixes it thanks!
@peterskeide explains it perfectly. With one field access it works, with more than one I get this error:

and all the fields of the collections become readonly / uneditable.

Commenting again to add that I don't have any field-level access checks, but I do have multiple (create, read, update, delete) collection-level access checks.

Same issue here. This only happens when there are field level access control checks on the collection.

I'm closing this as it has become stale. Tag me if this is still an issue and we can re-open it.

authors: DanRibbens LorenzoInvernizzi billyhawkes jasperverbeet peterskeide revnelson


text:
Add analytics

Analytics will help give us an idea of what people are looking at most and where to focus efforts.
I don't think Google analytics in the browser will be good enough because many of our target audience blocks those. Any ideas?

I've been working on a thing.
https://github.com/vesparny/fair-analytics

That looks great! One thing. I really like the user privacy thing. Is there any way to get the country and/or language preference of the user? This will help us know what languages to focus on getting translations for.

Whoops! Agreed this to the wing repo! lol

source: gharchive/issue

authors: kentcdodds vesparny

repo: paypal/glamorous

license: MIT

license_type: permissive

license_source: github-api


text:
Deprecation

A few changes

Removes track function so the tracker component never sends out beacon again for shopping events

replaced by: https://github.com/paypal/paypal-muse-components/pull/107/files

source: gharchive/pull-request

repo: paypal/paypal-muse-components


text:
🛑 Departamento is down

In af209cb, Departamento (http://cca.unb.br/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Departamento is back up in 3468bd3 after 18 minutes.


text:
🛑 Departamento is down

In bcc6644, Departamento (http://cca.unb.br/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Departamento is back up in e321811 after 12 minutes.


text:
🛑 Departamento is down

In 0086657, Departamento (http://cca.unb.br/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Departamento is back up in ebb78c1 after 25 minutes.


text:
Update to Java11

Update dependencies to support Java11

Done.

repo: pazuzu-io/pazuzu-registry


text:
🛑 PBrain CN tenant is down

In 0213b9d, PBrain CN tenant (https://www.pbraindemo.cn/xiaomi-airpurifier) was down:

HTTP code: 0
Response time: 0 ms

Resolved: PBrain CN tenant is back up in 40a481d after 1 hour, 11 minutes.

repo: pbrainbiz/upptime


text:
Does this work on rollup chains (Optimism, Arbitrum)?

Pardon me for opening an issue for a simple question (may be worth it to enable discussions on this repo?)
I'm currently using the deterministic-deployment-proxy to deploy contracts deterministic addresses across chains. One of the issues with that it can't be used on Arbitrum and Optimism..
Is xdeployer able to deploy to rollup chains as well?

@paulrberg good point: I enabled discussions here.
Regarding your question: yes xdeployer is able to deploy on rollup chains. Currently, however, only the testnets are supported (I will add later the prod networks):

optimismtestnet
arbitrumtestnet

For testing purposes, I just deployed a mock ERC20 including constructors on both rollup chains:

optimismtestnet: MyToken
arbitrumtestnet: MyToken

Also (just for testing) I also transferred the tokens and it worked smoothly:

However, you must increase the gasLimit significantly to deploy on Arbitrum as it seems. Also, in case interested check here for the standard Arbitrum ERC-20 interface.

I enabled discussions here.

Note that you can convert this issue to a discussion. There's a button on the right-hand side of the screen.

However, you must increase the gasLimit significantly to deploy on Arbitrum as it seems

Yes, we have to do that because of the way the rollup chain txs are batched. That was actually my fear, that bumping the gas limit would break xdeployer.

authors: paulrberg pcaversaccio

repo: pcaversaccio/xdeployer


text:
some records are missing file number

This is missing the the active license export. Looking at the larger file with all applications it appears that contained the correct file numbers however the files don't contain the active callsign. From an initial look this might be a result of the vanity application process.
ex: https://hamcall.dev/ww0cj.json

fixed with 135fbfa6d258d23fae98cdb8fb2a2b7110fd6816

repo: pcunning/hamcall


text:
Font metrics for Zapfdingbats not found

When testing a pdf creation program I iterated over the 14 core fonts and tried to get both the pdf font object and its corresponding metrics object for each. All attempts were successful except for the Zaphdingbats: its font object was succesfully obtained BUT the metrics object was not:
my $afm = Font::AFM.new: :name<Zapfdingbats>;
Can't find the AFM file for Zapfdingbats (Zapfdingbats.afm)
 in method load-afm-metrics at /home/tbrowde/.raku/sources/71E60D8F710457D56668C01B4743FB3D93178B9D (Font::AFM) line 263
 in submethod TWEAK at /home/tbrowde/.raku/sources/71E60D8F710457D56668C01B4743FB3D93178B9D (Font::AFM) line 207
 in block  at ./fonts.raku line 21
 in block <unit> at ./fonts.raku line 19

The program I used for testing is in the dev subdirectory of my PDF::Document repo at https://github.com/tbrowder/PDF-Document

I think the README and doco for Font::AFM is pretty misleading (and that error message isn't great).
Font::AFM.new: :$name is scanning the users system font directories for the given font,
If all you want is metrics for the core PDF fonts, there's some prebuilt classes for that: Font::Metrics::zapfdingbats,  Font::Metrics::courier, etc.
These are listed in the README.

I think the README and doco for Font::AFM is pretty misleading (and that error message isn't great).
Font::AFM.new: :$name is scanning the users system font directories for the given font,
If all you want is metrics for the core PDF fonts, there's some prebuilt classes for that: Font::Metrics::zapfdingbats,  Font::Metrics::courier, etc.
These are listed in the README.

Ah, sorry, there is already a core-font method which IS being called and from PDF::Content::Font::CoreFont.
Leave it with me.

Ah, sorry, there is already a core-font method which IS being called and from PDF::Content::Font::CoreFont.
Leave it with me.

Have put in a PR to use the core-font method.
The Font::AFM README doesn't even mention this method and could generally do with some TLC.

Have put in a PR to use the core-font method.
The Font::AFM README doesn't even mention this method and could generally do with some TLC.

Font::AFM 1.24.3 has been uploaded to CPAN with documentation updates, including:

Include core-font() example in synopsis
Change misleading documentation for new(). Now describes what this method really does (scan you local system for matching fonts. This is probably a fairly archaic thing to do in 2020).
Mention Font::FreeType and HarfBuzz which can be used to obtain font metrics for other fonts.

Font::AFM 1.24.3 has been uploaded to CPAN with documentation updates, including:

Include core-font() example in synopsis
Change misleading documentation for new(). Now describes what this method really does (scan you local system for matching fonts. This is probably a fairly archaic thing to do in 2020).
Mention Font::FreeType and HarfBuzz which can be used to obtain font metrics for other fonts.

authors: dwarring tbrowder

repo: pdf-raku/Font-AFM-raku


text:
Support initial value for metadata

It would be interesting to be able to set a default value for metadata in the yaml file

In GitLab by @ksiero on Feb 6, 2020, 09:43
Closes with MR !221

removed milestone

In GitLab by @ksiero on Jun 22, 2020, 08:31
closed via merge request !240

In GitLab by @ksiero on Jun 20, 2020, 12:43
mentioned in commit 178e7010045a76b143de3973319439df8a0712bd

In GitLab by @ksiero on Jun 22, 2020, 08:31
closed via commit 178e7010045a76b143de3973319439df8a0712bd

repo: pdidev/pdi


text:
Does MetaBIDx also gives abundance score ?

Wanted to know if/Does MetaBIDx also gives abundance score of species it gets aligned to ?

@arpit20328 MetaBIDx only identifies species, and doesn't give abundance.

repo: pdtrang/MetaBIDx


text:
抓取失败，显示warning

使用的系统、浏览器、Pxer版本

Windows 10
Google Chrome 67.0.3396.62（正式版本） （64 位）
Pxer 稳定版 v7

Bug发生时的地址
https://www.pixiv.net/member_illust.php?id=1113943

Bug发生时Pxer的状态
能够正常跑完，结束以后主面板有提示warning，并在左侧显示24690070:Cannot read property 'innerHTML' of null
print结果为都为零
Bug发生的具体情况
能正常跑完，结束后有warning，抓不到网址

同遇到一个问题，不知道是否解决了？


text:
🛑 amazingair is down

In 1460a7d, amazingair (https://amazingairsolutions.co) was down:

HTTP code: 0
Response time: 0 ms

Resolved: amazingair is back up in 5d44eda.

repo: pearanalytics/uptime


text:
🛑 hdnew is down

In c4c56d7, hdnew (https://www.thetriallawyers.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: hdnew is back up in d7eeb13.


text:
Update for SDK 3 on aplite

Update for Aplite, SDK 3

:+1:

authors: C-D-Lewis MRoyPebble

repo: pebble-examples/actionmenu-multilevel

license: mit

license_source: bigquery


text:
Change ID arg semantics

Closes #53, but extends to all commands that can take a name or ID as the argument. This changes these commands by unifying name and ID arguments/options to a single argument that can take either of the two.
ID's must be prefixed with id:, i.e. the argument "id:123" is parsed into the integer 123, while the argument "123" is parsed into the string "123".

Let's hope this doesn't break too many commands!

repo: pederhan/harbor-cli


text:
BaseRecordController

Created the BaseRecordController component to allow setting up a custom RecordController to record video in other formats that accept h264 and aac or in mp4 using something different from AndroidMuxer (Xuggler or JCodec for example)
By default the AndroidMuxerRecordController class is used.

I like it.
I will modify it a bit to fix a bug found and merge.

I fixed all and refactored the code.
Thank you for sharing :)

Thank you pedro for your library :)!
As soon as possibile i create a repo with 2-3 recordcontroller i wrote (i must refactor some code) so then you'll decide if create a new repo as RTSP-Server

I will add it to README for sure but I'm not sure if create my own repo or only fork it.
I think the fork is the best solution because the code is yours.

Hello pedro!
I have created a repo with two recordcontrollers ;).
https://github.com/FunnyDevs/rtmp-rtsp-stream-client-java-recordcontrollers

Hello pedro! I have created a repo with two recordcontrollers ;).
https://github.com/FunnyDevs/rtmp-rtsp-stream-client-java-recordcontrollers

I like it but only one hint.
Instead of set video configuration in constructor you can get it from MediaFormat like this:
    @Override
    public void setVideoFormat(MediaFormat videoFormat, boolean isOnlyVideo) {
        int width = videoFormat.getInteger(MediaFormat.KEY_WIDTH);
        int height = videoFormat.getInteger(MediaFormat.KEY_HEIGHT);
        int fps = videoFormat.getInteger(MediaFormat.KEY_FRAME_RATE);
        int videoBitrate = videoFormat.getInteger(MediaFormat.KEY_BIT_RATE);
    }

    @Override
    public void setAudioFormat(MediaFormat audioFormat, boolean isOnlyVideo) {
        int sampleRate = audioFormat.getInteger(MediaFormat.KEY_SAMPLE_RATE);
        int channels = audioFormat.getInteger(MediaFormat.KEY_CHANNEL_COUNT);
        int audioBitrate = audioFormat.getInteger(MediaFormat.KEY_BIT_RATE);
        //I should get a way to notify you this if necessary. It isn't a proper way to do it.
        int sampleSize = new MicrophoneManager(null).getMaxInputSize();
    }

This is the real usage of this methods and by this way you can make it really easy to use because you don't need know anything about your configuration. It is configured with stream object parameters.

I don't recommend it.
Stream without audio or without video are supported so I recommend you change record status when the last of both is called.
One way is use 2 bolleans to know if that method was called and start record when both boolean are true
Also, If you can support record only video or only audio it is better to fully support all record modes.
You can know it using "boolean isOnlyVideo" (in setAudioFormat it should be named isOnlyAudio, it is a typo)

Ok, i should have modified the files but, before pushing them, i would like to finish testing onlyVideo and onlyAudio conditions.
To test onlyVideo i have called only prepareVideo(), but how could i test onlyAudio? If I call prepareAudio() without prepareVideo() i obtain a crash
 java.lang.NullPointerException: Attempt to invoke virtual method 'void android.media.MediaCodec.setCallback(android.media.MediaCodec$Callback, android.os.Handler)' on a null object reference
        at com.pedro.encoder.BaseEncoder.initCodec(BaseEncoder.java:70)
        at com.pedro.encoder.BaseEncoder.start(BaseEncoder.java:61)

You can use RtmpOnlyAudio class


text:
The new host wrapper

After profiling about DLS loading, I found EAS's HW was so slow about file operations. It was designed for twenty years ago.
So I reimplemented it, using CRT and some OS-specific APIs. The old one is preserved as optional.
It works on Linux and Windows now, blazing fast (compared than before).

The CRT file operations are only used when file->readAt is NULL.

Benchmark (not authoritative, for information only):
OmegaGMGS2.dls, ~266 MiB, with 484 insts, 15947 regions, 15944 arts and 2255 waves
Old HW:
./sonivoxrender -d ~/OmegaGMGS2.dls  23.50s user 22.76s system 99% cpu 46.503 total

New HW:
./sonivoxrender -d ~/OmegaGMGS2.dls  0.41s user 0.20s system 99% cpu 0.612 total

Benchmark (not authoritative, for information only):
OmegaGMGS2.dls, ~266 MiB, with 484 insts, 15947 regions, 15944 arts and 2255 waves Old HW:
./sonivoxrender -d ~/OmegaGMGS2.dls  23.50s user 22.76s system 99% cpu 46.503 total

New HW:
./sonivoxrender -d ~/OmegaGMGS2.dls  0.41s user 0.20s system 99% cpu 0.612 total

Where can I find OmegaGMGS2.dls? I can only find the sf2 version of it.

Where can I find OmegaGMGS2.dls? I can only find the sf2 version of it.

It is converted from the sf2 version, using Extreme Sample Converter.
BTW, the eas_mdls limits are increased to make it load.

Where can I find OmegaGMGS2.dls? I can only find the sf2 version of it.

It is converted from the sf2 version, using Extreme Sample Converter.

I've tested Airfont 340.dls, ~80 MiB from this page and the benefit is evident.
Please take a look to my modifications. I want to do more checks before a new release.

Please take a look to my modifications. I want to do more checks before a new release.

EAS's duplicated handle has its own position indicator, but as I know, dup()-ed file descriptors share the same I/O object (so file position is same).
Use dup() to implement DupHandle will not work on Linux (as I tested it), but according to CI it works on FreeBSD.

according to CI it works on FreeBSD and MacOS. Confused about how it works

CI is only proving that the code is compiling, and the unit tests are passing. But unit tests are only testing MIDI file processing without DLS files. Other manual tests with the sonivoxrender program show that FreeBSD reads correctly all DLS files, but macOS doesn't. What is needed:

Unit tests that include DLS. The main problem is having a free and reasonable sized DLS to be included in the repository for testing.
I am not happy with the special Linux implementation. There should be a single DupHandle implementation for all POSIX platforms.

The alternative right now is to enable at configuration time the new host wrapper for some platforms and the old wrapper for others. This doesn't feel right for a release.

I found that even passing ReadAt to the new HW, it is still much faster than the old one:
./sonivoxrender -d ~/OmegaGMGS2.dls  3.19s user 1.80s system 99% cpu 5.022 total

The alternative right now is to enable at configuration time the new host wrapper for some platforms and the old wrapper for others. This doesn't feel right for a release.

For these platforms, fallback to internal ReadAt in the new HW is okay. I will implement it in the next PR.

repo: pedrolcl/sonivox


text:
Guidance To Fine Tune BigBird for Sequence Classification for multiclass

Hi I am trying to implement the same notebook for bigbird fine tuning but I am confused with the regression layer and drop out what are changes the I need to make for classification. Your guidance will be a great help.
Thanks,
Abhishek Tomar

While I have not used BigBird before, generally a sigmoid activation function is used for binary classification and the softmax activation function is used for multiclass classification. You can try to replace the regression layer with either the sigmoid or softmax output layer (https://machinelearningmastery.com/choose-an-activation-function-for-deep-learning/).
Dropout is usually used for regularization, but it was omitted in my case since it did not improve results. But you may try it out with your model and dataset to see if dropout could work for you.


text:
🛑 API MERCADO PUBLICO is down

In 268684f, API MERCADO PUBLICO (https://api.mercadopublico.cl/servicios/v1/publico/ordenesdecompra.json?fecha=30122022&CodigoOrganismo=7312&ticket=E1951384-BF88-4EE5-8C33-934680281626) was down:

HTTP code: 0
Response time: 0 ms

Resolved: API MERCADO PUBLICO is back up in e2bf5e3 after 37 minutes.


text:
🛑 AVIS is down

In 74eb41b, AVIS (https://sscoquimbo.avislatam.com/modulos/login/index.cfm) was down:

HTTP code: 0
Response time: 0 ms

Resolved: AVIS is back up in 9bef22e.


text:
🛑 Beyond.pl is down

In 350e6fd, Beyond.pl (http://www.beyond.pl) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Beyond.pl is back up in 0f7f22f.

repo: pejotes/upptime


text:
🛑 Beyond.pl is down

In d13eb80, Beyond.pl (http://www.beyond.pl) was down:

HTTP code: 403
Response time: 686 ms

Resolved: Beyond.pl is back up in b209838 after 33 minutes.


text:
doc verifier token

I found necessary to pass the verifier token in the callback action, that was for fitbit
https://wiki.fitbit.com/display/API/OAuth+Authentication+in+the+Fitbit+API

Closing for no activity.

repo: pelle/oauth


text:
Patch callback doesn't seem to work

Here's a simplified example of it not working. I intended to use it for something like Redux middleware, but the patchCallback function is never called.

Looking at the tests revealed that it's called patchListener instead.

repo: pelotom/use-methods


text:
🛑 Dinas Tenaga Kerja is down

In b25034e, Dinas Tenaga Kerja (https://disnaker.bekasikota.go.id) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Dinas Tenaga Kerja is back up in 8ddb6a1.

repo: pemkotbekasi/website-status


text:
WIP: Various improvements

Don't merge until backend profile branch is merged first.
Implement subscription button (needs styling)

Add placeholder for empty profile fields

Fix button icons with correct coloring and spacing

Implement allowing club owners to specify schools/majors to target

Various small bug fixes
Implement better invitation page (with public toggling)

From the invitation page, can we make the name of the club a link to the club page?

Great idea, I'll implement that!

Just a few comments after those are resolved I think we are gtg. This PR looks very solid.

authors: cacabo ezwang

repo: pennlabs/clubs


text:
[BACKLOG-37149] Added error message forJava 8 with Windows 11 when op…

…ening Spoon UI. Also Added Java 8 swt.jar to maintain Java 8 compatibility on Windows 10

:white_check_mark: Build finished in 2m 54s
Build command:
mvn clean verify -B -e -Daudit -amd -pl \
assemblies/client,assemblies/core/client,assemblies/core/static,assemblies/static

:exclamation: No tests found!
:information_source:  This is an automatic message

authors: buildguy urena-kenny

repo: pentaho/pentaho-kettle


text:
Manually check that relaying works with new proof specs and new apphash

Following the changes to the AppHash and ProofSpecs made in #3390, we should manually check that relaying works to flag any changes early, before we finish pushing changes through the rest of the IBC stack and restoring interchaintest support.

xref https://github.com/penumbra-zone/hermes/pull/10 ; @avahowell said this is expected to work but isn't working because of an apparently unrelated rpc serialization issue.

Alternatively, it sounds like https://github.com/cosmos/relayer/pull/1355 could allow testing this using rly.

Yesterday we confirmed that IBC packets can be relayed from Penumbra Testnet 64 to Osmosis Testnet 5: https://github.com/penumbra-zone/penumbra/pull/3567 That packet was relayed via Hermes, the instance that PL runs. Separately, SL has restored go-relayer compat with Penumbra, as evidenced by the recent integration of Penumbra into interchaintest. :rainbow:

authors: conorsch hdevalence

repo: penumbra-zone/penumbra


text:
Failure to verify proof leaf hash leads to incorrect output proofs being accepted

Describe the bug
When updating to use the TCT, these lines:
https://github.com/penumbra-zone/penumbra/blob/41879a8cc5ac8683f49c20bf0f0b04a3b31c26f6/crypto/src/proofs/transparent.rs#L95-L123
were replaced with these lines:
https://github.com/penumbra-zone/penumbra/blob/0b2ba0231333b91fc79266c2509e8f05402033f9/crypto/src/proofs/transparent.rs#L92-L95
Although penumbra_tct::Proof::verify encapsulates a lot of the previous logic, it does not check that the Hash contained as the leaf of the Proof is the same as Hash::of the commitment we are trying to prove the inclusion of. This means that any inclusion proof will pass this check, not just one for the commitment in question.
This was discovered because of https://github.com/penumbra-zone/penumbra/issues/858: the sensibility check implemented in this assertion when producing the proof is the same check that must be made for soundness when verifying the proof; when the assertion was not flipped on (because it's turned off in release mode), the absence of any error meant that we had to be missing a verification step.
Steps to fix
Check that self.note_commitment_proof.commitment() == self.note_commitment, or (more appealingly) reduce the duplication and do not have a separate note_commitment field in the structure.

This seems relatively easy to fix and good to prioritize.

authors: hdevalence plaidfinch


text:
Implement worker scaffolding to enable parallelization

this work stream is well summarized by this comment. After completing this, we'll be able to parallelize the trial decryption since the polyfill now allows us to launch workers for individual sub-tasks, for instance transaction building.
while the polyfill uses offscreen, the rest of the services package wasn't launched that way and is still running in the background script in the service worker. In the present situation, everything — including all top-level tasks not launched with new Worker— runs on the main extension thread. However, after this phase, services no longer use chrome.runtime and could be moved into the offscreen entry in a separate phase.

closing in favor of https://github.com/penumbra-zone/web/issues/1787

repo: penumbra-zone/web


text:
Initial proposal for API design changes

Looking for feedback on implementation details
A couple notes:

Not all endpoints have been updated

See examples of create DB cluster and list DB clusters

Create DB cluster API endpoints returns HTTP 200 with an empty response

I would argue there's little use for the response to include the newly created object while it would hurt performance for API users who don't need this data
The user can always issue a GET request to get info about a DB cluster

I will try to provide an answer from general stand point, mostly rely on API usage convenience (actually the reason why API is created) and user experience.

Is the performance issue that big of an issue? Not sure about this. It seems to me like a premature optimization. On the other hand we have convenience of the API users who don't have to make an additional request to check everything is created as expected. Or in case of our frontend - to update the UI.

Performance is not the only concern. There's a couple other criteria as well:

Well, right now we don't have neither measures nor complains regarding performance. Without that it looks like a  premature optimization because we can't prove that:

performance really becomes better after such changes
the issue with performance is more important/valuable rather than user experience, so that we may decrease the API usage convenience level for the sake of performance improvement.

The response would be used by a negligible percentage of users.

We don't have any statistics to claim that.
I agree that this approach (omitting full resource in a response body) may be very beneficial in case the returned body  is completely the same as request that user sent. But in our case we have an opposite situation. Looking at CreateDatabaseCluster object schema:

there is only one mandatory part of parameters - engine. So it can be quite common situation that user sets mandatory param only.
in order to obtain the exact values that server decided to use for the rest of fields in DBCluster object user needs to send an additional request.

This complicates the user experience since he has to do an extra actions in order to achieve his goal.
On another hand - in case user doesn't need the response (such situations really take place) - he just ignores the body. No extra actions are required for that from user side.

You can always issue another GET request if you need the response

An additional request is ALWAY more expensive than returning the body (from user and server stand point):

user shall initiate a new request:
1.1 build request
1.2 initiate a new connection with TLS handshake. Yes, it is possible to re-use the existing connection, but this is not so widely used practise so far.
1.3 send request

server shall process the request:
2.1 authentication
2.1 authorization
2.3 validation
2.4 fetching the resource to return
2.5 send response

So if we are talking about general performance optimisation, we have:

1.1 - 1.3 + 2.1 - 2.5 actions in case of additional GET request
vs
2.4 - 2.5 actions for case of returning the resource in response.

Returning a response may require additional code which creates room for errors

there are actions to mitigate it: code review, tests, ...

I understand it may look as a no-brainer at first sight to return the response, but after analyzing the use cases and the percentage of users affected, it becomes more clear this brings only marginal benefits.

Right now we try to stick to general standards/guidelines/best practises for building API and product in general.
Returning the new object fits with the REST principle of "Uniform Interface - Manipulation of resources through representations." The complete object is the representation of the new state of the object that was created.
There is a really excellent reference for API design, here: Best Practices for Designing a Pragmatic RESTful API
To sum up:

I agree that this approach may take place
It decreases the level of API usage conviniese
We need to have a strong evidences that is it worth being applied here right now.

Later the situation may change and there will be a better time for considering it.
For now - let's postpone with adoption.

authors: maxkondr michal-kralik

repo: percona/percona-everest-backend


text:
GITHUB-ISSUE-1812: Fix azure storage restore by removing hardcoded trailing slash

Fixes #1812

Test name
Status

affinity-8-0
passed

auto-tuning-8-0
passed

cross-site-8-0
passed

demand-backup-cloud-8-0
passed

demand-backup-encrypted-with-tls-8-0
passed

demand-backup-8-0
passed

haproxy-5-7
passed

haproxy-8-0
passed

init-deploy-5-7
passed

init-deploy-8-0
passed

limits-8-0
passed

monitoring-2-0-8-0
passed

one-pod-5-7
passed

one-pod-8-0
passed

pitr-8-0
passed

pitr-gap-errors-8-0
passed

proxy-protocol-8-0
passed

proxysql-sidecar-res-limits-8-0
passed

pvc-resize-5-7
passed

pvc-resize-8-0
passed

recreate-8-0
passed

restore-to-encrypted-cluster-8-0
passed

scaling-proxysql-8-0
passed

scaling-8-0
passed

scheduled-backup-5-7
passed

scheduled-backup-8-0
passed

security-context-8-0
passed

smart-update1-8-0
passed

smart-update2-8-0
passed

storage-8-0
passed

tls-issue-cert-manager-ref-8-0
passed

tls-issue-cert-manager-8-0
passed

tls-issue-self-8-0
passed

upgrade-consistency-8-0
passed

upgrade-haproxy-5-7
passed

upgrade-haproxy-8-0
passed

upgrade-proxysql-5-7
failure

upgrade-proxysql-8-0
passed

users-5-7
passed

users-8-0
passed

validation-hook-8-0
passed

We run 41 out of 41

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1813/commits/cc72977361c042c57998cd48e0ed21eaaec95470
image: perconalab/percona-xtradb-cluster-operator:PR-1813-cc729773

Test name
Status

affinity-8-0
passed

auto-tuning-8-0
passed

cross-site-8-0
passed

demand-backup-cloud-8-0
passed

demand-backup-encrypted-with-tls-8-0
passed

demand-backup-8-0
passed

haproxy-5-7
passed

haproxy-8-0
passed

init-deploy-5-7
failure

init-deploy-8-0
passed

limits-8-0
passed

monitoring-2-0-8-0
passed

one-pod-5-7
passed

one-pod-8-0
passed

pitr-8-0
passed

pitr-gap-errors-8-0
passed

proxy-protocol-8-0
passed

proxysql-sidecar-res-limits-8-0
passed

pvc-resize-5-7
passed

pvc-resize-8-0
passed

recreate-8-0
passed

restore-to-encrypted-cluster-8-0
passed

scaling-proxysql-8-0
passed

scaling-8-0
passed

scheduled-backup-5-7
passed

scheduled-backup-8-0
passed

security-context-8-0
passed

smart-update1-8-0
passed

smart-update2-8-0
passed

storage-8-0
passed

tls-issue-cert-manager-ref-8-0
passed

tls-issue-cert-manager-8-0
passed

tls-issue-self-8-0
passed

upgrade-consistency-8-0
passed

upgrade-haproxy-5-7
passed

upgrade-haproxy-8-0
passed

upgrade-proxysql-5-7
passed

upgrade-proxysql-8-0
passed

users-5-7
passed

users-8-0
passed

validation-hook-8-0
passed

We run 41 out of 41

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1813/commits/44a71cfa967635e0804a5b9a3ccd700f479bf9ec
image: perconalab/percona-xtradb-cluster-operator:PR-1813-44a71cfa

Test name
Status

affinity-8-0
passed

auto-tuning-8-0
passed

cross-site-8-0
passed

demand-backup-cloud-8-0
passed

demand-backup-encrypted-with-tls-8-0
passed

demand-backup-8-0
passed

haproxy-5-7
passed

haproxy-8-0
passed

init-deploy-5-7
passed

init-deploy-8-0
passed

limits-8-0
passed

monitoring-2-0-8-0
passed

one-pod-5-7
passed

one-pod-8-0
passed

pitr-8-0
passed

pitr-gap-errors-8-0
passed

proxy-protocol-8-0
passed

proxysql-sidecar-res-limits-8-0
passed

pvc-resize-5-7
passed

pvc-resize-8-0
passed

recreate-8-0
passed

restore-to-encrypted-cluster-8-0
passed

scaling-proxysql-8-0
passed

scaling-8-0
passed

scheduled-backup-5-7
passed

scheduled-backup-8-0
passed

security-context-8-0
passed

smart-update1-8-0
passed

smart-update2-8-0
passed

storage-8-0
passed

tls-issue-cert-manager-ref-8-0
passed

tls-issue-cert-manager-8-0
passed

tls-issue-self-8-0
passed

upgrade-consistency-8-0
passed

upgrade-haproxy-5-7
passed

upgrade-haproxy-8-0
passed

upgrade-proxysql-5-7
passed

upgrade-proxysql-8-0
passed

users-5-7
passed

users-8-0
passed

validation-hook-8-0
passed

We run 41 out of 41

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1813/commits/44a71cfa967635e0804a5b9a3ccd700f479bf9ec
image: perconalab/percona-xtradb-cluster-operator:PR-1813-44a71cfa

Can we test this change on azure? @nmarukovich

Test name
Status

affinity-8-0
passed

auto-tuning-8-0
passed

cross-site-8-0
passed

demand-backup-cloud-8-0
passed

demand-backup-encrypted-with-tls-8-0
failure

demand-backup-8-0
failure

haproxy-5-7
passed

haproxy-8-0
passed

init-deploy-5-7
passed

init-deploy-8-0
passed

limits-8-0
passed

monitoring-2-0-8-0
failure

one-pod-5-7
passed

one-pod-8-0
passed

pitr-8-0
passed

pitr-gap-errors-8-0
passed

proxy-protocol-8-0
passed

proxysql-sidecar-res-limits-8-0
passed

pvc-resize-5-7
passed

pvc-resize-8-0
passed

recreate-8-0
passed

restore-to-encrypted-cluster-8-0
passed

scaling-proxysql-8-0
passed

scaling-8-0
failure

scheduled-backup-5-7
passed

scheduled-backup-8-0
passed

security-context-8-0
passed

smart-update1-8-0
passed

smart-update2-8-0
passed

storage-8-0
passed

tls-issue-cert-manager-ref-8-0
passed

tls-issue-cert-manager-8-0
passed

tls-issue-self-8-0
passed

upgrade-consistency-8-0
passed

upgrade-haproxy-5-7
passed

upgrade-haproxy-8-0
passed

upgrade-proxysql-5-7
failure

upgrade-proxysql-8-0
passed

users-5-7
passed

users-8-0
passed

validation-hook-8-0
passed

We run 41 out of 41

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1813/commits/874dda7728888262fabfc67c4195c7b9256cd1f3
image: perconalab/percona-xtradb-cluster-operator:PR-1813-874dda77

Test name
Status

affinity-8-0
passed

auto-tuning-8-0
passed

cross-site-8-0
passed

demand-backup-cloud-8-0
passed

demand-backup-encrypted-with-tls-8-0
passed

demand-backup-8-0
passed

haproxy-5-7
passed

haproxy-8-0
passed

init-deploy-5-7
passed

init-deploy-8-0
passed

limits-8-0
passed

monitoring-2-0-8-0
passed

one-pod-5-7
passed

one-pod-8-0
passed

pitr-8-0
passed

pitr-gap-errors-8-0
passed

proxy-protocol-8-0
passed

proxysql-sidecar-res-limits-8-0
passed

pvc-resize-5-7
passed

pvc-resize-8-0
passed

recreate-8-0
passed

restore-to-encrypted-cluster-8-0
passed

scaling-proxysql-8-0
passed

scaling-8-0
passed

scheduled-backup-5-7
passed

scheduled-backup-8-0
passed

security-context-8-0
passed

smart-update1-8-0
passed

smart-update2-8-0
passed

storage-8-0
passed

tls-issue-cert-manager-ref-8-0
passed

tls-issue-cert-manager-8-0
passed

tls-issue-self-8-0
passed

upgrade-consistency-8-0
passed

upgrade-haproxy-5-7
passed

upgrade-haproxy-8-0
passed

upgrade-proxysql-5-7
passed

upgrade-proxysql-8-0
passed

users-5-7
passed

users-8-0
passed

validation-hook-8-0
passed

We run 41 out of 41

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1813/commits/874dda7728888262fabfc67c4195c7b9256cd1f3
image: perconalab/percona-xtradb-cluster-operator:PR-1813-874dda77

Test name
Status

affinity-8-0
passed

auto-tuning-8-0
passed

cross-site-8-0
passed

demand-backup-cloud-8-0
passed

demand-backup-encrypted-with-tls-8-0
passed

demand-backup-8-0
passed

haproxy-5-7
passed

haproxy-8-0
passed

init-deploy-5-7
passed

init-deploy-8-0
passed

limits-8-0
passed

monitoring-2-0-8-0
passed

one-pod-5-7
passed

one-pod-8-0
passed

pitr-8-0
passed

pitr-gap-errors-8-0
passed

proxy-protocol-8-0
passed

proxysql-sidecar-res-limits-8-0
passed

pvc-resize-5-7
passed

pvc-resize-8-0
passed

recreate-8-0
passed

restore-to-encrypted-cluster-8-0
passed

scaling-proxysql-8-0
passed

scaling-8-0
passed

scheduled-backup-5-7
passed

scheduled-backup-8-0
passed

security-context-8-0
passed

smart-update1-8-0
failure

smart-update2-8-0
passed

storage-8-0
passed

tls-issue-cert-manager-ref-8-0
passed

tls-issue-cert-manager-8-0
passed

tls-issue-self-8-0
passed

upgrade-consistency-8-0
passed

upgrade-haproxy-5-7
passed

upgrade-haproxy-8-0
passed

upgrade-proxysql-5-7
failure

upgrade-proxysql-8-0
passed

users-5-7
passed

users-8-0
passed

validation-hook-8-0
passed

We run 41 out of 41

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1813/commits/b47e51a877e020817f27706832ed03cc8014ca67
image: perconalab/percona-xtradb-cluster-operator:PR-1813-b47e51a8

Test name
Status

affinity-8-0
passed

auto-tuning-8-0
passed

cross-site-8-0
passed

demand-backup-cloud-8-0
passed

demand-backup-encrypted-with-tls-8-0
passed

demand-backup-8-0
passed

haproxy-5-7
passed

haproxy-8-0
passed

init-deploy-5-7
passed

init-deploy-8-0
passed

limits-8-0
passed

monitoring-2-0-8-0
passed

one-pod-5-7
passed

one-pod-8-0
passed

pitr-8-0
passed

pitr-gap-errors-8-0
failure

proxy-protocol-8-0
passed

proxysql-sidecar-res-limits-8-0
passed

pvc-resize-5-7
passed

pvc-resize-8-0
passed

recreate-8-0
passed

restore-to-encrypted-cluster-8-0
passed

scaling-proxysql-8-0
passed

scaling-8-0
passed

scheduled-backup-5-7
passed

scheduled-backup-8-0
passed

security-context-8-0
passed

smart-update1-8-0
passed

smart-update2-8-0
passed

storage-8-0
passed

tls-issue-cert-manager-ref-8-0
passed

tls-issue-cert-manager-8-0
passed

tls-issue-self-8-0
passed

upgrade-consistency-8-0
passed

upgrade-haproxy-5-7
passed

upgrade-haproxy-8-0
passed

upgrade-proxysql-5-7
passed

upgrade-proxysql-8-0
failure

users-5-7
failure

users-8-0
passed

validation-hook-8-0
passed

We run 41 out of 41

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1813/commits/7aabd61239269885db257c554d6552ad1ccab6fe
image: perconalab/percona-xtradb-cluster-operator:PR-1813-7aabd612

Test name
Status

affinity-8-0
passed

auto-tuning-8-0
passed

cross-site-8-0
passed

demand-backup-cloud-8-0
passed

demand-backup-encrypted-with-tls-8-0
passed

demand-backup-8-0
passed

haproxy-5-7
passed

haproxy-8-0
passed

init-deploy-5-7
passed

init-deploy-8-0
passed

limits-8-0
passed

monitoring-2-0-8-0
passed

one-pod-5-7
passed

one-pod-8-0
passed

pitr-8-0
passed

pitr-gap-errors-8-0
passed

proxy-protocol-8-0
passed

proxysql-sidecar-res-limits-8-0
passed

pvc-resize-5-7
passed

pvc-resize-8-0
passed

recreate-8-0
passed

restore-to-encrypted-cluster-8-0
passed

scaling-proxysql-8-0
passed

scaling-8-0
passed

scheduled-backup-5-7
passed

scheduled-backup-8-0
passed

security-context-8-0
passed

smart-update1-8-0
passed

smart-update2-8-0
passed

storage-8-0
passed

tls-issue-cert-manager-ref-8-0
passed

tls-issue-cert-manager-8-0
passed

tls-issue-self-8-0
passed

upgrade-consistency-8-0
passed

upgrade-haproxy-5-7
passed

upgrade-haproxy-8-0
passed

upgrade-proxysql-5-7
passed

upgrade-proxysql-8-0
passed

users-5-7
passed

users-8-0
passed

validation-hook-8-0
passed

We run 41 out of 41

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1813/commits/7aabd61239269885db257c554d6552ad1ccab6fe
image: perconalab/percona-xtradb-cluster-operator:PR-1813-7aabd612

@dcaputo-harmoni, thank you for your contribution.

authors: JNKPercona dcaputo-harmoni egegunes hors

repo: percona/percona-xtradb-cluster-operator


text:
AndroidX

migration to AndroidX
right now i cannot build due to some maven publishing taks which are not working on my machine (or due to the new gradle plugin..)

I've fixed the mentioned issues (imports and versioning) and fixed the travis build by copying the licences to $ANDROID_HOME/licences
It is failing now due to the publishing plugin. I don't know how you usually deal with this.

Might be related to https://github.com/novoda/bintray-release could you try to update?

no luck...bintray-release does not seem to support gradle-4.6 but the android-gradle plugin for AndroidX should be the 3.2.0-alpha*
https://github.com/novoda/bintray-release/issues/177

Umm.. 🤔
I found kind of official plugin and seems there is a PR for that but haven't merged yet.
https://github.com/bintray/gradle-bintray-plugin/pull/230

Unfortunately I have no resources atm to look into bintray (have not yet deployed any artefacts to any kind of artifactory :-)).

Umm ok, it would be actually a huge change so it might be better to be addressed by @permissions-dispatcher/maintainers 🤔Really sorry to say that but this is actually a great work!

Let me take over this work.

@forkch if you don't mind, could I change your author mail address to your GitHub email address? This change lead to associate your commit with your GitHub account.

Sure no problem!

Done. See #494
I will close this PR since all @forkch works are there. Thank you for your contribution!

Thank you @forkch !

authors: forkch hotchemi shiraji


text:
Add matchedRoute field to Request object

Currently on a Spark request you are able to retrieve the requested URI or path. However, there's no way to work backwards from the URI to find out which route it was mapped to. I should mention that this is specifically only an issue when parameters are involved. I can see that the request's pathInfo is "/users/bob" but I can't pull the matched route of "/users/:username".
For context, we want to inspect requests in the after() block and instrument them by route.
This is a pretty tiny change so if I get approval I'd be happy to submit a PR. Thanks ya'll. :)

This would also resolve #959

@tipsy do you have an idea of when @perwendel will be back? it seems he's been gone almost 3 months

Sorry @robax, I don't really know.

This has been merged now!

Thanks you @tipsy !

Thank you!

authors: adriancole jon-ruckwood robax tipsy

repo: perwendel/spark


text:
Http2 support

Http2 support added. To be able to use http2 pay attention to add right version of alnp library to boot classpath.

@gimlet2 Do you know in which version of http2 will be available?

This is great. @gimlet2, could you update this?

@tipsy, updated

To be able to use http2 pay attention to add right version of alnp library to boot classpath.
How can I do this?

Can you add some tests?

@tipsy  Are we just missing tests?  Is that the only thing blocking this from delivery?

@tdalbo92 From what I can see, yes. But @perwendel needs to merge this one.

@tipsy beautiful, thanks for the update.  I'm interested in getting this in as part of a new project I'm writing.
If there's help needed with the tests, I can give it my best to write some.

any news on that? Would love to use http2 instead of websockets !

I'm baffled by this.
Doesn't look like anything complicated by reading the pull request changes, and yet after 3 years still here it is.

Any updates?

Any progress on getting in http2 support done.

I did a new PR from where gimlet2 left, added the tests and updated the code so it works with the current Jetty version and also added support for HTTP 2 clear protocol https://github.com/perwendel/spark/pull/1183

authors: blahmutz camdenorrb gimlet2 gorros gsivamal luneo7 raffaeleragni tdalbo92 tipsy


text:
Mismatched gem dependencies on tty-screen

Hi! Thanks for providing this gem. I wanted to let you know that c3e2221d causes problems when installing the tty gem, producing output like the following:
ERROR:  While executing gem ... (Gem::DependencyError)
    Unable to resolve dependencies: tty requires tty-screen (~> 0.1.0); tty-progressbar requires tty-screen (~> 0.2.0); tty-table requires tty-screen (~> 0.1)

Is it possible to get them all in sync? I can also resolve this by updating my gem file to require a previous version of tty, but I wasn't sure if you'd seen this error.

Hi Brian! I will resolve this issue by bringing all dependencies in sync and release new tty version - hold tight.

Sorry forgotten to let you know about the release. I've released tty version 0.2.0 that addresses this issue. Give it a try and let me know if it works for you.

authors: floatplane peter-murach

repo: peter-murach/tty-progressbar


text:
Switch to inheriting from Rack's Abstract::PersistedSecure

This update changes the Dalli session store implementation to inherit from Abstract::PersistedSecure rather than Abstract::Persisted.  Effectively this change decouples the public id value (the one placed in the cookie) from the key used for internal lookups in memcached.
This should fix #765 .
It is also likely a breaking change, as it will invalidate any outstanding sessions.

@GCorbel I know it's been a while since you reported #765 , but if you're still around and so inclined it would be helpful to confirm that this fix addresses your issue.  Thanks.

Hi, I give it a try on a Rails 6.0.4.1 application. I just changed to use :
gem 'dalli', github: 'petergoldstein/dalli', branch: 'feature/switch_to_persisted_secure'

And added session[:session_id] in a controller and I still have NoMethodError: undefined method public_id' for "13ef84661f6d4cf846edfe8ac5a7fd03":String`. I checked, and the application really use your code.

Thanks @GCorbel .  Any chance you'd be able to post your test app in a repo?  That would make it easier to nail this down.

@GCorbel You can ignore my last request.  I believe I found the issue and so I don't need a test app.  If you could rerun your test with the latest code from this PR, I'd appreciate it.  Thanks.

I confirm it works. session.id.class gives Rack::Session::SessionId and session[:session_id] gives the correct id.

@GCorbel Great.  I'll merge and release (probably as 3.1.0) soon.

authors: GCorbel petergoldstein

repo: petergoldstein/dalli


text:
Implemented Binary and Display traits for FixedBitSet.

Providing some implementation for #34. Need feedback.

Looks good

Cool, thanks!
Let's be open to tweaks to the format in the future. Maybe _ separators every 16 or 32 bits would be good to keep it readable.

authors: Dolphindalt bluss

repo: petgraph/fixedbitset


text:
User-reported issues don't trigger notification

Describe the bug
When a user reports an issue on a media item, no Telegram notification is sent.

The notification system for Telegram isn't hooked up to issues, issues are currently only sent to email. We're in the process of rebuilding our backend api and this is a valid enhancement but not a bug.

authors: AshDyson djdembeck

repo: petio-team/petio


text:
Made "More Reading" a header so it can be easily linked to.

I recommend this reading list to people trying to learn promises a lot.

closing 2.0 pr, please retarget to 2.x branch if still applicable

authors: airportyh petkaantonov

repo: petkaantonov/bluebird


text:
Fix login to do without GALX

Google is no longer serving GALX on the login page.  The fix is simple: just
remove GALX.

Good find.  GALX should also be removed from lines 84, 85, and 89.

Thanks, working !

Thanks, chris001.  I fixed 84, 85, and 89 as well.

authors: Sfinx chris001 gkuenning

repo: pettazz/pygooglevoice


text:
Verify that "Pătrângeni" is listed in the dropdown menu when searched

Description:
"Pătrângeni" should appear in the dropdown menu with correct spelling and locality information.
Preconditions:
The website is up and running.

Step 1
Type "Pătrângeni" in the search bar.
Expected results:
The dropdown is listing "Pătrângeni" as the first location in the dropdown menu.

Step 2
Observe the dropdown menu.
Expected results:
"Pătrângeni" is displayed and selectable from the dropdown menu.


text:
"Chirnogi" is listed in the drop-down menu

Description:
"Chirnogi" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Chirnogi" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Chirnogi, CĂLĂRAȘI (Ulmu)".


text:
"Glod", "Lunca" are listed in the drop-down menu

Description:
When user writes ”Glod” and "Lunca" the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Lunca"

Expected results
The location appears in the drop-down menu as "Sat Lunca, DÂMBOVIȚA (Moroeni)".


text:
The cursor disappears when you search for a job

URL: https://admin.peviitor.ro/
Browser: Chrome
Device: Laptop
OS: Windows 11
STEPS TO REPRODUCE:

Open the URL in the browser.
Search for "Prohuman" company.
Click on "Vizualizeaza joburi".
Search for "russian"

EXPECTED RESULTS:
The search field should be active.
ACTUAL RESULTS:
The search field is not active.
https://github.com/user-attachments/assets/7f3dd3c7-20ad-485b-b367-6b0eebc25d5f

Fixed...
Cautarea nu se mai face automat ci dupa apasarea butonului cauta sau apasarea tastei enter

repo: peviitor-ro/validator-ui


text:
is it a small fault in batch_dice

in the medicaldetectiontoolkit/utils/model_utils.py, lines 883, batch_dice loss function,
I think should add
if y[:, 1:, ...].max() == 0:
y = 1 - y
pred = 1 - pred
for the patch if it has no roi.

in the medicaldetectiontoolkit/utils/model_utils.py, lines 883, batch_dice loss function,
I think should add
if y[:, 1:, ...].max() == 0:
y = 1 - y
pred = 1 - pred
for the patch if it has no roi.

repo: pfjaeger/medicaldetectiontoolkit


text:
Swap argument positions in all examples

This PR fixes #274.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #275   +/-   ##
=======================================
  Coverage   82.24%   82.24%
=======================================
  Files         116      116
  Lines        5834     5834
=======================================
  Hits         4798     4798
  Misses       1036     1036

authors: codecov-io mihaimorariu

repo: pfnet-research/chainer-chemistry


text:
Support fmod=1 in Mod op

rel: https://github.com/chainer/chainer/pull/8110

/test

repo: pfnet-research/chainer-compiler


text:
Fix x-in-ipv6 tunnels for gre and gif

It needs to do "ifconfig inet6 tunnel xxxx" instead of "ifconfig tunnel xxxx" if the remote address is an ipv6 address.
Tested and working on my production router.

Before this pull request can be accepted you must first sign a CLA as described at https://www.pfsense.org/about-pfsense/#cla. Please read for more details.

CLA sent, not interested in any attribution, just want this fixed for my own router. Code is public domain.

repo: pfsense/pfsense


text:
Add OnAppV4 record update as Dynamic DNS Client

https://docs.onapp.com/display/40API/Edit+DNS+Records

Before this pull request can be accepted you must first sign a CLA as described at https://www.pfsense.org/about-pfsense/#cla. Please read for more details.

Sorry, I have just done it.

Please add OnApp item to header at:
https://github.com/pfsense/pfsense/blob/master/src/etc/inc/dyndns.class#L59

Did it, thank you

@TroyDesign can you please rebase your fork? It cannot be automatically merged anymore

No response to rebase request in six months

authors: TroyDesign pfSense-git-updates rbgarga sbeaver-netgate


text:
Is this the right way with using resolvers on a type?

I have the following type which contains:
 type<VehicleResponse> {
            description = "Gadget returned from original GadgetProxy"

            property(VehicleResponse::gadget) {
                description = "Integer value representing type of vehicle"
            }

            property<InventoryParts?>("inventory") {
                resolver { response, fromDate: LocalDate ->
                    getInventoryParts(response.gadget.stockNumber, fromDate)
                }
            }
        }

I also have :
 query("getInventory") {
            description = "Returns the parts from the inventory."
            resolver { stockNumber: String, fromDate: LocalDate ->
                getInventoryParts(stockNumber, fromDate)
            }.withArgs {
                arg<String> { name = "fromDate"; description = "Date to check inventory stock" }
            }
        }

As you can see there is a bit of duplication (getInventoryParts). So my question is am I using the resolver correctly when defining the schema or can I some how from VehicleResponse go to query("getInventory") rather than creating a resolver in my VehicleResponse type?

This looks correct. All your logic would be in the getInventoryParts function, so calling it from different places is perfectly fine.

Okay, thank you so much.

authors: jeggy shamresh

repo: pgutkowski/KGraphQL


text:
🛑 proxy index is down

In 4fb0656, proxy index (https://index.phantomic.live/) was down:

HTTP code: 404
Response time: 141 ms

Resolved: proxy index is back up in 2ca1683 after 1 hour, 8 minutes.


text:
Package fails to install under R 4.0

According to its DESCRIPTION, the package requires R 4.0. However, the following line uses the native pipe operator, which is only available from R 4.1 onwards:
https://github.com/pharmaverse/admiraldev/blob/86c007a71a99a417ac9b8dbbc36672afc0e8a1b0/R/process_set_values_to.R#L96

Ohhh...goodspot @klmr !!
@pharmaverse/admiral and @pharmaverse/admiral_comm
ummm....how should we approach:

Remove native pipe and update to magrittr pipe
Require admiral/admiraldev to only work with 4.1  and keep native pipe?

Ohhh...goodspot @klmr !!
@pharmaverse/admiral and @pharmaverse/admiral_comm
ummm....how should we approach:

Remove native pipe and update to magrittr pipe
Require admiral/admiraldev to only work with 4.1  and keep native pipe?

I'd vote for the second option - R 4.1 is quite old anyway.

So should we handle this as an official bug fix with a re-release?

So should we handle this as an official bug fix with a re-release?

I don't think so. The min R version is something that is not correct is many many many pkgs on CRAN. I would just fix in dev, and it'll be correct in the next release

Could we add a CI/CD workflow which tries to install and test the package with the minimum versions stated in DESCRIPTION?

Perhaps we can just say somewhere on the website we support the last 2, 3, 4 (whatever we choose) versions of R, and supplement our workflows with the R CMD Check from Posit that makes it easy to check the last n versions of R.
          - {os: ubuntu-latest,   r: 'devel', http-user-agent: 'release'}
          - {os: ubuntu-latest,   r: 'release'}
          - {os: ubuntu-latest,   r: 'oldrel-1'}
          - {os: ubuntu-latest,   r: 'oldrel-2'}
          - {os: ubuntu-latest,   r: 'oldrel-3'}
          - {os: ubuntu-latest,   r: 'oldrel-4'}

We say we develop using the earliest of the last 3 versions of R, in documentation:
https://pharmaverse.github.io/admiraldev/articles/programming_strategy.html#r-and-package-versions-for-development

I believe we are testing 4.4, 4.3, 4.2 with our CI actions.  We should add 4.1 to mix.

We already have the Posit workflow file here, and we can just uncomment the lines.
https://github.com/pharmaverse/admiraldev/blob/86c007a71a99a417ac9b8dbbc36672afc0e8a1b0/.github/workflows/R-CMD-check.yaml#L32

ah i see - ummm... can we re-work the admiralci one since it is used in admiral as well as well as the other admiral packages.  Just be nice to all pull from the same spot.  I'm totally fine to simplify the one in admiralci!

We say we develop using the earliest of the last 3 versions of R, in documentation: https://pharmaverse.github.io/admiraldev/articles/programming_strategy.html#r-and-package-versions-for-development

There it says

Currently we test admiral against the three latest R Versions and the closest snapshots of packages to those R versions.

Is the part "and the closest snapshots of packages to those R versions" up to date? I remember vaguely that we changed our strategy to use the latest version of the packages for all R versions.

I believe we are testing 4.4, 4.3, 4.2 with our CI actions. We should add 4.1 to mix.

I think testing more R versions doesn't solve the issue. If we had tested with 4.1, we still wouldn't have discovered the problem.
If we can add a workflow which tests the package on the minimum R version and minimum package versions as specified in the DESCRIPTION file, it would be helpful. @cicdguy , is this possible?
Otherwise, I would rely on the developers and users that they report wrong minimum versions.

I believe we are testing 4.4, 4.3, 4.2 with our CI actions. We should add 4.1 to mix.

I think testing more R versions doesn't solve the issue. If we had tested with 4.1, we still wouldn't have discovered the problem.
If we can add a workflow which tests the package on the minimum R version and minimum package versions as specified in the DESCRIPTION file, it would be helpful. @cicdguy , is this possible?
Otherwise, I would rely on the developers and users that they report wrong minimum versions.

My understanding is that we are saying in our current DESCRIPTION that admiral works with R version 4.0.  However, we are using a pipe |> that was introduced in R version 4.1 in a few lines of code
If we had been testing the package build on 4.0 I believe we would of found this issue out.  Maybe we should just give a try to experiment.
Anyways, I do think the minimum version we suggest in the DESCRIPTION should be tested no matter what! :)

authors: bms63 bundfussr ddsjoberg klmr manciniedoardo millerg23

repo: pharmaverse/admiraldev


text:
Create derive_var_bvcacritxfl function

Background Information
Create function derive_var_bcvacritxfl which can create criterion flags for CBVA gain/loss of x letters. Functionality could be as follows:
derive_var_bcvacritxfl <- function(
  ...
  gain_values,
  lose_values,
  ....
){
<function code>
}

which could be called like this:
adbcva <- adbcva %>%
    derive_var_bcvacritxfl(
        gain_values = c(10,15),
        lose_values = c(5,10)
     )

to generate an updated ADBCVA dataset containing criterion flags CRIT1FL, CRIT2FL, CRIT3FL, CRIT4FL for gaining 10 letters, gaining 15 letters, losing 5 letter and losing 10 letters respectively, along with associated variables CRIT1, CRIT2, CRIT3, CRIT4 encoding the criterion for the flag (eg, CRIT1 = "10 letter BCVA gain" for BCVA records and blank otherwise).
Proposed convention:

CRITxFL with 1<=x<=10 used for BCVA gain categories
CRITxFL with 11<=x<20 used for BCVA loss categories

This will be quite a central function in {admiralophtha}, so we should aim to provide:

many examples in the function roxygen code
vignette
example in the ADBCVA template.

Note: we should also add avoidloss_values and avoidgain_values corresponding to avoiding a loss/gain of x letters. An appropriate convention can also be considered like the one suggested above.

Sorry for my delay coding. I missed push my branch so copy my code here.
I have some questions.
i) Do gain_values and loss _values always have 2 vectors? If not, I need to modify code which.
ii) Sorry for I am not famillier with ADBCVA, how to define gain_values or loss _values. Do we usulally use minus numbers?
I will confirm with our in-house ophthalmologist tomorrow, and will modify.
`derived_var_bvcacritxfl <- function(
adbcva_1,
x,
gain_values,
lose_values
){
adbcva_1 %>%
mutate(CRIT1FL = ifelse((!!as.name(x) >= gain_values[1] & !!as.name(x) < gain_values[2]),
paste(gain_values[1], " letter BCVA gain"), "" )) %>%
mutate(CRIT2FL = ifelse((!!as.name(x) >= gain_values[2]),
paste(gain_values[2], " letter BCVA gain"), "" )) %>%
mutate(CRIT3FL = ifelse((!!as.name(x) <= -(lose_values[1] & !!as.name(x) < -(lose_values[2])),
paste(lose_values[1], " letter BCVA loss"), "" )) %>%
mutate(CRIT4FL = ifelse((!!as.name(x) <= -(lose_values[2])),
paste(lose_values[2], " letter BCVA loss"), "" ))
}`

Thanks for this Yuki. A couple of notes:

We really should be working in a separate branch, so I will make a new branch and copy the code across.
We should be following the programming strategy listed here, which has info about variable names and formatting.
I think it would make sense to loop over gain_values and lose_values rather than having many mutate statements.

I am happy to implement the above myself and then circle back with you to show you what I've done and how I've done it 😄

repo: pharmaverse/admiralophtha


text:
Incorrect description for GameClockUpdatedSeconds

I think there might be a copy/paste error in the payload for https://github.com/pharuxtan/sos-emulator/blob/322820218773c0ca1ae4a5a28a5d634b04c98a46/src/Components/Payloads/GameClockUpdatedSeconds.svelte#L13
which is the same description as
https://github.com/pharuxtan/sos-emulator/blob/322820218773c0ca1ae4a5a28a5d634b04c98a46/src/Components/Payloads/GameRoundStartedGo.svelte#L13
Perhaps a better description could be "This payload is sent when the seconds of the clock are updated"?

Yup, thanks for notice that 😄
I'll change that now, if you notice any other typo feel free to tell me

authors: johanalkstal pharuxtan

repo: pharuxtan/sos-emulator


text:
Session object should be extensibie

motivation
When user auth delegate to outer api server, phenyl authentication want to save outer-api-server-auth-token.
image Code
class MemberUserDefinition<ExtraSession> extends StandardUserDefinition {

  async authentication(): Promise {
    const result = await authPhenylAndOtherApiServer()
    const {  expiredAt, entityName, userId, versionId } = result // form phenyl
    const extraSession :ExtraSession = result.outerServerToken // fromOuterApiServer
    const preSession = { expiredAt, entityName, userId, extraSession}
        return {
          ok: 1,
          preSession,
          user,
          versionId,
        }
      }
  }
 // and able to get session.extraSession

You mean, "session object should be extensibie". Yes, it's already extensible but type problem. We'll fix the type problem.

ok, sorry for my poor English

I realized that $Subtype<Session> is the solution for this.

In typescript branch, Session will be Session<T> where T is extended properties attached to Session. Currently, some codes haven't used Session<T> yet. We should replace all the Sessions into Session<T> gradually.

authors: KONDO-Yuuki shinout

repo: phenyl-js/phenyl


text:
Unreachable code line in normalize method of Vector4.js

As part of the work for DOT (see #49), I noticed this in Vector4.js,

    /**
     * Normalizes this vector (rescales to where the magnitude is 1), changing this vector.
     * @public
     *
     * This is the mutable form of the function normalized(). This will mutate (change) this vector, in addition to
     * returning this vector itself.
     *
     * @returns {Vector4}
     */
    normalize: function() {
      var mag = this.magnitude();
      if ( mag === 0 ) {
        throw new Error( 'Cannot normalize a zero-magnitude vector' );
      }
      else {
        return this.divideScalar( mag );
      }
      return this;
    }
  } );

```

Am I not correct in saying that the "return this" line is redundant because it can never be called?

Assigning to @jonathanolson  for review

I recommend to write it like so:
      var magnitude = this.magnitude();
      assert && assert( magnitude !== 0, 'Cannot normalize a zero-magnitude vector' );
      return this.divideScalar( magnitude );

If this seems good to you, go for it @phetsims/martin!

Also, @veillette, I want to make sure you are aware that dot has 1420 unit tests--may be useful as you work on dot.
http://localhost/dot/tests/qunit/unit-tests.html

The normalize function was written as suggested by samreid

authors: samreid veillette

repo: phetsims/dot


text:
Also support Foggit WH3000?

My Weatherstation Foggit WH3000 lao has the port 45000 open and Sso I tried the adapter :-)
Result is that he can read out the software but is not reading more:-( Any chance to?
Debug Log:
2020-06-20 21:43:18.692  - info: sainlogic.0 (9208) starting. Version 0.6.2 in /opt/iobroker/node_modules/iobroker.sainlogic, node: v14.4.0, js-controller: 3.1.4
2020-06-20 21:43:18.708  - info: sainlogic.0 (9208) Starting Scheduler
2020-06-20 21:43:18.709  - info: sainlogic.0 (9208) Scheduler call ws_getfirmware activated
2020-06-20 21:43:18.709  - info: sainlogic.0 (9208) Scheduler call ws_getcurrent activated
2020-06-20 21:43:18.709  - info: sainlogic.0 (9208) Scheduler call ws_getmax activated
2020-06-20 21:43:18.709  - info: sainlogic.0 (9208) Scheduler call ws_getmin activated
2020-06-20 21:43:18.709  - info: sainlogic.0 (9208) Scheduler call ws_getmaxdaily activated
2020-06-20 21:43:18.710  - info: sainlogic.0 (9208) Scheduler call ws_getmindaily activated
2020-06-20 21:43:18.710  - debug: sainlogic.0 (9208) Weatherstation IP: 192.168.178.58
2020-06-20 21:43:18.710  - debug: sainlogic.0 (9208) Weatherstation port: 45000
2020-06-20 21:43:18.710  - debug: sainlogic.0 (9208) Scheduler Interval: 15

2020-06-20 21:43:33.711  - info: sainlogic.0 (9208) Scheduler pulling for new data
2020-06-20 21:43:33.736  - debug: sainlogic.0 (9208) Scheduler connected to weather station run 255,255,80,3,83
2020-06-20 21:43:33.743  - debug: sainlogic.0 (9208) FW Scheduler Received data string: ffff501511456173795765617468657256312e312e34
2020-06-20 21:43:33.743  - debug: sainlogic.0 (9208) Data Command received: 80 subcommand 69
2020-06-20 21:43:33.744  - debug: sainlogic.0 (9208) Data object: {"softwaretype":"EasyWeatherV1.1.4"}
2020-06-20 21:43:33.744  - info: sainlogic.0 (9208) Scheduler updating IOBroker states
2020-06-20 21:43:33.745  - debug: sainlogic.0 (9208) Setting value from data for info.softwaretype to EasyWeatherV1.1.4
2020-06-20 21:43:33.746  - debug: sainlogic.0 (9208) Scheduler connected to weather station run 255,255,11,0,6,4,4,25

2020-06-20 21:43:48.711  - info: sainlogic.0 (9208) Scheduler pulling for new data
2020-06-20 21:43:48.711  - debug: sainlogic.0 (9208) Scheduler connected to weather station run 255,255,80,3,83
2020-06-20 21:43:48.722  - debug: sainlogic.0 (9208) FW Scheduler Received data string: ffff501511456173795765617468657256312e312e34
2020-06-20 21:43:48.722  - debug: sainlogic.0 (9208) Data Command received: 80 subcommand 69
2020-06-20 21:43:48.722  - debug: sainlogic.0 (9208) Data object: {"softwaretype":"EasyWeatherV1.1.4"}
2020-06-20 21:43:48.722  - info: sainlogic.0 (9208) Scheduler updating IOBroker states
2020-06-20 21:43:48.723  - debug: sainlogic.0 (9208) Setting value from data for info.softwaretype to EasyWeatherV1.1.4
2020-06-20 21:43:48.723  - debug: sainlogic.0 (9208) Scheduler connected to weather station run 255,255,11,0,6,4,4,25

2020-06-20 21:44:03.711  - info: sainlogic.0 (9208) Scheduler pulling for new data
2020-06-20 21:44:03.712  - error: sainlogic.0 (9208) Connection error on 192.168.178.58:45000: Error: connect EISCONN 192.168.178.58:45000 - Local (192.168.178.79:46968)
2020-06-20 21:44:03.713  - debug: sainlogic.0 (9208) FW Scheduler Connection closed

2020-06-20 21:44:18.711  - info: sainlogic.0 (9208) Scheduler pulling for new data
2020-06-20 21:44:18.721  - debug: sainlogic.0 (9208) Scheduler connected to weather station run 255,255,80,3,83
2020-06-20 21:44:18.722  - debug: sainlogic.0 (9208) Scheduler connected to weather station run 255,255,80,3,83
2020-06-20 21:44:18.737  - debug: sainlogic.0 (9208) FW Scheduler Received data string: ffff501511456173795765617468657256312e312e34
2020-06-20 21:44:18.737  - debug: sainlogic.0 (9208) Data Command received: 80 subcommand 69
2020-06-20 21:44:18.737  - debug: sainlogic.0 (9208) Data object: {"softwaretype":"EasyWeatherV1.1.4"}
2020-06-20 21:44:18.737  - info: sainlogic.0 (9208) Scheduler updating IOBroker states
2020-06-20 21:44:18.738  - debug: sainlogic.0 (9208) Setting value from data for info.softwaretype to EasyWeatherV1.1.4
2020-06-20 21:44:18.738  - debug: sainlogic.0 (9208) Scheduler connected to weather station run 255,255,11,0,6,4,4,25
2020-06-20 21:44:18.790  - debug: sainlogic.0 (9208) FW Scheduler Received data string: ffff501511456173795765617468657256312e312e34
2020-06-20 21:44:18.790  - debug: sainlogic.0 (9208) Data Command received: 80 subcommand 69
2020-06-20 21:44:18.790  - debug: sainlogic.0 (9208) Data object: {"softwaretype":"EasyWeatherV1.1.4"}
2020-06-20 21:44:18.791  - info: sainlogic.0 (9208) Scheduler updating IOBroker states
2020-06-20 21:44:18.791  - debug: sainlogic.0 (9208) Setting value from data for info.softwaretype to EasyWeatherV1.1.4
2020-06-20 21:44:18.791  - debug: sainlogic.0 (9208) Scheduler connected to weather station run 255,255,11,0,6,5,5,27
2020-06-20 21:44:19.175  - debug: host.io-test.compactgroup1 object change system.adapter.sainlogic.0 (from: system.adapter.admin.0)
2020-06-20 21:44:19.175  - debug: host.io-test object change system.adapter.sainlogic.0 (from: system.adapter.admin.0)

BTW: When I see that log correct then something is strange because sometimes the connection not seems to be closed  (at least log missing).

This seems a bit weird and indeed a similar pattern as with the Eurochron EFWS2900. The get-firmware command returns just fine, other commands do not return anything, not even an error. Hence i keep the connection open waiting on an answer :/
If someone finds our which commands these stations need I can surely adjust this in the adapter. But at this point it is difficult for me.
Does the WeatherSmartIP app on windows for your station? I would expect this to have similar issues. Might also be that your firmware is just to old. Mine lists EasyWeatherV1.5.0. Can you check if an update is possible using WSView on your phone?

wstool is finding the device but not offers any fimrware updates. it say firmware 1.1.4 is current one. wsTool is the "official" tool Frogitt tells about. It sensd my data to WU at the moment. configured via wsTool (and I also can add Weathercloud or weatherobservations
WsView (i tested now) and offers me to update firmware to 1.5.0 ... hm... :-) Should I do it? ;-)

Ok, I do the update :-) fingers crossed :-)

Hm no luck ... it says "Downloading ..." for 2-3 mins on the phone, and then "update fail" and thats all.
I also tried restarting the display ... no luck ... same again. I think wstool is doing something different ten WSView :-(

Do you have a download URL for the weather station PC software

I have written to Froggit and asked for an update ... lets see

The WeatherStation IP is available via ELV: https://files2.elv.com/public/25/2504/250408/Internet/ws980wifi_weathersmartip_v1_1_0426.zip
Let me know what Froggit's answer is.

Ok, Software installed, find the device but also shows no values ...

Alright, kind of expected.
So let's wait for Froggit - otherwise it seems your station is more similar to the Eurocron and does not reply to the commands :/
Maybe it is different command strings that it requires. If you are curious and adventurious enough you can check the lib/constants.js, all commands are listed their in hex code. :D

Very Interesting. No answer from them but today I just tried again and it downloaded and updated successfully via WSTool. Now it is listed there. Will try adapter later ;-) seems that the download of firmware did not worked.

Ok, so it now has the 1.5.0FW but still:
2020-07-03 22:08:14.850  - debug: sainlogic.0 (3088) Weatherstation IP: 192.168.178.58
2020-07-03 22:08:14.850  - debug: sainlogic.0 (3088) Weatherstation port: 45000
2020-07-03 22:08:14.850  - debug: sainlogic.0 (3088) Scheduler Interval: 15
2020-07-03 22:08:28.895  - info: influxdb.0 (2108) Store 301 buffered influxDB history points
2020-07-03 22:08:29.851  - info: sainlogic.0 (3088) Scheduler pulling for new data
2020-07-03 22:08:29.927  - debug: sainlogic.0 (3088) Scheduler connected to weather station run 255,255,80,3,83
2020-07-03 22:08:29.932  - debug: sainlogic.0 (3088) FW Scheduler Received data string: ffff501511456173795765617468657256312e352e30
2020-07-03 22:08:29.933  - debug: sainlogic.0 (3088) Data Command received: 80 subcommand 69
2020-07-03 22:08:29.934  - debug: sainlogic.0 (3088) Data object: {"softwaretype":"EasyWeatherV1.5.0"}
2020-07-03 22:08:29.934  - info: sainlogic.0 (3088) Scheduler updating IOBroker states
2020-07-03 22:08:29.935  - debug: sainlogic.0 (3088) Setting value from data for info.softwaretype to EasyWeatherV1.5.0
2020-07-03 22:08:29.936  - debug: sainlogic.0 (3088) Scheduler connected to weather station run 255,255,11,0,6,4,4,25
2020-07-03 22:08:44.851  - info: sainlogic.0 (3088) Scheduler pulling for new data
2020-07-03 22:08:44.852  - debug: sainlogic.0 (3088) Scheduler connected to weather station run 255,255,80,3,83
2020-07-03 22:08:44.855  - debug: sainlogic.0 (3088) FW Scheduler Received data string: ffff501511456173795765617468657256312e352e30
2020-07-03 22:08:44.855  - debug: sainlogic.0 (3088) Data Command received: 80 subcommand 69
2020-07-03 22:08:44.855  - debug: sainlogic.0 (3088) Data object: {"softwaretype":"EasyWeatherV1.5.0"}
2020-07-03 22:08:44.855  - info: sainlogic.0 (3088) Scheduler updating IOBroker states
2020-07-03 22:08:44.856  - debug: sainlogic.0 (3088) Setting value from data for info.softwaretype to EasyWeatherV1.5.0
2020-07-03 22:08:44.856  - debug: sainlogic.0 (3088) Scheduler connected to weather station run 255,255,11,0,6,4,4,25
2020-07-03 22:08:59.852  - info: sainlogic.0 (3088) Scheduler pulling for new data
2020-07-03 22:08:59.853  - error: sainlogic.0 (3088) Connection error on 192.168.178.58:45000: Error: connect EISCONN 192.168.178.58:45000 - Local (192.168.178.79:37174)
2020-07-03 22:08:59.854  - debug: sainlogic.0 (3088) FW Scheduler Connection closed

BTW: EISCONN means that the socket is alkready connected so you might have cases where your code is not optimal! Comes only sometimes ... but still no answer :-(

ANd the windows program also do not work ... so it seems to be something different

The connection issue is due to the fact the station does not send back a result. I should probably add a timeout as my code is waiting on the response.

Timeout is a good idea ;-)
But in fact I now could after the fw Update think about using the „Webserver“ Mode. ;-)

Closing w/o update.
Station will only work in Listener mode.


text:
Play video file from CDN does not work

Hi,
Is it possible to play video (previously decoded with ffmpeg) hosted in CDN with http endpoint?
Thanks,
Hanan

Yes, but you need to add a CORS header for the .ts file on the CDN, if you want to embed the video from another domain.
I.e. for all .ts files on the CDN, specify:
Access-Control-Allow-Origin: *
and JSMpeg will be able to load the video from the CDN.

Fixed.
Thanks !

authors: hbenlulu phoboslab

repo: phoboslab/jsmpeg


text:
A request for publishing a new release

There are some additional improvements since prior version. Among them, commit worth mentioning is:

https://github.com/phoenixframework/phoenix_live_reload/pull/138

Maybe, it's time to give it a new release, such as v1.4.2 ?

We will soon, as we work towards v1.5.

Thank you for letting me know about this.
I will close this issue for now and look forward to version 1.5. ;)


text:
serve "Project already exists" on 7.1.1

PhoneGap CLI, node & npm versions
Phonegap: 7.1.1
nodejs: v4.2.6
npm: 5.3.0
Expected Behaviour
App is served with no errors
Actual Behaviour
The first time phonegap serve is run on a project, phonegap creates a "serve" directory in the project directory and runs it. After stopping the server and running phonegap serve a error message stating "Project already exists! Delete and recreate" appears.
Steps to Reproduce
I cloned a project from git (which I can't share unfortunately), but it seems to happen on any app template.

Clone repository from git. My repo contains the follow files/folders. The rest are gitignored.

www
.bithoundrc
.bowerrc
.gitignore
bower.json
config.xml
package-lock.json

cd in the project folder

phonegap serve
*Phonegap creates a "serve" directory in my project that contains a demo app with index.html etc.

Ctrl+C to cancel

phonegap serve
Error is displayed

Error messages
"Project already exists! Delete and recreate"
Furthermore
This error does not occur on 7.0.1. Some behaviour has been changed (as is expected between minor versions). I've rolled-back until this is resolved or someone can explain what I'm doing wrong XD.

Hi @samlittler,
Just tried this myself using version 7.1.1 and am not able to reproduce this error. There is no "serve" directory and I am able to run the command multiple times consecutively. Can you try completely uninstalling phonegap, npm uninstall -g phonegap, and reinstalling, npm install -g phonegap. Maybe there is something that's gone wrong with your install.

Hi @surajpindoria,
Thanks for your aid in this. I tried that to no avail I'm afraid. Please see the output from my command line attached.
`samlittler@samlittler-p6-2302eam:~/phonegap$ ll
total 8
drwxrwxr-x  2 samlittler samlittler 4096 Dec 15 09:18 ./
drwxr-xr-x 53 samlittler samlittler 4096 Dec 13 11:43 ../
samlittler@samlittler-p6-2302eam:~/phonegap$ sudo npm uninstall -g phonegap
removed 907 packages in 7.176s
samlittler@samlittler-p6-2302eam:~/phonegap$ sudo npm install -g phonegap
npm WARN deprecated connect@2.30.2: connect 2.x series is deprecated
npm WARN deprecated node-uuid@1.4.8: Use uuid module instead
npm WARN deprecated tough-cookie@2.2.2: ReDoS vulnerability parsing Set-Cookie https://nodesecurity.io/advisories/130
/usr/local/bin/phonegap -> /usr/local/lib/node_modules/phonegap/bin/phonegap.js
npm WARN optional SKIPPING OPTIONAL DEPENDENCY: fsevents@1.1.3 (node_modules/phonegap/node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for fsevents@1.1.3: wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})

phonegap@7.1.1
added 1023 packages in 18.042s
samlittler@samlittler-p6-2302eam:~/phonegap$ git clone git@github.com:REDACTED/REDACTED
Cloning into 'REDACTED'...
remote: Counting objects: 834, done.
remote: Compressing objects: 100% (552/552), done.
remote: Total 834 (delta 290), reused 783 (delta 239), pack-reused 0
Receiving objects: 100% (834/834), 4.88 MiB | 1.96 MiB/s, done.
Resolving deltas: 100% (290/290), done.
Checking connectivity... done.
samlittler@samlittler-p6-2302eam:~/phonegap$ cd REDACTED/
samlittler@samlittler-p6-2302eam:~/phonegap/REDACTED$ phonegap serve
Creating Browser project. Path: serve
[phonegap] starting app server...
[phonegap] listening on 10.0.0.192:3000
[phonegap]
[phonegap] ctrl-c to stop the server
[phonegap]
[VARIOUS PROJECT FILES REDACTED]
[phonegap] 200 /plugins/org.apache.cordova.file/www/DirectoryEntry.js
[phonegap] 200 /plugins/org.apache.cordova.file/www/DirectoryReader.js
[phonegap] 200 /plugins/org.apache.cordova.file/www/Entry.js
[phonegap] 200 /plugins/org.apache.cordova.file/www/File.js
[phonegap] 200 /plugins/org.apache.cordova.file/www/FileEntry.js
[phonegap] 200 /plugins/org.apache.cordova.file/www/FileError.js
[phonegap] 200 /plugins/org.apache.cordova.file/www/FileReader.js
[phonegap] 200 /plugins/org.apache.cordova.file/www/FileSystem.js
[phonegap] 200 /plugins/org.apache.cordova.file/www/FileUploadOptions.js
[phonegap] 200 /plugins/org.apache.cordova.file/www/FileUploadResult.js
[phonegap] 200 /plugins/org.apache.cordova.file/www/FileWriter.js
[phonegap] 200 /plugins/org.apache.cordova.file/www/Flags.js
[phonegap] 200 /plugins/org.apache.cordova.file/www/LocalFileSystem.js
[phonegap] 200 /plugins/org.apache.cordova.file/www/Metadata.js
[phonegap] 200 /plugins/org.apache.cordova.file/www/ProgressEvent.js
[phonegap] 200 /plugins/org.apache.cordova.file/www/fileSystems.js
[phonegap] 200 /plugins/org.apache.cordova.file/www/requestFileSystem.js
[phonegap] 200 /plugins/org.apache.cordova.file/www/resolveLocalFileSystemURI.js
[phonegap] 200 /plugins/org.apache.cordova.file/www/ios/FileSystem.js
[phonegap] 200 /plugins/org.apache.cordova.file/www/fileSystems-roots.js
[phonegap] 200 /plugins/org.apache.cordova.file/www/fileSystemPaths.js
[phonegap] 200 /plugins/org.apache.cordova.file-transfer/www/FileTransferError.js
[phonegap] 200 /plugins/org.apache.cordova.file-transfer/www/FileTransfer.js
[phonegap] 200 /plugins/org.apache.cordova.geolocation/www/Coordinates.js
[phonegap] 200 /plugins/org.apache.cordova.geolocation/www/PositionError.js
[phonegap] 200 /plugins/org.apache.cordova.geolocation/www/Position.js
[phonegap] 200 /plugins/org.apache.cordova.geolocation/www/geolocation.js
[phonegap] 200 /plugins/org.apache.cordova.globalization/www/GlobalizationError.js
[phonegap] 200 /plugins/org.apache.cordova.globalization/www/globalization.js
[phonegap] 200 /plugins/org.apache.cordova.inappbrowser/www/inappbrowser.js
[phonegap] 200 /plugins/org.apache.cordova.media/www/MediaError.js
[phonegap] 200 /plugins/org.apache.cordova.media/www/Media.js
[phonegap] 200 /plugins/org.apache.cordova.media-capture/www/CaptureAudioOptions.js
[phonegap] 200 /plugins/org.apache.cordova.media-capture/www/CaptureImageOptions.js
[phonegap] 200 /plugins/org.apache.cordova.media-capture/www/CaptureVideoOptions.js
[phonegap] 200 /plugins/org.apache.cordova.media-capture/www/CaptureError.js
[phonegap] 200 /plugins/org.apache.cordova.media-capture/www/MediaFileData.js
[phonegap] 200 /plugins/org.apache.cordova.media-capture/www/MediaFile.js
[phonegap] 200 /plugins/org.apache.cordova.media-capture/www/capture.js
[phonegap] 200 /plugins/org.apache.cordova.network-information/www/network.js
[phonegap] 200 /plugins/org.apache.cordova.network-information/www/Connection.js
[phonegap] 200 /plugins/org.apache.cordova.splashscreen/www/splashscreen.js
[phonegap] 200 /plugins/org.apache.cordova.statusbar/www/statusbar.js
[phonegap] 200 /plugins/org.apache.cordova.vibration/www/vibration.js
[phonegap] 200 /plugins/org.apache.cordova.battery-status/www/battery.js
[phonegap] 200 /plugins/org.apache.cordova.camera/www/CameraConstants.js
[phonegap] 200 /plugins/org.apache.cordova.camera/www/CameraPopoverOptions.js
[phonegap] 200 /plugins/org.apache.cordova.camera/www/Camera.js
[phonegap] 200 /plugins/org.apache.cordova.camera/www/ios/CameraPopoverHandle.js
[phonegap] 200 /plugins/org.apache.cordova.console/www/console-via-logger.js
[phonegap] 200 /plugins/org.apache.cordova.console/www/logger.js
[phonegap] 200 /plugins/org.apache.cordova.contacts/www/contacts.js
[phonegap] 200 /plugins/org.apache.cordova.contacts/www/Contact.js
[phonegap] 200 /plugins/org.apache.cordova.contacts/www/ContactAddress.js
[phonegap] 200 /plugins/org.apache.cordova.contacts/www/ContactError.js
[phonegap] 200 /plugins/org.apache.cordova.contacts/www/ContactField.js
[phonegap] 200 /plugins/org.apache.cordova.contacts/www/ContactFindOptions.js
[phonegap] 200 /plugins/org.apache.cordova.contacts/www/ContactName.js
[phonegap] 200 /plugins/org.apache.cordova.contacts/www/ContactOrganization.js
[phonegap] 200 /plugins/org.apache.cordova.contacts/www/ContactFieldType.js
[phonegap] 200 /plugins/org.apache.cordova.contacts/www/ios/contacts.js
[phonegap] 200 /plugins/org.apache.cordova.contacts/www/ios/Contact.js
[phonegap] 200 /plugins/org.apache.cordova.device/www/device.js
[phonegap] 200 /plugins/org.apache.cordova.device-motion/www/Acceleration.js
[phonegap] 200 /plugins/org.apache.cordova.device-motion/www/accelerometer.js
[phonegap] 200 /plugins/org.apache.cordova.device-orientation/www/CompassError.js
[phonegap] 200 /plugins/org.apache.cordova.device-orientation/www/CompassHeading.js
[phonegap] 200 /plugins/org.apache.cordova.device-orientation/www/compass.js
[phonegap] 200 /plugins/org.apache.cordova.dialogs/www/notification.js
[phonegap] 200 /plugins/nl.x-services.plugins.insomnia/www/Insomnia.js
[phonegap] 200 /plugins/ADBMobile/sdks/Cordova/ADBMobile/Shared/ADBHelper.js
[phonegap] 404 /ServiceWorker.js
Using cordova-fetch for cordova-browser@~4.1.0

Adding browser project...
Creating Browser project. Path: platform
Unable to load PlatformApi from platform. Error: Unhandled "error" event. (Does not appear to implement platform Api.)
Error: Unhandled "error" event. (The platform "browser" does not appear to be a valid cordova platform. It is missing API.js. browser not supported.)
Project already exists! Delete and recreate
samlittler@samlittler-p6-2302eam:~/phonegap/REDACTED$
samlittler@samlittler-p6-2302eam:~/phonegap/REDACTED$ ll
total 88
drwxrwxr-x 9 samlittler samlittler  4096 Dec 15 09:20 ./
drwxrwxr-x 3 samlittler samlittler  4096 Dec 15 09:20 ../
-rw-rw-r-- 1 samlittler samlittler  1229 Dec 15 09:20 .bithoundrc
-rw-rw-r-- 1 samlittler samlittler   816 Dec 15 09:20 bower.json
-rw-rw-r-- 1 samlittler samlittler    34 Dec 15 09:20 .bowerrc
-rw-rw-r-- 1 samlittler samlittler  8140 Dec 15 09:20 config.xml
drwxrwxr-x 8 samlittler samlittler  4096 Dec 15 09:20 .git/
-rw-rw-r-- 1 samlittler samlittler   117 Dec 15 09:20 .gitignore
drwxrwxr-x 2 samlittler samlittler  4096 Dec 15 09:20 hooks/
drwxrwxr-x 3 samlittler samlittler  4096 Dec 15 09:20 node_modules/
-rw-rw-r-- 1 samlittler samlittler   214 Dec 15 09:20 package.json
-rw-rw-r-- 1 samlittler samlittler 23200 Dec 15 09:20 package-lock.json
drwxrwxr-x 9 samlittler samlittler  4096 Dec 15 09:20 platform/
drwxrwxr-x 2 samlittler samlittler  4096 Dec 15 09:20 platforms/
drwxrwxr-x 9 samlittler samlittler  4096 Dec 15 09:20 serve/
drwxrwxr-x 9 samlittler samlittler  4096 Dec 15 09:20 www/
samlittler@samlittler-p6-2302eam:~/phonegap/REDACTED$ `
On this occasion, I couldn't get phonegap to serve the first time.

Looks it is downloading an old version of the browser platform which is causing some issues. Can you remove that, phonegap platform rm browser, and then try to serve again.

@surajpindoria Excellent, that seems to have resolved it!

@samlittler,
Nice, glad you got it working!

authors: samlittler surajpindoria

repo: phonegap/phonegap-cli


text:
Add Windows 10 Support

Created a Windows Runtime Component (Universal Windows) for all platform types for the zxing.winmd.  Created a Windows Runtime Component (Universal Windows) for the WinRTBarcodeReader.  Added it to the plugin and applied a version attribute to the framework entries.
Fixes #87

@TimBarham, this looks good, can we merge this?
@RyanWFiorini, only one question: is it possible to reuse the same files (at least Reader.cs and ZXing library for AnyCPU, as they seems to be equal) for both Win8 and Win10 versions of the component?

I noticed you closed this request.  I didn't see the previous message to me.  I can use the same file for both projects.  Do you want to reopen the request and I will make the changes?

No, this is not needed, because your changes pulled into master in https://github.com/phonegap/phonegap-plugin-barcodescanner/commit/9c2da016e4ddbd973a596568c750b7bb48ea4409. Thanks!

Also I played with new mediaCapture APi in Windows 10 and realized, that we don't need for this WinMD component anymore. The only thing required is ZXing assemblies, rebuilt for Win10 from your PR

authors: RyanWFiorini vladimir-kotikov

repo: phonegap/phonegap-plugin-barcodescanner


text:
Clear Notifications on Android

Hi everyone,
I would like to clear notifications on android, even if my app is not in the background.
Is there a way to do that?
Use case: Chat.
My user receives a message, as a push, reads it on another device and I would like the push to be removed then.

Closing as duplicate of #346.

authors: Nilos macdonst

repo: phonegap/phonegap-plugin-push


text:
How to generate a action-button JSON in java

I was going through this link https://github.com/phonegap/phonegap-plugin-push/blob/master/docs/PAYLOAD.md#action-buttons
I am using https://github.com/google/gcm to send notification from java. My question is how can i make

"data": {
    "title": "AUX Scrum",
    "message": "Scrum: Daily touchbase @ 10am Please be on time so we can cover everything on the agenda.",
    "actions": [
        { "icon": "emailGuests", "title": "EMAIL GUESTS", "callback": "app.emailGuests", "foreground": true},
        { "icon": "snooze", "title": "SNOOZE", "callback": "app.snooze", "foreground": false}
    ]
}

I dont see any method that can produce this kind JSON. addData(String, String) only except String

@mkj-git it's just JSON. There are many Java packages that let you manipulate JSON.
http://stackoverflow.com/a/31743324

That i got it. My question is the GCM server only allow to send anything in <String, String> format. I can't see any method that allow you to send any other format except <String,Steing>.
e.g addData(String, String), setData(HashMap<String, String>) are the only method to send data. and both are accepting only Strings.

You can send any JSON to GCM server, how you create it is up to you. If the library you use right now only allows you to add strings search for another

can you suggest some good GCM java library

I'm not familiar with any GCM java library, the only push notifications I've sent from a java server just did a POST to the GCM server

@mkj-git use node-gcm, there is an example script in the docs:
https://github.com/phonegap/phonegap-plugin-push/blob/master/docs/PAYLOAD.md#action-buttons

that is for nodejs i am looking for java

authors: jcesarmobile macdonst mkj-git


text:
Update Phaser Sandbox version

Version

Phaser Version:
3.17.0

Browser:
Any

Description
While the shader examples on labs.phaser.io do work, trying to modify them on Phaser Sandbox doesn't work because the Phaser version on the Sandbox is outdated (3.15.1)
Example
This works: http://labs.phaser.io/view.html?src=src\game%20objects\shaders\change%20shader%20texture.js
This doesn't: http://labs.phaser.io/edit.html?src=src\game%20objects\shaders\change%20shader%20texture.js

Done :)

authors: photonstorm pistacchio

repo: photonstorm/phaser


text:
Applied fixes from StyleCI

This pull request applies code style fixes from an analysis carried out by StyleCI.

For more information, click here.

Current coverage is 26.05%

Merging #2 into master will not affect coverage as of 357be53

@@            master      #2   diff @@
======================================
  Files            2       2
  Stmts          142     142
  Branches         0       0
  Methods         21      21
======================================
  Hit             37      37
  Partial          0       0
  Missed         105     105

Review entire Coverage Diff as of 357be53

Powered by Codecov. Updated on successful CI builds.

authors: Nyholm codecov-io

repo: php-cache/adapter-common


text:
You cannot use the "Symfony\Component\HttpClient\Psr18Client" as the "psr/http-client" package is not installed

PHP version: 8.2.0
Description
php-http/discovery plugin installs a psr18 client that cannot be discovered
How to reproduce
composer.json:
{
  "config": {
    "allow-plugins": {
      "php-http/discovery": true
    }
  },
  "require": {
    "php-http/client-implementation": "^1.0",
    "php-http/discovery": "^1.18"
  }
}

test.php:
<?php
require 'vendor/autoload.php';
$client = \Http\Discovery\Psr18ClientDiscovery::find();
var_dump(get_class($client));

composer update
php test.php

Results:
Fatal error: Uncaught LogicException: You cannot use the "Symfony\Component\HttpClient\Psr18Client" as the "psr/http-client" package is not installed. Try running "composer require php-http/discovery psr/http-client-implementation:*". in /usr/src/myapp/vendor/symfony/http-client/Psr18Client.php on line 37

Possible Solution
As a workaround, I could require psr/http-client, but we don't directly require or use it.

ok, took me a moment but i think this is a confusion between the psr client and the legacy php-http client abstraction.
if you want psr-18, you don't need to require php-http/client-implementation but psr/http-client-implementation

if you manually require psr/http-client, it will only work by chance because the symfony http client has adapters for both http client interfaces.

Changing my requirements to use psr/http-client-implementation gives the same error:
{
  "config": {
    "allow-plugins": {
      "php-http/discovery": true
    }
  },
  "require": {
    "psr/http-client-implementation": "*",
    "php-http/discovery": "^1.18"
  }
}

But, I was mistaken earlier when I said we did not directly rely on psr/http-client - we do, and so I should add that dependency to our library.
I'd like to understand better about "it will only work by chance" though, I'd much prefer it to work on purpose! :)

php-http/client-implementation is the virtual package for the httplug interfaces by php-http. after we built that, we proposed psr-18, which is now accepted and defines an "official" http client interface. the virtual package for psr18 is psr/http-client-implementation.
we did not deprecate httplug because it also provides an asynchronous client, and psr-18 is only about synchronous http requests.
symfony/http-client provides both an integration with psr18 and with the legacy php-http client - this is why i said it would work by chance if you requries a php-http client implementation but then expect to find a psr18 client.
all that said, i just ran your minimal composer.json and indeed psr/http-client is not installed automatically. i think this is indeed a mistake of discovery. i think #249 would be the fix. can you try with that branch if your project then indeed works?

Confirmed as working using dev-composer-plugin branch. I get psr/http-client, and discovery works as expected.

authors: brettmc dbu

repo: php-http/discovery


text:
Remove Zend UriFactory

Q
A

Bug fix?
yes

New feature?
no

BC breaks?
yes

Deprecations?
no

Related tickets
-

Documentation
-

License
MIT

What's in this PR?
PR is primarily context to explain proposal to remove Zend UriFactory support.
Why?
The Zend UriFactory is not PSR-17 compliant in the way that it generates it's URIs.
Example:
$uri = new \Zend\Diactoros\Uri('testing');
$this->assertEquals('testing', (string) $uri);

This test case would fail as the resulting Uri will actually be /testing.
Prepending a / is an assumption which has consequences - mentioned on\Psr\Http\Message\UriInterface
* Normally, the empty path "" and absolute path "/" are considered equal as
* defined in RFC 7230 Section 2.7.3. But this method MUST NOT automatically
* do this normalization because in contexts with a trimmed base path, e.g.
* the front controller, this difference becomes significant. It's the task
* of the user to handle both "" and "/".

More info here:
https://github.com/php-fig/http-message/blob/master/src/UriInterface.php#L134
As this UriFactory is prepending a / and not allowing the user to handle both "" and "/", different UriFactories can not be used interchangeably - preventing Liskov Substitution principle here.
Recently this caused us an issue where in many of our services, our new client operated as expected - primarily using versions of Guzzle. However, the Zend UriFactory was used in one of our legacy systems. Since our services have a path as part of their base_uri, this was stripped and our URIs were malformed due to prepending / - we addressed this issue by using league/uri.
Checklist

[ ] Updated CHANGELOG.md to describe BC breaks / deprecations | new feature | bugfix
[ ] Documentation pull request created (if not simply a bugfix)

To Do

[ ] The checklist above - primarily raising the PR to first discuss the approach and gain feedback

i am worried that this would lead to problems for existing users of discovery that had been using zend UriFactory and would end up with a system that does not discover a uri factory anymore after upgrading.
zend has been renamed laminas (which we also support, but at lower priority). i guess zend things will not be updated anymore (and the change in the factory would be tricky as their users will have learned the wrong behaviour and rely on it...)? is laminas UriFactory correct?
we could do a new major version of discovery that drops no longer maintained providers like zend.
@Nyholm wdyt?

@dbu you could add zendframework/zend-diactoros as conflict when you drop the interface.
Laminas is out there for years now, even laminas dropped backwards compatibility layer for zendframework components while conflicting with them in several components while it replace the appropriate zendframework component in earlier versions:
https://github.com/laminas/laminas-diactoros/blob/b2355dc58253c480aa25ba085845a5aabf84955d/composer.json#L58
https://github.com/laminas/laminas-diactoros/commit/8c644be420dd168f622d75af8cfd7eed5896c6f7

Closing in favor of #225, thanks for pushing this.

authors: acairns boesing dbu nicolas-grekas


text:
Add paragraph word wrap support

Closes https://github.com/php-tui/php-tui/issues/202 and https://github.com/php-tui/php-tui/issues/203
This implementation is heavily based on Ratatui's. You can review tests (the last two ones are the best ones) to understand the differences between breaking by Word vs WordTrimmed (there is a significant difference).
The None variant is simply an alias for Character. Notice that None is the default wrapping logic used if none is specified.

Nice work!

authors: KennedyTedesco dantleech

repo: php-tui/php-tui