Bio Weekly - Home Mouthwash Collection Metrics

Please generate the six new tables and one new figure for the home mouthwash collection metrics. These new metrics will be added to the weekly biospecimen metrics report following Table 26; the mock-up tables are numbered to reflect this.
Requestor(s): Domonique
Table or Figure title: See mock-up for titles.
Metrics to be updated or created: Create new metrics
Variables needed (include variable name and Concept ID):

RcrtES_Site_v1r0 (827220437) – Sites
BioSpm_Setting_v1r0 (650516960) – Collection setting
BioRec_CollectFinal_v1r0 (410912345) - Complete Collection
BioRec_CollectFinalTime_v1r0 (556788178) – Collection date/time
BioFin_BaseMouthCol_v1r0 (684635302) – Baseline mouthwash collection
BioKit_KitType_v1r0 (379252329) – Kit Type
BioKit_KitStatus_v1r0 (221592017) – Kit status
BioKit_KitShipTm_v1r0  (661940160) – Date/time kit shipped
BioKit_KitRecdTm_v1r0 (826941471) – Date/time kit received
BioKit_CollCardRet_v1r0 (137401245) – Collection card returned flag
(Source Question) Select Package Condition - Home Collections (633640710)

Specify numerator and denominator by variable name where applicable: See mock-up for denominator and numerator for each table (below each table in bold)
Mock-up table if applicable:
Mock-Up_HC-MW Metrics_102323.docx
Notes: N/A
Approved by:

@HopkinsDC  Quick question, without data I can try writing a code for these tables but won't know if they worked until I have actual data. I may try this in stage if we have home collection data there for these variables. Are we still on the DEV phase?

@KELSEYDOWLING7 Yes, we are still in the dev phase. We will be testing home collections in dev today and tomorrow, so there should be data in dev to test the code on next week, but until then, you're fine to just draft the code for now. I'll add a comment here when we finish our initial testing in dev so you do a test run.

@HopkinsDC  Ok Thanks! The structure of DEV won't work for the code but I'll just draft the code for now like you said

@HopkinsDC  For Table 28, should kit_status="Pending" be a column as well, or should I exclude those or put them in with no status?

@KELSEYDOWLING7 Kit status is present in both the Kit Table and the Participants Table. "Pending" will only exist in the Kit Table for kits that have been created but have not yet been linked to a participant. "Address Printed" only exists in the Participants Table for when the participant's address is printed in the BPTL dashboard; the kit status is NULL until then. Once linked, the kit status in both tables is updated to "Assign" and will update to "Shipped" and "Received" simultaneously as these events occur.
Here's the home mouthwash collection data structure for your reference: https://nih.app.box.com/file/1306274192476

@HopkinsDC Ok so there shouldn't be any participants in Table 28 that should have a status of pending? Or do you want that column added?
Also here in the mock up you have a column called "Two of More Conditions Selected" but then ask to have it excluded below. Just want to confirm whether it should be in there or not.

@KELSEYDOWLING7 No, don't add a Pending column; participants will not have a Pending status.

@HopkinsDC  Got it, thanks. This code should be ready to test once we have data in prod

@KELSEYDOWLING7 Hi Kelsey, home collections will be in stage on 11/27 or 11/28. Can the home collection metrics code be tested in stage?

@HopkinsDC  If all the variables needed for the tables are in the biospecimen or participants table in BQ, then yes.
Right now these variables aren't in there (which may make sense for now, but would just need to be answered to test)
-137401245
-633640710
-950521660
-545319575
-938338155
-205954477
-289239334
-992420392
-541085383

@KELSEYDOWLING7 A new table was created for the home collections (the Kit Table), which is where the variables listed above, and other variables for the tables, are. These Kit Table variables will have values in stage after we test on 11/28.
Will you be able to test these at that time?

@HopkinsDC  Oh that's right, I saw Nicole mention the Kit table to Jake. As long as that table has been flattened in stage and those questions answers, yes I would be able to test these at that time

@KELSEYDOWLING7 Awesome, thank you! I'll follow-up here when the stage testing is done so you can test.

@KELSEYDOWLING7 Testing in stage is complete for the most part, so you can try out your home collection metrics code now.

@KELSEYDOWLING7 The tables you were able to run look fine except for Table 27. The columns are mislabeled; can you update those to the labels I have in the mock-up? Also, I think I used the wrong CID for Table 27's numerator. Please update the table using the following denominator and numerator below:
Denominator = Complete clinical collections (650516960 = 1; 410912345 = 1)
Numerator = Mouthwash collected in home setting (915179629 = 2 and 684635302 = 1) - If 915179629 or 684635302 is NULL, the record should be included in the "No Home Mouthwash Collected" column count.

@HopkinsDC  Thank you for pointing that out! There was a filter missing in Table 27. But I think the denominator would need to be 650516960 = 2 (Collection setting=home) instead of 650516960 = 2 (Collection setting=research) right?

@KELSEYDOWLING7 It'll be better to look at the mouthwash collection setting specifically (BioSpm_MWSettingBL_v1r0 - 915179629=2 [Home]) for Table 27's denominator.

So the problem is that we have 8 values for d_143615646_d_826941471, but for all 8 d_410912345 (BioRec_CollectFinal_v1r0) is nulll.... and that's a filter when I first pull the data in from BQ.
As for table 32 that was my mistake, the responses aren't nested the way I thought they would be so the code was off

@KELSEYDOWLING7 Table 32 looks good, but can you add a "Total" row to it?
That's weird that there's a filter for stage data; can you remove it, or is that filter for all data pulled from BQ?

@HopkinsDC  Yes, here's the updated table:

And that's the filter we decided to add to all the biospecimen data a while back, so I'd have to do a separate BQ pull if we want only this data to include non-finalized collections

@KELSEYDOWLING7 The row titles are showing as 1, 2, 3, and 4 now instead of the condition names; can you fix that?
Okay, in that case, don't worry about testing Tables 30 and 31 in stage. You can just have the code ready for when we have production data and check them then; we'll make adjustments if needed. Thanks!

@KELSEYDOWLING7 Hi Kelsey, please make the following changes to the home mouthwash collection metrics:
Table 27

Change title "Table 27. Baseline Home Mouthwash Collection Status Among Clinical Collections by Site" to --> Table 7.1: Baseline Home Mouthwash Collection Status Among Eligible Participants
Change the last column's label to "Total Eligible Participants"
Add the following footnote --> Note: “Eligible participants” are those who had a clinical collection with blood or urine collected on or after January 1, 2024, and did not refuse, withdraw, or pass away prior to January 2024. Home mouthwash collections were launched for eligible participants in January 2024. Home mouthwash collections for backlog collections (before January 2024) will be launched at a later date.
Updated the denominator of this table to be the same as Table 28 [Table 7.2] below.

Table 28

Change title "Table 28. Home Mouthwash Collection Kit Status Among Eligible Participants by Site" to --> Table 7.2: Baseline Home Mouthwash Kit Status Among Eligible Participants
Add the following footnote --> Note: “Eligible participants” are those who had a clinical collection with blood or urine collected on or after January 1, 2024, and did not refuse, withdraw, or pass away prior to January 2024. Home mouthwash collections were launched for eligible participants in January 2024. Home mouthwash collections for backlog collections (before January 2024) will be launched at a later date.
Updated the denominator of the table to be

Table 29

Change title "Table 29. Collection Card Status Among Retuned Home Mouthwash Collection Kits by Site" to --> Table 7.3: Collection Card Status Among Received Baseline Home Mouthwash Kits

Table 30

Change title "Table 30. Time (in Hours) from Home Mouthwash Collection Kit Shipment to BPTL Receipt by Site " to --> Table 7.4: Time (in Days) from Baseline Home Mouthwash Kit Shipment to BPTL Receipt
Update table calculation to be in Days, not in hours

Table 31

Change title "Table 31. Time (in Hours) from Clinical Biospecimen Collection to BPTL Home Mouthwash Collection Kit Shipment by Site " to --> Table 7.5: Time (in Days) from Baseline Clinical Collection to Home Mouthwash Kit Shipment
Update table calculation to be in Days, not in hours
Change the collection date/time variable used in the calculation to BioCol_ColTime_v1r0 (678166505)

Table 32

Change title "Table 32. Home Mouthwash Collection Kit Condition" to --> Table 7.6: Kit Conditions Among Received Baseline Home Mouthwash Kits

Figure 5

Change title "Figure 5. Weekly Shipped and Received Home Mouthwash Collection Kits" to --> Figure 7.1: Weekly Shipped and Received Baseline Home Mouthwash Kits
Change the y-axis title to "Number of Kits"
Replace the underscore with a space in the legend title "Kit_Status"

@HopkinsDC  For table 27, should the date of collection be BioCol_ColTime_v1r0 (678166505)?

@KELSEYDOWLING7 Yes, please use BioCol_ColTime_v1r0 (678166505) for the date of collection, and yes, keep both and add the other variables you listed. As a note for the other variables being added.

@HopkinsDC  Ok thanks. Unfortunately we have 0 participants that fit that description in Stage so I'll have to wait for more data

@KELSEYDOWLING7 That's fine. I doubt that part was included in the stage testing for home collections anyway. Thank you!

@KELSEYDOWLING7 , Michelle and I will review this so you can hold for a bit on this

@KELSEYDOWLING7 We are looking at the Weekly Biospecimen Metrics Report. Two home mouthwash collection kits were shipped on Friday, we expected to see the Home Mouthwash data in the report today. None have been received yet, but we should see the sent status for the first two kits. Do you know why this isn't showing up?
Also, home mouthwash collection was delayed please adjust the footnotes for Table 7.1 and 7.2 to reflect April launch (not January):
Note: “Eligible participants” are those who had a clinical collection with blood or urine collected on or after April 1, 2024, and did not refuse, withdraw, or pass away prior to April 2024. Home mouthwash collections were launched for eligible participants in April 2024. Home mouthwash collections for backlog collections (before April 2024) will be launched at a later date.
Also, please update titles for Table 7.3-7.6 and Figure 7.1 to reflect new title convention Baseline-[rest of title]

@erincschwartz  Hi Erin, yes I haven't put any of the MW code into the report yet because not all the variables were in prod on Friday. If any were missing it would cause the whole report to crash and it looks like we're still waiting on many of them to be populated. I believe Michelle mentioned on the Punch List call that we should be getting about 20 kits back tomorrow. That should hopefully give us a majority of them if not all.
As far as table 7.1 and 7.2 and figure 7.2 are you referring to these on the markup?

If so then yes I will update those now with the footnotes and title change. Once we have all the variables I'll rerun the report and post that section here for aproval before updating the automation. If we have all the variables by the end of the week, do you need me to update the report this Friday for next Monday's report as opposed to the next automation update scheduled for Apritl 19th for the April 22nd report?

@KELSEYDOWLING7 Yes, I was referring to Table 27 & 28 shown above, just used the new naming convention. To clarify, 2 kits were sent Friday; we expect an additional 18 kits to be sent today, for total of 20 sent out. We do not know how quickly any of the kits will be returned. That said, if we have kits returned and all the data available, can you push it through this Friday? If it is a huge burden to do this out of cycle, I can talk to Bio Team about waiting. I know we are anxious to start seeing the data!

