feat: pr files analyzer feature to analyze graphical assets 🤖

Close #4144
Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md

Hey the @MohitGupta121,
Please review this PR and merge this with level3 label, because this is one of the core contribution to this Repository


Error and Reporting Alignment : Phase 1

What kind of change does this PR introduce?
Refactor, RFC Implementation
Did you add tests for your changes?
Not yet needed, but tests were updated to satisfy changes in this PR.
If relevant, link to documentation update:
Not yet needed, but will occur when the validation and deprecation messages are enabled.
Summary
This PR is Phase 1 for changes proposed in RFC #6534. It introduces WebpackErrorArray and implemented a refactor to WebpackError. This is setup required for future work. The use of Errors across the webpack codebase is inconsistent and spread out. Many tests rely on arbitrary tests that differ greatly. It's for those reasons that a phased approach to fulfill the RFC is required.
To be implemented in later phases to follow shortly:

Create proper Error types which extend WebpackError for code using new Error
Enforcement in WebpackError construction
Enforcement of types pushed to WebpackErrorArray
Standardization of toString for WebpackError

Does this PR introduce a breaking change?
Nope
Other information

As mentioned off-Github: This PR is being blocked based on commented-out code within the PR, despite the fact that no guidelines exist within CONTRIBUTING.md which explicitly forbid that practice, and despite the fact that the comments are pertinent to quick-to-follow phases related to and reliant upon the code within the //TODO: blocks, and that the commented-out code is expressive and speaks to intent and guidance for changes to follow.
I understand that disagreements with a maintainer's choices are inevitable, but I disagree with applying arbitrary rules to Pull Requests on a whim. For these reasons I'm closing this PR and abandoning my effort to improve webpack's Error use. This is a net loss for the user experience.


Fix auto version issue

Pull Request Test Coverage Report for Build 100

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 65.979%

Totals

Change from base Build 89:
0.0%

Covered Lines:
64

Relevant Lines:
97

💛  - Coveralls


column ordering - haybaler_taxonomy.py

Hi @poer-sophia
it would be great - having spoken with Jens too - to change the order of the columns in the haybaler_taxonomy.py output.
Specifically, the taxonomy should be inserted in the final columns, after the more important abundance values.
We currently have :
1_AP011540_1_Rothia_mucilaginosa_DY_18_DNA__complete_genome_BAC Rothia  Bacteria;Actinobacteria;Actinomycetia;Micrococcales;Micrococcaceae;Rothia;      2264603.0       59.62   0.0
0.0     2142.86
But it is more readable if we add the columns 2 and 3 to the end of the column list/ df
Rothia  Bacteria;Actinobacteria;Actinomycetia;Micrococcales;Micrococcaceae;Rothia;
What do you think ?

Yes that makes sense

done by Sophia


19 and 29 override default fonts

Original issues: #19, #29
Includes the changes from #26, #30
Instructions to install the release here: https://github.com/jon-dez/tldraw-in-obsidian/releases/tag/issue-19-and-29--2
The video below shows how to use font files that are located in your vault to override the default fonts in tldraw.
https://github.com/user-attachments/assets/23475d47-e534-4701-bef4-d1afbb6c6f97

Closed, included in #33.


SAMLError: Missing entity_id specification

Python package pysaml2 version 4.6.1 will return the error SAMLError: Missing entity_id specification.
Downgrading the pysaml version to 4.0.3 allows the SAML assertion to post successfully.

Same here

This is very useful, thanks everyone!

SAML2 Service Providers should have an entityid, just add:
'entityid': 'https://example.com/sp/metadata',

Or some unique URL on the applications domain (ideally should point to where metadata xml can be retrieved).

Yeah, downgrading the version worked fine !!..

SAML2 Service Providers should have an entityid, just add the following to the settings ( https://github.com/jpf/okta-pysaml2-example/blob/master/app.py#L97 ):
'entityid': 'https://example.com/sp/metadata',

Or some unique URL on the applications domain (ideally should point to where metadata xml can be retrieved).

This worked well for me. thanks.

In case it's unclear, the entityID must match the destination URL / audience URI in the SAML 2.0 settings of the IdP.


fizmo-ncursesw vs. fizmo-sdl2, ncursesw status line truncated

In this test situation of 3 columns in the status line, fizmo-sdl2 seems to render the output correctly - identical to the Inform 7 IDE.  But when run with fizmo-ncursesw or fizmo-console it is heavily truncated no matter how wide the screen is.
Inform 7 source, which can be compiled to .z8 file: https://raw.githubusercontent.com/BroadcastGames/Inform7_StoryCode0/master/storyFilesPile0/StatusLineOutputTest0.ni
Also note that 'wear blindfold' seems to only remove the first status line and not correctly clear the second status line. This bug happens on the sdl2 version and all versions I tested.

In fizmo-ncursesw the status line cannot be re-sized in a “live” manner when the window is resized, as opposed to the regular text output, which is always reformatted as soon as window changes size. The Z-Machine however will detect the new screen size as soon as some input has been processed and the status line should appear correctly after this. fizmo-console doesn't support any upper window and therefore no status line (since the upper window requires the screen to be able to absolutely position the cursor which cannot be dont from “raw” C).
The code which is executed after entering the “wear blindfold” command runs a loop overwriting only the top line with spaces, there is no output for the second line. There may be, of course, other issues which might prevent any such exiting code to run, but at the moment it simply looks as if only one line is actually being cleared.


get_as() always returns true

I'm wondering if I'm using this wrong. This example program:
#include "cpptoml.h"
#include <iostream>
#include <strstream>

int main() {
  std::strstream toml;
  toml << "Int = 64\nBool = false";
  auto parser = cpptoml::parser(toml);
  auto table = parser.parse();
  std::cout << *table << std::endl;
  bool b = *table->get_as<bool>("Bool");
  std::cout << "Bool = " << std::boolalpha << b << " # expect false"
            << std::endl;
}

outputs the following:
Bool = false
Int = 64

Bool = true # expect false

Shoot, I bet the recent commit to allow type conversions in fetching ints is what broke this...

Yep. This should be fixed as of 048ce87ed078f40511f184c9d00b9daafbf8cd5a. This was particularly nasty since the serialization still did the correct thing, and thus the unit tests still all passed (except for the two known failures).
...whoops.

Everything works on my end. Thanks very much for the assistance!


Member profile page dates appear to be American

Describe the bug
I just noticed that the dates on the member profile page appear to be formatted like the American MM/DD/YYYY standard. This should be verified and changed to the Australian / everywhere else standard. Ideally, if not too difficult, this should actually use localisation to format them as per the device's localisation/region setting.

this should be resolved by a recent commit on dev and will be included in the next release.


Other extensions than Lilico not shown in "Connect" widget

Instructions
Since this week we stopped seeing our extension among the available ones for connection despite being included in window.fcl_extensions list. Previously, registered extensions were listed as expected. Seems to be related to a recently merged PR: https://github.com/onflow/fcl-js/pull/1202
Problem
Other extensions than Lilico/Blocto stopped appearing among available extensions when logging in with FCL
Steps to Reproduce
The issue can be reproduced with the extension example app the https://github.com/onflow/wallet-extension-example - the example flow extension won't appear in the harness app, just Blocto (and not sure why not even Lilico shows up).

The culprit is this recent commit in fcl-discovery: https://github.com/onflow/fcl-discovery/commit/8119f74b94353a856d151bedfe40bea841c073e0
From what I can tell it moved the handling of the extensions found in window.fcl_extensions to the server side. Unfortunately, the server side drops any extensions it does not have pre-supplied metadata for here: https://github.com/onflow/fcl-discovery/blob/164cea8cedb97719d243a707aac7969f6c9e916d/data/metadata.json
The specific reason is the filterServicesByPlatform function, which tries to filter services by platform, but it drops any that it can't find the metadata for: https://github.com/onflow/fcl-discovery/blob/164cea8cedb97719d243a707aac7969f6c9e916d/helpers/services.js#L74-L83
A solution for this would be to somehow exclude the window.fcl_extensions services from this check, since it can be assumed that they are supported on the platform. Perhaps this could be done by just changing the order of the filtering pipe.

Thanks @refi93 and @gyorgy-kurucz-vacuum  for catching this. You are correct in that extensions should not require metadata to show. It's a bug. Fix looks good on Discovery. We'll get it merged.


Disable AggregateFailures by default

What is the purpose of this pull request?
Disable AggregateFailures by default and keep AggregateExamples enabled.
According to docs, it had to be enabled explicitly previously.
What changes did you make? (overview)
Disabled AggregateFailures in RuboCop default config.
Is there anything you'd like reviewers to focus on?
AggregateExamples is enabled, so for those who have enabled AggregateFailures, double the offences will be shown, and a deprecation warning regarding AggregateFailures.
It will bring double the attention to deprecation, and fix is trivial - remove AggregateFailures from user RuboCop config.
Checklist

[-] I've added tests for this change
[-] I've added a Changelog entry
[-] I've updated a documentation

Thanks!


Champion and Spells Images not loading.

Hi, I was able to succesfully run the LOL UI using the npm install --force command since it gave me an error on installation the first time I tried to run the program. Now is recognizing the League Client Match but when I pick champions to ban or to select most of them appear with a failed image icon as shown in the picture attached. Any thoughts on how to fix the issue?

Are you using the following url: http://localhost:3000/?backend=ws://localhost:8999
Are there any other problems with the project? Console error messages, etc.?

Hello, it seems like the images are not available locally. To fix this issue, try to delete the "cache" folder, and restart the backend component. This should cause it to download everything again.


php7{1,2,3}: bump for "security release"

Motivation for this change
See linked announcements.
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

@dtzWill Looks good, this should be backported as well. Mind doing the cherry-picking?


Permission denied on the database when installing

Good evening,
I wanted to install panichd from a perfectly functional ticketit installation. Unfortunately I get this error just after clicking on "Install now" at this step. The access was working with ticket it before, I don't see what could block it.
I tried a php artisan cache:clear and to chmod 777 bootstrap and storage directories (for test of course) but it didn't work. Any ideas? Thanks

@antvet It is problem with access to migration file. Check chmods for migrations directory in database

Thank you @Mistral, it worked. However, I got this error about symlinks now... Any idea?

@antvet You have still problems with not enough user permissions (http server? or logged user?).


danger-swiftlint 0.11.0 is throwing false positives for delegate protocol class rule

and you can see that line 104:

running swiftlint locally won't throw this warning. any idea what could be happening here?

Hmm, it sounds like the danger plugin isn't using the same .swiftlint.yml as when you run locally. Could you check your usage against the docs? Thanks!

@dubemike  I had the same problem. The swiftlint binary that came with the gem was not up to date in my case. Setting the binary_path to my own swiftlint installation fixed that for me :)

