Update Senegal contraceptive switching matrices

Correct matrix multiplication or Senegal contraceptive switching matrices.

Matrices have been updated. Code is here: https://github.com/InstituteforDiseaseModeling/FP-DHS/blob/master/Switching_matrices/Switching calendar analysis DHS.R
Matrices are attached and here: https://github.com/InstituteforDiseaseModeling/FP-DHS/blob/master/Switching_matrices/Results/matrices_Senegal_DHS_2022-11-30.csv
matrices_Senegal_DHS_2022-11-30.csv

Updated formatting to match existing code (initiation is within the same matrix) and reordered to match inputs. New version attached.
matrices_Senegal_DHS_2022-12-05.csv

Realized I forgot to apply pop weights. Added in this version @MObrien-IDM .
matrices_Senegal_DHS_2022-12-09.csv


GH Actions: use Azure Artifacts binary cache

vcpkg won't upload packages from the GitHub Actions cache to Azure Artifacts, so the packages need to be rebuilt. They are being uploaded now: https://dev.azure.com/mixxx/vcpkg/_packaging?_a=feed&feed=dependencies
So PRs from forks will still save to the GitHub Actions cache, but they will have to be rebuilt after merging before they get uploaded to Azure Pipelines.

vcpkg won't upload packages from the GitHub Actions cache to Azure Artifacts, so the packages need to be rebuilt. They are being uploaded now: https://dev.azure.com/mixxx/vcpkg/_packaging?_a=feed&feed=dependencies
So PRs from forks will still save to the GitHub Actions cache, but they will have to be rebuilt after merging before they get uploaded to Azure Pipelines.


V2 docs improvements

Work In Progress - Do Not Merge
Changes to make the docs look and work a bit better...  Let me know what you think / like.

@SimonClark this is good, but I can see where this is heading.
My first reaction is that anyone should be able to add to documentation, especially non-API documentation. I’m a HUGE fan of WIKI, and think that users can contribute changes or content very easily. I’m not so sure about MD files.
Having said that, I’m open to experimenting.

My first reaction is that anyone should be able to add to documentation, especially non-API documentation. I’m a HUGE fan of WIKI, and think that users can contribute changes or content very easily. I’m not so sure about MD files.

Yeah, I think wiki based documentation is good goal.  Eventually we need something more robust than jsdoc, or that works with jsdoc to add comments and wiki functionality. I've got some ideas.
This PR is a vanity piece for me.   Adding the tutorials and homepage content seemed like a good thing too.

Now, that said, there is a "Disqus" option in the docdash theme.  It would be somewhat trivial to add on-page discussions.  But... if/when we moved to something more customized, we might lose all existing comments.

I think more valuable than user-editable docs, in the short term, would be a user-contributable script repository.  Something like Thingiverse, but for jscad scripts.


[Question] is AQ trained from human games?

Hi!
is AQ trained from human games?
Or is AQ trained from bot generated games? or self-play games? or a mixed set of games?

I'd like to now that too

This issue mentions an older version, or hasn't been discussed in a long time, so we close it.


:sparkles: (llm|lld): delete local app data on uninstall and uninstallAll

✅ Checklist

[x] npx changeset was attached.
[x] Covered by automatic tests.
[x] Impact of the changes:

delete app data when restore is finished (wip: handle UserRefusedOnDevice case)
updated the apprunner state to know when to delete app data backup during uninstall
refactor execWithTransport in its own file
update lld to use new flow
update llm to use new flow

📝 Description
This PR makes sure to remove local app data when uninstalling app through the manager on LLD/LLM + all changes required in LLC to make it work

❓ Context

JIRA or GitHub link: LIVE-13454 [LIVE-12688]

🧐 Checklist for the PR Reviewers

The code aligns with the requirements described in the linked JIRA or GitHub issue.
The PR description clearly documents the changes made and explains any technical trade-offs or design decisions.
There are no undocumented trade-offs, technical debt, or maintainability issues.
The PR has been tested thoroughly, and any potential edge cases have been considered and handled.
Any new dependencies have been justified and documented.
Performance considerations have been taken into account. (changes have been profiled or benchmarked if necessary)

Some things to fix in async unit tests. Also I think the name of the deleteAppDataBackup is maybe a bit too vague. And I'm not sure I understand where the actual deletion of locally stored app data is done for app uninstallation, deleteAppDataUseCaseDI is never called in uninstall flows as far as I understand 🤔

Things are done here https://github.com/LedgerHQ/ledger-live/pull/7744/files#diff-cadd4d39653d7214faf8f5dd226d1a6e9c4fc35c45e66a08787170bcc9097d71 to know how to handle each case. (the state)
And there is no "deletion" when uninstalling https://github.com/LedgerHQ/ledger-live/pull/7744/files#diff-64aaf9798ce046311b2798bcaa0ebd94cf9c3308faf867ac000a861a6c150a87R19 we just skip the backup.
But fair point I will rename deleteAppDataBackup to make more sense


Large flutter test suite can fail with "Failed to load: Too many open files"

Running large test suites can cause "Failed to load: Too many open files".
ulimit is the default macOS 10.14.5 (256).
Steps to Reproduce
Running all the tests in packages/flutter at the same time demonstrates this problem.
$ cd packages/flutter
$ flutter test

Logs
flutter test --verbose results (with scrubbed directory names):
test_verbose.txt
00:57 +1113 ~25: flutter/packages/flutter/test/cupertino/scaffold_test.dart: Contents have automatic sliver padding between translucent bars                                    [  +18 ms] test 175: error caught during test; sending to test framework
[   +3 ms] test 175: cleaning up...
[        ] test 175: deleting temporary directory
[   +3 ms] test 175: shutting down test harness socket server
00:57 +1113 ~25 -1: loading flutter/packages/flutter/test/cupertino/action_sheet_test.dart [E]
  Failed to load "flutter/packages/flutter/test/cupertino/action_sheet_test.dart":
  Too many open files
    Command: flutter/bin/cache/artifacts/engine/darwin-x64/flutter_tester --disable-observatory --enable-checked-mode --verify-entry-points --enable-software-rendering --skia-deterministic-rendering --enable-dart-profiling --non-interactive --use-test-fonts --packages=flutter/packages/flutter/.packages /var/folders/mv/qlhc4vsj6tqct166rw3zwmth00mfq2/T/flutter_test_listener.Q2tyNY/listener.dart.dill

$ ulimit -n
256

$ flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel unknown, v1.7.5-pre.17, on Mac OS X 10.14.5 18F203, locale en-US)

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 10.2.1)
[✓] iOS tools - develop for iOS devices
[✓] Android Studio (version 3.4)
[✓] VS Code (version 1.35.0)
[✓] Connected device (1 available)

• No issues found!

