Update LLVM 18.1.1

Update LLVM v18.1.1, and perhaps fixed submodule (src/config).

So here we are updating from 18.1.0rc to 18.1.1?   (Can you mention both these versions in the PR title).
I'm not sure why ever pinned to an rc version but its good that we are now back to an official release.

I don't believe an rc was used, it was a mistake in the LLVM release process.

Is there a benefit to accept LLVM 18.1.2?

As a heads up I've opened https://github.com/WebAssembly/wasi-sdk/pull/402 to update to 18.1.2 which additionally addresses changes necessary to keep things working with components.


python310Packages.pyinfra: 2.0.1 -> 2.1

Description of changes
https://github.com/Fizzadar/pyinfra/releases/tag/v2.1

enable tests

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 171530 run on x86_64-linux 1

2 packages built:

pyinfra (python39Packages.pyinfra)
python310Packages.pyinfra

Result of nixpkgs-review pr 171530 run on x86_64-linux 1

2 packages built:

pyinfra (python39Packages.pyinfra)
python310Packages.pyinfra

Just out of curiosity...
Is it really preferable to use GitHub as source for packages instead of PyPi?

Is it really preferable to use GitHub as source for packages instead of PyPi?

I switched to GitHub because the source tarball from PyPI didn't contains all files which are required to run the tests. I assume that upstream skipped the fixtures to reduce the size of the tarballs.

Result of nixpkgs-review pr 171530 run on x86_64-linux 1

2 packages built:

pyinfra (python39Packages.pyinfra)
python310Packages.pyinfra


New package: TropicalNN v1.0.0

Registering package: TropicalNN
Repository: https://github.com/Paul-Lez/TropicalNN.jl
Created by: @Paul-Lez
Version: v1.0.0
Commit: 055c7f4c894f05d800cca4747f77ef4cdaa587b4
Reviewed by: @Paul-Lez
Reference: https://github.com/Paul-Lez/TropicalNN.jl/issues/1#issuecomment-2412258485
Description: Tropical Geometric Computations for Neural Networks

Thank you for submitting your package! However, please make sure to add some documentation before registering. At the very least, that would be a description of the package's purpose and a small usage example in the README.
For a version 1.0, there would be additional requirements:

Must define the stable API. That generally means complete documentation.
Must have tests with reasonable coverage (on the order of 80%). Otherwise, there's no way to know if the package actually implements its API correctly
Should somewhat reasonably fill the described scope of the package. No major gaps in functionality.

You probably want to register at v0.1, not v1.0

Thank you for submitting your package! However, please make sure to add some documentation before registering. At the very least, that would be a description of the package's purpose and a small usage example in the README.
For a version 1.0, there would be additional requirements:

Must define the stable API. That generally means complete documentation.
Must have tests with reasonable coverage (on the order of 80%). Otherwise, there's no way to know if the package actually implements its API correctly
Should somewhat reasonably fill the described scope of the package. No major gaps in functionality.

You probably want to register at v0.1, not v1.0

Hi @goerz, thanks for the advice! I'll start adding these now and change the registered version to v0.1.

Thank you for submitting your package! However, please make sure to add some documentation before registering. At the very least, that would be a description of the package's purpose and a small usage example in the README.
For a version 1.0, there would be additional requirements:

Must define the stable API. That generally means complete documentation.
Must have tests with reasonable coverage (on the order of 80%). Otherwise, there's no way to know if the package actually implements its API correctly
Should somewhat reasonably fill the described scope of the package. No major gaps in functionality.

You probably want to register at v0.1, not v1.0

Hi @goerz, thanks for the advice! I'll start adding these now and change the registered version to v0.1.

Hi again @goerz! I've updated the repository to give a description of what this package does in the README, as well as some examples of how this code should be used. A large portion of the code is also documented (in the files) with additional examples.
Do let me know if I should add any extra stuff for the v0.1 release (I do plan add a substantial amount of unit tests for the following release).

Closing in favor of https://github.com/JuliaRegistries/General/pull/117343


reformat: ignore opam-switches

Explain your changes:

opam-switches has been added to handle different switches. It is created by the Makefile. However, make reformat will try to reformat the files in the directory.

Explain how you tested your changes:

N/A

cc @querolita who was stuck because of this.
Checklist:

[ ] Dependency versions are unchanged

Notify Velocity team if dependencies must change in CI

[ ] Modified the current draft of release notes with details on what is completed or incomplete within this project
[ ] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[ ] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[ ] All tests pass (CI will check this if you didn't)
[ ] Serialized types are in stable-versioned modules
[ ] Does this close issues? List them

Closes #0000

!ci-build-me

Helping https://github.com/MinaProtocol/mina/pull/16057


Unable to center paragraph.

I have an InlineCssTextArea where I append a line of text and then attempt to center the entire paragraph. I've tried setting the setParagraphStyle and setStyle, but neither seems to work. I've attached a sample that shows the behavior, any ideas where I might be going wrong?
import javafx.application.Application;
import javafx.scene.Scene;
import javafx.scene.layout.GridPane;
import javafx.stage.Stage;
import org.fxmisc.richtext.InlineCssTextArea;

public class Example extends Application {

    public static void main(String args[]){
        launch(Example.class);
    }
    @Override
    public void start(Stage primaryStage) throws Exception {

        GridPane root = new GridPane();
        Scene scene = new Scene(root, 900.0,900.0);
        InlineCssTextArea area = new InlineCssTextArea();
        root.add(area, 0, 5, 4, 1);
        primaryStage.setTitle("Example");
        primaryStage.setScene(scene);
        primaryStage.show();

        area.appendText("Test");

        area.setParagraphStyle(area.getCurrentParagraph(), "-fx-text-align: center;");
        area.setStyle(area.getCurrentParagraph(), "-fx-text-align: center;");

    }
}

That looks correct. Have you examined it with Scenic View?
On Jul 27, 2017 4:10 PM, "Patrick Dattilio"<EMAIL_ADDRESS>wrote:

I have an InlineCssTextArea where I append a line of text and then attempt
to center the entire paragraph. I've tried setting the setParagraphStyle
and setStyle, but neither seems to work. I've attached a sample that
shows the behavior, any ideas where I might be going wrong?
import javafx.application.Application;
import javafx.scene.Scene;
import javafx.scene.layout.GridPane;
import javafx.stage.Stage;
import org.fxmisc.richtext.InlineCssTextArea;
public class Example extends Application {
public static void main(String args[]){
    launch(Example.class);
}
@Override
public void start(Stage primaryStage) throws Exception {

    GridPane root = new GridPane();
    Scene scene = new Scene(root, 900.0,900.0);
    InlineCssTextArea area = new InlineCssTextArea();
    root.add(area, 0, 5, 4, 1);
    primaryStage.setTitle("Example");
    primaryStage.setScene(scene);
    primaryStage.show();

    area.appendText("Test");

    area.setParagraphStyle(area.getCurrentParagraph(), "-fx-text-align: center;");
    area.setStyle(area.getCurrentParagraph(), "-fx-text-align: center;");

}

}
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/TomasMikula/RichTextFX/issues/548, or mute the thread
https://github.com/notifications/unsubscribe-auth/AIBfbabQ6LMjbf48qBhlB5a5F-2mola-ks5sSUMygaJpZM4OmDCn
.

Yes, the style appears to be set on both the TextExt and the ParagraphText but "Test" remains left aligned.

Hmm... strange.
Also, which version are you using?
On Jul 27, 2017 6:19 PM, "Patrick Dattilio"<EMAIL_ADDRESS>wrote:

Yes, the style appears to be set on both the TextExt and the ParagraphText
but "Test" remains left aligned.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/TomasMikula/RichTextFX/issues/548#issuecomment-318555277,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIBfbQ0DTyFDM7MpYyqkRwkfPjW-kKWSks5sSWFogaJpZM4OmDCn
.

"-fx-text-align" -> "-fx-text-alignment". Simple typo. Sorry about that!


Missing SqlBuilder Java Classes

Currently, the Java classes provide the possibility to analyze a SQL statement and get a resolved AST. However, the it is not possible to create a SQL statement from the resolved AST, because the required Java classes are missing (even though the corresponding C++ classes are available), i.e. SqlBuilder.java and its dependencies.

This was improved a bit in c610a21ffdc110293c1c7bd255a2674ebc7ec7a8 - you can call Analyzer.BuildExpression, but there still isn't a great way to actually modify a ResolvedExpr (if that is your goal).
Can you talk about your usecase?

We are maintaining a data warehouse of dozens of BigQuery datasets with sensitive personal data of our customers. By GDPR, many of collegues do not have the permission to actually see some sensitive personal parts of the raw data (this includes parts like customer numbers and even pseudonyms), but they have the permission to analyze and work with this sensitive data to some extend. Therefore, we build our collegues a web interface (GAE ftw.) to submit their queries, which does essentially two things:

It computes a data lineage of the resulting schema of the query. Based on this lineage, we are able to annotate the columns of the resulting table as sensitive or not sensitive. For instance, if you have a table customer_data that contains a sensitive column name and you submit the query SELECT CONCAT(name, "foo") AS funny_name FROM customer_data, then funny_name should be classified as sensitive.
It augments the input query in order to generate some metadata for the computed cells of the resulting table. For example (very, very simplified), we allow our users to "see" a data cell if it is an aggregate of at least 20 different values. To achieve this, we compute, based on an input querySELECT shop_id, AVG(customer_age) FROM customer_data GROUP BY shop_id

the new querySELECT shop_id, AVG(customer_age), COUNT(DISTINCT customer_age) AS hidden_count FROM customer_data GROUP BY shop_id

and show a resulting column only if hidden_count is larger than 20. (Again, this is extremely simplified, because this is obviously by far not enough to ensure data protection.)

For both of these tasks we are currently using Apache Calcite, but the ZetaSQL dialect contains some parts (e.g. the non-standard EXCEPT in SELECT * EXCEPT(foo)) that are not supported by Calcite and are really hard or even impossible to implement using Calcite. Thus, we were very happy to see that the ZetaSQL analyzer was published here. While we nearly managed to fully implement the lineage computation using ZetaSQL (despite the missing docs), we are not able to modify the resolved AST and unparse it to an SQL statement that we can submit to BigQuery, because SQLBuilder.java and its surrounding classes are not provided.

Christoph, this looks very similar to an effort we are working on. Have you have any success in this regards, yet?

@cvonredapt I took over working on that issue from @christophstockhusen and came up with #29 to solve this issue.
Actually, the gRPC interfaces are already in place, you just have to extend the Analyzer interface to use it.

Latest release contains a rewriting visitor which can help with the read-modify-write use case.
See examples in RewritingVisitorTest.java (sorry, not really any useful documentation yet)


Fixing issues with PNG thumbnails

The version of pillow was so old the library that was trying to convert the images was failing silently.
To Test:

Try adding a PNG to a recipe and make sure the thumbnail is generated.

Coverage remained the same at 90.636% when pulling ab7d2f9bb7f557a116624ffa1f43cbd2412cd944 on fix/png-thumbnails into ef8135b4e071eaf2fad21d8ec36ebc26cfe8b4d1 on master.


CPOL-97 First implementation to show last triggered time.

For this to work we need https://github.com/RedHatInsights/custom-policies-engine/pull/39 first.
Also it would be good if the engine could allow to batch retrieve the data (in a more compact way).
Signed-off-by: Heiko W. Rupp<EMAIL_ADDRESS>

I'm not sure what you mean with batch fetching. You can already fetch all the triggers per tenant in a single call?

Is your last sentence a question to yourself?
The UI fetches policies in batches of n policies. Default for n is 10. So
fetching potentially 100 policies from the engine may not be desired.
For batching I mean if it is possible to fetch exactly those 10 .
Also I can imagine that returning a map <triggerId,triggerTime> may be more
economical than always fetching/transferring full trigger structures
Michael Burman<EMAIL_ADDRESS>schrieb am Fr. 20. März 2020 um
08:52:

I'm not sure what you mean with batch fetching. You can already fetch all
the triggers per tenant in a single call?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/RedHatInsights/custom-policies-ui-backend/pull/54#issuecomment-601573229,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABS25RKFXJX7CRFKWG37KTRIMOB5ANCNFSM4LELUVMA
.

--
Reg. Adresse: Red Hat GmbH, Technopark II, Haus C,
Werner-von-Siemens-Ring 14, D-85630 Grasbrunn
Handelsregister: Amtsgericht München HRB 153243
Geschäftsführer: Charles Cachera, Michael Cunningham, Michael O'Neill, Eric
Shander

"For batching I mean if it is possible to fetch exactly those 10". Yes, engine supports paged fetching as well as with a list of triggerIds.

I mark this as ready-for-review. We need to rewrite the getPolicies() case once the engine supports batch fetching of something with lastEval time. Currently TriggerHandler.findTriggers() does only return thing Triggers, that do not have the lastEval time nor the conditions.


chore: add usePython3 and increase version of ort-action

Codecov Report

Merging #91 (1d03915) into main (be123bd) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##            main     #91   +/-   ##
=====================================
  Coverage   7.52%   7.52%
=====================================
  Files          9       9
  Lines        226     226
=====================================
  Hits          17      17
  Misses       209     209

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 be123bd...1d03915. Read the comment docs.

:tada: This PR is included in version 1.0.8-develop.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add posteriors for effects to plot with CRI

Add derived parameters of the fixed effects to make effect sizes on the real scale with credible intervals. For example,
for(i in 1:R) {
eff_elev[i] <- exp(alpha0 + alpha1 * elev[i] + alpha2 * elev2[i])
}

This will be useful for plotting in R so practitioners can see the effects on the abundance and detection scales with CRI rather than just the log and logit scales.

Consider making elev_prime = 450 - 2050 by 10 or something like that so it's the posterior prediction and smoother


[WIP] model: tensorflow: Refactor tensorflow model

references #383
the tensorflow model will be refactored to improve code reusability and make things more neat.

Hello @pdxjohnny , thanks for pointing that out, I was thinking of moving the accuracy and predict to TensorflowModelContext? But I am not able to find how to do it. I was thinking maybe a decorator will help but, I am not sure how, can you help me with this.

@pdxjohnny , i moved some of the common code in predict method to TensorflowModelContext. and regarding the accuracy method, i did not feel it needed refactoring, so its left untouched. please let me know of any changes you may want : )

