Problems with lorentz transform (symbolic module)

🐞 Problem
There is a lot to be desired regarding lorentz_transform function in the classes of symbolic module. This issue consolidates a number of problems into a single issue.
🎯 Goal

[ ] Devise a testing strategy for lorentz_transform across all the classes and implement them.
[ ] lorentz_transform implemented in ChristoffelSymbols is probably incorrect, Christoffel Symbols being a pseudo-tensor. Research about this, and change the implementation along with suitable tests.
[ ] Currently, "parent metric" is not preserved while during lorentz transformation. Do something about this. (This issue duplicates #355 )

💡 Possible solutions

This issue needs extensive research. Comment on the problem being addressed and discuss in the comments before opening a PR.

📋  Steps to solve the problem

Comment below about what you've started working on.
Add, commit, push your changes
Submit a pull request and add this in comments - Addresses #<put issue number here>
Ask for a review in comments section of pull request
Celebrate your contribution to this project 🎉

Can i take up this issue ?

That’s a collection of issues, each requiring deep theoretical knowledge. I wouldn’t recommend that you take these up. Good first issues, maybe??
Regards,
Ritwik

On 02-Feb-2020, at 3:06 PM, sagnik-chatterjee<EMAIL_ADDRESS>wrote:

Can i take up this issue ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hi, I know how "lorentz transformation" of Christoffel symbols should work, since they are not actual tensors. However, I am new to this project, so would be happy if someone wants to work together on it, I can provide some references and formula for that.

Hi, I know how "lorentz transformation" of Christoffel symbols should work, since they are not actual tensors. However, I am new to this project, so would be happy if someone wants to work together on it, I can provide some references and formula for that.

I would love to collaborate, let's get things done with #427 first.
BTW, it would be great help!

@gimseng However, I am thinking of deprecating lorentz transform and implementing change of basis vectors, lorentz transform being a special case of the latter.
What do you say?

Sure, I think doing a general coordinate transformation (or you call it basis change) is more relevant and useful for a GR package. I could start working on this. However, I wanted to do it probably with git and stuff so maybe it will be a little slower, and I will ask lots of questions to make sure I don't screw with master and I want to do rebase and squashing properly. If its ok with you all, I will go dig around to figure out how and what should be done.
On another note, I think for Lorentz transformation, the Christoffel will transform like appropriate Lorentz tensors, since the "extra terms" vanish for Lorentz-transfomation. So they are Lorentz tensors but not general-covariant tensors. If need be, and if some points me towards the right place, I could write a few lines to explain this in a doc or issue ticket or something.
Finally, what is this issue with "parent metric"? Is this a class inheritance thing or some GR thing?

Okay then! Have a look at the symbolic module. There are many examples on their usage. Let’s start with general coordinate transform then.
With this we won’t have the luxury of ignoring extra terms for pseudo-tensors. Also we need to devise a strategy on how to implement them!
Regards,
Ritwik

On 12-Feb-2020, at 1:21 PM, Gim Seng<EMAIL_ADDRESS>wrote:

Sure, I think doing a general coordinate transformation (or you call it basis change) is more relevant and useful for a GR package. I could start working on this. However, I wanted to do it probably with git and stuff so maybe it will be a little slower, and I will ask lots of questions to make sure I don't screw with master and I want to do rebase and squashing properly. If its ok with you all, I will go dig around to figure out how and what should be done.
On another note, I think for Lorentz transformation, the Christoffel will transform like appropriate Lorentz tensors, since the "extra terms" vanish for Lorentz-transfomation. So they are Lorentz tensors but not general-covariant tensors. If need be, and if some points me towards the right place, I could write a few lines to explain this in a doc or issue ticket or something.
Finally, what is this issue with "parent metric"? Is this a class inheritance thing or some GR thing?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

See from_metric classmethod in ChristoffelSymbols, you would understand what  parent_metric is about. It actually preserves the metric from which other quantities are derived.

I see, if I understood correctly, the structure of implementations are such that you do something.from_somethingelse(...) to actually calculation, say Christoffel / Riemann / Ricci from the appropriate object (say metric/Christoffel/Riemann).
So the Christoffel method has a from_metric method, which computes the Christoffel symbol, and return as a tensor class.
If I understood the issue correctly, now under Lorentz/general coordinate transformations, do we want to keep track of which metric a Christoffel symbol was constructed from? If we did, when we  applied a transformation/basis change to the Christoffel symbol, returning the Christoffel symbol in a new coordinate/basis, the question/issue is then: do we transform the original metric (in original coordinates/basis) to the metric in new coordinate and assigning this to be the parent metric of the Christoffel symbol in new coordinate?

@gimseng Yes, you understood this right. Along with transformation of the derived quantity, we also need to transform the metric from which it is derived, and set the new metric as parent_metric.
But we would need to decide on whether to drop lorentz altogether in favor of general transformation. I say yes!
If it is yes from your side also, we can first work on general transformation, and later drop lorentz, saving us the efforts here!

And this parent_metric thing would also apply in general transformations. There's no running from this problem :(

Also, here is a link detailing mathematics behind transformation of christoffels

hi I am a little busy these two weeks, maybe I will check in again after I finished my busy weeks?

Sure ! Open source is people coming together to contribute at their own convenience.
Regards,
Ritwik

On 14-Feb-2020, at 2:24 PM, Gim Seng<EMAIL_ADDRESS>wrote:

hi I am a little busy these two weeks, maybe I will check in again after I finished my busy weeks?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or unsubscribe.

I think, it is better to have mathematical operations as mixins that suitable objects can then derive from. Perhaps one can look into this.
Also ref: https://github.com/einsteinpy/einsteinpy/issues/497#issuecomment-1295053256.


The SkipButton can not been change height

Some time i can not tap the skipbutton .
I think the height of skipbutton is so low.
But when i set the height of skipbutton,
It can not work.
Maybe autosize or Constraint cause it?

I just fixed this issue here  #127, it was a height constraint problem.

Resolved in #129


Update to v1.1.0

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] 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.
I do have some suggestions for making it better though...
For recipe:

License is not an SPDX identifier (or a custom LicenseRef) nor an SPDX license expression.

Documentation on acceptable licenses can be found here.

@conda-forge-admin, please rerender

@mhearne-usgs Can you look at this? This seems to be an issue with the wrong compiler getting installed (or different than what the build script is expecting) in the linux builds. This is the relevant portion of the docker log:
  creating build/temp.linux-x86_64-3.6/gmprocess
  creating build/temp.linux-x86_64-3.6/gmprocess/metrics
  clang -DNDEBUG -fwrapv -O2 -Wall -Wstrict-prototypes -march=nocona -mtune=haswell -ftree-vectorize -fPIC -fstack-protector-strong -fno-plt -O2 -ffunction-sections -pipe -isystem /home/conda/feedstock_root/build_artifacts/gmprocess_1598804790600/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_p/include -fdebug-prefix-map=/home/conda/feedstock_root/build_artifacts/gmprocess_1598804790600/work=/usr/local/src/conda/gmprocess-1.1.0 -fdebug-prefix-map=/home/conda/feedstock_root/build_artifacts/gmprocess_1598804790600/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_p=/usr/local/src/conda-prefix -DNDEBUG -D_FORTIFY_SOURCE=2 -O2 -isystem /home/conda/feedstock_root/build_artifacts/gmprocess_1598804790600/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_p/include -fPIC -Igmprocess/metrics -I/home/conda/feedstock_root/build_artifacts/gmprocess_1598804790600/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_p/lib/python3.6/site-packages/numpy/core/include -I/home/conda/feedstock_root/build_artifacts/gmprocess_1598804790600/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_p/include/python3.6m -c gmprocess/metrics/oscillators.c -o build/temp.linux-x86_64-3.6/gmprocess/metrics/oscillators.o
  unable to execute 'clang': No such file or directory
  error: command 'clang' failed with exit status 1
  Building wheel for gmprocess (setup.py): finished with status 'error'

The only thing we do currently to specify the c compiler in the recipe.yml is:
  build:
    - {{ compiler('c') }}

I've been looking for documentation on this, but I can't find any info on this type of error or how to specify different c compilers (e.g., clang vs gcc).

Also, note that it seems like it has to be something specific to the conda forge build process because our CI builds were successful.

@conda-forge-admin, please update for conda-build 3

@conda-forge-admin, please rerender


Does this gem support "setex" call

I have a use case where I would like to use "SETEX" call to set the key with expiration in single call. Currently I am doing this in two separate calls and if possible I would like to avoid one extra call to redis

I do not believe this is currently supported, but it seems like a great
addition. I would be willing to accept a patch with tests to add this
functionality.   We would probably need to check for the version of Redis,
and/or for the existence of SETEX to support older Redis servers.
On Fri, Feb 10, 2017 at 7:54 AM Nachiket Londhe<EMAIL_ADDRESS>wrote:

I have a use case where I would like to use "SETEX" call to set the key
with expiration in single call. Currently I am doing this in two separate
calls and if possible I would like to avoid one extra call to redis
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nateware/redis-objects/issues/205, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAcoOZf8JUpDxnsSr1Xsp7A-OVgzwzdks5rbIhNgaJpZM4L9iHE
.

Great I will try my best to get it out this week end, take a look if you
would like
On Fri, Feb 10, 2017 at 11:48 AM, Nate Wiger<EMAIL_ADDRESS>wrote:

I do not believe this is currently supported, but it seems like a great
addition. I would be willing to accept a patch with tests to add this
functionality. We would probably need to check for the version of Redis,
and/or for the existence of SETEX to support older Redis servers.
On Fri, Feb 10, 2017 at 7:54 AM Nachiket Londhe<EMAIL_ADDRESS>wrote:

I have a use case where I would like to use "SETEX" call to set the key
with expiration in single call. Currently I am doing this in two separate
calls and if possible I would like to avoid one extra call to redis
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nateware/redis-objects/issues/205, or mute the
thread
<https://github.com/notifications/unsubscribe-
auth/AAAcoOZf8JUpDxnsSr1Xsp7A-OVgzwzdks5rbIhNgaJpZM4L9iHE>
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nateware/redis-objects/issues/205#issuecomment-278996544,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQSGphEWxnW3ZQUpRsT5ShC8VB4WYt6Cks5rbJTxgaJpZM4L9iHE
.


Update to require calicoctl

Description
Update node-to-node encryption doc to require calicoctl. OS user did not know where to find calicoctl.

Merge to nightly
Cherry pick to /master

Deploy preview for calico ready!
Built with commit 4bde3072246b7be82978d00252f6cc19112ef869
https://deploy-preview-3641--calico.netlify.app

Deploy preview for calico ready!
Built with commit 9bc540b9e5db3a8131b082333eecceb1d40c237c
https://deploy-preview-3641--calico.netlify.app

