Fix parameter name so string2hex has correct generated .d.ts signature

Description of change
After testing an update from v5.2.4 to 5.3.0, I noticed an error relating to string2hex. After the update, it seems the generated .d.ts signature changed to accidentally not accept any arguments. This doesn't affect tests or the generated JavaScript at runtime since the actual code itself and TypeScript annotation does still correctly accept a string parameter. But when using Pixi as an external dependency in a TypeScript project, something like PIXI.utils.string2hex('#FFFFFF') will throw a TypeScript error.
Inspired by a similar recent PR https://github.com/pixijs/pixi.js/pull/6703 that seems to fix up some issues created from the refactor in https://github.com/pixijs/pixi.js/pull/6634, this simply adds a parameter name to the JSDoc so the generated pixi.js.d.ts function signature is correct.
This fix can be tested by running npm run types before and after applying the commit and seeing the generated signature for string2hex in dist/types/pixi.js.d.ts.
Please let me know if I missed anything!
Pre-Merge Checklist

[x] Documentation is changed or added
[x] Lint process passed (npm run lint)
[x] Tests passed (npm run test)

Thank you @prokopcm 🎉


punctuation's added

and ->  ,  and

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Please resolve conflicts -- there have been others editing the same file -- and then dismiss my review.  I will be happy to merge.

@sidharthpunathil note that I cannot merge with conflicts; please resolve to unblock me. Thanks!

Hi @sidharthpunathil ,
Please note that I am unable to pull until you resolve the conflicts.  There's a button "Resolve conflicts" that might help, or you might prefer to use the command line.
Thanks!

Thanks! Nearly there; please remove the rollback of an unrelated change and I can merge.

@sidharthpunathil FYI this on the last review round is still blocking me from merging.


Improve API test coverage (v2)

A second go at a branch that descended into a conflict/rebase nightmare. Cherry picked the relevant commits from the old branch (see https://github.com/alphagov/digitalmarketplace-api/pull/603)
In the interests of not having the rebase nightmare happen again, I left out the reorganisation of the large test class for another time. The goal of having better test coverage for the modules involved has still hopefully been achieved!

Looks good :+1:
Would be nice to get the reordering into classes in, but yeah maybe not right now.

Thanks @TheDoubleK!


[Code Addition Request]: Movie Recommendation System with Machine Learning

Have you completed your first issue?

[X] I have completed my first issue

Guidelines

[X] I have read the guidelines
[X] I have the link to my latest merged PR

Latest Merged PR Link
https://github.com/UTSAVS26/PyVerse/pull/426
Project Description
This project implements a content-based movie recommendation system using TF-IDF and sigmoid kernel to suggest movies with similar plot summaries based on the user's input.
Full Name
Deepanshu Baghel
Participant Role
GSSOC

@UTSAVS26


Azure Function Bot upgrade from v<IP_ADDRESS> to v3.13.1 causes failure to connect

Bot Info

SDK Platform: .NET
SDK Version: 3.13.1
Active Channels: Bot Framework Emulator v3.5.35
Deployment Environment: local development with Function emulator

Issue Description
I have a Function Bot that works just find with v<IP_ADDRESS>. When I update the NuGet libraries to v3.13.1, everything builds. But when I run the function and attempt to connect via Bot Framework Emulator, I get an error in the function emulator window that it can't find the <IP_ADDRESS> version of Microsoft.Bot.Connector
Reproduction Steps

Create Azure Function from template
Copy files from Bot template into function
Install <IP_ADDRESS> Bot libraries
Run function from Visual Studio (it will prompt to upgrade the Function environment. do it)
Connect to function endpoint from Bot Framework Emulator
Have successful conversation with Bot
Upgrade to 3.13.1 Bot libraries, run function
During connection, observe error 500 in Bot Framework Emulator Log and observe error in function window attempting to locate Microsoft.Bot.Connector v<IP_ADDRESS>

Expected Behavior
Bot Emulator should connect without error
Works:
Microsoft.Bot.Builder v<IP_ADDRESS>
Microsoft.Bot.Builder.Azure v3.2.5
Microsoft.Bot.Connector v<IP_ADDRESS>
Microsoft.NET.Sdk.Functions v1.0.8
Fails:
Microsoft.Bot.Builder v3.13.1
Microsoft.Bot.Builder.Azure v3.2.5
Microsoft.Bot.Connector v3.13.1
Microsoft.NET.Sdk.Functions v1.0.8

After researching, I believe this is connected to the fact that Azure Functions has no mechanism for binding redirects. Microsoft.Bot.Builder.Azure v3.2.5 is linked to Bot.Builder v<IP_ADDRESS>. Please release an updated Bot.Builder.Azure that links to the current libraries, and that should solve my problem.
Thanks!

Also, I just tried to update to the alpha of Bot.Builder.Azure. Failure:
Azure Functions requires Newtonsoft.Json = 9.0.1
Microsoft.Bot.Builder.Azure 4.0.0-alpha20180221 requires Newtonsoft.Json >= 10.0.3
So please go back and release a Microsoft.Bot.Builder.Azure v3.2.6 that requires Newtonsoft.Json = 9.0.1 and Microsoft.Bot.Builder = <IP_ADDRESS>. It would be most appreciated.

The v4 alpha version is for Bot Builder v4, not v3.x
You are right, BotBuilder-Azure needs to be updated to use the latest BotBuilder  packages.  In fact, the library itself has already been updated: https://github.com/Microsoft/BotBuilder-Azure/pull/52/files  It just hasn't been released yet.  There is no timeline or schedule for the release, but I assume it will be soon.
Lastly, this is an issue with https://github.com/Microsoft/BotBuilder-Azure not with the BotBuilder.  Please open an issue with that repository if you wish to pursue this further.  Or, wait for the team to release the next nuget package.  Sorry for the inconvenience.


Let build be customized by env vars for OSX

Signed-off-by: Ed Snible<EMAIL_ADDRESS>Which problem is this PR solving?
Resolves #3445
Short description of the changes
With this change, I can build on a Mac.  The biggest problem is that the Mac doesn't let the containers see the real localhost, but instead puts the Mac's localhost at host.docker.internal.
First, I must tell Docker to let host.docker.internal be insecure:
cat <<EOF | > /tmp/buildkitd.toml
[registry."host.docker.internal:5000"]
  http = true
EOF

docker buildx create --use --config /tmp/buildkitd.toml

I then build using
make build-binaries-linux
PLATFORMS=linux/amd64 DOCKER_REPO=host.docker.internal:5000 make docker-images-jaeger-backend
(Note that I am confident about every change except for the --load.  I am not sure what that is needed!  Without it, the build complained that "No output specified for docker-container driver. Build result will only remain in the build cache. To push result image into registry use --push or to load image into docker use --load ")

Thanks for your patience everyone.
I removed ~/.docker, reinstalled Docker Desktop for Mac, and rebooted my laptop.
After those environment changes everything works without --load.
After those environment changes I find I don't need DOCKER_REGISTRY=host.docker.internal:5000 any more.  Something in my old config needed it; on the fresh Docker Desktop install it works fine without it.  Thus, I can't test overriding DOCKER_REGISTRY.
Thus, the only value of this PR is the ability to speed builds by doing PLATFORMS=linux/amd64 make docker-images-jaeger-backend.  Would it make sense to

close this PR
revise this PR just for PLATFORMS
keep DOCKER_REGISTRY exposed for users whose configuration requires it
?


Improve a11y in the search experience

Axe also complains about non-unique labels, but that might just be because their IDs are generated programatically? As far as I can see, they are unique
Acceptance Criteria:

[ ] Wave suggests that the search input on the homepage/search page should have a .

Either provide a  or use the aria label by thing?

[ ] Resolve Wave issues with KS filters on search page

I think this could be because the filter inputs are within the associated label tags?

duplicate


Persistent context example throws a ScopeMismatch exception

Using pytest-playwright 0.1.1 with playwright 1.11.0, I used to be able to create a persistent context like shown below -
import pytest

from playwright.sync_api import Playwright

from typing import Dict

@pytest.fixture(scope="session")
def context(
    playwright: Playwright,
    browser_name: str,
    browser_type_launch_args: Dict,
    browser_context_args: Dict
):
    context = getattr(playwright, browser_name).launch_persistent_context("./foobar", **{
        **browser_type_launch_args,
        **browser_context_args
    })

    context.set_default_navigation_timeout(90000)
    context.set_default_timeout(90000)

    yield context
    context.close()

def test_example(page):
    page.goto("https://example.com")
    assert page.inner_text('h1') == 'Example Domain'
    page.click("text=More information")

I recently upgraded to pytest-playwright 0.2.0 and playwright 1.14.1, and took the persistent context snippet published on https://playwright.dev/python/docs/test-runners#persistent-context to understand how it's done on the latest version. Unfortunately the snippet seems to throw a ScopeMismatch error, because browse_context_args has function scope -
# conftest.py
import pytest
from playwright.sync_api import BrowserType
from typing import Dict

@pytest.fixture(scope="session")
def context(
    browser_type: BrowserType,
    browser_type_launch_args: Dict,
    browser_context_args: Dict
):
    context = browser_type.launch_persistent_context("./foobar", **{
        **browser_type_launch_args,
        **browser_context_args,
        "locale": "de-DE",
    })
    yield context
    context.close()

def test_example(page):
    page.goto("https://example.com")
    assert page.inner_text('h1') == 'Example Domain'
    page.click("text=More information")

Sample Output:
$ pytest .
========================================================================================== test session starts ==========================================================================================
platform win32 -- Python 3.9.6, pytest-6.2.3, py-1.10.0, pluggy-0.13.1
rootdir: C:\Users\fowliena\Documents\repositories\persistentcontext
plugins: asyncio-0.15.1, base-url-1.4.2, forked-1.3.0, json-report-1.4.0, metadata-1.11.0, playwright-0.2.0, xdist-2.2.1
collected 1 item

test_stuff.py E                                                                                [100%]

================================================================================================ ERRORS =================================================================================================
________________________________________________________________________________ ERROR at setup of test_example[chromium] ________________________________________________________________________________
Exception ignored in: <function BaseSubprocessTransport.__del__ at 0x0000011E28530DC0>
Traceback (most recent call last):
  File "c:\users\fowliena\appdata\local\programs\python\python39\lib\asyncio\base_subprocess.py", line 126, in __del__
    self.close()
  File "c:\users\fowliena\appdata\local\programs\python\python39\lib\asyncio\base_subprocess.py", line 104, in close
    proto.pipe.close()
  File "c:\users\fowliena\appdata\local\programs\python\python39\lib\asyncio\proactor_events.py", line 108, in close
    self._loop.call_soon(self._call_connection_lost, None)
  File "c:\users\fowliena\appdata\local\programs\python\python39\lib\asyncio\base_events.py", line 746, in call_soon
    self._check_closed()
  File "c:\users\fowliena\appdata\local\programs\python\python39\lib\asyncio\base_events.py", line 510, in _check_closed
    raise RuntimeError('Event loop is closed')
RuntimeError: Event loop is closed
ScopeMismatch: You tried to access the 'function' scoped fixture 'browser_context_args' with a 'session' scoped request object, involved factories
test_stuff.py:7:  def context(browser_type: playwright.sync_api._generated.BrowserType, browser_type_launch_args: Dict, browser_context_args: Dict)
..\..\..\appdata\local\programs\python\python39\lib\site-packages\pytest_playwright\pytest_playwright.py:135:  def browser_context_args(pytestconfig: Any, playwright: playwright.sync_api._generated.Playwright, device: Optional[str]) -> Dict
======================================================================================== short test summary info ========================================================================================
ERROR test_stuff.py::test_example[chromium]

Is this due to the snippet being incorrect, or am I missing something with the way I'm creating a persistent context?
I'm running Python 3.9.6, with the following packages installed -
$ pip list installed
Package                    Version
-------------------------- ---------
apipkg                     1.5
atomicwrites               1.4.0
attrs                      21.2.0
autopep8                   1.5.6
certifi                    2020.12.5
chardet                    4.0.0
click                      8.0.1
colorama                   0.4.4
coverage                   5.5
execnet                    1.8.1
flake8                     3.9.1
flake8-docstrings          1.6.0
flake8-import-order        0.18.1
flake8-junit-report        2.1.0
flake8-polyfill            1.0.2
future                     0.18.2
gitdb                      4.0.7
GitPython                  3.1.18
greenlet                   1.1.0
hvac                       0.10.10
idna                       2.10
iniconfig                  1.1.1
Jinja2                     2.11.3
joblib                     1.0.1
livereload                 2.6.3
lunr                       0.5.8
Markdown                   3.3.4
MarkupSafe                 2.0.1
mccabe                     0.6.1
mkdocs                     1.1.2
mkdocs-autorefs            0.2.1
mkdocs-gen-files           0.3.3
mkdocs-material            7.1.6
mkdocs-material-extensions 1.0.1
mkdocstrings               0.15.1
mypy                       0.910
mypy-extensions            0.4.3
nltk                       3.6.2
packaging                  20.9
pathspec                   0.8.1
pep8-naming                0.11.1
pip                        21.1.3
playwright                 1.14.1
pluggy                     0.13.1
py                         1.10.0
pycodestyle                2.7.0
pydocstyle                 6.0.0
pyee                       8.1.0
pyflakes                   2.3.1
pygit                      0.1
Pygments                   2.9.0
pymdown-extensions         8.2
pyparsing                  2.4.7
pytest                     6.2.3
pytest-asyncio             0.15.1
pytest-base-url            1.4.2
pytest-forked              1.3.0
pytest-json-report         1.4.0
pytest-metadata            1.11.0
pytest-playwright          0.2.0
pytest-xdist               2.2.1
python-slugify             5.0.2
pytkdocs                   0.11.1
PyYAML                     5.4.1
regex                      2021.4.4
requests                   2.25.1
semver                     2.13.0
setuptools                 56.0.0
six                        1.16.0
smmap                      4.0.0
snowballstemmer            2.1.0
tblib                      1.7.0
text-unidecode             1.3
toml                       0.10.2
tornado                    6.1
tqdm                       4.61.0
typed-ast                  1.4.3
typing-extensions          <IP_ADDRESS>
urllib3                    1.26.4
websockets                 9.0.1
yamllint                   1.26.1

Believe I may have found the issue. In https://github.com/microsoft/playwright-pytest/pull/70, the browser_context_args fixture was changed from session scoped to the default scope (function).
So to use a persistent context we' d have to change the scope of browse_context_args back to session. Is there a nicer way to do this other than copy & pasting the existing fixture and modifying the annotation?
import pytest
from playwright.sync_api import BrowserType, Playwright
from typing import Any, Dict, Optional

@pytest.fixture(scope="session")
def browser_context_args(
    pytestconfig: Any,
    playwright: Playwright,
    device: Optional[str],
) -> Dict:
    context_args = {}
    if device:
        context_args.update(playwright.devices[device])
    base_url = pytestconfig.getoption("--base-url")
    if base_url:
        context_args["base_url"] = base_url

    video_option = pytestconfig.getoption("--video")
    capture_video = video_option in ["on", "retain-on-failure"]
    if capture_video:
        context_args["record_video_dir"] = artifacts_folder

    return context_args

@pytest.fixture(scope="session")
def context(
    browser_type: BrowserType,
    browser_type_launch_args: Dict,
    browser_context_args: Dict
):
    context = browser_type.launch_persistent_context("./foobar", **{
        **browser_type_launch_args,
        **browser_context_args,
        "locale": "de-DE",
    })
    yield context
    context.close()

def test_example(page):
    page.goto("https://example.com")
    assert page.inner_text('h1') == 'Example Domain'
    page.click("text=More information")

Hi, thank you for your great verbose bug report. You are correct, since one of the last releases we changed the scopes from browser_type_launch_args from session to function.
We advice people to use context on the function scope what we by default use to enable context isolation per test. Are you intentionally having context on the session scope?

I was under the belief it needed to be session scope to use a persistent context (so I can login exactly once at the start of the test session, and preserve the authentication state for subsequent tests). If I can achieve the same thing with function scope then I'll give that a bash.
Thanks for the quick response!

When I change the browser_type_launch_args from session to function, my tests now load a new browser context for each test rather than 1 for all the tests. This has slowed the run time quite a bit

I'm just wondering what I need to do to open a new tab for each test in my Test Class, rather than a new Browser for each one? Since the change from Session to Function scope for the browser_type_launch_arg I have not been able to do this.


Take too long for saving change to local db

Hi 3lvis,
I am using SyncDB, in some cases it takes 7-10 seconds to finish saving change.
Is there any way to reduce saving time?
Thanks and Regards,

Hi @jetstar89,
I hope you're having a nice weekend :) Can I get some more information in the operation that you're doing? Having your Core Data model and an example of your JSON file might help a lot to profile the save.
Also, is it blocking your main thread or you don't have problems with that?

Hi 3lvis, It isn't blocking my main thread, cos I save in background thread.
But it takes quite much time for completion.

@jetstar89 Could you help me providing an example project I can use to reproduce this issue?
Thanks in advance :)