Nice work @sk-ip!! Thanks for cleaning things up in there! :)


NoCredentialsError due to wrongly breaking the retry loop

Description
In some cases IMDSFetcher._fetch_metadata_token can ignore your AWS_METADATA_SERVICE_NUM_ATTEMPTS configuration, and you will end up getting NoCredentialsError on the first attempt.
Details
Our web app runs on EC2 with role-based authentication and relies on InstanceMetadataFetcher.
We have this config for the metadata service:

AWS_METADATA_SERVICE_TIMEOUT=5
AWS_METADATA_SERVICE_NUM_ATTEMPTS=3

When the load goes high and autoscaling launches a new EC2 instance, the web app of the new instance starts to bomb the metadata service with credential requests. (Boto sessions are created upon the first need, then they persist for the lifetime of the workers, but due to the high load on startup all the workers are busy getting the credentials in parallel).
As a result of this scenario, a low number of boto sessions fail to get credential, and due to #1791 they get stuck in repeatedly raising NoCredentialsError.
We attempted a trivial workaround by setting the number of retries higher:

AWS_METADATA_SERVICE_TIMEOUT=5
AWS_METADATA_SERVICE_NUM_ATTEMPTS=12

But no change, the same chain of NoCredentialsError again.
By adding logs to our code we identified that instead of exhausting all the configured 12 attempts, InstanceMetadataFetcher gave up after the first one.
Looking at the code it was easy to point out the reason in IMDSFetcher._fetch_metadata_token:
If the first attempt ends with ReadTimeoutError in the for loop of _fetch_metadata_token then the loop will break prematurely, failing to exhaust configured AWS_METADATA_SERVICE_NUM_ATTEMPTS. If you look at the definition of RETRYABLE_HTTP_ERRORS at the top of the module, you can find it also has ReadTimeoutError, which indicates it is planned as retriable.
RETRYABLE_HTTP_ERRORS = (
    ReadTimeoutError, EndpointConnectionError, ConnectionClosedError,
    ConnectTimeoutError,
)
...
   def _fetch_metadata_token(self):
        self._assert_enabled()
        url = self._base_url + self._TOKEN_PATH
        headers = {
            'x-aws-ec2-metadata-token-ttl-seconds': self._TOKEN_TTL,
        }
        self._add_user_agent(headers)
        request = botocore.awsrequest.AWSRequest(
            method='PUT', url=url, headers=headers)
        for i in range(self._num_attempts):
            try:
                response = self._session.send(request.prepare())
                if response.status_code == 200:
                    return response.text
                elif response.status_code in (404, 403, 405):
                    return None
                elif response.status_code in (400,):
                    raise BadIMDSRequestError(request)
            except ReadTimeoutError:
                return None
            except RETRYABLE_HTTP_ERRORS as e:
                logger.debug(
                    "Caught retryable HTTP exception while making metadata "
                    "service request to %s: %s", url, e, exc_info=True)
            except HTTPClientError as e:
                if isinstance(e.kwargs.get('error'), LocationParseError):
                    raise InvalidIMDSEndpointError(endpoint=url, error=e)
                else:
                    raise
        return None

Expected behavior
ReadTimeoutError should not break the loop, because it violates the documented behaviour of AWS_METADATA_SERVICE_NUM_ATTEMPTS.
Commit and PR introducing this
The ReadTimeoutError handler was added by commit 66ee37d8965c371fcfa3060d8fb9946b3fe15d78 and PR #1895.
Obviously there was a reason for adding the change, but it created a contradiction with the list of RETRYABLE_HTTP_ERRORS, and also violated the documented behaviour of AWS_METADATA_SERVICE_NUM_ATTEMPTS.
Workaround

AWS_METADATA_SERVICE_TIMEOUT=60
AWS_METADATA_SERVICE_NUM_ATTEMPTS=1

Setting the values above defeats the impact of early exit on ReadTimeoutError.
Related issues
Unresolved issues found by this search might be related.

