Add Background Music v0.1.1

Adds Background Music, an app to pause/un-pause music in the background
such as Spotify and iTunes.
After making all changes to the cask:

[x] brew cask audit --download background-music.rb is error-free.
[x] brew cask style --fix background-music.rb reports no offenses.
[x] The commit message includes the cask’s name and version.

Additionally, if adding a new cask:

[x] Named the cask according to the token reference.
[x] brew cask install background-music worked successfully.
[x] brew cask uninstall background-music worked successfully.
[x] Checked there are no open pull requests for the same cask.
[x] Checked the cask was not already refused in closed issues.
[x] Checked the cask is submitted to the correct repo.

I can't uninstall it with those errors:
brew cask uninstall background-music
==> Uninstalling Cask background-music
==> Running uninstall process for background-music; your password may be necessa
==> Removing launchctl service com.bearisdriving.BGM.XPCHelper
Password:
==> Quitting application ID com.bearisdriving.BGM.App
==> Running uninstall script /Applications/Background Music.app/Contents/Resourc
Error: No such file or directory - /Applications/Background
Follow the instructions here:
  https://github.com/caskroom/homebrew-cask#reporting-bugs
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/2.3.0/open3.rb:199:in `spawn'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/2.3.0/open3.rb:199:in `popen_run'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/2.3.0/open3.rb:95:in `popen3'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/system_command.rb:88:in `each_output_line'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/system_command.rb:24:in `run!'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/system_command.rb:13:in `run'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/artifact/abstract_uninstall.rb:199:in `uninstall_script'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/artifact/abstract_uninstall.rb:53:in `block in dispatch_uninstall_directives'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/artifact/abstract_uninstall.rb:50:in `each'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/artifact/abstract_uninstall.rb:50:in `dispatch_uninstall_directives'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/artifact/uninstall.rb:7:in `uninstall_phase'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/installer.rb:384:in `block in uninstall_artifacts'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/2.3.0/set.rb:674:in `each'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/2.3.0/set.rb:674:in `each'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/installer.rb:381:in `uninstall_artifacts'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/installer.rb:370:in `uninstall'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/uninstall.rb:22:in `block in run'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/uninstall.rb:12:in `each'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/uninstall.rb:12:in `run'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/abstract_command.rb:35:in `run'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:97:in `run_command'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:167:in `run'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:131:in `run'
/usr/local/Homebrew/Library/Homebrew/cmd/cask.rb:8:in `cask'
/usr/local/Homebrew/Library/Homebrew/brew.rb:100:in `<main>'

Interesting. What version of macOS are you running?

my version is : 10.13.1 (17B48)

https://github.com/caskroom/homebrew-cask/issues/40908

The output of 'brew doctor' and 'brew config'

On Nov 17, 2017, at 6:48 AM, 卜木<EMAIL_ADDRESS>wrote:
my version is : 10.13.1 (17B48)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@SConaway
brew doctor:
bumu@bumudembp:~|⇒  brew doctor
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry and just ignore them. Thanks!

Warning: "config" scripts exist outside your system or Homebrew directories.
`./configure` scripts often look for *-config scripts to determine if
software packages are installed, and what additional flags to use when
compiling and linking.

Having additional scripts in your path can confuse software installed via
Homebrew if the config script overrides a system or Homebrew provided
script of the same name. We found the following "config" scripts:
  /Users/bumu/.pyenv/shims/python3.5m-config
  /Users/bumu/.pyenv/shims/icu-config
  /Users/bumu/.pyenv/shims/libpng16-config
  /Users/bumu/.pyenv/shims/python3.6m-config
  /Users/bumu/.pyenv/shims/python3.5-config
  /Users/bumu/.pyenv/shims/python-config
  /Users/bumu/.pyenv/shims/python3-config
  /Users/bumu/.pyenv/shims/python3.6-config

Warning: You have unlinked kegs in your Cellar
Leaving kegs unlinked can lead to build-trouble and cause brews that depend on
those kegs to fail to run properly once built. Run `brew link` on these:
  emacs-plus
  python3
  python

brew config:
HOMEBREW_VERSION: 1.3.7
ORIGIN: https://github.com/Homebrew/brew
HEAD: 441c212f2dd70f053ee21eac8398c5fe5dfd0551
Last commit: 7 days ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: 077cbecbbb20397f6580000e73c9af58c5351509
Core tap last commit: 12 hours ago
HOMEBREW_PREFIX: /usr/local
HOMEBREW_GIT_CONFIG_FILE: /usr/local/Homebrew/.git/config
CPU: quad-core 64-bit kabylake
Homebrew Ruby: 2.3.3 => /usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/bin/ruby
Clang: 9.0 build 900
Git: 2.15.0 => /usr/local/bin/git
Curl: 7.54.0 => /usr/bin/curl
Perl: /usr/local/bin/perl => /usr/local/Cellar/perl/5.26.1/bin/perl
Python: /usr/local/opt/python/libexec/bin/python => /usr/local/Cellar/python/2.7.14/Frameworks/Python.framework/Versions/2.7/bin/python2.7
Ruby: /usr/local/bin/ruby => /usr/local/Cellar/ruby/2.4.2_1/bin/ruby
Java: 1.8.0_144
macOS: 10.13.1-x86_64
Xcode: 9.1
CLT: <IP_ADDRESS>.1.1508540944
X11: N/A

thanks for your attention in my issues.

Can you create a new issue? Thanks.

@SConaway There is an open issue for this: https://github.com/caskroom/homebrew-cask/issues/40908

@SConaway Sorry


[Feature Request]: Enhance the Contributor Page with better Design

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
Description:
The current design of the Contributor Page requires enhancements to improve visual appeal.
Proposed Changes:

Redesign the layout for a cleaner and more modern look with better spacing and alignment.
Introduce profile cards for each contributor, displaying their photos, names, roles, and contributions.
Ensure the page is fully responsive for optimal viewing on all devices.

Demo
https://github.com/user-attachments/assets/9b8a83de-3a5a-42ec-8f45-0fb8031c9a77
https://github.com/user-attachments/assets/1a9b9e34-4da5-4b30-b118-81fb2c8d49c9
Use Case

Improved Visual Layout: Redesign the Contributor Page to feature a cleaner, more modern layout with better spacing and alignment for enhanced readability.

Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@Trisha-tech please look into the issue and if you want me to work on it. do mention which design


MSDKUI-1654: Improve GuidanceManeuverData handling

The GuidanceManeuverData model contains nullable properties and,
therefore, GuidanceManeuverView should reflect that.
Current implementation only updates the view if the values carried by
GuidanceManeuverData are not nil. This creates a problem, since a new
value with nil would keep the previous view value.
This commit changes that. If a new model contains nil, the view will
take that in consideration and reflect whatever the model has.
Signed-off-by: Otavio Cordeiro<EMAIL_ADDRESS>

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@07c889e). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master      #11   +/-   ##
=========================================
  Coverage          ?   88.29%
=========================================
  Files             ?       57
  Lines             ?     3682
  Branches          ?        0
=========================================
  Hits              ?     3251
  Misses            ?      431
  Partials          ?        0

Flag
Coverage Δ

#ios
88.29% <100%> (?)

Impacted Files
Coverage Δ

MSDKUI/Classes/GuidanceManeuverView.swift
79.54% <100%> (ø)

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 07c889e...bc60950. Read the comment docs.


django-wfs refurbishment with wfs-1.1.0 and geojson support.

Hello,
I've worked out a refurbishment of the django-wfs project with geojson output and wfs-1.1.0 support.
The project has been adapted to the GeoDjango integration as of django-1.9, so the project will not be running under django-1.8 or even older versions.
I also ported to code to python3, please be graceful, if I introduced some python2 glitches on my way, I did my best to follow django' python3 porting guide, which contains instructions to keep the code compatible with python2.
I've tested XSD-validation both for wfs-1.0.0 and wfs-1.1.0. Since qgis currently acts as a wfs-1.0.0 client  only, I had no opportunity to do integration tests with a wfs-1.1.0 client.
The geojson output has been tested together with openlayers3 as a client.
Please consider my changes and try to undertake more intergation tests.
Hopefully, my changes are a starting point for drawing attention of a wider user group to django-wfs.
Best regards, Wolfgang

FYI, I've now been able to test all my changes under python-2.7.9 and everything works like a charm ;-)

Hi Vascop,
Did you find time to review my changes/additions? I'd really like to get feedback, because actually I do not know, if there are other WFS-like initiatives around in the geo-django community.
TIA, Wolfgang

Hi Vascop,
Any news on reviewing my WFS extensions?
Best regards, Wolfgang


Updated .bg-* and .text-* utilities break when CSS variables are not enabled in v5.1.0

In one of my projects I include the needed Bootstrap SCSS files manually and specifically exclude bootstrap/scss/root, first because I don't rely on most of the CSS variables and secondly because adding all CSS variables increases the CSS file size by about 50-60kb (uncompressed).
With v5.1.0 all .bg-* and .text-* utilities break, as the needed CSS variables are missing, e.g.
.text-white {
    --bs-text-opacity: 1;
    color: rgba(var(--bs-white-rgb), var(--bs-text-opacity)) !important; // --bs-white-rgb is not set
}

Is there a simple way to disable CSS variables generation or to reduce the number of CSS variables? The only way forward right now would be to revert the utility sets back to $theme-colors, but that still leaves an additional --bs-text-opacity: 1; and does not take care of --bs-body-color and the like:
$utilities-text-colors: $theme-colors;
$utilities-bg-colors: $theme-colors;

bootstrap/scss/root required, this variables uses in many files.

The whole bg-, text- update in 5.1.0 has seemingly complicated the entire process of adding a new theme-color. The only way I could find to get the bg- and text- classes regenerated for the new "custom" theme color is....
@import "functions";
@import "variables";
@import "mixins";

$custom: #df711b;

$custom-theme-colors: (
  "custom": $custom
);

$theme-colors: map-merge($theme-colors, $custom-theme-colors);
$theme-colors-rgb: map-loop($theme-colors, to-rgb, "$value");
$utilities-colors: map-merge($utilities-colors, $theme-colors-rgb);
$utilities-text-colors: map-loop($utilities-colors, rgba-css-var, "$key", "text");
$utilities-bg-colors: map-loop($utilities-colors, rgba-css-var, "$key", "bg");

@import "bootstrap";

Whereas in 5.0.2 one could simply follow the example in the docs...
@import "functions";
@import "variables";
@import "mixins";

$custom-colors: (
  "custom-color": #df711b
);

$theme-colors: map-merge($theme-colors, $custom-colors);

