Configure Initial Setup for SAP Mobile Cards

Tutorial URL: https://developers.sap.com/tutorials/cp-starter-digitalexp-mobilecards-1-setup.html
Please specify the step you are referring to
Step 6
Getting no pop-up on clicking the Create Card Template icon in SuccessFactors My TimeSheet.

Hi @jayeshsaini
Could you check if you  are seeing a warning message in "Mobile Services cockpit" like mentioned in this issue ?
https://github.com/SAPDocuments/Tutorials/issues/4022
If so, could you follow the steps as i mentioned there and try again.

Hi @jitendrakansal
I'm facing the following issue when scanning the QR code and setting up Mobile Cards App on my phone. Not even showing the Welcome Card and any other cards defined by me in subscriptions.
Not sure if the issue is with trail account and Connectivity.

Also when clicking on the Create Card Template icon in SuccessFactors My TimeSheet, I'm getting no popup as showed in tutorial for putting the API Key value. Please check the following step:

@jayeshsaini Could we have a short call if you are available? Can you send me a short note to my email address
jitendra.kansal@sapdotcom , i will send meeting request.

Sure

@thecodester i had a short call with @jayeshsaini ..issue has been resolved, you can close it.

@jitendrakansal If there is something with the solution that you and @jayeshsaini discussed that would be helpful for others, please add it to this issue. And thanks for handling.


MACOS(the dashboard need to be started manually)

the dashboard need to be started manually?  Can it be started by the terminal?

Not at this time. How would you want that to work?

I hope npm or yarn run can it automatically....thanks.


Type inference from default expression doesn’t compile in free function

