It's misleading to have the archive button in the toolbar when viewing the archive

Even though Mail.app has the same behaviour, Terrence found it confusing to have the Archive button in the toolbar when viewing the archive folder. It does, however, make sense to have it here as running a rule allows files to be moved to different locations. Perhaps the correct answer would be to make it more obvious that this button runs a rule, rather than the simple operation of moving a file to the archive?

I've added a tooltip calling the archive button out as the 'Rules Wizard', and changed the icon to try to make it clear that it's equally applicable when in the archive, and in the inbox.

See #99.


IDEA/Eclipse plugins aren't downloading sources when mavenLocal() is used

This is a transfer of GRADLE-2832 from JIRA (originally from a post in the Gradle Forums).  I'm running into this issue myself, and I see that nobody has advocated for the issue, so it didn't get transferred to GitHub.  An identical issue was mentioned in regards to the Eclipse plugin in a different forum post.
The problem occurs when the mavenLocal() repository does not have the source JAR, but another repository declared after it does.  In this case, the IDEA/Eclipse plugin will not search any of the later repositories for the artifact sources.
Per the comments on the forums, the crux of the issue seems to be that Maven tools treat the local repository differently than Gradle does.  Namely, the Maven tools will recognize that when only some of the artifacts are in the local repository, it should fetch the others when needed.  Gradle, on the other hand, appears to treat the local repository the same as any other repository: if the artifacts aren't there, then it doesn't bother checking elsewhere for them.
Here is a simplified version of the example from the old IDEA ticket:
apply plugin: 'maven'
apply plugin: 'idea'

repositories {
        mavenLocal()
        mavenCentral()
    }
}

idea {
    module {
        downloadJavadoc = true
        downloadSources = true
    }
}

I should note that I'm having this issue within IDEA itself, and not using the plugins as part of the build.  I'm guessing that this issue has the same cause as the plugin issue, though I haven't thoroughly investigated whether that's the case.

Is there any progress or comment for this? The issue is extremely annoying and forces to change order of repositories each time before and after downloading needed sources. Also for security sensitive projects it may be a security issue, since by making public repository first one to check names of internal dependencies may be exposed to it.

While this isn't the most upvoted issue ever, it looks like there is still some interest from the community other than myself in seeing this resolved.

Our company is in the transition of the old build system to Gradle and most of our developers use Eclipse this is one of the most annoying problems our developers also run into. A change of the behaviour would be great here.

I don't think it was the case when the issue was originally filed, but the Gradle documentation now states that mavenLocal() should generally not be used as a repository, with a few exceptions that should be carefully configured.  I think the right answer if possible is to switch your builds to no longer use mavenLocal(), or to configure it to only narrowly be used in the contexts where it's necessary.

As a general advice, you should avoid adding mavenLocal() as a repository. There are different issues with using mavenLocal() that you should be aware of:

Maven uses it as a cache, not a repository, meaning it can contain partial modules.

For example, if Maven never downloaded the source or javadoc files for a given module, Gradle will not find them either since it searches for files in a single repository once a module has been found.

As a local repository, Gradle does not trust its content, because:

[...]

To mitigate the fact that metadata and/or artifacts can be changed, Gradle does not perform any caching for local repositories

[...]

There are a few cases where you might have to use mavenLocal():

For interoperability with Maven

For example, project A is built with Maven, project B is built with Gradle, and you need to share the artifacts during development
It is always preferable to use an internal full featured repository instead
In case this is not possible, you should limit this to local builds only

For interoperability with Gradle itself

In a multi-repository world, you want to check that changes to project A work with project B
It is preferable to use composite builds for this use case
If for some reason neither composite builds nor full featured repository are possible, then mavenLocal() is a last resort option

After all these warnings, if you end up using mavenLocal(), consider combining it with a repository filter. This will make sure it only provides what is expected and nothing else.


Frontend: Instances are only shown when the version filter is set

Description
The instance view shows no instances because the version= parameter has no value set which results in a HTTP 400 bad request being thrown: parameter "version" in query has an error: empty value is not allowed
Environment and steps to reproduce

In the UI, go to Application → Group → Instances and see that no instances are shown unless a version filter is set

Expected behavior
All instances are shown
Additional information

I also have this issue on latest version (2.8.0)

For me it is fixed since 2.8.1:


Extend a Workflow Definition with an Exclusive Gateway

Tutorial URL: https://developers.sap.com/tutorials/cp-wf-03-add-excl-gw.html
Please specify the step you are referring to
Can't belive the images are gone from Sap developer tutorials. This issue has been happening intermittently. Why this problem is not fixed permanently??

I see the images now. Do you?
I will check why this happens.

Yes i can see it now.. But this happened last year also. Please find out the root cause.

Will do ...


Log stacktraces on error

I sometimes see random errors in the log output with no real way to follow up on them.
Consider the following log line - it would be nice to know which JSON file was being parsed:
IMAGE_NAME=ghcr.io%2Fspr-networks%2Fsuper_api%3Alatest unexpected end of JSON input

is this logs under docker-compose logs?
we have a little bit more information under http://spr/admin/events where there's at least a link to the line that prints the error

This is enabled in v0.2.23+


Allow chaining with built records

This will allow things like:
build(:user, name: "john") |> on_free_trial |> create |> add_comment(title: "toto")

So that you can easily modify the built record, then call create and do has_many/has_one relationships with the saved record. I will be adding the functionality to not save associations on build in the next PR, which should make this a bit more robust

LGTM

Thanks!
On Fri, Sep 25, 2015 at 3:28 PM, Blake Williams<EMAIL_ADDRESS>wrote:

LGTM
Reply to this email directly or view it on GitHub:
https://github.com/thoughtbot/ex_machina/pull/21#issuecomment-143331579


Some little typo fixes

Codecov Report

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

@@           Coverage Diff           @@
##           master      #63   +/-   ##
=======================================
  Coverage   82.69%   82.69%
=======================================
  Files          31       31
  Lines         705      705
  Branches      118      118
=======================================
  Hits          583      583
  Misses        114      114
  Partials        8        8

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 cf76b98...a69ea69. Read the comment docs.


[JENKINS-46076] Reordering events so WorkflowRun.completed not set until finish cause printed & log closed

JENKINS-46076
After incremental deployment, downstream bumps in:

[X] https://github.com/jenkinsci/workflow-cps-plugin/pull/294
[X] https://github.com/jenkinsci/kubernetes-plugin/pull/496
[X] https://github.com/jenkinsci/pipeline-model-definition-plugin/pull/336

https://github.com/jenkinsci/workflow-job-plugin/blob/fb2c9f5778e044e309ee887375fbd70b5f4cb872/src/main/java/org/jenkinsci/plugins/workflow/job/WorkflowRun.java#L214 - one last reference to "logCopyGuard"

https://github.com/jenkinsci/workflow-job-plugin/blob/fb2c9f5778e044e309ee887375fbd70b5f4cb872/src/main/java/org/jenkinsci/plugins/workflow/job/WorkflowRun.java#L214 - last reference to copyGuard  (I can't edit it or I'd just do it).

Now waiting for INFRA-2140.

Seems to have worked—all downstream builds pass.

Hey @dwnusbaum, would you mind running a release of the plugin? I'd like to integrate the changes in this PR in downstream plugins

@Vlatombe Ok, I will try to release the plugin today.

cool, thanks!

I suppose you can just suggest changes in the downstream POMs after it has been released.

2.33 was released, I added suggestion to the downstream PRs linked here.


The sample given :continuationsample.js is for blobs and not tables

The link given that has a sample is for blobs not Tables: https://github.com/Azure/azure-storage-node/blob/master/examples/samples/continuationsample.js

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

ID: 168890e7-484f-6d2f-9eec-3a97dcd44e6f
Version Independent ID: ffd475a2-052e-7ce0-851a-622453a14010
Content: How to use Azure Table storage or Azure Cosmos DB from Node.js
Content Source: articles/cosmos-db/table-storage-how-to-use-nodejs.md
Service: cosmos-db
GitHub Login: @mimig1
Microsoft Alias: mimig

@shraboin Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

#in-process

@MicahMcKittrick-MSFT - Thanks for the feedback. I've removed the link to the blob sample. I will follow up when the link is live on the site.

The link has been removed from the page: https://docs.microsoft.com/en-us/azure/cosmos-db/table-storage-how-to-use-nodejs#use-continuation-tokens. #please-close


Add zoom/pan viewport support

This is probably just a simple plugin away from working, or we could implement it manually

This is done. I'm waiting to merge a bug fix to a zoom/pan plugin upstream but other than that it works and people are happy with it.


"Connection pool is full, discarding connection" warning

This warning pops up sometimes when running the OSXCollector Analyze Filter:
requests.packages.urllib3.connectionpool: WARNING  Connection pool is full, discarding connection: investigate.api.opendns.com

Not sure if it means that a particular connection is dropped and the results are never gonna be obtained for a certain domain.

As per requests documentation in module {{requests.packages.urllib3.connectionpool}}:

If the pool is already full, the connection is closed and discarded because we exceeded maxsize. If connections are discarded frequently, then maxsize should be increased.

I don't think this is a big problem, but we may need to increase the pool size, so that the connections are not discarded that often.

I am receiving the folowing error:
WARNING:urllib3.connectionpool:Connection pool is full, discarding connection: www.virustotal.com

Many times when trying to get a report for ~2000 domains. Has this connection pool issue been resolved?
I get it if I send all 2000 at once or if I iterate over the domains making an API call per domain.
#!/usr/bin/python
import os
import time
from threat_intel.virustotal import VirusTotalApi

vt = VirusTotalApi(os.environ["VIRUS_TOTAL"], resources_per_req=1)
fo = open("questionable_domains.txt", "r")
domains = fo.readlines()

# print(vt.get_domain_reports(domains))

for domain in domains:
    print(vt.get_domain_reports(domain))

The reason for this issue was that the customized SSL Adapter mounted to each session was using a max HTTPConnectionPool size of 10. Specifically, this meant that each connection pool corresponding to a specific host could only issue 10 requests asynchronously per batch before filling up and discarding the connection. The fix was as simple as specifying for the adapter to take in a maximum pool size corresponding to the max of the grequest batch size and 10.

Fixed with #74.


Adding support for non dollar virtual columns names

Summary
Adding support for having non-dollar('$') character in the MySQL virtual(generated) columns. Currently, the virtual columns are named as 'i_urn$platform' now we can also use '0' as the delimiter i.e the virtual column could be named 'i_urn0platform'.
Testing Done
./gradlew build
Added unit tests for all the changes.
Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable)

