Android: Scanner UI broken on Android 11

I added the library to our app for a .NET MAUI App on iOS and Android.
On Android we have issues only on Android 11, above versions work fine.
Depending on where the camera gets enabled in code behind (in OnAppearing or OnNavigatedTo) several different UI issues are happening.
A reproduction project and screenshots are attached.
Some comments with descriptions which screenshot matches which case of implementation are within ScanPage.xaml.cs and in ScanPage.xaml are the 2 versions of surrounding UI elements for the CameraView.
Hope you can help me out.
BarcodeScanningIssue.zip

Hello.
I've taken a quick look at your example on an Android 11 emulator as I don't have a device with this version on hand. I've noticed a few things:

await Methods.AskForRequiredPermissionAsync(); is missing. Camera won't start without it.
Border didn't work whatever I did.
Frame worked when HeightRequest was set on some fixed value (for example 300) or if Grid.RowDefinitions were set to fixed value. In my experience this is a broader MAUI problem as I had problems with calculated values on Grid with other controls in the past.
For this example you need to call cameraView.Handler?.DisconnectHandler(); to prevent memory leaks. Readme has an example.

I really don't know why is Frame working and Border isn't... Maybe someone can chime in and help...

Thanks for your reply.
In the main project we have the permissions request, in the sample project I shortly granted permission via the app settings from the home screen - sorry for forgetting to mention that.
We'd like to not set a fixed height, but let the Border use any available height the several device screens offer using the layout. I will have a closer look at the call of cameraView.Handler?.DisconnectHandler(); not quite sure if that helps as needed, but I will try and let you know.
Would be nice if anybody has a clue how to solve that, on that side MAUI has still a lot of problems and needed workarounds to be done by us.

Just to give you a heads-up I think I found a solution to this problem.
Although official AndroidX documentation mentions:

Use a SurfaceView for the preview when possible.

it looks like the solution is to set PreviewView ImplementationMode to Compatible, which uses TextureView as it integrates seamlessly to the UI hierarchy and it looks like this fixes the problems with the layout. But it has some drawbacks, mainly slower performance.
I'll give it a few more days to test and if everything goes well I'll publish a new release.

That sounds promising.
Could you provide a preview NuGet of that possibly fix? Then I can also test that in our app.

Thanks for the NuGet Update, that fixed the problems on our side.


Error on compiling

Configure project :shared
fabric-installer.json not found in dependencies!

Task :shared:compileJava FAILED

FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':shared:compileJava'.

Could not resolve all dependencies for configuration ':shared:compileClasspath'.
The project declares repositories, effectively ignoring the repositories you have declared in the settings.
You can figure out how project repositories are declared by configuring your build to fail on project repositories.
See https://docs.gradle.org/8.1.1/userguide/declaring_repositories.html#sub:fail_build_on_project_repositories for details.
Could not find :Geyser-Fabric:.
Searched in the following locations:
- file:/C:/Users/Glitch/Downloads/Hydraulic/libs/Geyser-Fabric.jar
Required by:
project :shared

I'm getting the same error.

try to remove this as they said cuz they already do it on geyser
// TODO: Remove this once custom block support
//       is merged into the master branch. This is
//       far from ideal, but is the easiest way to
//       get this working for the time being.
compileOnly(":Geyser-Fabric")

Download https://geysermc.org/download#fabric create a libs folder place the jar in the libs folder and rebuild

Should be resolved with the latest version


better timeseries notebooks

improve color bar

Coverage decreased (-0.08%) to 80.844% when pulling 2bf35d2a4b589bccd2de86c924bd4d3566349baa on rnaer into 17e5e8783085b12b37c83ce5755a4523c6cddf17 on master.

Coverage decreased (-0.08%) to 80.844% when pulling 2bf35d2a4b589bccd2de86c924bd4d3566349baa on rnaer into 17e5e8783085b12b37c83ce5755a4523c6cddf17 on master.


Sorry for doing this, but I don't how how else to do it.

Just wanted to say HI DUDE. If you are Steven, then its been a loooong time my man.  🥳
http://networkdls.com/

um i do not know what you are talking about i am Aarav Paryemalani

im a teen game dev

Sorry. My mistake.

Well, if you don't mind me asking - what languages are you using?


Question: Is grouping supported in Select control ?

Just wondering if grouping is possible in Select control ? Thanks.

@SMrockend Nope, currently md2-select support simple and multiple selections only. will provide this feature later.


Fix Tray Icon in packaged/installer release

Fix path for Tray icon

Thanks for the PR, I have fixed this in another manner: https://github.com/block-core/blockcore-hub/commit/5460035d8831ce82b73dbba19bafc77fbd088383


LinalgRefactor - Remove old linalg::range_fill and linalg::max

Can you send viktors patch in a separate first PR?

Nevermind about the seperate patch, this just refactors things, so merging


Fix setProps when IDs aren't provided

Fixes #165

@chriddyp this is awesome (my :dancer: stands BTW, despite the extra discussion). But @byronz merged some of the if(id) hacks in https://github.com/plotly/dash-core-components/pull/523, do we need to remove them now? Seems like they could actually be harmful once this PR is merged.

Yeah - I think I'll follow up with a PR that cleans up some of the DCC logic. I think that the id => setState is actually still "safe" but it could be cleaned up - many of the components can become functional & stateless


Blob type support

Add a new type Blob to gluesql::data::Value.
It may match to Blob in sqlparser::ast::DataType and HexStringLiteral(String) in sqlparser::ast::Value.

Expand sqlparser::ast::Value to store raw bytes[].
Then someone who wants to build AST without sql string, they can use raw bytes without converting into string.

@KyGost

Oh, I don't recall why I was tagged in this?

I barely remember. You were trying to insert blob, but there's no feature supported so you might use sqlparser directly? or used text.

It seems that BYTEA, which has a different name but performs the same function, is implemented as https://github.com/gluesql/gluesql/pull/586.
How about closing issue?

resolved in #586


Check logging messages

[ ] Indication of the script start/end
[ ] Check levels

Check levels:

use level INFO really just for most important stuffs
use CRITICAL (most of ERROR should be CRITICAL) and exit together
use WARNING if something NON optional was not provided but we can still continue => don't print anything if optional thing was not provided
do something with commented logging
evaluate script (also consider using compare jsons script)
add more logging to some scripts, e.g. intent_json2.csv.py


bug: IONIC 4 source map files out of sync

Ionic version:
[x] 4.4
Current behavior:
Debugging with Webstorm: debugger stops at wrong line or does not stop at all.
IONIC 3 is no problem, so i think it is an Ionic 4 source map files problem.
Ionic info:
Ionic:
ionic (Ionic CLI)             : 4.12.0 (/Users/aj/.npm-global/lib/node_modules/ionic)
Ionic Framework               : @ionic/angular 4.4.0
@angular-devkit/build-angular : 0.13.9
@angular-devkit/schematics    : 7.3.9
@angular/cli                  : 7.3.9
@ionic/angular-toolkit        : 1.5.1
Cordova:
cordova (Cordova CLI) : 8.1.2<EMAIL_ADDRESS>Cordova Platforms     : android 7.1.4, browser 5.0.4, ios 5.0.1
Cordova Plugins       : cordova-plugin-ionic-keyboard 2.1.3, cordova-plugin-ionic-webview 4.0.1, (and 7 other plugins)
System:
ios-deploy : 2.0.0
ios-sim    : 8.0.1
NodeJS     : v8.9.4 (/usr/local/bin/node)
npm        : 5.6.0
OS         : macOS Mojave
Xcode      : Xcode 10.2.1 Build version 10E1001

Hi there,
Thanks for the issue. Can you provide a repository where this is happening?
Thanks!

HI @liamdebeasi,
unfortunately no, I have a non-disclosure agreement on this project.
But to reproduce you just have to start a new project and start debugging.
Then add a few lines of code. After that the debugger in Webstorm is out of sync.
Thanks!

Hi there,
I do not need your project's code, but rather a sample repo with code the shows this happening. Without a code example, I am unable to reproduce this issue.
Thanks!

Hi,
here is a dummy project with this supposed bug: https://www.dropbox.com/s/4rokw2vue9d7iq8/18280.zip?dl=0
I dded an AuthGuard, copy and paste some code lines, after a while the debugger from Webstorm got out of sync, see attached image.... The chrome debugger is still in sync, the real line number for "return false" is 49. If i set the breakpoint in Webstorm on line 61, it does not get hit.
Thanks

Hi there,
Thanks for the follow up. If Chrome is parsing the source maps correctly and hitting breakpoints where it should this probably is not an Ionic issue. There seem to be a few posts on the JetBrains website and StackOverflow for similar issues after looking this issue up on Google.
For further help with this feel free to post on our forum or our slack.
I am going to close this as this is not an Ionic issue. Thanks!

HI,
my description was wrong. The real line number is 61, so Google chrome hits the wrong line, the source maps did not get updated correctly.
Thanks


Comments with names

I forgot to update to the base branch to master after the dependent branch pull request was approved.


operator t8c-certified (42.58.0-beta.1)

operator t8c-certified (42.58.0-beta.1)

/pipeline restart operator-release-pipeline

/pipeline restart operator-release-pipeline


Support nested embedded case classes and optionals

Fixes #697
Solution
Allow unlimited nesting of embedded case classes and optionals.
Checklist

[x] Unit test all changes
[ ] Update README.md if applicable
[ ] Add [WIP] to the pull request title if it's work in progress
[x] Squash commits that aren't meaningful changes
[x] Run sbt scalariformFormat test:scalariformFormat to make sure that the source files are formatted

@getquill/maintainers

@getquill/maintainers Ready for review.


TypeError: view must be a callable or a list/tuple in the case of include().

How to modify the urls.py file when the file is trying to look for the views.py file?

This is a Django question, not a todo question, but maybe we can help. What does your urls.py file look like now?

No response from user, and this is not a django-todo issue. Closing.


warnings in Authelia as preparation for V5.0

option 'domain' is deprecated in v4.38.0 and has been replaced by a multi-domain configuration: this has automatically been mapped for you but you will need to adjust your configuration to remove this message and receive the latest messages"

So far no ready made configuration for this change seems to be available

https://www.authelia.com/roadmap/active/multi-domain-protection/
Feature is not finished yet, recommend waiting


Custom sorting & marking for some tasks: categories such as "fast", "important"?

It kind of overcomplicates things.


Conflict resolution for replaced arrays and objects

TL;DR: Array and object values returned by Automerge.getConflicts cannot be assigned to a document as part of conflict resolution.

The documentation for managing conflicts does not demonstrate how to use the conflict information to resolve a conflict, so I could be doing this wrong. I assumed I would be able to use the values in the conflict information directly to resolve a conflict like so:
const result = Automerge.merge(doc1, doc2)
const conflicts = Automerge.getConflicts(result, 'list')

Automerge.change(doc2, d => {
  d.list = Object.values(conflicts).pop(); // bogus "pop" resolution strategy :)
});

While this works for primitive values, the approach yields a RangeError: Cannot create a reference to an existing document object for arrays and objects. The value portion of the opId -> value map includes _objectId and other Automerge symbols.
Am I using Automerge.getConflicts incorrectly or should it be returning a subtly different result?
Test Case
let doc1 = Automerge.init()
doc1 = Automerge.change(doc1, d => d.list = [])
let doc2 = Automerge.clone(doc1)

doc1 = Automerge.change(doc1, d => d.list = [])
doc2 = Automerge.change(doc2, d => d.list = [])

