Update Corsican translation for FreeOTP for Android 2.0.4

Hello,
This is an update of Corsican localization to take into account the following commit:

https://github.com/freeotp/freeotp-android/commit/a6157bc338c4bf8a3cd73da86829d20bd8f6d0c7 Replace hardcoded welcome string

The format of the whole XML file has been standardized by using the 'Pretty print - indent only' function of Notepad++'s plugin XML Tools. In a nutshell, it makes the XML code easier to read and reduces the file size by using tabs instead of spaces as indent.
Best regards,
Patriccollu.

Thank you for the PR @Patriccollu - I should have checked this before releasing 2.0.4, I hope it is okay to wait for this to be included in the next release.

Ahh disregard my previous comment, I see this is a minimal change unrelated to functionality.


Refreshing Node.js supported versions

Updating the supported node versions to match the current list, including previews.
Fixes #756

Oh right, tests probably need updating too 😅

I'm not sure how to fix the failing test. The test that fails is this one here and it fails because it "thinks" we have v3 of the core tools installed but we need v4 (since v3 didn't support Node18 but now v3 does).
The problem is that we now have some overlaps in the version support (v3 and v4 both support Node 18 and Node 20), resulting in it not being able to tell what to install.
Probably need an overhaul of that test but I'm not sure what it should look like.

Hold off on merging this one for right now
Hey @aaronpowell - I'll get this merged once we have the big CVE/ESM move done.

@aaronpowell Could you please change the cached version here that will fix the unit test? In src/core/func-core-tools.spec.ts line 227:
fct.setCachedInstalledSystemCoreToolsVersion(2);
This will force the binary to be out of sync with the version of Node and return the expected downloaded binary

@aaronpowell Based on the timeline of our upcoming release, I've created a PR including your changes and the unit test fix at #869


[BitwiseCopyable] Imported type improvements.

Tested inference for imported types.  Don't infer when an imported type has storage that isn't imported into Swift.  Allow a type to explicitly state conformance via __swift_attr__.

@swift-ci please test


Prevent maxTTL being set to NaN

Add a check when iterating policyFields to prevent maxTTL being set to NaN which prevents referencesTTL being set.
referencesTTL not being set causes the cache to behave in very unpredictable ways.
I've also added validation for ttl and policy.ttl to throw an error if either isNaN
maxTTL = Math.max(maxTTL, undefined)
maxTTL = Math.max(maxTTL, NaN)

nice!
lgtm


Remove Home page of food access map; default to the Search page

See slide 1 of https://docs.google.com/presentation/d/1ktVrZegMr6E6Fi3PgSDvq6ntIqsuMxZD5vbt6mH19Ow/ for details!

@mike-feeney has an pull request in the pipeline for this that hasn't yet been made into an official pull request, so this is very close to being done.

Pull request made, so this is done! 🍾


pre_install.sh fails with code 1 at setup-miniconda@v3/installer step

Hi folks, I am 100% sure this not you, but core conda, have a look at our failed GA - this is happening across the board for us, and it's been happening since last night, also, we've seen this about two weeks ago, and within 24h it resolved itself. Just a heads up in case others start reporting this, and also, any clue what's the matter? Many thanks for your great Actions 🍻

I think you are referring to the Mambaforge brownouts. Please read the announcement in the warning text.
More details at https://conda-forge.org/news/2024/07/29/sunsetting-mambaforge/, but TLDR: Use miniforge instead of mambaforge.

I think you are referring to the Mambaforge brownouts. Please read the announcement in the warning text.
More details at https://conda-forge.org/news/2024/07/29/sunsetting-mambaforge/, but TLDR: Use miniforge instead of mambaforge.

@jaimergp whoa this is most useful, mate, cheers! I was brushing all this under a conda failure carpet, but in this case, we have to take action! I'll be onto this, many thanks for the pointer 🍺


Added top level meta support for JSON API

Closes #55

👍

[Current coverage][cc-pull] is 98.22%

