Fix MSTransferor logic for block distribution

Fixes #9551
Superseeds #9552
Status
In development
Description
It contains a few bug fixes, like:

a bug reading the data structure (accessing a dict instead of blockSize)
decrease number of chunks (sites) when there are less blocks than sites to distribute data
fix bug where transfer IDs might be skipped from the transfer doc (when a chunk has 0 blocks)

Is it backward compatible (if not, which system it affects?)
yes
Related PRs
none
External dependencies / deployment changes
none

test this please

I've done something that should be done since the beginning, implement a few unit tests for some of those methods. With that, I managed to spot two more issues. I'm going to test it all again.

test this please

This is good to go from my side.


Auto Bump Guardian and Garden submodules when a new version of garden-runc-release is available

Thank you for submitting an issue to the diego-release repository. We appreciate the feedback. To help us address your issue, please fill out the sections in the following template to the best of your ability:
Summary
We have a concourse job that verifies if Garden and Guardian submodules are out of sync with the latest garden-runc-release. In order to update the SHA, it requires a human intervention to bump to the latest. We should be able to automate this, so that when a new version of garden-runc-release is available, we bump the submodules in diego-release.
Expected Result
When the next version of garden-runc-release is out, we no longer have to bump the Garden and Guardian submodules, and validate-garden-version job remains green

The pipeline job is now working.  Garden and Guardian are currently up to date, so we'll have to keep an eye on the job next time https://github.com/cloudfoundry/garden-runc-release posts a new release. Closing out the issue.


Create manually triggered Github Action for running CI on behalf of forks

This PR introduces a GIthub Action which can be triggered manually to clone a fork to a new branch and open a draft PR with the "e2e" label which will (hopefully) run the e2e jobs.
Inputs:

PR number type: string (the PR to clone from)
Reviewed type: boolean (to confirm the user has reviewed the PR for usage/leakage of secrets)

Also:

The user triggering the action is enforced to be on the wrangler team

One more thing we might consider is to update this section of CONTRIBUTING.md, explaining that some tests may fail and that it requires a wrangler team member to run the action.


fix typo, and add dimension example

add example of how dimensions can be used

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank you @crs2here for your contribution and congratulations on your first PR merge to this repo.


chore: complement chaos

What problem does this PR solve?

This PR complement the experiment creation, now you can create IO Chaos, Kernel Chaos, Time Chaos and Stress Chaos from the dashboard.
Similarly, you can also use YAML to create any type of experiment.
For local review, please first run yarn to install updated deps.

Codecov Report

Merging #679 into master will decrease coverage by 1.11%.
The diff coverage is 33.18%.

@@            Coverage Diff             @@
##           master     #679      +/-   ##
==========================================
- Coverage   55.78%   54.67%   -1.12%
==========================================
  Files          68       68
  Lines        4383     4569     +186
==========================================
+ Hits         2445     2498      +53
- Misses       1768     1883     +115
- Partials      170      188      +18

Impacted Files
Coverage Δ

api/v1alpha1/podchaos_types.go
37.14% <ø> (ø)

controllers/networkchaos/ipset/ipset.go
37.14% <0.00%> (ø)

pkg/utils/chaosdaemon.go
51.21% <0.00%> (ø)

pkg/utils/grpc.go
0.00% <0.00%> (ø)

pkg/webhook/inject/inject.go
73.49% <0.00%> (-0.68%)
:arrow_down:

pkg/store/experiment/experiment.go
24.27% <12.28%> (-14.51%)
:arrow_down:

pkg/httpfs/fs.go
31.48% <31.48%> (ø)

controllers/twophase/types.go
54.20% <40.00%> (-1.68%)
:arrow_down:

controllers/networkchaos/netem/types.go
47.27% <50.00%> (ø)

pkg/utils/selector.go
52.41% <66.66%> (+0.17%)
:arrow_up:

... and 6 more

Continue to review full report at Codecov.

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

I really like this refactoring to put labels and annotations at the first page instead of the scope page. It is much clear now that these are labels of the chaos experiment, not the label selector in the scope

Overall this looks really awesome! This is a huge effort, only with some small nits about help messages. I don't think I am able to dive deeply to review the frontend code. If someone else can help then it would be great!

Unfortunately, now I am the only person that mainly dev with the front end. 🤣
So far, this is not a big problem. In the early stage of the dashboard, one person can maintain a rapid development status. I hope that more people can participate in the future, so that I can be free to do some other work, such as the API part.

@yeya24 I didn't reply to all comments. I will mark as resolved directly when I fixed.

/merge

/run-e2e-test

/run-e2e-test

/run-e2e-test

/merge

/run-e2e-test

/run-e2e-test

/run-e2e-test


Ref: Add error and stacktrace parameter to logger

:scroll: Description
Adds  error and stacktrace parameter to logger.
This is in order to get better bug reports and also to improve the output of our logs.
This is a non-breaking change because they're added as optional named parameters.
:bulb: Motivation and Context
Closes #462
:green_heart: How did you test it?
Existing test pass
:pencil: Checklist

[x] I reviewed submitted code
[ ] I added tests to verify changes
[x] I updated the docs if needed
[x] All tests passing
[x] No breaking changes

:crystal_ball: Next steps

Codecov Report

Merging #503 (663a691) into main (040738e) will increase coverage by 4.13%.
The diff coverage is n/a.

:exclamation: Current head 663a691 differs from pull request most recent head 34efa46. Consider uploading reports for the commit 34efa46 to get more accurate results

@@            Coverage Diff             @@
##             main     #503      +/-   ##
==========================================
+ Coverage   91.49%   95.63%   +4.13%
==========================================
  Files          69        6      -63
  Lines        2293      229    -2064
==========================================
- Hits         2098      219    -1879
+ Misses        195       10     -185

Impacted Files
Coverage Δ

flutter/lib/src/file_system_transport.dart
100.00% <ø> (ø)

dart/lib/src/sentry.dart

dart/lib/src/protocol/debug_image.dart

dart/lib/src/protocol/sentry_runtime.dart

dart/lib/src/protocol/max_request_body_size.dart

...art/lib/src/http_client/failed_request_client.dart

dart/lib/src/platform/_io_platform.dart

dart/lib/src/sentry_envelope_item_header.dart

dart/lib/src/protocol/debug_meta.dart

dart/lib/src/protocol/sdk_version.dart

... and 48 more

Continue to review full report at Codecov.

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


Add a loading indicator

We need to have a loading indicator when user clicks on Sign in button -

@brockwhittaker is this quick?

Hi @akashnimare I would like to work on this.
Can you assign this issue to me?

@kritisingh1 comment @zulipbot claim to claim it.

@akashnimare claim

No, you have to comment @zulipbot claim only.

@zulipbot claim

@kritisingh1 Can you accept the GitHub organization invite, so I can directly assign you this task?

@lonerz I have accepted the GitHub organization invite. Thanks a lot.

Let me know if you need any help @kritisingh1, would be happy to help. :)

Yeah I am still working on it and will soon make a PR. Apologies for delay
On Sun, Sep 24, 2017 at 1:24 PM, zulipbot<EMAIL_ADDRESS>wrote:

Hello @kritisingh1 https://github.com/kritisingh1, you claimed this
issue to work on it, but this issue and any referenced pull requests
haven't been updated for 7 days. Are you still working on this issue?
If so, please update this issue by leaving a comment on this issue to let
me know that you're still working on it. Otherwise, I'll automatically
remove you from this issue in 3 days.
If you've decided to work on something else, simply comment @zulipbot
abandon so that someone else can claim it and continue from where you
left off.
Thank you for your valuable contributions to Zulip!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zulip/zulip/issues/6477#issuecomment-331693780, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AU_BvrU-fOxRD1fKtkl8KM69Be-k2j94ks5slgqxgaJpZM4PUe9l
.

@zulipbot claim

@zulipbot abandon

@zulipbot claim

I want help. I cant find specifically where I can addthis code segment for loading sign!

@minervapanda you should be able to find this via git grep, but I think it's templates/zerver/login.html for any HTML, and static/js/portico/signup.js for any JS.

@zulipbot claim

#7684  SignIn: Add a loading indicator

@timabbott  please review.


test(exports): rm status from top level exports

Summary
Attempting to isolate an error happening with version 0.0.7 released.
Failed to compile.

<EMAIL_ADDRESS>Attempted import error: 'Status' is not exported from '@textile/core-storage'.

You can see the failure here https://github.com/textileio/storage-js-basic-demo/actions/runs/1118230723
It appears to come from this commit:
https://github.com/textileio/storage-js/commit/0d5a877685109386f75d9fd76121b56b3a8e73d7

because Status is just a typed string, the client souldn't need to worry about the type export at all. just removing to try and clear that issue highlighted above. going to 🤠 this on passed tests since hackers are waiting for the lib and it's blocking usage.


register_with_elb.sh on asg with spot instances fails to transition from standby to in service

It appears to get stuck on a waiting for lifecycle action step.
It is almost as through transitioning the spot instance to standby causes a new ec2_instance_launching lifecycle transition and then it gets stuck waiting for another deployment hook
I have one instance where this works and another where it doesn't and ultimately the lifecycle hook times out the instance is terminated and a new instance is spun up.
Found more about this.  Looks like a spot instance transitions through EC2_INSTANCE_LAUNCHING when moving from Standby to InService which causes all deployments targeting it to run again as though it was just created.  Including the currently running deployment!  It never finishes as you might imagine due to this recursion...
Dropping a spot instance to standby in the console and then putting it back in service works fine but unclear what console is doing...
Very weird...

Hi,
The issue was that lifecycle hooks were applied to spot instances when they were attached to an Auto Scaling Group or when they were exiting Standby. That was incorrect behavior as lifecycle hooks should only be applied to newly launched instances. The fix for this issue has been deployed to all public regions as of Nov 3.


precompilation

