Direct metadata writing for segmentation pipelines

Was having major trouble getting neuroconv.tools.roiextractors.write_segmentation to properly handle fully custom metadata for multiple plane segmentations + RoiResponseSeries; likely because none of our previous ophys testing data had been built with those assumptions/capability, and since I'm running out of time to get this project done I just opted to do that part directly through PyNWB. I plan to look back and refactor/simplify it in NeuroConv once I have some time.
Also the single- vs. dual- color have different metadata, so I split those altogether from the globally common metadata, so now there are multiple .yml files and the main script decides which ones to pull from. Easier than having a single big/messy one that then gets selectively modified in the code.

Found the source of the issue, summarized in sister PR on HDMF: https://github.com/hdmf-dev/hdmf/pull/780
Added fix here, too. Will add tests tomorrow or next week


Total annihilation

This software deleted every file on every computer in my house. How can I recover them?

Hmm, it was supposed to burn your house done once that was done.  I'll need to push some updates to the destroy_house method and will get back to you.
[For the loyal hoards following this project (0 as of now), this troll is my pal of many decades. If only I could delete his computers / house from here.]


[Specifications] Compile QA list for Shannon Feature Descriptions

Objective
Compile a list of open questions per feature to seed docs + community coms
Origin Document
Ask via Community Builder Call
Goals

List of questions that will be answered async by the dev team
Facilitate community coms to align everyone on available features for future Shannon releases

Deliverables

[ ] Question List
[ ] Answers for each Question
[ ] Format for community coms

Non-goals / Non-deliverables

Anything beyond documentation
Morse

Creator: [ @RossiNYC ]
Co-Owners: [ @Olshansk ]


⚠ Bofrost.be has degraded performance

In 6ef3645, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1225 ms

Resolved: Bofrost.be performance has improved in a460c73.


Kibana 6.5.x Support

Any chance for 6.5.x support? Kibana 6.5.x has some nice features we want to use, but we cannot go without your awesome plugin.

I will port to 6.5 as soon as I fix a couple of issues/regressions in 6.4.x

Quick look shows this will not be a simple port, expect a week or so before I can get it completed.

Hi there
thanks for your email. 6.5.1 is 100%, thanks. And I do understand that you
do it during your free time, so please, no pressure. Thanks a lot.
Johannes Mahne
c: +27 81 381 9988 <0813819988>
e<EMAIL_ADDRESS>www.conor.co.za | Suite 4, Block C, Southdowns Office Park, 22 Karee St,
Centurion, South Africa
https://maps.google.com/?q=22+Karee+St,+Centurion,
+South+Africa&entry=gmail&source=g
On Sat, 24 Nov 2018 at 17:36, Pierre Padovani<EMAIL_ADDRESS>wrote:

Quick look shows this will not be a simple port, expect a week or so
before I can get it completed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ppadovani/KibanaNestedSupportPlugin/issues/100#issuecomment-441375950,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADIXLHMRQHsVpszmKyHZeryrBFCUU-XGks5uyWd4gaJpZM4Yu4zJ
.

I have some changes on my laptop... 6.5.X is broken... they refactored a bunch of things that I was untangling before work and life got in the way. I'm hoping to find some time this week and weekend to get back to the code. If you try and build the code as it stands it will not work for 6.5.X.

Patiently waiting for this release. Downgrading Kibana from 6.5.x to 6.4.2 seems to have broken a lot of things for me.

Is 6.4.2 of my plugin broken? My holiday vacation is coming up soon, so I'll have more time to spend finishing up the 6.5.X support. My goal is to have it complete before the end of the year.

Update: Partial success... have it up and running, still a bunch of issues to resolve... but am confident this is doable. Will post an update in a few days.

I wouldn't even know where to begin diagnosing the issue. A copy of Kibana without this plugin behaves better? For example, sometimes it get's stuck and won't show any new data. Even after unselecting all filters, refreshing browser, etc. I have to stop the Kibana process in order for it to show me new data.

Hi! Any news here? I add myself to the queue of concerned people. Plugin is not working at all with 6.4.2 for me. Not even detecting well non-nested objects, and breaking the whole discover view.

Hey folks... I have found my time severely limited due to work and life. Additionally even though I was able to get things partially up, the changes that the Kibana team made in moving to React will require mostly rewriting large chunks of what I've done.
Bottom line, I don't know when/if I'll get this version done. I will spend time over this next week fixing older versions. If anyone wants to take a stab at this feel free to give it a go.

@ppadovani you might want to add an up-for-grabs label to this issue (and others) and also submit it to https://up-for-grabs.net/

Thanks for pointing that out... submitted.


Lower aten.as_strided when it can be simplified into ttnn.reshape

Ticket

Partially resolves #587

Problem description
As per discussion in #587, we only convert aten.as_strided that is equivalent to ttnn.reshape.  This would convert all our known input variations for now.

A general implementation is best done as a kernel op. Its element rearrangement is far more complicated than slicing. https://pytorch.org/docs/stable/generated/torch.as_strided.html
However, our use cases can be simplified into reshape. The stride argument matches the default strides of size for each case. I am working on a conversion that only lowers these cases. https://github.com/tenstorrent/pytorch2.0_ttnn/blob/main/docs/operations/aten.as_strided.default.md

What's changed

[x] Convert aten.as_strided to ttnn.reshape when such simplification is correct
[x] Pass auto generated test cases
[x] Pass manual test cases

CI tests running at https://github.com/tenstorrent/pytorch2.0_ttnn/actions/runs/12471891387


f-strings for Berry

Now that I have some decent experience with programming in Berry, I'm am not completely happy with string formatting. string.format() is slightly verbose. I would like to add an equivalent to Python's f-strings.
Example:
string.format("id=%i cluster=0x%04X name=%s", id, cluster, self.name)
would become:
f"id={id:i} cluster={cluster:04X} name={self.name}"
I made a POC that sounds promising. Changing the parser would be a daunting task. Instead, I use a "transpiler" approach. The lexer contains code to detect f-string syntax and rewrite the code to generate the string.format syntax.
The lexer transforms f"id={id:i} cluster={cluster:04X} name={self.name}" to ("id=%i cluster=0x%04X name=%s", id, cluster, self.name) and then just feeds the normal parser.

Also see discussion on Discord.
https://discord.com/channels/479389167382691863/826169659811168276/1120818883713040476

Done: https://github.com/berry-lang/berry/pull/355

Hi @s-hadinger
Is that working only as compile-time like in python ? which means that I can't use a string variable ?
I'm looking if there is a way to use format string like in python:
# fulltopic = "{prefix}/{topic}/{subtopic}"
fulltopic = fulltopic=string.replace(string.replace(tasmota.cmd("fulltopic")['FullTopic'],"%prefix%","{prefix}"),"%topic%","{topic}")+"{subtopic}"
# publish_topic = "cmnd/devname/power"
publish_topic = fulltopic.format(prefix="cmnd", topic="devname", subtopic="power")
# or
publish_topic = fulltopic.format(**{"prefix":"cmnd", "topic":"devname", "subtopic":"power"})

As you imagine the idea is to use named variable in the format string instead of positional values
Thanks

f-strings are purely compile time synctactic sugar to the format() function. You could however write a helper function that does something similar.

Yes, indeed I simply string replace:
fulltopic=tasmota.cmd("fulltopic")['FullTopic']

def make_topic(prefix, topic, subtopic)
  return string.replace(string.replace(fulltopic,"%prefix%",prefix),"%topic%",topic) + subtopic
end

If you really want, you can have non-constant f-strings expanded by going via compile, to get the syntactic sugar expanded, but that's not very efficient.
var prefix='stat' var topic='device' var leaf='what'
var fformat=/fst-> compile("return f'" + fst + "'")()
print(fformat('{topic}/{prefix}/{leaf}'))

I wouldn't recommend using compile with dynamic content. This is always dangerous ; like eval() in many languages


code for getting the retrieval results

Hi, thanks for the nice work! I've been having trouble getting the retrieval results recently. Would you mind sharing the code that got the retrieval  results?

Hi, unfortunately I don't have easy access to that code anymore, but we did not do anything special for our retrieval results beyond what's mentioned in the paper, using off-the-shelf BM25 and Contriever. Thanks for your interest!

Could you inform whether any Python library is used for BM25 retrieval? It seems very slow to search the entire Wiki_DPR text for each entity. Or any techniques to speed it up?

@Chengkai-Huang Yeah, I also because of the slow retrieval, with a batch of 1024 sentence matrix operation may be a bit faster, but the overall speed is still very slow.

We used https://pypi.org/project/rank-bm25/ for BM25

@Chengkai-Huang Same Question, it takes 55.31s to retrieval 1 question, and it takes 200 hours to complete the BM25 retrieval of the dataset.
@AlexTMallen Hi, could you share the BM25/contriever result file of the popQA dataset for download?

Here are the retrieved docs for EQ and PopQA!
EntityQuestions_bm25_500_retrieved_docs.json
PopQA_BM25.jsonl.zip

Here are the retrieved docs for EQ and PopQA! EntityQuestions_bm25_500_retrieved_docs.json PopQA_BM25.jsonl.zip

Thanks, that helps a lot!

Hello. Many thanks for sharing BM25's retrieval results, is there any possibility for sharing contriver as well? I might miss it, but I think the google drive's sharing link for retriever results just points to an empty folder.

Unfortunately I don't believe I have access to those anymore.

EntityQuestions_bm25_500_retrieved_docs.json

Great! Another question, If I do not hope to test the retriver, could I directly use the s_uri and o_uri as the retrived results for testing? It might be an oracle results of the retriving, if I understand correctly


Split Form.validate

Split Form.validate into Form.validate and _get_extras for better sub-classing.

Apologies for not getting to this for so long.
Your PR is failing the style check part of the CI. WTForms uses the Black code style. If this is still something you're interested in having included into WTForms please fix the code style.

@omarryhan I rebased your patch to get the latest changes.
This patch would require tests involving that _get_extras method, else you won't have a guarantee that a future refactoring won't break this.
What do you think?

Sure.
I pulled the branch that you rebased but I couldn't run the tests, because of this error:
  File "/home/omar/.local/lib/python3.6/site-packages/setuptools/wheel.py", line 156, in _convert_metadata
    os.rename(dist_info, egg_info)
PermissionError: [Errno 13] Permission denied: '/home/omar/wtforms/.eggs/pytz-2019.3-py3.6.egg/pytz-2019.3.dist-info' -> '/home/omar/wtforms/.eggs/pytz-2019.3-py3.6.egg/EGG-INFO'