Deploy preview for calico ready!
Built with commit 20d3c3e29dab59c6d2ae2ade562eeeea36ba5600
https://deploy-preview-3641--calico.netlify.app


Array of nested template structs/class not completing

A class/struct that has array of template struct with nested template argument (foo in the example) doesn't recognise the class. In this example, TemplateExample holds an array of the template struct NestedStruct.
template <int foo>
struct NestedStruct
{
	int a = foo;
	int b = foo + 1;

	int boo()
	{
		return a + b;
	}
};

template<int foo>
struct TemplateExample
{
	NestedStruct<foo> arr[5];

	int boo(int index)
	{
		// Auto Completion not working
		return arr[index].boo();
	}
};

Logs
logs.txt
System information
Output of clangd --version: clangd version 10.0.0-4ubuntu1
Editor/LSP plugin: Sublime 4107 with sublime_lsp
Operating system: Ubuntu 20.04

this is what the ast node for arr[0].boo(); looks like:
        |-CallExpr 0xaf3f6a0 <line:15:5, col:16> '<dependent type>'
        | `-CXXDependentScopeMemberExpr 0xaf3f658 <col:5, col:12> '<dependent type>' lvalue .boo
        |   `-ArraySubscriptExpr 0xaf3f638 <col:5, col:10> '<dependent type>' lvalue
        |     |-MemberExpr 0xaf3f5e8 <col:5> 'NestedStruct<foo> [5]' lvalue ->arr 0xaf109d8
        |     | `-CXXThisExpr 0xaf3f5d8 <col:5> 'TemplateExample<foo> *' implicit this
        |     `-IntegerLiteral 0xaf3f618 <col:9> 'int' 0

unfortunately ArraySubscriptExpr produces a dependent type rather than a NestedStruct<foo>. I am not sure how complicated it would be, but we can try to make use of underlying type for subscripts on arrays/pointers since they can't (?) have overloaded operator[].

We can hack more rules into code completion getApproximateType
But I think clang's AST might be being too conservative here in not typing the expr and we can fix it at that level.
If i'm reading the rules right:

overloading isn't an issue here (over.sub, over.match.oper) if we can prove the LHS doesn't have class type
in the non-overload case, one arg must be pointer or array, and the other must be integer or enum (LHS vs RHS doesn't matter). So if LHS is pointer-or-array, we can assume it's the base and thus determine the type.

https://reviews.llvm.org/D107275
Not sure exactly how far to go with this, hopefully Richard can weigh in


Documentation

Use MkDocs and Material theme.
Github workflow pushes to gh-pages when a .md file is changed in the docs folder.
I only added a docs/Readme.md and a docs/getting-started/Readme.md.

Search is included but it does not use lunr, sorry ;-).
Can be tested locally:

Run pip3 install -r docs/requirements.txt to install dependencies.
Run mkdocs serve to start the site.

Should be available at https://bleroy.github.io/lunr-core/
Ex: https://github.com/agriffard/lunr-core/tree/gh-pages

Thanks for the contribution!


[AutoPR managementgroups/resource-manager] typo: managementgroups/resource-manager/Microsoft.Management

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/4709

(message created by the CI based on PR content)

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/2934


Typo / ambiguous sentence "Create a new connection to the all connectors."

Typo / confusing sentence - Import the solution (step 3):

"Create a new connection to the all connectors."

Clarify "new connection to all connectors":
This can be interpreted as actually creating new connectors instead of using existing connectors present if the reader has also followed the documentation for installing core components (step 9) and the approvals connector by creating a dummy approvals flow (Initialize flow approval tables in your environment)

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

ID: e14181d4-574f-95e3-c2c9-545116549c9b
Version Independent ID: bf449b92-6756-e08a-a58c-d561e36947cb
Content: Set up governance components - Power Platform
Content Source: power-platform/guidance/coe/setup-governance-components.md
Service: power-platform
Sub-service: guidance
GitHub Login: @manuelap-msft
Microsoft Alias: mapichle

Made a local change, will be updated in December. Thank you!


OpenXR support

Will there be an OpenXR support in the future? Revive started to support it and I hope this can solve the warped display for most of the Oculus games

I don't know much about OpenXR. Will it replace SteamVR(OpenVR)?
Can we see games which support OpenXR in near future?


Type-Fortified build

Optional build with as many runtime type-checks as possible.
Note: We could use TypeScript for the library itself, but it won't protect the users.

+1 for TypeScript. Types are great documentation.

i would much prefer flowtype personally
it's a stronger language and it's going to win

@phadej What's your current take on using TypeScript for the project itself? I have been checking out this project with the TypeScript typings: while the do compile, they're just plain wrong in a lot of places.
For starters the check result Result type does not even allow for the desired true state. Also Promise is completely absent from the typings, though it works great in practice (just differently from what I would expect from the typings).
Would you actually desire stricter types to help users verify their code before execution? In that case I would agree with your proposal to convert the codebase to TypeScript first to get full type guarantees in this project, and then into the hands of the users 🙃

In retrospect, typescript won, and I was wrong


[E-Document Core] Added field Service Participant Id to Vendor and Customer

Customer and Vendor needs a field to save identification data related to E-Document Service. Used in E-Document connectors
Fixes #27060
Fixes AB#544798

@aholstrup1 This pull request has replaced #27051 due to using merge instead of rebase policy. Thank you for your patience

@geschwint What if customer is sending to multiple services?

@Groenbech96 You are raising a valid question here. I'll raise this question in our channel

Comment on last commit. Resolving object number conflict. Just allocated next available number. Error in build

Closing this PR as MS is providing a solution in PR #27509


Full finetune < 16GB

Context

We'd like to enable a variant of full finetune that trains in < 16GB of RAM for users with consumer grade GPUs that have limited GPU RAM. This PR enables the full finetune to fit into that memory requirement.

Changelog

Add support for running optimizer in backward. This is done by following this tutorial: https://pytorch.org/tutorials/intermediate/optimizer_step_in_backward_tutorial.html in _setup_optimizer.
Add OptimInBackwardWrapper to checkpoint the optimizer states when running in backward + unittests.
Update relevant portions of training code such as not calling step, zero_grad when using optimizer in backward.
Disable optim state dict checkpointing for optimizer in backward, as this is nontrivial to implement and not supported yet.
Warn when user is loading in an optimizer state checkpoint but optimizer in backward is enabled. In this case, the checkpoint is not loaded in.
Change configuration to use PagedAdamW optimizer, when using this, memory is < 16GB:

GPU peak memory allocation: 13.993540096GB, GPU peak memory reserved: 15.974006784GB, GPU peak memory active: 13.993540096GB

Test plan

Run with bnb PagedAdamW + optim in backward + full bf16: tune full_finetune_single_device --config recipes/configs/full_finetune_single_device.yaml &> out &

Memory does not spike above 16GB reserved -
Memory Stats::
 3450     GPU peak memory allocation: 13.92 GB
 3451     GPU peak memory reserved: 15.97 GB
 3452     GPU peak memory active: 13.92 GB

Loss curves:

Eval result:
truthfulqa_mc2: {'acc,none': 0.4372944697596623, 'acc_stderr,none': 0.014862338279863522, 'alias': 'truthfulqa_mc2'}

Checkpoint save/load of optimizer states

Run and save recipe state: tune full_finetune_single_device --config recipes/configs/full_finetune_single_device.yaml max_steps_per_epoch=1 device=cuda:1
Load recipe state, including optimizer state, back in for training: set resume_from_checkpoint: True and recipe_checkpoint: /tmp/llama2/recipe_state.pt, then run: tune full_finetune_single_device --config recipes/configs/full_finetune_single_device.yaml device=cuda:1 --> training resume works

Discussion points

Configuring bnb optimizers. Currently we are just able to do this in the config itself, user can just swap torch.optim.AdamW for bitsandbytes.optim.PagedAdamW for example. Can definitely have some documentation around this, but wondering if we need to do anything further for configuration.
Currently, I'm mostly testing w/PagedAdamW but the 8bit adam from BNB works as well and can be similarly configured. PagedAdamW results in more memory savings (~14GB peak memory vs 27GB for 8 bit adam).
Checkpointing of optimizer in backward. For a fully functional recipe, checkpointing optimizer states for optimizer in backward is critical. We can do this with distributed's _apply_optimizer_in_backward API + distributed_state_dict, though the downsides are that the latter API is only avail. in more recent versions of torch, and the former API is private. OTOH, to implement checkpointing using the technique in https://pytorch.org/tutorials/intermediate/optimizer_step_in_backward_tutorial.html is not well understood from a technical perspective (hasn't been done before, and there are tricky things to figure out like how to key the optimizers by a UUID instead of a numerical index, which won't work when there are multiple optimizers). Using the former implementation is also useful since it is better tested and designed than building something ourselves.
UX / exposing this memory efficient config. Currently I've just modified the config in place, but we can easily land a separate memory efficient config. We could even author an entirely separate recipe for users who don't want to enable optimizer in backward and deal with the UX complexity of using this.

Follow up work

Documentation for using bitsandbytes optimizers, update documentation for memory efficiency in general

If there are opportunities to make the optimizer wrapper have identical APIs to a normal optimizer, we can hide a lot of logic within the wrapper class and we won't need to check if self._optimizer_in_bwd in so many locations. I'm worried this will confuse new users esp since this is one of the main flagship recipes

So I'm actually reluctant to have the optimizer wrapper have identical APIs as a normal optimizer. This would mean it would need to implement a .step(), which would likely be a noop since the step actually happens in the backward pass. In general I'm a bit reluctant to add these sort of pass through abstractions that are just no-ops, as the user needs to click through the implementation to find it doesn't do anything (as expected).
I do agree that all the if/else'ing here is not ideal.

In general I'm a bit reluctant to add these sort of pass through abstractions that are just no-ops, as the user needs to click through the implementation to find it doesn't do anything (as expected).

I see how this is odd but imo this is preferable to adding more if-else statements depending on certain parameters in the config. I've seen the runaway version of this where more features were added and the entire training recipe was bloated with these if-else checks that it was hard to even parse the core logic. Only advanced users may need to parse the logic of the optimizer in backward - I'd rather keep that more hidden so we don't risk muddying the core logic of a recipe that most beginner users will use as a starting point. Maybe there are alternative ways to do this by creating more modular methods?

In general I'm a bit reluctant to add these sort of pass through abstractions that are just no-ops, as the user needs to click through the implementation to find it doesn't do anything (as expected).