@erincschwartz  Yes, if we have all the variables needed then I can talk to Jake about updating the automation on Friday, or running it manually on Monday

@KELSEYDOWLING7 Kit status is present in both the Kit Table and the Participants Table. "Pending" will only exist in the Kit Table for kits that have been created but have not yet been linked to a participant. "Address Printed" only exists in the Participants Table for when the participant's address is printed in the BPTL dashboard; the kit status is NULL until then. Once linked, the kit status in both tables is updated to "Assign" and will update to "Shipped" and "Received" simultaneously as these events occur.
Here's the home mouthwash collection data structure for your reference: https://nih.app.box.com/file/1306274192476

@KELSEYDOWLING7 Just pointing out that Madhuri updated the home mouthwash collection data structure document, linked above. The only variable she updated was "BioKit_Mouthwash" which I do not believe your reports are using but wanted to let you know in case.

A few of the variables I need did have a nesting structure change so thank you for confirming!

@erincschwartz  It's still too early for a majority of these tables and Figures to run.
So for the first, second, and third HMW, that collection we did receive back but this variable BioCol_ColTime_v1r0 is null for that person so those tables won't run. Joe's email mentioned that he can see a value for that variable, but the value wasn't entered correctly. It's a date instead of a timestamp I'm assuming that's why BQ isn't handling it correctly.
@jacobmpeters this variable d_678166505 has two rows with different values for the same participant  in the raw table, one that's null and one is a date variable instead of datetime. Can we flatten that as is and add T00:00:00.000Z or does that have to be fixed upstream?
BioKit_KitRecdTmBL_v1r0 is null for everyone so the fourth and fifth table won't run.
None of the package conditions were filled out so the sixth table won't run.
BioKit_CollCardMissing_v1r0 is null so the last table won't run.

@jacobmpeters  Of course!

@erincschwartz  A few tables were able to be run this morning! Going to be working with Jake on the duplicate rows for the BioCol_ColTime_v1r0 that is throwing some of the numbers off. But let me know if anything else stands out that needs to be changed. I updated the footnotes for Table 7.1, Table 7.2, and Figure 7.1 as requested above.

@KELSEYDOWLING7 Please see notes on the updates we discussed on our call. I received confirmation on Table 7.5 from Bio Team, note included below:
Table 7.1 - Update title to: Baseline- Home Mouthwash Kit Status Among Eligible Participants
- Change columns to:    Total Eligible-- Total Kits Shipped -- Total Kits Received
Table 7.2 - Remove this table
Table 7.3-  Update title: Baseline- Collection Card Status Among Home Mouthwash Kits Received
Table 7.4 - Add  "N" between site name and min
Table 7.5 - Add  "N" between site name and min
- Clinical Collection is date of clinical blood or urine collection, whichever occurred first
Table 7.7 - Update title to:  Baseline- Mouthwash Biospecimen Survey Status Among Mouthwash Kits in 'Shipped' Status
- Update "Total Shipped Kits" column header to "Total Kits in 'Shipped' Status"
Figure 7.1 Update start date on X axis
- Make sure this is including all kits ever shipped that week, not just those in 'shipped' status
Table 7.9- Change title to: Baseline- Mouthwash Biospecimen Survey Completion Time Relative to Collection Time Reported on Collection Card
- Change Before Collection column header to:  "1+ Day(s) before Collection"
- Add footer-  Note: Only includes data from mouthwash kits returned with completed Collection Card

@erincschwartz  For consistency it may make more sense for Table 7.1 columns to be total shipped, total received, and THEN total eligible. In all the other tables we have the column with the denominator on the far right as reference, unless we want eligible to mean eligible but not yet shipped or received. Also for that table I would need the conditionality for what makes a participant eligible.

Also do you know if something like this might be acceptable for Figure 7.1?

@KELSEYDOWLING7 Confirming my understanding, is the dashed line a cumulative total for all weeks?

The dashed is the weekly total of Shipped (Shipped+Received). Although now that I write that, that assumes that the receiving date and shipped date are the same week each week.... this may be more complicated then I thought. Please disregard the figure.

@erincschwartz  Sorry for all the back and forth! I take that back. I was able to use the weekly shipped date counts as we discussed to verify the "Total Shipped" is the total number of kits shipped per week, regardless of whether they were received that week. I overlaid that line on the existing plot that just looks at counts of Kit Statuses and the plot above is correct, if that style is acceptable.
I can also remove the "Shipped" counts if its confusing. I'm struggling to get the "Total Shipped" line to also have the dots on the week, which is why I left "Shipped" on there for now as a reference. Please let me know if that makes sense or if you have any questions

@erincschwartz  Thanks! I'll update the title numbers now that Table 7.2 is removed, but I didn't want to make it confusing for this review. Also, the current conditionality for Figure 7.1 (kit type=mouthwash) is preventing any counts of address printed from showing. Not sure if that's a problem

@erincschwartz  You're right, thank you for pointing that out. there was a typo in one of the table merges. Please let me know how this looks.

Two participants aren't counted in that table that did have a MW kit shipped to them because their first blood or urine collection time was after kit shipment (same day but later that day): Connect IDs<PHONE_NUMBER> and<PHONE_NUMBER>. Please let me know if you want the table updated to include times of 0

@KELSEYDOWLING7 I will take the comment above back to the Bio team for those two cases, Connect IDs<PHONE_NUMBER> and<PHONE_NUMBER>----the sort order for printing labels/assigning kits was just updated in the Prod push, so kits will likely not ever be shipped the same day as blood/urine collection.
Table 7.6 is showing that 5 kits were receipted with "no conditions selected," however, we just confirmed in the Dash that FNL must select package condition before receipting the kit in the Dash. Can you look into this?
Also, still working to perfect Figure 7.1----seeing more data in there now makes us think we do not actually want the "shipped" count showing. We just want to see absolute values: how many kits were shipped out from FNL in a given week and how many kits in total were received in a given week. With removal of this line, we can convert the dashed line for Total Shipped into a solid line, yellow works.
Please clarify your note above: Also, the current conditionality for Figure 7.1 (kit type=mouthwash) is preventing any counts of address printed from showing. Not sure if that's a problem. Address printed should not factor into this Figure, so I am not sure what this means.
Thanks!

@erincschwartz  7.6 was linked to the delayed kit_assembly flattening. That's now fixed.

Table 7.1- shipped status removed. If counts for address printed aren't needed here, please disregard the comment. Please also disregard the b in the table title, I was playing around with different versions

@KELSEYDOWLING7 Thanks for fixing 7.6!
Figure 7.1 looks off now, the first week 2 were shipped, the second week 18 were shipped, third week 100 were shipped. Also, forgot to mention earlier, can we start at 3/25, to get rid of even more blank space along the x-axis? And, can you eliminate showing the data for the current week, since it is not complete? And, if run on Monday morning, will likely always show a plummet to 0?

@erincschwartz  I'll look into the counts for 7.1, specifically for 4/1, and have updated the x axis. The dates are counting the Monday prior until the day before, which when run, would be Sunday so there will always be full counts. For example Monday will have 4/29/24 on the x-axis but will have the full count of 4/22-4/28 and would never plummet to 0 unless there were no counts or if it was run mid-week before anything was shipped.

@erincschwartz

@KELSEYDOWLING7  Thank you!
For Table 7.9, please add columns for:  2 Days before Collection, 3 Days before Collection, and 4+ Days before Collection
We were originally concerned about people completing the survey much after the collection but now we realize we need more insight into numbers of participants who are completing it well before actually completing the collection.

@erincschwartz  Sure thing! I added 1 day before collection as well to account for all scenarios

Hi Kelsey,
Thank you, that is very helpful.  It is showing us that a large percentage of participants are filling out the mouthwash survey too soon, which is something we need to address.
Thank you!
Stephanie

@KELSEYDOWLING7 We noticed a discrepancy in the KPCO data. In table 7.1 above, it shows that 5 kits have been returned; Tables 7.3 and 7.4 seem to show that 7 kits have been returned. Can you look into this?
Also, can you add 1 new table and 2 new figures:
New Table
Title:  Table 7.2 Baseline- Home Mouthwash Kit Return Rate
*This table will mimic Table 7.1 but will only have 4 columns:  Site,  Shipped (this would be total ever shipped);  Received; Total
*Can this data have a 1 week lag to account for lag time to get to the participant, complete collection and then ship the sample back to FNL? Footnote: Shipped data are current through [date 1 week ago].
New Figures
Title: Figure 7.2 Baseline- Home Mouthwash Kit Return Rate

Kits shipped would be the blue fill part and the kits received could be the line; left Y axis= # Kits Shipped; right Y axis Cumulative Return Rate; X axis = Date
Footnote: Shipped data are current through [date 1 week ago].
Title: Figure 7.3

Y axis=  Number of Kits Shipped
X axis = Site
Coral bar= Total Kits Shipped
Turquoise bar= Total Kits Returned

@erincschwartz  Thank you for letting me know.
@brotzmanmj  The counts all seem to be in order now. Please let me know if/when the team decides on how they want to lag Table 7.2 (with received lagged as well or not). For now I have not lagged either. I also recommend changing the column name from 'Shipped' to 'Ever Shipped' for clarification. Lastly, the 'Total' in that table really has no meaning, as we're counting those ever shipped (including received) as well as received.
For Figure 7.2, please note that a second y axis is very hard to control. It will only take up a very small portion of the bottom right axis, so that line may never really look all that different. Meaning that I could have the second y axis from 0-30% or 0-50% but I can't get the ticks to stretch further up the right y axis, only increase the numbers on the tick line. It will not really be a proportional line. I think it worked significantly better in the operations report only because it was less than 10%. Here we have values of 30%+ but it looks the same as the 3% at first glance. Instead, I've made a slight adjusted to the request. Please let me know your thoughts on this.

Please note their are 13 received kit that are excluded based on the criteria requested. Meaning BL D/T Clin Blood Collection or BL D/T Clin Urine Collection were collected before 4/1/24 and we've received their kits

@brotzmanmj  Updated Table 7.2 as requested

I also realized why the line in 7.2 looked much lower than we were expecting. The 24% was mapping to the number 24 on the y axis. Instead I made this second mountain the cumulative returns as a count, and put the percentage as the label.

This looks great, thanks

Great! Is it set to close the issue?


JOSE: Versioning

