After the new version of vendor/susy unable to compile sass

[x] This is a question about using the theme.
[ ] This is a feature request.
[x] I have updated all gems with bundle update.
[x] I have tested locally with bundle exec jekyll build.
[x] I believe this to be a bug with the theme --- not Jekyll, GitHub Pages or one of the bundled plugins.

Environment informations

Minimal Mistakes version:

$ grep minimal Gemfile
gem "minimal-mistakes-jekyll", :git => "https://github.com/mmistakes/minimal-mistakes.git", :branch => "master"
$ grep minimal Gemfile.lock
  remote: https://github.com/mmistakes/minimal-mistakes.git
    minimal-mistakes-jekyll (4.5.0)
  minimal-mistakes-jekyll!

github-pages or jekyll gem version:

$ gem --version
2.6.12
$ jekyll --version
jekyll 3.5.1

Operating system:

Linux 559d6849779c 4.10.0-28-generic #32-Ubuntu SMP Fri Jun 30 05:32:18 UTC 2017 x86_64 GNU/Linux

Expected behavior
Configuration file: /usr/src/app/_config.yml
            Source: /usr/src/app
       Destination: /usr/src/app/_site
 Incremental build: enabled
      Generating...

Steps to reproduce the behavior
$ git clone https://github.com/mmistakes/minimal-mistakes.git
$ cd minimal-mistakes
$ git reset --hard 3e781220a1cb5d47312a3db25d7c6ae6dc375b2b
$ wget https://raw.githubusercontent.com/kmassada/kmassada.github.io/jekyll-3.5/Dockerfile
$ docker build . -t minimal
$ docker run -p 4000:4000 -v "$PWD":/usr/src/app -w /usr/src/app minimal
Configuration file: /usr/src/app/_config.yml
       Deprecation: The 'gems' configuration option has been renamed to 'plugins'. Please update your config file accordingly.
            Source: /usr/src/app
       Destination: /usr/src/app/_site
 Incremental build: enabled
      Generating...
  Conversion error: Jekyll::Converters::Scss encountered an error while converting 'assets/css/main.scss':
                    Invalid US-ASCII character "\xE2" on line 54
jekyll 3.5.2 | Error:  Invalid US-ASCII character "\xE2" on line 54

additional steps
file list diff between 4.4 and 4.5.
After resetting each file, one at the time. I was able to identify the problem with _sass/minimal-mistakes/vendor/susy, need guidance on how to fix it.
 $ git diff origin master --name-only
Dockerfile
Gemfile.lock
README.md
_config.yml
_data/ui-text.yml
_includes/analytics.html
_includes/page__hero.html
_layouts/default.html
_layouts/single.html
_sass/minimal-mistakes.scss
_sass/minimal-mistakes/_archive.scss
_sass/minimal-mistakes/_footer.scss
_sass/minimal-mistakes/_masthead.scss
_sass/minimal-mistakes/_navigation.scss
_sass/minimal-mistakes/_page.scss
_sass/minimal-mistakes/_reset.scss
_sass/minimal-mistakes/_sidebar.scss
_sass/minimal-mistakes/_syntax.scss
_sass/minimal-mistakes/_tables.scss
_sass/minimal-mistakes/_utilities.scss
_sass/minimal-mistakes/_variables.scss
_sass/minimal-mistakes/vendor/susy/_susy-prefix.scss
_sass/minimal-mistakes/vendor/susy/_susy.scss
_sass/minimal-mistakes/vendor/susy/_susyone.scss
_sass/minimal-mistakes/vendor/susy/plugins/_svg-grid.scss
_sass/minimal-mistakes/vendor/susy/plugins/svg-grid/_prefix.scss
_sass/minimal-mistakes/vendor/susy/plugins/svg-grid/_svg-api.scss
_sass/minimal-mistakes/vendor/susy/plugins/svg-grid/_svg-grid-math.scss
_sass/minimal-mistakes/vendor/susy/plugins/svg-grid/_svg-settings.scss
_sass/minimal-mistakes/vendor/susy/plugins/svg-grid/_svg-unprefix.scss
_sass/minimal-mistakes/vendor/susy/plugins/svg-grid/_svg-utilities.scss
_sass/minimal-mistakes/vendor/susy/susy/_api.scss
_sass/minimal-mistakes/vendor/susy/susy/_normalize.scss
_sass/minimal-mistakes/vendor/susy/susy/_parse.scss
_sass/minimal-mistakes/vendor/susy/susy/_settings.scss
_sass/minimal-mistakes/vendor/susy/susy/_su-math.scss
_sass/minimal-mistakes/vendor/susy/susy/_su-validate.scss
_sass/minimal-mistakes/vendor/susy/susy/_su.scss
_sass/minimal-mistakes/vendor/susy/susy/_syntax-helpers.scss
_sass/minimal-mistakes/vendor/susy/susy/_unprefix.scss
_sass/minimal-mistakes/vendor/susy/susy/_utilities.scss
_sass/minimal-mistakes/vendor/susy/susy/language/_susy.scss
_sass/minimal-mistakes/vendor/susy/susy/language/_susyone.scss
_sass/minimal-mistakes/vendor/susy/susy/language/susy/_background.scss
_sass/minimal-mistakes/vendor/susy/susy/language/susy/_bleed.scss
_sass/minimal-mistakes/vendor/susy/susy/language/susy/_box-sizing.scss
_sass/minimal-mistakes/vendor/susy/susy/language/susy/_breakpoint-plugin.scss
_sass/minimal-mistakes/vendor/susy/susy/language/susy/_container.scss
_sass/minimal-mistakes/vendor/susy/susy/language/susy/_context.scss
_sass/minimal-mistakes/vendor/susy/susy/language/susy/_gallery.scss
_sass/minimal-mistakes/vendor/susy/susy/language/susy/_grids.scss
_sass/minimal-mistakes/vendor/susy/susy/language/susy/_gutters.scss
_sass/minimal-mistakes/vendor/susy/susy/language/susy/_isolate.scss
_sass/minimal-mistakes/vendor/susy/susy/language/susy/_margins.scss
_sass/minimal-mistakes/vendor/susy/susy/language/susy/_padding.scss
_sass/minimal-mistakes/vendor/susy/susy/language/susy/_rows.scss
_sass/minimal-mistakes/vendor/susy/susy/language/susy/_settings.scss
_sass/minimal-mistakes/vendor/susy/susy/language/susy/_span.scss
_sass/minimal-mistakes/vendor/susy/susy/language/susy/_validation.scss
_sass/minimal-mistakes/vendor/susy/susy/language/susyone/_background.scss
_sass/minimal-mistakes/vendor/susy/susy/language/susyone/_functions.scss
_sass/minimal-mistakes/vendor/susy/susy/language/susyone/_grid.scss
_sass/minimal-mistakes/vendor/susy/susy/language/susyone/_isolation.scss
_sass/minimal-mistakes/vendor/susy/susy/language/susyone/_margin.scss
_sass/minimal-mistakes/vendor/susy/susy/language/susyone/_media.scss
_sass/minimal-mistakes/vendor/susy/susy/language/susyone/_padding.scss
_sass/minimal-mistakes/vendor/susy/susy/language/susyone/_settings.scss
_sass/minimal-mistakes/vendor/susy/susy/output/_float.scss
_sass/minimal-mistakes/vendor/susy/susy/output/_shared.scss
_sass/minimal-mistakes/vendor/susy/susy/output/_support.scss
_sass/minimal-mistakes/vendor/susy/susy/output/float/_container.scss
_sass/minimal-mistakes/vendor/susy/susy/output/float/_end.scss
_sass/minimal-mistakes/vendor/susy/susy/output/float/_isolate.scss
_sass/minimal-mistakes/vendor/susy/susy/output/float/_span.scss
_sass/minimal-mistakes/vendor/susy/susy/output/shared/_background.scss
_sass/minimal-mistakes/vendor/susy/susy/output/shared/_container.scss
_sass/minimal-mistakes/vendor/susy/susy/output/shared/_direction.scss
_sass/minimal-mistakes/vendor/susy/susy/output/shared/_inspect.scss
_sass/minimal-mistakes/vendor/susy/susy/output/shared/_margins.scss
_sass/minimal-mistakes/vendor/susy/susy/output/shared/_output.scss
_sass/minimal-mistakes/vendor/susy/susy/output/shared/_padding.scss
_sass/minimal-mistakes/vendor/susy/susy/output/support/_background.scss
_sass/minimal-mistakes/vendor/susy/susy/output/support/_box-sizing.scss
_sass/minimal-mistakes/vendor/susy/susy/output/support/_clearfix.scss
_sass/minimal-mistakes/vendor/susy/susy/output/support/_prefix.scss
_sass/minimal-mistakes/vendor/susy/susy/output/support/_rem.scss
_sass/minimal-mistakes/vendor/susy/susy/output/support/_support.scss
_sass/minimal-mistakes/vendor/susy/susy/su/_grid.scss
_sass/minimal-mistakes/vendor/susy/susy/su/_settings.scss
_sass/minimal-mistakes/vendor/susy/susy/su/_utilities.scss
_sass/minimal-mistakes/vendor/susy/susy/su/_validation.scss
assets/js/_main.js
assets/js/main.min.js

I have nothing to do with Susy so can't really say since those Sass files are included in the theme untouched.
Smells like a possible file encoding issue from this error you're getting.
Invalid US-ASCII character "\xE2" on line 54
jekyll 3.5.2 | Error: Invalid US-ASCII character "\xE2" on line 54

I also don't have experience with Docker so it could be something there, or related to you have a different version of the Sass gem installed.

spun a cloud instance, setup works fine for both yours and mine. which indicates problem perhaps with my dev setup
#Linux jekyll-test 4.10.0-32-generic #36~16.04.1-Ubuntu SMP Wed Aug 9 09:19:02 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
git clone https://github.com/mmistakes/minimal-mistakes.git
cd minimal-mistakes
sudo apt-get install rubygems ruby-dev
sudo apt-get install linux-headers-$(uname -r) build-essential  zlib1g-dev
sudo gem install jekyll bundler
sudo gem install sass --no-user-install
bundle install
bundle exec jekyll serve -H <IP_ADDRESS> --incremental --watch

@kmassada That wouldn't surprise me. It's usually the case with Ruby and gems. Going to close this as it doesn't appear to be theme related.
Feel free to re-open if that's not the case.

I want to add details in case anyone comes across this too. cloud images tend to not have locales configured.
apt-get install locales
locale-gen en_US.UTF-8
localedef -i en_US -f UTF-8 en_US.UTF-8
LANG="en_US.UTF-8" LC_ALL="en_US.UTF-8" bundle exec jekyll serve -H <IP_ADDRESS> --incremental --watch

Thanks @kmassada I used your solution to adapt my Dockerfile:
FROM ruby:2

RUN apt-get update && \
    apt-get install locales && \
    locale-gen en_US.UTF-8 && \
    localedef -i en_US -f UTF-8 en_US.UTF-8

ENV LANG en_US.UTF-8
ENV LC_ALL en_US.UTF-8

RUN gem install bundler

And there is my docker-compose.yml file:
version: '2'

services:
    jekyll:
        build: docker/jekyll
        volumes:
            - .:/var/www/html
        working_dir: /var/www/html
        ports:
            - 4000:4000
        command: bash -c "bundle install && bundle exec jekyll serve --host=<IP_ADDRESS>"

