[GJ] Translation to Gujarati - Ch0 Setup

This pull request is to add Gujarati Language as a translation for the course.
This is specifically for Chepter0- Setup
Issue: #284

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Hi @lewtun,
Sure, i have started working on translating chapter 1.


UI for reporting errors in config/operations

This issue is based on an older conversation which happened around version 0.138.
Problem
It's easy to miss certain errors in wundergraph.config.ts.
Cause
This happens due to combination of two factors:

wundergraph keeps going after errors in config and in operations
errors get log in logs scroll.

The "keep going despite errors" strategy is useful for interactive development (you can edit a .graphql operation and the server will keep going which the operation is half-complete).
However non-trivial config might contain number of operations. Each operation generates a log line "operation registered". As the result errors about skipped operations get lost in log scroll.
In my experience, I don't use interactive development feature because I need to scroll up to check if there are any issues in operations. As the result, I Ctrl+c, Ctrl+l to clean the terminal and then start the server again.
Suggested solution
I personally, would prefer a cli switch to stop on the first error.
Alternatively, WG team mentioned development of a "terminal UI" to get list of errors.

We just released our new TUI in 0.144.1 and love to get your feedback on this.


[Bug] Nested variables do not work in the Base Environment

Insomnia 6.0.1
Details
It appears that nested variables do not work when defined in the Base Environment:
"db": {
  "password": "foo"
}

{{ db.password }} yields attempted to output null or undefined value.
Workaround is to define nested variables inside of each environment.

I have a feeling you're probably overwriting the db property in a sub environment (?)
Note: environments are joined using a shallow merge.

I am using the same nested structures in the Base environment and sub-environments. I was assuming that sub-environments would be deep merged on top of the base.

No, it's a strict overwrite, unfortunately

Any reason why it shouldn't be a deep merge? It makes the nested environment variables less useful if they can't be abstracted to the base environment and overridden where needed.

There are pros and cons to both methods. Shallow merge was chosen at the beginning so that's what it is. It would be difficult to change now without breaking existing users setups

I'm actually not sure this would break anyone. If db.password in the base environment suddenly starts working, sub-environments that currently use that variable would still override it.

There are many other use cases, many of which we can't even imagine.
One example is if someone includes a user object in a base environment and overrides it in a sub environment with different keys. Perhaps they are serializing it to JSON and sending it in the request body.


Add Azure OpenAI Service support

Fix for #93

I bumped the openai-client to 1.0.12, but we should still address this issue.

I pushed the update, but didn't have a chance to test it properly - as soon as I get back to my other machine, I'll check if it still works correctly.

Yeah, you are sending API key on Authorization header although it should be api-key.

Yeah, you are sending API key on Authorization header although it should be api-key.

It should support both, our company uses Bearer authentication for this and the api-key won't work.

Yeah, you are sending API key on Authorization header although it should be api-key.

If anything, the plugin should support both. Our company uses Bearer authentication for this and the api-key won't work. And this is the default approach for fine-grained Azure access  (we do have a possibility to work with different types of keys, which we can put in the api-key header, but they are for development purposes. The one used on daily basis are for Bearer auth and generated by Azure's cognitiveservices).

I'm not sure if there's a way to detect the necessary auth type. I noticed, that regular api-key's are rather short, while the Bearer ones are 1988 characters long.
We could always send both headers, or conditionally based on the key length, or add yet another checkbox in settings for authentication type. Neither of which seems clean.
Best would be to introduce complete authentication with azure-identity, but that's not that trivial to implement (at least with my limited familiarity with Azure). Currently I simply run az login & az account get-access-token, which I paste to the CodeGPT settings.

I think let's not overcomplicate things (at least for now).
Sending both headers might not work, or even if it does, you can't really guarantee which header is actually being used.
Setting the header based on the key length seems a bit hacky.
I like the checkbox logic, it's easy to implement and provides info which auth method is actually being used. Besides, the Azure doc says "you can use either API Keys or Azure Active Directory".


Move the content of the subheader into a footer