for precompilation a compile function like this would need to receive lib and helpers in the inner function
(function(lib,helpers) {
    return function(data){
        var o=lib.elementOpen,c=lib.elementClose,t=lib.text,v=lib.elementVoid;
        o('div', null, null, 'class', 'box');
            t( data.content );
        c('div');
        helpers['my-console']({data:7+8});}
})

something like
(function() {
    return function(data, lib, helpers){
        // throw error or something if not lib and helpers
        var o=lib.elementOpen,c=lib.elementClose,t=lib.text,v=lib.elementVoid;
        o('div', null, null, 'class', 'box');
            t( data.content );
        c('div');
        helpers['my-console']({data:7+8});}
})

var template = require('app.tpl');
idom.patch(document.getElementById('app'), function () {
    template(data, idom, helpers);
});

and maybe helper functions should be protected so we can render without all helpers functions registered.
i'm just sharing some ideias here, what think about this ? i'm thinking about writing a transform for browserify

Thank you for your attention.
Unfortunately, we cannot add parameters (lib, helpers) directly into the rendering function because it is transmitted as a call parameter to the function path, and for compatibility with the future versions of (incremental-dom)https://github.com/google/incremental-dom] we cannot change the semantics of calling a rendering function. This is why lib and helpers are transmitted via closure, and not as call parameters.
But we can suggest the following option, if the second parameter is absent during the compilation of the template, which means you won't transmit the link to the library:
var renderFn = itemplate.compile(templateStr[, IncrementalDOM]);

In this case it's not a rendering function that will be compiled, it will be just a function without closure and wrapping:
function (data, lib, helpers){
        // throw error or something if not lib and helpers
        var o=lib.elementOpen,c=lib.elementClose,t=lib.text,v=lib.elementVoid;
        o('div', null, null, 'class', 'box');
            t( data.content );
        c('div');
        helpers['my-console']({data:7+8});
}

Here it's up to you to decide how to call it and transmit compilation parameters; for example, as you suggested:
idom.patch(document.getElementById('app'), function () {
    template(data, idom, helpers);
});

Thank you for the suggestion. It allows to work with templates with more flexibility; for example, call several templates in one rendering function, or introduce additional logic, such as filtration.

done $-)


Fix Generators build

If XamlX has been built locally, the Avalonia.Generators project won't compile due to duplicate generated assembly information.
Fixed by not including the XamlX\obj folder (like in Avalonia.Build.Tasks).

You can test this PR using the following package version. 11.0.999-cibuild0031504-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]

You can test this PR using the following package version. 11.0.999-cibuild0031582-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]

You can test this PR using the following package version. 11.0.999-cibuild0031595-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]


Supply "IsConnected" state info to components

This is a partial implementation of what we need for #8786. It's pretty basic and just follows from existing patterns.
Still to do: E2E tests showing the swap from "prerendering/disconnected" into "connected", with a component that renders different output in the two cases.
Question: Do we need some other flag to differentiate the "prerendering" case from the "was interactive but then disconnected" case? I'm not sure we have a use case for that.

Temporarily rebased onto https://github.com/aspnet/AspNetCore/pull/8910 so that the E2E test can work.
@javiercn, I know you already approved this, but now I've added the E2E test it might be worth taking another look. Maybe the format of test will be useful in your prerendering PR too (which I will read this afternoon).

