Omit new events for plugin extension for tabline.wez

I recently developed a tab bar plugin: https://github.com/michaelbrusegard/tabline.wez and I want to have an extension for resurrect. I specifically need the events for when the fuzzy finder is opened and when it is closed. If you have any good ideas for how the tabline should look with the extension that would be cool. Also if you have some feedback in how I can make the plugin better I am very open for that

I think so, the events are for enabling and disabling a custom tab bar specific to the plugin. As long as I have that it will work

I just submitted a PR that adds "start" and "finished" emitters that also return window and pane.
This should hopefully resolve this issue
https://github.com/MLFlexer/resurrect.wezterm/pull/59

@michaelbrusegard your plugin has a cool look and feel out of the box, and config should be familiar for people who use nvim.
@MLFlexer I don't know your thoughts, but I think it might be cool if a tab section showed the md_sleep_off icon along with the window title if the resurrect fuzzy picker is open.

your plugin has a cool look and feel out of the box

I am so glad you think so! I hope it can be useful for many people. The extension can customize the tab bar completely or just slightly (it is very versatile). It can also show a small loading indicator for the loading event. I can probably create something myself but I think it is better if you help decide how the extension should work and which events it should use. Though personally, I would like at least a custom look when you are inside the picker (for both the workspace switcher and for resurrect :)

Sorry for my slow response. Uni has started again, so I don't have as much free time as I had over the summer break 😃
@michaelbrusegard I have merged  #59  adding the emitters

@MLFlexer I don't know your thoughts, but I think it might be cool if a tab section showed the md_sleep_off icon along with the window title if the resurrect fuzzy picker is open.

I havn't had time to try out the plugin, so I don't really have any strong oppinions about this. But it sounds fine 😄

The resurrect.fuzzy_load.finished event is not working, but the events for https://github.com/MLFlexer/smart_workspace_switcher.wezterm/issues/34 is working correctly. I was also wondering if there are any other information that would be useful to have in the tab-extension for this plugin. Right now I am showing a Selection text when using the fuzzy picker. I also added a loading indicator that shows up on periodic save (I have not seen it yet so that event may also have issues, but It can also be that I have never looked at the tab-bar when a periodic save has happened).
I do not really need the window and pane to be returned for these events since I am not using them though it does not harm. Also it would be nice if there was separate events for resurrecting window or workspace when opening the fuzzyloader so I can show different text for each. I dont need separate close events

The resurrect.fuzzy_load.finished event is not working, but the events for MLFlexer/smart_workspace_switcher.wezterm#34 is working correctly. I was also wondering if there are any other information that would be useful to have in the tab-extension for this plugin. Right now I am showing a Selection text when using the fuzzy picker. I also added a loading indicator that shows up on periodic save (I have not seen it yet so that event may also have issues, but It can also be that I have never looked at the tab-bar when a periodic save has happened).
I do not really need the window and pane to be returned for these events since I am not using them though it does not harm. Also it would be nice if there was separate events for resurrecting window or workspace when opening the fuzzyloader so I can show different text for each. I dont need separate close events

Were you able to get it working?
I was able to use the events in my local config (tested on Mac and Windows) with:
   local event_listeners = {
    { event = 'resurrect.fuzzy_load',                        text = wezterm.nerdfonts.md_sleep_off .. ' Resurrect ' },
    { event = 'smart_workspace_switcher.workspace_switcher', text = wezterm.nerdfonts.cod_terminal_tmux .. ' Smart Workspace Switcher ' },
  }

  for _, v in ipairs(event_listeners) do
    wezterm.on(v.event .. '.start', function(window, _)
      local colors = wezterm.get_builtin_color_schemes()['Catppuccin Mocha']

      window:set_right_status(wezterm.format({
        { Foreground = { Color = colors.ansi[6] } },
        { Text = v.text },
      }))
    end)
    wezterm.on(v.event .. '.finished', function(window, _)
      window:set_right_status(wezterm.format({
        { Text = '' },
      }))
    end)
  end

@joncrangle I am so stupid... I wrote stop instead of finished
The tabline.wez extension for resurrect now works perfectly. Are you able to test if the cpu and ram components works on windows or linux if you are using the tabline plugin

@joncrangle I am so stupid... I wrote stop instead of finished The tabline.wez extension for resurrect now works as expected. Are you able to test if the cpu and ram components works on windows or linux if you are using the tabline plugin

No worries! I can confirm that the cpu and ram components work on Arch linux. They did not work on Windows: error output from debug window:
19:46:07.782 ERROR wezterm_gui::termwindow > while processing update-status event: runtime error: ...ZstablinesDswez\plugin\tabline\components\window\cpu.lua:58: bad argument #2 to 'format' (number expected, got string)
stack traceback:
        [C]: in function 'string.format'
        ...ZstablinesDswez\plugin\tabline\components\window\cpu.lua:58: in function 'tabline.components.window.cpu.update'
        ...michaelbrusegardsZstablinesDswez\plugin\tabline\util.lua:86: in function 'tabline.util.extract_components'
        ...elbrusegardsZstablinesDswez\plugin\tabline\component.lua:74: in upvalue 'create_sections'
        ...elbrusegardsZstablinesDswez\plugin\tabline\component.lua:135: in function 'tabline.component.set_status'
        ...DscomsZsmichaelbrusegardsZstablinesDswez/plugin/init.lua:38: in function <...DscomsZsmichaelbrusegardsZstablinesDswez/plugin/init.lua:37>

A couple other things I found:

I tried using the resurrect extension, but there's a spelling error on the filename so it doesn't import properly. If you rename ressurect.lua to resurrect.lua it will fix it.

If the process isn't found, it might be worth querying the domain instead of defaulting to ssh. There are all kinds of possible domains that a tab might be (SSHMUX, DOCKER, etc.). I noticed this because when you bring up the Wezterm debug menu, it's tab process title defaulted to ssh so I found it a bit strange.

Feature request(s) (I can create an issue or discussion in your repo if you prefer):

Are you interested in adding an option involving process icons within the tab bar? That's what I use in my config at the moment, and I like it because it lets me have a very minimal tab bar. I modified it from folke's dotfiles, so I think the attribution is important within the code block below. I wasn't sure how you wanted to deal with this option, so at the moment it's structured as a different file, but you might prefer providing it as opts to process.lua if it's a feature you're interested in incorporating.

local wezterm = require('wezterm')

local foreground_process_name = ''

local icons = {
  ['C:\\WINDOWS\\system32\\cmd.exe'] = wezterm.nerdfonts.md_console_line,
  ['Topgrade'] = wezterm.nerdfonts.md_rocket_launch,
  ['bash'] = wezterm.nerdfonts.cod_terminal_bash,
  ['btm'] = wezterm.nerdfonts.mdi_chart_donut_variant,
  ['btop'] = wezterm.nerdfonts.md_chart_areaspline,
  ['btop4win++'] = wezterm.nerdfonts.md_chart_areaspline,
  ['cargo'] = wezterm.nerdfonts.dev_rust,
  ['curl'] = wezterm.nerdfonts.mdi_flattr,
  ['docker'] = wezterm.nerdfonts.linux_docker,
  ['docker-compose'] = wezterm.nerdfonts.linux_docker,
  ['fish'] = wezterm.nerdfonts.md_fish,
  ['gh'] = wezterm.nerdfonts.dev_github_badge,
  ['git'] = wezterm.nerdfonts.dev_git,
  ['go'] = wezterm.nerdfonts.seti_go,
  ['htop'] = wezterm.nerdfonts.md_chart_areaspline,
  ['kubectl'] = wezterm.nerdfonts.linux_docker,
  ['kuberlr'] = wezterm.nerdfonts.linux_docker,
  ['lazydocker'] = wezterm.nerdfonts.linux_docker,
  ['lazygit'] = wezterm.nerdfonts.cod_github,
  ['lua'] = wezterm.nerdfonts.seti_lua,
  ['make'] = wezterm.nerdfonts.seti_makefile,
  ['node'] = wezterm.nerdfonts.mdi_hexagon,
  ['nvim'] = wezterm.nerdfonts.custom_neovim,
  ['pacman'] = wezterm.nerdfonts.md_pac_man,
  ['paru'] = wezterm.nerdfonts.md_pac_man,
  ['powershell.exe'] = wezterm.nerdfonts.md_console,
  ['psql'] = wezterm.nerdfonts.dev_postgresql,
  ['pwsh.exe'] = wezterm.nerdfonts.md_console,
  ['ruby'] = wezterm.nerdfonts.cod_ruby,
  ['sudo'] = wezterm.nerdfonts.fa_hashtag,
  ['vim'] = wezterm.nerdfonts.dev_vim,
  ['wget'] = wezterm.nerdfonts.mdi_arrow_down_box,
  ['zsh'] = wezterm.nerdfonts.dev_terminal,
}

return {
  update = function(tab)
    -- Inspiration: @folke https://github.com/folke/dot/blob/master/config/wezterm/tabs.lua
    if tab.active_pane and tab.active_pane.foreground_process_name then
      foreground_process_name = tab.active_pane.foreground_process_name
      foreground_process_name = foreground_process_name:match('([^/]+)/?$') or
          foreground_process_name
    end
    local foreground_process_icon = icons[foreground_process_name] or ''
    return foreground_process_name ~= '' and foreground_process_icon or wezterm.nerdfonts.dev_terminal
  end,
}

Also modified from folke, an indicator if a pane is zoomed, which I also incorporated into my current tab bar. You can check out mine at https://github.com/joncrangle/wezterm/blob/master/tabs.lua where I also implemented the zoom indication feature.

Those are som great ideas. Maybe discuss it in the tabline repo instead, because I didnt see the message until now haha.


use keys returned from getTypeProps as refs

This might be heresy, but in one case I have nested refs inside of custom templates. When validating structs it iterates over all refs and will fail to call validate on my own refs (which are not tcomb generated components). I am open to other ways of achieving this.

Hi,
Thanks for your PR.
getInputs() uses the keys of the object returned by getTypeProps() as refs, what about?
const props = this.getTypeProps() // <= same implementation of getInputs()
for (const ref in props) {
  if (this.refs.hasOwnProperty(ref)) {
    ....
  }
}

Note. Both validate() and getValue() should be changed accordingly.

Ah I didn't see getTypeProps() but I figured it must exist somewhere. Thanks for pointing that out.

@gcanti I think this PR makes sense considering we should only be iterating over known props instead of all possible refs inside of a struct component. Does this look good to you?

Ah, yes. Sorry, I didn't notice your last commit yesterday, going to merge

Released in https://github.com/gcanti/tcomb-form/releases/tag/v0.7.9, thanks @velveteer


Azure Storage Blobs Sample issues

Section link1,link2:

Reason: The lack of  create blob code after 876 lines of code.
Suggestion:
Add
 await containerClient.CreateAsync();
after
var containerClient = new BlobContainerClient(ConnectionString, containerName);
@jongio for notification.

Close this issue, due to this issue had been fixed at commit.


