airline could not resolve airline theme 'material'.

environment

vim: 8.1.1467

vim-airline: latest (v0.10)

OS: manjaro linux (xfce)

airline configuration:
set termguicolors
set background=dark
colorscheme material
let g:airline_theme='material'

terminal: xfce terminal

$TERM variable: xterm-256color

color configuration (:set t_Co?): 256

actual behavior
airline could not resolve airline theme 'material'
expected behavior
to work :)
I have installed
Plug 'vim-airline/vim-airline'
Plug 'vim-airline/vim-airline-themes'
It worked fine before this update!!
For solutions tired reinstalling both airline and airline themes as stated in Error after update #880

Before what update?

Am 02.07.2019 um 03:20 schrieb Goku-San<EMAIL_ADDRESS>environment
vim: 8.1.1467
vim-airline: latest (v0.10)
OS: manjaro linux (xfce)
What is your airline configuration:
set termguicolors
set background=dark
colorscheme material
let g:airline_theme='material'
terminal: xfce terminal
$TERM variable: xterm-256color
color configuration (:set t_Co?): 256
actual behavior
airline could not resolve airline theme 'material'
expected behavior
to work :)
I have installed
Plug 'vim-airline/vim-airline'
Plug 'vim-airline/vim-airline-themes'
It worked fine before this update!!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

There is no material theme in https://github.com/vim-airline/vim-airline-themes/tree/master/autoload/airline/themes
So it cannot work. Question is, where did you have this theme from?

Hello, @chrisbra
I updated yesterday with vim-plug, and several plugins were updated, I restarted vim and I got the error
This is the theme I am using kaicataldo/material.vim
I used the above setup for gVim when I was on window 7 five months ago, and for five now in terminal vim with linux, never had any issues :)
Didn't know that there was no material theme in airline-themes :) it just worked

This is the theme I am using kaicataldo/material.vim

That only has a lightline theme, not an airline one.


Request : Cheat : Weapon Switching as a Free Action

No hurry :)

if you are on turnbased can't the unlimited actions options work, could just turn it on, do your switch weapon then turn it off, you don't often switch weapons in combat anyway so it's not super inconvenient.

Unlimited Actions doesn't want to work anymore, I found, and additionally, it should be a free action as soon as you get a BAB of +1, so long as you do a move action!

Unlimited actions also dont work for me


weird highlight on javascript

VSCode Version:0.10.11
OS Version:Win7

Like below
res.vary("Accept-Encoding*");*

Fixed in the insider build. You can give it a try from our insiders release today: http://code.visualstudio.com/Download#insiders


Add option to not clean build directory prior to performing action

We are repeatedly building SourceKit-LSP in CI because we clean the build directory prior to to testing & installing, which shouldn’t really be necessary.
rdar://82448307

@swift-ci Please test

@swift-ci Please test

Are we rebuilding in the install phase still?

Yes, the SPM commands are still run as normal, this flag simply doesn't wipe the previous build before the test/install steps, which was forcing a from-scratch build for each step. SPM is pretty smart and simply applies the needed changes to update the past build, so that rpath flag works, though it's always possible that something else is missed on these non-clean builds that the tests simply haven't caught.
I had disabled this unnecessary cleaning on my Android CI that cross-compiles all new snapshot tags of this repo for Android, so I'm happy to remove that now. 😄


Disable Auto Output

I am developing a pack based loosely around the early days of just IC and BC. In hopes to try and fit EnderIO into this theme I was looking for a config option to disable the pushing and pulling that EnderIO machines do. For example, a playing could set the top of the block to input mode, but this would only allow a different block or pipe to insert into the slot, notfor the block to pull from a chest. For mods like IC2-exp this is possible by disabling the upgrade that allows this behavior, but as EnderIO has this built into the machines GUI, it isn't possible from what I can tell.

This is one of the core features of Ender IO...


kfctl instructions need KUBEFLOW_TAG

Instructions on the kfctl page are missing the requirement to set the environment variable KUBEFLOW_TAG=0.5.1. Add this before CLIENT_ID and use it in place of the text "<release tag>".
When this env var is not set, strange error messages occur on kfctl apply, which do not reference this environment variable, making it impossible to troubleshoot.

@texasmichelle Can you describe what you did and what errors you saw?
I'm pretty sure the kfctl go binary no longer uses that tag.
https://github.com/kubeflow/kubeflow/search?q=KUBEFLOW_TAG&unscoped_q=KUBEFLOW_TAG
I believe its only the kfctl.sh shell scripts that use that environment variable.
The instructions you linked to are using the go binary.

When I tried to reproduce, turns out KUBEFLOW_TAG is not the problem. Closing this issue to open one in kubeflow/kubeflow.


[PackageModel] Delete unused LibraryType

This patch removes LibraryType enum in Product, which is not used anymore.

@swift-ci please test and merge


Kong v0.12.3 support and probes value tweaks

/assign @unguiculus

/ok-to-test

@pbaezab thanks, keeping initial probe delay higher for livenessProbe than readinessProbe  makes sense but why timeout needs to be higher too?

How one probe timeoutSeconds affects others, I checked the doc https://kubernetes.io/docs/tasks/configure-pod-container/configure-liveness-readiness-probes/ and the example used in there not even set explicit timeoutSeconds, so basically same default value for both the probes. Mind providing me link to the best practices for liveness and readiness.
Regarding initialDelaySeconds, I am keeping it high because I want probe to wait for migrations job to finish.

ping @unguiculus, waiting for your approval, thanks.

@shashiranjan84 generally you should have the livenessProbes fire more frequently because if the pod is not responding then you want kubernetes to know asap so that it stops routing the traffic to that pod. If your deployment has more than one replica this is improves the resilience to failures.
The readiness probes are generally triggered less frequently, and especially when you have the same test for both probes it does not make sense to have the same frequency for both probes.

@sameersbn rebased, would you please review or should we wait for @unguiculus to review?

/ok-to-test

@sameersbn no worries, I did get confuse for a moment as redinessProbe usually controls the traffic, but somehow got convinced.
I bumped liveness probe periodSeconds to 30. Please review and approve.

/lgtm


[WIP] fix(startup): wait for disk crd to be loaded before scanning disks

As part of the initial startup, if the disk CRDs are not loaded,
the discovery process fails due to errors in updating. A restart
will be required to fix this after the disk CRDs are available in
etcd.
Signed-off-by: kmova<EMAIL_ADDRESS>

Codecov Report

Merging #79 into master will decrease coverage by 0.29%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master      #79     +/-   ##
=========================================
- Coverage   56.21%   55.91%   -0.3%
=========================================
  Files          41       41
  Lines        1521     1529      +8
=========================================
  Hits          855      855
- Misses        609      617      +8
  Partials       57       57

Impacted Files
Coverage Δ

cmd/controller/controller.go
19.17% <0%> (-2.37%)
:arrow_down:

Continue to review full report at Codecov.

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

Logs after making the above change:
kiran_mova@kmova-dev:~/go/src/github.com/kmova/openebs$ kubectl logs -n openebs openebs-ndm-dnzwm | more
E0905 10:05:51.263175       1 controller.go:152] Disk CRD is not available yet. Retrying after 10s, error: the server could not find the requ
ested resource (get disks.openebs.io)
I0905 10:06:01.268492       1 controller.go:156] Disk CRD is available
I0905 10:06:01.268521       1 filter.go:58] registering filters
I0905 10:06:01.268540       1 filter.go:70] configured os disk exclude filter : state enable
I0905 10:06:01.268900       1 filter.go:70] configured vendor filter : state enable
I0905 10:06:01.268919       1 filter.go:70] configured path filter : state enable
I0905 10:06:01.268939       1 probe.go:57] registering probes
I0905 10:06:01.268955       1 probe.go:83] configured smart probe : state enable
I0905 10:06:01.269045       1 probe.go:83] configured udev probe : state enable
I0905 10:06:01.269329       1 udevprobe.go:186] starting udev probe listener


Add tempo to audio

This change is

Paging @bmcfee:  I added tempo, and added a track level Timing, so we can get mean tempo for an entire track easily.  What do you think?

Paging @bmcfee: I added tempo, and added a track level Timing, so we can get mean tempo for an entire track easily. What do you think?

Cool, thanks for adding that.  I'm not sure mean tempo is the right aggregation; if it's half in 120 and half in 160, i don't think 140 is a reasonable summary.  Median or mode might be a better aggregator.. what do you think?

Yeah, I'll go for median - will merge later tonight.


Not responsive

When I change size of window. I got :

Looks  bad

Thanks @corners2wall opla.ai website will be completely rewrite during next iterations. We will take care about responsive design. ;-)

New website is deployed as a minimal landing page. More stuffs are coming.
so i close this issue, thanks again @corners2wall ;-)


Godot 4 Regressions

Keeping track of differences spotted between Godot 3.5 and Godot 4.  This is only the outstanding ones; Paul has fixed a bunch of them, and I just fixed quite a few of the remaining.

[x] When you press Build City, the highlighted city name has white font; it should have black font (likely a style issue)
[ ] The I-Beam in the same dialog is white (and thus hard to see), rather than black (likely a style issue?)
[x] The cursor is also white rather than black and hard to see
[ ] The "End Turn" button should not be highlightable nor clickable while the "Build City" dialog is open, but is (this seems to apply to all the buttons as well, though only End Turn has a clear effect)
[ ] The spacing between right-click-menu items is slightly larger
[x] The font color on hovered right-click-menu items is orange, not black (likely font_color_hover -> font_hover_color in style overrides)
[x] The advisor art above the Build City Popup has some odd transparency issues going on, maybe needs shadows turned off?
[ ] The lower-right info box font is slightly too large (or misaligned?) (most visible when you fortify all and "ENTER or SPACEBAR for next turn" starts too far to the left)

thanks for helping to close a bunch of these out, the source of the regressions can be tricky to track down!


cleanup: rename EncryptionTypeString() to EncryptionType.String()

This makes it easier to log the EncryptionType as string, or int,
whatever is preferred. Standard fmt formatting notations like %s or %d
can be used now.

@mergifyio queue

@mergifyio rebase

@mergifyio queue

@mergifyio queue

@mergifyio requeue

@mergifyio queue

@mergifyio refresh

@mergifyio queue


Include Frontend in Dockerimage

Is your feature request related to a problem? Please describe.
Runtime and config for frontend isn't included in Docker Image
Describe the solution you'd like
When I start the Docker Image I want the frontend to start as well as the backend.
I cannot use anything without having a frontend.
Describe alternatives you've considered
As described in README, how to start frontend does not work.
[root@dcf04c93d633 webui]# npm install
bash: npm: command not found

