Move celo-fullnode and common charts out from Monorepo

Description
This PR introduces the changes to celotool to allow the deployment of "external" charts in oci registries, using the existing installGenericHelmChart and upgradeGenericHelmChart present functions. In particular, with this new change there is a need to introduce an optional chartVersion parameter to these functions, so you can select the desired remote chart version. It was needed to refactor the functions parameters to allow named parameters, that should make the code more readable and easy to maintain for both cases (charts as files and remote charts).
Using these new functions, the celo-fullnode and common charts have been deleted and now are installed from a remote oci repository. The new charts are in https://github.com/celo-org/charts (expected to be public soon).
Tested
Tested in some deployments, and with --helmdryrun in different clusters. It may be useful to test also in some Azure cluster (still pending)
Related issues

Fixes https://github.com/celo-org/infrastructure/issues/409

Backwards compatibility
It should be backwards compatible.
Documentation
The set of community facing docs that have been added/modified because of this change

yess!
going for that Most lines deleted in October Bonus ;)


Broken JSON in case heat pump returns NULL value

Hi,
due to whatever reason, my heatpumps returns "nothing" when querying
{0x60,5,203,1,-1,"Betrieb_Störung_x60"}
This is taken over 1:1 in the MQTT JSON file, which causes "broken JSON format" errors downstream

I assume the code needs to be changed to put a null into the JSON in such cases
Thanks for investigating!

Hi @DieterTHeck. I've seen where it is. Thanks for reporting. I'll fix it ASAP.

Ok, it should be fixed now!

Hi,
thanks, brilliant.
I just

updated converter.h with lastest version
recompiled without error
OTA update works fine with the buggy NULL value re-enabled
MQTT shows "-" now

Home Assistant templates work w/o errors now

Looks like it is working :-)


How to add styles to an element "trigger" - popoverTriggerClass

Hey. You wrote a very good component.
But why there is no props "popoverTriggerClass", so that I can add my custom styles for the trigger element.
Code Popover.vue
<div
      ref="trigger"
      class="trigger"
      // my modification
      // :class="['trigger', popoverTriggerClass]"
      style="display: inline-block;"
      :aria-describedby="popoverId"
      :tabindex="trigger.indexOf('focus') !== -1 ? 0 : undefined"
></div>

Just ran into this as well. I was able to select the element via the trigger CSS class in a global CSS block, ie. not scoped, but it would be great to allow custom classes on that element.


Is there a guide how to test report functionality?

Hi,All
When I use the lighting project, I see that the reporting function has been added, but when I trigger it via the local button, there is no wireless message reported by the device. How should I verify this reporting function?

What's your hardware setup?
App log should show you reporting happen - for example (linux all-clusters-app)
[1630583381.359413][974:974] CHIP:DMG: <RE> Sending report...
[1630583381.359476][974:974] CHIP:DMG: IM RH moving to [Reporting]
[1630583381.359638][974:974] CHIP:IN: Secure message was encrypted: Msg ID 1
[1630583381.359720][974:974] CHIP:IN: Encrypted message 0x5769b8 to 0x5F419ACFB36E2E1C of type 5 and protocolId 5 on exchange 44256.
[1630583381.359811][974:974] CHIP:IN: Sending msg 0x5769b8 to 0x5F419ACFB36E2E1C at utc time: 64765164 msec
[1630583381.359878][974:974] CHIP:IN: Sending secure msg on generic transport
[1630583381.365141][974:974] CHIP:IN: Secure msg send status ../../third_party/connectedhomeip/src/inet/IPEndPointBasis.cpp:917: Success
[1630583381.365327][974:974] CHIP:DMG: <RE> ReportsInFlight = 1 with readHandler 0, RE has no more messages
[1630583381.483279][974:974] CHIP:EM: Received message of type 0x40 with vendorId 0x0000 and protocolId 0x0000 on exchange 44256
[1630583381.483504][974:974] CHIP:DMG: in state Reporting, receive status report, protocol id is 5, protocol code is 0
[1630583381.483583][974:974] CHIP:DMG: <RE> OnReportConfirm: NumReports = 0
[1630583381.483645][974:974] CHIP:DMG: IM RH moving to [Uninitialized]
[1630583381.483791][974:974] CHIP:IN: Secure message was encrypted: Msg ID 2
[1630583381.483874][974:974] CHIP:IN: Encrypted message 0xbeb83860 to 0x5F419ACFB36E2E1C of type 16 and protocolId 0 on exchange 44256.
[1630583381.483965][974:974] CHIP:IN: Sending msg 0xbeb83860 to 0x5F419ACFB36E2E1C at utc time: 64765289 msec
[1630583381.484030][974:974] CHIP:IN: Sending secure msg on generic transport
[1630583381.484353][974:974] CHIP:IN: Secure msg send status ../../third_party/connectedhomeip/src/inet/IPEndPointBasis.cpp:917: Success

Can you please follow up with @kidip ? Thanks!


List documents UI needs update

Above is the current UI for list documents. It has the following problems:

It does not tell how many unresolved doubtful transactions are pending manual curation.
It does not tell number of line items extracted
It does not show account details and time details.
User should be able to define identifiers for the file after uploading the file. For example, user should be able to mark something as statement for the month of June for Acccount xx

[X] how many unresolved doubtful transactions are pending manual curation.
[X] number of line items extracted
[X]  account details
[X] time details
[X] Ability to add a description for the statement
[X] Store file in s3


Change logging when admin sends a gift

Is your feature request related to a problem? Please describe.

Bot logs a warning when tries to accept offer from owner that was a gift because there's no confirmation required.
Describe the feature you'd like

Describe alternatives you've considered

Additional context

I am not sure what this issue is about @danocmx @offish ?

Can this be closed?


PWA Push notification support

I've published my PWA app to store like 2 years ago. But push notifications don't work anymore. I am using web push procotol.
Document says Windows WNS is mandatory so does it mean that web push doesn't work any more for PWA store apps?
Also WNS link returns 404 (link below).
https://docs.microsoft.com/en-us/windows/uwp/controls-and-patterns/tiles-and-notifications-windows-push-notification-services--wns--overview.md

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

ID: 7f55203c-7b09-e20b-2769-6977b2b80c26
Version Independent ID: 6ab62be6-5201-0ead-d912-a271f88ec00c
Content: Progressive Web Apps (EdgeHTML) on Windows - Microsoft Edge Development
Content Source: microsoft-edge/progressive-web-apps-edgehtml/index.md
Product: microsoft-edge
Technology: pwa
GitHub Login: @MSEdgeTeam
Microsoft Alias: msedgedevrel

Web push should continue to work for PWA store apps.  What errors are you experiencing?  Can you identify the affected app?
We need more information to investigate.  Thanks!
Steve Becker
Software Engineer, Microsoft

Hello,
Thank you so much for your response. Then recently it's about my app's
integration. I've deleted the previous Live sdk application but cannot
create a new one on my new account.
Here are the application details;
Package/Identity/Name garaj.GuildsofWoW
Package Family Name (PFN) garaj.GuildsofWoW_0yd3bcpxvd0kg
Package SID
S-1-15-2-1525687717-4291355751-1504712405-4283844914-3909918357-701748778-1116766085
On Fri, Oct 9, 2020 at 9:45 PM Steven Becker<EMAIL_ADDRESS>wrote:

Web push should continue to work for PWA store apps. What errors are you
experiencing? Can you identify the affected app?
We need more information to investigate. Thanks!
Steve Becker
Software Engineer, Microsoft
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/edge-developer/issues/922#issuecomment-706344803,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA23JHDGR6T7SZVOVY6Y5STSJ5K4HANCNFSM4R6JSI2Q
.

--
Serkan YAZICIOĞLU

Microsoft's push notification service is delivering messages to the garaj.GuildsofWoW app you identified.  Could you provide more details about the problems you're experiencing?  Does this app handle push channel expiration?  PWA-based web push subscriptions expire after 30 days.


build arm64-win

🤔 This is a ...

[ ] New feature
[ ] Bug fix
[ ] Performance optimization
[ ] Enhancement feature
[ ] Refactoring
[ ] Code style optimization
[ ] Test Case
[ ] Branch merge
[ ] Workflow
[ ] Other (about what?)

🔗 Related issue link

closed #210
💡 Background and solution

📝 Changelog

☑ Self-Check before Merge
⚠ Please check all items below before requesting a reviewing. ⚠

[ ] Doc is updated/provided or not needed
[ ] Changelog is provided or not needed

the hickory-dns build is failing and needs a bit more work


Abort async tasks using asyncio

Closes #1084

Successful PR Checklist:

[x] Tests

[ ] (not applicable?)

[x] Documentation

[ ] (not applicable?)

PR label(s):

[ ] https://github.com/procrastinate-org/procrastinate/labels/PR type%3A breaking 💥
[x] https://github.com/procrastinate-org/procrastinate/labels/PR type%3A feature ⭐
[ ] https://github.com/procrastinate-org/procrastinate/labels/PR type%3A bugfix 🕵
[ ] https://github.com/procrastinate-org/procrastinate/labels/PR type%3A miscellaneous 👾
[ ] https://github.com/procrastinate-org/procrastinate/labels/PR type%3A dependencies 🤖
[ ] https://github.com/procrastinate-org/procrastinate/labels/PR type%3A documentation 📚

Changes
This modifies the way jobs are being aborted when the task is async, by cancelling the underlying asyncio Task that represents the running job.

Congratulations again :)


Separate trade start logic from trade monitoring for exit

Currently the trade loop runs once for a given period. If for example you run the loop on 1 hour candles it runs the entire trade loop once every hour. This also means that active trades get checked for their exit conditions only once every hour. I think the trade monitoring for exit conditions could benefit from being a separate process that can run more often compared to the logic for finding a decent entry point.

A lot of this has been fixed by adding a SellTimer next to the BuyTimer on develop.


KAFKA-7477: Improve Streams close timeout semantics

Second part of KIP-358.
This changes based on previous PR discussion.
Default close timeout is 30 seconds.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Hello, @vvcephei.
Can you take a look?

Tests passed.
@mjsax Can you take a look?

We should also add a test KafkaStreamsTest to test that the new method throws for negative values and does not block for zero (ie, please add two tests)

Hello, @mjsax
Looks like a didn't understand initial task properly :)
Thank you for the clarification!

waitOnState reworked.
Tests added.
Current methods behaviour preserved.

Please, take a look.

Failure unrelated.
Retest this, please.

@mjsax @vvcephei
Thanks for the review.
I addressed all your comments.

Merged to trunk and cherry-picked to 2.1 branch.


TSP: Unable to resolve sub module path for dual published ESM/CJS package

Page URL:
TypeScript Link Here
Example Code:
Code above
import { Type } from '@sinclair/typebox'        // ok
import { Value } from '@sinclair/typebox/value' // not found

