Amplify Mock  AWSAppSyncRealTimeProvider Connection error

Note: If your issue/bug is regarding the AWS Amplify Console service, please log it in the
Amplify Console GitHub Issue Tracker

Describe the bug
Create a fresh react project and add amplify to it with an API. On running amplify mock api the graphql subscriptions are not working as expected with the AWSAppSyncRealTimeProvider having a connection failed exception after timeout.
Amplify CLI Version
<EMAIL_ADDRESS>To Reproduce
Following demo from aws blog post by Nikhil Dabhade
https://aws.amazon.com/blogs/mobile/amplify-framework-local-mocking/
npx create-react-app myapp
amplify init
amplify add api
npm i aws-amplify

replace App.js with code below
import React, { useEffect, useReducer } from "react";
import Amplify from "@aws-amplify/core";
import { API, graphqlOperation } from "aws-amplify";
import { createTodo } from "./graphql/mutations";
import { listTodos } from "./graphql/queries";
import { onCreateTodo, onUpdateTodo } from "./graphql/subscriptions";

import config from "./aws-exports";
Amplify.configure(config); // Configure Amplify

const initialState = { todos: [] };
const reducer = (state, action) => {
  switch (action.type) {
    case "QUERY":
      return { ...state, todos: action.todos };
    case "SUBSCRIPTION":
      return { ...state, todos: [...state.todos, action.todo] };
    default:
      return state;
  }
};

async function createNewTodo() {
  const todo = { name: "Use AppSync", description: "Realtime and Offline" };
  await API.graphql(graphqlOperation(createTodo, { input: todo }));
}
function App() {
  const [state, dispatch] = useReducer(reducer, initialState);

  useEffect(() => {
    getData();
    const subscription = API.graphql(graphqlOperation(onCreateTodo)).subscribe({
      next: eventData => {
        const todo = eventData.value.data.onCreateTodo;
        dispatch({ type: "SUBSCRIPTION", todo });
      }
    });
    return () => {
      subscription.unsubscribe();
    };
  }, []);

  async function getData() {
    const todoData = await API.graphql(graphqlOperation(listTodos));
    dispatch({ type: "QUERY", todos: todoData.data.listTodos.items });
  }

  return (

        <button onClick={createNewTodo}>Add Todo</button>

        {state.todos.map((todo, i) => (

            {todo.name} : {todo.description}

        ))}

  );
}
export default App;

amplify mock api
npm start
Expected behavior
From the http://localhost:20002/ local endpoint using graphiql to create a mutation create a todo item. This would then be automatically updated on the http://localhost:3000/ endpoint via the graphql subscription.
Screenshots

Desktop (please complete the following information):

OS: Mac OSX 10.13.6
Node Version. 10.17.0

Additional context
The mutations are being persisted to the local database and the changes can be viewed by refreshing the page, however, the websocket is not updating in realtime.

@riacoding Mock does not yet support the new WebSocket based subscriptions, only MQTT and the code you're trying to run using the new AmplifyJS library is leveraging the new WebSocket based subscriptions, hence creating this error for you.

@attilah is there a roadmap to have this feature implemented? I understand the issue but it seems more of a regression as it was working in 1.2.4. I have been looking at the repo ,Am I right to assume that the mock would need to implement a pure WS server instead of MQTT to handle the new AWSAppSyncRealtimeProvider?

@riacoding - any success on this?

They said it’s in the pipeline for getting fixed but there is no timeline
as of yet
On Wed, Feb 19, 2020 at 10:53 AM narayanncube<EMAIL_ADDRESS>wrote:

@riacoding https://github.com/riacoding - any success on this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aws-amplify/amplify-cli/issues/2935?email_source=notifications&email_token=AACZHNSYFVXF7D4RHLINN53RDV537A5CNFSM4JWMMZF2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMJA5NQ#issuecomment-588385974,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACZHNX3W3ICTVLNMF7BOVDRDV537ANCNFSM4JWMMZFQ
.

Ok
Thanks for replying

They said it’s in the pipeline for getting fixed but there is no timeline as of yet
…
On Wed, Feb 19, 2020 at 10:53 AM narayanncube @.***> wrote: @riacoding https://github.com/riacoding - any success on this? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#2935?email_source=notifications&email_token=AACZHNSYFVXF7D4RHLINN53RDV537A5CNFSM4JWMMZF2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMJA5NQ#issuecomment-588385974>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACZHNX3W3ICTVLNMF7BOVDRDV537ANCNFSM4JWMMZFQ .

any update on this @riacoding

Not at this time. Last I heard it was on the backlog but with no priority.
Hopefully the more awareness the quicker the fix
On Sun, Mar 15, 2020 at 11:55 AM Camin McCluskey<EMAIL_ADDRESS>wrote:

any update on this @riacoding https://github.com/riacoding
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aws-amplify/amplify-cli/issues/2935#issuecomment-599249813,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACZHNX5RHDOC6QPNJFLJLLRHUQBFANCNFSM4JWMMZFQ
.

Facing this issue as well still.

we are at the second wave of god damn covid!
this thing is still not working...

+1 for this fix

Unsure if this is the same original issue above, but I filed this bug regarding Mock AppSync Subscriptions.
https://github.com/aws-amplify/amplify-cli/issues/9621

Hey y'all as noted in @mgarabedian 's comment this item is currently being tracked on #9621. Closing in favor of tracking 9621. As a workaround we can use a proxy or frontend app config to rewrite /graphql/realtime to /graphql during mock.


Apply mask to query with view

Change-type: patch

Mask is not applied properly when calling stream.emit('query', ...).

@karaxuna Can you follow up with a PR that adds a test fo this please?

@karaxuna Can you follow up with a PR that adds a test fo this please?

@LucianBuzzo https://github.com/product-os/jellyfish-core/pull/1241


Support Testcontainers framework in junit testings

What changes are proposed in this pull request?
Add testcontainers test support for relevant unit testings.
Why are the changes needed?
For certain junit test suites such as etcd related, testcontainers framework needs to be supported
Does this PR introduce any user facing changes?
N/A

alluxio-bot, merge this please.


pkg: hyperstart: Fix MTU type from string to integer

The MTU parameter was working with hyperstart but it is only because
the cast worked. This patch makes sure that we are passing the MTU
as an integer as it is expected by hyperstart.

Coverage remained the same at 65.601% when pulling b9292d2499a8c820f694b6d0d90903deb0fcef74 on sboeuf/fix_mtu into 4e3f56a880420c78ba1e0049e5b42856265bdc82 on master.

wow.
lgtm

@sameo could you take a look please ?

LGTM


installation issue

I am having a hard time installing erdapfel with npm
npm WARN deprecated<EMAIL_ADDRESS>Package no longer supported. Contact<EMAIL_ADDRESS>for more info.
npm WARN deprecated<EMAIL_ADDRESS>Please note that v5.0.1+ of superagent removes User-Agent header by default, therefore you may need to add it yourself (e.g. GitHub blocks requests without a User-Agent header).  This notice will go away with v5.0.2+ once it is released.
npm WARN deprecated<EMAIL_ADDRESS>core-js@<3 is no longer maintained and not recommended for usage due to the number of issues. Please, upgrade your dependencies to the actual version of core-js@3.
npm WARN deprecated<EMAIL_ADDRESS>use String.prototype.padStart()
npm ERR! code ENOENT
npm ERR! syscall spawn git
npm ERR! path git
npm ERR! errno -4058
npm ERR! enoent Error while executing:
npm ERR! enoent undefined ls-remote -h -t https://github.com/QwantResearch/map-style-builder.git
npm ERR! enoent
npm ERR! enoent
npm ERR! enoent spawn git ENOENT
npm ERR! enoent This is related to npm not being able to find a file.
npm ERR! enoent

Could you kindly help me out, as seems that some of the packs are outdated?
Tried erlier npm versions with no luck
cheers!

it was an issue with my git path..sorry for inconvinience caused. however, the installation proceedure is very difficult to accomplish, lots of errors, timeout in downloading dependencies etc.
i gave up

Hi!
Sorry for the difficulties you encountered, don't hesitate to ask for more help if you get interested in the project again.
If you were running on Windows, note that while the process is way more complicated than on Linux there is now a bit of documentation about how to proceed.
Sorry for the late reply.


taterclient-ddnet: 9.0.0 -> 9.0.1

https://github.com/sjrc6/TaterClient-ddnet/compare/9.0.0...V9.0.1
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 360538

x86_64-linux

:white_check_mark: 1 package built:

taterclient-ddnet

x86_64-darwin

:white_check_mark: 1 package built:

taterclient-ddnet

aarch64-darwin

:white_check_mark: 1 package built:

taterclient-ddnet


Publish new version of react-scripts

Is this a bug report?
No. Request for publish of new version of react-scripts
Issue
Snyk reports a vulnerability for<EMAIL_ADDRESS>through the dependencies<EMAIL_ADDRESS>›<EMAIL_ADDRESS>›<EMAIL_ADDRESS>›<EMAIL_ADDRESS>›<EMAIL_ADDRESS>›<EMAIL_ADDRESS>The issue is this dependency of<EMAIL_ADDRESS>https://github.com/facebook/create-react-app/blob/v2.1.8/packages/react-scripts/package.json#L51
This dependency is bumped and fixed in https://github.com/facebook/create-react-app/pull/6278 but there is not a new version on https://www.npmjs.com/package/react-scripts
Could you publish a new version of react-scripts?

This will be fixed in the 3.0 release, which should be available soon. We are not planning any more releases of the 2.x version. In the meantime you can try an alpha release of 3.0 here: #6475

Thanks for the quick response 👍


toString

Unifies toString method for the different regressions
close #1

Coverage increased (+4.7%) to 96.32% when pulling 7036d3a970788ec5b95f65037d1486f1bc637fcd on toString into ebf4c88a8c0a01745844b2248adcfd7e7f11ec6d on master.

Why all the Istanbul ignore?

in base-regression it's because in the other classes are implemented the methods, and the load function it's pretty much the same for all the regressions

IMO it is not a good reason to disable the coverage analysis. It should
only be done if it is not possible or too difficult to add tests. It is not
bad to have a coverage <100%. Means there is room for improvement.
Le lun. 26 sept. 2016 17:25, Miguel Angel Asencio Hurtado <
<EMAIL_ADDRESS>a écrit :

in base-regression it's because in the other classes are implemented the
methods, and the load function it's pretty much the same for all the
regressions
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/mljs/regression/pull/24#issuecomment-249603506, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACPmF5DcZy6-p48tXg9VbwoVn1CR2U2qks5qt-P7gaJpZM4KGo8S
.

@targos @lpatiny now the exponents are like x^2 and there's test cases for the base-regression, I still have the doubt if the toLaTeX should have the same rules? I change it to have f(x), but I would like that you see the test cases for that and tell me what you think how is it now

Coverage increased (+3.6%) to 95.186% when pulling 1254294bb1bc754c42cecbcb351175cb1c87f308 on toString into ebf4c88a8c0a01745844b2248adcfd7e7f11ec6d on master.

For the the way it looks now is really good

@lpatiny but those are not the ones with the latest changes, but indeed it shouldnt change the rendering