@stealthycoin any chance we could get some clarification? Is the plan to drop timeouts as a retrying cause and change the meaning of AWS_METADATA_SERVICE_NUM_ATTEMPTS? The lack of retrying combined with no exception getting raised downstream (#1791) causes poisoned containers in Kubernetes setups.

Hi @JustinTArthur,
My apologies for the delayed response.  I saw you probably had a chance to look at #1895— this change was made shortly after the rollout of imdsV2, where changes that we made to the credential-fetching strategy broke users whose deployments were incompatible with imdsV2 (see https://github.com/boto/botocore/issues/1892, https://github.com/boto/botocore/issues/1897 and https://github.com/aws/aws-cli/issues/4682).
At the time, ReadTimeout errors were retry-able, but in each case of incompatibility, customers were hitting a ReadTimeoutError, so the thought was to have this fail fast so they could quickly fall back to imdsvV1.  Reverting this change would likely break people again.
I've spoken with the team about this and we're willing to consider a flag or something to that effect that would allow ReadTimeout errors to be retry-able, but we're interested to hear more about what would be most helpful in getting around this.

https://github.com/boto/botocore/pull/1752/files
made by this


AppVeyor CI check is removed on PRs and failing on master

It seems it was removed on April, shouldn't it be readded? The current Travis test does not ensure dom.generated.d.ts has correct TypeScript syntax and type structure.

yes please. a PR would be appreciated.


feat(metadata): add metadata controller module

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Which docs this PR relation:
Ref #
Which toolchain this PR adaption:
No Breaking Change
Special notes for your reviewer:

Codecov Report

Merging #138 (9376256) into main (9c22040) will decrease coverage by 0.69%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #138      +/-   ##
==========================================
- Coverage   37.26%   36.56%   -0.70%
==========================================
  Files          78       81       +3
  Lines        5370     5472     +102
==========================================
  Hits         2001     2001
- Misses       3369     3471     +102

Impacted Files
Coverage Δ

core/consensus/src/adapter.rs
0.00% <0.00%> (ø)

core/consensus/src/consensus.rs
0.00% <0.00%> (ø)

core/consensus/src/engine.rs
0.00% <0.00%> (ø)

core/consensus/src/synchronization.rs
0.00% <0.00%> (ø)

core/mempool/src/adapter/mod.rs
46.47% <0.00%> (+0.19%)
:arrow_up:

core/mempool/src/lib.rs
66.16% <0.00%> (-0.51%)
:arrow_down:

core/mempool/src/tests/mod.rs
91.74% <0.00%> (ø)

core/metadata/src/adapter.rs
0.00% <0.00%> (ø)

core/metadata/src/lib.rs
0.00% <0.00%> (ø)

core/metadata/src/metadata_abi.rs
0.00% <0.00%> (ø)

... and 4 more


useMeasure hook elements box-sizing option

New Features
Current implementation of the useMeasure hook calculates the element bounds based on content-box model. It is a little unexpected behaviour when the project and all the layout is based on border-box model. Thus, current hook ignores paddings calculating element's width and height.
What is the new or updated feature that you are suggesting?
Suggesting adding the additional hook input parameter which has to define whether calculate content-box or border-box of the element.
Example of code calculating the border-box based content size:
//...
const [observerHandler] = useRafCallback<UseResizeObserverCallback>((entry) => {
    setMeasures({
        height: entry.borderBoxSize[0].blockSize,
        width: entry.borderBoxSize[0].inlineSize,
     });
});
// ...

And extend this logic into current calculation:
const [observerHandler] = useRafCallback<UseResizeObserverCallback>((entry) => {
    // For example:
    if (boxSizing === 'border-box') {
         setMeasures({
            height: entry.borderBoxSize[0].blockSize,
            width: entry.borderBoxSize[0].inlineSize,
         });
     } else {
         setMeasures({ width: entry.contentRect.width, height: entry.contentRect.height });
    }
});

Why should this feature be included?
Mostly border-box model is used, and content-box calculations are unexpected behaviour as expected to calculate the box-sizing model needed. Makes sense to add optional parameter to override the initial box-sizing model of the calculation to prevent unexpected miscalculations and debugging.

IMO it should be done the other way.
user must be able to provide matcher - function that matches captured ResizeObserverEntry to our height\width


preview the output of preprocessing

After preprocessing, there should be a way to preview the outputs
e.g. it should be possible to

preview the registered image and make sure the registration is reasonable
preview the segmentation and make sure it's reasonable
look at the jacobian image and/or feature image, to help the user understand exactly what is being input into the analysis

Would it make sense to attempt to use vtk.js for this? And/or, would vtk.js even be the right tool for this? Based on this page, it seems that only ITK.js can render .nii files.

Yes I think it makes sense, and it will be good practice for later approaching #32.

Based on this page, it seems that only ITK.js can render .nii files.

Based on that page it looks like the approach that mimics paraview would be to read the nifti files with itk.js and then visualize them with vtk.js. I haven't used itk.js before; would be nice to know more about this

@AlmightyYakob I noticed that if I go to the paraview glance web app and give it one of the sample nifti files it loads nicely.
Can we dig into how glance is doing it? Glance repo

@AlmightyYakob I noticed that if I go to the paraview glance web app and give it one of the sample nifti files it loads nicely.
Can we dig into how glance is doing it? Glance repo

Great find! Yes, I think that's a good idea.

I think a good first case to look at is previewing segmentations. It could be a slice view showing an overlay of two layers:

the original bias-corrected brain-extracted registered image (the output saved here) and
the segmentation label map (the output saved here)

with two sliders:

one slider controls which slice is being viewed
another slider controls the opacity of the segmentation label map overlay

and maybe the ability to scroll to zoom? What do you think-- is this a reasonable amount of stuff to cram into a popup window?

The scope of this particular issue has been reduced to previewing the segmentation only.


Variable in configuration file for primary color

Hi there,
Thanks so much for developing this theme! I'm really enjoying it.
I was wondering if it's possible to add an option that lets a variable in the configuration.yaml override the theme's primary color? I’ve set my own color, but it’s a bit tedious to update it whenever the theme gets updated.
I know it’s not a huge issue, but it would be a nice quality-of-life improvement.

Hi @nitras,
I'm glad you're enjoying Graphite! The idea of a user configurable primary theme color is fantastic. While there's no straightforward way to implement user defined colors similar to the HA default theme to my knowledge, your suggestion got me thinking.
I came up with a solution that leverages Graphite's token abstraction and a simple patch script. After testing it this morning, I'm happy to say it works seamlessly! The process involves scanning the theme files and replacing the primary color tokens with a value provided by the user via a script. It's an easy and relatively elegant approach.
I'll be sharing an theme update later today with this custom feature for you and anyone else interested. Thanks again for suggesting this!

Hi @nitras,
I'm glad you're enjoying Graphite! The idea of a user configurable primary theme color is fantastic. While there's no straightforward way to implement user defined colors similar to the HA default theme to my knowledge, your suggestion got me thinking.
I came up with a solution that leverages Graphite's token abstraction and a simple patch script. After testing it this morning, I'm happy to say it works seamlessly! The process involves scanning the theme files and replacing the primary color tokens with a value provided by the user via a script. It's an easy and relatively elegant approach.
I'll be sharing an theme update later today with this custom feature for you and anyone else interested. Thanks again for suggesting this!

Thank you so much! It works perfectly. I really appreciate you seeing the value in this—sometimes I tend to overthink these things.

Great to hear it's working well for you! I'm always up for a challenge if it means making the theme more user-friendly. I also believe that many themes tend to be overly imposing. While this can be great when everything works perfectly and matches the taste, it often makes things less accessible for others. I'm really glad we have such a supportive community here at Home Assistant that values helping one another. It would also mean a lot if you could leave a star since the HACS ranking is based on it 😊

Great to hear it's working well for you! I'm always up for a challenge if it means making the theme more user-friendly. I also believe that many themes tend to be overly imposing. While this can be great when everything works perfectly and matches the taste, it often makes things less accessible for others. I'm really glad we have such a supportive community here at Home Assistant that values helping one another. It would also mean a lot if you could leave a star since the HACS ranking is based on it 😊

Absolutely ;) Happy to be the 87th star.
Thanks!


[BUG]: asset rename isn't updated downstream

Describe the bug

asset rename isn't updated downstream

To Reproduce

Drop in a model and attach it another operator
Rename the model
Notice how the name of the model is still the old one if you were to hover over the left button for drilldown nav

Screenshots
https://github.com/user-attachments/assets/0db24b8e-e0e6-452f-80a9-1e5e138ec96a

Dealt in another issue


Haskell package servant doesn't compile on darwin due to python2.7-future

Issue description
The servant package has an override in configuration-common.nix which tacks on documentation building with sphinx. Sphinx depends via some path on python2.7-future which has only linux mentioned as a viable platform, so the whole build bombs during validation on darwin.
I had trouble overriding this by using packageOverrides and haskell.lib.overrideCabal but couldn't get the postInstall to be overridden. I tried this specifically:
  import <nixpkgs> {
    packageOverrides = pkgs: rec {
      haskellPackages = pkgs.haskellPackages.override {
        overrides = self: super: {
          servant = pkgs.haskell.lib.overrideCabal super.servant (old: {
            postInstall = "";
          });
        };
      };
    };
  }

I did have success cabal2nixing servant and overriding servant with that copy, like this:
  import <nixpkgs> {
    packageOverrides = pkgs: rec {
      haskellPackages = pkgs.haskellPackages.override {
        overrides = self: super: {
          # Do this because the servant package is patched in nixpkgs to build the documentation, which uses sphinx. Sphinx doesn't work on Darwin
          # because python2.7-future is not available on that platform.  We don't care about the docs so instead just use a clean expression
          # generated by cabal2nix.
          servant = self.callPackage ./servant.nix {};
        };
      };
    };
  }

It would be really swell if it was possible to disable the doc build like you can with other haskell packages using dontHaddock or similar, though just having it not blow up and not generate the docs would be fine in my view - the docs are available at http://haskell-servant.readthedocs.io/en/stable/
Steps to reproduce
nix-build '<nixpkgs>' -A haskellPackages.servant fails with:
error: Package ‘python2.7-future-0.15.2’ in ‘/nix/store/wq90d75hc8k4rz3syd9x7sk7x295a9cf-nixpkgs-17.03pre99928.61c48d9/nixpkgs/pkgs/top-level/python-packages.nix:11828’ is not supported on ‘x86_64-darwin’, refusing to evaluate.
a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.nixpkgs/config.nix.

(use ‘--show-trace’ to show detailed location information)

Technical details

System: macOS Sierra 10.12.3
Nix version: nix-env (Nix) 1.11.4
Nixpkgs version: 17.03pre99928.61c48d9

cc @Profpatsch

Have you tried just removing that linux restriction for python2.7-future?
Maybe it builds just fine.

@profpatsch I had not. I just tried it and it seems to build without error. should I make a PR to that effect?

Yes!

@Profpatsch #22837


Bump to v1.0.5

Backport some changes to c/storage to fix a potential unusable container store if the system restarts during certain c/storage ops.

@baude @rhatdan @vrothberg @giuseppe @TomSweeneyRedHat PTAL

LGTM