Merging [#72][cc-pull] into [master][cc-base-branch] will increase coverage by <.01%

@@             master        #72   diff @@
==========================================
  Files             7          7
  Lines           393        394     +1
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            386        387     +1
  Misses            7          7
  Partials          0          0

[][cc-pull]

Powered by Codecov. Last updated by [ff84a87...8782053][cc-compare]
[cc-base-branch]: https://codecov.io/gh/devlucky/Kakapo/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/devlucky/Kakapo/compare/ff84a87cc79ec55b7d7d33e4288acde47d24a28c...8782053823ace46d1be2c451e7dda5abedb234e3
[cc-pull]: https://codecov.io/gh/devlucky/Kakapo/pull/72?src=pr


Hashable literal

Make class SCons.Subst.Literal hashable.
When using Python 3, substitution of Literal objects requires the objects to be hashable, otherwise an error will be thrown. The returned hash value is that of the lstr member.
This is not a feature update so I didn't write a test case and there is no need for updated documentation/readme imo.
Contributor Checklist:

[x] I have created a new test or updated the unit tests to cover the new/changed functionality.
[x] I have updated master/src/CHANGES.txt directory (and read the README.txt in that directory)
[ ] I have updated the appropriate documentation

Can you add a test case with your simple reproducer?
We're missing a testcase for this or we would have caught it prior to 3.0.0 release.

Added a test which will hopefully work on all platforms (not using $_RPATH).

LGTM!


can't find 'AMR' AF in gnomad_db_v3.1.2

hello,
when i use the gnomad_db_v3.1.2 ,i can't find the 'AMR' AF in the resluts, how can i get it from gnomad_db_v3.1.2 database?
thanks

Hello @loverlyday
here you can find the 23 columns, which I extracted from the gnomAD vcf file and inserted into the SQL database.
Indeed, "AMR" is not part of it.
INFO=<ID=AF_amr,Number=A,Type=Float,Description="Alternate allele frequency in samples of Latino ancestry">

You can recreate the SQL database by yourself by following these steps and including - AF_amr here.
You can use AF_popmax from the already existing SQL database to approximate the upper bound of the AF

Hello,
I tried to follow these instructions to make a SQL database with additional population data. However, the Snakemake program only outputted createTSVtables.ipynb and 15G of .tsv.gz files (most of which "finished" at the same timestamp). It did not output the SQL database or insertVariants.ipynb. No errors were thrown, and there is space for the database to be made. How can I fix this?
Thank you!

Hello @pkiehl2002,
Here you specify the location of the database: https://github.com/KalinNonchev/gnomAD_DB/blob/master/script_config.yaml
By default it is: test_out
Can you find it there?

Please open an issue next time to continue the discussion as your problem is not related to the one above.


Filtering

Hi, I see there is an ability in Spreadcheetah to create an auto filter. Is it possible to also create a filter rule so rows get filtered out when the file is opened by the user? In my case, for example, I am looking for not equals blank. If not, is there any workarounds?
Thank you,
Michael Chinitz

Hi, unfortunately this is not possible yet, but it can be added in the future. I'll keep the issue open as a remainder.


Fix warnings in Python 3.8

*Issue : Fixes #659 *
Description of changes:

Import ABC from collections.abc
Fix syntax warnings due to comparison of literals using is

Job PR-660-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-660/1/index.html

Thanks for the contribution, it looks good to me! @jwmueller please take a look and merge if you don't have any concerns.


Clarify that the Psychopy tasks repository MUST NOT be in a dirty state

The SOPs should clearly indicate that the repository on the psychopy laptop should be on a 'reproducible' state.
Ideally, this means 'on detached HEAD' mode, after checking out the latest commit tagged. While developing, we can live with having the repo on the master branch in a clean state (meaning, no staged or unstaged changes, in sync with the central repo's HEAD commit).
To achieve this, it would be necessary that:
1 (preferably) we do not develop the tasks on the presentation laptop. This doesn't mean we don't test them on that laptop, but we just don't work on the code there unless it is strictly necessary (e.g., a specific bug happening only on the laptop).
2 if developing in the laptop cannot be avoided, then develop under your personal user rather than the common user.
All of this should clearly be indicated in the SOPs

I think we have arrived to some understanding here and the SOPs are clear. Closing.


Review CS261 requirements

Are ruby, lua5.3, and lsb-core necessary for 261, or were they just added for generic UUG purposes? Are they better better served in a role for a different course?

There is no intentional use of ruby or lua in 261, so if they are not dependencies of something (check, logisim, or valgrind, maybe?) then they do not need to be included. I do intend to use Ruby in CS430 next semester. I'm honestly not sure about lsb-core; does that package actually provide anything or is it just a way of installing a bunch of common build dependencies at once?

Ruby is a system dependency, so it wouldn't really be going away, but we wouldn't call it out as a dependency of 261. Logisim and valgrind come from apt packages, so they should trigger installations of any dependencies as determined by their maintainers. lsb-core is a strange package that pulls in a lot of odds and ends, but I think build-essential might be a closer target to what we're looking for, which is already in our list.
If you're looking for the VM for 430, let's get an issue open to track the dependencies for that and create a dedicated install target.

Debian package for lsb-core: https://packages.debian.org/sid/lsb-core
Debian package for build-essential: https://packages.debian.org/sid/build-essential
lsb-core was not included in the latest stable release of Debian; not sure what that means for future versions of Mint. Especially since it depends on a variety of unnecessary tools, we could try to see what build-essential is missing and just pull those packages in.


Cache labels

Since labels shouldn’t change often, I thought of a clever way to cache them and speed up managing issues:

Use a global store and key on the owner and repo
On VC load, check if values exist in the cache

If they do, don’t fetch from network
Else fetch

Then either on fetch or manual refresh, update the store and UI.
Could apply this to assign, reviewers, and milestones too.

Bug Report Dump (Auto-generated)

Version 1.15.0 (SEE BUILD PHASE)
Device: iPhone X (iOS 11.1.2)
TestFlight: false

Dupe of #348?

can anyone give more context? I didn't understand requirement properly. screenshot?

Check the first comment in this thread. We should add caching to labels / reviewers / milestones. As this has to do with caching, I don't see a screenshot helping?

If we were going to cache branches, would this meet expectations:
class RepositoryBranchesCache {
    static let shared = RepositoryBranchesCache()
    private let cache = FlatCache()

    func get(owner: String, repo: String) -> Branches? {
        return cache.get(id: "\(owner)/\(repo)") as Branches?
    }

    func set(owner: String, repo: String, branches: [String]) {
        let branches = Branches(
            owner: owner,
            repo: repo,
            branches: branches
        )
        cache.set(value: branches)
    }
}
struct Branches: Cachable {
    let owner: String
    let repo: String
    let branches: [String]
    var id: String { return "\(owner)/\(repo)" }
}


Update FAQ.rst

Coverage remained the same at 100.0% when pulling 1bdba60b4b6244a9f1df4902f8fb052546f7c73d on faq-improvements into 4282b887aafed6c6be14ecc78e4886d86a27333a on master.


Inconsistent LogLevel configuration option

The LogLevel configuration option in the config.yaml file does not apply in a consistent manner. For example, the logs generated at some places may be more than configured level.

This has been fixed in 23.07 release with configuration for log level in config.yml file. The config file now controls log level across manager components.


Feature/arangosearch scatter node

Please note that for legal reasons we require you to sign the
Contributor Agreement
before we can accept your pull requests.

http://jenkins01.arangodb.biz:8080/job/arangodb-matrix-pr-linux/701/


docs: gRPC unix socket usage

Signed-off-by: Lorenzo Fontana<EMAIL_ADDRESS>What type of PR is this?
/kind content
Any specific area of the project related to this PR?
/area documentation
What this PR does / why we need it:
Related to the changes to Falco:

https://github.com/falcosecurity/client-go/pull/43

Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:

/hold

Wait until https://github.com/falcosecurity/falco/pull/1217 is merged.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign fntlnz
You can assign the PR to them by writing /assign @fntlnz in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: e3d359614eae7c63e94189721243d933d3dae16a

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leodido
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leodido]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: kris-nova, leodido
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [kris-nova,leodido]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/hold cancel

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: kris-nova, leodido, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [kris-nova,leodido,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Particle3D scale does not work in billboard mode.

Godot version
Godot_v4.1-dev1
System information
Ubuntu 22.04, RTX3060ti
Issue description
Particle3D scale does not work in billboard mode.
Steps to reproduce
Add 3D particles, enable billboard particle.
Minimal reproduction project
Particles test.zip

Do you have keep_scale checked or unchecked?

Do you have keep_scale checked or unchecked?

Wow!
It seems that this was not required before. Thank you.

I think you are right. You are not the first person to encounter this issue.
For context, the option was introduced in https://github.com/godotengine/godot/pull/65353. Previously the scale was always included, but it was included in a way that broke when using non-uniform scales or rotations.
Enabling keep_scale by default introduces an extra matrix multiplication, but it ensures correct behaviour

Yes, I think that would be the best default behaviour. Clay, what do you think about making an extension of StandardMaterial that extends the class and just puts better defaults for particles? (unshaded, transparent, keep_scale on, etc.). No special shader code, just different defaults?

For future reference, I have looked into enabling keep_scale by default for only particles billboard mode and it will be quite difficult given how material flags are implemented as keep_scale is shared between the particles billboard modes. At this point since 4.0 has been out already for several months, we may just be stuck with having it off by default.

Perhaps we can have the particle node then pre-configure a spatial material for its first draw pass?

Perhaps we can have the particle node then pre-configure a spatial material for its first draw pass?

That might be possible, as long as we could figure out a nice UX for it. Personally, I prefer just extending SpatialMaterial and having a "ParticlesRenderMaterial" that has good defaults for particles

Let's do that with a separate proposal then. I'll put it in my TODO list.


[PE-D][Tester A] No command to view patient

Based on the requirement that this should be an application focusing on CLI, I feel that it might be good to include a command 'view PATIENT_INDEX' to view patient by their index. If not the patient can only be viewed by mouse click.

Labels: severity.Medium type.FeatureFlaw
original: CShuwen/ped#8

include in the user guide show {index}

Noted

Added, fixed


Add test service

@James-Mart I've made those changes except the Rust wrapper import, I can't crack that... will try get on a call and sus it.

Feel free to merge before you start getting merge conflicts


Refactor Task Results Substitution

Changes

While working on When Expressions, found that this refactor would make
it easier to extend Task Results Substitution to When Expressions.
Specifically moving the logic to get the results refs to resultrefresolution.go
and splitting the logic to two functions with separate concerns.
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[x] Includes tests (if functionality changed/added)
[x] Includes docs (if user facing)
[x] Commit messages follow commit message best practices
[x] Release notes block has been filled in or deleted (only if no user facing changes)

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.
Release Notes
    NONE

/cc @bobcatfish

/test check-pr-has-kind-label

/kind cleanup

Thanks for this refactoring @jerop 🙏
/approve

/lgtm


Support swiping to open panes on mobile

Work started in the swiper-pls-swiping branch. Currently supports some swipes and not others


Properties defined on an Array are not unwrapped in template

Version
3.0.0-rc.5
Reproduction link
https://codepen.io/ByScripts/pen/YzqwEER
Steps to reproduce
<template>

{{ obj.prop }}

{{ arr.prop }}

</template>

What is expected?
Both prop properties should be unwrapped
What is actually happening?
Only the object prop is unwrapped. The array prop is not.

See #1682


Google Drive / Photos integration

Hi,
I installed StackEdit via helm on k8s cluster and created OAuth2 credentials in Google Cloud, but the integration is not working - Google asks for permissions, but returns without authenticating - no error displayed, but debugger shows 400 error.
Any idea?
Thanks

Found the issues:

The API key was invalid
Also People API should be enabled under Google Cloud project


Consider making a RFC for a style convention

So we can discuss it!

I will. I expect there will be multiple RFCs. The plan has been to get rustfmt into good shape with lots of options, then use RFCs to decide on the defaults as a Rust style guide. I'm not exactly sure the RFC process is the right one to do this, but there will be somewhere public and formal-ish for making these decisions. Probably some time a bit later this year.


Missing: BlockchainRetrievalServiceAsync and ConfigServiceAsync

When doing maven import via IntelliJ those files are missing:

com.yoghurt.crypto.transactions.shared.service.BlockchainRetrievalServiceAsync;
com.yoghurt.crypto.transactions.shared.service.ConfigServiceAsync;

These are generated by a maven plugin. I'm unfamiliar with how IntelliJ handles this as I use Eclipse myself.

Ide doesn't matter here, just make sure that you get those files when importing a fresh copy of the project.

Is this still an issue 3a later? I'm tempted to get my hands dirty, too.


Insert Row data Using a Query in snowflake using terraform

Is your feature request related to a problem? Please describe.
Yes, if we want to do automation of inserting data in a table, we can use this option.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
There is a requirement to insert row data in a snowflake table, and i almost tried all the options available from the last 1 week, and need help in this scenario.
Example:
INSERT INTO mytable (id, name, age)
VALUES (1, 'John Doe', 30);
this is one of a similar example which i want to run using terraform.
Describe the solution you'd like
May be any resource module, where we can run sql or python scripts.
Describe alternatives you've considered
No alternatives found as for now.

Hey @kushal1313. Thanks for reporting the issue.
We have introduced snowflake_unsafe_execute resource in v0.77.0 release. Does it fit your needs?

Hey @kushal1313 , I will close this issue, because of the inactivity and because the unsafe resource is currently usable in such a scenario.


Make the plugin runtime safe for concurrent usage

It was previously reading/writing/overwriting files in-place without any
synchronization mechanism.
This commit introduces a tmp directory inside the cluster's dir, which
is used as a staging directory for writing files before they are
eventually moved atomically to their final location.
https://jira.mesosphere.com/browse/DCOS_OSS-4843

@mesosphere-ci run integration tests

@mesosphere-ci run integration tests

failed on dcos-launch again, re-running

@mesosphere-ci run integration tests


[WIP] Introducing Federal Senate script to Rosie!

This PR is a work in progress, but it has started today.
The things that need to be done are:

[x] Create a sample randomic dataset to use to create tests.
[x] Finish the adapter.py script, writing the tests first (that is the reason why I stopped the script)
[x] Learn how to run the tests only for federal senate
[ ] Check if everything is working
[ ] :shipit:

Any opinion is important, so feel free to do so :)
(I never do this alone, so I stopped putting my name in the begining)

Coverage decreased (-1.4%) to 95.172% when pulling 94f316137a55d1807bc83e1917528208c4f5d273 on introducing-federal-senate into cceb0ff3f6b4c22a5673a35cfe76b335e043de8a on master.

Coverage decreased (-3.6%) to 92.939% when pulling dd19bfb2c814701299cdd99114bc21c61ccc98bd on introducing-federal-senate into cceb0ff3f6b4c22a5673a35cfe76b335e043de8a on master.

Coverage decreased (-3.8%) to 92.799% when pulling 1241f0819fe19aa3327747a83b4c222471cd2dda on introducing-federal-senate into cceb0ff3f6b4c22a5673a35cfe76b335e043de8a on master.

Coverage decreased (-3.8%) to 92.81% when pulling 3d5f2b92514817b2e36305267885d4795d3fac7a on introducing-federal-senate into cceb0ff3f6b4c22a5673a35cfe76b335e043de8a on master.

Coverage decreased (-4.2%) to 92.37% when pulling 15814418dea222ee6bed7228ab2d13bf893e8f15 on introducing-federal-senate into cceb0ff3f6b4c22a5673a35cfe76b335e043de8a on master.

Coverage decreased (-1.3%) to 95.246% when pulling 247a2db0562b2b35ab2b33b855632d51b686c998 on introducing-federal-senate into cceb0ff3f6b4c22a5673a35cfe76b335e043de8a on master.

@jtemporal there is only one thing missing, @cuducos asked if we can comment on the code why we create another column for invalid_cpnj_cpf, I can do that later, but for now I will only admire that good work you made yesterday <3
👏

Coverage decreased (-1.3%) to 95.246% when pulling 40e1de3bdeb120e213d24a137011c9585c7f1718 on introducing-federal-senate into cceb0ff3f6b4c22a5673a35cfe76b335e043de8a on master.

Learn how to run the tests only for federal senate

I believe commit 40e1de3 took care of that ;)

Coverage decreased (-1.3%) to 95.246% when pulling 6d00705a76883cf0f4c3a5e0846f2640c49f55c1 on introducing-federal-senate into cceb0ff3f6b4c22a5673a35cfe76b335e043de8a on master.

Coverage decreased (-1.2%) to 95.367% when pulling cb62ce8515783e7db763dd3d9e8d9a4abd778ac2 on introducing-federal-senate into cceb0ff3f6b4c22a5673a35cfe76b335e043de8a on master.

Coverage decreased (-1.2%) to 95.367% when pulling cb62ce8515783e7db763dd3d9e8d9a4abd778ac2 on introducing-federal-senate into cceb0ff3f6b4c22a5673a35cfe76b335e043de8a on master.

Coverage decreased (-1.2%) to 95.367% when pulling d704a0a7c550bb69a573d150500280375fd91a61 on introducing-federal-senate into cceb0ff3f6b4c22a5673a35cfe76b335e043de8a on master.