There is something weird with the rendering. If I write one of the equations that have fractions in latex, I get this:

But in the

Luc's comment, It looks like the value over and the fraction line is bold.

then this is what you expect?
@mljs/collaborators

ping @mljs/collaborators

I wonder if when you write a f(x) the 'f' should not be a little bit more fancy.
If I look in wikipedia: https://en.wikipedia.org/wiki/Function_(mathematics)
I see something like

I found this package in LaTeX, but I'm still not convinced in force the user to use an extra library https://www.ctan.org/pkg/mathrsfs

I think actually that it will be rendered correctly based on some example

I think you should merge the code


add description new plugin wordcloud

There is an issue with the line endings I think https://docs.github.com/en/github/using-git/configuring-git-to-handle-line-endings#global-settings-for-line-endings. Merged via c74a0dd6, thanks.


can't run on win10

go install<EMAIL_ADDRESS># github.com/unionj-cloud/go-doudou/svc
C:\Users\CF\go\pkg\mod\github.com\unionj-cloud\go-doudou@v0.7.2\svc\svc.go:305:12: undefined: syscall.Kill

@xcrossed issue is fixed. Please try go install<EMAIL_ADDRESS>Thank you for your feedback!


Cannot drag chart on x axis

Description
Chart does not load with most recent candle in view, and you cannot drag the chart along x axis
Steps to Reproduce

Go to UNIDAI.MF21 marke
Try to drag the chart to get to the latest candles

Expected behavior

Chart should load with most recent candle showing
Should be able to drag the chart along its x axis

Screenshots

Device details
Desktop (please complete the following information):

OS: macos
chrome
Version 111.0.5563.64

zoom is also broken (probably related to the same cause)

https://user-images.githubusercontent.com/13255539/229072410-fb1b0b6a-f529-45db-9075-fbeedf53b2f2.mov

The proximate cause is that the data going into the chart is in reverse time order, i.e. the first element of the array is the latest candle and the last element is the oldest candle. Presumably this changed at some point in the past few months/weeks.
We had something similar with the depth chart last year, where it was being sent out of order data.
That this keeps occurring suggests to me that I drop the expectation that the data is sorted when passed to the charts and sort the data inside the pennant library. I've created an issue for this: https://github.com/vegaprotocol/pennant/issues/808.
If you want an immediate fix I would sort the data in the trading application.


Bug fix 3.7/database initial syncer needs patience at times

Scope & Purpose
Wait for getting keys for initial sync more intelligently

[x] :hankey: Bugfix (requires CHANGELOG entry)
[ ] :pizza: New feature (requires CHANGELOG entry, feature documentation and release notes)
[x] :fire: Performance improvement
[ ] :hammer: Refactoring/simplification
[x] :book: CHANGELOG entry made

Backports:

[ ] No backports required
[ ] Backports required for: (Please specify versions and link PRs)

Related Information
(Please reference tickets / specification / other PRs etc)

[ ] Docs PR:
[ ] Enterprise PR:
[ ] GitHub issue / Jira ticket number:
[ ] Design document:

Testing & Verification
(Please pick either of the following options)

[ ] This change is a trivial rework / code cleanup without any test coverage.
[ ] The behavior in this PR was manually tested
[ ] This change is already covered by existing tests, such as (please describe tests).
[ ] This PR adds tests that were used to verify all changes:

[ ] Added new C++ Unit tests
[x] Added new integration tests (e.g. in shell_server / shell_server_aql)
[ ] Added new resilience tests (only if the feature is impacted by failovers)

[ ] There are tests in an external testing repository:
[ ] I ensured this code runs with ASan / TSan or other static verification tools

Link to Jenkins PR run:
https://jenkins01.arangodb.biz/view/PR/job/arangodb-matrix-pr/13549/
Documentation

All new features should be accompanied by corresponding documentation.
Bugs and features should furthermore be documented in the CHANGELOG so that
support, end users, and other developers have a concise overview.

[ ] Added entry to Release Notes
[ ] Added a new section in the Manual
[ ] Added a new section in the HTTP API
[ ] Added Swagger examples for the HTTP API
[ ] Updated license information in LICENSES-OTHER-COMPONENTS.md for 3rd party libraries

External contributors / CLA Note
Please note that for legal reasons we require you to sign the Contributor Agreement
before we can accept your pull requests.

http://jenkins.arangodb.biz:8080/job/arangodb-matrix-pr/14025/

Tests blue except unrelated known failure.

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/14040/

Tests blue.


Updated demo file

The definition that overrides the default template required a wrapping div with the panel class

LGTM.  Thanks.


Make transform styles work

I've picked up where @paulyoung left off in #6 so this pull request makes the transform styles work (shadowOffset did already worked previously):

Thanks @paulyoung and @mk for this!


Configurable web controller - refactor

I think we need a meeting to discuss most of this, looking through the changes here I feel there is a misunderstanding of quite of few decisions made in the previous PR.


Add full Stackable functionality for UIStackView derivatives

Right now, if you make a class deriving from UIStackView, you don't get full Stackable functionality on it, as the various extensions in this library only apply to UIStackViews. This PR fixes this issue.
Additionally, I wrote tests to support the development of this PR. To that end, I've added the test files as a target in the package itself. Because this gives us an easy point of entry for testing from the command line, I've also added automated testing via GitHub Actions.

Thanks @heyltsjay and @chrisballinger!


New cluster_mask property in Clustering

compute it at the beginning
with new _update() method which updates cluster counts and masks

New BaseClusterInfo class for holding cluster-dependent information
ClusterMetadata(BaseClusterInfo) holds cluster information as specified by the user: cluster group and cluster color for now
ClusterStats(BaseClusterInfo) provide functions computing cluster statistics with cache invalidated after clustering actions:

quality
number of unmasked channels
cluster mask
mean channel
number of spikes
ACG
mean waveform(s)
firing rate histogram


feat: Workflow to publish wasm plugin tester as Docker container

This manually-executed workflow builds and publishes the wasm plugin tester.
I confirmed this version works as intended on my fork:
https://github.com/martijneken/service-extensions-samples/actions/runs/9141236770

[x] Tests pass
[x] Appropriate changes to documentation are included in the PR

Very cool!


[0.2.4] Connection multiplexer leak

Given 32x1000 file copy tasks its RSS creeps quite rapidly.

whatever it might have been, it's gone


Accordion: Open accordion item by default