const result = Automerge.merge(doc1, doc2)
const conflicts = Automerge.getConflicts(result, 'list')

// the following throws: RangeError: Cannot create a reference to an existing document object
Automerge.change(doc2, d => d.list = Object.values(conflicts)[0])

This is a bug, we should be able to do this by checking that the object being assigned is one of the already existing objects at that location. I'll try and take a look at it soon.
In the meantime you'll have to effectively convert the conflicted object to a normal json object via the (hidden) Automerge.toJS function.

Okay, interesting. Is there an equivalent of Automerge.toJs in the javascript-only implementation?

Ah I see, I assumed you were using the latest javascript libarary at @automerge/automerge. If that's not the case I'm not sure what's in the old API
However, even if you are using the latest library (which does have toJS) this approach still doesn't work because the toJS function doesn't completely clear out the bookkeeping we do to track what object ID it is. You may have to serialize to a JSON string and back again. This is obviously gross, I'll work on fixing this soon.

I suspect this is a non-trivial change that warrants some consideration.
If a user is "deciding" to go with another actor's operation as the "winner" of a conflict, the bookkeeping information might be relevant depending on the desired semantics.
When Automerge automatically selects a winning operation, it retains all of the bookkeeping information about the object so it can correctly absorb future changes and merges to that object. Stripping the bookkeeping out would preclude that from happening. The conflict resolution would be making an entirely new nested object rather than "choosing another operation as a winner". The later would allow future merges into that object.
Given the project stance that hooking directly into conflict resolution is not desired, I suspect "entirely new nested object" is fine. But I did want to surface this so it's taken into consideration.

The fix I am suggesting is just that we modify the logic of the proxy which handles the set operation on doc.list (in this example) such that if the value being set is an object which is already a member of the conflict set at that key, then we allow the operation. This seems to me to be uncontroversial because the purpose of forbidding setting an existing object is to retain the tree structure of the document - automerge does not support cycles, or move (yet). If the value is already present at the given key there should be no problem.
As to stripping out the bookkeeping, I was really talking about why the hack of using toJS won't work. toJS is hidden from the docs at the moment because I'm not sure whether it should exist at all, but if it does exist then I think it should strip out the bookkeeping to avoid surprising footguns such as this one. But yeah, I really think the fix is to not throw if the object is in the conflict set.

Ah nice, that sounds like a reasonable approach. For our particular use-case, this will likely operate in an ideal manner.
We're managing multiple documents and performing forward-only merges in order to forward-cascade changes. Retaining lineage would be important as the forward-merges would enable future changes to continue being applied to the same objects. Having said that, we'll be looking to eliminate object / array re-assignment. I actually see the instance in question as a data modelling bug on our end.
I'll monkey-patch our Automerge to experiment with this. If I find anything of concern, I'll report back; otherwise, thanks for the help and speedy discourse!


Password prompt and flickers

I've tried this theme and it looks fantastic. However, I have noticed some issues
There is no password prompt when trying to boot from an encrypted partition, so I have to press escape or guess when I can enter it.
I also have white flickers at the end of the boot animation.

same here, could we please get this updated @zenorocha

would appreciate a password prompt for encrypted disks.


Funkyheatmap dark theme

Describe your changes
Issue ticket number and link
Partially addresses #176
Checklist before requesting a review

[x] I have performed a self-review of my code

Check the correct box. Does this PR contain:

[ ] Breaking changes
[ ] New functionality
[ ] Major changes
[x] Minor changes
[ ] Bug fixes

[ ] Proposed changes are described in the CHANGELOG.md

[x] CI Preview succeeds and looks good!

For the colour schemes, I think @rcannood had a plan to have 2 funkyheatmap cells with light and dark palettes and then to switch them via #200


Default value for zend.assertions

Sail Version: 1.10.2
Laravel Version: 8.61.0
PHP Version: 8.0.11
OS: macOS

Description:
Not sure where this question belongs but I was wondering what are your thought on the default value for zend.assertions on dev environments. Shouldn't it be active in development?

assert() is a language construct in PHP 7, allowing for the definition of expectations: assertions that take effect in development and testing environments, but are optimised away to have zero cost in production.

See: https://www.php.net/manual/en/function.assert.php
Steps To Reproduce:
$ sail php -ini | grep zend.assertions
zend.assertions => -1 => -1

You can update your custom runtime to enable it if you like.


primod.RechargeSvatMapping doesn't drop layer dimension

The RechargeSvatMapping requires a 2D field of active Recharge cells which need to be coupled to SVATs.
The recharge grid can be provided in two ways:

A 2D grid with dims (y, x)
A 3D grid with dims (layer, y, x), where the layer dimension is length one.

In the latter case, we call xarray .drop_vars("layer")  to drop the layer dimension.
However, I'm getting reports this doesn't actually drop the dimension, but creates an empty dimension, which still needs to be squeezed.
Furthermore I see we have not separate unit tests for this mapping to test its validation.

I think the right method here would've been .squeeze("layer", drop=True) instead.


0.6.1-SNAPSHOT is not published to maven

Release 0.7.1 should be showing up in Maven any moment now :)


Order Entry Transaction Update issue

I'm trying to update an OE transaction and when I send the information I receive "Element 'update_sotransaction': The attribute 'key' is required but missing.. Line: 1, column: 0." I'm sending the Transaction definition and the Document number, but I am uncertain as to what the "key" that it is referring to is... any Ideas?
Here is the pertinent part of the code for reference:
OrderEntryTransactionUpdate update = new OrderEntryTransactionUpdate()
{
//hard code invoice number
TransactionDefinition = "Contract Invoice",
DocumentNumber = "arbitrary invoice number"
};
Task updateTask = client.Execute(update);
updateTask.Wait();
            Console.WriteLine("Updated Order Entry Transaction Contract Invoice ");

TransactionId -- https://github.com/Intacct/intacct-sdk-net/blob/master/Intacct.SDK.Tests/Functions/OrderEntry/OrderEntryTransactionUpdateTest.cs#L42


Fix crash when resetting flag

base.Removed already sets the entity's Scene property to null, which is what SceneAs accesses. Since Removed gets the scene as a parameter, we can just use that.

Thanks!


Bug when using 2 filters on the same line

Hi,
I tried to compile the following line:
 {{ $cash_acct->value | money }} growing at {{ $cash_acct->growth_rate | percentage }}
it compiles to:
<?php echo e(\Pine\BladeFilters\BladeFilters::percentage(\Pine\BladeFilters\BladeFilters::money); ?> growing at <?php echo e($cash_acct->growth_rate($cash_acct->value))); ?>
so it mixes up the filters
Now if I do:
{{ $cash_acct->value | money }} growing at
{{ $cash_acct->growth_rate | percentage }}

then I get the correct:
 <?php echo e(\Pine\BladeFilters\BladeFilters::money($cash_acct->value)); ?> growing at
                            <?php echo e(\Pine\BladeFilters\BladeFilters::percentage($cash_acct->growth_rate)); ?>

This is preventing me from using the package.

@AdrienPoupa Hi,
Thanks for your feedback. You are right, this was a bug, that the package caused.
Now, I set the regexp to non-greedy, so it should work well even if it's one or multiple lines as well.
I pushed a new release, so you may update your packages via composer to pull in the latest code.
After that, I think you need to recompile the views, but they should be good now.
Please let me know if it works for you as well!
Thanks!

Great thank you!


Update remoteapp-officesubscription.md

updating per eric's request to state specifically that office 2016 is not officially supported in ARA.

Hi @jobende, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (jobende). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@jobende : Thanks for your contribution to the Azure documentation! The author, @piotrci, has been notified to review your proposed change.

@jobende - Hey, Josh - just a minor typo fix:
You can use your Office 365 subscription to activate and share Office 2013 in Azure RemoteApp deployments.

#sign-off - This is ready to go live.

Only the author of the article can use the sign off comment to add the ready-to-merge label.

Merging per conversation w/@lizap, as the author is no longer in org/role.


Transfer small visual changes

There are few things I would like to change/improve in current transfer design. Figma link
Changes:

remove shadow from list items in more section

remove shadow in Continue button

add info text under token tags with what token you are sending on which network

add checkmark to address textfield when address is correct

add fixed dollar sign to amount field

add trash icon next to amount field when multiple recipients (decrease size of amount field). Please make sure the icons have at least 40x40px touchpoint around them.

make text in amount field a placeholder. Currently there is 0 as a number by default and I have to remove it or I type amount like "0222".

Dimensions:

make the card 660px wide (should be 6 columns)
make left&right padding 3rem
make wallet address textfield bigger to 70% of the width, that should be 390px, (we need more space here than in amount)

remove shadow in Continue button

styling for disabled button

otherwise nice changes

remove shadow in Continue button

styling for disabled button
otherwise nice changes

mby use k-grey stroke as is currently used in prod (cart, buy button)? Looks better imo and already implemented.

The buy button in the gallery item is IMO by accident, as it does not match the rest (cart button). Black stroke is more versatile when using it connected to the same height container.

like here or the buy button that you attached
It is a detail, so we can keep the stroke how it is, but please, definitely change the font color

updated

👋

Updated the delete icon from "close" to "trash".

add fixed dollar sign to amount field

the NeoInput can add icon from Font Awesome directly in the right. is this acceptable?  i may change the font color to grey

add fixed dollar sign to amount field

the `NeoInput` can add icon from `Font Awesome` directly in the right. is this acceptable?

Yes, thats ok


[release-4.15] OCPBUGS-25798: Deprecation snapshot backport

Cherry-pick of #654

/lgtm
/approve

Looks like cluster creation failure.
/retest

We're running into test flakes now that have yet to be backported. I'll give this one more shot, but otherwise we'll need to wait for that to get this in.
/test e2e-gcp-olm

/label cherry-pick-approved
/label qe-approved

test success, detail https://issues.redhat.com/browse/OCPBUGS-25798

Merged in #670; closing.


Update manage-virtual-network.md

Add a warning that Microsoft recommends virtual networks be configured with private address space or public address space owned by the customers organization.  This will prevent problems generally for the customer.  If the customer uses public space owned by Microsoft, some Azure services/products may not function.  This change proposal is a result of a support escalation.

@davegoddard : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@davegoddard this looks good, except for the last line, which is confusing. Can you remove it?

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@allegradomel - I removed the last line per your request. Thanks for reviewing the PR. I'll merge it now.


Unify xbsa_checklist.yml and sbsa_checklist.yml

My project does not have clean plans defined so I ended with two files doing the same stuff.

Done


Missing HTTP response header in auditlog prevents mlogc from parsing correctly in Nginx.

I installed Nginx + modsecurity + mlogc.
When I try to parse the audit logs via cron, using mlogc-batch-load, I get the following error:
[Mon Nov 09 16:51:07 2015] [2] [31066/1dfb8a8] Invalid entry (failed to match regex): nginx <IP_ADDRESS> - - [09/Nov/2015:15:37:00 +0000] \"GET /1QsIkLNV.nl HTTP/1.1\" - 168 \"-\" \"-\" AcEwAcAcAcjcAcAcAcAcQcAc \"-\" /20151109/20151109-1537/20151109-153700-AcEwAcAcAcjcAcAcAcAcQcAc 0 3605 md5:0e5bb92e1a063ffcfbffd71b1cf785c0