I already included something similar in my draft PR :(


Multiple keybindings for color picker

Description of the new feature / enhancement
It'd be very handy if I could have one keybinding for the color picker (Win + C) and a different one for picking a color and opening the color editor (Win + Shift + C). Ideally I wish I could change which format the color picker shows/copies before picking a color using the scroll wheel or something, but the first option is more likely.
Scenario when this would be used?
Most of the time I just need the HEX values of a color, but sometimes I need the RGB values, and I end up having to change my settings to be able to check them.
Supporting information
No response

Thanks for your message. This is a Duplicate of an other issue. Additional comments may be added there. /dup #12778


Add script to check manually installed plugin

Examples:
Preinstalled plugins: (bundled by default)

fluent-plugin-calyptia-monitoring

fluent-plugin-concat ...

fluent-plugin-utmpx

fluent-plugin-webhdfs

Currently Installed Plugins:

fluent-plugin-calyptia-monitoring
fluent-plugin-elasticsearch ...
fluent-plugin-utmpx
fluent-plugin-webhdfs

Manually Installed Plugins:

fluent-plugin-concat

Wow! Thanks!

found that it needs to be fixed on deb platform.

found that it needs to be fixed on deb platform.
/opt/fluent/lib/ruby/3.2.0/bundler.rb:310:in `rescue in root':  (Bundler::Dsl::DSLError)
[!] There was an error parsing `Gemfile`: Could not locate Gemfile or .bundle/ directory. Bundler cannot continue.

 #  from /opt/fluent/share/Gemfile:5
 #  -------------------------------------------
 #
 >  source "https://rubygems.org"
 #

🤔

It seems that we need BUNDLE_GEMFILE environmental variable so that Bundler can find Gemfile.
ENV["BUNDLE_GEMFILE"] = ...

It seems that we need BUNDLE_GEMFILE environmental variable so that Bundler can find Gemfile.

Thanks!

Once script was deployed for a package, when fluent-diagtool was implemented such a feature, it is hard to remove it in point view of compatibility, so keep it aside.


Resource usage logger broken in Python 2

The resource usage logger does not work in Python 2, execution fails with the following stack trace:
Exception in thread Thread-1:
Traceback (most recent call last):
  File "/home/bandela/conda/envs/esmvaltool2/lib/python2.7/threading.py", line 801, in __bootstrap_inner
    self.run()
  File "/home/bandela/conda/envs/esmvaltool2/lib/python2.7/threading.py", line 754, in run
    self.__target(*self.__args, **self.__kwargs)
  File "/home/bandela/src/esmvaltool/esmvaltool/_task.py", line 100, in _log_resource_usage
    for msg in _get_resource_usage(process, start_time, children):
  File "/home/bandela/src/esmvaltool/esmvaltool/_task.py", line 85, in _get_resource_usage
    entries = [round(entry, p) for entry, p in zip(entries, precision)]
TypeError: 'NoneType' object cannot be interpreted as an index

@bouweandela I had to fix this some time ago when testing the performance comparison between Py2-3 and iris1-2, I remember I removed precision since that is something Py3-specific, I think

@valeriupredoi I am also experiencing such kind of problem. Can you please suggest a common way to fix it?
Thank you very much!

I'll look into this asap. Cheers, it completely fell off my radar
On Wed, 10 Oct 2018, 11:35 ValerioLembo<EMAIL_ADDRESS>wrote:

@valeriupredoi https://github.com/valeriupredoi I am also experiencing
such kind of problem. Can you please suggest a common way to fix it?
Thank you very much!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ESMValGroup/ESMValTool/issues/433#issuecomment-428505361,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AbpCowJY6WCln5YnOD5Nfe28peWOd6A9ks5ujb9JgaJpZM4UtmRE
.

Note that Python 2 will only be maintained for another year, so I recommend doing any new development work in Python 3.

That is true! But I would take care of porting the script to Py3 after having restructured it for version 2.0 of ESMValTool.

Note by @evertrol: On Mac OS the resource usage logger does not work either, because Process.io_counters is not available.

This will be no longer relevant if we drop Python 2 support #267


5.2.0 Preview 2 ETA/Plan?

Just went to try 5.2.0 Preview 1 and ran into a few issues, then i noticed that the Preview 1 package is a month old.(https://www.nuget.org/packages/Microsoft.IdentityModel.Protocols.WsFederation/5.2.0-preview1-408290725)
Well that explains things, as the things i wanted to use were completed in the last month.
#703
#763
Is there a plan for a preview 2?

@brentschmaltz I'd like to at least get a working build on myget to validate some of the fixes. That could wait until you've finished your feature work if needs be.

@Zoxive @Tratcher POR is to drop preview2 next week, we are finishing up some XML validation.


Add VALUE_IN UDF to support multi-valued column group-by without extra values

Refactor all layers above Projection
Transform layer treat all columns as expressions
Re-design the TransformFunction interface to support dictionary in UDF
Add UDF support to both aggrgation columns and group-by columns
VALUE_IN UDF can act on multi-valued group-by columns and returns only the values inside the UDF

TODO: refactor selection to work on top of transform so that we can add UDF to selection in the future

Codecov Report

Merging #2652 into master will decrease coverage by 0.56%.
The diff coverage is 59.89%.

@@            Coverage Diff            @@
##           master   #2652      +/-   ##
=========================================
- Coverage   69.47%   68.9%   -0.57%
=========================================
  Files         860     861       +1
  Lines       39927   40385     +458
  Branches     5372    5467      +95
=========================================
+ Hits        27739   27829      +90
- Misses      10370   10731     +361
- Partials     1818    1825       +7

Impacted Files
Coverage Δ

...ore/operator/docvalsets/ProjectionBlockValSet.java
84.21% <ø> (ø)
:arrow_up:

...om/linkedin/pinot/core/common/BaseBlockValSet.java
4.54% <ø> (+0.19%)
:arrow_up:

...common/datetime/converter/EpochToSDFConverter.java
100% <ø> (ø)

...e/query/aggregation/groupby/GroupKeyGenerator.java
100% <ø> (ø)
:arrow_up:

...t/common/datetime/converter/DateTimeConverter.java
74.54% <ø> (ø)

...t/common/datetime/converter/SDFToSDFConverter.java
0% <ø> (ø)

...in/pinot/core/plan/ColumnarDataSourcePlanNode.java
54.54% <ø> (ø)
:arrow_up:

...mmon/datetime/converter/EpochToEpochConverter.java
100% <ø> (ø)

...common/datetime/converter/SDFToEpochConverter.java
100% <ø> (ø)

.../core/operator/docvalsets/ConstantBlockValSet.java
0% <0%> (-22.23%)
:arrow_down:

... and 81 more

Continue to review full report at Codecov.

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


Github Release not found at Alpine based Container

Used Release
Container: alpine:latest
asdf plugin: v1.0.2
chezmoi version: 2.40.0
Panic Output
...
* Downloading chezmoi release 2.40.0...
  curl: (22) The requested URL returned error: 404
  asdf-chezmoi: Could not download https://github.com/twpayne/chezmoi/releases/download/v2.40.0/chezmoi_2.40.0_linux_amd64-musl.tar.gz
  FAILED: install exited with an error

full log at https://github.com/nolte/asdf-chezmoi/actions/runs/6302247241/job/17109025400
Expected Behavior
The asdf plugin will be load the Release archive from some existing endpoint, for Alpine with some asset name  like chezmoi_2.40.0_linux-musl_amd64.tar.gz.
Actual Behavior
The Plugin looks for some not existing file chezmoi_2.40.0_linux_amd64-musl.tar.gz

Actual
chezmoi_2.40.0_linux_amd64-musl.tar.gz

Expected
chezmoi_2.40.0_linux-musl_amd64.tar.gz

Steps to Reproduce
Use the plugin inside a alpine based container build, or as GitHub Action
References

https://github.com/joke/asdf-chezmoi/issues/34

@nolte thanks for reporting the issue. the problem should be fixed now.


Matching Tag for custom extension

Issue Type: Feature Request
i have create my own extension (.vbb) which using both html and own customize element. When click the open tag to find the end tag of the element, all the word identical to the open tag is highlighted. Is this a bug or ?
VS Code version: Code 1.38.1 (b37e54c98e1a74ba89e03073e5a3761284e3ffb0, 2019-09-11T13:35:15.005Z)
OS version: Windows_NT x64 10.0.17763

This is a feature. You get the same behavior in text and any other files.

What is the solution in order to achieve my expectation ? Is it nothing can do from my side? Sorry I'm new with visual studio code extension.

You can add your own text decorations: https://github.com/microsoft/vscode-extension-samples/tree/master/decorator-sample
Fancy example: https://marketplace.visualstudio.com/items?itemName=vincaslt.highlight-matching-tag


Also enable use of Tatoeba-Challenge NMT models?

Could we even use the Tatoeba Challenge NMT models from https://github.com/Helsinki-NLP/Tatoeba-Challenge/tree/master/models? Does it work to use iso639-3 language codes? Can we even use models that cover language groups? (see https://github.com/Helsinki-NLP/Tatoeba-Challenge/blob/master/results/tatoeba-results-langgroup.md)

I tried https://huggingface.co/Helsinki-NLP/opus-mt-en-zh and it works. But I have to change the language code to iso-639 (e.g. eng-> en) for Trados to use.

Currently only bilingual models are properly supported (even the multilingual OPUS models probably won't work correctly). There's already some adhoc conversion happening from ISO-639-2 (three-letter) to ISO-639-1 (two-letter), since Trados uses 639-2 and memoQ 639-1, but should make a proper facility for conversion between the different code schemes.
Related to the Tatoeba-Challenge models, I should add a possibility of defining multiple model repositories in the UI (right now it's just one repository, and the settings can only be changed in the config files) so that Tatoeba-Challenge repository can be used. That would also be useful for local model repositories for companies that customize models etc.

The new release of OPUS-CAT now support Tatoeba-Challenge models (including multilingual models), so I'm closing this.


saver.restore(sess, args.model) in demo

args.model is ' ',When I run the demo,I meet many problems,such as make lib,I look others issues solve those proplems.
but ,when I run "saver.restore(sess, args.model)" this words,the program error and stop,I found there is no model ,may be it has existed just I do not know where ,may be it is not exist and need me to get the model by some way.
I do not know what to do , Is there someone can tell me what should I do , now I am studying tf . thank you very muck .

I download list ,it work.


Default parameters are not returned by Repo.insert/2

Environment

key
value

Elixir version (elixir -v)
1.4.0

Database and version
PostgreSQL 9.6.1

Ecto version (mix deps)
2.1.3

Database adapter and version (mix deps)
postgrex 0.13.0

Operating system
Archlinux

Current behavior
I have model created like this (with a default value for city_id):
defmodule Sample.Repo.Migrations.CreateUser do
  use Ecto.Migration

  def change do
    create table(:users) do
      add :first_name, :string
      add :last_name, :string
      add :city_id, :integer, default: 5
      timestamps()
    end
  end
end

The Repo.insert/2 does not return the default value 5 for city_id:
iex> Sample.Repo.insert(%Sample.User{first_name: "Test"})
{:ok,
 %Sample.User{__meta__: #Ecto.Schema.Metadata<:loaded, "users">, city_id: nil,
  first_name: "Test", last_name: nil, id: 1, inserted_at: ~N[2017-01-26 20:43:32.748233],
  updated_at: ~N[2017-01-26 20:43:32.752669]}}

This only happens with the index/2 and getting the same record after being inserted, does have the default value:
iex> Sample.Repo.get(Sample.User, 1)
 %Sample.User{__meta__: #Ecto.Schema.Metadata<:loaded, "users">, city_id: 5,
  first_name: "Test", last_name: nil, id: 1, inserted_at: ~N[2017-01-26 20:43:32.748233],
  updated_at: ~N[2017-01-26 20:43:32.752669]}}

Expected behavior
insert/2 returned result should have the default values already assigned.

By default Elixir will not query the database to return field values after insert. You can turn this behaviour on by passing the read_after_writes option to the field macro or using the schema-level default.


Loading the model or training in any way gives an environment variable error

🐛 Bug
When I'm trying to load the model in some way, I keep getting an error that the USER environment variable is missing, I have attempted to fix this by overriding the variable manually, however I still get an error raise RuntimeError("Where are you running this?! Check DATA_LOCATIONS.") RuntimeError: Where are you running this?! Check DATA_LOCATIONS.
To Reproduce
Steps to reproduce the behavior (always include the command you ran):

Run opt-baselines   -n 2 -g 8   -p test_v0   --model-size 1.3b   --azure   --checkpoints-dir "model_ckpts"   --no-save-dir  # Remove this if you want to print out full save-dir path
See error Traceback (most recent call last): File "/home/studio-lab-user/.conda/envs/studiolab/bin/opt-baselines", line 33, in <module>  sys.exit(load_entry_point('metaseq', 'console_scripts', 'opt-baselines')()) File "/home/studio-lab-user/metaseq/metaseq/launcher/opt_baselines.py", line 313, in cli_main  sweep_main( File "/home/studio-lab-user/metaseq/metaseq/launcher/sweep.py", line 374, in main  args = _get_args(add_extra_options_func, scheduler_args) File "/home/studio-lab-user/metaseq/metaseq/launcher/sweep.py", line 238, in _get_args  _modify_arg_defaults_based_on_env(env, args) File "/home/studio-lab-user/metaseq/metaseq/launcher/sweep.py", line 263, in _modify_arg_defaults_based_on_env  os.environ["USER"], File "/home/studio-lab-user/.conda/envs/studiolab/lib/python3.9/os.py", line 679, in __getitem__  raise KeyError(key) from None KeyError: 'USER'

Environment

python and torch: as specified within the setup readme
OS (e.g., Linux, Windows, MacOS): linux (AWS instance)
How you installed metaseq (pip, source): source

Guess we need to change that to os.getuser()?

I tried this but apparently it gives a function doesn't exist error, I'm assuming it's a Linux specific issue maybe?

https://stackoverflow.com/a/842096 my memory was slightly off


Ren'py games don't run in sandbox

Test game: https://emptyfortress.itch.io/gat-life-boyfriend-bar
Logs: https://github.com/itchio/itch-compatibility-watchlist/issues/170#issuecomment-248633403
I wasn't able to get logs myself - log.txt doesn't contain anything interesting, and I think isolate still swallows stdout/stderr atm?

I think I know what the issue is - the generated .exe immediately returns instead of waiting on the child. We should find a way to wait on all children spawned under isolate.exe - maybe Job objects will do the trick!

This has been fixed in the job branch - gotta clean it up a bit and fix stdout/stderr redirection for print-heavy games, but that branch already runs Gat Life: Boyfriend Bar top-notch!

(And as added benefits, it lets you run multiple games in the sandbox instead of just the one - weeee squashing all the bug reports about CreateEnvironmentBlock: Error 5, Access Denied)

The job branch was merged a while ago!


optimize walking of L-path vertices

Fixes #176.
Uses a modified shortest path algorithm to avoid re-walking edges that have already been walked. It does add computation time, so the subtype configuration enabled this optimization only if it helps the shape (reduces vertices) and doesn't bog down the browser.

I am pretty sure 176 was referring to the Space Filler Wiper pattern, which is in space_filler/SpaceFiller.js. That doesn't appear to be affected by this change.

I am pretty sure 176 was referring to the Space Filler Wiper pattern, which is in space_filler/SpaceFiller.js. That doesn't appear to be affected by this change.

Click the link on 176 above. It's the penrose optimization issue.

Penrose tile space filler

If you Create New Layer -> Space Filler -> Type: There is a penrose tile there. I think that is what they are referring to. It's in space_filler.js

If you Create New Layer -> Space Filler -> Type: There is a penrose tile there. I think that is what they are referring to. It's in space_filler.js
Wow, I both missed this and misunderstood your comment. I've added in both places. I had to reduce the maxIterations for Penrose by 1 because the optimization takes too long to run, but it does reduce the vertex count considerably.


massa functions k8s operator

With the arrival of the first version of Massa close, we need to start developing Operator. It would be interesting to have both proxy and operator functional in the same release.
Previously I imagined using the pre-existing Operator, but maybe it is better to start something that has a fine adjustment with our proxy.
In terms of technologies I see two main options, the first using Go and the second, less traditional but very simple, using Elixir with the Bonny library.
I've been looking at the examples and it seems to me to be very easy to implement an Operator with Bonny, I didn't think there was a need for the experience in Elixir not even with the development of operators to use this library.
I suggest that you take a look at this to help the decision.
https://github.com/coryodaniel/bonny

I've had a look at bonny. From what I see, also with their issues list:
https://github.com/coryodaniel/bonny/issues
the project seems to be pretty much dormant. With Issues pretty old and no one working on them.
But, if bonny is suitable and if it's very easy to have an operator working early, why not going this route.
How would an operator with bonny be different in than the one we have with cloudstate?

If Bonny isn't the right tool, that's fine. I don't see any problems with going with Go. The question I asked here was more about making an Operator to work with Massa, I mentioned Bonny because he thought it was an interesting option for the practice of Elixir. But that's okay, if not.
I think the difference to what exists today is in the kubernette artifacts that will be generated at the end and some change in the CRD that does not make sense in our use case, for example Akka configurations.
Of course, we have to think in terms of compatibility with the existing Operator, but I don't think that's what should guide our work.

Of course, we have to think in terms of compatibility with the existing Operator, but I don't think that's what should guide our work.

I inclined to diverge the operator. The Cloudstate Operators is not our one. We might change what is needed to change. WDYT? I think we would completely fork/copy the existing CS operator.

My only fear of copying is to bring all the complexities of the Cloudstate Operator. That was one of the reasons why I suggested Bonny because I saw controllers that use only a single module and can be quite functional.
I found the cloudstate operator, said by those who don't know anything about Go

I see. Lets start with bonny if that works. I saw the same done by CS as they did not know Go and used Scala. But also they switched back to Go because it was too much of a hassle.
Alas, I think we have more Go knowledge than the CS guys had.

Let's think before we decide. I just suggest that we do a POC on both technologies and evaluate the two approaches in a practical way and then go with what we think is best.
What do you think?
See an example for Bonny:
https://github.com/coryodaniel/hello_operator

I open this https://github.com/coryodaniel/bonny/issues/102

This might be a more interesting operator to look at than the hello world.
Current bonny version is fairly stable. There are some issues if the masters undergo an upgrade. The rest of the open tickets are features to add after k8s.ex 1.0 is released.

This might be a more interesting operator to look at than the hello world.
Current bonny version is fairly stable. There are some issues if the masters undergo an upgrade. The rest of the open tickets are features to add after k8s.ex 1.0 is released.

Great job with Bonny @coryodaniel  and Thank you for the feedbacks

Oh, and v1 CRDs are supported, it is in the master branch (pr #101), but hasn’t been released yet.

Oh, and v1 CRDs are supported, it is in the master branch (pr #101), but hasn’t been released yet.

Great!
@coryodaniel I'm trying to create an operator using Bonny but I'm not getting much progress, I can deploy the operator in a kubernetes cluster using kind, the deployment is created and I can see the event logs. However, the resources that my Operator tries to create do not appear anywhere. The only resource I was able to create was a configmap, but StatefulSet and Service are not created and I don't see anything in the logs.
Source code:
https://github.com/sleipnir/permastate-operator
Running CRD:
kubectl apply -f example.yaml
I created the issue https://github.com/coryodaniel/bonny/issues/104

Just jumped over here from your discussions on bonny. We have written operators with bonny and they operate very well. Also, if you need to implement admission webhooks, have a look at bonny_plug.

Just jumped over here from your discussions on bonny. We have written operators with bonny and they operate very well. Also, if you need to implement admission webhooks, have a look at bonny_plug.

Thank you @mruoss I will take a look in the plug plugin

Create another repository to handle this
https://github.com/eigr/permastate-operator

I'm closing this issue once our Operator is functional.
New controller-related issues can be opened directly in their own repository.


NGSessionPool poolSize always zero?

I think this is a mistake.
NGSessionPool(NGServer server, int poolsize) {
    this.server = server;
    this.poolSize = Math.min(0, poolsize);

    pool = new NGSession[poolSize];
    poolEntries = 0;
}

If poolsize is negative, an exception is thrown. If poolsize is positive, poolSize is zero.

Originally I thought this project created a pool of JVMs, because it would be so difficult to make it work properly with a single JVM.  But that's apparently what it's trying to do, with some tricks and an accidentally unbounded thread pool.
Instead of making a pull request that opens up those previously unexecuted codepaths, here's one #59 that updates the code to make explicit what it has been doing in practice anyway, since at least 2012.


Jeffrey zoo exploit.

People can avoid the Jeffrey Zoo by leaving and rejoining. Role persistance automatically adds the Jeffrey role however mee6 gives them the member role whenever they join. The member role has higher priority than the jeffrey role.
There are multiple solutions to this.

Potential solution:
Have Jeffreyfier remove the role after a new member joins who has the jeffery role.
Sample code: Detecting-New-Members

Commenting to hopefully get this solution moving :)

People can no longer escape the zoo by leaving and rejoining 😄


Allow 'align' functions to work with IDs

I don't have GraphicalObjects, and it's difficult to get them; it'd be much easier to just give your library IDs and have you align the corresponding objects.

Align and distribute functions support both element ids and their associated graphical object ids.


add DependencyControl support

Adds a function to BM, DM, PT and requireffi modules that creates a DependencyControl version record. Since these modules are DepCtrl dependencies, this function/method can and will be called only after DependencyControl has finished loading.
This change is required to make DependencyControl capable of self-update, which requires required modules to also be updatable when new version are required.

related DependencyControl commit: https://github.com/TypesettingTools/DependencyControl/commit/819b9c201eda5fa8127256065e8b5d8ce3e6d89c

Merged.


Remove TSPI references to avoid TPM1.2 interactions

Summary
Remove TSPI references to avoid TPM1.2 interactions.
Tests
vgopal@:$go test ./...
ok github.com/google/go-attestation/attest (cached)
? github.com/google/go-attestation/attest/attest-tool [no test files]
? github.com/google/go-attestation/attest/attest-tool/internal [no test files]
ok github.com/google/go-attestation/attest/attest-tool/internal/eventlog (cached)
ok github.com/google/go-attestation/attest/internal (cached)
ok github.com/google/go-attestation/attributecert (cached)
? github.com/google/go-attestation/ci [no test files]
? github.com/google/go-attestation/oid [no test files]
vgopal@: go-attestation:$

Also tested the following operations using the attest-tool (not pasting output here intentionally),

Listing PCRs, making AK and self-attesting, quote, measurement-log.

LGTM.


JenkinsQueueJob: jenkins job job creation failed with status code 400 bad request with exception message job xxx not found

Required Information
Question, Bug, or Feature?
Type: Question
Enter Task Name: JenkinsQueueJobV2
Environment
Azure Devops Server 2020

Agent - Hosted or Private: Private

Issue Description
Jenkins Queue Job job creation failed with status code 400 bad request with exception message job xxx not found
Task logs
2021-11-23T02:10:17.7730786Z Job creation failed.
2021-11-23T02:10:17.7744382Z HttpResponse.statusCode=400
2021-11-23T02:10:17.7746986Z HttpResponse.statusMessage=Bad Request
2021-11-23T02:10:17.7749369Z java.lang.IllegalArgumentException: Job: xxx not found
2021-11-23T02:10:17.7751354Z              at hudson.plugins.tfs.TeamBuildEndpoint.getJob(TeamBuildEndpoint.java:269)
2021-11-23T02:10:17.7754205Z              at hudson.plugins.tfs.TeamBuildEndpoint.innerDispatch(TeamBuildEndpoint.java:292)
2021-11-23T02:10:17.7757025Z              at hudson.plugins.tfs.TeamBuildEndpoint.dispatch(TeamBuildEndpoint.java:167)
2021-11-23T02:10:17.7758481Z              at hudson.plugins.tfs.TeamBuildEndpoint.doBuild(TeamBuildEndpoint.java:335)
Troubleshooting
We can see job xxx in Jenkins configure dashboard, but not sure why the log says Job not found.
Find the source of method hudson.plugins.tfs.TeamBuildEndpoint.getJob on the top of the exception call stack as below. Please help to check about this issue.
https://github.com/jenkinsci/tfs-plugin/blob/master/tfs/src/main/java/hudson/plugins/tfs/TeamBuildEndpoint.java

BTW, some user has reported similar issue in issue https://github.com/microsoft/azure-pipelines-tasks/issues/3135, please refer the last question in the link.

@anatolybolshakov Is there any update?

Hi @MichaelMWW I can't reproduce the issue - does it happen sporadically or constantly for you? Do you have some stable repro steps for this?

@anatolybolshakov Prerequisites and background as below.
Jenkins version: 2.298
Jenkins pipeline type: Multi-branch pipeline which uses Azure Repo git source
Jenkins TFS Plugin: Team Foundation Server Plug-in 5.157.1
Azure DevOps on-premises Server version: 2020 Update 1
Azure Repo default branch -> Branch Policies -> Build Validation to validate code by pre-merging and build pull requests changes.
Build Validation pipeline uses JenkinsQueueJob to trigger Jenkins multi-branch pipeline when a PullRequest to the default branch created.
The yaml pipeline is like that:

Configured Build validation for main branch:

Hi @MichaelMWW thanks! I managed to reproduce it for Multi-branch pipeline - investigating this issue currently, will update you shortly once there are any results.

@MichaelMWW could you please try to open  /team-build/build/ - does it return any error?

@MichaelMWW Jenkins task uses < jenkins-server >/team-build/build endpoint to submit new job. This endpoint comes with https://github.com/jenkinsci/tfs-plugin - as I see there is an issue while reaching out to this endpoint for multibranch job at the moment, so it seems to be some issue on tfs-plugin side.

@anatolybolshakov Thanks for your update, so what's your suggested next actions, create an issue for Jenkins tfs plugin, I saw there is no place to create new issue for https://github.com/jenkinsci/tfs-plugin.

@MichaelMWW yes, I see it too - let us clarify if there is someone we could reach out about it.

Hi @MichaelMWW since this plugin is owned by Jenkins - as we've checked the better way to report this issue is to open a ticket on https://issues.jenkins.io/secure/Dashboard.jspa

@anatolybolshakov It seems that the Jenkins TFS plugin has no update for a long time. Regarding to below Jenkins queue job task official link, it mentioned argument isMultibranchJob and it supports this argument, but it failed on the scenario in this github issue, can we update this document to mentioned that to avoid confusion for our users?
Doc link:
https://docs.microsoft.com/en-us/azure/devops/pipelines/tasks/build/jenkins-queue-job?view=azure-devops#arguments

@MichaelMWW sure, let us take a look at the documentation update here also.


Feature/autosuggest section header

** Please provide as much detail as possible, but feel free to remove irrelevant sections **
{{ Reason for change }}
Added the section header for the autosuggest component

@ZhenYongSeekAsia How the section header looks like? ;)

Is that by design that Section labels are not following the standard padding?

Section header should share the same padding as regular list items.
Is it something equivalent to <optgroup> in HTML which shouldn't be selectable?

Section header should share the same padding as regular list items.
Is it something equivalent to <optgroup> in HTML which shouldn't be selectable?

actually the section header can be a component, for the current usage there will be a clear button inside the section header, so the selectable should be control in the application but not style guide


[TACHYON-1247] Fixed deadlock issue in BlockStoreContext and FileSystemContext

https://tachyon.atlassian.net/browse/TACHYON-1247

Also, a high level comment: should we also push this into branch-0.8?

@yupeng9 Thanks for the fix! Out of curiosity, how did you find the problem?
ResourcePool should be thread-safe, so can we drop the corresponding acquire*Client synchonize modifier?

@yupeng9 Could you reopen this to branch 0.8 and add a unit test? Also, is this a temporary solution? I'm not sure how this will interact with resetContext if it is not synchronized. Perhaps we need a different kind of locking scheme (ie. anything can execute except resetContext which should lock everything).

@aaudiber, I encountered a deadlock when I had several threads concurrently write into Tachyon. @calvinjia, unit tests must be deterministic, so I don't think concurrency can be unit tested in a deterministic way. The closest thing I can think of is to spawn lots of threads, but such tests shouldn't be crafted as unit test. Maybe @jsimsa can shed some light on the best practice on concurrency testing.

@aaudiber, I think synchronized keyword can be dropped on acquire*Client. @calvinjia, why do we need reset in *Context? By looking it use, it's mainly called in initialization and testing, if so, do we need it to be a public method?

Will reopen in 0.8


add build folder to git

This enables GitHub URI resolution

This PR just adds the build folder to git, nothing more. There are no changes to the code. I am admin-merging this to avoid wasting someone's time with a review.


Is 7.7.4 to 8.0.0 a move to the beta?

I'm using 7.7.4 and while doing my regular HACS updates I saw this add-on was wanting to update but did read "This change relies on go2rtc configuration. Please wait until a new version is released and do not get from master", so I'm assuming that 8.0.0 is still in development and I should not pull the trigger.
My question is more around the go2rtc and other changes - will it be a simple upgrade to 8.x when it's finalized or will it be better to just remove the current version entirely and start over?  That warning sound ominous!  The old version used RTSP Simple Server so I don't know if it's a big deal or not.
Thanks for this amazing extension, I have a LOT of things tied to it so I want to be deliberate in updating!

You want to use this version of WebRTC instead (you can add it as a custom repository in HACS, and uninstall the old one):  https://github.com/fuatakgun/WebRTC
This post and some comments later on are useful: https://community.home-assistant.io/t/eufy-security-integration/318353/2056

it is time to move to version 8 if you are interested in more stable streaming, if it is already good enough for you, stay with v7 but it will not be updated.


Namespaces dropdown empty in VM wizard

Seems like namespaces in the Create VM from wizard window are not properly fetched. The list is empty. Deploying a VM from yaml works like a charm!

this may be due to the permissions, we should use projects in that case

I did not see any 403 permission denied errors looking at the network stream. Also, I have loaded the admin credentials. Furthermore, I can create a VM through a yaml file and browse all the namespaces..

hmm, that is weird. Did the other fields get populated successfully ?

Yes they did, can you point out where in the code this data is being loaded?

here: https://github.com/kubevirt/web-ui/blob/master/frontend/public/kubevirt/components/modals/create-vm-modal.js#L20


Set current culture when matching localized route

Taken from the MVC team feedback:

Consider taking this a step further and setting the current culture when you match a localized route. Our localization system does this, and use it as well, so this will play with all of the culture-sensitive features we have. The optimal way for you to do this would be to attach an IAsyncResourceFilter to the action model 👍

Since culture is added to route data, this can be done with RouteDataRequestCultureProvider, so nothing to implement here. Something like
RequestLocalizationOptions requestLocalizationOptions = new RequestLocalizationOptions
{
	DefaultRequestCulture = new RequestCulture(defaultCulture),

	// Formatting numbers, dates, etc.
	SupportedCultures = supportedCultures,

	// UI strings that we have localized.
	SupportedUICultures = supportedCultures,
	RequestCultureProviders = new List<IRequestCultureProvider>()
	{
		new RouteDataRequestCultureProvider(),
                new AcceptLanguageHeaderRequestCultureProvider(),
	},
};


Clean repo

We should clean our repo. There are some "big" files in the history. Don't know to safely remove them yet.
0B	./.git/objects/info
7,3M	./.git/objects/pack
7,7M	./.git/objects
 12K	./.git/refs/heads
 28K	./.git/refs/remotes/origin
 28K	./.git/refs/remotes
  0B	./.git/refs/tags
 40K	./.git/refs
7,8M	./.git

We cant remove these files from the repository, but we can talk about remove the history, when we know we won't rollback. I recommend to do this only in the master-branch.

Agreed on the only on master thing.
I know that .git saves the history and we shouldn't mess with that. I think the nearly 8MB come from the images of the circutes we had in the repo.
But maybe this fixes itself after updating master. If not, we should investigate further. Pulling 8MB for this repo is not ok.

I thinm we can compress a lot in master.. so close here?

issues are repo wide, not branch wide right? So since its not fixed yet, maybe don't close it just yet.

./sign


[Slider] focus animation bug

http://www.material-ui.com/#/components/slider
When moving a slider its "breathing" animation has a bug: sometimes it works, sometimes it doesn't (grab, move, release, grab, move, release)

Looks like an issue with the focus handling. I can reproduce it on mobile.
What environnement are you using?

I'm on Chrome

We plan on migrating the component to the v1-beta branch. I'm closing the issue for #4793.


VWIP Topoplot refactor

Have plot_evoked_topo use plot_compare_evokeds.
Benefits: looks nicer, CIs, code reuse, API consistency.
preemptively tagging @sappelhoff @mmagnuski @cbrnr @choldgraf ...
closes #4746

Ok the current implementation is bad. Will hopefully solve it better with @drammock :)

Thanks for looking over it though @massich


cmd/go: go version -m shows that replace directive was ignored

What version of Go are you using (go version)?

$ go version
go version go1.14.4 linux/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/mark/.cache/chgo/cache/go1.14.4"
GOENV="/home/mark/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/mark/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/opt/goroots/go1.14.4"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/opt/goroots/go1.14.4/pkg/tool/linux_amd64"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/mark/src/testing1/go.mod"
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 -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build605705998=/tmp/go-build -gno-record-gcc-switches"

What did you do?
While attempting to follow the recommended way of tracking tool dependencies for a module, I discovered that the version of the tool installed didn't respect the replace directive in the go.mod file.
go.mod:
module testing

go 1.14

require (
	// …
	k8s.io/code-generator v0.18.0
	// …
)

replace (
	// …
	k8s.io/code-generator => k8s.io/code-generator v0.0.0-20190311093542-50b561225d70
	// …
)

tools.go:
// +build tools

package tools

import (
	_ "k8s.io/code-generator/cmd/deepcopy-gen"
)

With these files in place, I installed the tool via:
$ GOBIN=$PWD/bin go install k8s.io/code-generator/cmd/deepcopy-gen
go: finding module for package k8s.io/klog
go: finding module for package k8s.io/gengo/args
go: finding module for package k8s.io/gengo/examples/deepcopy-gen/generators
go: finding module for package github.com/spf13/pflag
go: downloading k8s.io/gengo v0.0.0-20200518160137-fb547a11e5e0
go: found github.com/spf13/pflag in github.com/spf13/pflag v1.0.5
go: found k8s.io/gengo/args in k8s.io/gengo v0.0.0-20200518160137-fb547a11e5e0
go: found k8s.io/klog in k8s.io/klog v1.0.0
go: downloading golang.org/x/tools v0.0.0-20200505023115-26f46d2f7ef8

Unfortunately, go version -m … indicates that the non-replaced version was installed rather than the replaced version.
What did you expect to see?
$ go version -m bin/deepcopy-gen
bin/deepcopy-gen: go1.14.4
	path	k8s.io/code-generator/cmd/deepcopy-gen
	mod	k8s.io/code-generator	v0.0.0-20190311093542-50b561225d70	h1:lgPp615xLHxN84RBd+viA/oHzJfI0miFYFH4T9wpPQ4=
	…

What did you see instead?
$ go version -m bin/deepcopy-gen
bin/deepcopy-gen: go1.14.4
	path	k8s.io/code-generator/cmd/deepcopy-gen
	mod	k8s.io/code-generator	v0.18.0	h1:0xIRWzym+qMgVpGmLESDeMfz/orwgxwxFFAo1xfGNtQ=
	…

To confirm that it was actually respecting the base require rather than something else, I tried changing the version to v0.17.0:
module testing

go 1.14

require (
	// …
	k8s.io/code-generator v0.17.0
	// …
)

replace (
	// …
	k8s.io/code-generator => k8s.io/code-generator v0.0.0-20190311093542-50b561225d70
	// …
)

Which produced:
$ go version -m bin/deepcopy-gen
bin/deepcopy-gen: go1.14.4
	path	k8s.io/code-generator/cmd/deepcopy-gen
	mod	k8s.io/code-generator	v0.17.0
	…

So it is following the require, but appears to be ignoring the replace.

Reproduced with Go 1.14.4 but not master. go list -json shows the replacement is applied. Might be an issue with go version -m.

Ah, this was fixed in CL 220898, which was a fix for a closely related issue, #37392. The text produced by modload.PackageBuildInfo is reported by both go version -m and runtime/debug.ReadBuildInfo, so these two issues have a common root cause. I'll close this one. Thanks for reporting though!


AMapView不能作为Widget添加

flutter 0.9.4，计划像示例里面一样添加AMapView作为组件
    return new Scaffold(
      appBar: new AppBar(title: new Text('Titles')),
      body: Expanded(child: AMapView(
      ))
    );

但是ide直接报错说 The arguemennt type 'AMapView' can't be assigned to the parameter type Widget'`.
请问这个问题如何解决？

你这个应该是属于flutter页面内嵌原生页面了，可能还没实现。目前这种应该是直接Push出一个原生页面的

你好,
I would also be interesting in an AMap Widget. Is this something you plan to do? I might support that, however the chinese documentation is a bit challenging...

你好,
I would also be interesting in an AMap Widget. Is this something you plan to do? I might support that, however the chinese documentation is a bit challenging...

hello.
你还打算继续研究 “flutter” 的地图插件吗？


Update meta.yaml

If there is no objection, I would like to reduce the restriction for xorg-libx11 as only xorg-libx11=1.7 has been build for osx-arm64. Doing so would allow performing a migration for that architecture.

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge/xorg-libxext, this is ready for review

@conda-forge-admin, please rerender


Blob in docker can't persist to docker volume

Which service(blob, file, queue, table) does this issue concern?
Blob
Which version of the Azurite was used?
3.32.0
Where do you get Azurite? (npm, DockerHub, NuGet, Visual Studio Code Extension)
DockerHub
What's the Node.js version?
v20.10.0
What problem was encountered?
    local_dev_azurite:
        platform: ${PLATFORM}
        image: mcr.microsoft.com/azure-storage/azurite
        hostname: azurite
        command: "azurite-blob --blobHost <IP_ADDRESS> --blobPort 10000"
        ports:
            - "10000:10000"
        volumes:
            - /d/az/data:/data

Steps to reproduce the issue?

start the docker;
upload file to blob with Java SDK;
the returned url is abled to download the file;
but not file in the /data

Have you found a mitigation/solution?
no

I tried /opt/azurite/__blobstorage__ as well. Not working.

@jhuang-simwell
Azurite has its own data structure to save the data uploaded to it.
You can only download from the Azure Storage API, but can't access the blob as a file directly. (Like you can't access blob as file from public Azure.)
Besides that, Azurite data will save in the folder specified by "-l", so if you add a parameter to the command "-l /data", you should can find the Azurite saved file in the /data folder, however it's in a specific structure.