Hi 3lvis, could we have a private channel, Like slack, skype or any thing else.
Regards,

@jetstar89 you can write me in a privat channel in https://gitter.im/SyncDB/Sync

Haven't seen any other issues related to this so might be an isolated case and not something related to the library, closing unless anything changes.


The elements position in pdf is not exactly as seen on screen

I am using Angular in my project. I have multiple cards in my page, I am trying to convert my page into pdf. But, the pdf is not the exact version of my page. The card positions are little to the left and some are to the right.

Please refer to above images


Added two settings to optionally perform stricter json deserializations

MemberNameMatchHandling: forces case-sensitive lookup of member name.
MemberTypeConversionHandling: prevents implicit type conversion, e.g. assigning a json string to an int member.

Hi @JamesNK, I am using ASP.NET 5 MVC 6 and I would like to have a case sensitive JSON deserialization and I do not want to add a dependency on Newtonsoft Schema.
I think that the natural place to configure or override it is on InputFormatter's ContractResolver, but I cannot find a way to do it since GetClosestMatchProperty is who make it case insensitive.
Could you give me any clue?

:watch:

Hello again @JamesNK
7+ years have passed. What's your stance on json validations? My use case is a customer-facing REST API accepting large json objects. I don't want to be lax during deserialization, so that I don't break my customer implementations in case I ever need to become strict (e.g. if I move the endpoint to a different development stack). Could I prepare a new version of this MR for the latest version? Would you accept it?
If not, could you explain how Json.net schema can be used to validate casing and prohibit implicit type conversions (e.g. mapping a json string literal such as "3" to an integer property)? Can it be done without generating actual schemas, just based off C# POCOs?
Thank you!


Kafka Tests Fail

There is a intermittent / persistent bug that causes the kafka tests to fail.
We've tried backoffs / delays. But sometimes it seems it just doesn't want to get out of bed. Locally, the presents as failing on the first run, then passing IMMEDIATELY afterwards (sometimes on the third run... rarer). This seems to be tied to the configuration of the the TestContainer.

Think we fixed it. Added retry in the right places. Mostly I think it was a race condition between the producer and the consumer. Now both retry if they're surprised by the current state.


Add get_object to interpret JSON settings

Ticket
N/A
Purpose
Allow JSON objects and arrays to be passed into as environment variable settings, needed for Storage Internationalization project.
Changes
Adds get_object to the base Settings class, in the correct alphabetical order.
Side effects
None
QA Notes
N/A
Deployment Notes
Can now use JSON in environment variables, e.g.
export OSFSTORAGE_PROVIDER_CONFIG_PARITY_PROVIDER_SETTINGS='{
  "bucket": "cos-osf-stage-files-ca-1-parity"
}'

Coverage decreased (-0.007%) to 89.874% when pulling 540a1c39b6e6a8459b20d8ab6042fba0d77585b9 on icereval:feature/settings-get-object into e2e626680cba93dbd4e99cc789c26d3fd248704a on CenterForOpenScience:develop.


Added missing step

Identified and fixed missing step.
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

This was resolved as part of a large documentation refresh. Thank you.


Licensing and forks

Hello there!
The gatsby-plugin-hotjar plugin still works nicely. But it does not provide any option to supply some custom data to comply with CMPs that do not feature automated blocking.
I want to create a fork ClickTrap-Media/gatsby-plugin-klaro-hotjar to support KIProtect Klaro, but I just saw that there is no license in the repository. As you can read in GitHub's ToS a repository without
a license can be read and forked, but it is not allowed to modify the source code or create alternative releases.
As I want to add the code necessary to use the plugin and publish the package for others to use on npm, it would be great if you might add some license that allows to fork, edit and publish the project (under a name like gatsby-plugin-klaro-hotjar). The original plugin will surely be linked in the description at the top of the README.md.

@pascal-zarrad sorry for the delay with it. I've added a MIT license. I hope it works for you.

@pascal-zarrad sorry for the delay with it. I've added a MIT license. I hope it works for you.

@pavloko Yes that works, thank you!

@pavloko Yes that works, thank you!


User Story 04 - View stored interests and hobbies

A user should be able to view the stored interests and hobbies (keywords)
Components

Interests & Hobbies Page (view)

Displays stored records

Buttons

Edit
View Occupations

Authorization required

From Django

Interests & Hobbies (keywords)

Interests & Hobbies page planned to be dynamic:

Edit button should change field to editable (for story 5 functionality)


NIFI-13412 [MiNiFi] Handle absolute paths in Sync/Resource C2 command

Summary
NIFI-13412
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[x] Apache NiFi Jira issue created

Pull Request Tracking

[x] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[x] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[x] Pull Request based on current revision of the main branch
[x] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[x] Build completed using mvn clean install -P contrib-check

[x] JDK 21

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

Moved the related logic to the validation part, and created a new unit test. The previous implementation was dropped with the faulty test

Thanks for the fix, merging


Auto-generation of basic ufuncs with numba

