Unable to preview google doc or sheet in v6.7.0

Description
When trying to preview a google doc or sheet I get the error.
Previewing failed. Must be a Google document or spreadsheet.
To Reproduce
Try and preview a google doc or sheet.
Expected behavior
You should be able to preview a doc or a sheet.

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

GitHub release
v6.7.1

Your semantic-release bot :package::rocket:


Unable to query for all connections

Describe the bug
Abbreviated Schema:
User @model {
recommendations: [Recommendation] @connection(name:"User_Recommendation")
}
Restaurant @model {
name: String!
recommendations: [Recommendation] @connection("Restaurant_Recommendation")
}
Recommendation @model {
caption: String!
user: User! @connection(name:"User_Recommendation")
restaurant: Restaurant! @connection(name:"Restaurant_Recommendation")
}
When I create a new recommendation, I see it created in the database. It has the proper recommendationUserId and recommendationRestaurantId fields. But when I query for the user or the restaurant (like below) I do not see all of the Recommendations that should be connected.
query getUser {
getUser(id:"<proper_id_here>") {
recommendations {
items {
restaurant {
name
}
caption
}
}
}
}
To Reproduce

Create a User
Create a Restaurant
Create multiple Recommendations with the User/Restaurant
Attempt to query for the User's recommendations

Expected behavior
All recommendations created should appear (w/ captions 1 - 19) when using this query:
query getUser {
getUser(id:"6b822cf0-4352-4170-b888-25d96d2de549") {
recommendations {
items {
restaurant {
name
}
caption
}
}
}
}
What I actually get:
{
"data": {
"getUser": {
"recommendations": {
"items": [
{
"restaurant": {
"name": "Edo Salon"
},
"caption": "Hair test 12 w/ photo "
},
{
"restaurant": {
"name": "Edo Salon"
},
"caption": "Hair 18"
},
{
"restaurant": {
"name": "Edo Salon"
},
"caption": "Hair 15 w/ photo "
},
{
"restaurant": {
"name": "Edo Salon"
},
"caption": "Hair 6 w/ photo"
},
{
"restaurant": {
"name": "Edo Salon"
},
"caption": "Hair test 3 w/ photo"
},
{
"restaurant": {
"name": "Edo Salon"
},
"caption": "Hair test1"
},
{
"restaurant": {
"name": "Edo Salon"
},
"caption": "Hair 4 no photo"
}
]
}
}
}
}
Desktop (please complete the following information):

OS: iOS
Browser is chrome (version 74.0.3729.169)
Amplify 1.1.2

Smartphone (please complete the following information):

iPhone6s
OS: iOS 12.2
Browser: Chrome

*Closed because the problem was not using nextToken properly


unique_together

I have created a question on SO
http://stackoverflow.com/questions/28799949/django-polymorphic-models-with-unique-together
Someone suggested to use abstract=True but I am not sure if this works properly in this app.

Here is how I handled this problem.  Bear in mind that I use PostGres as my
database and I do not know if the same problems occur with other databases
(though I guess that they do).
Unique together constraints can only be applied to a single table or view
in PostGres.  This means that out of the box Django/Django-polymorphic
cannot express database-enforced unique constraints on a combination of
fields that are in both a parent table and a child table of Django models
in an inheritance hierarchy.
If you really want database enforced unique constraints on these fields,
you can do one of these two things:

copy any parent-model fields that are involved in the unique constraint
into the child's table, and express the unique constraint on the child's
fields and the fields copied from the parent, or
create a view on the child that includes both the fields from the parent
and those from the child, and express the unique constraint on this view.

Hope that helps.
On Mon, Mar 2, 2015 at 2:06 AM, variable<EMAIL_ADDRESS>wrote:

I have created a question on SO
http://stackoverflow.com/questions/28799949/django-polymorphic-models-with-unique-together
Someone suggested to use abstract=True but I am not sure if this works
properly in this app.
—
Reply to this email directly or view it on GitHub
https://github.com/chrisglass/django_polymorphic/issues/123.

Closing per accepted answer on StackOverflow! :+1:

Sorry for asking, but can you provide more detailed example/explanation of these methods? Seems like first won't work with fields with "related_name" and i don't completely understand second method


v3.0 plan, spam database and automation

This is the plan for v3.0, integrate spam database and automation, plz share your ideas if it doesn't cover your use case.
Table of Contents

Spam Database
Automation Workflow

Triggers
Actions
Presets

Examples

DNC daily .csv
Generate regex rule from local .csv
Daily backup

Spam database
With the Automation Workflow(see below), it'll be possible to import any database into this app.
There'll be a new item "Database" in the Quick Settings section.
Automation Workflow
On the setting page, there'll be a section showing all workflows.
A workflow is made of two things:

A trigger, for when it should be executed.
A sequence of Actions, each executed in order, the output of one action becomes the input for the next.

Triggers

manually
hourly
daily
weekly

Actions

network operations:

http_download

config

url: "https://...", supports

output

ByteArray

parsers:

parse_csv

config

column_mapping, e.g.: { "pattern": "Company_Phone_Number" }

input

ByteArray

output

RuleList

regex rules

add_regex_rule, create a new number rule like (11|22|...)

config

description
priority

input

RuleList

merge_regex_rule, merge numbers to the rule with the specific description, create if not exists

same as add_regex_rule, priority is only used when creating new rule

replace_regex_rule: replace a rule with the same description, and create if not exists

same as above

spam database:

spam_db_add

config

priority

input

RuleList

file operations:

read_file

config

dir, e.g.: "{Downloads}"
filename, e.g.: "input.csv"

output

ByteArray

backup and restore:

backup_export
config

dir, e.g.: "{Downloads}"
filename, e.g.: "SpamBlocker.auto.bak"

Presets
A workflow can be built from scratch with the UI, there will be a build dialog, however, if it's widely used, it will be included in the presets.
A preset is simply a json, it can be imported to generate the workflow.
Examples
DNC daily .csv
It will download DNC daily .csv and import the numbers into the database:
{
  "trigger": {
    "type": "weekly",
    "weekdays": [1,2,3,4,5],
    "at": "01:00:00"
  },
  "actions": [
    {
      "type": "http_download",
      "url": "https://www.ftc.gov/sites/default/files/DNC_Complaint_Numbers_{year}-{month}-{day}.csv"
    },
    {
      "type": "parse_csv",
      "column_mapping": { "pattern": "Company_Phone_Number" }
    },
    {
      "type": "spam_db_add"
    }
  ]
}

Generate regex rule from local .csv:
{
  "trigger": {
    "type": "daily",
    "at": "00:00:00"
  },
  "actions": [
    {
      "type": "read_file",
      "dir": "{Downloads}",
      "filename": "1.csv"
    },
    {
      "type": "parse_csv",
      "column_mapping": { "pattern": "my_number_column" }
    },
    {
      "type": "replace_regex_rule",
      "description": "imported from 1.csv",
      "priority": 10
    }
  ]
}

Daily backup:
{
  "trigger": {
    "type": "daily",
    "at": "00:00:00"
  },
  "actions": [
    {
      "type": "backup_export",
      "dir": "{Downloads}",
      "filename": "SpamBlocker.auto.bak"
    },
  ]
}

@Umotas

Every number can be spam today and cannot be spam tomorrow.

It can be solved by adding an automated workflow db_cleanup, with a configurable expiry, it will be included in the presets.

synchronize / parse local CSV-file

There's a feature request for it. I put it here just to show that it can grab data from anywhere, HTTP, local files, or even IPFS databases in the future. It can be easily supported by adding a new action protocol.
No plan for custom themes, the dark theme is just for reducing eye strain in the midnight.
@realgooseman

Having a custom filter + third-party crowdsourced spam database would make SpamBlocker very powerful.

Exactly, it will be capable of integrating all public databases into this app.

@realgooseman

Will the spam databases be included in the app with enable/disable buttons or will the user have to import them manually ?

The database will be a new row in the quick settings, looks like:

Contact ...
STIR ...
Database            [ 12345 records ]  [switch box]
...

And there will be a new section "Automation", just like the regex rules, it's a list of workflow items. Click a workflow to popup the configuration dialog, you can edit it with UI, no need to manually compose the json, the json is only for exporting a workflow so that you can share it to others. If you set a workflow as "schedule = daily, actions = [ http_download, import_to_db ]", then the database will get updated everyday, if you change it to "schedule = manually", then you can manually run it.

Are there European public spam databases, I believe there is STIR/SHAKEN for US citizens but I'm not familiar with European alternative ?

I don't know if there are any public European databases, but if there are any, I'll add them to the presets. And from google, because SHAKEN doesn't work for international calls, it would be ineffective unless other countries also implement it. So far, France seems to be the only European country that has implemented it (in 2023), maybe there will be more countries in the future.

Truecaller/Hiya

