[P2P Distance] Create UpdateMoneyRequestDistanceRate

Create UpdateMoneyRequestDistanceRate API command in Web-E and Auth following this plan
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0116944cbb002d044c
Upwork Job ID:<PHONE_NUMBER>350415360
Last Price Increase: 2024-02-20

Not overdue

Hey hey I'm happy to take this one one - do we have a dealine for implementing this bad boi?

Woah I totally missed that you were working on this! Thanks so much! I'm happy to help review the rest of it. Please DM me when it's ready for review of if you have any questions.

I'm trying to figure out how to get an existing $distance amount from an existing transaction

Ah yeah that part wasn't described super clearly. You can get the existing distance from the transaction NVPs. Something like this.
$transaction->getNVPs()["customUnit"]["quantity"];

Thanks @neil-marcellini ! Feel free to post-review the Auth PR if you're interested, but I'm actually OOO tomorrow so won't be continuing the Web-E changes till Monday! Here's my WIP PR for web-e: https://github.com/Expensify/Web-Expensify/pull/41495
And thanks for explaining where to get the distance!! I haven't worked much with distance requests, and it definitely isn't immediately obvious that it's in the quantity of the custom unit, so thanks for clarifying!! 🙏 hopefully I'll have that PR ready for review on Monday 🙏

Yep, check out [$500] [P2P Distance] Create a new Rate field. There's a PR up now which might be complete enough for testing [App] Feat/36985 create new rate field.

aah nice, it also looks like the "Edit rate" functionality will be added in [$500] [HOLD 36985] [P2P Distance] Create UpdateMoneyRequestRate in App, so I mayyyy wait to test with the changes made there?

@paultsimura the payload params we're expecting on UpdateDistanceRequestRate are as follows:

(string $authToken, string $transactionID, string $customUnitRateID, int $optimisticReportActionID = 0)

Aah that's totally fine if we name the report action ID param reportActionID, in our backend I'll just quickly rename it to optimistic* so it's clear we're in optimistic land :D

Nice, sounds good - let me know if you need help but feel free to work on that fix in your PR 👍 🙏

Eek, I'm moving so slow here b/c it's a bit confusing how to break a new computeDistanceCustomUnit function out of computeRouteAndCustomUnit in the most efficient way possible 😅

@neil-marcellini question for you, which I asked in another issue, here: https://github.com/Expensify/App/issues/31673#issuecomment-2051719369
This command UpdateMoneyRequestRate will be used for P2P AND for money requests in workspaces, right? If so, I might have to deal with tax updates for workspace money requests, eh?

FYI I couldn't quite get this one ready by my EOD - I had to run out a bit early, I'll either be back late this evening or over the weekend for a few hours to make up for this.
@neil-marcellini (sorry to keep pinging you) - If you have time today, it would be great if you could give https://github.com/Expensify/Web-Expensify/pull/41495 a preliminary review -> Like to see if I'm going a decent direction that you were planning in your doc. It looked easyish and straightforwardish, but currently it's breaking TrackExpenseTest and there's some psalm errors which, for the life of me, i can't figure out why they don't exist in main 😅

@neil-marcellini question for you, which I asked in another issue, here: #31673 (comment)
This command UpdateMoneyRequestRate will be used for P2P AND for money requests in workspaces, right? If so, I might have to deal with tax updates for workspace money requests, eh?

Only workspace requests. I answered in detail over on that issue here.

Thanks @neil-marcellini 🙏
Ok so it seeeeeems like we ARE going to have to deal with tax stuff possibly, but will cross that bridge in a bit 😅
For now, I'm going to have to rewrite a few things, due to us agreeing (here) that the current implementation isn't 1:1:1. Here's my new plan of action for now:
Web-E:

Continue accepting authToken, transactionID, customUnitRateID, reportActionID params
Since we don't want to do a Read call to get the transaction ID in PHP, we can't check if the transaction is a custom unit distance, AND we can't get the linked policy chat report, AND we can't get the transaction's policy data
So we'll pass the params directly to Auth to validate & do all the updates
We also can't do TransactionUtils::pushTransactionUpdate at the end, since this reads iouReportID, iouThreadReportID, both report's actions, their NVPs, and possibly more

Auth:

Do all the things Web-E was going in point 2 above in Peek
Also in Peek, Compute the custom distance amount & merchant, and custom unit info (from TransactionAPI::getP2PDistanceRequestRate or TransactionAPI::getDistanceRequestRate)
Use those bits ^ in the currently existing auth.moneyRequest.updateRate function
Figure out how to send all onyx updates via `queueOnyxUpdates(, , )

Thanks Alex, that plan sounds pretty good. I think you'll be happy to hear that my top focus right now is this Migrate TransactionUtils::pushTransactionUpdate function to auth. Once that's done you should be able to use it in the new UpdateMoneyRequestRate Auth command.
I'm tackling that migration in two parts. In the first part I'll move most of it to Auth, and leave a small part in PHP to send the modified expense report action and the violations, because it's a massive amount of work to move those over. I'm working on a plan to distribute that migration work across the team, which will be the final part of moving pushTransactionUpdate to Auth.

Oh siiiick, that's great that we're taking that in chunks and i won't have to think about that part 😅 I'll get started on other pieces of the plan tomorrow, taking into consideration that i won't have to deal with migrating pushTransactionUpdate 👍 🙏

I like this plan, I can also make this change in the Auth command 👍

Web-e PR is up and waiting on Auth PR
Auth PR is slowly in progress, but is waiting on at least 2 other PRs most likely (Monil's PR for some Tax updates in Auth & Neil's PR for moving pushTransactionUpdate to Auth

I should be able to get back into this tomorrow, though i still believe i won't be able to complete this until the other tasks I mentioned are done - I'll try to find those WIP PRs to link here

Tax stuff: https://github.com/Expensify/Auth/pull/10517

Ooh ok some updates:

ForexUtils::format was moved to Currency::format, which had some mega refactoring in https://github.com/Expensify/Auth/pull/10625 (merged)
Transaction::generateTaxUDF was built in https://github.com/Expensify/Auth/pull/10517 (merged)
Transaction::pushTransactionUpdate still being implemented in https://github.com/Expensify/Auth/pull/10487

Alright @neil-marcellini  - my Auth PR https://github.com/Expensify/Auth/pull/10638 is feelin much better - I haven't added in the stuff from ^ above PRs, but it's at least doing some of the web-e stuff... Do you mind taking a quick look to give any thoughts on where it's at for now?

Thanks for the initial review! Good questions, I got a few resolved but didn't have enough time (half day today & tomorrow) so will try to move forward tomorrow or early next week

WIP! Currently working on updating Currency::format to be able to format doubles as well, since currently it only accepts ints

FYI https://github.com/Expensify/Auth/pull/10809 just got merged, which unblocks https://github.com/Expensify/Auth/pull/10638 - hoping to get that one finished this week, as i believe all the other PRs it was held on are merged!

Made a bit of progress adding a few more tests today, didn't have much time w/ deploy duty

Ok @neil-marcellini I think I made a lot of progress on the PR today - I added auth.transaction.pushTransactionUpdate and related tests, fixed a few things, and made sure currency doubling looks good.
Mind giving it another text in the next few days? I'm probably going to have to chat with @MonilBhavsar to figure out what kind of tax stuff i may have to incorporate as well

Had to prioritize some API optimizations today, hoping to get back to this wednesday

tryiingggg to get this done by EOW

@MonilBhavsar is planning to help with the tax changes early this week, since tax stuff is confusing a.f.

Commenting again to hopefully remove the Overdue label

Waiting for updates from @MonilBhavsar on the PR

Auth PR was merged!! 🚀  I'm OOO this week but will get to the web-e changes to make sure it works well once I'm back!

ermagerd all PRs are finally merged! And web-e is on staging, so we're almost done with this!

Web-E Went to prod 4 days ago! @neil-marcellini want me to close this out? ORr should we keep it open while the App PR is being tested w/ all the latest code?

@Beamanator nah we can close this, there's a separate App issue to implement the front end part. You're work here is done! Unless of course Pavlo has found a legitimate bug 😬 😆
[$500][P2P Distance] Create UpdateMoneyRequestDistanceRate in App

Yo @paultsimura I'd be happy to help look into this, could you give me more specific test steps so I can try to reproduce & figure out exactly what you're seeing? :D

Nice, ok so there's definitely a bug, it LOOKS like the problem is that some C++ code isn't accurately converting an integer value into a float 😅

Ok ya so the problem

@paultsimura Most likely tomorrow, as long as it gets reviewed and merged today

Seems a lot of people were off at the end of last week for the 4th of July, so the changes have not been merged yet 🙏

Fix got to Prod end of last week! 🚀


Renderdoc shows that attachment is discarded every vkCmdNextSubpass() call if the attachment has VK_ATTACHMENT_STORE_OP_DONT_CARE

Description
We are creating a render pass with some attachments having a store op VK_ATTACHMENT_STORE_OP_DONT_CARE. Right-Clicking on one of these attachments shows this:

The line marked as GREEN shows the correct discard call, which happens when we end the render pass. The ones marked with a RED line, they show the wrong discards and clicking it will show the vkCmdNextSubpass() call:

The line marked as BLUE is also a false discard call (or a discard call shown incorrectly) but this time is when a barrier is added for this image before the render pass.
This attachment is loaded with VK_ATTACHMENT_LOAD_OP_CLEAR.
Changing the store op for this attachment to VK_ATTACHMENT_STORE_OP_STORE will result in the expected behavior:

This happens for all the attachment that have VK_ATTACHMENT_STORE_OP_DONT_CARE as their store op in the same render pass. The behavior of the attachment throughout the pass is correct and the final image is also correct, so we believe it might be an issue with RenderDoc displaying wrong information. What we would expect to see is the discard label only when vkCmdEndRenderPass is called.
Steps to reproduce

Create a render pass with more than one subpasses
Set the store op of one of the attachments to VK_ATTACHMENT_STORE_OP_DONT_CARE

This should be enough to reproduce this.
What we run in our application is a regular deferred render pass with a GBuffer subpass, an in-between subpass, a lighting subpass and a post fx subpass. Overall, we have 5 color attachments (3 gbuffers, 1 for the in-between subpass and the final color) + a depth-stencil attachment. The GBuffer attachments and the in-between attachment have store op VK_ATTACHMENT_STORE_OP_DONT_CARE since we don't care about them after the lighting subpass. The attachment usages are:
GBuffer subpass:

GBuffer0 -> Color attachment (render target)
GBuffer1 -> Color attachment
GBuffer2 -> Color attachment
In-Between attachment -> Unused
Final color -> Unused
DS -> Write DS

In-Between subpass:

GBuffer0 -> Preserve
GBuffer1 -> Preserve
GBuffer2 -> Preserve
In-Between attachment -> Color attachment
Final color -> Unused
DS -> Write DS

Lighting subpass:

GBuffer0 -> Input
GBuffer1 -> Input
GBuffer2 -> Input
In-Between attachment -> Input
Final color -> Color attachment
DS -> Input

Post fx subpass:

GBuffer0 -> Unused
GBuffer1 -> Unused
GBuffer2 -> Unused
In-Between attachment -> Unused
Final color -> Input
DS -> Input

Environment

RenderDoc version: v1.26
Operating System: Windows
Graphics API: Vulkan
GPU: RTX 2070 super (also tested on 3080)
GPU Driver: 531.29

Just to be sure I have this clear since I don't have a direct repro case and can't test this, the only issue is the multiple discard listings within the multi-subpass renderpass? The discard patterns aren't showing up incorrectly with filling the wrong attachment or at the wrong time?
I can see that yes discard usage will be added every time instead of just at the end, though in theory it's undefined where the discard actually happens between final use of an attachment & end of render pass, in RenderDoc it logically happens at the end.

Thank you for your very quick response!
Yes, we believe that RenderDoc is incorrectly showing that the contents of the attachment were discarded even though the attachment is later used as an input, color attachment or preserved. As you can see in the first picture , the attachment is used after a discard label is shown. For example, after the clear (which happens when the attachment is loaded) there is a discard, then the attachment is used as a color attachment (Where the green flag is) then it shows to be discarded again and then it's used as input. We can clearly see that the attachment is properly used as input later and we don't get any validation errors from the validation layers (or any other visual artifacts).
We only expect to see a possible discard at the last subpass where the attachment is no longer used and/or when we end the render pass. What I cannot tell is what the driver actually does. If the driver indeed discards the contents with every vkCmdNextSubpass call then what render doc shows us should be correct but it would mean that we should probably see garbage values for these attachments in every subpass, but we don't, we see that the attachment is correct. This is why we strongly believe this might be an issue related to RenderDoc.
Thanks again for the quick response and if you need any more information let me know. Unfortunately, I cannot share any captures with you as this is a private project.

No problem at all, I just wanted to make sure I wasn't missing the main bug if it's only this. The resource usage is entirely generated by RenderDoc, it looks like there's just an omission that applied discards on every subpass - likely not noticed before now because subpasses are rarely used.
It's fine that you can't share a capture - most likely I would not be able to load it successfully since really only desktop captures can be portable, there is too much hardware/driver variation that can't be resolved for captures from mobile devices. That commit should fix it I believe, but it would take me longer to reconstruct your repro case (given its complexity) than it is for you to re-test. If you're able to build RenderDoc could you try to see how it behaves with that change, or if not could you check the next nightly build and update this issue?

Perfect, thanks!
I will give the next nightly build that includes this commit a try and let you know.

Hi @baldurk, just took a look at the latest nightly build and the issue is now fixed:


Cleanup docs and types for upcoming release

These were notes from @themoonrat on Slack.

[x] Add proper Renderer.extract deprecation
[x] Improve docs for backgroundAlpha & contextAlpha (rename to useContextAlpha)
[x] Remove package names from deprecation warnings for resources
[x] Fix BlobResource namespace in documentation
[x] Add proper RenderTexture.create deprecation for (width, height, scaleMode, resolution)

Codecov Report

Merging #7135 (72cc26d) into dev (0bff570) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               dev     #7135   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           17        17
  Lines          699       699
=========================================
  Hits           699       699

Continue to review full report at Codecov.

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

Codecov Report

Merging #7135 (72cc26d) into dev (0bff570) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               dev     #7135   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           17        17
  Lines          699       699
=========================================
  Hits           699       699

Continue to review full report at Codecov.

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


How to turn it OFF into an Apache Virtualhost Directory?

Hello,
i'm on an Apache Directory
I tried
<IfModule pagespeed_module>
ModPagespeed off //this one alone, the below alone, even the 2 together
ModPagespeed unplugged
</IfModule>

nothing to do.

If I turn it off on pagespeed.conf I can't enable it into the Directory
If I turn it on on pagespeed.conf I can't disable it into the Directory

am I missing something... ?
Thank you

I had to do
<IfModule pagespeed_module> ModPagespeed unplugged </IfModule>
outside the Directory
but this way it disables mod_pagespeed for the entire Virtualhost.
At the moment, I can't understand if possible to disable mod_pagespeed per Directory
Can you better explain this matter, please?
Thanks

Can you try 'standby'?  Here's a brief description on the names:

ModPagespeed standby: disables the module in a way that it can be turned on again by query-param or at a lower level
ModPagespeed unplugged: completely disables the module for the server, with no way to turn it on by query-param or in a subdirectory
ModPagespeed on: enables the module in the directory & below, as long as the state is not unplugged

Brief history: we originally had "on" and "off".  Later we added unplugged because we wanted site owners a way to hit a big kill switch in case something went wrong. off meant standby in MPS, but off meant unplugged in ngx_pagespeed (NPS), which was an oversight when we built NPS.
To make this consistent (since the modules share doc) we decided to deprecate off and ask users to explicitly say unplugged or standby which work the same way in both servers.

I'm on 1.12.34
so, I can't even use standby and it returns error on the syntax
for now, I need to
ModPagespeedDisallow "*mydomain.com/mydir/*" inside pagespeed.conf
anyway, thank you

On MPS 1.12, "off" should work as you described, but Disallow is fine as well.

I tried "off" but it didn't work in the Directory, probably I got confused by my cookieless domain
eg. static.domain because in the page source I still saw many pagespeed lines...
so, in order not to write other rules for the static domain, I used disallow...
ok, thanks for now


Tooltips vanish immediately after they popup

Describe The Bug
Tapping the small info "i" icon shows the tooltip but then the tooltip disappears before you can read it. Example in Settings > Network > mDNS Resolver.
iOS 17.2.1 / Safari
Logs
No response
Config
No response
Homebridge UI Version
4.55.1
Homebridge Version
1.7.0
Node.js Version
20.11.0
Operating System
Official Homebridge Raspberry Pi Image
Environment Info
None
Raspberry Pi Model
Raspberry Pi 1 B

Not stale

Hi @jamieburchell
In fact I just so happened to remove some of these tooltips in the beta version of the UI. Perhaps you can give it a try!

This should be fixed in the new releases of the UI.


Strange behavior of regex

Version
2.5.17
Reproduction link
https://jsfiddle.net/eywraw8t/466635/
Steps to reproduce
Start to enter a valid email and the flag isValid takes the wrong value on a valid expression. For example, if you specify a regular expression in its pure form, then unexpected behavior is not noticed.
What is expected?
It is expected that to validate a field, it will be possible to import rules that lie outside an instance or function.
What is actually happening?
It has the form of a sequence: "true", "false", "true", etc. Dependent on the length of the string, since the veracity of the expression.

It's not related to Vue. RegExps in JavaScript can be stateful.

You can learn more on MDN:

test() called multiple times on the same global regular expression instance will advance past the previous match.


examples are missing

[Enter feedback here]

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

ID: 078ae3f6-204f-923d-1804-00b5bebd370a
Version Independent ID: 8c65025a-a737-5597-adb0-3b421e48da1a
Content: Work item Batch Updates - Azure DevOps Services REST API
Content Source: docs-ref-conceptual/6.0/wit/WorkItemBatchUpdate.md
Product: devops
Technology: devops-ref
GitHub Login: @wnjenkin
Microsoft Alias: whjenkin

Examples are missing

@oktep thanks for reporting we will try to get this sorted out.
Related to #455


Error Computation

Hi,
I noticed in your code for error computation ( evaluateActionWise ) the cumulative error is the average of errors for all actions. Ideally, it should be a weighted average. I observed that per action averaging gives slightly better error than the weighted average. Is that also how the numbers in your paper are reported ? Correct me if I'm wrong, the numbers reported in the Protocols for Human3.6 report average error across all test frames, so it should be a weighted average there.

Ideally, it should be a weighted average.

How would that be ideal?
Where would the weights come from?
Do you know of other papers that do that or do you think that is the protocol that other papers follow?
Please point us to sources before saying that we have an error in our results, or that other ways of reporting results would be more ideal. Please understand that what you are saying is not a minor claim.

I observed that per action averaging gives slightly better error than the weighted average. Is that also how the numbers in your paper are reported ?

What you see in this repo is what we reported. You can reproduce all the results yourself -- this is why we release the code.

Correct me if I'm wrong, the numbers reported in the Protocols for Human3.6 report average error across all test frames, so it should be a weighted average there.

Like I said, this is what this code does. If that is what you see in the code, that is what we report in the paper. You can train yourself and double check.
If you have trouble reproducing our results, please open another issue.

Hi, my intention isn't to make spurious claims, so let me qualify the statement.
I'm referring to line 248 and line 275 in predict_3dpose.py. Passing the evaluateActionWise flag would yield an average across actions, else an average across the entire test set. From the documentation, it seems evaluateActionWise is used for Eva, where numbers are reported action and subject wise. I'm not sure that is what you're doing, so please confirm.
By weighting I'm implying weighting using the number of samples in each action, so that the mean gives the true average across the entire test set, else actions with higher and lower number of test frames are given equal weightage.
No paper mentions that they average the error across actions, so I'm assuming the reported errors are the average error across the entire test set. If you can point me to a paper that says this is what they do, I'd be happy to admit my assumption is false.

Hi, my intention isn't to make spurious claims, so let me qualify the statement.

Thanks! Sorry I don't mean to overreact. Also stressed with the deadline over here.

From the documentation, it seems evaluateActionWise is used for Eva, where numbers are reported action and subject wise. I'm not sure that is what you're doing, so please confirm.

For HumanEva we did the same as for H3.6M. That is, compute each error individually and then report the average of that. A simple way to verify that is to compute

By weighting I'm implying weighting using the number of samples in each action, so that the mean gives the true average across the entire test set, else actions with higher and lower number of test frames are given equal weightage.

That makes sense if you want to weight some actions more than others, but I don't see why you would want to do that -- actions with more frames could be easier, so average of averages makes more sense in  my head, as it forces us to do well on all the actions. I think this is rather subjective.
However, to the best of my knowledge, previous work follows our protocol -- that is, the average of averages across actions. A simple way to verify this is to copy the table results to a spreadsheet, and see that the avg column is indeed the average of the other numbers. This is true for our tables, which include results we took from previous work. Eg in

In any case, if you think reporting the error over the entire set makes more sense, I suggest you report both numbers in your paper, as that number by itself would not be comparable to results reported in previous work (including ours).
Good luck with the deadline!

Thanx a lot for the clarification. Good luck with the deadline as well !


Add ability to view all Tag Tabs

Closes #9625
Closes #7405

Supports all operations from Tag Tabs
Supports swap/insert mode
Can be saved as a Tag Tab with tagtabs as the tab name

Is there a reason this isn't allowed to be in the client?

Sorry adam I didn't mean it like that but I really want this

On my end I only see "all checks passed", hence my confusion.
I mean no harm.

Is it possible to make the new worn items button to be replaced with the Runelite + button? Toggle maybe?

#10575 solves that


(BOLT-365) Update nodes usage

Remove Eg. as inconsistent docs practice. Update the documentation and example.
New output
    -n, --nodes NODES                Node(s) to connect to in URI format or group name (Optional)
                                      --nodes localhost,node_group,ssh://nix.com:2222,winrm://windows.puppet.com

                                     * URI format is [protocol://]host[:port]
                                     * NODES can either be comma-separated, '@<file>' to read
                                     * nodes from a file, or '-' to read from stdin
                                     * Windows nodes must specify protocol with winrm://
                                     * protocol is `ssh` by default; may be any TRANSPORT listed below
                                     * port defaults to `22` for SSH
                                     * port defaults to `5985` or `5986` for WinRM, based on the --[no-]ssl setting

@katelopresti thoughts on these updates?

  * NODES can either be comma-separated, '@<file>' to read
  * nodes from a file, or '-' to read from stdin

wrapping with * confuses me.
* Windows nodes must specify protocol with winrm://
* protocol is `ssh` by default; may be any TRANSPORT listed below
* port defaults to `22` for SSH
* port defaults to `5985` or `5986` for WinRM, based on the --[no-]ssl setting

especially the first line seems confusing without  "unless otherwise configured"

Updated.

CLA signed by all contributors.

Updated for text specified on the ticket. Also removed Optional because it was pretty much never helpful.


Duration cleanup

A cleanup of the duration pull request (#22):

duration code and documentation after review. Also pushing the data repo with the TTE files used in the documentation example.

remove test data from test package and have them downloaded using gdt-data.
unit testing needs more work.

Closed by pull request #40


Connecting to the KWB module

Hi,
If I understand correctly, you can use the module with serial connection or with direct connection with RJ45 cable from KWB module to the computer or am I missing something? I have my KWB module connected to the web but can't get any data or even connect with it thru my router.
Thank you

The connection is always serial for this module. Newer versions of the easyfire might have direct connection options, but this module does not support this.


[BUG] Backport of https://github.com/rancher/rancher/issues/41746 PR to 2.7 release line

In case folks are looking for the link: https://github.com/rancher/rancher/issues/41746

There is no customer justification for it to be backported. @CamrynCarter  said there needs to be a customer justification if a backport is required.
So, I will close this. @diogoasouza Let me if I am wrong.


Unintended attribute name for actor's hp in data structure?

It is rather minor issue and most likely just a small naming mistake. I just noticed that base hp refers to max hp, but max hp attribute does not? I simply cannot find what hp.max is used for in such case.
If this structure was intended tho, I will close the issue report, cheers.

This is intended.


oncotree_to_oncotree.py should support python3

Update to python3
Set interpreter and add to docs
Update OncoTree-Mapping-Tool.md

Co-authored-by: thomasyu888<EMAIL_ADDRESS>Co-authored-by: Thomas Yu<EMAIL_ADDRESS>Co-authored-by: thomas.yu<EMAIL_ADDRESS>

#208 has been merged to a feature branch to facilitate the running of Jenkins tests. Jenkins unit tests have failed with this message:

#208 has been merged to a feature branch to facilitate the running of Jenkins tests. Jenkins unit tests have failed with this message:

@thomasyu888 : I've updated your changes to run cleanly on our python 3.4 installation. Unit tests are still failing for a different reason, but we will try to work that out today.
Could you download this branch (convert-to-python-3) and test out the updated script on your system / with your version of python and make sure it runs as expected? I assume that code which runs in python 3.4 will also run on python 3.6/3.7...  but we should make sure.

@thomasyu888 : I've updated your changes to run cleanly on our python 3.4 installation. Unit tests are still failing for a different reason, but we will try to work that out today.
Could you download this branch (convert-to-python-3) and test out the updated script on your system / with your version of python and make sure it runs as expected? I assume that code which runs in python 3.4 will also run on python 3.6/3.7...  but we should make sure.

Works on 3.7 and 3.8.

Works on 3.7 and 3.8.

Jenkins has been configured to include python3 in its execution path and tests for the scripts module are now running properly, but a failure is now being observed because some tests are assuming output order of a dictionary key set, which seems to have been altered between python 2.7 and python 3.4:
14:16:43   File "/srv/www/jenkins/jobs/oncotree/workspace/scripts/test/test_oncotree_to_oncotree.py", line 82, in test_resolve_multiple_future_possible_target_oncotree_codes_no_new_children
14:16:43     self.run_resolve_oncotree_codes_test("ALL", set(["BLL", "TLL"]), False, format_oncotree_code_options("ALL", "{BLL,TLL}", 0), False)
14:16:43   File "/srv/www/jenkins/jobs/oncotree/workspace/scripts/test/test_oncotree_to_oncotree.py", line 200, in run_resolve_oncotree_codes_test
14:16:43     self.assertEqual(expected_oncotree_code_option, actual_oncotree_code_option)
14:16:43 AssertionError: 'ALL -> {BLL,TLL}' != 'ALL -> {TLL,BLL}'
14:16:43 - ALL -> {BLL,TLL}
14:16:43 ?         ^   ^
14:16:43 + ALL -> {TLL,BLL}
14:16:43 ?         ^   ^
Tests checks need to be altered to compare collections without output order assumptions

Jenkins has been configured to include python3 in its execution path and tests for the scripts module are now running properly, but a failure is now being observed because some tests are assuming output order of a dictionary key set, which seems to have been altered between python 2.7 and python 3.4:
14:16:43   File "/srv/www/jenkins/jobs/oncotree/workspace/scripts/test/test_oncotree_to_oncotree.py", line 82, in test_resolve_multiple_future_possible_target_oncotree_codes_no_new_children
14:16:43     self.run_resolve_oncotree_codes_test("ALL", set(["BLL", "TLL"]), False, format_oncotree_code_options("ALL", "{BLL,TLL}", 0), False)
14:16:43   File "/srv/www/jenkins/jobs/oncotree/workspace/scripts/test/test_oncotree_to_oncotree.py", line 200, in run_resolve_oncotree_codes_test
14:16:43     self.assertEqual(expected_oncotree_code_option, actual_oncotree_code_option)
14:16:43 AssertionError: 'ALL -> {BLL,TLL}' != 'ALL -> {TLL,BLL}'
14:16:43 - ALL -> {BLL,TLL}
14:16:43 ?         ^   ^
14:16:43 + ALL -> {TLL,BLL}
14:16:43 ?         ^   ^
Tests checks need to be altered to compare collections without output order assumptions

Tests checks need to be altered to compare collections without output order assumptions

This has been accomplished by making output order deterministic / sorted.


Correction for raw affiliation Department of Physiology and Pharmacology, Western University

Correction needed for raw affiliation Department of Physiology and Pharmacology, Western University
raw_affiliation_name: Department of Physiology and Pharmacology, Western University
new_rors: 02grkyz14
previous_rors: 05cgtjz78
works_examples: W4384028982;W4367301719;W4389938404;W4389083180;W4389947649;W4389099140
contact: 3afbe738474e487d48a02949afbbe232:09db8a25dbce681c @ uwo.ca

This issue was accepted and ingested by the OpenAlex team on 2024-12-19. The new affiliations should be visible within the next 7 days.


"_update_ps1: command not found" when launching an inner shell

I'm not sure if this is an issue but if by mistake I launch an inner shell like so sh (bash) all the prompt that follows will not work properly and I will get back the following error for each line prompt
_update_ps1: command not found

I wasn't able to reproduce this. If you are still having this issue could you maybe provide a screen shot? Since this issue is really old I'd like to close, but feel free to open if need be.

Just git pulled and re-run install.py and this still happens! I'll try and get a screenshot but you should be able to reproduce by typing 'su anyothernormaluserhere' in a normal shell.

@jay-to-the-dee are you exporting _update_ps1 in your bashrc? The README used to say to do
export PROMPT_COMMAND="_update_ps1; $PROMPT_COMMAND"

But we have since changed this to recommend doing just:
PROMPT_COMMAND="_update_ps1; $PROMPT_COMMAND"

See https://github.com/milkbikis/powerline-shell#bash. I think this will fix your issue.

Boom! And that indeed fixes the bug! Thanks a lot! :)

Great! no problem

Thanks a lot....
that fixes  my problem


Multiple App Registration vs Using Same Bot AAD ID

[Enter feedback here]
Can  A developer use the same Bot ID (AAD ID) to expose an API and assign permissions.
Scenario 1 :

Scenario 2 :

In the above scenarios developer portal fields would be updated like the following:

What would be the Microsoft Recommended Approach ?

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

ID: 11c68c28-b6d6-ef7e-f8f9-ebebe487378b
Version Independent ID: c5985867-3795-4280-ef0d-1d66ee30da6e
Content: Enable resource-specific consent in Teams - Teams
Content Source: msteams-platform/graph-api/rsc/resource-specific-consent.md
Service: msteams
GitHub Login: @AkJo
Microsoft Alias: lajanuar

@osheengithub - It is recommended to use same Bot app ID to expose an API and assign permissions.

But the docs mentions new AD App registration.
Also when we provision Teams Toolkit Sample using latest VS 2022, it provisions 2 different Apps on Azure AD, One for Bot and one for AAD
Earlier it use to provision a single app. Can we get some clarity on this.

Hello @osheengithub - We will re-verify this, also we will check with engineering team and get back to you.

Hello @osheengithub - Sorry for delay in response.
We will check the updates and let you know.


Per-Environment Base Offsets for 'Parent to Base Game UI' (Plus others.)

Is it because of an issue?
The base game UI is not in the same orientation for every environment. For example, EDM makes the UI elements smaller and by extension, lower. Counters+ UI elements do not account for this even with 'parent to base game UI' and the setting that attaches the Z position both on.
Describe your solution
Per-environment size/position offsets that are built into the code. (New environments would of course just use the default environment value prior to updating the mod for them.) As long as the UI follows the differences in base UI positions properly, there shouldn't need to be any change to the in-game customization settings. Disabling the 'parent' option should be sufficient to disable this automatic adjustment as well.
A programmer yourself?
I've only worked extensively with the code base in Second Life, almost a decade ago, which used 'LSL' and 'Mono'. I've heard they're a mix of Java and C# however, and I do intend to learn how to code mods eventually, so I may in fact eventually come help with this, as there are other 'enhancements' that I have in mind for this mod.
Other enhancements in mind;

A 'display example values instead of labels' toggle.
'Display combo (with/without FC bars) and multiplier elements in customization environment' toggle,
Select preview environment (should tie in with the original enhancement's base UI offsets.)
Additive offsets for base game UI which apply to the Parent function.
Make the Z position slider become an additional offset when 'use base Z value' is ticked.
Finer per-counter height adjustments than just -1 to 4.
Per-counter text size adjustment.

It sounds like an existing mechanism to try and automatically find those environment offsets broke with a recent update (or recent environment). I'll have to look into it.

Oh! Well hopefully it's a simple fix then, as, yeah, specifically with the EDM environment if I have something sitting below the combo counter, said counter will have sunken down and overlapped that hud element.
If it matters: I am on 1.27 via Legacy Launcher, and using Counters+ v<IP_ADDRESS>


[SIEMINT-85] DDS: Trend Micro Vision One XDR: Crawler Integration v1.0.0

What does this PR do?
This is a initial release PR of Trend Micro Vision One XDR integration including all the required assets.
Additional Notes

Crawler code for this integration has been committed in its respective repo
Pipeline and Facet group created for this integration are available in our sandbox and would be shared separately with the required teams.
Samples for the pipeline review would also be shared separately with the required teams.
OOTB detection rules JSON would be shared separately with the required teams as a part of separate repository.
Since during the standard attribute remapping we are not preserving the source attributes as per suggested best practices, it would result in filters using these standard attributes populating the values of other integrations as well as per current Datadog behaviour

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Changelog entries must be created for modifications to shipped code
[ ] Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
[ ] If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

Thanks, created DOCS-8655 to review

Docs are approved, looks like there's a merge conflict to resolve though

@cswatt, I have resolved the merge conflicts

The test samples allow to test nested pipelines, the only non standard facet is correctly namespaced, and the remappers look ok. I move it to testing in staging. One question, is it intentional that the service tag is preset by the log source in this integration ?

Yes, It will be preset as it's a crawler based integration and it will be added in code only.


Clean up and document Gradle Managed Devices

See the additions to the readme, but the short version is to run ./gradlew gmdTestQuick to run all instrumented tests quickly.

First time running, ran into this:
Execution failed for task ':designcompose:gmdApi33Setup'.
> A failure occurred while executing com.android.build.gradle.internal.tasks.ManagedDeviceInstrumentationTestSetupTask$ManagedDeviceSetupRunnable
   > java.lang.IllegalStateException: Snapshot setup ran successfully, but the snapshot failed to be created. This is
     likely to a lack of disk space for the snapshot. Try the cleanManagedDevices
     task with the --unused-only flag to remove any unused devices for this project.

First time running, ran into this:
Execution failed for task ':designcompose:gmdApi33Setup'.
> A failure occurred while executing com.android.build.gradle.internal.tasks.ManagedDeviceInstrumentationTestSetupTask$ManagedDeviceSetupRunnable
   > java.lang.IllegalStateException: Snapshot setup ran successfully, but the snapshot failed to be created. This is
     likely to a lack of disk space for the snapshot. Try the cleanManagedDevices
     task with the --unused-only flag to remove any unused devices for this project.

Did you try what the error suggested?

I cleaned then ran the quick test, and got:
> Task :tutorial-app:mergeDebugAndroidTestTestResultProtos
Test execution completed. See the report at: file:///data/src/automotive-design-compose/reference-apps/tutorial/app/build/reports/androidTests/managedDevice/debug/allDevices/index.html

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':tutorial-app:gmdAtdApi30DebugAndroidTest'.
> Computer Architecture Environment not initialized properly.

I suspect there's something wrong with your system, hopefully it's an easy fix. Let's jump on a chat to figure it out when you have time.

@iamralpht could you try running ./gradlew gmdTestQuick when you get a chance? I spent some time with Tim and we were unable to get this to work on my machine.


OT appears ahead of OTel in searches

@austinlparker posted on Slack, https://cloud-native.slack.com/archives/C02UN96HZH6/p1650296911544559:

https://twitter.com/bwplotka/status/1515715889299136517
@bwplotka
It's crazy and confusing to NOT see @opentelemetry on the first page of @Google search for "tracing Go" or Go client for tracing...:crying_cat_face:
@opentracing docs pop up and are nice (deprecated though).
I don't what happened, but this needs some work, especially for new in the ecosystem:see_no_evil:

[x] https://github.com/opentracing/opentracing.io/pull/462

I'll track the GA/GSC for a little while before closing this.

Added you to the OpenTracing GA, check email.

Thanks @austinlparker. Could you grant me edit access so that I can also view the Google Search Console results?

OT no longer appears before OTel pages -- though there are other pages before any OTel website pages show up. I'm going to close this now that the original concern has been addressed.


<= ( 1 + 2 ) not working

hi, i was testing few examples and looks like this case is not working. is it valid?
res: peg$SyntaxError {message: "Expected "-", ".", [0-9], [?], [A-Z], [_], [a-z], …ꯥꯨ꯭ﬞ-︠-︯], string, or whitespace but "(" found.", expected: Array(10), found: "(", location: Object, name: "SyntaxError"…}

+1

@gustavomick This is an invalid expression. According to the FEEL grammar, the right-hand side token in a "simple positive unary test" (the type of expression as mentioned) can only be a literal or a variable (called a name in FEEL) and cannot be an expression. But <= ( 1 + 2 ) can be achieved by assigning (1 + 2) to a context variable, and using the variable name in the expression as illustrated in the below example,

const FEEL = require('./dist/feel');

var context = '{a : (1 + 2)}'
var text = '>= a';
var parsedContext = FEEL.parse(context);
var parsedGrammar = FEEL.parse(text);

parsedContext.build().then(ctx => {
    return parsedGrammar.build(ctx);
}).then(result => {
    console.log(result(4));
}).catch(err => console.log(err));

Thanks for the explanation, i checked that ( page 108 ).
My example thou was may be too trivial, i was thinking more about setting a threshold/alarm example, where if X <= ( Y * 1.10 ) => True.
if you don't mind to give me your thoughts on this. thanks.

I think you can do "if X <= ( Y * 1.10 ) then true else false" and take X and Y as part of context or payload.

@pragyandas thanks but in this context i have 2 inputs, X and Y , both are numbers, so i have to return X if my expression is at column X so that rule resolves as true / valid.  "if X <= ( Y * 1.10 ) then X else !X" is like that??
this example is related to discussion on usability, where something that should be very trivial to write for user, ends convoluted bc spec has maturity few pendings. thinking out loud thou, appreciate your feedback on this matter. thanks


Restore input checked attribute special case

Turns out the DOM is annoying and removeAttribute("checked") does remove the checked attribute but HTML latches onto user input state if the checkbox is toggled manually.
Demo in the playground:
export default function StickyCheckbox() {
  const state = useState({ checked: true })
  return <>
      <input type="checkbox" checked={state.checked} />
      <button onClick={() => { state.checked = false; }}>Reset</button>
  </>
}

Observe that the reset button works correctly if you don't touch the checkbox
Toggle the checkbox manually off/on
Observe that the reset button does not work anymore

It's possible that it's required for option selected as well.

Hit a somewhat expected snapshot error, but would have expected the snapshot to at least contain checked instead of checked="true"
  ● kaiku › should toggle attributes for certain boolean attributes

    expect(received).toMatchSnapshot()

    Snapshot name: `kaiku should toggle attributes for certain boolean attributes 4`

    Snapshot: "
<input type=\"text\" disabled=\"true\"><input type=\"checkbox\" checked=\"true\"><select><option selected=\"true\"></option></select>
"
    Received: "
<input type=\"text\" disabled=\"true\"><input type=\"checkbox\"><select><option selected=\"true\"></option></select>
"

Fixed in c0db6d1590f606643b36626f7050eea8cde80a05


[PG-144] Vertical: Initial Plugin Version

[x] Setup repository
[x] Adapt CI and set merge prerequisites
[x] Fix initial lint issues
[ ] Adapt README
[ ] Support Binary Switch
[ ] Unit Test Coverage

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@855c7d2). Click here to learn what that means.
The diff coverage is n/a.

@@          Coverage Diff           @@
##             main      #1   +/-   ##
======================================
  Coverage        ?   4.00%
======================================
  Files           ?       4
  Lines           ?      50
  Branches        ?       4
======================================
  Hits            ?       2
  Misses          ?      48
  Partials        ?       0

Continue to review full report at Codecov.

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


95 percent limits

For example, data sets like https://github.com/jxeeno/aust-govt-covid19-vaccine-pdf/blob/master/docs/data/2021-10-07.json now have a lot of entries "stuck" at 95%:
                {
                    "ageLower": 80,
                    "ageUpper": 84,
                    "firstDoseCount": 509508,
                    "firstDosePct": 95,
                    "secondDoseCount": 434980,
                    "secondDosePct": 82.3,
                    "femalePct": 95,
                    "malePct": 95
                },

Is the data source limiting that, or is that happening somewhere in the data-acquisition workflow of this repo?
(Maybe the people who put together the published data realize that the base denominators for population are not accurate enough, and they want to avoid "hard questions" if the reported rate in some category gets up to 99% and then ticks over to 100.1%!)
When we understand the reason, there could be a note about that in README.

Yeah, I think this is probably something we'll let consumers of this data feed (e.g. yourself) to handle derivatives or interpretations of data.

True - the reverse-engineered percentages are derived from the raw data anyway, so better that this repo is just responsible for providing various convenient views/formats of the actual raw data.


Set initial value on the propsFactory in the repo constructor

Which @ngneat/elf-* package(s) are relevant/releated to the feature request?
store
Description
There's the case in Angular when I want to set the initial value in the construction after getting some options token. Yes, we have withProps method but as said in this issue(it works as expected) https://github.com/ngneat/elf/issues/238 while resetting we'll get an initial value passed to the propsFactory. It can work if we'll create a class method that will create a props factory and we'll evoke it before creating a store but it's too complicated and demands setting props to the class property to be able to be seen.
Proposed solution
I see two ways of doing it.

withProps overwrite the initial value(I'd prefer this one); As we create separate propsFactory for each store that it will work fine ;
create one more method in each props factory setPropsInitialValue;

If we create propsFacroty once for multiple usages it'll change an initial value for all who're using this props factory but with the second variant, you just can hide and not export setPropsInitialValue.
Alternatives considered

Do you want to create a pull request?
Yes

Can you please add code examples? I don't think I understood.

In this case, as for now, I can set a filters in the constructor but on resetting them I'll reset them to the value that is inside of filtersPropsFactory but I want it to be reset to the value I got from DI.
const { withFilters } = filtersPropsFactory();

class {
  constructor(@inject(DEFAULT_FILTERS) filters: {}) {
     this.store = this.createStore(filters);
  }

  private createStore(filters: {}): typeof store {
    const store = createStore(
      { name: 'name' },
      withFilters(filters),
    );

    return store;
  }
}

to fix this either withFilters should replace an initial value of filtersPropsFactory or add new method
const { withFilters, setFiltersInitialValue } = filtersPropsFactory();

class {
  constructor(@inject(DEFAULT_FILTERS) filters: {}) {
     setFiltersInitialValue(filters);
     this.store = this.createStore();
  }

  private createStore(): typeof store {
    const store = createStore(
      { name: 'name' },
      withFilters(),
    );

    return store;
  }
}

I guess, we can make something like this too:
const { withVersion } = propsFactory('version', { initialValue: 1 });

// you can not pass an initial value to the `withVersion` if you passed to the `propsFactory`
withVersion();

const { withVersion } = propsFactory<T>('version');

// you must pass an initial value to the `withVersion` if you didn't pass to the `propsFactory`
withVersion(1);


Could not load zlib1.dll - Noob question

I'm sorry to ask this question, both because it has already been up in issues #43 but also because I'm sure this is not a problem with Nimdata but is most likely just me not being able to figure out Windows
New to Nim and haven't used Windows for the last 12 years so total noob in every way
choosenim 0.8.4
Nim 1.6.6
Windows 11
I hope a kind soul will help me out here

I haven't done any coding under Windows in a long time either. Can you try to compile and run:
import zip/zlib

If that reproduces your problem you might have better chances asking the question in a general Nim issue tracker, because it might be a general installation/core issue.

Away from the battle station so will try that later and get back to you.
Have a great day

Unfortunately, that did not work :-(
Warning: imported and not used: 'zlib' [UnusedImport] could not load: zlib1.dll


redo Hubspot events

Had to undo the changes yesterday due to a 🔥 . Could you ✅ please?

Let's wait this to be merged since we need the events to be created in Hubspot (otherwise we'll receive tons of Rollbar reports)


Exception: git failed

Hello @letmaik, I'm not sure if this is something that you can help me with, but I am trying to pip install rawpy on my Mac OS X 10.9.5 with "pip install --user rawpy," but I run into this error:
Installing collected packages: rawpy
  Running setup.py install for rawpy ... error
    Complete output from command /opt/local/Library/Frameworks/Python.framework/Versions/2.7/Resources/Python.app/Contents/MacOS/Python -u -c "import setuptools, tokenize;__file__='/private/var/folders/yt/47j3bxc91z7b3_j9pjdq5kb40000gn/T/pip-build-xJgffe/rawpy/setup.py';exec(compile(getattr(tokenize, 'open', open)(__file__).read().replace('\r\n', '\n'), __file__, 'exec'))" install --record /var/folders/yt/47j3bxc91z7b3_j9pjdq5kb40000gn/T/pip-4GBXMf-record/install-record.txt --single-version-externally-managed --compile --user --prefix=:
    LibRaw git submodule is not cloned yet, will invoke "git submodule update --init" now
    fatal: Not a git repository (or any of the parent directories): .git
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/private/var/folders/yt/47j3bxc91z7b3_j9pjdq5kb40000gn/T/pip-build-xJgffe/rawpy/setup.py", line 322, in <module>
        mac_libraw_compile()
      File "/private/var/folders/yt/47j3bxc91z7b3_j9pjdq5kb40000gn/T/pip-build-xJgffe/rawpy/setup.py", line 283, in mac_libraw_compile
        clone_submodules()
      File "/private/var/folders/yt/47j3bxc91z7b3_j9pjdq5kb40000gn/T/pip-build-xJgffe/rawpy/setup.py", line 127, in clone_submodules
        raise Exception('git failed')
    Exception: git failed

Not sure why this is happening, or if it's on my end or the installer's end. Any thoughts? Thanks.

That shouldn't happen. Can you run the following on the terminal please and copy paste the output?
pip --version
python --version

Yep!
pip --version:
pip 8.0.3 from /opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages (python 2.7)

python --version:
Python 2.7.11

OK, it doesn't have to do with your Python, but the binary wheels I provide are only for macOS >= 10.10 and you have 10.9.5. Can you upgrade?

Hi @letmaik, I've tried to do that, but I've run into High Sierra issues. Will have to address those first. In the meantime, I am now trying to install rawpy on the 32-bit Linux Ubuntu partition of my netbook, and I ran into an issue there. Will open a ticket related to that. Sorry for the trouble...

Let's try one more thing that might work for your Mac.

Download rawpy-0.10.1-cp27-cp27m-macosx_10_10_x86_64.whl to some folder.
Rename the file to rawpy-0.10.1-cp27-cp27m-macosx_10_6_intel.macosx_10_9_intel.macosx_10_9_x86_64.macosx_10_10_intel.macosx_10_10_x86_64.whl
Open a terminal, navigate to that folder and do pip install rawpy-0.10.1-cp27-cp27m-macosx_10_6_intel.macosx_10_9_intel.macosx_10_9_x86_64.macosx_10_10_intel.macosx_10_10_x86_64.whl

This kind of stuff is not guaranteed to work in all cases but there are good chance it will. By doing that we're pretending that the wheel was built on an older macOS version and is hence compatible to it.

@letmaik! That worked. Thank you so much. Clever solution. The High Sierra upgrade dependencies will take more time than I want to devote right now, so I appreciate the workaround.


Fix StyleCI configuration (continuous integration)

StyleCI fails due to a configuration error. This is confusing when creating a pull request.
It seems like the configution has a linting option, which is not longer available.

@c9s @GM-Alex @jhdxr @marcioAlmada @shinnya
Could one of you guys take a look at this? Should be quick to fix and it's making GitHub mark all recent PRs as failing.

Thank you for reporting! I removed linting option from the StyleCI setting of this repo.

I think we should remove the style CI. It's really bothering.

I removed the webhook.


Add a copy of the MIT license

It is good practice to add a copy of the license to the repository.

Thanks for making details! :smiley:


Tabbing out on hiring network page

Have the companies link out on new tabs https://suncoast.io/academy/network

See notes on #70


Add Sora High School (https://soraschools.com/).

Sora is an accredited online high school based in Atlanta, Georgia. They have an engineering track that requires JavaScript and TypeScript programming, for which IntelliJ Ultimate would be excellent.
https://soraschools.com/

@lkesteloot Pull request merged. Thank you!


Move Search to the RightDockedModal

If you haven’t already, check out our contributing guidelines for onboarding!

view the job on upwork here.
Platform - version:
current
Currently, clicking the search icon changes the functionality of the Left Hand Nav in order to show search results. We've recently introduced a right-docked modal which would be a better fit for search functionality. We're leaving the search icon in the same place as it is today (on the left), but we're going to engage the right-docked modal to show the search bar and search results when the icon is clicked. This implementation is similar to NewChatPage and NewGroupPage navigation wise.
Action Performed (reproducible steps):

Click the search icon
Search for a user using the search bar

Expected Result:

The search bar and results should utilize RightDockedModal
Note: ensure we grey out the main page content, as we do for the other pages in the RightDockedModal

Actual Result:

The search bar and results utilize the Left Hand Nav

Notes/Photos/Videos:

I would love to work on this issue.
I have worked on the SearchView, RightDockedModal, and NewGroup(/Chat)Page, so I know exactly the steps needed to reproduce the desired behavior.
Sent the proposal Upwork (Lucas Neuhaus).
Proposal

Change the SearchView component to SearchPage (basically just changing the name and the folder location);
Delete everything related to it on SidebarLinks;
Create a new route in src/ROUTES: SEARCH: '/search';
Call it on the HomePage, using the same pattern as the other RightDockedModal pages;
Verify everything works on all platforms.

Questions

Should the arrow keys actions be disabled as they are on the other RightDockedModal pages?

We would probably be able to delete the isChatSwitcherActive ONYXKEYS entry as well since it wouldn't control anything anymore.

Yep, sounds good @lucas-neuhaus-dev. I think we'd really be looking to remove all traces of anything referencing a ChatSwitcher as part of this step.

Should the arrow keys actions be disabled as they are on the other RightDockedModal pages?

I'm not too sure about this question. cc @shawnborton to see what he thinks.

I think we want to have the arrow functionality work for Search like it currently does.


Upgrade to<EMAIL_ADDRESS>

Is your enhancement related to a problem? Please describe.
<EMAIL_ADDRESS>was just released, which upgrades a lot of packages and removes a lot of polyfill code required for IE11. Now that new projects are dropping IE11 support, we can safely upgrade our version in the scaffold. Will probably resolve #71
Designs
No response
Describe alternatives you've considered
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

I totally missed this issue but I have updated 10up-toolkit to 2.x already in https://github.com/10up/wp-scaffold/commit/81ea51f270f5c9602a0f56a0422c89937e5125ef


Finish tying in process_labels code to create_iq_snapshot (integrate using python not shell scripts)

The objective here is to eliminate the use of a shell script to integrate ceilyn's code that creates the matrix into a normal run.
This is a basic integration as part of the V1.0 MVP.
It's OK if the code is not clean, as long as it  is bug free.

Example of the current state of the integration.
create_iq_snapshot_init \
     --collection_flag "snapshot" \
     --collection_timestamp "" \
     --sprint_name "20230503sprint" \
     --src_dir_name "" \
     --src_file_name "" \
     --organization_name "IQSS" \
     --project_name "IQSS/dataverse" \
     --src_type "api"\
     --workflow_name "tst"

#
# usage: process_labels.py [-h] --input INPUT --output OUTPUT [--matrix MATRIX]
# process_labels.py: error: the following arguments are required: --input, --output
# perftest@ubuntu:~/DevCode/github-com-mreekie/iqss_gh_reporting$
DIRNAME=/home/perftest/iqss_gh_reporting/run/out/April_26_2023_sprint-end
INFILE=2023_05_10-14_37_49-April262023-end-create_iq_snapshot-api-20230510143749-sized.tsv
INFILECORE=$(echo "${INFILE}" | cut -d'-' -f1-7)

OUTFILE="${INFILECORE}-matrix_summary .tsv"
MATRIXFILE="${INFILECORE}-matrix.tsv"

cat<<EOF
${DIRNAME}
${INFILE}
${INFILECORE}
${OUTFILE}
${MATRIXFILE}
cp ${DIRNAME}/${INFILE} ${DIRNAME}/input.tsv

EOF

cp "${DIRNAME}/${INFILE}" "${DIRNAME}/input.tsv"

process_labels --input "${DIRNAME}/input.tsv" --output "${DIRNAME}/outputput.tsv" --matrix "${DIRNAME}/matrix.tsv"

cp  "${DIRNAME}/outputput.tsv" "${DIRNAME}/${OUTFILE}"
cp "${DIRNAME}/matrix.tsv" "${DIRNAME}/${MATRIXFILE}"

Here's the dev/test loop for  this:
in pycharm in the run configuration:
set working directory:

/home/perftest/DevCode/github-com-mreekie/iqss_gh_reporting/scripts
set the GITHUB_TOKEN

cd /home/perftest/DevCode/github-com-mreekie/iqss_gh_reporting/scripts
./workflow-May032023sprint-snapshot.sh

the input_file.yaml is now in scripts directory
Start the debugger for iqss_gh_reporting
Run create_iq_snapshot in debug mode

OK, now I'll try this built locally.
make change to create_iq_snapshot in project: iqss_gh_reporting
cd ~/DevCode/github-com-mreekie/iqss_gh_reporting
bash -cl "/usr/bin/python3.10 /home/perftest/DevCode/github-com-mreekie/iqss_gh_reporting/setup.py bdist_wheel"; pip install .
pushd /home/perftest/DevCode/github-com-mreekie/iqss_gh_reporting/scripts
./workflow-May032023sprint-snapshot.sh
create_iq_snapshot
popd

Test the checkin from the repo
Switch to the "builds" project.
Update it from github
cd ~/DevCode/github-com-mreekie/builds
bash -cl "/usr/bin/python3.10 /home/perftest/DevCode/github-com-mreekie/iqss_gh_reporting/setup.py bdist_wheel"; pip install .
pushd /home/perftest/DevCode/github-com-mreekie/iqss_gh_reporting/scripts
./workflow-May032023sprint-snapshot.sh
create_iq_snapshot
popd

I'm going to leave this issue open for now since this was just a cursory check.
This issue is ready for Ceilyn to give it a shot and I guess that would act as our informal review and QA.

Ceilyn and I looked at this today.
The last piece left to this is transition away from her code reading from a file for the first step and instead just taking the already present dataframe.
   xfmrd_df = xfmr.SprintCardSizer(sprint_data).df
    sprint_data.df = xfmrd_df
    sprint_data.write(postfix="sized")

    summarized_data = xfmr.SprintSizeSummarizer(sprint_data)

    # read the input file
    # Todo: This does not need to be read. The data frame can be passed in.
    snapshot_file = sprint_data.dest_dir_name + "/" + sprint_data.dest_file_name + "-sized.tsv"
    df = plu.read_sized_snapshot(snapshot_file)

Ran into this.
Traceback (most recent call last):
File "/home/perftest/.local/bin/create_iq_snapshot", line 168, in <module>
main()
File "/home/perftest/.local/bin/create_iq_snapshot", line 111, in main
new_df = plu.process_sized_snapshot(sprint_data.df, labels='Labels', key='CardURL')
File "/home/perftest/.local/lib/python3.10/site-packages/iqss_gh_reporting/process_labels_util.py", line 137, in process_sized_snapshot
val = _parse_labels(label_str)
File "/home/perftest/.local/lib/python3.10/site-packages/iqss_gh_reporting/process_labels_util.py", line 62, in _parse_labels
label_list = labels.split(',')
AttributeError: 'list' object has no attribute 'split'

At line 136, it assumes the labels are expressed as a comma separated string.
This is safe when they are read from a file.
        if (label_str):
            val = _parse_labels(label_str)
            parsed_labels[key_value] = val
``

147 legacy.py
- I'll ensure that this value is stored as a string when the data is collected.
- This appears to have resolved the issue.

                    RequiredDfColumnHeaderNames.value("labels"): str(card_content.labels),

Also turned up an issue in pdata.py while looking.

I was overly aggressive in cleaning strings for directory paths.
I was turning - into _

Fixed in line 55
'out_dir':  sanitize_filepath(dest_dir_name + '/' + sprint_name, platform="auto"),

I can also add a check to _parse_labels to throw a useful exception if the labels are not structured as comma separated strings.

I can also add a check to _parse_labels to throw a useful exception if the labels are not structured as comma separated strings.

I think we're good here for the MVP.
We should take that into account in the next rev though. It's been years too long, but I think something like this falls under the concept of a contract or something?  Everyone agrees on how data is represented, etc?  Not sure.

I'm closing this.
Looking back at our short list objectives in the description, I think they are met.


Mein Computer startet wegen HackBGRT nicht mehr

my computer doesn't start anymore because of HackBGRT.  I installed HackBGRT, after a while I wanted to uninstall it and deleted the folder a: / Boot / EFI / HackBGRT.  now my computer always starts with an error message that cannot be skipped.  what can i do if i don't want to reset my computer?  By the way, I have an Ubuntu CD.

My Email is<EMAIL_ADDRESS>.

My Email is<EMAIL_ADDRESS>.

I read that it is at your own risk but I ask for an answer anyway..

I read that it is at your own risk but I ask for an answer anyway..

HackBGRT: Failed to load configuration (\EFT VlackBGRT\config.txt) 1 HackBGRT: No config. no command line HackBGRT has failed. Use parameter debug=1 for details. Get a Windows Install disk or a recovery disk to fix your boot.
HackBGRT version: v1.5.1
Press any key to exit.

HackBGRT: Failed to load configuration (\EFT VlackBGRT\config.txt) 1 HackBGRT: No config. no command line HackBGRT has failed. Use parameter debug=1 for details. Get a Windows Install disk or a recovery disk to fix your boot.
HackBGRT version: v1.5.1
Press any key to exit.

I can access my hard drive from Ubuntu

I can access my hard drive from Ubuntu

my computer doesn't start anymore because of HackBGRT.

That's not true. Your computer doesn't start anymore because you manually deleted boot files. HackBGRT setup has an option for uninstalling, and if you had used that, you wouldn't have this problem.
Your screenshot clearly tells you what to do: "Get a Windows Install disk or a recovery disk to fix your boot."
Just to repeat: You need to restore \EFI\Microsoft\Boot\bootmgfw.efi from a backup of from web, or you can use Windows installation media to fix boot issues, like the message says.

my computer doesn't start anymore because of HackBGRT.

That's not true. Your computer doesn't start anymore because you manually deleted boot files. HackBGRT setup has an option for uninstalling, and if you had used that, you wouldn't have this problem.
Your screenshot clearly tells you what to do: "Get a Windows Install disk or a recovery disk to fix your boot."
Just to repeat: You need to restore \EFI\Microsoft\Boot\bootmgfw.efi from a backup of from web, or you can use Windows installation media to fix boot issues, like the message says.


Problem with compiling older versions 1.6.6, 1.7.0

Hello, I have problems with compiling coredns+kubernetai of 1.6.6 and 1.7.0 versions.
1.8 builds fine, for coredns 1.6.6 I get:
# github.com/coredns/coredns/plugin/trace
plugin/trace/trace.go:68:20: undefined: zipkintracer.NewHTTPCollector
plugin/trace/trace.go:73:14: undefined: zipkintracer.NewRecorder
plugin/trace/trace.go:74:18: undefined: zipkintracer.NewTracer
plugin/trace/trace.go:74:45: undefined: zipkintracer.ClientServerSameSpan
# github.com/coredns/coredns/plugin/kubernetes/object
plugin/kubernetes/object/object.go:92:36: undefined: "k8s.io/apimachinery/pkg/apis/meta/v1".Initializers
plugin/kubernetes/object/object.go:95:47: undefined: "k8s.io/apimachinery/pkg/apis/meta/v1".Initializers

for coredns 1.7.0:
plugin/trace/trace.go:66:20: undefined: zipkintracer.NewHTTPCollector
plugin/trace/trace.go:71:14: undefined: zipkintracer.NewRecorder
plugin/trace/trace.go:72:18: undefined: zipkintracer.NewTracer
plugin/trace/trace.go:72:45: undefined: zipkintracer.ClientServerSameSpan
# github.com/coredns/kubernetai/plugin/kubernetai
/go/pkg/mod/github.com/coredns/kubernetai@v0.0.0-20201023153034-a3ae9db639d4/plugin/kubernetai/setup.go:30:22: cannot use c (type *"github.com/coredns/caddy".Controller) as type *"github.com/caddyserver/caddy".Controller in argument to k.RegisterKubeCache
/go/pkg/mod/github.com/coredns/kubernetai@v0.0.0-20201023153034-a3ae9db639d4/plugin/kubernetai/setup.go:33:21: cannot use c (type *"github.com/coredns/caddy".Controller) as type *"github.com/caddyserver/caddy".Controller in argument to dnsserver.GetConfig
/go/pkg/mod/github.com/coredns/kubernetai@v0.0.0-20201023153034-a3ae9db639d4/plugin/kubernetai/setup.go:50:36: cannot use c (type *"github.com/coredns/caddy".Controller) as type *"github.com/caddyserver/caddy".Controller in argument to kubernetes.ParseStanza

Images are built as follows:
FROM golang:alpine AS builder

ARG COREDNS_VERSION=v1.7.0

ENV GO111MODULE=on \
    CGO_ENABLED=0 \
    GOOS=linux \
    GOARCH=amd64

RUN apk update && apk add --no-cache git

WORKDIR $GOPATH/src/github.com/coredns

RUN git clone --depth 1 --branch $COREDNS_VERSION https://github.com/coredns/coredns.git && \
    cd coredns && \
    go get github.com/coredns/kubernetai && \
    sed -i 's/kubernetes:kubernetes/kubernetes:kubernetes\nkubernetai:github.com\/coredns\/kubernetai\/plugin\/kubernetai/' plugin.cfg && \
    go generate && \
    go build -o /go/bin

If I replace go get github.com/coredns/kubernetai with go get github.com/coredns/kubernetai@196a693161c7230a8c4a29bb7c4db1838311547f ("bump to coredns" 1.6.5 commit) then it seems, that 1.6.6 is built fine (but nevertheless here is the version of kubernetai for coredns 1.6.5, and the coredns itself is version 1.6.6, not sure if this is correctly).
for 1.7.0 I get:
/go/pkg/mod/github.com/coredns/kubernetai@v0.0.0-20191114152720-196a693161c7/plugin/kubernetai/setup.go:24:24: not enough arguments in call to k.InitKubeCache
        have ()
        want (context.Context)

Since AWS recommends using coredns 1.6.6 and 1.7.0 for cluster versions 1.17 and 1.18 respectively, I would like to be able to do so.
Could you please suggest the correct way to build older versions 1.6.6 and 1.7.0?

Since AWS recommends using coredns 1.6.6 and 1.7.0 for cluster versions 1.17 and 1.18 respectively, I would like to be able to do so.

That seems like bad advice. Use the most recent version for all K8s versions, unless there is a specific backwards-incompatible change that you cannot tolerate.

Since AWS recommends using coredns 1.6.6 and 1.7.0 for cluster versions 1.17 and 1.18 respectively, I would like to be able to do so.

This seems like an outdated recommendation - perhaps related to a specific bug in past.  Indeed, the linked documentation does not currently appear to make references to specific versions of CoreDNS.


feat: introduce wasmbus lattice

Related Issues

#361
This does not actually handle any lattice commands, just the core functionality and some command parsing
Release Information

Consumer Impact

Testing

Built on platform(s)

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

Tested on platform(s)

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

Unit Test(s)

Acceptance or Integration

Manual Verification

Tested alongside OTP as part of bigger changeset

I can't tell from my phone but it looks like the 'default' lattice has been hard coded. Just want to mention here that lattice ids other than default need to be supported

I can't tell from my phone but it looks like the 'default' lattice has been hard coded. Just want to mention here that lattice ids other than default need to be supported

@brooksmtownsend already pointed that out in https://github.com/wasmCloud/wasmCloud/pull/366#discussion_r1260157830
This will be addressed in a follow-up


Add short cut links to speed up traversal between vertexes in case of regular edges

During the addition of links to the edge records during edge creation, also add links directly between vertices. This will allow us to:

Restrict the cardinality of edges, which is very important.
Allow to execute navigation between vertices with the same speed as we do right now for lightweight edges.

Hi,
I think I reverted this

Sure no problem


Next release

Added

Update manifest standard to use icon property instead a custom one (#388)
[TX-Builder] Transaction library, import and export (#348) (#349)

Fixed

EOA Address not clearing ABI field (#359)
Tuple type not working as expected (#390)

Chore

Bump dependencies (#409)

Regression plan for the release https://gno.testrail.io/index.php?/runs/view/1410&group_by=cases:section_id&group_order=asc

Regression is all green!
Only comment that I have is that on tx history screen txs done via the PR branch are being displayed like a multisend call (the one above):

Let me know if we would need to talk to the backend team or it is just something related to PR branches :)

I can confirm that icons are visible from tx history screen. However, I think there is something a bit off with backend on Gnosis Chain. Look at how drains are initially displayed on Gnosis Chain:

I perform a drain & check tx history

(Displayed as separate send txs)

I create a second drain & check tx history

(second Drain displayed as separate send txs, first one pops up but the separate send txs for it persist)

I wait for a while and go back to tx history screen

(All bueno)


How add the reference Intel.Realsense.dll in VB.net

| Required Info
| Camera Model D435i
| Firmware Version <IP_ADDRESS>
| Operating System & Version Win 10 Pro for OEM
| Platform PC
| SDK Version .NET 4.6
| Language VB.NET(2015)
| Segment Others
I try to develop a app with VB.net for my D435i. I could already add the reference Intel.Realsense.dll, but  the realsense2.dll always failed. I do not know why.
I found someone  have made it.
Such as:
https://github.com/IntelRealSense/librealsense/issues/2019
How I add the reference  realsense2.dll in the VB.NET?

Thanks very much!
You are right. I added the reference Intel.Realsense.dll, and pasting the realsense2.dll into the Debug folder manually. It works with Vb.net.


Documentation doesn't link to the GitHub information page

When clicking the three dots against the Tronity integration in the Devices and Services page, the Documentation option just re-opens the same devices and services page. It should link here: https://github.com/tronity/homeassistant
This is the menu I am referring to:

Thanks for letting us know about the issue. We've fixed it now, so when you click on it, it'll take you to the correct page.


Support Window 10 base image for Windows Service 2022

Is your feature request related to a problem? Please describe.
Based on a company policy we are still using Windows 10 and we need to update from Windows Server 2019 container to Windows Server 2022 container.
but the process of building locally is becoming a big issue since we cannot use Windows 10 to pull/run the Windows Server 2022 container
Describe the solution you'd like
Support for Windows 10
Describe alternatives you've considered
None
Additional context
None

Hey @miaooss, thank you for sharing this feedback. Currently we all of our investments are focused on Win11, WS2022 and beyond.
I've shared this request with the team, so we are aware of it.
I'll close this issue for now since this isn't a priority at the moment, but we'll track this thread for any new comments.


Handle incorrect phrases

Do not raise in case if phrase is not correct

Thanks for this @oltarasenko, this library definitely needs better error handling. I like how easy to follow the pipelines are but in reality a with would be much better here.
I will merge this because I think we'll still need this as a last resort even if we fix the variable length phrase.


To contribute scraped car dataset from various websites

Can you please assign me this issue. I have scraped a new dataset using scrapy.

That will be great ! @Tejaswi-Kumar

Hey, I added the dataset, thanks for assigning the issue. If everything is in order please accept the pull request.
Thanks and regards!


Source file naming convention

[x] This only affects the JavaScript OpenTelemetry library
[x] This may affect other libraries, but I would like to get opinions here first

In the repo, there are files named in hex-to-base64.ts style, and some are BoundInstrument.ts style, and some are correlation_context. Should the file naming style be kept consistent across the project? And which style would the project prefer?

I prefer not to use caps in filenames as it can cause issues when some developers use case insensitive file systems. I don't have a strong preference for snake_case vs kebab-case

Yes, it can be nasty on case insensitive fs if the pattern contains capital characters. For reference, snake_case is used in Node.js core.

Would like some weigh-ins from @open-telemetry/javascript-approvers and @mayurkale22  but I'm ok with it.

This is a dupe of #38


GPU-Util=0

why the gpu-util=0

it seems to spend too much time in collater process when loading train data. so the GPU is waiting this . do you have good idea to solve it? thanks


Allocator api for immutable Buffer

Which issue does this PR close?

For discussing https://github.com/apache/arrow-rs/pull/6336
Rationale for this change

What changes are included in this PR?
Implementing the Allocator for immutable Buffer correspondingly

Are there any user-facing changes?

Extra context
Notice that this patch would fail in some cases as I haven't change https://github.com/apache/arrow-rs/blob/e67f17e80be4c51f8b62746e79f16e853fd2ed0b/arrow-buffer/src/buffer/immutable.rs#L319-L322

Sorry for the inconvenience. This branch is on top of waynexia:allocator-api. New change since
2ad2e3e


vertical pickers

any idea why I have the picker vertical not horizontal ??

Did you import the CSS too?

I forgot to. thanks for the response :+1:


Selecting wrong one when view is reordered

I am reordering some rows of these input fields based on whcih one they type into first. For example, there are three rows, and if they type into the second one, with the top one empty, then the second row becomes the top row.
If i do this, when i press tab, comma, enter etc, it will move it correctly, highlight the correct one, but then the correct highlight gets removed and it highlights the one in the second position (where the one that should be highlighted used to be).

Can you post a reproduction in Plunker?


Fix - Fail to serve with built-in server as instructed

Error at line 25:
$ php -S <IP_ADDRESS>:8080 -t public/ public/index.php

Error in logging output:
PHP 7.0.28-0ubuntu<IP_ADDRESS> Development Server started at Mon Mar 26 09:42:37 2018
Listening on http://<IP_ADDRESS>:8080
Document root is /home/userj/My/repos/ZendFrameworkRibs/ZF3/helloworld/public
Press Ctrl-C to quit.
[Mon Mar 26 09:42:53 2018] PHP Warning:  Unknown: failed to open stream: No such file or directory in Unknown on line 0
[Mon Mar 26 09:42:53 2018] PHP Fatal error:  Unknown: Failed opening required '/home/userj/My/repos/ZendFrameworkRibs/ZF3/helloworld/public/public/index.php' (include_path='.:/usr/share/php') in Unknown on line 0

@peterleephp I think we should go just with php -S <IP_ADDRESS>:8080 -t public/.
See also: https://github.com/zendframework/zend-expressive-skeleton/pull/216

Would it be more explicit as suggested by php --help:
Usage: php [options] [-f] <file> [--] [args...]
   php [options] -r <code> [--] [args...]
   php [options] [-B <begin_code>] -R <code> [-E <end_code>] [--] [args...]
   php [options] [-B <begin_code>] -F <file> [-E <end_code>] [--] [args...]
   php [options] -S <addr>:<port> [-t docroot] [router]
   php [options] -- [args...]
   php [options] -a

  -a               Run interactively
  -c <path>|<file> Look for php.ini file in this directory
  -n               No configuration (ini) files will be used
  -d foo[=bar]     Define INI entry foo with value 'bar'
  -e               Generate extended information for debugger/profiler
  -f <file>        Parse and execute <file>.
  -h               This help
  -i               PHP information
  -l               Syntax check only (lint)
  -m               Show compiled in modules
  -r <code>        Run PHP <code> without using script tags <?..?>
  -B <begin_code>  Run PHP <begin_code> before processing input lines
  -R <code>        Run PHP <code> for every input line
  -F <file>        Parse and execute <file> for every input line
  -E <end_code>    Run PHP <end_code> after processing all input lines
  -H               Hide any passed arguments from external tools.
  -S <addr>:<port> Run with built-in web server.
  -t <docroot>     Specify document root <docroot> for built-in web server.
  -s               Output HTML syntax highlighted source.
  -v               Version number
  -w               Output source with stripped comments and whitespace.
  -z <file>        Load Zend extension <file>.

  args...          Arguments passed to script. Use -- args when first argument
                   starts with - or script is read from stdin

  --ini            Show configuration file names

  --rf <name>      Show information about function <name>.
  --rc <name>      Show information about class <name>.
  --re <name>      Show information about extension <name>.
  --rz <name>      Show information about Zend extension <name>.
  --ri <name>      Show configuration for extension <name>.

@peterleephp Believe me, we tried different version before and we have always issues with them on different systems or different PHP version, what I suggested above and linked is the latest version we agreed on, which is working on all OS and latest PHP versions as there described.

Ok, I believe you @webimpress, updated the pull request.
My search does show trace of issues on this PHP command on different OS over the time:
https://github.com/zendframework/ZendSkeletonApplication/pull/147
https://github.com/zendframework/zend-expressive/pull/42
https://github.com/zendframework/tutorials/pull/71
https://github.com/zendframework/ZendSkeletonApplication/pull/406

Thanks, @peterleephp. Merged to develop for release with 3.1.0.


Create and Update Employee Cost Rates via SDK

I notice there's no support in the SDK documentation for Employee Cost Rates, which is documented as "Legacy".
Is this functionality obsolete? Is it marked for removal? Has it been replaced by something else?
What's the current method for specifying the hourly rate or annual salary of a new employee or updating the salary or hourly rate of an existing employee?

The documentation shows what is available via the API. The SDK doesn't provide methods to access everything in the API. Sometimes you need to do it manually which is a pain in the neck. Legacy doesn't mean it is obsolete. It is just using the old methods which will always be available and in some cases are the only way to do certain things. If the documentation shows it is available in the API, and there isn't anything in the SDK for it, you will need to construct the XML yourself rather than rely on inbuilt functionality.


error processing custom format.

Can't figure out the error thrown when using the custom format.
$ echo "[GIN] 2020/07/25 - 03:01:04 | 200 |      67.081µs |   <IP_ADDRESS> | GET      /" | goaccess --log-format='%^ %d %^ %t |  %s | %D.%^ | %~%h | %~%m %U' --date-format=%Y/%m/%d --time-format=%T
Parsed 1 lines producing the following errors:

Missing braces '{}' and ignore chars for specifier '%h'

Format Errors - Verify your log/date/time format

Version:
$ goaccess -V
GoAccess - 1.2.
For more details visit: http://goaccess.io
Copyright (C) 2009-2016 by Gerardo Orellana

Please attach a file with a few lines from your access log. Thanks

sample.txt
Attached a file with few lines from the log.

Thanks,
Nikhil.

Interesting format, but this should do it in v1.4:
goaccess sample.txt --log-format='[%v] %d - %t %^ %s |%~%D.%^ |%~%h %^ %m %~%U' --date-format=%Y/%m/%d --time-format=%T

I switched to using goaccess 1.4, but cant get it to work with the sample text.
When i run the command mentioned above, i get a "Log Format Configuration" screen.
Tried selecting every log format, but get an error:
"Token '[GIN]' doesn't match specifier '<>'
where, the  changes with the selected log format.

@nikhil-kartha could you please post a few lines from your access log and the command you are running? Thanks

Sorry, it's working fine today. Not sure what the issue was yesterday.
I used the command you provided:
goaccess sample.txt --log-format='[%v] %d - %t %^ %s |%~%D.%^ |%~%h %^ %m %~%U' --date-format=%Y/%m/%d --time-format=%T

I tried it with a bigger sample data, few mb's, it works great!


Vickel: Watch avinashkatta.in

Vickel requests the watch of the watch_keyword avinashkatta\.in. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

avinashkatta\.in has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


Add variables to tweak deployment for stream service

Shouldn't the with_repository_cache variable also be used to exclude the repo-cache related pieces from the worker template? Or can that be done independently?

It should, eventually, but this can be, I think, merged without that part.


can not download Demo video

Dear Master,
I want to check Demo video through attached link (https://www.aliyundrive.com/s/ycDqPLz5e3Z/folder/61bef2d66bbb3e8803544251adf525354f3240bd)
But, I don't know how to download it....
Could you please share Demo video through Google Drive?
Thanks,
kwanghyuk

We are glad about the concern of our work. We provide the demo links on Google Drive here:
Original videos: https://drive.google.com/drive/folders/1M2OB4p5E8r0ywW82iU89DLJ9ErK9jvUE?usp=sharing
Replaced demos: https://drive.google.com/drive/folders/1P7VQgqHvRClOOghztaBEMrIFdBcNKsJf?usp=sharing
Comparison on POT: https://drive.google.com/drive/folders/15ww81nZJK-u1ROns4AKW_qY7mdKApFKe?usp=sharing

Dear Master,
I want to check Demo video through attached link (https://www.aliyundrive.com/s/ycDqPLz5e3Z/folder/61bef2d66bbb3e8803544251adf525354f3240bd) But, I don't know how to download it.... Could you please share Demo video through Google Drive?
Thanks, kwanghyuk

We are glad about the concern of our work. We provide the demo links on Google Drive here:
Original videos: https://drive.google.com/drive/folders/1M2OB4p5E8r0ywW82iU89DLJ9ErK9jvUE?usp=sharing
Replaced demos: https://drive.google.com/drive/folders/1P7VQgqHvRClOOghztaBEMrIFdBcNKsJf?usp=sharing
Comparison on POT: https://drive.google.com/drive/folders/15ww81nZJK-u1ROns4AKW_qY7mdKApFKe?usp=sharing


Added php 7 with nginx

I have added nginx with php-fpm.
P.S It requires docker-compose

@ashikkalavadiya Thank you for the PR!
Actually, the purpose of this Dockerfile is a little bit different. We are using this image to just run our CI server and test our PHP code. It is not being used as a local development environment.
Thanks again :)


Improve log readability, minor code cleanups

reduces log level of cargo metadata parsing step, this is not significantly important but noisy
added an empty line before project tasks, makes it easier visually to find where a different project build started
refactored up some code

thanks for the PR. i see build is failing.

Example of the improvement:

i see build is failing.

@sagiegurari will check

@berkus looks good thanks. just fyi, it will probably be a while until i release this as its not big functional change or a bug fix.


Resource not implemented for MatchboxSocket

the trait bound `bevy_matchbox::MatchboxSocket<bevy_matchbox::prelude::MultipleChannels>: bevy::prelude::Resource` is not satisfied
the following other types implement trait `bevy::prelude::Resource`:
  SocketState
  bevy::a11y::AccessibilityRequested

Totally unsure of the bevy magic at play here, but when trying to use the MatchboxSocket in a project I got this error for one of my systems,
fn sys {
    mut socket: ResMut<MatchboxSocket<MultipleChannels>>,

Check the above PR, and you'll get the full error stack. I have no clue how to fix this

Ok so I think this is related to #175 because I tested a solution of adding a sealed-type version and I didn't get any issues.
Does Bevy not allow resources to be generic? It certainly seems like an anti-pattern or at the least has very poor ergonomics.
The real solution we should do is probably have 3 types: MatchboxMultiChannelSocket, MatchboxSingleChannelSocket, and MatchboxEmptySocket` with the generics erased...
Unless someone knows of a better solution?