@skrah
Here're the scripts to generate a .o file with basic ufuncs supported by numba:
https://gist.github.com/sklam/977dd295324db10868542567f526ecf6
Usage:

python ufunc_gen.py to create umath.o.
run cc -shared -o umath.so umath.o.
test with python check_ufunc_gen.py.

See top of file comments for details like ABI and symbol mangling.

@sklam This is fantastic, thanks!

The non-strided loops work nicely here.  As I understood, strided loops in the last dimension are not yet supported in Numba. How difficult would it be to add that?
For example, these tests would also pass:
https://github.com/plures/gumath/blob/master/python/test_gumath.py#L89

I thought the numba generated kernel doesn't care about the striding.  It should always read the strides from the argument.  Let me check.

You're right.  It should be easy to fix.


Installation hangs on GKE

Hi @nicolaferraro ,
I'm trying to run a simple REST DSL on GKE.

Install Kamel on GKE -

kubectl  get pods
NAME                                READY     STATUS    RESTARTS   AGE
camel-k-operator-7fb9dc6fb4-88shs   0/1       Pending   0          4m

Looks like the camel-k-operator hangs here.

Any thoughts?

More logs:
Events:
Type     Reason            Age               From               Message

Warning  FailedScheduling  2m (x26 over 8m)  default-scheduler  PersistentVolumeClaim is not bound: "camel-k-builder" (repeated 2 times)

Thanks @kasun04 for reporting this issue. Camel K has not been tried on GKE yet and such problems were expected to happen.
Currently the binary data is "passed" to Kaniko through a shared PV (RWM) that is not available in all Kubernetes implementations (including GKE, I think).
We can solve this issue e.g. by co-locating the kaniko builder and the maven builder pods.
Another issue that we need to solve for GKE is supporting image push secrets (that are not needed on Minikube).
Current support for Kubernetes is limited to Minikube at the moment.
We need to fix this issue to support also GKE oob.

thanks @nicolaferraro .
Does it work with Docker for mac at least?

It works with Docker for mac (not on minikube).
I've deployed a sample integration examples/routes-rest.js and was expecting it to create svc, ingress etc so that I can invoke my RESTful service.
Do I have to manually create ingress, service etc for that? Here are the pod and svc information.
kubectl get pods
NAME                                READY     STATUS    RESTARTS   AGE
camel-k-operator-64c5686848-4zkzt   1/1       Running   0          2h
./kamel get
NAME		CONTEXT	STATUS
routes
routes-rest
kubectl get svc
NAME               TYPE        CLUSTER-IP    EXTERNAL-IP   PORT(S)     AGE
camel-k-operator   ClusterIP   <IP_ADDRESS>           60000/TCP   2h
kubernetes         ClusterIP   <IP_ADDRESS>             443/TCP     3h


Jobs with Parallel, Each, and SubJobs

I can see validations around ensuring a job doesn't contain a mix of Parallel, Each, and SubJobs, and I can see the commits that made those changes, but I can't see why that's the case. I can't see it in the docs either.
Is there a reason we can't do it?

A job can contain a mix. A task can't.


Creative search does not use the language

Every time I launch Minecraft, I can't use creative search until I press the F3+J keyboard shortcut, every time I restart Minecraft, the search for items using the language in Creative mode lost.
Minecraft 1.18.2
LanguageReload v1.2.1
Video: https://youtu.be/JNGyf6qDV3g
This problem is similar to mine, but it's a little different.

That's actually a Dash Loader bug.
https://user-images.githubusercontent.com/17567951/174054882-558b567b-892c-45d4-b3e2-88a85482a0e5.mp4

Ohh, okay, thanks


Rule in ALB for default-backend is not created

alb ingress version: 1.0-beta.4
I created an ingress resource with alb-ingress-controller, the rule for default-backend is not created and the nodeport set for the first ingress resource is set to last of ALB rule.
ALB rule by created alb-ingress-controller in management console.
The rule for "/echo" and last is set to same target group.

Is it supposed to be such a rule?(I created this manually)

ingress resource:
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  name: echoserver
  annotations:
    kubernetes.io/ingress.class: alb
    alb.ingress.kubernetes.io/scheme: internet-facing
    alb.ingress.kubernetes.io/subnets: subnet-XXXX,subnet-XXXX
    alb.ingress.kubernetes.io/security-groups: sg-XXXXX
    alb.ingress.kubernetes.io/tags: Environment=dev,Team=test
spec:
  rules:
  - host: example.test.info
    http:
      paths:
      - path: /echo
        backend:
          serviceName: echoserver
          servicePort: 80

This is a duplicate of #405


next-error jumps to second error/warning if there is no output preceding the first

In some cases, cargo will not output anything like "Compiling crate" (I don't know why...).  Then, the first error/warning is apparently considered "visited" by next-error it jumps immediately to the second error.  With previous-error you can still go back to it.
(In normal compilation-mode compilation-start inserts the timestamp and the command at the beginning, so this never happens.  Why is compilation-start not used by rustic-compile?)

Hi birkenfeld. There were several reasons, but I wrote this almost as long ago as it was that I talked to you. I'd have to think about it.

No worry, I just switched modes again and wanted to share my immediate findings. It is of course not a problem for now to use normal compilation mode again, but I'm already missing the nice colors :)

Thanks I appreciate that ;)
You could try to motivate @mookid to merge https://github.com/rust-lang/rust-mode/pull/397 by tesging his branch =)

Is this fixed now ?

Which commit could have fixed it?

Wrong issue, sorry.

-> #355


Cucumber Linux isn't supported

Description
Could you add Cucumber Linux?

Output of cat /etc/os-release, lsb-release, lsb_release. Also  what package manager does it use?

Yeah, about that. I actually went out of my way to find an obscure af distro to submit, without realizing I would then have to download it, install it, and do the things in order to get it in neofetch.
So I thought "f*ck it" and flashed it on a USB drive. But it's in an early alpha and I can't get it to boot. Whether you decide to go through the trouble of adding it is up to you I guess.

/etc/os-release isn't a file that exists in my system, but /etc/lsb-release gives:
# Do NOT edit this file. It is generated automatically as part of the Cucumber
# Linux installation process.
DISTRIB_ID="Cucumber Linux"
DISTRIB_RELEASE="1.1"
DISTRIB_CODENAME="Luke"
DISTRIB_DESCRIPTION="Scott Court's Cucumber Linux Distribution"

# The following fields are not required as part of the LSB. They are here to
# provide supplemental information only.
DISTRIB_TYPE="stable"
DISTRIB_ROLLING_RELEASE="1.x"
DISTRIB_ARCH=i686

lsb_release is the same also.


[Master]Fix boot volume not extend

get device name of root fs by 'lsblk' rather than 'df', as sometimes 'df' give stale dev name
execute 'partprobe' before 'resize2fs', so that 'resize2fs' can recognize the extended partition

Signed-off-by: Da Long Wang<EMAIL_ADDRESS>

some conflict here..

@jackydalong please backport this, thanks

backported to 1.5.n @jichenjc


Check if game is lost

If there are no assets and you have less money than cheapest trip, you lost

if the budget is >1 million, you won


Forward geo error: invalid type: sequence, expected a string

use geocoding::{Opencage, Forward, Point};   - using version 0.3.1 in cargo.toml
let oc: Opencage = Opencage::new("xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx".to_string());
let address = "Schwabing, München";
let res: Vec<Point> = oc.forward(&address).unwrap();
let first_result = &res[0];
The third line above responds with the error below:
thread 'main' panicked at 'called Result::unwrap() on an Err value: Error { kind: Json(Error("invalid type: sequence, expected a string", line: 1, column: 393)), url: None }', src/main.rs:91:53
Could it be due to a change in API with OpenCage? I tried OpenStreetMap too, which also responds with a type error. 3rd option not workable as it's limited to Switzerland vs being worldwide.
Use case is to get long lat by address (city, state, zip, country).

Hi,
Ed from OpenCage here.
Can you establish at all what is causing the error, I mean which part of the response it is dying on?
The most recent change was over a month ago (see Change Log), which I suppose this could be related to.
Basically in the components portion of the response there can now be a key ISO_3166-2 which is a list of ISO 3166-2 codes for the location. Here is a detailed blog post describing the addition to the API.
I hope that helps.

sorry, I guess I am not formulating my question clearly. Yes, I see the error message, it was posted in the initial message in this thread. My question is which part of the JSON is causing this errror in the decoding? Which content is at "column 392"?
Please bear in mind I know nothing about Rust and am not the maintainer of this module.
Our API is returning:
     ...
      "components": {
        "ISO_3166-1_alpha-2": "DE",
        "ISO_3166-1_alpha-3": "DEU",
        "ISO_3166-2": [
          "DE-BY"
        ],
        "_category": "place",
        "_type": "neighbourhood",
        "city": "München",
        "city_district": "Schwabing-Freimann",
        "continent": "Europe",
        "country": "Deutschland",
        "country_code": "de",
        "political_union": "European Union",
        "postcode": "80805",
        "state": "Bayern",
        "state_code": "BY",
        "suburb": "Schwabing"
      },

my guess is it is the newly-added ISO_3166-2 that is causing problems as Rust is expecting the value to be a string, not a list. Please see the blog post to understand why it needs to be a list (in short: many countries have multiple ISO 3166-2 codes).

@freyfogle this seems to be the issue indeed as components is defined as a HashMap<String, String>.

Probably can test this by replacing the String value with something like HashMap<String, Component>
#[derive(Debug, Clone, Serialize, Deserialize)]
pub enum Component {
    String(String),
    List(Vec<String>),
}

Hi Ed, I'm not feeding or getting any json.

yes, you are getting back JSON when you query our API
Please see pur demo page: https://opencagedata.com/demo
Put in the query "Schwabing, München" and you can see exactly the JSON we are returning.

or HashMap<String, serde_json::Value>

That seems to work. @b4l would you like to push a PR with the fix?

@urschrei yes, I can do that.

v0.4.0 is on crates.io, with the fix. Thanks @freyfogle and @b4l for the diagnosis and PR

That error doesn't originate in geocoding I'm afraid.

@AppyCat this seems to be expected, resulting from how the blocking reqwest Client interacts with the tokio runtime: https://github.com/seanmonstar/reqwest/issues/1017

Hi yes, there are many differences between OpenStreetMap and OpenCage.
I dont think the rust SDK is really the place for the discussion
Spend some time reading the OpenCage geocoding API docs.


PHP: Strange keyword highlightning in string

Keywords get highlighted in strings. Is this intended? If this issue isn't related to this project, please tell me which module is responsible for this behavior.

I am interested in figuring out this too.


[from noice.nvim] Anyone experience this strange indent on the menu popup?

Hi, I just encountered a problem with noice.nvim. It seems that this is caused by nui.nvim according to the thread there but I'm not sure. Really need your help... (please forgive my copy-paste, since I believe I have provided enough context there.)

