setCurrencyCode is not a function

We are getting this error 'setCurrencyCode is not a function' from time to time and have tracked it down to this code. It seems that when store is not initialized, self.setCurrencyCode is called in the ready callback. The problem is that self is mParticleInstance and does not have a function called setCurrencyCode, because setCurrencyCode is a method of mParticle.eCommerce.
https://github.com/mParticle/mparticle-web-sdk/blob/c61398e92e76c3057bb23a10813155591dd5bd1b/src/mp-instance.js#L619-L633

@bryoncdiscovery thanks for bring this to our attention! We will take a look ASAP.

@bryoncdiscovery Sorry for the delay in responding.
It looks like you're a few iterations behind our current SDK. Can you let me know if you've tried updating to our latest SDK and if this is still an issue?
Would you also be willing to share some usage examples of how you're calling setCurrencyCode? I'd like to set up something similar to your usage to test for this so I can make sure I've properly resolved the issue.
Thanks!

@bryoncdiscovery Sorry for the delay in responding.
It looks like you're a few iterations behind our current SDK. Can you let me know if you've tried updating to our latest SDK and if this is still an issue?
Would you also be willing to share some usage examples of how you're calling setCurrencyCode? I'd like to set up something similar to your usage to test for this so I can make sure I've properly resolved the issue.
Thanks!

@bryoncdiscovery this is fixed as of v2.12.5 which we just released. I am closing this for now, but please reopen if there are any additional issues.  Thanks for the report!


Version 2.0.7

Landed in https://github.com/metarhia/impress/commit/d16c183e257bf08b301b7a5bb5b680a1c29e1652


No example for Connect-AzureAD using Service Principal with a Secret Key

The only example for using Connect-AzureAD using a Service Principal is with a certificate. Sometimes this isn't practical, I could provide some code for an example using a Client Secret as I've had to work that out, which I did with some assistance from users on the PowerShell channel on Slack.com

Hello @psignoret, could you help us with adding the requested example.

Hi @msewaweru
Sorry, ADHD, I'm always really busy and so I often want to help out or respond etc. I basically ended up creating a function I'm using do do this, but it's a bit complicated, in that some cmdlets need 'Scope', my function has a switch parameter called -UseScope, so if that's set, then the URL used for getting an access token is "https://graph.microsoft.com/.default" otherwise it's "https://graph.microsoft.com".
I'll try to share shortly, just checking if my code is suitable, generic enough. Thanks for chasing though.

I've extracted the code from my function, as I wasn't too sure about it. It feels a bit long winded for an example in the documentation and could also do with a list of commands that require the access token using the resource and those that require a scope access token.
For example, my full script used resource access token for Get-AzureADMSTrustFrameworkPolicy, Set-AzureADMSTrustFrameworkPolicy and New-AzureADMSTrustFrameworkPolicy and used the scope access token for Get-AzureADApplication and Get-AzureADServicePrincipal
$TenantId = # tenant id
$ClientId = # client id
$ClientKey = # client key or password
$UseScope = # true or false - switch in function

$B2CUri = "https://login.microsoftonline.com/$TenantId/oauth2/token"
$xwwwform = "application/x-www-form-urlencoded"
$granttype = "client_credentials"
$GraphResource = "https://graph.microsoft.com"
$GraphScope = "https://graph.microsoft.com/.default"

# Setting Request Body with scope for Graph
$scopeRequestBody = @{
    grant_type    = $granttype
    client_id     = $ClientId
    client_secret = $ClientKey
    scope         = $GraphScope
}

# Setting Request Body with resource for Graph
$requestBody = @{
    grant_type    = $granttype
    client_id     = $ClientId
    client_secret = $ClientKey
    resource      = $GraphResource
}

# Making the Web Request with Scope and Resource for Graph API and setting the response variables
$ScopeResponse = Invoke-WebRequest -Uri $B2CUri -Method Post -ContentType $xwwwform -Body $scopeRequestBody -UseBasicParsing
$ResourceResponse = Invoke-WebRequest -Uri $B2CUri -Method Post -ContentType $xwwwform -Body $requestBody -UseBasicParsing

# Setting the Access token variables for Graph API from the responses using scope and resource requests
$ScopeAccessToken = $ScopeResponse.Content | ConvertFrom-Json | Select-Object -ExpandProperty access_token
$ResourceAccessToken = $ResourceResponse.Content | ConvertFrom-Json | Select-Object -ExpandProperty access_token

# Making an initial connection to Azure AD B2C using the Graph API Scope access token to enable the script
# to query Azure AD B2C for the Azure AD Domain and the in use credential
$ctx = Connect-AzureAD -MsAccessToken $ScopeAccessToken -TenantId $TenantId -AccountId $ClientId -AadAccessToken $ScopeAccessToken

$CtxTenant = Get-AzureADDomain | Select-Object -ExpandProperty Name
Write-Verbose "Tenant domain for Azure AD B2C is $CtxTenant"

$CtxTenantId = $ctx.TenantId
Write-Verbose "Tenant ID for Azure AD B2C is $CtxTenantId"

$Ctxuser = Get-AzureADServicePrincipal -Filter "appid eq '$ClientId'" | Select-Object -ExpandProperty DisplayName
Write-Verbose "User for Azure AD B2C is $Ctxuser"

# This is just for information
$CtxTenantShort = $CtxTenant.Replace(".onmicrosoft.com", "")
Write-Verbose "Tenant short name for Azure AD B2C is $CtxTenantShort"

# If we haven't called the function with the UseScope switch, then we need to make another connection using the Graph API
# resource access token to replace the connection already make with the scope access token
if (!($true -eq $UseScope)) {
    $ctx = Connect-AzureAD -MsAccessToken $ResourceAccessToken -TenantId $TenantId -AccountId $ClientId -AadAccessToken $ResourceAccessToken
}

return $ctx