LGTM
(and thought I'd done so last night....)

/lgtm


docker inspect not reporting correct MAC address

I created a container with docker run -tid --publish-service=service5.prod.calico busybox
When I run docker inspect, I can see that most of the NetworkSettings are populated but the MacAddress isn't. Can anyone else reproduce this problem or is it just me? I see this problem with the overlay driver and my remote driver (calico) but not on the bridge driver.
Inspect output is below
docker inspect boring_newton
[
{
    "Id": "e991512c9e33c8c2151e1bb40f11f05d3df5f5ab33c4663924ad788006a9495b",
    "Created": "2015-07-21T16:44:17.143703918Z",
    "Path": "/bin/sh",
    "Args": [],
    "State": {
        "Running": true,
        "Paused": false,
        "Restarting": false,
        "OOMKilled": false,
        "Dead": false,
        "Pid": 14685,
        "ExitCode": 0,
        "Error": "",
        "StartedAt": "2015-07-21T16:44:17.594565389Z",
        "FinishedAt": "0001-01-01T00:00:00Z"
    },
    "Image": "8c2e06607696bd4afb3d03b687e361cc43cf8ec1a4a725bc96e39f05ba97dd55",
    "NetworkSettings": {
        "Bridge": "",
        "EndpointID": "a2dc14eb484c6dc6f29c7e6488edf5a639259857f32e3ca72366a950f626f6cf",
        "Gateway": "<IP_ADDRESS>",
        "GlobalIPv6Address": "",
        "GlobalIPv6PrefixLen": 0,
        "HairpinMode": false,
        "IPAddress": "<IP_ADDRESS>",
        "IPPrefixLen": 32,
        "IPv6Gateway": "",
        "LinkLocalIPv6Address": "",
        "LinkLocalIPv6PrefixLen": 0,
        "MacAddress": "",
        "NetworkID": "b35a41ef11c698e2972abc2ec029b6872d078eba70e5af01a9986a83e109b989",
        "PortMapping": null,
        "Ports": {},
        "SandboxKey": "/var/run/docker/netns/e991512c9e33",
        "SecondaryIPAddresses": null,
        "SecondaryIPv6Addresses": null
    },
    "ResolvConfPath": "/var/lib/docker/containers/e991512c9e33c8c2151e1bb40f11f05d3df5f5ab33c4663924ad788006a9495b/resolv.conf",
    "HostnamePath": "/var/lib/docker/containers/e991512c9e33c8c2151e1bb40f11f05d3df5f5ab33c4663924ad788006a9495b/hostname",
    "HostsPath": "/var/lib/docker/containers/e991512c9e33c8c2151e1bb40f11f05d3df5f5ab33c4663924ad788006a9495b/hosts",
    "LogPath": "/var/lib/docker/containers/e991512c9e33c8c2151e1bb40f11f05d3df5f5ab33c4663924ad788006a9495b/e991512c9e33c8c2151e1bb40f11f05d3df5f5ab33c4663924ad788006a9495b-json.log",
    "Name": "/boring_newton",
    "RestartCount": 0,
    "Driver": "aufs",
    "ExecDriver": "native-0.2",
    "MountLabel": "",
    "ProcessLabel": "",
    "Volumes": {},
    "VolumesRW": {},
    "AppArmorProfile": "",
    "ExecIDs": null,
    "HostConfig": {
        "Binds": null,
        "ContainerIDFile": "",
        "LxcConf": [],
        "Memory": 0,
        "MemorySwap": 0,
        "CpuShares": 0,
        "CpuPeriod": 0,
        "CpusetCpus": "",
        "CpusetMems": "",
        "CpuQuota": 0,
        "BlkioWeight": 0,
        "OomKillDisable": false,
        "Privileged": false,
        "PortBindings": {},
        "Links": null,
        "PublishAllPorts": false,
        "Dns": null,
        "DnsSearch": null,
        "ExtraHosts": null,
        "VolumesFrom": null,
        "Devices": [],
        "NetworkMode": "default",
        "IpcMode": "",
        "PidMode": "",
        "UTSMode": "",
        "CapAdd": null,
        "CapDrop": null,
        "RestartPolicy": {
            "Name": "no",
            "MaximumRetryCount": 0
        },
        "SecurityOpt": null,
        "ReadonlyRootfs": false,
        "Ulimits": null,
        "LogConfig": {
            "Type": "json-file",
            "Config": {}
        },
        "CgroupParent": ""
    },
    "GraphDriver": {
        "Name": "aufs",
        "Data": null
    },
    "Config": {
        "Hostname": "e991512c9e33",
        "Domainname": "",
        "User": "",
        "AttachStdin": false,
        "AttachStdout": false,
        "AttachStderr": false,
        "ExposedPorts": null,
        "PublishService": "service5.prod.calico",
        "Tty": true,
        "OpenStdin": true,
        "StdinOnce": false,
        "Env": null,
        "Cmd": [
            "/bin/sh"
        ],
        "Image": "busybox",
        "Volumes": null,
        "VolumeDriver": "",
        "WorkingDir": "",
        "Entrypoint": null,
        "NetworkDisabled": false,
        "MacAddress": "",
        "OnBuild": null,
        "Labels": {}
    }
}
]

#dibs

I think there is a need to define the keys for some of the iinformation returned from a network plugin as part of the endpointOperInfo such that they can be processed by docker. In the patch I just pushed I use MacAddress as the key for the MAC address information as I think it is what is used by the bridge driver. So for the MAC address to show up with docker inspect, the plugin needs to return the information with that key. If this is the correct approach, I can update the Overlay driver as well.

Does it need to be added to the endpointOperInfo? The response to the createEndpoint request already includes the MAC address.

@tomdee No, but the question is essentially what keys are to be used (for both receiving parameters by the plugin or returning information back to docker/libnetwork) and how to make sure these are well defined so plugins can be developed more easily.
@mavenugo @dave-tucker @aboch  I have looked into how best to specify the keys that network plugins can expect to get in the incoming requests and what they need to use to pass the information to libnetwork.
I think we want to have all the keys defined in a single place where the plugins can possibly directly use (at least when the plugin is written in go) or can be easily referenced while the plugin is being developed in other languages.
There are a couple of options I can think of:
1- Define and use the labels as specified in libnetwork/netlabel/labels.go
There are a couple of issues that make this less than desirable. There are keys being passed to network plugins that are not defined here. They are essentially obtained while the data structures defined in libnetwork/drivers/remote/api/api.go are marshaled. We either need to expand the net labels to include them or have the information needed by plugins in more than one place.
The other issue with this approach is that we will need to explicitly use these labels as the json encoding of fields specified in libnetwork/drivers/remote/api/api.go. This is certainly doable, but I believe by using encoding/json we have to explicitly specify these keys as strings (and not as reference to the exiting net label strings) as in for example
MacAddress  net.HardwareAddr json:com.docker.network.endpoint.macaddress
This means we have to hardcode these labels in two places, once as part of the netlabels and once in the api.go.
2- Use the libnetwork/drivers/remote/api/api.go as the single place where network plugins get these keys and possibly explicitly specify them through statements like:
MacAddress  net.HardwareAddr json:macAddress
and provide a function in the remote driver to parse/convert these keys to netlabel labels.
There is a small number of labels that need conversion and this can be done by a new function parseEndpointOperInfoResult which will be similar to what the bridge network functions parseEndpointOptions and parseContainerOptions are.
This approach requires updating this function when new labels are defined in netlabel. One can do this in a somewhat generic way for certain labels however, we still need to update the conversion function or table when new labels are added. However, for developing a network plugin one needs to only use/know the api package.
I think the second option is cleaner. What do you think? Are there better ways of doing this that you can think of?
Please let me know what you think and I will update the patch accordingly.

tried on master and looks like the mac is returned:
[...]
"NetworkSettings": {
            "Bridge": "",
            "SandboxID": "d527c3afde3c9a29629e4028f397e04b24dbb60319c475da677c4295ea0850c6",
            "HairpinMode": false,
            "LinkLocalIPv6Address": "",
            "LinkLocalIPv6PrefixLen": 0,
            "Ports": {},
            "SandboxKey": "/var/run/docker/netns/d527c3afde3c",
            "SecondaryIPAddresses": null,
            "SecondaryIPv6Addresses": null,
            "EndpointID": "",
            "Gateway": "",
            "GlobalIPv6Address": "",
            "GlobalIPv6PrefixLen": 0,
            "IPAddress": "",
            "IPPrefixLen": 0,
            "IPv6Gateway": "",
            "MacAddress": "",
            "Networks": {
                "net2": {
                    "IPAMConfig": {
                        "IPv4Address": "<IP_ADDRESS>"
                    },
                    "Links": null,
                    "Aliases": [
                        "d5464ea1cd19"
                    ],
                    "NetworkID": "rqfrlaafewitn2vttqfmqpn3s",
                    "EndpointID": "32a53d8b389719fa6958b139cf151f1fe956f9a7dd4859f433f5f73eb7a30bfb",
                    "Gateway": "",
                    "IPAddress": "<IP_ADDRESS>",
                    "IPPrefixLen": 24,
                    "IPv6Gateway": "",
                    "GlobalIPv6Address": "",
                    "GlobalIPv6PrefixLen": 0,
                    "MacAddress": "02:42:0a:00:01:02",
                    "DriverOpts": null
                }
            }
        }
    }


added factorial in haskell

Hello, thanks for the submission. I am not really sure what this is for, but we currently do not accept code without a corresponding chapter. If you want to write a chapter on this, then we might think about it.
Thanks!


Enablement of cloud drive for OpenStack in KubeOne

See the mother issue for explanation, enabled / disabled option should be available for this feature in KubeOne.

/label customer-request
/label sig/cluster-management
/label kind/feature

/label feature

/label kind/feature

/kind feature


Add copyright header to all files

I've followed the recommendation of the CDF regarding copyright headers, and attributed it to "The Spinnaker Authors" without a year.

As part of preparing this repo for moving under the spinnaker org, I also looked into adding CONTRIBUTING and CODE_OF_CONDUCT files.  It looks like these are actually set at the org-level in spinnaker/.github which indicates they should propagate to all sub-repos.  (I'm not sure exactly how that happens though as I don't see those in all the repos currently under spinnaker.)

Here is the copyright guidance provided by the CDF.


[FLINK-29441] Setup (fix) convergence configuration

What is the purpose of the change
Setup (fix) convergence configuration such that convergence is checked on CI
Brief change log

Enable convergence check (still requires -Pcheck-convergence to run it)

Verifying this change
Run locally:

Enabled: mvn validate -Pcheck-convergence
Disabled: mvn validate

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? n/a

Can see convergence check running in action logs:
[INFO] --- maven-enforcer-plugin:3.0.0-M1:enforce (dependency-convergence) @ flink-connector-dynamodb-parent ---
[INFO] --- maven-enforcer-plugin:3.0.0-M1:enforce (dependency-convergence) @ flink-connector-dynamodb ---
[INFO] --- maven-enforcer-plugin:3.0.0-M1:enforce (dependency-convergence) @ flink-sql-connector-dynamodb ---

Should we enforce minimum required Maven version too? https://github.com/apache/flink-connector-elasticsearch/blob/main/pom.xml#L1001

Should we enforce minimum required Maven version too? https://github.com/apache/flink-connector-elasticsearch/blob/main/pom.xml#L1001

We already inherit all these:
[INFO] --- maven-enforcer-plugin:3.0.0-M1:enforce (enforce-maven-version) @ flink-sql-connector-dynamodb ---
[INFO] --- maven-enforcer-plugin:3.0.0-M1:enforce (enforce-maven) @ flink-sql-connector-dynamodb ---
[INFO] --- maven-enforcer-plugin:3.0.0-M1:enforce (ban-unsafe-snakeyaml) @ flink-sql-connector-dynamodb ---
[INFO] --- maven-enforcer-plugin:3.0.0-M1:enforce (ban-unsafe-jackson) @ flink-sql-connector-dynamodb ---
[INFO] --- maven-enforcer-plugin:3.0.0-M1:enforce (forbid-log4j-1) @ flink-sql-connector-dynamodb ---
[INFO] --- maven-enforcer-plugin:3.0.0-M1:enforce (forbid-direct-akka-rpc-dependencies) @ flink-sql-connector-dynamodb ---
[INFO] --- maven-enforcer-plugin:3.0.0-M1:enforce (forbid-direct-table-planner-dependencies) @ flink-sql-connector-dynamodb ---
[INFO] --- maven-enforcer-plugin:3.0.0-M1:enforce (enforce-versions) @ flink-sql-connector-dynamodb ---


Update govuk-frontend to 3.10.0

Update govuk-frontend to 3.10.0

You can preview this change here:
Built with commit 96c3da64c4f6cf4598aa9e90a99c1f4552fa1e48
https://deploy-preview-1404--govuk-design-system-preview.netlify.app


OSError: Authentication failed. Maybe your password is invalid?

