rgw/logging: basic set of tests for bucket logging

bucket logging is implemented here:  https://github.com/ceph/ceph/pull/59808
~~https://github.com/ceph/ceph/pull/54815~~

add versioned bucket tests

test is passing. see: https://github.com/ceph/ceph/pull/60700#issuecomment-2493444419

why is it necessary to duplicate the s3 extensions file from the ceph repo? i'd rather not worry about trying to keep those in sync

why is it necessary to duplicate the s3 extensions file from the ceph repo? i'd rather not worry about trying to keep those in sync

the other (and better) option is to fetch it from the ceph repo, but in my case I had to have a way to fetch it from a branch on my ceph fork.

cherry-picked to ceph-master


Fixing #7264 issue

Fixing #7264 isuse.

@suraj-webkul
Not working, Still, duplicate addresses are getting saved.Please find the screenshot here

@suraj-webkul
This is working fine.

Conflicts not resolved properly take help from @Sanjaybhattwebkul.

@suraj-webkul
This is working fine

@suraj-webkul
This is working fine.


Can not install bgheatmaps

Dear @FedeClaudi , I can not install bgheatmaps  via  pip install bgheatmaps.
ERROR: Could not find a version that satisfies the requirement bgheatmaps (from versions: none)
ERROR: No matching distribution found for bgheatmaps

Dear @chaoshengt, you seem to have a typo. The right installation is
pip install bgheatmap

To be fair, the typo is in the readme @FedeClaudi

My bad! Typos fixed now, pip install bgheatmap should work.

My bad! Typos fixed now, pip install bgheatmap should work.

It is done, Thank you! :)


feat: shadow commit for unify enums into one linking status

This is an empty commit in order to create a release that contains https://github.com/tuist/XcodeGraph/pull/32

There was a bug in the release job logic: https://github.com/tuist/XcodeGraph/pull/38
This PR shouldn't be necessary anymore.


Parse and insert keycloak roles scopes into JupyterHub

Reference Issues or PRs
Fixes #2432

What does this implement/fix?
Put a x in the boxes that apply

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds a feature)
[ ] Breaking change (fix or feature that would cause existing features not to work as expected)
[ ] Documentation Update
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build related changes
[ ] Other (please describe):

Testing

[ ] Did you test the pull request locally?
[ ] Did you add new tests?

Any other comments?

Do you know how long the login sequence with the REST API call takes? Should we add a log message printing out delta in milliseconds to be able to monitor this in logs?

I have not measured it but I didn't notice any observable latency. Sure, worth adding the time delta in logs.

Last time some pytest tests failed in local deployment run, let's see if those pass this time before merging.

Yes, I noticed tests/tests_deployment/test_jupyterhub_ssh.py::test_simple_jupyterhub_ssh failed, I am guessing it's a bit flaky, lets see though. 🤞


Ability to retrieve Allowance Module instance

Context / issue
We are manually retrieving the Allowance Module contract instance whereas other deployments are loaded via the SDK.
Proposed solution
Implement the ability to load the Allowance Module contracts via the SDK.

I would say this task should stay out of the Safe Core SDK and stick to the Safe core contracts included in safe-deployments  package.
Although the Core SDK is used by our web interface, its functionality should remain generic and not implement logic for custom modules, guards, tokens, etc.
If the allowance module is added to safe-deployments  package, with all the addresses for the different networks, etc. I'm okay with it, but in general I think it should work only with the Safe core contracts.


