Add unit testing of fdlibm

The project is relying on a variant of fdlibm, which has aleady
been edited but never verified for correctness. This patch adds
unit testing of fdlibm by:

introducing a test generator that uses a trusted libm
implementation to calculate correct and expected results of math
functions (tools/gen-test-fdlibm.c),
adding tests created with the generator that stress all publicly
exported functions of jerry's fdlibm
(tests/unit/test-fdlibm.inc.h), and
adding a unit test file to drive the generated tests
(tests/unit/test-fdlibm.c).

Note: The test generator is not expected to be executed often, thus
it is not wired into the build system. If it gets edited, it must
be built locally and must be used to re-generate the .inc.h file.
During development, it turned out that tests/unit/test-common.h
included the system header math.h, which was only a bad smell until
now but became a real header conflict issue with the introduction
of the fdlibm unit test. Thus, this patch also changes the include
to fdlibm-math.h.
JerryScript-DCO-1.0-Signed-off-by: Akos Kiss<EMAIL_ADDRESS>

Do we really need to add the source code of the generator? There is no make target for it and I think it will be never used again. Can't just simply add generation macros for the test? BTW, the generator should be under tools/unit-test folder and we should add a generate-fdlibm-unit-tests.sh to tools

In general I'd like to see the usage of JERRY_ASSERT like in other unit test files and avoid the usage of printf.

@LaszloLango

Do we really need to add the source code of the generator? [...] I think it will be never used again.

The source code generator might be needed in the future if

an error is found in fdlibm (e.g., in some corner case) and we'd like to add a regression test for it not to allow creep it back next time, or
we'd simply like to increase the branch coverage of the existing functions with new test cases, or
our version of fdlibm is extended to contain more math functions (note that the version we have is a heavily trimmed down one of the original) and we'd like to cover them with tests as well.

There is no make target for it

That was intentional. We shouldn't rebuild the .inc.h. file all the time, only when something changes in the .c. And we certainly shouldn't build the .c with the toolchain we use for building jerry, since jerry might be cross-built but this file is always to be built for and executed on the local developer machine. If someone has knowledge on how to achieve that within the same build system as we use for jerry, do share, please.

Can't just simply add generation macros for the test?

I cannot think of a working approach that way. We'd need to link a trusted libm and fdlibm to the same unit test to have fdlibm's math functions and that libm's math functions called and their results compared.

BTW, the generator should be under tools/unit-test folder

Then, it'd be built with the rest of the unit tests. CMakeLists has file(GLOB SOURCE_UNIT_TEST_MAIN_MODULES tests/unit/*.c). Tried and errored for me. A possible resolution might have been to change the GLOB in CMakeLists, but then I've put it under tools anyway, since other generators reside there as well (e.g., generator.sh and print-unicode-ranges.sh).
(BTW, will print-unicode-ranges.sh be re-executed ever? Considering that it's output has already been manually edited, according to lit-unicode-ranges.inc.h.)

In general I'd like to see the usage of JERRY_ASSERT like in other unit test files

The problem with the assert approach is that it exits on the very first failure and we will have no clue whether that's the only problem with the implementation or there are many more. I've tries that and it's very annoying. By returning a status code at the end, we can see all the failures that we have (at least what's revealed by the test suite).

avoid the usage of printf.

Why is that? test-api.c also uses printf. Seeing the difference (or match) between the computed and expected results is a huge help in debugging. See my reply above to @galpeter . (I'm not completely strong about this, but it's good to debug without the need to edit the test code and only have to focus on the code that has the issue.)

Then, it'd be built with the rest of the unit tests. CMakeLists has file(GLOB SOURCE_UNIT_TEST_MAIN_MODULES tests/unit/*.c).
That is not what I suggested. I suggested tools/unit-test not tests/unit

Why is that?
Consistency. If we mix up everything, then the project would be hard to maintain at some point. Even if we talk about unit tests. I don't like JERRY_ASSERT, but in that case you should provide a better solution for the existing tests first.

@LaszloLango

Consistency. If we mix up everything, then the project would be hard to maintain at some point. Even if we talk about unit tests. I don't like JERRY_ASSERT, but in that case you should provide a better solution for the existing tests first.

The proposed test is not the first to use exit status code to signal error. TEST_INIT in test-common.h exits main with return 1; if something is wrong with /dev/urandom. All but one unit tests (test-date-helpers.c) use it. And test-string-number.c uses exit status code only to signal pass/fail, doesn't use JERRY_ASSERT at all. The proposed test is not the first to use printf either: test-api.c prints quite some info during execution.
So, all the things the proposed test does, are already being done by other tests. I don't see how this breaks consistency. (If someone thinks that unit tests should be unified somehow to be more consistent, then it can be done before this patch or after this patch, but unification and fdlibm testing don't seem to be related to me.)

@galpeter @LaszloLango I've updated the patch. I have to admit, it addresses the previous comments only partially:

the generator code (gen-test-fdlibm.c) has been moved out from tools and a shell script (gen-test-fdlibm.sh) is added to tools to build and run the generator and put its output to the right place (test-fdlibm.inc.h) -- this indeed makes its use easier,
the generated file does mention now how it was generated.

What went against the comments:

now the generator resides in tests/unit: this way it is checked by vera and cppcheck and is also closer to its generated output (might be easier to relate them) -- this needed a slight change in CMakeLists,
I haven't removed printfs and haven't added asserts (yet?) -- heated conversation about those is still ongoing.

How does it look now?

Thanks for the update, I'd still prefer tools/unit-test/gen-test-fdlibm.c