The grey line that shows the owner of the current Pod (that is shown when you're visiting another Pod) can be a bit obtrusive. We suggest that we move it into a footer.
Additionally, we suggest to make this footer "sticky", as described at https://css-tricks.com/couple-takes-sticky-footer/ .

Tim also suggest us adding a link to https://solidproject.org/ and to link to the root of the Pod in the text "the Pod".

I propose to make the footer as simple as possible, as shown in the example below:

This was done in https://github.com/solid/mashlib/pull/49


PE-42104 - Adding Jobstat to SAT CLI (continuation of PR-38)

A new feature named "Jobstat," allows users and admins to get a system-wide view of application status data via the SAT toolkit. The change is backward compatible as long as it has access to the elasticsearch API. The feature is meant to be implemented on any system running PALS. Currently, it is being developed for Slice.
Original PR-38, cannot be changed as the intern went back to university after the summer. So, creating a new PR instead.

I am not able to sync with the main sat codebase as this code is a fork from Elmir's code/fork. Can we take this to a branch in Cray-HPE/sat (not master)?


Support for adding multiple swift-bridge packages

Hello, is there a way to add multiple swift-bridge packages to an Xcode project? If I try to add an additional package using this method https://chinedufn.github.io/swift-bridge/building/swift-packages/index.html, I get an error from Xcode saying that there are conflicts. Here is a screenshot:

Is there a workaround to this? Thank you!

So the workaround for this, per the error, was to rename RustXcframework in the second module to RustXcframework2. This seems to work ok as a hack for now.

Sorry to bother, but reopening this because I'm unable to compile the project. I get the errors:
Multiple commands produce '...../Build/Products/Debug-iphoneos/include/SwiftBridgeCore.h'

Multiple commands produce '...../Build/Products/Debug-iphoneos/include/module.modulemap'

Not sure how to get around this.

@annethereshewent
Could you share a minimal example to be able to reproduce the error?

Hey there, I was able to get around this by moving some files around. I'm not sure this is a bug but more of a limitation of how you import modules into swift. I used the how-to in the readme of this repo to get around the issue for now. https://github.com/jessegrosjean/swift-cargo-problem


Removing unused test keys

What? and Why?
Removing test keys as they are not used anywhere
Checklist

[ ] CHANGELOG.md updated? (if required)

Codecov Report

Merging #40 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #40   +/-   ##
=======================================
  Coverage   83.71%   83.71%
=======================================
  Files          10       10
  Lines         571      571
=======================================
  Hits          478      478
  Misses         93       93

Continue to review full report at Codecov.

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


Ensure Orbit used the threading option

See comments in https://github.com/pyvista/pyvista/pull/719

Done


Implement Union-Find algorithm

This PR includes the creation of visualisers and logic for Union-Find algorithm. An image of the Union-Find algorithm is shown below and can also be accessed at https://aia.davidsha.me (temporary).

Features

[x] Union operations and find operations are properly visualised in both an array view and tree view format.
[x] Included the ability to toggle on and off path compression during a find operation.
[x] Included adequate background information, more information, and instructions into the respective tabs on the right.

Changes
There were also some changes made to the codebase for the betterment of the developer and code reviewer.

[x] A Union-Find specific GitHub Actions workflow unionFind.yml has been created for testing the UF algorithm functions run as intended.
[x] npm build and npm start now runs through a newly added package run-script-os so that Windows users and Unix users alike can work on the same codebase without issues.
[x] npm run test-uf script runs the union-find jest tests.
[x] Recommended VSCode workspace settings are included so that a new developer may have the most luck when diving into the codebase in the future.
[x] prettier.config.js has been updated to include semicolons as part of formatting convention.
[x] parser.js now includes functionality for \Note blocks within \Code blocks.
[x] parser.js now also includes functionality for a \NewLine which is equivalent to a \n.
[x] Fixed displaying explanations when "?" button is pressed. Before, explanations were falsely shown when clicking on a line, as opposed to clicking on the actual "?" button.

Merged with #34.


tests: drivers: Fix coverity issue 198873

Fix coverity issue 198873
Signed-off-by: Mieszko Mierunski<EMAIL_ADDRESS>Fixes #16574
I don't know how to check if coverity issues are fixed locally, if there is some test that I should perform please let me know

@Mierunski please, rebase this one


MetaLidar: Windows Compatibility?

Why is Linux the only supported system for developing Unreal Engine projects?
Can I not use Unreal Engine on a Windows platform?

Hi, @wakasu
I did not dig into this issue, I remember not being able to get an accurate timestamp when testing on Windows. This plugin requires a high-resolution timestamp to create an accurate data packet.

Apologies for the delayed response.
My objective is to have a simulation environment running on Windows. However, for the time being, I will be setting up a Linux development environment. I will also look into the accuracy of timestamps when working on Windows.
Cheers.


Enable previewing drafts

We need to factor in draft previewing.  Postlight's implementation does this nicely and we should follow suit.

Functionality added.  See bd6ec03.


Leave and Document Request Letter Generator

Key features:

Responsive
User Friendly UI
Option to Download letter as PDF

Tech Stack:
HTML, CSS and JS

@gurjeetsinghvirdee Please assign this issue.


[blocked upstream] parse tslint JSON output

This passes warning and span information on to neomake.
Fixes #1783.

process_json has been merged by now (#1848), so could (and should) be used here.

Awesome. Added them in the latest commit.

i wonder if recently merged #2010 json updates will help here?

@davidosomething This is effectively blocked on https://github.com/palantir/tslint/pull/3816

I am closing this for now, given that there appears to be no work done upstream for this.
Happy to re-open later, of course.


[SPARK-13063] [YARN] Make the SPARK YARN STAGING DIR as configurable

What changes were proposed in this pull request?
Made the SPARK YARN STAGING DIR as configurable with the configuration as 'spark.yarn.staging-dir'.
How was this patch tested?
I have verified it manually by running applications on yarn, If the 'spark.yarn.staging-dir' is configured then the value used as staging directory otherwise uses the default value i.e. file system’s home directory for the user.

Can one of the admins verify this patch?

ok to test

Test build #54622 has started for PR 12082 at commit 1afb29a.

Test build #54622 has finished for PR 12082 at commit 1afb29a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54622/
Test PASSed.

Thanks @tgravescs for looking into the patch.

Test build #54695 has started for PR 12082 at commit 42848ad.

Test build #54695 has finished for PR 12082 at commit 42848ad.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54695/
Test PASSed.

Test build #55007 has started for PR 12082 at commit 49ac38c.

Test build #55007 has finished for PR 12082 at commit 49ac38c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/55007/
Test PASSed.

+1


[BUG] Hydrogen theme fails to deserialize

Describe the bug
TOML deserialization of Hydrogen.toml theme fails.
To Reproduce
Steps to reproduce the behavior:

Copy contrib/themes/Hydrogen.toml to your local themes folder
Try to use Hydrogen theme.

Expected behavior
Macchina output appears in the Hydrogen theme.
Actual behavior
Macchina output appears in the default theme, silently failing to apply the Hydrogen theme.
System Information
You don't have to provide this information if you're not comfortable doing so, but it'll help us solve the issue a lot faster.

Operating System: macOS
Terminal: iTerm2
Macchina's version: 6.0.1

The error message we get from the toml deserializer internally is:
Could not create theme Hydrogen.toml: ParsingError(Error { inner: ErrorInner { kind: Custom, line: Some(8), col: 18, at: Some(166), message: "invalid type: string \"Light\", expected struct Palette", key: ["palette"] } })
Which means, that the line palette         = "Light" in the Hydrogen.toml file is wrong. Palette should be expanded into its own section like in the other themes.

I'll push out a fix in v6.0.2

Hydrogen displays properly with https://github.com/Macchina-CLI/macchina/commit/1e7fff019c0b763768dc79272d5fd23afae742ea


fix(editorUtils): update card JSON when group by multiselect changes

Closes https://github.com/carbon-design-system/carbon-addons-iot-react/issues/3674
Summary

Within Dashboard Editor, a data table card with existing columns does not update the card JSON when the "Group by" multi-select has new items selected which are also added as columns. Similarly, when unchecking items from "Group by" multi-select they are not removed from groupBy array.

Change List (commits, features, bugs, etc)

fix(editorutils): update columns destination property when group by multiselect changes
fix(editorutils): add test scenario for data table card data grouping

Acceptance Test (how to verify the PR)

adding/removing group by data items should correctly reflect in the card json

Regression Test (how to make sure this PR doesn't break old functionality)

columns are added to the table when group by data items is selected

Things to look for during review

[ ] Make sure all references to iot or bx class prefix is using the prefix variable
[ ] (React) All major areas have a data-testid attribute. New test ids should have test written to ensure they are not changed or removed.
[ ] UI should be checked in RTL mode to ensure the proper handling of layout and text.
[ ] All strings should be translatable.
[ ] The code should pass a11y scans (The storybook a11y knob should show no violations). New components should have a11y test written.
[ ] Unit test should be written and should have a coverage of 90% or higher in all areas.
[ ] All components should be passing visual regression test. For new styles or components either a visual regression test should be written for all permutations or the base image updated.
[ ] Changes or new components should either write new or update existing documentation.
[ ] PR should link and close out an existing issue

This all looks fine but the "steps" in acceptance and regression test area seem to happen in next

@herleraja ?


FINERACT-1722: Fix loan repayment schedule with multitranche

Description
The is an issue in the Loan repayment schedule, when the Loan product is multidisbursed and the disbursed amount is different (lower or higher) than approved amount
Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!

[ ] Write the commit message as per https://github.com/apache/fineract/#pull-requests

[ ] Acknowledge that we will not review PRs that are not passing the build ("green") - it is your responsibility to get a proposed PR to pass the build, not primarily the project's maintainers.

[ ] Create/update unit or integration tests for verifying the changes made.

[ ] Follow coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions.

[ ] Add required Swagger annotation and update API documentation at fineract-provider/src/main/resources/static/legacy-docs/apiLive.htm with details of any API changes

[ ] Submission is not a "code dump".  (Large changes can be made "in repository" via a branch.  Ask on the developer mailing list for guidance, if required.)

FYI our guidelines for code reviews are at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide.

@josehernandezfintecheandomx tests are failing. Please check.

See my above concerns

The next test case was added as the original issue, disburse a lower amount than the approved amount


feat: add serde feature

This solves https://github.com/smart-leds-rs/smart-leds/issues/16.
(I only serialize/deserialize colors programatically as/to JSON so I dont need to be able to deserialize contants like "WHITE" or "BLACK", only previously programatically serialized json like {"r":255,"g":0,"b":0}).

As is I won't merge this, as serde is much too large of a dependency for an always-on feature with small value for most.
If you just want to serialize/deserialize, you could either choose the option of adding rgb as a dependency with the serde feature enabled to your binary (no changes needed for this repo) or adding the serde feature as an optional feature here (which then also needs to be passed through for the smart-leds crate). I'm not sure about adding the latter option if the former works just as well and is just a bit more obtuse.

That's exactly what I did for now, adding the rgb crate as a dependency with the serde feature enabled, and it works. I think it looks ugly though, because my crate does not really use rgb.
Let me know if you change your mind about "the latter option", I'll be glad to update my PR then.

Finally decided to update the PR anyway.

I think this is fine so long as it's opt-in.

Thank you, looks good to me


Editor alpha - plugins store filter not working

Hi,
In the alpha4, opening the plugins store, clicking on Installed does nothing, it has the same effect as All, shows all plugins.
Imi

Fixed


Request: More settings

Ideas for settings:
Number of Note Slots on screen
Colour options (for stuff like module #s and Usernames)
whatever else may be interesting or helpful to be able to change

damn it formatting

I'd much rather change the number of note slots to be something that everyone is happy with then adding a setting for it. Unless there is a real need for having something be configurable, I'm not going to add a setting.
Although it's not documented, you can already set the color of the module IDs. I don't know what you mean by username colors, those are based on what someone's color is in chat.
You'll need to be more specific, otherwise I can never close this issue.

You basically answered the request verbatim. other than those 2 (and your reason is good to not add them) I feel a good setting would be to be able to turn off the leaderboard/points system.

Like I said, if you can find streamers who would want to have that I'd be willing to add it but there's no point in me adding something that people aren't going to use. Especially because there is already a ton of settings already and I feel like we could probably remove some of them.

Closing due to lack of information.


merge from arlac77/npm-package-template-esm-only

.travis.yml

chore(travis): (jobs.include)
chore(travis): add 12.13.0 remove 12.12.0 (jobs.include.node_js)
chore(travis): (script)

Coverage remained the same at 90.11% when pulling 9ba61c86e26c971650bed6bfe32d100a69d81217 on npm-template-sync/1 into e6b878161da0f1aa67cdc0964e6bc067ddd47f78 on master.

Codecov Report

Merging #495 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #495   +/-   ##
=======================================
  Coverage   89.02%   89.02%
=======================================
  Files           2        2
  Lines         328      328
  Branches       26       26
=======================================
  Hits          292      292
  Misses         36       36

Continue to review full report at Codecov.

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

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Kms key support

Adding support to set the KMS key, as discussed in #36 running make readme appears to remove all the inputs/outputs from the readme.

@stefansedich  thanks.
What OS are you using (Mac, Windows, Linux)?
We need to rebuild README correctly (we can't merge it like this).
Try:
make init
make readme/deps
make readme

and if any errors, please port it here.

@aknysh am running on linux, running those commands results in the readme as part of this PR, just removes the inputs/outputs.

@stefansedich  please rebase

@aknysh done I still cannot generate docs without everything being removed.


[BUG] Training Fails - GK700RBii

Checking that we have all of the required audio files...
Found v3_0_0.wav, version 3.0.0
Delays:
Blip  1: 67
Blip  2: 67
After aplying safety factor of 4, the final delay is 63
Plotting the latency for manual inspection...
Set latency to recommended 63
V3 checks...
Replicate ESR is 2.01217866.
Validation replicates have a self-ESR of over 0.01. Your gear doesn't sound like itself when played twice!

output.zip
Possible causes:

Your signal chain is too noisy.
There's a time-based effect (chorus, delay, reverb) turned on.
Some knob got moved while reamping.
You started reamping before the amp had time to warm up fully.
Failed checks!

Exiting core training...
No model returned; skip exporting!

Looks like your output.wav is slightly longer than v3 - or at least, the one in your zip file is.  I tried shortening it up and it seems to be working (ESR 0.001):
https://drive.google.com/file/d/1WE-mPt1XHalcksKQBYznNkJ5s47j8jFn/view?usp=sharing

@vossenv can you send me the edited WAV file? I made double sure they are the same length and still get the ESR error.

@jtemple967  sure:
https://drive.google.com/file/d/1J0Y9XjLuH6fN7Y3jqL33Hg5_HlaxjF90/view?usp=drive_link

@vossenv okay, then with your help I'm making progress. I figured out how I was exporting a longer file in my DAW and through the export process.  I can't quite get my file length to match yours exactly but I have gotten the ESR down to just over .01. It still fails but the ESR is much better. I will continue tweaking the exported file.

Okay, back at it trying to get this to train. So after seeing most people using Reaper I downloaded Reaper and did the capture/export with that tool. The sizes of the files match exactly.  However...the ESR is still over 2.0. I'm just scratching my head here trying to figure out what I'm doing wrong.
@vossenv would you mind sharing what settings you use when you Render your WAV files in Reaper?

Success!!!!
My initial setup was using both Ardour and Reaper on Ubuntu with my Focusrite 2i2 without any Focusrite drivers installed. I don't think there's an official Linux driver for the 2i2.  Part of my problem may have been that if I turned up the input from the amp past a certain level I was getting a low frequency feedback. That might have been the way Ubuntu routed audio to/from the interface.
Here's what I did to resolve the issues (some of these may or may not have had an impact):

I decided to boot into the Windows partition of my PC to run the capture
I downloaded/installed Reaper
I installed the drivers for my Focusrite 2i2. As soon as I did that it indicated there was an update so I installed the updated firmware.  Prior to the update (both on Ubuntu and Windows) I was getting some white noise on the input and after the update that stopped. Additionally, the low frequency feedback also stopped on Windows so I'm going to attribute that to using the official Focusrite driver.
I recorded the output of my amp with Reaper on Windows and exported the WAV file, being sure to not add the trailing "tail". I exported with 48kHz/24 bit/mono.
Somehow I was using the outdated training colab page. Not sure how I did that, may have been saved in my browsing history.  Here's the link that I first landed on: https://colab.research.google.com/github/sdatkinson/neural-amp-modeler/blob/bd1d8a5/bin/train/easy_colab.ipynb#scrollTo=xOzo3NxqjZjl. When I switched over to Windows I had to redownload the V3 wav and saw that the colab page that I got from the NAM documentation page was different.
The ESR was .0027124 and training completed!


pip build fails on mac

While running pip install pqkmeans:
[ 28%] Building CXX object CMakeFiles/_pqkmeans.dir/src/clustering/bkmeans.cpp.o
    In file included from /private/var/folders/pz/ckv4fct52c7_826fg4x48vgm0000gn/T/pip-install-khtx799x/pqkmeans/src/_pqkmeans.cpp:3:
    In file included from /private/var/folders/pz/ckv4fct52c7_826fg4x48vgm0000gn/T/pip-install-khtx799x/pqkmeans/src/encoder/encoder_sample.h:6:
    In file included from /private/var/folders/pz/ckv4fct52c7_826fg4x48vgm0000gn/T/pip-install-khtx799x/pqkmeans/ext/pybind/include/pybind11/stl.h:34:
    /Library/Developer/CommandLineTools/usr/bin/../include/c++/v1/experimental/optional:18:3: warning: "<experimental/optional> has been removed. Use <optional> instead." [-W#warnings]
    # warning "<experimental/optional> has been removed. Use <optional> instead."
      ^
    In file included from /private/var/folders/pz/ckv4fct52c7_826fg4x48vgm0000gn/T/pip-install-khtx799x/pqkmeans/src/_pqkmeans.cpp:3:
    In file included from /private/var/folders/pz/ckv4fct52c7_826fg4x48vgm0000gn/T/pip-install-khtx799x/pqkmeans/src/encoder/encoder_sample.h:6:
    /private/var/folders/pz/ckv4fct52c7_826fg4x48vgm0000gn/T/pip-install-khtx799x/pqkmeans/ext/pybind/include/pybind11/stl.h:283:46: error: no member named 'experimental' in namespace 'std'
    template<typename T> struct type_caster<std::experimental::optional<T>>
                                            ~~~~~^
    /private/var/folders/pz/ckv4fct52c7_826fg4x48vgm0000gn/T/pip-install-khtx799x/pqkmeans/ext/pybind/include/pybind11/stl.h:283:69: error: 'T' does not refer to a value
    template<typename T> struct type_caster<std::experimental::optional<T>>
                                                                        ^
    /private/var/folders/pz/ckv4fct52c7_826fg4x48vgm0000gn/T/pip-install-khtx799x/pqkmeans/ext/pybind/include/pybind11/stl.h:283:19: note: declared here
    template<typename T> struct type_caster<std::experimental::optional<T>>
                      ^
    /private/var/folders/pz/ckv4fct52c7_826fg4x48vgm0000gn/T/pip-install-khtx799x/pqkmeans/ext/pybind/include/pybind11/stl.h:286:36: error: no member named 'experimental' in namespace 'std'
    template<> struct type_caster<std::experimental::nullopt_t>
                                  ~~~~~^
    1 warning and 3 errors generated.
    make[2]: *** [CMakeFiles/_pqkmeans.dir/src/_pqkmeans.cpp.o] Error 1
    make[2]: *** Waiting for unfinished jobs....
    make[1]: *** [CMakeFiles/_pqkmeans.dir/all] Error 2
    make: *** [all] Error 2

My enviroment details is:
ProductName:	Mac OS X
ProductVersion:	10.15
BuildVersion:	19A583

Darwin MacBook-Pro.local 19.0.0 Darwin Kernel Version 19.0.0: Wed Sep 25 20:18:50 PDT 2019; root:xnu-6153.11.26~2/RELEASE_X86_64 x86_64

@Zardinality We have updated this library to be available on latest Mac OS (see #33 for more details).
Could you try pip install pqkmeans again?


Input and Form Item - Linking Form Item to Input for Accessibility

Info
Right now we have an arialabel property for our form components. This is because currently our Form Item field isn't actually read by a screen reader, so we need some way to announce a form component and what it's intended for. This is not the recommended way of having accessibility for form elements. The W3C recommends having a label element (Form Item) with a for property that is the id property of the associated form element. The W3C only recommends using an aria-label property if visual users can ascertain the intent of the form element through context of where it's located on the site, and thus don't need a label.
If you look at Angular Material, they have an interesting way of dealing with this issue. They use a label element with a for attribute, targeting an auto generated id on the associated Input element.
Right now we don't even use a label element, we have a div with a class of label that contains the text.
This ticket applies to all form "input" elements i.e. input, radio, dropdown, checkbox, textarea
Acceptance Criteria:

Investigate if we can:

Use a label element instead of a div for our Form Item component
Generate a random id for our Input component
Attach that randomly generated id to the for property for the above label element

:tada: This issue has been resolved in version 1.17.0-alpha.71 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 3.0.0-alpha.4 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 4.17.0-alpha.32 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


add timezone to max

fix timezone part 2

can you first test this with private deployment to verifiy it works?


Docker image for Socket.IO + Server for DevTool

Hey,
First, thanks a lot for your package, really helpful for my Capacitor project !
To ease my workflow with React Query, I made a little Docker image that start a Socket.io server + a Vite server that displays the devtool !
I needed this because my back is not really able to deliver React, and I don't want it too, this allows to access your devtools with just a docker container running !
Maybe we can add this. to your doc to help peoples start with this project ?
Here is the link: https://hub.docker.com/repository/docker/navalex/rq_devtool/general

I make a PR right now to change this ;)
And to answer your question about the specific needs for Capacitor, actually this repo seems to do exactly what someone would need to use a react-query devtool, so I guess this project is enough for this issue :)


Doesn't fully cover how to handle Credit Card Payments

Here's the scenario that most customers will face:

Invoice is generated.
Payment is taken against a credit card.
Funds are not deposited immediately, instead they typically are stored in "undeposited funds"
Credit Card Processing company completes the transaction by depositing funds into your bank account in a lump sum.
Visa and MasterCard do this by doing a lump sum payment of the total of all transactions for period. Amex does this as lump sum payment of total of all transactions less their fees.
Customer needs to link the undeposited funds to the actual lump sum and do an entry from undeposited funds to the bank account (and in the case of Amex their CC cost account on a/p).

I can find no documentation anywhere on how this should be done and it isn't clear from this documentation if this is how or there is another process. If this is how, then it needs to be made far more clear. If this isn't how, then a new item in this section needs to be created that fully describes this process of recording CC payments without funds and then receiving funds and reconciling.  Hopefully this is just incomplete documentation and not a major missing function of the product.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 1b825e6d-f005-b0a4-c2a4-57bf9ea3a483
Version Independent ID: 1607ed34-6562-bbf4-dd4b-d76533a6f99a
Content: Use Automatic Application to Reconcile Payments - Business Central
Content Source: business-central/receivables-how-reconcile-payments-auto-application.md
Service: dynamics365-business-central
GitHub Login: @SorenGP
Microsoft Alias: sgroespe

Hi - Thank you for the feedback
In Help for payment processing, we deliberately do not specifically describe a credit-card process or that of any other payment means. Instead, we refer generically to "payments" and to the resulting bank transactions. This because payment means vary geographically and change over time, but also because the nature of Business Central is to be an application platform, not a customer solution.
Accordingly, the standard version of  Business Central does not have dedicated functionality for the credit card scenario that you describe, however the scenario is most likely supported with a comparable method using the standard functionality. See for example this topic about handling lump-sum customer payments: https://docs.microsoft.com/en-us/dynamics365/business-central/receivables-how-reconcile-customer-payments-list-unpaid-sales-documents#to-reconcile-lump-payments. Also, the Deposits feature in the standard US version may be relevant for you. For more information, see https://docs.microsoft.com/en-us/dynamics365/business-central/localfunctionality/unitedstates/how-to-create-deposits.
If you need dedicated credit card functionality in Business Central, try to search for a relevant extension on AppSource: https://appsource.microsoft.com/en-us/marketplace/apps?product=dynamics-365-business-central.
Regards
Soren, Business Central writer


[MRG] Add ask-n-tell example to README

Fixes #425
ping @glouppe

Codecov Report

Merging #459 into master will decrease coverage by 0.03%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master     #459      +/-   ##
==========================================
- Coverage   86.46%   86.42%   -0.04%
==========================================
  Files          22       22
  Lines        1581     1584       +3
==========================================
+ Hits         1367     1369       +2
- Misses        214      215       +1

Impacted Files
Coverage Δ

skopt/utils.py
97.84% <66.66%> (-0.69%)
:arrow_down:

Continue to review full report at Codecov.

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

lgtm


Bug when creating subusers

Pre-release 0.6.0-pre7: Just a minor graphical bug, but when clicking on a server and go to the Subusers tab, I provide the email address for the user and then want to select their permissions. However, anything I check doesn't seem to actively reflect that change (ie: the box doesn't show that it's checked). If I click "Update Subuser", it actually does show that what I clicked on is checked now.

It seems to be working fine for me, what browser are you using?
If you have time to waste you could also install the latest version from develop and test if the issue still occurs :P

I fixed this and forgot to close it.


Skip dry run on missing resources for keycloak

Without it, the keycloak crds are failed to be synced.

The change isn't tested in the e2e suite, merging.


How to change output video codec?

So by default the demo.py outputting result.mp4 which is x264, crf 25, 256x256 video. But how to change these paramethers? I run a search across all files but didn't find anything. Can anyone help me with that?
Higher res video should look better even with 256x256 pictures, because of motion
Also, crf is too high
Seems like it only saves video here, and encoding done somewhere else
imageio.mimsave(opt.result_video, [img_as_ubyte(frame) for frame in predictions], fps=fps)

@AliaksandrSiarohin why are you ignoring me?

imageio.help('.mp4')


Fix failing 3d-tiles test

Just waiting until the CI passes.

There's failures in release: https://travis-ci.org/AnalyticalGraphicsInc/cesium/builds/210688218

There were two intermittent failures which I can't reproduce locally with the same command that travis runs.
I adjusted the 404-related test to give it a greater timeout and changed the file name, which seems to have helped. I haven't seen the other test about ModelInstanceCollection culling in a while, so maybe it just works now...
The jsHint errors appeared recently and are also fixed.


add ability to override getTokenFromResponse with custom method

It'd be nice if we, developer, had the ability to override this method.
Recently I was playing a little with Aurelia and this plugin and used Nodal to get an API running. Nodal authentication controller returns the access_token like this:
{   "meta": {     "total": 1,     "count": 1,     "offset": 0,     "error": null   },   "data": [     {       "user_id": 1,       "access_token": "360e4cb7d49051d1c0991a299921aca9",       "token_type": "bearer",       "expires_at": "2016-09-07T21:42:43.439Z",     }   ] }
I was trying to get the access token via accessTokenProp: 'data', accessTokenName: 'access_token', accessTokenRoot = true w/o success.
Maybe it is just me and I don't know how to set the config values properly.
Maybe just adding a isArray check

yeah, array might not work atm. i think our approach needs rethinking to make it more flexible in general.

@ricbermo would that work for you  https://github.com/SpoonX/aurelia-authentication/pull/232/commits/26edc37939c8d42a6d750c627b6ec663ff9787f8  ?

@doktordirk yes sir, it'd totally do the job, thank you

done


Angular wizard buttons not placed at bottom of form, regression in 12.0.9

Describe the bug
For clr-wizard in 12.0.9 and 12.0.10, buttons are placed directly below form instead of being locked vertically at the bottom of the form. When navigating between pages with forms of varying vertical height, the buttons move position impacting usability.
How to reproduce

For any clr-wizard create a page with some default buttons.
Observe buttons are not placed in bottom-right corner.

Using browser dev tools, the form content div now uses "modal-body-wrapper" whereas prior to 12.0.9 it was "modal-body". Manually changing the class on this div to "modal-body" resolves the issue. I believe this regression is a side-effect of this commit:
https://github.com/vmware/clarity/pull/6561/commits/541c92707fe845b05bb42eb3f6cc9f6f1bc04e57
Only workaround found was to revert back to 12.0.8.
Expected behavior
Wizard buttons should be placed vertically at the bottom.
Versions
Clarity project:

[ ] Clarity Core
[x] Clarity Angular/UI

Clarity version:

[ ] v3.x
[ ] v4.x
[x] v5.x

Framework:

[x] Angular
[ ] React
[ ] Vue
[ ] Other:

Framework version:
Angular 12.2.15
Device:

Type: PC
OS: Windows 10
Browser: Chrome
Version: 97.0.4692.99

Additional notes

I have the same problem with Clarity version 13.0.0. Adding this CSS rule works for me.
.clr-wizard .modal-body-wrapper {
  height: 100%;
  width: 100%;
}

reproducible on the website


Permission error with organization GitHub App

I created a GitHub App in our organization but got the following error:
2021-02-26T04:10:32.399Z	ERROR	controllers.ApplicationSyncStatus	unable to add a comment	{"application": "...", "comment": ..., "error": "query error from GitHub API: Resource not accessible by integration"}

It seems this query caused the issue:
https://github.com/int128/argocd-commenter/blob/1ed4c397e9ac99a5eb7e2bde9302f4dff56c4fde/pkg/github/comment.go#L17-L30

I tried query for a public repository (https://github.com/int128/sandbox/commit/33ba4d602be92a2986517e6df962394c0e6dd876) and got a result:
query {
  repository(owner: "int128", name: "sandbox") {
    object(oid: "33ba4d602be92a2986517e6df962394c0e6dd876") {
      ... on Commit {
        associatedPullRequests(first: 1) {
          nodes {
            id
            number
          }
        }
      }
    }
  }
}

{"data":{"repository":{"object":{"associatedPullRequests":{"nodes":[{"id":"MDExOlB1bGxSZXF1ZXN0MzA5NTQzNzUx","number":4}]}}}}}

I tried query for a private repository and got a result:
query {
  repository(owner: "lab-hidetake-org", name: "sandbox-private") {
    object(oid: "6e17eb224fa532a0b1a7b259a11483d1543112f6") {
      ... on Commit {
        associatedPullRequests(first: 1) {
          nodes {
            id
            number
          }
        }
      }
    }
  }
}

{"data":{"repository":{"object":null}},"errors":[{"type":"FORBIDDEN","path":["repository","object"],"extensions":{"saml_failure":false},"locations":[{"line":4,"column":5}],"message":"Resource not accessible by integration"}]}

I could fix the issue by the following steps:
(1) Change the permissions
Add "Contents: Read-only" to the permission of GitHub App and save it:

(2) Accept the change
Open installed GitHub App and accept the new permissions:


CI: pytest timeout

What does this PR do?
add particular timeout fr some hanging tests...
noticed in https://github.com/PyTorchLightning/metrics/pull/798#issuecomment-1021970991
Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure to update the docs?
[x] Did you write any new necessary tests?

PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

@justusschock I am already investigating speeding up testing in https://github.com/PyTorchLightning/metrics/pull/820 because I also think it just takes too long time now.

Although we should rather think about speeding up certain tests. Our pipeline really takes long these days...

I agree this is just surfacing the too slow and making them as fail instead of hanging all test suit and then you would not know which it is as log show just the one right before it...

I am already investigating speeding up testing in #820 because I also think it just takes too long time now.

Super COOL! :rabbit:


Profiling Node.js

https://nodejs.org/en/docs/guides/simple-profiling/

http://www.willvillanueva.com/the-node-js-profiling-guide-that-hasnt-existed-profiling-node-js-applications-part-1/

https://nodesource.com/blog/profiling-node-js-applications/

https://github.com/jam3/devtool

https://github.com/GoogleChrome/devtools-docs/issues/53

https://github.com/petkaantonov/bluebird/wiki/Optimization-killers

https://github.com/thlorenz/v8-perf/blob/master/compiler.md

Heap Size

Heap: Current heap size (MB)
RSS: Resident set size (MB)
V8 Full GC: Heap size sampled immediately after a full garbage collection (MB)

V8 optimizer
Note the signs ~ and * before function names:

means that the function was optimized by V8.

~ means that the function was not optimized.

http://stackoverflow.com/questions/1911015/how-do-i-debug-node-js-applications/16512303#16512303


feat: less verbose CI output by default

Closes #905
Moved current implementation to verbose reporter. Now reporter only prints test files

With failed test:


Full Deploys for multi device

If i do a full deploys or restart Node Red, i find many devices in red not connected (Listen and Power State write).
If a deploys a single ewelink node, i do not have any problem.

Duplicate


Baggage creation breaks external APIs

How do you use Sentry?
Self-hosted/on-premise
Version
1.10.1
Steps to Reproduce
I just spent literal 5 hours debugging a very weird issue in our production system. A specific API call to an external system started to return an error code 400.
After a long search, I could pinpoint the change to our recent upgrade of sentry-sdk, and specifically to this pull request: https://github.com/getsentry/sentry-python/pull/1589
Starting from 1.9.8, Sentry will add a Baggage request to all external HTTP targets we call with requests.get() by default. I'm sure this is really helpful when calling internal systems that are also sentry-instrumented, but in our case we happen to call an external API that happens to break when it sees that header.
Our temporary solution is now to set propagate_traces=False, which is sad, because it also breaks propagation to useful targets (such as celery). We'd very much appreciate a more detailed way to configure this.
Expected Result
Sentry is not modifying requests to external APIs by default.
Actual Result
Sentry is adding a Baggage header to requests to external APIs even though we never asked for it.

hey @raphaelm apologies for that, we'll add a trace_propagation_targets config to enable control over which origins we add outgoing headers to.
It's interesting that baggage broke your setup because we also always added sentry-trace before by default.

we'll add a trace_propagation_targets config to enable control over which origins we add outgoing headers to.

That'd be perfect!

It's interesting that baggage broke your setup because we also always added sentry-trace before by default.

Yeah… The target origin in our case is a German government entity, not sure what they're doing over there.

Related to this Issue https://github.com/getsentry/sentry-python/issues/1785

I will close this issue in favor of the Issue that adds the solution to this problem to the SDK:
https://github.com/getsentry/sentry-python/issues/1785
So @raphaelm please check the linked Issue for progress! Thanks!


Odata Schema

Hi Chris
How do I pull the schema to get the collections names and data types
regards
K

Hiya, OData supports getting the schema either as JSON or XML at this URL: http://localhost:8000/odata/$metadata. You must provide an Accept HTTP header to specify the format, either application/json or application/xml. For example this curl request:
curl -H 'Accept: application/xml' 'http://localhost:8000/odata/$metadata'

Will return the CSDL metadata in XML format https://docs.oasis-open.org/odata/odata-csdl-xml/v4.01/odata-csdl-xml-v4.01.html

Thanks


Ignore dirty commits in the submodule

The main Dotbot readme recommends ignoring 'dirty' commits in the submodule as part of the installation, this updates the script to match.

(I realise this hasn't been updated for years! But I just used it to install dotbot and still seems very useful!)

Thanks!


Repo sync for protected CLA branch

The pull request is created from master637728553226464667 to master to fix git push error for protected CLA branch

@shsagir : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Should be made for Mac too

Thanks for providing this library. I am reviewing Desk API client libraries for potential inclusion in an internal project. I was not able to complete my review of this library because:
While reviewing this one I note that it is not designed to work with Mac. When I look through the code the only class that seems like it is problematic is this one: DSAPINetworkIndicatorController.h
I think you should trying to not depend on UIKit and instead of Foundation only if possible like you did in the rest of the project. From my read of this class it seems it would be straight forward to support both OSX and iOS using #if def for this one file.
Is there a reason that I am not seeing that this project only supports iOS?

Hi @yepher, the original intent was to only support iOS. You're absolutely right though, this could be easily made to run on macOS also. This is the first request we've had to support macOS. I'll pass it along to our Product Manager. I can't make any guesses when we would work on it.
If you feel adventurous, you can clone it down and if you get it to work in a macOS project, issue a PR against the develop branch and we'll happily review/merge your changes.

@nartiles There you go. There is a first pull request. It should be 100% compatible and minimal risk for existing iOS projects.
Next step is to create a new OSx target but I won't have time for that today.


DEFAULT_CONFIG is required

Svgo.config.js must be placed in the config directory regardless of whether -c config is configured
let svgoConfig = await loadConfig(DEFAULT_CONFIG);

Looks like you're right, @aegorx. I will try to fix this as soon as I can! Thanks for reporting.

@aegorx please install the latest version from NPM (1.0.4) or download the latest version from Github.
Again, thanks for finding this :)


Drop utils-space in next version

It's not useful or flexible enough.

Hi @necolas, I'm curious and I would like to know more about the lack of utility/flexibility of this kind of utilities. Can you explain me or recommend me an article about this? Have you recommendations for managing spaces?
Thank you in advance (I wonder for a long time).

ping @simonsmith can you give me an opinion ?

Hey @florianbouvot,
Generally it was found that utils-space was too generic to be its own package, as each site/application often has quite different requirements and couldn't be met by something like:
.u-marginBn { margin-bottom: 0 !important; }
.u-marginBs { margin-bottom: 5px !important; }
.u-marginBm { margin-bottom: 10px !important; }
.u-marginBl { margin-bottom: 20px !important; }

On top of that it's generally easier to have a containing component that can look after the spacing of its children, like so:

One main benefit of this approach is easily controlling the spacing across breakpoints, which isn't easily done with utility classes. You can read more about that in this issue.
That being said, if you need to use spacing classes, then by all means implement them yourself as they can be useful. I wrote postcss-generate-preset to help with mass production of these utils.

@simonsmith thank you very much for your answer and your reactivity ;)


Remove use of deprecated Signal.providing_args

The attribute was deprecated in upstream commit
https://github.com/django/django/commit/769cee525222bb155735aba31d6174d73c271f3c.

Thanks!


Adding some try catch to various places that cause hard errors, espec…

Adding some try catch to various places that cause hard errors, especially for empty buffers; fixing issue with parseInt and listen port on command line not being set; fixed issue with namespaces from config not being correctly passed to the epp xml building code; nameservers are technically optional for createDomain; more try catch is necessary along with recovery for timeouts

The change to the way namespaces are passed is something I would like to have verified with other registries as it is currently breaking the tests. The current implementation worked fine when I used it in the past and no other users have reported issues with it (yet).
If indeed it is valid, then the tests should also be updated accordingly.

The problem I noticed is that the previous code was not honoring the
settings in the config.json, it was pretty much ignoring them or partially
ignoring in the case of domain commands. I'll make the example config and
tests match what the code had before outputting before, that way there is
no change for the example and the tests, and then they should pass as
before. Another pull request coming today.
On Sat, Nov 12, 2016 at 11:52 AM, William Travis Holton <
<EMAIL_ADDRESS>wrote:

The change to the way namespaces are passed is something I would like to
have verified with other registries as it is currently breaking the tests.
The current implementation worked fine when I used it in the past and no
other users have reported issues with it (yet).
If indeed it is valid, then the tests should also be updated accordingly.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/heytrav/nodepp/pull/10#issuecomment-260137414, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADKssMNO2bb11DEEbN-KzyAX2BtwWeo-ks5q9fzsgaJpZM4Kvrij
.

--
Christopher Schaub
http://chris.schaub.com


Documentation not updated

The documentation should be adapted to the current state of development, e.g. for version 7.2.2 according to the documentation ES256KSigner: The supported encodings for the private key are hex, base64, base64url, and base58btc.
However, according to code only Uint8Array is allowed: function ES256KSigner(privateKey: Uint8Array, recoverable = false).
Note: Before version 6.0.0 it was indeed possible to pass the key in various binary-to-text encodings.

Thank you for reporting this!
Are you planning to open a PR with the fix?


Issue receiving many transactions in parallel

When starting the wallet and there are many transactions to be recieved from the same mint, wallet likely get bricked on "duplicate b_B" errors returned by the mint. Issue is cause by the fact, that multiple transactions acess and overwrite recovery index counter that is now increased before making a call to the mint in order to allow recovery in case the app will quit before receiving the response from the mint.
Solution is to implement pessimistic lock on recovery index counters specific to each transactionId and released only after transaction completed storing of new proofs.

Fix released in v0.1.5-beta-37 and in new native release v0.1.6-beta.


stuck in graph validating - TubularCellLupusNephritis

Dataset/group this task is for:

project uuid - 97fca723-d9e9-4263-9f67-335416086f47
link to ingest UI

Relevant People

Wrangler Ida
dev

Description of the problem with the dataset
This seems to be a size issue when running the graph validation - similar to #1050
The project has 38582 entities (mostly cell supensions) and 173443 links
I've tried to do graph validation locally but the project on't hydrate from the xls
ingest-graph-validator hydrate xls /Users/idazucchi/Documents/Wrangling/TubularCellLupusNephritis/TubularCellLupusNephritis_ontologies.xlsx
23-03-03 11:53:04 [ingest_graph_validator.ingest_graph_validator] - INFO: starting neo4j docker instance
23-03-03 11:53:05 [ingest_graph_validator.ingest_graph_validator] - INFO: attached to backend container [neo4j-server]
23-03-03 11:53:05 [ingest_graph_validator.ingest_graph_validator] - INFO: Connecting to neo4j...
23-03-03 11:53:05 [ingest_graph_validator.hydrators.hydrator] - INFO: started xls hydrator for file [/Users/idazucchi/Documents/Wrangling/TubularCellLupusNephritis/TubularCellLupusNephritis_ontologies.xlsx]
23-03-03 11:53:55 [ingest_graph_validator.utils] - INFO: [import_spreadsheet] took [50343] ms
23-03-03 11:53:57 [ingest_graph_validator.hydrators.hydrator] - INFO: imported 38582 nodes
23-03-03 11:54:07 [ingest_graph_validator.hydrators.hydrator] - INFO: imported 231257 edges

it has been hanging there for 3h+
Wei solved the same issue with HumanMelanocyteDevelopment by running an older version of the ingest graph validation tool but this time it failed to hydrate
(ingest-graph-validator) wteh@C02DM256ML85 Ingest-graph-validator % ingest-graph-validator hydrate xls TubularCellLupusNephritis_ontologies.xlsx
23-03-03 10:27:43 [ingest_graph_validator.ingest_graph_validator] - INFO: starting neo4j docker instance
23-03-03 10:27:43 [ingest_graph_validator.ingest_graph_validator] - INFO: attached to backend container [neo4j-server]
23-03-03 10:27:47 [ingest_graph_validator.hydrators.hydrator] - INFO: started xls hydrator for file [TubularCellLupusNephritis_ontologies.xlsx]
23-03-03 10:28:37 [ingest_graph_validator.utils] - INFO: [import_spreadsheet] took [49893] ms
Traceback (most recent call last):
  File "/opt/anaconda3/envs/ingest-graph-validator/bin/ingest-graph-validator", line 8, in <module>
    sys.exit(entry_point())
  File "/opt/anaconda3/envs/ingest-graph-validator/lib/python3.8/site-packages/click/core.py", line 764, in __call__
    return self.main(*args, **kwargs)
  File "/opt/anaconda3/envs/ingest-graph-validator/lib/python3.8/site-packages/click/core.py", line 717, in main
    rv = self.invoke(ctx)
  File "/opt/anaconda3/envs/ingest-graph-validator/lib/python3.8/site-packages/click/core.py", line 1137, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/opt/anaconda3/envs/ingest-graph-validator/lib/python3.8/site-packages/click/core.py", line 1137, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/opt/anaconda3/envs/ingest-graph-validator/lib/python3.8/site-packages/click/core.py", line 956, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/opt/anaconda3/envs/ingest-graph-validator/lib/python3.8/site-packages/click/core.py", line 555, in invoke
    return callback(*args, **kwargs)
  File "/opt/anaconda3/envs/ingest-graph-validator/lib/python3.8/site-packages/click/decorators.py", line 17, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/opt/anaconda3/envs/ingest-graph-validator/lib/python3.8/site-packages/ingest_graph_validator/hydrators/hydrator_definitions.py", line 25, in xls
    XlsHydrator(ctx.obj.graph, xls_filename).hydrate()
  File "/opt/anaconda3/envs/ingest-graph-validator/lib/python3.8/site-packages/ingest_graph_validator/hydrators/xls_hydrator.py", line 31, in __init__
    self._nodes = self.get_nodes()
  File "/opt/anaconda3/envs/ingest-graph-validator/lib/python3.8/site-packages/ingest_graph_validator/hydrators/xls_hydrator.py", line 49, in get_nodes
    for node_type in self._entity_map.entities_dict_by_type.keys():
AttributeError: 'NoneType' object has no attribute 'entities_dict_by_type'

Acceptance criteria for the task:

[ ] TubularCellLupusNephritis graph valid in ingest

Found and fixed the issue and adding graph_test_set tests here
(ingest-graph-validator) wteh@C02DM256ML85 ingest-graph-validator % ingest-graph-validator action test graph_test_set 23-03-06 12:50:30 [ingest_graph_validator.ingest_graph_validator] - INFO: starting neo4j docker instance 23-03-06 12:50:30 [ingest_graph_validator.ingest_graph_validator] - INFO: attached to backend container [neo4j-server] 23-03-06 12:50:30 [ingest_graph_validator.actions.test_action] - INFO: loading tests 23-03-06 12:50:30 [ingest_graph_validator.actions.test_action] - INFO: loaded [19] test queries 23-03-06 12:50:30 [ingest_graph_validator.actions.test_action] - INFO: running tests 23-03-06 12:50:32 [ingest_graph_validator.actions.test_action] - ERROR: test [contains_cell_barcode_info.adoc] failed: non-empty result. 23-03-06 12:50:32 [ingest_graph_validator.actions.test_action] - ERROR: result: [{'p': (_19380:library_preparation_protocol:protocol {cdna_library_amplification_method.ontology: 'OBI:0000415', cdna_library_amplification_method.ontology_label: 'polymerase chain reaction', cdna_library_amplification_method.text: 'PCR', describedBy: 'https://schema.humancellatlas.org/type/protocol/sequencing/6.3.1/library_preparation_protocol', end_bias: 'full length', id: 'library_Fluidigm', input_nucleic_acid_molecule.ontology: 'OBI:0000869', input_nucleic_acid_molecule.ontology_label: 'polyA RNA', input_nucleic_acid_molecule.text: 'polyA RNA', library_construction_method.ontology: 'EFO:0010058', library_construction_method.ontology_label: 'Fluidigm C1-based library preparation', library_construction_method.text: 'Fluidigm C1-based library preparation', nucleic_acid_source: 'single cell', primer: 'poly-dT', protocol_core.protocol_description: 'Single-cell suspensions at a concentration of 200,000\u2009cells\u2009ml\u20131 and no less than 2,000 cells were loaded into a medium (10\u201317\u2009\u00b5m) diameter, C1 HT, 800-well, integrated microfluidic chip (IFC) (Fluidigm) and processed according to the Fluidigm C1 HT protocol revision A, using the recommended standard mRNA-seq reagents and program. The IFC is divided into two 400-well sections, allowing for loading of skin and kidney samples matched by individual on the same chip. Following the manufacturer\u2019s recommendation, 7\u2009\u00b5l of 10% phosphate-buffered saline Tween was added to the valve fluid to reduce surface tension. The occupation of single-cell capture sites was verified by using a Zeiss Axiovert 200 inverted microscope averaging 500 single-cell captures per 800-well chip, and wells containing either debris or multiple captures were excluded from downstream analysis. The captured cells were lysed, poly(A) mRNA was reverse transcribed and complementary DNA pre-amplified using the SMARTer Ultra Low RNA kit (Clontech) in the Fluidigm C1 Single-Cell Auto Prep system. To monitor cDNA library conversion, a cocktail of synthetic RNA spikes, nos. 1, 4 and 7 of the Ambion ArrayControl RNA Spikes (ThermoFisher), was prepared as described in the Fluidigm C1 protocol and added to the lysis reaction (Mix A) for each experiment. After the initial rounds of PCR on the IFC, products were harvested, exonuclease treated and further amplified according to the Fluidigm protocol.\n\nPreamplified cDNA libraries were tagmented and barcoded using the Nextera XT Library Preparation Kit (Illumina) with indexing according to the Fluidigm C1 HT protocol revision A. An enrichment primer to select for the 3\u2032-ends was added during this step. PCR products originating from up to 800 cells per chip were pooled together using the 20 barcodes recommended by Fluidigm.', protocol_core.protocol_id: 'library_Fluidigm', protocol_core.protocol_name: 'Library preparation Fluidigm C1 HT', schema_type: 'protocol', spike_in_kit.manufacturer: 'ThermoFisher', spike_in_kit.retail_name: 'Ambion ArrayControl RNA Spike nos. 1, 4 and 7', strand: 'unstranded'}), '"Missing information for cell barcode"': 'Missing information for cell barcode', 'labels(p)': ['protocol', 'library_preparation_protocol']}] 23-03-06 12:50:35 [ingest_graph_validator.actions.test_action] - ERROR: test [contains_umi_barcode_info.adoc] failed: non-empty result. 23-03-06 12:50:35 [ingest_graph_validator.actions.test_action] - ERROR: result: [{'p': (_19380:library_preparation_protocol:protocol {cdna_library_amplification_method.ontology: 'OBI:0000415', cdna_library_amplification_method.ontology_label: 'polymerase chain reaction', cdna_library_amplification_method.text: 'PCR', describedBy: 'https://schema.humancellatlas.org/type/protocol/sequencing/6.3.1/library_preparation_protocol', end_bias: 'full length', id: 'library_Fluidigm', input_nucleic_acid_molecule.ontology: 'OBI:0000869', input_nucleic_acid_molecule.ontology_label: 'polyA RNA', input_nucleic_acid_molecule.text: 'polyA RNA', library_construction_method.ontology: 'EFO:0010058', library_construction_method.ontology_label: 'Fluidigm C1-based library preparation', library_construction_method.text: 'Fluidigm C1-based library preparation', nucleic_acid_source: 'single cell', primer: 'poly-dT', protocol_core.protocol_description: 'Single-cell suspensions at a concentration of 200,000\u2009cells\u2009ml\u20131 and no less than 2,000 cells were loaded into a medium (10\u201317\u2009\u00b5m) diameter, C1 HT, 800-well, integrated microfluidic chip (IFC) (Fluidigm) and processed according to the Fluidigm C1 HT protocol revision A, using the recommended standard mRNA-seq reagents and program. The IFC is divided into two 400-well sections, allowing for loading of skin and kidney samples matched by individual on the same chip. Following the manufacturer\u2019s recommendation, 7\u2009\u00b5l of 10% phosphate-buffered saline Tween was added to the valve fluid to reduce surface tension. The occupation of single-cell capture sites was verified by using a Zeiss Axiovert 200 inverted microscope averaging 500 single-cell captures per 800-well chip, and wells containing either debris or multiple captures were excluded from downstream analysis. The captured cells were lysed, poly(A) mRNA was reverse transcribed and complementary DNA pre-amplified using the SMARTer Ultra Low RNA kit (Clontech) in the Fluidigm C1 Single-Cell Auto Prep system. To monitor cDNA library conversion, a cocktail of synthetic RNA spikes, nos. 1, 4 and 7 of the Ambion ArrayControl RNA Spikes (ThermoFisher), was prepared as described in the Fluidigm C1 protocol and added to the lysis reaction (Mix A) for each experiment. After the initial rounds of PCR on the IFC, products were harvested, exonuclease treated and further amplified according to the Fluidigm protocol.\n\nPreamplified cDNA libraries were tagmented and barcoded using the Nextera XT Library Preparation Kit (Illumina) with indexing according to the Fluidigm C1 HT protocol revision A. An enrichment primer to select for the 3\u2032-ends was added during this step. PCR products originating from up to 800 cells per chip were pooled together using the 20 barcodes recommended by Fluidigm.', protocol_core.protocol_id: 'library_Fluidigm', protocol_core.protocol_name: 'Library preparation Fluidigm C1 HT', schema_type: 'protocol', spike_in_kit.manufacturer: 'ThermoFisher', spike_in_kit.retail_name: 'Ambion ArrayControl RNA Spike nos. 1, 4 and 7', strand: 'unstranded'}), '"Missing information for UMI barcode"': 'Missing information for UMI barcode', 'labels(p)': ['protocol', 'library_preparation_protocol']}] 23-03-06 12:50:35 [ingest_graph_validator.actions.test_action] - INFO: all tests finished

Passed graph validation!
(ingest-graph-validator) wteh@C02DM256ML85 ingest-graph-validator % ingest-graph-validator action test graph_test_set
23-03-06 12:54:42 [ingest_graph_validator.ingest_graph_validator] - INFO: starting neo4j docker instance
23-03-06 12:54:42 [ingest_graph_validator.ingest_graph_validator] - INFO: attached to backend container [neo4j-server]
23-03-06 12:54:42 [ingest_graph_validator.actions.test_action] - INFO: loading tests
23-03-06 12:54:42 [ingest_graph_validator.actions.test_action] - INFO: loaded [15] test queries
23-03-06 12:54:42 [ingest_graph_validator.actions.test_action] - INFO: running tests
23-03-06 12:54:44 [ingest_graph_validator.actions.test_action] - INFO: all tests finished
(ingest-graph-validator) wteh@C02DM256ML85 ingest-graph-validator %

Made a PR which fixes the previous error in ingest-graph-validator - see here


Add tool to apply vector field simulations to graph model

How can you bias the random perturbation to make interesting output?
Requires #62 and #71
Are there perturbations that might work better on straight geometries instead of the graph?

This could maybe be done by supporting graph input with the streamlines tool?


Deploying modules to prod times out with invalid schema failures

[ask me for full logs]
Renamed modules: MM, EX
debug:MM: Waiting for deployment dpl-MM-588lmgdyc2g8nfbg to become ready
...
debug:EX: Generating main module
debug:EX: Tidying go.mod files
...
debug:EX: Compiling
...
info:EX: Module built (14.46s)
info:EX: Deploying module
debug:EX: Uploading 1/1 files
...
debug:EX: Uploaded ___/modules/EX/.ftl/main as e64a93547727f8cc087d89a1a32d80094e243c2acafbb096acf1e1dc444fcd40:main
debug:EX: Unary RPC failed: unknown: invalid module schema: invalid schema: /home/runner/_work/__/___/___/modules/EX/___.go:_____: unknown reference "MM.____", is the type annotated and exported?: /xyz.block.ftl.v1.ControllerService/CreateDeployment
... [more logs]

Error: The action 'Deploy pfi' has timed out after 10 minutes.

Some thoughts:
Did MM get deployed fully?
EX is built before MM get's deployed?
Shouldn't it wait for it's dependency to be active?

@alec-brooks:

We were getting similar invalid schema logs in previous deploys that ended up being successful, but those seemingly didn’t brick those endpoints after being deployed

Looks fixed due to https://github.com/TBD54566975/ftl/pull/2110


Repo contains the DNA?

I don't think the repo should contain the compiled DNA and hApp as it makes the downloads huge. Shouldn't they be release artifacts?

with this in the cargo.toml file ..  i still get quite a heavy download ..  am I doing something wrong?
[dependencies]
hc_zome_profiles = {git = "https://github.com/holochain-open-dev/profiles", package = "hc_zome_profiles"}

@nphias nothing wrong no... Also notice that zippy was talking about when cloning the git repository, as oposed as just using the crate like you are. I don't think there is a way to make it lighter since it basically only depends on the hdk.


CLI desktop shortcut does not appear on Ubuntu Unity

Closed in favor of https://github.com/CCI-Tools/cate-core/issues/398


Feature/floatbar for X11

Here you go. This fixes Issue #4756

@freerdp-bot test

@mmattes very nice! Thank you!


RangeError: Maximum call stack size exceeded

I'm getting a problem in where if I upgrade from 2.1.0 I get a call stack error.
W20161012-15:49:43.074(1)? (STDERR) /home/jlacey/.meteor/packages/meteor-tool/.1.3.5_1.1bjykex++os.linux.x86_64+web.browser+web.cordova/mt-os.linux.x86_64/dev_bundle/server-lib/node_modules/fibers/future.js:280
W20161012-15:49:43.074(1)? (STDERR) 						throw(ex);
W20161012-15:49:43.074(1)? (STDERR) 						      ^
W20161012-15:49:43.074(1)? (STDERR) RangeError: Maximum call stack size exceeded

I've simplified the problem I'm having into these model below.
AS = {};

AS.Object = Astro.Class.create({
	name:  'AS.Object',
});

AS.Object.extend({
	fields: {
    /**
     * @NOTE
     * Commenting out these fields, will stop the call stack error
     */
		attachments: {
			type: [AS.Object],
			optional: true
		},
		author: {
			type: AS.Object,
			optional: true
		},
	},
});

/**
 * @NOTE
 * Commenting the activity class, will stop the call stack error
 */
AS.Activity = Astro.Class.create({
	name: 'AS.Activity',
  /**
   * @NOTE
   * Commenting the collection will stop the call stack error.
   */
	collection: new Meteor.Collection('as.activities'),
	fields: {
		fieldA: {
			type: AS.Object,
		},
	},
});

I've created a test case which can be found here, https://github.com/Centiq/Astro-Call-Stack.
<EMAIL_ADDRESS>updating meteor still continues to throw the error.

2.1.1, 2.1.2, 2.1.3, 2.1.4, All throw the call stack error.
However 2.1.0 works fine.

Thanks!

Ah right, thanks for noticing it. The problem are nested indexes that were introduced in 2.1.1. It goes through the list of fields and for object fields through its fields and so on. It goes into infinite loop where. I have to implement mechanism of detecting loops. I will try to prepare some fix today.

Ahh I see, thank you for taking a look at this.
I've had a look at the code you've described and I've made this change
_.each(fields, (field) => {
	if (field.type.class !== Class) { // <!----- here
		_.extend(
			indexes,
			prefixIndexes(field.type.class.definition.indexes, field.name),
			prefixIndexes(collectNestedIndexes(field.type.class), field.name)
		);
	}
});

I no longer get the call stack error, but I don't know how viable this fix is because I don't know how the whole indexes looping works (I've not had a look at that). I'm assuming because your already checking the fields of that class you don't need to have a look inside it.

In your case such a fix solves problem but it's not universal. It would still be vulnerable. Let's take an example:
A class
  field: B class

B class
  field: A class

The A class has a field of type B which is a class, so your check would not apply here. And when checking fields of the B class we would encounter a field of the A type and we would have a loop. The check I have to make is something like stack. Adding all the classes that were already checked onto the stack. If a given class is already on the stack then do not check that class. I think it should solve a problem.

Fixed and published in 2.1.5


DataPollManager: Update some of the methods/variables names.

No logic/algo change in this PR. Updated some variable/method names (to help with code readability) and some of the comments/documentation.

Codecov Report

Merging #1561 into master will increase coverage by 0.53%.
The diff coverage is 37.5%.

@@            Coverage Diff             @@
##           master    #1561      +/-   ##
==========================================
+ Coverage   69.14%   69.67%   +0.53%
==========================================
  Files         156      156
  Lines       20117    20116       -1
  Branches     2480     2480
==========================================
+ Hits        13910    14016     +106
+ Misses       5304     5202     -102
+ Partials      903      898       -5

Impacted Files
Coverage Δ

src/core/thread/data_poll_manager.hpp
0% <ø> (ø)
:arrow_up:

src/core/thread/mesh_forwarder.cpp
77.71% <100%> (+0.68%)
:arrow_up:

src/core/thread/data_poll_manager.cpp
61.06% <30.76%> (ø)
:arrow_up:

src/core/thread/mle.cpp
81.98% <50%> (+0.42%)
:arrow_up:

examples/platforms/posix/random.c
85% <0%> (-15%)
:arrow_down:

src/core/thread/mle.hpp
93.1% <0%> (-1.73%)
:arrow_down:

src/ncp/ncp_base.cpp
35.13% <0%> (-1.24%)
:arrow_down:

src/core/meshcop/dtls.cpp
88.2% <0%> (-1.03%)
:arrow_down:

src/core/net/udp6.cpp
82.52% <0%> (-0.98%)
:arrow_down:

src/core/thread/network_data_leader_ftd.cpp
68.45% <0%> (+0.23%)
:arrow_up:

... and 16 more

Continue to review full report at Codecov.

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


bump ES version support

Seems to work fine in ES 5.2.0, should continue to work throughout ES 5.x.

I do this on my server. Works fine!

Solve #2


Implementing MergeSort Algorithm in C++

Do you want this Algorithm to be implementable on arrays or linked lists?
A much better algorithm to implement arrays is Quick Sort and for linked lists it is Merge Sort.

I thought that all the sorting (at least basic) algorithms  are going to be implemented in the future. So, to cope with existing implantation, I planned to do with arrays. I can also do with linked lists. Which one should I do? arrays or linked lists or both?

Hey,
Thanks for being interesting in contributing to codezilla!
Go for both of them, it'll be great!

Thank you for the opportunity to contribute.
Issue #31 is actually implementing merge sort with arrays and linked lists. I am almost ready with linked list version. However, issue #31 is closed. Should I open new one or just move on?


arch migrators

closes #8
the migrator seems to think this depends on the fenics recipe, presumably because fenics-ufl used to be published from there, so applying this one by hand

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


add optional cron job for certificate renewal

This creates a cronjob that sets up a daily job to re-run letsencrypt-auto with same arguments plus the --keep-until-expiring argument so that certificate will be renewed when it is close to expiring. I tested this on a server with two websites on it and it scheduled two jobs which ran OK but didn't renew certificate because they aren't close to expiring. When I ran the command manually with the renew-by-default argument it did renew the certificate (but early).

Hi @hdeadman! Thanks for the PR! I've left some comments inline. Also, can you please add a test for this feature?

OK, thanks for the feedback. I made the updates you requested.

Thanks!


haxe game of life

What does the program do ?
It's a game of life implemented for the console

In what programming language it is written?
Haxe

Thank you for participating  in hacktoberfest ❣ if you liked working on this repo you can share it with friends and star this project for helping others


az deployment sub create add --mode flag (complete\Incremental)

There is any option to create RG using  az deployment sub create (subscription scope)
with mode parameters?
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 6c21eb60-cfe9-0ef7-8d93-71a8af273a04
Version Independent ID: a97f1562-8602-f55e-df4e-7523f437edcf
Content: az deployment sub
Content Source: src/azure-cli/azure/cli/command_modules/resource/_help.py
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

add to S172

@motim6026 May I ask what do you mean by the mode parameters?

@motim6026 Hi, You can declare the deployment mode of resources in the ARM template, for example:
"resources": [
  {
      "type": "Microsoft.Resources/deployments",
      "apiVersion": "2017-05-10",
      "name": "linkedTemplate",
      "properties": {
          "mode": "Incremental",
          <nested-template-or-external-template>
      }
  }
]

Do you have any other questions?

@motim6026 any update?

Because there is no reply to this question for a long time, I will close it temporarily. If you have any questions, please feel free to let us know.

Can Incremental Mode be declared in bicep?

Can Incremental Mode be declared in bicep?

@Shenglol Could you please help answer this question?

Does anyone have an answer to this yet?
Using incremental mode with bicep deployments..


Stopping guard doesn't always kill webpack --watch

I haven't really triggered what is causing it. But i'm finding that when i'm kiling guard and starting it again, zombie webpack's are running still.

Thanks for catching this! I was able to reproduce by kill -9'ing my Guard instance.
Looks like behavior should work properly when Guard is stepped down via $ exit, but if the parent thread is halted, we're not doing anything to ensure that child process is eliminated.
Should be able to get this fixed shortly :)

@blainekasten Should be working correctly now in master. Mind pulling it down and checking your use case before I close this up?
Thanks!

It seems to be working. I downloaded the new version and haven't had an
issue at all. thanks for the quick update! :+1:
On Tue, Mar 10, 2015 at 1:19 PM, Kevin Gisi<EMAIL_ADDRESS>wrote:

@blainekasten https://github.com/blainekasten Should be working
correctly now in master. Mind pulling it down and checking your use case
before I close this up?
Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/gisikw/guard-webpack/issues/1#issuecomment-78114895.

Cheers! Pushed a bugfix version update to RubyGems :)


Bump "safe" dependencies

This bump some "safe" dependencies atomically (as 1 update per commit). By "safe" dependency, I means it tried to update dependency that are not constrained, nor overridden, and that do not track master.
This has been done using sec, which mens:

It tried to update the dependency
Ran tests to validate it doesn't break (at least the build) (go test ./... or go test ./pkg/..., …).
Commit if the test command was successful.

sec is really experimental, but my end-goal is to have some kind of bot that could update those safe deps for us :angel:.
Also, this never update to latest either (if latest is the latest release, even if it's a major version bump). It's only updating minor or fixes versions.
Related PR "upstream"

https://github.com/knative/pkg/pull/200
https://github.com/knative/serving/pull/2697
https://github.com/knative/build/pull/500

/assign @mattmoor

I don't think updating dependencies to latest should be automatic, it really should be a decision by the project to pick up this new version or not.
One of reasons, the go community has build tools to manage dependencies such as dep or even go modules is to allow project to select which version they want to use.

you have a valid point, but I'm still not sure about the automated update of dependencies, I guess creating a PR is always a good thing, then someone will have to review it, but even minor versions updates I prefer to be an intentional decision

/test pull-knative-build-pipeline-go-coverage

/lgtm
/approved

/approve


Build Erlang 27

Needed to run tests on Erlang 27 on circleci.
MIM-2251

Do we want to update the underlying ubuntu version too? 24.04 was released a while back already :)

this one https://hub.docker.com/layers/library/ubuntu/24.04/images/sha256-c920ba4cfca05503764b785c16b76d43c83a6df8d1ab107e7e6610000d94315c?context=explore? last version is 22 days ago and 22.10 is still beta probably

What do you mean by this? Ubuntu 24.04 is official and fully released 🤔

uff, it is not underlying ubuntu version, it is cimg ubuntu, that's why tagging is weird.
I switched to current.


docker version of molecule not working.

molecule --version
molecule, version <IP_ADDRESS>a3.dev12
I am using it as a alias command that uses docker.
 alias molecule='docker run --rm -it -v "$(pwd)":/tmp/$(basename "${PWD}"):ro -v /var/run/docker.sock:/var/run/docker.sock -w /tmp/$(basename "${PWD}") quay.io/ansible/molecule molecule'
Molecule installation method (one of):
Docker
Ansible installation method (one of):
Arch linux package management
Detail any linters or test runners used:
Desired Behavior
molecule init role --driver-name docker --role-name ansible-role
Creating a new role named ansible-role
Actual Behaviour
molecule init role --driver-name docker --role-name ansible-role
I only get output of
--> Initializing new role ansible-role...
Traceback (most recent call last):
File "/usr/local/bin/molecule", line 10, in
sys.exit(main())
File "/usr/local/lib/python3.7/site-packages/click/core.py", line 722, in call
return self.main(*args, **kwargs)
File "/usr/local/lib/python3.7/site-packages/click/core.py", line 697, in main
rv = self.invoke(ctx)
File "/usr/local/lib/python3.7/site-packages/click/core.py", line 1066, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/usr/local/lib/python3.7/site-packages/click/core.py", line 1066, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/usr/local/lib/python3.7/site-packages/click/core.py", line 895, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/usr/local/lib/python3.7/site-packages/click/core.py", line 535, in invoke
return callback(*args, **kwargs)
File "/usr/local/lib/python3.7/site-packages/click/decorators.py", line 17, in new_func
return f(get_current_context(), *args, **kwargs)
File "/usr/local/lib/python3.7/site-packages/molecule/command/init/role.py", line 131, in role
r.execute()
File "/usr/local/lib/python3.7/site-packages/molecule/command/init/role.py", line 64, in execute
self._process_templates('role', self._command_args, role_directory)
File "/usr/local/lib/python3.7/site-packages/molecule/command/init/base.py", line 61, in _process_templates
no_input=True,
File "/usr/local/lib/python3.7/site-packages/cookiecutter/main.py", line 94, in cookiecutter
output_dir=output_dir
File "/usr/local/lib/python3.7/site-packages/cookiecutter/generate.py", line 364, in generate_files
generate_file(project_dir, infile, context, env)
File "/usr/local/lib/python3.7/site-packages/cookiecutter/generate.py", line 176, in generate_file
with io.open(outfile, 'w', encoding='utf-8') as fh:
FileNotFoundError: [Errno 2] No such file or directory: '/tmp/molecule/ansible-role/README.md'

Do not report bugs before reproducing them with the code of the master branch!
This text was in the template for a reason.


Improve IPv4 address validation for services_dhcp

The input pattern that goes with Form_IpAddress by default allows for IPv4 and IPv6 valid characters. The back-end validation here is checking for IPv4 addresses, so it seems reasonable that the front-end input pattern checks might as well be restricted to the IPv4 valid characters. Unneeded setPattern have also been removed.
The exception is Gateway, where the special string "none" is allowed, so I just left the setPattern as it is (yes, it could be modified to only allow the letters "n" "o" and "e").

If the other proposed change to enhance the input pattern "help" text for Form_IpAddress goes ahead, then this sort of thing will also be needed, otherwise the "help" will be misleading, making users think they can put in an IPv6 address.
In any case, this front-end input pattern validation might as well be a bit tighter than it was.
If this seems "a good thing" then I will look at the other uses of Form_IpAddress and see where they need to be made specific to V4 or V6.


#606: Update rubocop requirement from = 1.27.0 to = 1.28.2

null

View rendered readme.md

Codecov Report

Merging #607 (1833469) into master (af47d54) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #607   +/-   ##
=======================================
  Coverage   86.54%   86.54%
=======================================
  Files          57       57
  Lines        1642     1642
=======================================
  Hits         1421     1421
  Misses        221      221

Impacted Files
Coverage Δ

lib/lazylead/cli/app.rb
97.14% <ø> (ø)

lib/lazylead/model.rb
93.45% <100.00%> (ø)

Continue to review full report at Codecov.

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


Update solution to allow for space(s) after document

Challenge Learn how Script Tags and Document Ready Work has an issue.
This code should be a valid solution. The regex allows spaces before "document" but not after.
jQuery's documentation shows "document" nested between two spaces but FCC's regex prevents this from being a valid solution.
"assert(code.match(/\\$\\s*?\\(\\s*?document\\)\\.ready\\s*?\\(\\s*?function\\s*?\\(\\s*?\\)\\s*?\\{/g), 'message: You should add $(document).ready(function() { to the beginning of your script element.');"

Link to test
My code:

jQuery Playground

#left-well

        <button class="btn btn-default target" id="target1">#target1</button>
        <button class="btn btn-default target" id="target2">#target2</button>
        <button class="btn btn-default target" id="target3">#target3</button>

#right-well

        <button class="btn btn-default target" id="target4">#target4</button>
        <button class="btn btn-default target" id="target5">#target5</button>
        <button class="btn btn-default target" id="target6">#target6</button>

Hi, can I try to tackle this?

#8368 This seemed to work locally, but I'm a noob at regex please give me feedback if it is wrong.

I will QA for you @champlainTwombls. You need to make your changes in a separate branch.

Sorry about that. I will do that and make a new pull request as soon as I can.

shall I try?

@farhd Defiantly!


Fix pull clasenh "file emxxx not found" #4515

Fix pull clasenh "file emxxx not found" #4515

look good. could you please add a test case that has some enho includes to this repo?
https://github.com/abapGit-tests/ENHO-INCLUDES

https://github.com/abapGit-tests/ENHO-INCLUDES/pull/1 created by @sandraros

thanks 👍


add bedtime lullaby

A try for making a sleep timer. It is easier to let one try than explaining it.
For short: I tried to add the google clock sleep timer at first, but ended up deleting the activity.
Also merging my old fork made a mess, in reality there just are a few changes to some files. In April should be the first commit which truly matters.

I don't doubt that your proposal is interesting, but I'm really sorry, your PR is a bit of a mess... 73 commits for a single modification request is way too much to analyze especially since most of the commits concern my own modifications mixed with yours.
Please, update your main branch from mine or even clone the project again; create a new branch from your main branch dedicated to you modification and repropose your PR.
Let me remind you again: one PR = one modification, just to make things clearer.
In any case, I'll give it a try in the next days, as I'm very interested in improving the bedtime mode.

Yes, this time I did it. As I mentioned at the beginning ifyou look at the changed files you can see, that they don't correspond the 73 commits.
Anyway it still is just a draft and not ready for review jet. I just wanted to have one so I don't forget to fix stuff and since I didn't want to trouble you I left this pull request as a draft.
Just for the future, I open draft pull request relatively early when my feature clearly isn't structured and WIP.
Also DO NOT JET give it a try since this is still WIP and will stay so till at least next Sunday. I will message you if I think it is in at least beta stage and the sleeptimer itself works reliable (I noticed now it is still up to chance).

OK ok. Take your time working on it and thanks for working on it.
Let me know when I can start testing.
If you can, take the opportunity to make a new PR without all those useless commits appearing.
I won't be making any more changes for a while unless someone reports a major problem with the application.

I (somehow) can't stop the service when timed (the notification can) and don't know how to. I'll leave this to someone else and provide my work as base. Also I notice, that one jet can't decide the length of the sleep timer. to do this copy the code from the other spinner and adapt it.

@BlackyHawky  think I now fixed the issue I wrote of. I will now resolve the conflicts and start developing the clock app again. This is just a side note but I also laid the ground work for media alarm with this in a more viewable way (the com.best.music package).
Best regards and thanks for still developing this app


removed content style plugin

this pr removes content style copy plugin which has became redundant.

@JohnBra have removed sass dependancy

LGTM
Thanks again for the contribution! I highly appreciate it @mehimanshupatil !


README - add namespace

Adds namespace to the readme's example JSON

Closing as this change was included in Config JSON's PR #31 which has now been merged \o/


tour: provide answers to exercises

Context: https://go.dev/tour/methods/22
Hi rather than make exercises a school-like academic engagement to prove understanding please can you provide the answers to exercises available (with a single click)? This would be useful to professional developers new to Go limited in time for answering questions.
Thanks

Created this:
package main

import (
	"golang.org/x/tour/reader"
)

type MyReader struct{}

func (MyReader) Read(b []byte) (n int, err error) {
	for i := 0; i < n; i++ {
		b = append(b, "a"[0])
	}
	return n, nil
}

func main() {
	reader.Validate(MyReader{})
}

which is most likely wrong, but I cannot find a way how to see what I did wrong.


Tracking: Ideas for useful helper functions

I'll try keeping track of some useful helper functions to include in willow-rs.

[ ] Some form of Range::new_exact:
impl Range<Path> {
  fn new_exact_path(path: Path) -> Self;
}

An alternative for this would be:
trait Successor { // which is actually equivalent to `trait SubspaceId` at this point.
  fn successor(&self) -> Option<Self>;
}

impl<T: Successor> Range<T> {
  fn new_exact(value: T) -> Self;
}

[ ] Helpers for creating Paths from arrays of bytes/strings:
impl Path {
  fn from_bytes(&[&[u8]]) -> Self;
  fn from_strs(&[&str]) -> Self;
}

[ ] Helpers for printing Paths. Perhaps just a nice Path: Debug instance that interprets the path segments as utf8, if possible, or otherwise escapes to hex or similar. Alternatively something like Path::to_string(&self) -> Option<String> that tries to format using / and utf8, but fails if the path would have to be escaped.

[ ] Easily converting an Area into a Range3d: Area::to_range3d(&self) -> Range3d. Alternatively: a From/Into instance.

[ ] Easily converting an Area into an AreaOfInterest: AreaOfInterest::new_area(area: Area) -> Self or similar

Thank you for these suggestions, I'm agreed on all counts. I like the Successor trait, I think the only drawback would be needing a newtype to implement it on u64 for timestamps.

FWIW, I personally find it just easy enough to do a Range::new_closed, if it's a Range<u64>, so I'd be fine without a newtype.


ADFS Overview/What's new in ADFS: PSH explained

Description:
As discussed for more than a year now in issue ticket #2514 (What is "PSH" in the context of this document?), this page needs an explanation for the origin of the initialism "PSH", seeing that it is a source of confusion rather than helpful reading.
Thanks to @jordanmills (Jordan Mills), @husamhilal, @schnitz76, @jgallucci32, and @ThomasKur (Thomas Kurth) for following up.
Thanks in particular to @jgallucci32 for providing the correct codestyle.

Changes proposed:

Preface the first entry of "PSH" with the full name "PowerShell"
Place first entry of the initialism PSH in parentheses (PSH)
Follow up last entry of PSH with the name in parentheses (PowerShell)
Apply Camel Case: "Powershell" -> PowerShell
Typo correction: "Brose" -> Browse
Replace 2 old TechNet link URLs with their permanent redirect URLs
Whitespace changes:

Remove redundant end-of-line blanks
Move HTML </br> codes from start of line to end of previous line
Bullet point list hyphen: remove single preceding blank (no indent)
Normalize bullet points to 1 single space between text and bullet
Standardize on blank line after H2/H3/H4 headings
Standardize on 2 blank readability lines before H2/H3/H4 headings (to separate from lists spaced out with blank lines)
Reduce occurrences of 3 contiguous blank lines to 2
Add blank line between metadata and page title

Ticket closure or reference:
Closes #2514

#sign-off

#sign-off


Remove Default Span Configuration

Currently in Opentelemtry JS there is an instance of configuration for adding default span attributes. The use case for this overlaps heavily with the resource sdk. In addition seems this was just replicated from OpenCensus. Since this is just replicated functionality it is best to remove it.
The implementations can be found here:

https://github.com/open-telemetry/opentelemetry-js/blob/e9b2cf9aeb1daf5ffbab800681bfe1cafc636576/packages/opentelemetry-node/src/NodeTracerProvider.ts#L38
https://github.com/open-telemetry/opentelemetry-js/blob/master/packages/opentelemetry-tracing/src/types.ts#L30

@mayurkale22 do you have any opinion here? I was not around in the OC days or when this was introduced.
My gut feeling is to remove it.

I believe this field was there in the earlier version of specs. Seems like this concept is not present in other languages, I am fine to remove it.

@aravinsiva are you planning on making a PR or should I mark this as up for grabs?

 No problem

Closed via #1342


Input-output templates refactor

Description
Removed defaults and regexp for selects and booleans
Test
No changes
Coverage
No changes
Scalastyle
No info

LGTM


Release new version

1. What version of Caddy are you running (caddy -version)?
0.9.4
https://hub.docker.com/r/abiosoft/caddy/
2. What are you trying to do?
About a month and a half ago, the home automation software I use started using websockets. Since I have caddy setup to terminate tls and act as a reverse proxy for this service, it is broken in 0.9.4 (Should be fixed by PR #1309)
3.
With docker: /usr/bin/caddy -conf=/etc/Caddyfile -log=stdout -http2=false
5. What did you expect to see?
websocket connections do not return a 502
6. What did you see instead (give full error messages and/or log)?
No errors in logs.
Websocket connections stall. All https connections are ok.
7. How can someone who is starting from scratch reproduce this behavior as minimally as possible?
Set up home assistant, fronted by caddy with a similar proxy config. https://home-assistant.io/

I will get around to it eventually! Releases are half-day projects until the new site is up and running (then it will just be the push of a button).

@petey does introducing an edge or nightly version of the docker image makes sense. An image that builds daily or on every push to master.
The caveat is that you aren't guaranteed an iota of stability or up to date documentation. Documentation usually get updated during releases.

Also, I thought https://github.com/mholt/caddy/pull/1316 was the fix for Home Assistant?

@abiosoft Yeah, something like that would be greatly appreciated. I'm fine with accepting the risk of using pre-release software if it is labeled that way. I can always change my docker compose to pull a different label if it doesn't work, and it affects no one but myself.

@mholt Wasn't aware of that issue. Could be. I haven't had the opportunity to build a docker image from source to test any of this out.

Yeah, the code that is on master isn't yet enough to get home assistant working again. FWIW, here's the docker image I set up to build the server from source: https://hub.docker.com/r/linuxpete/caddy-from-source/

New version going out in a few hours. Thanks!

YEA, thank you @mholt!

FYI https://hub.docker.com/r/abiosoft/caddy/ also updated to 0.9.5