[Fixes #2056]Adds selenium tests for checking the functionality of Add to calendar…

… button

Checklist

[x] I have read the Contribution & Best practices Guide.
[x] My branch is up-to-date with the Upstream development branch.
[ ] I have added necessary documentation (if appropriate)

Short description of what this resolves:
Adds selenium tests for checking the presence and functionality of Add to calendar button.
Fixes #2056

@geekyd @ParthS007 @shubham3211 please review.


Get xiaomi message for xxxx failed

All time I have this error:
This error originated from a custom integration.
Logger: custom_components.xiaomi_miot.sensor
Source: custom_components/xiaomi_miot/sensor.py:569
Integration: Xiaomi Miot Auto (documentation, issues)
First occurred: 09:35:06 (193 occurrences)
Last logged: 12:52:38
Get xiaomi message for<PHONE_NUMBER> failed: {'code': 0, 'message': 'ok', 'result': {'messages': None}}

Your Mihome APP account has not received any device or system notifications.

@al-one can you please explain the reason for this message? my log got spammed with this message 24/7

@al-one +1, Why the issue is closed?

Try master branch.

it seems that the error disappeared on the master branch. thank you, @al-one <3

@al-one In master the particular problem is gone. However, there also yet another warnings spam from miot logs. There's a ton of issues and I don't know Chinese at all. Not sure was it reported or not. According to debug the resource unreachable from different region IPs, so, shouldn't be a rare case, just left it here.
Logger: custom_components.xiaomi_miot.core.xiaomi_cloud
Source: custom_components/xiaomi_miot/core/xiaomi_cloud.py:204
Integration: Xiaomi Miot Auto ([documentation](https://github.com/al-one/hass-xiaomi-miot), [issues](https://github.com/al-one/hass-xiaomi-miot/issues))
First occurred: 13:48:05 (68 occurrences)
Last logged: 16:03:58

Request xiaomi api: v2/message/v2/typelist {} failed, response: None

You can now disable the Mihome notification entity from the integration options if it doesn't work for you.


Error while buidling locally

I am trying to build this "lambda-directly-optimized" using cargo lambda watch and getting below error..
Compiling rustc_version v0.4.0
Compiling strum_macros v0.25.3
Compiling hdrhistogram v7.5.2
error[E0282]: type annotations needed for Box<_>
--> /Users/ps/.cargo/registry/src/index.crates.io-6f17d22bba15001f/time-0.3.30/src/format_description/parse/mod.rs:83:9
|
83 |     let items = format_items
|         ^^^^^
...
86 |     Ok(items.into())
|              ---- type must be known at this point
|
help: consider giving items an explicit type, where the placeholders _ are specified
|
83 |     let items: Box<_> = format_items
|              ++++++++
Compiling http-body v0.4.5
Compiling deno-proc-macro-rules-macros v0.3.2
Compiling cmake v0.1.50
Compiling encoding_rs v0.8.31

I upgraded the packages and that resolved the issue.


[CPDEV-95355] add_node and remove_node if there are notready workers

Description
It is not possible to run add_node and remove_node if there are unavailable workers in cluster.yaml.
Solution
The accessibility check will be performed only for control-plains, balancers and new nodes.
The same for prepare.check.sudoer task
coredns-configmap.yaml and coredns-deployment-patch.yaml will save only on control-plains
Excluded offline nodes for the cache_packages task
Updating etc_hosts will not be executed for offline nodes during the prepare.dns.etc_hosts task
Test Cases
Steps:

Stop worker node in the cluster
Run  add_node for new VM
Run  remove_node for new VM

Results:

Before
After

failed
succeeded

Checklist

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] Integration CI passed
[x] Unit tests. If Yes list of new/changed tests with brief description
[x] There is no merge conflicts

Added the test_remove_node_if_worker_offline and test_add_node_if_worker_offline to test add_node and remove_node procedures if there are offline workers in the cluster.

Run add_node for new VM with --exclude='prepare.dns.etc_hosts'
Run remove_node for new VM with --exclude='update.etc_hosts'

Maybe add explicit check inside these tasks if they are run with workers not being online, and raise exception. Alternatively, and more complex, add the check before running of the flow.

Run add_node for new VM with --exclude='prepare.dns.etc_hosts'
Run remove_node for new VM with --exclude='update.etc_hosts'

Maybe add explicit check inside these tasks if they are run with workers not being online, and raise exception. Alternatively, and more complex, add the check before running of the flow.

I think that until we change the entries scheme for etc_hosts, we should just exclude offline nodes from system_prepare_dns_etc_hosts for now. I've done this.


Quaver Bot Made from MemoryLib confusion

So, I couldn't contact the original user because they archived the thing. It's called crochet and it's a quaver bot made with a modified version of memory lib, I have no idea how to run it and I'm very very confused, I haven't done much coding before and I want to learn how so I can run this thing.

this is not the repository for crotchet. if they archived the repo, they are most likely not willing to support it and i have no experience with it.
cc @Azukee


Lock in static node version.

Testing framework fails on later versions of node and should be updated.
This will lock node version at the latest stable point until updates to testing are done.

I believe tests are failing do the the node version Travis is using.
It seems I'm unable to lock it in though.

Thanks fo this!
You'll want to edit the travis.yml file rather than .nvmrc.
https://docs.travis-ci.com/user/languages/javascript-with-nodejs/#Even-more-Nodejs-versions

I just looked at one commit and not the pr as a whole. You did the right thing but I believe Travis doesn't test changes to the Travis settings in PRs for security reasons. We will need to merge before it passes. I will verify and merge if confirmed.


Dropout is not required to apply to hidden states

Usually LSTM does not apply dropout to hidden states.

LGTM!

FYI, although LSTM does not usually apply dropout to hidden states, there is a form of  (Variational) dropout whereby the same dropout mask is applied to the hidden states at every time-step.

I think dropout is applied when layer > 1.
 if layer > 0:
   # At first layer, dropout will not be applied?
    x = dropout.dropout(x, ratio=dropout_ratio, train=train)

Yes, I know that there are a lot of variations of LSTMs. I'm trying to make compatible implementation with cuDNN's one.


FAST: Clear default resourcemanager.projectCreator and billing.admin authoritatively at the root of the org

For tracking: on freshly created GCP organizations, there's a default "domain:" level binding at the root of the org for the roles resourcemanager.projectCreator and billing.admin. FAST should authoritatively manage these roles, removing these default (explicit) grants.

projectCreator is already managed authoritatively. We only have to do the same for roles/billing.creator


snapshot fails due to tombstone file

Bug Report
What did you do?
Try to create a snapshot with the admin api curl -XPOST http://prometheus-server:80/api/v1/admin/tsdb/snapshot
What did you expect to see?
Success message.
What did you see instead? Under which circumstances?
"error": "create snapshot: error snapshotting block: /data/00019T5RPWJRT35RKQFTF1D7V0: create snapshot tombstones: link /data/00019T5RPWJRT35RKQFTF1D7V0/tombstones /data/snapshots/20190529T094304Z-129c08ec7fd4bec4/00019T5RPWJRT35RKQFTF1D7V0/tombstones: no such file or directory"

The tombstones directory does not exist on disk but the snapshot still tries to snapshot it.
Environment

Prometheus version:
  version 2.9.2

Can you replicate this every time?
In general a tombstone file is created with every block.

ping @vandewilly

closing for no activity. Feel free to reopen if you find the time to continue troubleshooting.


Enabling dropout layers to be used in the actor and critic models.

The K.learning_phase() flag was not being passed correctly to k.function(), resulting in AttributeError: 'Tensor' object has no attribute '_keras_shape'  since the return value of K.learning_phase() was getting treated as normal input tensor to the model. This change fixes this. The reason for this is the segment self.actor(critic_inputs). the actor should not receive K.learning_phase()

Spotted some errors will open it if I can fix them.

Hi @matthiasplappert can we put this through?

I see, makes sense. Why is the special case for Theano necessary though? Shouldn't the same argument equally apply to Theano?

@matthiasplappert I don't really know. Some how Theano expects the learningPhase flag all the time. I was getting errors otherwise.

Alright, merging this since tests seem to be fine.


cli: create abstraction for interacting with SCM web services

As part of project https://github.com/systemslab/popper/projects/9, this issue will track the work needed to create an abstraction to interact with SCM web services (github, gitlab, bitbucket, etc.).
The main idea is to create a thin API such that most of the repo-related functionality in the utils.py file can run on any SCM service (gitlab, bitbucket, github). This can be done by leveraging the python wrappers for Github, Gitlab and Bitbucket REST APIs.

Using these REST APIs might require defining some place to save auth details for these. Probably somewhere separate from popper.yml since you probably don't want these sitting in plain text in version control.

@eltrufas thanks for the feedback! At some point the popper archive implemented the functionality where it would store this in a file locally in the repo and put the name of that file in .gitignore. We'd need to go back on the commit history to look for this

Gave this a bit more of thought, I've updated the title and description as a result. It simplifies things as we don't need to use a REST API anymore (we do everything through Git)

Hey ivo, i have an issue, im not sure how to handle urls from different services. Before we could do user/repo/dir-path but now im not sure how to handle a generic git url without breaking back compatibility.
Here is my code: https://github.com/barbaraDorame/popper/tree/issue-483
Thanks in advance!

hey @barbaraDorame , I left some comments on https://github.com/barbaraDorame/popper/commit/ac75e00c22f1666ece9ea1e737070941a8d2c873 . Please take a look


Add new Style/AccessorGrouping cop

Closes #3983
# EnforcedStyle: grouped (default)

# bad
class Foo
  attr_reader :bar
  attr_reader :baz
end

# good
class Foo
  attr_reader :bar, :baz
end

# EnforcedStyle: separated

# bad
class Foo
  attr_reader :bar, :baz
end

# good
class Foo
  attr_reader :bar
  attr_reader :baz
end

Updated.

FYI, this cop is not compatible with sorbet. See this example
This code:
class Original
  extend T::Sig

  sig { returns(String) }
  attr_reader :foo

  sig { returns(Integer) }
  attr_reader :bar
end

Is auto corrected to this:
class AutoCorrected
  extend T::Sig

  sig { returns(String) }
  attr_reader :foo, :bar

  sig { returns(Integer) }
end

@ghiculescu I'm not super familiar with Sorbet yet, I imagine the "separated" style is fine. Is it mandatory, actually?

Yes, that works. The problems are

that doesn't work if they return different types
we are autocorrecting to invalid syntax

To be fair, I'm not sure if it's rubocop's responsibility to deal with that. i'm just commenting so other sorbet users are aware of this.

I believe it warrants at least a note in the doc: #8828
I believe the cop already considers comments, but maybe our notion of "comment" should be expanded for Sorbet.


Static images are not cached

Calls to https://test.sram.surf.nl/static/media/*  return a "cache-control: private" header teruggeven which hurts performance.
We should make sure that static content is cached.
It seems nginx is serving content from /static directly, without going though flask.  I can't find where this cache header is set though.

Can be easily confirmed when creating a new CO on test and choose the option to "Select from gallery". The PNG files have a "cache-control: private" heading.

This is also an easy check:
$ curl -kI https://sbs.scz-vm.net/static/media/users.36978667c99f35360ce5605552b80fae.svg
HTTP/2 200
server: nginx/1.18.0
date: Thu, 11 May 2023 07:34:07 GMT
content-type: image/svg+xml
content-length: 581
last-modified: Mon, 08 May 2023 12:53:27 GMT
etag: "6458f0c7-245"
expires: Thu, 11 May 2023 08:34:07 GMT
cache-control: max-age=3600
cache-control: public, no-transform
accept-ranges: bytes
x-frame-options: sameorigin
x-xss-protection: 1; mode=block
referrer-policy: strict-origin-when-cross-origin
x-content-type-options: nosniff
content-security-policy: default-src 'self'; frame-src 'none'; form-action 'self' http://*.scz-vm.net; frame-ancestors 'none'; block-all-mixed-content;
feature-policy: accelerometer: none; ambient-light-sensor: none; autoplay: none; battery: none; camera: none; display-capture: none; document-domain: none; encrypted-media: none; execution-while-not-rendered: self; execution-while-out-of-viewport: self; fullscreen: none; geolocation: none; gyroscope: none; layout-animations: self; legacy-image-formats: none; magnetometer: none; microphone: none; midi: none; navigation-override: none; oversized-images: self; payment: none; picture-in-picture: none; publickey-credentials: self; sync-xhr: self; usb: none; vr: none; wake-lock: none; xr-spatial-tracking: none
set-cookie: SERVERID=sbs.vm.scz-vm.net|ZFyac; path=/; HttpOnly; Secure; SameSite=None
cache-control: private
strict-transport-security: max-age=31536000; includeSubDomains; preload;

Hmm.... it looks like it's not nginx but propbably haproxy that's injecting the extra cache-control header?
# curl -kI https://localhost:90/static/media/users.36978667c99f35360ce5605552b80fae.svg
HTTP/2 200
server: nginx/1.18.0
date: Thu, 11 May 2023 08:21:48 GMT
content-type: image/svg+xml
content-length: 581
last-modified: Mon, 08 May 2023 12:53:27 GMT
etag: "6458f0c7-245"
expires: Thu, 11 May 2023 09:21:48 GMT
cache-control: max-age=3600
cache-control: public, no-transform
accept-ranges: bytes

Works.


Fix: PHP 8.2 accessing (unused) dynamic property $breadcrumbs is depr…

…ecated in LinkSorterTest.php
This property was never used in the test and I believe was copied into the setup method for the test when the file was originally created using the same setup as the BreadcrumbsTest which is the only other test with the same chunk of code.  I removed the reference to the dynamic attribute and the associated use statement, and that test now passes successfully.

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

Fixed issues

Thanks!


Questionable implementation of "promotes" for EditorialWork and Trailer

The class "Trailer" promotes "only" Editorial group" and also "Editorial work".
Semantically, this is inconsistent, since as two statements this is contractictory:

Alternatively, this could be modelled using an "or"'d restriction, like:

Also, the class Editorial Work already declares "promotes Editorial work", which would be inherited to "Trailer".
However, the class "Trailer" declares that redundantly:
EditorialWork:
###  http://www.ebu.ch/metadata/ontologies/ebucoreplus#EditorialWork
ec:EditorialWork rdf:type owl:Class ;
                 rdfs:subClassOf ec:EditorialObject ,
                                 [ rdf:type owl:Restriction ;
                                   owl:onProperty ec:hasEditorialGroup ;
                                   owl:allValuesFrom ec:EditorialGroup
                                 ] ,
                                 [ rdf:type owl:Restriction ;
                                   owl:onProperty ec:hasEditorialWork ;
                                   owl:allValuesFrom ec:EditorialWork
                                 ] ,
                                 [ rdf:type owl:Restriction ;
                                   owl:onProperty ec:promotes ;
                                   owl:allValuesFrom ec:EditorialWork
                                 ] ;

Trailer:
###  http://www.ebu.ch/metadata/ontologies/ebucoreplus#Trailer
ec:Trailer rdf:type owl:Class ;
           rdfs:subClassOf ec:EditorialWork ,
                           [ rdf:type owl:Restriction ;
                             owl:onProperty ec:promotes ;
                             owl:allValuesFrom ec:EditorialGroup
                           ] ,
                           [ rdf:type owl:Restriction ;
                             owl:onProperty ec:promotes ;
                             owl:allValuesFrom ec:EditorialWork
                           ] ;

Thanks to Stefan Kollien here, who detected that during his app implementation.

Thank you, @alexander-schulze , for documenting this.
Obviously, the inconsistent use of ´promotes´ was introduced together with the class hierarchy of ´Trailer´. It is still true, that ´EditorialWork´ promotes ´EditorialWork´ or ´EditorialGroup´. ´Trailer´ is just a subclass of ´EditorialWork´ and can inherit the property ´promotes´.  Therefore the proposal is:

remove the 2 ´promotes´ class restrictions from ´Trailer´
make ´promotes´ a class restriction of ´EditorialWork´ for ´EditorialGroup´ OR ´EditorialWork´

Pull request will follow.


Set up Oracle Cloud Infrastructure account

As an alternative to CSC's servers, we would like to take a look into setting up an Oracle Cloud account to make use of their very free-tier ARM-based VMs (4 virtual cores + 24GB RAM).

Oracle Cloud Infrastructure account created. Credentials can be acquired from the relevant slack channel.
VM instance has also been created:


tracking down strings in mercurial not hooked up on gitbook

https://github.com/mozilla-l10n/localizer-documentation/blob/master/tools/mercurial/tracking_back_string_to_bug.md is referenced from https://mozilla-l10n.github.io/localizer-documentation/tools/mercurial/, but doesn't render.

Sadly GitBook only convert files referenced in summary.

https://mozilla-l10n.github.io/localizer-documentation/tools/mercurial/tracking_back_string_to_bug.html


how to split r,g,b channel image from Mat data pointer

I known Cv2.Split can complete this funtion,but some time,we have to use pointer convert to r,g,b channel,like this
           Mat image = Cv2.ImRead("E:\\animal.jpg");
            int out_h = image.Height;
            int out_w = image.Width;
            int channel_step = out_h * out_w;
            Mat rmat =new Mat(out_h, out_w, MatType.CV_32FC1, image.CvPtr);
            Mat gmat = new Mat(out_h, out_w, MatType.CV_32FC1, image.CvPtr+channel_step);
            Mat bmat = new Mat(out_h, out_w, MatType.CV_32FC1, image.CvPtr+2 * channel_step);

            rmat *= 255f;
            gmat *= 255f;
            bmat *= 255f;
            Cv2.ImShow("R",rmat);
            Cv2.ImShow("G", gmat);
            Cv2.ImShow("B", bmat);
            Cv2.WaitKey(0);
            Cv2.DestroyAllWindows();

but the code can not run well,how modify it to work normally?

You should first consider using Cv2.Split(), but you may find the following page helpful for other methods:
https://stackoverflow.com/questions/21619609/split-channels-from-rgb-array-in-c-without-opencv


It's possible use this library with Express ?

This is my code:
`
import * as express from 'express';
import * as bodyParser from 'body-parser';
import * as dialogflowController from './controllers/dialogflow';
const app: express.Application = express();
app.use(bodyParser.json());
app.post('/echo', dialogflowController.doActions);
const http = require('http');
const httpServer = http.createServer(app);
httpServer.listen(80, function () { // ascolta sulla porta 80 per comando: ngrok http 80
console.log('HTTP Started!');
});
// dialogflowController controller unit
import { Request, Response, NextFunction } from 'express';
const { WebhookClient, Card, Suggestion } = require('dialogflow-fulfillment');
export let doActions = (req: Request, res: Response, next: NextFunction) => {
console.log('here it works');
const agent = new WebhookClient({ req, res });
console.log('crashes to the previous line');
};
`
And this is the error
Error: Request can NOT be empty. [Node]     at new WebhookClient (D:\Progetti\node_modules\dialogflow-fulfillment\src\dialogflow-fulfillment.js:58:13) [Node]     at exports.doActions (D:\Progetti\dist\controllers\dialogflow.js:14:19) [Node]     at Layer.handle [as handle_request] (D:\Progetti\node_modules\express\lib\router\layer.js:95:5)

Solved:
https://stackoverflow.com/questions/50818512/how-use-dialogflow-fulfillment-nodejs-with-express/50818810#50818810


Soumyadeep/conf sauvola

Description

We propose a novel binarization technique that improves upon the Sauvola's algorithm by preserving more foreground information in the binarized document-images. The computational complexity of the proposed algorithm is the same as the Sauvola's algorithm, thereby making it suitable even for mobile-computing based applications.
Related Issues

Checklist:

[ x] I have followed the contribution guidelines and code style for this project.
[x ] I have added tests covering my contributions.
[x ] I have updated the documentation accordingly.
[ x] This PR is being made to staging and not master
[ x] I will squash merge this PR into staging

@soumyadeep9 Thanks for updating the pdf link. Please also remove the __pycache__ directory since I guess it's not needed, and I will merge then. Looks great!!

@soumyadeep9 Thanks for updating the pdf link. Please also remove the pycache directory since I guess it's not needed, and I will merge then. Looks great!!

It is done

@soumyadeep9 Thanks for updating the pdf link. Please also remove the pycache directory since I guess it's not needed, and I will merge then. Looks great!!

It is done

Great! It's merged now into staging! Great contribution!


Typo in the title of one link in the Standard Connectors section : "Blog" instead of "Blob" (Azure "Blog" Storage).

[Enter feedback here]

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

ID: 470961ba-3289-e684-f027-8f82e826b804
Version Independent ID: b31439c7-fbbf-9530-aee9-ec99e2a6328a
Content: Managed connectors for Azure Logic Apps - Azure Logic Apps
Content Source: articles/connectors/managed.md
Service: logic-apps
GitHub Login: @ecfan
Microsoft Alias: estfan

@alephenne, That's a good catch! Thanks for taking time to share your feedback. We have raised a PR and the changes will published soon.

@alephenne, The PR has been merged and the doc reflects the changes. Thanks again for bringing this to our attention. It's much appreciated.


remove engine details in POST /volumes/create API

This PR did:
remove engine details in POST /volumes/create API
Fixed #2260
Signed-off-by: allencloud<EMAIL_ADDRESS>

LGTM

ping @dongluochen @vieux @nishanttotla

PTAL @vieux @dongluochen @nishanttotla
I think it is a Bug currently Swarm has. We had better improve the API part.

LGTM


Small fixes for german locale

Some small fixes for the german translation :)
Thanks for the module!

awesome thank you!

Codecov Report

Merging #176 (bb42e8e) into main (2118d3c) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #176   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           33        33
  Lines         4567      4567
  Branches      1099      1030   -69
=========================================
  Hits          4567      4567

Flag
Coverage Δ

unittests
100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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 2118d3c...bb42e8e. Read the comment docs.


"Invalid aria prop aria-labelled-by" warning when using aria-labelledby attribute in  file

Is this a bug report?
Yes.
Did you try recovering your dependencies?
Yes.
Environment
System:
OS: macOS 10.14.3
CPU: x64 Intel(R) Xeon(R) CPU E5-1650 v2 @ 3.50GHz
Binaries:
Node: 11.7.0 - /usr/local/bin/node
npm: 6.8.0 - /usr/local/bin/npm
Browsers:
Chrome: 72.0.3626.109
Firefox: 63.0.3
Safari: 12.0.3
npmPackages:
react: ^16.7.0 => 16.7.0
react-dom: ^16.7.0 => 16.7.0
react-scripts: ^2.1.5 => 2.1.5
npmGlobalPackages:
create-react-app: Not Found
Steps to Reproduce

Spin up a new cra project with npx create-react-app [project-name]

Edit the src/logo.svg file to look like this.

Import the SVG element into the src/App.js file as a component with this:
import { ReactComponent as BadAttribute } from "./src/logo.svg".

Replace the <Logo /> component in src/App.js with this:
<BadAttribute className="App-logo" />

run npm run start or yarn start

Expected Behavior
An inline, accessible SVG element with the correctly spelled aria-labelledby attribute should be rendered in the DOM.
Actual Behavior
An inline SVG element with an incorrectly spelled aria-labelled-by attribute is rendered in the DOM.
This warning is produced in the console.
Reproducible Demo
Clone this repo to see the bug in action (fresh CRA project).
Thoughts
I have a feeling this is related to svgr, but I was unable to track down exactly where or how the magic is happening here. If someone can point me in the right direction, I'd love to investigate further and contribute.
In the repo linked above, I also included a correctly rendered aria-labelledby attribute on the component. This is a fine alternative for now, but I think one of the nice things about SVG is that it's a package deal. With the attribute added in JSX, it refers to markup outside the current working context, which isn't terrible but it's also not ideal. It could potentially mean refactoring every SVG that's included in the app for accessibility.

@neoziro Is this a bug in SVGR?

@iansu @neoziro I investigated a bit further. This appears to be reproducible in the SVGR playground.

If the bug is known and reproducible, can you please submit a new issue in SVGR? Feel free to submit a PR to fix it, I will review, merge and release quickly. It should be a little issue in hast-util-to-babel-ast.

Gladly! @neoziro Here's the newly created issue. I'll provide some more details there momentarily.

Hi there, just housekeeping: This issue has been resolved with this commit 😄


Merge target_groups_defaults

Resolves the issue brought up in #80.
Currently, a user must specify all target_group_defaults even if they wish to override just one value, ie:
target_groups_defaults = {
  cookie_duration = 86400
  deregistration_delay = 300
  health_check_healthy_threshold = 3
  health_check_interval = 10
  health_check_matcher = "200-299"
  health_check_path = "/my-custom-health-check-path"
  health_check_port = "traffic-port"
  health_check_timeout = 5
  health_check_unhealthy_threshold = 3
  stickiness_enabled = false
  target_type = "instance"
}

We should merge this module's defaults with those provided by the user, letting the user simply pass in whichever values that should be overridden:
target_groups_defaults = {
   health_check_path = "/my-custom-health-check-path"
}

This would be very useful, could someone look at Travis and figure out why the region variable is not being passed into the build?

Very useful!
Who can look into Travis CI?

Ping @brandoconnor

This is excellent. Thanks for your patience as I was on holiday for a good bit there. Reviewing now.


document cog.Path and cog.File

The new pydantic version of Cog includes two new custom input types cog.Path and cog.File. These should be documented in the Python API doc.

cog.File: a file-like object representing a file
cog.Path: a path to a file on disk

Note: cog.Path extends pathlib.Path, so model others can use it as as stand-in replacement for Python Path they're used to.

#452


✨ Adding cluster-template.yaml for clusterctl

What this PR does / why we need it:
This will enable us to use clusterctl generate config to create the sample configs for deploying a cluster.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #124
Signed-off-by: Chris Hein<EMAIL_ADDRESS>/milestone v0.1.x
/kind feature

/assign @charleszheng44

/assign @weiling61

/unassign @charleszheng44


Maven artifacts are empty

Hi,
I wanted to try out your library and added the dependency from maven central. Unfortunately, the artifact you pushed to maven central only contains a MANIFEST.MF, so something appears to have gone wrong during the build of your artifact.
See:
https://repo1.maven.org/maven2/com/github/ajozwik/akka-smtp-server_2.12/<IP_ADDRESS>/

I see the files. Maybe there are unavailable in your location?

The files are there but they only contain a Manifest File, no code.

Please use:
libraryDependencies += "com.github.ajozwik" %% "akka-smtp" % <version>


Numba does not error out when an if statement does not account for all conditions

[Yes] I have tried using the latest released version of Numba (most recent is
visible in the change log (https://github.com/numba/numba/blob/master/CHANGE_LOG).

[Yes] I have included a self contained code sample to reproduce the problem.
i.e. it's possible to run as 'python bug.py'.

I discovered that I had a hanging if statement which, when Numba was disabled, encountered a condition that should have been covered by an 'else' part of the conditional but wasn't, causing the function to fail. Numba, however, never threw an error, which led me to investigate why.
It turns out that Numba will implicitly assign and return the variable. I tried searching to see if this is expected/designed behavior but couldn't find anything.
I want to show that a variable will be successfully returned even in the event of a hanging if statement (incompletely enumerated conditions)
import numba
from numba import njit, types
import numpy as np

@njit(parallel=False)
def reproduce_bug_bool(test_condition_bool):

    if test_condition_bool:
        test_var = True

    return test_var

@njit(parallel=False)
def reproduce_bug_float(test_condition_bool):

    if test_condition_bool:
        test_var = 10.0

    return test_var

@njit(parallel=False)
def reproduce_bug_int(test_condition_bool):

    if test_condition_bool:
        test_var = 10

    return test_var

num_trials = 10
test_cases = np.asarray([True, False, True, False, True, False, True, False, True, False])

for t in range(num_trials):
    print("Test case: ", test_cases[t])
    returned_bool = reproduce_bug_bool(test_cases[t])
    returned_float = reproduce_bug_float(test_cases[t])
    returned_int = reproduce_bug_int(test_cases[t])

    print("Returned bool var: ", returned_bool)
    print("Returned float var: ", returned_float)
    print("Returned int var: ", returned_int)

All results seem both arbitrary and dangerous; I would expect them to throw errors.
In test case 1 (bool), the unaccounted-for condition implicitly takes on the same value as in the accounted-for condition (True). (I originally ran my code in version 0.46, and the value came back as False, which seemed dangerous but plausibly correct.)
In test case 2 (float) the unaccounted-for condition implicitly takes on the value of 0.0, rather arbitrarily.
In test case 3 (int), the unaccounted-for condition implicitly takes on the same value as in the accounted-for condition (10).

I can reproduce this behaviour. I'd expected to find a duplicate issue but haven't found one so far.

I was rather surprised not to find this already as well!

I can reproduce the strange behavior as well. I would like to contribute by writing a pull request to fix this problem directly. I believe @sklam is in charge with exception handling... where might the issue be caused?

I can reproduce the strange behavior as well. I would like to contribute by writing a pull request to fix this problem directly. I believe @sklam is in charge with exception handling... where might the issue be caused?

Unfortunately, determining where in the compilation process that this issue is caused will be the first step in writing a PR to fix the issue.

I can reproduce the strange behavior as well. I would like to contribute by writing a pull request to fix this problem directly. I believe @sklam is in charge with exception handling... where might the issue be caused?

Unfortunately, determining where in the compilation process that this issue is caused will be the first step in writing a PR to fix the issue.

Right. So if you try returning test_var without numba an error is thrown:
UnboundLocalError: local variable 'test_var' referenced before assignment.
I will look into the python documentation for the code snippet where this appears, and hopefully the implementation will be similar.

Tracking unbounded variables
@vpagano1, the handling of unbound variable will be very different from cpython. I tried to find the issue or discussion we had on how to track unbounded variables but I cannot currently find the record. I will try to explain the challenges in tracking unbounded variables in Numba.
In CPython, local variables are kept in the active frame as a list of PyObject*. (See LOAD_FAST, which is the most common case). The PyObject* can be NULL to indicate that it is never defined.
In Numba, local variables are stored as "plain-old data" like in a C program. An int is usually just 8-bytes. A ndarray is a C-structure. However, they are not nullable. To track unbounded variables, the compiler will need to insert additional storage for the "liveness" marker.
Currently, Numba does not track the liveness at runtime. It is always zero-initializing all variables at the start of the function.
Unexpected results
With the zero-initialization, I would expect the result to be stable. If the branch is not taken, all bytes in that variable should be zeros. Somehow, I think it is doing something different. For instance, the boolean function has the following in the cpython wrapper:
  %.33 = call i32 @"_ZN8__main__22reproduce_bug_bool$241Eb"(i8* nonnull %.28, { i8*, i32, i8* }** undef, i8 %.32) #2
  %.58 = call i8* @PyBool_FromLong(i64 1)   # constant one!?!?!?!
  ret i8* %.58

but the lowered function body has:
define i32 @"_ZN8__main__22reproduce_bug_bool$241Eb"(i8* noalias nocapture %retptr, { i8*, i32, i8* }** noalias nocapture readnone %excinfo, i8 %arg.test_condition_bool) local_unnamed_addr #0 {
entry:
  %.7.not = icmp ne i8 %arg.test_condition_bool, 0
  %.22 = zext i1 %.7.not to i8
  store i8 %.22, i8* %retptr, align 1
  ret i32 0
}

The actual function is correctly computing base on the input. But the wrapper is bypassing the returned value and always returning True. I think we have another bug.

I found the problem for the unexpected result. Type inference (or constant propagation) is incorrectly assuming the return type to be a constant because there is no other assignment.
Following is output from reproduce_bug_bool.inspect_types()
    # --- LINE 11 ---
    #   jump 10
    # label 10
    #   del $4pred
    #   $12return_value.1 = cast(value=test_var)  :: Literal[bool](True)
    #   del test_var
    #   return $12return_value.1

    return test_var

Liveness analysis is a nice feature for numba.
Shall we consider add a liveness analysis compilation pass for numba? I feel there should exist some mature liveness analysis algorithms. Perhaps we can choose one to implement.
I did meet some segfault issues at runtime, whose root cause is UnboundedError/NamedError in this test case.
I used gdb_init and etc to find the root cause, but it's much slower than if numba directly throws an error during compilation.


🚩: Milestone 9: Space Travelers' Hub | feature/rocketBooking

🚩: Milestone 9: Space Travelers' Hub | feature/rocketBooking - >>branch<< -
Here is a summary of what has been done:

Added the functionality to reserve and cancel a rocket.

In this Branch, I have MODIFIED the succeeding 📂* folders and 📄** files

📂* src/components/

**📄 Rockets.jsx

Added the condicional rendering for the button dispatch and the button classes.

📂* src/redux/rockets/

**📄 rocketsSlice.js

Created the reserverRocket and cancelRocket reducers to handle the state of the rockets.

Thank you for taking the time to review this PR ⭐!

STATUS: APPROVED! 🍾 🎆 🥇
Hi @Ruthmy
You did a great job working on this milestone, your milestone is complete! 🚀
There is nothing else to say other than... it's time to merge it 🤖
Congratulations! 🎉

Cheers, and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.


Allow running eclipselsp as installed by system package on GNU/Linux

This is a continuation of https://github.com/w0rp/ale/pull/2514
Thanks for the feedback on the last PR and please allow me to suggest a different approach based on the suggestions presented there.
I removed the code that creates the ~/.jdtls config directory and replaced it with an optional parameter that takes a path to the config directory. If this parameter is not present then ALE keeps the previous behavior of looking for the config directory within eclipselsp installation directory.
I've also added a mention to this in the documentation.
This way ALE has no side-effects and does no configuration for eclipselsp but still allows the users to have the configuration directory wherever they wish.
Is it acceptable like this?
Please note that the main goal of this PR is to have ALE working with eclipselsp when it is installed by a distro package. As is, ALE assumes that the config dir is within the installation dir. That can't be the case with a distro package because eclipselsp is installed to a non-writable dir like /usr/share/java/jdtls/.
Should I attempt adding any particular tests to cover these additions?
This implementation was inspired by https://github.com/w0rp/ale/pull/2520.
Thank you for your time.

Thanks a lot for the feedback, would the test included in the commit above be acceptable and desirable?

That's good. The Windows test fails because the test path needs to be simplified for Windows, but I'll fix that. I also noticed that the paths in the command aren't escaped, so I'll fix those.

Cheers! :beers:


Ripple-carry addition - arithmetic namespace functions

First portion of the arithmetic namespace functionality:

Added Ripple-carry little-endian addition operations to Arithmetic namespace: RippleCarryIncByL, RippleCarryIncByLE, and RippleCarryAddLE, and a few auxiliary internal operations.
Also added wrapper functions IncByL, IncByLE, and AddLE.
Added functions TrailingZeroCountI and TrailingZeroCountL to Math namespace and ApplyXorInPlaceL to Arithmetic namespace.
Added exhaustive and branching case tests. Large test functions are moved to separate .qs files.

Acknowledgements:

Thanks to @msoeken for providing most code and tests
Thanks to @swernli for help with fitting tests into our testing approach

Thanks for updating the tests to be more granular! As we discussed, I think we may want to do a similar refactoring to break up tests in other places in the libraries, but that is out of scope for this PR. I'll file an issue to cover that and we can continue that part of the discussion there.

Thanks for updating the tests to be more granular! As we discussed, I think we may want to do a similar refactoring to break up tests in other places in the libraries, but that is out of scope for this PR. I'll file an issue to cover that and we can continue that part of the discussion there.

Filed as #832


Change copy on deprecation page to "retired" from "deprecated" to hel…

…p non-english readers

@cadecairos can we point people to https://github.com/mozilla/popcorn-editor/issues, where we are working on a white labeled Popcorn Editor? Cc @nokski

@benrito we totally can and should!
@KevZawacki how can we word this?

@benrito @KevZawacki I filed #625 to work on linking to popcorn editor

I will set up a meeting to discuss; we may want to do some repo magic too

On Oct 15, 2015, at 10:47 AM, Christopher De Cairos<EMAIL_ADDRESS>wrote:
@benrito @KevZawacki I filed #625 to work on linking to popcorn editor
—
Reply to this email directly or view it on GitHub.

@cadecairos @benrito Happy to help with any wording. A bit in the dark on the context. Has someone adopted Popcorn? Are we linking to their iteration?

@KevZawacki The repo we're linking to is https://github.com/mozilla/popcorn-editor - My understanding is that it's the front-end timeline style editing environment we built for popcorn maker, but made to be embeddable in any context and without the back-end server bits that popcorn maker came with originally.
@benrito did I get that mostly right?

Yeah, that’s right; I am going to organize a meeting to figure out the overall approach, but for now it should be OK to add a single line to the README that says something to that effect.
@nolski can probably help come up with an appropriate line.

On Oct 16, 2015, at 11:46 AM, Christopher De Cairos<EMAIL_ADDRESS>wrote:
@KevZawacki https://github.com/KevZawacki The repo we're linking to is https://github.com/mozilla/popcorn-editor https://github.com/mozilla/popcorn-editor - My understanding is that it's the front-end timeline style editing environment we built for popcorn maker, but made to be embeddable in any context and without the back-end server bits that popcorn maker came with originally.
@benrito https://github.com/benrito did I get that mostly right?
—
Reply to this email directly or view it on GitHub https://github.com/mozilla/popcorn.webmaker.org/pull/624#issuecomment-148751364.

So, the updated copy might read something like: "You can read more about this on our blog. You can also adopt and tinker with the Popcorn Maker code here."

@KevZawacki That copy sounds good to me :)

Closing in favour of #626


Add checks from blueprint test_component,

allow fill assignment tables for exactly one circuit

waiting for: https://github.com/NilFoundation/evm-assigner/pull/22


[Issue-112] Upgrade pravega client version to 0.9.1

Signed-off-by: zhongle.wang<EMAIL_ADDRESS>Change log description
Upgrade pravega version to the latest commit pravega/pravega@26b543c
Purpose of the change
Fix #112
What the code does
Upgrade pravega version to the latest commit pravega/pravega@26b543c
How to verify it
./gradlew build passes

Let's change the target branch to r0.9-spark3.0 so that the naming convention is consistent with the other branches.


UnicodeEncodeError when downloading/decrypting using Reports().get_report_document()

Describe the bug
Python throws a UnicodeEncodeError exception when using Reports().get_report_document().
To Reproduce
Steps to reproduce the behavior:
# report request
createReportResponse = Reports(credentials=credentials
    ,marketplace=marketplaces.Marketplaces.CA).create_report(
        reportType="GET_BRAND_ANALYTICS_SEARCH_TERMS_REPORT",
        reportOptions={"reportPeriod": "WEEK"},
        dataStartTime="2021-10-03",
        dataEndTime="2021-10-09"
        )

# report id
report_id = createReportResponse.payload['reportId']
print("Report ID: ", report_id)

# get report (loop)
getReportResponse = Reports(credentials=credentials, marketplace=marketplaces.Marketplaces.CA).get_report(report_id=report_id)
while getReportResponse.payload['processingStatus'] != 'DONE':
    print("Report status is: %s. Sleeping for 10 seconds..." % getReportResponse.payload['processingStatus'])
    time.sleep(10)
    getReportResponse = Reports(credentials=credentials, marketplace=marketplaces.Marketplaces.CA).get_report(report_id=report_id)

# report document id
report_document_id = getReportResponse.payload['reportDocumentId']
print("Report document ID: ", report_document_id)

# create filename
output_file = report_document_id + ".json"

# get report document
getReportDocumentResponse = Reports(credentials=credentials,
    marketplace=marketplaces.Marketplaces.CA).get_report_document(
        document_id=report_document_id,
        decrypt=True,
        file=output_file,
        character_code='utf-8'
        )

Expected behavior
Report download, decrypts, and decompresses without throwing an exception.
Desktop (please complete the following information):

OS: Windows 10

Additional context
I've tried specifying different values for character_code such as iso-8859-1 but I still get an exception.

Can you please download the report without decrypting it and check it? Is the data in the report confidential, or can you share it?

The report is confidential so I cannot share it.
What should I check now that I've downloaded the report?
Thanks in advance for your help!
Also, why doesn't python-amazon-sp-api use the `2021-06-30' version of the SP-API for Reports which removes the need to decrypt reports?

I'm pretty sure I found the issue here: https://github.com/saleweaver/python-amazon-sp-api/blob/79a7e0b12052982cc49652c3e5693684ceaaa78b/sp_api/api/reports/reports.py#L116
Proposed change:
with open(file, "w", encoding=character_code) as text_file:

FYI: https://github.com/saleweaver/python-amazon-sp-api/pull/272

Thanks, that should fix it.
I'll generate the code for the new versions the next few days, encryption can be removed for Feeds as well.

Thanks.
How would a developer use pip to pull the latest version of reports.py?

Thanks.
How would a developer use pip to pull the latest version of reports.py?

You mean https://stackoverflow.com/questions/15268953/how-to-install-python-package-from-github?

I was more looking for how to pull the most up-to-date version of a package. When I tried the normal method (i.e. pip install python-amazon-sp-api --upgrade) I didn't get the latest version of the reports.py file.

I was more looking for how to pull the most up-to-date version of a package. When I tried the normal method (i.e. pip install python-amazon-sp-api --upgrade) I didn't get the latest version of the reports.py file.

Yeah, I haven't released yet. I'll push a release in a minute, you should be able to download using pip in a couple of minutes.
What you want to do is described in the stackoverflow answer, that would get you any state of the package you want, e.g. latest.

got it. Thanks!

I've pushed a version with the new client ReportsV2 if you want to try it.


Add more filters to resolve for index schema

Currently we only support equality predicate, we should also add range check and isNull.

Fixed in #18.


Fix hass.data (fixes #8288)

Description:
Change entry which was missed.
Related issue (if applicable): fixes #8288
Checklist:
If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass

Cherry picked for 0.48.1


Normalize ThemeData.cardTheme

Following https://github.com/flutter/flutter/pull/151914, this PR is to normalize ThemeData.cardTheme; change the CardTheme cardTheme property to CardThemeData cardTheme in ThemeData. The type change may cause breaking changes if ThemeData.cardTheme is used in apps, a migration guide will be created. There's no way to create a dart fix because we can't add a "@deprecated" label for CardTheme; CardTheme is a new InheritedWidget class now.
Addresses the "theme normalization" sub project within https://github.com/flutter/flutter/issues/91772
G3 Fix
This PR required a g3fix(Google internal) change: cl/661878832
Breaking Change
The type of ThemeData.cardTheme was CardTheme, and now is replaced with CardThemeData. If the app defined its own cardTheme in ThemeData, then it may have breaking changes. CardTheme still has .of(BuildContext) method but it returns CardThemeData now.
Before fix:
ThemeData(
  cardTheme: CardTheme(elevation: xxx, color: xxx),
)
...

final CardTheme cardTheme = CardTheme.of(context);

After fix:
ThemeData(
  cardTheme: CardThemeData(elevation: xxx, color: xxx),
)
...

final CardThemeData cardTheme = CardTheme.of(context);

Pre-launch Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I signed the [CLA].
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is [test-exempt].
[x] I followed the [breaking change policy] and added [Data Driven Fixes] where supported.
[x] All existing and new tests are passing.

Will wait until rest of the component themes(DialogTheme, InputDecorationTheme and TabBarTheme) have fixes ready to merge because I think it might be better to include them together in one breaking change migration.

While reading for learning purpose, I spotted some typos.

Thanks a lot for the suggestions:) Just updated!


fix spelling of 'readme.md' -> 'readme.MD' in pyproject.toml

Otherwise a fresh poetry install would error by mismatch in md != MD:
$ poetry install
Installing dependencies from lock file

Package operations: 49 installs, 1 update, 0 removals

  • Installing click (8.1.7)
...
...
  • Installing types-toml (<IP_ADDRESS>)

[Errno 2] No such file or directory: '/home/stefano/personal/Datastax/code/other_clones/astrapy/README.md'

Not the right solution (If you do that, poetry build doesn't use the readme for the package description)


Onebox and oEmbed support

Resolves #181 by nicely formatting links to a project on the same server, on a single line.
Adds oEmbed support using the micawber library.
Additional updates to the project include Project fixtures and tests, Talisman configuration, and improvements to Responsive Web support for the new FlipClock release.
There may be some minor breaking changes to theme customizations in this release.

@gonzalocasas deployed to now.makezurich.ch


Upgrade to OCaml 4.08+beta2

For reference, this is what OCaml 4.08+beta2 seems to require (issue #69).

I don't understand why Travis CI is complaining, and I don't have the time to investigate.

cc @xclerc

@jhwoodyatt I opened a PR on your fork to fix travis: https://github.com/jhwoodyatt/ppx_tools/pull/1

all green. cc @alainfrisch @gasche (not sure who's maintaining this)

I know I'm not, but maybe I can try to have a look later in the week if nobody has stepped up. (cc @alainfrisch @nojb @xclerc @diml)

It's embarrassing that I don't know how to check the list of people with merge rights (to me or to the Github interface), but it may be that only @alainfrish is in this set. I would warmly recommend to add @nojb to it, though :-)

I have merge rights! But I can't add more people to the repository.
I also have very little time right now, so I won't be able to make a release myself for a few weeks.

While testing the master branch on 4.08 I found a small issue -- the generated code still mentions Pervasives -- for which I propose a fix in #72. Would someone look at it and consider merging it?

Shouldn't be this merged one already? It is pretty obvious it is good one.

I believe metaquot is currently broken with this feature.
One should ignore pexp_loc_stack, ptyp_loc_stack, ppat_loc_stack when generating patterns on the ast. see what's done for location and attributes: https://github.com/ocaml-ppx/ppx_tools/blob/master/ppx_metaquot.ml#L175
Also see the patch in ppx_tools_versionned https://github.com/ocaml-ppx/ppx_tools_versioned/pull/22/files#diff-7ab097557bb7183454f885e4595acc38R180


Request: make Flutter able to pass a callback function to iOS native side.

Use case
Our SDK build with Flutter, it's able to call the native side (iOS and Android) through the bridge. Because iOS resist all the app which include idfa things, so we decided to take the idfa logic out, let customer pass the callback function to the Flutter SDK. Then we will pass the customer designed function block to our iOS native side.
After did the research, I found the way to define the callback function is using typedef, but I didn't find any useful info or example to actually pass any callback function to iOS. So I have this feature request.
//in Flutter, .dart file
typedef adSupportBlock = string Function();  // define the idfa callback function, no argument, return string back.
void setCallback(Function() f){
        adSupportBlock = f;
}
//in the dart app somewhere, we pass the an anonymous function in which is defined by the customer to the adSupportBlock
setCallback(() -> {
   return 'teststring';
})

/*
Most of the case, in flutter dart, we just map the function name to the native side function name using invokeMethod(), and pass the native side needed arguments in. And in the iOS bridge (.swift file), when the call match the function name, we call the native SDK directly. Like the following:
   //in .dart
   final MethodChannel _channel = const MethodChannel('flutter_sdk');
   Future<void> setOptOut(bool optOut) async {
       Map<String, dynamic> properties = _baseProperties();
       properties['optOut'] = optOut;
       return await _channel.invokeMethod('setOptOut', jsonEncode(properties));
  }
   //in .swift
   public func handle(_ call: FlutterMethodCall, result: @escaping FlutterResult) {
      ...
      case "setOptOut":
           let optOut = args["optOut"] as! Bool
          TestSDK.instance(withName: instanceName).optOut = optOut
          result(true)
      ...
   }

*/
But is it possible to pass the callback function or function pointer to native side? And what will the .swift iOS native side looks like?
//in .dart file
Future<void> adSupportBlock(Function adSupportBlock) async {
      return await _channel.invokeMethod(??? how to pass the adSupportBlock to native);
}
//in .swift file
 public func handle(_ call: FlutterMethodCall, result: @escaping FlutterResult) {
    ...
    case "adSupportBlock":
        ???? how to get the callback from flutter side?
        result(true)
    ...
 }

Any update?

But is it possible to pass the callback function or function pointer to native side?

This is not possible; the way to get this kind of behavior would be too keep the callback on the Dart side, and pass something that lets you refer to it (e.g., an ID it is stored under), then use method channels to trigger that.
The menubar plugin, for instance, uses that pattern.


How to obtain Books3 dataset?

Hi, thanks for the awesome work. However, during dataset collection, I found Books3 in RP has already been removed due to copyright issue. Is there any other possible resource for this dataset? I thought it was indispensible and significant for the training process as you mentioned in your paper.

Thanks for the fast reply! May I confirm once again that you made RP-filter (section 7 page 8) using only PG19 for books domain (without any S3 data)?


Add r/rerun option to runner

Would be nice to be able to re-run a started day w/o having to quit.

never mind, I now see that it does that automatically when code was changed.  Only question then is: if I use an IDE which automatically saves the code after every change, would that not re-run that way too often while I edit the code?


[AnnotationsToAttribute] Skip not in test on AnnotationWithValueToAttributeRector

Like other rules, the AnnotationWithValueToAttributeRector should verify in test class as well.

All checks have passed 🎉 @TomasVotruba I am merging it ;)


ARXIVNG-1408 cross-list support

This introduces support for cross-list requests. This is a little different from adding cross-list categories during a new or replacement submission. What we're doing here is creating a new request (on the submission as Submission.user_requests: List[UserRequest]) with a set of additional cross-list categories that the user would like added to an existing version.
Here's an example with a new submission that I started (got through classification steps), and then used the /{submission_id}/publish shortcut to artificially "publish" it.

The template needs a little work (looking for your direction @eawoods), but here's what the cross-list view looks like at the beginning.

You can select a category that is not already on the submission, and add it to the request.

You can add a bunch.

And then submit the request.

I added a route (dev/test only) for artificially "applying" the request. This is equivalent to the cross-list submission getting announced in the classic system. You can apply a cross-list request by making a GET request to /{submission id}/apply_cross. This is just for testing so there isn't any visual feedback, but notice that the status of the request has changed.

You can also simulate rejection by visiting /{submission id}/reject_cross. Again, dev/test only. This will even work if you've already "applied" the request. Note the change in request status.

There may be some bugs lurking here or there. Don't forget to re-install dependencies, and consider starting with a fresh sqlite db.

Pull Request Test Coverage Report for Build 424

76 of 205 (37.07%)  changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-3.6%) to 71.058%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

submit/filters.py
5
6
83.33%

submit/controllers/util.py
4
6
66.67%

submit/controllers/classification.py
8
12
66.67%

submit/routes/ui.py
15
33
45.45%

submit/util.py
4
36
11.11%

submit/controllers/cross.py
38
110
34.55%

Totals

Change from base Build 418:
-3.6%

Covered Lines:
1505

Relevant Lines:
2118

💛  - Coveralls


Link request

could you give the link to the original repo? Can't find it on github right away

All tasks have been described by me, so it's unlikely you'll find them anywhere else but in this repository. I've spent a long time gathering employee evaluation structures from various sources and tailored them to suit the specific needs of my department. You can find the my final structure in this repository. Whether it suits you or not, it's up to you to decide :)