Additional context
The frontend is compiled in the Dockerfile HERE.
But there is no nodejs and no nginx installed in the "Final image" - as it is described in your documentation HERE.
So of course if I try to connect to the running Service on port 9999 (which is the backend) I only get: mix: no Session-ID header.

Docker support is work in progress, I just added a new PR https://github.com/Studio-Link/mix/pull/46 with some improvements.


Update nf-winuser-createwindowexw.md

Specify message order list to documentation. If this "order" of messages being sent is not by design or intentional, feel free to delete this pr.

@AzAgarampur : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Update dashboards.md

🚀 Deployed on https://66f448622e66632e699d8549--cozy-fox-0defec.netlify.app


fix: issue #367

I took a look and figured out that this fix only works correctly if you clearing all of the toasts, which toast.remove will do since it's overloaded (undefined clears all). In fact I would rather have removeById but that's another discussion.
I figured out that what is not happening is when you click 'remove' PAUSE_END action is not being fired (onMouseLeave not being triggered).
Either fire that action on remove and dismiss, or extract the logic of that PAUSE_END into separate function and reuse.
You need this because current fix is not counting on pauseDuration that is being adjusted. This is revealed if you remove by id and then they all disappear all at once after largest pause, which is not desired behaviour.
Finally, this is really tricky bug because 'dismiss' is also affected, but because dismiss hides the toast, you accidentally trigger END_PAUSE because wrapper is there and onMouseLeave triggers.
All that to say here is the fix.
   case ActionType.REMOVE_TOAST: {
      let newState = state;
      if (state.pausedAt !== undefined) {
        newState = reducer(state, {
          type: ActionType.END_PAUSE,
          time: Date.now()
        });
      }

      if (action.toastId === undefined) {
        return { ...newState, toasts: [] };
      }
      return {
        ...newState,
        toasts: newState.toasts.filter((t) => t.id !== action.toastId),
      };
    }


Update README.md

Landing Page Project
Table of Contents

Instructions

Instructions
Landing Page Project HTML was used in addition to CSS and JavaScript for the work of the project, as it consists of:
-4 sections that have been added to the page.
-Navigation is built dynamically as an unordered list.

clear which section is being viewed while scrolling through the page
This was done by adding 2 function in java script
function to create list for navbar .
function to add active section.

Hi,
We are closing out the issue because there are no enhancements proposed here.
Thank you


Add C API for power supply, node types and samples

Power Supply API provides a way to get information about battery,
UPS, AC or DC power supply.
This initial version is a synchronous API heavily biased by
Linux implementation.
It doesn't provide events.
Way more information could be provided, like energy,
charge, health and other stuff that eventually few drivers
will provide and hardly will be consumed.
So I'm trying to define a set of relevant properties to expose,
and we should try to provide the same for other platforms.
@ibriano helped me with a quick research about it on RIOT
and it seems there isn't anything like that, no defined
interface. Eventually some
drivers provide some information, like voltage.
Later I intend to investigate how it's done on yet another
platform / project to make sure we get an API that at
least work for Linux and this other one.
Signed-off-by: Bruno Dilly<EMAIL_ADDRESS>

The API overall and the commits look good.

Looks ok to me, fixing the comments.

merged with fixes


fix typos and LSST DESC broken link

Nice work. I'm going to close  this PR so other's can complete the assignment.


5.1.0: JWT with "cty":"JWT" in header cannot be read

In System.IdentityModel.Tokens.Jwt 5.1.0, JWTs with a valid header like:
{
  "alg": "HS256",
  "typ": "JWT",
  "cty": "JWT"
}

