(#325) Disable rpcauthorization for shim layer

The shim layer has buggy authorization handling when using nested facts, so disable and just rely on the Choria server authorization that handles nested facts without issue Fixes #325

Verified at site using nested fact authorization:
[root@vestal-agent-test-rhel8-live-0xb1 support]# grep rpcauthorization /etc/choria/choria-shim.cfg
rpcauthorization = 0

This is the authorization:
[root@vestal-agent-test-rhel8-live-0xb1 support]# grep mysql /etc/choria/policies/bolt_tasks.policy
allow	choria=mysql_demo.choria	*	gce.instance.attributes.mysql_demo=true	*

This is the task working:
[tdockendorf@blue-vestal-choria-util-dev-us-west1-x5sb ~]$ mco tasks run choria::ping --message "hello world" -F gce.instance.attributes.mysql_demo=true
Retrieving task metadata for task choria::ping from the Puppet Server
Attempting to download and run task choria::ping on 1 nodes

Downloading and verifying 1 file(s) from the Puppet Server to all nodes: ✓  1 / 1

Running task choria::ping and waiting up to 60 seconds for it to complete

Summary for task 22a0747d5a205b66bd04df43993b7943

                       Task Name: choria::ping
                          Caller: choria=mysql_demo.choria
                       Completed: 1
                         Running: 0

                      Successful: 1
                          Failed: 0

                Average Run Time: 0.06s

Thank you!


GalacticScale 1.10 - plural satellite issues

BepInEx + Thouma GalacticScale Plugin, no other mods
Game Seed 3440 5065, all default settings (64 solar systems, 1x multiplier, default generated config-file by r2-modmanager)
Starting Solar system contains a ice-giant (Vouskiaho - II ) with 1 moon ((Vouskiaho - II - I). Game says this moon has plural satellites. Orbit of the other moons show/imply those are satellite of the ice-giant, even though the ice-giant does not show the 'plural satellite' attribute. May be a visual bug only.

should be fixed now - reopen if needed


drishti_add_test() features

As expected, there are a few follow up issues and features related to drishti_add_test() that will be useful for day-to-day use, as this was merged for initial use before being feature complete (if that ever happens).
ios-deploy: It is difficult to kill ios-deploy calls, which I believe is due to reliance on lldb for launching apps.  There is some discussion about this online.
PARALLEL: The testing mechanism should be updated for parallel use.  Assuming 'adb' and 'ios-deploy' can run concurrently (TODO: confirm in docs/implementation) we can use a toolchain specific installation path on the test device (in the case of Android) or a unique BUNDLE_ID in the case of iOS in order to avoid conflicts.  This is essentially the same as what would happen in HOST builds.  This should provide protection for parallel builds+testing assuming each build has a unique toolchain.  If the same toolchain is used in multiple builds we could still have a problem.  This is an unlikely scenario, but it should at least be documented in the drishti_add_test interface.  To cover all cases we could consider adding file(LOCK ...) inside CMake, where the lock would be created in a shared path between different toolchain builds (${HUNTER_ROOT}, ${CMAKE_SOURCE_DIR}/_locks, etc.)
The toolchain addition currently looks like this for iOS and ANDROID.
if(ANDROID)
    # ...
    if(CMAKE_TOOLCHAIN_FILE)
      get_filename_component(TOOLCHAIN_NAME ${CMAKE_TOOLCHAIN_FILE} NAME_WE)
      set(TESTING_DIR "${DRISHTI_ANDROID_DEVICE_TESTING_ROOT}/${PROJECT_NAME}/${TOOLCHAIN_NAME}")
    else()
      set(TESTING_DIR "${DRISHTI_ANDROID_DEVICE_TESTING_ROOT}/${PROJECT_NAME}")
    endif()
    # ...
elseif(IOS)
    # ...
    if(CMAKE_TOOLCHAIN_FILE)
      get_filename_component(TOOLCHAIN_NAME ${CMAKE_TOOLCHAIN_FILE} NAME_WE)
      set(BUNDLE_ID "${DRISHTI_IOS_BUNDLE_IDENTIFIER}.${TOOLCHAIN_NAME}.${APP_TARGET}")
    else()
      set(BUNDLE_ID "${DRISHTI_IOS_BUNDLE_IDENTIFIER}.${APP_TARGET}")
    endif()
   # ...
endif()

A CLEAN and DOWNLOAD policy of some sort should be added.
For general build tests, having a mechanism to start from a clean slate is desirable:

for android we can rm -rf /data/local/tmp/drishtisdk before starting all tests
(optional) rm -rf /data/local/tmp/drishtisdk after running, however, having an option for some persistent output could be desirable -- example: we may want to debug an image processing algorithm with some annotated image, or compare the results between an iOS and an Android run... such a feature could be accomplished with something like a DOWNLOAD device_folder host_folder feature.

Note: I've noticed that the ios-deploy device tests seem to end up installing each test-drishti-<module> application twice.

Project migrated to https://github.com/hunter-packages/gauze


Update to go 1.22

The change request to regex/syntax has been merged (https://github.com/golang/go/issues/57950). The change should be in go 1.22. As soon as it's out, create a PR to update rassemble-go.

Done: https://github.com/itchyny/rassemble-go/pull/2


Reflect on a groupby multiplexing variant wrt argtop etc.

    - argtop(k, <expr>, <expr>?, separator?)
        expression, joined by a pipe character ('|') or by the provided separator.
    - distinct_values(<expr>, separator?) -> string
        the provided separator.
    - top(K, <expr>, separator?)
        joined by a pipe character ('|') or by the provided separator.
    - top_counts(k, <expr>, separator?)
        joined by a pipe character ('|') or by the provided separator.
        Sorted list, pipe-separated, of all the types seen in the values.
    - values(<expr>, separator?) -> string
        the provided separator.

Using flags, -l working like k top. Basically top with counts vs. argtop, vs. distinct values? We can also give the context around the row itself through -s.

What's more, the top & argtop aggs are not their own counterparts, probably need another top, and a most_common


Add support for youtube gaming

I would like to use this extension, but I much prefer youtube gaming to the main youtube. If you add support for youtube gaming, I will immediately start using this extension

@yamboy1 Thanks for your feedback! I actually wasn't aware YouTube Gaming existed. I'll look into this.

Great, thanks


Named selectors

This adds Named Selectors example to docs

[ x ] Documentation

Codecov Report

Merging #81 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master     #81   +/-   ##
======================================
  Coverage    87.4%   87.4%
======================================
  Files          15      15
  Lines         524     524
  Branches      106     106
======================================
  Hits          458     458
  Misses         53      53
  Partials       13      13

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 70fd17c...b427bb4. Read the comment docs.


Do no generate new address automatically

The onViewCreated of the WalletReceive fragment should wait for the user to press the "Generate new address" button before displaying an address.

Fixed with bc315f91255f22fb3d0e18b747a28fce53211294.

Fixed with bc315f91255f22fb3d0e18b747a28fce53211294.


Lsp stays attached when switching to Netrw

Weird, issue here, when I open lua for example, it attaches lua_ls and everything works, but when I switch back to netrw the lsp stays attached and I get syntax errors, etc. Here is my setup:
lsp.lua file
local M = {}

M.lazy = {
  {
    'VonHeikemen/lsp-zero.nvim',
    branch = 'v2.x',
    dependencies = {
      -- LSP Support
      {'neovim/nvim-lspconfig'},             -- Required
      {                                      -- Optional
        'williamboman/mason.nvim',
        build = function()
          pcall(vim.cmd, 'MasonUpdate')
        end,
      },
      {'williamboman/mason-lspconfig.nvim'}, -- Optional

      -- Autocompletion
      {'hrsh7th/nvim-cmp'},     -- Required
      {'hrsh7th/cmp-nvim-lsp'}, -- Required
      {'L3MON4D3/LuaSnip'},     -- Required
    }
  }
}

M.setup = function ()
  local lsp = require("lsp-zero").preset({})

  lsp.on_attach(function (client, bufnr)
    lsp.default_keymaps({buffer = bufnr})
  end)

  lsp.ensure_installed({"lua_ls"})

  lsp.setup()
end

return M

init.lua
local lazypath = vim.fn.stdpath("data") .. "/lazy/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({
    "git",
    "clone",
    "--filter=blob:none",
    "https://github.com/folke/lazy.nvim.git",
    "--branch=stable", -- latest stable release
    lazypath,
  })
end
vim.opt.rtp:prepend(lazypath)

require("lazy").setup({
  require("dev.plugin.telescope"),
  require("dev.plugin.colorscheme"),
  require("dev.plugin.nvim-colorizer"),
  require("dev.plugin.harpoon"),
  require("dev.plugin.treesitter"),
  require("dev.plugin.lsp").lazy,
})

-- setup lsp
require("dev.plugin.lsp").setup()

as you can see it is just calling the setup and reading lazy fields from lsp.lua
Weirdly running :lua vim.cmd.Ex() works without the issue but having a keybind vim.keymap.set("n", "<leader>pv", vim.cmd.Ex) does not

This certainly looks like a bug in Neovim (or maybe just netrw). My guess is something is wrong with the value of the filetype.
I suggest using neovim v0.9.

I'm having the same problem on a nightly build:
NVIM v0.10.0-dev-278+ga72e795ed
It's easy to reproduce. Just open a file with Netrw then return to Netrw with :Ex. It'll apply the same language server to the Netrw page that was used for the file that was opened.
Pressing <C-g> while on the Netrw page indicates "[No Name]" before opening any file and indicates the name of the file that was open after returning with :Ex.
Others are having the same problem: https://www.reddit.com/r/neovim/comments/12ye5rc/lsp_errors_in_netrw_windows/

Yeah I am gonna try and see if it was an issue outside of lsp-zero, I dont remember this from my old config.


User study

Ref: https://ntnu.blackboard.com/ultra/courses/_34370_1/cl/outline

Discuss:

questions and walk-through scenarios
Interview objects - people who actually would use the app


Let the api to choose the default network driver.

That way swarm can understand the user's intention.
@tiborvass it turns out that we already have a bunch of tests in integration-cli/docker_api_network_test.go that already assume the driver is an empty string and let the daemon to select bridge as default network. I added an extra test here, but I don't think it's necessary.
Closes #17500
Signed-off-by: David Calavera<EMAIL_ADDRESS>

@calavera that's interesting, then why wasn't it caught?

@calavera needs rebase :D #thingsgotoofast

LGTM

Rebased, I removed the test because we have others that do the same. See:
https://github.com/docker/docker/blob/master/integration-cli/docker_api_network_test.go#L26
@tiborvass because we have a check for empty in https://github.com/docker/docker/blob/master/daemon/network.go#L91

thanks @calavera. LGTM.
I honestly don't remember why I made bridge as default at the client side.
But as you rightly spotted, it is not required.

@mavenugo it's not that it's not required, it's a bug if it's there :)

@tiborvass okay. i remember why I added it :) and it is not a bug. It was addressing this comment : https://github.com/docker/docker/pull/16645#issuecomment-145595087

having the right default in the help output makes sense. I don't know how we can check who set that value :pensive:

@calavera if we don't already, we could have an INFO log in the daemon logs to tell which driver was selected, better than nothing.

Okay, I found how to solve this issue for real, yay for reading the docs!
It turns out you can detect whether a user set a flag or not with flagSet.IsSet. So, we send an empty string if the user didn't set bridge as the driver. That way swarm knows what to do with the value.
@tiborvass, @mavenugo, @vieux please, take a look.

These are the parameters we send to the api when I run docker network create --driver bridge foo:
DEBU[0076] Calling POST /v1.22/networks/create
DEBU[0076] form data: map["options":map[] "name":"newbridge" "check_duplicate":%!q(bool=true) "driver":"bridge" "ipam":map["driver":"default" "config":[]]]