Issue:
I'm not sure if this is a critical issue, but following a recent update to support dual published ESM / CJS in TypeBox.  I have noticed that the TSP is no longer able to locate the /value sub module (or any other sub module), however the top level module import continues to work fine. I have checked to ensure TS resolution of sub module paths are valid for the package via the link below, I believe everything is setup correctly.
https://arethetypeswrong.github.io/?p=%40sinclair%2Ftypebox%400.32.4
I should note that this package uses a somewhat non-typical package redirection setup to support dual publishing. The technique it uses is described at the following link / project.
https://github.com/andrewbranch/example-subpath-exports-ts-compat/tree/main/examples/node_modules/package-json-redirects
Would be amazing if this could be looked at as I use the TSP almost daily to help users who have submitting issues to the TypeBox project. It would be a shame to lose the sub module imports as many type inference features exists for these sub modules, and it would be difficult to demonstrate functionality and share code via other means.
Many Thanks

This is the repo for TypeScript's website... Did you mean to file this on the typebox repo?

Hi @jakebailey !

This is the repo for TypeScript's website... Did you mean to file this on the typebox repo?

Haha, yes, I meant to submit here :) (assuming this is the correct place to submit issues for the TypeScript Playground?). If there is a better place to submit issues for the TypeScript Playground, please let me know! :)

So, as for the issue itself, it seems specific to sub module resolution on the TypeScript Playground, where it's lost the ability to import sub modules from the TypeBox package. Note that this issue only relates to the TypeScript Playground, as importing these sub modules is perfectly fine in a local environment (vscode, language service and compiler)
Note that this issue has only appeared since TypeBox introduced ESM + CJS dual publishing. It currently uses a package redirection technique (linked above) to support both dual publishing. This technique seems to be both "valid" and yet very "non-typical", with the technique seemingly tripping up resolution on the TSP.
It may be helpful to take a look at how the package setup on NPM (noting that sub module directory only contain a package.json file which redirects the imports to elsewhere)
https://www.npmjs.com/package/@sinclair/typebox?activeTab=code

So, I am open to this being an issue with TypeBox (and would certainly fix if that is the case), but as far as I can tell everything seems to be configured correctly (as indicated by this tool)

Was posting this issue to see if there may be something a miss with the TSP where type definitions are resolved differently to the compiler/language service.

Oops. I didn't see who filed the issue.
It's possible that the playground's implementation of ATA can't handle it. Likely we should yank the new "APA" out of vscode.dev (which runs an actual package manager via wasm in the browser).

To be honest, I don't quite get the layout; the one tshy uses is a lot simpler and likely would work out of the box.

@jakebailey Thanks for the follow up on this!

the one tshy uses is a lot simpler and likely would work out of the box.

Yeah, I've just become aware of tshy, will be taking a deeper look into it the new year.

To be honest, I don't quite get the layout

Yeah, the setup is a quite different. It was selected to support dual publishing and submodule pathing contrasted with the following criteria.

Modules must be resolvable without a tsconfig.json file present.
Modules must be resolvable with moduleResolution configured to  node configured (a common misconfiguration)
Sub modules resolvable in all cases (this one has been challenging)

The package-json-redirect technique seems to achieve the above and makes TS and Node happy locally, it also enables downstream bundlers (esbuild / webpack) to do their thing...however the configuration hasn't been entirely without issue .... it's just that it seems to yield less issues than other layouts I've tried! :)

It's possible that the playground's implementation of ATA can't handle it. Likely we should yank the new "APA" out of vscode.dev (which runs an actual package manager via wasm in the browser).

This would be amazing!


forc plugin registry / index

Purely for discoverability, it would be useful if users could "announce" their custom forc plugins to some centralized, searchable site.
We likely shouldn't take on the responsibility of hosting the plugins themselves as they can be totally arbitrary programs in any language, however it could still be useful to host a site where users can publish an IPFS hash and some metadata about their plugin, or something along these lines.

Related: https://github.com/FuelLabs/sway/issues/2180


Fix protobuf run dependency on old libignition-msgs5 Windows packages

Checklist

[ ] Used a static YAML file for the patch if possible (instructions).
[ ] Only wrote code directly into generate_patch_json.py if absolutely necessary.
[ ] Ran python show_diff.py and posted the output as part of the PR.
[ ] Modifications won't affect packages built in the future.

<!-- Put any other comments or information here --!>

Please submit these patches as YAML.

Sure! I had no idea about the new YAML system and I just noticed it in the PR message! Thanks for setting it up, I will convert this to YAML.

(I am currently with limited bandwith (not a metaphor, I have limited GB on my phone), so I opened this to mark down the logic necessary to encode this https://github.com/conda-forge/libignition-msgs1-feedstock/issues/30#issuecomment-1682551859 . I will iterate on this locally when I have more bandwith.

Superseded by https://github.com/conda-forge/conda-forge-repodata-patches-feedstock/pull/506 .


Add os-diff to the adoption workflow

Just FYI, we need to wait a little to land this PR because I have pushed breaking changes waiting for being merged at the os-diff side: https://github.com/openstack-k8s-operators/os-diff/pull/25

This does not touch tests, i'll merge.


Passed, Completed and Terminated statements do not include duration

I believe that Passed, Completed and Terminated statements must include duration.

+1

Good catch, agreed.

Resolved by commit https://github.com/adlnet/cmi5-Client-Library/commit/964724b1e1ddbcedf1bb4364d9b109474496da16

Great, thank you for this.


Towny - Unable to create new towns (NullPointerException)

Hi,
Describe the bug
Towny isn't able to create a Town when Gringotts is used as economy plugin.
To Reproduce

Setup a Paper 1.18.2 Minecraft server.
Download Vault .7.3-b131.
Download Gringotts 2.12.3.
Download Towny <IP_ADDRESS>.
As a player in-game, execute /town create Test.

Expected behavior
The Towny's town "Test" should be created.
Screenshots

Desktop (please complete the following information):

Server Type : Paper
Server Version : Paper 379 (MC: 1.18.2) (Implementing API version 1.18.2-R0.1-SNAPSHOT) (Git: 276d830)
Gringotts Version : 2.12.3 (compiled from master)

Additional context

Error log : https://pastebin.com/E1aCq0iz
Vault, Gringotts and Towny are using their default configuration.

I have the same issue. I tried with version of Gringotts 2.12.1 and town creation seems to be working ^^
I'm gonna try to track where is the issue between both versions

I have the same issue. I tried with version of Gringotts 2.12.1 and town creation seems to be working ^^ I'm gonna try to track where is the issue between both versions

Oh, you're right.
The problem seems to be with the 8eea3fb commit.
The problem is not present in the previous commit (ae0eac3).

Yes, seems like it's from TownHolderProvider.java line 54.

I rollbacked the changes and built the plugin and it works fine!
Here are the changes that I rollbacked:

NationHolderProvider.java l.52 -> l.64; l.68; l.79 -> l.84
TownHolderProvider.java l.51 -> l.63; l.67; l.78 -> l.83.

The previous code uses deprecated method from Towny but it should be fixable.


I seek and seek and find that token rubish not on google api website.

What Google Rubbish is need to get it work?  It offer me Google.AI, Google Identity Toolkit , TokenServiceAPI, GMAIL-API and and and

Hahaha
I will never come back and look for bether solution with good documentation.


use 'AWS_REGION' instead of 'AWS_DEFAULT_REGION' when formatting credentials output

What type of PR is this?
feature
What this PR does / Why we need it?
When outputting cloud credentials with the -o env flag, the current output will contain the region as AWS_DEFAULT_REGION. Looking at the AWS docs, this seems to be dropping in favor of AWS_REGION. Furthermore, some AWS SDKs don't even acknowledge the AWS_DEFAULT_REGION envar.
Resolves OSD-23927

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 44.58%. Comparing base (a8a9d8a) to head (8eaa112).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #443   +/-   ##
=======================================
  Coverage   44.58%   44.58%
=======================================
  Files          63       63
  Lines        5291     5291
=======================================
  Hits         2359     2359
  Misses       2615     2615
  Partials      317      317

Files
Coverage Δ

pkg/credentials/aws.go
16.66% <ø> (ø)

/lgtm
/approve


minor lattice update to avoid deprecated msg & relocated CPU% to avoid overlap

Hi @schollz ,
Here's a small PR to address the following:

When loading the script: there was a 'new_pattern' is deprecated; use 'new_sprocket' instead. message
The "muted" message and the CPU% were overlapping so it has been relocated to the opposite side of the grid UI

Thank you again for this lovely script and for considering this lil' contribution!

Amazing, thank you sooooo much


Add<EMAIL_ADDRESS>which includes the library's cpp sources.

Hello @wep21, @Vertexwahn, modules you maintain (boost.coroutine) have been updated in this PR. Please review the changes.

@bazel-io skip_check unstable_url


✨ Feature: Concealing aliases

The nature of the feature:

[X] Parser(syntax, conceal etc. related)
[ ] Renderer(style, options etc. related)
[ ] Support(plugin support, language support etc. related)
[ ] Other

Description:
First of all thanks for this great plugin.
Obsidian has a syntax for aliases. It is also supported for instance by Markdown-oxide.
Suggestion
Backlinks such as [[Jane Doe|Jane]] could be concealed to just [Jane] (currently it is concealed to [Jane Doe|Jane]). I assume most people (myself included) use aliases to not have to look at the full link.


Implement GraphQL metrics query

This PR brings a metrics GraphQL query and type. The supported fields correspond to the 2nd-5th SQL queries listed here.

computation of served families incl. filtering
computation of sales incl. filtering
computation of stock overview
computation of moved stock overview

The product list can be fetched via the GraphQL API in a straightforward way.
@aerinsol

I settled for the name metrics instead of statistics. Imho a statistical value is e.g. an average or a variance but not a counted number. Any objections?
What is the difference between the first and the second query in the document?
When computing the number of sales, should we add Transaction.tokens > 0 to avoid counting the transactions that actually represent granted tokens?
When computing moved stock, we use WHERE NOT location.visible but when computing stock overview, we use WHERE location.visible = 1. Is that on purpose?
All queries return info for the current client's organisation. Should the god user be capable of fetching data for any organisation?
I compared the SQL queries generated by peewee with the ones from the document. They match in functionality I reckon. I still have to perform a sanity comparison by running the SQL queries directly on the production database. Sanity comparison was successful!

As usual let me know about any remarks :)

Hey @pylipp , sorry for the late reply. Here are my answers:

Thanks!

What is the difference between the first and the second query in the document?

We have a functionality where people are grouped into families, so each family has a family head (you should see a self-referential parent_id in the field there). So, the difference is counting all individual people vs. family groups.

Is it correct that transactions are always booked via the family head?
The new metrics query only provides the number of served families. I'll add the number of served people, too.

When computing the number of sales, should we add Transaction.tokens > 0 to avoid counting the transactions that actually represent granted tokens?

No, because there are a lot of transactions that don't use any granted tokens (i.e. product prices are zero or null). We should, however, exclude negative tokens, so I will update that query :)

Does it make sense to also update the queries for computing the number of served families/people too? For my understanding, negative tokens correspond to 'granted tokens'?

When computing moved stock, we use WHERE NOT location.visible but when computing stock overview, we use WHERE location.visible = 1. Is that on purpose?

Yes. "invisible' locations are locations that are not counted as stock, i.e., the LOST "location" (should be solved when we correctly handle status via @leo-labs PR), and outgoing locations, where they're being sent on to another location, or being distributed en masse often in a non-choice way.

