awsmobile creating appsync: create graphql api fails.

Hi, I can never successfully add appsync to my project from awsmobile CLI.
version 1.1.5
following error occurs after awsmobile push:
`
⠼ creating appsync: create graphql api...create graphal API err [...project]
{ Error: connect ETIMEDOUT <IP_ADDRESS>:443
at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1161:14)
message: 'connect ETIMEDOUT <IP_ADDRESS>:443',
errno: 'ETIMEDOUT',
code: 'NetworkingError',
syscall: 'connect',
address: '<IP_ADDRESS>',
port: 443,
region: 'eu-west-2',
hostname: 'appsync.eu-west-2.amazonaws.com',
retryable: true,
time: 2018-08-05T07:38:58.690Z }
appsync creation failed
{ Error: connect ETIMEDOUT <IP_ADDRESS>:443
at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1161:14)
message: 'connect ETIMEDOUT <IP_ADDRESS>:443',
errno: 'ETIMEDOUT',
code: 'NetworkingError',
syscall: 'connect',
address: '<IP_ADDRESS>',
port: 443,
region: 'eu-west-2',
hostname: 'appsync.eu-west-2.amazonaws.com',
retryable: true,
time: 2018-08-05T07:38:58.690Z }
updating backend project: [...project]
Successfully updated the backend awsmobile project: [...project]
`

I think the issue is because AppSync is not available on EU (London) region.
Current supported regions are
US East (N. Virginia), US West (Oregon), US East (Ohio), EU (Ireland), EU (Frankfurt), Asia Pacific (Tokyo), Asia Pacific (Sydney), Asia Pacific (Mumbai) and Asia Pacific (Singapore).
You can try on eu-central-1 or eu-west-1 regions that are closer to you.

Ah right thats what I thought @elorzafe, but I don't get to choose region on the --prompt 🤔


sqlite3.dll remains locked after running the tests

After I run all the tests in the SQLite.csproj project there remains a lock on sqlite3.dll file preventing me from rebuilding or rerunning the tests until I restart Visual Studio. Curiously enough, I don't encounter this problem running the same tests from SQLiteMetroTests.csproj. This is the error I get:
Unable to copy file "C:\Users\Damir\Documents\GitHub\sqlite-net\lib\x86\sqlite3.dll" to "bin\Debug\sqlite3.dll". The process cannot access the file 'bin\Debug\sqlite3.dll' because it is being used by another process.

I'm using Visual Studio 2012 RC built-in test runner with NUnit Test Adapter (Beta 2) v0.92. Is anyone else encountering this issue? Could it be a bug in Visual Studio or the adapter?

As per http://stackoverflow.com/a/15748825/541420 you can use the following command in "Build Events > Pre-build event commande line" to avoid a batch file and an error if the process is not running:
taskkill /F /FI "pid gt 0" /IM vstest.executionengine.x86.exe
Hope if helps.

I have the same exact problem. IIS remain stuck because that file is locked even if all database access has been completed.
This generally indicate a reference leak on the DB usage. Killing the hosting process (IIS or IISExpress) is not a solution.
Unfortunately, tracking down these handle leaks are very very difficult. There should be an API to reset all current connections and deallocate everything in a clean way, but there isn't.


Dots representing the progress instead checked scenarios description

Hey Folks,
Earlier I got this versions of protractor and framework:
    "protractor-cucumber-framework": "4.0.7",
    "protractor-jasmine2-screenshot-reporter": "0.3.2",

now when I run proctractor:cucumber task, the output is dots and letters that represents result of each test instead checked scenario descriptions like before the update.
My question is: Is possible use some configuration or flag (in CLI) to use scenario descriptions instead dots?

@Odravison
To which version of Cucumber are you referring? Version 3 doesn't have the pretty formatter anymore.
Secondly it's a question about a third party library, please ask it there. I'm going to close it for now.

First, thanks in advance about fast answer.
Thank you @wswebcreation, about this information. I didn't know who is responsible for this output logs.
Yeah, my cucumber is above version 3 =/
Unfortunately they didn't make this feature as optional configs, or they did? I will ask there, I'm sorry about wrong place to this issue.
Thanks again @wswebcreation

Hi @Odravison
Protractor gets the log from CucumberJS. protractor-cucumber-framework is the "middleware" that passes all the data between protractor and CucumberJS.
For reporting, have you also taken a look at this plugin that uses this kind of report ;-)

Thank you! ;)

I just have tested this plugin, it's very helpful, however, see the scenarios while tests are running was more helpful, to me.


improve usage doc for starlark

There are 3 sections in the usage doc now:

Overview
Function
FunctionConfig

I will update our template if this makes sense.

@frankfarzan PTAL


Amavis improvement 2020 spring

Define several error reasons and patterns
Add lhost-amavis-03.eml into sample email directory

Coverage increased (+0.005%) to 95.15% when pulling 779116a500d09369893728e23c904358b72f959f on amavis-improvement-2020-spring into 9e717e0e222e851ab20e11d671ed4fcb997fd9a1 on master.


Escape post title when rendering disqus JS snippet

Single quotes in post titles don't break the JS anymore

Thanks!


EKF: Optical flow and Range Finder Updates

Fixes https://github.com/PX4/Firmware/issues/9398 and improves robustness of flow operation to operator handling pre-flight.
Needs to be tested against https://github.com/PX4/Firmware/issues/9414
TESTING
TODO

I tested this with the vehicle in hand with only optical flow aiding, and it seems to have fixed the bad behaviour I had been seeing. Range aid also seems to function correctly now.
Will be flight testing soon.

Still drifting when flow aiding is on, but data is invalid :

@mhkabir Can you please provide a link to your log

@priseborough Here you go : https://logs.px4.io/plot_app?log=1f62deb9-a479-4895-b839-584a548a3bca

@mhkabir the innovations are zero mean:

Velocity estimate is also close to zero:

Position drift is minimal:

What is the problem you are seeing?
The EKF assumes a zero optical flow rate if not in air and flow data quality is below the minimum.

Ah okay I did not realise that this was by design. Thanks for the clarification!

Did 4 flights with this branch, with a perfect success on every flight. 250-sized racer vehicle is equipped with a TFMini Lidar, PX4Flow with 8mm lens (running slightly modified research firmware), and Drotek M8P GPS (RTK disabled) and Visual-Inertial Odometry (not enabled for these flights). Running PX4 master + Commander patches from https://github.com/PX4/Firmware/pull/9435 + this branch of ECL.
Flight 1 : No GPS initially, lock gained in flight. Good example of OF speed limit functioning and then gradually releasing. https://logs.px4.io/plot_app?log=858354d4-39bd-4400-85f8-383f3c7d4f56
Flight 2 : GPS lock since takeoff. OF + GPS aiding. https://logs.px4.io/plot_app?log=b17af656-9b43-42bd-ac02-d103e6407334
Flight 3 : Position hold and short trajectory in shadow of building. Horrible GPS. Windy conditions. https://logs.px4.io/plot_app?log=1d532995-ce0d-4908-ac08-2788f53e997d
Flight 4 : Long trajectory in shadow of building. Horrible GPS. Windy conditions. https://logs.px4.io/plot_app?log=62437c5a-8be4-435f-a068-0cff96be3ffe
Vehicle needs some controller tuning, but apart from that, performance was very good!


Can I start Korean Translation?

Hi, Can I start Ko Translation? Solutions in this page helped me a lot.
It would be great to have a Korean version.

@kyeongsoosoo sure, you can start by creating a folder with a language code for Korean and start adding Markdown there with the translation. Make sure that the metadata is there and up to date. I can help you by setting up the skeleton for it and you will add more on the fly there.

[ ] create a folder ko in the root
[ ] the file structure mirrors the structure of other solutions, including filenames

As far as I remember, that is all you need to do for getting a Korean version. In case I forgot something, please ping me for any further questions and I'll help you set it up.
Thanks for your contribution!

Thanks for your guide.
Now, I'm looking for a co-worker who can help with reviews.
Will begin working soon!

I'm closing the issue as resolved. If you have any other questions, please open a new issue 🙇🏻


Unable to Locate Downloadable Code

Hello,
I am eagerly looking forward to your amazing tool's demo. I attempted to download the code and understood the general explanation, but I could not locate the actual source files. If the source has already been published, I would be grateful if you could direct me to it. I am looking forward to future updates.
Thank you.

thats cus there is none

Thank you.


Lsp finder file title (Find: xxx) not automatically closing on close/file open

Hi!
Really love the plugin, but I have a small issue with the lsp_finder.
When i try to either close it with 'q' or open a file the small title window with "Find: xxx" never closes. See pictures below.

Use Lspsaga  lsp_finder on "authorizedFetch"

Lsp finder window opens and i select the definition with for example 's'

The file opens i a split, but the small title window with 'Find: authorizedFetch' does not close automatically. Leaving me to having to close it manually.

Hope anyone can help me with this.
Neovim version: 0.7.0

I can't reproduce your problem. Please test with a min config

Works like a charm now! Awesome job :-)


Consumer callback is called infinitely while extracting compressed 0byte entry