And this is what we send when I run docker network create bar:
DEBU[0100] Calling POST /v1.22/networks/create
DEBU[0100] form data: map["options":map[] "name":"bridge2" "check_duplicate":%!q(bool=true) "driver":"" "ipam":map["driver":"default" "config":[]]]

See that when I(the user) set the driver to bridge, we send that value to the api, but we don't send anything if I don't set the driver.
Moreover, docker network create -h still shows bridge as the default option:
Usage:  docker network create [OPTIONS] NETWORK-NAME

Creates a new network with a name specified by the user

  --aux-address=map[]      auxiliary ipv4 or ipv6 addresses used by Network driver
  -d, --driver=bridge      Driver to manage the Network
  --gateway=[]             ipv4 or ipv6 Gateway for the master subnet

even better thanks @LGTM

@calavera nice, that's what we also do for --tls. If others are fine with this i'm fine too

LGTM

dunno what windows ci is doing, merging


core-lib: Add docstring for add-hook-trigger!

I was browsing the source trying to understand how/when these doom hooks get triggered and found this TODO. Hope this description is accurate and helpful!

Thanks!


[feat] Docker Support

Hi maintainer @lllyasviel 🐱
There are too many feedbacks about running problems due to environmental reasons in the project, maybe there should be a stable Docker image(s)?
If necessary, I can provide some basic templates, including traditional gpu or coreml env.
// Played this project for a few days, it is still very interesting. Hope the project is getting better and better. 🍻

+1

I look forward to this feature.

I've set up a PR for this at #256.
Did an example run with the scribble generation. You guys can test it out.

When the docker builds, the models are not copied directly into the image since these models are so big.


Upgrade Ruby to 2.3.1

Signed-off-by: Dan Kohn<EMAIL_ADDRESS>

[Current coverage][cc-pull] is 96.54%

Merging [#357][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master       #357   diff @@
==========================================
  Files            32         32
  Lines           781        781
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            754        754
  Misses           27         27
  Partials          0          0

Powered by Codecov. Last updated by [6ad7b5f...59275ea][cc-compare]
[cc-base-branch]: https://codecov.io/gh/linuxfoundation/cii-best-practices-badge/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/linuxfoundation/cii-best-practices-badge/compare/6ad7b5f0a8e2ccb3a682b540ddb75e23977afac4...59275eaa679ff30628e98f90898d276d48c76e3c
[cc-pull]: https://codecov.io/gh/linuxfoundation/cii-best-practices-badge/pull/357?src=pr

Makes sense. I am currently only on my phone.  Is the last line of uninstall_gems.sh terminated with a normal newline?
I see some indications of oddness.
https://github.com/linuxfoundation/cii-best-practices-badge/blob/59275eaa679ff30628e98f90898d276d48c76e3c/lib/uninstall_gems.sh

The INSTALL.md instructions were wrong.  They probably should just install whatever .ruby-version says.

Updated.


User authentication

I wanted to request a feature or maybe get your take on this as a feature. I've been looking for something in Go to replace the vast PHP file upload apps that replicate Dropbox or puush. Although in Go, the only scripts I've found are temporary uploaders or single user file managers.
Do you think it'd be possible to support a user authentication system with signing up and isolated root folders for each user's uploads?

Hi camjac251, sorry but that sounds like another project all together :)
This project is a temporary url shortener and will include temporary files and temporary text blobs (like pastebin).
However one of the planed features is multiple logged in users that will be able to manage their links, text blobs and files so I guess that you could use it as stated in your request even if that is not the main purpose of the project (except for the sign up part).
When the features are implemented you can just fork the repo and remove all (for you) unnecessary parts or just use it "as is".
Cheers!

Different design and purpose, shorter will be a simple url shortener.


Settings property update