The GitHub page indicates that this repository is currently under construction for the 2021 course (with a link to the 2020 archival materials in the README), but given that this paper is describing materials from the 2020 course then perhaps the associated release for this work should be the NMA2020 release (https://github.com/NeuromatchAcademy/course-content/releases/tag/NMA2020)? Currently the JOSE system appears to be interpreting the release version as “1” which I think this incorrect.
This is reviewer feedback associated with submission to JOSE: https://github.com/openjournals/jose-reviews/issues/118

The group has decided to make it about the 2021 version (with many other consequences) so I am now looking at how to get JOSE / Whedon to recognize a specific version.


resolve NaN on Map and Legend

resolved issue #112

Map legend showing max 1 when there is value 0 or in exponential form.
Removed NaN to 0. As legend was not able to encode NaN to any color. NaN was due to exponential values between 0 to 1.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 5 (0.0%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage remained the same at 20.749%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

frontend/src/components/Sidebar/DistrictMap.tsx
0
5
0.0%

Files with Coverage Reduction
New Missed Lines
%

frontend/src/components/Sidebar/DistrictMap.tsx
2
0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
231

Relevant Lines:
900

💛  - Coveralls

Faded districts are showing up now.
Max value for legend is showing up 1 where in actual it is fractions of 0 or 0.


[WIP] IME and Dead Key support

Issue: Oni today does not support 'dead keys' or IME on all platforms, and this is a blocker for several users using non-US keyboard layouts.
Defect: The browser (and Electron by extension) does support dead keys and IME out of the box - for form fields, text areas, etc. However, Oni's current input strategy just tacks a keydown handler on the document body. This is sufficient for US keyboard layouts, but for IME / dead-keys, this isn't handled correctly on all platforms.
Fix: Use a hidden input field to support the IME and dead-key scenario. In non-IME/dead key cases, the input will pass-through as it does today (resolving via the strategy in Keyboard.ts). When we detect dead key or IME composition, the text field will become visible, to give feedback on the composition.
Once completed, this would address a class of the issues around keyboard input - like #542, #396, and #638

I am confused as to why this needs careful treatment, but maybe it's because I'm not familiar with electron and how it implements the key* events.
Is it possible that keypress could be a better fit?
Also, I found these events that may be helpful to avoid having to store the composition state:

https://developer.mozilla.org/en-US/docs/Web/Events/compositionstart
https://developer.mozilla.org/en-US/docs/Web/Events/compositionupdate
https://developer.mozilla.org/en-US/docs/Web/Events/compositionend

This my daily editor today so I'm interested in helping to solve these daily editor issues, unfortunately my time is very constrained these days.

Excited to see this being worked on, thanks!


Using ament_target_dependencies when possible

See ros2/ros2#658

CI:

Linux
Linux-aarch64
macOS
Windows

This is a follow up from https://github.com/ros/class_loader/pull/121

Linux CI randomly failed while running tf2_sensor_msgs tests. The others CI jobs passed or had known failures. Linux CI again:

Linux

@nuclearsandwich If you approve this PR, please also merge it as I don't have write access in this repo.


Feature to "fix" filtering on  multi-valued dimensions

Currently, if you have a row in druid that has a multi-valued dimension with values ["v1", "v2", "v3"] and you send a query grouping by that dimension with filter for value "v1". In the response you will get rows containing "v2" and "v3" as well.
For more details see the UT MultiValuedDimensionTest.testGroupByWithDimFilter() in this PR.
This PR introduces the feature to add regex or list-of-string-values filters in new DimensionSpec implementations which can be used to do proper filtering on multi-valued dimensions so that unwanted rows are discarded as early in processing pipeline as possible.
Not having this ability causes unnecessary major resource utilization across brokers and historicals where result set would sometime contain 90% unwanted rows.

@himanshug can you comment on the operational difference between the RegexFilteredDimensionSpec and an extraction filter using regex?

@drcrallen filter in DimensionSpec only make sense for multi-valued dimensions. Existing [Regex]Dim filter is used to select row from segment. one row would explode into multiple rows based on multivalued dimension. Then DimensionSpec filter would apply on these exploded rows and select from them.
In many cases the regex pattern used might be same in both places but it can be different as well depending upon the use case.

@cheddar review comment addressed.

@fjy updated the doc, see if it is better now?

m adding some more examples to doc.

I'm :+1:, I find the docs easier to read and understand now as well.  @fjy ?


bst cache fails to authenticate due to SSL

See original issue on GitLab
In GitLab by [Gitlab user @adds68] on Jul 31, 2018, 17:08
Summary
When running bst build, our CI runners try to contact our cache server for push/pull, but receive: "Failed to initialize remote https://testcache.codethink.co.uk:11002: Method not found!" as a response.
Steps to reproduce
The CI/cache server are configured to use 1.1.4 and use the same configuration that is found here: https://buildstream.gitlab.io/buildstream/install_artifacts.html?highlight=cache server#configuring-buildstream-to-use-remote-caches
What is the current bug behavior?
The build fails to pull/push anything from the cache server
What is the expected correct behavior?
Buildstream to pull elements from the cache server and push after new things have been built
Relevant logs and/or screenshots
https://gitlab.com/freedesktop-sdk/freedesktop-sdk/-/jobs/85713745
Possible fixes
Other relevant information

BuildStream version affected: /milestone %BuildStream_v1.x

In GitLab by [Gitlab user @adds68] on Jul 31, 2018, 20:39
After some further investigation, i decided to shut down the bst push/pull instance and nginx running on our server. To make sure i could isolate the issue to one service (all managed via systemd).
[Unit]
Description=Buildstream Artifact pull server
After=remote-fs.target network-online.target

[Service]
Environment="LC_ALL=C.UTF-8"
ExecStart=/usr/local/bin/bst-artifact-server --port 11001 --server-key {{certs_path}}{{ssl_url}}/privkey.pem --server-cert {{certs_path}}{{ssl_url}}/fullchain.pem /home/artifacts/artifacts/
User=artifacts

[Install]
WantedBy=multi-user.target

We currently use certificates provided via Lets Encrypt, which i thought could be the issue in the first place, so this lead me to head back to the documentation and follow the steps from scratch.
I generated a new self signed key and modified the systemd service file to use these (owned by the artifacts user), i then copied .crt to my local machine and build against the new service.
On my client i received:
[--:--:--][][] START   Initializing remote caches
E0731 20:21:00.483169370   14501 ssl_transport_security.cc:1063] Handshake failed with fatal error SSL_ERROR_SSL: error:1000
007d:SSL routines:OPENSSL_internal:CERTIFICATE_VERIFY_FAILED.
[--:--:--][][] WARNING Failed to initialize remote https://testcache.codethink.co.uk:11001: Connect Failed
[00:00:00][][] SUCCESS Initializing remote caches
[--:--:--][][] START   Resolving cached state
[00:00:01][][] SUCCESS Resolving cached state

But on the server i do not get any logs from journalctl at all.
However if a request is sent via the browser, i can see:
4175 secure_endpoint.cc:181]     Decryption error: TSI_DATA_CORRUPTED
4175 ssl_transport_security.cc:453] Corruption detected.
4175 ssl_transport_security.cc:429] error:10000412:SSL routines:OPENSSL_internal:SSLV3_ALERT_BAD_CERTIFICATE
4175 secure_endpoint.cc:181]     Decryption error: TSI_DATA_CORRUPTED
4175 ssl_transport_security.cc:453] Corruption detected.
4175 ssl_transport_security.cc:429] error:10000412:SSL routines:OPENSSL_internal:SSLV3_ALERT_BAD_CERTIFICATE
4175 secure_endpoint.cc:181]     Decryption error: TSI_DATA_CORRUPTED
4175 ssl_transport_security.cc:453] Corruption detected.
4175 ssl_transport_security.cc:429] error:10000412:SSL routines:OPENSSL_internal:SSLV3_ALERT_BAD_CERTIFICATE
4175 secure_endpoint.cc:181]     Decryption error: TSI_DATA_CORRUPTED

I have also made sure i am using the latest 1.1.4 version of bst also!

In GitLab by [Gitlab user @adds68] on Jul 31, 2018, 20:42
changed title from bst cache {-responds: "Failed to initialize remote https://testcache.codethink.co.uk:11002: Method not found!"-} to bst cache {+fails to authenticate due to SSL+}

In GitLab by [Gitlab user @jjardon] on Aug 2, 2018, 18:11
[Gitlab user @adds68] I think you manage to solve this; Can we close this now?

In GitLab by [Gitlab user @tristanvb] on Aug 6, 2018, 07:32
I think this is solved, but there are some side effects we should track and improve as a result of this.
Should we open a separate issue for that ? or update the title of this issue and repurpose it for the problems we've discovered ?
Messy output
It looks like the ArtifactCache is launching something which causes stuff to be printed directly to stdout/stderr, without being logged or going through the proper logging channels
This message should not reach the terminal, unless it is properly logged and explicitly given as a Message detail string:
E0731 20:21:00.483169370   14501 ssl_transport_security.cc:1063] Handshake failed with fatal error SSL_ERROR_SSL: error:1000
007d:SSL routines:OPENSSL_internal:CERTIFICATE_VERIFY_FAILED.

More helpful error message
Is there anything BuildStream can discern from this error, and report something helpful to the user ?


it blacks half the screen when i use it

Can you share what did you do to get the black screen?


Add stagger map support

Support staggered map.
See #403

Is the file not found error from BytesResourceReader user-facing at all?
I think user will not fact  file not found error because I only use this loader load a tmx. If a tmx use extern png it will load behind.
But i'm not sure TSX tileset. rs-tiled has some tileset's attribute bug so it cannot read tsx correctly.


[Card] zDepth undocumented

On the card component, the zDepth prop is not listed in documentation

Closing for #3172


There may be a memory leak?

@KISSMonX Could you update your go langserver? This is likely a duplicate of the now fixed issue https://github.com/sourcegraph/go-langserver/issues/178

same issue

This extension looks unavailable, there is my config: -usebinarypkgcache=false -trace, after a while, memory usage increase crazy, then will be killed by system.

Is there any way for user to debug this extension, may be I can provide more log with debug info

Please try without setting usebinarypkgcache=false, that will cause greatly
instead memory usage.
If still having a problem, see readme to capture a heap profile please.
On Oct 20, 2017 3:17 AM, "lixiaohui"<EMAIL_ADDRESS>wrote:
Is there any way for user to debug this extension, may be I can provide
more log with debug info
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sourcegraph/go-langserver/issues/208#issuecomment-338167086,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADBrOLF1gjqRXIMeQ2byk_5YQNNNc8nVks5suHNWgaJpZM4QAFSP
.

Ohh, but without this option for some package the hover info is missing.

Looks as if during my type sending contentChanges event, memory increase crazy.

We don't support using that option in a desktop environment. It is used in our server environment where the working copy is immutable and type information is shared between different working copies.
If you do not have memory issues without the flag, are you happy to close this and file an issue for the missing hover info?

Sure, thank you! However, this issue isn't open by me 🤪


Update aws-java-sdk-bundle to 1.12.607

About this PR
📦 Updates com.amazonaws:aws-java-sdk-bundle from 1.12.590 to 1.12.607
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.amazonaws", artifactId = "aws-java-sdk-bundle" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.amazonaws", artifactId = "aws-java-sdk-bundle" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #3032.


gnome-music 3.20 fails to load plugins

Issue description
When launching gnome-music from a terminal, an error message is displayed:

grl-registry.c:1255: Could not open directory '/nix/store/mzq7732z6by6a729634sjkl0w5prdz7z-grilo-0.3.0/lib/grilo-0.3': Error opening directory
14:47:17 ERROR   Failed to load plugins

indeed, there is no directory grilo-0.3 there, but I have the impression the library files should be searched in /nix/store/rxzgpy3k2z8crp34ykk9d7p8bprcsh0z-grilo-plugins-0.3.1/lib/grilo-0.3 instead (please note the derivation name grilo vs. grilo-plugins).
Technical details

