[Other] intl dependency version conflict in flutter v3.10.0

Flutter tools Because no versions of pluto_grid match >7.0.1 <8.0.0 and pluto_grid 7.0.1 depends on intl ^0.18.1, pluto_grid ^7.0.1 requires intl ^0.18.1.
And because every version of flutter_localizations from sdk depends on intl 0.18.0, pluto_grid ^7.0.1 is incompatible with flutter_localizations from sdk.
So, because site depends on both flutter_localizations from sdk and pluto_grid ^7.0.1, version solving failed.

Flutter version:
Flutter 3.10.0 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 84a1e904f4 (2 days ago) • 2023-05-09 07:41:44 -0700
Engine • revision d44b5a94c9
Tools • Dart 3.0.0 • DevTools 2.23.1

Workaround:
dependency_overrides:
  intl: ^0.18.0

I see it was introduced in recent commit: https://github.com/bosskmk/pluto_grid/commit/14119df97f41bbb16d52bc92a1eec739a09bb475

Any update on this, why does pluto_grid always use intl version that's not bundled in the latest flutter release?

Same issue:
Resolving dependencies...
Because no versions of pluto_grid match >7.0.1 <8.0.0 and pluto_grid 7.0.1 depends on intl ^0.18.1, pluto_grid ^7.0.1 requires intl ^0.18.1.
And because every version of flutter_localizations from sdk depends on intl 0.18.0, pluto_grid ^7.0.1 is incompatible with flutter_localizations from sdk.
So, because nasiya_savdo depends on both flutter_localizations from sdk and pluto_grid ^7.0.1, version solving failed.
exit code 1

Any update on this, why does pluto_grid always use intl version that's not bundled in the latest flutter release?

@Nithsua
Are you referring to the intl version included in the bundle as in the link below?
https://github.com/flutter/flutter/blob/84a1e904f44f9b0e9c4510138010edcc653163f8/packages/flutter_localizations/pubspec.yaml#L11


Add license

@kathrinbr @phil-opp I need a comment from you in this PR that you accept the new licenses
I accept that all my contributions are dual licensed under MIT and Apache 2.0

I accept that all my contributions are dual licensed under MIT and Apache 2.0

I accept that all my contributions are dual licensed under MIT and Apache 2.0


Failed to parse JSON from "https://api.travis-ci.org/jobs/147525158". Error: Unexpected token F

See https://github.com/semantic-release/semantic-release/issues/265. I think this is an issue within travis-after-all. I’m happy to help dig into it, I’d appreciate pointers / ideas of how to debug it

workes again, see https://github.com/semantic-release/semantic-release/issues/265

From https://github.com/semantic-release/semantic-release/issues/265:

Turns out the API call to https://api.travis-ci.org/jobs/147525158 returns Forbidden. Any idea why that might happen?

@gr2m That might have been due to a service outage, or Travis CI may have changed something related to the number of requests you can do to their API? 😕

There was no documented service outage during this time, I double checked. I pinged Travis support about it, let’s see what they have to say :)

I pinged Travis support about it, let’s see what they have to say :)

@gr2m Great, thanks! :)


AVRO-2240 Fix py3 Setup

Removes the assert statement that prevents python2 from interacting with setup.py at all. Also fixes an incompatibility and apparent brokenness with chmod in the setup.py script.

What's the next step for this?

I don't know. I'm waiting for feedback from the py team...

@cutting @Fokko can you advise here, please? Who do we need to approve and merge this PR?

@kojiromike @zymergen-luke I'd like to have #353 merged first. Still some issues with the tests of the custom encoders. Then we can rebase onto master, and run tests here as well.

Hi, Piotr,
Could you please re-post your comment on the list? It seems like a better
place for this discussion than the closed ticket.
On Thu, Nov 8, 2018 at 04:22 Piotr Gołąb<EMAIL_ADDRESS>wrote:

Oh no, it seems it's too late to vote! I left a comment here:
https://issues.apache.org/jira/browse/AVRO-2240?focusedCommentId=16679485&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-16679485
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/avro/pull/347#issuecomment-436927460, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABfmX0vviGXvU2P8udO8OVQuJRYkZ5zjks5us_fTgaJpZM4XaYV0
.

Yeah, done. Sorry, I wasn't in the mailing list before.


Module caching memory leak

🐛 Bug Report
This is a crosspost of https://github.com/kulshekhar/ts-jest/issues/1967 (Please read through this issue for context)
When transpiling TypeScript with a minimal transformer, there is a significant memory leak.
To Reproduce
Steps to reproduce the behavior:

git clone git@github.com:willsoto/jest-repro.git
git switch jest-memory-leak-custom-transpiler
npm ci
Run any of the test commands

Expected behavior

Modules should be cached once for the duration of the test run or allowed to be garbage collected after the test suite has completed.
Link to repl or repo (highly encouraged)
https://github.com/willsoto/jest-repro (switch to the jest-memory-leak-custom-transpiler branch)
envinfo

  System:
    OS: macOS 10.15.6
    CPU: (8) x64 Intel(R) Core(TM) i7-4980HQ CPU @ 2.80GHz
  Binaries:
    Node: 12.18.3 - ~/.volta/tools/image/node/12.18.3/bin/node
    Yarn: 1.22.4 - ~/.volta/tools/image/yarn/1.22.4/bin/yarn
    npm: 6.14.5 - ~/.volta/tools/image/npm/6.14.5/bin/npm
  npmPackages:
    jest: ^26.4.2 => 26.4.2

cc/ @ahnpnl

Bump, we're still hitting this issue across several of our repos

Do we know if this behaviour changed recently? Like did it used to not do this in an older version of Jest for example?

I have exactly the same issue in my project. Fingers crossed to find a fix soon

I have exactly the same issue in my project. Fingers crossed to find a fix soon

Having the same issue. Even using 24.0.0. Don't think it's related to the transformer though as I had the same issue when transpiling the TS and then running Jest on normal JS code.
Also see this repo: https://github.com/jaredjj3/jest-memory-leak-demo (however exposing GC did not work for me).

Having the same issue. Even using 24.0.0. Don't think it's related to the transformer though as I had the same issue when transpiling the TS and then running Jest on normal JS code.
Also see this repo: https://github.com/jaredjj3/jest-memory-leak-demo (however exposing GC did not work for me).

I put this commit on the ts-jest thread but it is relevant here i believe:
https://github.com/kulshekhar/ts-jest/issues/1967#issuecomment-764694233
When running with --expose-gc and --logHeapSize I was finding inconsistent results with sometimes the memory ballooning to over 6gb but others times never breaking 300mb.
The scenario I have found to make recreate it on my tests 100% of the time is to run jest with --no-cache or --resetCache before running the tests. If the cache is enabled then (obviously I guess) it can make this issue not apparent because loading the modules from the cache does not run the transformer and this does not have this leak

I put this commit on the ts-jest thread but it is relevant here i believe:
https://github.com/kulshekhar/ts-jest/issues/1967#issuecomment-764694233
When running with --expose-gc and --logHeapSize I was finding inconsistent results with sometimes the memory ballooning to over 6gb but others times never breaking 300mb.
The scenario I have found to make recreate it on my tests 100% of the time is to run jest with --no-cache or --resetCache before running the tests. If the cache is enabled then (obviously I guess) it can make this issue not apparent because loading the modules from the cache does not run the transformer and this does not have this leak

I'm having memory leak issue and it seems related.
Here's my reproducing sample.
https://github.com/takayukioda/reproduce-lab/tree/master/nest-e2e-memoryleak
here's commands to clone specific directory.
git clone --depth 1 --filter=blob:none --sparse https://github.com/takayukioda/reproduce-lab
cd reproduce-lab
git sparse-checkout init --cone
git sparse-checkout set nestjs-e2e-memoryleak

Any news about this problem. I'm stuck on jest and ts-jest 23 as 24+ or higher leads to memory leak. Calling GC manually doesn't fix it.
For my project:
Jest 23 and ts-jest 23: 1.2 GB heap size
Jest 24 and ts-jest 24 or higher: 12 GB heap size

Any news? I have the same issue.

Any news? I have the same issue.

Maybe you can try to isolate modules https://jestjs.io/docs/jest-object#jestisolatemodulesfn. my project is working with ts-jest and we are investigating if using the isolatedModules (sacrificing test checking) https://huafu.github.io/ts-jest/user/config/isolatedModules is the correct solution.

Maybe you can try to isolate modules https://jestjs.io/docs/jest-object#jestisolatemodulesfn. my project is working with ts-jest and we are investigating if using the isolatedModules (sacrificing test checking) https://huafu.github.io/ts-jest/user/config/isolatedModules is the correct solution.

Any luck with that mitigation ?  We are currently hitting the same issues and wondering if it's worth a try.

Any updates here?

For my project, its more about version of Node.js than version of Jest.  With Node 16.13, running our unit tests with --runInBand --logHeapUsage, only 20 test files complete before heap space runs out.  Merely switching back to Node 14.15, all 81 of our test files complete.  Heap usage really decelerates with Node 14.15.  Heap usage reaches a higher number with Node 1415 (1818MB vs. 1472MB), but never hits out of memory.  Graph of memory usage vs test file number:

Test files do not run in the exact same order, but close enough that that is not an issue.


commonjs support?

I know it's 2023 and all ...

Yep!
On Wed, Jun 21, 2023, 1:42 AM Armando Faz @.***> wrote:

something along the lines of Dual ESM and CommonJS?
https://nodejs.org/api/packages.html#dual-commonjses-module-packages
—
Reply to this email directly, view it on GitHub
https://github.com/cloudflare/voprf-ts/issues/30#issuecomment-1599321576,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEAHGYSRD4DDOZP7I5ZK33XMHVHZANCNFSM6AAAAAAXZQV7L4
.
You are receiving this because you authored the thread.Message ID:
@.***>


fix: remove warning tentang kami on test

Closes
#717  Fix warnings in Tentang Kami page test
Description
I think there is a paradox between ImageLoaderProps and Image when the layout fill

You need to use width on ImageLoaderProps https://github.com/vercel/next.js/blob/daadfd347f1f66a725c7f85cf5dbf284010ec8a1/packages/next/client/image.tsx#L25

Image with src "v1628780633/128877540-e3a9b168-4fab-489c-8001-344347e3fb84_kce5oc.jpg" has a "loader" property that does not implement width. Please implement it or use the "unoptimized" property instead.
 Read more: https://nextjs.org/docs/messages/next-image-missing-loader-width

If you use "fill" layout you cannot use height || width  https://github.com/vercel/next.js/blob/canary/packages/next/client/image.tsx#L378

 Image with src "v1628562551/128724600-f7c60a95-9ab9-4761-bcbb-a25fa3596a1a_npxy45.png" and "layout='fill'"
 has unused properties assigned. Please remove "width" and "height".

Current Tasks

[x] Figuring out what happened and remove the warning
[ ] Let's have some discussion to find the best way fix this

