chore: update unenv dependency version

What this PR solves / how to test
Update unenv - Pulled fixes are listed in the changeset
See https://github.com/unjs/unenv/issues/314
Fixes #6684
Author has addressed the following

Tests

[ ] TODO (before merge)
[ ] Tests included
[x] Tests not necessary because: unenv update

E2E Tests CI Job required? (Use "e2e" label or ask maintainer to run separately)

[ ] I don't know
[x] Required
[ ] Not required because:

Changeset (Changeset guidelines)

[ ] TODO (before merge)
[x] Changeset included
[ ] Changeset not necessary because:

Public documentation

[ ] TODO (before merge)
[ ] Cloudflare docs PR(s):
[x] Documentation not necessary because: unenv update

/cc @anonrig

The error in the nodejs-hybrid-app is:
[ERROR] service core:user:nodejs-hybrid-app: Uncaught TypeError: Cannot read properties of undefined (reading 'bind')

Building the worker to dist, then changing wrangler.toml to point to the compiled output and adding no_bundle = truegives the actual line that is failing:
addEventListener: performance3.addEventListener.bind(performance3),

So it appears that addEventListener is not defined
Elsewhere in the code we can see:
performance3 = globalThis.performance ?? new Performance2();

The fake Performance2 class has addEventListener() but it is not used because globalThis.performance is defined, and is missing that function.
From the docs performance is defined and only has now() and timeOrigin(). Confirmed in workerd source code.
So this polyfill is a bit broken for Cloudflare. I wonder if we need to be more fine grained about polyfilling that global?

Great analysis @petebacondarwin

So it appears that addEventListener is not defined
Elsewhere in the code we can see:
performance3 = globalThis.performance ?? new Performance2();

The fake Performance2 class has addEventListener() but it is not used because globalThis.performance is defined, and is missing that function.

unenv should inject the polyfill in the globalScope here. But it looks like this is not working.
Also https://github.com/unjs/unenv/issues/318

Thanks Pete!


Add seq2seq eval benchmark callback

Similar to #441, this PR adds an evaluation benchmark for generative tasks as machine translation.
Description

This additional evaluation benchmark is self-contained and can be triggered via the do_causal_lm_eval configuration. First it will generate completions for every sample in the eval set (configured via eval_max_new_tokens) and then score these against the reference completions via 🤗 Evaluate. Metrics can be chosen among a subset of supported metrics and skipped with a warning if the corresponding libraries are not available.
A few notes on this PR:

I believe it could be further improved by removing duplicated code (e.g. it shares the same generation as LogPredictionCallback)
🤗 Evaluate requires additional libraries to be installed to compute metrics (e.g. sacrebleu is needed to compute BLEU). I didn't make to add this libraries as optional, so far code will simply raise a warning if the libraries for the requested metrics are missing. I believe this could be handled in a better way (maybe adding a [metrics] entry to the extras_require?)

Tagging @winglian and @tmm1 who maybe can help? Cheers.
Motivation and Context

Enable Axolotl to compute generative evaluation metrics to better support generative tasks (e.g. machine translation, language modelling, etc.).
How has this been tested?

End-to-end SFT fine-tuning of an existing LLama-2-hf model with and without the feature.
Screenshots (if appropriate)
Types of changes

[x] Add CausalLMBenchEvalCallback callback for generative tasks evaluation
[x] Add do_causal_lm_eval to AxolotlTrainingArguments
[x] Refactor eval_table_max_new_tokens into eval_max_new_tokens
[x] Bump evaluate to 0.4.1 for COMET fix (see related issue)
[ ] Test behaviour with sample_packing=True
[ ] Add unit and integration tests

Social Handles (Optional)

Is this intended as a replacement of the log prediction callback?
Looks good so far. I assume you want to finish off the last two items before we merge?

Is this intended as a replacement of the log prediction callback?

I believe the callbacks serve similar purposes but are different in the objective: the log callback generates a few (e.g. 5) samples for logging purposes while the causal_lm benchmark generates completions for all the dataset to compute metrics for generative tasks.
Ideally these two would share the same code for the generation (model.generate rather than predict from this thread) and store them somewhere to be re-used if do_causal_lm_eval=True. Do you have any suggestions how to achieve it?

Looks good so far. I assume you want to finish off the last two items before we merge?

I'll have a look at the case when sample_packing=True but don't currently have time to create ad-hoc tests.

@winglian please have a look now if this looks good to you. Do you have further suggestions?


Create and insert Mean as well as Median (with diffs) into MetricRAG table

Context
AB#238511 AB#238512 AB#236925
Change proposed in this pull request

Added new columns to MetricRAG table
During pipeline run, insert mean and median to correct MetricRAG columns

Guidance to review
A future piece of work will deprecate the Mean columns. For now, both averages will exist so as to not break the service. This can be validated by running the data-pipeline locally, after having first run the database migrations.
Checklist (add/remove as appropriate)