I have to type one letter to get rid of it. I don't like it.
https://user-images.githubusercontent.com/24765272/209911616-2686f8d8-2160-4fa0-9e0a-e057117a8916.mov
This is my config, if you could take a little time to test it. (As you can see, I didn't add anything related to indentation, and I only added nui into dependencies without any customization; The font I'm using is this Fantasque San Mono (Regular) maybe this is related but I'm not sure...).
(Also, it looks like the padding is applied only when my cursor moved. Not sure...)
config = function ()
  local wininfo = vim.fn.getwininfo()[1]
  local offset_top = math.floor(wininfo.height/4)
  local styles = {
    position = { col = '50%', row = offset_top },
    border = { style = 'shadow', padding = { 0, 0, 0, 0 } },
    size = { width = 100 },
  }
  require('noice').setup {
    lsp = {
      -- override markdown rendering so that **cmp** and other plugins use **Treesitter**
      override = {
        ['vim.lsp.util.convert_input_to_markdown_lines'] = true,
        ['vim.lsp.util.stylize_markdown'] = true,
        ['cmp.entry.get_documentation'] = true,
      },
    },
    hover = { enabled = true },
    signature = { enabled = true },
    messages = { view_search = false },
    views = {
      cmdline_popup = vim.tbl_deep_extend('force', {}, styles, {
        win_options = {
          winhighlight = { Normal = 'NoiceCmdlinePopup' },
          winblend = 10,
        },
        size = { height = 2 },
      }),
      popupmenu = vim.tbl_deep_extend('force', {}, styles, {
        win_options = {
          winhighlight = { Normal = 'NoiceCmdlineItems' },
          winblend = 15,
        },
        position = { row = offset_top+3 },
        size = { height = 15 },
      }),
    },
    presets = {
      lsp_doc_border = true,
      long_message_to_split = true,
      inc_rename = true,
    },
  }
  vim.api.nvim_create_autocmd({ 'ColorScheme' }, {
    pattern = '*',
    callback = function ()
      vim.cmd([[
        hi NoiceCmdlinePopup guifg=#b8b8ff guibg=#101010
        hi NoiceCmdlineItems guifg=#c9d1d9 guibg=#101010
      ]])
    end
  })
end

I tried the exact same config and it doesn't happen for me.
If you can create a minimal reproducible issue with nui.nvim (not noice.nvim), I'd be happy to look at it further.


Typescript samples are incomplete

Neither of these contain a wwwroot, so when run they fail to load:
https://github.com/aspnet/AspNetCore.Docs/tree/master/aspnetcore/tutorials/signalr-typescript-webpack/sample
I download and open the SignalRWebPack.csproj in VS2019 community, and attempted to run the project, and since there's nothing in wwwroot it cannot connect to the site.
These are linked from: https://docs.microsoft.com/en-us/aspnet/core/tutorials/signalr-typescript-webpack?view=aspnetcore-2.2&tabs=visual-studio
I've spent a couple weeks trying to get through this tutorial, and it's frustrating that you don't even have a valid working sample project.

Our .gitignore removes wwwroot to prevent bloat. The samples are primarily to document the code. They sometimes need a little work to run.

I've spent a couple weeks trying to get through this tutorial

Were you unable to follow the instruction at
Use ASP.NET Core SignalR with TypeScript and Webpack
If so, Can you create an issue from the doc page? That will connect the doc to the issue. Go to the bottom of the page and select the Content feedback issue.

Per comment hub:
Can we get a less complicated startup that doesn't use webpack and libman? The article should explain how to setup typescript without these complexities, and leave them for a separate article.
npm init -y doesnot create any package.jkson
cc @scottaddie


Intermittent FPS drops

After conducting binary testing (50% on, 50% off, then repeat) on plugins, I have determined that Herbi AFK is causing intermittent FPS drops. Every 15 or so mins FPS will drop to 20 FPS for a few seconds--not horribly long, but enough to cause issues in-game.

@nomadicvice ill look into it, I have some questions:

where were you in game when you noticed the lag? in the herbiboar area of elsewhere?
can you post a screenshot of your herbi-afk configs

Can do! Also a preface that I've now also noticed this issue happening with the equipment inspector plugin (both that AND herbi AFK would cause the drops when solely enabled), so I'll be reaching out to them on their github as well.

This has been happening all over the game. Areas I've noticed include wintertodt, tempoross, hunter areas, edge bank, etc


[Question] Item Inventories for Mining Missions

Hello Zmaster587,
I was wondering what forms of item inventories is accepted by Advanced Rocketry. I'm aware that normal chests work, and I have to assume shulker boxes do as well (I haven't tested it yet since I've not gotten to the end just yet.), but Stone Storages from Multi-Storage don't seem to work. Or are the only accepted inventories vanilla?
Also, as a side note, I can see the chests when I shift-right click the rocket entity, but when I try and open them, I'm immediately booted out. For my specs, they are written in https://github.com/zmaster587/AdvancedRocketry/issues/998
Thank you,
greenking13

In theory, any block that implements IInventory should work.  If you're running 1.12 it's possible that they switched to the item capability system and removed IInventory as an implemented interface.
As for opening the chests in the rocket that sound like a failure of the ASM code that overrides the distance check.   how big is the rocket and how far from it are you standing when you try to open the chests.  Also are you running anything like sponge?

We are not running sponge or the likes. The rocket is...

that. So, quite small and fat. I am also standing about the same distance away as I am currently. (It is not currently built, I disassembled it due to mining mission issues. We seem to have borked the asteroids such that they only have air of 0+-0 as their contents, even if it says it has XYZ of ores. I'm waiting until the pack dev pops on later today to discuss this more in depth with him.)

Every block from computercraft isn't working too. I think it is because every peripheral uses the same block id(computercraft:peripheral), but different metadata.


Several clusters using same Hdfs namespace

Hello,
For our project, we're forced to use the same Hdfs namespace (hosted in a remote distributed storage) between 2 TDP Clusters.
Unfortunately in this context, the last one who deploys recovers the Kerberos rights (e.g. we use an outsourced and shared KDC/LDAP), whereas the other one lost them, especially for the Edge Node (e.g. headless keytabs).
After checking, we noticed that the principals for some main service accounts (e.g. hdfs, yarn, ...) are recorded like that, for example : "hdfs@realm", and are not suffixed with this kind of pattern "hdfs/<ansible_fqdn>@realm".
As a consequence, it seems impossible for the KDC instance to make a difference between clusters nodes names, and then share the same Hdfs namespace.
The workaround we found is to force the creation of those principals with suffixed ansible FQDN like "hdfs/<ansible_fqdn>@realm" in all.yml (e.g. yarn_headless_principal: yarn/{{ ansible_fqdn }}).
Thanks to this, we are now able to distinguish users whether they come from one cluster or another.
Our questions about it :

Is there a reason why headless keytabs are not suffixed with the ansible_fqdn (e.g. as described in this issue) ?
Why are some of them written without being set with variable (e.g. principal: "zookeeper/{{ ansible_fqdn }}" in this tasks, contrary to this one hbase_principal: "{{ hbase_rest_kerberos_principal }}") ?

Thank you in advance for your help and if this is not clear do not hesitate to ask us,
Best regards,
C.L.

Hello @ChristopheLRTE ,
Regarding you questions:

Is there a reason why headless keytabs are not suffixed with the ansible_fqdn (e.g. as described in this issue) ?
--> Having a "headless" precisely means that: a principal that is not bound to a host. This is made on purpose.

Why are some of them written without being set with variable (e.g. principal: "zookeeper/{{ ansible_fqdn }}" in this tasks, contrary to this one hbase_principal: "{{ hbase_rest_kerberos_principal }}") ?
--> Indeed there is a difference in the way we deal with service principals from one role to another.

Feel free to open a PR if you need to variabilize any role's service principal 👍.
As for the headless keytab, what might work for you would be to define the principals like hdfs-cluster1@REALM and hdfs-cluster2@REALM to differentiate. auth_to_local config should then do the job from here.


Reset unavailable after 12 hours

if a call is marked as "unavailable", and is older than 12 hours, then when it's read back into the log, it's skipped
when re-entering the app, each screen that display a log will check to see if there are any old, unavailable calls in the log, and will remove them (by reloading the log) and then reloads the visible table.

Addresses #96

I think the better option is to just not treat Unavailable as Completed?
We still want to log Unavailable and also be able to log all possible attempts over time, for the Your Impact screen, and this would remove that level of detail in the logs.
Thoughts @subdigital?

actually, yeah, that makes sense.

Going to close this PR, as I think @mccarron is right. I'm still thinking about the best approach to solve this (maybe a isCompleted flag on ContactLog, but in any case it warrants a new PR.


Moving HYPRE memory allocation macros to a common location [hypre-mem-macro-dev]

This PR hopes to avoid code duplication when defining hyper memory allocation macros which are currently needed in two locations. There is at least one other PR #584 which will also need these definitions and there may be more in the future.

PR
Author
Editor
Reviewers
Assignment
Approval
Merge

#1172
@mlstowell
@v-dobrev
@delyank + @v-dobrev
04/29/20
⌛due 05/13/20
⌛due 05/20/20

The error in travis seems to be real. @mlstowell, can you double check?

The error in travis seems to be real. @mlstowell, can you double check?

I took a quick look earlier and didn't find anything.  I've had a Valgrind job in the queue all afternoon but it hasn't started yet.  I'll keep an eye on it.

The error in travis seems to be real. @mlstowell, can you double check?

I took a quick look earlier and didn't find anything. I've had a Valgrind job in the queue all afternoon but it hasn't started yet. I'll keep an eye on it.

I merged master into this branch last night and the tests now pass.

LGTM

Merged in next for testing...


Pragrawa

Adding ProtectionPolicy Generated Code

Can one of the admins verify this patch?

Hi @pragrawa, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.azure.com.

TTYL, AZPRBOT;

By mistake I had send this request. I will close it.


Replace react-helmet with built in functionality from gatsby

Warning from yarn build step
warning gatsby-plugin-react-helmet: Gatsby now has built-in support for modifying the document head. Learn more at https://gatsby.dev/gatsby-head

https://github.com/gatsbyjs/gatsby/issues/37701

From what I understand, this is not a bug, this is simply a warning message from Gatsby to YOU, saying that you don't need to use react-helmet anymore.


TPS display

It would be nice if this could display the current server TPS as well.

I am happy to add this in update v4.3.0, which will also add horizontal equipment hud, text alignment, and a fix for debug screen and hud overlap.


Now to to access the application via ALB created

Hey Folks,
I am using aws-alb-ingress-controller in my k8s setup. I created ingress which ultimately creating the alb loadbalancer and target group. I used route53 with the same alb.
I am getting "This site can’t be reached" when hitting the route53 url. I even tested the record set. I don't see any error then why I can't access my application?
DNS request sent to Route 53
xxxx.xxx.co.     IN     A
EDNS0 client subnet IP
24
DNS response code
NOERROR
Protocol
UDP
Response returned by Route 53
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>

Also when I did the port-forward on the pod(alb is attached to the same pod via service). I am able to access the application. That means alb is not able to access the service/pod. I even checked the health. Both worker node is healthy.

Can you check the logs
kubectl logs -f alb-ingress-controller-cdd84df5d-tpx95 -n kube-system

Hi, seems you are using a private ALB.  Is it intended?
If not, you should add annotation alb.ingress.kubernetes.io/scheme: internet-facing to your ingress to make it a public ALB.


Reset the $_SESSION variable if the session isn't started

It seems if the session isn't started the function session_unset() doesn't work.
I've added an else statement where we assign a new array to the session variable to reset it.
Locally the tests are succeeding now 👍

The build also seems to run nicely: https://travis-ci.org/onelogin/php-saml/builds/621519897


Invoker

in the future we will need integration tests, a simple way to do this is to use invoker

@l3r8yJ ok, I will mark this


How to use the trained model?

Do you have pre-trained model?
How to use onnx version for such model?
I tried https://github.com/HemingwayLee/deepfont-implement, and the result seems not correct for Japanese fonts.

I have trained a toy model for Japanese fonts months ago. You can download the model here. It can only recognize the following five fonts. You can download the fonts here.

GenEiAntiqueNv5-M.ttf
GenEiKoburiMin6-R.ttf
GenEiLateMinN_v2.ttf
GenEiMGothic2-Bold.ttf
GenEiMGothic2-Regular.ttf

I have refactored my code and I am currently trying to train a larger model which can recognize 169 Japanese fonts. I may release it soon.

Good. Will wait for your large model to test.
Ms gothic etc.

import torch
cnn_net = torch.load('../data/models/cnn_model.torch')
from PIL import Image
cnn_net = torch.load(r'L:/font/cnn_model.torch')
img = Image.open(img_path)
img = Image.open(r'F:/24.png')
img = img.copy().convert("L")
label = cnn_net(img)
print(label)
报错===========================================
D:\Python3.8.8\python.exe F:/pycharm2020.2/universal-font-recognition-main/eval.py
Traceback (most recent call last):
File "F:/pycharm2020.2/universal-font-recognition-main/eval.py", line 12, in
label = cnn_net(img)
File "D:\Python3.8.8\lib\site-packages\torch\nn\modules\module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "F:\pycharm2020.2\universal-font-recognition-main\utils\CNN.py", line 48, in forward
X = self.Cu(X)
File "D:\Python3.8.8\lib\site-packages\torch\nn\modules\module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "D:\Python3.8.8\lib\site-packages\torch\nn\modules\container.py", line 204, in forward
input = module(input)
File "D:\Python3.8.8\lib\site-packages\torch\nn\modules\module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "D:\Python3.8.8\lib\site-packages\torch\nn\modules\conv.py", line 463, in forward
return self._conv_forward(input, self.weight, self.bias)
File "D:\Python3.8.8\lib\site-packages\torch\nn\modules\conv.py", line 459, in _conv_forward
return F.conv2d(input, weight, bias, self.stride,
TypeError: conv2d() received an invalid combination of arguments - got (Image, Parameter, Parameter, tuple, tuple, tuple, int), but expected one of:

(Tensor input, Tensor weight, Tensor bias, tuple of ints stride, tuple of ints padding, tuple of ints dilation, int groups)
didn't match because some of the arguments have invalid types: (!Image!, !Parameter!, !Parameter!, !tuple!, !tuple!, !tuple!, int)
(Tensor input, Tensor weight, Tensor bias, tuple of ints stride, str padding, tuple of ints dilation, int groups)
didn't match because some of the arguments have invalid types: (!Image!, !Parameter!, !Parameter!, !tuple!, !tuple!, !tuple!, int)

Process finished with exit code 1


The two rabbits are still not staying in sync.

The two rabbits are still not staying in sync.

Are you talking the 2 degrees C or the CO2 ppm?

Linked to #6.
I might even say #6 replaces this. What do you think?


order of filters

I'm using ransack for search and I added a filter for searching
filter :q, apply: ->(_records, value, _options) {
    Driver.search(value.first).result
  }

The problem is when I try to use filters on the resource, for example, when I do drivers?filter[supplierId]=4&filter[q]=name
It doesn't work correctly and I get all results without the first filter applied. It only works when I order the filters, for example, drivers?filter[q]=name&filter[supplierId]=4 works correctly
I don't understand what's causing this and why the order is important

This is because you're not using the records parameter in your custom ransack filter. Filters need to build on the existing record set, otherwise they essentially reset everything (hence why it works only when the custom filter is applied first). This can also cause problems with anything else that builds up the query, such as sorting.
I haven't used ransack personally, but I believe just calling the search method on the records parameter will work.

Thank you for clarifying.


Exit sequencer when sequencing fails.

This PR allows the main thread to call http.Shutdown
when the sequencing thread exits.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #963   +/-   ##
=======================================
  Coverage   45.76%   45.76%
=======================================
  Files          28       28
  Lines        1993     1993
=======================================
  Hits          912      912
  Misses        901      901
  Partials      180      180

Continue to review full report at Codecov.

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


Change Feeds.get_feed_submission_result() to use method POST

I could not find documented which method should be used on each call. The MWS docs say GET or POST. But GET returns 404 for this request, and the scratchpad calls this as a POST. Maybe POST should be the new default, but I can't say I know enough about the MWS API to make that call.

Thanks for the pr.
Closing as this project is no longer maintained.
Please head over to https://github.com/python-amazon-mws/python-amazon-mws.


ion-datetime Keeps sliding even if finger long gone

Short description of the problem:
Im testing the ion-datetime on android 6.0 and it looks like that when I slide my finger and then lift it up
the scroller keep sliding slowly and it makes it hard to aim to the correct inpu
What behavior are you expecting?
Im expecting for the scroller to stop faster.
Steps to reproduce:

take android 6.0.1(might be able to reproduce on other version and os, but thats what I have)
create an ion-datetime
try to scroll, both fast and slow.

Which Ionic Version? 1.x or 2.x
2.0.beta.7
Run ionic info from terminal/cmd prompt: (paste output below)
Cordova CLI: 6.0.0
Gulp version:  CLI version 3.9.0
Gulp local:   Local version 3.9.1
Ionic Framework Version: 2.0.0-beta.7
Ionic CLI Version: 2.0.0-beta.24
Ionic App Lib Version: 2.0.0-beta.14
OS: Distributor ID:	Ubuntu Description:	Ubuntu 14.04.4 LTS
Node Version: v5.9.0
Can someone confirm this?

@brandyscarney  @jgw96  any news regarding this issue?

Hello, thanks for opening an issue with us! Would you be able to provide a plunker that demonstrates this issue? Thanks for using Ionic!

@jgw96 dont think plunker is relvant here since I was able to reproduce this only on android device  (didnt test it on ios). Tell me if you still need a plunker example

Hello @royipressburger, thanks but i was already able to reproduce this issue and will be investigating it more. Thanks for using Ionic!

@jgw96 any news regarding this issue?

I'm having the same problem with Android 6.0.1 and 4.0.2 and with iOS. The ion-date is very slow to select the dates
On browser this doesn't happen, it is fast to scroll down or up on the field.

Same here on Windows 10 universal app (mobile and desktop). Ion-date is verry slow in comparaison to the browser.

Same problems on Ionic 2 beta 11, browser and device.
Cordova : 6.3.1
ionic CLI : 2.0.0-beta.32
node : 6.2.2
gulp : 3.9.1

Same problems on Ionic 2 beta 11, browser and device.
Cordova : 6.3.1
ionic CLI : 2.0.0-beta.32
node : 6.2.2
gulp : 3.9.1

Any news regarding this issue?

Same issue here, slow on iOS and Android, both on physical devices and simulators.
Ionic 2 beta 11

I'm using the same version as @arturalkaim and I have the exact same problem, reproducible both on iOS and Android

Performance of Datetime and Picker has been improved a lot, I will keep this issues open since I think this particular performance issue can be improved even more.

For information a simple input type "date" will trigger the native date
selector, work both in android and ios. No more issues with the cordova
plugin or laggy experience with the ion-datepicker ;)
2016-11-15 11:50 GMT+01:00 Manu Mtz.-Almeida<EMAIL_ADDRESS>
Performance of Datetime and Picker has been improved a lot, I will keep
this issues open since I think this particular performance issue can be
improved even more.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/driftyco/ionic/issues/6613#issuecomment-260609804,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AChoZ21Rx19D-yKLPhC1it1oJ3tC1xrXks5q-Y59gaJpZM4IkDrx
.

--
Léger Stéphane
Designer UI / Developer Front-end06 03 99 30 53
www.legerstephane.com http://www.legerstephane.com

Hello! With the release of RC3 this issue is now fixed. The issue here was that the datepicker was so laggy that it was hard to pick the actual date you wanted. In rc3 we did a ton of perf improvements to the datepicker that solved this issue. Thanks for using Ionic everyone!

Thanks, Can confirm that this issue is fixed in rc3


mem-cache: Implementation of SMS prefetcher

This PR adds the SMS prefetcher described in this  paper.
On branch sms
Changes to be committed:
modified: src/mem/cache/prefetch/Prefetcher.py
modified: src/mem/cache/prefetch/SConscript
new file: src/mem/cache/prefetch/sms.cc
new file: src/mem/cache/prefetch/sms.hh
Change-Id: I68d3bb6cf07385177d0f776fb958f652cfc41489

@AbhishekUoR, the tests aren't running for this PR. We believe there's a bug with GitHub that occurs when a PR is initially created on the stable branch and then later changed to develop—GitHub doesn't seem to handle that well. I'll try closing and reopening the PR, but if that doesn't trigger the tests, I suggest you close this PR and open a new one with the same changes, targeting the develop branch instead of stable. Thank you for understanding.

Hi @AbhishekUoR, thanks for addressing my comments When reviewing the functionality I noticed we actually already have a spatial prefetcher implementation which is the Spatio-Temporal Memory Streaming Prefetcher (STeMS). Which I presume is building up on SMS adding temporal streaming.
Did you notice that prefetcher implementation in gem5 when you developed SMS?
I believe the two prefetchers should try to reuse as much code as possible. For example the AGT entry definition in the STeMS.
Another comment I have is that I see you hardcoded an LRU replacement policy with a deque. It would also be nice if we could make it configurable with the ReplacementPolicy Param, and to use the AssociativeCache as well.
Please let me know your thoughts

Hello @giactra, I initially tried using STeMS as SMS by disabling the TEMS component, but I found that they differ in how they learn patterns. Another reason for keeping SMS as a separate prefetcher is that ARM cores specifically use SMS and GSMS (for larger pages).
I chose not to use the STeMS infrastructure to develop independent code that I can test without worrying about issues related to associative set implementations, such as those mentioned here: https://gem5-review.googlesource.com/c/public/gem5/+/37597.
I agree with the idea of using a replacement policy parameter, which I may consider for future changes to SMS. Currently, I am focusing on implementing other prefetcher-related changes and bug fixes in gem5.

Hello @giactra, I initially tried using STeMS as SMS by disabling the TEMS component, but I found that they differ in how they learn patterns.

I didn't look into STeMS, but I was not proposing for you to tweak it to make it work like SMS, rather to extract the common code and use it as a base for both (if it is not the algorithm I believe there might still be chance to reuse some data structures). Anyway it is not a big deal and the amount of lines we save is probably trivial

I chose not to use the STeMS infrastructure to develop independent code that I can test without worrying about issues related to associative set implementations, such as those mentioned here: https://gem5-review.googlesource.com/c/public/gem5/+/37597.

I looked into the commit. I don't see anything functionally wrong in the indexing policy implementation as it is. Rather I agree the current interface makes its usage very weird for tables (instead of caches) because of the size/entry_size different meaning. But setting the (entry_size=1) and (size = assoc) effectively makes it work as a fully associative indexing policy which is what you are hardcoding in this PR
You can see this in the BaseIndexingPolicy constructor
https://github.com/gem5/gem5/blob/stable/src/mem/cache/tags/indexing_policies/base.cc#L60C1-L60C48
  numSets(p.size / (p.entry_size * assoc)),
  setShift(floorLog2(p.entry_size)), setMask(numSets - 1), sets(numSets),

Becomes
  numSets(1),
  setShift(0), setMask(0), sets(1),

Anyway I don't want this to be a blocker and I'd rather have this contribution merged rather than forcing you to reuse existing code.
As you are likely running some experiments to check the performance implementation of these prefetchers, you are probably the best person to make the proposed changes while ensuring there is no performance regression

Hi @AbhishekUoR and @Setu-Gupta,
Thank you for your contribution! Could you please update us on the status of this PR? I noticed that PRs #1461, #1454, and #1441 seem to be related and add prefetchers. It would be helpful if you could create a discussion post to track the status of these PRs and inform the community about the changes. Additionally, we have a gem5 developer's meeting next Thursday, October 10th. It would be great if you could join us to discuss these updates.

Hi @AbhishekUoR, do you think you will have some time to review my change requests?
If not, as I believe changes are minimal, we can consider merging this as is and we will update the changes ourselves with a separate PR. Let me know what you think about it


Custom storage key prefix

Closes #114.
The refactor part created a nasty indentation diff, maybe I should keep the indentation as before, it's quite awkward as it is already. (fixed this)
Makes it possible to change the prefix by
angular.module('app')
    .config(function ($sessionStorageProvider, $localStorageProvider) {
        $sessionStorageProvider.setKeyPrefix('customPrefix-');
        $localStorageProvider.setKeyprefix('fooBar-');
    });

Great one! Thanks!

I've got error
Uncaught Error: [$injector:modulerr] Failed to instantiate module webApp due to:
TypeError: $localStorageProvider.setKeyPrefix is not a function

Hey @furyscript!
Yes. the minified version is only updated on releases. I'll push one for you now to be inline with the current code.

Ok, I will wait next released, when the all are fixed :) Do you know when it maybe release?

I'll push one this weekend (since I'm away tomorrow) with this stuff and some more.
Thanks for being patient :)