Ah, I suppose the computation of that query then changes anyway with the update of the box-location-state logic. Did you see my response to this comment related to the topic ?

All queries return info for the current client's organisation. Should the god user be capable of fetching data for any organisation?

Yes. Also I should probably discuss with you and @vahidbazzaz we will probably want to have a way of granting a user API read-only access to all organizations without also granting godadmin access to the application.

Okay, I'll try to extend the PR to make the god user capable.

Is it correct that transactions are always booked via the family head?

Yes (for now).

When computing the number of sales, should we add Transaction.tokens > 0 to avoid counting the transactions that actually represent granted tokens?

No, because there are a lot of transactions that don't use any granted tokens (i.e. product prices are zero or null). We should, however, exclude negative tokens, so I will update that query :)

Does it make sense to also update the queries for computing the number of served families/people too? For my understanding, negative tokens correspond to 'granted tokens'?

No, this is not correct. It most likely refers to reversed purchases. Yes, we can update those queries of served families to exclude negative tokens.

When computing moved stock, we use WHERE NOT location.visible but when computing stock overview, we use WHERE location.visible = 1. Is that on purpose?

Yes. "invisible' locations are locations that are not counted as stock, i.e., the LOST "location" (should be solved when we correctly handle status via @leo-labs PR), and outgoing locations, where they're being sent on to another location, or being distributed en masse often in a non-choice way.

Ah, I suppose the computation of that query then changes anyway with the update of the box-location-state logic. Did you see my response to this comment related to the topic ?

Yes, let's continue the conversation there.

All queries return info for the current client's organisation. Should the god user be capable of fetching data for any organisation?

Yes. Also I should probably discuss with you and @vahidbazzaz we will probably want to have a way of granting a user API read-only access to all organizations without also granting godadmin access to the application.

Okay, I'll try to extend the PR to make the god user capable.

To be clear - there are two roles that are needed, but we can put this in a separate PR. 1) Godadmin can do everything across all organizations. 2) Read-only Boxtribute internal user can read across all organizations.
We can do #1 first.

All queries return info for the current client's organisation. Should the god user be capable of fetching data for any organisation?

Yes. Also I should probably discuss with you and @vahidbazzaz we will probably want to have a way of granting a user API read-only access to all organizations without also granting godadmin access to the application.

Okay, I'll try to extend the PR to make the god user capable.

To be clear - there are two roles that are needed, but we can put this in a separate PR. 1) Godadmin can do everything across all organizations. 2) Read-only Boxtribute internal user can read across all organizations.
We can do #1 first.

This is implemented, I'll merge the PR then.
Thanks for the helpful clarifications :)


toBuffer with streams

Hi! I want to resize an image and then write the output to a Buffer using sharp, where my input is a stream,  but I cannot get it to work. What is the best way to achieve this? I do not want to save the image anywhere, I just want to access the data in the buffer. The images I want to do this for are of type 'svg'. I have tried the following, but it does not seem to generate the correct data in the buffer:
function resizeImage(inputStream, width, height) {
  var resizeTransform = sharp().resize(width, height);
  return new Promise(function (resolve, reject) {
    inputStream.pipe(resizeTransform).toBuffer()
    .then(function(data) {
      console.log(data);
    })
    inputStream.on('end', resolve);
    inputStream.on('error', reject);
    resizeTransform.on('error', reject);
  });
}

Hello, there are some examples of Stream-based input with Buffer(-as-a-Promise) based output in the unit tests - see https://github.com/lovell/sharp/blob/d8765f955de2f037881da7487773ad02cea539cc/test/unit/io.js#L52-L89
Ignoring error handling on inputStream, your example might look something like:
function resizeImage(inputStream, width, height) {
  var resizeTransform = sharp().resize(width, height);
  inputStream.pipe(resizeTransform);
  return resizeTransform.toBuffer();
}

Hi! That helped, thank you very much!

But I am not sure I understand what the buffer 'data' will contain. I read that it is supposed to be the output image data, but I seem to get the same data for every different image that I try. If I for example want to resize an image to be only one pixel, and I want access the RGB value for the colour of this pixel, how can I do this?

I solved this by using the .raw() function!


Add config file

One could control things that are not controllable via CLI:

colors

Custom path for sample and ls could also be taken from config

Custom path for sample and ls could also be taken from config

Very much related to #58.
@AbhayAysola I wonder what would be your usecase? I would imagine it is flexibility. The idea behind putting everything in ~/.mltype is to be able to run mlt sample and mlt sample from anywhere.

That way, I could seperate models by folder to organise them well. Like Python related models in a Python folder, General English related go in English folder etc. When I want to use other models, I would just switch the path. The config file also wouldn't have to be manually updated, a command like mlt config  could also work.

default number of chars to generate
set key combination to exit

The config file logic was added in #89.
One can easily set defaults to all options for any of the commands of the CLI (e.g. number of characters)
Additionally, define any  parameters inside of the general section then can be read anywhere within the code
mltype.utils.get_config_file. So there can be new pull requests using this logic.


visual indicator of session status

an icon or word or possibly both/either should show up somewhere on the screen whenever the client is either hosting (even if no one is connected) or visiting

does this mean it should be a session view after all instead of a scene view


Challenge is invalid! possible missing "/" in url

I have followed the nice example at https://www.aaflalo.me/2016/09/dehydrated-bash-client-lets-encrypt/ and have things setup on an Ubuntu 14.04
But I seem to have a missing slash "/" somewhere and I cant find it.
My output is:
Processing sub.mydomain.dk
 + Signing domains...
 + Generating private key...
 + Generating signing request...
 + Requesting challenge for sub.mydomain.dk...
 + Responding to challenge for sub.mydomain.dk...
ERROR: Challenge is invalid! (returned: invalid) (result: {
  "type": "http-01",
  "status": "invalid",
  "error": {
    "type": "urn:acme:error:connection",
    "detail": "Could not connect to sub.mydomain.dk.well-known",
    "status": 400
  },
  "uri": "https://acme-staging.api.letsencrypt.org/acme/challenge/6LxKELET2eL6diD-ovIPrKE9LHKQNGU2SmZh3MLsDc4/16900890",
  "token": "KP8Um5SjkvoYi41fE2Hmsg7Gn4BClvPx3CjpNcqzTCo",
  "keyAuthorization": "KP8Um5SjkvoYi41fE2Hmsg7Gn4BClvPx3CjpNcqzTCo.RzSA_osSH4_aPw-z2ZyR0ob05YOIOwsnySON7jA3zzE",
  "validationRecord": [
    {
      "url": "http://sub.mydomain.dk/.well-known/acme-challenge/KP8Um5SjkvoYi41fE2Hmsg7Gn4BClvPx3CjpNcqzTCo",
      "hostname": "sub.mydomain.dk",
      "port": "80",
      "addressesResolved": [
        "<IP_ADDRESS>"
      ],
      "addressUsed": "<IP_ADDRESS>"
    },
    {
      "url": "https://sub.mydomain.dk.well-known/acme-challenge/KP8Um5SjkvoYi41fE2Hmsg7Gn4BClvPx3CjpNcqzTCo",
      "hostname": "sub.mydomain.dk.well-known",
      "port": "443",
      "addressesResolved": null,
      "addressUsed": ""
    }
  ]
})

I notice that the hostname is prepended with ".well-known" Could that be the issue ?
Regards and thanks for a nice script
Supergoof

This looks like you are doing a redirect on your webserver, and it is missing the '/' on the redirected path, nothing to do with this script or Let's Encrypt.

I don't think it's that because I have the same problem and if I access the url in the browser with 'http' instead of 'https' it redirects correctly. And the alias is created like in your doc file.

@cdarken you can clearly see that it first tries the http:// url with correct path and only tries the broken path afterwards, and that only happens if your server is sending a redirect. none of my example configs do redirects, so this has to be something else in your configuration.
since i did this mistake myself a few times i know how easy it is to make, check using curl or something else, your browser might not be a good debugging tool if you have/had hsts enabled, it will do its own redirect and not ask the server on what to do.


Add abusive flag

closes #722

This looks good to me at a glance, will let someone more familiar with the codebase also review.
As a sidenote, I noticed we have fewer strings for spam flags (4) than the other types (5 each). Maybe something we should investigate? https://github.com/Charcoal-SE/metasmoke/blob/master/app/models/user.rb#L200

@Undo1 4 different strings is correct for spam flags, whereas the other types of flags have 5. Both English & Portuguese use "spam" for the string.


Add tutorial GIF on how identify yourself

For complete clarity, it would be nice to record a short GIF of how to identify yourself in the Hackathon server and attach this to the post in the registration channel.

No longer relevant due to #5 -- identification will become much simpler! 🎉


kubeadm reset

[ x] Documention problem

kubeadm init and kubeadm reset
This is a...

[ ] Feature Request
[ x] Bug Report

Problem:
I follow this page https://kubernetes.io/docs/setup/independent/high-availability/ to install kubernetes-1.9.2 when I execute kubeadm --init --config=config.yaml due to download of kubernetes core components The network is slow and I need to terminate kubeadm init execution and re-edit config.yaml. Need to perform kubeadm reset, but when this command is executed, delete my /etc/kubernetes/pki/ directory, my etcd cluster key and related key are in the /etc/kubernetes/pki/etcd directory, Kubeadm reset deletes one of my etcd nodes' secret keys. The official documentation for the installation is to put the etcd key in /etc/kubernetes/pki/etcd. Is this conflict?
[root@k8s-master-001 kubernetes]# kubeadm reset
[preflight] Running pre-flight checks.
[reset] Stopping the kubelet service.
[reset] Unmounting mounted directories in "/var/lib/kubelet"
[reset] Removing kubernetes-managed containers.
[reset] No etcd manifest found in "/etc/kubernetes/manifests/etcd.yaml". Assuming external etcd.
[reset] Deleting contents of stateful directories: [/var/lib/kubelet /etc/cni/net.d /var/lib/dockershim /var/run/kubernetes]
[reset] Deleting contents of config directories: [/etc/kubernetes/manifests /etc/kubernetes/pki]
[reset] Deleting files: [/etc/kubernetes/admin.conf /etc/kubernetes/kubelet.conf /etc/kubernetes/controller-manager.conf /etc/kubernetes/scheduler.conf]
Proposed Solution:
Page to Update:
https://kubernetes.io/...
https://kubernetes.io/docs/setup/independent/high-availability/

@IBUMBLEBEE :wave: This issue sounds more like a request for support and less like an issue specifically for docs. You can bring your question to the #kubernetes-users channel in Kubernetes slack. You can also search resources like Stack Overflow for answers to similar questions.


chore: add runtimeArgs in VSCode launch.json

This is a fix to ensure debugger stops at correct breakpoint in VSCode
Refs: https://github.com/facebook/jest/issues/6918

Summary
The contributors who use VSCode editor are not able to use debugger, as it doesn't stop at correct points as described in #6918
The issue was discussed in https://github.com/jest-community/vscode-jest/issues/372 where @connectdotz pointed out debugger configuration to be added as Jest uses flow (documentation)
This PR adds that debugger configuration "runtimeArgs": ["-r", "flow-remove-types/register"] in VSCode launch configuration
Test plan

