Create run level links for Beaver

This commit fixes an issue where the beaver deamon would not
restart after each reboot because it is not linked inside of the
legacy init /etc/rcX.d directories. This issue would also cause
commands like 'update-rc.d -f beaver enable 2 3 4 5' to fail.
This fix is done by adding a task inside of
beaver_post_install.yml that makes sure the beaver service is
listed in the appropriate rcX.d folders.
Connects https://github.com/rcbops/rpc-openstack/issues/934

Backport to kilo of #1277

👍  pending checks

not tested but seems legit. 👍


add python3 support

I think we can/should merge this pull request, because by fork is based on this branch. This would make it easier to merge my changes.

changed base to new branch "develop"
@garyp Please review the pull request


Abs time moments

IMPORTANT: Please search among the Pull request before creating one.
Title

Add more efficient computation of variance of time to absorption.
@Marius1311 I've settled on this, feel free to suggest otherwise:

writing 'absorption_pseudotime' and 'absorption_pseudotime_uncert' to adata.obs + to Estimator objcect (previously were written only to estimator I think)
I've kept the time 'til absorption + variance hidden under _absorption_time_mean and _absorption_time_var respectively

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] New feature (non-breaking change which adds functionality)

Description

some improvements to logging.

How has this been tested?

I will add test later today which uses the normal equation.
Closes

#closes #308

How do you transform the variance on the mean absorption time to give you the variance on the inverted mean absorption time?

I've been thinking whether it's correct to call it pseudotime - it's the time until absorption if we start at that given cell. I guess MFPT would be better suited for this - I think a formulat exists for it's variance so I will check it out later.

Right, it's like a mean first passage time for a whole group of cells, i.e. all cells in the absorbing state.

Only that it must be conditional, because a normal mean first passage time in an absorbing chain would be infinity. That's because it's the average over all possible paths that terminate in a given state. Some of these paths may be absorbed by other absorbing states, hence have infinite time. This would make the average infinite. So you have to condition to only average over those paths that actually terminate in your state of interest. So I would call this a conditional mean first passage time.
Compare with https://www.tandfonline.com/doi/abs/10.1080/17509653.2013.783187?journalCode=tmse20

Just read through the abstract, it's exactly that: a conditional mean first passage time.


Nav main item stylings - Main nav

In the main menu, the selected section should be in black and there should be a red underline beneath it, with a height of 2px, per design in Figma.
When a section is chosen and expanded in the main menu, the selected section is not visible, and the selection remains on the "Home" section.

Let's make sure we have a component / variant for this behavior of the dropdown


CoreRemoting interactive with Classic remoting.

I modify Examples code, CoreRemoting with CoreRemoting and classic remoting with classic remoting are work well. But, CoreRemoting call classic remoting or classic remoting call CoreRemoting are failed.
Maybe it is my fault. CoreRemoting can interactive with classic remoting? If support, can provide example?

CoreRemoting is not wire compatible with classic .NET Remoting.
So it is not possible to connect a CoreRemoting client to a classic .NET Remoting server and vice versa.
Both client and server must be migrated from classic .NET Remoting to CoreRemoting.
If a one step migration is not possible, CoreRemoting and classic .NET Remoting can run side by side - using different TCP ports - inside the same processes.

@theRainbird would it be possible to make it compatible and map the types by hand e.g. on the client side before send and after receive (a bit like in #35 )?
would be great to not change all servers and be able to use the CoreRemoting client on a classic remoting server.
if you want maybe i can be of help :) i think it would be best to discuss this first as there might be some extensions points to create in CoreRemoting to implement this as a feature?
greetings

would it be possible to make it compatible and map the types by hand e.g. on the client side before send
and after receive (a bit like in https://github.com/theRainbird/CoreRemoting/issues/35 )?

No, mapping types on client side would not help. CoreRemoting doesn't implement the same network protocol on application layer like Classic .NET Remoting. So a Classic .NET Remoting server is not able to understand and process network packages sent from a CoreRemoting client.
When using CoreRemoting to migrate your application, client and server must be migrated to use CoreRemoting.


Add github-action-locks

This Action let's you add execution locking around your GitHub Account workflows

Fixed, apologies for missing that.

Thanks!


Misleading documentation for Kyma Dashboard access

Description

I want to deploy Kyma on a fresh Gardener cluster and access Busola dashboard. I follow installation guide.
I first provisioned new cluster on Gardener, then deployed kyma with kyma deploy command. According to the installation guide (section Default Installation)I should be able to access dashboard with the link: https://dashboard.kyma-project.io/, but the page does not open.

Expected result

Documentation tells me how to access Kyma dashboard in a convenient way. I propose to link to busola run with docker.
Actual result

I don't know how to access Kyma dashboard.

We should use the kyma dashboard command to open the dashboard


Unorganized list is numeric

There's an ordered list in "Ojects and Data structures" section, but it says  "Well, here's an unorganized list of reasons why" right before the list starts.
So which one is it? This could be a major issue for someone :)
https://github.com/ryanmcdermott/clean-code-javascript/blob/master/README.md#objects-and-data-structures

Duly noted and duly fixed in 50850a937040c341f7ec8aff95b5293b046081a0
Thank you!


Clang compilation fixes (TRIVIAL)

Another round of fixes to make it all compile with clang again.

+1


Upgrading to 3.1 best practices.

I am upgrading from 3.0 to 3.1 and it appears that the preferred method to authenticate routes is to nest the routes under an authenticated parent route.
So in order to do this migration I would have to move all my routes, controllers, templates and rename all transition to functions.
I wouldn't have a problem with this but in this case all routes are authenticated except for one login page.
It seems like there should be a way to use the applications route beforeModel hook to do the authentication and skip it only on the login route.
I was hoping the following would work but it is buggy.  It works on page load but if you transition to a route after after the application has loaded it is not requiring authentication.
// routes/application.js
export default class ApplicationRoute extends Route {
  @service session;

  beforeModel(transition) {
    if (transition.targetName !== 'login') {
      this.session.requireAuthentication(transition, 'login');
    }
  }
}

// routes/login.js
export default class LoginRoute extends Route {
  @service session;

  beforeModel(transition) {
    this.session.prohibitAuthentication('index');
  }
}

Is there any other way to pull this off without reworking my whole application?
Thanks for taking the time to look at this.
~ aaron

A good practice is to wrapper authenticated routes:
Router.map(function() {
  this.route('login');

  this.route('authenticated', { path: '/' }, function() {
    // your authenticated routes.
  });
});

If you can not do that, your approach should work.

I was hoping the following would work but it is buggy. It works on page load but if you transition to a route after after the application has loaded it is not requiring authentication.

That is the expected behavior. The requireAuthentication  is not called every transition, it is just called on page load.

@alejandrodevs the backed does return a 401 and the ember-simple-auth used to handle the error using the application route mixin.
Should this error be handled in the application route or the data adapter?

Ember simple auth handle 401 in the DataAdapterMixin which is not longer recommended because Mixins deprecation. Now you can remove completely the mixn and add to your application adapter:
  handleResponse(status, ...manyMoreArgs) {
    if (status === 401) this.session.invalidate();
    return super.handleResponse(...arguments);
  }

@alejandrodevs thank you.  This should be added to the readme.


drivers: modem: AT commands sent before OK from previous is received

Describe the bug
The gsm_ppp.c driver send the first setup_modem_info_cmds before the OK is received from the last setup_cmds.
This is due to no delay after the last setup_cmds:
https://github.com/zephyrproject-rtos/zephyr/blob/265394e30efd3f31fdbece6e49fae73a60eec8d9/drivers/modem/modem_cmd_handler.c#L572
Expected behavior
Delay between all AT commands.

I had NO CARRIER issues on a BG95 modem and found this inconsistency, and was not sure what the impact was, therefore I opened the issue.
With a logic analyzer there was a small overlap where the next AT command was sent before both \r and \n was received.
However I wondered where the 50 ms came from in the first place, but was not able to find anything in 3GPP standards.

The only reason of a delay is when modem needs some time after executed command in order to prepare for the next one. I've seen such requirements stated in modem datasheets some time ago, though I don't remember which ones they were. I guess that this 50ms delay is just an arbitrary number. Maybe it was a requirement of the first modem that was tested with that framework, who knows.
I think that we still need such delay mechanism for modems that require that. Such delay should be configurable in Device Tree, so that by default there is no delay (because there is no reason to have one in general, we are just operating on AT commands).
Delay should not only be part of modem_cmd_handler_setup_cmds(), but it should rather be enforced between every AT command, so that there is the same delay between commands from subsequent modem_cmd_handler_setup_cmds() calls (which is what you try to achieve, right?). Question is, what to do with commands that operate on sockets? Are they affected as well?

With a logic analyzer there was a small overlap where the next AT command was sent before both \r and \n was received.

This is related, but a bit different issue from delaying between sent commands. It might be worth to wait for \r\n characters before assuming that AT command execution was finished.

Very good points.
Can you elaborate on the socket issue? Do you think there is a need to add delays there as well?

Delay should not only be part of modem_cmd_handler_setup_cmds(), but it should rather be enforced between every AT command, so that there is the same delay between commands from subsequent modem_cmd_handler_setup_cmds() calls (which is what you try to achieve, right?)

Yes.
These are the issues I see:

Fix bug where delay is not the same between the AT commands (which is what I attempt to fix here
Possible to set delay between AT commands via DTS.
Maybe an issue on commands on sockets.
Await "\r\n" before issuing another AT command.

Can you elaborate on the socket issue? Do you think there is a need to add delays there as well?

Sure. I though about alternative approach to have delays between issued AT commands. The idea was to add a delay in modem_cmd_send*() family of functions, so that delay is enforced between each, no matter from where it is issued (right now delays are only part of modem_cmd_handler_setup_cmds*(). The biggest issue with that alternative approach is that there would be delay between each packet transmission on socket, so network performance would suffer. Maybe we will need that at some point, maybe not. It would be better to have a hardware that requires such behavior before implementing it.
So in summary, I have commented about socket operations because of two reasons:

maybe we need to add delay between socket operations as well (it is likely that modem firmware requires delay between each AT command, no matter if that is "setup phase" or "networking/socket communication"),
I have concerns about networking performance if we decide to add delays between each AT command (not only between AT commands in "setup phase").


Orb Migrate Docker via UI hangs when there is not enough disk space

Describe the bug
When I try to use the UI to migrate from Docker, it just hangs at "Gathering info":

Using the CLI shows the underlying issue:
➜  ~ orb migrate docker
INFO[0000] Starting Docker Desktop
INFO[0000] Gathering info
INFO[0000]                                               progress=2.3255813953488373
Not enough free disk space in Docker Desktop: 5 GB required. Please free up space or increase virtual disk size in Docker Desktop settings.
➜  ~

To Reproduce

Ensure there's not enough disk space for this process (less than 5 GB).
Use the UI to migrate from Docker to Orb.

Expected behavior
Same error that shows in the CLI should be shown rather than hanging forever.
Diagnostic report (REQUIRED)
OrbStack info:
Version: 1.7.2
Commit: 50f93373f351fe839fd72948e6aad032774c0f6c (v1.7.2)
System info:
macOS: 14.6.1 (23G93)
CPU: arm64, 12 cores
CPU model: Apple M3 Pro
Model: Mac15,7
Memory: 36 GiB
Full report: https://orbstack.dev/_admin/diag/orbstack-diagreport_2024-09-03T16-38-25.731310Z.zip

Screenshots and additional context (optional)
No response

Fixed for the next version.

Released in v1.7.3.

It's still happening.


"Work Queues" example

Work Queues
In this one we'll create a Work Queue that will be used to distribute time-consuming tasks among multiple workers.
The main idea behind Work Queues (aka: Task Queues) is to avoid doing a resource-intensive task immediately and having to wait for it to complete. Instead we schedule the task to be done later. We encapsulate a task as a message and send it to the queue. A worker process running in the background will pop the tasks and eventually execute the job. When you run many workers the tasks will be shared between them.

Closed by #142561b7.


Time to train

Hello, great work with this paper and repo!
I would like to ask you how much time you spent training the model (for the dataset Phoenix12) and what kind gpu you used for the training. Because I am trying to replicate it but with other dataset (specificly the Phoenix14-T), and in my first test I spent around 14h to train 10 epochs. I used a TitanXP with 12Gb for the training and a batch = 1.
Thank you again for your work and congratulation for this repo.

Thanks for your attention, it takes about half an hour to train one epoch on Phoenix14 with batch size=2 on a single 3090, and the total training process (40 epochs) costs about 20 hours. Using multiple GPUs can speed up this process.

Great! Thank you for your early answer.


IOS / Android - Extra spacing above the header at additional details page

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Click on Fab icon >
Send money >
onfido verification >
Additional details page

Expected Result:
There shouldn't be extra space above the header
Actual Result:
There's extra space above the header on native
Workaround:
unknown
Platform:

Where is this issue occurring?

iOS
Android

Version Number: 1.1.86-4
Reproducible in staging?: Y
Reproducible in production?: Y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

![180063394-2ead695b-eb00-4ad2-b489-1c2fd7988b02](https://user-images.githubusercontent.com/43995119/184224579-a509c5e3-615a-495d-bbf3-ffeddeb54d7a.png)
Expensify/Expensify Issue URL:
Issue reported by: @thesahindia
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1659042770219849
View all open jobs on GitHub

I can't triage this properly because I have "UK" wallet so can only opt to send money outside expensify.

Can you provide more specific reproduction steps please @thesahindia, not sure exactly how you are getting to that screen

Bump on the above request @thesahindia

I tested it by rendering this page at a different route. But there are two ways to reach this page ( I found them on some other issues )
1st way (not really sure about this but you have to in some beta for this) -

Click on Fab icon
Send money
Complete the Onfido verification

2nd -

Go to Settings
Go to Payments
Click on Transfer Balance (you must have some balance available)

@mvtglobally, please help if you have the exact steps to repro

Found the steps -

The account already has a bank account added.
Hit green plus button
Send money
Enter amount
Enter email
Pay with Expensify

cc: @MitchExpensify

Thanks @thesahindia, I followed those steps and still cannot reproduce this

@mvtglobally mind giving this a buddy check for me? Are you able to reproduce this?

Lil bump @mvtglobally, thank you!

@MitchExpensify I was not able to repro, but asked the team as well to see if anyone can repro

Thank you! Will see if they can and go from there

Nobody able to reproduce, closing

@MitchExpensify, this is still reproducible
(ss from latest main branch build)

Got a PR up to fix this and a few related bugs: https://github.com/Expensify/App/pull/11257

Can we reopen as this is eligible for the reporting?

@MitchExpensify Let's make sure @thesahindia gets his reporting bonus here. Thanks

Offer sent - Upwork Job

Offer sent - Upwork Job

Accepted 🚀

Paid, thanks @thesahindia !


Get our TOS rated by TOS;DR

Terms of Service; Didn't Read is

a user rights initiative to rate and label website terms & privacy policies, from very good Class A to very bad Class E.

I think it would be a good step to have our Terms reviewed by this peer-reviewed body.
Their Get Involved section contains information on how to submit for review.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

This is something we could try again with #4117.

Closing in light of our decision to shut down Gratipay.
Thank you all for a great run, and I'm sorry it didn't work out! 😞  💃

Interestingly enough, Gratipay was just added a week ago: https://github.com/tosdr/tosback2/commit/107471c9669c54d62911154a165f15bb1188255e


rehex: 0.5.3 -> 0.5.4

Description of changes
https://github.com/solemnwarning/rehex/releases/tag/0.5.4
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

@ofborg eval

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

1 package built:

rehex


Create Kruskal's Algorithm

Kruskal’s algorithm to find the minimum cost spanning tree uses the greedy approach. The Greedy Choice is to pick the smallest weight edge that does not cause a cycle in the MST constructed so far.

PR Checklist:

[ ] My submission is formatted according to the guidelines in the contributing guide
[ ] My addition is on refer on the language README.md file
[ ] My addition does not have a spelling problem
[ ] My submission has a proper and user-friendly description of the algorithm
[ ] My submission has the time complexity of the algorithm
[ ] My submission has sample input-output of the program (NOT FOR PYTHON)

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[ ] New algorithm
[ ] Optimization in previous algorithms
[ ] Code style update
[ ] Refactor
[ ] Documentation
[ ] Other, please describe:

Briefly describe the changes in this PR

Other information:

Hi @Purvesh77,
Could you help me to check which pull request is the correct one #991 and #992?
These pull request will be closed, and please re-open it for the correct one.


refactor: rename Polygon2D to Polygon

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

63 of 63 (100.0%)  changed or added relevant lines in 11 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 82.609%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5092

Relevant Lines:
6164

💛  - Coveralls


[bug] v2 sep from @tauri-apps/api/path does not return correct separator

Describe the bug
sep  from "@tauri-apps/api/path" resolves to
function sep() {
  return window.__TAURI_INTERNALS__.plugins.path.sep;
}

instead of the correct separator.
I've enabled the following capabilities
"path:allow-basename",
"path:allow-dirname",
"path:allow-extname",
"path:allow-join",
"path:allow-normalize",
"path:allow-resolve",
"path:allow-resolve-directory",
"path:allow-is-absolute",

Reproduction
import { sep } from "@tauri-apps/api/path";
console.log(sep);

Expected behavior
Return the current separator
Full tauri info output
[✔] Environment
    - OS: Mac OS 14.4.0 X64
    ✔ Xcode Command Line Tools: installed
    ✔ rustc: 1.77.2 (25ef9e3d8 2024-04-09)
    ✔ cargo: 1.77.2 (e52e36006 2024-03-26)
    ✔ rustup: 1.27.0 (bbb9276d2 2024-03-08)
    ✔ Rust toolchain: stable-aarch64-apple-darwin (default)
    - node: 21.6.1
    - pnpm: 8.15.2
    - npm: 10.2.4
    - bun: 1.0.26

[-] Packages
    - tauri [RUST]: 2.0.0-beta.19
    - tauri-build [RUST]: 2.0.0-beta.15
    - wry [RUST]: 0.39.5
    - tao [RUST]: 0.28.0
    - tauri-cli [RUST]: 2.0.0-beta.17
    - @tauri-apps/api [NPM]: 2.0.0-beta.11
    - @tauri-apps/cli [NPM]: 2.0.0-beta.17

[-] App
    - build-type: bundle
    - CSP: unset
    - frontendDist: ../dist
    - devUrl: http://localhost:1420/
    - framework: React
    - bundler: Vite

Stack trace
No response
Additional context
No response

Was changed to a function in #7160


with-styled-components/README.md Update

README File Change Word
Before
yarn create next-app --example with-styled-components with-styled-components-app
After
yarn create-next-app --example with-styled-components with-styled-components-app

yarn create is a yarn subcommand, it's correct.


[BUG] Event generation fails for FFImageLoading

Describe the bug
When you install Pharmacist in a Xamarin.Forms project that has FFImageLoading it makes the event generation to fail. It can't basically handle the events that FF exposes cause they are in a static class with all the events.
Steps To Reproduce

Install Pharmacist latest (1.2.2) into a Xamarin.Forms app that has FFImageLoading as well
Build the project
It fails on building

Expected behavior
The events should be generated.
Screenshots

I'm doing some work on Pharmacist today so will add this to the test bed.

I believe this problem has been solved since it's not happening for me anymore.

Hi @glennawatson.
I'm gonna have a try tomorrow to see what happens.
Thank you!

Hey @glennawatson, sorry for the late answer, I tried here again with the version 1.5.3 and FF latest and the same issue still happens.
Just to mention, the packages I have installed here are exactly:

Pharmacist.MsBuild 1.5.3
Pharmacist.Common 1.5.3
Xamarin.FFImageLoading.Forms <IP_ADDRESS>2
Xamarin.FFImageLoading.Svg.Forms <IP_ADDRESS>2
Xamarin.FFImageLoading.Transformations <IP_ADDRESS>2

The one that produces the issue to me is the Xamarin.FFImageLoading.Forms itself.
This is what gets generated:
Pharmacist.NuGet.cs.txt

@winterdouglas what does your project look like? Are you doing this in your .net standard shared project or on the final project?

These packages are installed in the .net standard forms project itself, not in the platform projects. Nothing special about them.
Here's a sample project where the error happens:
PharmacistFFImageLoading.zip
It's a basic Xamarin.Forms app and the only packages that are installed there are FFImageLoading.Forms and Pharmacist.MsBuild and Pharmacist.Common.

This was solved with 1.5.14 release.

Gonna give it a try and will let you know. Thanks @glennawatson


client.keepConnected does not seem to function properly

Judging by the code it seems that this should handle automatic re-connection to a disconnected or dropped FreeSwitch connection, however it never seems to work or reconnect in my case. When sending commands it always returns and error:
{ [Error]   res: {},   args:     { when: 'send on closed socket',      command: 'sendmsg',      args:        { 'execute-app-name': 'reloadxml',         'execute-app-arg': '',         'call-command': 'execute' } } }
The code I am trying to use is as follows:
let eslClient; let client = esl.client(function () {     eslClient = this;   }, (err) => {     log('ERROR!', err);   } ); client.keepConnected(8021, '<IP_ADDRESS>')
So 'eslClient' is used to call on the commands like reloadxml, however when I stop FreeSwitch, then start it back up, it never seems to create the new socket connection. You help would be really appreciated. Thanks

Just finished this solution and seems to do the job:
`let client;
// Connect "inbound" to Freeswitch ESL:
const reconnect = () => {
console.log('Reconnecting!');
if (client) {
client.destroy();
}
client = esl.client(function() {
eslClient = this;
}, (err) => {
log('ERROR!', err);
}
);
client.connect(8021, '<IP_ADDRESS>');
client.on('error', function(err) {
log('ERROR!', err);
if (err.code === 'ECONNREFUSED') {
setTimeout(() => {
reconnect();
}, 5000);
}
});
}
reconnect();`

@newtondev hmmm.. the code was changed recently (in 7.0.0), but older versions should be as bad as the current one. I'm not sure why I coded keepConnected this way, there's no way it can work. :(
One needs to (re)create a new socket as you described. I'll factorize keepConnected out into a new object.

This patch removes the broken keepConnected.
The new feature is exported as reconnect( connect_options, [options,] handler [,report] ) and works similarly to client([options,]handler[,report]) but keeps the connection open. It returns a function which will terminate the connection attempts:
var stop_client = FS.reconnect({host:'<IP_ADDRESS>',port:8021},handler);
…
stop_client()

Integrated in v8.0.0


open-telemetry stable HTTP semantic conventions

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you expect to see?
With the HTTP semantic conventions declared stable in v1.23.0 and it's big impact on all the attributes, Traefik should support it already in v3.0 before GA.
This would potentially avoid a lot of extra work and frustration later on, if enabling the brand new OTel functionality and having to update/refactor the whole observability stack to deal with all the changed attributes a view releases later.

Hey @schewara,
Thanks for your suggestion.
We are interested in this issue but are unsure about the use case and the traction it will receive, so we are going to leave the status as "kind/proposal" to give the community time to let us know that they would like this.
We will reevaluate as people respond.

some of these are implemented in https://github.com/traefik/traefik/pull/10223/files#diff-f1e0d24911164f850e8c1e92b5e73073d417eb5ca067ad3feba01bc6d6211082R109-R130

Hey @schewara,
Thanks for your suggestion.
We are interested in this issue but are unsure about the use case and the traction it will receive, so we are going to leave the status as "kind/proposal" to give the community time to let us know that they would like this.
We will  as people respond.

traefik will got benefits from APMs evolution for free

For example

APM vendors provide by default a filtering UI for trace with a specific name.
If you follow semconv, we can make an opentelemetry-semconv Grafana dashboard
If you follow your own metrics, I will make a dashboard dedicated to traefik.

This is more labor intensive and will slow down maintenance if I am busy.

Closed by https://github.com/traefik/traefik/pull/10223

@mmatur

Please re-open this.
Traefikv3 supports Opentelemetry, but metric semantics doesn't conforts semantic conventions of Opentelemetry.

[traefik v3(docs)]
traefik_entrypoint_requests_total
traefik_entrypoint_requests_tls_total
traefik_entrypoint_request_duration_seconds
traefik_entrypoint_requests_bytes_total
traefik_entrypoint_responses_bytes_total

[Opentelemetry (semconv docs)]
https://github.com/open-telemetry/semantic-conventions/blob/v1.23.0/docs/http/http-metrics.md

thank you!


[Bug] After switch hanging protocol from primaryAxial to default, viewport image looks like broken

Describe the Bug
Hi guys,
I am working with a fork version which forked since 4 months ago.
I faced with an issue when trying switch hanging protocol from mpr to default.
The first time loaded and render is ok.
I switch to protocol primaryAxial is ok. But when switch back to protocol default, it looks like image is crashed.
But one more time when I switch to protocol primaryAxial, the viewports look fine.

Maybe I was wrong in wording, so I raise this issue to ask about some hints to debug this issue
Steps to Reproduce

Run app
Open the first study CTA Head and Neck

Switch to hanging protocol primaryAxial
Switch back to handing protocol default

The current behavior
After switch hanging protocol from primaryAxial to default, viewport image looks like broken
The expected behavior
I expect that viewport image will be not broken
OS
macOS 14.5
Node version
v18.16.1
Browser
Chrome 125.0.6422.61

Or does stack viewport corrupted?

Now I can reproduce the error.
I made a command that toggle between volume and stack viewport.
toggleSwitchMPR: async ({ orientation }) => { 			// console.log('switch viewport to MPR - orientation', orientation); 			const activeViewportID = viewportGridService.getActiveViewportId(); 			const viewportInfo = cornerstoneViewportService.getViewportInfo(activeViewportID); 			const currentViewportData = viewportInfo.viewportData; 			const displaySetInstanceUID = currentViewportData.data.length 				? currentViewportData.data[0].displaySetInstanceUID 				: currentViewportData.data.displaySetInstanceUID; 			const displaySet = displaySetService.getDisplaySetByUID(displaySetInstanceUID); 			const dataSource = extensionManager.getActiveDataSource()[0]; 			if (currentViewportData.viewportType === 'stack') { 				const viewportOptions = { 					viewportType: 'volume', 					orientation: orientation 				}; 				const viewportData = await cornerstoneCacheService.createViewportData([displaySet], viewportOptions, dataSource); 				// console.log(viewportData); 				const publicViewportOptions: PublicViewportOptions = { 					id: orientation, 					viewportType: 'volume', 					toolGroupId: 'mpr', 					orientation: orientation 				}; 				cornerstoneViewportService.setViewportData( 					activeViewportID, 					viewportData, 					publicViewportOptions, 					viewportInfo.displaySetOptions 				); 			} else { 				const viewportOptions = { 					viewportType: 'stack' 				}; 				const viewportData = await cornerstoneCacheService.createViewportData([displaySet], viewportOptions, dataSource); 				// console.log(viewportData); 				const publicViewportOptions: PublicViewportOptions = { 					viewportType: 'stack', 					toolGroupId: 'default', 					initialImageOptions: { 						preset: 'middle' 					} 				}; 				cornerstoneViewportService.setViewportData( 					activeViewportID, 					viewportData, 					publicViewportOptions, 					viewportInfo.displaySetOptions 				); 			} 		}

@ahlaughland I reallied that I am using this config to reduce memory sizing
useNorm16Texture: true,
preferSizeOverAccuracy: true

Also found a config about rendering engine
If I config cornerstone like this, the issue is solved
cornerstone.setConfiguration({
	...cornerstone.getConfiguration(),
	rendering: {
		...cornerstone.getConfiguration().rendering,
		strictZSpacingForVolumeViewport: appConfig.strictZSpacingForVolumeViewport
	},
	enableCacheOptimization: false
});

Thanks, I will research this enableCacheOptimization option.

Can you check latest OHIF which has latest Cornerstone3D 2.0 additions?

We released OHIF 3.9 which address this issue.


Rotativa pdf bottom position is wrong after upgrade visual studio 17.8

After upgrade to visual studio version 17.8, there is somethings wrong in pdf bottom position as below
I'm currently using Rotativa.AspNetCore 1.3.2 with the below parameters
return new ViewAsPdf(view_name, model, view_data) {     PageSize = Rotativa.AspNetCore.Options.Size.A4,     PageOrientation = Rotativa.AspNetCore.Options.Orientation.Portrait,     PageMargins = new Rotativa.AspNetCore.Options.Margins(2, 0, 5, 0),     CustomSwitches = "--disable-smart-shrinking",     ContentDisposition = Rotativa.AspNetCore.Options.ContentDisposition.Inline };

Visual studio 17.6

After upgrade to 17.8

css for the bottom div element
`#invoice_bottom {
padding-top: 5px;
margin-left: 30px;
border-top: 1px solid #ddd;
height: 100px;
width: 720px;
position: fixed;
left: 0;
bottom: 10px;
}

@media print {
@page {
		size: A4 portrait; /* can use also 'landscape' for orientation */
		margin: 0;
}

	html, body {
    	width: 210mm;
    	height: 297mm;
    	display: block;
	}

}
`
I have no clue on this problem at all.
Could you please check if anything changed with the latest version of Visual studio build environment ?

In case you haven't fixed this: you encase a part of your CSS in @media print. But you haven't provided the --print-media-type custom switch. So this CSS is ignored by wkhtmltopdf. It should have been ignored regardless of the version of the IDE you are running. Wkhtmltopdf shouldn't be affected by the version of Visual Studio.
I recreated this, with no CSS except for the CSS you provided (plus background: yellow). Because the height of the document isn't specified in the CSS, the fixed element is placed on top of the lowest element:

Now I add the custom --print-media-type flag:
return new ViewAsPdf("Contact", null, ViewData)
{
    PageSize = Rotativa.AspNetCore.Options.Size.A4,
    PageOrientation = Rotativa.AspNetCore.Options.Orientation.Portrait,
    PageMargins = new Rotativa.AspNetCore.Options.Margins(0, 0, 0, 0),
    CustomSwitches = "--disable-smart-shrinking --print-media-type",
    ContentDisposition = Rotativa.AspNetCore.Options.ContentDisposition.Inline
};

And the result is as expected:

I set the margins to 0 (new Rotativa.AspNetCore.Options.Margins(0, 0, 0, 0)) as the fixed element otherwise creates another page.
If you don't want to add a custom switch you can also delete the @media print in the CSS.
As it is correct behavior that the print media was ignored (thus not aligning the footer element on the bottom), my guess is that something in your content or CSS changed the height of the document, thus moving the footer.


Incorrect tooltip for Execute Cell button on Linux

Describe the bug
I'm running Polyglot on Pop!_OS 22.04 (a flavor of Ubuntu 22.04) and building text for the book and noticed an incorrect shortcut key for the "Execute Cell" button describing the shortcut as Super + Enter. Super is a substitute for the Windows key on most keyboards. The correct shortcut is still Control + Enter as it is in Windows.
Please complete the following:
Which version of .NET Interactive are you using? (In a notebook, run the #!about magic command. ):

OS

[ ] Windows 11
[ ] Windows 10
[ ] macOS
[x] Linux (Pop!_OS 22.04)
[ ] iOS
[ ] Android

Browser

[ ] Chrome
[ ] Edge
[ ] Firefox
[ ] Safari

Frontend

[ ] Jupyter Notebook
[ ] Jupyter Lab
[ ] nteract
[x] Visual Studio Code
[ ] Visual Studio Code Insiders
[ ] Visual Studio
[ ] Other (please specify)

Screenshots

This is likely more than just the execute cell shortcut. I just noticed a mistake on the validate markdown shortcut as well:

I'd recommend reviewing Linux shortcuts for correctness.

This is VS Code functionality, not Polyglot Notebooks.
@rebornix Is this on your radar?

We are using ctrl+enter as the default keybindings
primary: KeyMod.WinCtrl | KeyCode.Enter,
win: {
	primary: KeyMod.CtrlCmd | KeyMod.Alt | KeyCode.Enter
},

@IntegerMan do you have the Jupyter Keymap installed?

@rebornix I did indeed. I didn't think I did since I have a very minimal install in a new workspace, but perhaps I added it and don't remember doing so. Disabling that gives me better tooltips for the Stop Editing Cell, but not the execute cell shortcut:

Note that the keymap is disabled in this screenshot.


Improved conda-build linking phase in conda-build 3.1.3

Just a heads-up, as I know you've had issues with the "fixing linking" time in conda-build.  This PR just went in: https://github.com/conda/conda-build/pull/2564
It very dramatically improves this issue.  This will be in conda-build 3.1.3.  Conda-forge is not quite using conda-build 3 yet - that was last discussed in our meeting on Monday.  Agenda/notes are at https://paper.dropbox.com/doc/Compilers-conda-build-3-update-20171211-JSvRO2Lk2Z7Bbf2zDaG9p

Thanks @msarahan. This should also enable sagelib which had hundreds of dylibs.

Works great.


Check if a response has been sent?

Is there any way to test if a response has been sent already?
For context, I'm finding that some non-existing url queries end up just hanging, so I created
# Timeout Handling
((req,res) ->
    setTimeout(
        ->
            # Prepare
            requestInfo = {url: req.headers.host+req.url, ip: req.connection.remoteAddress, status: res.statusCode}

            # Attempt Timeout
            try
                res.send(408) # Request Timeout
                console.log 'request timeout:', requestInfo
                #res.end() # End Response
                #console.log 'end response:', requestInfo
            catch err
                # Chances are the request sent fine
                false
        30*1000
    )
)(req,res)

but even with the catch it restarts the entire process

use res.writableEnded


JSON schema evaluation fails for Kafka Source Connector

Hello, I am using this library with Kafka source connector. Below are my connect worker configurations -
    "key.converter.region"                        = "us-east-1"
    "key.converter.registry.name"                 = var.schema_registry_name
    "key.converter.compatibility"                 = "FULL"
    "key.converter.schemaAutoRegistrationEnabled" = true
    "key.converter.dataFormat"="JSON"
    "key.converter.avroRecordType"="GENERIC_RECORD"
    "key.converter.schemaNameGenerationClass" = "<CustomNamingStrategyClass>"
    "value.converter.region"                        = "us-east-1"
    "value.converter.registry.name"                 = var.schema_registry_name
    "value.converter.compatibility"                 = "FULL"
    "value.converter.schemaAutoRegistrationEnabled" = true
    "value.converter.dataFormat"="JSON"
    "value.converter.avroRecordType"="GENERIC_RECORD"
    "key.converter.schemaNameGenerationClass" = "<CustomNamingStrategyClass>"

As you see, compatibility mode has been set to FULL but updating my source db table with any optional field gives me error saying Error while fetching or registering schema.  I have also tries most of the compatibility modes but end up getting same issue.
Here is the stack trace of the error -
[Worker-0acd35d51b07406e0] Caused by: com.amazonaws.services.schemaregistry.exception.AWSSchemaRegistryException: Failed to get schemaVersionId by schema definition for schema name = users
--

[Worker-055733311b878809b] Caused by: org.apache.kafka.connect.errors.DataException: Converting Kafka Connect data to byte[] failed due to serialization error:
--
[Worker-055733311b878809b] 	at com.amazonaws.services.schemaregistry.kafkaconnect.jsonschema.JsonSchemaConverter.fromConnectData(JsonSchemaConverter.java:145)
[Worker-055733311b878809b] 	at org.apache.kafka.connect.storage.Converter.fromConnectData(Converter.java:63)
[Worker-055733311b878809b] 	at org.apache.kafka.connect.runtime.WorkerSourceTask.lambda$convertTransformedRecord$3(WorkerSourceTask.java:316)
[Worker-055733311b878809b] 	at org.apache.kafka.connect.runtime.errors.RetryWithToleranceOperator.execAndRetry(RetryWithToleranceOperator.java:156)
[Worker-055733311b878809b] 	at org.apache.kafka.connect.runtime.errors.RetryWithToleranceOperator.execAndHandleError(RetryWithToleranceOperator.java:190)
[Worker-055733311b878809b] 	... 11 more
[Worker-055733311b878809b] Caused by: com.amazonaws.services.schemaregistry.exception.AWSSchemaRegistryException: Exception occurred while fetching or registering schema definition = <schema definition>
[Worker-055733311b878809b] 	at com.amazonaws.services.schemaregistry.common.SchemaByDefinitionFetcher.getORRegisterSchemaVersionId(SchemaByDefinitionFetcher.java:99)
[Worker-055733311b878809b] 	at com.amazonaws.services.schemaregistry.serializers.GlueSchemaRegistrySerializationFacade.getOrRegisterSchemaVersion(GlueSchemaRegistrySerializationFacade.java:86)
[Worker-055733311b878809b] 	at com.amazonaws.services.schemaregistry.serializers.GlueSchemaRegistryKafkaSerializer.serialize(GlueSchemaRegistryKafkaSerializer.java:113)
[Worker-055733311b878809b] 	at com.amazonaws.services.schemaregistry.kafkaconnect.jsonschema.JsonSchemaConverter.fromConnectData(JsonSchemaConverter.java:143)
[Worker-055733311b878809b] 	... 15 more
[Worker-055733311b878809b] Caused by: com.amazonaws.services.schemaregistry.exception.AWSSchemaRegistryException: Failed to get schemaVersionId by schema definition for schema name = platform_users
[Worker-055733311b878809b] 	at com.amazonaws.services.schemaregistry.common.AWSSchemaRegistryClient.getSchemaVersionIdByDefinition(AWSSchemaRegistryClient.java:148)
[Worker-055733311b878809b] 	at com.amazonaws.services.schemaregistry.common.SchemaByDefinitionFetcher$SchemaDefinitionToVersionCache.load(SchemaByDefinitionFetcher.java:110)
[Worker-055733311b878809b] 	at com.amazonaws.services.schemaregistry.common.SchemaByDefinitionFetcher$SchemaDefinitionToVersionCache.load(SchemaByDefinitionFetcher.java:106)
[Worker-055733311b878809b] 	at com.google.common.cache.LocalCache$LoadingValueReference.loadFuture(LocalCache.java:3529)
[Worker-055733311b878809b] 	at com.google.common.cache.LocalCache$Segment.loadSync(LocalCache.java:2278)
[Worker-055733311b878809b] 	at com.google.common.cache.LocalCache$Segment.lockedGetOrLoad(LocalCache.java:2155)
[Worker-055733311b878809b] 	at com.google.common.cache.LocalCache$Segment.get(LocalCache.java:2045)
[Worker-055733311b878809b] 	at com.google.common.cache.LocalCache.get(LocalCache.java:3951)
[Worker-055733311b878809b] 	at com.google.common.cache.LocalCache.getOrLoad(LocalCache.java:3974)
[Worker-055733311b878809b] 	at com.google.common.cache.LocalCache$LocalLoadingCache.get(LocalCache.java:4935)
[Worker-055733311b878809b] 	at com.amazonaws.services.schemaregistry.common.SchemaByDefinitionFetcher.getORRegisterSchemaVersionId(SchemaByDefinitionFetcher.java:74)
[Worker-055733311b878809b] 	... 18 more

[Worker-0acd35d51b07406e0] Caused by: com.amazonaws.services.schemaregistry.exception.AWSSchemaRegistryException: Schema Found but status is FAILURE

I tried using AVRO converter with the same worker properties and it allows me update the source table with optional fields. I was able to get the updated data in the topic.
Is there any property I am missing for JSON schema, or it's not supported?

I am having a similar issue, except my error at the bottom of the stack trace is:
[Worker-0dabc170e2f071b0e] Caused by: software.amazon.awssdk.services.glue.model.EntityNotFoundException: Schema version is not found. (Service: Glue, Status Code: 400, Request ID: <requestID>),
even though its clearly there by the name that they show one error up...

The compatibility rules can vary between data formats. Without the schema, it is hard to debug the compatibility rule. Can you please provide a simplified version of the schema you are trying to use? Also, Full is more restrictive compatibility mode as it mandates the changes to be both forward and backward compatible.


Rename Chroma Palette -> Palettes & Hues 🎨

Hey @ajayts07, I actually had to rename my extension, do you mind updating the README, against this PR?
Thanks.

Sure I will
On Thu, 31 Oct 2024 at 10:02, Mayank Singh @.***> wrote:

Hey @ajayts07 https://github.com/ajayts07, I actually had to rename my
extension, do you mind updating the README, against this PR?
Thanks.
You can view, comment on, or merge this pull request online at:
https://github.com/themeselection/best-chrome-extensions/pull/6
Commit Summary

e26c1e7
https://github.com/themeselection/best-chrome-extensions/pull/6/commits/e26c1e7b0af4397a495d529134c01294faf7a9e3
Rename

File Changes
(1 file
https://github.com/themeselection/best-chrome-extensions/pull/6/files)

M README.md
https://github.com/themeselection/best-chrome-extensions/pull/6/files#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5
(2)

Patch Links:

https://github.com/themeselection/best-chrome-extensions/pull/6.patch
https://github.com/themeselection/best-chrome-extensions/pull/6.diff

—
Reply to this email directly, view it on GitHub
https://github.com/themeselection/best-chrome-extensions/pull/6, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AZIAQYYFEVPSPKJ7O7TDGELZ6GXFHAVCNFSM6AAAAABQ5QJXJ6VHI2DSMVQWIX3LMV43ASLTON2WKOZSGYZDKOJUGU2DKNI
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Anand Patel
Marketing Executive
ThemeSelection


Bridge branch 20230919 152659

Changelog category :

New Feature
Bug Fix
Improvement
Performance Improvement
Backward Incompatible Change
Build/Testing/Packaging Improvement
Documentation (changelog entry is not required)
Other
Not for changelog (changelog entry is not required)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
...
Documentation entry for user-facing changes

[ ] Documentation is written (mandatory for new features)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: dmthuc:x: TKaxeYou have signed the CLA already but the status is still pending? Let us recheck it.


ignore coverage on !CI_DEBUG

Ignore coverage when CI_DEBUG is false as never hit in tests.
Checklist:

[x] Securely signed commits

travis build green 🎉


Added UI Changes

Modified the Home page and added sidebar

@Sriramkashyapks as we discuss in call please rephrase the issue by considering the react js technology


validate the edge name at the task contex write

If non existing edge is given, we should throw an error

This is fixed now


Anchor point's lands on wrong position

I'm developing a one page website, with multiple sections, which also include some links to the subpages.
The problem I have is, that since it is a single page, I'm using anchor links on subpages to get the user back where he left.
But in safari, when the user comes back from the subpage, the index page opens on the anchor link location, but the position is wrong, about 70px more to the bottom of the anchor point in html. But after the anismition load's the page, then the anchor jumps to it's place.
The weirdest part is, that this doesn't happen when you refresh the website with the anchor link inside the link like index.html#section, it happens only when you link back from subpage.

Here is the link to the developing page, it might help you.


Clarify installation instructions

When I read the installation instructions, it didn't come across to me that the npm package could be used without a CJS module bundler. I tried to reorganize the instructions here to make it a bit more clear.
===
The rest of the commit msg:

The previous instructions may have come across that the only way to use Redux with npm was with a module bundler, like Browserify or Webpack.

This reorganizes the text in an effort to more explicitly explain that it's only the extensions require a CJS module bundler.

Let's add an explicit paragraph explaining how to install and use the UMD build.
It's available both inside NPM package and standalone via npmcdn.
In fact you can even use it with Bower:
bower install redux=https://npmcdn.com/redux/dist/redux.js

@jmeas Can you add that npmcdn link? Other than that, this looks good! :smile:

@jmeas Any chance you can get to this? I can cherry pick in and add the npmcdn link for you, if not.

I'll add it to my to-do list, and try to get to it in a few days.
Since all of the builds export UMD, I think there's some good rephrasing that can be done to capture that important detail.

There's also the counter-vanilla example now that uses npmcdn, which you may way to point to.

Hmmm..."counter-vanilla"? Not sure if I know what ya mean.

https://github.com/rackt/redux/blob/master/examples/counter-vanilla/index.html

Ah, right, haha. Cool, and that's a great idea. I'll add a link to that!

Mmk, updated with some of the feedback y'all gave. Let me know what ya think, and if there are any other changes you think I should make.

I can fix the spaces. That aside, how are the text changes?

This hasn’t been updated in a while so I made some changes inspired by this in 7b859625b727aa7f8e7fffd4481693d81b82a532.
Thank you!

Thanks @gaearon :v:


cancermodels.org ingress controller

As a dev and user I want services to be available on the internet via dev.cancermodels.org and cancermodels.org
ACCEPTANCE CRITERIA

[x] Ingress controller has been mapped to internal resources on both hx and hh
[x] URL behave as desired
[x] external load balancer www6 forwards traffic to the ingress controllers on HX and HH to allow internet access

Waiting for the wild card domain.

@CsabaHalmagyi any follow ups to this?
We can put Dev up on CancerModels.org if it's easier. I'm not sure if we want a wild card domain if we are not going to expose dev.cancermodels.org

Maybe we can limit traffic to EBI IP's only on the EBI external load balancer.

No update from purchasing about this, will follow up.

moving to the next sprint

done


feat(core): add _compute_transaction_hash rpc

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@41fc844). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop     #201   +/-   ##
==========================================
  Coverage           ?   95.58%
==========================================
  Files              ?        9
  Lines              ?      272
  Branches           ?       65
==========================================
  Hits               ?      260
  Misses             ?       11
  Partials           ?        1

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 41fc844...9f81f28. Read the comment docs.


crossplane: fix read many template

Description of changes: ReadMany template was using .Items subfield for all ReadMany outputs but that's not true for all ReadMany resources. For example, RDS DBCluster has .DBClusters in the returned object. ACK already addresses this in set output code but Crossplane does a filtering before that step so we needed to check whether the filtered list still has any items.
Fixes https://github.com/crossplane/provider-aws/issues/492
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@jaypipes thanks for the review! I just rebased and it should be good to go.

@jaypipes thanks for the review! I just rebased and it should be good to go.


[RSpec] Fixed class cast exceptions & npe

Current coverage is 72.68% (diff: 100%)

Merging #65 into master will decrease coverage by 0.15%

@@             master        #65   diff @@
==========================================
  Files            27         27
  Lines           637        637
  Methods           0          0
  Messages          0          0
  Branches         82         82
==========================================
- Hits            464        463     -1
- Misses          143        144     +1
  Partials         30         30

Powered by Codecov. Last update 753e030...474869c


Raise warning if type is not passed

I was banging my head to day because the slider inputs would not pass data to the backend. I could not figure out why.
@app.post("/slider-update")
def slider_update(data):
    print("updating chart")
    print(data)
    return Div(
        P(f"You have pressed the button {count} times."),
        generate_chart()
    )

Can you spot the issue?
This was the fix:
@app.post("/slider-update")
def slider_update(data: dict):
    print("updating chart")
    print(data)
    return Div(
        P(f"You have pressed the button {count} times."),
        generate_chart()
    )

The type needed to be there, without it ... the dictionary is just none. A warning that the type information is missing would have really helped here.

Good point. Fixing that now.


Allow pytest v4 and v5 as well

Fixes https://github.com/kensho-technologies/pytest-annotate/issues/6
I noticed (anecdotally) that pytest-annotate works wonders also using pytest 5.1

Thanks for the PR! We should try to figure out why pylint on Travis is failing, it might need a version bump as well. Then as soon as we are able to merge, I'm happy to push a new release to PyPI.

It seems to have been a linter bug, and I think I may have fixed it by updating to a newer version.

Released as v1.0.3: https://pypi.org/project/pytest-annotate/
Thanks for the PR!


Keep track of the previous channels

Following #489, this re-adds the old behavior of returning to the previous channel on close. While not exactly super intuitive, it still more useful than going to a server window even if it's random and has the advantage of bringing you back to where you were after a /whois or ZNC's *status or after chatting with a bot such as NickServ or ChanServ.
I don't see how reordering the DOM like that could have a negative impact since the others are display:none anyway, and completely removes the need for the z-index trickery.

How expensive is the DOM shuffling in this case?

I haven't felt any difference so far personally but I don't have hundreds of tabs open either tho. I don't see why it would be that expensive they're all hidden: only the visible one ends up being rendered anyway.

Mind checking in devtools what perf it has?

I'm still not set about messing with the DOM for something like this. Partly because of prejudice (the common rule of "Don't mess with the DOM if you don't have to", and we don't have to here) and therefore seems like overkill to me, partly because of potential unintended consequences.
A few things worth thinking of:

If user has preloaded a ton of previous messages, does this have a performance issue?
What about low-end devices? Is there a chance the app crashes on them when closing a chat?
Do we lose any kind of metadata when shuffling this, such as event handlers (can't think of anything else)?
Do prefetched content gets prefetched again?
Is the drag-and-drop affected by this?
etc.

Sorry to be that guy, I'm just afraid to trigger a whole new world of bugs when we thought this was consequence-free. Maybe (and hopefully) the answer to all these questions is no, but it's worth being sure.

You guys are really way too scared of the DOM. It's an array.  The DOM is only a data structure. Adding and removing a node to the DOM is just as fast as an object from an array and putting it back in.
What's expensive is layout changes and redraws. This causes neither a relayout nor a redraw, because nothing visible changes. The renderer loads all nodes in that tree and only one is displayed and will be laid out again. Its dimensions doesn't change, so nothing happens. End of it.
Actually, I can't even measure it because the sticky scroll on the other hand does force a redraw each time it is triggered (due to scroll).
And on top of that, it HAS to improve performance because the z-index only changes the stacking order. Unless they were set to display: none;, they will all be redrawn each on top of the other because each layer can be transparent. At its best it does exactly the same as reordering the DOM, but abusing CSS for it instead.
The answer to all of your concerns is "No", both as per what I just said and also because I've been running it for weeks and didn't see any difference.
 It only leads to even worse workarounds that gives the impression of being better (see the z-index we had before and try to tell me it's not bad) just because it doesn't touch the oooo scary DOM.

Well someone was grumpy :D
Just to be clear, I think the whole debate is not "your solution vs. the z-index way" but "your solution vs. the current way" (current way being send the user back to the network window of that server when closing the active window).
And in that regard, whether there is a performance issue or not, I do think it's a bit overkill to move the entire content of chan in the small eventuality that a user closes a window and that this is the active window.
Putting all my comments and concerns above aside, I see 2 issues:

This is per-network, so if I'm on network A, channel 1, then go to network B, channel 1 and close it, I will get sent to the last channel on network B that I visited. This was already confusing enough on Shout (this behavior only works when closing multiple channels in a row, otherwise it's a real usability mess), it's worse here.
In a big channel list, the user may be sent to another channel far away. Slack does this and it's so messed up that you cannot not grumble about this :D

I think we are trying to find the most computer-sciency way to solve this. I would say the 2 following options are nicer in terms of usability:

When closing the active window, send the user back to the server window (as currently).
When closing the active window, send the user back to the window right above or below the one just closed.

I like the second option because it guarantees not jumping the users all over the place. What do you think? Does that seem reasonable?
(Option 2 is what Chrome does when closing active tabs, with the slight difference that if user opens a tab, stays on that tab whatever he does there, then closes it, they get sent back to the tab they were on when opening it, but this is confusing in many ways so I think we should stay clear of that, at least for now)

@astorije Are you fine with this approach and can we merge it?

@astorije Are you fine with this approach and can we merge it?

Nah, I think this is very confusing in terms of UX. The 2 biggest concerns (but not the only ones) I have are the following:

This does not work across networks. If I'm on network A, channel X, then move to channel Y living on network B, and close channel Y, I will not move back to channel X but to whatever I was on previously on network B.
As @YaManicKill mentioned in #573 (actually, it might have been on the IRC channel), it's tricky to move across channels because of potentially unread messages/markers. While it's ok to move back to the channel we very last visited, any further step risks to mark channels as read.

I think @YaManicKill's alternative is by far the best: when closing the current channel, user gets sent to a blank screen. That allows for 2 independent improvements:

That blank screen can be transformed into an informative dashboard (summary of open channels, missed notifications, errors/disconnects while away (or not), etc.) and become our landing page.
To fulfill @maxpoulin64's use case, we can implement something similar to what Chrome does: after opening a new window, closing it could transfer the user the the previous active channel only if they haven't switched across channels. This only works for the last one and does not bubble like an MRU cache because this is terrible for UX (users do not remember where they were more than 1 step before and asking them to do so is a cognitive exercise we should not do).

I believe this is the most optimal UX decisions we could implement (well at least, the most optimal I can think of at the moment lol).
Note that both 1 and 2 can happen separately, but until any of these happen, I think we should keep the current mechanism. I'll try to take a stab at either of them when I have some spare time.
My take on this is to close both PRs (but #573 is already closed) and rely on current mechanism for now.

Should we close this? Looks like it's not going to go anywhere. Any objections?

While I think this solution is fine, I will close it due to no activity, and will leave it to the "Home" tab idea.


343 update notifications ruby client dependancy

Context
Sadly, we cannot actually update to the mail-notify 2.0 for a couple of reasons, but this PR does take us one step closer by removing dependency on an unintended behaviour (headers that are actually options and never get passed to notify api)

The 'preview' method is no longer available in a the test object without mocking the notify response.
We have a Capybara dependancy that the update breaks.

We'll probably need to wait until further updates to upgrade.
Changes proposed in this pull request

stop storing mail template and mailer as 'headers'

Guidance to review
NA
Things to check

[ ] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] If this code adds a column to the DB, decide whether it needs to be in analytics yml file or analytics blocklist
[ ] API release notes have been updated if necessary
[ ] If it adds a significant user-facing change, is it documented in the CHANGELOG?
[ ] Required environment variables have been updated added to the Azure KeyVault
[ ] Inform data insights team due to database changes
[x] Make sure all information from the Trello card is in here
[x] Rebased main
[x] Cleaned commit history
[x] Tested by running locally
[x] Add PR link to Trello card

 Prepare to Update notifications-ruby-client dependancy


LBRY office on learn

add LBRY Office address on learn page because lbry.fund has the LBRY Office Address..
Open separate ticket as Mr Jeremy said on my PR earlier...

close for now


/tmp not necessarily a ramdisk, data potentially saved onto non-volatile storage

Reading

Create a temporary directory which will be cleared on reboot and set it as the GnuPG directory:
$ export GNUPGHOME=$(mktemp -d)

I had to double-check. And indeed /tmp/ - where mktemp creates temporary directories by default - is not a ramdisk - at least not on Debian.
Yes, it's getting cleared on reboot, however it was once written to persistent storage, meaning the storage device (thumb drive, spinning disk, memory card, you name it..) once had this data being saved on.
I think this needs to be stated and/or the guide should include a setup for creating a ramdisk, e.g. via mount -t tmpfs -o size=128M myramdisk /tmp/ramdisk and mktemp being invoked with -p /tmp/ramdisk.

Happy to create a PR if agreed

Great idea - please send the PR!

Great idea - please send the PR!

I'd suggest checking for the existence of /run/user/$(id -u), and set that as $TMPDIR for mktemp to use.

I'd suggest checking for the existence of /run/user/$(id -u), and set that as $TMPDIR for mktemp to use.

I lost track of what's standard and/or how distributions do it differently.
If you say /run/user/$(id -u) always exists and is always a ramdisk and us independant from systemd being used or not, I'm good and happy with using this one.

I don't think /run/user/$(id -u) is guaranteed to be a tmpfs mount, and it tends to only exist on systems using systemd "user slices" in my experience, which is a lot of distributions but not all of them.
Making a custom tmpfs mount is probably the best way to ensure that you are operating in a ramdisk only.

If there's a better, yet compatible, way to do temporary files, let's document it in a PR.


Write up requirement docs for issues

Documentation is needed to outline the necessary requirements an issue must meet before it is considered "ready to implement"
Additionally issue templates need to be created and existing ones updated to include questions about upcoming deadlines, priority, and resource availability to accomplish (following Carbon's example).
Considered done when:

[x] There are issue templates for bugs and feature request and all templates follow the Carbon example outlined above.
[x] We have documentation in our wiki/README about the items required for each issue in order to be ready to implement by a developer.

CONTRIBUTING.md contains info on required items to consider an item 'ready to implement'
The above PR, #614, updates issue templates as well as the PR template.

:tada: This issue has been resolved in version 2.10.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


How to get the content(e.g. images) from request with out cookie?

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Hey @HtwoOtwo in order to best help you we'll need more information about how to solve this issue.
Without any further context I'm unable to understand how to best help you.
Closing this for the meantime.

Apologies for not providing enough details earlier. What I meant is, how can I access my images? Currently, both my Label Studio and data are stored in place A. However, I need to access this data from place B. Currently, the only way I can do this is by using "request" to fetch the images via their URLs, but it requires cookies. Is there any other method to achieve this? Thank you.


vscode: 1.41.0 -> 1.41.1

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/vscode/versions.
meta.description for vscode is: '"Open source source code editor developed by Microsoft for Windows,\nLinux and macOS\n"'.
meta.homepage for vscode is: '"https://code.visualstudio.com/"

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 1.41.1 with grep in /nix/store/v9jxhgrl7s4dlrs1hzkwswhl76vrd2b8-vscode-1.41.1

directory tree listing: https://gist.github.com/78e75598d1e926e4da2d83ed92ad5721

du listing: https://gist.github.com/4550da8b6744e27c6cd91295d121a227

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)
2 package rebuild(s)
2 x86_64-linux rebuild(s)
0 i686-linux rebuild(s)
2 x86_64-darwin rebuild(s)
0 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
vscode
vscode-with-extensions

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/v9jxhgrl7s4dlrs1hzkwswhl76vrd2b8-vscode-1.41.1 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A vscode https://github.com/r-ryantm/nixpkgs/archive/1166c5750e56c718cd1c4885d815573615ef59a5.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/v9jxhgrl7s4dlrs1hzkwswhl76vrd2b8-vscode-1.41.1
ls -la /nix/store/v9jxhgrl7s4dlrs1hzkwswhl76vrd2b8-vscode-1.41.1/bin

cc @eadwu @Synthetica9 for testing.

Missing a hash for darwin though.

addressed by https://github.com/NixOS/nixpkgs/pull/76678


Make new loading screens to match content and receiver dimensions

Closes https://github.com/relaycc/receiver/issues/108
Here is a video of the loading screen not lining up with the dimensions of the receiver window or the conversations list that it links to
https://user-images.githubusercontent.com/81312527/202504493-52a2408e-83e4-4c1f-9613-78f13dde18b6.mov

@BrianS111 Are you working on this?

No, but I can if it's something we want done @killthebuddh4


Added the contributors page and properly designed it.

Pull Request Template for GSoC 2024
Related Issue
closes: #144
Description of Changes
I have properly done the contributors page and it is working super fine and good looking too and professional too. you can check in the image below
Type of Pull Request

[ ] Bug Fix
[X] New Feature
[ ] Performance Improvement
[ ] Documentation Update
[ ] Code Refactoring
[ ] Security Enhancement
[ ] Other (Specify): _______________

Screenshots / Demo (if applicable)

Checklist for Contributor
use'X' in [ ] to check it .

[X] I have reviewed my code before submission.
[X] I have read and followed the project's Contribution Guidelines.
[X] I have tested the code thoroughly.
[X] I have attached related issue numbers, screenshots, or videos (if applicable).
[X] I have added comments to my code where necessary for clarity.
[X] I have checked the existing code base and my changes do not introduce any breaking changes.
[X] My PR addresses only one issue; I have created separate PRs for other issues.
[X] I have ensured that the PR does not include unrelated files.
[X] I am mindful of the project's coding standards and best practices.

@dhairyagothi Check this PR and merge it and assign this to me and put relevent labels like gssoc 2024 ext, hacktoberfest and level. i think it can be level 3 because the page is very important in open source to showcase the efforts of the people contributing in building it and it is not static it is dynamic as a new contributor is added to your repo it will be automatically added there so this is the beauty of this.

very well done bro @tarunkumar2005 thanku so much .... your contribution is valuable i loved the proper UI


C++: Check licenses with RAT in CI

Reporter: Uwe Korn / @xhochy
Assignee: Uwe Korn / @xhochy
Related issues:

parquet-cpp 1.0.0 release (blocks)

Note: This issue was originally created as PARQUET-773. Please see the migration documentation for further details.

Wes McKinney / @wesm:
Issue resolved by pull request 189
https://github.com/apache/parquet-cpp/pull/189

Uwe Korn / @xhochy:
PR: https://github.com/apache/parquet-cpp/pull/189


Once again fails to convey the concepts and raises more questions than answers

The "Set the font in XAML" section explains how to set up a font family. The "Use a custom font" section explains how to import a custom font. Now, the examples in "Use a custom font" are completely different than the ones in "Set the font in XAML". Are both required, or are they different flavours? My guess is that "Use a custom font" explains the new ExportFont annotation and it supersedes "Set the font in XAML", is this correct? In that case please update the documentation and don't just add to it. If you look for font issues on the Xamarin.Forms GitHub there's a ton of reports because nobody knows how to use it, because the documentation is always written in such a way that you are left guessing as to what should be done. Please do better on this front.
Shouldn't the entire "Set the font in XAML" section be removed, or at the very least shouldn't it be placed AFTER the "Use a custom font" with further elaboration that it's a legacy way. Human beings read from top to bottom and they will read the outdated section first, then the new way, and get extremely confused. This nonsense is wasting time on developers fronts and also Xamarin.Forms's devs time as there will be bug reports because people can't follow the documentation.
https://devblogs.microsoft.com/xamarin/embedded-fonts-xamarin-forms/ is an example that explains this in a concise manner.
Also, the "Set font info per platform" section begins with "Alternatively, the Device.RuntimePlatform property can be used to set different font names on each platform". Alternatively to what? It's the first time that topic is discussed.

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

ID: fe0572c4-91be-fc86-d130-242db2bf470b
Version Independent ID: 1238fdd7-c378-6d0d-654e-55656a70d386
Content: Fonts in Xamarin.Forms - Xamarin
Content Source: docs/xamarin-forms/user-interface/text/fonts.md
Product: xamarin
Technology: xamarin-forms
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Hi @Tommigun1980
Yes, "Use a custom font" supersedes "Set the font in XAML". The issue here is that "use a custom font" requires Forms 4.5 or higher. Many users are still on older versions, and so "Set the font in XAML" is required. On top of that custom fonts didn't work very well when they were introduced, and so "Set the font in XAML" was still required as a fallback.
Going back to your overall point, I agree that this doc needs an overhaul. I'll add it to the backlog.

Hi David.
Why not say that in the docs then? It has two sections that explain completely different approaches. How would a user know that one is the old way and one the new way if, you know, the docs don't tell that?

Hi @Tommigun1980
It's a result of the messy way custom fonts were introduced into Xamarin.Forms. I agree the doc needs fixing, and it will be.

I've just refreshed the doc, and it will be live soon.


Traffic lights should fail if there is power cut

Idea spawned from, and dependent upon, #128
If there is power cut (as game can detect for buildings), it would be nice if the traffic signals could fail.
In many cases a failed traffic light will show yellow flashing light (running from battery power or something?) to warn that there is a junction, or just be completely unlit.
Some infos in wikipedia: https://en.wikipedia.org/wiki/Traffic_light_control_and_coordination#Other_types_of_control

I don't know about other places, but when a traffic light has failed here, it will flash the red light over and over.  In terms of power outages, I'm not sure it they act differently, and I'm also not sure if there is anything in terms of backup power for the lights. I know sometimes the lights will just be out entirely (no lit signals), and in that situation you continue to treat it like a 4-way stop. Technically, it's supposed to be right-of-way, but everyone does 4-way because it makes way more sense.

Yes, I'd love the street lights to go out. A proper power cut!
Maybe lights could 'malfunction' (causing flashing yellow or red depending on user preference) if the road is unmaintained for too long? :D

Yes! I could totally see if I could do that. I'm just going to note this here, I think the option name "Road lights are power-dependent" as a toggle (and enabled by default) is a good option name.

For power detection we'd ideally need some way to cache it, it seems wasteful needing to check every frame, and also wasteful to check every segment (there can be 38k segments). I assume the base game has a decent approach to staggering the electricity checking across frames?
It might be worth inspecting source of Real Time mod, as that had to deal with electricity. Specifically, RT would turn lights out at night, but if there was a power cut the game would turn lights back on when power was restored, so Real Time had to handle that scenario.

Since I'm taking finals for my high school right now, I might be able to experiment a bit but I'll focus on this 2-4 weeks from now. Hope that's okay!

@aubergine10, luckily enough, Real Time didn't have to bother with electricity. For the lights out feature, Real Time just semi-deactivates the buildings, the game does the rest. So there is no code for electricity handling or similar things in Real Time, no need to study the code.
Maybe the same approach could be used for the traffic lights too.

My apologies but with my recent increase in real life activities and my lack of participation in TM:PE in recent months, I think this issue would be better suited to someone else. If it's still around when I come back I'd look forward to fixing it, but for now I'm going to unassign myself. Sorry!


setup ArbitraryTransformer with new testing appraoch

Issue: https://github.com/lvgig/tubular/issues/207
Made changes to bring ArbitraryImputer in line with new test format, also refactored BaseImputer tests to ease the rest of the rollout across imputers.
Introduced new 'FITS' class attribute to simplify test inheritance process and turning on/off relevant tests, if the reviewer approves then suggest to rollout across transformers. Possibly default to True in BaseTransformer and set to False where needed in main transformers.

Should we be adding type hints to the tests?

Can we update the changelog with these changes?

updated changelog here https://github.com/lvgig/tubular/pull/212/commits/f5234c86c0a94857a4a1508b70870e0a515fd86f

re type hints in tests.. maybe?
But think that would be one to spin out into another issue as it doesn't currently seem to be done elsewhere

Happy to leave the type hints for now. I think there's just an outstanding merge conflict then happy to approve!

nice thanks :) should be resolved now!


String pool for MemoryPoolIteratorExtensions.GetAsciiString()

Just a question for curiosity. When reading the request header (and few other things), a new string is allocated in MemoryPoolIteratorExtensions.GetAsciiString().
But in case of several requests for a same connection:

Each request header contains few strings to allocate.
Some strings have probably the same length between requests.

So I was asking myself if a small but very optimized string pool attached to the Frame instance would not be intersesting to limit string allocation when several requests are done in a same connection.
I have made a prototype, and of course string allocation count goes down to 0 after the first request. Nevertheless to be honest I didn't see significant performance gain. But my computer is quite old and my prototype is not optimal, so I'm not sure that it was a reliable benchmark.
Of course the difference would not be huge for a large MVC View request, but for a "plaintext" performance test?

Is your prototype anything like @benaadams #411 PR from last year? At the time, it showed a small enough improvement in plaintext benchmark (which is pretty much a best case scenario for this type of change), that we decided it wasn't worth the additional complexity.

I had not found the issue. Yes, it is the same starting observation.
There is a difference in the implementation: instead of hashing the string and then copying it if not found, my prototype is only focused on the string length:
	public const int PoolSize = 32;

        private string[] _pool;

        private uint _used;

        public string Get(int length)
        {
            if (_used == uint.MaxValue ||
                _charCount + length > MaxCharCount)
            {
                return new string('\0', length);
            }

            int freeIndex = -1;
            for (int i = 0; i < PoolSize; i++)
            {
                if ((_used & (1 << i)) == 0)
                {
                    var str = _pool[i];

                    if (str == null)
                    {
                        freeIndex = i;
                    }
                    else if (str.Length == length)
                    {
                        _used |= 1u << i;
                        return str;
                    }
                    else if (freeIndex < 0)
                    {
                        freeIndex = i;
                    }
                }
            }

            _used |= 1u << freeIndex;
            _charCount += length;
            _pool[freeIndex] = new string('\0', length);

            return _pool[freeIndex];
        }

       // Called for every request
        public void Reset()
        {
            _used = 0;
            _charCount = 0;
        }

But I clearly don't know if it makes a big difference with the #411 algorithm...

We are closing this issue because no further action is planned for this issue. If you still have any issues or questions, please log a new issue with any additional details that you have.


Train 166

High Level Description

upgrade rex-ray to v0.9.0 #1430
Add dcos checks #1459
fixes duplicate iptable rule #1518
Admin Router: AR NEXT #1546
Add test verifying profile symlink #1575

@mesosphere-mergebot bump-ee

Please review: @branden , @mellenburg , @GoelDeepak , @vespian
After review, this could be merged: @spahl , @lingmann

LGTM

@mesosphere-mergebot label ship-it


Choose File is not working on Safari 12

Hi, I have the following scenario;
macOs High Sierra
Safari 12
RF 3.1.2
RF-S2L 3.0.0
RF-SL  3.3.1
selenium 3.141.0

I'm not able to upload files in a form. My tests running on chrome uses the keyword:
Choose File   <locator>    <file_path>

The error I got is:
InvalidArgumentException: Message: One or more files could not be selected.

This tracker is for bug reports and enhancement requests related to Robot Framework core only. Issues related to specific library should be submitted to their issue trackers. Questions are best asked on support forums such as mailing list and Slack listed at http://robotframework.org.

Notice that if the same test works with Chrome, the problem is most likely in Safari driver.


[Finalize + Organization + Account Stacks] Insufficient permissions for actions

Describe the bug
When running the uninstaller, deleting the :

AWSAccelerator-FinalizeStack-<AWS_ACCOUNT_ID>-us-east-1
AWSAccelerator-OrganizationsStack-<AWS_ACCOUNT_ID>-ap-southeast-2
AWSAccelerator-AccountsStack-<AWS_ACCOUNT_ID>-us-east-1

fails with:

Received  response status [FAILED] from custom resource. Message returned:  AccessDeniedException: You don't have permissions to access this  resource.     at de_AccessDeniedExceptionRes (/var/task/index.js:84:73182)     at de_CommandError (/var/task/index.js:84:67567)     at process.processTicksAndRejections  (node:internal/process/task_queues:95:5)     at async /var/task/index.js:2:11287     at async /var/task/index.js:3:6621     at async /var/task/index.js:13:16356     at async /var/task/index.js:1:15425     at async D7 (/var/task/index.js:13:21595)     at async /var/task/index.js:13:21376     at async ACe (/var/task/index.js:86:63000) (RequestId:  c9d6bca3-12d5-4bc4-a147-f80c5670b843)
--

To Reproduce
Delete the AWSAccelerator-FinalizeStack-<AWS_ACCOUNT_ID>-us-east-1 stack when SCPs are attached to any OU
Expected behavior
Deletion without issue
Please complete the following information about the solution:

[x] Version: 1.10.1.
[x] Region: HOME_REGION=ap-southeast-2 , stack region=us-east-1
[ ] Was the solution modified from the version published on this repository?
[ ] If the answer to the previous question was yes, are the changes available on GitHub?
[x] Have you checked your service quotas for the services this solution uses?
[x] Were there any errors in the CloudWatch Logs? Logs in Cloudtrail

Screenshots

Additional context
{
    "eventVersion": "1.09",
    "userIdentity": {
        "type": "AssumedRole",
        "principalId": "XXXXX:AWSAccelerator-FinalizeSt-CustomOrganizationsAttac-KN1YGfATsNrc",
        "arn": "arn:aws:sts::YYYYYYY:assumed-role/AWSAccelerator-FinalizeSt-CustomOrganizationsAttach-e1MdSHKhGXHk/AWSAccelerator-FinalizeSt-CustomOrganizationsAttac-KN1YGfATsNrc",
        "accountId": "YYYYYYY",
        "accessKeyId": "ASIA47CRVUCI7CSUK65Z",
        "sessionContext": {
            "sessionIssuer": {
                "type": "Role",
                "principalId": "XXXXX",
                "arn": "arn:aws:iam::YYYYYYY:role/AWSAccelerator-FinalizeSt-CustomOrganizationsAttach-e1MdSHKhGXHk",
                "accountId": "YYYYYYY",
                "userName": "AWSAccelerator-FinalizeSt-CustomOrganizationsAttach-e1MdSHKhGXHk"
            },
            "attributes": {
                "creationDate": "2024-12-18T22:00:32Z",
                "mfaAuthenticated": "false"
            }
        }
    },
    "eventTime": "2024-12-18T22:00:34Z",
    "eventSource": "organizations.amazonaws.com",
    "eventName": "ListPolicies",
    "awsRegion": "us-east-1",
    "sourceIPAddress": "<IP_ADDRESS>",
    "userAgent": "aws-sdk-js/3.638.0 ua/2.0 os/linux#5.10.227-239.884.amzn2.x86_64 lang/js md/nodejs#18.20.4 api/organizations#3.638.0 exec-env/AWS_Lambda_nodejs18.x AwsSolution/SO0199/1.10.1",
    "errorCode": "AccessDenied",
    "errorMessage": "User: arn:aws:sts::YYYYYYY:assumed-role/AWSAccelerator-FinalizeSt-CustomOrganizationsAttach-e1MdSHKhGXHk/AWSAccelerator-FinalizeSt-CustomOrganizationsAttac-KN1YGfATsNrc is not authorized to perform: organizations:ListPolicies on resource: * because no identity-based policy allows the organizations:ListPolicies action",
    "requestParameters": null,
    "responseElements": null,
    "requestID": "e8f1e381-08aa-49ac-8886-9eea851a6fb2",
    "eventID": "2657b7db-0535-459d-851c-f015b276e5bc",
    "readOnly": true,
    "eventType": "AwsApiCall",
    "managementEvent": true,
    "recipientAccountId": "YYYYYYY",
    "eventCategory": "Management",
    "tlsDetails": {
        "tlsVersion": "TLSv1.3",
        "cipherSuite": "TLS_AES_128_GCM_SHA256",
        "clientProvidedHostHeader": "organizations.us-east-1.amazonaws.com"
    }
}

Hi @richardkeit!
Thank you again for reporting this. I'm taking a look at your pull request right now.


Create year round up update presentation

https://blockscienceteam.slack.com/archives/CA70QMRRC/p1716244022186489


Can't drop until moving the item below the target (horizontal sorting)

Reproduction online
Video:

I'm seeing this too. It looks like it doesn't sort until the mouse position moves below the middle.

Yep, which makes sense for vertical sorting, but not for horizontal one.

This appears to be fixed by setting the option:
direction: 'horizontal'

https://jsfiddle.net/k4xcdg43/

That's so lame of me :)
Thanks for pointing it out!

Don't worry, I did exactly the same!


[Feat]: Add Multiple workflows for GitHub Repository[Description]

What feature?

lock.yml: This file is used for "Locking". It is likely used to prevent discussions or information from being modified or altered by locking conversations or threads.
prevent_multiple_issues.yml: This file is used for "Preventing Multiple Issues". It helps in handling multiple issues in a project simultaneously by providing rules for managing and avoiding conflicting or overlapping issues.

Add screenshots
Add screenshots
Code of Conduct

[x] I agree to follow this project's Code of Conduct

@NarendraKoya999, as of now, we don't need that feature. For that reason, I'll be closing this issue.


dvtm unsets LS_COLORS

is that on purpose (why)?
it doesn't do that when i set DVTM_TERM to "screen" or "xterm" before launching dvtm, or if i set TERM to either of those from within dvtm.
but with that there's at least one (new) issue: the cursor isn't properly reset, or something - let's say my cursor is somewhere in the middle of the screen (for example, after i do "echo -e 'foo\nfoo\nfoo\nfoo\nfoo'). then, when i open and close some fullscreen program (e.g. "man man"), the cursor is suddenly on the 1st line of the screen, not where it was before, and the output from before (the 5 "foo"s) is still there.
is that expected behavior?
thanks!
(btw this happens with dvtm-0.15 on linux)

dvtm should not unset the LS_COLORS environmental variable. More likely your shell startup scripts do not set it (or only if $TERM is a "known" value such as xterm or screen.
Also keep in mind that dvtm creates non-login shells (as apposed to for example tmux which AFAIK creates login shells), make sure you are modifying the correct shell startup script.

yes, you're right. it was because i aliased "ls" to "ls --color=auto". it works fine with "--color=always" instead. thanks!

actually what i said is wrong. i just didn't set LS_COLORS correctly - i used to reset it for every non-login shell (with the dircolors command, and within dvtm it just gets set to an empty string), it works fine when i only set it for login-shells. so as you said.


Correction for raw affiliation Eco&Sols, Univ. Montpellier, CIRAD, INRAE, IRD, Montpellier SupAgro, Montpellier, France

Correction needed for raw affiliation Eco&Sols, Univ. Montpellier, CIRAD, INRAE, IRD, Montpellier SupAgro, Montpellier, France
raw_affiliation_name: Eco&Sols, Univ. Montpellier, CIRAD, INRAE, IRD, Montpellier SupAgro, Montpellier, France
new_rors: 03rnk6m14;05kpkpg04;05q3vnk25;051escj72;003vg9w96;00b80ez59
previous_rors: 03rnk6m14;05kpkpg04;05q3vnk25;051escj72;003vg9w96
works_examples: W4243579926;W4247919721
contact: acf1e66e0ccdd8e7d233f8d8d48f0156:0cd061541357de6e3e4bfbf9f4a5b18fd282 @ inrae.fr

This issue was accepted and ingested by the OpenAlex team on 2024-11-18. The new affiliations should be visible within the next 7 days.


Add data-workspace unit tests to test-extensions-unit scripts

Adding this so that the data-workspace code coverage gets included in the code coverage reports in PRs. @Charles-Gagnon let me know if anything else needs to be updated to include them.

Pull Request Test Coverage Report for Build 703350959

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.4%) to 43.577%

Totals

Change from base Build 703302562:
0.4%

Covered Lines:
26053

Relevant Lines:
54605

💛  - Coveralls

Oh wait - it's already added there. You're not getting code coverage for these?


Few user questions!

Hi developers! Happy New Year!
I have few user questions:

How I can turn on logging to file?
My example:
from lackey import *
a = "C:\go.log"
DebugMaster.setLogFile(a)

Result:
DebugMaster.setLogFile(a)
TypeError: setLogFile() missing 1 required positional argument: 'filepath'

Can I somehow set confidence? (lever of accuracy):

[info] Found match for pattern 'C:\Program Files (x86)\Python36-32\NinjaBot\head
er_of_platform.png' at (351,146) with confidence (0.9999997019767761). Target at
(434,156)
in some cases I want to set different level of confidence.

Happy New Year to you too!
The best resource for getting acquainted with the typical usage like this is the SikuliX documentation, as Lackey is intended to be a drop-in replacement.
First, DebugMaster is the uninstantiated class; what you're looking for is the Debug object:
from lackey import *
a = "C:\go.log"
Debug.setLogFile(a)

For more details, check out the log and debug messages documentation.
Second, you absolutely can set the level of confidence for finding a pattern. This can be configured globally by changing Settings.MinSimilarity, or individually by passing a Pattern object instead of just the path.
By default, Settings.MinSimilarity is 0.7. If you don't want to change this globally, you can set it individually by calling the pattern's similar() method:
Settings.MinSimilarity = 0.8
find("image_to_find.png") # Will only find matches with confidence 0.8 or greater
find(Pattern("image_to_find.png").similar(0.9)) # Will only find matches with confidence 0.9 or greater

For more details, check out the Pattern documentation.
Hope this helps!

2nd - it works! Thank you!
1st > about DebugMaster...
I just want to write logging from cmd console (on windows) to file.
Please,, provide me instruction how I can write log information to file.


[WIP] Custom Render Context

PR Details

Add missing extension point to support custom render context

Custom Window
Custom Platform

Description
Mostly putting internal things in public
Related Issue

#870
Motivation and Context

These changes allow stride to be hosted in a custom window platform like:

Headless
Remote
GLFW
vvvv
Avalonia

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[x ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

@tebjan you have more points to open ?

Hey, very glad to see this happening. I hope we will have time to discuss these things in our next meeting and come back to you with ideas and/or other points to consider.

I think most important would be Xamarin.Forms / .Net MAUI Custom Renderer/Handler I tried to work on it and I would do it but still Months it is not possible to build Android Apps. For UWP/WinUI for example creating a Custom Renderer would be extremly simple (https://github.com/xamarin/XamarinCommunityToolkit/issues/549) but for Android I have to test it. So I could not work fourther on.

Hi, how is it going with this PR? Looking forward to discussing the bounty reward for it if you have the time.

I think most important would be Xamarin.Forms / .Net MAUI Custom Renderer/Handler I tried to work on it and I would do it but still Months it is not possible to build Android Apps. For UWP/WinUI for example creating a Custom Renderer would be extremly simple (xamarin/XamarinCommunityToolkit#549) but for Android I have to test it. So I could not work fourther on.

Is testing on Android the only thing holding up this PR? I have several Android devices that are already set up for development, just let me know what needs to be tested.

Any news to this?


Couldn't load IImageService

Hi all, I am bumping my head for 5 hours trying to find a solution for a simple problem. I created a new LocalFileImageService but I am getting the following error
Couldn't load IImageService: MCWeb_3SR.Klase.LocalFileImageService2, MCWeb_3SR.Klase
Description: An unhandled exception occurred during the execution of the current web request. Please review the stack trace for more information about the error and where it originated in the code.
Exception Details: System.TypeLoadException: Couldn't load IImageService: MCWeb_3SR.Klase.LocalFileImageService2, MCWeb_3SR.Klase
Here is my config entry
<service name="LocalFileImageService2" type="MCWeb_3SR.Klase.LocalFileImageService2, MCWeb_3SR.Klase" />

end here is my Service code

namespace MCWeb_3SR.Klase
{
    using System;
    using System.Collections.Generic;
    using System.IO;
    using System.Net;
    using System.Threading.Tasks;
    using System.Web;
    using ImageProcessor.Web.Helpers;
    using ImageProcessor.Web.Services;

    /// <summary>
    /// The local file image service for retrieving images from the file system.
    /// </summary>
    public class LocalFileImageService2 : IImageService
    {
        /// <summary>
        /// Gets or sets the prefix for the given implementation.
        /// <remarks>
        /// This value is used as a prefix for any image requests that should use this service.
        /// </remarks>
        /// </summary>
        public string Prefix { get; set; } = string.Empty;

        /// <summary>
        /// Gets a value indicating whether the image service requests files from
        /// the locally based file system.
        /// </summary>
        public bool IsFileLocalService => true;

        /// <summary>
        /// Gets or sets any additional settings required by the service.
        /// </summary>
        public Dictionary<string, string> Settings { get; set; }

        /// <summary>
        /// Gets or sets the white list of <see cref="System.Uri"/>.
        /// </summary>
        public Uri[] WhiteList { get; set; }

        /// <summary>
        /// Gets a value indicating whether the current request passes sanitizing rules.
        /// </summary>
        /// <param name="path">
        /// The image path.
        /// </param>
        /// <returns>
        /// <c>True</c> if the request is valid; otherwise, <c>False</c>.
        /// </returns>
        public bool IsValidRequest(string path)
        {
            return ImageHelpers.IsValidImageExtension(path);
        }

        /// <summary>
        /// Gets the image using the given identifier.
        /// </summary>
        /// <param name="id">
        /// The value identifying the image to fetch.
        /// </param>
        /// <returns>
        /// The <see cref="System.Byte"/> array containing the image data.
        /// </returns>
        public async Task<byte[]> GetImage(object id)
        {
            string path = id.ToString();
            byte[] buffer;

            // Check to see if the file exists.
            if (!File.Exists(path))
            {

                path = System.Web.Hosting.HostingEnvironment.MapPath("~/images/NemaSlike.png");
                //throw new HttpException((int)HttpStatusCode.NotFound, $"No image exists at {path}");
            }

            using (FileStream file = new FileStream(path, FileMode.Open, FileAccess.Read, FileShare.Read, 4096, true))
            {
                buffer = new byte[file.Length];
                await file.ReadAsync(buffer, 0, (int)file.Length);
            }

            return buffer;
        }
    }
}`

What might be a problem ?

Hi @srdjo
Where are you storing the class? In compiled format or in your App_Code folder?
Cheers
James

I was able to load your service in ImageProcessor.TestWebsite within the solution using the following config.
<service name="LocalFileImageService2" type="MCWeb_3SR.Klase.LocalFileImageService2, ImageProcessor.TestWebsite" />

Not that the assembly name after the comma has to exactly match the name of your assembly.