In addition to that, the suggested way of extending color utility classes generates HEX color values instead of RGBA and still keeps useless --bs-text-opacity: 1; in every custom color class

The whole bg-, text- update in 5.1.0 has seemingly complicated the entire process of adding a new theme-color. The only way I could find to get the bg- and text- classes regenerated for the new "custom" theme color is....
@import "functions";
@import "variables";
@import "mixins";

Thanks!
This really needs to be written in the official documentation!

I consider this either a regression or a lack in documentation.
Thanks @iatek for the above!

Oh, and btw the same is true if you need to add spacing classes:
@import "node_modules/bootstrap/scss/functions";
@import "node_modules/bootstrap/scss/variables";

// Extend the spacings
$spacers: map-merge(
  $spacers, (
    6: ($spacer * 6)
  )
);

arrrggghhhh add +1 to frustration of note. this change has had me tearing my hair out.
i sort of got it working with
$theme-colors: (
...
);
$theme-colors-rgb: map-loop($theme-colors, to-rgb, "$value");
$utilities-bg-colors: map-loop($utilities-colors, rgba-css-var, "$key", "bg");

but not quite.

Sorry we've been slow to investigate and identify a path forward. We'll need a little more time to experiment and validate the issues identified here. Definitely not in our interest to be causing headaches for folks :).

Hehe thanks mdo. I think the 5.1 update took a few people by surprise.

Left a comment at https://github.com/twbs/bootstrap/issues/34734#issuecomment-916290592. Please read that and let me know what you all think about that potential solution. Seems like the best path forward to me so far.

Hi @mdo,
Honestly I don't understand why you guys went to the rgba path anyways. It created so many extra css variables along with the the extra bs-opacity variable on every colour utility class.
The colour utility class were perfect in v5.0
I don't know maybe I'm feeling this way as I see no extra benefit of all these hassle.

@isaumya This is why: https://getbootstrap.com/docs/5.1/utilities/colors/#opacity. It allows for on the fly manipulation of colors. We've stated we're heading to more CSS variables since developing v5, and there's more coming for components (see #34443, #34600, and #34622).

@mdo I think lots of people are missing the whole point of introducing RGBa with opacity CSS var. If i understand correctly, this is needed for new text and background opacity logic. But the same can be achieved with regular opacity-* utilities. If that's correct, this means we have a bunch of RGB values defined in :root just for setting the opacity. But this doesn't help to extend color palette, because the suggested method still generates HEX values with opacity CSS var. Why don't we use HSL and HSLa instead, where we can simply change lightness to generate additional tints/shades of *-500 colors and change alpha to control opacity? A few examples here.

I love bootstrap but yikes. This all feels like woosh direction gone. I trust you guys to do what's best for the project and I'm in no position to know the big picture with it, but eish. To a long time user at this point I'm just going "I trust the devs" more than what I trust what I'm seeing.
Maybe consider splitting bs up more or having opt in on stuff? The "core" compiled bs with very basic components is still huge.
How many projects does the RGB thing even help?
Maybe with 6 go back to reboot then add stuff to it?
Thanks team. I appreciate the work tho!

First up, it’s definitely my fault that these problems have occurred. I didn’t test these changes enough. We’ll be looking into some Sass testing we can add to help avoid situations like this in the future.

I think lots of people are missing the whole point of introducing RGBa with opacity CSS var. If i understand correctly, this is needed for new text and background opacity logic.
...
Why don't we use HSL and HSLa instead, where we can simply change lightness to generate additional tints/shades of *-500 colors and change alpha to control opacity?

HSL is great, but setting colors up with three variables instead of one feels overkill. Experience has shown me most folks are familiar with hex and RGB, so we went that route. The goal wasn’t to really extend the palette—that’s optional for everyone, and the very use case we want to solve for with this issue—but rather to make components and utilities stronger and easier to customize. These RGB values also accomplish what you want with the alpha of HSLA—that’s how we use them for bg and color utilities now.

But the same can be achieved with regular opacity-* utilities.

The same cannot be achieved with opacity utilities. These variable/utility changes apply to color, background-color, and eventually border-color. Changing the opacity of would affect the whole element instead of those individual properties. Sure there are clever ways around this, but usually with added complications.

How many projects does the RGB thing even help?

Considering we only shipped it a month ago, I’m not entirely sure yet :). Give it time for folks to get used to though. For what it’s worth, Tailwind does something like this already and it seems super helpful. Give it a go and let me know what you think.

Maybe consider splitting bs up more or having opt in on stuff? The "core" compiled bs with very basic components is still huge.
...
Maybe with 6 go back to reboot then add stuff to it?

There won’t be any “going back” to Reboot. Reboot came out of Normalize.css and Bootstrap 4. We never started with that :). Bootstrap has been a compilation of components, layout, plugins, and more for the last decade. It’ll most likely stay that way for the next ;).
Regarding file size, if you can find a way to decrease file size while doing all Bootstrap does, I’d love to hear about it :). We try to keep it lean but convenient. That aside, doing a custom build and using PurgeCSS is a great way to streamline it all :).

Regarding file size, if you can find a way to decrease file size while doing all Bootstrap does, I’d love to hear about it :). We try to keep it lean but convenient. That aside, doing a custom build and using PurgeCSS is a great way to streamline it all :).

That's my biggest hassle with this change, as CSS variables alone account of over 50-60kb (uncompressed). Would it be somehow feasible to add an option to include/exclude CSS variables?

That could be possible. Where are you getting 50-60kb from though? It's maybe a few if you include literally every one of them.

That could be possible. Where are you getting 50-60kb from though? It's maybe a few if you include literally every one of them.

I just compared the final result with and without including @import "bootstrap/scss/root";

_root.scss is 55 lines of CSS—there's no way it's 50-60kb :). I just checked again, 2kb difference.
# With _root.scss
204316 Sep  9 16:20 bootstrap.css

# Without _root
202579 Sep  9 16:21 bootstrap.css

In fact you are right, I just compared 5.0.2 without _root.css and 5.1.1 with:
5.0.2 without
208K 10. Sep 11:28 bootstrap.css
5.1.1 with
216K 10. Sep 11:29 bootstrap.css
Minifying this will give me a 6kb difference. So the above mentioned difference of 50-60kb was most likely due to not importing other Bootstrap components.

Just for reference, this is the correct include to include:
$theme-colors-rgb: map-loop($theme-colors, to-rgb, "$value");
$utilities-colors: map-merge($utilities-colors, $theme-colors-rgb);
$utilities-text-colors: map-loop($utilities-text, rgba-css-var, "$key", "text");
$utilities-bg-colors: map-loop($utilities-colors, rgba-css-var, "$key", "bg");

$utilities-text-colors should loop $utilities-text not $utilities-colors

The whole bg-, text- update in 5.1.0 has seemingly complicated the entire process of adding a new theme-color. The only way I could find to get the bg- and text- classes regenerated for the new "custom" theme color is....
@import "functions";
@import "variables";
@import "mixins";

$custom: #df711b;

$custom-theme-colors: (
  "custom": $custom
);

$theme-colors: map-merge($theme-colors, $custom-theme-colors);
$theme-colors-rgb: map-loop($theme-colors, to-rgb, "$value");
$utilities-colors: map-merge($utilities-colors, $theme-colors-rgb);
$utilities-text-colors: map-loop($utilities-colors, rgba-css-var, "$key", "text");
$utilities-bg-colors: map-loop($utilities-colors, rgba-css-var, "$key", "bg");

@import "bootstrap";

I discovered this on an SO question and here's the working code demo
Whereas in 5.0.2 one could simply follow the example in the docs...
@import "functions";
@import "variables";
@import "mixins";

$custom-colors: (
  "custom-color": #df711b
);

$theme-colors: map-merge($theme-colors, $custom-colors);

This doesn't work entirely for $utilities-text-colors, you will be missing .text-white etc.

I know @tomsommer,
I had to spend a few hours to solve that issue in v5.1.2 and here is what I come up with and it works perfectly.
@import "functions";

// Overwrite your variables
$custom-color1: red !default;
$custom-color2: blue !default;

$min-contrast-ratio: 3;
$link-decoration: none;

// Import the bootstrap variables
@import "variables";

$extra-theme-colors: (
	"gray-dark":   $gray-800,
	"custom-color1": $custom-color1
);

$extra-general-colors: (
	"custom-color2": $custom-color2
);

// Run map-merge() to mer our custom maps with the default Bootstrap maps
$colors: map-remove( $colors, "gray-dark" );
$colors: map-merge( $colors, $extra-general-colors );

$theme-colors: map-merge($theme-colors, $extra-theme-colors );

// ---------- MOST IMPORTANT ---------------
// Extra stuffs we need to do for generating the proper theme color classes (.bg-*, .text-*)
$theme-colors-rgb: map-loop($theme-colors, to-rgb, "$value");

$utilities-text: map-merge($theme-colors, $utilities-text);
$utilities-text-colors: map-loop($utilities-text, rgba-css-var, "$key", "text");

$utilities-bg: map-merge($theme-colors, $utilities-bg);
$utilities-bg-colors: map-loop($utilities-bg, rgba-css-var, "$key", "bg");

// And then continue importing the rest
@import "mixins";
@import "utilities";
// and so on...

This will work. Try and enjoy. :)
Honestly, I really wish I can go back to the Boostrat v5.0 days without all these complexities around adding own custom colors.

The "problem" goes  even beyond the maps.
Modifying anything in _variables.scss that depends on anything else is a cumbersome. Even adding an extra element to $spacers requires you to manually set $spacer: 1rem; before doing anything, even though it's a default, because you cannot change $spacers with a map-merge() after _variables.scss is included, because $gutters and $negative-spacers depends on $spacers (just an example), and so these won't be changed. It's just really a mess to keep track of.
Really something to think about how to make easier, my two cents.

Yeah, this was definitely a shortcoming of Sass we didn’t fully explore at the time. It’s something we’ll 100% have to revisit in v6, but until then, I want to make sure we consider the best options and ship something that’s functional and as easy to document/explain as possible.
What do folks think of that PR I opened to separate out Sass maps? I might need to include more maps there like $spacers it sounds like, but I’m hoping the principle of it helps. We might also need to smooth out the utilities map merges if the latest comments here are right.

I like the idea of splitting it to its own files. But would rather say that it should be split in such a way that if you don't need colours then don't include it (spaces, colours, et al)
I honestly prefer "more" files vs larger files in cases like this.
If the pr fixes the maps thing then I'm 100% behind it. Just gotten bitten again with a .bg-white suddenly disappearing on update again and used many short words again. (Just had to add the stuff in my custom variables file as per above again)