Using bimmer_connected on the command line as my 0.116.4 home assistant install reported an error on the bmw_connected_drive integration, this is the output I see.
I know my password is correct, I've changed it to try and 'reset' the app and website both work for me, so I think maybe BMW have changed something (I'm in the UK).
Am I the person having issues?
DEBUG:bimmer_connected.account:Getting vehicle list
DEBUG:bimmer_connected.account:getting new oauth token
DEBUG:urllib3.connectionpool:Starting new HTTPS connection (1): customer.bmwgroup.com:443
ERROR:bimmer_connected.account:Authentication failed. Maybe your password is invalid?
ERROR:bimmer_connected.account:('Connection aborted.', RemoteDisconnected('Remote end closed connection without response',))
Traceback (most recent call last):
File "/home/dan/.local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 677, in urlopen
chunked=chunked,
File "/home/dan/.local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 426, in _make_request
six.raise_from(e, None)
File "", line 3, in raise_from
File "/home/dan/.local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 421, in _make_request
httplib_response = conn.getresponse()
File "/usr/lib/python3.6/http/client.py", line 1331, in getresponse
response.begin()
File "/usr/lib/python3.6/http/client.py", line 297, in begin
version, status, reason = self._read_status()
File "/usr/lib/python3.6/http/client.py", line 266, in _read_status
raise RemoteDisconnected("Remote end closed connection without"
http.client.RemoteDisconnected: Remote end closed connection without response
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/dan/.local/lib/python3.6/site-packages/requests/adapters.py", line 449, in send
timeout=timeout
File "/home/dan/.local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 727, in urlopen
method, url, error=e, _pool=self, _stacktrace=sys.exc_info()[2]
File "/home/dan/.local/lib/python3.6/site-packages/urllib3/util/retry.py", line 410, in increment
raise six.reraise(type(error), error, _stacktrace)
File "/home/dan/.local/lib/python3.6/site-packages/urllib3/packages/six.py", line 734, in reraise
raise value.with_traceback(tb)
File "/home/dan/.local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 677, in urlopen
chunked=chunked,
File "/home/dan/.local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 426, in _make_request
six.raise_from(e, None)
File "", line 3, in raise_from
File "/home/dan/.local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 421, in _make_request
httplib_response = conn.getresponse()
File "/usr/lib/python3.6/http/client.py", line 1331, in getresponse
response.begin()
File "/usr/lib/python3.6/http/client.py", line 297, in begin
version, status, reason = self._read_status()
File "/usr/lib/python3.6/http/client.py", line 266, in _read_status
raise RemoteDisconnected("Remote end closed connection without"
urllib3.exceptions.ProtocolError: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response',))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/dan/.local/lib/python3.6/site-packages/bimmer_connected/account.py", line 112, in _get_oauth_token
expected_response=expected_response_code, post=True)
File "/home/dan/.local/lib/python3.6/site-packages/bimmer_connected/account.py", line 155, in send_request
params=params)
File "/home/dan/.local/lib/python3.6/site-packages/requests/api.py", line 119, in post
return request('post', url, data=data, json=json, **kwargs)
File "/home/dan/.local/lib/python3.6/site-packages/requests/api.py", line 61, in request
return session.request(method=method, url=url, **kwargs)
File "/home/dan/.local/lib/python3.6/site-packages/requests/sessions.py", line 530, in request
resp = self.send(prep, **send_kwargs)
File "/home/dan/.local/lib/python3.6/site-packages/requests/sessions.py", line 643, in send
r = adapter.send(request, **kwargs)
File "/home/dan/.local/lib/python3.6/site-packages/requests/adapters.py", line 498, in send
raise ConnectionError(err, request=request)
requests.exceptions.ConnectionError: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response',))
Traceback (most recent call last):
File "/home/dan/.local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 677, in urlopen
chunked=chunked,
File "/home/dan/.local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 426, in _make_request
six.raise_from(e, None)
File "", line 3, in raise_from
File "/home/dan/.local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 421, in _make_request
httplib_response = conn.getresponse()
File "/usr/lib/python3.6/http/client.py", line 1331, in getresponse
response.begin()
File "/usr/lib/python3.6/http/client.py", line 297, in begin
version, status, reason = self._read_status()
File "/usr/lib/python3.6/http/client.py", line 266, in _read_status
raise RemoteDisconnected("Remote end closed connection without"
http.client.RemoteDisconnected: Remote end closed connection without response
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/dan/.local/lib/python3.6/site-packages/requests/adapters.py", line 449, in send
timeout=timeout
File "/home/dan/.local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 727, in urlopen
method, url, error=e, _pool=self, _stacktrace=sys.exc_info()[2]
File "/home/dan/.local/lib/python3.6/site-packages/urllib3/util/retry.py", line 410, in increment
raise six.reraise(type(error), error, _stacktrace)
File "/home/dan/.local/lib/python3.6/site-packages/urllib3/packages/six.py", line 734, in reraise
raise value.with_traceback(tb)
File "/home/dan/.local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 677, in urlopen
chunked=chunked,
File "/home/dan/.local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 426, in _make_request
six.raise_from(e, None)
File "", line 3, in raise_from
File "/home/dan/.local/lib/python3.6/site-packages/urllib3/connectionpool.py", line 421, in _make_request
httplib_response = conn.getresponse()
File "/usr/lib/python3.6/http/client.py", line 1331, in getresponse
response.begin()
File "/usr/lib/python3.6/http/client.py", line 297, in begin
version, status, reason = self._read_status()
File "/usr/lib/python3.6/http/client.py", line 266, in _read_status
raise RemoteDisconnected("Remote end closed connection without"
urllib3.exceptions.ProtocolError: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response',))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/dan/.local/lib/python3.6/site-packages/bimmer_connected/account.py", line 112, in _get_oauth_token
expected_response=expected_response_code, post=True)
File "/home/dan/.local/lib/python3.6/site-packages/bimmer_connected/account.py", line 155, in send_request
params=params)
File "/home/dan/.local/lib/python3.6/site-packages/requests/api.py", line 119, in post
return request('post', url, data=data, json=json, **kwargs)
File "/home/dan/.local/lib/python3.6/site-packages/requests/api.py", line 61, in request
return session.request(method=method, url=url, **kwargs)
File "/home/dan/.local/lib/python3.6/site-packages/requests/sessions.py", line 530, in request
resp = self.send(prep, **send_kwargs)
File "/home/dan/.local/lib/python3.6/site-packages/requests/sessions.py", line 643, in send
r = adapter.send(request, **kwargs)
File "/home/dan/.local/lib/python3.6/site-packages/requests/adapters.py", line 498, in send
raise ConnectionError(err, request=request)
requests.exceptions.ConnectionError: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response',))
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/home/dan/.local/bin/bimmerconnected", line 11, in
sys.exit(main())
File "/home/dan/.local/lib/python3.6/site-packages/bimmer_connected/cli.py", line 240, in main
args.func(args)
File "/home/dan/.local/lib/python3.6/site-packages/bimmer_connected/cli.py", line 116, in fingerprint
log_responses=time_dir)
File "/home/dan/.local/lib/python3.6/site-packages/bimmer_connected/account.py", line 61, in init
self._get_vehicles()
File "/home/dan/.local/lib/python3.6/site-packages/bimmer_connected/account.py", line 226, in _get_vehicles
self._get_oauth_token()
File "/home/dan/.local/lib/python3.6/site-packages/bimmer_connected/account.py", line 117, in _get_oauth_token
raise OSError(msg) from exception
OSError: Authentication failed. Maybe your password is invalid?

Looks to be as reported in #208 hadn't spotted this issue further down the responses in that thread, closing and adding to that report


Korean translation is incorrect in "Task asynchronous programming model"

en-us: A synchronous method returns when its work is complete (step 5), but an async method returns a task value when its work is suspended (steps 3 and 6).
ko-kr: 비동기 메서드는 작업이 완료될 때 반환되지만(5단계) 비동기 메서드는 작업이 일시 중단될 때 반환됩니다.
Since synchronous means '동기' and async means '비동기' in Korean, ko-kr should be

~비~ 동기 메서드는 작업이 완료될 때 반환되지만(5단계) 비동기 메서드는 작업이 일시 중단될 때 반환됩니다.(3~6단계)

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

ID: fbd97c63-2887-1891-fb08-9cd78659c2db
Version Independent ID: 1ae4494a-ca0b-a1da-cdcf-522ef79e0fc5
Content: The Task Asynchronous Programming (TAP) model with async and await (C#)'
Content Source: docs/csharp/programming-guide/concepts/async/task-asynchronous-programming-model.md
Product: dotnet-csharp
Technology: csharp-async
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

@JasonCard

Hello @dasg34
Thank you for your contribution! Since your comments are specific to the localization of the documentation, we have moved this issue to the corresponding Microsoft team and will close this one.
Kind regards,
Microsoft DOCS Global Experience Team


Make everything asynchronous

Right now the module is very synchronous with the callbacks and some code execution.
I'm mostly concerned about the TTL check period which goes through all the keys synchronously.
Everything should be deferred using process.nextTick / setImmediate / setTimeout(fn,0)

That said, doing a process.nextTick() after every operation will add a ton of overhead and make the operation take a really long time.


Problems for USBatchNorm2d and fedalign loss

Hi, thanks for the great work.
However, I have some problems while reading code. Not sure if I miss something.
In slimmable_ops.py line 146-147, should they by
self.running_mean[:c]
self.running_var[:c]

By the way, for line 122, 136, and 137, I can't figure out why you need an additional self.bn=nn.BatchNorm2d here?

One more question. In methods/fedalign.py line 54,  not sure why  you multiply (loss_CE/loss.item()) before loss?
loss = self.args.mu*(loss_CE/loss.item())*loss

Hello @calvin89029! Thank you for your interest.
Here, track_running_stats=False, and therefore self.running_mean and self.running_var are None. This means that the batchnorm operation on lines 144-152 will use batch statistics, which is what we want for the subnetworks. For the full network, we maintain the running mean and variance, which is why we have self.bn (track_running_stats=True by default). This is based on the repos I referenced here. Nonetheless, it could probably be simplified by setting track_running_stats=True here, remove self.bn, replace lines 136 and 137 with
self.running_mean[:c]
self.running_var[:c]

And replace lines 146 and 147 with
None,
None,

But in the end it should be the same :)
For your second question, I discuss this in Section E of the supplemental material. Basically, it's just to ensure that loss is on relatively the same scale with loss_CE.
Hope this helps!

Hi @mmendiet, thank you for the reply.
For the first question, yeah I think that will be much clearer.
For the second one, I just reimplement your work and train on some other dataset. However, if I train without gradient clip and loss_CE scaling, the Lipschitz loss seems to get larger and larger along the training process and finally explode.  Did you encounter this problem too? And is this the reason why you add these two tricks?


Missing Cargo.lock

Since this is a binary project it should distribute its Cargo.lock. Some distros depend on it for reproducible builds.

Hi!
I'm not a big fan of having a Cargo.lock file in the source code, rather than a minimum version.
When the project is being compiled, it's nice to have the latest compatible version of all dependencies, rather than aging, possibly insecure or broken (due to changes in the compiler) versions that have been hard-coded in a Cargo.lock file.
This is an executable file, so it doesn't matter much, but with libraries, these Cargo.lock file also tend to require many versions of the exact same sub-dependencies, just because the dependencies includeCargo.lock files generated at different times.
For building reproducible packages, you may want to add this Cargo.lock file.
But I'd prefer to avoid including it in the project.


[Feature Request]

Describe the solution you'd like
A clear and concise description of what you want to happen.

I would love to see a colour switching feature, which allows the main colour scheme to be changed in the settings from the default blue.

Additional context
Add any other context or screenshots about the feature request here.

N/A

This already exists


Update KubeCon 2024-2025 events

This PR makes the "attend KubeCon" links up-to-date on the main page. It removes the past KubeCon China event and adds an upcoming European one in 2025.

/approve


Hide not matched tab during search?

This is not necessary because we can show multiple windows in one view.


JSONSchemaType doesn't infer nullable types

What version of Ajv are you using? Does the issue happen if you use the latest version?
8.11.2
This issue is not present in 8.11.0
Your typescript code

import { JSONSchemaType } from "ajv";

const schema: JSONSchemaType<{ foo: string | null }> = {
	type: "object",
	properties: {
		foo: { type: "string", nullable: true },
	},
	required: ["foo"],
};

void schema;

Typescript compiler error messages
Type '{ type: "object"; properties: { foo: { type: "string"; nullable: true; }; }; required: "foo"[]; }' is not assignable to type 'UncheckedJSONSchemaType<{ foo: string | null; }, false>'.
  The types of 'properties.foo' are incompatible between these types.
    Type '{ type: "string"; nullable: true; }' is not assignable to type '{ $ref: string; } | ({ anyOf: readonly UncheckedJSONSchemaType<string | null, false>[]; } & { [keyword: string]: any; $id?: string | undefined; $ref?: string | undefined; $defs?: Record<...> | undefined; definitions?: Record<...> | undefined; } & { ...; }) | ({ ...; } & ... 1 more ... & { ...; }) | ({ ...; } & ... 2...'.
      Types of property 'nullable' are incompatible.
        Type 'true' is not assignable to type 'false'.ts(2322)

Describe the change that should be made to address the issue?
There should be no TypeScript errors. As mentioned, this does not give any compiler errors in version 8.11.0
Are you going to resolve the issue?
I'm not familiar with the internals of ajv