@egilkh I didn't include any docs in this PR, will you fix this or do you want me to open another PR?

If you want to make one please do, if not I'll add it to my list and get it
done eventually :)
On Thu, Aug 6, 2015 at 10:43 AM, William Boman<EMAIL_ADDRESS>wrote:

@egilkh https://github.com/egilkh I didn't include any docs in this PR,
will you fix this or do you want me to open another PR?
—
Reply to this email directly or view it on GitHub
https://github.com/gsklee/ngStorage/pull/126#issuecomment-128293277.

--
Egil Hanger
Heimdal IT AS
<EMAIL_ADDRESS>/ 95737838


Fix corrupt storage data on Electron update

Closes #35

Been trying everything. The only solution I could find is to delete all the user data...

Need to merge and publish ASAP because everybody is facing this problem.


Will you publish the addition of Typescript types to npm?

Is your feature request related to a problem? Please describe.
-
Describe the solution you'd like
I saw that you merged my addition of Typescript types 😄. I was wandering when you would publish these to npm so that I can start using them in my project.
Describe alternatives you've considered
-
Additional context
Btw I've also added the command: npm run gen:types to generate the types from the source code. So whenever you make a change to the project in the future, please make sure to run this command before you commit the changes 🙂.

Hi! Sorry for the delay.
We've just published a new version. Could you please verify if the types are working?
Thank You.