Maintaining such proprietary databases is time and money consuming, if they were to make it publicly accessible, it could be easily integrated with the workflow (if it's free and opensource), but it's impossible for them to do that.

I don't know whether only the suggestions related to spam database integration are welcomed.
What I feel this app really need is
An optional way to filter/delete the actual message instead of just the notifications.
There is another abandoned fdroid app Blacklist Blocker  which does this thing well. What misses in that app is the regex checking. It only allows sender blacklisting. It would be complete, if we can bring in those features of that app in this one.

@thomasjimmy
The database can't be generic without a flexible automation system.
About deleting messages, from that app's readme:

it is required to give to the application the permissions of "Default SMS-application" in the Settings.

That's why it's capable of deleting messages, it has to be the default app.

Hi @aj3423
Can we get a feature, where users can choose an option to delete the messages from the actual message instead of just filtering notifications? Those users who choose this feature alone need to give the "Default SMS-application" permission. This will increase the usability to a large extent.

@thomasjimmy Making it the "Default SMS-application" means it must do whatever an SMS app does, such as handling SMS/MMS conversation, group messaging, a fancy UI, etc. That would be making a brand new app, all the efforts, just for deleting messages? Nah..

I think implementing the database is a good idea if it does not detract from the primary original configurability of Spamblocker of giving the user maximum control of the number of ways to block spam as his situation demands which may vary according to locale and circumstances.
For example, in some countries/regions, telemarketers are almost unheard of while scam callers are the predominant villains. To my understanding, in such cases, a database will be almost useless since scammers typically use "normal" numbers to make calls and such numbers are too random and dynamic and rarely permanent enough to make it to a database which is more suited to targeting telemarketers.
That is why, to me, a powerful blacklist / whitelist configurable system is the best core of a spam-blocking app.

Just did a quick search, Begone is an app? It's supposed to support any kind of data, can you show me a sample XML from Begone?

The first version is done, for those who want to try the automation and database, here's the action build: https://github.com/aj3423/SpamBlocker/actions/runs/11217112568

There seems to be an issue about reading files that are not created by this app, maybe writing as well. I'll fix this later.
BTW, the {pattern} should be a json, for example {'my_number': 'pattern'}, which means map column my_number to pattern. If the column is already named  pattern, then there is no need to map, simply use a empty json {}.

@Umotas
Well, I thought it won't require any permission when accessing the Downloads folder, just like the backup export/import. But actually it requires permission, when you use that file dialog to select a file in the Downloads folder, that system file picker will temporarily grant that permission for you. However, when the workflow reads a file, there isn't such user interference, that's why it caused the permission error.
To solve this, now the app will ask for file storage permission when any file action is used.
Updated in the action build: https://github.com/aj3423/SpamBlocker/actions/runs/11247096939

The numbers should have been imported, The UI is not refreshed, try restarting the app.

@aj3423
A Begone  xml list : https://github.com/oulixalt/BegoneListeFR/blob/Liste-complète/Begone-FR.xml

@Nosnahc It's now supported: https://github.com/aj3423/SpamBlocker/actions/runs/11265746145
This xml doesn't seem to be up-to-date, I'm not adding it to the presets. You can import this configuration to test:

{
    "desc": "Begone",
    "schedule": {
        "type": "Daily",
        "time": {
            "hour": 0,
            "min": 0
        }
    },
    "actions": [
        {
            "type": "HttpDownload",
            "url": "https://raw.githubusercontent.com/oulixalt/BegoneListeFR/refs/heads/Liste-compl%C3%A8te/Begone-FR.xml"
        },
        {
            "type": "ParseXML",
            "xpath": "//dict/key[text()='number']/following-sibling::string[1]"
        },
        {
            "type": "ConvertNumber",
            "from": "(\\+)",
            "flags": 5,
            "to": ""
        },
        {
            "type": "ConvertNumber",
            "from": "(#)",
            "flags": 5,
            "to": "."
        },
        {
            "type": "ImportAsRegexRule",
            "description": "BeGone",
            "priority": 0
        }
    ],
    "enabled": false
}

How does Quick Copy work? I read the pop-up help about matching verification codes or OTP and tried to use regex to match a sample SMS containing a verification code but there's no Copy: 123456 button in the notification.
Sample SMS:
"Your verification number is 12345678. Please enter this and press the # key."
Regex tried:
(?<=is\s)\d{8}(?=\.)

I'm closing this since v3 is ready, welcome to test (the latest tag)
@noto10 This issues is only for discussion about v3, I created another issue for the regex problem: https://github.com/aj3423/SpamBlocker/issues/150

How can I replace my regex rules with Workflow? I tried "type": "ReplaceAsRegexRule" but it's failure

How can I replace my regex rules with Workflow? I tried "type": "ReplaceAsRegexRule" but it's failure

Not sure what you mean by "replace regex rule", please fire a new issue, because everyone in this post will get notified, I'm locking this post.


Add password validators

Story
At the moment it's possible to register a new user with passwords that have only one char or don't follow a minimum security protocol.
To enable it we need to add to our settings:
AUTH_PASSWORD_VALIDATORS = [
    {
        'NAME': 'django.contrib.auth.password_validation.UserAttributeSimilarityValidator',
    },
    {
        'NAME': 'django.contrib.auth.password_validation.MinimumLengthValidator',
    },
    {
        'NAME': 'django.contrib.auth.password_validation.CommonPasswordValidator',
    },
    {
        'NAME': 'django.contrib.auth.password_validation.NumericPasswordValidator',
    },
]

Acceptance criteria
GIVEN I enable the password validators in our settings
AND a new user registers going to our register page
THEN we validate the password

Moved: https://github.com/toladata/ActivityAPI/issues/26


Add configuration for build time dependencies

Resolves #145.
In accordance with https://peps.python.org/pep-0517/ , the build time dependency NumPy should be specified, as otherwise QML cannot be installed unless NumPy is already installed on the system, which breaks installation in fresh virtual environments.
This change appropriately specifies the build time dependency, and so should fix the issue. Once it is released, the following commands should run without error:
python3 -m venv venv
source venv/bin/activate
pip install qml

It is also worth noting as shown in the warning message in #143 that numpy.distutils as used in setup.py is depracated and should be migrated away from (https://numpy.org/devdocs/reference/distutils_status_migration.html). However, this is a bigger change which should be done in a separate pull request

Please note that further development for the QML package is now happening on the qmlcode/qmllib repository. qmllib is a port of the QML's Fortran core functionality with fewer dependencies and numpy 2.0 compatibility. You are encouraged to open any future issues or pull requests there.


adding field sorting to setState / getState?

Had a request today to save the multi column sorted state to localStorage - so had to extend the setState and getState methods.
Can create a  pull request for the sorting to be added to the columns state if the feature could be potentially useful?

get/set sort state can already be done via it's own api. this isn't part of the main column get/set state as it isn't managed by the columnController (inside my code). i'll keep this issue open as i may refactor to get it responsible, then the main get/set state will do this also.

it is just the case of ColumnController having a property columns -> and each column inside that array already contains sort property and is easy to expose (alternatively it should not belong there?)
Feel free to close if it goes against your design principles.
Thank you

no, makes sense. column controller should do it all. just the evolutionary nature of the design is why it's the way it is now.

see "Saving / Restoring State, Sort and Filter" at bottom of this page:
https://www.ag-grid.com/javascript-grid-column-api/
this is the preferred way of getting and saving the sort order.
we could change the design, however given it's the way it is now, changing the design would be a breaking change for people, so am leaving sleeping dogs lie.


Change IsSecond to match the design section

The third bullet in the design section says that the second unique key gets coefficient 1, but the algorithm doesn't match that.
This change will make the algo description match the design section.

It should be correct as it is. Say we have pk1 != pk2
Then if I'm not mistaken IsSecond([pk1, pk2], pk2) = true.

Run KeyAggCoeff([77, 43], 2) // 1-based index
IsSecond([77,43],2) will return false and a_2 will become != 1
The result is that the second unique key will not get coefficient 1, which contradicts the Design section.

Oops, forget about this. IsSecond is correct.

This part of the spec has already confused more people https://github.com/ElementsProject/secp256k1-zkp/pull/157#discussion_r777616214 and others. I'm wondering if we can write this down more clearly.

Maybe we can write a FindSecondUniqueKey which differs because

it will return a single pubkey and not work on indices (I think this is be clearer)
it's more natural because it iterates over they keys only once (this is how you'd implement it)

But I think the core of the problem is that the algorithm looks wrong at first glance. (I thought the same when I first saw it). But that's hard to fix. The algorithm itself is correct and as simple as it gets. Our brains here are wrong, not the spec. ;)

No matter how we write the algorithm, we could add a few input/output examples

One slightly related question:
If the keys are [44, 80, 80], then IsSecond([44, 80, 80], 3) would return true, right? This means that if the second unique key is repeated, all of them will get coefficient 1. Is that intentional? It seems intentional because the check compares actual keys, and not indices in "Return true if pkj = pki, otherwise return false"

@kallerosenbaum Yes this is intentional and the variant that is proven correct in the appendix of the MuSig2 paper (we call this variant MuSig2*).


Powrmatic