It looks like an issue with my Windows Subsystem Linux. Unfortunately, this is the only environment I am able to work on right now.
Initially, I wanted to create the extra method in the PR to make this lib work with https://github.com/omarryhan/sanic-wtf, but then I figured that it wasn't necessary. So feel free to close this pull request if you don't feel like writing tests for this PR.
Thanks

Thanks for working on this and following up. I'm going to close it, I don't think we need to split it apart. The standard way to override Form.validate isn't affected by this:
class MyForm(Form):
    def validate(self):
        result = super().validate()

        if not result:
            return False

        return my_other_validation

Also, #584 allows passing extra_validators to validate.


feat: adding upgrade support

adding upgrade support in python ext + fixing evidence run dev to work locally + inside codespaces

@edgarrmondragon Thank you for the help with the CI changes - I haven't used ruff before 😅


Database structure

Basic database structure has been added, along with some basic seeding functions.
It is working as intended, but would like some feedback on the current iteration of the branch. Otherwise it can go straight into main and can be amended at a later date when we need to make changes.

I was able to cleanup the Main.go by using 1 seed function, and also created a singular tierlist to start with associations. I wanted to talk about the Items > Tiers relationship more. Currently, they are uncategorised from the seeding.
Its likely that I would need to hardcode the Items within the Tiers section:
		Tiers: []models.Tier{
			{Text: "S", Colour: "f4ee1d"},
			{Text: "A", Colour: "d81621"},
			{Text: "B", Colour: "00bef1"},
			{Text: "C", Colour: "37f100"},
			{Text: "D", Colour: "ffffff"},
			{Text: "F", Colour: "000000"},
		},

                  Tiers: []models.Tier{
                      {
                          Text: "S",
                          Colour: "f4ee1d",
                          Items: []models.Item{
                              {Text: "Item 1", Image: "item1.png"},
                              {Text: "Item 2", Image: "item2.png"},
                          },
                      },
                      {
                          Text: "A",
                          Colour: "d81621",
                          Items: []models.Item{
                              {Text: "Item 3", Image: "item3.png"},
                          },
                      },

I am not entirely sure if we want to do that as of yet, or keep things uncategorised for now.

I'm backwards and forwards on the best place for Items. I think for now to keep it simple we can just have a "soft link" between Tiers and Items. Where Items reference a Tier by its text field, if an Item references a Tier that does not exist we just count it as uncategorised. e.g. Item with tier field "S" will be grouped under a Tier that has i "S" as its text field, if the Item has a tier field of "U", it'd be uncategorised since know Tiers exist with that text field.
Can come back in the future to make a more concrete link once we've got a better feel on the flow and interactions around the tierlists.

Not that I disagree because I'm not entirely sure either, but just looking through the code - when creating the seeding data, we have currently set it up as three tierlists but they all have the same tier texts ("S", "A" etc...). If we were to create a soft link via the text boxes, we would have to check the Tier: Text & Tier: TierlistID for a match, otherwise it is "U" for uncategorised if it cannot find a tier with the corresponding tierlistid.
We can have:
type Item struct {
	ID int `gorm:"primaryKey" json:"id"`
	TierlistID int `json:"tierlist_id"`
	Text string `json:"text"`
	Image string `json:"image"`
	TierID int `json:"tier_id"`
	TierText string `json:"tier_text"` // this field has been added
	Tier Tier `gorm:"foreignKey:TierID"`
	Tierlist Tierlist `gorm:"foreignKey:TierlistID"`
}

TierText would provide no link via Gorm unless we create a check function (BeforeCreate) which will do the above and I guess create the link for us. Otherwise, we keep it very simple and manually add a TierText: "S" or "U" and have no link for now, with a view of amending as you said, when we get a better understanding.

Amended so that the TierText is available on items and also manually added data to our current tables. Each tierlist currently has a "U" for uncategorised but as with all our work, it's subject to change.
I may even add a default as "U" if not given a TierText via the struct.
Let me know your thoughts.


Add Quarkus json logging extension

Related jira issue:
https://issues.redhat.com/browse/DBZ-4114

Simple change overall. It's worth noting that with this extension the logs are JSON formatted by default.
If that's a concern, it can be overridden with the config quarkus.log.console.json=false, but I'm unsure where that could be set in the codebase

I think it's fine to change the format to JSON by default, as long as users can set it back. We'll only have to call out the changed default behavior in the release notes. @jpechane, any objections?


no windows either

I guess this is just an iOS software tool, this DOES NOT RUN on Windows... will play with it and see what makes it tick.

Well, it is a React library, you have to include it in your javascript project in order to use it.


Update database-instant-file-initialization.md

Add the "Perform volume maintenance tasks" right to the service account, as the setup.exe is doing it. Downside when using an AD account: Configuration manager will not show the setting, as it just looks for the StartName of the service. We wrote about the topic here: https://blog.ordix.de/instant-file-initialization-microsoft-sql-server-set-up-check

@WilliamDAssafMSFT
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@andreasjordan We appreciate your feedback. I have reached out to the security team. It may take weeks or months to get a response and update the content, and we'll let you know here when it happens.

#sign-off


StreamClientTest now spins up an instance of interledger-rs in a docker container (via testcontainers)

Necessary ILP accounts are created as part of test setup
Signed-off-by: nhartner<EMAIL_ADDRESS>

It doesn't make sense to me to put a test in the Java connector project for a class in the Quilt project. If there's a bug in StreamClient, it should be caught as part of its project build. Especially when you have an open issue like https://github.com/hyperledger/quilt/issues/229 that will need additional tests to verify the fix.
My reason for making this change the way I did was:

The build was failing on this test
The client should be tested against something that is close as possible to a real server
The test can download and spin up the Interledger-rs docker image quickly (under 5s)
The Java ILP connector is not dockerized yet and I assumed it still does not fully implement streams
The change set is small and self-contained

But let's pick the best method to test this class/module. I'll defer to your judgement if you think we are betting off testing StreamClient by hitting StreamReceiver.

@nhartner Your rationale makes sense to me - let's keep going with this one using Interledger-rs.
Maybe also:

Once Java Connector get dockerized, let's consider using that instead of, or in addition to, the Rust Connector in this harness to provide solid integration testing of the client (so we know the client works against the Java stack too).

Decide how best to test the StreamServer. I'll move that over to Slack.


REMIND of debug

I notice that when some of you do debug of JS files, some errors will occur.
It may be the issue of firebase since every updates of our feature will probably have minor changes on firebase structure, if you do debug of the features, I suggest you might try to read the code first to understand the new updated structure. Sometimes you need to delete the previous structure to avoid errors.

OK


custom error page

make a custom 404 page, so when user acces the wrong url, 404 page will be shown

not just 404, need to make custom for 500,404,400 ect page


Show error messages in terminal when peer creation fails

Expected Behavior
Show error messages on terminal
Current Behavior
In terminal no error messages, however CC says it has error messages
Possible Solution
In the end peer creation show output of error file and make "Press Enter to finish installation", so user user can read before closes
Steps to Reproduce

Create peer
While peer creation, press Destroy when Peer status is PoweOFF or Not Ready

Context (Environment)
All OS

it will look like this on MacOS

Win10

Done in Master.

fixed on prod.


[Question] What is the fundamental difference of LM and custom templates?

I find them quite similar in concept. Should them not be the same?

Closing due to inactivity. You are welcome to open new issues for further questions :-)


fast/extras/0-cicd-github creates corrupted PNG (base 64 encoded content)

Describe the bug
Applying fast/extras/0-cicd-github creates corrupted PNG output and ultimately fails with error.
Environment
❯ terraform -version
Terraform v1.5.6
on darwin_amd64

Also tried with opentofu (same result)
❯ tofu --version
OpenTofu v1.8.1
on darwin_amd64

❯ git rev-parse --short HEAD
0420dec3

To Reproduce
Create a tfvars with any repo:
repositories = {
  fast_00_bootstrap = {
    create_options = {
      description = "FAST bootstrap."
      features = {
        issues = true
      }
    }
    populate_from = "../../stages/0-bootstrap"
  }

Execute: terraform init + apply
Look at PNG in the output - they are base64 encoded text, not the original binary content.
Expected behavior
Files in the new repos to be intact, and mostly identical to the original.
Result
Terraform output and/or error messages
github_repository_file.default["fast_02_networking/IAM.md"]: Creation complete after 36s [id=fast_02_networking/IAM.md]
github_repository_file.default["fast_00_bootstrap/IAM.md"]: Creation complete after 35s [id=fast_00_bootstrap/IAM.md]
github_repository_file.default["fast_01_resman/outputs-gcs.tf"]: Creation complete after 36s [id=fast_01_resman/outputs-gcs.tf]
github_repository_file.default["fast_00_bootstrap/variables.tf"]: Creation complete after 36s [id=fast_00_bootstrap/variables.tf]
github_repository_file.default["fast_00_bootstrap/diagram.png"]: Creation complete after 36s [id=fast_00_bootstrap/diagram.png]
github_repository_file.default["fast_00_bootstrap/outputs.tf"]: Creation complete after 36s [id=fast_00_bootstrap/outputs.tf]
╷
│ Error: unsupported content encoding: none
│
│   with github_repository_file.default["fast_02_networking/diagram-ncc.png"],
│   on main.tf line 159, in resource "github_repository_file" "default":
│  159: resource "github_repository_file" "default" {
│
╵

Additional context
Add any other context about the problem here

I can't see how https://registry.terraform.io/providers/integrations/github/latest/docs/resources/repository_file actually supports uploading a binary file (there's no flag to let it know that the content is base64 encoded).

This commit totally doesn't work (for me)
https://github.com/GoogleCloudPlatform/cloud-foundation-fabric/commit/a514ce0ef5743b065c1f1788f0b9ec3d9d3650b0#diff-75d6741a4ff5654842d3a983ac5376d3485db0914e89f004825fda996162c569
@jayBana

See https://github.com/integrations/terraform-provider-github/issues/710

My guess is that the actual error message comes from around here.
But due how the code looks as of now you should not get this exact message. This suggests that you use v5.43.0 or earlier of github provider. Is this the case? Can you try with newer version?

My last attempt was with opentofu:
❯ tofu --version
OpenTofu v1.8.1
on darwin_amd64
+ provider registry.opentofu.org/hashicorp/tls v4.0.5
+ provider registry.opentofu.org/integrations/github v4.31.0

PS: it "works fine" if I comment out the *.png line in main.tf - but obviously doesn't include those files in the output repo.
Will reinstall terraform, and see which version it installed. Sec.

Even with latest terraform, and running terraform init it installed:
❯ terraform --version
Terraform v1.9.4
on darwin_amd64
+ provider registry.terraform.io/hashicorp/tls v4.0.5
+ provider registry.terraform.io/integrations/github v4.31.0

Updating required_providers to "~> 6.0"
❯ terraform --version
Terraform v1.9.4
on darwin_amd64
+ provider registry.terraform.io/hashicorp/tls v4.0.5
+ provider registry.terraform.io/integrations/github v6.2.3

Then applying the stock terraform, I didn't get any error, but the PNG is still corrupt:

Given there seems to be little to fix on our side, can we close this issue?

Maybe it would be worth it to mention this in the stage README, what do you think?

Yes, documenting "this process will corrupt any binary files, including PNG".  In the alternative, maybe a non-terraform solution would work better?


Form fields styling

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
❌

Fixed issues

👍


Add package configuration feature

Is there a way to tell eb_deployer how ti should package the application?
Perhaps something similar to ebs-deploy
Thanks!

Hi, @thiagofelix
How to package application is kind of platform dependent. For platform like java packaging is a complex process, should always handled by build tools such maven and gradle. On the other hand, on platform like ruby and node (maybe python), packaging is as simple as single line of shell command "git ls | zip @".  So I think it is better let end user doing the packaging.


markdown compatibility

I was looking at http://<IP_ADDRESS>:5080/_browse/#/local-npm and the "How it works" section tells of the http://<IP_ADDRESS>:16984/_utils entry point - but because the markdown parser used seems a little bit flaky (I haven't investigated any further yet) the url http://<IP_ADDRESS>:16984/utils is shown instead with a bit in italic (because of the two underline characters. Compare the output with https://github.com/nolanlawson/local-npm#how-it-works. This is just one example.
What markdown library is used in the deps? Perhaps a bit of config can fix its output.

The markdown library is this one; I admit I don't recall exactly which one it is. Pull requests to improve it would be accepted. :)

fixed in #125


robotraconteur: 1.2.2-1 in 'rolling/distribution.yaml' [bloom]

Increasing version of package(s) in repository robotraconteur to 1.2.2-1:

upstream repository: https://github.com/robotraconteur/robotraconteur.git
release repository: https://github.com/robotraconteur-packaging/robotraconteur-ros2-release.git
distro file: rolling/distribution.yaml
bloom version: 0.12.0
previous version for package: null

The release repository of this package is not part of ros2-gbp, please consider moving it to this org. Here you have some instructions on how to create a ros2-gbp release repository for a ROS 2 package.
Quoting from the how-to:
"It is now strongly recommended for release repositories to live in this dedicated GitHub organization."

I just realized all the other distros release from the same from the same non ros2-gbp repo. Any chance we could start transitioning to ros2-gbp?

See https://github.com/ros2-gbp/ros2-gbp-github-org/issues/582


Add uptime to the relay monitoring schema

It would be nice to have a standard way to measure a relay's uptime. Currently we have the /eth/v1/builder/status endpoint and this is inadequate as there is no way to determine if a relay is creating artificial uptime by returning a static page.
Relay monitor has this endpoint /monitor/v1/faults.
Would be nice to either:
a) rename to from /monitor/v1/faults to /monitor/v1/stats and include { faults: {...} } as a part of the payload
b) add a new monitor/v1/stats that includes uptime stat along with other relevant metrics