@LaszloLango updated (couldn't resist to tweak your request a bit though: the new directory is called tools/unt-tests, as plural form is used everywhere else)

@akiss77, thanks. It looks much better.

That was intentional. We shouldn't rebuild the .inc.h. file all the time, only when something changes in the .c. And we certainly shouldn't build the .c with the toolchain we use for building jerry, since jerry might be cross-built but this file is always to be built for and executed on the local developer machine. If someone has knowledge on how to achieve that within the same build system as we use for jerry, do share, please.

Now we have a bash script that can build and generate tests/unit/test-fdlibm.inc.h. What if we call the script in test-unit target of Makefile? I think that will always call the system gcc, but may have problems if there is no gcc. :(

LGTM

@LaszloLango While waiting for a second approval, I've been thinking about your last comment and might have come up with a variant that addresses some issues you've raised. I've added a Makefile to tools/unit-tests, which builds and cleans the generator, and made the bash script in tools call make (not gcc directly, as before).
Moreover, the Makefile uses standard variables to specify the compiler and the reference math lib (CC=gcc and LDFLAGS=-lm), so should a project maintainer ever need to work on a machine that does not have gcc and/or use something different from libm, then the override of Makefile's defaults will be as easy as setting the environment variables with the same name.
The PR is now updated and implements the above approach.

LGTM

@akiss77, still LGTM.


Form Builder compatibility bugs on Safari

Description from QA Testing:

Side menu appears and then disappears.

It appears when I select the text box not the drop down menu. It doesn’t work.

Can’t change order of the questions or duplicate or add more or remove.

Tested an confirmed there is an issue.

It appears when I select the text box not the drop down menu. It doesn’t work.

Drop menu is going away so that will be out of the picture
Workaround for now is to slightly shrink the browser window so you end up with the non-floating menu


SDB: In Soviet Russia, C has booleans ##refactor

Detailed description
Change arguments type from int to bool. Those are internal static function, so no other code could possibly be affected. There are booleans all over the place in that file, so we are not losing any portability.
PS Editing raw *.patches to match new paths is a surprisingly valid technique.
Test plan
None
Closing issues
Originally (mistakenly) posted at: https://github.com/radareorg/radare2/pull/18120

thanks!


Certain LFGs not being picked up

For example:  "need tank WC" not being added to LFG window.

Right now that is intended, before the actual release of classic I'm going to expand parsing the chat messages. Ill post an update here once that is done!

The chat parsing is now a bit more lenient, but now LFG messages might pop up inbetween now like "tank looking for wc group" is going to show up, however messages like "need tank for wc" will show up now. The update will be released with the next update tonight.


Could you release the pretrained model?

Ok, I will release them at this weekend.
ZhuojieChen @.***>于2021年8月5日 周四下午9:13写道：

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cxzhou95/XLSR/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIHI4LZFTXBZM6INJR5JPSDT3KFAXANCNFSM5BT2YLXQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

Thank you

@JackyChan97  I have uploaded the pre-trained model, please take a look. The performance (PSNR) of my implementation (x3 SR) on the DIV2K validation dataset is 30.092dB (30.10dB reported in the paper).
The performance of the quantized model (x3 SR) on the DIV2K validation dataset is 29.71dB (29.82dB reported in the paper). And I think this performance could be improved after I try different learning rates and training epochs.
However, I found the inference speed of XLSR's quantized model is slower than its float model because of pytorch framework (take too much time for the convolution when the number of channel is small, e.g., 8, 16, 32).

Thank you so much, I will have a try.


Improve test times by removing "-race"

Improve test times by removing the -race condition on running the unit
tests

Please see my comment here https://github.com/kubernetes-incubator/kompose/issues/247#issuecomment-257672539
The race detector isn't your issue, its the fact the unit test script doesn't cache builds combined with kompose having a huge number of dependencies.


ViewPagerAndroid has been extracted from react-native core

Reopening #682
Current behaviour
Getting warning that ViewPagerAndroid has been extracted from react-native core and will be removed in a future release.
Expected behaviour
React-native-tab-view to import the ViewPager from the community module instead of rn-core module.
Your Environment

software
version

ios or android

react-native
0.59.4

react-native-tab-view
2.0.3

react-native-gesture-handler
1

react-native-reanimated
1

node
11.8.0

npm or yarn
6.9.0

The 2.0.3 version doesn't use ViewPagerAndroid. try resetting metro cache with react-native start --reset-cache


Need a more defined story for TS + ASP.NET Core 1 (was ASP.NET 5)

We are receiving a lot of questions about how to use ASP.NET Core 1 (formerly ASP.NET 5) and TypeScript.
Specifically @toresenneseth posted an issue "What is the Typescript story with ASP.NET 5" where he asks

What is the Typescript story with ASP.NET 5? At the moment (VS 2015 CTP6) it doesn't seem to work very well at all. Since it's still pre-release software I don't expect everything to work smoothly, but I need to know if you're planning on making them work well together for the final release or not.

He is not the only one with questions about how TS should be used with ASP.NET Core 1. We have had several issues logged in the tooling repo.
I think that some users are going down the MSBuild route, and others gulp/grunt but it seems like both sets of users are running into issues and there are no resources for them to unblock themselves. We need a more well defined usage of TS + ASP.NET Core 1. We may just have a documentation gap here which could be easily fixed. If we need to make updates to the ASP.NET Core 1 project system in Visual Studio please let me know how we can help.
cc @shanselman @DamianEdwards @danroth27

We have the wiki page at https://github.com/Microsoft/TypeScript/wiki/Using-TypeScript-With-ASP.NET-5 to cover this specific topic, but it's not super discoverable and could do with some clean-up. I'll see if there might be a better location with our site redesign (cc @DanielRosenwasser who is working on docs).
p.s. I notice the issue in the ASP.NET forum you link to for you "specifically..." example is nearly a year old (the questions were posted in Mar 2015), which was before the wiki page was created. Do you have more recent examples of continuing questions here we can reference?

@danroth27 @sayedihashimi I'll look into updating our walkthrough docs to go from "File / New Project" to working TypeScript & ASP.NET5 project (probably using Angular2 as the framework, as that's also a common request). What is the starter template/configuration you think this would be best based off? (Feel free to ping me internally if easier for links or attachments, etc., my alias is billti).

@billti It would be good show folks how to setup typescript in the default web app project. If you want to show Angular as well it might make more sense to start from Empty.

Hi, I finally fixed all the issues updating to the latest bits of angular 2 and aurelia, and if it can help I can I can upload simple projects using these frameworks working and exchanging data back and forth with a web api controller

@lucamorelli, @danroth27, and @sayedihashimi can you take a look at https://github.com/Microsoft/TypeScript-Handbook/pull/203

#203 is merged now.


Add resolve() for getting local paths

This PR adds a method resolve() which turns a URI into a local path - if possible. It's intended to support https://github.com/ros/sdformat_urdf/pull/1

Orthogonal, but since we are looking at it here, can we rename retriever.h to retriever.hpp, and deprecate #include <retriever.h>? It's OK to do this as a follow-up too, but please open an issue in that case.

Did in a separate PR https://github.com/ros/resource_retriever/pull/51

Addressing the PR feedback brought up a bunch of corner cases, enough that it seems like it would make sense to add a dependency on a URI parser. Since I no longer need this in ros/sdformat_urdf#1 I'm going to close it instead.


[FEAT] Footnote syntax for cloze hints and Extra (back of card) fields

Is your feature request related to a problem? Please describe.
We can't specify cloze hints or Extra fields (for "back of card" info). Adding this info could be very verbose and clutter up our notes. we'd like to make this clean.
Describe the solution you'd like
footnotes can provide a succinct way to define the Cloze Extra field, a.k.a. "the back of the card".
Obviously, Hint is a per-cloze thing. Extra, however, is typically a per-note thing, i.e. since we're using a single note to generate N cloze-variations of a question about the note, we can generalize the Extra (back of the card) to explain the entire note as a whole in more detail. I call this out because Extra could be defined in a shared footnote body, down where it's not cluttering up your notes. Note that this works because multiple footnote references can refer to the same footnote body.
Re: cloze numbering with footnotes, since markdown footnote identifiers can be any alphanumeric sequence, Generally speaking, the design should not require the user to use any specific numbering or naming convention...
...except in the case of trying to define each cloze's hint in the footnote id, which I think is a good idea. :-) Note, however, that  cloze hints may commonly be duplicated in multiple cloze notes. So, prefixes or suffixes might need to be used to make their corresponding footnote ids unique. Or, as @MotaOcimar suggested, perhaps we use separate footnotes for hints and extra. Perhaps the convention is simply:

Numeric Footnote ids are used for Extra (back of the card).
Non-Numeric Footnote ids are used for hints.

Clozes that want to define a hint would use [^hint]; cloze notes that want to define back of card information must define at least one numeric footnote id [^1] and body. (If this results in implementation/parsing difficulties (since each cloze becomes its own distinct card, and perhaps card siblings might not be able to find each other's Extra field easily/efficiently), I suppose it would be acceptable to require each cloze to have the same Extra footnote.) There may be multiple footnotes on each cloze but as has been shown these are minimally visible in both edit and preview modes, which is the whole point of using footnotes.
Note that in this scheme the hint footnotes don't really need a body, since their information is conveyed through their id. However, at least on GitHub, they don't render unless there is a complete footnote reference/body pair. In the example below, I include them with empty bodies so they'll render.
Note that the footnote syntax described above doesn't need to be restricted to highlight-clozes only. In theory, you should be able to define cloze hints and Extra/card backs using highlight and/or Anki {{cloze}} syntax to define the clozes themselves. I vastly prefer the highlight form bc it sticks to standard md syntax, so I'll show that as an example below.
The ==rain==[^type_of_weather][^1] in ==Spain==[^country] falls mostly in the ==plains==[^topography].
which renders like this (sorry GitHub md doesn't seem to support highlights):
The ==rain==[^type_of_weather][^1] in ==Spain==[^country] falls mostly in the ==plains==[^topography].
The footnote bodies would look like this in markdown/edit mode:
[^1]: [Spain](https://en.wikipedia.org/wiki/Spain) is a country in southwestern [Europe](https://en.wikipedia.org/wiki/Europe) with parts of territory in the [Atlantic Ocean](https://en.wikipedia.org/wiki/Atlantic_Ocean) and across the [Mediterranean Sea](https://en.wikipedia.org/wiki/Mediterranean_Sea). Here is some [more information on Rain](https://en.wikipedia.org/wiki/Rain). Also note that the [Rain In Spain Doesn't _Always_ Stay On The Plain](https://spainguides.com/weather/rain-spain/).
[^type_of_weather]:
[^country]:
[^topography]:

(See bottom of comment for rendered footnotes)
[^1]: Spain is a country in southwestern Europe with parts of territory in the Atlantic Ocean and across the Mediterranean Sea. Here is some more information on Rain. Also note that the Rain In Spain Doesn't Always Stay On The Plain.
[^type_of_weather]:
[^country]:
[^topography]:

I have the same request! Hints can be put in brackets, sure, but I fail to see a way to implement the Extra field for clozes in the current version.
Alternatively one may use the bullet points for Extras: write the Extras field inside the toggle. For this to function, though, the card should fold/collapse everything togglable when showing the front of the card EXCEPT the one(s) that the actual cloze is in, and that is currently not supported.

Hi @timblaktu,  @MotaOcimar has released a PR #943 that I'm currently reviewing. The scope of his PR covers some of the functionality you described here, so I'm wanting to clarify some of the points you made.
Alternative Functionality
Note that in #943, there is "Generalized Cloze Overlapping" functionality (I think we need a better name for this) implementing issue #426. If I understand correctly, then this would provide the ability to finely tune what is shown on the back of the card. It doesn't use your suggested footnote notation, but the back of the card info can at least be placed on a separate line.
Additionally there have been a few people requesting an easy way to show the underlying note when reviewing a flash card, e.g. #786. This would simplify writing questions, because there would be no need to write specific "back of the card" info. Perhaps not as good when reviewing cards.
Between functionality covered by #426 and #786, would that obviate the need for your idea of the footnote syntax extra field?
Terminology

Obviously, Hint is a per-cloze thing. Extra, however, is typically a per-note thing, i.e. since we're using a single note to generate N cloze-variations of a question about the note

To make sure we are on the same page, regarding the term "note" I think it should be "question". The plug-in uses the terminology "note" to mean the entire markdown file.
A note can contain zero or more "questions" (terminology in code, not sure about user documentation).
And a question generates 1 or more "cards". In terms of cloze type questions, current functionality creates one card for each cloze in the question.
Extra -  Definition

footnotes can provide a succinct way to define the Cloze Extra field, a.k.a. "the back of the card".

Do you mean that the back of the card should be only the cloze extra field? Or does "extra" mean in addition to what would normally be shown on the back of a cloze type question?

Hi,
I've just made a release v1.13.0 that ships with custom cloze patterns 🥳 all thanks to #943. The updated documentation is live here.
I've tested the release but not as extensively as I normally would. Perhaps we need a beta version of the plugin? That being said, I'm on standby to fix bugs ASAP. Just give me a shout.
Cheers.


Fug: is code red or black?

core defines inline code `code` as black
https://github.com/db-ui/core/blob/3619b722292621d84989a8fa8bf2cecdc22a59ae/source/_patterns/00-base/_init.global.scss#L64
but DB UI elements shows it mostly "color: inherit" style
don't know where it comes from https://github.com/search?q=repo%3Adb-ui%2Felements Consolas&type=code
what is the correct one and is it a Feature in core and a Bug in Elements or a Bug in Core and Feature in Elements?
regards
~Marcel

No, i can`t find any declarative section for it neither on core nor on elements (nor on marketing portal).
I just opened the playground apps and hacked a code block in :)

@childnode I found that declaration, it's in there quite a while now. As code isn't defined in the standard, but we've only defined it within the technical implementation, it's not necessarily something that we've ever talked about with the creative colleagues. So if there's a problem that you've identified with the red color or the inconsistency, that you've reported, we could have a look at it.


feat: Homebrew Installation

Proposed Change
Setup goreleaser to create homebrew formula.
This is blocked until this repo becomes public as it requires public access to the download URL and creation of the not yet created hombrew-observiq-collector repo.
Sample install output:
 ~ > brew install observiq/observiq-otel-collector/observiq-otel-collector
==> Downloading https://github.com/observIQ/observiq-otel-collector/releases/download/v0.4.1/observiq-otel-collector-v0.4.1-darwin-arm64.tar.gz
Already downloaded: /Users/cphelps/Library/Caches/Homebrew/downloads/6112799930ee4e1efbe026712fb69657be44ec306c193bd6026549b1aaab28bf--observiq-otel-collector-v0.4.1-darwin-arm64.tar.gz
==> Installing observiq-otel-collector from observiq/observiq-otel-collector
==> Caveats
****************************************************************
After installation please run the following to finish setup:

  # Default location for the opentelemetry-java-contrib-jmx-metrics.jar
  sudo cp /opt/homebrew/opt/observiq-otel-collector/lib/opentelemetry-java-contrib-jmx-metrics.jar /opt/

The configuration file that is run by the service is located at /opt/homebrew/opt/observiq-otel-collector/config.yaml.

If you are configuring a logreceiver the plugin directory is located at /opt/homebrew/opt/observiq-otel-collector/plugins.
****************************************************************

****************************************************************
Below are services commands to run to manage the collector service.
If you wish to run the collector at boot prefix these commands with sudo
otherwise the service will run at login.

To start the collector service run:

  brew services start observiq/observiq-otel-collector/observiq-otel-collector

To stop the collector service run:

  brew services stop observiq/observiq-otel-collector/observiq-otel-collector

To restart the collector service run:

  brew services restart observiq/observiq-otel-collector/observiq-otel-collector
****************************************************************

****************************************************************
To uninstall the collector and its dependencies run the following commands:

    brew services stop observiq/observiq-otel-collector/observiq-otel-collector
    brew uninstall observiq/observiq-otel-collector/observiq-otel-collector
    launchctl remove com.observiq.collector

    # If you moved the opentelemetry-java-contrib-jmx-metrics.jar
    sudo rm /opt/opentelemetry-java-contrib-jmx-metrics.jar

****************************************************************

To start observiq/observiq-otel-collector/observiq-otel-collector now and restart at login:
  brew services start observiq/observiq-otel-collector/observiq-otel-collector
==> Summary
🍺  /opt/homebrew/Cellar/observiq-otel-collector/0.4.1: 82 files, 157.8MB, built in 1 second
==> Running `brew cleanup observiq-otel-collector`...
Disable this behaviour by setting HOMEBREW_NO_INSTALL_CLEANUP.
Hide these hints with HOMEBREW_NO_ENV_HINTS (see `man brew`).

Checklist

[x] Changes are tested
[ ] CI has passed

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

@antonblock

Not familiar with the requirements for Homebrew+git, but is it possible to only have 1 observiq-otel-collector in the name?

I'm not sure to be honest. When I tapped the public repo that's what it listed as the formula even though the formula had a single name.

@cpheps
All I could find was this, which only said the homebrew- suffix can be left off, nothing about the full name


endPoint.play does not return if call end by RemoteParty

we have a sample conneting to a local PhonerLite.
AsyncMain(process.mainModule == module, async () => {
    const srf = new Srf();
    srf.connect({host: '<IP_ADDRESS>', port: 9022, secret: 'cymru'});
    let hostport = await waitForEvent<string>(srf, 'connect');
    console.log('Connected ', hostport);
    const mrf = new Mrf(srf);
    let mediaserver = await mrf.connect({listenAddress: '<IP_ADDRESS>', address: '<IP_ADDRESS>', port: 8021, secret: 'ClueCon'});
    console.log('MediaServer Available');
    let endPoint = await mediaserver.createEndpoint();
    console.log('Endpoint created',endPoint.local.sdp);
    let dialog = await srf.createUAC('sip:ahr@<IP_ADDRESS>:5070', {
        localSdp: endPoint.local.sdp,
    }, {
        cbRequest: (err,req, res) => {
            console.log('CbReq...', err, );
        },
        cbProvisional: (res) => {
            console.log('ProvisionalResp...',res.status+' '+res.reason)
        }
    });
    await endPoint.dialog.modify(dialog.remote.sdp);
    dialog.on('destroy', (msg, reason) => {
        endPoint.destroy();
        console.log('got bye', reason);
    })
    console.log('Established');
    await waitFor(1000);
    await endPoint.play(['silence_stream://1000', 'voicemail/8000/vm-record_message.wav']);
    console.log('Done Playing')
    await endPoint.execute('send_dtmf', '1234@100');
    // console.log('done Play file');
    let answer = await endPoint.playCollect({file: 'silence_stream://200', min: 1, max: 4});
    console.log(answer);
    await waitFor(5*1000);
    console.log('Dest. Dialog');
    await dialog.destroy();
    console.log('Done');

});

The Playback works, but if the remoteparty (Phonerlite) hangs up during playback the Promise
await endPoint.play(['silence_stream://1000', 'voicemail/8000/vm-record_message.wav']);

never ends...
The destroy event from Dialog is fired
console.log('got bye', reason);

The Result is the same with or without
await endPoint.dialog.modify(dialog.remote.sdp);

It is realy needed to Populate to remote.sdp (i think so?)

Yes, this is how it currently works.  If you destroy the Endpoint (as you do in the 'destroy' handler for the sip dialog) then any operations are terminated by freeswitch and those promises won't resolve .
I suppose it would be more correct to have those promises be rejected (or callbacks invoked with an error) that indicate the call / session was terminated?  In practice, I have never had the need to want to receive those since once the call is over I implicitly know all operations have ceased and I mostly proceed to doing my cleanup code.  I can take a deeper look at this, but would be good to know what you expect....a Promise rejection, as I suggest above?

A Promise rejection wood be perfect. Even if I know the Endpoint is destroyed i‘m not able to reject a Promis, even if I have the promise (and normally only the creator of the promise should resolve or reject).
Best solution would be promises „owned“ by
Endpoint so all promises can be rejected if the endpoint is destroyed/ crashed whatever.
Von meinem iPhone gesendet
Am 23.11.2018 um 19:00 schrieb Dave Horton <notifications<EMAIL_ADDRESS>Yes, this is how it currently works. If you destroy the Endpoint (as you do in the 'destroy' handler for the sip dialog) then any operations are terminated by freeswitch and those promises won't resolve .
I suppose it would be more correct to have those promises be rejected (or callbacks invoked with an error) that indicate the call / session was terminated? In practice, I have never had the need to want to receive those since once the call is over I implicitly know all operations have ceased and I mostly proceed to doing my cleanup code. I can take a deeper look at this, but would be good to know what you expect....a Promise rejection, as I suggest above?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/davehorton/drachtio-fsmrf/issues/9#issuecomment-441296275, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AYvpiO1F36TBfmT2O1te7Nc-y665pZdWks5uyDezgaJpZM4YwlgY.

How do you end a Promise if your Code waits on the Promise.
Maybe this is only needed for persons like me, working with async functions.
Von meinem iPhone gesendet
Am 23.11.2018 um 19:00 schrieb Dave Horton <notifications<EMAIL_ADDRESS>Yes, this is how it currently works. If you destroy the Endpoint (as you do in the 'destroy' handler for the sip dialog) then any operations are terminated by freeswitch and those promises won't resolve .
I suppose it would be more correct to have those promises be rejected (or callbacks invoked with an error) that indicate the call / session was terminated? In practice, I have never had the need to want to receive those since once the call is over I implicitly know all operations have ceased and I mostly proceed to doing my cleanup code. I can take a deeper look at this, but would be good to know what you expect....a Promise rejection, as I suggest above?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/davehorton/drachtio-fsmrf/issues/9#issuecomment-441296275, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AYvpiO1F36TBfmT2O1te7Nc-y665pZdWks5uyDezgaJpZM4YwlgY.

At the end I think a Promise should always end by the component which created the Promise (even by timeout). Only this way it’s possible for booth sites to continue correctly (means „at a trusted state“).
Second problem would be a possible memory leak by unresolved promises / or never called callbacks!

Maybe this is only needed for persons like me, working with async functions.
Von meinem iPhone gesendet
Am 23.11.2018 um 19:00 schrieb Dave Horton <notifications<EMAIL_ADDRESS>Yes, this is how it currently works. If you destroy the Endpoint (as you do in the 'destroy' handler for the sip dialog) then any operations are terminated by freeswitch and those promises won't resolve .
I suppose it would be more correct to have those promises be rejected (or callbacks invoked with an error) that indicate the call / session was terminated? In practice, I have never had the need to want to receive those since once the call is over I implicitly know all operations have ceased and I mostly proceed to doing my cleanup code. I can take a deeper look at this, but would be good to know what you expect....a Promise rejection, as I suggest above?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/davehorton/drachtio-fsmrf/issues/9#issuecomment-441296275, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AYvpiO1F36TBfmT2O1te7Nc-y665pZdWks5uyDezgaJpZM4YwlgY.

we had solved the problem by implementing a BreakablePromise. This class implements a Set of Promises. If an event (like destroy) occurs, all promises in unresolved( like on .play or .playCollect) will be broken (resolved) with a special error.

ok.  I'm still going to look into properly resolving/rejecting those promises.  let's leave this issue open until I do so.


Remove unnecessary code in cupy.fft

The parameters s and axes of _fft are expected not to be integers. The implementation of master accepts integers unexpectedly. This PR fixes the behavior.

LGTM!


Module for Symbolic Calculations / Computer Algebra System

I think a module that performs symbolic calculations with expressions could be very useful. This would manipulate algebraic expressions by simplifying, factoring, expanding, etc., providing exact answers. Ideally with something like this, you would be able to define your own variables and functions along with rules to simplify them. I also think that this is something that should be including because it is something that is far from trivial to implement.
With Swift's custom operators and strong typing, this will be extremely powerful.
There are plenty of use case for this. Firstly, as previously, this would allow you to solve equations with exact value rather than estimations. Moreover, this would allow for the definition of types whose properties themselves are expressions, thus allowing you to perform calculations and easily derive formulas. For example, one could define a point type with expressions as their coordinates, enabling to easily get the formula for the distance between two points.
struct Point {
    var x, y: Expression
}

let p1 = Point(x: "x1", y: "y1")
let p2 = Point(x: "x2", y: "y2")
let lengthFormula = sqrt((p1.x - p2.x) ^ 2 + (p1.y - p2.y) ^ 2)

print(lengthFormula)
// Prints "√((x1 - x2)² + (y1 - y2)²)"

Expressions would also be able to do things like isolate for variables:
var expression: Expression = "x ^ 2 + 1 = 3y + 5"
expression.isolate(for: "x")

print(expression)
// Prints "x = ±√(3y + 4)"

For mathematical computations, something like this has the potential to really make Swift stand out, opening it to numerous applications.

Out-of-scope for Swift Numerics. I agree that it's interesting, but there's already a few years worth of projects lined up (and this is, after all, symbolic computation instead of numeric).
This should absolutely not be read as discouraging you or others to push for or work on this; I am only saying that this package isn't the right home for it.


fixed link to manage stale devices

@trevordueck : Thanks for your contribution! The author(s) have been notified to review your proposed change.

This looks good; we'll merge it. Thanks @trevordueck


Improve Doxygen comments for Mesh::GetNodes and related functions

Clarify when GetNodes can return NULL, and explain the role of EnsureNodes.

This PR is now under review (see the table in the PR description). To help with the review process, please do not force push to the branch.

Merged in next for testing...


Add support for 'tags' and 'releases'.

Add support for generated tags and releases.
For example: https://gitlab.com/lucaapp/android/-/releases

This is the first version of the proposed release/tag model for gitlab2prov.
A gitlab release contains:

release assets (compressed source code archives, optional files generated by CI/CD such as .apk)
release evidence (a .json file containing a snapshot of data that is related to the release, automatically generated for each release)
a git tag (used to mark a specific point in the repository history)

Important Note
Releases can be edited after they've been created. For example, a developer with the appropriate authorization can change the title and the description of a release. The /releases api endpoint does not return such changes and returns only the latest revision of the release. Changes in the title or description can not be detected after they've been made. At least not by using the /releases endpoint. It is as of now unclear whether there is a mechanism that could provide such information. There are webhooks for each new release but there aren't any that i know of, that are triggered when a release is edited.

@onyame I pushed the source code for the initial model to docs/model.py. Do you have any comments on what needs to be changed?

@onyame I pushed the source code for the initial model to docs/model.py. Do you have any comments on what needs to be changed?

Please use one single Entity Release_Asset only, which should replace Release_Assets and the four specific Entities source*.

The Enitity Release can have arbitrary number of members Release_Asset.

Release_Asset should have (at least) the attributes filepath, format, uri.

I removed the attribute field description_html from release entities. The HTML tags created some issues when serializing to json/xml and while rendering dot files. As the same description text without HTML formatting is already present in the attribute field description, this should be fine without further replacements as we do not loose important information.


Dropdown selections do not need a border, especially not 2pt

Not sure what you mean. The element with the border is the input. This is the default focus style for all our inputs.

I have meant the dropdown select, which does not have a border on the screenshot, because i have changed it via browser inspect :) normally it has a 2pt border, but to me it should not have any border

Understood. The behavior is know:

focus by arrow key shows a focus ring
hover does not


Create rolebindings for etcdbackups in cluster's ns

https://github.com/rancher/rancher/issues/20911
Although etcdbackups are scoped to a cluster, they are rancher mgmt resources,
and need to be added to the clusterManagmentPlaneResources list. The crtb_handler.go
ensures all cluster members get access to resources in this list by creating
role bindings.
The "user" global role has get/list/watch permissions on etcdbackups, so cluster
members with 'user" role don't need this and will be able to see etcdbackups
even without this resource being in the clusterManagmentPlaneResources list.
But any user having "user-base" global role or created with any custom permissions
will require this change so they can see the etcdbackups in their cluster.

Not sure how to add tests for this. The integration test will need an rke provisioned cluster, otherwise the option to take snapshot won't be available. Not sure how to add that, and don't see a proper place to add a unit test either.

Add management.cattle.io for the value

lgtm


YouTube requires no simultaneous playing of videos

Is there a way to make only one video play at a time?
YouTube terms require it and I have a page with multiple videos.

I've replied in Slack, this is possible as per this demo:
https://jsfiddle.net/zkn8yhba/2/

I'm getting this errorr when I try to implement your code.
www-widgetapi.js:221 Failed to execute 'postMessage' on 'DOMWindow': The target origin provided ('https://www.youtube.com') does not match the recipient window's origin ('http://<IP_ADDRESS>:8080').

I got it working.


Update dartiverse sample to use requires.1.9 tag

This sample using async/await features
Too late for 1.9 gold, but we should do a merge-to-stable for the 1.9 dot release

How are we managing our samples in a post-Editor world?

We don't care about updating these tags for the Editor any more.

so no other tools downloads samples using the scheme we had in the editor? Does that mean we can stop adding these labels in the samples themselves too?


Breadcrumbs with new settings editor results in empty window after restart

If I close the VS Code window while the new settings editor is open, after restarting VS Code, the window is empty:

After disabling the breadcrumbs, everything works fine.
Developer Toolbar shows the following errors:
  ERR Cannot read property 'scheme' of null: TypeError: Cannot read property 'scheme' of null
    at e.update (file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:4240:311)
    at t.updateBreadcrumbsControl (file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:4324:841)
    at t.openEditor (file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:4326:986)
    at t.doShowEditor (file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:4887:998)
    at t.restoreEditors (file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:4881:795)
    at new t (file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:4874:924)
    at t.create (file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:111:430)
    at e._createInstance (file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:1374:325)
    at e.createInstance (file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:1373:600)
    at Function.t.createFromSerialized (file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:4875:130)
    at r.doCreateGroupView (file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:4907:432)
    at Object.fromJSON (file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:4913:897)
    at Function.t.deserializeNode (file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:487:885)
    at Function.t.deserializeNode (file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:487:781)
    at Function.t.deserialize (file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:488:476)
    at r.doCreateGridControlWithState (file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:4913:823)
    at r.doCreateGridControlWithPreviousState (file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:4913:621)
    at r.doCreateGridControl (file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:4913:135)
    at r.createContentArea (file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:4912:606)
    at r.t.create (file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:2440:455)
    at t.createEditorPart (file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:5367:31)
    at t.renderWorkbench (file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:5365:576)
    at t.startup (file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:5346:380)
    at t.createWorkbench (file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:5400:782)
    at t.renderContents (file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:5400:205)
    at t.open (file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:5408:363)
    at file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:5416:28
    at Object.g [as _notify] (file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:151:807)
    at Object.enter (file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:155:85)
    at n.Class.derive._oncancel._run (file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:156:932)
    at n.Class.derive._oncancel._completed (file:///C:/Program Files/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.main.js:156:373)
    at runCallback (timers.js:672:20)
    at tryOnImmediate (timers.js:645:5)
    at processImmediate [as _immediateCallback] (timers.js:617:5)

and
TypeError: Cannot read property 'scheme' of null
    at e.update (breadcrumbsControl.ts:199)
    at t.updateBreadcrumbsControl (tabsTitleControl.ts:141)
    at t.openEditor (tabsTitleControl.ts:262)
    at t.doShowEditor (editorGroupView.ts:790)
    at t.restoreEditors (editorGroupView.ts:418)
    at new t (editorGroupView.ts:145)
    at t.create (types.ts:166)
    at e._createInstance (instantiationService.ts:120)
    at e.createInstance (instantiationService.ts:73)
    at Function.t.createFromSerialized (editorGroupView.ts:59)
    at r.doCreateGroupView (editorPart.ts:497)
    at Object.fromJSON (editorPart.ts:862)
    at Function.t.deserializeNode (grid.ts:435)
    at Function.t.deserializeNode (grid.ts:428)
    at Function.t.deserialize (grid.ts:464)
    at r.doCreateGridControlWithState (editorPart.ts:856)
    at r.doCreateGridControlWithPreviousState (editorPart.ts:838)
    at r.doCreateGridControl (editorPart.ts:811)
    at r.createContentArea (editorPart.ts:789)
    at r.t.create (part.ts:53)
    at t.createEditorPart (workbench.ts:1065)
    at t.renderWorkbench (workbench.ts:989)
    at t.startup (workbench.ts:286)
    at t.createWorkbench (shell.ts:189)
    at t.renderContents (shell.ts:166)
    at t.open (shell.ts:459)
    at main.ts:103
    at Object.g [as _notify] (winjs.base.js:1191)
    at Object.enter (winjs.base.js:867)
    at n.Class.derive._oncancel._run (winjs.base.js:1089)
    at n.Class.derive._oncancel._completed (winjs.base.js:1057)
    at runCallback (timers.js:672)
    at tryOnImmediate (timers.js:645)
    at processImmediate [as _immediateCallback] (timers.js:617)

VSCode Version: 1.26.0-insider, b48367a8652bd0aa862638bbc24555a70823d3bd
OS Version: Win10

fixed via


BootVM to QEMU !!

Hi guys i face this problem and i have allready done the all instalation of macOS after rebooting i face that :

cobrax_king@cobrax-King-PC:~$ sudo '/home/cobrax_king/Bureau/ra1nstorm-helper-0.9.1/stage2/BootVM.sh'
[sudo] Mot de passe de cobrax_king :
ioctl(TUNSETIFF): Device or resource busy
ls: impossible d'accéder à '/sys/kernel/iommu_groups//devices/': Aucun fichier ou dossier de ce type
ls: impossible d'accéder à '/sys/kernel/iommu_groups//devices/': Aucun fichier ou dossier de ce type
QEMU 2.11.1 monitor - type 'help' for more information
(qemu) qemu-system-x86_64: warning: host doesn't support requested feature: CPUID.01H:ECX.pcid [bit 17]
qemu-system-x86_64: warning: host doesn't support requested feature: CPUID.01H:ECX.pcid [bit 17]
qemu-system-x86_64: warning: host doesn't support requested feature: CPUID.01H:ECX.pcid [bit 17]
qemu-system-x86_64: warning: host doesn't support requested feature: CPUID.01H:ECX.pcid [bit 17]
qemu-system-x86_64: -device vfio-pci,host=00:12.2,bus=pcie.0: vfio error: 0000:00:12.2: no iommu_group found: No such file or directory

me too！

then....the solution -_-

me too！

then....the solution !!

Known issue, what CPU?

iface same problm ,'
i try os in folder but dosnt detect devices

Max Memory Bandwidth
21 GB/s

Processor Graphics

Processor Graphics ‡
Intel® HD Graphics 2000

Graphics Base Frequency
850 MHz

Graphics Max Dynamic Frequency
1.10 GHz

Intel® Quick Sync Video
Yes

Intel® InTru™ 3D Technology
Yes

Intel® Flexible Display Interface (Intel® FDI)
Yes

Intel® Clear Video HD Technology
Yes

of Displays Supported ‡ | 2
Device ID | 0x102
Expansion Options |
PCI Express Revision | 2
Max # of PCI Express Lanes | 16
Package Specifications |
Sockets Supported | FCLGA1155
Max CPU Configuration | 1
TCASE | 69.1°C
Package Size | 37.5mm x 37.5mm
Advanced Technologies |
Intel® Turbo Boost Technology ‡ | No
Intel® vPro™ Platform Eligibility ‡ | No
Intel® Hyper-Threading Technology ‡ | Yes
Intel® Virtualization Technology (VT-x) ‡ | Yes
Intel® Virtualization Technology for Directed I/O (VT-d) ‡ | No
Intel® VT-x with Extended Page Tables (EPT) ‡ | Yes
Intel® 64 ‡ | Yes
Instruction Set | 64-bit
Instruction Set Extensions | Intel® SSE4.1, Intel® SSE4.2, Intel® AVX
Idle States | Yes


Add coveralls support

Add the coveralls package to the requirements.txt. After
applying our repo key to the CircleCI config, this should
enable Coveralls reports on CircleCI.

Nice, I was thinking of doing this myself at some point as well.
LGTM.


@bind-value doesn't work correctly with globalization features

These two work differently:
<input type="number" @bind="numberNumber" />
<input type="number" @bind-value="numberNumber" />

@bind is aware of the fact that we're using a culture-invariant field and will pass the invariant culture into the binding.
However, @bind-value isn't doing this.

The fix is to add two additional mappings to BindAttributes
    [BindInputElement("date", "value", "value", "onchange", true, "yyyy-MM-dd")]
    [BindInputElement("number", "value", "value", "onchange", true, null)]

push commit guys:)


Updated Readme to include npm to install

I am not sure if the 3rd step also needs to have an npm option too, or if npm install means step 2 and 3 isn't needed. Thanks.

The instructions are a lot simpler now, but thank you!


Fix formatting on completions documentation guide

Pull Request Summary
Malformed formatting on Completions guide - https://llm-engine.scale.com/guides/completions/

Test Plan and Usage Guide
How did you validate that your PR works correctly? How do you run or demo the code? Provide enough detail so a reviewer can reasonably reproduce the testing procedure. Paste example command line invocations if applicable.

thanks for fixing!


Test API

Problem Detected

Sorry, I was testing an API for another related issue in cccatalog-api repo and have mistakenly created this issue. Please delete it.


System.InvalidOperationException: 'No browser configured.'

I was following this guide to having auth0 authentication in Xamarin.Forms, but I ran across an exception that I can't find any information about:
at IdentityModel.OidcClient.AuthorizeClient.AuthorizeAsync (IdentityModel.OidcClient.Browser.DisplayMode displayMode, System.Int32 timeout, System.Object extraParameters) [0x00036] in C:\local\identity\model\IdentityModel.OidcClient\src\AuthorizeClient.cs:36
at IdentityModel.OidcClient.OidcClient.LoginAsync (IdentityModel.OidcClient.LoginRequest request) [0x000ee] in C:\local\identity\model\IdentityModel.OidcClient\src\OidcClient.cs:94
var auth0LoginResult = await _auth0Client.LoginAsync(new { audience = Auth0Config.Audience });

This happens on both emulator and device in Android. I set up the intent filter on my activity. I was using Azure AD B2C login on the same device, so I don't think it has to do with my browser. I see here where the exception is, but the guide I was following didn't have to set the browser property.
Auth0.Core - 6.5.4
Auth0.Android/iOS - 3.0.0
Xamarin.Forms - <IP_ADDRESS>7036

Sorry for the very late response, this must have fallen between the cracks.
Over three years later, I can no longer reproduce this issue so I am closing this. But again, sorry for missing this.


Dependabot 0.14 crashes on azure_helpers.rb:35:in `branch_delete': undefined method

When using 0.14 it crashes on me with the following log
https://pastebin.com/g283zEEs
It is too big to paste here.

I will look into this and revert.
The last change to that particular method was in #440
Meanwhile, I'd appreciate any other context you can provide

@air2 Is this still happening?

I am not sure, because I use the latest version now, So it is not happening in the latest version


"Crintești" is listed in the drop-down menu

Description:
"Crintești" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Crintești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Crintești, Argeș (Topoloveni)".


edge_security_policy is not optional

TL;DR
The attribute edge_security_policy is not optional, although the upgrade guide says that it is optional.
https://github.com/terraform-google-modules/terraform-google-lb-http/blob/a0ea2acb64ab64a130b7aced40aa25644924499b/variables.tf#L91
Expected behavior
You do not need to specify a value for the attribute.
Observed behavior
You are required to specify a value for the attribute.
Terraform Configuration
irrelevant - can be directly observed in the variables.tf

Terraform Version
Terraform v1.3.2
on windows_amd64
+ provider registry.terraform.io/hashicorp/google v4.65.2
+ provider registry.terraform.io/hashicorp/google-beta v4.65.2
+ provider registry.terraform.io/hashicorp/local v2.4.0
+ provider registry.terraform.io/hashicorp/random v3.5.1

Additional information
No response

@uap-universe Based on following logic do you think it should be set to "" by default?
  # To achieve a null backend edge_security_policy, set each.value.edge_security_policy to "" (empty string), otherwise, it fallsback to var.edge_security_policy.
  edge_security_policy = lookup(each.value, "edge_security_policy") == "" ? null : (lookup(each.value, "edge_security_policy") == null ? var.edge_security_policy : each.value.edge_security_policy)

@uap-universe ok I will set it to null local config in PR #333


fix OpenVINO doc

1、修改OpenVINO文档，说明VPU下使用2020.4版本的OpenVINO

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

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

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

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


How to set the current directory?

for example, I have a react-native app located in /app, and when I'm trying to use console:

I think we can use a .rnconsole file to specify where really the project file is, in that file you can specify a relative path.

@beansoftapp is it implemented?
if not, I think it's better to save it under .idea folder

https://github.com/beansoftapp/react-native-console/blob/master/react-native-console.jar, you can download it now , 1.0.8 will be public at IDEA plugin center soon. See github read me of how to use, so far only supports edit config file.

And still thanks for your idea that in fact i need this function too when coding a library project with a runnable example project.


Avoid shortening of linkified code

Fixes #789

This is still happening for me in the latest version of the Firefox extension. Is this fix not shipped yet, or is it a regression?

Regression, if you're using the version from AMO and not a locally-installed one.
Can you post a link where you're seeing the issue + screenshot?

Cannot reproduce on a public repo. Maybe Firefox didn't update automatically before or something...
Sorry for raising the alarm. :sweat_smile:

No worries, it was actually a regression and it was fixed a few days after your report. #1145 Thank you!


Fixed build warning in GDAL support module

Function GetOpenCVTypeName is not used
GDAL detector is erroneously turned ON when IMREAD_UNCHANGED flags value is passed to imread

:+1:


Refactor LifecycleComponent package path

Description
Refactor LifecycleComponent package path
Issues Resolved
N/A
Check List

[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Integration test is failing because the change in core is not released to maven repo yet.


Editable dropdown

Are there any options that allow for dropdown to included options PLUS editing - i.e. enter your own value?

Nope.


Add additional information to workshop request page

[ ] ask how many pupils are in computer science related courses / AGs of the pupils wanting to visiting

[ ] provide checkbox asking wether they wish for tour over the campus

[ ] provide checkbox asking wether they wish for informations about the study options at HPIs

[x] make sure admins and organizers have access to results of this data

@annkatrinkuessner does the 4th point mean that only admins and organizers should see it, or that coaches can see it too but the other two should have special tests etc

everybody involved in organizing the camps should have access to the send data. therefore, coaches, as well as admins and organizers, should have access.

As far as I can tell this is already the case. All three roles have access to the "show page" of requests.


Add Fantom Opera and Fantom Testnet support

Adding support for Fantom Opera and Fantom Testnet networks and their Etherscan compatible client called FTMScan
Reference:
https://github.com/safe-global/safe-deployments/pull/243

CLA Assistant Lite bot: Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.
I have read the CLA Document and I hereby sign the CLA
You can retrigger this bot by commenting recheck in this Pull Request

Fixing it here, thanks for the heads up :)
https://github.com/safe-global/safe-eth-py/pull/610

I have read the CLA Document and I hereby sign the CLA

@HonzaDajc I think is going to be faster if you remove the 1.2.0 addresses here also because this was not still approved, https://github.com/safe-global/safe-deployments/pull/243

@HonzaDajc I think is going to be faster if you remove the 1.2.0 addresses here also because this was not still approved, safe-global/safe-deployments#243

Ok, I removed v1.2.0 contracts from addresses

Could you rebase so we can merge?