Bump expensify-common to v2.0.57

Details

Bump expensify-common to v2.0.57 to fix the issue of <emoji> being mistaken as <em>.
Fixed Issues

$ https://github.com/Expensify/App/issues/45635
PROPOSAL: https://github.com/Expensify/App/issues/45635#issuecomment-2235052840
Related PRs
https://github.com/Expensify/expensify-common/pull/760
Tests

Send this message:

~👿~ _abc_

Edit the message
Verify the message in edit composer matches the original message in step 1
Open workspace room
Change room description to the message in step 1 and save
Open room description page again
Verify the room description matches the original description in step 1

[x] Verify that no errors appear in the JS console

Offline tests

NA
QA Steps

Same as Tests

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

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

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Cherry-picked to staging by https://github.com/roryabraham in version: 9.0.14-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

This seems like a potential cause of this issue. I haven't tested yet, just leaving a note.

Nevermind, found the cause elsewhere.

🚀 Deployed to production by https://github.com/roryabraham in version: 9.0.14-6 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


hotfix: fix build.gradle.kts from the integration-tests package, so the integration tests can be executed

PR Checklist
PR Structure

[x] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[x] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[x] This PR's title starts with name of package that is most changed in the PR, ex.
paymentservice.stellar, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[x] This PR adds tests for the most critical parts of the new functionality or fixes.

What
Fix build.gradle.kts from the integration-tests package.
Why
So the integration tests can be executed.

Reference Server Preview is available here:https://anchor-ref-pr505.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr505.previews.kube001.services.stellar-ops.com/