Hi! Sorry for the delayed response.
The new version including the types works, thank you very much for publishing it!
Cheers


Unable to Access storage.list in CastLabs Electron 28 with Chromium 120 After Offline License Download

Description:
We are experiencing an issue after updating to CastLabs Electron version 28 (Chromium version 120). Previously, using CastLabs Electron version 27, we could access storage.list data across multiple domains without any problems. However, after the update, this functionality is no longer working as expected.
Steps to Reproduce:
We use Shaka Player version 2.0.0 for offline license download.
We open multiple Electron windows with different domains (e.g., localhost1, localhost2, localhost3) to speed up the offline download license process.
We randomly allocate the files to download the offline license.
After downloading, when playing the content, we open the player on the respective domain to access the relevant IndexedDB data.
We try to access the offline downloaded license through storage.list but receive an empty array.
If we open the console, navigate to the Application tab, and check IndexedDB, we can see the offline downloaded data.
Expected Behavior:
The storage.list should return the data of the offline downloaded licenses, just like it did with CastLabs Electron version 27.
Actual Behavior:
storage.list returns an empty array, even though the data is visible in IndexedDB when manually checked.
Additional Information:
Shaka Player Version: 2.0.0
CastLabs Electron Version: 28
Chromium Version: 120
The same setup worked as expected in CastLabs Electron version 27.
Could there be any recent security changes or restrictions introduced in Electron 28 or Chromium 120 that might be causing this issue?

Would you please help me?

Version 2.0.0 is no longer supported, see our policy at: https://github.com/shaka-project/shaka-player/blob/main/maintained-branches.md
Please test it with a newer version of Shaka, sorry!

Used v4.10 as per your recommendation, still same issue , is there any cross domain related changes happed in chromium recent released versions, we are download the offline license in localhost1 stored with same domain in indexedDB, while accessing offline license parent application is launching with localhost and player is loading in ifram with localhost1but storage.list() giving empty array ,please give the solution   , is anything related to "StoragePartitioning"

@joeyparrish can you help here? Thanks!

I assume it has to do with the different domains used by your application.  You should be able to investigate in the dev console, in the "Application" tab, which lets you see what's stored in IndexedDB.


Use path fixtures in more tests

Also fix the data_dir fixture to work if you're already in the tests directory.

Coverage decreased (-0.3%) to 82.729% when pulling 8cf3a6b54103ac768d0fd0174782c8c3a90b9fd7 on QuLogic:test-path-fixtures into 8aea0967e94a38afea4e4b1f97aa9cc2910bfb2e on Toblerity:master.

Coverage decreased (-0.3%) to 82.729% when pulling 8cf3a6b54103ac768d0fd0174782c8c3a90b9fd7 on QuLogic:test-path-fixtures into 8aea0967e94a38afea4e4b1f97aa9cc2910bfb2e on Toblerity:master.


Testing gcp

merge changes to master

merge changes to master with changes to workflow - namespace and service account for deployment, service and pods

Reviewed


[Refactor] Remove dead code from depthwise_conv2d for Intel graphics

After fix a66186b, I saw that it should be necessary to do the same fix
for depthwise_conv2d for intel graphics. I saw that we never used the
removed code and it is just the same code from
cuda/depthwise_conv2d.py. So we can use the cuda implementation when it
will be necessary.
Thanks for contributing to TVM!   Please refer to guideline https://tvm.apache.org/docs/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from Reviewers by @ them in the pull request thread.

@echuraev BTW is intel_graphics target supposed to be better than the generic opencl target? I did auto scheduling experiment on icelake iGPU using generic opencl and got good performance result (comparing relative performance with CUDA + NV GPU).

@elvin-n did some experiments with intel graphics. Maybe he can give more accurate information about intel grahics.

@masahi
I observed much better performance on Iris XE graphics with intel graphics than with generic gpu.
On ONNX Mobilenet v2 Generic OpenCL auto scheduler - 14ms, with AUTOTVM IG conv2d_NCHWc 16ms, with autoscheduler conv_packed_data 9ms, after some fixes in IG conv2d_NCHWc and autoschewduler - 4ms, I have not created PR yet since have some questions to IG primitives and wanted to verify everything on previous generation of Intel Graphics.
Eventually I was ably to find Intel graphics (UHD graphics 630) and the situation was much less optimistic, still continue evaluations.

Intel graphics introduce two convolution - NCHWc and packed_data(by fact it is blocked convolution by out channels with further mandatory conversion to planar layout). Both of these convolutions had accuracy issue, I fixed it with only for packed_data and !PR8201 was accepted, but have not fixed accuracy of NCHWc yet
I would say that in many cases NCHWc kernels themselves are better than generic OpenCL, but often additional conversion of data into NCHWc<-> NCHW eats all performance boost and even make topology work slower
I have not finished comparison on UHD630. On the same time I see 2x slowdown on this platform comparing to competitors. Wanted to take a look into this as well.
Probably it would be more correct way to add model distinguishing UHD and IrisXE in addition to the -device=intel_graphics, but there are several independent places in TVM dealing with device/model and it's hard for me to figure out the right scope of changes for adding model

Several more questions to you

which topologies have you used for your experiment with Skylake graphics?
can you try to tune with target="opencl -device=intel_graphics"?

Interesting! Since our intel_graphics work was done a couple of years ago, I didn't expect it to be relevant today.

I tested on following model/input, with Gen11, 1.1 fp32 peak tflops.

