Update status to NOTE

c.f. #6, this should now be ready to publish.
@ylafon to add status text

Yay! Sorry I bailed on this. Glad you're picking it up.


Fix/search widget

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


Defect: PartsAssist: Carousal Block is not as expected as cards navigation is not displayed; Text size is incorrect

Steps:

Navigate to https://main--vg-partsasist--hlxsites.hlx.page/ from index.docx by clicking preview
Validate the  Carousal block

Actual:
Carousal Block is not as expected as cards navigation is not displayed; Text size is incorrect

Will be fixed wtih #81

Validated on partsassist.com (postgolive) and able to see the correct results, agree to defect closure:


setup a supply_main file similar to demand_main file

This file needs to be run to get the following files:

Geothermal.csv
SWP.csv
PV.csv
PVT_35.csv
SC_60.csv
SC_75.csv

Right now they are being shipped along with the reference-case-zug

@JIMENOFONSECA : Can you let me know how the above mentioned 'csv' files are being created. For 'reference-case-zug', the above mentioned files are already present in 'baseline -> outputs -> data -> potentials', whereas for 'reference-case-open', the 'potentials' folder has files with names as 'xx_PV' and 'xx_sensors'. I think these might be used in 'photovoltaic.py' file which requires sensors_data

Geothermal.csv

I suggest to take it out form the optimization
SWP.csv
I suggest to take it out from the optimization
PV.csv, PVT_35.csv, SC_60.csv, SC_75.csv
Created from the PV scripts in technologies. they are being redeveloped in #336 and arre waiting to merge

Ok will work on this after #336 is merged

At the moment, the results of the PV generation are saved for individual buildings.
The old results 'PV.csv' are the aggregation of the entire district.
@BhargavaKrishnaSreepathi , how does the current optimization procedure handle this information, does it decide the amount of panel installed at the building level?

documentation Thuy-an.PDF
@shanshanhsieh : Nope. It actually takes the total area and then assigns a percentage of this area for PV, PVT and SC. It is not done at building level.

@BhargavaKrishnaSreepathi do you see this be finished in spring May?
I personally would put it to the first priority of the coming sprint, since it will have to wait until #636 is merged as well.

@shanshanhsieh: I think it will take longer than that, as both of us have other issues to wrap up.

This is not necessary anymore, as the guide of running CEA is made available. Which includes a separate section for potentials calculation. The remaining steps are already incorporated into optimization in various PRs


Disable the rebase-merge button

As informally discussed in Slack, we disable the rebase-merge button, and just leave the squash-merge button to ensure that one commit (in the unstable branch) corresponds to one PR, so that the cherry-pick procedure can be more simple during releasing new versions.

Thanks all. Merging...


HW sync undefined behaviour

Required Info

Camera Model
D415

Firmware Version
5.9.14

Operating System & Version
Win10

Platform
PC

SDK Version
2.13 & 2.14

Language
C++

Segment

Issue Description
Hello, doing some experiments with HW syncing.

Does the HW sync imply Internal (Depth & Color Sensor) Sync? That is, are the depth-color streams of the same device synced? (I think that the answer is NO).
Setting a device as MASTER (and forgetting about any other device) seems to have undefined behavior regarding color. I monitored the following:

The device does not produce any color frames
the device produces color and depth frames, but not at the correct rate (MASTER's clock)
The devices hangs.
What may cause that behavior?

Even if I set the device as master, and then open the rs-viewer, the color stream is not available. However, the depth stream seems to be OK.
Any help is really appreciated.
P.S.: Does HW sync concern only the depth sensor or the color too? If it does apply only for the depth sensor, is there a way to achieve Depth-Color Sync?

[Realsense Customer Engineering Team Comment]
Hi
Here is how to enable the Sync feature on D415:
0) Refer to the attached picture;

RS-viewer must be version 2.13.0 or newer; Firmware version must be <IP_ADDRESS> or newer;
Depth and Color sensors must be set at same frame rate (i.e. 30fps )
"Inter Cam Sync Mode" set to "1.000"; (Default: 0.000; Master: 1.000; Slave: 2.000).
Color sensor, the "Enable Auto Exposure" must be Unchecked !!!
Keep in mind: Always start Depth camera first, then start the Color sensor; (Depth is master, Color is Slave)
To verify the Sync feature, see the number under icon "i"; they should be same. (see attached picture: "30.01" fps for both Depth and Color)

Thanks

Hello,
Thank you for your response. The steps I followed are just the same as the ones you mention (however there is no attachment). The device's behavior is still random. (I noticed that with the 2.14 rs-viewer the rgb sensor does not stream at all, but with the 2.13 sometimes it works as expected and sometimes hangs. Sometimes I get a Hardware Notification about a RT MIPI Error.)
The following is a link to a gif that captures my screen and presents the randomness I mentioned.
(The file was bigger than what github allows, so I uploaded it on imgur).
rs-viewer 2.13 screen capture

[Realsense Customer Engineering Team Comment]
Hi
After you turn off RGB camera (under Slave mode), please wait 5 seconds to let it fully discharge and then you can turn on RGB camera again. This usually can bring up RGB camera normally.
Thanks

Hello! Is the "Inter Cam Sync Mode" setting in the Realsense viewer? Or we should set it by code?

[Realsense Customer Engineering Team Comment]
Hi
Are you asking Sync feature on a single camera or on multiple cameras?
Here is the Whitepaper regarding camera sync usage: see "Multi-camera_Configurations_whitepaper.PDF"
https://realsense.intel.com/intel-realsense-downloads/#whitepaper
Thanks

Hi,
I have managed to have perfect time-synchronized flow (all RGB+Depth at 30 FPS) between two cameras with the realsense viewer app. But when I try to do the same without the app GUI, RGB wants always to be the master over depth (29.8 FPS instead of 30 FPS) and only the depth flows of the two cameras seems time-sync, not the rgb flows since the RGB and depth flow of the same device are not sync. I have tried to launch depth sensor before rgb sensor with sensor.start but without success.
Is there any way to force the Depth sensor as the master over RGB?


questions about grouth.npz

hi
thank you for sharing your excellent work!
I am trying to make my ground_truth.npz for my own dataset. but I have some questions:

does 'utmQ' and 'utmQ' contains the utm coordinates of all pictures in query and database dataset?
I have create my datasets with 7 places. Each place have 400 pictures for database and 80 pictures for query. is the number of pictures enough for training (simply using the traning code provided for maliipary dataset)?
I have try training on my own dataset (descirbed in question 2.). And I got the following result:

====> Calculating recall @ N
====> Recall@1: 0.0600
====> Recall@5: 0.1200
====> Recall@10: 0.1800
====> Recall@20: 0.3000
====> Recall@50: 0.6000
====> Recall@100: 0.8200
I wonder the result is good or not?
I am looking forward to your reply , thank you!

Hi,

Yes
No, this is most likely not sufficient. But why not use a pre trained model and use that to extract features on your dataset?
The results look poor considering there are only 7 places.

You might want to read our recent “Visual Place Recognition: A Tutorial” paper for some more intuitions.


BigFraction - double constructor

Maybe I don't understand this constructor correctly:

https://github.com/Hipparchus-Math/hipparchus/blob/3c785280519b0c351003cf890f2ff4e917f6a354/hipparchus-core/src/main/java/org/hipparchus/fraction/BigFraction.java#L272

but what I'm looking for is a conversion where the result n - numerator and d - denominator of the v - double value satisfy the condition:
|(n/d) - v| < (10^c) / (d^2)

with c = -4 for example.

What about adding a stream generator that would produce a Stream<BigFraction>? This way, you could decide by yourself when to select the fraction you want using the Stream API, for example with filter and findFirst.

What about adding a stream generator that would produce a Stream<BigFraction>? This way, you could decide by yourself when to select the fraction you want using the Stream API, for example with filter and findFirst.

That seems to be a good idea. Do you have an example how to do it?

I have added static methods convergents in both Fraction and BigFraction to generate streams of fractions.
You can look at the unit tests to see how one specific convergent can be selected from the stream.
Beware that your criterion with a threshold of 10^c/d^2 will probably never be fulfilled. One theorem for errors bounds in continued fractions states that |α - pₖ/qₖ| < 1/(qₖqₖ₊₁). With this boundary, the numerator is 1 and not 10^c, and the denominator qₖqₖ₊₁ is bigger than qₖ^2 as the denominator series is an increasing series.

I have added static methods convergents in both Fraction and BigFraction to generate streams of fractions. You can look at the unit tests to see how one specific convergent can be selected from the stream.

I also commented this on the commit:
Shouldn't the spliterator created from the generatingIterator() also report the ORDERED characteristics, since the elements should not be arbitrarily reordered?
Spliterator.DISTINCT | Spliterator.ORDERED


Array Plus Array in Code Wars plus test file

Array Plus Array in Code Wars
test file included in the test folder
:)

Thanks for the update 👍


Safari src/srcset behavior

What problem does this feature solve?
Safari on iOS and macOS (other browsers not sure?) render an <img> tag src attribute initially before calculating an <img> tag srcset attribute. This basically means the following 2 javascript examples behave differently:
// example 1, srcset first
const newImage = new Image();
newImage.srcset = 'https://domain.com/image.jpg?w=200 200w';
newImage.src = 'https://domain.com/image.jpg';
document.body.appendChild(newImage);

// example 2, src first
const newImage = new Image();
newImage.src = 'https://domain.com/image.jpg';
newImage.srcset = 'https://domain.com/image.jpg?w=200 200w';
document.body.appendChild(newImage);

Safari will send 1 network request for for the srcset URL for example 1, but 2 network requests for both the src and srcset URLs for example 2. I believe most people would not expect example 2 behavior.
This behavior occurs in Vue templates because of how the template compiles down to javascript. So writing:
<img
  :src="someSrc"
  :srcset="someSrcset"
>

which compiles down to something like (pseudocode) in vue:
...
("img",{src:someSrc, srcset:someSrcset})
...

will experience double network requests in Safari.
I wonder if vue could optimize the template generation to append the srcset attribute first when constructing an image tag? I know there's some funny business around object property order and simply rearranging src and srcset in the object above might have some caveats... so this request, although simple, may have some unappreciated complexities. I at least wanted to start the conversation around what I think is a seemingly trivial and common example for anyone using srcset, and how Safari behaves. Simply put, a common practice could be resulting in basically double (or more!) bandwidth usage, let alone degrading the render time based on image loading priority.
What does the proposed API look like?
If this is something that makes sense to address, there would be no changes to the developer-facing side. I imagine it would just be an edge case hardcode in vue template compiler for inserting srcset before src potentially?
Totally understand if this isn't "vue's" problem to solve, just logging something I found unexpected. I really think this change would immediately impact performance significantly for anyone using srcset.

Replicated this behavior as well for safari mobile
https://github.com/vuejs/core/issues/6391#issue-1324834994

Same as https://github.com/vuejs/core/issues/4680#issuecomment-927166733

Just leaving a follow up note that the same problem/behavior happens in safari desktop/mobile for a template like:
<img
  :srcset="https://domain.com/image.jpg?w=200 200w, https://domain.com/image.jpg?w=100 100w"
  sizes="50px"