Expanded the proc macro for Resource and got this:
// Recursive expansion of Resource! macro
// =======================================

impl<C: BuildablePlurality> bevy::ecs::system::Resource for MatchboxSocket<C> where
    Self: Send + Sync + 'static
{
}

Certainly looks generic to me

Ah, it looks like you've managed to build bevy_matchbox with bevy 0.10 and bevy_ggrs_example with bevy 0.9. @johanhelsing and I discussed this possible issue when I first added bevy_matchbox, but didn't expect it to blow up in our faces so immediately

I think we should only focus Bevy 0.10+, since this is so new.

Let's just update to Bevy 0.10.

How do you propose we upgrade bevy_ggrs in the example?

If we want something more official than my git branch for the example, I guess we could ask @gschup kindly if he has the time and would be ok with a final reflect-based release for Bevy 0.10?
Or I could at least make a tag on my 0.10-compatible bevy_ggrs branch.
#130

I think I should definitely merge your branch of bevy 0.10. I am in my final months of writing my thesis, so my time on this is limited right now. I actually have some time right now. Is this PR still up-to-date? Can you please make sure it has everything we need? @johanhelsing

https://github.com/gschup/bevy_ggrs/pull/51 builds on https://github.com/gschup/bevy_ggrs/pull/50 and has some extra very low-hanging fruits (ambiguous rollback schedule is an error, expose schedule label for more ergonomic usage) and is what I've been using when updating the tutorial for Bevy 0.10

