Question: core methods accessibility from scenefrom

Hi,
we have a core session we want to migrate to ArFragment to be able to use its methods.
It is possible to use the standard core methods from within ArFragment and just add the desired functionalities or we have to re write all the code for ArFragment?
Do we have any example?
Thanks.

If I'm reading this correctly, you already have an ArCore Session and you'd like to reuse it in ArFragment.  In v1.0 there is no API to use a Session created elsewhere.  I'll record that as a feature request.
If you want to call you own functions on the Session we have created this is possible.  You need to get the ArSceneView via ArFragment.getArSceneView()
Then call ArSceneView.getSession().

Hi,
I have my own methods impelemented in a core Session. I am using GLES shaders and a lot of code working without sceneform.
Question is, may use the core functionalities inside sceneform ArFragment ARcore session?
This way with little work I can cut&paste the actual code to the ArFragment and avail of the new features without the need to rewrite it. EG: I can keep my shaders and mix them with the new code which will use sceneform specific content.

Can we get more info about exactly what you are doing?
Unfortunately, you can not add handwritten OpenGL into Sceneform as of V1.0.  Our next release will have more shader functionality so we may be able to achieve a similar effect, but you still will not be able to copy and paste a shader over.

Ok,
I ll continue to work with the core functionalities then until will be quick to migrate on this.
Thanks again, you can close the case.


register with smartAPI

See https://github.com/dcppc/data-stewards/issues/5 - we are in the same position as the gtex people as we don't hand edit the yaml, it is generated from annotations and we are using a framework that supports openapiv2
I think these steps will work:
Write a build/makefile step that will take
https://api.monarchinitiative.org/api/swagger.json
Use https://www.npmjs.com/package/api-spec-converter
to convert to open API v3, and register

Currently our swagger JSON is Swagger 2.0, which is generated by Flask-RESTPlus.
But there are some issues with the generated JSON where it fails validation. Minor issues here and there, but it gets in the way of using the swagger JSON with SmartAPI and also converting the JSON from Swagger 2.0 to OASv3.0.
Note to self: Investigate into alternatives to Flask-RESTPlus, that generates OASv3.0 automatically instead of writing them manually.


Cloud backups

@MattDHill
export type BackupTarget = DiskBackupTarget | CifsBackupTarget | FtpBackupTarget

// physical disk, DiskInfo previously defined
export type DiskBackupTarget = { type: 'disk' } & DiskInfo

// Windows shared folder
export interface CifsBackupTarget {
    type: 'cifs'
    hostname: string
    path: string
    username: string
}

// FTP server
export interface FtpBackupTarget {
    type: 'ftp'
    hostname: string
    path: string
    username: string
}

backup.list-targets :: {} -> BackupTarget[]
backup.create :: { target: BackupTarget, target-password: string | null, old-password: string | null, password: string } -> WithRevision<null>
package.backup.restore :: { ids: string[], target: BackupTarget, target-password: string | null, old-password: string | null, password: string } -> WithRevision<null>
setup.execute :: { embassy-logicalname: string, embassy-password: string, recovery-location: BackupTarget, recovery-location-password: string | null, recovery-password: string } -> SetupResult

disk.backup-info should be eliminated in favor of backup.target.info, which takes in a target-id instead of a logicalname.

afaik, Matt your role on this ticket is complete and it is entirely waiting on the embassyd side at this point right?

Minus testing on live BE, correct.


show "completed ingest records" for more than 30 days

because ingests are sometimes done and not completed/published, it would be helpful to be able to see completed ingest records for more than 30 days, ideally for six months and potentially up to the previous year/365 days. Ideally, also, we would be able to filter for unpublished records so that we could limit the queue to those which had not been reviewed/published.

The update were pushed to production prematurely due to other bug fixes but it appears to be stable.  I'm going to begin adding a "publish all" by collection for recent imports.


where is the example code about datetime in Mysql?

Setup
Versions

**Rust:**1.36.0-x86_64-pc-windows-msvc
Diesel: 1.4.0
Database: 8.0.17
Operating System Windows 10

Feature Flags

diesel:
diesel = { version = "1.4.0", features = ["mysql", "chrono"] }

Problem Description
the trait diesel::Queryable<diesel::sql_types::Datetime, diesel::mysql::Mysql> is not implemented for diesel::sql_types::Datetime
What are you trying to accomplish?
known how to use the type about datetime
What is the expected output?
What is the actual output?
^^^^ the trait diesel::Queryable<diesel::sql_types::Datetime, diesel::mysql::Mysql> is not implemented for diesel::sql_types::Datetime
Are you seeing any additional errors?
Steps to reproduce

Checklist

[ ] I have already looked over the issue tracker for similar issues.
[ ] This issue can be reproduced on Rust's stable channel. (Your issue will be
closed if this is not the case)

Our issue tracker is used to track bugs and feature request for diesel. For asking questions please use our gitter channel.
To answer that specific question see this section of our documentation.

Diesel maps "Rust types" (e.g. i32) to and from "SQL types" (e.g. diesel::sql_types::Integer). You can find all the types supported by Diesel in the sql_types module. These types are only used to represent a SQL type. You should never put them on your Queryable structs.

Actually I am wondering if this is a bug. I added a column of datetime in getting-started demo,
CREATE TABLE `posts` (
  `id` int(11) NOT NULL AUTO_INCREMENT,
  `title` varchar(255) NOT NULL,
  `body` text NOT NULL,
  `published` tinyint(1) NOT NULL DEFAULT '0',
  `create_time` DATETIME default NOW(),
  PRIMARY KEY (`id`)
) ENGINE=InnoDB AUTO_INCREMENT=0 DEFAULT CHARSET=utf8mb4;

And then the automatically generated schema.rs has :
table! {
    posts (id) {
        id -> Integer,
        title -> Varchar,
        body -> Text,
        published -> Bool,
        create_time -> Nullable<Datetime>,
    }
}

And then, it gives a compiling error:
the trait FromSql<diesel::sql_types::Nullable<diesel::sql_types::Datetime>, Mysql> is not implemented for chrono::naive::datetime::NaiveDateTime
However, if changed Nullable to Timestamp, the error is gone.
Like this example:
https://github.com/pymongo/diesel_crud/blob/main/src/main.rs


Feature request: Include .strings file comment in generated type documentation

For example, if I have in my .strings file:
"search-placeholder" = "Search %@"; /* Parameter is name of category. */

it'd be awesome to have the generated function and property documentation do something like
/// Base translation: Search %@
///
/// Locales: Base
///
/// - Note: Parameter is name of category.

so you get the localizer note right at the code use site. It's a helpful reminder of what the string is meant for and what the parameters are. It also makes bad/outdated localizer notes a bit more visible so maybe they can be fixed.
I believe genstrings puts the comment as I've done in my example .strings file, so that's probably as close as we'll get to a standard for comment placement.
Loving R.swift btw, thanks for making it!

@tomlokhorst Do you know if this is easy to do? Are the comments available to us when we parse the file?
Sounds like a nice feature!

Comments were part of the original design. But they fell off, because the default parsing of a plist file to a NSDictionary doesn't expose them.
I still like the feature, though!
To parse the comments we'd need to create a plist parser that exposes them. Or find a third-party parser that works, I haven't looked for one.

On 26 Jun 2016, at 16:57, Mathijs Kadijk<EMAIL_ADDRESS>wrote:
@tomlokhorst Do you know if this is easy to do? Are the comments available to us when we parse the file?
Sounds like a nice feature!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

I had no idea you could parse .strings as a plist using NSDictionary! I can't decide if that's really cool or slightly horrifying ;) But now that you mention it, if you tack curly braces around a .strings file you get an ASCII plist.
I'd assumed there was a lil parser for this already in R.swift. Using NSDictionary makes this a larger feature request than I thought it would be. Sorry about that.
Some random notes for anyone who takes this on (maybe me), or maybe for evaluating an existing parser to bring in:

The parser needs to consider escapes (e.g. \" doesn't mark the end of the key/value), which I've seen done badly by lots of code that ostensibly parses .strings files.
There's also \Uxxxx escapes to maybe consider. Surrogate pairs are accepted here by NSDictionary so watch out for those.
I may be wrong about where to put the comment; the Resource Programming Guide says the convention is to put the comment on the line before the string it's describing.

More notes to myself/whoever:
/*
/*
*/
"nested comments?" = "nope";

NSBundle.mainBundle().localizedStringForKey("nested comments?", value: nil, table: nil)

returns nope

To add to the list of possibilities:

No quotes are needed, in some cases: key = Value;
Duplicate keys are ignored (example)
Multiline keys and values are allowed (example)


Cleanup payload

@fkorotkov I believe I found/fixed the issue.
What I believe it was is that in the curl statement, there was a value for X-Author which I set to jumbocakeyumyum#0001 but I believe that bash saw the # and thought of it as a comment meaning a malformed payload error.  This may or may not be the fix 🙏.

Have you tested it on your repository with actions?

@fkorotkov Just did.  The status check no longer fails but I can't check discord as it is blocked here at school :(  I think that this is the fix however.


Cleanup supported versions.

Drop support for Python 3.7 due to end-of-life on 27 June 2023.
Run deployment actions on ubuntu-latest.
Make setup.py and tox.ini consistent with supported versions.

Codecov Report
Patch and project coverage have no change.

Comparison is base (a42ac84) 100.00% compared to head (6f8a8bf) 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff            @@
##              main       #77   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines          129       129
=========================================
  Hits           129       129

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Cutting and pasting items in an unordered list results in misplaced items

Issue Summary
When trying to cut and then paste an item from an unordered list to a different spot in the list, said item gets placed at the very bottom of the list.
https://github.com/TryGhost/Ghost/assets/27488257/18666d54-7c1f-4b67-8591-388e2f15ec97
Steps to Reproduce

Create several items in an unordered list
Cut one of those items
Go to another spot in the middle of the list and paste that item in
See it gets placed at the bottom of the list

Ghost Version
5.68
Node.js Version
Yes
How did you install Ghost?
Ghost(Pro)
Database type
MySQL 5.7
Browser & OS version
Firefox 118.0.2, macOS 13.6
Relevant log / error output
No response
Code of Conduct

[X] I agree to be friendly and polite to people in this repository

Thanks for the report. This is an upstream issue that we're aware of and tracking.

This seems to persist even after resolving the upstream issue. The behaviour upstream isn't perfect as it still injects an empty ListItem node when using Shift+Up selection to not just grab the text... but it at least doesn't dump it at the end of the list like Koenig does, so we've got something to look in to at this point.


Issue with Shaka Player on TizenTV - HLS Multi Audio Playback Problem

Have you read the Tutorials?
Certainly?
Have you read the FAQ and checked for duplicate open issues?
Yes
What version of Shaka Player are you using?
4.3.8
Please ask your question
Dear Community,
I am writing to seek assistance with a specific issue I am facing while using the Shaka Player on TizenTV (SamsungTV) with HLS video URLs. I have implemented the player to handle both single audio and multi audio streams, but unfortunately, I am encountering playback problems when using the multi audio HLS URLs.
The error message I am receiving is as follows:
Skipping unsupported HLS stream audio/aac audio/aac/tur/test.m3u8
Raw formats are not yet supported. Skipping audio/aac
"CHUNK_DEMUXER_ERROR_APPEND_FAILED: audio object type 0x40 does not match what is specified in the mimetype
For reference:

Shaka Player CDN URL: https://cdnjs.cloudflare.com/ajax/libs/shaka-player/4.3.8/shaka-player.compiled.js
Relevant code snippet:

var MediaPlayer = (function () {
  var video; // Declare the video variable to make it accessible throughout the module.

  var _loadUrl = function () {
    // Use the correct 'window.player.load' method to load the HLS stream.
    window.player.load('https://testencoding.s3.eu-central-1.amazonaws.com/output_videos/videos/2666d83b-2ccf-4a7f-a533-0104b10a24c3/hls/playlist.m3u8', null, 'application/x-mpegURL')
      .then(function () {
        video.play();
      })
      .catch(function (e) {
        console.log('--', e.code);
      });
  };

  var _initPlayer = function () {
    video = document.getElementById("video");
    var player = new shaka.Player(video);
    player.configure({
      streaming: {
        bufferingGoal: 180,
        rebufferingGoal: 5,
      },
    });
    window.player = player;
    // Configure Shaka Player with preferred audio language
    const config = {
      preferredAudioLanguage: 'es', // Set the preferred audio language code (e.g., 'en' for English)
    };

    // Load HLS content and start playback
    player.configure(config);
    _loadUrl(); // Move the _loadUrl() call inside the _initPlayer function.
  };

  var init = function () {
    shaka.polyfill.installAll();
    if (shaka.Player.isBrowserSupported()) {
      _initPlayer();
    }
  };

  return { init };
})();

document.addEventListener("DOMContentLoaded", MediaPlayer.init, false);

Video URLs:

Single Audio: https://testencoding.s3.eu-central-1.amazonaws.com/output_videos/videos/videos/7a8d35b8-fddb-4109-bd05-3be32bf3f1bb/hls/playlist.m3u8
Multi Audio: https://testencoding.s3.eu-central-1.amazonaws.com/output_videos/videos/2666d83b-2ccf-4a7f-a533-0104b10a24c3/hls/playlist.m3u8

The multi audio urls causing this issue, while the mpd multi audio urls are working fine.
I would highly appreciate it if someone could shed some light on why this problem is occurring and provide guidance on how to resolve it. If you need any additional information or clarifications from my end, please do not hesitate to ask. PFA
shakaPlayer_samplecode.zip
Thank you for your time and assistance in advance.

Can you test with the main branch? We fixed a lot of HLS issues in the main branch. Thanks!


feat: ability to specify CA certificates for LDAP

LDAP servers are often hosted on an organization's internal network. And access to the LDAP server is open only via the LDAP protocol over SSL (LDAPS).
In this case, there may be a problem of the LDAP client not trusting the server. The X.509 certificate trust chain will not be built.
You need to specify trusted CA certificates manually.
This PR offers a solution to this problem.

@ovsinc store cert in the "Certs" tab: https://door.casdoor.com/certs , see how applications and payment providers use it

Did I understand you correctly, you suggest to bind to the "cert" object instead of the CA certificate input field?
But as far as I can see, the "cert" object is intended for security settings: jwt, generation of the rsa key and a derived server certificate. This multitude of possibilities and fields can confuse the administrator.
The CA certificate is downloaded by the administrator, not generated. And most likely this CA certificate will be used only on the LDAP configuration form.
I can suggest and implement an alternative - add a new form "CA cert". It will implement the ability to download CA certificates.
@hsluoyz

@ovsinc

All different kinds of certs go to the "Certs" tab.
The cert content can be large, so will not be carried in a provider get or list API call.
The cert can be shared by multiple LDAP settings, so it's still useful for separate storage

@hsluoyz Thanks for the reply.
I agree with your arguments. But, it still seems to me that the "certs" form is overloaded with objects of different purposes.
Do you mind if I rework the "certs" form in the next PR? For example, the form can be divided into Tabs by purpose: server certificate and private key; jwt settings; CA certificates.
In this form, the use of the "certs" object looks convenient.

@ovsinc you can define a new scope or type in: https://door.casdoor.com/certs . Different options can also control which UI elements to be visible in the cert edit page

Excellent. I will take advantage of your suggestion: I will add a new scope and UI behavior on the "certs" form.
When it's ready, I'll write.
@hsluoyz

I have added CA certificate support in LDAP. Added a CACert certificate scope.

On the LDAP form, a certificate can only be used with a CACert scope.

I had to expand the scope of work a little.
Only a certificate with a JWT scope can be used on the Application form. Added a scope filter.

In addition, I added backend validation on the Certs form:

the certificate and the RSA private key are validated;
the CA certificate is validated.

@hsluoyz

I can't answer your comment in object/cert.go
I'll answer here.
You can switch the scope to JWT and then the certificate and key will be incompatible.
This can lead to errors that are difficult to detect.


svcat version returns the k8s version, not the Service Catalog API version

$ svcat version
Client Version: UNKNOWN
Server Version: v1.9.6

v1.9.6 is the version of k8s that I'm running, not the version of service catalog.
I've tried a few endpoints and can't figure out if our aggregated api even exposes a /version endpoint. So this fix may require more work on the server.
// example of how to manually hit any path you like
func (sdk *SDK) ServerVersion() (*version.Info, error) {
	body, err := sdk.ServiceCatalog().RESTClient().Get().AbsPath("/apis/servicecatalog.k8s.io/version").Do().Raw()
	if err != nil {
		return nil, err
	}
	var info version.Info
	err = json.Unmarshal(body, &info)
	if err != nil {
		return nil, fmt.Errorf("got '%s': %v", string(body), err)
	}
	return &info, nil
}

// example of how to retrieve a path under our aggregated api
func (sdk *SDK) ServerVersion() (*version.Info, error) {
	body, err := sdk.ServiceCatalog().RESTClient().Get().Name("version").Do().Raw()
	...
}

Here's the code in question:
https://github.com/kubernetes-incubator/service-catalog/blob/d297cedebd9b762598e569a797ad6b44b4e0ccb7/pkg/svcat/service-catalog/version.go#L26-L33

To my knowledge and looking I don't think we expose an endpoint with the catalog version.  I just tried adding one and it seems to work along with the RBAC changes...  Did you have thoughts on a good URL?  /version is taken/handled by the core apiserver framework we're using so I think we'll need something else.  /service-catalog-version or /svcat-version ?


[T3A5][W11-A2]Ong Heng Le

Ready for review.

Accepting without review since T3A3 and T3A4 were reviewed in detail. You are advised to cross check those comments with your code for this activity too. We also discussed the right way to do this exercise in the tutorial.


Synology page on docs - wrong influxes

User comment -
Followed the guide on unifipoller.com for Synology > needs a change to the doc.
If you use the :latest tag, you'll end up with a 2.0 version of Influx and that doesn't work.

Fixed


Error in Value

Run the script predict_temperatures.R and get the error:
[1] "2071 of 3221 loops"
Joining by: "agency_id"
Auto-disconnecting postgres connection (23487, 8)
Error in `[<-.data.frame`(`*tmp*`, is.na(df[, names(B[-1])]), value = c(867345.965986395,  :
  'value' is the wrong length

The error occurs in loops 2061 & 2071 when running in chunks of 50. The "value" in the error is identical in both loops. The featureid associated with the 2071 groups are:
 [1] "833626" "833627" "833628" "833629" "833630" "833631" "833632" "833633" "833634"
[10] "833635" "833636" "833637" "833638" "833639" "833640" "833641" "833642" "833643"
[19] "833644" "833645" "833646" "833647" "833648" "833649" "833650" "833651" "833652"
[28] "833653" "833654" "833655" "833656" "833657" "833658" "833659" "833660" "833661"
[37] "833662" "833663" "833664" "833665" "833666" "833667" "833668" "833669" "833670"
[46] "833671" "833672" "833673" "833674" "833675"

The problem is likely in the predict function or in one of the prediction helper functions. It's strange that it works for almost all loops and it did work for all loops when I was doing it in chunks of 100. My guess is that it has to do with all of the 50 featureid having NA for some value which causes a problem. The likelihood of all having NA is low and even lower with 100 featureid, which would explain the discrepancy.

It seems like the problem was in the prepPredictDF function. I added the dplyr::select() to the line within the autoregressive loops and it seems to have fixed the problem.
df[ , names(B[-1])][is.na(df[ , names(B[-1])])] <- colMeans(dplyr::select(B, B.ar1))


Error validating access token: Session has expired

Hi, I'm using this component for a few months but today the feed disappears of my page and I'm getting this access token error, it's seems that the token has expired, there is a method to allow the token to not expire?
{
"error": {
"message": "Error validating access token: Session has expired on Tuesday, 09-Nov-21 07:10:15 PST. The current time is Wednesday, 10-Nov-21 07:57:55 PST.",
"type": "OAuthException",
"code": 190,
"fbtrace_id": "A1k9iGqkQ8hyDAZqNTeGZLz"
}
}

When my token expires it throws and error and breaks the entire site.

Error validating access token: Session has expired

Error in function InstagramFeed in ./node_modules/react-ig-feed/dist/index.modern.js:69
Cannot read properties of undefined (reading 'slice')
  67 |     className: styles.instagramItems,
  68 |     ref: placeholder
> 69 |   }, showImage && data.slice(0, counter).map(function (item, index) {
     |   ^
  70 |     return /*#__PURE__*/React.createElement("div", {
  71 |       key: index,
  72 |       className: styles.instagramItem


Navtree - report function properties so that they can be identified as functions

https://github.com/Microsoft/vscode/issues/51988

TypeScript Version:  3.0.0-dev.20180616

Search Terms:

navtree
method, function, property

Code
For the code:
class K {
    p = 1
    f = () => {}
    m() {}
}

Run a navtree request
Expected behavior:
Some way to identify that f is a function
Actual behavior:
Both p and f are reported the same way, as properties.
Result: {
    "text": "<global>",
    "kind": "script",
    "kindModifiers": "",
    "spans": [
        {
            "start": {
                "line": 1,
                "offset": 1
            },
            "end": {
                "line": 5,
                "offset": 3
            }
        }
    ],
    "childItems": [
        {
            "text": "K",
            "kind": "class",
            "kindModifiers": "",
            "spans": [
                {
                    "start": {
                        "line": 1,
                        "offset": 1
                    },
                    "end": {
                        "line": 5,
                        "offset": 2
                    }
                }
            ],
            "nameSpan": {
                "start": {
                    "line": 1,
                    "offset": 7
                },
                "end": {
                    "line": 1,
                    "offset": 8
                }
            },
            "childItems": [
                {
                    "text": "f",
                    "kind": "property",
                    "kindModifiers": "",
                    "spans": [
                        {
                            "start": {
                                "line": 3,
                                "offset": 5
                            },
                            "end": {
                                "line": 3,
                                "offset": 17
                            }
                        }
                    ],
                    "nameSpan": {
                        "start": {
                            "line": 3,
                            "offset": 5
                        },
                        "end": {
                            "line": 3,
                            "offset": 6
                        }
                    }
                },
                {
                    "text": "m",
                    "kind": "method",
                    "kindModifiers": "",
                    "spans": [
                        {
                            "start": {
                                "line": 4,
                                "offset": 5
                            },
                            "end": {
                                "line": 4,
                                "offset": 11
                            }
                        }
                    ],
                    "nameSpan": {
                        "start": {
                            "line": 4,
                            "offset": 5
                        },
                        "end": {
                            "line": 4,
                            "offset": 6
                        }
                    }
                },
                {
                    "text": "p",
                    "kind": "property",
                    "kindModifiers": "",
                    "spans": [
                        {
                            "start": {
                                "line": 2,
                                "offset": 5
                            },
                            "end": {
                                "line": 2,
                                "offset": 10
                            }
                        }
                    ],
                    "nameSpan": {
                        "start": {
                            "line": 2,
                            "offset": 5
                        },
                        "end": {
                            "line": 2,
                            "offset": 6
                        }
                    }
                }
            ]
        }
    ]
}

Related Issues:

#14830

@andy-ms we did try this in the past. i can not remember what were the issues we ran into.. do you recall?

I don't remember trying this before. One caveat navTree doesn't use the type checker so we would have to specifically be looking for an arrow function node, rather than supporting any callable type. But if you prefer it, we could report f as a method instead of a property, or add a new ScriptElementKindModifier.callable.

i think that was the issue... sometimes we would report it as a functions and sometimes we would not..

I think we are better off keeping it consistent.

Sounds good. We can always revisit this if we start hearing about this more often


cupy.nanargmax() throwing exception

Description
I have a 2D array allocated on GPU and I need to use the cuPy's nanargmax() function to find the maximum value's index in each row. Some of the values could be NaN. Since the 2D array is quite large (here is in the sample, I have am showing a smaller one), I don't want to transfer it to the CPU and then use numpy's nanargmax()(which is working but transfer to CPU takes long time).
To Reproduce
# Write the code here
  import cupy as cp
  import numpy as np

  # Create a test array with a shape similar to (1250, 552)
  # test_array should have some NaN values and some non-NaN values

  test_array_gpu = cp.array(np.random.randn(1250, 552))  # Random values
  # Set some values to NaN for testing purposes
  test_array_gpu[200, 100] = cp.nan
  test_array_gpu[500, 200] = cp.nan
  test_array_gpu[1000, 300] = cp.nan

  # Apply cp.nanargmax along axis=1 (find the index of the max element in each row ignoring NaNs)
  best_fit_indices_gpu = cp.nanargmax(test_array_gpu, axis=1)

  # Print the result
  print("Best fit indices (indices of max values, ignoring NaNs):")
  print(best_fit_indices_gpu)

Installation
Conda-Forge (conda install ...)
Environment
# Paste the output here
OS                           : Linux-5.15.0-1029-nvidia-x86_64-with-glibc2.35
Python Version               : 3.10.13
CuPy Version                 : 13.0.0
CuPy Platform                : NVIDIA CUDA
NumPy Version                : 1.26.3
SciPy Version                : 1.12.0
Cython Build Version         : 0.29.37
Cython Runtime Version       : None
CUDA Root                    : /usr/local/cuda
nvcc PATH                    : /usr/local/cuda/bin/nvcc
CUDA Build Version           : 12000
CUDA Driver Version          : 12020
CUDA Runtime Version         : 12000 (linked to CuPy) / 12030 (locally installed)
cuBLAS Version               : 120304
cuFFT Version                : 11012
cuRAND Version               : 10304
cuSOLVER Version             : (11, 5, 4)
cuSPARSE Version             : 12200
NVRTC Version                : (12, 3)
Thrust Version               : 200001
CUB Build Version            : 200200
Jitify Build Version         : c08b8c6
cuDNN Build Version          : None
cuDNN Version                : None
NCCL Build Version           : None
NCCL Runtime Version         : None
cuTENSOR Version             : None
cuSPARSELt Build Version     : None
Device 0 Name                : Tesla V100-DGXS-32GB
Device 0 Compute Capability  : 70
Device 0 PCI Bus ID          : 0000:07:00.0
Device 1 Name                : Tesla V100-DGXS-32GB
Device 1 Compute Capability  : 70
Device 1 PCI Bus ID          : 0000:08:00.0
Device 2 Name                : Tesla V100-DGXS-32GB
Device 2 Compute Capability  : 70
Device 2 PCI Bus ID          : 0000:0E:00.0
Device 3 Name                : Tesla V100-DGXS-32GB
Device 3 Compute Capability  : 70
Device 3 PCI Bus ID          : 0000:0F:00.0

Additional Information
No response

Can you share the stacktrace of the exception?

This version is outdated, try the latest v13.3.0 release and please report back if the problem still occurs.
CuPy Version                 : 13.0.0


feat: add dependency management

Description
Adds capability to add dependencies for policy assignments and policy role assignments.
This provides a nice workaround to the limitation of not being able to pass unknown values into the alz provider.

Type of Change

[ ] Non-module change (e.g. CI/CD, documentation, etc.)
[x] Azure Verified Module updates:

[ ] Bugfix containing backwards compatible bug fixes

[ ] Someone has opened a bug report issue, and I have included "Closes #{bug_report_issue_number}" in the PR description.
[ ] The bug was found by the module author, and no one has opened an issue to report it yet.

[x] Feature update backwards compatible feature updates.
[ ] Breaking changes.
[ ] Update to documentation

Checklist

[ ] I'm sure there are no other open Pull Requests for the same update/change
[ ] My corresponding pipelines / checks run clean and green without any errors or warnings
[ ] I did run all  pre-commit checks

Hi @matt-FFFFFF ,
I ❤ this. The idea to use terraform_data  and sha256 as some kind of "intermediate" to have a chain depends_on can rely on - awesome!
I am sure this will, together with the better retry logic of AzAPI v2, help a lot!

Thanks @c-baumgartner!
terraform_data might be my new favourite resource 😂


CLD-338-rendering-PGW

@asad-rafter I'm afraid that this is not in the right direction. This uses onload, but also dynamically injects the script ensuring it will be loaded much later than needed. This slows down the loading of the PGW which causes increases in the LCP.
window.cldProductGallery is not assigned in
https://github.com/cloudinary/cloudinary_sfcc_site_cartridge/blob/018a043514adde3fba5f22362e03b741e89c975c/cartridges/cloudinary_sfra_changes/cartridge/templates/default/product/productDetails.isml#L12
Could this be the issue?

Yes, this is the issue. So the problem is, sometimes in milliseconds difference  loads late and the cloudinary object from other scripts assigned to window.cldProductGallery, in that case it breaks the PGW because the script is not available in DOM. Let me know if you have any better solution for this

@asad-rafter I'm afraid that this is not in the right direction. This uses onload, but also dynamically injects the script ensuring it will be loaded much later than needed. This slows down the loading of the PGW which causes increases in the LCP.
window.cldProductGallery is not assigned in
https://github.com/cloudinary/cloudinary_sfcc_site_cartridge/blob/018a043514adde3fba5f22362e03b741e89c975c/cartridges/cloudinary_sfra_changes/cartridge/templates/default/product/productDetails.isml#L12
Could this be the issue?

Yes, this is the issue. So the problem is, sometimes in milliseconds difference  loads late and the cloudinary object from other scripts assigned to window.cldProductGallery, in that case it breaks the PGW because the script is not available in DOM. Let me know if you have any better solution for this

@asad-rafter The new code adds loading time, and still allows such a condition. Please remove the onload changes and assign window.cldProductGallery after
https://github.com/cloudinary/cloudinary_sfcc_site_cartridge/blob/018a043514adde3fba5f22362e03b741e89c975c/cartridges/cloudinary_sfra_changes/cartridge/templates/default/product/productDetails.isml#L12
exactly like it is done in all other places you call the widget

PGW is only called on PDP, where we are using the onload


[TW#28163] Posting an event with data alloctaes a copy of the data but does not free it

Environment

Development Kit:      ESP32-PICO-Kit
Module or chip used:  ESP32-PICO-D4
IDF version: v3.3-beta1-134-g1023ff73f
Build System:  Make
Compiler version: 1.22.0-80-g6c4433a5
Operating System:     Windows
Power Supply:         USB

Problem Description
When posting an even uisng esp_event_post() or esp_event_post_to() with data (i.e. parameter event_data != NULL and event_data_size > 0), a copy of the event data is allocated by post_instance_create() and attached to the post which is then sent to the event queue.
That copy of the event data is never freed, except when sending to the event queue has failed.
I would expect that the event loop (i.e. esp_event_loop_run()) would free that copy of the event data after passing it to all registered event handlers, i.e at the end of the while loop. But this does not seem to happen. Instead the copy remains allocted, causing a memory leak if many such events are posted.
Expected Behavior
Event data to be freed in event loop (i.e. esp_event_loop_run()) after passing it to all registered event handlers, i.e at the end of the while loop.
Actual Behavior
Event data is not freed, leading to a memory leak.
Steps to repropduce
Post an event using esp_event_post() or esp_event_post_to() with data.
E.g. like it is illustrated in example 'system/esp_event/default_event_loop':
https://github.com/espressif/esp-idf/tree/master/examples/system/esp_event/default_event_loop
Code to reproduce this issue
See example 'system/esp_event/default_event_loop':
https://github.com/espressif/esp-idf/tree/master/examples/system/esp_event/default_event_loop
Debug Logs
None.
Other items if possible
None.

I also wonder why function post_instance_create() defines event_data_copy as void **, while the event data itself is just void*. Also in struct esp_event_post_instance it is defined as void **, although it should just be void * in my opinion.
Function esp_event_loop_run() then passed post.data to the handler, and esp_event_handler_t is defined with void* event_data. So post.data should really be void *, not void **. However, this does not really hurt.

The minimal fix for this issue should be in functioon esp_event_loop_run():
    while(xQueueReceive(loop->queue, &post, ticks_to_run) == pdTRUE) {
        esp_event_base_instance_t* base = NULL;
        esp_event_id_instance_t* event = NULL;
        ...
        bool exec = false;

        for (int i = LOOP_LEVEL_HANDLER; i <= EVENT_LEVEL_HANDLER; i++) {
            if (handlers_list[i] != NULL) {
                esp_event_handler_instance_t* it;
                ...
            }
            exec |= true;
        }

        free(post.data);    /* <---- add this line */

        if (ticks_to_run != portMAX_DELAY) {
           ...

Besides that post.data should be changed to just void * as well as event_data_copy in post_instance_create().


Make srv_async_unix public

The problem with srv_async is that it creates constraints on the naming of the unix socket, specifically, that the name must end in :<port>.
This is not compatible with my use case (wiring up to a firecracker-vm vsocket, which has its own naming convention for sockets).
This is a trivial PR to simply expose the srv_async_unix method so I can provide my own address.

Merged. Thanks.


To test STO where owners have reserved MSC

Here's a way to test STO with owner(s) who have MSC that are reserved, e.g. because they have a tx20 sale (Dex phase 1) that's active.
Assign the same number of MSC, e.g. 100.0, to 3 addresses. Then:

Address 1 issues a tx20 for all 100.0 MSC, so his available balance is 0.0, reserved balance is 100.0
Address 2 issues a tx20 for 10.0 MSC, so his available balance is 90.0, reserved balance is 10.0
Address 3 doesn't do anything with his 100.0 MSC, so his available balance is 100.0, reserved balance is 0.0

The actor address acquires a number of MSC, e.g. 100.0 or more. Then the actor address submits an STO for all but 1.0 of his MSC. This will leave enough MSC available for the transfer fee for 100,000,000 recipient owners.
After the STO, check the balances of the 3 addresses. The (available balance + reserved balance) for all three should be exactly equal, with the available balance for each of them increased by the same amount.

Hmm.. even if there are several rows in the test plan with "TDiv1" etc., for each test new addresses and properties are created, intended to reduce side effects and to have self sustaining tests. This works very well for tests involving properties, but unfortunally we can't simply reset the whole TMSC/MSC state on demand.
This is a fundamentally different approach than I used with mastercore-rpc-tests where each test (or rather a test sequence) spawns at least one new, pristine node.
Long story short: I think it's given that we have no control over the MSC/TMSC state and therefore can't predict or command how many owners there are.

@dexX7 Sorry, I wasn't clear. For this test it doesn't matter how many other MSC owners there are or how many MSC they own. I'm just interested in seeing that the 3 newly created addresses all end up with exactly the same number of MSC after the STO - because before the STO those 3 addresses all have the same (available balance + reserved balance) total.
The 3 would end up with different amounts only if the number of MSC to be distributed runs out after 1 or 2 of the 3 get their distribution, which seems highly unlikely.

@marv-engine: my mistake actually, I focused on the inability to control MSC holders and dismissed it without grasping the actual test. :)
I'll probably push it later.

@marv-engine: is this one fine? https://github.com/dexX7/bitcoin-spock/commit/23e62e594d6ac75231b129ce7445b1cf7a09d503
As precaution: the test is skipped, if there are more owners than the actor can pay in fees.
A note regarding the test plans (CC: @msgilligan): while sto-testplan.tsv is completely parsed and used to generate tests, sto-testplan-special.tsv is not processed automatically, but hard coded, because the test class can't handle the unusual situations or constraints.
Tests in MSCSendToOwnersTestPlanSpec.groovy use executeSendToOwners which is overwritten in MSCSendToOwnersTestPlanRawSpec.groovy and each test (including the special ones) run two times: once by using the RPC call sendtoowners_MP and a second time by constructing a raw transaction (based on unobfuscated Omni data) via sendrawtx_MP to bypass any RPC checks, e.g. "property does not exist", to test the interpretation layer of Master/Omni Core.

@dexX7 - yes, your code for this test looks good to me.
On Sunday, February 1, 2015, dexX7<EMAIL_ADDRESS>wrote:

@marv-engine https://github.com/marv-engine: is this one fine? dexX7@
23e62e5
https://github.com/dexX7/bitcoin-spock/commit/23e62e594d6ac75231b129ce7445b1cf7a09d503
As precaution: the test is skipped, if there are more owners than the
actor can pay in fees.
A note regarding the test plans (CC: @msgilligan
https://github.com/msgilligan): while sto-testplan.tsv
https://github.com/msgilligan/bitcoin-spock/blob/master/src/integ/groovy/foundation/omni/test/rpc/sto/sto-testplan.tsv
is completely parsed and used to generate tests, sto-testplan-special.tsv
https://github.com/msgilligan/bitcoin-spock/blob/master/src/integ/groovy/foundation/omni/test/rpc/sto/sto-testplan-special.tsv
is not processed automatically, but hard coded, because the test class
can't handle the unusual situations or constraints.
Tests in MSCSendToOwnersTestPlanSpec.groovy
https://github.com/msgilligan/bitcoin-spock/blob/master/src/integ/groovy/foundation/omni/test/rpc/sto/MSCSendToOwnersTestPlanSpec.groovy
use executeSendToOwners
https://github.com/msgilligan/bitcoin-spock/blob/master/src/integ/groovy/foundation/omni/test/rpc/sto/MSCSendToOwnersTestPlanSpec.groovy#L382-L398
which is overwritten in MSCSendToOwnersTestPlanRawSpec.groovy
https://github.com/msgilligan/bitcoin-spock/blob/master/src/integ/groovy/foundation/omni/test/rpc/sto/MSCSendToOwnersTestPlanRawSpec.groovy
and each test (including the special ones) run two times: once by using the
RPC call sendtoowners_MP and a second time by constructing a raw
transaction (based on unobfuscated Omni data) via sendrawtx_MP to bypass
any RPC checks, e.g. "property does not exist", to test the interpretation
layer of Master/Omni Core.
—
Reply to this email directly or view it on GitHub
https://github.com/msgilligan/bitcoin-spock/issues/38#issuecomment-72363506
.

--
Marv Schneider
<EMAIL_ADDRESS>

@msgilligan: can be closed, I think.


how to install

add readme

I'm so sorry, but this theme  is still under development 😅
Currently, some refactoring  and documentation of components including README is in progress.
I try to release it at the end of March at the latest.


⚠ Bofrost.at has degraded performance

In 2fdb0c2, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 814 ms

Resolved: Bofrost.at performance has improved in 32ad90f.


Error when install bleno via npm

Hi, @sandeepmistry
There is an error occurred when install this package via npm on my Intel Edison with Ostro OS, the log message show as below:
root@edison:/tmp# npm install bleno
npm WARN optional dep failed, continuing<EMAIL_ADDRESS>-
><EMAIL_ADDRESS>install /tmp/node_modules/bleno/node_modules/bluetooth-hci-socket/node_modules/usb
> node-pre-gyp install --fallback-to-build

node-pre-gyp ERR! Tried to download: https://github.com/tessel/node-usb/releases/download/1.2.0/usb_bindings-v1.2.0-node-v46-linux-ia32.tar.gz
node-pre-gyp ERR! Pre-built binaries not found for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v46 ABI) (falling back to source compile with node-gyp)
...

The usb_bindings-v1.2.0-node-v46-linux-ia32.tar.gz can not found in github, but the bleno can be required and the all tests are pass after install this package. Is there anything wrong?

Hi @wanghongjuan,
This should be fine, please let me know if one of the built-in bleno examples don't function.
 xpc-connection is only needed on OS X and usb is only needed on Windows (optional for Linux).


Langraph Implementation: Analysing and fixing the consumer flow

Update 4 DEC -

Flow was fixed upto search.  The issue was with every break in the langraph loop, it was always returning to one single node when called again. For now adding conditions to that one loop only

Problem Statement

Numerous interrupts were added in the flow for human feedback, referred to as "human in the loop" in Langraph terminology.
After each interrupt, regardless of the last step, the flow always returned to the same node, causing continuity issues.

Update

Integrated all Beckn API calls (select, init, and confirm) using Langraph.
Added conditional logic to the single node to redirect to different nodes based on the previous node.
Separated node handlers and conditional edges implementations into separate files for better organization.

Next Steps

Refactor to follow Langraph best practices.

Analyze why the flow was returning to the same node after interruptions and fix it.
Ideally, the resume node should be handled by Langraph itself. Further debugging is needed.

Branch with updated code: feature/langgraph-updated


Apps reCAPTCHA gives error.

this app does not work
it always says try again
the sequel doesn't

Can you reproduce this issue on our community?

data66
2 hours ago
#4
Please post issues at github where the issue tracker is. Thanks! https://github.com/moxi9/reCAPTCHA

bu sorunu çözmek için ne yapmalıyım ?

what should I do to solve this problem ?

Its though to debug the issue as we cannot reproduce this on our live community. Have you tried to enable debug mode to see if any errors return?


Bad Responsiveness

The website is having bad responsive to smaller screen devices, and needs improvement.

@Vin205 Kindly assign it to me. I'll make it responsive.

@SatyaJaiss assigned!

@SatyaJaiss your changes are accepted!


Question: Frontend Project!

Hey @nectarboy !
I came across this project and it looks really cool, emulation works perfect and integration aspect seems really cool. I know nothing about html, css, or JavaScript lol, and to learn, I want create a simple front end (a cross platform front end that could also work on mobile :) ).  I know that this project is under the MIT license which is pretty permissive, but I still wanted to ask if it's cool with you if I can use pollenboy as a backend core?
Thanks!

Hi, thanks !
Course you can :D as long as credit goes where its due no problem
I always appreciate my projects being useful to others so feel free


add sentence length distribution plot

Resolves #89. Implements sentence length distribution plot. Fixes to documentation accordingly.
@meghdadFar Thank you for your previous reply. Unfortunately, it is not possible to push from my local branch as I do not have permission. Changes to the fork can be seen by checking out the feature/sentence-length branch. Also, thank you for calling attention to the correct documentation directory. Changes in this new pull request were made to the sphinx-docs folder.

@jgregoriods It seems the merging does not automatically add you to the collaborators (so that you can push your branches), hence I invited you. :)


deps(saml): update SAML version

Thanks for submitting a PR! Please check the boxes below:

[x] I have run pre-commit to check linting
[ ] I have added information to docs/ if required so people know about the feature!
[x] I have filled in the "Changes" section below?
[x] I have filled in the "How did you test this code" section below?
[x] I have used a Conventional Commit title for this Pull Request

Changes
Update SAML version in enterprise / SaaS deployments. The main reason for this is update / add the endpoints for managing SAML configuration and mappings to support the FE work as part of #4055 . For full information on what has changed see release notes in SAML repository here.
How did you test this code?
N/a

LGTM


[FR] Allow to bypass Route53 DNS record.

Related to #79.
Add a flag that will disable automatic Route53 record creation, so it is possible to use other DNS servers, or fallback to just adding the custom domain name to Cloudfront.

You may choose to not pass any domains. In such case, it will result in CloudFront's domain to be used.
Afterwards, you can manually create whatever DNS records you need that will target Cloudfront.

Considering Cloudfront is target alias, I really don't think you can simply plug-n-play different DNS server. You will need to create hosted zone on AWS and setup NS for it.
Aka.

I have server example.com on GoDaddy,
I want to use app.example.com on AWS.
So I need to create new hosted zone app.example.com and on GoDaddy's end, I need to set NS records to use AWS's resolved.

This results in you always having hosted zone on AWS as you cannot point other DNS server to Cloudfront's IP (as Cloudfront does not have IP).
I hope I understood the request correctly, let me know.

The use case would be, for example having example.com on Cloudflare for example, and lets say that I want to deploy the nextjs site to app.example.com, not to example.com, i could set up a custom domain name on Cloudfront and also set a cname record at Cloudflare that is pointed at the cloudfront dristribution url (for example d111111abcdef8.cloudfront.net), so I would not need R53 records.
The whole thing would llok something like this:

Cloudfront: Distribution Url: d111111abcdef8.cloudfront.net, Alternate Domain: app.example.com
Cloudflare: app.example.com --(CNAME)--> d111111abcdef8.cloudfront.net

In such case, we should be good as we are now. I wouldn't recommend using CNAMEs personally, but feel free to not pass any domain names to CLI command and you will end-up with Cloudfront's url.

This does not seem like very reasonable pattern to be honest. I wouldn't recommend using CNAME for Cloudfront, seems like you are just going around setting up DNS properly.
I cannot support this out of box as manual steps are required (setting DNS records on Cloudflare to validate certificate). Considering the focus on AWS and your scenario not being 100% automated, I'm gonna close this.
Feel free to create stack without domain names and manually add them after creation via AWS Console.


vendor in istio types

Required for #339 (breaking up into smaller commits)
The only change made here is:
dep ensure -add knative.dev/pkg/apis/istio/v1alpha3

/assign @yuzisun

This change is

Just confirming you did this with dep ensure, right?

yas:
dep ensure -add knative.dev/pkg/apis/istio/v1alpha3

/retest

/retest

@yuzisun @ellis-bigelow can approve?

Looks good!
/lgtm
/approve


[HttpPlatformHandler]: Forward common headers from reverse proxy

This bug is to track the work that needs to happen so that items forwarded from httpplatformhandler show up in the user's application.
There are several parts to this:

Windows Auth (coming soon)
Client certs (not yet available)
Https scheme

We could do some of it in middleware (like WindowsAuth) but that would mean it wouldn't work out of the box. It could also be in the template by default so that we don't couple to the HttpPlatformHandler in any of our core components.

Also of interest are the remote ip & port and the reverse proxy public IP & port. I'd recommend doing it all in middleware.
Eventually we'll need the TLS token binding.

I think scheme is the only one we'd want to support by default.

HttpPlatformHandler needs to forward them all, it's up to the app how many forwarders they check.

Well let the user experience drive that. This is our new ootb experience for IIS and it can't be degraded in obvious ways. We'll discuss it in the design meeting

A few other interop questions:
100-continue support
http/2 termination (should just work, it happens in http.sys)
client disconnects

We decided to make this a separate middleware UseHttpPlatformHandler().

What about nested applications/child paths? Is that still something that will be supported out of the box? Or will we need to configure the application's base path (a.k.a. ~/) manually?

Won't be supported out of the box. I haven't tried it myself with the platform handler.

@Eilon @muratg Let's create a new repo for this: IIS or IISIntegration. I think it will just replace Helios in the build hierarchy.

@davidfowl  repo name?

@Tratcher move this bug over

This issue was moved to aspnet/IISIntegration#2


Fix the race condition with the GC in CachingSpringLoadBalancerFactory

Fix for #2443

@jebeaudet Feign has been moved to its own project can you please submit a PR there https://github.com/spring-cloud/spring-cloud-openfeign


[BUILD] Fix lint-python.

What changes were proposed in this pull request?
This pr fixes lint-python.
./python/pyspark/accumulators.py:231:9: E306 expected 1 blank line before a nested definition, found 0
./python/pyspark/accumulators.py:257:101: E501 line too long (107 > 100 characters)
./python/pyspark/accumulators.py:264:1: E302 expected 2 blank lines, found 1
./python/pyspark/accumulators.py:281:1: E302 expected 2 blank lines, found 1

How was this patch tested?
Executed lint-python manually.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/1682/
Test PASSed.

Merged build finished. Test PASSed.

Test build #94033 has started for PR 21973 at commit e405a9a.

LGTM

Test build #94033 has finished for PR 21973 at commit e405a9a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94033/
Test PASSed.

Thanks! merging to master.


Editor issues

This is a list with issues I encountered with the new editor:

[x] Auto-complete does not use "intelligent filtering" like the old one did (Filtering is strictly from left to right according to what you type (lower-cased))
[x] Auto-complete does not close itself when pressing "Esc"
[ ] Setting editor-insert-marker is not working
[ ] No syntax highlighting for the settings-file

Setting editor-insert-marker is not working

It definitely worked when I implemented it ;), but there were some issues with escaping that was necessary in the INI-file. Have you tried with a simple string only containing normal characters?

No syntax highlighting for the settings-file

I've intentionally removed the code for this, as it's overkill for a format that should go away anyway.
Off-topic: I've been busy and travelling a lot the past month (and will be still until mid-September probably) which is why I have been quite inactive. I might have time to do some changes and fixes sporadically.

I've intentionally removed the code for this, as it's overkill for a format that should go away anyway.

Oh, and totally true. Will remove this todo.

Off-topic: I've been busy and travelling a lot the past month (and will be still until mid-September probably) which is why I have been quite inactive. I might have time to do some changes and fixes sporadically.

No worries and save travels! The only "big" thing would be releasing a new version, as a lot has changed in the meantime (design, filters, editor, ...). Should we keep that until the end of September?


ErrorCode 17999

Description
iOS users randomly receive ErrorCode 17999
 Error Domain=FIRAuthErrorDomain Code=17999 \"An internal error has occurred, print and inspect the error details for more information.\" UserInfo={FIRAuthErrorUserInfoNameKey=ERROR_INTERNAL_ERROR, NSLocalizedDescription=An internal error has occurred, print and inspect the error details for more information., NSUnderlyingError=0x302e96790 {Error Domain=FIRAuthInternalErrorDomain Code=3 \"(null)\" UserInfo={NSUnderlyingError=0x302c5a8b0 {Error Domain=com.google.HTTPStatus Code=401 \"(null)\" UserInfo={data={length = 284, bytes = 0x7b0a20 ... }, data_content_type=application/json; charset=UTF-8}}, FIRAuthErrorUserInfoDeserializedResponseKey={\n    code = 401;\n    errors =     (\n                {\n            domain = global;\n            message = \"Firebase App Check token is invalid.\";\n            reason = unauthorized;\n        }\n    );\n    message = \"Firebase App Check token is invalid.\";\n    status = UNAUTHENTICATED;\n}}}}"

Could be a regression of https://github.com/firebase/firebase-ios-sdk/issues/11926 which was fixed. I am unable to replicate this locally but many users are saying they are encountering this with some saying a re-install isn't resolving it.
I feel that the AppCheck token isn't being re-generated when a 17999 error is received.
Reproducing the issue
Unable to repro locally, but users just attempt to login with username / password using signInWithEmailPassword with AppCheck enabled.

Hi @jostster, which App Check provider(s) are being used in the project?

Hi @jostster, which App Check provider(s) are being used in the project?

We are using AppAttestProvider(app: app)

Looks like more users are seeing this according to our support team. Users have stated they haven't changed devices but maybe they have restored from backup or for some reason the system is holding on to an old token?

Users have stated they haven't changed devices but maybe they have restored from backup or for some reason the system is holding on to an old token?

In the case where the backup is creating the invalid state, the proposed fix for https://github.com/firebase/firebase-ios-sdk/issues/12629 may address this.

Users have stated they haven't changed devices but maybe they have restored from backup or for some reason the system is holding on to an old token?

In the case where the backup is creating the invalid state, the proposed fix for #12629 may address this.

That is a possibility but I didn't see anything from that issue about 17999 error code. We have also asked users with this issue to delete and reinstall the application which they say isn't resolving their issue. I have added back code into our next release that will reset the attest key when a 17999 occurs. We will see if that resolves the issue.

@jostster, from the reports you've seen, does this seem like an issue that won't go away once it starts happening?

@ncooke3 it seems hit or miss. Some are saying that reinstalling the app does not resolve the issue while others that specifically stated, they bought a new phone and restored from cloud said that deleting and reinstalling did fix the issue.

Got it. Thanks, @jostster. I'll release the fix for https://github.com/firebase/firebase-ios-sdk/issues/12629 early next week and we can then see if it addresses this issue. I have a feeling they are somewhat related.

Hi @jostster, are you still facing this issue with the AppCheckCore dependency bumped to 10.19.1? https://github.com/firebase/firebase-ios-sdk/issues/12629#issuecomment-2110215864

@ncooke3 I haven't had a chance to implement it as it wasn't available in the latest release version at the time. I am on PTO but will add it into our June release

I also getting this error, I didn't change any code, and it worked in the morning.
[log] e.message Error Domain=FIRAuthErrorDomain Code=17999 "An internal error has occurred, print and inspect the error details for more information." UserInfo={FIRAuthErrorUserInfoNameKey=ERROR_INTERNAL_ERROR, NSLocalizedDescription=An internal error has occurred, print and inspect the error details for more information., NSUnderlyingError=0x3029a9fe0 {Error Domain=FIRAuthInternalErrorDomain Code=3 "(null)" UserInfo={FIRAuthErrorUserInfoDataKey={length = 1628, bytes = 0x3c21444f 43545950 45206874 6d6c3e0a ... 2e3c2f69 6e733e0a }, NSUnderlyingError=0x3029a2610 {Error Domain=com.google.HTTPStatus Code=403 "(null)" UserInfo={data={length = 1628, bytes = 0x3c21444f 43545950 45206874 6d6c3e0a ... 2e3c2f69 6e733e0a }, data_content_type=text/html; charset=UTF-8}}}}}

     String? token =
          await FirebaseAuth.instance.currentUser!.getIdToken(force);

@jostster

UPDATE
This error comes when device connect to VPN. If VPN is off its working. What is the issue?

Hi everyone, I have these token failures to Apple (FB13812838) and am still awaiting a response. I will follow-up and report back as I learn more.

For confirmation we had one user that updated their iOS version and was locked out because of the AppCheck. We asked them to uninstall our app, then reinstall it from the AppStore and they got back to us saying that reinstalling allowed them to get back in.

@ncooke3 is it possible for firebase to request a new token on these failures? With iOS 18 out now, and as users are updating, they are contacting our support with this error that the AppCheck is giving them a 17,999 error. Really the only thing we can try is having them delete the application and reinstall it from the app store and hope that fixes it for them.

Yes @jostster, I will look into this ahead of the next release. I suspect this is due to the SDK storing an app attest artifact in user defaults, which is included in iCloud backups. Since the artifact is device specific, I suspect this may explain this failure path that upgrading customers are experiencing.

Thanks @ncooke3 . I'm curious, is there a different key as I created this little hack as a work around which did work before but with this still in our application and users still encountering this error, I was curious if there is another place I should be resetting the token.
#import "FirebaseAppCheckProviderOverride.h"
@import Firebase;
@import FirebaseAppCheck;

@implementation FirebaseAppCheckProviderOverride

- (FBLPromise<NSNull *> *)resetAttestation {
    FIRAppAttestProvider* provider = [[FIRAppAttestProvider alloc] initWithApp:[FIRApp defaultApp]];
    return [provider performSelector:@selector(resetAttestation)];
}

@end

Hi @jostster, I mixed up my thoughts as the retry logic to wipe out the key ID was already implemented last year to address https://github.com/firebase/firebase-ios-sdk/issues/11926.
Since reinstalling the app seems to fix the issue, I'm trying to tease out what persistent state is the culprit.
Two additional questions to clarify some details.

Are you using any other providers beside app attest?
Are you using an app group to share data across multiples apps you develop?

We do use the debug app attest provider, but that is only for dev builds. For production where we are seeing this issue we only use the default AppAttestProvider(app:). We also do not use App Groups or have that capability enabled.

@ncooke3 Would recaptcha be an alternative for AppCheck? Our use for AppCheck is to prevent bad actors from hitting the google api directly and bruteforce logins.

Hi @jostster, reCAPTCHA could be a solution to consider. It is documented to use Apple's App Attest, so if App Attest is the underlying issue here, then similar failures could occur.
When the Auth SDK makes a request (e.g. password login), an app check token is requested to include in the request. Tokens can be requested with and without forcing a refresh of the token. In Auth's case, app check token's are not force refreshed, so whatever token is cached will be included in the request. This is done to be as performant as possible.
In this case though, I think it could make sense to retry the sign in request with a fresh app check token to see if that fixes the issue or at least provides some more clues as to what is going wrong.
Here's an example of what this could look like:
      Task {
        do {
          try await Auth.auth().signIn(withEmail: email, password: password)
        } catch let error as NSError {
          // Catch the App Check related error, force refresh the stored App
          // Check token, and make retry signing in.
          guard error.code == AuthErrorCode.internalError.rawValue else {
            throw error // Error is unrelated (e.g. account does not exist).
          }

          // This call will fetch a new app check token and reset the cached
          // attestation artifacts if needed.
          _ = try await AppCheck.appCheck().token(forcingRefresh: true)

          // Retry the sign in. This should pick up the new, refreshed token.
          try await Auth.auth().signIn(withEmail: email, password: password)
        }
      }

Thanks @ncooke3 I was looking at the app-check repo earlier and noticed this open issue which relates to the token(forcingRefresh: true) returning the cached token if a token(forcingRefresh: false) is in progress. I'm wondering if this might be something causing this issue?

I don't think so because the Auth SDK uses token(forcingRefresh: false) so a cached token will be returned regardless of whether a concurrent token(forcingRefresh: true) is happening.

@jostster, if you would like, I can take a closer look into the logs to better understand which part of the App Attest handshake is failing. I just need a project number. Feel free to send it to nickcooke+gh at google dot com

@jostster, if you would like, I can take a closer look into the logs to better understand which part of the App Attest handshake is failing. I just need a project number. Feel free to send it to nickcooke+gh at google dot com

Sent, ty!

Hi @jostster, after a closer look, I think this may be a #12468 surfacing when the the Auth SDK requests the App Check token. If the App Check token cannot be successfully retrieved, then the App Check SDK will give a place holder token to include that will result in the error message given. Today I released a fix similar to what was done in https://github.com/firebase/firebase-ios-sdk/issues/11926. Please take a look at my comment in the corresponding issue for updating instructions: https://github.com/firebase/firebase-ios-sdk/issues/12468#issuecomment-2398154247
I'm going to mark this issue as closed, but we can re-open if the persists.


Omit final self from use statements.

Thank you for the PR, however, this normalization is not a safe one to do during formatting, see https://github.com/rust-lang/rust/issues/60941 and #3568 for more information.
I suspect this should be closed, although the formatting change in the test file with the aliased super::self is an interesting case. I'd have to dig back through things to see whether or not that's a separate bug or intentional

Thank you for the PR, however, this normalization is not a safe one to do during formatting, see https://github.com/rust-lang/rust/issues/60941 and #3568 for more information.
I suspect this should be closed, although the formatting change in the test file with the aliased super::self is an interesting case. I'd have to dig back through things to see whether or not that's a separate bug or intentional

I see; thanks for the links. I'll close this for now.

I see; thanks for the links. I'll close this for now.


Integrate Github with Slack for v4-dev

Anyone doing this needs access to the GitHub repo AND be an admin on Slack. Just hit this URL and click-click https://getbootstrap.slack.com/services/new/github.

GitHub/Slack is pretty noisy

is that a bad thing @thomas-mcdonald? Depending on the person or organization that isn't a bad thing.

FWIW, we can explore adding the integration to a separate room. I don't think we'd inundate #general with all that noise given the number of folks there.

@kkirsche we had it enabled in our internal room for a while - the activity on this project means it will probably drown out discussion

@thomas-mcdonald I wasn't in those so I can't say. I know the alpha brings up a ton of activity but before it wasn't enough In my opinion to be a problem. I don't know what you consider drowning out though

@thomas-mcdonald @mdo a #git channel has served my team well. Even with dozens upon dozens of commits to master every day, it's actually surprisingly quiet in my experience - typically announcing groups of commits and actions periodically rather than each event individually.

@cassler sounds cool - I hadn't thought about putting it in another room

Added to the #git room.


ClockFace and circle is placed in wrong position

I have replaced and fixed all issues such as PropTypes and remove react-native-svg by using expo. However, the clock face and circle is placed in wrong position.
Can anyone help?

Compare with #22, the solution is there!

@HighSoftWare96  yeah, I also fixed this issue.


Echo between android mobile and web call

We have implemented agora call between react-native mobile and web side.
While call there is echo on the web app. But it occurs only when the call is made between web and Android app. There is no such problem when call is made from the iOS app.
Echo occurs when the web user speaks slowly or after a pause. When the user speaks continuously there is no echo.
We went through Self-check list provided by Agora (https://docs.agora.io/en/Video/faq/echo) but no luck so far.
Platform: Android
"react-native": "0.66.3"
"react-native-agora": "^3.5.1"
Thanks in advance

I've got the same issue.
To find the cause for this issue, I've done some further testing. Specifically I've tested the following Web-SDKs with the react-native-sdk version above:

"agora-rtc-sdk": "^3.6.7"
"agora-rtc-sdk-ng": "^4.7.3"

The issue appeared for both versions.
Strangely enough the issue only occurs in calls between web and react native apps.
If there are only web clients in the call or only react-native clients in the call, no such issue appears.


TopDown Engine

SteamDB app page links to a few games using this
https://steamdb.info/app/2241880/
https://steamdb.info/app/1734900/
https://steamdb.info/app/2090760/
https://steamdb.info/app/1272290/
https://steamdb.info/app/1586740/
Brief explanation of the change
Add TopDown Engine.

We don't want to detect unity addons


Ignore raw and unicode

closes #14

Closing this as more changes are needed and it is a bit difficult for me to fix this PR. Thank you @esauvisky .


Added an option to send scaleform message

You can change the title of the scaleforms in txAdminClient/txa_client.lua at line 2 (default "Staff announce")
You can change the time of the message in txAdminClient/txa_server.lua at line 71 (default 3sec)
Hope you like it,
I love your job, awesome release
preview:

That's useful.
Will have to take a look after the next update.

IT doesnt work

@rex2630
Its not up to date, i have to edit this.

Ok,please do that,because i really like it :)

Will u edit this anytime soon, please?


Enable HTTPS for Content Routing Type = HTTP

Would like to enable HTTPS for delivery services using HTTP 302 redirects, as is currently possible when using DNS as the Content Routing type.

Is this about using HTTP to Request Router and then return an HTTPS location or an end-to-end HTTPS? What I would require is simply a Location: https://blawhile using HTTP to TR.

this is about TR listening on 443 w HTTPS.
you want HTTP routing and HTTPS caching?

@knutsel Yes. Should I open a separate issue?

@smalenfant yes, please create a new issue labeled with enhancement and Traffic Router.


Outputting context vectors

I'd like to get the context vectors out too (not just word vectors). Is this possible with current implementation?
The idea is to try to decouple words and contexts completely, ala Levy&Goldberg's "dependency based embeddings", to experiment with functional similarities.

How can I extract the vectors (for individual words)?
I.E, given a new text corpus, I want to transform each individual word (given its context) into the vector space representation, for later use.
Thanks!

One way is to make a dictionary where keys = words the model was trained on and values = glove representation of the words, i.e.
model = fitted Glove() model
D = {word: model.word_vectors[model.dictionary[word]] for word in model.dictionary.keys()}
Example: suppose "music" was in the training corpus and model.no_components  = 200. Then D["music"] is a vector of floats of length 200.


this will fail in version 2023.3.

Log Message:
Logger: homeassistant.helpers.frame
Source: helpers/frame.py:77
First occurred: 18:40:44 (1 occurrences)
Last logged: 18:40:44
Detected integration that called async_setup_platforms instead of awaiting async_forward_entry_setups; this will fail in version 2023.3. Please report issue to the custom integration author for solarman using this method at custom_components/solarman/init.py, line 18: hass.config_entries.async_setup_platforms(entry, PLATFORMS)

Indeed is failing:
Logger: homeassistant.helpers.entity
Source: custom_components/solarman/solarman.py:234
Integration: Solarman (documentation, issues)
First occurred: 19:49:17 (40 occurrences)
Last logged: 19:51:45
Update for sensor.gerador_input_active_power fails
Update for sensor.gerador_output_apparent_power fails
Update for sensor.gerador_output_active_power fails
Update for sensor.gerador_output_reactive_power fails
Update for sensor.gerador_inverter_id fails
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 548, in async_update_ha_state
await self.async_device_update()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 746, in async_device_update
raise exc
File "/usr/local/lib/python3.10/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
File "/config/custom_components/solarman/sensor.py", line 140, in update
self.inverter.update()
File "/usr/src/homeassistant/homeassistant/util/init.py", line 190, in wrapper
result = method(*args, **kwargs)
File "/config/custom_components/solarman/solarman.py", line 175, in update
self.get_statistics()
File "/config/custom_components/solarman/solarman.py", line 234, in get_statistics
sock.close()
UnboundLocalError: local variable 'sock' referenced before assignment

Is there any workaround for this? Great integration but the last HA update has broken it. Do I need to roll back or is there something I can change locally?

Is there an update for this? Basically you should update to 2023.03.1 or later because there was a security issue, so a solution for the problem with the newer version would be important.

Is there a plan to fix this issue?

Try redownloading the main version instead of the latest release v1.4.3.

the initial issue should be fixed with this commit https://github.com/StephanJoubert/home_assistant_solarman/commit/0cd2e8d0f98133682522a7e622e29373b5410227 in release 1.4.3


WRO-12043: Increased code coverage for Slider and Spinner

Checklist

[x] I have read and understand the contribution guide
[ ] A CHANGELOG entry is included
[ ] At least one test case is included for this feature or bug fix
[x] Documentation was added or is not needed
[ ] This is an API breaking change

Issue Resolved / Feature Added
Added missing tests for

Slider: drag and drop and for wheel events
Spinner: default aria label

Resolution
Additional Considerations
Util function for drag and drop was copied for the example on the testing library website: https://testing-library.com/docs/example-drag/
Links
WRO-12043
Comments
Enact-DCO-1.0-Signed-off-by: Daniel Stoian<EMAIL_ADDRESS>

Good job!
And I confirmed that the coverage cannot be increased anymore.
Thank you for adding missing test cases even if they are not related to code cov increase.
Please check the a questions about "setTimeout", I'm not sure so please let me know which is correct

@jeonghee27 , I applied your suggestion. it works. However, i don't understand why Travis CI is still pending


Read register with R command is not working in esp32 (Always showing 81)

You did not state which ESP board... Also, do you change the register before reading it? If you don't write something to the register, it will always return the same value. Also, check your wiring and circuitry. I remember your nickname from YouTube and based on your question there, I am not 100% sure that you do everything correctly (no offence).
I have an ESP32-WROOM-32 in front of me as I am writing this reply, and I can read and write any registers and it works perfectly.

I haven't changed the value of the register, so shouldn't it give the current default register value?

I am still convinced that there's something wrong on your side. I have the circuit in front of me and it works well. Incorrect wiring, using the incorrect SPI (especially on an ESP32) and so on... Since you haven't provided any details, no images, no schematics, I won't be able to help.


OCI-CommitmentDiscountType

Not applicable.
Comment:

The OCI Universal Credit annual commitment model typically involves a discretionary discount percentage, categorized as a negotiated discount rather than a commitment-based discount.


Use --focus=<focus> to focus profiling on a named focus area

Given this example:
$ wp profile --verbose
+----------------+------------------+
| Field          | Value            |
+----------------+------------------+
| hook_count     | 10737            |
| hook_time      | 0.19295263290405 |
| execution_time | 2.438s           |
| memory_usage   | 49.5mb           |
| query_count    | 192              |
| query_time     | 0.021s           |
+----------------+------------------+

For any of these named overview fields, I should be able to --focus=<field> to focus the profiling on a specific area. For instance, --focus=hook_count would give me a list of the common hooks.

+1


[RFR] Fixed an issue with the parsing of spaces option

Super sorry about that @maxhoffmann. Should be better know (I hope). Feel free to release directly, I bumped version number.

no worries. merged and published version 2.2.1.


Cannot find message Listeners in Version 3.7.2

Hi,
i'm using the react native connectycube sdk on my development.the version i'm using is 3.72,
when i tried using this code. (from react native documentation in connectycube doc site)
ConnectyCube.chat.onDeliveredStatusListener = function (messageId, dialogId, userId) { console.log("[ConnectyCube.chat.onDeliveredStatusListener] callback:", messageId, dialogId, userId);};
the.onDeliveredStatusListeneris not defined.
also i tried the other listeners. but i cant't find the listeners as mentioned in the documentation site.

how can i use that? it seems that version is not compatible in my react native version (0.63.2)
please help me on this issue.
Thank you.

Hi, @mikeangelsilva
Symbol "_" uses for methods' names to show that the method is private. So, this listeners uses inside the ConnectyCube SDK, you do not need to use they. It is conventional rule for JavaScript language.
Do not fully trust to a code editor's extension. It does not work as you expect. JS is not compiled language.
Chat listeners are not methods of the Chat class. You should define each chat's listener by a function.
Just try the code in development.

Hi, @mikeangelsilva
Symbol "_" uses for methods' names to show that the method is private. So, this listeners uses inside the ConnectyCube SDK, you do not need to use they. It is conventional rule for JavaScript language.
Do not fully trust to a code editor's extension. It does not work as you expect. JS is not compiled language.
Chat listeners are not methods of the Chat class. You should define each chat's listener by a function.
Just try the code in development.

tried using it. still not working the listener. please help me. we cant really proceed in the next step because of this listener.

this is the code i used.

also tried this.

not working. :(

Show your logs, please. The code snippet you have provided is correct, but this is not enough to understand the problem.
You can get logs from Metro serrver / Chrome debugger / AndroidStudio / XCode


Search parses CropStartTime from url if present

I download a lot of videos that are given by links that contain a timestamp (e.g. https://www.twitch.tv/videos/1021452561?t=16960s).
When searching for the video based on the url, the timestamp is discarded, and the user has to manually input it. This is extra-annoying if the timestamp is given in seconds only (as in the example) instead of hours, minutes and seconds.
This pull request adds parsing of the timestamp from the url and forwarding it to the download.

After a lot of playing around with this and testing it in daily work, I decided not to merge it. At first this seems convenient but it can cause trouble down the line.
Often I receive links to Twitch videos that contain the start time parameter in the query, just because the other guy didn't remove it or does not even know that it's there. If you now copy the link from the browser to TL, that actually DOES NOT mean that I only want to download the video from the given timestamp onwards. Maybe the timestamp is there, but I need the whole video? Who knows? Could be, could be not. In MY case more often I wanted to download the whole video even if the parameter was there.
That brings us to the conclusion:
If a user wants to only download a part of the video, he MUST tell Twitch Leecher. TL cannot decide, what to user wants in this case.

What do you think about parsing and inputting into the start time field, but not automatically enabling actual cropping (i.e. without , CropStart = video.StartTime != TimeSpan.Zero)?
This way, the start time from the url is available without the user having to manually calculate if he wants it but not changing the default behavior? The user would still have to enable start time cropping manually.

That could be a compromise. I'll implement that quickly in the "helix" branch. There the parsing method for the "3h35m55s" kinda timestamps is already there, because the helix API does this for the video duration now (which is abolutely atrocious API design). So you don't need to do anything and we can keep this PR closed. Appreciate the input!


[5.9] StackPromotion: fix a crash due to a problem in liferange evaluation

Explanation: The analysis to check if an alloc_ref outlives its "inner" liferange had a bug which resulted in a crash in the StackPromotion pass. It is very rare that this bug shows up and I only found it by coincidence. Still, it makes sense to fix it in 5.9 because the risk is very low.

Issue: rdar://112275272

Risk: Very low. It's a small change which makes the optimization more conservative.

Testing: With a regression test

Reviewer: @atrick

Main branch PR: https://github.com/apple/swift/pull/67331

@swift-ci test

We're getting close to 5.9 and I don't think this is critical enough.  Let's hold for later.


AI2 CRF Train changes

Improved Header model which yields metrics broadly comparable to Wapiti. The Title normalized is 3% down in F1, but other metrics (author, venue, year) are up by at least as much in total.
CRF=AI2 (old)
Time elapsed in milliseconds: 678692
Metric  Precision       Recall  F1
venueNonEmpty   1       0.63    0.77
venueNormalized 0.07    0.05    0.06
authorLastNameNormalized        0.91    0.9     0.91
year    0.95    0.46    0.62
yearNonEmpty    1       0.49    0.65
authorLastNameNormalizedFirstInitial    0.9     0.89    0.9
authorFullName  0.77    0.76    0.77
titleNonEmpty   1       0.98    0.99
venueExact      0.07    0.05    0.06
titleExact      0.54    0.53    0.54
titleNormalized 0.9     0.88    0.89
bibliography    0.36    0.89    0.51
CRF=WAPITI
Time elapsed in milliseconds: 1038056
Metric  Precision       Recall  F1
venueNonEmpty   1       0.61    0.76
venueNormalized 0.06    0.04    0.05
authorLastNameNormalized        0.91    0.89    0.9
year    0.99    0.39    0.57
yearNonEmpty    1       0.4     0.57
authorLastNameNormalizedFirstInitial    0.9     0.88    0.89
authorFullName  0.77    0.75    0.76
titleNonEmpty   1       0.98    0.99
venueExact      0.06    0.04    0.05
titleExact      0.55    0.54    0.55
titleNormalized 0.93    0.91    0.92
bibliography    0.36    0.88    0.51

Just grabbing from @rodneykinney as he's on vacation and I want to get this integrated.


Update README file with build and usage instructions

I've used PR Pilot to understand the project and update the README file.
Prompt saved in prompts/ directory for visibility

/pilot Add emojis to the README


Fixed a typo on NestedUpdateOperation.

Just a quick fix.
This would throw an undefined variable $request when trying to update a model.

Hi @DiyanDev, thank you very much for your help.
It is merged now.


GHSA-35fr-h7jr-hh86 Credit Assignment

Hey Armeria Team,
GitHub recently rolled out a credit assignment system to GitHub.
Would you be willing to assign credit to me for disclosing this vulnerability to your team?
https://github.com/line/armeria/security/advisories/GHSA-35fr-h7jr-hh86
Thank you so much!
Cheers,
Jonathan Leitschuh

Hi @JLLeitschuh, we'd be happy to assign credit to you, but I don't know how to do so. Would you mind explaining a little bit on how to assign credit to someone in GitHub's vulnerability page? I don't see anything noticeable in the vulnerability page. Maybe it's not rolled out to all repositories? 🤔

Hey @trustin
If you click 'edit' on the advisory (top right), then you go to the bottom you should see 'add a user', type my username 'JLLeitschuh' and click the '+' button.

Unfortunately, GitHub doesn't let me edit the advisory saying 'This advisory is being reviewed by GitHub and can no longer be edited.' 🤔

I've reached out to someone on GitHub staff to help out here.

👋  Hello from the Github CNA team.  @JLLeitschuh pinged me about this as  were discussing some unrelated stuff about advisory credits.
After taking a look, it seems this advisory was unfortunately stuck in a "locked" state due to a bug that has since been addressed.  Basically we changed our logic for locking/unlocking advisory editing and unfortunately it seems we neglected to migrate the data for a few advisories with pending cve requests.  The effect is they were permanently locked to editing.
I have manually unlocked this particular advisory so it should now be possible to add @JLLeitschuh or any other github user to the credits per the above instructions from Jonathan.
To be clear:  The only time an advisory is expected to be locked for editing is when a CVE Request is pending, and even then the expected behavior is it automatically unlocks 72 hours after the cve request is sent if GitHub does not respond with a cve assignment in that time window.   My personal opinion is we should not prevent edits at any time and potentially that behavior change will come this year.
Thanks for raising this @JLLeitschuh 🙇

@trustin, can you follow up here? Just let me know if you need anything on my end! 😄
Thanks @rschultheis!

Friendly ping @trustin 😄

Sorry I'm late! Just added you to the Credits field. It seems like you need to accept it.

Done! Thank you so much! 😄
It was a pleasure working with you on this issue.

Thanks a lot for your patience on this. Looking forward to your reports in the future, although no vulnerability would be the best :wink:


Stabilize seal_debug_message

Stabilize without changes as introduced in #8773. It is implemented in ink! and works as intended.

bot merge


API sk_ prefix

Functions need a consistent prefix throughout the project!

Used an sk namespace instead of an sk_ prefix. Solves the same issue, but results in shorter code.


Adding verbosity to e2e.sh

Using bash 'trap DEBUG' to print relevant commands before execution

Look in the Travis job - You can see the trap debug of commands in a cyan color:
https://travis-ci.com/submariner-io/submariner/builds/120213965

Nice.
Is it possible to call trap_commands once instead of inside every function?

I'm afraid not, since every function seems to have its own scope.
But if we want to make this verbosity during whole 'ci' and 'test' scripts,
we can move this function to:
https://github.com/submariner-io/submariner/tree/master/scripts/lib
And then source it in:
https://github.com/submariner-io/submariner/blob/master/scripts/e2e
https://github.com/submariner-io/submariner/blob/master/scripts/evi
On Tue, Jul 23, 2019 at 6:17 PM Miguel Angel Ajo Pelayo <
<EMAIL_ADDRESS>wrote:

Nice.
Is it possible to call trap_commands once instead of inside every function?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/submariner-io/submariner/pull/115?email_source=notifications&email_token=ACLUMWLGDDSQQZKJ3BVBPKLQA4OITA5CNFSM4IGD2MT2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2TOZ5I#issuecomment-514256117,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACLUMWJ7AKDTR7MQZKMYZXLQA4OITANCNFSM4IGD2MTQ
.

Is it possible to call trap_commands once instead of inside every function?

set -x works for a complete shell invocation, so it would only need to be set once per shell script; but it doesn’t allow as much fine-tuning of the output (only a prefix given in PS4).

I'm afraid not, since every function seems to have its own scope. But if we want to make this verbosity during whole 'ci' and 'test' scripts, we can move this function to: https://github.com/submariner-io/submariner/tree/master/scripts/lib And then source it in: https://github.com/submariner-io/submariner/blob/master/scripts/e2e https://github.com/submariner-io/submariner/blob/master/scripts/evi
…
On Tue, Jul 23, 2019 at 6:17 PM Miguel Angel Ajo Pelayo < @.***> wrote: Nice. Is it possible to call trap_commands once instead of inside every function? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#115?email_source=notifications&email_token=ACLUMWLGDDSQQZKJ3BVBPKLQA4OITA5CNFSM4IGD2MT2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2TOZ5I#issuecomment-514256117>, or mute the thread https://github.com/notifications/unsubscribe-auth/ACLUMWJ7AKDTR7MQZKMYZXLQA4OITANCNFSM4IGD2MTQ .

I like that but let's leave it to another patch.
I tested locally and I understand what you mean, the trap scope does not get inside the functions sadly.
I'm ok with the current format, the output is really nice.  If you @dimaunx (I see your +1) @skitt are ok I can merge.

I like this lot, it improves debugging.

I guess we should also add it into the ci script:
https://github.com/submariner-io/submariner/blob/master/scripts/ci
Then we'll see more debug info there too.
Let's add another patch to move this function into a general lib in:
https://github.com/submariner-io/submariner/tree/master/scripts/lib
On Wed, Jul 24, 2019 at 11:46 AM Dmitry Groisman<EMAIL_ADDRESS>wrote:

I like this lot, it improves debugging.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/submariner-io/submariner/pull/115?email_source=notifications&email_token=ACLUMWPAJIPTZLVAA4RVA73QBAJFZA5CNFSM4IGD2MT2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2VUCPA#issuecomment-514539836,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACLUMWOKGAQAVURZMXHSFZTQBAJFZANCNFSM4IGD2MTQ
.

It's now conflicting :), can you use the chance to move the common trap stuff to scripts/lib?

done.
Travis job: https://travis-ci.com/submariner-io/submariner/builds/120489195
On Wed, Jul 24, 2019 at 5:04 PM Miguel Angel Ajo Pelayo <
<EMAIL_ADDRESS>wrote:

It's now conflicting :), can you use the chance to move the common trap
stuff to scripts/lib?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/submariner-io/submariner/pull/115?email_source=notifications&email_token=ACLUMWOWEHNIHTUEU5GNWCDQBBOORA5CNFSM4IGD2MT2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2WOABY#issuecomment-514646023,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACLUMWPXI4IYNGNVQTE3K3LQBBOORANCNFSM4IGD2MTQ
.


Synthesis failed for Google.Cloud.DataCatalog.V1

Hello! Autosynth couldn't regenerate Google.Cloud.DataCatalog.V1. :broken_heart:
Here's the output from running synth.py:
Cloning into 'working_repo'...
Switched to a new branch 'autosynth-Google.Cloud.DataCatalog.V1'
Cloning into '/tmpfs/tmp/tmp1r4m5dqa/googleapis'...
Note: checking out '88316b63a486002727e14032d104690541179fc9'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:

  git checkout -b <new-branch-name>

HEAD is now at 88316b63a Generated synth.py files with multiple commits enabled
Note: checking out '7be2811ad17013a5ea24cd75dfd9e399dd6e18fe'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:

  git checkout -b <new-branch-name>

HEAD is now at 7be2811a fix: Update gapic-generator version to pickup discogapic fixes
Switched to a new branch 'autosynth-Google.Cloud.DataCatalog.V1-2'
Running synthtool
['/tmpfs/src/git/autosynth/env/bin/python3', '-m', 'synthtool', '--metadata', 'synth.metadata', 'synth.py', '--']
2020-03-30 07:23:22,196 synthtool > Executing /tmpfs/src/git/autosynth/working_repo/apis/Google.Cloud.DataCatalog.V1/synth.py.
Contents of preconfig file:
{"preclonedRepos": {"https://github.com/googleapis/google-cloud-dotnet.git": "/tmpfs/src/git/autosynth/working_repo", "https://github.com/googleapis/googleapis.git": "/tmpfs/tmp/tmp1r4m5dqa/googleapis"}}Extracted repo location: /tmpfs/tmp/tmp1r4m5dqa/googleapis
generateapis.sh: line 40: declare: GOOGLEAPIS: readonly variable
2020-03-30 07:23:22,244 synthtool > Failed executing /bin/bash generateapis.sh --check_compatibility Google.Cloud.DataCatalog.V1:

None
2020-03-30 07:23:22,245 synthtool > Wrote metadata to synth.metadata.
Traceback (most recent call last):
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/synthtool/__main__.py", line 102, in <module>
    main()
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/synthtool/__main__.py", line 94, in main
    spec.loader.exec_module(synth_module)  # type: ignore
  File "<frozen importlib._bootstrap_external>", line 678, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/tmpfs/src/git/autosynth/working_repo/apis/Google.Cloud.DataCatalog.V1/synth.py", line 22, in <module>
    hide_output = False)
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/synthtool/shell.py", line 39, in run
    raise exc
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/synthtool/shell.py", line 33, in run
    encoding="utf-8",
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/subprocess.py", line 438, in run
    output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '('/bin/bash', 'generateapis.sh', '--check_compatibility', 'Google.Cloud.DataCatalog.V1')' returned non-zero exit status 1.
Synthesis failed
Traceback (most recent call last):
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/tmpfs/src/git/autosynth/autosynth/synth.py", line 482, in <module>
    main()
  File "/tmpfs/src/git/autosynth/autosynth/synth.py", line 334, in main
    return _inner_main(temp_dir)
  File "/tmpfs/src/git/autosynth/autosynth/synth.py", line 429, in _inner_main
    branch,
  File "/tmpfs/src/git/autosynth/autosynth/synth.py", line 135, in synthesize_loop
    metadata_path, extra_args, deprecated_execution, environ, synthesize_py_path,
  File "/tmpfs/src/git/autosynth/autosynth/synth.py", line 278, in synthesize
    synth_proc.check_returncode()  # Raise an exception.
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/subprocess.py", line 389, in check_returncode
    self.stderr)
subprocess.CalledProcessError: Command '['/tmpfs/src/git/autosynth/env/bin/python3', '-m', 'synthtool', '--metadata', 'synth.metadata', 'synth.py', '--']' returned non-zero exit status 1.

Google internal developers can see the full log here.

Autosynth passed, closing! :green_heart:


onEndReached doesn't work as expected using ListView on Android

When I try render a list of items using ListView, onEndReached method of ListView is triggered without any interaction, as far as I understood it should only be called whenever all rows of the datasource is rendered but it doesn't work as it intended. It renders all the rows given to its datasource regardless of whether they are displayed on the screen or not.
I already asked a question on Stackoverflow and explained the details over there:
http://stackoverflow.com/questions/36218698/onendreached-doesnt-work-as-expected-using-listview-on-android-with-reactnative
React-Native version: 0.22
Mobile Platform: Android and IOS
Does anyone have any idea what might be the problem?
Thanks

I guess I understood my problem. My ListView was under a ScrollView component as I had a search box in the same view. After moving ListView outside of the ScrollView it started to work as expected. I was also able to add search box under ListView using renderHeader prop.

sou gay