I noticed when attempting to set
{
actions: {
edit:false,
delete: false,
update: false
}
in my page settings that it wasn't carrying forward into the table settings.
after looking at the settings property update code its looks like
deepExtend(this.settings, this.defaultSettings);
works when it is changed to
deepExtend(this.defaultSettings, this.settings);
otherwise the page level settings get overwritten by the defaults;
https://github.com/akveo/ng2-smart-table/blob/master/src/ng2-smart-table/ng2-smart-table.component.ts#L181

@geneeblack yeah, this is a regression issue from this commit https://github.com/akveo/ng2-smart-table/commit/2ebac0ff9c36035ce17c02a851a5a46277f00f09, will have a look soon.

guys, could you try latest changes from master and let me know if everything works as expected? Thanks.

The way I was using settings was to send a partial description of new settings, i.e.
settings = {
actions: {
edit:false,
delete: false,
update: false
columns: {...}
}
I can see in #20 that they may have been attempting to set the settings.columns directly on the settings property rather than the assigning the whole settings property. I can see where this wouldn't work because it is not triggering the settings change. Maybe some type of general accessor 'set(propertykey) that would then trigger the property change might be middle ground where individual properties in settings could be changed while still triggering the change. Otherwise you could strong type the settings object and have onchange for each element. Although I am happy to send the whole settings object with only those differences that I wish to change from the default as long as those settings are overlaid on top of the defaults i.e. deepcopy(defaults, mysettings).

@geneeblack now this should work as you described, you can pass the whole settings object with the changes you need and it will be merged on top of the default settings object.

If I am understanding this right, you are only talking about when you pass the settings object into the component in the view template. What if I wanted to make a change to the table with a settings property after the view is rendered? For example, if I have a button bound to a function in my component that I want to hide the sub header, would I do it like this:
toggleSubHeader() {
console.log('you toggled the sub header');
this.settings.hideSubHeader = !this.settings.hideSubHeader;
}
or do I need to then pass the settings into the smart table component again somehow?

@jkon Hi, have the same question, this way don't seems to work for me , have you find a solution ?

unfortunately, you have to change this.settings to trigger onChanges event (it's a global angular2 binding issue)
so the right way is  :
toggleSubHeader() {
console.log('you toggled the sub header');
this.settings = {
hideSubHeader:!this.settings.hideSubHeader
};
}

I had a lot of other customizations that I wanted to do with this package so I made a copy of the repo locally and exposed the initGrid() function. Probably not smartest way, but it allows me to make all the changes I want.

Not a fan of this approach as it involves jQuery, but use jQuery to find the smart filters and wire up a button to trigger the toggle.
  public toggleFilter() {
    const filterRow = this.$('.ng2-smart-filters');
    this.hideSubHeader = !this.hideSubHeader;
    this.hideSubHeader ? filterRow.hide() : filterRow.show();
  }


Add docs for subcommands

:sparkles: Docs for subcommands, updated tests, and others.
Part of several local changes still to be pushed.

Codecov Report

Merging #28 into master will increase coverage by 2.63%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #28      +/-   ##
==========================================
+ Coverage   97.32%   99.95%   +2.63%
==========================================
  Files          99      135      +36
  Lines        1983     2388     +405
==========================================
+ Hits         1930     2387     +457
+ Misses         53        1      -52

Impacted Files
Coverage Δ

...est_subcommands/test_name_help/test_tutorial008.py
100% <100%> (ø)

docs/src/subcommands/name_help/tutorial004.py
100% <100%> (ø)

...test_tutorial/test_subcommands/test_tutorial002.py
100% <100%> (ø)

...est_subcommands/test_name_help/test_tutorial003.py
100% <100%> (ø)

...est_subcommands/test_name_help/test_tutorial006.py
100% <100%> (ø)

...test_tutorial/test_subcommands/test_tutorial001.py
100% <100%> (ø)

docs/src/subcommands/tutorial003/users.py
100% <100%> (ø)

...ommands/test_callback_override/test_tutorial003.py
100% <100%> (ø)

...ommands/test_callback_override/test_tutorial001.py
100% <100%> (ø)

docs/src/subcommands/tutorial001/users.py
100% <100%> (ø)

... and 65 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 d12d00e...5d7eb22. Read the comment docs.


[HOLD for payment 2023-01-05] [HOLD #13392] [$1000] Android - Avatar is not displayed on push notification

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Issue found when executing PR https://github.com/Expensify/App/pull/12540
Action Performed:

Sign into the Android app
Sign into another account on web and upload a custom profile picture for that account
Background the Android app
Send a message from the account signed in on web to the account signed in on Android
Verify that a push notification appears and that the notification appears a message style notification, including the avatar image of the web user.

Expected Result:
Avatar image of the web user is displayed after received a push notification in Android app
Actual Result:
Avatar image of the web user isn't displayed after received a push notification in Android app
Workaround:
Unknown
Platform:

Where is this issue occurring?

Android

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/203633324-ac7e4d46-4d31-45c2-a62e-338725caa2ff.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0171d7d835c1969527
Upwork Job ID:<PHONE_NUMBER>845291520

Bug0 Triage Checklist
Note: see this SO for more information.

[ ] The "bug" is actually a bug
[ ] The bug is not a duplicate report of an existing GH.

If it is, close the GH and add any novel details to the original GH instead

[ ] The bug is reproducible, following the reproduction steps.

[ ] If the GH doesn’t have steps to reliably reproduce the bug and you figure it out, then please add them.
[ ] If you’re unable to reproduce the bug, add the Needs reproduction label.
[ ] Comment on the issue outlining the steps you took to try to reproduce the bug, your results and tag the issue reporter and the Applause QA member who created the issue. Ask them to clarify reproduction steps and/or to check the reproduction steps again. Close issue.

[ ] The GH template is filled out as fully as possible

The GH body and title are clear (ie. could an external contributor understand it and work on it?)

[ ] The GH was created by an Expensify employee or a QA tester
[ ] If the bug is an OldDot issue, you should decide whether it is widespread enough to justify fixing or it is better to wait for reunification. If it’s better to wait, close the GH & provide this justification
[ ] If there's a link to Slack, check the discussion to see if we decided not to fix it
[ ] Decide if the GH should be resolved by an External contributor or Internal engineer, add the appropriate label

Reproduced following the steps in OP:

I could reproduce the issue as well on the debug build and started to debug the issue.
The following error occurred when fetching the icon and appears to occur on Android O (onwards):
12-01 17:21:31.539 24579 24622 E NotificationProvider: Failed to fetch icon
12-01 17:21:31.539 24579 24622 E NotificationProvider: java.lang.IllegalArgumentException: Software rendering doesn't support hardware bitmaps
12-01 17:21:31.539 24579 24622 E NotificationProvider: 	at android.graphics.BaseCanvas.throwIfHwBitmapInSwMode(BaseCanvas.java:706)
12-01 17:21:31.539 24579 24622 E NotificationProvider: 	at android.graphics.BaseCanvas.throwIfCannotDraw(BaseCanvas.java:81)
12-01 17:21:31.539 24579 24622 E NotificationProvider: 	at android.graphics.BaseCanvas.drawBitmap(BaseCanvas.java:139)
12-01 17:21:31.539 24579 24622 E NotificationProvider: 	at android.graphics.Canvas.drawBitmap(Canvas.java:1604)
12-01 17:21:31.539 24579 24622 E NotificationProvider: 	at com.expensify.chat.customairshipextender.CustomNotificationProvider.getCroppedBitmap(CustomNotificationProvider.java:110)
12-01 17:21:31.539 24579 24622 E NotificationProvider: 	at com.expensify.chat.customairshipextender.CustomNotificationProvider.fetchIcon(CustomNotificationProvider.java:265)
12-01 17:21:31.539 24579 24622 E NotificationProvider: 	at com.expensify.chat.customairshipextender.CustomNotificationProvider.applyMessageStyle(CustomNotificationProvider.java:151)
12-01 17:21:31.539 24579 24622 E NotificationProvider: 	at com.expensify.chat.customairshipextender.CustomNotificationProvider.onExtendBuilder(CustomNotificationProvider.java:76)
12-01 17:21:31.539 24579 24622 E NotificationProvider: 	at com.urbanairship.push.notifications.AirshipNotificationProvider.onCreateNotification(AirshipNotificationProvider.java:210)
12-01 17:21:31.539 24579 24622 E NotificationProvider: 	at com.urbanairship.push.IncomingPushRunnable.postProcessPush(IncomingPushRunnable.java:196)
12-01 17:21:31.539 24579 24622 E NotificationProvider: 	at com.urbanairship.push.IncomingPushRunnable.processPush(IncomingPushRunnable.java:144)
12-01 17:21:31.539 24579 24622 E NotificationProvider: 	at com.urbanairship.push.IncomingPushRunnable.run(IncomingPushRunnable.java:97)
12-01 17:21:31.539 24579 24622 E NotificationProvider: 	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:463)
12-01 17:21:31.539 24579 24622 E NotificationProvider: 	at java.util.concurrent.FutureTask.run(FutureTask.java:264)
12-01 17:21:31.539 24579 24622 E NotificationProvider: 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1137)
12-01 17:21:31.539 24579 24622 E NotificationProvider: 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:637)
12-01 17:21:31.539 24579 24622 E NotificationProvider: 	at com.urbanairship.util.AirshipThreadFactory$1.run(AirshipThreadFactory.java:50)
12-01 17:21:31.539 24579 24622 E NotificationProvider: 	at java.lang.Thread.run(Thread.java:1012)

To resolve the issue, we could convert the avatar to the desired bitmap (as used by the canvas we create in getCroppedBitmap)
// android/app/src/main/java/com/expensify/chat/customairshipextender/CustomNotificationProvider.java
public Bitmap getCroppedBitmap(Bitmap bitmap) {
        bitmap = bitmap.copy(Config.ARGB_8888, true);
        // ...
}

Thats would resolve the bug as seen here.

Thanks for the proposal, it looks like this is indeed the source of the issue! Just out of curiosity, is there a reason why we don't use PixelCopy? https://bumptech.github.io/glide/doc/hardwarebitmaps.html

@StefanNemeth Thanks for the proposal, it looks like this is indeed the source of the issue! Just out of curiosity, is there a reason why we don't use PixelCopy? https://bumptech.github.io/glide/doc/hardwarebitmaps.html

According to the documentation PixelCopy allows for copy operations from Surface to Bitmap (e.g. for screenshots). We are dealing with bitmaps here so bitmap.copy is the proper way to do it afaik.

Got it, that makes sense. @parasharrajat What are your thoughts on the above proposal? 🙏

I am not an Android native guy but can we just use bitmap.setConfig(Config.ARGB_8888) before drawing? it seems that it will have the same effect.
https://github.com/Expensify/App/blame/main/android/app/src/main/java/com/expensify/chat/customairshipextender/CustomNotificationProvider.java#L107
I don't see where we create the Bitmap with Hardware config. It may be coming directly from AirShip.

Ok, Thanks for the explanation. I am good with both. I have asked internally to review it before we proceed.

Ok @StefanNemeth 's proposal looks good to me.
Cc: @robertjchen
:ribbon: :eyes: :ribbon:  C+ reviewed

Hey @robertjchen ! Just wanted to add here the reasoning why I internally confirmed that the solution is good:
The avatar is downloaded as a hardware bitmap (a new optimised bitmap added in Android O) but we crop it to make it circular. Crop operation uses software Canvas which doesn't support hardware bitmap rendering. @StefanNemeth solution basically converts the hardware bitmap into a software bitmap (bitmap.copy(Config.ARGB_8888, true)) which is successfully rendered by the Canvas.

Thanks for the confirmation @cristipaval !
@dylanexpensify Let's move forward with hiring @StefanNemeth for this 👍

Created the PR

@StefanNemeth offer sent! @parasharrajat please apply when you get a moment 😄

ah, @robertjchen looks like @arielgreen is the BZ member for this 😂

Am I eligible for the 50% bonus as the PR created within a few hours after assignment was held?

@StefanNemeth yes, I think that is fair. Paid. @parasharrajat sending offer now.

[@parasharrajat / @robertjchen] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:

it's not clear to me what exactly caused the regression in the linked PR. I recall that notification profile images have been broken for weeks/months now.

Ahhh, I see. That PR was January 2022, not 2023 😄

All payments have been issued.

Hey @arielgreen I think this should be added as a new regression test, as it ensures we test for custom profile pics as well as the default icons:
Ensure rounded profile images are displayed for notifications

Android and iOS only
Sign in as a new account (userA) on the test platform (iOS/Android)
Sign in as userB on a different device/browser/platformfor that account
Background the app as userA
Send a message from userB to userA
Verify that a push notification appears for userA and that the notification includes a rounded default avatar
As userB upload a custom profile picture for that account
Send a message from userB to userA
Verify that a push notification appears for userA and that the notification includes a rounded version of the custom profile image

Discussing regression test here.

https://github.com/Expensify/Expensify/issues/256144


[Feature Request] Time tracking and time boxing enhancements

Hello,
my goal is to add better time boxing and task time tracking capabilities to obsidian. Day Planner has enabled myself to plan my day better but there are some issues/ enhancements I would like to address. I do not want to just create this issue, but I am also looking forward helping to implement parts where my help is wanted.

-I have problems creating tasks in my daily note that are for another day. (EXAMPLE WILL FOLLOW SOON HERE). Is this an expected behavior or not? I haven't looked at the code yet, maybe you could show me the specific part where the problem lies.

-I would love to see Day Planner to respect recurring tasks in advance. Currently Day Planner only shows tasks that are created. But recurring tasks are only created when the last task is completed. I hope you see the problem here, because I cannot Plan weeks in advance, when I'm not seeing all my recurring tasks. Yes I could implement daily recurring tasks in my daily template, but that wouldn't be a great solution.

-I'm currently not seeing a standardized way to store intervals someone worked on a task. In #330 you mentioned one example of a format, but for example eatgrass/obsidian-pomodoro-timer has a completely different approach to the problem. I also don't know if it would be a better idea to implement these formats directly into the obsidian-tasks plugin. I hope @claremacrae joins this discussion/ issue and shares her ideas. Maybe something from https://github.com/obsidian-tasks-group/obsidian-tasks/issues/2502 could be helpful.

I saw that your tracker branch is inactive for some time but I would love to see the feature shown in https://github.com/obsidian-tasks-group/obsidian-tasks/issues/2502 released into production.
I'm looking forward to your feedback and hope that we find a solution that we can then code into the respective plugin. Each plugin should focus on its core capabilities.
Thank you.
~ Marius

Hello,
thank you for the fast reply. I will separate this into multiple issues. Furthermore I will not reference the tasks plugin anymore, because of obvious reasons stated in your referenced issue about the separation of task and time tracking.
~ Marius

@DeveloperMarius Thanks very much.
It may be that Tasks can help support the implementation of some aspects of this, such as perhaps allowed its date fields to have optional time values...
That definitely would be in-scope for tasks, and we have lots of requests for that, including:

https://github.com/obsidian-tasks-group/obsidian-tasks/discussions/64

So I will be very happy to be mentioned in more focussed issues at the appropriate time...


Feature Request: Contacts API v3.0

Contacts API v3.0 is available at the endpoint: https://www.google.com/m8/feeds
I am already using googleapis. It will be great if the Contacts API endpoint is available for use.

I get a 404 when I click the link you provided.


Remove java.xml.bind dependency.

java.xml.bind is under a separate module in Java 9 and needs to be
included with a command line switch. Implement GZip trailer calculation
without javax.xml.bind.DatatypeConverter for better Java 9
compatibility.

Codecov Report

Merging #1438 into master will decrease coverage by 0.32%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1438      +/-   ##
==========================================
- Coverage   73.84%   73.51%   -0.33%
==========================================
  Files         235      235
  Lines        5050     5049       -1
  Branches      279      287       +8
==========================================
- Hits         3729     3712      -17
- Misses       1321     1337      +16

Impacted Files
Coverage Δ

...main/scala/org/http4s/server/middleware/GZip.scala
95.65% <100%> (-0.1%)
:arrow_down:

.../scala/org/http4s/testing/ArbitraryInstances.scala
92.21% <0%> (-6.97%)
:arrow_down:

...src/main/scala/org/http4s/client/PoolManager.scala
68.31% <0%> (+0.99%)
:arrow_up:

Continue to review full report at Codecov.

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


[QUESTION] [GPU] [Angle] [WPF] Any safe way to FlushAsync() or Flush in other thread?

Canvas.Flush() is taking a lot of time (up to 2 seconds) in my scenarios with huge amount of elements on a canvas, blocking my render thread.
I want to draw next frame, while i'm waiting for previous frame render. Some kind of double buffer render.
Is anyone faced this type issue and found a good solution?
I'm trying to launch Flush() in another thread. It works for a while, works perfectly(huge lag reduce), but unsafe, has random crashes on flush and image draw skia calls.
AFAIK All Flush() is do - is just an endless loop wait for GPU response. So I think I'm doing something wrong and cannot find an obvious solution.

I've found, that the reason of the long flush() calls in a most of scenarios is just a small ResourceCacheLimit in GrContext.
If skia needs more then a cache limit to draw a frame - it will create and delete textures from cache on each frame to draw it. It takes a lot of time.
After enlarging it the problem do not exist anymore, so I don't need to find a way to run flush in other thread.

Thanks for the update on this. Why are there so many textures? Is it related to what you are drawing?

Yes. In our app users can make really big files with thousands of small textures and some huge textures. Small ResourceCacheLimit is a real sneaky bottleneck. Was really hard to find.


Deprecate enable-ha cli command

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Steps to reproduce

Does this PR introduce a user-facing change?

Deprecates the `kots enable-ha` CLI command

Does this PR require documentation?

NONE

Looks like cobra also has a Deprecated field for this, though I have no issues with doing it this way either.


When to use includePolyfill: true?

It is not clear to me what the status on includePolyfill: true is. There are some outstanding issues/PRs from 2017:
https://github.com/babel/ember-cli-babel/issues/166
https://github.com/babel/ember-cli-babel/pull/142
https://github.com/babel/ember-cli-babel/pull/187
Then when looking at core-js, it says that babel-preset-env should take care of that, but not sure if that applies here (see https://github.com/zloirock/core-js/issues/188)?
Also, I'm not quite sure how this plays together with https://github.com/machty/ember-maybe-import-regenerator.
Basically, in what scenarios do I need to set includePolyfill: true? If we target fairly modern browsers (e.g. last 1 chrome, safari, firefox), what do we need to include? The polyfill? Or only ember-maybe-import-regenerator? Or neither of them? I could not find any definite information on this, and since the polyfill does add about 30KB gzipped to our total bundle size, it would be a fairly big benefit in being able to drop it.
Maybe someone can help clear this up for me :)

@mydea https://github.com/pzuraq/ember-cli-babel-polyfills is a recent addon that aims to help resolve your question ...

Going to close this as polyfill support was removed in v8.


Pass config dtype into RotaryEmbedding

Notes
Use the same dtype before and after calling RotaryEmbedding. Related to issue 595.
Not passing dtype into RotaryEmbedding was causing the keys to be cast to bfloat16. This caused the linked issue where running decode with float32 was throwing this exception:
AssertionError: Key and Value Dtypes should match

Note that I applied the same change for the query since this was also being cast to bfloat16. Let me know if I should keep the query casting the same as it was.
Testing

Successfully ran decode with bfloat16/float32. I was able to see key/value (and query) dtype matching for both runs. I also saw them not matching before this change

key: Traced<ShapedArray(float32[4,1,16,128])>with<DynamicJaxprTrace(level=3/0)>
value: Traced<ShapedArray(float32[4,1,16,128])>with<DynamicJaxprTrace(level=3/0)>

Successfully ran training with bfloat16/float32

Add a test!

Add a test!

Done, thanks @rwitten. Great to hear from you and hope things are going well


Wrong (outdated) module path in go.mod

Could you please put the current path https://github.com/qax-os/excelize here https://github.com/qax-os/excelize/blob/master/go.mod (qax-os instead of xuri) - go mod tidy fails because of it

Thanks for your issue. Go allows module path inconsistence with the repository path, there is no plan to change the Go module path. Please delete go mod cache $GOPATH/pkg/mod/cache by go clean -modcache and try again, also reference issues #393, #403, #488, #522, #973, #1020, #1079, #1472 and #1744.

Thanks for your issue. Go allows module path inconsistence with the repository path, there is no plan to change the Go module path. Please delete go mod cache $GOPATH/pkg/mod/cache by go clean -modcache and try again, also reference issues #393, #403, #488, #522, #973, #1020, #1079, #1472 and #1744. I'll close this issue. If you have any questions, please let me know, and you can reopen this anytime.


nixos/nextcloud: Add ensureUsers option

Description of changes
Aiming to get rid of the config option in the Nextcloud module, having them all in the free form type extraOptions which can be renamed to settings in accordance to RFC42 in a later PR.
Instead of defining a single admin user with config.adminuser and config.adminpassFile switch to a more generic approach which is used by other modules, having a ensureUsers attribute set.
services.nextcloud = {
  enable = true;
  package = pkgs.nextcloud27;
  hostName = "localhost";
  config = {
    adminpassFile = "${pkgs.writeText "adminpass" "hunter2"}";
  };
  database.createLocally = true;
  ensureUsers = {
    user1 = {
      email = "user1@localhost";
      passwordFile = /secrets/user1.secret;
    };
    user2 = {
      email = "user1@localhost";
      passwordFile = /secrets/user1.secret;
    };
  };
};

For now I would like to only add the ensureUsers option to add additional users. In a later PR I would like to manage admin user configuration with this option.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[x] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@Ma27 Good point, I agree on this. But this argument somehow also applies to the current implementation of nextcloud.config.adminuser and nextcloud.config.adminpasswordFile.
As pointed out in the PR description, it would be nice to move all config options into the freeForm type extraOptions which could be renamed to settings.
So if I manage to migrate the adminuser into ensureUsers we'll still stick to a non-declerative approach as before but then with a more generic interface?
Being able to add additional users is helpful with bootstrapping test and dev environments using the NixOS Nextcloud module :)
But nethertheless I understand your concern :)