Started the upgrade to bevy 0.10. Got everything migrated except for the stages.
Feel free to add onto #190


SimpleDragHandler.createForwardingListener creates dead nodes on Android and Chromebook

From https://github.com/phetsims/circuit-construction-kit-common/issues/426 @phet-steele and @jessegreenberg identified that SimpleDragHandler.createForwardingListener creates toolbox icons that, when tapped rapidly on Android or Chromebook, creates "dead" nodes which cannot be moved.  We have seen this problem on CCK and Projectile Motion.  @phet-steele can you report if this symptom appeared elsewhere?
It would be good to address this before CCK 1.0, so I'll assign a high priority. Tagging @ariel-phet so he is aware of the scope of the problem.  Assigning to @jonathanolson for investigation.  Labeling for developer meeting so everyone is aware of the problem and so we can discuss & allocate other resources as necessary.

Note the image https://github.com/phetsims/circuit-construction-kit-common/issues/426#issuecomment-339734063 that indicates the input listener gets a down and move but no up/end.

Steele tested with sceneryLog=OnInput, and it looks like the browser isn't sending the pointerup:

I'll want to verify that pointercancel is set up to be listened to.

@phet-steele, this is weird, but can to reproduce with rootRenderer=canvas?

Sims using createForwardingListener:
capacitor-lab-basics
circuit-construction-kit-common
equality-explorer
fluid-pressure-and-flow
projectile-motion