uptime would be neat to see
my only concern is getting a super precise signal but if we are ok w/ some lossy-ness then I think the relay monitor could support this
do you have anything particular in mind?
I would think to start w/ a simple poll of /eth/v1/builder/status although you raise a good point about static or cached assets making this endpoint a little less meaningful
I don't really want to maintain a set of routines per relay to do some arbitrary liveness check though...
another thought I had was to encourage an ecosystem norm that relays expose metrics although im not sure how to do this in a DoS-resistant way; another option is to encourage a norm that relays just expose a liveness check for this purpose with reputation backing the claim that it is a reliable signal and not cached etc in some way
what do you think?


Add runtime Expression node

#2115
Add runtime Expression node
This PR introduces an Expression node that evaluates mathematical expressions during graph runtime. This is an initial implementation that currently only supports standalone expressions. Support for graph inputs is blocked pending variadic inputs implementation.
Changes:

Implement basic ExpressionNode that evaluates math expressions using math_parser
Register the Expression node in the node registry
Add DocumentNode Definition for Expression node
Add type system support in network_interface for Expression nodes

Current limitations:

Can only evaluate expressions without inputs from the graph
Will be expanded to support graph inputs once variadic inputs are implemented

Testing:

Unit test added for basic expression evaluation

Please advise correction and improvement.

Thanks for your work on this so far.
The goal of the expression node is to allow the node's output to be used within the graph (e.g. to set the width of the rectangle) as seen below. If you have any questions regarding this, feel free to ask in the discord chat.
Thanks again for this contribution.

Thanks for the feedback @0HyperCube . I jumped in a bit too quickly without fully understanding the context. Apologies for the confusion! Could you let me know which Discord channel is best to ask questions so I can clarify what I should be doing next? Thanks again for your patience and help.

@jacinyan No problem. You can ask in #task-help or #development.

!build

!build

!build

This works well, thanks for your work on this.
I changed the default of the input to be not exposed and set to "1 + 1", which should make it clearer about how the node works.

Hi @0HyperCube, thanks for reviewing and approving the changes! I've read GitHub's documentation about workflows approval for first-time contributors, and I believe the remaining steps are on the maintainers' side. Just wanted to confirm if my understanding is correct?

!build

Ok, with some further changes, this finally meets my original goals for the Math node.

Deferred for later:

Making this work with DVec2
Deleting all the other math nodes (with an upgrade script on document import to convert into the equivalent Math nodes), blocked by DVec2 support
Visual error handling when an expression is invalid
Fixing some issues in the expression calculator library for things like abs() and other math functions


Use different fast-forward

** What **
Using a different fast forward
** Why **
Because the other one doesn't work

/fast-forward


FileNotFoundError when running in subdirectory

To reproduce:
$ mkdir -p project/subdir
$ cd project
$ pipenv --three   # Init project in "project"
$ cd subdir
$ pipenv run python

Error message:
Traceback (most recent call last):
  File "bin/pipenv", line 11, in <module>
    sys.exit(cli())
  File "venvs/pipenv/lib/python3.6/site-packages/pipenv/vendor/click/core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "venvs/pipenv/lib/python3.6/site-packages/pipenv/vendor/click/core.py", line 697, in main
    rv = self.invoke(ctx)
  File "venvs/pipenv/lib/python3.6/site-packages/pipenv/vendor/click/core.py", line 1066, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "venvs/pipenv/lib/python3.6/site-packages/pipenv/vendor/click/core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "venvs/pipenv/lib/python3.6/site-packages/pipenv/vendor/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "venvs/pipenv/lib/python3.6/site-packages/pipenv/cli.py", line 2036, in run
    ensure_project(three=three, python=python, validate=False)
  File "venvs/pipenv/lib/python3.6/site-packages/pipenv/cli.py", line 594, in ensure_project
    ensure_pipfile(validate=validate)
  File "venvs/pipenv/lib/python3.6/site-packages/pipenv/cli.py", line 298, in ensure_pipfile
    if project.pipfile_is_empty:
  File "venvs/pipenv/lib/python3.6/site-packages/pipenv/project.py", line 357, in pipfile_is_empty
    with open('Pipfile', 'r') as f:
FileNotFoundError: [Errno 2] No such file or directory: 'Pipfile'

$ pipenv --version
pipenv, version 8.0.1

Thanks for opening this issue @uranusjr. This appears to be a bug in pipfile_is_empty that uses our PIPENV_MAX_DEPTH variable to check if a Pipfile exists, but then unconditionally assumes the Pipfile is in the current directory when it finds one. I've pushed f31a29f which will now open the same file it found while searching to perform the read.
This should be available in the next release.


fix: 3980 - refresh of the ordered nutrients

What

There was a typo in off-dart about mono and poly unsaturated fats, that prevented us from using them. That was fixed in 2.5.1
The current PR upgrades to 2.5.1, and also changes the ordered nutrient cache management - now we refresh frequently the ordered nutrient cache (we used to download only once).

Screenshot

before
after

Fixes bug(s)

Fixes: #3980

Impacted files

ordered_nutrients_cache.dart: now we cache the downloaded data in static - it used to be in database but then was never refreshed
pubspec.lock: wtf
pubspec.yaml: upgraded openfoodfacts to 2.5.1

Codecov Report

Merging #3997 (401634b) into develop (9d17997) will decrease coverage by 0.03%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           develop    #3997      +/-   ##
===========================================
- Coverage    10.97%   10.95%   -0.03%
===========================================
  Files          265      266       +1
  Lines        13071    13103      +32
===========================================
  Hits          1435     1435
- Misses       11636    11668      +32

Impacted Files
Coverage Δ

...app/lib/pages/product/ordered_nutrients_cache.dart
0.00% <0.00%> (ø)

... and 7 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Closes #1: Enable Postgres based backend

Fixes #1
Proposed Changes

Enables Postgres based backend

Ignore my previous comment, after the demo, it is clear to me now.


Releases are deleted when mirror repository is synchronized

Description
By some reason the created releases are deleted when mirror repository is synchronized even when the tags have not been recreated.
Steps

Sign-in to Gitea
Click "+"
Click "New Migration"
Click "GitHub"
Put https://github.com/tailix/libkernaux in the field "Migrate / Clone From URL"
Check "This repository will be a mirror"
Click the button "Create repository"
Click "7 tags" in the bar (the number may be different)
Click "New release" on tag "v0.6.1"
Put foobar in the field "Title"
Click "Publish release"
Click on the tab "Settings"
Click the button "Synchronize Now"
Wait a few (2-5) minutes
Refresh the page

Expected
You should see the label "Releases [1]" (the number as a badge) among the tabs.
Got
You see the label "Releases" (without a number) among the tabs.

Gitea Version
1.17.3 (self-hosted), 1.19.0+dev-166-g0a85537c7 (try.gitea.io)
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots
No response
Git Version
2.37.2
Operating System
Ubuntu 22.10
How are you running Gitea?
https://try.gitea.io & self-hosted
Database
PostgreSQL

Hm, maybe we should not allow to create release on a mirror repository?

Why not keep them on sync?

When setting up a migration from github and selecting the mirroring option, this disables the ability to sync labels, issues, pull requests, releases, milestones. Why would those be touched anyways during a sync when they're not configurable?
I just setup a script to do the mirroring of releases and their assets on a whole bunch of repos only to find everything gone the next day. The attachments still exist in the attachments directory but are no longer associated with any release.

Also interested in addressing this problem which arises when you use the new CI features in Gitea on a mirrored repository, with CI publishing local releases.

Any pointers where to start fixing this in the code?

My point is to prevent users to create releases on a mirror repository currently.
The second method is to not delete releases when syncing tags.

Please don't prevent release creation on a mirror -- this is how we test/debug our CI processes.