It looks like this issue has been resolved in Swiftlint version 0.23.0.
Would the fix involve updating the SWIFTLINT_VERSION in version.rb? Would this likely be a breaking change?

That makes sense to me. Since we're not upgrading the Swiftlint version to a new major version, I don't think this is a breaking change. Anyone up to test locally and send a PR?

There are a couple of breaking changes with SwiftLint from the current version 0.20.1 outlined from their Changelog below:

SwiftLint Version
Breaking Change

0.21.0
Xcode 8.3 or later and Swift 3.1 or later are required to build.

0.22.0
Nested configurations will now be merged with parent configurations rather than replace them outright.

0.23.0 & 0.23.1
None

0.24.0
SwiftLint now requires Xcode 9 and Swift 3.2+ to build and Remove SwiftExpressionKind.other.

What do you all think? Since the issue has been resolved with 0.23.0, it'd be nice to get there at least.
I imagine not many people are still using Xcode 8.3. I'm unsure about the nested configurations as I don't use them myself.

@ajfigueroa Those changes are for building SwiftLint. This plugin use the prebuilt SwiftLint binary so it doesn't matter which Xcode people are using. You can use the SWIFTLINT_VERSION environment variable to override the default version, or you can specify a path to your SwiftLint binary.
swiftlint.binary_path = '/path/to/swiftlint'

@thii Oh, that's neat! My post can be disregarded then.
Sent with GitHawk

Swiftlint version was updated in the plugin. Closing this.


tectonic console cert problems

Issue Report Template
Tectonic Version
I think 1.5.6 can't open tectonic console
Environment
BareMetal
Expected Behavior
Open tectonic console
Actual Behavior
Tectonic console container won't start
Reproduction Steps
Create Ingress
kind: Ingress
apiVersion: extensions/v1beta1
metadata:
  name: tectonic.example.com
  namespace: kube-lego
  annotations:
    kubernetes.io/ingress.class: tectonic
    kubernetes.io/tls-acme: 'true'
spec:
  tls:
    - hosts:
      - tectonic.example.com
      secretName: tectonic-tls-secret
  rules:
    - host: tectonic.example.com
      http:
        paths:
          - path: /.well-known/acme-challenge
            backend:
              serviceName: kube-lego-tectonic
              servicePort: 80

Rename tectonic-tls-secret to tectonic-tls-secret-old
Wait for cert.
Recognise that kubectl doesn't work anymore.
Remove kube-lego ingress.
Failing tectonic console and kubectl with error:
error: error fetching provider config: Get https://tectonic.example.com/identity/.well-known/openid-configuration: x509: certificate is valid for ingress.local, not tectonic.example.com
Other Information
Try to restart tectonic console, after that the container constatly restarts with message:
Provider config sync still failing, retrying in 2s: Get https://tectonic.example.com/identity/.well-known/openid-configuration: x509: certificate is valid for ingress.local, not tectonic.example.com
Is the any chance to get back into the system?

In the assets.zip that the installer generated, there is a kubeconfig that is "root" and can be used to bypass identity. Using this with kubectl should give you the ability to add the ingress rule back and correct any other misconfigurations.
I'm curious how the cert was signed for ingress.local. All of my LE certs contain the expected hostname without other SANs.

@robszumski I didn't get kube-lego cross namespace running, so I created a Ingress for every hostname, maybe that is the problem?

tectonic says that I have to be logged in to the server. Is this correct?
KUBECONFIG=assets/auth/kubeconfig kubectl get pods --insecure-skip-tls-verify=true

@kersten What is the actual error? I don't think I've seen that one before.

@robszumski
error: You must be logged in to the server (the server has asked for the client to provide credentials)

Ah wait, let me check something

Sorry, for that touble. Had the wrong kubeconfig.
Ok, now I will try to get tectonic-console back working

ok, got the system back online. But one error is left: Chrome now doesn't let me visit the site:
Attackers might be trying to steal your information from tectonic.example.com (for example, passwords, messages or credit cards). NET::ERR_CERT_AUTHORITY_INVALID
You cannot visit tectonic.example.com at the moment because the website uses HSTS. Network errors and attacks are usually temporary, so this page will probably work later.
This was not the case before.

@kersten this is a Chrome thing, once a domain has enabled HSTS it won't let you not use it. The best way to solve this is to add the Tectonic CA to your trusted chain for TLS connections.

ok, than I am fine. Thank you!

@kersten did we resolve your issue? If so, I'll close this issue!

@mfburnett Yes, sorry for late response. The issue is now solved.


Reset substreams/firehose block ingestor backoff

Fixes #4976

Will re-implement on top of the current master branch with integration tests and resubmit


HHH-15802 SubQuery with 'in' results in java.lang.ClassCastExceptionclass org.hibernate.metamodel.mapping.internal.BasicEntityIdentifierMappingImpl cannot be cast to class org.hibernate.metamodel.mapping.EntityValuedModelPart

https://hibernate.atlassian.net/browse/HHH-15802

The following will be added to the 6.2 migration guide:
[[query-path-comparison]]
=== Query Path comparison

As of 6.2, comparisons of paths are type checked early. This means that a comparison predicate in HQL or JPA Criteria
might fail to construct if the types of the left and right hand side are not compatible.

In general, two types T1 and T2 are considered compatible if

* T1 == T2
* T1 instanceof T2 or T2 instanceof T1
* T1 is temporal and T2 is temporal
* T1 or T2 is unknown
* T1 can be widened/coerced to T2, or the other way around

Widening/Coercion usually refers to e.g. widening an integer to a long, but can also mean
that a string constant can be interpreted as enum when comparing against an enum attribute.

Note that a comparison of a temporal attribute against a string literal worked before

```sql
from MyEntity e where e.temporalAttribute > '2020-01-01'
```

but has to be changed to the proper temporal literal now

```sql
from MyEntity e where e.temporalAttribute > date 2020-01-01
```

Hi Beikov,
I am upgrading the springboot from 2.7.18 to 3.3.3.
Hibernate versions also upgraded as part of it to 6.5.2.Final.
After that I am facing the same ClassCastException as mentioned in the description.
I also tried in the latest version 6.6.1, I am still getting the same exception.
After debugging i found out that this exception is occurring when we are trying to get getSingleResult() from a criteria query which supposed to return count rows.
Could you please help me with this.

@adityadwadasi Please try to create a reproducer with our test case template and if you are able to reproduce the issue, create a bug ticket in our issue tracker and attach that reproducer.


Replace split with explicit token segments count

This small PR is changing the way how token segments are count. Split is replaced by explicit for loop to  counts segments. It should reduce number of allocations made during token validation and positively impact performance of that operation.

would ideally have liked a few unit tests to be exercised ensuring similar behavior, but overall I see no issues with this change and fewer string allocations makes a lot of sense

/azp run

/azp run

@krlm can you please rebase this against dev, thanks.

@krlm can you please rebase this against dev, thanks.

@brentschmaltz, @krlm what's the verdict on this PR? It's over a year old. Is it going to make it?
Reason I'm here is that we've been reviewing expensive string.Split calls across all 1st party Azure services and this one bubbled to the top. In fact, there are four in System.IdentityModel.Tokens.Jwt:

System.IdentityModel.Tokens.Jwt.JwtSecurityTokenHandler.ValidateToken
System.IdentityModel.Tokens.Jwt.JwtSecurityTokenHandler.ReadJwtToken
System.IdentityModel.Tokens.Jwt.JwtSecurityTokenHandler.CanReadToken
System.IdentityModel.Tokens.Jwt.JsonWebToken..ctor

The first three be eliminated entirely by "counting dots" -- like this PR does.
The last one is harder to avoid -- particularly since the substrings are actually needed.

This pr is more comprehensive https://github.com/AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet/pull/1645, we will try and get this one in


Responsive design implemented

Summary
The file initially was following poor CSS methodologies. I implemented responsive units and refactored the pre-existing code

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

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

Please try to install ldoc.
cd spec
ldoc .
cd docs && live-server


cargo: updater misses a valid update (update_not_possible)

Package manager/ecosystem
cargo
Manifest contents prior to update
https://github.com/coreos/zincati/blob/67f9f2cd0a376b4dc2406100083f3270634aef8c/Cargo.toml
Updated dependency
prometheus, expected update from 0.9.0 to 0.10.0 (released on 2020-09-01).
What you expected to see, versus what you actually saw
Dependabot is skipping this update, with the following messages logged:
  proxy | time="2020-09-08T05:14:56Z" level=info msg="proxy starting" commit=6c7a5acb2b2982f6bbe35988d635b24d7afa72ee
updater | time="2020-09-08T05:15:01Z" level=info msg="guest starting" commit=3f2146432118fdd1c0106af94aa1ee3180c134d7
updater | time="2020-09-08T05:15:01Z" level=info msg="starting job..." fetcher_timeout=5m0s job_id=54064398 updater_timeout=45m0s updater_version=0.119.2-44f545d18b91fba30ffc1bf83f991775bf34ab91
...
updater | INFO <job_54064398> Checking if prometheus 0.9.0 needs updating
updater | INFO <job_54064398> Latest version is 0.10.0
updater | INFO <job_54064398> Requirements to unlock update_not_possible
updater | INFO <job_54064398> Requirements update strategy bump_versions
updater | INFO <job_54064398> No update possible for prometheus 0.9.0

This is somehow new/recent, as previous updates to the same dependency were fine, e.g. https://github.com/coreos/zincati/pull/289.

I've been getting this as well. Dependabot worked for me for a couple PRs, then I updated dependabot.yml to include every workspace in my repo and it hasn't made a PR since. Not sure if that is related.

@andrewbanchich FYI I have a wild guess that ^ versions may play a role in this, see https://github.com/dependabot/dependabot-core/issues/2532#issuecomment-706071885.

I can confirm that carets were the root cause of my troubles. I'm closing this, keeping #2532 open as a duplicate.

I'm seeing this message when using the Unix epoch as a version for a private Ruby gem. Oddly, it looks like Dependabot worked exactly one time and since then has not caught newer versions of the gem.


Fix tmpl imports

Bundle Go template files into executable
Move templates into generator package
Fix README commands

Codecov Report

Merging #65 (b0d6130) into master (b3d4c31) will decrease coverage by 0.38%.
The diff coverage is 80.00%.