@ariel-phet said this may not be a blocking issue for CCK.

@phet-steele, this is weird, but can to reproduce with rootRenderer=canvas?

@jonathanolson yes.

@phet-steele can you report if this symptom appeared elsewhere?

Sims using createForwardingListener:
capacitor-lab-basics
circuit-construction-kit-common
equality-explorer
fluid-pressure-and-flow
projectile-motion (published)

capacitor-lab-basics - No, but this sim doesn't "create" nor "dispose" something in the areas where this is used (the switches) so the use case doesn't exist (or is harder).
circuit-construction-kit-common - Yup.
equality-explorer - No.
fluid-pressure-and-flow - No, but this sim doesn't create anything on a "start" drag (or tap), so the use case doesn't exist (or is harder).
projectile-motion (published) - You bet.

Edited above to say I CAN repro in equality-explorer

I don't see how this is specific to createForwardingListener, or the sims that use it.

I don't see how this is specific to createForwardingListener, or the sims that use it.

@jonathanolson I think it was just a hunch that has proved somewhat true so far?

I don't see how this is specific to createForwardingListener, or the sims that use it.
@jonathanolson I think it was just a hunch that has proved somewhat true so far?

Okay maybe not, the reproduction in equality-explorer was done without the use of createForwardingListener. There, I did it with items on the scale, not from the toolbox.