Microsoft HEVCVideoExtension is included in Windows 11 22H2

Description
Microsoft.HEVCVideoExtension is now included by default in Windows 11 22H2, and therefore it can be removed from the provisioning packages.
Context
Looking at insider build 22621 from May it seems that Microsoft.HEVCVideoExtension is now included by default in Windows. That is nice, if this makes it to stable. Previously you had to pay for the HEVC extension, or sideload the free manufacturer only version which is the one listed above.

Store Extension
Package
Price

HEVC Video Extensions
Microsoft.HEVCVideoExtensions
$0.99

HEVC Video Extensions from Device Manufacturer
Microsoft.HEVCVideoExtension
Free

Notice the omission of the s, in the free and now included package name.
Originally posted by @aisgbnok in https://github.com/aisgbnok/provisioning/issues/30#issuecomment-1207312892

FYI, I verified on a clean ISO from Microsoft that this is true again. Just to make sure nothing changed in the past few months.


[bug]: Marketplace icons are not displaying

Prerequisites

[X] I have searched for existing issues that already report this problem.

embassyOS Version
next
Device
Laptop/Desktop
Device OS
MacOS
Device OS Version
all of them
Browser
Firefox
Browser Version
1
Current Behavior

Expected Behavior
a
Steps to Reproduce
a
Anything else?
a

