Error getting QrCode

I got an error while Connecting:
"message":"wppservice.loop: Connection failed: interface conversion: interface {} is nil, not string"
My version is:
{2, 2121, 6}
Does anyone have a similar problem?

@LucasSPanaro  update to latest  version {2, 2126, 14} in session.go file

It seems the fix is changing the version to {2,2126,14}

Got the same error again, with whatsappBusiness, version <IP_ADDRESS>:
"wppservice.loop: Connection failed: interface conversion: interface {} is nil, not string"

Got the same error again, with whatsappBusiness, version <IP_ADDRESS>:
"wppservice.loop: Connection failed: interface conversion: interface {} is nil, not string"

Whatsappweb works and sends 2.2126.14, but it doesn't work with go-whatsapp sending the same payload. It answers with: [\"Cmd\",{\"type\":\"update\"}]

It seems the connection fail error changed to timeout.
Now most of our clients are connected with 2.2126.14, but they are getting timeout when trying to send new messages.
Anybody experiencing something similar?

We have same problema..
"Websocket keepalive for 0xc0002e8b40 failed: connection timed out"
Stay on air few minutes and down...
Em 02/08/2021 14:18, Paulo Cabral Sanz escreveu:

It seems the connection fail error changed to timeout.
Now most of our clients are connected with 2.2126.14, but they are getting
timeout when trying to send new messages.
Anybody experiencing something similar?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Rhymen/go-whatsapp/issues/604#issuecomment-891195996, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABWBCCSGDED6YFV77DNSDJLT23HO7ANCNFSM5BMY3ESA.

It seems we fixed it by increasing the timeout, anybody experiencing the same?

Parece que corrigimos aumentando o tempo limite do NewConn, alguém experimentando o mesmo?

Changed the time for how many Paulo?


Need the ability to instruct Phinx to not replicate queries