Slack discussion with @phet-steele regarding equality-explorer:

@phet-steele it was a lot easier for me to do it to items on the scale…not in the toolbox. is that still event forwarding?
@pixelzoom items on the scale do not use createForwardingListener. createForwardingListener is only used when one object (eg, icon in the toolbox) creates another object (eg item the user is dragging). in such a case, the drag sequence starts with the icon in the toolbox, and subsequent events (drag, endDrag) must be forwarded from the icon to the item that the user is dragging. in the case of items on the scale, the entire drag sequence (start, drag, end) involves only the item.  so there’s no forwarding involved. so if you’re seeing the issue with items on the scale, the problem is not in createForwardingListener

I don't see how this is specific to createForwardingListener, or the sims that use it.

So this is a problem for all input listeners, not just createForwardingListener?

So this is a problem for all input listeners, not just createForwardingListener?

If Scenery never receives the "up" from the browser, it definitely could potentially affect any listener. It just may be generally easier to "trigger" the issue in cases where we would want to use createForwardingListener.
Reading up to see if I missed a case that we'd want to treat as "up" in the specification.

@phet-steele, can you test a version of a sim published before September 2015 to see if it exhibits the same bugginess? I'm curious if the setPointerCapture() added as part of https://github.com/phetsims/scenery/issues/464 may by chance be related.