I see how this is odd but imo this is preferable to adding more if-else statements depending on certain parameters in the config. I've seen the runaway version of this where more features were added and the entire training recipe was bloated with these if-else checks that it was hard to even parse the core logic. Only advanced users may need to parse the logic of the optimizer in backward - I'd rather keep that more hidden so we don't risk muddying the core logic of a recipe that most beginner users will use as a starting point. Maybe there are alternative ways to do this by creating more modular methods?

This makes sense. I'd also like inputs from @kartikayk and @ebsmothers on this. One alternative I considered was writing an entirely new recipe, but decided not to go in this direction after discussing with @kartikayk offline. For now, I have an idea on how to separate this out into reusable components, so will put up something with that for folks to take a look at.

@rohan-varma Is there a way to achieve the same using torchao? BnB support on AMD is requires complex setup, compile from source did not work for me.


metropolis: implement removing nodes

Currently there is no way to remove a node from a cluster

There's two things here:

A proper, safe decomissioning flow, including draining, Kubernetes state sync, etc.
A way to just remove dead nodes and generally 'clean up'.

We mostly need the latter, so that's what we should probably implement first. Then the decomissioning flow will include the latter, either automatically or as part of the process to be done by operators.

Isnt this already implemented?

Partially - we can remove everything but control plane nodes.

We need this to be done until mid-september

With https://review.monogon.dev/c/monogon/+/3343 merged we will be able to safely and fully remove nodes from consensus.
A full decommissioning flow is blocked by SPIFFE support and general auth refactor, as that mostly revolves around distributing some revocation lists to make sure a node can't reconnect to the cluster. However, we can still remove non-decommissioned nodes by setting DeleteNodeRequest.SafetyBypassNotDecommissioned.

After removing etcd membership from a node, etcd panics which takes down the entire node. The panic is in IsLocalMemberLearner, stack trace:
go.etcd.io/etcd/server/v3/etcdserver/api/membership.(*RaftCluster).IsLocalMemberLearner(0xc000879380)
	external/gazelle~~go_deps~io_etcd_go_etcd_server_v3/etcdserver/api/membership/cluster.go:859 +0x26d
go.etcd.io/etcd/server/v3/etcdserver.(*EtcdServer).IsLearner(0xc001020c18?)
	external/gazelle~~go_deps~io_etcd_go_etcd_server_v3/etcdserver/server.go:2813 +0x1a
go.etcd.io/etcd/server/v3/etcdserver/api/v3rpc.(*maintenanceServer).Status(0xc001020bb0, {0xc001d38638?, 0xc001d38638?}, 0xc001d38690?)
	external/gazelle~~go_deps~io_etcd_go_etcd_server_v3/etcdserver/api/v3rpc/maintenance.go:228 +0x137
go.etcd.io/etcd/server/v3/etcdserver/api/v3rpc.(*authMaintenanceServer).Status(0x4a5c340?, {0x3122cf8?, 0xc0026f2f00?}, 0x2d0ebd8?)
	external/gazelle~~go_deps~io_etcd_go_etcd_server_v3/etcdserver/api/v3rpc/maintenance.go:306 +0x25

I'm not sure how to fix the panic.

We could just return false in IsLocalMemberLearner instead of panicking. This is technically correct in that a node that is no longer a member is also not a learner anymore, but clients might not expect this.
We could change the function to return (isLearner bool, ok bool) or similar to force clients to handle the case where the local node is not a member. But this is a breaking change in a public interface and thus also not ideal.
Some existing callers of IsLearner first check IsMemberExist to avoid the panic, but that does not completely solve the problem because it's possible that a member is removed between the two calls.

Another way to improve the situation is to move etcd into a separate process such that a panic in etcd does not restart the entire machine. This is #349.

Removing consensus nodes is now possible with these changes merged:
https://review.monogon.dev/c/monogon/+/3437
https://review.monogon.dev/c/monogon/+/3438
https://review.monogon.dev/c/monogon/+/3454
There are still some rough edges that could be improved:
Bootstrap node does not stop consensus
The bootstrap node will not stop consensus when it has never rebooted and the role is removed, because the bootstrap data takes priority over the role.
https://github.com/monogon-dev/monogon/blob/d5538b52d7a8739f7123458c10973be36b27b9ff/metropolis/node/core/roleserve/worker_controlplane.go#L134-L136
Move leadership
When removing the consensus role from a node which is currently either etcd or curator leader, there is some downtime until a new leader is elected. It would be nicer to first move leadership to another node in this case. The challenge with moving the leadership is that maintenance.MoveLeader can only be called on the etcd leader node, and etcd leadership is currently independent from curator leadership. We could solve that by making the curator leadership follow etcd leadership, which might have performance benefits as well.
See discussion in https://review.monogon.dev/c/monogon/+/3437


HNC: Complete security review for sub-project graduation

Complete an security review with approvals from at least one sig-auth technical lead as part of the graduation criteria to become a sub-project.
https://github.com/kubernetes/enhancements/blob/master/keps/sig-auth/1687-hierarchical-namespaces-subproject/README.md#graduation-criteria

/remove-lifecycle stale
On Mon, Nov 9, 2020 at 10:47 PM fejta-bot<EMAIL_ADDRESS>wrote:

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually
close.
If this issue is safe to close now please do so with /close.
Send feedback to sig-testing, kubernetes/test-infra and/or fejta
https://github.com/fejta.
/lifecycle stale
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes-sigs/multi-tenancy/issues/749#issuecomment-724431737,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE43PZB5QTCLUCS3T5TYVV3SPCZWXANCNFSM4NGJEAEQ
.

This is done.
/close

was this review published somewhere?


Create dynamic chat

how to create a chat dynamically as it is done when opened by the url: web.whatsapp.com/send?phone=number

Unfortunately, this lib doesn’t have this feature. We need a previous created chat (with someone else) to start a new conversation. I think that feature can be developed after study the app and app2.js, but no one else spend time to develop that functionality.

no? it's so easy...
var id = new window.Store.UserConstructor(idToFind);
Store.Chat.find(id).then(e=>{e.sendMessage('blablabla');});

Friends, thanks for the answers. @Bobaoapae's response is not exactly what it wanted, but it already serves well. Thank you!

No, with this code i can send message without have a chat created with user

and if only want the Chat object... just
var id = new window.Store.UserConstructor(idToFind);
Store.Chat.find(id);
Now the Chat object it's in Store.Chat.models, just not visible in web interface

Check this out -> http://prntscr.com/kn3wup

Whaaaaat? This is a very strange implementation of a find method, but a really good think :). Maybe we can explore how wp creates new chats to prevent that bug on view.

@bobaoapae This is so interesting, but I'm getting the error:
window.Store.UserConstructor is not a constructor
I'm using the last version of wapi.js. How do you getting the UserConstructor?

@bobaoapae Fala João Victor, preciso falar com você, teria algum email para entrar em contato?

@italomatoss<EMAIL_ADDRESS>


passthrough option

service can forward the matched request to one of the definied upstreams and patch the response

closed with #6


make compatible with 3.11

address https://github.com/rbarrois/aionotify/issues/18. (drops support for 3.4, I think)

@rbarrois - is there any progress towards getting this merged?
The lack of Python 3.11 support from this module causes Home Assistant's keyboard_remote integration to break in the next release of Home Assistant, as said release runs on Python 3.11.

@rbarrois can you transfer project to new maintainer?

@oolivero45 @spumer  I just sent @rbarrois messages on Linkedin and Twitter regarding this. Maybe if enough of us reach out they might be able to help, otherwise a fork might be in store?

Fixed! Thanks a lot.
As a side note, I'm more easily reached by email than over social media :)

@rbarrois Awesome, thanks :)


Pysmbc

Motivation for this change
https://askubuntu.com/questions/548361/browse-button-disabled-in-windows-printer-via-samba-for-printer-shares
Things done

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

[x] NixOS
[ ] macOS
[ ] Linux

N/A Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

This commit seems to break Travis and nox review build, e.g. at https://travis-ci.org/NixOS/nixpkgs/jobs/282048362.
The license is wrong, "gplv2" instead of gpl2 or more correctly I guess gpl2Plus, if you look at the license notices in the c files and setup.py.

Fixed in https://github.com/NixOS/nixpkgs/commit/c1e872d289b69f377252b56414a7ec560c98c27b

Changed to gpl2Plus, thank you!


remove the 0.5 scaling term