It's okay, I will try to re-upload all logos using a proper dimension.
So we can set the width.
Previously, I just force it to 200px since some images are too big.


[Bug]: line value outside range warning when calling extract method/widget more than once

Describe the bug
call plug#begin('~/.config/nvim/plugged')
  Plug 'nvim-lua/plenary.nvim'
  Plug 'akinsho/flutter-tools.nvim'
call plug#end()

lua << EOF
  require("flutter-tools").setup{}
EOF

When using the extract method/widget code action more than once  a 'line value outside range' warning will be displayed.
This warning will also be displayed if you undo the code action. The action/undo will execute despite the error message.
Closing and opening nvim will again allow you to do 1 extract action before the warning is shown.
Error executing vim.schedule lua callback: .../plugged/flutter-tools.nvim/lua/flutter-tools/labels.lua:15: line value outside range
stack traceback:
        [C]: in function 'nvim_buf_set_extmark'
        .../plugged/flutter-tools.nvim/lua/flutter-tools/labels.lua:15: in function 'render_labels'
        .../plugged/flutter-tools.nvim/lua/flutter-tools/labels.lua:39: in function 'func'
        ...gged/flutter-tools.nvim/lua/flutter-tools/utils/init.lua:230: in function 'handler'
        ...r/neovim/HEAD-046950f/share/nvim/runtime/lua/vim/lsp.lua:796: in function ''

To Reproduce
Steps to reproduce the behavior:

Create blank flutter project and cd into it
nvim lib/main.dart
Navigate to something you want to extract
:lua vim.lsp.buf.code_action() and pick either extract widget or extract method
undo || navigate to another thing you want to extract and repeat 4.

Expected behavior
For the code action to execute without displaying a warning.
Screenshots

System information

OS: Ubuntu 20.04.3 LTS && macOS Monterey 12.2.1
Flutter version:

Flutter 2.11.0-0.0.pre.633 • channel master • https://github.com/flutter/flutter.git
Framework • revision 54a9d65282 (21 hours ago) • 2022-02-19 10:09:26 -0800
Engine • revision f6ea6186ee
Tools • Dart 2.17.0 (build 2.17.0-125.0.dev) • DevTools 2.11.0

Is flutter in $PATH: Yes
neovim version:

NVIM v0.7.0-dev+1067-ga1f34b248
Build type: Release
LuaJIT 2.1.0-beta3

Additional context
I did mention this in #130 as well, but since it's a different problem entirerly I thought it best to open a separate issue.
The actual actions / undos seem to execute perfectly fine, it's just a somewhat annoying error message so I don't know how critical it is to be fixed.

@bjornevik thanks for raising this. In the example you mentioned it doesn't sound like you save the file after completing an extract code action. So it's unlikely that the language server will know that the file has changed. So it will have a reference to incorrect line numbers since the line numbers will now be different. Can you try again but making sure you save in between code actions and see whether or not that solves the issue?

@akinsho Same result with saving.

It does not happen if I save and then reload the buffer with :e

I've pushed a fix which should stop the disruptive error. Ultimately though the fix I think should maybe be in nvim core since making sure the buffer is sync with the language server isn't really the job of this plugin but nvim-core. Since I think that's the issue

@akinsho looked a bit through :h nvim_buf_set_extmark, could we potentially set strict = false (introduced in  neovim#17001) in the actual call?

@bjornevik thanks for pointing that out, although I don't think that's a solution to this issue. The problem isn't really with the extmark function but rather with the fact that when you use the extract method/widget functionality this changes the buffer content so then trying to place widget labels doesn't work since the buffer has changed but the lsp seems to be unaware of the change till the buffer is reloaded. I think the solution would be figuring out why the lsp is not aware of the updated buffer state after the refactor and so why subsequent labels requests don't know that the buffer has changed.


Simplify API for querying for Accounts

While working on #147 in preparation for #144 and #145, I noticed that the AccountsQuery can be simplified to adhere to the pattern followed by the other APIs in the library.
The API can be simplified to,
val result = accountsQuery
      .withTypes("com.google", "com.hotmail")
      .associatedWith(rawContact1, rawContact2)
      .find()

[x] Update https://github.com/vestrel00/contacts-android/blob/main/howto/howto-query-accounts.md
[x] Update README
[x] Update blogs

This will be closed in scope of #147

Included in release; https://github.com/vestrel00/contacts-android/releases/tag/0.1.10


missing php intl package (important for TYPO3 v9)

What happened (or feature request):
TYPO3 v9 requires php intl module to be present.
Please use a complexity rating of 1-5 (5 is high) for a feature request. (High complexity implies more PR planning)
1

@tmotyl Seems straightforward and I'll get it in this upcoming sprint before our next release of DDEV next Monday/Tuesday. Thanks!
BTW, are there any other notable differences in TYPO3 v9 that we should be aware of? We currently have detection to differentiate between Drupal 6, 7, and 8 but I'm not entirely sure if there are infrastructure level differences between TYPO3 v8 and v9 to warrant splitting them apart. Happy to split that off into its own ticket if you feel it's a larger discussion while we focus this on the specific request for the php intl package.

The other difference is that TYPO3 9 requires PHP 7.2 when 8.7 works starting from 7.0. There are no other difference in infrastructure requirements.


feature: stdin / stdout

Hello, I have wanted to used sort-package-json  with dprint which handles the file io and needs stdin and standout, such as:
$ cat package.json | sort-pakcage-json - > package.json
Would this be beneficial in other contexts?

I like the idea. PRs welcome :slightly_smiling_face:


feat: add upgrade-single-requirement gh workflow

Description:
Added GH workflow to upgrade a single package. Copied from https://raw.githubusercontent.com/openedx/edx-platform/20e09b3fe68f23a5e2832cd18254dc00e5fa6b2c/.github/workflows/upgrade-one-python-dependency.yml
Reviewers:

[ ] tag reviewer

Merge checklist:

[ ] All reviewers approved
[ ] CI build is green
[N/A ] Version bumped
[ ] Changelog record added
[ ] Documentation updated (not only docstrings)
[ ] Commits are squashed

Post merge:

[ N/A ] Create a tag
[ N/A ] Check new version is pushed to PyPi after tag-triggered build is
finished.
[ ] Delete working branch (if not needed anymore)

Closing. This can be reopened when and if it is required.


ToolTip.Tip binding does not update

This
<Slider ToolTip.Tip="{Binding $self.Value, StringFormat={0:F1}}"/>
does not update the tool tip as you slide the slider, moving the mouse away and back again does show the updated value.

Fixed with https://github.com/AvaloniaUI/Avalonia/pull/4574

Fixed with https://github.com/AvaloniaUI/Avalonia/pull/4574


Third test for 'Waypoint: Create a Set of Checkboxes' appears in quotes when passed.

The third test appears as  "Make sure each of your label elements has a closing tag."  when successful. It should not be in quotes.

Note: The quotes do not appear when the test is failed.
Challenge Waypoint: Create a Set of Checkboxes has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_4) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/46.0.2490.80 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:
<link href="http://fonts.googleapis.com/css?family=Lobster" rel="stylesheet" type="text/css">

CatPhotoApp

Click here for cat photos.

Things cats love:

cat nip

laser pointers

lasagna

Top 3 things cats hate:

flea treatment

thunder

other cats

<form action="/submit-cat-photo">
  <input type="radio" name="indoor-outdoor"> Indoor
  <input type="radio" name="indoor-outdoor"> Outdoor
  <input type="text" placeholder="cat photo URL" required>
  <button type="submit">Submit</button>
</form>

Confirmed.  When this test was added, the author used double rather than single quotes.  Should be an easy fix.
Thanks for the catch!


[Jakarta] kogito-runtimes: SB: OASTests are failing

repo/branch: kogito-runtimes:2.x
Tests failing:

org.kie.kogito.integrationtests.springboot.OASTest.testOASisValid
org.kie.kogito.integrationtests.springboot.OASTest.testOASisSwaggerUICompatible

Error message:
Expecting empty but was: ["Unable to read location `http://localhost:46581/v3/api-docs`"]

Stacktrace:
Caused by: java.io.FileNotFoundException: http://localhost:46581/v3/api-docs

Solved by https://github.com/kiegroup/kogito-runtimes/pull/3191


leading to 404 page on apptoolbox page

By mistake i raised this issue in polymer/docs kindly ignore or resolve.
on page

https://www.polymer-project.org/2.0/toolbox/

header section
Edit on GitHub link leading to  404 page

https://github.com/Polymer/docs/edit/master/app/2.0/toolbox/.md

some how ".md" is added in end of link which has to be removed
to resolve 404 redirection.
i also added a screen shot with highlighted error area

Closing as duplicate of https://github.com/Polymer/docs/issues/2387


Activity Bar Icons don't show

VSCode Version: 1.44.0-insider (user setup)
OS Version: Windows 10 Enterprise 10.0.18363

Steps to Reproduce:

Start Visual Studio Code
No Icons except the default ones appear.
Not sure if there is something wrong with my User Profile in normal Code or Insiders
With the Extensions loaded I can hover over the Activity Bar where they should be and I can click on the link and start the extension just no icon being displayed.

Does this issue occur when all extensions are disabled?: No due to there be no Extensions with Icons needing to be loaded. I have tried it in the non-Insiders version and loading only one or two Extensions and the Icons don't work then either.

Does it reproduce if you reload the window?
Can you please try using fresh user data dir. Start code from command line as follows
code --user-data-dir <path to a temp user data dir>
Open Developer tools (Developer: Toggle Developer Tools). Go to elements tab and inspect the element. Let me know where the icons are located. See eg:

Can you please open network tab and check what is being returned for this svg file?

Hey @sandy081 thanks for your help. I think I may have spotted the issue, another App is the Default for SVG files which may be causing VS Code to fail to display the Icon. You can see from the screen below the behaviour I am getting, this is in VS Code and the same happens in the Insiders version.

/duplicate
https://github.com/microsoft/vscode/issues/92323


add an input option to short-circuit post-init-comms

PR Summary
Given the segfaults happening due to PostInitializationCommunication, I'm adding a runtime option to disable it. Adds an input parameter in the phoebus block.
In the long term, we need to pull #118 through. Alternatively, changes upstream in Parthenon may make this driver stage unnecessary. @brryan what are your thoughts?

PR Checklist

[x] Adds a test for any bugs fixed. Adds tests for new features.
[x] Format your changes by calling scripts/bash/format.sh.
[x] Explain what you did.

This runs successfully on 4 devkit nodes.
I can't comment on the accuracy of the code with/without the boundary inits, but it removes the impediment.


Moves ServiceBuilder to Core, adds Google\Cloud\Version

Addresses #440

Moves Google\Cloud\ServiceBuilder to Google\Cloud\Core\ServiceBuilder
Adds Google\Cloud\Version::VERSION for checking the parent repo's version

This will need to be updated in google\gax

Throws \Exception when a known service is requested from ServiceBuilder that does not exist.
Updates Core entrypoint to be ServiceBuilder, and google/cloud entrypoint to be NULL.