/rerun-check Custom Validations


Missing textures on suns, doors(?) (maybe other places too)

Hi, at some point after version 1.3.25 a change was made that makes the sun on most modded maps (though not all for some reason, the new "vulcan" moon seems fine) a missing purple texture. It also does the same thing to the doors on the kokiri forest map from the mega map pack. I can confirm that on version 1.3.25 neither of these issues occurred. Though I haven't been able to use versions after 1.3.26 due to the terrain change, and can now update only because of 1.3.37 reverting that change (thank you!) so I'm not sure what was changed that causes this bug....
Pic for comparison, and pic of the missing textures on the kokiri entrances.

Thank you for upkeeping this mod!

The mappers must check for missing sprite and materials in their map before bundle it with the last version of the sdk


Add function standardization tests

All functions needs and should have the same basic parameters:

Organization

Should always be required

Projectname (where needed)

If included should be mandatory

Create tests in Module.tests.ps1 to verify they are correct to avoid parameter name confusion and drift.

We currently have two name standards for Projectname: ProjectName and Project.
What should we use or should we support all variants?
Regarding Projectname needs to be mandatory if present, Get-ADOPSProject currently lists all projects or single one based on a optional Project parameter.
I see two options for the tests, ether we exclude the Get-ADOPSProject function from that test or we split the function in two. One to get all projects without the Projectname parameter and one to get a single project with a mandatory Projectname parameter.
Any thoughts?

I made a PR standardizing ProjectName to Project in all occurrences a few days ago, my vote is to stay with Project as the parameter name.

Just to clarify @bjompen, you mean that it should be mandatory to add Organization as a parameter, not that the parameter should be mandatory for the user? In that case I agree with it being a good test :)

Correct, I was very unclear.
Organization parameter should be in all functions, but not mandatory for the user.

More standard test we should include:
All functions should have the files

.\Source<private / public>\FunctionName.ps1
.\Tests\FunctionName.Tests.ps1

If it is a public function it should also have

.\Docs\Help\FunctionName.md


fix: improves form timer (old)

Summary | Résumé
Improves the form timer by handling the case of forms with groups. If a form has groups, the delay calculation includes required questions based on the group history. The time a user spends on a form is now also included in the calculation.
This is a high risk PR since it updates forms-forms submission logic.
Test
The form timer should not display in the form-builder Preview. The form timer is only for forms-forms
Groups Form
Create a form with groups and add a few required questions. View the form you created and open the dev console. Reload the form (to restart the timer) and quickly fill in the form, counting the number of required questions. When you get to the Review page, check the console and see if the final delay makes sense.
For example, a form that has 10 required questions and has been view for 5 seconds, should have a delay of
2 + (10 - 5) * 2 = 12 seconds
Note that the countdown begins once the Review page loads. So if you linger on the Review page for e.g. 5 seconds and then click the submit button, count down delay should start at e.g. 7 seconds (with a total delay listed of 12 seconds)
Non-Groups Form
Create a form without groups and add a few required questions. Following the same steps above. The difference is the time a form is viewed is not taken into account (existing behaviour - countdown begins when the single page form is shown).

I have a question - is it suppose to calculate the time spent on each required question throughout the entire form, prior to the Review Page? Or is it that you need to hang around the Review Page for a certain amount of time (per each required question) before you can submit?
(Asking because when I tested, I waited 10 seconds while filling out each required field. But then when I got to the Review Page, I still had to wait to submit the form?)

Moved to this PR that takes a simpler approach https://github.com/cds-snc/platform-forms-client/pull/4437


Repeated classes removed

Hello,
I'm using Jade with MeteorJS and the "incredible" Semantic-UI. Today I noticed that the jade compiler is removing the repeated classes of the elements. This should be ok but not for the semantic ui, for example:
div.ui.middle.aligned.center.aligned.grid
the second aligned class is removed and the div is not "center aligned"
there is any way the avoid that ? a jade config {remove.repeated.classes:false} por example ?
Thanks

This is not related to jade compiler but to Meteor itself. There was an issue about this https://github.com/Semantic-Org/Semantic-UI/issues/1526


WSL2 upgrade just prints help text

Windows Version
Microsoft Windows [Version 10.0.20348.587]
WSL Version
Invalid command line option: --version (followed by help text)
Are you using WSL 1 or WSL 2?

[ ] WSL 2
[X] WSL 1

Kernel Version
4.4.0-2348
Distro Version
Ubuntu 20.04 LTS
Other Software
Docker Desktop for Windows v4.17.0
Repro Steps
On a clean installation of Windows Server 2022 version 10.0.20348.587, follow the guide official Microsoft guid and simply run wsl --install and follow the steps.
Expected Behavior
I should have had the option to select Ubuntu 22.04, and it would be installed using WSL2.
Actual Behavior
I was multitasking and do not remember the exact steps I took after running wsl --install but everything appeared to set up normally. I thought that I had selected Ubuntu 22.04 during that process.
When I tried to use Docker Desktop I was greeting with a strange Newtonsoft.Json error and began to troubleshoot.
I noticed that wsl -l -v outputs Ubuntu   Stopped   1 for NAME   STATE   VERSION. I attempted to run wsl --set-version Ubuntu 2 and was greeted with:
PS C:\Windows\system32> wsl --set-version Ubuntu 2
Copyright (c) Microsoft Corporation. All rights reserved.

Usage: wsl.exe [Argument] [Options...] [CommandLine]

Arguments for running Linux binaries:
...

Interestingly, I am also unable to win wsl --version, maybe because I'm running version 1 for some odd reason:
PS C:\Windows\system32> wsl --version
Invalid command line option: --version
Copyright (c) Microsoft Corporation. All rights reserved.

Usage: wsl.exe [Argument] [Options...] [CommandLine]

Why the heck am I running WSL1 on Server 2022, and how do I upgrade it?
Diagnostic Logs
No response

There was an issue in the Server 2022 branch where the --set-version option got removed. We are working on bringing a fix back, but in the meantime you should be able to use wsl.exe --set-default-version 2 before you register your distro.

I try to run ANY wsl command and it prints just the help message (-l, --set-default-version, anything.)


Test, DO NOT MERGE

Making this as PR so I can easily make a test build in this dependency mess.

Can one of the admins verify this patch?  Type 'ok to test' to test.

ok to test

ok to test

ok to test

ok to test

This will probably fix the GroupInvalidationEvent snitch group thing, but the test build is broken so I cant test it. Should I just merge it, hope the build works there and fix it if needed or do you want to figure out what broke when trying to build this?


Hardcode the string "true" in the HTML's value parameter

A small issue was that the code has to call .toString on value in order to pass it into the "elements.args.selectedInput" method (on line 11, now line 10).
This is because in the views, Play converts Strings (and maybe other types) to a "play.twirl.api.Html" (to work nicely with the view template); it probably does not expect further method calls.
Since the value for a checkbox is always the string "true", it is not necessary for the code to check the args Map for the value of "value", it is sufficient to just pass the String "true" into selectedInput. The same goes for the HTML (instead of referencing @value, "true" can be hardcoded); although, since checkboxes are always "true", it is not really necessary to have the value in the HTML

As discussed earlier, let's pause this one for now because we probably need to do some more extensive refactoring to make it easier to follow the commonised code in these templates.

@techncl We should close this PR for now, until we decide whether to do some refactoring of the general error handling.

Sure!


cmd/go: go generate fails silently when run on a package in a nested workspace module

I noticed the following while working on https://go.dev/cl/440181. I have a go.work file using both x/tools and x/tools/gopls. When running go generate on a package in x/tools/gopls, the command exits with status 0 without doing anything.
When run from the gopls module, the go generate command works as expected. This can be seen from the following shell history, where I start from a clean workspace (at patchset 2 of the above CL), make an arbitrary edit in one of the generated files, and run go generate expecting it to revert the diff.
$ git status
On branch accessdenied
Your branch is ahead of 'origin/master' by 1 commit.

nothing to commit, working tree clean
$ echo "// arbitrary edit" >> gopls/internal/robustio/robustio.go
$ git status
On branch accessdenied
Your branch is ahead of 'origin/master' by 1 commit.

Changes not staged for commit:
        modified:   gopls/internal/robustio/robustio.go

no changes added to commit
$ go generate ./gopls/internal/robustio
$ echo $?
0
$ git status
On branch accessdenied
Your branch is ahead of 'origin/master' by 1 commit.

Changes not staged for commit:
        modified:   gopls/internal/robustio/robustio.go

no changes added to commit
$ cd gopls && go generate ./internal/robustio
$ git status
On branch accessdenied
Your branch is ahead of 'origin/master' by 1 commit.

nothing to commit, working tree clean

Duplicate of #29751

Hmm, not quite a duplicate (because of workspace mode).

@gopherbot please backport to 1.21 and 1.22. go generate doesn't work in workspace mode and it's a one-line fix


Phil Sayer Platform A/B announcement

The announcement for Platform A/B by Phil are not able to play
5e3179c37dc94a85b6c269eba5e3f5b3
Platform B
c7dbe04c0c014e94a474659f33af01a2
Platform A

Yeah I got the same thing

And also 'St Albans' is apparantly missing on both Next Train and Live Trains on Phil Sayer generator

Please open a separate issue for unrelated problems.
This is because the station is called St Albans City rather than simply St Albans. As there was never a recording with "City", this wasn't included by default.

Would you also be able to double check if Hampton Wick is in your file please, as it is missing from the list as well.

Please open a separate issue for unrelated problems.


[BUG][1.1.0] Places all small entities insteod of just pipes

Game version: 1.1.87
Mod version: 1.1.0
I tried in /editor and without it to place inserts, splitters, belts, chests etc., but the mod placed second one near it.

I'll send a save file in several minutes, because I can't reproduce it in other games.

big__pipe_mod.zip

huh, now it happens only in editor mode.

I haven't been able to reproduce this. Please comment here if you encounter this behavior again.


MIgrate to otel_sdk instead of prometheus_client

Description

Refactor to the different client
Align metadata
Align metric naming
Use same metadata in logs
Adjust existing dashboards and alerts

Reasons

Attachments

Release Notes