Remove the 0.5 scaling term in the full leapfrog step.
The original paper of "Neal's MCMC using Hamiltoniann" (http://www.mcmchandbook.net/HandbookChapter5.pdf, page 125) does not contain this term.
Also, both of the following link do not contain the 0.5 scaling term
line 357 of https://github.com/koepsell/pyhmc/blob/master/hmc2.py
line 199 of https://github.com/koepsell/pyhmc/blob/master/hmc2.m

I think the code here is trying to do half-step updates in momentum, and it needs a half step momentum update after the full step update.


Commute is never correct.

I am using the Assistant on a Raspberry PI - with the Google AIY kit and I am connected to the internet via an ISP.
If I say "OK Google - Good Morning" I get the usual greeting, then the weather, then details of my commute. BUT I am sitting in my own home - I don't need to commute - I am already there. If I ask the assistant where I live it has my address absolutely pinned down (from my google account).
The commute information is always wrong - it must be based upon my derived location using some kind of geolocation software (presumably based upon my ISP's allocated IP address). According to the assistant - the distance I have to commute varies from 7 to 16 miles.
Telling me what my commute to my home is - when I am sitting in my home - is a farce. To make matters worse I cannot find any way to tell the assistant to stop telling me about my non existent commute - HELP it is driving me bananas.

@sheridat sorry for the late followup, can you confirm which sample you're running?

@proppy  Hello. As you can see above I've asked the AIY folk to confirm what I am running but I think it is google-assistant-grpc==0.0.2 but I don't really know.

@proppy Hi I heard back and can confirm that is is google-assistant-grpc==0.0.2. I've just upgrade to 0.0.3 and that has the same behaviour

Same behaviour applies to me also for python sdk library

Can you try with the updated version of the library or gRPC?
We've added support for custom setting of location or street address, so responses should be more accurate.
google-assistant-library 0.1.0
google-assistant-grpc 0.1.0

If you're still seeing issues, you can manually set your location in each request with the gRPC DeviceLocation message.


Add ConTejas Code as Partner

PR Checklist

[ ] Addresses an existing open issue: fixes #000
[ ] That issue was marked as status: accepting prs
[x] Steps in CONTRIBUTING.md were taken

Overview
I:

Fixed .DS_Stores being included in the repo
Fixed text alignment on the Partners component
Added the ConTejas Code Podcast as a partner

@all-contributors please add @tejasq for code.

🤖 Beep boop! This comment was added automatically by all-contributors-auto-action.
Not all contributions can be detected from Git & GitHub alone. Please comment any missing contribution types this bot missed.
...and of course, thank you for contributing! 💙


Use conda-forge

As a sizeable portion of our dependencies are in conda-forge at this point, start using conda-forge for CI. Also, make some tweaks to handle some renaming, version constraints, and the conda-forge model for BLAS selection.

Once this is working, we will cut a new release v0.1.0a51, which will include these changes.

Coverage decreased (-7.9%) to 82.668% when pulling 7c48f2632b22da936c06adf0b5d998670c37c807 on jakirkham:use_conda-forge into f507a1f69afd7e92b6d96666b266bf161d722e65 on nanshe-org:master.


Script hanging after python bot.py run

Rasa Core version: current version
Python version: 2.7
Operating system : Ubuntu 16.10
Issue:
I downloaded restaurantbot then I run python bot.py train-nlu and python bot.py train-dialogue
after I run python bot.py run but script giving warning and after input nothing working.
/home/arshid/.local/lib/python2.7/site-packages/h5py/init.py:36: FutureWarning: Conversion of the second argument of issubdtype from float to np.floating is deprecated. In future, it will be treated as np.float64 == np.dtype(float).type.
from ._conv import register_converters as _register_converters
Using TensorFlow backend.
2018-01-16 17:13:54.468177: I tensorflow/core/platform/cpu_feature_guard.cc:137] Your CPU supports instructions that this TensorFlow binary was not compiled to use: SSE4.1 SSE4.2 AVX
Bot loaded. Type a message and press enter :
hi
Content of domain file (if used & relevant):

The respones [] is coming from action_listen.
In debugging, I see greet with highest confidence. MessageProcessor finds the highest probability at index 0. It looks up the action at index 0 and finds action_listen, whose run method is then executed resulting in [] being printed to the screen.
It looks to me like the probabilities are too low. The highest we get from both policies in the Supervised Learning tutorial is 0.11995545774698257. I'll hazard a guess that that defaults to "I don't know", hence action_listen.

We have just recently updated that example, thanks to @Ghostvv

I will close this, as this is an issue with the training data which has been fixed in the example that is part of the docs.
To get more insights into what the bot is doing when running it on the command line, append --debug when running it to get more outputs.


Customer feedback

lack of user testimonials #46

Title :Customer Feedback Added
Issue No. :#46
Code Stack : slider,react,
Close #46

Video (mandatory)

https://github.com/user-attachments/assets/6367f0c2-ff14-4eab-b1a4-52ee2c372539
Checklist:

[x ] I have mentioned the issue number in my Pull Request.
[ x] I have commented my code, particularly in hard-to-understand areas
[ x] I have created a helpful and easy to understand README.md if its a new page/tech stack
[ x] I have gone through the  contributing.md file before contributing

Additional context (Mandatory )
Are you contributing under any Open-source programme?

[ x] I'm a GSSOC-EXT contributor
[ x] I'm a HACKTOBERFEST contributor

@jeevan10017  is it now visible?


Kernel selector does not indicate that I'm currently connected to my Jupyter session

Testing #8778
After performing the steps in the test plan, if I open the kernel selector, I see the following:

It says I am currently connected to the remote kernel that I selected, but there is also a Jupyter session entry with 1 existing connection, and that does not say that I'm currently connected to that session. If anything, I'd expect the kernel selector list to say that that is my active connection.

I agree, however this is how VS Code API works, when you reload VS Code the connection will be restored correctly and you'll see the right kernel marked as the currently selected kernel.
We've looked into this and we don't think this can be resolved.
VS Code treats controllers as the actual kernel & stores the mappings, when in reality in jupyter these things are like kernel specifications which could spanw a real kernel that can get listed in the dropdown.
TLDR - I don't think we can do anything about this, we've look into this.
Upon reloading VS Code the correct kernel is displayed as selected.

The known use cases for how this bug could cause an issue doesn't seem worth fixing for now.


Update from python 3.9.11 to 3.10.14

Description
Bump from python 3.9 to 3.10 for flexibility such as union types: https://peps.python.org/pep-0604/
Will automatically resolve:

https://github.com/gusye1234/nano-graphrag/actions/runs/10999180637/job/30538791197

TBA for python version upgrade, closing this PR for now.


Add-on becomes unresponsive

Description of the issue
Some (if not all) zigbee devices can no longer be controlled without any errors in log
Requires overall restart after which all devices are controllable again
after some time (varies from 30 minutes to several hours) devices (switch and user interface) provide haptic feedback but control of target is not possible
Addon version
1.3.31-1
Platform
Home Assistant 2023.10.3
Supervisor 2023.10.0
Operating System 11.0
Frontend 20231005.0 - latest
Logs of the issue (if applicable)
No response

Hardware:
/dev/serial/by-id/usb-ITead_Sonoff_Zigbee_3.0_USB_Dongle_Plus_be7177ea2e1aec1181df76e5f01c6278-if00-port0

The situation has been "resolved"
I cannot figure out what changed/happened but entire system seems to be back as it was


used dfmt to format code & move private blocks to the beginning

used dfmt to format code
move private blocks to the beginning

Fixed this myself and made other changes


test: add unit test for cri

Signed-off-by: Zou Rui<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
add unit test for cri
Ⅱ. Does this pull request fix one issue?

NONE
Ⅲ. Describe how you did it
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

I will add more unit test for cri these days. Please do not merge this now.

Codecov Report

Merging #709 into master will decrease coverage by 0.03%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master    #709      +/-   ##
=========================================
- Coverage   10.94%   10.9%   -0.04%
=========================================
  Files          93      93
  Lines        5482    5482
=========================================
- Hits          600     598       -2
- Misses       4831    4836       +5
+ Partials       51      48       -3

Impacted Files
Coverage Δ

daemon/mgr/cri_utils.go
43.58% <100%> (-0.68%)
:arrow_down:

Continue to review full report at Codecov.

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

You may need to review this pr, since I have deleted the whole cri_test.go file.
Every function in that file is corresponding to cri_utils.go.
So, I remove those functions to cri_utils_test.go. @allencloud

There are still some unit tests missing in cri_utils_test.go, since those functions in cri_utils.go are unfinished. Maybe we can merge this pr now and more work will be done in the future.

LGTM

Thanks for your work. @ZouRui89


[OPENTOK-42902] Issue with Audio when minimizing app

I'm using opentok to publish and subscribe to audio and video, but there's an issue with publishing audio after putting the application in the background (minimize), it stops working.
The audio stops working at exactly 1 minute after minimizing the device.
Note: I can still hear other audio publishers.
The audio works again as soon as I put the app in the foreground.
Logs:
2021-02-17 18:24:24.945 30465-30480/? V/APM_AudioPolicyManager: setAppState(uid:10552, state:2)
2021-02-17 18:24:24.945 30465-30480/? V/APM_AudioPolicyManager: setAppState(uid:10552, state:2)
2021-02-17 18:24:25.427 30465-30481/? V/APM_AudioPolicyManager: setAppState(uid:10552, state:2)
2021-02-17 18:24:25.428 30465-30481/? V/APM_AudioPolicyManager: setAppState(uid:10552, state:1)
2021-02-17 18:24:25.428 30465-30481/? V/APM_AudioPolicyManager: setAppState(uid:10552, state:1)

minimized the app and after 1 minute:
2021-02-17 18:25:25.430 30465-30481/? V/APM_AudioPolicyManager: setAppState(uid:10552, state:0)
2021-02-17 18:25:25.436 30465-30481/? V/APM_AudioPolicyManager: setAppState(uid:10552, state:0)
2021-02-17 18:25:25.437 1267-1645/? V/AudioService.RecordingActivityMonitor: updateSoundAppPolicy packageName = com.testonly.package
2021-02-17 18:25:25.440 14736-14844/? I/A: Mic not exclusive, user is in voice call or video recording.

As you can see the state is set to 0? I'm not sure what that means. But that's the reason why the audio cannot be heard anymore.   It seems like the microphone of the device is being muted?
And as soon as I put the app again in the foreground, I'm receiving this logs: As you can see it is setting it to state: 2 again. The audio works now at this point.
2021-02-17 18:28:43.200 30465-30480/? V/APM_AudioPolicyManager: setAppState(uid:10552, state:0)
2021-02-17 18:28:43.200 30465-30480/? V/APM_AudioPolicyManager: setAppState(uid:10552, state:0)
2021-02-17 18:28:43.200 30465-30480/? V/APM_AudioPolicyManager: setAppState(uid:10552, state:2)
2021-02-17 18:28:43.203 30465-30480/? V/APM_AudioPolicyManager: setAppState(uid:10552, state:2)
2021-02-17 18:28:43.682 30465-30481/? V/APM_AudioPolicyManager: setAppState(uid:10552, state:2)
2021-02-17 18:28:43.683 30465-30481/? V/APM_AudioPolicyManager: setAppState(uid:10552, state:2)

Device: Samsung Galaxy A31 - Android 10
Opentok version:  2.18.1,  also tried 2.15.3 and still got the same issue.

any update on this? I think i should use foreground service?
I've researched and found the values for appState.
https://android.googlesource.com/platform/system/media/+/master/audio/include/system/audio_policy.h
typedef enum {
   APP_STATE_IDLE = 0,   /* client is idle: cannot capture */
  APP_STATE_FOREGROUND, /* client has a foreground service: can capture */
  APP_STATE_TOP, /* client has a visible UI: can capture and select use case */
  } app_state_t;

We are taking a look at this @marcqtan

@marcqtan, apologies for the belated reply.
This isn't a bug.  When an application is minimized, there's privacy concerns if it continues to capture audio.
cc'ing @kapilmishr, @Daniel085, since this is a product question.


[improvement] send merged runtime filter asynchrously and improve runtime filter waiting

Proposed changes
Issue Number: close #xxx
Problem Summary:

Send merged runtime filter asynchrously to improve effeciency.
For tpch query 3, using 100G data, 1Fe, 3BEs, parallel_fragment_exec_instance_num = 8, time for sending merged RF improve from 129 ms to 50 ms.
sql:

    l_orderkey,
    sum(l_extendedprice * (1 - l_discount)) as revenue,
    o_orderdate,
    o_shippriority
from
    (
        select l_orderkey, l_extendedprice, l_discount, o_orderdate, o_shippriority, o_custkey from
        lineitem join orders
        where l_orderkey = o_orderkey
        and o_orderdate < date '1995-03-15'
        and l_shipdate > date '1995-03-15'
    ) t1 join customer c [broadcast]  on c.c_custkey = t1.o_custkey
    where c_mktsegment = 'BUILDING'
group by
    l_orderkey,
    o_orderdate,
    o_shippriority
order by
    revenue desc,
    o_orderdate
limit 10;

Waiting runtime filters for at most 1 seconds even for multiple runtime filters.

Checklist(Required)

Does it affect the original behavior: (Yes/No/I Don't know)
Has unit tests been added: (Yes/No/No Need)
Has document been added or modified: (Yes/No/No Need)
Does it need to update dependencies: (Yes/No)
Are there any changes that cannot be rolled back: (Yes/No)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

LGTM

Hi @jacktengg , please rebase to make p0 test happy.

Please add ssb 100g performance test. I am afraid there is performance loss.

@jacktengg
hi, why removed the logic that multiple filters only wait 1s at most. Is there a performance loss in the test?
This is a TODO for the first implementation of the runtime filter. Currently, ScanNode will wait at most n seconds, and n is equal to the number of runtime filters, which will cause serious performance loss in extreme cases.

@jacktengg hi, why removed the logic that multiple filters only wait 1s at most. Is there a performance loss in the test?
This is a TODO for the first implementation of the runtime filter. Currently, ScanNode will wait at most n seconds, and n is equal to the number of runtime filters, which will cause serious performance loss in extreme cases.

No enough test have been done for the change of 'multiple filters only wait 1s at most', so  the performance impact is not sure for mutiple runtime filters in on ScanNode.
E.g., waiting a RF for 1.5 seconds and it can filter a lot of rows, VS waiting for 1 seconds and the RF is missed, I have not tested this situation, not sure which is better.
So in this PR will not include the change of RF waiting.

@jacktengg hi, why removed the logic that multiple filters only wait 1s at most. Is there a performance loss in the test?
This is a TODO for the first implementation of the runtime filter. Currently, ScanNode will wait at most n seconds, and n is equal to the number of runtime filters, which will cause serious performance loss in extreme cases.

No enough test have been done for the change of 'multiple filters only wait 1s at most', so the performance impact is not sure for mutiple runtime filters in on ScanNode.
E.g., waiting a RF for 1.5 seconds and it can filter a lot of rows, VS waiting for 1 seconds and the RF is missed, I have not tested this situation, not sure which is better.
So in this PR will not include the change of RF waiting.

If waiting for 1 second causes RF to be missed, runtime_filter_wait_time_ms should be increased.
Otherwise it will only wait 1.5 seconds if ScanNodeapply is greater than 1 RF, which is uncontrollable.
Number of RFs is not directly related to Waiting time per RF.
I think 'multiple filters only wait 1s at most' is a viable optimization, maybe do some testing.


Suggestion: Add a way to set slide cover images

Hey @jdan, thanks for writing Cleaver!
This PR is a suggestion more than a request to merge code. I wanted to add background cover images to some of my slides, so I did.
To add a cover image to a slide, just add <!-- background: http://example.com/batmobile.jpg --> to the body of any slide. It will generate the background CSS to set the background image for each individual slide. It will set the CSS background size for all slides to cover.
Let me know what you think. I'd be happy to re-implement this in a different way if you like the idea but not the code.

Hey @mplewis, thanks for the PR.
I'm not quite sure I understand the need for this - since one can pretty easily add a background image by bringing in some CSS.
I'm hesitant to start generating CSS from the original markdown, because that may set an unfortunate precedent where folks also want things like <!-- font-size: --> or <!-- color -->. What do you think?

When I built my last presentation with Cleaver, I used lots of cover images. I could have put the URLs in CSS and written
#slide-1: {
  background-image: ...;
}

I felt this decoupled my cover image data from my slides. And if I changed the order or number of slides, then my cover image locations changed to different slides too.
I agree though: using HTML comments is messy. I like the idea of adding slide metadata, but I don't think that's the solution. YAML front matter seems like a neat way to go, but I don't know what the syntax for that would look like on a slide by slide basis.

btw: This was inspired by Shower, an HTML5 presentation engine which supports cover images.

You can also annotate slides like so :)
-- class1 class2