Thinking something like this could allow for the accordion item with value="2" open by default
{{#bs-accordion default="2" as |acc|}}
Would this be easy to achieve (or is it already possible)? I know I can probably do it in jQuery for my project but trying to avoid jQuery.
Thanks!
{{#bs-accordion default="2" as |acc|}}
  {{#acc.item value="1" title="First item"}}

Lorem ipsum dolor sit amet, consectetur adipisicing elit. Consectetur ea eos hic neque quia sequi sunt. Amet at atque corporis earum et fugiat illum magnam nisi sapiente voluptatem. Laudantium, quam?

  {{/acc.item}}
  {{#acc.item value="2" as |aitem|}}
    {{#aitem.title}}
      Second item
    {{/aitem.title}}
    {{#aitem.body}}

Lorem ipsum dolor sit amet, consectetur adipisicing elit. Consectetur ea eos hic neque quia sequi sunt. Amet at atque corporis earum et fugiat illum magnam nisi sapiente voluptatem. Laudantium, quam?

    {{/aitem.body}}
  {{/acc.item}}
  {{#acc.item value="3" title="Third item"}}

Lorem ipsum dolor sit amet, consectetur adipisicing elit. Consectetur ea eos hic neque quia sequi sunt. Amet at atque corporis earum et fugiat illum magnam nisi sapiente voluptatem. Laudantium, quam?

  {{/acc.item}}
{{/bs-accordion}}

Set selected (see docs). Does that do what you want?

Thanks, it does exactly. I must have missed it as I was reading through the docs!


Update for compatibility with pyrate-limiter v3

Has there been any progress on this issue? I am getting this error message:
    from .requests_ratelimiter import *
  File "~/anaconda3/lib/python3.11/site-packages/requests_ratelimiter/requests_ratelimiter.py", line 8, in <module>
    from pyrate_limiter import Duration, Limiter, RequestRate
ImportError: cannot import name 'RequestRate' from 'pyrate_limiter' (~/anaconda3/lib/python3.11/site-packages/pyrate_limiter/__init__.py)```

It's supposed to be pinned to ^2.8, but it looks like I neglected to do that for the conda-forge recipe. I'll get that updated.
As for moving to pyrate-limiter v3, it seems like there are a few features missing (which the maintainer is still working on), so I'm holding off on this issue until that's sorted out.

This has been fixed on conda-forge in v0.4.2.

This has been fixed on conda-forge in v0.4.2.

Great, thanks!

It's supposed to be pinned to ^2.8, but it looks like I neglected to do that for the conda-forge recipe. I'll get that updated.
As for moving to pyrate-limiter v3, it seems like there are a few features missing (which the maintainer is still working on), so I'm holding off on this issue until that's sorted out.

Hi JWCook - Thanks for maintaining this lib. Just curious if this can be upgraded to work with pyrate-limiter v3 now? :)
Thanks!

Probably! I will take another look soon to see if pyrate-limiter v3 has re-implemented all the features I need. If not, I'll submit some issues and/or PRs for whatever is missing.

Probably! I will take another look soon to see if pyrate-limiter v3 has re-implemented all the features I need. If not, I'll submit some issues and/or PRs for whatever is missing.

Look forward to it. Thanks very much!

The main thing still missing from v3 is the multiprocess-compatible SQLite backend I originally submitted here: https://github.com/vutran1710/PyrateLimiter/pull/68
That's a pretty important feature that I and others are using via requests-ratelimiter. I made an issue for that upstream.
Is there a specific feature you'd like to use in pyrate-limiter v3? Just wondering if there's a short-term workaround I could provide.

Thanks for following Jordan - much appreciated!
I've managed to get a workaround for now, so it's not a problem. I just run in to issues when upgrading packages via pip as it results in some conflicts with other packages that I use. However that's a minor issue for now as I can manually sort it out by deleting the latest pyrate-limiter package and just reinstalling requests-ratelimiter, which then installs the appropriate version of pyrate-limiter.
Hopefully the upstream issues get resolved soon!
Thanks again for following up on this!

Is there a specific feature it provides that you'd like to use? Just wondering if there's a short-term workaround I could provide.
@JWCook to add to this discussion, I'm personnally interested by the new PostgresBucket backend.

My use case is a package running API calls but used in different pods/containers: I need to track this whatever the actual number of pods currently running. I was thinking of either trying to create a new custom bucket on a S3 storage system or using the PostgresBucket (which is not supporting async, and I'm not sure about the implications yet)...
Note : if not that, the package I'm speaking of has it's own way of tracking calls which is stored in a parquet file, which could easily be stored on a S3... But I'm way more in favor of using requests-ratelimiter, though I'm not sure that's (yet) possible...
Do you have any suggestions?


def transcription factor TFIID complex

TFIID complex pays a central role in core promoter
recognition and (the regulation of ?) transcription
initiation by PolII.
TFIID is composed of TATA binding protein (TBP) and a
number of TBP-associated factors (TAFs).
Reported by: ValWood
Original Ticket: "geneontology/ontology-requests/1530":https://sourceforge.net/p/geneontology/ontology-requests/1530

thanks www.movied.org


Fix the default value for Ruby Hash.

LGTM

What is the motivation for the second commit?

Oops, I've forgotten I sent a PR previously -- I'm revisiting this by a different approach. Close the PR -- will create another PR with the new approach. Sorry.


fields with this.hasOne are not insertedOrUpdated correctly and are given a value from a different relationship

TL;DR; fields with this.hasOne are not insertedOrUpdated correctly and are given a value from a different relationship.
I have a Resume model with the following fields:
    ....
      qualities: this.hasMany(DualInputValue, "resume_id"),
      thriveProfessionally: this.hasOne(DualInputValue, "resume_id"),
      whatColleguesNeedToKnow: this.hasOne(DualInputValue, "resume_id"),
      thingsInFutureProjects: this.hasMany(DualInputValue, "resume_id"),

All of these use the same model DualInputValue, this model has the following fields:
      id: this.increment(),
      resume_id: this.attr(null),
      type: this.attr(""),
      dutch_value: this.attr(""),
      english_value: this.attr(""),

Basically, the resume contains two input fields where the user can give a Dutch and English translation for the same form entry.
Now here is the problem, when I create a new Resume I give several default values only they end up mixed. This only goes for the Resume fields which have a "this.hasOne(DualInputValue, "resume_id")" value:
How I create the new Resume:
// mutations
const mutations = {
 createResume(state: any, resume_id: number) {
    Resume.insertOrUpdate({
      data: {
        id: resume_id,
        qualities: [{ resume_id, type: "quality" }],
        thriveProfessionally: {
           resume_id,
           type: "thriveProfessionally",
           dutch_value: "LOL",
           english_value: "BBQ"
        },
         whatColleguesNeedToKnow: {
           resume_id,
           type: "whatColleguesNeedToKnow",
          dutch_value: "LOL",
           english_value: "BBQ"
        },
        thingsInFutureProjects: [
          {
            resume_id,
            type: "thingsInFutureProjects"
          },
          {
            resume_id,
            type: "thingsInFutureProjects"
          },
          {
            resume_id,
            type: "thingsInFutureProjects"
          }
        ],
        certificates: [
          {
            resume_id,
            type: "certificates"
          }
        ],
        competenties: [
          {
            resume_id,
            type: "competenties"
          }
        ],
        studies: [
          {
            resume_id,
            type: "studies"
          }
        ],
        hobbiesAndInterests: [
          {
            resume_id,
            type: "hobbiesAndInterests"
          }
        ],
      }
    });
  }
}

And then I would like an output of all data including the relationships of this resume:
// getters
const getters = {
  resumeData: (state: any) => (id: number) => {
    return Resume.query()
      .whereId(id)
      .withAll()
      .first();
    }
};

But I end up with the following results:
{
  "id": 431,
  "firstName": "",
  "lastName": "",
  "region": "",
  "nationality": "",
  "driversLicenses": [],
  "languages": [],
  "educations": [],
  "qualities": [],
  "thriveProfessionally": {
    "id": 18,
    "resume_id": 431,
    "type": "hobbiesAndInterests",
    "dutch_value": "",
    "english_value": ""
  },
  "whatColleguesNeedToKnow": {
    "id": 18,
    "resume_id": 431,
    "type": "hobbiesAndInterests",
    "dutch_value": "",
    "english_value": ""
  },
  "thingsInFutureProjects": [
    {
      "id": 12,
      "resume_id": 431,
      "type": "thingsInFutureProjects",
      "dutch_value": "",
      "english_value": ""
    },
    {
      "id": 13,
      "resume_id": 431,
      "type": "thingsInFutureProjects",
      "dutch_value": "",
      "english_value": ""
    },
    {
      "id": 14,
      "resume_id": 431,
      "type": "thingsInFutureProjects",
      "dutch_value": "",
      "english_value": ""
    }
  ],
  "certificates": [
    {
      "id": 15,
      "resume_id": 431,
      "type": "certificates",
      "dutch_value": "",
      "english_value": ""
    }
  ],
  "competenties": [
    {
      "id": 16,
      "resume_id": 431,
      "type": "competenties",
      "dutch_value": "",
      "english_value": ""
    }
  ],
  "studies": [
    {
      "id": 17,
      "resume_id": 431,
      "type": "studies",
      "dutch_value": "",
      "english_value": ""
    }
  ],
  "hobbiesAndInterests": [
    {
      "id": 18,
      "resume_id": 431,
      "type": "hobbiesAndInterests",
      "dutch_value": "",
      "english_value": ""
    }
  ]
}

What happens is that  "thriveProfessionally" and "whatColleguesNeedToKnow" are both given a type of "hobbiesAndInterests". Weirdly enough, if I change the order of the data in the createResume function, then a different wrong type is inserted.
It seems to take the last field given in the data {}, takes that type and then uses that for fields which have a "hasOne" relationship.
Thank you for reading!

#437 Hi! thank you for the report.
Well, this behavior kind of happening by nature of the RDM data structure. The problem is that you have 2 hasOne relationship.
Has One relationship assume that Resume only has one DualInputValue model relationship in the store. With your example, there will be many DualInputValue with same resume_id. And Has One relationship will stop looking up DualInputVal when it finds the first matching record. That is why you will get a different result if you change the array order.
In this case, you shouldn't be using Has One relationship, because the relationship is not only one, it has many. Instead, filter the result based on type filed. You could query relationships like this.
Resume.query().with('thriveProfessionally', (query) => {
  query.where('type', 'thriveProfessionally')
}).with('whatColleguesNeedToKnow', (query) => {
  query.where('type', 'whatColleguesNeedToKnow')
}).get()

Thank you so much for explaining this! I managed to solve this thanks to your explanation!


Queue tweeter

This PR shows a proposed design for a class that pulls from a queue and tweets. I like that the message_transformer function can be used for people to customize the class to send any kind of tweet. (I am thinking of sending polls!)
One drawback is that I removed some of the customizability of send_tweet in terms of being able to pass in different combinations of API/Auth/Client. If thats important, it could be brought back, but I wasn't sure of the use case for it so I simplified the twitter auth specifying.
Let me know what you think.

Glad you like it! I'll update the README with several examples and send a PR for the updated README.
Here's a poll tweet sent by QueueTweeter:
https://twitter.com/JustPyThings/status/1573680143154880512
Regarding the tests, I'm fairly new to pytest so I'm not sure if I mocked everything in the optimal way, feel free to rejigger things if there's a better way.


feat: add 4844 header fields and consensus checks

This adds EIP-4844 header fields:

blob_gas_used
excess_blob_gas

And their according consensus header validation checks. The method calculate_excess_blob_gas is added which is specified in the EIP-4844 header extension section.
RLP encoding and decoding are updated in a similar way to the withdrawals root.
Fixes #3941

failing proptest cases, needs an encoding / decoding fix - working on it

will find some reference RLP data, may have to generate it using geth code

Added RLP tests from devnet-7 and EF tests


MINOR: Clean-up client javadoc warnings

More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.
To validate fix run ./gradlew javadoc.  Note that this PR only addresses client javadoc warnings, there's #9461 for the streams warnings
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Only javadoc changes, so merging

Merged #9463 into trunk

cherry-picked to 2.7


Autofill is disabled on Android

I'm loading a form that has a autofill field, but on Android isn't working.
On console is showing:
W/cr_AwAutofillManager(28324): Autofill is disabled: AutofillManager isn't available in given Context.
W/IInputConnectionWrapper(28324): getSelectedText on inactive InputConnection
W/IInputConnectionWrapper(28324): getTextAfterCursor on inactive InputConnection

Hi, I am also having the same issue. Any update on this? Thanks!

same here!

Solution for this? Anyone known that?


Ramadan 2018 goals, tasks, features

Salam all,
Wanted to create this issue as a discussion for Ramadan 2018 goals, tasks, features and bugs. Anyone in the community is welcomed to add comments and/or suggestions. Feel free to comment suggestions and I could add them to a list in this comment (I will edit it continuously).
cc @ahmedre @sharabash @hammady @naveed-ahmad @sabeurthabti

The search feature needs to be worked on. The one on the legacy site works so much better. Before typing this comment again, I just tried it out and typed qulillahumma in both version. The new version didn't give any results where as the legacy site gave the exact ayah I was looking for.

We can work on improving the homepage - I think we wanted a better design anyway.
Bring quran.audio to the same standards as quran.com (maybe even share audio players_
I'd love to do some out reach at events here in the UK and get people excited and contributing to quran.com (not just code)
Maybe quran.audio Android app (I know I dropped the ball on that one)

Salam 'alaykum and jazakum Allah Khayran for all of your efforts!
I have a small set of easy proposals that don't require much work that I think may benefit the project, specifically the security of the websites.
Since there's a push everywhere to move to HTTPS I think the first task should be to:

Moving everything to HTTPS:

There are still some domains that don't even support HTTPS (legacy.quran.com, salah.com, ...) Fortunately Let's Encrypt is free and easy to setup (with automation tools like certbot.eff.org).
Once done one can add them to the HSTS preload list to make sure that they're always loaded over HTTPS (they can be included there even if they don't meet the full criteria, I think but I may be wrong, of the HSTS preload list hstspreload.org by contacting its maintainers).

Ameliorating security headers

For example adding a CSP and set the Secure flag for cookies, as well as adding Subresource Integrity for scripts that are loaded https://observatory.mozilla.org/analyze/quran.com

Moving to jsDeliver whenever possible

One of the reasons (besides the speed, the number of CDNs that they use, ...etc) is that they are guaranteed to not be blocked in China.

Hello everyone, I've only known about quran.com today as I was thinking about implementing something similar but seems like it's a better idea to help this project than to start a new one.
Unfortunately I'm mainly a python developer with some intermediate experience in the frontend land [ angular & angularjs ] so It wouldn't be easy for me to contribute in the frontend or backend parts, but I'll surly follow the issues and see if I can help with anything or to see if I can help with any of the python based parts of the project.
As for suggestion of new features or improvements,
One thing that will be very good to have is Tafser of each Aya in the reading page.
Thank you all for the amazing effort and hope to be of any help to this amazing project.

@mxahmed  React is not as hard. I promise. I am also a python dev but i was able to learn react in a few weeks and start contributing. InshaAllah may Allah make it easy for you. I also want to embark on a python project here at quran.com for teaching people how to learn classical Arabic with AI not theafter ramadhan inshaAllah and it will be in python maybe we could work together then inshaAllah.

I don't think the search needs smartening — it needs dumbing down. Take the term that quran.com suggests: "Noah". The 2nd result doesn't even have Noah in the ayah — it's just mentioned in between brackets by some of the translators.

Assalamu Alaikum
We could add to the goals moving of statics artefacts of quran.audio to CDN and maybe issue quran/audio.quran.com#105?

@mustafa0x i agree. we need to fix search again. it used to be much much better. it's just a big pain to tweak :(


added clickhouse operator added to nodeSelector

@medicharlachiranjeevi Thanks for raising the PR. Can you also add a few lines on what does this PR do so that reviewers can review accordingly

Added node selector if useNodeSelector: true for clickhouse operator.(clickhouse:true)

Added nodeSelector label for ClickhouseOperator


Using HTTP instead of CURL

I've come across certain scenarios, where CURL would have a blank response while HTTP response confirms the sub-domain is redirected.
Basis the fingerprint by 0xpatrik for Amazon AWS at https://github.com/EdOverflow/can-i-take-over-xyz/issues/36.
Wouldn't using HTTP instead of CURL result in fewer false positives?

Maybe? This would need some digging and a vulnerable subdomain to compare both ways and seeing what results in a TP. Do you have an example?

I was looking at the status of brochure.belkin.com via tko-subs:
brochure.belkin.com,s3-website-us-west-1.amazonaws.com.,amazonaws,true,false,Can't CURL it but dig shows a dead DNS record
The curl's response was blank, while the HTTP returned:
HTTP/1.1 301 Moved Permanently
Content-Length: 0
Date: Tue, 28 May 2019 23:28:28 GMT
Location: http://www.belkin.com/us/belkinbusiness/
Server: AmazonS3
x-amz-id-2: 63dEzBVOsJaPo3cFk0zEsihspVuTYAxeGHLgsigexQe8nxCh/6l4LI0f3SVEehmIPHIhOAI3gmY=
x-amz-request-id: 627231689EBCB87D
This is just one of the examples I was looking and saw the false positive.

Since it had been a while since we wrote this, I took a look at the code and found this - https://github.com/anshumanbh/tko-subs/blob/master/tko-subs.go#L475
It looks like we default to assume its vulnerable if curl doesn't return anything. I assume this is reasonable as compared to calling it not vulnerable.
@mhmdiaa what do you think?

@anshumanbh instead of curl, is it possible we test the http command. The outputs of both can then be compared to see lesser false-positives.

Hi @sumgro and @anshumanbh!
I'm going to explain the issue quickly to make sure we are on the same page.
The reason tko-subs assumes brochure.belkin.com is vulnerable is, as @anshumanbh said, we've defaulted to assuming domains are vulnerable if they don't respond over https (or http if specified in the providers-data.csv file). And when this domain is requested over https, it times out (although it responds over http).
I think @sumgro's suggestion is to request each domain over both http and https and call it vulnerable only if we don't get a response over either protocol, which is a reasonable solution although it could decrease performance.
I believe this is an inaccurate data issue, so the typical way to eliminate this specific false positive would be to set "http" to "true" for "amazonaws" in the providers-data.csv file; however, I think this is a good opportunity to rethink how we're doing things. @anshumanbh, is there a reason we use https by default here instead of http? I took a random sample of our known providers and all of them seem to respond normally over http, and some of them respond normally over http only as we know, so wouldn't http make more sense as the default protocol?
So my suggested solution is to use http by default (thus solving the issue) and deprecate the "http" field of the providers' data CSV file (or maybe turn it into an "https" switch that does the opposite in case we come across a provider that works over https only)
Let me know what you think and I'll write a fix ASAP!
P.S. We don't use curl, the command-line utility, for http requests BTW. It's kind of a misnomer that has been there for a while since we used to use curl in the early versions of tko-subs. Currently, we use Go's http package. Admittedly, we should edit the code/documentation to reflect this fact.

@mhmdiaa The only reason I can think of why I included the http field in the first place is because I believe it was Tumblr or Heroku (either one of these) that wouldn't respond properly on HTTP. It had to be sent over HTTPS. I think trying both should be a reasonable middle ground here. Don't think it would impact the performance by a lot. And, maybe we can remove the http field from the providers file all together?

@anshumanbh you mean Tumblr and Heroku wouldn't respond over https? (That's what the current providers-data.csv file indicates.) They do respond over http like most of the other providers
Not sure about the performance impact. I'm going to have to run some performance tests and compare both approaches. But I mean, if we can avoid making that extra request in about 90% of the cases, this must be the better way.

Hey,
It’s been a fairly long time.
Checking back on any updates to testing use of https and http to resolve the targets.
Thanks.


Add Quadratic equation calculator

use  Ax^2 + Bx + C = 0  format

Hey! I want to work on this , can it be assign to me?🙂

sure @SOUMOJIT-CHAKRABORTY do it!

Hey @davejane25 , I would like to work on it . Please assign it to me.

completed with #19


Undefined selectedFleet in Natsuiro Listener

I was trying to use Natsuiro theme at the latest commit.

ah this happened to me as well, and the Inspection screen stays in the "Waiting" mode

This issue and #527 actually are the same bug report, just different description. :sparkles:

ah I merged back Listeners.js to the main script file xD
figured this was more trouble than I hoped for so... kek
will push later

@bamboo3250 oh lol XD

can check latest push if still error? thanks :)

@dragonjet it works :+1:

aye thnx


Implement Langchain's LLM LCEL feature to LLMs.

We now use official openai library only, and use external service like vLLM or LocalAI for running custom models.
But it is kind of hard to implement for beggingers.
Plus, many services is hard to use with openai library. (Like PaLM API, huggingface inference endpoints)
So, I think just compatible with Langchain is better. (Also, I think langchain's new LCEL is kind of cool)

great blog
I think we can implement our retrieval to LCEL using Passage => text as easily.
Like, extract contents from List of Passages.
Then, you can use your own PromptTemplate and RunnableMap for using retrievals.
The pipeline can be really simple.
Pipeline will be the same when we change to LCEL, because pipeline's inside will change, but outside usage must be same.
Because pipeline is like a cookbook for RAG workflows


the f macro is unused in some targets

r? @alexcrichton
(rust_highfive has picked a reviewer for you, use r? to override)

@bors: r+

:pushpin: Commit 9b74d9972b4f14804231da213fbb8392c3f54f50 has been approved by alexcrichton

:hourglass: Testing commit 9b74d9972b4f14804231da213fbb8392c3f54f50 with merge ac2a6928dd543315500f50a184388256e13a8817...

:broken_heart: Test failed - status-travis

@alexcrichton this fails because the beta of x86_64-apple-darwin is still not shipped with rustup (even though the beta of i686-apple-darwin now started shipping again). I manually merged a PR before that had the same issue.

:broken_heart: Test failed - status-travis

@bors: retry

:hourglass: Testing commit 9b74d9972b4f14804231da213fbb8392c3f54f50 with merge 5fb08ff82821567a4f33e5513b1f51d3c3f1761d...

:broken_heart: Test failed - status-travis

@bors: retry

:hourglass: Testing commit 9b74d9972b4f14804231da213fbb8392c3f54f50 with merge 5cc006652a242c35b6fe5b615ccd39d9356a2346...

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 5cc006652a242c35b6fe5b615ccd39d9356a2346 to master...


User interface: Add placeholder for all known ACs and Categories

This adds a placeholder file for each or the known groups of Acceptance criteria. Meaning that approbation can run and show what categories/files are still todo.
Also, tidy up the readme/index for the user-interface folder. This includes breaking staking and governance into 2 separate categories (in response to feedback)

[x] add the category JSON to this PR

I'm not well versed in github enough to review this.  As long as it doesnt do anything to break the existing core report I'm happy.
Conceptually yes I agree we should go with teh placeholder files for approbation.

@JonRay15 it won’t interfere with core. Looks fine to me @campbellssource


Cannot run with gunicorn, missing import

[2021-01-06 12:09:08 +0000] [1] [INFO] Starting gunicorn 20.0.4
[2021-01-06 12:09:08 +0000] [1] [INFO] Listening at: http://<IP_ADDRESS>:9726 (1)
[2021-01-06 12:09:08 +0000] [1] [INFO] Using worker: threads
[2021-01-06 12:09:08 +0000] [8] [INFO] Booting worker with pid: 8
[2021-01-06 12:09:08 +0000] [8] [ERROR] Exception in worker process
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/site-packages/gunicorn/arbiter.py", line 583, in spawn_worker
    worker.init_process()
  File "/usr/local/lib/python3.8/site-packages/gunicorn/workers/gthread.py", line 92, in init_process
    super().init_process()
  File "/usr/local/lib/python3.8/site-packages/gunicorn/workers/base.py", line 119, in init_process
    self.load_wsgi()
  File "/usr/local/lib/python3.8/site-packages/gunicorn/workers/base.py", line 144, in load_wsgi
    self.wsgi = self.app.wsgi()
  File "/usr/local/lib/python3.8/site-packages/gunicorn/app/base.py", line 67, in wsgi
    self.callable = self.load()
  File "/usr/local/lib/python3.8/site-packages/gunicorn/app/wsgiapp.py", line 49, in load
    return self.load_wsgiapp()
  File "/usr/local/lib/python3.8/site-packages/gunicorn/app/wsgiapp.py", line 39, in load_wsgiapp
    return util.import_app(self.app_uri)
  File "/usr/local/lib/python3.8/site-packages/gunicorn/util.py", line 411, in import_app
    app = app(*args, **kwargs)
  File "/app/rq_exporter/exporter.py", line 57, in create_app
    worker_class = import_attribute(config.RQ_WORKER_CLASS)
NameError: name 'import_attribute' is not defined
[2021-01-06 12:09:08 +0000] [8] [INFO] Worker exiting (pid: 8)
[2021-01-06 12:09:08 +0000] [1] [INFO] Shutting down: Master
[2021-01-06 12:09:08 +0000] [1] [INFO] Reason: Worker failed to boot.

Hi, thank you for reporting this error, I'll take care of it.

Hi, thank you for reporting this error, I'll take care of it.

Fixed in v1.5.1 and v1.6.0.

Fixed in v1.5.1 and v1.6.0.

@mdawar  That was quick! Thank you very much

@mdawar  That was quick! Thank you very much


Unity warns about OnMouse_ event handlers on Android build

Unity warns OnMouse_ event handlers on Android build. On Unity 4.6.2f1, the following warning is on its console when Unity builds Android application.

Game scripts or other custom code contains OnMouse_ event handlers. Presence of such handlers  might impact performance on handheld devices.
UnityEditor.BuildPlayerWindow:BuildPlayerAndRun()

I will create a pull request.

Thank you!
I've merged.


Save sidebar preference when collapsing/expanding

🚀 Feature Proposal
When the sidebar collapses or expands, I would like the setting saved so that when I refresh the page or when I use a different device, that setting is saved.
Technical Notes

Add a new database "user"
Add new user preferences object for the user
Add new redux action
Save the user preferences when the user expands or collapses the side bar

I wonder if saving the preference would be enough per cookie, rather than per user?

Good question @kumikokashii. In the first case, the settings would be saved on the browser, in the other case the user could bring back his settings on another browser/PC too.

I came up with this use case. If user was on Desktop and let's say had all of sidebar expanded. When the same user goes to another hospital and uses Tablet to log in. Screen size is much smaller, and he/she may not want all sidebar expanded? Mobile has no sidebar.

I think this issue has to be split in two:

user settings object inside user's record saved in db
sidebar preference: we have to save also the viewport of the preference so we can address what @kumikokashii said. I think we can let the user decide per three different views:
desktop
tablets
mobile (for this specific settings, mobile will not be selectable)

This feature seems to me more something for version 2.1. Am I wrong?

This feature seems to me more something for version 2.1. Am I wrong?

Agree ✋

No, you are not. We will have many of these, let's call them, "settings". However we have to finalise the document that explains the user story, before the release of 2.0.

Hey all! I see that you have decided to save this for a later milestone. I was thinking about taking a crack at it over the next two weeks and was wondering if that would be advisable by the team?
If so, the document that @fox1t mentions would be useful to see.

Just some clarification on the tasks involved:

There does not seem to currently be any implementation of "users". There are language settings which are powered by i18next from my understanding and that seems to be browser based.
This task would mean adding users schema for the db located in the core/server repos?
Does this task involve creating a sign on system for the user, and if so, what would that look like? Google OAuth? Register with HospitalRun specific credentials? Provisioned to users by the HospitalRun team?

I'll get started testing the persistence of this setting by mocking the source of truth (where the setting will be saved per user) for now.

So after some more investigating, I see that the demo on the HospitalRun website actually has an authentication and user system. Is this part of the 1.x releases and just has not been updated for the 2.0 release yet?

Hi @Elveskevtar,

We are in the process of defining the user entity: our users are CouchDB users that handles both authentication and data. May I ask you if you have any experience with CouchDB?
Yes, we need to add database user schema representation to core repo, however it depends on point 1
Yes: first we are going to define our registration and login process. We need to support our own login (internal) and also external one: the internal is handled by couchdb itself and this is the first we are going to develop. After the internal is working like expected we are going to add external ones: we going to develop a custom server side plugin that reaches external systems, authenticate the user and adds the JWT/token to couchdb to authenticate the user.

So after some more investigating, I see that the demo on the HospitalRun website actually has an authentication and user system. Is this part of the 1.x releases and just has not been updated for the 2.0 release yet?

The 1.0.0-beta has several issues about how it handled the authentication. We are building it ground up in order to support what i mentioned in my last reply here.

Thank you for the quick update! I don't have experience with CouchDB but I do have experience with NoSQL databases. I took a look at the json models to define the schema for it in core and it looks pretty intuitive.
As for authentication, that makes sense. Will users need to be connected to the internet to authenticate at first or is that the point of the external login? I have significant experience in making auth systems but that task is probably a little beyond me as a first time contributor.
Regardless, where would you recommend that I start? I think that I could get the persistence working while the user schema is fleshed out.

Hi guys. Any update on this issue can I take it?

Hi @blestab @fox1t @jackcmeyer @morrme @tehKapa could I work on this?

Hi @blestab @fox1t @tehKapa @jackcmeyer @morrme, could I please take this on?

@aguptamusic Sure !

@ArtemGoldsmith sorry that we missed your message!

Update: I'm holding off on this issue because @jackcmeyer says:
"The core team is deciding to move away from CouchDB/PouchDB in favor of some more modern offline-first practices, such as RxDB + GraphQL sync. Since this migration will be a large effort, we simply removed the login stuff along with removing the code that made syncing happen. It appears that we have left behind some of these artifacts."


[Rector] Update to Rector 0.12.17 and Re-run it

Update Rector to 0.12.17
Re-run it to apply more removal on unneded variable in assign
Apply typed property on final class protected property.
Ignore phpstan 1.4.7 notice on Binary operation "/" between string and 8 results in an error. on system/Encryption/Handlers/OpenSSLHandler.php

Checklist:

[x] Securely signed commits

All green 🎉

I think this should supersede #5764

@paulbalandan probably not yet, feel free to create failing test case to rector-src so we can try to solve https://github.com/rectorphp/rector-src, the tests mostly under rules-tests.

@paulbalandan we can update OpenSSLHandler when PR https://github.com/codeigniter4/CodeIgniter4/pull/5764 updated, I ignored PHPStan notice for now to avoid regression in functionality.

@samsonasik Thank you!


Singleton Entity Question

It seems to me that some entities can be singletons so that they are always unique and you don't need an Id to find the component underneath it. Good idea, bad idea? How would I implement it?

Hi Don :)
I wouldn't add a generic way for the singleton pattern for entities. It always depends on the specific use case. But in my ears having a component that is only on one entity sounds more like it should be a system instead of an entity. Systems are singletons by default and it may be worth considering.
If you post your specific use case we can discuss it in detail.

I can remember we had some (rare) cases in the past. One of which that comes into my mind is the idea of a "map" entity, which holds information such as the width and height of the map.
In this case, we had a single entity with one or more components holding these properties. The entity was created from a blueprint, so we could load the data from disk.
As soon as the entity was created by some system (e.g. the MapSystem), an event was raised, notifying all other systems of the id of the entity (e.g. MapInitializedEvent, with a single property, Entity = 123).

That's true, thanks for the reminder :) In case of the map the thing was that only one map was ever active at the same time, but there were multiple different maps in the game.
Then it makes sense to handle them as entities, not as a system, and inform the game systems about the creation (and deletion) of a map via events.

Closing it as the questions seems to be answered


13.2.0 causing SteamVR to start in safe mode

When i start ALVR it runs SteamVR but in save mode, telling me alvr_server is disabled. If i let it restart it's still the same.
I hadn't this problem with 13.1.0!

If you re-enable the ALVR add-on for SteamVR (details here), does is still crash? If so, please attach the SteamVR logs for review (it tells you how to get the logs on the troubleshooting page).

I allready re-enabled the plugin and restart. I don't see any crash. It just disables alvr_server (and Gamepad) again, over and over again. I have to check for the log.

Strange after some restarts and driver (un)registration it finally works. I don't know exactly how i did this. So never mind at the moment - thanks!


feat(module): component optional when generating module

As mentioned in #3365
Usage: ng g m foo --component

Noob me forgot about e2e tests. I will fix and update tomorrow.

@filipesilva  Sorry if this is out of line, but I was just wondering if there's a way to trigger the Travis build without having to commit anything. It seems the build cancelled when I pushed the last commit.

We're discussing this internally to see if that's the way we want to move forward. The other solution would be to have components generated by default and have a --no-component flag (or --component=false).

If you guys choose to go for the second option I can change the code to fit the need.

Okay we're going with this PR as is.

Could you add a simple e2e test that actually call your flag? I want to make sure we avoid regressions.

Other than adding a test, this LGTM.

@hansl Correct me if I'm wrong, but the --routing has a e2e test here with it's new functionality. Also, the module default behavior is being tested [here)(https://github.com/baruchvlz/angular-cli/blob/c97a322ba4d97618b60db6ad297b308683d879e4/tests/e2e/tests/generate/module/module-basic.ts), also with the new behavior.

@baruchvlz yes, but I want a test that actually uses --component. There's no testing on using the flag itself, which is the regression I want to avoid (say we get rid of the flag entirely later on).

The --component flag itself doesn't exist in the current code base, so I don't think there's any regression. I added the --component flag based on the suggestion in the issue and then removed it when it was decided to go a different direction, but the flag was never introduced into the code.
Sorry if I'm misunderstanding, I don't have experience working with such large projects.

No it's okay. We'll add a --component flag if/when people request it. LGTM.

I think this default is a bit confusing.
Example: https://github.com/angular/angular-cli/issues/3255#issuecomment-270818087


ci: add main.go to sonar exclusions

This file is not run in our test right now as the bootstrapping happens in the suite test file.

:tada: This PR is included in version 0.6.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


BUG Restore travis artifacts

Fixes #6046

Good to merge when green

Behat tests currently fail on CMS occasionally. Not due to this PR, which is the point of getting artifacts fixed again. :) I suggest we fix this even with the failure.

Merging as this will help diagnose the intermittent failure still present in the CMS build.


Feature: add constraint "groups"

related to #5404 it would be nice to write a plugin as
    deps.constraints( dch -> {
        dch.create( "testCompile", "org.junit.jupiter:junit-jupiter-api:5.+" ).withAlias("junit");
        dch.create( "testRuntime", "org.junit.jupiter:junit-jupiter-engine:5.+" ).withAlias("jupiter");
        dch.create(  "testImplementation", "org.assertj:assertj-core:3.8+" ).withAlias("assertj");
        dch.create( "testImplementation", "org.mockito:mockito-core:3.+").withAlias("mockito")
        dch.group("junit5").of( "jupiter", "junit" );
        dch.group("test").of( "assertj" ).with("junit5");
        dch.group( "mock-test").of("mockito").with("test");
    } );

then use it like
 dependencies {
     use("test")
 }

or
 dependencies {
     use("mock-test")
 }

which would import -api as a test compile dep, and engine as a test runtime, this is similar spring-boot-starter modules.

one thought is being able to write a Plugin<DependencyConstraintHandler> however the problem I have with this, is really 2 things 1. Plugins can't use dynamic versions... 2. I don't see it as a plugin so I'd prefer it not be in that block.

Thanks for the idea.
Work is still ongoing around constraints and concepts to group them together. Your idea seems similar to what Maven BOMs are used for and we have plans for supporting and defining similar things in Gradle.
As such, closing this issue. Please watch out for announcements around this in the future.


Classification 2 classes and 60 Querypoints

I have 2 classes and I want to see if the 60 points are one of these 2 classes. The problem is that I can’t get much data when I implement this algorithm with more than 1 nearest neighbor, cause I get a lot of -1 output otherwise.
please how can I use this algorithm with 2 classes and 60 points to be classified with k=7 without getting -1 in the output? Please can I iterate in the algorithm and adding points to the classes to be looked for as neighbors or the algorithm do that automatically?

Hi and thanks for using knn-cpp. I am happy to help you as best as I can.
First off, can you give me a bit more details on your data? Like:

What dimensionality do your points have?
What data do you use to build the KDTree?
What data do you use to query the KDTree?

Maybe you can also post some example code on how you use the KDTree, so I can get more insights into your problem.

Thank you so much for your reply as well. Actually I have solved the problem. I have used the library wrong. Now my Datapoints are the EEG Chanels Features and the query points are the reference features, which I need to look for their next k Neighbors.
The KDTree is the one in the readme file with Euclidean distance.
From the distances Matrix I do the classification.
thank you deeply


Only change peaq-dev's token/name to Agung related info

This is the first change for clarifying the peaq-dev parachain and agung parachain. Therefore, we changed the name of the peaq-dev to the agung network; however, in the runtime module, we are still using the peaq-dev but not agung runtime module

It's outdated and we've removed it already


python310Packages.databricks-cli: 0.17.1 -> 0.17.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.databricks-cli is: "A command line interface for Databricks"
meta.homepage for python310Packages.databricks-cli is: "https://github.com/databricks/databricks-cli"
meta.changelog for python310Packages.databricks-cli is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.17.2 with grep in /nix/store/065z8i3p2v0vxn072bk20a3sl2ifbl7s-python3.10-databricks-cli-0.17.2
found 0.17.2 in filename of file in /nix/store/065z8i3p2v0vxn072bk20a3sl2ifbl7s-python3.10-databricks-cli-0.17.2

Rebuild report (if merged into master) (click to expand)

6 total rebuild path(s)

6 package rebuild(s)

First fifty rebuilds by attrpath
dbx
mlflow-server
python310Packages.databricks-cli
python310Packages.mlflow
python39Packages.databricks-cli
python39Packages.mlflow

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/065z8i3p2v0vxn072bk20a3sl2ifbl7s-python3.10-databricks-cli-0.17.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.databricks-cli https://github.com/r-ryantm/nixpkgs/archive/560f62e4186e81ab3ce88119c345d8ecaaf3016d.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/065z8i3p2v0vxn072bk20a3sl2ifbl7s-python3.10-databricks-cli-0.17.2
ls -la /nix/store/065z8i3p2v0vxn072bk20a3sl2ifbl7s-python3.10-databricks-cli-0.17.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

6 packages built:

dbx
mlflow-server
python310Packages.databricks-cli
python310Packages.mlflow
python39Packages.databricks-cli
python39Packages.mlflow

Maintainer pings
cc @tbenst for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.databricks-cli is: "A command line interface for Databricks"
meta.homepage for python310Packages.databricks-cli is: "https://github.com/databricks/databricks-cli"
meta.changelog for python310Packages.databricks-cli is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.17.3 with grep in /nix/store/ppz9y59rh7yxcas6s2zwcw4yhmmfcyd7-python3.10-databricks-cli-0.17.3
found 0.17.3 in filename of file in /nix/store/ppz9y59rh7yxcas6s2zwcw4yhmmfcyd7-python3.10-databricks-cli-0.17.3

Rebuild report (if merged into master) (click to expand)

6 total rebuild path(s)

6 package rebuild(s)

First fifty rebuilds by attrpath
dbx
mlflow-server
python310Packages.databricks-cli
python310Packages.mlflow
python39Packages.databricks-cli
python39Packages.mlflow

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ppz9y59rh7yxcas6s2zwcw4yhmmfcyd7-python3.10-databricks-cli-0.17.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.databricks-cli https://github.com/r-ryantm/nixpkgs/archive/1e2363fd985f9be42f00e7158dd21ae05a50e3b0.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ppz9y59rh7yxcas6s2zwcw4yhmmfcyd7-python3.10-databricks-cli-0.17.3
ls -la /nix/store/ppz9y59rh7yxcas6s2zwcw4yhmmfcyd7-python3.10-databricks-cli-0.17.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

6 packages built:

dbx
mlflow-server
python310Packages.databricks-cli
python310Packages.mlflow
python39Packages.databricks-cli
python39Packages.mlflow

Maintainer pings
cc @tbenst for testing.

Result of nixpkgs-review pr 187506 run on x86_64-linux 1

6 packages built:

dbx
mlflow-server
python310Packages.databricks-cli
python310Packages.mlflow
python39Packages.databricks-cli
python39Packages.mlflow

@tbenst You could try to enable the tests by fetching the sources from GitHub :)


Remove appdata.go and tweak config file reading

Hi @poszu there seems to be a fair amount of code dedicated here to supporting multiple default paths for config. What about just making the user explicitly provide config with a flag?

Hi @poszu there seems to be a fair amount of code dedicated here to supporting multiple default paths for config. What about just making the user explicitly provide config with a flag?

True it will look at a few places. However, there is a flag to provide path to a config file. If not set it will look in the:

current directory,
in OS config directory (for example, ~/.config/poet/poet.conf on linux).

Is it necessary? Probably not. @nj, you are the main user of the poet - what is important to you? I can make it look for config only when -c <path> is given on CLI and remove looking in the aforementioned dirs.

I would also prefer the user to have to provide paths explicitly instead of using too much logic to define a default path.
Normally I would say if os.UserCacheDir() errors, the app should just log the error and os.Exit(1), but since this is executed even when a custom path is set that would be counterproductive and changing the code to only determine the default path when no custom path is set is probably a bigger effort?


Remove unused variable(also fixes standard error), fix error while loading module

Actually, It was because of the path error.

What do you think?

Thanks!

:D

Nice


EKS Controller is asking for the ClusterName or reference when deploying a NodeGroup

Describe the bug
According to the documentation, it is not needed to pass either the cluster name or the cluster name reference. What happens is, after setting up a manifest as follows:
apiVersion: eks.services.k8s.aws/v1alpha1
kind: Nodegroup
metadata:
  name: t3-small-spot
  annotations:
    argocd.argoproj.io/sync-wave: "0"
spec:
  clusterName: whatever
  amiType: AL2_x86_64
  capacityType: SPOT
  diskSize: 40
  instanceTypes:
  - t3.small
    intent: apps
  name: t3-small-spot
  scalingConfig:
    desiredSize: 0
    maxSize: 3
    minSize: 0
  subnets:
  - subnet-12345
  - subnet-123456
  tags:
    k8s.io/cluster-autoscaler/node-template/label/intent: apps
    k8s.io/cluster-autoscaler/node-template/taint/spotInstance: "true:PreferNoSchedule"
  taints:
  - effect: PreferNoSchedule
    key: spotInstance
    value: "true"

I see the following logs by checking the controller logs:
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem
	/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.0/pkg/internal/controller/controller.go:266
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2
	/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.0/pkg/internal/controller/controller.go:227
2023-01-11T13:43:55.499Z	ERROR	controller.nodegroup	Reconciler error	{"reconciler group": "eks.services.k8s.aws", "reconciler kind": "Nodegroup", "name": "t3-small-spot", "namespace": "kube-system", "error": "resource reference wrapper or ID required: ClusterName,ClusterRef"}

Steps to reproduce

# create the manifest
cat > manifest.yaml <<EOF
\ apiVersion: eks.services.k8s.aws/v1alpha1
kind: Nodegroup
metadata:
  name: t3-small-spot
  annotations:
    argocd.argoproj.io/sync-wave: "0"
spec:
  clusterName: whatever
  amiType: AL2_x86_64
  capacityType: SPOT
  diskSize: 40
  instanceTypes:
  - t3.small
    intent: apps
  name: t3-small-spot
  scalingConfig:
    desiredSize: 0
    maxSize: 3
    minSize: 0
  subnets:
  - subnet-12345
  - subnet-123456
  tags:
    k8s.io/cluster-autoscaler/node-template/label/intent: apps
    k8s.io/cluster-autoscaler/node-template/taint/spotInstance: "true:PreferNoSchedule"
  taints:
  - effect: PreferNoSchedule
    key: spotInstance
    value: "true"
\ EOF

# apply the manifest
kubectl apply -f manifest.yaml

# check the logs of eks controller pod
kubectl logs <pod>

Expected outcome
I expected to see a node groupd named t3-small-spot on the EKS console since I passed all the required arguments described in the documentation.
Environment

Kubernetes version 1.23
Using EKS (yes/no), if so version? yes, 1.23
AWS service targeted (S3, RDS, etc.) EKS

After adding the clusterName ref to the manifest, I checked the logs and I see the following:
ERROR	controller.nodegroup	Reconciler error	{"reconciler group": "eks.services.k8s.aws", "reconciler kind": "Nodegroup", "name": "t3-small-spot", "namespace": "kube-system", "error": "resource reference wrapper or ID required: NodeRole,NodeRoleRef"}

Again, I can see in the same documentation I linked that neither the node role nor node role ref as required. I understand that it must exist one or another, however, I think the documentation must be improveded then.

I understand that it must exist one or another, however, I think the documentation must be improveded then.

Correct. The reference is a replacement for a concrete value. Our documentation doesn't show required fields with mutual exclusivity (the docs don't have context about which fields are references), so the best we can do is show that both are "optional".
Do you know of any other documentation sources that show this type of behaviour that would serve as a good reference for us to improve ours?

right of the bat I can remember Freshservice's API docs. Notice the requester_id description.


Android web app capability broken

Describe the bug

Navigate to an installation of pi-gallery2 in Chrome on Android
Create a web app in the Chrome menu with Add to Home screen and click through the creation process
Navigate to the created icon on the home screen and click it
An errorneous screen is shown

Screenshots

Environment

OS:  Android 11 on Google Pixel 4a 5G
Browser: Chrome on Android

Used app version

bpatrik/pigallery2:1.8.2-alpine linux/arm/v7

Solution idea
This is maybe due to a missing meta annotation in the index.html:
<meta name="mobile-web-app-capable" content="yes">

I found out what the issue is.  The start_url parameter in the manifest.json is set as the publicUrl parameter from the configuration.  The issue is that publicUrl is empty by default and so is start_url then, which does not work.
I would suggest to set start_url either as publicUrl or . in case publicUrl is empty.

I found out what the issue is.  The start_url parameter in the manifest.json is set as the publicUrl parameter from the configuration.  The issue is that publicUrl is empty by default and so is start_url then, which does not work.
I would suggest to set start_url either as publicUrl or . in case publicUrl is empty.


Is dependency on Node.js core needed?

It seems a bit strange to see references to process.env.NODE_ENV for a library that seems isomorphic. I'm trying to use SkateJS which depends on incremental-dom and build with RollupJS. This hopefully works fine, but references to process obviously blows up.
P.S. I did work around it by configuring an inject plugin for process, but should that be needed?

IMHO using process.env.NODE_ENV !== 'production' is pretty great and straight forward since no code modifications are necessary for testing and everyone knows what's meant by it right away since it's so popular.
Use in production is covered by gulpfile.js babel transform-inline-environment-variables

P.S. I did work around it by configuring an inject plugin for process, but should that be needed?

If you want to build from source, you need to define it. There are also prebuilt UMD bundles (incremental-dom.js and incremental-dom-min.js) that remove the check and just have/don't have asserts enabled.
Expanding on @mashedcode's point, it allows you to integrate it into your overall build process and centrally flip assertions on/off for all the libraries you need with a single build config. The alternative would be to have a separate debug global that you could define in your build. Since most other libraries use process.env.NODE_ENV, I opted to follow the rest of the ecosystem.

I will close the issue as it is something you can configure with all build chains I know of. However I wish there was some future option in ES6, macros perhaps, to resolve this at build time.


33 test failures on windows due to diff_test case sensitivity issue

33 tests are failing on windows because skydoc is writing CRLF line endings, whilst the expected files are in repo with LF endings.
bazel --windows_enable_symlinks test //doc/shlib:execute_binary.md_difftest --enable_runfiles
INFO: From Testing //doc/shlib:execute_binary.md_difftest:
==================== Test output for //doc/shlib:execute_binary.md_difftest:
FINDSTR: /= ignored
FINDSTR: /" ignored
FINDSTR: /= ignored
FINDSTR: /" ignored
FAIL: files "doc/shlib/execute_binary.md" and "doc/shlib/execute_binary.md_" differ.

bazel --windows_enable_symlinks test //doc/shlib:execute_binary.md_difftest --noenable_runfiles
INFO: From Testing //doc/bazeldoc:build_rules_overview.md_difftest:
==================== Test output for //doc/bazeldoc:build_rules_overview.md_difftest:
FAIL: files "doc/bazeldoc/build_rules_overview.md" and "doc/bazeldoc/build_rules_overview.md_" differ.
================================================================================

This issue would be fixed by https://github.com/bazelbuild/bazel-skylib/pull/527, if approved and upstreamed
Failures
//doc/bazeldoc:api.md_difftest                                           FAILED in 0.8s
    ERROR   .doc/bazeldoc/api.md_difftest-test.bat (0.0s)
//doc/bazeldoc:build_rules_overview.md_difftest                          FAILED in 0.7s
    ERROR   .doc/bazeldoc/build_rules_overview.md_difftest-test.bat (0.0s)
//doc/bazeldoc:doc_utilities.md_difftest                                 FAILED in 0.7s
    ERROR   .doc/bazeldoc/doc_utilities.md_difftest-test.bat (0.0s)
//doc/bazeldoc:providers.md_difftest                                     FAILED in 0.8s
    ERROR   .doc/bazeldoc/providers.md_difftest-test.bat (0.0s)
//doc/bzlformat:rules_and_macros_overview.md_difftest                    FAILED in 0.7s
    ERROR   .doc/bzlformat/rules_and_macros_overview.md_difftest-test.bat (0.0s)
//doc/bzllib:api.md_difftest                                             FAILED in 0.8s
    ERROR   .doc/bzllib/api.md_difftest-test.bat (0.0s)
//doc/bzllib:bazel_labels.md_difftest                                    FAILED in 0.8s
    ERROR   .doc/bzllib/bazel_labels.md_difftest-test.bat (0.0s)
//doc/bzllib:filter_srcs.md_difftest                                     FAILED in 0.8s
    ERROR   .doc/bzllib/filter_srcs.md_difftest-test.bat (0.0s)
//doc/bzllib:lists.md_difftest                                           FAILED in 0.7s
    ERROR   .doc/bzllib/lists.md_difftest-test.bat (0.0s)
//doc/bzllib:rules.md_difftest                                           FAILED in 0.5s
    ERROR   .doc/bzllib/rules.md_difftest-test.bat (0.0s)
//doc/bzllib:src_utils.md_difftest                                       FAILED in 0.7s
    ERROR   .doc/bzllib/src_utils.md_difftest-test.bat (0.0s)
//doc/bzlrelease:api.md_difftest                                         FAILED in 0.8s
    ERROR   .doc/bzlrelease/api.md_difftest-test.bat (0.0s)
//doc/bzlrelease:create_release.md_difftest                              FAILED in 0.7s
    ERROR   .doc/bzlrelease/create_release.md_difftest-test.bat (0.0s)
//doc/bzlrelease:generate_release_notes.md_difftest                      FAILED in 0.7s
    ERROR   .doc/bzlrelease/generate_release_notes.md_difftest-test.bat (0.0s)
//doc/bzlrelease:generate_workspace_snippet.md_difftest                  FAILED in 0.8s
    ERROR   .doc/bzlrelease/generate_workspace_snippet.md_difftest-test.bat (0.0s)
//doc/bzlrelease:hash_sha256.md_difftest                                 FAILED in 0.7s
    ERROR   .doc/bzlrelease/hash_sha256.md_difftest-test.bat (0.0s)
//doc/bzlrelease:release_archive.md_difftest                             FAILED in 0.7s
    ERROR   .doc/bzlrelease/release_archive.md_difftest-test.bat (0.0s)
//doc/bzlrelease:rules.md_difftest                                       FAILED in 0.8s
    ERROR   .doc/bzlrelease/rules.md_difftest-test.bat (0.0s)
//doc/bzlrelease:update_readme.md_difftest                               FAILED in 0.8s
    ERROR   .doc/bzlrelease/update_readme.md_difftest-test.bat (0.0s)
//doc/bzltidy:rules_and_macros_overview.md_difftest                      FAILED in 0.8s
    ERROR   .doc/bzltidy/rules_and_macros_overview.md_difftest-test.bat (0.0s)
//doc/shlib:api.md_difftest                                              FAILED in 0.8s
    ERROR   .doc/shlib/api.md_difftest-test.bat (0.0s)
//doc/shlib:execute_binary.md_difftest                                   FAILED in 0.6s
    ERROR   .doc/shlib/execute_binary.md_difftest-test.bat (0.0s)
//doc/shlib:rules.md_difftest                                            FAILED in 0.6s
    ERROR   .doc/shlib/rules.md_difftest-test.bat (0.0s)
//doc/updatesrc:api.md_difftest                                          FAILED in 0.9s
    ERROR   .doc/updatesrc/api.md_difftest-test.bat (0.0s)
//doc/updatesrc:providers_overview.md_difftest                           FAILED in 0.7s
    ERROR   .doc/updatesrc/providers_overview.md_difftest-test.bat (0.0s)
//doc/updatesrc:rules_and_macros_overview.md_difftest                    FAILED in 0.9s
    ERROR   .doc/updatesrc/rules_and_macros_overview.md_difftest-test.bat (0.0s)
//doc/updatesrc:update_srcs.md_difftest                                  FAILED in 0.8s
    ERROR   .doc/updatesrc/update_srcs.md_difftest-test.bat (0.0s)
//tests/bzlformat_tests/tools_tests/missing_pkgs_tests/common_tests:bzlformat_BUILD_bazel_fmttest FAILED in 0.6s
    ERROR   .tests/bzlformat_tests/tools_tests/missing_pkgs_tests/common_tests/bzlformat_BUILD_bazel_fmttest-test.bat (0.0s)
//tests/bzlrelease_tests/rules_tests/generate_release_notes_tests:bzlformat_BUILD_bazel_fmttest FAILED in 0.6s
    ERROR   .tests/bzlrelease_tests/rules_tests/generate_release_notes_tests/bzlformat_BUILD_bazel_fmttest-test.bat (0.0s)
//tests/bzlrelease_tests/rules_tests/generate_workspace_snippet_tests:bzlformat_BUILD_bazel_fmttest FAILED in 0.5s
    ERROR   .tests/bzlrelease_tests/rules_tests/generate_workspace_snippet_tests/bzlformat_BUILD_bazel_fmttest-test.bat (0.0s)
//tests/bzlrelease_tests/rules_tests/release_artifact_tests:bzlformat_BUILD_bazel_fmttest FAILED in 0.8s
    ERROR   .tests/bzlrelease_tests/rules_tests/release_artifact_tests/bzlformat_BUILD_bazel_fmttest-test.bat (0.0s)
//tests/markdown_tests/markdown_generate_toc_tests:structured_custom_toc_header_test FAILED in 0.4s
    ERROR   .tests/markdown_tests/markdown_generate_toc_tests/structured_custom_toc_header_test-test.bat (0.0s)
//tests/markdown_tests/markdown_generate_toc_tests:structured_keep_toc_header_entry_test FAILED in 0.5s
    ERROR   .tests/markdown_tests/markdown_generate_toc_tests/structured_keep_toc_header_entry_test-test.bat (0.0s)
//tests/markdown_tests/markdown_generate_toc_tests:structured_no_toc_header_test FAILED in 0.5s
    ERROR   .tests/markdown_tests/markdown_generate_toc_tests/structured_no_toc_header_test-test.bat (0.0s)
Test cases: finished with 295 passing and 34 failing out of 329 test cases

A typical difference is:
D:\workdir\github\forks\bazel-starlib2>diff "D:\workdir\github\forks\bazel-starlib2\doc\bazeldoc\build_rules_overview.md" "D:\udu\b\356umzpb\execroot\_main\bazel-out\x64_windows-fastbuild\bin\doc\bazeldoc\build_rules_overview.md_" | cat -v
1,15c1,15
< <!-- Generated with Stardoc, Do Not Edit! -->
< # Build Rules
<
< The macros described below are used to generate, test and copy
< Starlark documentation.
<
< On this page:
<
<   * [doc_for_provs](#doc_for_provs)
<   * [stardoc_for_prov](#stardoc_for_prov)
<   * [stardoc_for_provs](#stardoc_for_provs)
<   * [write_doc](#write_doc)
<   * [write_file_list](#write_file_list)
<   * [write_header](#write_header)
<
---
> <!-- Generated with Stardoc, Do Not Edit! -->^M
> # Build Rules^M
> ^M
> The macros described below are used to generate, test and copy ^M
> Starlark documentation.^M
> ^M
> On this page:^M
> ^M
>   * [doc_for_provs](#doc_for_provs)^M
>   * [stardoc_for_prov](#stardoc_for_prov)^M
>   * [stardoc_for_provs](#stardoc_for_provs)^M
>   * [write_doc](#write_doc)^M
>   * [write_file_list](#write_file_list)^M
>   * [write_header](#write_header)^M
> ^M

This issue would be fixed by https://github.com/bazelbuild/bazel-skylib/pull/527, if approved and upstreamed


Tiles with sprites larger than nominal tile width are cropped right/down

This is a stub issue for the behavior discussed in https://github.com/photonstorm/phaser/issues/1439#issuecomment-67734788
It probably relates to using the nominal (vs. actual) tile size measurement when drawing and/or computing the area to update.

I wonder if an easy fix for this might be to scan all the tilesets used on a layer and calculate the largest width / height used across all of them. Then use these values to calculate the update area dimensions.

In order to keep the Github issues clear this feature request has been moved to the Phaser 3 wishlist.


update live

Bunch of changes and improvements to the website:
New trademark page, federation page, updates to event page, improvements to support and promote pages.

all tested on staging.

You tested this?
;)

Kidding, you're doing great work with the site. It looks really good and it's easy to find stuff.

@carlaschroder thanks ;-)
Now waiting for deployment... Will kick sysadmin agian D:


Type aliases deprecation

Description
We would like to deprecate the Nested and the Test type aliases for using the annotations of JUnit instead.
The deprecation notice should indicate that this will be an error in the next minor release (see issue #48).
Dependencies
This issue is blocked by the following ones:

[x] #35

Checklist

[ ] Deprecate the Nested type alias with a warning level and refactor its usages.
[ ] Deprecate the Test type alias with a warning level and refactor its usages.
[ ] Update the Unreleased section in changelog.

The Kotools Assert project will be archived.
We recommend to use the testing library provided by Kotlin instead.


Only the first button with class "button" is working with the ".button" touchend event

In the list of buttons rendered by transparency.js, only the first button is causing the .button touchend event to be fired. Even when a new button with the .button class is added manually so that is not rendered by transparency.js, the touchend event does not fire.

The issue was that JavaScript wasn't getting the updated DOM in which the buttons were rendered, so I had to put the touchend event listener inside of the function that has the render call in it.


on startup creation of maps

for pulling recent Config-like structures for all the available projects (project_names). on startup, builder-backend is pinged to collect all such configs and stored globally.

as per contracts discussed for creation, has been followed. please review and merge it.
for testing it locally, try running it with BUILDER_BACKEND_URL='http://builder.vernacular.ai' py.test -v within the slu directory.


Failures of MathNet.Numerics.Tests.MKL

I have tested MathNet.Numerics.Tests.MKL project with MathNet.Numerics.MKL.Win (v2.3.0, MKL 2018.0 R1).
Math.NET Numerics Configuration:
Version 4.9.0
Built for .Net Framework 4.6.1
Linear Algebra Provider: Intel MKL (x64; revision 12; MKL 2018.0 Update 1)
Fourier Transform Provider: Intel MKL (x64; revision 12; MKL 2018.0 Update 1)
Max Degree of Parallelism: 8
Parallelize Elements: 300
Parallelize Order: 64
Check Distribution Parameters: True
Thread-Safe RNGs: True
Operating System: Microsoft Windows NT 6.2.9200.0
Framework: 4.0.30319.42000

There are 43 failed tests and 2 errors. Most of them may be minor and related to accuracy :
1) Failed : MathNet.Numerics.UnitTests.DistributionTests.Multivariate.MatrixNormalTests.ValidateDensity
Not equal within 16 places. Expected:0.000156829273664912; Actual:0.000156829273664912
   위치: MathNet.Numerics.UnitTests.AssertHelpers.AlmostEqualRelative(Double expected, Double actual, Int32 decimalPlaces)

7) Failed : MathNet.Numerics.UnitTests.LinearAlgebraTests.Complex.Factorization.UserCholeskyTests.CanFactorizeRandomMatrix(1)
  Expected: (0.353088257803903, 0)
  But was:  (0.353088257803903, -8.01291052930066E-21)
   위치: MathNet.Numerics.UnitTests.LinearAlgebraTests.Complex.Factorization.UserCholeskyTests.CanFactorizeRandomMatrix(Int32 order)

And some are related with LUFactor() ( => LAPACKE_dgetrf function of Intel MKL):
40) Error : MathNet.Numerics.UnitTests.RootFindingTests.BroydenTest.Seveneq3b
MathNet.Numerics.InvalidParameterException : An invalid parameter was passed to a native method, parameter number : 4
   위치: MathNet.Numerics.Providers.LinearAlgebra.Mkl.MklLinearAlgebraProvider.LUFactor(Double[] data, Int32 order, Int32[] ipiv)
   위치: MathNet.Numerics.LinearAlgebra.Double.Factorization.DenseLU.Create(DenseMatrix matrix)
   위치: MathNet.Numerics.RootFinding.Broyden.TryFindRootWithJacobianStep(Func`2 f, Double[] initialGuess, Double accuracy, Int32 maxIterations, Double jacobianStepSize, Double[]& root)
   위치: MathNet.Numerics.UnitTests.RootFindingTests.BroydenTest.Seveneq3b()

Am I missing something? MKL 2019 R5 gives the same result, as well.

Yes, this has been bothering me for a while as well. I finally had some time to look into it, and noticed the following:

Differences between different CPUs, e.g. Skylake vs Sandy Bridge. This was reduced by explicitly setting in the tests the MKL consistency level to AVX.
Differences in NaN handling between managed and MKL, e.g. when unbounded root finding algorithms move outside of the supported domain of the function. The behavior is slightly more consistent now (they at least result in the same exception).
Minor accuracy differences between managed and MKL, which could have all kind of reasons.

On the two machines I've tried, one of them with Intel Parallel Studio 2020.1, the other only with Intel MKL 2020.1, all the tests now pass. On a clean checkout, I've started the tests with ./build.sh mkltest (see also) without any other changes or steps. I wonder whether they now pass on your environment as well?

With MKL 2019.0 U5 on Coffee Lake(i7-9700KF), all the tests pass, but MKL 2018.0 U1 failed only at
MathNet.Numerics.UnitTests.LinearAlgebraTests.Double.Factorization.EvdTests.CanFactorizeRandomSquareMatrix(50)
Not equal within 8 relative places. Expected:1.3356091897454E-06; Actual:1.33560918183506E-06

Now, I am not sure why I can't build the MKL project. I got the following errors:
---------------------------------------------------------------------
Build Time Report
---------------------------------------------------------------------
Target         Duration
------         --------
Start          00:00:00.0001280
Clean          00:00:00.4952310
ApplyVersion   00:00:00.0302953
Prepare        00:00:00.0000304
MklWinBuild    Failure
Total:         00:00:12.4002579
---------------------------------------------------------------------
Status:        Failure
---------------------------------------------------------------------
---------------------------------------------------------------------
  1) Building D:\Codes\mathnet-numerics\MathNet.Numerics.MKL.sln failed with exitcode 1.
  2) : C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\Sdks\Microsoft.NET.Sdk\build\Microsoft.NET.TargetFrameworkInference.targets(112,5): The current .NET SDK does not support targeting .NET Core 3.1.  Either target .NET Core 1.1 or lower, or use a version of the .NET SDK that supports .NET Core 3.1.
  3) : C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\Sdks\Microsoft.NET.Sdk\build\Microsoft.NET.TargetFrameworkInference.targets(126,5): The current .NET SDK does not support targeting .NET Standard 2.0.  Either target .NET Standard 1.6 or lower, or use a version of the .NET SDK that supports .NET Standard 2.0.
  4) : C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\Sdks\Microsoft.NET.Sdk\build\Microsoft.NET.TargetFrameworkInference.targets(112,5): The current .NET SDK does not support targeting .NET Core 3.1.  Either target .NET Core 1.1 or lower, or use a version of the .NET SDK that supports .NET Core 3.1. 3) : C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\Sdks\Microsoft.NET.Sdk\build\Microsoft.NET.TargetFrameworkInference.targets(126,5): The current .NET SDK does not support targeting .NET Standard 2.0.  Either target .NET Standard 1.6 or lower, or use a version of the .NET SDK that supports .NET Standard 2.0.
  4) : C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\Sdks\Microsoft.NET.Sdk\build\Microsoft.NET.TargetFrameworkInference.targets(112,5): The current .NET SDK does not support targeting .NET Core 3.1.  Either target .NET Core 1.1 or lower, or use a version of the .NET SDK that supports .NET Core 3.1.
---------------------------------------------------------------------

.NET Core SDK 3.1.101, VS2017, and MKL 2019 U5 were installed.

Thanks for testing!
Weird - in case you still have the build output, at the beginning you should see a print-out of dotnet info - does it indeed state Version: 3.1.101?

build.cmd build on the latest master works well.
The build output of MklWinBuild on the master is here:
MklWinBuild.txt
There is a warning NU1503 for dotnet restore MathNet.Numerics.MKL.sln. Is it problematic?
Starting Target: MklWinBuild (==> Prepare)
dotnet restore MathNet.Numerics.MKL.sln --verbosity minimal /p:StrongName="False" /nr:false
D:\Codes\mathnet-numerics\src\NativeProviders\Windows\MKL\MKLWrapper.vcxproj : warning NU1503: Skipping restore for project 'D:\Codes\mathnet-numerics\src\NativeProviders\Windows\MKL\MKLWrapper.vcxproj'. The project file may be invalid or missing targets required for restore. [D:\Codes\mathnet-numerics\MathNet.Numerics.MKL.sln]

The corresponding lines on my working directSparseSolver branch(#695) is
Starting Target: MklWinBuild (==> Prepare)
Building project: MathNet.Numerics.MKL.sln
  C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\15.0\Bin\MSBuild.exe  MathNet.Numerics.MKL.sln /t:Restore /m /nologo /nodeReuse:False  /v:m  /p:RestorePackages="False" /p:Configuration="Release" /p:StrongName="False" /logger:Fake.MsBuildLogger+ErrorLogger,"D:\Codes\mathnet-numerics\packages\build\FAKE\tools\FakeLib.dll"

By the way, I uninstalled VS2019 in January.

@cdrnet, are you using VS2019? If yes, I will check whether VS2019 fixes the errors.

I use Rider primarily, but have both VS2017 and VS2019 installed on all machines I user for development. Opening, building and testing the MKL solution works for me in both VS2017 and VS2019. The solution still uses the VS2017 C++ platform toolset (v141).
That does not necessarily imply that it would also work in VS2017 if VS2019 is not installed.
Just to rule this out: if you open the Visual Studio installer, it does not report a warning that the installation is not complete? I've had similar errors in the past when one of the updates failed to complete.
Warning NU1503 I have as well on my machine, seems to be fine (it does not need to restore anything).

I've just realized what might be the issue: for MKL we still build directly with MsBuild, not with the .NET SDK tools (the main build uses the .NET Core SDK). VS2017 does not directly/officially support .NET Core 3. I suspect it might still work on my machine because the MsBuild coming with VS2019 does indeed support .NET Core 3. My VS2017 might effectively build using the MsBuild from VS2019. So it seems to me technically we do indeed no longer fully support VS2017.
Maybe it could also work to use dotnet msbuild instead of MsBuild, need to check. I'd expect VC++ to be stripped out there though.

In VS2019, the Intel Performance Libraries section was not shown in the property pages of the MKL project, but it was listed in VS2017. There may be something wrong configuration in my PC.
When I changed the platform toolset from v141 to v142, build.cmd mkltest passed successfully.

It seems v142 is working well for me as well in VS2019. I've pushed it to master. So it seems we can finally resolve this issue? Thanks again!


Issues adding name and attribute value locations

There is a possible use case of using the xml crate for parsing user interface components written in MXML as part of the Apache Royale framework, where it's necessary to know the first and last line numbers and columns of a certain element name, attribute name, or attribute value ((OwnedName, TextSpan) or (String, TextSpan) for attribute values).
I've tried forking this crate, but it looks like the constructed spans have the same first TextPosition as the last TextPosition, pointing always to after the last character of the qualified name or attribute value. Here's an example of what I modified from xml-rs:
impl PullParser {
    fn read_qualified_name<F>(&mut self, t: Token, target: QualifiedNameTarget, on_name: F) -> Option<Result>
      where F: Fn(&mut PullParser, Token, (OwnedName, TextSpan)) -> Option<Result> {
        let start = self.lexer.position();

        ...

        let invoke_callback = move |this: &mut PullParser, t| {
            let name = this.take_buf();
            match name.parse() {
                Ok(name) => on_name(this, t, (name, TextSpan { start, end: this.lexer.position() })),
                Err(()) => Some(this.error(SyntaxError::InvalidQualifiedName(name.into()))),
            }
        };

        ...
    }
}

Here's a dummy debugging function:
#[test]
fn f1() {
    use crate::reader::XmlEvent;
    let my_xml = r#"<?xml version="1.0" encoding="UTF-8"?>
<s:Application
    xmlns:fx="http://ns.adobe.com/flex"
    xmlns:s="library://ns.adobe.com/flex/spark"
    creationComplete="initialize()">
</s:Application>
    "#;
    let mut reader = EventReader::new(my_xml.as_bytes());
    while let Ok(event) = reader.next() {
        match event {
            XmlEvent::StartElement { name, attributes, .. } => {
                println!("{}", name.1);
                for attr in attributes {
                    println!("{}={}", attr.name.1, attr.value.1);
                }
            },
            XmlEvent::EndDocument => {
                break;
            },
            _ => {},
        }
    }
}

This crate tracks positions only for reporting errors, so often the position is not updated until an error happens, and then only points to the invalid markup.
Positions of elements in the source are not part of XML's data model. This crate won't be able to provide that.

@hydroper  I suppose this crate is what you need https://github.com/RazrFalcon/roxmltree

@sify21 It looks to only store the location for the entire element rather than also the opening and closing tag names, the attribute names, the attribute values, and so on, but thanks for the pointer!


Mark tests that sleep as @medium

The @medium annotation is an alias for @group medium.
If the PHP_Invoker package is installed and strict mode is enabled, a medium test will fail if it takes longer than 10 seconds to execute. This timeout is configurable via the timeoutForMediumTests attribute in the XML configuration file.

You can skip these tests while developing by running
phpunit --exclude-group medium

This will significantly speed up the execution of the suite.
Includes PR #80

Can you please rebase?

(I though I could retrigger travis)

Thank you