Description
A function definition that relies on Type inference from default expression (SE-0347 compiles when it's defined inside a type (a method), but does't compile when it's defined as a free function.
Reproduction
As a method, this compiles:
struct MyTask {
    func sleep<C: Clock>(
        until deadline: C.Instant,
        clock: C = ContinuousClock()
    ) async throws {}
}

But: the same function signature as a free function doesn't compile:
func sleep<C: Clock>(
    until deadline: C.Instant,
    // ❌: Cannot use default expression for inference of 'C'
    // because it is inferrable from parameters #0, #1
    clock: C = ContinuousClock()
) async throws {}

Expected behavior
The free function should compile.
Environment
macOS 14.3, Xcode 15.3 (Swift 5.10)
$ swift --version
swift-driver version: <IP_ADDRESS> Apple Swift version 5.10 (swiftlang-<IP_ADDRESS> clang-15<IP_ADDRESS>)
Target: arm64-apple-macosx14.0

Additional information
Forum thread where this is being discussed: https://forums.swift.org/t/type-inference-from-default-expression-doesnt-compile-in-free-function/70521/3

Same issue on Xcode 16.1

Still the same issue on Xcode 16.1. Adding the method inside a Task seems to word as a workaround.
extension Task {
    func withTimeout<C: Clock>(
        _ timeout: C.Instant.Duration,
        clock: C = ContinuousClock(),
        task: @escaping @Sendable () async throws -> Success
    ) async throws -> Success {
            ...
    }
}


Update pyScss

Description of what has changed

Updated pyScss to avoid install bug from https://github.com/Kronuz/pyScss/issues/385

Also updates typed-ast for same bug: https://github.com/python/typed_ast/issues/126


Add tracer for dataloader events

Unfortunately it looks like there isn't really anything there that looks interesting enough to put into attributes, but events are there.
Signed-off-by: Maciej Szlosarczyk<EMAIL_ADDRESS>

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please ask for approval from maciej-szlosarczyk after the PR has been reviewed.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 511fb94bfcd2f68f7b7232b07e54058082cbbcc4

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: onimsha
To complete the pull request process, please ask for approval from maciej-szlosarczyk after the PR has been reviewed.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: antonsatin, onimsha
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [antonsatin]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Account for user supplied input height in determining cursor position.

Fix #505.
👋 Hey @bryphe! Excited to make my first PR to revery. This simply fixes issues around cursor positioning with inputs that are not at the default height. All this involves, I think, is taking into account the possibility of a user-supplied height on the Input component, and using that in lieu of the defaultHeight to calculate the cursor's marginTop. If no height is provided, we fallback to defaultHeight. Here's what it looks like with a 200px tall Input, for example.

Let me know if there's anything else you'd like me to add here, happy to add an example into source ✅ Thanks for all your hard work on revery!

Hi @parkerziegler ! Great to see you here - thank you for taking on #505 ! 🎉
Fix looks good to me - awesome PR!
I saw the CI was failing so I merged master to pick up #574 , and ran esy format... I think CI should pass now 🤞

Ah sweet, I'll make sure to do that on future PRs. Yeah @kitten and I were joking that we both made our first PRs to revery on the same day 😂

Looks like CI is green! 🎉

Yeah @kitten and I were joking that we both made our first PRs to revery on the same day 😂

Haha ya it was cool to see you both submit PRs at the same time. Figured you guys coordinated it or something 😉 Formidable represent!
Bringing this in now - thanks for the contribution, @parkerziegler !


feat(cmd): Support custom cache

Signed-off-by: Ce Gao<EMAIL_ADDRESS>
What this PR does / why we need it:
We keep the cache and config.json in $HOME/.ormb/ by default. But users can customize it using rootPath in $HOME/.ormb/config.yaml
Which issue(s) this PR is related to (optional, link to 3rd issue(s)):
Fixes #
Reference to #

Special notes for your reviewer:
/cc @your-reviewer

Release note:

NONE

/assign @codeflitting

/lgtm

bot will not automatically add /approve label now? @bbbmj

/

/approve

/approve


Print vmap warnings by default

torch._C._debug_only_display_vmap_fallback_warnings -> Use a functorch API instead of this
Turn on warnings by default so that it is clear we are not promising good perf

Done, FYI @Chillee

I guess it's not pushed yet?


Compilation error with LDC

Tracking https://github.com/etcimon/memutils/issues/19:

Compilation error with LDC 1.27.0-beta3:
memutils-1.0.4\memutils\source\memutils\allocators.d(18,8): Error: module std.exception import enforceEx not found, did you mean template std.exception.enforce(E : Throwable = Exception) if (is(typeof(new E("", string.init, size_t.init)) : Throwable) || is(typeof(new E(string.init, size_t.init)) : Throwable))?

Switching to memutils~master results in this error in libasync:

../../../../.dub/packages/libasync-master/libasync/source/libasync/internals/memory.d(17,8): Error: module std.exception import enforceEx not found, did you mean template std.exception.enforce(E : Throwable = Exception) if (is(typeof(new E("", string.init, size_t.init)) : Throwable) || is(typeof(new E(string.init, size_t.init)) : Throwable))?

See #91.
See also #92.

I am not very professional user of GitHub - so I have no idea why your PR is not pass the tests, but it helps me to build and run lighttp with steps below:

build memutils from ~master
set add-local with version 1.0.4 in dub (it is very sad that the installation from github and branches are prohibited)
download chances/libasync branch "fix-signal"
set add-local with version 8.6.0
dub build/dub run with dependency lighttp

Ping @etcimon


Get info from GitHub

I have already tried to make use of GitHub API but its a mess, unauthenticated API requests are limited to 60 requests per hour.
So optimization required

I can help with the API - any specific files you need editing? If you could provide the paths and what's expected, I'd be glad to help!

Ohh, yes. The Website directory contains the web contents where you can find the /Website/index.js file has API calls. So that should be a good start.
If you are good in NodeJS you can also checkout the NodeJS directory. It has'nt started yet
Happy coding. 😄

When you close the first PR about documentation and the frontend I'll make a new one to link to my changes in the backend.  :+1:

Posting a poll to make changes in the way of getting info of the user.
https://strawpoll.com/341rzfpuu


eruti

VSCode Version:
OS Version:

Steps to Reproduce:

Please provide some information about what is going wrong


Add a handler for unknown options

As discussed in #225, it could be nice to have a way to handle unknown options specified by users like .unknownOptions(), or have some kind of "wildcard" suffix like --option.*
One use case would be to let users "pass an object" with properties that are not known in advance.
mycli create-object-in-database --object.name "test object" --object.content "hello world" --object.custom_property "really cool"

Letting users have an unknown handler would let them manage in this example all properties starting with --object.*, though it could be also directly handled by cliffy if it isn't too hard to implement

My use-case is similar, I have syntax like that:
mycli run --action action1 -- --text foo --value 5

which is kind of short for below:
mycli run --action action1 --params.text foo --params.value 5

Here the params are only known during runtime, eg only known after the action function is called. the cli does not know the format of those params.


Istio fault injection routing rule from Ingress

I am trying to get the fault injection abort rule to work from Ingress ( I use traefik on my local minikube)  with my local application. For instance, say Grafana,
Service definition:
apiVersion: v1
kind: Service
metadata:
  name: myhost-grafana
  labels:
    app: grafana
    chart: grafana-0.1.0
    release: foiled-panda
    heritage: Tiller
spec:
  type: ClusterIP
  # type: NodePort
  ports:
    - port: 3000
      targetPort: 3000
      protocol: TCP
      name: grafana
  selector:
    app: grafana

Ingress:
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  name: myhost-grafana
  labels:
    app: grafana
    chart: grafana-0.1.0
    heritage: Tiller
  annotations:
      0: "map[kubernetes.io/ingress.class:istio]"
      1: "map[kubernetes.io/ingress.class:traefik]"
spec:
  rules:
    - host: "grafana.local"
      http:
        paths:
          - path: /
            backend:
              serviceName: myhost-grafana
              servicePort: 3000

Istio abort rule:
apiVersion: config.istio.io/v1alpha2
kind: RouteRule
metadata:
  name: deny-route
spec:
  destination:
    name: myhost-grafana
  match:
    source:
      labels:
        app: grafana
  precedence: 1
  route:
  - weight: 100
  httpFault:
    abort:
      percent: 100
      httpStatus: 403

But all I see when I curl is ,
$ curl -I http://grafana.local
HTTP/1.1 200 OK
Content-Type: text/html; charset=UTF-8
Date: Thu, 23 Nov 2017 12:05:47 GMT
Set-Cookie: grafana_sess=195f2cc07373ac95; Path=/; HttpOnly

I also tried curl from within the minikube vm with the  endpointIP:port . The output is same.
May be am missing some configuration. Has someone got this working for your application?  Any help/suggestions would be greatly appreciated.

fixed by #796 .. Fault injection is not available in ingress.


Organisational Chart Upload validation error

Started POST "/form/26/attachments.json" for <IP_ADDRESS> at 2015-05-20 16:18:02 +0300
Processing by FormController#add_attachment as JSON
  Parameters: {"authenticity_token"=>"abwZG0QWYPpBlnZdN8eSCA275tVWo/HfdusHn0CDKZBY9sOhBlZ17b9N2R+3oB6K679L6AsZuK6xAo07EmjpVw==", "question_key"=>"org_chart", "form"=>{"file"=>#<ActionDispatch::Http::UploadedFile:0x007fca94eab350 @tempfile=#<Tempfile:/tmp/RackMultipart20150520-9066-1pn6l75.jpg>, @original_filename="friday.jpg", @content_type="image/jpeg", @headers="Content-Disposition: form-data; name=\"form[file]\"; filename=\"friday.jpg\"\r\nContent-Type: image/jpeg\r\n">}, "id"=>"26"}
  User Load (0.4ms)  SELECT  "users".* FROM "users" WHERE "users"."id" = $1  ORDER BY "users"."id" ASC LIMIT 1  [["id", 1]]
  Account Load (0.2ms)  SELECT  "accounts".* FROM "accounts" WHERE "accounts"."id" = $1 LIMIT 1  [["id", 1]]
  FormAnswer Load (0.3ms)  SELECT  "form_answers".* FROM "form_answers" WHERE "form_answers"."account_id" = $1 AND "form_answers"."id" = $2 LIMIT 1  [["account_id", 1], ["id", 26]]
  FormAnswerAttachment Load (0.2ms)  SELECT "form_answer_attachments".* FROM "form_answer_attachments" WHERE "form_answer_attachments"."form_answer_id" = $1  [["form_answer_id", 26]]
   (0.3ms)  BEGIN
   (0.2ms)  SELECT COUNT(*) FROM "form_answer_attachments" WHERE "form_answer_attachments"."question_key" = $1  [["question_key", "org_chart"]]
   (0.2ms)  ROLLBACK
Completed 422 Unprocessable Entity in 39ms

ActiveRecord::RecordInvalid - Validation failed: Maximum amount of these kind of files reached.:
  activerecord (4.2.0) lib/active_record/validations.rb:79:in `raise_record_invalid'
  activerecord (4.2.0) lib/active_record/validations.rb:43:in `save!'
  activerecord (4.2.0) lib/active_record/attribute_methods/dirty.rb:29:in `save!'
  activerecord (4.2.0) lib/active_record/transactions.rb:291:in `block in save!'

Validation is need to be corrected, because if there is at least of one FormAnswerAttachment with "org_chart" question key in db -> then always would be raised validation issue.
  def question_key_correctness
    # possible improvements:
    # - validating all upload questions in this way
    # - smart collecting max-attachments information from the form structure
    if question_key == "org_chart" && new_record?
      if self.class.where(question_key: self.question_key).count > 0
        errors.add(:base, "Maximum amount of these kind of files reached.")
      end
    end
  end

Probably need to add form_answer_id condition  above ^

This one is fixed


fix: Whitelist the 'charset' type param to include it in the response

Other params are put in a typeParams property.
Fixes #1513

Tests def need added if you could!

Closing until you submit a better PR


Add support for wdio v5

This project does not support the new version of WebdriverIO.
@rhysd is there any plan on making it compatible with v5?
Thanks in advance.

What needs to be done to support wdio v5?

WebdriverIO has an open issue to port this library into the core packages. This would be a lot easier to do than convince someone to update a library that hasn't been touched in 2 years. https://github.com/webdriverio/webdriverio/issues/3693

I'm working on porting this service to the wdio mono repo (webdriverio/webdriverio/issues/3693)

We have absorbed the project into the WebdriverIO org. I recommend to use @wdio/appium-service from now on.


Cannot install Composer

Hi all,
I am unable to install Composer.
I keep getting exit code 1.
This is my log
The Composer installer script was not successful [exit code 1].
OpenSSL failed with a 'certificate verify failed' error. This indicates a problem with the Certificate Authority file(s) on your system, which either cannot be found or may be out of date.
Script Output:
The "https://getcomposer.org/versions" file could not be downloaded: SSL operation failed with code 1. OpenSSL Error messages:
error:1416F086:SSL routines:tls_process_server_certificate:certificate verify failed
Failed to enable crypto
Failed to open stream: operation failed

No log file supplied.


[Silabs] Adds support for ICD management cluster for WiFi lighting application

Adding changes for Lighting app wifi zap file for ICD Management Cluster.

Light App won't support ICDM Cluster.. It is applicable for Lock app(Sleepy) for both Thread and wifi. Discussed with Mathieu. Hence closing this Pull Request.


tamper protection of results

[7:34 PM] Dreibaby: Specify generates an ID with every report and specified can generate the ID based on arbitrary information inside the report and compare

After discussing this with Drei we came to the conclusion this is ultimately unnecessary. The default path for uploading data doesn't even allow for users to tamper with the data. Even if they used the manual upload method, a flag on this could cause other issues with whomever is reading the data.
Closing this as a reconsidered design choice.


Mapnik v2 spec

With<EMAIL_ADDRESS>landed we should make sure we are all up to date.

now working on updated pull at #1546


Broken API / syntacktic pull request / citation use for related utility

Hey,
I'm currently trying to use your Python wrapper to create a Python utility that can sync between Todoist and Habitica, but the API update issue means that the master copy won't work as is. While I'm developing it, I don't have an issue using syntacktic's version with the API update (see the pull request), but it'd be awesome if by the time I finish this thing I can direct people straight to pip install habitica for the API wrapper and then whatever I wind up naming the syncer program. I think it'd be simpler. I know it's been a super short timeframe and not sure if this project is being very tightly monitored at this time, but I wanted to let you know before I did anything.
Otherwise, how would you like me to cite your work when I finish my project? I'll default to linking your work here, obviously, with all attribution attached.
Cheers,
Erin

Also, I apologize if this isn't the place for this comment! I'm new to using Github as a means of contacting and also creating open source utilities, so I'm still learning the norms of social interactions here.

This is the right place! I've just been super busy at work and not prioritizing this.
I've merged the v3 PR and a couple others, things with habitica v0.0.15 seem to be working again.
Sync tool: nice!


DeserializeSeed support

Sometimes, it's necessary for a type to only implement DeserializeSeed, and not Deserialize. serde_yaml should support this use case, especially with the functions related to from_str. I think the type should look something like this:
fn from_str_seed<T>(seed: T, s: &str) -> Result<T::Value>
    where T: DeserializeSeed

As we discussed in https://github.com/serde-rs/json/issues/282 - I think this is better addressed by https://github.com/dtolnay/serde-yaml/issues/44. Something like:
seed.deserialize(&mut serde_yaml::Deserializer::from_str(s))


add support for managing tags

Closes #26

Codecov Report

Merging #26 into master will increase coverage by 0.81%.
The diff coverage is 77.04%.

@@            Coverage Diff             @@
##           master      #26      +/-   ##
==========================================
+ Coverage   73.77%   74.59%   +0.81%
==========================================
  Files           4        5       +1
  Lines         183      244      +61
==========================================
+ Hits          135      182      +47
- Misses         27       34       +7
- Partials       21       28       +7

Impacted Files
Coverage Δ

tag.go
77.04% <77.04%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5af8d7a...8f21e6d. Read the comment docs.


Roll Flutter Engine from a01548f5fa82 to 072a9ca3773c (1 revision)

https://github.com/flutter/engine/compare/a01548f5fa82...072a9ca3773c
2023-01-05<EMAIL_ADDRESS>Add TextProvider and TextEdit patterns to AXPlatformNodeWin (flutter/engine#38646)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

43b912090224614885a297f8d189ad648b6d7258


Adding onIdleCamera event for Google Maps

Calculation to get the bounds of the map northEast and southWest corners are not exact with the help of region data provided with deltas. Hence a new event is being added when the camera is idle at a position. It adds the enhancement claimed under issue #356
Usage - Wants to show the favourite places inside a particular region when user moves camera.

Does any other open PR do the same thing?

No.
How did you test this PR?

Tested via Xcode and Android Studio in both iOS & Android platform for compatibility.

@ShivamKJJW besides my comment LGTM, also please fix the conflict ❤
@alvelig @christopherdro 🐽

Will this API work now that we're using Google Maps 3.0.3?

Closing as being an old PR and not a clear reply. Thanks for all the effort 😻


Signatures appear vertical when signing document

When I add my drawn signature to a vertical PDF, then redownload the file, the added signature is completely vertical on the right-hand side of the page - not in the location where I placed it on the PDF (bottom left)

@HiDef888 How did you solve the Problem. I have the same issue. Trying to insert a png or jpeg as a signature

Can you run the pdf through the pdf repair and see if to changes anything?

Thx @Frooodle repairing solved the problem
What also solved the problem is to use "view PDF" (with the unrepaired PDF) and insert the image from there.
Thanks for your fast help

@Frooodle forgot to mention. Repair after signing didn't solve the issue


Need Solution for the Issue :Object reference not set to an instance of an object

Hi I am using the template ID with embedded sign using the example 32 and envelope update for the 33
Kindly help me to resolve the issue
this is for the envelope Update
namespace Example\Controllers\Examples\eSignature;

use DocuSign\eSign\Api\EnvelopesApi\UpdateOptions;
use DocuSign\eSign\Client\ApiException;
use DocuSign\eSign\Model\EnvelopeDefinition;
use DocuSign\eSign\Model\Workflow;
use DocuSign\eSign\Model\Recipients;
use DocuSign\eSign\Model\Signer;
use DocuSign\eSign\Model\RecipientViewRequest;
use Example\Controllers\eSignBaseController;
use Example\Services\SignatureClientService;
use Example\Services\RouterService;

class EG033UnpauseSignatureWorkflow extends eSignBaseController
{
    /** signatureClientService */
    private $clientService;

    /** RouterService */
    private $routerService;

    /** Specific template arguments */
    private $args;

    private $eg = "eg033"; # Reference (and URL) for this example

    /**
     * Create a new controller instance.
     *
     * @return void
     */
    public function __construct()
    {
        $this->args = $this->getTemplateArgs();
        $this->clientService = new SignatureClientService($this->args);
        $this->routerService = new RouterService();
        parent::controller($this->eg, $this->routerService, basename(__FILE__));
    }

    /**
     * 1. Check the token
     * 2. Call the worker method
     *
     * @return void
     * @throws ApiException for API problems and perhaps file access \Exception, too
     */
    public function createController(): void
    {
        $minimum_buffer_min = 3;
        if ($this->routerService->ds_token_ok($minimum_buffer_min)) {
            # 1. Call the worker method
            # More data validation would be a good idea here
            # Strip anything other than characters listed
            $results = $this->worker($this->args);

            if ($results) {
                $_SESSION['pause_envelope_id'] = false;
                # Redirect the user to the embedded signing
                # Don't use an iFrame!
                # State can be stored/recovered using the framework's session or a
                # query parameter on the returnUrl (see the make recipient_view_request method)
                header('Location: ' . $results["redirect_url"]);
                exit;
              } else {
            $this->clientService->needToReAuth($this->eg);
        }
    }

    public function worker($args): array
    {

          $envelope_api = $this->clientService->getEnvelopeApi();
         $env = new EnvelopeDefinition([
            'workflow' => new Workflow(['workflow_status' => 'in_progress']
        ),

        ]);

        $envelope_option = new UpdateOptions();
        $envelope_option -> setAdvancedUpdate('true');
        $envelope_option -> setResendEnvelope('true');
        $signer1 = new Signer([ # The signer1
                'email' => ' '#### ',
                'name' => 'GH',
                'recipient_id' => "1",
                'routing_order' => "1",
                'role_name' => 'd',
                'client_user_id' =>  '#### ',
        ]);

        $signer2 = new Signer([ # The signer2
            'email' => '#### ',
            'name' => 'MH',
            'recipient_id' => "2",
            'routing_order' => "2",
            'role_name' => 'p',
            'client_user_id' =>  '#### ',
        ]);
            $recipients = new Recipients([
                'signers' => [$signer1, $signer2, ],
            ]) ;
            $env -> setTemplateId( '#### ');
            $env->setTemplateRoles([$signer1, $signer2]);

       $authentication_method = 'None';
        $recipient_view_request = $this->clientService->getRecipientViewRequest(
            $authentication_method,
            $args['receipent_arguments']);
            $envelope = $envelope_api->update(
                $args['account_id'],
                $args['pause_envelope_id'],
                $envelope=$env,
                $options=$envelope_option
            );
            $envelope_id = $envelope['envelope_id'];

                 $results = $this->clientService->getRecipientView($args['account_id'], $envelope_id,
                 $recipient_view_request);
                return ['envelope_id' => $envelope_id, 'redirect_url' => $results['url']];

    }

    /**
     * Get specific template arguments
     *
     * @return array
     */
    private function getTemplateArgs(): array
    {
        $signer2_name  = preg_replace('/([^\w \-\@\.\,])+/', '', $_SESSION['signer2_name']);
        $signer2_email = preg_replace('/([^\w \-\@\.\,])+/', '', $_SESSION['signer2_email']);
        $receipent_arguments =[
            'signer_email' =>  '#### ',
            'signer_name' => 'MH',
            'signer_client_id' => '#### ',,
            'ds_return_url' => $GLOBALS['app_url'] . 'index.php?page=ds_return',
            'recipient_id' => '2',
            'routing_order' => '2',
            'role_name' => 'p',
        ];
        $args = [
            'account_id' => $_SESSION['ds_account_id'],
            'base_path' => $_SESSION['ds_base_path'],
            'ds_access_token' => $_SESSION['ds_access_token'],
            'pause_envelope_id' => '#### ',
            'receipent_arguments' => $receipent_arguments,
        ];
       return $args;
    }
}

     * Get specific template arguments
     *
     * @return array
     */
    private function getTemplateArgs(): array
    {
        $signer2_name  = preg_replace('/([^\w \-\@\.\,])+/', '', $_SESSION['signer2_name']);
        $signer2_email = preg_replace('/([^\w \-\@\.\,])+/', '', $_SESSION['signer2_email']);
        $receipent_arguments =[
            'signer_email' =>  '#### ',
            'signer_name' => 'MH',
            'signer_client_id' => '#### ',,
            'ds_return_url' => $GLOBALS['app_url'] . 'index.php?page=ds_return',
            'recipient_id' => '2',
            'routing_order' => '2',
            'role_name' => 'p',
        ];
        $args = [
            'account_id' => $_SESSION['ds_account_id'],
            'base_path' => $_SESSION['ds_base_path'],
            'ds_access_token' => $_SESSION['ds_access_token'],
            'pause_envelope_id' => '#### ',
            'receipent_arguments' => $receipent_arguments,
        ];
       return $args;
    }
}

Certain parts of this look like they need to be reworked -  there's an extra comma next to the signer_client_id that could be causing you problems.  I see that the recipient id routing order and role name are hardcoded as args for the 2nd user. The view request client user id, name, and email need to match or the view request will not work. You're sending those off to the view request that presumably generates per each user - that would mean those args need to be tailored to each user per each getRecipientViewRequest.
also           'email' => ' '#### ', had an extra single quote but I don't think that's your issue.
I tried to run your code but was unable to replicate your issue (and ran into my own separate unrelated issue that made it impossible to get your snippet working.  Is there a specific stacktrace your getting this in relation to in the code, like a line number for this file or perhaps it leads from an error in a different file that ex 33 is using?


Is possible to checkout the source code task-vision wasm?

Hi @schmidt-sebastian, I just want to ask: is that possible to know where is the source code of task vision wasm? I found https://storage.googleapis.com/mediapipe-assets/wasm/vision_wasm_internal.js / https://storage.googleapis.com/mediapipe-assets/wasm/vision_wasm_internal.wasm, and whether the source code of vision_wasm_internal is available for external developers?

Hi @sucong0826,
Unfortunately, significant portions of the C++ Wasm sources are missing. The GPU code and the code that invokes MediaPipe are not open source. However, most of the calculators are available and can be found in the tasks/cc directory. It is not currently possible to build custom Wasm blobs.
Thank you!!


[Storage Manager] Use Real Chunk IDs

Contributes to #111.
Using real chunk ids.
Problem
A chunk id describes the position of a chunk within a table. If the table is not known, a chunk cannot derive its chunk id. Therefore, lets put it like this, that the chunk gets it id once the table is added.

Heyoo, before I review, can you rebase on master? 😄

Continued in #99.


Firmware version served from API route

I believe this commit solves #83.
Current default in the React app is the same env variable, however. Please suggest what to put, although it doesn't impact much now.
Please review and suggest changes or merge.
Thank you.

In app.js, use the .env variable REACT_APP_CURRENT_VERSION instead of "0.5.x" when returning the dummy firmware. Everything else looks good, thanks!

Done!


fix(decorators): export decorators

Decorators were not being exported. So when I was following the readme to simply decorate my message-handling methods, there was of course an error. I can skirt around this by referencing the file-structure of the module for now, but this PR is a permanent change to fix this.

Thanks!


Fixes #1483 - fix reference for Queue empty exception class

Looks like @ZhuozhaoLi got basically the same fix for this submitted as #1482 before your PR - I've merged #1482.


tor-insecure: new service, based on tor

continue #48625
this is just a quick draft
is this a good way to refactor the tor service? better ideas?
i moved nixos/modules/services/security/tor.nix to tor-service-factory.nix
and i call the "tor-service-factory" from tor.nix and tor-insecure.nix
Motivation for this change
copy-paste from #48625

If the configuration doesn't enable Tor client features,
generate a configuration file that explicitly includes this disabling
to allow such non-anonymous hidden services to be created

both should be possible
i mean, i want to use tor to both

provide non-anonymous hidden services with HiddenServiceSingleHopMode = true
have an anonymous system-wide tor client with the usual 3 hops

seems like this requires running two separate tor daemons

One tor daemon could run in a nixos container with port forwarding.

sounds too complicated ...
the non-anonymous tor daemon is not listening (SocksPort 0)
but needs to connect to local servers (http server, etc)
it just needs a separate DataDirectory, for example /var/lib/tor_insecure
and HiddenServiceDir /var/lib/tor_insecure/onion/service-name
to clarify, the "insecure" mode of tor is useful for NAT punching

Non Anonymous Onion Encryption and NAT Traversal
Can NAT traversal be Tor's killer feature?
Direct onion services over Tor

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

todo: force separation of the "secure" tor service and tor-insecure
tor service forced config
HiddenServiceSingleHopMode = false;
HiddenServiceNonAnonymousMode = false;

tor-insecure service forced config
HiddenServiceSingleHopMode = true;
HiddenServiceNonAnonymousMode = true;
SocksPort = 0;

user was banned https://github.com/NixOS/moderation/blob/main/moderation-log.md


Passing custom environment variables to Docker

Hi!
I am trying to ensure that a custom environment variable that I use at build-time is passed into manylinux builds, but it seems we currently only corner-case certain env var prefixes such as CARGO_* and RUST*.
My use-case here is to create a Rust const that is dependent on environment variables at compilation-time.
My workaround is currently to add a RUST* prefix to my environment variable to circumvent this: https://github.com/Eventual-Inc/Daft/pull/432
However, it would be useful if maturin-action supported the specification of environment variables that are to be passed into the docker container at build-time!
A previous issue that was filed for this same request: #69

Added docker-options option in #135, please try
docker-options: -e ENV_VAR_NAME

or
docker-options: -e ENV_VAR_NAME=value


Data, data, data

I put together tree diagrams. Doing so raised a couple thoughts/questions.

When should we ask about location? This should be the very first question, or more specifically, a location should be the "trigger" word that launches the questionnaire. We want users to answer the least number of questions, and it seems impossible to recommend a resource without knowing using location. Plus, the suburbs have only two options, so users in the suburbs need only answer a couple questions, before reaching a state of termination. (We'd need to know they were in the suburbs to build out the shorter paths.)

I see three question-and-answer paths that lead to two sources:
18-24; slept at my home or that of a friend/family; unstable housing situation
Coordinate Entry Access points
Youth Diversion Supports (the logic table marks this one as being prioritized before CEA points)
18-24; slept outside
Coordinate Entry Access
Chicago Shelter System
25+; slept outside
Coordinate Entry Access
Chicago Shelter System

Or should those last two paths only recommend CES points? And the Chicago Shelter System should be recommend to users who pass through all states without receiving a specified source, right?

city_paths.pdf
suburban_paths.pdf

Logic table data
'The Bad Idea for “Fixing” Homelessness That Just. Won’t. Die.' (Article shared by @derekeder!)
This article (however emotionally charged) raises a good question about the CSH data - we assume that users are individuals. Is this always the case? Or should we ask a question about children? spouses? pets? Would the recommended resource change, if say, an user had a kid or a dog?

Great question! Let's be sure to raise this with the client.
On Tue, Aug 14, 2018 at 3:14 PM Regina Compton<EMAIL_ADDRESS>wrote:

Logic table data
'The Bad Idea for “Fixing” Homelessness That Just. Won’t. Die.'
https://medium.com/@candacefaber/the-bad-idea-for-fixing-homelessness-that-just-wont-die-7f26a1f3ac4f
(Article shared by @derekeder https://github.com/derekeder!)
This article (however emotionally charged) raises a good question about
the CSH data - we assume that users are individuals. Is this always the
case? Or should we ask a question about children? spouses? pets? Would the
recommended resource change, if say, an user had a kid or a dog?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/datamade/csh-platform/issues/3#issuecomment-413000506,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAgxbWD3HA1kWVrUupLJZMA1Txv2dDwpks5uQy98gaJpZM4VwnU5
.

No, the survey outcome does not change if the user has a kid, spouse, or dog.


New package: Stardates v1.0.0

Registering package: Stardates
Repository: https://github.com/chrisoei/Stardates.jl
Created by: @chrisoei
Version: v1.0.0
Commit: bbcb052419225c4c019e478ea3202d1d99acede5
Reviewed by: @chrisoei
Reference: https://github.com/chrisoei/Stardates.jl/commit/bbcb052419225c4c019e478ea3202d1d99acede5#commitcomment-35530819

Your new package pull request met all of the guidelines for automatic merging.
I will automatically merge this pull request after the mandatory waiting period has elapsed.
If you want to prevent this pull request from being auto-merged, simply leave a comment.
(If you want to post a comment without blocking auto-merging, you must include the text [noblock] in your comment.)

[noblock]


Implement pc_actions_ROL and corresponding tests

This PR implements the function pc_actions_ROL in simuvex/vex/ccall.py and the corresponding tests in function test_ccall() in file tests/test_vex.py as discussed in #13

Success!


ENH: ability to pass ef_search in query

Issue #, if available:
#116
Description of changes:

Add ef_search field into KNNQueryBuilder and KNNQuery and modified the relevant APIs
Pass in ef_search from query when getting/loading index for query
Modified both UnitTest and IntegrationTests to cover the enhancement

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

TODO: device test cases for dynamic ef_search.

@vamshin Do not have a good strategy to test Cache rebuilt on updating index setting. Otherwise the PR is ready for another look.

@jmazanec15 @vamshin Ready for another look.
I added testKNNStatsAfterUpdateIndexSetting to cover cache miss after update index setting. It is a little different from vamshin's proposal as I noticed that KNN stats(misses, hits) gets reset after updating index setting. But it still makes sure the first search after update misses.


vulkan: Update volk, headers and glslang to 1.3.204 (take 2)

Redo of #57948 which was reverted due to #57978.
It's still the same commits and thus has the same bug for now, so opening as draft.
We need to fix our push constants apparently, as per @clayjohn:

Looks like its intended behaviour. https://github.com/KhronosGroup/glslang/pull/2810
We should go through our push constants and remove the binding assignment.

Let's merge sure to get this tested on all desktop OSes at least this time. I'm still puzzled that #57948 was working OK for both @bruvzg on macOS and me on Linux... but I don't see any other PR merged in between that would trigger this bug.
Edit: Seems we were both fooled by the shader cache which didn't update properly, and still gave us old, working shaders.

It looks like specifying a binding for push_constants is not supported behaviour and now glslang is erroring when it is used. What we need to do is go through our .glsl files and removing the binding assignment.
For example, this:
https://github.com/godotengine/godot/blob/cf13f8af648c075282e596744e48d60c25fae283/servers/rendering/renderer_rd/shaders/blit.glsl#L7
should become:
layout(push_constant, std140) uniform Pos {

That seems to fix it for me, I'll add this change as first commit in this PR.

Tested on my system with both Intel HD 630 and Radeon RX Vega M, the current state of this PR seems to work fine.
I tried both with and without res://.godot in a project, and both with and without ~/.local/share/godot/shader_cache.
Marking as ready, though it still needs more testing results before we can merge it.

Tested on Windows 10 with RTX 2080: the editor works normally without any error

Tested on my desktop system (Windows 10 build 19044.1526, RTX 3060Ti driver version 511.65) and on my notebook (Linux[OpenSUSE Tumbleweed] with kernel 5.16.5, MX150 driver version 470.103). It seems to work fine, before and after the deletion of the shader cache.
The test project I used contains several fog volumes and nearly all lighting effects(SDFGI, SSGI, SSAO, etc.) are enabled, so it should cover a large part of what could potentially break :)


control when suspending functions emit - is this possible?

AFAIK, there's no way to run an assertion after a suspending function is invoked but before it returns (resumes with a return value). Is this possible?
Currently, mocking suspending functions return immediately when invoked. I would like to control the timing of when they resume with a value. Is this possible? Use case for loading some async data is:

suspending function is invoked (without returning yet)
assert that loading indicator is shown
allow previously invoked suspending function to resume with value
assert that loading indicator is no longer shown

Does MockitoK have a way to do this?

@ZakTaccardi Did you try the solution @bohsen proposed? Is it sufficient enough?

I actually haven't had the time to try it yet. I do plan on giving it a spin and reporting back here


enable release pipeline on master

enable release pipeline on master
There are 3 stages

Build:  which runs when a PR changes (daily|monthly)/build/.*
Test:  which runs when a PR changes (daily|monthly)/test/.*
Release:  which runs when a PR changes (daily|monthly)/release/.*

They are chained by using post submits which trigger a PR for the next stage.
build -> test -> release

@rkpagadala: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-upgrade-tests-1.0.1.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-upgrade-tests-1.0.1

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

@rkpagadala: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-upgrade-tests-1.0.1.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-upgrade-tests-1.0.1

prow/daily-upgrade-tests-1.0.2.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-upgrade-tests-1.0.2

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

@rkpagadala: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-upgrade-tests-1.0.1.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-upgrade-tests-1.0.1

prow/daily-upgrade-tests-1.0.2.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-upgrade-tests-1.0.2

prow/daily-unit-tests.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-unit-tests

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

@rkpagadala: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-upgrade-tests-1.0.1.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-upgrade-tests-1.0.1

prow/daily-upgrade-tests-1.0.2.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-upgrade-tests-1.0.2

prow/daily-unit-tests.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-unit-tests

prow/istioctl-tests.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test istioctl-tests

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

@rkpagadala: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-upgrade-tests-1.0.1.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-upgrade-tests-1.0.1

prow/daily-upgrade-tests-1.0.2.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-upgrade-tests-1.0.2

prow/daily-unit-tests.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-unit-tests

prow/istioctl-tests.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test istioctl-tests

prow/daily-e2e-pilot-no_auth.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-e2e-pilot-no_auth

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

@rkpagadala: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-upgrade-tests-1.0.1.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-upgrade-tests-1.0.1

prow/daily-upgrade-tests-1.0.2.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-upgrade-tests-1.0.2

prow/daily-unit-tests.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-unit-tests

prow/istioctl-tests.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test istioctl-tests

prow/daily-e2e-pilot-no_auth.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-e2e-pilot-no_auth

prow/daily-e2e-bookinfoTests.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-e2e-bookinfoTests

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

@rkpagadala: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-upgrade-tests-1.0.1.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-upgrade-tests-1.0.1

prow/daily-upgrade-tests-1.0.2.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-upgrade-tests-1.0.2

prow/daily-unit-tests.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-unit-tests

prow/istioctl-tests.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test istioctl-tests

prow/daily-e2e-pilot-no_auth.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-e2e-pilot-no_auth

prow/daily-e2e-bookinfoTests.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-e2e-bookinfoTests

prow/daily-e2e-simpleTests.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-e2e-simpleTests

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

@rkpagadala: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-upgrade-tests-1.0.1.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-upgrade-tests-1.0.1

prow/daily-upgrade-tests-1.0.2.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-upgrade-tests-1.0.2

prow/daily-unit-tests.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-unit-tests

prow/istioctl-tests.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test istioctl-tests

prow/daily-e2e-pilot-no_auth.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-e2e-pilot-no_auth

prow/daily-e2e-bookinfoTests.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-e2e-bookinfoTests

prow/daily-e2e-simpleTests.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-e2e-simpleTests

prow/daily-e2e-mixer-no_auth.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-e2e-mixer-no_auth

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

@rkpagadala: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-upgrade-tests-1.0.1.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-upgrade-tests-1.0.1

prow/daily-upgrade-tests-1.0.2.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-upgrade-tests-1.0.2

prow/daily-unit-tests.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-unit-tests

prow/istioctl-tests.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test istioctl-tests

prow/daily-e2e-pilot-no_auth.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-e2e-pilot-no_auth

prow/daily-e2e-bookinfoTests.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-e2e-bookinfoTests

prow/daily-e2e-simpleTests.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-e2e-simpleTests

prow/daily-e2e-mixer-no_auth.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-e2e-mixer-no_auth

prow/daily-e2e-dashboard.sh
0ddb3c15bb3ee5f97989b92ca177900c2597c51e
link
/test daily-e2e-dashboard

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


Allow disabling swipe gesture on OSX

I view a lot of code snippets and/or other "wide" content in Zulip, and find its default behavior on OS X of window navigation when I use a two-finger swipe to scroll through content in the current window annoying. Enough so that I created the attached script to remove https://github.com/zulip/zulip-electron/blob/master/app/main/preload.js#L13-L14 from my local app when/if my local Zulip app is updated.
zulip-disable-swipe.sh.txt

@akashnimare do you think we should make this a setting, or just remove that code for everyone?

This is an experimental feature for macOS; if nobody seems to use it we could just remove it for now.

Okay, removing this experimental feature for now. Will make it optional once we have the centralized preference settings .


[Feature]: Multi-arch in-place update through instance manager injection for multiple architectures

Is there an existing issue already for this feature request/idea?

[X] I have searched for an existing issue, and could not find anything. I believe this is a new feature request to be evaluated.

What problem is this feature going to solve? Why should it be added?
Currently, the In-place update of the instance manager requires all pods (operators and operands) to run on the same architecture, meaning that this feature doesn't yet support hybrid/mixed architectures Clusters (e.g running a given Cluster pods on either amd64 or arm64 nodes).
At present, the operator contains a validation check preventing in-place updates in a cluster containing mixed architectures.
Describe the solution you'd like
Starting with 4ee639f: we've enhanced our multi-arch images to store the binary of each supported architecture in every architecture image.
Following up with 87e6591: we've added in the operator a catalog of all the available architectures contained in the running operator image, together with a new Cluster status field named AvailableArchitectures which lists all the architectures available for a target Cluster (each paired with their unique hash).
This patch aims to start using the matching architecture's binary during in-place updates, enabling the operator to inject the correct binary in any Pod having a supported architecture.
The check preventing in-place updates in a cluster containing mixed architectures will be also relaxed.
Describe alternatives you've considered
Not provided.
Additional context
No response
Backport?
Yes
Are you willing to actively contribute to this feature?
Yes
Code of Conduct

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

One proposal which is related to this, I believe: Can we make the warning that's currently shown in the logs a bit clearer? I've just set up my first cluster on a mixed ARM64/AMD64 k8s cluster and got this warning:
{"level":"info","ts":"2024-02-24T11:09:50Z",
"msg":"Warning: mismatch architecture between controller and instances. This is an unsupported configuration.",
"controller":"cluster","controllerGroup":"postgresql.cnpg.io","controllerKind":"Cluster","Cluster":{"name":"keycloak-pg-cluster","namespace":"testsetup"},"namespace":"testsetup","name":"keycloak-pg-cluster","reconcileID":"e4cf6ddc-b4a2-435c-8846-65c8a33ae77b","uuid":"3aa05ab2-d305-11ee-b17a-3a3f9a721d3b","controllerArch":"amd64","instanceArch":"arm64","pod":"keycloak-pg-cluster-2"}

This made me think that mixed architectures are not supported in general, but if I'm reading this issue correctly, it's only related to in-place updates of the instance manager, and otherwise mixed-arch clusters are fine?
I think it would be good to only show this warning when there is a mix of architectures and in-place updates are actually enabled.

@mmeier86 Absolutely! Btw that warning is generated by the same check I've mentioned earlier, and right now it's only being used to validate if an instance manager online update (e.g in-place) would be possible.
So I agree it probably shouldn't be emitted in case a user didn't enable in-place updates of the instance manager.

This made me think that mixed architectures are not supported in general, but if I'm reading this issue correctly, it's only related to in-place updates of the instance manager, and otherwise mixed-arch clusters are fine?

That's correct, mixed architectures clusters should be working properly provided that you are using the default rolling update method to upgrade the operator. Given that each pod is terminated and recreated in a rolling update fashion, the new version of the instance manager will be downloaded by the bootstrap-controller initContainer of each pod.
I guess what's missing is that this kind of setup is not currently documented, nor we are explicitly mentioning in the doc if this is a supported scenario. So we should clarify that..

@mmeier86 Absolutely! Btw that warning is generated by the same check I've mentioned earlier, and right now it's only being used to validate if an instance manager online update (e.g in-place) would be possible. So I agree it probably shouldn't be emitted in case a user didn't enable in-place updates of the instance manager.

This made me think that mixed architectures are not supported in general, but if I'm reading this issue correctly, it's only related to in-place updates of the instance manager, and otherwise mixed-arch clusters are fine?

That's correct, mixed architectures clusters should be working properly provided that you are using the default rolling update method to upgrade the operator. Given that each pod is terminated and recreated in a rolling update fashion, the new version of the instance manager will be downloaded by the bootstrap-controller initContainer of each pod. I guess what's missing is that this kind of setup is not currently documented, nor it's explicitly being mentioned in the doc if this is a supported scenario. So yeah that should be clarified..

Amazing, I'm mixing x86_64 and ARM for cost reduction in the entire cluster. Is good to know that effort is made to support this. I think this is the only Postgresql operator that does support this scenario. I'm using node affinity to separeae workloads, but I will test remove this restriction in HA postgresql. ❤


Error when running npm install<EMAIL_ADDRESS>https://github.com/AzureAD/microsoft-authentication-library-for-js/packages/292089

Please follow the issue template below. Failure to do so will result in a delay in answering your question.

Library

[ ] @azure/microsoft-authentication-library-for-js

https://github.com/AzureAD/microsoft-authentication-library-for-js/packages/292089
Important: Please fill in your exact version number above, e.g<EMAIL_ADDRESS>Framework
msal-react-native-android-poc 1.0.5
Description
Error Message

Security

[ ] Is this issue security related?
no

Regression

[ ] Did this behavior work before?
Version: not sure

MSAL Configuration
// Provide configuration values here.
// For Azure B2C issues, please include your policies.

Reproduction steps
// Provide relevant code snippets here.
// For Azure B2C issues, please include your policies.

Expected behavior
library is not working
Browsers/Environment
Terminal. React Native

This package is only published to the Github Registry, not the npm registry, so you will need to update your npmrc to use the Github Registry for the @azuread scope: https://github.com/AzureAD/microsoft-authentication-library-for-js/blob/dev/samples/msal-react-native-android-poc-sample/.npmrc
Furthermore, this package is a proof-of-concept, and not meant for production usage.

Thanks @jasonnutter !
We have an API App Service in Azure and we created two AD Apps one is for the server-side and another is for the client (Mobile App) side.
We were unable to implement React Native AD Authentication using the Client App ID and TENANT ID from this link https://docs.microsoft.com/en-us/graph/tutorials/react-native
Which library from https://github.com/AzureAD should we utilize to accomplish React Native AD Auth users to login only within our organization using our Tenant ID?

@kkadapa You can try using MSAL for Objective C and MSAL for Android directly (which the POC does), or finding a third-party library (such as the one recommended in the tutorial).


Adapters for CLUD actions

Move CLUD actions to an adapter:
import Store from "json-api-store";

var ajaxAdapter = new Store.AjaxAdapter("http://api.example.com/v1/");
var storeA = new Store();
var storeB = new Store(adapter);

// error: no adapter
storeA.create("products", "1", {});

// all good
storeB.create("products", "1", {});

This approach makes isomorphic support possible.

This has been abandoned in favour of jumping straight to using adapters. See #2.


Adopting the run-condition-plugin

Greetings team, I want to adopt the https://github.com/jenkinsci/run-condition-plugin inorder to update the existing feature and add new functionality to the existing plugin.

Submitter checklist for adding or changing permissions

Always

[x] Add link to plugin/component Git repository in description above

For a newly hosted plugin only

[ ] Add link to resolved HOSTING issue in description above

For a new permissions file only

[ ] Make sure the file is created in permissions/ directory
[ ] artifactId (pom.xml) is used for name (permissions YAML file).
[ ] groupId / artifactId (pom.xml) are correctly represented in path (permissions YAML file)
[ ] Check that the file is named plugin-${artifactId}.yml for plugins

When adding new uploaders (this includes newly created permissions files)

[x] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[x] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[x] Make sure to @mention the users being added so their GitHub account names are known if they require GitHub merge access (see below).
[x] All newly added users have logged in to Artifactory at least once

Reviewer checklist (not for requesters!)

[ ] Check this if newly added person also needs to be given merge permission to the GitHub repo (please @ the people/person with their GitHub username in this issue as well). If needed, it can be done using an IRC Bot command
[ ] Check that the $pluginId Developers team has Admin permissions while granting the access.
[ ] In the case of plugin adoption, ensure that the Jenkins Jira default assignee is either removed or changed to the new maintainer.
[ ] If security contacts are changed (this includes add/remove), ping the security officer (currently @Wadeck) in this pull request. If an email contact is changed, wait for approval from the security officer.

There are IRC Bot commands for it

please fill out the template

@timja Sir, I have already login to the Artifactory and jira


Display GraphExplorer version in client UI

Issue 77
Description of changes:

Display the version number in the top bar of all workspaces.
The version number is extracted from the package.json version number and injected into the application through the GRAPH_EXP_VERSION variable, which can be globally accessed by the application components.

Can we remove the "V" from this? I think it looks cleaner without. Let's also add some "Release" documentation on what exactly needs to be changed in which files when we are ready to increment the release number: https://github.com/aws/graph-explorer/blob/main/additionaldocs/development.md

Looks good to me. Adding @michaelnchin for one more review. Let's merge this last, when we're ready to release.


husky_bringup package is not found in ROS2

How do I install husky_bringup package in my husky robot with ROS2? When I run ros2 run husky_bringup install, it says the package is not found

@niuhaoyu16 it is pending being synced on Friday.  You can switch to the test repo to get it now.

Need to create a workspace and source the package


Add audio source to video-viewer

I'd like to add audio source via usb microphone to video-viewer.py.
I couldn't find audio options in (https://github.com/dusty-nv/jetson-inference/blob/master/docs/aux-streaming.md).
Do you have any audio options for video-viewer?

Hi, sorry it doesn't do audio output

From: Yu Nishimura @.>
Sent: Wednesday, September 15, 2021 10:40:07 PM
To: dusty-nv/jetson-utils @.>
Cc: Subscribed @.***>
Subject: [dusty-nv/jetson-utils] Add audio source to video-viewer (#93)
I'd like to add audio source to video-viewer.py.
I couldn't find audio options in (https://github.com/dusty-nv/jetson-inference/blob/master/docs/aux-streaming.md).
Do you have any audio options for video-viewer?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/dusty-nv/jetson-utils/issues/93, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADVEGK7KABNRMCO55KEPIKTUCFKIPANCNFSM5EDXKY4A.
Triage notifications on the go with GitHub Mobile for iOShttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|dustinf%40nvidia.com|9cc290ccf4bf4499168708d978bb4c2f|43083d15727340c1b7db39efd9ccc17a|0|0|637673568097843714|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=x9CMK6jtnML%2Fxh7GSpApdp5cMKYXs4umJecp0aUns20%3D&reserved=0 or Androidhttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|dustinf%40nvidia.com|9cc290ccf4bf4499168708d978bb4c2f|43083d15727340c1b7db39efd9ccc17a|0|0|637673568097853715|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=XwDpFWPQLDykiijZ17TFMHcGOqJY8GBxrOU1Ge2rohU%3D&reserved=0.

Thanks,  I will modify your code to use audio output!


chore: bump dev dependencies and add package-lock

No change to logic.
This reduces install spam by bumping codecov to avoid depending on a
deprecated graceful-fs version, as well as adding a package-lock.json.

Codecov Report

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

@@          Coverage Diff           @@
##           master    #864   +/-   ##
======================================
  Coverage    95.8%   95.8%
======================================
  Files          34      34
  Lines        1262    1262
======================================
  Hits         1209    1209
  Misses         53      53

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update aa9d443...3166707. Read the comment docs.

LGTM


Only store most recent 255 blocks in the Blockchain object

This patch makes the spec forget blocks older than 255 blocks ago. This is all that is required in order to implement the EVM. This saves about 5GB of memory usage by the end of Frontier.
I experimented with the deque object for this, but the complexity it introduced wasn't worth it.
Cute Animal Picture

s/255/256?

We need the block hash for 256th most recent block, but we get it by looking at the parent of the 255th most recent block.


Don't log an error if API key not set

Requirements for Contributing to this repository

Fill out the template below. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
The pull request must only fix one issue at the time.
The pull request must update the test suite to demonstrate the changed functionality.
After you create the pull request, all status checks must be pass before a maintainer reviews your contribution. For more details, please see CONTRIBUTING.

What does this PR do?
Fixes https://github.com/jenkinsci/datadog-plugin/issues/42

Description of the Change
Doesn't create a client if the apikey is null, all code has been updated to handle null

Alternate Designs

Possible Drawbacks

Verification Process
tested with mvn hpi:run

Additional Notes

Release Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bug fix MUST have appropriate tests (unit, integration, etc...)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have one changelog/ label attached. If applicable it should have the backward-incompatible label attached.
[ ] PR should not have do-not-merge/ label attached.
[ ] If Applicable, issue must have kind/ and severity/ labels attached at least.

Looks covered in https://github.com/jenkinsci/datadog-plugin/pull/44

Yes indeed! Still, thanks a lot for letting us know 😁


Improved: VIEW permissions - invoice contact mechs (OFBIZ-12428)

Though there is an entity InvoiceContactMech, this cannot be accessed by user with VIEW permissions when accessing an invoice.
See:
for entity data: https://demo-trunk.ofbiz.apache.org/webtools/control/entity/find/InvoiceContactMech
test invoice: https://demo-trunk.ofbiz.apache.org/accounting/control/invoiceOverview?invoiceId=demo11001
Modified:
AccountingMenus.xml

added menu-item for contact mechs to InvoiceTabBar menu
controller.xml
added request-map and view-map for request invoiceContactMech
InvoiceScreens.xml
added screen for invoice contact mech(s)
InvoiceForms.xml
added grid to display contact mech details

Hi Pierre,
Did you check locally? It seems we have a test issue.

HI Jacques,
Yes I did. Are you referring to the 'Java CI with Gradle/build (pull_request) failing check mentioned above?
It seems there is something weird going on there.
I did the obligatory ./gradlew cleanAll loadAll testIntegration. Beyond the multitude of errors we see all the time with running the integration test, I could not find anything wrong (may have overlooked something).
Subsequently I did the ./gradlew ofbiz in my dev setup, and activated the invoiceContactMech request for invoice="demo11001", no records shown. After adding an InvoiceContactMech record via entity management in webtools for the same invoice, I refreshed the aforementioned screen and data appears. So everything seems ok.

Hi @JacquesLeRoux,
You have anymore insights?

Hi Jacques,
It appears everything is ok now. Was the missing label the culprit?


Display a more meaningful error message when the song cannot be played because playing the video on other websites has been disabled by the uploader

For example this song cannot be played in Streamus and Streamus could show a more meaningful error message, maybe accompanied by a button to listen to the song on YouTube (by opening a new YouTube tab).

Hey,
Duplicate of #293
I've added a filter in for search results to make sure songs which can't be played in embedded players aren't shown in search results, but I agree the dialog should be better, too. YouTube just doesn't give me a lot of info on why the song could not be played. Just an error code which happens for a lot of different reasons.


small grammar fix

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


very old runtime nodejs10.x

@artemrys   Hello team , application deployment is failing due to very old nodejs runtime , Please update to nodejs18.x  @theletterf

@jainvijay01 hey, I am not part of the team who owns this repository and it seems it did not get an update for 4 years.


Read TF version from poetry.lock

The problem:
We were using sed-style to read TF version from pyproject.toml, which leads us to problems when the version constraints aren't cover in the sed command. We need to find a way to read TensorFlow version from poetry.lock directly.
Proposed changes:
Use python's toml library to parse and read TensorFlow version from poetry.lock.

/modeltest
include:
 - dataset: ["Carbon Bot"]
   config: ["Sparse + DIET(bow) + ResponseSelector(bow)"]


Add navigate callback

needs docs still

Coverage increased (+1.4%) to 94.366% when pulling 43c7426ed338bce37345914819c06d735fc75be6 on add-navigate-callback into 873c76a8a901bd0e7ce665e62e4aac094a3c0809 on master.

Coverage increased (+1.4%) to 94.366% when pulling a13d0b50985d406b02e45c5e1d26aa32222d53af on add-navigate-callback into 873c76a8a901bd0e7ce665e62e4aac094a3c0809 on master.

Coverage increased (+1.4%) to 94.366% when pulling a13d0b50985d406b02e45c5e1d26aa32222d53af on add-navigate-callback into 873c76a8a901bd0e7ce665e62e4aac094a3c0809 on master.


Add @Test annotation to JUnit4 example test

Without this the test doesn't compile, build fails with "java.lang.Exception: No runnable methods"

Thanks for your pull request.  It looks like this may be your first contribution to a Google open source project, in which case you'll need to sign a Contributor License Agreement (CLA).
:memo: Please visit https://cla.developers.google.com/ to sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll verify.  Thanks.

If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address.  Check your existing CLA data and verify that your email is set on your git commits.
If you signed the CLA as a corporation, please let us know the company's name.

I signed it!

Many thanks!


connection.spec.js tests doesn't use different connection options

It doesn't use options supplied by helper.allTests - all test runs use default settings

My bad... not broken

although point 1 still seems to apply. Unless I am missing something.

The tests are redundant but otherwise this is nothing bad. The connection.spec does not need any provided options from that function. As long as the test suite does not take very long it's not of big concern in my opinion. But feel free to remove the redundancy if you want to.

Redundancy is fine. I was a bit confused originally.
By redundancy do you mean that it essentially runs the test with the same options multiple times?
The only confusing thing here that it creates impression (from the test log) that it uses multiple connection options, so it would be cleaner not to use helper.allTests here.
But I agree it is not critical.


[stable/airflow] Use ingress.web.host value instead of hardcoded loca…

…lhost:8080
For AIRFLOW__WEBSERVER__BASE_URL environment variable
Signed-off-by: Olivier Monnier<EMAIL_ADDRESS>
Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @gsemet Please review

/lgtm
nice one


same query, different results

Hi ranaroussi,
Firstly, thanks for yfinance. It's been helpful in the past.
I noticed an interesting bug today. Running the same query (below) in two different instances produced differing values. They were run simultaneously on colab notebooks.
yf.Ticker('AMZN').info.get('marketCap')
yields...

Notebook A:<PHONE_NUMBER>2
Notebook B:<PHONE_NUMBER>272 (correct)

Notebook A is calling the data as part of a list of tickers whereas B is just 'AMZN'.  In Notebook A, I double-checked the code and ran the query just for 'AMZN' (as above) and kept pulling<PHONE_NUMBER>2.
Any ideas what's going wrong here?
Thanks

Is marketCap the only difference in info? If there's a lot of differences then indicates Yahoo returned bad data.

good point. here's a list of the differences when pulling .info
There are 84 fields of the 158 that are different

index
a
b

targetLowPrice
104
110

targetMedianPrice
170
172

earningsGrowth

numberOfAnalystOpinions
51
48

targetMeanPrice
169.37
171.18

recommendationMean
1.8
1.7

isEsgPopulated
true
false

underlyingSymbol

NaN

underlyingExchangeSymbol

NaN

headSymbol

NaN

uuid
261fd26b-0151-3813-b0d0-97e4ed4c6505
NaN

annualHoldingsTurnover

enterpriseToRevenue
3.029
2.761

beta3Year

enterpriseToEbitda
27.968
25.497

52WeekChange

-0.27699292

morningStarRiskRating

forwardEps
70.96
2.31

revenueQuarterlyGrowth

sharesOutstanding
507148000
10187599872

fundInceptionDate

annualReportExpenseRatio

totalAssets

bookValue
205
12.904

sharesShort
81630928
71375116

sharesPercentSharesOut
0.008
0.0069999998

fundFamily

heldPercentInstitutions
0.60442
0.59828997

trailingEps
52.562
1.09

lastDividendValue

SandP52WeekChange

-0.122783005

priceToBook
0.5960488
9.469157

heldPercentInsiders
0.09864
0.09859

yield

shortRatio
1.18
1.4

sharesShortPreviousMonthDate
1655251200
1659052800

floatShares
9171244349
9172874358

beta
1.246487
1.3332

enterpriseValue
1471660556288
1341667278848

threeYearAverageReturn

lastSplitDate
936230400
1654473600

lastSplitFactor
2:1
20:1

legalType

morningStarOverallRating

earningsQuarterlyGrowth

priceToSalesTrailing12Months
0.12753275
2.5618806

dateShortInterest
1657843200
1661904000

pegRatio
7.17
34.08

ytdReturn

forwardPE
1.7219561
52.896107

lastCapGain

shortPercentOfFloat
0.0092
0.008

sharesShortPriorMonth
112243921
67798677

category

fiveYearAverageReturn

trailingAnnualDividendYield

0

volume24Hr

navPrice

trailingAnnualDividendRate

0

expireDate

algorithm

dividendRate

exDividendDate

circulatingSupply

startDate

trailingPE
2.3246832
112.100914

regularMarketVolume
47645938
47543613

lastMarket

maxSupply

openInterest

marketCap
61968416768
1244822896640

volumeAllCurrencies

strikePrice

askSize
1000
800

volume
47645938
47543613

fromCurrency

fiveYearAvgDividendYield

dividendYield

bidSize
900
1200

lastDividendDate
NaN

impliedSharesOutstanding
NaN
0

toCurrency
NaN

coinMarketCapLink
NaN

trailingPegRatio
NaN
4.9641

colab must be caching the bad data, can you clear it?

re-ran it today, still bogus data in notebook A.
Is it a thing for yahoo to spam bad data to clients?

Only time I've encountered this is when I am spamming Yahoo (looping over many tickers). But after some time AFK a repeat query returns good data. So it is odd for you to still be getting bad data, unless your notebook is spamming.
Are you spamming?

I don't believe I am spamming. I am running through a list of the same 30 tickers each time I test notebook A. I'll give your suggestion a shot though. Thanks!

Actually, before you use that code can you try using latest main instead? It has optimised info creation, might help. Instructions: https://github.com/yfinance-fork-team/yfinance-x/discussions/14

Since my last response those fixes have been pushed into PIP. Can you try latest release?

@mreous714 Please provide a working code snippet we can use to reproduce your issue.

stepped away from this project for a bit due to the issues I have been having. tried again today and same issue.
table=pd.read_html('https://en.wikipedia.org/wiki/List_of_S%26P_500_companies')
df = table[0]
stocks=list(df.loc[:100,'Symbol'])
stocks.sort()
universe_data = dict.fromkeys(stocks)
shares_keys = ['sharesOutstanding','floatShares','marketCap','regularMarketPreviousClose','regularMarketVolume']
for x in stocks:
temp_stock = yf.Ticker(x)
shares_val=[temp_stock.info.get(key) for key in shares_keys]
universe_data[x] = dict(list(zip(shares_keys,shares_val)))
universe_data['AMZN']
yf.Ticker('AMZN').info.get('sharesOutstanding')

I should add I pip installed the latest yfinance version via colabs
Successfully installed requests-2.28.1 yfinance-0.1.89

Yahoo does not like many requests for info[], a loop of 100 will trigger them. I don't know why they return bad data instead of just blocking you (temporarily).
Try a combination of Ticker.history() and Ticker.shares. If that helps, then try experimental Ticker.shares_full() for up-to-date share count (see #1133).


Can't define OutputDataConfig for TrainingStep

It doesn't look like it's possible to specify the output path for the model artifacts in the TrainingStep class. You can only use the default output directory or define it in the estimator object before you pass it to the TrainingStep. In either case it doesn't look like its possible to set the output path from the execution input or the output of a prior task.

+1 on this.
One could potentially work around this by overwriting the sagemaker default bucket (https://github.com/aws/sagemaker-python-sdk/issues/1402), but this doesn't allow you to specify a path, only a different bucket.
As the API integration supports creating the OutputDataConfig (https://docs.aws.amazon.com/sagemaker/latest/APIReference/API_CreateTrainingJob.html), it makes sense to expose it via this SDK directly.

Implemented in  #142  which has now been merged 🎉


pdns-recursor: 4.8.3 -> 4.8.4

Description of changes
Version bump
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested via pdns-recursor.tests
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

https://www.openwall.com/lists/oss-security/2023/04/04/1

CVE-2023-26437

Please add Fixes: CVE-2023-26437 into the commit message, so anyone can easily  grep for it.


RFC: Breaking API Change: drivers: can: rework support for manual bus-off recovery

Since all CAN controllers drivers seem to support automatic recovery (for any future drivers for hardware without this hardware capability this can easily be implemented in the driver), change the Zephyr CAN controller API policy to:

Always enable automatic bus recovery upon driver initialization, regardless of Kconfig options. Since CAN controllers are initialized in "stopped" state, no unwanted bus-off recovery will be started at this point.

Invert and rename the Kconfig CONFIG_CAN_AUTO_BUS_OFF_RECOVERY, which is enabled by default, to CONFIG_CAN_MANUAL_RECOVERY_MODE, which is disabled by default. Enabling CONFIG_CAN_MANUAL_RECOVERY_MODE=y enables support for the can_recover() API function and a new manual recovery mode (see next bullet). Keeping this guarded by Kconfig allows keeping the flash footprint down for applications not using manual bus-off recovery.

Introduce a new CAN controller operational mode CAN_MODE_MANUAL_RECOVERY. Support for this is only enabled if CONFIG_CAN_MANUAL_RECOVERY_MODE=y. Having this as a mode allows applications to inquire whether the CAN controller supports manual recovery mode via the can_get_capabilities() API function and either fail or rely on automatic recovery - and it allows CAN controller drivers not supporting manual recovery mode to fail early in can_set_mode() during application startup instead of failing when can_recover() is called at a later point in time.

RFC: #69459

RFC ready for review.

Look good to me.
But I see that nxp s32 canxl driver is missing capability for manual recovery mode. Please update this and check this in other drivers.@henrikbrixandersen
https://github.com/zephyrproject-rtos/zephyr/blob/8992d05b0fde9c8437a0191a6d4e973634ebf918/drivers/can/can_nxp_s32_canxl.c#L153

Look good to me. But I see that nxp s32 canxl driver is missing capability for manual recovery mode. Please update this and check this in other drivers.@henrikbrixandersen

Nicely spotted, thanks. I have updated the patch.

Rebased on main to resolve merge conflicts in doc/releases/migration-guide-3.7.rst and drivers/can/can_stm32_bxcan.c.


App Transport Security error

Hi Dalton, I have problems to make a http request in order I got this messagge
App Transport Security has blocked a cleartext HTTP (http://) resource load since it is insecure. Temporary exceptions can be configured via your app's Info.plist file.
error: The resource could not be loaded because the App Transport Security policy requires the use of a secure connection.
Thanks for your help

Sorry for issues. I've solved nice tool


getFileHandle(create: true) throws NotFoundError in a valid direcotry.

 file_system_access_api:
    dependency: "direct main"
    description:
      name: file_system_access_api
      url: "https://pub.dartlang.org"
    source: hosted
    version: "1.0.1"

$ flutter --version
Flutter 3.3.10 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 135454af32 (2 days ago) • 2022-12-15 07:36:55 -0800
Engine • revision 3316dd8728
Tools • Dart 2.18.6 • DevTools 2.15.0

Sorry for my misunderstanding.
I thought that getFileHandle returns null when the file dones't exist.

Thank you for pointing this out.
It is indeed not clear, both getFileHandle and getDirectoryHandle returns a nullable type. Which is not expected according to JavaScript API, as there is a special NotFoundError thrown in such case.
I re-open to fix this and make return type non-nullable.


Windows 10/11 context menu editor

Description of the new feature / enhancement
As the title said, I suggest to add Windows 10/11 context menu editor
Scenario when this would be used?
Allow user to add/remove/regroup Windows 10/11 context menu items
Supporting information
No response

Hi Mahmoud,
Couldn't agree more. I did not see your ticket before I added mine: https://github.com/microsoft/PowerToys/issues/31721
Team PowerToys, feel free to merge the two requests.


CreateUser method doesn't validate email field

It could cause strange string added in database

Create does call validate with the UserValidator which is responsible for email validation
https://github.com/aspnet/Identity/blob/dev/src/Microsoft.AspNetCore.Identity/UserManager.cs#L422


SECRET_FILE_SIZE does not work as expected

Description
With SECRET_FILE_SIZE=10, a file upload of a 10MB file fails.
Relevant log output:
{"level":30,"time":1692055043495,"pid":19,"hostname":"hemmelig","reqId":"req-9","res":{"statusCode":413},"err":{"type":"FastifyError","message":"Request body is too large","stack":"FastifyError: Request body is too large\n    at rawBody (/home/node/hemmelig/node_modules/fastify/lib/contentTypeParser.js:207:16)\n    at ContentTypeParser.run (/home/node/hemmelig/node_modules/fastify/lib/contentTypeParser.js:167:5)\n    at handleRequest (/home/node/hemmelig/node_modules/fastify/lib/handleRequest.js:41:33)\n    at runPreParsing (/home/node/hemmelig/node_modules/fastify/lib/route.js:528:5)\n    at next (/home/node/hemmelig/node_modules/fastify/lib/hooks.js:168:7)\n    at handleResolve (/home/node/hemmelig/node_modules/fastify/lib/hooks.js:185:5)","name":"FastifyError","code":"FST_ERR_CTP_BODY_TOO_LARGE","statusCode":413},"msg":"Request body is too large"}
{"level":30,"time":1692055043496,"pid":19,"hostname":"hemmelig","reqId":"req-9","res":{"statusCode":413},"responseTime":3.1347219944000244,"msg":"request completed"}
{"level":30,"time":1692055073172,"pid":19,"hostname":"hemmelig","reqId":"req-a","req":{"method":"GET","url":"/","hostname":"localhost:3000","remoteAddress":"<IP_ADDRESS>","remotePort":59382},"msg":"incoming request"}

Screenshots
No response
Additional information
I noticed, that a SECRET_FILE_SIZE value of 19 or higher is needed, for a 10MB file upload.

Yes, this might be due to the file being encoded in the frontend before being shipped to the server, so it is kind of a bug, but not really. I think it should be documented better.

I think you are right. When uploading the 10MB test file, the transferred bytes are way more than 10MB.
From the HTTP request headers:
Content-Length: 18642131
Content-Type: application/json

Uploading the file json encoded seems to be the cause. I looked at other solutions for comparison.
filebrowser uses Content-Type: application/octet-stream. When uploading the 10MB test file to filebrowser, only 10MB are transfered (unencoded).

It should not be due to the json body, but due to it is being encrypted before shipped to the backend. Here is how it works: https://github.com/HemmeligOrg/Hemmelig.app/issues/163#issuecomment-1465126691


Ingest Zip Files via REST Api

VectorFlow should support uploading zip files, unzipping them and embedding all eligible documents inside.

There is no longer a need to send zip file since VectorFlow now supports multi-file ingestion - https://github.com/dgarnitz/vectorflow/pull/82


should record events during pod/node was watched, pod scheduled/deleted and failed tasks

as title described, events should be recorded to let the users know what happens during one scheduling task:
1, event for one pod watched, task submitted to firmament, failed/successful tasks from firmament and the reasons
2, event for on node watched/deleted

event for pod supported by #138


Overview of needed quality control information

This is a summary issue to list the quality control information piccr is supposed to deliver; both in the stand-alone version as well as when used along with Cpt-Picarr.
These (and more detailed) information are also available from the dataProcessingGuide.Rmd.
Quality control of the measured and processed data is requested to be possible with piccr on three levels: on (1) a per-injection basis, (2) a per-sample basis, and (3) a per-file basis.
Per-injection basis
On this level, the following quality control information is obtained directly from the raw Picarro measurement files:

standard deviation of the injection value for each measured isotopic species from the measurement integration time in the cavity;
water vapour level (mean and standard deviation) during the injection.

Per-sample basis
The final isotopic value of a sample or standard is obtained from averaging across a certain (settable) number of injections with the following quality control information:

the mean water vapour level and its standard deviation for this sample/standard;
the standard deviation of the mean isotope value for each isotopic species of this sample/standard;
the deviation from the true isotope value (only for standards).

Per-file basis
On the per-file basis, piccr should provide the following quality control information:

the specific deviation from the true value for the independent quality control standard;
the root mean square deviation across the deviations of all measured standards from their true values;
the pooled standard deviation across all measured samples and standards;
quality control information from the memory correction process (memory coefficients: mean values and values for each analysed standard);
quality control information from the calibration process (such as standard deviation of residuals, p-value of regression) [tbd];
quality control information from the drift correction process (drift slopes, p-value of regression) [tbd].

Definitions
The root mean square deviation is defined as

where the delta term is the deviation for standard i from its true value and k ist the number of standards.
The pooled standard deviation is the square root of the pooled variance which is defined as

where \sigma_i is the standard deviation of the mean isotope value for vial i from averaging across n_i injections, and N is the total number of analysed vials.
Measurement uncertainty for project
The overall measurement uncertainty for a given measurement project, which contains M single measurements (i.e. Picarro files), can then be assessed with the root mean square deviation of the quality control standards,

where the delta term is the deviation for measurement file i of the quality control standard from its true value (Note: if each measurement file has several control standards, one can use each file's root mean square deviation from these control standards here).

This issue is related to:
#15
EarthSystemDiagnostics/cpt-picarr#39
EarthSystemDiagnostics/cpt-picarr#13
EarthSystemDiagnostics/cpt-picarr#12
EarthSystemDiagnostics/cpt-picarr#11

@thomas-muench Thank you for putting together this detailed summary.
It would be great if you could be involved in the implementation. Maybe you can take care of the picrr side of things (making sure that piccr outputs all required information). Then I could work on integrating the output into cpt. picarr. Does that work for you?

We decided to first make a short meeting to set up a general piccr output structure for the quality control information. Then I will work on the piccr implementation of this output, and @twollnik will prepare cpt-picarr to handle it.

We decided on the following general output structure:
Output for individual data set
Data output
The measurement data output will include the following data frames:

raw: the original measurement data before any processing was done;
memoryCorrected: the data after applying the memory correction;
calibrated: the data after applying only a calibration using first-block standards;
calibratedAndDriftCorrected: the data after applying a linear drift correction and a calibration using first-block standards, or after applying a double calibration (with inherent drift correction);
processed: the final data from averaging across n injections.

Quality control output
The quality control output, as outlined above, splits into information delivered along with the raw or processed data, and into separate information:

data-delivered:

water vapour level information
SD of mean isotope values

separate information:

deviationsFromTrue: data frame with the deviations from the true values for all measured standards = columns Identifier 1, block, d18O_measured, d18O_true, d18O_deviation and the same last three columns for dD
qualityControl: named numeric vector with the deviation from the true value for the quality control standard for d18O and dD
rmsd.DeviationsFromTrue: named numeric vector with the rmsd of DeviationsFromTrue for d18O and dD
pooledStdDev: named numeric vector with the pooled standard deviation for d18O and dD for the data set
memoryCoefficients: data frame with memory coefficients = mean and indivdiual values for each analysed standards = columns Inj No, mean, <standard-name1, ..., each for d18O and dD
calibrationParams: data frame with columns block = standard block used for calibration, pvalue = p-value of the calibration regression, d18O_RMSDofResiduals, dD_RMSDofResiduals = RMSD of calibration regression residuals, d18O_Slope, dD_Slope, d18O_Intercept, dD_Intercept =  slope and intercept of calibration for both isotope species, timeMean = mean measurement time since start for this block
[only for calibration_method=1] driftParams: data frame for drift parameters with columns variable = name of the standard used for the estimate or the mean estimate, d18O_alpha, dD_alpha = estimated drift rates, pvalue = p-value of the linear drift regression, d18O_RMSDofResiduals, dD_RMSDofResiduals = RMSD of drift regression residuals (p-value and RMSD values are NA for the mean estimate)

Output for M data sets
The output for M processed data sets is a list of length M, where each list element i is a list containing all of the above output.
@twollnik Please have a look over this proposed structure and tell me if you are fine with it or if any changes (e.g. variable names) are necessary from your point of view.

@thomas-muench Thank you for taking the time to write this. I think that you have captured everything that we talked about (and more). I have one question and some remarks.
The question: Under quality control output >> seperate information you mention named vectors a few times. What names would you choose? (e.g. names(qualityControl) equals what?)
The remarks:

I suggest adding a component name that contains the file name of the raw dataset. That way we can output the actual names of the input datasets and not just file numbers to make the output more understandable.
I suggest to stick to our camelCase naming convention and not use underscores or points in the component names. (This applies to deviationsFromTrue, rmsd.DeviationsFromTrue, calibrationParams, and driftParams.

Things we should not forget

[ ] writeDataToFile(..) needs to be updated to be able to work with the new format.
[ ] outputSummaryFile(..) needs to be updated to work with the new format and to include more quality control information.
[ ] The roxygen docstrings need to be updated for all functions that are changed.
[ ] The tests should be updated continously. They should always be green.

@twollnik Thanks for your feedback.
I have updated the comment to adopt a consistent camelCase naming convention and included the name parameter.
Regarding your question:
What I meant was in each case d18O and dD as names for the vectors, since the respective quantities are all one numeric value for each isotope species. Or should we rather instead use named lists to be more consistent?

Thanks for integrating my feedback.

Or should we rather instead use named lists to be more consistent?
Yes, good idea.

Ok, thanks, I will edit the structure to also use lists for the respective quantities.

@thomas-muench I suggest renaming the component qualityControl to controlStandardDeviation to be more precise.

@thomas-muench I added the file piccrMockOutput to the cpt picarr repository. You can download the file and then execute load("path/to/piccrMockOutput") to load the variable piccrMockOutput into your workspace. It contains example output in the format that cpt picarr expects. (Note that some values are NULL)

@twollnik Thanks for the mock variable; this is helpful.

@thomas-muench I suggest renaming the component qualityControl to controlStandardDeviation to be more precise.

I would not use this name since it sounds like the "control standard deviation (SD)" value. Instead, how about deviationControlStandard or deviationOfControlStandard?

I like deviationOfControlStandard best.

I like deviationOfControlStandard best.

Updated accordingly.

@thomas-muench

the mean water vapour level and its standard deviation for this sample/standard;

At the moment this information is not included in the processed data. Would you prefer to include two columns H2O_Mean and H2O_SD in the processed data or should this information be calculated by cpt picarr?

Good point; I would have cpt picarr to calculate this.

@twollnik Should I initialize the output structure such that it always contains all possible elements but certain elements might be NULL when a specific processing step was not switched on (e.g. memoryCoefficients, ...)?
I also would suggest to rename pooledStdDev to pooledSD, since SD is the common abbreviation for the standard deviation.

@twollnik Should I initialize the output structure such that it always contains all possible elements but certain elements might be NULL when a specific processing step was not switched on (e.g. memoryCoefficients, ...)?

Yes, please.

I also would suggest to rename pooledStdDev to pooledSD, since SD is the common abbreviation for the standard deviation.

I agree.

Left to do here:

[ ] output calibration parameter in calibrationParams
[ ] output linear drift regression parameter in driftParams

Finally completed and closed by #53.