Screen recording of the issue:

Screen recording of the fix:

Thanks for the PR! I don't use VS Code, so pinged a couple on the team who does 🙂

Yeah, this should be fine 👍


Fix outstanding backup manager issues

This PR continues the work in #30 which wasn't quite finished.
The main fix so far is that --column-statistics=0 is added to the backup script.

Currently having a problem: while trying to more rigorously test database backups, I can backup and restore a full-data site (great).  However, when I try to backup and restore the "empty" database, I'm getting the error ERROR 1146 (42S02) at line 322: Table 'mysql.innodb_index_stats' doesn't exist.  Here are my repro steps:

Use this branch
Start with everything down (python deploy_tool.py local_windows dc down to be sure)
Create DB (python deploy_tool.py local_windows create_db)
Apply "empty" database (python deploy_tool.py local_windows restore_empty_db)
Start site (python deploy_tool.py local_windows start_site)
Verify site's up in browser: http://localhost
SSH into backup manager on port 22001
a. /backup_mysql.sh
Unzip .gz file created in sqlbackup
Tear down site (python deploy_tool.py local_windows dc down)
Create DB second time (python deploy_tool.py local_windows create_db)
Attempt to restore DB (python deploy_tool.py local_windows restore_db)

This is where I get the error.  FWIW, the backup created at 7a is 5.4 MB whereas empty_schema.sql is 1.5 MB (probably generally as expected since the site start adds a bunch of transient state stuff I assume)