Add entry for GOV.UK Accounts

We didn't have anything in our main developer docs for these.
Screenshots

I wonder if we should do this, though? Accounts are meant to be separate from "GOV.UK", including their apps. So by listing them here, we kind of saying "we" maintain them. We should probably check with @barrucadu or @huwd about what makes sense here - definitely think there should be a clear statement about accounts in here, somewhere.

@benthorner it was never clear to me that they are that separate. @richardTowers some steer from you perhaps please? Today's situation made for an interesting event. The entries here are pointers/harvest their repos anyway.

@benthorner maybe I can add entries to link to their manual at least from our Manual somewhere?

@chao-xian yup, I reckon we should have at least something in here that links to the other manual. I think it's really on the Accounts team to write that, though: we (all of us) don't currently have a shared understanding of the relationship between "GOV.UK" and "GOV.UK Accounts", why there's a separate manual for it, and what should go in each of them.

@chao-xian yup, I reckon we should have at least something in here that links to the other manual. I think it's really on the Accounts team to write that, though: we (all of us) don't currently have a shared understanding of the relationship between "GOV.UK" and "GOV.UK Accounts", why there's a separate manual for it, and what should go in each of them.

Heya, mmm, yeah this is a good debate to be having.
Ben is correct, the approach we've been taking is that we might be more akin to something like pay/verify in future and so we've been following their playbook on docs.
So we've got this repo: https://github.com/alphagov/govuk-account-team-manual (currently exceedingly sparse!)
which is up here: https://docs.account.publishing.service.gov.uk/#gov-uk-accounts-technical-documentation
A link sounds like a good idea. We may add / remove / change those repos so we'd need to maintain those links in GOV.UK docs.
:thinking: That said...@chao-xian isn't wrong. I do question sometimes how much we are actually a separate thing from GOV.UK right now, or when that separation might actually occur in a meaningful way.
Currently we're growing rapidly - we may rely on 2ndline / on call in some circumstance, and the Account system currently is not effectively isolated from GOV.UK services (we made compromises especially with the email alert system which draws them closer together than they might otherwise be...).
This will be resolved in time, as accounts broadens to take on multiple services.
In the meantime a link to our docs seems like a good idea and then I think @brucebolt, @bevanloon, @barrucadu, @richardTowers and I should get together on an Accounts ticket to clarify:

how integrated into GOV.UK this thing actually is during these prototype phases
what our support expecation is / might be in the future
given those, what we need docs to do and if they make more sense in a GDS model, or where GOV.UK folks might go looking for them?

That sound sensible to both of you @chao-xian / @benthorner?

Heya, mmm, yeah this is a good debate to be having.
Ben is correct, the approach we've been taking is that we might be more akin to something like pay/verify in future and so we've been following their playbook on docs.
So we've got this repo: https://github.com/alphagov/govuk-account-team-manual (currently exceedingly sparse!)
which is up here: https://docs.account.publishing.service.gov.uk/#gov-uk-accounts-technical-documentation
A link sounds like a good idea. We may add / remove / change those repos so we'd need to maintain those links in GOV.UK docs.
:thinking: That said...@chao-xian isn't wrong. I do question sometimes how much we are actually a separate thing from GOV.UK right now, or when that separation might actually occur in a meaningful way.
Currently we're growing rapidly - we may rely on 2ndline / on call in some circumstance, and the Account system currently is not effectively isolated from GOV.UK services (we made compromises especially with the email alert system which draws them closer together than they might otherwise be...).
This will be resolved in time, as accounts broadens to take on multiple services.
In the meantime a link to our docs seems like a good idea and then I think @brucebolt, @bevanloon, @barrucadu, @richardTowers and I should get together on an Accounts ticket to clarify:

how integrated into GOV.UK this thing actually is during these prototype phases
what our support expecation is / might be in the future
given those, what we need docs to do and if they make more sense in a GDS model, or where GOV.UK folks might go looking for them?

That sound sensible to both of you @chao-xian / @benthorner?

@huwd I was going by the user need of 2ndline helping Dana to deploy stuff on NYE. Which I reckon will help steer your conversations. I'll keep this PR open then. Thanks!

@huwd I was going by the user need of 2ndline helping Dana to deploy stuff on NYE. Which I reckon will help steer your conversations. I'll keep this PR open then. Thanks!

Ah ha, looks like this is already a source of friction then.
I've created a card, we've got planning today so I'll discuss with the team.
We might need to wait until later in the week when everyone is back.

Ah ha, looks like this is already a source of friction then.
I've created a card, we've got planning today so I'll discuss with the team.
We might need to wait until later in the week when everyone is back.

Sorry @chao-xian I know this one has lingered.
We're due to have a team chat tomorrow which should help sort out some of the bigger questions here.
Then @jonathanglassman has been getting all our team docs into shape. We'll ensure some of that makes it back into the govuk dev docs once we've had those chats.

Sorry @chao-xian I know this one has lingered.
We're due to have a team chat tomorrow which should help sort out some of the bigger questions here.
Then @jonathanglassman has been getting all our team docs into shape. We'll ensure some of that makes it back into the govuk dev docs once we've had those chats.

Had a chat with @chao-xian yesterday where he showed me how the GOV.UK Developer Docs worked. I've added the GOV.UK Account team manual and tech docs to the manual section. However, the preview of these is not working so I'll grab some help from the GOV.UK Account team. Have linked the trello card below as per GOV.UK Account team agile practice.
https://trello.com/c/hGrGtMzS/584-add-account-tech-docs-links-to-main-govuk-developer-docs

Had a chat with @chao-xian yesterday where he showed me how the GOV.UK Developer Docs worked. I've added the GOV.UK Account team manual and tech docs to the manual section. However, the preview of these is not working so I'll grab some help from the GOV.UK Account team. Have linked the trello card below as per GOV.UK Account team agile practice.
https://trello.com/c/hGrGtMzS/584-add-account-tech-docs-links-to-main-govuk-developer-docs


Revert "Add node shutdown"

Reverts aicers/review-web#133
Relates #170

Codecov Report
Attention: Patch coverage is 50.00000% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 51.48%. Comparing base (086f95d) to head (1709aeb).

Files
Patch %
Lines

src/graphql/node/control.rs
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #171      +/-   ##
==========================================
- Coverage   51.48%   51.48%   -0.01%
==========================================
  Files          65       65
  Lines       10839    10710     -129
==========================================
- Hits         5581     5514      -67
+ Misses       5258     5196      -62

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Since there are many PRs pending merge and this is a minor issue, I'll close it after making @sophie-cluml and @dayeon5470 aware.


Fix nillable finite type in array with array literal

Fixes #10940

Codecov Report

:exclamation: No coverage uploaded for pull request base (release-0.990.0@091afd7). Click here to learn what that means.
The diff coverage is 76.92%.

@@                Coverage Diff                 @@
##             release-0.990.0   #12730   +/-   ##
==================================================
  Coverage                   ?   59.86%
  Complexity                 ?      659
==================================================
  Files                      ?     2066
  Lines                      ?   101598
  Branches                   ?    12928
==================================================
  Hits                       ?    60822
  Misses                     ?    35589
  Partials                   ?     5187

Impacted Files
Coverage Δ
Complexity Δ

...alang/compiler/semantics/analyzer/TypeChecker.java
86.52% <76.92%> (ø)
0 <0> (?)

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 091afd7...1ff1d7d. Read the comment docs.


[Feature]: Add data download to data analysis portal.

Guidelines

[X] I agree to follow this project's Contributing Guidelines.

Description
People want to download the data from CKAN, we should contstrain how that happens to make a clean download.
Problem
No
Proposed Solution
Add a download button on the data_analysis portal. Make a warning message if people try to download data that isn't current.
Alternatives Considered
Ask people to download raw from data.ca.gov

https://github.com/Moore-Institute-4-Plastic-Pollution-Res/Microplastic_Data_Portal/commit/a72e082e9500bf5980c92fef54362a20fdb916c2


Update Composer dependencies (2022-01-12-21-13)

Visual regression test passed!
View the visual regression test report


Remove config

Closes #938.
TODO:

[x] remove fallbacks on env vars, as discussed with Trent
[x] run a validation/check on the config dict provided

GAS_LIMIT only used from env, should we keep it in the config dict?

@trentmc @mariacarmina Here's the gist of what I did here:

removed the Config class and object-based configuring, including config.ini
removed fallbacks to env variables when configuring Ocean. The reason for this is simple: users will be able to instantiate multiple Ocean objects i.e. ganache_ocean, mumbai_ocean, etc. and compare between them
Ocean class validates the config given to it for presence and types of all configuration options, no fallback, no guessing. Every Ocean class needs to be configured with all the options.
updated Example Config: I kept most of it in place but flattened the result into a simple dict. I think this logic it is valuable and we should not remove it. If we do, however, decide to remove it, we should do that in a separate PR. ExampleConfig now handles the "guessing and defaulting" part for users, if they wish to use it, while the Ocean class does not care about that. It requires everything fully and properly configured. I believe this is a good balance between UX and thoroughness.
fixed a bunch of tests, refactored all places where config was involved etc.

The readme tests were skipped when I started this issue. I believe we should have a follow-up issue to fix all the necessary readme tests and other skipped tests as needed.
Pending discussion:
I have noticed that the GAS_LIMIT option inside the config was not actually used in gas calculations. I suggest we either work it into the calculation some way, or drop it from the config dict.


[FEA] Optimize Shuffle coalesce performance when handling very small sized partitioned batches

Is your feature request related to a problem? Please describe.
Currently shuffle will partition the columnar batch into partitioned batches.
https://github.com/NVIDIA/spark-rapids/blob/65ec25d85503484874d3b157d4e2775159187436/sql-plugin/src/main/scala/com/nvidia/spark/rapids/GpuHashPartitioningBase.scala#L35-L47
And those partitioned batches will go through GpuColumnarBatchSerializer in GpuShuffleExchangeExec to serialize it.
https://github.com/NVIDIA/spark-rapids/blob/65ec25d85503484874d3b157d4e2775159187436/sql-plugin/src/main/scala/org/apache/spark/sql/rapids/execution/GpuShuffleExchangeExecBase.scala#L341-L344
https://github.com/NVIDIA/spark-rapids/blob/branch-24.04/sql-plugin/src/main/scala/com/nvidia/spark/rapids/GpuColumnarBatchSerializer.scala#L170-L174
And the deserialization path, using shuffle hash join as an example, it will first go through the path: deserialization -> concat+load -> filter (e.g. null filtering masking or RequireSingleBatch) to meet CoalesceSizeGoal.
https://github.com/NVIDIA/spark-rapids/blob/65ec25d85503484874d3b157d4e2775159187436/sql-plugin/src/main/scala/com/nvidia/spark/rapids/GpuShuffleCoalesceExec.scala#L213-L218
https://github.com/NVIDIA/spark-rapids/blob/65ec25d85503484874d3b157d4e2775159187436/sql-plugin/src/main/scala/com/nvidia/spark/rapids/GpuShuffledHashJoinExec.scala#L295
If the shuffle write total size is not big enough, saying 30MB with 8 batches while we had 2000 partition number, shuffle write side will generate 8 * 2000 total batches to go through process above. And each batch is ~ 15.36 KB. From nsys traces below, there're clear bubbles for both shuffle write and read side.
The shuffle write does have some overlap with GPU computation but since those small sized batch serialization making a long tailing time after GPU finished all his work.