This is actually not a problem. Those icons can be expected to always be there once the registry is updated.


Unknown route switching language

Hi @meeroslav integrating your localize-router I found what I believe is an issue, correct me if I am wrong, what I am trying to do is that: when a user click on a new language the page is translated and the navigation is updated to march the new language, in summary the basic idea of this module.
The way the feature is implemented is that:
I created 2 buttons and on click I invoke a function on my module that using the LocalizeRouterService#changeLanguage method set the new language.
HTML
<span class="pull-left ">
      <button style="padding-left: 0 !important; " class="nav-link d-inline " (click)="switchLanguage( 'en') ">
          English
      </button>
      <button style="padding-left: 0 !important; " class="nav-link d-inline " (click)="switchLanguage( 'it') ">
          Italian
      </button>
</span>

Component
@Component({
  selector: 'app-nav',
  templateUrl: './nav.component.html',
  styleUrls: ['./nav.component.css']
})
export class NavComponent implements OnInit {

  constructor(private localize: LocalizeRouterService) { }

  ngOnInit() {
    console.log('ROUTES', this.localize.parser.routes);
  }

  switchLanguage(language: string) {
    this.localize.changeLanguage(language);
  }

}

What I observe is this when I click the button the page is translated and the links are updated to the new language as well but when I try to click on any of the links the route is not recognized and the following error is thrown