Thank you @Ma27 for the feedback, sounds reasonable! Going to look on how to integrate this into an external flake module in case it's useful for someone

Is someone still interested in using the ensureUsers option, I added a practical configuration example to the NixOS wiki https://nixos.wiki/wiki/Nextcloud#Add_users_declaratively


Update sbt-git to 1.0.0

Updates com.typesafe.sbt:sbt-git from 0.9.3 to 1.0.0.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention @scala-steward in the comments below.
Have a nice day!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [{ groupId = "com.typesafe.sbt", artifactId = "sbt-git" }]

superseded by #72


Error thown when choosing other recipe first time

When you use the option: Choose Other Recipe in the period screen.
On the first time another recipe is selected, an error 404 is trown.
URL looks like:
localhost:8080/;jsessionid=EB21CC0A9257CF5BA5C225D7026F8F97
second time error does not occur

Can not reproduce in dev. So marking as closed for now


Hyperlink in mermaid SVG diagram is not working

Hi,
I am using kroki to render the following example mermaid diagram:
graph LR;
    A-->B;
    B-->C;
    C-->D;
    D-->E;
    click A "http://www.github.com" _blank
    click B "http://www.github.com" "Open this in a new tab" _blank
    click C href "http://www.github.com" _blank
    click D href "http://www.github.com" "Open this in a new tab" _blank

Output in sphinx page:

Looks like the hyperlink in SVG is not clickable.
I tried enabling inline SVG in conf.py, but still the same output. Although when I open SVG image alone in the new TAB, hyperlinks work fine.
Am I missing something?

Confirmed...sadly...


allow selection of tags without prepending + [discussion]

The +tag syntax makes sense when defining a task; but it trips me up when switching  contexts etc.
Would it make sense to allow tags to be referenced without the +?

I have the same feeling. I wanted to be consistent, but it seems I'm not the only one who has trouble with. I will push sth soon. Thanks :)

I made the + optional for context and worktime commands. It will be available in the next release.


TGI docker error: LLama2 70b GPTQ or AWQ models do not load.  weight model.layers.0.self_attn.q_proj.weight does not exist