### Some slide

...

And then in your css:
.class1 {
   /* ... */
}

That's a good idea, thanks! I'll try that next time.

Let me know if it's super confusing, perhaps we could improve it.

I think the only thing I would suggest is adding an example to the readme showing how you can add a cover image. Maybe add a presentation that uses a cover image too, to show off how that might work.

Mhm it's in the readme :)

#slide-N (for example, #slide-3) allows you to identify a particular full-bleed slide by its position in the slideshow. It extends to the bounds of the page.

http://jdan.github.io/cleaver/#4 shows this off.

Gaaaah I feel really bad now for totally missing that! Thanks again.
On Thu, Jun 11, 2015 at 4:53 PM Jordan Scales<EMAIL_ADDRESS>wrote:

Mhm it's in the readme https://github.com/jdan/cleaver#markup :)
#slide-N (for example, #slide-3) allows you to identify a particular
full-bleed slide by its position in the slideshow. It extends to the bounds
of the page.
http://jdan.github.io/cleaver/#4 shows this off.
—
Reply to this email directly or view it on GitHub
https://github.com/jdan/cleaver/pull/119#issuecomment-111286605.


Add pasteur integration to README

Hey @jberg, thanks for an awesome library! I used it to build pasteur.cc, a tool for rendering visualizations to mp4 video files with audio.

Awesome!!! great job @markneub , so cool!


Extensions: Hide Workspace Thumbnails

The reason for workspace thumbnails is in case users feel "lost" without some more symbolic representation. At the same time, the GNOME Design team is exploring a different way to more subtly give that representation. In the meantime, it may be worth trying to get rid of this sort of sometimes-visible duplication of the workspaces.
https://extensions.gnome.org/extension/5275/hide-workspace-thumbnails/
Maintained by my colleague @dylanmccall 😄

One could also move windows to different workspaces dragging them on those thumbnails.


fix/chat: Improve performance in streaming responses in long chats

Webviews' RPC system registers "a lot" (in a long chat, hundreds) of 'message' event handlers. Each of these typically causes the following work:

Deliver the message, including creating MessageEvent.data. For chat transcripts this can be a large object.
Walk the whole event data, testing whether each value is a URI, and if so, instantiating a URI for it.
Examine a field, like type or RPC stream ID, and decide to discard the message because it is for a different subscriber.

This work is dispatched synchronously to the hundreds of listeners on the webview main thread, causing us to have long pauses where we don't produce a frame. In extreme cases we can hang the renderer enough for VSCode to trigger a webview reload, (...but our webview reloading may not be robust because we do not use retainContextWhenHidden) leaving users with a blank sidebar/chat panel.
This change makes the hydrate message-check one property-discard steps extremely cheap by using a Proxy to lazily do the hydration. The typical case will allocate two proxies, cache one in a WeakMap, and return the primitive string value for the message type or RPC ID by reading through the proxy to the underlying message... effectively O(1) instead of O(message size). All of the objects in this fast path are short lived so should be ~free to collect.
This change does not address the root cause of too many event listeners. That is recommended follow-up work.
Before and after videos demonstrating the effect of this simple change:
https://www.loom.com/share/dd37421120114f8f805a97ff0de7f366
https://www.loom.com/share/3ca61e9ab493400884ba44e49e55dbf0
Test plan
This is a performance change, there should be no functional difference in behavior.
To verify performance improved:

Run the extension
Load your longest chat... dozens of messages and type a new chat query with a long response like write a Java bytecode verifier in Rust
Cmd-Shift-P, Developer: Open Webview Developer Tools
Performance tab, start recording
Back in VSCode, submit the chat. Verify that the response streams smoothly.
Back in Webview developer tools, stop the profiler. Verify that there are no (or few) dropped frames.
Lastly, verify with a heap snapshot that the heap has not regressed.

Profile before this change: Note the chat response takes ~52s with ~46s of script time and there are many slow (red) tasks in excess of 100-200ms each:

Profile after this change: Note that the total time is about 13s with 7s of script time and we only had four slow tasks (red marks) of about 50ms each. The extra rendering time is good thing, reflecting we produced a lot more frames much more quickly = smoothness. Note the CPU utilization (top line) is no longer pegged and spilling onto a second core.

Note, with a long enough chat and slow enough machine, this will lag again.
Changelog

Chat response streaming is much smoother, especially for long chats.

It looks like this breaks @-mentions with line ranges. I changed the behavior of the hydrate function a bit, so I will look into that... I may need to update a caller.

Expect CI to cycle green, please take a look.
If this is too complicated, another simpler solution is to make onMessage take a fast filter and a slow callback. It makes like tedious for callers because they need to duplicate the set of objects they handle in their callback's switch (message.type) ..., and it might be slower for big messages which we only use subsets of. But with some API craft we can probably make something palatable, if this is too complex.


Network Trace - Info popup does not display contents in IE

@MikeMillerGIS
Steps to reproduce:

Run trace
Click on any result feature added to map to display info-popup. Notice it displays contents correctly.
Click any row in widget's result panel to highlight it on map
Click the same feature you clicked in step 2. Notice the info-popup opens, but the contents are empty.
This is reproducible in all IE versions.

@MikeMillerGIS
After investigating this issue, we could not find any resolution for it. We have opened a thread on geonet for suggestions/resolutions. Please let us know if you have any suggestions or inputs.
We will not be able to include a fix for this in the upcoming 6/4 delivery.


the trustAnchors parameter must be non-empty