@@            Coverage Diff             @@
##           master      #65      +/-   ##
==========================================
- Coverage   73.90%   73.52%   -0.39%
==========================================
  Files          11       11
  Lines         525      525
==========================================
- Hits          388      386       -2
- Misses        110      111       +1
- Partials       27       28       +1

Impacted Files
Coverage Δ

grafikgen/cli.go
0.00% <0.00%> (ø)

evaluator/evaluator.go
69.69% <100.00%> (ø)

generator/generator.go
73.03% <100.00%> (-2.25%)
: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 b3d4c31...b0d6130. Read the comment docs.


Spelling docs

This fixes some misspellings in Swift documentation, it is split per https://github.com/apple/swift/pull/42421#issuecomment-1101092698

I think this is supposed to be split between user facing documentation and compiler documentation, but I'm having trouble figuring out what that is. When someone identifies which files are which, I'll split this PR.

@swift-ci smoke test

Note to self: squash merge this

Windows failure is unrelated to these changes
⛵


unexpected negative rewards in bench monitor

Apologies if this is an issue with baselines, rather than this repo.  But I'm not sure at this point.
I built a custom environment that has only positive rewards, and a cap/limit to the total rewards over an episode.  Yet occasionally the monitor files have negative reward values or values greater than the cap.  The large values might be explained by division by std dev, as mentioned in issue #88.  But I can't fathom why negative values should appear and haven't been able to track it down in the code.
I also see values larger than the cap in the max reward values reported by main.py (to stdout).  But I do not observe negative values.
Any thoughts/suggestions?

As I should have guessed, I had a code bug that led to this problem.  Has a=b, but needed a=np.copy(b).  And then ripple effects.  Oops.


docs: update version ranges in README

Since the v9.5.0 release, we support themes for 7.3 as well.

Simple enough.


NexusReader fails to open separate flats and darks

When flats and darks are separate files all methods of NexusReader give an error.

File "/home/algol/anaconda3/envs/CIL/lib/python3.5/site-packages/ccpi/io/reader.py", line 155, in
get_projection_dimensions
projections = file['entry1/tomo_entry/data/data']
KeyError: 'Unable to open object (component not found)'

Sample with separate darks is here (temporary):
https://www.dropbox.com/s/3b8n0znva1ivwuz/darksNXS.zip?dl=0

But if they are on different files, you'd better use 2 readers?
That'd work. Would we expect this to be the standard? In such case there should be some tests to perform regarding sizes of the flat/darks.

There can be two different readers, however NexusReader should give a message something like "no flats/darks have been in the file". So then one goes and look for separate files. This is quite standard when one performs 4D studies with start/stop scan, They do flats/darks before the experiment and then proceed taking projections only.

Calling load_flat or load_dark should raise an Error if darks and flats are not in the file.
The reader is not aware, and cannot be, of the existence of other files.

projections = file['entry1/tomo_entry/data/data']
KeyError: 'Unable to open object (component not found)'

seems to indicate that there is no data in the file, not that just flats and darks are missing.

Right... The path to data is the following /entry1/instrument/pco1_hw_hdf_nochunking/data
I found it using h5dump. See the screenshot. Looking at some of my code I found that separate flats/darks indeed can be accessed with entry1/tomo_entry/data/data but apparently no always! Is there a way to read the path from the file or add this path as a possible alternative for now?

and yes /entry/instrument/detector/data did not work... I managed to read from matlab:
darks = h5read(strcat('74408', '.nxs'), '/entry1/instrument/pco1_hw_hdf_nochunking/data', [1 100 1], [P 1 40]);

Ok, I've extracted it like this in Python:
f = h5py.File("/home/algol/Documents/DATA/74240.nxs",'r')
darks = list(f['/entry1/instrument/pco1_hw_hdf_nochunking/data'])

Have a look at the nexus_read_slice branch.
This example should do what you want: it extracts one slice (or subset)
https://github.com/vais-ral/CCPi-Framework/blob/nexus_read_slice/Wrappers/Python/wip/multifile_nexus.py
reader = NexusReader(os.path.join( os.path.abspath(directory) , '74331.nxs'))
acquisition_data = reader.get_acquisition_data_slice(200)
nparray = acquisition_data.array

returns an AcquisitionData for 2D. You can then access the Numpy array at nparray

BTW the NexusReader doesn't use the /entry1/instrument/pco1_hw_hdf_nochunking/data path but the standard entry1/tomo_entry/data/data which is an external link

OK, and the access to flats and darks is performed by using h5py independently or NexusReader will process them as well?

That's a good one. The reader will access the flats and darks in the files you provided, given the appropriate path to the data. The problem is that I haven't created a method to read the whole dataset :D I'll add it.
There's no image_key in these files. The image_key is an array of length `#projections +#flats +#darks" with 0,1,2, where 0 indicates a projection, 1 flat and 2 dark. The data and the projections and the darks are all stacked in a single array, hence the need to find which is which.


Add a "recently added content" section to the site

Hello, is this task still available?
I'm interested in the task.

@mercyikpe sorry, this is not available. We need to spec this out further before we can open it back up for contribution.

We're not going to move forward with this feature in the immediate future, and perhaps not ever. Closing this out for now. If we do this at some point we'll start fresh with an outline of what we're thinking and the value it'll provide to users.


[Arts 423] [Subfeature] - Integrating SC config server into Practitioner service

Description
Changes:
renamed config-server to be mts-config-server - this is to ensure it is understood to be the only 'mts' config server.
added search-paths to the config-server properties to ensure the newly created directories in the config-server repo are recognised.
Refactored PractitionerConfiguration to get the service working with the .json file held in the git-repo rather than local resources.
Removed any application.properties from practitioner - only leaving the bootstrap property file.
Added webconfig to the pom for practitioner-service
Added the spring.cloud.config.profile and spring.cloud.config.label  to the service bootstrap property file rather than the config-server application.property file - this is to allow the calling service to make the decision what profile/label should be used.
Removed the sc-config-server package as its a duplicate of mts-config-server
Removed application, log4j and liquibase property files form Trial-config-service as they are now pulled from the config-server repo.
Refactored the TrialConfigController to ensure the property values from the config server are used.
Also in addition to the changes in this repo, I added service specific directories to the /spring-config-server repo and moved the service specific properties to the relevant directories. The structure of that repo will be default/shared properties held at the base directory with service specific property files held under the service specific directories.
I tested the changes using Postman and all worked ok, I also tested changing the property files held in the /spring-config-server repo and ensured they were picked up without restarting the server. This proves the @RefreshScope is working as expected.
I had to change some JUnit tests due to the refactoring, I havent ensured the tests cover the refactored code so some additional work will be required for that but that is out of the scope of this story. That should be included in the integration of the config-server to existing MTS services story(s).

merging after Elad/Anats reviews

merging after Elad/Anats reviews


Bump version of MGC

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

could we briefly specify the changes included in this new MGC version that could impact us here?


[wip] Token Return

ensures that list_v2 always has a token avaliable

@xbrianh @ttung I'm not sure why the Travis CI tests are failing, but the continuous-integration/travis-ci/ pass; it appears to be pulling the same .yml file, unless the test runner on Travis CI is misconfigured.


Remove debug statements

It gets pretty annoying when trying to debug my own code

Ah yes, we can definitely turn them into some configurable macros, maybe lock it behind a feature

I was talkin about logging macros provided by the log crate, actually.

temporarily resolved by a debug macro, will make an issue to continue this


Bootstrap Template Scrolling nav

While I was working with this template I noticed when you re-sized the height of your brower window the elements on the page starts overlapping each other.
solution:
1.go into the file "scrolling-nav.css"
2. go to line 34 or search for the class "intro-section"
3. remove the "height property" or make it an comment to disable it
After you've done this the problem should be fixed :)

There is no such example/template in Bootstrap.

You are presumably referring to this third-party template:
https://github.com/IronSummitMedia/startbootstrap-scrolling-nav

yes, I'm sorry I didn't know it was a third-party template


Eager load associations to speed up GET /data-sets

A small optimisation in order to speed up the retrieval of all data sets through the GET /data-sets endpoint.
This optimisation is in support of a usability improvement to the Performance Platform admin tool which requires that the admin tool makes use of the /data-sets endpoint for the first time - see https://github.com/alphagov/performanceplatform-admin/tree/select-groups-and-types  and https://www.agileplannerapp.com/boards/15088/cards/8879.
By using Django's select_related QuerySet method to eager load the data group and type associations for each data set, the overall query count is reduced significantly. This in turn reduces the time taken to retrieve all data sets. In development, the time taken to retrieve all data sets was observed to drop from circa 3.5 seconds to circa 1.2 seconds.
There is scope to reduce the time taken further by optimising the retrieval of data type schema information within the DataSet#serialize() method (its call to get_schema). This could be achieved, for example, by passing a look-up dict of schema information to the serialize method.

Moved select_related functionality to DataSetView in response to comment by @jcbashdown (good spot, Jacob!)
Interestingly select_related does tolerate *fields that do not exist in the model. This explains why tests passed when the select_related was added directly to the list function in ResourceView despite ResourceView being inherited from by several views.


Extending StoryblokComponent with pageData prop for context passing

Description
Context / Problem
As we know, NextJS does not have native data passing mechanism from server to server component (something like Astro has Astro.locals, which is available in components, middleware and endpoints), hence we can only use prop-drilling in NextJS.
Here is a quick comparison of the mechanisms available in Astro vs NextJS:

NextJS has reasons for not having this, and it's due to segmentation (explained here https://github.com/vercel/next.js/issues/43704#issuecomment-2090798307). For the same reasons, you can't access the URL / Pathname information in server components.
Suggested solution or improvement
Workaround with Storyblok approach
However, not having the possibility to pass data server-to-server can be a challenge, there is a pattern how can we overcome this, and here is how?
There might be users who want to implement the following use cases (just a few examples):

Fetch the data from CMS about the page theme - light / dark - and switch descendant components look based on that
Fetch pricing plan data for the whole page and pass down the information for various deeper components to consume so data like price or discount, can be used in multiple components without the need to fetch them in each instance.
When rendering f.e. [...slug]/page.tsx), resolve the "locale" for the page and pass it down to Storyblok components so they can react to it, hence pre-render language switcher choices according to business logic.

The pattern we use is to always pass pageData prop to StoryblokComponent:
<Section
  heading={headingBlok && <StoryblokComponent blok={headingBlok} pageData={pageData} />}
  subheading={
    subheadingBlok && <StoryblokComponent blok={subheadingBlok} pageData={pageData} />
  }
  {...storyblokEditable(blok)}