Please don't prevent release creation on a mirror -- this is how we test/debug our CI processes.

OK. Then you would like the second option. But how should Gitea handle when remote tags are deleted from your view point?

Do you mean local tags?

Do you mean local tags?

I mean as a mirror repository, if the source repository in the remote deleted some tags, when syncing the mirror repository, whether Gitea need to delete the release after the tags deleted?

I would suggest to handle it the same as GitHub does. I think if you delete a tag for a release that release gets deleted or converted to draft (not 100% sure). Deleting the release in all cases is undesirable for me.
In my mirror structure I want to manually (with the API) re-create the releases and mirror the binaries too.

Do you mean local tags?

I mean as a mirror repository, if the source repository in the remote deleted some tags, when syncing the mirror repository, whether Gitea need to delete the release after the tags deleted?

Not sure I care much about tags being deleted in the source repo since I'd never do that. I thought this problem was about this:

Release created in mirror repo -- a release has a 1:1 relationship with a tag, implying that a tag must be created in the mirror in order to create a release there.
Now sync from the source repo.
The mirror is supposed to be read-only, but in step 1 we create a tag locally, hence it is not read-only.
What to do about that? Delete the locally created tags, so the repo matches the source? But that means deleting the release.

Yes, that's why I think we need to disable creating releases in a mirror repository.

Ok, we're on same page then.
For my purposes, I don't need tags associated with locally-created releases to be preserved in the mirror.
I would like to have releases created in the mirror be preserved.
If, as @mrexodia suggests above it is already the case that a release on a non-mirror repo will survive if its associated tag is deleted, then I'd like that same behavior on a mirror.

Yes, that's why I think we need to disable creating releases in a mirror repository.

In my case this forces me to use a non-mirror repository and manually mirror + recreate releases. My use case is hosting a mirror with release binaries in another region. The GitHub repository I mirror from is the main development place. Currently I’m forced to use a non-mirror already so it’s fine, but in general disabling releases altogether prevents my use case of a mirror completely…

Because #20311 will support mirror releases from upstream. So I think release creation on mirror repositories must be disabled. Otherwise, it will result in possible conflicts.

I think there are two separate use cases for mirrored repositories:

Migrating a project from GitHub etc.
Keeping a local mirror of code dependencies (does not protect from force-pushed deletions ofc).

Mine is the latter. In addition, I'm building my own signed releases from those mirrors. Because of this bug, I have to manually sync a fork instead of having automatic releases.
Two proposals regarding https://github.com/go-gitea/gitea/pull/20311:

When release syncing is available, allow to deselect it to be able to have local releases. When activating it later, warn that all local releases will be deleted if there are any.
Or introduce another repo/migration type that specifically addresses (2) (code mirroring only) and allow local releases there without being able to mirror remote releases/issues etc.

Not sure about the difficulty of those proposals, just throwing it out there. I really appreciate your hard work on Gitea!

This is clearly a bug.
If i mirror a repository it disables all the other options (such as releases) so i understand that it ignores them.
I needed to attach new releases in a public read-only organization and every time it syncs any release uploaded on the mirrored repo will be deleted.

Or maybe we can convert a release as draft automatically if the related tag has been removed when mirroring.

I am not sure my issues is the same since most people seem to be talking about mirroring from github initially and/or creating the releases on the gitea side.
My use case is a private gitea repo that mirrors to github. The commits sync fine to the remote github repo. I want to take advantage of the Actions on Github so I create the release there - manually for now. Everything works fine until I push another commit to my private gitea instance which then syncs to Github. At this point the releases disappear because they've been converted to drafts. Is this the same issue everyone is having?
Is it happening because the gitea side does not have the tag I have to create on the github side? How should I proceed if I want to keep the same workflow?

Any results? This bug has been around for a year now. I want to use Gitea to backup Github data, including binaries from releases, but this bug makes it impossible to backup releases at all.

This bug was introduced by #19125.
Ref: https://github.com/go-gitea/gitea/pull/19125/files#diff-2aa04998a791c30e5a02b49a97c07fcd93d50e8b31640ce2ddb1afeebf605d02R481
I think we need a method to not delete all previous releases but also keep better performance.

Also, although the Release list is lost, the uploaded attachment files are not deleted. And every time you upload again, a copy of the attachment will be saved again, and no de-duplication will be done. This had caused me to store 500GB+ of invalid data on my disk. Is there any way I can delete this invalid data now?

This bug was introduced by #19125. Ref: https://github.com/go-gitea/gitea/pull/19125/files#diff-2aa04998a791c30e5a02b49a97c07fcd93d50e8b31640ce2ddb1afeebf605d02R481
I think we need a method to not delete all previous releases but also keep better performance.
The bottleneck is upset is very slow and the batch insert is not. Or maybe we can load all tag names into memory first.