[KYUUBI #1112] Upgrade scala to 2.12.15

Why are the changes needed?

Close #1112.
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Codecov Report

Merging #1120 (9bb1085) into master (7da30bd) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1120   +/-   ##
=========================================
  Coverage     79.19%   79.19%
  Complexity       90       90
=========================================
  Files           177      177
  Lines          6620     6620
  Branches        783      783
=========================================
  Hits           5243     5243
- Misses          923      924    +1
+ Partials        454      453    -1

Impacted Files
Coverage Δ

.../scala/org/apache/kyuubi/server/KyuubiServer.scala
56.52% <0.00%> (-2.18%)
:arrow_down:

...pache/kyuubi/sql/KyuubiQueryStagePreparation.scala
81.37% <0.00%> (+0.98%)
:arrow_up:

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 7da30bd...9bb1085. Read the comment docs.

Thanks, merging to master for v1.4.0


pipenv fails to install my repo: pipenv 2022.8.24

Issue description
pipenv install fails with new version (2022.8.24) for my github repo.
This morning I made some changes to my github repo, committed to a new branch and made a merge request. None of the changes touched pipenv settings, or changed the Pipfile. To my surprise, a couple of jobs which run pipenv install --dev --python=3.8 --skip-lock failed even before they seem to have installed any packages.
So I spun up a container with ubuntu:latest, built python 3.8 from source, installed git, checked out my repo and followed each command my github runner had followed. When I got to the pipenv install --dev --python=3.8 --skip-lock command, I got an error.
Expected result
I expected that, as usual, pipenv install --dev --python=3.8 --skip-lock would work.
Actual result
This command failed, with the following error trace:
 Traceback (most recent call last):
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pkg_resources/_vendor/packaging/requirements.py", line 98, in __init__
    req = REQUIREMENT.parseString(requirement_string)
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pkg_resources/_vendor/pyparsing.py", line 1654, in parseString
    raise exc
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pkg_resources/_vendor/pyparsing.py", line 1644, in parseString
    loc, tokens = self._parse( instring, 0 )
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pkg_resources/_vendor/pyparsing.py", line 1402, in _parseNoCache
    loc,tokens = self.parseImpl( instring, preloc, doActions )
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pkg_resources/_vendor/pyparsing.py", line 3417, in parseImpl
    loc, exprtokens = e._parse( instring, loc, doActions )
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pkg_resources/_vendor/pyparsing.py", line 1402, in _parseNoCache
    loc,tokens = self.parseImpl( instring, preloc, doActions )
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pkg_resources/_vendor/pyparsing.py", line 3739, in parseImpl
    return self.expr._parse( instring, loc, doActions, callPreParse=False )
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pkg_resources/_vendor/pyparsing.py", line 1402, in _parseNoCache
    loc,tokens = self.parseImpl( instring, preloc, doActions )
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pkg_resources/_vendor/pyparsing.py", line 3400, in parseImpl
    loc, resultlist = self.exprs[0]._parse( instring, loc, doActions, callPreParse=False )
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pkg_resources/_vendor/pyparsing.py", line 1406, in _parseNoCache
    loc,tokens = self.parseImpl( instring, preloc, doActions )
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pkg_resources/_vendor/pyparsing.py", line 2711, in parseImpl
    raise ParseException(instring, loc, self.errmsg, self)
pkg_resources._vendor.pyparsing.ParseException: Expected W:(abcd...) (at char 0), (line:1, col:1)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pipenv/vendor/requirementslib/models/requirements.py", line 966, in _parse_name_from_line
    self._requirement = init_requirement(self.line)
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pipenv/vendor/requirementslib/models/utils.py", line 194, in init_requirement
    req = Requirement.parse(name)
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pkg_resources/__init__.py", line 3139, in parse
    req, = parse_requirements(s)
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pkg_resources/__init__.py", line 3084, in parse_requirements
    yield Requirement(line)
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pkg_resources/__init__.py", line 3094, in __init__
    super(Requirement, self).__init__(requirement_string)
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pkg_resources/_vendor/packaging/requirements.py", line 100, in __init__
    raise InvalidRequirement(
pkg_resources.extern.packaging.requirements.InvalidRequirement: Parse error at "'.#egg='": Expected W:(abcd...)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/opt/hostedtoolcache/Python/3.8.13/x64/bin/pipenv", line 8, in <module>
    sys.exit(cli())
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pipenv/vendor/click/core.py", line 1128, in __call__
    return self.main(*args, **kwargs)
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pipenv/cli/options.py", line 56, in main
    return super().main(*args, **kwargs, windows_expand_args=False)
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pipenv/vendor/click/core.py", line 1053, in main
    rv = self.invoke(ctx)
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pipenv/vendor/click/core.py", line 1659, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pipenv/vendor/click/core.py", line 1395, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pipenv/vendor/click/core.py", line 754, in invoke
    return __callback(*args, **kwargs)
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pipenv/vendor/click/decorators.py", line 84, in new_func
    return ctx.invoke(f, obj, *args, **kwargs)
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pipenv/vendor/click/core.py", line 754, in invoke
    return __callback(*args, **kwargs)
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pipenv/cli/command.py", line 233, in install
    do_install(
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pipenv/core.py", line 2064, in do_install
    do_init(
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pipenv/core.py", line 1333, in do_init
    do_install_dependencies(
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pipenv/core.py", line 861, in do_install_dependencies
    batch_install(
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pipenv/core.py", line 752, in batch_install
    dep.is_direct_url
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pipenv/vendor/requirementslib/models/requirements.py", line 2856, in is_direct_url
    or (self.line_instance.is_direct_url or self.req.parsed_line.is_direct_url)
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pipenv/vendor/requirementslib/models/requirements.py", line 2554, in line_instance
    self.line_instance = self.get_line_instance()
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pipenv/vendor/requirementslib/models/requirements.py", line 2548, in get_line_instance
    return Line(line)
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pipenv/vendor/requirementslib/models/requirements.py", line 173, in __init__
    self.parse()
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pipenv/vendor/requirementslib/models/requirements.py", line 1304, in parse
    self.parse_name()
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pipenv/vendor/requirementslib/models/requirements.py", line 1027, in parse_name
    name = self._parse_name_from_line()
  File "/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/pipenv/vendor/requirementslib/models/requirements.py", line 968, in _parse_name_from_line
    raise RequirementError(
pipenv.vendor.requirementslib.exceptions.RequirementError: Failed parsing requirement from '.#egg='

Steps to replicate
docker run -it ubuntu

Then, inside the ubuntu container:
apt-get update
apt-get upgrade

apt-get install git
apt-get install -y make build-essential libssl-dev zlib1g-dev \
       libbz2-dev libreadline-dev libsqlite3-dev wget curl llvm \
       libncurses5-dev libncursesw5-dev xz-utils tk-dev
apt-get install python3-pip

wget https://www.python.org/ftp/python/3.8.13/Python-3.8.13.tgz
tar xvf Python-3.8.13.tgz
cd Python-3.8.13
./configure --enable-optimizations --with-ensurepip=install
make -j 8
make altinstall
python3.8 --version
cd ~

git config --global --add safe.directory /home/runner/work/diffcalc_API/diffcalc_API
git init /home/runner/work/diffcalc_API/diffcalc_API
cd /home/runner/work/diffcalc_API/diffcalc_API/
git remote add origin https://github.com/RAYemelyanova/diffcalc_API
git config --local gc.auto 0
git config --local --name-only --get-regexp core\.sshCommand
git -c protocol.version=2 fetch --prune --progress --no-recurse-submodules origin +refs/heads/*:refs/remotes/origin/* +refs/tags/*:refs/tags/*
git branch --list --remote origin/addResponseModels
git rev-parse refs/remotes/origin/addResponseModels
git checkout --progress --force -B addResponseModels refs/remotes/origin/addResponseModels

sed -i 's/editable *= *true/editable = false/' Pipfile

pip install pipenv
pipenv install --python 3.8 --dev --skip-lock

HOWEVER, running pip uninstall pipenv and then installing pipenv 2022.8.5, and re-running the pip install command, pipenv installs fine.

$ pipenv --support yields something that's over 65565 characters, so I don't know how I can post it here. Just look at the Pipfile.lock on the github branch addResponseModels at  https://github.com/RAYemelyanova/diffcalc_API

@RAYemelyanova I believe the problem is that lock file contains pipenv itself:
        "pipenv": {
            "hashes": [
                "sha256:6cc26b3b2095c2fbc71a5157443e98d80f779b0613e201bf77f745258ee036fb",
                "sha256:e9333cdd365aeb9c3f9460ac2d84c4750e3ffa8e766fe3d5de03d4496b55d851"
            ],
            "markers": "python_version >= '3.7'",
            "version": "==2022.8.5"
        },

So you're uninstalling the latest version of pipenv and replacing it with an older one is problematic -- I would suggest leaving pipenvout of thePipfileandPipfile.lock` altogether.

Ah, I see. Thanks for clarifying.


Pass a synthetic page variable in posts and other AsciiDoc files to indicate that the source is in AsciiDoc

In my layout, I have a chunk of code that looks like this:
{{ page.title | markdownify }}

I want to support AsciiDoc posts too so that I can modify that code to:
{% if page.asciidoc %}
  {{ page.title | asciidocify }}
{% else %}
  {{ page.title | markdownify }}
{% endif %}

But looks like at the moment there's no such variable added to the page object out of the box.
As a workaround, we can pass asciidoc: true in the front-matter of posts, but having this built-in functionality built-in will prevent errors (e.g. forgetting to add the variable to the front matter).

You can already check for the document property:
{% if page.document %}
  {{ page.title | asciidocify }}
{% else %}
  {{ page.title | markdownify }}
{% endif %}

However, I realize that's rather ambiguous. I could support the idea of adding a boolean property as well.


Unable to set scrollWheelZoom argument for dl.Map()

@emilhe Thanks for your work on this package! It's incredibly useful
When instantiating a dash_leaflet.Map() object the scrollWheelZoom parameter isn't accepting boolean arguments.
Code to reproduce:
import dash  # 1.12.0
import dash_leaflet as dl  # 0.0.16

app = dash.Dash()
app.layout = dl.Map(
    dl.TileLayer(),
    style={'width': '1000px', 'height': '500px'},
    scrollWheelZoom=False)

if __name__ == '__main__':
    app.run_server(debug=True)

Error message:

Thanks! I just (0.0.17) corrected the type verification of the scrollWheelZoom  parameter, so it should work now :)

Awesome, thanks for your help! I just updated the package and the scrollWheelZoom parameter is working now


Keep generated SMIL for partial tracks

If one uses the IngestService and adds so called "partial"-tracks to a media package, the ingest service generates a SMIL-Catalog for these partial tracks. Unfortunately this generated SMIL-Catalog is removed if the option "skip.catalogs.for.existing.events" is set to true, which is the default. This pull request fixes this behaviour.

@lwillmann, looks good. Just wondering if we don't want this in r/8.x since it's a bug fix.

@lkiesow yes, I think it would be good to have this in r/8.x. Is there something I have to do?

Yes, you would probably have to rebase your branch, push this again and then update this pull request to point towards r/8.x (“Edit” right of the pull requests title).
For the rebase, it's more or less:
git fetch upstream  ← assuming `upstream` is the remote pointing to the community repo
git checkout keep-generated-smil
git rebase -i upstream/r/8.x  ← remove all commits except yours
git push --force-with-lease origin keep-generated-smil

We can take a short look at this together on Tuesday if you want

Ok, thanks for the guidance @lars. Rebased to r/8.x


v0.2.1 does not swipe from left

Upgrading to v0.2.1 stopped the left swiping of pages to work. Using v0.1.6 with no other changes it works like a charm,
Only thing I changed (besides the version) was to go from onlySwipeFromEdge: true to canOnlySwipeFromEdge: true
Flutter doctor:
[✓] Flutter (Channel stable, 2.0.1, on Mac OS X 10.15.5 19F101 darwin-x64, locale en-NZ)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio (version 3.5)
[✓] VS Code (version 1.54.1)
[✓] Connected device (2 available)

Could you please add some code that reproduces the problem? The example app included in this project demonstrates the canOnlySwipeFromEdge parameter and works fine.

Ideally I will be able to do this. It is on my project at work and would require tracking down why it does not work. Could be down to the routing logic I have. When I have time I will look into it more

I am experiencing the same issue and tracked down the issue that's causing this bug, and it's "WillPopScope" if it is present it will not work.
return WillPopScope(
onWillPop: () async {
return true;
},
child: Scaffold(),
},
Hopefully you can recreate this issue!

@MarekBeers That seems to be the expected behavior: flutter/flutter#14203 discusses that swiping back does not work when using WillPopScope, even if its callback returns true. This package only extends CupertinoPageRoute to allow swiping back from anywhere on the page.

Ooo okay because in version 0.1.6 this plugin worked with WillPopScope.

That's interesting; I can try to investigate that but I'm currently in the process of writing my bachelor's thesis, so it likely won't be until July.

Mystery solved! I added WillPopScope to correct Android back button behavior and now I can't swipe to go back on iOS.
Working around it by setting onWillPop to null when I don't need it. In some cases that is straightforward, and probably what I should have been doing in the first place:
Old:
onWillPop: () async { if (editing) { ... } } : null

New:
onWillPop: editing ? () async { ... } : null

in others, where I really only used it to control Android back button behavior, this works:
onWillPop: defaultTargetPlatform == TargetPlatform.android ? () async { ... } : null,


[Question] Add microphone button to record speech?

I want to add microphone button in my asp.net core app, and to record speech and then using the bing speech-to-text API to convert it to text. I know the implementation of the API but I don't know how to add microphone in the view and then to use the API. I want to create something similar to google keyboard microphone.

@msBot1111 Can you provide some context for your project? How does this relate to Bot Framework specifically?

It's not related to Bot Framework. It's core app.


Updated the Cosmosdb  module

All Submissions:

[YES] Have you followed the guidelines in our Contributing document?
[YES] Have you added an explanation of what your changes do and why you'd like us to include them?
[YES] I have updated the documentation accordingly.
[YES] I have added tests to cover my changes.
[YES] All new and existing tests passed.
[YES] My code follows the code style of this project.
[YES] I ran lint checks locally prior to submission.
[YES] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

What is the current behavior?

The Cosmosdb module is updated to include the networking aspect
separated the test variables in a separate file
An issue is identified in the integration test, the Issue is created and those lines of code are commented out for the time being.
Read me is updated

Does this introduce a breaking change?

[NO]

Any relevant logs, error output, etc?

(If it’s long, please paste to https://ghostbin.com/ and insert the link here.)
Other information

#403


tweak to Webmin links

I didn't realize these links already existed in the UI when Webmin is enabled.  This PR just tweaks the look a little bit & adds an external-link icon:

Relates to #1351

Merged these changes into the open #1354  PR as they depend on each other anyhow.


Possible PR

I have a research paper on adding color to scene reconstruction that just got rejected from openMVG/awesome_3DReconstruction_list. Would mine qualify for this repository? It may be a bit off-topic, so I'm not making a PR yet.

Thanks for asking. I don't think it's related, so I am closing this issue. If you have anything that's related to multi-object tracker, feel free to open an issue or PR :)

Wow. This was made a LOOOONG time ago. So much has changed since then. I might have closed this PR myself if I had known about it!


feat(config): pass querystring config to qs module

Allow configuring of qs options via form.querystring parameter
Fixes #7

Coverage increased (+0.1%) to 79.947% when pulling 53853f396336ae7c5bab9ed8839b0072f4bc94b4 on webdevian:patch-1 into a4443553e90bcf3e3719db761e5e2e76406ce05e on adonisjs:master.

Coverage increased (+0.1%) to 79.947% when pulling 53853f396336ae7c5bab9ed8839b0072f4bc94b4 on webdevian:patch-1 into a4443553e90bcf3e3719db761e5e2e76406ce05e on adonisjs:master.

Appveyor break seems unrelated to this change

What do you think @thetutlage?

Thanks. Somehow I was not watching this repo, so missed it completely


Using version 4 completely with SCSS (all mixins)

I'm fiddling with Bootstrap 4 and I really like how you can apply different components via @include make-row(); etc. But currently I don't know how to apply e. g. forms or headline styles.
I wonder: will it be possible to apply forms, headline styles etc. just like this? (Let's say @include form-group(); and @include h1();)
Is it just not there because version 4 is still alpha? Or is it supposed to work in a different way? Or not at all?
And if it works the way I imagine, will the docs include a SCSS version of all components or a general documentation "how to work with SCSS"?

@mdo has stated in previous issues that providing mixins for everything is a non-goal. (I'm sad about that, but it admittedly would require adding a lot of trivial boilerplate.) We do provide mixins for many components though.

will the docs include a SCSS version of all components

We aim to document relevant Sass mixins/variables/etc. when applicable, e.g. http://v4-alpha.getbootstrap.com/layout/grid/#sass-mixins . We're likely missing a lot of such docs at the moment, partly because we're still in Alpha and thus our Sass is still in flux.

or a general documentation "how to work with SCSS"?

Giving some general tips on using Bootstrap via Sass is on our to-do list: http://v4-alpha.getbootstrap.com/getting-started/best-practices/

Okay, thanks a lot. It's fine if there aren't mixins for simple things like buttons. The grid mixins are a great help already.
Actually I've just noticed I can use @extend to apply styles from Bootstrap classes to other classes or elements. This does exactly what I was looking for.
Just an example to help others who want to do the same:
button {
    @extend .btn;
    @extend .btn-primary;
}


initial port of code review policy from google docs

This is just a port of the google docs version

@dimagi/team-commcare-hq @dimagi/commcare-mobile proposing that we keep this here. cc @biyeun

:+1:


[UI/UX][HIGH PRI] Connect to pulling infected info from backend

When a user indicates they have COVID symptoms, we can pull from the server.
This is done in this repo, need to wire to the UX wireframe

Pulling from new firebase


nixos/oci-containers: Remove dependency on system.path

Motivation for this change
Every oci-containers service depends on system.path which means it will be needlessly restarted every time a package is added/removed from the system path. To fix this I just launch the container backend through bash. Since the backend is already configured through the service PATH, it will automatically pick the correct one.
I also noticed that we weren't taking into account the configured docker.package when using the docker backend, so I also fixed that.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg test oci-containers

@GrahamcOfBorg test oci-containers

@GrahamcOfBorg test oci-containers

@GrahamcOfBorg test oci-containers

Thank you!

Thank you!


Color Scheme

Ensure that the whole site has a proper color scheme

Revised the CSS to ensure the entire site has similar colors and nothing too far from each other


UB-1503: epic1 : Fix ubiqutiy flex Mount API to fetch the right Scale PV mountpoint

This PR is needed  for getting mounpoints based on respective backends during mount operation.

This change is

controller/controller_test.go, line 505 at r2 (raw file):
Previously, shay-berman wrote…
I am expecting to see more UT to cover the new things you added to controller.go
because it looks like you have lack of UT here.

Yes will add more UT. Kindly let me know If we can finalise above code changes so that I can start on writing UT.

controller/controller_test.go, line 505 at r2 (raw file):
Previously, yadaven (Yadavendra Yadav) wrote…
Yes will add more UT. Kindly let me know If we can finalise above code changes so that I can start on writing UT.

Added UT in commit b5ef6fea93782fd1089021f4d512a0445a11b07a

controller/errors.go, line 137 at r2 (raw file):
Previously, yadaven (Yadavendra Yadav) wrote…
ok will make this change

Modified const/type for Missing mountpoint error to contain SpectrumScale. Changes have been added as part of commit b5ef6fea93782fd1089021f4d512a0445a11b07a

controller/controller.go, line 530 at r2 (raw file):
Previously, shay-berman wrote…
ok good, so just clarify - you removed the mountpoint from the DB and instead everytime getVolumeConfig called it automatically get the mountpoint from the Scale system?   (BTW i didn't see any PR that propose this fix, so please clarify it)
in addition, please make sure that you don't need to make it as 2 commands - volumemountpoint, ok = volumeConfig["mountpoint"]..... and then to make it a string.  just make sure its ok.   and add UT that check case that mountpoint key not exist.

While getting VolumeConfig we always got mountpoint from Scale system. This code is already present and thus no PR for that ( we can refer getVolumeMountPoint in local/spectrumscale/spectrumscale.go.) .
Currently volumeConfig is map[string]interface{}. Since value for key is interface{}. we need to typecast it to string.

controller/controller.go, line 222 at r17 (raw file):
Previously, shay-berman wrote…
BTW
please check that the detach\attach or mount\unmount doesn't use this doIsAttached - because if so then maybe you should skip calling it if its scale backend.
please check and let me know if it could lead you to an issue.
thanks

doIsAttached is called from below functions

doDetach

IsAttached

For IsAttached we return “Not Supported”
doDetach is called from Detach & doLegacyDetach.
We return “Not Supported” for Detach and we do not call doLegacyDetach during unmount.

glide.yaml, line 113 at r17 (raw file):
Previously, shay-berman wrote…
just make sure its the latest ubiqutiy commit in dev.
I think its now 422904c4c777d359341f22687ab6e3c9b459c470

Sure we will make sure to use latest ubiquity commit.

cmd/provisioner/main/main.go, line 46 at r17 (raw file):
Previously, shay-berman wrote…
why this code is in this PR?
isn't it a code that already exist in dev?  (please make sure you will not have collision)

This is due to rebase to dev

controller/controller.go, line 208 at r17 (raw file):
Previously, shay-berman wrote…
You forgot to handle idempotancy here as mentioned in the requirement: "add GetVolume if scale then not support (if PV not exist then just return error)"
So you need to check if getvolume return error because PV not exist in ubiqutiy-DB, and if so(PV not in ubiqutiy-db) then you should continue OK with idempotent warning (but if getvolume failed due to other error then you should fail with error). Exactly the same as -> https://github.com/IBM/ubiquity-k8s/blob/master/controller/controller.go#L457
Also please add here a comment with # TODO, later we should consider to get the volume backend by using the IsAttachRequest.opts instead of calling to ubiqutiy server.

This has been taken care in latest commit 30810a7

controller/controller.go, line 257 at r17 (raw file):
Previously, shay-berman wrote…
wait you have here the same bug as mentioned above.
you need to check if the getvolume error is specifically mentioned that PV not exist, and only then you should return success with warnning idempotancy message.
please fix, it should be the same as https://github.com/IBM/ubiquity-k8s/blob/master/controller/controller.go#L457

This has been taken care in latest commit 30810a7

controller/controller.go, line 544 at r17 (raw file):
Previously, shay-berman wrote…
I prefer more explicit here. So it will be clear that its different for scale vs scbe and also check if its other backend (if so then fail with not supported backend.)
something like this:
if (volume.Backend == resources.SpectrumScale) {
            # Scale has no detach process after unmount
	return c.successFlexVolumeResponse("")
    } else if (volume.Backend == resources.SCBE) {
    // Do legacy detach (means trigger detach as part of the umount from the k8s node)
	    if err := c.doLegacyDetach(unmountRequest); err != nil {
  	return c.failureFlexVolumeResponse(err, "")
    }

return c.successFlexVolumeResponse("")
} else {
print some error like backend not supported (I think there is already error like that you can leverage -> see PvBackendNotSupportedError
)
}
In addition, I would expect to see UT for this scale if\else

This has been taken care in 30810a7. We have not added UT for this because if backend is not valid getMounterForBackend will fail and control will not reach till this step.

controller/controller.go, line 977 at r17 (raw file):
Previously, shay-berman wrote…
what is this code?
how its related to PR detail?  it looks like olga code that is already in DEV.
please check it out

This is due to rebase to dev

controller/controller.go, line 989 at r17 (raw file):
Previously, shay-berman wrote…
also this one?

this is die to rebase to dev

controller/controller.go, line 1022 at r17 (raw file):
Previously, shay-berman wrote…
maybe its reviewable issue. it look like olga code and i can see it here as added into the PR.   So i think its ok.

This is due to rebase to dev

controller/controller_test.go, line 1031 at r17 (raw file):
Previously, shay-berman wrote…
need to fix it
it should succeed only if getvolume return error that the PV not in ubiqutiy DB, else it should fail. as mentioned above. please fix UT according.

We have fixed this UT in latest commit 30810a7

controller/controller_test.go, line 1068 at r17 (raw file):
Previously, shay-berman wrote…
only if PV not exist in DB error, else it should fail.

We have fixed this UT in latest commit 30810a7

CI sanity and extended pass OK.
@deeghuge  go ahead and merge it to dev (in "squash and merge" mode)


Code adaptation for TF 2.0

Code adaptation for TF 2.0

Hi,
thanks for the adjustment for TF2.0 . The only down side i see here, is that it wont work with TF versions below 1.13. Since i was already planning to release the pyro-nn Version for TF1.14 and TF 2.0 the pullrequest fits perfectly, but i would postpone the merge until the realse of the Pyro-nn comatible version.
best,
Christopher

Hi,
pyronn and the layers are now reworked to be compatible with tensorflow 2.0 and eager execution. All examples are adapted to the new tf2 style. Thus i close this pullrequest.
Thanks for your input.


Switch base16-schemes to new schemes repo

Describe the bug
The base16-schemes package uses the tinted-theming/base16-schemes repository, which has now been archived by the owner in favor of tinted-theming/schemes
Steps To Reproduce
Steps to reproduce the behavior:

Search for the base16-schemes package on search.nixos.org
Either click on the homepage link or find the repo from the source
Find out that it's archived

Expected behavior
The package should use the new repo to include more schemes that aren't available in the old repo.
Screenshots
The README of the old repo:

Notify maintainers
@DamienCassou

Add a :+1: reaction to issues you find important.

@fabiandrinksmilk Can you please test https://github.com/NixOS/nixpkgs/pull/286272?


adding to the readme

Coverage remained the same at 60.0% when pulling b8b21c5582bc068587a8317e33254bd1c64ff766 on dxwu:readmeToolsUpdate into df07c6e2117086f5f2e6fe51f1e195dd5cc15652 on TryStatsN:master.


Reduce pageSize when downloading to 10

Overview
At a pageSize of 50 with detail=true we are seeing response times near 30s, sometimes timing out.
In some brief testing reducing the pageSize to 10 results in requests that take 3 to 6 seconds. Setting a pageSize of 20 doubles the response time but interestingly setting a pageSize of 5 doe not reduce the request time. There appears to be a performance floor.
Demo

Testing Instructions

Open the network tab
Download a CSV and verify that the data requests have a page size of 10

Checklist

[ ] fixup! commits have been squashed
[ ] CI passes after rebase
[X] CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines

@TaiWilkin reviewed this with me on a screenshare so I will merge this.


Generate bindings for custom uxml components

Unity supports re-use of uxml components in other uxml components.  Rosalina doesn't generate bindings for these custom uxml components.
Pseudo-ish code for an idea about what to generate:
// CustomComponent.g.cs
public partial class CustomComponent
{
    private VisualElement _root;

    public CustomComponent(VisualElement root)
    {
        _root = root;
    }

    public Button MyButton { get; private set; }

    public void Initialize()
    {
        MyButton = (Button)_root?.Q("MyButton");
    }
}

...

// MainUIDocument.g.cs
public partial class MainUIDocument
{
    [SerializeField]
    private UIDocument _document;

    public CustomComponent MyCustomComponent { get; private set; }

    public VisualElement Root
    {
        get
        {
            return _document?.rootVisualElement;
        }
    }

    public void InitializeDocument()
    {
        MyCustomComponent = new CustomComponent((VisualElement)Root?.Q("MyCustomComponent"));
    }
}

In general, I have uxml components that will never be a top-level UI element.  They will always be composed by something else, so their [SerializeField] UIDocument _document field is unnecessary.

At the moment, Rosalina supports only native types:
https://github.com/Eastrall/Rosalina/blob/9ed4c726ec0badb3a31da9d53bc8c2b65bd88ac8/Editor/Scripts/Generator/UIPropertyTypes.cs#L7-L30
I believe this is a good idea to add support of custom components, let's plan this for next releases. Thanks for the suggestion!

I've been thinking how I could implement this, and I suggest the following code generation process:

Note: this is non-dependent wether the document is a "custom component" or any other component.
If I wanted to specify the component type within Rosalina, we'll need to wait for version 2023.X because Unity team is working on a way to extend the UI Builder inspector.

We could then imagine, a dropdown where you can select your "component" type (document, custom component, editor extension, ...)

// Menu.g.cs
public partial class Menu
{
    [SerializeField]
    private UIDocument _document;

    private VisualElement _rootElement;

    public Label Title { get; private set; }

    public MenuItems Items { get; private set; }

    public VisualElement Root => _rootElement ??= _document?.rootVisualElement;

    public InteractionMenu()
    {
    }

    public InteractionMenu(VisualElement rootElement)
    {
        _rootElement = rootElement;
    }

    public void InitializeDocument()
    {
        Title = (Label)Root?.Q("Title");
        MenuItemsContainer = new MenuItems((VisualElement)Root?.Q("Items"));
    }
}

// MenuItems.g.cs
public partial class InteractionMenuItem
{
    [SerializeField]
    private UIDocument _document; // Not used, but keeping it for now.

    private VisualElement _rootElement;

    public Label Title { get; private set; }

    public VisualElement MenuItemContainer { get; private set; }

    // Here _rootElement will be used since this instance will be initialized with the `VisualElement` constructor.
    public VisualElement Root => _rootElement ??= _document?.rootVisualElement;

    public InteractionMenuItem()
    {
    }

    public InteractionMenuItem(VisualElement root)
    {
        _rootElement = root;
    }

    public void InitializeDocument()
    {
        Title = (Label)Root?.Q("Title");
        MenuItemContainer = (VisualElement)Root?.Q("MenuItemContainer");
    }
}

Be careful though, the MenuItems script cannot be a MonoBehavior, otherwise, Unity will show you a warning that you should not create an instance of a MonoBehavior using the new keyword.
Please let me know your thoughts about this implementation.

The empty constructors pose a chance to break some code: Defining empty constructors means the other sides of partial implementations can't define them.  Anyone currently defining the empty constructor in their app-level code will break.
I'm not entirely sure how people use Rosalina.  If people bake a ton of logic in their partial implementations, then they'd probably want to write some code in the default constructor.  If people use it as a pure binding layer, then they most likely wouldn't care to lose access to the default constructor.
The other thing I see, which I missed as well, is invoking InitializeDocument()  on InteractionMenuItem's.  You'd probably want to call that in the InitializeDocument() of InteractionMenu.cs

In the implementation I posted, the code for the composed component was purpose built for it being composed, which lets the default constructors go undefined.  I think if you try to make it generalized for both use cases (top-level vs. composed), it's going to be a game of compromise.
I'm not familiar with how the auto-generation code works, but maybe you can generate different versions of the file based on it's usage.  In your example above, when the autogeneration code gets to Menu.uxml, it'll see that it contains a MenuItems template.  Instead of generating the default MenuItems.g.cs, you could generate a MenuItems.Component.g.cs that only has a single constructor expecting a VisualElement.  And when the code generation gets to MenuItems.uxml, it'll generate the default MenuItems.g.cs, which can be used as a normal top-level ui component.
// Menu.g.cs (Top Level Component)
public partial class Menu
{
    [SerializeField]
    private UIDocument _document;

    public Label Title { get; private set; }

    public MenuItemsComponentItems { get; private set; }

    public VisualElement Root
    {
        get
        {
            return _document?.rootVisualElement;
        }
    }

    public void InitializeDocument()
    {
        Title = (Label)Root?.Q("Title");
        Items= new MenuItemsComponent(Root?.Q("Items"));
        Items.InitializeDocument();
    }
}

// MenuItems.Component.g.cs (Composed Component)
public partial class MenuItemsComponent
{
   private VisualElement _root;

    public InteractionMenuItemComponent(VisualElement root)
    {
        _root = root;
    }

    public Label Title { get; private set; }

    public VisualElement MenuItemContainer { get; private set; }

    public void InitializeDocument()
    {
        Title = (Label)_root?.Q("Title");
        MenuItemContainer = (VisualElement)_root?.Q("MenuItemContainer");
    }
}

// MenuItems.g.cs (Top Level Component)
public partial class MenuItems
{
    [SerializeField]
    private UIDocument _document; // Not used, but keeping it for now.

    public Label Title { get; private set; }

    public VisualElement MenuItemContainer { get; private set; }

    public VisualElement Root
    {
        get
        {
            return _document?.rootVisualElement;
        }
    }

    public void InitializeDocument()
    {
        Title = (Label)Root?.Q("Title");
        MenuItemContainer = (VisualElement)Root?.Q("MenuItemContainer");
    }
}

You are correct, the default constructor generation is a mistake. I thought that without a default constructor, Unity wouldn't be able to instanciate the Menu document as a component on a game object. Looks like I was wrong, since I removed it and it still worked.

I'm not familiar with how the auto-generation code works, but maybe you can generate different versions of the file based on it's usage. In your example above, when the autogeneration code gets to Menu.uxml, it'll see that it contains a MenuItems template. Instead of generating the default MenuItems.g.cs, you could generate a MenuItems.Component.g.cs that only has a single constructor expecting a VisualElement. And when the code generation gets to MenuItems.uxml, it'll generate the default MenuItems.g.cs, which can be used as a normal top-level ui component.

The code generation mechanism is triggered by uxml file. Which means that every uxml file is treated one by one. Thus looking for what uxml is a component wouldn't work in that case.
I tried to mix up both code generation processes because I currently don't have any other way to identify what is a component and is not.
But, once the Unity team finishes the UI Builder extension mechanism, I would be able to create a field that would allow the developer to specify what "kind" of document it is. For example, a document, a component, an extension, etc...
Another approach would be to have something more "manual" which could be integrated along with the suggestion made on issue #26 : Manual generation
By doing a right-click on the UXML file, you could :

Activate / Deactivate the generation process
Choose how to generate the UXML (document or component)

All this would be stored in the RosalinaSettings and the code generator would generate the correct code without having a UIDocument field for custom component.

Solved in #27, see release : https://github.com/Eastrall/Rosalina/releases/tag/v4.0.0


The following resources (authamplifyIdentityBrokerAuth) failed to deploy when creating a new env.

I created the stack with the dev env without any problems, then I wanted to create another prod environment but I am encountering a problem.
1-I created successfully the environment via CLI:
$ amplify env add prod
Deployment state saved successfully.
✔ Initialized provider successfully.
✅ Initialized your environment successfully.
✅ Your project has been successfully initialized and connected to the cloud!
2-I then ran the command  $ amplify push but I get an error that I cannot decipher:
**🛑 The following resources failed to deploy:**
Resource Name: authamplifyIdentityBrokerAuth (AWS::CloudFormation::Stack)
Event Type: create
Reason: Parameters: [functionamplifyIdentityBrokerVerifyAuthChallengeName, functionamplifyIdentityBrokerCreateAuthChallengeName, functionamplifyIdentityBrokerCustomMessageArn, functionamplifyIdentityBrokerCustomMessageName, functionamplifyIdentityBrokerDefineAuthChallengeName, functionamplifyIdentityBrokerDefineAuthChallengeArn, functionamplifyIdentityBrokerVerifyAuthChallengeArn, functionamplifyIdentityBrokerCreateAuthChallengeArn] do not exist in the template

**🛑 Resource is not in the state stackUpdateComplete**
Name: authamplifyIdentityBrokerAuth (AWS::CloudFormation::Stack), Event Type: create, Reason: Parameters: [functionamplifyIdentityBrokerVerifyAuthChallengeName, functionamplifyIdentityBrokerCreateAuthChallengeName, functionamplifyIdentityBrokerCustomMessageArn, functionamplifyIdentityBrokerCustomMessageName, functionamplifyIdentityBrokerDefineAuthChallengeName, functionamplifyIdentityBrokerDefineAuthChallengeArn, functionamplifyIdentityBrokerVerifyAuthChallengeArn, functionamplifyIdentityBrokerCreateAuthChallengeArn] do not exist in the template, IsCustomResource: false

My problem was related to the amplify cli version.
I fixed the problem with these commands:

In my dev branch I restore all files: amplify env pull --restore
Then I downgrade the amply cli to v11: npm install -g<EMAIL_ADDRESS>Then I created the prod env: amplify env add prod
Finally I push the stack online: amplify env push --force


"Invalid Exclude" warnings when added to Xcode 13 Beta

I'm getting the following warnings when adding this package to a project in Xcode Beta 13:
Invalid Exclude '.../SourcePackages/checkouts/Unrar.swift/Sources/Cunrar/arcmem.hpp': File not found.
Invalid Exclude '.../SourcePackages/checkouts/Unrar.swift/Sources/Cunrar/arcmem.cpp': File not found.

My understanding is that it's because "arcmem.cpp" and "arcmem.hpp" are listed in the exclude section for the "Cunrar" target in Package.swift, but those files are not in the package.

My understanding is that it's because "arcmem.cpp" and "arcmem.hpp" are listed in the exclude section for the "Cunrar" target in Package.swift, but those files are not in the package.

Thanks for your report! I released v0.3.7.


Fixed error with query_id printing

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en
For changelog. Remove if this is non-significant change.
Category (leave one):

Bug Fix

Short description (up to few sentences):
Query id was not printed in logs for some threads.

TODO: Untangle attachQueryContext and attachQuery. These methods are non-obvious.


Specify version of open-next in default build command

I think it would be nice to have the version of open-next pegged to the latest known working version. Currently the build command is const buildCommand = this.props.buildCommand ?? 'npx --yes open-next build.
I'd propose specifying the version of open-next so that it's at least a known working version. That would have saved at least a number of people from running into issues with the current release as it's not working with v2.x.
For whatever it's worth, this is what SST NextjSite does:
...
buildCommand: "npx --yes<EMAIL_ADDRESS>build",
...

Perhaps this isn't worth the trade off in terms of having to keep up with open-next releases, but to me it would be worth the trade off.

As long as users can override it, that sounds good

@kevin-mitchell, can you only peg major version so minor and patch fixes are updated?

I opened a small PR just to peg the current version of the project to open-next@1 so something working can be deployed for anybody new to the project.

Closed in #127


What non-standard libraries are used in the original ElMD implementation?

Curious what is the motivation for this repo.

The original implementation on pypi was written in pure python which was about 5-10x slower than using pot (think the numba branch is likely comparable in speed but had issues getting it to work).
pot is what I mean by standard library

I also wanted to test using the elemd as a kernel function so wanted a signature f(A,B) rather than g(A).h(A) .

Gotcha. I also needed some speedup from the original implementation. I ended up implementing a CUDA/Numba version of Earth Mover's Distance and integrating that into a forked version of ElM2D. I like how simple the code is in EleMD - pretty straightforward. I've enjoyed looking through your various repos!


Can't delete volume after created

Version - Master 12/2
Steps:

Create a mysql container (make sure volume is created and listed in Host details)
Delete/Purge container
Go to Storage tab in Host details page
Try to remove volume

Results:
You can't remove, the drop-down menu only has View in API and no delete
Expected:
Should be able to remove the volume.

I believe this is a duplicate of https://github.com/rancher/rancher/issues/2796


Remove problematic chars from asset filenames

Issue: AP-3864
What Changed

The storybook server seems to have a problem serving files with percent signs. This removes those.
How to test

See the working asset test in the Chromatic E2E build

Change Type

[ ] maintenance
[ ] documentation
[x] patch
[ ] minor
[ ] major

📦 Published PR as canary version: 0.0.45--canary.44.271d657.0

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

:rocket: PR was released in v0.0.46 :rocket:


Calendar widget

Is calendar widget something that could be in the library at some point ? I have a magento customer that ask it. I am not sure it make sense for magento but for wordpress it will probably be needed anyway.

This is something that can be useful, for sure. I'm not sure I'd like this as part of the core widgets (a calendar library is a project by itself).
Maybe we should just provide the default widget that will work with HTML5 date elements, and show a few examples of plugging external calendar libraries (jQuery, etc).

Yes html5 date element is indeed a good idea.

Actually I think a widget able to do numerical refinement and manipulate/display date from timestamp should be in the core widgets to address all the media/news use cases.
If we talk about predefined range, it also make sens to propose a manual range where the calendar seems to be the right UI elements.
This could be the really mandatory widget:

the html5 calendar is really poor and don't really allow to select range in a fun or intuitive way.

This can be done in an "external" widget (community)

I am currently writing an API proposal, I am ok to write it afterwards. I will need that anyway


registers an AMD module if possible

Sets it as an AMD module with no deps if required, apart from setting the global object.
This is a small fix. It tries not to change too much how things work in terms of modules or the build process. I guess bigger changes could be made, if the project wants to take that route.
cheers! - fixes #691

Looks lovely. I'm curious, have you had a chance to test this on IE? I figure that typeof define test should be fine everywhere, but who knows with IE8/9.
With the addition of this feature, we should mention it in the docs. I'm thinking somewhere on this first page:
http://openseadragon.github.io/docs/
Any thoughts on how best to mention it? If you'd like to make a patch for that as well, it's over in:
https://github.com/openseadragon/site-build/blob/master/doc-home.md

@Emigre ping! :)

Hi @iangilman ! Sorry for the delay! I haven't forgotten abut it: I've been travelling for the last week and I've just returned today. I have a removal this weekend but I'll come back to this on Sunday or Monday.

I've just tested it in IE8 in Windows and it works fine. I've mentioned it very briefly in the home page as you suggested - that seems enough to me, but check it to see it you like it.

@Emigre, awesome, thank you!


'octmul.h' file not found

Can't compile for target ARM since octmul.h referenced from oblas_neon.h is missing.

octmul.h was replaced with new tables in octmul_hilo.h, i updated the oblas_neon.h to reflect that change. please note though that oblas_neon is essentially oblas_classic as i have not implemented neon intrinsics yet.


Add shortcuts for view alignment

Context
Some applications like Magnet on MacOS allow users to align the views with keyboard. As a developer-oriented windowing system, it is better for zen-desktop to have such a functionality by default.
Summary
Added keyboard shortcuts  to align the focused view & move focuses.
How to check behavior
Build zen-desktop & try the shortcuts added to the README.

Sorry for leaving the PR for a while, but now I think all the points are fixed.
(and sorry for the messy commit log due to something about rebase ...)


wpan: misc. cleanup and add mac

This fixes a few bugs I found with the last pr and adds basic mac functionality.
One thing I noticed was that mac was a bit different than ble: it does not use the linked list for events. In response, I modified the code structure a bit to mask the difference with ble so that there is a consistent api between the two.

@OueslatiGhaith just have a look at this to see if it looks reasonable.


Task/g361 network in http

Added optional para 'NetworkType' to Http to bypass Rest calls for retrieving networkType.
Tested on NamespaceHttp.spec.ts in e2e

Pull Request Test Coverage Report for Build 850

0 of 0   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 846:
0.0%

Covered Lines:
10

Relevant Lines:
10

💛  - Coveralls


Hl 1361 missing full application

Description :sparkles:
HL-1361
This time ensure, that the full application attachment is included in the pre fetch query, where it was not previously included.
Issues :bug:
Testing :alembic:
Screenshots :camera_flash:
Additional notes :spiral_notepad:

APPLICANT branch is deployed to platta: https://helsinkilisa-ui-pr3089.dev.hel.ninja :rocket::rocket::rocket:

API branch is deployed to platta: https://helsinkilisa-pr3089.api.dev.hel.ninja/healthz :rocket::rocket::rocket:

HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr3089.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://helsinkilisa-ui-handler-pr3089.dev.hel.ninja :laughing::tada::tada::tada:

TestCafe result is success for https://helsinkilisa-ui-pr3089.dev.hel.ninja :laughing::tada::tada::tada:


InsertColumn has a bug in DataFrame

If a DataFrame has 1 column of Length 0, and I insert a new column of Length > 0, InsertColumn should throw. It doesn't at the moment.

Hello @pgovind ,
Firstly, i'd like to thank you and the rest of your team for creating this data library. It was very useful and performed well on a finance project i have been working on.
Can i help contribute to this project and this issue?
I have recreated this issue and created a test for it as well.
The below proposed check fixes the issue as well as passes all the existing tests. Shall i create a PR for this?
       protected override void InsertItem(int columnIndex, DataFrameColumn column)
        {
            column = column ?? throw new ArgumentNullException(nameof(column));
            if (RowCount > 0 && column.Length != RowCount)
            {
                throw new ArgumentException(Strings.MismatchedColumnLengths, nameof(column));
            }

+           if(Count >= 1 && RowCount==0 && column.Length != RowCount)
+          {
+               throw new ArgumentException(Strings.MismatchedColumnLengths, nameof(column));
+           }

            if (_columnNameToIndexDictionary.ContainsKey(column.Name))
            {
                throw new ArgumentException(string.Format(Strings.DuplicateColumnName, column.Name), nameof(column));
            }
            RowCount = column.Length;
            _columnNames.Insert(columnIndex, column.Name);
            _columnNameToIndexDictionary[column.Name] = columnIndex;
            for (int i = columnIndex + 1; i < Count; i++)
            {
                _columnNameToIndexDictionary[_columnNames[i]]++;
            }
            base.InsertItem(columnIndex, column);
            ColumnsChanged?.Invoke();
        }

Can i help contribute to this project and this issue?

Yes!!! I think your fix looks good as proposed! I'd say go ahead and open a PR!!

Thanks pgovind, I will make the PR at some point today including the test

Fixed with https://github.com/dotnet/corefxlab/pull/2937


autoUpdate does not always work when floating element is portaled

Reproduction: https://codesandbox.io/s/adoring-lake-p7e0ps?file=/src/App.tsx
If the floating element is wrapped in a FloatingPortal, the autoUpdate feature does not seem to always work properly. In the reproduction, if you resize the browser viewport, the Tooltip part is not repositioned.
https://user-images.githubusercontent.com/1892091/162899673-d1904be2-0e28-45aa-ac22-e3e2f4829706.mp4
If I render the floating element conditionally, it seems to work (see https://codesandbox.io/s/affectionate-scott-kbyopd?file=/src/App.tsx) - but in my actual code that does not seem to make a difference.
I'm not sure if I'm missing anything, as the more complex Dropdown and similar examples from the docs are working.

It looks like you need to use refs.floating.current as a dependency in this case. The reason is the FloatingPortal generates 2 renders, but the floating element doesn't exist by the time the effect in your component is called.
The reason it works in the docs is because they use open && and pass that as a dependency, which happens after the FloatingPortal double render happens


New logos

Apps
StablePay
https://stablepay.io/

Apps
EthDNS
https://www.wealdtech.com/articles/using-ethdns-to-host-your-dns-domain/

Mobile Wallets
Rainbow
https://rainbow.me/

Apps
KnownOrigin
https://knownorigin.io/

Apps
Podcrypt
https://podcrypt.app/

Apps
Defi Watch
https://defiwatch.io/

App
Accointing
https://www.accointing.com/

Apps
Origin Protocol
https://www.originprotocol.com/en

Hey @brantlymillegan, could I please bother you to have a look at https://github.com/ensdomains/ens.domains/issues/49

Apps
Now.eth
https://now.substack.com/p/introducing-noweth

Added all except now.eth which I will handle with separate PR.
https://github.com/ensdomains/ens.domains/pull/55


[performance] Array Operations are extremely slow

We are using java-based crypto libraries and they works extremely slow in j2objc. 90% of time in encryption is reading and writing to arrays and such simple operation takes a lot of time unlike plain objc implementation. Especially IOSLongArray_Get is very slow.

Crypto code has to do lots of number calculating and array accesses, as otherwise the plaintext would be trivial to decode. My hunch is that range-checking is slowing your code down, since IOSLongArray_Get() is an inlined function that just checks the range and then returns the indexed element using a straight C array dereference. You can turn off index-checking by defining J2OBJC_DISABLE_ARRAY_BOUND_CHECKS, though doing so for the whole j2objc runtime is obviously risky.
I haven't tried this, but since both IOSLongArray_Get() and IOSArray_checkIndex() are inlined functions, it should be possible to disable range checking in just one Java source by adding this near the top of the file (such as after the imports and before the class declaration):
/*-[
#define J2OBJC_DISABLE_ARRAY_BOUND_CHECKS 1
]-*/

This JNI block will be added to the generated .m file for the Java source, so it won't affect classes that #include the .h for this Java file.
Another area to investigate (once range-checking is disabled) is whether performance is an issue on all platforms, or just 32-bit or 64-bit ones. Since j2objc generates arch-independent code, we can't help with platform-specific issues, but if that's an issue it's likely that there are related discussions on StackOverflow for ways to speed up C code on specific iOS platforms.
If you want us to investigate this further, please reopen this issue with a micro-benchmark that demonstrates the problem.

Thank you @tomball J2OBJC_DISABLE_ARRAY_BOUND_CHECKS speeds up my encryption ~100x times!

That's great news, I'm glad you're unblocked.


Add created timestamps to prompb

Adds created timestamps to prompb.
Adding this in a separate PR because #12733 is getting too big

cc @bwplotka @macxamin @TheSpiritXIII

NOTE: As Björn mentioned this is just copy of changes in client_model


inner columns shorter than outer columns

Why when I make a column layout are the inner columns shorter than the outer columns? it looks strange.

can you post a screenshot?

Very very easy to reproduce. Just make some columns. The outer ones have less padding than the inner ones.
--
Mark Amber
(815) 713 - 8116
On February 16, 2016 at 2:44:26 PM, Peter Ivanov<EMAIL_ADDRESS>wrote:
can you post a screenshot?
—
Reply to this email directly or view it on GitHub.

Haven't tried to reproduce this but i think this doesn't have anything to do with the columns themselves but with the (row-)container. The (row-)container also has a padding, so when the padding of the columns and row is set to 15px you'll always have 30px in total, even on the outer sides.

Indeed, that is the culprit. default.css:257 RE the first child padding-left:0; I wonder what changing that to 10 will break elsewhere…
--
Mark Amber
(815) 713 - 8116
On February 17, 2016 at 5:10:54 PM, Thomas Vansteenkiste<EMAIL_ADDRESS>wrote:
Haven't tried to reproduce this but i think this doesn't have anything to do with the columns themselves but with the (row-)container. The (row-)container also has a padding, so when the padding of the columns and row is set to 15px you'll always have 30px in total, even on the outer sides.
—
Reply to this email directly or view it on GitHub.

I guess it relies on the container of the columns to have a padding of 10, so just put padding there?
(mw-row probably)


Getting stderr of failed task(s) in a common on-error condition task in mistral workflow.

Hi,
I'm working with a mistral workflow and on any on-error condition I'm calling a task calling_on_error in the workflow on any on-error and want to pass stderr  of failed task in the calling_on_error.
Publishing is not working as it only works when a task succeeded and not in case of failure.
Is there any way I can have stderr of previous task to calling_on_error  task as I want to pass this another subsequent task for say for sending email notification.
<% task(Name_of_Task).result.stderr %>
will not help as my Name_of_Task will keep changing.
Thanks,
Parveen Sharma

https://stackstorm.slack.com/archives/community/p1492117899815875

+1 this is very helpful for debugging workflows.


add example:docker moderate-mysql-init-sql-data

Thanks. Please run terraform fmt for future commits.


[_]: feat/add public sharing GET meta and POST sharings

Codecov Report

Merging #173 (e051aec) into master (3299091) will decrease coverage by 0.07%.
The diff coverage is 50.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##           master     #173      +/-   ##
==========================================
- Coverage   66.56%   66.50%   -0.07%
==========================================
  Files          39       39
  Lines        1032     1036       +4
  Branches      144      144
==========================================
+ Hits          687      689       +2
- Misses        307      309       +2
  Partials       38       38

Files Changed
Coverage Δ

src/drive/share/index.ts
48.17% <50.00%> (+0.05%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Fix bicubic interpolation in image.warp

The existing bicubic sampling produces artifacts. Fixes issue #103

Here is the new output of the test program (see #103 )

@colesbury FYI just pushed an update (see #105).


fix issue 3733 making result.get_counts() return also a list of dicts

Summary
Added an optional check that allows to return a list of dicts in case the method get_counts() of the class result.py is called with no argument (experiment=None).
In this way it is achieved the enhancement for example to execute:
plot_histogram(result.get_counts())
Details and comments
Correct version of the previous #3737 to fix #3733

Anybody saw the latest commits I did? Last week I added the test and all the other reviews to merge the enhancement. Should anyone take a look at it @jyu00 @1ucian0 @kdk ?


Cleaning code

@takutosato This addresses your comments on a recent PR, then does a few more things to clean M2, then binges on IntelliJ's automatic code cleanups.  You can skim the last several commits if you trust the IDE and the Travis tests.

Also, this is low priority -- much lower than the M2 integration tests PR.

Some notes on the commits:

got rid of M2 debug -- earlier we discussed keeping this because conditional breakpoints are slow in the IDE, but personally I would rather just write an one-off if statement and put a breakpoint inside rather permanently clutter the code.

inlined redundant variables -- replaced the pattern result = something; return result with simply return something in several places

remove unnecessary throws clauses -- deleted throws Exception from methods that do not in fact throw these exceptions.

remove redundant super -- no need to say super.method() when you can just say method().  Otherwise, what's the point of inheritance?

remove redundant this -- it's not Python.

replace fully qualified names with imports -- eg java.lang.Util.List --> List.


[WIP] Only enable diagnostics and fix settings change detection

Fixes #242.
#289 causes a few issues to do with parser-generated diagnostics, and there are more diagnostics I wasn't aware of coming from ErrorSink. This is a simplified PR that does the minimum to enable diagnostics (ignoring "Unspecified" at publish time) and fixes the setting change bug.
We'll have to revisit where diagnostics should be stored to access via code actions in a different PR. There are only two diagnostics that actually come out of the server which can be disabled with python.analysis.disabled for now.
I'll close #289 in favor of this one.

Test would be good. Like declare a variable, observer squiggly, change file, squiggly goes away, change back, it reappears. @AlexanderSher - do we have something like this for unresolved imports?

Okay, maybe not as simple as I thought. The test helper SendDidChangeConfiguration only accepts completion options, then reimplements part of DidChangeConfiguration, specifically the part that I'm trying to test... So I'll have to do some extra work to actually give the server a proper config change message to run the real code.

I've added a test which changes the settings and re-checks the diagnostics. The overall diagnostic testing code and settings editing code is really strange, though, and could probably be improved.
All of the "work" for changing settings is actually done in ServerBase at the RPC method level on the JToken. It'd be nice if all of that hand-written JToken decoding was done automatically so that the tests could actually call it...


redundant commit navigation in clean merge commits

The commit navigation on the bottom clean merge commits seems redundant.

source commit
But maybe the effort in excluding specific pages based on different criteria might not be worth it though.

In this case they’re only “redundant” because they’re so close together. I think the logic would be something like “display if there are more than 10 diff lines on screen”
but like you said maybe it’s not worth it.


Can't uv run --with private package index

> uv -V
uv 0.3.2 (c5440001c 2024-08-23)

We have a private package index:
> pip config list
global.extra-index-url='https://pypi.python.org/simple'
global.index-url='https:*****'

Private packages can be installed with uv without any issue:
> uv pip install privately-hosted-package
Resolved 38 packages in 3.54s
Installed 38 packages in 44ms

But trying to run a command using --with does not work:
> uv run --with privately-hosted-package echo "success"
  × No solution found when resolving `--with` dependencies:
  ╰─▶ Because privately-hosted-package was not found in the package registry and you require
      privately-hosted-package, we can conclude that your requirements are unsatisfiable.

Note that manually specifying the index urls seems to work fine:
> uv run --with privately-hosted-package --index-url 'https://*****' --extra-index-url 'https://pypi.python.org/simple' echo "success"
success

Hi! We don't read the pip configuration file (see #1404)

Hi! We don't read the pip configuration file

Interesting. Any idea why uv pip install worked then?

You can specify your index URL in a uv configuration file? https://docs.astral.sh/uv/configuration/files/

That could be a good option, thank you.

Is it possible you have the index URL set in a configuration file somewhere already?

Oh, sorry I missed that part. That's confusing. Were they run in the same directory?

Is the index URL in a uv configuration file in a [uv.pip] section instead of [uv]?

Is the index URL in a uv configuration file in a [uv.pip] section instead of [uv]?

This was the issue
Removing the table header from the uv.toml file lets me execute uv run --with privately-hosted-package echo "success"

Awesome! Documentation on that at https://docs.astral.sh/uv/configuration/files/#configuring-the-pip-interface


Vibration to signal snooze

It would be great if you could add the possibility to have the phone vibrate when I turn it around to activate snooze. Thus I would have a haptic response to know that snooze is activated and I could continue sleeping without looking at my display.

@Anomueller
I'm not against this enhancement.
Here's what could be implemented:

Let the user choose whether or not to activate vibrations for the 2 “Snooze” and “Dismiss” buttons in “Customize alarm display” settings.
For the “Snooze” button, create a double vibration and for the “Dismiss” button, create a single vibration.

Would you agree with this?


libStorage 0.3.3-rc3

This patch updates REX-Ray to work with libStorage 0.3.3-rc3 and its new volume attachment query/filter framework.

Current coverage is 8.36% (diff: 6.66%)

Merging #619 into release/0.6.1 will decrease coverage by 0.53%

@@           release/0.6.1      #619   diff @@
==============================================
  Files                 15        15
  Lines               2112      2115     +3
  Methods                0         0
  Messages               0         0
  Branches               0         0
==============================================
- Hits                 188       177    -11
- Misses              1919      1934    +15
+ Partials               5         4     -1

Powered by Codecov. Last update 469c8f6...010c874

Closing in favor or #622.


uniapp小程序支持吗

Clear and concise description of the problem
uniapp小程序，引用组件，不会在当前页面的.json中引用当前组件，这个有什么解决方案吗-
Suggested solution

Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

这个问题可能得去 uni-app 提 issue 了。找他们适配一下。

这个问题可能得去 uni-app 提 issue 了。找他们适配一下。

😂好吧

https://github.com/dcloudio/uni-app/issues/3748


remove ZBR_INFRA_HOST usage

ZBR_INFRA_HOST should be replaced everywhere by ZBR_HOSTNAME  variable

done.
during the testing please cover two scenarios:

regular setup from develop branch
-> all should work as usual. also verify that INFRA_HOST jenkins global variable has default port value like:
<IP_ADDRESS>:80
setup 1.0 version, perform basic acceptance and upgrade to the latest
-> all should work as uausl and INFRA_HOST jenkins global var updated using port as well


Add tests for Series arithmetic and comparison methods and fix Series.div

For me this auto generation looks too complicated.
Maybe organize each test like this:
def test_impl()
return [A + B,
A += B,
A.add(B)]
and then check each result from list?
Or create separate test for each case?


Support PlanetScale in Edge Client

Problem
Right now, the only way to use the Prisma Edge client (@prisma/client/edge) is by using the Data Proxy. I cannot use PlanetScale at the edge with Prisma at the moment.
Suggested solution
Support a PlanetScale mySQL connection string for the edge client
Alternatives
I am forced to use Kysely Planetscale dialect to generate a typesafe client at the edge. I'd like to use Prisma, but this limitation is a huge blocker.
Additional context
It'd be nice to support PlanetScale at the edge and have a typesafe way to access a DB without having to use Kysely or other query builders. Prisma DX is unparalleled for schema and pushing schema updates, but I am forced to drop the helpful client at the edge in favor of other query builders.

You can totally use a Planetscale connection string with Data Proxy, and our Edge Client.
But you can not use the serverless driver from PlanetScale - which is a different way to talk to their database. Is that what you mean? If so, then this request is already tracked in https://github.com/prisma/prisma/issues/15265 and this issue can be closed.

That issue describes what I need perfectly @janpio. I'd like to avoid having to use the Data Proxy if possible. So somehow landing support in the Prisma Edge Client to talk to the serverless driver directly would be ideal.
We can go ahead and close.


Match Prop Rotation Before Tool Activation

First off, awesome mod!
When placing prop fences with devui (fence networks don't seem to work), the rotation of the fence prop does not match the rotation of the prop before activating the tool.

This makes it hard to precisely place directional props as the manual rotation arrow buttons might not be precise enough over long distances to match edges

Confirmed!  Yeah, there's no fence mode currently, and I need to add one.
I'm assuming that what you really want is for it to be aligned with the line, and not just your original alignment?

Ideally yes

A fence mode has been added in version 1.0.3 - is this what you were after?

Awesome, yes that's what I was looking for


Support Event Tracing for Windows (ETW)

http://msdn.microsoft.com/en-us/library/windows/desktop/bb968803(v=vs.85).aspx
I think we should have at least one use case before  we implement this, so posepone for now.

Windows driver developers (including Windows kernel developers) tend to prefer Windows software trace preprocessor (WPP), which is backed by the ETW technology, over debug prints in production code, since it runs fast, does not leave hard-coded strings in compiled files (they are instead, stored in PDB files), and as straightforward to use as debug prints.
If DebugView++ implements a capability to view WPP logs in real-time, that would be attractive for such developers because the only tools that support it are: TraceView, which UI was written 15 years ago and pain to use, and TraceView Plus, which is not free, as far as I am aware of.
This will still be somewhat large work but at least you do not need any driver file, unlike support of the kernel-mode debug message capturing.

@janwilmans Here's an example: https://github.com/Microsoft/Windows-classic-samples/blob/master/Samples/Win7Samples/winbase/Eventing/WppConsumer/WppDumper.cpp (Didn't test..)

@harriv thanks! I will try this, at first glance this looks like the 'receiving part' right? the side I would need to do, which is perfect to have an example of, however, what about the sending side ? any example in c++ and/or  C#?

Hi @janwilmans
I also quickly coded the provider part with WPP and TraceLogging: etw.zip
Some instructions and brief explanations are there as comments. Those are expected ouputs from the sample programs.
WPP

TraceLogging (which, I care less)

wow thanks! I will have to examine this!


Error code -2010

Hey mate,
refering to the same issue of #73, I'm getting the same error now...
Tried with my old version 1.4, as well as with 1.6, ending up with error code -2010 (from binance), saying "Account has insufficient balance for requested action".
What's interesting is that this doesn't always happen.. Sometimes its working, sometimes it's not.. For me, it feels like the bot tries to place the sell order, before the buy order got executed, so it has insufficient balance to create the sell order.. Really weird somehow...
You have any idea or are familiar with this issue? Trying to fix as well... What's weird is that code didn't change on my side since the last pump i participated and there, everything worked like a charm.... Maybe a problem with the Binance API?

Okay, so it seems like I found a more or less working fix for me to try in a pump today...
I just set the amount to sell to coinOrderQty*0.98, which seems to do the trick for now...
I noticed, that the bot seems to somehow and sometimes use the same value of amount to sell, so my wallet showed something like 57.8 of the coin and the bot tried to place a sell order of 58.2, so of course, it didn't work, that's why that seems to be a temp fix for me to try today...
However, I noticed that there actually seems to be some ind of issue with that quantity in general, as in one test, I bought 38 QTUM, I could also see that number in my order history - however prinitng the coinOrderInfo variable gave me a qty of 29.... it's really weird and I don't know why this happens, maybe they hat some change on the api or sth?

"quotedCoin": "BTC",
"buyLimit": 1,
"percentOfWallet":100,
"manualQuoted": 0.00000000,
"profitMargin": 150,
"stopLoss": 0.90,
"endpoint": "api1",

With these values, I have an error corrected. BUT when I gave a buy order for any coin, the price of the coin is 1379 satoshi and the order value for buying is 1342 satoshi. does not get instant. Do you have the same problem?

Why is it giving a lower purchase order than the instant coin price? Do you have the same problem?

Why is it giving a lower purchase order than the instant coin price? Do you have the same problem?

This is happening because your buyLimit is 1. See the readme's descirption of the buyLimit parameter.

Okay, so it seems like I found a more or less working fix for me to try in a pump today...
I just set the amount to sell to coinOrderQty*0.98, which seems to do the trick for now...
I noticed, that the bot seems to somehow and sometimes use the same value of amount to sell, so my wallet showed something like 57.8 of the coin and the bot tried to place a sell order of 58.2, so of course, it didn't work, that's why that seems to be a temp fix for me to try today...
However, I noticed that there actually seems to be some ind of issue with that quantity in general, as in one test, I bought 38 QTUM, I could also see that number in my order history - however prinitng the coinOrderInfo variable gave me a qty of 29.... it's really weird and I don't know why this happens, maybe they hat some change on the api or sth?

This is weird I've encountered it a few times too. If I have some free time I'll try and do some rigorous testing into what is causing this error so I can fix it.

@fj317 Can we give a manual sale?

The bot should attempt to manually sell at market price if an error occurs already.

@fj317 I have solved this end as follows; the balance is first 50% and then it continues to operate 100%. does not repeat the error then.

So you set the percentOfWallet to 50?

YES is trading with 50% first. I can then use it 100%. the error then goes away.

So what should i do to solve the problem ? Set my "percentOfWallet" to 50? Or can i set it to 100?

Yeah, another missed chance on a nice pump....
Again, I got the error when trying to buy, saying "A BinanceAPI error has occurred. Code = -2010".., further down saying: "Account has insufficient balance for requested action"...
Getting kinda annoying right now :(

@fj317 Hi just some tips from a bot seller,
Firstly, python would auto round up when it send the amount to binance which means if amount of coin purchased are 0.1789 it will set sell order 0.179 which would cause insufficient balance
Secondly, if the user has no bnb for fees it would take the coin X purchased and minus some for fees e.g. purchased amount 0.1789 and after fees might be 0.177 and again it will trigger another insufficient balance
Thirdly, most common problem some coins/tokens are traded in whole number which means no decimal e.g. if I purchased 2 coin X and I don't have any bnb for fees it would deduct again from that coin and becomes 1.997 and if you try to set sell order it will throw lot size error and if you manually sell it, binance would convert it to 1 and that 0.997 you will have to go to your wallet settings and convert
Probably the best temporary fix for this error with no lot size problem would be if user has no bnb for fees then take (amount of coin purchase - lot size), because * 0.98 doesn't work for all coins some would throw lot size error, doing this at most just leaving like $0.3 behind but best would be to have some bnb for fees

Okay, let's just get some BNB then, shouldn't be a problem :)

Problem 1 might occur if it is left on default it will auto round up, if format it based on precision then that would not occur. Something like newValue1 = "{:0.0{}f}".format(newValue1, precision)

Firstly, python would auto round up when it send the amount to binance which means if amount of coin purchased are 0.1789 it will set sell order 0.179 which would cause insufficient balance

The rounding shouldn't be an issue since I created my own rounding function that rounds the coin to the correct number of decimal places depending upon how many decimal places Binance uses.

Secondly, if the user has no bnb for fees it would take the coin X purchased and minus some for fees e.g. purchased amount 0.1789 and after fees might be 0.177 and again it will trigger another insufficient balance

This is written into the readme already with good instructions.

Thirdly, most common problem some coins/tokens are traded in whole number which means no decimal e.g. if I purchased 2 coin X and I don't have any bnb for fees it would deduct again from that coin and becomes 1.997 and if you try to set sell order it will throw lot size error and if you manually sell it, binance would convert it to 1 and that 0.997 you will have to go to your wallet settings and convert

If users had BNB, like recommended, this wouldn't be an issue. Also I'll have to spend a lot of time implementing the logic when the alternative is getting some BNB which is very easy and quick.

i also have this issue with BNB available.
when doing a testrun it gives:
Investing amount for BTC: 0.001
Investing amount in USD: 59.12

Coin pair: sc
Buy order has been made!
Processing sell order.
A BinanceAPI error has occurred. Code = -2010

config:
    "quotedCoin": "BTC",
    "buyLimit": 1.15,
    "percentOfWallet": 10,
    "manualQuoted": 0.00100000,
    "profitMargin": 101,
    "stopLoss": 0.90,
    "endpoint": "default",

It's because your profit margin is so low I think. The program needs the profit margin to be higher than the current price of the coin otherwise it'll fail. Since your profit margin is 101 the program tries to sell the coin at the same price it bought it at basically and this results in an error since OCO orders have strict rules.

Ah, lol. Thought 101 meant something else ;-) will test that later. Tnx for the pointer.


Design and implement CSVelte\Writer family of classes

#72 So far I've done quite a bit with reading CSV files, but nothing with writing them. Need to design the interface first. Take a look at how other libraries are doing writing, how PHP itself does it, and then throw all of it away and write it in the most intuitive and user-friendly manner possible. Think through the most common use cases for generating CSV files and try to come up with an interface that will make them as painless as possible.
Things to keep in mind:

Writing CSV files is one of those things that people who don't even usually write code, try to do. It's one of the first things I ever wrote as far as programming goes. For this reason, try to write this thing in such a way that anybody can use it, but without sacrificing on good design. Try to encourage these people to learn proper coding techniques.
Eventually you may have filters/mutators for the data contained in a CSV file as well so be sure to keep that in mind.

Checklist:

[ ] Design Writer interface
[x] Design Writable interface
[ ] Need as easy a way s possible to convert a CSV file from one format to another. Think about how this would work. Eventually you may have filters/mutators for the data contained in a CSV file as well so be sure to keep that in mind.
[ ] Buffer write operations so that you can call things like setHeaderRow() and change flavor and all that jivey divey goodness at any time.
[ ] Need to write a test to ensure that when a file already exists, it is completely overwritten and that no existing data is left over in the file (if, for instance, you write less data than was originally in the file)
[ ] Need to add an option for appending a file rather than overwriting it
[ ] Need to write a test to ensure that when a file doesn't exist, it is created (and that if it cannot be created, an appropriate exception is thrown)

The majority of this will be added in v0.2, pushing to that version.


DisplaceEntityEvent.Teleport.TargetPlayer fired for natural movement.

@bloodmc
Latest SF master and I'm getting event triggers for whenever any movement occurs under Teleport.TargetPlayer. This should only happen under Move.

This is an issue with how the interface inheritance is working for this event in 1.8.9. To prevent plugins from breaking, this won't be addressed in API 4's lifetime. This has been resolved in bleeding/bleeding-1.10.


Provide a homeDir system variable for use in launch.json

VSCode Version: 1.2.1
OS Version:  OSX 10.10 and OSX 10.11

Steps to Reproduce:

Folow tutorials on creating .vscode/launch.json
Set paths that start at the Home Directory (eg: %USERPROFILE% on Windows, $HOME / ~ on Linux)

It would be nice to have a homeDir system variable to use in the Launch.json.  I have been using the ruby plugin and I have looked over a number of examples, issues and tutorials.  One thing that strikes me is that the launch.json looses some utility if it is not reusable.  Take for example the RSpec launch that I am working through now while using RVM on a Mac:
        {
            "name": "RSpec - all",
            "type": "Ruby",
            "request": "launch",
            "cwd": "${workspaceRoot}",
            "program": "${workspaceRoot}/bin/rspec",
            "useBundler": true,
            "pathToBundler": "/Users/<username>/.rvm/gems/ruby-2.3.0/wrappers/bundle",
            "showDebuggerOutput": true,
            "pathToRDebugIDE": "${workspaceRoot}/bin/rdebug-ide",
            "args": [
                "-I",
                "${workspaceRoot}"
            ]
        },

While this works:
"pathToBundler": "/Users/<username>/.rvm/gems/ruby-2.3.0/wrappers/bundle",
It would be much easier/shareable/reusable to have:
"pathToBundler": "${homeDir}/.rvm/gems/ruby-2.3.0/wrappers/bundle",
Similar to what is available for workspaceRoot.
I speculate the complexity here would come in from working for all OS's. (eg: %USERPROFILE% on Windows, $HOME / ~ on Linux).

It is possible to reference enviroment varialbes in your launch.json, for instance to access the PATH env variable use the following syntax ${env.PATH}
More details can be found here http://code.visualstudio.com/docs/editor/tasks#_variable-substitution
I suspect that you can acess the HOME via env variables


csl1883 + tests for master

Original issue here: https://github.com/input-output-hk/cardano-sl/pull/1919
The purpose of this PR is to create tests which are available in the master branch and which are missing in #1919 branch.
For more info see #1919
TODO

[x] write automated tests for /api/txs/payments/batch
- this will be variant of https://github.com/input-output-hk/cardano-sl/blob/master/wallet/test/Test/Pos/Wallet/Web/Methods/PaymentSpec.hs#L45
[x] remove "purescript": "^0.11.5", from package.json

@adinapoli-iohk got it.
Lets proceed as you said:

feel free to crack on with the tests, but my preference would be to merge just #1919 on cardano-sl-1.0, then cherry-pick the merge into master and then open a separate PR with just the tests against master

we will close this PR when tests are written.

Thanks @akegalj , appreciated as always!

@pva701 note that current batch payment spec is super set of newPayment spec. So it will cover both.

Also, I don't see an update of purescript api. dunno whether it should be done :man_shrugging:

@pva701

Also, I don't see an update of purescript api. dunno whether it should be done

it won't be needed. This is needed only for Binance exchange and they are integrating via direct http endpoints. Purescript api is not used by any exchange so far and Daedalus UI is also moving away from it. We will deprecate it really soon

@pva701 @adinapoli-iohk it should be ready for re-review.
I am happy with squashing these commits when merging them.
I have simplified json serilization from:
{
    from: "the_input_moneysource"
  , to:
      [ {address: "addr1", amount: 30}
      , {address: "addr2", amount: 40}
      , {address: "addr3", amount: 30}
      ]
  , inputSelectionPolicy: null
}

to:
{
    from: "the_input_moneysource"
  , to:
      { ["addr1", 30]
      ,  ["addr2", 40]
      ,  ["addr3", 30]
      ]
  , inputSelectionPolicy: null
}

After it builds on my machine I will test json serilization one more time and edit this comment to confirm (as I never really serialized json tuples - ie used FromJSON a, FromJSON b => FromJSON (a,b) instance)

Sorry to be a pain in the neck @akegalj , but I'm not very comfortable with this serialisation scheme:
{
    from: "the_input_moneysource"
  , to:
      { ["addr1", 30]
      ,  ["addr2", 40]
      ,  ["addr3", 30]
      ]
  , groupingPolicy: null
}

Why are we giving up lovely and self descriptive JSON objects in favour of nasty heterogeneous lists? 😉  What am I missing?

Why are we giving up lovely and self descriptive JSON objects in favour of nasty heterogeneous lists? 😉 What am I missing?
It looks simpler (in my mind TM) I am ok with list of objects though. Will revert back

We are almost there, @akegalj ! Would you address the conflicts so we can this merged? :)

@adinapoli-iohk This should be it - waiting for CI

@akegalj I have decided to squash these into one clean commit which is easy to revert, in case. Thanks Ante!


everything worked perfectly but when in game my champion AFK

like i said everything worked perfectly but when in game my champion AFK
my mouse is plug
sometimes cursor is moving but not click
when in lobby everything fine only problem in game

What version are you using?

lastest version 2.2.2

Same problem

i have a fix, if you running it using python , create a file start.bat in the same folder as main.pyw and then paste this code :
start.bat
@setlocal enableextensions @cd /d "%~dp0" py main.pyw pause
run this file start.bat as administrator,
if you are using the exe just run it as administrator.

@waaikanon does anything here solve your issue