And I could run jekyll server


Panic in getFromStringItem

In version v2.9.0 and earlier there's a panic in this function. The exact line and panic is in the comment below.
// getFromStringItem build shared string item offset list from system temporary
// file at one time, and return value by given to string index.
func (f *File) getFromStringItem(index int) string {
	if f.sharedStringTemp != nil {
		if len(f.sharedStringItem) <= index {
			return strconv.Itoa(index)
		}
		offsetRange := f.sharedStringItem[index]
		buf := make([]byte, offsetRange[1]-offsetRange[0]) // panic: runtime error: index out of range [1] with length 0
		if _, err := f.sharedStringTemp.ReadAt(buf, int64(offsetRange[0])); err != nil {
			return strconv.Itoa(index)
		}
		return string(buf)
	}

I am not really familiar with this so I don't know what the fix has to be.

Thanks for your issue. Could you show us a complete, standalone example program or reproducible demo? If you open an existing workbook, please provide the file attachment without confidential info.

It seems still not have enough info of code and attachments to reproduce this issue, so I'll close this issue, if you have any questions, please let me know to reopen this anytime.


Override deps requires profile tuple

Currently to override deps you have to do:
{overrides,
 [{override, eleveldb,
   [
   {{deps,default}, []}
   ]
  }]
}.

There should be no need for default.

This is actually a dupe of https://github.com/rebar/rebar3/issues/704


Updated Microsoft PowerBI Desktop to version 2.111.590.0.

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.2 schema?

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

Resolves #89954.

@wingetbot run

@jedieaston, wingetbot has updated this in https://github.com/microsoft/winget-pkgs/pull/89929. You can merge the changes if there's anything extra this PR adds. Otherwise, this can be closed.

Duplicate of #89929


Firefox: ICE failed, your TURN server appears to be broken

Running the webrtc_videoroom example in this repository, I am able to connect to a room in Chrome with no issue, however attempting to do the same in Firefox always fails, and the console logs the error "WebRTC: ICE failed, your TURN server appears to be broken, see about:webrtc for more details"
Running on Arch Linux, Firefox Developer version 122.0b9.
Attached is the about:webrtc (saved as .txt because Github won't let me upload .HTML)
aboutWebrtc.txt
Let me know if there is any more information I can provide!

Gah! Basic instructions are hard. Running with EXTERNAL_IP=<IPv4 address> mix phx.server  as instructed in the README solved the issue. Similarly, binding to my actual IP rather than loopback or <IP_ADDRESS> resolved the issue with my own code.


Swarm integration tests should not require network

#24490 added network requirement for these tests but there shouldn't be a need for that. Original problem seems to come from listening address detection and there are already PRs(#24237) that change that. If that PR doesn't fully fix it we should fix the remaining issues instead of skipping these tests.
@runcom @thaJeztah

@tonistiigi The PR #24237 has already been merged. I removed network requirement and all tests still passed with networking disabled. I created a PR #25258 to remove testRequires(c, Network) from the integration tests.


Rename readme_zh-cn.md to readme_zh-CN.md

Typo

closing cuz i already solved it

Hi awslawsl233, does CN really need to be uppercase? cuz i already changed
it to lowercase in the links of the readme files
if u need to, make another PR Thanks
On Thu, Feb 24, 2022 at 21:58 awslawsl233 @.***> wrote:

Typo
You can view, comment on, or merge this pull request online at:
https://github.com/xvzc/SpoofDPI/pull/12
Commit Summary

6015da3
https://github.com/xvzc/SpoofDPI/pull/12/commits/6015da3fe567704f5ac0de32b34ff7649c353cee
Rename readme_zh-cn.md to readme_zh-CN.md

File Changes
(1 file https://github.com/xvzc/SpoofDPI/pull/12/files)

R readme_zh-CN.md
https://github.com/xvzc/SpoofDPI/pull/12/files#diff-6793591b5fc00f4fb81f4a5dc0ef11f67656eb165e5c8895589b9bb6f2e43e7c
(0)

Patch Links:

https://github.com/xvzc/SpoofDPI/pull/12.patch
https://github.com/xvzc/SpoofDPI/pull/12.diff

—
Reply to this email directly, view it on GitHub
https://github.com/xvzc/SpoofDPI/pull/12, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AK3Z72OGVGCG7DBOSXJXPS3U4YTNXANCNFSM5PHJ2USQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

No, not very necessary.

Ok, thanks.
On Thu, Feb 24, 2022 at 22:06 awslawsl233 @.***> wrote:

No, not very necessary.
—
Reply to this email directly, view it on GitHub
https://github.com/xvzc/SpoofDPI/pull/12#issuecomment-1049839932, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AK3Z72JFSQPVJHDZC7U7M7LU4YUONANCNFSM5PHJ2USQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you modified the open/close state.Message
ID: @.***>


[Bug]: Hitting connection pool error when trying to config ORACLE datasource

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When I tried to config staging with same creds as that of Prod it is observed that the user is displayed with error message
Steps To Reproduce

Navigate to oracle DS
Config the prod DS
Navigate to staging and try to config same creds as of Prod
Click on test config on Staging
and observe that user is displayed with error

Public Sample App
No response
Environment
Deploy Preview
Issue video log
No response
Version
Cloud

Closing since this was a config issue


Make optional packages optional

This slims down a docker build from 2.4 GB to 1.2 GB.

Nice going. Thanks.


Allow direct A* grid search

As mentioned in [1, Ch. III 6.3] in 'chessboard-like grid worlds' (many small obstacles have a lot of extremities!) it can be better to use A* right away (implemented in graph_search.py).
add option to use A* directly
automatically use A* when a certain ratio of extremities vs. grid tiles is reached

add API to pass just a graph (point cloud with connections)

Already implemented (without use-case specific optimisations) in the networkx library: https://networkx.org/documentation/stable/reference/algorithms/shortest_paths.html#module-networkx.algorithms.shortest_paths.astar


More work...

Type of change

Bugfix

Description
I think this caused a lot of random test failure in CI due to address space controller spinning in requeue.
Checklist

[ ] Update/write design documentation in ./documentation/design
[ ] Write tests and make sure they pass
[ ] Update documentation
[ ] Check RBAC rights for Kubernetes / OpenShift roles
[ ] Try your changes from Pod inside your Kubernetes and OpenShift cluster, not just locally
[ ] Reference relevant issue(s) and close them after merging
[ ] Update CHANGELOG.md

Sorry, wrong branch!


Message constructors

Hi,
While looking at the Message class, I noticed that it has three constructors
Message(MessageType type, span<byte> data)
Message(MessageType type, string& data)
Message(MessageType type, string_view data)

The last constructor works fine with const strings, because the string view is essentially const. However the first two constructors don't work with const data. Also I see that the third constructor internally uses a const_cast to make the string_view data non-const.
I think it makes sense to change the constructors to
Message(MessageType type, span<byte> data)
Message(MessageType type, span<const byte> data)
Message(MessageType type, const string& data)
Message(MessageType type, string_view data)

This way it will accept any const strings and spans without a problem.
Then it also makes sense to change
    span<byte> data;

to
    span<const byte> data;

This will remove the need for const_casts in the constructors.
What do you think?

Maybe it is possible to get rid of this constructor
Message(MessageType type, string& data)

because the string_view constructor takes care of the case when the argument is const strings&.

Hi,
See https://github.com/rbeeli/websocketclient-cpp/commit/78068b3aa3a4e59ce96006465bfe00894e576cce. I have added a span<const byte> constructor and dropped string& since string_view covers that case as you said. Message, for now, is writeable, maybe at some point I'll consider making it read-only, i.e. span<const byte>. One reason is to be able to do in-place parsing.

Great, this will let me remove one or two explicit string constructors in my code. Thank you!


Disappeared icons

https://wintercms.com/docs/ui/icon

Odd. Fixed by clearing the server cache. Thanks for the report!


add support banner container

Why:
fix issue

What's being changed:
fix support banner block add container for large monitors

Check off the following:

[x] All of the tests are passing.
[x] I have reviewed my changes in staging.
[x] For content changes, I have reviewed the localization checklist
[x] For content changes, I have reviewed the Content style guide for GitHub Docs.

review page https://docs-388--fix-support-banner.herokuapp.com/en


V2 - Project Console - Need API for Enable branch protection feature

Need API for Enable branch protection feature for github and gerrit in project console
wireframe https://wireframepro.mockflow.com/view/administrative-console#/page/D3c036a1a1d2cf80d8ac83dae32036445/sidebar/off

@makkalot let's start investigating this feature.  Essentially, we:

we will need to adjust the permissions of the EasyCLA GitHub app to allow admin controls to be able to adjust the repository branch protection
Via the GitHub API - can we query to see if "branch protection" is set up on the "default" branch? Yes/No?
If not setup, can we add branch protection via the API? Yes/No?
If yes, can we specify 1) EasyCLA is a required check 2) admins can't by-pass

@makkalot I have reviewed your video. Excellent work.  Based on this, can you:

identify GH permissions required to 1) check and 2) set/adjust these settings. I need to understand and document these permissions changes...
develop a v2 API to determine the branch protection status (as you have done with your CLI example). Input: GH repository, Output: branch protection enabled on default branch true/false. If enabled, is: enforce admin enabled true/false, status checks for EasyCLA enabled true/false
develop a v2 API to add/adjust branch protection: Input. GH repository, branch name, enforce admin: true/false, enable status checks (accepts zero or more): [EasyCLA: on, Other check: off, Some other check: on].  If checks on GH repo set, but not in our input list, do not modify.

thoughts?

yeah, makes sense @dealako

Looks like the least perms we need are like :

repo

public_repo

admin:org

write:org
read:org

For more info about the permissions can check : https://docs.github.com/en/developers/apps/scopes-for-oauth-apps#available-scopes

@makkalot @dealako
Could you please provide the link to API Doc so that I can proceed with the testing of the ticket. I am not able to find the endpoint in https://api-gw.dev.platform.linuxfoundation.org/cla-service/v4/api-docs#tag/github-organizations

@nirupamav it's under github-repositories :

https://api-gw.dev.platform.linuxfoundation.org/cla-service/v4/api-docs#operation/getProjectGithubRepositoryBranchProtection
https://api-gw.dev.platform.linuxfoundation.org/cla-service/v4/api-docs#operation/updateProjectGithubRepositoryBranchProtection

Tested and observed below issues
Get Action
1. Able to fetch data of repository branch protection details when project Id that is not associated with the repository Id is provided in the URI or when invalid project Id is provided in the URI

2. Error message is not returned in the response when invalid repository Id is provided

In github for the repository, when default branch is master. But when default branch is the branch that I created then status checks is returned with EasyCLA information

Post Action
1. Able to Post data of repository branch protection  when project Id that is not associated with the repository Id is provided in the URI or when invalid project Id is provided in the URI
2. Error message is not returned in the response when invalid repository Id is provided
3. When rule is not created in github, not able to enable branch protection

4. As per below comment in ticket discussion, Endpoint should accept branch name also as input so that protection can be enabled for that particular branch. As of now it works only for default branch

5. As per below comment in ticket discussion, existing rules set in github should not be modified. However few settings are removed

Settings configured from github

After updating branch protection from endpoint

@dealako @makkalot
What is he expected behavior in below scenarios
Payload1
{
  "enforce_admin": false,
  "status_checks": [
    {
      "enabled": false,
      "name": "EasyCLA"
    }
  ]
}