Hello, I tried to solve the issue.
This is what I did:
This change refactors the different clients, aligns metadata and metric naming, uses consistent metadata in logs, and adjusts existing dashboards and alerts. The modifications aim to improve code consistency and maintainability across the kyma-metrics-collector project.
You can review changes in this commit: https://github.com/lattaai12/kyma-project-kyma-metrics-collector-121/commit/767429cc37f3038164180a1a205452b3149db1f8.

[!CAUTION]
Disclaimer: The commit was created by Latta AI and you should never copy paste this code before you check the correctness of generated code. Solution might not be complete, you should use this code as an inspiration only.

This issue was tried to solve for free by Latta AI - https://latta.ai/ourmission
If you no longer want Latta AI to attempt solving issues on your repository, you can block this account.


Custom dumps for DateTime* objects (+convert data providers to generators)

Q
A

Is bugfix?
✔/❌

New feature?
✔/❌

Breaks BC?
✔/❌

Tests are broken.
Need a line for CHANGELOG.


Use REM for comments, instead of ::

When :: labels are used as comments within a bracketed code block or FOR command, the command processor will expect (not unreasonably) that every label is followed by at least one command. (That way if you jump to the label it will have something to execute.) In fact the CMD shell will attempt to execute the second line even if it is formatted as a label.
Anything which is not a valid command, even a blank line or a closing parenthesis will cause a :: style comment to fail:
(
 Echo This example will fail
 :: some comment
)

(
 :NormalLabel & Echo this will not be executed.
 :SecondaryLine & Echo This will be executed.
)

When working within parentheses/brackets it is safer to use REM for all comment lines.
(
 REM something & Echo this will not be executed.
 REM somethingelse & Echo This also will not be executed.
)

I didn't even know that could be an issue, tyty


Manual scrolling.

Implementation:

Allow the user to scroll through messages, being with the scroll wheel, up/down arrow keys, and even j or k.
When scrolling through messages, the currently selected one should be bold.
Allow user and moderator actions on a selected message such as copy and delete (if possible).

There are different users within Twitch, and then subsets of those users. Let's try to find a way to cater to all of them, either through different options in the config file or general-purpose manual scrolling. Given that someone has gone out of their way to download and setup the binary of this program, we can assume that they're most likely willing to either read large chunks of chat or freeze every so often to see if a message really is what they saw.
Implementation suggestions as of now when manual scrolling mode is activated for the chat:

The new messages get stored in the VecDeque for later viewing. Once the user exists out of manual scrolling mode (which only gives the previously configured amount of messages backwards, stored in config.terminal.maximum_messages), they get to see the most recent messages as a whole.
An amount of config.terminal.maximum_messages / 2 as a floored float are available backwards and forwards (for new messages). Once new messages appear, there are two options:

Remove messages from the back of the VecDeque so there's room for new messages.
Use a different data structure (if needed) to allow for adjust the amount of messages to be stored in the front, so all new messages are available while only some backwards are.

For channel and user searching, a function will be created to go through the items. The popup window will not expand beyond a certain point.
TODO:

[ ] Chat
[ ] Users
[ ] Channels

This will be implemented in #74.


Dynamically parse DBCs

Now you can edit any DBC file and start PlotJuggler and it should grab your new changes without a complete re-compile of PlotJuggler!

This is about 50-70% slower on C3, hyundai_kia_generic for example takes consistently 10.5 ms while on PC it's consistently 6.3 ms.
For toyota_nodsu_pt_generated, it's consistently 3.1-3.2 ms while on PC it's 1.3-1.5 ms


3D euler transformation parameter information inconsistent with visual check

Hello,
I am applying the first step of this notebook on image registration with SimpleITK.
It is about the inital transformation:
initial_transform = sitk.CenteredTransformInitializer(fixed_image,
                                                      moving_image,
                                                      sitk.Euler3DTransform(),
                                                      sitk.CenteredTransformInitializerFilter.GEOMETRY)

moving_resampled = sitk.Resample(moving_image, fixed_image, initial_transform, sitk.sitkLinear, 0.0, moving_image.GetPixelID())

I modified it slightly to serve my needs:
initial_transform = sitk.CenteredTransformInitializer(fixed_image,
                                                       moving_image,
                                                       sitk.Euler3DTransform(),
                                                       sitk.CenteredTransformInitializerFilter.MOMENTS)

moving_resampled = sitk.Resample(moving_image, fixed_image, initial_transform, sitk.sitkLinear, 0.0, moving_image.GetPixelID())
moving_resampled_numpy = sitk.GetArrayFromImage(moving_resampled)

I was curious about what rotations and translations are actually performed, so I obtained the parameters of the transform and printed the information of the transform:
print(initial_transform)
initial_transform.GetParameters()

Which gave me back:
itk::simple::Transform
 Euler3DTransform (0000020DA7BADF80)
   RTTI typeinfo:   class itk::Euler3DTransform<double>
   Reference Count: 1
   Modified Time: 1142
   Debug: Off
   Object Name:
   Observers:
     none
   Matrix:
     1 0 0
     0 1 0
     0 0 1
   Offset: [-11.6416, 3.42275, -9.36365]
   Center: [178.924, 146.712, 161.544]
   Translation: [-11.6416, 3.42275, -9.36365]
   Inverse:
     1 0 0
     0 1 0
     0 0 1
   Singular: 0
   Euler's angles: AngleX=0 AngleY=0 AngleZ=0
   m_ComputeZYX = 0

and
(0.0, 0.0, 0.0, -11.641622423527735, 3.42275104972623, -9.363647551070386)
respectively.
Can I now conclude that this transformation only consists of a translation in three directions and no rotation?
This contradicts with my visual findings:

where I think especially the shaded area cannot come from just a translation in three directions.
Can someone help me with this?
Thank you!

Hello @IlvaHou55,
Your observations are correct. The initial_transform is just a 3D translation.
I disagree with the statement that the "shaded area cannot come from just a translation in three directions". It is hard to visually confirm this, especially when using a single slice of a 3D object after translation.
To confirm that a transform is mapping points as expected you can use the RegistrationPointDataAquisition GUI component which is part of the notebooks repository.
Clicking on a distinct point in one image, will map it to the other image, see code below:
%matplotlib notebook
import gui

gui.RegistrationPointDataAquisition(
fixed_image,
moving_image,
figure_size=(8, 4),
known_transformation=initial_transform
);

@zivy I guess you are right. I've also been taking screenshots of different slices and see that the registered volume lacks behind about two slices which would suggest only translation indeed. Could the shaded areas come from interpolation for example?
And thank you very much for the RegistrationPointDataAquisition GUI, very useful tool!

@zivy one additional question, I see that the RegistrationPointDataAquisition GUI should also work for 3D images (using a scroll bar between slices). However, this does not pop up for my case. Any idea why?

Hello @IlvaHou55,
The shaded values are due to the type of interpolation you are applying. As you appear to be working with a binary image  (guessing here based on the displayed images) you should be using sitk.sitkNearestNeighbor instead of sitk.sitkLinear. The linear interpolation introduces values that did not exist in the original image while nearest neighbor ensures that no new values are created.


Improve support for Systems that cannot provide total capacity

Note: It's reporting 1B since the metrics exporter of Noobaa is broken.

/retest

Working as expected with Noobaa.

/cherrypick release-4.10

/lgtm

/cherrypick release-4.10-compatibility


react 14.x and 15.x supported?

0.14 works fine. 15 has the same problem as Typescript https://github.com/Microsoft/TypeScript/issues/7270

Fixed in v5.0.0


Missing changelog update

Version 2.0.3 released, please add a changelog entry. ❤

As a maintainer of georap, I am stuck at email-validator 1.0.7 because the missing changelog. Please, keep them up to date for your users' sake.


xAxis time with different labels

Hi, I have a set of records where each record has two values: 'date' (timestamp) and 'status' (number).
I would like to use the xAxis of type time, so that the values are shown on a timeline.
But I want to show as labels of the axis the correspondent 'status' values.
Is there a way to achieve that?
Record example: { date: new Date(), status: 42, value: 1.2 }

You can use formatter to add custom text in label.
xAxis: {
  axisLabl: {
    formatter: function (timestamp) { return ''; }
  }
}

The problem is that I don't have access to the other values of the record in the formatter function.
Moreover, using the xAxis as a "time" axis, it can take any time value, not just the ones that have data.
Example: if you have 2 timestamps corresponding to the 2 dates 01/06/2016 10:00 and 01/06/2016 11:00  (so only the time is different), it can show a label with 10:30 in the middle...


[kong] Allow the proxy container's lifecycle hooks to be overridden

What this PR does / why we need it:
Allows the proxy container's lifecycle hooks to be overridden. In order to prevent dropped requests during the shutdown process, it can help to sleep first before signaling the shutdown to the container process. This gives kube-proxy and other ingress controllers time to remove the IP of the terminating pod from their iptables rules or load balancing configurations. There was a KubeCon NA talk that demonstrated this: video + slides
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes # N/A

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] PR is based off the current tip of the next branch and targets next, not master
[x] New or modified sections of values.yaml are documented in the README.md
[x] Title of the PR and commit headers start with chart name (e.g. [kong])

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

I see that it's been merged, but to speak to the point raised by @rainest, it seems this also opens up the opportunity to put something other than a sleep into the value.
My immediate thought when I saw this was that maybe one wcould curl another endpoint that was reporting info on their system and then only execute kong quit once that endpoint said it was safe to do. I've had cases, for example, where I've had to drain off 100s of thousands of web socket connections over a controlled period of time that was significantly longer than 15 seconds.

Please fill in the contributor form to claim your Kong swag: https://github.com/Kong/kong/blob/master/CONTRIBUTING.md#contributor-t-shirt


[HOLD for payment 2023-10-09] [$500] Chat - Loader is shown on top of image even after image is loaded in attachment carousel

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open a report
Send 3-4 images/pdf's
Scroll through carousel multiple times

Expected Result:
The loader should not appear on top of the image if the image is already loaded
Actual Result:
The loader appears on top of the image even if the image is already loaded
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.57-5
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/24da66b5-0f34-44ce-9dec-aa65cd031136
https://github.com/Expensify/App/assets/78819774/06168b45-fd23-4a75-8274-738311208f41
Expensify/Expensify Issue URL:
Issue reported by: @krishna2323
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1692443921240289
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01e48d51a0940adaac
Upwork Job ID:<PHONE_NUMBER>322225152
Last Price Increase: 2023-09-12
Automatic offers:

Krishna2323 | Contributor | 26743909
krishna2323 | Reporter | 26743910