And the below shown that serialization on each batch will all write offset as well as data but with lots of overhead (the yellow box "serialize batch" besides the green box "write sliced"). And with lots of small batches, the overhead of serialization getting significant while the data write itself is not that much. We should figure out a way to improve this performance issue.

On the other hand in deserialization, the first stream batch was blocked until we met the build goal. And lots of time is spent in read head + read batch with lots of iterations.

Describe the solution you'd like
There're two ideas in general could help this:

Introduce a write side coalesce mechanism before flushing out those small batches. To achieve this goal, we need to maintain some buffer to threshold the batch serialization other than immediately flushing them out.
Optimize shuffle coalesce performance by some of the CPU work onto GPU.

Describe alternatives you've considered
AQE may not help too much as we still need the optimize both shuffle write and read with those small batches with the same shuffle partition number for shuffle write.

The GPU coalesce on the read side is tracked by #10402.


Add "Select All Results" in the "Select All" drop-down

Storage Explorer Version: 1.2.0
Platform/OS Version: Windows
Architecture: ia32
Regression From:
I have a large table with several partitions. As a one-time maintenance operation, I have to delete ~250K rows with the same partition key
Azure Table API doesn't support batch delete, despite the feature being requested since 2012.
It would be VERY helpful if Azure Storage Explorer implemented this by allowing the user to select all query results, even the ones that have not been cached. Currently, I'd have to "Select All Cached" 250 times and click "Delete" 250 times - a total of 750 clicks.

Thanks for the suggestion. We'll look into this.


Update scala-compiler, scala-library, ... to 2.12.13

Updates

org.scala-lang:scala-compiler
org.scala-lang:scala-library
org.scala-lang:scala-reflect

from 2.12.12 to 2.12.13.
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 me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-lang" } ]

labels: library-update, semver-patch

I think it's failing because of this https://github.com/scoverage/scalac-scoverage-plugin/issues/305

Codecov Report

Merging #1675 (34dd5e3) into master (da67318) will decrease coverage by 0.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1675      +/-   ##
==========================================
- Coverage   86.74%   86.66%   -0.09%
==========================================
  Files          59       59
  Lines        2399     2399
  Branches      142      136       -6
==========================================
- Hits         2081     2079       -2
- Misses        318      320       +2

Impacted Files
Coverage Δ

...re/shared/src/main/scala/io/circe/KeyDecoder.scala
97.29% <0.00%> (-2.71%)
:arrow_down:

...rc/main/scala/io/circe/numbers/BiggerDecimal.scala
85.96% <0.00%> (-1.17%)
:arrow_down:

.../core/shared/src/main/scala/io/circe/Decoder.scala
85.07% <0.00%> (+0.26%)
: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 da67318...34dd5e3. Read the comment docs.


Could not find androidx.compose.ui:ui-tooling-preview:1.6.10-beta01

Describe the bug
Just migrating the project from 1.6.2 to 1.6.10-beta01 triggered errors who makes compilation impossible
Affected platforms

Android,
compilation on desktop is ok

Versions

Kotlin version*: 2.0.0-RC1
Compose Multiplatform version*: 1.6.10-beta01

> Could not resolve all files for configuration ':composeApp:debugRuntimeClasspath'.
   > Could not find androidx.compose.ui:ui-tooling-preview:1.6.10-beta01.
     Searched in the following locations:
       - https://dl.google.com/dl/android/maven2/androidx/compose/ui/ui-tooling-preview/1.6.10-beta01/ui-tooling-preview-1.6.10-beta01.pom
       - https://repo.maven.apache.org/maven2/androidx/compose/ui/ui-tooling-preview/1.6.10-beta01/ui-tooling-preview-1.6.10-beta01.pom
       - https://maven.pkg.jetbrains.space/public/p/compose/dev/androidx/compose/ui/ui-tooling-preview/1.6.10-beta01/ui-tooling-preview-1.6.10-beta01.pom
     Required by:
         project :composeApp

Additional context
    commonMain.dependencies {
        implementation(compose.components.uiToolingPreview)

}

implementation(compose.components.uiToolingPreview)

compose.components.uiToolingPreview doesn't contain links to androidx.compose.ui:ui-tooling-preview - I checked that, and it works for Android/Desktop.
Maybe you have another dependency on androidx.compose.ui:ui-tooling-preview somewhere? It is a part of Jetpack Compose, which has its own versioning, and doesn't have 1.6.10-beta01. For Jetpack Compose you should use a different version (the latest version is 1.6.6).
It is a guess, but you might use a template https://kmp.jetbrains.com/, which generates this:
[versions]
...
compose = "1.6.5"
compose-plugin = "1.6.2"
...

compose here is Jetpack Compose
compose-plugin is Compose Multiplatform
We will fix this confusion here