Considerations:

Is the new Version class appropriate?
Can we have ServiceBuilder return GAPIC-only services as well, i.e. DLP, VideoIntelligence, etc?
Is there a better way to run tests for this other than unregistering the autoloaders? It seems to work. Another way would be to run shell_exec('php -r ....');

The main concern I have here is that this may be considered a BC break. It's a bit of a grey area because the ServiceBuilder is currently part of the google/cloud package which we do not plan to mark 1.0 on - but what about users who may be using GA libraries instantiated using the ServiceBuilder?

Is the new Version class appropriate?

Yeah, makes sense to me.

Can we have ServiceBuilder return GAPIC-only services as well, i.e. DLP, VideoIntelligence, etc?

We should probably do that. Before we do, though, we may want to consolidate our configuration options between GAPIC and veneer clients as it could otherwise get unwieldy having them all be documented together here. What do you think of us taking the time to refactor parts of Core (perhaps even putting this work into GAX? If we go that route, it may require removing the GAX's dependency on gRPC) so we can share some of the underlying concepts with what we have in the GAPIC clients as well?

Is there a better way to run tests for this other than unregistering the autoloaders?

The approach you took makes sense to me 👍.

I removed all changes to the GAPIC classes, and added VERSION to the deprecated Google\Cloud\ServiceBuilder for backwards compatibility. PTAL!

Rebased and updated for v0.37.0

@dwsupplee @jdpedrie PTAL, when can we merge this?

@bshaffer we are planning to cut a release soon and if possible would like to get this in :).

Please note now two VERSION constants exist, one for BC in Google\Cloud\ServiceContainer, and one in the new Google\Cloud\Version class.
Does a script update these, or are they updated manually upon each release?


mgmg: smp: Change smp_shell_rx_byte to process data in bulk

The smp_shell_rx_byte has been renamed to smp_shell_rx_bytes and now
accepts data buffer pointer and its size as parameters. Return value
has been changed to size_t and represents number of bytes processed from
the given buffer.
The change has been done to more efficiently serve most common scenario
when the function is called in loop to process buffer, byte by byte.
Previously such operation required passing each byte separately,
with the change the function will work directly on source buffer reducing
number of calls and byte copy operations.
Signed-off-by: Dominik Ermel<EMAIL_ADDRESS>

Is it possible to add a unit test for these function (If you think this make sense)?

I will take a look into this.

Hi,
What about adding a new backend to the shell: shell_uart_smp?
This starts to be more and more complex.

What about adding a new backend to the shell: shell_uart_smp?
This starts to be more and more complex.
@mniestroj any thoughts?

Should be compatible with https://github.com/zephyrproject-rtos/zephyr/pull/27023.

What about adding a new backend to the shell: shell_uart_smp?
This starts to be more and more complex.

@de-nordic @jakub-uC please take a look at #27674, this is my idea on how to simplify RX handling in ISR.
In general we can benefit from all #27023, #27674 and this PR as well. They seem all compatible.

s/mgmg/mgmt/ in the commit title.


fix(ios): make sure notificationMessage is immutable

Description

Update to make sure that notificationMessage is correctly storing an NSDictionary.
Create a mutable copy when the content needs to be updated and then restore an immutable copy.
Related Issue

Resolves #314
Closes #315
Motivation and Context

Fix issue reported in #314 to prevent app crashes.
How Has This Been Tested?

Deployed app to iOS 18.1 device.
Pushed payload from AWS Pinpoint:
{
  "APNSMessage": {
    "aps": {
      "alert": "Sample message for iOS endpoints",
      "content-available": "1"
    }
  }
}

Pushed payload from Amazon SNS
{
  "APNS_SANDBOX": "{\"aps\":{\"alert\":\"Sample message for iOS development endpoints\", \"content-available\": \"1\"}}"
}

Push plugin was initialized with the following:
    const push = PushNotification.init({
        ios: {
            alert: true,
            badge: true,
            sound: true
        }
    });

Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

I tried installing this commit to test it but:
cordova plugin add https://github.com/havesource/cordova-plugin-push/pull/317/commits/0117971a75a56cd4b1ac20af8a205a5aac11903b
Failed to fetch plugin https://github.com/havesource/cordova-plugin-push/pull/317/commits/0117971a75a56cd4b1ac20af8a205a5aac11903b via registry.
Probably this is either a connection problem, or plugin spec is incorrect.
Check your connection and plugin name/version/URL.
CordovaError: Error: TAR_BAD_ARCHIVE: Unrecognized archive format

@geoidesic I don't think the provided method is a valid way to check out a plugin. It should follow a pattern that npm understands.
I recommend using the following command to check out the PR:
cordova plugin add github:havesource/cordova-plugin-push\#fix/GH-314

@erisu thx for that. I was able to install that version. It does fix the crash. However, the foreground property is NULL?
{
    "additionalData": {
        "message": "Carl Hoing",
        "matchID": 27466,
        "matchOpCode": "trlxo6bsnxxz83in7cuphycsbabqcmc5lmom5oqiqdfaymuctjfn3y9bgxt5twb5",
        "foreground": "",
        "coldstart": "",
        "matchKey": "ogmhf6j1vexgngce2clqepnl4cedarid"
    },
    "message": "bump",
    "title": "Carl Hoing",
    "count": 7,
    "sound": "default"
}

This PR is unrelated to foreground or coldstart.
It only addresses the app crash caused by attempting to edit an immutable object.
You'll need to open a new issue ticket and provide detailed information about how you're configuring the PushNotification.init settings for your app, as well as the payload you're sending. You may also want to review your JS code, because if the foreground or coldstart key exists, it can only be true or false based on what I see in the native code.
This PR and the original issue ticket are not the appropriate places to discuss additional issues.
I will merge as fixed.

Right. I notice I was wrong about the NULL value, that was a transformation error in my logging tool.


Change License

I suggest to change it from
Copyright (c) 2013-2015 Mikko Rantanen, Frederick Gnodtke

to
Copyright (c) 2013-2015 Mikko Rantanen and other contributors

I agree with the proposition but still wanted a way to list contributors other than git log.
Created AUTHORS file. More details in the above commit message.


Please add a match step example

Hello, could you add an example on how to perform the match-select pattern with gremilin scala?
This is an example from the Tinkerpop documentation:
gremlin> g.V().match('a',
            __.as('a').out('created').as('b'),
            __.as('b').has('name', 'lop'),
            __.as('b').in('created').as('c'),
            __.as('c').has('age', 29)).
          select('a', 'c').by('name')

Being a total newcomer I tried to invoke the match step on a GremlinScala class, but seems to me it's not implemented yet.
Then I tried to invoke it directly on the graph object. This is the code I have right now:
    val graph = TinkerGraph.open()
    var gs = GremlinScala(graph)
    val vertex = gs.addVertex("hello")
    var vertex2 = gs.addVertex("world")
    vertex.addEdge("followed_by", vertex2, Map())

    val result = graph.V.outE().`match`(
      "a",
      __.as("a").inV().as("b") // what to put here ?
    ) // .....

Which does not seem to work. What should I substitute to __ to have the code compile?
Perhaps a little example in the readme would help many others :)

Hello, after much digging I found a solution inside the Tinkerpop 3.0.0M6 documentation (yes, the previous version).
    val graph = TinkerGraph.open()
    var gs = GremlinScala(graph)
    val vertex = gs.addVertex("hello")
    var vertex2 = gs.addVertex("world")
    vertex.addEdge("followed_by", vertex2, Map())

    val result = graph.V.outE().`match`(
      "a",
      graph.of().as("a").inV().as("b")
    ).select()

Hope it will be useful for somebody else.
Bye!

Thanks for sharing. I will provide a sample usage for all steps, however at the moment the API is moving too much and my time is limited.


Update README.md

Tiny mistake in the readme makes copy pasting this code less awesome. This PR fixes that.

Nice! Great catch @aisflat439

@all-contributors add @aisflat439 for docs


Show/hide element

Hello,
my panel have a main select and a lot of fields, each select option needs a specific subset of fields and not all.
Is it possible to show/hide programmatically a specific element in order to simplify the user navigation?
Thank you,
Fabrizio

Hello Mikhail,
thank you for your enthusiasm.
I have an idea about a different use of your plugin.
Grafana is a powerful software but have a BIG limitation, the template variables are not groupable in a specific GUI like a command panel.  Morover if you have more panels, charts table etc, and each of these panels use a specific variable or a subset of variables, it's very difficult for the final user to understand the panel dynamic.
Your plugin already do this with a few operations, I wrote a basic guide for the other users.
Please take a look at the attached files, I have also a suggestion to improve the plugin.
VolkovLabs.pdf
VolkovLabs - changes.pdf
VolkovLabs_TEST_dashboard.txt
Thank you,
Fabrizio

@ftasso, this is awesome.
I will take a look and get back to you shortly.

We implemented various JavaScript hooks and refactored element states in v3.
Version 3.1.0 introduced conditional element visibility.


precompile

@VarLad I've opened this to discuss https://github.com/FedeClaudi/Term.jl/issues/141
This is a branch from master while before I was testing the development branch for the up-coming version, I thought this might make things easier.
BEFORE using snoopcompile

AFTER

Codecov Report

Merging #147 (dc7e5c4) into master (7456581) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #147   +/-   ##
=======================================
  Coverage   94.55%   94.55%
=======================================
  Files          34       34
  Lines        1965     1965
=======================================
  Hits         1858     1858
  Misses        107      107

Impacted Files
Coverage Δ

src/Term.jl
90.00% <ø> (ø)

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

@FedeClaudi Also, in your "AFTER", notice that you're including the Precompile time with the compile time.
I'd recommend to close the Julia session after your first using Term and then do
@time using Term
@time Panel("test")

again

I see what you mean, the first time using Term is called its a bit slower but after that it loads faster for future sessions, neat.

Which is appropriate (it increases the precompile time, doubles the load time, but reduces the execution time)

I guess what I'm confused about is that despite precompilation @time Panel() shows that some compilation is still done:

julia> @time using Term
  0.391873 seconds (815.66 k allocations: 54.100 MiB, 2.91% compilation time: 85% of which was recompilation)

julia> @time Panel()
  0.081357 seconds (6.94 k allocations: 322.232 KiB, 99.33% compilation time)
╭──────────────────────────────────────────────────────────────────────────────────────╮
╰──────────────────────────────────────────────────────────────────────────────────────╯

julia> @time Panel()
  0.000223 seconds (338 allocations: 22.773 KiB)
╭──────────────────────────────────────────────────────────────────────────────────────╮
╰──────────────────────────────────────────────────────────────────────────────────────╯

Not that it makes a huge difference, but I would've thought that the first @time Panel() would be the same as the second since Panel() should be pre-compiled?

@FedeClaudi

Not that it makes a huge difference, but I would've thought that the first @time Panel() would be the same as the second since Panel() should be pre-compiled?