Proposal
Please re-state the problem that we are trying to solve in this issue.
Chat - Loader is shown on top of image even after image is loaded in attachment carousel
What is the root cause of that problem?
https://github.com/Expensify/App/blob/d09a1b66fd957f68a985cc25ebd372ac35e47de0/src/components/Attachments/AttachmentCarousel/Pager/AttachmentCarouselPage.js#L115C27-L115C27
forgot to add
onLoadEnd={() => setIsImageLoading(false)}
What changes do you think we should make in order to solve the problem?
we need add this
onLoadEnd={() => setIsImageLoading(false)}

We're discussing moving the shortcuts modal to the RHN, so I'm going to close this issue.

I'm sorry, I closed the wrong issue.

I'm sure we've fixed this before. Here's one issue where we fixed an issue of the carousel loading every image multiple times: https://github.com/Expensify/App/issues/15922. @kidroca @thesahindia you worked on that issue so I'm hoping you're familiar - is this a regression of that one?

Posted in the original Slack thread again.

@thesahindia confirmed that we fixed this and it's a regression. Adding external label, I'd like to first see if we can track down the regression, because the person who caused it should ideally fix it.

Reassigning the external label. @eVoloshchak could you please take a look at the proposals we've received so far? Thanks!

@eVoloshchak could you please review these proposals? I can choose another C+ again if you're not available to review by tomorrow.

@pradeepmdk, I can still reproduce this using your proposal
The bug doesn't occur when you use arrow buttons for navigation, only when you're swiping between images manually
https://github.com/Expensify/App/assets/9059945/228c5970-e877-49d8-bd96-53619080393d

@Krishna2323, thank you for the proposal!
I wonder if there is a better way to resolve this, adding a delay (i.e. setTimeout) should be the last resort.
Actually, this this comment summed up this perfectly:

Given that this is a special case where there is no standard way to detect the cache, thus delaying the loader is fine. The main purpose of the settimeout is to schedule a task for the future and this issue seems like a perfect match where we want to delay the loader for a short time which validates the use of settimeout

The same logic you're proposing has already been implemented for ImageWithSizeCalculation, is that correct? If so, could we add it to the Image component itself? I think having the same logic implemented in two places isn't ideal

@eVoloshchak Proposal updated https://github.com/Expensify/App/issues/26046#issuecomment-1694928149

@Krishna2323's updated proposal looks good to me
This isn't ideal, but there is no standard way to detect if the image is cached, so it's the next best thing. We'll also be shifting this logic from ImageWithSizeCalculation to a generic Image, so while not perfect, I think that's the approach we should take
🎀👀🎀 C+ reviewed!

The PR that introduced the bug has been identified. Link to the PR: I'm unable to pinpoint the exact PR that caused this, the root cause for this was present in the initial implementation
The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: N/A
A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: I don't think an additional discussion is needed, the bug is specific to this particular component, there is no universal approach/checklist item we could implement

Regression Test Proposal

Open any chat
Send 3-4 images
Scroll through the carousel multiple times
Verify that the loader doesn't briefly appear on top of the image if the image is already loaded

Do we agree 👍 or 👎

@Krishna2323 thanks for the reminder on the issue creation date. I updated the offers accordingly.
@eVoloshchak does not require payment (Eligible for Manual Requests)
@Krishna2323 requires payment offer (Contributor)
@Krishna2323 requires payment offer (Contributor)
Summarizing payouts for this issue:
Reporter: @Krishna2323 $250 (paid via Upwork)
Contributor: @Krishna2323 $1000 (paid via Upwork)
Contributor+: @eVoloshchak $1000 (payable via Manual Request)
Above payments include efficiency bonus 🎉
Upwork job: https://www.upwork.com/jobs/~01e48d51a0940adaac

@eVoloshchak let me know once you've sent the manual request, I'll close the issue at that point. Thanks!

@sakluger, request has been sent, thank you!

$1,000 payment approved for @eVoloshchak based on BZ summary.


Package should not be annotated @CheckReturnValue -- generated protobuf code fails Error Prone checks

Package com.google.common.html.types is annotated with @CheckReturnValue: https://github.com/google/safe-html-types/blob/abccc1eb91b4dd1ca817c61f3c743fb3a3320fb6/types/src/main/java/com/google/common/html/types/package-info.java#L23
This causes builds that enforce the annotation (like Error Prone) to fail on generated protocol buffers which reference the safe html types defined in this repository.
Example:
import "webutil/html/types/html.proto";

message Test {
  optional webutil.html.types.SafeUrlProto url_field = 1;
}