[x] Work items have been linked (use AB#)
[x] Your code builds clean without any errors or warnings
[x] You have run all unit/integration tests and they pass
[x] Your branch has been rebased onto main
[x] You have tested by running locally
[ ] You have reviewed with UX/Design

By implementing this way there is a risk that the Web/API will return and display the mean, when it should be displaying the median.

Agreed. It was hoped the changes elsewhere in the repo would follow shortly afterwards to reduce the impact, especially before a new pipeline run is completed. If it is preferred, the API and Web changes can all be done as part of a single PR.

My preference would be for the changes to be in separate PRs.
It was more the QA impact and awareness of the scope of these changes. There might have been an assumption that the prop/field would just be renamed therefore no risk of showing the wrong value (the code would break or show nothing).
QA will need to be aware of all places where these values are surfaced.

There might have been an assumption that the prop/field would just be renamed

Perhaps that is the wider question: should the 'average' be 'median' everywhere for RAGs and 'mean' dropped completely, or is there a case where one or the other may be consumed? I was trying to be cautious with the implementation, but if there is no need for both then the follow up work to clean this up should follow swiftly afterwards.

Median should be used everywhere at the moment. However I'm not sure if mean would have any value in future.
https://learn.microsoft.com/en-us/sql/relational-databases/system-stored-procedures/sp-rename-transact-sql?view=sql-server-ver16

Closing in anticipation of alternative implementation to simply rename the original column rather than support both median and mean.


Print an error if the godot-cpp submodule hasn't been initialized

This makes the error that is printed easier to understand, with a command given as a solution to run.
The error message is also colored to match how core Godot displays this kind of build errors.
Preview

Thanks!


Different rdf icon

We should use a different rdf-icon for the "show raw data"-toggle in the user menu, the icon we currently use looks too much like the default "share" icon. also the padding of the icon seems to be too excessive and therefore the icon is displayed too small, we should leave the icon padding to the css and not include a padding in the svgs in general, so we can adapt paddings within stylesheet instead.

fixed


newrelic-infrastructure-v3: KSM discovery uses code defaults

For discovering KSM, we are using constants holding the selector, scheme, and port:
https://github.com/newrelic/nri-kubernetes/blob/f6d1980085e98d434d2ce48e16729179bcbce3db/src/ksm/scraper.go#L22-L24
It would be good if we could align this behavior with other component's, and provide this defaults as config in the values.yaml.

Hey @imnot-abot,
It is possible by specifying them in the values.
ksm:
  config:
    selector: foo=bar

Ideally those should already be in there so it's obvious where to changed, which is what this issue is about :)


Don't add Content-Encoding and Transfer-Encoding if no body

This is a fix for issue #117.
This fix disables setting the Content-Encoding and Transfer-Encoding headers when a request is created with compress=True but does not have a body.

Coverage increased (+0.0006%) to 98.213% when pulling be7bfb850f9922f03ac7fb0afd2726c0c9d8e1ac on remove-content-headers-no-body into be9a084888bd4fbbf18e9fa21b815051f2dd5eb4 on master.

Thanks!


Fix linter warnings

This fixes everything VSCode is showing as a "Problem":

Capitalize common abbreviations (e.g. Id => ID)
Add comments to all exported methods
Capitalize Dockerfile instructions

Codecov Report

Merging #102 into master will not change coverage.
The diff coverage is 69.91%.

@@          Coverage Diff           @@
##           master    #102   +/-   ##
======================================
  Coverage    58.1%   58.1%
======================================
  Files          21      21
  Lines        2270    2270
======================================
  Hits         1319    1319
  Misses        807     807
  Partials      144     144

Thanks! This breaks our exported models, but we do need to use EntityID instead of EntityId.


Apply versioning to Options topic

Fixes #7504
@HaoK I'm applying versioning (moniker ranges) in place of text references to versions. It's probably a good time for you to double-check and see if the versions mentioned are correct.

@scottaddie I think @HaoK is out. This is just converting tabs over to versioning. Let's go ahead with it.


Setup

Added setup.py file, moved code into the src directory (for packaging). To try it out run pip3 install . in the base directory.

Thanks for this contribution! As you can probably tell I am not an expert in these sorts of things, so it is much appreciated!


Issue with provisioning a site with OfficedevPnp since Friday June 17th

Hi all,
We deploy certain SharePoint Online sites with the help of OfficedevPnp. This always worked fine and last Thursday we have created several sites with it. On Friday however something probably changed we now get an error.
We use the schema's of OfficeDev from august 2015 so maybe it's just outdated but would like the confirmation that installing OfficedevPnp actually does solve the issue we are experiencing since friday.
The error we get starts with [List instances Data Rows] [1] [Error] Creating listitem failed: exception from HRESULT: 0x80131904: at Microsoft.SharePoint.Client.ClientRequest.ProcessResponseStream(Stream responseStream)......
And gives:
"Error during provisioning of template Lists.xml - Error
Anybody any experience with this or does anyone knows it something changed on SharePoint Online which causes OfficeDevPnP to fail?
Like to know more.

+1 from me. We too are experiencing the same issue that started occurring about the same time.

@kbarret3  which version of devPnP are you using? also an older version ?

Hi @angelanouwens. Yes my code base was I think on the first release that included the capability to provision a site based on a template site. The OfficeDevPnPCore package is at version 1.3.0615.0

Thanks for your reaction. We are currently very busy working on the issue and probably updaten devPnP as soon as possible to see if this solves the issue.
Will keep you updated when we have found a solution
From: kbarret3<EMAIL_ADDRESS>Sent: vrijdag 24 juni 2016 0:41
To: OfficeDev/PnP
Cc: Nouwens, Angela; Mention
Subject: Re: [OfficeDev/PnP] Issue with provisioning a site with OfficedevPnp since Friday June 17th (#1465)
Hi @angelanouwenshttps://github.com/angelanouwens. Yes my code base was I think on the first release that included the capability to provision a site based on a template site. The OfficeDevPnPCore package is at version 1.3.0615.0
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/OfficeDev/PnP/issues/1465#issuecomment-228207355, or mute the threadhttps://github.com/notifications/unsubscribe/ATJ7QgG6CyAZb_NB8v8PMoN7sed8dgS_ks5qOwuHgaJpZM4I7s9u.
This message contains information that may be privileged or confidential and is the property of the Capgemini Group. It is intended only for the person to whom it is addressed. If you are not the intended recipient, you are not authorized to read, print, retain, copy, disseminate, distribute, or use this message or any part thereof. If you receive this message in error, please notify the sender immediately and delete all copies of this message.

+1 We have the same issue/error. Also the schema from august 2015.

This issue was moved to OfficeDev/PnP-Sites-Core#770


Support plugins configuration in omnisharp.json

Locations for OminSharp’s plugins can currently be configured only via –plugin command line argument. But when OminSharp is activated as a part of ms-vscode.csharp extension, other VSCode extensions cannot use this approach to supply their own plugins for OmniSharp. The only exception are code actions which can be configured in omnisharp.json via RoslynExtensionsOptions. The proposed changes enable OmniSharp to discover any plugin using the same technique via new Plugins section in omnisharp.json, i.e.:
"Plugins": {
    "LocationPaths": [ "path_to_plugin1.dll", "path_to_plugin2.dll" ]
}

An example of VSCode extension that provides plugins via omnisharp.json is Roslynator which does it via RoslynExtensionsOptions.

thanks - overall this is a good idea and a welcome addition.
I am not convinced that this is the best implementation though. I'd rather see types under OmniSharp.Options to be used in a consistent, strongly typed way rather than using the raw IConfigurationRoot to bind to them multiple times in multiple places.
This would mean that we need to refactor the code a bit to perhaps delay the decision about plugin discovery or move the options bootstrapping up.
I will have a closer look when I have some time.

I'm with @filipw I like the idea but the solution feels wrong somehow.  I'm going to have to circle back to this, this weekend after my local code camp.

Good feedback! Switched to using strongly typed object for declaring locations for plugins.

@filipw @david-driscoll Curious what you guys think of the latest changes.

thanks a lot - it will be reviewed carefully as soon as there is some free time.
but after https://github.com/OmniSharp/omnisharp-roslyn/pull/1625 we don't want to rush any changes without thoroughly testing and pulling locally.

@filipw a gentle ping since it's been few weeks. thanks

sorry! I will do my best to review in the coming days! 🙏

any thoughts? @david-driscoll @mholo65 @JoeRobich
if not IMHO this is good to merge and a very valuable feature

LGTM

thanks!


Systemd notify support

Experimental support for systemd notify
From the discussion in chat

This is not perfect yet, so lets discuss here what is needed for "good enough" level

It uses https://crates.io/crates/sd-notify crate. The way that crate is written I assume we do not even need to hide it behind a feature flag: On a non-systemd (maybe even non Linux) it does no harm, as it determines systemd availability by just checking existence of a file. On a a systemd system, even when starting manually on the commandline, a notify doesn't hurt, systemd will just ignore it (I assume that is probably how all notify-enabled dameons work)

When to signal ready state: I put the code in the last reachable statement in main before the GRPC socket is up. So strictly speaking there is still a race conditions. As at this point all parsing an dinitlaization is done, I assume for practical reasosn it would solve your issue @g-scott-murray    However anyone knows a "better" place to put this code, i.e. any callback/event when GRPC server is up (strictyl speaking VISS also might not be ready yet, if started in a thread before). I did not find anything obvious. Do you have an idea @argerus , what might work preferably without putting too much infrastructure in?

Looks good to me, but I haven't tested it. Would probably still make sense to put it behind a feature flag in order to not add a hard dependency (think compiling for QNX etc).
A possible improvement could be to bind the socket before notifying systemd, e.g.
    let socket_addr = addr.into();
    let listener = TcpListener::bind(socket_addr).await?;

    // Notify systemd here <---

    serve_with_incoming_shutdown(listener, ...)

I moved the notify, so it is fired after bindng the socket.
I did not create an "explicit" feature flag, instead I used an existing flag, and now only build this when building for a Linux OS. I verified that the code is not included when building for mac OS.
if running on a Linux without systemd, this has no negative effect (the code detects whether systemd is in use or not). As the create is very small and includes zero dependencies, I figured not much would be gained by hiding it behind an explicit user-configurable flag.
-> Works for me, ready to review/merge fmpov


gflags change (cc3d12b) fails sophus installation

vcpkg install sophus yields
ninja: error: 'C:/vcpkg/installed/x64-windows/debug/lib/gflags.lib', needed by 'test/ceres/test_ceres_se3.exe', missing and no known rule to make it

Related #3045

Thanks for posting!
I'm not able to reproduce this issue; can you list the version + features of ceres, sophus, and gflags that you're installing?

It's vcpkg install ceres[*] and vcpkg install sophus. Not sure if it makes a difference but I am using a custom triplet x64-windows-dev but it should be irrelevant because the custom triplet looks like
set(VCPKG_TARGET_ARCHITECTURE x64)
set(VCPKG_CRT_LINKAGE dynamic)

if(PORT STREQUAL "glew")
  set(VCPKG_LIBRARY_LINKAGE static)
else()
  set(VCPKG_LIBRARY_LINKAGE dynamic)
endif()


node.js 6 fixes

The enclosed changes make assetgraph's systemjs-builder tests work with node.js 6. Previously it failed like this: https://travis-ci.org/assetgraph/assetgraph/jobs/126310648
The configPathCache thing seems a bit weird to me -- when would an object ever have a map method?

packageConfigPaths is supposed to be an array, so this may be caused somewhere else in the process, by passing a non-array into system.config here. Perhaps this is happening during a serialization process of the configuration?
I've also added Node 6 testing to the main branch here which needed another fix as well.

packageConfigPaths is supposed to be an array, so this may be caused somewhere else in the process, by passing a non-array into system.config here. Perhaps this is happening during a serialization process of the configuration?

Doesn't look like any of the tests do that. Hmm...

It seems like you've got this working? Can I close?

Yeah, thanks.  It's clearly working now :)


Update push to use master of chef

This should fix the build failures for push
http://manhattan.ci.chef.co/job/push-jobs-client-build/90/
There is still some work to do because we require 11.12.2 in some places inside of the push client.

@chef/omnibus-maintainers
I'd like to merge this in so that we can get all the builds green again.
This fails the windows build but that looks like a slave failure.

:+1:

:+1:


Retry behavior

Hi,
it seems that when doing a query, if the DNS server does not reply, the query will be automatically retried, either by aiodns or by pycares. Do you know if there is a way to set this behavior?
For instance choose the max number of retries?

You should be able to configure that with a kwarg to the resolver init. Check the pycares docs since we accept those options.


Set application_name for internal connections to computes

This will help when analyzing the origins of connections to a compute like in 0.

Is anyone aware of any other places we might be creating internal connections?

Would it be useful to add a neon/ prefix for these application names? Could help create better SQL queries.

We create connections in compute_ctl. We have a monitor that connects every so often

We should also set the application name for vm-monitor connections:
https://github.com/neondatabase/neon/blob/d25f7e3dd575878df49925bead4c797a61757751/compute_tools/src/bin/compute_ctl.rs#L734-L741


Error: Nesting detected after update to nextjs 7.0.1

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
Nextjs won't start because of multiple errors of "Error: Nesting detected"
If the current behavior is a bug, please provide the steps to reproduce and possibly a minimal demo or testcase in the form of a Next.js app, CodeSandbox URL or similar
Create a component with media query like:
            <style jsx>{`
                .hero {
                    position: relative;
                    padding-bottom: 100vh;

                }
                img {
                        height 100vh;
                        filter: brightness(${this.props.brightness || "50%"});
                }
                @media ${mediaQueries.mobile} {
                    aside {
                        position: absolute;
                        left: 0;
                        right: 0;
                        bottom: ${rem(20)};
                        display: flex;
                        flex: 0 0;
                        overflow-x: auto;
                    }
                }
`}</style>

What is the expected behavior?
Media query shall work
Environment (include versions)

OS: Windows
Browser: Any
styled-jsx (version): 3.1.0

Did this work in previous versions?
With Next 6 && 2.2.6 everything works.

This was false diagnose.
Error was caused by comment block in the CSS


Update Cilium, OpenStack CCM, and Cinder CSI (January 2023)

What this PR does / why we need it:

Update Cilium from v1.12.3 to v1.12.5
Update OpenStack CCM to v1.26.0 for Kubernetes 1.26+ clusters
Update OpenStack Cinder CSI to v1.26.0 for Kubernetes 1.26+ clusters

This also includes its dependencies such as CSI attacher, provisioner...

Remove enabled-by-default API server feature gates for OpenStack

CSIMigrationOpenStack and ExpandCSIVolumes are enabled by default at least since 1.23

Which issue(s) this PR fixes:
xref #2562
What type of PR is this?
/kind feature
Does this PR introduce a user-facing change? Then add your Release Note here:
- Update Cilium from v1.12.3 to v1.12.5
- Update OpenStack CCM to v1.26.0 for Kubernetes 1.26+ clusters
- Update OpenStack Cinder CSI to v1.26.0 for Kubernetes 1.26+ clusters

Documentation:
NONE

/lgtm
/approve


How to enable first and last day of month?

I was wondering if there is option to disable all other days of month but not first and last day.
Is this possible with some minor effort?

Will be closing as a how-to question, it should be asked at StackOverflow


feat: add interactive selection for tasks if task was not found

Adds an interactive task selector as mentioned in #1274.

adds cask inquire
replaces task not found error with interactive selection of available tasks

Did not find an easy way to test this. Not sure if theres an easy way send user input to the console.

wow does it work? it's crazy how little code that was

nice, it works great!

I saw you filed a bug on demand which isn't too surprising, it hasn't been used that much yet but I think it's got a lot fo promise. I wasn't very happy with inquirer

Yeah was surpised too how easy this was. The issue in demand is probably an edge-case which i ran into because i used the terminal in my IDE with aprox. 5 rows height only.


Repo sync for protected CLA branch

The pull request is created from main637956831118304963sync_temp to main to fix git push error for protected CLA branch

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


and suddenly it doesn't work anymore

Using istaloader for a while now in different version and no Problem appeared.
Today i tried to download a Hastag and i get this error:
Traceback (most recent call last):
File "main.py", line 464, in
File "main.py", line 461, in __main
File "main.py", line 431, in main
File "main.py", line 149, in _main
File "instaloader\instaloader.py", line 1023, in download_hashtag
File "instaloader\structures.py", line 1302, in from_name
File "instaloader\structures.py", line 1316, in _obtain_metadata
File "instaloader\structures.py", line 1311, in _query
KeyError: 'graphql'
tried different versions and i figured out that i depends on the login
When i dont login, the downloads start, but after a few you will be redirected to Login page.
-> after a few attempts i cant eben download 1 pic, without being redirected to the login page
Can someone help out whats the matter with the login?
or at least explain whats happening?
using win standalone

you missed important information, at least:

instaloader version
command line you used

concerning the limit with redirection to login, see #1067

tl;dr hashtag scraping should still work if you're not logged in.
I'm currently using 4.6.2 in production, but I've confirmed that this issue does exist in latest version 4.7.1.
/explore/tags/_?__a=1 endpoint has changed for some users.  Looks like IG is rolling out a change.
The expected return format looks like this: {"graphql":{"hashtag":{"id":"17843783215038189","name":"burrito"
This is the new return format: {"data":{"id":17843783215038189,"name":"burrito"
I only see the new format when logged into instagram.  It looks like the entire structure has changed so Hashtag logic will require a refactor.

I come across  the same error KeyError: 'graphql' when running Hashtag.from_name(L.context, "hashtag_name")
I am using instaloader 4.7.1

i also have the same problem! my code used to work and i didn't change anything.. My version is also 4.7.1

I also have the same Error (KeyError: 'graphql') when running: instance.download_hashtag(hashtag="fridakahloinspired",max_count=10) . My code used to work to. My version: 4.7.1

This Hashtag.from_name(L.context, keyword) works for me again if i'm not logged in.. Has something with Instaloader or Instagram changed? Does someone know how much requests with Hashtag.from_name can i do before i'll be redirected to Instagram Login?

@nlsnhrwld Did you find some solutions?

@Lahmidi-Mustapha  Until now there is no solution, i guess. (My expierence was, that I can download like 6 Pictures until I be redirected)
When i unterstand @rick-mmi  right, there has been a structure change on the Instragram-site, when you are logged in. These changes have to be adapted first in instaloader, if this changes can be adapt. I dont know. But i guess

@nlsnhrwld Thanks! i switched to Graph Api from Instagram and it works pretty well and is also stable

@Lahmidi-Mustapha You need to have a business account for the Instagram API, dont you? and your App also has to be reviewed by facebook, is that right? or is that only if you want to have public pictures?

I was teaching a workshop with instaloader this morning and 4-5 different people in the workshop encountered this exact issue. I think what @rick-mmi said is the issue - e.g. Instagram has changed its format so instaloader needs to change the Hashtag loading logic.

@nlsnhrwld i created an Instagram profile for my Webapp just to retrieve hashtags from there. Then i changed it to a business profile after that i register my personal facebook account as a developper account and from there i could get an Access Token to the Facebook Graph Api. It's works very smooth and fast. I can now make up to 200 request per Hour without getting a too many requests error or putting a sleep() between my requests to avoid them.

Confirming issue on my end as well in both 4.6.2 and 4.7.1 with python 3.6.

Having issue too with 4.7.1

Duplicate of #1080.


Release 0.5.2

0.5.2 (2021-02-08)
Features

changelog:  add sections (ff062cf)
mobile: tip shared url (865302b)

Maintenance

lint: setup pre-commit hook (686ee0c)

Minor build adding the share option to mobile menu.


Update failed: error: pathspec 'origin/master' did not match any file(s) known to git. : exit status 1 on the git.rancher.io

What kind of request is this (question/bug/enhancement/feature request):
Question/Issue
Steps to reproduce (least amount of steps as possible):
From the Rancher UI:
Apps -> Manage Catalogs
Result:
I'm getting error message on any catalog library install from the git.ranchar.io:
Update failed: error: pathspec 'origin/master' did not match any file(s) known to git. : exit status 1

Other details that may be helpful:
2020/08/12 16:32:17 [ERROR] CatalogController helm3-library [catalog] failed with : Update failed: error: pathspec 'origin/master' did not match any file(s) known to git.
: exit status 1
2020/08/12 16:32:17 [ERROR] CatalogController system-library [catalog] failed with : Update failed: error: pathspec 'origin/release-v2.4' did not match any file(s) known to git.
: exit status 1, [system-image-upgrade-catalog-controller] failed with : upgrade cluster c-chx55 system service alerting failed: get template cattle-global-data:system-library-rancher-monitoring failed, catalogTemplate.management.cattle.io "cattle-global-data/system-library-rancher-monitoring" not found
2020/08/12 16:32:17 [ERROR] CatalogController library [catalog] failed with : Update failed: error: pathspec 'origin/master' did not match any file(s) known to git.
 Does your rancher run behind a proxy or something like that?

+1 same error

I've had issues with those repos too. I just switched to getting them directly from GitHub.
For https://git.rancher.io/helm3-charts use https://github.com/rancher/helm3-charts
For https://git.rancher.io/charts use https://github.com/rancher/charts
For https://git.rancher.io/system-charts use https://github.com/rancher/system-charts


Update version to 2.2

Opening this PR just want to double confirm I won't miss something for updating version.

Handled in #1222


Added user-relevant info summarization to HTML

Additional fields are added to the generated HTML web page:

archive name
timestamp when the run was initiated
execution duration
final verdict, if all the checks passed or some problems were detected

Closing as this is already covered in #19.


Bump vsoch/pull-request-action version

Description
describe the purpose of the change here
Resolves <issue nr here>
PR Checklist

[ ] Tests added
[ ] Changelog updated

trying to solve release branch build issue


CloudFrontResult needs to be exported?

What version of Hono are you using?
4.3.10
What runtime/platform is your app running on?
Lambda@Edge
What steps can reproduce the bug?
The linter is failing just following the tutorial. It's having an issue with the return type of the handle function (CloudFrontResult).

I think CloudFrontResult may just need to be exported.
What is the expected behavior?
No lint errors from the basic tutorial.
What do you see instead?
Lint error on handle function.
Additional information
No response

What is your VSCode TypeScript version?
Or try 5.4.5 and see if the bug still exists

@ronster37
Hmm..yes, strage. I think this is not a Node.js matter but a TypeScript problem. Can you try to create the project with the following command again?
npm create hono@latest -- --template lambda-edge

Yes, that worked. The tsconfig.json file differs greatly from the one generated by following the documentation. Because of this, I was able to locate the issue.
Here is the tsconfig.json generated using cdk init app -l typescript:
  "compilerOptions": {
    "target": "ES2020",
    "module": "commonjs",
    "lib": [
      "es2020",
      "dom"
    ],
    "declaration": true,
    "strict": true,
    "noImplicitAny": true,
    "strictNullChecks": true,
    "noImplicitThis": true,
    "alwaysStrict": true,
    "noUnusedLocals": false,
    "noUnusedParameters": false,
    "noImplicitReturns": true,
    "noFallthroughCasesInSwitch": false,
    "inlineSourceMap": true,
    "inlineSources": true,
    "experimentalDecorators": true,
    "strictPropertyInitialization": false,
    "typeRoots": [
      "./node_modules/@types"
    ]
  },
  "exclude": [
    "node_modules",
    "cdk.out"
  ]
}

Removing (or setting to false) the declaration attribute solves the issue. I added "skipLibChecks" and "skipDefaultLibCheck" but it had no effect.
I think removing it might be ok since I will not be building a library.

@ronster37
Thank you for sharing the tsconfig.json! But it is weird. It does not show the error in my environment with your config.

I think removing it might be ok since I will not be building a library.

Yeah. It may be so.

I'm going to close this out for now.
For anyone else who encounters this remove "declaration": true fixes this problem.


fix: don't use any in signature of the mxGraphModel filter functions

The filters only apply to mxCell so simplify the signature of filter functions and remove any.
Rationale

filterCells: https://github.com/jgraph/mxgraph/blob/v4.2.2/javascript/src/js/model/mxGraphModel.js#L361-L382
filterDescendants: https://github.com/jgraph/mxgraph/blob/v4.2.2/javascript/src/js/model/mxGraphModel.js#L401-L450

LGTM +1


Fetched logs are sent to third party infrastructure for long-term storage

AS A Participant
I WANT TO store logs in a third-party storage
SO THAT I could maintain a  needed backup of logs for when it may be needed

Logs will be sent to a third-party infrastructure storage/database according to the internal policy


Bug Report ---- erors !!!!

Bug Report ---- erors !!!!
wallet.log
Environment

Sia version: -----  1.3.0
OS:  ---- Win 10 64-bit

Sia roadmap

host.log
gateway.log
siad-output.log

Please dear developers fix these problems! .... or explain how to fix them yourself!

This looks like a duplicate of #1605.
Recommendation to devs: Close as duplicate.


Get permission to share Wasabi credentials

Before getting on a soapbox about this, make sure The Allen is ok with me sharing the credentials to their Wasabi storage.

Email sent to Rachel Dalley on 2019-11-15


Adding functions to convert units

Has there been debate about adding in a function to convert units? I am not sure if you would want to just have a function that just converts to SI unit or allow people to select units, but it would be nice to have and maybe both. If it just want to SI units then you could theoretically convert a column of different units which would be very nice. I am not sure if this would live in admiral or not. Would you ever convert units going from the eCRF to SDTM? Or only SDTM to ADaM?
@pharmaverse/core-team
@pharmaverse/admiral

The unit conversions are normally done at SDTM creation, not usually when creating ADaM. Its a lot of work to maintain these conversions, as there can also be a lot of synonyms.
In saying that, in some of the lab grading the grade criteria can be in US/CV units and I've had to convert to SI unit, but this is rare.

We convert units for deriving the age grouping variables (derive_var_age_years()). We discussed implementing unit conversion for derived parameters like BMI or BSA and decided not to implement it because of the complexity.
I think a unit conversion function would be useful (for SDTM and ADaM) but as Gordon pointed out it is a lot of work to implement and maintain it.

sounds like a good potential for a new dedicated pharmaverse package. will raise with council reps. we're still figuring out how best to advertise such gaps to allow individuals or companies to register interest to help develop

There must already be a package on CRAN which does this? For example: https://cran.r-project.org/web/packages/units/vignettes/measurement_units_in_R.html

FYI all interested - when Oak (SDTM R package from Roche) open sources later in the year there are some functions for this:
oak_import_lab_conversion_factors.R, oak_import_lab_unit_thesaurus.R, oak_merge_lab_with_conversion_factors.R, oak_merge_lab_with_unit_thesaurus.R.
A question we can then ask down the line is whether beneficial for other pharmaverse packages if we split these into their own dedicated package. It won't be highest priority for that team but as I say something for down the line.


fix(tsconfig.json): add DOM.Iterable to compilerOptions of tsconfig.json

What is it?

[ ] Feature / enhancement
[X] Bug
[ ] Docs / tests

Description
resolves issue: #1344
Use cases and why

Expected Behaviour
Code example:
export const onPost: RequestHandler = async ({ request }) => {
  const payload = Object.fromEntries(await request.formData());
}

Actual Behaviour
TypeScript warning:
  Overload 1 of 2, '(entries: Iterable<readonly [PropertyKey, any]>): { [k: string]: any; }', gave the following error.
    Argument of type 'FormData' is not assignable to parameter of type 'Iterable<readonly [PropertyKey, any]>'.
      Property '[Symbol.iterator]' is missing in type 'FormData' but required in type 'Iterable<readonly [PropertyKey, any]>'.
  Overload 2 of 2, '(entries: Iterable<readonly any[]>): any', gave the following error.
    Argument of type 'FormData' is not assignable to parameter of type 'Iterable<readonly any[]>'.
      Property '[Symbol.iterator]' is missing in type 'FormData' but required in type 'Iterable<readonly any[]>'.

Checklist:

[x] My code follows the developer guidelines of this project
[x] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation
[ ] Added new tests to cover the fix / functionality

Seems like lint ci is failing probably because your IDE changed the identation of the file?

Seems like lint ci is failing probably because your IDE changed the identation of the file?

@manucorporat, line ending got changed from LF to CRLF.
You can add prettier fix to lint-staged, so when commiting it will run prettier hook or else add gitattributes file.
I have updated the file with LF, also ran prettier fix, could you please rerun the workflows?

Great suggestion @OmkarKirpan ! thanks
Want to add a PR for that?

Great suggestion @OmkarKirpan ! thanks
Want to add a separate PR for that?

Sure, I'll do that.


About touch

toreton from 4pda write me for know how touch can solve. Here's my answer for him:
It all depends on device model and firmware: Global or other location.
_But if the files (listed below) from the /vendor/firmware folder of both devices are the same and compatible, then add the files in the /vendor/firmware folder:
novatek_ts_fw.bin
novatek_ts_fw_jdi.bin
novatek_ts_fw_tm.bin
Recompile the TWRP. Touch screen test._

Thanks for the suggestion.
According to the stock kernel config for 8 5G, I see two touch panels — from ilitek and novatek. I don't know if there are more...
Firmwares are available for above two in /vendor/firmware/tp/
Adding to this recovery's ramdisk will fix it definitely.

Didn't work...

Hi everyone. I need help. I would like to install TWRP but I'm using Windows and with adb commands I can't send the phone to fastboot mode. Can you help me?

Hi everyone. I need help. I would like to install TWRP but I'm using Windows and with adb commands I can't send the phone to fastboot mode. Can you help me?

Follow my guide:
https://forum.xda-developers.com/t/a-guide-to-unlock-the-bootloader.4306487/
Please read it.

Thanx bru 😁

Fixed by 9f1e5d56baefcee1068453a0313acf3b3a9f9e5c


AuthorizeRouteView, DefaultLayout can take dynamic type but does not work as expected

Describe the bug
The AuthorizeRouteView is not using the correct default layout
To Reproduce

Expected behavior
After login, navigate to the counter page should use MainLayout instead of LoginLayout.
@danroth27  did I do sth wrong here?

@jingzheshan thanks for contacting us.
I'm not able to repro this. Could you provide a minimal repro project so that we can help you identify the issue?

On second look, I believe your issue is that the CascadingAuthenticationState must wrap the entire component. Here is a simplified sample project that shows this working.
dynamicauthlayout.zip

@javiercn thanks for your time and investigation.
Here is a demo sln you can try.
demo.zip
I have mocked the code as close as to yours, hope you can reproduce the issue. The main difference right now I think I am using Blazor WebAssembly client-side instead of server-side. BTW. I could confirm your solution works as I expect. 👍

@jingzheshan For some reason the cascading authentication state doesn't get updated.
@SteveSandersonMS Can you take a look here? I can't see what's wrong.

There's a bit too much going on in demo.zip for me to narrow in on any specific thing causing a problem.
@jingzheshan I would recommend that you keep comparing your implementation against @javiercn's until you track down what specific difference means that his works and yours doesn't. If at that stage you think it's a bug in the framework, please describe which specific API call is producing the wrong result.

We are closing this issue for the time being. If as @SteveSandersonMS mentions you end up tracking down the root cause and you think it's a bug, feel free to comment on the issue and we'll re-evaluate.
Thanks!


removed vendor from repo

You should not have vendor folder in your repo, as composer auto generates it.
Also added it to git ignore.

OK. Hope the project root still have composer.json


What's wrong in my train model?

I trained my model using to detect ship.
And get the following results:

something wrong about my training data？

How many images were in your  train folder during training?
What was your minimum_percentage_probability during detection?

* How many images were in your ` train` folder during training?

* What was your `minimum_percentage_probability` during detection?

1.about 300
2.I knew I can set minimum_percentage_probability during detection, but as you see, some rectangle percent is so close, and I just make it default 30.

* How many images were in your ` train` folder during training?

* What was your `minimum_percentage_probability` during detection?

btw, do i need to make each pic just one target?
some of my pics is like the following and i user the labelImg mark them all:

Hello layer master, how do you implement the segmentation algorithm on your own data set, do you have a email, i want to learn something from you .@xiluzi

Hello layer master, how do you implement the segmentation algorithm on your own data set, do you have a email, i want to learn something from you .@xiluzi

Are you chinese? My mail is<EMAIL_ADDRESS>

@xiluzi 300 images seems good for a single class. However, you have to ensure you pick a well trained model to achieve good accuracy.

The model you used for the above result is from what epoch/experiment?

@xiluzi 300 images seems good for a single class. However, you have to ensure you pick a well trained model to achieve good accuracy.
* The model you used for the above result is from what epoch/experiment?

hi, the result is from above 100 epoch. If you don't mind, i upload my sample pictures in the following url.
https://github.com/xiluzi/training_demo


Add a graphic preview for each concepts

It would be interesting to see what is apps for teams, tabs, bot, on this page ? So that the developer can explore the capabilities

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

ID: 38f8810b-ac90-1258-a025-0fb37dc7f48d
Version Independent ID: 807f4d11-6331-17db-ee65-a9f9ccca65da
Content: Concepts - Teams
Content Source: msteams-platform/concepts/concepts-overview.md
Product: msteams
GitHub Login: @o365devx
Microsoft Alias: o365devx

@NormSohl any ideas?

The reader would usually follow the links in the first column for that information. This is just a navigational topic. We could add more conceptual material here, but the risk is that it would become cumbersome and difficult to manage.  The links don't work in the github version of the docs, however they work just fine in the published version.


Store cached data in XDG-defined paths, not ~/.clippy

A lot of linux users really dislike random apps cluttering ~/.foo directories; It's recommended to use XDG paths instead.
https://standards.freedesktop.org/basedir-spec/basedir-spec-latest.html
Clippy would use $XDG_CACHE_HOME/clippy, defaulting to ~/.cache/clippy/

Didn't know about that, thanks :) I suppose ~/.cache/clippy would be a good default for macos too.
You don't know what's the convention on windows, by chance?
And maybe you'd like to try a PR? This would probably be a change to the default in ClippyPlugin

Windows would probably be %APPDATA_LOCAL% iirc but I don't use it myself so
I'm not sure. If I have time I can pr in. Likely I'll make it check the old
location and move the files to avoid extra bandwidth usage if I have time
to write it
On Tue, Dec 20, 2016, 3:59 AM Adam Warski<EMAIL_ADDRESS>wrote:

Didn't know about that, thanks :) I suppose ~/.cache/clippy would be a
good default for macos too.
You don't know what's the convention on windows, by chance?
And maybe you'd like to try a PR? This would probably be a change to the
default in ClippyPlugin
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/softwaremill/scala-clippy/issues/29#issuecomment-268190078,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAA5NLpSMF98BOW-R9MtgWe7dtkQYw14ks5rJ5kFgaJpZM4LQvSw
.

If you would find the time, that would be great :)
Not sure if it's worth moving from the old location, as the gzipped file is now about 2KB

I'm not sure how best to handle windows. Right now it's using the user.home java property, but I'd need to switch to checking environment variables. I suppose I could check for the windows one along with the xdg ones, it just seems a little unfortunate that I'll have to effectively add code path branches vs what it's doing now

Well, if different OSes have different conventions for cache-like local storage, then adding conditional code wouldn't seem wrong in any way to me :) Maybe this could be separate object?


Can you support sanic?

This is a great library.
i wish it support sanic!

This library is for tornado, so it likely won't support Sanic. But a sanic-sqlalchemy would be interesting.


Dependency - requests

1inch.py is looking for the METADATA file in requests  Lib\site-packages\requests-2.28.1.dist-info
requests-2.25.1.dist-info has the METADATA file but 2.28 does not.
Workaround
pip install 1inch.py --ignore-installed requests

Would you mind submitting a PR to fix it? Don't think the build needs 2.28 particularly :) @snub-fighter


installation issue

hello! I'm trying to install pivpn but when it ask to choose a user when i hit OK (with pi user) installation exit an report this error
::: Your system is up to date! Continuing with PiVPN installation...
:::    Checking for git... already installed!
:::    Checking for tar... already installed!
:::    Checking for wget... already installed!
:::    Checking for curl... already installed!
:::    Checking for grep... already installed!
:::    Checking for dnsutils... already installed!
:::    Checking for whiptail... already installed!
:::    Checking for net-tools... already installed!
:::    Checking for bsdmainutils... already installed!
:::    Checking for dhcpcd5... already installed!
:::    Checking for iptables-persistent... already installed!
::: Static IP already configured.
::: Using User: pi
:::
::: Checking for existing base files...
:::    Checking /usr/local/src/pivpn is a repo...:::     Updating repo in /usr/local/src/pivpn...main: line 967: cd: /usr/local/src/pivpn: No such file or directory
I try to flash sd card with clean installation but nothing change. any suggestion? thanks

Every week the same issue. Do you guys even read the FAQ or at least existing issues to look for a solution?


flux platform as a switch

Description:
Flux Switch for Home-Assistant.
The idea for this component was inspired by:
https://community.home-assistant.io/t/circadian-rythm-for-lights/685/17
https://github.com/KpaBap/hue-flux/blob/master/hue-flux.py
This component will change the temperature of your lights similar to
the way flux works on your computer.  When the switch is on, it will change your lights every 30 seconds.  The light color is determined by the 3 colortemp values, start_colortemp, sunset_colortemp, and stop_colortemp, and the start_time and stop_time.  See code for more detail.
You must give a list of light entities to control in the
configuration.yaml.
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#481
Example entry for configuration.yaml (if applicable):
switch:
  platform: flux
  name: Fluxer                # optional, default "Flux"
  lights:
    - light.desk
    - light.lamp
  start_time: "7:00"          # optional, default to sunrise time
  stop_time: "22:00"          # optional, default 22:00
  start_colortemp: 4000       # optional, default 4000
  sunset_colortemp: 3000      # optional, default 3000
  stop_colortemp: 1900        # optional, default 1900
  brightness: 100             # optional, default 90

Checklist:
If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

This depends on home-assistant/home-assistant#2095
Still working on test cases

I tried to do it like this but the remove_listener doesn't seem to be working.  I don't get any errors.  See anything wrong?
    def turn_on(self, **kwargs):
        """Turn on flux."""
        self._state = True
        self.update_ha_state()
        self.flux_update()
        self.tracker = track_time_change(self.hass, self.flux_update, second=[0, 30])

    def turn_off(self, **kwargs):
        """Turn off flux."""
        self._state = False
        self.update_ha_state()
        self.hass.bus.remove_listener(EVENT_TIME_CHANGED, self.tracker)

That should work
On Tue, May 17, 2016, 18:11 Nolan Gilley<EMAIL_ADDRESS>wrote:

I tried to do it like this but the remove_listener doesn't seem to be
working. I don't get any errors. See anything wrong?
def turn_on(self, **kwargs):
    """Turn on flux."""
    self._state = True
    self.update_ha_state()
    self.flux_update()
    self.tracker = track_time_change(self.hass, self.flux_update, second=[0, 30])

def turn_off(self, **kwargs):
    """Turn off flux."""
    self._state = False
    self.update_ha_state()
    self.hass.bus.remove_listener(EVENT_TIME_CHANGED, self.tracker)

—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/home-assistant/home-assistant/pull/2097#issuecomment-219899149

I thought so too.  I was about to push the code so you could look at the whole file when I noticed this pylint error:
E:121, 8: Assigning to function call which doesn't return (assignment-from-no-return)
from this line:
self.tracker = track_time_change(self.hass, self.flux_update, second=[0, 30])

Ah track time change has a bug it should return but doesn't. I thought I
fixed it. Use time changed UTC for now
On Tue, May 17, 2016, 18:52 Nolan Gilley<EMAIL_ADDRESS>wrote:

I thought so too. I was about to push the code so you could look at the
whole file when I noticed this pylint error:
E:121, 8: Assigning to function call which doesn't return
(assignment-from-no-return)
from this line:
self.tracker = track_time_change(self.hass, self.flux_update, second=[0,
30])
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/home-assistant/home-assistant/pull/2097#issuecomment-219904290

I think I need some guidance writing tests. If I execute _setup_component, how can I then get the Entity that it creates?
I'm thinking I should setup the component with _setup_component, then get the created Entity somehow and have it run flux_update.  Then, I would check the new x,y values based on the time and start/stop variables.  Am I on the right track?

You should not need access to the entity, instead you need to fire fake
time changed events that trigger your time listeners that get attached when
switch is turned on.
On Wed, May 18, 2016, 12:39 Nolan Gilley<EMAIL_ADDRESS>wrote:

I think I need some guidance writing tests. If I execute _setup_component,
how can I then get the Entity that it creates?
I'm thinking I should setup the component with _setup_component, then get
the created Entity somehow and have it run flux_update. Then, I would
check the new x,y values based on the time and start/stop variables. Am I
on the right track?
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/home-assistant/home-assistant/pull/2097#issuecomment-220135386

Ok thanks, I'll give that a shot.  I don't know what I was thinking with the last validation commit. I'll go back to the right way.

Still struggling with the tests.
I was able to create the 3 test lights and I can fire time events, but I'm not sure how to get the flux switch working :disappointed:

So a simple sanity test would be:

Turn 1 light on, 1 light off
Set sun state to set at 5pm
Turn Flux switch off
fire time event for 5.30pm
Test lights did not change
Turn Flux switch on
Test light that is on changed

Do the same for sunrise (change times)

Looks good! :dolphin:


Add github release actions - mkdocs and helm on gh-pages

This pull request defines workflow for publishing basic documentation and helm charts on github pages.
There are additional steps to be taken before merging:

Create an orphaned branch gh-pages with command git switch --orphan gh-pages
Make initial commit on this branch and push it to the remote repo
Setup github pages to be hosted on the root of this branch (Settings -> Pages -> Branch -> gh-pages / (root) )

In the effect helm repo will be available to clients under path gh-pages_url/charts
And documentation generated from mkdocs will be in gh-pages-url/docs
Check my fork to see how it works in action: https://github.com/BbqGamer/ncm-issuer
To see docs enter: https://bbqgamer.github.io/ncm-issuer/docs
To sync helm repo run: helm repo add https://bbqgamer.github.io/ncm-issuer/charts

LGTM


Please invite me to the community

Name
MD ALAMIN A.
Discord Username (if applicable)
No response
Additional Context
This community is interesting, and I love it. I would like to join in order to improve and contribute to more open source projects, as I am a beginner at contributing to open source projects. I also want to join in order to meet more filipino devs and help each other!

@Alamin0x01

@Alamin0x01

@Alamin0x01

@

@Alamin0x01


Fix setting of hiscore stats after combat level

result.getSkill(null) throws an IllegalArgumentException, which meant the hiscores were never set beyond combat level.
Introduced in https://github.com/runelite/runelite/pull/3035/files#diff-63f5372602c8f4a5d59848b8189532adR467

Hang on, not a fan of this flow actually as it's not clear what's happening (previous if-elseif didn't help). Will see if I can rework it into a simpler flow.

Okay, reworked the flow so it's more similar to how it was before, as the else if introduced some confusion. Given that 'combat' is a pretty special case (in that it's not actually an actual lookup, but a derivation), I've left it in the if block. The else block now contains all logic for the 'normal' hiscore skills, which do follow a proper lookup flow.

Tested and works. Weird that cb/total/bh/clues are treated as skills!

It is really weird, I spent way too much time trying to understand the code, tried leaving comments wherever I could.


Linux link failure when using python 3.5

current link errors are like
[ 27%] Linking CXX executable gadgetron_test_python

/usr/lib/gcc/x86_64-linux-gnu/5/../../../x86_64-linux-gnu/libboost_python.so: undefined reference to `PyString_InternFromString'

This is with USE_SYSTEM_BOOST=On but using a virtualenv for Python 3.5 (Python 2.7 works). It seems Gadgetron needs to link with libboost_python-py35.so.

possibly a gadgetron issue? Master's still failing: https://travis-ci.org/CCPPETMR/SIRF-SuperBuild
I've also just added https://travis-ci.org/CCPPETMR/SIRF (PR: https://github.com/CCPPETMR/SIRF/pull/69) which tried to use SuperBuild:master and SuperBuild:v0.9.2 to build SIRF:coverage

This gadgetron linking error will disappear onc we update its tag to later than the merge with https://github.com/gadgetron/gadgetron/pull/552, i.e. at least https://github.com/gadgetron/gadgetron/commit/21d584927ee35c8d351a0bd72284b472da555d57. However, I see a few gadgetron bug fixes later which might be useful for us, e.g. https://github.com/gadgetron/gadgetron/commit/6a0482e0cbcf5bfea3a55fdd110b485ad0a9bdf0 and https://github.com/gadgetron/gadgetron/commit/e7eb430673eb3272e8a821b51750c0a2a96dafed.
@evgueni-ovtchinnikov, can you update version_config.cmake to use the latter and run the tests?

successfully tested with the gadgetron commit e23da883867819947fe16ca6ab9bf467079b7369 - is it recent enough?

as I said above  (but rather unclearly!), please test with https://github.com/gadgetron/gadgetron/commit/e7eb430673eb3272e8a821b51750c0a2a96dafed

still failing: https://travis-ci.org/CCPPETMR/SIRF/builds/311558374

All PET errors seem to be of the form
Command: "/home/travis/virtualenv/python3.5.3/bin/python" "test1.py"
Directory: /home/travis/build/CCPPETMR/SIRF-SuperBuild/SIRF/src/xSTIR/pSTIR/tests/
"PET_TEST1" start time: Dec 04 22:57 UTC
Output:
----------------------------------------------------------
/home/travis/virtualenv/python3.5.3/bin/python: can't open file 'test1.py': [Errno 2] No such file or directory

That's of course because you switched to nosetest  but didn't adapt the ctest lines yet (the nosetests at the end do work ok.
There's one MR test that fails
"MR_TEST3" start time: Dec 04 22:57 UTC
Output:
----------------------------------------------------------
Exception ignored in: <bound method pTest.__del__ of <pUtilities.pTest object at 0x7fb2982364a8>>
Traceback (most recent call last):
  File "/home/travis/build/CCPPETMR/SIRF-SuperBuild/INSTALL/python/pUtilities.py", line 216, in __del__
    raise ValueError("%d failures" % self.failed)
ValueError: 13 failures
ismrmrd-python-tools not installed

The exception worries me (we've had a discussion on something that cannot be caught before), can't remember where). @evgueni-ovtchinnikov does this test need ismrmrd-python-tools?

successfully tested with the gadgetron commit e23da883867819947fe16ca6ab9bf467079b7369 - is it recent enough?

"MR_TEST3" start time: Dec 04 22:57 UTC
Output:
----------------------------------------------------------
Exception ignored in: <bound method pTest.__del__ of <pUtilities.pTest object at 0x7fb2982364a8>>
Traceback (most recent call last):
  File "/home/travis/build/CCPPETMR/SIRF-SuperBuild/INSTALL/python/pUtilities.py", line 216, in __del__
    raise ValueError("%d failures" % self.failed)
ValueError: 13 failures
ismrmrd-python-tools not installed

This is very puzzling.
For one thing, my line 216 in pUtilities.py is
expected = self.data[self.ntest]
Furthermore, I could not find the line
raise ValueError("%d failures" % self.failed)
in any of my files.
As for
ismrmrd-python-tools not installed
it is not an error but just warning - I will move it to its proper place so that it does not pop up every time pGadgetron is imported.

as I said above (but rather unclearly!), please test with gadgetron/gadgetron@e7eb430

Done, all well. Should I push?

yes please

As for
ismrmrd-python-tools not installed
it is not an error but just warning - I will move it to its proper place so that it does not pop up every time pGadgetron is imported.

Some clarification is needed: this is normally just a warning, which, however, becomes a fatal error if the user's script tries to compute coil sensitivity maps using Inati method - so, I now moved (on devel) the import of ismrmrd-python-tools to CSM calculation method.

        as I said above (but rather unclearly!), please test with gadgetron/gadgetron@e7eb430

    Done, all well. Should I push?

yes please

Done

    As for
    ismrmrd-python-tools not installed
    it is not an error but just warning - I will move it to its proper place so that it does not pop up every time pGadgetron is imported.

Some clarification is needed: this is normally just a warning, which, however, becomes a fatal error if the user's script tries to compute coil sensitivity maps using Inati method - so, I now moved (on devel) the import of ismrmrd-python-tools to CSM calculation method.

Forgot to write: none of the tests uses Inati method for CSM calculations (one demo does, though).

@evgueni-ovtchinnikov can this be closed?


pdf_converter: could not convert string to float

Hi there!
I'm trying to convert pdf files on a mac. I seem to be hitting an error on all of the documents i've attempted to try. Here's the stack trace:
`
INFO:main:Processing PDF file '/Users/bdavey/Desktop/prints/ben.pdf'
INFO:main:Extracting to SVG page 1
INFO:main:Unable to revert mtime: /Library/Fonts
INFO:main:verbs_action: Invalid verb: com.klowner.filter.applytransform.noprefs
InkscapeApplication::close_document: Window vector not empty!
INFO:main:Extracting to SVG page 2
INFO:main:
INFO:main:verbs_action: Invalid verb: com.klowner.filter.applytransform.noprefs
InkscapeApplication::close_document: Window vector not empty!
INFO:main:Extracting to SVG page 3
INFO:main:
INFO:main:verbs_action: Invalid verb: com.klowner.filter.applytransform.noprefs
InkscapeApplication::close_document: Window vector not empty!
INFO:main:Creating Rm Lines file for /Users/bdavey/Desktop/prints/ben/page_1.svg
Traceback (most recent call last):
File "scripts/pdf_converter.py", line 399, in
generate_rmlines_and_upload(out_dir)
File "scripts/pdf_converter.py", line 339, in generate_rmlines_and_upload
rm = RMLines.from_svg(f.open("rb"))
File "/Users/bdavey/dev/remarkable-layers/rmlines/rmobject/lines.py", line 45, in from_svg
return svg_to_rmlines(buffer)
File "/Users/bdavey/dev/remarkable-layers/rmlines/svg.py", line 91, in svg_to_rmlines
layer.extend(svg_path_to_strokes(path.attrib["d"]))
File "/Users/bdavey/dev/remarkable-layers/rmlines/svg.py", line 58, in svg_path_to_strokes
points = svg_path_d_to_points(d)
File "/Users/bdavey/dev/remarkable-layers/rmlines/svg.py", line 31, in svg_path_d_to_points
point = [float(args), point[1]]
ValueError: could not convert string to float: '1.8125 l 0.018555 0.2988285 0.05957 0.2636715 0.09668 0.2348634'
`
inkscape version info:
inkscape --version Inkscape 1.0.1 (c497b03c, 2020-09-10)     Pango version: 1.43.0
Thanks for the amazing work on this!

I'm not sure really. I am also using the same version of Inkscape as you are (albeit on Ubuntu Linux).
It might be the missing applytransforms plugin. I had thought it came bundled with Inkscape 1.0+. Perhaps try installing it manually from their github repo One can confirm it's available with following commands:
$ inkscape --verb-list | grep klowner
com.klowner.filter.applytransform: Apply Transform
com.klowner.filter.applytransform.noprefs: Apply Transform (No preferences)

I reckon that might be it. Seems to be missing from my dist. I'll try rebuilding with the applytransformations plugin.
Thanks for all of the amazing work once again. I'm so excited to be able to select content from pdfs.

can confirm that this works on mac, albeit the applytransformations plugin needs to be installed manually from https://github.com/Klowner/inkscape-applytransforms
Thanks a million for your help!


Exception after deletion of unmodified entity

If an entity will be deleted before making any modifications, the lastModificationData is missing. Because modifiedDate of EntityAuditEvent is mandatory, the createdBy/createdAt-values could be used instead.
2016-03-23 14:16:45.967 DEBUG 5457 --- [Executor-2] d.i.r.w.c.a.AsyncEntityAuditEventWriter  : -------------- Post DELETE audit  --------------
Hibernate: select max(entityaudi0_.commit_version) as col_0_0_ from jhi_entity_audit_event entityaudi0_ where entityaudi0_.entity_type=? and entityaudi0_.entity_id=?
2016-03-23 14:16:45.971 ERROR 5457 --- [Executor-2] d.i.r.w.c.a.AsyncEntityAuditEventWriter  : Exception while persisting audit entity for User{...} error: {}

javax.validation.ConstraintViolationException: Validation failed for classes [domain.EntityAuditEvent] during persist time for groups [javax.validation.groups.Default, ]
List of constraint violations:[
	ConstraintViolationImpl{interpolatedMessage='darf nicht null sein', propertyPath=modifiedDate, rootBeanClass=class domain.EntityAuditEvent, messageTemplate='{javax.validation.constraints.NotNull.message}'}
]
	at org.hibernate.cfg.beanvalidation.BeanValidationEventListener.validate(BeanValidationEventListener.java:160)
	at org.hibernate.cfg.beanvalidation.BeanValidationEventListener.onPreInsert(BeanValidationEventListener.java:95)
´´´

can you check with latest release?

I heck the sourcecode and dit not find any related changes:
EntityAuditEvent.modifiedDate has NotNull Annotation / table has notNullable constraint.
AsyncEntityAuditEventWriter still use getLastModifiedDate() for DELETE-Event. But LastModifiedDate will be only filled by @PreUpdate Events (Spring Data). But an INSERT on Entity creation can not fire such an event.

@gbrehmer would you like to do a PR to fix this?

I checked this and I dont find any issues. during a delete action the modified by and modified date values are set by spring and we read that in the AsyncEntityAuditEventWriter I tried and its working fine.

Closing as I couldn't reproduce. plz update if you have more info

Ok sorry I spoke too fast. This happens only for the user entity since it was not created when this module was added

need to check further

Ok the issue is that the default users are created via liquibase and hence audit info is not captured

this needs to be updated in JHipster

You're right, sorry for any inconvenience.
I didn't saw, that AbstractAuditingEntity set lastModifiedDate initially to Date.now. I was looking for methods calling the setter of lastModifiedDate ;) We are prefilling our database for other entities with liquibase, too.


⚠ gorzdrav.org has degraded performance

In 8ba2a65, gorzdrav.org (https://gorzdrav.org) experienced degraded performance:

HTTP code: 200
Response time: 5505 ms

Resolved: gorzdrav.org performance has improved in d6bba6b.


Recent Transaction Notification Issue

After viewing a transaction the recent transaction notification counter does not delete/decrease.
To Recreate:

Interact with a recent transaction
Back out and see that the notification counter is still there

Functioning as designed. If you think it should do something different please offer some requirements.

Maybe we should just drop the whole notification dot thing? Btw, those blue dots on the wallet are meant to color code with the wallet color; another confusing thing when compared with the sample app.


Use mopidy 2.2 also on master

Use fix from #771 also on master.
Supposed to fix issue #783

With this fix i'm able to use spotify, files from disk are not fixed by this.

@DarthChoco thanks for the update, so this PR fixes at least some parts of #783.
@MiczFlor I suggest to merge this PR although not all issues are fixed.


Minor doc fix in finetuning directory

These are some minor fixes. I'll go ahead and merge them once the tests pass.


[README] Add GitPOAP Badge to Display Number of Minted GitPOAPs for Contributors

Hey all, this PR adds a GitPOAP Badge to the README that displays the number of minted GitPOAPs for this repository by contributors to this repo.
You can see an example of this in our Documentation repository.
This should help would-be contributors as well as existing contributors find out that they will/have received GitPOAPs for their contributions.
CC: @colfax23 @kayla-henrie

Hey @kayla-henrie just circling back to this and noticed it was committed to the main branch. Can you add these changes to the dev branch? I tried switching the branch the PR was submitted to but there were too many changes and would have required a rebase which I can't do from your repo


Develop

Codecov Report

Merging #13 into master will decrease coverage by 48.87%.
The diff coverage is 0%.

@@           Coverage Diff            @@
##           master   #13       +/-   ##
========================================
- Coverage   48.87%    0%   -48.88%
========================================
  Files           1     1
  Lines         133    20      -113
========================================
- Hits           65     0       -65
+ Misses         57    20       -37
+ Partials       11     0       -11

Impacted Files
Coverage Δ

main.go
0% <0%> (-48.88%)
: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 f53b29b...021f4b0. Read the comment docs.


Feature Request: Blacklist for Richtext elements

I was happy to find the whitelister hook for richtext elements, but a bit disappointed that I could not blacklist elements, which is much more important for me having users copying stuff from word and who knows what...
Instead of having a blacklister, it would be also ok to have an option to overwrite the original whitelist, not merge it..
In reality it seems not such a big problem, unless you copy html directly into your chrome dev tools richtext html field, but I want to make sure...
Also a way to replace html elements would be great, for example the <b> and <i> tags from hello js with <strong> and <em>

It looks like it should be possible to 'blacklist' an element with the following code (untested):
def reject_tag(tag):
    tag.unwrap()

@hooks.register('construct_whitelister_element_rules')
def whitelister_element_rules():
    return {
        'h1': reject_tag
    }

You could probably use something similar to replace the tag with another one (although, if I recall correctly, the HTML spec specifically advises that HTML authoring tools shouldn't replace <i> with <em> unless they're absolutely certain that the author intended it as emphasis, rather than, say, a book title.)
I agree we ought to make this easier, though. (I'm surprised that the requirement to remove elements from the whitelist hasn't come up before - I guess most people aren't really on board with the Wagtail philosophy of taking flexibility away from editors :-) ) A couple of easy ways to do that:

provide the above reject_tag method in wagtail.wagtailcore.whitelist, alongside allow_without_attributes
in https://github.com/torchbox/wagtail/blob/ba807e5f45812e85bf83b49bf04d2c796ef63cda/wagtail/wagtailcore/whitelist.py#L136-L144, treat a None or False rule the same way as a KeyError (so that the above code could be simply return {'h1': False}).

I totally agree that it's necessary to be able to change the configuration of the whitelist. The whitelist is the right approach (instead of a blacklist), it just have to be fully configurable.
But I believe it's just the tip of the iceberg. What we actually want are configurable features for rich text editors like @gasman described in https://github.com/torchbox/wagtail/pull/2175#issuecomment-194838184. That means there is a single place where the features (e.g. HTML elements) are configured, which then makes it possible to keep the visible rich text editor buttons, client-side clean up logic (e.g. paste and drag & drop content) and the backend whitelist in sync according to the activated rich text editor features. For a smooth user experience it's important they are in sync, otherwise I could for example paste some markup, it will be displayed in the editor and only after I hit the save button and reload the editor I will be able to see that some of my pasted content was transformed/removed.
Of course, the configurable whitelist is one step in the right direction! 👍

Related: #3736, which implements the concept of rich text 'features'. Incorporating whitelisting logic into those will be the next logical development step.


“Feature Request: Support for PlaceholderAPI Variables in BetterHud Text”

Dear BetterHud plugin author,
Your BetterHud plugin has been incredibly helpful for my creations. It would be even better if it could include the following feature.
Feature suggestion: Please add support for PlaceholderAPI variables, such as %betterhud_text_Hello!%, which would result in the text "Hello." Since your plugin’s text feature is so powerful, I would love to be able to apply it in other areas as well.
Thank you for all the support your plugin has provided!

nice


bug: pre-release builds should show correct version and should link to changelog

Version
v0.17.1701356806
Describe the bug
If you're running a pre-release build:

The sidebar shows the incorrect version:

Expected behavior

Show the correct version (e.g. v0.17.1701356806)
Link to https://github.com/sourcegraph/cody/blob/main/vscode/CHANGELOG.md

Additional context
No response

Marking as P2 since I don't see this on release builds.


type Tool example fails

Description of the bug:
<EMAIL_ADDRESS>OS: Windows 10
Go: 1.22.4
at this line:
res, err := session.SendMessage(ctx, genai.Text("What is the weather like in New York?"))
if err != nil {
log.Fatal(err)
}
returns "googleapi: Error 400: exit status 1"
Actual vs expected behavior:
no error
Any other information you'd like to share?
Confirmed same GOOGLE_API_KEY working on a Python code.
https://colab.research.google.com/github/google/generative-ai-docs/blob/main/site/en/gemini-api/docs/function-calling/python.ipynb

I confirm I can see this too - investigating


Updating Developer Guide with Undo and Redo Commands

Codecov Report
Base: 72.47% // Head: 72.47% // No change to project coverage :thumbsup:

Coverage data is based on head (de53849) compared to base (f7dd6ac).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master      #74   +/-   ##
=========================================
  Coverage     72.47%   72.47%
  Complexity      525      525
=========================================
  Files            89       89
  Lines          1617     1617
  Branches        174      174
=========================================
  Hits           1172     1172
  Misses          380      380
  Partials         65       65

Impacted Files
Coverage Δ

src/main/java/seedu/address/Main.java
0.00% <ø> (ø)

.../seedu/address/logic/parser/AddressBookParser.java
81.81% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Testing /network/clock

Issue Number
#1394
Overview

82b01bc7df06ca89a26ab7c9a4c999bbd00faf86
address that network/clock is not available on Windows at this stage

0f7c759ae15bb505b87a8c9d4039d1e25d9bdb3a
use pendingOnWindows for network/clock integration test

d49e8de165da61c1712bfe7174ee71ab05becdaf
revise log error levels

Comments

bors r+


list-selection _onClick

_onClick(evt) {
  let target = evt.target;
  while (target && target !== this && !target._isListItem) {
    target = target.parentNode;
  }

  if (target.item && target._isListItem) {
    evt.preventDefault();
    evt.stopPropagation();
    this.selectedId = target.item.id;
  }
  this.onClick(evt);
}

I understand that maybe the user that subscribes to onClick event wants to know on which target was called the event. But i think that at least we could send to onClick as second parameter the target found in the while statement. I really don't wont to repeat that iteration in my code.

evt.target: the DOM node that the user clicked upon
this: If accessed via the onClick Mixin, refers to the Conversation List Item
this.item: If accessed via the onClick Mixin, refers to the layer.Conversation instance represented by this List Item.

:+1:


Cannot load the model

Hi, I am unable to load the pytorch model when opening percival chromium browser. When looking at the code, it looks like GetModelPath is looking for a Sq2.json file to be used with fdeep.load_model.
Am i missing something here?

Sorry for the delay. Here it is 9b079c582b602a301e0753283b58d4325982bfb5

thank you!


Remove unused project directories for cert-manager and kube-vip

After migrating to the new org names for cert-manager and kube-vip, we no longer build them from these folders and can just remove them.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm

/approve

/cherrypick release-0.10


Pascal Binding Deconstructor does not work when named other than "ReleaseInstance".

Need to change
w.Writeln ("    FWrapper.ReleaseInstance(self);");
with
w.Writeln ("    FWrapper.%s(self);", componentdefinition.Global.ReleaseMethod);
as it is done in the Cpp bindings.

Thanks for the fix!


subctl leaks log messages

This happens with subctl gather:

Cluster "skitt-1"
Gathering information from cluster "skitt-1"
✓ Gathering service-discovery logs
✓ Found 3 pods matching label selector "component=submariner-lighthouse"
✓ Found 6 pods matching label selector "dns.operator.openshift.io/daemonset-dns=default"
⠈⠱ Gathering service-discovery resources W0303 18:41:54.480010 1402561 warnings.go:67] discovery.k8s.io/v1beta1 EndpointSlice is deprecated in v1.21+, unavailable in v1.25+; use discovery.k8s.io/v1 EndpointSlice
✓ Gathering service-discovery resources
✓ Found 0 serviceexports in namespace ""
✓ Found 0 serviceimports in namespace ""
✓ Found 0 endpointslices by label selector "endpointslice.kubernetes.io/managed-by=lighthouse-agent.submariner.io" in namespace ""
✓ Found 1 configmaps by label selector "component=submariner-lighthouse" in namespace "submariner-operator"
✓ Found 1 configmaps by field selector "metadata.name=dns-default" in namespace "openshift-dns"
✓ Found 0 services by label selector "submariner.io/exportedServiceRef" in namespace ""
⢎⡱ Gathering broker logs W0303 18:41:55.927931 1402561 warnings.go:67] discovery.k8s.io/v1beta1 EndpointSlice is deprecated in v1.21+, unavailable in v1.25+; use discovery.k8s.io/v1 EndpointSlice
✓ Gathering broker resources
✓ Found 2 endpoints in namespace "submariner-k8s-broker"
✓ Found 2 clusters in namespace "submariner-k8s-broker"
✓ Found 0 endpointslices by label selector "endpointslice.kubernetes.io/managed-by=lighthouse-agent.submariner.io" in namespace "submariner-k8s-broker"
✓ Found 0 serviceimports in namespace "submariner-k8s-broker"
✓ Gathering operator resources
✓ Found 1 submariners in namespace "submariner-operator"
✓ Found 1 servicediscoveries in namespace "submariner-operator"
✓ Found 1 deployments by field selector "metadata.name=submariner-operator" in namespace "submariner-operator"
✓ Found 1 daemonsets by label selector "app=submariner-gateway" in namespace "submariner-operator"
✓ Found 1 daemonsets by label selector "app=submariner-routeagent" in namespace "submariner-operator"
✓ Found 0 daemonsets by label selector "app=submariner-globalnet" in namespace "submariner-operator"
✓ Found 0 deployments by label selector "app=submariner-networkplugin-syncer" in namespace "submariner-operator"
✓ Found 1 deployments by label selector "app=submariner-lighthouse-agent" in namespace "submariner-operator"
✓ Found 1 deployments by label selector "app=submariner-lighthouse-coredns" in namespace "submariner-operator"
⚠ Gathering operator logs
✓ Found 1 pods matching label selector "name=submariner-operator"
⚠ Found logs for previous instances of pod submariner-operator-686cfdd6c9-m9dgp
⚠ Gathering connectivity logs
✓ Found 2 pods matching label selector "app=submariner-gateway"
⚠ Found logs for previous instances of pod submariner-gateway-q6jjr
⚠ Found logs for previous instances of pod submariner-gateway-rgctp
✓ Found 7 pods matching label selector "app=submariner-routeagent"
⚠ Found logs for previous instances of pod submariner-routeagent-spdqq
✓ Found 0 pods matching label selector "app=submariner-globalnet"
✓ Found 0 pods matching label selector "app=submariner-networkplugin-syncer"
✓ Gathering connectivity resources
✓ Gathering CNI data from 7 pods matching label selector "app=submariner-routeagent"
✓ Gathering CNI data from 2 pods matching label selector "app=submariner-gateway"
✓ Gathering globalnet data from 0 pods matching label selector "app=submariner-globalnet"
✓ Gathering cable driver data from 2 pods matching label selector "app=submariner-gateway"
✓ Found 2 endpoints in namespace "submariner-operator"
✓ Found 2 clusters in namespace "submariner-operator"
✓ Found 2 gateways in namespace "submariner-operator"
✓ Found 0 clusterglobalegressips in namespace ""
✓ Found 0 globalegressips in namespace ""
✓ Found 0 globalingressips in namespace ""
⢎⡠ Gathering broker logs Files are stored under directory "submariner-20220303174151"

Note “W0303 18:41:55.927931 1402561 warnings.go:67] discovery.k8s.io/v1beta1 EndpointSlice is deprecated in v1.21+, unavailable in v1.25+; use discovery.k8s.io/v1 EndpointSlice” which shouldn’t appear (even if the problem remains).
It also happens with any failure to load a kubecontext:

Mar  3 18:48:39.814: ERROR: Unable to load kubeconfig file kubeconfig for context admin2, this is a non-recoverable error
Mar  3 18:48:39.815: ERROR: loadConfig err: context "admin2" does not exist
Non-recoverable error: context "admin2" does not exist

It also happens with any failure to load a kubecontext:

Mar  3 18:48:39.814: ERROR: Unable to load kubeconfig file kubeconfig for context admin2, this is a non-recoverable error
Mar  3 18:48:39.815: ERROR: loadConfig err: context "admin2" does not exist
Non-recoverable error: context "admin2" does not exist

I'm not seeing it, which command(s) did you run to get that?
[root@d1c8c4ed677b submariner-operator]# ./bin/subctl gather --kubecontexts bubu
Error getting REST Config for cluster: error creating client config: context "bubu" does not exist

subctl version: devel-90e342b5

[root@d1c8c4ed677b submariner-operator]# ./bin/subctl gather --kubecontexts bubu --kubeconfig dudu
Error getting REST Config for cluster: error creating client config: stat dudu: no such file or directory

subctl version: devel-90e342b5

[root@d1c8c4ed677b submariner-operator]# ./bin/subctl gather  --kubeconfig dudu
Error getting REST Config for cluster: error creating kube config: stat dudu: no such file or directory

subctl version: devel-90e342b5

Generally it's problematic, since there's no good way to disable 3rd party libraries from logging from what I can tell.
We could capture the entire stdout during the call perhaps..?
Probably the best solution would be to use v1 API but AFAICT it's only available since K8s 1.21 and we aim to support 1.20 on 0.12

OK after extensive digging around I found https://kubernetes.io/blog/2020/09/03/warnings/#customize-client-handling and am sending a patch to use that, I'll also open an issue to change the API.

Proposed solution output:
[root@0e8f396ef662 submariner-operator]# ./bin/subctl gather --kubecontexts cluster1
Cluster "cluster1"
Gathering information from cluster "cluster1"
 ✓ Gathering connectivity logs
 ✓ Found 1 pods matching label selector "app=submariner-gateway"
 ✓ Found 3 pods matching label selector "app=submariner-routeagent"
 ✓ Found 0 pods matching label selector "app=submariner-globalnet"
 ✓ Found 0 pods matching label selector "app=submariner-networkplugin-syncer"
 ✓ Gathering connectivity resources
 ✓ Gathering CNI data from 3 pods matching label selector "app=submariner-routeagent"
 ✓ Gathering CNI data from 1 pods matching label selector "app=submariner-gateway"
 ✓ Gathering globalnet data from 0 pods matching label selector "app=submariner-globalnet"
 ✓ Gathering cable driver data from 1 pods matching label selector "app=submariner-gateway"
 ✓ Found 2 endpoints in namespace "submariner-operator"
 ✓ Found 2 clusters in namespace "submariner-operator"
 ✓ Found 1 gateways in namespace "submariner-operator"
 ✓ Found 0 clusterglobalegressips in namespace ""
 ✓ Found 0 globalegressips in namespace ""
 ✓ Found 0 globalingressips in namespace ""
 ✓ Gathering service-discovery logs
 ✓ Found 3 pods matching label selector "component=submariner-lighthouse"
 ✓ Found 2 pods matching label selector "k8s-app=kube-dns"
 ✓ Gathering service-discovery resources
 ✓ Found 0 serviceexports in namespace ""
 ✓ Found 0 serviceimports in namespace ""
 ✓ Found 0 endpointslices by label selector "endpointslice.kubernetes.io/managed-by=lighthouse-agent.submariner.io" in namespace ""
 ✓ Found 1 configmaps by label selector "component=submariner-lighthouse" in namespace "submariner-operator"
 ✓ Found 1 configmaps by field selector "metadata.name=coredns" in namespace "kube-system"
 ✓ Found 0 services by label selector "submariner.io/exportedServiceRef" in namespace ""
 ✓ Gathering broker resources
 ✓ Found 2 endpoints in namespace "submariner-k8s-broker"
 ✓ Found 2 clusters in namespace "submariner-k8s-broker"
 ✓ Found 0 endpointslices by label selector "endpointslice.kubernetes.io/managed-by=lighthouse-agent.submariner.io" in namespace "submariner-k8s-broker"
 ✓ Found 0 serviceimports in namespace "submariner-k8s-broker"
 ⚠ Gathering operator logs
 ✓ Found 1 pods matching label selector "name=submariner-operator"
 ⚠ Found logs for previous instances of pod submariner-operator-769c88dbfd-qqgtd
 ✓ Gathering operator resources
 ✓ Found 1 submariners in namespace "submariner-operator"
 ✓ Found 1 servicediscoveries in namespace "submariner-operator"
 ✓ Found 1 deployments by field selector "metadata.name=submariner-operator" in namespace "submariner-operator"
 ✓ Found 1 daemonsets by label selector "app=submariner-gateway" in namespace "submariner-operator"
 ✓ Found 1 daemonsets by label selector "app=submariner-routeagent" in namespace "submariner-operator"
 ✓ Found 0 daemonsets by label selector "app=submariner-globalnet" in namespace "submariner-operator"
 ✓ Found 0 deployments by label selector "app=submariner-networkplugin-syncer" in namespace "submariner-operator"
 ✓ Found 1 deployments by label selector "app=submariner-lighthouse-agent" in namespace "submariner-operator"
 ✓ Found 1 deployments by label selector "app=submariner-lighthouse-coredns" in namespace "submariner-operator"
⢎⠁ Gathering broker logs Files are stored under directory "submariner-20220306154740"

Encountered following Kubernetes warnings while running:
Warning: discovery.k8s.io/v1beta1 EndpointSlice is deprecated in v1.21+, unavailable in v1.25+; use discovery.k8s.io/v1 EndpointSlice

Opened #1911 for switching the API


Reload breaks anything autocomplete-related

Summary
Using Client.reload_extension breaks autocomplete in slash commands.
Reproduction Steps
I'm using Client.reload_extension for reloading a file full of slash commands, however it breaks my autocomplete and therefore it just doesn't work for some reason.
Minimal Reproducible Code
@commands.command
async def reload(ctx, extension):
    client.reload_extension(f"cogs.{extension}")
    await ctx.send("Reloaded!")

@nextcord.slash_command(guild_ids=[123456789], description="Your favorite dogs!")
async def favorite_dogs(interaction, dog: str=nextcord.SlashOption(name="dog", description="Your favorite dog's breed.")):
    await ctx.send("Your favorite dog is {}!".format(dog))

@favorite_dogs.on_autocomplete("dog")
async def favorite_dog_autocomplete(interaction, dog: str):
    if not dog:
        await interaction.response.send_autocomplete(["husky", "idk dog breeds"])
        return
    await interaction.response.send_autocomplete([i for i in ["husky", "idk dog breeds"] if i.lower().startswith(dog.lower())])

Expected Results
Well... for it to work.
Actual Results
It didn't work.
Error: ValueError: Received autocomplete interaction for warn but command isn't found/associated! (it worked before reloading)
Intents
nextcord.Intents.default() and message content.
System Information

Python v3.10.5-final
nextcord v2.0.0-final
aiohttp v3.8.1
system info: Windows 10 10.0.22622

Checklist

[X] I have searched the open issues for duplicates.
[X] I have shown the entire traceback, if possible.
[X] I have removed my token from display, if visible.

Additional Context
No response

This Minimal Reproducible Code appears to be missing self. Are you sure these cogs are setup correctly?

This Minimal Reproducible Code appears to be missing self. Are you sure these cogs are setup correctly?

i wrote the reproducible code by hand, so thats not the problem.
also i just edited it, i forgot to paste the error.

There are no specifics for only a Cog, however, you can use this after you load a cog in order to sync application commands on your bot.
If you wanted to provide more specific params, such as specific guilds or global you can look into this method.
This fixed the issue for my test bot, however, if your reloading your cogs a lot it may pay to look into alternatives as this does require hitting the API.

There are no specifics for only a Cog, however, you can use this after you load a cog in order to sync application commands on your bot. If you wanted to provide more specific params, such as specific guilds or global you can look into this method.
This fixed the issue for my test bot, however, if your reloading your cogs a lot it may pay to look into alternatives as this does require hitting the API.

it fixed the issue for my bot too, but i think this should still be fixed to not cause further problems

Heyo! When a cog is unloaded, all information the bot has about the application commands in that cog is removed from cache, such as what command ID's correspond to what callbacks. When a cog is (re)added to the bot, the bot will have the new application commands added locally, but won't know their command IDs.
In order for the bot to figure out what command IDs should be routed to which callbacks it needs to do at least one HTTP request to Discord for global commands, and at least one request per guild that your bot/cog has at least one application command for. Mix Discord enforcing rate limiting to keep people from hitting their platform too hard, and that puts things in a slightly awkward position.
Either NC would resync your commands for you every time you reload a cog, hitting Discord with HTTP requests behind the scenes, or the bot dev should manually call sync_all_application_commands when their cog(s) are all reloaded. I suppose a kwarg could be added to Client/Bot for syncing behind the scenes, but hmm.

Heyo! When a cog is unloaded, all information the bot has about the application commands in that cog is removed from cache, such as what command ID's correspond to what callbacks. When a cog is (re)added to the bot, the bot will have the new application commands added locally, but won't know their command IDs.
In order for the bot to figure out what command IDs should be routed to which callbacks it needs to do at least one HTTP request to Discord for global commands, and at least one request per guild that your bot/cog has application commands for. Mix Discord enforcing rate limiting to keep people from hitting their platform too hard, and that puts things in a slightly awkward position HTTP-wise.
Either NC would resync your commands for you every time you reload a cog, hitting Discord with HTTP requests behind the scenes, or the bot dev should manually call sync_all_application_commands when their cog(s) are all reloaded. I suppose a kwarg could be added to Client/Bot for syncing behind the scenes, but hmm.

I personally would be all for adding a kwarg to Bot, as there are already lots of requests that happen "behind the scenes" such as deleting unknown commands and syncing on connect. Documentation would have to most likely be edited to explain why (un)load_extension does not sync application commands by itself by default.


Pull request/d6b97610

This simply takes care of the changes in julia syntax that have occurred recently. I've also fixed up the Levinson tests.

Argh I'm missing this part of the patch:
-levinson(A::AbstractToeplitz, b::AbstractVector) = levinson!(A, copy(b), zeros(length(b)))
+levinson(A::SymmetricToeplitz, b::AbstractVector) = levinson!(A.vc, copy(b), zeros(length(b)))

Hi @maximerischard. Thanks for the PR and sorry for not replying earlier. It's been a while since I looked into this package, so I'll have to wrap my head around it again.
I've enabled Travis testing now so could you try to update your PR. Then hopefully, the tests will pass.

@maximerischard I've rebased your changes and pushed them to master so the package should work now on 0.4. Thanks for the fixes.


github_repository_dependabot_security_updates.this[0] resource can't be created

While updating resource after version updates:
Terraform will perform the following actions:

  # github_repository_dependabot_security_updates.this[0] will be created
  + resource "github_repository_dependabot_security_updates" "this" {
      + enabled    = false
      + id         = (known after apply)
      + repository = "<REPOSITORY_NAME>"
    }

Plan: 1 to add, 0 to change, 0 to destroy.

Following error occurs:
github_repository_dependabot_security_updates.this[0]: Creating...
╷
│ Error: DELETE https://api.github.com/repos/<ORGANIZATION>/<REPOSITORY_NAME>/automated-security-fixes: 422 Vulnerability alerts must be enabled to configure automated security fixes. []
│
│   with github_repository_dependabot_security_updates.this[0],
│   on main.tf line 278, in resource "github_repository_dependabot_security_updates" "this":
│  278: resource "github_repository_dependabot_security_updates" "this" {
│
╵

Temporary workaround is importing the resource using terraform import github_repository_dependabot_security_updates.this[0] <REPOSITORY_NAME>. But ultimately the resource creation should be skipped if var.vulnerability_alerts is false.


change defalut value of TrieSpec

Codecov Report

Merging #112 into develop will decrease coverage by 0.98%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           develop    #112      +/-   ##
==========================================
- Coverage    35.19%   34.2%   -0.99%
==========================================
  Files          150     150
  Lines        25967   25936      -31
==========================================
- Hits          9138    8872     -266
- Misses       16829   17064     +235

Impacted Files
Coverage Δ

util/src/trie/mod.rs
16.66% <0%> (ø)
:arrow_up:

rlp_derive/src/de.rs
0% <0%> (-20.29%)
:arrow_down:

rlp/src/stream.rs
36.36% <0%> (-1.71%)
: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 dc57141...87be9e0. Read the comment docs.


Fix maven code example to match instructions

Why:
Closes #16562

What's being changed:

Check off the following:

[x] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

@dawinkweon
Thanks so much for opening a PR! I'll get this triaged for review :zap:

Good catch @dawinkweon - thanks for the fix. LGTM.

@ethomson Hi there, could we get an approval to get this in? Thanks, Dawin.

<> Cntrl e

#...Delete...
#16563
#17488
#15325
#17486
#17485

@

...Delete ALL SYSTEM FILES.xls

......
...(https://archiveprogram.github.com/)...

[ ]

[ ]


Feature: Support creating histograms and showing images for dataframes

Glad that on UI side, Spyder 5 came back to Spyder 3 functionalities.
(Spyder 4 was truly a regression..)
Still missing many good features of Spyder 3:
Histogram from Variable Explorer ( ie one can check easily things)
https://docs.spyder-ide.org/3/variableexplorer.html

Hi @pprokevin thank you for the feedback! I think the features mentioned are still available from the Variable Explorer context menu:

The only difference is that their order/placement on the context menu is different but from a quick local check seems like are working as expected 🤔. Checked locally using the latest Spyder 5 (version 5.4.3) and the following example code:
import numpy as np

a = np.random.randint(100, size =(50))
b = np.random.random((16, 16))

Are you having troubles using the context menu actions? What version of Spyder are you currently using?
Any other info to better understand the issue/feature request is greatly appreciated. Let us know!

Thank you for the extra info @pprokevin ! So basically the idea will be to extend the support of the Histogram and Show image actions to dataframes, right?

I created a prototype so that we can discuss what this should look like. Let me know what you think. If there are no reactions, I will turn the prototype into a PR.
I think it makes most sense to create a histogram from the columns of a dataframe. In the video, you can see that when I right-clicking on a cell of a dataframe in the Dataframe Editor and select the new "Histogram" item, Spyder shows a histogram of the column that the cell is in. If multiple cells are selected, the plot has multiple histograms.
I don't think that "show image" on a DataFrame is a common action, so I would propose not to implement this. If we do want to do this, we need to decide how images are normally stored in a data frame?

Some points from the weekly developer call a few days ago:

Wait for PR #20546 which adds a toolbar to the dataframe viewer and add a button there to make the functionality more discoverable
Maybe have a way to use the different pandas functions that make plots in a dropdown way
The plots are not run in the console so they are shown in separate window; see also the new issue #21145 which is about the same problem in the collection editor.


[Design] Avatar Icon not Centered.

Avatar circle has more gap on bottom than top.

Adding margin-top: 7px to [ .LeaderboardContributor__user-avatar ] make it look better:

@officialabdulrehman I don't think this is meant to be centered! If you look at the contributor on position 3, there are more tasks as compared to 1 and 2. This card will expand in height and hence avatar will be placed differently for each contributor. We wanna avoid that.
Let's get @kristykjlee's view on this.