>
  {content &&
    content.map((contentBlok) => (
      <StoryblokComponent blok={contentBlok} key={contentBlok._uid} pageData={pageData} />
    ))}
</Section>

So we always know that the other component has access to that contextual data. The only thing is that developers should not forget to pass that data down each time in component implementation - this might have the potential to be enforced by wrapping StoryblokComponent with custom logic and using it, or the component in this SDK could do so.
Closing remarks
This may not necessarily be something to implement or do in the code, as it is possible to implement in projects even now, because StoryblokComponent implementation allows us to pass any rest props down to components, but I am sure this pattern could be outlined somewhere in the documentation of the SDK as an example of how to achieve one or the other sophisticated pattern.
Additional context
No response
Validations

[X] Follow our Code of Conduct

Thanks @edvinasjurele!


140 more comprehensive ontoViewers logs

Signed-off-by: Patrycja Miazek<EMAIL_ADDRESS>#140

For me logs are much more readable, they contain a lot of information, they are not divided into different files by type, but in general they are more readable, pr still needs some improvements.
you don't specify the number of lines, so it returns the whole file from a given date (maybe it should be, for a default full file if you give a value it's n last lines), there is no way to collect logs from several days, I don't know if there should be such a possibility, I put it up for discussion @mereolog

One file per day is fine.


setup-envtest: init at 0.18.0

Description of changes

This is a small tool that manages binaries for envtest. It can be used to download new binaries, list currently installed and available ones, and clean up versions.

https://github.com/kubernetes-sigs/controller-runtime/tree/main/tools/setup-envtest
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

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

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

@maxbrunet, sorry for the delayed response and thank you for testing on aarch64-darwin.
I have set the attribute __darwinAllowLocalNetworking = true;, as desired.


Add: common faces for man and woman

Related with https://github.com/hlissner/emacs-doom-themes/pull/325.
Note that themes palenight and moonlight already set their own faces for man.

Perfect!

Given that orange/green are a little bit too specific and didn't look pretty nice in other themes that are not Solarized I changed the overstrike and0 underline colors to operators and keywords respectively (which keeps it the same for Solarized) so it looks better on other themes. Hope it's ok.


Prevent TEA_PKGS=' ' from breaking everything

Fixes https://github.com/teaxyz/pantry/issues/823

Coverage: 34.591%. Remained the same when pulling 4a918795507b3ba2cc3f0fc29fd34980686b9014 on fix-TEA_PKGS-empty into e6acb2b2e97e46f512c3f0de619ac5281f77b820 on main.


Convert.ToString returning type when ToString method is available

Convert.ToString currently returns an object's type rather than calling its ToString method.
A similar issue logged under #2887 has just been fixed by pull request #3414, but the behavior reported here is still not addressed.
Steps To Reproduce
https://deck.net/f1e618d16a662f5d7679b68facd6667c
With test case (thrid assertion) for issue #2887 below:
https://deck.net/a4ca8bde2174382a4c7ec9cfdde6acd3
https://dotnetfiddle.net/IDwe5k
Expected Result
Success: t.ToString() is t.Value.
Success: Convert.ToString(t) is t.Value.
Success: Convert.ToString() returns 'System.Object'.

Actual Result
Success: t.ToString() is t.Value.
Failure: Convert.ToString(t) is t.Value.
Success: Convert.ToString() returns 'System.Object'.

See also

Bridge.NET forum thread: Convert.ToString returning type when ToString method is available

@fabriciomurta thanks for taking care.

Fixed! The fix will make it to the next Bridge version (see attached milestone).

@fabriciomurta thanks. Appreciated


Enable the simultaneous use of Flex delegate and other delegate

Fixes problem when Flex Delegate and other (e.g. external) delegate is used.

Can you add tests in subgraph_test to show what this fixes?

Hi @robert-kalmar This PR is in draft, any update on this? Please. Thank you!

Hi @robert-kalmar This PR is in draft, any update on this? Please. Thank you!

Hi @robert-kalmar Any update on this PR? Please. Thank you!

@gbaned , no update yet. We still did not figure out how to test the fix without external delegates being called.

Hi @robert-kalmar Any update on this PR? Please. Thank you!


Asp.net core web api that returns File is throwing No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'null' is therefore not allowed access

From @adriangh83 on August 10, 2017 16:37
Hello,
I have a problem when trying to return a File on an .net core (1.1) rest service.  I get "XMLHttpRequest cannot load http://localhost:83/api/attachment/content/22?_=1502382297330. No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'null' is therefore not allowed access. The response had HTTP status code 500."
If I change the controller code so I return Ok and not File I do not get this error anymore.
Can you help me?
Thanks.
Startup code:
services.AddCors(options =>
            {
                options.AddPolicy("AllowAllHeaders",
                      builder =>
                      {
                          builder.AllowAnyOrigin()
                                 .AllowAnyHeader()
                                 .AllowAnyMethod()
                                 .AllowCredentials();
                      });
            });

app.UseCors("AllowAllHeaders");
app.UseMvc();

Controller code:
var contentType = MimeMapping.GetMimeMapping(Path.GetExtension(attachmentPath));
using (var stream = new FileStream(attachment.CaleAtasament, FileMode.Open))
{
           return File(stream, contentType, attachmentName);
}

Client Code:
var getUrl = baseUrl + 'attachment/content/' + $('#tbAttachmentId').val();
                jQuery.ajax({
                    url: getUrl,
                    cache: false,
                    crossDomain: true,
                    type: 'GET',
                    beforeSend: function (xhr) {   //Include the bearer token in header
                        xhr.setRequestHeader("Authorization", 'Bearer ' + jwt);
                    },
                });

Formatting for style only by @Eilon.
Copied from original issue: aspnet/Security#1357

From @dallancarr on August 10, 2017 20:54
Can we see the full controller action code snippet? Also check exception it's returning 500 so hence not returning CORS headers.
So possibly not a security issue.

For some reason the previewer doesn't format the code so I've attached some screenshots. I'm sorry that some variable names are not written in English.

I should mention that on the client side I've used an ajax request so that I can send a JWT token in header.
After creating the issue I've tried using anonymous on the server side and windows.location.href on the client side and I didn't get the same CORS error. But using this approach I cannot attach the JWT token in header.

Trying to use another approach I find out that the error appears because I closed the stream.
If I use the following code I no longer get that CORS error:

Still don't know why I get CORS error on server error.
Sorry about wasting your time and keep up the good work. :)

This issue was moved to aspnet/Home#2374


[WIP] Increase pre-boot-sleep and limit upload retries

What is the problem that this PR is trying to fix?
What approach did you choose and why?
How can you test this?
What feedback would you like, if any?

This caused trusty builds to start failing due to SSH errors and be requeued so I reverted the changes.


Try using the latest version of this repo acheong08 / ChatGPT.

It supports loging into your chatgpt acoount with email and password and it works.

I just checked your issue. Thank you for that.


feat: Create using_attributes context managers and have openai instrumentation read from it

@fjcasti1 Can you break this into separate PRs for the different libraries? The reason being that targeting multiple libraries at once in a single PR will cause a race condition in the release pipeline. It's not hard to resolve manually, but it's simpler to make each PR update only a single package.

@fjcasti1 Can you break this into separate PRs for the different libraries? The reason being that targeting multiple libraries at once in a single PR will cause a race condition in the release pipeline. It's not hard to resolve manually, but it's simpler to make each PR update only a single package.

Of course!

nice work!


36 as a user i would like to consistently view the same navigation bar on all pages so that i can browse the website without getting lost

This change tests that the navbar links are valid. This change makes sure the navbar in all the listed pages have the listed links with the listed wording.

All of the checks are failing above?

Fixed by changing "test.only" to "test"

All checks appear to pass now.  Did you want me to merge?


issue1369

Check List:

Fixes #1369
Unit tests : NO / NA
Javadoc with a code example (API only) : NO / NA

added symmetrical case for isEqualToComparingFieldByFieldRecursively