Summary
Extract a compressed 0byte file, Consumer callback is called infinitely.
I write my code using entry.uncompressedSize for bufferSize to reduce count of consumer closure is called.
(like below test)
My app has infinitely loop.
Steps to Reproduce
This test never stop.
Put testExtractCopressedEmptyData.zip to Tests/ZIPFoundationTests/Resources
func testExtractCopressedEmptyData() {
    let expectation = XCTestExpectation(description: "Called extract consumer once")
    let archive = self.archive(for: #function, mode: .read)
    // empty.txt is compressed emtpy file.
    guard let entry = archive["empty.txt"] else { XCTFail("Failed to extract entry."); return }
    do {
        // Use uncompressedSize to reduce count of closure called
        let checksum = try archive.extract(entry, bufferSize: UInt32(entry.uncompressedSize)) { (data) in
            XCTAssertEqual(data.count, 0)
            expectation.fulfill()
        }
        XCTAssert(entry.checksum == checksum)
    } catch {
        XCTFail("Failed to unzip data descriptor archive")
    }
    wait(for: [expectation], timeout: 3.0)
}

To stop infinity loop, I can rewrite the extract code:
let checksum = try archive.extract(entry, bufferSize: max(UInt32(entry.uncompressedSize), 1)) { /* consumer code */ }

However, consumer calls twice. (It should be once)
With Xcode debugger, I notice that the test calls Data.process method with bufferSize = 0.
Data.process creates 0-byte Data inifinity.
NOTE: macOS can creates Zip file have 0byte compressed entry. I create sample zip file and attach in this issue.
It contains 0 byte "empty.txt" and 86336byte "faust.txt"
(zip command creates stored no-compressed entry for 0 byte file)
$ unzip -v Tests/ZIPFoundationTests/Resources/testExtractCopressedEmptyData.zip
Archive:  Tests/ZIPFoundationTests/Resources/testExtractCopressedEmptyData.zip
 Length   Method    Size  Cmpr    Date    Time   CRC-32   Name
--------  ------  ------- ---- ---------- ----- --------  ----
       0  Defl:N        2   0% 10-23-2020 16:13 00000000  empty.txt
  222214  Defl:N    86336  61% 06-19-2014 17:09 af9c0a0d  faust.txt
--------          -------  ---                            -------
  222214            86338  61%                            2 files

Method Defl:N means deflated entry.
Expected Results
consumer closure called only once with empty data (Data())
Actual Results
consumer closure called infinitely.
Regression & Version
I am using v0.9.11
Xcode 12.2 & macOS 10.15.7
Related Link

@weichsel I test with current development branch. It causes when entry is file (not directory).

Thanks for updating the issue and for providing a test file.
compressedSize being bigger than uncompressedSize for zero-length entries is an edge case I didn't think about 🙈.
I can't promise when I'll find time to fix this, but if you need this, you can submit a PR and I'll do my best to review it.


Haskell package hdocs is broken in unstable.

Issue description
When trying to install hdocs from unstable, I get the following:
nix-shell -I nixpkgs=https://github.com/NixOS/nixpkgs-channels/archive/nixos-unstable.tar.gz -p "haskell.packages.ghc7103.ghcWithPackages (pkgs: [pkgs.hdocs])"
these derivations will be built:
  /nix/store/abcnjqb3mddl4l2ad3z9m9jf7b7s749x-hdocs-<IP_ADDRESS>.drv
  /nix/store/finjj64h410yz01ilg5zikc3m5pvfzqv-ghc-7.10.3.drv
building path(s) ‘/nix/store/ypnpm4blff38fxl8dmawga2kslh50131-hdocs-<IP_ADDRESS>’
setupCompilerEnvironmentPhase
Build with /nix/store/xijka1pvhm02w4phzpp21b3iazkd4h0c-ghc-7.10.3.
unpacking sources
unpacking source archive /nix/store/sdl9a4xbl0spxfkqg974ascvd59ljy64-hdocs-<IP_ADDRESS>.tar.gz
source root is hdocs-<IP_ADDRESS>
setting SOURCE_DATE_EPOCH to timestamp<PHONE_NUMBER> of file hdocs-<IP_ADDRESS>/tools/hdocs.hs
patching sources
compileBuildDriverPhase
setupCompileFlags: -package-db=/tmp/nix-build-hdocs-<IP_ADDRESS>.drv-0/package.conf.d -j1 -threaded
[1 of 1] Compiling Main             ( Setup.hs, /tmp/nix-build-hdocs-<IP_ADDRESS>.drv-0/Main.o )
Linking Setup ...
configuring
configureFlags: --verbose --prefix=/nix/store/ypnpm4blff38fxl8dmawga2kslh50131-hdocs-<IP_ADDRESS> --libdir=$prefix/lib/$compiler --libsubdir=$pkgid --with-gcc=gcc --package-db=/tmp/nix-build-hdocs-<IP_ADDRESS>.drv-0/package.conf.d --ghc-option=-optl=-Wl,-rpath=/nix/store/ypnpm4blff38fxl8dmawga2kslh50131-hdocs-<IP_ADDRESS>/lib/ghc-7.10.3/hdocs-<IP_ADDRESS> --enable-split-objs --disable-library-profiling --disable-executable-profiling --enable-shared --enable-library-vanilla --enable-executable-dynamic --enable-tests
Configuring hdocs-<IP_ADDRESS>...
Setup: At least the following dependencies are missing:
haddock-library ==1.2.*
builder for ‘/nix/store/abcnjqb3mddl4l2ad3z9m9jf7b7s749x-hdocs-<IP_ADDRESS>.drv’ failed with exit code 1
cannot build derivation ‘/nix/store/finjj64h410yz01ilg5zikc3m5pvfzqv-ghc-7.10.3.drv’: 1 dependencies couldn't be built
error: build of ‘/nix/store/finjj64h410yz01ilg5zikc3m5pvfzqv-ghc-7.10.3.drv’ failed
/run/current-system/sw/bin/nix-shell: failed to build all dependencies

Steps to reproduce
Run the following:
nix-shell -I nixpkgs=https://github.com/NixOS/nixpkgs-channels/archive/nixos-unstable.tar.gz -p "haskell.packages.ghc7103.ghcWithPackages (pkgs: [pkgs.hdocs])"
Technical details

System: 16.03.909.f199be0 (Emu)
Nix version: nix-env (Nix) 1.11.2
Nixpkgs version: "16.03.909.f199be0"

The ghc7103 package set is not particularly well supported. If someone has a PR to fix this issue, then we'll be happy to merge it.


include meta data in tasks

Is your feature request related to a problem? Please describe.
Actually this is not a feature request, I just want your opinion on this workflow:
I have a lot of metadata I want to include in the json tasks, so that I can filter these tasks later in LabelStudio.
Describe the solution you'd like
I figured out that meta data can be included in a task like this:
[
    {
        "data": {
            "image": "s3:/...1.jpg",
            "meta_stuff_I_want_to_add": "meta data string1",    # <--
            "more_meta_stuff": "meta data string2"              # <--
        },
        "annotations": [],
        "predictions": [
            {
                "result": [
                    {....

After importing to LS, the meta data appears like this:

Also, the meta data is later saved in the annotation which is synced to a cloud storage, which I need.
(1)
So, is this task setup of the "correct"? Is there any other, better way to do this?
(2)
In order to keep filtering in labelstudio performant, should I rather work with one-hot encoding:
[
    {
        "data": {
            "image": "s3:/...1.jpg",
            "meta_feature_1": True
            "meta_feature_2": False
            ...

or stick to strings:
[
    {
        "data": {
            "image": "s3:/...1.jpg",
            "feature": "meta_feature_1"
            ...

What is your experience? With one-hot encoding I might end up with 100+ additional columns. Would this be a problem for the Backend DB or Labelstudio itself?
Thanks!

Thanks, I will try

Hi @heidi-humansignal,
I wonder if labelstudio can work with Booleans?
[     {         "data": {             "image": "s3:/...1.jpg",             "meta_feature_1": True, ...
The "True" in the metadata appears as string in labelstudio with no option to switch to Boolean. In order to properly filter, I changed the data to a number int.
[     {         "data": {             "image": "s3:/...1.jpg",             "meta_feature_1": 1, ...
With this, I can set the datatype to "int" in Labelstudio and filter properly. But this is a bit painful, as later in my workflow, when exporting the annotations, I have to convert the int in task.data.meta_feature_1 back to Boolean.
So, does Labelstudio somehow allow to import Boolean?


[opentitan-tool/cw310] Rom detection timeouts trigger test failures

On the cw310 when invoked with bazel, Rom detection timeouts trigger test failures, Instead of loading a bit-stream, if the rom detection times out the tests appear to fail.
Engineering a timeout by decreasing the limit in sw/host/opentitantool/src/command/load_bitstream.rs causes tests to fail, not to load a new bitstream and proceed as I'd expect.

I don't know if this is serious as it may be some of the software isn't ready at the time of the earlier timeout, so it may be that a problematic bitstream cannot cause this.

Hmm, looks like I may have brought in a defect from master and gotten confused.


Exemptions/Exceptions in Pepr Core

Is your feature request related to a problem? Please describe.
Have an exemption clause where specific failing objects are allowed to pass through admission. This can already be done in Pepr Core today through filters. This issue is here for tracking purposes.
Describe the solution you'd like

Given an exemption is created
When a failing object that is on the exemption list hits Admission
Then it is approved and goes into the cluster

Describe alternatives you've considered
(optional) A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

@cmwylie19 should this issue be closed

@cmwylie19 should this issue be closed?

oh yes, thought automation was supposed to do it for us so need to update that!


GROOVY-9456: Bump Gradle Spotbugs plugin to 4.0.0

Merged. Thanks.

Unfortunately, latest spotbugs plugin is still broken for us. All reports with errors are truncated so as not to show the interesting details.

Sad to know...
P.S. I am trying to create a project on the SonarCloud:
https://issues.apache.org/jira/browse/INFRA-19971
Here is the link to show all current Apache projects on the SonarCloud
https://sonarcloud.io/organizations/apache/projects

I'm trying to downgrade its version for now...


more algorithms: Butina and/or DBSCAN

Dear Patrick,
Maybe some other algorithms are more adapted to molecular datasets.
Butina, Darko. "Unsupervised data base clustering based on daylight's fingerprint and Tanimoto similarity: A fast and automated way to cluster small and large data sets." Journal of Chemical Information and Computer Sciences 39.4 (1999): 747-750.
https://pubs.acs.org/doi/10.1021/ci9803381

I think the following algorithm is not so known in chemoinformatics, but interesting:
Ester, Martin, et al. "A density-based algorithm for discovering clusters in large spatial databases with noise." Kdd. Vol. 96. No. 34. 1996.
https://www.aaai.org/Papers/KDD/1996/KDD96-037.pdf

Unfortunately, I don't have a google account, so I cannot comment in your blog
(https://practicalcheminformatics.blogspot.com/2019/04/clustering-21-million-compounds-for-5.html)
Regards,
F.

Thanks for the comments.  I'm very aware of the Butina algorithm, I use it all the time.  Unfortunately, that algorithm scales as n-squared with the number of molecules.  As such, it's not great for large datasets.  I haven't tried the other algorithm, but I'll check it out.
Pat

The authors say that DBSCAN has O(n * log(n)) complexity.
So, it seems to not require the full (all to all) distance matrix.
But, you will need a data structure to run region queries (like a mu-tree or a bisector tree, or a vantage-point tree).

Maybe DBSCAN to do the overall clustering, then Butina to enumerate representatives of each cluster of interest would be quite interesting.


Can se set Currency other than USD

Hi,
I have followed your step everything working fine. But while I am adding Indian currency "INR" I got error message.
Got Http response code 400 when accessing https://api.sandbox.paypal.com/v1/payments/payment.

Please help

CURRENCY
(Required) The currency code.Note: CURRENCY is applicable only to processors that support transaction-level currency. The PayPal acquirer allows PayPal Payments Advanced and PayPal Payments Pro merchants to run transactions using any of the following six currencies with a single account.AUD - Australian dollar CAD - Canadian dollar EUR - Euro GBP - British pound JPY - Japanese Yen USD - US dollar Limitations: Three characters.

https://developer.paypal.com/docs/classic/payflow/integration-guide/#paypal-credit-card-transaction-request-parameters


Adding Azure OpenAI LLM Provider Support "azureopenai"

Adding native Azure OpenAI LLM provider support including the embedding models.
Env. variables required for support to work:
AZURE_OPENAI_ENDPOINT="INSERT_ENDPOINT_URL e.g. https://xxxx.openai.azure.com/"
AZURE_OPENAI_API_KEY="INSERT_AZURE_OPENAI_ENDPOINT_KEY"
OPENAI_API_VERSION="INSERT_AZUREOPENAI_API_VERSION_NO e. 2023-08-01-preview"
OPENAI_API_TYPE="azure"
Note also new entry in config.py to specify the Azure OpenAI embedding model name:
self.azure_embedding_model = os.getenv('AZURE_EMBEDDING_MODEL', "INSERT_EMBEDDIGN_MODEL_DEPLOYMENT_NAME")

hey @norrisp90 thanks for this! Can you please revert so that defaults still hold such as in config file, etc? After that happy to merge!

hey @norrisp90 thanks for this! Can you please revert so that defaults still hold such as in config file, etc? After that happy to merge!

@assafelovic Ok, done, 2nd commit moves the Axure embedding model name to an env var AZURE_EMBEDDING_MODEL


Trigger issue when new ex_doc releases have been released.

This should be doable with github actions I believe. It would make keeping up to date easier.

Definitely agree with this one. Which option would be more convenient:

Create a cron action that checks like every week/month for a new version of ex_doc
In ex_doc repostirory, create a on_release action that uses checkout action to create an issue in rebar3_ex_doc repo.


Restructure the decode function

imap_utf8() is causing me some issues with email subjects which I want to resolve by using $this->mime_header_decode() instead.
The first problem is that Swedish special characters like "ä" ("\xC3\xA4") is decoded as "ä" ("\x61\xcc\x88") which can cause unexpected errors with regex for example.
The second is that some email subjects have two encoded strings in the subject, and imap_utf8() only decodes one of them. For example:
"=?UTF-8?Q?Re=3A_Uppdaterat_=C3=A4rende_=28447899=29=2C_kostnader_f=C3=B6r_hj=C3=A4?= =?UTF-8?Q?lp_med_stadge=C3=A4ndring_enligt_ny_lagstiftning?="
This solution should handle all cases, but I added imap_utf8() inside an else statement in case it catches something I'm not aware of.
Beyond that, the two if statements I removed should be dead code since the call to imap_utf8() already decodes $value.
And the removed else if statement should be redundant because there is an identical if statement below that.
It's my first time contributing to a project so I'm happy with any feedback that you have.

Hi @istid ,
thanks a lot for your pull request!
Best regards & happy coding,


Remove task for travis to update package.json version number on release

Remove task for travis to update package.json version on release as new stricter repository rules will cause this to fail.

 Set up stricter rules on repo as per engineering guide

can - provider: npm go from travis too?

No, we still need to deploy to npm for the prototype kit


Bug with ClassInitialize

          > Latest version (not preview) and we were using debug. Had to switch to testcleanup instead.

Same here :(
Originally posted by @enisak in https://github.com/microsoft/testfx/issues/143#issuecomment-1033470772

I am confused as there seems to be two different "things" (one feature request and one bug) being discussed here.
As far as I can see, the inheritance of the class initialize is implemented and is working properly (we recently added a bunch of integration tests for this: https://github.com/microsoft/testfx/blob/2d2ab810a78fa98855a9e56cb5a18d7f963c45f2/test/IntegrationTests/MSTest.VstestConsoleWrapper.IntegrationTests/SuiteLifeCycleTests.cs).
@enisak Could you please let us know which version of MSTest you are using? Also, would you mind providing a small repro of what's not working so we can investigate (cc @engyebrahim)?

I am confused as there seems to be two different "things" (one feature request and one bug) being discussed here.
As far as I can see, the inheritance of the class initialize is implemented and is working properly (we recently added a bunch of integration tests for this: https://github.com/microsoft/testfx/blob/2d2ab810a78fa98855a9e56cb5a18d7f963c45f2/test/IntegrationTests/MSTest.VstestConsoleWrapper.IntegrationTests/SuiteLifeCycleTests.cs).
@enisak Could you please let us know which version of MSTest you are using? Also, would you mind providing a small repro of what's not working so we can investigate (cc @engyebrahim)?

Hi there,
I am not currently working on the same thing, but I do remember that Class Initialize was working as expected: being triggered once per class (which inherits base class containing class initialize method). However, the issue was that class cleanup did not follow the pattern. So, at the end, I have actually created Assembly cleanup method in a way that it becomes a workaround to overcome the issue (business logic could fit somehow into that).
I do not remember which version of MSTest was, but surely the latest version that was available on Feb 9, 2022

If still not clear, please let me know to provide some code to help you to reproduce the issue.

It seems to me that what you are asking for is available (not sure since which version) but we have a test for derived class cleanup that seems to be working well: https://github.com/microsoft/testfx/blob/main/test/IntegrationTests/TestAssets/SuiteLifeCycleTestProject/LifeCycleClassInitializeAndCleanupBeforeEachDerivedClass.cs#L90
I will move forward by closing as cannot reproduce. If you see some problem please reopen or create a new issue with a reproducer so we can investigate.


Control a destination of generator/postgres.GenerateDSN

Problem
We would like to control the destination directory of postgres.GenerateDSN, but inside this function we have a hardcoded value - the database name and schema path.
Desired solution
I see several options:

Add option to GenerateDSN (backward compatibility issue)
Create a new GenerateDSN with functional options for this and other changes (as in Dave Cheney's article). I can recommend this tool
Export generator/postgres.postgresQuerySet as DefaultQuerySet() metadata.DialectQuerySet

For our case, we would like to have the second option, hiding the complexity of the generator implementation.

Hi @kazhuravlev.
It is possible to change destination directories using  Generator Customization, check these tests for more details: TestGeneratorTemplate_Schema_ChangePath, TestGeneratorTemplate_Model_ChangePath and TestGeneratorTemplate_SQLBuilder_ChangePath.

Of course I can manage the base directory. But I can't manage the rest of the directories (after the specified one).
When I set the resulting directory like ./output, I get a result like ./output/<DBNAME>/<SCHEMA_NAME>. But in my case there are multiple databases with different schemas inside each. So I don't want to have additional directories after ./output . Only ./output/[model|table] (instead of ./output/some__database_name/some__schema_name/[model|table] ).
Initial problem - I have local databases (dev dbs) and multiple databases in production. Each database is a separate tenant (client) with the same schema but different data. After generating, I get a little strange code, which contains the name of the database.
Main argument: The database name is only a configuration option and must be controlled by the jet user. Ideally, the schema name is the same.

To remove database name, you can use relative path "../".
err := postgres.Generate( // or GenerateDSN
	"./output",
	dbConnection,
	template.Default(postgres2.Dialect).
		UseSchema(func(schemaMetaData metadata.Schema) template.Schema {
			return template.DefaultSchema(schemaMetaData).UsePath("../")
		}),
)

Now generator will generate to ./output/[model|table|enum] folders.

Thanks, solved!


DMP-3216: Update angular and deps to v18.0.0

Jira link (if applicable)
DMP-3216
Change description
Angular 18 has been released so I've upgraded to the latest version

@deanbullock-hmcts if this good to go?


can you also please provide the test vectors for the RoCE module? Thank you !!

Hi David, can you also please provide the test vectors for the RoCE module (or the corresponding scripts to generate such test vectors)?
In the script they should be
write_read_read_large_receiver.in
write_read_read_large_receiver.out
rdma_txwriteread.in
rdma_txwriteread.out

Thank you very much!

Hi @hao310rui140326 . Were you able to generate any of those test files?


Related filters. - don't work in DSW

example

@gnibeda the related filters is a default feature in ZEN it does not set manually. You just assign two filters and they work together. Here is the demonstration
This is how it works in ZEN

And this is on DSW

And here is the link to example
https://pie-chart-bug.demo.community.intersystems.com/dsw/index.html#/USER/test.dashboard

We already have an endpoint for retrieving a filtered collection of filters. When we manually enter some data, the filter endpoint collects related filters.
Try to manualy input year value here
https://pie-chart-bug.demo.community.intersystems.com/dsw/index.html#/USER/Pareto Charts/Pareto Chart for Category.dashboard

Done in 3.2.18

@evshvarov could you test this on your data?

This is implemented and already working. The issue can be closed

done


Issue in fresh install on MacOS via Homebrew

To save time and help us identify the issue a bug report should at least contain the following:

a useful description of the bug - "It's not working" isn't good enough - you must try harder ;)

There seems to be an issue with the version served by homebrew.  It complains that the $DISPLAY environment variable isn't properly set & I can't find any instructions to set it.

the steps to reproduce the bug

Install for the first time on a 2016 MacBook* with a fully updated MacOS:
brew install freerdp
Once that completes, try to run any freerdp command:
% xfreerdp  [21:08:47:507] [10378:117b2e00] [ERROR][com.freerdp.client.x11] - failed to open display:  [21:08:47:508] [10378:117b2e00] [ERROR][com.freerdp.client.x11] - Please check that the $DISPLAY environment variable is properly set.
% xfreerdp --help [21:09:57:462] [10389:1a213e00] [ERROR][com.freerdp.client.x11] - failed to open display:  [21:09:57:463] [10389:1a213e00] [ERROR][com.freerdp.client.x11] - Please check that the $DISPLAY environment variable is properly set.
% xfreerdp --version [21:12:57:104] [10398:0b72be00] [ERROR][com.freerdp.client.x11] - failed to open display:  [21:12:57:106] [10398:0b72be00] [ERROR][com.freerdp.client.x11] - Please check that the $DISPLAY environment variable is properly set.
% xfreerdp --buildconfig [21:22:28:922] [10460:0faffe00] [ERROR][com.freerdp.client.x11] - failed to open display:  [21:22:28:923] [10460:0faffe00] [ERROR][com.freerdp.client.x11] - Please check that the $DISPLAY environment variable is properly set.
Problem survives a full reboot.
*: Could the small screen size be a factor?

operating System, architecture, distribution e.g. linux, amd64, debian

mac OS Big Sur 11.6

you need X11 installed to use xfreerdp

how to resolve this problem?


Test: check_valid

this function require checking

I could write several test cases and use all_valid_moves() to test the function.


FR: Print brew upgrade instructions when kurtosis cli users are using an outdated version of kurtosis cli

Background & motivation
Kurtosis releases updates quite often. This is awesome, but this also means users will likely be using out-of-date versions of Kurtosis.
Currently the WARN looks like this:
WARN[2023-08-18T12:14:39+01:00] You are running an old version of the Kurtosis CLI; we suggest you to update it to the latest version, '0.81.9'
WARN[2023-08-18T12:14:39+01:00] You can manually upgrade the CLI tool following these instructions: https://docs.kurtosis.com/install#upgrading

Desired behaviour
When I am using an outdated version of Kurtosis CLI, I would like the terminal to print the following:
WARN[2023-08-18T12:14:39+01:00] You are running an old version of the Kurtosis CLI; we suggest you to update it to the latest version, '0.81.9'
WARN[2023-08-18T12:14:39+01:00] Upgrade using: brew update && brew upgrade kurtosis-tech/tap/kurtosis-cli && kurtosis engine restart. For other ways to upgrade, visit: https://docs.kurtosis.com/install#upgrading

How important is this to you?
Nice to have; this feature would make using Kurtosis more enjoyable.

One step better would be a WARN line that reads
WARN[2023-08-18T12:14:39+01:00] You are running an old version of the Kurtosis CLI.

Would you like to upgrade the Kurtosis CLI? (Y/N):

@leeederek Derek agreed, and this message has bugged me for a little while. Unfortunately, Homebrew isn't the only way users install our CLI. What about this?
WARN[2023-08-18T12:14:39+01:00] You are running an old version of the Kurtosis CLI. To update to the latest (0.81.9), run the following:
     <print specific version information; e.g. `brew update && brew install kurtosis-tech/tap/kurtosis-cli`>

Fixed by https://github.com/kurtosis-tech/kurtosis/pull/1254


Config lang lib  and configurable keyword are not showing as suggestions

Description:
When I type config in the vscode plugin it does not show any import to lang.config module. But for runtime module suggestions are showed as import ballerina/runtime.
Once I manually add import ballerina/'lang.config then when I type conf, it shows the config module and its functions.

Also, there are no suggestions for the configurable keyword as well.
Test Environment:
Ubuntu , Sl alpha 2 pack, and vs code plugin

It seems ballerina/lang.runtime also does not suggest if you type runtime

Fixed with #28657. Hence closing the issue


[otl] Upgrade to version 4.0.448

Related issue: #8794. There are no features of this port need to test.

/azp run

/azp run


Virus detection while downloading in Win11

Virus detection while downloading in Win11
While downloading the latest version, Windows 11 defender is deleting the .exe portion automatically saying "interrupted: virus detected"

similar to #1002 - its a false positive.


microk8s kubeflow install failure due to disk pressure

Summary

Trying to install Kubeflow add-on on Microk8s
What Should Happen Instead?

Successful install
Reproduction Steps

microk8s install
sudo snap install juju --classic3.
juju bootstrap microk8s

Introspection Report

inspection-report-20220616_185221.tar.gz
Can you suggest a fix?

Are you interested in contributing with a fix?

Is there a way to increase disk space provided by microk8s?

Is there a way to increase disk space provided by microk8s? There's like 4T of space on the server left

Seeing that your disk is mounted on the /home folder, you can try the following to move all microk8s data under /home/<username>/microk8s-data.
 Make sure you understand what the commands below are doing before executing them. Proceed at your own risk
# run a shell inside the microk8s snap environment
sudo snap run --shell microk8s

# stop all services, make sure only containerd is running
snap stop microk8s
snap start microk8s.daemon-containerd

# stop all running containers, then kill containerd as well
. $SNAP/actions/common/utils.sh
stop_all_containers
snap stop microk8s

# replace <username> as needed
export TARGET="/home/<username>/microk8s-data"

# move all microk8s data into the target directory, and create a symlink from the previous location
mkdir "$TARGET"
mv /var/snap/microk8s/* "$TARGET"
ln -s "$TARGET" /var/snap/microk8s

# finally, start microk8s. pods may take some time to come up
snap start microk8s

I get a
- Run service command "start" for services ["daemon-apiserver" "daemon-apiserver-kicker" "daemon-cluster-agent" "daemon-containerd" "daemon-control-plane-kicker" "daemon-controller-manager" "daemon-etcd" "daemon-flanneld" "daemon-k8s-dqlite" "daemon-kubelet" "daemon-kubelite" "daemon-proxy" "daemon-scheduler" "daemon-traefik"] of snap "microk8s" (systemctl command [start snap.microk8s.daemon-containerd.service] failed with exit status 1: Job for snap.microk8s.daemon-containerd.service failed because the control process exited with error code. See "systemctl status snap.microk8s.daemon-containerd.service" and "journalctl -xe" for details.
)

I can't remove microk8s either
<EMAIL_ADDRESS>~]$ sudo snap remove microk8s
error: cannot perform the following tasks:
- Save data of snap "microk8s" in automatic snapshot set #9 (cannot create archive:
tar: common/microk8s-data/3272/var/kubernetes/backend/kine.sock\:12379: socket ignored
tar: common/microk8s-data/common/run/containerd.sock.ttrpc: socket ignored)

Sorry for taking to follow up. And sorry x2 because it looks like I have a typo in the original list of commands:
The commands instead would be:
# move all microk8s data into the target directory, and create a symlink from the previous location
export TARGET="/home/<username>/microk8s-data"
mkdir -p "$TARGET"
mv /var/snap/microk8s "$TARGET"
ln -s "$TARGET" /var/snap/microk8s

I've edited my previous comment accordingly for future readers.


[react-router@3] ES6 Classes everywhere. HOC on ContextUtils

Considering React 16 will no longer provide createReactClass in the default package I thought it could be useful to have react-router@3 free of that dependency.
This could reduce the library's footprint (if other libraries starts dropping createReactClass too) and possibly allow react-router@3 to make use of the performance improvements offered by React 16+.
What's in this PR:

We had mixins in ContextUtils.js. I converted them to Higher Order Components instead. The mixins are still exported for now, to keep compatibility with anyone who depended on it (but they shouldn't, really).
I changed the functions which used the mixins and wrapped them around the HOCs.
Rewrote every createReactClass (except for the tests) to use React.Component

While I appreciate the effort, this kind of turmoil in the internals of the code could have unintended side effects that would break our semver commitments. In particular, the HoC is going to cause a change in the component tree, which would likely be confusing to most users because of its visibility (just open up Dev Tools to see).
React.createClass isn't deprecated. Just including it in the core react package is deprecated (mainly so they can start slimming down bundle sizes). I've already suggested changing the wording from "deprecation" to "moving", since that will hopefully make it clear these APIs are sticking around.
Sorry about closing something that you clearly put a lot of effort into. Unfortunately, it's not something I feel comfortable merging in.

No problem. I understand your concerns.
I actually only needed (personally) the refactored ContextUtils thing, and
thought it could be useful for someone else. And after changing it to a HoC
Class it was pretty straightforward to change the createReactClass stuff.
And yeah, I know the package will still be around, but doesn't make much
sense to bundle "duplicate code" into an app.
Em ter, 11 de abr de 2017 às 21:14, Tim Dorr<EMAIL_ADDRESS>escreveu:

While I appreciate the effort, this kind of turmoil in the internals of
the code could have unintended side effects that would break our semver
commitments. In particular, the HoC is going to cause a change in the
component tree, which would likely be confusing to most users because of
its visibility (just open up Dev Tools to see).
React.createClass isn't deprecated. Just including it in the core react
package is deprecated (mainly so they can start slimming down bundle
sizes). I've already suggested changing the wording from "deprecation" to
"moving", since that will hopefully make it clear these APIs are sticking
around.
Sorry about closing something that you clearly put a lot of effort into.
Unfortunately, it's not something I feel comfortable merging in.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ReactTraining/react-router/pull/4957#issuecomment-293436249,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAa64hczSOW3f9u2jW2u0FXKQOXDYTDLks5rvBdogaJpZM4M6ntB
.


⚠ ClamAV has degraded performance

In e4e66c7, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 50 ms

Resolved: ClamAV performance has improved in 2c00bfd.


Convert Xorg from Docbook to CommonMark

This issue tracks the conversion of /doc/builders/packages/xorg.xml from Docbook to CommonMark per RFC 72.
If you intend to work on this conversion, please comment below!
Conversion instructions

Comment here that you intend to work on it. (Please also post if you stop working on it.)
Checkout a new branch of nixpkgs master.
Edit doc/builders/packages/index.xml: change xorg.xml to xorg.section.xml.
Rename doc/builders/packages/xorg.xml to xorg.section.md.
Convert the contents from Docbook to CommonMark. Follow conversion best practices. Use existing .md sections for inspiration.
Use nix-build from the doc/ folder to build the manual.
Preview the manual on your computer at http://<IP_ADDRESS>:8000/manual.html after running from doc/ folder:

nix-shell -p python3 --run 'python3 -m http.server --directory ./result/share/doc/nixpkgs'

When it looks good, submit a PR and reference this issue.

I think I can do this.


Flashed program hangs after power reset

I can't tell if this is an issue with probe-rs but I don't have any better options so I hope somebody can help me understand what's going on here.
Describe the bug
When I flash my code to the board (I'm using a nrf52840 DK), either by using the probe-run tool or cargo flash, it runs just fine for the first time. But if I reset the board by turning the power off and on, my program seems to hang in a seemingly arbitrary place somewhere near the beginning, e.g., if my code sets the color of some neopixels right after startup, the neopixels light up in the respective colors, but nothing after that gets executed (even if I just try to set different colors right after that).
The specific neopixel code in my example above doesn't seem to be at fault – if I remove anything noepixel related, it still hangs somewhere, but I can't tell where.
Interestingly, there's a previous revision of my code that doesn't trigger this behavior – it runs just fine even after a power cycle. Unfortunately, I can't tell what are the specific changes between the revisions that are causing the issue (the hardware part also changes so it's hard to bisect, reversing the most obvious changes in software doesn't help). Also, I think I've experienced this issue previously manifest and later disappear at an even earlier stage in development, so this maybe is somehow related some specific instructions that the compiler uses?
I also tried compiling the code in release and debug modes, but that didn't have an effect.
To Reproduce
I don't have a minimal repro code, unfortunately, but in my project, the steps that I take are as follows:

The project is initialized using https://github.com/knurling-rs/app-template, I've written a fair bit of code, I'm using a nrf52840 DK
Build the code using cargo build --bin some-bin
Flash the program using probe-run --erase-all --chip nRF52840_xxAA target/thumbv7em-none-eabihf/debug/some-bin

Alternatively cargo flash --chip nRF52840_xxAA --allow-erase-all --bin some-bin can be used

Observe that everything is running as expected (RGB LEDs light up in the selected colors, LCD shows text, etc.)
Turn off and on the power button on the nrf52840 DK to reset it.
Observe that program has stopped executing at some point near the beginning of the program (RGB LEDs light up in the selected colors for the first time, but nothing else happens, LCD doesn't show text, etc.)

Expected behavior
Step 6. produces the same result as step 4.
Stacktrace
N/A
Desktop (please complete the following information):

Linux (Arch)

Do you have the RTT buffer set to blocking or override? If it's set to blocking (which I think is default with defmt) it will not be able to write more logs after some time and lock up. Could that be what happens?

It seems that defmt now defaults to non-blocking mode for RTT – https://github.com/knurling-rs/defmt/pull/174  (I'm using defmt-rtt 0.4.0, so I should be fine in this regard), also here's some discussion on how it all works together https://github.com/knurling-rs/defmt/issues/133
Also cargo flash --chip nRF52840_xxAA --allow-erase-all --bin some-bin works fine for me for the first time (things only break after I power cycle), and that doesn't output anything to the terminal, so it probably means that RTT buffers aren't causing the issue (at least not in an obvious way).

It seems the issue is timing related. I managed to make my code run even after I power cycle the device (thereby fixing or, rather, sidestepping the issue) by adding a few delays in my code. Here's the commit that fixed the issue if anyone's interested https://github.com/rihardsk/airlog/commit/613b6b954488ab87f2b5d2ad7b3ccf1141a61585
I guess this makes it clear that probe-rs is not at fault here, but, rather, probe-rs maybe somehow introduced some subtle timing differences which made the bug not appear, though I'm not sure how. I get that flashing and running with probe-run could introduce such timing differences because of defmt blocking on RTT stuff underneath, but how is it possible with cargo-flash which doesn't output anything (and so it doesn't seem to be attaching any probes)?
One possibility that somebody suggested to me is that HFCLK could somehow be relevant here, but I don't yet understand what it is exactly and allegedly there were some evidence against this possibility. I still have to look into this.
I guess this means that this issue should be closed? If so, please feel free to do so. Though I still very much welcome any ideas on what might be going on here, i.e., what is cargo-flash doing that might make the timings different for the issue to go away (or maybe it's something different entirely?). I will close the issue later myself, if there's no activity here.


QSIs allowing containers to be placed in pockets

Description

When done, the bug allows you to have a crate in your pocket that can be opened and closed (and locked), effectively kidnapping people.
Reproduction

Have two QSIs linked
Place one inside of a crate or locker
Get inside of a disposal bin
Activate the QSI
Place it in your pocket before it swaps

Screenshots

Gotta ask it before Kadeo - Can we have pocket ripleys to kidnap people? :trollface:

Spin inverters are so constantly fucked lmao.

was this fixed with the recent PR or is this still a problem

I never checked pockets while making the pr but it does prevent stuff from getting teleported into your hands so I would imagine this is fixed.

hands and inventory are separate systems so probably not if you are using hands for logic and not containers

hands and inventory are separate systems so probably not if you are using hands for logic and not containers

The fix used container logic
#29200

well... I tested this and its still a bug so I guess I can try and fix that...

Silly... yeah, it looks like its the ordering of how it decides what to teleport. It gets its parent container, then the top most container. If the topmost container is anchored, then it just swaps the QSI instead of trying to swap the first container that is not anchored.
https://github.com/user-attachments/assets/031aecee-bae7-4422-9d5d-a6471e7401ef


Fixing CSS issues

Privacy toggler, horizontal align (Fixes #5777).

Grey logo, vertical align.

retest this please

:sparkles:


Blue-Green deployement strategy

Blue/Green deployement strategy
This is feature request.
With blue-Green deployement strategy I have 2 stacks (say stack_a and stack_b) plus a reverse-proxy container that swaps beetween stack_a and stack_b.
The green stack is the stable version (initially stack_a).
While the blue stack is the development version (initially stack_a).
The stack where to deploy is determined at runtime by reading the reverse-proxy current status with this command: docker exec -t proxy_container /status.sh.
After the deploy we can manually swap the blue/green in our reverse-proxy.
After the swap the green stack is stack_b, and the blue stack is stack_a.
I'm awaiting your considerations, opinions, questions or suggestions.
Thanks for everyone.

I think you could probably use rancher-gitlab-deploy to help with this as part of a wider system, but I don't think this functionality belongs in this project directly. The scope of the tool is just to push containers from GitLab into Rancher.
You can guide rancher-gitlab-deploy to update stack_a or stack_b by using the --stack argument.

Do you know how can I run /status.sh on container from inside gitlab? maybe using some external tool?

I'm not sure, sorry.

Maybe ask the question on the Rancher forums.


Two fixes for Docker integration tests

Add a command line argument to run_multiversion.sh which has it start
a docker daemon. This allows it to run directly inside an outermost
docker-integration container instead of running outside all containers.

Install Docker 1.7.1 intead of 1.7.0-rc1 in the docker-integration
container. 1.7.0 has a bug that prevents "docker cp" from working
properly.

Signed-off-by: Aaron Lehmann<EMAIL_ADDRESS>

LGTM


Investigate the template that came with Jesse's repo

Is it hand-built or based on something else? Is there a newer version? et cetera

Is there documentation for Rasmus Andersson's template, @jesserobertson? (A quick Google says no.)

Don't think so. I pretty much just looked through his git repo for info.

Closed by e613ba9 and the move to Bootstrap.

Err, I mean e70aa11.


Witness: add option to encode account with codesize/codehash but witout code

This adds an option to encode an account with codesize and codehash, but without code. This is useful for the case when an account is "touched", but not executed.
However, if we were to change the meaning of Bytecode to "merkleized-bytecode", then it would be preferable to encode codesize and codehash next to the merklized bytecode, otherwise in most cases,  all code needs to be submitted.

@axic feel free to "Squash and merge" after all typos are fixed.

However, if we were to change the meaning of Bytecode to "merkleized-bytecode", then it would be preferable to encode codesize and codehash next to the merklized bytecode, otherwise in most cases, all code needs to be submitted.

@mandrigin @poemm I think it would be useful to mention the above as a note in the spec. What would be a good place to mention it? Alternatively we can track this as an issue?

@mandrigin I don't have rights to merge btw.

I think it would be useful to mention the above as a note in the spec. What would be a good place to mention it? Alternatively we can track this as an issue?

Good idea, we can swap <Bytecode> for <Bytecode_Tree_Node>. I say track this in an issue. Since this may not be ready for version 1, we could also discuss whether it is useful to include some extra dummy bytes in preparation for introducing <Bytecode_Tree_Node> with minimal changes.
Maybe there should be a "Rationale" doc or section to explain the various quirks, warts, and design decisions in the spec.

@axic we have an issue to support merkelized code in this spec: https://github.com/ethereum/stateless-ethereum-specs/issues/11

I thought about this change.
This change removes canonicalness of witnesses -- two witnesses can encode the same thing.
Canonicalness can be recovered with a rule that each bytecode in a block witness must be executed at least once in the block. We need similar rules to enforce completeness and non-bloatedness of witnesses.
I am realizing that because witnesses depend on execution, we may have to specify them together.
I still support merging this because (i) we don't have a written design goal for canonicalness of witnesses and (ii) we will eventually find more optimizations like this and have to resolve their canonicalness. I made a personal note about this and will return to it.

Fixed all comments, should be ready to merge.

thanks @axic!


Ballerina program panics with exit code 0

Description:
When a Ballerina program panics, the exit code returns 0. This should be 1 since the program did not exit normally.
This is a serious issue since we use ballerina (1.2.8) for the standard library release pipeline processes, where the release is failed in the middle, the ballerina program panics, but the pipeline is marked as passed.
Steps to reproduce:
Try the following code:
public function main() {
    error err = error("Sample", message = "Sample Error");
    panic err;
}

$ ballerina run sample.bal
$ echo $?

This prints 0.
Affected Versions:
jBallerina: 1.2.8
Ballerina Tool: 0.8.8
** Related Issues:**
#20967, #21624

The mentioned PR is fixed the issue for Ballerina 2.x. But this should be fixed at 1.2.x branch. Reopening.


Multiplayer + Raycasting?

Is your feature request related to a problem? Please describe.
This is not a problem, just a future feature request :)
We love the multiplayer extension and we love the raycasting extension, and would love to see multiplayer ability available in a game that uses raycasting in the future!
Describe the solution you'd like
Allow for multiplayer in a game that incorporates raycasting!
Describe alternatives you've considered
We tried using it currently, and it is not possible while in raycasting mode.
Additional context
Thanks for considering our request!

@jwunderl Looks like the way the raycasting extension renders the screen doesn't work with how the multiplayer board reads it.

It looks to me like it plays as expected https://arcade.makecode.com/--multiplayer?host=59169-39428-75390-60169, it just looks like another instance of https://github.com/microsoft/pxt-arcade/issues/5113 as this explicitly different views per player to work nicely; when we have multiple cameras we'd have to have player specific renderables & raycasting should be relatively simple to update to work there in the same way any player specific ui would be done.

Here's one that repros for me: https://arcade.makecode.com/--multiplayer?host=S27122-90434-15832-10838

@bmarslandCN what was your repro case?

Ah I see, looks like it must have been an older version of the extension in the link I shared, it now digs a bit deeper than I expected and is overriding the actually place where we call into the sim to render: https://github.com/AqeeAqee/pxt-raycasting/blob/master/render_raycasting.ts#L395-L403 I'm not sure I see the benefit in that right away -- it looks like it's done when swapping between top down / raycasting view, but that should work fine to either render directly onto screen or as a renderable without too much change.

I believe the multi-cam is what is needed - I made a simple tilemap maze then used the raycast and multiplayer extensions to try creating a multiplayer version, but only one player was able to use the "myself sprite" function to navigate the raycasted tilemap. The other players had blank screens until the raycasted player toggled the view to the regular tilemap, then all players could see and navigate through the tilemap.

Sounds like a fix should be made in the raycasting extension, if I read that right.

Hi all ! Sorry for "late reply", I just happened to this.
Yes, as @jwunderl said to support this feature completely, we need multi cam.
The bug shown in @eanders-ms test project, is because I invoked a tempScreen, and update to low level Screen directly (skiped pasting it to the screen img object then update the low level Screen by arcade), for perfermence considering.
Hi @bmarslandCN
A quick fix could be:
game.onShade(()=>screen.drawImage((Render.raycastingRender as any).tempScreen,0,0))
You can add this line to anywhere of your code.
Glad to hear you like my Raycasting extension. And, actually I am not a team member of Arcade. You can contact me on Raycasting 3D render – Blocks Edition about this extension, or pin me at forum at any time for any question.
@jwunderl
The reasons why invoke tempScreen instead of rendering on the screen are:

The particle feature draw doesn't have a target parameter, it can only draw directly onto screen img.
       particle.__draw(this.camera)
Screen has contents rendering for current frame already at that time. So calling particle.__draw() for each sprite, then blit them from screen to tempScreen.
Blit these 2D drawing output(particles or sayText or etc. ) of arcade features to 3D rendering (with some transform parameters), we need another surface, that is the tempScreen.
As mentioned, for perfermence, I didn't blit tempScreen back to screen. This would be a little faster, especially on hardware consoles. But this really leads to some issues, like this. I had tried to write them in renderable or sprite.update/draw pattern, it's possible and looks better, but abondant finally for a little perf impact. But nowadays, thing like sim in high res and online features more and more pop. Less and less guys play on console, I feel. So maybe refator them in next release.


Update for inclusion in notebook

Depends on https://github.com/spyder-ide/pywinpty/pull/62.  Tested locally in the notebook using https://github.com/blink1073/notebook/tree/win-tornado.  Here's a screenshot with cmd.exe (the default on Windows if there is no SHELL set.

Here it is with export SHELL=`which bash`:

cc @jasongrout

Wow - super excited about this! I haven't had time to look at this repo as yet, but I'll make time if there's any testing or otherwise that might help get Windows terminal support into JupyterLab.
I'm running JupyterLab on Windows Containers via JupyterHub so happy to test on my setup.

See https://github.com/takluyver/terminado/pull/44 for a better option.


[d&d][Table]Unexpected error show when accept drop from 'Metric' in 'Percentage Metric'

Unexpected error show when drop from 'Metric' in 'Percentage Metric'
Expected results
No error show
Actual results
Unexpected error show when drop from 'Metric' in 'Percentage Metric'
Screenshots
https://user-images.githubusercontent.com/81597121/127795512-1d5c1b6a-cfbf-47b4-ae22-5af20d339e21.mov
=
How to reproduce the bug

Open Table 'Breakdown of developer Type'
Drop COUNT(Developer_type) to Metrics from left column
Drop COUNT(First time developer) to Percentage Metrics from left column
Drop COUNT(Developer_type) from Metric to Percentage Metrics
see error

Environment
(please complete the following information):

superset version: master
python version: python --version
node.js version: node -v
any feature flags active:

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[ ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
Add any other context about the problem here.

this is a feature request, not a bug. but we should have better indication for non-supported cross field

Feels to me that Unexpected errors are bugs

Feels to me that Unexpected errors are bugs

For this stage, the expected behavior should be to highlight cross field dropping in Red. but we can certainly skip to make it support drop. @mistercrunch

@junlincc I bet that's the same problem as with dragging column from rows to columns control. Of course it's a bug, but I'm fairly sure that it used to work as expected

@junlincc I saw 'Filter' can accept drop from other fields, is this expected? should i field bugs for that?

Saw same issue in example chart 'Parallel Coordinates' chart, when drag and drop "Metrics' to "color metrics'
https://user-images.githubusercontent.com/81597121/127830289-8239f789-af0c-45dd-bb54-7c4134f2f395.mov

@jinghua-qa in general the issue occurs when we have 2 controls of the same type. The fix is in progress

@junlincc I saw 'Filter' can accept drop from other fields, is this expected? should i field bugs for that?

Could you please show a recording/steps to reproduce? Filter works fine for me

Fix: https://github.com/apache/superset/pull/16020


[Feature request] Support --host flag for development server

Hi.
All the sparks commands works great, but one that I'm still running with CLI is the local development server with --host flag. I run multiple client projects and some of them use subdomains, so the only way to support them is setting my own hosts.
It could be great if the extension can support this feature!
CodeIgniter 4 host CLI option

Hi Alex,
The last update of this extension was in CI 4.4. I have to update it to add some new commands from CI 4.5 version.
I am going to do some testing to see if I can add this functionality.

Hello @Alex-Licea,
Sorry for the delay!
I recently released an update package that includes this flag. While some of the improvements do not directly affect the client side, they will contribute to more efficient code handling and facilitate future optimizations.
If you have any ideas or suggestions for new features, feel free to share them.
I'll be willing to consider new features to improve the extension.


Add webhook handler to update PR workload queues

This is the third step to implement tracking the pull request workload designed in #1753
In this patch we enable handling GH webhooks when a PR is assigned or unassigned.

@jackh726 I've removed a lot of code ^^, now I have a better understanding. PR is ready for a first review when you have time. Thanks!

thank you @GuillaumeGomez it looks much better now :)

@apiraino can you make a PR to rust-lang/rust to add the config option to the triagebot.toml


Preview on filer is limited to 10 lines

Environment (please complete the following information):

OS: MacOS
Vim version: 8.2.2500
vim-clap version: master
Have you reproduced with a minimal vimrc: No
Have you updated to the latest plugin version: Yes
Have you upgraded to/compiled the latest Rust binary: Yes

Describe the bug
It seems like the preview pane in filer only shows the first 10 lines of the file when hovering on them.
Clap debug
            has cargo: 1
            has maple: /Users/amousavi/.vim/plugged/vim-clap/target/release/maple
           maple info: version 0.1.23 (git v0.23), built for x86_64-apple-darwin by rustc 1.50.0 (cb75ad5db 2021-02-10).
         has +python3: 1
has py dynamic module: 1
            has ctags: ctags with JSON output support
    Current sync impl: Lua
     Current FileType:
Third Party Providers: []
       Global Options:
    let g:clap#autoload_dir = '/Users/amousavi/.vim/plugged/vim-clap/autoload'
    let g:clap#popup#display = {'shrink': function('164'), 'open': function('<SNR>112_create_display'), 'width': 81, 'shrink_if_undersize': function('163')}
    let g:clap#popup#preview = {'line_count': function('<SNR>111__line_count'), 'show': function('165'), 'hide': function('166'), 'clear': function('167'), 'add_highlight': function('168'), 'get_lines': f
unction('<SNR>111__get_lines'), 'getbufvar': function('<SNR>111__getbufvar'), 'setbufvar_batch': function('<SNR>111__setbufvar_batch'), 'setbufvar': function('<SNR>111__setbufvar'), 'winid': 1018, 'win_is
_valid': function('<SNR>111__win_is_valid'), 'goto_win': function('<SNR>111__goto_win'), 'set_syntax': function('169'), 'bufnr': 6}
    let g:clap#provider_alias = {'gfiles': 'git_files', 'hist:': 'command_history', 'hist/': 'search_history'}
    let g:clap_background_shadow_blend = 50
    let g:clap_disable_bottom_top = 0
    let g:clap_disable_matches_indicator = v:false
    let g:clap_disable_optional_async = v:false
    let g:clap_disable_run_rooter = v:true
    let g:clap_enable_background_shadow = v:true
    let g:clap_enable_debug = v:false
    let g:clap_enable_icon = 1
    let g:clap_forerunner_status_sign = {'done': '•', 'running': '!', 'using_cache': '*'}
    let g:clap_indicator_winid = 1019
    let g:clap_insert_mode_only = v:false
    let g:clap_layout = {'relative': 'editor'}
    let g:clap_multi_selection_warning_silent = 0
    let g:clap_no_matches_msg = 'NO MATCHES FOUND'
    let g:clap_open_action = {'ctrl-v': 'vsplit', 'ctrl-x': 'split', 'ctrl-t': 'tab split'}
    let g:clap_open_preview = 'always'
    let g:clap_popup_border = 'rounded'
    let g:clap_preview_direction = 'LR'
    let g:clap_preview_size = 5
    let g:clap_providers_relaunch_code = '@@'
    let g:clap_search_box_border_style = 'nil'
    let g:clap_search_box_border_symbols = {'nil': ['', ''], 'curve': ['', ''], 'arrow': ['', '']}
    let g:clap_spinner_winid = 1021
    let g:clap_theme = 'dogrun'
  Provider Variables:
                     **[]**

Increase the preview size to fit your screen size, e.g., let g:clap_preview_size = 10 will show 20 lines in the preview window.

In https://github.com/liuchengxu/vim-clap/commit/b6d82bc748a80577a31f2a40eb36947d70197a67 the default behaviour has been changed to fulfil the whole preview window, please pull the latest commit and compile the Rust binary.

Works great! thanks 👍

I just realized I have the same issue with buffers. Could be cool to have defaulted to window window height as well.

Should be resolved in https://github.com/liuchengxu/vim-clap/commit/9bb839bd9c5a5130a1794583741cbcf6406eb074 @arashm

Works perfectly 👍 Thanks


[ LUMI-184] Build lumigator before tests

What's changing
This runs both integration test suites against the current code. Not the latest tag (which currently represents the latest code built, and as such could literally be any branch)
How to test it
Run pytest in backend and sdk projects

Thanks a lot @chainlink 🙏 I very much like that now this is doing what make local-up was before.


did not show help for node js modules

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.60
OS Version:

Steps to Reproduce:

did not show help for node js modules
both vscode and vscode insiders

what did you used ?
it must add
import { join } from "path";
instead of create function

we should never update vscode


Change UI component of GKE end-to-end guide to not use a separate IP address

The initial version of the GKE end-to-end guide (PR #151) uses a separate IP address for the web UI.
Since we go to the trouble of setting up a static domain earlier on, we shouldn't use a separate IP address in the web UI step. One approach could be to add a service definition similar to tf-hub-lb or central-dashboard.

@amygdala I'm tempted to close this issue as obsolete. WDYT? Here's the doc and the related sample README

yes, IIRC that got addressed... let's close it.

/close


NONE: Add UI and styling for the admin connect team flow

Describe
Adding the screen for connecting a figma team. Upon success, it will render the success screen. Upon error, right now it will show the permissions error. We should update the /admin/teams/:team_id/connect route to return a 403 on when it fails to connect the team due to permissions errors and 500 for other errors when appropriate.
Screenshots:
Connecting a team:

When inputting an invalid URL:

When there's an error:

Success:

Notable changes:

updated /admin/teams/:team_id/connect to return a teamSummary in the success case since we want to render the team name of the team we just connected in the success message

Things to flag:

I'm not sure if I need to be added to a feature flag for this to work but I was getting 415 errors with await jiraService.setConfigurationStateInAppProperties() which was recently added to connect-figma-team-use-case.ts. I just commented that part out when testing this flow

Test Plan:

open the configure page
try to connect a random url
see that it fails inline
try to connect a figma team url you don't have admin permissions for
see that it fails with the permissions section message
try to connect a figma team url you do have the permissions for
see that the success screen is rendered upon completion
see that you can close this page with the x, exit setup, and see connected screens

LGTM thanks @Roystbeef

ugh i hate pending comments - thought I replied yesterday lmao


Create SB Liquidity Filter

We need a simple filter to notify Market Creator when SB liquidity is empty (also to be used for SB Mega template)
B/c SB users can only take....the Market creator might only want to add "Offers" only....this would fail the Trading/exchange liquidity spread filter but could still pass SB.
Could be as simple as tweaking tje current liquidity spread filter for trading:

Have offers on every outcome (except invalid)

@joeykrug is this doable.....does the current filter work if every outcome has offers on all outcomes (excluding invalid)......this did seems to fail when I tested it in market creation, so need confirmation if this should pass.

closing using amm instead


Allow custom hex colors for elements without having to add colors to Tailwind config

Description
It would be amazing to have color props of elements such as <UButton/> to accept hex color values. Example below:
<UButton
    block
    variant="solid"
    color="[#e3e34f]"
/>

Taking a look at component configs, this approach could currently work out of the box if colors weren't injected as bg-{color}-500. It is honestly very tedious to write in-place :ui props because you have to dive lot deeper to simply change a color.
Additional context
No response

This is unfortunately not possible, if you look at the config of a button it uses multiple shades of the color for the hover and focus states alongside the different variants: https://github.com/nuxt/ui/blob/dev/src/runtime/ui.config/elements/button.ts#L56

what could be done to color a single button in place? my :ui prop has stopped working after upgrading to 2.11.0. I guess I'll have to create a color inside tailwind config for this.
{color: {black: {solid: 'bg-[#4285F4] text-white'}}}

You might need to also set the strategy: 'override' in the ui prop, otherwise classes will be merged and it won't work on dark mode as you didn't override it.
https://ui.nuxt.com/dev/getting-started/theming#ui-prop

that works. thanks!


use getrandom(2) to securely generate random numbers

As discussed in #12, this is a more secure method of generating random numbers while also being relatively simple and not requiring any dependencies

The way getrandom() works is that it simply fills a buffer with the requested number of random bytes as opposed to actually returning a number.

The way getrandom() works is that it simply fills a buffer with the requested number of random bytes as opposed to actually returning a number.
In order to actually get a number, we first pass an integer array as our buffer to getrandom(), and then read off each integer one by one. You could call getrandom() every single time you generate a number, however this would be needlessly slow as syscalls can be quite expensive, which is why the array has been made large enough to store several integers. Each time we call our function, it simply increments an internal pointer and returns the next array element (only calling getrandom() and resetting the pointer once we reach the end of the array).

By the way, you can add me on discord if you want to have a better discussion about the source code, and I will need your opinion about some things here.


Extract emulated instruction in their own function

… easier way

FYI I will move this commit on top the virt module refactoring.

Merged as part of  #275, closing this PR.


Module typing don't get picked up before I open it in editor manually.

Version: 1.37.1
Commit: f06011ac164ae4dc8e753a3fe7f9549844d15e35
Date: 2019-08-15T16:16:34.800Z
Electron: 4.2.7
Chrome: 69.0.3497.128
Node.js: 10.11.0
V8: 6.9.427.31-electron.0
OS: Darwin x64 18.7.0
Steps to Reproduce:

Clone the sample project.
npm install
Open the index.ts, and the Intellisense won't get typings picked up.
Find the index.d.ts from the node_modules and open it in another tab.
Back to the index.ts and the auto import get working.

Does this issue occur when all extensions are disabled?: Yes

This is a configuration issue. VS Code will not automatically try loading all globals from node modules as this would be very noisy
As a first step, try creating a jsconfig.json


Hey!

Hey guys, why didn't you say something? :D This is awesome!!!

Some parts could still use work (especially documentation - there's more to this than what's in the README), but I'm glad you like it!
We mentioned y'all here, but I guess it got lost in the void :P

Oh, my bad. I just read the apidoc: "Please let us know when you use the API via email<EMAIL_ADDRESS>lol
Would you be okay if we make a PR adding this repo to the API doc for discoverability once this project (and its documentation) is in a more complete state? It isn't documented, but there's also an "api" module providing a simple wrapper for the JSON API in Retrofit/Rx that we plan to maintain (though it'd be nice if Urbandroid could maintain their own, it's also simple enough that it doesn't really need one, idk).
I'm having trouble deciding how to build the second part of this project; it wasn't intended to be just a simple frontend for DKMA. Ideally, I wanted to provide some extensible implementations of the developer workarounds listed on the site, as well as a series of "steps" to be presented to a user that open the relevant settings screens to work around issues. That seems less maintainable in the long run, though, as many of these steps only apply to specific devices :P


feat(mat-tabs): override material to remove tabs label min-width

Issue
https://github.com/gravitee-io/issues/issues/6765
Description
Remove unnecessary min-width inside mat tabs to have smaller tabs when the text is not large
Additional context
cockpit seems to have the same feedback. what do you think about doing it globally ?
https://github.com/gravitee-io/gravitee-cockpit/blob/82cefaa5d99ada4d5907ada13ba813e7add31511/gravitee-cockpit-ui/src/app/designer/designer-model-edit/design/components/inspectors/inspector.scss#L17

[x] Validate this with Cockpit teams

📚  View the storybook of this branch here

Gravitee.io Automatic Deployment
A prerelease version of this package has been published! 🚀
To use it in your project run the following lines:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

:tada: This PR is included in version 2.5.0 :tada:
The release is available on:

npm package (@latest dist-tag)
npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Skid And Pump Dance Right And Dance Left is ERROR

This Is Same As Issue OF GF Anims Stuck OR Something Else

explain

fixed in https://github.com/KadeDev/Kade-Engine/commit/d707c5efc6057b5221c974459ddb125dbbf24c03


Z_ISR_DECLARE: update for C++ support

The implicit conversion of pointer-to-function to pointer-to-void is
not acceptable in C++.  Provide a C-style explicit cast to force the
reinterpretation.
Fixes #24322

This is not a complete fix. It causes issues with ISR_DIRECT_DELCARE, whose created function is int (*)().

An example of how to use ISR_DIRECT_DECLARE Taken from https://docs.zephyrproject.org/latest/reference/kernel/other/interrupts.html#c.IRQ_DIRECT_CONNECT
ISR_DIRECT_DECLARE(my_isr)
{
   do_stuff();
   ISR_DIRECT_PM(); /* PM done after servicing interrupt for best latency */
   return 1; /* We should check if scheduling decision should be made */
}

void my_isr_installer(void)
{
   ...
   IRQ_DIRECT_CONNECT(MY_DEV_IRQ, MY_DEV_PRIO, my_isr, MY_IRQ_FLAGS);
   irq_enable(MY_DEV_IRQ);
   ...
}

Implementation of the macro ISR_DIRECT_DELCARE:
#define ARCH_ISR_DIRECT_DECLARE(name) \
	static inline int name##_body(void); \
	__attribute__ ((interrupt ("IRQ"))) void name(void) \
	{ \
		int check_reschedule; \
		ISR_DIRECT_HEADER(); \
		check_reschedule = name##_body(); \
		ISR_DIRECT_FOOTER(check_reschedule); \
	} \
	static inline int name##_body(void)

Seems like ISR_DIRECT_DECLARE needs an extern "C" added conditionally.  But... how much do we really care about this?

This is not a complete fix. It causes issues with ISR_DIRECT_DELCARE, whose created function is int (*)().

I'm unable to confirm that.  I tested the fix with both IRQ_CONNECT and IRQ_DIRECT_CONNECT examples from your original description, and with the new example (after providing defines for MY_FOO), and get no diagnostics.  Both variants end up using the same macro where the fix occurs.  (I can't confirm that it works, just that it builds.)
Could you create a gist with a complete example that shows the problem?

My apologies, looks like my error was an artifact of a non-pristine build. My test main.cpp files from the issue compile.

Seems like ISR_DIRECT_DECLARE needs an extern "C" added conditionally.

This shouldn't be necessary since fact-of IRQ is indicated by a function attribute, not the spelling of the external linker symbol.

Seems like ISR_DIRECT_DECLARE needs an extern "C" added conditionally.

This shouldn't be necessary since fact-of IRQ is indicated by a function attribute, not the spelling of the external linker symbol.

Not to mention that it is already wrapped in an
#ifdef __cplusplus
extern "C" {
#endif


How to use it with: Angular2 Seed

How can i use ng-bootstrap with Angular2 Seed

You can check out the examples part on angular2-seed project there is at least one using ng2-bootstrap.

@joelmuskwe can I close this issue?


cilium

List the direct Git links to be added to our Augur database
https://www.cncf.io/projects/cilium/

I would be interested in this as well :)

closed as its collected into the db


Queueing DiskIO can lead to memory overflow

The disk write queueing behavior can be dangerous when the images are being cached at set intervals.  The problem is that if the workload was big enough to cause queueing once, it is likely to need it again on a subsequent run, and the system will get further and further behind.  The queueing causes SDWebImage to keep a reference to the images in memory, meaning that we run out of memory.
Example that causes problem:
Image Size: 1920x1080
Number of Images: 20
Batch Interval: Every 1.5 seconds
Problems occur for us after 2 minutes of runtime

You can disable the memory caching and force SDWebImage to load from disk-cache everytime. Set SDImageCache shouldCacheImagesInMemory to NO.


Update ru-RU 04.03.17

Please remove the repeated lines, if possible. For example, many lines "all".

Need the latest "commit" and the name "Update ru-RU 4.3.17".
something I did not understand how to send only the latter. I hope you will understand.


Correctly handle tests --name '.' shortcut

Respects virtual cases but doesn't select other tests named as prefix of
the current directory
Fix #1185

Looks good, thanks for fixing this. I was just not sure about the extra translate_dot() method. Wouldn't it be more straightforward to just extend the path in the proper condition branch? Suggested in 60a2694.

Well, I understand but then it is not path (as tmt attribute) but kind of 'replacement for dot'. I don't mind change though, feel free to squash.

Also removed rm "$rlRun_LOG" lines there as beakerlib now takes care of the log file cleanup. Please, have a look if everything's ok.

Cool, I didn't cache this beakerlib change, very good to know. Much easier now

I see. Would s/path/pattern/ make this a bit more clear/selfexplanatory name for the variable?

Very good name


Git commands silently fail

When I try pushing to a remote branch that is ahead of my local branch, I get no feedback that this is the case. I think this is happening for most commands that error as I tried rebasing onto a branch using neogit and it did the same thing.

If a command fails you should be able to see the output in the command history.

Ah nice. I think it would be nice to have a notification that it failed in the same way we get the success notification? It's mainly a problem for me as I work on a huge mono repo and the push/pull sometimes takes a while so I never know if it's still pushing or if it failed.
Also I didn't say in my initial comment but thanks for the great plugin! :D

Ah nice. I think it would be nice to have a notification that it failed in the same way we get the success notification?

If the command returns an error code we actually display a notification, so it is weird that you didn't get a notification 🤔
What is the return code in the command history? (The leftmost number)

Oh interesting! I'm out at the moment but I'll try and repro it when I get back

OK I can't repro this anymore :confused: Sorry for the time wasting!


[Bug] Copy to compared file

When attempting to copy files that exist either "only in my folder" or "only in the compared folder", all files are currently being copied instead of only copying the files that do not already exist.

Thanks for reporting!
I'll check that.


Make it hard to submit a blank accidentally

A user reported answers not appearing in the script, sometimes.
I believe that this happened when someone clicked "submit this one" with nothing entered in the box, for a lib they had already answered.
Make this hard to do from the front end, since this is probably not what this user intended.
Or otherwise fix it.

I think doing gh-521 will accomplish this.

TODO

[ ] Add a separate button for already submitted libs, like Blank out my answer.
[ ] Explain in the footer that a funny default will be used for anything left blank.

It looks like my builds are failing because my GitHub Actions role can't delete what it needs to in S3.
delete failed: s3://douglasnaphasmadliberatio-frontendcreatehaggadahbu-fljgub7temb1/create-haggadah/_next/static/qlluQHxPhz5kceKAcRib0/_buildManifest.js An error occurred (AccessDenied) when calling the DeleteObject operation: Access Denied

You should only be able to submit a blank by clicking a "blank this one out" button.
That button should only appear if there is an already-submitted answer.
It should appear beside the already-submitted answer.
If there is an already-submitted answer, the Submit button should say "update answer." The explanatory text should say "update this answer and advance to the next one," or "update this answer" if we're on the last one.
If no text is in the input box, the "Submit" button or "update answer" button should be disabled.


Add travis-ci support, including OSX builds

A travis-ci account has been created and linked with this repo, but no builds are running because there is no .travis.yml configuration.
Exit Criteria: A .travis.yml configuration is added that enables Rust CI best-practices & OSX support with Travis CI, and builds are passing.
Rust CI best-practices:

https://docs.travis-ci.com/user/languages/rust/

For inspiration:

https://github.com/sharkdp/hyperfine/blob/master/.travis.yml

WIP https://github.com/jasonpeacock/led-bargraph/tree/travis-ci-1


Ansible support to multiple distros

See #389

I think there's more to do than just this. You'll need to conditionally check fedora and Ubuntu on more things like.. selinux

Closing this issue, I am still open to PRs but they need to be more complete than this.

Hi @onedr0p, I'd like to finish this feature. Could you please reopen this PR?

You will have to open a new PR it seems.


fix the issue of text being cut in TextView with margin

Thank you for your contribution!
The passed value for the isWrapRequired method needs to be fixed indeed, but this doesn't fix the #62.
The children used as an example in the #62 don't have margins.

Also is it possible to add a test case in the FlexboxAndroidTest, the test would fail unless the change in this PR is merged?

the argument "isHorizontal" I add is to support the directions requirment, u know the this layout support both Horizontal and Vertical, which takes diffrent params into calculation. Hozizontal mode need leftMargin and rightMargin, while Vertical mode need topMargin and bottomMargin,so this argument is needed.
As to the unneeded format, i don't notice them until i make the pull request, can u just
merge the useful part and ignore the unneeded part?

And this does solve the text being cutted problem.

i have updated the code:

remove the unneeded changes;
remove the extra argument i added.
the code fix the issue of text being cut in TextView with margin, which is different with #62 ,sorry for the wrong title, but the bug of text being cut does exit and this will fix it.

My question:

Where does this unneeded changes (extra spaces) come frome? as i have done as you said to set AndroidStyle as my project style and format it.

Thanks for the update.
Left some comments.

My question:

Where does this unneeded changes (extra spaces) come frome? as i have done as you said to set AndroidStyle as my project style and format it.
Please make sure you imported the correct format file.

And how about a test?

Also is it possible to add a test case in the FlexboxAndroidTest, the test would fail unless the change in > this PR is merged?

@thagikura

Thanks for your advice, i have updated the code with adding the margins to the fifth argument of isWrapReuiqred and leave it unchanged.
Continue my question of the Style Problem, could u tell me the specific name of the "extra spaces" happen to me in the previous code? which i can go to check the style and find where is the problem.
As the test case, you mind give me a hint how to add it ?

Hi,
Thanks for your continuous effort.
For 2, sorry I don't understand about the question 2. The #62 should be fixed by #70. And as I commented in the code review, this PR is a different (potential) issue where there is a flex line wrapping problem in the case flex line wrapping doesn't happen (but should've happened) because of the child's horizontal margin (or vertical margin) isn't taken into account when judging if a wrapping is required.
For 3, please refer to the existing FlexboxAndroidTest under the flexbox/src/androidTest.

@thagikura  i have updated the code with a test case

@keithlanding
Thanks for the update.
I'm so sorry... after another consideration, I realized #70 wasn't a correct fix for #62.
And realized #74 is the right way to fix it.
The problem was when judging if a new flex line is required (isWrapRequired method), the padding values from the parent and the margin values from the child weren't taken into account.
The change in #74 overlap the change in this PR.
(Actually #62 was caused by the missing padding values from the parent, but child's margin values were also missing for the isWrapRequired method)
So I'd like to merge the #74 and close this PR.
Apologies for not being able to merge this PR. At first I thought the change in this PR didn't overlap my PR.. I appreciate if you understand the situation.
Thanks.

@thagikura Got it, go ahead, fixing the issue is what matters.


firebase_auth build warnings

#37230 related.
[00:35:48]: ▸ Compiling FirebaseAuthPlugin.m
[00:35:48]: ▸ ⚠  /Users/jon/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.14.0+1/ios/Classes/FirebaseAuthPlugin.m:161:40: incompatible pointer to integer conversion assigning to 'BOOL' (aka 'signed char') from 'id _Nullable' [-Wint-conversion]
[00:35:48]: ▸ actionCodeSettings.handleCodeInApp = call.arguments[@"handleCodeInApp"];
[00:35:48]: ▸                                                                         ~~~   ^~~~~~~~~~
[00:35:48]: ▸ ⚠  /Users/jon/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.14.0+1/ios/Classes/FirebaseAuthPlugin.m:164:47: incompatible pointer to integer conversion sending 'id _Nullable' to parameter of type 'BOOL' (aka 'signed char') [-Wint-conversion]
[00:35:48]: ▸ installIfNotAvailable:call.arguments[@"androidInstallIfNotAvailable"]
[00:35:48]: ▸                                        ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
[00:35:48]: ▸ ⚠  /Users/jon/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.14.0+1/ios/Classes/FirebaseAuthPlugin.m:204:10: incompatible pointer to integer conversion initializing 'BOOL' (aka 'signed char') with an expression of type 'id _Nullable' [-Wint-conversion]
[00:35:48]: ▸ BOOL refresh = [args objectForKey:@"refresh"];
[00:35:48]: ▸                                       ^
[00:35:48]: ▸ ⚠  /Users/jon/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.14.0+1/ios/Classes/FirebaseAuthPlugin.m:72:1: conflicting return type in implementation of 'application:didReceiveRemoteNotification:fetchCompletionHandler:': 'BOOL' (aka 'signed char') vs 'bool' [-Wmismatched-return-types]
[00:35:48]: ▸ - (bool)application:(UIApplication *)application
[00:35:48]: ▸          ^         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
[00:35:48]: ▸ Compiling firebase_auth-dummy.m
[00:35:48]: ▸ Compiling FirebaseAuthPlugin.m
[00:35:48]: ▸ ⚠  /Users/jon/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.14.0+1/ios/Classes/FirebaseAuthPlugin.m:220:63: implicit conversion loses integer precision: 'long' to 'int' [-Wshorten-64-to-32]
[00:35:48]: ▸ [NSNumber numberWithInt:expirationTimestamp],
[00:35:48]: ▸ ^  ~~~~
[00:35:48]: ▸ ⚠  /Users/jon/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.14.0+1/ios/Classes/FirebaseAuthPlugin.m:221:78: implicit conversion loses integer precision: 'long' to 'int' [-Wshorten-64-to-32]
[00:35:48]: ▸ @"authTimestamp" : [NSNumber numberWithInt:authTimestamp],
[00:35:48]: ▸                                       ~                       ^~~~~~~~~~~~~~~~~~~
[00:35:48]: ▸ ⚠  /Users/jon/.pub-cache/hosted/pub.dartlang.org/firebase_auth-0.14.0+1/ios/Classes/FirebaseAuthPlugin.m:222:82: implicit conversion loses integer precision: 'long' to 'int' [-Wshorten-64-to-32]
[00:35:48]: ▸ @"issuedAtTimestamp" : [NSNumber numberWithInt:issuedAtTimestamp],
[00:35:48]: ▸                                                      ~                       ^~~~~~~~~~~~~

@lookfirst Please provide  a small self-contained app that demonstrates the problem. You can include small examples inline if they're enclosed by backticks, or just provide a public link to a github "gist" - https://gist.github.com/.
Also please provide your flutter doctor -v output.

@BondarenkoStas I just gave you the output of the compile errors. How about you show me the CI output that is a clean build.
Oh and I can't even find the source code anymore since someone moved it and didn't update the documentation for where it is now.
https://github.com/flutter/plugins/commit/3ad2d1d300b0d901ea9739e3b9f08eeebda8be04

@lookfirst
This issue has been moved to https://github.com/FirebaseExtended/flutterfire/issues/1083. Any further collaboration will be done there.


Support for CoAuthoredBy

It would be quite nice if this repo used "CoAuthoredBy" for commits.
Why?  Because if it did,.each BlueSky used added via GitHub issues would count as an additional contributor to this repo.
You should be able to use https://GitHub.com/StartAutomating/ugit to accomplish this automatically ( though you'd also be able to add the trailers yourself if you wish ).
Please let me know if you'd like a PR.

@StartAutomating would love to add this. Happy to take a PR.
I didn't even know this was possible!


Add gfx1201 to supported ARCH list

[misc] Add gfx1201 to supported arch list

@qli88
You are adding ARCH GFX 12 but are you sure that it's compiling fine on ROCm stack?
We have https://ontrack-internal.amd.com/browse/SWDEV-503927 issue when we compile GFX12 with GFX942
Thanks.

@qli88 You are adding ARCH GFX 12 but are you sure that it's compiling fine on ROCm stack? We have https://ontrack-internal.amd.com/browse/SWDEV-503927 issue when we compile GFX12 with GFX942
Thanks.

It is verified to compile and work.
The issue you're mentioning is about not being able to mix gfx942 and gfx12x in the same build. You have to pick one and either will compile and work. This is a current limitation due to them using different hardware FP8 formats.


quiche: destroy file event before close socket during connection migration.

Signed-off-by: Dan Zhang<EMAIL_ADDRESS>The old socket should be closed after corresponding  file_event is destroyed. Otherwise the socket might be picked up by other socket() call while file_event is still operating on it. It causes TSAN failure in QuicHttpIntegrationTest.ConnectionMigration: https://dev.azure.com/cncf/envoy/_build/results?buildId=41584&view=logs&j=5eb754ee-449a-545e-2b62-07fe525d7cbc&t=1110550d-5f5f-55a9-69fd-d02adf65e59a&l=4016
Risk Level: low
Testing: tested QuicHttpIntegrationTest.ConnectionMigration under tsan

Nice catch!  Mind adding a similar comment in the code, explaining why the file event needs to be closed where it is?  Otherwise LGTM!

Nice catch! Mind adding a similar comment in the code, explaining why the file event needs to be closed where it is? Otherwise LGTM!

done


Issue 381: Update version to 0.3.2

Change log description
Update version to 0.3.2
Purpose of the change
Fixes #381
What the code does
Update version to 0.3.2 to prepare for release.
How to verify it
All tests pass

As the github actions build is failing, I locally run the tests with the branch.
test result: ok. 53 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 6.34s


Uncaught SyntaxError: Invalid or unexpected token error when running the app locally through ngrok

Issue summary
So just this week when i run "shopify app serve"
i get my ngrok link with /login instead of /auth
i'm guessing that the /login came from me updating the shopify cli
but when i change the url to /auth now my app does not load
i tried deploying it on heroku, it works on there
so my guess is there is an issue with ngrok
also in the console it shows the error "Uncaught SyntaxError: Invalid or unexpected token at..."
basically it cant parse the file that it loads.
Expected behavior
So i've been using the same setup for about 6 months and i dint change anything i only updated the shopify cli since i thought tt might solve the above issue, i updated it from 2.x to the latest 2.x
expected  behavior was that it would just load the app
Actual behavior
the page just stops loading cause it can't parse the file

was there some change in ngrok that i missed is anyone else facing this issue ?

Okay so some updates, this issue was mainly getting caused due to ngrok timing out the requests
i tried this app by switching to localtunnel (https://theboroer.github.io/localtunnel-www/)
this does work but i link ngrok since i dont have to manually setup the tunnel stuff, i'll contact ngrok about this issue and see if i can find some other solution


Lasers

Hello @DavidDierickx! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file piescope/lm/detector.py:

Line 5:1: E302 expected 2 blank lines, found 1
Line 38:1: E305 expected 2 blank lines after class or function definition, found 1

I am not sure what tests to add for the serial_write and get_frame.

I'd also like your opinion on whether the laser stuff should have it's own class. Let's discuss

Nice work @DavidDierickx! All checks are green, so I'm going to merge this now.
Like we talked about on Monday I've put the laser code into a class.
laser.py now has two functions:

initialize_lasers() enables all four lasers. If you don't have COM6 available, then you'll need to connect to a different port and pass the pyserial object in as an argument to this function.
connect_serial_port() creates a pyserial Serial() object that you can pass into initialize_lasers()

and one Laser class, with methods:

enable()
emit(duration)
disable()

and Laser class attributes:

laser_power percentage laser power. They are initialized at 1% power, and you can adjust it in increments of 0.01 (everything is rounded to 2 decimal places). Float values are fine here, we should test that on the hardware too
WAVELENGTH is the wavelength in nanometers
NAME is the laser name, eg: "laser1"
SERIAL_PORT is the pyserial Serial() object the laser will write commands to

Example
This is how you'd interact with it.
If you don't have the COM6 serial port available, you'd need to connect to another available port first
from piescope.lm import laser
my_serial_port = laser.connect_serial_port()
laser1, laser2, laser3, laser4 = laser.initialize_lasers(my_serial_port)

otherwise you can just use the defaults
laser1, laser2, laser3, laser4 = laser.initialize_lasers()
laser1.laser_power = 2  # Increase the laser power to 2%
laser1.emit(0.1)        # turn the laser on for one tenth of a second

The serial communication port is opened and closed for each command, so it shouldn't block anything else. Mind you, it is fairly common to just open a port and leave it open, so we can do that if need be later.


Fix setState callback

Second argument of setState should be wrapped with a function, otherwise addEventListener will execute before setState.

@pd4d10 Yes, thank you :)