Error: Cannot match any routes. URL Segment: 'it/howitworks'

doing some tests I can see that the error goes away and all work fine if after I click the button I manually change the language on the address bar and refresh the page with the new URL.
I spent few minutes debugging and I believe the issue is related to the LocalizeRouteParser#_translateRouteTree method that on line 163 of the master branch should invoke the LocalizeRouteParser#_translateProperty setting to true the 3rd prefixLang parameter.
This is because, as far as my understanding goes, in this case the route will be prepended with the new language and as a result the routes configuration will be updated and back on LocalizeRouterService#changeLanguage the browser will be refreshed with the new route and as a result the system will start working correctly.
I am not sure if there are other implications that is something that need to be considered.
Looking at the bug list I believe this is a duplicate of #139 but I'll leave this to you to judge, in the meantime I will update my switchLanguage method so that it refreshes the browser page with new language and update on this.
Thanks,
~Q

Hi @meeroslav done some more test and the only way that seem to work is to reload the full website so if I write something like that:
switchLanguage(language: string) {
    this.localize.changeLanguage(language);
    console.log('AFTER CHANGE ROUTES', this.localize.parser.routes);
    let currentUrl: string = this.router.url;
    let parts: Array<string> = /\/(.+)\/(.+)/.exec(currentUrl);
    if (parts) {
      let currentLanguage = parts[1];
      let currentPage = parts[2];
      if (currentLanguage != language) {
        let newUrl = `/${language}/${currentPage}`;
        // this.router.navigateByUrl(newUrl);
        window.location.href = newUrl;
      }
    } else {
      console.log('current URL does not match');
      window.location.href = `/${language}`;
    }
  }