Hello and apologies for hijacking, I do not expect the problem to be rooted in this repo.
I do have a Powrmatic Vision Compact, which is basically the very same device as the Innova 2.0 Mini, just rebranded.
Since using the WiFi automation I have noticed that the device completely disappears from WiFi for hours or even days (comes back on after power down). This makes remote controlling very flaky, has someone else noticed such behaviour?
side note:
I had a brief look at the ESP-01 fitted to the control board. The (lean) firmware seems not to be Arduino based, which is not much of a surprise. I used the exploration to get a firmware backup.
edit: I too get returned "sw":{"V":"1.0.42"}, which I assume to be the firmware of the ESP-01.

Yeah that firmware is not very stable, when we are too aggressive on the polling it crashes and needs a restart. This is why I originally set the scan interval to 10 minutes in the home assistant integration.
Now it is configurable by the user. On my setup, I have it set at 3 minutes between polls and it seems to be doing ok. But when I started working on this library, I think 30 seconds would make it crash from time to time.

I did not (yet) thank you for your work, a big Thank You.
Occasionally I also do set the time of the device on "/api/v/1/set/time" with something like "tz=Europe/London&value=1730307624" <- being UTC Linux time stamp. Albeit its RTC seems to revert to 0 infrequently, when without access to the internet.
It does store the timezone in the flash. I hope it counterchecks for changes rather writing senselessly.


Do we have node-exporter dependency

Hi,
First of all, thank you for this awesome exporter.
I have a small question

The example dashboard needs the node_exporter installed. In order to select the nodes that belong to the ElasticSearch cluster, we rely on a label cluster. Depending on your setup, it can be derived from the platform metadata:

Readme denotes the above.
I see avg(irate(node_cpu{cluster=~"$cluster"}[10s])) by(mode) *100 in Avg. CPU Usage  dashboard.
And the cluster variable is label_values(elasticsearch_cluster_health_status,cluster) which is coming from es metrics.
How new labels related to each other?
because in my grafana dashboard

Hi, got the same issue, did you solve it?


Make newToken and newBaseInstance internal

One possible issue:
_installDandelionApps uses _getToken to associate a token with the DandelionVoting app. The value returned from _getToken comes from the sender's previous call to newToken, which overwrites any existing token value stored in the senderDeployedContracts mapping. Note that this means the value returned from _getToken may not be associated with an existing dao instance.
To avoid this issue we should make both newToken and newBaseInstance internal

closed by #31


Refactor interfaces to allow moving command tests

Trying to simplify the unit tests and move them into the commands directory (i.e., next to the commands they are testing).
PTAL @Connor1996

I suggest extract tikv/storage/kvstore into tikv/transaction, LGTM anyway

This sounds like a good idea. I think I will finish the transaction work, then do another refactor to move things into their 'right' places.

It's too weird to define InnerServer in storage

Why is it weird? It seems to me to make sense because the interface is defined where it is used. We could also pull it further up


New 'algorithmic' Z80 instruction decoder

This implements a new Z80 instruction decoder which which actually decodes the opcodes instead of branching over a huge nested switch-case. It's much less code, but also about 20% slower with only very moderate size reduction. The brute force switch-case decoding is probably better (but less elegant). The decoders can be toggled with the cmake option YAKC_Z80_DECODER.

With z80emu, I also use a decoder type emulator instead of the usual switch/cases. Regarding code size, the z80emu.o needed to run zextests, is around 14k (stripped). Whereas yaze's simz80.o (switch/cases type) is around 44k (stripped). Both compiled with gcc -O3 on Linux 64-bit. So I'm a bit surprised that you only got "moderate" size reduction. As for execution speed, it's of course slower, but not that much. When I developped z80emu (2006), I've done quite a bit of comparisons using the zextests as a benchmark. Yaze's was the fastest by a long way (and fully zexall ok). MZ80 which generates a switch-case in 32-bit assembly, making careful use of x86 flags and 8-bit/16-bit registers (after all, x86 descend from 8080), was actually slower than any C emulator. The Athlon XP I used was already optimized for full 32-bit  execution. I remember beating Marat Fayzullin's Z80 which is switch/cases (and not zexall ok). I tested one or two other emulators. Unfortunatelly, I lost the actual numbers, but zextests can be executed in around 1m30s if I remember correctly.

Thanks for the interesting info :) I was looking at the size reduction in the emscripten-compiled and gzipped version of the emulator, since I was interested how much the download size could be reduced. In that case the size reduction was only around 6 KByte (IFIR) from about 400 KByte overall. I guess in that case asm.js compressed size behaves very different than native x86 code.


Clicking a class crashes the app

Clicking on any item crashes the app, most likely due to the way the ZonedDateTime object is currently being handled.


Add markdown support to Title card

Would it be possible to add Markdown support to the Title card?

Apologies posted in the wrong place.


Mission Mode

Mission mode support would be cool imo

Stuff I was thinking about for mission mode:

Allow for individual packs to be downloaded much like CT distributions would be (although if a mission mode pack uses certain cts, you'd probably only allow it to be downloaded if the corresponding ct pack is also already downloaded)

File formats? I feel like keeping the mission mode files where everything is in one big ass file would probably be annoying for mission authors, so splitting everything into modular files per mission might be better. Also need to decide if it would be better to use the original mission file format, use the competition file format or use a custom file format. (competition files do have more options, but just a custom file format might be best if authors want to code custom mission gameplay modes, etc...)

Timer going down vs. up and character/vehicle being locked or not should probably also be options for the mission distro author to choose.

steal implement features from Palapeli's tournament museum like ghost saving, replays, etc...


No solution but still incorrect initialisation

I have a model with a MultiCostRegular.
The constraint emits a Contradiction during the initial propagation.
This is catched that way in SearchLoop.java:
private void initialPropagation() {
   this.env.worldPush(); // store state before initial propagation; w = 0 -> 1
   try {
       solver.getEngine().propagate();
   } catch (ContradictionException e) {
       solver.getEngine().flush();
       solver.setFeasible(FALSE);
       interrupt(MSG_INIT);
       smList.onContradiction(e);
       this.env.worldPop();
       return;
   }

interrupt() sets nextState to RESUME then the call to launch() lead to this exception:
            throw new SolverException("!! The search has not been initialized.\n" +
                    "!! Be sure you are respecting one of these call configurations :\n " +
                    "\tfindSolution ( nextSolution )* | findAllSolutions | findOptimalSolution\n");

So the problem here is that, to me, the solver is exaggerating the situation. Everything has been initialized correctly. The only real problem is that the solver detected the model has no solution during the initial propagation.

I refactored the SearchLoop object, this behavior is not encountered anymore.

I tried with the snapshot version. Now I have still a SolverException: "ERROR [Solver] The entire search space has been explored. Consider resetting the search loop."

Sorry, I didn't deploy the snapshot. That's done now.

Still got the issue with choco-solver-3.3.2-20150615.090524-4.jar (the last on the sonatype snapshot repository)

It is ok now. I messed up at something into my code.
Thanks for the patch


Add Manifest: added missing manifest for th-ch.YouTubeMusic 1.6.3

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest ?
[x] Have you tested your manifest locally with winget install --manifest ?
[x] Does your manifest conform to the 1.5 schema?

Note:  is the name of the directory containing the manifest you're submitting.

This PR adds missing manifest 1.6.3 for the th-ch.YouTubeMusic package. Note: currently, versions newer than 1.20.0 are not being recognized as newer by winget, due to these versions being prefixed with 'v'.
Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:


[GQSP] Optimally scaling polynomial approximations

Context: P is a QSP polynomial if $|P(e^{i\theta})| \le 1$ for every $\theta \in [0, 2\pi)$.
When computing polynomial approximations for functions (that have infinite series), the polynomials may end up having max abs. value on the complex unit circle larger than 1, even if the actual function's absolute value is always at most 1 on the unit circle.
Current fix (https://github.com/quantumlib/Qualtran/pull/851) evaluate polynomial on (1e5) uniformly spaced points on the complex unit circle to compute the maximum absolute value, and scale down P by that.
If anyone has ideas for more efficient ways to compute the optimal scaling, please do share!
cc @tanujkhattar @skushnir123 @NoureldinYosri @ncrubin

how large is the degree of the polynomial?

The current simulation tests use up to ~ 300. We would ideally want to test it on degrees upto 1e7.


Bluetooth: Mesh: Expose destination address in bt_mesh_msg_ctx

There are valid use cases where the model layer must know the true
destination address. So far only the fact that it was one of the
addresses that the model subscribes to (its element's unicast
included) has been knowable.
Solve the issue by moving the destination address from the internal
net_rx context to the public bt_mesh_msg_ctx struct.
Fixes #7453
Signed-off-by: Johan Hedberg<EMAIL_ADDRESS>

@bluetooth-mdw FYI

Codecov Report

Merging #7454 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #7454      +/-   ##
==========================================
+ Coverage   55.26%   55.26%   +<.01%
==========================================
  Files         467      467
  Lines       51643    51643
  Branches     9886     9886
==========================================
+ Hits        28540    28541       +1
+ Misses      19193    19192       -1
  Partials     3910     3910

Impacted Files
Coverage Δ

lib/posix/pthread.c
69.84% <0%> (+0.5%)
:arrow_up:

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 3bbe87e...2e05dd7. Read the comment docs.

@Vudentz @carlescufi I think this can be merged. I've tested it to be working ok.


Assignment 1

TITLE: UofT-DSI |  Python - Assignment 1
What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
create an anagram checker that compares two words with/without case sensitivity
What did you learn from the changes you have made?
order of operations matter. It's much easier to sort the words after they were split.
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
Yes, perhaps there's an easier way to remove spaces. perhaps join method is not required to remove spaces?
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
Yes, I was trying to understand the difference between sort vs. sorted method.
How were these changes tested?
I used various words to test, e.g. those with spaces, those without, with/without case sensitivity
A reference to a related issue in your repository (if applicable)
N/A
Checklist

[x] I can confirm that my changes are working as intended

You did a great job here!
Please go ahead and close this pull request now. Thanks!


Many InvocationTargetExceptions in pt branch

E.g. InvocationTargetException for field 'weight' of com.graphhopper.routing.VirtualEdgeIteratorState instance

Thanks!
Should we fix the problem that the VirtualEdgeIterator throws exceptions (which one)?

Hmm, not sure if it's a problem. For a getter, throwing an exception to mean "not available" is not very common, but it's not.. as such.. wrong. The code now does what it should: Not include the attribute when the getter throws. And I'm just serializing these things for debugging, anyway. Not sure if this will even be there when we merge.
I'd be ok with just leaving it until we think it may be a problem for the second time. :-)

(I just stuffed the class into a JSON-Serializer which just scrapes off all the getters which were probably not even meant for that, so.. it's not the class's fault.)


TFC Reloaded 2 mix Poetica

Describe the bug/crashreport
Game crash in loading screen
To Reproduce
open the modpack
Expected behavior
open
Screenshots
If applicable, add screenshots to help explain your problem.
Crash report Link
crash-2022-04-15_21.10.41-client.txt
Version

Minecraft: : 1.12.2
TFC Tinkers : 1.5.6
Forge : <IP_ADDRESS>60

Additional mods involved
idk

downgrade rocksalt.


guarantee(loc != NULL) failed: missing saved register

I just encountered this JVM-internal error while profiling and suspect that it might be related to the profiler, as I haven't seen this error outside of profiling yet.
#
# A fatal error has been detected by the Java Runtime Environment:
#
#  Internal Error (oopMap.cpp:352), pid=7174, tid=7200
#  guarantee(loc != NULL) failed: missing saved register
#
# JRE version: OpenJDK Runtime Environment Corretto-<IP_ADDRESS>.1 (11.0.10+9) (build 11.0.10+9-LTS)
# Java VM: OpenJDK 64-Bit Server VM Corretto-<IP_ADDRESS>.1 (11.0.10+9-LTS, mixed mode, tiered, compressed oops, g1 gc, linux-amd64)

I can't share the full dump publicly, but let me know if you need more info or if you think this is unrelated to async-profiler.

Does not look familiar to me.
Can you mail the entire log (with possibly erased sensitive information) to < upload at pangin.pro >? This is my private address.
Has the crash happened just once?

Yes, it happened only once and hasn't happened again in the next dozen profiling runs. So it's not a critical issue for me.

I didn't succeed in reproducing this particular crash, however, I committed a couple of stability improvements that might be related. Feel free to reopen the issue if you encounter the problem again with the most recent async-profiler.

I didn't succeed in reproducing this particular crash, however, I committed a couple of stability improvements that might be related. Feel free to reopen the issue if you encounter the problem again with the most recent async-profiler.

I've finally found the root cause of the crash and fixed it.


Add support for Protobuild CLI

This adds support for showing instructions on using Protobuild to add or install a NuGet package.
In addition, it detects packages that were created with Protobuild and thus have additional metadata included in them to make cross-platform projects work better, and recommends the use of the client in these scenarios.

@hach-que,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

In order to evaluate whether a CLI can be listed on the package details page, we look at a pre-defined set of criteria. In the case of this PR, we evaluated that the criteria of “Significant adoption in .NET community” is not something we can currently establish and hence we will not be able to accept this PR.
NuGet.org uses the “User Agent” string supplied by the CLI to determine usage. In case you are not already setting the “User Agent” in NuGet.org requests, you should consider setting this to something that will help us uniquely identify usage of the client.
If you would like to discuss further about this, please create an issue on NuGet/NuGetGallery repo or reach out to us at<EMAIL_ADDRESS>

Closing the PR for now (See above comment for details).
If you would like to discuss further about this, please create an issue on NuGet/NuGetGallery repo or reach out to us at<EMAIL_ADDRESS>

We'll add User-Agent (since I'm having to update the binary to support the X-NuGet-Protocol-Version: 4.1.0 header). However, it should be noted that to date there's over 300,000 downloads of packages which support the Protobuild-enhanced package protocol.


Support templated output

This PR includes a few commits with the primary purpose of adding a --template flag to support customized output.
See individual commits for details.

Closes #10


⚠ Bofrost.es has degraded performance

In 354f08c, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1686 ms

Resolved: Bofrost.es performance has improved in 9a97f7b after 14 minutes.


add logic to accept aria or bg-image on button with no text

Resolves https://github.com/bbc/bbc-a11y/issues/245
Summary
Make it possible to have a button with no text as long as there is a supporting aria-label, aria-labelledby or a background-image.
Details
Adds the logic in the tests for Fields must have labels or titles for buttons.
Motivation and Context
We want this expected behaviour.
Given a page with the following markup:
<button class="arrows__chevron"></button> /* .arrows__chevron has background-image style */

Then the test "Fields must have labels or titles" passes.
Given a page with the following markup:
<button aria-label="Scroll carousel right"></button>

Then the test "Fields must have labels or titles" passes.
Given a page with the following markup:
<button aria-labelledby="S1 itemTitle"></button>

Then the test "Fields must have labels or titles" passes.
How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

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

Checklist:

[ ] I've added tests for my code
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Hi,
Perhaps I am missing something but a background image isn’t an accessible label. There’s no way to provide alternative text and users who cannot load or see the image will be left with no label for the element.
I think this change goes against the purpose of the requirement. Background images are not sufficient to provide a label for a button.

Thanks for you input @JamieKnightBBC. I'll feed this back in the original issue and remove the background image changes if there's agreement on this.

@JamieKnightBBC having a look over the issue I think I was misunderstanding what changes were needed. I've changed the test to fail only when the button has no text and is using a background-image with no aria-label. I've pushed the changes.

Agree with Jamie, and have left comments on #245 ... a button should have an accessible label that is available programmatically as well as visibly.

it is OK in some instances for the programmatic label to be provided to the accessibility tree via an aria label or labelled by attribute, so that should pass the check.
it is not OK for there not to be a visible button, which could be an image, or text that is in some way different to the aria label, such as "share" with and expanded aria label that conveys what will be shared that is evident from the layout.
it is OK for the visible button to be a symbol rather than text, such as the chevron example, and an alt attribute or aria label helps facilitate this.
a button should be visible without CSS or javascript, though it can look different, such as CSS using a background image and moving text out of sight that then becomes visible without CSS.

It would be good to ensure any examples in feature files are all examples of good practice, or if failing examples are only demonstrating one thing that would fail.
Feel free to Slack me if anything needs clarification.

Closing in favour of https://github.com/bbc/bbc-a11y/pull/313
Still to do in that PR:

aria-labelledby support
checking for existence of background-image (I think I need some clarity on this suggestion)

@EmmaJP


[FEAT] Website does not have proper meta tags

Is your feature request related to a problem? Please describe.
The website does not have many important meta tags. This means that the website is not as visible in search results, and users are less likely to click on the website's links. This will help the website rank higher in search results and attract more visitors.
Describe the solution you'd like
Add meta tags like description, keywords, open graph, twitter, language, etc.

Please assign me this issue so that I can work on this issue under GSSOC'23.


Use env.HOME as a checkout reference

Closes: #1803
Signed-off-by: Morgan Davies<EMAIL_ADDRESS>

Draft for now while awaiting review from @AdamBrousseau @sxa

Might be that the openjdk_tests file is executed before cygwin boots up and we don't have HOME manually set on the windows boxes? Someone would need to dig into one of them and take a look though

@AdamBrousseau At J9, do you have HOME set on windows itself (not cygwin)?

I setup a VM for myself using the Adopt Playbooks and HOME is set to /home/vagrant in both powershell and cygwin so I don't think it's that.

What about a normal windows command prompt?

Hmmm that's using a UNIX style path which suggests to me it was the cygwin setup that did it as a system / user environment variable

@sxa Same variables. UNIX style path for HOME is also seen on command prompt

OK it must have been set as an environment variable by the cygwin installer then, but for some reason not on the affected machines. Do we know how many machines are affected? Presumably not all of them...

A quick look through the Failed jobs I've seen it on at least these four machines. But there could be more
test-aws-win2019-x64-1
test-aws-win2019-x64-2
test-softlayer-win2012r2-x64-2
test-azure-win2012r2-x64-1

The only online test Windows machine that I didn't see producing this error is https://ci.adoptopenjdk.net/computer/test-azure-win2012r2-x64-1

Looking at an OpenJ9 machine. I don't see HOME set in the Windows config nor in a command prompt.
Are all the Adopt (win) machines connected via JNLP (Windows Service)? All ours are ssh direct from master.

We do not have ssh on our windows machines - they all call back to the jenkins server

@M-Davies You've put test-azure-win2012r2-x64-1 in the list, and then said you don't see the error on it? Those are the only 4 online windows machines

My mistake, that machine also sees this error https://ci.adoptopenjdk.net/view/Test_openjdk/job/Test_openjdk11_hs_sanity.openjdk_x86-64_windows/227/console

From the looks of it, we do have HOME set on :
test-aws-win2019-x64-1
test-softlayer-win2012r2-x64-2
test-azure-win2012r2-x64-1

At least in the same way you checked it (printenv on both cygwin64 terminal & cmd).
(I don't have the credentials for the second aws machine).

I get the impression it could be due to differences in the Jenkins specific plugins (scm for instance), assuming the OpenJ9 machines are setup with our playbooks with minimal change, and OpenJ9 use a different CI to Adopt. I'm unable to find any instance on any of those machines where HOMEPATH is defined, but HOME isn't.
Given that the $HOME variable is there  and AFAIK there isn't any other issues about the $HOME variable not being found, I don't think it's an issue to just use HOMEPATH :-)

Has this PR actually been tested at Adopt? Since home is in the env, maybe it's something plugin/Jenkins side as Will says.

Hmm. It looks like this PR doesn't work with HOMEPATH either (I really need to remember that grinders can test openjdk-test branches!)
https://ci.adoptopenjdk.net/view/Test_grinder/job/Grinder/3132/console

Do we recall the reason the Adopt Win machines are setup using jnlp and not ssh? At this point that's the only difference i see.

It seems HOMEPATH needs to be prepended with env to pick it up
https://github.com/AdoptOpenJDK/openjdk-tests/compare/master...M-Davies:repo_scm_test
https://ci.adoptopenjdk.net/view/Test_grinder/job/Grinder/3135/console

Doing the same for HOME works too on windows.
https://ci.adoptopenjdk.net/view/Test_grinder/job/Grinder/3136/console
I'll do some more testing with other OS's to ensure it doesn't affect them. If that works out, it should be ok to shift this PR to instead use env.HOME over all of the other ideas discussed here.

I've done some testing with internal and Adopt jenkins instances. Both work with env.HOME on all the platforms I could test. I'll update this PR then await some more reviews

Hmm, I wasn't able to bounce jenkins over the weekend. I wonder if there's a plugin concern here.

Kicked off

https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk11u/job/jdk11u-windows-x64-openj9/614/
https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk11u/job/jdk11u-windows-x64-openj9-windowsXL/355/
https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk11u/job/jdk11u-windows-x86-32-hotspot/550/
https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk11u/job/jdk11u-windows-x64-hotspot/607/

to see if the bounce last night did anything.

Looks like the bounce didn't fix anything https://ci.adoptopenjdk.net/job/Test_openjdk11_hs_sanity.openjdk_x86-64_windows/231/console is still showing the issue. Can this be pushed through if everyone is happy with it?

The adoptopenjdk restart was done yetserday but a further one occurred this afternoon

Trying again with https://ci.adoptopenjdk.net/job/build-scripts/job/openjdk11-pipeline/979/console after the 2nd restart

Same error after the 2nd restart https://ci.adoptopenjdk.net/job/Test_openjdk11_j9_sanity.openjdk_x86-64_windows/226/console

Same error after the 2nd restart https://ci.adoptopenjdk.net/job/Test_openjdk11_j9_sanity.openjdk_x86-64_windows/226/console

I got a bit lost with the very long thread here. Is the https://ci.adoptopenjdk.net/job/Test_openjdk11_j9_sanity.openjdk_x86-64_windows/226/console test effectively manually applying this PR and it's not working?

Same error after the 2nd restart https://ci.adoptopenjdk.net/job/Test_openjdk11_j9_sanity.openjdk_x86-64_windows/226/console

I got a bit lost with the very long thread here. Is the https://ci.adoptopenjdk.net/job/Test_openjdk11_j9_sanity.openjdk_x86-64_windows/226/console test effectively manually applying this PR and it's not working?

It was thought that the jenkins restarts would have resolved the error raised in https://github.com/AdoptOpenJDK/openjdk-tests/issues/1803 so merging of this was put on hold. However, this has turned out not to be the case and as such, this PR is still a valid fix for the original error.
This has been manually applied in testing grinders and it seems to work (both on adopt and ibm grinders) so I'm just waiting on @smlambert 's review to make sure she is happy with the changes here.

Running https://ci.adoptopenjdk.net/job/build-scripts/job/openjdk11-pipeline/981/console to confirm the fix worked


modify transform endpoints

What changes were proposed in this pull request?
as discussed with @huitseeker
I modified CSVSparkTransformServer

/transformincremental : deal with a single record and return transformed record
/transform : deal with a batchRecord and return transformed record
/transformedincrementalarray : deal with a single record and return 64ed ndarray
/transformedarray : deal with a batchRecord and return 64ed ndarray

https://github.com/SkymindIO/lagom-skil-api/issues/167
How was this patch tested?
Test code(CSVSparkTransformServerTest) included
/cc @huitseeker

See also https://github.com/SkymindIO/lagom-skil-api/pull/170

 Improving API endpoints

@agibsonccc The naming switch was hinted at here : https://github.com/SkymindIO/lagom-skil-api/pull/170/files (you merged it)
and in the text of the PR : https://github.com/deeplearning4j/DataVec/pull/295#issue-231540115
and finally, as to the "why", here: https://github.com/SkymindIO/lagom-skil-api/issues/167#issuecomment-303255367

Ahh I see what happened now. Github hid the side by side changes from me. It looked look like batch and co didn't get fully migrated. That's the only reason I commented. I agreed with the reasoning.

@huitseeker thanks for precise explanation :)


chi.FileServer does not server index.html when asking explicitly for it

Using the example for chi.FileServer and adding a dummy index.html, when asking for curl -i http://localhost:3333/files/index.html the file is not served, but instead a redirect to Location: ./ is given.
This behaviour does not occur with the original http.FileServer.

Seems like the default behaviour of http.FileServer.


Cleanup the front page

[x] remove all the example HTML
[x] change the title
[ ] change the icon to something nice

#26


Customizing The Background Controls

Hello,
How can we customize the background player notification panel on Android? Right now it's just a grey box with a play button!

You have to set which capabilities you want to support. See updateOptions.


Bump dependencies versions

I'm trying to update the libraries on the project but I'm getting 403 on mesosphere maven.
error]         download error: Caught java.io.IOException: Server returned HTTP response code: 403 for URL: http://downloads.mesosphere.io/maven/com/twitter/util-core_2.11/6.37.1/util-core_2.11-6.37.1.pom (Server returned HTTP response code: 403 for URL: http://downloads.mesosphere.io/maven/com/twitter/util-core_2.11/6.37.1/util-core_2.11-6.37.1.pom) while downloading http://downloads.mesosphere.io/maven/com/twitter/util-core_2.11/6.37.1/util-core_2.11-6.37.1.pom
[error]         not found:
[error]             /Users/rafael/.ivy2/local/com.twitter/util-core_2.11/6.37.1/ivys/ivy.xml
[error]             https://repo1.maven.org/maven2/com/twitter/util-core_2.11/6.37.1/util-core_2.11-6.37.1.pom
[error]             https://jcenter.bintray.com/com/twitter/util-core_2.11/6.37.1/util-core_2.11-6.37.1.pom
[error]             http://repo.spray.io/com/twitter/util-core_2.11/6.37.1/util-core_2.11-6.37.1.pom
[error]             http://dl.bintray.com/emueller/maven/com/twitter/util-core_2.11/6.37.1/util-core_2.11-6.37.1.pom
[error]             http://repo.typesafe.com/typesafe/releases/com/twitter/util-core_2.11/6.37.1/util-core_2.11-6.37.1.pom
[error] Total time: 8 s, completed 28-Aug-2018 20:40:01

Any ideas?

checking...

twitter repo isn't in our repo list... it will need to be added... I don't remember the details but I believe this is OSS and available.

ill Check.  Is there a cache in mesosphere maven ? Or they changed where it is hosted?

Weird that it seems to be looking at _2.11

tried updating only play version and got the same issue.
[error]         download error: Caught java.io.IOException: Server returned HTTP response code: 403 for URL: http://downloads.mesosphere.io/maven/com/typesafe/play/play-json_2.12/2.6.18/play-json_2.12-2.6.18.pom (Server returned HTTP response code: 403 for URL: http://downloads.mesosphere.io/maven/com/typesafe/play/play-json_2.12/2.6.18/play-json_2.12-2.6.18.pom) while downloading http://downloads.mesosphere.io/maven/com/typesafe/play/play-json_2.12/2.6.18/play-json_2.12-2.6.18.pom
[error]         not found:
[error]             /Users/rafael/.ivy2/local/com.typesafe.play/play-json_2.12/2.6.18/ivys/ivy.xml
[error]             https://repo1.maven.org/maven2/com/typesafe/play/play-json_2.12/2.6.18/play-json_2.12-2.6.18.pom
[error]             https://jcenter.bintray.com/com/typesafe/play/play-json_2.12/2.6.18/play-json_2.12-2.6.18.pom
[error]             http://repo.spray.io/com/typesafe/play/play-json_2.12/2.6.18/play-json_2.12-2.6.18.pom
[error]             http://dl.bintray.com/emueller/maven/com/typesafe/play/play-json_2.12/2.6.18/play-json_2.12-2.6.18.pom
[error]             http://repo.typesafe.com/typesafe/releases/com/typesafe/play/play-json_2.12/2.6.18/play-json_2.12-2.6.18.pom
[error] Total time: 12 s, completed 28-Aug-2018 21:29:06

maybe sbt is not dealing well with a 403 from mesosphere maven

I'll take a look on this tomorrow US time if @alenkacz doesn't during EU time

Thanks @kensipe
I'm on EU time at the moment. This PR is not yet finished and I created now to be able to get some help with this 403 issue. As soon as the dependencies can be downloaded properly I'll continue and start adjusting the code in case a dependency update requires it.

Hey guys, any updates on this issue?

Issue found.
Seems that TwitterCommons in the newer version required finagle-core-java which is not able to find and play-json don't have version 2.6.18 (only 2.6.10).
I did those changes and I get dependencies to resolve.

Can someone label this pull request as work in progress since I don't have permissions to do it?

Updated the PR now

BTW, the first green build.

Late in EU now, I’ll work on the comments sometime tomorrow but thanks for the thorough look on it! Few free to send all comments now and I’ll send an update tomorrow

There are 2 unresolved comments here. Please comment in case I need to do any more changes :mega:

I don't think there is anything else left. I left https://github.com/dcos/metronome/pull/254#discussion_r215053229 and https://github.com/dcos/metronome/pull/254#discussion_r215051232 as unresolved comments because I didn't know if you guys were happy with them. But if you guys are happy then I won't add anything to this PR anymore

I'm pretty happy... I triggered the build... I'll update the label.

I ran all the tests locally now. Seems like everything passed. Can you send me the message or the log of why the build failed?

@grillorafael looks like an env issue... looking to resolve it and re-run

@grillorafael this generally looks good... however we are looking to have as few implicit as possible.   It looks like you have extended the list.   Could we switch that back?  It isn't part of the need for the bump in dependencies as well.     That change and we can get this merged.

Will do the change 👍

Thanks @grillorafael !

Realizing just now that it's not really clear since we don't have lots of (any?) contributing guidelines for Metronome :) What we have written for Marathon applies, though:
https://github.com/mesosphere/marathon/blob/master/docs/docs/code-culture.md#on-implicits :)