I get the exact same problem right on my project but I think the new behaviour is closer to JsonSchema specification
And BTW, if I try to put ['string', null] as a type in my JSONSchemaType I get this error:
Type '{ type: ("number" | null)[]; }' is not assignable to type '{ type: "number" | "integer"; }'.

I found the same problem, I downgraded to version 8.11.0 meanwhile

Seem's that this is a bug introduced with the commit https://github.com/ajv-validator/ajv/commit/00b3939ba545e87f585b5ee5e93d26f025454fc6
IMO it's against the JSON Schema specification.
JSON Schema imposes no restrictions on type: JSON Schema can describe any JSON value, including, for example, null http://json-schema.org/draft/2020-12/json-schema-core.html
And from the link @juliensnz posted It’s important to remember that in JSON, null isn’t equivalent to something being absent.
With this change, it became impossible to define a required field that has a null value.
{
  type: "object",
  properties: {
    foo: { type: "null" },
  },
  required: ["foo"],
}

While it should be perfectly fine.

Same problem with {type: 'integer', nullable: true}

Same issue for me:
type User = {
    firstName: string | null
}

const userSchema: JSONSchemaType<User> = {
    type: "object",
    properties: {
        firstName: {
            type: "string",
            nullable: true
        }
    },
    required: [
        "firstName"
    ]
}

The types of 'properties.firstName' are incompatible between these types.
...
Types of property 'nullable' are incompatible.
Type 'true' is not assignable to type 'false'.

I have to change my type from string | null to string | undefined, which makes no sense. In both TS and JSON there's a difference between "set x to nothing" and "I didn't mention x" ;-)

another solution (not prettier) could be to do:
type: ['string', 'null'] as unknown as 'string',

For reference, document about migrating from OpenAPI 3.0 to 3.1 describes it as
// OpenAPI v3.0
{ type: 'string', nullable: true }

// OpenAPI v3.1
{ type: ['string', 'null'] }

Also ran into this issue. It appears the problem is with the definition of Nullable<T>:
type Nullable<T> = undefined extends T
  ? {
      nullable: true
      const?: null // any other value here would make `null` fail
      enum?: (T | null)[] // `null` must be explicitely included in "enum"
      default?: T | null
    }
  : {
      const?: T
      enum?: T[]
      default?: T
    }

undefined does not extend null, and therefore declaring a nullable property as per the definition in the Ajv JSON Schema documentation will cause a Typescript compilation error.
For example, the following schema:
{
  "type": "object",
  "properties": {
    "foo": {
      "type": "string",
      "nullable": true
    }
  }
}

will fail to validate against the following type declaration:
type MyObject = { foo: string | null }

but will successfully validate against:
type MyObject = { foo: string | undefined }
type MyObject = { foo?: string }

...which is obviously unintended behaviour.
I believe the fix should be as simple as making Nullable<T> check against null rather than undefined. However, the test suite is very clearly using nullable for optional/undefined properties rather than null ones, so there seems to be some internal disagreement with Ajv's own specification.

any update on this?

This probably doesn't help but it appears if you make the field optional (e.g. add ?) then the schema works.
e.g.
import { JSONSchemaType } from "ajv";

const schema: JSONSchemaType<{ foo?: string | null }> = {
	type: "object",
	properties: {
		foo: { type: "string", nullable: true },
	},
	required: ["foo"],
};

void schema;

In my case ajv was coalescing null to 0 -- by adding ? I was able to add nullable: true and ajv also stopped coalescing.

Hello everyone, who is encountered this issue!
What helped me here to get rid of this issue at least temporary:

Use normal scheme links, no need for ? or something else, like it was done in the initial issue message

import { JSONSchemaType } from "ajv";

const schema: JSONSchemaType<{ foo: string | null }> = {
	type: "object",
	properties: {
		foo: { type: "string", nullable: true },
	},
	required: ["foo"],
};

void schema;

Use ajv version 8.11.0
Use typescript version 5.0.4: for example typescript 5.2.2 proceeded reproduce this issue even with previously mentioned item.

Overall, looks like issue connected not only with ajv, but with typescript dependency changes too, unfortunately, there is not enough time for taking deeper look into possible reasons. Hope, that it will help someone to resolve it locally in your projects and maybe found the root cause and fix it in the next update.

I will top this all with this workaround:
  { type: 'string', nullable: true as false }

Just see it as a Zen Koan

And if you embed this into another schema you must not use 'nullable' (8.17.1)
e.g.  oneOf: [schema]
now I have to duplicate the code
at least the behavior must be consistent to follow the DRY principle.

Ran into this issue as well in ajv v8.12.0 with Typescript v5.3.3

I encountered this after following OpenAI's suggestions to use null unions https://platform.openai.com/docs/guides/structured-outputs/all-fields-must-be-required. I'm not sure how to specify the field in the required array and also have it be nullable (either with a null union or nullable: true) in a way that makes TypeScript happy.


Fix  usage

Multiple changes, in different commits:

remove image width from <img src>
fix sizes attributes (detailed in the commit message)
remove useless <source>s
use <img> for the fallback instead of the second <source>
improve code readability

I hope it is simple enough to read and understand. I can explain more if needed.

Thank you lots for doing all of this! @nhoizey
Your notes on fixing the sizes attribute are very in depth and I really appreciate it! 👍

Yes indeed, Lighthouse uses a mobile viewport by default (Moto G4 - 360x640) for audits. That is the only thing I could think of too, that the images were smaller than required.
I'm still a bit bugged why lighthouse thinks it could be lighter when 33.3vw is included in sizes, but I agree that 1100KB-1300KB is a much more logical size for that many images.
Ok great. I will add the lazy loading attribute to images 4 to 15. Thanks! @nhoizey

I found the code of the audit:
https://github.com/GoogleChrome/lighthouse/blob/master/lighthouse-core/audits/byte-efficiency/uses-responsive-images.js#L84-L85
It shows an issue if an image weights at least 4KiB more than necessary, but the wasted KiB are computed from the number of wasted pixels. That's an approximation.
Anyway, as you say, they use a 360px wide viewport, which means the images are 296px wide (360 - 2 * 2 * 16 = 296), but loaded images are actually 320px wide, hence the "waste" they report.
You could add images that are really 296px wide (or round it to 300px) to please Lighthouse, but it might not be the best choice for actual users of the site. That's the limit of such audit based on one single arbitrary viewport width.
I started working on Daltons, a tool to help chose the best image widths, but it is not ready for production yet, unfortunately.