@dev-priporov we have yet decided to address #1369 so this PR is being put on hold.
One comment though, the PR does not contain any tests, if the PR solves an issue it should contain tests to prove the issue was solved (but again don't spend time on it until we decide to address the issue).
Anyway thanks for taking the time to submit a PR.

superseded by the recursive comparison which is not symetrical by design


Fix broken CI tests

[x] Fix missing import in cw721-metadata-uri
[x] Update schemas

Looks good to me.


Feature request: "layouts" config option, similar to "includes"

It would be great if there was a layouts config option to configure a directory where layouts can be found (if the path does not start with a "."), similar to the "includes" option for resolving include paths. Could that option be added?

include and layout are just two ways to import template files but the file loader doesn't know which tag originates the import.
But you can create your own loader with a different behavior. Take a look to the default file loader. It's just a class with two methods: load and resolve. You can create your own resolve logic, for example, if the path starts with layout:, search the files in a different directory.
export class MyLoader implements Loader {
  resolve(from: string, file: string): string {
    if (file.startsWith(".")) {
      // relative path
    }

    if (file.startsWith("layout:") {
      // look in the layouts folder
    }

    // look in includes folder.
  }
}

Aha, thanks for the pointer. I like the idea of a path qualifier, like "layout:". For now, I just put my template in the includes directory :)


RISC-V/PLIC: set proper nomenclature for threshold offset (plus typos fixing)

Hi, I made this PR just to "correct" the name of some variables/macros which does not respect the nomenclature provided by the PLIC specs.
As you can see in RISC-V PLIC specifications, offset 0x200000 is used to access the THRESHOLD for context 0 instead of CLAIM/COMPLETE (0x200004) for context 0, so the related code has been renamed accordingly.
At the same time, I've also fixed some typos in "threshold related" functions.
Hope you will find this useful.

I was about to suggest the same, it seems more accurate to me to call it context.

So you're saying to change PLIC_CLAIMCMPLT_OFF to a sort of PLIC_CNTXT_OFF and plic_claimcomplte_emul to plic_cntxt_emul?

Exactly.

It can be a good solution, the only thing I'm a bit concerned about is that context is used to distinguish the execution contexts in also other fields (like the enabled interrupt sources).

But I get your point here. They don't specify a name for that "interrupt handling" region as they do for the APLIC (i.e. IDC). So I feel there is no perfect solution for this, so I'd be ok to leave your PR as is.

Thanks again @filippofontana !

Thank you guys for accepting this solution. I hope that this will be clear also for others!


Remove post-deploy Icinga alert for Smokey

https://trello.com/c/vIt6bDXI/258-stop-full-post-deploy-runs-of-smokey
Smokey is run twice after most deployments:

A full run triggered by the Deploy_App job ^1.

A partial run triggered by the Deploy_App_Downstream
job (for CD apps).

This commit removes a single Icinga alert that would be updated
every time the job runs. The introduction of (2) has made this
meaningless: it's unclear if the status of the alert relates to
a full or partial run of Smokey.
We also run Smokey in a continuous "loop", with an Icinga alert
for each feature. These more fine-grained Smokey Loop alerts also
make the general one we're removing here redundant i.e. we should
just remove it instead of fixing it.
As I was making this change I noticed a bunch of variables that
aren't used anymore so I've removed them.

Thanks for reviewing @ChrisBAshton !
Would you mind deploying this, since I don't have access?
I've already tested it on Integration so I don't you'll need to do any more testing.


Duplicating spawners

When you mine a spawner with silk touch you get the message that it is sent to inv, and it does, but it also drops as an item! Running latest build of paper for 1.19.3 and latest update of silk spawners

Hi, sorry I am unable to reproduce this issue. Which server software are/were you running (Spigot/Paper/...)?

I'm having the same issue, I have a video of it. https://clipchamp.com/watch/ImQdD9X1XCV

I'm having the same issue, I have a video of it. https://clipchamp.com/watch/ImQdD9X1XCV

I am still unable to reproduce this. I've tested Spigot and Paper version 1.19.4 and Paper version 1.20.1.
Which server software are you running (Paper/Spigot/Purpur/...)?
Which plugins are installed?
Additionally, could you please provide the config file located at plugins/SilkSpawners_v2/config.yml?


android 64-bit  support?

i see there is no android 64-bit support.  when we will have this?

Where do you see that there's no support for that? Support for arm64, x86_64 and mips64 should be available since around the 1.1 release, at least in 1.2 and 1.3.

@mstorsjo  https://github.com/cisco/openh264#os-support

Ah, I see. Ok, I'll make sure that gets fixed.

There is a glitch in the script, at least on Mac OS. The correct command line is
make OS=android ARCH=arm64 TARGET=android-21 NDKLEVEL=21

On the face of it, this duplication is not necessary, but at least it deserves to be documented.


Option to handle Back key

It would be nice to have an option to consume Back key event and don't pass it to rest of activity.

@Euphrates-Media That's a very good idea.

With v1.5, OnBackPressListener added, but still needs a trick. OnBackPressListener will be always triggered whether it is cancelable or not. Thus you can use cancelable=false and trigger the event.


meter values updating every 4 minutes, how to bring back to 1 minute and how to add additional current & voltage measurends

OCPP is updating every 4 min and it should update every 1 minute

Hi, to add more measurands, refer to this example:
https://github.com/matth-x/ESP32_WiFi_V4.x/blob/cc5f5dff60ce423fec1370c26ecfdf56c85a22b5/src/ocpp.cpp#L151-L156
Note that you also need to activate the reading in OCPP by changing the configuration. The MeterValuesSampledData needs to contain the new measurand (e.g. Current.Import).
And yes, currently the library collects 4 measurements before sending them to the OCPP server. I will change the default behavior so it sends every measurement immediately. In the meantime you can lower the sample interval to 15s, then you will get the updated values every minute.
Feel free to reopen if this doesn't solve this issue.

Can you suggest how to change 4 minute to 1 minute for , for meter values update every minute to steve ?


Updating docker image to cover Log4j Solr vulnerability.

Hello, I thought this would be the best place to mention that to address #https://github.com/advisories/GHSA-xxfh-x98p-j8fr you should consider updating the nbgallery-solr image currently on Docker Hub ASAP. The solr image you use as a base has already been fixed and updated, so a simple rebuild and update to Docker Hub should be sufficient. Thanks!

Sorry we missed this, but we did upgrade to 8.11.1 a while back (which solves all of the log4j vulnerabilities that Solr was potentially vulnerable to) but we are also in the process of trying to upgrade to 9.0 which even patches up to latest so scanning tools will be happy.


one last try to get this test de-flaked

My hunch is that the lack of a Task.Run means things aren't yielding properly. If this doesn't de-flake things, I'm just gonna skip the dang test :).
/cc @moozzyk @mikaelm12

The other one that fails frequently (even though the tests pass):
  info: fail: Microsoft.AspNetCore.Sockets.ConnectionManager[0]
  info:       Failed disposing connection 8655f364-c114-416f-8ce0-4de234759fe1
  info: System.ObjectDisposedException: Cannot access a disposed object.
  info: Object name: 'WebSocketConnection'.

https://travis-ci.org/aspnet/SignalR/jobs/203977072 has both failures.

I believe my other PR will fix that (#218)

Actually, there is still a possibility for it to occur due to a race between someone closing the socket and a protocol error, so I'm adding some code there to avoid the exception. We don't care if the race occurs so the exception is unnecessary.

🐑 :it:


Fix KeyData iteration not including potentially empty trains

The ExtraDataFunctor is currently able to iterate over both DataCollection and KeyData objects. The latter however does not iterate over empty trains by default, which the former does. This PR fixes KeyData by additionally passing include_empty=True.

Ah, the CI is failing because Python 3.6 is no longer available on the default Github actions runner. I imagine you'll want to drop some old versions from CI & add newer ones - most of our packages are currently tested on Python 3.9 and above.
If for some reason you do need to maintain Python 3.6 compatibility, it's not difficult, you can add a separate CI job which runs on Ubuntu 20.04, like this:
https://github.com/pypa/flit/blob/7cc7d330d38d48ef865c07e5086a917278b19050/.github/workflows/test.yml#L46-L50

Thanks for review!
I will look into dropping Python 3.6 support later.


enviando documentação do notify

I made the change directly into the main branch because there were many files (that should not be here) in this pull request. To avoid errors like this, I updated the gitignore file.


Is this project maintained?

What are you trying to achieve?
Im trying to see if codecept is a valid alternative for testcafe, by going through getting started documentation but i got stuck on first command, then on second command from docs.
What do you get instead?
Im trying to generate some example, because codeceptjs run --steps  or codeceptjs run my_test.js does not run anything for me.
➜  lol git:(master) ✗ npx codeceptjs generate:test
Creating a new test...
----------------------
Cannot read property 'match' of undefined
➜  lol git:(master) ✗ node -v
v14.4.0

Provide test source code if related

Details

CodeceptJS version: 3.05
NodeJS Version: 14.4.0
Operating System: mac os 10.14.6
puppeteer || webdriverio || protractor || testcafe version (if related)
Configuration file: none

Yeah, there is understandable that this is an open source not commercial one, thus everyone's contributions are in their free time. I'm still using codeceptjs and working fine from my end. :).

Yeah im pretty sure once you know it its good, but learning curve is where it lies when it comes to adoption/switching from a different solution. API looks very promising, but if i cant get to API phase, it doesnt matter ;)

Aha, I'm coming from different frameworks thou! I was using many before landing on codeceptjs. :)

@pavelloz yeah, If a project was not released in 24 hours it may be considered abandoned
https://www.theolognion.com/npm-now-marks-packages-as-abandoned-after-24h-since-the-last-commit/
According to error, you seem to have a broken config. Try to rerun npx codeceptjs init script in a fresh directory
Join our Slack for such kind of questions.
http://bit.ly/chat-codeceptjs

Usually unmaintained docs mean unmaintained project, because noone reports / fixes bugs in it, especially in getting started.

@pavelloz Probably not a healthy comment on open-source communities. If you want to see if a project is maintained or not, PTAL at the forums and community discussions, especially recent activity. By that scale, CodeceptJS is very much maintained. Also, contribution is very much welcome from anyone, meaning if you see something broken, feel free to raise a PR. Lastly, it's highly unlikely that something basic is broken given that its used by thousands of people. Happy testing :)


Unit Tests for Graph

Not all methods are coverd due to difficulty of using interface (Graphs cannot be compare in assert method, different ways of casting to string, local settings produces error in github). You can develop more tests if you have time.

Here needs changing for Culture-Invariant printing ;)
https://github.com/Kri5t0fK/SouvlakiSearch/blob/84f694f5085a26e10aaaa1c814c0bc027375cb5f/SouvlakMVP/SouvlakMVP/Graph.cs#L107
string str = this.position.ToString("G", CultureInfo.InvariantCulture) + " : [";

Here needs changing for Culture-Invariant printing ;)
https://github.com/Kri5t0fK/SouvlakiSearch/blob/84f694f5085a26e10aaaa1c814c0bc027375cb5f/SouvlakMVP/SouvlakMVP/Graph.cs#L107
string str = this.position.ToString("G", CultureInfo.InvariantCulture) + " : [";

give back toString() tests

Other solution -> change CurrentCulture in Testing scope:
CultureInfo.CurrentCulture = CultureInfo.InvariantCulture;


Update _index.md

Updates for: https://mattermost.atlassian.net/browse/MM-35211
Partner and Experimental labels.
@hanzei, the content I've added is placeholder content as I don't have the definitions for those labels. If you could point me to the source definition I'll edit this PR. Thank you!

@aaronrothschild, ping on this. :)


No audio output for plugins under Linux

The LV2 and VST3 plugins (1.02) on Linux (Fedora 38 using Reaper 6.82) the plugin shows 2 input channels, no audio output channels.  Everything appears to be running, but no audio output is generated.  The standalone binary works.
Same version of the VST3 plugin on Windows using  Reaper 6.82 shows 2 channels of output, and works.

Did you build yourself or use binaries I built?

Can you try https://github.com/FigBug/Wavetable/releases and let me know if it fixes it?

Maybe fixed in 681be5731b4e50b9ecd9754f08406d5c60de0400

Sorry for the delay, yes that did fix the issue!  Thank you!


Pasted images no longer show on the Canvas, also impacts images that were working fine before.

Preflight Checklist

[ X ] I agree to follow the Code of Conduct that this project adheres to.
[ X ] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Describe the bug
I can no longer successfully copy/paste screenshots into the canvas. When I paste, I do see a blank placeholder for the image. When I try to edit the image, I see that the binary code for the PNG is there. What seems to be missing is the declaration to use base64 in the encoding bit of the PNG. The IMAGE url shows "data:image/png,binary", when I copy and paste this URL into a JSFIDDLE and add base64 to the string, it does show: "data:image/png;base64,binary"
To Reproduce
Steps to reproduce the behavior:

Paste any image into a blank canvas
You will see a placeholder without an image
Right click on the placeholder and click on Edit image
Copy paste the Data URL for the image (starts with data:image/png,)
Open JSFiddle.com and create a  tag and paste what you just copied between the ""
Hit run and confirm that the image is not showing (https://jsfiddle.net/9wpbx75a/)
Replace the beginning of what you pasted in the src, so that it starts with "data:image/png;base64,"
Hit run again and confirm that the image is now showing correctly (https://jsfiddle.net/9wpbx75a/1/)

Expected behavior
Images should be shown in the canvas
Screenshots
See JSFIDDLE's and explanation on top.
draw.io version (In the Help->About menu of the draw.io editor):

draw.io version 13.6.8 (in Confluence)
draw.io version 13.6.8 (https://app.diagrams.net/)

Desktop (please complete the following information):

OS: iOS 10.14.6
Browser Chrome v85, Firefox v80

Hi, I'm facing the same issue. I'm unable to copy paste screenshots OR view screenshots added in previously saved files. Please help as it's massively impacting my project.

Fixed in 13.6.9. Thanks for the report!

Any idea when Confluence picks up the new version? Thanks for the quick fix.

Confluence Cloud will see it when you see 13.6.9 in the Help menu at https://confluence.draw.io


Add undefined to Typescript keywords

This PR will add undefined to the list of keywords for Typescript that was missing so far.

@RunDevelopment npm decided to update the lockfile automatically, I just reverted the automated changes and only included my addition to the Typescript component files.

Thank you for contributing!


adding option to specify client_id for MSI

My team recently ran into an error when using an MSI to get a token from an AKS cluster. The agent pool for this particular cluster had two MSIs, the one we created and another one that came from AzSecPack. Other clusters with our same application were fine, but only had the single MSI we were expecting to be there.
We spun up an ubuntu container and ran some curl commands
curl 'http://<IP_ADDRESS>/metadata/identity/oauth2/token?api-version=2018-02-01&resource={ourResource}' -H Metadata:true
resulted in
{   "error":"invalid_request",   "error_description":"Multiple user assigned identities exist, please specify the clientId / resourceId of the identity in the token request" }
and
curl 'http://<IP_ADDRESS>/metadata/identity/oauth2/token?api-version=2018-02-01&resource={ourResource}&client_id={ourClientId}' -H Metadata:true
resulted in a good response.
Looking at https://docs.microsoft.com/en-us/azure/app-service/overview-managed-identity?tabs=portal%2Cdotnet#rest-protocol-examples I saw that one can specify the client_id as a query parameter on the URL to fetch the token, so that is what I am trying to do in this pull request.

Thanks, I have added the other query parameters and switched to Option<String>.

I dug into this more, as the mi_res_id vs msi_res_id was bothering me.  I think the documentation is wrong.  The CLI, specs, and multiple language SDK implementations all refer to msi_res_id.
Note, I opened a work item to address the document  typo here, with references to multiple SDKs and the REST API spec:  https://github.com/MicrosoftDocs/azure-docs/issues/92662

Thank you for the investigation, I've been convinced and replaced mi_res_id for msi_res_id. I also opened another issue for the other document I originally read. https://github.com/MicrosoftDocs/azure-docs/issues/92664


Reference

Do you know if there is a tool like this if I'm using PostgreSQL node.js driver and elasticsearch.js?

Sorry, I know nothing about Node.js


ARCamera shows old suggestions after saving an observation

Describe the bug
The ARCamera shows the last suggestion after saving an observation and opening the ARCamera again.
To Reproduce
Steps to reproduce the behavior:

Open the ARCamera
Point it at something it shows a suggestion for
When you get a suggestion, tap the shutter to navigate to ObsEdit
Save the observation
Open the ARCamera again
The last suggestion you saw in the ARCamera is still on screen

Expected behavior
When you open the ARCamera, there should not be any suggestion on screen.
Context (please complete the following information):

Device: iPhone SE (3rd generation)
OS: iOS 17.3.1
App version: 0.28.0 (80)

Additional context
This one might be best suited for @jtklein as I think it has to do with how long the native code is preserving its buffer of recent suggestions.
Carrie reported this and posted a screen cap in Slack: https://inaturalist.slack.com/archives/C02G5AATVGA/p1710766186596349

@albullington notes that there's a related issue in the new flow where if you nav from ARCamera to Suggestion to ObsEdit, then back out to ARCamera, your old suggestion is still visible.

I did try to fix this by adding a method to the inat-vision plugin to reset the internal state of stored results but calling this function on the camera screen getting focus or navigating away from it did not fix this issue here. In Seek we reset the predictions state when the camera is focused which might help here as well.

I tried resetting the result as well as calling resetStoredResults( ) on navigation blur, with no luck.
One interesting thing about this bug is that the previous result sometimes reappears maybe 5 seconds later, long after the user has moved the camera away from the previous organism. This makes me think that we're processing previous camera frames that we should no longer be processing.
Repro steps:

Take a photo of a Pilea with ARCamera
Tap to navigate to Suggestions
Point phone at the ground (not at the Pilea)
Go back to ARCamera

At this point, you'll see Pilea still on the screen in TaxonResult for a moment, then it disappears. But alarmingly, Pilea will reappear as a prediction a few seconds later, even though the user has not pointed their phone back at the Pilea.
I'm going to unassign myself since it seems more likely something that needs to be fixed in the vision camera.

This was reported again by @tiwane and still appears to be happening in the latest build, 0.39.0+105

These repro steps are no longer applicable because there is a new step with the suggestions screen. When I follow the steps, and additionally accept a suggestion between tapping the shutter and saving the obs, I can not reproduce this.
@tiwane could you provide up-to-date repro conditions?


apply_scale_32 not lowering to ARM NEON sqdmulh instructions

Preamble
ARM NEON has fixed-point multiplication instructions, like sqdmulh. Existing NN inference solutions (TFLite, ruy, XNNPACK) use them. It's not possible to match their performance in quantized workloads without using them. These instructions are important primarily[^1] in quantization rescalings of internal int32 accumulators at the end of each quantized layer.
ARM has already successfully invested effort in specifying rescalings in a way that allows bit-exact agreement across implementations that use sqdmulh and implementations that perform plain int64 arithmetic (https://github.com/google/ruy/pull/227).
So my understanding has been that it is the intent that the fixed-point multiplications in the TOSA spec are implementable by means of sqdmulh. Unfortunately it seems that some details are currently preventing that from happening, and that fixing that would require untangling multipl issues scattered across: the TOSA spec, the MLIR TosaToLinalg pass, and codegen passes specializing to ARM codegen. So the present issue is filed as an overall tracking, motivating issue for all that.
Requirements to be able to use sqdmulh
In order to be implementable as a sqdmulh, a fixed-point multiplication needs to satisfy 2 criteria:

The int64 value ((a * b) >> shift) must be in int32 range, with the only exception being saturating when both a and b are INT32_MIN.
In the case where both operands are INT32_MIN, the result must be allowed to be off by 1 (saturating 2^31 to INT32_MAX).

Getting TOSA's apply_scale_32 to meet the requirements to use sqdmulh
Meeting the no-INT32_MIN requirement
Crucially, apply_scale_32 has a REQUIRE(multiplier>=0) statement that ensures that the multiplier operand is never INT32_MIN, so the above requirement 2 is automatically met.  In fact, I guess that that was the motivation for that REQUIRE(multiplier>=0) statement.
One note though: that REQUIRE(multiplier>=0) information is lost in the TosaToArith pass. Either one needs to find a way for TosaToArith to preserve that information, or one needs to have explicit lowering to sqrdmulh intrinsic before TosaToArith.
Meeting the no-overflowing-int32 requirement
In practice it is the case in at least 90% of use cases that the shift amount is a compile-time constant and is at least 31. That automatically ensures that ((a * b) >> shift) is in int32 range.
So we should have a pattern identifying that case and generating a sqrdmulh intrinsic for that case.
For the remaining cases where the shift amount either isn't known at compile time, or is less than 31, conformance with the current TOSA spec requires materializing i64 products. A currently non-conformant approach (mirroring what TFLite backends do) would be to left-shift the pre-rescale int32 values by (31-shift), before passing that to sqdmulh. That left-shift could overflow, and the looseness here is in assuming that the overflow won't actually happen.
Maybe an attribute could be added to TOSA ops that perform an apply_scale_32, allowing to specify that behavior. The TFLite-to-TOSA import pass would be able to set that attribute since TFLite is already making that assumption.
[^1]: Secondarily, they may also be used in implementations of math functions, when a fixed-point strategy is followed, but that's less essential as there are alternatives, such as dequantizing to float or lookup tables, depending on the function at hand.

@bjacob This should be fixed for given we have a 32-bit version of the lowering now. https://github.com/llvm/llvm-project/commit/9294a1e9a8ab8fed60ee1b0737e944077b5339cd

does 'fixed' mean 'actually generate a fixed-point mul instruction like sqdmulh' or some intermediate goal?

At this point we generate almost purely i32 operations that should pass through the vectorization code. The only section that is not i32 attempts to "encourage" the compile towards sqdmulh. Is there a good way to see whether the backend compiler actually pattern matches the behavior?
https://github.com/llvm/llvm-project/blob/main/mlir/test/Conversion/TosaToArith/tosa-to-arith.mlir#L29
If not we would need to have some intrinsic we can insert instead of the minimal i64 code above.

Before going into the backend to see whether it uses the opportunity to generate a sqdmulh, we need to make sure that we actually create that opportunity.  The requirements are outlined in this PR's description > "Getting TOSA's apply_scale_32 to meet the requirements to use sqdmulh".
In particular, the IR going into the backend must convey the information that at least one of the operands is not INT32_MIN. As explained in the PR description above, that is implied by the tosa.apply_scale having a REQUIRE(multiplier>=0), but that information is lost in TosaToArith. The above tosa-to-arith.mlir link in the previous comment confirms that information is lost: there is no way for the compiler backend to know that both sides can't simultaneously be INT32_MIN, making it legal to compile this as a sqdmulh.
The other part as explained in the above PR description is that tosa.apply_scale_32 allows the shift argument to be as low as 2. The sqdmulh instruction can be used only if shift>=31, and that needs to be statically known.

Hi @bjacob. You mention in the “meeting the no-overflowing-int32 requirement” of the issue statement that a left shift of (31-shift) on the value to be scaled could overflow an int32. I think the third line of the TOSA spec for apply_scale_32 is relevant to this. This REQUIRE statement requires “value” to be in a range for which left shift of up to (32-shift) does not overflow (for shift<32).

Thanks @dominicsymes , I had missed this!
So this leaves only the other issue, of avoiding the INT32_MIN * INT32_MIN case which would overflow / where sqdmulh saturates?


Following requirement(s) failed: PDO extension loaded with Sqlite support

I have installed SQLite ( sudo apt-get install php-sqlite3). and after that restarted the apache2 service (sudo service apache2 restart). still facing this issue when trying to install cockpit.
configuration
OS - Ubuntu 18.04
SQLite -  v 3.22.0
PHP - 7.1
..

I ran into the same issue, turns out the php that my apache2 service was using is different from the one I was installing sqlite3 to. Make a basic php script that calls phpinfo();  and open it from the browser. Then check which version of php is apache2 using, in my case I had to run sudo apt-get install php5.6-sqlite3


MINOR: Improvements on log4j

add thread id as prefix in state directory classes; also added logs for lock activities.
add logging for task creation / suspension.
add more information in rebalance listener logging.
add restored number of records into changlog reader.

ping @dguy @mjsax @enothereska for reviews.

In StoreChangelogReader.java, could we add some info on how many records were restored, in the restore() function, i.e.,
log.debug("Took {} ms to restore active state of {} records", time.milliseconds() - start, MISSING);
I've found that important when debugging restores. @dguy what do you think?

@enothereska @dguy comments addressed

Merged to trunk.


 Luckily Helm seems to ignore the issue and cluster charts in use were unaffected.
The defaults can be removed as the definition of network already contains the correct defaults.


on Android setting TextInput keyboardType prop will not work as expected

[x] I have reviewed the documentation
[x] I have searched existing issues
[x] I am using the latest React Native version

on Android setting TextInput keyboardType prop will not work as expected with the following types: 'numbers-and-punctuation', 'url', 'number-pad', 'name-phone-pad', 'decimal-pad', 'twitter', 'web-search', 'visible-password'
Environment
Environment:
OS: macOS High Sierra 10.13.4
Node: 8.9.1
Yarn: 1.6.0
npm: 5.7.1
Watchman: 4.9.0
Xcode: Xcode 9.3 Build version 9E145
Android Studio: 3.1 AI-173.4720617
Packages: (wanted => installed)
react: 16.3.1 => 16.3.1
react-native: 0.55.3 => 0.55.3
Steps to Reproduce
I just tested with different types from documentation
enum('numbers-and-punctuation', 'url', 'number-pad', 'name-phone-pad', 'decimal-pad', 'twitter', 'web-search', 'visible-password')
Actual Behavior
the keyboard is not the right one, it is the same as for ascii-capable

Right in the documentation it says:
iOS Only

The following values work on iOS only:

ascii-capable
numbers-and-punctuation
url
number-pad
name-phone-pad
decimal-pad
twitter
web-search

https://facebook.github.io/react-native/docs/textinput.html

@zibs I see now, the documentation is confusing , right near under Android section there is a table and I was thinking is applying to Android only.. ok then, I will close this issue

@zibs, i see that now the docs say:
The following values work across platforms:

default
number-pad
decimal-pad
numeric
email-address
phone-pad

but Android seems still to show default tab if set to number-pad or decimal-pad anyway.
Environment:
<EMAIL_ADDRESS><EMAIL_ADDRESS>

@devey  upgrade your RN version to 57


fix: night mode tweaks

A couple of tweaks to night mode

Allows the ability to see the selection of text in the editor:
Before

After

Make the test result odd rows darker:
Before

After

Fixes partially freeCodeCamp/freeCodeCamp/#17782

@thatnini @Bouncey I feel super bad about what I'm going to say. I don't want to discard someone's contributions, but we can get the vs-dark theme to work and resolve the text selection issue in the process if we, um, remove all the night styles for the code editor. :(
Here is the result on my local copy:

@huyenltnguyen No need to feel bad, if that is the fix then that is great. Have those changes been pushed up to this branch?

Don't feel bad, I just wasn't sure if we wanted to continue using the custom theme or vs-dark for the editor, so my original fix was just to remove the offending lines and leave the rest as is.
I've now removed what I believe to be the rest of the css that was used to create the custom dark theme for the editor.

I've now removed what I believe to be the rest of the css that was used to create the custom dark theme for the editor.

Perfect. Thanks for making these changes. This LGTM

Thanks @huyenltnguyen and @thatnini :+1:

Thanks @thatnini .

Hey @huyenltnguyen @thatnini. I was the one who implemented the colors for the code editor. Don't feel bad. There were a lot of issues with my implementation. I understand why you had to take my styles out. It works better this way.
@Bouncey  @raisedadead. I am so sorry for all the issues I caused with night mode. I didn't qa my code as properly as I should have. It was one of my first open source contributions, and I was too eager to design the code editor. I'll stick to more of the beginner issues from now on to get a better understanding of the codebase.

@renojvarghese @thatnini @huyenltnguyen
Do not worry, I would rather recommend on working on anything that you wish like. This is Git. Everything is detectable and reversible. So breaking things is encouraged and OKAY. This is the only way we can learn ensure good quality.
Thanks a lot for for working on the getting the night mode working. Keep up the good work.
Cheers and Happy Contributing.


Cannot observe the crash in simple C program

Hi, I am trying to use aflgo on the following simple program. I followed the steps described here and I set the target as line 13 (i.e., the statement val = 1;). The following distance values are obtained after instrumentation: test.c:13,1.0 test.c:6,2.0. I am able to run the fuzzer, however, I am not able to observe the crash in this program. The abort statement is never reached even when the initial seed should be able to reach it. I am using LLVM version 11. Is there something that I am missing?
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>

int main(int argc, char** argv) {
  char buf[8];
  read(0, buf, 8);

  int val = 0;

  if (buf[0] == '2') {
    val = 1;
  }

  if (val == 1) {
    abort();
  }

  return 0;
}

This is likely due to the abort statement not leading to a signal that is recognized as a crash by AFL.
Not tested, but you could manually raise a SIGSEGV: raise(SIGSEGV)


add epilog

This PR adds an epilog to the options (like ARGPARSE_DESCRIPTION is optional)

@aberezin - thanks for the contribution! Can you please update your changes to the test code to retain python2 compatibility?
% python2 test.py
  File "test.py", line 12
    def run(script,*args,expect_exit_1=False):
                                     ^
SyntaxError: invalid syntax

also, I try to stick with with PEP8 formatting rules. I'd be happy to merge an updated PR.


split up config file

Pull Request Test Coverage Report for Build 961341710

11 of 19 (57.89%)  changed or added relevant lines in 14 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 77.476%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/config/commands.ts
0
1
0.0%

src/ui/modules/closeOnEsc.ts
0
1
0.0%

src/index.ts
0
6
0.0%

Totals

Change from base Build 961291425:
-0.2%

Covered Lines:
370

Relevant Lines:
480

💛  - Coveralls


Could not find executor for data source type: grafana-influxdb-flux-datasource

2020-05-22T21:53:03+0000 lvl=eror msg="Request Completed" logger=context userId=1 orgId=1 uname=admin method=POST path=/api/ds/query status=500 remote_addr=<IP_ADDRESS> time_ms=11 size=34 referer="http://grafana.everyonecounts.de/explore?orgId=1&left=["now-1h","now","Flux (InfluxDB) [BETA]",{},{"mode":"Metrics"},{"ui":[true,true,true,"none"]}]"
t=2020-05-22T21:53:05+0000 lvl=eror msg="Metric request error" logger=context userId=1 orgId=1 uname=admin error="Could not find executor for data source type: grafana-influxdb-flux-datasource" remote_addr=<IP_ADDRESS>
Using Grafana 7.0.0 and Flux 7.0.0.

I have no tried to build it with yarn for my arm64, but still missing gpx_influx_linux_arm64.
t=2020-05-22T22:50:05+0000 lvl=eror msg="Failed to start plugin" logger=plugins.backend pluginId=grafana-influxdb-flux-datasource error="fork/exec /opt/grafana-influxdb-flux-datasource/dist/gpx_influx_linux_arm64: no such file or directory"

ahh -- the problem is we do not yet have an arm build!

Tried to build, but seems to fail, in the nd I have no /dist/gpx_influx_linux_arm64.
ubuntu@ip-172-31-47-213:/opt/grafana-influxdb-flux-datasource$ sudo yarn install
yarn install v1.22.4
[1/4] Resolving packages...
[2/4] Fetching packages...
warning Pattern ["@grafana/ui@next"] is trying to unpack in the same destination "/usr/local/share/.cache/yarn/v6/npm-@grafana-ui-7.0.0-beta.3-5a664b91c1e155205ea67bfb53bffb8f14f7db27-integrity/node_modules/@grafana/ui" as pattern ["@grafana/ui@7.0.0-beta.3","<EMAIL_ADDRESS>This could result in non-deterministic behavior, skipping.
info<EMAIL_ADDRESS>The platform "linux" is incompatible with this module.
info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
info<EMAIL_ADDRESS>The platform "linux" is incompatible with this module.
info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
[3/4] Linking dependencies...
warning "@grafana/ui ><EMAIL_ADDRESS>has unmet peer dependency "@storybook/api@>= 5.x".
warning "@grafana/ui ><EMAIL_ADDRESS>has unmet peer dependency "@storybook/addons@>= 5.x".
warning "@grafana/ui ><EMAIL_ADDRESS>has unmet peer dependency "@storybook/components@>= 5.x".
warning "@grafana/ui ><EMAIL_ADDRESS>has unmet peer dependency "@storybook/core-events@>= 5.x".
warning "@grafana/ui ><EMAIL_ADDRESS>has unmet peer dependency "@storybook/theming@>= 5.x".
warning "@grafana/runtime > @grafana/ui > @grafana/slate-react ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "@grafana/toolkit ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>[4/4] Building fresh packages...
Done in 329.97s.
ubuntu@ip-172-31-47-213:/opt/grafana-influxdb-flux-datasource$

flux support was added to grafana 7.1 (released today :tada: )
https://www.influxdata.com/blog/how-grafana-dashboard-influxdb-flux-influxql/
that has ARM support, so you should be good


Improve documentation for ClusterConfig section

I'm trying to increase the timeout of the ScaledownIdletime.
I added the following ClusterConfig/SlurmSettings/ScaledownIdletime to my cluster config file:
ParallelClusterConfig:
  Version: 3.11.1
  Architecture: x86_64
  Image:
    Os: rocky8
    CustomAmi: ami-0d68c6538XXXXXXX  # pcluster-3-11-1-Rocky-8-x86-64-ami-0d002XXXXXXXXXX 2024-10-24T03-12-55.412Z
  DisableSimultaneousMultithreading: true
  ClusterConfig:
    SlurmSettings:
      ScaledownIdletime: 20

I've discovered this is the wrong syntax.  Your documentation only states make ClusterConfig a dict.  I look at the config_schema.py and not much there to go on either.  I've tried multiple variations, including:
ClusterConfig:
  Scheduling:
    SlurmSettings:
      ScaledownIdletime: 20

just can't figure it out.  This latter example at least throws an error by the python code.
I have been able to get the simple case of tags to work:
ClusterConfig:
  Tags:
    - Key: Project
      Value: amazing

Can you please add some examples (specifically my need) to your documentation and this issue?
How do I add a section in the config file to change the ScaledownIdletime?

Your code looks correct. I'm testing it right now.
ClusterConfig:
  Scheduling:
    SlurmSettings:
      ScaledownIdletime: 20

I made the change in my configuration.
I downloaded the config file that get generated and confirmed that the setting shows up in the ParallelCluster config file.
I updated my cluster and it successfully updated the config and ParallelCluster.
I checked in slurm_parallelcluster.conf and confirmed that SuspendTime is set to 1200 seconds which is 20 minutes.
So I think that it is working.

PLBKAC. Solved.
Here is the error I was getting, it happens right after the AMI builds section is output (I've copied and pasted a bit of that here to give you context):
"Rocky": {
    "8": {
        "arm64": {},
        "x86_64": {}
    },
    "9": {
        "arm64": {},
        "x86_64": {}
    }
}

}
Traceback (most recent call last):
File "/proj/work/gwolski/aws-eda-slurm-cluster-3.11.1/source/app.py", line 31, in
CdkSlurmStack(app, app.node.try_get_context('stack_name'), env=cdk_env,
File "/users/gwolski/.local/lib/python3.11/site-packages/jsii/_runtime.py", line 118, in call
inst = super(JSIIMeta, cast(JSIIMeta, cls)).call(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/proj/work/gwolski/aws-eda-slurm-cluster-3.11.1/source/cdk/cdk_slurm_stack.py", line 143, in init
self.create_parallel_cluster_config()
File "/proj/work/gwolski/aws-eda-slurm-cluster-3.11.1/source/cdk/cdk_slurm_stack.py", line 2557, in create_parallel_cluster_config
self.parallel_cluster_config['Scheduling']['Scheduler'] = 'slurm'
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^
TypeError: list indices must be integers or slices, not str
Subprocess exited with error 1
I tried so many variants, I must have copied and pasted the wrong code in my issue here.  Here is the offensive code that caused the above error that I should have realized is wrong.
ClusterConfig:
  Scheduling:
    - SlurmSettings:
        ScaledownIdletime: 20

Note the '-' in front of the SlurmSettings.  Argh.   Damn (tired) user.  Never file a ticket when you are tired. Thank you.
I have now used the appropriate code, as you have shown, and I see the correct entry in the YAML file when downloaded with PCUI and also the value SuspendTime=1200 in the slurm_parallelcluster.conf file.  All good.


Login failed

Getting this everyime I try to login, and I'm positive the user/password is correct

[03/13/15 18:24:07] Login failed, -103 - Oh no! Your login temporarily failed, so please try again later. If your login continues to fail, please visit https://support.snapchat.com/a/failed-login :)

My script:

from SnapWrap import Snapchat
import logging
class CustomBot(Snapchat):
def on_snap(self, snap):
self.post_story(snap)
logging.getLogger("requests").setLevel(logging.DEBUG)
bot = CustomBot(*["xxxx", "xxxx"])
bot.begin()

Logging in using https://github.com/agermanidis/SnapchatBot/ works just fine

Thanks!


update pynacl/python versions and bump version to 11.0b16

Checklist

[ ] Pre-commit hook is installed to do formatting checks before committing code...etc
[ ] Tests included
[ ] Build pass
[ ] Documentation
[ ] Code format and docstrings

Codecov Report
Base: 69.64% // Head: 72.15% // Increases project coverage by +2.51% :tada:

Coverage data is based on head (0ba9a60) compared to base (0437550).
Patch has no changes to coverable lines.

Additional details and impacted files
@@               Coverage Diff               @@
##           development     #607      +/-   ##
===============================================
+ Coverage        69.64%   72.15%   +2.51%
===============================================
  Files               84       78       -6
  Lines             4246     3534     -712
===============================================
- Hits              2957     2550     -407
+ Misses            1289      984     -305

Impacted Files
Coverage Δ

jumpscale/clients/docker/docker.py
74.54% <0.00%> (-3.64%)
:arrow_down:

jumpscale/sals/nettools/__init__.py
63.66% <0.00%> (-2.16%)
:arrow_down:

jumpscale/sals/fs/__init__.py

jumpscale/clients/docker/__init__.py

jumpscale/servers/openresty/__init__.py

jumpscale/sals/process/__init__.py

jumpscale/tools/alerthandler/alerthandler.py

jumpscale/core/executors/__init__.py

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.


PyPI Package?

Hey, I really love your work here. It is super helpful for what I'm working on right now.
Are you guys planning to release a PyPI package of this in the near future?

There is no such plan in near future, the license is pretty flexible though, so anyone can wrap the code into a PyPI package of their own, I'm OK with that.

@dophist That's really kind of you! I'd love to help you wrap the code into a PyPI package when I get my tasks at hand done.


Rename plugin to extension

Rename plugin folder to extension
Rename test-plugin to test-extensions
Rename app to integration-test

is the plural on test-extensions a typo @melloware ?

Sorry yes it is!


Docs example

Pull Request
What type of PR is this? (check all applicable)

[ ] Refactor
[ ] Feature
[ ] Bug Fix
[ ] Optimization
[x] Documentation Update

Description
Check it out with browser.
file:///home/UR USERNAME HERE/rov-24/docs_output/gui/index.html
Related Tickets & Documents

Related Issue #
Closes #

@benjaminwp18 This is more of a demo so it doesn't you need to install ros-humble or anything just open the index.html in your browser.


receive: add float histogram support

As followup to https://github.com/thanos-io/thanos/pull/6071 and preparation for compact (downsampling) and rule native histogram support, which will both produce float histograms, we add float histogram support to the store chunk iterator and for the receiver.
This PR also includes improvements of the native histogram e2e test to support float histograms.
Additionally histogram to/from proto functions, which are taken from prometheus, were updated and remote_engine which uses the functions was fixed.
Part of https://github.com/thanos-io/thanos/issues/5907

Some suggestions otherwise LGTM. Sorry for the delay. And please don't use force push so that it would be easier to review 😄

Force pushed only for DCO fix and rebase on main. 😀


When digit 1 in binary has no information content: How to distinguish number "0" and number "1"?

Ahoj,
in your video around minute:second 16:02 you argue that the first digit in binary has no information content at all.
(You argue:

we can remove all leading 0s,
then the first leftover digit is always a 1, so counting information content it can be omitted.)

But if doing so: How do you then distinguish the number "0" from the number "1"?
Number "0" would be represented by the empty list already (deletion of leading zeros), so we need the digit 1 when representing the number 1 in order to distinguish it from the number "0".
I see your argument flawed in this point -- where am I wrong?
Regards!

The section of the video about multiplication tables (and their technical nonexistence in binary) uses the 0 digit on its own to represent 0.

What that is suposed to mean is that for any number written,
l..ll
l.llll
lll.l
or what have you, every single one starts with a 1, for it to be a number.  So, the proposal is to always drop that first 1 when writing and just assume its existence. These are the same numbers as above.
..ll
.llll
ll.l
So in all cases, including those where the number starts with 0, we know there is an assumed 1 at the start that is now written.  In binary this contains the same information either way, vut in any other base that first digit still contains some information, albeit less than the other digits.

Ahoj @LincolnatorTheGreat, whom do you reply to?
In case you reply to my initial post:

What that is suposed to mean is that for any number written, […] every single one starts with a 1, for it to be a number.

No. That assumption is false in its absolute generality:
ZERO does not start with a 1.
So, if we have e.g.
.
l
l.
ll

and we apply

the proposal is to always drop that first 1 when writing and just assume its existence.

we get

.
.
l

How we distinguish here

* the one in the 1st line and the one in the 3rd line?
* Regarding the one in the 2nd line, how do we know that there is a number at all?

Regards!

Though I am long late to this thread, I still think I have something to say, even if dreirund doesn't see.
I think the implication of the fact that "every non-zero binary number starts with a ", is not that everyone should start writing numbers with one less digit, but instead that it makes binary defeat every number larger than it when fighting the Information Content per Digit vs. More Digits battle.
That battle isn't really real, it's just one way of comparing bases which is often done wrong and which favors binary, not e or three.
It is in fact, impossible, without some kind of metadata (which would take exactly one bit of information), to write numbers with no leading 1, because numbers like 10 (Two, with its leading one to distinguish from zero) and 10 (Five, with its leading one removed) are identical, even as you go beyond the zero vs. one problem.
You could, in theory, dedicate an additional bit at the start of the number to telling the viewer if they need to add a leading one in their head, but then what's even the point because you're adding an extra bit either way.
Nobody is writing numbers with less digits, sadly, but some are getting radix economy wrong.

Your presumption that the intent is to write numbers with less digits is flawed, upon closer inspection of the video, which I had not done, seemingly nor the video's author, though they scripted the whole thing🤔.
They say in the video:

[On radix economy]
The key is that leading digit, carrying slightly less information than the rest.
In most bases, this slight decrease in information makes hardly any difference. But in binary, where the only two options for a digit are zero or one, excluding zero means it can only be one.
Since it can only be one value, it contributes no information whatsoever.

At this point you could reasonably say that there is a flaw, in that you must in fact include the leading 1 (or, when writing 0% of numbers, 0) when writing numbers down. However, the next part challenges that.

It still matters, of course - it contributes to arithmetic after all - but in terms of actually determining the number's value, it might as well not be there.

I think the implication of this sentence is that you do still have to write it (it contributes to arithmetic, after all), but when playing the mostly imaginary game of "Which input gets the lowest output when plugged into the radix economy formula!" (kind of like a certain game show) then binary wins.
I hope this helps, in rescrepanising* this discrepancy.
Good luck in your travels through the forest of all knowledge.
*That's a pretty fun word I just made up, isn't it?