External Traffic is not flowing via EGRESS GATEWAY

Bug Description
Traffic from our micro service to external service is not flowing via egress gateway.
Our app will be running inside kubernetes cluster with istio-injection enabled. The microservice makes an external request to our host machine (The external service will be running on port 5020 of host machine) [Request by microservice: "http://<IP_ADDRESS>:5020/politics/latest" ], here <IP_ADDRESS> is my host system IP.
EGRESS_GATEWAY.txt
In kiali dashboard, The traffic to external service(ie, logs-svc) is not passing via egress gateway. It is directly flowing from microservice to logs-svc(external service). I need to forward that traffic to egress gateway.
Version
isitoctl version
client version : 1.12.1
control plane version : 1.12.1
data plane version: 1.12.1(5 proxies)
kubectl version --short
Client Version:v1.22.4
Server Version:v1.22.5+k3s1

Additional Information
No response
Affected product area

[ ] Docs

[ ] Installation

[ ] Networking

[ ] Performance and Scalability

[X] Extensions and Telemetry

[ ] Security

[ ] Test and Release

[ ] User Experience

[ ] Developer Infrastructure

[ ]

[ ]

[ ]

[ ] - [ ] Upgrade

[ ] Multi Cluster

[ ] Virtual Machine

[ ] Control Plane Revisions

Is this the right place to submit this?

[X] This is not a security vulnerability
[X] This is not a question about how to use Istio

The config is badly formatted. Please correct the format for easy checking.

Please verify the EGRESS_GATEWAY.txt file, which is formatted correctly.

The virtual service should match on port 5020 instead of 80 for mesh gateway.

can you please share any code format to deploy egress gateway which is running as an external service on rest API.

Sorry I don't follow your ask.

Yeah i've changed the port and deployed , even though the traffic is not flowing out from the egress gateway and it's showing some unkonwn. please refer to the screenshot below.
EGRESS_GATEWAY.txt

Envoy access log from the sidecar proxy and egress gateway will be more useful to understand what happened. https://istio.io/latest/docs/tasks/observability/logs/access-log/


Beacon needs to power on again to be detected

I am able to detect my beacons but only when i power on them. If beacon is already powered on then it does not detect it.
I need to manually power off the beacon and then power on.
I need to keep the beacon status updated every time. I am using Gimbal  Series 21 beacons.
Please let me know if you need more details to understand the problem.

@Gayatrideshmukh what code are you using for scanning. I think you will need to enable scanning with duplicates ...

Actually i allwed the duplicated but still it was not showing until i power on them.
I did more analysis and found that hcitool.lescan also not showing the device until i power on them.
I found another library 'node-bleacon' which is helping me. its detecting the devices which are already on with few more details like accuaracy , distance in meter etc. but the problm with this library is i am not able to unquely identify devices.
I tried to change in lib of node-bleacon by adding
bleacon.address = peripheral.address and so many things tried in lib also.
Can you please help me to identify the unique feature so that i can differentiate the devices ?
Also one more problem is 'Its not detecting series 10 beacon' . its working with series 11 beacons.
Thanks,
Gayatri

Following is the code using noble
var noble = require('noble');
noble.on('stateChange', function(state) {
if (state === 'poweredOn')
noble.startScanning([], true); // any service UUID, allow duplicates
else
noble.stopScanning();
});
noble.on('discover', function(peripheral) {
var macAddress = peripheral.uuid;
var rss = peripheral.rssi;
if(macAddress=='a4d856183313' || macAddress =='a4d85618e05e' || macAddress =='a4d8560c110a')
{
console.log('found device: ', macAddress, ' ', rss);
}
});
The problem  : need to power on to detect. hcitool.lescan also doing same.

Hi Sandeep,
By using noble , we are able to detect the beacons which are already powered on by chanining the code like below.
if (state === 'poweredOn')
{
noble.startScanning([], true); // any service UUID, allow duplicates
}
else
{
noble.on('stateChange', function(state) {
noble.startScanning([], true);
});
}
Now we are almost there but the problem is after few seconds device stop detecting.
Its not continuously detecting device. Is it something like device stops advertising , if yes then what we need to do to continuously detect it.

Hi ,
My issue is resolved by pushing using the Bleacon library.
By pushing the proper configuration by Gimbal manager , it helped me.
We set unique UUID, major , minor from Gimbal manager for the beacon.
We applied assigned configuration to those beacon.
after pushing those configuration from app store beacon manager, we were getting the uuid which we were specified.
Hope this help someone who will be reading this.
Thanks,
Gayatri

Closing this as resolved. @Gayatrideshmukh thanks for the updates.


Remove pycache from the repository

This is currently part of the repo, I think it should be removed and gitignored.

Similarly, I think doc/build should not be part of the repository.

I agree - and I would add a .gitignore such as the github default for python to avoid these kind of files being accidentally commited:
https://github.com/github/gitignore/blob/master/Python.gitignore

Similarly, I think doc/build should not be part of the repository.

I had thought to leave it in there, so users could directly check the sphinx-generated html page after cloning the repository without having to install the requirements.

Similarly, I think doc/build should not be part of the repository.

I had thought to leave it in there, so users could directly check the sphinx-generated html page after cloning the repository without having to install the requirements.

That makes sense, but in general it's a good idea to avoid putting build output / auto-generated files in the git repo as it tends to add noise to commits & pull requests: e.g. someone changes a couple of lines in one source file, builds the docs (with their different local version of sphinx), does git commit -a, and makes a pull request which then has hundreds of changed files.

I would also say that the value of the best practice keep build artifacts out of the repository outweighs the advantage of being able to look at the docs immediately.


fix macOS build

fixes #17

Thanks.  Unfortunately, this breaks the build on Linux:
../src/qt5_in_gtk.cpp:128:31: error: invalid cast from type ‘GdkNativeWindow’ {aka ‘unsigned int’} to type ‘WId’ {aka ‘long long unsigned int’}