Thanks for your answer, and sorry for wasting your time. In my build.gradle.kts, under the Android block, I had:
dependencies {
    debugImplementation(libs.compose.ui.tooling)

and in my catalog, compose-tooling is defined as:
compose-tooling = { module = "androidx.compose.ui:ui-tooling", version.ref = "compose" }
So yeah problem was that . Thanks again

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Change value modifier to make StandardLockService extendable for future extension

This pull request changes the modifiers of standard lock service to protected for the case where only certain parts of the lock service need to be rewritten.
An example is this pr in the Keyspace extension.
https://github.com/liquibase/liquibase-amazon-keyspaces/pull/8

@nvoxland Thanks, I will work on getting this passed the CI and we can go from there.

@nvoxland  Also this PR needs the "SafeToBuild" tag for the CI.

@nvoxland It looks like the CI thinks that this pr is not safe to build...

@nvoxland  What do I need to do to make Liquibase pro stable?

@nvoxland What do I need to do to make Liquibase pro stable? or is this even needed for this to merge in?

Hello, @Cliftonz! There is nothing you can do from your side to help with the stability of the Liquibase Pro build. I am kicking off the functional and test harness test suites manually; when those pass, I'll mark this as ready to merge. Thank you for the PR and for your patience!


Wrong Edges locations

Describe the Bug
When the node location is very high(like 1668230) The calculation of the arrows position is not well calculated (there is an offset)
See attached example

[ ]

Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
Create Node with x = 1668230, y = 1668230 and Try adding an arrow that Coming out from the same NODE
Expected behavior
That the arrows came out of the circle of the NODE
Screenshots or Videos

Platform

OS: Windows
Browser: Chrome
Version: 10.2.1

Additional context
No response

same here

@linoytzarfatyMicrosoft I can't reproduce this issue. I created a codesandbox for this: https://codesandbox.io/s/happy-lamarr-dviclh?file=/src/initial-elements.js. Please re-open if the issue still persists.
@myatniy this is a different issue. You need to remove the setEdges call in your timeout. You only want to update the nodes.


ENH: Data manipulations to facilitate modeling and registration: smoothing & subsampling

Resolves: #108

@oesteban This PR could be useful for Gaussian processes. Is it ready for us to take over and work on it?

This would need to add tests. I'm preparing some data to do that.

This PR carries too much weight with it. I'm going to split it in two (potentially three).


style: putting 4 shadows inputs in one div

🛠 Fixes Issue
Closes #257
👨💻 Changes proposed
Updating the style of the box shadow tool. The 4 inputs are now presented inside a single box now
✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers
I also renamed "box-shodow-inputs" to "box-shadow-inputs"
I remain available if my PR doesn't respect your specs
📷 Screenshots

Why ?

Why ?

Ah, I didn't mean to close this without a comment, but I only accept PRs from people who were assigned to work on the issue.
So I will be merging the PR of someone who was actually assigned to work on the issue, thanks for wanting to contribute tho.
But the rules were set, and you didn't follow them

Why ?

Ah, I didn't mean to close this without a comment, but I only accept PRs from people who were assigned to work on the issue.
So I will be merging the PR of someone who was actually assigned to work on the issue, thanks for wanting to contribute tho.
But the rules were set, and you didn't follow them

I asked to get it :/
https://github.com/Dun-sin/Code-Magic/issues/257#issuecomment-1272563895

Why ?

Ah, I didn't mean to close this without a comment, but I only accept PRs from people who were assigned to work on the issue.
So I will be merging the PR of someone who was actually assigned to work on the issue, thanks for wanting to contribute tho.
But the rules were set, and you didn't follow them

I asked to get it :/ #257 (comment)

I think there's a difference between "I'm taking it" and "can I take it?". This project isn't maintained by a robot, but a human being.
I also didn't assign you to it, you asking doesn't justify being assigned

Why ?

Ah, I didn't mean to close this without a comment, but I only accept PRs from people who were assigned to work on the issue.
So I will be merging the PR of someone who was actually assigned to work on the issue, thanks for wanting to contribute tho.
But the rules were set, and you didn't follow them

I asked to get it :/ #257 (comment)

I think there's a difference between "I'm taking it" and "can I take it?" One is a command and one is a question. This project isn't maintained by a robot, but a human being. I also didn't assign you to it, you asking doesn't justify being assigned

Sad :/


Run the tests nightly

Codecov Report

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

@@             Coverage Diff              @@
##            master     #607       +/-   ##
============================================
- Coverage   100.00%   28.62%   -71.38%
============================================
  Files           60       60
  Lines         4046     4046
  Branches       266      266
============================================
- Hits          4046     1158     -2888
- Misses           0     2847     +2847
- Partials         0       41       +41

Impacted Files
Coverage Δ

tests/mock_vws/test_query.py
0.00% <0.00%> (-100.00%)
:arrow_down:

tests/mock_vws/test_usage.py
0.00% <0.00%> (-100.00%)
:arrow_down:

tests/mock_vws/test_add_target.py
0.00% <0.00%> (-100.00%)
:arrow_down:

tests/mock_vws/test_date_header.py
0.00% <0.00%> (-100.00%)
:arrow_down:

tests/mock_vws/test_target_list.py
0.00% <0.00%> (-100.00%)
:arrow_down:

tests/mock_vws/test_invalid_json.py
0.00% <0.00%> (-100.00%)
:arrow_down:

tests/mock_vws/test_update_target.py
0.00% <0.00%> (-100.00%)
:arrow_down:

tests/mock_vws/test_content_length.py
0.00% <0.00%> (-100.00%)
:arrow_down:

tests/mock_vws/test_get_duplicates.py
0.00% <0.00%> (-100.00%)
:arrow_down:

tests/mock_vws/test_target_summary.py
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 42 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 0aee30e...9883ced. Read the comment docs.


[DevOps] Why DevOps Need Application Release Orchestration

Proposed title of article
Why DevOps Need Application Release Orchestration
Proposed article introduction
DevOps coordination is focused on the automation of a large number of tasks that can run at the same time. Therefore, We can say that, without release orchestration, DevOps cannot truly and completely be implemented.
Release orchestration simplifies and automates manual, repetitive tasks, contributing to lower costs, reducing time-to-market, and greater transparency in the software development and delivery cycle.
In this post, will discuss why Application Release Orchestration (ARO) matters, who it benefits, and top ARO tools. We will also discuss No-Code DevOps Orchestration and how it promotes delivery.
Key takeaways

Why ARO matters.
Who does ARO benefit?
No-Code DevOps Orchestration
Top ARO tools.

Article quality
My article covers additional topics and is more in-depth.
References
Please list links to any published content/research that you intend to use to support/guide this article.
N/A

Good afternoon and thank you for submitting your topic suggestion.
Your topic form has been entered into our queue and should be reviewed (for approval) as soon as a content moderator is finished reviewing the ones in the queue before it.

Good topic @Snzovi - approved. Make sure your article is unique and adds to what is already available in official documentations and other sources.


PEP8 formatting fixes and merge conflict fixes

PEP8 compliant formatting.
Also fixed an issue in the Logcat() function that was caused by an earlier merge conflict resolution gone wrong.

Coverage remained the same at 42.691% when pulling f500b60d44f2efb15a9b8bd471dcd04e7c6b25e2 on greateggsgreg:PEP8-format-fixes into 0b15f4fc69c8b9ed87a82964fd6d2eed8d637f21 on google:master.

That's one way to get your diff count up :)


Expose ignored projects in Web UI

We expect that users/orgs adding ocaml-ci will select specific projects to build, what often happens is someone installs the ocaml-ci GH app and enables build for all repositories. We warn about this situation in the logs and don't build everything.
The experience would be better if we exposed the projects we have discarded and clearly noted why they are not building.

Duplicate of your own #392 :computer: .


Simplify mappings from ACT to SPS for SIPROTEC 5 devices

Currently we use an option to ignore the CDC check for preconfigured attributes.
https://github.com/OpenEnergyTools/scl-lib/blob/06423179eeb18d2bb0d411fee280ec44dbace514/tExtRef/subscribe.ts#L19
However this means that subscriptions are clumsy within the subscriber plugin (the user has to go to settings and select this option which set the option on the function and then remove it later):

The most common use case seems to be subscribing an SPS to an ACT which is very common in SIPROTEC 5 devices.
We short circuit other tests as part of subscription if this option is set:
https://github.com/OpenEnergyTools/scl-lib/blob/06423179eeb18d2bb0d411fee280ec44dbace514/tExtRef/doesFcdaMeetExtRefRestrictions.ts#L55-L58
I think that the CDCs are badly structured to cause this issue but I don't see any TISSUES discussing this.
Do we have some better options?

I could perhaps check the CDC within the subscriber plugin and pass the flag rather than requiring the user to set it
Ideally though I would delegate this to the SIPROTEC 5 plugin https://github.com/danyill/oscd-subscriber-lb-siemens The difficulty with this is that historically it assumes the first event does something but perhaps it could work without this. Then we could scope this improvement more narrowly to SIPROTEC 5 (maybe other manufacturers solve this differently)?
Maybe the subscribe function in scl-lib could accept a list of acceptable cdc matches

Ideally though I would delegate this to the SIPROTEC 5 plugin https://github.com/danyill/oscd-subscriber-lb-siemens The difficulty with this is that historically it assumes the first event does something but perhaps it could work without this. Then we could scope this improvement more narrowly to SIPROTEC 5 (maybe other manufacturers solve this differently)?

I wonder whether it would be a mis-use of our API to look at an edit, know it would fail (because of the CDC mismatch) and then add our own edit in using the Siemens plugin.

When we discussed at our later standup we agreed that an ACT <> SPS and vice versa could be an option to the subscribe function in scl-lib and also in the various matching functions.
I'll try to prepare a PR on this over the next little while.


Integrate 2018 EIA860 data

The EIA has released the official 2018 data for Form 860, so we need to integrate it.

Hi there! I have a super basic question comment here. Are we able to use 2018 EIA 860 and EIA 923 data yet? I get the following error when I try so perhaps my software is out of date?

Hi yes! It's been integrated into the codebase but hasn't been released yet. We've been sidetracked with customer contract work. If you're installing from the github repo, you can pull the latest version and it should work.  If you're installing from conda/pip then it'll be a little while before we get it released.

Thanks for the head's up. I had done pip so that makes sense. I hope all is well!


Unable to build Example - Exit code 1

Using the latest version of Xcode Release we are unable to build the example projects.
We tried

pod install
pod update
But we still get the same error:

Linting Swift files at path ..
No lintable files found at path '..'
Command /bin/sh failed with exit code 1

more detailed error:
Loading configuration from '../.swiftlint.yml'
configuration error: 'unused_closure_parameter' is not a valid rule identifier
configuration error: 'weak_delegate' is not a valid rule identifier
configuration error: 'large_tuple' is not a valid rule identifier
Valid rule identifiers:
trailing_newline
opening_brace
custom_rules
empty_count
comma
colon
force_cast
type_name
trailing_semicolon
force_try
force_unwrapping
function_body_length
nesting
variable_name
conditional_binding_cascade
function_parameter_count
operator_whitespace
control_statement
legacy_constant
line_length
return_arrow_whitespace
trailing_whitespace
closing_brace
statement_position
type_body_length
todo
legacy_constructor
cyclomatic_complexity
missing_docs
valid_docs
file_length
leading_whitespace
legacy_cggeometry_functions
Linting Swift files at path ..
No lintable files found at path '..'
Command /bin/sh failed with exit code 1

It looks like its the requirement ot install Swiftlint in order to use the library.   can we maybe shift to using the cocoapods version of the module so it doesn't require installing a seperate program into the mac to use this library?

When I update the build script to use the syntax specified on the swift lint page - it still seems to create an error.  I would log a bug there- but there are over 150+ issues at the moment.

closing out old issues, feel free to reopen


calc() is being evaluated and the result is being outputted to CSS instead of the calc() function being outputted to CSS

NPM version (npm -v): 3.10.10
Node version (node -v): 6.12.0
Node Process (node -p process.versions): { http_parser: '2.7.0',
node: '6.12.0',
v8: '5.1.281.108',
uv: '1.15.0',
zlib: '1.2.11',
ares: '1.10.1-DEV',
icu: '58.2',
modules: '48',
openssl: '1.0.2m' }
Node Platform (node -p process.platform): orm
darwin
Node architecture (node -p process.arch): x64
node-sass version (node -p "require('node-sass').info"): node-sass	3.13.1	(Wrapper)	[JavaScript]
libsass  	3.3.6	(Sass Compiler)	[C/C++]
npm node-sass versions (npm ls node-sass): 3.13.1

We have a Sass function and mixin that responsively changes the font size based on viewport width. It does this by calculating a constant rate of change and then outputting a calc() value that is essentially calc(Xvw + Yrem). The key is that the css is a calc() with the vw in it so that it can constantly be evaluating the font size based on the viewport width and changing accordingly. Our function worked great when we were using node v4.2.2 (npm v3.8.6) but not on node version 6 or 7. Instead, the calc() will be evaluated and then a static value will be outputted to the stylesheet (say font-size: 4.25rem; instead of the desired font-size: calc(1.4vw + 2.64rem); or whatever).
I would expect the calc() to be outputted to the stylesheet instead of being evaluated and then the result being outputted.

Can you reproduce this with the latest, you're on an old version

This should be resolved with #2071

Hello,
I tried updating with the latest version and it still wasn't working. This module was being used by another called gulp-sass and when I updated this package it seemed to fix it. node-sass is now outputting calc() correctly now with node 6 and 7.
Thank you.

@tleifj  how did you fix this ? I am still having it

This shouldn't be the case in node-sass 4.8
On Fri., 20 Apr. 2018, 1:31 am James Campbell<EMAIL_ADDRESS>wrote:

@tleifj https://github.com/tleifj how did you fix this ? I am still
having it
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sass/node-sass/issues/2144#issuecomment-382779974,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAjZWBFY02LxcBenha1qRC1-Itj0o40Nks5tqK3SgaJpZM4QVg5t
.


Square.OkHttp3.UrlConnection 4.9.1

@moljac
Now that PRs for Square.OkHttp3 and Square.OkIO have been merged, could you please update Square.OkHttp3.UrlConnection ?
Thanks!

@moljac Could you please review and merge this PR when you have the time ? Thanks


defaultHeader is 'null' for StackNavigator screens nested in a TabNavigator

I have following structure:
TabNavigator

StackNavigator 1
StackNavigator 2
...

On each StackNavigator I have configured some header styles which are applied and visible:
navigationOptions: {
  header: {
    tintColor: 'red',
  },
},

For certain screens within a StackNavigator I want to change the default header but I can't extend the default because the default is null:
static navigationOptions = {
  header: (navigation, defaultHeader) => {
    console.log(defaultHeader) // null
    return {
      ...defaultHeader,
      tintColor: 'blue', // This is not applied
    }
  }
},

Can you post the full code where you're declaring the navigators?

Root Navigator
const Tabs = TabNavigator({
  Home: {
    screen: TabHome,
    path: 'home',
    navigationOptions: {
      tabBar: navigation => {
        return {
          icon: ({ tintColor, focused }) => {
            return <TabIcon icon={ 'home' } color={ tintColor } />
          },
        }
      }
    },
  },
...
}, {
  tabBarOptions: {
    style: styles.tabBar,
    showLabel: false,
  },
})

Home Tab Navigator
const TabHome = StackNavigator({
  Home: {
    screen: HomeScreen,
    navigationOptions: {
      header: {
        style: { backgroundColor: 'yellow' },
      },
    },
  },
...
}, {
  headerMode: 'screen',
})

Home screen component
static navigationOptions = {
  header: (navigation, defaultHeader) => ({
    ...defaultHeader,
    style: { backgroundColor: 'red' },
  }),
}

I worked around this issue for now by removing the navigateOptions from the screen completely and configuring everything directly in the StackNavigator as follows:
const TabHome = StackNavigator({
  Home: {
    screen: HomeScreen,
    navigationOptions: {
      header: (navigation, defaultHeader) => ({
        ...defaultHeader,
        style: {
          backgroundColor: 'red',
        },
      }),
    },
  },
...
}, {
  navigationOptions: {
    header: {
      style: {
        backgroundColor: 'yellow',
      },
    },
  },
})

Since there hasn't been any activity on this in a while I'm going to close it (and since a lot has changed). If it still exists please open a new issue. Thanks!


[Feature]: the ability to disable switching the focus of a document to a DropdownMenuItem on hover

Describe the feature
the ability to disable switching the focus of a document to a DropdownMenuItem on hover
use cases:

The search field embedded in the list constantly loses focus when the user hovers over the list to, for example, scroll through it and view the results

When the height of the list is limited and the last element is not fully displayed, then when you move the cursor over the list, the last element focuses, which causes the list to twitch back and forth in an ugly way

Additional information

[ ] I intend to submit a PR for this feature.
[ ] I have already implemented and/or tested this feature.

@masawik  I'm not sure I understand the ticket. Can you please elaborate more, or perhaps add a stacklbitz to explain what are you looking for?

Closing as stale


Rename aggregate_metric_spec col schema to spec

schema is a reserved word in SQL, spec isn't.
@jamiesnape @cpatrick  PTAL

LGTMIAWTIH

Travis may never be happy, even if he should be.

The push check passing is good enough for me.

I take it back, my apologies Travis.


Add a Middlewares algebra that allows to modify Endpoint

Addresses https://github.com/endpoints4s/endpoints4s/issues/794 and part of https://github.com/endpoints4s/endpoints4s/issues/844
The Middlewares algebra defines methods to change the Request[A] and Response[B] in an Endpoint[A, B].
It also adds methods to add headers to Request and Response and add queryString to Request.
While the Middlewares algebra is separate from the Endpoints algebra, all Endpoints interpreters implements it. To do so, several interpreters had to be changed, especially the client ones in order to be able to retrieve the Request and Response from an Endpoint. For instance in akka-http-client the type of Endpoint is changed from A => Future[B] to:
case class Endpoint[A, B](request: Request[A], response: Response[B]) extends (A => Future[B]) {
  def apply(a: A): Future[B] = ...
}

The AuthenticatedEndpoints algebra demonstrates a usage of these new capabilities. The client and server interpreters for this algebra are independent from the underlying HTTP framework and only uses the new operations provided by Middlewares. Since AuthenticatedEndpoints provides similar features as the BasicAuthentication algebra, it has been deprecated.

Thanks for making this PR @mleclercq. Can you maybe process the review comments? Otherwise I also have some time tomorrow (or otherwise next week) to continue this PR.

Sorry for the delay for responding to the comments. I had a couple of very busy weeks...
I've rebased my branch on @agboom ones which includes the change proposed for akka-client (type Request[A] = A => Future[HttpRequest]).
I did the following changes:

merge the Middleware algebras in the "main ones".
rename the withHeaders/QueryString to addHeaders/QueryString as suggested by @agboom.

Thank you @mleclercq!
This PR introduces several important changes, so we need to make sure this is the right way to go :)
A couple of things that I’m not super happy with:

as mentioned in a comment above, the fact that we don’t control whether to call or not the logic of the endpoint to which we apply the middleware,
the fact that we have three traits AuthenticatedEndpoints, AuthenticatedEndpointsServer, and AuthenticatedEndpointsClient (btw why don’t we need an AuthenticatedEndpointsOpenapi?)

maybe its possible to move everything in AuthenticatedEndpoints (that would imply using the same concrete type for UsernamePassword)

The operations you have introduced to manipulate requests and responses remember me the discussion in #47 (so, it also solves a problem we had for some time).
Another thing middleware should help us achieve is to implement #104. We should try to implement a “telemetry” middleware, as another validation.

@mleclercq I have implemented my suggestion here, let me know what you think: https://github.com/mleclercq/endpoints/compare/middleware-take-two...julienrf:just-authenticated-endpoints

Hi,
Just a heads up that I have made some progress on my branch, I have just pushed a commit that introduces a way to partially provide the logic of endpoints (such as handling authentication things): https://github.com/julienrf/endpoints/commit/1a5d48475c08c591b27230993d244c69386179e6
I believe this is going in an interesting direction (you can see how I could re-implement the basic authentication as a middleware). I still need to implement it for all the interpreters (so far I only did OpenAPI and play-client).
Please let me know what you think about it. Maybe it’s introducing unnecessary complexity?

Great to see some progress here! I don't think the Middleware approach makes it much more complicated. And it's nice that we can move some things to the algebras that used to live in the interpreters.
I had to look twice to understand how we can have a client/server implementation in the algebra (since I saw the usage before the definition) but the ScalaDoc is pretty clear. Also I think it is important to note that using Middlewares is not required. And we currently have quite complicated code because of a lack of 'middlewares/composability'.
A minor point. I think it would be nice if Credentials and BearerToken to have a shared trait so you can just say an endpoint with auth.

Hello! I managed to get everything implemented. You can see the result here: https://github.com/endpoints4s/endpoints4s/compare/master...julienrf:just-authenticated-endpoints
There are still a few things that are missing (tests, examples, and documentation). Also, some things to polish.
More importantly, middlewares are currently made of “synchronous” functions (ie, they can’t return a Future, and they can’t return an Effect in the http4s interpreter). I think supporting asynchronous functions would be doable, but we should also support “effectful” functions in http4s, ideally, which would be a bit more complex to do.

Since the diff is quite large, here are the main highlights:

It is now possible to implement exactly what we used to implement in the BasicAuthentication module, but without having to add any code to the interpreters: everything is now achieved at “the algebra level”, by using middlewares.
Given an endpoint of type Endpoint[Int, String], you can call withBasicAuth() on it to get an Endpoint[(Int, Credentials), Option[String]]. That resulting endpoint carries authentication credentials in its request, and may return a Forbidden response in case the credentials don’t grant access to the underlying resource.
The withBasicAuth method is defined here:def endpointWithBasicAuth[A, B, AC](
    endpoint: Endpoint[A, B],
    realm: Option[String] = None,
    unauthorizedDocs: Option[String] = None,
    forbiddenDocs: Option[String] = None
)(implicit tupler: Tupler.Aux[A, Credentials, AC]): Endpoint[AC, Option[B]] = {
  // `endpoint` enriched with request headers and additional error responses
  val enrichedEndpoint: Endpoint[(A, Option[Credentials]), Either[Either[Unit, Unit], B]] =
    endpoint
      .mapRequest(_.addHeaders(basicAuthHeader))
      .mapResponse(response =>
        unauthorizedResponse("Basic", realm, unauthorizedDocs)
          .orElse(forbiddenResponse(forbiddenDocs))
          .orElse(response)
      )
  // add a middleware that takes care of the authentication part
  enrichedEndpoint
    .withMiddleware[AC, Option[B]](authenticationMiddleware)
}

What it does is that it first transforms the given Endpoint[A, B] by adding an Authorization header to its request, and by adding alternative response statuses (Unauthorized and Forbidden).
Then, that modified endpoint is plugged to a middleware that provides the client and server logic around the initial Endpoint[A, B].
The middleware is defined here:private def authenticationMiddleware[A, B, C, `(A, C)`](implicit
    tupler: Tupler.Aux[A, C, `(A, C)`]
): Middleware[(A, Option[C]), Either[Either[Unit, Unit], B], `(A, C)`, Option[B]] =
  Middleware(
    serverAction = (aAndCredentials: (A, Option[C])) => {
      val (a, maybeCredentials) = aAndCredentials
      maybeCredentials match {
        case Some(credentials) =>
          // The request contains credentials: invoke the logic of the
          // resulting endpoint, after some transformation
          Middleware.Continue(tupler(a, credentials))
        case None =>
          // No credentials in the request headers: bypass the resulting
          // endpoint’s logic by immediately returning an Unauthorized response
          Middleware.Bypass(Left(Left(())) /* Unauthorized */ )
      }
    },
    fromNewResponse = {
      case Some(b) => Right(b)
      case None    => Left(Right(())) // Forbidden
    },
    fromNewRequest = aAndCredentials => {
      val (a, credentials) = tupler.unapply(aAndCredentials)
      // Always provide the credentials in requests
      (a, Some(credentials))
    },
    toNewResponse = {
      case Right(b)        => Some(b)
      case Left(Right(())) => None
      case Left(Left(()))  => throw new Exception("Unexpected status 'Unauthorized'")
    }
  )

The serverAction defines what server interpreters should do when they receive requests. If there are no credentials the Unauthorized response is returned (without even going through the logic provided for the endpoint). Otherwise, the credentials are added to the request information. So, it transforms the information carried by requests from (A, Option[Credentials]) to (A, Credentials). The parameters fromNewResponse, fromNewRequest, and toNewResponse define how to map the requests and responses of the resulting endpoint to the requests and responses of the so-called “enriched” endpoint. Here, we say that a response containing None in the resulting endpoint should be mapped to the Forbidden response, for instance.

In conclusion, I still want to conduct a few more experiences but so far I am convinced that the changes proposed by @mleclercq should be accepted. As a first step, we could have a PR that only introduces the operations for transforming requests and responses (without the AuthenticatedRequests algebra, which will be superseded by the middlewares in my branch). Does that make sense?

Great work @julienrf! Can you explain to me why we need fromNewResponse and toNewResponse? I don't understand yet why we need to have both directions.
Other than that I think two PRs would be good. The mapRequest and mapResponse would already be very useful for us. That said, I also don't mind to review this as a whole. There are already 4 developers that have been involved and could likely do a review. I don't mind taking the time and review the whole PR next week. That said, if we want more time to test/verify the middleware approach we can split it up into two PRs.
In any case I am willing to help and have some time next week.

Can you explain to me why we need fromNewResponse and toNewResponse?

fromNewResponse is used by servers to transform the response provided by the user logic into the response of the oiriginal endpoint.
toNewResponse is used by clients to transform the response send by the server to the new response type.
I think it will be easier to introduce things gradually, so I am in favor of doing separate PRs. Unfortunately, I won’t have much time in the next two months, so any help is appreciated. Maybe one of you @bmeesters or @mleclercq could create a PR introducing mapRequest and mapResponse?

Ah yes, that explanation for fromNewResponse and toNewResponse makes sense.
If you want a shot @mleclercq at the transformations just let me know. Otherwise I will start on it next week.


Added a Stock Videos Category

Made a new category for stock videos and added 12 websites where you can find the same.
@markodenic can you please merge this pull request  as this is my first pr and I'm really excited. If you want some changes then please suggest me. One more request, Sir can you please label the issue with hacktoberfest and hacktoberfest-accepted as I have participated in World's largest open source festival and we are supposed to submit 4pr. I will always be obliged to you for this.
Thanks

@breakerh Sir, its an request can you please label this one as hacktoberfest and hacktoberfest-accepted. As I have participated in an open source festival and we have to make 4 successfull PRs with this label.

Thank you so much @breakerh


How to attach a specific Managed Disk ?

Hello guys, just wondering how can i attach a specific managed disk (for example created from an old VHDs disk).
I'm migrating my clusters to the last ACS release and i want to migrate my application disks from old VHDS to Managed Disks and then specify into the deployment Yaml file, my managed disk for that application (i don't need a dynamic provisioning for those Statefull applications).
Used to do :
 - azureDisk:
          diskURI: https://integrationk8s.blob.core.windows.net/jenkins-master/example.vhd
          diskName: jenkins-master-data

Since we don't have any URI with managed Disks, i'm really not sure how to do it.
Thanks !