I think this might have come about because the db container and the backup_manager container are running different versions of the mysql tools. (5.7 v's 8.0).
On backup_manager:
# mysqldump -V
mysqldump  Ver 8.0.33-0ubuntu<IP_ADDRESS> for Linux on x86_64 ((Ubuntu))

# mysqldump --host ropewiki_db --all-databases --add-drop-database --single-transaction --user=root --password=NotReal
[...]
/*!40000 ALTER TABLE `columns_priv` ENABLE KEYS */;
UNLOCK TABLES;
mysqldump: Couldn't execute 'SELECT COLUMN_NAME,                       JSON_EXTRACT(HISTOGRAM, '$."number-of-buckets-specified"')                FROM information_schema.COLUMN_STATISTICS                WHERE SCHEMA_NAME = 'mysql' AND TABLE_NAME = 'columns_priv';': Unknown table 'COLUMN_STATISTICS' in information_schema (1109)

On db:
# mysqldump -V
mysqldump  Ver 10.13 Distrib 5.7.42, for Linux (x86_64)

# mysqldump --host ropewiki_db --all-databases --add-drop-database --single-transaction --user=root --password=NotReal
[..... no error.....]

It maaaay be better to have them match. But from what I read just adding the --column-statistics=0 is acceptable too.

I think the MySQL version mismatch is probably the likely cause, but I'm not sure what to do about it.  The above procedure still reproduces the problem for me (see below), I don't see an easy way to install mysql-client 5.7, and we'd like to update the server but that's certainly a project for another day.  Prepping the DB by applying this doesn't immediately fix the issue, but I may be applying it the wrong way.
c:\bjp\ropewiki\app>python deploy_tool.py local_windows restore_db
2023-05-23T23:33:46.613240 Restoring backup c:\ropewiki\mount\sqlbackup\all-backup-2023-05-24-063222.sql...
2023-05-23T23:33:46.614240   (NOTE: this operation usually takes a few minutes)
2023-05-23T23:33:46.620242   RUN type c:\ropewiki\mount\sqlbackup\all-backup-2023-05-24-063222.sql | docker container exec -i local_windows_ropewiki_db_1 mysql -uropewiki -pthispasswordonlyworksuntildbisrestored ropewiki
mysql: [Warning] Using a password on the command line interface can be insecure.
ERROR 1146 (42S02) at line 322: Table 'mysql.innodb_index_stats' doesn't exist
The process tried to write to a nonexistent pipe.
2023-05-23T23:33:47.395071 Error running command 'type c:\ropewiki\mount\sqlbackup\all-backup-2023-05-24-063222.sql | docker container exec -i local_windows_ropewiki_db_1 mysql -uropewiki -pthispasswordonlyworksuntildbisrestored ropewiki'; rerunning to capture result...
2023-05-23T23:33:47.395071   RUN type c:\ropewiki\mount\sqlbackup\all-backup-2023-05-24-063222.sql | docker container exec -i local_windows_ropewiki_db_1 mysql -uropewiki -pthispasswordonlyworksuntildbisrestored ropewiki
mysql: [Warning] Using a password on the command line interface can be insecure.
ERROR 1146 (42S02) at line 322: Table 'mysql.innodb_index_stats' doesn't exist
The process tried to write to a nonexistent pipe.

ERROR: Return code 1

I wonder if restoring the DB via the backup_manager makes a difference. (So the same tools were used to both create and restore the db).
e.g. deploy_tool would run something like....
cat backup.sql | docker container exec -i dev_ropewiki_backup_manager_1 mysql -h ropewiki_db ...

I've tested these new changes by successfully doing the following:

Perform the reproduction steps above, but observe that restore_db completes successfully
Following step 1, make a change to the site, backup again, clear everything, load backup, confirm site change is present
Restore the all-backup-2023-05-28-100001.sql production backup locally and confirm that the site looks ok
Restore the same backup to dev.rope.wiki along with rsync'd images folder, and visually verify front page looks good

The approach now taken is to just use the same exact base image from the database container for backup manager.  The suggestion to interact with the DB almost exclusively through backup manager is also adopted as that seems like good practice.


Implimented parsley instance selector UI and functionality

Implemented a parsley instance selector functionality with a pop up UI that allows you to select an instance of parsley to connect to.

This change is

I pushed the requested changes

Reviewed 2 of 3 files at r12, 1 of 1 files at r13, all commit messages.
Reviewable status: all files reviewed, 8 unresolved discussions (waiting on @Darth-Raazi, @KavinSatheeskumar, @MaisimZaman, and @Rio6)

sinks/dashboard/dashboard.py line 251 at r11 (raw file):
Previously, BluCodeGH (BluCode) wrote…
bump. Please click on the reviewable button to reply to specific comments.

I removed self.last_received

Outdated PR, closing


Gateway: Enforce a plain-text message size limit

We have now confirmed that compression does allow for larger plain-text messages to make it through bottle's body size check.  It's only lookin at the 'raw', and thus compressed if applicable, size.
Normal gzip will compress things to 10-11% their original size, and thus the 1MiB bottle limit means potentially a ~10MiB plain-text message.
However it's possible to craft nasty gzip bombs that are small on the wire but decompress to much smaller.  Thus we should see if zlib has any way to limit the amount of memory/buffer it will use in decompression and throw an exception if exceded.

Suggestion from A_D for how to enforce a plain-text limit:
import gzip, io
incomingstream: bytes = None # wherever this comes from
stream_file = io.bytesIO(incomingstream) # this can be skipped if incomingstream is already a file (eg a socket)
gz_file = gzip.GzipFile(fileobj=stream_file)

buffer = b''
while len(buffer) < MAX_BUFFER:
  buffer += gz_file.read(1024)

And Spansh has supplied a 1GB gzip bomb, see https://discord.com/channels/164411426939600896/205369618284544000/934051715957792818


call the success callback after the features are added

fixes #12138

Thanks, @kikitte!

@kikitte Cool!


Only deploy endpoint from master

The PR changes the CI configuration to only deploy_endpoint on master, and only if extact_endpoint job succeeded

This would still deploy the endpoint if just the extract_endpoint tests passed. I propose instead that we fan-in all stages so that we only deploy anything IIF all tests pass.


No colors in build output

Hi, you have to configure the compiler to output colors, here is my build configuration for C for your reference.
{
  "cmd": ["/usr/bin/clang", "-fcolor-diagnostics", "-std=c99", "-Wall", "-Werror", "-Wno-unused-variable", "-Wno-gnu-empty-initializer", "-Wno-vla", "${file}", "-o", "${file_path}/${file_base_name}.out"],
  "selector": "source.c",
  "file_regex": "^.*?(/.+):(\\d+):(\\d+):.*error:"
}


html2canvas not generating complete canvas  when used in loop.

Hi,
I am using html2canvas version: 1.0.0-rc.5.
facing a peculiar issue when I tried to run html2canvas in a loop.
I am using Html2Canvas Plugin to convert div (containing Highcharts svg, html content ) to canvas. and capture all the methods in a promise Array as shown in the below code.
for (let i = 0; i < svgArray.length; i++) {
promiseArray.push(
html2canvas(element, {
allowTaint: true,
removeContainer: true,
height: height,
width: width,
useCORS: true
});
);
Note: width, height are related to A4 dimensions.
Promise.all(promiseArray).then(responseArray => {
//looping response array to get the images uri
}
The images are not rendering properly as i figured it out that the canvas was not generating completely. when i try to render the images they are either being cutoff at the top or coming completely.
This issue is occurring when run through localhost but occurring consistently when deployed in production.
Can you please help me with this issue? I m struggling like anything, need help !!!
Specifications:

html2canvas version tested with:
Browser & version: Chrome
Operating system: Mac

Same problem

Anybody please respond, I need help badly

I solved this by using setTimeout and promises because I noticed that html2canvas needs a little bit of time between captures.
The following code returns an array of page objects. Each page consisting of a body and the same header.
If you have this HTML:

  Header

Page 1

Page 2

You would use it like this:
capturePagesAsPromises().then(pages => {
  pages.forEach(page => {
    document.body.appendChild(page.header);
    document.body.appendChild(page.body);
  })
})

var HEADER_SELECTOR = '[data-pdf-header]';
var BODY_SELECTOR = '[data-pdf-body]';
var initialTimeBetweenCapturesInMs = 2000;
var timeToIncreaseBetweenCapturesInMs = 2000;

function capturePagesAsPromises() {
  return new Promise(function(resolve, reject) {
    var header = document.querySelector(HEADER_SELECTOR);
    var headerPromise = _captureElement(header);
    var bodies = Array.from(document.querySelectorAll(BODY_SELECTOR));
    var bodiesPromises = bodies.map(function(body) {
      return _captureElement(body);
    });
    var promises = bodiesPromises.map(function(bodyPromise) {
      return Promise.all([headerPromise, bodyPromise]);
    });

    Promise.all(promises)
      .then(function(pageCaptures) {
        resolve(_createPageObjects(pageCaptures));
      })
      .catch(reject);
  });
}

function _captureElement(element) {
  return new Promise(function(resolve, reject) {
    setTimeout(function() {
      html2canvas(element)
        .then(function(canvas) {
          resolve(canvas);
        })
        .catch(reject);
    }, (initialTimeBetweenCapturesInMs += timeToIncreaseBetweenCapturesInMs));
  });
}

function _createPageObjects(pageCaptures) {
  return pageCaptures.map(function(pageCapture) {
    return {
      header: pageCapture[0],
      body: pageCapture[1]
    };
  });
}

Not shown here but make sure you scroll the window to the top and prevent the user from scrolling down during captures because you could get cropped images if you don't.


Conversion of DataFrame to R's data.frame

Although I have already produced code for this (see below) I'm posting this as an issue rather than a pull request to discuss the design, because there are some issues open in my code:

Series of dtype object need an explicit cast or rpy2's numpy conversion will treat them improperly
The performance has not been profiled
Probably some room for optimizations
Proper name for the function
The generation of an intermediate OrdDict object may cause problems in case of very large datasets

The code in the current form is posted below. If there is interest, I will work towards integrating it in pandas.rpy.common and add unit tests.
import numpy as np
import rpy2.robjects as robjects
import rpy2.robjects.numpy2ri as numpy2ri
from rpy2.robjects.packages import importr
import rpy2.rlike.container as rlc

def dataset_to_data_frame(dataset, strings_as_factors=True):

    # Activate conversion for numpy objects
    robjects.conversion.py2ri = numpy2ri.numpy2ri
    robjects.numpy2ri.activate()

    base = importr("base")
    columns = rlc.OrdDict()

    for column in dataset:
        value = dataset[column]

        # object type requires explicit cast
        if value.dtype == np.object:
            value = robjects.StrVector(value)
             #FIXME: how to generalize it?
            if not strings_as_factors:
                value = base.I(value)

        columns[column] = value

    dataframe = robjects.DataFrame(columns)
    dataframe.rownames = robjects.StrVector(dataset.index)

    # To prevent side-effects in other code
    robjects.conversion.pi2ri = robjects.default_py2ri

    return dataframe

I've been experimenting with alternative solutions so far, but at the moment most of them, save for the dict intermediate, are horribly slow.

I think I might get some code to merge in for 0.8.0, but I'd need to adapt my unit tests (which use unittest) to pandas. Is there any document on how unit tests are handled in pandas and any guidelines to follow?

Progress: the current implementation (not integrated in my pandas clone yet as I have no idea on how to handle things like MultiIndex, which I don't use in my normal workflow). The main advantage is that now nans are translated to R's NA:
import numpy as np
import rpy2.robjects as robjects
from rpy2.robjects.packages import importr
import rpy2.rlike.container as rlc

def dataset_to_data_frame(dataset, strings_as_factors=True):

    base = importr("base")
    columns = rlc.OrdDict()

    # Type casting is more efficient than rpy2's own numpy2ri

    vectors = {np.float64: robjects.FloatVector,
               np.float32: robjects.FloatVector,
               np.float: robjects.FloatVector,
               np.int: robjects.IntVector,
               np.object_: robjects.StrVector,
               np.str: robjects.StrVector}

    columns = rlc.OrdDict()

    for column in dataset:
        value = dataset[column]
        value = vectors[value.dtype.type](value)

        # These SHOULD be fast as they use vector operations

        if isinstance(value, robjects.StrVector):
            value.rx[value.ro == "nan"] = robjects.NA_Character
        else:
            value.rx[base.is_nan(value)] = robjects.NA_Logical

        if not strings_as_factors:
            value = base.I(value)

        columns[column] = value

    dataframe = robjects.DataFrame(columns)

    del columns

    dataframe.rownames = robjects.StrVector(dataset.index)

    return dataframe


Angular 14

Fix #1948

[x] Dependencies
[x] Typescript errors to fix

Can you solve the conflict plz @qmonmert ?
As dependabot worked the last night... :-p

@pascalgrimaud done ✔


Dub bug

The "unittest" configuration is built by default. Because of this, when exiting main, assert from btl is triggered
btl:autoptr 1.0.0-rc.4: target for configuration "unittest" is up to date.
btl:string 1.0.0-rc.4: target for configuration "unittest" is up to date.
btl:vector 1.0.0-rc.4: target for configuration "unittest" is up to date

I add new default(empty) configuration for all packages.
The triggered asserts from btl unittest configuration trigger when some memory is not freed. It can happend when string/smart_ptr/vector is inside gc allocated array (dlang bug for copy ctors), for value sequence on the stack (dlang bug for dtors) or some other bug. Can you create some example when this assert trigger in your code?

Thanks, I have a global associative array with weak pointers in my code. It will have to be forcibly removed.


message/state: The event id is required in the response when sending events.

As per https://github.com/matrix-org/matrix-doc/pull/2525. The spec has been fixed.

Thanks! :)


[SMALLFIX] Mark Metrics.registerGauges in default block master

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/19320/
Test PASSed.


[Web] Add auto-formatting to HTML files.

Uses html-eslint for HTML file and eslint-plugin-html for inline JavaScript.
Use HTML5 (not XHTML), remove CDATA and trailing slashes for self closing tags.
Add format checks to CI.
Closes #66291 .
We should double-check the validity of all the HTML changes (auto-closing tags?) and the schema for the editor.html (xhtml doesn't seem correct.)

Thanks!

Thanks!


Fix annotation changes and tuple union changes

Purpose

Fix annotation changes and tuple union changes

Fixes #15947
Fixes #16146
Check List

[x] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[x] Checked Tooling Support (#)
[x] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[x] Integration Tests
[ ] Ballerina By Example Tests

[x] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

:exclamation: No coverage uploaded for pull request base (jballerina@437c497). Click here to learn what that means.
The diff coverage is 72%.

@@              Coverage Diff              @@
##             jballerina   #16168   +/-   ##
=============================================
  Coverage              ?   30.05%
  Complexity            ?     3092
=============================================
  Files                 ?      754
  Lines                 ?    31455
  Branches              ?     4148
=============================================
  Hits                  ?     9453
  Misses                ?    21161
  Partials              ?      841

Impacted Files
Coverage Δ
Complexity Δ

...xtproviders/FunctionDefinitionContextProvider.java
82.45% <ø> (ø)
17 <0> (?)

...providers/ExternalFunctionBodyContextProvider.java
66.66% <66.66%> (ø)
5 <5> (?)

...lerinalang/langserver/completions/TreeVisitor.java
76.62% <66.66%> (ø)
110 <0> (?)

...server/completions/util/CompletionVisitorUtil.java
84.68% <85.71%> (ø)
21 <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 437c497...3478c85. Read the comment docs.


Fix minor typos

Many thanks for submitting your Pull Request :heart:!
Please specify parts this PR updates:

[x] Specification
[ ] Schema
[ ] Examples
[ ] Extensions
[ ] Roadmap
[ ] Use Cases
[ ] Community
[ ] TCK
[ ] Other

What this PR does / why we need it:
Fix minor typos in specification.md
Special notes for reviewers:
Additional information (if needed):

Nice! Thank you @yzhao244

@tsurdilo I followed the instructions from the following DCO details link. However, the DCO check still failed :(
(https://github.com/serverlessworkflow/specification/pull/390/checks?check_run_id=2668355843)

Can you please sign your commit?

git commit --amend --no-edit --signoff
git push --force-with-lease origin typo-fix

@tsurdilo I followed the instructions from the following DCO details link. However, the DCO check still failed :(
(https://github.com/serverlessworkflow/specification/pull/390/checks?check_run_id=2668355843)

Your commit is not signed :(

Yeah I'm not sure how it works, but I usually get problems, too. Added autosign feature to VisualStudio, it does use my GPG key, but DCO systematically fails if not using bash to signoff


Information about GSoC

#62
@EverVino

I just did a very superficial review here .. for now looks good .. let's move it forward we can treat any issue here later.
thanks @Anavelyz


Unwanted Jars are sent to Bnd from bundle-plugin

After digging deeper into problem here (https://github.com/bndtools/bnd/issues/1276) I've found that problem partially comes from this gradle-bundle-plugin - implementation actually sends all transitive dependencies to Bnd, not the ones explicitly defined in build configuration. This, of course, has no sense for building of the bundle. Such behavior occurs at least with Gradle 2.9.
    static File[] getClasspath(Jar jarTask) {
        def project = jarTask.project
        def configuration = project.configurations.runtime.copyRecursive()
        def excludeDependencies = project.bundle.excludeDependencies
        excludeDependencies.each {
            configuration.exclude(it)
        }
        configuration.files
    }

more appropriate implementation is as of now
    static File[] getClasspath(Jar jarTask) {
        def project = jarTask.project
        def configuration = project.configurations.runtime.copyRecursive()
        def excludeDependencies = project.bundle.excludeDependencies
        def classPath = []

        project.configurations.runtime.resolvedConfiguration.firstLevelModuleDependencies.each { dependency ->
            dependency.moduleArtifacts.each { artifact ->
                classPath << artifact.file
            }
        }

        classPath
    }

Additionally, doc/readme says
Exclusion of dependencies from the classpath passed to Bnd
bundle {
    exclude module: 'guava'
    exclude group: 'org.jmock'
}

according to Gradle's docs ( https://docs.gradle.org/current/javadoc/org/gradle/api/artifacts/Configuration.html#exclude(java.util.Map) ) this works only for transient dependencies and does not work for dependencies defined in 'compile' or 'runtime' configuratons.

@xtracoder, hmm, I don't know why they put that into the docs, you can
actually check that it works not only for transient dependencies.
On 20 December 2015 at 21:18, xtracoder<EMAIL_ADDRESS>wrote:

After digging deeper into problem here (
https://githubcom/bndtools/bnd/issues/1276) I've found that problem
partially comes from this gradle-bundle-plugin - implementation actually
sends all transitive dependencies to Bnd, not the ones explicitly defined
in build configuration This, of course, has no sense for building of the
bundle Such behavior occurs at least with Gradle 29
static File[] getClasspath(Jar jarTask) {
    def project = jarTaskproject
    def configuration = projectconfigurationsruntimecopyRecursive()
    def excludeDependencies = projectbundleexcludeDependencies
    excludeDependencieseach {
        configurationexclude(it)
    }
    configurationfiles
}

more appropriate implementation is as of now
static File[] getClasspath(Jar jarTask) {
    def project = jarTaskproject
    def configuration = projectconfigurationsruntimecopyRecursive()
    def excludeDependencies = projectbundleexcludeDependencies
    def classPath = []

    projectconfigurationsruntimeresolvedConfigurationfirstLevelModuleDependencieseach { dependency ->
        dependencymoduleArtifactseach { artifact ->
            classPath << artifactfile
        }
    }

    classPath
}

Additionally, doc/readme says
Exclusion of dependencies from the classpath passed to Bnd
bundle {
exclude module: 'guava'
exclude group: 'orgjmock'
}
according to Gradle's docs (
https://docsgradleorg/current/javadoc/org/gradle/api/artifacts/Configurationhtml#exclude(javautilMap)
) this works only for transient dependencies and does not work for
dependencies defined in 'compile' or 'runtime' configuratons
—
Reply to this email directly or view it on GitHub
https://github.com/TomDmitriev/gradle-bundle-plugin/issues/43.

--
Yours sincerely,
Artyom Dmitriev

Re: you can actually check that it works not only for transient dependencies.
Yes, it does work. Most probably i've made a 'typo' or so when experimenting and it was not excluded in my test.


Website uses project-name/org-name instead of org-name/project-name

Usually everything on GitHub uses the format org-name/project-name.
Is it intentional that the website (https://forgoodfirstissue.github.com/) has these reversed?

Came to report this as well. 😅
Looks like that is rendered from here (and yes, they are indeed backwards):
https://github.com/github/forgoodfirstissue/blob/732d8f2409db04f2416ad0447ac82c4ffd6dce89/components/RepositoryItemTopBar.tsx#L28-L35

Offered up a PR:

https://github.com/github/forgoodfirstissue/pull/83


Delete LoJaxSmallAgent.exe

You can clone the repo using HTTPS of GitHub CLI without any issue... if you are trying to download the zip, then some error may occur.

LoJaxSmallAgent.exe is just for testing purpose, don't run it on your working system.

Where is file LoJaxSmallAgent.exe sirr ?

Can I request the file, please?

LoJaxSmallAgent.exe was a malicious file that's why GitHub asked to delete
it from the repository. Sorry, I don't have that file with me anymore. You
can try downloading some other malicious executable file from the internet.
On Tue, Jul 11, 2023 at 4:13 PM DendyGandara @.***>
wrote:

Can I request the file, please?
—
Reply to this email directly, view it on GitHub
https://github.com/Kiinitix/Malware-Detection-using-Machine-learning/pull/1#issuecomment-1630593016,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIJS5VLLGWC7QX6OYXNOH5DXPUU6PANCNFSM5WASLRHQ
.
You are receiving this because you modified the open/close state.Message
ID: <Kiinitix/Malware-Detection-using-Machine-learning/pull/1/c1630593016@
github.com>

can u send file to my email ? please sirr, thnk u
my email :
<EMAIL_ADDRESS>

I was googling for that executable file and found this repository with my
old code. You can download that file from this repository. And thank you
for this query, If it weren't for you I would have never discovered this
repository and this guy didn't even mention me in the reference list, so I
raised an issue. Thank you. You can mail me anytime if you encounter any
problems with the code.
Repo Link: https://github.com/Amol512/Malware_Detector
On Tue, Jul 11, 2023 at 5:13 PM DendyGandara @.***>
wrote:

can u send file to my email ? please sirr, thnk u
my email :
@.***
—
Reply to this email directly, view it on GitHub
https://github.com/Kiinitix/Malware-Detection-using-Machine-learning/pull/1#issuecomment-1630671105,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIJS5VOL3C5BMJFYZF5DW73XPU34PANCNFSM5WASLRHQ
.
You are receiving this because you modified the open/close state.Message
ID: <Kiinitix/Malware-Detection-using-Machine-learning/pull/1/c1630671105@
github.com>

thnk u sir

Welcome to antimalware detector

PE scanner
URL scanner
Exit

Enter your choice : 2
'python3' is not recognized as an internal or external command,
operable program or batch file.
Do you want to search again? (y/n)
sir please why always show 'python3' is not recognized as an internal or external command,
operable program or batch file." i am using windows 10 with cmd
thnk u

sirr this program cant be running in os windows ?
can u have a program in windows? thnk u

ok, I'll try dockerizing it soon.

Done dockerizing it. Accordingly modified the readme. You can refer to it for installation using docker.


Unlicensed popups are back

I think I'm supposed to submit an issue when this happens? Have manually updated using https://a.dove.isdumb.one/list.txt no luck.

yep. i met 2. what should we do to help troubleshoot this problem?

"Adobe tends to update their telemetry checking subdomain every two weeks. Feel free to open an issue if you already tried updating the list and nothing happened. It'd be appreciated if you could use Fiddler / Charles Proxy when opening the program, sort by "/integritychecker/machineevents/v1" and include the subdomain it was requested from. Please include the URL as text instead of just pasting a screenshot, your submission will be rejected if you don't paste the URL."
Also, please stop using that first word

I don't understand "sort by "/integritychecker/machineevents/v1" and include the subdomain it was requested from" but using Charles and some trial & error, I've found that the IP causing the problem is a2104gz1mh.adobe.io

I don't understand "sort by "/integritychecker/machineevents/v1" and include the subdomain it was requested from" but using Charles and some trial & error, I've found that the IP causing the problem is a2104gz1mh.adobe.io

This was added some days ago thanks to your contribution. Have a nice day!


Critical errors upgrading from v7.5.2

@jonyo, I wish I would have run a test upgrade sooner. In any case:

It fails hard on 7.5.3 -> 7.6.0

From the looks of it, the problem is because mySQL is running in strict mode and thus default values are required for all fields.

So, the table is:

I updated the table to set the DEFAULT to NULL (which is what you see above. There was no default value before.
Anyway, that moves us along to the next error:

Make sure you set strict mode to true in config, I think that is default.

Make sure you set strict mode to true in config, I think that is default though.

Yep, mine is set to '0'
Let me change and try again.

ha ha, yep. That was the issue. Thanks @jonyo


As of right now a LocationEntity can use AMovableBody subclasses.

❗This issue is mostly a reminder, no need to rush fixing this.
Proposed solution

[ ] #50

This issue is being handeled by another user story issue


UI for service

What type of PR is this?
/kind feature

What this PR does / why we need it:

implement ui for service management

Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

- add management for service in the dashboard

/status in-progress

/assign @RainbowMango @samzong

/status in-review

/status in-review

Please cc me again once it is ready for review.

/status in-review

Please cc me again once it is ready for review.

@RainbowMango i've rebased the code and recheck the feature locally, let's move forward.


SuggestedAction support in Skype channel

Version
Microsoft.Bot.Builder 4.0.7
Bot Framework Emulator (V4 PREVIEW) version: 4.0.0-preview.40025
Describe the bug
SuggestedAction is not supported in Skype (Windows, app. v. <IP_ADDRESS>)
To Reproduce
Send a message activity containing a SuggestedAction
await context.SendActivityAsync(MessageFactory.SuggestedActions(new CardAction[] { new CardAction(type: ActionTypes.PostBack, title: "Test it", text: "Test it", value: "Show me the shop catalog") }));

When executing last statement, I always get an exception Operation returned an invalid status code 'BadRequest'
   at Microsoft.Bot.Connector.Conversations.ReplyToActivityWithHttpMessagesAsync(String conversationId, String activityId, Activity activity, Dictionary`2 customHeaders, CancellationToken cancellationToken) in D:\a\1\s\libraries\Microsoft.Bot.Connector\Conversations.cs:line 1121
   at Microsoft.Bot.Connector.ConversationsExtensions.ReplyToActivityAsync(IConversations operations, String conversationId, String activityId, Activity activity, CancellationToken cancellationToken) in D:\a\1\s\libraries\Microsoft.Bot.Connector\ConversationsExtensions.cs:line 241
   at Microsoft.Bot.Builder.BotFrameworkAdapter.SendActivitiesAsync(ITurnContext turnContext, Activity[] activities, CancellationToken cancellationToken) in D:\a\1\s\libraries\Microsoft.Bot.Builder\BotFrameworkAdapter.cs:line 298
   at Microsoft.Bot.Builder.TurnContext.<>c__DisplayClass22_0.<<SendActivitiesAsync>g__SendActivitiesThroughAdapter|1>d.MoveNext() in D:\a\1\s\libraries\Microsoft.Bot.Builder\TurnContext.cs:line 267
--- End of stack trace from previous location where exception was thrown ---
   at Microsoft.Bot.Builder.TurnContext.SendActivityAsync(IActivity activity, CancellationToken cancellationToken) in D:\a\1\s\libraries\Microsoft.Bot.Builder\TurnContext.cs:line 196
   at Microsoft.eShopOnContainers.Bot.API.Dialogs.Help.HelpDialog.ProcessHelpChoices(WaterfallStepContext dc, CancellationToken cancellationToken) in C:\Users\frank\Work\eShoConAI\src\Bots\Bot.Core.API\Dialogs\Help\HelpDialog.cs:line 54
   at Microsoft.Bot.Builder.Dialogs.WaterfallDialog.OnStepAsync(WaterfallStepContext stepContext, CancellationToken cancellationToken) in D:\a\1\s\libraries\Microsoft.Bot.Builder.Dialogs\WaterfallDialog.cs:line 108
   at Microsoft.Bot.Builder.Dialogs.WaterfallDialog.RunStepAsync(DialogContext dc, Int32 index, DialogReason reason, Object result, CancellationToken cancellationToken) in D:\a\1\s\libraries\Microsoft.Bot.Builder.Dialogs\WaterfallDialog.cs:line 130
   at Microsoft.Bot.Builder.Dialogs.WaterfallDialog.ResumeDialogAsync(DialogContext dc, DialogReason reason, Object result, CancellationToken cancellationToken) in D:\a\1\s\libraries\Microsoft.Bot.Builder.Dialogs\WaterfallDialog.cs:line 103
   at Microsoft.Bot.Builder.Dialogs.DialogContext.EndDialogAsync(Object result, CancellationToken cancellationToken) in D:\a\1\s\libraries\Microsoft.Bot.Builder.Dialogs\DialogContext.cs:line 169
   at Microsoft.Bot.Builder.Dialogs.Prompt`1.ContinueDialogAsync(DialogContext dc, CancellationToken cancellationToken) in D:\a\1\s\libraries\Microsoft.Bot.Builder.Dialogs\Prompts\Prompt.cs:line 98
   at Microsoft.Bot.Builder.Dialogs.DialogContext.ContinueDialogAsync(CancellationToken cancellationToken) in D:\a\1\s\libraries\Microsoft.Bot.Builder.Dialogs\DialogContext.cs:line 130
   at Microsoft.Bot.Builder.Dialogs.ComponentDialog.ContinueDialogAsync(DialogContext outerDc, CancellationToken cancellationToken) in D:\a\1\s\libraries\Microsoft.Bot.Builder.Dialogs\ComponentDialog.cs:line 65
   at Microsoft.Bot.Builder.Dialogs.DialogContext.ContinueDialogAsync(CancellationToken cancellationToken) in D:\a\1\s\libraries\Microsoft.Bot.Builder.Dialogs\DialogContext.cs:line 130
   at Microsoft.eShopOnContainers.Bot.API.Dialogs.Main.RouterDialog.OnContinueDialogAsync(DialogContext innerDc, CancellationToken cancellationToken) in C:\Users\frank\Work\eShoConAI\src\Bots\Bot.Core.API\Dialogs\Main\RouterDialog.cs:line 34
   at Microsoft.Bot.Builder.Dialogs.ComponentDialog.ContinueDialogAsync(DialogContext outerDc, CancellationToken cancellationToken) in D:\a\1\s\libraries\Microsoft.Bot.Builder.Dialogs\ComponentDialog.cs:line 65
   at Microsoft.Bot.Builder.Dialogs.DialogContext.ContinueDialogAsync(CancellationToken cancellationToken) in D:\a\1\s\libraries\Microsoft.Bot.Builder.Dialogs\DialogContext.cs:line 130
   at Microsoft.eShopOnContainers.Bot.API.eShopBot.OnTurnAsync(ITurnContext turnContext, CancellationToken cancellationToken) in C:\Users\frank\Work\eShoConAI\src\Bots\Bot.Core.API\eShopBot.cs:line 69
   at Microsoft.Bot.Builder.AutoSaveStateMiddleware.OnTurnAsync(ITurnContext turnContext, NextDelegate next, CancellationToken cancellationToken) in D:\a\1\s\libraries\Microsoft.Bot.Builder\AutoSaveStateMiddleware.cs:line 60
   at Microsoft.Bot.Builder.MiddlewareSet.ReceiveActivityWithStatusAsync(ITurnContext turnContext, BotCallbackHandler callback, CancellationToken cancellationToken) in D:\a\1\s\libraries\Microsoft.Bot.Builder\MiddlewareSet.cs:line 55
   at Microsoft.Bot.Builder.BotAdapter.RunPipelineAsync(ITurnContext turnContext, BotCallbackHandler callback, CancellationToken cancellationToken) in D:\a\1\s\libraries\Microsoft.Bot.Builder\BotAdapter.cs:line 167

Same code works in emulator. I also tested with other ActionTypes (ActionTypes.ImBack and ActionTypes.PostBack). I also tested other cases, for instance, a suggested action containing many CardAction, and it also fails.
Expected behavior
SuggestedAction/CardAction works in Skype channel
[bug]

Sorry to bother again ... but your affirmation is not totally correct
Adding the following line:
await context.SendActivityAsync(MessageFactory.SuggestedActions(new CardAction[] { new CardAction(type: ActionTypes.PostBack, title: "Test it", text: "Test it", value: "Show me the shop catalog") }, text: "I don't need this prompt"));

I've got the following output in skype:

which is not what I want, because I don't need the extra prompt. Besides, without the text parameter, works flawlessly in the emulator.
But one thing is correct: adding the text parameter, the click is handled correctly and I don't get any exception. But the behaviour is very different between the emulator and Skype app

All channels have their own behavior as the rest API for each channel is not exactly the same.  The emulator is not a good baseline for channel behavior.  For example, some channels do not even render suggested actions like slack.  Skype requires text or an attachment to be included with the activity or you will get the error you were seeing.   To debug locally in a channel see this blog.  If you are targeting a specific channel this is how you should do you debugging while developing because as I mentioned before the emulator is not the same as every channel.


Issues with Quick Open (⌘⇧O)

Maybe I'm too fast, but when I type something on the IDE, then decided to open another file by using Quick Open, it dismisses the Quick Open box, and I accidentally make changes to the file, which I had to erase and do a Quick Open again, only to have it dismiss and for me to accidentally make a change on the file and correct it and reopen Quick Open... I could go on forever.
The only temporary workaround is to... WAIT. After making a change on the file, I have to wait maybe 1 or 2 seconds, then open Quick Type, type a file name and open it.
It's not a big issue right now. But when I do stuff fast, it is an issue that I'd like to be removed. The Xcode Copilot works great, but there are some instances like this that makes me want to not use it.
Here's the current version and settings I'm using.

You can revert to use version 0.11.0, which newer uses Apple Scripts to get real-time suggestions. I will bring the tweak I disabled in 0.11.1 back in the next release.

A new version 0.11.2 is released


Combine streets

Define which segments are one street

Find street names/ geographical information

How to submit an issue?

[x] Have you checked previously open/closed issues to see if your question has already been asked & addressed?

If there is an open issue similar to your issue. Comment on that open issue with additional detail indicating that you too are having the same issue. This allows the project sponsor to prioritize incoming issues.

[x] Have you provided a clear description of your issue so that the project sponsor is able to answer all your questions with as few back-n-forths as possible?
[x] (Please acknowledge by checking this box) If a screenshot or visual helps better communicate some complex idea, have you provided one?. Draw.io, Lucidchart, and Autodraw are some excellent and quick drawing tools.
[x] (Please acknowledge by checking this box) When an issue is resolved, please post the steps to resolution or clearly describe the resolution before marking the issue closed.

This will help future SOLVERS read through how your issue was solved and may help them solve their issue quicker.

[x] Have you been kind, courteous, and considerate in your questions?
[x] Have you considered using Stackedit.io, a handy tool to quickly write in markdown?

Hmm @jazzzchan can you please provide more detail for this? The description as it stands does not seem clear.


Rename get* to build*

:tada: This PR is included in version 4.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Insufficient handling of capabilities

In the module class, there is a function HasCapability.  It is suppose to return true of the module has a certain capability.  It is implemented by looking for the capability in question in the list of OpCapability instruction.  The problem is that this is not enough.
When one capability depends on another, the other is implicitly enabled, even if it is not explicitly present.  This needs to be improved.

@alan-baker suggested we add this to the feature manager.  As we read each capability, the feature manger keeps track of all explicitly and implicitly enabled capabilities.
The name of the function on the module could change to HasExplicitCapability or something like that.

The validator handles this transitive closure in ValidationState_t::RegisterCapability.
Agree the optimizer needs the transitive closure too.

Fixed by #1122


Add workflow for publishing images to ghcr.io

Description

This PR adds a new workflow for publishing the API, ingestion server, index worker, and analytics docker images to ghcr.io every time a release is cut. GitHub should just auto-magically allow this repo to publish under ghcr.io/wordpress, but we'll see!
This will help eliminate the need to set up Dockerhub credentials as repository secrets. GHCR is also free for open source projects!
This PR is sort of a priority because we have so little space on the catalog EC2 box that I don't want to build the image there, I'd rather pull it from somewhere (but we also don't have easy access to the Dockerhub creds).
See the repositories! :tada: :

https://ghcr.io/wordpress/openverse-api
https://ghcr.io/wordpress/openverse-ingestion-server
https://ghcr.io/wordpress/openverse-index-worker
https://ghcr.io/wordpress/openverse-analytics

Testing Instructions

(will be tested when this PR is opened)
Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[x] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

@dhruvkb it is fully free for OSS repos: https://github.com/features/packages#pricing

Referencing #429 as it affects work done in this PR.

I made the PR #437 as a proof-of-concept of how #429 would affect the workflow in this repo in a positive way.

Pinging @rbadillap here so you can take a look at this. It could be relevant to future work on an api deployent pipeline.

Closing in favor of #437


ZendDeveloperTools is not installable with new skeleton application

First I install the new skeleton application and choose the minimal install with no optional packages:
$ composer create-project "zendframework/skeleton-application:dev-develop" zend-project

Then I want to install the ZendDeveloperTools:
$ composer require zendframework/zend-developer-tools:dev-master

I get the following messages:
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Installation request for zendframework/zend-developer-tools dev-master -> satisfiable by zendframework/zend-developer-tools[dev-master].
    - zendframework/zend-developer-tools dev-master requires zendframework/zend-mvc 2.* -> satisfiable by zendframework/zend-mvc[2.0.3, 2.0.4, 2.0.5, 2.0.6, 2.0.7, 2.0.8, 2.1.0, 2.1.1, 2.1.2, 2.1.3, 2.1.4, 2.1.5, 2.1.6, 2.2.0, 2.2.0rc1, 2.2.0rc2, 2.2.0rc3, 2.2.1, 2.2.10, 2.2.2, 2.2.3, 2.2.4, 2.2.5, 2.2.6, 2.2.7, 2.2.8, 2.2.9, 2.3.0, 2.3.1, 2.3.2, 2.3.3, 2.3.4, 2.3.5, 2.3.6, 2.3.7, 2.3.8, 2.3.9, 2.4.0, 2.4.0rc1, 2.4.0rc2, 2.4.0rc3, 2.4.0rc4, 2.4.0rc5, 2.4.0rc6, 2.4.0rc7, 2.4.1, 2.4.10, 2.4.2, 2.4.3, 2.4.4, 2.4.5, 2.4.6, 2.4.7, 2.4.8, 2.4.9, 2.5.0, 2.5.1, 2.5.2, 2.5.3, 2.6.0, 2.6.1, 2.6.2, 2.6.3, 2.7.0, 2.7.1, 2.7.2, 2.7.3, 2.7.4, 2.7.5, 2.7.6, 2.7.7, 2.7.8] but these conflict with your requirements or minimum-stability.

Installation failed, reverting ./composer.json to its original content.

there is pull request to that support https://github.com/zendframework/ZendDeveloperTools/pull/210 waiting for merge

new PR for it with incorporated feedbacks https://github.com/zendframework/ZendDeveloperTools/pull/213 , if you want to test:
Steps to reproduces:

add "zendframework/zend-developer-tools": "dev-zf3" to your composer.json inside require-dev
add my forked ZDT to your composer.json

"repositories" : [ {
"type" : "vcs",
"url" : "git@github.com:samsonasik/ZendDeveloperTools.git"
}],

composer update
register 'ZendDeveloperTools' to your config/application.config.php
cp vendor/zendframework/zend-developer-tools/config/zenddevelopertools.local.php.dist config/autoload/zenddevelopertools.local.php

Fixed now, can be closed. Just tested with new Skeleton


autodetect ANSI capabilities only if user DOES NOT define use_colors

If the user defines use_colors, then it does not try to detect terminal capabilities, it just uses it.
This solves some scenarios like logging to consoles that do not report correctly if they support ANSI colors or not. Example: Jenkins, IntelliJ.
If the user wants the to use the autodetect capabilities, then use_colors should be None.

Coverage increased (+2.1%) to 93.659% when pulling 980aef2fb6860d2e2f159cca80aa2afa7a78d8e5 on pabloa:master into<PHONE_NUMBER>bc39057f824c545f2decab49c019ad on xolox:master.

Hi and thanks for the feedback. I've just published coloredlogs release 14.1 which fixes this issue.


Hyprpaper IPC: Wallpaper all monitors

Currently in the Hyprpaper configuration using the following line would set your wallpaper for all monitors.
wallpaper = ,/path/to/image.png
However when doing the same using IPC it fails silently, giving an OK.

(the wallpaper didn't change but IPC gave an OK)
Expected Behaviour
The wallpaper for all monitors should of changed but no wallpapers changed at all.

I have the same problem even now:

Hyprland 0.40
hyprpaper 0.7


feat: allow adding content that is not fully materialized

For example when generating a backup.
The whole file never exists outside of iroh, but rather it should be streamed into iroh, piece by piece and at the end be stored by iroh (on disk or in memory) but without the user having to fully have it in memory or on disk at any point in time.

I don't quite understand this. How can it be neither on disk nor on memory?
Currently you can write into a file, then add this file either by reference or by reflink (on a FS that supports it), which is 0 overhead. What more do you want?

Looking at the store api, there is import_bytes to import a single Bytes, import_stream to import a stream of Bytes, and import_reader which is implemented in terms of the former usually.
They don't really do anything incrementally right now, but just create the file as a temp file and then compute an outboard and change it to a permanent file. But in principle they could.
I see that in the API there is this:

https://docs.rs/iroh/latest/iroh/client/struct.BlobsClient.html#method.add_from_path
https://docs.rs/iroh/latest/iroh/client/struct.BlobsClient.html#method.add_reader
https://docs.rs/iroh/latest/iroh/client/struct.BlobsClient.html#method.add_bytes

In the store API there are these 3 but also

https://docs.rs/iroh-bytes/0.9.0/iroh_bytes/store/trait.Store.html#tymethod.import_stream

Should we just expose the one below as well and be done with it?

Sounds good, lets just have parity 👍 and yes seems that the basic functionality I was looking for is already there


Training CI cleanup catch-all

Most items from review discussion on #1557

[ ] fix FULL_TRAIN so that training does not run twice on when FULL_TRAIN is true
[ ] establish if re-installation of deepspeed is necessary and if it is a fix in the CLI or in the training repo
[ ] remove packaging installation in the e2e workflow. Figure out why it is causing failures since it is a requirement in training repo
[ ] update training-lib based training to do training based on the output of ilab data generate instead of a file checked in to the tree - scripts/test-data/train_all_pruned_SDG.jsonl
[ ] update docs/ci.md to reflect where the training-lib is used

@cdoern did https://github.com/instructlab/instructlab/pull/1557 fix this?

@nathan-weinberg no this should be handled as part of the CI cleanup that I think @alimaredia is working on


#398 - tidy up queries

Should close #398.
Optimized some queries on dashboard and vacation requests page - removed some duplicated queries and cached some similar queries with laragear/cache-query.

Should we add key CACHE_QUERY_STORE to .env file?

Added

I can't create request, because type of request are not fetch

It happens when I have APP_URL like localhost:8751 - I use this, because on localhost login by Google works - on blumilk environment the above error doesn't occur.

I can't create request, because type of request are not fetch

It happens when I have APP_URL like localhost:8751 - I use this, because on localhost login by Google works - on blumilk environment the above error doesn't occur.

Did u try do that on other branch?
Maybe its general problem?

Okay, after removing 's' from https://toby.blumilk.localhost/ it works (login by Google and creating requests)


Introduced SQL reporting extension

Solves #77

Example report from SampleStories here:
reports.zip
Example screen:

@mbocek changed our intercepting template from inheritance to composition, can you please review again?


Pytest color is not showing up running as a task

I created a Test task that runs the py.test. This plugin can show some color but not quit good. But vscode-log-output-colorizer do not show color at all.

The Python extension is overriding the colorization in the Output panel.  There can only be one active colorizer in the output channel at any time - this is a limitation with VS Code itself.  It's been reported that the Python, Code Runner, and Quokka extensions will cause conflicts.  There are also a few others that I can't remember the names of off hand.  If you want the Output Colorizer to have precedence, then you might have to disable the Python extension (or others).


Implemented func: Create Node

Few questions:
1, are we use the advertise address as a docker daemon endpoint?
2, what about the first node want join into the cluster?

@cmingxu  This PR is trying to solve the problem: we have to add docker tcp entrypoint again, although we have give the node advertiseAddr

1, are we use the advertise address as a docker daemon endpoint?
Yes, It make sense

2, what about the first node want join into the cluster?
The PR don't solve it. We need init the first manager still.

retest this please

@vitan I think the advertise address and docker daemon endpoint are different things, "advertise address" is a swarmkit concept for inter node connectivity while docker daemon is for API request(both tcp and unix sock are fine)

@cmingxu  Sure. But both the IP value of advertise addr and docker daemon endpoint are constantly same one. Please show me an negative example if possible.

@cmingxu  I will Merge it firstly. Please make another PR is any problem


Cannot create new environment with templates

Hi
I am trying to create a new environment which includes a template.
Despite populating the "Templates" parameter, the result is a standard blank Dataverse environment.
I used the code below
  name: create-environment action
  uses<EMAIL_ADDRESS>  id: create_environment
  with:
    user-name: ${{ github.event.inputs.admin_username }}
    password-secret: ${{ github.event.inputs.admin_password }}
    name: ${{ github.event.inputs.env_name }}
    region: ${{ github.event.inputs.env_region }}
    type: SubscriptionBasedTrial
    currency: ${{ github.event.inputs.env_currency }}
    language: ${{ github.event.inputs.env_language }}
    domain: ${{ github.event.inputs.env_domain }}
    templates: ${{ github.event.inputs.env_templates }}

This is the resulting output

Is it possible to specify a template? Am I missing something? The same parameter works on Power Shell Cmdlets.
Thank you!

@gabrielesilvagni  sorry for the delay in reply :-(
investigating on our side; since you have not attached any error spew, I assume in your case, the action succeeded without showing errors or failures, but the resulting env is not configured with the template, correct?

Yes, you are correct. The operation succeeded but no template has been applied.

the create-environment actions simply hands the list of templates through to the EnvManagement API. If this reproes, we would need a more detailed log for us to have our partner team investigate.
If such reproes are consistent, it will also reproduce when calling pac admin create ... --templates <templates>. The CLI does keep a local trace log that we'd need to investigate; it is placed in a subfolder logs/ next to the pac.exe's location


Enforce MudListItem type with runtime exception

Reported by @jairbubbles:

Hi everyone, I had a small issue with the Dense parameter in MudListItem. There was a mismatch between T in the list and in the item and as a result the items were not "dense" while the list itself was. (the MudList cascading parameter was not properly set in GetDense() )
I created a small snippet to illustrate the issue:
https://try.mudblazor.com/snippet/cucokMulIzoXqFIR
I feel like we should have an error if the generic types don't match. What do you think?
Please note that in my case, there were nested components involved so the issue was not that obvious to understand.

Originally posted by @jairbubbles in https://github.com/MudBlazor/MudBlazor/issues/8447#issuecomment-2263161174

Would be nice to implent this. I see more and more that people use different generic type in MudList and MudListItem


waitForFrames timeout - all calls changed to default - 5s

all calls to waitForFrames timeouts changed to default - 5 sec

must check

corrected - now ok


Sample Flutter app doesn't run, lib/main.dart not found

Sorry, I can't copy out from the console. Here's a screenshot:

sky_tools 0.0.16 is in the pubspec.lock (latest at this time)
closed all my atom windows
New window
Packages, Dart, Create new Flutter app
Waiting until done
Cmd-R

Atom 1.0.19, dartlang 0.4.9, SDK 1.13.0-dev.2.0 running on macos.

FWIW I stopped the run using the little square. I then clicked the little x. Then I hit Cmd-R again. Same error, but this time, I got a bigger stack trace.

Can you try running .packages/sky/sky_tool install from the CLI and see if the issue is that the APK is outdated on the device?

My sky_tools wasn't updating, so it's very possible I had my plugin, local project, and globally activated packages all out of sync.
Closing this because I manually updated sky_tools. I'll keep an eye on it.


Fix opencoarrays_version in windows-install.sh

Avg response time
coverage on master

Summary of changes
Updated the definition of opencoarrays_version in windows-install.sh to reflect a months-old change in version numbering.
Rationale for changes
Build on Windows Subsystem for Linux was failing.
Additional info and certifications
This pull request (PR) is a:

[X] Bug fix
[ ] Feature addition
[ ] Other, Please describe:

I certify that

[X] I reviewed and followed the contributing guidelines, including
- Increasing test coverage for all feature-addition PRs
- Increasing test coverage for all bug-fix PRs for which there
does not already exist a related test that failed before the PR
- At least maintaining test coverage for all other PRs
- Ensuring that all tests pass when run locally
- Naming PR  to indicate work in progress (WIP) and to attach the PR
to the appropriate bug report or feature request issue
- White space (no trailing white space or white space errors may
be introduced)
- Commenting code where it is non-obvious and non-trivial
- Logically atomic, self consistent and coherent commits
- Commit message content
- Waiting 24 hours before self-approving the PR to give another
OpenCoarrays developer a chance to review my proposed code

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.**** seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@zbeekman I went back to commits as old as late April and found that the Windows installer has been broken for a rather long time.   This caused me to realize that we could actually be testing this script in Travis-CI.  Because the script runs under Windows Subsystem for Linux, it's actually just a bash script that expects to run on Ubuntu 16.04 or later.  There is nothing in the script specific to Windows.  Please set up for Travis-CI to test this script if it's easy to do so.  Let me know if I should create an issue for this.

ah, sorry, this was my fault. I  made the change that originally caused this and neglected to update the windows script.
LGTM

The shebang at the start of the file is wrong: https://github.com/sourceryinstitute/OpenCoarrays/pull/441/files#diff-761c977a48b8e2a4ee665b286772ca5dR1
missing a ! after the leading #
also, shellcheck has many warnings

also style issues:
$ ./developer-scripts/style.pl windows-install.sh
windows-install.sh[112]: [ -z "${LOG_LEVEL:-}" ] && emergency "Cannot continue without LOG_LEVEL. "
windows-install.sh[115]: if [[ "${arg_d}" == "${__flag_present}" ]]; then[ ]
windows-install.sh[117]:    if [ "$(( LOG_LEVEL < print_debug_only ))" -ne 0 ]; then
windows-install.sh[147]: export this_script="$(basename "$0")"
windows-install.sh[161]:   opencoarrays_version=$text_before_language_keyword
windows-install.sh[166]: export build_path="${OPENCOARRAYS_SRC_DIR%/}"/prerequisites/builds/opencoarrays/$opencoarrays_version
windows-install.sh[197]:   if [[ $major_release -lt 16  ]]; then
windows-install.sh[199]:   elif [[ $major_release -eq 16  ]]; then
windows-install.sh[200]:     if [[ $minor_release -lt "04"  ]]; then
windows-install.sh[209]:     info "$opencoarrays_version"
windows-install.sh[230]: [ ]
windows-install.sh[233]:   if ! type "$CMAKE" >& /dev/null; then
windows-install.sh[236]:   if ! type "$CXX" >& /dev/null; then
windows-install.sh[239]:   if ! type "$FC" >& /dev/null; then
windows-install.sh[242]: [  ]
windows-install.sh[271]: [  ]
windows-install.sh[274]:   if [[ -d "$build_path" ]]; then
windows-install.sh[275]:     rm -rf "$build_path"
windows-install.sh[277]:   mkdir -p "$build_path"
windows-install.sh[278]:   cd "$build_path"
windows-install.sh[280]:   info "FC=\"$FC\" CC=\"$CC\"  \"$CMAKE\" \"$OPENCOARRAYS_SRC_DIR\" -DCMAKE_INSTALL_PREFIX=\"$install_prefix\""
windows-install.sh[281]:   FC="$FC" CC="$CC" "$CMAKE" "$OPENCOARRAYS_SRC_DIR" -DCMAKE_INSTALL_PREFIX="$install_prefix"
windows-install.sh[283]:   info "make -j $arg_j"
windows-install.sh[284]:   make -j $arg_j
windows-install.sh[288]:   if [[ -f "$install_prefix"/lib/libcaf_mpi.a && -f "${install_prefix}/bin/caf"  && -f "${install_prefix}/bin/cafrun"  ]]; then
windows-install.sh[290]:     info "$install_prefix"
windows-install.sh[293]:     emergency "$install_prefix"
windows-install.sh[296]:   loopback_line=`grep $NAME /etc/hosts`
windows-install.sh[299]:     info "<IP_ADDRESS> $NAME"

@rouson hmmm it also appears that git wasn't configured correctly, so it didn't record the author info correctly. As a consequence, it thinks that the CLA needs to be signed again. I can probably forcibly fix this myself.

yeah, it looks like you didn't configure git properly in a new environment:
Author<EMAIL_ADDRESS> Date:   Fri Sep 1 17:54:31 2017 -0700

LGTM
let's merge after tests run & pass


web site is offline

plz fix kgr8thx

sorry, fixed!

Getting timeouts: https://downforeveryoneorjustme.com/zapalytics.com

home wifi where i host is having downtime, should hopefully be back up soon