@jhuang-simwell
Azurite has its own data structure to save the data uploaded to it. You can only download from the Azure Storage API, but can't access the blob as a file directly. (Like you can't access blob as file from public Azure.)
Besides that, Azurite data will save in the folder specified by "-l", so if you add a parameter to the command "-l /data", you should can find the Azurite saved file in the /data folder, however it's in a specific structure.

I see. I don't mean to inspect the raw data. I just want to make the data persist so that whenever I remove the container, I can still access the data I uploaded in the previous container when I start a new container. I will try the -l option.

@jhuang-simwell
I will close this issue as you should can find the files after add "-l" parameter to specify the file location.
Feel free to contact us if need any further assistance on Azurite.


After rioxarray 1.3 is released, remove OpenData from TopographicDataSource.__post_init__()

Right now, rioxarray can't open data directly from cloud storage buckets, so TopographicDataSource.__post_init__() downloads files to the VM's local hard disk using OpenData.  But rioxarray v1.3 will enable rioxarray to open directly from cloud storage buckets, so we can remove OpenData from TopographicDataSource.__post_init__()
See https://github.com/corteva/rioxarray/issues/246#issuecomment-968515945

Good spot, and nice to always reduce our codebase

Itll be ' rasterio 1.3' which is near out ...


Track method fixes