When I compile with docker，Execute the ./build.sh command directly in the project directory，I'm having trouble downloading the pom。
Plugin org.apache.maven.plugins:maven-checkstyle-plugin:3.1.1 or one of its dependencies could not be resolved: Failed to read artifact descriptor for org.apache.maven.plugins:maven-checkstyle-plugin:jar:3.1.1: Could not transfer artifact org.apache.maven.plugins:maven-checkstyle-plugin:pom:3.1.1 from/to spring-plugins (https://repo.spring.io/plugins-release/): Transfer failed for https://repo.spring.io/plugins-release/org/apache/maven/plugins/maven-checkstyle-plugin/3.1.1/maven-checkstyle-plugin-3.1.1.pom: java.lang.RuntimeException: Unexpected error: java.security.InvalidAlgorithmParameterException: the trustAnchors parameter must be non-empty -> [Help 1]
I tried to use Alibaba Cloud's image and configure the proxy, but I couldn't solve it.How to solve this problem?

cd fe mvn clean package -Dmaven.wagon.http.ssl.insecure=true -Dmaven.wagon.http.ssl.allowall=true


chore(project): fix formatting in README

Signed-off-by: Xavier Coulon<EMAIL_ADDRESS>

replaced by #28


Implement numpy func via numba parfor

We need to investigate how to implement functions via parfor.

@1e-to can you add more details to the ticket?

I think it is related to dpnp integration and is obsolete.


external volumes: UI shows volume attached even though app is suspended and no tasks are active

/cc @Poltergeist

After a quick chat with @kolloch we came to the conclusion that in fact the UI should not make any assumption about the attached/detached state of an external volume.  The correct fix for this is to drop that column entirely from the UI at this point: to actually display such info the API must include some changes that depend on Mesos. We probably need a separate issue for that.
cc @air

sounds good to me
On Mon, Apr 11, 2016 at 11:51 AM, Pierluigi Cau<EMAIL_ADDRESS>wrote:

After a quick chat with @kolloch https://github.com/kolloch we came to
the conclusion that in fact the UI should not make any assumption about the
attached/detached state of an external volume. The correct fix for this is
to drop that column entirely from the UI at this point: to actually display
such info the API must include some changes that depend on Mesos. We
probably need a separate issue for that.
cc @air https://github.com/air
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/mesosphere/marathon/issues/3724#issuecomment-208414074


[3.0.0-rc1] "Query: Could not save query" when saving query in web UI (though query is saved)

When saving a query via the web UI, a popup appears at screen bottom, "Query: Could not save query" (even though the query is actually saved). Also get a "Query: Could not delete query" when deleting a saved query (even though it is actually deleted).

I'm getting this same error, or at least a very similar one.
When I go to the query list after saving a query, it sometimes shows the query twice.
HOWEVER when reloading the page, all the queries disappear, so the query does not appear to have been saved permanently to the database.

Thanks. Also this was related to #1872 . Both fixed.


Cannot find module 'events' on browser

On version 1.8.1, loading on browser got error
Uncaught Error: Cannot find module 'events'

which is traced back to EventEmitter.js
if (process.env.PARSE_BUILD === 'react-native') {
  module.exports = require('EventEmitter');
} else {
  module.exports = require('events').EventEmitter;
}

It is a new build configuration settings, or an error ?

@lucianmat the code on master has the fix for this issue. https://github.com/ParsePlatform/Parse-SDK-JS/pull/240 You can use 1.7.1 in the meantime. We'll create a release once we've resolved #234.

resolved in 1.8.2


Highly Privileged Roles

How come the Teams Admin role and Application Admin role are not considered to be Highly Privileged, (Like SharePoint and Exchange)?

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

ID: 826a3309-03a5-60b4-aa4c-63f385150455
Version Independent ID: 1d2e2f1c-2805-5cda-a4ad-76fedbc3f525
Content: Best practices for secure admin access - Azure Active Directory
Content Source: articles/active-directory/users-groups-roles/directory-admin-roles-secure.md
Service: active-directory
Sub-service: users-groups-roles
GitHub Login: @curtand
Microsoft Alias: curtand

@DeanGross Thanks for your Feedback! We will investigate this and get back to you as soon as possible,

@DeanGross thanks for providing your feedback again, I don't believe this is correct. As essentially any admin role is considered to be a high privileged role.
I'm looking into this and will make the changes to the docs accordingly. Thanks for the help!

@DeanGross I published this content for another team. I'll contact the author to see if there are more roles included.

Hi @DeanGross The author seems to have left the company. The message of the article is really "secure the roles that are important to you." The designation of those few as highly privileged roles  is an example, not a statement that those roles are treated differently in some way.
#please-close


Error in maturity scale SMSF

There is an error in the SMSF maturity scale in vocabs, due to an error in the WK report. Karen from WGBIOP asks if we can fix this.
See below.
Hi Joana,
I’m contacting you as chair of WGBIOP regarding a maturity scale question.
We found an issue with the maturity stage in the SMSF maturity scale. Just a short resume of what happened:
The SMSF scale is the conversion of the WKMATCH scale, the only difference being the coding is no longer in digits (1-6), but in letters (A-F).
In the WKMATCH report, there was a mistake as there were two versions of the same table for the stages 3a and 3b:
Wrong version in WKMATCH report
Correct version in WKMATCH report
Of course, as always happens, the wrong version of this table was used for the conversion of the stages into the SMSF scale that became the official scale to report in the ICES databases.
The question is if this can be fixed in the vocab and the databases:
So the correct WKMATCH is:
3a :  spawning capable
3b : actively spawning
During WKSMSF this became (due to mistake in WKMATCH report, but now integrated as ICES vocab)
Ca : actively spawning
Cb : spawning capable
So we would like to correct in the ICES vocab (so that WKMATCH and WKASMSF scale have the correct conversion) to:
Ca : spawning capable
Cb : actively spawning
We checked in Datras, Acoustic and RDBES and it seems that those substages are used in several countries, but not too much (DK, GB-SCT, LT, LV, SE, FO). So it would also involve changes in the databases.
Is it an option to make those corrections in the vocab and afterwards in the databases?

@pcrjoana to ask whether there is a reason to suspect that data have been submitted using 'switched' descriptions.
We recommend that the report is amended.

@pcrjoana WKMATCH scale was proposed in 2012, but was revised and well-documented by WKASMSF in 2018
https://ices-library.figshare.com/articles/report/WKASMSF_-_Report_of_the_Workshop_for_Advancing_Sexual_Maturity_Staging_in_Fish_WKASMSF_/19212915
Table 4.1 in the report shows the revised scale from 2018 recommended for use, also implemented in ICES vocabs

@pcrjoana to share correspondence with @Osanna123


[WIP] Update aws distro approvers in owners

The currently assigned AWS approvers no longer work on the AWS distro. This PR updates to people working on the project and docs.
/cc @goswamig

I will have approve once you got membership

/close
some of our team members got access as part of #3125. Lets open a new PR with updated list if we still need access


Update ModLinks.xml

Tried to add new mod, Stop Hitting Yourself. I believe everything should be okay, as I tried looking at other mods' xml manifests, but I had not done this before, so there could still be errors.

It seems that your repo doesn't contain the source code but just a build of the mod - typically people upload the source code in the repository and make a release which contains the dll (or, more often, a zip containing the dll and pdb, which means that bug reports are more helpful)

Source code added (i hope that's all of it).
Makes sense mods should be open source.
thanks for the help


Add owners for pipeline tasks

Is your feature request related to a problem? Please describe.
We need to account Airflow tasks, but it's possible to define owners only for DAGs (ingested as pipelines to OM)
Describe the solution you'd like
Add ownership to pipeline tasks.
Describe alternatives you've considered
None.

@harshach for this we'll need to think how we want to show that info on the UI, since tasks for now are just a list view in the pipeline details

we will show it as another column under the list view

UI pending @chirag-madlani


CI: Also test and lint Metal (MacOS/iOS) code

This all is really terrible as metal-rs is depending on ancient unmaintained mac crates that haven't been touched over four years.
Really hoping the upstream migration to objc2 is being pushed through sooner than later.


Update README.md

Fixed some grammatical errors in README.md

Closing this as the readme seems fine without these changes.


No CMAKE_CUDA_COMPILER could be found

Note: Providing complete information in the most concise form is the best way to get help. This issue template serves as the checklist for essential information to most of the technical issues and bug reports. For non-technical issues and feature requests, feel free to present the information in what you believe is the best form.
For Q & A and discussion, please start a discussion thread at https://discuss.mxnet.io
Description
(Brief description of the problem in no more than 2 sentences.)
The CUDA compiler identification is unknown .
CMake Error at CMakeLists.txt:73 (project):  No CMAKE_CUDA_COMPILER could be found.
Environment info (Required)
win10+vs2015+cuda10.1+cmake3.15.0-rc3
gpu:rtx2080ti
D:\dev>nvcc -V
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2019 NVIDIA Corporation
Built on Wed_Apr_24_19:11:20_Pacific_Daylight_Time_2019
Cuda compilation tools, release 10.1, V10.1.168
What to do:
1. Download the diagnosis script from https://raw.githubusercontent.com/apache/incubator-mxnet/master/tools/diagnose.py
2. Run the script using `python diagnose.py` and paste its output here.

----------Python Info----------
Version      : 3.6.3
Compiler     : MSC v.1900 64 bit (AMD64)
Build        : ('default', 'Oct 15 2017 03:27:45')
Arch         : ('64bit', 'WindowsPE')
------------Pip Info-----------
Version      : 19.1.1
Directory    : C:\ProgramData\Anaconda3\lib\site-packages\pip
----------MXNet Info-----------
Hashtag not found. Not installed from pre-built package.
----------System Info----------
Platform     : Windows-10-10.0.17763-SP0
system       : Windows
node         : DESKTOP-LI615VE
release      : 10
version      : 10.0.17763
----------Hardware Info----------
machine      : AMD64
processor    : Intel64 Family 6 Model 79 Stepping 1, GenuineIntel
Name
Intel(R) Xeon(R) CPU E5-2640 v4 @ 2.40GHz
Intel(R) Xeon(R) CPU E5-2640 v4 @ 2.40GHz
----------Network Test----------
Setting timeout: 10
Timing for MXNet: https://github.com/apache/incubator-mxnet, DNS: 0.0600 sec, LOAD: 1.4430 sec.
Timing for Gluon Tutorial(en): http://gluon.mxnet.io, DNS: 0.3461 sec, LOAD: 2.7975 sec.
Timing for Gluon Tutorial(cn): https://zh.gluon.ai, DNS: 1.2440 sec, LOAD: 2.9702 sec.
Error open FashionMNIST: https://apache-mxnet.s3-accelerate.dualstack.amazonaws.com/gluon/dataset/fashion-mnist/train-labels-idx1-ubyte.gz, <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:777)>, DNS finished in 0.34665799140930176 sec.
Timing for PYPI: https://pypi.python.org/pypi/pip, DNS: 0.0030 sec, LOAD: 4.7989 sec.
Timing for Conda: https://repo.continuum.io/pkgs/free/, DNS: 0.0281 sec, LOAD: 2.9924 sec.
#########################################
Package used (Python/R/Scala/Julia):
(I'm using ...)
For Scala user, please provide:

Java version: (java -version)
Maven version: (mvn -version)
Scala runtime if applicable: (scala -version)

For R user, please provide R sessionInfo():
Build info (Required if built from source)
Compiler (gcc/clang/mingw/visual studio):
MXNet commit hash:
(Paste the output of git rev-parse HEAD here.)
CMakeError.log
CMakeOutput.log
Build config:
(Paste the content of config.mk, or the build command.)
Error Message:
(Paste the complete error message, including stack trace.)
Selecting Windows SDK version 10.0.14393.0 to target Windows 10.0.17763.
The C compiler identification is MSVC 19.0.24215.1
The CXX compiler identification is MSVC 19.0.24215.1
Check for working C compiler: D:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/cl.exe
Check for working C compiler: D:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/cl.exe -- works
Detecting C compiler ABI info
Detecting C compiler ABI info - done
Detecting C compile features
Detecting C compile features - done
Check for working CXX compiler: D:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/cl.exe
Check for working CXX compiler: D:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/cl.exe -- works
Detecting CXX compiler ABI info
Detecting CXX compiler ABI info - done
Detecting CXX compile features
Detecting CXX compile features - done
CMAKE_CROSSCOMPILING FALSE
CMAKE_HOST_SYSTEM_PROCESSOR AMD64
CMAKE_SYSTEM_PROCESSOR AMD64
CMAKE_SYSTEM_NAME Windows
CMake version '3.15.0-rc3' using generator 'Visual Studio 14 2015'
The CUDA compiler identification is unknown
CMake Error at CMakeLists.txt:73 (project):
No CMAKE_CUDA_COMPILER could be found.
Minimum reproducible example
(If you are using your own code, please provide a short script that reproduces the error. Otherwise, please provide link to the existing example.)
Steps to reproduce
(Paste the commands you ran that produced the error.)

What have you tried to solve it?

@mxnet-label-bot add [build, cmake, cuda]

I got the same error

tried various CMake versions, the same error

@SpaceView   on ubuntu, I add
set(CMAKE_CUDA_COMPILER "/usr/local/cuda-9.0/bin/nvcc")

solved this pro ！ thanks

yes nvcc should be in the path seems.

with cuda 9.2 as in the CI environment this is not a problem as nvcc is added to the path.

Let's add a note about CMAKE_CUDA_COMPILER to the install instructions.

May I ask whether can I not use CUDA, if I only build a cpu mxnet version?

May I ask whether can I not use CUDA, if I only build a cpu mxnet version?

Of course. If you compile with cmake, just set USE_CUDA=0

May I ask whether can I not use CUDA, if I only build a cpu mxnet version?

Of course. If you compile with cmake, just set USE_CUDA=0

Thank you


XVIZMetadataBuilder.log_info does not work as intended

According to spec https://avs.auto/#/xviz/api-reference/xviz-builder/xviz-metadata-builder?section=methods,
XVIZMetadataBuilder.log_info should allow custom field to be added to log_info. However this is constrained by protocol buffer definition as the LogInfo message can only have the start_time and end_time fields

Referring to the Python library

Did you solve this issue?

Using a hack. Add a dummy stream like /log_info/xyz. Place the value in the source field which accepts free text

Ok, thank you)

Sorry, this only works when using the JSON and the docs are not clear about that. I will keep this open until I update the docs.


add me as maintainer [ci skip] [skip ci]

@pmlandwehr I'd like to be added as maintainer if you agree

thanks @ocefpaf!

No problem. Thanks for taking care of this package!


Coder filters: add an exact string literal search (instead of substrings)

Sometimes we want to find short strings used in source code. In some cases, using queries like 'st' gtSubstringLiteralCaseSensitiveMatch produce too much noise. We therefore want to provide filters that compare whole literal strings.

Searching ST string
As a substring

As a string


⚠ WSINGRESOS - RECIBOPAGO has degraded performance

In c695880, WSINGRESOS - RECIBOPAGO (https://srvshyweb.yucatan.gob.mx/cgi-bin/wspd_cgi.sh/WService=wsingresos/internet/servicios/wsRecibosPagosWeb.r) experienced degraded performance:

HTTP code: 200
Response time: 9213 ms

Resolved: WSINGRESOS - RECIBOPAGO performance has improved in 7584af6.


Amend and reorganise troubleshooting section

While committing the FinnGen sumstat ingestion changes, I encountered a very annoying bug with my pre-commit configuration. It wouldn't let me commit anything, throwing around some obscure errors with JavaScript tracebacks.
After some digging, it turned out the issue was a conflict between a system-wide NodeJS version, and a version pre-commit actually wanted. I've researched ways to fix this, and added instructions to the documentation.
I've also amended instructions for Pyenv/Poetry reinstallation to be more complete.
Finally, seeing as we now have four troubleshooting entries, I've reorganised them to occupy a separate page, troubleshooting.md

Codecov Report

Merging #96 (6b1f319) into main (d3f2e1b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #96   +/-   ##
=======================================
  Coverage   80.39%   80.39%
=======================================
  Files          43       43
  Lines        1214     1214
=======================================
  Hits          976      976
  Misses        238      238


Auto insert inline background-image when clicks the input element

hi,
I implement a date picker with my custom calendar icon as background-image
but when I click the date-picker, I found the component will auto insert the inline styles to the <input> element,
then my custom icon will be replaced because the inline background-image is set to !important
Is there any way to prevent this problem? Thanks.

Sorry, I found out the problem is one of my chrome extension (something for English grammar check),I close this issue now.


FancyZones for secondary window failure

Microsoft PowerToys version
v0.55.2
Running as admin

[X] Yes

Area(s) with issue?
FancyZones Editor
Steps to reproduce
FancyZones only work on the main window of the application, but not on the secondary window of the application, such as Cinema 4D or Houdini
✔ Expected Behavior
FancyZones is expected to apply to all Windows on the desktop
❌ Actual Behavior
FancyZones only work on the main window of the application, but not on the secondary window of the application, such as Cinema 4D or Houdini
Other Software
Cinema 4D，Houdini。。。

I suppose it's a duplicate of https://github.com/microsoft/PowerToys/issues/15426.

@Hi~ @SeraphimaZ
Thanks for your reply.
I think I didn't make it clear, so I tried to make a video to illustrate it, and hopefully it makes sense.
https://user-images.githubusercontent.com/31401503/154416662-18deaab0-8ad1-46f1-be49-fd2073410db7.mp4

Thank you for the video! Your issue is related to that one I mentioned. We'll add an option that allows snapping child windows, so your problem will be fixed as well.

That's great. Thank you so much.
stay young and beautiful forever to you.
Seraphima Zykova @.***> 于2022年2月17日周四 17:59写道：

Thank you for the video! Your issue is related to that one I mentioned.
We'll add an option that allows snapping child windows, so your problem
will be fixed as well.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/16385#issuecomment-1042770046,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHPSMH2RYZ3S2VN6VRT6CATU3TBGJANCNFSM5OQJ3WJQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

That's great. Thank you so much. ❤
Seraphima Zykova @.***> 于2022年2月17日周四 17:59写道：

Thank you for the video! Your issue is related to that one I mentioned.
We'll add an option that allows snapping child windows, so your problem
will be fixed as well.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/16385#issuecomment-1042770046,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHPSMH2RYZ3S2VN6VRT6CATU3TBGJANCNFSM5OQJ3WJQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>


Use -fconcepts-diagnostics-depth only when compiling C++ files with GCC

This allows compilation with GCC as the host compiler and something else that doesn't support -fconcepts-diagnostics-depth as the device compiler. For example clang (and hip-clang) does not support the flag.

/ok to test


Add more applications with new Analyzer test

Update tests for new analyzer and adding few more sample applications.
Related to https://github.com/konveyor/go-konveyor-tests/issues/13

@dymurray Updated this application analysis PR creating 3 stages (more in README) and fixed analysis test for stage0 (executed by default in this PR main workflow). This covers applications and Pathfinder example that tests very basic flow, but that should be green now.

There are some TODOs that can be addressed in next PRs. An example of analysis test case for Tomcat application: https://github.com/konveyor/go-konveyor-tests/pull/15/files#diff-b2d6bbd29a1f86e4e159b33f90255df8a9857992c8176affe65d3c269b4dd621

Passes locally, but API dies on github :-/


NoneType' object is not subscriptable

Can somebody help me out here please, The workflow was working fine i literarily created images 5 munities right before this error ( i haven't close computer, or even the browser page, if im not wrong and reading the error message correctly the issue is from the refiner checkpoint, and maybe it got corrupted but i replaced it with another copy and still the issue remains, what should i do ?

Same for me


[InstagramBridge] Fix incorrect cache timeout calculation

It is expected that getCacheTimeout returns integer. In fact
it returned boolean value which lead to situation, where Instagram feeds
were not cached.

So it always returned true which probably is cast to 1 further up. Nice catch.


date conversion issue

The tool did not convert the date correctly; format was (mdy); so had to use another tool to fix

Yes, it's been notices. Haven't have time to correct that...


AVRO-2230 Assignable line_ in Entity

Fixes AVRO-2230 by making the line_ assignable. The line_ in JsonDom.hh is only used for reporting the line an error occurs on (AVRO-2113), and does not need to be const.

Fixed in #326


Give user option to disable the configure TUI during installation

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

[X] Feature

Have you discussed this change with the InvokeAI team?

[X] Yes
[ ] No, because:

Have you updated all relevant documentation?

[X] No - this should go into release notes.

Description
During installation, the installer will now ask the user whether they wish to perform a manual or automatic configuration of invokeai. If they choose automatic (the default), then the install is performed without running the TUI of the invokeai-configure script. Otherwise the console-based interface is activated as usual.
This script also bumps up the default model RAM cache size to 7.5, which improves performance on SDXL models.

https://github.com/invoke-ai/InvokeAI/pull/4675
to fix style checks

I fear configuring a static cache amount / should we not be basing this off the amount of RAM the user has in some fashion?

Hmm yeah maybe we should default to no cache at all? It may be easier to handle the initial install automatically, but then direct users to consider updating their cache sizes in the yaml.

I’d rather select the RAM cache based off of total RAM than disable the cache completely. Without cache users will have miserable performance; every model will be loaded from disk before each generation. Currently, the code makes the RAM cache 7.5 GB, which is large enough to hold a single SDXL model. Typical RAM is 16-32 GB, but I suppose there could be some users with only 8. How about the following table of defaults?
RAM  (GB)       CACHE SIZE
——————         —————
8                 0
16                4
32               7.5
64               15

Yeah, that makes sense.
Regarding the default values - I think it's fairly safe to assume that if a user is loading up invokeai to generate, they likely want to use their system to the fullest.
I'd be inclined to bump the RAM cache defaults to half the system RAM.

Yeah, that makes sense.
Regarding the default values - I think it's fairly safe to assume that if a user is loading up invokeai to generate, they likely want to use their system to the fullest.
I'd be inclined to bump the RAM cache defaults to half the system RAM.

I think that's too aggressive. Without any models loaded the web server uses nearly 2GB of RAM. So add 4 GB of RAM cache to that and you've only got 2 GB of free memory left on a 8G machine. On the other end, I've got a 64 GB machine. Do I really want 32 GB going to Invoke?

Latest commit uses this logic:
    return (
        15.0 if MAX_RAM >= 60 else 7.5 if MAX_RAM >= 30 else 4.0 if MAX_RAM >= 14 else 2.1
    )

On my system, at least, psutil.virtual_memory().total is not returning slightly lower numbers than expected for the installed RAM in the system, so I have chosen slightly lower threshold values. The lowest value, 2.1, is enough to hold SD-1.5 without reloading.

This seems fair


Remove default ENV to allow user provide environment variables via env files

NextJS app allow user define environment variables by providing a .env.local file in /app/.
Those related lines in the Dockerfile are setting values to the environment variables, which will be passed in as process.env, which ranked as the top one in the loading order
This PR is removing these default variables so that user can just mount a .env.local file to set environment variable, which can avoid include the sensitive information in the docker-compose file.

@ileodo I'm assuming you are going to mount .env.local file inside the docker container so that NextJS would load the env by default?

any chance to get an approval on this? thx


short notes break the browser

https://jsfiddle.net/037sn9yb/
This is an occasionally appearing issue, possibly related to performance.
Please tweak the start('+0.1') to 0.01 0.2 back and forth, or tweak some params of the synth.
you will get "Aw,Snap" from Chrome.
I have tried multiple times, it will come out...

Seems to be an issue with chrome's implementation of cancelAndHoldAtTime. If i set AudioParam.prototype.cancelAndHoldAtTime = false, Tone.js will use it's fallback and i no longer get the crash. example.
Would be good to file a bug with chromium if you can distill it to a minimal example

superb

gonna leave this open to remind me to push a fix


Component Upgrades

Core - Full Integration Build 5151 outcome was FAILURE using a merge of 0270bb412b355fc91112deddde67551dcd116e8d
Summary: Tests passed: 3879, ignored: 130; exit code 137 (new) Build time: 00:52:06


create classes for each bitmap type

1-, 4-, 8-, 16-, 24- and 32- bits per pixel bitmaps.
Create them into a dedicated directory bitmaps.
Will be a copy of code already coded in BMPLoader classes.
Will eventually lead to the refactoring of BMPLoader classes.

Completed. Tested.


Handle
 and  block preserving HTML

Fixes #178

Ideally this should go into https://github.com/mixmark-io/turndown but it looks like unmaintained recently... oh well.

Task linked: CU-384hf76 [PR] Handle
 and  block preserving HTML

I'm not really a fan of modifying the turndown.js file.
Mainly because if and when I want to update turndown to the latest commit then it's going to be a pain to copy over and reapply the changes.
Also, because in the past I've been asked (by the Firefox add-on team IIRC) to link to the exact version of turndown used, so they can compare and make sure I didn't add anything nefarious in there sneakily.
So I'd rather just not touch it.
I did some experimentation locally, and you should be able to just add a new rule to the turndown service for the fenced code blocks and it will override whatever the default is, so you should be able to move the whole thing into the background.js. The only issue I had was the call to repeat which is easily just replaced with the implementation (i.e. Array(fenceSize + 1).join(fenceChar))

The other thing is, do you have specific sites you're testing on?
I copied the code from #178 and threw it in a temporary HTML file for testing, and even then I noticed that the second case (
strong text
) was being replaced with
```
<strong>strong text</strong>
```

instead of using **

Another thing to note here is there is very similar code in the GFM plugin which checks for a div with a class that matches the regex here: https://github.com/deathau/markdownload/blob/1ac4eea1a379ea9348fec8a9f0d5874d39705261/src/background/turndown-plugin-gfm.js#L4 with a pre element inside it. Obviously that's GitHub-flavoured text highlighting it's matching there, but it's probably worth doing something similar for popuplar syntax highlighter syntax like prism.js, etc... 🤔

@deathau Restored turndown.js and moved the logic into background.js

The other thing is, do you have specific sites you're testing on? I copied the code from #178 and threw it in a temporary HTML file for testing, and even then I noticed that the second case (\
\strong text\\
) was being replaced with

strong text

instead of using **

In my opinion, the way it behaves is exactly how it meant to. Everything in
...
 should be used exactly as is, so we can use any HTML snippets inside. So expecting ** instead of  is not right, in my opinion

In my opinion, the way it behaves is exactly how it meant to. Everything in \
...\
 should be used exactly as is, so we can use any HTML snippets inside. So expecting ** instead of \ is not right, in my opinion

Fair, but the other option (
strong text
 does translate  to **, which seems inconsistent to me.

@deathau I guess the best solution would be to make
  escaping configurable in your plugin because all the approaches have their pros and cons


SyntaxError during installation

See http://stackoverflow.com/questions/27553414/pip-install-django-import-export-raises-syntaxerror
Apparently this has something to do with tablib and Python 2.x vs Python 3.x.
The readme https://github.com/bmihelac/django-import-export/blob/master/README.rst indicates that the requirements are "Python 2.7+ or Python 3.3+".  I would assume that If tablib doesn't support Python 2.7+, and django-import-export required tablib, then django-import-export should also not support Python 2.7.x?

I'm closing this, please reopen if needed.


oneOf models are rendered with incorrect member names [RustGenerator]

Describe the bug
Given the following schema, member names and serde macros are rendered with the index of the member appended to the model name.
    WebrtcSettingsApplyRequest:
      name: WebrtcSettingsApplyRequest
      type: object
      additionalProperties: false
      properties:
        video_src:
          x-parser-schema-id: VideoSource
          oneOf:
            - $ref: "#/components/schemas/Camera"
            - $ref: "#/components/schemas/PlaybackVideo"
      required:
        - video_src

The rust code currently rendered:
// VideoSource represents a union of types: Camera0, PlaybackVideo1
#[derive(Clone, Debug, Deserialize, Eq, Hash, Ord, PartialEq, PartialOrd, Serialize)]
pub enum VideoSource {
    #[serde(rename="Camera0")]
    Camera0(crate::Camera),
    #[serde(rename="PlaybackVideo1")]
    PlaybackVideo1(crate::PlaybackVideo),
}

Expected behavior
The schema above should render the following model:
// VideoSource represents a union of types: Camera, PlaybackVideo
#[derive(Clone, Debug, Deserialize, Eq, Hash, Ord, PartialEq, PartialOrd, Serialize)]
pub enum VideoSource {
    #[serde(rename="Camera")]
    Camera(crate::Camera),
    #[serde(rename="PlaybackVideo")]
    PlaybackVideo(crate::PlaybackVideo),
}

Dropping some debugging notes. Looks like ConstrainedUnionModel is getting an array of ConstrainedReferenceModel with the wrong types (Cat0, Dog1, StickInsect2).


DataType.of should be non case sensitive

Currently DataType.of only accepts uppercase names like FLOAT32.  Anyone used to using python will expect float32 and similar lowercase names to work.  It seems like it may have been a design decision by it's inclusion in the docs, but I don't see a reason not to add it (probably as a toUppercase on the arg rather than extra cases) and I expect almost everyone will run into this at some point.

This will likely be closed by #139

Hi @rnett , I was about to say so, SIG JVM have agreed to go forward with #139, which leverages even more Java idiomatic type features in preference to a custom or pythonic type system.
I suggest we close this issue so that you can continue to provide your valuable feedbacks on the next PRs to come that will be related to this

Agreed.


Performance difference between TCP and UDP servers

I observed a huge difference between TCP and UDP servers:
$ time dig @localhost google.com
real 0m1.037s

while
$ time dig +tcp @localhost google.com
real	0m0.040s

Looking at the code, I have found that m, s, e := reader.ReadUDP(l, rtimeout) inside Server.serveUDP is holding up the response. Added the following logs:
	glog.V(15).Info("serveUDP is ready")
	// deadline is not used here
	for {
		m, s, e := reader.ReadUDP(l, rtimeout)
		glog.V(15).Info("Out of ReadUDP")
		srv.lock.RLock()
		if !srv.started {
			srv.lock.RUnlock()
			return nil
		}
		srv.lock.RUnlock()
		if e != nil {
			continue
		}
		srv.inFlight.Add(1)
		glog.V(15).Infof("Got message %s:%s:%s", m, l, s)
		go srv.serve(s.RemoteAddr(), handler, m, l, s, nil)
	}

This shows serverUDP is ready is logged immediately as soon as the server is started and Got message is logged immediately after the last Out of ReadUDP. However reader.ReadUDP is always held up as long as the timeout.
However while reducing rtimeout shows the code going through the loop much faster, but the queries still take about a second to come back. Profiling the rest of the code after this point doesn't show any points causing this (everything else after Got message log go through in less than 20ms). Also nothing in the rest of the code is different between TCP and UDP.
This is very puzzling. Any thoughts as to why this could be?

To take out any of my custom code, I tried using https://github.com/kenshinx/godns and the results are the same. Using time dig @<IP_ADDRESS> +tcp google.com and time dig @<IP_ADDRESS> +notcp google.com doesn't make difference in performance.

Interesting observation... In the end this code calls: ReadFromSessionUDP, which calls ReadMsgUDP, is there a bug in there?
From the looks of it, it does not seem this code (i.e. what you reference) does anything whacky...

I tried to replicate this behaviour using go1.5.1 on a FreeBSD 10.2 VM and https://github.com/kenshinx/godns. I didn't really get any distinct result but that could be due to any number of things. Can you share some more details on your environment?

I think I managed to get to the bottom of this. This was a combination of multiple things on top of each other (some local and some systemic) that caused the issue:
Issue with localhost:

Using localhost with UDP was significantly slower than <IP_ADDRESS> as for TCP, performance was identical between localhost and <IP_ADDRESS>.
Using a different name in /etc/hosts for <IP_ADDRESS> instead of localhost yielded in identical performance between TCP and UDP.
Using an alias for <IP_ADDRESS> in /etc/hosts file doesn't affect the performance of queries. Both TCP and UDP work just as performant regardless of the IP or address.

This might be isolated to my test machine.
SingleInFlight flag:

Changing the SingleInFlight flag for UDP to false improved the performance. It however didn't make any significant difference with TCP.

Good summary, thanks... BTW, use of "localhost" on modern OS often results in using "::1", just something to recall next time we chat about this.


[RHDM-1512] Update EAP version from 7.3.4 on PAM/DM 7.10.x

See https://issues.redhat.com/browse/RHDM-1512
Thanks for submitting your Pull Request!
Please make sure your PR meets the following requirements:

[ ] Pull Request title is properly formatted: [RHDM-XYZ] Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket
[ ] Attached commits represent units of work and are properly formatted
[ ] You have read and agreed to the Developer Certificate of Origin (DCO) (see CONTRIBUTING.md)
[ ] Every commit contains Signed-off-by: Your Name<EMAIL_ADDRESS>- use git commit -s

@spolti

@Kaitou786 do you mind to verify why the bot didn't request the reviews and add the labels?

@Kaitou786 do you mind to verify why the bot didn't request the reviews and add the labels?

Not sure, but right now I see the bot is working as expected

@desmax74 before merge, can you please check the items on the PR description that applies to this change?
Sry for keeping asking it, but the example for other contributors needs to came from us :)