PR updated. I brought back wire[Controller] to Api module. If you prefer explicit parameter passing in there let me know and I'll rollback. I added the changes to my latest commit https://github.com/dcos/metronome/pull/254/commits/14a1dafcfc1956ac93ffba2f8685d06bf7ae26a1


USER STORY: Audit Report

Full User Story
As an admin user, I want to receive and Audit report, so that I can more easily catch errors and reconcile reports against my accounting system before submitting to Treasury.
Action Plan

[ ] Initial design drafted
[ ] Initial acceptance criteria drafted
[ ] High level technical approach & feature flag use determined
[ ] Designs & Acceptance criteria finalized
[ ] Eng task breakout complete
[ ] Assessment of whether partner training & communication actions are needed
[ ] Acceptance criteria testing completed & passed in staging
[ ] Feature flags enabled in production

Engineering Tasks
Acceptance Criteria
USDR Admin & Org Admin should have access to generate an audit report for the current reporting period.
On the uploads page, next to the send treasury report by email, a button should display that says "Send Audit Report by Email"

this should only be visible to USDR Admin & Org Admin (not Org Staff)
once clicked, a confirmation toast should appear saying "Audit report is being generated - this may take up to 1 hour to send to your email."
Within an hour, the audit report email should be sent to the email of the user who clicked the button. The audit report should reflect data that is up to date with what is in the tool at the time the button is clicked.