We have a problem which is: we need Rosie to update the data from Federal Senate every time she runs just like she does with the Chamber of Deputies data.
Right now that doesn't happen. Also, she is expecting that we already have a federal_senate_reimbursements.xz because of how the data path is built on the adapter
We need help to mock update_datasets() or figure out a way to fix this issue.
On hold until we finish this, soon will be resolved.
cc @cuducos

We need help to mock Adapter.update_datasets() or figure out a way to fix this issue.

Running the update method by default, but mocking it in tests: that sounds like a really good approach IMHO.

Running the update method by default, but mocking it in tests: that sounds like a really good approach IMHO.

This idea sounds really good to me, that could be a way, if it looks good to @jtemporal we can work on it tomorrow, or I can try it later.

This idea sounds really good to me, that could be a way, if it looks good to @jtemporal we can work on it tomorrow, or I can try it later

that's the plan

Coverage decreased (-1.2%) to 95.367% when pulling 545ab5780214b8ac5cd18c599b5b4a246b7bb5e3 on introducing-federal-senate into cceb0ff3f6b4c22a5673a35cfe76b335e043de8a on master.

Coverage decreased (-1.2%) to 95.367% when pulling 5d7f1572cfdfdbb37629bcba4a8a765289a32fb4 on introducing-federal-senate into cceb0ff3f6b4c22a5673a35cfe76b335e043de8a on master.

Coverage decreased (-1.2%) to 95.382% when pulling efe32b9314c9d35620bdcfe601504bd19e60e0a1 on introducing-federal-senate into cceb0ff3f6b4c22a5673a35cfe76b335e043de8a on master.

Coverage decreased (-1.2%) to 95.39% when pulling 9427bed02ef247822f71363737dcc04f22e04bd1 on introducing-federal-senate into cceb0ff3f6b4c22a5673a35cfe76b335e043de8a on master.


Add Host::register_test_contract_wasm

What
Add Host::register_test_contract_wasm, which accepts a WASM byte slice and a contract ID to register it against.
Why
When writing tests, developers will want to register not only their own contract, as is doable via Host::register_test_contract, but also other contracts for which they have only the WASM for so that they can do broader integration testing.
Some examples in the soroban-examples repo have achieved this by creating the other contract with the Host::create_contract_from_contract function. The problem with this approach is that the contract being created is created as if it was created by the developers contract. This is conceptually not aligned with how a developer thinks about deploying the other contract, because a contract that a developer writes won't always be deploying the contract it is interacting with.
Therefore it is helpful if we allow developers to simply register contracts without going through all the steps to deploy that contracts from within their own.
This is only for tests infrastructure and is gated behind the testutils feature.

See https://github.com/stellar/rs-soroban-sdk/pull/438 for an example of how this will be used.

I'm not sure I understand the motivation for this change. I wasn't able to find any examples that actually do what's described as

Some examples in the soroban-examples repo have achieved this by creating the other contract with the Host::create_contract_from_contract function. The problem with this approach is that the contract being created is created as if it was created by the developers contract. This is conceptually not aligned with how a developer thinks about deploying the other contract, because a contract that a developer writes won't always be deploying the contract it is interacting with.