System Info
text-generation-inference docker, did not install text-generation-launcher
cargo 1.72.1 (103a7ff2e 2023-08-15)
Model tried : TheBloke/Llama-2-70B-chat-GPTQ
Hardware: A10 GPU, g5.12xlarge, 4 GPUs
NVIDIA-SMI 535.104.05
Driver Version: 535.104.05
CUDA Version: 12.2
Deployment: AWS EC2 containers
We just spun up the docker for various models to try. When we try GPTQ or AWQ versions of LLAMA 2 70b, docker fails to load as model initialization fails with below error.
Command run:
docker run --gpus all --shm-size 1g -p 8080:80 -v $volume:/data ghcr.io/huggingface/text-generation-inference:latest --model-id TheBloke/Llama-2-70B-chat-GPTQ
Error: weight model.layers.0.self_attn.q_proj.weight does not exist
Details:
File "/opt/conda/lib/python3.9/site-packages/text_generation_server/cli.py", line 83, in serve
server.serve(
File "/opt/conda/lib/python3.9/site-packages/text_generation_server/server.py", line 207, in serve
asyncio.run(
File "/opt/conda/lib/python3.9/asyncio/runners.py", line 44, in run
return loop.run_until_complete(main)
File "/opt/conda/lib/python3.9/asyncio/base_events.py", line 647, in run_until_complete
return future.result()
File "/opt/conda/lib/python3.9/site-packages/text_generation_server/server.py", line 159, in serve_inner
model = get_model(
File "/opt/conda/lib/python3.9/site-packages/text_generation_server/models/init.py", line 201, in get_model
return FlashLlama(
File "/opt/conda/lib/python3.9/site-packages/text_generation_server/models/flash_llama.py", line 68, in init
model = FlashLlamaForCausalLM(config, weights)
File "/opt/conda/lib/python3.9/site-packages/text_generation_server/models/custom_modeling/flash_llama_modeling.py", line 478, in init
self.model = FlashLlamaModel(config, weights)
File "/opt/conda/lib/python3.9/site-packages/text_generation_server/models/custom_modeling/flash_llama_modeling.py", line 416, in init
[
File "/opt/conda/lib/python3.9/site-packages/text_generation_server/models/custom_modeling/flash_llama_modeling.py", line 417, in
FlashLlamaLayer(
File "/opt/conda/lib/python3.9/site-packages/text_generation_server/models/custom_modeling/flash_llama_modeling.py", line 353, in init
self.self_attn = FlashLlamaAttention(
File "/opt/conda/lib/python3.9/site-packages/text_generation_server/models/custom_modeling/flash_llama_modeling.py", line 233, in init
self.query_key_value = load_attention(config, prefix, weights)
File "/opt/conda/lib/python3.9/site-packages/text_generation_server/models/custom_modeling/flash_llama_modeling.py", line 154, in load_attention
return _load_gqa(config, prefix, weights)
File "/opt/conda/lib/python3.9/site-packages/text_generation_server/models/custom_modeling/flash_llama_modeling.py", line 177, in _load_gqa
weight = weights.get_multi_weights_col(
File "/opt/conda/lib/python3.9/site-packages/text_generation_server/utils/weights.py", line 217, in get_multi_weights_col
w = [self.get_sharded(f"{p}.weight", dim=0) for p in prefixes]
File "/opt/conda/lib/python3.9/site-packages/text_generation_server/utils/weights.py", line 217, in
w = [self.get_sharded(f"{p}.weight", dim=0) for p in prefixes]
File "/opt/conda/lib/python3.9/site-packages/text_generation_server/utils/weights.py", line 112, in get_sharded
filename, tensor_name = self.get_filename(tensor_name)
File "/opt/conda/lib/python3.9/site-packages/text_generation_server/utils/weights.py", line 63, in get_filename
raise RuntimeError(f"weight {tensor_name} does not exist")
RuntimeError: weight model.layers.0.self_attn.q_proj.weight does not exist
rank=0
Error: ShardCannotStart
2023-10-07T04:16:56.653807Z ERROR text_generation_launcher: Shard 0 failed to start
2023-10-07T04:16:56.653841Z  INFO text_generation_launcher: Shutting down shards
Information

[X] Docker
[ ] The CLI directly

Tasks

[X] An officially supported command
[ ] My own modifications

Reproduction
docker run --gpus all --shm-size 1g -p 8080:80 -v $volume:/data ghcr.io/huggingface/text-generation-inference:latest --model-id TheBloke/Llama-2-70B-chat-GPTQ
Expected behavior
Docker spins up with text-generation-inference service, with LLAMA 2 70b GPTQ models OR LLAMA 2 AWQ versions of the models.

Was there a solution to this?

+1, seeing this issue on GPTQ models consistently, would love to hear the solution @nm-narasimha


Can no longer add events to calendar in the Learning Space theme

In testing creating sites Kristen and I have found a discrepancy between the theme on HC and on MSU-C. This same issue exists on production.
On HC there is no option under "New" to enter a new event: https://hctester100-rosie.hcommons-dev.org/
On MSU-C that link : https://bonnie-rosiemsu.msu.hcommons.org/
It looks like there's a plugin missing on hc-dev that exists on msu-c.
On MSU Commons the only place to add an event is that top bar under new. Events do not exist in the left-hand nav bar in wp-admin. Is there a way to add them there as well?
Rosie Test is hctester100.

Learning space theme developed by MLA. Pass it off to MLA's developer?

I'm not sure why the toolbar is different across networks, but the functionality is there. I was able to add an event on the test site on HC. This post gives instructions on how to do so via wp-admin: https://learningspace.hcommons.org/assignment/create-an-event/.

Should do some investigation about what's going on here.

This appears to be working.


Flags overwriting config file

Previously the test file would over write any flags passed to
influx_stress, now flags overwrite config file options

@otoolep

+1, thanks.


Conv3D: gemm consuming large amount of memory causing TF to crash

I run a 3d conv network.
I have a problem
Has anyone encountered it?
Predict volume shape:  (24, 31, 31)
2019-06-06 16:58:04.595653: F tensorflow/stream_executor/rocm/rocm_dnn.cc:494] miopen only supports 4D tensors, dim=5 not allowed
GPU:radeon vii

@nuu9323226 could you provide the steps to repro your issue?
3D convolution support has been added in the following PR:
https://github.com/ROCmSoftwarePlatform/tensorflow-upstream/pull/381
Please use the following docker image to preview the support and let us know if that works:
rocm/tensorflow:rocm2.4-tf1.14-python3-preview
CC @jerryyin

@sunway513
i install docker and run rocm/tensorflow:rocm2.4-tf1.14-python3-preview
But still fail
`
Computer:  None
Using TensorFlow backend.
WARNING: Logging before flag parsing goes to stderr.
W0611 09:44:09.838078<PHONE_NUMBER>87200 deprecation_wrapper.py:119] From step2_train_nodule_detector.py:27: The name tf.ConfigProto is deprecated. Please use tf.compat.v1.ConfigProto instead.
W0611 09:44:09.838232<PHONE_NUMBER>87200 deprecation_wrapper.py:119] From step2_train_nodule_detector.py:29: The name tf.Session is deprecated. Please use tf.compat.v1.Session instead.
2019-06-11 09:44:09.869179: I tensorflow/core/platform/profile_utils/cpu_utils.cc:94] CPU Frequency:<PHONE_NUMBER> Hz
2019-06-11 09:44:09.870005: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x74b20d0 executing computations on platform Host. Devices:
2019-06-11 09:44:09.870022: I tensorflow/compiler/xla/service/service.cc:175]   StreamExecutor device (0): ,
2019-06-11 09:44:09.871167: I tensorflow/stream_executor/platform/default/dso_loader.cc:43] Successfully opened dynamic library libhip_hcc.so
2019-06-11 09:44:09.871574: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x7514d80 executing computations on platform ROCM. Devices:
2019-06-11 09:44:09.871583: I tensorflow/compiler/xla/service/service.cc:175]   StreamExecutor device (0): Device 66af, AMDGPU ISA version: gfx906
2019-06-11 09:44:09.871588: I tensorflow/compiler/xla/service/service.cc:175]   StreamExecutor device (1): Device 66af, AMDGPU ISA version: gfx906
2019-06-11 09:44:09.907733: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1613] Found device 0 with properties:
name: Device 66af
AMDGPU ISA: gfx906
memoryClockRate (GHz) 1.802
pciBusID 0000:05:00.0
2019-06-11 09:44:09.907784: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1613] Found device 1 with properties:
name: Device 66af
AMDGPU ISA: gfx906
memoryClockRate (GHz) 1.802
pciBusID 0000:08:00.0
2019-06-11 09:44:09.907836: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1715] Adding visible gpu devices: 0, 1
2019-06-11 09:44:09.907852: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1145] Device interconnect StreamExecutor with strength 1 edge matrix:
2019-06-11 09:44:09.907860: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1151]      0 1
2019-06-11 09:44:09.907866: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1164] 0:   N N
2019-06-11 09:44:09.907872: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1164] 1:   N N
2019-06-11 09:44:09.907969: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1288] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 8184 MB memory) -> physical GPU (device: 0, name: Device 66af, pci bus id: 0000:05:00.0)
2019-06-11 09:44:09.987733: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1288] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:1 with 8184 MB memory) -> physical GPU (device: 1, name: Device 66af, pci bus id: 0000:08:00.0)
Get train/holdout files.
Pos samples:  217
Pos samples manual:  31
Ndsb3 samples:  12
0  ndsb3 pos labels train
0  ndsb3 neg labels train
0  ndsb3 pos labels holdout
0  ndsb3 neg labels holdout
Edge samples:  159784
Luna samples:  356660
Falsepos LUNA count:  6233
Pos 248
ndsb2 pos:  0
ndsb2 neg:  0
Pos 50
ndsb2 pos:  0
ndsb2 neg:  0
Train count:  561901 , holdout count:  108454
W0611 09:44:12.604604<PHONE_NUMBER>87200 deprecation_wrapper.py:119] From /usr/local/lib/python3.5/dist-packages/keras/backend/tensorflow_backend.py:321: The name tf.placeholder is deprecated. Please use tf.compat.v1.placeholder instead.
W0611 09:44:12.606200<PHONE_NUMBER>87200 deprecation.py:506] From /usr/local/lib/python3.5/dist-packages/keras/backend/tensorflow_backend.py:634: calling RandomUniform.init (from tensorflow.python.ops.init_ops) with dtype is deprecated and will be removed in a future version.
Instructions for updating:
Call initializer instance with the dtype argument instead of passing it to the constructor
W0611 09:44:12.610019<PHONE_NUMBER>87200 deprecation.py:506] From /usr/local/lib/python3.5/dist-packages/keras/backend/tensorflow_backend.py:491: calling Constant.init (from tensorflow.python.ops.init_ops) with dtype is deprecated and will be removed in a future version.
Instructions for updating:
Call initializer instance with the dtype argument instead of passing it to the constructor
W0611 09:44:12.666750<PHONE_NUMBER>87200 deprecation.py:506] From /usr/local/lib/python3.5/dist-packages/keras/backend/tensorflow_backend.py:1047: calling reduce_prod_v1 (from tensorflow.python.ops.math_ops) with keep_dims is deprecated and will be removed in a future version.
Instructions for updating:
keep_dims is deprecated, use keepdims instead
W0611 09:44:12.684238<PHONE_NUMBER>87200 deprecation_wrapper.py:119] From /usr/local/lib/python3.5/dist-packages/keras/optimizers.py:658: The name tf.train.Optimizer is deprecated. Please use tf.compat.v1.train.Optimizer instead.
W0611 09:44:12.687177<PHONE_NUMBER>87200 deprecation_wrapper.py:119] From /usr/local/lib/python3.5/dist-packages/keras/backend/tensorflow_backend.py:2446: The name tf.log is deprecated. Please use tf.math.log instead.
W0611 09:44:12.688659<PHONE_NUMBER>87200 deprecation_wrapper.py:119] From /usr/local/lib/python3.5/dist-packages/keras/backend/tensorflow_backend.py:2448: The name tf.nn.sigmoid_cross_entropy_with_logits is deprecated. Please use tf.nn.sigmoid_cross_entropy_with_logits instead.

Layer (type)                                  Output Shape                   Param #         Connected to
input_1 (InputLayer)                          (None, 32, 32, 32, 1)          0

averagepooling3d_1 (AveragePooling3D)         (None, 16, 32, 32, 1)          0               input_1[0][0]

conv1 (Convolution3D)                         (None, 16, 32, 32, 64)         1792            averagepooling3d_1[0][0]

pool1 (MaxPooling3D)                          (None, 16, 16, 16, 64)         0               conv1[0][0]

conv2 (Convolution3D)                         (None, 16, 16, 16, 128)        221312          pool1[0][0]

pool2 (MaxPooling3D)                          (None, 8, 8, 8, 128)           0               conv2[0][0]

conv3a (Convolution3D)                        (None, 8, 8, 8, 256)           884992          pool2[0][0]

conv3b (Convolution3D)                        (None, 8, 8, 8,<PHONE_NUMBER>         conv3a[0][0]

pool3 (MaxPooling3D)                          (None, 4, 4, 4, 256)           0               conv3b[0][0]

conv4a (Convolution3D)                        (None, 4, 4, 4,<PHONE_NUMBER>         pool3[0][0]

conv4b (Convolution3D)                        (None, 4, 4, 4,<PHONE_NUMBER>         conv4a[0][0]

pool4 (MaxPooling3D)                          (None, 2, 2, 2, 512)           0               conv4b[0][0]

last_64 (Convolution3D)                       (None, 1, 1, 1, 64)            262208          pool4[0][0]

out_class_last (Convolution3D)                (None, 1, 1, 1, 1)             65              last_64[0][0]

out_malignancy_last (Convolution3D)           (None, 1, 1, 1, 1)             65              last_64[0][0]

out_class (Flatten)                           (None, 1)                      0               out_class_last[0][0]

out_malignancy (Flatten)                      (None, 1)                      0               out_malignancy_last[0][0]
Total params: 13,758,018
Trainable params: 13,758,018
Non-trainable params: 0

W0611 09:44:12.950066<PHONE_NUMBER>87200 deprecation_wrapper.py:119] From /usr/local/lib/python3.5/dist-packages/keras/backend/tensorflow_backend.py:736: The name tf.assign is deprecated. Please use tf.compat.v1.assign instead.
learnrate:  0.001  epoch:  0
2019-06-11 09:44:13.452114: E tensorflow/stream_executor/rocm/rocm_driver.cc:630] failed to allocate 7.99G<PHONE_NUMBER> bytes) from device: hipError_t(1002)
2019-06-11 09:44:13.452147: E tensorflow/stream_executor/rocm/rocm_driver.cc:630] failed to allocate 7.19G<PHONE_NUMBER> bytes) from device: hipError_t(1002)
2019-06-11 09:44:13.452162: E tensorflow/stream_executor/rocm/rocm_driver.cc:630] failed to allocate 6.47G<PHONE_NUMBER> bytes) from device: hipError_t(1002)
2019-06-11 09:44:13.452176: E tensorflow/stream_executor/rocm/rocm_driver.cc:630] failed to allocate 5.83G<PHONE_NUMBER> bytes) from device: hipError_t(1002)
2019-06-11 09:44:13.452193: E tensorflow/stream_executor/rocm/rocm_driver.cc:630] failed to allocate 5.24G<PHONE_NUMBER> bytes) from device: hipError_t(1002)
2019-06-11 09:44:13.452234: E tensorflow/stream_executor/rocm/rocm_driver.cc:630] failed to allocate 4.72G<PHONE_NUMBER> bytes) from device: hipError_t(1002)
2019-06-11 09:44:13.452255: E tensorflow/stream_executor/rocm/rocm_driver.cc:630] failed to allocate 4.25G<PHONE_NUMBER> bytes) from device: hipError_t(1002)
2019-06-11 09:44:13.452278: E tensorflow/stream_executor/rocm/rocm_driver.cc:630] failed to allocate 3.82G<PHONE_NUMBER> bytes) from device: hipError_t(1002)
2019-06-11 09:44:13.452299: E tensorflow/stream_executor/rocm/rocm_driver.cc:630] failed to allocate 3.44G<PHONE_NUMBER> bytes) from device: hipError_t(1002)
2019-06-11 09:44:13.452320: E tensorflow/stream_executor/rocm/rocm_driver.cc:630] failed to allocate 3.10G<PHONE_NUMBER> bytes) from device: hipError_t(1002)
2019-06-11 09:44:13.452341: E tensorflow/stream_executor/rocm/rocm_driver.cc:630] failed to allocate 2.79G<PHONE_NUMBER> bytes) from device: hipError_t(1002)
2019-06-11 09:44:13.452363: E tensorflow/stream_executor/rocm/rocm_driver.cc:630] failed to allocate 2.51G<PHONE_NUMBER> bytes) from device: hipError_t(1002)
2019-06-11 09:44:13.452383: E tensorflow/stream_executor/rocm/rocm_driver.cc:630] failed to allocate 2.26G<PHONE_NUMBER> bytes) from device: hipError_t(1002)
2019-06-11 09:44:13.452404: E tensorflow/stream_executor/rocm/rocm_driver.cc:630] failed to allocate 2.03G<PHONE_NUMBER> bytes) from device: hipError_t(1002)
2019-06-11 09:44:13.452425: E tensorflow/stream_executor/rocm/rocm_driver.cc:630] failed to allocate 1.83G<PHONE_NUMBER> bytes) from device: hipError_t(1002)
2019-06-11 09:44:13.452445: E tensorflow/stream_executor/rocm/rocm_driver.cc:630] failed to allocate 1.65G<PHONE_NUMBER> bytes) from device: hipError_t(1002)
2019-06-11 09:44:13.452465: E tensorflow/stream_executor/rocm/rocm_driver.cc:630] failed to allocate 1.48G<PHONE_NUMBER> bytes) from device: hipError_t(1002)
2019-06-11 09:44:13.452485: E tensorflow/stream_executor/rocm/rocm_driver.cc:630] failed to allocate 1.33G<PHONE_NUMBER> bytes) from device: hipError_t(1002)
2019-06-11 09:44:13.452505: E tensorflow/stream_executor/rocm/rocm_driver.cc:630] failed to allocate 1.20G<PHONE_NUMBER> bytes) from device: hipError_t(1002)
2019-06-11 09:44:13.452526: E tensorflow/stream_executor/rocm/rocm_driver.cc:630] failed to allocate 1.08G<PHONE_NUMBER> bytes) from device: hipError_t(1002)
2019-06-11 09:44:13.452547: E tensorflow/stream_executor/rocm/rocm_driver.cc:630] failed to allocate 994.98M<PHONE_NUMBER> bytes) from device: hipError_t(1002)
2019-06-11 09:44:13.452568: E tensorflow/stream_executor/rocm/rocm_driver.cc:630] failed to allocate 895.48M (938983680 bytes) from device: hipError_t(1002)
2019-06-11 09:44:13.452588: E tensorflow/stream_executor/rocm/rocm_driver.cc:630] failed to allocate 805.94M (845085440 bytes) from device: hipError_t(1002)
2019-06-11 09:44:13.452608: E tensorflow/stream_executor/rocm/rocm_driver.cc:630] failed to allocate 725.34M (760577024 bytes) from device: hipError_t(1002)
2019-06-11 09:44:13.452627: E tensorflow/stream_executor/rocm/rocm_driver.cc:630] failed to allocate 652.81M (684519424 bytes) from device: hipError_t(1002)
2019-06-11 09:44:15.019331: E tensorflow/stream_executor/rocm/rocm_driver.cc:493] failed to memset memory: HIP_ERROR_InvalidValue
Traceback (most recent call last):
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/client/session.py", line 1354, in _do_call
return fn(*args)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/client/session.py", line 1339, in _run_fn
options, feed_dict, fetch_list, target_list, run_metadata)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/client/session.py", line 1427, in _call_tf_sessionrun
run_metadata)
tensorflow.python.framework.errors_impl.FailedPreconditionError: Failed to memcopy into scratch buffer for device 0
[[{{node _SOURCE}}]]
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "step2_train_nodule_detector.py", line 394, in
train(train_full_set=True, load_weights_path=None, model_name="luna16_full", fold_count=-1, manual_labels=False)
File "step2_train_nodule_detector.py", line 387, in train
model.fit_generator(train_gen, len(train_files) / 1, 12, validation_data=holdout_gen, nb_val_samples=len(holdout_files) / 1, callbacks=[checkpoint, checkpoint_fixed_name, learnrate_scheduler])
File "/usr/local/lib/python3.5/dist-packages/keras/engine/training.py", line 1516, in fit_generator
callbacks.on_epoch_begin(epoch)
File "/usr/local/lib/python3.5/dist-packages/keras/callbacks.py", line 62, in on_epoch_begin
callback.on_epoch_begin(epoch, logs)
File "/usr/local/lib/python3.5/dist-packages/keras/callbacks.py", line 534, in on_epoch_begin
K.set_value(self.model.optimizer.lr, lr)
File "/usr/local/lib/python3.5/dist-packages/keras/backend/tensorflow_backend.py", line 1853, in set_value
get_session().run(assign_op, feed_dict={assign_placeholder: value})
File "/usr/local/lib/python3.5/dist-packages/keras/backend/tensorflow_backend.py", line 125, in get_session
_initialize_variables()
File "/usr/local/lib/python3.5/dist-packages/keras/backend/tensorflow_backend.py", line 282, in _initialize_variables
sess.run(tf.variables_initializer(uninitialized_variables))
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/client/session.py", line 948, in run
run_metadata_ptr)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/client/session.py", line 1171, in _run
feed_dict_tensor, options, run_metadata)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/client/session.py", line 1348, in _do_run
run_metadata)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/client/session.py", line 1368, in _do_call
raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.FailedPreconditionError: Failed to memcopy into scratch buffer for device 0
[[{{node _SOURCE}}]]
`

Hi @nuu9323226 , the error message is not directly associated with the 3D convolution support.
Can you run the HIP unit test can let's see if we can get some hints there?
https://github.com/ROCm-Developer-Tools/HIP/tree/master/tests

@sunway513 Thanks for letting me know.
After using the new container, the log confirms that it is run with conv3d support, which fixed the original issue of not pulling correct tf version for 3d convolution. The follow-up error log is a separate issue when running conv3d.
Looking at the error code, it looks like we can do very little in the short term. The error code indicates the user's system run out of memory, because:

For the specific config, miopen is requesting huge amount of memory, to a max amount of 7.99GB. It may not be realistic for the user to be able to have a 7.99GB of memory for one operation especially considering that TF tends to run multiple nodes in parallel
miopen intermediate mode has the potential to reduce memory footprint, however, it does not support 3d convolution for now

@jerryyin , can you provide the MIOpen config that requires 7.99GB VMEM?
cc @whchung , @daniellowell

@sunway513 I have no more than the developer provided. I can get more details if @nuu9323226 let us know more details of how to reproduce.  According to his log, the following line is printed per memory request from miopen. There are plenty of other requests like this as well.

2019-06-11 09:44:13.452114: E tensorflow/stream_executor/rocm/rocm_driver.cc:630] failed to allocate 7.99G<PHONE_NUMBER> bytes) from device: hipError_t(1002)

The config needs to be one of the below according to earlier log:

conv1 (Convolution3D) (None, 16, 32, 32, 64)
conv2 (Convolution3D) (None, 16, 16, 16, 128)
conv3a (Convolution3D) (None, 8, 8, 8, 256)
conv3b (Convolution3D) (None, 8, 8, 8, 256)
conv4a (Convolution3D) (None, 4, 4, 4, 512)
conv4b (Convolution3D) (None, 4, 4, 4, 512)

Hi @nuu9323226 , could you help provide the steps to repro this issue for @jerryyin so we can further investigate it?

@sunway513 I'm not sure there's value in us continuing the investigation as there is no short term fix. I see this as more of a feature request to us. If there is any plans to fix it, it should be redirected to @daniellowell. To confirm it,
@daniellowell, Could you confirm that the comment above is accurate? To give a summary:

The conv3d config with dimension of either (16, 32, 32, 64), or (16, 16, 16, 128) costing more than 5 gigs of workspace
miopen immediate mode does not do optimizations on conv3d yet

If you can confirm the above, please let us know your opinion of whether treating this as a feature request and close it, or if there is any short term plan in reducing the workspace memory consumption from 3d conv configs reported from this issue.

@jerryyin Immediate mode does support 3-d convolutions, however it again falls back to GEMM since only GEMM supports 3-d. We do have plans to implement 3-D convolution in more kernels, but it may be one or two releases.

Thanks @jerryyin and @daniellowell for the clarifications.
Let's hold this ticket and visit it back after MIOpen support is in place.
I believe it would be valuable to have a real use case from the community in place, so we can use it to validate our future work :-)

Thanks @daniellowell. @sunway513 Hopefully we know how to reproduce it by then.
@nuu9323226 Could you provide instructions for reproducing the issue?

Ping @nuu9323226 , can you provide instructions to repro the issue?

This is possibly a duplicate to #619. The series of conv3D run out of memory issue due to large gemm sizes required.

Ping @nuu9323226 , can you provide instructions to repro the issue?

Hi Sunway, and nuu9323226 - I experienced something very similar where the memory required by miopen to instantiate BVLC GoogleNet model using hipcaffe would grow quite exponentially resulting in my Radeon VII running out of memory.
I solved it by deleting everything under ~/.cache/miopen and restarting the training. I know I had some object files in there generated by a previous mixture of versions of hip & mipoen. I know this is a script kiddy solution. But I'm an almost totally unreconstructed C programmer, and in the face of all this newfangled ML stuff I am, in fact, a script kiddy.

Hi @emerth , thanks for the comments.
Every time you upgrade ROCm builds installed on your system, it's desired to clean up the MIOpen cache under ~/.cache folder.
For details, please refer to the following doc:
https://github.com/ROCmSoftwarePlatform/MIOpen#persistent-program-cache

Hi Peng,
I just wanted to send thanks to you and the ROCm & miopen team. With release 2.9 ROCm now runs Caffe (hipcaffe) well enough that I can run deepdream-type codes. In the past I would hit bugs in code generation when running the sort of recursive code deepdream uses. I seem to recall you forwarded an issue from this to the miopen or Clang people on my behalf last year, and the fix arrived in 2.9! So, thanks!

Closing due to inactivity. Feel free to comment/reopen if you are still seeing issues.


Deployment: Always set domain filters defined in externalDNS.domainFilterList, not only on AWS.

Enables setting domain filters defined in externalDNS.domainFilterList. Before this option was used only on AWS.

Checklist

[x] Added a CHANGELOG entry

Testing
The instance of external-dns installed as part of Giant Swarm platform releases watches services in the kube-system namespace with annotations giantswarm.io/external-dns=managed and external-dns.alpha.kubernetes.io/hostname matching the clusters base domain. (You can find this in the deployments args --domain-filter value)
You can take this example Service, apply it to your cluster. Change the external-dns.alpha.kubernetes.io/hostname annotation to match your clusters base domain.
then:

Check external-dns logs for lines like Desired change: CREATE test.your.configured.domain.gigantic.io CNAME
Try to resolve the domain (https://www.dnstester.net/)

apiVersion: v1
kind: Service
metadata:
  annotations:
    external-dns.alpha.kubernetes.io/hostname: test.your.configured.domain.gigantic.io
    external-dns.alpha.kubernetes.io/ttl: "60"
    giantswarm.io/external-dns: managed
  name: test-external-dns
  namespace: kube-system
spec:
  type: ExternalName
  externalName: www.giantswarm.io

For testing upgrades:

Create the service and check for creation
Upgrade
Delete the service and check for deletion

Default app on AWS releases

[ ] Fresh install works
[ ] Upgrade works

Default app on Azure releases

[ ] Fresh install works
[ ] Upgrade works

Optional app (KVM)

[ ] Fresh install works
[ ] Upgrade works

I would appreciate including a reason for this change in the changelog and PR body.

On hold for now, will be continued after https://github.com/giantswarm/external-dns-app/pull/160.


Does it support "Wemo Mini Smart Plug"?

Does the library support "Wemo Mini Smart Plug"?

Probably not yet, but it's a simple addition. If you've got one we can add the support pretty quickly. I don't have one yet.
On Feb 23, 2017, 5:18 PM -0600, Daniyar Yeralin<EMAIL_ADDRESS>wrote:

Does the library support "Wemo Mini Smart Plug"?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub (https://github.com/iancmcc/ouimeaux/issues/154), or mute the thread (https://github.com/notifications/unsubscribe-auth/AADic7CEmMPHzCoxAqWB1cn7zb_T3IRLks5rfhPMgaJpZM4MKoNz).

Yep, I have one. You can explain the process to me, and I can try myself

@iancmcc ?

Sorry, I haven't forgotten, but my day job got extremely busy.
The basic idea is simply to watch logging during discovery (i.e., run wemo -d list) and look for this log message:
https://github.com/iancmcc/ouimeaux/blob/develop/ouimeaux/environment.py#L137
Assuming that the smart plug has its own USN, you'll get it logged, and then it's a simple matter of adding it to the checks right above that log message, probably using klass = Switch (if it has extra functionality, it could have its own class to support that, but Switch will most likely work).
On Feb 28, 2017, 7:18 PM -0600, Daniyar Yeralin<EMAIL_ADDRESS>wrote:

@iancmcc ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@iancmcc Actually, it worked w/o any changes.
I ran wemo list
and it recognized "Wemo Mini Smart Plug" as ('Switch:', 'Wemo Mini')


ERROR: pidwait: pidfd_open() not implemented in Linux < 5.3

The error message is as follows：
$ docker logs -f --tail=200 dsm

❯ Starting Virtual DSM for Docker v4.08...
❯ Install: Downloading DSM_VirtualDSM_42218.pat...
/DSM_VirtualDSM_422 100%[===================>] 362.05M  6.62MB/s    in 50s
❯ Install: Extracting downloaded image...
❯ Install: Extracting system partition...
❯ Install: Installing system partition...
pidwait: pidfd_open() not implemented in Linux < 5.3
❯ Starting Virtual DSM for Docker v4.08...
pidwait: pidfd_open() not implemented in Linux < 5.3

It is not known whether it is related to the kernel version of the host machine：
$ Linux home 5.0.2-aml-s905 #5.77 SMP PREEMPT Mon Apr 1 17:23:16 MSK 2019 aarch64 aarch64 aarch64 GNU/Linux

This is my launch command
docker run -itd --name=dsm  \
--privileged \
-p 6001:5000 \
-e CPU_CORES=1 \
-e RAM_SIZE=1024M \
-e DISK_SIZE=100G \
--device=/dev/kvm \
--cap-add NET_ADMIN \
--stop-timeout 60 \
-v /data/dsm:/storage \
kroese/virtual-dsm:latest

Oooh. The current host on which I am running the container is my soft routing device. Upgrading the kernel can be difficult

unraid 6.9.2
kernel version: 5.10.28-Unraid
still error:
dsm  | ❯ Starting Virtual DSM for Docker v4.09...
dsm  | ❯ Install: Downloading installer...
dsm  | ❯ Install: Downloading DSM_VirtualDSM_64570.pat...
dsm  |
dsm  |      0K ........ ........ ........ ........  8% 10.9M 30s
dsm  |  32768K ........ ........ ........ ........ 17% 17.1M 22s
dsm  |  65536K ........ ........ ........ ........ 26% 17.1M 18s
dsm  |  98304K ........ ........ ........ ........ 35% 17.1M 15s
dsm  | 131072K ........ ........ ........ ........ 44% 17.1M 13s
dsm  | 163840K ........ ........ ........ ........ 53% 17.1M 11s
dsm  | 196608K ........ ........ ........ ........ 62% 17.2M 8s
dsm  | 229376K ........ ........ ........ ........ 71% 17.1M 6s
dsm  | 262144K ........ ........ ........ ........ 80% 17.1M 4s
dsm  | 294912K ........ ........ ........ ........ 89% 17.1M 2s
dsm  | 327680K ........ ........ ........ ........ 98% 17.1M 0s
dsm  | 360448K .....                              100% 17.8M=22s
dsm  |
dsm  | ❯ Install: Extracting downloaded image...
dsm  | ❯ Install: Extracting system partition...
dsm  | ❯ Install: Installing system partition...
dsm  | pidwait: pidfd_open() not implemented in Linux < 5.3
dsm  | ❯ Starting Virtual DSM for Docker v4.09...
dsm  | pidwait: pidfd_open() not implemented in Linux < 5.3
dsm  | ❯ Starting Virtual DSM for Docker v4.09...
dsm  | pidwait: pidfd_open() not implemented in Linux < 5.3
dsm  | ❯ Starting Virtual DSM for Docker v4.09...
dsm  | pidwait: pidfd_open() not implemented in Linux < 5.3
dsm  | ❯ Starting Virtual DSM for Docker v4.09...
dsm  | pidwait: pidfd_open() not implemented in Linux < 5.3
dsm  | ❯ Starting Virtual DSM for Docker v4.09...
dsm  | pidwait: pidfd_open() not implemented in Linux < 5.3
dsm  | ❯ Starting Virtual DSM for Docker v4.09...
dsm  | pidwait: pidfd_open() not implemented in Linux < 5.3
dsm  | ❯ Starting Virtual DSM for Docker v4.09...
dsm  | pidwait: pidfd_open() not implemented in Linux < 5.3
dsm  | ❯ Starting Virtual DSM for Docker v4.09...
dsm  | pidwait: pidfd_open() not implemented in Linux < 5.3
dsm  | ❯ Starting Virtual DSM for Docker v4.09...
dsm  | pidwait: pidfd_open() not implemented in Linux < 5.3
dsm  | ❯ Starting Virtual DSM for Docker v4.09...
dsm  | pidwait: pidfd_open() not implemented in Linux < 5.3

@DDSDerek Really strange.. Your kernel is 5.10 so you should never get this error that you need kernel 5.3 or higher? Many people installed this container on Unraid without problems, so is it possible for you to update to a more recent release?

I will not choose to continue to upgrade Unraid to a higher version.Is there any other solution?

@DDSDerek Should be solved now in the latest release (v4.10)

Thanks, it has been resolved


build(treeshaking): export es6 modules so the can be treeshook

BREAKING CHANGE: This potentially requires consuming apps to update their webpack config to take advantage of treeshaking, and it hasn’t yet been tested with gatsby.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

Coverage remained the same at 88.49% when pulling b3a93452c1497390069c0d1f900d4d2211e02814 on djoy/babelize into d6d8a0a61aba43e858e2e9e81c35527b397fee0e on master.


add code for app managed environment

This code adds an initial implementation for the container app managed environment resource.
Current E2E tests cover default & an internal workload profile (I hope, if I merged the code into this repo right! 😂)
Regarding the use of AzAPI
I think AzAPI is a good choice for this resource.

aspects of Container Apps are moving faster than the AzureRM resource can keep up
feedback on the latest Terraform on Azure Community Call re-enforced the support commitments around AzAPI and the validity for these use cases.
AVM modules by their nature are designed to abstract and hide essential complexity, a calling module consumer doesn't need to be concerned that AzAPI is used.
There's no specific guidance from the AVM wiki I could find that suggests whether AzAPI is inappropriate (in fact, if i was in the product support group - i'd probably prefer AzAPI to be used because it looks more like Bicep 😂)

That said I note some disadvantages

the means to manage changes to API versions is not clear
the outputs may need some tidying to make them easier to consume
workload profiles has just (in the last day or so), landed in AzureRM (still not released, last i checked).

Regarding your statement "the means to manage changes to API versions is not clear", it's not mentioned somewhere you're right but also when using AzApi, this falls back to SFR1 - the latest “non-preview” version of the API SHOULD be used.
I'm in conversation with the team about the current spec about API versions to be used and we'll update the documentation as required.

@segraef i was more meaning these sorts of concerns;

how we would transition between API versions - would it simply be a breaking change requiring an updated API version, etc.
whether we'd have to support multiple (which would be very messy),

e.g. what if we did want to offer access to preview features (noting the caveats about how their descriptions need to be decorated), but a customer didn't want to be using a preview API?

Would be good to have some guidance on the appropriateness (or not) or azapi as well in general, I think.

ran the E2E tests in my fork:
https://github.com/kewalaka/terraform-azurerm-avm-res-app-managedenvironment/actions/runs/7124518461

@segraef not sure if i'm doing something wrong, but your comments are from an old review.  If I look at the "files" section on this PR, I see what I expect.
it looks like this repo is missing the 'test' environment as well, needed for the e2e tests.  I don't have access to set that up.

@segraef not sure if i'm doing something wrong, but your comments are from an old review. If I look at the "files" section on this PR, I see what I expect.
it looks like this repo is missing the 'test' environment as well, needed for the e2e tests. I don't have access to set that up.

You're not doing anything wrong, it was me looking at another commit or not having the page refreshed.
I'm currently setting up the e2e workflows and will run them then but for me/us your successful run is also fine for now.

Cool, in which case can this be merged?
On Fri, 8 Dec 2023, 17:55 Sebastian Gräf, @.***> wrote:

@segraef https://github.com/segraef not sure if i'm doing something
wrong, but your comments are from an old review. If I look at the "files"
section on this PR, I see what I expect.
it looks like this repo is missing the 'test' environment as well, needed
for the e2e tests. I don't have access to set that up.
You're not doing anything wrong, it was me looking at another commit or
not having the page refreshed.
I'm currently setting up the e2e workflows and will run them then but for
me/us your successful run is also fine for now.
—
Reply to this email directly, view it on GitHub
https://github.com/Azure/terraform-azurerm-avm-res-app-managedenvironment/pull/1#issuecomment-1846546469,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYAGXWBOI5RHIOESE6D2XLYIKMTZAVCNFSM6AAAAAA76354AKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNBWGU2DMNBWHE
.
You are receiving this because you authored the thread.Message ID:
<Azure/terraform-azurerm-avm-res-app-managedenvironment/pull/1/c1846546469
@github.com>

Moving #1 to #3 to have e2e enabled.


⚠ Samsung has degraded performance

In e3f461d, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2485 ms

Resolved: Samsung performance has improved in 1b055e4.


Getting duplicate subjects, come in random, don't break the interface

Some videos come to be classified more than once by the same user. They come in random (for example 2 days after the 1st classification) and the browser console shows no errors. We recognize them due to the comments we made after the 1st classification. It is an old problem which happens to various volunteers.
Some examples:
http://talk.chimpandsee.org/#/subjects/ACP00062ny
http://talk.chimpandsee.org/#/subjects/ACP0007bsa
http://talk.chimpandsee.org/#/subjects/ACP0007frf
http://talk.chimpandsee.org/#/subjects/ACP0006t2k
http://talk.chimpandsee.org/#/subjects/ACP0007p0n
http://talk.chimpandsee.org/#/subjects/ACP0007ey9
http://talk.chimpandsee.org/#/subjects/ACP0007zcx
http://talk.chimpandsee.org/#/subjects/ACP0007fly
http://talk.chimpandsee.org/#/subjects/ACP000asl3

I created a tag group for them http://talk.chimpandsee.org/#/collections/CCPL0000o9
It will collect all videos hashtagged with 'repeated'.

This will be the same problem as #132.


Remove async break on terminate with clrdbg

Remove async break on terminate with clrdbg as it's not needed and causes asserts on clrdbg shutdown with chk bits.

@gregg-miskelly @chuckries @rajkumar42

LGTM

:+1:

:+1:


Switch to nalgebra for statically allocated vectors

The StaticMat struct was always meant as a placeholder until a more performant alternative is available.
nalgebra provides statically allocated vectors and matrices. It is well maintained, more feature complete, and more performant.
As a consequence the derive api is not possible for vectors anymore. In this PR it is replaced with more explicit functions for, e.g., gradients, Hessians and Jacobians. Of course, it is still possible to construct dual numbers from their real and dual parts.
Some disadvantages

The dependency tree is exploding and compile times are up.
the string representation of dual numbers is a bit wonky (maybe override it by hand?).
Due to the somewhat stringent trait bounds in nalgebra, the trait bounds in num-dual also have to be tightened. This has no effect on applications of dual numbers.

Overall, this should be an advantage with regard to the maintainability, though.

With the considerations in #53 the derive methods are renamed to derivative. They are now only available for scalar dual numbers.


Reminder improvements

[ ] You should be able to remind others rather than just yourself (.remind @person "bleh" in blah)
[ ] You should be able to list reminders

Closed in favour of #97


Class-based coercers

Pull Request Test Coverage Report for Build 944

109 of 160 (68.13%) changed or added relevant lines in 24 files are covered.
30 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.5%) to 81.763%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/SchemaBuilder/DefinitionBuilder.php
1
2
50.0%

src/SchemaValidator/Rule/TypesRule.php
0
1
0.0%

src/Type/Coercer/BooleanCoercer.php
2
3
66.67%

src/Type/Coercer/IntCoercer.php
11
12
91.67%

src/Type/Definition/ExtensionASTNodesTrait.php
0
1
0.0%

src/Type/Definition/EnumType.php
6
7
85.71%

src/Type/Coercer/StringCoercer.php
7
9
77.78%

src/Type/Definition/ResolveTrait.php
3
6
50.0%

src/Type/CoercerProvider.php
0
5
0.0%

src/Type/Definition/ScalarType.php
12
17
70.59%

Files with Coverage Reduction
New Missed Lines
%

src/Util/helpers.php
1
88.89%

src/Type/Definition/EnumType.php
2
76.0%

src/Type/ScalarTypesProvider.php
3
66.15%

src/Util/coercers.php
24
0.0%

Totals

Change from base Build 934:
-0.5%

Covered Lines:
4425

Relevant Lines:
5412

💛  - Coveralls

Pull Request Test Coverage Report for Build 944

109 of 160 (68.13%) changed or added relevant lines in 24 files are covered.
30 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.5%) to 81.763%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/SchemaBuilder/DefinitionBuilder.php
1
2
50.0%

src/SchemaValidator/Rule/TypesRule.php
0
1
0.0%

src/Type/Coercer/BooleanCoercer.php
2
3
66.67%

src/Type/Coercer/IntCoercer.php
11
12
91.67%

src/Type/Definition/ExtensionASTNodesTrait.php
0
1
0.0%

src/Type/Definition/EnumType.php
6
7
85.71%

src/Type/Coercer/StringCoercer.php
7
9
77.78%

src/Type/Definition/ResolveTrait.php
3
6
50.0%

src/Type/CoercerProvider.php
0
5
0.0%

src/Type/Definition/ScalarType.php
12
17
70.59%

Files with Coverage Reduction
New Missed Lines
%

src/Util/helpers.php
1
88.89%

src/Type/Definition/EnumType.php
2
76.0%

src/Type/ScalarTypesProvider.php
3
66.15%

src/Util/coercers.php
24
0.0%

Totals

Change from base Build 934:
-0.5%

Covered Lines:
4425

Relevant Lines:
5412

💛  - Coveralls


Failed when jumpDefinition with error msg "invalid AST"

`
versions
vim version: NVIM v0.8.0-v0.8.0
node version: v16.18.1
coc.nvim version: 0.0.82-11257dc 2022-10-08 18:52:57 -0700
coc.nvim directory: /srv/data/J/cao_y1/.config/nvim/plugged/coc.nvim
term: xterm-256color
platform: linux
Log of coc.nvim
2022-11-30T09:03:59.085 INFO (pid:66361) [services] - registered service "languageserver.bash"
2022-11-30T09:03:59.087 INFO (pid:66361) [services] - registered service "languageserver.golang"
2022-11-30T09:03:59.158 INFO (pid:66361) [services] - registered service "yaml"
2022-11-30T09:03:59.159 INFO (pid:66361) [services] - LanguageClient YAML Support state change: stopped => starting
2022-11-30T09:03:59.378 INFO (pid:66361) [plugin] - coc.nvim initialized with node: v16.18.1 after 480ms
2022-11-30T09:03:59.446 INFO (pid:66361) [services] - registered service "clangd"
2022-11-30T09:03:59.446 INFO (pid:66361) [services] - LanguageClient clangd state change: stopped => starting
2022-11-30T09:03:59.471 INFO (pid:66361) [language-client-index] - Language server "clangd" started with 66384
2022-11-30T09:03:59.626 INFO (pid:66361) [services] - LanguageClient YAML Support state change: starting => running
2022-11-30T09:03:59.640 INFO (pid:66361) [services] - service yaml started
2022-11-30T09:03:59.643 INFO (pid:66361) [services] - LanguageClient clangd state change: starting => running
2022-11-30T09:03:59.648 INFO (pid:66361) [services] - service clangd started
2022-11-30T09:03:59.674 INFO (pid:66361) [attach] - receive notification: runCommand [
'coc-helper.internal.didVimEvent_m2_v0_15_2_explorer_3',
'CocDiagnosticChange'
]
2022-11-30T09:03:59.675 ERROR (pid:66361) [node-client] - Error on notification "runCommand": Command: coc-helper.internal.didVimEvent_m2_v0_15_2_explorer_3 not found Error
at NeovimClient2.echoError (/srv/data/J/cao_y1/.config/nvim/plugged/coc.nvim/build/index.js:9833:41)
at NeovimClient2. (/srv/data/J/cao_y1/.config/nvim/plugged/coc.nvim/build/index.js:78479:16)
at processTicksAndRejections (node:internal/process/task_queues:96:5)
2022-11-30T09:03:59.675 ERROR (pid:66361) [attach] - Error: Command: coc-helper.internal.didVimEvent_m2_v0_15_2_explorer_3 not found
at CommandManager.executeCommand (/srv/data/J/cao_y1/.config/nvim/plugged/coc.nvim/build/index.js:70274:17)
at CommandManager.fireCommand (/srv/data/J/cao_y1/.config/nvim/plugged/coc.nvim/build/index.js:70280:30)
at processTicksAndRejections (node:internal/process/task_queues:96:5)
at async Commands.runCommand (/srv/data/J/cao_y1/.config/nvim/plugged/coc.nvim/build/index.js:73174:14)
at async Plugin.cocAction (/srv/data/J/cao_y1/.config/nvim/plugged/coc.nvim/build/index.js:78377:15)
at async NeovimClient2. (/srv/data/J/cao_y1/.config/nvim/plugged/coc.nvim/build/index.js:78477:11)
2022-11-30T09:08:40.528 INFO (pid:66361) [attach] - receive notification: jumpDefinition []
2022-11-30T09:08:40.545 ERROR (pid:66361) [provider-manager] - Provider error on provideDefinition: ResponseError2: invalid AST
at handleResponse (/srv/data/J/cao_y1/.config/nvim/plugged/coc.nvim/build/index.js:12216:40)
at processMessageQueue (/srv/data/J/cao_y1/.config/nvim/plugged/coc.nvim/build/index.js:12039:13)
at Immediate. (/srv/data/J/cao_y1/.config/nvim/plugged/coc.nvim/build/index.js:12025:11)
at processImmediate (node:internal/timers:466:21) {
code: -32001,
data: undefined
}
2022-11-30T09:08:45.011 INFO (pid:66361) [attach] - receive notification: showInfo []
`
I successfully installed coc.clangd and config it with right compile_compile.json in $XDG_CONFIG_HOME/clangd/config.yaml
Also config the following clangd related content
1   "clangd.path": "~/.config/coc/extensions/coc-clangd-data/install/15.0.3/clangd_15.0.3/bin/clangd",
2   "clangd.enabled": true
It seems coc-helper.internal.didVimEvent_m2_v0_15_2_explorer_3 was not found.
I want to know
1.If it will cause jumpDefinition failed?
2.What else should i config beyond CocConfig and clangd.yaml?
It will be very pleased if someone have experienced the similar error and willing to share.

coc-helper.internal.didVimEvent_m2_v0_15_2_explorer_3 was not found.

Do you use con-explorer? It's coc-helper's issue, not coc-clangd.

coc-helper.internal.didVimEvent_m2_v0_15_2_explorer_3 was not found.

Do you use con-explorer? It's coc-helper's issue, not coc-clangd.
@fannheyward
Thanks for your quick reply!
Yes, I have used coc-explorer.
Ok, What about Q2?
What else should i do to successfully jumpDefinition with no error (invalid AST)

My CocConfig and config.yaml under clangd are as following:
CocConfig:
{
"suggest.noselect": true,
"suggest.enablePreselect": false,
"codeLens.enable": true,
"languageserver": {
"golang": {
"command": "gopls",
"rootPatterns": ["go.work", "go.mod", ".vim/", ".git/", ".hg/"],
"filetypes": ["go"],
"initializationOptions": {
"usePlaceholders": true
}
}
},
"explorer.icon.enableNerdfont": true,
"explorer.file.showHiddenFiles": true,
"explorer.enableFloatinput": true,
"clangd.path": "~/.config/coc/extensions/coc-clangd-data/install/15.0.3/clangd_15.0.3/bin/clangd",
"clangd.enabled": true
}
config.yaml:
If:
PathMatch: "./tsd-blueware-bluego/."
CompileFlags:
CompilationDatabase: "/srv/data/J/cao_y1/recipes-svw-sop2/dev/build/phone/bt-middleware/tsd-blueware-bluego-dev/1/workspace"
If:
PathMatch: "./tsd-blueware-api/."
CompileFlags:
CompilationDatabase: "/srv/data/J/cao_y1/recipes-svw-sop2/dev/build/phone/bt-middleware/tsd-blueware-api-dev/1/workspace"
If:
PathMatch: "./tsd-blueware-mib3-bluego-api/."
CompileFlags:
CompilationDatabase: "/srv/data/J/cao_y1/recipes-svw-sop2/dev/build/phone/bt-middleware/tsd-blueware-mib3-bluego-api-dev/1/workspace"
If:
PathMatch: "./ext-bluego/."
CompileFlags:
CompilationDatabase: "/srv/data/J/cao_y1/recipes-svw-sop2/dev/build/phone/bt-middleware/ext-bluego-dev/1/workspace"

invalid AST

This error is came from clangd, clangd failed to jump because AST error.
coc-clangd is just a client, the server didn't returns definition locations, coc-clangd can't jump.

invalid AST

This error is came from clangd, clangd failed to jump because AST error.
coc-clangd is just a client, the server didn't returns definition locations, coc-clangd can't jump.

Thank you VM, I will check where goes wrong in clangd.
BTW How can enable verbose level log with coc.clangd,
Maybe i can enable by command, Just curious.

invalid AST

This error is came from clangd, clangd failed to jump because AST error.
coc-clangd is just a client, the server didn't returns definition locations, coc-clangd can't jump.

Thank you VM, I will check where goes wrong in clangd. BTW How can enable verbose level log with coc.clangd, Maybe i can enable by command, Just curious.

ITS OK, I found that, Over the below argument.

clangd.arguments
arguments for clangd server
[]

invalid AST

This error is came from clangd, clangd failed to jump because AST error.
coc-clangd is just a client, the server didn't returns definition locations, coc-clangd can't jump.

Thank you VM, I will check where goes wrong in clangd. BTW How can enable verbose level log with coc.clangd, Maybe i can enable by command, Just curious.

ITS OK, I found that, Over the below argument.
clangd.arguments	arguments for clangd server	[]

Finally, Problem Solved.
This answer did greate job.
https://github.com/clangd/clangd/issues/734
The root cause is one of my cross compiler parameter seems doesn't recognized by cland server.

I encountered a similar problem when cross-compiling the Linux 5.4 kernel code. Replacing all cc parameters in the compile_commands.json file with arm-none-linux-gnueabihf-gcc solved the problem.


PANIC whilst trying to do migrate task

Description
Hello,
I'm running a fresh Gitea instance (latest 1.22.3 as of today with no repository at all yet as it's a new instance).
I wanted to mirror a Github repository (private).
As you can see below on the screenshot, I'm getting the following error:
PANIC whilst trying to do migrate task: runtime error: index out of range [2] with length 2
I have no clue about how to resolve this.
FWIW, The PAT token I'm using from Github is a classic one with everything checked (just to be sure it's not a permission issue).
According to older similar issues, this kind of error was supposed to be fixed from earlier versions already.
I'm ready to provide more information but right now I'm not sure what can be useful to share.
Thanks.
Gitea Version
1.22.3
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots

Git Version
No response
Operating System
No response
How are you running Gitea?
I'm running Gitea using docker-compose.
Database
PostgreSQL

Okay my bad! I made a typo on the repository address 🤦♂

Can you paste the stack trace here from the log file or console?


Dummy commit just to ask @thednp a message

Hi @thednp ! I'm interested in submitting a few PRs against v4 of your BSN library (https://cdn.jsdelivr.net/npm/bootstrap.native@2.0.27/dist/bootstrap-native-v4.js), but I'm confused about how to properly target my branches.  In your master branch, I only see code for v3, but not v4.  Could you please advise on the best way to submit PRs that affect only v4 of your library?  Many thanks!

The current master is V4 only and is the current active branch.
CDNs are not YET updated with the current master.

@thednp thanks for your quick reply!  I'm still unsure about how to proceed, though.  You wrote that "the current master is V4", but it seems like the dist directory (https://github.com/thednp/bootstrap.native/tree/master/dist) only contains V3 files.
I can see the V4 files in that directory only after choosing one of the recent tags:  https://github.com/thednp/bootstrap.native/tree/2.0.27/dist.  But as far as the master branch is concerned, it only has V3's files.

Did you check the content of the files?

Did you check the content of the files?

Yeah, and in fact we can look only at the very top of each file to see that something is out of whack.
If master is truly V4 only, then I would expect master's version of /dist/bootstrap-native.js to contain this as its first line: // Native Javascript for Bootstrap 4 v2.0.27 | © dnp_theme | MIT-License
But instead, its first line is: Native JavaScript for Bootstrap v3.0.0 (https://thednp.github.io/bootstrap.native/)

Lemme drop this for you
https://github.com/thednp/bootstrap.native/issues/306

Ah, that helps, thank you.  I was conflating your new V3 (based on Twitter Bootstrap's V4) with Twitter's old V3.  Sorry for the hassle!