@anhowe is this a supported (or available, but unsupported) workflow?

Should be great to handle it for update scenarios :D

@theobolo static provisioning for azure disk does not support managed disk, you could only use existing azure unmanaged disk in unmanaged VM

@theobolo I was wrong, actually k8s already support attach a specifc manage disk, below is an example:
https://raw.githubusercontent.com/andyzhangx/Demo/master/linux/azuredisk/nginx-pod-azuredisk-static-mgrdisk.yaml
For detailed steps, you could refer to following link under Static Provisioning
https://github.com/andyzhangx/Demo/tree/master/linux/azuredisk

@theobolo have you tried that, is it ok to close it?

Oups Sorry i was in vacation ... I gonna try this as soon as possible !


V1.0.0

Pull Request Test Coverage Report for Build 73

26 of 26 (100.0%) changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+3.3%) to 90.222%

Totals

Change from base Build 31:
3.3%

Covered Lines:
127

Relevant Lines:
131

💛  - Coveralls

Approved

Approved


Drag the file to open invalid/Cut operation invalid

What's the Problem?

I want to open a file by drag it to some software, but it can't open, most of the softwares support drag and open the files, so I hope  this problem could be solved.
By the way, the cut file operations cannot be performed, no matter the shortcut or right-click, which is pretty strange.

Solution/Idea
fix the problem of draging and open files, and fix the invalid cut operations.
Alternatives
None
Priorities

Capability
Priority

This proposal will allow developers to accomplish W
Must

This proposal will allow end users to accomplish X
Should

This proposal will allow developers to accomplish Y
Could

This proposal will allow end users to accomplish Z
Won't

Files Version
版本: <IP_ADDRESS> OS Version: 10.0.19043.1526
Windows Version
Windows 10 21H1 19043.1526
Comments
No response

What file types are you draging to what program.
You will need to upload the logs files directly to github. You can get them in files settings > about > open log folder.

for exampe, to drag a .xls file or a .dbf file to Microsoft Excel 2019, Excel acts like I didn't drag anything to it, and by the way, when I drag the file, there is a red forbidden Icon occured, please see the screenshot.

I'm sorry, but I didn't find the position to upload the logs files, because I'm a newcomer.

Have you tried draging it around the window a bit before going to excel it sometime is slow and it beeing worked on. Files can't be moved/copied into the same folder it's in that why it denied in your screenshot.

There has been some work recently on this in the preview version https://files.community/download/preview. Have you tried using that?


Pin node version

Node 23.0.0 was released 2 days ago and broke hooks in this repository. The log I receive is the following:
An unexpected error has occurred: CalledProcessError: command: ('/home/stas/.cache/pre-commit/repo6fs1kess/node_env-default/bin/node', '/home/stas/.cache/pre-commit/repo6fs1kess/node_env-default/bin/npm', 'pack')
return code: 254
stdout: (none)
stderr:
    (node:2233787) ExperimentalWarning: Support for loading ES Module in require() is an experimental feature and might change at any time
    (Use `node --trace-warnings ...` to show where the warning was created)
    npm warn tarball tarball data for file:/home/stas/.cache/pre-commit/repo6fs1kess/ (null) seems to be corrupted. Trying again.
    npm warn tarball tarball data for file:/home/stas/.cache/pre-commit/repo6fs1kess/ (null) seems to be corrupted. Trying again.
    npm error code ENOENT
    npm error syscall lstat
    npm error path /home/stas/.cache/pre-commit/repo6fs1kess/ode_env-default/lib/node_modules/npm/node_modules/sprintf-js/dist/.gitattributes
    npm error errno -2
    npm error enoent ENOENT: no such file or directory, lstat '/home/stas/.cache/pre-commit/repo6fs1kess/ode_env-default/lib/node_modules/npm/node_modules/sprintf-js/dist/.gitattributes'
    npm error enoent This is related to npm not being able to find a file.
    npm error enoent
    npm error A complete log of this run can be found in: /home/stas/.npm/_logs/2024-10-18T16_37_40_681Z-debug-0.log
Check the log at /home/stas/.cache/pre-commit/pre-commit.log

NPM log is no more helpful. This is something related to node v23, and pre-commit defaults to using latest version. The problem is that npm pack works, while node npm pack for some reason does not, and pre-commit invokes the latter.
To reproduce (if this doesn't fail, try pre-commit clean prior to running this):
cd $(mktemp -d)
git init
cat >.pre-commit-config.yaml <<EOF
repos:
-   repo: https://github.com/biomejs/pre-commit
    rev: "v0.5.0"
    hooks:
    -   id: biome-check
        additional_dependencies: ["@biomejs/biome"]
EOF
pre-commit install
pre-commit install-hooks

Works:
cd $(mktemp -d)
git init
cat >.pre-commit-config.yaml <<EOF
repos:
-   repo: https://github.com/biomejs/pre-commit
    rev: "v0.5.0"
    hooks:
    -   id: biome-check
        additional_dependencies: ["@biomejs/biome"]
        language_version: '22.0.0'
EOF
pre-commit install
pre-commit install-hooks

language_version can be overridden by end users, but it's expected that hooks work "out of the box", hence pinning language_version here would be very helpful. (to make it clear, this does not depend on system node installation, so won't be exposed to users directly in any way)

Here's an upstream confirmation: already discovered in pre-commit/pre-commit#3339 and caused by nodejs/node#55410.


error in driver during machine creation on 2.4.5

When creating clusters in vCenter 7.0, Rancher 2.4.5 reports
Error creating machine: Error in driver during machine creation: The name 'k8s-dev-worker-1' already exists.
The VMs are made in vCenter, but do not get powered on, i have tried to manually power them on, but rancher ends up removing them and re-deploying.
However the same vCenter 7.0 with Rancher 2.3.5 works fine.
2020/07/02 18:21:06 [ERROR] NodeController c-5v46w/m-5fksc [node-controller] failed with : Error creating machine: Error in driver during machine creation: The name 'k8s-dev-worker-1' already exists.
2020/07/02 18:23:07 [ERROR] NodeController c-5v46w/m-5fksc [node-controller] failed with : nodes.management.cattle.io "m-5fksc" not found

Rancher
v2.4.5

User Interface
v2.4.28

Helm
v2.16.8-rancher1

Machine
v0.15.0-rancher43

Rancher
v2.3.5

User Interface
v2.4.22

Helm
v2.14.3-rancher1

Machine
v0.15.0-rancher29

v2.4.2 also works..


Update stypes.xsd to include enumeration of NCTS5.1 with and without space

As discussed in NCTS5 Hypercare call 08/07 11:30 with @jjr2000.

This appears to be duplicate, @sonisn1988 just beat you to it with ticket #62


Markdown footnote validation

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

VS Code Version: 1.73.1
OS Version: Windows_NT x64 10.0.19044

Steps to Reproduce:

Create new markdown file
Type a text:

text[^1]
[^1]: footnote

See problems:
(×) No link definition found: '^1' (link.no-such-reference) [Ln 1, Col 6]

Try separating footnotes and citations by one line.

Try separating footnotes and citations by one line.

Nothing changed.


Issues downloading containers via Kitematic

Expected behavior
Press create button on a new container. It starts to download container to local repository.
Actual behavior
(HTTP code 500) server error - {"message":"Get https://registry-1.docker.io/v2/: EOF"}
Information about the Issue
Steps to reproduce the behavior

open Kitematic
try creating several new containers, not present in local repository

Please make sure that you're on a stable connection and use a solid DNS such as CloudFlare DNS <IP_ADDRESS>
https://blog.cloudflare.com/announcing-1111/


Visualizations: when date is chosen for overlay, legend text is wrong

I'm using the PRISM2 study & Death date, which has 3 unique values:  2018-05-13, 2018-10-30, and 2019-01-24
When I select a date variable with < 8 unique values as my overlay variable, the legend does not look right (says "Data" instead of the date):

so death date has only 3 values but a data shape of continuous, which is whats causing the unexpected behavior. i can make handling this case stronger in plot.data. but i also wonder if its shape should really be categorical? @danicahelb @jaycolin


No error handling for null tiles.

Is this still in development? I am looking at using Koop to add layers into AGOL using the feature server output from static geojson files. This seems like the simplest method for accomplishing this outside of loading the data into AGOL. If so I will add more issues. If not disregard.
If geojsonVT(data).getTile(z, x, y) is null then the plugin crashes as it calls vtpbf using a null tile. A simple fix would be something like the following:
  if (!tile) {
    res.writeHead(204, { 'Access-Control-Allow-Origin': '*' })
    return res.end()
  }

hi @reyemtm  - we would definitely like to keep moving forward on this plugin; just looking for contributors.  You should submit your fix as PR and I will happily review.
BTW, Koop sets CORS wide-open internally, so I am not sure you need the { 'Access-Control-Allow-Origin': '*' }, but I haven't looked into it.

Sounds good. There are a few other updates I have in mind. I have another
issue which might be related to the core...response time is slow to get to
the output plugin...not sure how to debug that one.
On Wed, May 22, 2019, 12:22 PM Rich Gwozdz<EMAIL_ADDRESS>wrote:

hi @reyemtm https://github.com/reyemtm - we would definitely like to
keep moving forward on this plugin; just looking for contributors. You
should submit your fix as PR and I will happily review.
BTW, Koop sets CORS wide-open internally, so I am not sure you need the {
'Access-Control-Allow-Origin': '*' }, but I haven't looked into it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/koopjs/koop-output-vector-tiles/issues/3?email_source=notifications&email_token=ABQ2HUN7C3UONMV6KZSWEYTPWVXLNA5CNFSM4HOV4PXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODV7S62I#issuecomment-494874473,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABQ2HUKPKTDMUJQBHMELZBLPWVXLNANCNFSM4HOV4PXA
.

What provider are you using?

Local file geojson provider. You can check the request received log in the
output plugin. I think I am right in that the data file is passed to memory
by default?
On Wed, May 22, 2019, 4:19 PM Rich Gwozdz<EMAIL_ADDRESS>wrote:

What provider are you using?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/koopjs/koop-output-vector-tiles/issues/3?email_source=notifications&email_token=ABQ2HUPI7BHGNAO5GP64MVLPWWTELA5CNFSM4HOV4PXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWAGY6A#issuecomment-494955640,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABQ2HUKYE3JHLTDPXXW5IILPWWTELANCNFSM4HOV4PXA
.

Yes, that provider has data cached for 10 seconds (geojson.ttl = 10).  After 10 seconds, it will fire getData again and re-read the file.

Oh thanks that explains what I am experiencing. I will look at adjusting
the cache timeout.
On Wed, May 22, 2019, 4:57 PM Rich Gwozdz<EMAIL_ADDRESS>wrote:

Yes, that provider has data cached for 10 seconds (geojson.ttl = 10).
After 10 seconds, it will fire getData again and re-read the file.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/koopjs/koop-output-vector-tiles/issues/3?email_source=notifications&email_token=ABQ2HUJ3ITBKUCIMNVZGEF3PWWXSVA5CNFSM4HOV4PXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWAKIYY#issuecomment-494969955,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABQ2HUNQA2HEKDPTFKYBVODPWWXSVANCNFSM4HOV4PXA
.

A couple more thoughts on this cache. Increasing the cache ttl fixed the
behavior I was seeing. Any reason to keep this limit low? Is there a fear
that keeping the file in cache will lead to high mem usage, or is it that
the underlying file could change? Trying to figure out the reasoning so I
can better justify my approach.
On Wed, May 22, 2019, 4:57 PM Rich Gwozdz<EMAIL_ADDRESS>wrote:

Yes, that provider has data cached for 10 seconds (geojson.ttl = 10).
After 10 seconds, it will fire getData again and re-read the file.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/koopjs/koop-output-vector-tiles/issues/3?email_source=notifications&email_token=ABQ2HUJ3ITBKUCIMNVZGEF3PWWXSVA5CNFSM4HOV4PXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWAKIYY#issuecomment-494969955,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABQ2HUNQA2HEKDPTFKYBVODPWWXSVANCNFSM4HOV4PXA
.

There's no particular reason the ttl is to 10 seconds.  I use this provider mainly for debugging and testing koop-core, FeatureServer, and Winnow, so I haven't really scrutinize performance details.
Ideally, the ttl should be set by a provider-namespaced config value, so that it's not hardcoded in the getData function.  Feel free to submit a PR on that repo, or maybe I can squeeze in the change sometime soon.


fix(host): return cache image error

What this PR does / why we need it:

[x] Smoke testing completed

If yes, write the release branches name in the below format and submit the related cherry-pick PR:

release/3.8

/area host
/cc @zexi

/hold

/hold cancel
/area host
/lgtm
/approve


Auto-tunneling not working when there is a wifi connection without internet access

Opening a different issue for this mostly to inform other users since this can be sketchy to detect
When you have both wifi and mobile data on (I think the standard configuration as of today) but you live or are in a zone where there is a wifi with captive portal or that does not provide internet access and is a trusted ssid the auto-tunneling breaks
(I wrote captive portal but may be any reason why wifi doesnt provide internet access)
This happens because the android os knows that the wifi doesnt provide internet access hence uses mobile data but the app checks only if you are, in general, connected to a trusted ssid
the logic in order to fix this should be something like:

if(connected to trusted_ssid AND wifi provides internet) then disable vpn
if(connected to trusted_ssid AND mobile data is in use) then enable vpn

from the screenshot is possible to see the wifi that states "No internet connection avialable", the mobile data ON in the notification panel and the vpn tunnel wrongly off (since yes i am also connected to a trusted ssid but actually I am on mobile data)
My main goal is to inform since I did not manage to understand why was not working until i went to check into wifi settings
To be fair I do not know if through android api this kind of control would be possible but forgetting the wifi network may be an option such as considering if to add that network as a trusted one with this issue in mind

Thanks for the detailed issue! This is definitely an edge case I've never tested before. I look into it.

@mich2k This should be fixed as soon as google approves version 2.3.7. Let me know if anything doesn't work as expected! Thanks!

@mich2k The fix for this is live now. Please let me know if it doesn't work as expected!


WIP: Create and synchronise a Cassandra service

Cassandra cluster database pods need to connect to each other.
A service provides a DNS name for the cluster and each pod will be given a stable hostname under that DNS name.

https://kubernetes.io/docs/tutorials/stateful-application/cassandra/#creating-a-cassandra-headless-service

When a CassandraCluster resource is added or modified, the Service control will ensure that a headless service is created for the cluster and it will update the service configuration if the CassandraCluster configuration changes.
Fixes: https://gitlab.jetstack.net/marshal/navigator/issues/26
CassandraController now creates a Service resource.

Closing in favour of https://github.com/jetstack/navigator/pull/58


How to center align text in a textfield while using acroform

Is there a way i can specify the alignment of a text in PDTextfield in a PDAcroform?
I tried using the Quad constants but, doesn't work.
textField.q = QUADDING_CENTERED

The function call should be textfield.setQ(QUADDING_CENTERED), does that solve the problem?

I had to call textField.acroForm.q = QUADDING_CENTERED and it worked. I am using Kotlin and so used this syntax.


changes to work with can-connect 2 & can-observation 4

Not Fixed
Error
in session/session_tests-x-provider.js, the handler method never gets called.  It is making the assumption that the Session.on('current', handler) will be called when a session is created, and that doesn't seem to be happening anymore.
Needs fix
Did the binding of a constructor event change?
Error
Files not found:
GET http://localhost:8080/node_modules/can-fixture/node_modules/can-connect/node_modules/steal-stache/package.json 404 (Not Found)
GET http://localhost:8080/node_modules/can-define/node_modules/can-compute/node_modules/can-stache-key/package.json 404 (Not Found)
GET http://localhost:8080/node_modules/can-define/node_modules/can-stache-key/node_modules/can-reflect-promise/package.json 404 (Not Found)

Needs fix
Why are these files not found?  Are they causing problems?
Error
Unhandled promise rejection -> NotAuthenticated
{"name":"NotAuthenticated","message":"Could not find stored JWT and no authentication strategy was given","code":401,"className":"not-authenticated","errors":{}}