Current dependencies on/for this PR:

main

PR #42

PR #43

PR #44

PR #45

PR #46   👈

This comment was auto-generated by Graphite.


Carla 0.9.2 : Information of sensor parent is giving as None

Hi,
I have written a script similar to manual_control.py in which I have attached Lidar,Camera and Collision sensors to vehicle.
Now I want to listen the data collected from these sensors from another client attached to the same server.
I have created Client_2 where I have retrieved the "Hero" vehicle (spawned in manual_control.py) and I got sensors list through get_actors(). However when I try to find parent of those sensors, It is giving me parent as None.
Kindly help me to listen these sensors data from client_2.

Let's continue on #1225 as it seems the same problem.


Question about post-install.sh

I'm looking to build this container on my own without calling docker pull, and noticed the following lines at the end of the dockerfile:
# add-in post installation file for permissions
COPY post-install.sh $APP_DIR/
RUN chmod 500 $APP_DIR/post-install.sh

It looks like there is some expectation to run the file afterward, I was wondering what the proper way to call this script is following the docker build?  I'm currently running into permissions when I just try to have it call the script following the docker build by running ./post-install.sh

There's an example folder that imports IronAlpine as the baseline image, adds some more packages on top, and then runs post-install.sh. The reason (presumably) that the base-line image doesn't run this is that it's meant for downstream users of IronAlpine to execute it only after they've finished adding new packages to their IronAlpine-derived custom builds.