**Payload2**

{
"enforce_admin": false,
"status_checks": [
{
"enabled": true,
"name": "EasyCLA"
}
]
}
Payload3
{
  "enforce_admin": true,
  "status_checks": [
    {
      "enabled": false,
      "name": "EasyCLA"
    }
  ]
}

Payload4
{
  "enforce_admin": true,
  "status_checks": [
    {
      "enabled": true,
      "name": "EasyCLA"
    }
  ]
}

PR is created
If branch protection is enabled if EasyCLA is true (required to pass before merging) if enforce admin is enabled => cannot bypass the EasyCLA check (unable to merge under all conditions/users)
If branch protection is enabled, if EasyCLA is true (required to pass before merging), if enforce admin is disabled => maintainers cannot bypass the EasyCLA check (unable to merge under all conditions/users), but Admins/Owners CAN.
If branch protection is enabled, if EasyCLA is false (required to pass before merging), if enforce admin is enabled => maintainers CAN bypass the EasyCLA check (unable to merge under all conditions/users)
If no branch protection, => maintainers CAN bypass the EasyCLA check (unable to merge under all conditions/users)

regarding #3 - Gerrit doesn't have an API for us to enable "Branch Protection". This is a GitHub only concept.  GitLab may be similar to GH, but Gerrit doesn't have anything like this.
AutoEnable for Gerrit - Gerrit currently does not have an API to allow us to configure this.

@nirupamav @makkalot I pushed and merged a PR which adds the branch protection endpoint to ACS-CLI, see: https://github.com/LF-Engineering/acs-cli/pull/313

@dealako as per the wireframe, the Enable Branch protection is at Github organization level. However the API needs the repositoryID as a required path parameter.

Create GitHub Organization: https://api-gw.dev.platform.linuxfoundation.org/cla-service/v4/api-docs#operation/addProjectGithubOrganization
Update GitHub Organization: https://api-gw.dev.platform.linuxfoundation.org/cla-service/v4/api-docs#operation/updateProjectGithubOrganizationConfig

please note the branchProtectionEnabled flag in the payload.

@makkalot @dealako
Is the https://api-gw.dev.platform.linuxfoundation.org/cla-service/v4/api-docs#operation/updateProjectGithubRepositoryBranchProtection still valid. I am not able to perform any action as it returns 400 even for valid data

This ticket is no more valid as Branch protection is enabled at org level. The new changes with below endpoints will be tested in #1914
Create GitHub Organization: https://api-gw.dev.platform.linuxfoundation.org/cla-service/v4/api-docs#operation/addProjectGithubOrganization
Update GitHub Organization: https://api-gw.dev.platform.linuxfoundation.org/cla-service/v4/api-docs#operation/updateProjectGithubOrganizationConfig


Fix _done function not dealing with compilation being undefined

What kind of change does this PR introduce?
#3934 introduced a bug as the title says, this fixes that.
Did you add tests for your changes?
No, any pointer to where add this test too? It seems there are no tests for the watcher handler being called correctly with err.
Summary
This should land ASAP because currently webpack is broken for errors in some plugins.
Does this PR introduce a breaking change?
No.
Other information
None.

//cc @timse @sokra

Thanks


Allowing same file to be uploaded.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #110   +/-   ##
=======================================
  Coverage   78.84%   78.84%
=======================================
  Files          42       42
  Lines        1952     1952
  Branches      305      305
=======================================
  Hits         1539     1539
  Misses        413      413

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 575537e...624091c. Read the comment docs.


Support RuntimeClasses for Windows Pod Scheduling

Is your feature request related to a problem? Please describe.
Kubernetes documentation recommends to use RuntimeClasses to simplify Windows Pod Scheduling however this is not possible when using EKS as the webhook code is just looking for NodeSelector configuration in PodSpec.
Describe the solution you'd like
Looking in PodSpec in runtimeClassName is defined and search for NodeSelector config in specified RuntimeClass otherwise look NodeSelector in PodSpec.
Describe alternatives you've considered
Additional context
Take into account that defining different configurations for NodeSelector in both PodSpec and RuntimeClass will make Kubernetes scheduler to return an error.

This issue can be closed now as no changes are required on the controller, the runtimeClass nodeSelector is automatically merged into pod by the RuntimeClass Admissions Controller https://github.com/kubernetes/enhancements/blob/master/keps/sig-node/585-runtime-class/README.md#runtimeclass-admission-controller
Also see https://github.com/aws/containers-roadmap/issues/1106


[Bug]

Describe the bug
On demo page after add new product and new form order show error on product:
Deprecated: json_decode(): Passing null to parameter #1 ($json) of type string is deprecated in /var/www/html/modules/Formbuilder/Service.php on line 243 Deprecated: json_decode(): Passing null to parameter #1 ($json) of type string is deprecated in /var/www/html/modules/Formbuilder/Service.php on line 244 Deprecated: json_decode(): Passing null to parameter #1 ($json) of type string is deprecated in /var/www/html/modules/Formbuilder/Service.php on line 244 Deprecated: json_decode(): Passing null to parameter #1 ($json) of type string is deprecated in /var/www/html/modules/Formbuilder/Service.php on line 244 Deprecated: json_decode(): Passing null to parameter #1 ($json) of type string is deprecated in /var/www/html/modules/Formbuilder/Service.php on line 243 Deprecated: json_decode(): Passing null to parameter #1 ($json) of type string is deprecated in /var/www/html/modules/Formbuilder/Service.php on line 244 Deprecated: json_decode(): Passing null to parameter #1 ($json) of type string is deprecated in /var/www/html/modules/Formbuilder/Service.php on line 244 Deprecated: json_decode(): Passing null to parameter #1 ($json) of type string is deprecated in /var/www/html/modules/Formbuilder/Service.php on line 244
How to reproduce

Go to page https://demo.fossbilling.org/order/test-product
https://demo.fossbilling.org/admin/extension/settings/formbuilder?id=1

Expected behavior
after add new product
Screenshots
No response
FOSSBilling version
0.5.0
Module version
No response
Device
No response
Information
No response
Additional context
No response

Disabled Debug mode....


Introducing a float type?

We often have to customize the number type to be able to deal with float numbers, using something similar to:
nga.field('float_attribute', 'number').attributes({ step: 'any' }).format('0.000');

What about offering a float type out of the box, including the previous configuration?

Great idea, and quite easy to achieve.

PR merged. Closing.


I changed my name

Please I am sorry for all the trouble we have caused you but count it all joy. Your PR is about to be merged and you will be able to see your name on this url.
To enable me to do this, READ the README again carefully, update your code with the latest and push the update. I am here for you for the next 6 hours before I close the PR so you can initiate a fresh PR which will be easier to merge.

Okay sir
On Mon, Aug 15, 2022, 9:19 PM Olúwásetèmi @.***> wrote:

Please I am sorry for all the trouble we have caused you but count it all
joy. Your PR is about to be merged and you will be able to see your name on
this url https://altschool-opensource-names.oluwasetemi.dev/.
To enable me to do this, READ the README again carefully, update your
code with the latest and push the update. I am here for you for the next 6
hours before I close the PR so you can initiate a fresh PR which will be
easier to merge.
—
Reply to this email directly, view it on GitHub
https://github.com/Oluwasetemi/altschool-opensource-names/pull/330#issuecomment-1215747745,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXVHWZA3HBKO5UIOPEEIG73VZKQ5DANCNFSM56HWQJVQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


type '({LegacyJavaScriptObject buffer})' is not a subtype of type 'JSObject'

I am getting strange error:
DartError: TypeError: Record (buffer: Instance of 'LegacyJavaScriptObject'): type '({LegacyJavaScriptObject buffer})' is not a subtype of type 'JSObject'

When I am run this code
entries: [
  GPUBindGroupEntry(
    binding: 0,
    resource: sampler,
  ),
  GPUBindGroupEntry(
    binding: 1,
    resource: (
      buffer: blurParamsBuffer <-- Error
    ) as GPUBindingResource
  ),
] as JSArray<GPUBindGroupEntry>,

As you can see resource type can be different, that is why it's just JSObject in definition
typedef GPUBindingResource = JSObject;

extension type GPUBindGroupEntry._(JSObject _) implements JSObject {
  external factory GPUBindGroupEntry({
    required GPUIndex32 binding,
    required GPUBindingResource resource,
  });

  external GPUIndex32 get binding;
  external set binding(GPUIndex32 value);
  external GPUBindingResource get resource;
  external set resource(GPUBindingResource value);
}

But this cause error. My bindings from SDK 3.6, but version of SDK 3.5.4 on "macos_x64".
Source: compute example from repo.

Are you using dart:html or similar? I believe that's why you're seeing the Legacy object.
CC @srujzs

The error is correct, you're casting a record (which itself contains a LegacyJavaScriptObject but is not one) to a JSObject:
resource: (buffer: blurParamsBuffer) as GPUBindingResource

You likely want another dictionary type for this type e.g.
extension type GPUBindingResource._(JSObject _) implements JSObject {
  external factory GPUBindingResource({
    required GPUBuffer buffer,
  });
}
...
resource: GPUBindingResource(buffer: blurParamsBuffer)

Thanks, I was desperate yesterday. By the way better to hide all typed lists in webglX.dart or add to them additional methods. This types too basic to not be covered. Now they declared like that
typedef Float32List = JSObject;

But in code when you using it, you forced to hide it
import 'dart:js_interop';
import 'package:web/web.dart' hide Float32List, Uint32List;

As I understand no dart:web_gl user is migrated to package:web yet.

I'm doing the same thing. Those conflicts are a bit annoying as you can't really do anything with the Float32List from package:web.
But there is an open issue for this: https://github.com/dart-lang/web/issues/58

Yup, there's an open issue (although it'd be a breaking change).
Another option is to always import it with a prefix, but that's going to be quite verbose when you're using lots of types.


CI for Node

Initialize with two players
Starta the game with

Initializes with two players
Starts the game with a random player
Sets the current player to be whoever it is not

ads

Initializes with two players


Question: Can I have the API hosted separately from where my swashbuckle configuration is?

I'm using Swashbuckle 5.6.0 and have an API hosted with IIS on my machine. Whenever I use the "try it out" button, the project in which i have setup up swashbuckle creates an Instance of my API without resolving any dependancies. The problem comes from the fact that I cannot separate the 'try it out' button to have it hit my hosted api, ex: "https://foo.com/api/dosomething" because it tries to make a call to the same root url that I have hosted the Documentation, ex"https://foo.com/doc/dosomething".
I have searched for similar type of problems here and on stack-overflow but couldn't find a reasonable answer.
TL:DR Can I have the "try it out" button configured to hit a specific url?

I have found a workaround for the issue.
Since the api is hosted separately from the documentation, I had the problem of not being able to dictate where the "Try it Out" button sent the request to (by default it sends it to the same url as the root). By implementing IDocumentFilter i was able to use the swaggerDoc.host and swaggerDoc.basePath to manipulate the endpoint target. The workaround provided here #356 was not fruitful for me.


Baseline completions and signature help

Background: I need to update a lot of jsdoc tests since #41877 makes the jsdoc parse tree much more complete. This leads to a lot of painstaking, repetitive work updating fourslash tests.
In this PR, I add baselining to completions and signature help, to match the baselining for quick info. I also changed the filename, which should really be the containing filename.
Limitations