mlperf ssd-resnet34: input shape (1, 3, 1200, 1200)
DETR (https://github.com/facebookresearch/detr): input shape (1, 3, 750, 800)

The result was compared against GTX 1070 ti, with peak 8 TFLOPS.
All tuning was done with auto scheduler and NHWC layout.
End to end result:
mlperf ssd resnet 34
gen11: 1116.40 ms
1070 ti: 99.34 ms

DETR
gen11: 292.5 ms
1070 ti: 33.1 ms

I consider this results excellent for Gen11, given HW peak difference and the fact that auto scheduler was hyper optimized for CUDA + dGPU.

I can certainly trying tuning with intel_graphics. Thanks for the suggestion!

The problem is that scheduling for intel_graphics convolutions use(d) "warp" scope while it is not defined neither for OpenCL nor for Metal. And I do not know how it could work previously. I have not seen that warp scope was ever supported for OpenCL backend in TVM


Error while uploading

Receiving the following error when trying to upload - An error occurred on the server. Please check your inputs.
The inputs were the ZIP of the ad and the project name. Unsure of what else to check. This issue started to occur today, as far as I know. It was working yesterday.

I am looking into this issue. Will update this thread when I have more
information.
clc
On Wed, Jun 7, 2017 at 2:32 AM, zumek<EMAIL_ADDRESS>wrote:

Receiving the following error when trying to upload - An error occurred on
the server. Please check your inputs.
The inputs were the ZIP of the ad and the project name. Unsure of what
else to check. This issue started to occur today, as far as I know. It was
working yesterday.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/InteractiveAdvertisingBureau/HTML5-AdValidator/issues/15,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAouci_BUBB81ldysaFLfcX3hJMpsEVdks5sBm4SgaJpZM4NycXR
.

The root cause was a disk space issue. I have removed old files and the system is not back online and available for use.


MYSQL support

it would be useful if there is mysql support in the plugin so that if people have a bungee cord network that people who for example have a car also have it in another server

Yep, current system for saving vehicles isn't optimal, however, were I to implement new one, it would be necessary to move the existing data to the new system as well, which seems to be the most difficult part.
To understand, I am not sure whether I can just create a new system and tell everybody to delete all their cars and update - it would bring many problems.
For now, this is impossible, however, I would love to consider it with the introduction of this new system.

@Zettovec it's an option to let people choose the way of saving data, most people would be okay saving data into a yaml file. But bigger servers like having the option to choose for a database like mysql or SQLite. Giving the option would already be great!

I worked on an implementation with: https://www.spigotmc.org/resources/ndatabase-fast-data-model-creation-powerful-async-sync-api.107793/ a few months ago.
But I had to stop due to lack of time. As I unfortunately moved to a new OS 2 weeks ago, I no longer have the code.
It was possible, although very tiring, because you have to recast all the existing data to the NDatabase Format.


Failing to restore AdaBelief optimizer from checkpoint

Description
Brief Description of the PR:
This PR solves the same issue reported in https://github.com/tensorflow/addons/issues/2361, by avoiding storing a int32 variable.
Type of change

[x] Bug fix

Checklist:

[x] I've properly formatted my code according to the guidelines

[x] By running Black + Flake8
[x] By running pre-commit hooks

@juntang-zhuang
You are owner of some files modified in this pull request.
Would you kindly review the changes whenever you have the time to?
Thank you very much.

Thanks @bhack! When do you think it will be possible to have a release of tensorflow addons so that it is possible to have AdaBelief in production? The nightly version might not be accepted in my company :)
thx

You can subscribe to https://github.com/tensorflow/addons/pull/2696

LGTM, thanks for fixing the problem.


请问如果我的训练视频不是25fps，哪些地方需要修改呢？？

数据预处理？以及渲染生成视频的部分？谢谢

You need to use ffmpeg to process the original video to 25fps, 512x512.

You need to use ffmpeg to process the original video to 25fps, 512x512.

i want to train model about 256x256，how can i do?
thanks in advance


Skip components by env-var

Required for migrating components being in the OSS component list, like telemetry or serverless
Configurable via https://github.com/kyma-project/control-plane/pull/2933

/test all

/test pull-main-cli-integration-k3d pull-main-cli-kyma-release-upgrade-k3d

/test pull-main-cli-kyma-release-upgrade-k3d

/test pull-reconciler-checkconfig


StarRatinngOptions glowColor is being ignored

starRatingOptions: StarRatingOptions(
      initialRating: 0.0,
      glowColor: Colors.green,
    ),

Still the stars are shown in orange
I migrated code from rate_my_app: ^0.6.1+3 to rate_my_app: ^1.1.0+1
Previously this worked fine with old format of styles which I found better
starRatingOptions: StarRatingOptions(
        initialRating: null,
        starsBorderColor: Colors.green,
        starsFillColor: Colors.green),

with new version I converted to
    starRatingOptions: StarRatingOptions(
      initialRating: 0.0,
      glowColor: Colors.green,
    ),

but as mentioned color does not work. Would be great to again have possiblilty change border and star fill color like in previous version, was very clear customisable and worked great

Glow color does not modify widget color. It only changes the 'glow' when you interact with the rating bar.
I added a PR to fix this issue and give the ability to use custom widgets. Hopefully, it will get merged. Or maybe somebody will come up with a better solution.

Hi, thanks I would suggest you just revert it as it was back in old version  ^0.6.1+3, worked fantastic allowing to customise both colors like:
starRatingOptions: StarRatingOptions(
        initialRating: null,
        starsBorderColor: Colors.green,
        starsFillColor: Colors.green),

Well, I added only the item color change. I'm not a writer of this repo, so I don't know why they changed it, but I think that it is because of the underline dependency, 'flutter_rating_bar' changed. It has no longer has those fill-color and border-color properties. So I don't know whether it is possible to revert those changes. But I'll try to come up with some workaround to give support for both color changes.

This PR #104 now adds support for border color customization as well


[DEVONthink] Error upon typing any search term

Extension
https://www.raycast.com/bgnfu7re/devonthink
Description
I have attached the error as a file as it was too long to fit in the description box.
devonthink_raycast_error.md
Steps To Reproduce
Error message too long to be allowed by Github so I cut off half of it.
Current Behaviour
"something went wrong" error message/image whenever I type in a search term. Sometimes I am able to see the initial search results but as soon as I press a key to navigate through them, the error immediately pops up.
Sometimes the error will be there from the beginning.

Expected Behaviour
No response

@godette please try disabling this tick


Exactly 1 match required, but found more than 1 | arm-template (schema)

Most of these issues were fixed in 0.8.1, released 2 days ago.


[DO NOT MERGE] 3) Rename edition description and description string

https://trello.com/c/Fpgjdydt/915-sort-out-the-json-description-field-on-edition

All looks good 👍  - nice commit messages too! I think once the comments are addressed we should give it a spin on integration

This is working as expected on integration 🎇

Whoops, good spot, will fix now!
Emma Beynon
Junior Technologist, GOV.UK http://gov.uk/ Custom Formats
Government Digital Service

On 27 Apr 2017, at 09:33, Kevin Dew<EMAIL_ADDRESS>wrote:
@kevindew requested changes on this pull request.
In app/presenters/edition_presenter.rb https://github.com/alphagov/publishing-api/pull/887#discussion_r113641884:

@@ -49,6 +50,13 @@ def rendered_details
{ details: details_presenter.details }
end

We're keeping this until such time as we decide to remove description_json

entirely, so that we don't lose the data in case we decide to revert.

def description=(value)

 super(value)

 self.description_json = { "value" => value }

end

I think this has ended up in the wrong class? should be in the edition model
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/alphagov/publishing-api/pull/887#pullrequestreview-35046253, or mute the thread https://github.com/notifications/unsubscribe-auth/AMz-VKVI1RAiD11frJGwez_RTtuXm6Zqks5r0FLMgaJpZM4NDSPe.


pkg/cluster: Use Action for fixupClusterSPObjectID

Which issue this PR addresses:
[ARO-1919] AAD: Migrate ARO-RP Azure AD Graph -> Microsoft Graph
What this PR does / why we need it:
Minor follow-up to https://github.com/Azure/ARO-RP/pull/1970 where @bennerv requested this in his review.
The fixupClusterSPObjectID step in PUCM uses only the MS Graph client (no ARM calls), and the client already handles HTTP 401 Unauthorized errors by refreshing its authorization token. So there's no need to use the AuthorizationRetryingAction wrapper.
The 401 Unauthorized error handling logic for the MS Graph client is in the github.com/microsoft/kiota-http-go module.
Test plan for issue:

Create an ARO cluster.
Run az aro update --refresh-credentials on it.
Verify it completes successfully.

Is there any documentation that needs to be updated for this PR?
No, this is just an internal change.

Holding until we can sort out some permission issues in the dev subscription.  It's blocking a successful test.
$ az aro update --refresh-credentials --name $CLUSTER --resource-group $RESOURCEGROUP
...
Traceback (most recent call last):
  File "/home/mbarnes/git/github.com/mbarnes/ARO-RP/python/az/aro/azext_aro/custom.py", line 552, in cluster_application_update
    client_secret = aad.add_password(app)
  File "/home/mbarnes/git/github.com/mbarnes/ARO-RP/python/az/aro/azext_aro/_aad.py", line 55, in add_password
    cred = self.client.application_add_password(obj_id, {})
  File "/opt/az/lib/python3.10/site-packages/azure/cli/command_modules/role/_msgrpah/_graph_client.py", line 132, in application_add_password
    result = self._send("POST", "/applications/{id}/addPassword".format(id=id), body=body)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/command_modules/role/_msgrpah/_graph_client.py", line 55, in _send
    raise GraphError(ex.response.json()['error']['message'], ex.response) from ex
azure.cli.command_modules.role._msgrpah._graph_client.GraphError: Insufficient privileges to complete the operation.

@mbarnes were you ever able to get around the graph issues after the Azure AD admin made changes?

@mbarnes anything needed for this to get merged? Can we look to remove the ready-for-review label if its gonna stay on hold?

I guess it could use re-testing since the PR is a year-and-a-half old now.  It's just a minor cleanup to remove some unnecessary logic around MS Graph calls.  Or we could wait until ARO is fully migrated to azure-sdk-for-go, at which time AuthorizationRetryingAction should die naturally.

Closed in favour of: https://github.com/Azure/ARO-RP/pull/3694


sha256.h

I submitted a comment in Youtube, apologies for the duplicates.   When I compile, it does not find sha256.h. Can you help?

You need to install Cryosuite in your arduino library
https://github.com/jjssoftware/Cryptosuite


feat: list all raids on /raids

As title + fixing up of formatting and things

Super basic (read: ugly) listing of all raids on /raids
I think this should be merged in now. I will finish off the wireframe for this screen, and get someone on it.

Super basic (read: ugly) listing of all raids on /raids
I think this should be merged in now. I will finish off the wireframe for this screen, and get someone on it.


[move-prover] Implemented the spec pragmas "emits_is_partial" and "emits_is_strict"

(emits_is_partial == false) by default. If emits_is_partial == true, Prover skips the completeness check of the emits specs.

By default, (emits_is_strict == false), and Prover checks nothing about events if a function has no emits spec. If emits_is_partial == true, Prover checks that each function with no emits specs actually emits no events.

TODO:

Support the emits verification with opaque functions

Motivation
To support the "emits_is_partial" and "emits_is_strict" pragmas
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
cargo test

/land


Create LICENSE

Pull Request Test Coverage Report for Build 13

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

Totals

Change from base Build 10:
0.0%

Covered Lines:
73

Relevant Lines:
73

💛  - Coveralls


Results about occlusion edge detection

Hi,
Thanks for releasing your code! By the way, whether can you kindly offer results concerning about occlusion edge detection on PIOD/BSDS? For DOOBNet repo, they also offered this score.
Thanks by advance!

Hi,
The results can be obtained by running the testing script directly through the models we have uploaded. We will offer the occlusion edge detection results soon.
Best

The results have been updated. Good luck.


texture.set_flags does not work

Operating system or device, Godot version, GPU Model and driver (if graphics related):
macOS 10.13.1, Godot3 #bf45e569c
Issue description:
When using a Stream Texture, the set_flags method isn't working. This might be something about the new import workflow, but I'm using that to ensure, that Texture.FLAG_REPEAT is set in my Scrolling Background plugin.
I've attached a sample project, that outputs the current texture flags, modifies them and outputs them again. There's no change in the output value.
Steps to reproduce:

Add a Sprite
Select an image as a texture (I guess, this creates a StreamTexture)
in the script use set_flag to alter the texture flags to add the FLAG_REPEAT to it
➡ No change

Link to minimal example project:
TestSetFlags.zip

I tested on Ubuntu 17.04, changing texture flags in runtime don't seems to have any impact on the Sprite node in 237be766d4b0247d7ded0a9b85da12a989c561c8:

I'm unsure if this is normal in current import work-flow, but since we still have set_flags method inside Texture then I assume it should be usable...


Have to add quotes and escape them for startkey/endkey to work

        database.doc.all "dbname", {
          startkey: "\"alpha\"",
          endkey: "\"bravo\ufff0\"",
          include_docs: true
        }
        .each (row) => ...

Is this by design? This isn't the case for PouchDB. The extra escaped quotes are ugly.
(sorry for the coffeescript!)

This isn't by design. I believe this is the format that CouchDB expects and I agree that it is ugly. I'll take a closer look when I get a chance. Thanks for bringing this up!

I got a chance to look closer into this and this aspect appears to be something rooted in the CouchDB API. For example, you have to wrap the key in quotes even when using just curl:
curl -X GET 'localhost:5984/mydb/_all_docs?include_docs=true&startkey="bravo"'

As such, I'm not really sure that making the change in Slouch is the best move at this point.
You can make this a little prettier by using a single quote to wrap the keys, e.g.
slouch.doc.all('dbname', { startkey: '"bravo"' }).each(function (doc) {
  console.log(doc);
});

You have to wrap the startkey in quotes because the value passed to couchdb has to be a valid json object.
If you're trying to provide an interface for people moving from curl then I guess it makes sense to do this. I'm not sure what Nano does, but for PouchDB you don't have to. So I guess it comes down to who is your target user.


node-red-node-mongodb 66-mongodb.js, node-red aborts during start

I just installed node version 7.1.0 and re-installed node-red-node-mongodb
It falls over the connect timer (setTimeout) and will not start node-red:
in 66-mongodb.js:
var connectToDB = function() {
MongoClient.connect(node.mongoConfig.url, function(err, db) {
if (err) {
node.status({fill:"red",shape:"ring",text:RED._("mongodb.status.error")});
node.error(err);
node.tout = setTimeout(connectToDB(), 10000);
I removed () so the line is now:
node.tout = setTimeout(connectToDB, 10000);
Now node-red restarts properly...

Can you include the actual error message please

Actually ignore that last request, just spotted what you meant

Hi - thanks @pelis
closed by - a9d669411c7947bff22bda250e134dc11ded75d1
now v0.0.10


Inheritance not working

I've just updated my server to 1.12, and I'm having problems with PermissionsEx inheritance for essentials. It only appears to be for essentials though, everything else seems fine. For example, one of my Trusted members can use /fly, but I (owner rank) can not, even though it appears I should be able to. I'm unsure exactly if it's pex or essentials that is the issue, so I'm posting this on both pages in hopes of an answer. Below is my permissions.
https://pastebin.com/7SWQXjqx

The problem is likely in the group "Mod".  "Mod" inherits from "TMod", which does not exist, so the essentials.fly permission node never makes its way up the chain to the "Owner" group.

Crap, completely overlooked that. Updating the server at 1AM isn't the best idea. Thanks for that buddy, feel really dumb now.

You're welcome. Come on back if you need help with anything else. :-)


@action decorator in es6

How could I  replace the @action decorator in a react component? do I need to wrap my class component functions somehow?

@cancamilo please file general mobx questions on the mobx or mobx-react repo's


[VSCode] Auto completion not works for public func

Description:

Public function auto complete not works

Steps to reproduce:

Type public func and do [Ctrl] + [Space]

Affected Versions:
v0.982.0-RC
OS, DB, other environment details and versions:
Windows 10
Suggested Labels (optional):
[Type/UX]

Thank you Jerad for reporting this. We are fixing this.


Not able to render BitmapFont properly, I've generated font by gdx-freetype

When I render BitmapFont, it's not properly render. I am using latest version 1.9.6. When I tried with version 1.5.5, it's works fine.
I tried many different .ttf, Result are almost same for all files.
This is my Test Code :
public class MyGdxGame extends ApplicationAdapter {

	SpriteBatch batch;
	Texture img;

	BitmapFont font;
	OrthographicCamera camera;

	@Override
	public void create () {
		batch = new SpriteBatch();
		img = new Texture("badlogic.jpg");

		float w=Gdx.graphics.getWidth();
		float h=Gdx.graphics.getHeight();

		float worldWidth=400;
		float worldHeight=h / (w / worldWidth);

		camera=new OrthographicCamera();
		camera.setToOrtho(false,worldWidth,worldHeight);

		FreeTypeFontGenerator freeTypeFontGenerator=new FreeTypeFontGenerator(Gdx.files.internal("ariblk.ttf"));
		FreeTypeFontGenerator.FreeTypeFontParameter ftp=new FreeTypeFontGenerator.FreeTypeFontParameter();
		ftp.size=400;
		font=freeTypeFontGenerator.generateFont(ftp);

		font.setScale(0.04f);  // when I use version 1.5.5
               //font.getData().setScale(0.04f);    // when I use version 1.9.6
	}

	@Override
	public void render () {

		Gdx.gl.glClearColor(1, 0, 0, 1);
		Gdx.gl.glClear(GL20.GL_COLOR_BUFFER_BIT);

		batch.setProjectionMatrix(camera.combined);

		batch.begin();
		font.draw(batch,"LEADERBOARD",10,200);
		batch.end();
	}
}

And Result with both version

Does the same thing happen by using

font.getData().scale(0.04f);

instead of

font.getData().setScale(0.04f);

Because when I look into the docs, there are 3 different methods for the scaling of BitmapFontData:

setScale(float scaleX, float scaleY), individual axes
setScale(float scaleXY), both axes
scale(float amount), relative scale

I think you should be using the 3rd one instead of the 2nd, because you're right now not scaling it relatively.

@MrGussio,  I know the difference between these three methods.
1st  : -  Scales the font by the specified amounts on respective axes
2nd  : -  Do same thing as in 1st, with same amount on both directions/axes
3rd  : - Sets the font's scale relative to the current scale.
but this is not my question, Please read my issue once more !
When I scale down my font with large amount then not able to draw properly, I've attached image for the same.
Also no rendering problem when I use  version 1.5.5. There is some bitmapFont refactoring  on March 24, 2015 so I use previous version 1.5.5 for testing.

You need to do this font.setUseIntegerPositions(false); if you want to scale fonts to very small sizes.

@piotr-j Thanks for suggestion, but still same result with your suggestion.

Are you certain the the generator is the problem here? Try making the font in hiero instead and see if it renders correctly.

When I tried same .ttf file with gdx version 1.5.5, It's working fine so I conclude, there is some issue in BitmapFont Refectoring.

Many things changed. Its entirely possible that generator is broken for some fonts. Make the bitmap font in hiero and see if it works.

Yeah lot of changes. I've tested many .ttf files so I don't think so problem is in .ttf file, If problem is in .ttf then why result with version 1.5.5 is fine.  I have to use gdx-freetype this is my 1st requirement so I can't use hiero that generate BitmapFont(.fnt file and associated .png file)

Your 1st requirement should be to find out which part is broken. You have a .ttf on one end and broken rendering on the other. Try to figure out a way to narrow down where the problem is.


Estimate AWS cost for Test and Production environments

When trying to get AWS credits, they asked that we provide an estimate using their calculator.
We need a Test and Production environment. The Test env can be small as it's only used by contributors for integration testing and demos. The Production environment will need to handle a yet to be determined amount of load for the county-level and state-level milestones.
Test (US-West-1)

MySQL RDS - Single-AZ T3 Medium w/ 16GB GP SSD EBS
ECS - 2 x EC2 T3 Small w/ 16GB GP SSD EBS each
Application LB for ECS
1 GB of S3
VPC

Production (US-West-1)

MySQL RDS - Single-AZ T3 Medium w/ 16GB GP SSD EBS
ECS - 3 x EC2 T3 Small w/ 16GB GP SSD EBS each
Application LB for ECS
1 GB of S3
VPC

Shared

Domain on Route53

The DB doesn't need to be Mulit-AZ to start, but if the project gets any kind of traction and we care about uptime, we will need to move to a Multi-AZ MySQL RDS instance. The above uses General Purpose SSD, which under load may not provide the IO performance required. Before reaching a state-level amount of load we may need to upgrade to Provisioned SSD.
We assume we only need T3 Smalls for the Frontend servers. but depending on the memory footprint we may need to upgrade to T3 Mediums. The amount of EBS space required for the DBs and Frontend may also grow, but EBS is cheap so not a huge concern. As load increases, we will want to add additional EC2 instances to ECS. We have no way to reliably predict growth in load until we have more insight into the services real-work performance.
Use the AWS calculator to determine the cost of the environments. Determine the increase in cost if must use T3.Mediums instead of Smalls.

If anyone has comments, concerns, or feedback on the above requirements please comment.

First pass at this is $320.50/month.
For DB’s, assumed On-Demand.
For data transfers, made no assumptions.
https://calculator.s3.amazonaws.com/index.html#key=calc-BBCE25D6-9E59-41AC-B7C5-F15FE55072CB&r=SFO

Looking over the details, I don't think we need the VPN Connections for the VPC. Vanilla VPCs may not cost anything.

Revised without VPN is $247.30/month
https://calculator.s3.amazonaws.com/index.html#key=calc-1E39D743-C92D-4FDB-B4F1-552AF01AA37D&r=SFO
Reference pricing for T3 Mediums upgrade is $338.05/month
Going to submit the $247.30/month link for credits.

Looks good to me. Thanks @mattsayre.


use secure version of postgresql

Fixes #9543
Purpose of the pull request
use secure lib
Brief change log
pom change

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Codecov Report

Merging #9573 (ccba085) into dev (32f76e4) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head ccba085 differs from pull request most recent head 18c8b16. Consider uploading reports for the commit 18c8b16 to get more accurate results

@@            Coverage Diff            @@
##                dev    #9573   +/-   ##
=========================================
  Coverage     39.94%   39.94%
  Complexity     4432     4432
=========================================
  Files           831      831
  Lines         33328    33328
  Branches       3680     3680
=========================================
  Hits          13314    13314
  Misses        18785    18785
  Partials       1229     1229

Continue to review full report at Codecov.

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


Make exit code not 0 when validation fails

Currently, the validator returns a 0 exit code when validation fails:
[glatard@sapajou demo]$ bids-validator ./tar_subjects
The directory ./tar_subjects failed an initial Quick Test. This means the basic names and structure of the files and directories do not comply with BIDS specification. For more info go to http://bids.neuroimaging.io/
[glatard@sapajou demo]$ echo $?
0

Returning a non-zero value would be useful to stop a BIDS app when the input dataset is not valid.

This seems to be happening only if the quick test fails.