Hi @bcronrath! The main goal of post-install.sh is 1) setting strict permissions after you are done installing and 2) removing the package managers once the container is finished.
#!/usr/bin/env sh

# fail if a command fails
set -e
set -o pipefail

# remove apk package manager
find / -type f -iname '*apk*' -xdev -delete
find / -type d -iname '*apk*' -print0 -xdev | xargs -0 rm -r --

# set rx to all directories, except data directory/
find "$APP_DIR" -type d -exec chmod 500 {} +

# set r to all files
find "$APP_DIR" -type f -exec chmod 400 {} +
chmod -R u=rwx "$DATA_DIR/"

# chown all app files
chown $APP_USER:$APP_USER -R $APP_DIR $DATA_DIR

# finally remove this file
rm "$0"

Thank you both for the information!


dot.tk and friends?

.tk and other domains are available for free. Check out dot.tk

Thanks, I've added this.


Add cancellable double click handler

Hi,
This is in addition to the already existing cancellable simple click. I use this new handler on table rows.
Thank you.

Thanks!
However, we are currently working on allowing arbitrary user-defined event handlers in Vdom, based on a notion of "event decoder" (same as in Elm). We will also reimplement all existing handlers with this new system.
This is currently being developed and tested in our internal version at LexiFi, and we will push it upstream afterwards.
Your proposal could then be implemented in "user land" (but we can of course also add the function to the Vdom interface -- to be discussed). Anyway, I suggest to wait for a few weeks, to avoid a conflict with our ongoing development.
cc @aureliensaue who is working on this extension.

Ok, no problem, i will wait for this new version. Thanks!

Hi @slegrand45, the new version has now been released: https://github.com/LexiFi/ocaml-vdom/tree/0.3!

Excellent news! Thank you very much.


No function names on plot

I tried to run the example from the README (code below), and I get almost the same plot, except there are no function names written on it. Does this need to be enabled somehow or is this a bug? (nothing appears on hovering over the plot either)

using Profile
using ProfileSVG

function profile_test(n)
    for i = 1:n
        A = randn(100,100,20)
        m = maximum(A)
        Am = mapslices(sum, A; dims=2)
        B = A[:,:,5]
        Bsort = mapslices(sort, B; dims=1)
        b = rand(100)
        C = B.*b
    end
end

profile_test(1)   # run once to compile

Profile.clear()
@profview profile_test(10)

Versions

Julia: 1.4.0
ProfileSVG: 0.1.0
IJulia: 1.21.1

I think I've seen that too but I think it depends on how you're visualizing the SVG file? It's been a long time and I can't remember. You might want to inspect the produced SVG file to check whether the names are there somewhere.
I generally recommend ProfileVega.jl or ProfileView.jl over this package. They all use FlameGraphs.jl as the core, but the SVG output has not been maintained and needs someone to help (as indicated in the README).

I can't reproduce this issue (on Chrome and Firefox on Windows 10).

I think this is a dependency problem with the AMD (Asynchronous Module Definition). If Snap.svg is already loaded on Jupyter, ProfileSVG will use it. This problem happens if other packages or plugins used Snap.svg.

I think it's worth checking out PR #12. However, the URL in the General registry has not been updated yet, so you need to set the new URL manually.

If the problem has not been fixed, please reopen this.


⚠ Bofrost.es has degraded performance

In 02cd016, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1632 ms

Resolved: Bofrost.es performance has improved in 4ab946b after 6 minutes.


Document the new gitops feature flag

Add documentation for https://github.com/rancher/rancher/issues/32808

Closed via https://github.com/rancher/docs/pull/3429


Tab or Pane not closing

Environment
Windows Terminal Preview: Version: 1.6.10272.0 installed from Microsoft Store.
Any other software running in terminal.
Steps to reproduce
For example, you run pwsh.exe and cmd.exe in different tabs/panes and want to kill cmd.exe from pwsh.exe:
Get-Process cmd | Stop-Process # or
(Get-Process cmd).kill()  #or
taskkill /im cmd.exe /f

The panel or tab where the cmd.exe was launched does not close after it is killed. This applies to any application and to any method of closing it. Even from the Task Manager.
#Expected behavior
The tab or panel must close when the application is stopped.
Actual behavior
The tab or the pane does not close if you close an application running in it from another one.

Set closeOnExit to always. This is working as intended: cmd effectively crashed, so Terminal leaves the pane open so you can see what happened.

closeOnExit dictates when panes are automatically destroyed :smile:

Thanks! Strange, but I passed by this option.


Add constants for common defaults such as IID(Normal())

          The default version of this is so common, would it be worth having a constant like

const NormalIID = IID()

Which can be used in places like the generate_latent method for HierarchicalNormal?
Originally posted by @SamuelBrand1 in https://github.com/CDCgov/Rt-without-renewal/pull/438#discussion_r1879847119

Yes. I'm abit agnostic, but I personally like const s for really commonly used objects. ymmv though.


