Data Export option has vanished

The Data Export area illustrated in the screenshot no longer exists in the PPAC setting, and I'm unable to find the option anywhere else in the admin center:

Would be really great if the documentation could be updated to clearly indicate where this option has been moved to

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

ID: 88890037-f7de-6084-f506-dc917adc0508
Version Independent ID: 95103288-f976-3b83-d089-be6d315e563f
Content: Export data to Application Insights - Power Platform
Content Source: power-platform/admin/set-up-export-application-insights.md
Service: power-platform
Sub-service: admin
GitHub Login: @iwinoto-msft
Microsoft Alias: t-iwinoto

The screen shot above is from the PPAC solution - not the documentation.
The visibility of the user experience menu item is based on the authenticated users roles and/or licensing matrix.
Please verify that the user is a member of one of these roles: Power Platform admin, Dynamics 365 admin, or Microsoft 365 Global admin.
See https://learn.microsoft.com/en-us/power-platform/admin/self-service-analytics

Right that menu is not available with proper role


 There's an open bug on codecov where it can't publish results that I'm working with them to resolve.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@2ee2afe). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #11   +/-   ##
=========================================
  Coverage          ?   63.63%
=========================================
  Files             ?        2
  Lines             ?       55
  Branches          ?        0
=========================================
  Hits              ?       35
  Misses            ?       20
  Partials          ?        0

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 2ee2afe...4a454e3. Read the comment docs.


Quarks not recognised in Atom environment

Hello,
just looked at atom-supercollider again, gets better and better, and I'll eventually very sure to use it as my regular IDE soon  :)
One thing I now noticed is that Quarks (from the git system) are not recognised, although I download them through atom-sc via Quarks.gui. Even after a restart, they are not visible...

are you on the latest sc from master ?
there was a change a few weeks ago.
I'll check when I get a chance. you might be right and I forgot to change
the atom side.
On Thu, May 7, 2015 at 6:09 PM LFSaw<EMAIL_ADDRESS>wrote:

Hello,
just looked at atom-supercollider again, gets better and better, and I'll
eventually very sure to use it as my regular IDE soon :)
One thing I now noticed is that Quarks (from the git system) are not
recognised, although I download them through atom-sc via Quarks.gui.
Even after a restart, they are not visible...
—
Reply to this email directly or view it on GitHub
https://github.com/crucialfelix/atom-supercollider/issues/34.

Quarks.gui; works for me in Atom. I cannot remember if I simply had to update or actually had to update the Quarks location after it moved. I can check later tonight when I am home from work, if that is helpful.

Yep, "latest" (a day old?) ...
thanks!
Till

On 07. May 2015, at 18:34, Chris Sattinger<EMAIL_ADDRESS>wrote:
are you on the latest sc from master ?
there was a change a few weeks ago.
I'll check when I get a chance. you might be right and I forgot to change
the atom side.
On Thu, May 7, 2015 at 6:09 PM LFSaw<EMAIL_ADDRESS>wrote:

Hello,
just looked at atom-supercollider again, gets better and better, and I'll
eventually very sure to use it as my regular IDE soon :)
One thing I now noticed is that Quarks (from the git system) are not
recognised, although I download them through atom-sc via Quarks.gui.
Even after a restart, they are not visible...
—
Reply to this email directly or view it on GitHub
https://github.com/crucialfelix/atom-supercollider/issues/34.

—
Reply to this email directly or view it on GitHub.

It might be easier to test Windows with Atom, as the showstopper bug seems to be a Qt issue...

that would be interesting. let me know.
On Fri, May 8, 2015 at 1:39 AM Rainer Schütz<EMAIL_ADDRESS>wrote:

It might be easier to test Windows with Atom, as the showstopper bug seems
to be a Qt issue...
—
Reply to this email directly or view it on GitHub
https://github.com/crucialfelix/atom-supercollider/issues/34#issuecomment-100047239
.

Finally got a chance to fix this. Sorry for the delay.
Its a bit snakey: supercolliderjs dynamically creates a config file. Right now its mainly just to override the Error handling reportError methods.
But it also supports a local .supercollider.yaml file so you can have simple dependency management for each project you are working on.  http://supercolliderjs.readthedocs.org/en/latest/configuration.html#supercollider-yaml-and-sclang-config-yaml
If you want to handle your dependencies that way (even I am not doing it yet) then you wouldn't use Quarks.gui
So anyway I had to override some methods in Quarks so that it would save to both the generated config and your original config (sclang_conf.yaml).  It was just writing to the dynamically generated config and that's why they weren't sticking.
One caveat: if you click the nice green Recompile Library button then supercollider-js does not notice and fails to re-install its interpreter helpers. So use shift-cmd-K to recompile from atom instead of using the button in sc gui.  Filing a separate bug for that.

btw. Atom has gotten much faster recently and the new One themes are imho really nice.


Feature Request: Profiles or global options

It would be nice if there was a way to set different profiles for all the options, so that it's easier to be able to set up all of my lights with adaptive lighting. As it is currently, I have to adjust all of the options individually when setting up a new area. But I set all of my areas the same, and I have about 30. So when I want to change something like a sunset offset, I have to go back and change all of them individually. A the minimum, a global settings option that would set all of areas to the same settings would be great.

This would be great!

This is the same concept as my issue #724 - I was requesting an service call to save the settings so that I can have a script that sets and saves the settings for all of the configs..
Right now I wait 5 min after boot and set the adaptive lighting options for all instances..
https://github.com/zpriddy/Home-Assistant/blob/main/scripts/set_adaptive_lighting.yaml
Having a service call to save these settings would solve both of our issues..


ci: run extended array based on the person who open the PR

Description
Making sure that the teams check run with the person who open the PR credential, not the run triggerer.
Issue linked
NA
Checklist

[x] I have visited and read Develop PyMAPDL.
[x] I have tested my changes locally
[x] I have added necessary documentation or updated existing documentation.
[x] I have followed the PyMAPDL coding style guidelines.
[x] I have added appropriate unit tests that also ensure the minimal coverage criteria.
[x] I have reviewed my changes before submitting this pull request.
[x] I have linked the issue or issues that are solved to the PR if any.
[x] I have assigned this PR to myself.
[x] I have marked this PR as draft if it is not ready to be reviewed yet.
[x] I have made sure that the title of my PR follows Commit naming conventions (e.g. feat: adding new MAPDL command)

@pyansys-ci-bot LGTM.


populate github refspec

this will populate the refspec field using the <src>:<dest> branches.
For example, lets say we have a pull request for feature branch new-topic that will get merged into master. The refspec would be stored as new-topic:master

note that I think the refspec field can be used for cloning bitbucket pull requests, which do not support merge refs
# clone base
git clone https://github.com/drone-plugins/drone-slack.git
cd drone-slack

# clone head from remote repo, refspec patch-1:master
git pull https://github.com/ProLoser/drone-slack.git patch-1:master


[Linear Layouts] Implement LL conversion for DotOperand(version=2)

Note that the current implementation uses DotOperandEncodingAttr::getWarpsPerCTA, which is buggy for cases where the warps are not of the form [numWarps, 1] or [1, numWarps]. This is alright, as this PR does not use the LL for conversions yet.
We will activate its use for a subset of DotOperandEncodings in a PR coming soon.

Looks good overall but I'll look more in details after the lit tests are fixed

Addressed the reviews


Custom seed doesn’t default to last seed

Typically when you play a random seed and switch to custom seed, it sets the seed to the last seed you played. Now it doesn’t

2 leads atm:

Settings aren't saved after sending the player into the rando
Upon checking the debugger, when doing a random seed, the seed is not set in the settings.


Adding options to display filesystem storage size output in a specific unit

What type of PR is this? (check all applicable)

[ ] Refactor
[x] Feature
[ ] Bug Fix
[ ] Optimization
[ ] Documentation Update
[ ] Other: Replace this with a description of the type of this PR

Description
This adds new formatting options to the filesystem which allow the user to force the displayed storage amount in to be in a specific unit, that does not change depending on amount available. For example the user could specify gb_free and the output would always be in Gigabytes, even when over 1000 or under 1.
While it is inconsistent with how storage device manufacturers market their products, I use the system that each unit is 1024 of the unit immediately smaller than it rather than 1000, to be consistent with existing code.
Related Issues & Documents

Documentation (check all applicable)

[x] This PR requires changes to the Wiki documentation (describe the changes)
[ ] This PR requires changes to the documentation inside the git repo (please add them to the PR).
[ ] Does not require documentation changes

The new options would need adding to the wiki

Please merge this!


Update banner with style from ubuntu.summit.com

Note: Do not merge until cfp's pr is merged
Note 2: Originally we were waiting on some brand assets, but as we don't have them yet we will use the banner from ubuntu.summit.com for now
Done
Updates banner with style from ubuntu.summit.com
Fixes
https://github.com/canonical/web-squad/issues/5686

Closing, as after discussion with a designer the banner has changed a lot. And the review will be included in the overview page pr


Missing documentation for variable ops.

Please make sure that this is a documentation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:doc_template
System information

TensorFlow version: v1.13
Doc Link:

Describe the documentation issue
Hi, I am looking for documentation on the ReadVariableOp and VarHandleOp. Specifically as to what are the inputs that they take and what do they produce exactly. I dug around the source code for mentions of these ops and found some calls here and here. While this hints at the signature of these ops, I would love to look at the implementation, which probably exists in this module, but I can not find it in this public repo's master branch. Could someone point me to this?
Alternately, is there any documentation on these ops written this way?
Thanks!
We welcome contributions by users. Will you be able to update submit a PR (use the doc style guide) to fix the doc Issue?

@smr97,
You can find the documentation of ReadVariableOp and VarHandleOp with their inputs and output below

https://www.tensorflow.org/api_docs/python/tf/raw_ops/ReadVariableOp
https://www.tensorflow.org/api_docs/python/tf/raw_ops/VarHandleOp

Marking the issue as closed. Please feel free to re-open if necessary. Thanks!


Command prompting returns [object Map] after 30 seconds

I've noticed this bug where if u let a command prompt for 30 seconds it would return [object Map] https://i.imgur.com/N4urwsW.gif that gif shows you what I mean, I'm guessing the issue is you're awaiting messages with no error handling


Create automated test suite

I don't know how easy it will be to mock DryWetMidi.

Hopefully not too bad, this library mostly uses it through the IInputDevice and IOutputDevice interfaces, encapsulated in a MidiClient wrapper class.
The most complicated part looks like BehringerXTouchExtenderControlSurface.Open() which calls two real methods on the library, InputDevice.GetByName(DeviceName), OutputDevice.GetByName(DeviceName). This may be hard to mock since they're static methods, so I may have to wrap them in an internal virtual method that the tests can override.

For publishing test reports, Testspace looks good and free for public open-source personal projects, and it works with GitHub Actions. There is a shield image available as well.

Added 519 unit tests using xUnit, FakeItEasy, and FluentAssertions. Runtime on my computer is about 1 second. Coverage is 100% of 386 statements.


Python 3.8 support for Visual Studio 2019

Python 3.8 is still not supported in Visual Studio 2019 IDE
Some of features are not supported as there is a warning when I create a new Python file in Visual Studio that my current Python is not supported in IDE(i.e. 3.8.5)
Visual Studio Code is a good alternative but Visual Studio 2019 is a better option for Python because it's a IDE

Dupe of #5822

Dupe of #5822

Dupe of #5822

Which has been open for 15 months without a peep from Microsoft.
For the record:
3.7 has been out of support for 7 months.
3.8 will go out of support in April.