We can't precompile everything :P
Also, I tried direct precompilation (without the SnoopPrecompile lines)
if ccall(:jl_generating_output, Cint, ()) == 1   # if we're precompiling the package
    let
        Panel()
        Panel("test") * Panel(Panel()) / hLine(20) |> tprint
        Panel(Panel("Text"))
    end
end

just before the module ends.
The results are similar-ish.
Whats concerning though is, I see that there is a lot of cache invalidation, which triggers recompilation. (for example with Panel("Some text") I believe this can be avoided with changes in the package itself, and lowering the invalidation where possible will make the entire package more usable.
https://timholy.github.io/SnoopCompile.jl/dev/snoopr/#invalidations
I think this is going to take more time and term effort, but it certainly looks doable. I'd recommend to keep the draft open for now.

Thank you! Precompilation and TTX is one of the areas of Julia I haven't digged into yet. Input from Term's users that want to improve the package is always the best motivation to get me to do things!

I'll look into whats specifically causing the invalidations with more time. :D

That would be great! I will also look into stuff, but I first need to learn a bit more about pre-compilation,methods invalidation etc... any help would be great.
P.s.: term is not very type stable at all, would that make pre-compilation less effective?

P.s.: term is not very type stable at all, would that make pre-compilation less effective?

I think that would contribute to the problems, yes.
I'd recommend into looking into https://timholy.github.io/SnoopCompile.jl/dev/ . The docs have a lot of good points and have a few tools to help you make your package faster. :D

As fully optimizing TTX will be a more involved project, I will merge this for now and leave the issue open as a reminder for when I will have the time. This will already help a little bit, I will release a new version today/tomorrow with this precompilation bit if you want to get it.


[SPARK-10714][SPARK-8632][SPARK-10685][SQL] Refactor Python UDF handling

This patch refactors Python UDF handling:

Extract the per-partition Python UDF calling logic from PythonRDD into a PythonRunner. PythonRunner itself expects iterator as input/output, and thus has no dependency on RDD. This way, we can use PythonRunner directly in a mapPartitions call, or in the future in an environment without RDDs.
Use PythonRunner in Spark SQL's BatchPythonEvaluation.
Updated BatchPythonEvaluation to only use its input once, rather than twice. This should fix Python UDF performance regression in Spark 1.5.

There are a number of small cleanups I wanted to do when I looked at the code, but I kept most of those out so the diff looks small.
This basically implements the approach in https://github.com/apache/spark/pull/8833, but with some code moving around so the correctness doesn't depend on the inner workings of Spark serialization and task execution.

cc @davies

Merged build triggered.

Merged build started.

Test build #42712 has started for   PR 8835 at commit 8bfe6c4.

Test build #42712 has finished for   PR 8835 at commit 8bfe6c4.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
  class WriterThread(

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42712/
Test FAILed.

Merged build finished. Test FAILed.

Could you try to re-use the code in Python UDF?

@davies I've updated BatchPythonEvaluation too.

Merged build triggered.

Merged build started.

Test build #42719 has started for   PR 8835 at commit 8d3c495.

Merged build triggered.

Merged build started.

Test build #42719 has finished for   PR 8835 at commit 8d3c495.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
  class WriterThread(

Test build #42724 has started for   PR 8835 at commit 5e55bf6.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42719/
Test PASSed.

Merged build finished. Test PASSed.

Test build #42724 has finished for   PR 8835 at commit 5e55bf6.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
  class WriterThread(

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42724/
Test PASSed.

Based on some offline discussion / debate, we've decided to merge this patch into both master and branch-1.5. I'm going to merge this now.


Renaming folder while downloading will mess things up

Have a quite big remote folder, let's call it "A"
Start to sync it with maestral
While syncing, rename the folder on your local computer to "B"

Maestral will throw a lot of sync errors, and the "B" folder on your PC will be uploaded to Dropbox (so you will have the "A" folder, with full content, and the "B" folder, with the content synced up until when you renamed it)

Thanks for the report! This is indeed a tricky issue, I'll need to think about what the best behaviour would be here. Redirecting any running downloads to the new location may be possible, but could come with unintended consequences.


avvio dotnet run cursore lampeggia ma non parte applicazione

Problem encountered on https://dotnet.microsoft.com/learn/aspnet/hello-world-tutorial/run
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.

Translation:

start dotnet run cursor flashes but no application starts

@boroil can you provide more information? What were you trying to do?

Closing due to no response.


Version bumps all around:

Howdy - I started doing this bring @hapi/hoek up to date, and was getting conflicts and version warnings with @hapi/hapi unless I brought it up to date as well.  When I did that, I started getting node errors until I moved to the current LTS, v12.
I think with more time you could make a "partial" update without introducing the need for newer node versions (@hapi/hapi v18 should work under node v10).  I figured I would make the most current version possible, get the tests fixed, and make sure the newer requirements were documented in the README and bump this to v5 to follow your previous semver patterns.

bring @hapi dependencies up to date
fix tests under newer Sinon api
update README with changes and new version requirements
bump version a full number

If it's too much of a jump I understand.  I figured you'd have to make the changes eventually, so why not rip the band-aid off.  =)
If you do merge it, please also publish to npm (I always forget).  Thanks again for your work on this package!

Published. Thanks for updating the package.

Awesome, thank you so much.


Feature request: ability to control instantiation of sqlite db

Wow, amazing work here--really high quality code and docs, and just what I'm looking for (sqlite-wasm + Kysely).
That said, I wish the API supported a way for me to pass in the sqlite DB instead of sqlocal creating it (i.e., the ability to override SQLocalProcessor.init). Would you be open to making a change along those lines, or considering a PR to that effect?
Some reasons this might be a nice feature:

dev can control version of sqlite-wasm being used, including the possibility of packages other than the official @sqlite.org/sqlite-wasm distro (e.g., maybe one compiled with custom sqlite extensions, etc.)
dev has better control over storage mechanism (e.g., could use new sqlite.oo1.JsStorageDb(), or their own logic for how to handle OPFS not being available, etc.)

this also allows dev to control flags passed in to OpfsDb constructor (e.g., use trace flag in dev mode); although this could also be exposed as a param similar to databasePath

There's difficulty in allowing one to just pass in the database object or a function since neither of these can be transferred from the SQLocal class to the worker through postMessage since functions and class instances cannot be cloned. As an alternative to this, I experimented a while back with allowing for passing a custom worker. This is why SQLocalProcessor is its own class and worker.ts just instantiates that. But if you are making your own version of SQLocalProcessor, you are re-implementing a lot of the library while being bound to the messaging system SQLocal uses, and I'm not sure there's much value in that over building your own full custom solution to use instead of SQLocal. SQLocal is meant to be an abstraction to cover most use-cases. Apps with more complex requirements may not be able to use an abstraction.
However, I do think adding options that can be passed to change the OpfsDb flags and tweak other behaviors is a good idea. I can definitely see someone wanting to use the trace flag, like you said, or use SQLocal to open a database file in read-only mode. This would not be too hard to add. I'll look into it.

Ah yeah, didn't think of the limitations of passing custom implementations across to the worker...really good point.
What do you think about allowing someone to pass in their own worker as an optional SQLocal constructor param (with the disclaimer that it's an unsupported escape hatch: "by using this, you accept responsibility for maintaining worker message compatibility, forfeit issue support, etc.")?
// myWorker.js
import { SQLocalProcessor } from '@sqlocal/processor';
import sqlite3InitModule from<EMAIL_ADDRESS>
// Dev gets to pick sqlite-wasm version and customize constructor param (e.g., sqlite logging) 🎉
const sqlite3 = await sqlite3InitModule(...);
const myDb = new this.sqlite3.oo1.OpfsDb(this.config.databasePath, 'cw');

const processor = new SQLocalProcessor(myDb); // 👈 New: pass in custom db instance

self.onmessage = (message) => processor.postMessage(message);
processor.onmessage = (message) => self.postMessage(message);

// myapp.js
import { SQLocalKysely } from "sqlocal/kysely";

const myWorker = new Worker(new URL('./myWorker', import.meta.url), { type: 'module' });
const { dialect } = new SQLocalKysely("foo.sqlite", myWorker); // 👈 New: pass in custom worker instance
// ...

It seems like the changes to SQLocalProcessor.init() would be fairly minimal? Basically "if custom DB passed in, use that, otherwise use the existing logic for creating it"?
There's just so much other great stuff in sqlocal--would be great to find a way for someone to avoid forking just to customize such a small amount of code.
Do you think something like this would work as an optional escape hatch?

I don't hate that idea, but I'm not sure it's necessary. Here's what I'm thinking right now. Along with the options for flags we mentioned above, there could be a storage option that accepts 'memory' | 'opfs' | 'local' | 'session'. If local or session is used, SQLocal will create an SQLocalProcessor that uses JsStorageDb; OpfsDb for opfs; DB for memory. This gives the freedom to use all the available storage mechanisms while keeping things simple. What do you think?


Try to fix sporadic CI test failure

There is a family of CI object server test failures that all look something like this:
RLMRealm *realm = // ...open a Realm
doSomething();
CHECK_COUNT(3, MyTestObject, realm);   // this passes
doSomethingElse();
waitForDownloads();
CHECK_COUNT(5, MyTestObject, realm);   // this fails; count is still 3

They are all impossible to reproduce, happen in different tests, but all take that same basic structure.
My hypothesis is that, since Realms are not refreshed until the current runloop iteration is complete, that there exists a possibility where waitForDownloads() completes quickly enough the second CHECK_COUNT() call is made before the runloop has had a chance to complete and the Realm has a chance to refresh.

I think my hypothesis was not correct, but I had another hypothesis, so I made more fixes. Will also run the tests locally to see if I can see any difference.
My new hypothesis is this: ROS takes some time to perform actions (e.g. process permission changes or get changes ready to be downloaded), but we don't account for that in our tests, so if ROS is slow the tests will fail.

I rebased this onto master (from master-3.0), since it's not breaking and since once it's in we can easily fold it into the WIP branch (but not vice versa).

If the high timeout values are indeed required, we should still report this on the sync repo as an area worth optimizing.

I agree with filing an issue. I'd like to merge in this change and let it "bake" for a while (see if it results in a trend of improved CI performance), then file an issue if it does look like this was the culprit.

Good news! The object server tests failed not because of anything I did, but because of the "child process" issue.

Good news! The object server tests failed not because of anything I did, but because of the "child process" issue.

are we tracking that one?

https://github.com/realm/ci/issues/145


Automatically deduce DPI

With this commit the DPI doesn't have to be listed in the ~/.Xresources and will be deduced automatically.

@AUTplayed thanks for feedback


ReplaceEmojisWithFunc: Unexpectedly trims strings

Function ReplaceEmojisWithFunc unexpectedly returns trimmed version of strings, thus modifying the original string. I would like to know the reason behind this, as it's not the responsibility of this function.

Hi @Mukhammadsaid19 ,
Thank you for the catch. I agree that trimming of spaces is not a responsibility of the replacer function so I fixed it in the PR: https://github.com/forPelevin/gomoji/pull/22


Upgrade to WildFly Core 1.0.0.Beta6

Windows Build 1444 is now running using a merge of 45dad8367deee52a7e91232589503bb37e8564a0

Linux Build 6364 is now running using a merge of 45dad8367deee52a7e91232589503bb37e8564a0

Linux with security manager Build 1292 is now running using a merge of 45dad8367deee52a7e91232589503bb37e8564a0

Linux with security manager Build 1292 outcome was SUCCESS using a merge of 45dad8367deee52a7e91232589503bb37e8564a0
Summary: Tests passed: 826, ignored: 358 Build time: 0:06:59

Linux Build 6364 outcome was SUCCESS using a merge of 45dad8367deee52a7e91232589503bb37e8564a0
Summary: Tests passed: 2858, ignored: 414, muted: 1 Build time: 0:48:13

Windows Build 1444 outcome was SUCCESS using a merge of 45dad8367deee52a7e91232589503bb37e8564a0
Summary: Tests passed: 2858, ignored: 414, muted: 1 Build time: 0:49:18


Consider standardizing range selector icon across these components

sl-date-picker and sl-date-range-picker look the same, and are based on bootstrap-datepicker.  sl-calendar is our own creation.

taking this for my PR to rule all glyphs


Inconsistent type inference when changing the order of type declarations

🔎 Search Terms
N/A — this is a very specific case, and I'm not sure which part of the code sample causes the bug.
🕗 Version & Regression Information
Tested on all versions starting from 5.0.1. The example repo uses 5.3.0-dev.20230905.
⏯ Playground Link
https://github.com/SimpleCreations/typescipt-compiler-inconsistent-behavior-poc
💻 Code
Steps to reproduce:

Clone the POC repo (see link above)
yarn install
node ./build.mjs

Observe the error listed below in "Actual behavior".

Switch to a branch where order of type declarations is slightly modified (type Footer is moved higher in the file): git checkout modified-type-order; or use the following diff:

diff --git a/src/graphql-types.ts b/src/graphql-types.ts
index 162aab1..cf252dd 100644
--- a/src/graphql-types.ts
+++ b/src/graphql-types.ts
@@ -6,14 +6,14 @@ export type Query = {
   footerCollection?: FooterCollection;
 }

-type ExternalLink = Entry & {
-  linkedFrom?: FooterCollection;
-};
-
 export type Footer = Entry & {
   footerLinksGroupCollection?: FooterLinksGroupCollection;
 };

+type ExternalLink = Entry & {
+  linkedFrom?: FooterCollection;
+};
+
 type FooterCollection = {
   items: Footer[];
 }

node ./build.mjs

Observe no error.
🙁 Actual behavior

Incorrectly reported error in the initial example:

src/query.transformer.ts:23:7 - error TS2322: Type 'Footer[] | undefined' is not assignable to type 'FooterTransformer | null | undefined'.
  Property 'field' is missing in type 'Footer[]' but required in type 'FooterTransformer'.

23 const y: FooterTransformer | null | undefined = x;
         ~

  src/query.transformer.ts:9:3
    9   field: unknown;
        ~~~~~
    'field' is declared here.

After changing the order of type declarations, the code compiles successfully.
In IDEs (e.g. VS Code) you wouldn't see any errors (the types are inferred as expected).

🙂 Expected behavior
The compiler's behavior is consistent regardless of the order of type declarations. (No error is expected in either case because the types in question are compatible.)
Additional information about the issue
This seems to be a bug somewhere deep down in the TS compiler. It may have something to do with types circularly referencing each other.
There are a few seeming unrelated ways to make the error disappear, e.g. by trying any of the following:

Removing Entry & from types;
Removing | {} union inside the FooterLinksGroup type;
Putting all code into a single file.

All this additionally confirms that this is a compiler bug and not a feature.
This is a bug I encountered in production and tried to reduce the example as much as possible. The need for a build with API instead of the CLI arises from the fact that my original code uses Nest CLI which internally uses the TS compiler API.

Managed to prepare a repro via TS Playground — https://github.com/microsoft/TypeScript/issues/55666
Closing in favor of the new issue with a better repro.


pull request for e2e test at 01/28/2017 18:26:12

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


change app.csp() with contentSecurityPolicy()

Thanks a lot for your support @NataliaRizzi! :-)
I assume that we can merge it. The Helmet API changed and this PR solve it. I changed the base branch from master to release-1.5
What do you think @ckarande? Should we merge it?

LGTM. Yes, we can merge it! 👍


Add version dropdown to sphinx website

We should add a version dropdown to the sphinx website such that documentation for a specific version can be easily accessed:
https://pydata-sphinx-theme.readthedocs.io/en/stable/user_guide/version-dropdown.html

See https://blog.esciencecenter.nl/versioned-documentation-using-only-github-actions-and-github-pages-1825296e31aa

Aw this looks promising, it's something I wanted to solve for ages but never found the time to look into.


Failing to install on Raspberry Pi3

Sorry if this is a noob problem, but I am completely blank on Python, especially on the Raspi. I wanted to set up the camera_proxy on my pihole Raspi, cloned the repository with git and wanted to install the prerequisites. But the very first is failing already:
pi@raspberrypi:~/camera_proxy $ sudo pip install av
Looking in indexes: https://pypi.org/simple, https://www.piwheels.org/simple
Collecting av
  Downloading https://files.pythonhosted.org/packages/40/e6/da37b548e9e1312fb11ffd0f31f8679e1b96d3859790a77e93a113394d34/av-8.0.3.tar.gz (2.3MB)
    100% |████████████████████████████████| 2.3MB 49kB/s
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-install-Mwk4pG/av/setup.py", line 9, in <module>
        from shlex import quote
    ImportError: cannot import name quote

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-install-Mwk4pG/av/

So it looks like something (quote?) is missing, but I do not know how to fix that...
Any help is highly appreciated!

I found I had to use pip3 instead of pip to install the libraries and python3 to run main.py.


[BoringSSL] Update third_party/boringssl-with-bazel

Change was created by the release automation script. See go/grpc-release.

Closed in favor https://github.com/grpc/grpc/pull/38274


Fix bug #4202

Fix bug #4202
Problem: line on the edge get cut
Example: https://jsfiddle.net/2tqmf2bg/
Pull request: https://jsfiddle.net/onnbzLuu/1/

@leonfed is there a way to unit test this?

@etimberg I do not know how to make a unit test, because it's changes in visual look of charts.

Closing due to inactivity

Adding my vote to reopen this pull request

@etimberg Why close a real bug?

@etimberg @simonbrunel Could you merge this, or at least review it? #4202 is still unfixed in 2.7.2. As @leonfed said, there isn't a clear way to unit test this, but that's no reason to close this without any further discussion.

Okay, for anyone looking at this, #5321 was merged after 2.7.2, and claims to fix #4202. I still don't think this PR was handled appropriately. However, http://www.chartjs.org/dist/master/Chart.min.js as of right now still exhibits #4202. I don't know if that's outdated or if #4202 is still not entirely fixed (that file is different from https://cdnjs.cloudflare.com/ajax/libs/Chart.js/2.7.2/Chart.min.js, at least).

These changes have been merged as part of #5321 but are not released yet.
@lethosor it works with master (https://jsfiddle.net/e8n4xd4z/8951/)


meaning of "image"

What is the meaning of the word "image" in this context? I link would be helpful since the word is too generic to search for effectively on google. I'm guessing the implication is that in this explanation, the two obj files are part of the same library or executable? If so, it doesn't explain to me why I'm getting this warning because the files being specified in the warning I'm getting, belong to two different libraries.
Thanks for any further insight!

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

ID: 82e663e7-83de-68f1-1f0a-29e7db4f2783
Version Independent ID: 53efb74d-9f84-63d7-8206-2bb6ca836d27
Content: Linker Tools Warning LNK4286
Content Source: docs/error-messages/tool-errors/linker-tools-warning-lnk4286.md
Service: visual-cpp
Sub-service: errors-warnings
GitHub Login: @TylerMSFT
Microsoft Alias: twhitney

Moved this issue to an internal work item tracking system because we are moving away from using GitHub issues to track C++ doc work.


OLS-1034: Enable W0603 Pylint issue on CI

Description
Enable W0603 Pylint issue on CI
We can do it as all issues found by this rule have been fixed already
Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library
[ ] Bump-up library or tool used for development (does not change the final image)
[x] CI configuration change
[ ] Konflux configuration change

Related Tickets & Documents

Related Issue #OLS-1034

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 74.89%. Comparing base (f093792) to head (6704506).
Report is 34 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #2044       +/-   ##
===========================================
- Coverage   96.95%   74.89%   -22.06%
===========================================
  Files          70       70
  Lines        2890     2892        +2
===========================================
- Hits         2802     2166      -636
- Misses         88      726      +638

see 35 files with indirect coverage changes

/test unit
/test verify

/test unit

/approve
/override "Red Hat Konflux / ols-enterprise-contract / lightspeed-service"

/lgtm

/override "ci/prow/4.17-e2e-ols-cluster"

/test integration
/test 4.15-images

/override "ci/prow/4.17-e2e-ols-cluster"

/override "ci/prow/integration"


Missing data for SRX2409974

Hi,
I wanted to use data from your service, but stumbled over a problem when trying to download data for SRX2409974. Trying to understand why, it looks like the alignment failed. At least the alignment rate is 0%. Is this really correct? Or is it simply a processing error? I also do not understand how you can filter out 14% duplicates with 0% aligned reads? This results in 0 peaks. This is an Encode sample and their processing, admittedly different, comes out with ~64k peaks:
https://www.encodeproject.org/files/ENCFF169XNB/
If it is a processing error, it might be the case it happened for several samples... Could that be?
Another example is SRX083227 with 0.9% aligned reads, 28% duplicates and 3 peaks.
What do you think?

Hi Karl,
I found that the most of the reads of SRX2409974 and SRX083227 have technical sequences as shown in atatched QC results, although such technical sequences are usually removed prior to submitting to SRA or GEO.
Exact alignment rate of SRX2409974 was 6.9E-06 (= 214 / 31167642), and 14 % (= 30 / 214) of aligned read was PCR duplicate.
Cheers,
Shinya

2018/07/09 17:18、Karl Nordström<EMAIL_ADDRESS>Hi,
I wanted to use data from your service, but stumbled over a problem when trying to download data for SRX2409974. Trying to understand why, it looks like the alignment failed. At least the alignment rate is 0%. Is this really correct? Or is it simply a processing error? I also do not understand how you can filter out 14% duplicates with 0% aligned reads? This results in 0 peaks. This is an Encode sample and their processing, admittedly different, comes out with ~64k peaks:
https://www.encodeproject.org/files/ENCFF169XNB/ https://www.encodeproject.org/files/ENCFF169XNB/
If it is a processing error, it might be the case it happened for several samples... Could that be?
Another example is SRX083227 with 0.9% aligned reads, 28% duplicates and 3 peaks.
What do you think?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/inutano/chip-atlas/issues/4, or mute the thread https://github.com/notifications/unsubscribe-auth/AMehTU1JXBwCKgMP8vl8DP9w4Dg1TOtKks5uExHpgaJpZM4VHRYF.

Hi Shinya,
thank you for the clarification.  I understand the numbers and the problem now.
The last half of each read is adapter. As an end-user, of course I would like to see these things handled, but I also see the complications. It would probably require reprocessing of a lot of data.
I'll close this issue...


Landing page

Create a landing page

Hi @debrajrout  can you please assign this to me?

yes

Can you please add hactoberfest-accepted label to this issue? I will start working

Can you please add hactoberfest-accepted label to this issue? I will start working

done

Hi @debrajrout  can you please review and approve?

Can you please assign this task to me? I want to contribute to this. And which tech stacks we can use for the landing page?

Hi @debrajrout  can you please close this issue? As the PR has been merged


Unhandled exception when there is no internet

Unhandled exception when there is no internet:

Reference:
https://github.com/AppImage/appimage.github.io/pull/785

This does not surprise me at all. We're still in the super early stages for this @probonopd - but thank you for the issue!


getTransactions incorrectly states startLedger as required

RPC Page: https://developers.stellar.org/docs/data/rpc/api-reference/methods/getTransactions
The documentation currently incorrectly indicates startLedger as always required. That is not correct.
startLedger and cursor cannot be sent at the same time.  Either use startLedger or cursor. The docs should say something to this respect so it's clear how to data.
Maybe also add an example of using cursor instead of startLedger.
curl -X POST \
-H 'Content-Type: application/json' \
-d '{
  "jsonrpc": "2.0",
  "id": 8675309,
  "method": "getTransactions",
  "params": {
    "pagination": {
      "cursor": "3086346319040513",
      "limit": 5
    }
  }
}' \
https://soroban-testnet.stellar.org:443 | jq

I think the same issue in documentation also exists on getEvents https://developers.stellar.org/docs/data/rpc/api-reference/methods/getEvents
cc @mollykarcher


refactor calls to cl_apl_ecatt_object=>get_tadir_entry

refactor calls to cl_apl_ecatt_object=>get_tadir_entry to use the abapgit tadir class instead

actually not possible, there is some object type conversion going on, closing


Do you have plans to support the IETF's QUIC protocol?

Hi
The IETF organization has split google's QUIC protocol into two protocols, QUIC and HTTP3.
QUIC is the perfect solution for lockstep games.Do you have plans to support the IETF's QUIC protocol?
Thanks!

Yes... see https://github.com/netty/netty/issues/3778

The work for QUIC is happening here and I think we are very close for a release: https://github.com/netty/netty-incubator-codec-quic


doc: release-notes-4.0: Add Silabs notes

Add noteworthy additions for Silabs socs, boards and drivers.

Thanks @asmellby! I think we might want to mention something in the "HALs" section as well? https://github.com/zephyrproject-rtos/zephyr/blob/c50777a8431157bd4ab3a2c2e2a37e8151ae07da/doc/releases/release-notes-4.0.rst?plain=1#L984-L990

Rebased to resolve conflicts with Renesas additions.

@asmellby new conflicts have crept in

Please rebase and fix the merge conflict


convert weeks to hours #1483

Why:
Closes #1483
What's being changed:
Weeks to hours converter has been added.

!assign

Hey @lukaszmielczarekdev, !assign command is only for good first issue !!!
Please choose another issue.
Thanks for your interest in contributing to this project.


Added TLS backend selection features

The main reason is that request's default TLS dependency pulls in openssl which wraps the OpenSSL C-lib via openssl-sys.
This not only adds a noticable hiccup to build times, it also requires a resp. C compiler toolchain which can be a PITA when cross-compilation for other platform targets is needed (e.g. Android or iOS).
reqwest offers a bunch of alternative TLS backends, exposed by feature flags.
With this PR they're all also available in the infobip_sdk crate.
This PR also puts some code behind #[cfg(feature=...)] gates to get rid of some warnings and further decrease build time.

That spellcheck will never succeed btw. for obvious reasons.

This now also fixes #13 and #16. There are some minir changes for developer UX too. I.e. releasing this requires at least a bump to v0.6.


Update python Docker tag to v3.11

This PR contains the following updates:

Package
Type
Update
Change

python
final
minor
3.10-slim -> 3.11-slim

python
stage
minor
3.10-slim -> 3.11-slim

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about these updates again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Renovate Ignore Notification
Because you closed this PR without merging, Renovate will ignore this update (3.11-slim). You will get a PR once a newer version is released. To ignore this dependency forever, add it to the ignoreDeps array of your Renovate config.
If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR.


Can this be used with puppet masterless and what would be best practices...

I'd love to use this gem along with Hiera, r10k, but could i use them with puppet masterless.
Any help or links would be great.
Thanks!

You can absolutely use this module in a masterless model. In fact most of the examples take that route to make it easy to get started. And one of the examples even uses Hiera.
With regards r10k, you could use r10k's dependency management features to manage a list of modules (like this one) in a masterless model, though many of the other features of r10k are about interacting with a master.
You might find [Pupa](https://github.com/daenney/pupa interesting too from @daenney.

A few more questions:
Does this module also manage the signing of the certs for puppet or is that even necessary in a masterless setup?  And if not, since this module supports setting up autoscaling groups, how is masterless puppet suppose to get onto a newly created node in an ASG?
What is the ENC in this kind of setup? Is that hiera but as commited code in the puppet repo?
Are there any good examples of using theforeman in this kind of masterless setup that you guys (puppetlabs) recommend?
Thanks!

Hi @dfuentes77. You would be better to ask more general questions like these in the puppet-user mailing list. The AWS module is at a lowevr level that the questions you're asking.
With regards to installing Puppet onto nodes provisioned by the module, you can either create an AMI which already have Puppet installed and running, or you can use the userdata script to install and run Puppet on instance creation. You can see a basic example here: https://github.com/puppetlabs/puppetlabs-aws/blob/master/examples/puppet-enterprise/pe_agent.pp#L11

Thanks you!


typo in Preface

"... as well as various new algorithmic ideas. some of which we cover in this book." ->
"... as well as various new algorithmic ideas, some of which we cover in this book."

fixed thanks


npx cli command

Summary
User story
As a developer, I would like to be able to execute markdownlint trough CLI, so that I will be able to setup CI jobs based it with ease.
Additional context
Something like this using npx:
npx markdownlint <path>

What do you think?

Forget it, I didn't see the reference on the manual. :upside_down_face:


cilium status: Fix exit code when components are disabled

Fixes: #3067
Reported-by: @ChiefAlexander
Signed-off-by: Thomas Graf<EMAIL_ADDRESS>

test-me-please


New SmoothScroll module based on Magellan scrollToLoc code.

Implementation for #9587
To enable smooth scroll on the internal links just add the data-smooth-scroll attribute to the parent container, sample code below, or via javascript initialization.

Section 1

The Magellan module now depends on the SmoothScroll module for the scrolling animation functionality.

Cheers @denisahac

Thank you @denisahac for this PR. Sorry, I planned to work on it before but did not have the time to. However, please warn when you are working on an issue.

However, please warn when you are working on an issue

hahahaaaaa i think have heared this before :wink:

Should've notified you guys! @ncoden will nudge you guys next time 😄

@denisahac end of line please ?

@IamManchanda I do not think the end of line is required, when you check the rest of the javascript codebase you'll see nothing at the end of the file.
You should only put a comment on your code to explain what you're trying to achieve, or for documentation purposes.

This looks great - given this is a new plugin, can you do a pass on a quick docs page for this as well? E.g. docs/page/smooth-scroll.md

Got you @kball. I will do the documentation page for this new javascript module 👍

Awesome buddy @denisahac

@ncoden @denisahac ... looking at the doc changes ... i do think there should be some kind of example ( demo page) or equivalent page to reflect on the smooth scroll plugin .... because its currently its not showing the functionality

@IamManchanda though the documentation for SmoothScroll didn't explicitly provide a working functionality, however it's implicitly provided by the Magellan in the right sidebar navigation.

@denisahac
But still bro .... you atleast need a example page or if that's not possible
then a simple codepen like for eg. => http://codepen.io/IamManchanda/pen/VPZMba
or something like => https://blackrockdigital.github.io/startbootstrap-scrolling-nav/
will be great .... because this isn't showing its actual functionality!

What is the current status of this PR?

Waiting for merge.

@ncoden and @IamManchanda you reviewed this - do you think it's good to go at this point?

Yeah but I would like to have a PR ( master) with a codepen example linked in the docs ... Once this is released !

Because current example lacks its actual usage and fwiw as pointed earlier

Great, merging! Thanks for the work on this @denisahac @ncoden @IamManchanda

Big cheers @denisahac  !


Divergence of implementation between Windows and non-Windows

https://github.com/apache/incubator-mxnet/blob/f00b9ab5b4410a91a8f6581da696a92f85fbccf6/src/operator/numpy/np_elemwise_broadcast_op.cu#L32-L41
Shouldn't be needed anymore as 64-bit toolchain is now required on Windows? (Added due to compiler OOM errors when 32-bit toolchain was in use)

In GluonNLP Windows CI, tests fail on Windows with errors like
2020-05-13T12:06:33.5890094Z >           raise get_last_ffi_error()
2020-05-13T12:06:33.5890252Z E           mxnet.base.MXNetError: Traceback (most recent call last):
2020-05-13T12:06:33.5890363Z E             File "c:\jenkins\workspace\mxnet\mxnet\src\imperative\./imperative_utils.h", line 251
2020-05-13T12:06:33.5893446Z E           MXNetError: Check failed: outputs[i]->dtype() == out_types[i] (0 vs. 1) : 0-th output has invalid dtype. Expecting 1 got 0 in operator _npi_uniform

This may be due to the diverging implementation described above, or there are more issues. @zhreshold @yzhliu

@BenjaminCHEN2016 is helping.

closed by #18523


When existing managed containers on the host get synced to the server as part of host registration, they are not created as part of managed network.

When existing managed containers on the host get synced to the server as part of host registration, they are not created as part of managed network.
Steps to reproduce the problem:
Registered a host as part of default environment.
Added few containers in managed network to this host .
Deleted this host from this environment ( Stopped the host VM and deleted the host from rancher server).
Added this host back to a different environment in the same rancher server.
When the host was added , the containers were synced to the server. But they were not deployed as part of managed network.
Expected behavior:
Network agent should be started and the container should get created in this managed network.
Question - docker inspect of the container has managed network ip address published. Will we give the same ip to this container when this container is synced to rancher server?
    "Labels": {
        "io.rancher.container.ip": "<IP_ADDRESS>/16",
        "io.rancher.container.uuid": "38376511-2ab1-476c-a0a0-ebd1750930ea"
    },

Dupe of #929. Closing


Move AppliedPTransform into the Runners Module

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[x] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[x] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[x] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[x] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

This class is for use of runner authors, and should not be used
elsewhere.

This probably should wait until recordAsOutput and finishSpecifyingOutput are merged

R: @kennknowles

Coverage increased (+0.04%) to 70.511% when pulling 52128511cb2ed614b63bce3969feb644528cd1a1 on tgroh:move_applied_ptransform into 8af5c28d4866801fefc300a3b42d30b271795426 on apache:master.

Coverage increased (+0.007%) to 70.475% when pulling 52128511cb2ed614b63bce3969feb644528cd1a1 on tgroh:move_applied_ptransform into 8af5c28d4866801fefc300a3b42d30b271795426 on apache:master.

Coverage increased (+0.003%) to 70.493% when pulling 953e07b02d0f7f2bc63acdb6aa021af48ad31d7f on tgroh:move_applied_ptransform into 3bffe0e0014bdd6ae73dc2e8ecfc2b61d066120c on apache:master.

LGTM


Create .DS_Store

@Boses
The .DS_Store files are files specific for Mac file systems, they are not relevant to this code base and therefore they should not be included in this repository.
Alternatively, we could update our .gitignore to ignore all .DS_Store files. That way they won't appear as changed files on git status.
If you are interested, you can create an PR to update our .gitignore or we can wait until next week and I can do it.
I am closing this PR for now.


KH2 Auto Attack only works in Combat State 1, but not Combat State 2

0 = Resting combat state
1 = Combat state 1 (regular fights, some scripted fights like Minnie Escort part 1)
2 = Combat state 2 (most scripted fights, all boss fights)
Currently the script only works in CS1, but not CS2.

Assuming 2 is the value for the other combat state this should be fixed now


Add decimals to SPL token account state

it seems kind of annoying that the only way to tell how many decimal places an SPL token account uses is to look up its mint.


What do we do with artworks without images?

For instance, https://www.artsy.net/artwork/jacopo-da-pontormo-study-of-a-nude-boy-partial-figure-study-recto-study-of-a-seated-man-verso

@broskoski maybe this should be handled in volt? @garrengotthardt thoughts here?

@starsirius Can you have a think about this?

In general, we require artworks to have images before publishing by admins; in fact, artworks created in CMS (Volt) should always have images as uploading images is the first step (this artwork was created and published in 2014 which predates Volt). So maybe

@garrengotthardt and the institutions team can confirm if there is actually a use case for artworks with no images.
Otherwise, I think it's a data issue and we should re-upload images for them. Force can probably make that assumption.

hey guys sorry just catching up on this - there is no use case for published works without images, and the GR and instititutions teams dont publish works unless there are images.
Since it looks like theres an image record for this work but the image is missing seems like two possible scenarios could be at play:

could be that this work was published in a rush before processing was complete and then it ended up failing — though unlikely here since it seems there is no original to even try to reprocess, or
the original image got lost for this when i think platform was working on changing storage of original images about 1-1.5 years ago

Sounds like we may want to do a sweep then and cleanup the data from Gravity’s end?
/cc @joeyAghion


How to unit test with redux without TestBed

This is a...

[ ] feature request
[ ] bug report
[x] usage question

What toolchain are you using for transpilation/bundling?

[x] @angular/cli
[ ] Custom @ngTools/webpack
[ ] Raw ngc
[ ] SystemJS
[ ] Rollup
[ ] Other

Environment
NodeJS Version:
Typescript Version:2.7.2
Angular Version: 5.1.1
@angular-redux/store version: 7.0.0
@angular/cli version: 1.6.1
OS: CentOS (Linux)
Description:
When talking about testing in Angular, many approaches can be found on google, stackoverflow, etc. I followed this instruction here to clearly differ between unit tests and integration tests. Writing unit tests without TestBed works very good until I had to mock an @select property for getting data from the redux store. I cannot directly assign an Observable to the @select property (because it doesn't support setters). I only want to test the pure functions of my components and not the store, services or other dependencies. Therefore I don't use TestBed and instanciate the component object itself instead. For the ng2-redux library (which is a deprecated fork of angular-redux)  I found the following issue  that has a solution for that. Unfortunately, that is no longer working in angular-redux, because NgRedux is an abstract class now, which cannot be instanciated directly.
#157 references this problem but uses the old ng2-redux library.
Is there any way to mock it without using TestBed?

Abstract your redux code to a separate state service and then use the MockNgRedux module. Here is an example.  I've stripped out some business logic and dumbed this down a bit to make it simpler.
import { Injectable } from '@angular/core';
import { select, dispatch } from '@angular-redux/store';
import { Observable } from 'rxjs/Observable';

import { DocumentsArchive } from '../models/documents-archive.interface';
import { DocumentsSelectors } from './documents.selectors';
import { DocumentsActions } from './documents.actions';

@Injectable()
export class DocumentsStateService {

  @select(DocumentsSelectors.documentsArchive) getDocumentsArchive$: Observable<DocumentsArchive[]>;

  @dispatch() onDocumentsArchiveReceipt = DocumentsActions.onDocumentsArchiveReceipt;

}

And to test it:
Pay attention to the code to "getSelectorStub" for mocking @select properties.
import { NgRedux } from '@angular-redux/store';
import { MockNgRedux } from '@angular-redux/store/testing';
import { DocumentsSelectors } from './documents.selectors';
import { DocumentsStateService } from './documents-state.service';
import { DocumentsTypes } from './documents.types';
import { getDocumentsMock } from '../services/documents-archive.mock';

describe('State Service: Documents', () => {
  let mockNgRedux: NgRedux<any>;
  let stateService: DocumentsStateService;

  beforeEach(() => {
    MockNgRedux.reset();
    mockNgRedux = MockNgRedux.getInstance();
    stateService = new DocumentsStateService();
  });

  describe('documents page loading', () => {

    it('should dispatch a documents archive received action', () => {
      const expectedAction = {
        type: DocumentsTypes.DOCUMENTS_ARCHIVE_RECEIVED,
        payload: {
          accountDocuments: [] as any[]
        }
      };
      spyOn(mockNgRedux, 'dispatch');
      stateService.onDocumentsArchiveReceipt({
        accountDocuments: []
      });
      expect(mockNgRedux.dispatch).toHaveBeenCalledWith(expectedAction);
    });

  });

  describe('documents archive', () => {

    it('should get a list of documents', (done: Function) => {
      const documentsArchiveMock = getDocumentsMock().accountDocuments;
      const documentsArchiveStub = MockNgRedux.getSelectorStub(DocumentsSelectors.documentsArchive);
      documentsArchiveStub.next(documentsArchiveMock);
      documentsArchiveStub.complete();

      stateService.getDocumentsArchive$.subscribe(collection => {
        expect(collections[0].documents[0].archiveDate).toEqual('2017-04-04');
        done();
      });
    });
  });

});

I will add that this pattern keeps all of your redux in one place and only requires one place to mock the redux. By injecting the state service into your components, the state service can easily be mocked with jasmine.spy objects for @disptach functions and BehaviorSubjects for the @select properties.

Just test the function you give to the selector in isolation. Testing that @select works is testing library code imho.


Release 1.0.0a16

(title says it all)

CLOSING after having merged #207 and release to PyPI + RTD.


Test additional python versions

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

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

Totals

Change from base Build<PHONE_NUMBER>8:
0.0%

Covered Lines:
2442

Relevant Lines:
2865

💛  - Coveralls


Add github actions

Closes #27

We might need to adjust the ssh command later depending on whose machine we use to deploy and the name of the ssh key


Fix Gallery dependencies

Fixes https://github.com/flutter/flutter/pull/63200

Verified the fix locally with flutter build apk.


Add fields to values schema

This updates missing fields coming from multicluster.
@ericvn @ayj @elfinhe PTAL if this is everything we need.

@ostromart: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_operator
6d13b274cbc198c9ce760cfd8ec4b780e7a07eac
link
/test gencheck_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Thanks for the suggestions, @elfinhe.

@ostromart: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_operator
27c0548eb58626db7c5650f2d247c91de92bfff4
link
/test gencheck_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ostromart: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_operator
27c0548eb58626db7c5650f2d247c91de92bfff4
link
/test gencheck_operator

e2e_operator
27c0548eb58626db7c5650f2d247c91de92bfff4
link
/test e2e_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ostromart this has a do not merge label

In response to a cherrypick label: #565 failed to apply on top of branch "release-1.4":
Applying: Make gen
error: Failed to merge in the changes.
Using index info to reconstruct a base tree...
M	pkg/apis/istio/v1alpha1/v1alpha1.pb.html
M	pkg/apis/istio/v1alpha1/values_types.pb.go
M	python/istio_api/pkg/apis/istio/v1alpha1/values_types_pb2.py
Falling back to patching base and 3-way merge...
Auto-merging python/istio_api/pkg/apis/istio/v1alpha1/values_types_pb2.py
CONFLICT (content): Merge conflict in python/istio_api/pkg/apis/istio/v1alpha1/values_types_pb2.py
Auto-merging pkg/apis/istio/v1alpha1/values_types.pb.go
CONFLICT (content): Merge conflict in pkg/apis/istio/v1alpha1/values_types.pb.go
Auto-merging pkg/apis/istio/v1alpha1/v1alpha1.pb.html
CONFLICT (content): Merge conflict in pkg/apis/istio/v1alpha1/v1alpha1.pb.html
Patch failed at 0002 Make gen


Starting repl from file on windows does not work ( all path separator are removed from the path)

Even after #90 (I use v0.9.0 and had the same issue with the previous version), start or refresh repl still remove all path separator from the path. Example:
:l d:MyBackupsandboxidristestDec.idr
(real path been D:\MyBackup\sandbox\idris\test\Dec.idr)

@Cedric-Venet OS version? plugin version? is ipkg being used?

Windows 10, extension v0.9.0 (is this the plugin you speak of?), not ipkg (I opened a folder in vscode but its just a few independant files)

@Cedric-Venet If you know some JS, you can try do this:
open C:\Users\your name\.vscode\extensions\zjhmale.idris-0.9.0\out\idris\commands.js, navigate to this block:
let startup = (uri) => {
  const pathToIdris = vscode.workspace.getConfiguration('idris').get('executablePath');
  const idrisPath = which.sync(pathToIdris)
  const pkgOpts = ipkg.getPkgOpts(innerCompilerOptions)
  term = vscode.window.createTerminal("Idris REPL", idrisPath, pkgOpts)

  if (innerCompilerOptions.src && uri.includes(innerCompilerOptions.src)) {
    term.sendText(`:cd ${path.resolve(innerCompilerOptions.src)}`)
    term.sendText(`:l ${path.relative(path.resolve(innerCompilerOptions.src), path.resolve(uri))}`)
    term.show()
  } else {
    term.sendText(`:l ${uri}`)
    term.show()
  }
}

change it into this:
let startup = (uri) => {
console.log(uri) // ← ADD THIS LINE
  const pathToIdris = vscode.workspace.getConfiguration('idris').get('executablePath');
  const idrisPath = which.sync(pathToIdris)
  const pkgOpts = ipkg.getPkgOpts(innerCompilerOptions)
  term = vscode.window.createTerminal("Idris REPL", idrisPath, pkgOpts)

  if (innerCompilerOptions.src && uri.includes(innerCompilerOptions.src)) {
    term.sendText(`:cd ${path.resolve(innerCompilerOptions.src)}`)
    term.sendText(`:l ${path.relative(path.resolve(innerCompilerOptions.src), path.resolve(uri))}`)
    term.show()
  } else {
    term.sendText(`:l ${uri}`)
    term.show()
  }
}```

Then restart VSCode, type `Toggle developer tool` in the command panel, and then fire the `Start REPL` command. See what the F12 developer tool shows.

I added those three lines:
console.log(uri)
console.log(:l ${uri})
console.log(innerCompilerOptions.src)
And I get:
[Extension Host] d:\MyBackup\sandbox\idris\test\Dec.idr
[Extension Host] :l d:\MyBackup\sandbox\idris\test\Dec.idr
[Extension Host] undefined
looks like its term.sendText which misbehave. I added .replace(/\/g, "/") before sending the text and it works now (note that using .replace(/\/g, "\\") did not work, which seems really strange to me):
term.sendText(:l ${uri}.replace(/\/g, "/"))
Thanks for the help, I can try to do a PR tonight if you want.

@be5invis Thanks for the help!
@Cedric-Venet Path separator missing in load path on windows is reported twice before #83 #65 and we are still facing this issue, sorry for that.
One thing is really wired that REPL can work perfectly with my windows10 virtual machine, I don't know if this issue is related to Visual Studio or MSBuild or some other concrete .Net frameworks.

And yes, the solution you got here looks good and can also work as expected both on OSX and Windows10 from my side, the reason why replace(/\/g, "\\") did not work is that \ need to be escaped.
Looking forward to your PR :)

@zjhmale markdown ate my '\' I tried .replace(/\\/g, "\\\\")  and it did not work its why I was surprised. It seems that on (both) of my computers '\' really don't go through.

@Cedric-Venet OK, but term.sendText(:l ${uri}.replace(/\\/g, "/")) is completely doable for this, just feel free to open a pull request for this :)


Weird freezing on VMWARE win10

It works half the time, and it crashes half the time, without any blue screen or explanation. The crash always happens after a successful load, and I can read all the messages in my debugger but the vm is frozen.

Just to clarify, are you crashing, blue screening, or freezing?

Closing due to inactivity.


Refactor: remove cookiecutter and replace it with jinja2 templates

Description
cookiecutter library was used to develop ucc-gen init feature quite fast, but it can be replaced by jinja2 library and we will reduce the number of dependencies we have in the project.

:tada: This issue has been resolved in version 5.30.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Library not showing up in Obsidian

Difficulty getting the Library to show up in Obsidian.
Plugin set up fine until I tried to open from within Obsidian and there was no option to get to Library via icon.


build: darwin/arm builders broken by content-based build CL

Looks like I broke the darwin/arm builders with my CL 73212. Investigating.
The log at https://build.golang.org/log/83cd0cb12afd4ff5069c6a9a72b124e02d21127c makes it look like there are two different make.bash runs, which maybe is part of the problem.
/cc @eliasnaur @crawshaw

Thanks for putting up with the difficult mobile os'es. Off the top of my head:

iostest.bash runs make.bash and then all.bash.
cmd/dist test rebuilds runtime/cgo with the lldb tag.

Not sure if any is relevant.

I can reproduce locally without an ios device. One unrelated point: iostest.bash runs all.bash --no-rebuild but that's not a thing.  all.bash always rebuilds; perhaps that was supposed to be run.bash --no-rebuild.
The root cause appears to be that when you do a cross-compile build it looks like the CC_FOR_TARGET gets recorded as the default $CC for both host and target builds, so the host binaries look out of date during the run.bash half (because they were not built with CC=$CC_FOR_TARGET).
Related to #8161 in part.

Off to a meeting but will keep at this later.

Supposed to be fixed now. We will see.

@eliasnaur I did fix the failure we were seeing. Now we are seeing different failures. I am not sure what to make of them. Can you take a look and maybe file a new issue if you can see more?

Thanks. It looks unrelated to the content-based caching. https://github.com/golang/go/issues/22612


Crash: SDWebImageDownloaderOperation.m line 85

SDWebImage Version
4.0.0

Integration Method
cocoapods /

Xcode Version
Xcode 8.2.1

Crashed: com.apple.main-thread
0  libobjc.A.dylib                0x19489bbdc objc_msgSend + 28
1  CoreFoundation                 0x182f41458 CFRelease + 524
2  CFNetwork                      0x182999f38 HTTPRequestParserClient::~HTTPRequestParserClient() + 52
3  CFNetwork                      0x182999ef4 HTTPRequestMessage::~HTTPRequestMessage() + 80
4  CoreFoundation                 0x182f41394 CFRelease + 328
5  CFNetwork                      0x18299a144 URLRequest::~URLRequest() + 152
6  CoreFoundation                 0x182f41394 CFRelease + 328
7  CFNetwork                      0x182aa9e68 -[NSURLRequestInternal dealloc] + 40
8  CFNetwork                      0x182aab124 -[NSURLRequest dealloc] + 48
9  libobjc.A.dylib                0x194886b1c object_cxxDestructFromClass(objc_object*, objc_class*) + 148
10 libobjc.A.dylib                0x194893f38 objc_destructInstance + 92
11 libobjc.A.dylib                0x194893f90 object_dispose + 28
12 Foundation                     0x183e96b14 -[NSOperation dealloc] + 104
13 ??????                         0x1008ae4fc -[SDWebImageDownloaderOperation dealloc] (SDWebImageDownloaderOperation.m:85)
14 libsystem_blocks.dylib         0x194f31910 _Block_release + 256
15 libdispatch.dylib              0x194ed9954 _dispatch_client_callout + 16
16 libdispatch.dylib              0x194ede20c _dispatch_main_queue_callback_4CF + 1608
17 CoreFoundation                 0x18301b7f8 CFRUNLOOP_IS_SERVICING_THE_MAIN_DISPATCH_QUEUE + 12
18 CoreFoundation                 0x1830198a0 __CFRunLoopRun + 1492
19 CoreFoundation                 0x182f452d4 CFRunLoopRunSpecific + 396
20 GraphicsServices               0x18c75b6fc GSEventRunModal + 168
21 UIKit                          0x187b0afac UIApplicationMain + 1488
22 ??????                         0x100144178 main (main.m:14)
23 libdyld.dylib                  0x194f06a08 start + 4

@xhzth7091 this is a duplicate of #1859 - that is why the template asks you to search for issues before opening a new one.
Please add to that thread - we released a fix in version 4.1.0, but let us know if that doesn't resolve it for you.


[feat] Add support for TMod version 3.1

Extend the TMod implementation to add support for modules 3.1.

Hello @victorusu, Thank you for submitting the Pull Request!

In the file reframe/core/modules.py:

Line 407:80: E501 line too long (82 > 79 characters)

Do see the ReFrame Coding Style Guide

Codecov Report

Merging #935 into master will decrease coverage by 0.18%.
The diff coverage is 15.21%.

@@            Coverage Diff             @@
##           master     #935      +/-   ##
==========================================
- Coverage   91.84%   91.65%   -0.19%
==========================================
  Files          80       81       +1
  Lines       10732    11030     +298
==========================================
+ Hits         9857    10110     +253
- Misses        875      920      +45

Impacted Files
Coverage Δ

reframe/core/modules.py
60.66% <15.21%> (-6.1%)
:arrow_down:

reframe/frontend/executors/__init__.py
97.92% <0%> (-0.31%)
:arrow_down:

reframe/frontend/dependency.py
96.77% <0%> (-0.17%)
:arrow_down:

reframe/frontend/executors/policies.py
96.98% <0%> (-0.07%)
:arrow_down:

reframe/core/containers.py
100% <0%> (ø)
:arrow_up:

...s/resources/checks_unlisted/dependencies/normal.py

unittests/resources/checks_unlisted/deps_simple.py
100% <0%> (ø)

...nittests/resources/checks_unlisted/deps_complex.py
97.67% <0%> (ø)

reframe/core/logging.py
83.54% <0%> (+0.05%)
:arrow_up:

reframe/frontend/statistics.py
90.29% <0%> (+0.09%)
:arrow_up:

... and 3 more

Continue to review full report at Codecov.

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


Need to handle Close App event in c++winrt

Need help on how to use CloseRequested in C++/Winrt program. Any pointers to working example would be really helpful.
Regards,
Kuldeep
[Enter feedback here]

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

ID: cd653644-7246-93a2-5fb1-035c487d548b
Version Independent ID: 0bc615cc-1401-a197-2454-e35bf1d256e0
Content: SystemNavigationManagerPreview.CloseRequested Event (Windows.UI.Core.Preview) - Windows UWP applications
Content Source: winrt-api-build/xml/Windows.UI.Core.Preview/SystemNavigationManagerPreview.xml
Product: uwp
Technology: winrt-reference
GitHub Login: @lastnameholiu
Microsoft Alias: alholiu

Thanks, @kuldeepmelligeri, and sorry about the long delay in responding.
For this, I'd suggest taking a look at Handle events by using delegates in C++/WinRT.
If you have this working in C# and want to port it to C++/WinRT, then we have resources such as the C++/WinRT content in general, and then the Porting the Clipboard sample to C++/WinRT from C#—a case study and Move to C++/WinRT from C# topics in particular.
So I'll close this issue for now.
-Steve


darwin: Stop the systray on quit rather than terminating the app

On Darwin, quitting the systray terminates the current process. That's always been a pain. This stops the event loop, and systray.Run should return after calling systray.Quit.

Changes applied.

Thanks for looking at this. I have tested and it does not seem to reliably work on completing the example code.
Any thoughts on this?
[anacrolix-master] ><> go run .
Requesting quit
Finished quitting
andy@Fyreball /Users/andy/Code/Fyne/systray/example                                                                                7s [8:55AM]
[anacrolix-master] ><> go run .
Requesting quit
andy@Fyreball /Users/andy/Code/Fyne/systray/example                                                                                7s [8:55AM]
[anacrolix-master] ><>

Yes, the quit is issued asynchronously, and main can exit before the quit handler completes. The correct behaviour is to report quitting after systray.Run returns. I've updated the example to make it deterministic in this respect.

Hello, were you able to test/confirm this?

quit is issued asynchronously, and main can exit before the quit handler completes.

Doesn't this seem problematic? We need to know that the handlers all complete. (for compatibility if nothing else surely?)

I think you misunderstand. I fixed the code so that it always reports completion, where previously it raced.

Cheers