Seems like the types differ significantly across platforms.  static_cast actually seems more sensible here to me (reinterpret_cast was just a guess, that's not appropriate for differently sized integers).  Haven't managed a Mac build yet, I can't figure out where/if the required Qt things (QMacCocoaViewContainer) are in homebrew...

Fixed (by instead reverting to C casts) in 2d18caf.


add info about arm64 repo

Added info about arm64 repo for Debian.

@arm64b ...

On the docker-ce/debian page, we provide docker run commands for x86_64 and armhf. Does arm64 have a unique command or is it the same as x86_64?

Hello @gbarr01 , on AArch64, if the image of hello-world has multi-arch support, we can docker run it just as x86_64 does: sudo docker run hello-world, if the hello-world image is non-multiarch, then we need use this one: sudo docker run arm64v8/hello-world. So the question is depended on the docker image itself: multi-arch or non-multiarch.  Does this answer make sense for you ❓

Hi, @arm64b, it makes perfect sense. Thanks so much.


error when import onnx of yolo8

I have one onnx file of trained from yolo8.  I want to use it in burn, as the backend.  Then i import the onnx file , and an error is throwed. The error like this:
 checking outputs       DEBUG burn_import::onnx::from_onnx: output name: /model.22/Shape_4_output_0       DEBUG burn_import::onnx::proto_conversion: Converting ONNX node with type "Concat"       DEBUG burn_import::onnx::from_onnx: renaming node "/model.22/Concat_3"       ERROR burn_import::logger: PANIC => panicked at /home/lin/.cargo/registry/src/mirrors.tuna.tsinghua.edu.cn-df7c3c540f42cdbd/burn-import-0.13.0/src/onnx/dim_inference.rs:178:10:   calledOption::unwrap()on aNone` value
--- stderr
thread 'main' panicked at /home/lin/.cargo/registry/src/mirrors.tuna.tsinghua.edu.cn-df7c3c540f42cdbd/burn-import-0.13.0/src/onnx/dim_inference.rs:178:10:
called Option::unwrap() on a None value`
This is a bug or not ,  i don't know.   So, anyone can help me , please. thanks .

@laggui    First, thanks your reply.
And the model is exported by yolo8 project of ultralytics.  So, can you try it when you have some time.
The yolo8  is so good , and many people use it.  Now, i use yolo8 model by tch-rs.
By export, and then import onnx file, it is a bridge from pytorch to burn.  It is a good way . And the other way is that writing the modle script again in burn from pytorch to burn , it is not good .
Most CNN models are trained by pytorch, So if you give a simple way from pytorch to burn, i think it must be excellent.

It would be great if you could link the onnx file so we could try out.

@antimora i am also facing a similar issue , not sure if i should create a new issue as it is already open.
this is the python code i use to generate onnx file
`
model_id = "distilbert-base-uncased-finetuned-sst-2-english"
feature = "sequence-classification"
model = AutoModelForSequenceClassification.from_pretrained(model_id)
tokenizer = AutoTokenizer.from_pretrained(model_id)
inputs = tokenizer("onnx is wonderful", return_tensors="pt")

torch.onnx.export(
model,
tuple(inputs.values()),
f="albert.onnx",
input_names=['input_ids', 'attention_mask'],
output_names=['logits'],
dynamic_axes={'input_ids': {0: 'batch_size', 1: 'sequence'},
'attention_mask': {0: 'batch_size', 1: 'sequence'},
'logits': {0: 'batch_size', 1: 'sequence'}},
do_constant_folding=True,
opset_version=16,
verbose=True
)
`
getting this error,
  ERROR burn_import::logger: PANIC => panicked at /home/kanak/.cargo/registry/src/index.crates.io-6f17d22bba15001f/burn-import-0.13.2/src/onnx/dim_inference.rs:281:40:   calledOption::unwrap()on aNonevalue
let me know if any other information i can provide(or if i should create a new issue), thanks

@antimora i am also facing a similar issue , not sure if i should create a new issue as this one is already open.
this is the python code i use to generate onnx file

model_id = "distilbert-base-uncased-finetuned-sst-2-english"

feature = "sequence-classification"

model = AutoModelForSequenceClassification.from_pretrained(model_id)

tokenizer = AutoTokenizer.from_pretrained(model_id)

inputs = tokenizer("onnx is wonderful", return_tensors="pt")

torch.onnx.export(

    model,

    tuple(inputs.values()),

    f="bert.onnx",

    input_names=['input_ids', 'attention_mask'],

    output_names=['logits'],

    dynamic_axes={'input_ids': {0: 'batch_size', 1: 'sequence'},

                  'attention_mask': {0: 'batch_size', 1: 'sequence'},

                  'logits': {0: 'batch_size', 1: 'sequence'}},

    do_constant_folding=True,

    opset_version=16,

    verbose=True

)

getting this error,

  DEBUG burn_import::onnx::from_onnx: input /distilbert/embeddings/Constant_3_output_0 matched constant node constant6

  ERROR burn_import::logger: PANIC => panicked at /home/kanak/.cargo/registry/src/index.crates.io-6f17d22bba15001f/burn-import-0.13.2/src/onnx/dim_inference.rs:281:40:

  called `Option::unwrap()` on a `None` value

let me know if any other information i can provide(or if i should create a new issue), thanks

It would be helpful if you file a new ticket.

i have created a new issue https://github.com/tracel-ai/burn/issues/1811


Fixed bug with confirm payment page completed other stories

Sophies changes and a fix to the confirm payment page.

lgtm :+1:


Running issue.

May I have the complete code with json file?
Now I can't run it under freqai.
By the way, I wish Israel win the war against Hamas.
World peace!

Hi,
I've been wanting to push an example for a while now.  I'll make sure share examples in the next couple of days!
Amen to that! ☮


Make sure we don't crash when RTL text is wrapped

What does the pull request do?

What is the current behavior?

What is the updated/expected behavior with this PR?

How was the solution implemented (if it's not obvious)?

Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Avaloniaui.net with user documentation

Breaking changes

Fixed issues

#4828

Checked with cibuild0011280, fixed #4828

#4848


Add timezone information to Telemetry data

What type of PR is this?
/kind feature
What does this PR do / why we need it:
$subject
Which issue(s) this PR fixes:
Fixes part of #4994
PR acceptance criteria:

[ ] Unit test

[ ] Integration test

[ ] Documentation

[ ] I have read the test guidelines

How to test changes / Special notes to the reviewer:
Segment data should contain the timezone information in the format "UTC +0530", "UTC 0000", etc.

/lgtm

/approve

/hold

/lgtm


Feature request: Inline virtual text for suggestions

Neovim 0.10 has been out for a while now, with support for virt_text_pos = "inline", i.e. virtual text which is visible in the middle of text, leaving the text in the buffer at its position visible instead of replacing it, in the style of LSP inlay hints.
It would be nice if copilot.lua used inline virtual text to display suggestions, as currently suggestions in the middle of a line obscure what remains on the line. This very often causes confusion around things like auto-pairs, e.g. in this example:

If I dismiss the suggestion, I can see that the opening bracket has been auto-paired, but I have no way to tell while the Copilot suggestion is active.

And if I accept the suggestion, it appears as though the closing bracket was added by Copilot, even though it didn't appear in the suggestion.

This is all much more noticeable if there is more than one remaining character on the line, e.g. when replacing the first argument of multiple in a function call, the suggestion may cover up the existing values for the following arguments.
Notes

In this example, when I did accept the suggestion, it actually moves my cursor after the closing bracket. Does that mean that the closing bracket is actually part of the suggestion and is being hidden, or automatically made to not conflict somehow?
Can inline virtual text have multiple lines?

Not having this is currently my top issue with using copilot. Having this would make the whole experience so much better


See if Minio throttling settings apply

I'm not sure if any of the Minio settings at https://github.com/minio/minio/tree/master/docs/throttle apply to Gateway-ST.  See if they matter / should be documented.

Closed by https://review.dev.storj.io/c/storj/gateway/+/6205.


Change modal width

Pull Request Test Coverage Report for Build 106

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 82.129%

Totals

Change from base Build 105:
0.0%

Covered Lines:
2561

Relevant Lines:
2924

💛  - Coveralls


Compatibility with JavaScript Modules API (RFC 176)

Hi! My code use JavaScript Modules API.
Sample code:
import EmberObject from '@ember/object';

And then I try to run tests with eslint-plugin-ember-best-practices, I get an exception:
TypeError: Cannot read property 'name' of undefined

First line from stack trace

Would you mind opening a PR to fix this issue? Or at least a reproduction would be helpful.


weex run ios后报错simList is not defined

here

have the same issue, adding let simList fixed
Environment:

weex -v   v1.1.0-beta.5
weexpack : v0.4.7-beta.11
weex-builder : v0.2.13-beta.2
weex-devtool : v0.3.2-beta.7
weex-previewer : v1.3.13-beta.5
node -v: v8.8.0
npm -v: 5.4.2

This issue is outdated for a long time and will be closed now. You can create a new one if you still have questions.


Doesn't work on hover w/ disabled elements

Hi,
Thanks for all the work, floating-vue works out great for us!
Context
I'm using Dropdown w/ a component that can be disabled, and I'm using hover as a trigger.
Need
I'd like to be able to use hover as a trigger, even with disabled inputs. Since disabled inputs don't fire mouseenter and mouseleave events, this is not possible right now.
Recommendation
I'd recommend listening to pointerenter and pointerleave events, as these are fired even when buttons are disabled.
POC
See here an example of my use-case, and a comparison between the pointer events, note how pointer events keep firing even after the button is disabled.

I still see the events on the disabled button:

Ouch. This may be platform dependent. I think there is not much to do about this (on the library level), given how inconsistently this is implemented across browsers.
I see this on MacOS 13.5.2 w/ Chrome Version 111.0.5563.64 (Official Build) (arm64) – fires pointer on disabled:

On the same platform w/ Safari – don't fire anything on disabled:

On the same platform w/ Firefox (117.0.1) – fires both on disabled:

I guess the best way is to wrap the input with a div


Feature auth support

module和component鉴权接口android和ios都已添加

@zshshr @cxfeng1 pls review this PR.
@168668660

It's 'Validate' not 'auth', change this PR's title.
Do not commit with chinese any longer, it's not meet our contribute requirement. Thanks. 😄

@168668660 pls fix the conflicts.
@cxfeng1 pls review the iOS part.

@168668660  what's going on?


Contextual Menu not available when using convert method (like mathml2chtml)

Issue Summary
When using the MathJax convert method, such as mathml2chtml to render Math content, the Contextual Menu that MathJax provides on "normally" typeset content is not available.
Is there a way to have the menu available when using the convert method?
Steps to Reproduce:
I made a codesandbox to display the issue: https://codesandbox.io/s/nervous-keldysh-sdnfqu?file=/src/index.js
Technical details:

MathJax Version: 3.2
Client OS: Manjaro Ruah 21.3.5
Browser: Vivaldi 5.3.2679.70

loading MathJax via
const script = document.createElement("script");
script.src =<EMAIL_ADDRESS>script.async = true;
document.head.appendChild(script);\

Your code sandbox example indicates that your are using that MathJax outputs outerHTML to set the innerHTML of its new location.  MathJax uses event listeners to implement its contextual menu (and other actions needing interaction), but outerHMTL does not retain those, so you are losing the event listeners at that step.  Using
    document.getElementById("math").appendChild(converted);

instead will preserve the menu code, and is more efficient than serializing and re-parsing the math in any case.

Ok, that make sense. Thank you!


309551087 lab1

↓↓↓ Notice ↓↓↓

[ ] Check your base repository and branch are belong to you.
[ ] The PR name need to include student ID and which lab assignment

template: ${student id} ${which lab}
e.g. 0756110 lab0

↑↑↑ Notice ↑↑↑

If you ensure the notice items is done, you can remove them.

Following section is optional, depend on you.

Description
a few message for overall goals of this pull request
Todos

[ ] what feature to do ?
[ ] what bug to fix ?
[ ] what enhancement to improve ?

@GrassLab/ta

Please submit your pull request to your own student id branch.
Close it for now.


Check duplicate lights on Spike by including node in address uniqueness

The recent update to check for duplicate light addresses triggered a false positive on Spike platforms, which use device.node plus device.number to identify a unique address. During the duplicate light check, a light with address 10-14 would fail as duplicating a light with address 8-14.
This PR checks for the presence of device.node and includes the value when creating the unique key for a light, thereby ensuring that lights of the same number on different nodes are not considered duplicates.

Good find. I think i want to solve this differently by including node + index in number (and probably separate variables for index and node).

Thanks for reporting this and proposing a fix.  That is really great! I hope I did not discourage you from contributing to MPF by modifying the approach slightly. This makes it a bit more universal in a sense that all platforms should behave according to their interface (and Spike was incorrect here).

Awesome, thanks for checking it out! You're the expert, I trust your approaches over mine :)


feat: made search bar accessible on homepage 🔍✨

Close #860
Description

Added proper  <label> element which has a "for" attribute that corresponds to the "id" attribute of the <input> element. This association is what makes the label functionally related to the input field.
When the user interacts with the label or its associated input, the browser handles the connection, ensuring proper focus and accessibility behavior.
By doing so, it helps users, including those with disabilities, better understand the purpose of the form control and interact with it more easily. The usage of <label> is crucial for all aspects of accessibility.
Additionally we make the <label> hidden by adding sr-only class, So that it couldn't visible to the user visually but it can be visible to the accessibility device user. We can ensure our search bar is completely accessible now.
Overall, this PR is intended to making the Search Bar accessible to accessibility devices with best standards.

Screenshots

We can't capture accessibility changes through screenshots as opposed to code changes since accessibility modifications are often imperceptible to the user, as their purpose is to assist disabled individuals.

Your review is appreciated @panwar8279 🙌🏻

Note to reviewer @panwar8279  🚨

There are some old PR's of mine which was opened 2 weeks ago by me, They are still not reviewed.
Please review all of them and merge them.
Here's the PR's

#820
#819
#799
#798


Undocumented options

Hey,
The following default options are not documented in usage.org:

strict-leech
strict-darkling-sh
strict-chaosmagician-sh
errata-cultist-power

Thanks. But only options that users might want to change are documented.
There are about a dozen of options, including the above four, that are essentially implementation details. They deal with cases where bugfixes, rules clarifications or outright rules changes would cause some existing games to become invalid. But there will never be any reason for anyone except the site admin to care about those options.


question: how to switch case result

In console, how to switch the current cases
For example, I wan to see the result of Case 2 in the screenshot below.

answered in #134

Ohh thanks


Image Resolution Issues for Data Acquisition and Testing

Hi, I've been following your excellent work, but I have a problem. I found that when your data is collected, the resolution of the single image (left, middle, right) collected is (320, 160, 60), while the single image collected during the test (left, middle, right) is (960 , 480, 120), I would like to ask whether this has an impact on the test results?Thank you very much。

This is an engineering trick to get around CARLAs bugged lens model.
We collect data at training time without image distortions at low resolution and FOV.
During test time we use a larger camera (with image distortions) and only use the the center part that correspons to the original resolution (and FOV) used in the dataset. Since image distortions are not strong in the center of the carma you end up with a mostly undistorted image.


Formulas with hyperlinks output empty values

xls2xlsx version: 0.1.5
Python version: 3.7.0
Operating System: OSX

Description
I am trying to convert an XLS file containing hyperlink formulas and this is not working. Instead, it will output empty values
Original XLS

Converted XLSX

What I Did
xls2xlsx /tmp/my_file.xls

Hi can you upload a test case please?
-joe
Sent from my AT&T iPhone

On Dec 9, 2020, at 12:45 PM, Ivan Rodriguez<EMAIL_ADDRESS>wrote:

xls2xlsx version: 0.1.5
Python version: 3.7.0
Operating System: OSX
Description
I am trying to convert an XLS file containing hyperlink formulas and this is not working. Instead, it will output empty values
What I Did
xls2xlsx /tmp/my_file.xls
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

xlrd 2.0.1 has just been released - please install that and see if this issue is fixed there.  If your file is a true xls file and not an HTML xls file, then xlrd does all of the work involved in reading the file.

Thanks I will try that, I meant to upload the used file but it contains some private info I was not able to anonymize or scrub

Ok if it’s still a problem, just try to make a similar small file that fails or only copy the bad cell(s) to a new file.
-joe
Sent from my AT&T iPhone

On Dec 14, 2020, at 11:41 AM, Ivan Rodriguez<EMAIL_ADDRESS>wrote:

Thanks I will try that, I meant to upload the used file but it contains some private info I was not able to anonymize or scrub
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Sorry I can't fix this without a sample input file.  Not even sure if the input is XLS or HTML embedded in XLS.


Checkout flow bug

There seems to be a bug. Can someone test and fix it please so I can pay? (Or I could ask one of my junior hackers.)

Thanks for making this issue @IvanAnishchuk, I'm happy to fix this and get it resolved ASAP for you!
Is there any chance you'd be able to tell me what sort of computer/browser you're using and if you have the Coinbase Wallet extension downloaded?(would be helpful for debugging)

Linux Firefox and chrome (chrome didn't even connect), tried mobile as well. No extension, no.

I tried w/ coinbase extension enabled as well with about the same result. Let me know if you need any more details

response is
{
    "jsonrpc": "2.0",
    "id": 8,
    "error": {
        "code": -32604,
        "message": "this request method is not supported"
    }
}

I have a team member asking questions about you guys, can you ping me in some DM maybe? Telegram or discord perhaps.

Ok, I managed to pay, this issue probably still need to be fixed (and maybe support other wallets?) thanks, looking forward to SG events.

Awesome, thanks so much and we look forward to having you @IvanAnishchuk! Separately we have an active ticket for this internally and I'll close this issue once we have it resolved in our next release.


fix(logger): switch to stderr

Resolves #1562

[x] This is a small change

Thanks for such a quick fix


Add QCSubmit to full-stack tests

This slows things down quite a bit, but I think is worth it:
Maybe one day we'll get the 4-core runners we are told we already have!


Migration Scripts Limitation for Kubernetes

There is a limitation regarding the migration scripts that we use whenever we run impact-graph, where the migration scripts can cause data duplication, this can cause a limitation for scalability in Kubernetes with data duplication once we deploy a new version of our apps running in a container. This is specifically important when we have more than one impact-graph container running and connected to the same database.

@mohammadranjbarz  @aminlatifi

AFAIK, the footprint of running migration scripts is saved on the database and each migration is run once per DB. So, probably we won't have any issue if we create multiple dockers and connect them to a same DB.

AFAIK, the footprint of running migration scripts is saved on the database and each migration is run once per DB. So, probably we won't have any issue if we create multiple dockers and connect them to a same DB.

Thanks for your feedback Amin. My concern is just when we want to update our apps with new version, and do a new release (Canary or Blue/Green for example). We might need to update the Production DB that is running, and for that I want to make sure that if we have (5 replicas of impact-graph for example) and all f them are being updated while we are doing a canary release, the migration scripts are only run once, and the DB is only changed once.
A.I, not changed with every new replica container that is running.
A good example would be:
Having a minimum of 2 replicas of impact graph running at all times, The first one is Master (Read/Write) Replica and the second one is Slave (Read Only) Replica, and the database update can be executed only on the master replica.
Do you think these kind of steps are needed??

AFAIK, the footprint of running migration scripts is saved on the database and each migration is run once per DB. So, probably we won't have any issue if we create multiple dockers and connect them to a same DB.

Thanks for your feedback Amin. My concern is just when we want to update our apps with new version, and do a new release (Canary or Blue/Green for example). We might need to update the Production DB that is running, and for that I want to make sure that if we have (5 replicas of impact-graph for example) and all f them are being updated while we are doing a canary release, the migration scripts are only run once, and the DB is only changed once.
A.I, not changed with every new replica container that is running.
A good example would be:
Having a minimum of 2 replicas of impact graph running at all times, The first one is Master (Read/Write) Replica and the second one is Slave (Read Only) Replica, and the database update can be executed only on the master replica.
Do you think these kind of steps are needed??

Good point. If I understood correctly, you are concerned about two replicas running a migration at the same time. So, do you think only master replica must be responsible for running migrations?

AFAIK, the footprint of running migration scripts is saved on the database and each migration is run once per DB. So, probably we won't have any issue if we create multiple dockers and connect them to a same DB.

Thanks for your feedback Amin. My concern is just when we want to update our apps with new version, and do a new release (Canary or Blue/Green for example). We might need to update the Production DB that is running, and for that I want to make sure that if we have (5 replicas of impact-graph for example) and all f them are being updated while we are doing a canary release, the migration scripts are only run once, and the DB is only changed once.
A.I, not changed with every new replica container that is running.
A good example would be:
Having a minimum of 2 replicas of impact graph running at all times, The first one is Master (Read/Write) Replica and the second one is Slave (Read Only) Replica, and the database update can be executed only on the master replica.
Do you think these kind of steps are needed??

Good point. If I understood correctly, you are concerned about two replicas running a migration at the same time. So, do you think only master replica must be responsible for running migrations?

Yes exactly, this could be one of the solutions, if there was a duplication in the migration scripts, if there was no duplication, then I think it's ok. But we have to test to know for sure

How we can move this issue to QA or done?

@divine-comedian Exactly, we are halting it for now. Starting it's development again could be a long time from now. I think we have two options, either keep it in icebox, or close it. Feel free to choose.

@mhmdksh
Is this epic closed? I had understood we were halting the Kubernetes Integration - is this still important?

Icebox!


fix: incorrect comparison of bytes and string objects

Address an issue in the is_url function where bytes objects are being compared to string objects using the == operator, leading to unexpected results.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 96.27%. Comparing base (98887ee) to head (d155e86).
Report is 1 commits behind head on development.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##           development     #202   +/-   ##
============================================
  Coverage        96.27%   96.27%
============================================
  Files                5        5
  Lines              483      483
============================================
  Hits               465      465
  Misses              18       18

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Make some automated optimizations using ruff

Description
Make some automated optimizations via the command: ruff check --select=PD,RET,RSE,RUF,SIM --fix --unsafe-fixes

https://docs.astral.sh/ruff

% ruff check --select=PD,RET,RSE,RUF,SIM --statistics
7	SIM401	[*] if-else-block-instead-of-dict-get
4	SIM210	[*] if-expr-with-true-false
3	SIM108	[*] if-else-block-instead-of-if-exp
2	PD002 	[*] pandas-use-of-inplace-argument
1	RSE102	[*] unnecessary-paren-on-raise-exception
1	RET506	[*] superfluous-else-raise
1	PD011 	[ ] pandas-use-of-dot-values
1	RUF010	[*] explicit-f-string-type-conversion
1	RUF100	[*] unused-noqa
[*] fixable with `ruff check --fix`

% ruff check --select=PD,RET,RSE,RUF,SIM --fix --unsafe-fixes
Found 23 errors (22 fixed, 1 remaining).

% ruff rule SIM401
if-else-block-instead-of-dict-get (SIM401)
Derived from the flake8-simplify linter.
Fix is sometimes available.
What it does
Checks for if statements that can be replaced with dict.get calls.
Why is this bad?
dict.get() calls can be used to replace if statements that assign a
value to a variable in both branches, falling back to a default value if
the key is not found. When possible, using dict.get is more concise and
more idiomatic.
Under preview mode, this rule will
also suggest replacing if-else expressions with dict.get calls.
Example
if "bar" in foo:
    value = foo["bar"]
else:
    value = 0

Use instead:
value = foo.get("bar", 0)

If preview mode is enabled:
value = foo["bar"] if "bar" in foo else 0

Use instead:
value = foo.get("bar", 0)

References

Python documentation: Mapping Types

Related Issue

Type of Change

[ ] 📚 Examples / docs / tutorials / dependencies update
[x] 🔧 Bug fix (non-breaking change which fixes an issue)
[ ] 🥂 Improvement (non-breaking change which improves an existing feature)
[ ] 🚀 New feature (non-breaking change which adds functionality)
[ ] 💥 Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🔐 Security fix

Checklist

[x] I've read the CODE_OF_CONDUCT.md document.
[x] I've read the CONTRIBUTING.md guide.
[ ] I've updated the code style using make codestyle.
[ ] I've written tests for all new methods and classes that I created.
[ ] I've written the docstring in Google format for all the methods and classes that I used.

@cclauss thanks for the PR :)
Please can you check the CI issues (don't bother about trivy ones, I should handle them in another PR)


keyboard shortcuts documentation

Is your feature request related to a problem? Please describe.
I am looking for a list of the default keyboard short cuts and how to change them.
I want to do simple things like toggle full scree, grab and release input.
I can't find and documentation describing what these shortcuts are or how to change them.
Describe the solution you'd like
In the user manual a section on the default keyboard shortcuts and possibly how to configure them.
Describe alternatives you've considered
I am going to start working through the code to see if I can Identify these so they can be added to the documentation.
Additional context
Add any other context or screenshots about the feature request here.

I found the following source files that would reference keyboard shortcuts.
Next step is to put this in to the wiki.
I will work on that a bit later.
https://github.com/FreeRDP/FreeRDP/blob/f002c7429b8e1ddb9198945ca329a4da1464b8dd/client/X11/xf_keyboard.c
https://github.com/FreeRDP/FreeRDP/tree/f002c7429b8e1ddb9198945ca329a4da1464b8dd/client

part of 3.x manpages for xfreerdp and sdl-freerdp


Support Vite 5

The package.json strictly calls out v4 - v5 just got released and I am currently able to run this on my V5 project, but its also pulling in vite v4 alongside v5 which probably doesn't need to happen.
I believe peer dependencies should be used in this case to support multiple major versions

Hey there, thanks for the Issue!
I am currently working on a newer version which fixes this problem!
Probably gonna drop next week..

Considering i got a lot of messages regarding Vite 5, i just want to state that this Package fully works with Vite 5 - there are no issues with it.

I haven't checked this package in a bit, but vite is still a direct dependency pictured above. That means when Vite has another major version release (v6) everyone is hung out to dry without installing both versions of vite into their node_modules.
I am suggesting a transition to using vite as a peerDependency to enable the usage of version ranges. There's a few pros to this model -

It only installs the user's desired version of vite and not what's specified in this project's dependencies. Fixes issue I've mentioned above
It only allows supported versions of vite to be installed in the user's project. (Not allowing users to use without officially supporting by adding to the dependency range)
A small PR just needs to be opened to add the range and update the example project to validate the plugin still works with the new version of vite.
Allows for multiple versions

 "peerDependencies": {
    "dependency-name": "^1.0.0 || ^2.0.0"
  }

Feel free to close this when 0.12 is published - I had some time before lunch to review and see it's now listed as a devDependency for the Plugin type which should resolve the issue that's happening now.
I hadn't dug into the code to check if you were using a utility function from vite, but happy the problem is resolving itself naturally without major changes.

Hey there!
Yes that is the current plan, right now there is still some parts used for vite, but these are removed to only types in the next Version 0.12.
Definitely moving vite towards peer :)

Moved this towards peer now as 0.12 will still take some more time!


Address/ landmark search

Like the eye icon that allows you to go to specific coordinates, it would be great to have another button that allows you to search for addresses and landmark. Like in openstreet map.

Is there an open API for converting landmarks into lat/lons

After some digging, I found this API: http://nominatim.openstreetmap.org/search?q="london"&format=json, which returns results in json (other formats are available, see http://wiki.openstreetmap.org/wiki/Nominatim).
Example:
[
  {
    "place_id": "179118272",
    "licence": "Data © OpenStreetMap contributors, ODbL 1.0. http://www.openstreetmap.org/copyright",
    "osm_type": "relation",
    "osm_id": "65606",
    "boundingbox": [
      "51.2867602",
      "51.6918741",
      "-0.5103751",
      "0.3340155"
    ],
    "lat": "51.5073219",
    "lon": "-0.1276474",
    "display_name": "London, Greater London, England, United Kingdom",
    "class": "place",
    "type": "city",
    "importance": 0.31637239413505,
    "icon": "http://nominatim.openstreetmap.org/images/mapicons/poi_place_city.p.20.png"
  }
  ...
]

We could use the first value (best effort) or we could run the query after a second of the last character is typed and show a list of value and then the user can select the one he wants to go to.

@nreese I start working on this feature in this branch. I am not a JS expert so any feedbacks you can provide will be very much appreciated.
I added a new 🔍  button to implement search. For now, I would like to do the API call and go to the coordinates of the first result. Then, it would be interesting to show in a drop-down slider the top n result (so that the user can have multiple choices) by firing the API query after 1 second of the last typed character and set the view to the user selected entry.

Also, the API returns a bounding box of the address/landmark and it would be cool to extrapolate the zoom level from that.

I found this work about implementing search in in leaflet, but I cannot find a way to integrate it into the map. https://github.com/perliedman/leaflet-control-geocoder/tree/master
Maybe for someone more expert than me in JS it is an easy task.


Liberapay: I want to give $1/month but account recipient not ready to receive new payments

Dear @zaquestion,
I wanted to thank you for lab. It is so nice and useful that I wanted to donate so that your work gets rewarded. I know it is not that much, but I wanted to give $0.25 per week. For some reason, Liberapay says I cannot donate:

@zaquestion I would advice to connect both stripe and paypal account:

stripe has lower fees
paypal has more users

Oh my, and here I thought that's was all setup.

Okay! I think I got it, thanks so much @MartinDelille -- I've also decided all donation will go straight to my farm (because thats my stripe account) and making lab great of course but thats a given. If you're ever in Seattle say hello and I'll give you some free greens :D https://www.tight5.farm/

Turning lines of code into food! The future is now! ;-)

Awesome, I'm both helping a developer AND the planet! ❤❤

Hi!
Now that Github has a sponsorship program, do you wish to enroll? Right now, Github matches the donated amount, so it is even better 😀 It also means more visibilty as your project will appear on peoples profiles.
https://help.github.com/en/articles/sponsoring-a-developer

@maelvls I signed up for the beta, but haven't seen anything come through yet... Perhaps theres just a setting I need to find?

Yeah I'm still on the waitlist, will defs update here when I get in :D Thanks for all the support! Also just FYI will be a bit less available this week (getting married), and later in month ("minimoon")

Congratulations!!! 🎉🎉🎉 I'm also going through many life changes right now, I feel you 😁
I went to one of my githubs projects settings and this button appears:

Looks like that setting lets you reference different funding platforms, including GitHub's if you're in it. So I got an error listing my user for the sponsorship, but was able to get the button up and at least reference the liberapay

Hi @zaquestion!
Did Github let you into the sponsor beta program yet? 😁

@maelvls ha! Your spidey senses must be working, I just got in a few days ago, it's on my list to get setup

Yey!! I’m now sponsoring you through the GitHub Sponsor thing! 🥳

@maelvls deeply appreciate it, that donation goes directly to my farm. Which is struggling with the ongoing pandemic and farmers market shutdowns in Seattle. Every dollar counts, thanks for using and supporting lab


Bug Report: The Hyperlink for 'Python' should take user to Python Documentation instead it takes to 'Full catalogs' and not specifically to Python Docs.

Reproduction steps
I will  make the correct link  which goes to Python Documentation attached with the hyperlink.
Expected behavior
Here in Python Section:

After clicking on 'Read more' it should take us to:

Actual Behavior
Here in Python Section:

After clicking on 'Read more' it is taking us to 'Full catalogs' instead of 'Python Documentation':

@SauravMukherjee44  can you please assign this issue to me , I would love to fix it for HACKTOBERFEST.
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Go for it! For support join our discord community: https://discord.com/servers/codein-community-831388874118660156


closeOnBackpressureLimit not in TS Docs

In the interface WebSocketBehavior, closeOnBackpressureLimit is not detailed as an option, however, it is referenced as a controlling option.
Should the option be present in this interface? I had to ignore a TS error when trying to enable the behavior.

Needs to be added can you open PR


Why the training is too slow.

I use 4 gpu

i have the same problem ,how do you solve it?

All of the loss computation is done on GPU 0. You can notice much more memory cunsumption on that card.  In my customed case, it took 1 sec in the loss.


Hide messages of user's alts on locks/bans

When a user is spamming, often times they are doing so under several accounts.  When this happens, a staff member has to manually hide each alts messages in the room, which takes up one line of the chat for each alt.  By doing this, it automatically does it on the same line as the lock  message at the same time, which both clutters up the chat less and makes it easier on the staff
PREVIEW:

Maybe a separate /hidealts. I don't want locking someone on a shared IP to suddenly hide all their classmates' messages.

What should that say in the chat? Something like,
panpawn the legend's alternate account's messages were cleared from Lobby by panpawn.

Maybe a separate /hidealts

Updated

Well, not literally /hidealts, that sounds too much like it hides your alt list.
/hidetextalts?

/hidealtstext

It doesn't necessarily even need text

Updated

Updated

Sorry for the confusion, does this look better?


Handle 422 responses from POST /oauth/tokens to API

At least some of the responses we've been seeing lately are from missing refresh tokens, so it might be Federated Identity-related.
https://rollbar.com/Heroku-3/identity/items/37/
/cc @heroku/enterprise-experience-team
Ready for review @heroku/api

@dmcinnes Ah, these are probably related to some of the session timeouts we've been working on.  Federated users don't have refresh tokens, which would result in the 422. This is still worth rescuing, but we're also working on some changes that will reduce these from coming in.

@naaman Cool! When we first created Fido we definitely punted on that part :smile:

Good call! +1.


Have change log link to tags

Have the change log link to tags, so you can navigate from the changelog to the codebase as it was for that release. (E.g. https://github.com/how-is/how_is/tree/v12.0.0)

Closed by https://github.com/how-is/how_is/pull/107.


auto_combine will be deprecated in xarray v0.15

First off, thanks a lot for writing this package! I'm current looking into using xgcm for analyzing my NEMO output, so I think xorca will save me a lot of headache to prepare my files.
During my first attempt using xorca, I ran into
FutureWarning: In xarray version 0.15 `auto_combine` will be deprecated. See
http://xarray.pydata.org/en/stable/combining.html#combining-multi

I will shortly submit a pull request that should solve this warning by replacing auto_combine with combine_by_coords. Hopefully this will make this package compatible with future versions of xarray.

Closed with #32.  Thanks @christophrenkl !


Add toThrowError matcher.

Closes #8

Hey @ibrahimbutt, based on your review below:
**toThrowError() should return true, regardless of what the error message is. You could do return true inside the catch block, because that code will only execute if there is an error.
The way toThrowError() works right now, should be the functionality of toThrowErrorMessage().**
I have separated the toThrowError and toThrowErrorMessage matchers with relevant tests which pass within the specRunner.html.


Migrate repo to SDP

Merging repo build changes to migrate content to Docs next-gen reference pipeline

Docs Build status updates of commit f958110:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 9549258:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Port is undefined

Hi. Seems the config Service is not working. For me e.g. port is undefined when starting the service.

Ideally we are supposed to create a .env file with a reference from the .env.example file and fill in the details there, by default the environment configuration is read from the .env file and hence there is no need to mention the envFilePath unless we need to separate out env file at per environment level, hope that helps and glad to see the issue is already resolved.
Marking this issues as closed hence !!


Improve node-to-node communication performance

Improve node-to-node communication performance by consuming pipe messages
in batches.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 31 Dec 2009 at 3:12

Merged into: #24

Original comment by<EMAIL_ADDRESS>on 14 Jan 2010 at 9:07

Changed state: Duplicate


TLA+ Basics Tutorial: typos and small changes

Description
This is a trivial PR that fixes typos and introduces small changes.
List of changes:

typos
I replaced the inequality operator # for the inequality operator /= (the one that is mentioned in the cheat sheet)
I made the tutorial text consistent with the model checker results (bob is happy at state 6, and not 8)
I changed the invariant in hello_world_typed.cfg. The previous one was referring to an invariant that was not defined.
I added a clarification sentence about typechecker being sometimes able to infer types of operators

For contributor use:

[ n/a] If applicable, unit tests are written and added to CI.
[ n/a] Ran go fmt, cargo fmt and etc. (or had formatting run automatically on all files edited)
[ +] Updated relevant documentation (docs/) and code comments.
[ n/a: because this is a small pass over the tutorial, there should be no issue related with it] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[+ ] Re-reviewed Files changed in the Github PR explorer.
[ n/a: because this is a small pass over the tutorial, it should not be included in the changelog] Added a changelog entry, using unclog.

Great, thanks so much!


Can't pass function in state, in location descriptor

Not mentioned in the docs that this is not possible, is it on purpose?
I'm doing something like;
    this.props.router.push({pathname:'/path/add/item', state: {
      onSave: state => ...

Version
2.0.0
Test Case
http://jsbin.com/sacerobuxi/edit?html,js,output
Steps to reproduce
Expected Behavior
Actual Behavior

That's correct. Location state is actually serialized and thus must be serializable.

Just asking, but why is it serialised?

On 11 Aug 2016, at 4:59 PM, Jimmy Jia<EMAIL_ADDRESS>wrote:
That's correct. Location state is actually serialized and thus must be serializable.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/reactjs/react-router/issues/3720#issuecomment-239168791, or mute the thread https://github.com/notifications/unsubscribe-auth/ABk1jCT-zPmA8MExZFFOooZ_C6-nE0nuks5qeyq1gaJpZM4Jh7mQ.

So it persists across things like page refreshes

I was thinking, isn't it possible to serialize functions?
Functions could be useful for passing around callbacks, is there a reason not to support that?
I might even pull request ':)


docs: add additional search options, typesense and local search

Motivation
This PR is a follow-up to https://github.com/facebook/docusaurus/issues/5446#issuecomment-910164722.
The motivation behind this PR is to provide multiple options for a search bar for users, in addition to Algolia. To this end, I've updated the first section of the docs to list out all the options, I've then added a section about Typesense (linking to Typesense's documentation) and another section about local search plugins.
Have you read the Contributing Guidelines on pull requests?
Yes.
Test Plan
I've already run this locally to make sure formatting is good, and that all links I added work fine.
Related PRs
None.

I've signed the CLA

As this PR is a bit old (was on holiday 2 weeks 😅 ) I'll fix and merge it now, but feel free to provide a live deployment in the future :)

Also added some useful things about Algolia that were not very well documented

Thanks for merging this @slorber! Great idea about setting up Typesense DocSearch on a Docusaurus site and including that as an example. I'll create a separate for PR it.


Compatibility with the latest autograd version

Hi,
I read your paper and wanted to give your package a try, but I encounter issues with the way you link to the autograd module.
First, the pip installation requires the autograd package, whereas according to the https://autograd.readthedocs.io/en/latest/installation.html one should now install the dragongrad package.
Installing the dragongrad package makes the autograd module available, but then I face a second issue, namely, your module calls autograd.np as a replacement for numpy, whereas according to https://autograd.readthedocs.io/en/latest/usage.html one should now load numpy separately from autograd.
Do you have a workaround for me, or can you update your package?

Hey
Thanks for reaching out.
The autograd you mentioned is different from the dependency listed in this package. The dependency listed in this package is autograd maintained by HIPS
The dragongrad you mentioned seemed to be a different one.


Possibility to combine clipped and non-clipped geometry on the same scene.

Hello,
Going through the documentation, I read this on the OSPGroup definition:

All geometries and volumes assigned to geometry or volume will be clipped. All clipping geometries from all groups and Instances will be combined together – a union of these areas will be applied to all other objects in the world.

I made a test scene to confirm the statement so that, even though I added the clipping geometry to a group, other group instances are clipped as well.
In our visualization tool, based on OSPRay, some of the use cases that we have require that some geometry is clipped, while other remains non clipped at all.
Is there a way to achieve this in the current version of OSPRay (2.8.0) or is it possible to have it?
Currently, to bypass this limitation, we have to implement our own renderers, adding a custom parameter to the material so that when we hit a primitive, depending on the aforementioned parameter and the clipping planes on the scene, we clip the geometry or not.
We are on the process of updating the version of OSPRay we use to the latest release, and this is the only bit that prevent us from using OSPRay out of the box.
Kind regards.

I just saw it is already a WIP feature at #504
Apologies for the double posting.


MultiTurnPromptBot responds out of order when run via Microsoft Teams

Sample information

Sample type: Sample
Sample language: dotnetcore
Sample name: MultiTurnPromptBot

Describe the bug
Conversation is out of order when run via Teams
To Reproduce
Steps to reproduce the behavior:

Publish to team.
Run in teams
Notice questions are re-asked and appear out of order.

Expected behavior
Waterfall steps should be maintained
Screenshots

@mjanulaitis  Unfortunately, I'm not able to reproduce this. What version of bot SDK are you running? (I'm running 4.7.0)

@dmvtech  I am running version 4.7.0.

@mjanulaitis Are you able to reproduce the issue when debugging locally and using ngrok/service bus? I'm wondering if we can maybe figure out why it is doing this in that way.
Alternatively, have you configured either your bot or bot channels registration for Application Insights? We can also use that to get an idea of what is happening.

I took a look and I think that checking for DialogTurnStatus.Waiting should be all that is needed. And this is already being done as this is using UserProfileDialog which inherits from ComponentDialog.
The other thought I had; is your App Service (plan) scaled out? I was just curious if maybe you have multiple instances of the bot that are running, and with memory state, you are essentially talking to two different running instances of the bot. (one message goes to one bot, the next message goes to the other). This would be solved by using a persistent state storage, multiple running bots would all use one state storage and know exactly where you were in conversation.

The bot should have only been on one server.  I am no longer working on this project and am unable to continue the support issue.  Thank you for your help thus far.


Add policy to control which RPC operations are re-tryable.

At the end of this task it will be possible to configure the bigtable::Table class with a policy to control which RPC operations can be retried based on their status codes.  The default policy should only retry operations that fail with grpc::StatusCode::ABORTED, grpc::StatusCode::UNAVAILABLE, or grpc::StatusCode::DEADLINE_EXCEEDED.

On second thought, I am not sure this is a useful policy.  The semantics of the error codes is pretty strict.  I am going to close it as invalid.


Update getting started artwork

This PR fixes #125877:

Replaces PNG screenshots with abstract SVG artwork
Adds theming support so that artwork colors will update with the user's them
Adds resizing rules to the SVGs themselves to ensure they scale appropriately

https://user-images.githubusercontent.com/25163139/124034648-59438200-d9b0-11eb-9971-857c1c8b9b27.mp4

https://github.com/microsoft/vscode/blob/920303b974e9ac5b9f7d30d5bc354db5a849c793/src/vs/workbench/contrib/welcome/gettingStarted/common/media/workspaceTrust.svg#L27
This line throws an XML attributes construct error

@4086606 thanks for that find! I'll push a fix shortly. How did you come by the error btw?

Prototyping SVG support in the image-preview extension :)

Nice. This is now fixed via https://github.com/microsoft/vscode/commit/4860b6f45b9c132949c58da526f8dc79669ee21b

Did a regex search and picked up another one at interactivePlayground.svg


CPP definitions from .cabal file not picked up (again)

I have some doctest that looks like this:
#if __GLASGOW_HASKELL__ >= 908 && !defined(CLASH_MULTIPLE_HIDDEN)
>>> :t readNew (blockRam (0 :> 1 :> Nil))
readNew (blockRam (0 :> 1 :> Nil))
  :: ...
     ...
     ... =>
     Signal dom addr -> Signal dom (Maybe (addr, a)) -> Signal dom a

#else
>>> :t readNew (blockRam (0 :> 1 :> Nil))
readNew (blockRam (0 :> 1 :> Nil))
  :: ...
     ...
     ...
     ...
     ... =>
     Signal dom addr -> Signal dom (Maybe (addr, a)) -> Signal dom a

#endif

which fails with:
[ERROR  ] [ThreadId 668] /home/christiaan/devel/clash-compiler/clash-prelude/src/Clash/Prelude/BlockRam.hs:842: failure in expression `:t readNew (blockRam (0 :> 1 :> Nil))'
[ERROR  ] [ThreadId 668] expected: readNew (blockRam (0 :> 1 :> Nil))
[ERROR  ] [ThreadId 668]             :: ...
[ERROR  ] [ThreadId 668]                ...
[ERROR  ] [ThreadId 668]                ... =>
[ERROR  ] [ThreadId 668]                Signal dom addr -> Signal dom (Maybe (addr, a)) -> Signal dom a
[ERROR  ] [ThreadId 668]  but got: readNew (blockRam (0 :> 1 :> Nil))
[ERROR  ] [ThreadId 668]             :: (KnownDomain dom, Eq addr,
[ERROR  ] [ThreadId 668]                 Hidden (AppendSymbol dom "_clk") (Clock dom),
[ERROR  ] [ThreadId 668]                 Hidden (AppendSymbol dom "_rst") (Reset dom),
[ERROR  ] [ThreadId 668]                        ^
[ERROR  ] [ThreadId 668]                 Hidden (AppendSymbol dom "_en") (Enable dom), Enum addr, NFDataX a,
[ERROR  ] [ThreadId 668]                 NFDataX addr, Num a) =>
[ERROR  ] [ThreadId 668]                Signal dom addr -> Signal dom (Maybe (addr, a)) -> Signal dom a

because CLASH_MULTIPLE_HIDDEN is not defined when doctests are run.
I would've reoponed #71 but I don't have permissions

Also, looking at https://github.com/martijnbastiaan/doctest-parallel/pull/73/files, which is now reverted, I wonder whether cppOptions was used correctly...
Because Cabal uses cppOptions like this:
https://github.com/haskell/cabal/blob/51da23dda752448c96a2971b43aacc2fd2e0cc59/Cabal/src/Distribution/Simple/GHC/Internal.hs#L633
ghcOptCppOptions = cppOptions bi

https://github.com/haskell/cabal/blob/51da23dda752448c96a2971b43aacc2fd2e0cc59/Cabal/src/Distribution/Simple/Program/GHC.hs#L741
 ["-optP" ++ opt | opt <- ghcOptCppOptions opts]

It's a bit sad I didn't document what cases broke --- I only mentioned that stuff broke in https://github.com/martijnbastiaan/doctest-parallel/pull/74. I'm tempted to reintroduce the functionality, this time using -optP.
Having said that, CPP/flag support will remain a sore point for doctest-parallel because we're manually parsing the .cabal files and extracting the relevant flags from there. This doesn't account for custom setups, cabal.project, etc.


Fix link to example files

Pull Request Test Coverage Report for Build 9

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

Totals

Change from base Build 8:
0.0%

Covered Lines:
889

Relevant Lines:
906

💛  - Coveralls


NamedTuple#to_h fails when a tuple is empty

def foo(**options)
  options.to_h
end

foo # Syntax error in expanded macro: macro_XXX: for empty hashes use '{} of KeyType => ValueType'

I would like to fix it, but I don't know what to_h should return in this case. Thank you.

Oh, it's easy, the method should used NamedTuple.new(...) instead of {...}.

But what would be the type of the resulting Hash in this case?

Oh... it's to_h, I was confused. Mmm... maybe we should check for this case. I think the Hash should be {} of NoReturn => NoReturn.
Probably the error is correct, using to_h on an empty named tuple doesn't make much sense anyway.

But that won't allow you to check existence of a key. Whatever the key of the type you want. Until we support object i would say we should raise or even not compile.

FIY, my use case:
struct NamedTuple
  def merge(other)
    T.from(to_h.merge(other.to_h))
  end
end

class C
  @options : NamedTuple(i: Int32, s: String, b: Bool)
  DEFAULT_OPTIONS = {i: 1, s: "str", b: false}

  def initialize(**options)
    @options = DEFAULT_OPTIONS.merge(options)
  end
end

If to_h occurs a compile error whenever a tuple is empty, the above code is useless. I'll find another solution. Thank you.

I personally don't think modelling C that way is a good idea. You should have variable @i, @s and @b in C, and make the initialize accept those as named arguments. All those named tuples and hashes will go away, and the code will be simpler and faster.

@asterite Thank you for your advice! I like named arguments but it's not suitable this time :). By the way, I defined to_h? method for my problem like:
struct NamedTuple
  def merge(other)
    if h = to_h?
      if other = other.to_h?
        T.from(h.merge(other))
      else
        self
      end
    else
      other
    end
  end

  def to_h?
    {% unless T.empty? %}
      {
        {% for key in T %}
          {{key.symbolize}} => self[{{key.symbolize}}].clone,
        {% end %}
      }
    {% end %}
  end
end

It's just FYI!

This issue was closed by https://github.com/crystal-lang/crystal/pull/4076.


dec crash for spec bitstream

using master branch.
please rename it with suffix .264 instead and try "./h264dec gop2.264 out.yuv".
gop2.txt

@huili2  Certainly, I will look into the issue.

@xiaotiansf  The crash still exists on master branch. Could you please check again?

Let me check.

@huili2 Is the same asset gop2.txt?

@huili2 I just synced to master. Retest many times with this asset gop2.txt and never reproduce the crash.

Strange. I repeated to crash .... Let me check on another PC.

I am on Widows 10 64-bit machine.

Mac seems OK but one linux reports error. Close this one for now.


chore(deps): update terraform-module

This PR contains the following updates:

Package
Type
Update
Change

github.com/terraform-ibm-modules/terraform-ibm-devsecops-alm
module
patch
v1.0.4-beta.5 -> v1.0.4-beta.6

github.com/terraform-ibm-modules/terraform-ibm-devsecops-cc-toolchain
module
patch
v1.0.6-beta.2 -> v1.0.6

github.com/terraform-ibm-modules/terraform-ibm-devsecops-cd-toolchain
module
patch
v1.0.6-beta.2 -> v1.0.6

github.com/terraform-ibm-modules/terraform-ibm-devsecops-ci-toolchain
module
patch
v1.0.6-beta.2 -> v1.0.6

Release Notes

terraform-ibm-modules/terraform-ibm-devsecops-alm
v1.0.4-beta.6
Compare Source
What's Changed

ci: update tests by @huayuenh in https://github.com/terraform-ibm-modules/terraform-ibm-devsecops-alm/pull/176

Full Changelog: https://github.com/terraform-ibm-modules/terraform-ibm-devsecops-alm/compare/v1.0.4-beta.5...v1.0.4-beta.6

terraform-ibm-modules/terraform-ibm-devsecops-cc-toolchain
v1.0.6
Compare Source
Full Changelog: https://github.com/terraform-ibm-modules/terraform-ibm-devsecops-cc-toolchain/compare/v1.0.5...v1.0.6

terraform-ibm-modules/terraform-ibm-devsecops-cd-toolchain
v1.0.6
Compare Source
Full Changelog: https://github.com/terraform-ibm-modules/terraform-ibm-devsecops-cd-toolchain/compare/v1.0.5...v1.0.6

terraform-ibm-modules/terraform-ibm-devsecops-ci-toolchain
v1.0.6
Compare Source
Full Changelog: https://github.com/terraform-ibm-modules/terraform-ibm-devsecops-ci-toolchain/compare/v1.0.5...v1.0.6

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

:tada: This PR is included in version 1.0.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add Take Command from Kerbal Stuff

This pull request was automatically generated by Kerbal Stuff on behalf of seanmcdougall, to add Take Command to CKAN.
Please direct questions about this pull request to seanmcdougall.

Please delete this pull request... I have released a new version of this plugin and will re-submit manually soon.


drivers/i2c/i2c_dw.c: make 64-bit clean

Use UINT_TO_POINTER() macros to silence warnings about casts.
Fixes: #19219
Signed-off-by: Charles E. Youse<EMAIL_ADDRESS>

I wonder if we should modify struct i2c_dw_dev_config's base_address member to be a uintptr_t instead of a u32_t like it is now.

Hmmm.


feat: improve telemetry

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

193 of 224 (86.16%)  changed or added relevant lines in 21 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.3%) to 77.689%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

plugins/receivers/slack/client.go
12
13
92.31%

core/notification/service.go
40
42
95.24%

core/receiver/service.go
20
22
90.91%

internal/store/postgres/receiver.go
7
9
77.78%

internal/store/postgres/template.go
6
8
75.0%

plugins/providers/cortex/service.go
1
3
33.33%

plugins/queues/postgresq/cleanup.go
0
2
0.0%

core/notification/handler.go
22
25
88.0%

plugins/queues/inmemory/queue.go
0
3
0.0%

plugins/providers/cortex/option.go
0
4
0.0%

Files with Coverage Reduction
New Missed Lines
%

internal/store/postgres/subscription.go
1
84.96%

plugins/providers/cortex/service.go
1
35.42%

Totals

Change from base Build<PHONE_NUMBER>:
0.3%

Covered Lines:
2824

Relevant Lines:
3635

💛  - Coveralls

making this PR as draft as I need to test some features more


Views not updating for containers added after other components.

Here's how to reproduce the bug:

Head here: https://ryanpcmcquen.org/react-habitat-poc/
Click ADD TO CART (it works).
Edit the HTML for the page in the dev tools, add in a container like this:

Run window.updateHabitat(); in the console to make the new Cart appear.
Edit the HTML again for the page in the dev tools, add in a container like this:

Run window.updateHabitat(); in the console to make the new ProductCard appear.
Now click ADD TO CART on the new ProductCard container.
Although the Redux state updates (as seen in the Redux dev tools), the Cart view does not update, as seen by the lack of the number incrementing.

Source: https://github.com/ryanpcmcquen/react-habitat-poc

Hi Ryan,
Welcome back! Thanks for your continued support 👍
I've checked out your code and had a look. Im a bit puzzled because I can replicate your issue following your steps. However, recreating your steps in code and everything works flawlessly!
For example, add this code at the end of your Manifest.js file
window.mount1 = function() {
    const target = document.createElement('div');
    target.innerHTML = '

';
    window.document.body.appendChild(target);
    window.updateHabitat();
};

window.mount2 = function() {
    const target = document.createElement('div');
    target.innerHTML = '

';
    window.document.body.appendChild(target);
    window.updateHabitat();
};

Then in your browser dev tools console

Enter command window.mount1(). Verify the shopping cart added to the end of the page.
Enter command window.mount2(). Verify the product card added to the end of the page.
Now click ADD TO CART on the new ProductCard container.
The cart views update correctly.

Can you please verify.
I have a feeling that editing the HTML in dev tools is doing something screwy with node references. Im just not sure if this is our bug or a bug with Provider in Redux yet.

That works for me also, weirdness! A workaround we found is moving the parentheses to be outside the span tag. This also works with the dev tools scenario. It feels like a JSX bug to me, but it could also be dev tools related.

And here is the fix for reference:
https://github.com/ryanpcmcquen/react-habitat-poc/commit/54b33411f5fb1cfed79f77d9f17e8dc670e30aa4


(encoding) How to get the amount of payload that has been used?

I am encoding a map where many entries are conditional, and at the end, would like to transmit only the part of the buffer that contains the encoded cbor. I assume the state struct contains that information somewhere, but casually looking at the header files, nothing pops out.

You can compare the start of the buffer to the payload_end member after encoding.


The file favicon.ico is being served over http when I have set everything to be served over https.

Welcome to Ghost's GitHub repo! 👋🎉
Do you need help or have a question? Please come chat in Slack: https://ghost.org/slack 👫. Found a bug? Please fill out the sections below... thank you 👍
Issue Summary
My blog is self-hosted using an https site and I am getting the following warning that is in Chrome. I've replaced my actual domain with  in the error message for my personal privacy.
/ghost/#/signin:1 Mixed Content: The page at 'https:///ghost/#/signin' was loaded over HTTPS, but requested an insecure image 'http:///content/images/2018/02/favicon.ico?t=1518474189048'. This content should also be served over HTTPS.
Steps to Reproduce

Configure NGINX as an SSL proxy to ghost
Browse to your personal ghost website /admin and login. After logging in, the lock icon on the browser changes to indicate insecure. Chrome throws the error listed above indicating that favicon.ico is being loaded over http (not https), despite the rest of the content being served as https.

Any other info e.g. Why do you consider this to be a bug? What did you expect to happen instead?
I would expect favicon.ico to be served over https like all of the rest of the data.
Technical details:

Ghost Version: 1.21.1
Node Version: 6.11.2
Browser/OS: FireFox and Chrome on OpenBSD
Database: sqlite

@mattschwartz03 you need to adjust Ghost's url config value to also be https. If you need support please use our Slack channel rather than opening GitHub issues - we use GitHub solely for bug tracking and on-going feature work and do not offer support here, thanks!


Some changes to compare

integrated the login and register bit with what I had,
added the (templates/auth, templates/email and templates/profile) for the user emails confirmation and reset,
added the email.py to handler sending emails and populated the profile and the contact us pages...…

modified routes.py and layout.html
app/init.py should be users.login


File adapter needs to open file with 'rb'

Using the :HashFile adapter and storing a pretty simple hash value as the key, Moneta will save a value it is incapable of loading.  The save adapter writes the marshaled value as:
::File.open(temp_file, 'wb') {|f| f.write(value) }
yet the load code simply calls:
::File.read(store_path(key)
The file is not loaded in binary mode and the marshaler raises the 'marshal data too short' exception.
The code should be:
::File.read(store_path(key), {mode: 'rb'})

Hiya! Do you have any plans to push this out to a new gem version?

There's an open issue for this -- see #87.  Basically this gem had widespread test failures that would have made releasing a bad idea.  They are almost all resolved now, but there are still a couple of 1.9.3 issues, according to Travis.

Gotcha. In case I can help: is it just the failing tests in runs 488.37 and 488.43 that are holding up the release?

@avdi, yes those are the two that we would want green, but just a warning that I think some of the failures may be intermittent.  When I last tried to get all the tests green it felt a bit like whac-a-mole.  Any help you can provide will be very much appreciated though!
@minad, are you happy for a new release to go ahead if we can get all outstanding tests to pass?

It looks like the failures were because the build server couldn't connect to remote repos.

I'm going to go ahead and say that yes, if we can fix those 1.9.3 travis problems, we'll be good to put out a new release of moneta.  Sorry for the delay.


Bug downloading cbioportal

Error executing process > 'cbioportal_proteindb'
Caused by:
Process cbioportal_proteindb terminated with an error exit status (1)
Command executed:
pypgatk_cli.py cbioportal-to-proteindb --config_file "/home/yperez/.nextflow/assets/bigbio/pgdb/configs/cbioportal_config.yaml" --input_mutation cbioportal_allstudies_data_mutations_mskcc.txt --input_cds Homo_sapiens.GRCh37.75.cds.all.fa --clinical_sample_file cbioportal_allstudies_data_clinical_sample.txt --tissue_type all  --output_db cbioPortal_proteinDB.fa
Command exit status:
1
Command output:
(empty)
Command error:
Traceback (most recent call last):
File "/usr/local/bin/pypgatk_cli.py", line 4, in
import('pkg_resources').run_script('pypgatk==0.0.5', 'pypgatk_cli.py')
File "/usr/lib/python3/dist-packages/pkg_resources/init.py", line 666, in run_script
self.require(requires)[0].run_script(script_name, ns)
File "/usr/lib/python3/dist-packages/pkg_resources/init.py", line 1446, in run_script
exec(code, namespace, namespace)
File "/usr/local/lib/python3.7/dist-packages/pypgatk-0.0.5-py3.7.egg/EGG-INFO/scripts/pypgatk_cli.py", line 43, in
cli()
File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 764, in call
return self.main(*args, **kwargs)
File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 717, in main
rv = self.invoke(ctx)
File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 1137, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 956, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 555, in invoke
return callback(*args, **kwargs)
File "/usr/local/lib/python3.7/dist-packages/click/decorators.py", line 17, in new_func
return f(get_current_context(), *args, **kwargs)
File "/usr/local/lib/python3.7/dist-packages/pypgatk-0.0.5-py3.7.egg/pypgatk/commands/cbioportal_to_proteindb.py", line 31, in cbioportal_to_proteindb
cosmic_to_proteindb_service = CancerGenomesService(config_file, pipeline_arguments)
File "/usr/local/lib/python3.7/dist-packages/pypgatk-0.0.5-py3.7.egg/pypgatk/cgenomes/cgenomes_proteindb.py", line 26, in init
super(CancerGenomesService, self).init(self.CONFIG_COSMIC_DATA, config_file, pipeline_arguments)
File "/usr/local/lib/python3.7/dist-packages/pypgatk-0.0.5-py3.7.egg/pypgatk/toolbox/general.py", line 50, in init
with open(yaml_configuration_file, 'r') as f:
FileNotFoundError: [Errno 2] No such file or directory: '/home/yperez/.nextflow/assets/bigbio/pgdb/configs/cbioportal_config.yaml'
Work dir:
/home/yperez/ms-work/proteogenomics-project/work/0e/3c333dd82854e49ac4acc869a24b33
Tip: you can replicate the issue by changing to the process work dir and entering the command bash .command.run

It says that it can't find the cbioportal configuration file. Can you make sure this file does exist on /home/yperez/.nextflow/assets/bigbio/pgdb/configs/cbioportal_config.yaml ?

I check and it exists. It is really a strange error.

indeed, I will run it locally tonight and will see.

Fixed.
the config file was directly given to the script-command without marking it as input in the process. the same thing was done to the cosmic process.


[SKIP CI] Update tenancy.md

Fixed #1536
This PR updates tenancy.md to match the up-to-date multi-tenancy behavior.

@lipingxue Can you please fix following broke link too from tenancy.md?
 Vmgroups can be created and managed via the [Admin CLI](/user-guide/admin-cli/#Vmgroup)

@shuklanirdesh82 I have addressed your comments, please take a look.


child widget collides with toggle icon

The 'child' widget, which is the widget displayed on the background, with a large width collides with toggle icon (visually). And is centered based on the whole action slider rather than the empty part of the slider. I have created a pr #7

I probably won't have time to take a closer look today. Until then, you can use your MR or manually wrap a Padding around child to solve the problem.
Thanks for creating issue and MR :)

Thank you for your time, here to help anytime!


Error handling refactoring

This looks great overall -- thanks, @alexander-chervony! I'd like to see some changes before we land it, but the changes should be relatively small.

Hey Chris


Added fix for 5.0.0-snapshot1

See https://github.com/hapifhir/hapi-fhir-jpaserver-starter/issues/328

Any chance this can go into 6.0.0?

wait ...


HomeAutomation: HVAC Register Vent Fans

| Ventbots Are Fans Of HVAC And Home Automation / Hackaday |

#1530 closes #1515


An import error in a KV file raises AttributeError in Kivy instead of ParserException

Software Versions

Python: 3.10.6
OS: Linux Mint 21
Kivy: 2.1.0
Kivy installation method: pip wheel

Describe the bug
An import error in a KV file raises AttributeError, which produces a long Kivy traceback.
Expected behavior
A ParserException would be more clear.
To Reproduce
The point of this is to raise an exception, so the code intentionally doesn't run, but does reproduce the issue.
It should reproduce the error in the latest master because the code there is the same as in my version:
https://github.com/kivy/kivy/blob/5744bc564dd99c6326438e628acd5f3e3e146ff8/kivy/lang/parser.py#L559

toggle_nav_drawer.zip

Code and Logs and screenshots
The only logs you need to know are the last line(s) shown in the screenshots.
Before fixing:

/home/owner/.virtualenvs/kivy/bin/python "toggle_nav_drawer.py" (in directory: /home/owner/metaprojects/examples/kivymd)
[INFO   ] [Logger      ] Record log in /home/owner/.kivy/logs/kivy_23-01-01_83.txt
[INFO   ] [Kivy        ] v2.1.0
[INFO   ] [Kivy        ] Installed at "/home/owner/.virtualenvs/kivy/lib/python3.10/site-packages/kivy/__init__.py"
[INFO   ] [Python      ] v3.10.6 (main, Nov 14 2022, 16:10:14) [GCC 11.3.0]
[INFO   ] [Python      ] Interpreter at "/home/owner/.virtualenvs/kivy/bin/python"
[INFO   ] [Logger      ] Purge log fired. Processing...
[INFO   ] [Logger      ] Purge finished!
[INFO   ] [Factory     ] 189 symbols loaded
[INFO   ] [Image       ] Providers: img_tex, img_dds, img_sdl2, img_pil (img_ffpyplayer ignored)
[INFO   ] [Window      ] Provider: sdl2
[INFO   ] [GL          ] Using the "OpenGL" graphics system
[INFO   ] [GL          ] Backend used <sdl2>
[INFO   ] [GL          ] OpenGL version <b'3.1 Mesa 22.0.5'>
[INFO   ] [GL          ] OpenGL vendor <b'Intel'>
[INFO   ] [GL          ] OpenGL renderer <b'Mesa Intel(R) HD Graphics 4000 (IVB GT2)'>
[INFO   ] [GL          ] OpenGL parsed version: 3, 1
[INFO   ] [GL          ] Shading version <b'1.40'>
[INFO   ] [GL          ] Texture max size <16384>
[INFO   ] [GL          ] Texture max units <16>
[INFO   ] [Window      ] auto add sdl2 input provider
[INFO   ] [Window      ] virtual keyboard not allowed, single mode, not docked
[INFO   ] [KivyMD      ] 1.1.1, git-Unknown, 2022-12-08 (installed at "/home/owner/.virtualenvs/kivy/lib/python3.10/site-packages/kivymd/__init__.py")
[INFO   ] [Text        ] Provider: sdl2
 Traceback (most recent call last):
   File "/home/owner/metaprojects/examples/kivymd/toggle_nav_drawer.py", line 60, in <module>
     Builder.load_file("toggle_nav_drawer.kv")
   File "/home/owner/.virtualenvs/kivy/lib/python3.10/site-packages/kivy/lang/builder.py", line 305, in load_file
     return self.load_string(data, **kwargs)
   File "/home/owner/.virtualenvs/kivy/lib/python3.10/site-packages/kivy/lang/builder.py", line 372, in load_string
     parser = Parser(content=string, filename=fn)
   File "/home/owner/.virtualenvs/kivy/lib/python3.10/site-packages/kivy/lang/parser.py", line 483, in __init__
     self.parse(content)
   File "/home/owner/.virtualenvs/kivy/lib/python3.10/site-packages/kivy/lang/parser.py", line 590, in parse
     self.execute_directives()
   File "/home/owner/.virtualenvs/kivy/lib/python3.10/site-packages/kivy/lang/parser.py", line 559, in execute_directives
     mod = getattr(mod, part)
 AttributeError: module 'kivymd.uix.navigationdrawer' has no attribute 'NavigationLayout'. Did you mean: 'MDNavigationLayout'?
Compilation failed.

After fixing (The IDE can detect the error, and the user can double-click the error to go to the line in the KV file)

If a KV file is not being used (If load_string), the exception will still be the more clear ParserException, but the last line containing the source filename will not appear. The user is already probably in the py file containing the issue anyway.

/home/owner/.virtualenvs/kivy/bin/python "toggle_nav_drawer.py" (in directory: /home/owner/metaprojects/examples/kivymd)
[INFO   ] [Logger      ] Record log in /home/owner/.kivy/logs/kivy_23-01-01_82.txt
[INFO   ] [Kivy        ] v2.1.0
[INFO   ] [Kivy        ] Installed at "/home/owner/.virtualenvs/kivy/lib/python3.10/site-packages/kivy/__init__.py"
[INFO   ] [Python      ] v3.10.6 (main, Nov 14 2022, 16:10:14) [GCC 11.3.0]
[INFO   ] [Python      ] Interpreter at "/home/owner/.virtualenvs/kivy/bin/python"
[INFO   ] [Logger      ] Purge log fired. Processing...
[INFO   ] [Logger      ] Purge finished!
[INFO   ] [Factory     ] 189 symbols loaded
[INFO   ] [Image       ] Providers: img_tex, img_dds, img_sdl2, img_pil (img_ffpyplayer ignored)
[INFO   ] [Window      ] Provider: sdl2
[INFO   ] [GL          ] Using the "OpenGL" graphics system
[INFO   ] [GL          ] Backend used <sdl2>
[INFO   ] [GL          ] OpenGL version <b'3.1 Mesa 22.0.5'>
[INFO   ] [GL          ] OpenGL vendor <b'Intel'>
[INFO   ] [GL          ] OpenGL renderer <b'Mesa Intel(R) HD Graphics 4000 (IVB GT2)'>
[INFO   ] [GL          ] OpenGL parsed version: 3, 1
[INFO   ] [GL          ] Shading version <b'1.40'>
[INFO   ] [GL          ] Texture max size <16384>
[INFO   ] [GL          ] Texture max units <16>
[INFO   ] [Window      ] auto add sdl2 input provider
[INFO   ] [Window      ] virtual keyboard not allowed, single mode, not docked
[INFO   ] [KivyMD      ] 1.1.1, git-Unknown, 2022-12-08 (installed at "/home/owner/.virtualenvs/kivy/lib/python3.10/site-packages/kivymd/__init__.py")
[INFO   ] [Text        ] Provider: sdl2
 Traceback (most recent call last):
   File "/home/owner/metaprojects/examples/kivymd/toggle_nav_drawer.py", line 60, in <module>
     Builder.load_file("toggle_nav_drawer.kv")
   File "/home/owner/.virtualenvs/kivy/lib/python3.10/site-packages/kivy/lang/builder.py", line 305, in load_file
     return self.load_string(data, **kwargs)
   File "/home/owner/.virtualenvs/kivy/lib/python3.10/site-packages/kivy/lang/builder.py", line 372, in load_string
     parser = Parser(content=string, filename=fn)
   File "/home/owner/.virtualenvs/kivy/lib/python3.10/site-packages/kivy/lang/parser.py", line 490, in __init__
     self.parse(content)
   File "/home/owner/.virtualenvs/kivy/lib/python3.10/site-packages/kivy/lang/parser.py", line 604, in parse
     self.execute_directives()
   File "/home/owner/.virtualenvs/kivy/lib/python3.10/site-packages/kivy/lang/parser.py", line 567, in execute_directives
     raise ParserException(
 kivy.lang.parser.ParserException: Parser: File "/home/owner/metaprojects/examples/kivymd/toggle_nav_drawer.kv", line 3:
 ...
       1:#:kivy 1.11.0
       2:#:import MDNavigationDrawer kivymd.uix.navigationdrawer.MDNavigationDrawer
 >>    3:#:import NavigationLayout kivymd.uix.navigationdrawer.NavigationLayout
       4:#:import NavigationDrawerDivider kivymd.uix.navigationdrawer.NavigationDrawerDivider
       5:#:import NavigationDrawerToolbar kivymd.uix.navigationdrawer.NavigationDrawerToolbar
 ...
 Module kivymd.uix.navigationdrawer does not contain NavigationLayout
 File "/home/owner/metaprojects/examples/kivymd/toggle_nav_drawer.kv", line 3
Compilation failed.

Additional context
The fix demonstrated above in "after" will be in a PR.

@Poikilos Do you plan to make a PR for this?

@Julian-O I assumed someone with more knowledge of the code could do it better. I could probably, with some time.

@Poikilos I see. I thought you had a PR on your own branch ready to go.
I am looking at the 10 months that have elapsed without any interest from anyone with "more knowledge of the code", and extrapolating that it may be a long time before anyone else gets to this.

@Poikilos I see. I thought you had a PR on your own branch ready to go.
I am looking at the 10 months that have elapsed without any interest from anyone with "more knowledge of the code", and extrapolating that it may be a long time before anyone else gets to this.

Ok I'll double check. Now that I reread my post, I recall I may have made a change on a local install and could potentially do a diff and put it in git.


Implement Node annotation operation

Going to be pushed by 9/20.

Expected by Monday. Chris has a preliminary set of nodes with FDA attributes from Vlado that will go into Automat.
An "annotate K-nodes" function for data we need for Workflow A should be available early next week.

As per meeting on 9/24, can be closed.


[Core] Refine Context.access modes

Description
https://github.com/OpenAssetIO/OpenAssetIO/issues/1016
Adds the kCreateRelated context access mode.
This is an access mode necessary to signal intent to create related entities, rather than mutations/versions of an entity. This comes up particularly in the "folder in a folder" case, where it's very likely that the regular method of determining version/new, (checking that the traitsets match,) will not be enough.
Also remove the idea of multiple writes, due to ther being no coherent use for these within the core API methods themselves, and so this behavior could be more accurately and coherently expressed as part of the UI layer itself.
Dosen't close the linked issue, as workflow tests havn't been explored yet. OpenAssetIO api surface has been prioritised, see https://github.com/OpenAssetIO/OpenAssetIO/issues/1016#issuecomment-1670967369

[x] I have updated the release notes.
[x] I have updated all relevant user documentation.

LGTM. I do note there are quite a few docs remaining that talk about kWrite without mentioning that kCreateRelated is also an option and may be more appropriate, e.g. ForHosts.dox and ForManagers.dox, along with various examples littered throughout the docs.

Good call - added https://github.com/OpenAssetIO/OpenAssetIO/issues/1043, so we can take a step back once all these related PRs are in.


Route name

This PR adds the component name to the route for easier debugging.
Closes #248.
@calebporzio: I haven't tested this code, because of the asset compilation. Am I missing something?

Good stuff, thanks @tillkruss

Also, I built and tested locally, looks good.

already added as a contributor, nice!


Updated deployment location to AIB location for Custom Image Build solution

Sets most resources to deploy to the same location as the AIB image template
Allows the primary and DR locations for the image version to be different than the AIB image template

I'll take a look at it @shawntmeyer

@shawntmeyer just pushed a commit that removes Global location.  Any other feedback?

Remove value of primary location from possible for secondary. That was fast!
Sent via the Samsung Galaxy S22 Ultra 5G, an AT&T 5G smartphone
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Jason Masten @.>
Sent: Wednesday, January 11, 2023 11:40:37 PM
To: Azure/avdaccelerator @.>
Cc: Shawn Meyer @.>; Mention @.>
Subject: Re: [Azure/avdaccelerator] Updated deployment location to AIB location for Custom Image Build solution (PR #278)
@shawntmeyerhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fshawntmeyer&data=05|01||e95650fef0ef4fb03c9b08daf44ec556|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|638090916405807328|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=DJjh7GzluT92CeQbLHMEWKn%2BbANaYzdIAVHRppVbpbM%3D&reserved=0 just pushed a commit that removes Global location. Any other feedback?
—
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Favdaccelerator%2Fpull%2F278%23issuecomment-1379771729&data=05|01||e95650fef0ef4fb03c9b08daf44ec556|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|638090916405807328|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=3URvDTpOtOdDI%2FY8ZymVTgj%2Fm2wH6G6bR9MBKeVFjpw%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FALWLDAJ56CBQH3JBUZ3SU6DWR54LLANCNFSM6AAAAAATYDWDKY&data=05|01||e95650fef0ef4fb03c9b08daf44ec556|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|638090916405807328|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=8rl4d0D81sbtbQKo4G14hcMVegklm2xjHwjSxea3lNY%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>

@shawntmeyer good call! I'll look into that too

@shawntmeyer just added the fix to ensure the primary and DR locations don't match

@jamasten  thanks for your contribution


Getting errro while executing

failed to execute ['dot', '-Tsvg'], make sure the Graphviz executables are on your systems' PATH
#for this code!
from flowgiston import FlowgistonChart
chart = FlowgistonChart()
chart.node("Hello, world")
chart

#1 add Graphviz\bin to the path environment