>

What happens is, Safari eagerly loads the srcset before the sizes attribute has been appended by vue, and so it will actually load the 200 width image and then immediately afterward load the 100 width image.
In my opinion, this behavior effectively makes using srcset broken within vue + safari. I understand this isn't really vue's fault, but bring it up again as unexpected behavior. To me, this makes it slightly larger than an "edge" case, as this is most certainly affecting every safari hit using srcset in any fashion with an img tag in vue.

I've experienced the same problem when working with Angular 15. I couldn't even imagine that the order of src and srcset attributes does matter.
Thanks for finding out what the reason is.

Hi everyone, thanks for reporting this issue!
The related Safari bug tracker ticket: https://bugs.webkit.org/show_bug.cgi?id=190031
Perhaps someone here can help get it fixed.

That's a long shot, but maybe you could help getting the bug fixed, @jensimmons?


Text is cut off in render_ui_to_texture example with high scale factors.

Or perhaps a better title is "bevy_ui text rendering always uses the primary window's scale factor, even when rendering to a texture."
Bevy version
9f8db0d
Relevant system information
AdapterInfo { name: "Apple M1 Max", vendor: 0, device: 0, device_type: IntegratedGpu, driver: "", driver_info: "", backend: Metal }
SystemInfo { os: "MacOS 13.6.2 ", kernel: "22.6.0", cpu: "", core_count: "10", memory: "64.0 GiB" }

What you did
Have high DPI screen or override scale factor with
.add_plugins(DefaultPlugins.set(WindowPlugin {
    primary_window: Some(Window {
        resolution: WindowResolution::default().with_scale_factor_override(1.),
        ..default()
    }),
    ..default()
}))

cargo run --example render_ui_to_texture
What went wrong
Cube says This is a cu.

Additional information
Related issue: https://github.com/bevyengine/bevy/issues/5621
If it is true that we now generally support "window-independent ui scaling," then it looks like some code in bevy_ui needs to be updated: https://github.com/bevyengine/bevy/blob/39cca41f3f602b19e9c9596e46fc3093bfadd553/crates/bevy_ui/src/widget/text.rs#L234-L238
If not, that issue may need to be reopened, and perhaps this one closed.

As of https://github.com/bevyengine/bevy/pull/10559, this should probably be thought of as per-camera UI scaling rather than per-window UI scaling.

As far as I can tell, text_system was just missed in #10559, and it's a bit awkward because text_system is also used in the context of Text2d.

This was fixed by #13697


move gateway into unified docker file

This is the last step to unify the various dockerfiles, move the gen-grpc-gateway container build into the root-level container.

Neat.


Code Form - Get existing data

In order to update the information, the form should be prefilled with the existing information about a project.
When a user selects an administration and an project, the fields should be populated with existing data so that they can be modified.

Surveys.io might be useful for forms.


Sign out doesn't work properly

Issue
When a user clicks sign out from the menu, he will be directed to the login page. But when we open the app again, we can directly go to map activity without sign in.
Steps to reproduce

Open app
Choose Signout from the menu
Close the app
Open app again

@CloudyPadmal The next time, please state clearly whether you want to work on the issue or someone else can claim it to avoid conflics.


$expand operation not respecting the filter parameter

Describe the bug
When we perform a $expand operation on our FHIR server on the US Core Condition Code ValueSet, found here, the server ignores the filter parameter. This seems to only happen if we initiate the $expand operation using the /ValueSet/$expand?filter=fracture&url=http://hl7.org/fhir/us/core/ValueSet/us-core-condition-code format. If we make the request to /ValueSet/us-core-condition-code/$expand?filter=fracture the filter parameter is respected. This behavior seems to happen for both normal value sets and expanded value sets. We have confirmed that this behavior can be reproduced using the service-category, found here.
To Reproduce
Steps to reproduce the behavior:

Start the HAPI FHIR Starter project
PUT the service-category CodeSystem, found here

curl -X PUT -H "Content-Type: application/json" http://localhost:8080/hapi-fhir-jpaserver/fhir/CodeSystem/service-category -d @{codeSystemFile}

PUT the service-category ValueSet, found here

curl -X PUT -H "Content-Type: application/json" http://localhost:8080/hapi-fhir-jpaserver/fhir/ValueSet/service-category -d @{valueSetFile}

Wait for the ValueSet to be expanded
Make a request to http://localhost:8080/hapi-fhir-jpaserver/fhir/ValueSet/service-category/$expand?filter=Hos
Make a request to http://localhost:8080/hapi-fhir-jpaserver/fhir/ValueSet/$expand?filter=Hos&url=http://hl7.org/fhir/ValueSet/service-category

Expected behavior
I expected both queries to respect the filter parameter and return an expansion with one element:

Instead the request with the ValueSet id explicitly provided respects the filter parameter while the query using the url parameter does not.
Screenshots
Using the ValueSet id:

Using url parameter:

Request Logs:

Environment:

HAPI FHIR Version: 5.0.2
OS: Ubuntu 18.04
Using built-in h2 backend for reproduction but it happens in our Postgresql backed environment as well

Please let me know if there is any additional information that you would need to diagnose the issue.

Duplicate of #2189

Closing, as #2189 is fixed. Please feel free to comment if this is not resolved.

I just checked with master branch build and this is unfortunately still not resolved. Weirder still, calling expand with ValueSet id now returns error
/ValueSet/administrative-gender/$expand?filter=fema
{     "resourceType": "OperationOutcome",     "issue": [         {             "severity": "error",             "code": "processing",             "diagnostics": "Resource ValueSet/administrative-gender is not known"         }     ] }
while calling
/ValueSet/$expand?url=http://hl7.org/fhir/ValueSet/administrative-gender&filter=fema
returns:
{     "resourceType": "ValueSet",     "status": "active",     "compose": {         "include": [             {                 "system": "http://hl7.org/fhir/administrative-gender"             }         ]     },     "expansion": {         "identifier": "f3cd5f01-2731-4013-9695-0d94ff30eda0",         "timestamp": "2021-02-16T21:57:01+01:00",         "offset": 0,         "parameter": [             {                 "name": "offset",                 "valueInteger": 0             },             {                 "name": "count",                 "valueInteger": 1000             }         ],         "contains": [             {                 "system": "http://hl7.org/fhir/administrative-gender",                 "code": "male",                 "display": "Male"             },             {                 "system": "http://hl7.org/fhir/administrative-gender",                 "code": "female",                 "display": "Female"             },             {                 "system": "http://hl7.org/fhir/administrative-gender",                 "code": "other",                 "display": "Other"             },             {                 "system": "http://hl7.org/fhir/administrative-gender",                 "code": "unknown",                 "display": "Unknown"             }         ]     } }

This is still not resolved in 5.4.0. Here's a test that reproduces the issue:
	 * A valueset expand with url and filter
	 */
	@Test
	public void testExpandValueSetContainingSystemIncludeWithCode() throws IOException {
		CodeSystem cs = loadResourceFromClasspath(CodeSystem.class, "/r4/iar/CodeSystem-iar-citizenship-status.xml");
		myCodeSystemDao.create(cs);

		ValueSet vs = loadResourceFromClasspath(ValueSet.class, "/r4/iar/ValueSet-iar-citizenship-status.xml");
		myValueSetDao.create(vs);

		ValueSet expansion = myValueSetDao.expandByIdentifier(
			"http://ccim.on.ca/fhir/iar/ValueSet/iar-citizenship-status",
			ValueSetExpansionOptions.forOffsetAndCount(0, 1000).setFilter("fugee"));
		ourLog.info(myFhirCtx.newJsonParser().setPrettyPrint(true).encodeResourceToString(expansion));

		assertEquals("Refugee", expansion.getExpansion().getContains().iterator().next().getDisplay());
	}


Fix track seeking not setting the track's time

Seems to not work correctly. The position is set, but in the next update cycle, the current time of the track isn't actually at the track's very end. Not sure why this is the case.


Jan 31 linked to Feb 1 in book

Hi !  The early February dates are off in the book for a week or less, I think by Feb one being linked to Jan 31.  I'm enjoying just now starting to go through your posts. Thank you for the resources!

Hi, thanks!
Can you please double-check if this is really the case?
I went through all tweets for Jan/Feb 2021, and couldn't see any which were incorrectly dated. Please let me know if you still see any inconsistency.