cannot be read, due to this code in JwtSecurityToken.DecodeJws():
        private void DecodeJws(string[] tokenParts)
        {
            // We do not support other content types for JWS.
            if (Header.Cty != null)
                throw LogHelper.LogExceptionMessage(new ArgumentException(string.Format(CultureInfo.InvariantCulture, LogMessages.IDX10723, tokenParts[1], RawData)));

Note that this is not an issue in the master branch, but is a (blocking for me) issue in the release branch, and in the existing released NuGet package.
This test fails (release branch only):
        [Fact]
        public void PayloadParsesWithCtyInHeader()
        {
            // This token has JWT header: { "alg": "HS256", "typ": "JWT", "cty": "JWT" }
            string token = "eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCIsImN0eSI6IkpXVCJ9.eyJzdWIiOiIxMjM0NTY3ODkwIiwibmFtZSI6IkpvaG4gRG9lIiwiYWRtaW4iOnRydWV9.E-Dl3jALtUnmEZzbFuyFV15wkqTsQU_zRci5u6yBrHc";

            var jwtHandler = new JwtSecurityTokenHandler();
            var jwt = jwtHandler.ReadJwtToken(token);
        }

Note that per https://tools.ietf.org/html/rfc7519#section-5.2 (covering the cty header), the current behavior is reasonable (and an implementation I'm integrating with is incorrect). However, this is a breaking change from the prior version of System.IdentityModel.Tokens.Jwt (version <IP_ADDRESS>8261200).
I think the biggest reason that this is a bug is that it prevents successful interoperability with other implementations (this is a blocking issue for me for integrating with https://www.aoe.com/en/products/forgerock/forgerock-openam.html). The question is whether this exception is truly necessary.

Note that per https://tools.ietf.org/html/rfc7519#section-5.2 (covering the cty header), the current behavior is reasonable (and an implementation I'm integrating with is incorrect). However, this is a breaking change from the prior version of System.IdentityModel.Tokens.Jwt (version <IP_ADDRESS>8261200).
I think the biggest reason that this is a bug is that it prevents successful interoperability with other implementations (this is a blocking issue for me for integrating with https://www.aoe.com/en/products/forgerock/forgerock-openam.html). The question is whether this exception is truly necessary.

@johncrim lax on receive and per spec on send is probably the way to adjust here. I marked this for the 5.1.2 release

@johncrim we will get this into 5.1.3

@johncrim committed into dev, will be in nightly builds.
5.1.3 will be released by Feb 28th.


update dep

Coverage increased (+0.09%) to 63.931% when pulling 1a858f710ae08cfa04b98f3b87cff966d566d652 on tianxiaoliang:master into 1e9b28ee595e513f39b5d81c6037216c2170a641 on ServiceComb:master.


Updated FEGA submission page

Complete update of FEGA submission page based on the new portals and credentials. Also, tried to clarify the legal agreements section.

I am not able to fix the videos.


Unable to connect to local server

Describe the bug
Unable to connect to local Homestead server - get this error
Error: Error invoking remote
method 'connectSsh': Error:
Timed out while waiting for
handshake
Versions
Ray version (you can see this in "About Ray"):1.15.0
You can use composer show to get the version numbers of:

spatie/ray package version: 1.27.1
spatie/laravel-ray package version (if applicable): 1.23.0

PHP version: 8.0.5
Laravel version (if applicable): v8.48.2
To Reproduce
Set connection to Homestead:

Expected behavior
Connect to server
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: [e.g. macOS] MacOSX
Version [e.g. 22] 11.4

Additional context
Add any other context about the problem here.

Can you ssh to your server using your terminal?


[FB Messenger] Checkbox Plugin support?

System Information

SDK Language: .NET
SDK Version: 3.9.0

Issue Description
Hello!
I'm adding Facebook Checkbox Plugin, and I have a trouble with getting opt-in callback - it just does not come to our server (I set a breakpoint in my Application_BeginRequest method and nothing happened).
As far as I understand it's not supported now? If so, then how can I workaround this issue?
I think I can set my own webhook, where I can switch between callbacks, ie:

a webhook request comes to my server
I see if it is an Plugin opt-in callback
If it is, then I process my own logic
Otherwise I send the request to Bot Framework url

But I wonder is there a more convenient way?

In case somebody have the same issue. Added my own webhook.
    [HttpPost]
    [HttpGet]
    [Route("api/FacebookWebHook/{*remaining}")]
    public async Task<HttpResponseMessage> CallbackAsync()
    {
        if (Request.Method == HttpMethod.Post)
        {
            await Request.Content.LoadIntoBufferAsync().ConfigureAwait(false);
            string json = await Request.Content.ReadAsStringAsync().ConfigureAwait(false);
            if (!string.IsNullOrEmpty(json))
            {
                JObject o = JObject.Parse(json);
                JToken token = o.SelectToken("entry[0].messaging[0].optin");
                if (token != null)
                {
                    var userRef = token.SelectToken("user_ref")?.Value<string>();
                    var userEmail = token.SelectToken("ref")?.Value<string>();

                    if (userRef != null && userEmail != null)
                    {
                        dynamic result = await _facebookClient
                            .PostTaskAsync("me/messages", new {
                                access_token = _appSettings.FacebookPageAccessToken,
                                recipient = new {
                                    user_ref = userRef,
                                },
                                sender_action = "mark_seen"
                                // message = new {
                                //     text = "Hi there!"
                                // }
                            })
                            .ConfigureAwait(false);
                    }
                }
            }
        }

        const string host = "facebook.botframework.com";
        string forwardUri = $"https://{host}/api/v1/bots/billycom{Request.RequestUri.Query}";

        Request.Headers.Remove("Host");
        Request.RequestUri = new Uri(forwardUri);
        if (Request.Method == HttpMethod.Get)
        {
            Request.Content = null;
        }

        var client = new HttpClient();
        HttpResponseMessage httpResponseMessage = await client
            .SendAsync(Request, HttpCompletionOption.ResponseHeadersRead)
            .ConfigureAwait(false);
        return httpResponseMessage;
    }

And more question - now I have a table for all my conversations. It looks like this:

However, when I use Checkbox Plugin there is no conversation with the user.
So, the question is - can I use BotId/ServiceUrl/ChannelId from the other conversation? i.e. is the BotId stable within the channel?

are you seeing behavior that suggests the BotId does not remain the same?


add analytics for auction results for artists you flow

The type of this PR is: Feature
This PR resolves CX-1617
Description
Add analytics for AuctionResultsForYou and AuctionResultsRail components
PR Checklist (tick all before merging)

[ ] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[ ] I have tested my changes on iOS and Android.
[ ] I have added tests for my changes, or my changes don't require testing, or I have included a link to a separate Jira ticket covering the tests.
[ ] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[ ] I have documented any follow-up work that this PR will require, or it does not require any.
[ ] I have added an app state migration, or my changes do not require one. (What are migrations?)
[ ] I have added a CHANGELOG.yml entry or my changes do not require one.

Dev changes

Add analytics for auction results for artists you flow

#squashongreen


Cannot disable telemetry collection

I am trying to disable telemetry collection on an Ubuntu host with a MSSQL container following the documentation here: https://docs.microsoft.com/en-us/sql/linux/sql-server-linux-customer-feedback?view=sql-server-linux-2017#on-docker
As a test:
echo '[telemetry]' >> ~/mssql/mssql.conf
echo 'customerfeedback = false' >> ~/mssql/mssql.conf

docker run -e 'ACCEPT_EULA=Y' -e 'MSSQL_SA_PASSWORD=<YourStrong!Passw0rd>' -p 1433:1433 -v ~/mssql:/var/opt/mssql -d microsoft/mssql-server-linux:2017-latest

I can bash into the container and see the /var/opt/mssql/mssql.conf file with customerfeedback = false in it.
I am monitoring the server with tcpdump -i eth0 -n port 443 and after approximately 5 minutes of the mssql container being online I see a flood of outbound packets going to IP <IP_ADDRESS>. Upon further inspection, this IP belongs to Microsoft. https://dig.whois.com.au/ip/<IP_ADDRESS>
This happens consistently every time I restart the mssql container and wait ~5 minutes. So either the customerfeedback = false config setting is not working or there is some other outbound data being sent to Microsoft.
How can I disable this completely?

I'm facing the same issue, with traffic going to <IP_ADDRESS> / <IP_ADDRESS> / <IP_ADDRESS>.
These IPs are from settings.data.microsoft.com and vortex.data.microsoft.com.
Would be nice to be able to disable all external traffic.
Thx 👍

We have "black holed" settings.data.microsoft.com and vortex.data.microsoft.com in /etc/hosts until we get more clarification on what is occurring here.
ref: https://github.com/bitwarden/core/commit/ffe5f37a64d72e23bb3398a4b9daa9a820e34841

We are checking into this... More info soon.

@twright-msft Was there any info update?

Why has this issue been closed?
No resolution, no explanation, nothing.

5 years later...
still collection telemetry data without possible opt-out.


text/markdown support

It would be pretty cool to have native markdown support.
I took a look in the source-code and from what I've been able to gather (as a C noob, basically), you're only handling gemini-formatted content (src/gmdocument.c), and you're manually rendering it (probably for best performance, I imagine).
That means that markdown support would require a custom frontend over a markdown compiler, I guess, or "another src/gmdocument" parsing markdown.
Considering the living hell that markdown is, I can probably imagine making a frontend over an existing markdown compiler, but I can probably guess that there are lots of thorny issues on the rendering side, e.g. tables, images, and such.
This would be no small feat, and while I originally created this issue with intent to propose help, now that I'm writing this sentence I kinda already feel overwhelmed by the required work.
My question is then: Was I correct in my assumption on how lagrange's gemini document rendering works?
If not, would you agree to take a bit of time explaining how the content handling / rendering is done?
And no matter what, would markdown support be an objective for a future major upgrade in your opinion?

Was I correct in my assumption on how lagrange's gemini document rendering works?

What you're saying is pretty much correct. GmDocument has been specifically written for text/gemini and text/plain, and introducing Markdown needs some non-trivial work.
Since there is no and will be no HTML renderer in Lagrange, the "full" Markdown cannot be supported in any case, but some sort of pragmatic "Gemini-flavored Markdown" would be doable. Certain elements like tables would probably have to be rendered as preformatted blocks. GmDocument would need to be modified to allow having a separate "visual text string" for each GmRun in addition to the source text, so it could apply inline formatting like bold/italic/etc. without breaking the text selection etc. code elsewhere. The text renderer would need some additional escape sequences to change the font inside a string. Hyperlinks would need some special handling, perhaps converting them all to the footnote style that Markdown supports. On the whole I would assume that GmDocument could be used without major modifications, though, since this Markdown preprocessing would be happening in separate class.

would markdown support be an objective for a future major upgrade in your opinion?

I've had Markdown support in my longer-term todo list since the beginning, but it hasn't been a priority because of how little it is actually being used (GUS shows only 44 hits, if the MIME types can be trusted).
Personally I would enjoy using Lagrange as a Markdown viewer, so I have a feeling I'll get around to working on this eventually.

It seems to me that you underestimate the importance of supporting markdown. Let me give you an example. I'm currently working on a project for lua scripting language. It is a documentation tool which parse source files and build standalone wiki in markdown format. This wiki can be browsed locally using web-browser extension or uploaded to github which will automatically convert it to html files. So, if I want to add gemini support to my tool, I need to write additional module to support its format. Maybe it's not really hard work but I just don't like this idea! Why do we need many different formats for the same thing? There is a lot markdown-based websites like blogs and if their owners want to move from http to gemini, they have to find (or make themself) some converter, but why?

i apply to this list too
i wish markdown support, in gemini protocol documentation they say gemini servers can serve markdown and some servers like agate can serve markdown and lagrange is cool but does not support markdown ... this is a problem... i also know some other people who ask for markdown and they then turn back on https and create minimalistic sites here like gemini but because its hard to find markdown support on gemini they go back to https, i my self also use markdown generated in obsidian.md and to reformat all into gmi is a problem like bold or italic or links
please support markdown

Naturally there is also opposition to this idea: gemini://skyjake.fi/gemlog/2021-10_re-mark-it-down.gmi
I'm a fan of Markdown myself. But I think the key aspect here is to accept that there isn't one canonical visual rendering of a Markdown document. Therefore, it's important to be able to see both the "real" document (i.e., the raw source version) and the rendered interpretation of it. This can be accomplished with #359.
To alleviate concerns regarding undermining of Gemtext's position as the primary content format, Markdown documents should default to the raw source view.

@lua-rocks:

Why do we need many different formats for the same thing? There is a lot markdown-based websites like blogs and if their owners want to move from http to gemini, they have to find (or make themself) some converter, but why?

One argument could be that someone will need to do the conversion or otherwise you're just looking at a plain text page in the Gemini client — not all Gemini clients will be able to display Markdown on their own. Doing this serverside saves everybody some trouble, and the content owner can oversee the process to ensure that the end result is good.
This is an interesting tension point: there's plenty of Markdown content out there, e.g., blogs and documentation as you say, that could be great to have available via Gemini. However, Gemtext (and plain text) being the only guaranteed content format in clients casts doubt whether that is feasible in practice. We should try to build toward a level playing field instead of favoring certain clients.

@skyjake I am not at liberty to argue with you. Ultimately, this is your project and you decide in what direction to go. Just from my point of view, the markdown browser (via any protocol, including locally) was just as or even more useful to people than the whole gemini-protocol with all its formats and features. But this is just my opinion, it is not meant to be the only truth :)

v1.8 has a simple Markdown-to-Gemtext converter for viewing local files (via file:// URLs). It is quite incomplete at the moment, so pretty experimental still.

It would be a very good idea to render markdown text, and even in gopher protocol when a file has a .md or .markdown extension.
I am radioamateur and gopher is a cool protocol to publish content over radio in Packet Radio with a small overhead, communication are often at 9600 bauds ! Gemini would be nice, but it is encrypted with TLS, and it is forbidden to crypt radio communication on amateur bands.
Many publishers write contents in markdown as input of their site/blog, it is easy to generate a gopher site in parallel.
Rendering md would be awesome and I'm sure Lagrange would become the best tool for Packet Radio network

Version 1.13 has an opt-in feature that enables automatically converting Markdown into a Gemtext document. This naturally leaves out most of the features of Markdown, except for horizontal rules that are now supported in Markdown documents.


Adding in autocomplete search with Semaphore

Overview
This PR adds into autocomplete searching on the homepage using SmartLogic's Semaphore to populate suggestions.
Checklist

[ ] PR has a descriptive enough title to be useful in changelogs

Testing Instructions

If needed, regenerate the Solr Schema
Update your settings_deployment with the SmartLogic API keys
In one terminal window docker-compose up solr-staging, in another run the server
Begin typing a topic, e.g. Metro Green Line
Confirm autocomplete appears and is functional
Choose a topic, confirm it goes to the appropriate search page when selected

Handles #433

btw, logging for testing purposes: here are current councilmatic subjects that are also in smartlogic, i.e., can be used to test the autocomplete flow:
lametro_councilmatic=# select distinct subject from lametro_subjectguid join councilmatic_core_subject on lametro_subjectguid.name = councilmatic_core_subject.subject;
                subject
---------------------------------------
 Title VI Requirements
 All Door Boarding
 Metro Green Line
 2ND QUARTER PERFORMANCE
 ENA (EXCLUSIVE NEGOTIATION AGREEMENT)
 RIDERSHIP INITIATIVES
 Measure R
 Measure M
(8 rows)


DataFrame.query gives false positives

Noticing some strange behaviour when using the query method. I'm expecting empty results from the query but once in a while I'm getting a non-empty dataframe returned.
import pandas as pd

testdf = pd.DataFrame(pd.np.random.randint(100, size=(10000, 2)))
testdf.columns = ['a', 'b']

missing_value = -2000
print missing_value in testdf['a'].values
>>>False

Doing this 10000 times with using the classical way:
found_count = 0
for i in xrange(10000):
    qq = testdf[testdf['a']==missing_value]
    if not qq.empty:
        print "Value found"
        found_count += 1
print "Times found: {}".format(found_count)
>>>0

Using query:
found_count = 0
for i in xrange(1000):
    qq = testdf.query('a==-2000')
    if not qq.empty:
        print "Value found"
        found_count += 1
print "Times found: {}".format(found_count)
>>>Value found
>>>Value found
>>>Times found: 2

or
found_count = 0
for i in xrange(1000):
    qq = testdf.query('a==@missing_value')
    if not qq.empty:
        print "Value found"
        found_count += 1
print "Times found: {}".format(found_count)
>>>Value found
>>>Times found: 1

The number of non-empty results varies but tends to happen 0.2% of the time. Is this a bug or have I misunderstood what query is supposed to to? Version is 0.17.0.

pls include a np.random.seed as impossible to replicated.
This is prob a floating point issue. numexpr actually compares floats I think (which is what is actually executing this).

Also happens with a dataframe containing strings:
testdf=pd.DataFrame([("one","two")]*10000)
testdf.columns=['a','b']

found_count = 0
for i in xrange(10000):
    qq = testdf.query('a=="three"')
    if not qq.empty:
        print "Value found"
        found_count += 1
print "Times found: {}".format(found_count)
>>>Value found
>>>Value found
>>>Value found
>>>...
>>>Times found: 13

pls show pd.show_versions()
this is not reproducible on master

INSTALLED VERSIONS
------------------
commit: None
python: 2.7.9.final.0
python-bits: 64
OS: Darwin
OS-release: 15.2.0
machine: x86_64
processor: i386
byteorder: little
LC_ALL: en_US.UTF-8
LANG: en_US.UTF-8

pandas: 0.17.0
nose: 1.3.7
pip: 7.1.2
setuptools: 18.4
Cython: 0.23.4
numpy: 1.10.1
scipy: 0.16.0
statsmodels: 0.6.1
IPython: 4.0.0
sphinx: 1.3.1
patsy: 0.4.0
dateutil: 2.4.2
pytz: 2015.6
blosc: None
bottleneck: None
tables: None
numexpr: 2.4.4
matplotlib: 1.4.3
openpyxl: None
xlrd: 0.9.4
xlwt: None
xlsxwriter: None
lxml: None
bs4: 4.3.2
html5lib: None
httplib2: None
apiclient: None
sqlalchemy: 1.0.9
pymysql: None
psycopg2: 2.6.1 (dt dec pq3 ext lo64)

so it might be numexpr 2.4.4 (though that is the latest in conda), you can try pip installing 2.4.6 which did fix some obscure indexing bugs.
I have quite similar versions of these packages & have not been able to repro on 0.17,0.17.1, or master
does it always happen?

Tried it on a different machine and also on a fresh IPython notebook and it random query results do not happen there. Only place I can reproduce it is in one notebook that has a several dataframes with >1e6 items each. In that particular notebook it always happens and I get random results from what should be a deterministic expression. I'll upgrade to the newer numexpr and see if it helps.

then its almost certainly numexpr. as the indexing issues only happen with > 500k element (note this was important information to have at the beginning).

I just reinstalled latest version of Anaconda which has numexpr 2.4.4 and then this started happening. Like OP, It happens very rarely and very hard to reproduce.
numexpr 2.4.6 has been ok so far.

Versions for the record.
INSTALLED VERSIONS
commit: None
python: 3.5.1.final.0
python-bits: 64
OS: Windows
OS-release: 7
machine: AMD64
processor: Intel64 Family 6 Model 42 Stepping 7, GenuineIntel
byteorder: little
LC_ALL: None
LANG: fi_FI
pandas: 0.17.1
nose: 1.3.7
pip: 7.1.2
setuptools: 18.5
Cython: 0.23.4
numpy: 1.10.1
scipy: 0.16.0
statsmodels: None
IPython: 4.0.1
sphinx: 1.3.1
patsy: 0.4.0
dateutil: 2.4.2
pytz: 2015.7
blosc: None
bottleneck: 1.0.0
tables: 3.2.2
numexpr: 2.4.4
matplotlib: 1.5.0
openpyxl: 2.2.6
xlrd: 0.9.4
xlwt: 1.0.0
xlsxwriter: 0.7.7
lxml: 3.4.4
bs4: 4.4.1
html5lib: None
httplib2: None
apiclient: None
sqlalchemy: 1.0.9
pymysql: None
psycopg2: None
Jinja2: 2.8

closing as this is a numexpr issue. maybe we should put in a doc-warning that we recommend > 2.4.6, though not available on conda...hmm I'll push on that.
so if anyone wants to put up a PR for install.rst great8.


Default values in privacy configuration are not working.

Description
According to the documentation the default value of privacy is “any”. However, when those values are left empty the server fails to start. The server should not fail to start; it just has to interpret that when there is empty in whichever privacy configuration’s properties means that the value is “any”.
This situation happens with every privacy configuration property located in the privacy_default’s folder.

The default values are set in the folder "META-INF / privacy_default", it is expected that if any required file is not in the folder, the application fails, since they are files required by the application. The user can overwrite these default values by creating the files in the folder "WEB-INF / privacy".
It is expected that the "src / META-INF" folder will only be modified by developers and not by users.


Inaccurate checkpoint: div classes are mixed up

Challenge http://www.freecodecamp.com/challenges/waypoint-create-bootstrap-wells has an issue. Please describe how to reproduce it, and include links to screenshots if possible.
Please note that the classes described in the first checkpoint are mixed up. The 's with class "well" should go within the 's with class "col-xs-6", instead of the other way around. Thank you!
Repro steps:
Go to http://www.freecodecamp.com/challenges/waypoint-create-bootstrap-wells
and observe the first checkpoint on the left.

duplicate of https://github.com/FreeCodeCamp/FreeCodeCamp/issues/1968

Closing as duplicate.


Updated pre-commit hooks versions and Reformatted files.

Used flynt automated tool to update from the old literal formatting method to f-strings, and also updated pre-commit hooks versions to latest versions. Tested all the files by running pre-commit run --all-files.

[ ] Closes #xxxx
[ ] Tests added / passed
[x] Passes pre-commit run --all-files

Can one of the admins verify this patch?
Admins can comment ok to test to allow this one PR to run or add to allowlist to allow all future PRs from the same author to run.


Db shrink optimization

tested

Are this db changes compatible with the old database? Have you tested update from the current dev chain to this new one?

sorry, forgot about it. will test

Are this db changes compatible with the old database? Have you tested update from the current dev chain to this new one?

sorry, forgot about it. will test

tested


MISSING: Conv3D for SYCL

Op file: conv_ops_3d.cc

Fixed by #133, except that some tests fail, which is likely due to Eigen::CuboidConvolution


Provide AUX and FLAPS assignment UI

This should be moved from RC cal to a separate screen.

Implemented.


fix typo

availabe -> available

/lgtm
/approve

@max8899 Thank you for your contribution.  Files under /blog are reviewed by a separate group of reviewers.  If you feel strongly about the changes you can contact the blog owners team located in https://github.com/kubernetes/website/blob/master/content/en/blog/OWNERS.   Since this PR cannot move forward through our standard review process I am closing this PR.


Move Bevy's fork of the ecs_bench_suite to the appropriate folder

I went looking for the latest bevy-specific ecs_bench_suite code and could not quickly find it.
Ideally, this should be found (or linked) in our benches/bevy_ecs folder.
The most up-to-date version is currently found on @cart's GitHub page here: https://github.com/cart/ecs_bench_suite/tree/bevy-benches. The latest upstream code is here: https://github.com/rust-gamedev/ecs_bench_suite
Once this is done, please also update CONTRIBUTING.md (see #2040) with directions to the benchmarks and add appropriate documentation on how to run the benchmarks in the README of the benches folder.
I'm also not entirely sure why we have two layers of benches folders; perhaps someone else can chime in.

I think it makes sense to have in-tree benches that are easy to run. We can probably just copy the benches from my ecs_bench_suite fork. However we should also drive the new benchmarks upstream. I'll probably get to it eventually, but if anyone else wants to pick that up, I'm cool with it.


ARROW-9131: [C++] Faster ascii_lower and ascii_upper.

Following up on #7418 I tried and benchmarked a different way for

ascii_lower
ascii_upper

Before (lower is similar):
--------------------------------------------------
Benchmark           Time           CPU Iterations
--------------------------------------------------
AsciiUpper_median    4922843 ns      4918961 ns           10 bytes_per_second=3.1457G/s items_per_second=213.17M/s

After:
--------------------------------------------------
Benchmark           Time           CPU Iterations
--------------------------------------------------
AsciiUpper_median    1391272 ns      1390014 ns           10 bytes_per_second=11.132G/s items_per_second=754.363M/s

This is a 3.7x speedup (on a AMD machine).
Using http://quick-bench.com/JaDErmVCY23Z1tu6YZns_KBt0qU I found 4.6x speedup for clang 9, 6.4x for GCC 9.2.
Also, the test is expanded a bit to include a non-ascii codepoint, to make explicit it is fine to upper
or lower case a utf8 string. The non-overlap encoding of utf8 make this ok (see section 2.5 of Unicode
Standard Core Specification v13.0).

I also thought that we could do a bit check instead of the range check, e.g. code_unit & 0b11100000) == 0b01100000, but that would also transform the backtick for instance (binary value 0b1100000).
The generated code looks vectorized indeed. I didn't look into the details of the generated code by clang and GCC, it seems their performance is a bit different, so we might be able to squeeze out a bit more if we want. Happy to look into that later (create a new issue), but I rather spend my time on other functions now.

It's ok, there's no need to further optimize those functions.

Travis-CI failure is unrelated.

Thank you @maartenbreddels !

Thanks, let me know if my workflow is ok, or if I can make things go smoother.
PS: I am looking for a document describing the kernel design. I see these two cases (if (batch[0].kind() == Datum::ARRAY) { and the else clause, but I am not sure I fully understand this. But I'm not sure where this is described, if it is.
PS2: Are there alternative channels for quick/small questions, or is this fine?

I think the current approach (implement kernels one-by-one) is reasonable and manageable for us (and for you as well I hope).
I don't think there's much documentation for now around the kernel design. Basically, kernels should usually be able to process two kinds of inputs (represented as Datums): arrays and scalars. That said, arrays are the dominant case, so if we leave scalars unimplemented in a given kernel, that's not an urgent problem.
For quick development questions, we have a public chat instance at https://ursalabs.zulipchat.com/ - just register though and you can chat with the team. The main channel there is the "dev" channel, and Zulip allows you to create subtopics - don't hesitate to use those!

I think the current approach (implement kernels one-by-one) is reasonable and manageable for us (and for you as well I hope).

No, this is fine.

That said, arrays are the dominant case, so if we leave scalars unimplemented in a given kernel, that's not an urgent problem.

👍

PS2: Are there alternative channels for quick/small questions, or is this fine?

Can we use<EMAIL_ADDRESS>so everyone can see the discussion and it's searchable via Google later?

Can we use<EMAIL_ADDRESS>so everyone can see the discussion and it's searchable via Google later?

I'm ok with that if not considered too noisy (like small cmake questions etc).

It shouldn't be a problem.


Java Style Guide: One-character local variables

https://google.github.io/styleguide/javaguide.html#s5.2.7-local-variable-names
does not say anything about one-character names.
Here there is comment that one-character names should be ok - https://github.com/google/guava/issues/1891#issuecomment-63231281
Please confirm that one-character names are ok.

I can confirm that one-character local variable names are permitted. As part of https://github.com/google/styleguide/pull/151 I loosened the guidance around single-character variable names. Note they are still discouraged (but permitted) in public method parameters.
While not detailed in the style guide (since it's subjective) we encourage people to limit their use of single-letter variable names to industry-wide standard usages (e.g. i, j, k for counters, e for exceptions, etc.).

@dimo414 , thanks for reply.

e for exceptions, etc

According to http://checkstyle.sourceforge.net/reports/google-style/guava/ , exception parameters are not allowed to be single characters. We could update regexp to be ^e|t|([a-z][a-zA-Z0-9]*)$ it will allow well known short names as e and t only.
Or you want to remove that constraint completely and allow any one character name ?

Note they are still discouraged (but permitted) in public method parameters.

we caught this update and addressed it. But we report violation on one-character arguments of public methods.
As representative of google style do you want completely allow one-character names ?
"discourage" never works well with humans, it is either violation or not.
We want that checksyle config follows guide as much as possible.
For examples of guava - http://checkstyle.sourceforge.net/reports/google-style/guava/ , search for "LocalVariableName"

(e.g. i, j, k for counters

we already have option to skip loops counters - example

Personally, I would encourage dropping the one-character requirement entirely as it's simply too subjective whether a given example is acceptable or not, and there are many common cases where it's perfectly reasonable.
I can't speak authoritatively about Google Style, but per @kevinb9n's comment in https://github.com/google/guava/issues/1891 ("I think that checkstyle should stop warning on single-character names.") I believe we're on the same page. Kevin certainly signed off on the style guide change I submitted that loosened the one-character language.
I appreciate your point that ""discourage" never works well with humans, it is either violation or not.", but I think it's too heavy-handed to ban them outright. I suggest simply allowing them.

Checkstyle config will be updated - checkstyle/checkstyle#3702


rstnet.pth

The link of rstnet.pth is invalid.

The link of rstnet.pth is invalid.

I've uploaded this file again.


Update README.md - Resolves #6

Contributing
If you want to contribute to Algorithmics and make it better, your help is very welcome. Contributing is also a great way to learn more about social coding on Github, new technologies and and their ecosystems and how to make constructive, helpful bug reports, feature requests and the noblest of all contributions: a good, clean pull request.
How to make a clean pull request
Look for a contribution instructions.

Create a personal fork of the Algorithmics on Github.
Clone the fork on your local machine. Your remote repo on Github is called origin.
Add the original repository as a remote called upstream.
If you created your fork a while ago be sure to pull upstream changes into your local repository.
Create a new branch to work on! Branch from develop if it exists, else from master.
Implement/fix your feature, comment your code.
Follow the code style of the project, including indentation.
If the project has tests run them!
Write or adapt tests as needed.
Add or change the documentation as needed.
Squash your commits into a single commit with git's interactive rebase. Create a new branch if necessary.
Push your branch to your fork on Github, the remote origin.
From your fork open a pull request in the correct branch.
And that's it, we will take care of the rest.

 Your commit message should describe what the commit, when applied, does to the code – not what you did to the code.

Hey @Pradyuman7, this PR resolves #6 with a more readable approach and emphasis on git commands. Can you look into this?


Fix changelog

What does this PR do?
Fixes changelog
Checklist

[x] This is something else

[x] I have updated CHANGELOG.asciidoc

@felixbarny Would please have a look?

:grey_exclamation: Build Aborted

The PR is not allowed to run in the CI yet

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Reason: The PR is not allowed to run in the CI yet

Start Time: 2022-01-14T09:27:41.181+0000

Duration: 4 min 42 sec

Commit: 860fcf866c2cb7ebf30fba4af96dd7ee58a308de

:robot: GitHub comments
To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

run benchmark tests : Run the benchmark test.

run compatibility tests : Run the JDK Compatibility test.

run integration tests : Run the APM-ITs.

run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)


Introduce gomemlimit

Is your feature request related to a problem?
Go1.19 introduces debug.SetMemoryLimit to set soft memory limit of a running process, which can help to avoid OOM in some scenarios.
More references:

https://tip.golang.org/doc/gc-guide
https://weaviate.io/blog/gomemlimit-a-game-changer-for-high-memory-applications
https://dzone.com/articles/how-we-solved-an-oom-issue-in-tidb-with-gomemlimit

Describe the feature you'd like
Add a server config to control memory limit
Describe alternatives you've considered
No response
Teachability, Documentation, Adoption, Migration Strategy
No response

/label affects-6.5


Caption blinking and settings handled on Video player

Description
LEARNER-7560

Handle Caption blinking
Persist Closed captions settings

Is the following issue linked to this PR?
On a fresh install, the captions appear on a video even when 'None' is selected by default on the captions popup.

Fixed


I want to try some NSFW words but it can't work.

I want to try some NSFW words on the huggingface "Language Models for Taiwanese Culture" page, but it would shows"很抱歉，我無法完成這個提示。這違反了OpenAI的內容政策，禁止描述或提供有關性內容的指令。我還能幫您做些什麼嗎？"

Our model operates within specific guidelines and usage policies, including content restrictions.

I think this issue is more about the the model's confusion about its own role, even if this model is not from OpenAI. e.g.
User: 你叫什麼名字
Bot: 我是一個開源的人工智能助理，你可以叫我 OpenAssistant。
User: 你是誰
Bot: 我是 Yen-Ting Lin，一位在人工智能和機器學習領域的研究人員。我可以回答你的問題並提供幫助。


Bug: 3.10.8 installs the stray file /usr/src/stringzilla/lib.c

Describe the bug
see subject
Steps to reproduce
build+install
Expected behavior
n/a
StringZilla version
3.10.8
Operating System
FreeBSD 14.1
Hardware architecture
x86
Which interface are you using?
C implementation
Contact Details
No response
Are you open to being tagged as a contributor?

[X] I am open to being mentioned in the project .git history as a contributor

Is there an existing issue for this?

[X] I have searched the existing issues

Code of Conduct

[X] I agree to follow this project's Code of Conduct

cc @Spixmaster, #191

@yurivict, what do you suggest?

The file is needed, https://github.com/unum-cloud/usearch/issues/522.

@Spixmaster, I wouldn’t reference that as a reason. We should and we use the git submodules there. The broader question is, what is the right installation path across different OSes? Hard-coding the path common to a couple of Linux distros is a bad solution.

what do you suggest?

Install it into $PREFIX/share/stringzilla/.
This is a standard place for "other" files that are needed.


Cluster overview is showing cores = nthreads * workers, instead of actual cores

What happened:
Cluster overview is showing misleading number of used cores.
cluster = SLURMCluster(
    queue='queue-name',
    project="project-name",
    cores=24,
    processes=1,
    memory="32GB",
    interface='bond0',
    walltime='4:00:00',
    extra=['--nthreads 2']
)

What you expected to happen:
When I set extra params of worker by passing extra, eg extra=['--nthreads 2'], number of cores is misleading, as it shows nthreads (used by workes) times workers, instead of actual reserved cores, which would be cores (as passed to eg. SLURMCluster) times jobs.
I guess such number came from assumption that 1 thread = 1 python job, but we could run multiprocessing within one python script (eg. via native ProcessPoolExecutor or joblib/loky). Therefore this should be changed as is somehow misleading when it comes to resource consumption of HPC (which might be charged)
dask==2021.1.0
dask-jobqueue==0.7.2
Python 3.8.5
Linux 3.10
pip

Hi @bgulowaty, thanks for posting here, and sorry for the delay.
I see what you'd want in your specific use case, but I think it is too edgy and wouldn't represent everyone's point of view.
SlurmCluster in an implementation of SpecCluster from dask.distributed, so it must also be consistent with other Clusters implementations. The widget you displayed just represent the Dask Cluster that is running, not necessarily the resources you've requested from your job queueing system (or K8S Cluster, or anything).
So from my point of view, this should not be changed.
In a general rule, I would advise against running Python multiprocessing into dask workers in most cases, this can be tricky. But you may want to run other binaries that are multi threaded.
If you want a consistent view on your cluster, you can also try to use Dask worker resources. This way, you would be able to express multiprocessing jobs along with single core ones.
And finally, it is interesting to see that in your case you specify an extra args for reducing the worker threads, and not for giving additionnal resource to individual jobs through scheduler options. What I mean is that in your case I would have done something like:
cluster = SLURMCluster(
    queue='queue-name',
    project="project-name",
    cores=2,
    processes=1,
    memory="32GB",
    interface='bond0',
    walltime='4:00:00',
    job_extra=['--ncpu=24'] #Or whatever Slurm option is
)

Hi @bgulowaty, thanks for posting here, and sorry for the delay.
I see what you'd want in your specific use case, but I think it is too edgy and wouldn't represent everyone's point of view.
SlurmCluster in an implementation of SpecCluster from dask.distributed, so it must also be consistent with other Clusters implementations. The widget you displayed just represent the Dask Cluster that is running, not necessarily the resources you've requested from your job queueing system (or K8S Cluster, or anything).
So from my point of view, this should not be changed.
In a general rule, I would advise against running Python multiprocessing into dask workers in most cases, this can be tricky. But you may want to run other binaries that are multi threaded.
If you want a consistent view on your cluster, you can also try to use Dask worker resources. This way, you would be able to express multiprocessing jobs along with single core ones.
And finally, it is interesting to see that in your case you specify an extra args for reducing the worker threads, and not for giving additionnal resource to individual jobs through scheduler options. What I mean is that in your case I would have done something like:
cluster = SLURMCluster(
    queue='queue-name',
    project="project-name",
    cores=2,
    processes=1,
    memory="32GB",
    interface='bond0',
    walltime='4:00:00',
    job_extra=['--ncpu=24'] #Or whatever Slurm option is
)


Add auth skip based on whitelisted ip addresses

You can now provide an overloaded flag called --skip-auth-ip-whitelist,
this will skip the authentication step if the client ip in the
X-Forwarded-For header is matches the whitelist provided to the proxy.
It will do both direct matches and CIDR block matches.
e.g. --skip-auth-ip-whitelist="<IP_ADDRESS>" will exactly match
<IP_ADDRESS> whilst --wkip-auth-ip-whitelist="<IP_ADDRESS>/29" will match
the range of <IP_ADDRESS> - <IP_ADDRESS>

Is this still relevant or can this be closed? Going through old PRs and found this.

Have we completely moved all our apps over to Yubikeys? If so we no longer need this fork

Two more are pending but after that, yes. Would rather not discuss on this public repo + PR :)


[AutoPR containerinstance/resource-manager] [DO NOT MERGE] Update readme.nodejs.md

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3544

Codecov Report

Merging #3044 into master will increase coverage by <.01%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master    #3044      +/-   ##
==========================================
+ Coverage   57.43%   57.43%   +<.01%
==========================================
  Files        7767     7767
  Lines      158336   158339       +3
==========================================
+ Hits        90937    90939       +2
- Misses      67399    67400       +1

Impacted Files
Coverage Δ

...re/mgmt/containerinstance/operations/operations.py
48.27% <0%> (ø)
:arrow_up:

...nce/operations/container_group_usage_operations.py
45.16% <0%> (ø)
:arrow_up:

...ntainerinstance/operations/container_operations.py
98.18% <100%> (+0.03%)
:arrow_up:

...instance/operations/container_groups_operations.py
58.27% <50%> (-0.12%)
:arrow_down:

Continue to review full report at Codecov.

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


Implemented aesthetic changes to UI

Implementing some of my suggested UI changes, might need @outruncolors help to rectify an issue with spacing on the header links.

@SIC0N I like most of the changes, but I'm not a fan of the mega-thick borders. If you reduce them and fix the conflicts, I'd be happy to merge this in!

@outruncolors conflicts resolved and borders reduced :)


Support for multithreading?

Not so much an issue, but more a suggestion, I noticed while in the latter stages of the Chapter 2 end to end tutorial, that despite supporting 8 threads, the more maths intensive end of the code was only ever running on one core in my CPU. Given the complexity and size of some datasets, it may be wiser to optimise the code to spread itself across all cores available?

I agree, some cells take too long, it's sometimes frustrating.  Some functions support an n_jobs argument that you can set to have the algorithm run in parallel on multiple cores (see the solution of exercise 1 in chapter 2 for an example, where n_jobs=4). That's the simplest way to speed up the notebook. One reason I did not do that is because I wanted to keep the examples as reproducible as possible. I set a random seed everywhere so that I would always get the same output, and I limited to one CPU for that same reason (with multiple cores, I would not necessarily get the

Hi @martynoconnor ,
Have you tried setting n_jobs to the number of cores on your machine?  Did it improve things?


Fails to load CreateThis_VRTK_GrabAttach and CreateThis_VRTK_Interactable

Unity 2017.1.0f3
SteamVR 1.2.2
VRTK 3.2.1
Fails to load CreateThis_VRTK_GrabAttach and CreateThis_VRTK_Interactable
Panel -> The associated script can not be loaded. Please fix any compile errors and assign a valid script.

What is the actual compile error? I think this is the result of the compile error not the error itself.

There is no compile error, the scripts just fail to load with the reported message against the panel object.

Do you have VRTK set per the Platform Custom Defines section in the README? Working fine without VRTK with 2017.1 and SteamVR 1.2.2 on my end.
Those scripts are right here and are pretty straight forward: https://github.com/createthis/createthis_vr_ui/blob/master/Assets/CreateThis/Scripts/VRTK/CreateThis_VRTK_GrabAttach.cs
I also wonder if this could be a case where Library needs to be deleted and rebuilt as Unity has some other version of the code cached.

Yes ;VRTK is set per the Platform Custom Defines.
Yes it is working fine for me also in 2017.1.0f3 and SteamVR 1.2.2 without VRTK.
I am not keen on messing with deleting the Library so tries it with 5.6.2f1 on a different machine and got the same notification against the Panel, but no actual compile error.
I can't force an error with either the Asset Store or GitHub versions of VRTK, but it does throw the following errors if I use the VRTK-release-3.3.0-alpha:
Assets/CreateThis/Scripts/VRTK/CreateThis_VRTK_Interactable.cs(10,30): error CS0115: `CreateThis.VRTK.CreateThis_VRTK_Interactable.StartTouching(UnityEngine.GameObject)' is marked as an override but no suitable method found to override
Assets/CreateThis/Scripts/VRTK/CreateThis_VRTK_Interactable.cs(21,30): error CS0115: `CreateThis.VRTK.CreateThis_VRTK_Interactable.StopTouching(UnityEngine.GameObject)' is marked as an override but no suitable method found to override

I downloaded the commit update to 2017 you just made today, started from scratch and noted the following when using the Asset store version of VRTK:
Assets/CreateThis/Scripts/VRTK/CreateThis_VRTK_Interactable.cs(10,30): warning CS0672: Member CreateThis.VRTK.CreateThis_VRTK_Interactable.StartTouching(UnityEngine.GameObject)' overrides obsolete member VRTK.VRTK_InteractableObject.StartTouching(UnityEngine.GameObject)'. Add the Obsolete attribute to `CreateThis.VRTK.CreateThis_VRTK_Interactable.StartTouching(UnityEngine.GameObject)'
Assets/CreateThis/Scripts/VRTK/CreateThis_VRTK_Interactable.cs(21,30): warning CS0672: Member CreateThis.VRTK.CreateThis_VRTK_Interactable.StopTouching(UnityEngine.GameObject)' overrides obsolete member VRTK.VRTK_InteractableObject.StopTouching(UnityEngine.GameObject)'. Add the Obsolete attribute to `CreateThis.VRTK.CreateThis_VRTK_Interactable.StopTouching(UnityEngine.GameObject)'
Assets/CreateThis/Scripts/VRTK/CreateThis_VRTK_Interactable.cs(11,18): warning CS0618: VRTK.VRTK_InteractableObject.StartTouching(UnityEngine.GameObject)' is obsolete: ``VRTK_InteractableObject.StartTouching(GameObject currentTouchingObject) has been replaced with VRTK_InteractableObject.StartTouching(VRTK_InteractTouch currentTouchingObject). This method will be removed in a future version of VRTK.'
Assets/CreateThis/Scripts/VRTK/CreateThis_VRTK_Interactable.cs(22,18): warning CS0618: VRTK.VRTK_InteractableObject.StopTouching(UnityEngine.GameObject)' is obsolete: ``VRTK_InteractableObject.StopTouching(GameObject previousTouchingObject) has been replaced with VRTK_InteractableObject.StopTouching(VRTK_InteractTouch previousTouchingObject). This method will be removed in a future version of VRTK.'

Is it an easy refactor? replace GameObject with VRTK_InteractTouch?


Implementation for standard handlers.

Hello.
I needed to implement an event of type PropertyChangedEventHandler. But the above code only implements EventHandler<MyEventArgs>. I've modified your code a little to suit my needs, but maybe this can be implemented in the main project?
Below is an example using my code:
        private readonly WeakEventSource<PropertyChangedEventHandler, PropertyChangedEventArgs> _PropertyChangedEventSource = new();
        public event PropertyChangedEventHandler PropertyChanged
        {
            add { _PropertyChangedEventSource.Subscribe(value); }
            remove { _PropertyChangedEventSource.Unsubscribe(value); }
        }

Hi @OlegKozhevnikov,

maybe this can be implemented in the main project?

Maybe, I'd need to see what the changes are. Did you change the existing WeakEventSource class to take an additional type parameter, or did you add a new class?

Yes, I used an additional type parameter.
The changes are small, but not compatible with EventHandler<MyEventArgs> . It would probably be easier to add a separate class for standard events?
public class WeakEventSource<TEventHandler, TEventArgs>
	where TEventHandler : Delegate
{
	...
	public void Subscribe(TEventHandler handler)
	{
		Subscribe(null, handler);
	}

	public void Subscribe(object lifetimeObject, TEventHandler handler)
	{
		Subscribe<DelegateCollection, OpenEventHandler, StrongHandler>(lifetimeObject, ref _handlers, handler);
	}

	public void Unsubscribe(TEventHandler handler)
	{
		Unsubscribe<OpenEventHandler, StrongHandler>(_handlers, handler);
	}
	....

@OlegKozhevnikov, if this improvement were to be implemented in the main project, it would have to maintain compatibility with previous versions somehow.
Maybe WeakEventSource<TEventArgs> could inherit WeakEventSource<EventHandler<TEventArgs>, TEventArgs>. Anyway, I'd need to see what you changed exactly. Could you submit a PR so that we can discuss it? (even if your PR currently isn't currently backwards compatible)
Thanks

Hi @thomaslevesque
The main problem is that PropertyChangedEventHandler is not a descendant of EventHandler. In the new class, I simply removed the EventHandler and added an explicit type to the type parameters.
In theory, you can simply add another file to the project, similar to AsyncEventHandler. This class will apply to all delegat.
Below is the full text of the new class. There are only 5 changes from the original where TEventHandler.

using System;
using static WeakEvent.WeakEventSourceHelper;

namespace WeakEvent
{
    /// <summary>
    /// An event with weak subscription, i.e. it won't keep handlers from being garbage collected.
    /// </summary>
    /// <typeparam name="TEventArgs">The type of the event's arguments.</typeparam>
    public class WeakEventSourceDelegate<TEventHandler, TEventArgs>
        where TEventHandler : Delegate
#if NET40
        where TEventArgs : EventArgs
#endif
    {
        internal DelegateCollection? _handlers;

        /// <summary>
        /// Raises the event by invoking each handler that hasn't been garbage collected.
        /// </summary>
        /// <param name="sender">The source of the event.</param>
        /// <param name="args">An object that contains the event data.</param>
        /// <remarks>The handlers are invoked one after the other, in the order they were subscribed in.</remarks>
        public void Raise(object? sender, TEventArgs args)
        {
            var validHandlers = GetValidHandlers(_handlers);
            foreach (var handler in validHandlers)
            {
                handler.Invoke(sender, args);
            }
        }

        /// <summary>
        /// Raises the event by invoking each handler that hasn't been garbage collected. Exceptions thrown by
        /// individual handlers are passed to the specified <c>exceptionHandler</c> to decide what to do with them.
        /// </summary>
        /// <param name="sender">The source of the event.</param>
        /// <param name="args">An object that contains the event data.</param>
        /// <param name="exceptionHandler">A delegate that handles exceptions thrown by individual handlers.
        /// Return <c>true</c> to indicate that the exception was handled.</param>
        /// <remarks>The handlers are invoked one after the other, in the order they were subscribed in.</remarks>
        public void Raise(object? sender, TEventArgs args, Func<Exception, bool> exceptionHandler)
        {
            if (exceptionHandler is null) throw new ArgumentNullException(nameof(exceptionHandler));
            var validHandlers = GetValidHandlers(_handlers);
            foreach (var handler in validHandlers)
            {
                try
                {
                    handler.Invoke(sender, args);
                }
                catch (Exception ex) when (exceptionHandler(ex))
                {
                }
            }
        }

        /// <summary>
        /// Adds an event handler.
        /// </summary>
        /// <param name="handler">The handler to subscribe.</param>
        /// <remarks>Only a weak reference to the handler's <c>Target</c> is kept, so that it can be garbage collected.</remarks>
        public void Subscribe(TEventHandler handler)
        {
            Subscribe(null, handler);
        }

        /// <summary>
        /// Adds an event handler, specifying a lifetime object.
        /// </summary>
        /// <param name="lifetimeObject">An object that keeps the handler alive as long as it's alive.</param>
        /// <param name="handler">The handler to subscribe.</param>
        /// <remarks>Only a weak reference to the handler's <c>Target</c> is kept, so that it can be garbage collected.
        /// However, as long as the <c>lifetime</c> object is alive, the handler will be kept alive. This is useful for
        /// subscribing with anonymous methods (e.g. lambda expressions).</remarks>
        public void Subscribe(object? lifetimeObject, TEventHandler handler)
        {
            Subscribe<DelegateCollection, OpenEventHandler, StrongHandler>(lifetimeObject, ref _handlers, handler);
        }

        /// <summary>
        /// Removes an event handler.
        /// </summary>
        /// <param name="handler">The handler to unsubscribe.</param>
        /// <remarks>The behavior is the same as that of <see cref="Delegate.Remove(Delegate, Delegate)"/>. Only the last instance
        /// of the handler's invocation list is removed. If the exact invocation list is not found, nothing is removed.</remarks>
        public void Unsubscribe(TEventHandler handler)
        {
            Unsubscribe<OpenEventHandler, StrongHandler>(_handlers, handler);
        }

        /// <summary>
        /// Removes an event handler that was subscribed with a lifetime object.
        /// </summary>
        /// <param name="lifetimeObject">The lifetime object that was associated with the handler.</param>
        /// <param name="handler">The handler to unsubscribe.</param>
        /// <remarks>The behavior is the same as that of <see cref="Delegate.Remove(Delegate, Delegate)"/>. Only the last instance
        /// of the handler's invocation list is removed. If the exact invocation list is not found, nothing is removed.</remarks>
        [Obsolete("This method is obsolete and will be removed in a future version. Use the Unsubscribe overload that doesn't take a lifetime object instead.")]
        public void Unsubscribe(object? lifetimeObject, TEventHandler handler)
        {
            Unsubscribe(handler);
        }

        internal delegate void OpenEventHandler(object? target, object? sender, TEventArgs e);

        internal struct StrongHandler
        {
            private readonly object? _target;
            private readonly OpenEventHandler _openHandler;

            public StrongHandler(object? target, OpenEventHandler openHandler)
            {
                _target = target;
                _openHandler = openHandler;
            }

            public void Invoke(object? sender, TEventArgs e)
            {
                _openHandler(_target, sender, e);
            }
        }

        internal class DelegateCollection : DelegateCollectionBase<OpenEventHandler, StrongHandler>
        {
            public DelegateCollection()
                : base((target, openHandler) => new StrongHandler(target, openHandler))
            {
            }
        }
    }
}

I can't provide the whole project, I have isolated the key points of the code.
Used like this:

public class BO_Base : INotifyPropertyChanged, IEditableObject
{
	private readonly WeakEventSourceDelegate<PropertyChangedEventHandler, PropertyChangedEventArgs> _PropertyChangedEventSource = new();
	public event PropertyChangedEventHandler PropertyChanged
	{
		add { _PropertyChangedEventSource.Subscribe(value); }
		remove { _PropertyChangedEventSource.Unsubscribe(value); }
	}

	#region Strong
	//public event PropertyChangedEventHandler PropertyChanged;
	//protected virtual void OnPropertyChanged(object sender, PropertyChangedEventArgs e)
	//{ PropertyChanged?.Invoke(sender, e); }
	#endregion

	protected void RaisePropertyChanged([CallerMemberName] string propertyName = "")
	{
		// Weak
		_PropertyChangedEventSource.Raise(this, new PropertyChangedEventArgs(propertyName));

		// Strong
		//PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName));
	}
}

:)

@OlegKozhevnikov I see, thanks.
This seems to work fine for e.g. PropertyChangedEventHandler, but what bothers me is that it won't actually work with all delegate types. There's nothing to prevent incorrect usage at compile time. For instance, I could write WeakEventSource<MyEventHandler, MyUnrelatedEventArgs>, and it would compile just fine, but would fail at runtime.
C# has no way to express that TEventHandler must be of the form void TEventHandler(object sender, TEventArgs e). That's actually one of the reasons why I didn't attempt to support arbitrary delegate types in the first place.
Of  course, this validation could be done at runtime, but it's not ideal...
Anyway, I don't think you really need a specialized class for your scenario. You could just do this:
	private readonly WeakEventSource<PropertyChangedEventArgs> _propertyChangedSource = new();
	public event PropertyChangedEventHandler PropertyChanged
	{
		add => _propertyChangedSource.Subscribe(new EventHandler<PropertyChangedEventArgs>(value));
		remove => _propertyChangedSource.Unsubscribe(new EventHandler<PropertyChangedEventArgs>(value));
	}

(there's no implicit conversion from PropertyChangedEventHandler to EventHandler<PropertyChangedEventArgs>, but the explicit conversion works fine, since the signature matches)
If you're using C# 9 or later, you can even use target-typed new expressions to do this:
	private readonly WeakEventSource<PropertyChangedEventArgs> _propertyChangedSource = new();
	public event PropertyChangedEventHandler PropertyChanged
	{
		add => _propertyChangedSource.Subscribe(new(value));
		remove => _propertyChangedSource.Unsubscribe(new(value));
	}

@thomaslevesque
It `s Magic!
Your solution is simple and elegant.
Thanks a lot!
P.S. I thought about the non-match of the handler and the argument, but it didn't matter to me.

@OlegKozhevnikov You're welcome, I'm glad it works for you!


clear src= on recycled s

Solves #351

Wondering if there is a way to make this target Firefox since that's the only browser affected by the src reuse issue.

It looks like it affects my Chrome as well.  You can try it out with my old codepen.

Ah okay, didn't realize it was in both.

Hey, just bumping this again:  is there a conflict with merging in this bugfix?

Nope, just on vacation. On my flight home actually!


Ember.EnumberableUtils.map is deprecated, please refactor to use Array.prototype.map

These may be affected:

https://github.com/softlayer/sl-ember-components/blob/c2d14ee32fc9089a7d24e89d382bead380309cf3/addon/components/sl-grid-cell.js#L111
https://github.com/softlayer/sl-ember-components/blob/9962cea468f9cb055422c6e679509864af0067b5/addon/components/sl-alert.js#L111
https://github.com/softlayer/sl-ember-components/blob/9962cea468f9cb055422c6e679509864af0067b5/addon/components/sl-button.js#L190
https://github.com/softlayer/sl-ember-components/blob/9962cea468f9cb055422c6e679509864af0067b5/addon/components/sl-button.js#L227
https://github.com/softlayer/sl-ember-components/blob/9962cea468f9cb055422c6e679509864af0067b5/addon/components/sl-drop-button.js#L150
https://github.com/softlayer/sl-ember-components/blob/9962cea468f9cb055422c6e679509864af0067b5/addon/components/sl-progress-bar.js#L138
https://github.com/softlayer/sl-ember-components/blob/0ce928e1f811edb0bf859b44759cca212aeabe7f/addon/components/sl-tab-panel.js#L231

Once Ember 1.13 is used by this addon

This issue can be closed. I checked each of the links above and they have already been refactored to not use Object.keys().map(). I also did a grep on the repo looking for Ember.EnumberableUtils.map,  EnumberableUtils.map or Object.keys().map but did not find any outstanding instances


Adding Digicert CIS Sourceplugin

Coverage decreased (-0.2%) to 60.538% when pulling 4be073c4a11bcf667a3fa173317887181f7a3bfc on kevgliss:digicert-backfill into 9e2578be1e7da27dcaf2d35bcc4e3b372249dba5 on Netflix:master.

Coverage decreased (-0.2%) to 60.538% when pulling 13555676f553c127605aceaf5db338bbf4fcf0af on kevgliss:digicert-backfill into 9e2578be1e7da27dcaf2d35bcc4e3b372249dba5 on Netflix:master.


tlsh: 3.4.5 -> 4.5.0

Motivation for this change
tlsh hasn't been updated in a long time, while there have been a few releases. Unfortunately the TLSH developers decided to not use tags for their new releases.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Please change the commit message and PR title to tlsh: 4.5.0 -> 3.4.5 to fit the contributing guide.

Please change the commit message and PR title to tlsh: 4.5.0 -> 3.4.5 to fit the contributing guide.

Since I don't want to downgrade I did it the other way around ;-)

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 111949 run on x86_64-darwin 1

1 package failed to build and already failed to build on hydra master:

diffoscope: log was empty

3 packages built:

python37Packages.tlsh
python38Packages.tlsh
python39Packages.tlsh

error: --- Error -----------------------------------------------------------------------------------------------------------------------------------------------------------------------
nixbuilder for '/nix/store/mikmbqjj49ppfjzfk3sgb7zxmlqldg0j-e2fsprogs-1.45.6.drv' failed with exit code 2; last 10 log lines:
/nix/store/2c9kjj14qhns6cd28cmm51jvy74yk4c4-Libsystem-osx-10.12.6/include/sys/_types/_off_t.h:30:25: note: 'off_t' declared here
typedef __darwin_off_t          off_t;
^
1 error generated.
make[2]: *** [Makefile:611: imager.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[2]: Leaving directory '/private/tmp/nix-build-e2fsprogs-1.45.6.drv-0/e2fsprogs-1.45.6/lib/ext2fs'
make[1]: *** [Makefile:445: all-libs-recursive] Error 1
make[1]: Leaving directory '/private/tmp/nix-build-e2fsprogs-1.45.6.drv-0/e2fsprogs-1.45.6'
make: *** [Makefile:366: all] Error 2
error: --- Error -----------------------------------------------------------------------------------------------------------------------------------------------------------------------
nix1 dependencies of derivation '/nix/store/291ricwi43wxd34adg4ypqirx8b3wy30-diffoscope-163.drv' failed to build
error: --- Error -----------------------------------------------------------------------------------------------------------------------------------------------------------------------
nix1 dependencies of derivation '/nix/store/wcr9780gc1z6dy5y2kwfc2nm86bjxjw7-env.drv' failed to build

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 111949 run on x86_64-linux 1

4 packages built:

diffoscope
python37Packages.tlsh
python38Packages.tlsh
python39Packages.tlsh

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 111949 run on x86_64-linux 1

4 packages built:

diffoscope
python37Packages.tlsh
python38Packages.tlsh
python39Packages.tlsh


MacOS Big Sur Support -- Empty Monarch Desktop Client.

The Monarch Desktop Client does not seem to support MacOS Big Sur.

We'll be testing Monarch on Big Sur in the next few weeks. We should be able to get to it before Big Sur is officially released.
I'll leave this issue open to track the work. Thanks for the feedback, @amouxkin.

Thank you for the quick response @fertrig , I really this project and was wondering if there were any way to contribute ?

I'm glad you like it! Someone from our team will reach out to see how you can help us out.

Great, looking forward to it.

@amouxkin Monarch is now working on Big Sur.


Conv1DTranspose Dilation support - Might be a bug, IDK.

Thank you for submitting a TensorFlow documentation issue. Per our GitHub
policy, we only address code/doc bugs, performance issues, feature requests, and
build/installation issues on GitHub.
The TensorFlow docs are open source! To get involved, read the documentation
contributor guide: https://www.tensorflow.org/community/contribute/docs
URL(s) with the issue:
Please provide a link to the documentation entry, for example:
https://www.tensorflow.org/api_docs/python/tf/keras/layers/Conv1DTranspose
Description of issue (what needs changing):
Conv1DTranspose - Dilation - Does not inform uses that dilation doesn't work for any value  of Dilation>1 because it isn't implemented yet.
Clear description
Currently documentation says:
"an integer, specifying the dilation rate to use for dilated convolution. Currently, specifying a dilation_rate value != 1 is incompatible with specifying a stride value != 1."
This may not be implemented yet in the newest of nightly build, but with my tf-nightly==2.5.0dev20200629 build this didn't work. I fear updating to new nightly builds in case in breaks my research code which relies on nightly builds until Conv1DTranspose is released in a supported build.
InvalidArgumentError:  Current libxsmm and customized CPU implementations do not yet support dilation rates larger than 1.
	 [[node test1_AE/decoder/conv1d_transpose/conv1d_transpose (defined at D:\Users\[username]\Desktop\libs_python\nn4n_autoencoder4.py:120) ]] [Op:__inference_train_function_2185]

Function call stack:
train_function

This is with stride == 1.
Correct links
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/python/keras/layers/convolutional.py
Parameters defined
Yes, setting my dilation to 1 gets rid of the issue.
Returns defined
Not necessary. (I'm not sure if you are asking if I have define returns in my code or if my code returns a defined value, or if the documentation claims to return something)
Raises listed and defined
InvalidArgumentError:  Current libxsmm and customized CPU implementations do not yet support dilation rates larger than 1.
	 [[node test1_AE/decoder/conv1d_transpose/conv1d_transpose (defined at D:\Users\[username]\Desktop\libs_python\nn4n_autoencoder4.py:120) ]] [Op:__inference_train_function_2185]

Function call stack:
train_function

Usage example
Nightly build, so no.
Request visuals, if applicable
No.
Submit a pull request?
I will not be doing so.
Test Code
Note 1: This is built with tf-nightly==2.5.0dev20200626 which was removed from the PyPi archive for unknown reasons.
Note 2: model.fit must be called for the error to occur. Simpy constructing and compiling the network was not enough to reproduce the error.
import tensorflow as tf
import tensorflow.keras as krs
import numpy as np

train_data = np.random.uniform(-1,1,(20,20))

inputs = krs.Input((20,1))

x = inputs

x = krs.layers.Conv1D(1,3,strides = 1,padding='same',dilation_rate=2,activation='relu')(x)
x = krs.layers.Flatten()(x)
x = krs.layers.Dense(10,activation='relu')(x)
x = krs.layers.Dense(2,activation='relu')(x)
x = krs.layers.Dense(10,activation='relu')(x)
x = krs.layers.Dense(20,activation='relu')(x)
x = krs.layers.Reshape(target_shape=(20,1))(x)
x = krs.layers.Conv1DTranspose(1,3,strides=1,dilation_rate=2,padding='same',activation='relu',output_padding=0)(x)
output = krs.layers.Flatten()(x)

model = krs.Model(inputs,output,name='test')

model.compile(optimizer='adam',loss='MSE')

model.summary()

model.fit(train_data,train_data)

@EnderWiggin14
Could you please update the issue template, we do not find any details.
Please provide with simple indented stand alone code such that we can replicate the issue faced or if possible please provide a colab gist with error faced.

Well, this was originally labeled as a Documentation issue so that "Details" section wasn't part of the standard template. I will try to get a short example added to the issue.

I ran the code on nightly, please find the gist here and on 2.3.0rc0 the dilation support error is seen as reported.

It is very odd that the TF nightly build used in the GIST (tf-nightly==2.4.0dev20200724) doesn't have Conv1DTranspose() support. I have another system that's running a slightly older tf-nightly build (tf-nightly==2.3.0dev20200622) which was still available 2-3 weeks ago but appears to now be deleted. This is one of the two builds that I actively use as well, and it supports the Conv1DTranspose() method, but also lacks the dilation support.

I can repro this with latest nightly build. See gist
When strides = 1 and dilation_rate = 1 in the tf.keras.layers.Conv1DTranspose layer the code executes successfully.
However for dilation_rate > 1 it fails.
Perhaps the documentation can be updated for the dilation_rate parameter. Thanks!

Docs are updated now.  See commit 75801da
Thanks!

Thanks! Is there any news as to when the dilation_rate > 1 will be supported?


How to use it in AWS Lambda?

I am a newbie to Lambda. I copied the below codes to my lambda_function.py and created a layer for all the libs, when I run the Lambda, it throws an error stating that lambda_handler isn't in the lambda function.
from fastapi import FastAPI
from lambda_handler import LambdaHandler, SnsEvent, LambdaResponse

handler = LambdaHandler()
...........
handler.fastapi_app = app

@asaarashi hey, in this case the handler is handler, so you need to reference lambda_function.handler in the Lambda function defintion in the AWS UI.


Bangla translation

Translation added for Bengali Language.

Thank you, but we don't maintain translations in devise itself. Please check the wiki for other options: https://github.com/heartcombo/devise/wiki/I18n


Add CommandSequence.takeoff to the public API

This is defined in the implementation class (not the interface class) and does not appear in the documentation.
It needs to be made visible in the documentation, ideally by defining it in the interface.
Note: I split this out from #64 so I can assign ownership to @mrpollo .

#134 adds the method to the documentation as part of the "class docs". I'm leaving this open because even though this will fix the documentation it doesn't do it in the right way - which is to add the method into the init.py .

@hamishwillee Is this technically a bug still? Looks like it's part of CommandSequence now (although we may move that into Vehicle proper?)

@tcr3dr No, this can be closed.
I will move the documentation for all methods under the methods themselves (out of the "class level" docs) as soon as we get some API stability.

@hamishwillee Sounds great.


Add env function similar to consul-template

@jrasell It's a mercurial file (https://www.mercurial-scm.org/wiki/DirState).  I think GoLand indexed that directory and wrote the dirstate.  I'll add that to the ignore.

So I'm not sure I should ignore those files ... they are part of the mercurial structure along with the cache folder.  Since the entire vendor directory is checked into the repo the contents of the dirstate and cache folder may change anytime someone indexes those files.  What do you want to do?

For instance, my current git status looks like this ...
ahurt1:levant ahurt$ git status
On branch feature/add-env-func
Untracked files:
  (use "git add <file>..." to include in what will be committed)

	.idea/
	vendor/github.com/hashicorp/go-getter/test-fixtures/basic-hg/.hg/cache/checkisexec
	vendor/github.com/hashicorp/go-getter/test-fixtures/basic-hg/.hg/cache/checklink
	vendor/github.com/hashicorp/go-getter/test-fixtures/basic-hg/.hg/cache/checklink-target
	vendor/github.com/hashicorp/go-getter/test-fixtures/basic-hg/.hg/cache/checknoexec
	vendor/github.com/hashicorp/go-getter/test-fixtures/basic-hg/.hg/cache/tags2-visible
	vendor/github.com/hashicorp/terraform/config/module/test-fixtures/basic-hg/.hg/cache/checkisexec
	vendor/github.com/hashicorp/terraform/config/module/test-fixtures/basic-hg/.hg/cache/checklink
	vendor/github.com/hashicorp/terraform/config/module/test-fixtures/basic-hg/.hg/cache/checklink-target
	vendor/github.com/hashicorp/terraform/config/module/test-fixtures/basic-hg/.hg/cache/checknoexec
	vendor/github.com/hashicorp/terraform/config/module/test-fixtures/basic-hg/.hg/cache/tags2-visible

nothing added to commit but untracked files present (use "git add" to track)

The reason the dirstate files showed as changed is they are already in the git history and the cache/* files are not.

@leprechau I have never used mercurial so can't really comment but I would have a preference for avoiding these files as if they are just caching files they shouldn't be required.

What do you think of just removing the .hg structure from under vendor?

Okay, I ignored the vendor/*/.hg/* structure and reverted the dirstate files back to the master revision.

@leprechau Do you have some examples / use cases that you can include for this? I'm mostly looking for inspiration on how to use it.

@stevenscg our use case is executing levant from within our Bamboo build system and using the exported environment variables from the build plan.


galene: init at 0.2

Motivation for this change
Status: draft
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 109414 1

1 package blacklisted:

tests.nixos-functions.nixos-test

1 package built:

galene

Result of nixpkgs-review pr 109414 1

1 package blacklisted:

tests.nixos-functions.nixos-test

1 package built:

galene

Sorry for the delay. I have left some comments, though I'm not sure how helpful they are. I hope at least some of them are... Let me know if you have any questions.

No problem. I'm gonna mark some of these comments as "resolved" but i'll wait until I resolve them all to push.

Sorry for the delay. I have left some comments, though I'm not sure how helpful they are. I hope at least some of them are... Let me know if you have any questions.

No problem. I'm gonna mark some of these comments as "resolved" but i'll wait until I resolve them all to push.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/456

/marvin opt-in
/status needs_reviewer


Add "quiet" option to suppress all log output

In working on my own repo hop, which uses nativefier under-the-hood, I found it very troublesome to suppress log output when using the programmatic API. This PR just adds a quick "quiet" option which will set the log level to "silent" and suppress all electron and nativefier output (except for errors, of course).
Let me know if there are any additional changes I need to make to get this merged!

Thanks for merging @ronjouch! When do you think you'll publish a new NPM version?


i can't getting any data from users table

I'm not sure I have the problem but I think it comes from the package
I have connections to other tables in the same tenant but when I try to get user data from the same tenant it tells me that the table does not exist in the central  tenants database
Is it an bug or is there a mistake I did not do, knowing that I followed the installation steps well
The strange thing is that it works with other tables and actually connects to the tenant database, but when I try to find a user through the App\Model\User, it tells me that the connection is on the central database and not on the same tenant database
SQLSTATE[42S02]: Base table or view not found: 1146 Table 'tenant.users' doesn't exist (SQL: select count(*) as aggregate from `users` where `account_type` = user)

Laravel Version: 9.*
package Version: 3.*
tenant database : tenant_foo
central database : tenants
Can someone tell me what to do?

anyone can help
i get data from another table like Role or Group
but He only wants to view the data through the  central database not by tenant  db

Hey!
Thanks for opening the issue. This doesn't seem to be a bug in package, seems to be related to your setup.
If you need quick support, you can visit our Discord community here: https://archte.ch/discord
Thanks