@jonathanolson and I discussed trying to create a minimal reproducible test case for this, which prints out all events received.  If we see the problem right away, we can report it to Google.  If not, we can take steps to make the example more "sim-like" until we notice the problem.

@phet-steele, can you test a version of a sim published before September 2015 to see if it exhibits the same bugginess? I'm curious if the setPointerCapture() added as part of #464 may by chance be related.

@jonathanolson, I think I'm seeing the issue in 1.0.0-dev.10 for Fractions Intro, which was made in 2014.

Marking for potential investigation as part of https://github.com/phetsims/circuit-construction-kit-common/issues/449.  @ariel-phet can you please check status/priority/assignment of this issue?

@ariel-phet can you please review https://github.com/phetsims/circuit-construction-kit-common/issues/426 and this issue, and prioritize/schedule/assign accordingly?


Cleaned up some code

Motivation
Clean up some component.
Have you read the Contributing Guidelines on pull requests?
ya.
Test Plan
Basically moved a complicated map function to outside of render's return.
Explantation
I chose this chunk of code smell because I wanted to reduce the amount of code inside of render's return, which should mostly contain just markup and as little logic in it as possible.

Thanks for the PR! Unfortunately, mapping to a component is not really considered logic, more like configuration. Moving the initialization of the component out of render also doesn't really add much value.


User options

new way to create, validate and transform options from many sources

Can I close this PR since you added the new one?

Yes
2017-05-23 18:23 GMT+02:00 wkrzemien<EMAIL_ADDRESS>
Can I close this PR since you added the new one?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/JPETTomography/j-pet-framework/pull/98#issuecomment-303455267,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOlHB9eq4KJ-akfIhxJTb056UPp4ql52ks5r8wf0gaJpZM4NHXqv
.

Superseded by #102