@msewaweru I suggest asking the owners of the PowerShell module whether they want to include an example using client secret. (In the Microsoft Graph PowerShell examples, for instance, this was deliberately omitted: https://docs.microsoft.com/graph/powershell/app-only)
@WillParry The main issue with the example script as you've shown it that it suggests the poor practice of including highly privileged credentials in a script. While I have no doubt at line 3 you mean "retrieve the client secret from a protected location", it's unfortunately very common for less experienced users to copy-paste the script, paste in their (highly privileged) client secret, and then share the script or store it in an insecure location.

@WillParry The main issue with the example script as you've shown it that it suggests the poor practice of including highly privileged credentials in a script. While I have no doubt at line 3 you mean "retrieve the client secret from a protected location", it's unfortunately very common for less experienced users to copy-paste the script, paste in their (highly privileged) client secret, and then share the script or store it in an insecure location.

A good point, perhaps I need to update my snippet and or function to make it more secure, I'm not storing any credentials in it of course, these come from the automation that I intended to run my script / function, but I see your point any it would be better if an example were to be provided that it show a very secure way of doing this. Thanks for the feedback.

@DerrickJKimani, please direct me to the owner of this cmdlet.


Whitespace goes green when paging up or down

Steps to reproduce

Open a longer file in vim (files less than a page don't seem to show it.)
<Ctrl+D> to go down.
Should notice all the white space go bright green.

System
Windows 10 Home Edition on Surface Pro
Ubuntu App (Windows Subsystem for Linux)

Had a passing thought to check if this happened with other color schemes and it does. So not an issue with molokai, sorry! Now to fix it for my environment...


Addressing comments raised in #3

This is a WIP pull request, intending to address comment raised in #3 that I missed.

Original comment:

DISTANCE is not implemented. I think that it shouldn't be an enum member.

One item that I don't know how to address is this comment:

You can write a custom an argparse action to perform cross-argument validation. It is a preferred way because it uses a argparse-builtin error handling.

I've posted my response there.


Object allocation is slower since 4.0.beta9

Godot version
v4.0 beta9
System information
Windows 11, 64bit
Issue description
Since migrating from Beta 8 to Beta 10, I've noticed a serious performance hit. I've tracked the problem down to a place where I'm creating a couple thousand objects. The object allocation used to take a couple microseconds, but ever since Beta 9 it takes hundreds of microseconds to construct a new object.
Steps to reproduce
Create a plain Godot class that extends Object. Create a few thousand of them. It takes a while.
Minimal reproduction project
AllocationExample.zip

If you can compile the engine from source, you could look into bisecting the regression to greatly speed up troubleshooting.

OK, I'll see what I can find.

I didn't test much, but I remember having this problem and that it might be fixed on newest master.

That would be awesome if so. I haven't been able to build from source yet, so I've yet to test it on master.

Beta 11 might be released soon, so you will be able to test easily.

Can you still reproduce this issue using Beta 14?


Fix service version in Swagger interface

The Swagger interface was using process.env.npm_package_version for the service version. The service no longer sets its version via package.json therefore npm_package_version is empty/undefined.
It now reads the version from the configuration service (from the about.version).

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.01%) to 91.192%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
4631

Relevant Lines:
4954

💛  - Coveralls


Update 150-db-push.mdx

Added a Note to tell users of PlanetScale that we recommend db push instead of migrate. Added a link to the Getting Started docs... but after the PlanetScale-specific docs are published, we can change that link to be more specific. And we can add more details to the upcoming Guide as well, about WHY we recommend db push. See Issue #7282.

@janpio agreed about the note might be more useful over on pages that talk about using prisma migrate... I'll go ahead and merge this PR, and then look for an eloquent way to do that, because there are a lot of docs about using migrate. :-)


Integrate with Turn.io

Allow clients that have Whatsapp numbers registered with Turn.io to use them with OCS.
https://whatsapp.turn.io/docs/category/turn-api

Might be helpful, I wrote this a few years ago: https://github.com/dimagi/turn-python. Not sure if they have made changes to their API, or have released better tools in the meantime.


[5.5][Parse] Disallow use of actor as a declaration modifier

Cherry-pick of https://github.com/apple/swift/pull/37516

Explanation:

Promote a warning into an error for declarations that use actor class syntax,
since actor is now a standalone contextual keyword and cannot be used as
a class declaration modifier.

Scope: Declarations that still use actor class syntax regardless of warnings that it has been deprecated.

Main Branch PR: https://github.com/apple/swift/pull/37516

Resolves: rdar://75753598

Risk: Low

Reviewed By: @etcwilde

Testing: Regression tests added to the suite

Resolves: rdar://75753598
(cherry picked from commit aeddab4dd0fcefae1472f2d90febb4e976901e80)

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - d1e48f0d534a14b4a65ba033ffdd885f3e691ba6

Build failed
Swift Test OS X Platform
Git Sha - d1e48f0d534a14b4a65ba033ffdd885f3e691ba6

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - 29d86a6443872393dad6c4df6b22ee260e98c741

@swift-ci please test Linux platform


Merge pull request #160 from openidl-org/develop

Merge develop into main for release tagging

Wrong base branch


Calling script results in "undefined method" error

I called it with these parameters:
transcode-video --main-audio 1=English --add-audio language=deu --audio-width all=double --no-auto-burn /title01.mkv

and the output was:
/usr/local/bin/transcode-video: undefined method `[]' for nil:NilClass

seems to be the "--main-audio" call in this case. Removing that results in no error aside fromm #8

@daDnaK Nice catch! Thanks for finding this. I can reproduce it, too. Sorry about that.
I'll start working on a fix after I get back from Mother's Day festivities today.

@daDnaK Found the bug.  This is what I get for trying to be clever with Ruby. :)

now you can try what happens if you omit the "-m" option when tagging ;-)

@daDnaK Nope. :) Seriously, I have a plan for release notes. I just need a little more time.

ah ok, take your time ;-)

Am 10.05.2015 um 18:53 schrieb Don Melton<EMAIL_ADDRESS>@daDnaK https://github.com/daDnaK Nope. :) Seriously, I have a plan for release notes. I just need a little more time.
—
Reply to this email directly or view it on GitHub https://github.com/donmelton/video_transcoding/issues/9#issuecomment-100668229.

@daDnaK OK, the fix is in just gem update and it should work now. Thanks again!

@daDnaK is now @JMoVS, as by your request (came around to do that)

@JMoVS Wow! Is that a new account? Or can your rename them?

you can rename them, it just won't relink previous mentions and won't redirect the profile page but will redirect repository addresses

I am getting this same issue with the most up-to-date version of the scripts.
transcode-video --big --quick --mp4 --main-audio 2 --crop 140:140:0:0 Jupiter\ Ascending\ \(2015\).mkv -o ../movie/Jupiter\ Ascending\ \(2015\)

returns
/Users/jason/.rvm/gems/ruby-2.2.1/bin/transcode-video: undefined method `[]' for nil:NilClass

@jsonbecker Then it's probably a different bug. I'll take a look when I'm back home tomorrow.
Question: does the .."/movie/Jupiter Ascending (2015)" already exist?

It does. Thanks for the prompt response.

@jsonbecker Sorry this took so long. And also sorry I didn't see the problem right from the beginning.
This is not a bug so much as a side effect of the way Ruby processes command line options.
Your command line will never work as is because you're passing an option after the input argument, i.e. you're doing something like this:
transcode-video --mp4 orginal.mkv --output directory/

See the problem? The options need to come before the input arguments. Just re-arrange your command like this and it should work fine:
transcode-video --big --quick --mp4 --main-audio 2 --crop 140:140:0:0 -o ../movie/Jupiter\ Ascending\ \(2015\) Jupiter\ Ascending\ \(2015\).mkv

That doesn’t seem to do it. If I remove —main-audio 2 it works fine (ish, I get this error [truehd @ 0x103801000] major sync info header checksum error because it can’t decode the first audio track right).

On Jul 7, 2015, at 5:37 PM, Don Melton<EMAIL_ADDRESS>wrote:
@jsonbecker https://github.com/jsonbecker Sorry this took so long. And also sorry I didn't see the problem right from the beginning.
This is not a bug so much as a side effect of the way Ruby processes command line options.
Your command line will never work as is because you're passing an option after the input argument, i.e. you're doing something like this:
transcode-video --mp4 orginal.mkv --output directory/
See the problem? The options need to come before the input arguments. Just re-arrange your command like this and it should work fine:
transcode-video --big --quick --mp4 --main-audio 2 --crop 140:140:0:0 -o ../movie/Jupiter\ Ascending\ (2015) Jupiter\ Ascending\ (2015).mkv
—
Reply to this email directly or view it on GitHub https://github.com/donmelton/video_transcoding/issues/9#issuecomment-119349853.

@jsonbecker OK, that... doesn't make any sense. Can I see the output of this command?
transcode-video --scan Jupiter\ Ascending\ \(2015\).mkv

So this is strange. VLC shows two audio tracks, the first one skips and the second one plays fine. But transcode-video --scan clearly shows only one track:
+ title 1:
  + stream: Jupiter Ascending (2015).mkv
  + duration: 02:07:34
  + size: 1920x1080, pixel aspect: 1/1, display aspect: 1.78, 23.976 fps
  + chapters:
    + 1: cells 0->0, 0 blocks, duration 00:08:45
    + 2: cells 0->0, 0 blocks, duration 00:10:46
    + 3: cells 0->0, 0 blocks, duration 00:12:02
    + 4: cells 0->0, 0 blocks, duration 00:10:09
    + 5: cells 0->0, 0 blocks, duration 00:07:53
    + 6: cells 0->0, 0 blocks, duration 00:08:54
    + 7: cells 0->0, 0 blocks, duration 00:10:50
    + 8: cells 0->0, 0 blocks, duration 00:09:47
    + 9: cells 0->0, 0 blocks, duration 00:10:02
    + 10: cells 0->0, 0 blocks, duration 00:06:59
    + 11: cells 0->0, 0 blocks, duration 00:07:18
    + 12: cells 0->0, 0 blocks, duration 00:09:32
    + 13: cells 0->0, 0 blocks, duration 00:05:01
    + 14: cells 0->0, 0 blocks, duration 00:09:35
  + audio tracks:
    + 1, English (AC3) (5.1 ch) (Dolby Digital EX) (iso639-2: eng), 48000Hz, 384000bps
  + subtitle tracks:
    + 1, English (iso639-2: eng) (Bitmap)(PGS)

@jsonbecker I would trust what HandBrakeCLI thinks that file contains then. And that explains the other problem you were having.
But it's entirely possible the file is corrupt somehow.


flycast: Add version 1.1

Here's a manifest for the great emulator that's mainly for Sega Dreamcast, but also other platforms like Naomi and Atomiswave (also based on Sega's hardware).
What's missing is a link to its cfg file since this is created on first run - by the way, should files that are linked use a symbolic link or a hard link, and what's the reasoning behind that?

by the way, should files that are linked use a symbolic link or a hard link, and what's the reasoning behind that?

I don't know what kind of link should be used. I guess the most reliable choice should be picked. Note that symbolic links can only be created by non-administrator users on Windows 10; I'm not sure if this limitation applies to hard links as well.

Yeah that's fine, as it's now it's also a non-issue since we don't have to make that choice; the manifest can't persist the emu.cfg as it doesn't exist from the get go anyway - and maybe the persist function even makes that choice for us.


Update JSDoc

Issue summary
Update JSDoc to follow the standards outlined in the Google JS StyleGuide and the JSDoc documentation (cheat-sheet too).
Expected behavior
Good looking JSDoc when generated with jsdoc path/to/file.js. Also add a script in package.json (such as the one here) as well as a conf.json file like the one here.
Actual behavior
Current correct JSDoc usage is flaky.

@ananyeet 's initial comment in chat for reference:
if you're making a type (i.e. trip, activity), make a typedef:
/**
 * An activity object.
 * @typedef {Object} ActivityInfo
 * @property {string} id The activity's ID in the database.
 * @property {string} tripId The activity's tripId in the database.
 * @property {string} title The activity's title.
 * @property {long} start_time Number of seconds since epoch of activity's start time.
 * @property {long} end_time Number of seconds since epoch of activity's end time.
 * @property {string} [description] The activity's description.
 */

1D arrays marked as string[] and 2D marked as Array.<string[]>  but this could be optional, just nice to read. Like I have an array that's marked as {Array.<string, ActivityInfo[]>}
btw code with the single ticks ` doesn't work, use  and
code on a new line uses  and
props look really nice like this:

/**
 * React component for a single day of activities.
 *
 * @property {Object} props ReactJS props.
 * @property {ActivityInfo[]} props.activities The list of activities for 'today'.
 * @property {string} props.date The date, formatted as 'MM/DD/YYYY'.
 */

and the real tag is @return not @returns
oh yeah and https://devhints.io/jsdoc this site is nice

Found another thing here, the @override annotation is preferred over @inheritdoc. Especially so when you are including other documentation in the same JSDoc comment like here:


Switch from oraclejdk7 to oraclejdk8

Applying this PR will:

Fix the Travis build, oraclejdk7 is no longer available in trusty
Update scala and sbt versions

Thanks for this. Happy to see the back side of Oracle :)


CIF-1426 - The URLProviderConfiguration service is misconfigured

Description
Update the default configuration of the UrlProvider service to use the new syntax.
Update to core-cif-components version 1.1.1.
Related Issue

Motivation and Context
The configuration syntax of the UrlProvider service has been updated in the latest version so the demo project has to be updated as well.
How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

@mhaack I think we have to check to check how and if this applies also for this archetype, the publish config was defined to work with the corresponding Sling mappings defined at https://github.com/adobe/aem-cif-project-archetype/tree/master/src/main/archetype/samplecontent/src/main/content/jcr_root/etc/map.publish


Duplicated Identifier "NexusPrisma"

Description
If you keep the default path for typegen, and then customize it, it will result in two files generated in different locations.
Because we're using global types, TS will complain because the global type alias NexusPrisma exists twice. Using an interface would remove the error (because global interfaces with same identifier are merged) but it would result in a silent issue because we don't want the interfaces to be merged anyway.
The best we can probably do is to document this as a known issue

Nice catch. Agree docs is the solution here, at library level. We should keep this issue in mind if we ever change the defaults. We could for example have the new version check for the old path. Etc.
Sent with GitHawk


Polymer linter support for double square brackets

What steps will reproduce the problem?
I'm using the [[ ]] syntax instead of {{ }} and my code works fine but the Polymer linter complains about this (I get warnings in Dart Editor).
Stuff like this:
[Warning from polymer (Linter) on mobile|lib/login.html]:
line 16, column 28 of lib/login.html: Invalid event handler body "[[login]]". Declare a method in your custom element "void handlerName(event, detail, target)" and use the form on-click="{{handlerName}}". See http://goo.gl/5HPeuP#polymer_21 for details.
on-click="[[login]]"
^^^^^^^^^^^^^^^^^^^^
Despite the warning the app works just fine because [[login]] is a valid binding expression.
The reason I'm using [[ ]] syntax instead of {{ }} is because the HTML template is run through Django template processor first.
Polymer linter should support [[ ]] syntax and not show warnings.

This issue has been moved to dart-lang/polymer-dart#347.


Fix output error occurring due to  check if it is a dict or not

The Transformers4Rrec end-to-end example gives an error Failed to process the request(s) for model 'mymodel_pt', message: ValueError: output of the forward function should be a dict when we send request to TIS. the reason for that is due to recent changes in the prediction tasks' outputs in TF4Rec repo (see https://github.com/NVIDIA-Merlin/Transformers4Rec/pull/546). This PR provides a quick fix to avoid such error. However, for long term solution Oliver's draft PR (see https://github.com/NVIDIA-Merlin/systems/pull/252) might be preferable.

The code modified by https://github.com/NVIDIA-Merlin/systems/pull/252 was never intended to be used directly—it was added to Systems as a starting point for refactoring toward the operator API, and some parts of the old code still exist there. If something is using the legacy serving code (e.g. export_pytorch_ensemble) from Systems instead of from NVTabular, it should be reverted to using the NVT version which is mostly frozen in order avoid regressions where we have insufficient test coverage.
Making minor fixes to the NVT code in order to keep it working with the rest of the Merlin ecosystem is fine if needed, but we've been avoiding making any major changes there. The corresponding code in Systems is open to major changes and therefore unstable; it's also highly likely to be removed in a future Systems version.

See https://github.com/NVIDIA-Merlin/systems/pull/265 for links to the open PRs related to legacy (non-Torchscript Python) serving for PyTorch. Possible that the fixes in https://github.com/NVIDIA-Merlin/systems/pull/252 should be applied to NVTabular instead?

See NVIDIA-Merlin/systems#265 for links to the open PRs related to legacy (non-Torchscript Python) serving for PyTorch. Possible that the fixes in NVIDIA-Merlin/systems#252 should be applied to NVTabular instead?

Thank you for the context @karlhigley, with Ronay we weren't sure if the legacy function will be included in systems or not.   So It makes sense to me to include the changes proposed by Oliver in https://github.com/NVIDIA-Merlin/systems/pull/265 in NVTabular instead (as a separate PR from this quick fix).


🔬 Support for calibration routines

The v0.1 version of QDMI had dedicated routines for calibration
int QDMI_device_quality_check(QDMI_Device dev, double *result);
int QDMI_device_quality_limit(QDMI_Device dev, double *result);
int QDMI_device_quality_calibrate(QDMI_Device dev);

These have been purposely removed as part of the transition towards v1 due to their limited usability.
However, calibration is an important aspect of integrating QC systems into HPC environments.
Hence, QDMI should allow to trigger calibration for devices.
@martin-knudsen: Just putting in motion the (re-)integration of this feature as part of the delayed deliverable on "continuous re-calibration" for MUNIQC-SC.

Thanks for creating this issue. I updated the title as well as the description of the issue to provide a little more information.
Adding calibration functionality to QDMI definitely makes sense.
It is just a matter of finding the right way to incorporate the required workflows.
Let's use this issue for brainstorming possible solutions.
@ystade and I briefly discussed this some time ago and we came to the conclusion that the current structure of QDMI would already allow one to incorporate calibration. The proposal on the table would be to

Define a new QDMI_PROGRAM_FORMAT_T for triggering calibration, e.g., QDMI_PROGRAM_FORMAT_CALIBRATION, and use the job submission system to trigger a "calibration job". In the end, it is just another job to be run on the system.

I'd assume there needs to be some kind of way to ask the device whether it needs to be calibrated, right? For that, the proposal would be to define a new QDMI_DEVICE_PROPERTY_T that allows to check exactly that, e.g., `QDMI_DEVICE_PROPERTY_NEEDSCALIBRATION"

Two questions that are less clear to me:

Is there any need for further properties/queries/results?
Is this design, where the client calls the respective functions, fine or is this something that we want the driver to manage? In the latter case, we would probably need to define new functions as part of the driver and the device interface, which are not exposed to the client.

Sounds cool. We should keep in mind that quantum devices tend to have multiple calibration routines though.

How would you suggest to trigger a specific calibration?
I guess we would also need to be able to query how many different types of calibration can be triggered.
Providers should be allowed to decide if calibration can be triggered from the QDMI client side. I guess we need something like QDMI_DEVICE_PROPERTY_CALIBRATIONSUPPORT.

Sounds cool. We should keep in mind that quantum devices tend to have multiple calibration routines though.

How would you suggest to trigger a specific calibration?
I guess we would also need to be able to query how many different types of calibration can be triggered.
Providers should be allowed to decide if calibration can be triggered from the QDMI client side. I guess we need something like QDMI_DEVICE_PROPERTY_CALIBRATIONSUPPORT.

Thanks for the comments!
I think the third part is the easiest to address, as it is currently supported. The job creation routine will report a "not supported" error, when a particular program format is not supported by the device.
The other two are a bit harder to address. At first, it seems hard to me to unify the different kinds of calibration across vendors and/or technologies. Standardization is probably hard here.
Given how we already have the capability to set parameters on a job, we could probably solve the problem of configuring the calibration that way, i.e., one would first create a job with a "calibration" program format and subsequently set some respective parameters on that job, before eventually submitting the job.
At first, while the precise parameters might be uncertain, one could make use of the custom parameters.
Does that make sense?
@ystade a sanity check from your side would probably also help here.

@ystade a sanity check from your side would probably also help here.

@burgholzer I actually wanted to reply something along the same lines as you did above. Hence, I am fully in line with your view. Especially, I think as the way of calibration is not standardized at this point in time, the implementation via dedicated calibration functions is even more difficult because my feeling suggests that this requires more standardization as the aforementioned proposal.
I just want to point out one more difficulty which I even do not know whether it is relevant. I think one part of point 3. of @echavarria-lrz reply

3. Providers should be allowed to decide if calibration can be triggered from the QDMI client side. I guess we need something like QDMI_DEVICE_PROPERTY_CALIBRATIONSUPPORT.

is that the client might set a limit fidelity and whenever the device falls below this fidelity limit it automatically starts a calibration round. In this case, the client does not submit a job to the device as the calibration is started by the device itself. However, once #117 is solved, such a limit can be set via a device parameter.

I don't think this is a feature we need to prioritize at the moment (or even support at any point):

the client might set a limit fidelity

It could be effectively managed by setting a limit on the client side and triggering a calibration round using the appropriate routine.
Regarding the following comment:

the device falls below this fidelity limit it automatically starts a calibration round

I wouldn't advise automating this process with such a straightforward approach (on any side). At the LRZ, we have encountered cases where devices became unstable precisely because of this. The solution we will adopt at the LRZ, for example, is to schedule calibration on the client side (whenever supported by the backend) based on, for example, the current load, time of the day, telemetry data, and so on.

Alright. Thanks for the productive discussion here. It seems that we have reached a state of agreement when it comes to the initial support for calibration routines.
Thus, I would propose to go forward with implementing the proposal in https://github.com/Munich-Quantum-Software-Stack/QDMI/issues/108#issuecomment-2520238480.
It might be worth to use that opportunity to also add a small bit of documentation on this in addition to the usual API docs.


KeyError: 'module name can't contain "."'

Hi, I am new to person re-identification. I found this model super excellent.
But, I am facing to a problem. Since KeyError: 'module name can't contain "."' happens every single running, I removed '.' from line 73 to 78 in "C:\Users#####\anaconda3\envs\STReID\lib\site-packages\torchvision\models\densenet.py". But still, I can't train the model. How can I sort out this problem?

It may be a good idea that you modified the official pytorch code in "C:\Users#####\anaconda3\envs\STReID\lib\site-packages\torchvision\models\densenet.py"
I suggest you use pdb.set_trace() debugger to trace the error and show me which line leads to this error.

Where can I find pdb.set_trace()?


Ensure custom trees are propagated for non-specified backend

I think this is the right fix and the tests are green. Merging.


Add support for new DXGI 16bpp format

In the DirectX Agility SDK 1.610, there is a new DXGI format DXGI_FORMAT_A4B4G4R4_UNORM. This PR adds support to DirectXTex for this format.

Related PR for test suite.


set minSdk 29, change repo to mavenCentral, kotlin codestyle and JVM args

Not sure if this talkback will be used by more than just GrapheneOS. If so, could drop minSdk a little lower. I just set it to 29 like the Camera app. Android 10.
Use normal memory allocation. I set the Kotlin codestyle to official like other repos in GrapheneOS. R8 full mode should be fine.
jcenter is also a deprecated and old repo. Should be using mavenCentral for updated libs.
Seems to work just fine with typical emulated Pixel 5.
Targeting SDK 31 requires intent changes in the code I haven't bothered to work out relating to IMMUTABILITY flags. That's as far as I know.
Signed-off-by: June<EMAIL_ADDRESS>

Targeting SDK 31 requires intent changes in the code I haven't bothered to work out relating to IMMUTABILITY flags. That's as far as I know.

Already resolved in #2, the only required thing was to set android:exported for BrailleEime service. Let''s stick with minSDK 26.
Useful and important suggestion to change jcenter repo.
Btw. This fork is obviously not only for GrapheneOS.

@PatrykMis Ah thank you for clarifying. I only made a PR here as I wasn't sure about some things.

Ok. So I think you can commit (eventually cherry-pick some commits from here) and then reopen PR#1 in GrapheneOS project. These are your changes so no need to open another PR.

I don't have push access to this repo so I had to fork. Was going to make a separate branch in here.
Will do though when I get time.


Add Github Enterprise support

Allows for environmental variables to be configured to access GitHub Enterprise
export GITHUB_DOMAIN=git.{YOURDOMAIN}.com
export GITHUB_PERSONAL_ACCESS_TOKEN={a personal access token with the correct permissions to read users}
export GITHUB_COOKIE={a cookie copied from an active browser session (this should work with pretty much all SSO implementations}
npm run start

Wouldn't it be easier to implement it similarly to how the code currently reads GitHub's html? Considering that there are GitLab graphs, that is. I could also be wrong considering I'm not too familiar with GitLab myself!

There are really only two differences between GitHub and GitHub Enterprise (GHE)

The API is at a different URL.
The user is logged in, usually via some kind of federated model (SSO)

In order to not implement the login flow, two-factor, etc, the easiest solution is to just expect the user to copy an active token from a browser session.
Other than changing the URLs and setting additional headers, this solution uses the same parser solution.

Just tried to access the GitLab url with a random existing username and you were right, it did ask me to login.
Thank you for your time and effort to create this PR, I will review it closer after work! 👏


Compiledoutput.drive is not a function

I'm getting this issue when trying to apply augur to a demo-app, which is a naked expressJs app.
TypeError: compiledOutput.drive is not a function
at executeInstructionsFromFile (/home/lpi/augur/ts/dist/src/utils.js:195:20)
at exports.run (/home/lpi/augur/ts/runner/run.js:225:23)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
Tried using the default spec.json, but it doesn't seem to do anything. Does this happen when I'm using a native function or something along those lines?

That doesn't seem to be the case: and the function in analysis.output seems to be a function.

Typically, you see that error in Augur when the program you're analyzing crashed; Augur is saying it has nothing left to analyze for taint flows. Augur doesn't crash if your code is using unsupported native functions.
Could you link me the code/repo you're trying to analyze? I can take a look and see what's going on.

Sure! Here's my test app which displays this. It's a pretty naked express app.
https://github.com/lpi/simple-taint-demo
Thank you!

Some other issues: I dropped express, and made a web server manually, this fails, albiet in a different way:
`const http = require('http');
const hostname = '<IP_ADDRESS>';
const port = 3000;
const server = http.createServer((req, res) => {
res.statusCode = 200;
var x = getPassword();
log(x);
res.setHeader('Content-Type', 'text/plain');
res.end('Hello World');
});
server.listen(port, hostname, () => {
console.log(Server running at http://${hostname}:${port}/);
});
function getPassword() {
return "123456";
}
function log (message) {
console.log(message);
}`
When I run this, augur gets stuck at 'Instrumenting code
=>  /home/lpi/demos/simple-taint-demo/index.js...
(    ● )^C
'
forever

I'm running nodeprof via docker, the test cases work for me. Changing my code to to
var x = getPassword();
log(x);I'm running nodeprof via docker, the test cases work for me. Changing my code to to

var x = getPassword();
log(x);

function getPassword() {
  return "123456";
}

function log (message) {
  console.log(message);
}

function getPassword() {
  return "123456";
}

function log (message) {
  console.log(message);
}

works as expected.


Accommodation for multiple python version

I was trying to setup the monitor locally when I ran into the issue of pipenv not accepting python version greater than 3.6 .
On further research, I found a similar issue raised here. This was indeed related to the Pipfile configuration.
I would be happy to resolve it.

The easiest fix, for now, might be to run it inside a Docker container that uses the python version the Pipfile expects.
If we can improve compatibility by updating the Pipfile accordingly, feel free to do so!

closed via https://github.com/mammuth/bitcoin-arbitrage-trading-bot/pull/48
Thanks!


Only resolve relative volumes if a Compose file is specified

Relative volumes are currently based on the path of the first Compose file. If libcompose is used as a library and a Compose file isn't specified then this causes a panic.
This checks to see if a Compose file was specified, and if not then it skips attempting to resolve relative volumes.
package main

import (
	"log"

	"github.com/docker/libcompose/docker"
	"github.com/docker/libcompose/project"
)

func main() {
	service := `
service:
  image: ubuntu
  command: echo Hello world!
  volumes:
    - /etc/selinux:/etc/selinux`

	project, err := docker.NewProject(&docker.Context{
		Context: project.Context{
			ComposeBytes: [][]byte{[]byte(service)},
			ProjectName:  "yeah-compose",
		},
	})

	if err != nil {
		log.Fatal(err)
	}

	project.Up()
}

panic: runtime error: index out of range

goroutine 15 [running]:
panic(0x912960, 0xc82000a0f0)
	/usr/local/go/src/runtime/panic.go:464 +0x3e6
github.com/docker/libcompose/docker.volumes(0xc82039d180, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0xc8202ce3e0, 0x1, ...)
	/home/joshwget/go/src/github.com/docker/libcompose/docker/convert.go:61 +0x2c5
github.com/docker/libcompose/docker.Convert(0xc82039d180, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0xc8202ce3e0, 0x1, ...)
	/home/joshwget/go/src/github.com/docker/libcompose/docker/convert.go:148 +0x541
github.com/docker/libcompose/docker.ConvertToAPI(0xc820373420, 0xc8203c4f50, 0x0, 0x0)
	/home/joshwget/go/src/github.com/docker/libcompose/docker/convert.go:46 +0x63
github.com/docker/libcompose/docker.(*Container).createContainer(0xc8203f0150, 0xc820390c08, 0x6, 0xc8202f8780, 0x40, 0x0, 0x23, 0x0, 0x0)
	/home/joshwget/go/src/github.com/docker/libcompose/docker/container.go:500 +0xf9
github.com/docker/libcompose/docker.(*Container).Recreate(0xc8203f0150, 0xc820390c08, 0x6, 0x1, 0x0, 0x0)
	/home/joshwget/go/src/github.com/docker/libcompose/docker/container.go:150 +0x888
github.com/docker/libcompose/docker.(*Service).recreateIfNeeded(0xc820373420, 0xc820390c08, 0x6, 0xc8203f0150, 0x0, 0x0)
	/home/joshwget/go/src/github.com/docker/libcompose/docker/service.go:297 +0x507
github.com/docker/libcompose/docker.(*Service).up.func1(0xc8203f0150, 0x0, 0x0)
	/home/joshwget/go/src/github.com/docker/libcompose/docker/service.go:272 +0x6f
github.com/docker/libcompose/docker.(*Service).eachContainer.func1.1(0x0, 0x0)
	/home/joshwget/go/src/github.com/docker/libcompose/docker/service.go:315 +0x33
github.com/docker/libcompose/utils.(*InParallel).Add.func1(0xc8203f0180, 0xc820426040)
	/home/joshwget/go/src/github.com/docker/libcompose/utils/util.go:26 +0x54
created by github.com/docker/libcompose/utils.(*InParallel).Add
	/home/joshwget/go/src/github.com/docker/libcompose/utils/util.go:30 +0x5d

Signed-off-by: Josh Curl<EMAIL_ADDRESS>

Thanks :+1:
@joshwget We should add a regression test on that :wink: (and the more I think about the more I feel we need an API integration test way more furnished than the cli one).

@vdemeester Added a test for this

LGTM 🐵


Unblock transfer queue processor by writing tasks back with higher IDs

Currently, failing to completely process one transfer task (e.g. because matching engine is down, or visibility store write is failing) would block the progress of the transfer queue processor goroutine (and, in extreme cases, causes the entire history service node to log fatal and restart).
Instead of blocking, it is better to rewrite the task back to cassandra with a higher task ID. This allows processing other tasks until the task is picked up again, and hopefully the reason for failing to process the task in the first place would be resolved by that time.

Also consider writing to a side / dead-letter queue and alert when processing repeatedly fails so that the failures can be inspected (e.g. a bug in the code prevents task from being processed)

@wxing1292 has already implemented a DLQ as part of cross-DC.


Settings -> Default route - production

Hi,
On a production server, the Default Route in Settings has no options to choose from.

In SettingsController@index filter probably removes all routes because of condition $route->action['prefix'] === '/admin' (without '/' this function returns the right routes - tested in artisan tinker)
Thanks

Resolved in PR: #Fix/Default-Route

This was fixed in v1.1.21


Improve Tests pt. 2

Let's use this issue as an ongoing story for improving our UI Test Strategy.
Related: https://github.com/mesosphere/marathon-ui/pull/511
Integration Tests Strategy

[ ] Define Integration Testing strategy and scope (what is the desired outcome? is testing within a browser against the Marathon API enough? which tools are suitable in an OSS project?) cc @air
[ ] get a working DCOS integration test runner set up
[ ] define and write some Integration Tests

ACs

[x] Audit the existing tests, organise them by scenario (ie: BDD (with DOM), Unit Tests, Integration Tests (?)) (https://github.com/mesosphere/marathon/issues/2929)
[x] separate unit tests from behavioural tests such that unit tests are clearly associated with the files they test (https://github.com/mesosphere/marathon/issues/2928)
[x] ensure that behavioural tests run with a DOM context to avoid the shallow renderer and children[0].props.children[0].props.children[0] type chains [https://github.com/mesosphere/marathon-ui/pull/547]
[x] add coverage numbers - [https://github.com/mesosphere/marathon-ui/pull/554]
[x] write and document some example tests
[ ] create jobs for areas where tests are missing
[ ] add Sinon for spies and other benefits

Thanks to @philipnrmn for the ideas.

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-3086. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-3086. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.


Add note about funding distribution to form

What does this PR do and why?
Adds a note describing that any can distribute funds on behalf of project. Also adds a missing word.
Screenshots or screen recordings

before
after

Acceptance checklist

[x] I have evaluted the Approval Guidelines for this PR.

cc @mejango @peripheralist
I'm not stoked with how many words are on this screen, but understand that there's been confusion about the distribution button that needed addressing. We can probably expand on this in the docs somewhere to reduce word count.
All good for now though 🚀

great 👏
agreed on the wordiness, but i think it's generally the right tactic to prioritize clarity over brevity


fix concatenation of anything_else

When concatenating fsms with differing alphabets, symbols from the others' alphabets do not map as if they were anything_else.
For example:
# [^a]
fsm1=fsm(
  alphabet={"a",anything_else},
  states={0,1},
  initial=0,
  finals={1},
  map={0:{anything_else:1}}
# .
fsm2=fsm(
  alphabet={anything_else},
  states={0,1},
  initial=0,
  finals={1},
  map={0:{anything_else:1}}
  )
# [^a].
assert (fsm1+fsm2).accepts("ba") # Fails

Looking at the concatenated result,
>>> print(fsm1+fsm2)
 name final? a anything_else
------------------------------
* 0    False    1
  1    False    2
  2    True

We see that "a" which is not in fsm2's alphabet does not get treated as if it were anything_else as it should.
To fix this, we check to see if the character is not in the alphabet and create the same transition as anything_else (if anything_else has such a transition).
This however doesn't seem to be an issue when concatenated through the lego module though.
Edit: lego merges the alphabet before converting to fsm.
>>> l1=lego.parse("[^a]")
>>> l2=lego.parse(".")
>>> print(l1.to_fsm()+l2.to_fsm())
  name final? a anything_else
------------------------------
* 0    False    1
  1    False    2
  2    True
>>> print((l1+l2).to_fsm())
  name final? a anything_else
------------------------------
* 0    False    1
  1    False  2 2
  2    True

The above example test has been added

This fix is in greenery version 3.3.2.


GCS Additonal e2e tests

Please squash commits & rebase so that we can merge.

All commits were Squashed, and the code was rebased. When the build is green. I'll notify you when it's ready to be merged.


Plug-in completely broken since latest update.

TV became unresponsive in HomeKit after update. I removed to try and re-add the device and now I cannot get it back into HomeKit.
After adding the bridge to HomeKit the tv doesn’t show up even after it has been set up in configuration… very frustrating.
Please help!

Did you manually add the tv in the home app? What errors do you get? Please post a debug log.

I'm having similar issues. Although I didn't try to re-add the device, it seems something broke after the iOS16 update. If I turn it on or off the plugin doesn't seem to register anything.

I had the same issue and then I came here and found this and got it working.
So it wasn't clear to me but..
You have to mannually add the TV and not the Bridge.


Handle PowerShell 6.0 executable which changed to pwsh

Please provide a configuration option where the linux executable can be forced to 'pwsh' instead of 'powershell' for versions of PowerShell 6.0 and greater.

Hello,
In 2.0 version it is possible to set a different executable.
Just pass it as an argument of the openSession method.
Best regards


Should not delete query.ppOptions.

Should not delete query.ppOptions in this line:
https://github.com/dearrrfish/preprocess-loader/blob/d49452a2943334c4c6d28b3c6f46cdee21769bf2/index.coffee#L53
Because when the next time the loader run the query.ppOptions is missing.

I'm also faced this issue. @dearrrfish can you please take a look?

It happens in case of using with webpack 2+ and options object, not via loader string.


feat: migrate Redis to public

Description
Migrating the internal Redis module to the public
Types of changes in this PR
No release required

[x] Examples or tests (addition or updates of examples or tests)
[x] Documentation update
[x] CI-related update (pipeline, etc.)
[ ] Other changes that don't affect Terraform code

Release required

[ ] Bug fix (patch release (x.x.X): Change that fixes an issue and is compatible with earlier versions)
[x] New feature (minor release (x.X.x): Change that adds functionality and is compatible with earlier versions)
[ ] Breaking change (major release (X.x.x): Change that is likely incompatible with previous versions)

Release notes content
Replace this text with information that users need to know about the bug fixes, features, and breaking changes. This information helps the merger write the commit message that is published in the release notes for the module.

Checklist for reviewers

[ ] The PR references a GitHub issue.
[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

Merge actions for mergers

Merge by using "Squash and merge".

Use a relevant conventional commit message that is based on the PR contents and any release notes provided by the PR author.
The commit message determines whether a new version of the module is needed, and if so, which semver increment to use (major, minor, or patch).

This PR has drifted quite alot to the code in our internal module, so I am going to close this PR to ensure a new one is created based on the code base at the time of migration


No module named 'networks'

By running the
python main.py --video_file data/videos/ayush.mp4 --path results/ayush \
  --camera_params "1671.770118, 540, 960" --camera_model "SIMPLE_PINHOLE" \
  --make_video

I get the following error:
Traceback (most recent call last):

File "/home/Projects/consistent_depth/consistent_depth/third_party/flownet2/models.py", line 12, in
from networks.resample2d_package.resample2d import Resample2d
ModuleNotFoundError: No module named 'networks'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "main.py", line 5, in
from process import DatasetProcessor
File "/home/Projects/consistent_depth/consistent_depth/process.py", line 10, in
from flow import Flow
File "/home/Projects/consistent_depth/consistent_depth/flow.py", line 15, in
import optical_flow_flownet2_homography
File "/home/Projects/consistent_depth/consistent_depth/optical_flow_flownet2_homography.py", line 11, in
from third_party.flownet2.models import FlowNet2
File "/home/Projects/consistent_depth/consistent_depth/third_party/flownet2/models.py", line 23, in
from .networks.channelnorm_package.channelnorm import ChannelNorm
File "/home/Projects/consistent_depth/consistent_depth/third_party/flownet2/networks/channelnorm_package/channelnorm.py", line 5, in
import channelnorm_cuda
ModuleNotFoundError: No module named 'channelnorm_cuda'

+1   I met the same problem

I think it's an issue of uncompiled CUDA pakages. The documentation is not fully detailed on this
You need a CUDA developer setup, so both CUDA and NVCC
then after cloning and setting up you need to go to third_party/flownet2/networks/ and for each of the folders, so channelnorm_package, correlation_package and resampl2d_package enter in the folder and run python setup.py install to install and compile the packages.


Typo

There is a typo in "ch of these commands has the same result." in "Examples" section. Probably should be "Each" rather than "ch".
File: engine/reference/commandline/export.md, CC @gbarr01

Thanks for reporting this @bigb123.
It seems this was fixed upstream, so it should make its way to the docs soon.

Oh... It seems @gbarr01 has a fix for this so I'm re-opening this issue.


load d3 from requirejs

d3 is being loaded from only commonjs and window.
So I added one more for amd, requirejs.
For some cases that using requiejs and not loading d3 in requirejs, added error handling when d3 is not loaded before loading d3-context-menu in requirejs, then d3 in window will be loaded.

Looks good, thanks! I'll try and update npm later tonight.


Bug: Missing "Start in" label in the shortcut properties

Description
When inspecting the properties of a file shortcut there is a text field below arguments for the "Start in" property without a label next to it. In the default Windows Explorer this property is called "Start in" and lets users select the directory in which a shortcut is executed.
The text field "C:\Users\lucas\Desktop" doesn't have any description:

Steps To Reproduce

Open the properties for an shortcut to an executable using the context menu or through the quick actions at the top.
Navigate to the "Shortcut" options on the left menu.
The text field below "Arguments" doesn't have a label.

Requirements
The "Start in" property should be properly labeled. An option without any description can be unintuitive for users.
Files Version
<IP_ADDRESS>
Windows Version
22H2 Build 19045.3208
Log File
debug.log

Thank you for reporting this bug.

I don't know where is the bug but these lines are important:

https://github.com/files-community/Files/blob/7de0fa4786e1b445fde661a597532e61b5308a5b/src/Files.App/Views/Properties/ShortcutPage.xaml#L64-L70
https://github.com/files-community/Files/blob/7de0fa4786e1b445fde661a597532e61b5308a5b/src/Files.App/Strings/en-US/Resources.resw#L531-L533

I still don't understand why x:uid= isn't being used...

@Jay-o-Way x:Uid isn't very readable.

This issue is resolved in main.


HSEARCH-2059 Upgrade to Hibernate ORM 5.0.5.Final and align related d…

…ependencies

https://hibernate.atlassian.net/browse/HSEARCH-2059

Merged https://github.com/hibernate/hibernate-search/pull/955


Implement  EU User Consent Policy - missing plugin

Godot version
4.2.1
Plugin version
3.1.0
System information
Sonoma 14.3.1 (23D60)
Phone information, if applicable
iPhone 8 and Pixel 5
Issue description
I'm implementing the consent form as instructed in the documentation, but when I'm executing at runtime the test the plugins are not loaded:

PoingGodotAdMobUserMessagingPlatform not found, make sure you marked all 'PoingAdMob' plugins on export tab
PoingGodotAdMobConsentInformation not found, make sure you marked all 'PoingAdMob' plugins on export tab
PoingGodotAdMobUserMessagingPlatform not found, make sure you marked all 'PoingAdMob' plugins on export tab

Ads are configured and working as expected with mediation (META)
Can you advice if there is an additional step to be completed? I would expect this plugin to be loaded with the AdMob plugin and the binaries included in the core and ads package.
Steps to reproduce

Implement consent form as for documentation
Run on device (iOS/Android)

Additional context
Message is generate when loading the consent form
UserMessagingPlatform.consent_information.get_is_consent_form_available():
as the _get_plugin method is not finding the given plugin

I needed to export the project with the debug flag once again and the plugin is now avaialble


icc build fails on NASA Pleiades

Original author: Thomas Quinn (@trquinn)
Original issue: https://charm.cs.illinois.edu/redmine/issues/1560

Building with
./build ChaNGa verbs-linux-x86_64 cuda smp icc -j8 --with-production
gives errors like:
../bin/charmc  -optimize -production  -I.   -c -o DummyLB.o DummyLB.C
In file included from sdag.h(28),
from CkMarshall.decl.h(6),
from charm++.h(134),
from LBDatabase.decl.h(3),
from LBDatabase.h(100),
from BaseLB.h(9),
from CentralLB.h(9),
from DummyLB.h(9),
from DummyLB.C(6):
../bin/../include/pup_stl.h(51): error: namespace "std" has no member "unique_pt
r"
inline void operator|(er &p, std::unique_ptr<T, std::default_delete> &ptr
);
^
Here is the icc version: icc -v
icc version 16.0.2 (gcc version 4.3.0 compatibility)

Original author: Thomas Quinn (@trquinn)
Original date: 2017-05-15 03:20:44

That last line was a clue: it looks like the intel compiler depends on the g++ libraries, so a modern gcc has to be loaded as well as the intel compiler.  On pleiades:
module load gcc
module load comp-intel
is needed.

Original date: 2017-05-15 03:55:34

Yes, that is true on a few other systems too. AFAIK we decided to require at least gcc v4.4 headers for 6.8.0, and we are planning for future releases after 6.8 to require full C++11 support. Phil knows more about the specifics here.

Original date: 2017-05-16 14:17:19

We ran into the exact same issue on some of the NERSC Cray systems.
We could maybe push a test that would trigger this failure into the configure process, and give a more explicit error explaining what went wrong and how to fix it. In the meanwhile, it should probably get at least a release notes entry.

Original date: 2017-05-16 17:37:15

If we don't catch this during configure and abort with a message explaining why it failed, I think we're basically asking for future emails to the charm mailing list about it.

Original date: 2017-05-18 19:34:10

Notes on the errors encountered with different icc/gcc version matchups:
|.gcc \ Intel            |.12              |.13     |.14       |.15      |.16       |.17         |
|<. 4.3 (Edison default) |                  |         |           |          | %{color:red}unique_ptr not defined in std% | %{color:red}unique_ptr not defined in std% |
|<. 4.4                  |module won't load - conflict|module won't load - conflict|module won't load - conflict|module won't load - conflict| module won't load - conflict with intel module| module won't load - conflict with intel module|
|<. 4.5 (edison n/a)     |   -              |  -      |    -      |  -       | -         | - |
|<. 4.6 (edison n/a)     |   -              |  -      |    -      |  -       | -         | - |
|<. 4.7                  |                  |         |           |          |%{color:red}icpc: internal error: Assertion failed (shared/driver/drvutils.c, line 670)%| %{color:red}icpc: internal error: Assertion failed (shared/driver/drvutils.c, line 670)%|
|<. 4.8                  |                  |         |           |          |%{color:green}good%| %{color:green}good% |
|<. 4.9                  |                  |         |           |          |           | %{color:green}good% |
|<. 5.3                  |                  |         |%{color:green}good% with older PrgEnv-intel/5.1.29|%{color:green}good%| %{color:green}good%| %{color:green}good% |
|<. 6.1                  |                  |         |           | %{color:red}see #1372%|Error #3802: Version 16.0 Update 2 is incompatible with GNU versions greater than or equal to 6.0, due to an incomplete implementation of SFINAE which is used extensively in gcc6 headers. If you wish to proceed, use option -wd3802| %{color:green}good%|
|_<. 6.2                  |                  |         |           |          |Error #3802: Version 16.0 Update 2 is incompatible with GNU versions greater than or equal to 6.0, due to an incomplete implementation of SFINAE which is used extensively in gcc6 headers. If you wish to proceed, use option -wd3802| %{color:green}good%|

Original date: 2017-05-18 20:37:45

The Intel 16 / GCC 6 combination errors at compile time. The configure output also shows error messages, but somehow doesn't detect those as failure, so it continues.

Original date: 2017-05-19 14:08:55

~~https://charm.cs.illinois.edu/gerrit/2542~~ https://github.com/UIUC-PPL/charm/commit/cd4d6f80b970ae702874a14404189f042e83478a

Original date: 2017-05-30 14:05:14

This seems to have broken the Edison autobuild targets.

Original date: 2017-05-30 14:51:39

Was working as intended. I had screwed up the login script configuration for the account used by autobuild, to say module load gcc/6.3.0 when the latest available was gcc/6.2.0. I switched it over to gcc/5.3.0 and confirmed that it now passes configure.

Original author: Jim Phillips (@jcphill)
Original date: 2017-05-30 16:03:28

This test is now failing on my build box because it doesn't pass through the "-gcc-name=gcc44 -gxx-name=g++44" options included on the build command line to icpc, and just runs "icpc -I../include -I. -c test.cpp -o test.o -h std=c++11" instead.

Original date: 2017-05-30 16:52:31

Jim:
I think the definitive resolution of the exact issue you're now seeing would be to pass $OPTS to the various compilers as we test them in configure. That would be a fairly high-risk change, and I feel like we've tried that at some point in the past, and encountered a lot of trouble.

Next, try out a sample program like netlrts-darwin-x86_64-cxx1y.2/tests/charm++/simplearrayhello

Original author: Jim Phillips (@jcphill)
Original date: 2017-05-30 17:43:00

Seems to have failed the auto-test.  Will test after that passes.

Original author: Jim Phillips (@jcphill)
Original date: 2017-05-30 18:28:54

The fix works. Thanks.

Original author: Jim Phillips (@jcphill)
Original date: 2017-07-06 21:56:49

When I pass OPTS_CXX="-gxx-name=g++44" to the build script it is preserved for future charmc invocations, which breaks on other machines.

Original date: 2017-07-06 22:04:46

Jim, could you please elaborate?

Original date: 2017-10-27 14:26:03

We need to update this for 6.9.0 and C++11 support. Our configure script points people to this issue if using ICC with old GCC headers.

Original author: Jim Phillips (@jcphill)
Original date: 2018-01-08 19:39:51

Wanted to update with two alternative solutions, neither of which preserves the options in the build config files.
First, rather than trying to pass the -gcc-name and -gxx-name options via the Charm++ build script, they can be set via the __INTEL_PRE_CFLAGS environment variable:
setenv __INTEL_PRE_CFLAGS "-gcc-name=gcc44 -gxx-name=g++44"

The modern way to install and access newer compilers is via the scl package (https://www.softwarecollections.org/en/scls/rhscl/devtoolset-6/):
scl enable devtoolset-6 -- ./build charm++ netlrts-linux-x86_64 ...


Intellicode Swp & ram high usage

Intellicode plugin hog up all the Swp space as well as a memory. For each new window, 12 new instance of Intellicode appears.
This causes vscode to crash as well as hangs the system.
This can be viewed by running htop.
Attaching images below for reference :

Image after disabling intellicode :

Environment:
OS: Ubuntu 19.10
vscode: v 1.40.2
intellicode: v 1.2.0

@tejassrivastava are you still seeing this issue? Can you give more information about your repro (what language are you using for instance, and do you see this problem all the time?)

Going to close this for now. Customers can feel free to re-open if they encounter this


[Event Request] report 780 "Certificate of Supply" - protected variables

Please set varibles TempServiceShipmentHeader and TempServiceShipmentLine to protected in report 780 "Certificate of Supply"
protected var
    TempServiceShipmentHeader: Record "Service Shipment Header" temporary;
    TempServiceShipmentLine: Record "Service Shipment Line" temporary;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 47978.


[release-1.7] Use istiod label selector

This is an automated cherry-pick of #27431
Resolve: https://github.com/istio/istio/issues/27756

Could you please add a release note?

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio_release-1.7
5b6bcdaec863f363e0d6c0424125644a3bf9ab00
link
/test unit-tests_istio_release-1.7

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test unit-tests_istio_release-1.7

Guys, in which version this fix will be available? As we are migrating to 1.7.3 and have this issue?


Problem with transparent png image

Dear,
I need to format a png image that is to paste a signature generator certificates.
The problem is that the image is transparent and reformatting (leave of 650x650 px) leaves completely black.
The code is:
$signature = Image::make($destinationPath.'/'.$fileName);
$width = $signature->width;
$height  = $signature->height;
$signature->resizeCanvas(($width > $height) ? $width : $height, ($width > $height) ? $width :    $height, 'center', false, 'rgba(0, 0, 0, 0)');
$signature->resize(650, 650);
$signature->save($destinationPath.'/'.$fileName);
The idea is that the canvas is completely transparent.
greetings from Chile !

Because you're using the width/height properties, I assume you're using version 1.*
I recommend to upgrade to version 2.*.

Dear, thank you very much worked perfect, actually it was version 1. *, Greetings!


Add ability to set custom underling grist texture

this can be done by placing your textures under kubejs/assets/minestuck/textures/entity/underlings/<GRIST>.png


Introduce Char primary type

The following
!(== 'A' "A")
!(get-type 'A')
!(get-type "A")
!("A")

outputs
[False]
[%Undefined%]
[String]
[('A')]

Indicating that even though "A" is displayed as 'A', the latter is neither a String nor a Char.
There are several fixes for that, the first one being to displaying "A" as "A".  In addition I would suggest to introduce a Char primary type that would be parsed with single quotes like traditional functional programming languages such as Haskell, Idris and Racket.
If you agree to that I don't mind taking care of it.

@ngeiswei , there is definitely some confusion. There are two reasons: the default way to print a string in Python is enclosing it into single quotes ' and parsing rules for a quoted sequences of characters are hardcoded in Rust.
When user runs a Python MeTTa interpreter both Python and Rust code works together. MeTTa parser (written in Rust) uses double quotes " to recognize a quoted sequence of characters as a token. Then the token is converted into a grounded string by the Python MeTTa standard library. Thus the resulting grounded atom is a Python object. The procedure of conversion to a string is written in Python. By default in Python string is printed using single quotes '. Before https://github.com/trueagi-io/hyperon-experimental/pull/266 it had been printed without quotes at all.
One issue here is that in Python we have a universal procedure to print all Python grounded atoms. And it uses single quotes for strings even when we know MeTTa accepts only double quoted sequences with spaces. Quick fix for this is to write a separate class to represent strings in Python and override the __repr__() method to print the string in double quotes. Also adding a single quoted character into Python and Rust MeTTa stdlib is trivial.

The more wider issue here is a bad decoupling of Tokenizer and SExprParser. Ideally there should be the only module responsible for a token recognition. But at the moment SExprParser does a part of the work by recognizing sequences of characters which are tokens, and Tokenizer specifies whether a sequence of characters is converted to a non-symbol atoms. In particular the SExprParser recognizes the double quoted sequences with spaces as tokens. This means Tokenizer which is configured by regular expressions cannot fully control how quoted character sequences are handled. The direct consequence is that one cannot add a single-quoted string (which includes spaces) by configuring the Tokenizer.
To fix this we need to combine SExprParser and Tokenizer to rule the process of parsing by tokens provided by the user. In such case the user can configure the tokens totally on a client side without modifying the code of parser. It means it is possible to configure single quoted strings or double quoted strings without harcoding the procedure inside the parser. But this fix will require more work on the Rust side and total rework of the parser. And it is a separate issue for sure.

@ngeiswei , I don't mind adding the character token, please feel free to fix it.


Object detector doesn't find matches

Hi, I have setup the current version of turicreate in linux and tried training a model for object detection. The training completes but the model doesn't find objects in test images, either trying predict() or exporting the model to the sample iOS app.
I have checked the training images and annotations are correct using util.draw_bounding_boxes() and it looks good.
I also tried the same data in the MakeML app, that seems to use turicreate 5.2.1 the model from that works correctly with their test tool and also in iOS.
Does anyone have any clue how to debug this? When training the loss is decreasing by a similar amount as seen in the MakeML app. I would rather be able to run it on my own hardware. Thanks.

I have re-installed linux and stared again and now I can create the model and it does find matches, so must have been something I was doing wrong. You can close this.
The model I am getting does not work as well as the one from MakeML though, I guess I could try installing turicreate 5.2.1 but that is a different question.

We have a serious issue with our TensorFlow implementation of object detection, this is what gets used on Linux. It's completely broken with our latest (6.2) release. If you have more than two classes in your dataset, then things have been broken since 6.0 (see #3031).
So if you have two or less classes/objects in your dataset, please use TuriCreate 6.1. If you have more than two classes, please use 5.8.
I am actively looking into these object detection issues. I'm planning to do a new release of TuriCreate once these issues are resolved.

I have installed 6.1 and it seems to be working as expected now (I have one object). Thanks.

We just released a point release (TuriCreate 6.2.2) with this bug fix. Upgrade your version of TuriCreate and you should be good to go.


Using other model on top of YOLOv3 to recognize other features

Hi all,
this is more like a question and not an issue...
Is there any way to use any model alongside YOLOv3 to recognize other vehicle features such as make and model...
My idea is to tweak the recognition process realized by YOLOv3 of the different objects as follows:

If YOLOv3 detects a vehicle in a frame, then with the help of OpenCV, I send the recognized vehicle (its cropped image) to another model to detect the remaining features (make and model)

The MongoDB collection that stores the recognition data needs to be adapted to allow the persistence of these extra recognized features

Such a scenario would be preferably deployed on a server due to the additional processing steps.
Any help/guidance in the code is much appreciated...
Thanks in advance!

Hi

You should look into darket to add this feature of running an extra model on top of YOLO on the bounding boxes .. In my opinion it is not very easy to do.. but should be possible

Regarding hooking this to OpenDataCam, if the extra models run fast and you are able to add this extra metadata to the JSON stream you can easily persist those in the MongoDB .. this is the easy part ;-)

Also you might be interested in looking into Deepstream v5 from nvidia which does this kind of thing.. but yes without a nice UI like OpenDataCam
Good luck !

Thanks for your help. I will have a look at Deepstream v5 :)

great, closing but feel free to report anything you can share back!


Bug: Integration With Golang Not Working

Describe the bug
dbConn.Query("UPDATE my_table SET flag = false WHERE some_int = 10", map[string]interface{}{}) works fine.
But
dbConn.Query("UPDATE my_table SET flag = false WHERE some_int >= 10", map[string]interface{}{})
dbConn.Query("UPDATE my_table SET flag = false WHERE some_int <= 10", map[string]interface{}{})
dbConn.Query("UPDATE my_table SET flag = false WHERE some_int > 10", map[string]interface{}{})
dbConn.Query("UPDATE my_table SET flag = false WHERE some_int < 10", map[string]interface{}{})

are all hanging until GoLang client terminates them after about 240 seconds of waiting.
Similar issues happen with SELECT queries. Comparing with integers using = is ok, comparing with inequality operators never works. It doesn't matter whether the argument is passed directly or mapped through the second param of Query().
Steps to reproduce
Run SurrealDB server 2.0.3, connect to it using the GoLang library, select namespace and database with Use(), and try to run the queries above on a non-existent table
Expected behaviour
I expected that all of the queries will successfully return in about no time, taking into account that they were all executed on a non-existing table
SurrealDB version
2.0.4 for linux on x86_64
Contact Details
<EMAIL_ADDRESS>Is there an existing issue for this?

[X] I have searched the existing issues

Code of Conduct

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

Hi @wojciechpietrzak which version of the Golang library are you using?


[feat] - Be able to differenciate anonymization jobs and run jobs

Is your feature request related to a problem? Please describe.
Since the mask-sobject framework can anonymize AND empty fields, it can be used for anonymization purposes (like post refresh or post data migration) and more often runs.
For example, you have a purge which occurs on a custom object, which id is referenced in a custom text field of another object (I know this is not the best implementation but sometimes you face some product limitations 😄 )
Once the data is purged, you need to empty the field which reference the deleted record, you can use mask-sobject framework to do this.
Describe the solution you'd like
It would be great if we could execute the batch in 'anon' context or in 'run' context.
Each MaskSObjectField___c should reference the context to run whith a custom picklist ('anon', 'run', 'both').
We should also add a comment field to give more context about the business need.
We should also add a schedulable class to be able to schedule run mode execution

Addressed by https://github.com/tprouvot/mask-sobject/commit/b6332772479f61bf5c416a7656112d4730e28361


Ensure iptables is installed

For support Ubuntu minimal version.

Hmm, can be moved package installing to debian and redhat persist configs? Of course, for support suse from an adjacent PR.

I will close this PR and instead open an issue to raise the question if iptables needs to be installed.


WIP: One click spin up a nostr relay

For now includes a simple enable enable disable button, will later include diagnostic UI and management UI  (key allow/ban lists and the like)

Codecov Report

Merging #1413 (c10d158) into main (0de2a62) will increase coverage by 0.10%.
The diff coverage is 88.63%.

@@            Coverage Diff             @@
##             main    #1413      +/-   ##
==========================================
+ Coverage   48.60%   48.71%   +0.10%
==========================================
  Files         294      298       +4
  Lines       16641    16685      +44
==========================================
+ Hits         8089     8128      +39
- Misses       8552     8557       +5

Impacted Files
Coverage Δ

lnbits/extensions/nostrrelay/views_api.py
81.81% <81.81%> (ø)

lnbits/extensions/nostrrelay/views.py
86.66% <86.66%> (ø)

lnbits/extensions/nostrrelay/__init__.py
91.66% <91.66%> (ø)

lnbits/extensions/nostrrelay/migrations.py
100.00% <100.00%> (ø)

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

Changes happening here https://github.com/lnbits/nostr-relay-extension


Add Frame Node: To group Nodes and move them together easily.

Problem
Currently, the only way to group Nodes is to do so temporarily: draw a square, select Nodes, move them around. This is enough when dealing with small dialogue trees, but as they grow, it can become difficult to organize properly.
Proposed Solution
Add a Frame item or FrameNode which has no connections, it's only purpose being to move all Nodes that are within as it is itself moved. The GraphFrame Godot Node seems to be made specifically for such a function.

GraphFrame added to the plugin. Thanks for the suggestion!


Create integration test utility to binary diff original & um2nc dev sourced netCDF data

Closes #131.
This PR adds a utility to compare netcdf data between different commit versions of um2nc-standalone. This is a loose check to determine if basic binary compatibility is maintained between this repo's 1st commit um2nc-standalone and the version a user has checked out.
NOTE: This is not exhaustive & some options are not tested, therefore may not be backwards binary equivalent!
NOTE 2: This complements unit testing to ensure the CLI works & default runs are binary equivalent.

Argh, it needs another review as I made it a non-draft....


Add CORS support?

THIRD_PARTY_APPS = [
    "crispy_forms",  # Form layouts.
    "django_extensions",  # Django extensions.
    "django_filters",  # Search filters for Django REST Framework.
    "django_saml2_auth",  # SAML support.
    "recurrence",  # Used to configure periodic scans.
    "rest_framework",  # Django REST Framework.
    "rest_framework.authtoken",  # Enable token authentication for Django REST Framework.
    "drf_yasg",  # Django Rest Framework Swagger and ReDoc support.
    # "django_baker"  # Django baker is used to create CRUD files for each model.
    "corsheaders",
]

MIDDLEWARE = [
    "corsheaders.middleware.CorsMiddleware",
.
.
.
]

CORS_ORIGIN_WHITELIST = (
    "http://<IP_ADDRESS>:3000",
    "https://scantron-test.example.com",
)

Not going to add this right now.


[MCS-DE-GO] -- Configurable timeout value for calls to Skill bots

When calling a skill bot we noticed that we have the 15s timeout limitation.
Is there any easy way of making this timeout configurable? Do we need to have our custom Skill Connector class?
We might have situations in the future where the skill bot is handling long running processes (e.g. calculating the quotation for a new insurance proposal or aggregating history data from the user).
@goergenj @benbrown @sgellock

@benbrown triage appropriately. I have it in R11, given it's a customer issue

@luhan2017 can you look at the SkillController class in Composer's runtime template to see if this is configurable via that hierarchy, or if we need to move this to an ask for the SDK? Thanks

Hi, do you want to set a timeout when the Bot calls a Skill or when the skill responds to the parent bot?
For the first one the SkllHttpClient class takes an HttpClient instance in the constructor where you can set your own timeout

@joaollq , did you try on @gabog 's suggestion to see whether it meets your requirement? We will support this in the upcoming release.

@joaollq Can you comment?


⚠ Bofrost.com has degraded performance

In d3ff9d7, Bofrost.com (https://www.bofrost.com) experienced degraded performance:

HTTP code: 200
Response time: 891 ms

Resolved: Bofrost.com performance has improved in 6b5b087 after 5 hours, 1 minute.


GH-39845: [C++][Parquet] Minor: avoid creating a new Reader object in Decoder::SetData

Rationale for this change

avoid creating a new Reader object in Decoder::SetData
What changes are included in this PR?

avoid creating a new Reader object in Decoder::SetData
Are these changes tested?

Already
Are there any user-facing changes?
no

Closes: #39845

Also cc @pitrou


Base16 Tomorrow Dark - Incorrect

https://github.com/AndrsDC/base16-vscode/commit/3e1e9fc7e2552ae608ab1012a9a2ee8451a88983
It seems that the transition from the old format to the new format for Tomorrow Dark might be incorrect?
For example the foreground color in the old version is 1d1f21 where-as the new color is d6d6d6.
The colors in VSCode are also very wrong.

Before:

After:

Yeah I noticed the same thing as well after being updated to the latest version 😕

I got the new files from updated .json files made for VSCode, while the old files were converted from .tmTheme. I will look look into it. Maybe I'll revert to the old files but I'll see if there's another solution.

I recently encountered this as well. I guess there's no way to revert to old versions of extensions... 😞
For the moment, I published a fork of this that reverts back to v1.4.0, as I wasn't sure what the correct fix would be to submit a PR here.
https://marketplace.visualstudio.com/items?itemName=jryans.base16-themes
I'll unpublish my fork once some kind of fix is found here.

Actually, if you guys only need Base 16 Tomorrow Dark theme, there is an solo extension called Base16 Tomorrow Dark+, that works fine for me.

What I did on the latest update was take the latest .thTheme files for this themes and convert them with the built in tool in VSCode. I'll have to hunt for old-ish files.


doc: add W3C Validation badge

Pull Request Test Coverage Report for Build 1354

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 1353:
0.0%

Covered Lines:
44

Relevant Lines:
44

💛  - Coveralls


Passing variables to file specified in --config option

I'm using PHP script as a config file for Phinx, is it possible to pass variable to that file? I know I can get enviroment variables in that PHP file but it's not what I need since I'm running phinx in bash loop and I'm trying to pass found folders to that script

Can you maybe set the folders as env vars?

I know I can get enviroment variables in that PHP file but it's not what I need

@bezdutchek why is it not what you need when that would work just fine?

@bezdutchek What I meant was, you can set the env vars (new made up ones) in bash and read them in the php config file and transform the data into anything you need


Update autoChargePylontech.R

Fix orientation variable

New version works good.  I do have a question about the daily_estimates.csv.. I dont seem to have any values for actual.  I see i can link it to home assistant.  But I dont use HA.  Can these be pulled from the inverter??

"date","actual","mean","forecast.solar","solcast"
2023-01-21,NA,9330,2175,16484
2023-01-21,NA,9330,2175,16484
2023-01-21,NA,9330,2175,16484
2023-01-21,NA,9330,2175,16484
2023-01-21,NA,9330,2175,16484
2023-01-21,NA,10120,2277,17962
2023-01-21,NA,10120,2277,17962
2023-01-23,NA,3791,0,3791,0
2023-01-23,NA,3861,0,3861,0

Fix orientation variable

It's now fixed. Keep in mind that orientation for forecast.solar is 0 for due south (not 180!).

New version works good. I do have a question about the daily_estimates.csv.. I dont seem to have any values for actual. I see i can link it to home assistant. But I dont use HA. Can these be pulled from the inverter??

"date","actual","mean","forecast.solar","solcast"
2023-01-21,NA,9330,2175,16484
2023-01-21,NA,9330,2175,16484
2023-01-21,NA,9330,2175,16484
2023-01-21,NA,9330,2175,16484
2023-01-21,NA,9330,2175,16484
2023-01-21,NA,10120,2277,17962
2023-01-21,NA,10120,2277,17962
2023-01-23,NA,3791,0,3791,0
2023-01-23,NA,3861,0,3861,0

It should be possible but that will depend on your inverter and whether you can pull data from it directly. If you can read from your inverter you can simply change the code that populates the field "actual" with data from your inverter. It shouldn't be too complicated.


Is there need to use a European reference grid at 1x1km resolution?

Based on previous conversations, we found that no worldwide grid is available and it is actually not needed for risk assessment (RA). We need it at European level for RA and at Belgian level for indicators. Based on what I can find from European Environmental Agency page I cannot find any grid at 1x1km resolution at European level, only 10x10km and 100x100km.
@DiederikStrubbe, @amyjsdavis : do you need to calculate occupancy at European level at 1x1km or is it fine for you to get 1x1km resolution at Belgian level?
If you really need, do you have ideas how to get it? Maybe we can build it by using all reference grids at country level and join them together but we will have a lot of problems at country borders where duplicates will occur. Something to think about... Let me know. If you have a shape file for it, could you please upload it in ./data/external/? Thanks.

After our TrIAS core meeting we know that:

we work at European level for risk assessment, so data and grid covering all Europe are needed.
do we need to work at 1x1km level or is it for you enough the grid at 10x10km? If 1x1km grid, then we have to get it somehow. @amyjsdavis: do you have it? And from who? Can be found on internet? Is it an "official" grid? I couldn't find it on EAA ineternetsite (only reference grids at 10x10km and 100x100km). We can make it ourselves in a documented, open and repeatable way (TrIAS style...) by stitching the country reference grids together. Just let us know. If you find that working at 10x10km scale is enough, then we use the reference grid already provided by EAA.

@damianooldoni : Yes I have a grid. I have to add Norway to it. I will send it to you when it is ready.

Hi @amyjsdavis as discussed on phone, once your finished sending the grid to KMI, can you:

Include the link to the 1km grid (without Norway) in this issue
Add the code for adding Norway to the grid in this repo: https://github.com/trias-project/eaa-reference-grid-1km

No need to send us the file.

https://www.eea.europa.eu/data-and-maps/data/reference-grid-for-land-cover-accounts-leac/zipped-shape-file-eea-reference-grid-1k-vector-data-polygon-leac-coverage/zipped-shape-file-eea-reference

Great, thanks! I noticed this grid only covers land. Does that mean that marine species will be ignored in the analyses?

They will require a different framework and different set of data (e.g see http://www.bio-oracle.org/ and http://www.emodnet.eu/,
We will tackle this after we model the terrestrial and aquatic (non-marine) species.


Changed default visibility for caret - ListItemRightCaretWithLabel

Details

Changed default visibility for caret - ListItemRightCaretWithLabel
Fixed header styles to adjust with new change

Fixed Issues
$ https://github.com/Expensify/App/issues/51053
PROPOSAL: https://github.com/Expensify/App/issues/51053#issuecomment-2420376995
Tests
Same as QA tests

[x] Verify that no errors appear in the JS console

Offline tests
Same as QA tests.
QA Steps
Prerequisite : A workspace with Distance Rates, Categories, Tags, Taxes, Report Fields.(Enable from More Features)

Navigate to settings > workspaces.
Open Distance Rates
Verify '>' is not visible for list item
Verify header text aligns with list item elements(ie. status aligns with enable/disable....).
Select a few items - verify header text aligns with list item elements(ie. status aligns with enable/disable....).
Repeat step 2-5 for Categories, Tags, Taxes, Report Fields.

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
https://github.com/user-attachments/assets/45e53ebc-838b-4668-a031-0d868a2048da

Android: mWeb Chrome
https://github.com/user-attachments/assets/3db366b0-e9c8-49aa-b8a1-a92fbac267a0

iOS: Native
Uploading Screen Recording 2024-10-22 at 2.08.17 PM.mov…

iOS: mWeb Safari
https://github.com/user-attachments/assets/af450cf2-ed02-4138-82c9-64d63b308b48

MacOS: Chrome / Safari
https://github.com/user-attachments/assets/93aa1de6-07c9-48a9-826d-2208f3b1fba8

MacOS: Desktop
https://github.com/user-attachments/assets/8840d35c-b057-490a-a423-129b4bf9ffa9

Adding videos in a while

Going to run a test build for us.

Can't seem to login with the test build but the screenshots above look good to me from a design perspective, cc @Expensify/design

Can't seem to login with the test build but the screenshots above look good to me from a design perspective,

Out of curiosity, what error you hitting? Is it infinite skeletons?

Nope, getting the bug page where you can either refresh or sign out.

Hmm I don't see what you are talking about, maybe you can bring this up in the open source channel for discussion though!

I am not yet invited to slack channel, but NVM.

Screenshots/video is looking pretty good to me too. Nice.

The screenshots and styles look good to me. But I don't quite understand why we had to write a totally new component CustomListHeader to accomplish this? Can someone help me understand that part?
I would think this should be as simple as just removing the caret icon from these table styles.

Got it, will let @dangrous weigh on on the technical aspects then :)

I don't think we want any carets anywhere on table rows anymore in case that's helpful for your question.

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.


Cutting code uses keyframe search

The download and cut script currently uses keyframe searching when cutting into clips. Full re-encoding is necessary to ensure frame alignment: http://www.markbuckler.com/post/cutting-ffmpeg/

Fixed with commit b62a98fefbf83e8737deb13656b5f81a47b07fbf
(https://github.com/mbuckler/youtube-bb/commit/b62a98fefbf83e8737deb13656b5f81a47b07fbf)


VSCode does not honor .gitignore file if the project has no .git directory

Issue Type: Bug
Remove .git directory or workspace's .git  file from the project.
Result: VSCode stops honoring the .gitignore file.
Expectation: VSCode honors .gitignore file irrespective of the presense of the .git directory or file.
Many other applications (e.g. rsync) honor .gitignore file no matter what :)
VS Code version: Code 1.31.1 (1b8e8302e405050205e69b59abb3559592bb9e60, 2019-02-12T02:16:38.656Z)
OS version: Darwin x64 17.7.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7920HQ CPU @ 3.10GHz (8 x 3100)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
16.00GB (0.82GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (24)

Extension
Author (truncated)
Version

vscode-direnv
Rub
0.0.2

numbered-bookmarks
ale
6.1.0

bracket-pair-colorizer
Coe
1.0.61

vscode-eslint
dba
1.8.0

githistory
don
0.4.6

gitlens
eam
9.5.1

tslint
eg2
1.0.43

prettier-vscode
esb
1.8.1

git-project-manager
fel
1.7.1

code-runner
for
0.9.7

vscode-pull-request-github
Git
0.4.0

githd
hui
2.1.0

rest-client
hum
0.21.1

asciidoctor-vscode
joa
2.3.3

my-keymap
MY
0.0.1

vscode-extension-auto-import
Nuc
1.4.3

indent-rainbow
ode
7.2.4

vscode-yaml
red
0.3.0

partial-diff
ryu
1.4.0

code-settings-sync
Sha
3.2.5

incremental-search
sie
0.2.1

html-preview-vscode
tht
0.2.5

json2yaml
tux
0.2.0

vscode-surround
yat
1.0.2

No, the design is to shell out to git for checking what's ignored, see https://git-scm.com/docs/git-check-ignore, because only that provides the true picture of what's being ignore by git.

I expect VSCode to ignore files and directories mentioned in .gitignore irrespective of the presence or absence of the .git directory. For example, it shouldn't search for files in those directories/files.
There are many well known utilities that understand and use .gitignore files to do the filtering. One of them is rsync which does not required git to be installed.
I understand why it could be hard to implement in VSCode, but I would called it as-implemented, not as-designed :)

For example, it shouldn't search for files in those directories/files.

It does that, but for greying out files in the explorer is consults git as the source of truth.

I understand why it could be hard to implement in VSCode,

This isn't about that. The problem would be the false impression, e.g you add a file to .gitignore, the explorer shows the file as ignored (greyed) but subsequent edits in that file show as modification because it's actually not ignored.

It does that

This is how I discovered the "problem". I made a copy of my project (don't ask me about reasons :) to do some work. Obviously I didn't copy the .git. After that search in files stopped ignoring directories mentioned in .gitignore so I got a lot of matches where I expected just a few of them. Frankly, this is the biggest inconvenience to me.

but subsequent edits in that file show as modification because it's actually not ignored.

We should define what does it mean to be 'ignored' :) To me, ignored means that the file is not considered to be a part of the project. It just resides there among other files (but ignored). In a sense, .git directory is not a part of the project. This is just how git stores things. That's why VSCode doesn't search in the .git directory.