but I feel like I'm looking for the wrong thing because I'm not understanding the motivation yet. Can you point me to one? There are some contracts (eg. liquidity pool) that actually do need to deploy another contract, and there are some tests (eg. for liquidity_pool and single_offer) which deploy other contracts as test contracts but not using WASM.
There are other ways to deploy WASM contracts that don't depend on doing it from in a contract. The create_contract_from_ed25519 host function exists, but isn't exposed in the SDK right now because it hasn't been needed. The main difference I see between what's in this PR and create_contract_from_ed25519 is the ability to choose the exact ID that the deployed contract receives. I suppose this could be an important feature if you want a hard-coded address in your contract (whether this is a good idea or not, I'm not really sure). Is the ability to deploy at a specific address the main motivation for this change?

I wasn't able to find any examples that actually do what's described as

Some examples in the soroban-examples repo have achieved this by creating the other contract with the Host::create_contract_from_contract function

I'm referring to this code where the liquidity pool contract deploys the token contract: https://github.com/stellar/soroban-examples/blob/04781cfadd9d6fc42dd80867e6fb2a8e5a06adda/liquidity_pool/src/token_contract.rs. I think this approach makes sense for the liquidity pool contract because it actually does as part of normal operation deploy contracts. Another contract I'm writing just invokes another contract and has no responsibility for deploying it.

The main difference I see between what's in this PR and create_contract_from_ed25519 is the ability to choose the exact ID that the deployed contract receives.

+1 another option would be to plumb through Host::create_contract_from_ed25519, but it seems odd to me for this to show up in the SDK given contracts can't use it, and it requires more code to use since a user needs an ed25519 key to be generated.
Since we already provide Host::register_contract, it feels more natural and simpler for us to also allow registering a wasm contract.
If we end up plumbing through create_contract_from_ed25519 we could consider removing register_contract_wasm.
Does the above address your concerns?

but it seems odd to me for this to show up in the SDK given contracts can't use it

A contract can create a contract on behalf of an ed25519 public key, it would just need to receive the signature as input.

and it requires more code to use since a user needs an ed25519 key to be generated

This is definitely true, and as such it's not really convenient to deploy a contract this way. I'm fine with merging this, especially for the purpose of making it possible to choose the ID.

I'm not too concerned about using a specific ID, unless of course a developer is hardcoding an ID in their code.

If this part doesn't seem relevant, it probably would have been better to add a wrapper method to the SDK that generates an ed25519 keypair and uses create_contract_from_ed25519 to actually do the work. This would mean that functionality is syntactic sugar from the host's perspective instead of having specific host functionality to support it.

I think it's relevant, just not how I ended up here. A developing hardcoding an ID would need this for testing, and have them jump through hoops relating to ed25519 feels irrelevant to most tests. Using randomly generated values inside a test can at times by inconvenient and get in the way for testing. I know @paulbellamy is also wrestling a little right now of how to build contract deployment into SDKs because the terms aren't intuitive. Even if we plumb through create_contract_from_ed25519 it will be a testutils only function, and it will just serve to expose impl details. I think this is okay as is, but we can revisit it if we see it create problems?


confusing error when not exporting types

TypeScript Version:  3.6.0-dev.20190615
Search Terms: not exported types, dynamic types
Code
https://github.com/robertIsaac/tsc-dynamic-type-issue
auth-response.ts
type AuthResponse = SuccessResponse | FailResponse;

interface SuccessResponse {
    success: true;
    token: string;
}

interface FailResponse {
    success: true;
    message: string;
}

insert-response.ts
type InsertResponse = SuccessResponse | FailResponse;

interface SuccessResponse {
    success: true;
    insertedId: string;
}

interface FailResponse {
    success: true;
    message: string;
}

user-api.ts
class UserApi {
    insertUser(username: string, password: string): InsertResponse {
        return {
            success: true,
            insertedId: 'id'
        };
    }

    login(username: string, password: string): AuthResponse {
        return {
            success: true,
            token: 'asd'
        };
    }
}

Expected behavior:
it success and compile
or
it throw
user-api.ts:2:53 - error TS2304: Cannot find name 'InsertResponse'.

2     insertUser(username: string, password: string): InsertResponse {
                                                      ~~~~~~~~~~~~~~

user-api.ts:9:48 - error TS2304: Cannot find name 'AuthResponse'.

9     login(username: string, password: string): AuthResponse {
                                                 ~~~~~~~~~~~~

Found 2 errors.

Actual behavior:
user-api.ts:3:9 - error TS2322: Type '{ success: true; insertedId: string; }' is not assignable to type 'InsertResponse'.
  Property 'token' is missing in type '{ success: true; insertedId: string; }' but required in type 'SuccessResponse'.

3         return {
          ~~~~~~~~
4             success: true,
  ~~~~~~~~~~~~~~~~~~~~~~~~~~
5             insertedId: 'id'
  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
6         };
  ~~~~~~~~~~

  auth-response.ts:5:5
    5     token: string;
          ~~~~~
    'token' is declared here.

user-api.ts:10:9 - error TS2322: Type '{ success: true; token: string; }' is not assignable to type 'InsertResponse'.
  Property 'insertedId' is missing in type '{ success: true; token: string; }' but required in type 'SuccessResponse'.

10         return {
           ~~~~~~~~
11             success: true,
   ~~~~~~~~~~~~~~~~~~~~~~~~~~
12             token: 'asd'
   ~~~~~~~~~~~~~~~~~~~~~~~~
13         };
   ~~~~~~~~~~

  insert-response.ts:5:5
    5     insertedId: string;
          ~~~~~~~~~~
    'insertedId' is declared here.

Found 2 errors.

Playground Link: can't be done as it depend on multi-files
Related Issues: didn't find any
notes
exporting the types and importing them into the used files (user-api.ts in this case) does solve the issue as in this PR
https://github.com/robertIsaac/tsc-dynamic-type-issue/pull/1/files
i don't really understand how it doesn't throw the Cannot find name 'InsertResponse'. error but i think the tsc engine trying to figure it out on itself but when it encounter two SuccessResponse it can't figure which one to use

This is working correctly.
When the same exact interface is declared twice, the declarations are merged: https://www.typescriptlang.org/docs/handbook/declaration-merging.html
The two declarations of SuccessResponse are considered the same interface because they're both at the global level.  When a file has no imports or exports, it's not treated as a module, and instead everything it does happens in the global scope: https://www.typescriptlang.org/docs/handbook/modules.html#introduction
Your code ends up creating the following declaration in the global scope:
interface SuccessResponse {
    success: true;
    token: string;
    insertedId: string;
}

Which the functions do not satisfy.  Exporting fixes it because once you have at least one import or export in a file it's now a module and the declarations are separate.

thanks a lot @nattthebear i understand now
can you please share a reference to understand deeply the TS global scope


Added section for Jira ticket creation for escalations

Summary
Added section for SET Jira ticket creation for escalations.

/check-cla


add option to check integrity/authenticity

For SWARM and IPFS this won't be necessary, as the address hashes also validate it.

We should use the same hashing mechanism as Swarm/IPFS to minimize code to add.

I think just an optional field for a sha256 checksum plus the internal check routines in the code should do it.

Done in 1.5.0.


allow dev_*.py modules to be located in testlib subdirs

recursively search for dev_*.py modules under testlib
Change-Id: If057f80fc3d8ce420c7974255ded9104eebb5852
Signed-off-by: Ross Brattain<EMAIL_ADDRESS>

bad topic branch


NodeJS MN crashes during domain-registry tests cases

During the test execution of the MN conformance tests in registration.spec.js the NodeJS MN crashes with following log:
nodejs      | [2017-06-07 08:41:05.714] [INFO] server - [C->S] new event :  {"id":1,"type":"read","from":"hyperty-runtime://localhost/1/registry","to":"domain://registry.localhost","body":{"resource":"user://google.com/openIdTest10"}}
nodejs      | [2017-06-07 08:41:05.714] [INFO] server - [PEP] PDP PolicySet 1/ Policy 2/ Rule 3 determined to permit. resulted from allow-overrides algorithm of PDP PolicySet 1
nodejs      | [2017-06-07 08:41:05.714] [INFO] server - [Domain-Registry-Manager], [ domain://registry.localhost/ ]: handle registry message :
nodejs      |  { id: 1,
nodejs      |   type: 'read',
nodejs      |   from: 'hyperty-runtime://localhost/1/registry',
nodejs      |   to: 'domain://registry.localhost',
nodejs      |   body: { resource: 'user://google.com/openIdTest10', auth: true } }
nodejs      | [2017-06-07 08:41:05.715] [INFO] server - msg.msg.type.toLowerCase() is: read
nodejs      | [2017-06-07 08:41:05.721] [INFO] server - [ domain://registry.localhost/ ] Reply from domain registry :
nodejs      |   { code: 404, description: undefined }
nodejs      | undefined:1
nodejs      | undefined
nodejs      | ^
nodejs      |
nodejs      | SyntaxError: Unexpected token u in JSON at position 0
nodejs      |     at Object.parse (native)
nodejs      |     at parseObj (/usr/src/app/src/main/components/rethink/DomainRegistryManager.js:50:25)
nodejs      |     at registryConnector.processMessage (/usr/src/app/src/main/components/rethink/DomainRegistryManager.js:71:23)
nodejs      |     at /usr/src/app/node_modules/dev-registry-domain/connector/src/hyperty.js:51:5
nodejs      |     at /usr/src/app/node_modules/async/retry.js:149:26
nodejs      |     at Request._callback (/usr/src/app/node_modules/dev-registry-domain/connector/src/js-request.js:49:14)
nodejs      |     at Request.self.callback (/usr/src/app/node_modules/request/request.js:188:22)
nodejs      |     at emitTwo (events.js:106:13)
nodejs      |     at Request.emit (events.js:191:7)
nodejs      |     at Request.<anonymous> (/usr/src/app/node_modules/request/request.js:1171:10)
nodejs exited with code 1

This issue is relevant for D6.4.

I guess this due to new specs.
When is the deadline @sdruesedow ?

afaik the D6.4 shall be ready mid of June. I need to repeat the tests and document them, so I think next Tuesday latest would be OK from my side. Do you think it's feasible?

OK

any updates?

I'm documenting the situation for D6.4 as it is

Hi,
From WP-6 perspective, I am fine with this as we have to only assess the work in WP-6 and provide feedback to the technical WPs.
But from an overall perspective, this will raise issues in the final review, i.e.,

that a technical issue, that features having had worked before, were not working any more,
that this situation had been reported to the technical WPs, and
that a solution was not found within a month, nor was there a documented follow-up discussion.

Just my two cents.

sorry for the delay reply,
Last week I was in Oulu, for EuCNC event.
I'm working on the issue right now, could please @sdruesedow guide me on how to test it locally ?

Are there some updates available? With the available develop-branch version the MN is still crashing during the registry tests.

The currently available MN still crashes during the registry testcases due to JSON parsing trouble. This is now documented in D6.4.

Issue reproducted : seems to be linked with commit from @Heriam  : https://github.com/reTHINK-project/dev-msg-node-nodejs/commit/bf20ec1de279ff9c59b396d9f2a0b76000eaeea3#diff-ddd9b51315ce0bbd46ea1d02a46f0a8d

@FredLuart yes I tried to solve the JSON parsing issue before and it worked. No idea why it produces new issues since I have not worked on it for a period. Maybe try to undo this commit (remove my change) to see if it works.

It seems that the error happens, when the registry returns the value "undefined" for the description of a 404 response. Parsing of "undefined" fails and the MN crashes.
In addition to reverting the commit I suggest to add a code snipped like this one on the registry response before returning it:
      try {
        body.value = JSON.parse(body.value);
      } catch (e) {
        console.log("[RegistryInterface] CALLBACK: unable to parse body.value \n", body.value);
      }

because the registry started at some point in the past to return a stringified JSON instead of a JSON Object in the "body.value" attribute of its response message.

@sdruesedow I have just pushed a patch on develop branch. I still have 1 test failed on :  "receive registration status updates". I am checking the issue.
Let me know if you need some help about D6.4

Hi @FredLuart. Great - Thanks! This test is expected to fail, because it is not yet supported by the RegistryConnector and/or the Registry itself (I'm not sure about that). But it is already in the spec and therefore part of the testcases.

@FredLuart I just updated and ran the tests again and can confirm that they are running successfully now. I'll update D6.4 accordingly. Thanks a lot!

@sdruesedow @FredLuart -- Thank you so much for solving the remaining issues with the code.  That was the last missing piece for a successful WP6 closure.

Indeed, I would like to second the appreciation for solving the issue.
From: emmelmann-fokus<EMAIL_ADDRESS>Sent: Monday, July 10, 2017 4:40 PM
To: reTHINK-project/dev-msg-node-nodejs
Cc: Subscribed
Subject: Re: [reTHINK-project/dev-msg-node-nodejs] NodeJS MN crashes during domain-registry tests cases (#30)
@sdruesedowhttps://github.com/sdruesedow @FredLuarthttps://github.com/fredluart -- Thank you so much for solving the remaining issues with the code. That was the last missing piece for a successful WP6 closure.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/reTHINK-project/dev-msg-node-nodejs/issues/30#issuecomment-314126314, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AKNmJsmbqMF5qeOPk__Nn3HCzy7Uq7dnks5sMje1gaJpZM4NyZGm.


Proxy settings of Dockerfile

Hi, there is issue on proxy when we build docker db images, it could be ER for dockerfile .
When we build a db image via Dockerfile in intranet,  docker build error out when running yum update.  I have checked proxy settings of docker daemon . Details in the note https://www.henryxieblogs.com/2018/09/proxy-settings-for-docker-daemon-and.html
docker itself access proxy well and docker login is working well but docker build still fails on yum update
eventually  i add below into Dockerfile to fix it
# add proxy
RUN echo "proxy=http://<proxy server ip address>:80" >> /etc/yum.conf

Hi @HenryXie1,
Proxy information is automatically passed on to the docker container as long as their are set.
Please set https_proxy (and http_proxy) before invoking buildDockerImage.sh:
export https_proxy=my.proxy.com:80
export http_proxy$https_proxy
./buildDockerImage.sh -e

You will see that the build script will propagate the proxy settings into the container.


UI request for customizable side panel in iPad mobile

Because screen real estate is at a premium in Obsidian Mobile, could there be a setting(s) to shrink the size of the two (left & right) side panes.

This is a duplicate of #101 — will investigate


Implement configuration service

Refactor the github-service, so it's agnostic to the underlying CVS. Consider modules for GitHub, GitLab, and BitBucket. At least implement the GitHub version.

Consider this while implementing:
When creating a project or onboarding a service, files are committed and pushed to GitHub.
Currently, the github-service immediately closes the websocket communication after a successful HTTP response from the GitHub API. However, GitHub takes some time to apply these changes.
Here, we can run into race conditions when e.g. further files are written in a following command.
Proposed Solution:
The github-service has to check whether the files are available on GitHub. Only when the files are available in GitHub, the github-service should return.


The :: bind operator probably won't be going forward

Hey guys,
From the looks of things, the bind operator won't be going forward.
I'm currently using this little helper:
export default function bindHandlers(Target) {
  return class BoundComponent extends Target {
    constructor(args...) {
      super(args...)

       // Get's the target prototypes properties
      const properties = Object.getOwnPropertyNames(Target.prototype)
      for (let methodIndex = 0, len = properties.length; methodIndex < len; methodIndex++) {

        const method = properties[methodIndex]
        // If the method's name starts with an _, bind the method
        if (method.startsWith('_')) {
          this[method.substring(1)] = this[method].bind(this)
        }
      }

    }
  }
}

Not sure if this is the best way to do it, but it's a decorator that binds all methods that begin with an _, removing the _ for the bound method. It may be better to just add a bindHandlers function to the prototype.
Do think the project should still use the bind operator or switch to an auto-binding option?

Thanks for pointing this out. It makes me a bit sad, but my IDE never really liked it, so my code might actually be "green" now.
The this keyword has always been a bit of a third rail in javascript. To be honest, I don't even see the need for the _ in your example. I'm never going to put a method on a javascript class that I want to ever be run with another this value. autobind-decorator supposedly does this, but I found that it didn't work too well whenever I used it.

There's another way of getting around the common use case of :: in Component event handlers:
class MyThing extends Component {
  const onClick = (e) => { this.isBound(); }

  render() { return
 }
}

Arrows to the rescue!

The arrow is a good idea. That still works with super, correct?
Also, this solves a problem with unbinding listeners.
This returns false: ::this.handle === ::this.handle.
meaning you can't un-attach bound listeners easily.

After testing, you can't call super with the arrow approach. Babel reports unexpected super.
Classes really should have been spec`ed to be bound by default. :grimacing:

IMHO, you shouldn't be using OO inheritance at all in React, so super is not an issue.

@quicksnap i've been using the same technique! in your code snippet i just wanna point out is that the click handler would have to be a class method for it to work ie..
class MyThing extends Component {
  boundHandler = () => this.isBound();
  render() { return
 }
}


Merge Attestation-Service and KBS

follow https://github.com/confidential-containers/confidential-containers/issues/169

Let's wait for the PR https://github.com/confidential-containers/guest-components/pull/388 to be merged and then continue with this PR.

Things have been fixed:

CODEOWNERS/LICENCE
import relationship
CI test

We still need the following things:

wait for https://github.com/confidential-containers/guest-components/pull/388 and change rev for kbs/tool
A new repo name
fix the guides due to the new repo name

follow confidential-containers/confidential-containers#169

I've added my comments to the issue.

My vote for the name would be to simply stick with kbs for now. If we come up with a good name that covers all of the components we can change it later. So far I don't think we have a great alternative.

LGTM

@fitzthum did you follow to the conversation in the issue? There were objections to this change and now it's merged without a concensus.

LGTM

@fitzthum did you follow to the conversation in the issue? There were objections to this change and now it's merged without a concensus.

@mythi
I must have done something bad that click the rebase button before reaching a concensus with you. We can revert the merging if needed.
Also, a better way is that let's keep talking about that in the issue or in slack (I recommend) to reach a consensus.

LGTM

@fitzthum did you follow to the conversation in the issue? There were objections to this change and now it's merged without a concensus.

@mythi
I must have done something bad that click the rebase button before reaching a concensus with you. We can revert the merging if needed.
Also, a better way is that let's keep talking about that in the issue or in slack (I recommend) to reach a consensus.


Refactor Register API in AppNotifications

Description: Refactoring the existing implementation to be able to do specialized operation depending on the Application type. This makes it easy to even introduce overloaded Register API that takes in assets to directly call into appropriate helpers to assist in registration process.

Will have to rework on this. Closing the PR for now.


'str' object has no attribute 'decode'

Line 64 in auth.py:

token = token.decode("utf-8")
I believe this line is not necessary in Python 3 and can be removed.
It also causes the above error in some cases.

Hello, thanks for the report. Feel free to open a pull request and add the appropriate test case, should be an easy first contrib !

Just figured it out!
token is a  token string encoded with: encode(HTTP_HEADER_ENCODING)
which makes line 64: token = token.decode("utf-8") necessary.
TLDR: No changes needed. False report.


Switch to bracket syntax for node attributes

As pointed out in #104, method access (node.foo.bar) is deprecated and will be removed in Chef 13, and newer versions of Chef produce a large deprecation warning when anything else than bracket syntax is used.
I need to have a look at the tests as well (to stay consistent).

Now we're using strings all over (I think at least). Need to run through the integration tests to make sure that nothing explodes unexpectedly.


Add MS2Deepscore to MZMine

Adding a module to MZMine that is able to make ms2deepscore (https://github.com/matchms/ms2deepscore) predictions.

[x] Load model
[x] Add test for making correct predictions for tensor
[x] Implement embedding creation. (without directly calculating cosine)
[x] Add method for tensorizing spectrum
[x] Create flexible method for tensorizing spectrum, based on model settings (the additional metadata is still fixed)
[x] Add test for correct tensorization
[x] Add integration test for correct embedding generation from spectrum
[x] Add dot product calculations to create similarity scores
[x] Add module to MZMine to use ms2deepscore for molecular networking and library searching
[x] Add a parameter to add the path to the ms2deepscore model (instead of default test model)
[x] Add automatic download of ms2deepscore model (from zenodo?)
[x] Add minSignals (maybe using SignalFiltersParameters)

I will look at it tomorrow. Sounds great so far.

@robinschmid Thanks for the review! (and sorry it took so long before I had time again...) Really helpful, to have this quick feedback loop, to get myself familiar with the syntax. I have applied your changes, all your comments were clear. I will now have a look at integrating ms2deepscore with the MZMine modules. Once I have this working I will ask for a new review.

In our documentation you should find a part in intellij where you can add a file filter for the copyright header. So formats that it should not apply to.

@robinschmid The current version has all basic functionality. MS/MS spectral networking (ms2deepscore) works. We can select a model file. I have added a test file to the resources/models so you can use this for testing (but the results will be meaningless). The larger models I tested myself also work fine. And visualization in interactive ion identity molecular networking is working.
Could you let me know if everything is implemented in a correct way and what steps we should still do?

Thank you for all the work @niekdejonge! Did @robinschmid and you discuss why MS2Deepscore does not implement the SpectralSimilarityFunction interface so we can use it everywhere where we use cosine similarity?

we have to check it. And see if we can come up with a good strategy for this. Not sure if the interface design is good.
Definitely need to check how we can combine it in one module in the end. Very different parameters needed for cosine / modified cosine than for MS2Deepscore.

The input takes a parameter set, so i think that should not be a problem, or?

MS2Deepscore works so differently that it does not really fit in. No mz tolerance, no minimum matched signals.
public abstract SpectralSimilarity getSimilarity(ParameterSet parameters, MZTolerance mzTol,
      int minMatch, DataPoint[] library, DataPoint[] query);

Best is to send a batch of spectral data to calculate the embeddings - then the similarity is a simple vector comparison by cosine sim. So the spectral alignment step is not needed. I think it might be good to have two interfaces somehow - one for vector based and one for spectral based.


feature: Helper utility to generate a collection of faked objects for bulk seeding

Helper utility to generate a collection of faked objects for bulk seeding
Use Case
When using RedwoodJS and seeding data with Prisma, I often find myself wanting to quickly generate N models populated with data.
For example, given the Prisma model:
model Person {
  id            Int    @id @default(autoincrement())
  fullName      String @unique
  postalAddress String
}

I may want to quickly generate 100 people.
Currently, I do something like this to construct 100 people with some fake data and then save ...
  const data: Prisma.PersonCreateArgs['data'][] = [
        ...Array(100).keys(),
      ].map((key) => {
        return {
          fullName: copycat.fullName(key),
          postalAddress: copycat.postalAddress(key),
        }
      })

// ...

  await db.person.createMany({ data, skipDuplicates: true })

But, I keep having to remember the somewhat clunky syntax
[ ...Array(100).keys(),].map((key) => {}

and it would be useful in copycat to have some method that can:

take an argument for how many data objets to generate
return a collection of those populated objects to save

I could envision several implementations:

a callback to set the "generate" methods ... in the above example the returned {fullName, postalCode}
option to return a collection (memory heavy) to use with createMany
option to "stream" so-to-speak so can create individual via Prisma create (to support databases other than Postgres, but if you are using Snaplet, not sure why... but could be useful still).
option to configure the "shape" like that in fictional to configure the object ... e.g., define:

const person =  {
          fullName,
          postalAddress,
        }

const person =  {
          name: fullName,
          postalAddress,
        }

so that no need to pass in the key per item.

Hey @dthyresson o/
The use case totally makes sense, thanks for bringing it up!
I actually think a combination of times and shape might help here:
import { copycat } from '@snaplet/copycat'
import { shape } from 'fictional'

const person = shape({
  fullName: copycat.fullName,
  postalAddress: copycat.postalAddress
})

copycat.times('someInputKey', 100, person)

/* =>
[
  {
    fullName: 'Arne Turner',
    postalAddress: '884 Williamson Inlet, Minot 3065, Lebanon'
  },
  {
    fullName: 'Casper Deckow',
    postalAddress: '141 Tierra Mountain, Kenner 6145, Cayman Islands'
  },
  // ...
]
*/

Would that work for you?
If it'd help we could also re-export shape from fictional so you only have to work with copycat
re streaming and memory, its a good point - times will all be in-memory. Would this be enough for your use case, or do you need to generate a very large number of items?

@justinvdm I didn't know about times ... that's perfect:
import { copycat } from '@snaplet/copycat'
import { shape } from 'fictional'

// ...
    const n = 100_000
    const batch = 10_000
    console.info(`Creating ${n} people to search through ...`)

    const person = shape({
      fullName: copycat.fullName,
      postalAddress: copycat.postalAddress,
    })

    for (var i = 1; i <= n / batch; i++) {
      const data: Prisma.PersonCreateArgs['data'][] = copycat.times(
        `person-${i}`,
        batch,
        person
      )

      console.info(
        `Creating ${data.length} x ${i} = ${data.length * i} records ...`
      )

      await db.person.createMany({ data, skipDuplicates: true })

Is working well for me to batch create 100k records (yes, I wouldn't typically seed this but I needed to demo a case insensitive search index and needed a larger dataset).
I'll write up a post for Redwood or maybe update the seed documentation to suggest this approach.

exporting shape might be a nice improvement
eventually, something the stream and create in batches could also be helpful.

// 1_000 times in batches of 100
copycat.inBatches('someInputKey', 1_000, 100, person), callback)

Great, glad times works :)

exporting shape might be a nice improvement

👍 I'll make a PR for this soon

eventually, something the stream and create in batches could also be helpful.

👍 I can see from your example how it would simplify things for use cases like seed scripts.
I'll close this one for now then, thank you for bringing this up.


Add "point_color_mode" and "point_color_domain" to the spec files

Fix error where lowercase constants should have been uppercase.
Support both for backwards compatibility.
Fixes https://github.com/uber/xviz/issues/547

Coverage decreased (-0.02%) to 71.827% when pulling 6564a5eb6f180c572c4fd24648ddbd6bbf6269b8 on tpw/fix-point-style-protobuf into 875d624e0fda2c80d7966404753fce6eb1edbb2b on master.


Show max sell price and total

Suggestion for showing total cost of the cargo based on max sell price (like on inara.cz)

@JohnTheCoolingFan Checking to make sure my implementation is similar to what you have in mind, do you want the max sell price to be displayed to the right of each commodity per unit (like average is right now), beneath the cargo list as a cumulative max value of all current cargo, both, or something else?

Added in 235d12b8036965af78a46700c82cc8afa8df3767.  Thanks for the suggestion!


Send context on callbacks - protect from side effects

The transition callbacks should receive and instance of the fsm event handler and the state from/to:
function onOpen(eh: EventHandler, from: State, to: State): Promise<void> {

    console.log("onOpen...");
    return eh.dispatch(Events.openComplete);
}

With your current code I would not be able to execute 2 instances of the same FSM, which is a problem if for example you want to use an FSM per user (to handle his user state).

the trick is to bind the callbacks to the object. see how callbacks are bound in src_test_\stateMachine.test.ts

the trick is to bind the callbacks to the object. see how callbacks are bound in src_test_\stateMachine.test.ts


White background fills entire output cell for Matplotlib plots

When I make a simple matplotlib plot, there is a white background covering the entire output cell area, which is pretty jarring in dark themes. I would expect it to only be white in the area immediately behind the figure. Also, I'm not sure why there is a small dark cutout in the lower right corner of the white background.

I'm guessing that it's built this way because by default, the figure background is transparent. However, even if the plot image was partially transparent, I would not expect that to affect any parts of the screen beside the image itself, so this might be undesirable behavior.
Steps to reproduce:

Create new blank notebook
Enter code in cell:

import matplotlib.pyplot as plt
plt.figure()
plt.plot([1,2], [1,2])
plt.show()

Run cell

Environment data

VS Code version: 1.61.0-insider
Jupyter Extension version: v2021.9.1001208778
Python Extension version: v2021.10.1210725981-dev
OS: Windows 10
Python 3.9.7
Type of virtual environment used: none
Jupyter server running: Local
No other extensions running besides Python + Jupyter

THanks for filing this issue.
I think we need to fix this, have had similar reports in the past as well https://github.com/microsoft/vscode-jupyter/issues/7066

The problem here is that to fix this we would likely need to parse the matplotlib html to tweak what the output looks like. Then on top of that we have VS Code themes to worry about. We're not likely going to make a change here in the near future.

Perhaps an option to turn it off then, with it on by default?

Re-opening to discuss this:
We can apply the background to the image element instead of the container DIV, that would solve the issue

I think that would be an ideal solution. Thank you!

Verified, but buttons are floating off on the right now, I'll open a new issue for that seems a bit off to me:

Hi,
I'm sorry if this isn't the right place, but I seem to have almost the exact opposite error!
My preference would be to have the figure of the plot all in white so the axis labels can be seen against a white background, like in IanMatthew's display.
I've tried all variants of the settings but I can't stop the overall figure from being transparent - am I missing something ?

@charlieh108 Can you post what you see in the </> button next to the chart? Looks like you might not be using the correct renderer here.

Hi @IanMatthewHuff  - thanks for responding !
I get the issue with both the svg+xml and png renderer, as so

!

In this photo I am using the atom colour theme, but I've had this problem using the vscode default, so I'm certain this is not the cause of the problem.
Thanks for the help !

@charlieh108 Looks like you might be missing the Jupyter renderers extension. VS Code separates execution from display so different extensions can provide different output renderings. The Jupyter renderers extension is supposed to install with the main Jupyter extension, but it looks like you are missing it. Try installing that, it should provide the right views:

You can see in my second image that it's using the renderer from that extension.

It was exactly that - thanks so much !

run the magic command before plotting your figure:
%matplotlib inline
You might need to run your plot code twice to see the effect of the above magic command.

I don't know what exactly it does, but there is a vscode setting which may or may not help in this instance:
 "jupyter.themeMatplotlibPlots": true,


allow injection of custom node store. fixes #14

Thanks, @justinedelson! great idea. I added a couple comments.


docs: add warning about using window's volumes with registry

Signed-off-by: Mary Anthony<EMAIL_ADDRESS>

You have a point. I've had company's happy to do this; depends on the editor.  You take this tack which I'm happy to do.

http://goo.gl/Wosd8c

LGTM!


Color in output

Awesome package, thanks. Any chance we can enable color in the aider output inside emacs aswell? Currently it's a bit harder to read the code snippets. Presumably we'd need to forward the color codes that aider already outputs correctly to emacs?

Yeah agree that the current comint based solution lost the code color from
aider output. i do try ansi color based change for the coming aider session
buffer, suggested by ChatGPT, doesn’t work. ChatGPT said aider might used
some more advanced coloring way.
On Mon, Oct 14, 2024 at 3:25 PM RomeoV @.***> wrote:

Awesome package, thanks. Any chance we can enable color in the aider
output inside emacs aswell? Currently it's a bit harder to read the code
snippets. Presumably we'd need to forward the color codes that aider
already outputs correctly to emacs?
—
Reply to this email directly, view it on GitHub
https://github.com/tninja/aider.el/issues/9, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFOPAMYQI4OURPUJXC3LSDZ3RAEFAVCNFSM6AAAAABP53JMU2VHI2DSMVQWIX3LMV43ASLTON2WKOZSGU4DOMJYGAYTSNI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Aider does not send the color codes to the shell because we are telling it this a "dumb" terminal via comint-terminfo-terminal. If you set it to something else like "eterm-color" (an Emacs terminal) or "xterm-color" it will colorize the output, but it also screws things up because it sends a lot of other commands that comint doesn't understand and also expects to get to receive "cursor position requests" from Emacs (which it doesn't send). I am not sure if there is a value that gives just color output with breaking other things. Aider uses prompt-toolkit to handle the coloring and it checks for a dumb terminal here. See also prompt-toolkit/python-prompt-toolkit/#1032. Unfortunately that setting both disables interactive editing and colors, so I don't see a way to only enable colors.

Trying to use the major mode of buffer which start aider, to highlight the comint aider buffer session. Tested elisp and python and it seems working

merged in https://github.com/tninja/aider.el/pull/21

Indeed it does work for elisp, but for some reason it does not for Elixir for me. I'm using elixir-ts-mode and prog-mode is its parent. I wonder if it's maybe because of tree-sitter somehow.

I asked LLM for aider.el, using this question (with aider.el file added): inside aider-run-aider, i use aider--inherit-source-highlight, to get code highlight from current major-mode. however, it doesn't work for some particular major mode, eg. elixir-ts-mode. Do you know why?
It do give me some example code. part of it looks like:
        (if source-tree-sitter
            (progn
              (require 'tree-sitter)
              (require 'tree-sitter-langs)
              (tree-sitter-mode)
              (tree-sitter-hl-mode))

But I am not familiar with elixir-ts-mode. wonder if you want to take a try and see if it works?

Thanks for the reply. I tried this and tried to iterate on top of that, but only got strings to be colored. I will try some more if I have time and some other ideas to try.

I don’t use elixir-ts-mode but if I had to guess it is because tree sitter parsers tend to be quite strict. Does it work better using the non-tree sister mode?


[#2518] feat(PyClient): Integrate pytest with Gradle to run python test with CICD

What changes were proposed in this pull request?

Use Gradle python plugin to manage python module
Automatic download conda and create python env in gravitino/.gradle/python directory
Use multiple python version 3.8, 3.9, 3.10,and 3.11 test ITs in Github Action
Support gradlew clean & gradlew build & gradlew :client:client-python:test -PpythonVersion=3.8

Why are the changes needed?
Use Gradle manager and development Python client module.
Fix: #2518
Does this PR introduce any user-facing change?

Use pythonVersion Gradle parameter setting Python version to compile and test Python client, gradlew :client:client-python:test -PpythonVersion=3.8
Use skipPythonITs Gradle parameter to control if skip Python test cases.

How was this patch tested?
CI Passed.

hi @shaofengshi  @zhaoyongjie @coolderli Please help me review this PR, Thanks!

Hi Xun, I have two comments:

is the "python-gradle-miniforge-plugin" a formal gradle plugin? The project seems not under maintained, the last commit is 2 years ago;
conda seems more heavy than venv; I'm not sure whether it can switch to venv;

For other parts, it looks good to me

hi @shaofengshi

is the "python-gradle-miniforge-plugin" a formal gradle plugin? The project seems not under maintained, the last commit is 2 years ago;

Yes, Because Python Gradle plugin is very few, I didn't found better  than to python-gradle-miniforge-plugin.
I think python-gradle-miniforge-plugin fulfill our needs.

conda seems more heavy than venv; I'm not sure whether it can switch to venv;

I didn't found a Python Gradle plugin would to support venv.
I think we could use it first.

hi @shaofengshi

is the "python-gradle-miniforge-plugin" a formal gradle plugin? The project seems not under maintained, the last commit is 2 years ago;

Yes, Because Python Gradle plugin is very few, I didn't found better than to python-gradle-miniforge-plugin. I think python-gradle-miniforge-plugin fulfill our needs.

conda seems more heavy than venv; I'm not sure whether it can switch to venv;

I didn't found a Python Gradle plugin would to support venv.
I think we could use it first.

OK, we can go with this first, and refine later.

Is this python environment removable, where is it installed?

Is this python environment removable, where is it installed?这个python环境是可移动的，安装在哪里？

Automatic download conda and create python env in gravitino/.gradle/python directory

@shaofengshi @zhaoyongjie @coolderli  Please help me review this PR, Thanks.

@xunliu Hi, I left two comments. Overall, LGTM. Thanks.

LGTM


add lodash to recommended preset

So users won't need to add it by themselves

Thanks for contributing! Published in v2.6.2.

@ganimomer what are your thoughts on https://github.com/wix/eslint-plugin-lodash/issues/180 ?


Social Fix

Fixes the broken [social] parameters in config.toml by placing them under [params], and fixes references to those parameters.

Hello @patrickderitter,
Thank you for submitting this , but just wondering why? Personally, I think it's more organised the way it's done...

Hi @nurlansu, thanks for taking the time to look at the PRs I sent in.
I made this change because you cannot (at least, not anymore) reference the [social] parameters using {{ .Site.Social.Email }}. This was the simplest way to fix that, but for organisation sake, I'll push some changes to them so that {{ .Site.Params.Social.Email }} can be used.

Hello @patrickderitter,
Thank you very much for this and sorry for the delayed response. I didn't know that [social] parameters were broken. Merging.
Cheers,
Nurlan S.


Support ignoretext for the translator

Support strings to ignore in the translation.
Ex: May want to not translate certain substrings in a localization like:
Company Name
Person Name
(swiftVar), %d, %@, etc...
This could also potentially be accomplished by allowing the developer to extend the before/after parse of the translation. Ex: give a callback function for the string before it is translated for mutation, and/or a callback function for the string after it is translated.
Thanks for any help!!!

I agree with @thejeff77 and @Jeehut!
It would be cool to add Google Translate support, because I think Google Translate is better than Microsoft Translate.
And there is one lifehack. It would be nice to add integration with Google Apps Script, because there is a free unlimited Google Translate. Here is an example article to understand what I mean.

I've just added the "good first issue" label, although I think I would prefer it a separate issue was created for each of the above mentioned features. The title here isn't very discoverable. @ownmas Could you create a separate issue for the Google Translate feature so we can prioritize and discuss it separately?
And as I said, I would be up to review a PR anytime, I just invested quite some time to fix several bugs recently and I don't plan to make any bigger changes myself before summer this year.


Package Requested

[X] I would like someone else to build the manifest.
[ ] I would like help so I can submit the manifest.
[X] I have performed a search and couldn't find this package.
[ ] I have checked that the installer for this package is not a standalone or portable executable or a .zip file
[ ] I think there is a new version available and I have provided the URL.

Please provide the following information
Publisher: microsoft
Publisher Url: https://www.microsoft.com/en-us/download/details.aspx?id=35
Package Name: directx
Description:
Package Version:
Installer URL: https://download.microsoft.com/download/1/7/1/1718CCC4-6315-4D8E-9543-8E28A4E18C4C/dxwebsetup.exe
Current Package Identifier (for new versions):

I have created the manifest and attempting to merge now
#63653

This merge is complete. To see the package type winget search directx. To install winget install Microsoft.DirectX

cc @denelon: Would you be able to close this? It's merged: https://github.com/microsoft/winget-pkgs/blob/635709300afe2c49bd1c9985c832233fcb5c2beb/manifests/m/Microsoft/DirectX/9.29.1974.1/Microsoft.DirectX.installer.yaml


scripts: don't even attempt to use Nix in non---nix modes

Interpret non-Nix modes as "don't use Nix at all, even for nix-shell", which restores complete independence from Nix in --cabal mode.

This PR does not result in breaking changes to upstream dependencies.

Checklist

[x] This PR contains all the work required to resolve the linked issue.

[ ] The work contained has sufficient documentation to describe what it does and how to do it.

[x] The work has sufficient tests and/or testing.

[x] I have committed clear and descriptive commits. Be considerate as somebody else will have to read these.

[x] I have added the appropriate labels to this PR.

bors r+


Adding axial expansion "support" for unshaped components

Description
In some cases, users may specify components within a block that meet the three criteria to check for axial linkage but do not have the class methods getBoundingCircleOuterDiameter and/or getCircleInnerDiameter -- and it doesn't make physical sense to add them. An example of this is for unshaped components.
This change adds a try/except clause to account for these types of component classes by addressing a runLog.warning and just not linking the components in question, rather than crashing.
Note: This could be perceived risky(ish) as some classes (new or existing) that don't have getBoundingCircleOuterDiameter and/or getCircleInnerDiameter but maybe should. This addition would gloss over this and just issue a runLog.warning that could be missed by a user, generating potentially unexpected results.

Checklist

[x] The code is understandable and maintainable to people beyond the author.
[x] Tests have been added/updated to verify that the new or changed code works.
[x] There is no commented out code in this PR.
[x] The commit message follows good practices.
[x] All docstrings are still up-to-date with these changes.

Will update the "implicitly" included list in https://github.com/terrapower/armi/pull/790 once this gets merged.

@john-science you're up when you have some time!

@john-science you're up when you have some time!

While the need behind the feature is quite interesting, the code is very simple.  I requested a minor addition to the testing, but then it looks good to me.


Error: undeclared field: 'reqPing'

Trying to run any of the examples (including the snippet in the README.md) results in this error:
~\.nimble\pkgs\discord-1.5.0\websocket\shared.nim(151, 49) template/generic instantiation from here
~\.nimble\pkgs\discord-1.5.0\websocket\shared.nim(180, 14) Error: undeclared field: 'reqPing'

Nim version used (nim -v):
Nim Compiler Version 0.17.0 (2017-05-18) [Windows: amd64]
Copyright (c) 2006-2017 by Andreas Rumpf

git hash: d3f0f87e81e262ee25fd528aae5b6db0bdf78d5e
active boot switches: -d:release

I've found the problem, gonna figure out a fix.

Works like a charm now, thanks.


Code scanning alerts on the PR’s Conversation tab (GitHub.com)

Summary
Code scanning will post security alerts to the PR’s conversation tab in a new way: a PR review composed of annotations. Users will be able to comment on any individual alert (annotation) to discuss it.
Intended Outcome
Currently, code scanning alerts are posted as PR checks and are sometimes missed by developers. At the same time, alerts can't be discussed in the PR itself. Tight integration in the conversation tab will address both points.
How will it work?
Code scanning uses checks (annotations) to communicate that an analysis is in progress and block the PR when an alert is flagged up. This behavior will remain unchanged.
In addition to these checks, the code scanning annotations will appear on the PR’s conversation tab as a PR review.

this has shipped! 🚢 https://github.blog/changelog/2022-06-02-users-can-view-and-comment-on-code-scanning-alerts-on-the-conversation-tab-in-a-pull-request/
closing out this issue.


[WIP] Filter pushdown

Filter pushdown through layouts
chunked
columns

Comments offline:

there's just one method that takes expression
add method from #979 for planning - call it sketch
something about shortcircuiting siblings

batching should be handled at range generation time. We shouldn't try to split read ranges into batches further

revived as #1124


Gitlab -> Lambda Event Bridge Test

I'm going to go ahead and close this considering how old the issue is.


Add Item dialog does not close

When I add an item to a board, the dialog is not closed when click ADD. All fields become gray and the item appears at the board. The only thing that works is x. After doing it a few times the colour picker of the dialog does not work any more.
Version TaskBoard-master

Open F12 dev tools and let me know if any errors appear when you add an item.

I found the same issue. It seems to be due to the mail config in api/mainConfig.php having some default/bogus SMTP settings. api/mailFactory.php creates a mail object and always uses these values (perhaps, by default, it shouldn't!).  I resolved the issue by commenting out the smtp setup lines in api/mailFactory.php:createMailObject.
...
/*
$mail->isSMTP();
$mail->Host = $MAIL_HOST;
$mail->SMTPAuth = true;
$mail->Username = $MAIL_USERNAME;
$mail->Password = $MAIL_PASSWORD;
$mail->SMTPSecure = $MAIL_SMTPSECURE;
$mail->Port = $MAIL_PORT;
*/
...

See also #246 #234

That was contributed code, which I didn't really delve into (my bad).
@mgkeeley Is there any reason to have that commented out block of code? If not, want to send a PR against dev?

Hi, no, I just commented it out as a minimal workaround to "get it working", and so others could do the same if they wanted to. I don't really have time to put together a better solution sorry. Maybe by default the properties in mailConfig should be commented out, and the mailFactory checks to see if those SMTP vars are set or not before configuring the mail item as sending via SMTP.


Azure DevOps Pipeline - MS hosted Agent, downloaded 'TabularEditor.exe' is not recognized

!!! For Tabular Editor 3 questions, go here --> https://github.com/TabularEditor/TabularEditor3/issues !!!
What are you trying to achieve?
Running the https://tabulareditor.github.io/2019/10/08/DevOps3.html example for a BIM build to AAS... I'm getting the  TabularEditor.exe not found issue.
Have read some similar issues/responses on that article discussion thread and this github... but no solution.
MS Hosted Agent --- Agent Pool = Azure Pipelines,  windows
We have other build and release pipelines running against different resources, but trying to get this to work with tabular editor.
Details below...
thanks!   Tony
======
INPUT SCRIPTS
repo root
C:\Users.....\source\repos\REPO_NAME\bids\Models\L2L\Model.bim
hence path is  \bids\Models\L2L\Model.bim

DOWNLOAD SCRIPT Inline

# Download URL for Tabular Editor portable:
$TabularEditorUrl = "https://github.com/otykier/TabularEditor/releases/download/2.16.7/TabularEditor.Portable.zip"

# Download destination (root of PowerShell script execution path):
$DownloadDestination = join-path (get-location) "TabularEditor.zip"

# Download from GitHub:
Invoke-WebRequest -Uri $TabularEditorUrl -OutFile $DownloadDestination

# Unzip Tabular Editor portable, and then delete the zip file:
Expand-Archive -Path $DownloadDestination -DestinationPath (get-location).Path
Remove-Item $DownloadDestination

DEPLOY SCRIPT -- inline

echo 'Build.SourcesDirectory = ' , $(Build.SourcesDirectory)

TabularEditor.exe "$(Build.SourcesDirectory)\bids\Models\L2L\Model.bim" -D "$(ASConnectionString)" "$(ASDatabaseName)" -O -C -P -R -M -W -E -V

OUTPUTS

OUTPUT Download task run

Starting: Download Tabular Editor
Generating script.
========================== Starting Command Output ===========================
"C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe" -NoLogo -NoProfile -NonInteractive -ExecutionPolicy Unrestricted -Command ". 'D:\a\_temp\527a32b5-447e-4d49-9544-e658ef7fd577.ps1'"
Finishing: Download Tabular Editor

OUTPUT DEPLOY SCRIPT

Starting: Deploy AS model using Tabular Editor
Generating script.
========================== Starting Command Output
"C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe" -NoLogo -NoProfile -NonInteractive -ExecutionPolicy Unrestricted -Command ". 'D:\a\_temp\545ce3c4-3433-41dd-9861-7cee4d877814.ps1'"
Build.SourcesDirectory =
D:\a\1\s
TabularEditor.exe : The term 'TabularEditor.exe' is not recognized as the name of a cmdlet, function, script file, or
operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try
again.
At D:\a\_temp\545ce3c4-3433-41dd-9861-7cee4d877814.ps1:5 char:1
+ TabularEditor.exe "D:\a\1\s\bids\Models\L2L\Model.bim" -D "Provider=M ...
+ ~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (TabularEditor.exe:String) [], ParentContainsErrorRecordException
    + FullyQualifiedErrorId : CommandNotFoundException

I recommend using a Command Line Task in step 2, instead of PowerShell, as PowerShell doesn't handle the output of TabularEditor.exe properly.
Also, you may need to specify the Working Directory in step 2, to ensure that it points to the location where Tabular Editor was downloaded and extracted in the previous step.

Command Line Step worked!
I did not have to specify the working directory path.
(can also deploy from TE3 directly).
Thanks, Tony
Still have to address the (power query data source related) warning items, but I knew about them and are addressing in another TE3 post.
##[warning]Information: Unprocessed object: partition 'Partition' on table 'XXX' (NoData)
##[warning]Information: Unprocessed object: column 'YYY'[ServerSiteSlicer] (CalculationNeeded)


SBT support

Description

This is https://github.com/pulumi/pulumi-java/pull/729 plus a few tiny fixes to make sure CI runs the tests.
Fixes # (issue)
Checklist

[ ] I have added tests that prove my fix is effective or that my feature works

[ ] I have updated the CHANGELOG-PENDING file with my change

[ ] Yes, there are changes in this PR that warrants bumping the Pulumi Service API version

:tada: :heart:

Can we expect a new release containing this fix anytime soon?

should be in v0.5.4 that was just released

So pulumi-language-java 0.5.4 is released, but it will be the "default" in Pulumi only on the next major Pulumi release. We just had one and there is a 2-week cadence. You'd need to grab this release by hand and put it in PATH to use it before the next Pulumi major rel.

Btw what would be a good example to test the com.pulumi.bootstrap.internal.Main packages functionality on?

com.pulumi.bootstrap.internal.Main is used by the language host to determine what pulumi plugins need to be downloaded on the CLI side.

I see. And is there an easy way how can I (or somebody more experienced with Pulumi) verify that it works for with the recently added Scala/sbt?

In general, if you add a pulumi provider dependency, e.g. the random provider, and the language host does not complain and the example works correctly, then it works.
There is also an undocumented internal feature.
If you modify the tests/examples/random/Pulumi.yaml to look like this:
name: random
description: An example of using pulumi-random with Java
runtime:
  name: java
  options:
    logtostderr: true
    v: 5

then the options part gets passed to the pulumi-language-java binary and after running pulumi up you can see that the functionality is working as expected:
Diagnostics:
  pulumi:pulumi:Stack (random-java-random):
    I0817 14:54:46.116815   94080 main.go:138] GetRequiredPlugins: program=.
    I0817 14:54:46.116843   94080 main.go:176] GetRequiredPlugins: Determining Pulumi plugins
    I0817 14:54:46.116854   94080 main.go:211] Language host launching process:  /usr/local/bin/mvn -Dorg.slf4j.simpleLogger.logFile=System.err --no-transfer-progress compile exec:java -DmainClass=com.pulumi.bootstrap.internal.Main -DmainArgs=packages
    I0817 14:54:47.938012   94080 main.go:240] '/usr/local/bin/mvn -Dorg.slf4j.simpleLogger.logFile=System.err --no-transfer-progress compile exec:java -DmainClass=com.pulumi.bootstrap.internal.Main -DmainArgs=packages' completed successfully
    I0817 14:54:47.938051   94080 main.go:189] GetRequiredPlugins: bootstrap raw output=[{"resource":true,"name":"random","version":"4.6.0"}]
    I0817 14:54:47.938147   94080 main.go:151] GetRequiredPlugins: Determining plugin dependency: random, 4.6.0
    I0817 14:54:47.938157   94080 main.go:168] GetRequiredPlugins: plugins=[name:"random" kind:"resource" version:"4.6.0"]
    I0817 14:54:48.001341   94080 main.go:248] Run: program=.
    I0817 14:54:48.001356   94080 main.go:267] Language host launching process:  /usr/local/bin/mvn --no-transfer-progress compile exec:java

here is the raw JSON output from the com.pulumi.bootstrap.internal.Main:
GetRequiredPlugins: bootstrap raw output=[{"resource":true,"name":"random","version":"4.6.0"}]

Does it work? how to get it to work?
t>pulumi up
Please choose a stack, or create a new one: zioplay2
Previewing update (zioplay2)

View Live: https://app.pulumi.com/PhilAndrew/minimalsbt/zioplay2/previews/fc292e91-4d28-490c-9a03-8dc76316a297

     Type                 Name                 Plan     Info
     pulumi:pulumi:Stack  minimalsbt-zioplay2           2 messages

Diagnostics:
  pulumi:pulumi:Stack (minimalsbt-zioplay2):
    I0820 17:56:13.120697    9548 main.go:150] GetRequiredPlugins: program=.
    I0820 17:56:13.120697    9548 main.go:188] GetRequiredPlugins: Determining Pulumi plugins

error: failed to discover plugin requirements: language host could not determine Pulumi packages: Failed to configure executor, tried: jar, maven, gradle, jbang

>pulumi version
v3.38.0

>

@PhilAndrew

tried: jar, maven, gradle, jbang

You're almost surely not using the newest version of pulumi-java -- the current version of Pulumi doesn't have the newest pulumi-java, see https://github.com/pulumi/pulumi-java/pull/791#issuecomment-1213197671


Adds Global Header CTA, initial Header/Mega Menu

Adds Submit a Complaint CTA, initial Header/Mega Menu
Additions

Atomic Submit a Complaint CTA.
Initial atomic Header (duplicate of https://github.com/cfpb/cfgov-refresh/pull/1342).
Initial atomic Mega Menu.

Testing

Visit the site, resize window, expand primary nav hamburger menu. Submit a Complaint appears in upper-right in header and bottom of mega menu primary nav.

Review

@KimberlyMunoz
@sebworks
@jimmynotjim

Screenshots
Desktop size:

Mobile size:

Instead of nesting, it seems it might be better to add modifiers to the CTA for their location specific styles. That'd help avoid having to reset styles from the heading nested CTA.

@jimmynotjim Addressed your comments and updated the name. Ready for a review again.

There's weird spacing on the mobile version between the beta banner and the rest of the page and witht the Submit a Complaint link.

@KimberlyMunoz please run gulp build

Same issue.

Pulled in another commit I missed and the gap is still there. Plus the focus should be on the first element of the menu when it opens. The submit a complaint button is completely skipped over for mobile screenreaders.

Gap fixed in https://github.com/cfpb/cfgov-refresh/commit/0668639b8faf7968628107ca15ce752e6e56b123

:+1: You might want to add the focus issue on to the mega-menu issue on JIRA as something to fix.

We discussed leaving the incorrect focus for now until this gets user tested just for the location change in Sprint Review, but ditto @KimberlyMunoz's comment about making sure it's documented as a future todo.

:+1:


Flutter Web: Canvas renderer is not usable offline

Hi,
For Flutter web you can choose the beautiful canvas renderer. Everything looks good and the same in each browsere.
But this renderer is loaded from a specific server on the internet if you open a flutter website.
But in some cases your flutter website is used in an intranet with no connection to the server that provides canvas kit.
In this case, the site is just blank. No error displayed to the user.
My points were as follows:

Could you make an option to host canvaskit with your site? So that if you build your flutter website ALL necessary files will be placed under build/web.
There is an option to set renderer to auto. This makes flutter use the html renderer on mobile. But I would prefer if the html renderer would be used automatically if on mobile, OR on desktop with no connection to the server to download canvaskit.
If the renderer is set to canvaskit and it cannot be loaded, there should be an understandable message that says, no connection to the internet or something.

Hi @Wissperwind
I see there's an open issue addressing the case you described.
Please follow up on that issue, Closing as duplicate of https://github.com/flutter/flutter/issues/70101
If you disagree, please write in the comments and I will reopen it.
Thank you


Stellar 0.6 support

This PR adds basic support for stellar 0.6, updating the XDR definitions and resolving test failures that were introduced.
Notably, a very subtle bug was fixed around the autoloading of the generated XDR constants.  Take note of the change to how we load the VERSION constant in the gemspec file.  I leave these notes here for history:
The symptoms: Stellar::Asset was missing many class methods that caused load errors
The issue: require_paths was not being obeyed.  lib was being included into the $LOAD_PATH before generated.
The solution:  don't manually modify the $LOAD_PATH in the gemspec

Note:  this PR unfortunately removes jruby from the CI suite.  We don't have the resources to properly support it for now, but if someone else would like to help us, I'd be happy to coordinate contributions to re-enable jruby testing.


Enable a blank greeting

Enable using a blank greeting.
Fixes #588 - "Hi mate, how are you?" cannot be changed and persisted

I'll make the file version change in another PR.


change logo

Update webpage logo with grayscale Spotify logo

Hey @jessej-samuel.
I have made a PR addressing this issue.
You can check it out https://github.com/jessej-samuel/spotipy/pull/18
Since I just replaced the favicon.ico file, I made the PR directly to the master branch.


How to turn your photo into cartoon using python

Article checklist
ATTENTION: In your PR - add the ISSUE (topic form) # that this PR corresponds to. You can do this by also adding a comment that states "This fixes #(enter your Idea Content Form #)" to link and close your Idea Content (issue) when the PR is merged.
NOTE: (Please ensure that you have only one open issue + linked pull request at a time. This will ensure that we complete the article in a timely manner from inception to publishing.)
If you have not already please go over our Resources Page for more tips and tricks on how to improve our overall technical writing.
Preliminary Checks - Formatting and Structure

[x] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.
[x] Does your article follow any of the suggested structure formats? - see example formats
[x] Is your article properly formatted in Markdown? - see Github Markdown guide
[x] Have you used the correct folder and file structure? - see Contribution Guidelines
[x] Is your article over 750 words? - Tool: Wordcounter.net
[x] Have you used our preferred heading structure? - H3 (###) and up
[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.
[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.
[x] Have you included a hero image that is 600x400 pixels and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images
[x] In your PR - add a comment that states "This fixes #(enter your Idea Content Form #)" to link and close your Idea Content (issue) when the PR is merged.

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid -  Grammarly
[x] Please place your article through a 3rd party plagiarism checker? We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[x] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide

Contribution guidelines
For first-time contributors and for more details, see Contributing Guidelines
This fixes #3047

@kayere the article should be good to review now

This fixes #3266

Hello @KanizoRGB, Thanks for sending this over. I will be reviewing this article.

@hectorkambow this is my article. I would like to be a peer reviewer

@Kaperskyguru - I had already started reviewing this article. Please make sure to replace my peer reviewer attribution with yours. Thanks.

@Kaperskyguru No progress so far?

I am working on it locally. Once I am done, I will push updates. Thanks.

@Kaperskyguru this review is taking longer than expected

@KanizoRGB
Yes. I know. Because I take my time to review properly.
I am done reviewing the article.
It's great content and well explained seems like it was reviewed before now.

Yeah @Kayere had already reviewed it. Can we move to the next label please?

plagiarism check done

@hector, I am done with the Tech Review. I just couldn't add Labels too. We had a chat on Slack.

@hector, I am done with the Tech Review. I just couldn't add Labels too. We had a chat on Slack.

@Kaperskyguru
Thank you Sir 👍

Overall great work Gents 👍
Please be sure to go through my changes to improve future articles and reviews @KanizoRGB
cc @Kaperskyguru

@KanizoRGB
Great article. While I was checking it out, noticed that your Colab notebook needs to be made public.
@Kaperskyguru As a peer-reviewer, we must ensure that all the links are working good.
Thanks.

@KanizoRGB
Great article. While I was checking it out, noticed that your Colab notebook needs to be made public.
@Kaperskyguru As a peer-reviewer, we must ensure that all the links are working good.
Thanks.

Great. This will be my top consideration.

@KanizoRGB
Great article. While I was checking it out, noticed that your Colab notebook needs to be made public.
@Kaperskyguru As a peer-reviewer, we must ensure that all the links are working good.
Thanks.

@srishilesh I'll make it public


flutter doctor cannot find flutter+dart plugin installed in android studio

Steps to Reproduce

flutter doctor cannot find flutter+dart plugin installed in android studio but changing channel to dev or master make it working fine
...
...

Logs

Logs

C:\flutter>flutter doctor -v
[√] Flutter (Channel stable, 1.22.6, on Microsoft Windows [Version 10.0.19042.746], locale en-IN)
• Flutter version 1.22.6 at C:\flutter
• Framework revision 9b2d32b605 (4 days ago), 2021-01-22 14:36:39 -0800
• Engine revision 2f0af37152
• Dart version 2.10.5
[√] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
• Android SDK at C:\AndroidSDK
• Platform android-29, build-tools 29.0.3
• ANDROID_HOME = C:\AndroidSDK
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
• All Android licenses accepted.
[!] Android Studio (version 4.1.0)
• Android Studio at C:\Program Files\Android\Android Studio
X Flutter plugin not installed; this adds Flutter specific functionality.
X Dart plugin not installed; this adds Dart specific functionality.
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
[!] Connected device
! No devices available

@testexample2016
This issue is already fixed and merged into master a while back, https://github.com/flutter/flutter/issues/67986.
Although you see those warnings in your flutter doctor -v log, the warnings should not matter and you should be able to continue using the plugins as usual.
You may try this solution to see if it works for you, https://github.com/flutter/flutter/issues/68429#issuecomment-740430602
Closing this as already fixed and there's no need to track.

@testexample2016
This issue is already fixed and merged into master a while back, https://github.com/flutter/flutter/issues/67986.
Although you see those warnings in your flutter doctor -v log, the warnings should not matter and you should be able to continue using the plugins as usual.
You may try this solution to see if it works for you, https://github.com/flutter/flutter/issues/68429#issuecomment-740430602
Closing this as already fixed and there's no need to track.

@testexample2016
This issue is already fixed and merged into master a while back, #67986.
Although you see those warnings in your flutter doctor -v log, the warnings should not matter and you should be able to continue using the plugins as usual.
You may try this solution to see if it works for you, #68429 (comment)
Closing this as already fixed and there's no need to track.

this issue still persist for Windows User