I suspect the problem is that mlogc-batch-load fails to parse the HTTP response header. In Apache's audit files, the section F (HTTP response headers) has this format:
--ac634d14-F--
HTTP/1.1 403 Forbidden
Content-Length: 301
Connection: close
Content-Type: text/html; charset=iso-8859-1

while in Nginx has this format:
--57dfa307-F--
HTTP/1.1
Content-Type: text/html
Content-Length: 168
Connection: keep-alive

mlogc-batch-load parses the string after "HTTP/1.1" and stores it in the variable response_status. In the logs generated by Nginx, it doesn't found that string and the variable is passed to mlogc with the value "-", causing the parse error.
In file modsecurity/apache2/msc_logging.c we have this relevant piece of code:
/**
 * Produce an audit log entry.
 */
void sec_audit_logger(modsec_rec *msr) {

        /* There are no response headers (or the status line) in HTTP 0.9 */
        if (msr->response_headers_sent) {
            if (msr->status_line != NULL) {
                text = apr_psprintf(msr->mp, "%s %s\n", msr->response_protocol,
                        msr->status_line);
            } else {
                text = apr_psprintf(msr->mp, "%s %u\n", msr->response_protocol,
                        msr->response_status);
            }
            sec_auditlog_write(msr, text, strlen(text));

After some debugging, I notice that msr->status_line has value "", which satisfies the condition of  "if (msr->status_line != NULL)" and then the status line gets printed as an empty string.
I think a workaround would be applying this patch.
diff -Naur modsecurity/apache2/msc_logging.c modsecurity-patched/apache2/msc_logging.c
--- modsecurity/apache2/msc_logging.c   2015-11-18 19:44:18.103869115 +0000
+++ modsecurity-patched/apache2/msc_logging.c   2015-11-18 20:03:26.826943022 +0000
@@ -853,7 +853,7 @@

         /* There are no response headers (or the status line) in HTTP 0.9 */
         if (msr->response_headers_sent) {
-            if (msr->status_line != NULL) {
+            if (msr->status_line != NULL && msr->status_line[0] != '\0' ) {
                 text = apr_psprintf(msr->mp, "%s %s\n", msr->response_protocol,
                         msr->status_line);
             } else {

I'm willing to do more tests or writing a patch if you'd like.
This happened to me in nginx 1.8.0 and ModSecurity 2.9.0.

Hi @charlymps,
We are current working on libModSecurity v3. The version 3 is a rewritten. The version 2.x (where this patch is applicable) will be available and maintained, thus, I would love to have your patch.

I traced this problem too for an independent issue since the problem is also present in all nginx auditlogs (native format)
at ../apache2/msc_logging.c:1897

1897	        if (msr->response_headers_sent) {
(gdb) print msr->response_headers_sent
$2 = 1
(gdb) s
1898	            if (msr->status_line != NULL) {
(gdb) print msr->status_line
$3 = 0x7cd4a9 ""
(gdb) s
1899	                text = apr_psprintf(msr->mp, "%s %s\n", msr->response_protocol,
(gdb)
It looks like the patch specified will work to fix this issue

Fxied by #1171


Part files downloaded then immediately deleted

Describe the bug
Using cdrm2, I can download files from a profile; I see the *.part files downloaded, then immediately deleted from local disk.
To Reproduce
Steps to reproduce the behavior:

Pull any model's profile and try to download

Expected behavior
Files should be downloaded as expected.
Screenshots/Logs
n/a
Desktop (please complete the following information):

OS: OS X, latest
Browser Chrome, latest
Version [e.g. 22]
binary or python
python version if installed via pip or pypi (installed via pip, ofscraper v3.3.10)

Additional context
Add any other context about the problem here.

Update: ran with debug on and I see this error firing
TRACEBACK  [logger.inner:193]  Traceback (most recent call last):                                                                                logger.py:193
            File "/Users/your_username/.pyenv/versions/3.11.1/lib/python3.11/contextlib.py", line 222, in __aexit__
              await self.gen.athrow(typ, value, traceback)
            File "/Users/your_username/.pyenv/versions/3.11.1/lib/python3.11/site-packages/ofscraper/classes/sessionbuilder.py", line 146, in _aio_funct_async
              yield r
            File "/Users/your_username/.pyenv/versions/3.11.1/lib/python3.11/site-packages/ofscraper/utils/download.py", line 401, in inner
              size_checker(temp,ele,total)
            File "/Users/your_username/.pyenv/versions/3.11.1/lib/python3.11/site-packages/ofscraper/utils/download.py", line 250, in size_checker
              raise Exception(s)
          Exception: Media:3004964113 Post:2168835679749 3024x4032_90f8a0d9632319752b6cf201dbb9b91b size mixmatch target: 1449702 vs current file: 1031622

@PsychoSid cool i think that worked! interestingly though, on v3.3.5, I can't see all models that I would be able to scrape paid content for; e.g.: I'm currently unsubbed to one profile, and would expect to be able to download messages from them, but that profile doesn't show up in the list.

Yeah I think I see the same I explicitly set mine with -u but I only have 4 anyway so it's easy for me.

Es porque el sitio donde hace la petición (cdrm2) para obtener las KID esta caido, solo te descargara los archivos encriptados (.part) y si en el programa le pones "partfileclean": true, te los borra
Usa "key-mode-default" manual o keydb

Also getting this on 3.3.11:
Progress: (0 photos, 0 videos, 0 audios, 0 skipped, 33 failed || 33/33||0 B/377.22 MB) ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 100% 0:02:41
With this in the logs:
 2023-11-12 07:22:55:\[logger.inner:193]  Traceback (most recent call last):
  File "/usr/lib/python3.11/contextlib.py", line 222, in __aexit__
    await self.gen.athrow(typ, value, traceback)
  File "/home/your_username/app/your_username/classes/sessionbuilder.py", line 146, in _aio_funct_async
    yield r
  File "/home/your_username/app/your_username/utils/download.py", line 401, in inner
    size_checker(temp,ele,total)
  File "/home/your_username/app/your_username/utils/download.py", line 250, in size_checker
    raise Exception(s)
Exception: Media:3069412081 Post:789443662 0hjeacla2vsrmfywhmtc1_source size mixmatch target: 10785016 vs current file: 10026844

 2023-11-12 07:22:55:\[logger.inner:193]  Media:3069412081 Post:789443662 0hjeacla2vsrmfywhmtc1_source size mixmatch target: 10785016 vs current file: 10026844

Reverting back to 3.3.5 is working.
Progress: (5 photos, 28 videos, 0 audios, 0 skipped, 0 failed || 33/33||377.22 MB/377.22 MB) ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 100% 0:00:08

I think 3.3.13 might fix this
there was a error in closing files


Current without autenticated

Dear @lazaronixon ,
thanks again for this wonderful gem. I have been playing around a bit, and i really like it.
Following up on #81 , i have another question - maybe you could help and point me into the right direction?
In my application layout, i have a navbar that shows the email-address of the currently logged in user (i.e., Current.user.email). This navbar is present in all views.
Normal pages (i.e., everything related to imprint, about, contact, ...) should be accessible for "guest" users, therefore i have the skip_before_action :authenticate in my PagesController.
However, when calling the pages#contact route, the Current.user.email is not shown, because Current.user is not present. This is, because the authenticate method is not called (i.e., i use the skip_before_action)
How can i still get the Current.user, if the user has been logged in before?
Thank you very much for helping me out. All the best,
Johannes

3 options…
1 - add a type to the user and customize the flow
2 - if it’s just the email set it as a cookie.
3 - create a guest model, and add a guest attribute to Current.rb, load it from cookies in a before_action…

Dear @lazaronixon ,
i am not sure, if i explained it correctly. Actually, i don't want some kind of Guest user (that can be "converted" into a real user, if the user signs up; as described in #75 ).
I want:

to make a controller accessible for everyone (i.e., logged in users, as well as users that are not currently logged in)
still get the Current.user, if the user has been logged in (before he moved to the page, i.e., if a cookie is set)

I guess, we could have another method, like get_session in the application_controller in order to address this issue:
# ApplicationController

before_action :get_session

def get_session
  return unless session_record = Session.find_by_id(cookies.signed[:session_token])
  Current.session = session_record
end

This way, the Current.user would work (if the user is logged in).
What do you think about this solution?
All the best,
Johannes

I think you could figure out if the same user is looking at two different URLs, but not which user is. That is separate and distinct from your authentication system, which is where current_user usually comes from, and has the responsibility for working out who you are dealing with, based on a shared secret like a username/password combo.
Everyone who interacts with a Web site (not Rails, I mean just any Web site served by any HTTP server) has a session. Rails helpfully exposes that server-side mechanism to you in the controller method session. You can write any key you want into that on one request and read it back out on another. By that key, you can identify an anonymous user from one page to another. You can't know anything about them that you didn't put into that session yourself. And if they move to a different device, or restart their computer, or even wait around long enough before returning, you cannot know that they are a returning user.
https://guides.rubyonrails.org/v7.1.0/action_controller_overview.html#session
The next step up from a session is a cookie. Those come in two flavors: temporary and permanent*. You have to set any values to them in your application, and again, Rails makes that pretty simple to do within a controller method. If you set a "permanent" cookie in one request/response loop, you can read it on the next, and be reasonably certain you have the same visitor (or at least their browser on a single device) between those two visits. This value will survive a restart of the browser or machine, but will not travel between devices, or even different browsers on the same device. The user can also erase this value at their whim, as it is stored on their computer. There's only ~56K of space in a cookie (may be more, haven't actually looked at the spec in a long time) so usually what you stash there is a key, and then any larger data you want to access about the cookie's owner is kept in your database, identified by that key.
https://api.rubyonrails.org/classes/ActionDispatch/Cookies.html
The CurrentAttributes mechanism does not do anything to keep track of the user for you. What it allows you to do is create cross-channels between the layers of a Rails application, such that your controllers (which are at the front line and have direct access to the Web server's session/cookie apparatus) can note the "current" information in a place where the models (which do not have any direct connection to an individual request) can read that and use it within their methods. All it does is give you a tidy way to refer to request-level information without passing that argument along in every method signature. It's still up to your application to put whatever information you want to be available at other layers of your application in there in the first place.
Most importantly: CurrentAttributes only sets these values for the duration of a single request/response loop. Anything you put in there on one page visit is gone after your server replies to that visit, and your application is responsible for setting it again during the next interaction.
https://api.rubyonrails.org/classes/ActiveSupport/CurrentAttributes.html
Walter
*Nothing can be set to last forever, but you can give it a significant time to live, up to 20 years. Temporary cookies are similar to sessions in terms of how your application interacts with them. They differ only in where the data is stored.

On Oct 5, 2023, at 2:36 AM, Johannes Schobel @.***> wrote:
Dear @lazaronixon ,
i am not sure, if i explained it correctly. Actually, i don't want some kind of Guest user (that can be "converted" into a real user, if the user signs up; as described in #75 ).
I want:
• to make a controller accessible for everyone (i.e., logged in users, as well as users that are not currently logged in)
• still get the Current.user, if the user has been logged in (before he moved to the page, i.e., if a cookie is set)
I guess, we could have another method, like get_session in the application_controller in order to address this issue:
ApplicationController
before_action :get_session
def get_session
return unless session_record = Session.find_by_id(cookies.signed[:session_token])
Current.session = session_record
end
This way, the Current.user would work (if the user is logged in).
What do you think about this solution?
All the best,
Johannes
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.

Dear @walterdavis ,
thank you very much for your extensive explanation and answer on this topic.
I think, having such a set_session method, as described above, solves my issue.
All the best,
Johannes

im surprised that it's not just me realize this issue @johannesschobel
when we work with another auth gem(read: devise) we can add <% if user_signed_in? %> tag(cmiiw) to the navbar section, but i've just notice that this gem basically built based on 37signals's HEY influence
then why this important? if you put bit more attention, you'll notice that HEY has 2 domains, www.hey.com ONLY FOR MARKETING and app.hey.com ONLY FOR MAIN SERVICE
with these 2 purpose, we know that authentication on app.hey.com trying to ISOLATE ENTIRE APP FROM PUBLIC EXPOSE rather than what rails programmer usually do
so if anyone plan to do something like in e-commerce site case, i suggest to add new line on main app controller to set session which can help to check if user already logged in or not


Fix issue with test exception that cannot be deserialized

Previously, a failure while deserializing an exception thrown by a test would cause the result processor proxy to blow up and fail and the test would be in an unknown state.  This changes it so that we capture failures that occur during deserialization of the exception and we replace the test exception with an exception that notifies the user that there was an exception with a given type, but we are unable to provide the full exception because it failed to deserialize when transmitting back to the build process.  In specific cases, we might be able to do something about the issue with deserialization, but in the general case, I think this is the best we can do.  We'll now at least report the test as failed and provide some minimal information to the user so that they know what exception was thrown and why we couldn't provide more.
Fixes #23602

@bot-gradle test this

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

I've triggered a build for you.

Will your fix be backported to Gradle 7 as well?

@Zordid Sorry for the late reply, for some reason I never saw your question.  We haven't done any 7.x patches since this fix was implemented, but I created a new issue to backport this to 7.x.  Assuming we do another 7.x release, we'll try to get this included.


Parsing directory

Add a functionality to parse a directory of feature files and generate the output directory of all test.js files

Added feature to parse directory


Allow import of KMZ

@bferguso I think KMZ is a zipped file of KML, and has to be extracted to get to KML file. If you are able to confirm this maybe we can close this ticket.

@emjohnst - yes that's correct. You should be able to import the unzipped KML file.


TypeScript import error in Astro integration with moduleResolution node16 or nodenext

UnoCSS version
0.51.12
Describe the bug
Getting TypeScript import error in astro.config.ts file when importing UnoCSS integration:
This expression is not callable.
  Type 'typeof import("stackblitz:/node_modules/unocss/dist/astro")' has no call signatures.(2349)

Despite the error, vite still works and the app starts normally. The issue is only during explicit tsc build. This only happens with TypeScript setting moduleResolution: "node16" (or nodenext).
I noticed that UnoCSS exports Astro integration like export { UnocssAstroIntegration as default }; when other integration that work with node16 module resolution seem to export just a default function, e.g. export default function mdx(partialMdxOptions?: Partial<MdxOptions>): AstroIntegration;.
Reproduction
https://stackblitz.com/edit/github-mrekxn?file=astro.config.ts
System Info
No response
Validations

[X] Read the Contributing Guidelines.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Same thing happens in sveltekit which also uses nodenext.


Updating to Swift 4.2

As Xcode 10.2 has been released, switching to Swift 4 is needed as it doesn't support Swift 3 anymore

I think the work is mostly done in #137 if you want to finish it.

PR https://github.com/SwiftGit2/SwiftGit2/pull/145 migrated the project to Swift 4.2, and PR https://github.com/SwiftGit2/SwiftGit2/pull/146 migrated the project to Swift 5.


Merge main into release/6.1

Merge the following PRs into release/6.1:

https://github.com/swiftlang/sourcekit-lsp/pull/1858
https://github.com/swiftlang/sourcekit-lsp/pull/1865
https://github.com/swiftlang/sourcekit-lsp/pull/1880
https://github.com/swiftlang/sourcekit-lsp/pull/1882
https://github.com/swiftlang/sourcekit-lsp/pull/1885
https://github.com/swiftlang/sourcekit-lsp/pull/1888

Do not merge the following PRs into release/6.1 because I want to qualify them on main for a little longer:

https://github.com/swiftlang/sourcekit-lsp/pull/1886

@swift-ci Please test


Avoid unexplained build break with VS2015 -- presumed compiler bug

Doesn't trigger an error on other compilers, or on newer MSVS. But this
minor rearrangement makes VS2015 happy, so let's just go with it and
not look under that rug.
Closes #2611
Signed-off-by: Larry Gritz<EMAIL_ADDRESS>

This just fixes one of the several compile errors reported in #2611. We should keep this PR open until we have fixed all of them.


[FEATURE] Making the contributors section dynamic

Describe the Feature you want to end

Making the contributors section dynamic in Readme.md
Expected behavior

The svg will look something like this with limit upto 54 contributors & will move from right to left.
Screenshots (optional)

Additional context (optional )

@MohitGupta121 Can I work on this as my first issue in this repo?

Hey @TusharKesarwani
Can you assign so I can get the work done?
Thanks.


Add JS runtime option infra and redirect-stdout-to-stderr option

Description of the change
The only functional change in this PR is adding support for -J redirect-stdout-to-stderr[=y|n] with a default of y to javy build. There are no other functional changes and behaviour without specifying that flag doesn't change since the existing code uses y for that setting implicitly.
At the code level, there's a few related changes in this PR:

Changing the static and dynamic codegen builders to use dedicated build functions instead of a single shared function so I can pass an argument to the static codegen build function and not to the dynamic build function. This involved deleting some code that is no longer referenced.
Adding a JsRuntimeOptionGroup struct, a JsRuntimeOption enum, and trait implementations to convert a Vec<JsRuntimeOption> to a JsRuntimeOptionGroup, and from a JsRuntimeOptionGroup to a javy_config::Config so we have a way of representing command line options for JS runtime settings.
For static module generation, use the JsRuntimeOptionGroup struct to generate the javy_config::Config instead of using Config::all() and pass it into the codegen struct.
For dynamic module generation, return an error if the JS runtime settings are different than the default settings.
Implement Default for the CodegenOptionGroup and use the defaults for converting from Vec<CodegenOption> to CodegenOptionGroup to keep it consistent with how JsRuntimeOptionGroup works.
Update the testing infrastructure to work with the new argument.
Update the testing infrastructure so we can inspect stderr from Javy when a compile or build command fails.

Why am I making this change?
See #702. I'm implementing this similarly to how I did for the codegen options. I'm also opting to return an error when generating a dynamic module if any of the JS runtime options are set to non-default values since those runtime options won't be applied to the generated module (the provider defines the runtime options). The refactoring of the codegen builder was because I wanted to try to make invalid state non-representable in the non-test code.
I can probably break this PR up if it's too large. I also tried avoiding any functional changes that I didn't have to so the PR could be kept smaller. Hence not bumping the provider version yet.
Checklist

[x] I've updated the relevant CHANGELOG files if necessary. Changes to javy-cli and javy-core do not require updating CHANGELOG files.
[x] I've updated the relevant crate versions if necessary. Versioning policy for library crates
[x] I've updated documentation including crate documentation if necessary.

The existing fuel consumption number on main is 37,309 for the log test is when compiled run on my Macbook.


Support notice

Add a note saying SB no longer actively updates rabix composer

@bogdang989 I think we should also add:
"If you need an up-to-date CWL editor composer, you can create an account either on the CGC http://cgc.sbgenomics.com/, Biodata Catalyst powered by Seven Bridges https://platform.sb.biodatacatalyst.nhlbi.nih.gov/, or on CAVATICA https://cavatica.sbgenomics.com/ and use the Web Composer available there. The Web Composer is also available on the Seven Bridges Commercial offerings"


How to use it to predict an RGB video？

Hi there, thanks for your interest! This repo only provides the code for skeleton-based action recognition, so it won't work with RGB videos directly. You could extend it with toolkits such as https://github.com/CMU-Perceptual-Computing-Lab/openpose that extracts skeletons from RGB videos.


installation of pycups fails -> python setup.py bdist_wheel did not run successfully.

Problem/Motivation

Fails to install pycups as a python package.
Getting the error: python setup.py bdist_wheel did not run successfully.

Expected behavior

(What you expected to happen)

Actual behavior

Collecting pycups
Downloading pycups-2.0.1.tar.gz (62 kB)
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 63.0/63.0 kB 4.2 MB/s eta 0:00:00
Preparing metadata (setup.py): started
Preparing metadata (setup.py): finished with status 'done'
Building wheels for collected packages: pycups
Building wheel for pycups (setup.py): started
Building wheel for pycups (setup.py): finished with status 'error'
error: subprocess-exited-with-error
× python setup.py bdist_wheel did not run successfully.
│ exit code: 1
╰─> [6 lines of output]
In file included from cupsconnection.c:22:
cupsconnection.h:25:10: fatal error: Python.h: No such file or directory
25 | #include <Python.h>
|          ^~~~~~~~~~
compilation terminated.
error: command '/usr/bin/gcc' failed with exit code 1
[end of output]

Steps to reproduce

(How can someone else make/see it happen)

Install AppDaemon add-on

Configure add-on and add the following python packages: cmake, wheel, pycups
add the following system packages:
cups-dev
gcc
libcrypto3
libssl3

Tried to force upgrade all packages on run time

Proposed changes

(If you have a proposed change, workaround or fix,
describe the rationale behind it)

Looking at the documentation of pycups, you probably also need to add python3-dev to the system packages.

It's a bit of a wacamole, I need to find a combination of packages that just works
ERROR: unable to select packages:
  python3-3.10.9-r1:
    breaks: python3-dev-3.10.12-r0[python3=3.10.12-r0]
    satisfies: world[python3=3.10.9-r1]
               glib-dev-2.74.7-r0[python3]
               py3-pip-22.3.1-r1[python3]
               py3-setuptools-65.6.0-r0[python3]
               py3-parsing-3.0.9-r0[python3]
               py3-packaging-21.3-r2[python3]
               py3-wheel-0.38.4-r0[python3]
               py3-retrying-1.3.3-r3[python3]
               py3-six-1.16.0-r3[python3]

Which version (of the add-on) are you running? Normally it ships with python 3.11.

Running 0.11.0, no available updates yet but I see the latest tag is quite a bit ahead

used a different repo and downloaded the latest version of the package. resolved the issue


Add issue template

Added the issue template #1066

I agree with @jrfnl. I find that issue templates can be annoying and overwhelming at times. I think that keeping it simple is good.
In this case, I think it is important to remember that we're dealing with developers here, not non-techie users of a piece of software. We probably don't need to walk them through how to submit a bug report or request a feature. They likely deal with software features and bugs every day. 😄 The structure for reports that is provided by the different sections is nice, but I don't think it's needed/worth it in this case.
I think that really all we need to do is mention the extra bits of information that are helpful specifically for bug reports for sniffs. Something like:
<!--
If you're submitting a bug report, it is helpful if you include the following information:

- A minimal snippet of example code that demonstrates the issue.
- The error code for the sniff that is being triggered (you can see the sniff error codes by running `phpcs` with the `-s` flag).
- The versions of PHPCS and PHP you are using.

Thanks!
-->

I think it is important to remember that we're dealing with developers here, not non-techie users of a piece of software

Not necessarily so once the theme review team switches over to their new process. For a lot of theme devs WPCS is just a piece of software and well outside their wheelhouse.

@jrfnl  True, something like that went through my mind, but I still contend that we probably don't need to give them this much generic instruction about how to write a feature request or bug report. I don't think it is as useful in our case. Mainly just the information specific to WPCS.

@JDGrimes I'm totally with you on that and I completely agree that something along the lines of your example + maybe PHPCS/WPCS/PHP version would suffice. I just wanted to add a little nuance about our users.

I will edit this when I get home :) Thanks for the feedback

Will this be ok?
<!--
If you're submitting a bug report, it is helpful if you include the following information:

- A minimal snippet of example code that demonstrates the issue.
- The error code for the sniff that is being triggered (you can see the sniff error codes by running `phpcs` with the `-s` flag).
- The versions of PHPCS, WPCS and PHP you are using.

If you are submitting a feature describe what that feature would be and why it should be useful.

Thanks!
-->


Add example to alarm for doing a step like every 2 hours

Based on some user feedback on #openwhisk Slack channel, it will be good to have an example on specifying steps in the docs using cron syntax.
Based on crontab.org
Step  values can be used in conjunction with ranges.  Fol-
             lowing a range with ``/<number>'' specifies skips  of  the
             number's value through the range.  For example, ``0-23/2''
             can be used in the hours field to specify  command  execu-
             tion  every other hour (the alternative in the V7 standard
             is ``0,2,4,6,8,10,12,14,16,18,20,22'').   Steps  are  also
             permitted after an asterisk, so if you want to say ``every
             two hours'', just use ``*/2''.

Example to add

* * */2 * * *: every 2 hours.

No Fixed by #1551


Spanner Enhancements

This PR adds retry to streams responses that receive an ABORTED error. Responses are buffered while waiting for a resume_token. Values will be returned when the buffer has reached its upper bound.
closes #1236

@quartzmo I have updated this PR with the bounds check matching the node implementation. I have removed the transaction retry and header changes and will re-submit them in separate PRs. Can you please review again?


EGVSR model leaving a grid pattern?

Sorry to bring up another apparent issue....

Had to scale it down to attach, but there's a grid-like pattern and overall the result looks quite odd.
I read over on doom9 that you said:

Please note though, that EGVSR does not auto-detect settings from the model file yet. So you should manually override any settings that needs it when you load the model for correct results.

and so I assume this result is due to not doing that and simply using:
from vsgan import EGVSR
clip = EGVSR(clip, device="cuda").\
load(r"C:\VapourSynth\vsgan_models\EGVSR\EGVSR_iter420000.pth").\
apply().\
clip
isn't going to work. But I have no idea how to override it to get the correct results?

Network settings are not automatically detected from EGVSR model files at the moment.
That model expects the settings: nb=10, degradation="BD" in load.

Thanks, that took care of that grid stuff. Though oddly without the grid, I'm getting a crash if I go a few frames into the sequence. Seem to not be getting as great results from EGVSR as I had expected based on their examples, but that's not on you.

I'm getting a crash if I go a few frames into the sequence.
Try setting the max VapourSynth threads to 1 to make sure you aren't getting more than (interval+1) frames every frame.
Since you super-res not 1, but 1+n(interval) frames every frame, the VRAM requirements are quite high.
On my system I barely get interval=3 or 5 to work and I have 11 GiB. Sadly, I don't know if there's some kind of memory leak or something as I don't really know what performance would be expected by other people's inference code.

Regardless, I try my absolute best to get as much performance with as little VRAM as possible. I've since done a few commits to try to help as much as I can, which will be out in the next version. It might not help, as simply VRAM requirements are quite high. But do try setting the max threads to 1 via core.num_threads = 1 as it will definitely help. It will automatically do this when you use EGVSR in the next version.
P.S., on the latest commit, through vs-pipe, with interval=3 on a 720x480 RGB24 input, I got 6.0 fps, which climbed from 4.5~ to 6 over about a minute. It might have even grown further than 6 fps but the video finished rendering just as it reached it.

v1.6.4 fixes a big memory leak. This will definitely help reduce your VRAM. It reduced mine from about 9.6 GiB to 2.6 GiB with interval=3. I can now go as far as interval=30 and reach 10.8 GiB/11 GiB.

Going to close this now as all questions seem to be addressed.


Population data from gridded cell rather than own NUTS3 interpolation

https://www.worldpop.org/geodata/summary?id=24777

Estimated total number of people per grid-cell. The dataset is available to download in Geotiff format at a resolution of 30 arc (approximately 1km at the equator). The projection is Geographic Coordinate System, WGS84. The units are number of people per pixel. The mapping approach is Random Forest-based dasymetric redistribution.

License
CC-BY-4.0
API Interface
with REST API
https://www.worldpop.org/sdi/introapi
Methods
https://www.worldpop.org/methods

Hi @skortmann, great! It indeed targets the https://github.com/PyPSA/pypsa-eur-sec/blob/master/scripts/build_population_layouts.py function. The motivation to use the raster file instead of the NUTS3 population data is that the interpolation between NUTS3 and clustered model regions may be quite inaccurate (population is assigned proportionally to overlapping area currently) and it may simplify the code (as it doesn't need to interpolation, just aggregation). Still it would need to disentangle rural from urban population. It would also be important to understand how NUTS3 interpolation and worldpop raster data differ.
I'd be happy to review a pull request if you want to have a first shot at replacing the dataset.


Indent open brace for impl when nested.

Fixes #1226

Thanks for the fix! Could you add a test please?

Done

Thanks!


Mouse Event memory leak

Stumbled across this today while going through some memory stuff on gl-engine. Haven't looked into it further, but it looks like MouseEvent is being retained somewhere. There's 1414 objects there.

This may be a Firefox thing. Checking it out.


New CruX metrics

Hey,
Are there plans to update this to support new metrics in CruX such as cumulative layout shift and largest contentful paint?

Yes we're looking into adding First Input Delay , CLS and LCP in a (near term) release.
We're working with the Google team on the exact key metrics to bring in (and they'll be in the API first then into the UI shortly after).
Thanks for the question!
Sent via Superhuman<EMAIL_ADDRESS>)
On Wed, May 27, 2020 at 10:28 AM, samunderwood <<EMAIL_ADDRESS>> wrote:

Hey,
Are there plans to update this to support new metrics in CruX such as
cumulative layout shift and largest contentful paint?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub (
https://github.com/edgemesh/crux.run/issues/10 ) , or unsubscribe (
https://github.com/notifications/unsubscribe-auth/AE6P44QIGS25SRMJWGX4K6TRTVEUXANCNFSM4NMLCPTA
).

Awesome! Another question, in the UI how are you categorising FCP as fast, average and slow? Strugglign to find any pre-defined metrics for this categorisation?

On Wednesday, May 27, 2020 at 6:47 pm, Jacob Loveless<EMAIL_ADDRESS>(mailto:notifications@github.com)> wrote:
Yes we're looking into adding First Input Delay , CLS and LCP in a (near term) release.
We're working with the Google team on the exact key metrics to bring in (and they'll be in the API first then into the UI shortly after).
Thanks for the question!
Sent via Superhuman<EMAIL_ADDRESS>)
On Wed, May 27, 2020 at 10:28 AM, samunderwood <<EMAIL_ADDRESS>> wrote:

Hey,
Are there plans to update this to support new metrics in CruX such as
cumulative layout shift and largest contentful paint?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub (
https://github.com/edgemesh/crux.run/issues/10 ) , or unsubscribe (
https://github.com/notifications/unsubscribe-auth/AE6P44QIGS25SRMJWGX4K6TRTVEUXANCNFSM4NMLCPTA
).

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub (https://github.com/edgemesh/crux.run/issues/10#issuecomment-634830768), or unsubscribe (https://github.com/notifications/unsubscribe-auth/ADVITCWF3N5WZMRF3EDSI6LRTVG3DANCNFSM4NMLCPTA).

You know … we sure don't!
 Will get that added as a route!
Sent via Superhuman<EMAIL_ADDRESS>)
On Thu, May 28, 2020 at 2:29 AM, samunderwood <<EMAIL_ADDRESS>> wrote:

Awesome! Another question, in the UI how are you categorising FCP as fast,
average and slow? Strugglign to find any pre-defined metrics for this
categorisation?

On Wednesday, May 27, 2020 at 6:47 pm, Jacob Loveless < notifications@ github.
com<EMAIL_ADDRESS>) (mailto: notifications@ github. com (
<EMAIL_ADDRESS>) )> wrote:
Yes we're looking into adding First Input Delay , CLS and LCP in a (near
term) release.
We're working with the Google team on the exact key metrics to bring in
(and they'll be in the API first then into the UI shortly after).
Thanks for the question!
Sent via Superhuman ( https:/ / sprh. mn/ ?vip=jacob. loveless@ edgemesh.
com<EMAIL_ADDRESS>) )
On Wed, May 27, 2020 at 10:28 AM, samunderwood < notifications@ github. com
<EMAIL_ADDRESS>) > wrote:

Hey,
Are there plans to update this to support new metrics in CruX such as
cumulative layout shift and largest contentful paint?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub (
https:/ / github. com/ edgemesh/ crux. run/ issues/ 10 (
https://github.com/edgemesh/crux.run/issues/10 ) ) , or unsubscribe (
https:/ / github. com/ notifications/ unsubscribe-auth/ AE6P44QIGS25SRMJWGX4K6TRTVEUXANCNFSM4NMLCPTA
(
https://github.com/notifications/unsubscribe-auth/AE6P44QIGS25SRMJWGX4K6TRTVEUXANCNFSM4NMLCPTA
)
).

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub ( https:/ / github. com/
edgemesh/ crux. run/ issues/ 10#issuecomment-634830768 (
https://github.com/edgemesh/crux.run/issues/10#issuecomment-634830768 ) ),
or unsubscribe ( https:/ / github. com/ notifications/ unsubscribe-auth/ ADVITCWF3N5WZMRF3EDSI6LRTVG3DANCNFSM4NMLCPTA
(
https://github.com/notifications/unsubscribe-auth/ADVITCWF3N5WZMRF3EDSI6LRTVG3DANCNFSM4NMLCPTA
) ).

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub (
https://github.com/edgemesh/crux.run/issues/10#issuecomment-635229291 ) ,
or unsubscribe (
https://github.com/notifications/unsubscribe-auth/AE6P44SIJUD2QJWTDLCZZVTRTYVJFANCNFSM4NMLCPTA
).

Awesome, thanks!

On Thursday, May 28, 2020 at 7:31 pm, Jacob Loveless<EMAIL_ADDRESS>(mailto:notifications@github.com)> wrote:
You know … we sure don't!
 Will get that added as a route!
Sent via Superhuman<EMAIL_ADDRESS>)
On Thu, May 28, 2020 at 2:29 AM, samunderwood <<EMAIL_ADDRESS>> wrote:

Awesome! Another question, in the UI how are you categorising FCP as fast,
average and slow? Strugglign to find any pre-defined metrics for this
categorisation?

On Wednesday, May 27, 2020 at 6:47 pm, Jacob Loveless < notifications@ github.
com<EMAIL_ADDRESS>) (mailto: notifications@ github. com (
<EMAIL_ADDRESS>) )> wrote:
Yes we're looking into adding First Input Delay , CLS and LCP in a (near
term) release.
We're working with the Google team on the exact key metrics to bring in
(and they'll be in the API first then into the UI shortly after).
Thanks for the question!
Sent via Superhuman ( https:/ / sprh. mn/ ?vip=jacob. loveless@ edgemesh.
com<EMAIL_ADDRESS>) )
On Wed, May 27, 2020 at 10:28 AM, samunderwood < notifications@ github. com
<EMAIL_ADDRESS>) > wrote:

Hey,
Are there plans to update this to support new metrics in CruX such as
cumulative layout shift and largest contentful paint?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub (
https:/ / github. com/ edgemesh/ crux. run/ issues/ 10 (
https://github.com/edgemesh/crux.run/issues/10 ) ) , or unsubscribe (
https:/ / github. com/ notifications/ unsubscribe-auth/ AE6P44QIGS25SRMJWGX4K6TRTVEUXANCNFSM4NMLCPTA
(
https://github.com/notifications/unsubscribe-auth/AE6P44QIGS25SRMJWGX4K6TRTVEUXANCNFSM4NMLCPTA
)
).

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub ( https:/ / github. com/
edgemesh/ crux. run/ issues/ 10#issuecomment-634830768 (
https://github.com/edgemesh/crux.run/issues/10#issuecomment-634830768 ) ),
or unsubscribe ( https:/ / github. com/ notifications/ unsubscribe-auth/ ADVITCWF3N5WZMRF3EDSI6LRTVG3DANCNFSM4NMLCPTA
(
https://github.com/notifications/unsubscribe-auth/ADVITCWF3N5WZMRF3EDSI6LRTVG3DANCNFSM4NMLCPTA
) ).

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub (
https://github.com/edgemesh/crux.run/issues/10#issuecomment-635229291 ) ,
or unsubscribe (
https://github.com/notifications/unsubscribe-auth/AE6P44SIJUD2QJWTDLCZZVTRTYVJFANCNFSM4NMLCPTA
).

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub (https://github.com/edgemesh/crux.run/issues/10#issuecomment-635521145), or unsubscribe (https://github.com/notifications/unsubscribe-auth/ADVITCVBCF5XSMVBIY5XMIDRT2UXPANCNFSM4NMLCPTA).

Is there any ETA for when you think you'll be able to add the new CrUX metrics to the API? 😊

Not the next release (May) but the one after (June).
Sent via Superhuman iOS<EMAIL_ADDRESS>)
On Thu, Jun 4 2020 at 7:54 AM, samunderwood <<EMAIL_ADDRESS>> wrote:

Is there any ETA for when you think you'll be able to add the new CrUX
metrics to the API? 😊
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub (
https://github.com/edgemesh/crux.run/issues/10#issuecomment-638902922 ) ,
or unsubscribe (
https://github.com/notifications/unsubscribe-auth/AE6P44QSPQVX6ZA76JIG2TDRU6YTPANCNFSM4NMLCPTA
).

This still planned for release this month? 😊

working on the rewrite of the parser as we speak!

Awesome, I have a blog post prepped for when this goes live to show how to use it with Google Sheets 😊


 Grant Expiry time should not be hard-coded and shall be handled similar to transmitExpiryTime

As per yesterday's demo on MOCK SAS V3.0, Grant Expiry time was hard-coded in xml files at CBRSConf folder. It will be a good option if Grant Expiry time can be handled similar to transmitExpiryTime, so that "grant renew as true" case can be handled correctly where MOCK SAS need to send a new time for Grant Expiry time.

Issue resolved as patyt of Mock-SAS updates uploaded 16-Aug-2017

Mock-SAS code uploaded 1-September-2017 addresses this according to section 11.7 in Mock-SAS tutorial “Handling of Grant Expiry Time, Transmission Expiry Time and Grant Renew”.

@saudawar please update

Verified. Now grant expiry is handled same as transmitExpiry with latest Mock SAS version.


Unable to handle TaskCanceledExceptions in HttpForwarder.SendAsync

Describe the bug
During load testing we see a lot of HTTP requests being cancelled, presumably by a client due to a timeout. Our API servers react to this by ignoring the corresponding Task-/OperationCanceledExceptions and setting the response code to 499. The gateway (implementing YARP) however still throws an unhandled Exception:

[2024-03-19 04:14:06] warn: Yarp.ReverseProxy.Forwarder.HttpForwarder[48]
RequestCanceled: The request was canceled before receiving a response.
System.Threading.Tasks.TaskCanceledException: The operation was canceled.
---> System.IO.IOException: Unable to read data from the transport connection: Operation canceled.
---> System.Net.Sockets.SocketException (125): Operation canceled
--- End of inner exception stack trace ---
at System.Net.Sockets.Socket.AwaitableSocketAsyncEventArgs.ThrowException(SocketError error, CancellationToken cancellationToken)
at System.Net.Sockets.Socket.AwaitableSocketAsyncEventArgs.System.Threading.Tasks.Sources.IValueTaskSource<System.Int32>.GetResult(Int16 token)
at System.Net.Http.HttpConnection.InitialFillAsync(Boolean async)
at System.Net.Http.HttpConnection.SendAsync(HttpRequestMessage request, Boolean async, CancellationToken cancellationToken)
--- End of inner exception stack trace ---
at System.Net.Http.HttpConnection.SendAsync(HttpRequestMessage request, Boolean async, CancellationToken cancellationToken)
at System.Net.Http.HttpConnectionPool.SendWithVersionDetectionAndRetryAsync(HttpRequestMessage request, Boolean async, Boolean doRequestAuth, CancellationToken cancellationToken)
at System.Net.Http.DiagnosticsHandler.SendAsyncCore(HttpRequestMessage request, Boolean async, CancellationToken cancellationToken)
at System.Net.Http.Metrics.MetricsHandler.SendAsyncWithMetrics(HttpRequestMessage request, Boolean async, CancellationToken cancellationToken)
at Yarp.ReverseProxy.Forwarder.HttpForwarder.SendAsync(HttpContext context, String destinationPrefix, HttpMessageInvoker httpClient, ForwarderRequestConfig requestConfig, HttpTransformer transformer, CancellationToken cancellationToken)

This exception seems to be thrown outside of our middleware pipeline as we're unable to handle it. Ideally we would like to react the same way our API servers do, ignoring the exception, and setting the HTTP response code to 499.
Yarp setup:
  services.AddReverseProxy().LoadFromMemory(GetRoutes(), GetClusters());
...
  app.MapReverseProxy(x => x.Use(HandleSafelyAsync)).RequireAuthorization();

  public static async Task HandleSafelyAsync(HttpContext context, Func<Task> next)
  {
      try
      {
          await next();
      }
      catch (Exception ex)
      {
          // this is never called
          var logger = context.RequestServices.GetRequiredService<ILogger<ExceptionHandlingMiddleware>>();
          logger.LogError(ex, "An error occurred while processing '{Route}'. HTTP {StatusCode}, RequestAborted: {RequestAborted}", context.Request.Path, context.Response.StatusCode, context.RequestAborted.IsCancellationRequested);
      }
  }

We're using Yarp.ReverseProxy version 2.1.0.

Looks like you ran into #2195.
These exceptions aren't uncaught (hence you don't see the catch triggering for them), but they are logged by YARP. Since cancellation errors are often outside of your control (e.g. client disconnects), this mainly leads to noise in logs, and #2195 tracks improving that.

Closing as a duplicate of #2195


Should a message of default values be removed after encode?

For example, the proto is:
message User { uint32 id = 1; }

A User of id 0 is as if a non-specified one after encode:
iex> u = User.new id: 0
%User{id: 0}
iex> User.encode u
""
iex> u = User.new
%User{id: 0}
iex> User.encode u
""

I think sometimes a explicit User of id 0 is needed, exprotobuf does so:
iex> u = User.new id: 0
%User{id: 0}
iex> User.encode u
<<8, 0>>
iex> u = User.new
%User{id: nil}
iex> User.encode u
""

It's not a bug. In proto3, you can tell wether a field is set to zero value or not set because there're default values for basic types(https://developers.google.com/protocol-buffers/docs/proto3#default). If you want to distinguish this, you can use proto2. btw, you should take an unofficial implementation(exprotobuf, gpb) as example. The official implementations(C++, Java, Go..) behave like what I do.


OCPBUGS-22854: The microshift-release-info RPM is no longer required and it contains sample blueprints

Also added code to the spec file to declare all the directories under /usr/share/microshift to make sure they are cleaned up on uninstall.
$ ls -Rl /usr/share/microshift/
/usr/share/microshift/:
total 0
drwxr-xr-x. 2 root root 65 Nov 21 14:04 blueprint
drwxr-xr-x. 2 root root 26 Nov 21 14:04 functions
drwxr-xr-x. 2 root root 61 Nov 21 14:04 release
drwxr-xr-x. 2 root root 38 Nov 21 14:04 spec

/usr/share/microshift/blueprint:
total 8
-rw-r--r--. 1 root root 3443 Nov 21 14:00 blueprint-aarch64.toml
-rw-r--r--. 1 root root 3441 Nov 21 14:00 blueprint-x86_64.toml

/usr/share/microshift/functions:
total 12
-rw-r--r--. 1 root root 8855 Nov 21 11:33 greenboot.sh

/usr/share/microshift/release:
total 8
-rw-r--r--. 1 root root 2469 Nov 21 11:19 release-aarch64.json
-rw-r--r--. 1 root root 2463 Nov 21 11:19 release-x86_64.json

/usr/share/microshift/spec:
total 8
-rw-r--r--. 1 root root 5002 Nov 21 11:54 config-openapi-spec.json

/retest-required

/retest-required

/test microshift-metal-tests

/retest-required

/test metal-periodic-test-arm microshift-metal-tests-arm ocp-conformance-rhel-eus-arm

/test metal-periodic-test-arm
/lgtm

/override ci/prow/metal-periodic-test-arm
The failure has nothing to do with the current change


Fix vmTemplate latestVersion isn't updated correctly when set the new created templateVersion as default

genclient:skipVerbs=update just control whether we generate update clientset or not, can't forbid the PUT API, so we get this back for testing.
Issue:
https://github.com/rancher/harvester/issues/121
https://github.com/rancher/harvester/issues/118

updated


Fixing the display text so it shows correctly

Display text now displays correctly for both snippeted and no snippeted response

Need to test if that conditional is needed in there and please squash this into a single commit.

Looks good to me. Cheers!


Update Pages “about”

Automatically generated by Netlify CMS

👷 Deploy Preview for unruffled-curie-30a9c9 processing.
🔨 Explore the source changes: 03cca70b8811b5dfdb0e3ae9b47a8fdb71f9bb90
🔍 Inspect the deploy log: https://app.netlify.com/sites/unruffled-curie-30a9c9/deploys/62009187eca05f0008347e32


Registration with email and log in.

When I create an email account on an Android device, i am not able to use that same account for a future log in with an iOS device, and vice versa.

Hi @besidesordinary
Can you please provide your flutter doctor -v,
your flutter run --verbose while attempting to register with email,
your pubspec.yaml and a minimal reproducible code sample.
Thank you

Hi @iapicca
This is what I get when i try to sign in from an iOS simulator, with account that has been created on an Android simulator (i would also like to mention that i also tried and tested with real devices)
Syncing files to device iPhone SE (2nd generation)...
Reloaded 0 of 737 libraries in 401ms.
flutter: PlatformException(ERROR_WRONG_PASSWORD, The password is invalid or the user does not have a password., null)
Here is  flutter doctor -v
ivan@Ivans-Air-3 pss_app % flutter doctor -v
[✓] Flutter (Channel dev, 1.19.0-5.0.pre, on Mac OS X 10.15.5 19F101, locale en-MK)
• Flutter version 1.19.0-5.0.pre at /Users/ivan/Documents/development/flutter
• Framework revision c264b70ec3 (11 days ago), 2020-06-08 09:27:34 -0700
• Engine revision c7f9725521
• Dart version 2.9.0 (build 2.9.0-13.0.dev 87c5e3612b)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
• Android SDK at /Users/ivan/Library/Android/sdk
• Platform android-29, build-tools 29.0.3
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 11.5)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 11.5, Build version 11E608c
• CocoaPods version 1.9.1
[✓] Android Studio (version 4.0)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 46.0.2
• Dart plugin version 193.7361
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
[✓] Connected device (1 available)
• iPhone SE (2nd generation) • 7DC3E83F-BA2E-4C4F-BBD9-E7ACA75110DE • ios • com.apple.CoreSimulator.SimRuntime.iOS-13-5 (simulator)
• No issues found!
And flutter run --verbose
[+2348 ms] executing: /usr/bin/xcrun simctl install 7DC3E83F-BA2E-4C4F-BBD9-E7ACA75110DE
/Users/ivan/Documents/development/pss_app/build/ios/iphonesimulator/Runner.app
[+3075 ms] executing: /usr/bin/plutil -convert json -o - /Users/ivan/Documents/development/pss_app/build/ios/iphonesimulator/Runner.app/Info.plist
[  +40 ms] Exit code 0 from: /usr/bin/plutil -convert json -o -
/Users/ivan/Documents/development/pss_app/build/ios/iphonesimulator/Runner.app/Info.plist
[   +2 ms]
{"DTCompiler":"com.apple.compilers.llvm.clang.1_0","CFBundleInfoDictionaryVersion":"6.0","UISupportedInterfaceOrientations~ipad":["UIInterfaceOrien
tationPortrait","UIInterfaceOrientationPortraitUpsideDown","UIInterfaceOrientationLandscapeLeft","UIInterfaceOrientationLandscapeRight"],"DTPlatfor
mVersion":"13.5","CFBundleName":"nexthop.pss","DTSDKName":"iphonesimulator13.5","UIViewControllerBasedStatusBarAppearance":false,"CFBundleIcons":{"
CFBundlePrimaryIcon":{"CFBundleIconFiles":["AppIcon60x60"],"CFBundleIconName":"AppIcon"}},"NSLocationAlwaysAndWhenInUseUsageDescription":"This app
needs access to location when open and in the
background.","LSRequiresIPhoneOS":true,"LSApplicationQueriesSchemes":["fbapi","fbapi20130214","fbapi20130410","fbapi20130702","fbapi20131010","fbap
i20131219","fbapi20140410","fbapi20140116","fbapi20150313","fbapi20150629","fbapi20160328","fbauth","fb-messenger-share-api","fbauth2","fbshareexte
nsion"],"NSLocationWhenInUseUsageDescription":"This app needs access to location when
open.","DTSDKBuild":"17F65","CFBundleShortVersionString":"1.0.0","CFBundleSupportedPlatforms":["iPhoneSimulator"],"NSLocationAlwaysUsageDescription
":"This app needs access to location when in the
background.","UISupportedInterfaceOrientations":["UIInterfaceOrientationPortrait","UIInterfaceOrientationLandscapeLeft","UIInterfaceOrientationLand
scapeRight"],"BuildMachineOSBuild":"19F101","DTPlatformBuild":"17F65","CFBundlePackageType":"APPL","DTXcodeBuild":"11E608c","CFBundleDevelopmentReg
ion":"en","MinimumOSVersion":"12.0","CFBundleVersion":"1","FacebookAppID":"2894734743946732","UIDeviceFamily":[1,2],"UILaunchStoryboardName":"Launc
hScreen","CFBundleIdentifier":"nexthop.pss","FacebookDisplayName":"PSS","DTXcode":"1150","CFBundleExecutable":"Runner","CFBundleIcons~ipad":{"CFBun
dlePrimaryIcon":{"CFBundleIconFiles":["AppIcon60x60","AppIcon76x76"],"CFBundleIconName":"AppIcon"}},"CFBundleSignature":"????","UIMainStoryboardFil
e":"Main","DTPlatformName":"iphonesimulator","CFBundleURLTypes":[{"CFBundleTypeRole":"Editor","CFBundleURLSchemes":["fb2894734743946732","com.googl
eusercontent.apps.20198450506-afk5tdijp3joo8ouqrd1d1doct3di0pe"]}]}
[  +18 ms] executing: /usr/bin/xcrun simctl launch 7DC3E83F-BA2E-4C4F-BBD9-E7ACA75110DE nexthop.pss --enable-dart-profiling --enable-checked-mode
--verify-entry-points --observatory-port=0
[+1274 ms] nexthop.pss: 6167
[   +1 ms] Waiting for observatory port to be available...
[+1733 ms] [Firebase/Crashlytics] Version 4.0.0-beta.7
[  +71 ms] [Crashlytics] Version 3.14.0 (144)
[   +9 ms] [Fabric] [Fabric +with] called multiple times. Only the first call is honored, please pass all kits you wish to initialize
[ +408 ms] Observatory URL on device: http://<IP_ADDRESS>:53443/ExhwD9EDcUE=/
[  +20 ms] Caching compiled dill
[ +114 ms] Connecting to service protocol: http://<IP_ADDRESS>:53443/ExhwD9EDcUE=/
[ +128 ms] [Fabric] failed to download settings Error Domain=FABNetworkError Code=-5 "(null)" UserInfo={status_code=403, type=2, request_id=,
content_type=application/json; charset=utf-8}
[ +457 ms] Successfully connected to service protocol: http://<IP_ADDRESS>:53443/ExhwD9EDcUE=/
[        ] Waiting for iPhone SE (2nd generation) to report its views...
[   +9 ms] Waiting for iPhone SE (2nd generation) to report its views... (completed in 9ms)
[  +20 ms] DevFS: Creating new filesystem on the device (null)
[  +28 ms] DevFS: Created new filesystem on the device
(file:///Users/ivan/Library/Developer/CoreSimulator/Devices/7DC3E83F-BA2E-4C4F-BBD9-E7ACA75110DE/data/Containers/Data/Application/6D318E8E-6173-4D1
A-93A3-9FB20637D197/tmp/pss_appg5p8RQ/pss_app/)
[   +7 ms] Updating assets
[+1168 ms] Syncing files to device iPhone SE (2nd generation)...
[   +4 ms] Scanning asset files
[  +18 ms] <- reset
[        ] Compiling dart to kernel with 0 updated files
[   +6 ms] <- recompile package:pss_app/main.dart 8dd5c73e-cf0d-4c08-8214-5629dfbb55af
[   +1 ms] <- 8dd5c73e-cf0d-4c08-8214-5629dfbb55af
[   +2 ms] [Crashlytics:Crash] Warning: NSUncaughtExceptionHandler is 'FBSDKExceptionHandler' in
'/Users/ivan/Library/Developer/CoreSimulator/Devices/7DC3E83F-BA2E-4C4F-BBD9-E7ACA75110DE/data/Containers/Bundle/Application/3832A469-978B-4E7A-ACF
D-DB242B7947D4/Runner.app/Frameworks/FBSDKCoreKit.framework/FBSDKCoreKit'
[ +160 ms] Updating files
[ +772 ms] DevFS: Sync finished
[        ] Syncing files to device iPhone SE (2nd generation)... (completed in 965ms)
[   +1 ms] Synced 3.5MB.
[   +2 ms] <- accept
[   +3 ms] Connected to _flutterView/0x7fca6c01b620.
[   +6 ms] Flutter run key commands.
[   +7 ms] r Hot reload. 🔥🔥🔥
[   +2 ms] R Hot restart.
[   +1 ms] h Repeat this help message.
[   +1 ms] d Detach (terminate "flutter run" but leave application running).
[   +1 ms] c Clear the screen
[        ] q Quit (terminate the application on the device).
[        ] An Observatory debugger and profiler on iPhone SE (2nd generation) is available at: http://<IP_ADDRESS>:53443/ExhwD9EDcUE=/
[+648742 ms] flutter: PlatformException(ERROR_WRONG_PASSWORD, The password is invalid or the user does not have a password., null)
Code sample:
final AuthService _auth = AuthService();
final _formKey1 = GlobalKey();
final _formKey2 = GlobalKey();
String email = '';
String password = '';
String error = '';
bool loading = false;
bool _ObscureText = true;
RaisedButton(
elevation: 0.0,
onPressed: () async {
if ( (_formKey1.currentState.validate()) && (_formKey2.currentState.validate()) ){
setState(() => loading = true);
dynamic result = await _auth.signInWithEmailAndPassword(email, password);
if (result == null) {
_auth.signOut();
setState(() {
loading = false;
error = '\nWrong email or password';
});
}
if(loading == true) {
Navigator.pushReplacement(
context,
MaterialPageRoute(builder: (context) => Wrapper()),
);
}
}
},
padding: EdgeInsets.all(15.0),
color: Colors.white,
child: SizedBox(
width: 500,
child: Text(
'Enter',
style: TextStyle(
color: Colors.red[400],
letterSpacing: 1.5,
fontSize: 18.0,
fontWeight: FontWeight.bold,
fontFamily: 'OpenSans',
),
textAlign: TextAlign.center,
),
),
),
Text(
error,
style: TextStyle(color: Colors.black, fontSize: 14.0),
textAlign: TextAlign.center,
)
pubspec.yaml
cupertino_icons: ^0.1.2
firebase_core: ^0.4.4+3
firebase_analytics: ^5.0.11
firebase_auth: ^0.16.0
google_sign_in: ^4.4.4
flutter_facebook_login: ^3.0.0
flutter_auth_buttons: ^0.8.0
firebase_database: ^3.1.5
provider: ^4.0.5+1
http: ^0.12.1
flutter_spinkit: ^4.1.2+1
firebase_crashlytics: ^0.1.3+3
slider_button: ^0.5.0
table_calendar: ^2.2.3
location: ^3.0.2
geolocator: ^5.3.1
fluttertoast: ^4.0.1
intl: ^0.16.1
device_info: any
shared_preferences: ^0.5.7
http_interceptor: any
flutter_picker: ^1.1.5
flutter_datetime_picker: ^1.3.8
package_info: ^0.4.1
Thank you!
Best regards
besidesordinary

Hi @besidesordinary
the issue might be caused by your implementation rather than a bug in flutterfire plugins
you may want to look at this possible solution
Closing, as this isn't an issue with FlutterFire itself. If you disagree, please write in the comments and I will reopen it.
Thank you


FR: Need a good way to inject secrets besides gcloud & GOOGLE_APPLICATION_CREDENTIALS

PR's #94 & #144 both want to find a way to improve ADC for non-google cloud providers.
The ADC algorithm is defined to:

Look at environment variable GOOGLE_APPLICATION_CREDENTIALS
Look for gcloud auth application-default login
Get a service account from the GCP Metadata server (GAE & GCP are a bit different, but it's the same idea)

We should see if there is a way to safely and securely inject a secret like a service account JSON into a container running on another cloud platform. (item 3 above).
@jonparrott said: environment variables should not be used to hold secrets.
This is a very important item for us. We are trying get to the point where docker containers can run anywhere, on GCP, on a competing cloud provider, in their own datacenter, or locally in their laptop with just a change in the configuration.

I'm not sure what more is available for us here.  @theacodes what other types of auth does python provide?

Environment variables pointing to a file that holds the secret (instead of putting the secret in the environment variable) is the best practice. Let's close this.


Errant GPL-3 licensed files in tests directory

Hi, the files tests/__init__.py and tests/nrlmsise00/__init__.py are GPL-3 licensed which I figure is a mistake with the rest of the library being MIT licensed.

Hi Kurt,
Sorry about that! It definitely was not intentional. I've corrected the files, and tagged and pushed version 0.1.72 to pypi with the correction.
Let me know if you find any other issues!
Yeah, I hadn't seen your comment until you submitted the issue - Gitter says I'm signed up for email notifications, but if I get them it's always a long time later :(


experimental: Add Maglev support

This adds support for the Maglev load-balancing algorithm to the experimental StateDB load balancing.
Update: In addition to the original goal of the PR, which is contained in the first commit, @joamaki added additional commit(s) improving the existing Maglev implementation (support for concurrency, migration to hive cells, and better data structures to avoid rewriting data from one to another).
Please ensure your pull request adheres to the following guidelines:

[x] For first time contributors, read Submitting a pull request
[ ] All code is covered by unit and/or runtime tests where feasible.
[x] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[x] If your commit description contains a Fixes: <commit-id> tag, then
please add the commit author[s] as reviewer[s] to this issue.
[x] All commits are signed off. See the section Developer’s Certificate of Origin
[ ] Provide a title or release-note blurb suitable for the release notes.
[x] Are you a user of Cilium? Please add yourself to the Users doc
[ ] Thanks for contributing!

@joamaki Please have a look.
@robscott The PR is not ready for review yet, but just FYI.

/test

/test

/test

/test

/test

/test

/test

One more thing regarding commit messages: are comparable benchmarks available somewhere for the previous implementation?

One more thing regarding commit messages: are comparable benchmarks available somewhere for the previous implementation?

Yep, in the commit messages.

Yep, in the commit messages.

Oh, got it now, it's in two subsequent commits for the old and new version respectively!

As already mentioned in https://github.com/cilium/cilium/pull/35430#discussion_r1834481481 and https://github.com/cilium/cilium/pull/35430#discussion_r1834525502, Jussi isolated his changes to a separate PR https://github.com/cilium/cilium/pull/35885, so I still need to remove them from here before merging.

Ok, I removed commits that have been copied to https://github.com/cilium/cilium/pull/35885, addressed minor comments from derailed@ and myself, and rebased onto main.

/test

/test

/test

/test

Similar as in https://github.com/cilium/cilium/pull/35810#issuecomment-2493344324, the latest review requests to cilium/envoy and cilium/sig-servicemesh are for this change in pkg/ciliumenvoyconfig/exp_test.go, which is a subset of the changes in pkg/loadbalancer/experimental/script_test.go.

/test

The last push is just squashing all commits into one. All tests were already green.

/test

/ci-clustermesh
/ci-e2e-upgrade
/ci-ginkgo
/ci-integration

/ci-clustermesh

/ci-e2e-upgrade

/ci-integration


Is there a way to pre-select a series of dates in inline multi selection mode ?

I would like to persist the user's selection and then re-apply it later. Does not have to be "pre-selected" -- just need a programmatic way to select dates.

      <ion-calendar [(ngModel)]="dateMulti"
                    [options]="optionsMulti"
                    (onChange)="onChange($event)"
                    [format]="YYYY-MM-DD">
      </ion-calendar>

...

dateMulti: string[] = [];
optionsMulti: CalendarComponentOptions = {
    pickMode: 'multi',
  };

constructor() {
  const localDate = localStorage.getItem('my_date');
    if (localDate) {
      this.dateMulti = JSON.parse(localDate);
    }
}

onChange($event) {
    localStorage.setItem('my_date', JSON.stringify($event));
  }

...

smack forehead, OMG, you're right, that was so obvious! Thanks!


Some fixes for lint

PR Checklist

Please check all that apply to this PR using "x":

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

PR Type
What kind of change does this PR introduce?

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

Fixes
Issue Number: partially #1064

Thanks for this. Will review today.

There were conflicts on fragment_file_view.xml after latest commits. I fixed them on latest forced push to my branch.


Cargo computer can be overflowed, which also overflows the cargo point values

Originally found by (Eris#Zero521)
How to reproduce:

Go to the cargo computer. Order an internals crate, and in the quantity section, type a very big number (ex: 99999999)
Click order.
The cargo console will order 20 of whatever you picked, and set the cargo point count at a very big number.
After a delay you now have infinite cargo points
Order all the guns and turn SS14 into a battle royale with shotguns

Correction: This only seems to work with internals crates. I've tested with all other crates, including other crates that cost <1000 (AME fuel) and only the internals crate can overflow.

fixed in #4057 presumably


Configuration Watcher and Service Names

For the Configuration Watcher to be able to make a refresh call to the application actuator, it looks like the Kubernetes service name has to match the spring boot app name (and config map name) or it won't be able to locate it to make the call. Currently, our services have a naming convention that is [app-name]-svc which is causing the Configuration Watcher to skip the update because it can't find a service with the correct name. Is there another way to tell the Configuration Watcher which service to look for without having to make the names match? We are just wondering if we are going to have to change the names of all our existing services to remove the -svc suffix.

No they must match at the moment but this is a reasonable enhancement

ok, thanks for the verification Ryan!

Hey @ryanjbaxter, the newer Spring boot version over 2.3.12 does not provide /actuator/refresh endpoint. Are you aware of it or do I miss some point?

That endpoint has existed and continues to exist for a long while now, you might need to enable it and expose it

I did as follow but still getting 404. Is it wrong? @ryanjbaxter
management:
    endpoints:
        web:
            exposure:
                include: '*'
    endpoint:
      restart:
        enabled: true
      refresh:
        enabled: true

When I go to http://localhost:8080/actuator I see the refresh endpoint using your example
{
"_links": {
"self": {
"href": "http://localhost:8080/actuator",
"templated": false
},
"refresh": {
"href": "http://localhost:8080/actuator/refresh",
"templated": false
}
}
}

@ryanjbaxter The reason why they are now available is after following deps this and this . AFAIU without any cloud dependency, it is not possible to get /refresh entpoint. With following [pom](https://github.com/mbaykara/spring-cloud-kubernetes/blob/01ffc1c9d5b1f52b913747dafbc33ba23084b108/pom.xml).xml you cannot set the /refresh ep.

That's why we suggest you use the Spring Cloud Bom and let it manage the dependencies for you.  The refresh endpoint is part of Spring Cloud Commons not Sprint Cloud Kubernetes https://docs.spring.io/spring-cloud-commons/docs/current/reference/html/#endpoints

Not sure.  Any help would be appreciated though, the team is very busy.

@sdaskam1 @ryanjbaxter but we already have such support via : https://docs.spring.io/spring-cloud-kubernetes/docs/current/reference/html/#monitoring-configmaps-and-secrets, specifically:
spring.cloud.kubernetes.configmap.apps

shouldn't this issue be closed?

But I am not sure the config map watcher will take that into account.  I think it was added after I implemented the config map watcher

I think it was added after I implemented the config map watcher

That is correct.

But I am not sure the config map watcher will take that into account

We have an integration test that enforces this actually: https://github.com/spring-cloud/spring-cloud-kubernetes/tree/main/spring-cloud-kubernetes-integration-tests/spring-cloud-kubernetes-client-configmap-event-reload-multiple-apps

Ah OK, then yes I think we can close this.


Add pre-commit hooks for linting and beautifying

Add pre-commit hooks (e.g. using husky) to lint and beautify code.
ESLint with Prettier plugin is a good option, but I'm open to any suggestions.
This will ensure that code is consistent across all devs and prevent unnecessary whitespace.

We should add this to both server and website (if it's not already in the website)


patent_id is missing from field list tables

I'm able to retrieve patent_id from many of the API's endpoints, but this field isn't listed in any of the field list tables (e.g., https://www.patentsview.org/api/location.html). Wasn't sure if that omission was on purpose or not.

Hello,
The patent_id and patent_number are synonymous. patent_id was originally intended to be an internal field to be used for joining different entities and patent_number to be used as the API field for the patent number. This is the reason for the lack of documentation on the field. While it is possible to query this field and while we don't have any plans for standardizing and de-duplicating API fields right now, I would wager that for any of the future standardization/cleanups, removing patent_id from the list of API fields would be a likely candidate.


Support for TouchID

https://github.com/99designs/aws-vault/pull/131 indicates the changes in that branch need to be upstreamed to this repo in order to support TouchID. Opening an issue here.
Would it be as simple as moving the files from that branch into a PR for this repo?

Hi there,
what about this?


Update scalactic to 3.2.11

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

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

labels: test-library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #121.


CF-CLI broken in brew...

It seems like the command to install the cf cli is broken. It seems like its un-related to the script, as a manual brew install cf-cli does not work either...  thoughts?
echo
echo "Installing Cloud Foundry Command-line Interface"
brew tap cloudfoundry/tap
brew install cf-cli
Does not work...
Installing Cloud Foundry Command-line Interface
==> Installing cf-cli from cloudfoundry/tap
==> Downloading https://cli.run.pivotal.io/stable?release=macosx64-binary&version=6.17.0&source=homebrew
curl: (6) Could not resolve host: cli.run.pivotal.io
Error: Failed to download resource "cf-cli"
Download failed: https://cli.run.pivotal.io/stable?release=macosx64-binary&version=6.17.0&source=homebrew

resolved... it turns out it was a DNS issue specific to the pivotal.io domain.


huge energy impact

App consumes an enormous amount of energy on OS X. On average 600 times more than the second most energy consumer app, I think this is an issue!

Yes, it's an issue, and I'm also waiting for this fix. : S
#162

Wow. Messenger App is off the chart, even when it is doing nothing.

We know this issue, we reduced the energy impact in the latest beta (see https://github.com/Aluxian/Facebook-Messenger-Desktop/issues/458), but we still have some problems when the window is visible (on mac it seems to be worse)...


plutus-use-cases: Add tutorial

Add a tutorial that shows how the crowdfunding contract works

@michaelpj I'm about to add the wallet actions for contributor and campaign owner, but I just realised that in order to collect the funds, the owner's wallet must have been watching the campaign address from the start. So we either need to use blockchain triggers after all, or we need to extend WalletAPI with the ability to add an address to the list of addresses that it is watching. What do you think?
I'm leaning towards the latter because it would be less complex.

I think letting you manually adjust the set of watched addresses isn't too bad.

@michaelpj I addressed all the issues (except the && one, see comments), it should be ready now

Hm, oh dear I think I see a problem. So the CI job currently pulls in all the split tests for things that have them, but it won't do that for the ones that don't so it won't build or test plutus-use-cases any more (since nothing depends on it).
Maybe just check that you can build it locally with nix and I'll do a PR to also build all the projects, which I think is the right solution.

Okay, it does indeed pass, so I'll merge this and the CI is also fixed.

Just saw your comments -

Maybe just check that you can build it locally with nix

I did that and it builds

Okay, it does indeed pass, so I'll merge this and the CI is also fixed.

👍 thanks!

One thought that occurred to me: possibly this tutorial belongs in wallet-api. At the moment I think the only thing it uses from plutus-use-cases is Language.PlutusTx.Validation, which arguably should move to live with Wallet.Validation in wallet-api.

possibly this tutorial belongs in wallet-api

Makes sense, I'm working on it