I only changed the tests that I care about for #41877 because I wanted to have time to inspect the baselines for this change. And I haven't planned out a way to update the tests. Ideas are welcome.
Quick info, signature help, smart selection and completions all use the same filename and are thus mutually exclusive in one test. Fourslash tests are mostly divided this way already, so I thought it was an acceptable limit. I had to split five or six existing, gigantic, horrible tests as a result.

One suggestion is that the baseline is appended so that you can add multiple commands baseline to same file .. I think it would be useful if you want to edit a file and see completions or quick info after the edit

Or it might be good to add a baselineAll variant that takes a list of baselines to produce.
Another problem I thought of is that the completions baselines are large, so asking for completions at every marker is unneeded.


Add Terrascan Sandbox link to README

Summary

Update README to include link to the Terrascan Sandbox

Codecov Report

Merging #1409 (1ef4d9d) into master (849eef7) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1409   +/-   ##
=======================================
  Coverage   77.27%   77.27%
=======================================
  Files         279      279
  Lines        7856     7856
=======================================
  Hits         6071     6071
  Misses       1426     1426
  Partials      359      359


Fetching registry library fails due to improper reference to index generator

Which area this feature is related to?
/kind bug

Which area this bug is related to?

/area registry
What versions of software are you using?
Go project
Operating System and version: N/A
Go Pkg Version: 1.19
Bug Summary
Describe the bug:
When fetching chained dependencies for the registry library, the command fails due to the index generator dependency not pointing to a valid revision reference. This is because of replace redirection on v0.0.0 which does not work when referencing the module externally.

To Reproduce:

Run go get github.com/devfile/registry-support/registry-library@latest.
Expected behavior

Consumers should be able to successfully fetch the registry library module along with all dependencies of it.
Any logs, error output, screenshots etc? Provide the devfile that sees this bug, if applicable

go<EMAIL_ADDRESS>reading github.com/devfile/registry-support/index/generator/go.mod at revision index/generator/v0.0.0: unknown revision index/generator/v0.0.0

Additional context

Any workaround?

Use earlier revisions of registry library which does not have this bug.
Suggestion on how to fix the bug

Change revision ref for index generator to latest one for both dependency reference and replace reference.

Using github.com/devfile/registry-support/index/generator v0.0.0-20240311135803-6215550f93d4 for index generator dependency reference.

Confirmed these changes fix the fetching of the registry library under devfile/library.

Need to create a backport patch for 1.1.x for this bug.

v1.1.x backport fix PR ready for review: https://github.com/devfile/registry-support/pull/223


[CHANGE] move the api base & nightbot base functions into two classes

more user-friendly,
better visibility

This will then look like this (used class names might change):
class ApiBase;

    def get_leaderboard(...):
        # ...

    def ...(...):
        # ...


Error on port 5500

I've only been using vs code for about a week now. Just when I've started to get used to it I'm getting a error on port '5500 please change in settings' alert when I attempt to use the live server extension.

Have you tried to change port ?
https://github.com/ritwickdey/vscode-live-server/blob/master/docs/settings.md

How exactly do you change the setting? I'm new to VS Code.

Live Server FAQS:  https://github.com/ritwickdey/vscode-live-server/blob/master/docs/faqs.md#how-to-configure-the-settings-in-my-project

Live Server settings : https://github.com/ritwickdey/vscode-live-server/blob/master/docs/settings.md

(official vscode docs : https://code.visualstudio.com/docs/getstarted/setting)

Is the issue still there?
Let me know if I can help further ?

Another option is to kill what's running on port 5500.

Then go back into VSC and start Live Server again. Works for me.

Hey I'm having a problem, I tried changing the port to 0 and I'm still getting error's. I even tried to kill the running port on the command line and it's still not working. What should I do?

Same problem for me. Now what?

@thisLinda lsof is a command for Linux, not Windows. No clue how the command is on windows. Try to google with "windows netstat openend ports command"

aHa!!!!
Thank you. I'll continue searching.

Solved itself after I ran the VSC update.

For Windows: (To kill 5500 Port)

Run command-line as an Administrator.

netstat -ano | findstr :5500
taskkill /PID typeyourPIDhere /F
Reference: https://stackoverflow.com/a/39633428/6120338

Have you tried to change port ?
https://github.com/ritwickdey/vscode-live-server/blob/master/docs/settings.md
ritwickdey, thank You much!)

@kapunahelewong thanks! I want to run two ports at same time though.

Hello Guys, I changed my port Number of my Live Server to 0. However, it tells me Server is now offine after I run it. What should I do?

Hello Guys, I changed my port Number of my Live Server to 0. However, it tells me Server is now offine after I run it. What should I do?

Sorry, it's offline!

pls can i use liveserve offline?

I also had problems with the port 5500, in my case it was the firewall, I solved the problem here => https://help.ubuntu.com/community/UFW
====== in terminal =====
sudo ufw allow 5500

I followed all the steps but not found the solutions


Emails from gmail to gmail not downloading

My wife and I both have K-9 on Android 11 set up with SMTP to our gmail accounts. Recently emails from gmail addresses to each of our gmail accounts have not been downloading to either of our K-9 apps even though they show up on our gmail accounts.
I don't know if this is a google problem or a K-9 problem.

Please use the forum for support requests.


WROOM ESP32 CH340C Pinout

Hi all
Trying this with a wroom-32, and I can't get my head around what D7 and D8 are on this board.
Looks like D7 may be GPIO13, but I can't figure out what D8 is at all, or what alternative pin would be ok to use. I have this wroom board with CH340C - https://www.aliexpress.com/item/1005006336429842.html
I think I have my head around the rest, here is my prepared yaml.
substitutions:
  name: victron_solar_controller
  external_components_source: github://KinDR007/VictronMPPT-ESPHOME@main

esphome:
  name: ${name}
  comment: Front Gate ESP32 controller
  area: Outside

external_components:
  - source: ${external_components_source}
    refresh: 0s

esp32:
  board: nodemcu-32s

wifi:
  ssid: !secret wifi_ssid
  password: !secret wifi_password

uart:
  id: uart_0
  tx_pin: D8  # Not used! The communication is read-only
  rx_pin: GPIO13
  baud_rate: 19200
  rx_buffer_size: 256

victron:
  uart_id: uart_0
  id: victron0
  throttle: 30s

sensor:
  - platform: victron
    victron_id: victron0
    max_power_yesterday:
      name: "${name} max power yesterday"
    max_power_today:
      name: "${name} max power today"
    yield_total:
      name: "${name} yield total"
    yield_yesterday:
      name: "${name} yield yesterday"
    yield_today:
      name: "${name} yield today"
    panel_voltage:
      name: "${name} panel voltage"
    panel_power:
      name: "${name} panel power"
    battery_current:
      name: "${name} panel current"
    battery_voltage:
      name: "${name} battery voltage"
    day_number:
      name: "${name} day number"
    charging_mode_id:
      name: "${name} charging mode id"
    error_code:
      name: "${name} error code"
    tracking_mode_id:
      name: "${name} tracking mode id"
    load_current:
      name: "${name} load current"

text_sensor:
  - platform: victron
    victron_id: victron0
    charging_mode:
      name: "${name} charging mode"
    error:
      name: "${name} error"
    tracking_mode:
      name: "${name} tracking mode"
    firmware_version:
      name: "${name} firmware version"
    device_type:
      name: "${name} device type"
    serial_number:
      name: "${name} serial number"

binary_sensor:
  - platform: victron
    victron_id: victron0
    load_state:
      name: "${name} load state"
    relay_state:
      name: "${name} relay state"

hi ,
use these pins :
GPIO16 as (RX)
GPIO17 as (TX) (default NC)
it is classic wiring for esp32 chips and UART 2
uart:
  id: uart_0
  tx_pin: GPIO17  # Not used! The communication is read-only
  rx_pin: GPIO16
  baud_rate: 19200
  rx_buffer_size: 256

Thankyou
I can't seem to get this to work either. At my desk it connects to wifi and I can visit it via localip or see logs in ESPHome. But there is obviously no data.
When I take it to the victron and plug in the white RX cable on the JST cable to GPIO 16, it won't ever seem to jump back on the wifi. Tested both with ground connected or unplugged.
INFO ESPHome 2024.4.2
INFO Reading configuration /config/esphome/esphome-web-41ff18.yaml...
INFO Updating https://github.com/KinDR007/VictronMPPT-ESPHOME.git@main
WARNING 'victron_solar_controller': Using the '_' (underscore) character in the hostname is discouraged as it can cause problems with some DHCP and local name services. For more information, see https://esphome.io/guides/faq.html#why-shouldn-t-i-use-underscores-in-my-device-name
INFO Starting log output from <IP_ADDRESS> using esphome API
WARNING Can't connect to ESPHome API for victron_solar_controller @ ip: Error connecting to [AddrInfo(family=<AddressFamily.AF_INET: 2>, type=<SocketKind.SOCK_STREAM: 1>, proto=6, sockaddr=IPv4Sockaddr(address='ip', port=6053))]: [Errno 113] Connect call failed ('ip', 6053) (SocketAPIError)
INFO Trying to connect to victron_solar_controller @ ip in the background

My gate controller is right at the limit of the property so thinking that it's a little too far away even though there is wifi there. I use a sonoff controller to actually monitor and control the gate at the same location. It complains occasionally but it does connect and stays connected for the most part.
Perhaps the wireless antenna on the esp board is a bit weaker. Thoughts on if you think this might be the problem or if I have done something else wrong. It is night time now but I can take some photos showing wiring etc tomorrow.
Should I look for a board with external antenna (if such a thing exists).