Dupe of #5822

Which has been open for 15 months without a peep from Microsoft.
For the record:
3.7 has been out of support for 7 months.
3.8 will go out of support in April.

Dupe of #5822

Which has been open for 15 months without a peep from Microsoft.
For the record:
3.7 has been out of support for 7 months.
3.8 will go out of support in April.

Visual Studio is stuck on Python 3.7.8 which is currently only on security updates. The security updates will go away on January 27, 2023 (2 years and 3 months).
The EOL dates are here:
https://endoflife.date/python
https://devguide.python.org/#status-of-python-branches


improve raw view ux

Add tooltips to icon buttons, have useful edit and delete buttons on all the cards. Add the ability to clear all the operations. Truncate the saved operations to save space. When editing an operation, it would be useful to have the full border around the text area, since they are quite large at times.

The UI fails to meet our Accessibility policy. It is unmaintainable and will be removed shortly.


Please add random seed

Please add random seed. This helps us by not having to change the prompt slightly all the time.

See #3 .
Seed is not returning consistent results due to a bug in llama.
I'll merge the seed branch to force different results but we won't see consistent results when reusing the seed.

Added! v2.0.0 will be out shortly with changes

Great Thanks.


How do you exclude with patterns?

for example, npm-run-all has the ability to do this:
    "lint": "npm-run-all --aggregate-output --continue-on-error --parallel 'lint:!(fix)'",

where it runs: lint:js, and lint:md but not lint:js:fix or lint:md:fix
can concurrently omit via pattern?

Nope, this isn't possible at the moment.
Best you can do is add some suffix to your scripts, e.g. lint:js:main, and then run concurrently "npm:lint:*:main"

would a PR to support this be of interest?

I don't know if many people ever needed something like this, so I'm keen to see how complex it can get -- if it turns out being a simple solution to implement and document, sure!


Unify notification title format for weworkremotely

Weworkremotely format: ${company}: ${title}
Desired format: ${title} @ ${company}

Weworkremotly notifications have desired format.


The first line is incorrect when formatting an array.

Description
The first line is incorrect when formatting an array.
Input
The code looked like this before beautification:
import { js_beautify as beautify } from 'js-beautify';

const template = [
    { param: 'params' }
];
const templateStr = JSON.stringify(template);
const output = beautify(templateStr, { indent_size: 2 });

Expected Output
The code should have looked like this after beautification:
[
    { param: 'param' }
]

Actual Output
The  code actually looked like this after beautification:
                [
    { param: 'param' }
]

Steps to Reproduce
Environment
OS: mac0s 10.12.3
Settings
Example:
{
    "indent_size": 2
}

I can't reproduce this on the jsbeautifier website.  Is there more code around the example you gave?

@bitwiseman See https://codepen.io/chenxu/pen/boLKRW?editors=1111

@xuchenchenBoy
Change the HTML in the linked paged to:

{{codes}}

And the formatting is fixed.

@bitwiseman Thanks.


persist backup block

persist backup blocks blow LIB will be pruned when fork db advance root

Change Type
Select ONE:

[ ] Documentation

[ ] Stability bug fix

[ ] Other

[ ] Other - special case

Testing Changes
Select ANY that apply:

[ ] New Tests

[ ] Existing Tests

[ ] Test Framework

[ ] CI System

[ ] Other

Consensus Changes

[ ] Consensus Changes

API Changes

[ ] fetch_block_by_id / fetch_block_by_num

Documentation Additions

[ ] Documentation Additions

with another design


Add test for some PHP special chars

Let's consider this source code:

<?php __('plain'); ?>

<?php __('DATE \a\t TIME'); ?>

<?php __("DATE \a\\t TIME"); ?>

<?php __("DATE \\a\\t TIME"); ?>

<?php __("FIELD\tFIELD"); ?>

Here we have only 3 different strings, but the php extractor finds these strings:
msgid "plain"
msgstr ""

msgid "DATE \\a\\t TIME"
msgstr ""

msgid "DATE \\a\\\\t TIME"
msgstr ""

msgid "DATE \\\\a\\\\t TIME"
msgstr ""

msgid "FIELD\\tFIELD"
msgstr ""

Here's the output of xgettext --language=PHP --keyword=__ special-chars.php:
msgid "plain"
msgstr ""

msgid "DATE \\a\\t TIME"
msgstr ""

msgid "FIELD\tFIELD"
msgstr ""

I know you may have some reticence about what I'm going to write, but the best approach IMHO would be to replace https://github.com/oscarotero/Gettext/blob/e93f74356280016bd50c41c62cc858c3005791e7/src/Utils/PhpFunctionsScanner.php#L42-L50 with:
$bufferFunctions[0][2][] = eval('return '.$value[1].';');

Contrary to what 99% of PHP developers thing, eval is not evil. You only need to know when to use it, and this is the case: $value[1] is for sure a PHP-encoded string (ie T_CONSTANT_ENCAPSED_STRING).
The only alternative I can see is to manually unescape the string, differentiating the behaviour if the string is single quoted (we only need to look for \ and ') or double quoted string (we'll have to look for all the cases reported here).

Yes, you're right: I'm reticent :D
If the string had other evaluable elements, like a variable: "hello $world" or "hello {$object->getWorld()}" they would be executed.

So, shall we parse the string char by char?

What about something like this:
function decodeString($value)
{
    $result = '';
    if ($value[0] === "'" || strpos($value, '$') === false) {
        $result = eval("return $value;");
    } else {
        // Manual and slow way of parse $value
    }

    return $result;
}

Here's the full code that should work:
function decodeString($value)
{
    static $simpleDictionary;
    $result = '';
    if ($value[0] === "'" || strpos($value, '$') === false) {
        $result = eval("return @$value;");
    } else {
        $value = substr($value, 1, -1);
        if (!isset($simpleDictionary)) {
            $simpleDecodeDictionary = array(
                '\\' => "\\",
                '$' => '$',
                '"' => '"',
            );
        }
        while (($p = strpos($value, '\\')) !== false) {
            if (!isset($value[$p + 1])) {
                break;
            }
            if ($p > 0) {
                $result .= substr($value, 0, $p);
            }
            $value = substr($value, $p + 1);
            if (isset($simpleDictionary[$value[0]])) {
                $result .= $simpleDictionary[$value[0]];
                $value = substr($value, 1);
            } elseif (preg_match('/^([a-wyz]|[0-7]{1,3}|x[0-9A-Fa-f]{1,2})/', $value, $m)) {
                $result .= eval('return "\\'.$m[1].'";');
                $value = substr($value, strlen($m[1]));
            } else {
                $result .= '\\';
            }
        }
        $result .= $value;
    }

    return $result;
}

If it's ok I can update this pull request accordingly

Even simpler:
function decodeString($value)
{
    $result = '';
    if ($value[0] === "'" || strpos($value, '$') === false) {
        $result = eval("return @$value;");
    } else {
        $value = substr($value, 1, -1);
        while (($p = strpos($value, '\\')) !== false) {
            if (!isset($value[$p + 1])) {
                break;
            }
            if ($p > 0) {
                $result .= substr($value, 0, $p);
            }
            $value = substr($value, $p + 1);
            if (preg_match('/^([\\$"]|[a-z0-9{}]+)/', $value, $m)) {
                $result .= eval('return "\\'.$m[1].'";');
                $value = substr($value, strlen($m[1]));
            } else {
                $result .= '\\';
            }
        }
        $result .= $value;
    }

    return $result;
}

...and without preg_match, for lightning fast execution:
function decodeString($value)
{
    $result = '';
    if ($value[0] === "'" || strpos($value, '$') === false) {
        $result = eval("return @$value;");
    } else {
        $value = substr($value, 1, -1);
        while (($p = strpos($value, '\\')) !== false) {
            if (!isset($value[$p + 1])) {
                break;
            }
            if ($p > 0) {
                $result .= substr($value, 0, $p);
            }
            $value = substr($value, $p + 1);
            $p = strpos($value, '$');
            if ($p === false) {
                $result .= eval('return "\\'.$value.'";');
                $value = '';
                break;
            }
            if ($p === 0) {
                $result .= '$';
                $value = substr($value, 1);
            }
            else {
                $result .= eval('return "\\'.substr($value, 0, $p).'";');
                $value = substr($value, $p);
            }
        }
        $result .= $value;
    }

    return $result;
}

I think it's a bit overcomplicated. Why not use an unique solution for all cases? For example (not tested):
function fix ($value) {
    if (strpos($value, '\\') !== false) {
        if ($value[0] === '"') {
            $value = preg_replace('/[^\\\]\\\([^nrtvefxu0-7\$\\"])/', '\\\\$1', $value);
        } else {
            $value = preg_replace("/[^\\\]\\\([^'])/", '\\\\\1', $value);
        }
    }

    return substr($value, 1, -1);
}

Eval is going to be used in very few cases, so I don't see any value using two solutions for the same problem.

think it's a bit overcomplicated. Why not use an unique solution for all cases? For example (not tested):

It's broken: see https://3v4l.org/WqGos

Eval is going to be used in very few cases, so I don't see any value using two solutions for the same problem.

Well, I'd use it for all the strings not enclosed in " or not containing $, and I think they are the 99%...

See the results from the approach of mine: https://3v4l.org/tQgip

Ok, it's a bit hard to fix this with regular expressions (at least for me).
I accept your method, but if the string not contain a backslash, avoid to use eval, please. A simple substr is faster.

Done... let's see what Trafix says :wink:

If this pull request will be merged, I have already another fix (the po export removes special chars like "\t" and could mess up multibyte strings)