Switch "goog" audio constraints back to optional

Was struggling to get the audio constraints to apply with the mandatory system. Temporary have put them all at the top level of the object. This probably breaks some browsers, especially ones that don't start with Google or end with Chrome. Will need to figure out the error that occurs when using the mandatory system.

Browsers simply ignore constraints they do not understand, so this shouldn't be an issue. See the green note in "Media Capture and Streams, 11. Constrainable Pattern" here.


Folder structure

I usually have attached notes or extra material in each project and it would be saved in the folder of the same name. This way if I need to resume a project I have everything organized quickly.
Is it possible to make the plugin compatible with a folder structure so that when creating a new project, area or resource a folder is created and inside a note with the same name as the folder?
Also if a project is archived the entire folder should be moved.

Hi @javiavid,
Thanks for your input. I have already thought about this. Creating a folder for the project is more in line with the PARA methodology.
I, personally, like it to use a flat structure that's why the first officially release of the plugin looks like this.
ICYMI, the areas are already created with an companion folder. However, the area note itself is not located in this folder. The reason for this is that you have faster and more direct access to your area with fewer clicks. The files in the companion folder are automatically linked in the area note when using my builtin template.
For the resources, I don't plan to support a folder structure for now. A flat structure offers you an more unbiased view of your resources and possibly new incentives if you are looking for something.
As an alternative for the moment, Obsidian supports tags, searches and filters, which are powerful tools and are sufficient to categorize and find a specific resource or group of resources.
For the projects, I will definitely take a look to make the plugin compatible with a folder structure like you mentioned.
There are also some other things I want to do and fix. In the coming days I will update the README to keep you all informed about the further development of the plugin.
This plugin is yet in an early stage of development and any feedback and ideas are very welcome.
I hope I could give you some insights and a better understanding of the current development.
Let me know if you have any further questions.

Thank you for your comments.  I will be attentive to the development of the plugin.

Want to let you know that support for a folder structure for projects is on the way and will be available with the next plugin release 👍
#3

Thanks!

Thanks!


etebase-server: 0.8.3 -> 0.9.1

Description of changes
Update etebase-server to latest release:
https://github.com/etesync/server/blob/master/ChangeLog.md
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)
[x] Tested, as applicable:

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

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

I've added a withLdap option which is true by default. Is this good or should it be disabled by default?
This is the first time I'm adding such an option, so I'm not sure how this is usually handled.
Also, I haven't actually tested the new LDAP functionality.

I've also just noticed that some packages use enableLdap/enableLDAP instead of withLdap/withLDAP.
And the capitalisation is also mixed 😅.

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

I've added a withLdap option which is true by default. Is this good or should it be disabled by default?
This is the first time I'm adding such an option, so I'm not sure how this is usually handled.

Why do we need the option at all? Just compile it in all the time.

@SuperSandro2000, well, one could save on the pythonPackages.ldap dependency.
It's also an optional dependency in the AUR package:
https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=etebase-server#n29


Add rest_total_hits_as_int to allowed Uri Params

This allow to get in ES7 again to total as int instead of an object.

Codecov Report

Merging #324 into 7.2 will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                7.2     #324   +/-   ##
=========================================
  Coverage     78.53%   78.53%
  Complexity      713      713
=========================================
  Files           123      123
  Lines          1887     1887
=========================================
  Hits           1482     1482
  Misses          405      405

Impacted Files
Coverage Δ
Complexity Δ

src/Search.php
21.91% <ø> (ø)
56.00 <0.00> (ø)

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 3b4424a...1220a3b. Read the comment docs.


Remove version fields from Cargo.toml that are not for creates.io

Version fields for redbpf packages are removed from Cargo.toml. But I only modified packages that are not uploaded to creates.io

One of the other issues here is that ˆ does not work with -rc pre-release version numbers in cargo, So that would also make sense to change in one go.


invalidate() method timeout

Describe the problem
If invalidate() is called and the server generates an error, it appears the client is directed to an error page with the appropriate details. ✅
But if invalidate() is called and there is simply no response, then what? Right now it appears happy to wait forever (where "forever" is at least the 5 minutes I waited). This could lead to inconsistent state if not handled.
Should we be expected to add our own timeout mechanism or could a default timeout option be set or invalidate() allow a timeout parameter that could generate an error on the client?
Describe the proposed solution
Either a global timeout or a per-invalidate timeout, that will generate a timeout error on the client (or a combination of timeout and retry mechanism).
Alternatives considered
No response
Importance
would make my life easier
Additional Information
No response

This plus #5305, plus a separate conversation about allowing non-URL arguments to invalidate (useful for client-side state management etc) all point to the need for a bit of a rethink of the API, so I'm marking this as a breaking change

Allowing a timeout only on invalidate() seems like a weird API if we don't also allow it on regular client-side navigations or on programmatic goto() calls - which seems to indicate to me that the concept of load timeouts should live somewhere else, separate from any of these specific reasons it should be called, but I don't know where that should be.

I was wondering if such an API is really necessary. A user could do something like await Promise.race([new Promise((_, reject) => setTimeout() => reject('timeout'), 1000), invalidate('..')]) to get the behavior themselves. But then I realized that this would not abort the invalidation if there's no other navigation/invalidation in the meantime (if there was, SvelteKit would know that is outdated and ignore the result), so if it would resolve some time later that could be confusing.

Coming back to this again, my workaround with Promise.race does work if you do
await Promise.race([
  new Promise((_, reject) => setTimeout() => goto($page.url), 1000),
  invalidate('..')
]);

This could be packed up into a little helper function. I'm inclined to move this post 1.0 to wait for more people expressing desire to have something like this built in where such a helper function (which also provides way more control, because you can do more than timeout) wouldn't suffice.
Other ways this could be solved:

some kind of abortLoading() function which stops any invalidation/navigation that currently takes place
passing in an abort signal to to invalidate/goto which you call when appropriate

Whether we do it now or later, I think the framework should have support built-in for it. And I think it should happen automatically and not require any special APIs. It should probably be configurable both globally and for an individual call

I still don't see why we would want this on programmatic invalidate()/goto() but not have a way to specify it for intercepted link clicks (or for SSR'd content, for that matter). What makes the programmatic calls special that we want to be able to specify a timeout for them, but not the other times the framework is calling load() automatically?
What would happen if we allowed a export const timeout = 1000; in +page.js? Is there a reason to want different timeouts in the four different situations that load() might be getting called?
If we want to be code-over-configuration about this (at least at first) what about a wrapper function around the whole load() function that Promise.races it with a given timeout that can vary between SSR and CSR if the user wants? Higher-order functions that augment the load() function (say, to inject additional arguments) are already a pattern we're recommending. It'd be nice to eventually let people specify those on all of their load() functions at once in a somewhat more DRY way. If we turn the problem in this issue into an augmenting-load() problem, we could eventually solve both at once.


(FACT-3058) Fix os.windows.release_id fact and add os.windows.display_version fact

This commit fixes os.windows.release_id fact on Windows 2022 by retrieving the information from the correct registry key. The ReleaseId registry key from SOFTWARE\Microsoft\Windows NT\CurrentVersion seems to be abandoned since build 2009 (is still kept on newer builds and versions of Windows but not updated anymore) and replaced by the DisplayVersion registry key from the same location.
This commit also adds a new fact, os.windows.release_id, with above information. For backwards compatibility reasons, when DisplayVersion is available, both facts will be populated from this registry key. When DisplayVersion is not available, only os.windows.release_id will be populated but from ReleaseId registry key instead (same as before this fix).
TODO in Facter 5: remove os.windows.release_id fact to avoid duplicated information
See https://github.com/puppetlabs/facter/pull/2439 for same implementation in Facter 4

jenkins please test this on windows2019-64a

jenkins please test this on windows2019-64a

jenkins please test this on windows2019-64a

Output on Windows 2022 with these changes (winver and facter):

Output on Windows 2022 without these changes (winver and facter):

Build for further testing: a44b5c630d52d530bd18945d617df491ac25fe9d
TODO: We should also manually test these on non-server Windows and older servers than 2022 to make sure of the behaviour.


Latex Array is not rendering on website, but works locally.

Issue Summary
I'm trying to write Latex array using MathJax, it works fine locally when I render it with bundle exec jekyll liveserve but when I push it to my GitHub pages repo it doesn't render. Simple equations work on the website, but I have a problem with arrays.
Steps to Reproduce:
$$
  \left(\begin{array}{lll}
  {a} & {b} & {c} \\
  {0} & {d} & {e} \\
  {0} & {0} & {f}
  \end{array}\right)
$$

Technical details:

MathJax Version: >3
Client OS: Win 10 64
Browser: Chrome

Supporting information:
My config files:
\includes\_lib\mathjax.html

used before the script that loads MathJax's latest.js file would do it.  But I suspect that the configuration script and the loading of latest.js is not something you have access to, so that might not work.
Note, that you are not using MathJax v3 but rather MathJax v2.7.4 (see the press release for v2.7.6 for why you are not getting the latest version of mathJax any longer).  You actually are loading both v2 and v3 (not very efficient), but  because the page has v2-style

My page looks OK now. Thank you very much. That was really sick error, but your preprocessing code did the job, thank you again ;)

Glad you were able to get it to work.  Thanks for letting us know.

Great answer, thanks, this tells me exactly what the problem is!
I'm stuck with the same problem, however, this code does not work for v3 as StartupHook has been replaced by ready() if I understood correctly from the docs.
I can't seem to figure out how to access "data.math" inside ready(). There doesn't seem to be any InputJax.TeX.prefilterHooks arrangement either. Any help would be great.

For v3 it is a little more complicated, since the comment strings are removed before the TeX input jax runs, so a pre-filter doesn't work in that case.  So you have to modify the code that collects the  DOM strings to make sure the CDATA comments are included.  Here is one possible solution:

This should come before the script that loads the MathJax component you are using, and if you already have other configuration, it needs to me merged into that.

This doesn't seem to work for me. On the page layout I have the following, which goes inside  ... .