Is this an issue?
Fixed
Error
can-connect/can/map/map must be configured with a List type
Error: Error loading<EMAIL_ADDRESS>at http://localhost:8080/test/test.js
Error: can-connect/can/map/map must be configured with a List type
      at Behavior.init (http://localhost:8080/node_modules/can-connect/can/map/map.js:31:11)
      at Behavior.init (http://localhost:8080/node_modules/can-connect/can/ref/ref.js:432:24)
      at connect (http://localhost:8080/node_modules/can-connect/connect.js:50:12)
      at runSessionTests (http://localhost:8080/session/session_tests-x-provider.js:123:23)
      at Object.eval (http://localhost:8080/session/session_test.js:77:1)
      at eval (http://localhost:8080/session/session_test.js:153:4)
      at eval (http://localhost:8080/session/session_test.js:153:98)
  Evaluating http://localhost:8080/session/session_test.js
  Evaluating http://localhost:8080/test/test.js

Fixed by
Add list to the session defineMap
  Session.List = DefineList.extend({
    '*': Session
  });

Error
Cannot define property _saving, object is not extensible
browser.js:3 Unhandled promise rejection TypeError: Cannot define property _saving, object is not extensible
    at Function.defineProperty (<anonymous>)
    at Function.define.property (can-define.js:309)
    at Object.defineExpando (define-helpers.js:29)
    at User.setKeyValue (map.js:52)
    at Object.setKeyValue (get-set.js:50)
    at Behavior.save (map.js:311)
    at User.eval [as save] (map.js:689)
    at eval (session_tests-x-provider.js:199)
    at eval (browser.js:3)
    at eval (browser.js:3)

Fixed by
define _saving property on User defineMap
Error
Cannot define property _destroying, object is not extensible
TypeError: Cannot define property _destroying, object is not extensible
    at Function.defineProperty (<anonymous>)
    at Function.define.property (can-define.js:309)
    at Object.defineExpando (define-helpers.js:29)
    at User.setKeyValue (map.js:52)
    at Object.setKeyValue (get-set.js:50)
    at Behavior.destroy (map.js:322)
    at User.eval [as destroy] (map.js:745)
    at eval (session_tests-x-provider.js:213)
    at eval (browser.js:3)
    at eval (browser.js:3)

Fixed by
define destroying property on User defineMap
Error
A bunch of these failures:
TypeError: Cannot define property _instanceDefinitions, object is not extensible
    at Function.defineProperty (<anonymous>)
    at Object.defineExpando (define-helpers.js:20)
    at Account.setKeyValue (map.js:52)
    at Object.setKeyValue (get-set.js:50)
    at Behavior.save (map.js:311)
    at Account.eval [as save] (map.js:689)
    at eval (session_tests-x-provider.js:343)
    at eval (browser.js:3)
    at eval (browser.js:3)
    at MutationObserver.f (browser.js:2)

Fixed by
Adding _saving and _destroying to Message DefineMap in service/service_tests-x-provider.js
Error
A bunch of tests were failing due to a deepEqual of a pre-saved map and a map retrieved via .find.  They were different because .save adds _saving and _destroying and one map had those props while the other didn't.
Fixed by
I worked around this issue for now calling  .serialize() on both maps which (somehow) makes them both have _saving and _destroying.  (?? not sure what's going on there).

Also, the whole _saving and _destroying stuff seems rather tedious to me..  should those values be added automatically like that?  Wouldn't that mean that anyone who wants to use .save would need to define _saving and _destroying ?

all above issues fixed by updating more of the dependencies :)


What happened to version 12.6.1?

Issue Transfer
This issue has been transferred from the Azure SDK for .NET repository, #24818.
Please be aware that @atrauzzi is the author of the original issue and include them for any questions or replies.
Details
I noticed that 12.6.1 is 404ing on maven central.  Is there any reason for this?

If you need further help
please create a ticket at aka.ms/azsupt.
Issue Type: Technical
Service Type: Blob Storage
Summary: #JavaSDKv12
Problem type: Development
Problem subtype: Client library or SDK

Put 12.6.1 back?

Can you move to 12.7? that is only minimal changes over 12.6.1. And none of your dependencies would change.


Fix If the width of the image is not a multiple of 4, the rendering r…

Fix If the width of the image is not a multiple of 4, the rendering result by image_view will be skewed. (#8166)

let's hold off on this one. This is due to a bug in the vk backend, but said bug is not simple to address.


Is it possible to select some frames of GIF before resize ?

Question about an existing feature
What are you trying to achieve?

I have big GIFs (10 - 20MB) with ~ 2-400 frames and want to resize it.
But with use lambda aws to handle, so spend more time to handle. (> 60 - 120 - ... seconds)
So could I select 100 frames from total frames before resize to reduce time handle or anyway to achieve !

When you searched for similar issues, what did you find that might be related?

Please provide a minimal, standalone code sample, without other dependencies, that demonstrates this question

const imageResized = image.clone().resize(300, 300);
const data = await imageResized.webp().toBuffer()

Please provide sample image(s) that help explain this question

7IsD-example-gif

Did you see the pages option of the constructor?
https://sharp.pixelplumbing.com/api-constructor
sharp(animatedGifInput, { pages: 100 })...

@lovell Thanks for quick reply. Sorry for my explain not detail.
With { pages: 100 } options it will get 100 pages first.
But I want 100 frames with indexes array like [0, 2, 4, ...200]

There's no support for a "step" - you'll have to extract frame-by-frame then reassemble, which will probably be slower than resizing anyway.
If you're using AWS Lambda for CPU-bound tasks, remember to assign as much RAM and therefore CPU time as possible.


wj32.ProcessHacker version 2.39

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

same as  https://github.com/microsoft/winget-pkgs/pull/373


Applicant gets Application ID Number when they submit program form

Dynamically generate an "application ID number" when the applicant hits submit. Show it on the "thank you" page once their answers are saved to the database.

It'd be helpful to get a bit more clear about our terminology around "application" since it's used in a few different ways right now. I propose we call this "program application ID number" to indicate that it is program-specific.

Yeah, the term `application' is overloaded. Sure, we can make that name change


python311Packages.dvc-data: 2.23.1 -> 2.24.0, python311Packages.dvc: 3.33.3 -> 3.33.4

Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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

[ ] 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/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 274091 run on x86_64-linux 1

14 packages built:

dvc (python311Packages.dvc)
dvc-with-remotes
dvc-with-remotes.dist
dvc.dist (python311Packages.dvc.dist)
python310Packages.dvc
python310Packages.dvc-data
python310Packages.dvc-data.dist
python310Packages.dvc.dist
python310Packages.dvclive
python310Packages.dvclive.dist
python311Packages.dvc-data
python311Packages.dvc-data.dist
python311Packages.dvclive
python311Packages.dvclive.dist


[Bug]: [9576] Failed to execute script 'pyi_rth_pkgres' due to unhandled exception!

部署方式
手动部署
版本
bilibili-buy_v2.8.6
异常情况
Traceback (most recent call last):
File "PyInstaller\loader\pyimod02_importers.py", line 187, in get_code
File "PyInstaller\loader\pyimod01_archive.py", line 134, in extract
PermissionError: [Errno 13] Permission denied: 'C:\Users\Administrator\Desktop\bilibili-buy_v2.8.6\bilibili-buy.exe'
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "PyInstaller\hooks\rthooks\pyi_rth_pkgres.py", line 158, in
File "PyInstaller\hooks\rthooks\pyi_rth_pkgres.py", line 36, in pyi_rthook
File "PyInstaller\loader\pyimod02_importers.py", line 419, in exec_module
File "pkg_resources_init.py", line 33, in
File "PyInstaller\loader\pyimod02_importers.py", line 419, in exec_module
File "email\parser.py", line 12, in
File "PyInstaller\loader\pyimod02_importers.py", line 419, in exec_module
File "email\feedparser.py", line 27, in
File "PyInstaller\loader\pyimod02_importers.py", line 419, in exec_module
File "email_policybase.py", line 9, in
File "PyInstaller\loader\pyimod02_importers.py", line 419, in exec_module
File "email\utils.py", line 28, in
File "PyInstaller\loader\pyimod02_importers.py", line 419, in exec_module
File "random.py", line 56, in
File "PyInstaller\loader\pyimod02_importers.py", line 400, in exec_module
File "PyInstaller\loader\pyimod02_importers.py", line 189, in get_code
ImportError: PyiFrozenImporter cannot handle module 'bisect'
[9576] Failed to execute script 'pyi_rth_pkgres' due to unhandled exception!
报错信息
Traceback (most recent call last):
File "PyInstaller\loader\pyimod02_importers.py", line 187, in get_code
File "PyInstaller\loader\pyimod01_archive.py", line 134, in extract
PermissionError: [Errno 13] Permission denied: 'C:\Users\Administrator\Desktop\bilibili-buy_v2.8.6\bilibili-buy.exe'
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "PyInstaller\hooks\rthooks\pyi_rth_pkgres.py", line 158, in
File "PyInstaller\hooks\rthooks\pyi_rth_pkgres.py", line 36, in pyi_rthook
File "PyInstaller\loader\pyimod02_importers.py", line 419, in exec_module
File "pkg_resources_init.py", line 33, in
File "PyInstaller\loader\pyimod02_importers.py", line 419, in exec_module
File "email\parser.py", line 12, in
File "PyInstaller\loader\pyimod02_importers.py", line 419, in exec_module
File "email\feedparser.py", line 27, in
File "PyInstaller\loader\pyimod02_importers.py", line 419, in exec_module
File "email_policybase.py", line 9, in
File "PyInstaller\loader\pyimod02_importers.py", line 419, in exec_module
File "email\utils.py", line 28, in
File "PyInstaller\loader\pyimod02_importers.py", line 419, in exec_module
File "random.py", line 56, in
File "PyInstaller\loader\pyimod02_importers.py", line 400, in exec_module
File "PyInstaller\loader\pyimod02_importers.py", line 189, in get_code
ImportError: PyiFrozenImporter cannot handle module 'bisect'
[9576] Failed to execute script 'pyi_rth_pkgres' due to unhandled exception!

试过了，关闭clash就好了，不知道是不是端口冲突


Remove branch filter on workflow trigger strategy

@huachaohuang @ZhongliGao I don't know why you guys change the branch name, but if it's the case and we have only interest branches on the upstream repository (i.e., no personal branch), we can trigger the workflow on every "push" and "pull request" event.
See also https://docs.github.com/en/actions/reference/events-that-trigger-workflows#configuring-workflow-events .

There are two reasons:

We are going to have another demo that may not share the code with the current one. So we need two different branches.
We will keep these demo branches forever to avoid breaking links to the code. But we don't want to keep them in the main branch.

Get it. Go ahead!


[Cosmos] Fix strictness issues in multiregion.spec

Fixes strictness issues in test/public/integration/multiregion.spec.ts in #12745.

I'm seeing some interesting icons when I run rushx build > log.txt 2>&1 and view the log.txt file. Is this intended?

@bzhang0 That is because the output is colored. You can disable coloring by removing the --pretty argument to tsc in the check:src:strict script in package.json.

Could you run rushx format?

Could you run rushx format?

Done!

Looks good, thanks!

I'm still getting an error for strict when I run rushx build. Is this because its dependency has an issue? It's still the same string | undefined issue that I had previously fixed but now it has appeared again after applying new changes.

I'm still getting an error for strict when I run rushx build. Is this because its dependency has an issue? It's still the same string | undefined issue that I had previously fixed but now it has appeared again after applying new changes.

Could you merge the remote main in your branch to see the error?

@deyaaeldeen sorry for the late reply! I merged remote main and ran rushx build with strict: true, and I'm receiving the following errors still:

This is after assigning plugin: async (context: RequestContext) like here

@bzhang0 Thanks! Could you delete this line: https://github.com/Azure/azure-sdk-for-js/blob/a28b7bd42afb21dfd92150791ee6e7b1673f85cc/sdk/cosmosdb/cosmos/tsconfig.strict.json#L175 and push that change?

@bzhang0 Thanks! Could you delete this line:
https://github.com/Azure/azure-sdk-for-js/blob/a28b7bd42afb21dfd92150791ee6e7b1673f85cc/sdk/cosmosdb/cosmos/tsconfig.strict.json#L175
and push that change?

Done!

@bzhang0 Thanks, and now the error is clear in CI and it is because endpoint is optional (marked with a question mark in RequestContext). Using RequestContext told us that endpoint is not guaranteed to be defined and we need to account for that by checking if it is defined in the first place before using it. Could you please add that check?
Also, multiregion.spec.ts imports stuff from src which means it is importing things that are not strict yet, remember when we talked about treating this as a graph? so we still cannot take it off the exception list in tsconfig.strict.json quite yet and we will have to add it back there for now in order to merge this PR.

@bzhang0 Thanks, and now the error is clear in CI and it is because endpoint is optional (marked with a question mark in RequestContext). Using RequestContext told us that endpoint is not guaranteed to be defined and we need to account for that by checking if it is defined in the first place before using it. Could you please add that check?
Also, multiregion.spec.ts imports stuff from src which means it is importing things that are not strict yet, remember when we talked about treating this as a graph? so we still cannot take it off the exception list in tsconfig.strict.json quite yet and we will have to add it back there for now in order to merge this PR.

Sounds great, I'll add a small check before each of the issues, then add multiregion.spec.ts back into tsconfig.strict.json. I also understand the issue with the imports, so I'll put my focus on on that area.

@bzhang0 Thanks! Could you delete this line:
https://github.com/Azure/azure-sdk-for-js/blob/a28b7bd42afb21dfd92150791ee6e7b1673f85cc/sdk/cosmosdb/cosmos/tsconfig.strict.json#L175
and push that change?

On a side note, how are you doing the code embed? I'm trying to do this in my new PR #20449 but I'm having trouble doing so.

I think it is not working because it is from your clone and not from our repo.


Readme tip to use environment variable to patch causes all Python installation for all versions to fail on MacOS Sonoma 14.0

Provide environment information
I have a MacOs with Sonoma 14.0. I followed the README of the plugin and set the env var ASDF_PYTHON_PATCH_URL.

To Reproduce
On MacOs set the URL accordingly to the README.
Then run
asdf install python 3.9.0
Describe the Bug
I attempted to install a version of python. The installation would fail without any explanation.
Expected Behaviour
I expected a clean installation of python.

A couple of things
Not all versions of python officially support Apple Silicon (Mx)
asdf-python uses pyenv which has a plugin called python-build its job is to apply patches and compile compatible versions. This is still limited because of other compatibly issues e.g compiler, sdks , library versions etc.
versions that should work out of the box, that is no ASDF_PYTHON_PATCH_URL set. work for me on Apple silicon with asdf-python are the following.

Python 3.7.12+
Python 3.8.10+
Python 3.9.1+
Python 3.10.0+
Python 3.11.0+
Python 3.12+ .....

I haven't gone any further because anything under 3.8 doesn’t receive security updates and is consider to be in end of life. Anything under 3.10 don't get most bug fixes.
There are some unofficial patches out there to get specific versions to work, personally, I don't use them.
Also make sure you have all the dependences installed required to build and require by python.
Homebrew, makes this easy for mac users and I believe even python-build has some builtin compatibility for homebrew.
I have just tried macOS Sonoma will all of the above and they work.
If you’re adamant on getting older versions to work then I suggest looking for unofficial back ports and unofficial patches.
Another option is to use the Intel versions this can be achieved via Rosetta 2 although I see most people have troubles to get it to compile python.  Other options are using Intel system emulation with a Linux VM  such as Qemu and Docker Buildx which also uses Qemu. Other than that, I would forget using them unless you are forced.
If you have any issues with the above versions without ASDF_PYTHON_PATCH_URL set.
Try looking at setting any verbose and/or tracing arguments if available, look at brew doctor for errors and brew list for missing python dependences. If you get any errors provide them and also check your $TMPDIR path for any logs regarding python-build etc and looking at the console app logs also wouldn’t hurt.


Migrate to null safety

Discussion 🗣
Migrate the whole project to null safety.
Context
Follow this guide to know about the migration: https://dart.dev/null-safety/migration-guide

@himanshusharma89 I'd like to work on this!

Sure @ShrutiJ-01. Also if you face some dependency issues do raise a new issue for that.

Hey @himanshusharma89  the version of packages "geocoder" and  "modal_progress_hud" which the project are using are not null safe. There is a null safe version of "modal_progress_hud"  and similar alternative for geocoder I've found. May I add these dependencies in the pubspec instead of old ones?

Sure @ShrutiJ-01