Here's what the diff will look like when importing and customizing Bootstrap after #34942 is merged. Taking the example above, we add a custom variable, put that in it's own map, and then map-merge it with $theme-colors. Since our $theme-colors map is now in a new stylesheet partial, the downstream reassigned maps will be updated properly.
  // Functions come first
  @import "functions";

  // Optional variable overrides here
+ $custom-color: #df711b;
+ $custom-theme-colors: (
+   "custom": $custom-color
+ );

  // Variables come next
  @import "variables";

+ // Optional Sass map overrides here
+ $theme-colors: map-merge($theme-colors, $custom-theme-colors);
+
+ // Followed by our default maps
+ @import "maps";
+
  // Rest of our imports
  @import "mixins";
  @import "utilities";
  @import "root";
  @import "reboot";
  // etc

Here's a quick output of a debug test I did. As you can see, it outputs the correct keys for all
@import "../scss/functions";

$custom-color: #df711b;
$custom-theme-colors: (
  "custom": $custom-color
);

@import "../scss/variables";

$theme-colors: map-merge($theme-colors, $custom-theme-colors);

@import "../scss/maps";

@debug "The following three maps should be the same, ending with the custom key.";
@debug map.keys($theme-colors);
@debug map.keys($theme-colors-rgb);
@debug map.keys($utilities-colors);
@debug "";
@debug "The following two maps should be the same: including custom and ending in black, white, and body.";
@debug map.keys($utilities-text-colors);
@debug map.keys($utilities-bg-colors);

➜ ~/work/bootstrap (split-vars-maps-import) npm run css-test

><EMAIL_ADDRESS>css-test /Users/mdo/work/bootstrap
> sass --style expanded --source-map --embed-sources --no-error-css test/index.scss:test/output.css

test/index.scss:35 Debug: The following three maps should be the same, ending with the custom key.
test/index.scss:36 Debug: "primary", "secondary", "success", "info", "warning", "danger", "light", "dark", "custom"
test/index.scss:37 Debug: "primary", "secondary", "success", "info", "warning", "danger", "light", "dark", "custom"
test/index.scss:38 Debug: "primary", "secondary", "success", "info", "warning", "danger", "light", "dark", "custom"
test/index.scss:40 Debug:
test/index.scss:41 Debug: The following two maps should be the same: including custom and ending in black, white, and body.
test/index.scss:42 Debug: "primary", "secondary", "success", "info", "warning", "danger", "light", "dark", "custom", "black", "white", "body"
test/index.scss:43 Debug: "primary", "secondary", "success", "info", "warning", "danger", "light", "dark", "custom", "black", "white", "body"

It's certainly better, and certainly an improvement, but you have to ask where the limit go:
You can't really include $spacers in this, then you would have to include anything that uses another variable in their value. Imagine changing $small-font-size after loading _variables.scss, you wouldn't get the result you imagine.
The problem is that people actually change $theme-colors, and they do it in a way that merges with the default $theme-colors - this is not really correct, at least in the way scss !default works. But you can't really propagate $theme-colors your self without loading all the colors first, and you can't do that without loading _variables.scss.
As a quick-fix, the PR is fine, but it really is something that needs a deep think. I think Tailwind has solved all this with Webpack?

I think $spacer/$spacers work, and related base variables and maps. Here's the output to an updated version of the above test where I set $spacers: 2rem, imported variables next, then compiled the debug test.
test/index.scss:40 Debug: The following spacers should be doubled the default values.
test/index.scss:41 Debug: Default: 0, 0.25rem, 0.5rem, 1rem, 1.5rem, 3rem
test/index.scss:42 Debug: Updated: 0, 0.5rem, 1rem, 2rem, 3rem, 6rem

The main issue seems to be updating maps that get re-used.

The point is if you do
@import "../scss/variables";

$spacers: map-merge($spacers, (
6: $spacer * 6,
));

You end up with a similar problem of $gutters not being updated.

just checking, adding in a "primary" into the custom-theme map would then change the primary';s colour right? this isnt just for "extending" it?
// variables.scss
$custom-color: #ff0000;
$custom-theme-colors: (
  "primary": $custom-color
);

// styles.css (the output)
.text-primary {
    color: #ff0000
}
.bg-primary {
    background: #ff0000;
}

If you just want to change an existing key's value in $theme-colors, you can override it vs using map-merge with a custom map.
@import "../scss/functions";

$primary: purple;

@import "../scss/variables";
@import "../scss/maps";
// etc

// Test for it
@debug "Test: Primary should be purple instead of #0d6efd (blue).";
@debug "Variable is " + $primary;
@debug "Theme colors map value is " + map.get($theme-colors, "primary");

test/index.scss:29 Debug: Test: Primary should be purple instead of #0d6efd (blue).
test/index.scss:30 Debug: Variable is purple
test/index.scss:31 Debug: Theme colors map value is purple

Looking at writing some tests for the spacers and more to see what we'd need to do.

@tomsommer Yes, the same does apply there. I misunderstood your earlier point I think and was only looking to see if spacers are updated if you change the $spacer value. That update does happen, but yes, the reassigned maps do not. I'm checking out the viability of moving those maps to _maps.scss as well.

The only way to solve this everywhere is to never modify variables after including _variables.scss, but of course that takes away some of the ease of Bootstrap.

  // Functions come first
  @import "functions";

  // Optional variable overrides here
+ $custom-color: #df711b;
+ $custom-theme-colors: (
+   "custom": $custom-color
+ );

  // Variables come next
  @import "variables";

+ // Optional Sass map overrides here
+ $theme-colors: map-merge($theme-colors, $custom-theme-colors);
+
+ // Followed by our default maps
+ @import "maps";
+
  // Rest of our imports
  @import "mixins";
  @import "utilities";
  @import "root";
  @import "reboot";
  // etc

I'm new to Sass and I'm completely lost here. A few weeks ago I completed a Bootstrap sass course and ran into this custom color issue. I thought I had the issue all figured out, but I guess not. I was using the first solution, the one without the _maps.scss. I know how to create the _maps.scss in bootstrap, and then import that file into my custom scss file.
I don't understand what do I put inside the maps.scss file to make all this work, am I importing other sass files?  I'm just new to Bootstrap sass and have had a really good time customizing and make bootstrap my own.  If some could explain what I need to do in 'newbie' terms would be really appreciated. Thank you.

The maps.scss file doesn't exist yet. The discussions here are around how to improve the situation in our next release.
Look up higher to the comments that suggest customizing the map and re-declaring the additional maps.

I'm not sure if this helps but I'm posting it here incase someone else tries the same thing.
I tried @mdo 's solution from PR #34942
$theme-colors: map-merge($theme-colors, $custom-theme-colors);
$theme-colors-rgb: map-loop($theme-colors, to-rgb, "$value");
$utilities-colors: map-merge($utilities-colors, $theme-colors-rgb);
$utilities-text-colors: map-loop($utilities-colors, rgba-css-var, "$key", "text");
$utilities-bg-colors: map-loop($utilities-colors, rgba-css-var, "$key", "bg");

and that breaks for example text-white as its no longer generated from the utilities after I recreated the maps.


docs: fix dead link in source.alias

Summary
remove dead link in builder-doc
Related Links

Checklist

[x] I have added changeset via pnpm run change.
[x] I have updated the documentation.
[ ] I have added tests to cover my changes.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

😄


[Tracking]: Current issues related to rspack

workbox-rspack-plugin works in general now, but there are some failed tests left due to rspack's limitations. This issue is used to track all the existing issues.

