[HUDI-633] limit archive file block size by number of bytes

What is the purpose of the pull request
With large clean files, archival process results in OOM. See HUDI-633. Limit archive file block size by number of bytes
Brief change log

Add option to limit  archival batch size by number of bytes in block in addition to maximum number of records allowed in a batch
This does not prevent OOM if a single record is larger than jvm heap size.
Note that in worst case, a single instant details can take up entire block. This likely has higher metadata overhead. But I think marginal increase in storage is acceptable for metadata.

Verify this pull request
This change added tests and can be verified as follows:

Run TestHoodieCommitArchiveLog#testArchiveTableWithLargeCleanFiles
Verified large clean file that caused OOM can be archived with specified config.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@satishkotha left some comments, I'm also assuming you will address the sizeToRead in a later PR ?
yes, that one is more challenging requires more elaborate change. If we can find a way to make these .clean files not get too large, that would be ideal. @nbalajee made some suggestions too. This will likely be another change.

@n3nash please take a look and let me know if you think this change is not needed

@satishkotha I feel we can table this PR for now, once we have a concrete plan on cleaner files being limited, lets discuss this PR at that time, moving to WIP for now..


eth locked in a pool after failed transaction

<!--
  Is this a bug report?
  If so, go back and select the "Bug report" option or your issue WILL be closed.
--!>
Please help, I have a bit of ETH locked.
Tx: https://etherscan.io/tx/0x14f95f9593118b47c70ea5d36dd8c490bb6572f37fc3e3ccd174f500cac32d45

Thanks in advance.

@brunojanvier77 It looks like you have tried to  If you got to https://thorchain.tools/ to enter your THORChain address thor162r62wymneyycvkqjvq9u7z2y4dj2exzl4szqg, you will see pending your pending ETH asset. Same result by checking pool/ETH.ETH/liquidity_provider endpoint.
 Then you add some RUNE to trigger deposit again OR withdraw your pending value. For more questions about to recover your asset there, go to Discord channel of Thorswap.


Correct to appropriate error message

Error message when ADDRESS_KEY(number-address) is missing is not appropriate.
Please fix it.
https://github.com/Finschia/cosmwasm/blob/03abb0871ca5cfe8b874561795bc59d12562002f/contracts/call-number/src/contract.rs#L55-L59
https://github.com/Finschia/cosmwasm/blob/03abb0871ca5cfe8b874561795bc59d12562002f/contracts/call-number/src/error.rs#L10-L11

closed via #321


Implement aggregate function "string_agg" and corresponding Julius clause

Implement aggregate function "string_agg" (as in Postgresql) ("listagg" in Oracle) and the corresponding Julius clause.

Implementation Hint:
https://github.com/nkarag/haskell-DBFunctor/blob/master/doc/JULIUS-TUTORIAL.md#ccaf

Implemented and tested! Merged into master branch.


Issue with k8s.io/zh/docs/setup/learning-environment/minikube/

This is a Bug Report

Problem:
Following command in section 2 is used to create a pod instead of creating a deployment.
kubectl run hello-minikube --image=k8s.gcr.io/echoserver:1.10 --port=8080
Proposed Solution:
Change it to be same with English page:
kubectl create deployment hello-minikube --image=k8s.gcr.io/echoserver:1.10
This command does not have --port parameter, need to add --port in the command kubectl expose deployment hello-minikube --type=NodePort.
Page to Update:
https://kubernetes.io/...

/language zh
/help

/assign

PR merged. We can close this issue.

/close


[WIP/backport/1.24] release: Cleanups/updates

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

waiting for #26159


search and replace advances too quickly

This isn't really a bug report but a request for a slight modification. I hope this is the right place to ask!
First off, I love Sublime's regex search and replace! That said, regex is never a bulletproof way to refactor and it's wise to not use "replace all" but rather go one and I time, carefully monitoring the effect of each replace.
But Sublime will perform the replace and then automatically scroll down to the next match in what feels like 0.0000001 nanosecond. My mere mortal eyeballs and brain cannot register + assess what has taken place in 0.0000001 nanosecond. I always have to hit undo to scroll back up but the change is undone before I can return to the spot.
Is there any way we can dial it down a bit for us humans?

Sorry, this is not the repository for the Sublime Text editor, just a plugin for interacting with Github from Sublime Text. You probably should post your suggestion in the forums: https://forum.sublimetext.com/


VoxelgridSampler broken when using with single_fit

Trying to use single_fit with the VoxelgridSampler outputs the following error message:

~/.local/lib/python3.6/site-packages/pyntcloud-0.1.0-py3.6.egg/pyntcloud/ransac/samplers.py in >get_sample(self)
67         # pick one point and get it's voxel index
68         idx = np.random.randint(0, len(self.points))
---> 69         voxel = self.voxelgrid.voxel_n[idx]
70
71         # get index of points inside that voxel and convert to probabilities
AttributeError: 'VoxelGrid' object has no attribute 'voxel_n'

Code that causes the error:
is_plane, model = single_fit(cloud.xyz, RansacPlane, VoxelgridSampler, return_model=True,                              model_kwargs={"max_dist": 0.2}, max_iterations=100,                              sampler_kwargs={"voxel_size": 0.1})
I assume this is because it never calls the compute() member function, however I'm not sure how to fix it properly.

Thanks for pointing out this bug. As you pointed, self.voxelgrid.compute is not being called in VoxelgridSampler.init

Hola @cosama!. It took me a while to found time to fix this (#188) and I ended up refactoring a lot of sutff to make the structures more usable from outside the PyntCloud class.
In theory (works for my and for some simple test), you can run:
is_plane, model = single_fit(
    points=cloud.xyz,
    model=RansacPlane,
    model_kwargs={"max_dist": 0.2},
    return_model=True,
    sampler=VoxelgridRansacSampler,
    sampler_kwargs={"size_x": 0.1, "size_y": 0.1, "size_z": 0.1},
    max_iterations=100
)

And the result should be what you were trying to do when you faced the bug.


Add job for CI using Prism parser

Follow up rubocop/rubocop-ast#277

Before submitting the PR make sure the following are checked:

[x] Feature branch is up-to-date with main (if not - rebase it).
[x] Squashed related commits together.
[-] Added tests.
[-] Updated documentation.
[-] Added an entry to the CHANGELOG.md if the new code introduces user-observable changes.
[x] The build (bundle exec rake) passes (be sure to run this locally, since it may produce updated documentation that you will need to commit).

I’d feel a tad more confident if we ran rake default, not just the spec task
OK. I updated this PR.

If there is a problem, we can delete this job so I merge this PR.


Fix multi renderpass in cb

Did you run clang-format on those test changes? Columns look wide.
Otherwise,
LGTM :+1:

Updated to fix formatting.

Cherry-picked to master.


2023 HUD Fixes!

Please read the README.md File for more information about what's currently broken/buggy that I haven't bothered to fix. I apologize for releasing this "Fix" in such a broken state but it's now a little playable, I don't plan on working on it anymore so anyone can feel free to fix my issues!
Sorry!
-SkyceCream

excellent work, thanks!


chore(deps): upgrade runtime dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-runtime-dependencies-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2684

Questions ?
Please refer to the Backport tool documentation


Unable to install kubernetes cli

urlopen error [WinError 10060] A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond>
To Reproduce
az aks install-cli
Expected behavior
A clear and concise description of what you expected to happen.
Environment summary
Install Method (e.g. pip, interactive script, apt-get, Docker, MSI, edge build) / CLI version (az --version) / OS version / Shell Type (e.g. bash, cmd.exe, Bash on Windows)
Additional context
error message:
<urlopen error [WinError 10060] A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond>
Traceback (most recent call last):
File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\urllib\request.py", line 1318, in do_open
encode_chunked=req.has_header('Transfer-encoding'))
File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\http\client.py", line 1239, in request
self._send_request(method, url, body, headers, encode_chunked)
File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\http\client.py", line 1285, in _send_request
self.endheaders(body, encode_chunked=encode_chunked)
File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\http\client.py", line 1234, in endheaders
self._send_output(message_body, encode_chunked=encode_chunked)
File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\http\client.py", line 1026, in _send_output
self.send(msg)
File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\http\client.py", line 964, in send
self.connect()
File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\http\client.py", line 1392, in connect
super().connect()
File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\http\client.py", line 936, in connect
(self.host,self.port), self.timeout, self.source_address)
File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\socket.py", line 724, in create_connection
raise err
File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\socket.py", line 713, in create_connection
sock.connect(sa)
TimeoutError: [WinError 10060] A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-qvpg2q31\knack\knack\cli.py", line 206, in invoke
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-qvpg2q31\azure-cli-core\azure\cli\core\commands_init_.py", line 351, in execute
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-qvpg2q31\azure-cli-core\azure\cli\core\commands_init_.py", line 409, in run_jobs_serially
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-qvpg2q31\azure-cli-core\azure\cli\core\commands_init.py", line 402, in run_job
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-qvpg2q31\six\six.py", line 693, in reraise
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-qvpg2q31\azure-cli-core\azure\cli\core\commands_init.py", line 379, in run_job
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-qvpg2q31\azure-cli-core\azure\cli\core\commands_init.py", line 171, in call
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-qvpg2q31\azure-cli-core\azure\cli\core_init_.py", line 451, in default_command_handler
File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-install-qvpg2q31\azure-cli-acs\azure\cli\command_modules\acs\custom.py", line 302, in k8s_install_cli
File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\urllib\request.py", line 223, in urlopen
return opener.open(url, data, timeout)
File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\urllib\request.py", line 526, in open
response = self._open(req, data)
File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\urllib\request.py", line 544, in _open
'_open', req)
File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\urllib\request.py", line 504, in _call_chain
result = func(*args)
File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\urllib\request.py", line 1361, in https_open
context=self._context, check_hostname=self._check_hostname)
File "C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\lib\urllib\request.py", line 1320, in do_open
raise URLError(err)
urllib.error.URLError: <urlopen error [WinError 10060] A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond>
To open an issue, please visit: https://aka.ms/azcli/issues

This looks like a transient network error.
@tjprescott this issue should be closed.


Disable  incremental build

set export CARGO_INCREMENTAL=0 to disable incremental build

@busyjay @kennytm PTAL

We will use memory disk, it seems resolve the bug, so I close this pr

Memory disk can not solve this problem


feat: Add entities to sources

Lets try this again since I've added a PR in web app that defines the node id as _id