Ah, ok, thank you. (sorry for the late, I'm currently a bit bussy)

See #92

Seems like sensiolab does not like eval https://insight.sensiolabs.com/projects/496dc2a6-43be-4046-a283-f8370239dd47/analyses/23 :disappointed:

I'm open to any valid alternatives... As I said, 99% of developers consider eval=evil, but I really don't understand why...
It's as dangerous as a dB query: if you let dangerous strings to be executed is bad, but it's not a valid reason to don't execute queries at all...

Yes, in this case I agree that it's not dangerous. But having those critical security errors is not nice.
Let's leave eval() until we find a better solution.


Only randomize on tap or left mouse-button click.

This allows you to right-click and select "Save As…" in Chrome to save the generated image as a PNG.
Previously, the image would randomize immediately so you could only save the subsequent generation.

Good call, thanks!


[BUG] The noon object has two version keys when creating the channel with enabled command

Describe the bug
When a guild does not contain yet the current channel, using the enabled flag with the noon command create the version property two times on the noon object.
Expected behavior
Only to have one version property.

Insights:

when the channel does not exist the noon object will be created with two versions
when the noon object is empty the version is not created

when the noon object is empty the version is not created
In fact, this scenario should never happen.
Either the object exists and you have a version either it does not exists.
I know that this could be checked but starting to not trust the interfaces will make the versioning completely useless and a breaking change will happen.

Damn I could not find how this can happen.
The tests are pretty strong and I do not think that something could be missing.
The logs are good but not great.
Could add for now new logs to details which part of the update is being done.

This was not an issue at all.
The Firebase preview show the noon version property AND the features version property at the same level.
The UI is just bad since there is no delimiter nor tab to separate those.


ffpyplayer audio inconsistently obeys SoundLoader volume setting

Versions

Python: 3.5.3
OS: Windows 7 64-bit
Kivy: 1.10.1
Kivy installation method: pip (under virtualenv)

Description
When using ffpyplayer as audio provider, and I've only tested .ogg Vorbis files, a SoundLoader object ignores the .volume property at the first .play() call. At the next call, it will obey its value, and it will work once more. After that, it seems to ignore .volume again.
I also found out that if you set .volume immediately after calling .play(), it will work the first time, but will still behave inconsistently in subsequent plays.
Code and Logs
Here's my log and a code sample that reproduces the issue. Note that I am using ffpyplayer because the kivy.deps.sdl2 audio provider will not play .ogg Vorbis files.
https://gist.github.com/tmarplatt/727b5e13dbd0b617582bb15420c2e448
Ogg file tested (or use your own) you can get it from:
https://freesound.org/people/zonkmachine/sounds/164854/

There is something else about ffpyplayer and ogg files, and please tell me if I should open another issue for it: CPU usage piles up after each sound ends playback. If I run the script above, CPU% increases linearly (in steps of about 16%).

@Julian-O I recommend closing this issue.  I was not able to reproduce the problem.
Win 11, Kivy 2.2.1, Python 3.11.4
I download the audio file the user suggested.  I used the code from the OP and created 2 additional scenarios, I could not replicate the issue.
# original file
from kivy.app import App
from kivy.core.audio import SoundLoader
from kivy.clock import Clock

class FFPyPlayerTest(App):

    def on_start(self):
        Clock.schedule_once(self.play, 1)

    def play(self, dt):
        self.sound = SoundLoader.load(filename='164854__zonkmachine__bit_bomber3.ogg')
        self.sound.bind(on_stop=self.play_next)
        self.sound.volume = .2
        self.sound.play()

    def play_next(self, *args):
        Clock.schedule_once(self.play, 2)

FFPyPlayerTest().run()

# removed loading from the loop
import os
os.environ['KIVY_AUDIO'] = 'ffpyplayer'

from kivy.app import App
from kivy.core.audio import SoundLoader
from kivy.clock import Clock

class FFPyPlayerTest(App):

    def on_start(self):
        self.sound = SoundLoader.load(filename='164854__zonkmachine__bit_bomber3.ogg')
        self.sound.bind(on_stop=self.play_next)
        Clock.schedule_once(self.play, 1)

    def play(self, dt):
        self.sound.play()
        self.sound.volume = .2

    def play_next(self, *args):
        Clock.schedule_once(self.play, 2)

FFPyPlayerTest().run()

# gui based test
import os
os.environ['KIVY_AUDIO'] = 'ffpyplayer'

from kivy.app import App
from kivy.core.audio import SoundLoader
from kivy.lang import Builder

kv = """
AnchorLayout:
    BoxLayout:
        size_hint_y: None
        height: dp(48)
        Button:
            text: 'play'
            on_release: app.play()
        ToggleButton:
            text: {'normal': 'loud', 'down': 'soft'}[self.state]
            on_text: app.set_volume(self.text)

"""

class FFPyPlayerTest(App):
    def __init__(self, **kwargs):
        super().__init__(**kwargs)
        self.sound = None

    def build(self):
        self.sound = SoundLoader.load(filename='164854__zonkmachine__bit_bomber3.ogg')
        return Builder.load_string(kv)

    def play(self):
        self.sound.play()

    def set_volume(self, text):
        self.sound.volume = 1 if text == 'loud' else 0.2

FFPyPlayerTest().run()

@ElliotGarbus: You are a champion!
Closing as fixed


fix(playout-gateway): improve handling and typing of TSR events (fixes some logging issues)

This PR is being opened on behalf of TV 2 Norge.

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Bug fix

What is the current behavior? (You can also link to an open issue here)

Some of the event listeners in playout-gateway for TSR are wrong, and they're wrong because the types are messed up, which led a previous developer to opt out of types for this section of the code.

What is the new behavior (if this is a feature change)?

This PR restores typing to this section of the code and fixes the relevant type errors, which fixes thing like error logging.

Other information:

This is a hack to work around limitations of the Promisify type used by threadedclass. Thought should be given to better alternatives before merging this.
Status

[x] Code documentation for the relevant parts in the code have been added/updated by the PR author
[x] The functionality has been tested by the PR author
[ ] The functionality has been tested by NRK

I have made the changes I was after, but this has conflicted with other changes in release50 so I think it could do with a retest before merging as each change took a different approach to fixing the logging.


Fixed #27557 -- Casted GEOSGeometry only when necessary

Thanks Pete Flugstad for the report.

What about the added comments?

Makes sense.


abount endpoint '{POST}/abc' and  '/abc'

Please answer these questions before submitting your issue.

Why do you submit this issue?
[ ] Question or discussion
[ ] Bug
[ ] Requirement
[ ] Feature or performance improvement

Question

I find '{POST}/login' and  '/login' in my service's endpoint  list ;
The interface of get mode is the same and this phenomenon exists at each endpoint .
And I want to know why it happened .
The framework of my project is spring-boot2.0.9 . The service registry is Eureka.

Which components do these two spans say?

I get this case, I don't know why? @wu-sheng ,skywalking7.0 ,springboot1.5.8.

Duplicates #4907. Will close that ticket with fix


Does disabling SSE4.1 on openal-soft improve backward compatibility?

I guess SSE4.1 is only used when CPU supports it, so disabling it at compile time doesn't improve compatibility with older computers.
https://github.com/native-toolkit/openal/commit/806867f198dfeb12de2f9f61c5aadd041eb544c3

Yes, I believe this helped support for some older systems, which might have been running Windows XP, I'm not sure. If we find this is not necessary, I'd be happy to rediscuss it :smile:


Persist problem with "Entity has identity through a foreign entity"

Hi guys,
I'm having a hard time understanding why Doctrine (version 2.5.3) won't handle the following schemas as expected. I made two example entities Test and TestTranslated. In the first example the TestTranslated table has a dedicated primary key and in the second example the TestTranslated entity has a compound primary key build out of two foreign keys.
... Running the code:
$em = $this->getDoctrine()->getEntityManager();

$a = new Test();
$a->setCity('Berlin');
$a->setDatetime(new \DateTime());

$c = $em->getRepository(Language::class)->findOneById('en');
$b = new TestTranslation();
$b->setTest($a);
$b->setLanguage($c);
$b->setLocation('Ljubljana');
$b->setMetaDescription('Meta description');
$b->setMetaKeyword('Meta keyword');
$b->setMetaTitle('Meta title');
$b->setSlug('ljubljana');

$a->addTestTranslation($b);

$em->persist($a);
$em->flush();

I get MySQL entries in both Test and TestTranslated entities with one flush. So all good.
Example2:
...Bundle\Entity\TestTranslation:
    type: entity
    table: testtranslation
    id:
        test:
            associationKey: true
        language:
            associationKey: true
    fields:
        location:
            type: text
            nullable: true
            length: 255
            fixed: false
        metaTitle:
            type: string
            nullable: true
            length: 75
            fixed: false
            column: meta_title
        metaDescription:
            type: string
            nullable: true
            length: 210
            fixed: false
            column: meta_description
        metaKeyword:
            type: string
            nullable: true
            length: 150
            fixed: false
            column: meta_keyword
        slug:
            type: string
            nullable: false
            length: 255
            fixed: false
    manyToOne:
        test:
            targetEntity: Test
            inversedBy: testTranslation
            joinColumns:
                test_id:
                    referencedColumnName: id
                    nullable: false
        language:
            targetEntity: Language
            inversedBy: testTranslation
            joinColumns:
                language_id:
                    referencedColumnName: id
                    nullable: false

In this case I get an error:
Entity of type ...Bundle\Entity\TestTranslation has identity through a foreign entity ...Bundle\Entity\Test, however this entity has no identity itself. You have to call EntityManager#persist() on the related entity and make sure that an identifier was generated before trying to persist '...Bundle\Entity\TestTranslation'. In case of Post Insert ID Generation (such as MySQL Auto-Increment or PostgreSQL SERIAL) this means you have to call EntityManager#flush() between both persist operations.
What I can do is run the code in the following way:
$em = $this->getDoctrine()->getEntityManager();

$a = new Test();
$a->setCity('Berlin');
$a->setDatetime(new \DateTime());

$em->persist($a);
$em->flush();

$c = $em->getRepository(Language::class)->findOneById('en');
$b = new TestTranslation();
$b->setTest($a);
$b->setLanguage($c);
$b->setLocation('Ljubljana');
$b->setMetaDescription('Meta description');
$b->setMetaKeyword('Meta keyword');
$b->setMetaTitle('Meta title');
$b->setSlug('ljubljana');

$a->addTestTranslation($b);

$em->flush();

but I don't understand why in this case Doctrine isn't that smart to deal with insert sequence. Meaning that it would first save Test entity and then TestTranslated. It works as expected with Example1 but not with Example2 although there's almost no difference in table schema. The only difference is the compound primary key.
I personally don't see any reason why Doctrine wouldn't act the same. Is there a reason behind this behavior or is it just expected that every table has a dedicated primary key? I prefer having a compound primary key.

This seems to be related to https://github.com/doctrine/doctrine2/pull/6701 (and duplicates #6531 and #6043).
@va5ja you can use a pre-generated value as ID in order to solve this for now (UUID, for example).

@va5ja please verify if this is actually a duplicate - I merged all related changes and will release 2.6.1 with them.

@va5ja thanks!


v6 removal of /twirp prefix makes upgrading hard

The current v6_prerelease branch changes the URL scheme. It goes from /twirp/<package>.<service>/<method> to just /<package>.<service>/<method>.
Unfortunately, this is difficult to upgrade into. Consider a service which has generated v5 code today, and which has a client which vendors that generated Go code. The service wants to upgrade to generating v6 code, and they currently register their Twirp service like this:
If they regenerate their code and don't tell the client to upgrade their vendored copy, then all client calls immediately start failing, since the client is hitting the old URL, which 404s. If the client upgrade first, same thing - all 404s. The client and server need to deploy an upgrade version simultaneously. That's unacceptably painful. We need something better.
What are the options?

Document code changes the service owner can make

We could tell service owners that they must run their service on two URL path prefixes if they are upgrading, at least until all their clients are updated. For example, they'd need to do this:
mux := http.NewServeMux()
mux.Handle(HaberdasherPathPrefix, svc)
mux.Handle("/twirp" + HaberdasherPathPrefix, http.StripPrefix("/twirp", svc))

This is not very pleasant. It's gross legacy code, and hard to know you need to do this; I don't like solutions that require manual action from our users. It's hard to know when you can remove this code, as well. And if users already have a mux, they could get confused.
We could make this more visible by renaming the PathPrefix constant. That way, code won't compile, and hopefully the user goes and looks up what's wrong and learns that they need to make a code change. But this is a bad option; upgrading should be painless.

Leave /twirp as a default prefix, but allow it to be customized

We could change the branch to default to using the /twirp prefix. But if the user specifies, we could remove the prefix in generated code. This would make upgrading easy - the user would only run into trouble if they enabled a custom prefix, which is kind of their own fault. Most upgrades would be painless.
But I already rejected this approach in https://github.com/twitchtv/twirp/issues/48, really. My argument there was that non-customizability is a feature, not a bug. It makes third-party generators harder to get right, and complicates the spec a bit. That said, it's worth looking at how we'd do this.
Protobuf files support options. We could have something like this:
// this path depends on your local machine :(
import "path/to/twitchtv/twirp/genconfig.proto"

service Haberdasher {
  option (twitchtv.twirp.genconfig.prefix) = "/whatever/you/want"
  rpc MakeHat(Size) returns (Hat)
}

The challenge is really that the option must be defined in some other .proto file which gets imported by the user. Twirp would have to distribute that .proto file somehow, and provide instructions on how to import it. This is tricky - protoc is not a friendly tool. But perhaps this is alright, since we hope that custom prefixes are the exceptional case, not the common one, so it's alright for them to be a little difficult.

Go back to requiring /twirp prefix

Finally, we could revert this change entirely. This is appealing to me - it keeps Twirp simple and the upgrade is really clean. I like that no third-party generators become incompatible, since the protocol remains the same.
But there are a few people who are using the prerelease branch today because they hate that prefix. I'd like to hear from them on their needs, once again. We kind of touched on them in #55, but that covered other topics, and I don't think we have good written reasons that removing the /twirp prefix specifically is worth the cost. I'd like to hear those arguments.

I lean towards option 3, in absence of clear reasons the /twirp prefix needs to be removed.

cc @rynop @jonathaningram @rmasp

Thank you for putting thought into this.
My vote is #1, then #2.  I love Twirp, but I don't want consumers knowing that I run it. Same reason you remove apache/nginx/php/blah product&version response headers from HTTP servers.  Also further locks me in to Twirp (hopefully you never leave Twitch :)
FWIW, I run v5 today behind a CDN.  My CDN adds the /twirp prefix for me.  My Twirp clients all don't send /twirp.  I'd prefer to use v6 and not have this extra config, but it's so far behind master I've been using v5 for new projects.
#1 is one line.  Someone using Twirp is already a bit adventurous/skilled, so IMO its a no-op.
I'm ok with the extra pain outlined in #2, because if this is the chosen option, I'm obviously the minority and should be expected to do extra work to get away from the default.

I think keeping things simple and stable is a better feature than removing /twirp. Though I'd prefer the /twirp prefix to be removed to hide a server's internals.
So my vote is either #2 or #3
For #3, I think we can maybe just document how to hackishly remove that prefix for any user who wants to do it like @rynop suggested. For example, by adding a Go middleware that just prepends the /twirp prefix.

My vote is for 2. It doesn't have to be a schema item though. I'd even be happy with a protoc plugin option, like the one I wrote for a v6_prerelease compatible protoc-gen-twirp_swagger
    --twirp_swagger_out=twirp_route="":./swaggerui/rpc

I've been socializing twirp across my organization in hopes of making it a standard microservice architecture. About 1/2 the engineers I show it too ask, "Can we get rid of this /twirp thing?" I also anticipate I would get exec blowback if we decided to build open APIs or expose APIs to partners.
In that case I might be forced in to #4: fork twirp and start a schism. That would be a last resort, for sure.

Option 1 doesn't look that bad to me. Is a major update and the instructions are clear: to update you need to support both with and without the prefix.
Option 2 looks decent. Although I think the prefix should default to empty. New twirp users should not have to worry about the prefix, they can always use a mux and a baseURL with prefix on their clients (which feels more intuitive). The upgrade instructions in this case would require to explicitly add the prefix option to "/twirp". This option doesn't need to be respected in other language implementations, because the easy way out is to just add that prefix in the baseURL when instantiating clients.
Option 3 seems like the easy thing to do now, but has the issue of always having to answer the same question over and over again for ever and ever. Maybe we can automate that by a new page in the documentation, with some known solutions for it. Maybe maintain a no-prefix branch?

I agree that keeping Twirp (and generally anything) customisation-free is good, so I'd probably prefer option 2 the least.
Regarding option 1. Putting aside anything to do with Twirp, APIs are often retired (after some fair warning and time hopefully), and I think service owners need to cater for this no matter if they use Twirp or not. I can personally say that I've retired certain endpoints and it was relatively easy to track the customers/clients that were calling the deprecated endpoints and give them a nudge to upgrade. With that in mind, option 1 would probably be my preferred option. It's a little gross, but not too difficult for the service owner to implement, along with good comments and commit messages.
Regarding option 3. As @rynop says, I think the /twirp prefix is the only thing that really looks like vendor lock-in. Correct me if I'm wrong, but everything else in Twirp looks pretty generic and so one could reasonably remove Twirp from the equation and clients nor the service owner would be none the wiser. I do love that Twirp can be used "just like a REST API" where you could hand out CURL commands and let customers implement their own clients. However, I don't think that is Twirp's primary use case: rather it's an edge case and so I think I'm reverting a little on my first opinion that this prefix is a deal-breaker. I think I'd still prefer option 1 but could live with option 3.
Thanks @spenczar for taking the time on this.

We are not running Twirp in production right now, but we are on our way to make Twirp a big part of our ecosystem. So we are not affected by this decision, although, coming as a new user I would prefer option #1.

For better or worse, we use a https://internal.gateway/<service>/ approach to routing our services, so we're already juggling http.StripPrefixs in various places.
Option 1 would get my vote :)