rspack.CopyRspackPlugin is trying to parse .js files and throw errors when module syntax is used. (https://github.com/web-infra-dev/rspack/issues/5194)
compilation.namedChunks.get(chunkOrGroup) does not include the splitted chunk when optimization.splitChunks.chunks is all, so the precache list may not be complete. (https://github.com/web-infra-dev/rspack/issues/5908)

Related: https://github.com/web-infra-dev/rspack/issues/4692

All resolved.


app Crashes on LG device with os version 7.1.1 while starting video recording.

I am using Camera2Api to record videos.
This example itself is crashing on device LG Q with OS version 7.1.1.
I have tried by removing line to set videoFrameRate.
Then it started to fail with error "IMediaRecorder died."
Please let me know if anyone finds the solution asap.

This sample has been migrated to a new location where we can accept Pull Requests (check README for more information).
As recommended by GitHub, we are closing all issues and pull requests now that this older repo will be archived.
If you still see this issue in the updated repo, please reopen the issue/PR there. Thank you!


FEAT: added dropdown with sign-out option in navbar

Closes #284
Description

Currently, When the user visits the RDS website, they sign in with Git hub but are unable to sign out. This feature will enable them to sign out by providing them with an option to sign out in a dropdown from the navbar, as shown below:

I agree with you @Muralidhar22 , but I am using the styled-jsx now and I will consider naming files more meaningfully 👍🏻

I think you should add the functionality where the dropdown closes when a user clicks outside of it too

I think you should add the functionality where the dropdown closes when a user clicks outside of it too

+1

I think you should add the functionality where the dropdown closes when a user clicks outside of it too

Added it @Maheima  👍🏻

I think you should add the functionality where the dropdown closes when a user clicks outside of it too

Added it @Maheima 👍🏻

So @kotesh-arya will it close the dropdown when clicking on the navbar or anywhere on screen?

Yes @SanketDhabarde !
It will close even if user clicks anywhere on the page 👍🏻


Handle large files better in the browser

_Moving this issue from https://github.com/feross/instant.io/issues/3 to here, for better visibility._
After lots of work on fixing memory leaks, I was able to seed a 1GB video from Firefox and stream it to Chrome! But there are still issues:

[ ] Can't seed files larger than 500MB in Chrome due to a chromium bug.
[x] Can't seed or download files larger than 1GB because of a buffer limitation. This limitation comes from V8 and we could probably lift this limit in feross/buffer and improve the situation in Firefox.

Besides fixing the above issues, future work should include using browser storage (#86) to get this data out of memory.

Update (5/26/2015):

Chrome supports seeding and video streaming (i.e. downloading) files up to 1GB now, due to this PR that implements streaming blob reading in the filestream package! Downloading via the download blob link is still limited to 500MB until the chromium bug is fixed.

I wondered how mega.nz dealing with the problem and found http://stackoverflow.com/questions/15994554/download-files-like-mega-co-nz . Looks like they are doing every ways hierarchically. How about mega.nz's solution?

But actually, I tried to upload 183.1MB file to mega.nz by Mac OSX Chrome (42.0.2311.135 (64-bit)), it crashed :(

[update] MEGA does not solve it with plugin-free. I tried downloading over 1GB file from MEGA by Firefox, MEGA required addon. Looks like there's so far no compatible way to manage large data within a browser.

Have you seen this https://github.com/eligrey/FileSaver.js/? I'm interested in downloading a blob without displaying a link on the page.

@jakefb I've not tried FileSaver.js, but they say they have a size limitation. Firefox 800MiB, Chrome 500MiB from README.md.

Chrome supports seeding and video streaming (i.e. downloading) files up to 1GB now, due to this PR that implements streaming blob reading in the filestream package! Downloading via the download blob link is still limited to 500MB until the chromium bug is fixed.

@kawa- I can download a 1.8GB file from mega just fine using my Firefox(dev)
https://mega.co.nz/#F!81skTRYK!Ybk7gc4oQfbOM_5P3POzcw

@aniquetahir Great, I've successfully downloaded the file of deepin_2014.3_amd64.iso (1.9GB) from your link in my Firefox ESR (31.7.0, yes it's old) without any addons.
MEGA using blob. I wonder if MEGA manages them.

WebTorrent should be able to handle downloading a 1.8GB file just fine in Firefox once we lift the 1GB limitation in buffer. I'll look at doing that now.

I just released a new version of the browserify buffer that has an increased max size limit of 2GB, instead of 1GB.
I just tested https://instant.io with a 1.5 GB file. Firefox (nightly) can seed and download it with no problems!
Chrome still can't seed files larger than 500GB because of the blob bug, but it can stream the 1.5 GB video/audio file without problems now! The download link doesn't work because of the same blob bug.
The Chrome team is actively working on that bug, so it's just a matter of time. :+1:

HTML5 filesystem API?

@alexeisavca The filesystem API is Chrome only, and it's non-standard and deprecated (and might be removed from Chrome eventually). I'd rather just wait for chrome to fix the blob bug. I've seen lots of commit activity on it recently – it'll be fixed soon.

@alexeisavca I actually attempted a workaround using the filesystem api for a chrome extension, and ran into more problems. The 500mb cap applies to all your blobs, so you can make a buffer splice it into blobs, and write it, but once you have created 500mb of them, you run into the same issue. So what you need to do is release the memory from blobs which are already written, but you can't.
We sort of hacked it using https://developer.mozilla.org/en-US/docs/Web/API/URL/revokeObjectURL, (even though we didn't create an object url out of the blob) and it worked about half the time. The half the time it didn't work, it crashed the browser.
I saw someone claimed some kind of workaround writing from indexdb, but I chose to stop trying to implement half baked solutions and wait for the bug to be fixed.

Does anyone tried using Streams API for downloading files on Chrome? It allows download huge files but seems an active service worker is required. A test where a 1 GB random file is generated and download: rawgit link.

The Streams API won't help us with generating a Blob to use with a download link, since Blobs still can't be more than 500MB in Chrome. But it's a good thought.

Have you guys looked into what those guys did handling big files? reep.io is their web page and they have the code open source in github. Maybe this can help bypassing the google chrome issues.

@feross Just don't use blobs: stream the download directly to the disk or temporally store chunks in IndexedDB then download creating in a Service Worker a stream using those stored chunks as source.
@ovidiualx It uses the deprecated FileSystem API, which @feross don't want to use.

@qgustavor How do you steam directly to disk?
And can you explain your indexdDB solution more in depth?

@ericwooley I wrote directme.ga when Streams API was implemented just on Canary, so maybe the spec changed and it's now broken. I will say how it works (or worked), so the only difference will be instead of using mega as a source just use WebTorrent as a source (both libraries uses Node streams).
Let's download some file, let's say Big Buck Bunny. Instead of downloading chunks (via HTTP or WebRTC) and storing it in a big blog it's used to feed a ReadableSteam, which is downloaded using Service Workers and finally being handled as a normal HTTP download, thus stored in the disk, reducing memory usage and using the disk as the storage.
As ServiceWorkers still can't connect to WebRTC peers IndexedDB is used as a intermediate storage between the download page and the service worker which will create the download stream. It can be also used to allow the WebTorrent download start before the HTTP download.

Whooot! @qgustavor I thought i was the only one that had that idea with service worker + fetch
cheers 🍺
I knew this was going to be possible long before cuz i could do it in the main thread with response + stream from an existing api that returned a stream but wasn't able to save it until it was possible to construct it in a service worker

@jimmywarting I never trought I was the only only which got this idea, but maybe one of the first which implemented it, as I don't found nothing about it when I created that website, just the spec. I think those who idealized and wrote Streams API got that idea first.
For the second part, sorry, I could not understand your English.

Sorry för my bad English. Yea, they probably thought they needed a way to save large data. But they didn't implement a way create a write stream to the filesystem

Did put up an simple example of how to do it in the readme
const client = new WebTorrent()
const torrentId = 'magnet:?xt=urn:btih:6a9759bffd5c0af65319979fb7832189f4f3c35d&dn=sintel.mp4&tr=wss%3A%2F%2Ftracker.btorrent.xyz&tr=wss%3A%2F%2Ftracker.fastcast.nz&tr=wss%3A%2F%2Ftracker.openwebtorrent.com&tr=wss%3A%2F%2Ftracker.webtorrent.io&ws=https%3A%2F%2Fwebtorrent.io%2Ftorrents%2Fsintel-1024-surround.mp4'
// Sintel, a free, Creative Commons movie

client.add(torrentId, torrent => {
    // Download the first file

    const file = torrent.files[0]
    const writeStream = streamSaver.createWriteStream(file.name)

    // Unfortunately we have two different stream protocol so we can't pipe.
    file.createReadStream()
        .on('data', data => writeStream.write(data))
        .on('end', () => writeStream.close())
})

@jimmywarting Cool module, and including the example is very helpful 👍

Thanks

FYI Chrome finally fixed the 500MB blob size limitation! https://bugs.chromium.org/p/chromium/issues/detail?id=375297#c107
This is shipping in tomorrow's Chrome Canary. We can test it out there and see if things work better now!

@feross  @jimmywarting  Can webtorrent or StreamSaver.js handle downloading 1GB+ files Download large data stream (> 1Gb) using javascript ?

Firefox 50 able to seed to 4GB (maybe more)
Chrome ~1GB, crash on 1.2GB

@guest271314 StreamSaver is able to save any size. But it has one limitation in that the service worker dies/restart after 10m. https://github.com/jimmywarting/StreamSaver.js/issues/39
So you need to be quick

What is the actual upper limit?

Chrome supports super large files now (>2GB). I'm closing this issue.


Add developer preview again, but point it to the introduction page.

QPPA-1709 Bring back the "Developer Preview" page in the dev docs
Add Developer Preview once again, but redirect it to the Introduction page until the CMS content team is able to update a CMS page that is linked to it.
Reviewer: @marimiyachi, @sarahnw

PR merged in commit b8944755dbd4e18923267be5d20fd59d00e7d4ed.


simplify configuration if rust source was installed via rustup

TL;DR

add a fallback when RUST_SRC_PATH is undefined and rustup is installed

background
rustup now comes with a way to install rust source code inside the root directory of each toolchain (https://github.com/rust-lang-nursery/rustup.rs/pull/659 and #626). rustup can be queried which toolchain is active and which binaries are used, which points to the toolchain directory. And rustc itself can be queried for the current toolchain root directory.
$ rustup which rustc
/home/colin/.multirust/toolchains/stable-x86_64-unknown-linux-gnu/bin/rustc
$ rustc --print sysroot
/home/colin/.multirust/toolchains/stable-x86_64-unknown-linux-gnu

If the rust source code was installed via rustup component add rust-src, then it will reside in
/home/colin/.multirust/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/src/
This can be rewritten in terms of the above query commands as either:

$(rustup which rustc | xargs dirname)/../lib/rustlib/src/rust/src/
$(rustc --print sysroot)/lib/rustlib/src/rust/src/

A direct query would be nicer of course, but this is all we have now.
suggestion
If RUST_SRC_PATH is undefined and rustup is installed,
then check if the source code is installed in the currently active toolchain (educated guess)

If source code is installed, then use it
else, give a hint like "please run rustup component add rust-src to install the rust source code"

discussion

more convenient first time setup
more convenient and powerful on subsequent toolchain changes, since it will automagically use the source code of the currently active toolchain
manual approach still possible

does this make sense to you?

We've actually got something like this in Racer already. The check_rust_sysroot() function appears to be exactly what you're recommending. There's not a published version of it yet, but that should be fixed soon.
Leaving this issue open as a reminder to fix docs

Ah cool. Btw, I also sent a pr to update the docs #626

Oh, great! Guess this can be closed then. Thanks for the write up :smile:

Well, if check_sys_root() lands, the docs should be updated again, because setting the environment variable wouldn't be necessary with the dust-up approach anymore, right?
Apart from that I am happy that this in the making. :-)


Documentation

Added optional requirements of sncosmo that we will be using

Coverage remained the same at 62.739% when pulling 6f23a2ede9650b44ca4f63c9eb633220d0fd9c23 on documentation into 535d7d2083511c420144b999cc0125b1c6ed64f6 on master.

Coverage remained the same at 62.739% when pulling 02bc12be9cdc53d6885de367b9f5f3b9c535bd40 on documentation into 535d7d2083511c420144b999cc0125b1c6ed64f6 on master.

demonstrate the LightCurve class using SNANASims
Added a script to generate requirements document from the conda and pip install script
Fixes #12 , #13

Coverage remained the same at 62.739% when pulling 95fc7b3ff6fd3f12630e33db269811174886155a on documentation into 535d7d2083511c420144b999cc0125b1c6ed64f6 on master.


New Tab Display text change

What's the Problem?
New Tab text change to home or root or some other relevant text.
No response
Solution/Idea

Alternatives
Instead of a new tab, type/display root or home. It will be more relevant. Because the new tab is suitable for blank pages, not for a page that is already showing drives.
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
Version: <IP_ADDRESS>
Windows Version
OS Version: 10.0.22000.194
Additional comment
No response

I'd agree with this, it's strange for the homepage to be called "Home" on the sidebar but "New tab" at the top.

Merging with #4139.


Upgrade text-buffer for setTextViaDiff fix

Refs https://github.com/atom/atom/issues/15225
@garretwilson This fixes a bug, which may be the cause of the problem you were seeing with atom-beautify, but I'm not yet positive.
I'm unfortunately going to be away from work until Monday evening, so I wanted to get this fix to you before I leave so you would have a chance to test it.

I need to go to bed, but once the windows CI build completes, you should be able to download a file called atom-windows.zip from this page.

Here is a direct download link.


Miss the option to print the PDF or store the result as PDF file

Hello,
Is any way to send the PDF character sheet to the printer or store it as a PDF file on the disk?
Best regards,
Yuri

In an older version, the PDF menu bar had a save option, and even now you can press ctrl-S to save a copy of the PDF, unfortunately the form data is not saved as part of that PDF.

I need to investigate, since any text entered by the user into the PDF since the window was last opened will appear in the saved PDF when ctrl-S is performed. So the issue seems to be that I'm not putting the data from the Actor into a place where it can be saved in by ctrl-S.

Using Ctrl-P to print and Ctrl-S to save a file (while the keyboard focus is in the PDF window) should work properly in version 0.48.1


Call Orsay native API

I would like to access the Orsay native API, it is possible through a toast app ?

Hi, @guipdutra
This will help you.
Thank you.

Thank you.


Question about GT pose of the provided replica datasets

Hello, thanks for your great job. I am curious about whether the 'traj.txt' is the ground truth pose or NICE-SLAM's result pose. If it is NICE-SLAM's result pose, could you provide the corresponding pose? Thank you!

It is the dataset from iMAP authors, traj.txt is the ground truth pose.


Info on stochasticity in Readme

Is the library deterministic? This would be a useful info to add to Readme.
Cheers!

Yes, it is deterministic. For the same text, the same language is always returned.


[Brackest auto-update] The update logs should be retained irrespective of whether the update is successful or failed

Description
[Brackest auto-update] The update logs should be retained irrespective of whether the update is successful or failed
Steps to Reproduce
Install Brackets 1.13
Click on Check for updates.
Click on Get it now.
Once download is complete, Click on Restart in the pop up
Go to  C:\Users<username>\AppData\Roaming\Brackets\updateTemp and check that there is a the log file generated
Once Brackets is auto launched go to Help->About Brackets
Verify that updated build number shown
Go to  C:\Users<username>\AppData\Roaming\Brackets\
Expected behavior: The log file should be retained irrespective of whether the update is successful or failed
Also the logs should be moved to C:\Users<username>\AppData\Local\Temp
Actual behavior: The log file is deleted once update is successful
Versions: Win 10
Release 1.13 build 1.13.0-17656

@DivyaBC , the removal of update logs on successful update is as per design. The intention is to not bloat up user's storage with heavy logs unless needed. Only in case of update failure is when we really need the logs to be retained, for further information about failure reasons. And so, if the update is successfully installed , keeping the logs is not needed.
Hence, not a bug.

@mbhavi , this ask was logged because we once saw an issue where the update was successful but on launching brackets, it was still the older version and because the logs were also cleared we could not figure out what went wrong. In such cases logs can be helpful if retained. hence, the ask to retain the logs irrespective of whether the update is successful or failure

we will keep the installer log within Brackets folder in Appdata.
On every subsequent auto update, we can cleanup previous installation logs.

Verified on 1.13 Prerelease Build 17689


docs: document non-zero amounts in State::increment_balances

Thought this would be good to add after debugging https://github.com/paradigmxyz/reth/pull/4840. This documents the same assumptions in CacheAccount::increment_balance, but in State::increment_balances as well. A debug_assert is added in increment_balance, to cause it to panic if the assumption is violated in tests.

Thought this would be good to add after debugging paradigmxyz/reth#4840. This documents the same assumptions in CacheAccount::increment_balance, but in State::increment_balances as well. A debug_assert is added in increment_balance, to cause it to panic if the assumption is violated in tests.

Added a commit that would skip zero balance increment.

Thought this would be good to add after debugging paradigmxyz/reth#4840. This documents the same assumptions in CacheAccount::increment_balance, but in State::increment_balances as well. A debug_assert is added in increment_balance, to cause it to panic if the assumption is violated in tests.

Added a commit that would skip zero balance increment. If you are okay with it, we can merge it

Yep, that should work!


chore: regex to match all go files

This PR
Adds go files for regex pattern

Ya I think this is a good idea


Improve account_balance indexing

The account_balance indexing is currently defined as:
  typedef multi_index_container<
     account_balance_object,
     indexed_by<
        ordered_unique< tag<by_id>, member< object, object_id_type, &object::id > >,
        ordered_unique< tag<by_balance>, composite_key<
                                            account_balance_object,
                                            member<account_balance_object, account_id_type, &account_balance_object::owner>,
                                            member<account_balance_object, asset_id_type, &account_balance_object::asset_type> >
                        >,
        ordered_non_unique< tag<by_account>, member<account_balance_object, account_id_type, &account_balance_object::owner> >,
        ordered_non_unique< tag<by_asset>, member<account_balance_object, asset_id_type, &account_balance_object::asset_type> >
        >
     > account_balance_object_multi_index_type;

The by_account index is redundant, as the by_balance index (which could be better named by_account_asset) can answer any query the by_account index can.
In addition, an index by (asset_type, balance) is required to support top_n authority.  Again, the by_asset functionality can be assumed by this new index.
So the solution is:

Rename by_balance to by_account_asset
Remove by_account and rewrite logic to use by_account_asset instead
Expand by_asset to composite key on (asset_type, balance)

The equal_range() of boost::multi_index_container will be useful for the new logic.

We need to have composite_key_compare like this:
        composite_key_compare<
           std::less< asset_id_type >,
           std::greater< share_type >,
           std::less< account_id_type >
        >

so that the ordering is increasing asset ID, decreasing balance within an asset ID, and increasing account ID.  So effectively the "rich list" for each asset, with ID as tiebreaker (a lower ID is ranked above a higher ID with equal balance).

That was easier than I expected.  Apparently by_asset wasn't actually being used by anything!


AssertionError in 2.3.1 when context='forkserver' / 'spawn' in Linux

Hi @dubovikmaster,
Our system updated to the latest 2.3.1, but I get the following assertion on Linux.
AssertionError: daemonic processes are not allowed to have children

Here's the exception stacktrace (in child process):
Traceback (most recent call last):
  File "/usr/lib/python3.8/multiprocessing/process.py", line 315, in _bootstrap
    self.run()
  File "/usr/lib/python3.8/multiprocessing/process.py", line 108, in run
    self._target(*self._args, **self._kwargs)
  File "/usr/lib/python3.8/multiprocessing/pool.py", line 114, in worker
    task = get()
  File "/usr/lib/python3.8/multiprocessing/queues.py", line 358, in get
    return _ForkingPickler.loads(res)
  File "/home/ubuntu/.local/lib/python3.8/site-packages/parallelbar/__init__.py", line 1, in <module>
    from .parallelbar import progress_map, progress_imap, progress_imapu, progress_starmap
  File "/home/ubuntu/.local/lib/python3.8/site-packages/parallelbar/parallelbar.py", line 15, in <module>
    from .wrappers import (
  File "/home/ubuntu/.local/lib/python3.8/site-packages/parallelbar/wrappers.py", line 10, in <module>
    from ._worker_queue import _WORKER_QUEUE
  File "/home/ubuntu/.local/lib/python3.8/site-packages/parallelbar/_worker_queue.py", line 7, in <module>
    _WORKER_QUEUE = mp.Manager().Queue()
  File "/usr/lib/python3.8/multiprocessing/context.py", line 57, in Manager
    m.start()
  File "/usr/lib/python3.8/multiprocessing/managers.py", line 579, in start
    self._process.start()
  File "/usr/lib/python3.8/multiprocessing/process.py", line 118, in start
    assert not _current_process._config.get('daemon'), \

From the stack it seems the assertion is because the child process loads the parallelbar module too, and the new version now always creates a Queue(). Is there anyway to avoid creating Queue() object in child process?

This issue doesn't occur when using context=fork,  but in my case I need use forkserver otherwise fsspec will hang

Hello! Can you give me a code example where you use parallelbar?


Adding UI tests for adding a feature

Fixes #120
Adds an instrumented test for adding a feature. Tests are constructed using the GivenWhenThen style. For example:

Test name: Add a feature
Given: a logged in user - with an active project which doesn't direct the user to add an observation when adding a feature.
When: they tap the "Add feature" FAB and choose a feature type.
Then: the observation marker is displayed in the center of the map screen.

Implementation details

[x] Refactor the authentication process so that a FakeAuthenticationManager can provide a fake user during tests
[x] Create a FakeRemoteDataStore to provide fake Project and Layer objects to tests
[x] Use trampoline as the io and ui Rx schedulers
[x] Add RxIdler
[x] Add databinding idling resource

Tests:

[x] Verify that tapping on the map does nothing when initially loaded
[x] Verify that adding a feature adds a map pin at the current crosshair position, and that tapping on that feature loads its details

Verification steps

Run the tests on an emulator or real device

@gino-m @shobhitagarwal1612 @scolsen PTAL.

Hi Don, what's the status on this? Did adding a longer wait in Thread.sleep() resolve the flakiness, at least temporarily?

Also, @JoseAlcerreca, is this the best we can do to work around async operations for now, or did we miss something?

@gino-m Myself and @JoseAlcerreca did some extensive investigation into this (PR here). The current conclusion is that whilst we can minimise the flakiness of tests by using idling resources the Thread.sleep is unavoidable.
I will selectively merge in the following code from the other PR:

Idling resource for databindings
Idling resource for Rx
BottomSheetBehavior class which has the capability of animations being disabled for testing

I will also experiment with reducing the sleep time so that we're not waiting unnecessarily for tests to finish.

I ended up removing the custom BottomSheetBehavior class because with the data binding idling resource the test will wait for the animation to finish.
I also figured that it was better to rely on the material version of BottomSheetBehavior than our own implementation (especially given all the issues we've had with the bottom sheet).
I'd like to merge this now as I think it provides a stable enough framework for UI testing.

🎉 🎉 🎉


Issue finding model

Hi, Thank you for making your great software available
I run into problems when  trying to get thin slices from thick slices of Head CT data
I installed KevinSR in a conda python 3.6 env on Ubuntu 22.04
I cloned the repo in Pycharms and run the code on CPU
When running "thins_slices = SOUP_GAN(thicks_ori, Z_FAC, 1)" I get  :
 Traceback (most recent call last):
  File "/home/dell/PycharmProjects/SOUP-GAN/Example_MF.py", line 25, in <module>
    thins_slices = SOUP_GAN(thicks_ori, Z_FAC, 1)
TypeError: SOUP_GAN() takes 2 positional arguments but 3 were given

When running thins_slices = SOUP_GAN(thicks_ori, Z_FAC) I get

File "/home/dell/PycharmProjects/SOUP-GAN/Example_MF.py", line 25, in
thins_slices = SOUP_GAN(thicks_ori, Z_FAC)
File "/home/dell/anaconda3/envs/KevinSR/lib/python3.6/site-packages/KevinSR/SOUP.py", line 21, in SOUP_GAN
new_model=keras.models.load_model('./Thin-to-thin')
File "/home/dell/anaconda3/envs/KevinSR/lib/python3.6/site-packages/keras/saving/save.py", line 205, in load_model
return saved_model_load.load(filepath, compile, options)
File "/home/dell/anaconda3/envs/KevinSR/lib/python3.6/site-packages/keras/saving/saved_model/load.py", line 108, in load
meta_graph_def = tf.internal.saved_model.parse_saved_model(path).meta_graphs[0]
File "/home/dell/anaconda3/envs/KevinSR/lib/python3.6/site-packages/tensorflow/python/saved_model/loader_impl.py", line 121, in parse_saved_model
constants.SAVED_MODEL_FILENAME_PB))
OSError: SavedModel file does not exist at: ./Thin-to-thin/{saved_model.pbtxt|saved_model.pb}
Process finished with exit code 1

This is the definition of the SOUP_GAN function in the SOUP.py file :

def SOUP_GAN(thicks_ori, Z_FAC):
thicks_ori = rescale_img(thicks_ori, max_val= 10000)

thins = zoom(thicks_ori, (1,1,Z_FAC))

new_model=keras.models.load_model('./Thin-to-thin')
thins_gen = thins.copy()

target = np.moveaxis(thins,-1,0)
target = target [...,np.newaxis]
target = target [np.newaxis,...]

index  = attention_coeff(target, Z_FAC)

target = new_model.predict([target,index])

target_small = target[0,...,0]
thins_gen = np.moveaxis(target_small, 0,-1)

return thins_gen

My understanding is that the SOUP_GAN function
-only takes two arguments ( not three as in example scripts)
-a Thin-to-Thin model by default and that this model is not installed with KevinSR
Is there any way to solve this ? Should I train a model on my own data ?
Best regards
Matthieu

Thanks for your questions. Below are my comments:
-only takes two arguments ( not three as in example scripts)
Actually, SOUP_GAN() takes three arguments, for example: CT1_sr = SOUP_GAN(CT1_thick, rb, 0)._
-a Thin-to-Thin model by default and that this model is not installed with KevinSR
The function with two types 0: thick-to-thin; 1: thin-to-thin are by default installed with KevnSR.
-Is there any way to solve this ? Should I train a model on my own data ?
Fine-tuning on user's data may improve the performance. Yet, the model was designed to be used directly. Please first check the version of KevinSR installed to see if it is up to date:  https://pypi.org/project/KevinSR/
Print dimensions of variables: thicks_ori, Z_FAC could help better understand the issue.

Thanks for your quick response
KevinSR version was 0.0.9, I think because the environment used Python3.6
and when typing "pip3 install KevinSR" it displayed error message for
installation of matplotlib (could not find version > 3.4)
I created new environment with python3.9
It is now working on CPU although I experience a lot of (memory ?) errors
when working with too many slices (or Z_FAC >3) : Process finished with
exit code 134 (interrupted by signal 6: SIGABRT)
Is there any way to export the predicted images to dicom ?
Le sam. 17 sept. 2022 à 22:30, Kevin @.***> a écrit :

Thanks for your questions. Below are my comments:
-only takes two arguments ( not three as in example scripts)
Actually, SOUP_GAN() takes three arguments, for example: CT1_sr =
SOUP_GAN(CT1_thick, rb, 0)._
-a Thin-to-Thin model by default and that this model is not installed with
KevinSR
The function with two types 0: thick-to-thin; 1: thin-to-thin are by
default installed with KevnSR.
-Is there any way to solve this ? Should I train a model on my own data ?
Fine-tuning on user's data may improve the performance. Yet, the model was
designed to be used directly. Please first check the version of KevinSR
installed to see if it is up to date: https://pypi.org/project/KevinSR/
Print dimensions of variables: thicks_ori, Z_FAC could help better
understand the issue.
—
Reply to this email directly, view it on GitHub
https://github.com/Mayo-Radiology-Informatics-Lab/SOUP-GAN/issues/2#issuecomment-1250136154,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHIU4FT5747T6ZDCGYRBYZTV6YS75ANCNFSM6AAAAAAQO7RYLI
.
You are receiving this because you authored the thread.Message ID:
@.***>


Univariate bivariate analysis

Linked Issue
Closes: #4
Describe the changes you've made
For Univariate Analysis added a scatter plot showing hazardous and nonhazardous for each column in the training set.
For Bivariate Analysis added scatter matrix, scatter matrix with regression line and scatter matrix showing hazardous and non-hazardous.
Type of change
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Yes, It has been tested.
Checklist:

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[ ] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] Any dependent changes have been merged and published in downstream modules.

Screenshots
Univariate Analysis

Bivariate Analysis

@abhishak3 Please resolve conflict in the PR. Otherwise it can't be merged.


docs: add hostnfly to company list

From old PR #4011

LGTM

Codecov Report

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

@@           Coverage Diff           @@
##           master    #7065   +/-   ##
=======================================
  Coverage   64.46%   64.46%
=======================================
  Files         421      421
  Lines       20537    20537
  Branches     2247     2247
=======================================
  Hits        13240    13240
  Misses       7170     7170
  Partials      127      127

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 84109be...8fbb329. Read the comment docs.


Shardeum Liberty 2.X Testnet

Website
https://docs.shardeum.org/network/endpoints#liberty-2x
ChainId
8081
RPC
https://liberty20.shardeum.org/
Block Explorer
https://explorer-liberty20.shardeum.org/

RPC is not working for us

We have deprecated this network. Closing this Issue and opening a  new issue with the new network details.


Parameter refactor v4

Some pretty big changes in here...

[x] Converting params and props to NamedTuples
[x] Removing chex dependency
[x] Rewriting to/from_unconstrained to use tree_map
[x] Rewriting log_det_jacobian_constrain to use tree_map
[x] Making all inference types NamedTuples
[x] Making HMM smoother functions compute transition probabilities by default (kwarg specified)
[x] Doing some general cleanup of the typing
[x] Removing unused imports in many files
[x] Fixing HMC notebook to use log det Jacobian
[x] Updating tests and notebooks to make sure everything still runs

I tried a few ways of combining parameters and their properties into one object, but in the end I think this is the most transparent approach. I converted them all to NamedTuple, which seems like JAX best practice. Note that props are still mutable, which could in theory cause some problems if you jit a function that closes over ParameterProperties and then change them.

Looks great! (See comments in slack.)
There are 2 failing tests, but I will merge anyway, and we can resolve this later.


Why would Firewall NAT traffic to public ip?

 From there, the firewall applies its firewall rules and NATs the packets to your load balancer's public IP address. "
Did you mean private? I am confused. Please explain.

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

ID: 07b33cf5-3747-59b6-29b4-9aa076a6d271
Version Independent ID: ddd15c52-275e-68e6-513e-07d42292b059
Content: Integrate Azure Firewall with Azure Standard Load Balancer
Content Source: articles/firewall/integrate-lb.md
Service: firewall
GitHub Login: @vhorne
Microsoft Alias: victorh

@Chai-NED In this scenario, the public Load Balancer can only load balance requests sent to the Public IP of the Load Balancer.
Traffic comes in via the Firewall's Public IP Address, and firewall rules are applied. Next, it goes to the Load Balancer's Public IP Address and load balancing rules are applied. It is then sent to a machine in the backend of the load balancer.

@Chai-NED We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Why are we converting an image from rgb to yuv??

I want to know the reason that why we are converting an image from rgb to yuv. What is the advantage of doing that before passing the image to the network??

You are separating the Y (Luminosity) from the U,V (Color Difference) Channels.
In RGB, all three channels contain almost the same amount of "sharpness" information, while in YUV, the Y channel mostly contain the information you need, with U and V being the colours only.
For example, in most ConvNet applications, the Y channel is always used (as it is the black and white version of that image), and the U and V channels can sometimes be discarded, or used in a different network that checks for colour.


cmd/go: test failure during build on 64-bit Fedora 28 after commit faafcc606e

What version of Go are you using (go version)?
go version devel +faafcc606e Wed Oct 31 00:17:00 2018 +0000 linux/amd64
This happens with this commit and later, and doesn't happen with 0ad332d80c and earlier.
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOOS="linux"

What did you do?
Running ./all.bash in a current git tip from faafcc606e onwards results in a test failure during build:
ok      cmd/fix 7.893s
go test proxy starting
go test proxy running at GOPROXY=http://<IP_ADDRESS>:34383/mod
go proxy: no archive rsc.io v1.5.0
go proxy: no archive example.com/split/subpkg v1.0.0
go proxy: no archive golang.org/x/text/language 14c0d48
go proxy: no archive golang.org/x/text/language 14c0d48
go proxy: no archive golang.org/x/text/language 14c0d48
go proxy: no archive golang.org/x/text/foo 14c0d48
go proxy: no archive golang.org/x 14c0d48
go proxy: no archive golang.org 14c0d48
--- FAIL: TestScript (0.00s)
    --- FAIL: TestScript/vendor_complex (0.56s)
        script_test.go:170:
            # smoke test for complex build configuration (0.547s)
            > go build -o complex.exe complex
            > [exec:gccgo] go build -compiler=gccgo -o complex.exe complex
            [stderr]
            # complex/nest/vendor/v1
            ar: no operation specified
            # complex/nest/vendor/v3
            ar: no operation specified
            # complex/vendor/v
            ar: no operation specified
            # complex/nest/sub/vendor/v2
            ar: no operation specified
            # complex/w
            ar: no operation specified
            [exit status 2]
            FAIL: testdata/script/vendor_complex.txt:3: unexpected command failure

FAIL
FAIL    cmd/go  36.895s

Fedora 28 has ar version 2.29.1-23.fc28.
Looking at the code change in that commit, I suspect that what is going wrong here is that Builder.run is being called with a null string as one of the command line arguments, and this is being passed on to ar, and it objects to being invoked as, effectively, ar "" rc .... I can reproduce a similar failure with ar by hand, for example:
$ ar "" t ./test/issue18902b.o
ar: no operation specified
$ ar t ./test/issue18902b.o
__.PKGDEF
_go_.o
$

CC @Helflym


⚠ Jenkins Plugins Artifactory has degraded performance

In 688e536, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1195 ms

Resolved: Jenkins Plugins Artifactory performance has improved in e74edd5 after 1 hour, 35 minutes.


Linux: RHost process does not get killed after disconnect

This was fixed when we switched back to use standard io streams.

This still repros. This is because broker runs as 'brokeruser', and hence does not have permission to kill the process running as another user.

While we do need to fix the ability of broker to kill processes, the host should also exit by itself when the broker closes its end of the stdout pipe. If that's not happening, it's a separate bug.

The host seems to run until either broker kills it by calling kill or when the broker itself exits. I think there might be a bug on the broker side. I will look into it.


data-bs-* attributes not working in Toast Component in v5.3.3 and v5.0.2

Prerequisites

[X] I have searched for duplicate or closed issues
[X] I have validated any HTML to avoid common problems
[X] I have read the contributing guidelines

Describe the issue
Whenever I configure toast with the data-bs-* attributes such as autohide, delay and animation they are not parsed. I even tried the element but no luck. I also tried in both variations of bundle.js and with popper.js & bootstrap.js.
The expectation is that the data attributes should be parsed and the component should be initialized without me writing no additional Javascript. I also tried to initialize the toast div, but no outcome.
Please find the code below and advise.
<!DOCTYPE html>

    <link<EMAIL_ADDRESS>rel="stylesheet" integrity="sha384-EVSTQN3/azprG1Anm3QDgpJLIm9Nao0Yz1ztcQTwFspd3yD65VohhpuuCOmLASjC" crossorigin="anonymous">

Test Bootstrap Data Attributes

Bootstrap 5 Toasts Options

            <button type="button"
                       class="btn btn-success"
                       onclick="showToast()">
                Show the Toast
            </button>

                  Bootstrap
                  11 mins ago
                  <button type="button" class="btn-close" data-bs-dismiss="toast" aria-label="Close"></button>

                  Hello, world! This is a toast message.

Reduced test cases
Please refer to the source code in the above issue description.
What operating system(s) are you seeing the problem on?
macOS
What browser(s) are you seeing the problem on?
Chrome, Safari
What version of Bootstrap are you using?
v5.0.2, v5.3.3

Hello @akhileshThapliyal
I've created a CodePen from your example, and data-bs-autohide="true" data-bs-animation="false" data-bs-delay="200" seem to work pretty well: the toast auto hides, has no animation, and is closed after 200ms.
I'm not sure to get what's the issue.

data-bs-autohide="true"

I want to show the toast by default, when I use data-bs-autohide="false" and initialise the toast using the new bootstrap.Toast(myToastEl); it doesn't show up. I have to then explicitly call .show() API. This is the problem.

Hey there,
I feel like the title and description of this issue might not be related to your actual problem because I can't see any issue with the behavior of the data-bs-* attributes compared to what's in our documentation right now. If I understand well, you're asking for a new feature that would display the toast by default without writing any Javascript ?

Hey there,
I feel like the title and description of this issue might not be related to your actual problem because I can't see any issue with the behavior of the data-bs-* attributes compared to what's in our documentation right now. If I understand well, you're asking for a new feature that would display the toast by default without writing any Javascript ?

yes, correct that is what the document also says's correct? The data attributes are parsed and evaluated by the bootstrap scripts. Hence, I thought that setting the autohide to false would show up on the page load by default :(.

As the documentation mentions,

autohide | boolean | true | Automatically hide the toast after the delay.

The autohide option avoid the toast to hide after the delay, but you need to display it first by calling the show() method. The documentation feels explicit enough to me. TBH I don't think we would implement such an option to avoid writing 3 lines of JS.
I'm closing it for now.

As the documentation mentions,

autohide | boolean | true | Automatically hide the toast after the delay.

The autohide option avoid the toast to hide after the delay, but you need to display it first by calling the show() method. The documentation feels explicit enough to me. TBH I don't think we would implement such an option to avoid writing 3 lines of JS.
I'm closing it for now.

Noted, thank you :)


how to go back to the default page via f7-link?

		<f7-toolbar labels>
			<f7-link icon-f7="home" text="home" href="/"></f7-link>
			<f7-link icon-f7="favorites" text="list" href="/list"></f7-link>
			<f7-link icon-f7="document_text_fill" text="chart" href="/chart"></f7-link>
		</f7-toolbar>

{
    path: '/chart/',
    component: require('./assets/vue/pages/chart.vue')
  },
  {
    path: '/list/',
    component: require('./assets/vue/pages/list.vue')
  },

link to other route is ok, but can't go back to the default first page, that page is main.vue, include f7-statusbar and #app(seems like main layout), so shell i set route '/' as main.vue? i dont think that's author purpose.
how should i do?

otherwise, i can make main.vue as a assets/pages,  on this way i can router pages as my wish.
but there has another problem as first enter the page is blank page~ because main.vue really dont load assets/pages/main.vue as default.....
and i tried to set redirect / to /main, seems not work

@fishenal any luck with this ?

Check how it is done in Kitchen sink at https://github.com/nolimits4web/Framework7-Vue/blob/master/kitchen-sink/index.html#L38-L41. There is main route specified for this https://github.com/nolimits4web/Framework7-Vue/blob/master/kitchen-sink/routes.js#L58-L60

well, seems like the key is to set a default 'url' in , i will try this, thanks very much.


Add Apache licenses to Python files - Part 1

Adding proper Apache license headers to all Python (.py) files.
Doing this in stages, this PR being "part 1" to make sure this works and reviewers can get a general feeling for what we are doing and provide comment before doing the rest.
Specifically, this PR updates the following files:
./openwhisk/core/actionProxy/actionproxy.py
./openwhisk/core/actionProxy/invoke.py
./openwhisk/core/pythonAction/cli/packagescripts/setup.py
./openwhisk/core/pythonAction/pythonrunner.py
./openwhisk/tools/build/checkLogs.py
./openwhisk/tools/build/scanCode.py
./openwhisk/tools/travis/box-upload.py
Subsequent PRs would then similarly address the remaining Python files. Specifically:
./openwhisk/ansible/callbacks/logformatter.py
./openwhisk/core/javaAction/init.py
./openwhisk/core/javaAction/run.py
./openwhisk/core/pythonAction/cli/packagescripts/init.py
./openwhisk/core/pythonAction/cli/wskaction.py
./openwhisk/core/pythonAction/cli/wskactivation.py
./openwhisk/core/pythonAction/cli/wskitem.py
./openwhisk/core/pythonAction/cli/wsknamespace.py
./openwhisk/core/pythonAction/cli/wskpackage.py
./openwhisk/core/pythonAction/cli/wskprop.py
./openwhisk/core/pythonAction/cli/wskrule.py
./openwhisk/core/pythonAction/cli/wsksdk.py
./openwhisk/core/pythonAction/cli/wsktrigger.py
./openwhisk/core/pythonAction/cli/wskutil.py
./openwhisk/core/swift3Action/buildandrecord.py
./openwhisk/core/swift3Action/swift3runner.py
./openwhisk/tests/dat/actions/hello.py
./openwhisk/tests/dat/actions/malformed.py
./openwhisk/tests/dat/actions/niam.py
./openwhisk/tests/dat/actions/stdenv.py
./openwhisk/tests/dat/actions/timedout.py
./openwhisk/tests/dat/actions/unicode.py
./openwhisk/tests/dat/blackbox/badaction/runner.py
./openwhisk/tools/admin/wskprop.py
./openwhisk/tools/admin/wskutil.py
./openwhisk/tools/db/replicateDbs.py
./openwhisk/tools/health/monitorUtil.py
./openwhisk/tools/json/validate.py

@mrutkows please take this opportunity and delete pythonAction/cli? (These files are redundant with the openwhisk-python-cli repo.)
@csantanapr FYI.

Yes 💣  pythonAction/cli

So delete all files in the pythonAction/cli subdir...

Deleted the CLI directory as requested.  If this PR is acceptable, please merge; I have "part 2" saved locally and would like to submit those on a separate PR.

Generally ok with this - shouldn't the license come first (top of the file) vs after some comments?
Are you intending for scanCode to work with these scripts?

scanCode should work this scripts, I think that's @markusthoemmes intention in his next PR.
merging this for now, and looking forward to learn some more about python and parsing the header @markusthoemmes
Going to mer this for now.

going to run a PG

I think that's @markusthoemmes intention in his next PR.

really?

PG3 360

merged ff8a88f97e4443e7d0f1853e5517acb17a2848ce

really? Did you mean @mrutkows?

What are you talking about? Your are imaging things 😸


The game crashed whilst initializing game Error: java.lang.IncompatibleClassChangeError: class dev.su5ed.sinytra.connector.mod.compat.fieldtypes.RedirectingIdMapper overrides final method net.minecraft.core.IdMapper.m_7942_(I)Ljava/lang/Object;

Describe the bug
Each time I try to run minecraft with the 47.1.3 forge client, it never makes it past the mod loading section. I originally had 47.1.30 downloaded on accident, but after reinstalling my client I got this error message instead. I am trying to run better end, and i have the extra dependencies installed besides the fabric api, because im using the forgified version. I tried looking around to see of this had been an issue for anyone else, but it looked like a quick fix by changing the client version.
Steps to reproduce

forge 47.1.3 client
better-end-2.1.6.jar
cave_dweller-1.20.1-1.6.4
Connector-1.0.0-beta.21+1.20.1-full
ConnectorExtras-1.4.1+1.20.1
fabric-api-0.90.0+1.9.28+1.20.1
FarmersDelight-1.20.1-1.2.3
geckolib-forge-1.20.1-4.2.3
bclib-3.0.13

all in mods folder
3. open minecraft launcher
4. run 47.1.3 client
5. watch it load until it gets to the mods section
6. crash
...
Logs

Additional context
No response

Try FFAPI 26, instead of 28. Better end works with connector, as I'm currently using it.

Please upload your debug.log

https://gist.github.com/neostasis/0c314e857388aa0a4e07b12a8c3742b4
This should be the correct link I think

That is still the same crash log. The log I asked for can be found at config/debug.log

I just found out i had the wrong version of better end downloaded after looking at the debug log, i think this should fix it. Here's the log anyways, though
debug.log


Change master to main

According to the new guidelines of awesome list we need to change the name of the default branch. See https://github.com/sindresorhus/awesome/pull/1798#issuecomment-644338164

I am very sorry for opening this issue. I fail to find this requirement in the new guidelines.

Since, it is a small project and a small change - if it makes people more comfortable, we can do it.

Done!


Batch Receive Messages is 0  after several time

Describe the bug
First I produce 1,000,000 messages into the topic and batch receive from it , but not ack them.
After one or two batch receive , I get  almost 250,000 messages ,  and later the num of batch receive messages is 0.
To Reproduce
batch policy
        consumer = client.newConsumer()
                .topic(topic)
                .subscriptionName(getRequired(JobPropKey.consumerGroup))
                .subscriptionType(SubscriptionType.Shared)
                .consumerName(DefaultValue.appName)
                .negativeAckRedeliveryDelay(5, TimeUnit.SECONDS)
                .receiverQueueSize(1000000)
                .batchReceivePolicy(BatchReceivePolicy.builder()
                        .maxNumMessages(1000000)
                        .maxNumBytes(1024 * 1024 * 512)
                        .timeout(10000, TimeUnit.MILLISECONDS)
                        .build())
                .subscribe();

Expected behavior
I just want to receive the messages
Screenshots
first receive

second receive

Desktop (please complete the following information):

OS: Linux Mac
Pulsar 2.7.0

Hi, can you run admin command pulsar-admin topics stats [topicname] to get the topic status and share the result? If broker has been restarted you might need to reproduce the issue so the topics stats is accurate.
And can you share the broker config, especially the message TTL, message retention and backlog quota settings.

The issue had no activity for 30 days, mark with Stale label.

Closing as stale. If it's still relevant to maintained versions, feel free to open a new issue


Feature/refresh token

Refresh token implementation works well. The frontend should update the returned refresh_token on the response from POST /auth/token, but I'll do this later. Anyway, the backend API returns the given refresh_token now.
Still, the backend API does not recognize the preview page generated by Firebase Hosting. But allowing CORS from the preview pages does not seem secure. I'll


[BUG] - clone instructions reference old syntax

Description
The clone step in the main readme is using an old syntax "--recursive".
Expected Behavior
The more appropriate option nowadays is "--recurse-submodule" as of git v2.13, which was released back in 2017.

Implemented the proposal. Thanks!


Virtual machine

So lets say . Currently this program doesn't doesn't support alt tab Or minimize right?,
I have an idea where the virtual machine come in?
Currently im trying to researching how to mount the drive directly to download folder intead of exporting it..

I don't understand your original post. Can you explain your problem more clearly?

sory to bother you..

I believe the tkinter window can run minimized. Perhaps I am misunderstanding?

oh it not the tkinter but the virtual box im minimize it, but then again im done with this issue thx tho for your help


[5.0] Confusion about exception thrown by ValidateRequests

The documentation says ValidateRequests throws an Illuminate\Contracts\Validation\ValidationException, whereas it actually throws an Illuminate\Http\Exception\HttpResponseException.

We're open to pull requests.

I was unsure how to fix it, otherwise I would have submitted a PR, whereas I'm confident Taylor could quite easily fix it.

The docs just need updating. It's too late to move the class about now.

For the record,

Documentation since 5.1 simply doesn't detail the type of exception
Laravel 5.2 will actually throw ValidationException

This just caught me out as well. It seems inconsistent that ValidatesRequests throws a ValidatorException and FormRequest throws a HttpResponseException
Shouldn't HttpResponseException be added to the dontReport at least ?


[FORM] Feature request

Suggestion/Submission below
add proxies amd fix discord

Add your own proxy, and discord should be fine


fix: enhance state gradient when bf16

FIX: https://github.com/sustcsonglin/flash-linear-attention/issues/36

Greetings maintainers, thank you for your work.
In this PR, I have fixed the issue of inconsistent gradients in the state of chunkrwkv (the original implementation had an absolute deviation that was 200-3000 times the original). The new implementation has an error of 1%-3%, which is simply achieved by introducing fp32 at a critical location.
After benchmarking, the performance is at the same level, with no significant decrease or improvement.
I have also updated the algorithm for measuring gradient error, and now use RMS to evaluate it. There is still a problem of large gradient errors in w and u, but considering the approximate implementation of chunk, this may be acceptable (for example, in my use case, I only need to backpropagate through the state, i.e. h).


Add .mailmap aliases to identify authors uniquely

See https://git-scm.com/docs/git-check-mailmap#_mapping_authors
How to categorize this PR?
/area open-source
/kind cleanup
/priority normal
What this PR does / why we need it:
Ensure authors are identified uniquely in git.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:
NONE

What's the motivation for this, i.e., why do we want to maintain such file in our repo(s)? Besides the fact that it contains duplicates, it will come with some maintenance effort. Would we also put it into our 100+ other repos? Is there an automatism that would maintain it?

ping @msohn

@msohn, any statement on above questions?

Run git shortlog -sne to see what this tries to fix. Apparently many authors use multiple identities (name, email address) in git commits they contribute to gardener. .mailmap is used by git shortlog [1] and git log --mailmap [2] to map multiple git identities of the same user to a single name and email address. You can also set log.mailmap=true in ~/.gitconfig to always use the mappings. I generated the draft of this .mailmap using [3].
I do not insist, so if you don't want to maintain such mapping simply close this PR.
If you are interested to maintain this I can try to generate this on all gardener repositories and create a merged .mailmap across all gardener repositories and we could check in the resulting .mailmap into all repositories.
[1] https://git-scm.com/docs/git-shortlog#_mapping_authors
[2] https://git-scm.com/docs/git-log#Documentation/git-log.txt---no-mailmap
[3] https://github.com/greenrd/genmailmap

Thanks for the explanation. If we would take this serious then we would need automation that maintains these mailmap files across all repositories. Otherwise, the content might get outdated eventually. If we have such automation then I'm fine with it, but I'd refrain from having it if the content is just manually generated and maintained because this cannot scale. @gardener/gardener-maintainers

I think this isn't fully automatable since some human needs to decide which identities belong to the same person which isn't always obvious

I would like to understand if this is just a cosmetic optimization for the case that someone runs git log.
Or is this used somewhere else (e.g. GitHub or some other processing)?

@msohn can you clarify on @timebertt's question so that we can decide how to proceed, please?

I do not like very much this mapping because it clearly exposes information that maybe not everybody would like to show publicly. Of course, the mails can be always retrieved from the commit metadata, except if the private email feature is used. Maybe the CLA also does not allow private email?

OK, let's
/close
the PR for now. It seems that the majority is against maintaining such a list.


Find a way to run the same tests multiple times with different settings

TenantStorageTest for each storage driver
All tests with predis and phpredis


[BUG]Knowledge Base sync fails

Describe the bug
I try to deploy 03-agent-with-kb-and-guardrails
git clone https://github.com/aws-samples/amazon-bedrock-samples.git
cd amazon-bedrock-samples/agents-for-bedrock/agent-blueprint-templates/
./blueprints.sh init
source ~/.bashrc
aws configure
./blueprints.sh deploy 03-agent-with-kb-and-guardrails

Cfn deploy is success, but Knowledge Base sync fails
Error messages in Sync history
Encountered error: Access Denied (Service: S3, Status Code: 403, Request ID: 3C7W8SDZFXKRB5E5, Extended Request ID: vTA/SackaEevUIZtZMMqHdntIhr97qSdSl45//+4v4jl799qH0FwKM6mvgakWDwCoon89zILntw=). Call to Amazon S3 Source did not succeed.

I check service role of KnowledgeBase.
{
    "Version": "2012-10-17",
    "Statement": [
        {
            "Action": "bedrock:InvokeModel",
            "Resource": "arn:aws:bedrock:us-east-1::foundation-model/amazon.titan-embed-text-v1",
            "Effect": "Allow",
            "Sid": "AllowKBToInvokeEmbedding"
        },
        {
            "Action": "aoss:APIAccessAll",
            "Resource": "arn:aws:aoss:us-east-1:637423213562:collection/ccygcjufi7yyog6ks2q9",
            "Effect": "Allow",
            "Sid": "AllowKBToAccessAOSS"
        },
        {
            "Condition": {
                "StringEquals": {
                    "aws:SourceAccount": "637423213562"
                }
            },
            "Action": [
                "s3:GetObject",
                "s3:ListBucket"
            ],
            "Resource": [
                "arn:aws:s3:::agentwithkbandguardrailss-agenticragstackagentblue-xgal3cstax2l/*",
                "arn:aws:s3:::agentwithkbandguardrailss-agenticragstackagentblue-xgal3cstax2l"
            ],
            "Effect": "Allow",
            "Sid": "AllowKBToAccessAssets"
        }
    ]
}

For comparison, we created a new knowledge base using the Management Console, and check it.
{
    "Version": "2012-10-17",
    "Statement": [
        {
            "Sid": "S3ListBucketStatement",
            "Effect": "Allow",
            "Action": [
                "s3:ListBucket"
            ],
            "Resource": [
                "arn:aws:s3:::bedrock-637423213562"
            ],
            "Condition": {
                "StringEquals": {
                    "aws:ResourceAccount": [
                        "637423213562"
                    ]
                }
            }
        },
        {
            "Sid": "S3GetObjectStatement",
            "Effect": "Allow",
            "Action": [
                "s3:GetObject"
            ],
            "Resource": [
                "arn:aws:s3:::bedrock-637423213562/kb-0721/",
                "arn:aws:s3:::bedrock-kb-custom-transformation/*",
                "arn:aws:s3:::bedrock-637423213562/kb-0719/*",
                "arn:aws:s3:::bedrock-637423213562/kb-0721/*"
            ],
            "Condition": {
                "StringEquals": {
                    "aws:ResourceAccount": [
                        "637423213562"
                    ]
                }
            }
        },
        {
            "Sid": "S3PutObjectStatement",
            "Effect": "Allow",
            "Action": [
                "s3:PutObject"
            ],
            "Resource": [
                "arn:aws:s3:::bedrock-kb-custom-transformation/*"
            ],
            "Condition": {
                "StringEquals": {
                    "aws:ResourceAccount": "637423213562"
                }
            }
        }
    ]
}

I found that there is a difference in the condition block
Create by agent blueprint
            "Condition": {
                "StringEquals": {
                    "aws:SourceAccount": "637423213562"
                }
            },

Create by management console
            "Condition": {
                "StringEquals": {
                    "aws:ResourceAccount": "637423213562"
                }
            }

I change aws:SourceAccount to aws:ResourceAccount , then retry is success.
Is this a specified behavior?
Target
aws --version
aws-cli/2.17.32 Python/3.11.9 Linux/<IP_ADDRESS>-microsoft-standard-WSL2 exe/x86_64.ubuntu.22
cdk version
2.152.0 (build faa7d79)
Host
Ubuntu 22.04 on WSL2

Hello, I fixed it removing the account check there https://github.com/awslabs/agents-for-amazon-bedrock-blueprints/commit/2b762b14d4c121172a921c5b21c13861db8ddd81
but your solution seems actually better. Did you check if it addresses the problem consistently? I see other permissions errors based on timing: https://github.com/awslabs/agents-for-amazon-bedrock-blueprints/issues/22