Just running packages/flutter/tests/widget/* was enough to hit the open file limit.
Upping my ulimit -n limit to 1024 was enough to stop hitting the open file limit.

Duplicate of https://github.com/flutter/flutter/issues/2976.


Siemens IoT2000/2040 - "Pull access denied for balenalib/iot2000-alpine-node, repository does not exist or may require 'docker login'"

I've started a forum post here discussing. I've started to look into fixes, but if anyone has any suggestions I'm all ears open!
https://forums.balena.io/t/node-red-on-siemens-iot-2040/25564

@joearkay I just released 2.x that is based on debian base images, so your IOT20XX will be happy :)

Awesome - thanks :)


CHIP Persistent data storage writes are not power-loss safe

In this discord thread: https://discord.com/channels/330944238910963714/1311124568081174628 a user reported an issue where following a power loss, all of their matter devices disappeared. From the Matter server logs:
2024-11-26 20:12:41.236 (MainThread) ERROR [chip.storage] Expecting ',' delimiter: line 29 column 111 (char 4111)
2024-11-26 20:12:41.236 (MainThread) CRITICAL [chip.storage] Could not load configuration from /data/chip.json - resetting configuration...

It looks like the configuration file was present, but had invalid data in it. The user was able to restore a backup correct the issue, but when I looked at the code to persist the storage: (edit: this code doesn't seem to be responsible for the user's issue, which happened in the Matter SDK code - but this code is incorrect.)
https://github.com/home-assistant-libs/python-matter-server/blob/a55bb8b0dc838cfee2eb0dd70f18655e26540efb/matter_server/server/storage.py#L149-L163
I noticed that the logic here is not power loss safe, and might result in a partially written or missing file for unlucky users.
The current logic does:

Rename chip.json to chip.json.backup. Following this, if power is lost, there may be no chip.json file present at all.
Write a new chip.json file. If the write is interrupted, or power is lost before the system flushes the write cache, a partially written or corrupt configuration file may be present on reboot, resulting in errors like this user saw.

I'd recommend the following logic instead:

(optional) Copy chip.json to chip.json.backup, leaving the existing file in place. (Ideally, run fsync on the newly written file to ensure the backup is updated before the original file gets replaced.)
Write the new data to a temporary file, like chip.json.new.
Run fsync (or fdatasync) on the newly written file to ensure the file contents are persisted on the disk. (see the notes on https://docs.python.org/3/library/os.html#os.fsync regarding flushing the python buffers). This is needed to ensure ordering - making sure that the file contents will be written before the file gets renamed.
Rename the temporary file to the final filename, atomically replacing the existing file.
(optional) Run fsync on the renamed file to ensure the rename is committed to storage. (This is only needed if data stored elsewhere - e.g. in a sqlite database - might depend on this data having been persisted)

… Actually I think I'm looking at the wrong code here - the error messages don't match up. Is the code in matter-server/server/storage.py used? is it for something else?

Looking closer, the actual code responsible for this might be in the Matter SDK itself, here:
https://github.com/project-chip/connectedhomeip/blob/e98f2b239aae657a10043bad9aa61f4717d059be/src/controller/python/chip/storage/__init__.py#L164-L183
Unfortunately, this code has the same issue, where a power loss during write (prior to the fsync) can result in an incompletely written file which will be corrupt on restart.

The only way to get that resolved is create a PR on the matter SDK repo. Nobody is going to pick up that issue report as the python bindings are never meant to be used the way we do. They're only created for the testing framework. So if you wnat this fixed, you'll need to create a PR there or wait for one of us to have some spare time ;-)
Even though this could be improved I still think you just need to make backups of your server.

Yes, backups are always needed, but if it can be easily fixed/improved, why not?

Files getting corrupted on hard power off is entirely preventable in software (assuming you're not running on broken hardware). We should be expecting Home Assistant will be able to automatically recover and keep working when booted up after a hard power off. The current situation, where it can hit a corrupt data file and drop all your Matter devices with nothing but a log message is unacceptable IMO.
Fixing this will prevent an entire class of "oh no, there was a power outage last night and none of my automations on matter devices worked this morning - what are you saying i need to restore a backup!?" issues.
Unfortunately while I have the ability to write and contribute the fix to the Matter SDK side, I'm not willing to sign their CLA at the moment.

I'm not saying we should not fix this (we're looking at it) but that besides making the software more forgiving you should always have backups as well. Sorry if that didn't come across. English is not my main language so my Dutch-style English is not always understood haha.
We are a super limited team (basically its just me and Stefan helps me out now and then) and low on contributions so we don't have lots of time to spend coding, its a constant balance of what to pickup and our time has very much been consumed by all these issues with Thread and bugs with TREL.
I'll do my best to create at least a temporary patch for this (as we now have seen a couple of reports already about this) before/on monday. Thanks!

I think I just hit this issue myself. Does the Docker container contain this fix?
I'm currently running (f07a9b2ff8bafeaf1be20ded7f4c2ff3452fe7b87d9c22aa72beaf980978a07b):
ghcr.io/home-assistant-libs/python-matter-server     stable             f07a9b2ff8ba   2 days ago      448MB

docker image inspect f07a9b2ff8bafeaf1be20ded7f4c2ff3452fe7b87d9c22aa72beaf980978a07b
        "RepoDigests": [
            "ghcr.io/home-assistant-libs/python-matter-server@sha256:1e371d6936c179fec67896180a8697448c2ccec628078f709bb4b271b040701b"
        ],

It seems up-to-date:
https://github.com/home-assistant-libs/python-matter-server/actions/runs/12470272943/job/34805323062
I'm not a fan of how difficult Github makes it to find the built images with their hash, etc. Should I be looking somewhere else?
All my Matter devices stopped responding for some reason. I have it working again now though. I still have the backed up data folder if it's useful. I'll open another bug if it happens again.

I think I just hit this issue myself. Does the Docker container contain this fix?

Unfortunately, the fix that has been committed isn't complete - it improves the situation, but there's an important data file written by the Matter SDK itself which also has this issue: https://github.com/project-chip/connectedhomeip/issues/36654 (in logs I've seen for other users hitting this problem, the Matter SDK file is the one that actually got corrupted).

I think there's still a bug in the server. It's not making the JSON file, I had to touch an empty file:
Traceback (most recent call last):
  File "/usr/local/bin/matter-server", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/usr/local/lib/python3.12/site-packages/matter_server/server/__main__.py", line 239, in main
    run(server.start(), shutdown_callback=handle_stop)
  File "/usr/local/lib/python3.12/site-packages/aiorun.py", line 370, in run
    raise pending_exception_to_raise
  File "/usr/local/lib/python3.12/site-packages/aiorun.py", line 270, in run
    loop.run_until_complete(shutdown_callback(loop))
  File "/usr/local/lib/python3.12/asyncio/base_events.py", line 686, in run_until_complete
    return future.result()
           ^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/matter_server/server/__main__.py", line 236, in handle_stop
    await server.stop()
  File "/usr/local/lib/python3.12/site-packages/matter_server/server/server.py", line 225, in stop
    await self.storage.stop()
  File "/usr/local/lib/python3.12/site-packages/matter_server/server/storage.py", line 55, in stop
    await self.async_save()
  File "/usr/local/lib/python3.12/site-packages/matter_server/server/storage.py", line 169, in async_save
    await self.server.loop.run_in_executor(None, do_save)
  File "/usr/local/lib/python3.12/concurrent/futures/thread.py", line 59, in run
    result = self.fn(*self.args, **self.kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/matter_server/server/storage.py", line 159, in do_save
    shutil.copy(self.filename, self.filename_backup)
  File "/usr/local/lib/python3.12/shutil.py", line 435, in copy
    copyfile(src, dst, follow_symlinks=follow_symlinks)
  File "/usr/local/lib/python3.12/shutil.py", line 260, in copyfile
    with open(src, 'rb') as fsrc:
         ^^^^^^^^^^^^^^^
FileNotFoundError: [Errno 2] No such file or directory: '/data/15481557425402580959.json'
# touch matter-data/15481557425402580959.json
# Now it persists properly when I `docker stop/start`

@aidanharris please do not pollute this issue report with an unrelated report. The issue you mention has been fixed this morning and a new release will be issued soon.

@aidanharris please do not pollute this issue report with an unrelated report. The issue you mention has been fixed this morning and a new release will be issued soon.

Sorry, about that. I did mark it as resolved (I should have checked first) but that's too late for people's email notifications.


MassInstallAction: Install the CompatHelper workflow on this repository

This pull request sets up the CompatHelper workflow on this repository. cc: @aminya

Codecov Report

Merging #8 into master will decrease coverage by 0.57%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master       #8      +/-   ##
==========================================
- Coverage   90.88%   90.31%   -0.58%
==========================================
  Files          12       12
  Lines         450      444       -6
==========================================
- Hits          409      401       -8
- Misses         41       43       +2

Impacted Files
Coverage Δ

src/helpers/juliaconfig.jl
77.27% <0.00%> (-11.62%)
:arrow_down:

src/helpers/standalone-exe.jl
94.59% <0.00%> (-0.15%)
:arrow_down:

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 9deca58...c60f64a. Read the comment docs.


Small improvements

Just some random things I saw in code.

Coverage increased (+0.3%) to 53.738% when pulling 45016ae3d4c7585061dd6a3fb759e08a85bc8889 on Menda:small-improvements into 67c465390c35eb28e6f4f5aaf715c496d21d5bc4 on toladata:dev-v2.


feat(dynatron): types improved

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Super Tab Not Working

FAQ

[X] I have checked the FAQ and it didn't resolve my problem.

Announcement

[X] I have checked Breaking change announcement.

Minimal reproducible full config
local cmp_status_ok, cmp = pcall(require, "cmp")
if not cmp_status_ok then
  return
end

-- WIP Protected call for vsnip
--local snip_status_ok, vsnip = pcall(require, "vim-vsnip")
--if not snip_status_ok then
--  return
--end

local has_words_before = function()
  local line, col = unpack(vim.api.nvim_win_get_cursor(0))
  return col ~= 0 and vim.api.nvim_buf_get_lines(0, line - 1, line, true)[1]:sub(col, col):match("%s") == nil
end

local feedkey = function(key, mode)
  vim.api.nvim_feedkeys(vim.api.nvim_replace_termcodes(key, true, true, true), mode, true)
end

-- Setup nvim-cmp.
local cmp = require('cmp')

local lsp_symbols = {
  Text = "   (Text) ",
  Method = "   (Method)",
  Function = "   (Function)",
  Constructor = "   (Constructor)",
  Field = " ﴲ  (Field)",
  Variable = "[] (Variable)",
  Class = "   (Class)",
  Interface = " ﰮ  (Interface)",
  Module = "   (Module)",
  Property = " 襁 (Property)",
  Unit = "   (Unit)",
  Value = "   (Value)",
  Enum = " 練 (Enum)",
  Keyword = "   (Keyword)",
  Snippet = "   (Snippet)",
  Color = "   (Color)",
  File = "   (File)",
  Reference = "   (Reference)",
  Folder = "   (Folder)",
  EnumMember = "   (EnumMember)",
  Constant = " ﲀ  (Constant)",
  Struct = " ﳤ  (Struct)",
  Event = "   (Event)",
  Operator = "   (Operator)",
  TypeParameter = "   (TypeParameter)"
}

cmp.setup {
  snippet = {
    expand = function(args)
      vim.fn["vsnip#anonymous"](args.body) -- For `vsnip` users.
    end,
  },
  mapping = cmp.mapping.preset.insert({
    ['<C-d>'] = cmp.mapping.scroll_docs(-4),
    ['<C-f>'] = cmp.mapping.scroll_docs(4),
    ['<C-x><C-s>'] = cmp.mapping.complete({
        config = {
        sources = {
          { name = 'vsnip' }
        }
      }
    }),
    ['<C-e>'] = cmp.mapping.close(),
    ['<C-y>'] = cmp.config.disable, -- If you want to remove the default `<C-y>` mapping, You can specify `cmp.config.disable` value.
    ['<CR>'] = cmp.mapping.confirm({ select = true }),

    ["<Tab>"] = cmp.mapping(function(fallback)
      if cmp.visible() then
        cmp.select_next_item()
      elseif vim.fn["vsnip#available"](1) == 1 then
        feedkey("<Plug>(vsnip-expand-or-jump)", "")
      elseif has_words_before() then
        cmp.complete()
      else
        fallback() -- The fallback function sends a already mapped key. In this case, it's probably `<Tab>`.
      end
    end, { "i", "s" }),

    ["<S-Tab>"] = cmp.mapping(function()
      if cmp.visible() then
        cmp.select_prev_item()
      elseif vim.fn["vsnip#jumpable"](-1) == 1 then
        feedkey("<Plug>(vsnip-jump-prev)", "")
      end
    end, { "i", "s" }),

  sources = cmp.config.sources ({
    { name = 'nvim_lsp' },
    { name = 'vsnip' }, -- For vsnip users.
    { name = 'path' },
    { name = 'treesitter' },
    { name = 'zsh' },
  },
  {
    name = 'buffer',
    option = {
        get_bufnrs = function() return { vim.api.nvim_get_current_buf() } end
      },
   }),
})
}

Description
When you go to insert mode you get a bunch of errors about an expected Lua string. nvim-cmp is not working at all once you are in insert.

Steps to reproduce
When you use the provided config you should get the error.
Expected behavior
nvim-cmp and super tab should work with vsnip and you should be prompted with completion.
Actual behavior
Errors pop up and you get no cmp.
Additional context
No response

I was able to resolve this. The issue was on me. You guys keep up the good work! Love the plugin.

I was able to resolve this. The issue was on me. You guys keep up the good work! Love the plugin.

What was your solution? Think of other people who might find this issue in the future.

Good point. I had found that I had the S-Tab out of the mapping section of my config. Here is what my new mapping section looks like.
mapping = cmp.mapping.preset.insert ({
   ['<C-b>'] = cmp.mapping.scroll_docs(-4),
   ['<C-f>'] = cmp.mapping.scroll_docs(4),
   ['<C-Space>'] = cmp.mapping.complete(),
   ['<C-e>'] = cmp.mapping.abort(),
   ['<CR>'] = cmp.mapping.confirm({ select = true }), -- Accept currently selected item. Set `select` to `false` to only confirm explicitly selected items.

   ["<Tab>"] = cmp.mapping(function(fallback)
     if cmp.visible() then
       cmp.select_next_item()
     elseif vim.fn["vsnip#available"](1) == 1 then
       feedkey("<Plug>(vsnip-expand-or-jump)", "")
     elseif has_words_before() then
       cmp.complete()
     else
       fallback() -- The fallback function sends a already mapped key. In this case, it's probably `<Tab>`.
     end
   end, { "i", "s" }),

   ['<S-Tab>'] = cmp.mapping(function()
     if cmp.visible() then
       cmp.select_prev_item()
     elseif vim.fn["vsnip#jumpable"](-1) == 1 then
       feedkey("<Plug>(vsnip-jump-prev)", "")
     end
   end, { "i", "s"}),
 }),


AniList Genre/Tags Not Working

Running script using anilist_genre or anilist_tag results in error "attribute not supported".

can you share the config for that collection

TV Shows - Anime.txt

which version of PMM are you using? if its current can you try the develop branch and see if your error still occurs

Just pulled latest develop. Same error.

can you send me you meta.log file

meta.log

the log say version 1.5.1 can you run this again with the develop branch and send me the log from that?

How do I switch branches? Tried git checkout develop and git pull but says up to date.

try git fetch

I figured it out, my docker command wasn't set to pull from develop. It's working now.


Random IO errors on write to terminal in Atom/VS Code, suspect to be caused by winpty

I, and others, have been getting weird IO errors/exceptions from Python using embedded terminal implementations in Atom and VS Code. Specifically platformio-atom-ide-terminal and the builtin embedded terminal of VS Code.
The errors are random, running something like this from a Python interpreter should reproduce the error:
for num in xrange(0, 1000):
    print num

The error/exception that you get randomly is (Randomly intermingled with output, of course):
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
IOError: [Errno 0] Error

Essentially it's a WriteFile/write/fwrite call that fails with an errno 0, which should never happen. Which makes it all the more likely that it's winpty, considering that winpty uses hooks to get its job done. (Thank you again, M$ Windows, for having no sane pty-like API).
Issues
https://github.com/platformio/platformio-atom-ide-terminal/issues/422
https://github.com/Microsoft/vscode/issues/40199
https://github.com/platformio/platformio-vscode-ide/issues/61
cc @rprichard @ivankravets @Tyriar

Thanks for filing this.

Which makes it all the more likely that it's winpty, considering that winpty uses hooks to get its job done.

FWIW, winpty doesn't really use "hooks". The interesting things that winpty does are:

It has a process that's always running in the background of a console, winpty-agent.exe.
It screen-scrapes the 2D console buffer and looks for changes.
When it's looking for changes, it "freezes" application writes to the console by selecting some text. While it's frozen, it sometimes writes a "marker" into the hidden console history so it can track how many times the console buffer has been scrolled. (If it didn't do this, it could still show you the 2D content of the visible console window, but you'd have no scrollback in your terminal. Once a line had scrolled up, it'd be gone.)

I wonder if Windows changed the behavior of writes when text is selected.
I'll need to hunt down that version of Windows and test it.

@rprichard Must have confused winpty with something else that I remembered used hooks for this 😛
I'm on Windows 10.0.16299.125 x64. This is what you get if you install Windows 10 x64 and update it to the latest update ATM.

I wonder if Windows changed the behavior of writes when text is selected.

They did, and it looks accidental. I filed this issue: https://github.com/Microsoft/console/issues/40.

This was fixed in Windows 10 version 1803 (https://github.com/Microsoft/vscode/issues/36630). @rprichard https://github.com/Microsoft/console/issues/40 can probably be closed as well.


Add support for Python 3.6 and 3.7

Currently the README specifies that Python 3.3, 3.4 and 3.5 are supported.
In order to merge https://github.com/celery/celery/pull/4720 safely we need you guys to support 3.6 and 3.7. Our tests pass, but we'd rather have official support in case something goes wrong.
Are these Python versions supported? Is the README simply outdated?

4.0 has been released and supports Python 3.x. If you run into any further issues you can open them in our new monorepo https://github.com/Azure/azure-sdk-for-python


80211au fails to compile on linuxPackages_latest

Describe the bug
When compiling unstable with 8812au and linuxPackages_latest
/build/source/os_dep/linux/ioctl_cfg80211.c: In function 'rtw_cfg80211_ch_switch_notify':
/build/source/os_dep/linux/ioctl_cfg80211.c:469:17: error: too many arguments to function 'cfg80211_ch_switch_started_notify'
  469 |                 cfg80211_ch_switch_started_notify(adapter->pnetdev, &chdef, 0, 0, false, 0);
      |                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from /build/source/include/osdep_service_linux.h:98,
                 from /build/source/include/osdep_service.h:54,
                 from /build/source/include/drv_types.h:27,
                 from /build/source/os_dep/linux/ioctl_cfg80211.c:17:
/nix/store/7j5krfzkiw31vkkv1jy13lpfzkiryn5s-linux-6.9-dev/lib/modules/6.9.0/source/include/net/cfg80211.h:8811:6: note: declared here
 8811 | void cfg80211_ch_switch_started_notify(struct net_device *dev,
      |      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/build/source/os_dep/linux/ioctl_cfg80211.c:488:9: error: too many arguments to function 'cfg80211_ch_switch_notify'
  488 |         cfg80211_ch_switch_notify(adapter->pnetdev, &chdef, 0, 0);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~
/nix/store/7j5krfzkiw31vkkv1jy13lpfzkiryn5s-linux-6.9-dev/lib/modules/6.9.0/source/include/net/cfg80211.h:8795:6: note: declared here
 8795 | void cfg80211_ch_switch_notify(struct net_device *dev,
      |      ^~~~~~~~~~~~~~~~~~~~~~~~~

Steps To Reproduce
Steps to reproduce the behavior:

Enable 8812au and the latest kernel:

  boot.kernelPackages = pkgs.linuxPackages_latest;
  boot.initrd.kernelModules = [
    "8812au"
  ];

Update
The error above should appear.

Expected behavior
There should be no error.
Screenshots
Additional context
The 8812au driver we currently use https://github.com/morrownr/8812au-20210629 is no longer maintained. Should probably switch to https://github.com/morrownr/8812au-20210820.
I'm not a maintainer but could probably provide the necessary changes if switching to 20210820 is an acceptable solution.
Notify maintainers
@moni
Metadata

Add a :+1: reaction to issues you find important.

Resolved in #326863


ARROW-2818: [Python] better error message when trying to convert sparse pandas data to arrow Table

https://issues.apache.org/jira/browse/ARROW-2818

Codecov Report

Merging #4497 into master will decrease coverage by 23.15%.
The diff coverage is 80%.

@@             Coverage Diff             @@
##           master    #4497       +/-   ##
===========================================
- Coverage   88.29%   65.13%   -23.16%
===========================================
  Files         846      475      -371
  Lines      103629    60570    -43059
  Branches     1253        0     -1253
===========================================
- Hits        91497    39453    -52044
- Misses      11885    21117     +9232
+ Partials      247        0      -247

Impacted Files
Coverage Δ

python/pyarrow/tests/test_pandas.py
94.94% <100%> (+0.01%)
:arrow_up:

python/pyarrow/pandas_compat.py
96.98% <100%> (+0.01%)
:arrow_up:

python/pyarrow/pandas-shim.pxi
64.83% <50%> (-0.69%)
:arrow_down:

cpp/src/arrow/util/memory.h
0% <0%> (-100%)
:arrow_down:

cpp/src/gandiva/date_utils.h
0% <0%> (-100%)
:arrow_down:

cpp/src/arrow/extension_type.h
0% <0%> (-100%)
:arrow_down:

cpp/src/arrow/compute/kernels/compare.h
0% <0%> (-100%)
:arrow_down:

cpp/src/arrow/util/memory.cc
0% <0%> (-100%)
:arrow_down:

cpp/src/arrow/filesystem/util-internal.cc
0% <0%> (-100%)
:arrow_down:

cpp/src/arrow/util/sse-util.h
0% <0%> (-100%)
:arrow_down:

... and 602 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 997226a...eddac8c. Read the comment docs.


Add Eclipse Adoptium "Security Audit" suggested compiler options for jdk-21.0.3+ gcc builds

As part of the Eclipse Adoptium security audit, the following gcc options were suggested as being used to enhance the security of the JDK binaries.
Code generation suggestions:

-fstack-protector-strong : Extends the basic stack protection without impacting all functions, see: https://lwn.net/Articles/584225/
--param=ssp-buffer-size=4 : param The minimum size of buffers (i.e. arrays) that receive stack smashing protection when -fstack-protector is used. This ties in with -fstack-protector-strong to better cover the stack protection.

Compiler Warning suggestions:

-Wformat : printf/scanf string checks
-D_FORTIFY_SOURCE=2 : string related compile time checks
-Wformat-security : enahnced printf/scanf string security checks
-Wshadow : warn about variable/function shadowing
-Wconversion : Warn for implicit conversions that may alter a value

@sxa fyi

dup of https://github.com/adoptium/temurin-build/issues/3685


Clarification that text-valued variables and attributes can be Unicode string or UTF-8 char arrays

See issue #141 for discussion of these changes.
Release checklist

[NA] Authors updated in cf-conventions.adoc? Add in two places: on line 3 and under .Additional Authors in About the authors.
[NA] Next version in cf-conventions.adoc up to date? Versioning inspired by SemVer.
[Y] history.adoc up to date?
[NA] Conformance document up to date?

This is a challenge! I did a bit more (unsatisfying" research into netcdf, string, and Unicode.
See my comment on #141, but I don't think this is ready to merge :-(

superseded by https://github.com/cf-convention/cf-conventions/pull/556


[FEATURE] Allow UnknownImport in starting actor

There's import error while starting actor.

I don't have this error in rust version of wasmcloud_host.
I maybe use dependences that uses wasmbindgen in compiling my Actors. But the functions I used may not eventually calls wasmbindgen functions.

Actors are not compatible with wasm bindgen. Bindgen creates rust code that relies on the server to have certain imports available, like those exposed by the bindgen JavaScript shim.
Actors get their capability from capability providers and cannot use bindgen.

If the bindgen function isn't used then it should be removed by post process cleanup of the wasm file before being signed by wash.

I am using https://github.com/rustwasm/wasm-snip, it works by snipping out "bindgen" functions

If snipping out the bindgen functions works then it should work just fine with the runtime. Actors should only ever import the requirements from the wasmbus protocol


@fermata="within"

From<EMAIL_ADDRESS>on August 01, 2012 16:51:15
Is it really necessary that @-fermata on //note etc. has a datatype offering "above", "below" and "within"? I think the last option makes no sense on fermatas…
Original issue: http://code.google.com/p/music-encoding/issues/detail?id=101

From<EMAIL_ADDRESS>on August 01, 2012 14:26:29
"within" is a side effect of using the data.STAFFREL datatype for @-place. You want another data type?

From<EMAIL_ADDRESS>on August 01, 2012 14:29:54
Yes, I guess so. "within" makes absolutely no sense here, or has anyone seen a fermata "within" the note its assigned to (or, to make it easier, just within the staff)? There might be other occasions where we use data.STAFFREL although we only want to allow "above" and "below", although I don't know one right now…

From<EMAIL_ADDRESS>on August 01, 2012 14:32:19
@-place expresses a relationship to the staff, not to the note.  Yes, I have seen fermatas within a staff.  It's difficult to read, but they exist nonetheless.

From<EMAIL_ADDRESS>on August 02, 2012 11:08:12
Labels: -Type-Enhancement Type-Bug

From<EMAIL_ADDRESS>on August 02, 2012 11:14:17
This is not a critical issue so I'm delaying it to 2013.
Labels: -Milestone-ReleaseNext Milestone-Release2013

From andrew.hankinson on October 17, 2012 07:52:52
Labels: -Milestone-Release2013 Milestone-ReleaseNext

From<EMAIL_ADDRESS>on November 15, 2012 07:29:01
substituted data.PLACE (above, below) for data.STAFFREL (above, below, within) in att.fermatapresent/@ferm
Status: Fixed

From<EMAIL_ADDRESS>on November 16, 2012 07:48:11
This issue was closed by revision r718 .


Publish only to Maven Central on CI

This avoids publishing to the test-local repo which, while cheap, does a bunch of file I/O which is slow on Macs.

Copying this! https://github.com/coil-kt/coil/pull/2302


Hooks for calling FastLanguageModel.for_inference() when model.eval() is called

It makes some intuitive sense to either have this hook by default or at least allow users to patch their model so this behavior exists. Would be great to be able to benefit from unsloths fast inference implementation without needing to update various trainers.
I've experimented with this in a modified DPO script and it works well.
https://gist.github.com/lapp0/e7d17884ed76669194c36e7fb3f64040#file-gistfile1-txt-L31-L44

@lapp0 Oh I think we internally use torch.inference_mode which will mimic .eval()

Sorry if I wasn't clear.
I'm working on creating a generalized base class for trl which should work with PPOTrainer, DPOTrainer, KTOTrainer, ORPOTrainer, and RLOOTrainer. (But to start it should only impact PPOTrainer and `RLOOTrainer) https://github.com/huggingface/trl/pull/1586
Currently the plan is to check is_unsloth_available, and if so, call
model.eval()
if is_unsloth_available():
    FastLanguageModel.for_inference(model)
model.generate(...)
model.train()
if is_unsloth_available()
    FastLanguageModel.for_training(model)

Just thought it might make sense on your end to add a hook to model.eval() so your fast inference implementation is automatically applied for every trainer using your library which performs eval.

Oh interesting - fair point I actually did not know on eval being able to store gradients so its just like torch.no_grad() Also very cool with the PR!


chore: [ADR-0004] use RadixUI

Description
Fixes #734
Related to #730

@KhudaDad414 thanks for reviewing this, could you also review https://github.com/asyncapi/studio/pull/730 as it's linked

/au

/rtm


Fix to build failure for duckdb_wasm sub-project test

Description
This is PR for issue #1888.
A test for duckdb_wasm sub-project has failed to build.
It resolves this issue.
Out-of-scope
To build test artifact is success, but executing test still fails.
Following testcases:
AllTypesTest.FullRangeTypes
This test is used a udf test_all_types.
It fails to not match expects columns with test_all_types columns.
It will be easy modifying.
ParquetLoadTest.LoadParquet
In 1.29.0, DuckDB extensions support enabled by default.
So calling duckdb_web_parquet_init(&db->database()) failes.
It will be easy modifying. (line-delete only)
Same for the following

ParquetLoadTest.LoadParquetTwice
FileSystemBufferTest.FlushFrameMemoryBugRegression
WebFileSystemTest.LoadParquet

CSVExportTest.TestExport
This test fails calling WebDB::Connection::InsertCSVFromPath.
{"exception_type":"Not implemented","exception_message":"Unsupported compression type for default file system"}

But this method does not have a compression option.
It makes no sense for me.
FilePageBufferTest.FixSingle
It fails calling TestableFilePageBuffer::BuffersFile after FilePageBuffer::FileRef::Release.
It seems that accessing released file buffer is incorrect.
It will need to make this specification clear.

I can include the fixes AllTypesTest.FullRangeTypes and ParquetLoadTest.LoadParquet this PR.
What do you think?


Update broken links

The docs directory appears to have moved, so I've updated the links.
I was unable to find a direct link for fork (The link titled 'Split up application state in large apps') but guessed from reading that it was referencing Domains.

What kind of change does this PR introduce?

Documentation

Does this PR introduce a breaking change?

No

If yes, please describe the impact and migration path for existing applications:
Does this PR fulfill these requirements:

[ ] All tests are passing
[ ] yarn pretty has been run
[ ] Commits reference open issues

Additional Information

Ack. Thank you so much. I guess I didn't clean up all of my mess when we moved to a mono-repo 😅 .
Thanks!


Feature request: "Stream Governance Packages" terraform resource

Hello! I've recently started using the confluent terraform provider, and so far it's been a great experience. Thank you for creating this provider! After creating a cluster, I noticed that features like Schema Registry were turned off by default, and I needed to select a Stream Governance Packages in order to turn that feature on. I don't see the Stream Governance Packages in the terraform provider, which means I would need to enable it by hand. We are trying to fully automate the process for creating kafka clusters and am hoping we can do that mostly through the terraform provider, with some help from the confluent API where needed. Can you add the Stream Governance Packages as a resource in terraform? At first glance, I don't see API support for Stream Governance Packages, so that may be the blocker here. If that's the case, where should I file a feature request to add Stream Governance Packages management to the API? Thank you!

Thanks for creating this issue @johnnyplaydrums and thanks for the kind words!
You're absolutely right, first we'd need to get a public API for it, I've just created an internal feature request for a relevant team 🤞. Will try to report back once I get the response.

I am updating from Mongey/confluent and the missing schema registry support is blocking us. The old provider is not longer compatible with the confluent API but the new one has no way to make api keys for schema registry. I tried using the values I imported from an existing resource, but they are not valid:
managed_resource {
    id          = "lsrc-XXXXX"
    api_version = "stream-governance/v2"
    kind        = "SchemaRegistry"

    environment {
      id = confluent_environment.environment.id
    }
  }

Gives error:
expected managed_resource.0.kind to be one of [Cluster], got SchemaRegistry
expected managed_resource.0.api_version to be one of [cmk/v2], got stream-governance/v2
invalid value for managed_resource.0.id (the resource ID must be of the form 'lkc-')

Also I see no way to retrieve the URL for the schema registry.

👋 @luggage66 that's very exciting to hear that you're trying to use TF Provider for Confluent!
We're aware of this issue (see #5) and it's on the roadmap.

Actually when I think about this issue again, I feel like enabling SG is the same thing as creating a SG cluster, in other words, this is a duplicate of #5 so I'll close this issue.

update: we released a new 1.10.0 version of TF Provider where we dded new confluent_stream_governance_cluster resource and a corresponding data source in an Open Preview lifecycle stage.
cc @johnnyplaydrums @luggage66

Really awesome turnaround time on this feature! Thank you all so much!


Support for jennybc/reprex in RDocs

Suggestion by @richierocks: Jenny bryan has been working on an easy way to create reproducible examples. You can create GitHub- or StackOverflow-flavored markdown chunks that you can easily include in an issue or question. https://github.com/jennybc/reprex
We should research whether one of the currently generated markdown formats are compitable with the markdown format that RDocs uses. If yes, great, and ask Jenny that she can include it as a supported platform. If not, suggest an additional markdown flavor to the reprex repo so it is supported.

@ludov04 commented on Fri Feb 17 2017
After trying to understand what reprex do, more closely, I'm not completely sure what to do here.
RDocumentation uses github flavored markdown, so it's already supported if people uses "gh" as a venue in reprex

@richierocks commented on Fri Feb 17 2017
@ludov04
That's useful to know; it wasn't clear to me what flavor of markdown was being used in RDocumentation examples. In that case this probably just needs documenting.
I suggest changing the advice text in RDocumentation example code blocks from

Use markdown to format your example

to

Use github-flavored markdown to format your example

Then, optionally, offer an update to the jennybc/reprex README to say that venue = "gh" should be used for creating DataCamp examples.

I think this can be abandoned at this point.


Add more detail to GitHub messages

This adds more detail to comments generated by the GitHub commit comment
and GitHub PR comment formatters by including the name of the Pronto
runner that generated the message and the warning level of the message.
In situations where a repo is using many different runners to report on
various different checks, it can be difficult to know which tool
generated which message. Adding the name of the runner allows this to be
more easily seen.
Adding the warning level to the message also could help to suggest which
messages are higher or lower priority if there are many messages being
generated.

@mmozuras can you merge this PR please?

@aergonaut could you provide a screenshot of what the resulting comments look like?

@doomspork There's an example of the generated text in the comments in the code

Generated by Pronto::Rubocop. Severity: ⚠ warning. Lorem ipsum dolor sit

@aergonaut I wanted to see what it would appear like in GitHub's UI rather than go by what the comments are showing.
I'm curious how it will look when combined with #154, my initial concern is that it'd get real noisy real quick with the extra meta data and a bunch of emojis.  Just my $0.02 though.

@aergonaut This is exactly what we're missing at the moment. Are you planning to continue working on that? (e.g. rebase)
@mmozuras Are you planning to merge it?

@aergonaut @mmozuras bump

This would be awesome to see in the GitLab formatter as well.

Sorry to let this PR fall off, I'm still interested in fixing it back up and getting it into a merge-able state. I'm now thinking the format might be too verbose and maybe a more concise format that still shows the same amount of data might be better.

What led me to this issue is that I simply wanted which runner threw the error. Currently to fix and retest a Reek issue, I have to run Brakeman, Rubocop and Reek, locally which takes time.

@davejlong I've found googling the exact error message almost always leads me to the culprit.
I do think this feature would be useful but I stand by my original comments with regards to the amount of noise this would produce.  If we could make the message configurable, that would suit most everyone's needs I think.
@aergonaut could this be used in the other formatters to maintain consistency?

@doomspork More configurable messages sounds like a good idea, I agree. Should I try to look at making Pronto in general expose some message format configuration options? Maybe I could make Comment store more information, or delegate to the Message class, and maybe use a configurable format string for the body, for example "%{runner} (%{level}): %{message}".

@aergonaut I like the idea of a configurable message with a formatted string like you've suggested!

@aergonaut I like the idea of configurable format very much!
I would expect it to work with every formatter, solving this PR too 😄.

Cool sounds great. I'll close this PR and open another one with a more generic solution!


Add dev-lang/zig-bin-0.9.1-r1 to repository.

I added dev-lang/zig-bin, tested on my local repo and everything work fine.

Thanks for the PR.


fix arg order in merge cmd

There are a few more breaking changes that I'd like to make to the cli. We should probably batch them together.

There are a few more breaking changes that I'd like to make to the cli. We should probably batch them together.

Codecov Report

Merging #1341 (c7aeacc) into master (c2c5dbe) will increase coverage by 42.15%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           master    #1341       +/-   ##
===========================================
+ Coverage        0   42.15%   +42.15%
===========================================
  Files           0      152      +152
  Lines           0    12155    +12155
===========================================
+ Hits            0     5124     +5124
- Misses          0     6321     +6321
- Partials        0      710      +710

Impacted Files
Coverage Δ

api/client.go
3.29% <0.00%> (ø)

api/controller.go
39.02% <0.00%> (ø)

catalog/rocks/cataloger.go
26.28% <0.00%> (ø)

catalog/rocks/entry_catalog.go
16.93% <0.00%> (ø)

graveler/graveler.go
37.36% <0.00%> (ø)

httputil/logging.go
0.00% <0.00%> (ø)

auth/cache.go
12.00% <0.00%> (ø)

gateway/operations/deleteobject.go
0.00% <0.00%> (ø)

metastore/hive/partition_collection.go
90.00% <0.00%> (ø)

graveler/ref/rev_parse.go
93.75% <0.00%> (ø)

... and 147 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 c2c5dbe...c7aeacc. Read the comment docs.


[DOCFIX] FileSystem doc update

Updates the doc to be consistent.

Thanks for the doc fixes, I've left some comments

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/8308/
Test PASSed.

+1 LGTM

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/8313/
Test FAILed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/8314/
Test PASSed.

+1 LGTM

+1 LGTM

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/8315/
Test PASSed.


Support for regions

Palo Alto Networks Cloud Python SDK version: v1.5.0
Python version: 2.7, 3.5+
Operating System: any

Description
There are two scenarios where the concept of region is relevant:

The region extracted from the base64 params provided by the Cortex Hub redirect which is used by the get_authorization_url() method, e.g. americas or europe.
The region specified in the API gateway URL, e.g. api.us.paloaltonetworks.com or api.eu.paloaltonetworks.com (currently defaults to us).

The first is relevant to the identity provider while performing authorization whereas the second determines what regional datacenter to direct API requests to (which should correspond to where the Cortex data lake tenant, et al. reside).
Currently, pancloud is missing a way to set or define a default region that could be used to determine which regional datacenter to direct API requests to. With the current API, you're forced to supply the full API gateway url to direct API requests to a region other than api.us.paloaltonetworks.com, which is the default.
Proposals

Add a default_region kwarg to the HTTPClient class. The value provided, e.g. us or eu, could be used to construct the default url used in all API requests made with that HTTPClient() object. Note that default_region would not be applicable to the Credentials class.
Add support for a PAN_DEFAULT_REGION environment variable. The behavior would be similar to proposal 1 except that the default_region constructor argument would take precedence over the envar.
Add support for a default_region or region to the credentials.json file or credentials store. This one feels a bit out of place, since the region would not necessarily be applicable to Credentials. Again, the region used by get_authorization_url() should normally be extracted from the base64 params passed by the Cortex Hub. It's worth noting that AWS boto3 credentials allow for specifying region. Note that the PAN_DEFAULT_REGION envar would take precedence over this value.

Other considerations
Another interesting approach would be to leverage a custom JWT claim to determine the regional URL, since the IdP would ostensibly have prior knowledge of the region used during authorization. The challenge with such an approach would be sharing the region across multiple instances of HTTPClient() since the value would, ostensibly, be extracted from the access_token in the fetch_tokens() or refresh() response (an operation performed within the scope of a single Credentials() object).
Another thing to note is that it is quite easy for an app to implement its own "region selector," since an application would also have knowledge of what region was used during authorization. This is the current recommended way to handle regional selection of url in the absence of default region support.

Hey @sserrata,
I think this would be good, as I've read this could be a really big change for any applications using the get_authorization_url method to derive the auth_base_url and state, meaning a state parameter is not provided by user, but would give us better populated dynamic groups.


docs: Fix deadlink in contributing guide

Signed-off-by: Xuanwo<EMAIL_ADDRESS>Fix #82

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1572

Relevant Lines:
1801

💛  - Coveralls


Github as search engine doesn't work

Whenever Github as a search engine is selected in the settings menu the text field shows "Google" and any submit gives me results from Google. All the other search options such as DuckDuckGo and Wolframalpha is working correctly.

Thanks for the note! Hopefully will fix this weekend.


[Tearsheet] Focus lost on stacked tearsheet

Hi guys,
some strange things going on here.
When I open a second tearsheet the focus does not move from the first one.
Also if I click on it the focus goes away.
here a sandbox
let me know if I can do more to solve this
Thanks

@ninjasun Thanks for this report -- a fix has now been made in PR #1134 and should be included in the next release.

@dcwarwick thanks.
I tried with release 0.97.0  but does not seem to work.
code

@ninjasun interesting: it doesn't seem to work when you don't have a selectorPrimaryFocus on the tearsheets -- you don't have a selectorPrimaryFocus set, but in my test case I was setting that. Sorry -- I will investigate and fix it for the case when selectorPrimaryFocus is not set!
That does mean that there is now a workaround you can use with 0.97.0: add a selectorPrimaryFocus to each of your tearsheets! Normally this would be a CSS selector within the tearsheet of the field to take focus, but you can actually just include selectorPrimaryFocus: '*' and that will provide the workaround to the focus trap. When I added selectorPrimaryFocus: '*' to the two tearsheets in your codesandbox I was able to click in the stacked tearsheet and focus worked as expected.
Incidentally, you should not be doing this with your tearsheets:

Preventing the tearsheets from rendering when they are not open breaks the entry and exit motion of the tearsheets.

Reopened, as the fix only works if selectorPrimaryFocus has been set. Also, setting an invalid selectorPrimaryFocus currently causes errors -- it would be better to fail silently.

@ninjasun  If you really need to exclude a tearsheet from rendering, I think you would need to use two flags: a 'render' flag and an 'open' flag. The 'render' flag would be used to say if (!render) { return null; } else { return (....tearsheet....); }, while the 'open' flag would be passed directly to the open prop of the tearsheet. When the tearsheet is to be shown, you would then need to set the 'render' flag to true and wait for an animation frame (or short timer) and set 'open' to true. When the tearsheet is to be closed you would need to set 'open' to false and then set a timeout of at least the Carbon $duration--moderate-02 interval then set 'render' to false. Alternatively, if you have expensive-to-render content then you could make that conditional on open === true, but be aware that this may cause parts of the content to vanish immediately while the exit animation is still happening. It's probably simplest just to have the tearsheet always rendered unless that makes a particular problem. I will add a note to the documentation to point this out.


Integration proposal - nClam antivirus

.NET Aspire issue link
No response
Overview
I've started an integration sample that uses nClam antivirus to scan file uploads for malware.  Source at:  https://github.com/csharpfritz/aspireantivirus
Would the team be interested in a contribution of that initial sample code and formalizing the feature?
Usage example
This integration could be used to scan uploads to a service to ensure that no malware is included
Additional context
No response
Help us help you
Yes, I'd like to be assigned to work on this item

AV to scan user provided files makes a lot of sense and you're making me question why I've never done that in a project before 🤣.
Would be happy to have it as a contribution, I notice your repo has a hosting integration only and the client is more directly using the client for ClamAV rather than an Aspire client integration. Would it make sense to also include a client integration to simplify the workflow of consuming it?

You're right, we should put together a client implementation as well... and I started one, then got a little confused.  I could use some assistance in tightening that client implementation, and would be happy to share the work I started.

If you want to get a pull request up @csharpfritz we can help iterate and refine it

AV to scan user provided files makes a lot of sense and you're making me question why I've never done that in a project before 🤣.

I'm with @aaronpowell... I would LOVE to see this and happy to lend a hand if needed

I have some content built and I can share it to the repoOn Nov 15, 2024, at 15:53, Dan Siegel @.***> wrote:Re: [CommunityToolkit/Aspire] Integration proposal - nClam antivirus (Issue #102)AV to scan user provided files makes a lot of sense and you're making me question why I've never done that in a project before 🤣.I'm with @aaronpowell... I would LOVE to see this and happy to lend a hand if needed—Reply to this email directly, view it on GitHub or unsubscribe.You are receiving this email because you commented on the thread.Triage notifications on the go with GitHub Mobile for iOS or Android.

I'll put together that PR with the nClam support this week


fix: use Nothing initialization without pure function caller

Current dependencies on/for this PR:

main

PR #86   👈

PR #87

This comment was auto-generated by Graphite.

@pevisscher started a stack merge that includes this pull request via Graphite.

@pevisscher merged this pull request with Graphite.


Add the ability to extend esbuild options

From reading some other issues, it sounds like you may be planning to remove esbuild all together. However, in the meantime, I keep running into scenarios that require some specific esbuild options (such as external). This PR allows you to add to the default options:
adapter: azure({
	esbuildOptions: {
		external: [
			'pg-native'
		]
	}
})

Thanks for the PR! Are there esbuild options you need to set besides external? If possible, I'd prefer to only expose external to minimize the API surface of the adapter.
Please also update the README with the new adapter configuration option.

No problem. The two I've come across so far external and keepNames.
Would you prefer it typed to just those two?
type Options = {
	debug?: boolean;
	customStaticWebAppConfig?: CustomStaticWebAppConfig;
	esbuildOptions?: Pick<esbuild.BuildOptions, 'external' | 'keepNames'>;
};

Would you prefer it typed to just those two?

Yep, only have those in the typings, and only pass those additional options to esbuild instead of spreading the whole object.
Can you also open an issue with the use cases this is solving for you? This will help me in the future to understand why those options are exposed if I ever remove/replace esbuild.

Fixes https://github.com/geoffrich/svelte-adapter-azure-swa/issues/56

Can you also open an issue with the use cases this is solving for you? This will help me in the future to understand why those options are exposed if I ever remove/replace esbuild.

I've added the use case for both options. I did see there have been some changes with the azure blob sdk, since I first ran into the need for keepNames. However, I haven't had a chance to see if it resolves the issue. If you prefer, I can reduce this PR to just exclude until I get a chance to confirm whether keepNames is still needed or not?

Thanks for opening the issue. Yes, please update this PR to only add exclude for now and open a separate PR for keepNames if you confirm it is still needed.

This is released in 0.9.0. Thanks for your patience!


Fix network example names in stress README

This fixes a few more naming issues with the latest readme update to the stress tutorial.

/check-enforcer override


Starter has issues getting input on Windows in Git Bash

If you use the starter in Git Bash, you can't select a schema from the schema options list. Keyboard input does not seem to be accepted.

Reproduced this today in MinGW64 prompt as well as in conhost.exe (cmd prompt).
Windows 10 14393
node, v4.4.3


⚠ Cioos National CKAN has degraded performance

In ef67628, Cioos National CKAN (https://catalogue.cioos.ca/) experienced degraded performance:

HTTP code: 200
Response time: 22289 ms

Resolved: Cioos National CKAN performance has improved in 81e6bc9 after 8 minutes.


predict the results

why the model will output mask-like image? i need your help

Sry, I do not find this bug in my output on SCUT-Syn using the same code,
and my results are normal.  And I find your results seem to contain some
input images, such as img_59.png and img_94.  Maybe there exist some
mistakes in your input data or the training process?
Aspen0309 @.***> 于2023年2月28日周二 15:38写道：

[image: image]
https://user-images.githubusercontent.com/126053393/221784883-ef641d34-66f5-4c3f-8555-936c0a90eadb.png
why the model will output mask-like image? i need your help
—
Reply to this email directly, view it on GitHub
https://github.com/lcy0604/EraseNet/issues/14, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ7RD3VP2GYDZMBSQP2ZBVLWZWTRDANCNFSM6AAAAAAVKLQFBA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I have the same question,how did you solve it. Thanks.


drt: functions changed to have more explicit output

This PR contain minor changed to some functions in PA. Functions changed take a reference to a container and fill said container. This PR changes it so instead it returns it.
This makes the functions role easier to understand, as its more obvious that its role is to return that structure and that it is an output, not an input.
This should not affect performance as due to move semantics instead of copying the returned structure its ownership is transferred to the caller.
Some functions have name changes:

prepPoint_pin_mergePinShapes → mergePinShapes (Conform to PR #5770)
initMaster2PinLayerRange → initMasterToPinLayerRange

Also LayerRange was changed to a pair instead of a tuple, as its always a tuple of 2 elements. This does not affect its use in code as it is accessed using auto [a, b] = LayerRange.

I'm satisfied but want to leave a chance for @osamahammad21 to comment before merging.


nacos+actuator has a error

env: nacos0.9.0,  spring-cloud-alibaba 0.2.1.RELEASE ,spring-cloud.Finchley.SR, spring-boot 2.0.8.RELEASE
jdk11
After adding the following reference:
<dependency>             <groupId>org.springframework.boot</groupId>             <artifactId>spring-boot-starter-actuator</artifactId>         </dependency>
Error occurred：
org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'nacosRegistration': Invocation of init method failed; nested exception is java.lang.NoClassDefFoundError: org/springframework/boot/actuate/autoconfigure/web/server/ManagementServerProperties
Get right after removing actuator

Could you please provide a Minimal, Complete, and Verifiable example?

https://github.com/sundong1982/spali.demo

works fine on my machine with jdk 11.

@fangjian0423 It was my nacos-server cluster that each node's log was exported to the same place, which caused three nacos-servers to start at the same time. Two of them were not running. I visited nacos-server through nginx before and did not notice this problem. Now I find this problem and have solved it.
thx:)


Panorama sky with image texture shows black sky in editor and produces endless draw errors in debugger

Godot version
4.0 dev 489f11ee0
System information
Win10, GLES3, RTX 2070 mobile
Issue description
When loading an image into the image slot on panorama sky material, the image does not show in the editor - the background just turns completely black instead although the texture does seem to be affecting the materials in the scene.
This Degubber shows thousands of this error:
ERROR: Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline
Steps to reproduce

start a 3d scene
add a world environment node
select Sky - New Sky
Set sky Material to new panorama sky material
Set Panorama to new image texture
load an image into the Image slot on the image texture
watch this error deluge the debugger: ERROR: Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline
sky turns completely black

Minimal reproduction project
No response

Does the problem go away if you save the scene then use Scene > Reload Saved Scene?

Does the problem go away if you save the scene then use Scene > Reload Saved Scene?

it would appear so, yes.

I can confirm this on master 489f11ee0.

Closed by: https://github.com/godotengine/godot/pull/54489


Linting some packages

Following #88, lint some packages : middlewares, provider and types 🐼.
This will ease the soon to be coming script/validate-golint script 😇.
🐸

LGTM!
Thank you @vdemeester for this!


re: Waterfalls - duplicate records in dataset

I will need to carefully filter the data because of duplicate records. The metadata .xml from SB states that any waterfall records that were closer together than 50 m were aggregated; yet, I randomly came across this example of a duplicated record:

Notably: If I were to be summing up the elevations through a spatial join with states, which I plan to do, I would be double counting these 15 ft for this one waterfall that has two duplicate records:

The possible solution might be to aggregate by the source name "Peterson-Falls-13253" or through NHD data- but will need to do this carefully to double check that this takes care of all duplicate records.
I would also like to look into why there are two separate World Waterfall Database record numbers in the screenshot above, yet only one page for this "Peterson-Falls-13253"
https://www.worldwaterfalldatabase.com/waterfall/Peterson-Falls-13253

Image showing mapping output for World Waterfall Database double verifying that there is only one "Peterson Falls" at this location (the falls to the NW are Montreal Falls)

Closing this issue for now, since I determined the impact on the final dataset is negligible. Much less impactful than the next issue, which is that the data release is grossly out of date...


Remove space stripping ERB tags from marriage-abroad templates

This replaces all uses of <%- -%> and <%= -%> with <% %> and <%= %> respectively.
We started using <%= -%> when converting from i18n YAML templates to ERB templates. Using -%> to strip trailing spaces/newlines allowed us to exactly replicate the outcomes that were previously generated by the i18n YAML system. This gave us confidence that the conversion was accurate. These are no longer necessary and so I've removed them in favour of consistently using <%= %>.
Using <%- -%> doesn't have any effect and can safely be replaced with <% %>. From the ActionView::Base docs:

When on a line that only contains whitespaces except for the tag, <% %> suppress leading and trailing whitespace, including the trailing newline. <% %> and <%- -%> are the same. Note however that <%= %> and <%= -%> are different: only the latter removes trailing whitespaces.

I ran all regression tests after making these changes. All changes to the artefacts are due to the addition of non-significant whitespace so I'm happy to accept the changes.

Minor: I think it might be clearer to group the 3 commits for each flow together, i.e. remove space-stripping ERB tags, update artefacts, update checksums. But I don't feel strongly about this.

Very minor: I find "Update artefacts" and "Update checksums" in the commit note subjects overly generic - I'd prefer "Update regression test artefacts" and "Update regression test checksums". However, I realise this would mean very long subject lines and perhaps we all know what you mean.

Very minor: It's pretty obvious, but I think it would be good to say that the regression tests were all passing in the commit notes for updating checksums.

Do you think it would be worth trying to add some kind of check that we don't re-introduce any of these? I wonder whether govuk-lint would allow this?

I think it might be worth making explicit the motivation behind this PR, which I assume is to simplify the code i.e. the non-space-stripping ERB tags are conceptually simpler and terser.

Apart from my minor comments, this looks good to me.

I think it might be worth making explicit the motivation behind this PR, which I assume is to simplify the code i.e. the non-space-stripping ERB tags are conceptually simpler and terser.

I've added the following sentence to the PR description: "We don't need the space/newline stripping behaviour and consistently using the same ERB tags leaves us with a slightly simpler system." Hopefully that makes sense!

Do you think it would be worth trying to add some kind of check that we don't re-introduce any of these? I wonder whether govuk-lint would allow this?

A very, very quick search suggests that Rubocop (and therefore govuk-lint-ruby) doesn't support parsing ERB templates.
I'm not sure it's worth the effort of trying to ensure these aren't introduced in future. I'm happy to leave it to see if we notice any more being added before trying to do something about it.

Minor: I think it might be clearer to group the 3 commits for each flow together, i.e. remove space-stripping ERB tags, update artefacts, update checksums. But I don't feel strongly about this.

This was a side effect of how I made these changes:

I used find/replace to replace the space-removing ERB tags.
I used git add <dir> and git commit <dir> to add the changes for a specific set of files (e.g. a single Smart Answer)
I ran all the regression tests
I used git add <dir> and git commit <dir> to add the changes for each Smart Answer
I updated the checksum data for each Smart Answer and committed those changes

I'm going to leave the commits as they are given the time I think it might take to combine the commits and ensure I've not missed anything.

Very minor: I find "Update artefacts" and "Update checksums" in the commit note subjects overly generic - I'd prefer "Update regression test artefacts" and "Update regression test checksums". However, I realise this would mean very long subject lines and perhaps we all know what you mean.

I'm happy to leave these as they are. Any potential confusion will hopefully be resolved when looking at the files in the commit (i.e. it contains changes to the regression test checksum file or regression test artefacts).


Extension not installable in VS Code version 1.87.2

I am unable to install the Easy Alt extension on Visual Studio Code version 1.87.2

Hi @jebitok-dev,
Thank you for filling the issue. I will look into it soon.

Hi @jebitok-dev,
Thank you for filling the issue. I will look into it soon.

Hi @jebitok-dev,
I just made some updates to this. Could you update the extension and vscode to see if this persists?
Also, it now works by pressing cmd/ctrl + E when in the alt attribute's quotation marks.
Thank you! 🙏🏽


Removing a server that's a secondary and adding it back

Hello,
After a geo-replicated database is disconnected from the primary using the ATER DATABASE '--' REMOVE SECONDARY ON SERVER '---' statement, is it possible to connect it back, without having to drop the database and recreate it?
If I run ADD SECONDARY for the database that was disconnected, I get this error message "The destination database name '---' already exists on the server '---'.". While I understand why, I was wondering if this can be overridden.
Thank you!

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

ID: baa6b22d-e969-1a7b-5598-a34e0f6314a6
Version Independent ID: 2d741fa1-3fea-7f78-5074-0627b1bfa129
Content: Rolling application upgrades - Azure SQL Database
Content Source: articles/sql-database/sql-database-manage-application-rolling-upgrade.md
Service: sql-database
Sub-service: high-availability
GitHub Login: @anosov1960
Microsoft Alias: sashan

@karthik25 We are checking this and will be getting back to you.

@NavtejSaini-MSFT Thank you! In a nutshell, I was wondering if I can "pause" the replication for a while and "unpause" it.


Replace default font with Nunito Sans

By default ,the Inter font from font/google is being used. If possible, we would like to use Nunito Sans to adhere to the design system.

https://github.com/gevuong/minimal-i18n-with-app-router/commit/3cfcf9db791195638236910d4c4e9e06aa271c9d


Tokens support

Adds token support so live preview tokens are stored in the database.
There are a couple of things that are out of the ordinary:

There's no TokenRepository contract, so we're just binding to the file repository class instead.
The file repository make method doesn't use the service container, so we have to override it.
Calling app(Token::class)::fromModel($model) doesn't work due to the constructor arguments, so this just calls Token::fromModel($model) directly.

These probably require PRs to core if we want them to work like everything else.
Let me know if anything needs changing.
The new update script assumes this will be in 3.5.0.

@ryanmitchell PR for supporting the preview tokens as we talked about. :)

Awesome, thanks for this.
I've added a PR to core for the cli install support: https://github.com/statamic/eloquent-driver/pull/277
We're switching to default repositories file rather than eloquent from the next release, so I've gone ahead and changed that in advance of that. You can just run the migration manually anyway, and opt-in (which is what I would expect on our installs).

There's no TokenRepository contract, so we're just binding to the file repository class instead.

 Since this PR is targeting Statamic 5, we can get a contract into core that you can bind to.

All looks good to me, but I'll let Jason merge it as he was working with you on this.