General email info:

Title: Your CPF audit report is ready to download
Sender name: USDR CPF Reporter<EMAIL_ADDRESS>
Email body:

The header should match the treasury report email - i.e. US Digital Response logo with "CPF Reporter" underneath
The body should say:
Your Audit Report is ready to download.
Click here to download your file ["click here" should hyperlink to the audit report download]
Or paste this link into your browser: [show full url]
This link will remain active for 7 days

Audit Report
Link to mockup
Link to cpf audit report field specifications- The audit report should be a .xlsx file

The file name should include the date and time it was generated (i.e. "cpfauditreport1001202412:22:00.xlsx")
The file should have one sheet named "Project Summaries"
The first row should reflect the column names included in row 1 of the mockup Link to mockup
Starting on row 2, the file will show all projects that have been reported on in an organization across all reporting periods that have at least one valid upload. Each row corresponds to one project, as denoted by project ID.
For each project, "general" (i.e. non-quarterly) data will display, as well as data for each quarter that the project has had valid uploads.
This file describes all general and quarterly data, including which column of the upload template it comes from. cpf audit report field specifications
The mockup file shows how the file should display, including a tab with data to show expected formatting. Link to mockup

This story is ready for eng review!


Example new Block creation tutorial

GraphQL etc coding is not my area of expertise. Would it be possible to have a short tutorial on how to add a couple of new simple blocks. HelloWorld and slightly above in complexity?

I do not want enter into GraphQL intricancies. Too complicated for a HTTP call from Web.

For new Block creation : please follow https://developers.google.com/blockly/guides/create-custom-blocks/overview

What do you want to create?

Thanks for the reply.  It was a general query. I'll look into that Blockly tute.
In the meantime I have found a partial way around that. I create a controller called Misc in which I can put some C# coding in a Get method. I then can call it from Blockly with parameters, get the processed result and use that. I have used it to create a detailed menu string from a CSV string.
Will post on my blog site, but the build on Azure is failing.

Here is my blog post on creating a Custom block via call to the Controller:
https://djzblogaccname137.z26.web.core.windows.net/softata/Softata-CustomBlocksviaASP.NETController-softata.html


Problem refresh data hostgroup

Hi,
I try restart grafana, remove plugin, add plugin again but doesn't refresh list of hostgroups in "select triggers" panel.
I created new hostgroups and I can't see them!
Any suggestion?
Thanks

Hi,
I could finally solve the problem, the user was connecting to the api had read only permissions, and by default zabbix doesn't put new hostgroups as allowed for this profile "read only"
Once I check user permissions in zabbix I could see the new hostgroups ;)
Regards!

6 years after this post saved me form hours of research !


Clusters and arrays for f103

Now svd2rust can't handle arrays with dimension 1.
But before it could.
Sometimes this is useful.

Closes #56 .

Does svd2rust process clusters?

Thanks for adding these features! I'm glad to get them into the upcoming release.


refactor: use shorthand syntax

So basically, I think this is the same kind of thing as #864.
In principle I'd accept a PR that fixed all of them. But to be honest, I'm not sure it's worth it. A lot of this code is going to get dropped Real Soon Now.


[8.x] [POC] Allow passing in Enum class to migration

Now that we can cast values to Enum's, it seems as though it would make sense to allow specifying the Enum class in the migration so that the class is the source of truth for the values
If you had the following enum class:
<?php

namespace App\Enums;

enum ServerStatus: string
{
    case Online = 'online';
    case Offline = 'offline';
}

You could pass the class to the migration:
<?php

use App\Enums\ServerStatus;
use Illuminate\Database\Migrations\Migration;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Support\Facades\Schema;

class CreateServersTable extends Migration
{
    /**
     * Run the migrations.
     *
     * @return void
     */
    public function up()
    {
        Schema::create('servers', function (Blueprint $table) {
            $table->id();

            $table->enum('status', ServerStatus::class);

            $table->timestamps();
        });
    }

    /**
     * Reverse the migrations.
     *
     * @return void
     */
    public function down()
    {
        Schema::dropIfExists('servers');
    }
}

I have not added any tests as I am not sure if anyone would like this idea, but I would be happy to add if this concept would be of value.

Argh, sorry. I had my VS Code formatted on.  Will fix.  Sorry.

This looks like a cool feature at first, but actually I'm not sure if this is a good idea to merge. The enum class can change over the development lifetime of your application (for example adding a new case in it) but the same migration should always produce the same database schema no matter at which time it has been run.

Agree with @X-Coder264
Database changes should not be tied with codebase

Ya, I am very torn about this too.  I am very torn about enum in a database as a whole, as it gets tricky when you need to add a value.  I was not sure what is the best way to deal with adding cases to the enum class & keeping the DB in sync or even if that is the responsibility of the framework or the responsibility of the programmer.
Great feedback.  Hopefully, someone has a good suggestion.

What's the status on this PR? Can this be reviewed again?

What's the status on this PR? Can this be reviewed again?

From a coding perspective, it is working, but I would be happy to add any tests if people approve/like the POC.

I'll let Taylor review this again.

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.

Totally understand.  I think that there is a need here, but just don't know if we have a good solution.  Maybe someone will come up with a great idea.


Prepare release

This PR does not address #8 right? I think we have to take care of this, since this lead to many doubled entries.