Here the latter half comes from the official documentation, while in the first half I'm simply stripping the dom content of all % <![CDATA[ ... %]] strings.
I see that github needs to allow math_engine: nil in Jekyll’s _config.yml for Kramdown settings. This is currently an open issue. Once that is solved, we won't need this fix I believe :)

OK, thanks for sharing your solution.  I suspect you don't need to the findScript render action any longer, since your event listener will replace all the old MathJax scripts with regular delimiters, so there won't be any left for that render action to find.

Hi, I've tried both of the presented solutions but neither has worked. I am using V2 and I noticed that OP's website is not rendering the maths, so I'm assuming that solution may be outdated.
Here is my mathjax.html

The last script is for mathjax to render any math that was missed in the first pass since I'm using a dynamic jekyll site.

@moribots, you are using \[...\] delimiters around the displayed math containing the arrays, bit have removed those delimiters from the displayMath list of delimiters.  So they are ignored by MathJax, and you end up getting the \begin{array}...\end{array} ending up as math delimiters because \begin{...}...\end{...} are taken as math delimiters when preocessEnvironments is true (which it is by default).
Try using
displayMath: [ ['$$','$$'], '[\\[','\\]'] ],

od use '$$...$$' around your displayed equations instead of '[...]'.

Thanks for the quick response!
Here is an example of an array I use, delimited by $$ ... $$
$$ A_{1b} = \left[\begin{array}{ccc} 1 & 0 & 0 \\ -D & 1 & 0 \\ 0 & 0 & 1 \end{array} \right] $$
If I understood correctly I think this fits your description of what should be working.

@moribots, the arrays on the link you provided above are not delimited by $$ but by \[...\].  That is why they are not working.  I do not see any on that page that use $$ delimiters.  Is that on a different page?  If so, can you provide a link to it?
The issue on the page for which you provided a link is the inconsistency between the configured delimiters ('$$...$$) and the ones used in the page ([...]`).

I just noticed something odd based on your comment:

This array delimited as you described by \[...\] on the live site is actually written as $$ A_{1b} = \left[\begin{array}{ccc} 1 & 0 & 0 \\ -D & 1 & 0 \\ 0 & 0 & 1 \end{array} \right] $$ in my code. Do you have any idea how this could have happened?

@moribots, whatever content management system you are using to create your pages (Jekyll?) is probably identifying the math (so that it doesn't further process the math itself) and inserting it into the page after creating the HTML needed for it.  It may change the delimiters to the standard LaTeX ones in that process.
So your solution is to change the configuration as I suggested above so that it will work both with your original delimiters and the ones that Jekyll uses.

Thanks! When I tried to add the backslash delimiter option it actually stopped rendering all of the maths instead of just the arrays like before. Does that give you any clues?
Yes  I am using Jekyll

That suggests that you made a syntax error in your configuration and MathJax couldn't process it.  Were there any errors in your browser console window?

When I run bundle exec jekyll serve I get no errors.
This is my whole config:

I hope that helps.

When I run bundle exec jekyll serve I get no errors.

I wasn't asking about running Jekyll, I was asking about when you load the page in the browser, do you get any errors in the browser's console window.

While you have set the inline math to include (...) as delimiters, you have not changed the disoplayMath to include [...] as delimiters, which is what you need in order to resolve the problem you are seeing, since the displayed equations are delimited by [...] not $$...$$ in the final page produced by Jekyll.

The last script is probably not needed, and is in the wrong place in any case.  At the point it runs, MathJax will likely not be available yet (since your previous script has the ASYNC attribute), so likely will generate an error about MathJax not being defined (it does for me).  Also, it doesn't really do anything since it used a typeset right after the initial typeset, and that probably won't find any new math since your page will not have had time to load any in between typesets.  I'd try leaving that out entirely, since it is likey not actually running in any case.
Davide

Thanks for the detailed response! I've gotten rid of the last script.
If I include displayMath: [ ['$$','$$'], '[\\[','\\]'] ], , none of my maths renders anymore.

If I include displayMath: [ ['$$','$$'], '[\\[','\\]'] ], , none of my maths renders locally anymore.

you have the quotation marks in the wrong place.  Try
displayMath: [ ['$$','$$'], ['\\[','\\]'] ],

instead.  Your way, the configuration failed to be processed, and so MathJax couldn't run.  You should probably see an error message in the browser console.

Thank you so much, that was it!

I come across this thread while struggling finding a solution to why all of the line break delimiter // get converted to
. The same code works locally. So I guess it's from the server side. I've tried with the following code, but it doesn't seem to work.
The website is at https://lhoangan.github.io/camera-params/, Equation (3)

Equation 3 renders properly for me in the page you link to.  Does that link not work for you?
Also note that v2.7.1 of MathJax is three and a half years old, so you may want to update to a more recent version.  The highest version of 2.x is 2.7.9.

It turns out that the problem lies with Kramdown 2.3.0 (or Jekyll 3.9.0).
My old version at local does not have problem with triple backslashes at
the end of line (1 extra for espace character), but the current version
used by GitHub page keeps interprete and convert double backslashes as hard
line-break ().
So I added an extra space behind the backslashes and now it is solved.
Thanks you for your time and for the info. I'll check it out.
On Sat, Sep 19, 2020, 20:50 Davide P. Cervone<EMAIL_ADDRESS>wrote:

Equation 3 renders properly for me in the page you link to. Does that link
not work for you?
Also note that v2.7.1 of MathJax is three and a half years old, so you may
want to update to a more recent version. The highest version of 2.x is
2.7.9.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mathjax/MathJax/issues/2377#issuecomment-695343835,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACHG2TS6JWHV2NP2LWNQ7W3SGT4QFANCNFSM4LRPPESA
.


Fix to allow custom CheckpointIO with strategy classes

What does this PR do?

Earlier since we initialized accelerator specific CheckpointIO in the strategy init, user's don't have an option to plugin custom CheckpointIO due to this:
https://github.com/Lightning-AI/lightning/blob/0e5312833fa70fb744f316f3db19c78b38eb9683/src/pytorch_lightning/trainer/connectors/accelerator_connector.py#L348-L352
Found while working on #13658
Does your PR introduce any breaking changes? If yes, please list them.

Before submitting

[ ] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes? (if necessary)
[x] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[x] Did you list all the breaking changes introduced by this pull request?
[x] Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃

need help with mypy

Thanks!


Rename go package to reconciler.io/dies

Users will need to update their go imports.
Before:
import diecorev1 "dies.dev/apis/core/v1"

After:
import diecorev1 "reconciler.io/dies/apis/core/v1"

Refs #133

I'm not sold on this change, but this is what it would look like.


Configure Publish Stage to upload API reference docs to S3

Closing since docs are on S3 and we have another issue to track publishing API reference through the publish stage


Routing Does not work

Hello,
I wrote a application with equivalent objects as the events calendar example, But my models are CamelCase( each model name prefixed with my app Acronym 'Cdj' to avoid naming conflicts with other plugins.
Wordpress permalink: Cus/%postname%/%post_id%/tom structure: I tried /%postname%/ and /%postname%/%post_id%/
in my class CdjActivite I added:
function to_url($object) {
return 'cdj_activites/'.$object->id;
}
In the debugger the code executes and takes effect on the Resulting url as I changed the return value to see the effect
I added route MvcRouter::public_connect('cdj_activites/{:id:[\d]+}/{:extra:(.*)}', array('controller' => 'cdj_activites', 'action' => 'show')); ( but I believe the predefined Router rules should do it I just did it to make sure.
but with the debugger I see that when I click on the View link from the Wp-Admin Cdj Activites List I
get invariably The requested URL /av-data/cdj_activites/3 was not found on this server. without the controller show method or the show view being invoked as I have breakpoints in the debugger..
av-data is my virtual host under apache
Pls advise I am new to your plugin I like the MVC concept. I can zip you the whole small application if you wish, but you may already have a simple explanation.

typo Wordpress permalink: Custom structure: I tried /%postname%/ and /%postname%/%post_id%/

cercledujeudi.zip

just added zip file of my very small plugin as additional doc

Alain--have you tried flushing the rewrite rules by saving permalinks?

Hello Robert yes I did without success. I am running this under VirtualBox - Ubuntu 16.04.
My .htaccess file contains # BEGIN WordPress

RewriteEngine On
RewriteBase /av-data/
RewriteRule ^index.php$ - [L]
RewriteCond %{REQUEST_FILENAME} !-f
RewriteCond %{REQUEST_FILENAME} !-d
RewriteRule . /av-data/index.php [L]

END WordPress
And my apache Virtual Directory  says DocumentRoot /var/www/html/av-data
I sense that maybe .htaccess needs to be more explicit than just RewriteRule . /av-data/index.php [L] ?
Many thanks!

I installed and tested the events calendar sample and I am hitting the same problem
so I know something is wrong in either setting permalinks/mod_rewrite ( enabled per command
sudo a2enmod rewrite) .htaccess( std wordpress) or ???.
What I noticed are the urls - in the admin part for example to list all Venues:
http://<IP_ADDRESS>/av-data/wp-admin/admin.php?page=mvc_venues
To Edit one venue:
http://<IP_ADDRESS>/av-data/wp-admin/admin.php?page=mvc_venues-edit&id=1
Works just like my app
http://<IP_ADDRESS>/av-data/wp-admin/admin.php?page=mvc_cdj_activites
and to edit
http://<IP_ADDRESS>/av-data/wp-admin/admin.php?page=mvc_cdj_activites-edit&id=3
but its the View link that fails both for the events calendar example:
The requested URL /av-data/venues/1/ was not found on this server.
just like for mine:
The requested URL /av-data/cdj_activites/3 was not found on this server.
So its the "Pretty link" PostName that does not seem to apply in the Wp-Admin panels( where a page= url type is applied.
Thanks for looking into this!

Alain, I'm sorry to say it sounds like you have configuration issues well beyond the scope of wp-mvc, with your WordPress installation specifically. Unfortunately, that isn't something we can help with here. Good luck!