System: nixos-unstable

I can no longer reproduce this. Instead I get the following error when I try to play a song:
Info <GstPbutils.DiscovererInfo object at 0x7f6f3402b900 (GstDiscovererInfo at 0x21a7550)>: error: gst-core-error-quark: Your GStreamer installation is missing a plug-in. (12)
02:46:38 WARNING	URI: file:///home/jtojnar/Music/Give%20My%20Love%20To%20London/01-give_my_love_to_london.mp3
02:46:38 WARNING	Error from element mpegaudioparse3: Internal data stream error.
02:46:38 WARNING	Debugging info:
     gstbaseparse.c(3597): gst_base_parse_loop (): /GstPlayBin:player/GstURIDecodeBin:uridecodebin0/GstDecodeBin:decodebin13/GstMpegAudioParse:mpegaudioparse3:
     streaming stopped, reason not-linked (-1)

Closing as it is probably fixed.


Doesn't work with lots of entries (google.com e.g.)

Installed the latest version on Mac OSX / Firefox 56.0.2.
Tested with addons.mozilla.org and it removed 3 entries.
If I try to delete all entries from google.com or google.de the script hangs for 5min and when it's done, nothing is deleted.
Possible ways to fix this error. Set a maximum numbers of entries to delete or loop and delete in batches.

Do you have the same problem with Firefox 69.0.2?
Extensions aside: with Waterfox Classic 56.2.14 on FreeBSD-CURRENT in single-process mode, if I use the Library window to filter then delete a few thousand entries, there are noticeable hangs.

There are also noticeable hangs with my Firefox 83 (with many other extensions enabled, and with a massive history).
When hundreds or thousands of entries are to be deleted, it seems that (within the constraints of Firefox) deletions occur in batches of 300.


Expose bpfman metrics via OpenTelemetry

bpfman may have some internal metrics that we might like to expose to opentelemetry.
In order for this to work:

bpfman would write metrics into the sled database when it is running
Another process bpfman-metrics-exporter (not to be confused with bpf-metrics-exporter) would use the Otel SDK to read the metrics from the database.

Is this issue requires additional work or to be closed as otel sdk is already integrated?

@harisudarsan1 So had to hunt down Dave (who reopened) to get clarification. This is still open. This Issue tracks adding bpfman specific metrics, which would allow someone to verify bpfman is working properly or debug an issue in bpfman. #579 is complete which tracked metrics on eBPF programs that are loaded.

Thanks @Billy99 for the clarification


Error when adjusting the number of qubits in PauliFeatureMap in VQC training process

Environment