No, this will be a separate PR.

Tests are allowed to fail for now. https://github.com/lexibank/yanglalo/pull/9/commits/47827ed730e307f7d612196e408db97626861586 and subsequent are a draft to use the online supplement as discussed in #8 to also fix https://github.com/lexibank/lexibank-analysed/issues/49. Note that this is a very quick first draft for using the other data source and I fully expect cognates etc. not to be working 100% correctly for now.
Things I need to do:

verify the data (particularly in comparison to the old version)
verify cognate sets
verify languages (theoretically, the online supplement version of the dataset includes a substantial amount of additional languages)
answer the question whether the data sheet I'm using (i.e. based on English glosses) is indeed one that can be used or if rather should be 33403_SOURCE4_2_A.cognatesets.csv

No, this will be a separate PR.

Apologies, I just continued working in this PR!

Tests are allowed to fail for now. 47827ed and subsequent are a draft to use the online supplement as discussed in #8 to also fix lexibank/lexibank-analysed#49. Note that this is a very quick first draft for using the other data source and I fully expect cognates etc. not to be working 100% correctly for now.
Things I need to do:
1. verify the data (particularly in comparison to the old version)

2. verify cognate sets

3. verify languages (theoretically, the online supplement version of the dataset includes a substantial amount of additional languages)

4. answer the question whether the data sheet I'm using (i.e. based on English glosses) is indeed one that can be used or if rather should be `33403_SOURCE4_2_A.cognatesets.csv`

After a first spot checking everything looks good with using the online supplement rather than the previous raw TSV. The cognate sets also seem correct to me. @LinguList, this would close #8 (but I'm still going to do some more checks and clean up the XLSX->CSV process). It fixes the issue with the cognates occuring multiple times and leading to doublets (triplets, etc.) in the lexeme list and Lexibank.
Also, the online supplement version of the data contains more language (varieties) than the previous raw data file ((x) denotes that those were present in the previous version of the raw data):

PLa (x)
CW-QY (x)
C-WC
C-LJ
C-LB
C-CJ
C-QS
CE-YA
W-SZP (x)
W- SLZ
W-DT
W-YL
E-DC (x)
E-HS
E-TS
XZ (x)
YL (young) (x, as just YL)
YL (old)
MD (x)
Eka