I’m new to GitHub and wondering if I can just reply here.  The date says Feb 1 but post from Jan 31 (then Feb 2 to Feb 1 post, and so on) until it gets to Feb 5 matches to Feb 5.   Unless I’m missing something.  I was making a table of posts I might want to refer back to later (I’m a new R user) and to share with my team who is going to start learning R soon, so noticed the discrepancy from the book date and twitter post date just for 4-5 days (that is as far as I’ve gotten).  Not a big deal at all but I wanted to mention since your book opening page mentioned to report things.  ☺
<EMAIL_ADDRESS>From: Indrajeet Patil @.>
Sent: Monday, February 7, 2022 10:51 AM
To: IndrajeetPatil/R-Function-A-Day-book @.>
Cc: Jodi Fender @.>; Author @.>
Subject: Re: [IndrajeetPatil/R-Function-A-Day-book] Jan 31 linked to Feb 1 in book (Issue #1)
• This message originated from outside the Long Beach Unified School District.  Do NOT click links or open attachments unless you recognize the sender and know that the contents are safe.
Hi, thanks!
Can you please double-check if this is really the case?
I went through all tweets for Jan/Feb 2021, and couldn't see any which were incorrectly dated. Please let me know if you still see any inconsistency.
—
Reply to this email directly, view it on GitHubhttps://github.com/IndrajeetPatil/R-Function-A-Day-book/issues/1#issuecomment-1031802156, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APZUQFK34OKOFPCQSPDZGH3U2AIBJANCNFSM5NYISGYA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>


[New game]: Cosmic Coin Collector

🎮 Game Request
Welcome to Cosmic Coin Collector, a thrilling space adventure where you embark on a mission to collect coins, dodge obstacles, and overcome hurdles while exploring a distant planet. Your goal is to gather as many coins as possible, destroy asteroids, and cover the maximum distance to achieve the highest score. Get ready to immerse yourself in an action-packed journey through the cosmos!
Game Logic:
Controls:

Use the arrow keys or swipe gestures to navigate your spaceship.
Press the spacebar or tap on the screen to shoot projectiles and destroy asteroids.

Gameplay:

The game is set on a visually stunning planet with a variety of landscapes, including mountains, canyons, and forests.
Coins will be scattered throughout the planet, appearing at random locations.
As you traverse the planet, various hurdles and obstacles will come in your path, such as meteor showers, enemy spacecraft, or laser barriers.
You must maneuver your spaceship skillfully to avoid collisions with the hurdles and collect as many coins as possible.
Some coins may require precise movements or special maneuvers to reach.
To destroy asteroids, aim and shoot projectiles by pressing the spacebar or tapping on the screen. Each destroyed asteroid adds to your score.

Point down the features
Scoring:

The score is calculated based on three factors: coins collected, asteroids destroyed, and total distance traveled.
Each coin collected adds a certain number of points to your score.
Destroying an asteroid grants additional points, depending on its size and difficulty.
Distance traveled contributes to your score. The longer you survive and explore, the higher your score will be.

Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

Hey @U-DAY-333 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


#1246 Unable to set oneofselect in MSE when reusing previous selection

fixes #1246
Signed-off-by: Jesus Rocha<EMAIL_ADDRESS>Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

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

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

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

@caritaou yes to all three

externalize Select string
move ST_mse_table_II into test
add unit tests

Any comments on the functional code fix? the UX (https://ibm-analytics.slack.com/archives/CQ378CWSJ/p1667945181474089)?

@JesusGuerrero I tested the code and it works well. Im just hesitant on the setTimeout part, it seems like a hack. I wonder if theres a better way to handle this

@JesusGuerrero I tested the code and it works well. Im just hesitant on the setTimeout part, it seems like a hack. I wonder if theres a better way to handle this

@caritaou I hear you on that, I will try to find another route. The setTimeout( fn, 0) is to queue immediately after the digest of the redux & componentUpdate. The sender has to set the value on the store, then store has to digest and update all the listeners, and then the store needs to get updated to a null value. Don't know any easier way to do this other than adding a queue to the store, and having the receiver pop (run function) the queue once it matches it by key.

@JesusGuerrero When testing this PR, when I select multiple rows and change common value to "Football". Rows are correctly updated to "Football". However, MSE dropdown still shows "Select" instead of "Football".


cargo-miri panic's on no_main binaries

While there are many other questions about what miri should do for binaries with #![no_main], the current behavior is to panic. In the spirit of #1001, it would be nice explicitly indicate that miri doesn't support testing freestanding binaries at the moment.

An example backtrace

[-bash] Wed 17 Jun 16:40 [[master $] /Volumes/code/helena-project/tock/boards/hail]
$ cargo miri test
    Checking hail v0.1.0 (/Volumes/code/helena-project/tock/boards/hail)
warning: unused import: `core::panic::PanicInfo`
 --> boards/hail/src/io.rs:2:5
  |
2 | use core::panic::PanicInfo;
  |     ^^^^^^^^^^^^^^^^^^^^^^
  |
  = note: `#[warn(unused_imports)]` on by default
warning: unused import: cortexm4
--> boards/hail/src/io.rs:4:5
|
4 | use cortexm4;
|     ^^^^^^^^
warning: unused import: kernel::debug
--> boards/hail/src/io.rs:5:5
|
5 | use kernel::debug;
|     ^^^^^^^^^^^^^
warning: unused import: kernel::hil::led
--> boards/hail/src/io.rs:7:5
|
7 | use kernel::hil::led;
|     ^^^^^^^^^^^^^^^^
warning: unused import: crate::CHIP
--> boards/hail/src/io.rs:10:5
|
10 | use crate::CHIP;
|     ^^^^^^^^^^^
warning: unused import: crate::PROCESSES
--> boards/hail/src/io.rs:11:5
|
11 | use crate::PROCESSES;
|     ^^^^^^^^^^^^^^^^
warning: static is never used: WRITER
--> boards/hail/src/io.rs:17:1
|
17 | static mut WRITER: Writer = Writer { initialized: false };
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= note: #[warn(dead_code)] on by default
thread 'rustc' panicked at 'no main function found!', src/tools/miri/src/bin/miri.rs:35:37
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
error: internal compiler error: unexpected panic
note: the compiler unexpectedly panicked. this is a bug.
note: we would appreciate a bug report: https://github.com/rust-lang/rust/blob/master/CONTRIBUTING.md#bug-reports
note: rustc 1.45.0-nightly (fe10f1a49 2020-06-02) running on x86_64-apple-darwin
note: compiler flags: -C opt-level=z -C embed-bitcode=no -C debuginfo=2 -C debug-assertions=on -C incremental
note: some of the compiler flags provided by cargo are hidden
warning: 7 warnings emitted
error: could not compile hail.
To learn more, run the command again with --verbose.

I've attached a backtrace, but any no_main binary will crash in a similar manner right now.
CC https://github.com/tock/tock/issues/1713

Thanks! I agree.
This is an easy bug to fix, well-suited as a first contribution. The panic arises here:
https://github.com/rust-lang/miri/blob/146ee66268bb38a226eb9737d1a595bcb2ac1dbd/src/bin/miri.rs#L35
And all it takes is replacing that by a better error message. rustc has some APIs for fatal errors; early_error might be suited here.


feat: RTL Support

Description

This pull request implements RTL support and closes #390
Complete

[x] Accordion
[x] Alert
[x] Asset
[x] Asset List
[x] Avatar
[x] Banner
[x] Bar Loader
[x] Breadcrumbs
[x] Button
[x] ButtonGroup
[x] Calendar
[x] Card
[x] Checkbox
[x] Circle Loader
[x] Coach Mark
[x] Cycle Button
[x] Dialog
[x] Drop Indicator
[x] Dropdown
[x] FieldLabel
[x] Icon
[x] IllustratedMessage
[x] InputGroup
[x] Menu
[x] Miller Columns
[x] Popover
[x] Quick Actions
[x] Radio
[x] Rating
[x] Rule
[x] Search
[x] Sidenav
[x] Site
[x] SplitButton
[x] SplitView
[x] StatusLight
[x] Stepper
[x] Switch
[x] Table
[x] Tabs
[x] Textfield
[x] Toast
[x] Treeview
[x] Typography
[x] Well
[x] StepList
[x] Pagination - Added spectrum-Pagination-nextButton and spectrum-Pagination-prevButton for button style examples
[x] Slider - supports switch from LTR to RTL, but not the other way around

Discussion required

[x] Tooltip - labels and icons render nicely, but left/right/bottom/top variants are physical, not logical. More discussion required with the React team to see if we want to replace this.

Support not required

[x] Dropzone
[x] FieldGroup
[x] Label
[x] Link
[x] Underlay

How and where has this been tested?

How this was tested: Lots of scrolly scroll. Pro tip: Hit Control + R to go to RTL, Control + N to go back to LTR 😎
Browser(s) and OS(s) this was tested with: Chrome on macOS

See the docs site here: https://git.corp.adobe.com/pages/lawdavis/spectrum-css-rtl/docs/

Design Review
Every component was checked against the Spectrum documentation to ensure it conforms to Internationalization Guidelines, if present.
Key:
✅- Conforms
✴- Not a stable component, no guidelines
⚠- Stable component, no guidelines
Components

⚠ Accordion - no guidelines, conforms to result of conversation on slack
✴ Alert - no guidelines (contribution)
✴ Asset - no guidelines (contribution)
✴ Asset List - no guidelines (contribution)
✅ Avatar - n/a (contribution)
✴ Banner - no guidelines (Badge contribution)
✅ Bar Loader - conforms to guidelines. Note: Percent sign must be moved manually
✴ Breadcrumbs - no guidelines (contribution)
⚠ Button - no guidelines, pinged on slack
✅ ButtonGroup - n/a
✴ Calendar - no guidelines (contribution)
⚠ Card - no guidelines
✅ Checkbox - conforms to guidelines.
✅ Circle Loader - conforms to guidelines.
✅ Coach Mark - conforms to guidelines. Note: x of y text must be manually mirrored (default directionality will make it of y x, examples include <bdo dir="ltr">2 of 8</bdo> to override this)
✅ Combobox - conforms to guidelines.
✅ Cycle Button - n/a (contribution)
⚠ Datepicker - conforms to Combobox, no guidelines (contribution)
✅ Dialog - conforms to guidelines.
✅ Drop Indicator - n/a (Drag and Drop contribution)
✅ Dropdown - conforms to guidelines.
✅ FieldLabel - conforms to Textfield guidelines.
⚠ Form - no guidelines, may need insight on label position in the future
⚠ Icon - no guidelines, but probably n/a
✴ IllustratedMessage - no guidelines, but probably n/a (contribution)
✴ Label - no guidelines (Badge contribution)
✅ Menu - conforms to Dropdown guidelines.
✅ Meter - conforms to Bar Loader guidelines.
✴ Miller Columns - no guidelines (contribution)
✴ Pagination Button - no guidelines (contribution)
✴ Pagination Explicit - no guidelines (contribution)
✴ Pagination Page - no guidelines (contribution)
⚠ Popover - no guidelines
⚠ Quick Actions - no guidelines
✅ Radio - conforms to guidelines.
⚠ Rating - no guidelines
✅ Rule - n/a
✅ Search - no guidelines
✅ Sidenav - conforms to guidelines.
✴ Site - no guidelines (K9 Panels contribution)
✴ SplitButton - no guidelines (contribution)
✴ SplitView - no guidelines (not even a contribution, no idea why this exists)
✅ StatusLight - conforms to guidelines.
✴ Stepper - no guidelines (Numeric Stepper Field contribution)
✅ Switch - conforms to guidelines.
✅ Table - conforms to guidelines.
✅ Tabs - conforms to guidelines.
✅ Textfield - conforms to guidelines.
✅ Toast - conforms to guidelines.
✴ Treeview - no guidelines (contribution)
✅ Typography - n/a
✅ Tooltip - conforms to guidelines, discussed on slack
✴ Well - no guidelines (Wells contribution)
✴ StepList - no guidelines (contribution)
✅ Slider - conforms to guidelines. Note: Percent sign must be moved manually

Screenshots

To-do list

[x] If my change impacts other components, I have tested to make sure they don't break.
[x] If my change impacts documentation, I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.

[x] Check for Spectrum RTL guidelines for each component, ensure we're following them
[x] File issues against Design on JIRA for anything we need their help or insight from
[ ] Fix DecoratedTextfield icon issue
[ ] Fix Checkbox position in Card
[ ] This pull request is ready to merge.

I added a VR test for this to inspect the default view. https://spectrum-visual-regression.ci.corp.adobe.com/job/spectrum-css-vr-test-asset/230/artifact/spectrum-css/backstop_data/html_report/index.html
There are a couple of default visual issues before RTL:

did you intentionally increase the viewbox of the component previews?

first chevron of the calendar is flipped

Miller columns are now cutting of the left-sided icons

toggles have increase in width

Could you please maybe not set the entire documentation to RTL?
I remember @NateBaldwinDesign  asked to only have the component preview switched to RTL.
I see this screenshot
https://user-images.githubusercontent.com/201344/74490054-30365600-4e7c-11ea-9ca9-2266ee757014.png
This PR will also have a significant impact how we write our CSS. Instead of using left, right top and widths we now have to use inset etc.
I am wondering why we have to progress with our own custom implementation where we are breaking-change every component and not use a more commonly used extension like RTLCSS
https://rtlcss.com/learn/usage-guide/options/

Good catches, @bernhard-adobe!
Viewbox of component previews
That looks wrong. It should not have that fixed height. Will fix.
Calendar icons
I saw that one myself and noted to fix it already. Thanks!
Miller columns
Good catch!
Toggles
I had to refactor some of the stuff RSP V3 had added for RTL support on the animations, looks like I messed something up. Good catch!
Entire documentation to RTL
I will try to address that. Note that, if we do this, we need to be CERTAIN that any :dir(rtl) specific hacks come after the :dir(ltr), otherwise this will break.
It is much more reliable to set the entire document to RTL, as that is closer to how things will happen in the wild.
RTL affecting how we write CSS
We're not writing a custom implementation for this at at all. In fact, we're not using the W3 specification for Logical Properties. See the MDN article on Logical Directions for an introduction to the new properties.
We did not write our own plugin for this, we're using the postcss-logical plugin to implement this specification, which is over 3 years old and has 2.2 million weekly downloads.
The RTLCSS approach is backwards to how we do things; it takes the old way of writing CSS and makes it work with RTL by generating a separate stylesheet. We want to do the opposite; we want to write CSS the way we'll be writing CSS in the future (with logical properties), and use PostCSS plugins to transform that code to work in the old way (like we do with Babel in the JS world).
I agree that it's a big change, but this prepares us for the future by letting us write future-CSS today. This is the PostCSS way, and exactly what we do/did with Custom Properties, Nesting, Focus Ring, etc. It's going to be hard for our brains, but it's better in the long run! In the mean time, I wrote a Sublime Text plugin that helps me transform the old style left/border-left/width etc into the new style to assist my learning, it's really helpful.

New visual regression test: https://spectrum-visual-regression.ci.corp.adobe.com/job/spectrum-css-vr-test-asset/231/artifact/spectrum-css/backstop_data/html_report/index.html

Decorated TextField Multiline example, the alert icon is shift out of the textarea.

Card component, the checkbox is not center.

@lazd Calendar


fix(theme-default): display sidebar emphasis text

Summary
Fixed the issue of the right sidebar not being displayed.
Related Issue
#952
Checklist

[ ] Tests updated (or not required).
[ ] Documentation updated (or not required).

Why is my CI always reporting errors on ut-macs due to test-title-suffix? @Timeless0911

Why is my CI always reporting errors on ut-macs due to test-title-suffix? @Timeless0911

fixed in #1022 , you can rebase the main branch


inconsistent rendering of amounts in xdr viewer

Amounts are returned with what seems to be an arbitrary form.
An amount value is returned as either a raw value or both fixed point and raw
Examples:
a trustline will show up with fields (right next to each other):

balance:<PHONE_NUMBER>25 this is the raw value
limit:<PHONE_NUMBER>00 (raw:<PHONE_NUMBER>000000000) both are returned

Lumens are returned the same way, for example in AccountEntry, you can see balance: 105032704 this is raw but in offers and payments it's the fixed point.
In general, I would imagine that any int64 that is supposed to be an amount should be rendered the same way.

Is this still an issue? An XDR to reproduce would be helpful, I'm not quite sure what I'm looking for.

yes, you can look at this example
Account and Trustline have balance in raw only, like balance: 487864711
Offer amount and Trustline limit are mixed.
Looks like right now the only gap is on balance so rendering it as mixed would probably fix this issue

Would be great to create type Amount int64 type in stellar-core so finding fields that should be rendered as amounts would be easier.

Fixed in new Lab


x/mobile: ios, No matching provisioning profiles found

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
1.7
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN="/Users/apple/workspace/go/bin"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/apple/workspace/go"
GORACE=""
GOROOT="/usr/local/go"
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/wp/ff6sz9qs6g71jnm12nj2kbyw0000gp/T/go-build159594453=/tmp/go-build -gno-record-gcc-switches -fno-common"
CXX="clang++"
CGO_ENABLED="1"

What did you do?
building this: https://github.com/golang/mobile/tree/master/example/network
What did you expect to see?
On android i get the green / blue screen as i expect.
On IOS i expected the same.
What did you see instead?
x-MacBook-Pro:network apple$ gomobile build --target=android
x-MacBook-Pro:network apple$ gomobile build --target=ios
gomobile: xcrun xcodebuild -configuration Release -project /var/folders/wp/ff6sz9qs6g71jnm12nj2kbyw0000gp/T/gomobile-work-486467550/main.xcodeproj failed: exit status 65
=== BUILD TARGET main OF PROJECT main WITH CONFIGURATION Release ===

Check dependencies
Code Sign error: No matching provisioning profiles found: No provisioning profiles with a valid signing identity (i.e. certificate and private key pair) matching the bundle identifier “org.golang.todo.network” were found.

** BUILD FAILED **

The following build commands failed:
	Check dependencies
(1 failure)

```

I get same ERROR ! How to fix it!

i never worked it out..
I can build other things for IOS, just not with gomobile

This is a duplicate of #12660, check there for workarounds.

hey @viru
Thats does sound exactly like what i experience too.  Thanks for letting me know.
The problem is that its not really fixed. The fix is to hardcode a namespace into the gomobile tool and then recompile.
@eliasnaur
Could someone on the gomobile team think about fixing this obvious tooling bug ?
What might be best is a command line flag for compiling for ISO ? This woudl then get ride of the hardcoded constant in the common code as shown at the https://github.com/golang/go/issues/12660

I agree that it seems like an obvious tooling bug, but unfortunately iOS provisioning and signing is not obvious to me. I'll gladly review any CL that attempts a fix.

@shurcool did you ever get past this ??

No, I haven't built for mobile recently. My latest status was https://github.com/golang/go/issues/12660#issuecomment-145306339.

in summary, for now I guess i will also just have to maintain my own fork and hack it too.
@shurcooL
thanks for update.
@eliasnaur
i doubt i have the skills to fix this. IOS signed etc is also newish to me, and when it works i am shocked half the time :)

I'm looking into fixing this.
I think the first solution should be to allow specifying the BundleID (and Android package ID) manually and direct people to create an application in XCode with the specified identifier. This would make gomobile easier to use now, with little effort. PS: Currently when I tried to use an existing profile then xcodebuild also required to specify DEVELOPMENT_TEAM (that can be detected from the provisioning profile).
Then as a next step there are few ideas:

Try to figure out how to create provisioning profile automatically (command-line tools or write the code). Unfortunately, I haven't yet found a nice way to do it. We can create BundleID with this https://play.golang.org/p/ZiOkpaU3HP... but this could still be problematic when you want to build application that is already registered and you don't have the signing key.
Use wildcard AppID for multiple apps (https://developer.apple.com/library/content/qa/qa1713/_index.html) -- this would avoid the need to create manually multiple profiles. There probably are restrictions when doing it in this way.
Combine XCode and gomobile. When selecting a new project there are templates "Empty" and "External-Build-System"... maybe we can use XCode to trigger "gomobile" with the correct parameters. Or create a custom template for gomobile.

Any thoughts?

PS: it seems that #12660 is pretty much the same issue? Should the discussion be moved there and this closed as a duplicate?

@egonelbre Thanks for picking this up.
yes #12660  is the same. i will close this, so it can be worked on at #12660


Support GGShadow service, add 'update' method

Hi @rpedigoni hope you won't mind merging back to master? Thanks for the interest and input!

@dzimine, README updated, can you please merge it? Thanks!

Sorry for not paying attention for a while I was out on something else. Merged now. Thanks!


RuntimeError: view size is not compatible with input tensor's size and stride

Hi,
I'm trying to run the code with the command like below,
python test_region_sequence.py --dataset coco --exp_name ours --sample_rl_nw
and
python test_region_set.py --dataset coco --exp_name ours --sample_rl_nw
, which are COCO Entities with CIDEr + NW optimization.
However, both give me same error like below,
Traceback (most recent call last): File "test_region_sequence.py", line 138, in <module> out, _ = model.beam_search((detections_i, ctrl_det_seqs_test_unique), eos_idxs=[text_field.vocab.stoi['<eos>'], -1], beam_size=5) File "/home/tejasrii/show_control_tell/models/CaptioningModel.py", line 151, in beam_search selected_logprob, selected_idx = torch.sort(seq_logprob.view(b_s, -1), -1, descending=True) RuntimeError: view size is not compatible with input tensor's size and stride (at least one dimension spans across two contiguous subspaces). Use .reshape(...) instead.
What should I do here..?
Thank you for sharing the code! :)
and Hope be safe from COVID-19.

Figured out with changing ".view()" to ".reshape" in line 151 CaptioningModel.py


 Try a subway or something :)
Then, any command that sends analytics, i.e. dart run or dart test hangs for a period of time.
To "fix", disable analytics (which ironically itself hangs for that same period):
dart --disable-analytics

Analytics collection should run "out of band", that is, it should not be blocking (I would not use await). I might go an additional step further and say that perhaps the actual mechanism that sends the analytics to a server should happen elsewhere, i.e. I should never have this happen:
dart run empty_file_with_void_main.dart
# ... script clearly executes, but 'dart' appears to hang for 10-30s

I might check with the flutter CLI folks to see how they handle it.

In our previous implementation of analytics it was possible to get a future for the last item sent, wait on that for a brief time w/ a 100ms-200ms timeout, and exit the process if there was a delay in sending (to not hang the VM exit waiting for analytics). We've since re-written the analytics implementation; I'm not sure who's responsible for the integration into the dart cli, but @bkonyi would be the best person to start with.

In our previous implementation of analytics it was possible to get a future for the last item sent, wait on that for a brief time w/ a 100ms-200ms timeout, and exit the process if there was a delay in sending (to not hang the VM exit waiting for analytics). We've since re-written the analytics implementation; I'm not sure who's responsible for the integration into the dart cli, but @bkonyi would be the best person to start with.

Yeah, I'm responsible for the CLI. I thought Elias had done some work to account for this (I know the legacy analytics timeout behavior was brought up), but maybe this never ended up happening.

So this call: https://github.com/dart-lang/sdk/blob/main/pkg/dartdev/lib/dartdev.dart#L262
Should default to timing out after 250ms: https://github.com/dart-lang/tools/blob/main/pkgs/unified_analytics/lib/src/analytics.dart#L548
I wonder if there is some other code path by which we don't run this finally? Or some other async call to unified_analytics is hanging, before we get to the finally?

Hmm, when we timeout, we call _gaClient.close();, but I'm thinking maybe that doesn't cancel our pending requests?

If I trace the code, it eventually gets to <IOClient>.close(), which does a .close(force: true), which should cancel pending requests.

If I trace the code, it eventually gets to <IOClient>.close(), which does a .close(force: true), which should cancel pending requests.

Ok, that makes sense. Also, assuming the flutter cli tool is not susceptible to this bug, then presumably Analytics.close() is closing these connections.

It also happen to me on a newly created project, it consistently freeze for 20 seconds before exiting when analytics is enable.

CC @DanTup
ref: https://github.com/Dart-Code/Dart-Code/issues/3270


Classroom create form, improved styles, and copy link functionality

What this PR does:

A few improvements to the classroom management table to get closer to the design
Copy link functionality for the classroom join link
Adds the Toast grommet component to show success of copying the join link. This component could be used to notify success of classroom creation and other actions. It auto-closes on its own. This is instead of the Notification component which has to be dismissed and is much larger in visual space. I'm open to using either/or/both, so let me know your thoughts.
A non-functional UI for the classroom create form. You can view it by clicking on the "Create Classroom" button

This can all be tested by using zootester1 using production database.
Next steps are more style improvements and making the create form functional.

@shaunanoordin Thanks for the thorough review. For your questions:

I think we need to confirm teacher registration this week for both I2A and WD as well as what the API expects. My guess is that it is not required for I2A.
Yes, there is a classroom details view that I haven't started. I imagine an edit button there that basically opens the same form that has inputs filled with current values. In that case, I can probably generalize the create form (I like to start with specific use cases then generalize if applicable)
Agreed about the join link

Apologies for being unclear. I am working on the functionality on a separate branch, so this can merge. I'll likely have the functional form branch ready today.

👍  on all of the above, and (I like to start with specific use cases then generalize if applicable) gets an extra 👌  just for being sensible.
OK, let's merge this!


Multiple language support in single file

I'm html web coder. I'm writing my html codes in to PHP file. I can change my language mode PHP to HTML form rigth bottom. But sometimes I'm using php codes in my file. Is there any multiple language support for single file ?

Don't change the mode from PHP to HTML. Instead, install a PHP extension:
E.g. https://marketplace.visualstudio.com/items?itemName=bmewburn.vscode-intelephense-client
File issues against the extensions if features are missing.


[Bug]: Text overlap from actual widget size when Label name is long text in Select family widgets

Is there an existing issue for this?

[X] I have searched the existing issues

Description

Steps To Reproduce

Add any select widget to canvas
Add label name as long text
Observe the size of the widget and see the long text. It is overlapping the canvas too

Public Sample App
No response
Version
Cloud - Release

Being fixed in #7064

@somangshu,
I think the issue has already been fixed by the PR at https://app.zenhub.com/workspace/o/appsmithorg/appsmith/issues/10600.
Could you please check that PR if it is actually fixing the issue?
After that, if this issue is not covered by that PR, please be more specific with the requirements about this PR

fixed in #10600


Update Readme

I'm just adding the links for the Chrome and Firefox extension to the readme

Thank you :heart:


removing PKI API and corresponding support

GRPC API does not need PKI and we can have a different clients using same certificates provided by same CA. CA by HDS is not used and will not be useful

If using an external CA:
On server:

Create CA
Generate server KP

On client:

Generate client KP
Submit public key to server to sign and generate client certificate
Download client's certificate
Download CA's certificate

For revocation; see CRL and OCSP support.

If using an external CA:
On server:

Create CA
Generate server KP

On client:

Generate client KP
Submit public key to server to sign and generate client certificate
Download client's certificate
Download CA's certificate

For revocation; see CRL and OCSP support.


1.2 panic on selecting top from subquery

Observed on 1.2 branch at commit e59d3f9f0b73ed312cae158f654990afa486b6ac.
Insert this data:
cpu,region=west,host=w u=1
cpu,region=west,host=w u=2
cpu,region=west,host=w u=3
cpu,region=east,host=e u=10
cpu,region=east,host=e u=20
cpu,region=east,host=e u=30

Execute this query:
select top(m, host, 3) from (select mean(u) as m from cpu)

Then the server process panics with:
panic: runtime error: index out of range

goroutine 126 [running]:
panic(0x60b480, 0xc42000c180)
	/usr/local/Cellar/go/1.7.3/libexec/src/runtime/panic.go:500 +0x1a1
github.com/influxdata/influxdb/influxql.filterFloatByUniqueTags(0xc421351ce0, 0x1, 0x1, 0xc4221dd9b8, 0x1, 0x1, 0x7b4788, 0xc421351ce0, 0xa3cd60, 0x1121989da0)
	/Users/mr/go/src/github.com/influxdata/influxdb/influxql/call_iterator.go:1020 +0x2b6
github.com/influxdata/influxdb/influxql.NewFloatTopReduceSliceFunc.func1(0xc421351ce0, 0x1, 0x1, 0xc421b69ba8, 0xc421b69c98, 0xc421b69c98)
	/Users/mr/go/src/github.com/influxdata/influxdb/influxql/call_iterator.go:814 +0x45b
github.com/influxdata/influxdb/influxql.(*FloatSliceFuncReducer).Emit(0xc421d848e0, 0xc421b69ba8, 0xc4201d80e9, 0x3)
	/Users/mr/go/src/github.com/influxdata/influxdb/influxql/functions.gen.go:111 +0x44
github.com/influxdata/influxdb/influxql.(*floatReduceFloatIterator).reduce(0xc4201863c0, 0xc420028ef0, 0x0, 0x18, 0xc420028f70, 0x46e312)
	/Users/mr/go/src/github.com/influxdata/influxdb/influxql/iterator.gen.go:1107 +0x807
github.com/influxdata/influxdb/influxql.(*floatReduceFloatIterator).Next(0xc4201863c0, 0xc4201a0000, 0x20002, 0xc420028f6c)
	/Users/mr/go/src/github.com/influxdata/influxdb/influxql/iterator.gen.go:1020 +0xa1
github.com/influxdata/influxdb/influxql.(*floatInterruptIterator).Next(0xc421d84800, 0x46e286, 0x180001, 0x0)
	/Users/mr/go/src/github.com/influxdata/influxdb/influxql/iterator.gen.go:777 +0x52
github.com/influxdata/influxdb/influxql.(*bufFloatIterator).Next(0xc421d84820, 0x46e2b4, 0x180001, 0x0)
	/Users/mr/go/src/github.com/influxdata/influxdb/influxql/iterator.gen.go:95 +0x3c
github.com/influxdata/influxdb/influxql.(*floatAuxIterator).stream(0xc421d84860)
	/Users/mr/go/src/github.com/influxdata/influxdb/influxql/iterator.gen.go:874 +0x32
created by github.com/influxdata/influxdb/influxql.(*floatAuxIterator).Start
	/Users/mr/go/src/github.com/influxdata/influxdb/influxql/iterator.gen.go:860 +0x3f
exit status 2

The same query on master as of 0b018caf876ae3c6fdb1b9816cc6bcf0301763a1 does not panic.

This is not a new regression. The same data and query panics on 1.2.0, 1.2.1, 1.2.2, and 1.2.3 tags as well.

Can you try master? It may have gotten fixed by #8081, but this one may have slipped through since host isn't available in the subquery.

That query did not panic on master as of 0b018ca (May 4). I didn't bisect to see the exact commit where it stopped panicking.

Then I'm going to close the issue for the moment. It seems like it got resolved with the general refactor of that section of code.

This panic takes down the 1.2 server process; it is not a recovered panic inside a query.

@rbetts is the non-recoverable panic worth trying to mitigate for 1.2 and worth a release?

I’m content to leave this unfixed on 1.2 given the scope of changes to the sub-query implementation between 1.2 and 1.3. We’ve been advising users with subquery issues to move to 1.3 for performance reasons as well.


Maybe insight into closed issue 46?

Reference: https://github.com/openai/tiktoken/issues/46
M1 Mac
macOS 13.3.1
python 3.9 and 3.10
Trying to install Microsoft guidance, which uses titoken as a dependency. First I got this:
ERROR: Could not find a version that satisfies the requirement tiktoken>=0.3 (from guidance) (from versions: none)
ERROR: No matching distribution found for tiktoken>=0.3 (from guidance)

So I tried pip install tiktoken and this error
ERROR: Could not find a version that satisfies the requirement tiktoken (from versions: none)
ERROR: No matching distribution found for tiktoken

Just going through the issues I thought I would try and force python 3.9 with python3.9 -m pip install tiktoken. That worked, and subsequently so did python3.9 -m pip install guidance
Anyway, my guess is those closed issues were also ARM Macs, and this is a specific issue with having 3.10 installed. Thanks.

You're probably not using the Python you think you're using, which is why pip install tiktoken fails, while python3.9 -m pip install tiktoken works. I'd figure out what Python that pip is pointing to, but yeah, in general python3.9 -m pip install tiktoken or python3.10 -m pip install tiktoken is going to be harder to mess up.

Same error as described by @TradingPlacesResearch trying to install tiktoken to use with guidance.
This time ubuntu.


Hoping the code will be soon released

Hey Yurri, first of all thanks for your interest in SIGNeRF. We are still polishing the code and waiting for responses of reviewers. So sadly we still need a bit of time, expect a code release at the end of february. If you need any comparative material for your research feel free to reach out via email and tell us the specifics you need.

Thanks a lot for your reply. Wish a desired result with your paper.

Sorry for the delay, the code is live now 😃


Ensure a plugin isn't reported as passed if no results were read

What this PR does / why we need it:
In the case where the e2e plugin reported results (a tarball) but
it did not contain an xml file of junit results we want to be sure
to NOT count this as a success.
It may be reasonable to consider this a failure but I think it is
more generic/safe to just report it as unknown and users can
handle that case differently.
Also, I found another bug which was causing daemonsets to not
properly be bubbling up their results.
Which issue(s) this PR fixes
Fixes #840
Special notes for your reviewer:
Release note:
Fixes a bug which caused `sonobuoy results` to report success if no results were processed at all. Modified this case to return 'unknown' instead.

Codecov Report

Merging #841 into master will increase coverage by 0.08%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #841      +/-   ##
==========================================
+ Coverage   46.19%   46.28%   +0.08%
==========================================
  Files          75       75
  Lines        4788     4790       +2
==========================================
+ Hits         2212     2217       +5
+ Misses       2446     2445       -1
+ Partials      130      128       -2

Impacted Files
Coverage Δ

pkg/client/results/processing.go
81.81% <100%> (+0.48%)
:arrow_up:

cmd/sonobuoy/app/status.go
57.54% <0%> (-1.89%)
:arrow_down:

pkg/plugin/aggregation/aggregator.go
83.2% <0%> (+4%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0fb9865...61fe463. Read the comment docs.


Fix Nix/Crane build of m1-da-light-node

Summary

RFCs: $\emptyset$.
Categories: cicd.

Changelog

Updated nix/m1-da-light-node.nix to fix crane build
Added nix/m1-da-light-node-relative-paths.patch to patch movementxyz/aptos-core deps

Testing
I don't currently know enough about the project to perform adequate testing of the resulting binary, but when run with no args it returns Error: Token not provided.
Outstanding issues

hey @johnbchron thanks for making this  PR .
On a Mackbook M2 I am getting this error.
Note: the entire team here at Movement Labs uses Macbooks, so this would need to be fixed. :)
nix build --extra-experimental-features "nix-command flakes"  '.#m1-da-light-node'
error:
       … while calling the 'derivationStrict' builtin
         at <nix/derivation-internal.nix>:9:12:
            8|
            9|   strict = derivationStrict drvAttrs;
             |            ^
           10|

       … while evaluating derivation 'm1-da-light-node-0.3.0'
         whose name attribute is located at /nix/store/asymc3nsl739p1wwr0w6xbjnqs3qb94p-source/pkgs/stdenv/generic/make-derivation.nix:331:7

       … while evaluating attribute 'buildInputs' of derivation 'm1-da-light-node-0.3.0'
         at /nix/store/asymc3nsl739p1wwr0w6xbjnqs3qb94p-source/pkgs/stdenv/generic/make-derivation.nix:378:7:
          377|       depsHostHost                = elemAt (elemAt dependencies 1) 0;
          378|       buildInputs                 = elemAt (elemAt dependencies 1) 1;
             |       ^
          379|       depsTargetTarget            = elemAt (elemAt dependencies 2) 0;

       (stack trace truncated; use '--show-trace' to show the full trace)

       error: Package ‘systemd-255.4’ in /nix/store/asymc3nsl739p1wwr0w6xbjnqs3qb94p-source/pkgs/os-specific/linux/systemd/default.nix:896 is not available on the requested hostPlatform:
         hostPlatform.config = "aarch64-apple-darwin"
         package.meta.platforms = [
           "aarch64-linux"
           "armv5tel-linux"
           "armv6l-linux"
           "armv7a-linux"
           "armv7l-linux"
           "i686-linux"
           "loongarch64-linux"
           "m68k-linux"
           "microblaze-linux"
           "microblazeel-linux"
           "mips-linux"
           "mips64-linux"
           "mips64el-linux"
           "mipsel-linux"
           "powerpc64-linux"
           "powerpc64le-linux"
           "riscv32-linux"
           "riscv64-linux"
           "s390-linux"
           "s390x-linux"
           "x86_64-linux"
         ]
         package.meta.badPlatforms = [
           {
             isStatic = true;
             parsed = { };
           }
         ]
       , refusing to evaluate.

       a) To temporarily allow packages that are unsupported for this system, you can use an environment variable
          for a single invocation of the nix tools.

            $ export NIXPKGS_ALLOW_UNSUPPORTED_SYSTEM=1

          Note: When using `nix shell`, `nix build`, `nix develop`, etc with a flake,
                then pass `--impure` in order to allow use of environment variables.

       b) For `nixos-rebuild` you can set
         { nixpkgs.config.allowUnsupportedSystem = true; }
       in configuration.nix to override this.

       c) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
         { allowUnsupportedSystem = true; }
       to ~/.config/nixpkgs/config.nix.

https://github.com/movementlabsxyz/movement/actions/runs/9462360759/job/26065152548
Commenting on this PR triggered one of our pipelines that has a runner for Macs :)

I added the necessary MacOS inputs and confirmed that it works on aarch64-darwin.

comment to trigger pipeline

I confirm that the build worked on my macbook


Don't use es6 classes

This is a contentious issue I realize, but using es6 classes in a library necessitates its use in any apps which consume that library. Many people are on the fence about es6 classes, and would prefer not to have them leak into their apps.

Hey Danny, thanks for opening the issue.
I agree that we should avoid classes when it's feasible, but I'm not sure that's the right choice for Model classes. Other parts of the codebase could be done without classes, though, but they're mainly internals.
The difficulty is that for models, you'll often want to chain methods. You add some validation on top of create and update. Then you add some hooks on top of that. Etc. That's super easy to do with classes super. That's harder with composing.
I'd be happy to hear you out if you have an idea of how a class-less implementation would look like for the user!

That makes sense, js doesn't make it easy to do that kind of classical inheritance without rolling your own  system.
Thanks for making this repo! Dealing with related data in my redux app was a huge blocker, but now it looks like I'm going to finish on time!


Move Calibre to be an AU package

@majkinetor can I get you to take a look at this PR?
I have tried to follow what you did for the python3 package, but I wanted to get your opinion on it.  Would you have done things differently?
Thanks!

Perfect.


Enable providers parent classes for usage across app not internally

Remi Rousselet, please make the parent classes of providers public so that we can use them and create our own components and custom providers. This is something we are really missing. Classes such as AsyncNotifierBase, AutoDisposeAsyncNotifierProviderImpl, etc.

Those are voluntarily not public. I have no plan to expose a way to make custom providers, as this would heavily restrict the evolution of Riverpod. That is because any new feature likely would mean a breaking change for folks who make custom providers.
Instead I'd suggest creating an issue explaining what you think is missing in Riverpod. We can then discuss how to implement those features without having to create custom providers.
Riverpod is fairly flexible. You should be able to extend its featureset without having to create custom providers.

Please take a look at this code. I created an extension that checks for errors in the provider before extracting data from it, because if an error has already occurred, it will keep returning it. However, since I don't have access to AutoDisposeAsyncNotifierProviderImpl, I couldn't make it more elegant, and as a result, I ended up with repetitions.
import 'package:flutter_riverpod/flutter_riverpod.dart';

extension RefExtension on Ref {
  Future<T> syncFutureProvider<T>(FutureProvider<T> provider) {
    if (exists(provider) && read(provider).hasError) {
      invalidate(provider);
    }
    return provider.future.read(container);
  }

  Future<T> syncAutoDisposeFutureProvider<T>(
      AsyncNotifierProvider<dynamic, T> provider) async {
    if (exists(provider) && read(provider).hasError) {
      invalidate(provider);
    }
    return provider.future.read(container);
  }

  Future<T> syncAsyncProvider<T>(
      AsyncNotifierProvider<dynamic, T> provider) async {
    if (exists(provider) && read(provider).hasError) {
      invalidate(provider);
    }
    return provider.future.read(container);
  }

  Future<T> syncAutoDisposeAsyncProvider<T>(
      AutoDisposeAsyncNotifierProvider<dynamic, T> provider) async {
    if (exists(provider) && read(provider).hasError) {
      invalidate(provider);
    }
    return provider.future.read(container);
  }

  Future<T> syncAutoDisposeAsyncProviderFamily<T, Z>(
      AutoDisposeAsyncNotifierProviderFamily<dynamic, T, Z> providerFamily,
      Z arg) async {
    final provider = providerFamily(arg);
    if (exists(provider) && read(provider).hasError) {
      invalidate(provider);
    }
    return provider.future.read(container);
  }

  Future<T> syncAsyncProviderFamily<T, Z>(
    AsyncNotifierProviderFamily<dynamic, T, Z> providerFamily,
    Z arg,
  ) async {
    final provider = providerFamily(arg);
    if (exists(provider) && read(provider).hasError) {
      invalidate(provider);
    }
    return provider.future.read(container);
  }
}

Thank you for your response, we will be waiting.


Sharing a project with multiple files

@bamurtaugh spotted this.
For the Arcade extension for VS Code, if you create a project with multiple files, and then create a share link, we don't package up the additional files in the shared project.

Thanks for filing! An additional idea to consider is making this opt-in (i.e. asking users before automatically including other files).
My scenario: I was creating a project from a template in vscode.dev/makecode. I was notified I needed to have a folder open for this (which I hadn't expected), so I opened a random folder on my machine. In this case, I wouldn't have wanted any random files in that folder to be included in my bundled and shared project.


Nvidia/cuda cpp lang

This PR adds a language ID for CUDA (in collaboration with Bob Brown and Colen Garoutte-Carson)

Closing in favor of https://github.com/microsoft/vscode/pull/119444


Use PTP codes in includes where possible

It seems Fuji's protocol is close to, but not the same as PTP/IP. The port numbers are even close (55740 instead of 15740 like canon and nikon use). Event codes are clearly PTP/IP-similiar, ie 0x100E is fire shutter in Fuji and in PTP/IP too.
So im just saying for the future you could grab libptp's includes and see what else maps 1:1.

Thanks for pointing this out. I obviously didn't know much about PTP when I started the project. Actually @tengelmeier mentioned at some point that https://github.com/tengelmeier/mtp-tools/ (has wireshark scripts for PTP) may be useful, but I never tried it.
Libptp may also be useful to check what or if features are supported that are not implemented in Fuji's app.
I wouldn't just add the headers as it seems to be GPL code. So I would have to think about license issues.

regarding the wireshark plugin, it specifically mentions Fuji XT-1 so could be interesting to research new commands.

Yeah I tried playing with the mtp-tools wireshark plugin. It does surface a few extra commands, but many are missing. There's not enough there to reverse-engineer the camera communications to the point I need.

What are you missing in detail?
For me it works sufficient to kind of understand the communication - just make sure you have also the 'packet bytes' section in the window visible (it is collapsed by default)
The remaining bytes of the payload like the value for SetDeviceProperty or GetDeviceProperty appear  there. You'll see where the payload remainder begins when you select e.g. pop -> transactionID in the Packet details pane.

The detail i'm missing is the codes themselves. For example, which code
means a new photo is ready, how do I get its identifier, and download the
photo? Which codes put the camera in or out of live-view mode, and once
there, what are the codes to trigger the camera, or focus etc?
Some of these are in the MTP tool, which I appreciate very much, thanks!
Others are not there yet.
cheers
-tim
On Wed, Feb 22, 2017 at 5:06 PM Thomas Engelmeier<EMAIL_ADDRESS>wrote:

What are you missing in detail?
For me it works sufficient to kind of understand the communication - just
make sure you have also the 'packet bytes' section in the window visible
(it is collapsed by default)
The remaining bytes of the payload like the value for SetDeviceProperty or
GetDeviceProperty appear there. You'll see where the payload remainder
begins when you select e.g. pop -> transactionID in the Packet details pane.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hkr/fuji-cam-wifi-tool/issues/3#issuecomment-281820033,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADmfLkbi7VpnrzryZwYXPcLl4wWjX4Mrks5rfLFqgaJpZM4Lw_Fw
.

OK, I see what you mean.
Basically PTP uses a mix of standardized commands / identifiers and allows vendor specific extensions.
Identifiers for these vendor extensions begin with 0xA (commands), 0xC (events) and  0xD (properties) - you'd need to reverse engineer the meaning for these identifiers.
Live view e.g. is for sure vendor specific as the 15740 standard has no commands for that feature.
Regarding device properties, there are three different PTP/IP host modes the camera uses ('PTP mode', 'PC mode', and then there is the mode where new images are sent to the computer). there is a good chance there are different properties for each of these modes available (I could also imagine the GUID in the OpenSession packets are used as kind of protocol identifier).
At least my X-T1 responds with an error to most of the device property codes sniffed when connected to an Mac via USB, so I'm missing something.
In most of the packet sniffs I also see usually the property 0xDF01 is set - it might also set an transfer or camera mode..

is there a link somewhere which describes the setup to intercept the communication between the app and the camera? I'm only interested in controlling the shutterspeed, should be detectable in the commands sent I think.

I reverse the official ios app, it has 3 class name which has "PTP" prefix.
PTPDeviecInfomation.h
PTPDeviecMacAddress.h
PTPDeviecModelInfomation.h


RSYNC Error - Error in RSYNC Protocol

Hey guys!
I can to run the action when deploying once of my sites, two days after a successful deploy to the env and I received this error rsync error: error in rsync protocol data stream (code 12) at io.c(228) [sender=v3.13.0_rc2-264-g725ac7fb] (full error log at the bottom). I have not changed the script over the either. Sometimes I get errors and I just run a redeploy and normally it sorts itself out and I noticed on this ticket, they kind of said the same thing here.
This is my script that takes the PHP files and transfers them to the WPE Env. I am not confident that this would reproduce the issue that I am having.
            - name: Deploy - Files to WPEngine Staging
              if: contains(github.ref, 'develop')
              uses<EMAIL_ADDRESS>              with:
                  # Deploy vars
                  WPE_SSHG_KEY_PRIVATE: ${{ secrets.WPE_SSHG_KEY_PRIVATE }}
                  WPE_ENV: ${{ secrets.WPE_INSTALL_STAGING }}
                  # Deploy Options
                  REMOTE_PATH: 'wp-content/themes/theme-name-2023/'
                  PHP_LINT: FALSE
                  FLAGS: -azvr --inplace --delete --exclude=".*"

Deploying refs/heads/develop to *** ...
# ***.ssh.wpengine.net:22 SSH-2.0-Go
prepping file perms...
file perms set...
Skipping PHP Linting.
OpenSSH_8.4p1, OpenSSL 1.1.1k  25 Mar 2021
debug1: Reading configuration data /etc/ssh/ssh_config
debug1: Authenticator provider $SSH_SK_PROVIDER did not resolve; disabling
debug1: Connecting to ***.ssh.wpengine.net [35.244.1[18](https://github.com/frankdigital/l1-2023-theme/actions/runs/8871757625/job/24355348368#step:7:19).20] port 22.
debug1: Connection established.
debug1: identity file /github/home/.ssh/github_action type 3
debug1: identity file /github/home/.ssh/github_action-cert type -1
debug1: Local version string SSH-2.0-OpenSSH_8.4
debug1: Remote protocol version 2.0, remote software version Go
debug1: no match: Go
debug1: Authenticating to ***.ssh.wpengine.net:22 as 'wpe_gha+***'
debug1: SSH2_MSG_KEXINIT sent
debug1: SSH2_MSG_KEXINIT received
debug1: kex: algorithm<EMAIL_ADDRESS>debug1: kex: host key algorithm: rsa-sha2-512
debug1: kex: server->client cipher: aes128-ctr MAC<EMAIL_ADDRESS>compression: none
debug1: kex: client->server cipher: aes128-ctr MAC<EMAIL_ADDRESS>compression: none
debug1: expecting SSH2_MSG_KEX_ECDH_REPLY
debug1: Server host key: ssh-rsa SHA256:T8IoIgBRJ3gDjA6/q7i33+nSrrpVfZipYtxo1rqvww4
Warning: Permanently added '***.ssh.wpengine.net,35.244.118.[20](https://github.com/frankdigital/l1-2023-theme/actions/runs/8871757625/job/24355348368#step:7:21)' (RSA) to the list of known hosts.
debug1: rekey out after<PHONE_NUMBER> blocks
debug1: SSH2_MSG_NEWKEYS sent
debug1: expecting SSH2_MSG_NEWKEYS
debug1: SSH2_MSG_NEWKEYS received
debug1: rekey in after<PHONE_NUMBER> blocks
debug1: Will attempt key: /github/home/.ssh/github_action ED25519 SHA256:9aTyrv1zIjzV4ZISKG28S8/ukVPohHhePjcp4qh2gCY explicit
debug1: SSH2_MSG_EXT_INFO received
debug1: kex_input_ext_info: server-sig-algs=<ssh-ed25519,sk-ssh-ed25519@openssh.com,sk-ecdsa-sha2-nistp256@openssh.com,ecdsa-sha2-nistp256,ecdsa-sha2-nistp384,ecdsa-sha2-nistp5[21](https://github.com/frankdigital/l1-2023-theme/actions/runs/8871757625/job/24355348368#step:7:22),rsa-sha2-256,rsa-sha2-512,ssh-rsa,ssh-dss>
debug1: kex_input_ext_info<EMAIL_ADDRESS>(unrecognised)
debug1: SSH2_MSG_SERVICE_ACCEPT received
debug1: Authentications that can continue: publickey
debug1: Next authentication method: publickey
debug1: Offering public key: /github/home/.ssh/github_action ED25519 SHA256:9aTyrv1zIjzV4ZISKG28S8/ukVPohHhePjcp4qh2gCY explicit
debug1: Server accepts key: /github/home/.ssh/github_action ED25519 SHA256:9aTyrv1zIjzV4ZISKG28S8/ukVPohHhePjcp4qh2gCY explicit
debug1: Authentication succeeded (publickey).
!!! MASTER SSH CONNECTION ESTABLISHED !!!
Authenticated to ***.ssh.wpengine.net ([<IP_ADDRESS>]:[22](https://github.com/frankdigital/l1-2023-theme/actions/runs/8871757625/job/24355348368#step:7:23)).
debug1: setting up multiplex master socket
debug1: channel 0: new [/github/home/.ssh/ctl/53cb216414143769a5486a537f80b41ec28b72c2]
debug1: forking to background
debug1: Entering interactive session.
debug1: pledge: id
OpenSSH_8.4p1, OpenSSL 1.1.1k  25 Mar 2021
debug1: Reading configuration data /etc/ssh/ssh_config
debug1: Authenticator provider $SSH_SK_PROVIDER did not resolve; disabling
debug1: multiplexing control connection
debug1: channel 1: new [mux-control]
debug1: channel 2: new [client-session]
debug1: Sending command: rsync --server -vlogDtprze.iLsfxCIvu --delete --inplace . sites/***/wp-content/themes/l1-capital-20[23](https://github.com/frankdigital/l1-2023-theme/actions/runs/8871757625/job/24355348368#step:7:24)/
debug1: mux_client_request_session: master session id: 2
debug1: client_input_channel_req: channel 2 rtype exit-status reply 0
Failed to create shell
debug1: channel 2: free: client-session, nchannels 3
debug1: channel 1: free: mux-control, nchannels 2
rsync: connection unexpectedly closed (0 bytes received so far) [sender]
rsync error: error in rsync protocol data stream (code 12) at io.c(228) [sender=v3.13.0_rc2-264-g7[25](https://github.com/frankdigital/l1-2023-theme/actions/runs/8871757625/job/24355348368#step:7:26)ac7fb]

Version information

Action version<EMAIL_ADDRESS>

Yep I get the same error too when I push to deploy.
Manually re-running the action in github deploys successfully.

Happening all the time now. Even if I re-run the failed job. The last 2 lines of the error are:
rsync: connection unexpectedly closed (0 bytes received so far) [sender]
rsync error: error in rsync protocol data stream (code 12) at io.c(231) [sender=3.2.7]

Yep - I am finding the same thing. I have an open support ticket with WPE so I will post anything they find here

Same issue here. I have contacted you support team twice, and they haven't helped me out. Now waiting for a solution a third time.

The past two days, I encountered the same issue. I was able to update a static image file, but could not upgrade ACF through the composer.json file on another project. I have also reached out to the support team.

Got it fixed on my end. WPE customer service rep, James H., suggested updating exclude.txt and ignoring mu-plugins/wp-cache-memcached. He mentioned that this was a recent addition to their must-use plugins.

This was fixed for me by contacting the WPEngine support team.
The supporter wrote:
_"I have gone ahead and cleared the git cache on the server for the environment as stuck caches here can cause issues like the one you are seeing more recently.
Please try the git push again and let us know if the changes are still not visible."_
After this, it worked again.

Hey everyone, we have had the same bottleneck the last week.
We couldn't figure out it on our own, so we asked WPEngine Support to deep into it.
if you have the same issue, the best solution would be to ask WPEngine Support about help.
Here is WPEngine comment regarding the issue:

So the issue was related to a git related node on the shared server on which youre hosted here. As this issue is related to a shared server node, we cannot disclose specifics for security reasons.
Regarding the comment you wish to leaveon the github thread, if any other users experience this issue, you can simply advise that they can contact us here in support and we can ensure the same changes are made for any other servers experiencing this issue

Just updating here that this is still an issue in some environments.

@jakemason @jaireina This seems to be a different issue. I'm having trouble even SSHing into my server. This seems to be related to this incident.

Yeah, same issue is just very widespread during this incident. Failure to create a shell for SSH in my case and the OPs.

The rsync error is secondary to the main error, "Failed to create shell" (about 4 lines above the end of the log). The "Failed to create shell" error is produced in the SSH Gateway and is an intermittent failure. It's not something the GitHub Action has any control over, unfortunately. The best thing to do is to re-run your workflow. If that doesn't help, please contact WP Engine Support (as many of you have done, thank you!). You can use the following to help them pinpoint your issue:

I am unable to connect to my site through the SSH Gateway. The error I receive is "Failed to create shell".

Again, this error will usually resolve itself quickly barring any active incidents on the WP Engine Status page.
We understand that even a temporary deployment interruption is frustrating and hope to have a better solution for this issue in the future.

Got it fixed on my end. WPE customer service rep, James H., suggested updating exclude.txt and ignoring mu-plugins/wp-cache-memcached. He mentioned that this was a recent addition to their must-use plugins.

Thanks for raising this @martylouis! It's unrelated to the "Failed to create shell" issue, but mu-plugins/wp-cache-memcached has been added to our default excludes list in the latest release.


Remove whitespace from type property descriptions

Closes #188

Codecov Report

Merging #189 (46be217) into main (a05c9eb) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #189   +/-   ##
=======================================
  Coverage   93.65%   93.65%
=======================================
  Files          17       17
  Lines         252      252
=======================================
  Hits          236      236
  Misses         16       16

Flag
Coverage Δ

dotnet
93.65% <ø> (ø)

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


Sync with inria repo

@hjmjohnson This is ready to be merged. Notice that the CI has not been enabled.


Implement New Toggle Switch Design Component

Description:
We need to implement a new toggle switch component that adheres to our design system. The component should have three main states:

Active State - Represents the "on" state of the toggle.
Inactive State - Represents the "off" state of the toggle.
Hover State - Visual feedback when a user hovers over the toggle.

The hover state should be configurable to be either active or inactive depending on the needs of the application. Additionally, when the hover state is active and a user hovers over the toggle, a tooltip should appear. The content of the tooltip will be dynamically fetched from the locales (translation) files to support internationalization.
Acceptance Criteria:

The toggle switch must have distinct visual representations for both active and inactive states.
Hover effects must be implemented and should be able to be enabled or disabled via a prop.
When hover is enabled and active, a tooltip should appear with content sourced from the appropriate locales file.
The component should be keyboard accessible, allowing toggling via the spacebar or enter key.
The component should be tested across all supported browsers.
The component's state should be controllable via props for use in controlled component scenarios.
Ensure that the component is accessible and meets WCAG 2.1 AA standards.
The implementation should be responsive and work well on both desktop and mobile devices.
The new element is available in storybook and well described

Functional Design Component Description (to be loaded into storybook):
## Functional Design Description: Toggle Switch Component

### Overview
The toggle switch component is a user interface element that allows users to switch between two states: on (active) and off (inactive). This component is commonly used for settings that can be enabled or disabled. The design is intuitive, requiring a simple click or tap to toggle the state.

### Usage
The toggle switch should be used when a user needs to perform an action that will turn a feature or setting on or off. It is not intended for actions that require submission or more complicated inputs.

#### Do's:
- Use the toggle switch for binary actions where a clear on/off or yes/no decision is needed.
- Ensure that the toggle's state is saved immediately or upon a form submission, as appropriate for the context.
- Provide visual feedback for all states: active, inactive, and hover (if enabled).
- Use the tooltip to provide additional information or clarification as necessary.
- Fetch tooltip content from locales files to ensure proper internationalization.

#### Don'ts:
- Do not use the toggle switch for actions that require more than a binary choice.
- Avoid using the toggle for critical features that require a confirmation step before activation.
- Do not clutter the interface with tooltips if they do not add value or necessary information.
- Do not use the toggle switch where a checkbox would be more appropriate, such as in a list of multiple options.

### States
1. **Active State (On):** The switch shows a visual indication that the feature is enabled. The active state should be distinct and easily identifiable.
2. **Inactive State (Off):** The switch shows that the feature is disabled. This state should be clearly different from the active state.
3. **Hover State:** When enabled, hovering over the toggle switch with a mouse or focusing with a keyboard will display a tooltip. This state is optional and can be enabled or disabled based on application needs.

### Tooltip
When hover is enabled, the tooltip appears providing additional context or information. The content of the tooltip is fetched dynamically from the locales files, supporting multiple languages and ensuring the component is ready for internationalization.

### Implementation Notes
- The component's state can be managed either through internal state or passed as props from a parent component.

Design Specs:

Refer to the design mockups attached to this ticket.
Use the color palette and typography defined in the design system.

size should be 60x30 for the first draft

@jjeroch great idea to add this component to the library!
Could we use the MUI toggle switch component as a reference? It covers all the best practice sizes, states and behaviours. The ON/OFF text would not be needed because it's a common interaction with the switch logic.
For the tooltip on hover: this should be included as an optional variation for the component, since it will be used in very special cases where detailed info is needed. It should not replace the base hover inteaction, that highlights the accessible component. A tooltip delay of 300-500 milliseconds would ensure that it's not conflicting in the interaction.
Gladly check out the MUI component, it would be great to add hover and focused state, size options and tooltip as optional component setting to the AC.
Cofinity-UX is happy to provide support along the development process!

@kunalgaurav-bmw

please remote the text
check the hover functionality
align the state,size options

if you got questions raise you hand @Cofinity-UX  can support

as aligned today in the demo please ensure to remove the ON / OFF text


Search UI — DO NOT DELETE

Description
WIP draft of current search UI work
Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

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

Checklist:

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

Thanks, @icaraps ! You are reviewing early, which is fine, but I haven't gone through and spectrumized it or refactored or...Still implementing so all that stuff comes later tonight. But this is fine too. Gives me an extra verification list. :)

I'll create a new PR based on the backend changes here and get it merged, deployed and released ...

Here's the new PR https://github.com/adobe/gatsby-theme-aio/pull/456

Closing as this is old and the search ui is in.


gulpfile.js and files from Resources folder in 2.2.0

Hello.
When I install React.Core package from NuGet, the gulpfile.js and files from Resources folder have been added to the my project. Why is this happening? Do I need really these files? I cannot find anything about it in the documentation.
Thanks!

As far as i understand it correctly you can now tell reactjs.net to load your own reactjs version.
If you still want to use the React version from react.js.net you don't need these files.
If you want to use reactjs from e.g. npm this will help you to bundle the reactjs version.
These files are just helper starting points to do so.
And you have to tell your ReactConfig that you are loading reactjs by ding:
        ReactSiteConfiguration.Configuration
            .SetLoadReact(false)

Hmm, that's to supposed to happen! You don't need those files at all. What version of Visual Studio are you using, and what version of ASP.NET?
Sorry for the delay in replying, I was on vacation.

These files comes from folder content from React.Core.2.2.0.nupkg, this is the expected behavior for content files - Nuget include content of content folder to project while installing packages.
If these files are completely not required in end projects using React.Net (I've checked, seems works fine without them) they should be excluded from .nupkg. Simple way to do so is to mark these files with Build Action None in project React.Core (right now files marked with Build Action Content).

Oh yeah, that sounds like a mistake. I'll fix this up soon. Thanks :)
Sent from my phone.
On Feb 14, 2016 6:22 AM, "Sergey Fadeev"<EMAIL_ADDRESS>wrote:

These files comes from folder content from React.Core.2.2.0.nupkg, this
is the expected behavior for content files - Nuget include content of
content folder to project while installing packages.
If these files are completely not required in end projects using React.Net
(I've checked, seems works fine without them) they should be excluded from
.nupkg. Simple way to do so is to mark these files with Build Action None
in project React.Core (right now files marked with Build Action Content).
—
Reply to this email directly or view it on GitHub
https://github.com/reactjs/React.NET/issues/222#issuecomment-183897728.

Release ReactJS.NET 2.2.1 which fixes this. Thanks for the bug report!


Deprecate UntrustedWorkloadRuntime Option

https://github.com/containerd/cri/pull/891 added the Runtimes option in containerd config. We should use that in the future.
Deprecation plan:

[ ] Announce in containerd 1.2 release note:UntrustedWorkloadRuntime is being deprecated, users should switch to use the Runtimes option.
[ ] Remove UntrustedWorkloadRuntime option from the configuration in next minor release.
[ ] Remove io.kubernetes.cri.untrusted-workload annotation support when RuntimeClass is GA.

Wonderful! 👍

RuntimeClass approaching GA kep work is now in progress was discussed at the sig-node call today https://docs.google.com/document/d/1Ne57gvidMEWXR70OxxnRkYquAoMpt56o75oZtg-OeBg


Adds filler response

OPTIONAL feature add
Pros:

shaves off about 20% from the critical period latency (end of user speech to start of reply audio playback)
lightens the mood

Cons:

not a game-changing latency improvement
is one more component for users to engage with
users need to run a modal run src.fillers::Fillers.prefill command to activate this feature

@charlesfrye let's decide if this is worth it before I update docs.

If it's optional, I say we keep it!

Very fun addition, so let's stick with it!


Image name extends beyond tooltip when hovering over text

Etcher version: Beta 17
Operating system and architecture: Windows 10 x64
Do you see any meaningful error information on DevTools? N/A

When hovering over the name of an image that's long and does not have natural separators, the text extends beyond the tooltip box.

This can be fixed by adding word-wrap: break-word to .tooltip-inner.
I can't test this in the newest version of Etcher right now (see #1031) so this may have already been resolved... I recall some talk of replacing this with a modal.

Thanks a lot for reporting! The tooltip is still present on the latest version. We have some plans to provide a new UX for this, but we should definitely fix it in the mid time.

This should do it.


[FP]: wildfly for CVE-2022-1278

Package URl
<EMAIL_ADDRESS>CPE
cpe:2.3:a:redhat:wildfly::::::::
CVE
CVE-2022-1278
ODC Integration
None
ODC Version
9.0.5
Description
Actual vulnerable component wildfly-microprofile-health-smallrye-16.0.0.Final.jar
Since https://github.com/jeremylong/DependencyCheck/issues/6404 was failed to evaluate, please consider this ticket
more jars were reported with same vulnerability for below mentioned Package URIs too
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

Resolved with #6408


Re-respect rubocop files config

This PR https://github.com/prontolabs/pronto-rubocop/pull/36 basically reverts changes from this PR https://github.com/prontolabs/pronto-rubocop/pull/16 "Process files included in .rubocop.yml"
It happen because starting from https://github.com/prontolabs/pronto-rubocop/pull/36/files#diff-bd66c995e7650e0ca147e089eb3732f7a0b7bb0de38733f83e97485254285317R7 it passes down ruby_patches from Pronto's runner instead of the whole @patches, so it again ignores the files configured in .rubocop.yml. In our case, it ignores .jbuilder extensions

Thanks for the contribution, @tejanium! I'll make a release at the end of the week :+1:


Add status coloring for mosaic bounding box

Right now, NACs show their status & phase using fill and stroke colors, but the mosaic bounding box does not.

Done


TODO in the future: Remove replace directive in go.mod

Helm has bumped containerd to v1.6.6 https://github.com/helm/helm/pull/11093, this is a safe version.
runc is also upgraded due to the indirect dependency of containerd
So after Fybirk bumps Helm to v3.9.1 and above in the future, replace will not be needed in go.mod.
https://github.com/fybrik/fybrik/blob/f0dc75ebe926e1466196f487770d5ac6cdca0f0e/go.mod#L243-L246

Wow, cool, good news. Thanks


layer selection for uploads

[ ] I added an entry to CHANGELOG.md if knowledge of this change could be valuable to users.

Here is a brief summary of what I did:

 Upload Dialog: Layer auswahl

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

236 of 243 (97.12%)  changed or added relevant lines in 2 files are covered.
126 unchanged lines in 7 files lost coverage.
Overall coverage increased (+0.1%) to 89.38%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

services/src/handlers/datasets.rs
116
123
94.31%

Files with Coverage Reduction
New Missed Lines
%

operators/src/source/ogr_source/mod.rs
1
95.4%

services/src/handlers/datasets.rs
1
94.53%

services/src/pro/handlers/datasets.rs
1
95.69%

services/src/pro/users/postgres_userdb.rs
3
95.73%

operators/src/processing/neighborhood_aggregate/aggregate.rs
5
97.55%

services/src/pro/contexts/postgres.rs
6
96.61%

operators/src/source/gdal_source/mod.rs
109
94.05%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
97320

Relevant Lines:
108883

💛  - Coveralls

bors merge


Add support for custom prometheus metrics defined by the application

This is for https://github.com/RedHatInsights/topological_inventory-ingress_api/pull/87 and potentially other applications that are currently using the Common Metrics classes.
Basically it allows passing in a hash defined by the application to register prometheus metrics and make helper methods for them as specified.
First pass only supports counters, but maybe I'll implement support for histograms etc in the future.
example implementation: https://github.com/RedHatInsights/topological_inventory-ingress_api/pull/87
# TODO:

[x] specs

@syncrou I updated the implementation to only require :name, :type, and :description. This way it'll just define methods based on the name, the instance_var is just the name parameter with _#{type} appended since it's not really exposed by the api. Let me know what you think!

@lindgrenj6 - Looks good but seems to be some Travis issues.

@mkanoor would you mind taking a look?


Kyma resilience: Introduce component testing support in reconciler

Description
The progress tracking of a component (checking if a K8s resource was reaching the READY status) is not expressive and does not ensure that the component really works.
The reconciler has to provide an option to retrieve quality indicators for a component and to make a decision about next actions (e.g. reconcile-component, trigger self-healing mechanisms etc.)
AC:

Define options to measure the correct installation of a component (e.g. using Prometheus metric, further healtch checks, running an external test suite etc)
Agree on the support options the reconciler will offer to verify that a component got correctly installed
Enhance the reconciler to support automated component checks (could require contract extension with KEB to provide also the information how to test a component to reconciler)
Implement integration tests to verify that component testing works properly

Reasons
Establish more reliable approach to measure the "health" status of a deployed component.
Attachments

Follow-up action from retrospective meeting.

We agreed during the PO call today to not investigate into it as we have first to gather some experiences whether the currently used readiness-probes aren't working out for us.

Functionalities of the Reconciler were migrated to the new architecture, this issue will not be worked on.