I'm not familiar with Gitea code. I've looked over the logic of this code and maybe changed this here to if _, err := db.DeleteByBean(ctx, &repo_model.Release{RepoID: repo.ID, IsTag: true}); err ! = nil { and that fixes the bug? I think the Tag data is just expected to be deleted here, right?

You can GC them with ./gitea doctor storages commands.
Yes, I think you are right. That's why I added this to 1.21.5.

I sent #28817. It will not remove mirror releases when mirroring.


event requests - report 393 Suggest Vendor Payments - OnBeforeIncludeVendor() and OnBeforeClearNegative()

We would like to provide an option to include vendors with credit balances in the payment suggestion. Therefore, can we please:

(1) replace the various bits of code that do something like this:
                CalcFields("Balance (LCY)");
                VendorBalance := "Balance (LCY)";

                if VendorBalance > 0 then begin
                    GetVendLedgEntries(true, false);
                    GetVendLedgEntries(false, false);
                    CheckAmounts(false);
                    ClearNegative;
                end;

specifically, in each, replace
                if VendorBalance > 0 then begin

with
                if IncludeVendor(Vendor, VendorBalance) then begin

where this new function would look like
local procedure IncludeVendor(Vendor: Record Vendor; VendorBalance: Decimal) Result: Boolean
var
    IsHandled: Boolean;
begin
    OnBeforeIncludeVendor(Vendor, VendorBalance, IsHandled, Result);

    if not IsHandled then
        exit(VendorBalance > 0);
end;

calling an event like
[IntegrationEvent(false, false)]
local procedure OnBeforeIncludeVendor(Vendor: Record Vendor; VendorBalance: Decimal; var IsHandled: Boolean; var Result: Boolean)

(2) This alone is not enough; we then also need to suppress ClearNegative() so that as well as not ignoring Vendors with credit Balance (LCY), we don't then regardless delete their temp entries per currency in credit. So, add here:
    local procedure ClearNegative()
    var
        TempCurrency: Record Currency temporary;
        PayableVendLedgEntry2: Record "Payable Vendor Ledger Entry" temporary;
        CurrencyBalance: Decimal;
        IsHandled: Boolean;
    begin
        OnBeforeClearNegative(IsHandled);

        if IsHandled then
            exit;

        [...]

where that new event will be
[IntegrationEvent(false, false)]
local procedure OnBeforeClearNegative(var IsHandled: Boolean)

Thanks!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


BDI: Record type works even when unmapped

Critical Changes
Changes
--Donation Record Type works in all orgs, regardless of help text mapping.
Issues Closed
New Metadata
Deleted Metadata

**lurch: attach W-031346

Tracking W-031346

Is the help text mapping a user-facing, documented feature? Is there ever a use case where a User would want to configure this field to map somewhere other than to Opportunity.RecordTypeId? With this change, we override whatever value is in the help text for this field on the Data Import object, so we might either want to check the help text value first to see if there is a User-configured alternate mapping, or document the fact that this Data Import field is now mapped directly to Opportunity.RecordTypeId regardless of whatever value is/isn't in the help text for this field.

@kenslewis  et al!
summarizing discussion from 2/27:
We are leaving this refactor as-is. BDI does not respect any custom mapping an admin may have included in the DonationRecordTypeName field. It is unclear why the help text was added at all. When BDI loops through mappings to update the opp, this field is explicitly ignored: https://github.com/SalesforceFoundation/Cumulus/blob/910ea10dcec4c10da4613cad7038b4d8e0734015/src/classes/BDI_Donations.cls#L440 and when writing Opportunity record type, it is used without regard for its mapping: https://github.com/SalesforceFoundation/Cumulus/blob/910ea10dcec4c10da4613cad7038b4d8e0734015/src/classes/BDI_Donations.cls#L380


question: can I change profiles from bash?

How do I switch between profiles from the command line?
I have some game binds that are fairly hazardous on the desktop, so it would be nice if i could script the switching of profiles.

My bad.. didn't see it until I looked a little deeper.
https://github.com/libratbag/libratbag/wiki/Usage
$ ratbagctl "$(ratbagctl list|sed -E 's/.:\ (.)\ /\1/')" profile active set 1
$ ratbagctl "$(ratbagctl list|sed -E 's/.:\ (.)\ /\1/')" profile active set 0

Couple of notes:

$ ratbagctl "$(ratbagctl list|sed -E 's/.:\ (.)\ /\1/')" profile active set 1

Better use:
$ ratbagctl G9x profile active set 1

(replace G9x with any part of your device that is unique enough to not be confused with anyone else shown by ratbagctl list)
And second, we do not guarantee a stable output/parameters from ratbagctl, so do not get angry if we change something in the future (we already did that once)

Nice!
No problem. Change is to be expected. Bash scripts are how I amuse myself.
For a while there a few months ago I was changing my .evrouterrc between 'Logitech Unifying Device' and'Logitech Performance MX'.
After about the third time I just scripted a permanent auto-select.


Convert readthedocs to use material theme

Close #22
Currently the code blocks with the line numbers look awkward, the line numbers column is so wide. Should we keep them?

Signed-off-by: Daniel Lim Wee Soong<EMAIL_ADDRESS>

We should figure out what needs to be done to fix the line numbers formatting.

@mithro I think we will need to modify the theme itself. I believe the theme we are using now is https://github.com/myyasuda/sphinx_materialdesign_theme. The last time that it was updated was 2 years ago. Shall we make a fork of it?

We already have a fork at https://github.com/SymbiFlow/sphinx_materialdesign_theme

@mithro

@daniellimws Did we fix the line number issue?

Oh right I forgot about the line numbers.

Fixed the formatting in https://github.com/SymbiFlow/sphinx_materialdesign_theme/pull/10


DISPLAY and DBUS enviroment variables in any terminal session

This script allows to have the DISPLAY and DBUS* environment variables availables in any session even if the session is through SSH. This pull request can be useful to fix #GPII-1838.

Hey @amatas, sorry for the delay but I haven't had the chance to follow up with @klown to confirm this resolves https://issues.gpii.net/browse/GPII-1838 Hoping to do that soon!

@amatas @avtar
I tried a version of this, but with strange results.
What I did was start with the current vagrant vm as provided by the GPII linux repository, but add @amatas' proposed desktop and script files to it (/etc/xdg/autostart/save-env-vars.desktop,  /usr/local/bin/save-env-vars.sh, and /etc/profile.d/load-vars.sh).  When the machine was rebooted, I got the unhappy GNOME dialog:
"Oh no! Something has gone wrong.  A problem has occurred and the system can't recover.  Please log out and try again".
Poking around using various vagrant ssh -c ...  commands revealed that there is a .vars file in vagrant's home directory, and its contents are the DBUS_SESSION_BUS_ADDRESS and DISPLAY variables.  So:  the autostart invocation of save-env-vars.sh is working fine.  The culprit is the load-var.sh in /etc/profile.d.
If I remove the load-var.sh file by doing vagrant ssh -d 'sudo rm /etc/profile.d/load-vars.sh', and then reboot, the machine comes up with no problems.  However, in that case the profile isn't properly set, since load-vars.sh isn't run (obviously).
Is there a branch of  GPII linux repository that provides a vagrant machine with @amatas' changes that I could try?

Hi @klown, I've just spinned up a VM using a clone of GPII Linux master repository and then I run the script/user-envs.sh as root with sudo. Finally, I did a vagrant reload to reboot the VM and it seemed that everything went well.
I check that the environment variables were exported using vagrant ssh -c "env | grep DBUS"

Hi @amatas, that's great, thanks!

I've just spinned up a VM using a clone of GPII Linux master repository ...

Do you have a branch with the specs for this VM somewhere in your github so I can try it?  Or, is @avtar reviewing this pull request, and it will be merged into master soon?

@amatas @avtar,
Confirming that this patch works with the checkOrcaStart.sh test script.
I've applied the patch, created a vagrant box with it, and started it up with none of the problems I noted above.
Next:  I'm going to apply my patches for GPII-442, do a grunt unit-tests and see if the one test that dependson the DBUS environment variable now works.  I'll let you know the outcome.

@amatas @avtar,
I wrote:

Next: I'm going to apply my patches for GPII-442, do a grunt unit-tests and see if the one test that depends on the DBUS environment variable now works. I'll let you know the outcome.

All good.  I'll add a :+1: comment to the GPII-1838 JIRA.
Thanks @amatas! :beer:

@amatas
I found one small problem.  After vagrant halt, when the machine is next started using vagrant up, I get the unhappy GNOME screen.
However, if I do the following:

remove the .vars file from the vagrant home directory (e.g., vagrant ssh -c 'rm .vars')
vagrant halt
vagrant up

The machine starts up fine.  Is there any way to trigger removal of the .vars file when running vagrant halt?

@amatas, using the /tmp directory might resolve the issue that @klown identified. Otherwise the /home/vagrant/.bash_logout can provide another option.

@avtar @klown could you please check if the issue related to the Gnome crashes has gone with the latests changes?

Thanks for your changes, @amatas. I made some minor suggestions.

@amatas @avtar Is there anything you need from me with respect to this pull request?

@klown it would help if you could see whether issue you reported earlier still exists with the most recent changes.

@avtar @amatas I have successfully built the 'inclusive/fedora24' box based on this pull.  But, when I attempt to use it with the master branch of GPII linux, I get an authentication failure:

...
==> default: Booting VM...
==> default: Waiting for machine to boot. This may take a few minutes...
default: SSH address: <IP_ADDRESS>:2222
default: SSH username: vagrant
default: SSH auth method: private key
default: Warning: Remote connection disconnect. Retrying...
default: Warning: Remote connection disconnect. Retrying...
...
Timed out while waiting for the machine to boot. ...

The machine has booted.  Something is off with the ssh keys, and the "linux" directory is not shared with the guest VM. What am I missing?
On the plus side, there is a vars file in /tmp/vagrant-DBUS-VARS/  with the expected DBUS_SESSION_BUS_ADDRESS and DISPLAY variables.  Also, the checkOrcaStart.sh test program works using vagrant ssh -c, except I have to provide a password.

@klown have you updated all Fedora packages by any chance?

@gtirloni I know we discussed this in IRC, but, for the record:  I built the box based on this pull request, and am using it for the linux VM.  So, as you said in #fluid-work, it should have all the updates.  But, you noted: "@amatas is investigating an issue where the vbox guest additions broke with the latest updates in fedora 24".  Does that explain why SSH authorization isn't working?

@avtar @amatas I have tested the latest changes, and I no longer see the problem reported above.  Also, this solves the problem reported in GPII-1838.

LGTM

LGTM


simplify http fliter check

There is no need to check http url when http header check result is false.

Codecov Report

Merging #178 (6363c55) into main (7b3f92d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #178   +/-   ##
=======================================
  Coverage   81.20%   81.20%
=======================================
  Files          44       44
  Lines        4846     4846
=======================================
  Hits         3935     3935
  Misses        694      694
  Partials      217      217

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 bdd7b96...6363c55. Read the comment docs.

Thanks for the PR.
All compilers do a short circuit evaluation, so the change hasn't simplified the check.
But as @benja-wu mentioned, it does make code easy to read, so we are glad to accept it.
I notice the commit message is in Chinese, to make it align with all other commits, could you please amend the commit and revise the message to English?


Fix maven-assembly-plugin setting in pinot-plugins to match apache release id

Make the execution id for maven-assembly-plugin in pinot-plugins to match the id in apache parent pom so that it can be correctly overridden.

@rfscholte @gortiz Please take a look and see if this is correct

cc @jadami10

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

Project coverage is 0.00%. Comparing base (59551e4) to head (e3560eb).
Report is 1281 commits behind head on master.

:exclamation:  There is a different number of reports uploaded between BASE (59551e4) and HEAD (e3560eb). Click for more details.
HEAD has 53 uploads less than BASE

Flag
BASE (59551e4)
HEAD (e3560eb)

integration
7
1

integration2
3
1

temurin
12
1

java-21
7
1

skip-bytebuffers-true
3
0

skip-bytebuffers-false
7
1

unittests
5
0

unittests1
2
0

java-11
5
0

unittests2
3
0

integration1
2
0

custom-integration1
2
0

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #14379       +/-   ##
=============================================
- Coverage     61.75%    0.00%   -61.76%
=============================================
  Files          2436        3     -2433
  Lines        133233        6   -133227
  Branches      20636        0    -20636
=============================================
- Hits          82274        0    -82274
+ Misses        44911        6    -44905
+ Partials       6048        0     -6048

Flag
Coverage Δ

custom-integration1
?

integration
0.00% <ø> (-0.01%)
:arrow_down:

integration1
?

integration2
0.00% <ø> (ø)

java-11
?

java-21
0.00% <ø> (-61.63%)
:arrow_down:

skip-bytebuffers-false
0.00% <ø> (-61.75%)
:arrow_down:

skip-bytebuffers-true
?

temurin
0.00% <ø> (-61.76%)
:arrow_down:

unittests
?

unittests1
?

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

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

I wouldn't do this, because both assemblies have a different purpose: the make-assembly should always be executed for plugins as it behaves like some packaging-plugin, creating the main deliverable. The source-release-assembly is only executed during release to generated the sources-jar (jar containing all the sourcefiles as required by deployments to maven central).
I do the see issue you're trying to solve. The -Papache-release triggers the maven-assembly-plugin (and with it all its execution-blocks), however you don't want it to be executed on pinot-plugins itself nor at the assembly-descriptor. So there are two options: only enable it on plugins or always enable it, but explicitly disabled on non-plugins. The original request was to activate it as easy as possible for plugins (like https://github.com/apache/pinot/blob/master/pinot-plugins/pinot-metrics/pinot-dropwizard/pom.xml#L40-L43), but we missed something.
Maybe the simplest solution is to move the assembly-plugin in the pinot-plugins to a separate profile, like this:
	<profile>
		<id>plugin-assembly</id>
		<activation>
			<file>
				<exists>src/main/resources/pinot-plugin.properties</exists>
			</file>
		</activation>
		<build>
			<plugins>
				<plugin>
					<groupId>org.apache.maven.plugins</groupId>
					<artifactId>maven-assembly-plugin</artifactId>
					<dependencies>
						<dependency>
							<groupId>org.apache.pinot</groupId>
							<artifactId>assembly-descriptor</artifactId>
							<version>${project.version}</version>
						</dependency>
					</dependencies>
					<executions>
						<execution>
							<id>make-assembly</id>
							<phase>package</phase>
							<goals>
								<goal>single</goal>
							</goals>
							<configuration>
								<descriptorRefs>
									<descriptorRef>pinot-plugin</descriptorRef>
								</descriptorRefs>
								<archiverConfig>
									<compress>${archiver.compress}</compress>
								</archiverConfig>
								<recompressZippedFiles>
									${archiver.recompressZippedFiles}</recompressZippedFiles>
							</configuration>
						</execution>
					</executions>
				</plugin>
			</plugins>
		</build>
	</profile>

@rfscholte Thanks for the suggestion! Made the change accordingly.
Do you see other changes required? IIUC, this PR will change the behavior to not run the assembly plugin by default. Do you see side effect of that?

@rfscholte Thanks for the suggestion! Made the change accordingly. Do you see other changes required? IIUC, this PR will change the behavior to not run the assembly plugin by default. Do you see side effect of that?

If it now works with both with or without -Papache-release I would expect it to be fine. The only downside I can think of, is that it is less clear how/why the zip is created, i.e. the existence of pinot-plugin.properties as defined in the parent.
For third parties that want to build a pinot-plugin, I would suggest to simply use the complete maven-assembly-plugin in their build.

I like this approach. Anyway, we are not using the new packaging right now, so this should be safe anything that disables that packaging right now. We are planning to work on this in the following months and part of the work to be done is to test a release using the new plugging packaging.


Error Android cannot read property launch of undefined

I have done everything indicated in the tutorial and at the time of opening the external app, indicates error in the property launch of undefined, version of platform android 6.1.2, version of cordova 6.3.1

i have the same Problem !

+1 here too

+1 here too

any update on this. still i am getting this error

I had the same problem but i removed all unnecessary plugins and files about this plugin. Then i run cordova plugin add cordova-plugin-app-launcher and now it's working fine.


Could you please add a feature to include websites?

Hello! I'd like to extract a specific topic from a specific link. These can be done differently, but not together. Would it be possible to do that?

hi!
Hello, I also faced the same issue.
And finally, I found out that entering the search query like this:
"site:https://www.cnn.com OR site:https://www.bloomberg.com keyword" works.


feat: modify runc install to accomodate configurable private overrides

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2292

Relevant Lines:
2887

💛  - Coveralls

taken care of with recent work / PRs


Active space approaches

This PR implements the CCSDt' method, where the T3 amplitudes have all indices in the active space. Hopefully CCSDt with split spaces will be possible down the line.
To do:

[ ] Optimisation for CCSDt'
[ ] UCCSDt'
[ ] Fix conversion routines
[ ] Add examples

Codecov Report
Patch coverage: 95.35% and project coverage change: -1.19 :warning:

Comparison is base (7bab745) 99.48% compared to head (27a9511) 98.30%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #19      +/-   ##
==========================================
- Coverage   99.48%   98.30%   -1.19%
==========================================
  Files          51       53       +2
  Lines       76248    77773    +1525
  Branches      709      708       -1
==========================================
+ Hits        75858    76455     +597
- Misses        266     1202     +936
+ Partials      124      116       -8

Impacted Files
Coverage Δ

ebcc/codegen/RCCSDtp.py
0.00% <ø> (ø)

ebcc/util.py
72.05% <14.28%> (+0.15%)
:arrow_up:

ebcc/dump.py
87.12% <72.72%> (-2.35%)
:arrow_down:

ebcc/ansatz.py
80.76% <79.31%> (+16.25%)
:arrow_up:

ebcc/rebcc.py
87.40% <86.82%> (+1.11%)
:arrow_up:

ebcc/space.py
92.78% <89.55%> (+3.69%)
:arrow_up:

ebcc/gebcc.py
94.64% <94.17%> (+3.76%)
:arrow_up:

ebcc/uebcc.py
97.05% <99.46%> (+1.90%)
:arrow_up:

ebcc/brueckner.py
90.58% <100.00%> (+0.16%)
:arrow_up:

ebcc/codegen/GCCSDtp.py
100.00% <100.00%> (ø)

... and 4 more

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


SQLite only supports TYPE_FORWARD_ONLY cursors

I have the error SQLite only supports TYPE_FORWARD_ONLY cursors when using sqlite-jdbc-3.23.1 loaded in libreoffice base <IP_ADDRESS> with oracle JRE 1.8.0_181
See also https://bitbucket.org/xerial/sqlite-jdbc/issues/120/javasqlsqlexception-sqlite-only-supports

same issue.... any progress on this?

This is not a bug in the driver.  JDBC drivers are permitted to support forward-only ResultSets, in part because that's the only kind that some database engines themselves support.  And SQLite is one such engine.

This is not a bug in the driver. JDBC drivers are permitted to support only forward-only ResultSets, in part because that's the only kind that some database engines themselves support. And SQLite is one such engine.

If it's not a bug, then what's going on here?  It's odd that the user is able to setup up a JDBC driver connection to assist with data manipulation only to be thrown this error message.  What's the point of allowing a data connection to be created, but not able to view or manipulate the data because the 'cursor' isn't supported.  If this isn't possible, then the data connections shouldn't be allowed.

If it's not a bug, then what's going on here? It's odd that the user is able to setup up a JDBC driver connection to assist with data manipulation only to be thrown this error message.

It's not a bug in the driver, but possibly it can be characterized as a bug in Base.  I think that's a bit harsh, though.  It could possibly be characterized as a user error to attempt to use a JDBC driver that does not satisfy Base's minimum requirements.
In any event, Base has requirements of JDBC drivers that this driver does not satisfy (and is within its rights not to satisfy).  It could conceivably check those requirements when the user first sets up a connection, but that does not rule out the possibility that something will change later.  When Base detects a failure, it provides sufficient information to characterize the issue.

What's the point of allowing a data connection to be created, but not able to view or manipulate the data because the 'cursor' isn't supported. If this isn't possible, then the data connections shouldn't be allowed.

Forward-only result sets -- the kind this driver provides -- are perfectly suitable for many data-manipulation tasks.  For substantially all tasks that SQLite is successfully used for, in fact, because forward-only access is the only mode that SQLite's C-language API and the underlying engine support.  It's not immediately clear to me that Base really requires more, vs. it asking for a feature that in practice it could do without.
Personally, I worked around this issue by installing an ODBC driver for SQLite, and setting up ODBC connections to my SQLite databases.  Base uses these successfully.  I am uncertain whether the ODBC driver does some kind of internal magic to provide for scrollability or whether Base just works with forward-only style in this configuration.  I would have preferred JDBC, but that's not a viable option, at least not with this driver, and again, that does not make this driver buggy.

See https://github.com/xerial/sqlite-jdbc/issues/682#issuecomment-950687433
This is not possible in SQLite.


drivers/disk: sdmmc: stm32: DMA header requested on F4 serie

Even if not used DMA HAL is required in F4 SD HAL driver. Just
as for L4/F7 series that have been added before.
Add it for this specific serie.
Signed-off-by: John Kjellberg<EMAIL_ADDRESS>

See Zephyr's contribution guidelines to fix your CI failure-one of your commits is missing body text. Other than that issue, this PR LGTM


Contact

Hey, nice tool ! Where can I contact you private ? Thanks !

Hi,
You can contact me through<EMAIL_ADDRESS>Thank you!


Use Bios targets in FindImports

Do not return any module that doesn't belong to the targets set.
If the targets set is empty, then lift this restriction.
The file existence check is probably redundant but keeping it around for now.
Fixes #733

Using the targets of the HscEnvEq doesn't work because they don't span across the whole component set. I'll rebase on top of #688 and use KnownFiles instead.

https://github.com/wz1000/ghcide/pull/10


Clinking Link opens pdf in browser, how to save as file instead?

I am using Puppeteer v11.0.0 with Chromium
I want this to work headed and headless.
I click a button that opens a PDF file in the browser.
After the PDF opens, in the DOM I see a tag like this
I'd like to intercept the request and just have the PDF save to my local disk without even opening in the browser if possible?
Thanks

I'm not sure this is a bug. The issue is puppeteer page.pdf generates links that assume puppeteer is running. I'd like to generate pdfs from html that use relative links that open new pdf files stored on local file system. This is something pdf supports. See https://stackoverflow.com/questions/17800596/relative-file-links-in-pdf-files
Is puppeteer source public? I'd like to modify the code myself just for my local env.

I aggree with @prmichaelsen it does not look as a bug to me because it's what the browser renders.


Update README.md

simpleion - simple
In order to - To
pyenv - piano
deployment, - deployment

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I'm assuming this was an automated script that went haywire. Closing.


JOSS Review: pytest-cov needed to run tests with default pyproject.toml options

This Issue has been raised as a part of the PyTOUGHREACT JOSS Review at: https://github.com/openjournals/joss-reviews/issues/6486
When I first tried to run pytest, I got the following error:

ERROR: usage: main.py [options] [file_or_dir] [file_or_dir] [...]
main.py: error: unrecognized arguments: --cov=pytoughreact
inifile: PyTOUGHREACT\pyproject.toml
rootdir: PyTOUGHREACT

After digging a little, I realized I needed pytest-cov for the --cov=pytoughreact option to be valid.
Since it is needed to generate the coverage report and run the tests without modifying the pyproject.toml first, I'd suggest adding pytest-cov installation to the notes in the following docs section: https://pytoughreact.readthedocs.io/en/latest/developer.html#test

Hi @blakeaw . Requirements for pytest-cov have been added to the documentation


TabBar - moving Tabs styles

HTML/CSS Post MVP (https://github.com/RaspberryPiFoundation/editor-ui/issues/493)
How does this differ from the current Tab implementation?
design

Also review existing tab styles

closed with merge


AMM Arbitrage - Could not find the exchange rate using rate_oracle_enabled FALSE

Describe the bug
Failed to start Uniswap Optimism strategy with rate_oracle_enabled False. OETH rates are not able at any available reliable rate oracle source (coingecko with extra-tokens are immediately exceed rate limits)
Steps To Reproduce

Clone and install latest development Client + gateway
connect uniswap optimism
Create/start amm_arb startegy using uniswap optimism

Actual:
Bot flooded with couldn't find exchange rate for OETH-USDC
Expected:
User able to use uniswap_ethereum_optimism
logs_conf_amm_arb_2.log
gwissueopt.zip
Screenshots

Release version
dev-1.18.0 /staging

Did you ever figure this out?

I was searching for the option to disable rate oracle since I was getting spammed by rate oracle errors as well, but I couldn't find any documentation.
Check here: https://hummingbot.org/strategy-configs/rate-oracle/ no mention.
Also, CLI does not hint about this option.


Typo for rqt package name

this didn't work in ubuntu 18 with dashing...which is why I raised the pull request.


Add tests for large-ish command-line analyzer classes

These classes need to have tests run by the bots.
com.google.dart.command.analyze.AnalyzerImpl
com.google.dart.command.analyze.AnalyzerMain
com.google.dart.command.analyze.AnalyzerOptions

I believe this was referring to the Java implementation of analyzer, which is no more, so I'm closing this as stale.


Account for else clauses in if return simplification

The example in fn4() was triggering a false positive in some code of mine.

Thank you.


Healthcheck: QEMU

Implement in vm service monitor action
Inspects the qemu log files of running VMs and reports if there was any errors.
See https://github.com/0-complexity/selfhealing/blob/master/jumpscripts/healthchecks/qemulogs_check.py

logs are parsed from  /var/log/libvirt/qemu/.log , to check for the word error
line with error will be added to healthcheck vailable at
http://{{url}}:{{port}}/health/nodes/{{ nodeid }} under vm-logs part

i tried 4 scenarios to corrupt the vms:

kill qemu proccess
kill nbdserver
remove vm server.socket file
put "error" word in /var/log/libvirt/qemu/{{vmname}}.log file

and after each case the health check was ok

software versions
master : 96700d6f

Steps

Create etcd cluster and storage in node 1
Create vm in node 2
kill node 1

Issue
check qemu health check  still ok
Software Version
0d238efa01e6f8ee291f2d3cec4d1e918aa39212

for Testing

create a node with storagecluster and etcd
create another node with vm and disk on sed storage cluster
login to vm
force close node with storage cluster
try read/write operation such as touch file
check logs in http://{{url}}:{{port}}/health/nodes/{{ nodeid }} under vm-logs part

Software Version
337169e32a2e4d28fb06178c06c6d93d2e4c47c0


chore: remove feature flag and reduce id to 5characters

Description

removes feature flag on donwload all button
shortens id in the download name

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


Longer task names should not break search

When a task name is more than 255 characters long, it will break the search algorithm:

levenshtein argument exceeds maximum length of 255 characters

This limit is hard-coded in Postgres (for performance reasons), but search should not break because of it.
Either the task names are cut off at 255 characters when filtering, or a fallback algorithm should be used for these longer names. I think option 1 is fine to start with.
Another possibility is to limit task name length, but I don't think that's warranted only because of this limitation.

This is fixed (in the simplest way) in 5163b2b05666ae4173b6f595e67660a6c7865378.
More advanced search improvements can happen independently from this bug fix.


current_user became not accessible

I'm using simple-navigation gem and its configuration is stored in config/navigation.rb. Previously to 2.0 it was fine. Currently on 2.0 current_user is inaccessible. It looks like the loading order has been changed.
P.S. Rails 3.2.1

Could you be more specific? I have no idea how this can be issue or what we can do to fix it.


Question about pretrained model

Hi, Mr. Liu.
thanks for your concise code! As a beginner, I learned a lot from reading your code.
Now I got a question about pretrained model. I can not download pre-trained model from link "https://github.com/Lornatang/ESRGAN-PyTorch/releases/download/0.1.0/ESRGAN16_DF2K-a03a643d.pth" when I execute "python test_benchmark.py -a esrgan16 --pretrained --gpu 0 data" in the command line. I try to open the link in Chrome browser and also got "404 not found".
I think maybe something wrong with the pre-trained model link, or my way of getting model is not right.
I am looking forward to your reply. You can contact me<EMAIL_ADDRESS>Thanks again.

Link fixed.


Add the barcode field in line item of the order resource on Admin rest API

Overview/summary
Can you please include barcode field on line_item object of the order resource on Admin rest API. https://shopify.dev/api/admin-rest/2022-07/resources/order#resource-object
...
Motivation

What inspired this enhancement?
We store the products with barcode as the unique key (making a composite key) and it's easier to get the product if we have barcode on the line item object, otherwise we have to query the product by variant id and find the barcode.
...

Area

[ ] Add any relevant Area: <area> labels to this issue

Checklist

[x] I have described this enhancement in a way that is actionable (if possible)

Please add this. It's so simple, but so much more is possible with this. I don't need to make an extra request to the product variant to fetch the barcode anymore..


rainbow-kit / wagmi

PR moves us from our existing implementation w/ web3modal / custom web3context implementation to rainbow-kit + wagmi.
Remaining Todo:

[x] Update Remaining Tests (Only a few more test failures). Connection mocks are done.
[x] Resolve test failure for import<EMAIL_ADDRESS>. Need to ignore this import for tests.
[x] Cleanup remaining unused files
[x] Resolve merge conflicts w/ develop

Notes for Review:

logo added to public folder for gnosis support. Manifest must use svg not png. (Should be able to manually add an app on gnosis via walletlink). Some further testing is needed here, but at a minimum it's an incremental improvement over existing functionality (Can't add a custom app w/ existing implementation)
--transformIgnorePatterns \"node_modules/(?!@rainbow-me)/\ needed to be added for jest tests to run. Slows down tests by ~2 min on CI Open Issue with Rainbow Team
NETWORKS was removed from networkDetails. The only place this was used was on the wrap page, and it was refactored out.
useAccount, useConnect, useNetwork hooks from wagmi are used throughout the app to replace web3context.

Screenshoots:

Thinking about a couple things I want to check here:

[ ] any events for gtag needed?
[ ] check handling of cross-chain, unsupported chains, etc.

Here is unsupported network experience:

Clicking on 'Unsupported Network' opens wallet drawer for user to interact as before.
Clicking on Unsupported Network or alert circle icon opens the network switcher.


Unable to command any message

Im trying to command a message and I can not make it work. Event the parameter example that comes in the lib.
I do not get the ack.
Trying to arm the drone:
void armVehicle() {
    CommandLong armVehicle = CommandLong(
        param1: 1,
        param2: 0,
        param3: 0,
        param4: 0,
        param5: 0,
        param6: 0,
        param7: 0,
        command: 400,
        targetSystem: 1,
        targetComponent: 1,
        confirmation: 0,
      );
    MavlinkFrame frame = MavlinkFrame.v2(0, 255, 1, armVehicle);
    var _serialized_frame = frame.serialize();

    _socket!.send(_serialized_frame, _socket!.address, _socket!.port);

Im also unable to send the heartbeat to the Px4
Heartbeat heartbeat = Heartbeat(
      customMode: 0,
      type: mavTypeGcs,
      autopilot: mavAutopilotInvalid,
      baseMode: 0,
      systemStatus: 0,
      mavlinkVersion: 0
    );

    MavlinkFrame frame = MavlinkFrame.v2(0, 255, 1, heartbeat);
    _socket!.send(frame.serialize(), InternetAddress.anyIPv4, 14540);

Hi,
The provided code snippet appears to work well for me on Ardupilot:
final command = CommandLong(
    command: mavCmdComponentArmDisarm,
    param1: 1,
    param2: 0,
    param3: 0,
    param4: 0,
    param5: 0,
    param6: 0,
    param7: 0,
    confirmation: 0,
    targetComponent: 0,
    targetSystem: 0
);
final frame = MavlinkFrame.v2(0, 0, 0, command);
_socket!.send(frame.serialize(), _test!.address, _test!.port);

Ensure that the socket is configured correctly after a successful RawDatagramSocket.bind call, and that _test is assigned the Datagram received when calling _socket.receive().
Feel free to reach out if you encounter any issues. If additional information about your application is needed, please provide more details, and I'll do my best to assist you. 😊


Problem using the browser of a mobile device

Hi @SaraVieira, using the browser of a mobile device I get this:

Maybe it's a project related problem: https://github.com/pmndrs/leva

Hey
Interesting because you are not supposed to get Leva in the mobile version :/
Can you tell me what your phone is? Or the screen size?
I get this one mine

@SaraVieira : I tried on two different devices a tablet or a mobile phone but I get the one shown in the picture.

Do you know the screen size of them?

@SaraVieira :  Resolution | 800x1280 pixels

I was only triggering at like 900, this should not show in any mobile device, so now after 1200 it disappears since it has no space


depth mask for mapper color loss

Hi @MohammadJohari ,
Thanks for your excellent work! I find that the depth mask is not used for the mapper color loss https://github.com/idiap/ESLAM/blob/main/src/Mapper.py#L362. Is it intended since I find the depth_mask for tracking color loss can make a lot of difference to the system? Thanks in advance.
Bests,
Weining

Hi @MohammadJohari ,
Thanks for your excellent work! I find that the depth mask is not used for the mapper color loss https://github.com/idiap/ESLAM/blob/main/src/Mapper.py#L362. Is it intended since I find the depth_mask for tracking color loss can make a lot of difference to the system? Thanks in advance.
Bests, Weining

I guess the RGB image is more accurate than depth image. So all sampled pixels in RGB images could be considered. But in depth image, some pixels are not valid with value of 0. It is highly relevant to SDF value, so depth mask is also implied on SDF loss.

Hello,
Thanks for your interest in our work!
Yes, it is intended. The intuition is that during mapping, we reconstruct the scene, and accordingly, we update the representation via RGB information. However, during tracking, we will only exploit the most reliable information.  That is why we use depth_mask to exclude pixels from contributing to tracking.


Crash discovery service with malformed WHOAREYOU packet

Description
Using a malformed RLP packet, we crash the discovery service of Teku permanently.
Attack scenario

We identify the public key of a Teku node, from which we compute its nodeId per discv5/ENR spec.
We craft a WHOAREYOU tag: sha256(dest-node-id || "WHOAREYOU")
We add 0xc1c0 as the body of the message, instead of the expected data.

See https://github.com/ethereum/public-attacknets/issues/18 for additional info

Indeed the discovery service stops responding after RLP decoding error. The reason is that in discovery there is a pipeline which is build upon RX stream. When the exception occurs it is not handler neither in packet handling code nor in RX stream error handler and thus the RX stream results in being closed abruptly and stops propagating further messages


Change file format from .sb (.smallbasic) to .svg in ReadShapes

SVG output is supported. But SVG input is not supported yet.

Closed in v1.96b.


update Building Realtime kernel tutorial, align it with the Ubuntu 20.04 release

Pull request is created based on the discussion at the ROS2 Real-time WG meeting

@clalancette, thanks for reviewing it, fixed

@clalancette, I have prepared a description of how to build an RT kernel for Raspberry Pi4 and need your advice.
Should I extend en existing page with it or create a new one? The kernel build is based on the Docker container I have created. What would be the best place to put a Dockerfile? Currently, it is located here https://github.com/razr/RTWG/blob/master/.gitpod.Dockerfile


Number only ID attribute is not allowed at chrome.

Number only ID attribute is not allowed at chrome.
like this
—
Uncaught DOMException: Failed to execute 'querySelectorAll' on 'Document': '#1671' is not a valid selector.
—
Fixed this issue.

Is the problem "numbers only" or "starts with numbers"? Your regexp is "starts with numbers"


Don't do event loop stuff when stopped.

Once we call stop on the app, get_current_app returns None because the app is reset. The problem is once this happens, kv rules and graphics instructions that reference the current app may get a None value and crash.
But ideally, once we stop the app, the clock should not continue ticking and the graphics should not be continued to update.

May be related: #6400 #6404


Fixed formatting/bugs

Fixes:

https://msdata.visualstudio.com/Database Systems/_workitems/edit/1073399
https://msdata.visualstudio.com/Database Systems/_workitems/edit/1069362
Bug where it showed 1 warning even when there were none.

Pull Request Test Coverage Report for Build 665716826

0 of 0   changed or added relevant lines in 0 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.002%) to 43.329%

Files with Coverage Reduction
New Missed Lines
%

extensions/import/src/wizard/pages/fileConfigPage.ts
3
72.99%

Totals

Change from base Build 665364557:
-0.002%

Covered Lines:
25414

Relevant Lines:
53516

💛  - Coveralls


https://validator.w3.org

🐛 Bug Report
there are lots of errors by validator.w3.org  and all errorrs are almost same:
Element p not allowed as child of element span in this context.
Errors are here
https://validator.w3.org/nu/?doc=https%3A%2F%2Fdocusaurus.io%2Fdocs%2Fen%2Fapi-pages

is this a major issue ? I personally feel this is quite trivial and doesnt affect end user.
there's no easy way to fix this on docu because the html is generated by markdown lib - Remarkable.  The generated markup includes a  element inside a  and that this is not allowed.
Wontfix

Thank you for clarification,
I have no idea about majority of issue just  informed you about error to contribute your project.


Last updated in overview is wrong for Mirror repositories

Gogs version (or commit ref): 0.9.0
Git version: 2.1.4
Operating system: Debian 8
Database:

[ ] PostgreSQL
[X] MySQL
[ ] SQLite

Can you reproduce the bug at http://try.gogs.io:

[X] Yes
[ ] No
[ ] Not relevant

Log gist:

Description
When seeing a repository in an overview it shows e.g. Updated 2 days ago when there were commits 1 hour ago.
For example on try.gogs.io:
https://try.gogs.io/cs
The Go repo (https://try.gogs.io/cs/Go/commits/master) shows (at time of writing): Updated 2 days ago
But when I click on it, it shows commits: 1 hour ago
Can this be fixed?
Thanks!
Rik

@rikvdh I can't reproduce this. Does bug exist now?

I have tried to create a repository and waited for a few minutes

On try.gogs.io
Version <IP_ADDRESS>0

The following html is generated (https://try.gogs.io/admin1 - repo test_issue_2807):

Updated 3 minutes ago

Patch has pushed to fix this issue, please test on develop branch or https://try.gogs.io.


Deployed a flist and it seems unstable

Deployed a flist, but the "CoreX service" / "server" seems to be unstable / unreachable. (My own network is stable and has no issues.)

@MathiasDeWeerdt Can you please always give the reservation ID when you open an issue.
If you are using node CBDY1Fu4CuxGpdU3zLL9QT5DGaRkxjpuJmzV6V5CBWg4 as access point for you network, there were some issue in the network causing what you see there. This was solved yesterday.
Can you confirm that you are using this node or not please.

Deployed a ubuntu container today and that seems to be working. I dont know which node I was using I just let the chat flow pick one out for me.


(Bug) walletconnect tries to reconnect when need to sign TX

Description
When performing a TX the native app selector is shown.
If then switching to the already open wallet app I see the TX to sign
BUT if i press on the wallet in the native app selector it tries to reconnect.
This seems like an issue in the wc Uri handling. if the uri doesnt contain connection information it should just open the walletconnect page without trying to connect.
Steps to Reproduce
ON ANDROID

go to any dapp (gooddapp/uniswap etc...)
connect using walletconnect and choose GoodWallet
perform any action that requires user signing (claim, swap)
the native wallet selector should now open
press on GoodWallet
the error popup is shown

@L03TJ3 still happens, please check the logs
https://images.zenhubusercontent.com/166818382/ba04db8a-6de9-4579-8ad2-33508699521b/c6ab7a53_227b_482c_8d03_437748d1cc26.mp4
https://app.zenhub.com/files/166818382/dcd18132-a4ea-423b-8e93-a32f08362e65/download

verified on prod


Added WiFi troubleshooting external dongle and additional information on how to backup/remove the map and added a link to the map editor.

Thank you for the PR, these are nice!


Auto - Refresh

Hope this question hasn't already been asked, but what would it take to add the ability to enable 'auto-refresh' feature?

That question/pull-request, indeed, has already been asked. https://github.com/puppet-community/puppetboard/pull/99

I'll be looking at this in the next few days folks, hang tight :smile:

Implemented in 177f6c234a975afc80938385ee17448de89ad33b


Cannot read properties of undefined (reading 'theme')

TypeError: Cannot read properties of undefined (reading 'theme')   at /cjs/theme/light.js

react-codemirror is a different package. Maybe try reporting this there.


[OS-3053] adding task complete handler

refer to https://blacklane.atlassian.net/browse/OS-3053 for more details

Able to delete successfully a Camunda process instance with business_key LHR_085b6d24-26cb-4217-bf70-a3a4db283df0.
{"level":"info","application":"camunda-sample-client","timestamp":"2022-09-23T15:49:50.358+02:00","message":"end"}


Add github release step to drone and real release to prod

closes #106

Create a backup of /opt/minitwit.db before merging to main


(docs)(PUP-5206) Note local filebucket default.

PUP-5206 changes puppet filebucket --local to use $clientbucketdir by default instead of $bucketdir. Note this in the command's man page.

@kris-bosland can you review and merge?

@kris-bosland via chat:

Hey Garrett, I think the sense on -local and -remote is incorrect in this doc update.  The clientbucketdir can be overridden with the bucket option, but the remote bucket cannot (or it's somewhere else).

For one, --bucket isn't even documented in this man page. So we should fix that, but probably not in this PR.
Also, if that's true, then the PUP-5206 acceptance criteria are wrong:

The --local flag to the filebucket command should always use the $clientbucketdir setting for its operations

Looking at the commit and your comment, the acceptance criteria should instead be:

The --local flag to the filebucket command uses the $clientbucketdir setting for its operations, unless overridden by the --bucket flag.

@jtappa @Magisus Can you confirm what I'm supposed to document here? This didn't seem complicated enough in the ticket to require pre-docs, but it sounds like there's either more going on or some information missing.

Sorry, I haven't been involved with the filebucket stuff, @joshcooper or @jtappa would know more, since they worked on the actual ticket.

@gguillotte, for --local, --bucket can override the '$clientbucketdir' setting for the location of the bucket.

My last comment was a little out of date, I didn't see the later comments.  Line 34 of the file does list the bucket option.

@kris-bosland It's in the list of options, yes, but we never describe what --bucket does — like overriding other options' defaults — in the man page.


enable travis; minor test-related fixes

The other changes were related to failures I was getting when running specs locally, but shouldn't affect anything else.
Fixes #29.

Aside from my comment above, :+1:.

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
Non-GitHub Verified Committers
There are 1 commit author(s) whose commits are authored by a non-GitHub verified email address. Chef will have to manually verify that they are authorized to contribute.
Please sign the CLA here.


[BUG] 2.6.0 release should be a major release

[ X ] I've read the docs of nestjs-pino
[ X ] I've read the docs of pino
[ X ] I couldn't find the same open issue of nestjs-pino
What is the current behavior?
2.6.0 upgrades to pino-http@7. Which has lots of breaking changes. Refer here
What is the expected behavior?
nestjs-pino 2.6.0 release should be released as a Major release.

2.6.0 upgrades to pino-http@7

@itizarsa but it keeps  support of previous major version of pino-http. Nothing changed for the current library and in the library API. It's up to end user which pino-http version to install as it's peer dependency


[After #806] Disable copy assignment / ctor for the model

Once a model starts to own some memory it shouldn't be able to copy the model as it the ownership of the memory is not clear.
Let's disable the copy assignment / ctor. If it needs to copy the graph (without assigning memory) I think we can have a static method designated for it.

:octocat: cibot: Thank you for posting issue #820. The person in charge will reply soon.


more DOS!!

Add more DOS Versions!
Only 1.25 and 2.00??
There is FreeDOS wich Supports al Versions!

DOS 6 please

+1

+1 for MS-DOS v6

CP-DOS | OS/2 ?  Windows 1.0?  Or the best of 1987, Windows/386 and Xenix 386!

MS-DOS 6 plz

MS DOS and non-NT Windows (because probably most NT functions are used in Windows 10) :)

One step at the time guys. Microsoft will probably release more versions as the time passes.
In the particular case of MS-DOS 6.x, the technology of doublespace/drivespace was licensed from Vertisoft. So I don't know if Microsoft would be allowed to making it open source.
And removing all the code of it, probably will require to invest time and money and Microsoft will not be interested in doing it.

The Windows 1.x/2.x source code would be more interesting to be honest. Maybe there would be comments in the code related to development?

I wonder whether Microsoft will release the source code of MS-DOS 6.22, or even MS-DOS 7.1 (part of Windows 98)

MS-DOS 6.22 FTW! ;)

https://www.youtube.com/watch?v=FONN-0uoTHI

we only need MS-DOS 5 because no PC should be without it!

@bitcrazed could you elaborate on that?


LPC55xxx flash driver

Adds flash support to lpc55xxx SoC's.
Tested with samples/drivers/flash_shell

"partiotions" mispelled in commit text of "dts: lpc55s69 all flash regions".

Some thoughts on what I might consider an acceptable way of doing this misaligned write. It should never happen as a result of calling the normal flash_write(). Ever. If we really want this, it should be a separate API. Code calling it needs to know that what it is doing is unsafe, and other code should never get this behavior just because of a configuration change.
Perhaps call the API flash_unsafe_misaligned_write_you_should_never_ever_call_this()?
This implementation wouldn't need to be specific to a driver.
But, at this point, why not just put this functionality into the flash shell. But, perhaps the flash shell should even print a warning on misaligned writes.
What about handling rewrites of data? Should the shell handle that as well? That will generate an error on many devices these days.

Realistically, the API could be called something like flash_unsafe_write_unaligned(). It doesn't really need to be that short of a name, because its use should be uncommon.

Removed unaligned sw managed flash write commit in order to get basic driver merged.
Commit will be moved to a separate pull request.

BTW, as far as the flash misalignment changes go, please put those into the flash shell, not as another patch that tries to add them to either this flash driver, or the flash subsystem itself.
The flash drivers and subsystem are working correctly, and should not contain workarounds that reduce reliability for every client, just because one client isn't implemented correctly.

Thanks @MaureenHelm I've updated and made another pull.
@d3zd3z the flash_shell was just one example where I needed unaligned writes; I can think of other examples such as logging applications that work with small data.
I find there is more benefit than risk, in having platform agnostic, alignment independent and easy to use write functions.

I find there is more benefit than risk, in having platform agnostic, alignment independent and easy to use write functions.

There is just no way to do this without making whatever system is doing it fail hopelessly when power is lost. Code that uses flash must know the characteristics of that flash device. Things like logging should be using a filesystem on top of a flash device, not the flash device itself.
I guess I don't really have a problem with a library like this (I don't think it should have any clients), but it is very important to me that the flash drivers themselves not be broken, and only support operations that can always be done with known characteristics.
Flash devices are hard to use. That's where there are filesystems that work on top of them. It doesn't help anyone to have things that might look like they make it easy to use, but only result in corruption and loss of data.


[ML][Pipelines] fix: compatibility with mldesigner

Description
Previous mldesigner import test will test class attributes only, but actually also require that the consistency of return value.
Updated test and code
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.

It seems there is formatting issue.


Add .papr support

Signed-off-by: Daniel J Walsh<EMAIL_ADDRESS>

CI is running, this is very promising

checking lint
make: ./.tool/lint: Command not found
make: *** [Makefile:67: lint] Error 127
make: Leaving directory '/root/gopath/src/github.com/projectatomic/libpod'

yourfault.../me runs away 😀
(srsly, giving a once-over of changes here...)

No idea why this closed.

I don't have permission to reopen - you're repo owner


Update Conemu , update clink 4.4

Made a new PR from the correct branch, against the correct branch.

Thanks.


Add support for Bitbucket Pipeline

Fixes #1835
Changes proposed:

[X] Add
[ ] Delete
[ ] Fix
[ ] Prepare

Add support for Bitbucket Pipeline

@robertohuertasm The icon doesn't meet our guidelines. Can you be a sport and pass it through Illustrator?


Feature Add: Max context length option

title.

Meh, second thought nah. Users can use too many different models, and its more likely the user will already know what the max length is or when things start to get wonky.