So the same .meshrc.yml is used in all environments.  Since there are no configurations for the staging and prod env vars, will we run into issues when deploying to those environments?  (I'm not sure if you tested what happens in sandbox if the .meshrc includes the entities service but does not set the entities URL env var).
Also, for this service, we've been configuring URLs using github environment variables / secrets, rather than setting them terraform.  could we switch to that pattern?  Unfortunately, that also means we need to update all of the deploy scripts to set the variables from Github (it might be time to create a reusable action for those things).


[WIP] pkg/obfuscate: Add Postgresql json operators to sql obfuscator

What does this PR do?
This PR adds support for Postgresql specific JSON and JSONB operators to the Obfuscate package.  It adds special handling in the tokenizer for multi-character json operators (#>, #>>, #-, @>, <@, ?|, ?&).
Motivation
This is in response to customer reports that json operators were causing queries to be truncated, because the # operators were being interpreted as comments in Postgres. e.g. select jsoncolumn #> '{field}' from foo results in select jsoncolumn after obfuscation.
Additional Notes
Possible Drawbacks / Trade-offs
Only #>>, #>, #- operators are causing obfuscation to truncate queries.  The other operators are not parsed correctly, but it only results in an additional space e.g. json < @ other_json.  It's incorrect but can still be understood by the user.  If we merge this PR query signatures for any query using these operators will change.
In the # operator case, this is acceptable, since the truncated queries don't make sense to the user.
If this trade-off isn't acceptable we can only update the parser to handle the # operators.
Describe how to test/QA your changes
Collect a SQL query from postgres that uses any of the listed JSON operators, you'll see that the obfuscated result correctly parses these operators after this change.
Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the config template has been updated.

I converted it to a draft. Best to work in "Draft" mode until you find you're ready for a review, and then request it.


Use primitive formatting for primitive references

Closes #163
A single recursive call in as_native_type is sufficient
to unpack the type from the reference.
Formatting functions (fmt.i32() etc.) are already taking
a reference, so this does not need to be changed.
Tests are adjusted, so that they pass.
This introduced change is not explicitly tested though.
#[derive(Format)]
struct S2<'a> {
    x: &'a i32,
    y: &'a u8,
}

now expands into
fn format(&self, f: &mut defmt::Formatter) {
    if f.needs_tag() {
        f.istr (&defmt::export::istr({
            #[link_section = ".defmt.{\"package\":\"use-test\",\"tag\":\"defmt_fmt\",\"data\":\"S2 {{ x: {:i32}, y: {:u8} }}\",\"disambiguator\":\"10404396863279507298\"}"]
            #[export_name = "{\"package\":\"use-test\",\"tag\":\"defmt_fmt\",\"data\":\"S2 {{ x: {:i32}, y: {:u8} }}\",\"disambiguator\":\"10404396863279507298\"}"]
            static S: u8 = 0;
            &S as * const u8 as usize
        })) ;
    }
    match self {
        Self { x, y } => {
            f.i32(x);
            f.u8(y);
        }
    }
}

bors r+


[Post] DMS Demo

These are all the things you can do with this DMS. And these are the things we plan for it to be able to do. What do you think?

Closed as obsolete.


Automatic error report

<!---   *** BEFORE SUBMITTING: PASTE CLIPBOARD HERE TO COMPLETE YOUR REPORT ***   ---!>
 'plugin = mod.PLUGIN_CLASS(self)
AttributeError: '_ModuleMock' object has no attribute 'PLUGIN_CLASS'

This is fixed in our latest version (3.3.1). Please update by opening the Anaconda Prompt and running there
conda update spyder


no visible focus indicator for buttons

Describe the bug
While buttons remain keyboard accessible, they appear not to be.  This behavior is not in compliance with the Web Content Accessibility Guidelines (WCAG)
If this is intentional, please explain.
Expected behavior
some form of visual feedback that indicates focus.  An outline, for example.
Screenshots
no focus indicator

default webkit focus indicator outline: -webkit-focus-ring-color auto 1px;

Additional context
We received a support request on Maverick, stating that the modal close button was inaccessible to students who must navigate via keyboard only.  I discovered that the modal didn't actually have a close button, just a div styled like one.  So my first instinct (being a genius after all) was to use a button instead of a div.  That lead me to notice that while the close button was now accessible, it still didn't appear to be.  Interestingly, leaving the div in place with the addition of tabindex=0 provides both keyboard accessibility & a visual focus indicator.  However, that seems semantically incorrect and probably doesn't play very nicely with screen readers..
Full discloser: Maverick is not actually using Optics (I don't think, could be wrong) However, it's css base is the Rolemodel design system, which lead me to check into whether or not Optics had the same issue.  Thanks!

@Jeremy-Walton, @scriswell, @zoopmaster   - not a pressing issue, but let me know if any of you have an opinion on this.  Thanks!

@OutlawAndy - Optics does have keyboard focus states. Is this an open Maverick support request that needs to be resolved?

@zoopmaster okay, thanks.  My mistake.  Closing this issue

@OutlawAndy I believe Maverick started in June of 2021 so while there is a lot that is similar, a lot about Optics has been improved since then.
Focus states on buttons have been in the last few version of Optics, but were revised slightly in the recent update. https://docs.optics.rolemodel.design/?path=/docs/components-button--docs


Boolean representation on GuzzleHttp\Psr7\build_query()

$data = [
    'true' => true,
    'false' => false
];

/*
 * GuzzleHttp\Psr7\build_query()
 */
var_dump(GuzzleHttp\Psr7\build_query($data));
// string(13) "true=1&false="

/*
 * \http_build_query()
 */
var_dump(\http_build_query($data));
// string(14) "true=1&false=0"

GuzzleHttp\Psr7\build_query() resolves false to '' but I think it should be 0 as well as \http_build_query().
I'll file a PR if it's reasonable for you too.

FYI this code is called by swagger-codegen.
All code generated by swagger-codegen is broken currently.


HDDS-5555. remove pipeline manager v1 code

What changes were proposed in this pull request?
remove pipeline manager v1 code
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-5555
How was this patch tested?
unit test

@bharatviswa504 PTAL!  this is a simple pre work of https://issues.apache.org/jira/browse/HDDS-5529

The patch LGTM, +1.

thanks @ChenSammi for the review!


Create User Routes

From reading the docs here, create user routes with respective restrictions. Controller functions are present in controllers > user.controller.js, auth and verify middlewares are present in middlewares folder.

Make sure to pull before starting your work.
For any doubts or difficulty, you can ping me on discord, I'll try to help you out as soon as possible.
All the Best!!

claim

Assigned to @GammaMicrowave

clem take 2


Update download-manager to use upstream download

download 4.x now is based on streams, so it will use a lot less memory by doing decompression on the fly

download now is based on streams but it's still storing the full file in memory, so it's useless to us as is it. We'll need to create our own download mechanism, luckily now the code of download is fairly simple and we can use it as basis :-)

I can take that since my logo PR depends on the download manager anyway ...

I'm having a review on it, 'download' use streams but it's still storing it
on memory so it's useless. I had a previous atempt of doing the downloads
directly but didn't fully work, I'll upload the changes so you can be able
to review them.
El 16/8/2016 7:43, "John Green"<EMAIL_ADDRESS>escribió:

I can take that since my logo PR depends on the download manager anyway
...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NodeOS/NodeOS/issues/290#issuecomment-240006587, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAgfviBg1AuluiWJv549mcG8yrzqC7J_ks5qgU4dgaJpZM4Jjp6O
.

@Coretool, I have just pushed my updated code without download module and also added some test coverage. I'll try to add later an integration test copied from nodeos-cross-toolchain that's the most complex of the usage cases, so let's hope we can find why it was not working...

I finally managed to make it work! :-D It was a bug on got. I need to test it a little more but so far it's fast and don't consume RAM, that's just what I wanted :-D If you want to take a look I've uploaded the code to the download-manager repo :-)

New version of download-manager is being used on all NodeOS packages. clossing.


Update example for menu payload

https://github.com/grammyjs/website/issues/369#issuecomment-1159617425

As talked with @KnorpelSenf and @roj1512 we will start with a new workflow to manage external contributions.
@CikiMomogi Can you change the base branch to grammyjs:menu-payload? We will add translations in other PR managed internally

We first have to create a branch with the correct name in the main repo before the base branch can be changed. I created grammyjs:menu-payload now and adjusted the base branch of this contribution.


Provided network configured Deadline when creating Transactions

The SDKs won't be able to create deadlines by default anymore. The clients would need to provide the network configured EpochAdjustment. EpochAdjustment used to be hardcoded in the SDK but not anymore. This value can be retrieved via the RepositoryFactory like generation hash and network type.
This task is to upgrade the Wallet to the latest TS SDK (alpha for now) and update the way it creates the Deadlines.
SDK issue and PR
https://github.com/nemtech/symbol-sdk-typescript-javascript/pull/686
https://github.com/nemtech/symbol-sdk-typescript-javascript/issues/681

It is a breaking change from Symbol-SDK (next release) affecting the Deadline.Create() which requires epochAdjustment now. The epochAdjustment value should be retrieved from the network properties. The value is different between networks now. (it used to be hardcoded).

@yilmazbahadir this is technical. Could you close if done?


MSID: 87198 Version: 1 DOI: rs.3.rs-1828415/v2

[PLACE MANUSCRIPT AND EDITOR DETAILS HERE WHEN AVAILABLE] See step 3
[PLACE PDF URL HERE WHEN AVAILABLE] See step 7
Step 1. Inform bioRxiv
Who can help: @QueenKraken, @nlisgo, @scottaubrey

[x] Manuscript is in data hub index (https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v1/by-publisher/elife/get-by-manuscript-id?manuscript_id=87198)

Step 2. Create preview of manuscript
Who can help: @fred-atherden, @nlisgo, @scottaubrey

[ ] Manuscript is available for preview (https://prod--epp.elifesciences.org/preview/87198/v1)

[ ] Trigger import for staging: https://prod--epp.elifesciences.org/import
[ ] Sync data folder with s3://prod-elife-epp-data/data
[ ] Trigger import for staging: https://staging--epp.elifesciences.org/import
[ ] Sync data folder with s3://staging-elife-epp-data/data
[ ] Pull request created to enhanced-preprint-data
[ ] Post link to pull request below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-manuscript
Step 3: Awaiting public reviews
Who can help: Editorial team

[ ] Public reviews available on sciety (https://sciety.org/articles/activity/10.21203/rs.3.rs-1828415/v2)
[ ] Manuscript and editor details at the top of this issue (Supplied by Editorial team)
[ ] Reviews returned by EPP - https://prod--epp.elifesciences.org/api/reviewed-preprints/87198/v1/reviews

Example
"msas": "Genetics and Genomics", "Neuroscience"
"msid": "84628"
"version": "1"
"preprintDoi": "10.1101/2<IP_ADDRESS>4241"
"articleType": "Reviewed Preprint"
"status": "Published from the original preprint after peer review and assessment by eLife."

"Reviewed Preprint posted": "2023-01-02"
"Sent for peer review": "2022-10-28"
"Posted to bioRxiv": "2022-11-21" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>4241v1")

Editors:
Reviewing Editor
Michael B Eisen
University of California, Berkeley, United States
Senior Editor
Michael B Eisen
University of California, Berkeley, United States

Step 4: Modify manuscripts.json (no PDF)

[ ] Reviewed preprint is published to EPP (https://elifesciences.org/reviewed-preprints/87198)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[ ] Request that a doi be registered

[ ] Post link to #enhanced-preprint comment below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]

Instructions to modify manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/publish-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/334/files
Once the pull request is merged in it should be available a few minutes later.

Request that a doi
Post the following in #enhanced-preprint:
@Fred can you register a doi for https://elifesciences.org/reviewed-preprints/87198

Step 5: Awaiting search reindex

[ ] Reviewed preprint is avaliable on journal homepage (https://elifesciences.org)

The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.

Additional info
If needed, the jenkins pipeline to reindex search can be triggered sooner.
https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 6: Published! Request PDF generation

[ ] Post the link to the PDF url at the top of the issue

[ ] Post link to #enhanced-preprint comment below

#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]
Post the following to the #enhanced-preprint on slack:
@Ryan Dix-Peek please can you generate a PDF for https://elifesciences.org/reviewed-preprints/87198

Step 7: Introduce PDF to data folder and git repo

[ ] PDF is avaliable at https://github.com/elifesciences/enhanced-preprints-data/raw/master/data/87198/v1/87198-v1.pdf

Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-pdf
Step 8: Add PDF url to manuscripts.json

[ ] Reviewed preprint PDF is available for download (https://prod--epp.elifesciences.org/reviewed-preprints/87198)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[PLACE LINK TO PULL REQUEST HERE]

Instructions to add PDF url to manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/add-pdf-url-to-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/397/files
Once the pull request is merged in it should be available a few minutes later.

Step 9: Done!

[ ] Kettle is on!

Have requested Exeter provide a MECA package.

Thanks Andy - yep, ETA on the XML is 12th June, so I expect we should be able to make that date.

MECA package uploaded to:
s3://prod-elife-epp-meca/10.21203/rs.3.rs-1828415-v2-meca.zip


Add documentation for the toObject function

Example usage of the 2 overloads:
var entries = [
  { id: 1, prop: 'val1' }
  { id: 2, prop: 'val1' }
]
output keyMap object = toObject(
  entries,
  entry => string(entry.id))
// returns JSON {"1":{"id":1,"prop":"val1"},"2":{"id":2,"prop":"val1"}}

var values = [
  { id: 1, properties: { prop: 'val1' } }
  { id: 2, properties: { prop: 'val1' } }
]
output valueMap object = toObject(
  values,
  entry => string(entry.id),
  entry => entry.properties)
// returns JSON {"1":{"prop":"val1"},"2":{"prop":"val1"}}

I would also explicitly mention that the key lambda must return a string - it cannot return an integer or other type (hence the usage of string(...) in the example.

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

ID: e25074ce-efdb-89f4-8603-72b944081662
Version Independent ID: c45698a8-97fd-5265-6d09-9fb6465cb837
Content: Bicep functions - Azure Resource Manager
Content Source: articles/azure-resource-manager/bicep/bicep-functions.md
Service: azure-resource-manager
Sub-service: bicep
GitHub Login: @mumian
Microsoft Alias: jgao

@anthony-c-martin
Thanks for your feedback! We will investigate and update as appropriate.

Content published - https://learn.microsoft.com/azure/azure-resource-manager/bicep/bicep-functions-lambda
#please-close

Thanks @mumian!


Live chart previews on the charts guide

Description
Adds the ability to render live previews of SDK components within guides. This updates the charts guide specifically to render live previews of the line charts used in the guide.
Screenshot(s)

I always forget that window is not available in the build. I will fix this.

@jerelmiller did you want to finish this up still?

@jerelmiller did you want to finish this up still?

Yes. Its currently failing the build so I just need to fix that.

Yes. Its currently failing the build so I just need to fix that.


[QUESTION] Backup POD always lands on the same node

The POD that generates a backup of the volumes, always falls on the same NODE, I believe it should have a balance so as not to degrade this node, see the image! can i adjust something?
Longhorn 1.3.0

@elton557 can you explain a bit info? also, please provide the support bundle. Thanks.

@c3y1huang This should be related to recurring job. Do you have any ideas?


trying to upgrade actions to stop the warnings

Just upgrading some modules, as well as the github action versions to prevent the deprecation warnings in the future.

merging since jeff approved, even if it was with the "wrong" account


ci(test): added automatic screenshot testing via Chromatic CI

The screenshot testing is possible thanks to the open source sponsorship by Chromatic.
The action always passes (even when visual changes are detected) except for cases when a story is broken. However we expect contributors and reviewers to check the results of the action (and accept or decline them in the Chromatic UI) by following a link in the build log.
Right now it runs on push, because running it on pull-request and blocking a PR needs more repo permissions anyway.
The Chromatic secret key is exposed 2 times – in package.json since it should help with testing in forked repos, and in chromatic.yml because we can't run the Github action without it and we have no rights yet for adding a secret to the repo as an environment variable.

Note that the open source status has not been confirmed yet (should happen really soon), and we are out of snapshots for this month (until 04.04).


refactor: use relative import from this typing package

Description:
replace some import ref from using confluent_kafka directly
Related Issue:
resolve #125
Type of change:

[x] New feature / Refactor / Enhancement (non-breaking change which adds new typings)

Checklist:

[ ] All code follows the project's coding style and conventions.
[ ] I have performed a self-review of my own code.
[ ] My changes generate no new warnings or errors.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding updates to the documentation.

:tada: This PR is included in version 0.5.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[BUG] cant create installer

:information_source: Info

Version: 2.8.16
:speech_balloon: Description
cant create installer
✓ parsing cli arguments
✓ validating config values
✓ cleaning temporary files
✓ creating app icons folder
✓ generating icons
✓ copying app icons
✓ copying VC libraries
✓ getting certificate publisher
✓ generate appx manifest
❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚ 64% generate package resource indexing files..
Microsoft (R) MakePRI Tool
Copyright (C) 2013 Microsoft. All rights reserved.

Usage:
------
    MakePri.exe new /pr <project root> /cf <config file> [options]

Example:
--------
    MakePri.exe new /pr C:\MyApp\src\ /cf C:\MyApp\priconfig.xml /mn
    C:\MyApp\AppXManifest.xml /of C:\MyApp\src\resources.pri /o

Description:
------------
    Creates a PRI file at [outputfile] by indexing all files in the
    [projectroot]and its subdirectories as directed by the [configxml]. The
    index will be assigned [indexname] to reference resources in the application

Required Parameters:
--------------------
    /ProjectRoot(pr)  : <FOLDERPATH> Root location of project files
    /ConfigXml(cf)    : <FILEPATH> Configuration file location. Use
                        'Makepri.exe createconfig' command to generate one

Options:
--------
    /OutputFile(of)   : <FILEPATH> Output location of PRI file, default is
                        [current directory]\resources.pri
    /Manifest(mn)     : <FILEPATH> Location of the application or component's
                        manifest. This parameter is ignored if [indexname]
                        is given. Default is [projectroot]\AppXManifest.xml
    /IndexName(in)    : <STRING> Name for the generated index of resources.
                        Typically matches the AppX package name, class library
                        simple name, etc. May be supplied via the
                        [manifest] parameter.
                        If IndexName is not specified and an AppX manifest
                        file is not present, the default name 'Application'
                        will be used.
    /VersionMajor(vma): <INTEGER> [Deprecated] Major version number for
                        index, default is 1
    /IndexLog(il)     : <FILEPATH> XML Log of indexed resources, no file
                        generated by default
    /AutoMerge(am)    : This flag is not recommended for normal use with AppX
                        packages. It causes Makepri.exe to set the auto
                        merge flag within the PRI file. Default is not set.
    /ReverseMap(rm)   : Generate a reverse mapping section in the PRI file
                        which can be used for debugging purposes.
    /MappingFile(mf)  : <MAPPINGFILETYPE> Generate a mapping file in the given
                        file format.
    /SchemaFile(sf)   : <FILEPATH> Output location of XML resource schema
                        description.
    /IndexOptions(io) : <OPTIONS> Options to provide detailed control over
                        behavior of resource indexers.
    /Overwrite(o)     : Overwrite an existing output file of the same name
                        without prompting
    /Verbose(v)       : Causes verbose messages to be output to the console
    /Help(h, ?)       : Display the usage help text
    /ExtensionDll(ex) : <FILEPATH> Location of the MRT environment extension
                        DLL. This DLL must be signed by a Microsoft-issued
                        certificate. Default is an empty path (no DLL
                        will be used).

    FOLDERPATH       - is a valid path to a folder
    FILEPATH         - is a path to a file, either relative to the current
                       directory or absolute
    MAPPINGFILETYPE  - Supported File type(s): 'AppX'

ERROR: PRI191: 0x80004005 - Appx manifest not found or is invalid. Please ensure well-formed manifest file is present. Or specify an index name with /in switch.
A name was started with an invalid character.

pub finished with exit code -1

:scroll: Pubspec.yaml

version: 3.0.178+1

environment:
  sdk: ">=2.15.0 <3.0.0"

dependencies:
  flutter:
    sdk: flutter
  get: ^4.1.4
  catcher: ^0.6.6
  sqflite: ^2.0.0+3
  async: ^2.6.1
  win32: ^2.2.9
  xpath_selector: ^2.2.2
  cryptography: ^2.0.5
  #video player
  volume_controller: ^2.0.2

  shared_preferences: ^2.0.13

    #flutter_vlc_player:
  #vlc_flutter:
    #git: https://github.com/zezo357/vlc-flutter
  video_player: ^2.2.18
  auto_orientation: ^2.0.2
  screen_brightness: ^0.1.1
  flutter_meedu: ^5.0.0
  flutter_spinkit: ^5.0.0
  meta: ^1.3.0
  dart_vlc: ^0.1.7
  share_plus: ^3.0.4

  window_manager: ^0.1.1
  screen_retriever: ^0.1.0
  oktoast: ^3.1.3+1

  #end of video player
  flutter_screenutil: "5.0.2"
  connectivity_plus: ^2.0.1
  path: ^1.8.0
  cached_network_image: ^3.0.0
  wakelock: ^0.6.1
  retry: ^3.1.0
  version: ^2.0.0
  permission_handler: ^8.1.1
  filesize: ^2.0.1
  fluttertoast: ^8.0.7
  helpers: ^1.1.3
  dio: ^4.0.0
  flutter_local_notifications: ^9.0.0
  queue: ^3.1.0
  hotkey_manager: ^0.1.2
  archive: ^3.1.6

  android_intent: ^2.0.2
  device_apps: ^2.1.1
  html: ^0.15.0
  file_picker: ^4.0.0
  workmanager: ^0.5.0-dev.8

  pull_to_refresh: ^2.0.0
  encrypt: ^5.0.0
  draggable_scrollbar: ^0.1.0
  auto_size_text: ^3.0.0-nullsafety.0
  intl: ^0.17.0
  fluttericon:
  android_path_provider: ^0.3.0
  open_file:
  collection: ^1.15.0-nullsafety.4
  liquid_progress_indicator: ^0.4.0
  menu_button: ^1.4.2+1
  sqflite_common_ffi:
  xml: ^5.3.1
  google_nav_bar: ^5.0.5

  #flutter_webview_plugin:
    #git: https://github.com/zezo357/flutter_webview_plugin

dev_dependencies:
  flutter_native_splash: ^2.0.1+1
  flutter_launcher_icons: ^0.9.0
  #msix: ^3.0.0-dev.1
  msix: ^2.8.16

  flutter_test:
    sdk: flutter
flutter_icons:
  android: "launcher_icon"
  windows: true
  ios: true
  image_path: "assets/icon.png"

msix_config:
  display_name: AnimeHere
  publisher_display_name: abdelaziz
  identity_name: com.zcreations.AnimeHere
  msix_version: <IP_ADDRESS>
  logo_path:  G:\My Work HDD2\AndroidStudioProjects\anime_here\windows\runner\resources\app_icon.ico
  start_menu_icon_path: G:\My Work HDD2\AndroidStudioProjects\anime_here\windows\runner\resources\app_icon.ico
  tile_icon_path: G:\My Work HDD2\AndroidStudioProjects\anime_here\windows\runner\resources\app_icon.ico

Hello
I'm trying to figure out what the source of the problem is,
can you please try again without the all msix_config: section

@YehudaKremer
this is the output without the msix_config
✓ parsing cli arguments
✓ validating config values
✓ cleaning temporary files
✓ creating app icons folder
✓ copying app icons
✓ copying VC libraries
✓ getting certificate publisher
✓ generate appx manifest
❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚ 61% generate package resource indexing files..
Microsoft (R) MakePRI Tool
Copyright (C) 2013 Microsoft. All rights reserved.

Usage:
------
    MakePri.exe new /pr <project root> /cf <config file> [options]

Example:
--------
    MakePri.exe new /pr C:\MyApp\src\ /cf C:\MyApp\priconfig.xml /mn
    C:\MyApp\AppXManifest.xml /of C:\MyApp\src\resources.pri /o

Description:
------------
    Creates a PRI file at [outputfile] by indexing all files in the
    [projectroot]and its subdirectories as directed by the [configxml]. The
    index will be assigned [indexname] to reference resources in the application

Required Parameters:
--------------------
    /ProjectRoot(pr)  : <FOLDERPATH> Root location of project files
    /ConfigXml(cf)    : <FILEPATH> Configuration file location. Use
                        'Makepri.exe createconfig' command to generate one

Options:
--------
    /OutputFile(of)   : <FILEPATH> Output location of PRI file, default is
                        [current directory]\resources.pri
    /Manifest(mn)     : <FILEPATH> Location of the application or component's
                        manifest. This parameter is ignored if [indexname]
                        is given. Default is [projectroot]\AppXManifest.xml
    /IndexName(in)    : <STRING> Name for the generated index of resources.
                        Typically matches the AppX package name, class library
                        simple name, etc. May be supplied via the
                        [manifest] parameter.
                        If IndexName is not specified and an AppX manifest
                        file is not present, the default name 'Application'
                        will be used.
    /VersionMajor(vma): <INTEGER> [Deprecated] Major version number for
                        index, default is 1
    /IndexLog(il)     : <FILEPATH> XML Log of indexed resources, no file
                        generated by default
    /AutoMerge(am)    : This flag is not recommended for normal use with AppX
                        packages. It causes Makepri.exe to set the auto
                        merge flag within the PRI file. Default is not set.
    /ReverseMap(rm)   : Generate a reverse mapping section in the PRI file
                        which can be used for debugging purposes.
    /MappingFile(mf)  : <MAPPINGFILETYPE> Generate a mapping file in the given
                        file format.
    /SchemaFile(sf)   : <FILEPATH> Output location of XML resource schema
                        description.
    /IndexOptions(io) : <OPTIONS> Options to provide detailed control over
                        behavior of resource indexers.
    /Overwrite(o)     : Overwrite an existing output file of the same name
                        without prompting
    /Verbose(v)       : Causes verbose messages to be output to the console
    /Help(h, ?)       : Display the usage help text
    /ExtensionDll(ex) : <FILEPATH> Location of the MRT environment extension
                        DLL. This DLL must be signed by a Microsoft-issued
                        certificate. Default is an empty path (no DLL
                        will be used).

    FOLDERPATH       - is a valid path to a folder
    FILEPATH         - is a path to a file, either relative to the current
                       directory or absolute
    MAPPINGFILETYPE  - Supported File type(s): 'AppX'

ERROR: PRI191: 0x80004005 - Appx manifest not found or is invalid. Please ensure well-formed manifest file is present. Or specify an index name with /in switch.
A name was started with an invalid character.

pub finished with exit code -1

can you please paste here the content of \build\windows\runner\Release\AppxManifest.xml

  <Package xmlns="http://schemas.microsoft.com/appx/manifest/foundation/windows10"
          xmlns:uap="http://schemas.microsoft.com/appx/manifest/uap/windows10"
          xmlns:uap2="http://schemas.microsoft.com/appx/manifest/uap/windows10/2"
          xmlns:uap3="http://schemas.microsoft.com/appx/manifest/uap/windows10/3"
          xmlns:uap4="http://schemas.microsoft.com/appx/manifest/uap/windows10/4"
          xmlns:uap6="http://schemas.microsoft.com/appx/manifest/uap/windows10/6"
          xmlns:uap7="http://schemas.microsoft.com/appx/manifest/uap/windows10/7"
          xmlns:uap8="http://schemas.microsoft.com/appx/manifest/uap/windows10/8"
          xmlns:uap10="http://schemas.microsoft.com/appx/manifest/uap/windows10/10"
          xmlns:iot="http://schemas.microsoft.com/appx/manifest/iot/windows10"
          xmlns:desktop="http://schemas.microsoft.com/appx/manifest/desktop/windows10"
          xmlns:desktop2="http://schemas.microsoft.com/appx/manifest/desktop/windows10/2"
          xmlns:desktop6="http://schemas.microsoft.com/appx/manifest/desktop/windows10/6"
          xmlns:rescap="http://schemas.microsoft.com/appx/manifest/foundation/windows10/restrictedcapabilities"
          xmlns:rescap3="http://schemas.microsoft.com/appx/manifest/foundation/windows10/restrictedcapabilities/3"
          xmlns:rescap6="http://schemas.microsoft.com/appx/manifest/foundation/windows10/restrictedcapabilities/6"
          xmlns:com="http://schemas.microsoft.com/appx/manifest/com/windows10"
          xmlns:com2="http://schemas.microsoft.com/appx/manifest/com/windows10/2"
          xmlns:com3="http://schemas.microsoft.com/appx/manifest/com/windows10/3"
          IgnorableNamespaces="uap3 desktop">
    <Identity Name="com.flutter.animehere" Version="<IP_ADDRESS>"
              Publisher="CN=Msix Testing, O=Msix Testing Corporation, S=Some-State, C=US" ProcessorArchitecture="x64" />
    <Properties>
      <DisplayName>animehere</DisplayName>
      <PublisherDisplayName>com.flutter.animehere</PublisherDisplayName>
      <Logo>Images\StoreLogo.png</Logo>
      <Description>Watch anime without ads <3</Description>
    </Properties>
    <Resources>
      <Resource Language="en-us" />
    </Resources>
    <Dependencies>
      <TargetDeviceFamily Name="Windows.Desktop" MinVersion="10.0.17763.0" MaxVersionTested="10.0.19042.630" />
    </Dependencies>
    <Capabilities>
      <Capability Name="internetClient" />
      <rescap:Capability Name="runFullTrust" />
      <DeviceCapability Name="location" />
      <DeviceCapability Name="microphone" />
      <DeviceCapability Name="webcam" />
    </Capabilities>
    <Applications>
      <Application Id="animehere" Executable="anime_here.exe" EntryPoint="Windows.FullTrustApplication">
        <uap:VisualElements BackgroundColor="transparent"
          DisplayName="animehere" Square150x150Logo="Images\Square150x150Logo.png"
          Square44x44Logo="Images\Square44x44Logo.png" Description="Watch anime without ads <3">
          <uap:DefaultTile ShortName="animehere" Square310x310Logo="Images\LargeTile.png"
          Square71x71Logo="Images\SmallTile.png" Wide310x150Logo="Images\Wide310x150Logo.png">
            <uap:ShowNameOnTiles>
              <uap:ShowOn Tile="square150x150Logo"/>
              <uap:ShowOn Tile="square310x310Logo"/>
              <uap:ShowOn Tile="wide310x150Logo"/>
            </uap:ShowNameOnTiles>
          </uap:DefaultTile>
          <uap:SplashScreen Image="Images\SplashScreen.png"/>
          <uap:LockScreen BadgeLogo="Images\BadgeLogo.png" Notification="badge"/>
        </uap:VisualElements>
          </Application>
    </Applications>
  </Package>```

Thank you!
Checking...

I think i found the problem, can you please try version 2.8.17

i guess its working?
✓ parsing cli arguments
✓ validating config values
✓ cleaning temporary files
✓ creating app icons folder
✓ copying app icons
✓ copying VC libraries
✓ getting certificate publisher
✓ generate appx manifest
✓ generate package resource indexing files
✓ packing
✓ cleaning temporary files
❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚ 84% installing certificate..
To install the test certificate run the command "flutter pub run msix:create" as administrator
pub finished with exit code -1

would love that
thanks ♥

okay wait a minute

it worked thanks ♥

admin dialog? in console?

i didnt choose yes?
rerunning the code to get the output to you
                                                                                G:\My Work HDD2\AndroidStudioProjects\anime_here>call flutter clean                                                     Deleting build...                                                  599ms                                                Deleting .dart_tool...                                              17ms                                                Deleting .packages...                                                0ms                                                Deleting Generated.xcconfig...                                       0ms                                                Deleting flutter_export_environment.sh...                            0ms
Deleting ephemeral...                                               24ms
Deleting .flutter-plugins-dependencies...                            0ms
Deleting .flutter-plugins...                                         0ms
Running "flutter pub get" in anime_here...                       2,716ms

 Building with sound null safety

Building Windows application...
copying windows app
The syntax of the command is incorrect.
creating msix installer...             20.7s
msix created: build\windows\runner\Release\anime_here.msix

Waiting for  995 seconds, press a key to continue ...

btw when sending updates should i change the msix version?

yes, lets day the your current version is <IP_ADDRESS>, then change it to <IP_ADDRESS> and create new msix installer.
thanks you for reporting those bugs, it's very helpful!
if you have any more problems please let me know

thanks


Entity jitters when setting location to cursor position

Based on the billBoardsOnTheGround example I wrote code to make an entity (Polygon) track the cursor on the globe. That code looks like this:
this.planet.renderer.events.on(
      "mousemove",
      function (e) {
        if (that.placingObject) {
          const pos = this.getLonLatFromPixelTerrain(e);
          that.pos = pos;
          that.entity.geometry.setGeometry({
            type: "Polygon",
            coordinates: [
              [
                [ pos.lon - 0.05, pos.lat - 0.05 ],
                [ pos.lon + 0.05, pos.lat - 0.05 ],
                [ pos.lon + 0.05, pos.lat + 0.05  ],
                [ pos.lon - 0.05, pos.lat + 0.05 ],
              ],
            ],
            style: {
              fillColor: "rgba(0,0,0,0.5)",
              lineColor: "rgba(0,0,0,0.5)",
            },
          });
        }
      },
      this.planet
    );

The example used a point and the setLatLon function to change the location. Since this is a polygon I figured the setGeometry was the right function in this case.
This code gives me the following behaviour:
https://user-images.githubusercontent.com/75586344/223946088-54a47f90-83e9-496e-a2f7-2426ef4d1db7.mp4
Is this a bug/unavoidable/not the right way to do this?

Ok I understand. Geometry, being a static entity, will always lead to this effect because it gets redrawn every time it is changed rather than 'moved'. Unless the VectorTileCreator is optimized even more so that you don't notice it being redrawn (just removing the deltaTime < 0.25 sadly isn't a quick fix).
Indeed Polylines seem to move much more smoothly, but they also seem to have some unexpected behavior when dragging them up close. I'll see if I can open an issue about that as well.
I also noticed GeoImages would be a usable alternative to draw a simple rectangle based on corner coordinates. I'll play around with that as well
Thanks for the clarification!


Add Mention-Bot to sugar

https://github.com/facebook/mention-bot

@satyan could you please just setup the mention-bot it is really simple

@sibeliusseraphini I've set it up.. thanks.

thanks


Cleanup code examples for relation animations

This was really difficult to read, should be slightly easier now

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

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

#sign-off

#sign-off


Primary Detail looks like flaming garbage on IE11

https://github.com/patternfly/patternfly-elements/pull/1352


Fix typo

IPSec -> IPsec

@jejackson-ms
Can you review the proposed changes?
IMPORTANT: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


[BUG] Unexpected value. Please add a @NgModule annotation.

🐞 Bug report
Command (mark with an x)

- [ ] new
- [x] build
- [ ] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Is this a regression?
Not, this is a older problem, i reported previusly in #13006 without providing a repo.
Description
 I receive a Unexpected value 'LayoutModule' error when building a library with a single NgModule dependency (one NgModule depends on other).
🔬 Minimal Reproduction
This repo contains a minimal example of the issue: https://github.com/prodisign/bug-reporting
ng build rh

🔥 Exception or Error

------------------------------------------------------------------------------
Building entry point '@prodisign/rh/model/client'
------------------------------------------------------------------------------
Compiling TypeScript sources through ngc
ERROR: Unexpected value 'LayoutModule in /home/jdavidls/Proyectos/prodisign/experiments/prodisign-workspace/dist/prodisign/rh/client/prodisign-rh-client.d.ts' imported by the module 'ModelModule in /home/jdavidls/Proyectos/prodisign/experiments/prodisign-workspace/projects/prodisign/rh/model/client/src/ModelModule.ts'. Please add a @NgModule annotation.

An unhandled exception occurred: Unexpected value 'LayoutModule in /home/jdavidls/Proyectos/prodisign/experiments/prodisign-workspace/dist/prodisign/rh/client/prodisign-rh-client.d.ts' imported by the module 'ModelModule in /home/jdavidls/Proyectos/prodisign/experiments/prodisign-workspace/projects/prodisign/rh/model/client/src/ModelModule.ts'. Please add a @NgModule annotation.

See "/tmp/ng-1Uj8ei/angular-errors.log" for further details.

🌍 Your Environment

Angular CLI: 8.3.18
Node: 10.15.3
OS: linux x64
Angular: 8.2.13
... animations, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router

Package                            Version
------------------------------------------------------------
@angular-devkit/architect          0.803.18
@angular-devkit/build-angular      0.803.18
@angular-devkit/build-ng-packagr   0.803.18
@angular-devkit/build-optimizer    0.803.18
@angular-devkit/build-webpack      0.803.18
@angular-devkit/core               8.3.18
@angular-devkit/schematics         8.3.18
@angular/cli                       8.3.18
@ngtools/webpack                   8.3.18
@schematics/angular                8.3.18
@schematics/update                 0.803.18
ng-packagr                         *5.7.1*
rxjs                               6.4.0
typescript                         3.5.3
webpack                            4.39.2

Edit:
In the repo, I am using a complex module disposition, doing use of secondary entrypoints. But other simplest dispositions throwns the same error.
Im reported the same issue at https://github.com/ng-packagr/ng-packagr/issues/1444.

This seems related to barrel files.
In projects/prodisign/rh/client/src/public_api.ts change export * from './layout/ to export * from './layout/index';
I am moving this to the FW repo as it's a compiler bug.


missing graphs

Can you run a check of missing links and missing figures and fix them?
For example, I found missing image files on this page:
https://alps.comp-phys.com/tutorials/python/general/latticehowtos/latticegraph/

Missing lattice graphs have been found. The pages have been updated.


Document versioning approach

Release end of sprint, nut can easily change.
Follow SEMVER, but with that breaking changes extra
Need to be bringing it into sprint planning/PI planning our major breaking changes
Need to communicate the regular releases (e.g. say 12 months) and how frequent and give people notice (give people the next tag)

https://github.com/mi6/ic-design-system/issues/202#issuecomment-1377645453


Entity Catalog: Address problematic store helpers

Most of the CF helpers can move from the store helpers folder.
Some are a bit more intertwined and more work is needed - specifically:
entity-relations-post-processor.ts
schema-tree-traverse.ts
all test files (*.spec.ts)

Double check that the pipeline work has unwound this from the core code. If its just a matter of moving the code then we can leave it for later. If the code is still use in core then we should fix it.

Fixed via #3837


Discussion: non-deterministic device name?

I'm not exactly sure where this belongs or what I'm asking for, but figured I should note it somewhere.  I've noticed in my efforts to get Tailscale installed automatically on a host and then use that device for further work in Terraform, that the device name used by Tailscale is not guaranteed.
In Terraform, I might want to create a server "enterprise" in AWS, set it's hostname as "enterprise", install & authorize Tailscale as part of the instance's first-boot configuration, and then wait_for (#72) the device so I can use it's IP to setup a DNS record for that host.
But, if there is already a device in Tailscale named "enterprise" then it looks like Tailscale will create the device as "enterprise1" and happily move on.  That obviously breaks things if then use tailscale_device with name as enterprise.example.com.
I've not yet tried to see Tailscale's behavior is any different using tailscale up --hostname.  Also, at least in my case, the impact of this is lessened if deletions (#68) become supported.

Might be related: https://github.com/tailscale/tailscale/issues/1532

Would you be better supported if the terraform provider allowed you to query devices based on an alternative field than name? The provider could allow you to search based on hostname so you get more deterministic behaviour when you use tailscale up --hostname?

@davidsbond thanks again for the time you are taking to discuss all of this.  Since we don't have a real provider that is creating the device and, under the hood, returning that device's information from the create API call, we have to lookup the device somehow, some way in a GET API call.  That, obviously, requires an identifier.
I think the only way that a different way to lookup the device, i.e. a field other than name, would be if tailscale up gave an option to accept an identifier and used it to create the device.  That identifier would then need to be exposed through the API.
As far as I can tell, giving some other field just doesn't matter b/c that value is not known ahead of time and can't be used to identify the device when using tailcale_device.

@davidsbond thanks again for the time you are taking to discuss all of this.

No problem, thanks for being a user and helping improve the provider.

Since we don't have a real provider that is creating the device and, under the hood, returning that device's information from the create API call, we have to lookup the device somehow, some way in a GET API call. That, obviously, requires an identifier.

Can the value to the --hostname flag not be that identifier currently? The documentation for the flag currently states:

hostname to use instead of the one provided by the OS

So in theory, you can provide anything that could be a valid hostname in this field and it will accept it. So you could obtain the specific device using the hostname you provide the tailscale up if the tailscale_device data source is updated to allow you to query on that field.

As far as I can tell, giving some other field just doesn't matter b/c that value is not known ahead of time and can't be used to identify the device when using tailcale_device.

Why can't it be known ahead of time if you can provide it to the tailscale up command? IIRC you mentioned previously about AWS instances where you were providing a bootstrapping script. I'm more familiar with GCP VMs, but they provide functionality that allows you to pass metadata into the VM which you can access from scripts. Assuming the same is possible with AWS instances, couldn't you do the following:

Use a variable in your terraform configuration for the hostname
Use said variable in your terraform configuration that creates the aws_instance, providing it as metadata or an environment variable (or however AWS lets you do it)
Obtain that metadata within your bootstrapping script which calls :
tailscale up --hostname ${TAILSCALE_HOSTNAME_FROM_SOMEWHERE}
Use a theoretical hostname field on the tailscale_device data source to look it up

So you could obtain the specific device using the hostname you provide the tailscale up command if the tailscale_device data source is updated to allow you to query on that field.

Isn't that what we do now?  The hostname becomes the host part of the device name, which is how tailscale_device works.  That all works fine and the process you described above to bootstrap a server is very similar to what I'm doing.

could avoid conflicting hostnames

This is really the key point.  Yes, I can do that now.  But it means the device's name in Tailscale now has some random value appended to it, which works but is ugly.  And, on the off chance something weird happens and you still have a conflicting hostname, there is nothing to indicate this.  The failure mode is silent, which I really don't like.
To be clear, this is the problem:
# Host 1
$ tailscale status
rsyring@meld:~$ tailscale status
...snip...
<IP_ADDRESS>  zinc                 randy.syring@ linux   -

data "tailscale_device" "this" {
  name = "zinc.example.com"
}

That data resource call would show the IP as <IP_ADDRESS>.
Now, let's say I don't realize that name is used and I run a cloud init script based on my Terraform scripts to run the equivalent of:
Host 2
$ tailscale up --hostname zinc --auth-key ...
Success.
$ tailscale status
<IP_ADDRESS>    zinc-1               randy.syring@ linux   -
...snip...
<IP_ADDRESS>  zinc                 randy.syring@ linux   -

Now, again, I have something like this in Terraform:

```terraform
data "tailscale_device" "this" {
  name = "zinc.example.com"
}

This will pull data from the first zinc host.  There will be no error.  The error will come later as I try to use the IP address, in a DNS record for example, and then can't figure out why the server I'm trying to connect to through Tailscale doesn't seem to be the server I want.
This isn't really a problem with this provider, it's just a silent footgun and I didn't want to keep the knowledge to myself.

Thanks for all the detail, without having some random string appended to the hostname I'm not sure any other workarounds are possible. Let's see how the team at Tailscale are regarding the issue you've raised. I'm happy to add any additional support that's within my capacity.
Unfortunately being a terraform provider makes resources etc only as available as the API we call. However, I can imagine that using Terraform/Pulumi will be important for other Tailscale users, so perhaps this use-case will be true for others. I only have very basic statistics that the Terraform registry provides regarding usage, but we have had 7k downloads of the provider at this point, which is perhaps a significant enough number to consider better support.

Tailscale has been working well for me, but this one issue is a real bummer when it comes to trying to use it at scale.
When I delete a k8s cluster and recreate it without remembering to go into the tailscale UI and manually delete each and every node, everything seems to work but it gets the ips of the old machines and then there are hard to debug connectivity issues later in the process.
Certainly if a new device pops up with a hostname that conflicts with another device with the same hostname that is ephemeral and in an unconnected state, it makes sense to delete the old device on tailscale's side immediately? It was going to be deleted automatically at some point, anyways.
If that makes people nervous an option could be added to tailscale_tailnet_key to perhaps on_ephemeral_hostname_conflict_remake or some such to make such behavior explicit.


Prep for v3.0.0 release

CHANGELOG was compressed to show only v2->v3 changes and metadata was
updated to refer to the v3 non-RC version.
What ticket does this PR close?
Connected to #219
Checklists
Change log

[x] The CHANGELOG has been updated, or
[ ] This PR does not include user-facing changes and doesn't require a CHANGELOG update

Test coverage

[ ] This PR includes new unit and integration tests to go with the code changes, or
[x] The changes in this PR do not require tests

Documentation

[ ] Docs (e.g. READMEs) were updated in this PR, and/or there is a follow-on issue to update docs, or
[x] This PR does not require updating any documentation

TBD if we can also update the README to say Trusted in this PR before tagging.


Opaaddon

Confirmed that OPA addon is running and works - will include examples and documentation once reviewers approve

Let's resolve the merge (rebase) and I will kick off the tests.

/do-e2e-tests

/do-e2e-tests

/do-e2e-tests

/do-e2e-tests

/do-e2e-tests

/do-e2e-tests

e2e faiiling. it is appmesh in this case, not really a descriptive error message, but it is related to the OPA add-on.

/do-e2e-tests

/do-e2e-tests

/do-e2e-tests

/do-e2e-tests

@parkand1  the e2e tests succeeded and I will merge the PR. Can you create a task for SSP-Patterns on how to use this add-on in practice? For example, create a blueprint with teams and clear OPA policies that can be demonstrable with the workloads repository.

Corresponding issue in SSP patterns:
https://github.com/aws-samples/ssp-eks-patterns/issues/4


Json property to Map<String, Map<String, Object>> not working [Hibernate 6.2.5.Final] [Mysql DB]

Hello,
I am using hypersistence-utils-hibernate-63:3.8.2 to store json in mysql 8 table with spring boot.
By using Hibernate 5 everything works well but with Hibernate 6 just map to Map<String, Object>. Looks like it's ignoring the second level of the json structure.
Here is the Entity class property :
    @Column( nullable = false, columnDefinition = "json")
    @Type(JsonType.class)
    private Map<String, Map<String, Object>> properties = new HashMap<>();

Data base JSON:
{
    "A": {
        "A1": [ {"id": 1, "name": "some name"} ],
        "A2": "some text"
    },
    "B": {
        "B1": [ {"id": 1, "name": "some name"} ],
        "B2": "some text"
    }
}

When return properties from the entity I get something like this:
{
    "A1": [ {"id": 1, "name": "some name"} ],
    "A2": "some text",
    "B1": [ {"id": 1, "name": "some name"} ],
    "B2": "some text"
}

As I said before, It seems like is mapping to a Map<String,Object> and ignoring the second map or the second level of the JSON structure.
I have been reading some of your post but I didn't find some related to Map<String,Map<>> but to Map<String,Object> like this:
Thanks in advance !

Working fine now, I had a bug in my service class.
My bad guys, sorry.


Added ability to parse json

This pull request should add support for parsing json so partially resolves one of the issues in #61

Thanks for the submission.

so partially resolves one of the issues in #61

But did you read that ticket? Because if you did (and also read README), you'd know that Picoweb is minimalist web framework which doesn't duplicate functionality already available in other modules/libraries. You can parse json using ujson.loads(string).

Then by your own definition the function should not parse the form data since the user could just call parse_qs(data). The problem is that while parsing json is supported by another module the pico web module has functions to help with reading the response however the form data is read into a dictionary and parsed as a form instead of providing the raw string.
I would propose that if the framework is not going to provide a json read function then the function below should be modified so that the raw response is made available to be used
def read_form_data(self):
size = int(self.headers[b"Content-Length"])
data = yield from self.reader.readexactly(size)
form = parse_qs(data.decode())
self.form = form
Here are  a few proposals
Option 1 - Save the (Raw Data) as a property on the Request so that it can be parsed by an external library without having to write a helper function for pico web module.
def read_form_data(self):
size = int(self.headers[b"Content-Length"])
self.data = yield from self.reader.readexactly(size)
form = parse_qs(self.data.decode())
self.form = form
Option 2 - Dont parse the data but return the raw response and let the calling application determine if its form data or json data.
def read_form_data(self):
size = int(self.headers[b"Content-Length"])
self.form = yield from self.reader.readexactly(size)
Using option 1 the web application can parse json by req.read_form_data()
then calling json.loads(req.data)
Using Option 2 The web application can parse the json by calling req.read_form_data()
then calling json.loads(req.form)
I would lean towards Option 1 because it maintains backwards compatibility without adding any lines of code but makes it very easy to parse the json while not having to re-write the code that reads the string from the request.

One other point is that if its such a bad idea to have a function in the HTTPRequest to parse the JSON Body which in todays day is a requirement on almost any web application then why is there a function called Jsonify provided by pico web which takes the dictionary and writes it to the response as Json?
Couldn’t the web application just call response.awrite(json.dumps(dict))?

Another 3rd option would be to define the function that reads the form data in such a way that it takes a parse data function as an agreement.
def read_form_data(self, parseFunction):
size = int(self.headers[b"Content-Length”])
data = yield from self.reader.readexactly(size)
self.form = parseFunction(data)
This allows the calling application to define how the data should be parsed (Json, Bson, Form Data, ect....) While still not having to deal with reading the raw response from the request.

Then by your own definition

My own definition is very simple: Be conservative with both adding new things, and removing existing things. And as this is minimalist project, be extra conservative with adding new things.

the function should not parse the form data

Dealing with HTML form data is what almost any standard (classic) web app would do, and hence convenience function is provided.

why is there a function called Jsonify provided by pico web which takes the dictionary and writes it to the response as Json?

Because it's pretty common usecase to develop a webapp which exports REST API with JSON responses. So, a convenience function for that was provided.

I would propose that if the framework is not going to provide a json read function then the function below should be modified

Please understand that there's no need to modify anything. Instead you do:
class MyWebApp(WebApp):
    # Add here whatever methods you want.

If you find that almost all your webapps use such an extended subclass, you can put it in a separate module for easy reuse, and even share it with others and maintain it yourself (answer questions, etc.). I encourage all that. All that was mentioned in #61 (I keep it open exactly to let people know all that without repeating it again and again).
But picoweb itself is intended to stay minimalist, and allow to develop small/simple webapps without adding much extra code overhead, while also being extensible (like above) to allow developing more complex webapps.
Thanks for the discussion.


[Release-1.27] - pprof statistics cannot be access when running server with --disable-agent

Backport fix for pprof statistics cannot be access when running server with --disable-agent

#10582

Validated using k3s commit id 112a185c346faafc8e2e5fe68fdb52e33aafabbd on release-1.27 branch
Environment Details
Infrastructure
Cloud EC2 instance
Node(s) CPU architecture, OS, and Version:
> cat /etc/os-release
NAME="SLES"
VERSION="15-SP5"
VERSION_ID="15.5"
PRETTY_NAME="SUSE Linux Enterprise Server 15 SP5"

Cluster Configuration:
Single node
Steps to reproduce

Install k3s passing --disable-agent  --enable-pprof
Open kube proxy running  kubectl proxy
Capture the profile data using go tool pprof --seconds=70 -top http://localhost:8001/debug/pprof/profile

Replication results:
ec2-user@ip-172-31-6-0:~> k3s -v
k3s version v1.27.16+k3s1 (a45cef49)
go version go1.22.5
ec2-user@ip-172-31-6-0:~> /usr/local/go/bin/go tool pprof --seconds=70 -top http://localhost:8001/debug/pprof/profile
Fetching profile over HTTP from http://localhost:8001/debug/pprof/profile?seconds=70
Please wait... (1m10s)
http://localhost:8001/debug/pprof/profile: server response: 404 Not Found
failed to fetch any source profiles
ec2-user@ip-172-31-6-0:~>

Validation results:
ec2-user@ip-172-31-6-0:~> k3s -v
k3s version v1.27.16+k3s-112a185c (112a185c)
go version go1.22.5
ec2-user@ip-172-31-6-0:~> /usr/local/go/bin/go tool pprof --seconds=70 -top http://localhost:8001/debug/pprof/profile
Fetching profile over HTTP from http://localhost:8001/debug/pprof/profile?seconds=70
Please wait... (1m10s)
Saved profile in /home/ec2-user/pprof/pprof.k3s.samples.cpu.004.pb.gz
File: k3s
Build ID: 49eae0faab5924ba5eeb1c8a50bcacd5e0171ca6
Type: cpu
Time: Aug 13, 2024 at 6:43pm (UTC)
Duration: 70s, Total samples = 1.83s ( 2.61%)
Showing nodes accounting for 1.83s, 100% of 1.83s total
      flat  flat%   sum%        cum   cum%
     0.23s 12.57% 12.57%      0.23s 12.57%  runtime/internal/syscall.Syscall6
     0.19s 10.38% 22.95%      0.19s 10.38%  runtime.futex
     0.12s  6.56% 29.51%      0.12s  6.56%  runtime.write1
     0.10s  5.46% 34.97%      0.10s  5.46%  runtime.cgocall
     0.09s  4.92% 39.89%      0.20s 10.93%  runtime.scanobject
     0.05s  2.73% 42.62%      0.25s 13.66%  runtime.netpoll
     0.03s  1.64% 44.26%      0.03s  1.64%  runtime.(*mspan).heapBitsSmallForAddr


[feature request] Method to allow Callable<> i.e. Gdx.app.postCallable()

I'm finding that, when developing for desktop, I have situations where I'm outside of the OpenGL context, but need to process something within its thread. Gdx.app.postRunnable is  a boon in these situations, but there are times when I need to use Future<> and future.get(), to be sure that what I want processed has completed properly.
Would it be possible to add this function to the package?
Thanks.

could help doing https://github.com/libgdx/libgdx/issues/2874

Might not be that easy, because on GWT Future still doesn't exist.
If I understand your request correctly, you want to block your background thread via Future.get until the GL thread has finished your Runnable and then proceed in the background thread?

For completeness, Callable also doesn't exist in GWT.

yes, noone, you are correct. Returning a processed result, would be icing on the cake.
regarding GWT, that's surprising. off-topic, and understandably that it's from another team, guava has an emulation of Callable since version 11, I believe.

Well, Future and Callable are just interfaces, so emulating them isn't really a problem (aside from implying support for the concurrent package when there is no real support).

The problem is that there's no real concurrency in GWT, which means any future emulation will dead lock :)
Sorru folks, you'll have to continue hand rolling this one.


Keycloak TF: failed to retrieve cryptographic signature for provider

In the past week I have run into the following error a few times when testing the QHub deployment:

during the Kubernetes test

using mrparker/keycloak v3.7.0

during the end-to-end integration test

using mrparker/keycloak v3.3.0

I reran the Kubernetes test again and the issue seemed to go away.

Hi @iameskild, it seems to be related to this https://www.terraform.io/cli/plugins/signing#plugin-signing. Maybe during those circumstances, the Hashicorp server was in maintenance (it seems that there are no recent issues with it though)?


Crashes on launch and leaves an empty window

Issue Type: Bug
I've gotten myself into a strange issue. I was debugging an extension and messing around with the grid layout and I ended up somehow causing an error when trying to close an empty editor group. Ever since then anytime I try to debug an extension it opens the shell of a vscode window, but quickly opens the devtools with the output below. And the vscode window never loads, its just a dead blank window. My guess (and hope) is that if I clear the local storage for that window it will be fixed. EDIT: Clearing the state for that workspace fixed the issue.

Old grammar file: file:///c%3A/Users/Eric/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/extensions/javascript/syntaxes/JavaScript.tmLanguage.json.

Old grammar file: file:///c%3A/Users/Eric/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/extensions/typescript-basics/syntaxes/TypeScript.tmLanguage.json.

Old grammar file: file:///c%3A/Users/Eric/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/extensions/typescript-basics/syntaxes/TypeScriptReact.tmLanguage.json.

Yeah, I was using insiders. I don't really know how to get into that situation -- although I have seen some strange issues (nothing I can repro reliably to really report) with debugging extensions with webviews (because they leave behind empty editor groups on reload) and very infrequently when I close one of the empty editor groups it will error out. But normally it doesn't really cause any issue, but this time it killed the window from ever loading right again (for that workspace).
My guess is that if you take my editorpart.state above, and add that to the local storage for one of your workspaces, it will repro the crash.

I have pushed https://github.com/Microsoft/vscode/commit/f0bfe7ce71b28e91b70c42622bce087f5e03807f to never fail restoring the grid as a workaround (I used to have that code before).

@eamodio Nice move with saving that editorPart.state!

Thanks 😄. I had tried to debug it, but not knowing the code and dealing with source map line/breakpoint issues I gave up and just captured the state.


Fix for the "Please Update!" text

This PR updates the URL/user agent to a newer version of the FaceApp API, which fixes the "Please Update!" text. It also changes the area in the JSON to check for filters, as that has been changed in the new API versions as well.

This update fixed it for me last night but today the message is back.


Key Value Extraction

🚀 The feature
Request for a feature where we could get the key and its value as output in forms like documents.
Motivation, pitch
I was working on Storing keys and values from forms,pdf etc and saving into rdbms,the current system dosent provide way to assign the key with values,so this feature would be of great help to all who are working on digitilisation of forms.
Alternatives
No response
Additional context
No response

Hi @mmkulkarni09 :wave:,
thanks for opening the issue.
With key value extraction you mean something like LayoutLM ?
Example: (scroll to the bottom)
https://github.com/NielsRogge/Transformers-Tutorials/blob/master/LayoutLMv3/Fine_tune_LayoutLMv3_on_FUNSD_(HuggingFace_Trainer).ipynb

Thanks for the reply @felixdittrich92,Can you Please suggest a way to directly use key-value parser on any forms by using pre-trained models?

KIE detection available in the next release


Possible fix for #356

closes #356
This works, but there may be a better way to do this, thus "possible" in the title.
And specifically, perhaps @atmorling has some insight into any relationship between #356 and the fix for https://github.com/wildlife-dynamics/ecoscope/issues/352 (which may point to a better solution).

I think something like this makes sense:
events_df["time"] = events_df["geojson"].apply(lambda x: ptime(x.get("properties").get("datetime"))
events_df = clean_time_cols(events_df)
events_df["geometry"] = events_df["geojson"].apply(lambda x: shape(x.get("geometry")))

Basically tease out the time data from the geojson before we do clean_time_cols
Then we shouldn't need any extra formatting in this function.

Oh cool, yeah if that works that sounds better! I figured you'd have a good insight on this.

If you'd like me to make a stab at that I can try, though tbh if you're keen, I think you're in a better place in re: familiarity with this section of code. So I'd be happy to just close this PR and have it superseded by your proposed fix.

Sure, will do!


Fixes samlAttributeName formats options from not appearing in dropdown

This PR will fix a bug described by David Curry in the cas-user email list.  There was a naming difference between the field sending the options in FormData from the server and the name of the field that was being used in the front end client.  This change fixes the client to match the field name sent from the server.

Coverage increased (+0.01%) to 45.503% when pulling e8db960eff636834450a229e9b781b19b0680036 on mgmt-saml-name-fix into e6d7b807bc00601c2f290acbede7a3c3f241b27d on 5.2.x.


add a new RD amit-bansal

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

rd/amit-bansal.yml
:white_check_mark:Succeeded

rd/index.yml
:white_check_mark:Succeeded

rd/map.yml
:white_check_mark:Succeeded

rd/media/profiles/amit-bansal.jpg
:white_check_mark:Succeeded

rd/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

rd/amit-bansal.yml
:white_check_mark:Succeeded

rd/index.yml
:white_check_mark:Succeeded

rd/map.yml
:white_check_mark:Succeeded

rd/media/profiles/amit-bansal.jpg
:white_check_mark:Succeeded

rd/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Update the go module and replace oci-image with oci-build and oci-publish

adds govulncheck CI job
adds golangci-lint verify check
uses new oci-build and oci-publish modules

NOTE for reviewer: the linters and linters-settings sections in the .golangci.yaml file are generated by the go Makefile module

/test pull-cert-manager-approver-policy-test

/approve


Transform.xform_inv() is not the inverse of Transform.xform()

Godot version:
3.2.1.stable
OS/device including version:
Windows10
Issue description:
The documentation claims that: xform "Transforms the given Vector3 by this transform" and xform_inv "Inverse-transforms the given Vector3 by this transform." but it is not the case.  The scale gets applied a second time instead of being undone.
https://docs.godotengine.org/en/stable/classes/class_transform.html?highlight=transform#class-transform-method-xform
Steps to reproduce:
var t = Transform(Vector3(2, 0, 0),
                  Vector3(0, 1, 0),
                  Vector3(0, 0, 1),
                  Vector3(0, 0, 0))
print(t.xform_inv(t.xform(Vector3(1,0,0))))
# --->  Result is (4, 0, 0) !!!

Minimal reproduction project:
See above.  (This has been simplified down from a more arbitrary transform I hit this problem with.)

Related to #26432? (Just a wild guess.)

What xform_inv does:
a * scale + origin
What it should do:
a / scale + origin

@Jummit, you omitted the rotation part and the origin should be subtracted, not added.
Current behaviour (for both 2D and 3D)
For a vector v and a transform t representing transformation defined by matrix M = T R S where T is translation matrix, R is rotation matrix, S is scaling matrix:
t.affine_inverse() corresponds to M-1 = S-1 R-1 T-1
t.inverse() corresponds to S R-1 T-1
t * t.affine_inverse() corresponds to M M-1 = (T R S)(S-1 R-1 T-1) = I (identity matrix)
t.affine_inverse() * t corresponds to M-1 M = (S-1 R-1 T-1)(T R S) = I
t * t.inverse() corresponds to (T R S)(S R-1 T-1) = T R S2 R-1 T-1
t.inverse() * t corresponds to (S R-1 T-1)(T R S) = S2
t.xform(v) corresponds to M v = T R S v
t.xform_inv(v) corresponds to S R-1 T-1 v
t.xform(t.xform_inverse(v)) corresponds to (T R S)(S R-1 T-1 v) = T R S2 R-1 T-1 v
t.xform_inv(t.x_form(v)) corresponds to (S R-1 T-1)(T R S v) = S2 v
So current behaviour of these methods is consistent. However naming isn't good, since it's perfectly reasonable to assume that (for example) result of t * t.inverse() should be an identity transform. There's also no xform... counter-part for affine_inverse method.
Possible changes
(for both Transform and Transform2D)

For 3.x version (as @goatchurchprime suggested):

enhancing documentation for xform_inv method (I'd say it's a must),
additionally method named something like xform_aff_inv could be added such that t.xform_aff_inv(v) would correspond to M-1 v = S-1 R-1 T-1 v.

For 4.0 version:

renaming (or removing) inverse and xform_inv methods since what these methods actually do is not what people expect them to do, they're the main reason of all this confusion (I couldn't come up with proper new names for these methods),
either renaming affine_inverse to inverse or not changing it at all,
adding xform... method corresponding to current affine_inverse method (with relevant name based on the point above).

What do you think about such changes?

This issue goes a little deeper than this. The real culprit here is the Basis class.
As it is documented, xform_inv in Basis only does the inverse transformation if the basis is a rotation-reflection. Implementation just uses its transposed version. To me this seems a serious error. A 3x3 matrix can express a very wide range of transforms and imo it is not enough to clarify in the docs that a method doesn't do what it is supposed to.
The workaround is to use Basis.inverse, but generally speaking it is not advisable to calculate an inverse of a matrix in vain.
The implementation of xform_inv in Transform looks like this:
_FORCE_INLINE_ Vector3 Transform3D::xform_inv(const Vector3 &p_vector) const {
	Vector3 v = p_vector - origin;

	return Vector3(
			(basis.rows[0][0] * v.x) + (basis.rows[1][0] * v.y) + (basis.rows[2][0] * v.z),
			(basis.rows[0][1] * v.x) + (basis.rows[1][1] * v.y) + (basis.rows[2][1] * v.z),
			(basis.rows[0][2] * v.x) + (basis.rows[1][2] * v.y) + (basis.rows[2][2] * v.z));
}

it also uses a simple transpose and even fails to utilize Basis.xform_inv.
So a full fix for this would be:

rename current Basis.xform_inv to Basis.xform_trans for those cases when you know you can use the optimization
implement Basis.xform_inv to perform a proper inverse transformation regardless of what the basis represents (Gaussian elimination?)
utilize Basis.xform_inv in Transform.xform_inv

Is this still the case in the latest documentation (4.2, 4.3)? It's a bit difficult to discern for me, given the removal of xform in favour of using the multiplication operator.

Is this still the case in the latest documentation (4.2, 4.3)? It's a bit difficult to discern for me, given the removal of xform in favour of using the multiplication operator.

AFAICT there's currently no issue with the documentation, it should be resolved in 3.x by #49662 and in 4.x by #83461.

Regarding https://github.com/godotengine/godot/issues/39433#issuecomment-1232927734:

So a full fix for this would be:

rename current Basis.xform_inv to Basis.xform_trans for those cases when you know you can use the optimization
implement Basis.xform_inv to perform a proper inverse transformation regardless of what the basis represents (Gaussian elimination?)
utilize Basis.xform_inv in Transform.xform_inv

Such changes are compatibility breaking, they could have been considered for 4.0 back then indeed, but no such changes have been done. Hence I'd say in 4.x we're settled with the current behavior, which is valid and documented (even if methods' names are questionable etc.).
So to me it seems that this issue can be closed, and if changes like suggested above are still desired then I think there should be a proposal opened for that in https://github.com/godotengine/godot-proposals (for consideration for 5.0 because of compat breaking nature).


This branch is to create news homepage

Add style for both desktop and mobile view.
Add navigation functionality for hamburger menu.
Add images and used CSS flexbox and media query to make it responsive.

Its plain Html/css project, so far looks good and responsive, good work with hamburger menu

can be merged with the main branch


nested axis x component with more than one children behaving not as expected

Not sure if this is the intended behavior but when I create a nested x-axis component within a y-axis component and the nested x-axis component has multiple children. The switch between outer y-axis component is restricted to only work when swiped on area not covered by the nested x-axis component.
Check out this http://www.webpackbin.com/EkTmBCD1M and try swiping up on any List A item. however swiping on any List B item registers fine.

I confirm, something weird is going one here.


Refactor namings

@laugustyniak any suggestions for renaming data parameter name? in Evaluator, Model, Task and Transformation ?

We can leave the data for now. I do not have a better naming idea.


Add tests for new header expose

Resolves #3

@rsamoilov sorry for the bothering :)

Hey @rsamoilov  I did what I could, please feel free to close the pull request if you felt that you need another approach.

Thanku for the comments, It should be good now


Bug: Response wont show

Your description here
Expected outcome
The response from the server should display.
Actual outcome
An error in the bottom left is shown Highlighter is unable to highlight the syntax, for then to not show any data.
If i look in the console, the following error is shown:
Variables evaluator Function not found Rgb

HTMLElement.log (chrome-extension://hgmloofddffdnphfgcellkdfbfbjeloo/index.js:102704:22)
    at HTMLElement._handleLog (chrome-extension://hgmloofddffdnphfgcellkdfbfbjeloo/index.js:102690:16)
    at HTMLDocument.handler (chrome-extension://hgmloofddffdnphfgcellkdfbfbjeloo/index.js:6026:17)
    at HTMLElement.fire (chrome-extension://hgmloofddffdnphfgcellkdfbfbjeloo/index.js:6813:6)
    at HTMLElement._callFn (chrome-extension://hgmloofddffdnphfgcellkdfbfbjeloo/index.js:67300:10)
    at HTMLElement._evalFunctions (chrome-extension://hgmloofddffdnphfgcellkdfbfbjeloo/index.js:67275:25)
    at HTMLElement.<anonymous> (chrome-extension://hgmloofddffdnphfgcellkdfbfbjeloo/index.js:67129:22)
    at <anonymous>

Versions
App: 10.0.3
Platform: 5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/61.0.3163.100 Safari/537.36
Steps to reproduce
The response is a large json string.

Hi,
Thank you for the issue report.
The variables evaluator error is not related to highlighting and displaying the response data.
I have to check how the body response display element works with invalid JSON data.

I've checked what is happening.
So in case of very large responses then the syntax highlighter won't finish work in 5 seconds it informs the application about difficulties. The app terminates the thread that is trying to parse the data and opens the "raw" view which basically just displays unparsed data.
The only possibility is that the payload is so large that even Chrome has difficulties with displaying it. In my test scenario I waited about 20 seconds before anything showed up.
There's nothing I can do about it. It's how the browser renders the view.
I'm closing this as there's nothing I can do about it.


Update ci.yml

Codecov Report

Merging #73 (1c8acb1) into main (e2aff3a) will increase coverage by 68.80%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           main      #73       +/-   ##
=========================================
+ Coverage      0   68.80%   +68.80%
=========================================
  Files         0       26       +26
  Lines         0     2382     +2382
=========================================
+ Hits          0     1639     +1639
- Misses        0      743      +743

Impacted Files
Coverage Δ

optpy-std/src/builtin.rs
0.00% <0.00%> (ø)

optpy-generator/src/lib.rs
87.09% <0.00%> (ø)

optpy-resolver/src/name/mod.rs
98.44% <0.00%> (ø)

optpy-parser/src/simplify/for_loop.rs
100.00% <0.00%> (ø)

optpy-parser/src/simplify/list_comprehension.rs
94.17% <0.00%> (ø)

optpy-resolver/src/lib.rs
100.00% <0.00%> (ø)

optpy-std/src/value.rs
0.00% <0.00%> (ø)

optpy-test-macro/src/lib.rs
94.44% <0.00%> (ø)

optpy-parser/src/expression/mod.rs
96.12% <0.00%> (ø)

optpy-std/src/cell.rs
0.00% <0.00%> (ø)

... and 16 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


VCS Collaborating Workflow

I with friend of mine run into some problems when trying to work on the same swift project through bitbucket.org. Every time after committing and pushing changes on my side and pulling on his side he gets warning that he is unable merge changes and he first need to discard some local changes. After he resolves somehow the problem and made new commit-push the same story repeats on my side.
Then we tried to share the project through drop box. But every rimes he make changes I was unable to run project because of errors like "TyphoonAssembly.h not found", or "TyphoonAssembly has duplicate" or something like that. After that I run pod update and project on my side runs well, but on his side he got new errors.
I've already seen [this post] (https://github.com/appsquickly/Typhoon/issues/59) but I'm still not clear what is the recommended workflow and how to avoid or deal with such issues. Thanks in advance and sorry for my english.

Hi @oligazar !
Thanks for opening an issue, however StackOverflow is the place to discuss usage problems like the one you're having.
If during that process we discover a bug or good candidate for a new featuer, we can open an issue here on Github :)
Thanks for using Typhoon!


enhancement: add sample gradle file

Description
Add sample build.gradle file for user help.

working from my end. @techoneel check once by running wrappid start mobile --env stage and wrappid setup

closing this issue since it's working as of now


Assignment2

Please pull changes from aiskunks:main to your yiyundotchen:main branch and make sure both are in sync. And then add your changes to your branch and raise a merge request from your yiyundotchen:main to aiskunks:main

Hi, I pulled changes from aiskunks:main to yiyundotchen:main branch,but don’t know how to raise a merge request again.
Instead of “Create pull request”,there’s only “View pull request” button.Should I change some part of the readme to resubmit it,or it’s just ok now?
Besides,how can I know if I joined the github repository or not?
Thank you for your notice,have a nice day.
From: Sagar Malhotra @.>
Sent: Wednesday, September 28, 2022 7:09 PM
To: aiskunks/Skunks_Skool @.>
Cc: Yiyun Chen @.>; Author @.>
Subject: Re: [aiskunks/Skunks_Skool] Assignment2 (PR #140)
Please pull changes from aiskunks:main to your yiyundotchen:main branch and make sure both are in sync. And then add your changes to your branch and raise a merge request from your yiyundotchen:main to aiskunks:main
—
Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Faiskunks%2FSkunks_Skool%2Fpull%2F140%23issuecomment-1261555354&data=05|01|chen.yiyun1%40northeastern.edu|fea2b0f8474b4f7794e708daa1a67494|a8eec281aaa34daeac9b9a398b9215e7|0|0|638000033539114417|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=cnq2mtIzyRiOcB7NJKjTX2%2BGzc5KQKu9nDIheDB%2Bp3U%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAVTBVA2TNUWGJM3WN2I5NIDWATFZPANCNFSM6AAAAAAQVLYRQI&data=05|01|chen.yiyun1%40northeastern.edu|fea2b0f8474b4f7794e708daa1a67494|a8eec281aaa34daeac9b9a398b9215e7|0|0|638000033539114417|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=v%2B68uge7PyTdkOB31WdffrGDdwlFJL2iJu3xqtx9vrg%3D&reserved=0.
You are receiving this because you authored the thread.Message ID: @.***>


Fix typo

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

Verified in UI screenshot. Thanks for the fix @changeworld.