Ah very nice find for the audit code. The wasted KiB are indeed an approximation using Math.round(totalBytes * wastedRatio). I feel more comfortable having the audit error show now that I understand how its computed.
This makes much more sense. The image weights can be optimized "a bit" more by changing the images to 296px or 300px wide since they are actually 320px and those extra wasted KiB are whats throwing the audit issue.
I see. I'm not sure I want to sacrifice user experience to please Lighthouse for a such a small optimization in KiB. Yes you're right. I didn't put the pieces together but the audit is based on a single arbitrary viewport width, it shouldn't be the truth for all viewports (That would be a nice Lighthouse feature to run audits at any viewport you'd like).
I took a look at Daltons and gave it a star. Once its ready for production I will most definitely use this tool for choosing image widths as I've figured out firsthand its not always easy.
Thank you for the great information and all your help on this project!

I will maybe write about such topics on my blog, I'll let you know.

Also, after adding loading="lazy" attribute to the fallback <img> element like you mentioned. The Lighthouse audit on live site for mobile yielded a much smaller page weight (462 KB). Woohoo!

Ok cool, I'm looking forward to it. I bookmarked your website as well. Lots of great articles you've posted there that I'm planning to read through in the coming weeks.

The Lighthouse audit on live site for mobile yielded a much smaller page weight (462 KB). Woohoo!

Awesome! 👍

Lots of great articles you've posted there that I'm planning to read through in the coming weeks.

Thanks!


Multi-cores

A quick question, does this run tests per-core in a separate process?

What I mean is that would this possibly run faster using this approach?
https://medium.com/@ClaytonXia/multiprocessing-in-nodejs-a810e7d0b071
Instead of threads it would delegate work across each core independently. It works wonders in Python.


All Day Event Request

I needed the ability to have an event not be bound to a start and end time just start and end dates. No matter what I tried I kept getting errors because it required time. I ended up editing the files that generate the URLs to remove the time but it might be nice to have the option to have all day events. It looks as if Google and iCal automatically mark an event as all day if there is no time in the URL. Not sure about Yahoo - don't have an account.
e.g. &dates=20180802/20180805

Added in 2.0.1


[Feature-Request] Delayed Animations with Built-in Animated Widgets

Use case
Delayed Animations with Built-in Animated Widgets , like :
   AnimatedOpacity(
        duration: const Duration(milliseconds: 250), // TODO : animate after backdrop opened
        delay: const Duration(milliseconds: 350), // REQUESTED FEATURE
        opacity: isBackDropOpen ? 1 : 0,
        child: myBackDrop(),
    ),

Proposal
didn't find any pub.dev package to do this in the Widgets themselves,
and I'm not sure if it's possible.
but it's obvious how much easier and straight-forward it would become to animate with delays this way.
❤

by delay I mean the animation and changes (the opacity in this example) happen after this given delay has passed after changes in the code (isBackDropOpen in this example).

This would be very useful, I know it can be achieved with a controller, but to have this built-in would make it a lot easier to understand.


How to know what opts are supported in a route / consul tag?

I can see that I can specify opts in a urlprefix consul tag, as in urlprefix-/ foo=1 bar=2, but I can't find documented anywhere what opts are actually recognized by fabio as meaningful.
For example, let's say I want to set the maxconn that a service instance will accept to 100 in the tag, so that fabio directs at most 100 simultaneous connections to that service instance. I can't tell whether I should use maxconn=100, proxy.maxconn=100, or if this feature is even supported at all.
How can I know what opts are supported, or how to translate opts from the config file to be used in a consul tag?  Can this be added to the documentation somewhere?

My bad. I need to update the docs more consistently. The opts from the config file are different than the opts on the routes. Right now the following opts are supported on a route:
strip=/foo         # strip '/foo' from the call to the upstream server
proto=tcp          # tcp upstream
proto=https        # https upstream
tlsskipverify=true # skip upstream tls verification

I'll update the docs.

Thanks for agreeing to update the docs; it's appreciated.
Is the per-instance maxconn feature something you would consider adding to fabio? traefik currently supports this feature.
Should I open a separate issue to discuss the maxconn feature?

What do you need this for?

We have some Ruby-based web servers that use forking processes to handle requests (a la unicorn), with each process handling one request at a time. Thus, the maximum number of simultaneous requests that each server can serve at a time is a fixed number (equal to the number of forked processes). Any additional requests that come in to the web server will be queued on listen socket.
However, these additional requests can be served more efficiently if they get queued at the proxy instead of the web server (because they can go to the next available web server in the entire pool, instead of getting stuck in an arbitrary round-robin-selected web server socket, which may happen to be serving very long-running requests). In HAProxy, we are currently using a per-server maxconn to achieve this behaviour. Since we are trying to replace our HAProxy + consul-template solution with fabio because of the traffic-interruption-during-reload issue, so we'd like to do something analogous with maxconn in fabio.

How would this work for multiple fabio instances?
Frank Schröder

On 25. Apr 2017, at 10:32, Joe Eli McIlvain<EMAIL_ADDRESS>wrote:
We have some Ruby-based web servers that use forking processes to handle requests (a la unicorn), with each process handling one request at a time. Thus, the maximum number of simultaneous requests that each server can serve at a time is a fixed number (equal to the number of forked processes). Any additional requests that come in to the web server will be queued on listen socket.
However, these additional requests can be served more efficiently if they get queued at the proxy instead of the web server (because they can go to the next available web server in the entire pool, instead of getting stuck in an arbitrary round-robin-selected web server socket, which may happen to be serving very long-running requests). In HAProxy, we are currently using a per-server maxconn to achieve this behaviour. Since we are trying to replace our HAProxy + consul-template solution with fabio because of the traffic-interruption-during-reload issue, so we'd like to do something analogous with maxconn in fabio.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

@magiconair - Sorry, I'm not sure I understand the implication of the question. Can you elaborate?

If you have one service that can handle 10 connections and announces it through consul this is simple to keep track of if you have a single fabio instance. However, once you spin up the next one (for LB or HA purposes) then all fabios need to coordinate since the service can only handle 10 connections. So how would this work in if the number of fabio instances is dynamic, i.e. they can come and go?

Good question. We haven't had to deal with that issue because we use a static number of HAProxy servers that is known by our Chef recipe at the time when we deploy our HAProxy configuration template.

I've updated the main README, the route config language docs on the wiki (https://github.com/fabiolb/fabio/wiki/Routing#config-language), the Quickstart on the Wiki (https://github.com/fabiolb/fabio/wiki/Quickstart) and the online documentation in fabio itself.
I'm going to merge this change and close this ticket since this the original question was about documentation and I don't want to morph tickets. I'll open another ticket for the maxconn question.


When I run gulp serve, on chrome it does not load the local workbench. I get ERR_CONNECTION_REFUSED

Category

[x] Bug

Describe the bug
I am trying to setup my environment for spfx development. This is not the first time that I am setting it up. I had done it multiple times in different systems using https://docs.microsoft.com/en-us/sharepoint/dev/spfx/set-up-your-development-environment
But now in my system where I set this up freshly, and do gulp serve , I get ERR_CONNECTION_RESET in chrome browser.
Steps to reproduce

Follow https://docs.microsoft.com/en-us/sharepoint/dev/spfx/set-up-your-development-environment
Run gulp serve

Expected behavior
Local workbench must load
Environment details (development & target environment)

Your Developer Environment: [ Windows 10 ]

Target Environment: [SharePoint Online]

Framework: [Node.js v10]

Browser(s): [Chrome version:86.0.4240.75 (Official Build) (64-bit)]

Tooling: [VS Code | SPFx v1.11.1 ]

Additional details: The more context you can provide, the easier it is (and therefore quicker) to help.

Additional context
I am able to go to https://localhost:4321/ and see all the directories
It is the same behaviour in all other browsers.
I have chrome version:86.0.4240.75 (Official Build) (64-bit)
I have Node v10.22.1
My npm is v6.14.6
My gulp is , 3.9.1
sharepoint generator is 1.11.1
I use Windows 10
I have run gulp trust-dev-cert
I have not run npm audit fix
I don't have postgress sql installed which usually blocks port 5432
I have tried unistalling all (gulp,yeoman, node,..) and reinstalling
I have disabled my antivirus and tried
I don't have any extensions in my chrome browser
In my temp folder, I don't see workbench.html created
Basically I have tried to create a webpart for sharepoint online and then run gulp serve
Below is my gulp serve output
Build target: DEBUG
[15:45:43] Using gulpfile H:\GitCentralRepo\central\spfx1\gulpfile.js
[15:45:43] Starting gulp
[15:45:43] Starting 'serve'...
[15:45:43] Starting subtask 'configure-sp-build-rig'...
[15:45:43] Finished subtask 'configure-sp-build-rig' after 6.44 ms
[15:45:43] Starting subtask 'spfx-serve'...
[15:45:43] [spfx-serve] To load your scripts, use this query string: ?debug=true&noredir=true&debugManifestsFile=https://localhost:4321/temp/manifests.js
[15:45:43] Starting server...
Starting api server on port 5432.
Registring api: /workbench
Registring api: /
[15:45:44] Finished subtask 'spfx-serve' after 435 ms
[15:45:44] Starting subtask 'pre-copy'...
[15:45:44] Finished subtask 'pre-copy' after 15 ms
[15:45:44] Starting subtask 'copy-static-assets'...
[15:45:44] Starting subtask 'sass'...
[15:45:44] Server started https://localhost:4321
[15:45:44] LiveReload started on port 35729
[15:45:44] Running server
[15:45:44] Opening https://localhost:5432/workbench using the default OS app
[15:45:44] Finished subtask 'copy-static-assets' after 64 ms
[15:45:44] Finished subtask 'sass' after 312 ms
[15:45:44] Starting subtask 'tslint'...
[15:45:45] [tslint] tslint version: 5.12.1
[15:45:45] Starting subtask 'tsc'...
[15:45:45] [tsc] typescript version: 3.3.4000
[15:45:48] Finished subtask 'tsc' after 3.38 s
[15:45:49] Finished subtask 'tslint' after 4.9 s
[15:45:49] Starting subtask 'post-copy'...
[15:45:49] Finished subtask 'post-copy' after 1.27 ms
[15:45:49] Starting subtask 'collectLocalizedResources'...
[15:45:49] Finished subtask 'collectLocalizedResources' after 4.43 ms
[15:45:49] Starting subtask 'configure-webpack'...
[15:45:50] Finished subtask 'configure-webpack' after 757 ms
[15:45:50] Starting subtask 'webpack'...
[15:45:51] Finished subtask 'webpack' after 1.52 s
[15:45:51] Starting subtask 'configure-webpack-external-bundling'...
[15:45:51] Finished subtask 'configure-webpack-external-bundling' after 1.98 ms
[15:45:51] Starting subtask 'copy-assets'...
[15:45:51] Finished subtask 'copy-assets' after 20 ms
[15:45:51] Starting subtask 'write-manifests'...
[15:45:52] Finished subtask 'write-manifests' after 1.06 s
[15:45:52] Starting subtask 'reload'...
[15:45:52] Finished subtask 'reload' after 2.08 ms
This my serve.json
{ "$schema": "https://developer.microsoft.com/json-schemas/core-build/serve.schema.json";, "port": 4321, "https": true, "initialPage": "https://localhost:5432/workbench", "api": { "port": 5432, "entryPath": "node_modules/@microsoft/sp-webpart-workbench/lib/api/" } }
I also changed the port to 8080 and also ran netstat still no luck
Thanks for your contribution! Sharing is caring.
Submission guidelines

!!IMPORTANT!! - All submissions must complete the baseline sections included in this template. Ignoring or deleting this template may result in closing the issue with the label type:incomplete-submission.
Follow our guidance on How To Create Good Issues.
If you are reporting an issue around any of the documents or articles, please ensure that you have referenced the specific file or URL of the related document.
If you have technical questions about the framework, we monitor the tags #spfx, #spfx-webparts, and #spfx-tooling on SharePoint StackExchange.
Remember to include sufficient details and context.
If you have multiple questions, suggestions, or bugs, please submit them in separate issues.

DELETE THIS SECTION BEFORE SUBMITTING
Thanks for your contribution! Sharing is caring.

I ran the command, npm install gulp yo @microsoft/generator-sharepoint --global , once again.
This time it installed different version of gulp and sharepoint generator..

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
But still when I run gulp serve, I am getting same error

This issue is now fixed. It was caused by the internet security (Kaspersky).
I went to Settings->Additional->Network
I selected Monitor selected ports only
The I clicked on "Select" next to it and unchecked "Monitor all network ports for applications that are vulnerable to network attacks"


ReadPendingInput plays unsafe byte/char games with pending data

Ported from MSFT: 18047766.
I fixed an immediate crash bug in MSFT: 17987369 that affected this area, but realized at the time that it's really bad that the ReadPendingInput function in the console host is playing games with a void pointer and mix-and-match of byte versus character counting.
This represents investigating what we can do to stop mix/matching the char and byte counts in ReadPendingInput.
@adiviness might be working on something like this with his changes to input to support emoji...

Yup, I'm going to have to fix this up as well.


Need ability to kick job ID from a queue

Moved to https://github.com/george-infinity/bsw/issues/8


kubectl get nodes error connection :443 timeout

Is this a request for help?: yes

Is this an ISSUE or FEATURE REQUEST? (choose one): Issue

What version of acs-engine?:
C:\Ravi>acs-engine version
Version: canary
GitCommit: unset
GitTreeState: unset

Orchestrator and version (e.g. Kubernetes, DC/OS, Swarm) Kubernetes
What happened:
Not able to reach the nodes in the cluster
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know:

I am having exactly the same problem with iptables MASQUERADE when deploying acs-engine onto custom VNET. Kubelet won't start:
Feb 07 11:46:45 k8s-master-62187370-0 systemd[1]: kubelet.service: Service hold-off time over, scheduling restart.
Feb 07 11:46:45 k8s-master-62187370-0 systemd[1]: Stopped Kubelet.
Feb 07 11:46:45 k8s-master-62187370-0 systemd[1]: Starting Kubelet...
Feb 07 11:46:45 k8s-master-62187370-0 systemd[1]: Started ACPI event daemon.
Feb 07 11:46:45 k8s-master-62187370-0 bash[6938]: Bad argument `MASQUERADE'
Feb 07 11:46:45 k8s-master-62187370-0 bash[6938]: Try `iptables -h' or 'iptables --help' for more information.
Feb 07 11:46:45 k8s-master-62187370-0 systemd[1]: kubelet.service: Control process exited, code=exited status=2
Feb 07 11:46:45 k8s-master-62187370-0 systemd[1]: Failed to start Kubelet.
Feb 07 11:46:45 k8s-master-62187370-0 systemd[1]: kubelet.service: Unit entered failed state.
Feb 07 11:46:45 k8s-master-62187370-0 systemd[1]: kubelet.service: Failed with result 'exit-code'.

Fixed it by adding "vnetCidr" parameter to masterProfile. Apparently firstConsecutiveStaticIP is not enough to setup custom VNET. You also need to specify master CIDR.

@mcwienczek are you using azure networkPolicy? or are you using none (kubenet)?

azure network policy

Fixed it by adding "vnetCidr" parameter to masterProfile. Apparently firstConsecutiveStaticIP is not enough to setup custom VNET. You also need to specify master CIDR.

Thank you. Saved my night. I was going crazy trying to figure that one out.


x/vgo: vgo get myitcv.io/gogenerate fails with "module root is myitcv.io"

What version of Go are you using (go version)?
go version go1.10.1 linux/amd64
go version go1.10.1 linux/amd64 vgo:2018-02-20.1

Specifically vgo is at 890b798475a0fc2108fa88d9b2810d5f768f5752.
Does this issue reproduce with the latest release?
Yes, per above.
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/myitcv/.cache/go-build"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/tmp/tmp.2LGDYmXt1s"
GORACE=""
GOROOT="/home/myitcv/gos"
GOTMPDIR=""
GOTOOLDIR="/home/myitcv/gos/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build814184507=/tmp/go-build -gno-record-gcc-switches"
VGOMODROOT=""

What did you do?
I am in the process of trying to migrate myitcv.io/... packages to be vgo-compatible. In the process I want to move away from each having its own repository to use a single repo: https://github.com/myitcv/x.
I have started with myitcv.io/gogenerate.  I believe I have the various go-import meta tags set correctly; indeed old go get continues to work.
But vgo get does not:
cd `mktemp -d`
export GOPATH=$PWD
mkdir blah
cd blah/
echo 'module "rubbish.com/blah"' >> go.mod
echo 'package blah' >> blah.go

This vgo get:
vgo get rsc.io/pdf

works as expected:
vgo: finding rsc.io/pdf v0.0.0-20180112171046-225057252246
vgo: downloading rsc.io/pdf v0.0.0-20180112171046-225057252246

But if I now do:
vgo get myitcv.io/gogenerate

I get an error:
vgo get myitcv.io/gogenerate: module root is "myitcv.io"

What did you expect to see?
vgo successfully downloading myitcv.io/gogenerate which is a module within the mono-repo at https://github.com/myitcv/x.
Or am I misunderstanding something here within respect to multiple modules in a single repo?
What did you see instead?
Per above.

vgo is correctly fetching the meta-data from your website. It then ends up in this code from domain.go:
	// First look for new module definition.
	for _, imp := range imports {
		println("checking imp", imp.Prefix, imp.VCS, imp.RepoRoot)
		if path == imp.Prefix || strings.HasPrefix(path, imp.Prefix+"/") {
			if imp.VCS == "mod" {
				u, err := url.Parse(imp.RepoRoot)
				if err != nil {
					return nil, fmt.Errorf("invalid module URL %q", imp.RepoRoot)
				} else if u.Scheme != "https" {
					// TODO: Allow -insecure flag as a build flag?
					return nil, fmt.Errorf("invalid module URL %q: must be HTTPS", imp.RepoRoot)
				}
				return newProxyRepo(imp.RepoRoot, imp.Prefix), nil
			}
		}
	}

At this point, your meta-data has VCS set to "git". This is correct, according to "go help importpath".
However, vgo is looking for a VCS of "mod". I don't know why, that's not documented in the importpath help message.
Could you try changing your VCS type to "mod" and we'll see if we can get vgo to go into that "newProxyRepo" call?

One time, inexplicably, I got this:
$ vgo get myitcv.io/gogenerate
checking imp myitcv.io/gogenerate git https://github.com/myitcv/x
vgo: finding myitcv.io/gogenerate v0.0.0-20180404133421-640b382e1fbd
checking imp myitcv.io/gogenerate git https://github.com/myitcv/x
panic: mistake

goroutine 1 [running]:
golang.org/x/vgo/vendor/cmd/go/internal/mvs.buildList(0xc420210151, 0x1f, 0x0, 0x0, 0x159a840, 0xc4201f8d20, 0x0, 0xc4201d7620, 0x10129f8, 0x50, ...)
	/Users/jallen/go/src/golang.org/x/vgo/vendor/cmd/go/internal/mvs/mvs.go:80 +0x101c
golang.org/x/vgo/vendor/cmd/go/internal/mvs.BuildList(0xc420210151, 0x1f, 0x0, 0x0, 0x159a840, 0xc4201f8d20, 0x1, 0x1, 0x0, 0x0, ...)
	/Users/jallen/go/src/golang.org/x/vgo/vendor/cmd/go/internal/mvs/mvs.go:33 +0x78
golang.org/x/vgo/vendor/cmd/go/internal/mvs.Upgrade(0xc420210151, 0x1f, 0x0, 0x0, 0x159a880, 0xc420392820, 0xc420392800, 0x1, 0x1, 0x1d5c6e8, ...)
	/Users/jallen/go/src/golang.org/x/vgo/vendor/cmd/go/internal/mvs/mvs.go:209 +0x262
golang.org/x/vgo/vendor/cmd/go/internal/vgo.runGet(0x17b7a60, 0xc4200a00b0, 0x1, 0x1)
	/Users/jallen/go/src/golang.org/x/vgo/vendor/cmd/go/internal/vgo/get.go:102 +0x829
golang.org/x/vgo/vendor/cmd/go.Main()
	/Users/jallen/go/src/golang.org/x/vgo/vendor/cmd/go/main.go:155 +0x82e
main.main()
	/Users/jallen/go/src/golang.org/x/vgo/main.go:37 +0x25

I cannot reproduce it.

Thanks for helping to look into this @jeffallen

One time, inexplicably, I got this:

Apologies, you caught me mid-test of a change to the go-import meta tags. Now reverted so you should see the original error as reported in my description.

Could you try changing your VCS type to "mod" and we'll see if we can get vgo to go into that "newProxyRepo" call?

I can't do that if I want to retain backwards compatibility with regular go users (added steps in my original post that confirm go get still works)
In any case, my understanding was that the mod "VCS" in a go-import meta tag is used for published modules (and that further more it can be specific in addition to the git or otherwise meta tag because there is fallback logic). i.e. the server that responds to mod needs to understand the download protocol: in this situation I'm not doing that, I'm serving source from Github.

It appears there is actually a test for this exact case (but golang.org/x/net/context is used instead of myitcv.io/gogenerate). In this test the response is:
GET https://golang.org/x/net?go-get=1
200 OK
Alt-Svc: hq=":443"; ma=2592000; quic=51303431; quic=51303339; quic=51303338; quic=51303337; quic=51303335,quic=":443"; ma=2592000; v="41,39,38,37,35"
Cache-Control: private
Content-Type: text/html; charset=utf-8
Date: Tue, 30 Jan 2018 03:40:58 GMT
Server: Google Frontend
Vary: Accept-Encoding
X-Cloud-Trace-Context: fd9216519bb8006b586b7fdda695bcd0

<!DOCTYPE html>

<meta http-equiv="Content-Type" content="text/html; charset=utf-8"/>
<meta name="go-import" content="golang.org/x/net git https://go.googlesource.com/net">
<meta name="go-source" content="golang.org/x/net https://github.com/golang/net/ https://github.com/golang/net/tree/master{/dir} https://github.com/golang/net/blob/master{/dir}/{file}#L{line}">
<meta http-equiv="refresh" content="0; url=https://godoc.org/golang.org/x/net">

Nothing to see here; move along.

which appears to be identical in form to my meta tags.
The expected output from the test is:
module root is "golang.org/x/net"

so I can only assume this behaviour is intentional for now, i.e. support for submodules is a TODO?

The mod directives are documented at https://research.swtch.com/vgo-module in the "Publishing" section.
If you wrote an import "myitcv.io/generate" then during a subsequent build modfetch.Import would try that path and then back down to "myitcv.io" which would succeed. But "vgo get" assumes it is being given module paths, not arbitrary import paths, and so it doesn't try prefixes. This is another thing to fix as part of the overall vgo get rethink (see also #24105).

Thanks @rsc

The mod directives are documented at https://research.swtch.com/vgo-module in the "Publishing" section.

Noted; I'll look at adding such a meta tag once I've got the regular go-import working with vgo.

If you wrote an import "myitcv.io/generate" then during a subsequent build modfetch.Import would try that path and then back down to "myitcv.io" which would succeed

Ah, ok. So if I now try:
cd `mktemp -d`
export GOPATH=$PWD
mkdir blah
cd blah/
echo 'module "rubbish.com/blah"' >> go.mod
echo 'package blah
import _ "myitcv.io/gogenerate"
' >> blah.go
vgo build

Then I get an error
vgo: resolving import "myitcv.io/gogenerate"
vgo: finding myitcv.io (latest)
vgo: adding myitcv.io v0.0.0-20180405194116-cdcde380fd71
vgo: finding myitcv.io v0.0.0-20180405194116-cdcde380fd71
vgo: downloading myitcv.io v0.0.0-20180405194116-cdcde380fd71
vgo: import "rubbish.com/blah" ->
        import "myitcv.io/gogenerate"<EMAIL_ADDRESS>open<EMAIL_ADDRESS>no such file or directory

If I list the contents of GOPATH:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
So it seems the gogenerate subdirectory (module) is not being detected?
The go.mod's in the https://github.com/myitcv/x repo are:
./go.mod
./gogenerate/go.mod

I'll do some digging to see if I can work out what's going on.

But "vgo get" assumes it is being given module paths, not arbitrary import paths, and so it doesn't try prefixes.

Got it. This is going to require a bit of a mental shift (potentially only for me?) because I'm used to go get either getting a package or installing a program. My understanding of the latter in vgo speak is the one should use vgo install X (per https://github.com/golang/go/issues/24051). But I'll follow https://github.com/golang/go/issues/24105 for any updates.

I'll do some digging to see if I can work out what's going on.

@rsc I'm going to park my investigation for now because I'm not entirely clear whether you are expecting this scenario (i.e. submodules within a repo) to work or whether it's simply a TODO.
As you said, I can see the fallback logic taking effect. So this first call to try fails with a *ModuleSubdirError. I was half expecting to see something like:
var p string

if err, ok := err.(*ModuleSubdirError); ok {
	p = err.ModulePath
} else {
	p = pathpkg.Dir(path)
}

But even then we lose the original submodule in the subsequent call to try. We almost need to be passing round the submodule path (if there is one, else "") so that the ultimate call to modfetch.Query also knows that we're looking for a submodule.
At this point however I'm honestly guessing as to what's intended, hence the pause!

As a further update to this issue, just to note the myitcv.io/gogenerate example will now "work" because I've setup the go-import meta tag with the mod VCS for this package.
I've therefore setup myitcv.io/blah to demonstrate the problem as originally described (I will also update my comments above for completeness)


[VTA] Add VTA PYNQ metal_test bitstream program logic and fix compile issue.

Issue:
VTAProgram not exist and cause compile error.
No logic to program the bitstream into FPGA.
metal test still use pynq 2.1 library which not support on latest
pynq 2.4.
Solution:
remove old VTAProgram.
when setting is pynq, program the bitstream during compile.
change DMA link library to libcma.
Thanks for contributing to TVM!   Please refer to guideline https://docs.tvm.ai/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from Reviewers.

@tmoreau89 could you help to review this patch?

There is currently a big PR (#3135) that will be merged in for VTA (that also affects how the bitstream is programmed). Metal tests are somewhat deprecated and need a rehaul. I will review this patch once the PR is merged in as to make sure that this fix wont' interfere with the changes.

Upon inspection this doesn't look like it will interfere with PR#3135. I'm gonna go ahead with the code review.

@tmoreau89 rpc_server.py change done,  vtaprogram.py already get changed int program_bitstream.py?
@liangfu  vtaprogram.py get renamed into program_bitstream.py, please find the change in this new file for comments address.

@tmoreau89 , all review comments already get addressed,   change get tested on pynq board and it work good, please kindly let me know if you have any other comments.

Thank you for addressing the comments @huajsj