Should those be included as well or were they excluded for a reason? (not an expert on the languages/studies so there might be a reason I'm not seeing here).

Thank you very much for checking! I'll prepare the updated release for Lexibank 2.0.


Problem

Undefined percentage on three different makes of blind motor. I know one is Zemismart, have I missed Something

The percentage must not be exposed to home assistant - I use zemismart zigbee blinds with no issues

I have recently bought some blinds so have done a bit of research, clearly not enough as I have a similar problem.  I have Somfy RTS blinds and for starters I wouldn't recommend them to anyone thinking of buying them and integrating with HomeAssistant or any other solutions for a number of reasons.
However that aside and back to the problem at hand, Somfy RTS like some other proprietary blind solutions are unidirectional meaning you can only send commands to the blinds (Open, Close, Stop or Custom Position), the blinds do not report their state back to the client/app nor can you open them a percentage.  As such as with my solution the percentage will be undefined as the blind does not provide this or any state.
My thought is the card could maybe have an option to remove the percentage rather than displaying it as undefined.  Also while we are asking maybe you could also include a custom present position button (for somfy this would call the service tahoma.set_cover_my_positon).  If you wanted to be really cleaver maybe just an Estimate state checkox so if you press open it opens 100% and if you select close it closes 0% or however it is supposed to work (how would I know mine just says undefined.)


When running now and you're not logged in, improve messaging

> Email:
> Success! To deploy, run `now` again

I'd say no. Let's just go on with how it works atm and make it deploy the current directly immediately.

https://github.com/zeit/now-cli/commit/af76505c9e02ad79f739e5c729c89c8b1973bcc4


Error on netflix addon startup

I'm submitting a ...

[ ] bug report
[ ] feature request
[x] support request

General information
Addon version used
0.14.6
Description
Error on Kodi start up : "headerdata"
And I can't open netflix addon, I get this error : "AddonSignals call timed out"
Installation

[x] I'm using this Netflix Repo
[ ] I'm using other Netflix Repo (Please specify which)
[ ] I'm using a different source (Please tell which)

Operating System

[ ] Linux (x86/x64)
[ ] OSX (x86/x64)
[ ] Windows (x86/x64)
[ ] Linux (ARM)
[x] Android

Additional informatin on the environment
I'm using a Formuler Z8 box with android 7
Debug log
https://paste.kodi.tv/ekehoropur.kodi

I need more info try this,
install this test and extract a new debug log
plugin.video.netflix_TEST1.zip

Ok installed and the new debug log : https://paste.kodi.tv/ewetudatuj.kodi

I suspect that this device is part of the andoird boxes not completely authorized to use netflix
I believe nothing can be done

ha ok
I didn't pay attention to this... when I buy this box... I'm disappointed.
Thanks for the help

if there is an firmware upgrade to android 8 you can try to do it
it is possible that updating it with more recent android then it works

I checked and this box is only compatible with WIDEVINE L3... not L1... :/
And I can't upgrade to android 8 on this box


[WIP] Initial use of SampleSet in PathMovers

Provides new versions of PathMovers which use SampleSet objects as input, and new version of the bootstrapping calculation which also does the same. My suggestion is to merge this in as soon the tasks below are complete (which should include all aspects in the test suite); other moves/calculation types can be fixed up in future PRs.
This is waiting on PR #97. Work in that is required for this, so we should finish that PR first.
Task List:

[x] Finish #97
[x] Modify ShootingMovers to use SampleSet
[x] Modify bootstrapping calculation to use SampleSet
[x] Include storage of the SampleSet as part of the calculation
[x] Rename MoveDetails attributes (closes #102).
[x] Update ipynbs for new versions of calculations

Extra stuff I included:

New EnsembleHopMove (also to be used in SRTIS)
New BootstrapPromotionMove, which shows how to string several moves together into a more complicated move
Some tests for PathMovers (in progress)

When complete, this closes #90.

I will check this. All the storing needs also tests. Maybe I can find a general way to test all storage at the same time.

Found the error... Actually not really an error. I forgot to add the simplest storage to the list of stores so add
self.sampleset = SampleSetStorage(store).register()

to storage/netcdf_storage.py at line 60 or so. Should be clear, there are more lines like this.
After that is gets stored correctly. However the ncdump seems to have a bug where not all variable that use variable length are shown correctly. If you look for specific variables they are listed correctly though.
ncdump -v sampleset_sample_idx trajectory.nc

or
ncdump -v trajectory_snapshot_idx trajectory.nc

I did not try to fiddle with your branch because this seems faster.

Ready for review and merging!

Looks good to me. I found some notes from myself that seem to be legacy of old stuff. I have to make a sweep of this anyway, so I will merge this now! Good work!


Be cautious about parser node merging

Some passes in p4c frontend will automatically merge 2 parser node into one. However, this could not be taken for granted if we consider the hardware constraints of the backend. For instance, there could be some limit over the # of extractions per parser node or the total extraction bit length per parser node.
We could remain the existing pass in the frontend for now and deal with "HUGE" parser node in later passes by splitting it into multiple ones.

We're going to need a pass to break up overlarge states anyway, so I don't think we need to worry about making states too large early in the process.


fixing dxgw metric collection

Issue #, if available:
DXGW metric collection got VGW methods and therefore wrong metrics are collected for DXGWs.
Description of changes:
Updating DXGW metric collection with DXGW methods.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thank you for your contribution @deltagarrett


Fix ImportError for TensorFlow Lite 2.17.0 on ARM Architecture

This PR resolves an ImportError in TensorFlow Lite 2.17.0 for ARMv7 architecture due to an undefined symbol in the tflite_runtime module.
Changes Made:
Cleaned the build environment.
Adjusted the build process for the ARMv7 target with Python 3.10.
Verified the generated wheel for necessary components.
Conducted import tests using a simple script (simpletest.py).
Issue Reference:
Addresses Issue #75562 regarding import failure.
Testing:
Executed in GitHub Codespaces; confirmed functionality through import tests.
Additional Notes:
Feedback on build configuration and test results is welcome!

Hi @saradha04 , Can you please sign CLA , thank you !

What is the undefined symbol that is being fixed by this change?
(Undefined symbols can sometimes be caused by using the wrong Android min SDK level, so this might be a matter of setting the correct Android min SDK level?)

Hi @saradha04 , Can you please sign CLA , thank you !


sha256sum hashes for every release

Hi,
Each release has the SHA256 hashes of the tarballs, however is there an easy way to get the hashes for the actual binaries without me downloading every release, untarring and hashing them manually? I am developing a tool which uses restic and need a way to determine if the provided binary is indeed a valid restic binary.
Many thanks.

There's currently no way to do that.
What we're doing is a pretty standard approach: Provide a SHA256SUMS file which can be used with sha256sum -c --ignore-missing to verify a downloaded file. Then extract it and use it.
Can you elaborate on your use case? Do you really need the hash for every released version of restic? Why do you need to verify the already downloaded/available restic binary?

I'm writing a utility that calls the restic binary and wanted to ensure it was genuine. It's really to protect people from themselves. Just doing "restic version" and parsing the text seems a bit too hacky.

I can't think of an easy way to do that with our current workflow, so I'm going to close this issue for now. Please feel free to add further comments. Thanks!


Make fuzzing work.

Fixes a Cargo error and adds a new make target, fuzz.
What have you changed?
There is a bit of complexity passing the right features to dependencies when fuzzing. This PR fixes that by adding dependencies to fuzz-targets. I also add a make target to make fuzzing more convenient.
What is the type of the changes?

Bugfix (a change which fixes an issue)
Engineering (engineering change which doesn't change any feature or fix any issue)

How is the PR tested?

Manual test (add detailed scripts or steps below)

make test and FUZZ_TARGET=fuzz_codec_bytes make test
Does this PR affect documentation (docs) or should it be mentioned in the release notes?
no
Does this PR affect tidb-ansible?
no
Refer to a related PR or issue link (optional)
Fixes #6591
PTAL @zhouqiang-cl @breeswish

LGTM. However CI failed :(

/test

The afl fuzz can work now!!!

@breeswish @zhouqiang-cl PTAL. Passing CI now, but the Makefile got a bit more complicated

@overvenus @breeswish PTAL again

/merge

/merge


Adds correct __repr__ to PyBuiltinMethod

Closes #2775

Thank you for contributing!

Thanks to @youknowone and @DimitrisJim for your quick and quality reviews! 🙂


compact block filter post

Cross-post of my compact block filter deep dive blog post

I'd like to highlight that this is a cross-post a bit more. Credit where credit is due. What do you think about an 'originally published on: ellemouton.com' somewhere here?

Oh lol, sorry about the wrapping 🙈  fixed to wrap at 80 chars 👍

What do you think about an 'originally published on: ellemouton.com' somewhere here?

cool, added that to the subtitle... not sure where else to put it? (is there someway to make it a link there rather? normal md linktags dont seem to work in the metadata part.)

cool, added that to the subtitle... not sure where else to put it? (is there someway to make it a link there rather? normal md linktags dont seem to work in the metadata part.)

with e4c2a21 you now can include
appearedfirston:
    label: "ellemouton.com"
    url: "https://ellemouton.com/posts/bip158/"

in the front matter of your post

awesome! updated 👍

Cool! Thanks for catching all those mistakes :) updated 🚀


cannot compile for new release

i have installed the trainer and everything is there, except for trainer.ini

Please show us the installer output.
Going forward we will only support installations using spt-aki. So if you try to compile against a vanilla EFT installation that's now expected to hit failures.


Networking part 2 electric boogaloo

mainly PlayerManager and EntityTracker

this is a draft review after its not

its not a draft now yay

still 1 change requested

still 1 change requested

no

i will close it if not fix


Add function for custom element registration

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Resolves: # (issue, if applicable)
Summary:
Please summarize the scope of the changes you have submitted, what the intent of the work is and anything that describes the before/after state of the project.
Add function that adds attribute to element when the component is custom registered
Type of change:
Please delete options that are not relevant.

[x] New capability
[ ] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

:tada: This PR is included in version 2.1.0-beta.2 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Appstudio update test-annotations-opmr

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-onzg-on-pull-request-
creating pipelinerun test-simple-build-onzg-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-yjrl-on-pull-request-
creating pipelinerun test-annotations-yjrl-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-nqsa-on-pull-request-
creating pipelinerun test-annotations-nqsa-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-kgek-on-pull-request-
creating pipelinerun test-simple-build-kgek-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-gvay-on-pull-request-
creating pipelinerun test-simple-build-gvay-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-bxtt-on-pull-request-
creating pipelinerun test-simple-build-bxtt-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-xtci-on-pull-request-
creating pipelinerun test-annotations-xtci-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-xosv-on-pull-request-
creating pipelinerun test-simple-build-xosv-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-wgvq-on-pull-request-
creating pipelinerun test-simple-build-wgvq-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-fglt-on-pull-request-
creating pipelinerun test-simple-build-fglt-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-fohw-on-pull-request-
creating pipelinerun test-annotations-fohw-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-eukh-on-pull-request-
creating pipelinerun test-annotations-eukh-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-cvzf-on-pull-request-
creating pipelinerun test-simple-build-cvzf-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-xaot-on-pull-request-
creating pipelinerun test-annotations-xaot-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-efea-on-pull-request-
creating pipelinerun test-simple-build-efea-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-gsnx-on-pull-request-
creating pipelinerun test-simple-build-gsnx-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-taux-on-pull-request-
creating pipelinerun test-simple-build-taux-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-igaw-on-pull-request-
creating pipelinerun test-simple-build-igaw-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-qlwi-on-pull-request-
creating pipelinerun test-annotations-qlwi-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-gjnx-on-pull-request-
creating pipelinerun test-annotations-gjnx-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-izee-on-pull-request-
creating pipelinerun test-simple-build-izee-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-xlnr-on-pull-request-
creating pipelinerun test-annotations-xlnr-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-owhs-on-pull-request-
creating pipelinerun test-simple-build-owhs-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-kvjs-on-pull-request-
creating pipelinerun test-simple-build-kvjs-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-htcz-on-pull-request-
creating pipelinerun test-annotations-htcz-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-qfje-on-pull-request-
creating pipelinerun test-simple-build-qfje-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-ckox-on-pull-request-
creating pipelinerun test-simple-build-ckox-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-qezd-on-pull-request-
creating pipelinerun test-simple-build-qezd-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-wznz-on-pull-request-
creating pipelinerun test-simple-build-wznz-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-nycf-on-pull-request-
creating pipelinerun test-simple-build-nycf-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-wfdk-on-pull-request-
creating pipelinerun test-simple-build-wfdk-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-bkgt-on-pull-request-
creating pipelinerun test-annotations-bkgt-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-pzcq-on-pull-request-
creating pipelinerun test-annotations-pzcq-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-ylok-on-pull-request-
creating pipelinerun test-simple-build-ylok-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-mbrb-on-pull-request-
creating pipelinerun test-simple-build-mbrb-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-izai-on-pull-request-
creating pipelinerun test-annotations-izai-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-qvet-on-pull-request-
creating pipelinerun test-annotations-qvet-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-jgvv-on-pull-request-
creating pipelinerun test-annotations-jgvv-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-yoht-on-pull-request-
creating pipelinerun test-annotations-yoht-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-knyw-on-pull-request-
creating pipelinerun test-simple-build-knyw-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-lbhf-on-pull-request-
creating pipelinerun test-annotations-lbhf-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-uwwa-on-pull-request-
creating pipelinerun test-annotations-uwwa-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-jihl-on-pull-request-
creating pipelinerun test-simple-build-jihl-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-irii-on-pull-request-
creating pipelinerun test-simple-build-irii-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-pmki-on-pull-request-
creating pipelinerun test-simple-build-pmki-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-phhr-on-pull-request-
creating pipelinerun test-simple-build-phhr-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-ygyv-on-pull-request-
creating pipelinerun test-simple-build-ygyv-on-pull-request- in namespace build-e2e-puxm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI/test-annotations-opmr-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
24 seconds
init

✅ Succeeded
33 seconds
clone-repository

✅ Succeeded
26 seconds
sast-snyk-check

✅ Succeeded
1 minute
build-container

✅ Succeeded
9 seconds
clair-scan

✅ Succeeded
26 seconds
deprecated-base-image-check

✅ Succeeded
59 seconds
clamav-scan

✅ Succeeded
23 seconds
inspect-image

✅ Succeeded
9 seconds
sbom-json-check

✅ Succeeded
6 seconds
label-check

✅ Succeeded
9 seconds
show-summary

✅ Succeeded
10 seconds
show-sbom


Support bulk read

Codecov Report
Attention: Patch coverage is 91.78082% with 12 lines in your changes missing coverage. Please review.

Project coverage is 96.65%. Comparing base (4c46bc7) to head (8496d19).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #300      +/-   ##
==========================================
- Coverage   96.73%   96.65%   -0.08%
==========================================
  Files         151      151
  Lines        8991     9107     +116
==========================================
+ Hits         8697     8802     +105
- Misses        294      305      +11

Files
Coverage Δ

include/async_mqtt/client.hpp
100.00% <ø> (ø)

include/async_mqtt/endpoint.hpp
100.00% <100.00%> (ø)

include/async_mqtt/impl/client_impl.hpp
100.00% <ø> (ø)

include/async_mqtt/impl/endpoint_recv.hpp
98.50% <ø> (-0.03%)
:arrow_down:

...clude/async_mqtt/util/impl/stream_write_packet.hpp
83.03% <100.00%> (ø)

include/async_mqtt/util/stream_traits.hpp
100.00% <ø> (ø)

include/async_mqtt/util/stream.hpp
91.89% <81.81%> (-4.41%)
:arrow_down:

...nclude/async_mqtt/util/impl/stream_read_packet.hpp
94.17% <91.93%> (-2.60%)
:arrow_down:

... and 1 file with indirect coverage changes