Those MPPT TX pins are 5vs, the ESP32-WROOM can only take a max of 3.6v` on its GPIO pins. You are probably overvolting the ESP32 board and it just crashes completely rather than dropping off of wifi.
You will need a voltage divider if you want to connect to GPIO16; check out the diagram in this repo here https://github.com/RalfJL/VE.Direct2MQTT/wiki/Hardware
I use a ESP32-WROOM with a BMV700, its TX pin is 3.3v as per Victron documentation here https://www.victronenergy.com/live/vedirect_protocol:faq#q4is_the_vedirect_interface_33_or_5v


don't know how to build these path

hey I've just updated to the latest nixinate master and it seems to be giving me the following error
don't know how to build these paths:
  /nix/store/cw9hr92idq1p5iady76xm4si9jqha2yw-nixos-rebuild
error: build of '/nix/store/cw9hr92idq1p5iady76xm4si9jqha2yw-nixos-rebuild' failed

Do I need to change anything ?

that was indeed what broke it. I have no idea what hermetic is, but you can't do it on a remote build apparently.
Thanks for the quick response!

@happysalada Hermetic is documented in the README. What it will do is copy nix and nixos-rebuild to the remote machine, rather than impurely executing the nixos-rebuild on the remote machine, assuming it is there.k

oh that makes sense, I'm on a darwin machine and trying to build on a linux machine. That's why it doesn't work. Ok, thanks for the explanation!


Any considerations for updates to use Llama 3.1?

I realize that it was just released yesterday (7/23/2024) and incorporating new/updated models can take a lot of time and energy, but was wondering if Llama 3.1 (with the extended context window) was something that is on the radar for inclusion with candle.
Is this something you typically only handle internally? Is it something you'd take outside contributions for?

If you just point the model at the new repo it should mostly work, it's largely the same architecture.
There is one change that's needed to incorporate some differences in the RoPE layer, but it should generate valid tokens without that.

@EricLBuehler incorporated the necessary change in mistral.rs, if someone wants to upstream it:
https://github.com/EricLBuehler/mistral.rs/pull/618

@zackangelo Funny enough I started experimenting with it yesterday in the candle repo cloned locally, and after making some adjustments to the configuration expectations (i.e. reading from config.json, it at least loaded the model but didn't inference.
Ill take a look at what Eric has done. I am hoping it's a relatively easy import. :)

if someone wants to upstream it

I can try taking a crack at it. Others here might be more successful, but for my project I could almost immediately use it if it were available.

@zackangelo Funny enough I started experimenting with it yesterday in the candle repo cloned locally, and after making some adjustments to the configuration expectations (i.e. reading from config.json, it at least loaded the model but didn't inference.
Ill take a look at what Eric has done. I am hoping it's a relatively easy import. :)

Did you get an error? The model implementation I'm currently using for Mixlayer is pretty close to what's in the examples right now and it works.

Nope, no error, it just didn't inference at all. It returned 0 tokens.

@BradyBonnette @zackangelo I just opened a PR for this!

Closing this due to https://github.com/huggingface/candle/pull/2359 getting merged.
Huge shout out to @EricLBuehler for getting it squared away in short order, and @LaurentMazare for getting it merged in so quickly!


Extremely long compilation times

I'm using an AMD node with over 128 cpus, but building rocBLAS with Tensile is excruciatingly slow.
Builds are taking more than 2 hours and 15 minutes. It builds almost completely single-threaded and some single clang processes are running for 20 minutes straight. 2 hours is spent compiling kernels.
What can I do to speed it up?
This is from the build logs:
################################################################################
# Tensile Create Library
# Detected local GPU with ISA: gfx906
# Asm caps for gfx803:SupportedISA=1 HasExplicitCO=0 HasExplicitNC=0 HasDirectToLds=1 HasAddLshl=0 HasSMulHi=0 HasCodeObjectV3=1 MaxVmcnt=15 SupportedSource=1
# Arch caps for gfx803:HasEccHalf=0 Waitcnt0Disabled=0 SeparateVscnt=0 CMPXWritesSGPR=1 HasWave32=0
# Asm caps for gfx900:SupportedISA=1 HasExplicitCO=1 HasExplicitNC=0 HasDirectToLds=1 HasAddLshl=1 HasSMulHi=1 HasCodeObjectV3=1 MaxVmcnt=63 SupportedSource=1
# Arch caps for gfx900:HasEccHalf=0 Waitcnt0Disabled=0 SeparateVscnt=0 CMPXWritesSGPR=1 HasWave32=0
# Asm caps for gfx906:SupportedISA=1 HasExplicitCO=1 HasExplicitNC=0 HasDirectToLds=1 HasAddLshl=1 HasSMulHi=1 HasCodeObjectV3=1 MaxVmcnt=63 SupportedSource=1
# Arch caps for gfx906:HasEccHalf=1 Waitcnt0Disabled=0 SeparateVscnt=0 CMPXWritesSGPR=1 HasWave32=0
# Asm caps for gfx908:SupportedISA=1 HasExplicitCO=1 HasExplicitNC=0 HasDirectToLds=1 HasAddLshl=1 HasSMulHi=1 HasCodeObjectV3=1 MaxVmcnt=63 SupportedSource=1
# Arch caps for gfx908:HasEccHalf=1 Waitcnt0Disabled=1 SeparateVscnt=0 CMPXWritesSGPR=1 HasWave32=0
# Asm caps for gfx1010:SupportedISA=1 HasExplicitCO=1 HasExplicitNC=1 HasDirectToLds=1 HasAddLshl=1 HasSMulHi=1 HasCodeObjectV3=1 MaxVmcnt=63 SupportedSource=0
# Arch caps for gfx1010:HasEccHalf=0 Waitcnt0Disabled=0 SeparateVscnt=1 CMPXWritesSGPR=0 HasWave32=1
# Asm caps for gfx000:SupportedISA=0 HasExplicitCO=0 HasExplicitNC=0 HasDirectToLds=0 HasAddLshl=0 HasSMulHi=0 HasCodeObjectV3=0 MaxVmcnt=0 SupportedSource=1
# Arch caps for gfx000:HasEccHalf=0 Waitcnt0Disabled=0 SeparateVscnt=0 CMPXWritesSGPR=1 HasWave32=0
Tensile::WARNING: Global parameter EmbedLibrary = None unrecognised.
# CodeObjectVersion from TensileCreateLibrary: V2
# CxxCompiler       from TensileCreateLibrary: hipcc
# Architecture      from TensileCreateLibrary: all

Generating kernels: Done.
*
Compiling source kernels: Launching 64 threads...
...
Compiling source kernels: Done.
# Kernel Building elapsed time = 7181.6 secs
# Tensile Library Writer DONE
################################################################################

-- TARGET: gfx803
-- TARGET: gfx900
-- TARGET: gfx906
-- TARGET: gfx908
-- Performing Test COMPILER_HAS_HIDDEN_VISIBILITY
-- Performing Test COMPILER_HAS_HIDDEN_VISIBILITY - Success
-- Performing Test COMPILER_HAS_HIDDEN_INLINE_VISIBILITY
-- Performing Test COMPILER_HAS_HIDDEN_INLINE_VISIBILITY - Success
-- Performing Test COMPILER_HAS_DEPRECATED_ATTR
-- Performing Test COMPILER_HAS_DEPRECATED_ATTR - Success
-- Configuring done
-- Generating done
-- Build files have been written to: /dev/shm/spack/hstoppel/spack-stage-rocblas-3.5.0-zhkw5kdvy5cvpzwsdd3rwkrmwdg3a3bi/spack-build
==> rocblas: Executing phase: 'build'
==> [2020-08-12-23:28:03.231710] 'make' '-j128'

@haampie , which ROCm version did you use to build rocBLAS?  If it is ROCm 3.5 or later, then you'd be using hipclang, which does take a lot longer to finish the build.  However, on a machine with plenty of main memory (say >= 128GB), one can set the following 2 environment variables:
export HIPCC_COMPILE_FLAGS_APPEND="-O3 -Wno-format-nonliteral -parallel-jobs=4"
export HIPCC_LINK_FLAGS_APPEND="-O3 -parallel-jobs=4"
which speeds up a rocBLAS build substantially.
The above suggestion applies to ROCm 3.5 or later.

Thanks, I will try this out :) memory shouldn't be a problem either


can't login with supabase 0.15.6+

Bug report
Describe the bug
Requests to http://localhost:54321/auth/v1/token?grant_type=password return 503 Service Temporarily Unavailable starting with release 0.15.6
When checking out sha 5a3db53 supabase.auth.signIn() works as expected.
To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:

git clone git@github.com:supabase/cli.git cli
git checkout 3a3f20b
go build
cd  # mine was created with supabase cli ~0.15.0
/cli start
Call supabase.auth.signIn({ email, password })

Expected behavior
Authorization succeeds.
System information

OS: macOS
Browser: Firefox (if applies) [e.g. chrome, safari]
Version of supabase-js: 1.28.5
Version of Node.js: v14.16.0

❯ git bisect log
# bad: [3a3f20bfa808bd13f8ff89c6e0c434ec3b6b1847] fix: etc. etc.
# good: [5a3db5398652b9203df68bdf0108fdda2cff1e76] fix: Apply extensions.sql before migrations
git bisect start '3a3f20bfa808bd13f8ff89c6e0c434ec3b6b1847' '5a3db53'
# bad: [c44acfef6a906ae550760325d8fe91bec63045a4] fix: --host localhost
git bisect bad c44acfef6a906ae550760325d8fe91bec63045a4
# good: [5659834bc3bf49de6d78cd8e83922bcc0ccf1850] chore: more ✨💅✨
git bisect good 5659834bc3bf49de6d78cd8e83922bcc0ccf1850
# bad: [ea41e841346a575d1fce760e8a8b5196bb3c3564] fix: update images
git bisect bad ea41e841346a575d1fce760e8a8b5196bb3c3564
# good: [90a8eae9aa8526bc5e8e99c2743dc5ae764803e3] chore: show `Dumping branches...` when stopping
git bisect good 90a8eae9aa8526bc5e8e99c2743dc5ae764803e3
# first bad commit: [ea41e841346a575d1fce760e8a8b5196bb3c3564] fix: update images

It works after doing supabase db reset and creating a new user.

Hmm, couldn't quite nail down what the issue is, but there was a problem when using the Postgres 14 image, though that should've been resolved in 0.15.6. Will keep this open for a while in case other people are having similar issues.

Looks to be a heisenbug, so closing for now.


Fix the previous chatBox espresso test's

015d486: Fixes wrong hint shown in the chatBox

Automated message from Dropbox CLA bot
@kunall17, it looks like you've already signed the Dropbox CLA. Thanks!

These compile & run but none of them pass. I think the problem is that I'm logged in when they start. Can you write up some docs on what state your app should be in when these start?
Functional tests are flakey, I'm a bigger proponent of unit tests for Android stuff.

This has been merged!


Default focus to the name block on new device, etc... views

Right now there's no default block, it'd be a little faster if we defaulted the view to setting focus in the name block of the view.
(adding as an issue to see if there's desire for the feature, before I work on the PR)

Personally, I'm not a big fan of autofocus, but let's see what others think.

I'm going to close this out as I don't really see the value in it.


inject custom authorization

Description
I would like to insert a unique header before each call. There is an "HttpInterceptor" that does this, but for some reason it does not run in the case of meilisearch/angularsearch. Is it possible to set your own token for each call before each call?
I would like to run the calls through my own backend, because meilisearch currently does not support that a specific api key can only see certain data within the index, and it is safer to assign it to the user's token. The backend would actually work as a proxy, validating the user data, then modifying the header and forwarding it locally to meilisearch.

Expected behavior
Compatibility with HttpInterceptor
Environment

OS: Windows 11
Browser: Google Chrome: 108.0.5359.125 x64
Meilisearch version: 0.30.4
instant-meilisearch version: ^0.10.1
angular-instantsearch version: ^4.3.1
Angular version: ^14.2.8

Code
For my own backend calls, the token is included perfectly in the header.
@Injectable()
export class TokenInterceptor implements HttpInterceptor {

  constructor(
    private inj: Injector,
    private auth: Auth,
  ) {
  }

  intercept(request: HttpRequest<any>, next: HttpHandler): Observable<HttpEvent<any>> {

    if (request.url.indexOf('oauthCallback') > -1) {
      return next.handle(request);
    }

    return from(this.auth.currentUser.getIdTokenResult()).pipe(
      switchMap(token => {
        request = request.clone({
          setHeaders: {Authorization: `Bearer ${token.token}`}
        });
        return next.handle(request);
      }),
    );
  }
}

Hey @Karman40, thanks a lot for the feedback :) I'm moving it to instant-meilisearch as the issue comes from the client and not angular itself.
About your interceptor not working, I'll investigate. Meanwhile, it is possible in Meilisearch to create tokens with specific rights on an index. See this doc.
If this does not cover what you need as permission, please tell us so we can add that in the product feedbacks!


Inspect code style

Some codes do not follow the code style guide rule.
Find the mistake and patch it.
Here is the code style rule in korean.
https://github.com/ztkmkoo/dss/wiki/Code-Style(KR)

Assign it to me.

@Amazansia
Hi, it looks i miss description bout class format.
class Foo {

    // static fields
    // static initializer
    // static methods
    // instance fields
    // constructors
    // instance methods
}


Allow user to select charge units: Ah or C

Version: 0.8.3
Platforms: All
Batteries use Ah, not the SI unit of C.  The software should do this conversion for when you want to work in Ah.  See the forum thread.

Duplicated of #39


Structure entry output format standardization (for crystal  coordinates)

We need a standardization of the structure entry type that allows communicating "where the atoms are", composition, etc.

Points related to structure :

How to deal with occupancies (also variable occupancies)
How to deal with correlation (in a similar way of what "assembly disorder groups" are in CIF files)
How to extend that to trajectories ? (keep in mind the variable occupancies above)

Here is the suggested format that resulted from the group discussion:
{
  "dimension_types": [
    1,
    1,
    1
  ],
  "lattice_vectors": [
    [
      4.0,
      0.0,
      0.0
    ],
    [
      0.0,
      4.0,
      0.0
    ],
    [
      0.0,
      0.0,
      4.1
    ]
  ],
  "cartesian_site_positions": [
    [
      0.0,
      0.0,
      0.0
    ],
    [
      2.0,
      2.0,
      1.95
    ],
    [
      2.0,
      2.0,
      2.15
    ],
    [
      2.0,
      2.0,
      0.0
    ],
    [
      2.0,
      0.0,
      2.0
    ],
    [
      0.0,
      2.0,
      2.0
    ]
  ],
  "species_at_sites": [
    "BaCa",
    "Ti",
    "Ti",
    "O",
    "O",
    "O"
  ],
  "species": {
    "BaCa": {
      "chemical_symbols": ["Ba", "Ca"],
      "concentration": [0.4, 0.5],
      "mass": 88.5
    },
    "O": {
      "chemical_symbols": ["O"],
      "concentration": [1.0],
      "mass": 15.9994
    },
    "Ti": {
      "chemical_symbols": ["Ti"],
      "concentration": [1.0],
      "mass": 47.867
    }
  },
  "assemblies": [
    {
      "sites_in_groups": [ [1], [2] ],
      "group_probabilities": [ 0.5, 0.5 ]
    }
  ],
}

And some comments:

dimension_types: Compulsory; follows definitions of https://esl.cecam.org/ESCDF_-_System but we allow only values of 0 (non-periodic direction specified by the lattice vector) or 1 (periodic)
lattice_vectors: Compulsory; optional for "dimension_types" = [0,0,0]. See  https://esl.cecam.org/ESCDF_for definition of which index runs faster -_System for the meaning. Always 3x3, units: angstrom
cartesian_site_positions: Compulsory; see also https://esl.cecam.org/ESCDF_-_System. Length: number of sites times 3. It is possible to have multiple sites with the same coordinates, e.g. if we have assemblies, see below
species_at_sites: compulsory;  must be of length equal to the number of sites. Contains the key of the species dictionary (see below), identifying the species at each site. Each site has only one species associated to it.
species dictionary: Compulsory. Key: the value of the species_at_sites.

chemical_symbols: compulsory, must be a list, even of list one. List  of chemical elements. "X" is allowed for a non-chemical element.
concentration: list, must have the same length as the corresponding chemical_symbols list. In general compulsory, except when it's a single list where the value is 1: [1.0]; then it can be omitted. Float numbers represent the relative concentration of that element. Sum of all numbers for a given specie must sum to a value >=0 and <=1 (a sum <1 means there is a probability to have a vacancy).
mass; optional, expressed in a.m.u. for each specie. This allows e.g. to make superlattices of C^12/C^13 (e.g. with two species with names C12 and C13, both with 100% C chemical symbol)

assemblies: Optional, absent if no partial occupancies, but client MUST check its presence as it changes the way to interpret the crystal structure.

sites_in_groups: index of the sites (0-based) that belong to each group for each assembly. Example: [[1], [2]]: two groups, one with the second site, one with the third. Example: [[1,2], [3]]: one group with the second and third site, one with the fourth. This flag is compulsory if assemblies are specified.
group_probabilities: statistical probability of each group. Must sum to >=0 and <=1. Sum <= 1 means there is some non-zero probability that the group  is completely absent. The flag is compulsory if assemblies are specified.

Summarizing the comment from @gmrigna :

in CIF, the expression of a mixture of elements on a particular site is simply expressed by repeating the coordinates. The proposal currently suggests to represent this differently without enforcing it. This will lead to people using our standard to express the same thing in different ways.

Approved with the following comments:

Two repeated coordinates refer to two positions in space that are technically not the same but too close to tell apart in the floating point representation. A structure where two different species are placed on these without the specification of an assembly to ensure they are not at those locations at the same time is technically a valid representation of a structure, but represents a physically non valid system where two atoms can be at nearly the same point in space at the same time.
The consequence of this is that someone interpreting the structure information does not need to "detect" repeated coordinates; coordinates that look repeated are still to be treated as conceptually different.
Another consequence of this is that it is theoretically not possible to represent an assembly where two different groups has one species at the exact same location. This situation is however practically indistinguishable from the species being at different positions that are similar within the floating point representation.
The separate x,y,z coordinates are allowed to be given as 'null' if unknown.

There are differing opinions on how to handle systems with atoms at unknown coordinates. How to resolve this is deferred into a separate issue. Two alternatives is to allow null in the coordinate lists, OR put an extra optional property that alongside the known coordinates describe the information that exists about the atoms at unknown positions.

The separate x,y,z coordinates are allowed to be given as 'null' if unknown. E.g., for disordered systems where coordinates are not fixed, and certain situations where light atoms can be near heavier atoms without knowing exactly where. However, we note that the latter situation can alternatively be represented by a species ~ "CH4", which can also be used in an assembly with "C" to indicate that it is not clear which one of the C atoms has the four H. (If it is known that all 4 always sits on on C...)
There is an extra required property all_coordinates_known that is true or false depending on whether any coordinate is a null value. (So clients can easily use filter to get only structures where all coordinates are known.)

Actually, there seems to be the third way to represent atoms with unknown positions: specify no site for them (so no 'null' coordinates), and have a species that is on no site.

I pretty much like the proposal.
One thing I dislike is that the concentration can be <1, I would very much prefer that vacancies are done explicitly using an X element or something similar.
Because if one is not careful with printing out, and things do not sum up exactly to 1 at which point doi you consider it not a rounding error but really a low probability vacancy (think 1/3,1/3,1/3)?
A minor thing that I like about the metainfo is that we have global names, so that if you say x it is clear without having to give the context, and then we basically enforced that one should use a bit longer names (with at least a _) as meta info names, so there is even another way to recognize metainfo names.


design: Run Envoy Gateway locally on host

Relates to https://github.com/envoyproxy/gateway/issues/1001 & https://github.com/envoyproxy/gateway/issues/1035

hey @AliceProxy thanks for taking a look at this PR, hoping to continue working on this PR and define the APIs within this PR so the design and API go in together


Update README.md

Corrected some grammatical mistakes.

Thanks for your valuable contribution 😊.Don't forget to 🌟our repository.


Review Select all rows command

Description
Now it's not obvious what is the exact purpose of this command and what is the difference between Evaluate sql expression and Select all rows options from the sql editor context menu. It looks like they do the same. We need to review this option. Maybe we could tune it to make automatic Select * from <selected text> top 100; as it was requested here #16074 or remove it.
DBeaver Version
DBeaver 22.3.3

Evaluate sql expression evaluates some parts of query, for example it can evaluate 1+2.
 Select all rows option allows to execute the query under the cursor without limit to fetch all data.

@E1izabeth any update? when will this feature available?

This issue is closed. In the previous comment I decided how this can be done in DBeaver

Can you please provide a sample how you intend to achieve this using mentioned solution?


Fix go cross

@agherzan @floion @michal-mazurek

@telphan this needs to reference an issue

Can you open an issue tab for this repo? @agherzan

@telphan there is already an issue in resinos for this (afaik)

Connects to resin-os/resinos#136


Add updated content when html is updated

@KendallDoesCoding Hi, there a few things I am not yet sure about. But as an experienced developer. You should be able to figure it out.
Here is the plan -

The python script at the basic level goes through the index.html file parses the table and writes it into the read me file.
But it has to be run every time there is a change in the html file.
Now through GH Action we can achieve this, but we cannot write a file in the repo.
That's where we need further help.

What do you think?

Alright, how about something like this -
name: Run Python script on index.html update
on:
  push:
    paths:
      - 'index.html'
jobs:
  build:
    runs-on: ubuntu-latest
    steps:
      - name: Check out repository code
        uses: actions/checkout@v2
      - name: Set up Python environment
        uses: actions/setup-python@v2
        with:
          python-version: 3.8
      - name: Install dependencies
        run: |
          pip install beautifulsoup4==4.10.0
      - name: Run Python script
        run: python main.py

@KendallDoesCoding Alright, how about something like this -
name: Run Python script on index.html update
on:
  push:
    paths:
      - 'index.html'
jobs:
  build:
    runs-on: ubuntu-latest
    steps:
      - name: Check out repository code
        uses: actions/checkout@v2
      - name: Set up Python environment
        uses: actions/setup-python@v2
        with:
          python-version: 3.8
      - name: Install dependencies
        run: |
          pip install beautifulsoup4==4.10.0
      - name: Run Python script
        run: python main.py

Perfect. What's the python script tho?


Add basic interaction with containers and images

It would be nice to at least be able to display the running containers and stored images...

nerdctl.lima container list - https://github.com/afbjorklund/lima-gui/wiki/Containers

nerdctl.lima image list - https://github.com/afbjorklund/lima-gui/wiki/Images

Start with nerdctl (containerd), but should be simple to add docker and podman support:

docker.lima

podman.lima

Also:

version

info

With the basic functionality of being able to select engine (nerdctl/docker/podman) to use.


Make processing work with rectangular arrays

Branch rectangular has been created for development of the sim processor class so that it can deal with rectangular arrays.  Testing of all methods used in the napari plug-in are successful so far, including GPU processing.
Most changes are in the baseSimProcessor.py but some also in _sim_widget.py to cope with the fact that frequency plane images are now rectangular too and circular annotations need to be placed correctly and/or replaced with ellipses.

@andreabassi78 Check that this is working OK for you, please.
If we are happy with rectangular frequency plane images for xcorr and spectrum then I don't think we need to reshape these for display.

Hi @pptman, I have tested the code and it works, the xcorr and spectrum are rectangular (with ellipses for the circles) and I also don't think this should be changed.
I also made some changes to this branch to fix the visualization issue that we had with python 3.10 (some of the changes are the same that you made in the fixesforpython3.10 branch).
We could promote this branch to the main one. What do you think?
I am not familiar with branch merging, and I'm not sure what are the correct steps (we now have at least 3 branches to merge: rectangular, fixesforpython3.10, ra_sim).
Can you let me know if everything works on your mac?
(I have changed some dependencies too, please let me know if this is causing problems)

@andreabassi78 I have done the merge.  Rectangular and fixesforpython3.1 were trivial as they are the same branch really.  ra_sim got a bit messy but with the right merge tool (I use pycharm's internal merge) it was easy to sort out.  It would on my mac with both my 3.9 and 3.10 versions, even after upgrading magicgui.  If you want to test the final merged version to make sure I haven't done anything wrong it is now on branch main again.  I'm not sure where you have changed dependencies (I can't see it in the git history) so am not really sure how to test this.

@andreabassi78 I notice the import changes that you have made but these don't work on my machine when running from pycharm (maybe I need to set something up?).  The error message is:
    from .baseSimProcessor import BaseSimProcessor
ImportError: attempted relative import with no known parent package

I also noticed the failed tox tests.  I noticed that one failure was to do with two environment variables on one line.  Meizhu found this too and we fixed it on the simulator but alas not here!

@andreabassi78 I took a look at the import error message I was getting and it seems that a better structure for us would be to move all the processing code into its own submodule, then the from .baseSimProcessor import BaseSimProcessor lines will work.  This works for me when running the code from within pycharm.  I have put the changes in a new branch processors_submodule.  I will test this from my jupyter notebook too.
Looking at the error messages from the tests it seems to be the line importing from module widget_settings that is not working.  I suspect this may be a related problem?

@andreabassi78 there is a solution that seems to work in branch processors_submodule but I suspect that this is not the real way to do it.

I pushed this branch as branch npe2 in order to trigger the testing workflow and it passes.  I agree, this is probably not the correct way to do it, though you could argue that the if__name__ == "__main__" is just catching the case when you are trying to run the code directly during testing/development.  So maybe the code used by else: is the correct way to write things.

@pptman Great to add the submodule processors, it makes the project clearer, even if it does not solve the dependencies project.
Adding the dots in e.g. "from .processors.hexSimProcessor import HexSimProcessor" is the right way to work with the module.
I looked at a few other plugins and nobody runs the script directly.
Therefore, in another project, I decided to remove the if __name__ == '__main__' part from the _widget.py file and create a separate py script to run the plugin.
It works, and it avoids the possible confusion of having if/else.
Please note that the _widget_runner.py file must be in the folder src. Having the _widget_runner.py in a separate file and folder might be uncomfortable but it makes the package clearer.
Some hints are here.
I would use this same approach for the sim-processor plugin if you like it.

I agree, remove the main stuff from the main code and create a short runner script in src.  In fact src is where I have to put my jupyter notebook to get that to run correctly too.
Mark
From: Andrea Bassi @.>
Date: Wednesday, 1 February 2023 at 07:36
To: andreabassi78/napari-sim-processor @.>
Cc: Neil, Mark A A @.>, Mention @.>
Subject: Re: [andreabassi78/napari-sim-processor] Make processing work with rectangular arrays (Issue #16)
This email from @.*** originates from outside Imperial. Do not click on links and attachments unless you recognise the sender. If you trust the sender, add them to your safe senders listhttps://spam.ic.ac.uk/SpamConsole/Senders.aspx to disable email stamping for this address.
@pptmanhttps://github.com/pptman Great to add the submodule processors, it makes the project clearer, even if it does not solve the dependencies project.
Adding the dots in e.g. "from .processors.hexSimProcessor import HexSimProcessor" is the right way to work with the module.
I looked at a few other plugins and nobody runs the script directly.
Therefore, in another projecthttps://github.com/andreabassi78/napari-psf-simulator/tree/main/src, I decided to remove the if name == 'main' part from the _widget.py file and create a separate py scripthttps://github.com/andreabassi78/napari-psf-simulator/blob/main/src/_widget_runner.py to run the plugin.
It works, and it avoids the possible confusion of having if/else.
Please note that the _widget_runner.py file must be in the folder src. Having the _widget_runner.py in a separate file and folder might be uncomfortable but it makes the package clearer.
Some hints are herehttps://iq-inc.com/importerror-attempted-relative-import/#packages-explained.
I would use this same approach for the sim-processor plugin if you like it.
—
Reply to this email directly, view it on GitHubhttps://github.com/andreabassi78/napari-sim-processor/issues/16#issuecomment-1411590048, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEB5BENSME3UUZDQFBYTEPTWVIG7FANCNFSM6AAAAAAT5YJ3JE.
You are receiving this because you were mentioned.Message ID: @.***>

Done, the runner is now in the src folder. Processing of rectangular passed all the tests.
The rectangular processing deserves a new version.
A new version v0.0.10, is now available on the Napari Hub.


Please add a change log to your releases

Please see the list of releases. From now on I'll include there the list of changes for each release.

Thanks


CI lint failure

Currently the lint CI workflow recurrently fails. The CI should be corrected or removed if not relevant.

Fixed in #57 👍


EditorSpinSlider loses click hold if mouse moves outside of input box

Godot version
4.0alpha12
System information
Win 10/64 NVIDIA GeForce GTX 1060/PCIe/SSE2
Issue description
When grabbing a slider in the inspector, you can click the circle and slide the values left and right. However if the mouse leaves the light blue box, the slider loses your held click. You then have to look away from the 3D viewport, then find and click the little circle again to resume adjusting the values. It's rather annoying when making adjustments to materials and many other items.

Switching the editor to Single Window Mode has no effect.
Steps to reproduce
Make a material, adjust the sliders, move the cursor outside of the input box, get annoyed.
Minimal reproduction project
n/a

Can you reproduce this in 3.4.4 or 3.5.rc7?

Works as expected in 3.5. Slider retains click hold, as the mouse moves anywhere. So I can keep my eyes on the viewport and not worry about where the mouse is.

As a side note, range properties created via gdextension also suffer from this bug, thus it is not limited to that material slider.

As digging through the code, I found some potentially related issue
void EditorSpinSlider::_grabber_gui_input(const Ref<InputEvent> &p_event) {
	Ref<InputEventMouseButton> mb = p_event;

	if (grabbing_grabber) {
		if (mb.is_valid()) {
			if (mb->get_button_index() == MouseButton::WHEEL_UP) {
				set_value(get_value() + get_step());
				mousewheel_over_grabber = true;
			} else if (mb->get_button_index() == MouseButton::WHEEL_DOWN) {
				set_value(get_value() - get_step());
				mousewheel_over_grabber = true;
			}
		}
	}

	if (mb.is_valid() && mb->get_button_index() == MouseButton::LEFT) {
		if (mb->is_pressed()) {
			grabbing_grabber = true;
			if (!mousewheel_over_grabber) {
				grabbing_ratio = get_as_ratio();
				grabbing_from = grabber->get_transform().xform(mb->get_position()).x;
			}
		} else {
			grabbing_grabber = false;
			mousewheel_over_grabber = false;
		}
	}

	Ref<InputEventMouseMotion> mm = p_event;
	if (mm.is_valid() && grabbing_grabber) {
		if (mousewheel_over_grabber) {
			return;
		}

		float scale_x = get_global_transform_with_canvas().get_scale().x;
		ERR_FAIL_COND(Math::is_zero_approx(scale_x));
		float grabbing_ofs = (grabber->get_transform().xform(mm->get_position()).x - grabbing_from) / float(grabber_range) / scale_x;
		set_as_ratio(grabbing_ratio + grabbing_ofs);
		update();
	}
}

The current code expects the user to press the left mouse button while simultaneously spinning the mouse wheel (for me this seems somewhat complicated). The common behavior of such a feature is to simply react on the mouse wheel spin when the cursor is over the slider. Can't imagine, that this is intended, but I wanted to ask before I do anything further.

Should work exactly as Godot 3 IMO. Grab slider, it slides no matter where the mouse moves until released.
Independent of the mouse click, if the wheel is changed while the mouse hovers in the box, it should change the value.
I think it should ignore the mouse wheel if it's already clicked and held.

I confirm the fix in https://github.com/godotengine/godot/pull/63598 works.


Add veterancy skill bonuses.

Hey Simon.  I may revisit it someday but, TBH, I haven't played in quite a while.

Test veterans in https://drive.google.com/drive/u/0/folders/1e9hu5NO_morudcX5zgtPtIk0jR69XwP_ - look OK?

That is really cool, Simon.


Light bugs when changing uniforms

The branch "lights" has a very stange bug on lights.
Just the fact of removing the uniform "ambient_color" in the "deferred_shading_point_light.fp" shader triggers the bug.
Adding a uniform "abient_color" to "deferred_shading_merge.fp" also triggers the bug.

@DorianPinaud can you take a look at it please ?

yep

Thx handsome
2015-03-20 17:36 GMT-04:00 DorianPinaud<EMAIL_ADDRESS>
yep
—
Reply to this email directly or view it on GitHub
https://github.com/Another-Game-Engine/AGE/issues/408#issuecomment-84155341
.

--
[image: dancing frog animation][image: dancing frog animation][image:
dancing frog animation][image: dancing frog animation][image: dancing frog
animation][image: dancing frog animation][image: dancing frog animation][image:
dancing frog animation][image: dancing frog animation][image: dancing frog
animation][image: dancing frog animation][image: dancing frog animation][image:
dancing frog animation][image: dancing frog animation]

I think that i fixed it, look at my last PR


eMail-notifications

new user registered
new message/conversation started
user subscribed your profile
new friendshiprequest
someone posted on your timeline

Mostly, it's the job of notifications module.
However email notification for Convos is really needed.


Ensures the environment variable is correctly set

This PR closes #1

Would it be possible also to generate a new release?

Hi @droidpl, thanks for your contribution!
We're still in early development stages, the changes you proposed are indeed valid however we're going to perform massive refactoring of the entire codebase so they may not be relevant in a few days.
We're in the process of building our community guidelines, Code of Conduct and contribution policy, after which we will begin monitoring the Pull Requests area and accept merges from outside contributors. I hope we will see you then :)


feature/78: update Ansible inventory documentation

Will there be some updates to the section ansible_inventory.md#k8s-connections? Some host vars used there that were intended to take deployment location properties seem outdated.


no adaface loss in source code

Hi, I didnt find any code about adaface loss in your training source code for pretrained adaface AdaFace + CFSM model. Can you provide adaface loss source code for that model?
The readme for CFSM model is hard to understand in the training with custom dataset part, can you also provide the custom dataset config for it too?

Thank you for your interest in our work. I will add the AdaFace loss soon.


Validate facility form fields

Related Issue or Background Info

Issue #917

Changes Proposed

Adds yup validation to facility fields
Makes sure to validate the correct fields
Adds same phone number validation on frontend as is on backend (google)

Additional Information

This has more boilerplate than I'd ultimately like. I think we could use react-hook-form or another established form library on the front-end to make our forms consistent, have less boilerplate, and integrate relatively easily with a validation library like yup.

Would like to ignore the SonarCloud duplication failure, but will defer to reviewers on that one.
Basically, I think we have some tech debt in that we don't have a form management library to abstract away my home-rolled validateField and clearError functions. Therefore, I wrote them and then duplicated them across two components and SonarCloud is not a fan of that.


Moran Scatterplots with equal bounds on X and Y axes?

I realised that the method to generate a Moran scatterplot adjusts the limits of the X and Y axes individually. For example, see this figure sourced from here:

Given how the Moran Scatterplot is designed, I'm thinking it might be more informative to set the bounds for both axes to be the same. That would avoid the effect in the figure above, where a statistically clear non-significant result might give the sense of a pronounced slope. The Y axis is expressed in the same units of the X, so I think it's safe to assumed that if the bounds are different, it's because of the actual values in the data.
What do folks think? @slumnitz, @sjsrey, @ljwolf

@darribas
Currently bounds are defined with:
ax.spines['left'].set_smart_bounds(True)
ax.spines['bottom'].set_smart_bounds(True)

in _create_moran_fig_ax.
I think Id decided to go this way before, because with non standardised data, the plot itself started to have weird shapes. e.g attribute values ranging from 0-500 and spatial lag ranging with significantly lower values, resulting in an almost unreadable plot. Also due to most plots being small multiples it started to be tricky to place them next to each other in a visually appealing way.
We could potentially integrate this for zstandardised values only? If zstandard=True? Otherwise maybe an option to disable smart_bounds_if needed in the function API?

For that to be true, wouldn't we need to set aspect=equal always?
Sent from Mobile
On Fri, Mar 29, 2019, 18:25 Stefanie Lumnitz<EMAIL_ADDRESS>wrote:

@darribas https://github.com/darribas
Currently bounds are defined with:
ax.spines['left'].set_smart_bounds(True)
ax.spines['bottom'].set_smart_bounds(True)
in _create_moran_fig_ax.
I think Id decided to go this way before, because with non standardised
data, the plot itself started to have weird shapes. e.g attribute values
ranging from 0-500 and spatial lag ranging with significantly lower values,
resulting in an almost unreadable plot. Also due to most plots being small
multiples it started to be tricky to place them next to each other in a
visually appealing way.
We could potentially integrate this for zstandardised values only? If
zstandard=True? Otherwise maybe an option to disable smart_bounds_if needed
in the function API?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pysal/splot/issues/51#issuecomment-478167834, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACJY815ZSSG17WffwzlSkLIufwODD1Orks5vbpLzgaJpZM4cTKQv
.

Maybe have an option that lets the user decide whether they want the same xlim and ylim or allow for the smart bounds approach?
if W is standardized, the lags range will be tighter than that of the original variable. This could be a case where relaxing the xlim=ylim conditions makes sense? Maybe a hook into the caller would let us know if w is standardized.

Smart. +1 on that.
Also, I think maybe if the plot is on standardized values, then equal aspect should always be true. If the plot is not standardized, then that should be optional.

@sjsrey And also maybe omitting outliers? I realized that when I was trying to replicate a plot I got from GeoDa. https://youtu.be/ZfOc2Hn-GFc?t=301 Otherwise, the plot will look weird sometimes:
import geopandas as gpd
from pysal.lib.weights import Queen
from pysal.explore.esda.moran import Moran_Local
from pysal.viz.splot.esda import plot_local_autocorrelation
# load dataset
path = gpd.datasets.get_path('naturalearth_lowres')
gdf = gpd.read_file(path)
# plot
queen_weights = Queen.from_dataframe(gdf)
moran_local = Moran_Local(gdf.pop_est, queen_weights)
plot_local_autocorrelation(moran_local, gdf, 'pop_est')

equal bounds is solved with #70


Security based on subscription

I'm working for a global organization, and in our to-be architecture we'd have messages of the same type arriving on the same topic. Market-specific applications would be able to subscribe to the topic and through a subscription-filter, they'd only be able to see events for their market.
This requires security based on a subscription. If a SAS token is given to the client application that is allowed to read from the topic, they can choose to use a different subscription (I believe?)
In this document, you often refer to this link to achieve this, but it seems broken: https://code.msdn.microsoft.com/Using-Shared-Access-e605b37c.
Is this feature on the roadmap? I feel it would be useful for multiple cases.

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

ID: 13478d1d-11eb-930a-02c4-559a1ef89fa6
Version Independent ID: be36cdf9-1e5e-c47f-e699-652dc42ee2b9
Content: Azure Service Bus access control with Shared Access Signatures
Content Source: articles/service-bus-messaging/service-bus-sas.md
Service: service-bus-messaging
GitHub Login: @axisc
Microsoft Alias: aschhab

Hi @adesmet Thanks for reaching out. We will review and update as appropriate.

@adesmet Just checking if you have gone through this and see if it helps.

@adesmet For now we will 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.


[Gardening] Update all Python scripting to conform to the style guidelines enforced by the black code-formatting tool.

The black tool is the Python Software Foundation's blessed formatting tool in the same spirit as gofmt. It's opinionated, but keeps all code consistent automatically. This change is completely non-functional and merely adds a proper flake8 config file and runs the black tool over the build-script.py and lit_tests/lit.cfg files.
Note: black is MIT licensed.

@swift-ci please test

@ahoppen I would love to add that in the future. We could enforce flake8 and black on all the Python sources. But for now this is more a manual transition.


⚠ Samsung has degraded performance

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

HTTP code: 200
Response time: 2597 ms

Resolved: Samsung performance has improved in 2d28357.


Rebase/integrate tests from #420

This constrains the title casing to Class titles, and adds
unit tests.

Coverage increased (+0.05%) to 90.934% when pulling 86415f9fc17b85a3025d1e30d54dda56c5a17cec on DavidS:update-tests into 7cbadb969ea65fd40f97347c8d55c91e21e0bee4 on rodjek:master.


Support "shouldFail" for Command

What would you like to be added:
Add "shouldFail" to the v1beta1.command type.
Why is this needed:
I'd like to confirm that executing a certain command fails.

A command can fail in many ways - with certain error codes or terminated by a number of signals. Perhaps we should think of an interface that supports this, like a expectedExitCode (default 0) that we could match against the exit code?
Although then asserting any non-zero exit code would be difficult.

I'll group this in the "theme" of commands asserts... which has a number of concerns beyond what @porridge mentions. (thanks Marcin!).  I will target this "theme" of activity for v0.7.0 which is an iteration away.
@gpaul more clarity for your use-case based on marcins comments would be great.
In fact, if you could provide a mock configuration and what your expectations around that would be really great!

Maybe something like
apiVersion: kudo.dev/v1alpha1
kind: TestStep
commands:
- command: ../../bin/dispatch login github --user=test --password=test
# Test that duplicate login fails.
- command: ../../bin/dispatch login github --user=test --password=test
  expectedExitCode: 1

This is the problem I was trying to test, by the way.


createdb command is undocumented and confusing

The docs about management commands don't include info about createdb. The only docs for the createdb command (in the installation instructions) leave me confused about why it exists, and not wanting to use it.
Since I want to be able to use other apps in my Mezzanine project, I don't really want to change my normal Django practices regarding 'syncdb' and 'migrate' - I know what they do already and when to use them, and so does my fabfile. I would need to know exactly what createdb will do if I do use it.
This leads me to wondering why mezzanine, a Django app, needs to include a command like this at all, especially one with a very generic name. If it is fixing a bug in Django or South, surely the fix should go in those projects?
If it is really intended that Mezzanine is supposed to take over your entire Django project, so that you are not supposed to use other apps, then please close this bug as invalid - but that's not the impression I got.
Thanks!

Firstly just to clarify - you can use syncdb / migrate as usual without ever using createdb, and Mezzanine also isn't meant to take over a project as far as preventing you from using other Django apps, in fact one of the core goals of Mezzanine is to work as closely as possible as a normal Django project, with whatever extra Django apps included as required.
So with that out of the way, createdb is just a shortcut for getting a new Mezzanine project up and running. All it does is run syncdb, and fake all of the initial migrations to get your database started. The problem it's meant to solve apart from being a shortcut for those, is a historical one, in that Mezzanine is comprised of multiple apps, each with their own set of migrations that depend on each other. These need to be run in a particular sequence to work correctly, and so when creating a new project in the past, these would fail. As far as I know there's no way for south to support cross-app sequencing of migrations. I recently learnt about being able to mark migrations as depending on other migrations, which may have been a solution back when this problem was a common issue for Mezzanine users, but I wasn't aware of it then.
As far as documentation goes, a note is included right after the only mention of the command: "The createdb command performs the same task as Django’s syncdb command, and also handles setting the initial migration state for South." - while fairly brief, it does describe what the command does. If anything it could use an extra note about the fact that you don't have to use if you don't want to. Please feel free to expand on it if you like and I'd be happy to merge it in.
Hope that clarifies things for you. I'll close this issue for now. If you'd like to discuss it further the mailing list is probably a better channel.

Thanks for the clarification. I do think one is needed, and more importantly, use of migration dependencies so that 'syncdb' and 'migrate' can be used normally. (I know that using '--fake' is preferred for this situation, but when you are deploying and want a single deployment path that works whether or not you happen to have added a new app to your project, it is nice to able to just do './manage.py syncdb && ./manage.py migrate').
A pull request will follow shortly!
On the same theme, if mezzanine is not supposed to necessarily take over your project, the installation instructions are quite lacking if you've got an existing project. A pull request for that is coming too!
Thanks.


Include list of recurring events

Monday internal ops call
Bimonthly sync
Quarterly openhouse

Added here: https://docs.dorg.tech/lifecycle/onboarding#recurring-events


Fix ThrottledStorageComponent not passing through health check

When using the ThrottledStorageComponent the health check (/actuator/health) always reports "UP" even when the backing storage component is in a bad state.
This change passes through the check from the delegate. I also formalized the name of components so that instead of reporting on the "ThrottledStorageComponent" in the health endpoint, you will get a more useful name such as "Throttled(ElasticsearchStorage)".

Ack. thanks for mentioning but I think this is the better optimization
On Mon, Jun 17, 2019, 10:52 PM lambcode<EMAIL_ADDRESS>wrote:

@lambcode commented on this pull request.
In zipkin/src/main/java/zipkin2/Component.java
https://github.com/apache/incubator-zipkin/pull/2632#discussion_r294338327
:

@@ -38,6 +38,11 @@ public CheckResult check() {
return CheckResult.OK;
}

public String name() {

I'm okay with this change. Just want to note that the name reported by the
health endpoint will change for some components. E.g "InMemoryStorage" will
become "InMemoryStorage{traceCount=123}"
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-zipkin/pull/2632?email_source=notifications&email_token=AAAPVVZGH6BBYGEBMVAWI2DP26QMFA5CNFSM4HYITGW2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB3XQYQY#discussion_r294338327,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAPVV2NR3UUXXDGREW2CUDP26QMFANCNFSM4HYITGWQ
.

thanks very much!


WIP 🚧 less docker-in-docker action

Description of changes
Configure Travis test run to use the PostgreSQL and Redis services already available in the Travis CI environment.
Issue Resolved

Fixes #

@robbkidd is this about ready?


Minor bug - Deleting a previewable file (md) temporarily disables visibility of first document selected

I don't think this is platform related

VSCode Version:
OS Version:

Steps to Reproduce:

Create a markdown file, open it up, with automatic preview enabled (is this configurable?)
Delete markdown file, previewer flags an error that the file is missing, close all currently open visible documents, and open a new one
New document doesn't appear at first, unless you open another one first.

Not a serious bug, but a bug nevertheless. Preview window should respond to deletion by closing not just an error notification.

Duplicate of #12033

It is not a duplicate. When this bug happens, the first source window you open other than the markdown does not appear, until you open a second one. I thought my bug description was quite clear.


44 add confirmation dialog to accept privicy and terms when you first visit the website

Component currently has Dialog set to true initially for testing purposes.
Close button should close the page but doesn't seem to do so right now
I assume when we develop a cookie policy it will also be featured here?

So just one comment for this. There shouldn't be a close button.

@MichaelC045 it seems like you can press escape to close the dialog. That shouldn't work.