it does work, the page is reloaded the new language is set and the navigation works with no problem.
What I noticed and that may void what I wrote above but I don't know the module well enough to say so is that the redirect route is not updated when the  the LocalizeRouterService#changeLanguage is invoked.
On Init

START ROUTES
(2) [{…}, {…}]
{children: Array(5), path: "en"}
{path: "", redirectTo: "en", pathMatch: "full"}

After LocalizeRouterService#changeLanguage

AFTER CHANGE ROUTES
(2) [{…}, {…}]
{children: Array(5), path: "it"}
{path: "", redirectTo: "en", pathMatch: "full"}

Thanks,
~Q

have the same problem, i think idea of this plugin is work exactly that your described in your problem
+1 for fix this behavior

same problem here, +1

anyone solved the problem? @meeroslav

Solve it by manually changing the locale in the url and then reload the page, the easiest way

@apolon23 my comment above provides the method I used to workaround the issue which in principle does what @Elmanderrr suggests as well.

So I got bored of seeing my app reloading as I consider it a very bad practice from technical and user experience point of view and decided to crack on figure the problem out, there are 2 issues that make this problem up:

The changeLanguage method expects router config to be reset when invoked
The router config is reset only when a localize service is started, and this is why the reload works

So the solution is simple just reset the config of the router after the routes translation is completed so the changeLanguage should be implemented as
swtchLanguage(language: string, options: NavigationExtras, useNavigateMethod: Boolean) {
    if (language != this.parser.currentLang) {
      const currentUrl: string = this.router.url;
      const newRoute: string = this.parser.translateRoute(currentUrl.replace(this.parser.currentLang, language));
      this.parser.translateRoutes(language).subscribe(() => {
        // reset the router config so that routes are dynamically loaded.
        this.router.resetConfig(this.parser.routes);

        const extras = Object.assign({ skipLocationChange: false }, options);
        if (useNavigateMethod) {
          this.router.navigate([newRoute], extras);
        } else {
          this.router.navigateByUrl(newRoute, extras);
        }
      });
    }
  }

with the method above I can navigate my app without any page reload.
I am creating a little project that tries to provide a way of approaching multil-anguage applications, is still very young and based as you can understand on this library. In order to use this new implementation load MultilangModule as
const routes: Routes = [...]

@NgModule({
  imports: [
    TranslateModule.forRoot({
      loader: {
        provide: TranslateLoader,
        useFactory: (http: HttpClient) => { return new TranslateHttpLoader(http, '/assets/locales/', '.json'); },
        deps: [HttpClient]
      }
    }),
    MultilangModule.forRoot(routes, {
      parser: {
        provide: LocalizeParser,
        useFactory: (translate, location, settings, http) =>
          new LocalizeRouterHttpLoader(translate, location, settings, http),
        deps: [TranslateService, Location, LocalizeRouterSettings, HttpClient]
      }
    }),
    RouterModule.forRoot(routes)],
  exports: [TranslateModule, LocalizeRouterModule, RouterModule]
})
export class AppRoutingModule {

}

and use it as:
constructor(private multilangService: MultilangService) {}

...
switchLanguage(language: string) {
    this.multilangService.swtchLanguage(language, {}, true);
}

I will create a PR on this library as soon as I get time.
Hope the above can help.
@meeroslav I think we can considered this issue as verified, if you are OK with it I will use this issue to create a PR as soon as I get time.