I'm dealing with MySQL replicated hosts all the time and it's mandatory that the DB alteration is not replicated through (so I can lower chances of production down time). I currently execute the migrations on the replication servers to avoid this down time.
Currently I'm duplicating these methods in each of my migration classes:
    /**
     * Disables replication for this query session.
     *
     * @return $this
     */
    protected function dontReplicate()
    {
        $this
            ->commitTransaction()
            ->execute("
                SET SESSION sql_log_bin=0;
            ");

        return $this->beginTransaction();
    }

    /**
     * @return $this
     */
    protected function beginTransaction()
    {
        if($this->getAdapter()->hasTransactions())
        {
            $this->getAdapter()->beginTransaction();
        }

        return $this;
    }

    /**
     * @return $this
     */
    protected function commitTransaction()
    {

        if($this->getAdapter()->hasTransactions())
        {
            $this->getAdapter()->commitTransaction();
        }

        return $this;
    }

And then I can use it like this:
    /**
     * Migrate Up.
     */
    public function up()
    {
        $this
            ->dontReplicate()
            ->execute("
                ALTER TABLE `my_database`.`my_table`
                CHANGE COLUMN `show_columns` `group_by` TEXT NULL
                    DEFAULT NULL
                    COMMENT 'JSONArray of fields to group by' ;
            ");
    }

Now obviously this works, but it's really redundant as you can see. I know we have the ability to specify our own base classes now (I still have to read up on how to do that), but I feel like this would be a great core feature for others to take advantage of.
The ideal feature would allow a configuration level setting to turn off/on replication for each environment. Finally the migration's abstract class would provide two methods to enable/disable replication.
$this->replicate();
$this->dontReplicate;
I'm more than happy to work on a PR if enough people want this.

Are you able to provide a pull request with your suggested changes?

Closing due to lack of activity or out of scope. If there is new interest, please open a PR.


747 update contrib

closes #747 (unsquased to simplify review)
Would need a few more passes (as usual with docs/writing, this can take time, so usually would go into separate PR).
(please feel free to close this PR for any reason. All good here)

Hey @abebeos, thanks you for contribution. Can you please squash the changes into a single commit, so we can push it as a "single atomic meaningful change"?

Ok, rebased&squashed


ensure algolia uuid exists before attempting to index

@brynnb and I were working with this library and found that, after the UUID change was introduced, existing pages/objects weren't be assigned UUIDs, which prevented them from being indexed. This change assigns a UUID to the object if one doesn't already exist at the time of indexing.

Thanks! Also think you’ll need to reindex everything up front to ensure everything in Algolia has a UUID


⚠ Samsung has degraded performance

In 3d8c383, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2207 ms

Resolved: Samsung performance has improved in e42e042.


Sub-Monoid Calculator

Enter calculator name -
Sub-Monoid Calculator
Describe calculator functionality -
Given a Monoid M and a Set A along with an operator 'op' on which it needs to be checked. The Calculator checks if the given Set A is a sub-monoid of the Monoid M or not.
Under which category you wish to place it -

[ ] Computer Science
[x] Mathematics
[ ] Physics
[ ] Chemistry
[ ] Biology
[ ] Art
[ ] Finance
[ ] Scientific
[ ] Health
[ ] Games/Sports
[x] Let project owner decide
[ ] Suggest a new category - (Answer here)

@1520229462,
Abiding by the contribution policy, not more than 2 issues can be assigned at a time.
Kindly comment once the previous issues are merged.


[ownership] Add an extra run of -Onone tests with diagnostics with -e…

…nable-ownership-stripping-after-serialization enabled.
Right now the stdlib/overlays can compile against -Onone tests with or without
-enable-ownership-stripping-after-serialization. This will help me to prevent
other work going on from breaking these properties.

I am taking advantage of the stdlib/overlays being compiled with -enable-ownership-stripping-* here. Specifically, -Onone tests can compiled in either modes now. This is letting me enable this in the tree slowly over time.

@swift-ci test and merge


Move Macro Type Analysis Code to dbt-common

Description
Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[x] I have run changie new to create a changelog entry

Codecov Report
Attention: Patch coverage is 54.41176% with 31 lines in your changes missing coverage. Please review.

Project coverage is 68.83%. Comparing base (243568e) to head (3a20cdc).

Files with missing lines
Patch %
Lines

dbt_common/clients/jinja_macro_call.py
53.73%
31 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #230      +/-   ##
==========================================
- Coverage   68.89%   68.83%   -0.06%
==========================================
  Files          52       53       +1
  Lines        3433     3501      +68
==========================================
+ Hits         2365     2410      +45
- Misses       1068     1091      +23

Flag
Coverage Δ

unit
68.83% <54.41%> (-0.06%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


update vep config etc

Update VEP settings cfr current bcbio config
bump modules
bring workflow naming in line with preprocessing
Add support for iGenomes/local genomes config, closes #32

PR checklist

[ ] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!

[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs

[ ] Make sure your code lints (nf-core lint).
[ ] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

@nvnieuwk , I'm going to merge this PR into your v1.1.0dev branch if the local tests pass.  I'm going to continue work in your branch while you're away.

I'll have a quick look now, I have a local change that overhauls all the tests, but I am waiting for the new update of nf-test to push it (because it won't work right now)

can always add the tests to your branch from another pr

Local tests pass, so I think it's good to go!


Show various features of partitioning

@IgorBaratta is there a way of chosing the partitioning algorithm (scotch, kahip, parmetis) and ghost mode at the same time? As far as I can tell this is not possible currently, see the changes/outputs in this PR

Issue made in DOLFINx for the remaining bit:
https://github.com/FEniCS/dolfinx/issues/2591


Rendering plain single Sequel Model object failed

Here is the code
require 'sqlite3'
require 'sequel'

DB = Sequel.sqlite
DB.create_table :items do
  primary_key :id
  String :name
  Float :price
end

items = DB[:items]

items.insert(:name => 'abc', :price => rand * 100)
items.insert(:name => 'def', :price => rand * 100)
items.insert(:name => 'ghi', :price => rand * 100)

class Item < Sequel::Model
end

require 'json'
require 'jsonapi/serializable'

class SerializableItem < JSONAPI::Serializable::Resource
  type 'item'

  attributes :name, :price
end

all_items = Item.all
first_item = Item.first

renderer = JSONAPI::Serializable::Renderer

renderer.render(all_items, class: SerializableItem) # this works

renderer.render(first_item, class: SerializableItem) # this does not work

I'm pretty sure the problem is originated in the  JSONAPI::Serializable::ResourceBuilder class where it checks if passed in objects respond to "each" and actually call "map" method.
Can we just check the passed in object respond to "map" method instead of "each"?

Hi @luciusgone – thanks for reporting this. I remember identifying this bug a while back (Sequel, for some reason, implements #each on models such as it iterates over the fields), and implementing a fix (namely checking for #to_ary instead of #each) but it got lost somehow. Will issue a PR immediately :+1:.

@luciusgone I just tested your code with #58 and it seems to work fine :+1:.


pre-trained model(gta2cityscapes)

Could you provide pre-trained model that can produce translated Cityscapes-style GTA images？
thanks

The translated images are released as well as the DRN model trained on those images, but the translation model was not part of the initial release and is not available at this time.


dr-update not found

Hello, I just got into deepracer and im not really that good.
I tried installing Deepracer-for-cloud on my WSL2 ubuntu distro. Installed all the the prerequisities and tested my cude on my WSL and works. I have also followed the  documentation on installing the DRfC by running this command:
bin/init.sh -a gpu -c local
However, when I tried dr-update it says that command not found. Hoping someone could help me with this.
Thank you in advance

You need to parse the bin/activate.sh file using source (see Local Training Troubleshooting):
https://aws-deepracer-community.github.io/deepracer-for-cloud/installation.html#local-training-troubleshooting
source bin/activate.sh
In the event that you do and you get errors. It may be that your shell is outdated. I'd recommend using bash4 or bash5.
For example macOS ships with bash3 so most likely you will need to update, assuming you have brew:
brew install bash


Update munit, munit-scalacheck to 0.7.16

Updates

org.scalameta:munit
org.scalameta:munit-scalacheck

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

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

labels: test-library-update, semver-patch

Codecov Report

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

@@           Coverage Diff           @@
##           master     #359   +/-   ##
=======================================
  Coverage   62.84%   62.84%
=======================================
  Files          38       38
  Lines        1281     1281
  Branches        8        8
=======================================
  Hits          805      805
  Misses        476      476

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 f02fdbb...bad925e. Read the comment docs.


Segment method doesn't seem to accept integers

Passing an integer to the .segment() method doesn't work as expected. It does work however if I change the integer to a string before passing it through.
Maybe this is a design choice, but it does seem to be unexpected behavior and doesn't fail in a descriptive way.
https://jsfiddle.net/k6advmLq/

Maybe this is a design choice, but it does seem to be unexpected behavior and doesn't fail in a descriptive way.

the docs for .segment() show that for the first argument numeric values are considered indices for getting or setting values. String values are appended as a new segment.


Make resolver changeable

Right now the resolver is configured as <IP_ADDRESS> it would be cool if this value could be changed...
Using the current configured dns instead of <IP_ADDRESS> seems to be a better default?

I guess you mean in the nginx template with OCSP stapling enabled, right?

yes, in a Amazon VPC you might want to resolve to local dns servers in order to query local Route53 Zones...

Implemented in master: 835072b2a812caaade9abf2dc3d2aa6e59e84314

Released in 2.0.0


Problem to intercept jQuery ajax requests (and also Angular $http requests)

Here is how I init (data is a simple hash):
          var restServer = new FakeRest.Server();
          restServer.init(data);
          var server = sinon.fakeServer.create();
          server.respondWith(restServer.getHandler());

Using pure JS works as expected:
          var req = new XMLHttpRequest();
          req.onreadystatechange = function() {
            console.log(req.responseText);
          };
          req.open("GET", "/authors", false);
          req.setRequestHeader( 'Accept', '*/*' );
          req.setRequestHeader( 'X-Requested-With', 'XMLHttpRequest');
          req.send(null);

If I try with jQuery the request seems to be caught but I get no response:
          var jqxhr = jQuery.ajax( { url: "/authors", accepts: { all: '*/*' } } )
          .done(function() {
            console.log( "success" );
          })
          .fail(function() {
            console.log( "error" );
          })
          .always(function() {
            console.log( "complete" );
          });

No console output.
The same with angularJS:
            $http({
              method: 'GET',
              url: '/authors'
            }).then(function successCallback(response) {
              console.log( 'ok' );
            }, function errorCallback(response) {
              console.log( 'err' );
            });

Any idea?

Did you load and register FakeRest before jQuery is loaded?

How could I do it?
I tried also to inject jQuery later on (for testing):
  <script>
    setTimeout( function() {
(function() {
    var script = document.createElement("SCRIPT");
    script.src = 'https://ajax.googleapis.com/ajax/libs/jquery/1.7.1/jquery.min.js';
    script.type = 'text/javascript';
    document.getElementsByTagName("head")[0].appendChild(script);
})();
    }, 1000 );
  </script>

Insert all the script tags mentioned in the Readme Usage section before loading jQuery.

That's what I tried.
Here are my script tags:
  <script src="FakeRest/dist/FakeRest.min.js"></script>
  <script src="sinon/sinon-1.17.3.js"></script>
  <script src="jquery/jquery-1.12.4.min.js"></script>

I tried in head or in the end of the body, same result.
If I inspect the ajax requests they are the same (headers are equals)
I put my code on plnkr if you want to take a look:
https://plnkr.co/edit/unEDpD8mMMhUUqwhfAQQ?p=preview

@fzaninotto  @blocknotes  problem is demonstrated here: http://codepen.io/airtonix/pen/BQKMjp

@fzaninotto  @blocknotes  Ok so it looks like when you set async: false on the jQuery.ajax call, it succeeds and fakerest can intercept.
But change it back to the default of async: true, then fakerest can't intercept.

I think the problem lies in sinon fake server
https://github.com/sinonjs/sinon/issues/510

@airtonix using async: false works for me too, probably it is caused by Sinon as pointed by @fzaninotto


How To Make Custom Themes

Something Like This Please Can Anyone Help Me

And How To Make A Button To Hide/show UI For Mobile Users

go in workspace in you exploit folder if you have solara

find the config or smthn like that, then edit it(idk man im triyn to help)

In the case of Quartyz Hub, they use a custom Fluent fork, which also adds the minimize icon.
Here's the link: https://github.com/xQuartyx/UILibrary/blob/main/Fluent (It doesn't seem to have the open source tree, just the modified build)


Update showall.dart

Description
Give a description of what this PR is about.
Jira Ticket

https://garrysjh.atlassian.net/browse/DIP-[your ticket number]

taskbar

we not using this page i think


Made compatible with Alamofire 5.0.0-rc2

This PR addresses the following issues:

Alamofire 5.0.0-rc.2 compatible version?
Generic type 'DataResponse' specialized with too few type parameters (got 1, but expected 2)

PTAL @tristanhimmelman
For those who are using Carthage, you can simply build from here for now:
github "ozgur/AlamofireObjectMapper" "master"

or Cocoapods:
pod 'AlamofireObjectMapper', :git => 'https://github.com/ozgur/AlamofireObjectMapper.git', :branch => 'master'

Hi
I'm still getting this error with below pods, i tried all mentioned Git master but nothing working.
Getting this error in XCODE11 with swift 5.
When i use AFDataRequest instead of DataRequest then issue go away. i don't whether its the correct approach to resolve the same ?
Please Help me to get out of it. thanks
pod 'Alamofire', '~> 5.0.0-rc.2'
pod 'ObjectMapper', '~> 3.4'
#pod 'AlamofireObjectMapper', :git => 'https://github.com/tristanhimmelman/AlamofireObjectMapper.git', :branch => 'master'
#pod 'AlamofireObjectMapper', :git => 'https://github.com/RomanPodymov/AlamofireObjectMapper.git', :branch => 'master'
#pod 'AlamofireObjectMapper', :git => 'https://github.com/ozgur/AlamofireObjectMapper.git', :branch => 'master'
pod 'AlamofireObjectMapper', '~> 6.2.0'
#pod 'AlamofireObjectMapper', '~> 5.2'
#pod 'Alamofire', '~> 4.0.0'


Capture guidance

First cut of capturing the details of the delivered guidance. Just capturing the attendee's name currently.

How about GuidanceRecord instead of Guidance?

Or maybe AppointmentRecord?

Any opinions @gjvis ?

Model renamed to AppointmentSummary per discussion

RuboCop!


scripts: Add GSI Building Script

Can be executed with 'python3 gsi.py'
Useful for Building GSIs with Ease xD

@techyminati Pru


fix(mpt): hint before fetch in insert

Description
When executing multiple blocks in the Kona derive pipeline, we ran into an issue where a TrieNode being requested by the fetcher before it was hinted in the state root execution. This is likely an oversight, but is necessary for multi-block execution with Kona.

Codecov Report
Attention: Patch coverage is 87.50000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 77.7%. Comparing base (5c30c4c) to head (efbd378).

Files
Patch %
Lines

crates/mpt/src/node.rs
82.3%
3 Missing :warning:

Additional details and impacted files

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

I'd rather not merge this until we understand what's going on.
It's interesting that this fixes the issue, but it's also a bit odd that it does. When inserting a new node, we do always have the full path to its parent opened within the trie. This is why we don't need to send hints for individual trie nodes within insert during regular operation, since we've front-loaded the account/storage proof hint.

Closing in favor of #396


chore: Update bundle for 2.14

Description
Keeping bundle upto date so that PR changes do not include unnecessary bundle updates

How Has This Been Tested?

Screenshot or short clip

Merge criteria

[ ] You have read the contributors guide.
[ ] Commit messages are meaningful - have a clear and concise summary and detailed explanation of what was changed and why.
[ ] Pull Request contains a description of the solution, a link to the JIRA issue, and to any dependent or related Pull Request.
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has manually tested the changes and verified that the changes work

/test rhods-operator-e2e

@VaishnaviHire  we could have https://github.com/red-hat-data-services/rhods-operator/pull/365 in (then drop this one)  or  get this in first then have 365, but both will need sync https://github.com/opendatahub-io/opendatahub-operator/pull/1275 to remove it afterward.

No longer relevant


Support grouping function for roll up

What problem does this PR solve?
Issue Number: close #7134
Problem Summary:
What is changed and how it works?
Support the grouping function and the passing of meta data for function.
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

What problem does this PR solve?
Issue Number: close #7134
Problem Summary:
What is changed and how it works?
Check List
Tests

[ ]  Unit test
[ ]  Integration test
[ ]  Manual test (add detailed scripts or steps below)
[ ]  No code

Side effects

[ ]  Performance regression: Consumes more CPU
[ ]  Performance regression: Consumes more Memory
[ ]  Breaking backward compatibility

Documentation

[ ]  Affects user behaviors
[ ]  Contains syntax changes
[ ]  Contains variable changes
[ ]  Contains experimental features
[ ]  Changes MySQL compatibility

Release note
None

/cc @ywqzzy

/run-all-tests

/run-all-tests

/merge

/cc @windtalker

/cc @windtalker

/cc @windtalker

/cc @windtalker

/run-all-tests

/run-integration-test

/run-integration-test

/run-integration-test

/run-integration-test

/run-integration-test

/rebuild

/rebuild

/run-integration-test


Add 2 examples on how to use injection in props/data

Relates to #5067, #5075

Let me know if you want me to fine tune this.

Should be better now 👍


Add interface for status and timeout to events

Add an interface for waiting for a specified amount of time for an event and getting a status associated with the event back once its signaled.
This is the interface for https://github.com/pytorch/glow/issues/2702

Lu is on "vacation" now, is this pr ready for review?

@zrphercule oh ok, yes it's ready

@zrphercule the idea is that the caller is supposed to query to find out which extensions are provided by the backend.


Fixed permissions for Docker

This adds a user called nodetube to the Docker container. Set the argument user_uid when building the container to your host system's user UID.

This is already fixed on master. Lot of issues I see with this so I'm closing.


[AKS] api-server auth access

Problem:
PAI services need query api-server data for monitor and metrics. api-server permission & auth problem caused some services cannot directly read the api-server data.
Problem screenshot:
Prometheus: x509: certificate is valid for apiserver
Service view no data
Watchdog can't access api-server
Challenge:
It is better to provide a common solution that each service can share. If not, each service is configured by itself.
Pai service involved:
Prometheus & Weportal & watchdog
Proposal & current status:
Leverage k8s RBAC & service (see reference) account to access api-server.
o	For prometheus:
Tested and create a PR
Code location: https://github.com/Microsoft/pai/blob/master/src/prometheus/deploy/prometheus-configmap.yaml.template
Solution: https://github.com/Microsoft/pai/pull/1458
o	For webportal
Code location: https://github.com/Microsoft/pai/blob/master/src/webportal/src/app/cluster-view/services/services.component.js#L44
Solution: service account + curl (attach token file) tested passed, haven't test under js script.
o	For watchdog
Code location: https://github.com/Microsoft/pai/blob/master/src/watchdog/src/watchdog.py#L244
Solution: service account + curl (attach token file) tested passed, haven't test under python script.
•	Need to do:

Try weather contains more general solutions to avoid configuration for each service component
If there is no better general method, implement the corresponding solution for each service.
•	Reference:
o	K8s RBAC: https://kubernetes.io/docs/reference/access-authn-authz/rbac/
o	K8s service account: https://kubernetes.io/docs/tasks/configure-pod-container/configure-service-account/

Dev plan

yanjie will create a pai/aks branch as master branch for receive all aks related pr
https://github.com/Microsoft/pai/tree/pai/aks
some knowledge could refer yanjga/aks-test driver related change
feature dev folk & submit pr to pai/aks branch
If all tests passed, then cherry pick pai/aks pr to master branch.

After discussion with @Gerhut , we prefer method1 to solve this problem and qixiang is help to try this solution.
the program summary:

3 proposals:
all use service account token.
method1: rest-server as proxy to attach token to api-server request call
method2: give token to client side, client side call service-view request attach tokens
method3: remove service-view page.
this is also related role binding issue. Related discussion is at:
https://github.com/Microsoft/pai/issues/1709

Related k8s dashboard access issue : https://github.com/Microsoft/pai/issues/1709

For prometheus and watchdog api-server access issue proposal summary:
will try to split PR:
1 make token file path configurable
2 prometheus access api-server and service account role binding
3 watcdog access api-server and service account role binding

for prometheus https://github.com/Microsoft/pai/pull/1939

for watchdog https://github.com/Microsoft/pai/pull/1710


feat!(math): contain degree sized elements univariate evalutions

Description

To compute degree of UnivariateEvaluations, it needs IFFT. Does it really mean anything in evaluation form? So this commit removes Degree(). Instead, UnivariatePolynomial and UnivariateEvaluations have NumElements() to count the number of elements. e.g, Considering constant polynomial, the degree of this polynomial should be 0, but previously it returns the number of elements.
UnivariateEvaluations except for zero polynomial have elements of MaxDegree. This affects equality operator, arithmetic operators and how it construct UnivariateEvaluations. e.g, Random() doesn't receive degree as an argument.
For performance reason, zero polynomial is allowed to have an empty vector.

BREAKING CHANGE: UnivariateEvaluations doesn't allow arbitrary number of evaluation vector any more.

@chokobole How about fixing ff2d8b7 to test commit instead of chore?

@chokobole How about fixing the commit description of aea2852 as well for two points?

real elements -> elements
MaxDegree->MaxDegree + 1

LGTM Before merging, could you please check this comment as well please? I'm not sure since there's no emoji left, so I think you might have missed it.

Oh I haven't checked it and fixed the commit type Thanks :)


Add spaces

A nice feature, to have, would be to add spaces that you could share with others.
space1
-- album1
-- album2
space2
-- album1
-- album2

Something like Collections or Multi-User?

More like https://github.com/electerious/Lychee/issues/254.


PHAuthorizationStatus.limited is not handled on iOS 14

Describe the bug
PHAuthorizationStatus.limited is not handled on iOS 14
To Reproduce

Select "Select photos" from the privacy confirm dialog.

Expected behavior
Screenshots

Environment (please complete the following information):

Device: Simulator
OS: iOS 14
Xcode Version: Version 12.0 beta 4 (12A8179i)

Installation Type

Cocoapods

Additional context

Can't run the example project.  occurs the same issue. "Type 'PHAuthorizationStatus' has no member 'limited'".
Environment
Xcode Version: 11.6 (11E708)
Device: Simulator
OS: 13.6

Added


Add method for exact builds of external toadz traits

For composability, would be nice to have a method that takes a metadata array and spits out the resulting toad.
Wouldn't need to be validated, we can leave that to the caller, and document the metadata format.
Creating a more user-friendly method with enums and validation would be a bit too much work for the ROI (imho)

implemented in abb2d9b6e11992ef9a4e0dc44a77f2ebfad72d15


Confusing guidance on Network Address Ranges

There are two bits of guidance in the Advanced Networking Deployment Parameters section which for me either contradict eachother, or require further elaboration to make more sense of the reason why:
Firstly, the documentation states: "Virtual network: The VNet into which you want to deploy the Kubernetes cluster."
Ok, that's straightforward enough, no confusion here.
But then:
"Kubernetes service address range: The Kubernetes service address range is the IP range from which addresses are assigned to Kubernetes services in your cluster"
"The Kubernetes service IP address range:

Must not be within the VNet IP address range of your cluster"

Specifically this last point is what presents the confusion for me because it is telling me to use a range outside of my Vnet. Am I to understand that pods will be assigned an IP within range of the Vnet, but, Kubernetes Services must live outside of the VNET in some separate Address Range?
If so, please can the documentation elaborate as to why.
Or, is the documentation actually trying to say that the Kubernetes Service address range should not be within the address range of the Subnet (rather than Vnet) that the cluster is deployed into?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 22ad4873-c73d-4b90-2063-927a1060d2a2
Version Independent ID: 8e6643ad-7333-a786-8712-3d17291e12a7
Content: Network configuration in Azure Kubernetes Service (AKS)
Content Source: articles/aks/networking-overview.md
Service: container-service
GitHub Login: @mmacy
Microsoft Alias: marsma

Thanks for the feedback! We are currently investigating and will update you shortly.

@MarkTopping The Kubernetes Service Address Range is IP Addresses that Kubernetes exposes to use it's services. These are not the IP Addresses of the physical machines that Kubernetes creates.
the IP Address range should be within your VNET, but not within the ranges where your cluster can deploy. Using the description of the doc, you would need to deploy it in a different Subnet than where you deployed the AKS Cluster, or give a range in the same subnet that will not conflict with physical machines being spun up. For instance, if your subnet allows 256 addresses, you expect AKS to consume 100, you can give .200 - .254 for the Service Address Range.

@MarkTopping Travis is correct: From a purely technical standpoint, you could specify a K8s service range that's in a different subnet, but within same VNet as your cluster. However, it's then also possible to later create a second (or third or fourth or...) subnet in the VNet that stomps over the K8s service range, causing potential conflicts. This is because the Azure Networking side doesn't have visibility into the K8s cluster to know not to allow that.
As such, that bullet's guidance is sound, and should be considered a best practice to avoid conflicts. Further, the K8s service range is not a routable range (within Azure, that is) and only K8s knows about it, routing packets from containers to a service within the cluster. To sum it up, this would be a valid config, and follows best practice guidance of using a service range outside the VNet:
VNet: <IP_ADDRESS>/16
Subnet: <IP_ADDRESS>/24
Kubernetes service range: <IP_ADDRESS>/16
Cc: @aanandr

Hi mmacy.
That also makes sense. I can work with that.
However, to say that Travis was right when he said:
"The IP Address range should be within your VNET, but not within the ranges where your cluster can deploy"
Well, you're guidance contrasts that by saying it is recommended for the IP Address Range to be outside of the VNET, not within.
And the documentation states it not as a recommendation, but uses the words 'must not'.
You're explanation here provides some useful and clear rational as to why its best to use a different address range - thank you.
Either way mind, if you use the Portal to provision a new cluster, the pre-create validation passes when you provision a cluster with the address range within or outside of the VNET; it might be better that validation did not pass when within. Anyhow, i'll proceed using your recommendation.
Thanks for your input.

@MarkTopping Sorry for the confusion here. I've clarified my comment by adding the "partially correct" qualifier to Travis' statement.
I'll update the article with more clarification on this guidance.
#in-progress

Great. Thank you

@mmacy - can we tweak the wordings a bit so that there is no confusion? For example we have to say that service IPs dont need to be created in a separate subnet. They are used only inside the cluster
@MarkTopping i didnt understand the last para of your response. Can you clarify

@aanandr
Sure... assuming you mean this:
"Either way mind, if you use the Portal to provision a new cluster, the pre-create validation passes when you provision a cluster with the address range within or outside of the VNET; it might be better that validation did not pass when within. Anyhow, i'll proceed using your recommendation."
It has nothing to do with this documentation; I was just pointing out an extra observation...
While I use the ACS-Engine myself, this comment refers to the validation of the AKS configuration that automatically occurs when provision a new cluster through the Azure Portal. Basically if you work though the various tabs to create a new AKS instance, and then when you click on the Review + Create tab which is the final tab before you actually create the instance, then a validation step kicks off. That validation step will report everything is ok when you place your Kubernetes Service Range within the VNET which is contradictory to to the guidance being offered above and during this conversation.
That said, subsequent creation of the cluster still works when the Kubernetes Service Address Range is placed inside the VNET, and the resulting cluster is operational; although (probably unrelated) I have suffered CNI errors within that cluster which is why I'm re-reviewing the underlying network setup.

Thanks Mark - when you specify a service IP range in the vnet space in the Networking tab you should have seen an error right there. I remember seeing it. Also, i dont think it lets you proceed further from that point. Let me know if this has been your observation.
Sent from Outlookhttp://aka.ms/weboutlook

From: Mark Topping<EMAIL_ADDRESS>Sent: Monday, August 6, 2018 11:48 PM
To: MicrosoftDocs/azure-docs
Cc: Aanand Ramachandran; Mention
Subject: Re: [MicrosoftDocs/azure-docs] Confusing guidance on Network Address Ranges (#12929)
@aanandrhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Faanandr&data=02|01|aanandr%40microsoft.com|ee1ec28efb3b446de8ed08d5fc31cbd3|72f988bf86f141af91ab2d7cd011db47|1|0|636692213171849261&sdata=ushBOeLnms8zbdy9IDk5BKZFnnLGjEtdY3BYvrs5oUY%3D&reserved=0
Sure... assuming you mean this:
"Either way mind, if you use the Portal to provision a new cluster, the pre-create validation passes when you provision a cluster with the address range within or outside of the VNET; it might be better that validation did not pass when within. Anyhow, i'll proceed using your recommendation."
It has nothing to do with this documentation; I was just pointing out an extra observation...
While I use the ACS-Engine myself, this comment refers to the validation of the AKS configuration that automatically occurs when provision a new cluster through the Azure Portal. Basically if you work though the various tabs to create a new AKS instance, and then when you click on the Review + Create tab which is the final tab before you actually create the instance, then a validation step kicks off. That validation step will report everything is ok when you place your Kubernetes Service Range within the VNET which is contradictory to to the guidance being offered above and during this conversation.
That said, subsequent creation of the cluster still works when the Kubernetes Service Address Range is placed inside the VNET, and the resulting cluster is operational; although (probably unrelated) I have suffered CNI errors within that cluster which is why I'm re-reviewing the underlying network setup.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F12929%23issuecomment-410952064&data=02|01|aanandr%40microsoft.com|ee1ec28efb3b446de8ed08d5fc31cbd3|72f988bf86f141af91ab2d7cd011db47|1|0|636692213171859271&sdata=Axw78IApFT55rG%2BZoPUS6nIIL83Rp%2Bny6X8vMwMqPpw%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAV3chYxNTnwo9O4S6amFMNXXyXb868beks5uOThBgaJpZM4VwU86&data=02|01|aanandr%40microsoft.com|ee1ec28efb3b446de8ed08d5fc31cbd3|72f988bf86f141af91ab2d7cd011db47|1|0|636692213171869280&sdata=BhjwDTHf6jcxqrq7x%2BNJtWofSpppu0MYqKKRaBL4mps%3D&reserved=0.

Hi AAnandr;
I'm afraid that's just not the case. Maybe when you try to specify a range in the Subnet space (which I've not tried), but you can definitely place it in the VNET Space.
i.e.
Vnet: <IP_ADDRESS>/12
K8s Address Space: <IP_ADDRESS>/16
Subnet: <IP_ADDRESS>/16
Results in no warnings on neither the Networking tab or the Review + Create tab.

@aanand I can confirm @MarkTopping's statement: no error is generated if you specify a subnet within the VNet for the K8s service range, as long as that subnet is different from the subnet in which you place the cluster. If you attempt to place the K8s service range in the same subnet as the cluster, then yes, an error is displayed.

Ok got it. Thanks for the clarifications.
@Marsh<EMAIL_ADDRESS>- can we include a question in the faq that covers this? We can talk about what the service IP is and the impact of overlap. The content is what you already have.
Sent from Outlookhttp://aka.ms/weboutlook

From: Marsh Macy<EMAIL_ADDRESS>Sent: Wednesday, August 8, 2018 7:57 AM
To: MicrosoftDocs/azure-docs
Cc: Aanand Ramachandran; Mention
Subject: Re: [MicrosoftDocs/azure-docs] Confusing guidance on Network Address Ranges (#12929)
@aanandhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Faanand&data=02|01|aanandr%40microsoft.com|9f284d1118cd4c2d843908d5fd3f32f9|72f988bf86f141af91ab2d7cd011db47|1|0|636693370245612536&sdata=hf2ntwxywOMPBLeNqn6ZgVMqx5SiKVpWYwOev5v%2F3ik%3D&reserved=0 I can confirm @MarkToppinghttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMarkTopping&data=02|01|aanandr%40microsoft.com|9f284d1118cd4c2d843908d5fd3f32f9|72f988bf86f141af91ab2d7cd011db47|1|0|636693370245612536&sdata=mWqCyJqu5HBnadLYAm5mjwi1btte4PwXvOq9y0ZKf7M%3D&reserved=0's statement: no error is generated if you specify a subnet within the VNet for the K8s service range, as long as that subnet is different from the subnet in which you place the cluster. If you attempt to place the K8s service range in the same subnet as the cluster, then yes, an error is displayed.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F12929%23issuecomment-411436023&data=02|01|aanandr%40microsoft.com|9f284d1118cd4c2d843908d5fd3f32f9|72f988bf86f141af91ab2d7cd011db47|1|0|636693370245612536&sdata=kijkzZ%2F%2FfxVU%2FLEFsVMcZvD1K%2F4S5HWNQu0hUjSKvfE%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAV3chbhKWxeVa3Y-oOwvD0r_FgfMBw7kks5uOvw9gaJpZM4VwU86&data=02|01|aanandr%40microsoft.com|9f284d1118cd4c2d843908d5fd3f32f9|72f988bf86f141af91ab2d7cd011db47|1|0|636693370245612536&sdata=gN%2BiLz3PpfyVj%2FP20qVMxCVHOAoTIWVe2bKIjpmfTgc%3D&reserved=0.

Somewhat off topic, but it would be helpful if the doc explained how to measure how much space you should need in the "Kubernetes service address range". Similar to how we calculate the space needed for the cluster subnet

Adding @mlearned who's the current article owner.

Also, it would help if the doc could explain how cbr0 and veth0 plays a part in AKS. A diagram of the network flow with these components would clear my confusion.

Two year later, and the documentation still has not been updated...


⚠ Community has degraded performance

In 7db95e9, Community (https://community.wrk.com/health) experienced degraded performance:

HTTP code: 200
Response time: 198 ms

Resolved: Community performance has improved in ddc96b9.


feat(providePlugin): Add ability to exclude files for ProvidePlugin

This feature will allow one to exclude files from the ProvidePlugin. This is useful if you wish to override a native function such as XMLHttpRequest. See #6226 for more details.
What kind of change does this PR introduce?
This is a feature and adds a new form of input in ProvidePlugin:
new ProvidePlugin({ module: "aaa", exclude: "bbb.js" })
new ProvidePlugin({ module: "aaa" })
new ProvidePlugin({ module: ["aaa", "bbb"] })

Did you add tests for your changes?
Yes
Does this PR introduce a breaking change?
No
What needs to be documented once your changes are merged?
Need to update the documentation to show the new feature and optional data structure.

I'd love to see this merge to enable use with polyfills. I guess this enables:
new ProvidePlugin({ some: "some-polyfill", exclude: require.resolve("some-polyfill") })

But actually, I think this should be done automatically in the some-polyfill specified. Why does it include the same plugin it provides in its substitutions?

@fregante I did look into this and that was my thought as well, but this would also be a breaking change. Not sure if anyone is using that as a feature.
The only reason this stalled is due to the windows failures which I was unable to recreate locally.


Support Docker Registry v2

As v1 is being deprecated we should support registry v2.

+1
On Fri 2 Oct 2015 at 2:31 p.m., Evan Hazlett<EMAIL_ADDRESS>wrote:

As v1 is being deprecated we should support registry v2.
—
Reply to this email directly or view it on GitHub
https://github.com/shipyard/shipyard/issues/625.

+1

v2 registry is a requirement for me, so +1.

+1

:+1:

:+1:

Awsome
Expecting authentication service for registry

+1

+1

+1

+1

+1

:+1:

+1

+1
any news on this?

Yes I have a branch started that works with v2.  The UI needs to be
refactored.
On Wed, Oct 28, 2015 at 6:04 PM, Mariusz Łuciów<EMAIL_ADDRESS>wrote:

+1
any news on this?
—
Reply to this email directly or view it on GitHub
https://github.com/shipyard/shipyard/issues/625#issuecomment-152011492.

+1 Any idea when this feature will be good to go? What skills are required to refactor the UI?

+1

+1

@starlight36 你来啦～

@RustJason 是啊，用上了

+1

+1

registry V2 +1

+1

+1

+1

+1 @ehazlett https://github.com/ehazlett/shipyard/tree/registry-v2 nice work, hope this gets productive soon

+1

+1

+1

+1

+1

If this includes authentication that'd be awesome

+1

+1
On Mon, Dec 28, 2015 at 8:49 AM, Abdellah<EMAIL_ADDRESS>wrote:

+1
—
Reply to this email directly or view it on GitHub
https://github.com/shipyard/shipyard/issues/625#issuecomment-167451245.

+1

While I'm sure it is appreciated to see that there is interest, adding +1s just ends up sending unnessary emails to everyone subscribed and PRs are always welcome :smile:

+1

+1

+1

+2

+1

After 5 months without solving this issue. Somebody are using other docker registry console that supports V2?

There is the Docker Trusted Registry from Docker that supports v2.
On Sun, Feb 14, 2016 at 1:55 PM, Ricardo Peironcely <
<EMAIL_ADDRESS>wrote:

After 5 months without solving this issue. Somebody are using other docker
registry console that supports V2?
—
Reply to this email directly or view it on GitHub
https://github.com/shipyard/shipyard/issues/625#issuecomment-183949563.

+1

+1

+1

+1

+1

+1

This is some kind of practical joke, isn't it?

+1 This seems to be the central issue for the v2 registry feature. As i am currently working on a project where shipyard would be the perfect fit, i am interested in getting this feature asap.
As a start, it would be great if there was an official dockerhub image shipyard:registry.v2 available as source for joint testing and/or documentation.

+1

+1

Could we just close this issue and admit that shipyard will never support registry v2?

@nfoonf if you would like to contribute I would be more than happy to review.  as stated in IRC:
10:03 <nfoonf> hi i am following this channel for nearly to months now
10:03 <nfoonf> and I finally come to the conclusion, that shipyard is practically totally dead
10:04 <nfoonf> for example the registryv2 support thread is mostly a practically joke
10:06 <nfoonf> as I sunk quite an amount of time (and thererfore money) into getting a productive environment for cluster management running based on shipyard I suggest, that you publicly mark this project as broken/unmaintained etc. to not let other people make the same mistake
10:07 <nfoonf> as in using a immature and unready project in the hope that seemingly minor bugs can and will be fixed or community input is actually in any way either wanted or honored
10:08 <ehazlett> it is an open source project and people's free effort and our own time (usually nights and weekends) cannot always be put priority as we have to maintain day jobs.  being an open source project we welcome contributions and i personally have continued to give direction towards the registry v2 branch.  i'm sorry you are upset you cannot get more effort into
10:08 <ehazlett> this.  i welcome any and all help.
10:12 <ehazlett> in fact there were 5 PRs merged in Feb including one of your own (thank you).  calling a project and its features "practically a joke" is insulting to everyone that has put time and effort into a project and these comments are why people leave open source.

I would love any and all help as my day job is taking priority.  I welcome any and all help.

+1

+1

now v2 is be cool?

Hello, everyone... we have incorporated the work that was started on branch https://github.com/shipyard/shipyard/tree/registry-v2 into our work for Shipyard / ILM project, which adds image testing and verification to Shipyard. Our work is still experimental, but if you want to give it a spin, you can clone our fork and switch to the demo branch https://github.com/eedevops/shipyard/tree/dev. We hope to eventually submit a pull request back to Shipyard.

+1

+1

We have a PR submitted for supporting V2, please check #852

+1


No module named 'langchain_community.document_loaders'; 'langchain_community' is not a package

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
from langchain_community.document_loaders import pdf
Error Message and Stack Trace (if applicable)
ModuleNotFoundError: No module named 'langchain_community.document_loaders'; 'langchain_community' is not a package
Description
我的python环境里确实有langchain_community这个包，其他包都没问题就这个有问题
System Info
windows ,python 3.9.8

Have you installed the langchain_community package?
https://pypi.org/project/langchain-community/
pip install langchain_community

I am still encountering the same issue despite having installed langchain_community.


Bug in extracting auxiliary data fields from BAM file

For some BAM records, attempting to find a nonexisting BAM auxiliary data field raises an unexpected error. This happens because getindex(record) calls getauxvalue, which calls findauxtag, which calls next_tag_position, which does not check for the data_size(record), and runs "off the edge". Thus the data after the proper end of the record is attempted to get parsed, which results in an error.
Expected Behavior
julia> haskey(record, "QQ")
false

Current Behavior
julia> haskey(record, "QQ")
ERROR: invalid type tag: ''
Stacktrace:
 [1] next_tag_position(::Array{UInt8,1}, ::Int64) at /home/jakni/.julia/dev/BioAlignments/src/bam/auxdata.jl:150
 [2] haskey(::BioAlignments.BAM.Record, ::String) at /home/jakni/.julia/dev/BioAlignments/src/bam/auxdata.jl:109
 [3] top-level scope at none:0

Possible Solution / Implementation
I propose one of two changes:

Changing AuxData so that it contains a view of the data rather than a copy. This makes it very cheap to instantiate, and so the AuxData can be a field of the BAM record, instantiated when the BAM file is instantiated. All operations on the aux fields then simply operates on the already-existing lightweight AuxData object.
Removing the AuxData object and rewrite all the aux operation to work directly on the record.

I can pitch a PR with either of those changes. I'd also like to add a get method to BAM records, similar to that of Dicts, to make it less awkward to tell Julia to "get this aux field if it exists, but don't raise an error".
Steps to Reproduce (for bugs)

Open any BAM file (you can use this small BAM file)
Run the following function to obtain a bad record:

function getbad(reader)
    record = BAM.Record()
    while !eof(reader)
        read!(reader, record)
        try
            x = haskey(record, "PW")
        catch
            close(reader)
            return record
        end
    end
end

You can then provoke the error easily:
haskey(record, "PW")

Closing this issue, as it should be fixed by the commit above.


apple me.com/mac.com/icloud.com vacation bounce (not RFC standard)

Hi,
I'm seeing some vacation bounce coming from apple (me.com , mac.com),
They don't use any RFC standard, but have specific headers
Return-Path<EMAIL_ADDRESS>X-Original-To<EMAIL_ADDRESS>From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>Subject: xxxx
Date: Mon, 16 Dec 2019 10:20:53 +0000
Message-id<EMAIL_ADDRESS>In-Reply-To<EMAIL_ADDRESS>References<EMAIL_ADDRESS>X-Apple-Criteria: {kind=VACATION,recipients=1396}
X-Apple-Action: VACATION
X-Apple-UUID: 30a87ac7-16dc-4b7c-bf82-3a5c6f784ff8
X-Apple-Server: st13p27ic-ztba19270301 mailgateway 1923B114
Content-type: text/plain; charset=UTF-8

I don't known if it's something that could be parsed by sisimai ?

As of present, the latest version of Sisimai can not parse such email above. However, to implement some code for detecting X-Apple-Action: VACATION or Subject: Auto reply: ... at lib/Sisimai/RFC3834.pm may make sisimai can parse.


Add codota search for apache/storm to Readme

The Storm codebase was semantically indexed in Codota - available at
https://www.codota.com/xref/#/github_apache_storm_560da9ada8cb8703008bbfdc/findUsages
I believe it will be useful for developers who want to learn and explore the code.

I'm not sure how useful codota is since it doesn't look like it indexes the clojure code. I'm not against adding it, but I'll wait to see what others think.
My only real concern is that providing an incomplete guide to the code base will result in confusion for anyone that wanted to learn about it.

I am +1 for merging this in.  With the JStorm merge I think it will become more useful for developers as it is now, it is mildly useful for users who what to see what the java code is doing.

LGTM. +1


Tobin gpdp2 GitHub merge

Port validation for get_physical_device_properties2 extension over to GitHub now that it's public.

Also, for some reason the appveyor build choked on 'qfp'.  Might be spurious.


Iteration-03 milestone done: organize bug bash and wrap up Iteration-03

Once all current issues of the Iteration-02 milestone are closed:

[ ] organize bug bash with docs as only self-service source
[ ] file all issues under the milestone
[ ] redirect all "deploy to azure" links to point to main branch instead of release branch
[ ] triage + fix all issues that are needed for Iteration-02
[ ] document the changes in the CHANGELOG
[ ] close the milestone.
[ ] issue a new tag for this release, with a fancy description of the changes

In https://github.com/Azure-Samples/azure-ml-federated-learning/blob/release-sdkv2-iteration-03/docs/provisioning/orchestrator_open.md
path to template is wrong, use fl_pairs instead

Add annotations in the config.yaml explaining where users should put their orch/silo settings.

For typical failure of permissions during training, add a troubleshooting guide with screenshot explaining if failure for access denied during training, maybe check permissions of silo compute on orchestrator storage.

Clarify parameters "existing *" in exisitng storage tyutorial

Add a simple way to set the permissions RBAC between silo and orchestrator


Added Chizuru Ichinose

Issue No: #4
Added Chizuru Ichinose
#hacktoberfest

Please sync your Fork & Open the pull request

Please sync your Fork & Open the pull request

do i have to reopen the pull request?


microphone not working on windows

Aknowledgements

[X] I have checked that there's no other issue describing the same or
similar problem that I currently have, regardless if it has been
closed or open.

[X] I can confirm that this is not an issue with the Discord website,
but it is a problem specific to the WebCord itself.

[ ] I have tried running the build from the master branch and it does
not have any fixes implemented according to my issue.

[ ] My issue describes one of the unstable and/or not fully implemented
features.

[ ] I have found a workaround to mitigate or temporarily fix this issue
(please write it in Additional context section).

Operating System / Platform
🪟 Windows
Operating system architecture
x64 (64-bit Intel/AMD)
Electron version
16.0.4
Application version
2.2.0
Bug description

I'm on windows and the microphone access is denied.
I found the same issue on another project pertaining this project: https://github.com/sandboxie-plus/Sandboxie/discussions/1118
Additional context
Notice: This issue was automatically generated by WebCord.

Did you make sure you've granted microphone permissions to Discord and optionally restarted the app if that wasn't enough?
With the latest release, WebCord should have disabled the microphone access unless the user granted it via the app settings (not Discord ones!).

yes i followed the FAQ , cleared discord from the task manager  but i cant seem too find a way on W10 to specifically give this app the permission to use the mic.
this is how it shows with the release 2.2.0 :

and with 3.0.0 beta8 :

the windows firewall popped-up once when i clicked on mic test - "lets check" , but after that the "help desk" keep appearing.
(also if that can help im on an old realtek version, i had to roll back due to laptop speakers crackling)

yes i followed the FAQ , cleared discord from the task manager but i cant seem too find a way on W10 to specifically give this app the permission to use the mic. this is how it shows with the release 2.2.0 :
and with 3.0.0 beta8 :
the windows firewall popped-up once when i clicked on mic test - "lets check" , but after that the "help desk" keep appearing.
(also if that can help im on an old realtek version, i had to roll back due to laptop speakers crackling)

Just in case this is Discord configuration, not app (WebCord-specific) ones.
Application can be configured by going to File → Settings on WebCord's menubar.
See #87, I feel like this issue might be duplicate of it.

thanks alot now it works, for some reason the app bar dont show up unless i press ALT . sorry for the nuissance

thanks alot now it works, for some reason the app bar dont show up unless i press ALT . sorry for the nuissance

You've probably set it to be hidden. Check the settings, there's an option to make it always visible or hide it automatically depending on what you prefer to do.

Actually I've found a bug in the code that made default permission handler to be in conflict with the one set for main window only.
I've made some changes to the code on my disk, I'll push it once I'll make sure everything else is ready for that as well.


OSIDB-3189: Filter jira users by task key

OSIDB-3189: Filter jira users by task key
Checklist:

[x] Commits consolidated
[x] Changelog updated
[x] Test cases added/updated
[x] Jira ticket updated

Summary:
Filter out jira users that does not have access to the current project.
The endpoint api/2/user/picker does not allow to specify a task ID and returns all the jira users, currently there isn't an available endpoint from the public API that allows this kind of filtering while maintaining  the option to query by any part of the user (name, username or email).
I ended up using an internal endpoint /internal/2/users/assignee that allows filtering via task id and querying any part of the user (name, username or email)
Considerations:
Snapshot testing
I took the opportunity to do a small proof of concept about snapshot testing, so we can start using it from now on.
This kind of tests will prevent us for doing regressions like in #365 as they match the whole html tree of the component, so any change in classes, attributes or structure will make the test fail.
This will also simplify testing, instead of manually expecting multiple values to exist or to have an specific value.
As an example, all this code:
https://github.com/RedHatProductSecurity/osim/blob/c1683742d1eb032ec5aaf54c275f86cfa3829e39/src/components/__tests__/CvssNISTForm.spec.ts#L89-L97
can be replaced with:
expect(wrapper.html()).toMatchSnapshot();

Here's the link to the test file and the generated snapshot
Closes OSIDB-3189

So when making an intentional UI change we would follow this process for updating a snapshot and commit the snapshot, sounds like--is that right?
Did we also want to use image snapshots for views? If so, I think that could wait for another PR.

So when making an intentional UI change we would follow this process for updating a snapshot and commit the snapshot, sounds like--is that right?

Exactly, it is as easy as vitest -u, could be made into a yarn script like we have yarn unit-test, i.e yarn unit-test-update

Did we also want to use image snapshots for views? If so, I think that could wait for another PR.

For visual snapshots we need to install https://github.com/americanexpress/jest-image-snapshot for the method expect().toMatchImageSnapshot() to be available.
Visual snapshots are great for style changes, maybe we could implement them only for the views instead of the components.
I think for that to work properly we need to first create mocks of everything backend related, as different data on the screenshot means that it will fail to compare, also we would need to host the snapshot somewhere on the repository, and having real data (even for stage data) could be a security risk, that said I think they can also be a great addition.
We could run the tests as a Workflow, here's an example from one of my repositories and a PR with the automated comments implying the tests failed https://github.com/MrMarble/drg-editor/pull/28.
Sadly I deleted the images from the hosted service so they can not be longer viewed but here's one screenshot I had saved of a test that failed to load the correct font

Screenshot


Added additions to commands to make sure they work

I was mistaken in the need for --trust-node=true as that is a config setting that can be made


Cannot open markdown link

Prerequisites

[ ] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: http://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
Cannot open markdown link
Steps to Reproduce

Create markdown file

[GitBook on GitHub](https://github.com/GitbookIO)
https://github.com/GitbookIO

hover link in line 1 (character 26 to 49)
Trigger "Link:Open" in Command Palette

Expected behavior: [What you expect to happen]
Link opened in browser
Actual behavior: [What actually happens]
Nothing happened
Reproduces how often: [What percentage of the time does it reproduce?]
100%
Versions
$ atom --version
Atom    : 1.17.2
Electron: 1.4.16
Chrome  : 53.0.2785.143
Node    : 6.5.0
$ apm --version
apm  1.18.2
npm  4.6.1
node 8.0.0 x64
python 2.7.13
git 2.13.0
$ apm ls | grep link
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>Additional Information
Note bare link in line 2 works

Electron: 1.4.16

Thanks for reaching out!
We've determined that this issue is being caused by the use of an unofficial build or distribution of Atom. Often these customized versions of Atom are modified versions of the Stable branch of Atom with mismatched versions of built-in components. These updated components are taken from the Beta channel or master branch and then injected into the Stable version and a new Atom package is generated. Because of the way Atom is constructed, using these mismatched components can cause mysterious and hard-to-diagnose problems. You can find out more about why we chose to not support unofficial distributions here.

I see, thanks for the info.

I'm on Arch and switch to aur/atom-editor-bin, which is based on the Debian release.
The issue persists. Is it still a mismatched versions issue?
$ atom --version
Atom    : 1.17.2
Electron: 1.3.15
Chrome  : 52.0.2743.82
Node    : 6.5.0
$ apm --version
apm  1.17.0
npm  3.10.5
node 6.9.5 x64
python 3.6.1
git 2.13.1

I'm on Arch and switch to aur/atom-editor-bin, which is based on the Debian release.
The issue persists. Is it still a mismatched versions issue?

Though there might not be version/component mismatches, it's still technically an unofficial - would you be able to install directly from https://atom.io?  Since I don't have Arch setup to test on right now, I'll tag as needs-repro for now.

I then downloaded https://github.com/atom/atom/releases/download/v1.17.2/atom-amd64.tar.gz.
I tried to trace the issue by adding console.log() to ~/Downloads/atom-1.17.2-amd64/resources/app/node_modules/link/lib but it there is no output from the console.
    activate: function() {
      console.log('link activated')
      return atom.commands.add('atom-workspace', 'link:open', openLink);
    }

Any hint on that?

I think if you start in dev mode that should work?
http://flight-manual.atom.io/hacking-atom/sections/contributing-to-official-atom-packages/#running-in-development-mode

Cannot confirm, no response:


Bug: Severely altering the behavior of the global Graph View in the Ars Magna theme

This plugin is severely altering the behavior of the global Graph View in the Ars Magna theme, for example by removing the configuration buttons

Sorry for the delay, can you help me understand what you mean (e.g. with a "before Map View" and "After Map View" screenshots)?
I installed Ars Magna and can't spot any differences in the global graph view.


Issue 9

Issue 9 - Generating Default README.md
Description
Generating a default README.md based off a template.
Type of Change

[X] Feature
[ ] Bug fix
[ ] Documentation

Contributors
@kailaniwoodard
Reminder
All GitHub Actions should be in a passing state before any pull request is merged.

@kailaniwoodard This PR is attempting to merge your changes into main, which is not the correct target branch -- you should be merging these changes into develop. Additionally, you need to ensure that linting passes, perhaps by deleting the example hello package, as your teammates have done.


Missing projectspecific deep userrights

Based on the changements of Typz we need some additional rights for users like the owners of Typz.
For an owner we need the following additional projectrights:

manage catagories
manage automatic actions
manage the rows of a board

We need them asap. For detailed information you could contact me.

It's done in master (currently under testing).


'random' import error

version v0.2.0
>>> from colors import random
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ImportError: cannot import name random
>>> from colors.base import random
>>>

What version of python is this?

2.7.9

Oh, I see the error. :) It's not in __all__. One sec.


Sync percentage is wrong

The percentage of synchronized blocks is different when running zec-qt-wallet (0.5.5) and zcashd (2.0.2) on the same datadir. The issue remains when --no-embedded is used as well.

This is interesting. I think you might have 2 zcashd running. Is there a zcashd tab in zec-qt-wallet? That’s the embedded zcashd which is running inside zec-qt-wallet which might be in a different sync state than external zcashd.

Oh, wait. You already tried —no-embedded, so that’s not the case. Very strange.
Does  the problem remain if you right click on the green check on the bottom right and click refresh?

Oh, I think I know what the problem is. zec-qt-wallet is showing the verification percentage while zcashd is showing the block download percentage. Verification is slightly lagging block download on your machine, so the percentages are different.
Thanks for reporting this. I’ll fix it.

Fixed in v0.5.6


Angular CLI production build placing duplicates of all my fonts in the root of my dist directory. Can this be avoided?

Do you have any solutions?

Please either update this issue or create a new issue following the new issue template. Thank you!
I also suggest checking out this article in the documentation: https://github.com/angular/angular-cli/wiki/stories-asset-configuration

Bug Report or Feature Request (mark with an x)
- [ ] bug report -> please search issues before submitting
- [ ] feature request

Versions.

Repro steps.

The log given by the failure.

Desired functionality.

Mention any other details that might be useful.

this issue should be reopened.

any solution? The same thing happens to me with images as well as with the sources

facing this same issue

I am also experiencing this issue. CLI version is 1.7.0

I think this is a duplicated of #6599

@vimalraj-a have you found a solution or a workaround to your problem?

This issue should be reopened, and https://github.com/angular/angular-cli/issues/6599 too

I have the same problem with fonts and images.

That's actually not a problem as that is the intended behavior.  If preferred, the resourcesOutputPath build option can be used to place the resources in a directory.


Coordinate Transformation from Sensor Body Frame to ECEF Frame

Hi,
Can anyone please guide me how I can transfer the coordinate frame of a saved file from sensor body frame to the ECEF frame in ROS platform? My goal is to get the "Rotation matrix from the navigation frame to the global reference frame". I can currently compute the rotation matrix from the quaternion using the Matlab.But I need to transfer the sensor coordinate frame in order to achieve my goal.I have shared my current INS output from rosbag to csv format here.
https://drive.google.com/file/d/1K7hY7ff5bi_ge7hiSUyftOMOF4niFgpM/view?usp=sharing
Any help will be very appreciated.
Thanks

In the top of inertial_sense.cpp include ISPose.h. This is a header file from the InertialSenseSDK:
#"include (custom directory)/SDK/src/ISPose.h"
In inertial_sense.cpp add this code in INS2_callback function:
float vel_ned[3];
quatRot(vel_ned, msg-qn2b, msg->uvw);


DIF10: Add GCMD Check for Vertical_Resolution_Range

Is this a new check? Please describe the check and the field.
</DIF/Data_Resolution/Vertical_Resolution_Range>:
Verify that element provided adheres to the GCMD "Vertical_Resolution_Ranges" list
Describe the checks in steps

Verify that the provided Vertical_Resolution_Range exists in the GCMD Vertical_Resolution_Ranges list
If it does not exist, output an error that says "The provided Vertical Resolution Range does not comply with the GCMD."

Is there a similar check already implemented? Please list the name of the check
N/A
Additional context
N/A

this was fixed in the feature/ummg_support branch


[OTHER] add ma’am to list

What would you like to share?
I saw that this one got through the bot check today as someone used it to address me
Additional information
No response

Hey , I would like to work on this. we have to add it to the allow list in config.ts right?

Yep in the allowlist of config.ts, following the alphabetical order

Yep in the preventwords of config.ts, following the alphabetical order

Also I need to remove it from prevent words also right?

Yep in the preventwords of config.ts, following the alphabetical order

But it's already there 🤔. So is the issue already solved

Yep it's there so I guess we can close it. Not sure how that person got through the bot check tho


Tax -The system message does not include tax amount change message if changing currency

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.65-5
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Action Performed:
Precondition:

The user is logged in
Workspace has enabled tax rate.
Workspace currency default is tax exempt (0%).
Foreign currency default have a different tax rate (non-zero rate).

Navigate to https://staging.new.expensify.com/
Select a workspace from preconditions
Go to workspace Inbox chat
Submit an expense in local currency by clicking on '+' button in the left bottom corner
Go to the transaction thread by clicking on the amount card
Click on Amount and change it to a foreign currency
Click Amount again
Change it back to local currency

Expected Result:
The system message should include tax amount change when tax rate changes as a result of switching currency
Actual Result:
The system message does not include tax amount change when the tax rate changes as a result of switching currency
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/user-attachments/assets/88655dd3-6c4a-4430-b820-4ac906b56250

View all open jobs on GitHub

Not a bug


--label==""

Thanos, Prometheus and Golang version used

What happened
What you expected to happen
How to reproduce it (as minimally and precisely as possible):
Full logs to relevant components

Anything else we need to know

dfa


Naming Convention for the Common Data Model (CDM)

Is there Naming Convention for the Common Data Model (CDM)?
For example, how should to name a person according to CDM, John Smith or JOHN SMITH or else?

When it comes to naming conventions for defining objects like entities and attributes, we try to adhere to camel-casing as a JSON standard. Anyplace that schema elements are defined, we expect camel-case. When making reference to CDM objects, you can expect camel-case to be used, remembering that JSON is case sensitive.
What you have shown with "John Smith" appears to be your data value. What is managed by CDM is primarily schema, not data. The only place data definitions tend to appear is in Constant entities:  representations of known enumerations (what Dataverse calls "choices" and used to call "option sets"). In these cases the values in the Constant entity can be in whatever format makes sense for the context - they generally map to sets of values already defined by the application.
FWIW, CDM is not designed to be a place to keep the data, but as a way to represent the schema that keeps the data.

Many thanks for your quick and clear answer!

Hello berikorazov. We note that you re-opened this issue immediately upon closing it. Was that intentional? If you have any further questions, please let us know. Otherwise, would you consider this issue closed?


Performance problem of method MQClientInstance#findBrokerVersion.

BUG REPORT

Please describe the issue you observed:

What did you do (The steps to reproduce)?
Method MQClientInstance#findBrokerVersion will invoke a RPC for each single pull when the brokerVersionTable not contains the specified key, which will cause a heavy load.

What did you expect to see?
Method MQClientInstance#findBrokerVersion can cache the version info after a query though RPC.

What did you see instead?
Version info did not cached.

Please tell us about your environment:
None.

Other information (e.g. detailed explanation, logs, related issues, suggestions how to fix, etc):
None

close dut to time too long.


Site Performance Recently

Somewhere in the last few months, the site has become very slow.  Most notably hitting the dropdown for a bot to select details.  It takes quite a few tries sometimes and is very slow.  The site also feels very sluggish in general compared to where it was a few months ago.
It feels inconsistent, sometimes it isn't so bad, other times it feels it really struggling.  I typically have most bots filtered out as I keep the filter at 50 minimum.

@themarkymark-steem is this still happening? I've been checking since you've submitted this issue and I haven't noticed it be slow at all.

Yes, clicking "Actions" is painfully slow a lot of the times and requires many clicks, sending video.

Problem solved by disabling uBlock Origin on the site, it appears to be what is slowing it down for me.  Telling it to ignore steembottracker.com gives me near-instant responsiveness now.


Emit undocumented infix operators as operators

js_of_ocaml uses ## and ##. respectively for JavaScript methods and properties. Those are accepted by ocamlc's parser as (undocumented) infix operators and represented as functions applied to two arguments (their operands):
$ cat infix.ml
let _ = a##b
$ ocamlc -dsource -dparsetree infix.ml
[
  structure_item (infix.ml[1,0+0]..[1,0+12])
    Pstr_value Nonrec
    [
      <def>
        pattern (infix.ml[1,0+4]..[1,0+5])
          Ppat_any
        expression (infix.ml[1,0+8]..[1,0+12])
          Pexp_apply
          expression (infix.ml[1,0+9]..[1,0+11])
            Pexp_ident "##" (infix.ml[1,0+9]..[1,0+11])
          [
            <arg>
            Nolabel
              expression (infix.ml[1,0+8]..[1,0+9])
                Pexp_ident "a" (infix.ml[1,0+8]..[1,0+9])
            <arg>
            Nolabel
              expression (infix.ml[1,0+11]..[1,0+12])
                Pexp_ident "b" (infix.ml[1,0+11]..[1,0+12])
          ]
    ]
]

let _ = ## a b
File "infix.ml", line 1, characters 9-11:
Error: '##' is not a valid value identifier.

However, the source code corresponding to this AST (copied from ocamlc -dsource's output, just above the error message) isn't syntactically valid anymore:
$ cat prefix.ml
let _ = ## a b
$ ocamlc -dparsetree prefix.ml
File "prefix.ml", line 1, characters 8-10:
Error: Syntax error

Could we pretty-print "operators" starting with # the same way as we do for +, -...?
/cc @vasilisp

CC @let-def :).

It seems it only affects the OCaml backend:
$ cat infix.re
let _ = a##b;
$ refmt -p ml infix.re
let _ = ## a b

This was a bug in the OCaml ast pretty printer that has been fixed in 4.06, note also that extension-only  operators like ## are documented in the language extension part of the reference manual : http://caml.inria.fr/pub/docs/manual-ocaml/extn.html#sec269 .

Thank you!

Going to call this fixed then


allow additional filtering on Timber\Post::terms() retrieval when calling wp…

…_get_post_terms
Issue
Getting the terms from a Timber\Post object using Timber\Post::terms() can not be filtered by terms that do not have a parent (they are the top-most-level term)
Solution
Add an additional argument to the method Timber\Post::terms that can be passed to the function wp_get_post_terms that is called within the method
Impact
Will have:

Next to no impact on codebase
Be backwards compatible
Allow much better filtering for the user

Usage Changes
Calling Timber\Post::terms now accepts a 4th argument that is an array of "Term query parameters", see WP_Term_Query::__construct() for supported arguments.

Coverage decreased (-1.4%) to 92.652% when pulling d5ea3fdba5a19fac84f4dfb16f7234b32eef3a61 on mdahlke:master into 5278a0ffcc701e796802ec14be9aa4e1883514d4 on timber:master.

@mdahlke Thanks for this pull request! I agree that we need this. Just today I ran into the same issue and had to solve it with a separate function. I’m very glad you raised this issue and provided a proposal for a solution 👍. It’s really great if we have so many developers who bring in their thoughts! 😎 But sometimes a little change like this can have a bigger impact than you initially thought or raise a bigger discussion.
Adding another parameter to the function might seem like the easiest solution. But then we’d have one more optional parameter in that function. I’m always hesitant about adding too many optional parameters, because then some questions arise. How should we define which parameter is the most important? If somebody wants to change only the last parameter, does he always need to check the defaults for the other parameters and pass them as well?
There are other areas in Timber where we have the same problem. I always propose to use an arguments array instead, because that seems like a good compromise in a situation like this. In this very particular case of getting terms, I’d even want to make a strong case for this:
When using Timber::get_terms(), we already need to pass the taxonomy along with other parameters. We use the same pattern that is used for WordPress’s get_terms() function. Couldn’t we use the same pattern here, so that we merge the tax and args into one array?
$terms = $post->terms( array(
    'taxonomy' => 'custom_tax',
    'orderby'  => 'count',
) );

I think it’s not important to know for a developer that in the back, Timber uses wp_get_post_terms(), which takes different parameters than get_terms().
Of course this would be a breaking change and we’d have to merge it into the 2.x branch instead. When we do that, I’d even go further and say that we use only one parameter:
$terms = $post->terms( array(
    'query' => [
        'taxonomy' => 'custom_tax',
        'orderby'  => 'count',
    ],
    'merge'      => false,
    'term_class' => 'My_Term_Class'
) );

An arguments array allows you to only pass the parameters that you want to change. Remember, we just merged in a pattern like this in #1778 for Timber\PostQuery:
$persons = new Timber\PostQuery( array(
    'query' => array(
        'post_type' => 'person',
    ),
    'post_class' => 'Person',
) );

Did I miss something? Any other opinions (@pascalknecht, @jarednova, @chrisvanpatten)?

@gchtr I totally agree. I like the pattern of #1778 where we have ~3x arguments. This could be implemented here in a way that doesn't break backwards compatibility via an adapter function (not sure if that's what it's called, but basically function overloading) that will run a modified function when that first argument is a two-dimensional array.
@mdahlke: is that something you'd be up for?

@jarednova I think that @gchtr solution is a much better way of implementing this than my quick and dirty solution. I say go that direction with it instead of just arbitrarily adding another argument.

@jarednova @mdahlke @pascalknecht Okay, I’m going to try to add this without breaking backwards compatibility 👍.

Thanks again for the PR @mdahlke! @gchtr ended up with a broader solution in #1845, but this is what got the ball rolling. So, while we're not going to merge in this PR know that it shall burn in our hearts forever — thanks again for contributing to 🌲 !


Use compiler_error on nightly

On stable it leaves the usual non-friendly random compiler error.

Coverage remained the same at 87.824% when pulling 887ba19dc0a777aa50bd1b4a2121462c46969466 on lu-zero:master into 091129df1a90b9c4aff45e456c7b7ba10c766652 on Geal:master.


Multipart/form-data request inference - Incorrect

Docs says:

Multipart/form-data request inference
The [ApiController] attribute applies an inference rule when an action parameter is annotated with the [FromForm] attribute. The multipart/form-data request content type is inferred.
To disable the default behavior, set the SuppressConsumesConstraintForFormFileParameters property to true in Startup.ConfigureServices:

I think this is incorrect, the "multipart/form-data" content-type should only be inferred for action params of type IFormFile used to upload files which normally uses multipart/form-data, and not for any type annotated with [FromForm] attribute. [FromForm] can be used for form fields encoded with URL-encoded form data as well. And I really think this is actually how it works, (Hence the suppression property name: SuppressConsumesConstraintForFormFileParameters) but the documentation is incorrect.
Please verify whether I am correct and fix the docs accordingly.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: b391017f-25e1-d5e1-c4f7-020408ade7b5
Version Independent ID: 263a7b64-2e42-bbfc-c1d3-61680438e1e4
Content: Create web APIs with ASP.NET Core
Content Source: aspnetcore/web-api/index.md
Product: aspnet-core
Technology: aspnetcore-webapi
GitHub Login: @Rick-Anderson
Microsoft Alias: scaddie

Hi @Rick-Anderson after testing it myself I found that I am completely right, when an action takes a param with the attribute [FromForm] it can still consume both "application/x-www-form-urlencoded" and also "multipart/form-data", so unlike docs states, [FromForm] does not infer "multipart/form-data request content type ".
However, I believe the docs should say that using IFormFile for the action's parameter type, that will indeed infer multipart/form-data , as I have tested it and see that this is indeed the case (when using IFormFile as the parameter type) then trying to call the api with content-type application/x-www-form-urlencoded will result in a 415 (media type is not supported). And to disable the default behavior, set the SuppressConsumesConstraintForFormFileParameters property to true in Startup.ConfigureService

Hi @Rick-Anderson, any news on this one?

Thanks!!!

@SDP190 Thanks for bringing this to our attention.


Bugfix so replacements are performed in the tail

The change as suggested with a new test to match :-)
Thanks :-)

looks good. will merge and deploy tonight


Built-in authenticator.

Suggestion: Built-in authenticator for authentication purposes.

Can you go into more detail?

A big nono to this. The point of the authenticator next to a password is that an unauthorized individual would, next to your password, also need physical access to your phone and your phone's unlock code. If you were to store the secret which generates authentication codes on your PC, your account can be compromised by having malware on it (keylogger for password and something that reads the secrets for 2FA). This strongly weakens the security you get for having an authenticator on your account. I would suggest closing this issue and not implement it.

Well, it's also possible to fetch secrets from phone & also you can have keyloggers on phone too, and phones unlock code is not even protection, it's a joke.

@gccltl If I handed you a phone with a 4 digit passcode with incremental wait periods for incorrect activities, you wouldn't be able to crack it in a timely manner what so over (make sure you read with time delays, as most phones do have). On top of that, most phones are now using bio-metrics with longer pass codes or phrases.
I think you're majorly missing the point of 2FA. It's supposed to provide a secondary, much more secure second layer of protection, and having that on the client itself is a security risk.
So, what I can recommend to you is using authy? They have a desktop client, if you really want to use that.

Modern phones do tend to be more secure than desktop computers, due to a combination of verified app marketplaces, tightly defined permissions, and encrypted internal storage. They're also designed by people who probably have much more experience in cybersecurity than any of us at RuneLite. 🕵

@Sharpienero Yes, you have a code, and you can try guessing it, what I'm saying that the passcode provides zero protection (placebo protection), it my stop some kid getting in dads phone, but you can get around the passcodes easily. The secrets in phone are stored as plain (not encrypted) database files.

@gccltl On an updated Android or iOS device, it is not possible to gain access to the data without knowing the passcode, since the data is encrypted.

Also, after a couple of attempts you have to wait 30 seconds before trying again. This wait time increases with every failed attempt. Good luck cracking it. You would also need physical access to the phone. Imagine someone from Venezuela attempting to hack a European player. It would not be efficient money-wise.

@Gamer1120 To elaborate a bit, hacking accounts that have authenticators on them is much easier (i.e shared passwords, cracked databases, etc) than snatching someone's phone. I agree in the idea that this ticket should be denied outright.

@Sharpienero Your comment made no sense. You are implying that is it easier to hack an account which uses password + authenticator than an account who has just a password. Do you mind elaborating on that?

I agree with what's been said here. Enabling 2FA on your account is to make sure in the cases your password is exposed, that they will need the one time pass as well.
Leaving the secret key with a 3rd party provider (e.g a RuneScape client) gives you a false sense of security, and the entire point of 2FA is gone. You leave your account in the hands of the RuneScape client to make it easier to login..
I'm surprised other clients offer this feature instead of educating the users that would use it.

This is effectively decreasing security rather than increasing it. Also, we don't want to store sensitive user data at all. This issue should be closed and put to rejected features probably.

+1 to deathbeam's comment. Completely agree.

@Sharpienero Ah, now it makes sense.

So I guess this can be closed, we will not be storing authenticator codes on our own.


add multi-account switching in notmuch module

Introduces a new config +notmuch-mail-folders which holds a list of mailfolders. The +notmuch-mail-folder config that already exists is used as the default. Nothing changes with normal usage of +notmuch/update for syncing email folders, but it now accepts a prefix argument (C-u / SPC u). When supplied the prefix argument allows selection of a mail folder.
This is a non breaking change, adding a new feature that helps users with multiple email folders without changing anything for users that do not.

@hlissner What can I do on this to make it merge ready?

Okay.


GCE on OSX

$ ./01-infrastructure-gcp.sh
Updated property [compute/region].
Updated property [compute/zone].
ERROR: (gcloud.compute.networks.create) The required property [project] is not currently set.
You may set it for your current workspace by running:

  $ gcloud config set project VALUE

or it can be set temporarily by the environment variable [CLOUDSDK_CORE_PROJECT]
$ gcloud -v
Google Cloud SDK 138.0.0
bq 2.0.24
bq-nix 2.0.24
core 2016.12.09
core-nix 2016.11.07
gcloud
gsutil 4.22
gsutil-nix 4.18

@tgraf You have to define a project where you have admin rights to create your instances. Try gcloud init to setup your account.

Right, might make sense to add it to the scripts.


line contact support

will line contact be supported in the future?

What do you mean by line contact?
Is it the contact in feet of Cassie robot?
If so, it will be possible after new Pinocchio feature is released and I update this repo for the new Pinocchio.
You then may be able to consider a line contact via couple of point contacts.

The current code cannot consider line contact by doing so.
It is no wonder it the optimization fails.
Even the optimizer converges, the solution may be meaningless.
This is because Pinocchio consider the external contact force as a corresponding joint force.
I’m not too sure, but the new Pinocchio feature can resolve it.

Thank you for being interested in this repo.


Updated readme and schema.sql typo correction

For users new to Vercel, updated the readme to accommodate for incorrect default settings. Also updated a typo in the SQL file.

For users new to Vercel, updated the readme to accommodate for incorrect default settings. Also updated a typo in the SQL file.

d35a7ffec555282abe3976002c668d67ce1d1114


Support for importing/exporting memory allocations

Vulkan has support for importing/exporting memory allocations via VK_KHR_external_memory_fd, This would be useful to avoid memory copies when IREE is integrated into a multi-device pipeline.

Related to #3378.

Duplicate of #7242. That's tracking the staging/upload buffer optimization where we'd use this with potentially several extensions we could make work.


Update interested-technical-writers.md

Added my entry as a participant for GSoD as TW.

Hi! could you resolve the conflicts and make another commit?


Add support for coming per product "libproduct.sh"

The goal is to move out from run.sh specific code to each product, put in a libproduct.sh file, and source if it's available.
Call a generic function available in both product productInit()
Once both products have their own libproduct.sh file, we can remove the if statement.

What do you see specific? So far I see only the logging and env vars capture.

v0.1.0 release in addon
v0.1.0 release in docker


Get IP of current user?

Can I use this package to get the IP or connection id of the user who is currently doing some action on the server, such as inserting a document in a collection?
This would be in order to have some superficial tracking of not logged in users. Thanks.

You can already do that with this.connection.clientAddress in any method of call, whether you are using this package or not.

Yes – thanks. The problem is that this.connection is undefined in the context where I need it:
MyCollection.before.insert(function (user_id, doc) {
  doc.authorIP = this.connection.clientAddress;
});

You can just call a meteor method and set a session var on response with the IP and/or connectionId and then you can use it anywhere on the client.
Meteor.methods({
    getConnectionInfo: function () {
        if (!!this.connection) {
            return [this.connection.id, this.connection.clientAddress];
        }
    }
})

@dalgard I suggest you read up on how this works in Javascript. It looks like you are trying to use the this variable from a method call inside a collection hook, which has a completely different API. You will need to pass that value over somehow, such as by using a Meteor environment variable.
In your case such a variable already exists in the form of DDP._CurrentInvocation, so you can do something like DDP._CurrentInvocation.get().connection.clientAddress anywhere on the server while a method is being processed. However, in other cases you will have to set such an environment variable yourself.

@riaan53 Thanks for the tip; need the value server-side, though.
@mizzao Really appreciate the info you're bringing to the table here! It's not that I don't understand javascript – I'm aware that those functions have a different context, but I thought I might leverage your library to get to the value. Your answer has helped me out beautifully, though (except for the small detail of using a "private" property, of course ;)).


Add customizable background

Both background color and checkers pattern.

It should be three things:  transparent, solid color and checkers pattern.  That's what I have in my tlRender branch.
Note that I found two things with your current implementation:

ui::checkers() with p.render->drawColorMesh() does not work on macOS (it seems macOS does not support colored VBOs?)
With your Qt and GLFW  sample code, your compositing doubles the alpha channel.  In mrv2, I separated the code to a separate offscreen buffer to avoid that.

First, by Transparent I mean the color of the viewport (I usually use a dark gray that is almost black but not quite), solid (which I usually leave it black and you can tell the square size with the viewport) and checkers which is obvious.  In mrv2, I used two offscreen buffers to do the compositing and to avoid having the checker colors in the color picker.
Regarding the macOS issue, I found it is a bug in the implementation of OpenGL color indices usage.  Find two simple examples in the zip file:
bad_macos_gl_index_colors.zip
(one ok that works and one bad that displays black).


Bundler error

Hi, when I include one of my own packages I receive following error during bundling and rebundlinf. I am not sure what am I looking at:
/Users/tomi/Github/packages/bimbam/.fusebox/cache/1.3.117/%2FUsers%2Ftomi%2FGithub%2Fpackages%2Fwebpack2-typescriptpublic%2Flouis.js/indexof%400.0.0 was not found

This error does not appear in initial build but only on rebuild ... following that Chrome goes crazy boiling CPU trying to snatch 'indexof' in HMR loop.
What is this indexof anyways? Thanks for any help!
These are references I have in my package:
"dependencies": {
    "mobx": "x.x.x",
    "mobx-react": "x.x.x",
    "rc-collapse": "^1.6.12",
    "react": "^15.4.2",
    "typestyle": "^1.1.1",
    "yester": "x.x.x"
  },
  "devDependencies": {
    "@types/react": "^15.0.10"
  }

can you nuke your .fusebox cache ?

I did, no luck, had to do npm i indexof and manually specify +indexof in bundler. It work like this so I consider this as resolved.

You can autoImport API to fix that!


Add hint about current docker context if not default

The recent addition of support for docker context is very nice :)
It could be a nice reminder for the user to add the name of the current docker context somewhere in the UI.
(if you don't want to add clunkyness, it could only be shown when this one is not default)
the information can be accessed via bash:
docker context show

Do you have a suggestion about where it should appear ?
I'm imagining a label & a menu to maybe change said context

Changing context may a bit more than straight forward. Because we'd need to restard all lazydocker internals and "reboot" the app and connect to a new docker engine.
The hint can surely be done though

Regarding where should be the name of the context shown: the Project sections seems good fit to me.
Either:

as a prefix
or as "@contextName" suffix
or as another line (probably very first one) stating "Context: contextName"