I don't think removing the /twirp URL path prefix is the right decision for Twirp.
Introducing this breaking change in a new major version of Twirp could affect some of its users too significantly. Some organizations that use Twirp would need to migrate hundreds of services for a change that may not have any business value. Even worse, the breaking change could result in unsuccessful migrations that may cause impact to services. I don't think this change would serve any purpose at Twitch, and Twitch is a sufficiently large user of Twirp to experience a significant amount of disruption from the change.
That being said, I do understand the need to change the path prefix for security or cosmetic purposes, and Twirp should have a sufficient answer for that use case even if it's not supported directly in Twirp's spec.
This problem seems solvable with code generation as a protoc plugin. I would prefer a plugin in comparison to a protobuf option because I think it is more clear to users that the mechanism is not supported by Twirp's spec. I also think it's appropriate to push the added complexity onto the users that have a need to deviate from Twirp's spec if it avoids breaking changes.
If there is interest for such a plugin, I am interested in gathering requirements for it and collaborating on an implementation. I'm interested in changing my view on this issue if presented with compelling arguments as well.
cc @rmasp

Twirp is not going to change the path format in the short term. The Twirp team has archived the draft for protocol v6.
However, it is still possible to implement a Twirp service without the /twirp prefix (or a different prefix) by using middleware and adapting the clients to call the right path.
As an example, this is how to build a Twirp service without the "/twirp" prefix.
Use middleware to strip the prefix in the service:
mux := http.NewServeMux()
mux.Handle(MyTwirpPathPrefix, http.StripPrefix("/twirp", svc))

All clients need to be configured to call into the URL without the prefix, which is different for different implementation or languages (this is the main reason why Twirp protocol can not be updated easily). For a Go client, a roundtripper can be used on the http.Client:
type StripPrefixTransport struct {
	Prefix string
	Transport http.RoundTripper
}

func (t *stripPrefixTransport) RoundTrip(r *http.Request) (*http.Response, error) {
	r = requestWithStripPrefix(r, t.Prefix)
	return t.Transport.RoundTrip(r)
}

func requestWithStripPrefix(r *http.Request, prefix string) r *http.Request {
	p := strings.TrimPrefix(r.URL.Path, prefix)
	if len(p) == len(r.URL.Path) {
		return r
	}
	r2 := new(Request)
	*r2 = *r
	r2.URL = new(url.URL)
	*r2.URL = *r.URL
	r2.URL.Path = p
	return r2
}

The StripPrefixTransport could be used on the *http.Client as Transport property.


Custom image module makes builder hang forever on initial load, then works after reload unless you dump your cache

Problem Description
I have a custom divi module that I use in the Divi builder. It works fine all the time EXCEPT for the first time you create a page or when using a new computer. I.e. when the builder isn't already cached. In that case, the builder hangs forever with the spinning wheel and you have to either hit the back button or reload the page. From then on the module will work fine, unless you switch to a new computer or force the browser to dump its cache. This wouldn't be a huge problem except that I use the Divi builder with scores of students each semester and this initial failure (and repeated failure as they tend to use a different computer each time) makes them hate Divi and think the whole site is janky. ANY help in resolving this would be much appreciated.  Elegant Themes said it wasn't their problem and I should ask you all... This is my first time ever using GitHub, so I apologize for any and all gaffes.
Steps To Reproduce

Add new post.
Load Divi Builder.
Add columns.
Add FIT image/caption module (a version of the Divi image module).

Screenshot and/or gif
I put the code in my repository.  The hang screen isn't that interesting, but here it is:

QA Checklist
Reporter

[ ] Helpful title used

Helpful title format: Scope(s) :: One Sentence To Describe The Issue
Example: VB :: Floating Toolbar :: Jumps To Top Of Page

[x] Detailed description
[x] Steps to reproduce included
[ ] Support Ticket URL included
[ ] Image and/or gif

Developer

[ ] Steps to reproduce proven solved
[ ] Test included and coverage acceptable and all Tests passing?
[ ] PR description has layman's description of problem and solution and "Areas Affected" listed/explained?
[ ] PR branch includes latest commits from master branch (at the time of checking)?
[ ] Estimated points on issue
[ ] Proper code documentation and conventions?
[ ] Proper RTL appearance
[ ] i18n used if applicable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?

Peer Review

[ ] PR branch includes latest commits from master branch (at the time of checking)?
[ ] Steps to reproduce proven solved
[ ] Affected areas are tested and ok
[ ] Proper code documentation and conventions
[ ] Proper RTL appearance
[ ] i18n used if applicable
[ ] Test(s) included and coverage acceptable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?

Quality Assurance

[ ] All tests passed on CircleCI.
[ ] Steps to reproduce passed, the fix works.
[ ] No errors in console or output
[ ] Affected areas are tested and ok
[ ] The fix does not change a customer's website in any surprising or unintended way

[ ] If Divi's design is adjusted, settings are migrated gracefully and invisibly

Design Review

[ ] UI is to spec
[ ] UX is to spec
[ ] Design is to spec
[ ] No surprising visual changes to a customer's website will be experienced after updating

[ ] Any/all visual changes are migrated gracefully

Dev Review

[ ] Test(s) included and coverage acceptable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?
[ ] i18n used if applicable
[ ] Implementation acceptable, DRY, Scaleable, future safe
[ ] Best practices used, cleverness and grokage, ES6 use, etc.

Hi @fashiontimeline ,
I was unable to reproduce the issue - could you please open up the developer console and screenshot any errors that may be shown when you trigger the issue?

It was a plugin conflict.  Found it using Health Check.  Thanks for the help!


⚠ Bofrost.be has degraded performance

In c5a7f14, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1804 ms

Resolved: Bofrost.be performance has improved in eea9f77.


ARMv7-R MPU Port Demos

Description
FreeRTOS Port for the ARMv7-R Architecture, tested on the Cortex R4 and Cortex R5 Processors
Demos related to the FreeRTOS-Kernel Pull Request for ARMv7-R MPU Ports.
The RM46 Demo was tested using the LAUNCHXL2-RM46, and the TMDXRM46HDK
The RM57 Demo was tested using the LAUNCHXL2-RM57L
and the TMDXRM57LHDK Development Kits.
Added a github workflow to perform a build of the project
The RM46 project's CMakeLists.txt uses fetch content to checkout the correct tag of the repo
The RM57 CMakeLists.txt project uses the existing submodule, which is why in this PR the submodule for the kernel has changed.
I can use either for the final workflow and CMakeLists.txt file, but wanted to raise this with both.
Both of these projects also have the necessary Code Composer Studio files and projects as well, which is mentioned in their READMEs.
All of the demo files are a duplicate, but I wasn't sure if we wanted to move them to the Common folder or to do what I had done.
Test Steps

Checklist:

[x] I have tested my changes. No regression in existing tests.
[x] I have modified and/or added unit-tests to cover the code changes in this Pull Request.

Related Issue

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

A few too many files to go through with a fine tooth comb. I'll approve to aboid blocking. We'll just need to keep an eye on this demo to ensure any bugs which pop up are dealt with.


Let SHERLOCK implement its own period grid

TLS period grid have a fixed equation and can't be easily modified. Therefore we could use a fork from tls and get what we need.

What about a computation of the density of the period grid for 2 sectors in a row, 3, 4, and so on. Then, adopt these densities when the sectors are largely separated such a sector 1 and 30 in TESS data ?


Add Nghe An College of Economics

@adminnace I am sorry, this request has signs of fraud. I have to close it without merging. If you are a student of this school, please apply with official document ("OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students).


chore: bump contracts version to 4.9.3

I suggest to bump contracts versions to 4.9.3
No changes in JS code required.
Related PRs:
#861
#862
#864
#865
#866
#867
#868
#869
Tests:

one skipped: - [skip] manifest › rename hardhat from unknown to dev manifest
all rest pass

$ yarn test
yarn run v1.22.19
$ tsc -b && wsrun -ms test
plugin-truffle-test has no test script, skipping missing
@openzeppelin/upgrades-core
$ tsc -b && hardhat compile --force && node scripts/copy-build-info.js && ava
...
 |   - [skip] manifest › rename hardhat from unknown to dev manifest
...
 |   399 tests passed
 |   1 test skipped
@openzeppelin/hardhat-upgrades
$ tsc -b && bash scripts/test.sh
 |
 |   307 tests passed
@openzeppelin/truffle-upgrades
$ yarn compile && bash test.sh
$ tsc -b
$ /openzeppelin-upgrades/node_modules/.bin/truffle test
 | This version of µWS is not compatible with your Node.js build:
 |
 | Error: Cannot find module '../binaries/uws_linux_x64_115.node'
 | Require stack:
 | -<EMAIL_ADDRESS> | - /openzeppelin-upgrades/node_modules/ganache/dist/node/core.js
 | - /openzeppelin-upgrades/node_modules/truffle/build/test.bundled.js
 | - /openzeppelin-upgrades/node_modules/original-require/index.js
 | - /openzeppelin-upgrades/node_modules/truffle/build/cli.bundled.js
 | Falling back to a NodeJS implementation; performance may be degraded.
 |
....
 |
 |   181 passing (53s)
 |
Done in 411.15s.

Thanks for suggesting this.  However, there are no identified changes between 4.8.3 to 4.9.3 that affect proxy functionality, so this version bump is not considered to be necessary.  See https://github.com/OpenZeppelin/openzeppelin-upgrades/issues/870#issuecomment-1716422933 for reasoning.


Remove unused & broken robot_sufix parameter

roslaunch tiago_moveit_config demo.launch was broken with
unused args [robot_sufix] for include of [/home/bence/moveit_ws/src/tiago_moveit_config/launch/fake_moveit_controller_manager.launch.xml]
and
unused args [robot_sufix] for include of [/home/bence/moveit_ws/src/tiago_moveit_config/launch/trajectory_execution.launch.xml]
I haven't touched robot_sufix in launch/tiago_moveit_controller_manager.launch.xml but I'd recommend sorting it's name out as it's inconsistent with the robot argument used in this and other packages.

@bmagyar I am afraid we cannot remove arg [robot_sufix] as it would break the compatibility with indigo-devel of TIAGo packages. I will check further the issue.


Can we drop "--service-cluster", "cluster0", "--service-node", "node0" for v2 config

I believe, but haven't checked yet that "--service-cluster", "cluster0", "--service-node", "node0" are optional with the gRPC api (they were part of the path params in the REST API). So maybe we can drop them and make the deployment manifests simpler.

Dropping the priority and removing the milestone for this bug. I think this might take some work with envoy upstream to remove these parameters, and at the end of the day, they're not such a weight to carry.

Closing as there is no obvious way to resolve this bug in the near term.


[Bug]: [Docs site] Links go to incorrect path

Which Remix packages are impacted?

[ ] remix (Remix core)

What version of Remix are you using?
n/a
Steps to Reproduce

Go to https://docs.remix.run (which redirects to docs.remix.run/v1)
Click any link in the body, eg. Quickstart, Jokes App Deep Dive, Conventional APIs

These links go to paths that don't exist eg. https://docs.remix.run/v1/v1/tutorials/blog
Expected Behavior
The link should either use absolute path with initial slash or have the v1 path removed.
Quickstart

Quickstart

Actual Behavior
Current docs pages use relative links.
Quickstart

Not applicable anymore


refactor(types): remove superfluous union types

Once you add "string" to a union type, any string literal values are ignored.
Here you see autocomplete of type Foo with "bar" and "foo". Cool.

But once you add | string, we no longer get autocomplete.

I moved the literal values into a JSDoc comment as I felt they still had merit, but removed them from the union type itself.

Shouldn't we remove the string type and keep the individual ones?
This will cause more maintenance, but more type-safety

we can use LiteralUnion from https://github.com/sindresorhus/type-fest/blob/main/source/literal-union.d.ts to keep both as well :)

@mcansh Seems even better imo!


Use externally hosted PDFs (Amazon S3, etc)

I recently ran into a version of this error (https://github.com/mozilla/pdf.js/issues/7153) when trying to use this gem to display PDFs hosted in an Amazon S3 bucket.  Originally thought I had a CORS issue but was able to solve it by overwriting /pdfjs_viewer/viewer.js and inserting my development and production urls into the HOSTED_VIEWER_ORIGINS around line 7100.
Would you be open to a pull request, either to add some documentation to the README, or to make HOSTED_VIEWER_ORIGINS dependent on an ENV variable?

@andyweiss1982 sure 👍

I too have a similar issue with S3 hosted pdf's throwing 403 Access denied errors, even after placing my appropriate URL's in the HOSTED_VIEWER_ORIGINS array.
My issue however is limited to Microsoft Edge browsers, and only Edge. I cannot replicate the issue in Chrome, Firefox, Safari etc.
Has anyone else experienced this? If so, have you a solution?
PDF.js v1.4.20 (build: b15f335)
Message: Unexpected server response (403) while retrieving PDF "https://s3.amazonaws.com/myrealbucket/myrealpdf.pdf".
My CORS policy is working for S3 hosted assets such as images, across all browsers, including Edge. This issue seems to be limited to PDF's streamed through pdf.js, via MS Edge only.
My Response headers on Chrome are as follows (works great):
Accept-Ranges:bytes
Access-Control-Allow-Credentials:true
Access-Control-Allow-Methods:GET, POST, HEAD
Access-Control-Allow-Origin:https://www.myrealdomain.com
Access-Control-Expose-Headers:Accept-Ranges, Content-Range, Content-Encoding, Content-Length
Cache-Control:max-age=31536000
Content-Length:65536
Content-Range:bytes<PHONE_NUMBER>79/706927
Content-Type:application/pdf
Date:Wed, 06 Jul 2016 20:41:14 GMT
ETag:"8612edf919de4c67678e33469673e5e7"
Expires:Fri, 12 May 2017 15:41:19 GMT
Last-Modified:Thu, 12 May 2016 15:41:20 GMT
Server:AmazonS3
Vary:Origin, Access-Control-Request-Headers, Access-Control-Request-Method
x-amz-id-2:LfLV8oJ/iS9UrOxbbyz/EfuB/r9LxC0S+1lgMG9fDJ4Ig3OsIkdAPPnJKh1RW00Py7T3a7BiZHk=
x-amz-request-id:6D9AF00B3BF4FE83
While my Response headers on MS Edge are:
Access-Control-Allow-Credentials: true
Access-Control-Allow-Methods: GET, POST, HEAD
Access-Control-Allow-Origin: https://www.pcrzone.com
Access-Control-Expose-Headers: Accept-Ranges, Content-Range, Content-Encoding, Content-Length
Content-Type: application/xml
Date: Wed, 06 Jul 2016 20:37:55 GMT
Server: AmazonS3
Transfer-Encoding: chunked
Vary: Origin, Access-Control-Request-Headers, Access-Control-Request-Method
x-amz-id-2: Dv77NpdsZ7ETCLfLZfCbYNlLtg19ZzQWB9UAhJzCoBRjJlXJq7wR5CHt0lyQRVxdQ4Tu62lB8tA=
x-amz-request-id: 367E32B636D46B6D
I notice some obvious differences between the two, especially Content-Type (but i suspect it defaults to xml due to 403 restriction).
Any advise would be greatly appreciated.
Thanks in advance!

Any news on this?

Another alternative to solve this is to use 'rack-reverse-proxy' gem. With this gem you can redirect your-domain.com/s3 -> to your-bucket.s3.aws.com. That could be safer that other alternatives.

'rack-reverse-proxy' gem is a good gem. However, since your file needs to send through your Rack Server, it can take pretty much time to do so.
Thus still would be good if we can configure the origin manually.

Hi @andyweiss1982 any update on this?


units: 2.21 -> 2.22

Description of changes
Application changelog:

Version 2.22 - 4 September 2022 (bug fix release)

Fixed bug: segfault on inverse conversions of tables in verbose moden
Added support for ExchangeRate-API in units_cur
Minor unit definition fixes and updates, including updates to
planetary data and a change in Rinfinity and hartree to increase accuracy.
Do not update currency.units at build

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Seems like this update has already been included?

https://github.com/NixOS/nixpkgs/pull/190261


Cached entities from the datastore is damaged

Summary
When create and delete service via kong's admin api frequently, the cached entities from the datastore is damaged.
Some logs as below:

upstream-global-tcnp-elasticsearch-client-f6375728-ea96-11e9-b77a-525400de400f-5969-elasticsearch.cc.oa.com is the host property of a service.  It is the name of an upstream, it should not be resolved by DNS.
Everything will be good after the kong restart.
Steps To Reproduce

Start kong
Create and delete service via kong's admin api frequently
After a while, the kong does not work.
Everything will be good after the kong restart

Additional Details & Logs

Kong version ($ kong version)
1.2.1

Kong debug-level startup logs ($ kong start --vv)
2019/11/06 09:40:22 [notice] 1#0: using the "epoll" event method
2019/11/06 09:40:22 [notice] 1#0: openresty/<IP_ADDRESS>
2019/11/06 09:40:22 [notice] 1#0: built by gcc 8.3.0 (Alpine 8.3.0)
2019/11/06 09:40:22 [notice] 1#0: OS: Linux 3.10.0-862.11.6.el7.x86_64
2019/11/06 09:40:22 [notice] 1#0: getrlimit(RLIMIT_NOFILE): 1048576:1048576
2019/11/06 09:40:22 [notice] 1#0: start worker processes

Operating system
CentOS 7.5

I have the same issue.

Hi @jeremyxu2010, @yaoice, sorry for the late reply. Could you confirm that if the issue still exists on the latest version (3.x)?


Update contract deployment to use safe singleton factory

Now that we have safe singleton factory (https://github.com/web3well/bls-wallet/issues/331#issuecomment-1308126246), update our contract deployment code to make use of this, allowing the contracts to be deployed to a deterministic address without relying on special keys.

This requires some care to pass constructor arguments. Just using the bytecode as I did https://github.com/voltrevo/arb1-exp-nov-2022/blob/a929341/scripts/create2FreeMoney.ts#L36 is correct if you don't have any constructor arguments.
I think the solution is to use ContractFactory.createInterface().encodeDeploy(...constructor args), but this isn't the full deployment code as you might expect. Instead, I think you need to append this result to the end of the bytecode.

Whilst this issue is worth completing, in the future this should not be required. Systems above addresses, like ENS and relevant CAIPs, should be able to provide a good ux for users to not make mistakes between chains.
https://forum.gnosis-safe.io/t/how-can-a-safe-hold-asset-on-multiple-chains/2242/11


[BUG] STT (whisper)

Hi, there! Using STT (whisper), I found a couple of bugs.  (I'm running the app on Visual Studio 2022 Debug)

Click on "Speech to Text to Speech" on/off several times will cause the Exception Error.

Closing the app (pressing "" icon) with "Speech to Text to Speech" Enabled, will not kill the app. There are dangling thread remaining preventing the app to exit.  You will see this behavior when the app is launched from Visual Studio Debug.

I think that's all for now and many thanks for the wonderful app.

I've applied a possible fix, I couldn't get the error to display for myself after a couple tries although it could still be there.

Fixed, recognizers are now stopped on form closing

https://github.com/VRCWizard/TTS-Voice-Wizard/releases/tag/v1.4.9

You are really wonderful. I'm so happy that I got to know this app and I'll try to help as much as I can to make this app better. I think the possibility of this app is limitless. I'm pretty sure you are thinking the same. ^^

Hi, here is the update.
1. Still seems to be the issue. Just to let you know. ^^
2.  has been fixed. Great!
Cheers!

Whisper can now NOT be stopped while it is launching to help avoid the AccessViolationException. If you can still reproduce the issue let me know the scenario.
https://github.com/VRCWizard/TTS-Voice-Wizard/releases/tag/v1.5.1

Sure, I'll.
Oh, there is compilation error at

missing, comboBoxProBranch.  ^^

Thanks. It compiles. 👍

Although it is still possible to cause error 1. to happen the likelihood has been drastically reduced by preventing users from stopping Whisper during it's startup so I'll be closing this issue.


[Tables] Adding azurite support

#17417

@v-xuto can you elaborate on how to best test these changes? Is there a way to spin up azurite in the azure devops? Are there any limitations to azurite or does it have the exact same functionality as the Tables service>

@v-xuto I tested locally and I get an error on the sample code you provided:
Error: *Internal Azurite Error*: content-type value application/json;odata=nometadata is not supported by Azurite.

@seankane-msft - I tested it locally based on the code you modified, and the result was successful. I use this repo: https://github.com/Azure/Azurite, switch to the branch table, start azurite. Below are the values of account_url and access_key that I used. The values of these variables are provided in the readme.
account_url = 'https://<IP_ADDRESS>:10002/devstoreaccount1'
access_key = 'Eby8vdM02xNOcqFlqUwJPLlmEtlCDXJ1OUzFT50uSRZ6IFsuFq2UVErCz4I6tq/K1SZFPTOtr/KBHBeksoGMGw=='

/azp run python - tables - ci


Clearer disable validation logic

In this PR I suggest to make clearer disable validation logic. It will be a flag which will be set in the pretrain_routine() so during the training the trainer will know if there will be validation runs or not. Now validation will be disabled if:

num_val_batches=0. It either means that there is no val_dataloader or that the user have set val_percent_check=0 (this part will work when #649 is merged).
validation_step was not overriden

Once this merged we will be able to finish #542.

@kuynzereb great job! sorry for the delay :)


UnicodeDecodeError in _scrape_response

There is this bit in the code:
            except SyntaxError as err:
                full_html = "%s" % response

It's trying to get the response into a str when the xml parsing fails.
But if that response contains unicode, that line will error.
The response it's trying to parse is:
'{"responseHeader":{"status":400,"QTime":281,"params":{"q":"*:*","facet.field":["facet_keywords_exact","grade","{!ex=category_exact}category_exact","hierarchy4"],"fl":"* score","start":"0","fq":["django_ct:(campsite.campsite)","{!geofilt pt=45.2065182704,0.826721191406 sfield=point d=25.749504}"],"f.facet_keywords_exact.facet.limit":"1000","rows":"1","facet":"on","wt":"json"}},"response":{"numFound":5,"start":0,"maxScore":1.0,"docs":[{"rating":"0","point":"45.4129972217,0.931974828309","grade":3,"text":"Camping le Repaire\\nCamping Le Repaire is in the beautiful region of P\xc3\xa9rigord, surrounded by trees, nature and rivers. Located only a short walk from the small village of Thiviers, this family-run campsite is ideal for a relaxing holiday with family and friends.\\r\\n\\r\\nOur facilities include:\\r\\n- two toilet/shower blocks\\r\\n- disabled access facilities\\r\\n- swimming pool\\r\\n- playground\\r\\n- bar and takeaway\\r\\n- laundry room\\r\\n\\r\\nPlay p\xc3\xa9tanque, play table football, ping pong or pool - or simply relax by the pool with an ice cream or an aperitif.\\r\\n\\r\\nOur friendly barbecues are not to be missed!\\nThiviers, Dordogne, Aquitaine, France\\nThiviers\\nDordogne\\nFrance\\nDordogne\\nfrance/aquitaine/dordogne/thiviers/camping-le-repaire\\ncamping-le-repaire\\n aa-pennant-none bar-nearby bar-or-club-house barbecues-allowed beach-excellent-water-quality beaches book-pitchup canoeingkayaking-nearby car-parking-by-pitch cycle-hire-nearby cycling-nearby disabled-facilities dogs-allowed dogs-allowed-all-year drainage-hook-up-points-for-tourers electrical-hook-up-points-for-tourers english family-friendly fishing free-wifi games-room gastronomic-delight hard-standings horse-riding-nearby is_balance_of_payment_taken_on_arrival jumbo-tent-pitches large-51-200-pitches launderette leisuretheme-park-nearby motorcycle-friendly motorhome-service-point nearby-farmers-market on-site-restaurantcafe outdoor-pool-nearby outdoor-swimming-pool peaceful play-area public-telephone rallies-welcome recycling-available restaurant-nearby shop-nearby shower-available take-away tennis-nearby toilet-block tv-room washing-up-area water-hook-up-points-for-tourers wifi \\nOur campsite is a few minutes from Thiviers, the capital of foie gras and truffles, so make sure you taste the exquisite local cuisine.\\r\\n\\r\\nA must visit is the Grottes de Villars (10km), a cave with 17,000-year-old prehistoric paintings, stalactites and other cave formations. \\r\\n\\r\\nKids will love the Chateau de Puyguilhem (11km), a sixteenth century castle listed as a historic monument by UNESCO.\\r\\n\\r\\nOr go to one of the many summer concerts in the ruins of Abbey de Boschaud, a twelfth century Cistercian monastery.\\n\\n    Barbecues allowed\\n\\n    Games room\\n\\n    Water hook-up points for tourers\\n\\n    On-site restaurant/cafe\\n\\n    Drainage hook-up points for tourers\\n\\n    Electrical hook-up points for tourers\\n\\n    Shower available\\n\\n    Cycling nearby\\n\\n    Nearby farmers' market\\n\\n    Fishing\\n\\n    Leisure/theme park nearby\\n\\n    Canoeing/kayaking nearby\\n\\n    Take away\\n\\n    Cycle hire nearby\\n\\n    Dogs allowed all year\\n\\n    Recycling available\\n\\n    Car parking by pitch\\n\\n    Launderette\\n\\n    Bar or club house\\n\\n    Public telephone\\n\\n    Horse riding nearby\\n\\n    Outdoor swimming pool\\n\\n    Play area\\n\\n    Toilet block\\n\\n    TV room\\n\\n    Washing-up area\\n\\n    Bar nearby\\n\\n    Outdoor pool nearby\\n\\n    Restaurant nearby\\n\\n    Shop nearby\\n\\n    Tennis nearby\\n\\n    Wifi\\n\\n    Hard standings\\n\\n    Motorhome service point\\n\\n    Disabled facilities\\n\\n    Dogs allowed\\n\\n    Family friendly\\n\\n    Jumbo tent pitches\\n\\n    Free wifi\\n\\n    Large (51-200 pitches)\\n\\n    Rallies welcome\\n\\n    Motorcycle friendly\\n\\n    Peaceful\\n\\n    Gastronomic delight\\n","has_primary_photo":true,"django_ct":"campsite.campsite","facilities":["barbecues-allowed","games-room","water-hook-up-points-for-tourers","on-site-restaurantcafe","drainage-hook-up-points-for-tourers","electrical-hook-up-points-for-tourers","shower-available","cycling-nearby","nearby-farmers-market","fishing","leisuretheme-park-nearby","canoeingkayaking-nearby","take-away","cycle-hire-nearby","dogs-allowed-all-year","recycling-available","car-parking-by-pitch","launderette","bar-or-club-house","public-telephone","horse-riding-nearby","outdoor-swimming-pool","play-area","toilet-block","tv-room","washing-up-area","bar-nearby","outdoor-pool-nearby","restaurant-nearby","shop-nearby","tennis-nearby","wifi","hard-standings","motorhome-service-point","disabled-facilities","dogs-allowed","family-friendly","jumbo-tent-pitches","free-wifi","large-51-200-pitches","rallies-welcome","motorcycle-friendly","peaceful","gastronomic-delight"],"lead_price_one_night":0.0,"hierarchy_tree_exact":["france/aquitaine/dordogne/thiviers/","france/aquitaine/dordogne/","france/aquitaine/","france/"],"id":"campsite.campsite.12273","bookable":true,"category":["tent-pitches","touring-pitches","motorhomes"],"django_id":"12273","content_autocomplete_text":"\\n{\\n    \\"value\\": \\"Camping le Repaire, Thiviers\\",\\n    \\"tokens\\": [\\"Thiviers\\", \\"france/aquitaine/dordogne/thiviers/camping\\\\u002Dle\\\\u002Drepaire\\"],\\n    \\"categories\\": [\\n                    {\\"sprite_class\\": \\"tents\\", \\"id\\": 4, \\"name\\": \\"Tent pitches\\"}\\n                  \\n                    ,{\\"sprite_class\\": \\"tourers\\", \\"id\\": 3, \\"name\\": \\"Touring pitches\\"}\\n                  \\n                    ,{\\"sprite_class\\": \\"motorhomes\\", \\"id\\": 10, \\"name\\": \\"Motorhomes\\"}\\n                  ],\\n    \\"name\\": \\"Camping le Repaire, Thiviers\\",\\n    \\"url\\": \\"france/aquitaine/dordogne/thiviers/camping\\\\u002Dle\\\\u002Drepaire\\",\\n    \\"bookable\\": true,\\n    \\"thumb\\": \\"https://media.pitchup.co.uk/images/2/image/upload/t_thumb_v2/v1383148631/camping\\\\u002Dle\\\\u002Drepaire/camping\\\\u002Dle\\\\u002Drepaire\\\\u002Dthe\\\\u002Dpond.jpg\\"\\n}","hierarchy0_exact":"france/","expected_value":6.488E-5,"hosted_online_booking":true,"category_ids":"4,3,10","campsite_id":12273,"parent_hierarchy_name":"Thiviers","content_autocomplete":"Camping le Repaire","rate_count":0,"hierarchy3_exact":"france/aquitaine/dordogne/thiviers/","name_sortable":"CampingleRepaire","available_pitches_tents":false,"category_exact":["tent-pitches","touring-pitches","motorhomes"],"available_pitches_lodges":false,"hierarchy1":"france/aquitaine/","enable_availability":true,"has_tagged_url":false,"available_to_search":true,"path":"france/aquitaine/dordogne/thiviers/camping-le-repaire","available_pitches_motorhomes":false,"hierarchy2_exact":"france/aquitaine/dordogne/","facet_keywords":["is_balance_of_payment_taken_on_arrival","barbecues-allowed","aa-pennant-none","nearby-farmers-market","beach-excellent-water-quality","fishing","recycling-available","outdoor-swimming-pool","tv-room","outdoor-pool-nearby","washing-up-area","dogs-allowed","car-parking-by-pitch","play-area","leisuretheme-park-nearby","public-telephone","on-site-restaurantcafe","peaceful","large-51-200-pitches","tennis-nearby","canoeingkayaking-nearby","cycle-hire-nearby","jumbo-tent-pitches","horse-riding-nearby","cycling-nearby","hard-standings","drainage-hook-up-points-for-tourers","take-away","free-wifi","shower-available","water-hook-up-points-for-tourers","dogs-allowed-all-year","gastronomic-delight","shop-nearby","bar-nearby","beaches","disabled-facilities","electrical-hook-up-points-for-tourers","motorhome-service-point","restaurant-nearby","wifi","bar-or-club-house","book-pitchup","rallies-welcome","family-friendly","launderette","motorcycle-friendly","english","games-room","toilet-block"],"facet_keywords_exact":["is_balance_of_payment_taken_on_arrival","barbecues-allowed","aa-pennant-none","nearby-farmers-market","beach-excellent-water-quality","fishing","recycling-available","outdoor-swimming-pool","tv-room","outdoor-pool-nearby","washing-up-area","dogs-allowed","car-parking-by-pitch","play-area","leisuretheme-park-nearby","public-telephone","on-site-restaurantcafe","peaceful","large-51-200-pitches","tennis-nearby","canoeingkayaking-nearby","cycle-hire-nearby","jumbo-tent-pitches","horse-riding-nearby","cycling-nearby","hard-standings","drainage-hook-up-points-for-tourers","take-away","free-wifi","shower-available","water-hook-up-points-for-tourers","dogs-allowed-all-year","gastronomic-delight","shop-nearby","bar-nearby","beaches","disabled-facilities","electrical-hook-up-points-for-tourers","motorhome-service-point","restaurant-nearby","wifi","bar-or-club-house","book-pitchup","rallies-welcome","family-friendly","launderette","motorcycle-friendly","english","games-room","toilet-block"],"name":"Camping le Repaire","hierarchy1_exact":"france/aquitaine/","available_pitches_caravans":false,"hierarchy_tree":["france/aquitaine/dordogne/thiviers/","france/aquitaine/dordogne/","france/aquitaine/","france/"],"available_pitches_rent_a_tent":false,"has_availability":true,"pitchtype_ids":[3915],"hierarchy0":"france/","hierarchy3":"france/aquitaine/dordogne/thiviers/","hierarchy2":"france/aquitaine/dordogne/","available_pitches_tourers":false,"last_booking_time":"2014-03-08T09:14:53.327Z","api_type":-1,"_version_":1489653176005033984,"score":1.0}]},"error":{"msg":"undefined field: \\"hierarchy4\\"","code":400}}\n'

See #123.


Fail to reboot with ARM/IAR

I build mbed-bootloader by:
mbed compile -m K64F -t ARM --profile tiny.json
Or:
mbed compile -m K64F -t IAR --profile tiny.json
At first boot, I can see log meessage:
[BOOT] mbed Bootloader
[BOOT] ARM:<PHONE_NUMBER>000000000000000000000000000000
[BOOT] OEM:<PHONE_NUMBER>000000000000000000000000000000
[BOOT] Layout: 0 9420
[BOOT] Active firmware integrity check:
[ERR ] Active firmware integrity check failed

But I see none after reboot.
If I flash attached K64F bootloader tools/mbed-bootloader-k64f-block_device-sotp-v3_3_0.bin in mbed-cloud-client-example (<IP_ADDRESS>), reboot is OK:
[BOOT] mbed Bootloader
[BOOT] ARM: 74CE36607E38CD63ECDD72CE2F1262D58BF55448
[BOOT] OEM:<PHONE_NUMBER>000000000000000000000000000000
[BOOT] Layout: 0 7188
[BOOT] Active firmware integrity check:
[ERR ] Active firmware integrity check failed
[BOOT] Slot 0 is empty
[ERR ] Active firmware invalid
[ERR ] Failed to jump to application!
[BOOT] mbed Bootloader
[BOOT] ARM: 74CE36607E38CD63ECDD72CE2F1262D58BF55448
[BOOT] OEM:<PHONE_NUMBER>000000000000000000000000000000
[BOOT] Layout: 0 7188
[BOOT] Active firmware integrity check:
[ERR ] Active firmware integrity check failed
[BOOT] Slot 0 is empty
[ERR ] Active firmware invalid
[ERR ] Failed to jump to application!

Target
K64F
ARM Toolchain
5.24
GCC_ARM Toolchain
GNU Tools for Arm Embedded Processors 7-2017-q4-major
IAR Toolchain
7.8
mbed-bootloader
v3.3.0
mbed-os
5.8.4

with ARMCC or IAR the binary size of the bootloader would be too large. But I cannot explain why you would fail to reboot as the bootloader should not be doing any writing.

@LiyouZhou Yes, with ARM/IAR, the binary size of bootloader exceeds 32KiB. I suggest adjusting mbed_app.json to make all GCC/ARM/IAR compiled binary workable. It is confusing that just GCC-compiled binary can work.

I'm closing this, as the issue is outdated.
We have made a choice to use only GCC for building our bootloaders.
Functionality should work on all compilers, but the size vary, so we only fine tune for one compiler, if required.


Start APK using this plugin

Can the plugin be used to start an external APK, or it can only run OS commands ?
Thanks for any info.
Mario

hi Mario,
I have not tried anything else than run OS commands.

Hi Peter, thanks for answering ! I supossed it.
Mario
El 20/02/2016 04:33, "Peter Vojtek"<EMAIL_ADDRESS>escribió:

hi Mario,
I have not tried anything else than run OS commands.
—
Reply to this email directly or view it on GitHub
https://github.com/petervojtek/cordova-plugin-shell-exec/issues/1#issuecomment-186536554
.

Hi Mazito
Please have a look here:
https://github.com/lampaa/com.lampa.startapp/issues/42
at the end of the page, you can found a working solution for your problem, actualized to the last version of lampaa.startapp plugin.
regards
MaX

Did you try using the pm command to start the package id?

http://stackoverflow.com/questions/4567904/how-to-start-an-application


Add more edge cases to the integration tests

Currently the tests in #53 test more for the "happy paths". More tests for edge cases should be added.

Done in #70


Add Required image-builder vSphere User Privs

Adds ContentLibrary.DownloadSession.
See mapping of UI to programmatic privileges here: https://gist.github.com/pljoel/98842cf88d78872bed891bba14a44ab8
Description of changes:
Adds missing privilege to image-builder.
User encountered error attempting to build ubuntu image without it.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm

/approve

/cherry-pick release-0.14
/cherry-pick release-0.15


Update init.py

This the v2 of our previous pull request, we merged the pull request because things became messy. This pull request is about 2 features
Favourite paths and
Find in file

So we will add the search in file in 3.2


lib: do not check nvm path if not default release

Currently the check for nvm mirror addresses will always happen,
even if the release in question is not default and provides it's own
url for finding headers.
This extra boolean check will ensure that non default releases will
use the embedded url for headers

it looks like there are some tests not passing here... I'm going to dig in and see what is going on tomorrow

If you're using NVM_* env vars then you're basically running in to https://github.com/nodejs/node-gyp/issues/787 which we can't really solve properly until node-gyp@4 where we can remove NVM_* entirely. Their introduction here was a mistake and needs to be reverted but we can't do it immediately because they are actually used and #878 is only just being rolled out now as an alternative.
The workaround is to not use NVM_* vars, or avoid nvm entirely for non-standard releases.

updated my nvm and all is right in the world again


Additional fields in /builders output needed by conductor.

https://www.aeolusproject.org/redmine/issues/2859
I'm not refactoring the obvious bits from the builders in the hopes we can just rip this out later when we stop acting as middleman for conductor/iwhd.

works as expected, thanks for this


Initial functionality

Pull requests:

https://github.com/Senzing/github-action-docker-buildx-build/pull/4
https://github.com/Senzing/github-action-docker-buildx-build/pull/5
https://github.com/Senzing/github-action-docker-buildx-build/pull/6


Should package search be case sensitive?

Most package names are all lower case, but some package names do have capital letters. Consider the package NetworkManager. At the moment making a request for networkmanager will not yield NetworkManager in the response. I propose that the package search should be case insensitive.

I like this idea, do you want to look into it?

Yes, I will look into it and submit a PR and we can evaluate from there!

One thing to keep in mind: ideally we would push the searching down to dnf eventually, rather than do it ourselves. So let's check that dnf supports whatever features we add, so that such a refactor would be transparent to the user.

I think performing case insensitive search may actually bring us closer to dnf's behavior:
https://dnf.readthedocs.io/en/latest/command_ref.html#search-command
Search Command
Command: search
Aliases: se

dnf [options] search [--all] <keywords>...Search package metadata for keywords. Keywords are matched as case-insensitive substrings, globbing is supported. By default lists packages that match all requested keys (AND operation). Keys are searched in package names and summaries. If the “–all” option is used, lists packages that match at least one of the keys (an OR operation). In addition the keys are searched in the package descriptions and URLs. The result is sorted from the most relevant results to the least.

Perfect! Thanks for checking :)


[docs] events page: lifecycle chart is not rendered

On Events section the mermaid chart is not rendered correctly.

Fixed, thank you for bringing this to my attention!


can not type on msdos from mobile devices

currently it doesn't support typing on msdos app on mobile devices.
any plan on this?
Btw great work 😍

Thanks for the feedback. Last night I upgraded to js-dos v7 which does have a virtual keyboard option. I haven't yet integrated it but this should be possible to do.

@DustinBrett Could there be a virtual controller for Doom and other games?

Yes since upgrading to js-dos v7 last night that is now very possible. I'll be adding it shortly I hope.

Closing v1 issues. Everything has been fully re-written.


add script for aws-opsworks-smooth-deploy

@petrabarus please review

rebase

lgtm. let me know when the commits squashed.


AkClipPlayer

A group of classes that utilize AVAudioPlayer to read multiple clips of audio from one or many audio files in a sequence.  Streams from file so memory footprint remains low for large files.  Sample accurate scheduling.

Another awesome addition to AudioKit, thank you!


PhoneNumber validator seems to be broken for at least country DE

I expected the validator to match the whole value given to isValid (or atleast a space-trimmed version of it), but it seems to also issue number parts as valid.
Using the following simple test program
require('vendor/autoload.php');

use Zend\I18n\Validator\PhoneNumber;

$phonenumbers = [
    '12345678 67890',
    '12345678a67890',
    'ab 12345678',
    '12345678cd',
    'ab12345678',
    'ab-12345678-cd',
];

$pv = new PhoneNumber([
    'allowed_types' => ['fixed', 'mobile'],
    'country' => 'DE'
]);

foreach ($phonenumbers as $pn) {
    var_dump($pv->isValid($pn));
}

I get with PHP 7.1
bool(true)
bool(true)
bool(true)
bool(true)
bool(true)
bool(true)

I'd be ok with the first one, but the others are just wrong IMHO.
The source of the problem seems to be preg_match's alternative (|) handling: Line-start (^) and line-end ($) matcher are handled for each alternative individually, thus in your regex strings the ^ and $ are only used for the first and the last alternative respectively.
Quoting the documentation: "Circumflex (^) need not be the first character of the pattern if a number of alternatives are involved, but it should be the first thing in each alternative in which it appears if the pattern is ever to match that branch. [..] Dollar ($) need not be the last character of the pattern if a number of alternatives are involved, but it should be the last item in any branch in which it appears. " [1].
From my point of view, there are 3 possibilities:

Add ^ and $ to each alternative
Use a non-capturing group like ^(?:a|b|c)$
It seems that \A and \Z match the beginning/end of the whole string, thus \Aa|b|c\Z should also work

Kind regards, Heiko
[1] http://php.net/manual/en/regexp.reference.anchors.php

@http
Sorry for the very late response. I will check this problem.
Thank you for reporting!


RUSTSEC-2021-0076 bump libsecp256k1

libsecp256k1 allows overflowing signatures
https://rustsec.org/advisories/RUSTSEC-2021-0076
Changes were made to conform to libsecp256k1 version differences.
Closes #9356

@sorpaas please check if that is the right way. I remember that you said something about some migration.

@sorpaas shouldn't we directly introduce the new host function? Host functions support versioning and we already use this versioning for a similar problem with schnorrkel.

@bkchr Yeah we should do it.

@trevor-crypto check this out https://github.com/paritytech/substrate/blob/master/primitives/io/src/lib.rs#L599-L600
@trevor-crypto We would need to do something similar here and use parse_overflowing_slice() in the version 1 and version 2 would use parse_standard_slice()

@bkchr thanks! Found my way there already by furiously grepping for 'host functions". I'm wondering how something like this will work with the TryFrom trait impl, since that is one place where parse_overflowing/standard_slice() needs to be used.

@trevor-crypto basically you need to revert https://github.com/paritytech/substrate/pull/9391/commits/c65328414a561180e16e44ff20d6de182f4a55ad because everything from now need to use the new function. Only the old host function needs to use the old function.

Added version 2 for ecdsa_verify. Please check carefully :pray:

@sorpaas please also check another time.

Rebased with master hoping that the tests would pass :confounded:

I wanted to jump in because I see that  https://github.com/paritytech/substrate/blob/7dcc77b982f59eaf6cec19499d981164b04a255d/primitives/io/src/lib.rs#L750 also uses secp256k1::Signature::parse_slice. Is there a reason why it does not need to get this change?

Thanks @girazoki
I guess I missed other places as well like bumping libp2p since libp2p-core uses libsecp256k1 too.

All traces of < v0.5.0 of libsecp256k1 have been removed. Once https://github.com/libp2p/rust-libp2p/pull/2151 is in a release, then everything should be on libsecp256k1 v0.6.0.

Nice! Is there a reason why sp_io::crypto::secp256k1_ecdsa_recover does not need to handle both parse_overflowing and parse_standard? Wouldnt it be the case that if we have a block/tx with a signature that overflows, if we just use the parse_standard would fail?

I think this assumes that we don't actually use secp256k1 outside of runtime. From what I know this seems to be the case.

I wanted to jump in because I see that
https://github.com/paritytech/substrate/blob/7dcc77b982f59eaf6cec19499d981164b04a255d/primitives/io/src/lib.rs#L750
also uses secp256k1::Signature::parse_slice. Is there a reason why it does not need to get this change?

Yeah this is right, this needs to be versioned as well.

@bkchr ok, so I will also add a v2 for secp256k1_ecdsa_recover and secp256k1_ecdsa_recover_compressed

Ty! And after a master merge we should be ready to merge this pr! Sorry for the delay!

Please check carefully! Thanks :grin: :pray:

@tomaka @kpp could please someone of you review the libp2p changes

@tomaka any why this test is failing:
---- discovery::tests::discovery_working stdout ----
thread 'discovery::tests::discovery_working' panicked at 'Unexpected event: NewListenAddr { listener_id: ListenerId(1), address: "/memory/9379582200911454507" }', client/network/src/discovery.rs:1036:37
stack backtrace:

New version of libp2p now reports all events, such as new local addresses, instead of only discovery-specific events. The test just needs to be adjusted to ignore anything that's not SwarmEvent::Behaviour instead of panicking.

The test just needs to be adjusted to ignore anything that's not SwarmEvent::Behaviour instead of panicking.

@tomaka done, thanks!

@sorpaas yeah, that is true. I didn't think it would conflict, since the RUSTSEC states that >=0.5.0 is fine.
It seems that this is already bumped on libp2p master. Maybe someone can push a release? Then after that I can bump the libp2p version

I don't see any reason to block this pr because some libp2p stuff isn't updated yet.

The merge conflict is just because of https://github.com/paritytech/substrate/pull/9490
You can solve it by replacing the file with your code.

I didn't mention this, but we don't make any use of the secp256k1 capabilities of libp2p. I don't see any reason to not merge this PR.

@trevor-crypto sorry :see_no_evil: Please merge master again and then I will merge this! Sorry for the problems

Rebased with master again. Had a lot of lock file conflicts, so hopefully everything worked out :grimacing:

bot merge

@bkchr Thanks for applying those fixes!

bot merge force


We are unable to login

Expected content
We are unable to login , error" Your IP address is not whitelisted. See http://send.gd/1PemvHz for more details."
Actual content
1 We are unable to login , error" Your IP address is not whitelisted. See http://send.gd/1PemvHz for more details."
2. We have reset the password still same issue
3. Supporting team not responding, all business emails not delivered
4. With out able to login, we can not reach support team, but we are unable to login from last 24 hours. Emails not delivered form last 30 hours too poor service from SendGrid
Link to page:
https://app.sendgrid.com/login

We have facing problem since Yesterday, We have raised ticket , not good response from SengGrid supporting Team,  very poor service, We do not recommend SendGrid We will look into good provider for "sending emails"
Thank you!

Hi @ganapathinuvem,
Can you please provide a support ticket number so I can try and follow up for you?

they have respond after 3 days, very poor service from them. We never recommand SendGrid
support.sendgrid.com/hc/requests/5280238
support.sendgrid.com/hc/requests/5276698
support.sendgrid.com/hc/requests/5276644
support.sendgrid.com/hc/requests/5273749
support.sendgrid.com/hc/requests/5276464
support.sendgrid.com/hc/requests/5276642
support.sendgrid.com/hc/requests/5277616


Build failed on FreeBSD/i386: error[E0432]: unresolved import wasmtime_asm_macros::asm_sym

Steps to Reproduce

Run build of devel/libwasmtime on FreeBSD 13.2/i386

Expected Results
A green/successful build is expected.
Actual Results
...
     Running `CARGO=/usr/local/bin/cargo CARGO_CRATE_NAME=wasmtime_fiber
+CARGO_MANIFEST_DIR=/wrkdirs/usr/ports/devel/libwasmtime/work/wasmtime-11.0.1/crates/fiber CARGO_PKG_AUTHORS='The Wasmtime Project
+Developers' CARGO_PKG_DESCRIPTION='Fiber support for Wasmtime' CARGO_PKG_HOMEPAGE='' CARGO_PKG_LICENSE='Apache-2.0 WITH
+LLVM-exception' CARGO_PKG_LICENSE_FILE='' CARGO_PKG_NAME=wasmtime-fiber CARGO_PKG_README=''
+CARGO_PKG_REPOSITORY='https://github.com/bytecodealliance/wasmtime' CARGO_PKG_RUST_VERSION='' CARGO_PKG_VERSION=11.0.1
+CARGO_PKG_VERSION_MAJOR=11 CARGO_PKG_VERSION_MINOR=0 CARGO_PKG_VERSION_PATCH=1 CARGO_PKG_VERSION_PRE=''
+LD_LIBRARY_PATH='/wrkdirs/usr/ports/devel/libwasmtime/work/target/release/deps:/usr/local/lib'
+OUT_DIR=/wrkdirs/usr/ports/devel/libwasmtime/work/target/i686-unknown-freebsd/release/build/wasmtime-fiber-1dbb6519007f3564/out
+/usr/local/bin/rustc --crate-name wasmtime_fiber --edition=2021 crates/fiber/src/lib.rs --error-format=json --json=diagnostic-re
ndered-ansi,artifacts,future-incompat --crate-type lib --emit=dep-info,metadata,link -C opt-level=3 -C panic=abort -C
+linker-plugin-lto -C codegen-units=1 -C metadata=06080e4acd1f33a8 -C extra-filename=-06080e4acd1f33a8 --out-dir
+/wrkdirs/usr/ports/devel/libwasmtime/work/target/i686-unknown-freebsd/release/deps --target i686-unknown-freebsd -C linker=cc -L
+dependency=/wrkdirs/usr/ports/devel/libwasmtime/work/target/i686-unknown-freebsd/release/deps -L
+dependency=/wrkdirs/usr/ports/devel/libwasmtime/work/target/release/deps --extern
+cfg_if=/wrkdirs/usr/ports/devel/libwasmtime/work/target/i686-unknown-freebsd/release/deps/libcfg_if-ec28bb3e84a99470.rmeta --extern
+rustix=/wrkdirs/usr/ports/devel/libwasmtime/work/target/i686-unknown-freebsd/release/deps/librustix-a6b2c0bec45f8f8c.rmeta --extern
+wasmtime_asm_macros=/wrkdirs/usr/ports/devel/libwasmtime/work/target/i686-unknown-freebsd/release/deps/libwasmtime_asm_macros-b681268
+2091809a9.rmeta -C link-arg=-fstack-protector-strong`
error[E0432]: unresolved import `wasmtime_asm_macros::asm_sym`
  --> crates/fiber/src/unix/x86.rs:13:37
   |
13 | use wasmtime_asm_macros::{asm_func, asm_sym};
   |                                     ^^^^^^^ no `asm_sym` in the root

For more information about this error, try `rustc --explain E0432`.
error: could not compile `wasmtime-fiber` (lib) due to previous error

Caused by:
  process didn't exit successfully: `CARGO=/usr/local/bin/cargo CARGO_CRATE_NAME=wasmtime_fiber

Versions and Environment
Wasmtime version or commit: 11.0.1
Operating system: FreeBSD 13.2
Architecture: i386
Extra Info
Here's the full build log
https://pkg-status.freebsd.org/beefy15/data/132i386-default/f68709c1b069/logs/libwasmtime-11.0.1.log

Unfortunately i386 is not supported (#1980); so this is an expected result. It would be great to have, but no one has had the time to contribute it -- sorry!
Closing in favor of that tracking issue.


Provide @2x and @3x assets

Cause people have "retina" screens

Migrated to https://github.com/pennlabs/penn-mobile-ios/issues/103


Cashu: add LNURLp

Codecov Report

Merging #1484 (a5c3954) into main (9467a8d) will decrease coverage by 0.14%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main    #1484      +/-   ##
==========================================
- Coverage   47.85%   47.71%   -0.14%
==========================================
  Files         264      264
  Lines       16158    16158
==========================================
- Hits         7732     7710      -22
- Misses       8426     8448      +22

Impacted Files
Coverage Δ

lnbits/extensions/lnurlp/migrations.py
81.48% <0.00%> (-14.82%)
:arrow_down:

lnbits/db.py
81.20% <0.00%> (-10.07%)
:arrow_down:

lnbits/utils/exchange_rates.py
86.44% <0.00%> (-3.39%)
:arrow_down:

lnbits/commands.py
61.40% <0.00%> (-1.76%)
:arrow_down:

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