Qiskit Terra version: 0.43.0
Python version: 3.9
Operating system: MacOS Ventura 13.3.1 (a) (22E772610a

What is happening?
When running a VQC training process with a PauliFeatureMap and RealAmplitudes for a QPSK signal classification task, I encountered a CircuitError that states "Register size must be an integer".
Here is the traceback:
Traceback (most recent call last):
File "/Users/jesusvilelajato/PycharmProjects/QuantumNNSNR/main.py", line 88, in
vqc = VQC(feature_map, var_form, optimizer, training_input)
File "/Users/jesusvilelajato/PycharmProjects/QuantumNNSNR/venv/lib/python3.9/site-packages/qiskit_machine_learning/algorithms/classifiers/vqc.py", line 105, in init
num_qubits, feature_map, ansatz = derive_num_qubits_feature_map_ansatz(
File "/Users/jesusvilelajato/PycharmProjects/QuantumNNSNR/venv/lib/python3.9/site-packages/qiskit_machine_learning/utils/adjust_num_qubits.py", line 66, in derive_num_qubits_feature_map_ansatz
_adjust_num_qubits(feature_map, "feature map", num_qubits)
File "/Users/jesusvilelajato/PycharmProjects/QuantumNNSNR/venv/lib/python3.9/site-packages/qiskit_machine_learning/utils/adjust_num_qubits.py", line 106, in _adjust_num_qubits
circuit.num_qubits = num_qubits
File "/Users/jesusvilelajato/PycharmProjects/QuantumNNSNR/venv/lib/python3.9/site-packages/qiskit/circuit/library/n_local/n_local.py", line 189, in num_qubits
self.qregs = [QuantumRegister(num_qubits, name="q")]
File "/Users/jesusvilelajato/PycharmProjects/QuantumNNSNR/venv/lib/python3.9/site-packages/qiskit/circuit/register.py", line 104, in init
raise CircuitError(
qiskit.circuit.exceptions.CircuitError: "Register size must be an integer. (PauliFeatureMap ' ┌───────────────────────────────────────┐\nq_0: ┤0 ├\n │ │\nq_1: ┤1 ├\n │ PauliFeatureMap(x[0],x[1],x[2],x[3]) │\nq_2: ┤2 ├\n │ │\nq_3: ┤3 ├\n └───────────────────────────────────────┘' was provided)"
How can we reproduce the issue?
Steps to reproduce the problem
I ran the following code snippet, where I'm creating a VQC classifier for QPSK signals:
import numpy as np
from qiskit import Aer
from qiskit.utils import QuantumInstance
from qiskit.algorithms.optimizers import COBYLA
from qiskit.circuit.library import RealAmplitudes
from qiskit_machine_learning.algorithms.classifiers import VQC
from qiskit.circuit.library import PauliFeatureMap
import numpy as np
from sklearn.model_selection import train_test_split
Define the coherent QPSK FEC 1Tb baud subaquatic cable system parameters
num_symbols = 10000
symbol_rate = 1e9
baud_rate = 2 * symbol_rate
symbols_per_packet = 10  # Define the number of symbols per packet
Define the QPSK signal
signal_I = np.random.choice([-1, 1], num_symbols)
signal_Q = np.random.choice([-1, 1], num_symbols)
Simulate noise and calculate SNR
noise_I = np.random.normal(scale=np.sqrt(signal_I.var()/10**(10/10)), size=num_symbols)
noise_Q = np.random.normal(scale=1.1*np.sqrt(signal_Q.var()/10**(10/10)), size=num_symbols)
Convert to binary sequence
binary_sequence = np.array([(i > 0) + 2 * (q > 0) for i, q in zip(signal_I, signal_Q)])
Split into packets
packets_I = np.array([signal_I[i:i+symbols_per_packet] for i in range(0, len(signal_I), symbols_per_packet)])
packets_Q = np.array([signal_Q[i:i+symbols_per_packet] for i in range(0, len(signal_Q), symbols_per_packet)])
noise_packets_I = np.array([noise_I[i:i+symbols_per_packet] for i in range(0, len(noise_I), symbols_per_packet)])
noise_packets_Q = np.array([noise_Q[i:i+symbols_per_packet] for i in range(0, len(noise_Q), symbols_per_packet)])
Calculate SNR for each packet
snr = 10 * np.log10((packets_I.var(axis=1) + packets_Q.var(axis=1)) / (noise_packets_I.var(axis=1) + noise_packets_Q.var(axis=1)))
Now, you can set the snr_threshold to the median of snr
snr_threshold = np.median(snr)  # dB
Prepare labels
labels = snr >= snr_threshold
Convert labels to integers
int_labels = np.array([int(i) for i in labels])
Prepare labels and convert them into strings (required by VQC)
str_labels = np.array([str(i) for i in int_labels])
Convert to binary sequence
binary_sequence = np.array([(i > 0) + 2 * (q > 0) for i, q in zip(signal_I, signal_Q)])
Split into packets
packets = np.array([binary_sequence[i:i+symbols_per_packet] for i in range(0, len(binary_sequence), symbols_per_packet)])
backend = Aer.get_backend('statevector_simulator')
print(backend.configuration())
feature_dim = 4
num_qubits = 4  # Set the number of qubits
feature_map = PauliFeatureMap(feature_dimension=num_qubits, reps=1, paulis=['X', 'Y', 'Z'])
var_form = RealAmplitudes(num_qubits, entanglement='linear', reps=1)
optimizer = COBYLA(maxiter=500)
Prepare data for VQC
train_data = np.array([np.histogram(packet, bins=4, range=(0, 4))[0]/symbols_per_packet for packet in packets])
training_input = {str(i): train_data[int_labels == i] for i in np.unique(int_labels)}
print(np.unique(str_labels))
print(train_data.shape)
print(int_labels.shape)
print(np.unique(int_labels, return_counts=True))
for i in np.unique(str_labels):  # Here is the correction: use str_labels instead of labels
print(f"Shape of class {i} data:", training_input[i].shape)
print("Feature map circuit:")
print(feature_map.draw())
print("Variational form circuit:")
print(var_form.draw())
print(f"Number of features in data: {train_data.shape[1]}")
print(f"Number of qubits in feature map: {feature_map.num_qubits}")
print(f"Number of qubits in ansatz: {var_form.num_qubits}")
vqc = VQC(feature_map, var_form, optimizer, training_input)
split data into training and testing sets
train_data, test_data, train_labels, test_labels = train_test_split(train_data, str_labels, test_size=0.2, random_state=42)
prepare training and test datasets in the required format
training_input = {str(i): train_data[train_labels == str(i)] for i in np.unique(train_labels)}
test_input = {str(i): test_data[test_labels == str(i)] for i in np.unique(test_labels)}
VQC training
vqc = VQC(optimizer, feature_map, var_form, training_input, test_input)
quantum_instance = QuantumInstance(backend)
result = vqc.run(quantum_instance)
model testing
predicted_labels = vqc.predict(test_data, quantum_instance)
calculate accuracy
accuracy = np.sum(predicted_labels == test_labels) / len(test_labels)
print('Accuracy:', accuracy)
What should happen?
I expected the VQC training process to adjust the number of qubits in the PauliFeatureMap without raising an error, and to proceed with the training process.
Suggested solutions
It seems there might be an issue with how the number of qubits is being adjusted in the PauliFeatureMap when used in the VQC process. Perhaps an investigation into how the number of qubits is adjusted and assigned within the adjust_num_qubits method could provide insight into what is going wrong.
Thank you in advance for your help!
Any suggestions?
No response

Code corrected based on updated VQC constructor.


Build AIR macOS apps  - Apple silicon (Universal Binary) and Intel-based Mac computers

Hello,
With vsCode-as3Mxml, is it possible to build AIR macOS apps  that run natively on both Apple silicon (Universal Binary) and Intel-based Mac computers or is rosetta 2 mandatory ?
Best regards

I believe that Rosetta 2 is still mandatory, but please refer to the AIR SDK release notes on HARMAN's website for details.
https://airsdk.harman.com/release_notes


将存放比对过程中的临时结果的数据库从sqlite替换到目标redis

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


docs: correct broken hyperlinks in README.md

Thanks a lot for your work with react-query!
I noticed that GitHub's markdown parser didn't correctly render the hyperlinks in the README.md - this is the fix for it! Probably one of the easiest PRs ☺
Here is a rich diff of the changes:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update memory-pressure-reporting.md

Updating pointer check.  Report from customer

@snakex2000 Thanks for helping to improve the documentation!


Update Shuffle

according to https://en.wikipedia.org/wiki/Fisher–Yates_shuffle
checks if j ≠ i may be omitted, and also remove checking logic will optimise performance(tested by 100000 times random).

Checklist

[x] I've looked at the contribution guidelines.
[x] This pull request is complete and ready for review.

Description

Cool, thanks for the update 🙂


.Net Core - Set JsonApiSerializer as Default Setting

Hello Everyone!!
I am really appreciating this library. Great Job. Maybe this is not the right place to ask, but I really dont know.
I am trying to get a global configuration using .NET CORE 2 MVC API.
I can easily convert an object using this code. This is a controller and I am converting and respond using Ok(Json).
        public IActionResult Get([FromRoute]string company)
        {
            var parts = _partService.GetAll(company);

            string json = JsonConvert.SerializeObject(parts, new JsonApiSerializerSettings());

            return Ok(json);
        }

. It Works, but add some backslashes to the JSON due to the duplicate serialization, from the JsonConvert and then from the MVC.
"{\"data\":[{\"type\":\"partdefinitionapimodel\",\"id\":\"2026001632590498180\",\"attributes\":{\"name\":\"Tax
I Tried to look over the internet and could not find any place where I could change globally the JsonSerializerSettings with NET CORE.
I am thinking there is something related to services.AddMvc().AddJsonOption inside Startup class but I really dont know where to look or how to do it.
Thanks in advance for any help!!

@prmces sorry I only noticed this question now.
Did you manage to resolve the issue? I am not able to reproduce

As I havent heard back I am assuming the issue has been resolved. If you can manage to reproduce feel free to open the issue again

Does Anyone knows how to setup JsonApiSerializer as the default input serializer on .NET 3.1? No matter how hard I try, I can't get the objects of my Binding Model to be different from null when sending a JsonApi formatted object.
Thanks


Remove snowman

What
No snowman
How to review
Look for snowman
Who can review
Anyone able to correctly identify a snowman.

☃


[Event Request] Codeunit 101 "Cust. Entry-SetAppl.ID" – New Event “OnCheckCustLedgEntryToApply”

[Event Request] Codeunit 101 "Cust. Entry-SetAppl.ID" – New Event “OnCheckCustLedgEntryToApply”
Could we please have a new publisher OnCheckCustLedgEntryToApply in Function SetApplId, Codeunit 101 "Cust. Entry-SetAppl.ID"?
if TempCustLedgEntry.FindSet then
repeat
CustLedgEntryToUpdate.Copy(TempCustLedgEntry);
CustLedgEntryToUpdate.TestField(Open, true);
// >>>>>>>>
OnCheckCustLedgEntryToApply(CustLedgEntryToUpdate,ApplyingCustLedgEntry)
// <<<<<<<<
// >>>>>>>>
[IntegrationEvent(false, false)]
local procedure OnCheckCustLedgEntryToApply (CustLedgEntryToApply: Record “Cust. Ledger Entry”; ApplyingCustLedgEntry: Record “Cust. Ledger Entry”)
begin
end;
// <<<<<<<<

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Missing RBAC permission to update volumesnapshots/status sub-resource

The manifests do not include an RBAC permission to update the volumesnapshots/status sub-resource:
forbidden: User "system:serviceaccount:kube-system:csi-cinder-controller-sa" cannot update resource "volumesnapshots/status" in API group "snapshot.storage.k8s.io" in the namespace "default"

/assign


[Fix] When network connection is unstable, UnityWebRequest downloadHandle content may be null

Maybe we need a nullcheck here

Thank you for your contribution! looks good to me. 👍


Run ktlint for whole project (including test classes)

I removed 2 ktlint checks from our pipeline:

ktlint_device_release_check (:Corona-Warn-App:ktlintDeviceReleaseCheck)
ktlint_device_for_testers_release_check (:Corona-Warn-App:ktlintDeviceForTestersReleaseCheck)

Instead, only this ktlint will now run:

ktlint_check (:Corona-Warn-App:ktlintCheck)

This now also checks if our test classes are formatted correctly.
❗ There is a problem: The 2 deleted checks are still shown (see below) and the pipeline seems to be stuck. Does anybody have an idea why this is the case? 🤔

❗ There is a problem: The 2 deleted checks are still shown (see below) and the pipeline seems to be stuck. Does anybody have an idea why this is the case? 🤔

I think @d4rken has to adapt it in the settings.

We can adjust the restrictions once this merged, if we adjust them now, all other PRs are blocked from merging.

@d4rken I talked with @AlexanderAlferov : We can only set these restrictions globally, right? So all releases need to have these config.yml changes. So does it make sense to make this PR against 1.15.2 after it got released and then merge 1.15.2 into 2.0.x ?

We can only set these restrictions globally, right?

The restrictions are enforced via regex on the branch names.

So does it make sense to make this PR against 1.15.2 after it got released and then merge 1.15.2 into 2.0.x ?

I'd say that depends on how many changes we have to make on 1.15.x to get the adjusted check to pass, i.e. do we have to reformat a lot?
If we have to reformat a lot, keep it against 2.0.x, for a few (if any) hotfix PRs against 1.15.x I can override the restriction manually.
If there is not a lot of formatting necessary, you can target 1.15.x.


Support default content type setting

I want all of my channels to use JSON serialization instead of the default Kryo. I can set the (output) content type on a per-channel basis using spring.cloud.stream.bindings.<channelName>.content-type, but this requires listing every channel individually.
I would like to be able to set a property such as spring.cloud.stream.defaults.content-type=application/json to make JSON the default.
(Also, there may currently be a way to do this programmatically in JavaConfig, but I haven't been able to figure out how; if there is, it would be a helpful addition to the docs.)

Hi,
This is currently supported as of the Chelsea release train asspring.cloud.stream.default.content-type - see  http://docs.spring.io/autorepo/docs/spring-cloud-stream-docs/Chelsea.SR1/reference/htmlsingle/#binding-properties .

I had managed to miss that, sorry! (As a note, STS doesn't recognize the default namespace. I don't know whether that's practical to remedy.)

No problem - could be a metadata issue!


Change Login Parameters

Changing Username e-mail address, password and access token

Coverage remained the same at 64.674% when pulling 2a219ca864ffd76b3c8dcc2eab89960ac8340cde on hwbrill:heroku-particle-api-js into 78a7eb9195c3f27f80282cc61875d789401898d2 on spark:master.

Coverage remained the same at 64.674% when pulling 2a219ca864ffd76b3c8dcc2eab89960ac8340cde on hwbrill:heroku-particle-api-js into 78a7eb9195c3f27f80282cc61875d789401898d2 on spark:master.

Don't know why this happened except... I don't know what I am doing!


I got this error in Run

Microsoft PowerToys version
0.66.0
Installation method
PowerToys auto-update
Running as admin
None
Area(s) with issue?
PowerToys Run
Steps to reproduce
Only happened once but how it happened was I

Quickly press alt+space
Quickly type log(10)/log(2)
It happened with no errors the next time I did it.
A screenshot of the error message is below in the actual behaviour section.

✔ Expected Behavior
A mathematical calculation to be done.
❌ Actual Behavior

Other Software
No response

oops, accidentally closed it.

Here is the image that was meant to be uploaded but isn't working

/dup #18166
Do you by any chance also have Adobe Preview Handler as mentioned in https://github.com/microsoft/PowerToys/issues/18166#issuecomment-1332887065 ?

@jaimecbernardo I do not have Adobe Preview Handler, however, looking at the other post, I actually do have korean characters installed but it wased used at that instance. (Main language on my computer is English)

For any users that are able to replicate this consistently, can you please update to 0.68.1 and disable thumbnail generation to verify if it fixes the issue? Please let us know on #18166!


"Denoising Diffusion Implicit Model" example broken in Colab

The Colab notebook that exemplifies the implementation of DDIM in Keras does not work using default configuration.
Error in training cell:
AttributeError                            Traceback (most recent call last)
[<ipython-input-17-12d5033d5855>](https://localhost:8080/#) in <module>
      6 # optimizer=tfa.optimizers.AdamW
      7 model.compile(
----> 8     optimizer=keras.optimizers.experimental.AdamW(
      9         learning_rate=learning_rate, weight_decay=weight_decay
     10     ),

AttributeError: module 'keras.api._v2.keras.optimizers' has no attribute 'experimental'

Using:

Tensorflow 2.9.0
Keras 2.9.0

EDIT:
It can be fixed by installing the library that includes AdamW optimizer as it's not installed by default in Colab:
!pip install tf-nightly

But this arises a new error on checkpoint callback:
---------------------------------------------------------------------------

TypeError                                 Traceback (most recent call last)

[<ipython-input-9-12d5033d5855>](https://localhost:8080/#) in <module>
     33     callbacks=[
     34         keras.callbacks.LambdaCallback(on_epoch_end=model.plot_images),
---> 35         checkpoint_callback,
     36     ],
     37 )

1 frames

[/usr/local/lib/python3.7/dist-packages/keras/utils/traceback_utils.py](https://localhost:8080/#) in error_handler(*args, **kwargs)
     65             return fn(*args, **kwargs)
     66         except Exception as e:
---> 67             filtered_tb = _process_traceback_frames(e.__traceback__)
     68             # To get the full stack trace, call:
     69             # `tf.debugging.disable_traceback_filtering()`

[<ipython-input-8-7570dd8bce01>](https://localhost:8080/#) in train_step(self, images)
    112
    113         gradients = tape.gradient(noise_loss, self.network.trainable_weights)
--> 114         self.optimizer.apply_gradients(zip(gradients, self.network.trainable_weights))
    115
    116         self.noise_loss_tracker.update_state(noise_loss)

TypeError: in user code:

    TypeError: tf___process_kwargs() takes 2 positional arguments but 4 were given

Hi,
Thanks for reporting the issue.
I was able to successfully run the example using Tensorflow 2.13 version.
Here is the Gist attached for reference https://gist.github.com/sachinprasadhs/981e2c55608f9ea7351971944d892951.
Please verify and close the issue.


FIX Don't raise a deprecation warning for xarray.ufuncs.{angle,iscomplex}

Closes https://github.com/pydata/xarray/issues/2609
This removes PendingDeprecationWarnings for xarray.ufuncs.{angle,iscomplex} which appear to be proper ufuncs and cannot be just used from numpy for now (as discussed in the parent issue).
Also adds more generic tests that numpy ufuncs do preserve the DataArray dtype.

Hello @rth! Thanks for submitting the PR.

There are no PEP8 issues in the file xarray/tests/test_ufuncs.py !

There are no PEP8 issues in the file xarray/ufuncs.py !

thanks @rth !


Migrate docs to new framework

Background
The docs are currently written in Pug and built as HTML:

Source
Published

There are various issues:

editing the docs is hard
publishing new docs is currently concatenating spaces between tags

I've got the build working, done some initial cleanup (links, ads, sponsorship links, version, install page) which will do, but  it would be nice (though not a necessity) to migrate to a modern markdown-driven docs framework.
Proposal
Convert source files:

[x] assess feasibility
[x] convert docs (script / package)
[ ] fix examples

Theme:

[ ] go with default?
[ ] or, rebuild using Spectre itself?

Build:

[ ] choose framework
[ ] rebuild docs + examples
[ ] implement search (Agolia, or such like)
[ ] publish on Netlify / vercel

Notes
Options:

VitePress
Astro
something else?

@ma4nn do you have any preferences?
I'm a Vue guy, but open to the right framework to get it done quickly.

Making good progress on converting the docs from HTML to Markdown, using some custom scripts and packages.

VitePress is certainly doing the job!

OK; see docs repo here:

http://localhost:5173/docs/components/badges.html


[DOCS] - example programs running not storing program

I wonder if in all the files to run programs e.g. client_single_party_compute/addition_simple.py we should also add code to upload the program in the file.
Otherwise, users have to store at the cli everytime before they can actually run those files.
The docs walk the user through this step, but it may get annoying for users to always have to store the program from the cli.
However, we should avoid confusion by having the docs tell them to upload from the cli then when they run the python script, it says "storing program". Maybe we have a bit in the python script that checks to see if the program is already stored, and if not, stores it for the user
Just an idea

Done - all examples have store_program in them now


Add Install Guide to repository

While the website is "down" or broken it would be useful if you have INSTALL instructions inside your repository.
Because to be honest: I have no idea what I have to do to get Cosmo ready.

Added the installation page. http://www.cosmocms.org/install


Accept Auth->ResourcePolicy for serverless.Auth

This is an attempt to fix #1525

Solved in https://github.com/cloudtools/troposphere/pull/1568

Closing. Thanks @ikben.


Cannot read property 'getFullYear' of null

Expected Behavior

Be able to successfully use minDate and maxDate and set them to null to get rid of them

Current Behavior

Getting issues with getFullYear still from this line https://github.com/chmln/flatpickr/blob/158dfcd9bee5bf37b933b475f2ee0b112d18d1e8/dist/flatpickr.js#L671

Possible Solution

Do a check for null, and clear it out

Steps to Reproduce (for bugs)

Set minDate or maxDate to null
2.
3.
4.

Your Environment

Version used: 2.0.1
Browser Name and version: Chrome
Operating System and version (desktop or mobile): OS X
Link to your project:

Could we do something across the whole codebase that ensures everything is converted to an actual date, so these things all work?

Found the issue. https://jsfiddle.net/chmln/3htf1hjw/
Setting minDate to null or any falsy value works fine.
But due to faulty logic in set() an invalid value gets set anyway.
Expect a release with everything fixed today/tmrw.
Thanks for reporting the issue

Thanks @chmln. Looking forward to a release where all my ember-flatpickr tests pass and I can release a new version 😄 . Should we port some of the tests over to here? Seems like these issues would have never happened in the first place, if the same tests were in place here.

@chmln I just updated to 2.0.4 and I still get TypeError: jumpDate.getFullYear is not a function

Fixed that with :
(DateUtil.prototype as any).createDate = (year, month, date, hours, minutes, seconds) => {   let ret = new Date(year, month, date, hours, minutes, seconds);   return ret.getMonth() == month ? ret : new Date(year, month + 1, 0, hours, minutes, seconds); }


nim-syscall for MacIntosh

I think that your ideas about writing software without reliance on large libraries are very interesting. That said, is it possible to adapt nim-syscall for Macintosh? Could you do that?

Honestly this repo was done more as a PoC than for actually writing software with. I'm not going to extend it to other platforms. If you manage to make a PR and get it working, I'd merge it though.


Add GKE Autopilot tests

What this PR does / why we need it:
Modifies the kots-e2e GitHub action to accept a few new inputs that enable running tests on Google Cloud Kubernetes Engine (GKE) Autopilot clusters. The most important new input is platform, which will accept either 'gke' or 'k3s' and run the end to end (e2e) tests on the given platform. There is also another input gke-service-key that is used as the GKE authentication token. All other inputs have reasonable default values, but can be used to change the cluster configuration if needed.
The modifications to the e2e action are steps that authenticate to GCloud, create a new autopilot cluster, then generate a kubectl config to the new cluster if the platform input is 'gee'. The test run continues as normal and after the tests are run, even on failure, the cluster is destroyed.
Each job in a workflow will create and destroy the cluster used by the job. While this leads to more clusters being created, it avoids the pitfalls of trying to make sure that when there are multiple jobs run on GKE clusters that there won't be failures synchronizing the creation, use of, and destruction of a cluster that is shared between jobs. The cost impact from this should be minimal.
The platforms are also run in a matrix with the k8s versions, although currently the created cluster uses the latest k8s version on GKE. If there is a test with multiple versions in a matrix with k3s, a single version of gke can be added with the include annotation:
test-name:
  strategy:
    matrix:
      k8s_version: [ v1.26.1, v1.25.6 ]
      platform: [ k3s ]
      include:
        - platform: gke
          k8s_version: "v1.26.1"

For estimating costs, let's assume that there could be a max of 5 PRs created per day and each PR will run 3 tests (3 clusters created) and each test starts 5 pods each using 1GB of memory and run for 1 hour. Since GKE charges by vCPU and memory usage, this will be 30 vCPUs and 30GB of memory being used for 1 hour per PR and with 5 PRs that's 150 of hourly units to charge each day. Based on the prices found here that totals to:
75 vCPU hours * $0.0445 / vCPU hour = $3.34
75 GB ram hours * $0.0049225 / ram hour = $0.37
Total daily cost = $3.71
Monthly cost (x20) = $74.20
The real number will be much lower, because this PR only enables a single GKE test and there are probably on average fewer than 5 PRs created per day.
The creation and destruction of a GKE cluster seems to take about 5-10 minutes each and running the test on the cluster adds maybe 1-2 minutes to the test as the GKE pods take longer to spin up than a k3s cluster. So on average this will add around 12-15 minutes to a test, but could take as long as 25 extra minutes. In practice everything usually runs on the faster end of those ranges, but there have been a few times where a GKE command takes a lot longer to run than normal.
The regular validate-smoke-test does not work on autopilot clusters due to a security restriction not allowing hostpaths to be mounted in write mode (docs), but the validate-air-gap-smoke-test does not need those permissions, so was used instead.
Many of the other testim tests could be modified to either have higher timeouts (deploys seem to take around 2 minutes before being ready) or "wait for" (testim action) the kotsadm web interface to say "Ready" instead of "Unknown".
Which issue(s) this PR fixes:

Fixes #SC-62335
Special notes for your reviewer:

The Testim step timeout for the validate-airgap-smoke-test checking for 'Ready' in the admin page after a deploy was increased to 5 minutes as it typically takes around 2 minutes for a deployment to be ready, but the testim test has a default timeout of 30 seconds (really 28 for some reason).
Steps to reproduce

Does this PR introduce a user-facing change?

NONE

Does this PR require documentation?

NONE

I agree with making the platform not required with a default of 'k3s', that's a good nitpick and implemented now


Add feature to set JSON-LD Frame for SPARQL Queries

In TriplyDB there is a feature in the SPARQL IDE to switch to JSON-LD Frame editor for a saved Query. In this view users can paste a JSON-LD Frame which is somehow stored with that query. It does not seem possible to store this frame and the SPARQL IDE state that comes with it using TriplyDB-JS. It look like it is thus impossible to programmatically create a SPARQL Query in an account making use of this feature.
Expected behavior:

Documentation on how to save a JSON-LD Frame when creating or changing a saved SPARQL Query.
Support for adding JSON-LD Frames in the API and TriplyDB-JS

Thanks for reporting this @mightymax ! We're looking into this, and should have this feature ready within two weeks.

Fixed in https://github.com/TriplyDB/TriplyDB-JS/pull/31
Released in v7.1.0
@mightymax


Change Chinese comment to English comment

We should add English comments so that we can get more developers to participate in our projects. I have added some comments and need some project members to review.

I have fixed some comments.Please review it again.Thanks.


Data Movement: Possible overwrite on copy out

When performing a copy out (from either directives or copy offload API), there is potential for users to overrwrite their data if they do not use a naming scheme for their data that is unique to the compute node (e.g. flux/mpi rank, hostname).
This example illustrates what happens today.
In this example, let's use a 2 rabbit, 4 compute system where each rabbit has 2 compute nodes. The user is using all 4 nodes for their flux job and they want to move their data out from gfs2 to global lustre. The application they are running across the computes simply dumps out a file named data.out. With 4 compute nodes, that's 4 data.out files.
Since each filesystem is unique for each compute node, the data movement effectively needs to merge those 4 filesystems into one destination location.
From the rabbit's perspective, these locations are mounted at:
rabbit-node1:
/mnt/nnf/a8e02956-de88-472d-9a5b-a64e4a54d568-0/0/ -> compute-node1
/mnt/nnf/a8e02956-de88-472d-9a5b-a64e4a54d568-0/1/ -> compute-node2

rabbit-node2:
/mnt/nnf/a8e02956-de88-472d-9a5b-a64e4a54d568-0/0/ -> compute-node3
/mnt/nnf/a8e02956-de88-472d-9a5b-a64e4a54d568-0/1/ -> compute-node4

The extra 0/ and 1/ directories are the index mount directories that map to each compute node.
So for this workflow, the $DW_JOB_MY_GFS2 variable would point at /mnt/nnf/a8e02956-de88-472d-9a5b-a64e4a54d568-0 without the trailing index mount directories. Since this value is used across all the compute nodes, it must remain the same.

If the user's copy_out directive looked like this:
$DW copy_out source=$DW_JOB_MY_GFS2 destination=/lus/global/user/my-job/

Then the resulting destination would look like this:
/lus/global/user/my-job/0/data.out
/lus/global/user/my-job/1/data.out

2 of the 4 data.out files are missing.
Since we are instructing dcp to copy everything out of `$DW_JOB_MY_GFS2 the index mount directories come with it. And since the indexs are the same accross the rabbits, 2 of those files get overwritten.

Our initial thoughts are to create some sort of unique identifer that combines the rabbit/compute relationship. That could then be appended to the destination to ensure that no data can be clobbered.
This simplest would be to use the rabbit hostname and the index directory, we could append this directory to the destination and the following would result when using the same destination=/lus/global/user/my-job/ from above:
/lus/global/user/my-job/rabbit-node-1-0/data.out
/lus/global/user/my-job/rabbit-node-1-1/data.out
/lus/global/user/my-job/rabbit-node-2-0/data.out
/lus/global/user/my-job/rabbit-node-2-1/data.out

I realize that the user most likely does not care about the rabbit hostname and index directory, but it would prevent users from overwriting their data if that data is not aptly organized.

This can be replicated by (assuming a 2 rabbit node system with 2 computes on each):
flux run -l -N4 --setattr=dw="#DW jobdw type=gfs2 capacity=10GiB name=my-gfs2 #DW copy_out source=\$DW_JOB_my_gfs2 destination=/lus/global/user/my-job/ profile=no-xattr" bash -c 'fallocate -l1G $DW_JOB_my_gfs2/data.out'

Which results in:
$ cd /lus/global/user/my-job
$ tree
.
|-- 0
|   `-- data.out
`-- 1
    `-- data.out

2 directories, 2 files

This issue is fixed via https://github.com/NearNodeFlash/nnf-sos/pull/257. Each index mount directory now contains the rabbit name, so you get a unique index mount directory for every compute node:
$ cd /lus/global/user/my-job
$ tree
.
├── rabbit-node-1-0
│   └── data.out
├── rabbit-node-1-1
│   └── data.out
├── rabbit-node-2-0
│   └── data.out
└── rabbit-node-2-1
    └── data.out

These directories don't mean much to the user, but this protects any from any potential data loss.
Note: One important thing here: the my-job/ directory must exist in the /lus/global/user/ directory. This is discussed in another issue: https://github.com/NearNodeFlash/NearNodeFlash.github.io/issues/130


Feature Request: Support TerserPlugin options

Laravel Mix Version: 6.0.6

Description:
terserOptions no longer support in Mix 6
mix.options({
    terser: {
      terserOptions: {
        compress: {
          drop_console: true,
        },
        extractComments: "all",
      },
    },
})

Steps To Reproduce:

Add the existing terserOptions to Mix 6

Already found the reference from other ticket. https://github.com/JeffreyWay/laravel-mix/issues/2738

Already found the reference from other ticket. https://github.com/JeffreyWay/laravel-mix/issues/2738


Changing bin size on individual pages slightly shifts spectra rightwards

@guillochon Good to know. I'd be interested in seeing this. I tried a few examples but didn't notice anything completely crazy.

Closing this for now. Problem not apparent at the moment.


Result.utils.marginal_counts returns inconsistent creg structure for multiple cregs

Information

Qiskit Terra version: master @ d26f273d
Python version: 3.6
Operating system: osx

What is the current behavior?
Steps to reproduce the problem

What is the expected behavior?
Suggested solutions

@kdk Just to confirm. The right answer should be {'111': 1024} and if the indices are changed
to [0, 1, 2, 3], it would be {'0 111': 1024} and [0, 2, 4] would be {'011': 1024} or {'0 11': 1024}?

@kdk Just to confirm. The right answer should be {'111': 1024} and if the indices are changed
to [0, 1, 2, 3], it would be {'0 111': 1024} and [0, 2, 4] would be {'011': 1024} or {'0 11': 1024}?

The first two examples are consistent with what I'd expect. For the last, I'd expect 0 11.

Thanks. I think I've found where the problem is, and right now I'm working on finding a reasonable solution. It's weird, there are about a dozen tests for marginal counts, but none of them seem to set memory_slots to less than the number of cregs.

This is meant as the start of a discussion about marginal_counts() which grows out of this issue.
The problem with a solution for the case above is that the final output, say [0, 2, 4] giving you '0 11' is that it is not unique. [0, 1, 4], [1, 2, 3], etc. will give you the same '0 11'. If this is passed to another function, all the original indices info is lost.
As an alternative, first consider how marginal_counts() currently works.

If it is passed a counts dict as an arg, it removes the spaces from any count strings and computes the marginal counts. For example, returning '011' in the case above.
If it is passed a Result object, it takes the counts from there and does the same marginal counts computation and then converts the returned string to a hex value in the counts dict and reduces the memory_slots count to the number of bits in the string. It then returns the modified Result object.

There are 2 problems here,

The counts string, '011' in our case, loses information of what indices were used. As a solution it would be possible to post-format the counts by adding spaces between cregs and a placeholder, maybe _ or x, for non-indices. So in our example the post-format string would be '0_ 1_1' or '0x 1x1'. Now it's clear what indices were chosen and these strings should work properly for any plot labels, since spaces are already used.
The second problem is that passing a Result object to marginal_counts() and then modifying it (or a copy of it) does not currently work if get_counts() is called on that Result. It might be possible to modify the creg_sizes along with memory_slots, but this creates a fair amount of overhead and in the end you are just left with '0 11' as output with no indices information.
A solution here would be to deprecate passing a Result object to marginal_counts() and instead add an indices kwarg to get_counts(). Then get_counts() will call marginal_counts() if indices is not None, and the post-formatted output returned, resulting in '0_ 1_1' or '0x 1x1' as the string part of the get_counts() output.

Any feedback would be appreciated.


Code cleanup

This PR contains several changes for cleaning up internal attribute handling, renaming various user options for clarity/consistency, and some reorganization of user-facing functions for handling data.

This is great-- I think you're still working on changes but let me know what testing you've done/if it's in a good state for me to run this on signal and nulls to see where it breaks!

I’ve run this on the example script locally and on the tutorial notebook.  Would be great to run this on some other dataset for sure, especially to test template subtraction and Planck signal subtraction.
I’m working on two things:

Combining get_masked_data, get_masked_fake_data and the data pieces in get_masked_sims into a single function
Reconfiguring the Planck things to use more general terminology, e.g. reference_signal subtraction and A/B reference halves for templates.

A longer term project is reconfiguring the likelihood code to make it easier to plug into other solvers, e.g. Cobaya.  That will be a separate PR though.

This is great-- I think you're still working on changes but let me know what testing you've done/if it's in a good state for me to run this on signal and nulls to see where it breaks!

I’ve run this on the example script locally and on the tutorial notebook. Would be great to run this on some other dataset for sure, especially to test template subtraction and Planck signal subtraction.
I’m working on two things:

Combining get_masked_data, get_masked_fake_data and the data pieces in get_masked_sims into a single function
Reconfiguring the Planck things to use more general terminology, e.g. reference_signal subtraction and A/B reference halves for templates.

A longer term project is reconfiguring the likelihood code to make it easier to plug into other solvers, e.g. Cobaya. That will be a separate PR though.

Given that there are more significant changes coming I'll hold off on a barrage of testing unless I find myself with some free time.

Ok, I think most of the code changes are in place now, and tested on the tutorial notebook and example script.  Still need to go through the tutorial notebook and make sure that all the text there reflects the changes properly.

I'm hands off on this now!  Any other changes I can think of should be separate PRs for later.


Glorfindel: Blacklist thebeautysystem.com

Glorfindel requests the blacklist of the following website(s):

thebeautysystem.com - MS search

thebeautysystem.com has been seen in 7 true positives, 0 false positives, and 0 NAAs.


Error AuctionCommand.php

Thread: Main
Error: Class Biswajit\AuctionHouse\AuctionCommand contains 1 abstract method and must therefore be declared abstract or implement the remaining methods (Biswajit\AuctionHouse\libs\CortexPE\Commando\IRunnable::getPermission)
File: plugins/FCAuctionHouse_dev-7.phar/src/Biswajit/AuctionHouse/AuctionCommand
Line: 29
Type: E_ERROR
Backtrace:
Code:
[20] use pocketmine\item\StringToItemParser;
[21] use pocketmine\item\VanillaItems;
[22] use pocketmine\nbt\NbtStreamReader;
[23] use pocketmine\network\mcpe\protocol\serializer\NetworkNbtSerializer;
[24] use pocketmine\player\GameMode;
[25] use pocketmine\player\Player;
[26] use pocketmine\Server;
[27] use pocketmine\utils\Config;
[28]
[29] class AuctionCommand extends BaseCommand
[30] {
[31]     public function __construct()
[32]     {
[33]         parent::__construct(Auction::getInstance(), "market", "Open market", ["auctionhouse", "hdv", "ah"]);
[34]         $this->setPermission("FCAuctionHouse.cmd.ah");
[35]     }
[36]
[37]     protected function prepare(): void
[38]     {
[39]         $this->registerSubCommand(new SellSubCommand(Auction::getInstance(), "sell", "Sell items", []));

"IRunnable::getPermission"
Problem umm

@YussGG just download it from GitHub and use custom commando

Ok


[MINOR][DOCS] Correct the usage example of Dataset in Java

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

Was this patch authored or co-authored using generative AI tooling?

Merged to master.


Repo page - paths under object name

I randomly tried this change and I think it's just straight up better. Any good reasons why we would not want the paths under the object name?

heh, no, looks very good IMHO 😄

breaks sorting by path, but that's like sorting by package->object, so you could just use the folders view


Prepare for 2.0.0 release

See https://github.com/naegelejd/mrd/tree/main

Re-did the documentation using vitepress and added Docker image build (for all C++ mrd tools).
The Docker image is not minimal and contains the full conda environment.
It is possible to make it minimal in the future, by

Making the CMakeLists.txt generated by yardl work out-of-the-box on Ubuntu 22.04 (a good idea) and,
Installing all dependencies using apt instead of conda


I have a process that consists of three steps:

Planning
Preparation
Execution

The last step leads to a product.
git-bob draw an SVG graphics demonstrating the process steps as boxes with arrows in between.

git-bob try to do this


Why using ARM Core M and doing an Espressif style?

Can somebody explain, why the Raspberry Pico has an ARM Core M but does not follow the ARM Core M software standards? So the SDK does not implement the benefits of ARM Core M and therefore is not compatible to the ARM Cortex M Eco-System, third parties, etc.? The big advantage of IP in the todays world is you getting hardware and software by the IP vendor - in this case the MCU Core by ARM. So why just using the hardware and the interesting part "Software" is ignored?
So nice SDK, but as soon a product with Pico comes to mass production, I see a lot of costs up-coming, because defined interfaces to third parties, tool suppliers, etc. are not existing. So for everything extra costs are required.
I was very happy to see Raspberry Pi foundation has an own MCU but as soon it came to software I was disappointed.

Thanks for this statement.


Get the Beta Test workflow defined

Basically, we upload the Beta version to Google Play as internal test version. All internal testers are added to the test track. The first round of testing takes place and issues are fixed. When we have a candidate ready, we bump up the release to open beta. so that more people can opt in to try the release.


Suggestion: Toggle behaviour for the TOP hotkey

I would love to have a config option to change the behaviour of the TOP hotkey to one where I have to hold the key to show the TOP box, instead of toggling it on and off.
I always used this option with WAILA, as I find it much more convenient.

That's a good idea

Implemented in next version


[Bug Report]: Align ADO and GH removal of validation deployments

Describe the bug
Align the condition on resource removal after validation deployments.
Desired output:

if validation deployments succeed, continue with removal
if validation deployments fail, continue with removal
if pipelines/workflow is cancelled, also cancel the removal

To reproduce
N/A
Code snippet
From .azuredevops\pipelineTemplates\jobs.validateModuleDeployment.yml

      # [Removal] task(s)
      #------------------
      - task: AzurePowerShell@5
        displayName: 'Remove deployed resources via [${{ parameters.serviceConnection }}]'
        condition: and(succeededOrFailed(), eq('${{ parameters.removeDeployment }}', 'True'), not(eq(variables['deploymentNames'],'')), not(startsWith(variables['deploymentNames'], 'variables[' )))
        inputs:

From .github\actions\templates\validateModuleDeployment\action.yml
    # [Deployment removal] task(s)
    # ----------------------------
    - name: 'Remove deployed resources'
      if: ${{ always() && inputs.removeDeployment == 'true' && steps.deploy_step.outputs.deploymentNames != '' }}
      uses: azure/powershell@v1
      with:

Relevant log output
NA

The alignment should be the other way as discussed with @AlexanderSehr. The GH condition on the same should be set to succeededOrFailed(), and not always(), as we might want to troubleshoot if a deployment messes up and we cancel a run.
Lets try with success() || failure()

Godspeed @MariusStorhaug


Added extension parameter to ImageResizer

Documentation page: https://wintercms.com/docs/v1.2/docs/services/image-resizing#available-parameters

I have a question:
does ImageResizer support custom image name?
Ex.:
{{ item.logo.thumb(200, 150, { 'imagename': 'logo-' ~ item.slug, 'extension': 'webp' }) }}

Result:
old-name.jpg -> logo-slug.webp

😱 Since when is this possible? I've been using JPGs for the last few years!
@LukeTowers Is it still good practice to use TinyPNG to make the images smaller first? Or is webp so small that this isn't needed anymore?

I assume a fallback to JPG is no longer needed? https://caniuse.com/webp

This enhancement is no longer needed for me because of the filter
https://github.com/wintercms/wn-tinypng-plugin/issues/2

😱 Since when is this possible? I've been using JPGs for the last few years!

This was available even before the fork.
Example:
<img src="{{ user.avatar.getThumb(90, 90, {'mode':'portrait', 'quality': 80, 'extension': 'webp'}) }}" width="90" height="90" alt="{{ user.name }}" />

😱 Since when is this possible? I've been using JPGs for the last few years!

One more for me!
I was completely unaware of this parameter.

Is this option related to other platform requirements?

The only requirement is that the source image file must have the extension jpg, jpeg, png, gif, webp.
https://github.com/wintercms/storm/blob/dd89dc140bfdb0e209437b2263b711b8fa655981/src/Database/Attach/File.php#L65

All parameters: https://github.com/wintercms/storm/blob/dd89dc140bfdb0e209437b2263b711b8fa655981/src/Database/Attach/File.php#L622-L629
Default values: https://github.com/wintercms/storm/blob/dd89dc140bfdb0e209437b2263b711b8fa655981/src/Database/Attach/File.php#L689-694

There is one more parameter that is not specified in the documentation. This is interlace.
interlace: Interlace image,  Boolean: false (disabled: default), true (enabled)
https://github.com/wintercms/storm/blob/dd89dc140bfdb0e209437b2263b711b8fa655981/src/Database/Attach/Resizer.php#L156-L176
In my opinion, this is something outdated for today's Internet.

I don't see in the code whether there is a parameter to set an SEO-friendly name for the file. Lighthouse doesn't like the generated file names.

The only requirement is that the source image file must have the extension jpg, jpeg, png, gif, webp. https://github.com/wintercms/storm/blob/dd89dc140bfdb0e209437b2263b711b8fa655981/src/Database/Attach/File.php#L65

I'd never realized before that this amazing feature was available!

There is one more parameter that is not specified in the documentation. This is interlace.
...
In my opinion, this is something outdated for today's Internet.

Totally agree.

I don't see in the code whether there is a parameter to set an SEO-friendly name for the file. Lighthouse doesn't like the generated file names.

Yes, this is a serious concern for good image SEO.

The ImageResizer's image URLs for all file types except for FileAttachements should be SEO friendly, is it not?


Added typescript definition

Description
Adds typescript definitions to the module. (Also fixes Travis CI for Windows)

Related Issue
fixes: https://github.com/adobe/jwt-auth/issues/28
fixes: https://github.com/adobe/jwt-auth/issues/30
Motivation and Context
Allows the module to be consumed via TypeScript.
How Has This Been Tested?
Used yarn link locally and included it in a TypeScript project.
Types of changes

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

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

@ittaibaratz any other changes?

@macdonst can you review? I am not sure about the Travis changes as well as including contributor personal details in the code.

@macdonst can you review? I am not sure about the Travis changes as well as including contributor personal details in the code.

@ittaibaratz I can take the TS header out if you want that is just the standard recommended details from the template here: https://www.typescriptlang.org/docs/handbook/declaration-files/templates/module-d-ts.html

@macdonst can you review? I am not sure about the Travis changes as well as including contributor personal details in the code.

@ittaibaratz I can take the TS header out if you want that is just the standard recommended details from the template here: https://www.typescriptlang.org/docs/handbook/declaration-files/templates/module-d-ts.html

@AndrewLeedham it would be great if you can use our standard header without any personal details. Your details as a contributor are visible to everyone through Github itself.
Also, the Typescript addition looks good to me, however I am not sure about the Travis CI changes and its a bit challenging to test it - So if you want the Typescript addition added soon, I'd recommend you break this pull request into 2 pull requests as these are two separate issues.

@ittaibaratz Ok I will refactor the comment header. I looped the travis changes in here because the tests will not pass for the PR otherwise.

@ittaibaratz Ok I will refactor the comment header. I looped the travis changes in here because the tests will not pass for the PR otherwise.

That's ok, we can still merge the pull request if we are confident the tests are not failing because of the code change.

Seems travis may have updated nvs or fixed the mismatch between linux and windows 🎉


about shadow_model train datasets

Hallo,i have a question about generating shadow model training data.
you are using a query-based synthesis approach.However, the display result of the synthesis data is far from that of the cifar-10 picture.
Can you tell me why?Thank you very much！

Hallo,i have a question about generating shadow model training data.
you are using a query-based synthesis approach.However, the display result of the synthesis data is far from that of the cifar-10 picture.
Can you tell me why?Thank you very much！

Hi,
The code of this paper doesn't contain data synthesis algorithm, so I just completed the algorithm on my own. I have contacted Dr. Shokri, the first author of the paper, but he told me the algorithm is designed for binary class. In other words, it doesn't fit for Cifar10 dataset. Therefore, we need to complete the data synthesis algorithm for Cifar10 dataset by ourselves.

Thank you very much for answering my question. One question is why the accuracy of the attack in your code is very high when the cifar-10 synthesis data obtained by using the query is quite different from the real data.And do you know what methods are available to synthesize simulated image information in the black box situation.

Thank you very much for answering my question. One question is why the accuracy of the attack in your code is very high when the cifar-10 synthesis data obtained by using the query is quite different from the real data.And do you know what methods are available to synthesize simulated image information in the black box situation.

The original code for the paper just replace some images with some random generated numbers, so I guess the key intuition lies on shadow models generation and shadow training (depends heavily on the guessing of shadow model). That might be why the accuracy of membership inference attack is high. For the second question, maybe you could refer to membership inference attack and adversarial example generation.

好滴，学长你的意思是攻击的精度与影子训练数据和影子模型直接相关吧。然后这个是从合成的数据集中选取一条数据生成的图像，是我生成的方式错误了吗，咋是这个结果。谢谢学长啦
------------------ 原始邮件 ------------------
发件人:                                                                                "icmpnorequest/MLSec"<EMAIL_ADDRESS>发送时间: 2020年10月7日(星期三) 下午3:10
收件人<EMAIL_ADDRESS>抄送: "王先森"<517928277<EMAIL_ADDRESS>主题: Re: [icmpnorequest/MLSec] about shadow_model train datasets (#2)
Thank you very much for answering my question. One question is why the accuracy of the attack in your code is very high when the cifar-10 synthesis data obtained by using the query is quite different from the real data.And do you know what methods are available to synthesize simulated image information in the black box situation.
The original code for the paper just replace some images with some random generated numbers, so I guess the key intuition lies on shadow models generation and shadow training (depends heavily on the guessing of shadow model). That might be why the accuracy of membership inference attack is high. For the second question, maybe you could refer to membership inference attack and adversarial example generation.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Create a user-guide/help pages

Explain how to work with the tool - how to search, navigate, and select (multiple!) elements, then how to load and remove related items, etc.
Explain how to use the detail panels, etc.
Create screen-casts, screen-shots, and a description.
Include it in the UI!

I've added a draft guidance document and an overview of the interface diagram -
https://github.com/covidgraph/visual-graph-explorer/blob/master/docs/vge-guidance.md
I still need to:

[ ] Add a link to the most recent data model
[ ] Identify a few real world examples and either write these up as step by step guides or do a screencast
[ ] Add more screenshots.

Happy for any feedback.


Improve zug

Some improvements to accept your recipe in Conan Center Index

Thanks a lot for all the great improvements :-)
Two things I see:

by changing the setting to settings = ("compiler", "arch", "os", "build_type"), you made the recipe depend on arch, os and build_type, which this library doesn't depend on them at all (you didn't use these flags in the recipe).
the only dependency is in the compiler, its version, and the c++ std.
the code you added is great, but why did you remove the dependency over Zc:externConstexpr when using VS?
This flag is needed when we use VS in C++14.

Use FULL settings ever. It will be the only way for Conan V2 which is closer to be released. You should manipulate the package ID on package_id method, so does not matter if you are using full settings.

Usually we take compiler flags as patch, or even from upstream. I see this is not the case, Zug has no more updates and that flag is too small for a patch. I just reverted to the previous cxxflags.

cool
how exactly do you take the compiler flags as a patch? from where?
Usually, they are part of the build system scripts (here CMake target) but how do you extract them from there? and to where?
I fill I miss a big part of Conan here :-| do you have an example? a link to docs?

Create a patch to change directly the upstream's cmake. For this specific case, externConstexpr is known: https://github.com/arximboldi/zug/issues/26#issuecomment-824004036

About patch example, there are tons in Conan Center Index, just search for "patches". Anyway, it's documented here and here

I understand patching the library code if we can't change it, but AFAIK this is not the issue here.
The issue here is with compiler flags when you use the library.
I need a way to get these flags from the library to the project which uses the Zug's Conan package.
How do I do that?
My understanding is that I need to put these flags in the recipe, as we did here:
    def package_info(self):
        if is_msvc(self):
            self.cpp_info.cxxflags = ["/Zc:externConstexpr"]

What other way does the library have which can be taken by the recipe and put in the package so a client can import the package?
What am I missing?
The link you put, tells the programmer to add this flag manually to his build script which uses Zug.
Or, the flag can be put in the CMakeLists.txt file, as a compiler option to the Zug target, but that means I will need to package that file too and add it as a CMake module, but that will only work for someone who imports the package through CMake.
Again, I fill I miss a fundamental thing here.

btw, about settings: https://github.com/conan-io/conan-center-index/blob/154c9edca773d52a790d917bf2c0b95796a7ce7b/docs/packaging_policy.md#settings

What other way does the library have which can be taken by the recipe and put in the package so a client can import the package?

https://docs.conan.io/en/latest/reference/conanfile/methods.html#cpp-info
You already did. When passing cxxflags, any Conan generator will inject those flags in the generated file.


Implementation of a secondary VM for the contract runtime that will execute WebAssembly

@RitaMAllenCA @rafal-ch
Yesterday's meeting notes:

Good progress so far, we have basic input passing, calling exports
On contract SDK there are macro-based exports hiding all the unnecessary boilerplate, and basic compile-time schema generator.
There's an established convention for passing inputs (previously:
Abstract key-value storage makes it possible to hook up the casper-storage trie.

Further steps:

Contract Versioning is a core feature, we should keep it in the new VM
Keep in mind backwards compatibility RuntimeArgs (we think it will work out of the box by passing serialized RuntimeArgs)
Next, focus on creating smart contracts, and calling into smart contracts.
Keep in mind backward compatibility and upgrade story for 1.x contracts with regards to named keys etc.