Attempting to compile with error prone will fail on the generated java protobuf code with a message like:
error: [CheckReturnValue] Ignored return value of method that is annotated with @CheckReturnValue
                subBuilder.mergeFrom(urlField_);
                                    ^
    (see http://errorprone.info/bugpattern/CheckReturnValue)
  Did you mean 'subBuilder = subBuilder.mergeFrom(urlField_);'?

error: [CheckReturnValue] Ignored return value of method that is annotated with @CheckReturnValue
    com.google.common.html.types.Html.getDescriptor();
                                                   ^
    (see http://errorprone.info/bugpattern/CheckReturnValue)
  Did you mean to remove this line?

Because the protocol buffer messages reside in the same java_package com.google.common.html.types, I don't believe there's a quick fix for this other than removing the annotation from from the package and instead annotating all classes in types/src/main/java/com/google/common/html/types individually.
Update: This actually also affects several of the non-protobuf classes--@CheckReturnValue shouldn't apply to the following builders:

SafeHtmlBuilder
SafeStyleBuilder
TrustedResourceUrlBuilder

Fixed via 7e6e902e52b206d1c369cc834f98e93911613aee

Pushed 1.0.8 to maven.  It's there but it'll take some time to show up in maven search results.


Update message_manager.py

Sticker doesnt have content_type and parse_mode

As discussed in chat: this is not the only problem with stickers. Additionally they cannot be edited and cannot be send as an edit for other media


Add OSBuddy links and popup menus to GE plugin

Add popup menus to both the offers and search tabs, allowing links to either the official GE prices or the OSBuddy GE prices. Add option to prefer OSBuddy prices when left clicking search results.
Resolves https://github.com/runelite/runelite/issues/4056.

You should probably just use 1 link, based on ```	@ConfigItem(
position = 3,
keyName = "enableOsbPrices",
name = "Enable OSB actively traded prices",
description = "Shows the OSBuddy actively traded price at the GE"
)
default boolean enableOsbPrices()

This could just be me, but I've always wanted a way to lookup prices for an item in the Offers tab to see if prices changed since I made the offer. I feel like the best way to do this is with a popup menu. I also compare OSBuddy and official prices quite often. I'm not sure if I agree with only having one link.
I think the preferOsbuddyGeLinks setting I added is redundant though and I should use the existing enableOsbPrices setting more. Things like only showing the Lookup OSBuddy GE price menu item if it's enabled, etc. I'll investigate more tomorrow.

Well, in GE plugin there should not be anything referring to OSB by default, so make sure that everything OSB related requires explicit enabling. Using the enableOsbPrices config option for that is ideal. I guess if you want, keep the right click menu and make OSB primary menu when the option is enabled, but when disabled there should not be any right click menu and it should simply point to GE.

Changed behaviour to be around the enableOsbPrices setting now.  Haven't factored OSB price in search yet, but I feel that could probably be it's own separate PR.

Looks like this needs rebase

Done.

Rebased again.

Needs rework now that https://github.com/runelite/runelite/pull/11940 is merged. Closing for now.


Support investigation of misses patterns by hour and line

Use case: Identifying of lack of execution by hour patterns
Example:
If we group by the execution metrics of line 64 of Electra Afikim from 1.7-30.7 by hour, we can easily identify patterns (take a look at 9:30):

Suggestion:

In the "gaps" page, allow selecting a range of dates.
When more than 1 day is selected, the results will be grouped -by hour, so instead of "V", "X", we'll see the following columns:

total planned
total executed
miss percentage - colored red-yellow-green.

@NoamGaash , I close this issue because I see that it was resolved and merged in Gaps patterns page #44 pr


Bug: Element-plus Select不显示Options

Version
Vben Admin V5
Describe the bug?
Element-plus Select不显示Options

Reproduction
{
fieldName: 'status',
component: 'Select',
componentProps: {
placeholder: '状态',
clearable: true,
options: [
{
label: '成功',
value: '200',
},
{
label: '失败',
value: '400',
},
],
},
},
System Info
Browsers

Relevant log output
No response
Validations

[X] Read the docs
[X] Ensure the code is up to date. (Some issues have been fixed in the latest version)
[X] I have searched the existing issues and checked that my issue does not duplicate any existing issues.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

ElementPlus 跟 Ant design vue不同，Select 组件根本就没有 options 属性。
https://element-plus.org/zh-CN/component/select.html#select-api

那我该怎么做呢？教我一下。我使用h函数也不行

我也遇到了这个问题，请问解决了吗

@752337625 这样可以


Misleading information on how controls works

Not technically a bug.
Controls are written on the main github page and are such:
"• You can double click the icon to immediately open the configuration sheet for that effect.
• You can right click the icon to immediately disable the effect.
• You can shift + right click the icon to be prompted to delete the effect."
However these controls are not what module actually offers and there is no way to make them such. What actually is offered in the module itself (see impeded screenshot):
"• You can double click the icon to immediately open the configuration sheet for that effect." - That's still true
"• You can right click the icon to immediately disable the effect." - It's configurable, but if set to "Disable" and not "Dialog" which is how it should have been working - then there is no way to delete an effect via Effects Panel.
"• You can shift + right click the icon to be prompted to delete the effect." - Not implemented at all. You can only delete by Right Click but only if it's configured that way.
I don't know whether it's just a problem of not updating the main page or the features of the module being lost throughout the versions, but either way it's very misleading and should probably be fixed either in the module itself (best, but harder way) or on the page, so that the people are not mislead (easier way).
Personally for me, I was lured by such crisp controls and were sad to see they are not actually implemented in the module.

Yeah, just forgot to update the readme. Sorry about that, will update them.


Handling of storagePrefix if not part of 'aggr-list' of the svm, should be taken care off in file 'storage_drivers/ontap/ontap_common.go'

INFO Created installer service account.            serviceaccount=trident-installer
INFO Waiting for object to be created.             objectName=clusterRole
INFO Created installer cluster role.               clusterrole=trident-installer
INFO Waiting for object to be created.             objectName=clusterRoleBinding
INFO Created installer cluster role binding.       clusterrolebinding=trident-installer
INFO Created installer configmap.                  configmap=trident-installer
INFO Waiting for object to be created.             objectName=installerPod
INFO Created installer pod.                        pod=trident-installer
INFO Waiting for Trident installer pod to start.
INFO Trident installer pod started.                namespace=trident pod=trident-installer
INFO Starting storage driver.                      backend=/setup/backend.json
INFO Controller serial numbers.                    serialNumbers="721825000190,721825000189"
WARN Please ensure all relevant hosts are added to the initiator group.  SVM=vs0 driver=ontap-san igroup=quark1
FATA Install failed; could not start the storage backend driver; unable to inspect ONTAP backend: runtime error: invalid memory address or nil pointer dereference
Stack trace:
goroutine 1 [running]:
runtime/debug.Stack(0xc420177078, 0x150d4c0, 0x21fe590)
/usr/local/go/src/runtime/debug/stack.go:24 +0xa7
github.com/netapp/trident/storage_drivers/ontap.getStorageBackendSpecsCommon.func1(0xc420178038)
/go/src/github.com/netapp/trident/storage_drivers/ontap/ontap_common.go:924 +0x6e
panic(0x150d4c0, 0x21fe590)
/usr/local/go/src/runtime/panic.go:502 +0x229
github.com/netapp/trident/storage_drivers/ontap/api/azgo.(*VserverInfoType).VserverAggrInfoList(...)
/go/src/github.com/netapp/trident/storage_drivers/ontap/api/azgo/type-vserver-info.go:504
github.com/netapp/trident/storage_drivers/ontap/api.Client.VserverGetAggregateNames(0xc420427f70, 0xd, 0xc420427f88, 0x3, 0xc420427f8b, 0x5, 0xc420427f80, 0x8, 0x0, 0xc4202a5080, ...)
/go/src/github.com/netapp/trident/storage_drivers/ontap/api/ontap.go:1678 +0x472
github.com/netapp/trident/storage_drivers/ontap.getStorageBackendSpecsCommon(0x187d7c0, 0xc42012a1c0, 0xc4203da600, 0xc4203c6780, 0x0, 0x0)
/go/src/github.com/netapp/trident/storage_drivers/ontap/ontap_common.go:929 +0x1b3
github.com/netapp/trident/storage_drivers/ontap.(*SANStorageDriver).GetStorageBackendSpecs(0xc42012a1c0, 0xc4203da600, 0xc420455270, 0xc4202a4cc0)
/go/src/github.com/netapp/trident/storage_drivers/ontap/ontap_san.go:572 +0xe6
github.com/netapp/trident/storage.NewStorageBackend(0x1897fa0, 0xc42012a1c0, 0xa, 0xc4201101c0, 0xd1)
/go/src/github.com/netapp/trident/storage/backend.go:82 +0xe5
github.com/netapp/trident/storage/factory.NewStorageBackendForConfig(0xc4201101c0, 0xd1, 0x0, 0x0, 0x0)
/go/src/github.com/netapp/trident/storage/factory/factory.go:79 +0x5e3
github.com/netapp/trident/cli/cmd.loadStorageDriver(0xc4202a47e0, 0xc420178698, 0x1)
/go/src/github.com/netapp/trident/cli/cmd/install.go:884 +0x225
github.com/netapp/trident/cli/cmd.installTrident(0xc420179cc8, 0x0)
/go/src/github.com/netapp/trident/cli/cmd/install.go:623 +0x3c0b
github.com/netapp/trident/cli/cmd.glob..func12(0x220ba80, 0xc4202c9440, 0x0, 0x12)
/go/src/github.com/netapp/trident/cli/cmd/install.go:161 +0x210
github.com/netapp/trident/vendor/github.com/spf13/cobra.(*Command).execute(0x220ba80, 0xc4202c9320, 0x12, 0x12, 0x220ba80, 0xc4202c9320)
/go/src/github.com/netapp/trident/vendor/github.com/spf13/cobra/command.go:766 +0x2c1
github.com/netapp/trident/vendor/github.com/spf13/cobra.(*Command).ExecuteC(0x220bf40, 0x0, 0xc420179f68, 0x137f8b5)
/go/src/github.com/netapp/trident/vendor/github.com/spf13/cobra/command.go:852 +0x30a
github.com/netapp/trident/vendor/github.com/spf13/cobra.(*Command).Execute(0x220bf40, 0xc420179f78, 0xc42008e058)
/go/src/github.com/netapp/trident/vendor/github.com/spf13/cobra/command.go:800 +0x2b
main.main()
/go/src/github.com/netapp/trident/cli/main.go:14 +0x3c
.  Resolve the issue; use 'tridentctl uninstall' to clean up; and try again.
We can clearly print that the user needs to add the aggregate into the aggr-list of the SVM rather than print this error which is very inconclusive and confusing:
FATA  Install failed; could not start the storage backend driver; unable to inspect ONTAP backend: runtime error: invalid memory address or nil pointer dereference

Can this be assigned back to me, I know the fix for this issue

Can this be assigned back to me, I know the fix for this issue

same issue, any advise？

Can this be assigned back to me, I know the fix for this issue

same issue, any advise？

To overcome the issue, you need to manually add the aggregate to the svm by using a CLI or a ZAPI call to the ONTAP cluster:
"vserver modify -vserver  -aggr-list "  --> CLI
"vserver-modify" or "vserver-modify-iter" --> ZAPI

This issue was fixed in 19.04.


Broken objects in hot/sav-acacia-denmyre (DESERT)

Describe the problem

What Should Happen
I guess there should have been some trees
What Biome
hot/sav-acacia-denmyre
Object Id
trees/acacia/denmyre3@131398087, trees/acacia/denmyre4@1811430193, trees/acacia/denmyre6@608287537, trees/acacia/denmyre1@909571517, trees/acacia/denmyre1@882435827
Minecraft Version
1.17.1
Iris Version
1.5.6
Relevant log output. (IF APPLICABLE)
No response
Checklist

[X] I am using an unmodified version of the overworld. (If you modified the overworld and see an issue, contact support)
[X] I am using the latest version of the overworld OR I have made sure this issue was not already resolved on the latest version.
[x] I am using Spigot, Paper, Tuinity, or Purpur. (If you are not, and still think it is a valid issue, contact support)

These are the objects Iris was complaining about not being able to find earlier as well. I think it was this Iris issue

Is this still happening on the latest @BenjaminHoegh ?

It does. The food I tried isn't working afaik

Could this be a replacer thing? Is that biome using edit tags?

yes:
{
            "chance": 0.3,
            "edit": [
                {
                    "find": [{"block": "spruce_leaves"}],
                    "replace": {"palette": [{
                        "block": "dark_oak_leaves",
                        "data": {"persistent": true}
                    }]}
                },
                {
                    "find": [{"block": "birch_leaves"}],
                    "replace": {"palette": [{
                        "block": "dark_oak_leaves",
                        "data": {"persistent": true}
                    }]}
                },
                {
                    "find": [{"block": "dark_oak_fence"}],
                    "replace": {"palette": [{
                        "block": "dark_oak_leaves",
                        "data": {"persistent": true}
                    }]}
                }
            ],
            "density": 2,
            "rotation": {
                "yAxis": {
                    "min": 0,
                    "max": 0,
                    "interval": 90,
                    "enabled": true
                },
                "enabled": true
            },
            "place": [
                "clutter/oakdenclutt1",
                "clutter/oakdenclutt2",
                "clutter/oakdenclutt3",
                "clutter/oakdenclutt4",
                "clutter/oakdenclutt5",
                "clutter/oakdenclutt6",
                "clutter/oakdenclutt7",
                "clutter/oakdenclutt8",
                "clutter/oakdenclutt9",
                "clutter/oakdenclutt10"
            ],
            "translate": {
                "x": 0,
                "y": -1,
                "z": 0
            }
        }

Maybe it's due to the recent change with the way we handle leaf persistency?

Looks like hot/sav-forest has the same issue or something familiar

This is an intended object. It's a dead tree.
Try clicking it with /ir o d and doing /ir o p <whatever you got back>

Im removing all  "data": {"persistent": true} tags because iris does this auto now.

Figured it out... ill make a fix for these artifacts later.


chore: update public suffix list

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


whereTranslationLike and accents

Hello,
if i have a name of an item which is like "Menu à la card" and the full name is given over to the whereTranslationLike function like
Class:whereTranslationLike('name', 'Menu à la card')->firstOrFail()
the function does not find any results.
It's possible this fails with other special chars too, i did not test this.
EDIT: The normal whereTranslation() function works with this string.

Cool


chore(main): release 1.0.0

:robot: I have created a release beep boop
1.0.0 (2022-12-04)
update

update (f6054cc)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/School-of-Website-Engineering/Back-end-management-system-E-commerce/releases/tag/v1.0.0 :sunflower:


Use an image instead of needing to build every time

Instead of making the image with npm run start, I can change that to make it npm run build, which will make an image up-to-date with everything, and dockerhub can build it in the cloud.
This makes it faster for redeployment, I only need to re-download the image, rather than taking up time to build it.
Changes needed:
Dockercompose file will need to have an image (probably payload-neo)
Dockercompose file will have to have an entry command, npm run start
Dockerfile will need to change the npm run start to npm run build

Should note that in order for this to work, the deployment webhook should be moved from github to dockerhub.


Unable to hot reload. No instance of Flutter is currently running.

When I change some code or strings of Text widget and save the file, below error show in the console.
Then after I need to relaunch application to run.
Initializing hot reload...
Unable to hot reload. No instance of Flutter is currently running.
Unhandled exception:
Bad state: No element
#0      List.first (dart:core-patch/dart:core/array.dart:165)
#1      HotRunner._reloadSources.<anonymous closure> (package:flutter_tools/src/run_hot.dart:542)
#2      _rootRunUnary (dart:async/zone.dart:1134)
#3      _CustomZone.runUnary (dart:async/zone.dart:1031)
#4      _FutureListener.handleValue (dart:async/future_impl.dart:129)
#5      Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:637)
#6      Future._propagateToListeners (dart:async/future_impl.dart:666)
#7      Future._completeWithValue (dart:async/future_impl.dart:481)
#8      Future._asyncComplete.<anonymous closure> (dart:async/future_impl.dart:511)
#9      _rootRun (dart:async/zone.dart:1126)
#10     _CustomZone.run (dart:async/zone.dart:1023)
#11     _CustomZone.bindCallback.<anonymous closure> (dart:async/zone.dart:949)
#12     _microtaskLoop (dart:async/schedule_microtask.dart:41)
#13     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50)
#14     _runPendingImmediateCallback (dart:isolate-patch/dart:isolate/isolate_patch.dart:113)
#15     _RawReceivePortImpl._handleMessage (dart:isolate-patch/dart:isolate/isolate_patch.dart:166)

Please add the output of flutter doctor

Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel beta, v0.3.2, on Linux, locale en_IN)
[!] Android toolchain - develop for Android devices (Android SDK 27.0.3)
✗ Android license status unknown.
[✓] Android Studio (version 2.4)
✗ Flutter plugin not installed; this adds Flutter specific functionality.
✗ Dart plugin not installed; this adds Dart specific functionality.
[✓] Android Studio (version 3.0)
✗ Flutter plugin not installed; this adds Flutter specific functionality.
✗ Dart plugin not installed; this adds Dart specific functionality.
[✓] Android Studio (version 3.1)
[✓] IntelliJ IDEA Community Edition (version 2018.1)
[✓] Connected devices (1 available)
! Doctor found issues in 1 category.

Seen this one twice today, on two different dev machines. This line assumes there is at least one reload report, which apparently isn't always true.

cc @aam

Reproduced twice across quite a lot of attempts. Both times were on first launch after clean builds.

Thanks for the report. This seems to be a different flavor of https://github.com/flutter/flutter/issues/14517, which was papered over. Generally speaking, lack of reload report from the VM should be treated as a fatal error since from that point on flutter on the host machine can't have any confidence in what is happening on the device.
Ideally we figure out why Dart VM didn't provide a report. If it is possible to repro this failure with engine with --trace_reload and --trace_reload_verbose added to the https://github.com/flutter/engine/blob/master/runtime/dart_vm.cc#L68, that should help

I believe this is stale. #14517 is fixed.
Please let us know if you're still seeing this problem.

Hot reload failed: Bad state: No element


Consider adding provenance information to all written data

Potential User Stories

A Perkeep format upgrader has all the information it needs to upgrade old data stores to new.
A Perkeep developer can evolve the the underlying schema.
An information archeologist can fully understand Perkeep content because it is fully self describing.
A content importer can understand when and how previously imported content was stored.
A user can see exactly how content entered Perkeep and trace it back to the original system.

There are some simple things we can do to enable this:

Store the version of Perkeep along with a reference to the config hash when data is written.
For importers store canonical information about the import, exact date imported, version of importer used, hash of config used, strong identifiers from the remote system etc.
Consider importing other implicit claims from external systems that can prove that the content originated there.

Open to any/all ideas on how to make the data stored more robust in a consistent way.

At least one bit of information we already have is the camliNodeType (e.g. twitter.com:tweet for a tweet node).


[Linux] use backing root in dehydrate functional tests

A number of tests in the DehydrateTests class fail on systems such as Linux where the virtualized repository's contents are not available at the mount location after the repository is unmounted.  This PR adjusts these tests so they refer instead to the backing root location anytime file paths are referenced after the repository is unmounted.
We also create a CheckDehydratedFolderAfterUnmount() helper method in DehydrateTests which encapsulates the different checks required on Windows and POSIX platforms.
This PR is probably most easily reviewed commit-by-commit (although the overall diff didn't turn out too bad either).  There is a whitespace-only commit (ce7ec2e3f5) for which I apologize, but it was just going to be a real headache trying to not to introduce additional missing CRs while refactoring, so I left that commit in to make the whole file consistent first.
Two particular commits I should note for extra scrutiny: first, in d19716138e, I pulled out the FolderDehydrateCreatedDirectoryParentFolderInModifiedPathsShouldOutputMessage() test as it appeared to me that since 7956bd95b3 it was now identical to the earlier FolderDehydrateParentFolderInModifiedPathsShouldOutputMessage() test.
And second, in a0264d6af5, I removed one call to GetVirtualPathTo() because as far as I could tell, it was being called an extra time when constructing the path to pass the fileToWrite to AppendAllText() in the FolderDehydrateFolderThatIsSubstringOfExistingFolder() test.  The current code in master looks like this:
string fileToWrite = this.Enlistment.GetVirtualPathTo(Path.Combine(folderToDehydrate, "App.config"));
this.fileSystem.ReadAllText(fileToRead);
this.fileSystem.AppendAllText(this.Enlistment.GetVirtualPathTo(fileToWrite), "Append content");

and ignoring the middle line, I just didn't see how this could be what was intended.
/cc @kewillford

/azp run GitHub VFSForGit Mac Functional Tests

/azp run GitHub VFSForGit Mac Functional Tests

/azp run GitHub VFSForGit Mac Functional Tests

/azp run GitHub VFSForGit Mac Functional Tests

/azp run GitHub VFSForGit Mac Functional Tests

Hi @wilbaker -- I believe this PR is ready for review, however, assiduous clicking on "Re-run failed tests" buttons has failed to get the Mac CI tests past the build stage; they both now seem to fail when trying to sign the kext.  Would you by any chance have a magic fix for that?
Thanks very much and cheers!

/azp run PR - Mac - Build and Unit Test

@chrisd8088 the PR - Mac - Build and Unit Test test is no longer required as there are some issues with the upgrade to Catalina (so don't worry about that failing).  The GitHub VFSForGit Mac Functional Tests build still runs the unit tests there so we still have coverage.
I will take a look through the changes, thanks!

Awesome, thank you!

Hi again @wilbaker ... just wondering if you had any thoughts on this PR.  I know there's a lot of renaming, but I hope it's all in a good cause (i.e., the final commit in the sequence, dfecd1f3aa8, is as clean as possible).
I should note again what's in the description, though, as @kewillford may also want to take a look at these:

Two particular commits I should note for extra scrutiny: first, in d19716138e, I pulled out the FolderDehydrateCreatedDirectoryParentFolderInModifiedPathsShouldOutputMessage() test as it appeared to me that since 7956bd95b3 it was now identical to the earlier FolderDehydrateParentFolderInModifiedPathsShouldOutputMessage() test.

And second, in a0264d6af5, I removed one call to GetVirtualPathTo() because as far as I could tell, it was being called an extra time when constructing the path to pass the fileToWrite to AppendAllText() in the FolderDehydrateFolderThatIsSubstringOfExistingFolder() test.  The current code in master looks like this:
string fileToWrite = this.Enlistment.GetVirtualPathTo(Path.Combine(folderToDehydrate, "App.config"));
this.fileSystem.ReadAllText(fileToRead);
this.fileSystem.AppendAllText(this.Enlistment.GetVirtualPathTo(fileToWrite), "Append content");

and ignoring the middle line, I just didn't see how this could be what was intended.

Hi again @wilbaker ... just wondering if you had any thoughts on this PR

@chrisd8088 apologies for the delay on this PR.  It's on my radar and I plan to review the changes.

Awesome, thanks, @wilbaker.  I know you're dealing with a bunch of sparse-related stuff right now too.

/azp run PR - Mac - Build and Unit Test

Hi @wilbaker (and @kewillford) -- I happened to notice that you might be starting on some new functional tests for dehydration in #1605, and I wondered if there was any chance I could persuade you to look over this PR before adding new dehydration tests, just because I'm selfishly hoping to avoid refactoring it again.  :-)
And I suppose I could argue that the effect of these changes is a net gain in readability ... I hoped for that, but may have introduced more verbosity than you desire.  Let me know what you think, and thanks as always!

Thanks for all the comments!  I'll try to work through these shortly -- much appreciated!

@wilbaker, @kewillford -- thanks for the close review and the suggestions!  I think I'm implemented them more-or-less as you intended and it does indeed, I think, improve the readability a lot.
Let me know what you think; this would clear up the last group of functional test failures on Linux not related to low-level implementation stuff which needs to be addressed in libprojfs.

Thanks very much!  I'll merge in a moment!  ❤


(WIP) useDragHooks/useDropHooks docs

Closes #2995
✅ Pull Request Checklist:

[ ] Included link to corresponding React Spectrum GitHub Issue.
[ ] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[ ] Filled out test instructions.
[ ] Updated documentation (if it already exists for this component).
[ ] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

🧢 Your Project:
RSP

Hope this PR can be merged early. I'm trying to use @react-aria/dnd to build a draggable Tree Component. It's hard to continue without documentation. 😂

@MrKou47 Thanks for your interest in the drag and drop hooks! Unfortunately this doc is actually covering the @react-spectrum/dnd drag hook wrapper so it won't go into depth regarding the aria drag and drop hooks themselves. I'd encourage you to look at these stories for examples of how you could go about using the aria drag and drop hooks themselves in the meantime! ListView is also a good place to look.

@devongovett did a rough update to the structure, is this what you were envisioning?

performing final edits/revisions/proof reading so back into wip


add a few valid dependency cases

disclaimer: I'm not a lawyer :)
based on Odoo's licenses:

LGPL-3/MIT module can depend on OEEL-1/OPL-1 module

also MIT module can depend on LGPL-3 module (but not on Apache module; removed Apache from the comment to avoid ambiguity; wasn't taken into account anyway)

Thanks. Would you mind adding a news fragment (58.feature - or is it a bugfix?) and a couple of test cases to reproduce the problem this fixes?

I addressed the issues in #74 . @sbidoul Do you mind having a look?

Thanks @ng-ife
Looking at this again, I have a concern. Imagine

A: AGPL
B: LGPL
C: proprietary
With A depending on B and B depending on C.

Before this change, B depending on C will be flagged.
With this change, B can depend on C. But then A (AGPL) depends on B, which depends on a proprietary module, so that is not ok?
Or do I miss something?

@sbidoul Thanks for your feedback.
Situation seems to be a bit tricky. (As indicated here.
Following this argumentation I would say A based on B should already be invalid.
What do you think?

Thanks @sbidoul for clarification. This way it really seems to be impossible to open source code that is based on proprietary software (in this case Odoo EE modules). This is to bad but I get the point. Will close #74 then.

I'm not saying I have the definitive answer :) It's a complex matter.
It might make sense to have open source that depends on proprietary, but if we allow it in manifestoo, we'll probably need a more sophisticated algorithm to detect transitive incompatibilities.

MIT module can depend on LGPL-3 module (but not on Apache

@nilshamerlinck out of curiosity, do you have a reference about MIT/Apache compatibility ?


lazyLoad don't load after slidesToShow changes.

I have a vertical slick, and the height of slick will fit height of window, so when window resize, slick's slideToShow will change.
and I'm using lazyLoad={true}, when slideToShow increase, the additional slide is not rendered.

the $0 item is what has been rendered, and those active but empty item is what should be rendered but empty.

All the related issues have been fixed in previous releases.


Type Annotation Failures

Thanks a ton for your work on this!
I can't seem to serialize my Rust-defined types, as written up in https://github.com/danburkert/prost#serializing-existing-types. This snippet from the README:
use prost::Message;

#[derive(Clone, Debug, PartialEq, Message)]
struct Person {
  pub id: String, // tag=1

  // NOTE: Old "name" field has been removed
  // pub name: String, // tag=2 (Removed)

  #[prost(tag="6")]
  pub given_name: String, // tag=6
  pub family_name: String, // tag=7
  pub formatted_name: String, // tag=8
}

Generates:
  |
4 | #[derive(Clone, Debug, PartialEq, Message)]
  |                                   ^^^^^^^
  |
  = help: message: called `Result::unwrap()` on an `Err` value: invalid message field Person.id

          Caused by:
              no type attribute

Adding the type-annotations inside the prost macro cures this.
Am I missing something? My impression is that since #73, the un-annotated version should work.
Thanks!

I should add: here are the Prost versions I'm using:

[dependencies]
prost = "0.6.1"
prost-types = "0.6.1"
# ...

[build-dependencies]
prost-build = "0.6.1"

I believe they are the latest.

Yeah it looks like the example in the README is wrong, thanks for pointing it out.  Type annotations are always necessary; that PR included type inference in a very early iteration, but it was removed because it's not robust.  The example should have been updated to include type annotations.

Sent quick, working update to the README example: #365.

Thanks!


Python source distribtests

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

Kokoro build triggered here: https://fusion.corp.google.com/invocation/bdd63f05-8619-43f8-9666-4e68f4e829d4/detail?search_pattern=artifact&include_inactive_projects=false

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

https://fusion.corp.google.com/runanalysis/test/prod%3Agrpc%2Fcore%2Fexperimental%2Fgrpc_build_artifacts_multiplatform/prod%3Agrpc%2Fcore%2Fexperimental%2Fgrpc_build_artifacts_multiplatform/KOKORO/bdd63f05-8619-43f8-9666-4e68f4e829d4/1524780424092/prod%3Agrpc%2Fcore%2Fexperimental%2Fgrpc_build_artifacts_multiplatform Build %23179/Target Transitions?target=grpc/core/experimental/grpc_build_artifacts_multiplatform
@matt-kwong For some reason the artifact build did not trigger the package build step. Retrying...

https://source.cloud.google.com/results/invocations/54faa296-ee3d-4909-96b9-0a2d14ea5e9b/targets/github%2Fgrpc/tests
What's the distinction between pythonand python_dev distribtests?

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

Distribtests passed in https://fusion.corp.google.com/invocation/024de03e-f567-49a5-a4b1-3648346a5104/buildlogs

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences


Mention correct order in SimplechannelPool javadocs

Cherry-picked into 4.0 (76d51b514aeb1041d1e1e8ddc450f2e046630ac9), 4.1 (2a848ce1c94100e9fd3e063740d66187f17a3e8c) and master (895ceb72a3d8cd30911b8f7887a5df2b7c50d2d0)


Discord Async Event doesn't update rich presence when using variables inside np_setpresence();

Tried methods like calling np_update(); once every 5 seconds (as opposed by new discord "update presence" method, as you can update only 5 times every 20 seconds), deleting and creating the main NekoPresence object, manually forcing async updates and so on. Presence never updated after first initialization. Tried testing with variables or built-in variables (like mouse_x and mouse_y), only updated once and never again, even if np_update(); is being called.
No errors shown, everyting is working fine. Fresh project without any other objects inside, only NekoPresence

Tried methods like calling np_update(); once every 5 seconds (as opposed by new discord "update presence" method, as you can update only 5 times every 20 seconds), deleting and creating the main NekoPresence object, manually forcing async updates and so on. Presence never updated after first initialization. Tried testing with variables or built-in variables (like mouse_x and mouse_y), only updated once and never again, even if np_update(); is being called.
No errors shown, everyting is working fine. Fresh project without any other objects inside, only NekoPresence

np_update is meant to be called every Step.


How to set column type when use HotColumn

I have tried below methods failed
hot-column(
    :data="'cells.value'"
    :type="'checkbox'"
    )

{
  settings: {
     type: 'checkbox'
  }
}

hot-column(
    :data="'cells.value'"
    :settings="settings"
    )

{
  settings: {
     type: 'checkbox'
  }
}

Hi @muei
would that https://jsfiddle.net/gvh6cfs1/ suit your project requirements?


Better fix for missing unsigned long to Poco::UInt64 alias

This partially reverts previous patch and adds conditional conversions for missing unsigned long.

This is totally incorrect (and still don't compile).
Both long and long long are 64 bit on both Linux and Mac OS X,
so POCO_LONG_IS_64_BIT is true.
The difference is only when int64_t is long (Linux) or long long (Mac OS X).

Ugh, you're right. Not sure how to enable TypeName<T> specializations only if the types are different though 😦

I couldn't figure out how to do this nicely with SFINAE, so I partially reverted this and used supported fixed-size types instead. I changed the return values as well to address https://github.com/yandex/ClickHouse/pull/1356#discussion_r144685939


[SPARK-44296][BUILD] Upgrade dropwizard metrics 4.2.19

What changes were proposed in this pull request?
This pr aims upgrade dropwizard metrics to 4.2.19.
Why are the changes needed?
The new version bring a bug fix related to metrics-jetty module:

https://github.com/dropwizard/metrics/pull/3379

Does this PR introduce any user-facing change?
No
How was this patch tested?
Pass Github Actions

Thanks @yaooqinn

Thanks @HyukjinKwon @yaooqinn

Merged to master


Spaces around/after named parameter

Should there be spaces after/around name parameters?
foo(
    arg1='like this?',
    arg2 = 'or like this?',
    args3= 'or like this?'
)

Closing this as it is mentioned in section 3.6 Whitespaces


M adds confirmation window when click 'accept request'

window has two btns confirm and cancel;
when confirm is clicked request is marked as inactive, etc. Later chat room window will be open on 'confirm' click;
when cancel is clicked window is closed and nothing happens;

This prevents translators from accidentally accepting requests.

Tested, approved, great work!


issue with Transporter

Hello
I'm trying to use transporter in my project, I made few changed for the function conflicting with tensorflow 2.0
now i'm trying to train my custom dataset, by this code
for idx, image_a_batch in enumerate(dataset_a):
for idx, image_b_batch in enumerate(dataset_b):
with tf.GradientTape() as tape:
transporter_results = model(image_a_batch, image_b_batch, is_training=True)
reconstructed_image_b = transporter_results['reconstructed_image_b']
loss = reconstruction_loss(image_b_batch, reconstructed_image_b)
params = model.trainable_variables
grads = tape.gradient(loss, params)
opt.apply(grads, params)
Until the loss function everything is working fine, but when trying to get the trainable_variables from the model I got this error
ValueError: Transporter(
encoder=Encoder(),
keypointer=KeyPointer(
num_keypoints=10,
gauss_std=0.1,
keypoint_encoder=Encoder(name='keypoint_encoder'),
),
decoder=Decoder(initial_filters=4, output_size=[128, 128]),
) does not currently contain any trainable_variables.
Most Sonnet modules create variables the first time they are called with an
input and requesting variables before this typically indicates a coding error.
You should refactor your code such that you request module variables after you
pass an example input to the module. For example:
module = Transporter(
encoder=Encoder(),
keypointer=KeyPointer(
               num_keypoints=10,
               gauss_std=0.1,
               keypoint_encoder=Encoder(name='keypoint_encoder'),
           ),
decoder=Decoder(initial_filters=4, output_size=[128, 128]),

)
output = module(input)
params = module.trainable_variables
tensorflow version is 2.2.0
sonnet is 2.0

You need tensorflow 1.13.1, as it states in requirements.txt
I think the only compatible Python versions are 3.6 & 3.7

You can download from these links
Python3.6: https://www.python.org/downloads/release/python-360/
Python3.7: https://www.python.org/downloads/release/python-370/


Unable to send notifications after NFS failure (@visual61 & @kryz_9850)

Describe the bug
After we had a NFS failure today these two atSigns could not send notifications. The log files looked fine and I was able to authenticate to the atServer and notifications were accepted by the secondary but never sent on to the recepient atSign.
This was tested by using at_talk before restarting the atSigns atServers.
attalk was able to log in to the atServer and would accept notifications but they were never receieved.
After restarting the atServer sending and receiving came back. I restarted visual61 and it could send and kryz_9850 could receive but could not send until it was also restarted.
Hopefully other atServers are not effected but these atServers are busy all the time and both atServers were effected. Interestingly the atGPS demo was un effeted and kryz_9850 sending to visual61 was not effceted in the first outage at 1:46 but was taken down on the 4PM outage see screen shots..
All client apps also had to be restarted before the notifications came through which was also ugly.
As soon as the atServer came back online there was a huge flurry of notifications so it would seem none were lost.
This could be a corner case or a major issue ??
Steps to reproduce
Unknown
Expected behavior
This should never happen
Additional context
KRYZ Graphs (stats from kryz_9850 to visual61)

GCP Secondaries load average during the NFS event

Possible post-mortem item?


Revert "Disable UBSAN in zig builds. (#3292)"

This reverts commit a316f9e7fc7f8e06852a40544e57f89018672ee4.
Issue #1891 was fixed again, so this is no longer needed.

@rcorre thanks for the review!


Ui text TextDirection implicitly hidden by this plugin TextDirection

I'm new to flutter, I tried to use this plugin with text that has TextDirection attribute, but it says Ui TextDirection implicitly hidden by this plugin TextDirection.

I don't know what that means. This package is not Flutter-specific, so that message is probably coming from Flutter, and you should ask there.

Sounds like there are members named TextDirection in 2 libraries imported at the same time.
You can use hide or as with your import
See also https://stackoverflow.com/questions/19723063/what-is-the-difference-between-show-and-as-in-an-import-statement

@zoechi import 'package:intl/intl.dart' as intl; Solved the issue.
Thanks!!

Oh, duh. Thanks @zoechi


Rename (refactor) parameter object doesn't rename the shape declaration

I have the following code:
 getExistingThing({
        id,
        spaceId
    }: {
        id: string,
        spaceId: string
    }): IThingInfo {
   // Use the params object here with destructuring...
}

When I tried to refactor id to thingId, it renamed it correctly in the parameter object variable and the symbols in the function body, but it did not rename it in the "shape" of the parameter object (I don't know what else to call it).
Expected:
    getExistingThing({
        thingId,
        spaceId
    }: {
        thingId: string, <<< should change here
        spaceId: string
    }): IThingInfo {

Got:
    getExistingThing({
        thingId,
        spaceId
    }: {
        id: string, <<< doesn't change though
        spaceId: string
    }): IThingInfo {

I would also love it if someone could tell me what the various parts of this kind of function declaration is called. I use it for named arguments which I love, but I just don't know what the various pieces are called. I just know it's awesomeness.

Also it fails in the reverse as well. If I perform the refactor on the shape declaration, it does not change the corresponding symbols in the parameter object, nor in the function body.

Ah, This apparently is a TypeScript problem and not a VS code problem, which is why I couldn't find it:
https://github.com/Microsoft/TypeScript/issues/6312

Yep, this is certainly annoying. I think it's root cause is related to issue #6459 that I created,  #6312 that my issue was close enough to label as 'duplicate', as well as other issues it's connected to.


 Particularly important when using integrations such as Pushover which have a monthly cap (which I have already hit due to the excessive amount of messages).
A test has also been added for the new functionality.

@martomi I will do your changes this week.

@martomi Completed everything as requested. I can also rebase the pull request if you want a single commit

I was waiting on the last piece of feedback to be addressed. But considering a decent amount of time has passed already, let me address that myself so we can merge it.


Phantom.js swallows errors

The following example doesn't output anything at all and I haven't found a way to detect the throw.
var page = require('webpage').create()

page.open('http://example.com/', function () {
  throw new Error('Kill me')
})

Duplicate. Please see #12973

Thanks!


KAFKA-6905: Document that Transformers may be re-used by Streams

This is a follow-up to #5022 which added documentation to the Processor
interface. This commit adds similar documentation to Transformer and
ValueTransformer.
Also, s/processor/transformer/ in the close() docs.

Call for second review @guozhangwang @bbejeck @vvcephei


feat: add Dockerfile

@keivanipchihagh review this merge request

You right, i forgot to remove it. it comes from the docker template.
On Sat, 24 Aug 2024, 15:50 Keivan Ipchi Hagh, @.***>
wrote:

@.**** commented on this pull request.
In Dockerfile
https://github.com/farbodbj/SubFinder/pull/3#discussion_r1729951294:

@@ -0,0 +1,25 @@
+FROM golang:1.21-alpine AS builder

+WORKDIR /go/src
+# Add gcc and libc-dev early so it is cached
+RUN set -xe \

&& apk add  gcc libc-dev git curl

Do we need gcc for this?
—
Reply to this email directly, view it on GitHub
https://github.com/farbodbj/SubFinder/pull/3#pullrequestreview-2258685209,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXUT4QMSDN5GO5ACKHM44VDZTB3B5AVCNFSM6AAAAABM6QJBDKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDENJYGY4DKMRQHE
.
You are receiving this because you authored the thread.Message ID:
@.***>


fix: IsNil fails on struct value

Fixes #501
IsNil() fails when given a struct value. That caused the oracle provider to fail. Other providers are not affected, they all use pointers :)

/approve

/merge


Add support for distributor TME (Issue #76)

This PR implements #76. Contrary to other distributor implementations, the code has to do additional XmlHTTPRequests to obtain pricing information and stock availability (because the website is not loaded as a single HTML page).
This implementation does up to four HTTP requests per part. A modified implementation could reduce this to two requests (explained in a TODO comment). Is it OK to postpone this optimization to the future?

I add you file in a new branch that I am creating https://github.com/hildogjr/KiCost/tree/restructuration (missing add in the documentation).
I intend to deal with #34 and improve the reliability of the code. I would like that you test (uncomment the line 38 in the file distributors_definitions.py in the folder distributors. In my connection, sometimes the TME got timeout.

Hi @hildogjr,

I add you file in a new branch that I am creating

Why?

A just tried to add because one of the proposed modification in my new branch is that the user do not need to modify the main kicost.py code to add new distributors or cad software. I think this would make more easy the community to add new modules (you do not need to understand) and maintenance (some people have this two different focus: software/kicad module and internet/distributor).
Also, I do not add the documentation to keep you request merge to main branch and authorship of the file when I request my merge.
I am from Brazil, now it is working. I will try to modify the number of attempts in the next time of time out, thanks by the tip.

I've just updated the commit so that it is no longer in conflict with the recent changes in master.

Hi @hildogjr,

A just tried to add because one of the proposed modification in my new branch is that the user do not need to modify the main kicost.py code to add new distributors or cad software.

The best approach is IMHO to add the support for TME in one PR and to do your refactoring in another PR (without the new file tme.py; I'm ok with updating it according to the new structure if the refactoring PR gets merged first). Otherwise the commit history is convoluted -- it's not a good idea to add the new distributor in a commit named Files restruturation (issue #77).

I've just updated the commit to be in sync with the last changes in the structure.

Adam, I merged your module into the master branch. It looks like getting part info out of TME is very complicated. Thanks for doing that! I added you to the list of authors.
I did correct a small error that appeared when running KiCost under Python 3.5: I had to decode the response from the TME URL request into a string, otherwise json.loads() would choke on it.

Hi @xesscorp, thank you and sorry for the error.

No problem, Adam. I do the same thing all the time. Thanks for the TME scraper!

Now that we are adding new distributors. We may add some control when call kicost able or disable some distributors and the number of retry.
We could add --retry [n] to modify HTML_RESPONSE_RETRIES and --distributors to just use the distributors that we want to scrap.
What do you think?

--distributors to just use the distributors that we want to scrap

The --include option does exactly what you want.

True, I read now.


Could I get the test set label to host a Kaggle inclass competition?

I am a first-year graduate student at SJTU and I am also a teaching assistant. I want to host a kaggle inclass competition with video level feature youtube-8m data for students. But I found that I couldn't have access to the data whose labels students don't know.
Could I get some test set label of Youtube 8m dataset for evaluating the result? I mean there is no need for the labels of all test set, part of them (may be the label of 200MB Video features) is  enough. Or if you have some other suggestions, I would also be very grateful!
Thanks in advance!

Hi,
I suggest using part of the validation set. Even a 1/10 sample of the validation set will match the test set results very closely.

I agree that they should release the test set after the competition has ended.
It might be fun and educational to have the students all contribute to build a test set.

On Oct 28, 2020, at 2:53, Nisarg Kothari<EMAIL_ADDRESS>wrote:
Hi,
I suggest using part of the validation set. Even a 1/10 sample of the validation set will match the test set results very closely.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hi,
I suggest using part of the validation set. Even a 1/10 sample of the validation set will match the test set results very closely.

Hi LeegleechN
Thanks for your prompt reply and suggestions! Maybe I should delabel part of the dataset and randomly sample them.
Thanks again.

I agree that they should release the test set after the competition has ended. It might be fun and educational to have the students all contribute to build a test set.

Hi ramcandrews,
Thanks for your suggestion. This sounds very challenging and interesting. I will talk with my professor later.


Frequent IDE crashes when editing python files

This happens at random times and on random files.  Can you advise how to collect a useful crash dump?  Unfortunately Visual Studio's crash recovery system kicks in and automatically restarts the IDE for me so I can't attach a debugger.
On the other hand, because of that, maybe you already have my crash dumps somewhere?  If I can provide you with some sort of ID by which you can look up my crashes, that might also help me produce a better bug report.
Thanks!

The simplest thing to check first would be Event Viewer. There may be an Error entry under Windows Logs/Application, with a useful call stack.

Yep, unfortunately the call stack is pretty generic.
Application: devenv.exe
Framework Version: v4.0.30319
Description: The process was terminated due to an unhandled exception.
Exception Info: System.ArgumentException
   at System.Collections.Generic.IntrospectiveSortUtilities.ThrowOrIgnoreBadComparer(System.Object)
   at System.Collections.Generic.ArraySortHelper`1[[System.__Canon, mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089]].Sort(System.__Canon[], Int32, Int32, System.Collections.Generic.IComparer`1<System.__Canon>)
   at System.Array.Sort[[System.__Canon, mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089]](System.__Canon[], Int32, Int32, System.Collections.Generic.IComparer`1<System.__Canon>)
   at System.Array.Sort[[System.__Canon, mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089]](System.__Canon[], System.Collections.Generic.IComparer`1<System.__Canon>)
   at System.Array.Sort[[System.__Canon, mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089]](System.__Canon[], System.Comparison`1<System.__Canon>)
   at Microsoft.PythonTools.PythonAnalysisClassifier+<OnNewAnalysis>d__6.MoveNext()
   at System.Runtime.CompilerServices.AsyncMethodBuilderCore+<>c.<ThrowAsync>b__6_1(System.Object)
   at System.Threading.QueueUserWorkItemCallback.WaitCallback_Context(System.Object)
   at System.Threading.ExecutionContext.RunInternal(System.Threading.ExecutionContext, System.Threading.ContextCallback, System.Object, Boolean)
   at System.Threading.ExecutionContext.Run(System.Threading.ExecutionContext, System.Threading.ContextCallback, System.Object, Boolean)
   at System.Threading.QueueUserWorkItemCallback.System.Threading.IThreadPoolWorkItem.ExecuteWorkItem()
   at System.Threading.ThreadPoolWorkQueue.Dispatch()
   at System.Threading._ThreadPoolWaitCallback.PerformWaitCallback()

But this triggered my memory.  One time I did manage to get a debugger attached (don't remember how) and it reproduced under the debugger.  There, I was able to see that it thought a comparison function was in use somewhere that was inconsistent (e.g. returned different results on same inputs, or something to that effect).

We have fixed a sorting issue very recently, and it was backported to the latest 5.3 update.
https://github.com/Microsoft/PTVS/issues/2921

That would do it.  I haven't downloaded the latest update yet.  I'll go ahead and close this, and re-open it if I encounter it again using the latest update.  Thanks!


fix script:

avoid race condition between save_mets...

in 1st cmd of pipeline when doing ocrd workspace find --download
in 2nd cmd of pipeline when doing ocrd workspace add
...by not using a pipeline at all!

re-use the old file ID (if possible), but replace input with output file group
pass on pageId, if it exists
exit on failure

Thanks!