@quirinobrizi
I have the same issue. When changing language, the component reload (it occurs when hitting navigate or navigateByUrl in the 'changeLanguage` function)
The function : LocalizeRouterService.traverseRouteSnapshot will return a redundant url path like "fr/index/index" and then the router navigate to an unknown url => go back to home.
After fixing the function LocalizeRouterService.traverseRouteSnapshot to return a valid url like "fr/index".
The component reload (ngOnInit is called) when a navigation occurs in the changeLanguage function :
this.router.resetConfig(this.parser.routes);
        if (useNavigateMethod) {
          this.router.navigate([url], extras);
        } else {
          this.router.navigateByUrl(url, extras);
        }

I've tried to implement your function switchLanguage() but i have the same bug : component is reloaded after navigation...
Any update on this ?

@Roelensdam i solved this problem like this:
onChange(language: string) {
window.location.href = this.router.url.replace(this.localize.parser.currentLang, language);
};

@apolon23
Thanks for the tip, but i DON'T want to reload the page. This make the whole app reload. I DON'T want the component to be reloaded either.

After further investigation, it seems linked to the immutability of the new router (Angular 6).
When performing a resetConfig before a navigation, the component reload as his reference is different from the previous config.
Any suggestions, ideas ?

@Roelensdam I am not familiar with Angular 6, I am between moving and have no much time to look into it I will as soon as I find some time... sorry about that.

@Roelensdam Problem fixed on angular 6 fork https://www.npmjs.com/package/@gilsdav/ngx-translate-router
@Greentube you can check what changed here: https://github.com/gilsdav/ngx-translate-router/commit/de42217aeb349e3adc577a490bf6ddefb22e42f0

@quirinobrizi, @Roelensdam, @gilsdav,
This should be solved on both version v1.0.1 (for Angular 5.0.0-5.2.6) and 2.0.0-RC.1 (for Angular 6+).
Can you please verify?

@meeroslav
doesn't work for me on 2.0.0-RC.1, language changed correctly, links are updated, but nothing happens when i trying to navigate to new urls

@meeroslav Some bugs there is no more on my repo but still in 2.0.0-RC.1:

If sub route is "" (empty string) changing langue will not found path so redirect to default "**".
ngInit is called every time we change lang. This is why I created my own ReuseStrategy.

You can test by cloning my repo and you change every reference of my fork to localize-router into scr folder (the demo app).
 We must come to "http://localhost:4200/en/mytest" and not to the home page.

@gilsdav, I like your approach with ReuseStrategy, but I would not make it part of localize-router.
I will, however, add the link to the README.
Based on the user's needs he/she can choose to have ngInit triggered on the lang change or not.
As for the empty subroute, I will look into it ... it's on my list.

@Elmanderrr, can you provide a minimal example with this behavior? Also, please open the new issue so I can target that specifically.

I was stuck with this issue when switching the route with parameters, such as listing-details/:id
When I use just the this.localize.changeLanguage(locale); for a url /ar/listing-details/7 it construct and redirect me to invalid URL en/listing-details%2F7;id=7, the expected URL redirect was /en/listing-details/7 which didn't happen.
Hence in my switch locale function, I decided to manually reconstruct the URL and navigate the user, here is the function I came up with, which is working fine for me.
import { LocalizeRouterService } from 'localize-router';
import { Router } from '@angular/router';

private localize: LocalizeRouterService,
private router: Router

const urlArray = this.router.url.split('/');
let switchToUrl = '/' + locale;
for (let i = 2; i < urlArray.length; i++) {
    switchToUrl += '/' + urlArray[i];
}
this.localize.changeLanguage(locale);
this.router.navigate([switchToUrl]);

I was excited it worked. Upon further testing it is still an issue, in some cases it still produces the same URL and in most cases it doesn't ...


Move affine transformation and vector 4f to regular util math

Signed-off-by: liach<EMAIL_ADDRESS>

Keeping this as mojang seems to randomly use some of these client math classes on server. Matrix4f rename was an example.


Just a few more UX improvements

Sorry, quite messy with multiple commits, as I was editing whilst working through the process for the first time.
Amazing work btw, bed has never been so level! Thanks.

Great changes, thanks!


Cache Homebrew to speed up build on macOS

Before creating a pull request

[x] Document new methods and classes (N/A)
[x] Format code with make format (N/A)

Before merging a pull request

[x] Set version target by selecting a milestone on the right side (N/A)
[x] Summarize this change in CHANGELOG.md (N/A)
[x] Add unit test(s) for this change (N/A)

Hope this works! :crossed_fingers:
Feel free to merge when you're ready.

It doesn't seem to work because of timeout in saving cache directories, which is just 3min. :disappointed:


no valid points: scale 0

When I run the example data , the error is no valid points: scale 0.
My environment is

RTX3090, torch 1.8+cu111, cuda11.1. (I try use the torch 1.6.0, but RTX3090 not match it. )
The torchsparse is install by rebuild.
repo commit is cd047e2356f68f60adfb923f33572d712f64b57a (HEAD -> master, origin/master, origin/HEAD)

The run code is
python3 demo.py --cfg ./config/demo.yaml
The log is:
2022-02-07 16:15:15.152 | INFO     | __main__:<module>:35 - Found SyncedPoses.txt, skipping data pre-processing...
2022-02-07 16:15:15.152 | INFO     | __main__:<module>:37 - Running NeuralRecon...
2022-02-07 16:15:15.154 | INFO     | __main__:<module>:50 - Initializing the model on GPU...
2022-02-07 16:15:17.009 | INFO     | __main__:<module>:58 - Resuming from ./checkpoints/model_000047.ckpt
2022-02-07 16:15:17.104 | INFO     | __main__:<module>:64 - Start inference..
  0%|                                                                                | 0/110 [00:00<?, ?it/s]2022-02-07 16:15:17.527 | WARNING  | models.neucon_network:forward:184 - no valid points: scale 0
  1%|█▍                                                                                | 1/110 [00:00<00:46,  2.32it/s]2022-02-07 16:15:17.590 | WARNING  | models.neucon_network:forward:184 - no valid points: scale 0
2022-02-07 16:15:17.654 | WARNING  | models.neucon_network:forward:184 - no valid points: scale 0
  3%|████▍                                                                                | 3/110 [00:00<00:16,  6.30it/s]

@vvcatstar I run into same issue. Do you find any solution?

+1

I carefully check the source code. This is due to torchsparse package. Under current setup, neuralrecon with torchsparse dependency can only work on cuda 10.2 and 11.1 or cpu mode.


Use this tool in windows occur a error with auto open the devtools for nodejs context

Version: 1.1.1
Platform: Windows 7 Professional Edition
Chrome Version: 58.0.3029.110 (64-bit)

Then I got the error when I start my express server

Hey @safarishi! Thanks for filing this issue. I'll look into Windows debugging when I get a chance later today. I've pushed off a new release because of some breaking changes I've made to how we launch the devtools so any fix for this as well will probably come out Monday/Tuesday.

thanks ~

When I upgrade chrome to 60 the devtools for the chrome has a change,
you can open devtools for nodejs

Thats so cool

This should be fixed as of the v1.2 release where rawkit specifically looks at your operating system and determines which "executable" to use (ie. "google chrome" on Mac, "google-chrome" on Linux & "chrome" on Windows). If you still find you're having issues, definitely let me know.


Advanced Settings Final

Added the theme to the advanced settings page and corrected the overflow error. Added a bit of documentation as well.

Please write good commit messages next time


maximum recursion depth exceeded

A cyclic import of JSON files causes a "maximum recursion depth" error in 2 variants:

JSON file imports itself: "maximum recursion depth exceeded while calling a Python object" (JPP_1152)
JSON file imports another file, that is already imported: "maximum recursion depth exceeded in __instancecheck__" (JPP_1153)

Both based on dynamic import paths.
Successor of https://github.com/test-fullautomation/python-jsonpreprocessor/issues/380 (imports are resolved now)

Hello Holger,
This ticket was solved on stabi branch.
Thank you,
Son

merged for version 0.13.1


Support context-based configs

Return different values based on a provided context that can include Ids and or tags

Moved to https://github.com/props-sh/props (https://linear.app/mihai-bojin/team/PROPS/all)


Recognize class names in command line

From docs work.
This is very slick - now you can call animations from the command line by their typename!
Just write:
bp BiblioPixelAnimations.matrix.Twinkle

or
bp '$bpa.matrix.Twinkle'

or
bp @bpa.matrix.Twinkle

Works with + notation, etc.
So now you do bp animations and then pick one to run - done.

+1
On Tue, Aug 21, 2018 at 8:01 AM Tom Swirly<EMAIL_ADDRESS>wrote:

From docs work.
This is very slick - now you can just write:
bp -s BiblioPixelAnimations.matrix.Twinkle
or
bp -s '$bpa.matrix.Twinkle'
or
bp -s @bpa.matrix.Twinkle
Works with + notation, etc.
You can view, comment on, or merge this pull request online at:
https://github.com/ManiacalLabs/BiblioPixel/pull/934
Commit Summary

Recognize class names in command line

File Changes

M bibliopixel/main/run.py
https://github.com/ManiacalLabs/BiblioPixel/pull/934/files#diff-0
(24)
M bibliopixel/project/fill.py
https://github.com/ManiacalLabs/BiblioPixel/pull/934/files#diff-1
(8)

Patch Links:

https://github.com/ManiacalLabs/BiblioPixel/pull/934.patch
https://github.com/ManiacalLabs/BiblioPixel/pull/934.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/pull/934, or mute the thread
https://github.com/notifications/unsubscribe-auth/AA6a6tMwfB-EJNXK7fILQ9DHabTk-Jlcks5uS_augaJpZM4WFowU
.

Ah, breakage, one moment... :-/


feat: use real pty

This allows to allocate a real PTY for the session, allowing to do things like running vim within a SSH session.
depends on https://github.com/charmbracelet/bubbletea/pull/879
depends on https://github.com/charmbracelet/ssh/pull/8
closes https://github.com/charmbracelet/wish/issues/196

If we're going to do that, we have to remove the ssh PtyWriter hack, and detect the color profile from the PTY https://github.com/charmbracelet/wish/pull/197/files#diff-79d3f1c4cf631bd3e71a422f4d73666477d1caa5b44e965c98d80e6551f4f8cfR93

Can be done in another PR, as its not strictly related

still pending changes on ssh cc/ @aymanbagabas

anyone against this not being a breaking change anymore?

anyone against this not being a breaking change anymore?

LGTM

@caarlos0 Do you plan on doing a release for this?


Add Insomnia (By Me)

Hey! I love your theme.
I made an insomnia theme for everforest, and I was wondering if you could include it.
Thanks again!

@talwat glad to see another port 👍 Feel free to edit the Wiki at https://github.com/sainnhe/everforest/wiki

Also, one more thing, is it possible to move the everforest docs to the wiki? It's much easier to read in my opinion.

Probably the installation section should be moved indeed.
These docs are supposed to be opened from inside Vim with :help everforest, but to get to this doc you need to install the color scheme.
Chicken and egg problem.

Yeah. I suggest maybe moving over troubleshooting and install information over to the wiki but keeping configuration documentation in the text file.

Thanks for updating the wiki 🙌


make group, chain, and choose compatible with RefRange

The first three of probably many functions that need to be fixed in order to resolve issue 18657.
Related forum discussion<EMAIL_ADDRESS>I'm not sure if this is the right direction. The alternative would be to add assignment to the range interface, and remove opAssign from RefRange. That would be less busywork, and being able to use assignment with ranges might be better for everyone's sanity. But it would also be more of a breaking change.

This has been open for three weeks, and the only discussion has been about optimizing for DMD or not (consensus: not). Ping?

In my opinion, it would be better to update the range spec and go from there.

I wouldn't mind that at all.

This has been open for three weeks, and the only discussion has been about optimizing for DMD or not (consensus: not). Ping?

This is because this is a complex and deep rooted topic that needs proper understanding, also related to unsupported unique ownership ranges.
FWIW using copy construction + move to avoid a broken opAssign is a hack that might bite us in the long-run. It's also problematic to leak such workarounds into the standard library, where people look up how to implement certain behaviors.
Whether the design issue is within refRange, the algorithms, the range spec, or the language someone still needs to exactly point out. And btw, it's the purpose of refRange to mutate the original range, so maybe it's not even reasonable to implement save.
Thanks for your work on this fix, but I think a deeper understanding of the problem would be more appropriate than simply fixing a few symptoms.

I've reconsidered based on @jmdavis's comments.
I think the option that will cause the least pain will keeping in the spirt of what RefRange is supposed to do, is to remove RefRange's forward range capabilities.
The range semantics still need to be defined, but that doesn't solve the issue of breaking something about RefRange. By removing save we keep the current intended behavior and remove the buggy behavior.

I think the option that will cause the least pain will keeping in the spirt of what RefRange is supposed to do, is to remove RefRange's forward range capabilities.

Maybe. I'm not so sure if the original goal of RefRange is worthwhile, though.
What the opAssign achieves is that the RefRange binds more tightly to its target than a pointer or class reference would. It's more like a ref parameter, which also cannot be rebound.
I think this behavior is actually somewhat surprising. I'd expect refRange(&foo) to work like &foo, just with the necessary adjustments to make it a proper range itself. I've had uses for that part of RefRange. I don't think I ever had a use for the stronger non-rebindable aspect. But it's of course possible that I just wasn't aware of relying on it.
Example:
import std.range: refRange;
import std.stdio: writeln;

void main()
{
    string s = "foo";

    auto p = &s;
    auto r = refRange(&s);

    auto p2 = p;
    auto r2 = r;

    p2 = ["bar"].ptr; /* Rebind p2. */
    writeln(*p); /* Still "foo". */

    r2 = refRange(["bar"].ptr); /* Can't actually rebind r2. */
    writeln(r); /* Surprisingly(?), it's "bar" now. */
}

The range semantics still need to be defined, but that doesn't solve the issue of breaking something about RefRange. By removing save we keep the current intended behavior and remove the buggy behavior.

You also lose the intended behavior of RefRange being a forward range. That's ok if it was a mistake in the first place, but does it buy us anything beyond that? Seems mostly a separate issue to me.
If non-surprising assignment becomes a requirement for ranges, we have to fix RefRange in that regard. And if funky opAssign overloads get deliberately allowed,  generic code can't assume anything about assignment, and we have to do what this PR does. In both cases, it doesn't really matter if RefRange has save or not.
One scenario where RefRange.save would become relevant is if we say that forward ranges must have non-surprising assignment, but input ranges don't have to. Doesn't really make sense to me, but that's the one case I can think of where removing save from RefRange would have a meaningful impact.

I'm not so sure if the original goal of RefRange is worthwhile, though.

My point was, refRange is a very important primitive, in particular with unique ownership to circumvent the copy vs. move problem. It's prolly also in widespread use.
If we need to fix sth. about this then we need to consider the whole ecosystem, including fallout, future directions, and learning hurdle/complexity.

I'd expect refRange(&foo) to work like &foo, just with the necessary adjustments to make it a proper range itself.

It this a valid expectation though? Pointers to ranges are valid InputRanges (try isInputRange!(typeof(&rng)).
Ironically where that breaks down is for auto saved = ptrRng.save; ptrRng = saved;, so RefRange just seems to fix that bit, thus supporting isForwardRange!(typeof(refRange(&rng))).
If RefRange behaved like a pointer, why would we need it in the first place?
Intuitively it seems there is a need for at least 2 use-cases:

Using refRange to consume (i.e. empty) a range with a normally preserving algorithm.
Particularly useful for forward ranges that only support O(N) popFrontN.
e.g. refRange(&fwdRng).until!(v => v > 10).sum
Using refRange to workaround internal copying of ranges, expecting that it preserves the current range just as passing a normal copyable one.
e.g. refRange(&uniqRng).sum; assert(!uniqRng.empty);

There might be other use-cases I'm not aware of right now!
Please re-read your bug report https://issues.dlang.org/show_bug.cgi?id=18657#c0, it's a bit one-sided and uses complex examples to argue for "doing the obvious thing".
Sorry to play bouncer here, I fully understand the urge to just fix sth., but this problem seems important enough to warrant an open mindset.

A little ping after three weeks of silence isn't being pushy, is it?

Not at all ;) sorry I've only just got around to this. This needs a rebase.

This needs a rebase.

Done. But this needs a decision more than it needs a rebase.

Happy birthday, PR #6346! :tada:
Still waiting on someone to decide if this is the way to fix issue 18657.

Thank you for pulling this, @thewilsonator. I hope you're aware that this was just the first step towards fixing the Bugzilla issue.
Here's the next batch: #6935. More will be needed.

This sounds like exactly the sort of "Generality Creep" identified by Andrei https://digitalmars.com/d/archives/digitalmars/D/Generality_creep_324867.html
Perhaps RefRange needs to be fundamentally rethought instead of changing every range algorithm.

This sounds like exactly the sort of "Generality Creep" identified by Andrei https://digitalmars.com/d/archives/digitalmars/D/Generality_creep_324867.html
Perhaps RefRange needs to be fundamentally rethought instead of changing every range algorithm.

Honestly, I think that it was a huge mistake to add RefRange in the first place (and I'm the one who added it). It's trying to fight against how ranges work to make certain use cases work better, and while it works in some situations, it's ultimately a mess.


Move file from cisagov/dotgov-data

💡 Summary
Move this file to this repo.
Motivation and context
This repo is a more appropriate home for this kind of data.

I think dotgov-home can serve the purpose of cataloging .gov hostnames (and I anticipate we'll increasingly do that). I just thought there was some duplication between the two repos. As long as that's not unnecessary duplication, I'm  happy to keep playing host.


[feature] ALSA & PulseAudio Equaliser

Hello,
As the title suggests, if it is possible to add Equaliser.
Edit 1:
For ALSA with your container I tried this method ALSA-EQ-Plugin and it works perfectly.
Steps to Install

apt update && apt install libasound2-plugin-equal
Create

/root/.asoundrc
...
pcm.!default {
type plug
slave.pcm plugequal;
}
ctl.!default {
type hw card 0
}
ctl.equal {
type equal;
}
pcm.plugequal {
type equal;
slave.pcm "plughw:0,0"; # NOTE this line MUST be your hardware device.
}
pcm.equal {
type plug;
slave.pcm plugequal;
}

In my case, audio is outputting from HDMI.

Hello, this is an interesting feature. I will be working on it as soon as I can.
About your asoundrc file, do you use "pcm.plugequal" as the alsa device in the mpd output?
In any case, I will probably start by adding the necessary packages, so by just mounting the asoundrc file and using the 'additional-outputs.txt' file, you will probably not need to do any package operation when the container is recreated. Then I will try to understand how to make this work using environment variable.

At first I did create additional-outputs.text like this:
audio_output {
  type "alsa"
  name "Alsa EQ"
  device "plug:plugequal"
  mixer_type "software"
}

Then I noticed that the EQ was automatically applied to the default output as well.

I have created /user/config/eq and mounted this volume with my EQ Present scripts in there. For example:
00-default.sh
...
#!/bin/bash
/usr/bin/amixer -D equal -q set '00. 31 Hz' 66
/usr/bin/amixer -D equal -q set '01. 63 Hz' 66
/usr/bin/amixer -D equal -q set '02. 125 Hz' 66
/usr/bin/amixer -D equal -q set '03. 250 Hz' 66
/usr/bin/amixer -D equal -q set '04. 500 Hz' 66
/usr/bin/amixer -D equal -q set '05. 1 kHz' 66
/usr/bin/amixer -D equal -q set '06. 2 kHz' 66
/usr/bin/amixer -D equal -q set '07. 4 kHz' 66
/usr/bin/amixer -D equal -q set '08. 8 kHz' 66
/usr/bin/amixer -D equal -q set '09. 16 kHz' 66
echo "EQ: reset"

Then I can run docker exec -it mpd /user/config/./<preset>.sh to apply it.

That's interesting :-)
There is already an image available, which includes the necessary packages. Can you please try it and see if this works for you? I expect that the manual steps required for adding the package should not be necessary now.

Tested and it is working smooth. 👍🏻

Good to hear that.
I will experiment with equalizer myself and probably come out with a release. I can't predict how long this will take though.

Hello, thank you for your feedback. I will the documentation from this ticket in an additional file in the doc folder and link it in the main README.md.
I will probably do that this saturday. Should you want to send a merge request with part of the documentation, it would be welcome of course. Just feel free to do that if you have some spare time.

Thank you for the contribution!

No problem at all. We can now close this issue.


Installation

I'm installing this on a simple project in Netlify, and I'm wondering if you could walk me through step-by-step exactly what to do. I'm still a rookie. So far I added this line to a new netlify.toml file: [[plugins]]
package = "netlify-plugin-inline-functions-env".
Thank you.

@bgschust What's the use case that you include the plugin to the project, did you see any errors or messages in netlify build log?

@bgschust What's the use case that you include the plugin to the project, did you see any errors or messages in netlify build log?

The project is called Cropify.org, which is a marketplace to connect local farmers with local buyers. There's currently a mobile app, and the web app is in development. Ideally I'll minimize terminal activities as much as possible to make prototyping faster and keep the code simple with Javascript functions + API calls.
I've gotten several errors with Netlify deployments, so I'll try to clarify that better directly with you.

The project is called Cropify.org, which is a marketplace to connect local farmers with local buyers. There's currently a mobile app, and the web app is in development. Ideally I'll minimize terminal activities as much as possible to make prototyping faster and keep the code simple with Javascript functions + API calls.
I've gotten several errors with Netlify deployments, so I'll try to clarify that better directly with you.


Camera RPi

Hi, does anyone know how to add the camera module from RPi as  "surveillance camera" ?
it works on HAP-JS, but I like this Home control better and would like to keep using.
Thanks

Just now I wrote my "question" :-)
https://github.com/brutella/hc/pull/93

I've spent some time looking at what it will take to support this. I think to do it right is going to require some large changes to hc.

I think this is going to depend on the OnValueGet hook from #100.
I think the tlv8 API is going to need an overhaul. The HAP configuration for the camera descriptors require a lot of complicated tlv8 structures. I think moving that to be based on structs and with tags for the type code is the way to go, then implementing a proper marshaling system for tlv8. This also needs the ability to have multiple entries with the same tag for some features (like supporting multiple resolutions) that the current tlv8 API can't do.

@icb- Looking at how HAP-Node does it, it appears there can be a "get" and "set" type of callback?: https://github.com/KhaosT/HAP-NodeJS/blob/master/lib/StreamController.js#L265-L272
Does our OnValueRemoteUpdate handle "gets" and "sets"?

@ipstatic

Does our OnValueRemoteUpdate handle "gets" and "sets"?

No, but see #100.

Hello everyone, I'm a beginner developer, and I'm interested in the camera on raspberries, if anyone made an accessory, I will be very grateful if you will give me the code and example of use, thanks)

There is a now an implementation of a HomeKit camera in hkcam.
Sorry that it took so long.


xeditable save button not working when used inside ng-repeat

Here is the template i am using

<item ng-repeat="phonenumber in contact.phoneNumbers"  >

	         	{{phonenumber.value}}
</item>

Cancel button works, where as save button doesnt.
PS: outside ng-repeat loop its work perfectly.

                {{phonenumber.value}}

modified 'item' tag with 'div' and it worked

Try implementing e-single, or make sure your item each item has a unique name. I bet if you check your form in bats tang, you will see what I'm talking about.

Batarang. Not bats tang.


Auto-complete for accessing resource files

I would like for my IDE to be able to recommend suggestions so that I can quickly access resources when I'm writing the tests, without having to refer to the resources directory.
I have two suggestions:

Implement the @Language tag that the Resources() class can inject languages, specifically the file-reference injection.
IntelliJ will then automatically suggest file paths

Unfortunately @Language isn't a Kotlin annotation, so it isn't suitable for Kotlin Multiplatform out of the box. But it can be converted to work for Kotlin Multiplatform projects, as demonstrated in Kotlinx Serialization.

Generate code so that I can access the files via variables.
For example, given that resources contains
.
└── resources/
    └── test-data/
        └── some-test-file.json

Then the kotlinx-resources plugin could generate the following code:
// build/generated/src/main/kotlin/ResourcesAccessors.kt

object Resource {
  /** `test-data/ directory */
  object testData {
    /** Load the file `test-data/some-test-file.json */
    val someTestFileJson = Resource("test-data/some-test-file.json")
  }
}

I could then quickly access the file in my test:
// src/commonTest/MyResourcesTests.kt
class MyResourcesTests {
  @Test
  fun `test json`() {
      val someTestData = Resource.testData.someTestFileJson.readText()
      // ...
  }
}

I'd suggest the 2nd option, basically what android does with the R class.
In case there is a change/removal it's being caught right away during compile time.


gguf supported?

gguf supported?

I thought not but I edited the test shell file in the scripts to accept gguf too and it worked