Feature/issue 455 submodules

Looks good. (This article)[https://docs.travis-ci.com/user/private-dependencies/] will probably help with the Travis problem we're seeing.

Thanks @bensinjin , I went away from using a submodule for the content ... don't want to break the repo for users ourside our org, but still want the content to only be available to us, so the manual approach is still preferred, I think. ui-strings handling should be much improved though.

It probably makes sense to move some of the CSV files from the content to the backend repo, that way we will update the content repo less often, which will be nice since it's a little cumbersome to make changes there with the versioning etc.

@bensinjin @ian-lin-PG and @tomy-pg feel free to have a look at this PR, I think it's ready for review.


would be nice to merge improvements from https://github.com/zzaakiirr/confluence-markdown-exporter fork

Just FYI, the fork:
https://github.com/zzaakiirr/confluence-markdown-exporter
Contains some nice improvements for the script. I think it would be good idea to incorporate at last some of them here.

That fork is a bit old and I don't have the time to dig through everything right now.
However, if you wish to make pull requests that:

are rebased to HEAD
only contain one commit
only have one feature/fix in them

I will take a look and merge them if it makes sense.

I'm grateful to you for making this script public :heart:
I'm trying to be a good citizen and contribute in the way I would love others to contribute to my repos.
Of course, feel free to do whatever you like with these contributions.
However, I'll use the https://github.com/zzaakiirr/confluence-markdown-exporter fork then, as it is free of most of the issues I reported here.

You are free to use whatever you want as long as you keep the license. This I had to update to contain my proper name, but feel free to use the old one.
The reason I won't willy-nilly merge your code is because the quality is subpar and your branch is dangerous to use.
You have removed my sanitisation routine that turns .. in titles into harmless _! No idea why you did that, but now you are exposing every single user of your code to arbitrary file overwrites on their systems. Give a page a title of ../../../../../../../../tmp/hello and you will see what I'm talking about.


Obsolete GO:0047847 deoxyuridine phosphorylase activity

id: GO:0047847
name: deoxyuridine phosphorylase activity
namespace: molecular_function
def: "Catalysis of the reaction: deoxyuridine + phosphate = uracil + deoxy-D-ribose 1-phosphate." [EC:<IP_ADDRESS>, MetaCyc:DEOXYURIDINE-PHOSPHORYLASE-RXN]
synonym: "2'-deoxyuridine:phosphate 2-deoxy-alpha-D-ribosyltransferase activity" RELATED [EC:<IP_ADDRESS>]
synonym: "deoxyuridine:phosphate deoxy-alpha-D-ribosyltransferase activity" RELATED [EC:<IP_ADDRESS>]
synonym: "deoxyuridine:phosphate deoxy-D-ribosyltransferase activity" RELATED [EC:<IP_ADDRESS>]
xref: EC:<IP_ADDRESS>
xref: MetaCyc:DEOXYURIDINE-PHOSPHORYLASE-RXN
xref: RHEA:22824
is_a: GO:0016763 ! pentosyltransferase activity
The EC and MetaCyc xrefs on the above term are obsolete (also see #24932).
EC:<IP_ADDRESS> has been transferred to <IP_ADDRESS>, <IP_ADDRESS> and <IP_ADDRESS>, and we already have GO terms for those:
id: GO:0016154
name: pyrimidine-nucleoside phosphorylase activity
xref: EC:<IP_ADDRESS>
id: GO:0004850
name: uridine phosphorylase activity
xref: EC:<IP_ADDRESS>
id: GO:0009032
name: thymidine phosphorylase activity
xref: EC:<IP_ADDRESS>
There are only 3 EXP annotations using GO:0047847, assigned by MGI:
Human UPP2 (PMID:12849978)
Mouse Upp1 (PMID:14154460 & PMID:7744869)
So, once those are fixed, I think we can obsolete GO:0047847 deoxyuridine phosphorylase activity

Maybe should improve GO:0016154 pyrimidine-nucleoside phosphorylase activity def to reflect EC reactions/comments:
GO def: "Catalysis of the reaction: pyrimidine nucleoside + phosphate = pyrimidine + alpha-D-ribose 1-phosphate." [EC:<IP_ADDRESS>]
EC Reaction catalysed:
phosphate + uridine <=> alpha-D-ribose 1-phosphate + uracil
cytidine + phosphate <=> alpha-D-ribose 1-phosphate + cytosine
2'-deoxyuridine + phosphate <=> 2-deoxy-alpha-D-ribose 1-phosphate + uracil
phosphate + thymidine <=> 2-deoxy-alpha-D-ribose 1-phosphate + thymine
EC comment:
Unlike EC <IP_ADDRESS> and EC <IP_ADDRESS>, this enzyme can accept both the ribonucleoside uridine and the 2'-deoxyribonucleosides 2'-deoxyuridine and thymidine.

Similarly, could improve GO:0004850 uridine phosphorylase activity def to reflect EC comment:
GO def: "Catalysis of the reaction: uridine + phosphate = uracil + alpha-D-ribose 1-phosphate." [EC:<IP_ADDRESS>]
EC comment:
The mammalian enzyme also accepts 2'-deoxyuridine.

The obsolete EC:<IP_ADDRESS> xref was addressed in #27410 - that xref was removed, but the 'GO:0047847 deoxyuridine phosphorylase activity' term was retained with remaining xrefs MetaCyc:DEOXYURIDINE-PHOSPHORYLASE-RXN and RHEA:22824.
I see that RHEA:22824 is now associated with EC:<IP_ADDRESS> at RHEA, which is mapped to pyrimidine-nucleoside phosphorylase activity in GO.
So, still need to address these points:

[ ] Remove obsolete MetaCyc:DEOXYURIDINE-PHOSPHORYLASE-RXN xref
[ ] Decide whether to keep GO:0047847 deoxyuridine phosphorylase activity
[ ] Consider adding the additional reaction/comments from EC (shown above) to the corresponding GO terms.

I've removed the obsolete MetaCyc:DEOXYURIDINE-PHOSPHORYLASE-RXN xref on and replaced with URA-PHOSPH-RXN.
I added EC:<IP_ADDRESS> and EC:<IP_ADDRESS> as broadMatch xrefs to GO:0047847


Adding forgotton wait to the upload records

All Submissions:
In the upload_records function the wait argument was not passing to the client.

[x] Contributions should target the dev branch. Did you create your branch from dev?
[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

New Feature Submissions:

[ ] Does your submission pass tests?
[ ] Have you installed pre-commit with pip3 install pre-commit and set up hooks with pre-commit install?

Changes to Core Features:

[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your core changes, as applicable?
[ ] Have you successfully ran tests with your changes locally?

@alimohammad1995 thanks a lot for fixing!


Update the checkbox animated icon

Update the AnimatedAcceptVisualSource with a new drop from design.
new:

old:

Currently investigating an issue with a clip on the right edge of the new design.

/azp run

/azp run


Add registry usage telemetry to track registry usage

Add devfile registry usage telemetry to track public registry usage.
Data of interested:

number of download of each stack or sample in the registry
which tool is being used for download, e.g. odo, che, dev console, dev workspace, etc.

Some thoughts I've had on this:

Telemetry should only be reported on the production registry (https://registry.devfile.io) and not staging

To this effect, most tests that are running against the registry right now (e.g.) should use the staging registry, so as not to pollute what's reported back to us.

Getting telemetry from the OCI server will be hard

We're just using the distribution binary reference implementation, and can't  modify it
Workaround: We can measure the usage via the registry REST API on the index server, but that means the registry library must also use REST API to retrieve stacks (it's using the OCI API currently)

Not sure how we'll be able to report the tool doing the download, it'll need to be part of the registry library I suppose?

This'll require more thinking, I'm coming up blank at the moment.

Updating my thoughts from April:

We won't be able to inject telemetry into the OCI registry binary without rolling our own OCI registry


Ability to specify the d3 defined prop in Line-, LineMark- and AreaSeries.

A PR that enables the ability the specify a defined prop in the Line-, LineMark- and AreaSeries based on this issue https://github.com/uber/react-vis/issues/457.
This let's users create series containing null (or undefined) data where the null values are not automatically converted to 0's, but can be ignored in the drawing process like in the image below.
<LineSeries defined={(d) => d.y !== null} data={data} />

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for the great feedback!
I made the requested changes. If you could review those that would be great!

@tijp Thanks for making this PR. I ran into this exact use case and it saved me a ton of time.

@tijp Thank you for this. I am not able to see and implement defined={} as a prop of LineSeries. Is this functionality still in place? Thank you

the property has been renamed to getNull


bug(docker): Images are inconsistently publishing multiplatform builds

Expected Behavior
All images on docker hub are all multi-platform (amd64/arm64).

Details
It appears the -debian images are meant to build and push on release via this github action workflow it is broken.
Below is a screenshot of this showing the inconsistency.

CC: @nunoocx and @gabriel-cx - are you manually building and pushing the images?

Specifically https://github.com/Checkmarx/kics/actions/runs/6800401414/job/18500698437 and https://github.com/Checkmarx/kics/actions/runs/6800401399/job/18488854312 are failing the debian (and ubi8) build and push.  Previously it was failing the generate .md table

I can't repro locally; docker builds and runs fine for me on my mac arm64 system with both 1.20.5 (buster) and 1.21.4 (bullseye) base images.  But I can't build and --load multi-platform due to a docker limitation, so it's a partial test.

Hi @JonZeolla ,
Thank you for your inputs.
We did have an issue regarding building some images in the latest release and we build and push manually the ones that failed.
However, we already fixed the root cause of that problem, so, in the next release we will not face similar issue!
"Previously it was failing the generate .md table" regarding this, you don't need to worry. We are analyzing the root cause, but this problem alone is not affecting the build or push of any image.
Does this information clarify your doubts?


Allow manually setting id of new entity

This should not be used for normal usage of the API, but it is helpful to simplify serialization

Ezby serialize write to the internal entity storage so this is not needes


Feature request: Advanced configuration of monitoring

As of right now, we have the predefined 10/30/60 seconds frequency, 5/10/15 seconds timeout and 1-3 attempts.
It would be nice to be able to have a bit more control over these values.
So a suggestion could be:
Under each setting (frequency, timeout and attempt(s)), you could but an input field (numeric values, minimum 1) and a dropdown with the options: ms, sec, min
Under attempts, it should (of course) only be a number >= 1.
This would make it much easier to adjust the monitor to your specific URL. Maybe it's a service, and you want to have a timeout below 500ms.. or it's a service that for some reason might take more than 15 seconds to do some processing in the background, and you want it to respond in less than 30 seconds.
Just stick to the different time durations in Go (time.Millisecond, time.Second, time.Minute), so you could easily do something like:
monitor.timeoutValue (of type int32) * monitor.timeoutDuration (of type time.Duration)

Thanks for writing this up after our discussion.
I'll take a look at this 👍

I think especially since the addition of the text based config it would be good to allow custom values, I'll take a look at this 👍


 I don't find any clear plan or contributors active working on this page: https://pulsar.apache.org/release-notes/timeline
If there's any, I'm happy to close this issue and follow that initiative.

cc @Anonymitaet @dave2wave

"Coming soon" is easy to be a liar. I don't find any clear plan or contributors active working on this page: https://pulsar.apache.org/release-notes/timeline
If there's any, I'm happy to close this issue and follow that initiative.

OK


Docs for creation of Facebook, Google, Github App

Fixes https://github.com/CircuitVerse/CircuitVerse/issues/792
Describe the changes you have made in this pr -

Made a seperate wiki page for creation of Google, Facebook, Github app and linked it with readme.

Can we do this in wiki instead

Fine @tachyons

Done @tachyons
Check the page here : https://github.com/CircuitVerse/CircuitVerse/wiki/Create-Apps

Resolve conflicts please

@tachyons I will remove the commit and will request @JoshHeng to add this is in his "Restructure Readme" PR .

I will close this PR, just check the wiki page.


[WIP]sync default profile

Process: I used the manifest migrate command to generate migrated default profile as a base, and did some manual update for existing default profile( The process still cannot be fully automated based on couple reasons, still WIP for that)
Changes include:

I added some k8s settings such as podaffinity to the profile even though they are default as empty, trying to keep it more consistent with the values.yaml format we use.
Some of the settings such as env or readinessprobe are not exposed because they are just hardcoded in the charts, remove those from profile.
3.[WIP part] The values pass through part is quite out of sync, I am still working to check and sync them if needed.

@richardwxn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

mandiff_operator
034e04a28a739e9e608e201e4581945c7ebf9a38
link
/test mandiff_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@richardwxn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

mandiff_operator
034e04a28a739e9e608e201e4581945c7ebf9a38
link
/test mandiff_operator

test_operator
034e04a28a739e9e608e201e4581945c7ebf9a38
link
/test test_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@richardwxn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

mandiff_operator
034e04a28a739e9e608e201e4581945c7ebf9a38
link
/test mandiff_operator

test_operator
034e04a28a739e9e608e201e4581945c7ebf9a38
link
/test test_operator

e2e_operator
034e04a28a739e9e608e201e4581945c7ebf9a38
link
/test e2e_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@richardwxn this is quite old and I'm not sure what the status is. If the default profile in sync now and can this be closed or is there any further work to be done?

@richardwxn: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

istio/operator is out of support window. If you wish to reopen this PR, feel free.


.req files on iOS are never deleted after successful completion of request

The request files continue to persist, even after a task has been successfully completed. This causes the app's storage usage to continually grow without being able to clear it.

That is a major bug, PRs welcome!

@ened Working towards it. First I realized that we need to name the req files with the taskId to make it easier to clean up.
https://github.com/jaysignorello/flutter_uploader/commit/e0e5e9d82aae7afe5adde1a1b9f363513a61b545


octoprint: 1.5.3 -> 1.6.1

Motivation for this change
OctoPrint is a mess for distros to package. And the upstream acknowledges as much, hence the amount of pinning required.
https://github.com/OctoPrint/OctoPrint/releases/tag/1.6.1
https://github.com/OctoPrint/OctoPrint/releases/tag/1.6.0
Addded patches to hardcode pip and ffmpeg, explanation in https://github.com/NixOS/nixpkgs/pull/135302/commits/72d6031607392eefac7ad1cb87ee9fdb70153b35.
Supersedes: #132164
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Thank you very much for your work on this.

You're welcome. I haven't tested this against an actual printer, so please report any breakage you find.

@mweinelt Well, there's actually one piece of breakage that isn't fixed here, and that's the fact that you have to add the octoprint user to dialout in order to access /dev/tty*, which would be better done in the service. Maybe you know of a way to do that? As it stands right now, if someone were to do services.octoprint.enable = true; and they were to plug a device into their USB port, Octoprint would see it, but be unable to access and use it.

Should be addressed by #135492.


Support for Purr-Data

With version 0.3 released, are there any plans to fully support Purr-Data?

it's the other way around, you should ask Purr Data to support cyclone 0.3

Please realize that Purr Data doesn't actually even fully support cyclone 0.2! One way or another, there's nothing really stopping Purr Data to include cyclone 0.3 - if the full support would be the reason, then it shouldn't include 0.2 as well...
I've opened many issues and requests for Purr Data to include cyclone 0.3, and it's been about 2 years I've done that, so I'm also just waiting.
There are a few externals that needed to be ported because of the GUI transition they made (and this also affects more GUI externals from other libraries in Pd Extended). This is true for both cyclone 0.2 and 0.3 versions.
Moreover, Purr Data doesn't also support some new features from the latest Pd Vanilla versions that would be required to fully support cyclone 0.3
some issues for reference:

https://git.purrdata.net/jwilkes/purr-data/issues/355
https://git.purrdata.net/jwilkes/purr-data/issues/460
https://git.purrdata.net/jwilkes/purr-data/issues/408

cheers

Thanks for all the info, Alexandre, I opened an issue requesting this from the Purr Data side.


fix(Controller): correct haptic duration argument

The first argument for device.TriggerHapticPulse is not strength, but
the duration of the pulse in microseconds.
This has been changed so ControllerActions.TriggerHapticPulse takes one
argument, the duration of the pulse in milliseconds.
There is not actually a haptic strength value in the OpenVR API. See here.

There were the following issues with your Pull Request

Commit: e7a4d0e084db9d1db72eb71f82b85523b6c6a367
Your commit message body contains a line that is longer than 72 characters

Guidelines are available at https://github.com/thestonefox/SteamVR_Unity_Toolkit

This message was auto-generated by https://gitcop.com

Have you tried the device.TriggerHapticPulse?
If you don't include it in an Update method, then it only happens for a split second at the intensity of the parameter you provide.
(well at least it did in SteamVR 1.0.8)
The purpose of my wrapper was to allow to provide a duration and an intensity. If device.TriggerHapticPulse works that way now then that's fine.

I was just trying to be informational. :confused:

The reason it happens for a split second is because the function takes microseconds, so it would barely be noticeable. It is still called in the Update method, but there isn't actually any way to increase the intensity of the haptics, which is why I removed the strength value.

You can increase the intensity by the number you put into device.HapticFeedback.
Try using my method and put in a low number for strength and then a high number (under 4000). Leave the time the same.
You'll get a different intensity of vibration.
As I say, unless they've actually changed this that's how it was working (physically actually working, not what their documentation says)

I guess you are correct. In my testing I didn't notice any difference when I changed the strength value, but apparently the haptic motor on one of my controllers is janky. The other controller I can notice the difference on.

Crazy how the OpenVR docs say one thing and it does something else.
I pulled my hair out reading those docs :/

:confused: I created an issue on the OpenVR repository. ValveSoftware/openvr#129

I considered being really active on their repos when I first discovered the amount of bugs in their stuff...then I quickly saw they hardly accept pull requests (or fix things!)
That's why I started this repo, just to get around that :)


Support ultra cheap Logic Analyzer USB 2.0 HS with 24MHz 8 Chan based on Cypress FX2/FX2LP chips

Support ultra cheap Logic Analyzer USB 2.0 HS with 24MHz SMSP 8 Chan based on Cypress FX2 chips
More details here: https://sigrok.org/wiki/Fx2lafw
Example of what we can find on Internet for less than 10USD

https://www.banggood.com/fr/USB-Logic-Analyzer-24M-8CH-Microcontroller-ARM-FPGA-Debug-Tool-p-1177821.html
https://www.aliexpress.com/wholesale?&SearchText=USB+Logic+Analyzer+24M

The same but more expensive less than 20 USD

https://www.sparkfun.com/products/15033
https://www.amazon.com/KeeYees-Analyzer-Device-Channel-Arduino/dp/B07K6HXDH1/ref=sr_1_3?dchild=1&keywords=USB+Logic+Analyzer+FX2&qid=1602009281&refinements=p_72%3A2661618011&rnid=2661617011&sr=8-3 (to be checked)

We can probably use FX2LA firmware as is. It's GPL but that's a non-issue since none of our code is linking to it, we can just redistribute it as as a separately licensed blob.
My understanding is it just spits out a firehose of 24 Msps sample data over USB.
Proposed architecture: create a custom TCP server under scopehal-apps that listens to the USB stream, runs software based triggering, then pushes waveforms to glscopeclient. Use a 2-socket protocol with SCPI data + raw binary waveforms, maybe somewhat compatible with the proposed MAXWELL and PicoScope socket protocols.
We can add arbitrarily complex protocol triggering in the future.

I am interested in this not because I'm a big fan of the FX2LAs, but because my custom scope will have most of the control logic on PC and send low level control commands to the hardware over USB.
This also provides a nice license boundary if I want to make the control software GPL.
Any opinions on the TCP server architecture, protocols, etc? Could you link the MAXWELL socket protocol you're mentioning?

I am interested in this not because I'm a big fan of the FX2LAs, but because my custom scope will have most of the control logic on PC and send low level control commands to the hardware over USB.

Did you see https://github.com/azonenberg/scopehal-pico-bridge ?
That implements something like what you are mentioning, but for Oscilloscopes from Pico Technology instead of FX2 LAs. Maybe you can get some ideas about protocols from there.

I see. Normal SCPI over the control channel. Is the data plane format standardized?

The data plane format for the Pico bridge isn't officially documented as it's subject to change at this point.
Informally, it's
uint16 num_channels
uint64 sampleInterval //femtoseconds
for i = 1 to num_channels
    uint64 channelID
    uint64 numSamples
    if channel is analog
        float32 gain
        float32 offset
        float32 phase //offset in fs from trigger event to sample clock
        uint16 rawADCValues[numSamples]
    if channel is a digital bank
        float32 phase //offset in fs from trigger event to sample clock
        uint16 rawDigitalPortValues[numSamples] //data for up to 8 channels in LSB, MSB unused (pico native data format)

operate under the assumption that you'll be making your own driver in libscopehal

Got it. Wasn't sure if you wanted to unify this protocol. If not, perfectly fine!

The protocol will probably get unified eventually but it's too early to think about doing so now, since we haven't made enough of them to have a good idea of what the needs of different instruments will be.
But the driver side will be different per instrument at least to some extent due to all of the quirks of particular hardware. A sizeable amount of the code in the Pico driver, for example, is implementations of all the rules to see whether a given sample rate is valid for a particular set of enabled channels.


ci: set up cargo-deny checking

Description

cargo-deny allows to restrict allowed crates based on different criteria, including licenses and sources.
It also warns about security advisories and duplicated dependencies (which could reduce binary size if de-duplicated).
Issues/PRs references

Depends on #373
Open Questions

Change checklist

[x] I have cleaned up my commit history and squashed fixup commits.
[x] I have followed the Coding Conventions.
[x] I have performed a self-review of my own code.
[x] I have made corresponding changes to the documentation.

offline discussion: find way to handle cargo-deny default config and our project specific settings.

handle project specifics in a more maintainable way

I modified the deny.toml configuration to only keep the keys with non-default values as we discussed. I also kept the comments provided by cargo-deny because the effect of some keys may otherwise be quite cryptic.

Perfect!

pls squash & merge


Fixed typo in settings

Description
There's a error in Settings where on turning on Male Voice , female voice plays. So, I changed the name to Female Voice, now things make sense there.

Fixes #15

kindly merge this pr with hacktoberfest label

@balayokesh @amittri1025 Sorry for the delay, I had some problems here.
Guys, I think the voices of Web_Speech_API changes according to OS or device, I really don't know.
Here (Ubuntu 22), the production version works well, the voice is correct. But on my phone (Android), I had only female voices, even if i enable Male voice.
So, I need to study the API better and understand how to solve this, for now I will not accept the PR.


[5.10][Concurrency] Allow default arguments to require actor isolation.

Explanation: This is the experimental implementation of allowing default arguments to call global-actor-isolated functions. Actor isolation checking is deferred to the caller; it is an error to use a default argument from across isolation domains. Though this is experimental, cherry-picking is useful to make cherry-picking other actor isolation checker bug fixes easier, and it leaves open the opportunity to close the data-race hole with global-actor-isolated instance properties in 5.10 if the feature is accepted.
Scope: Limited to the actor isolation checker. The implementation is gated behind -enable-experimental-feature IsolatedDefaultValues, so it will not impact existing code.
Risk: Low; the feature is not actually enabled.
Testing: Added new tests.
Reviewer: @xedin @ktoso
Issue: rdar://101058224
Main branch PR: https://github.com/apple/swift/pull/68794, https://github.com/apple/swift/pull/69030

@swift-ci please test


Update setup packages to detect and install .NET 6

PR Details
Update the setup prereqs and main setup installer projects to detect and, if needed, install .NET 6.   At the moment Advanced Installer only knows about 6.0.2 but that will be fine.
Also update to latest Advanced Installer, version 19.4.
Related Issue
Fixes #1410

[ ] Docs change / refactoring / dependency upgrade
[x] 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
Manually tested.

[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

@Xen2 is updating the build server to use AI 19.4 as well.

Note that the Advanced Installer GUI made all these changes, and I can't claim to know the exact effect of each one.  Manual testing of a locally built setup does show it works as expected.   Once Xen2 gets the build server updated, I'll run a build and test from an official build as well.

I looked through this XML and I think it's ok (lots of installer specific stuff) - so long as we can run it in a VM from a clean Windows installation and get to run Stride that's all we need.
I remember there being another requirement for VCRedist to run GameStudio, so maybe we could add that as well?

I remember there being another requirement for VCRedist to run GameStudio, so maybe we could add that as well?

While I'm sure that's so, it's missing from the prereqs web page.
Win 10+ comes with the Universal CRT so it shouldn't be an issue, it's only for older OSes that this might come into play.
And interestingly Advanced Installer doesn't even list a CRT as an option it supports out-of-the box.
If needed, perhaps we should just add it later since it hasn't been in these projects previously?

so long as we can run it in a VM from a clean Windows installation and get to run Stride that's all we need.

I've verified a locally built installer EXE on a clean VM (win10-21h2-pro) and it works as expected:  it detects that .NET FW 4.x is installed and that .NET6  is not.  It properly downloads and installs the missing prereq and then properly installs the Launcher.    The Launcher is started at the completion of setup and works as expected.
I'll re-run on a new clean VM with a StrideSetup.exe from the build server once this PR is accepted and a build run.

@manio143 - any objection with moving forward with the PR as-is since it seems to behave as expected?

None from me. Not sure if we're waiting on @xen2 to comment before merging?

OK, thanks.
Xen2 has the link to this PR so not sure.

There's also sources/prerequisites/prerequisites.aip for Stride itself
Supposedly launcher should have installed it, but I think it's still good to add it there in case the launcher has different requirements in the future (and it doesn't install .NET 6, for example if we pack launcher with .NET 6 runtime).

Thinking about it, we need to check if launcher already published with self-contained .NET 5/6 runtime.
If yes, both launcher setup don't need .NET 6.0, only Stride itself.

Pushed the latest based on clean VM testing.  In the end:

As @xen2 was thinking, since the launcher is a self-contained EXE, there is no need for a .NET 6 prereq.  I've removed that.
I've updated the Stride prereq installer and on a Win10 21h2 machine it successfully evaluates everything and discovers that everything is good-to-go (CRT, DirectX and .NET dependencies) and exits without doing anything else.   This was verified with event log entries.

In the end, the update is really to use 19.4 version of Advanced installer and to change the Stride prereq installer to look for .NET 6 instead of 5.

As an interesting aside, the Azure OS image comes with .NET 6 MSI in the OSes package cache, so it didn't wind up downloading it, just installing.

Ah, and same with the CRT, it turns out.   Since it was a very fast silent install it tricked me into thinking it didn't do anything.
It might be worth considering moving to the Universal CRT but we can save that for another day since this works.

Sounds like this PR is good to go, ready to merge ?


pandas read_excel reads slice_action, antigen row/column ints as floats

Describe the bug
Pandas read_excel reads slice_action and antigen row/columns ints as floats on the Jetson Nano. When df are merged, this leads to all non-float data becoming nan. This eventually leads Assertion Error: Plotting dataframe is empty. Please check the plotting keys error.
This is a non-issue on MacOS.
To Reproduce
Steps to reproduce the behavior:
On labelfree@labelfree-desktop, if one uses the pysero config in Pycharm to run -a -i "home/labelfree/googledrive/ELISAArrayReader/images_scienion/2021-06-23-11-54-27-CambodiaGCE_VLPconctest_plate1/pysero_2021-06-23-11-54-27-CambodiaGCE_VLPconctest_plate1_20210915_1542" -o "home/labelfree/googledrive/ELISAArrayReader/images_scienion/2021-06-23-11-54-27-CambodiaGCE_VLPconctest_plate1/pysero_2021-06-23-11-54-27-CambodiaGCE_VLPconctest_plate1_20210915_1542" -m "pysero_analysis_config.xlsx" in debug mode, this will reproduce the error.
Expected behavior
Since the metadata and config is accurately populated in this file, this should lead an output of standard curves with hue = antigen and subplots separated by serum ID. This png file should be located in the output folder and titled fig__mean.png
Operating environment (please complete the following information):

OS:  Jetson Nano
Versions of python packages: pandas 1.3.2 and openpyxl 3.0.7

I addressed this issue in the latest commit in the jn-edits branch. Necessary steps included changing dtype to object when pandas.read_excel is executing in read_plate_info and well2D_to_df1D and also removing (commented out for now) the #general slicing section of analyze_od.


django-summernote breaks after upgrading to Django 3

After upgrading my app to Django 3 I get the following error:
Traceback (most recent call last):
  File "manage.py", line 23, in <module>
    execute_from_command_line(sys.argv)
  File "/usr/local/lib/python3.7/site-packages/django/core/management/__init__.py", line 401, in execute_from_command_line
    utility.execute()
  File "/usr/local/lib/python3.7/site-packages/django/core/management/__init__.py", line 377, in execute
    django.setup()
  File "/usr/local/lib/python3.7/site-packages/django/__init__.py", line 24, in setup
    apps.populate(settings.INSTALLED_APPS)
  File "/usr/local/lib/python3.7/site-packages/django/apps/registry.py", line 114, in populate
    app_config.import_models()
  File "/usr/local/lib/python3.7/site-packages/django/apps/config.py", line 211, in import_models
    self.models_module = import_module(models_module_name)
  File "/usr/local/lib/python3.7/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1006, in _gcd_import
  File "<frozen importlib._bootstrap>", line 983, in _find_and_load
  File "<frozen importlib._bootstrap>", line 967, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 677, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 728, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/source/lib/models.py", line 8, in <module>
    from django_summernote.widgets import SummernoteInplaceWidget
  File "/usr/local/lib/python3.7/site-packages/django_summernote/widgets.py", line 4, in <module>
    from django.contrib.staticfiles.templatetags.staticfiles import static
ModuleNotFoundError: No module named 'django.contrib.staticfiles.templatetags'

regarding to https://docs.djangoproject.com/en/dev/internals/deprecation/ that module was removed.
After investigation of the Django sources replacing should be easy:
# django 2.x django\contrib\staticfiles\templatetags\staticfiles.py
def static(path):
    warnings.warn(
        'django.contrib.staticfiles.templatetags.static() is deprecated in '
        'favor of django.templatetags.static.static().',
        RemovedInDjango30Warning,
        stacklevel=2,
    )
    return _static(path)

I think the problem may be that the package at https://pypi.org/project/django-summernote/ is outdated. It dates from January 2019. As the problem ist already fixed in the summernote sources only a new package needs to get published.

try using the latest version
https://github.com/summernote/django-summernote.git@702b574920342766f56262725d87125719156878#egg=django_summernote

I understand that this is a possible solution. But I do not understand, why the package maintainer stopped to publish updates via pypi?

life happens I guess. I'm just happy it works at all and I didn't have to patch it myself :)

try using the latest version
pipenv install git+https://github.com/summernote/django-summernote.git@702b574920342766f56262725d87125719156878#egg=django_summernote

I tried your solution and python manage.py runserver worked, but in the form, a field that I use summernote shows a  messsage "<IP_ADDRESS> access is refused". Do you know how to fix this? thanks

I'm having the exact same problem as ismaemahh...

@ismaemahh https://docs.djangoproject.com/en/3.0/ref/settings/#std:setting-X_FRAME_OPTIONS https://docs.djangoproject.com/en/3.0/ref/clickjacking/

@lqez Is it a good idea to set X-Frame-Options: sameorigin in summernote?

He guys,
while installing summernote with pip i've got the error, that staticfields.templatetags are missing. These packages were removed in django 3.0, so i'am not able to install the editor. No change with the latest version. Help would be grate, thx!

I just published <IP_ADDRESS> with Django 3.0 support.
https://github.com/summernote/django-summernote/releases/tag/<IP_ADDRESS>

Who knows alternative module of django-summernote? I want use it for Django 3.1

Add this X_FRAME_OPTIONS = 'SAMEORIGIN' to your settings.py
See

Add this X_FRAME_OPTIONS = 'SAMEORIGIN' to your settings.py
See

It's solve the problem, thanks

I think we should write this on README.
Thanks to all!

Add this X_FRAME_OPTIONS = 'SAMEORIGIN' to your settings.py
See


Is there any swift equivalent?

Could you make this library for swift

@satyamub I'm sorry, no.
I have yet to learn swift.
Maybe someone can implement a copy with swift.
I authorize to everyone.


[PY3] TestDumpMaster.test_basic: Un-acked message

Steps to reproduce the problem:

(python3) py.test -k TestDumpMaster -v

What is the expected behavior?
No console output.
What went wrong?
test/mitmproxy/test_dump.py::TestDumpMaster::test_basic Exception ignored in: <bound method Reply.__del__ of <mitmproxy.controller.Reply object at 0x04E10570>>
Traceback (most recent call last):
  File "C:\Users\user\git\mitmproxy\mitmproxy\controller.py", line 267, in __del__
    raise exceptions.ControlException("Un-acked message: %s" % self.obj)
mitmproxy.exceptions.ControlException: Un-acked message: <Mock id='81855280'>
PASSED

Mitmproxy Version: master
Operating System: Windows x64

I see these on OSX and even on Travis from time to time.

fixed by #1474


ushort types being handled as System.UInt16

Properties that use the ushort data type are getting translated to System.UInt16, I would assume this would be handled as a primitive numerical data type.

Issue with fixed.


Milestone 2: Process data in models

Milestone 2: Process data in models
In this milestone I :

Created User model
Created Post model
Created Comment model
Created Like model
Set associations between the models
Created Custom methods for the above models

Users

A method that returns the 3 most recent posts for a given user.

Posts

A method that updates the posts counter for a user.
A method that returns the 5 most recent comments for a given post.

Comments

A method that updates the comments counter for a post.

Likes

A method that updates the likes counter for a post.

Used rails c command to open IRB and check the method implementations

Thanks for the review @descholar-ceo 🙌


Cleanup origin startup stdout

When watching folks during the PEARC tutorial, it was quite apparent how confusing our startup has become -- lots of "stuff" gets printed to stdout not from Pelican.
Here's what we have currently:
Initializing run environment for Pelican...
Jul 22, 2024 6:45:54 PM edu.uiuc.ncsa.security.core.util.MyLoggingFacade info
INFO: scitokens-server:Logging to file /dev/pts/0
Jul 22, 2024 6:45:54 PM edu.uiuc.ncsa.security.core.util.MyLoggingFacade info
INFO: scitokens-server:OAuth 2 for MyProxy, version 5.4.1 startup on Mon Jul 22 18:45:54 UTC 2024
Jul 22, 2024 6:45:54 PM edu.uiuc.ncsa.security.core.util.MyLoggingFacade info
INFO: scitokens-server:loading configuration.
Jul 22, 2024 6:45:54 PM edu.uiuc.ncsa.security.core.util.MyLoggingFacade info
INFO: oa4mp:Error loading authorization configuration. Disabling use of headers
Jul 22, 2024 6:45:54 PM edu.uiuc.ncsa.security.core.util.MyLoggingFacade info
INFO: oa4mp:ping enabled
Jul 22, 2024 6:45:54 PM edu.uiuc.ncsa.security.core.util.MyLoggingFacade info
INFO: oa4mp:No claim source configuration found.
Jul 22, 2024 6:45:54 PM edu.uiuc.ncsa.security.core.util.MyLoggingFacade info
INFO: oa4mp:Starting to load LDAP configuration.
No chain present for host cert; trying to derive one
No chain present; will use empty file
Starting Pelican...
Running pelican with arguments: origin serve -p 8444
Version: 7.9.3-next
Build Date: 2024-06-25T16:40:15Z
Build Commit: a024f4636b25ecccb26308769c0256d488d62392
Built By: goreleaser
2024/07/22 18:45:54 OK<PHONE_NUMBER>4529_create_db_tables.sql (2.76ms)
2024/07/22 18:45:54 goose: successfully migrated database to version:<PHONE_NUMBER>4529

Pelican admin interface is not initialized

I think we should dump:

All the oa4mp/Java stuff
The "No chain present" lines (may be related to the above?)
The Build date / commit /built by
Output from goose / ORM.

Marking as critical as I think this might be a simple fix but bigger payoff for new people.

Some additional details:

This occurs when you run the origin as a standalone container (there, the log files are redirected for logging -- the above is the stdout not using the logging system).  The tutorial where we noticed the errant printout is here if you'd like to reproduce.
The entry point for the container image is here.
I expect the OA4MP stdout is coming from here: https://github.com/PelicanPlatform/pelican/blob/main/images/entrypoint.sh#L38 .  We could redirect the output to a buffer and only print out the information if the command fails.
Similarly, the chain manipulation here could be hidden if there's no error.
For these two:

The Build date / commit /built by
Output from goose / ORM.
I don't know where they come from -- goose, at least, may need to be investigated to see if it can hook into a logging system.


Update all resolvers for Return ID

Update all the resolvers to call the new methods, remove all old methods from backend interface.
This will still compile, but not run right until I update keyvalue in a followup.
PR Checklist

[ ] All commits have a Developer Certificate of Origin (DCO) -- they are generated using -s flag to git commit.
[ ] All new changes are covered by tests
[ ] If GraphQL schema is changed, make generate has been run
[ ] If collectsub protobuf has been changed, make proto has been run
[ ] All CI checks are passing (tests and formatting)
[ ] All dependent PRs have already been merged

I'll just do it all together with the next one I guess


hack: Update preflight with ergonomics changes

@xugy99, @Anhui-tqhuang: I spent some time working with the Cloud Architects today on the onboarding process and I'd like to propose some changes to how the preflight script fits together. This PR sketches out the beginning of it by rewriting the --help; essentially I think we should split the script between two core use-cases: onboarding and productive-use.
Rather than using the --with-infra flag to add the DP overhead, the --onboard flag should check that the subscription is configured correctly, and that there is enough quota for a DP and a single non-HA E2s_v3 cluster. The customer should be able to do something like curl raw.githubusercontent.com/azure-preflight | bash -s -- $SUBSCRIPTION $REGION in their Azure web CLI and be off and running.
The other options remain for customers that are up and running but want to make sure that they have enough capacity to provision a new cluster. I don't think this second use case is ideal long-term - we want the portal/API to provide all the feedback the customer needs here.
I have a few other proposals to codify as well - e.g., you'll note here that I'm requiring <target-subscription> rather than assuming the default subscription in az is the target - but I'd like to get this first, largest change out there first to see what you guys think.

onboarding seems like a init step when customer first uses our services, in your case, onboarding is related to a specific region which makes me a bit confused 🤔

this looks good to me as a pre-first-flight check:

user has to know which subscription is going to be used
user has to know the first region it prefers for flight

@ba-edb do you want us to continue on patching to your branch?

onboarding seems like a init step when customer first uses our services, in your case, onboarding is related to a specific region which makes me a bit confused

The onboarding follows a specific script; I'm suggesting we try to make that as simple as possible with this script. But you're right about needing to preflight a second region - we still need the --infra option. Will amend.

@ba-edb do you want us to continue on patching to your branch?

Sure! I'm going to poke at this a bit more over the next couple of days as well. Thanks Guang.

@xugy99 I rebased this PR with main branch and made some tiny fix for the conflicts with @Simonzhaohui's comments. I also add an option --onboard based on the initial @ba-edb's comment.
@ba-edb @xugy99 I set this PR to read for review. Could you provide some comments for it?


ZEL 13: Code maturity

Code maturity wilk be the general focus over the next few months, we no longer need an issue for it


Create a 'reference status' report

Add a report for reviewing reference status of references that have been sent out/received where a candidate is in a post acceptance state.

Link to Trello card
https://trello.com/c/Idcf87an/301-create-a-reference-status-report-for-candidates-in-post-acceptance-states

Hi @David-Feetenby, thanks for putting this up for review again. I've got a few comments that I'd like addressing before we move this any further. I'll go through them in turn:

Ensuring we hide applications that are in no-longer-relevant statuses:

To replicate: I impersonate Cameron Carrot and view the report. I click on the link for Keele and North Staffordshire Teacher Education and click Winfred Hane's Application link. As they're "conditions not met", we probably no longer need to surface them on this report as they're not an active recruit for this provider.

We should scope the report to people in 'Conditions_Pending' and 'Recruited' at the very least, but we need to exclude candidates in unsuccessful 'post offer acceptance' statuses such as 'conditions_not_met'.

I can see the status of 'other' in the report, what do these represent? We shouldn't have this status as it doesn't appear to be useful, clicking into this users references, they only appear to have the 3 received. Looking at the user's application, they had one reference declined (we need to hide this) and one reference cancelled (we need to hide this).

I believe the action here is to hide referees from this report in the same way that we do on the provider tab.

Design comments (ensure we collaborate with Design colleagues)

I'll defer to @MaeveRoseveare and @Kateharries on this, but having seen this in a review app, I think we can amend a few things to improve it (in line with some of @JR-G's comments earlier. I think this will improve it and make it more digestible:

for each candidate, can we make it such that the name appears once for all references, and that links to the references page for that candidate
let's remove Application Choice ID, I'm not sure its utility will outweigh keeping this table simple
as we will use the application link in the Name section, we can remove that column too.

Hi @David-Feetenby, following up on your questions to me via Slack, and posting here for team visibility.

What candidate statuses count as no longer relevant?

There will be statuses that are not applicable in this report as they're prior to submission of an application, or before an offer is accepted. I believe these are: unsubmitted, application_not_sent, awaiting_provider_decision, interviewing, cancelled, declined, offer, offer_withdrawn
There are also statuses where we won't need to render the candidate because they're no longer progressing with an application at this specific provider: rejected, withdrawn, conditions_not_met
I believe this leaves the following statuses that are worth having in the report:
offer_deferred, recruited, conditions_pending

What to do with any references that are currently classified as 'other'

We only wish to show the references that have feedback_requested or 'feedback_provided'. Every other record should be filtered out in the other statuses
not_requested_yet
feedback_refused
email_bounced
cancelled
cancelled_at_end_of_cycle
This should mirror the behaviour we use in the following page: https://www.apply-for-teacher-training.service.gov.uk/provider/applications/XXXXXX/references

It's looking really good – great work! Left a comment above on the ActiveRecord query. I also have some 🤏  formatting questions:
When there are no applicable applications the placeholder text appears not to be rendering in our gov format:

And just to double check – should 'Not received' be in a format with a colour background, similar to received?


using rai to build raiproject docker images

I submitted a rai job with the following build file
rai:
  version: 0.2
resources:
  cpu:
    architecture: ppc64le
  network: true
commands:
  build_image:
    image_name: raiproject/scope:ppc64le-cuda80-latest
    dockerfile: "./ppc64le.cuda80.Dockerfile"
    no_cache: true

The server provides the following output
[ 2018-08-14 14:53:27.533701863 +0000 UTC] ✱ Downloading your code.
[ 2018-08-14 14:53:47.104687599 +0000 UTC] ✱ Docker image name cannot start with rai/ . Choose a different prefix.
[ 2018-08-14 14:53:47.17477052 +0000 UTC] ✱ Unable to create image raiproject/scope:ppc64le-cuda92-latest.

Is it possible to remove the restriction on using rai to build raiproject images?

https://github.com/rai-project/server/blob/master/work.go#L167


protobuf-c code generation using protoc-gen-c is not working

I installed protobuf-c[tools] package in x64-linux env and build protoc-gen-c successfully.
But, when I try generate c code using protoc and plugin built failed.
FYI) If I built protobuf-c not using vcpkg, code generation works well.
Environment

OS: ubuntu 18.04
Compiler: gcc (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0

To Reproduce
I made test.proto file like below
syntax = "proto3";

message AfTestSmall
{
    int32 v0 = 1;
    int32 v1 = 2;
    int32 v2 = 3;
    int32 v3 = 4;
}

and install package and run protoc
./vcpkg install protobuf-c[tools]
./installed/x64-linux/tools/protobuf/protoc --c_out=. --plugin=/home/dev/vcpkg/installed/x64-linux/tools/protoc-gen-c test.proto

then fail with message below
terminate called after throwing an instance of 'std::system_error'
  what():  Unknown error -1
--c_out: protoc-gen-c: Plugin killed by signal 6.

Confirmed.

It looks like crashed at protobuf/compiler/command_line_interface.cc line 2078:
  if (!subprocess.Communicate(request, &response, &communicate_error)) {
    *error = strings::Substitute("$0: $1", plugin_name, communicate_error);
    return false;
  }

@ras0219-msft Could you please take a look?
Thanks.

https://github.com/protobuf-c/protobuf-c/pull/425/commits/ce75229b6a2af6dcf701f94d981ad95a1952d8b6 should fix this. See https://github.com/protocolbuffers/protobuf/issues/5107, https://github.com/protocolbuffers/protobuf/issues/5140 for more info.


bugfix: fixed extravolumes mount in cilium-preflight

Please ensure your pull request adheres to the following guidelines:

[X] For first time contributors, read Submitting a pull request
[X] All code is covered by unit and/or runtime tests where feasible.
[X] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[ ] If your commit description contains a Fixes: <commit-id> tag, then
please add the commit author[s] as reviewer[s] to this issue.
[X] All commits are signed off. See the section Developer’s Certificate of Origin
[ ] Provide a title or release-note blurb suitable for the release notes.
[ ] Are you a user of Cilium? Please add yourself to the Users doc
[x] Thanks for contributing!

Fixes: #34961
Fixed condition to fix extraVolume
<!-- Enter the release note text here if needed or remove this section! -->

/test


Can't apply fontSize to headings

Hello!
I have simple React element with the following render method:
  render () {
    const bodyStyles = { fontSize: 12 }
    const titleStyles = { fontSize: 18 }

    return (
      <ScrollView style={styles.container}>
        <KeyboardAvoidingView behavior='position'>
          {
            this.props.items.map((item, index) => {
              return <View key={index} style={styles.item}>
                <Text style={styles.itemIndex}>{index + 1}</Text>
                <Markdown styles={{text: bodyStyles, heading: titleStyles, heading1: titleStyles}}>
                  # Heading
                  Ordinary text
                </Markdown>
              </View>
            })
          }
        </KeyboardAvoidingView>
      </ScrollView>
    )
  }

In this case, Heading has incorrect fontSize (12 instead of 18). I've tried without text style and with heading only, but no luck.
Used packages:
"react": "16.0.0-alpha.12",
"react-native": "0.47.1",
"react-native-simple-markdown": "^1.0.60-rc.3"

@CharlesMangwa , is it related to #37 ?

@CharlesMangwa, are there any news about this issue?

Hey @xvonabur! Sorry I didn't have too much time to work on this. Did you fix it? If so, could you send a PR please? Otherwise, I'll try to fix it for v1.2.

@CharlesMangwa , no. Unfortunately, I don't have enough time for this.

Might be related: I've noticed text style seem to override other styles, like those set for link. In my case I can't set a custom color to link if text also has one. Hope thiss helps to narrow it down, I probably don't have the time and energy to look into it myself.


Build Failure: Undefined Symbol undefined symbol: _ZN3Aws6Client9AWSClient20SetServiceClientNameERKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEE

Confirm by changing [ ] to [x] below to ensure that it's a bug:

[x] I've gone though Developer Guide and API reference
[x] I've searched for previous similar issues and didn't find any solution

Describe the bug
A build failure. Fuller notes below.
[ 92%] Linking CXX executable aws-cpp-sdk-core-tests
./aws-cpp-sdk-core-tests: symbol lookup error: ./aws-cpp-sdk-core-tests: undefined symbol: _ZN3Aws6Client9AWSClient20SetServiceClientNameERKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEE
aws-cpp-sdk-core-tests/CMakeFiles/aws-cpp-sdk-core-tests.dir/build.make:1300: recipe for target 'aws-cpp-sdk-core-tests/aws-cpp-sdk-core-tests' failed
make[2]: *** [aws-cpp-sdk-core-tests/aws-cpp-sdk-core-tests] Error 127
make[2]: *** Deleting file 'aws-cpp-sdk-core-tests/aws-cpp-sdk-core-tests'
CMakeFiles/Makefile2:1252: recipe for target 'aws-cpp-sdk-core-tests/CMakeFiles/aws-cpp-sdk-core-tests.dir/all' failed
make[1]: *** [aws-cpp-sdk-core-tests/CMakeFiles/aws-cpp-sdk-core-tests.dir/all] Error 2
Makefile:129: recipe for target 'all' failed
make: *** [all] Error 2
SDK version number
Master branch as of July 7, 2021
Platform/OS/Hardware/Device
Inside a docker container with Ubuntu 18.04 LTS.
To Reproduce (observed behavior)
git clone --recurse-submodules https://github.com/aws/aws-sdk-cpp.git
cd aws-sdk-cpp/
mkdir build
cd build
cmake .. -DCMAKE_BUILD_TYPE=Release -DBUILD_ONLY="ecr;s3"
make
This gets 92% done and then produces the dump listed earlier.
Expected behavior
A clean build.
Let me know if you need a docker container to work from.

@jplflyer ,
First and just in case this is blocking you, add -DENABLE_TESTING=OFF to your cmake command so it doesn't run the tests.
Do you know which test specifically is failing? And just to verify this is what I think it is, does it succeed after you run the make command a couple times?

@KaibaLopez Thanks for the reply.
Repeated makes produce the same error. I've tried going back to previous versions with something like:
 git checkout 1.9.40

I tried several parts and got the same error. If I go back to 1.9.1, I got a different build error. When I went to 1.8.186, I got yet another build error.
Linking CXX executable aws-cpp-sdk-core-tests
./aws-cpp-sdk-core-tests: symbol lookup error: ./aws-cpp-sdk-core-tests: undefined symbol:

However, the -DENABLE_TESTING=OFF you recommended got me a clean build.
I'm not sure which test is causing it -- I've included the build failure way at the top. I'm happy to use that as a workaround.

Ok, so if you are happy with the workaround feel free to close the issue, since it's going to be hard for me to pinpoint what is going on without knowing which test case is failing it would be a bunch of shooting in the dark for me so yea...
If you want to give it a few more tries to find the problem, can I ask which version of gcc/g++ and cmake are you using?
Also, try using 'sudo make', the tests do try to create files and read folders that might need admin permissions.

It's a build problem, not a run problem.
I'll make a Dockerfile that hits the error. That's what I did last time I had build problems with the SDK.

yea that works

Okay, I managed to dig further and make a docker container that highlights the problem. See this:
https://github.com/jplflyer/FailingAWS.git
The reason it took time was that the build succeeds under Ubuntu 18.04 and fails under 20.04 (which uses a newer version of GCC.
There's a README.md with directions on how to make the build fail.


bridge: Allow opendal be used as object_store::ObjectStore

It should be interesting to implement object_store::ObjectStore for the opendal's Operator.
After this work, object_store can use opendal's service too.

We can provide an object_store binding like what we do for python/nodejs

Although opendal doesn't support all operations provided by object_store, we can start from the basic (read/write/...)

I have implemented the basic support. Let's improve it later.


Change Labels of Zappi Energy and add Sum

based on discussion in https://github.com/CJNE/ha-myenergi/issues/217 I relabeled h[1-3]b to device_boosted and changed device_total to be sum of boosted + green energy.

Looks great, thanks! Merging.


BUG: scipy.optimize.milp sometimes crashes with HiGHS assertion failure in 1.9.2

Describe your issue.
Background: just upgraded from 1.9.1 to 1.9.2 to test out the fix for #16545.
Some MILP models that solved perfectly fine with scipy.optimize.milp in 1.9.1 crash hard in 1.9.2 from deep within the HiGHS code:
Running HiGHS 1.2.0 [date: 2021-07-09, git hash: n/a]
Copyright (c) 2022 ERGO-Code under MIT licence terms

Presolve is switched off

Solving MIP model with:
   358 rows
   332 cols (101 binary, 0 integer, 0 implied int., 231 continuous)
   961 nonzeros

        Nodes      |    B&B Tree     |            Objective Bounds              |  Dynamic Constraints |       Work
     Proc. InQueue |  Leaves   Expl. | BestBound       BestSol              Gap |   Cuts   InLp Confl. | LpIters     Time

         0       0         0   0.00%   0               inf                  inf        0      0      0         0     0.0s
         0       0         0   0.00%<PHONE_NUMBER>     inf                  inf        0      0      7        46     0.0s
Assertion failed: (fracval < localdom.col_upper_[col] - mipsolver.mipdata_->feastol), function selectBranchingCandidate, file HighsSearch.cpp, line 279

I tried to investigate this myself by looking into the HiGHS logs and the funny thing is that this assertion does not fail when I pass highs_debug_level with a value of 1, 2 or 3 (default=0) as a verbatim option to scipy.optimize.milp().
Also, I noticed that somehow the HiGHS version was downgraded from 1.2.2 to 1.2.0 between 1.9.1 and 1.9.2. Is this intentional? If not, I guess that might be the underlying culprit?
Reproducing Code Example
N/A

Error message
Assertion failed: (fracval < localdom.col_upper_[col] - mipsolver.mipdata_->feastol), function selectBranchingCandidate, file HighsSearch.cpp, line 279

SciPy/NumPy/Python version information
1.9.2 1.23.4 sys.version_info(major=3, minor=9, micro=13, releaselevel='final', serial=0)

This ring a bell to you guys @mckib2 @mdhaber ?

I don't think we intentionally downgraded HiGHS, and @mckib2 confirmed. How are you checking the HIGHS version?

Yeah that's also how I found out. What would be the course of action now? Bump HiGHS back to 1.2.2 in SciPy 1.9.3?

The version numbers are inconsequential, the reason they show up differently is based on the way you build them in scipy:

distutils builds will report the version scraped from the CMakeLists.txt (which doesn't always get incremented regularly by upstream maintainers)
meson builds will currently always report 1.2.0 (because we don't want to do dark magic to have meson be able to scrape the CMakeLists.txt)

The HiGHS submodule hash itself hasn't changed in the past 3 months, so the variability may be in whether you are using a distutils build or a meson build (which would be insidious!).
Can you provide a reproducer so I can try to reproduce the error?  (it also occurs to me, we probably want to be building in release which would remove assert statements, but it's also nice to catch these errors...)

The underlying optimisation model is quasi-infeasible due to its inherent poor numerical quality, and HiGHS trips on a feasibility tolerance check in the branch and bound phase.
In 1.9.1, HiGHS reported this particular model to be infeasible, which the Python code then would be able to handle properly. In 1.9.2, the assertion in HiGHS just nukes the whole Python process without any possibility errors...) of recovery. Needless to say, I prefer the 1.9.1 way of working.
The milp call is done quite deep in a program, and it would not be trivial to extract its exact input data.

@woutervbk gh-17430 I believe should resolve this issue.  Are you able to pull a nightly wheel or build from main to confirm if the issue is resolved or not?

@mckib2 I can confirm that I was able to successfully run all my unit tests with https://anaconda.org/scipy-wheels-nightly/scipy/1.10.0.dev0/download/scipy-1.10.0.dev0-cp39-cp39-macosx_12_0_arm64.whl, including the one for which I opened this issue.

Great! I'll close this and it should be available in the next upcoming release


How to convert pressure into a Air Speed in km/h

Hello,
I work on a telemetry for my Calmato RC plane.
Can you explain me how to convert your result to a km/h speed please ?
Thanks,
Pierre

The MS4525DO library is built to return the pressure from the sensor in units of Pa. We have an open-source air data library which can use the measured pressure to compute indicated airspeed in m/s using the Ias_mps function. Finally, you would need to convert m/s to km/h by multiplying the airspeed by 3.6.


Blog App: Feature/processing data in models

On this branch:

[ ]  I set up associations between models.

[ ]  I created the following methods:

A method that returns the 3 most recent posts for a given user.

A method that updates the posts counter for a user.

A method that returns the 5 most recent comments for a given post.

A method that updates the comments counter for a post.

A method that updates the likes counter for a post.

Hi @Megagig, it would be nice to remove the live demo section from your readme for now till you have a deployed link, please note this is a comment not a review 👍🏿

Hi @Megagig, it would be nice to remove the live demo section from your readme for now till you have a deployed link, please note this is a comment not a review 👍🏿

@prolajumokeoni Thank you so much for your comment. I will do so immediately.


Create ethernetnetworks with custom bandwidth

Scenario/Intent
Create network with bandwidth
Environment Details

Module Version: 5.3
Ansible Version: 2.8.4
OneView Appliance Version: 4.20.01
OneView Client API Version: 800
Python Version: 2.7.15+
python-hpOneView SDK Version: 5.0.0
Platform: Ubuntu

Steps to Reproduce

Install OneView python
git clone https://github.com/HewlettPackard/python-hpOneView.git --single-branch -b release/5.0.0-beta
cd python-hpOneView
sudo python setup.py install
pip install -r requirements.txt

Install Oneview-ansible
git clone  https://github.com/HewlettPackard/oneview-ansible.git --single-branch  -b enhancement/pass_by_name/network_set
cd oneview-ansible
sudo pip install -r requirements.txt

configure environment variables
export ANSIBLE_LIBRARY=/home/dung/oneview-ansible/library
export ANSIBLE_MODULE_UTILS=/home/dung/oneview-ansible/library/module_utils/

Run the example to create network

in oneview-ansible/examples
Expected Result
network created with bandwdith
Actual Result
An exception occurred during task execution. To see the full traceback, use -vvv. The error was: AttributeError: 'ConnectionTemplates' object has no attribute 'get'
fatal: [localhost]: FAILED! => {"changed": false, "module_stderr": "Traceback (most recent call last):\n  File "/home/dung/.ansible/tmp/ansible-tmp-1567518721.31-139722959710603/AnsiballZ_oneview_ethernet_network.py", line 114, in \n    _ansiballz_main()\n  File "/home/dung/.ansible/tmp/ansible-tmp-1567518721.31-139722959710603/AnsiballZ_oneview_ethernet_network.py", line 106, in _ansiballz_main\n    invoke_module(zipped_mod, temp_path, ANSIBALLZ_PARAMS)\n  File "/home/dung/.ansible/tmp/ansible-tmp-1567518721.31-139722959710603/AnsiballZ_oneview_ethernet_network.py", line 49, in invoke_module\n    imp.load_module('main', mod, module, MOD_DESC)\n  File "/tmp/ansible_oneview_ethernet_network_payload_Work2w/main.py", line 251, in \n  File "/tmp/ansible_oneview_ethernet_network_payload_Work2w/main.py", line 247, in main\n  File "/tmp/ansible_oneview_ethernet_network_payload_Work2w/ansible_oneview_ethernet_network_payload.zip/ansible/module_utils/oneview.py", line 301, in run\n  File "/tmp/ansible_oneview_ethernet_network_payload_Work2w/main.py", line 163, in execute_module\n  File "/tmp/ansible_oneview_ethernet_network_payload_Work2w/main.py", line 177, in _present\n  File "/tmp/ansible_oneview_ethernet_network_payload_Work2w/main.py", line 222, in _update_connection_template\nAttributeError: 'ConnectionTemplates' object has no attribute 'get'\n", "module_stdout": "", "msg": "MODULE FAILURE\nSee stdout/stderr for the exact error", "rc": 1}

@DungKHoang, Thanks for reporting this.
Looks like a bug in the module, can you please share the playbook you have used?

HEre is an example:
- name: Create ethernet network ch_oam_3
oneview_ethernet_network:
config: '{{config}}'
state: present
data:
type:                       'ethernet-networkV4'
name:                       'ch_oam_3'
ethernetNetworkType:        Tagged
purpose:                    General
smartLink:                  False
privateNetwork:             False
vlanId:                     2000
bandwidth:
typicalBandwidth:       1000
maximumBandwidth:       1000

@DungKHoang, Same playbook is working for me under the same environment settings, please make sure you are using the latest version of Ansible-OneView library and install dependency packages  by running the"pip install requirements.txt " command from the root directory.

As Sijeesh mentioned above, the playbook worked with the given environment details and it created a network from my end as well. Hence, closing it.

Closing as per Yuvarani's comment


Add Github Action to create+deploy a testnet

Following #17, it'd be great to be able to create and deploy a testnet using Github Actions.

(this isn't important to do this instant, but would be useful eventually, and hopefully soon)


Use hashFiles from GH actions toolkit

See https://github.com/actions/toolkit/issues/472

Closing, this functionality is now part of actions/setup-node.


Check if energy and info file are empty

EnergyFileReader: Check if input is an empty file

issue #46 includes the requested testing


[68] added utils.check_roles_enabled helper

This helper check are provided roles enabled on instance.
It might be very usefull if you want to run migration code only on specific roles.
Closes #68

It also look like you tests had failed for some reason. We'll need to fix it before merging.


add new button size and fix size typography

Add new button sizes
could you take a look @xavijam ?

:+1:


add support for riscv64

Bug/issue #, if applicable:
One of a series of updates required to build the swift toolchain on riscv64 host systems.
More info here - Swift on RISCV
Summary
Add support for riscv64 as host architecture.
Dependencies
none
Testing
build the riscv64 swift toolchain on a riscv64 host system
Steps:

Provide setup instructions.
Explain in detail how the functionality can be tested.

Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[n] Added tests
[n] Ran the ./bin/test script and it succeeded
[n] Updated documentation if necessary

@swift-ci Please test


automate hash calculation and email to user/coop upon completion of membership application form.

The member application form needs to send an email containing the membership hash used for payment and proving an ethereum  account upon payment of dues in eth.
The desire is to move to a private database ultimately but in the mean time googlescript to send the email would be expedient for a functional POC based on HJ's google  form.

I added a getBase64EncodedMD5(text) script and an "id hash" sheet to create the id string and hash for a user to be sent in the email to the applicant upon completion of the form.
I do not know what hash we should use. Greg suggested SHA-1 is not the best. I chose MD5 randomly.

SHA-512 should suffice. There may also be other ways to avoid damage from preimage attacks by taking care how we use the hash.
GAS (Google Apps Scipt) supports SHA-512. Why don't we go with that:
Enum DigestAlgorithm
'

I'm looking for a way to get a form that reacts on the given answers.
Question 1 has answers Yes an No
If you answer Yes then you see Question 2
else you see Question 3

Done with logical branched sections (standard feature)
https://docs.google.com/forms/d/e/1FAIpQLSecwGUVFNx_Xa_Qsw5bxLnaKstPS8kQnfrUGqpuf22rLDteDg/viewform


add simoleon website

I don't understand this error. can someone help?

Sorry, I'm a plain contributor just like you who can't merge PRs. I have my own correction PRs waiting forever too.

Sorry, I'm a plain contributor just like you who can't merge PRs.

sorry I tagged random people. do you know who I could tag?

https://github.com/solana-labs/token-list/commit/61f5f8b700316c27746646d2343f9ac567b05ab5

Wow! your prayers have been heard!
Hey @keone do you think you could merge my 2 open PRs modifying my own tokens?

https://github.com/solana-labs/token-list/pull/5159
https://github.com/solana-labs/token-list/pull/4993


[stable/gitlab-ce] Update to the recommended pvc patterns.

This is a split-off of PR #408 to just contain the chart gitlab-ce.
If there's any commonality in changes nessary between this PR and the others that are being submitted at once, please mention them in #408.
Some things to note:

This initial PR will not be updating the Chart.yaml's version string, as I feel it should be done manually per-pr once each chart's otherwise acceptable.
An automated script created the splits and the PRs, on a per-folder basis under stable, after copying the relevent files from a backup of my local copy of #408

@k8s-bot ok to test

I just created https://github.com/kubernetes/charts/pull/523 which should be merged before this to ensure that all the dependent charts are using the right fullname truncation.

@k8s-bot ok to test


testing Dialog

I am testing a method which opens a dialog by myDialog.open() and I am checking it as _get(Dialog.class)  But getting `java.lang.AssertionError: /my-view: No visible Dialog in UI[] matching Dialog
Is there any different way to test Dialog? I am using karibu-testing-v10-spring with vaadin 14.

The dialog should append itself as a child to the UI upon open, therefore _get(Dialog.class) should be able to locate it. Can you observe this behavior?

The UI itself.

Can you please elaborate what do you mean by "Can you observe this behavior?", The dialog opens and I can see it when the app is running, but in tests it doesn't get append to UI, and I am not getting any errors

Sure thing. Say you have this kind of a test:
    @Test
    public void testGreeting() {
        new Dialog().open();
        MockVaadin.clientRoundtrip();
        System.out.println(PrettyPrintTreeKt.toPrettyTree(UI.getCurrent()));
    }

It should print the following tree:
└── MockedUI[]
    ├── MainView[@class='centered-content', @style='width:100%', @theme='padding spacing']
    │   ├── TextField[label='Your name', value='', @class='bordered']
    │   └── Button[text='Say hello', @theme='primary']
    └── Dialog[]

Note that the Dialog component attaches itself as the direct child of the UI (represented by the MockedUI component). Also note the clientRoundtrip() function - dialog attaches itself lazily by default, and clientRoundtrip() forces eager attachment.
In your case, can you see the Dialog component in the component dump?

Thanks. It worked. I just added MockVaadin.clientRoundtrip() and _get(Dialog.class) was able to locate it.
I didn't know about MockVaadin.clientRoundtrip(). I am closing this issue now.

What's strange is that _get() should call MockVaadin.clientRoundtrip() automatically - you should not have to worry about it. I'll rather reopen this ticket and investigate.

Ok.
FYI, while closing the dialog I needed to use MockVaadin.clientRoundtrip()

I've upgraded Karibu-Testing testing suite to Vaadin 14.11.5 and ran the test suite. The test suite is really extensive and tests all sorts of scenarios, including the Dialogs. The tests seem to pass and I don't have to call MockVaadin.clientRoundtrip() manually. In other words, I can't reproduce the issue. I'll need the simplest possible app/test case which reproduces the issue.
Side question: Maybe you have hooked into TestingLifecycleHook and removed the call to clientRoundtrip()?

Right, I have hooked into TestingLifecycleHook, I didn't know that caused it. Thanks

Alrighty, no worries. I've updated the docs to stress out the importance of calling the original hook.

Is there any case in which we get NullPointerException on MockVaadin.clientRoundtrip() ?


feat: More robust multi-threaded speed testing with more generalized provider support

New Fast.com provider is not exposed to the user currently, but once options are added it can be.  This does a better job of evaluating high speed connections and follows more closely to official testers behavior.
I have imported some of my own code here, so if it looks like overkill that is likely why.  This gets me very reliable results over  many tests.   It should also scale the test to the size of the connection, like the actual tests.
No longer directly sub-classing HttpClient as I didn't see the benefit of going that route.

I'll merge it, when I'm doing a release.

Try again if still an issue please send the log:)


No source file named，，，

when i used the docker-dev image develop docker source，use the gdb tool like that，but，a error like “No source file named " happend.what wrong?
(gdb) pwd
Working directory /go/src/github.com/docker/docker/docker.
(gdb) cd
README.md  client.go  daemon.go  docker.go  flags.go
(gdb) cd
README.md  client.go  daemon.go  docker.go  flags.go
(gdb) break docker.go:20
No source file named docker.go.
Make breakpoint pending on future shared library load? (y or [n]) n
(gdb) pwd
Working directory /go/src/github.com/docker/docker/docker.

i resolve it

hi, i met the same problem, how did you solve it? thanks.


solana: figure out why solana CLI >=1.17 doesn't work

With 1.17, test validator dies with validator logs resembling:
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
thread 'solRpcEl' panicked at runtime/src/bank.rs:4387:14:
called `Option::unwrap()` on a `None` value
stack backtrace:
   0: rust_begin_unwind
             at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/std/src/panicking.rs:595:5
   1: core::panicking::panic_fmt
             at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/core/src/panicking.rs:67:14
   2: core::panicking::panic
             at /rustc/cc66ad468955717ab92600c770da8c1601a4ff33/library/core/src/panicking.rs:117:5
   3: solana_runtime::bank::Bank::simulate_transaction_unchecked
   4: solana_runtime::bank::Bank::simulate_transaction
   5: <solana_rpc::rpc::rpc_full::FullImpl as solana_rpc::rpc::rpc_full::rpc_impl_Full::gen_server::Full>::send_transaction
   6: <jsonrpc_core::delegates::DelegateMethodWithMeta<T,F> as jsonrpc_core::calls::RpcMethod<M>>::call
   7: jsonrpc_core::middleware::Middleware::on_call
   8: jsonrpc_core::io::MetaIoHandler<T,S>::handle_request
   9: jsonrpc_http_server::handler::RpcHandler<M,S>::process_body
  10: <jsonrpc_http_server::handler::RpcHandler<M,S> as core::future::future::Future>::poll
  11: <jsonrpc_http_server::handler::RpcHandler<M,S> as core::future::future::Future>::poll
  12: hyper::proto::h1::dispatch::Dispatcher<D,Bs,I,T>::poll_inner
  13: hyper::proto::h1::dispatch::Dispatcher<D,Bs,I,T>::poll_catch
  14: <hyper::server::conn::upgrades::UpgradeableConnection<I,S,E> as core::future::future::Future>::poll
  15: <hyper::server::server::new_svc::NewSvcTask<I,N,S,E,W> as core::future::future::Future>::poll
  16: tokio::loom::std::unsafe_cell::UnsafeCell<T>::with_mut
  17: tokio::runtime::task::core::Core<T,S>::poll
  18: tokio::runtime::task::harness::Harness<T,S>::poll
  19: tokio::runtime::scheduler::multi_thread::worker::Context::run_task
  20: tokio::runtime::scheduler::multi_thread::worker::Context::run
  21: tokio::runtime::context::scoped::Scoped<T>::set
  22: tokio::runtime::context::runtime::enter_runtime
  23: tokio::runtime::scheduler::multi_thread::worker::run
  24: tokio::loom::std::unsafe_cell::UnsafeCell<T>::with_mut
  25: tokio::runtime::task::core::Core<T,S>::poll
  26: tokio::runtime::task::harness::Harness<T,S>::poll
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

With >=1.18, <1.18.8, stack errors occur when invoking any instructions even with a refactor of account contexts.
And with >=1.18.8, stack errors can be resolved with a refactor of account contexts, but there is an issue with deserialization. One example is with the initialize instruction, where inputs resemble:
    const auctionParams: AuctionParameters = {
        userPenaltyRewardBps: 250_000, // 25%
        initialPenaltyBps: 250_000, // 25%
        duration: 2,
        gracePeriod: 5,
        penaltyPeriod: 10,
        minOfferDeltaBps: 20_000, // 2%
        securityDepositBase: uint64ToBN(4_200_000n),
        securityDepositBps: 5_000, // 0.5%
    };

But when debugging the Anchor instruction data, we get:
AuctionParameters { user_penalty_reward_bps: 1183200, initial_penalty_bps: 0, duration: 11768, grace_period: 0, penalty_period: 2, min_offer_delta_bps: 32720, security_deposit_base: 4200000, security_deposit_bps: 3 }

So right now, it is impossible to consider building without solana CLI 1.16.27.

And may be related to https://github.com/coral-xyz/anchor/issues/2835.

With 1.17, test validator dies with validator logs resembling:

Attaching an example log here.
validator-1708006722959.log

Be careful with CI like here: https://github.com/wormhole-foundation/example-native-token-transfers/pull/391.

In playing around with upticking another repo to Anchor 0.30.0, there may be hope here.
Also be aware of this: https://github.com/anza-xyz/agave/issues/522, which will require us to write a custom script to load forked programs because any instruction that performs CPI to any of these loaded programs will cause the following error in our test validator:
SendTransactionError: failed to send transaction: Transaction simulation failed: This program may not be used for executing instructions


UGP-11286

Please always provide the Jira Issue your PR is for, as well as test URLs where your change can be observed (before and after):
Jira ID: UGP-11286
Test URLs:

Before: https://main--exlm--adobe-experience-league.hlx.page/en/playlists
After: https://UGP-11286--exlm--adobe-experience-league.hlx.page/en/playlists?martech=off

Nice work on the PR!
One last item: Performance score is at 90, we can make it better, Let's discuss later today


Preprend regex string literals with an 'r' making them raw strings

Closes #39

@Hoffelhas If you would prefer this pushed to the 2.0.1 branch, let me know and I can reopen it for the minor release


Rz misclosure

Adapted computeMisclosure script work with unwrapStack.vrt file. Numerous improvements to the functionality and readability of the code implemented. Documentation and examples updated.

@rzinke the two-stage third-party dependency is creeping back in.


How to optimize Ubuntu 16.04 to reach the same level of performance as the 18.04 Quick Create VM?

Test Device - Surface Book 2 15" i7/16 GB/512 GB & Surface Pro 5 i7/16 GB/ 512 GB
VM - 4 Core/ Gen 2/ dynamic RAM & 2 Core/ Gen 2/ dynamic RAM
I have configured XRDP for Ubuntu 16.04 as per the steps in the onboarding session. The cursor movements are smooth however there's significant lag between keyboard key press and text display on screen.
Comparing with the quick create VM in Hyper-V (Ubuntu 18.04) on the same devices -

Virtually zero lag in text input in 18.04
18.04 VM boots faster
Application loading time (e.g. terminal/ firefox) is lower in 18.04

How can I further optimize my 16.04 VM to reach the same level of performance as the 18.04 Quick Create VM?

This repo is being archived, so closing this issue.

This repo is being archived, so closing this issue.


Updated JS admin translation catalog

@felixxm I'm so sorry, I may have missed that file in older releases, too.

@claudep np, Thanks :star:


JSX not resolved

Hi, I'm trying to make this work with my project, but again I cannot make the JSX to resolve.
I even added my own webpack.config.js with a following content but I still have to explicitly specify import ./a.jsx. This is a no-go for me since I'm compiling files from typescript.
const path = require('path');
module.exports = {
  module: {
    resolve: {
      extensions: ['', '.js', '.jsx']
    },
    loaders: [
      {
        test: /\.jsx?$/,
        loader: 'babel',
        query: { presets: ['react', 'es2015', 'stage-2'] },
        exclude: [path.resolve('./node_modules'), path.resolve(__dirname, 'node_modules')],
        include: [path.resolve('./'), __dirname],
      },
      {
        test: /\.css?$/,
        loaders: [ 'style', 'raw' ],
        include: path.resolve(__dirname, '../')
      }
    ]
  }
};

I tried to use pure typescript with a following config, yet it does not even resolve ts and tsx.
var path = require('path')
var webpack = require('webpack')
console.log("dd");
module.exports = {
  devtool: 'cheap-module-eval-source-map',
  entry: [
    'webpack-hot-middleware/client',
    './index'
  ],
  resolve: {
    extensions: ['', '.js', '.jsx', '.ts', '.tsx']
  },
  output: {
    path: path.join(__dirname, 'dist'),
    filename: 'bundle.js',
    publicPath: '/static/'
  },
  plugins: [
    new webpack.optimize.OccurenceOrderPlugin(),
    new webpack.HotModuleReplacementPlugin()
  ],
  module: {
    loaders: [
      {
        test: /\.jsx?$/,
        loaders: [ 'babel' ],
        exclude: /node_modules/,
        include: __dirname
      },
      {
        test: /\.tsx?$/,
        loaders: [ 'ts' ],
        exclude: /node_modules/,
        include: __dirname
      },
      {
        test: /\.css?$/,
        loaders: [ 'style', 'raw' ],
        include: __dirname
      }
    ]
  }
}

Try to run a fork and add ts loader to here: https://github.com/kadirahq/react-storybook/blob/master/src/server/webpack.config.js#L30
But above should work too. You need to use path.resolve('../', __dirname) instead of direct dirname.

@arunoda still no luck. I have downloaded the react-storybook-demo and renamed Header.js to Header.jsx and used following webpack.config.js and still am receiving the error:
var path = require('path')
var webpack = require('webpack')
console.log("dd");
module.exports = {
  devtool: 'cheap-module-eval-source-map',
  entry: [
    'webpack-hot-middleware/client',
    './index'
  ],
  resolve: {
    extensions: ['', '.js', '.jsx']
  },
  output: {
    path: path.join(__dirname, 'dist'),
    filename: 'bundle.js',
    publicPath: '/static/'
  },
  plugins: [
    new webpack.optimize.OccurenceOrderPlugin(),
    new webpack.HotModuleReplacementPlugin()
  ],
  module: {
    loaders: [
      {
        test: /\.jsx?$/,
        loader: 'babel',
        query: { presets: ['react', 'es2015', 'stage-2'] },
        exclude: [path.resolve('./node_modules'), path.resolve(__dirname, 'node_modules')],
        include: [path.resolve('./'), __dirname],
      },
      {
        test: /\.css?$/,
        loaders: [ 'style', 'raw' ],
        include: __dirname
      }
    ]
  }
}

I do not understand what is going on, looks like you do have support of jsx in the core: https://github.com/kadirahq/react-storybook/blob/master/src/server/webpack.config.js#L28-L34 but it just does not work even in your demo app.

SOLVED. I disected the way you compose the final config and following works (please see the "resolve" section in the parent of the child configuration):
const path = require('path');
module.exports = {
  resolve: {
    extensions: ['', '.js', '.jsx']
  },
  module: {
    loaders: [
      {
        test: /\.jsx?$/,
        loader: 'babel',
        query: { presets: ['react', 'es2015', 'stage-2'] },
        exclude: [path.resolve('./node_modules'), path.resolve(__dirname, 'node_modules')],
        include: [path.resolve('./'), __dirname],
      },
      {
        test: /\.css?$/,
        loaders: [ 'style', 'raw' ],
        include: path.resolve(__dirname, '../')
      }
    ]
  }
};

@tomitrescak can you add more clear description how did you manage to add a typescript support for the storybook?

How I managed it works with tsx:
const path = require('path');
module.exports = {
  module: {
    loaders: [
      {test: /\.css$/, loaders: ['style', 'css']},
      {test: /\.(tsx|ts)$/, exclude: /node_modules/, loaders: ['ts-loader']}
    ]
  },
  resolve: {
    extensions: ['', '.js', '.jsx', '.tsx', '.ts']
  },
  ts: {
    configFileName: path.resolve(__dirname, '../ts.conf.json')
  },
}

Hope this will help anybody.

Looks good! I have a bit different approach as I first compile to JS and let Storybook load javascript. I know it seems weird, but I like it this way, it's generally much faster (or it was when I was testing it couple years ago :)

Hmm, I tried with pure TS and no luck again :/ Looks like the resolver is not called for my ts files :(
// Export a function. Accept the base config as the only param.
module.exports = (storybookBaseConfig, configType) => {
  // configType has a value of 'DEVELOPMENT' or 'PRODUCTION'
  // You can change the configuration based on that.
  // 'PRODUCTION' is used when building the static version of storybook.

  // Make whatever fine-grained changes you need
  storybookBaseConfig.externals = {
    'jsdom': 'window',
    'cheerio': 'window',
    'react/lib/ExecutionEnvironment': true,
    'react/lib/ReactContext': 'window',
    'react/addons': true,
  };

  storybookBaseConfig.module.loaders.push(
    {
      test: /\.css?$/,
      loaders: ['style', 'raw'],
      include: path.resolve(__dirname, '../')
    }
  );

  storybookBaseConfig.module.loaders.push({
      test: /\.tsx?$/,
      loader: 'ts-loader',
      exclude: /node_modules/
  });

  storybookBaseConfig.devtool = 'eval';
  storybookBaseConfig.resolve = {
    extensions: ['', '.js', '.jsx', '.ts', '.tsx']
  };

  // Return the altered config
  return storybookBaseConfig;
};

@tomitrescak, try to add few more lines to your code:
module.exports = (storybookBaseConfig, configType) => {
  /* you code */
  storybookBaseConfig.ts = {
    configFileName: path.resolve(__dirname, '../path-to-ts-config.json')
  }
  return storybookBaseConfig;
};


Attribution line found in non-templated CC-BY/BY-SA license texts

License file: LICENSE
License hash: bec83478a124efb1beb8500ee952ffcc80a87b2b
Attribution: copyright and certain other rights. Our licenses are
License: Creative Commons Attribution 4.0
Confidence: 100.00%
Method: Licensee::Matchers::Dice

The regex defined at https://github.com/benbalter/licensee/blob/cbd96aed7dfd8e91ef780e029e5e6015c3486077/lib/licensee/matchers/copyright_matcher.rb#L9 and used at https://github.com/benbalter/licensee/blob/cbd96aed7dfd8e91ef780e029e5e6015c3486077/lib/licensee/project_files/license_file.rb#L43 matches some text within these licenses.
Few potential ways to fix:

Don't look for attribution in licenses known not to be templates (eg without [fill-in-fields])
Only look for attribution in first part of license text (some reasonable number of lines or characters)
Hardcode avoidance of particular matched line above

@mlinksva Can you point to an example license file? (e.g., how'd you generate the above output)? Think I know what the problem is, and should be an easy fix.

I ran on https://github.com/github/open-source-guide to get output, should've noted!

Fix in https://github.com/benbalter/licensee/issues/170.


Local Generation Failing on Hash Check

New issues must complete all sections of this template or they will be
automatically closed.
Use this template if your bug does not relate to a seed being completable due
to relic placement.
Seed Information
All Seeds
Randomizer version:
3.5.9
Operating system and version:
Windows 10
Version 20H2 (OS Build 19042.928)
Browser and version:
Running Locally, Powershell v.5.1.19
Emulator and version:
Not getting that far.
Bug description
PS F:\Repos\Rando> node randomize -p safe -vvv
Seed:<PHONE_NUMBER>939
TypeError: Cannot read property 'subtle' of undefined
at sha256 (F:\Repos\Rando\src\util.js:32:19)
at checked.sum (F:\Repos\Rando\src\util.js:672:12)
at randomize (F:\Repos\Rando\randomize:944:33)
at Object. (F:\Repos\Rando\randomize:1079:3)
at Module._compile (internal/modules/cjs/loader.js:1063:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1092:10)
at Module.load (internal/modules/cjs/loader.js:928:32)
at Function.Module._load (internal/modules/cjs/loader.js:769:14)
at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:72:12)
at internal/main/run_main_module.js:17:47
The same error happens when I put my Bin file as an input, or when I specify a ppf file as an output.
How to reproduce:
Running the command locally.  Not sure if I'm missing a prerequisite that's needed to run the Hash check locally.

Upgrade your node installation to at least 15.x


Update tutorial.md to reflect new required qubit method

Due to https://github.com/quantumlib/Cirq/pull/833

I'll take this.

Not sure what I need to update. The tutorial says GridQubits implement the QubitId class, which just means that they are equatable and hashable. This hasn't changed, right? I mean QubitId still contains those methods. Do i need to add a blurb about howQubitId sorts?


Proposal: Support default custom output path

Problem:
Currently there is no official support for setting a default custom path. This can be important for those that have different project structures or use a different server to host angular2 such as asp.net core which expects static files to be in a folder called wwwroot/.
There is also not a consistent way to unofficially set a default value for custom output path. For ng buld and ng serve as they are still just the default commands in ember cli you can create a .ember-cli file with the following and it will work
{
    "output-path": "wwwroot/"
}

However, for ng test it has the output folder hard-coded to dist/ and as it is, as an add-on command would not use .ember-cli
Proposal:
I was only able to find three commands that use an output-path flag, build, serve, test. The following are what I think would need to change or be completed to support this feature.

General

Support output path in angular-cli.json as a sub property of defaults. Would probably be called outputPath to match the cli flag output-path and the format in angular-cli.json already. Would look something like:
{
    "defaults": {
        "outputPath": "wwwroot/"
   }
}

_Required change:_ Update angular-cli.json schema

Test command

Remove hard-coded output path and instead retrieve it from
_Required change:_ Use this.project.ngConfig to get default settings and use it instead to construct buildOptions

Add output-path flag to be consistent with other commands.
_Required change:_ Add another option to availableOptions

Build Command

Create a BuildCommand in addon/ng2/commands. This would be a wrapper of the ember cli build command but would read default settings from angular-cli.json

Serve Command

Create a ServeCommand in addon/ng2/commands. This would be a wrapper of the ember cli serve command but would read default settings from angular-cli.json

Note: As I have told @hansl I am willing to go through the design process and work on this.

other use cases:

the need to have a different output path for different environments, see #801

@filipesilva @hansl what is the next step? Design docs or can I start working on this?

@JonCubed you're most welcome to make a PR for this functionality, make sure to add tests that verify it works as well.

@filipesilva sure no problems

Implemented as outDir in angular-cli.json currently. See https://github.com/angular/angular-cli/pull/1109#issuecomment-252507439 also.

Neither of option to create custom distribution folder is working for me. I tried following

ng build --outDir=customDist
ng build --output-path=customDist
ng build --outDir customDist
ng build --output-path customDist

nothing worked :(
Could anyone please help me out.
Thanks. Rahul

@rahul-winner looking at build.ts ng build --output-path customDist should work. Alternatively if you want it to be a default and not have to use the --output-path flag you can use --source-dir with ng new or ng init commands


Timeout after 30 sec

Hi, I can't upload large files. After 30 seconds, the connection is aborted.
If I configure dropzone.forcefallback=true then it works. But I think there is a hard coded limit of 200MiB in the file:
node_modules/formidable/lib/incoming_form.js line 28:
this.maxFileSize = opts.maxFileSize || 200 * 1024 * 1024;
Anybody managed to upload large files ?

Well spotted! I don't know Javascript but I guess opts.maxFileSize isn't being set so the file size defaults to 200MB. I changed || 200 to || 600 and was able to upload a 300MB. I had problems with that before.
The upload took longer than 30 seconds and I haven't had a connection abort.
Debian 9.5.0 amd64, setup_6.x from nodesource and the 1.0.6 zip.

I had the same problem and managed to solve it by increasing the KeepAlive Timeout in the apache2.conf .
After that I was able to upload a 8GB file successfully.
Hope this helps or at least can guide you into the right direction.

Well, we have to consider different use cases:

Without Apache (or Nginx or any kind of reverse proxy)
1a) with dropzone.forcefallback=false
1b) with dropzone.forcefallback=true
With Apache
2a) with HTTP/2
2b) without HTTP/2

The behaviour is different in all these cases. At first, I would forget Apache, because it's only plumbery. The main problem resides in YouTransfer itself I think, maybe in Dropzone.

There is no timeout specified. If we add a timeout of say 24 hours (timeout: 86400000) to the options object in fileupload.js then the problem will be solved. I've tested it.

After adding timeout parameter as @DNS-404 said, it solve the 30s timeout problem, but the file size cannot over 1G, otherwise it will has the File size is greater than possible Buffer: 0x3FFFFFFF error. it seems the nodejs's limit.
as I see, the big file is already in folder of root@a33a89536b11:/opt/youtransfer/upload_xxxx, but something wrong when moving them to uploads folder.

I'm getting the same problem when uploading a file above 1G.   How to get around this?  something in nodejs limit seems to be the problem indeed!


Unable to access local installed MetaMorpheus from non-administrative user (Windows)

Hi,
I'm having an issue with making MetaMorpheus available to all users in the windows machine.
I installed it with one of administrative account but non-administrative account can't even open the gui.
I tried with "run as" with administrative account but no luck so far.
What would be the best practice to make MetaMoepheus available to all users?
Thank you for sharing the awesome search engine.
best,
heejong

that is an interesting question. we will discuss it and respond to you soon.

Hi Heejong,
This is possible and we can hopefully make this happen in the release after next. So probably version 0.0.310. We use Wix to make our installer version, and I believe it's possible to code it to install MM for all users. I'd like to have this be an optional setting in the installer, which may be a bit trickier, hence the long-ish delay. But I will definitely look into this for you.

I was able to reproduce this error.
It looks like the installer settings are okay. Spritz installs with the same perMachine setup, and Spritz will start up from non-admin accounts, but while MetaMorpheus is visible in C:\Program Files, it doesn't pop open.

It looks like dotnet is available in both admin and non-admin accounts if installed. I wonder if there's something else about the admin environment that's missing.