sample question

In running the segment_sample file, I found that the speed is very slow and my image size is 512*512. How can I solve the problem of slow speed?

Well, going through the 1000 steps iteratively just takes a while. You can try to speed it up using a different timestep spacing. However, this comes at a cost of image quality

It's like this, either reduce the timestep or reduce the size of the image. I want to continue deep learning on the generated image obtained by sampling, but I found that the generation is really slow.

@xuecheng990531
can you tell me about you result in you dataset?
beacuse i use the Brast2020 dataset and do the same preprocessing as the paper did. But the result is not just a mask as in the paper. The result is a mixture of the mask and the original image.for example.

so i wonder what about your dataset.
If it is convenient, can you add my email or wechat. thanks a lot
email<EMAIL_ADDRESS>wechat: g479251920

@xuecheng990531 你能告诉我你在数据集中的结果吗？ 因为我使用 Brast2020 数据集并进行与论文相同的预处理。但结果并不像论文中那样只是一个面具。结果是蒙版和原始图像的混合。例如。 所以我想知道你的数据集怎么样。 方便的话可以加我邮箱或者微信。非常感谢<EMAIL_ADDRESS>微信：g479251920

我的也是这样，在最后采样的时候会生成多个mask，里面有一些mask的结果跟你这种是一样的，可以选择在训练的时候多迭代，也可以选择更加密集的timestep进行采样，效果会好很多，但是时间会更长。我觉得diffusion用在分割领域还是有些困难

@xuecheng990531 最后采样生成多个mask是因为作者加了for循环，这无所谓。不是很理解你说的效果会好很多是什么意思，更多次的迭代和更大的T效果好很多是指图像更清晰吗，但我想表达的是我用brast2020训练的分割结果不仅包含了分割的掩模，还包含原始mri的轮廓，我并不想生成原始的mri的轮廓，但作者的文章里只生成了掩模，所以我想是不是我哪里出了问题。看你在自己的数据集上训练了，所以想问下，你的结果是只有掩模还是说掩模和原图的混合。
非常期待你的回复。

@xuecheng990531 最后采样生成多个mask是因为作者加了for循环，这无所谓。不是很理解你说的效果会好很多是什么意思，更多次的迭代和更大的T效果好很多是指图像更清晰吗，但我想表达的是我用brast2020训练的分割结果不仅包含了分割的掩模，还包含原始mri的轮廓，我并不想生成原始的mri的轮廓，但作者的文章里只生成了掩模，所以我想是不是我哪里出了问题。看你在自己的数据集上训练了，所以想问下，你的结果是只有掩模还是说掩模和原图的混合。 非常期待你的回复。

就是训练迭代还不够

Hi
Yes, in the BRATSdataset you have always 4 MR sequences (T1, T1ce, T2 and FLAIR) as well as the segmentation mask.
You will need both for training of the diffusion model, as shown in Figure 2 of the paper. How the data should be stored can be found in the mini-example in the folder /data.

@JuliaWolleb
i wonder why my result contain the profile of the mri, not just the mask as the paper show.
it's mine, not clear maybe don't have enough training, i don't care it, but i can't stand the profile  in the result.

it can be see in you paper, the result is very clear ,

for most, it don't have any profile of the mri. I wonder if the result is raw, or the result is processed so don't have the profile. if the result is raw, maybe my result contain the profile has something wrong with it, can you tips why my result contains the profile.
[look forward to your reply](javascript.

Hi
So the profile does look a bit weird, but I would not worry about it too much.
The final output should be thresholded by using Otsu or thresholding at 0.5. Then you get the clear binary segmentation, just as you have it in my figure. I think in your result, this thresholding is missing


Build test suite of L8 data

Allowing us to compare results to other vetted approaches and assess the impact of changes.

One of the challenges to this task is that the cloudmasking algorithm must operate per-scene as several of the operations require global percentiles. So we can't do clipped subsets without biasing the results.
The other option is to do reduced resolution overviews but there could be some bias from resampling and scale issues.
Final option is to write a script to be run on an EC2 instance that pulls full datasets from s3://landsat-pds, runs the operational test suite. The results could be downsampled/clipped and transferred locally for inspection.


KIP contract bug

Describe the bug
i found that KIP37.sol in klaytn-contracts does not support KIP37 interface.
it seems from missing of totalSupply function
formal interfaceID is calculated by
/*
*     bytes4(keccak256(‘balanceOf(address,uint256)’)) == 0x00fdd58e
*     bytes4(keccak256(‘balanceOfBatch(address[],uint256[])’)) == 0x4e1273f4
*     bytes4(keccak256(‘setApprovalForAll(address,bool)’)) == 0xa22cb465
*     bytes4(keccak256(‘isApprovedForAll(address,address)’)) == 0xe985e9c5
*     bytes4(keccak256(‘safeTransferFrom(address,address,uint256,uint256,bytes)’)) == 0xf242432a
*     bytes4(keccak256(‘safeBatchTransferFrom(address,address,uint256[],uint256[],bytes)’)) == 0x2eb2c2d6
*     bytes4(keccak256(‘totalSupply(uint256)’)) == 0xbd85b039
*
*     => 0x00fdd58e ^ 0x4e1273f4 ^ 0xa22cb465 ^
*        0xe985e9c5 ^ 0xf242432a ^ 0x2eb2c2d6 ^ 0xbd85b039 == 0x6433ca1f
*/
but totalSupply function is moved to preset/KIP37Supply.sol
so,
interfaceId == type(IKIP37).interfaceId
in https://github.com/klaytn/klaytn-contracts/blob/master/contracts/KIP/token/KIP37/KIP37.sol#L45-L49
does not match with the formal interfaceId.
How to reproduce
Steps to reproduce the behavior:
Expected behavior
A clear and concise description of what you expected to happen.
Attachments
If applicable, add data, logs or screenshots to help explain your problem.
Environment (please complete the following information)

Klaytn version or git revision that exhibits the issue $ ken version
The version pragma of the smart contract(s) involved
OS and its version [e.g. macOS/10.14]

Additional context
Add any other context about the problem here.

Hi @JayChoi1736 thank you for highlighting this issue.
I can confirm the implemented IKIP37's interfaceId is 0xd9b67a26 while the standard KIP37's interfaceId was 0x6433ca1f. That's why interfaceId == type(IKIP37).interfaceId returns false
It is because (as you've mentioned) of skipping totalSupply(uint256) function from IKIP37.sol
So there are two ways to fix this issue

Declare totalSupply(uint256) function in IKIP37.sol & implementing it in KIP37.sol (as per KIP37 standard)
1.1. In this case I would suggest to merge KIP32Supply.sol with KIP37.sol
Update KIP37 to remove totalSupply(uint256) function from its interface and set its formal interfaceID 0xd9b67a26

@toniya-klaytn please suggest with which option I should proceed

@JayChoi1736  , You are right Formal interface id is : 0x6433ca1f.
When we forked contracts from openzepplien (oz), we want to follow the oz standard in contracts. However this breaks the KIP standard.
We can fix this by Declaring totalSupply(uint256) function in KIP37.sol as per the standard.


Blogs: Enable Pagination

Added the active page to the listBlogReq. An offset is added to the
listBlogs query and is created using the active page and limit.
Some logic changes including resetting the active page to 1 when
changing the 'Blogs per page' to avoid indexing issues.
TEST=manual
Ran page locally and validated feature works
Change-Id: Id55767108d20319debb7437ea8acd3d05b8b58e0

Code Review comments can be found at https://gerrithub.io/q/Id55767108d20319debb7437ea8acd3d05b8b58e0


Openssl trying to use the cipher algorithm from older version in newer version

Could we use the cipher algorithms from older versions in the newer versions?
Like "SEED-ECB". OpenSSL 1.0.1t version includes that cipher algorithm but does not include it in 1.1.1g version. I wanna use that cipher algorithm in 1.1.1g version.
Is it possible? If it is, how? :pray:

You need to load the legacy provider.
Have a look at the migration guide sections 1.5 and 6.2.

You need to load the legacy provider.
Have a look at the migration guide sections 1.5 and 6.2.

@paulidale your answer was about OpenSSL 3.0. @furkankahvecinsider is specifically asking about 1.1.1.

Like "SEED-ECB". OpenSSL 1.0.1t version includes that cipher algorithm but does not include it in 1.1.1g version. I wanna use that cipher algorithm in 1.1.1g version.

"SEED-ECB" support is included in OpenSSL 1.1.1. Why do you think it is not?

Perhaps you're using some build of OpenSSL-1.1.1 that uses no-seed?

You would have to rebuild the openssl with enable-seed build time option.

How can  I do to rebuild the OpenSSL with "enable-seed". I'm a little strange, sorry for that.

If you are using an OS distribution, they are likely changing the configuration when they package it. You will need to either patch the OS distribution (e.g. download RPM source, build, etc). Or download the original source from github/openssl.org and configure it yourself.
But this isn't really an OpenSSL issue, but a packaging issue.
You should ask the OS distributor why SEED is disabled.

Actually, I'm using this alpine Linux package, is this really matter?

Yes. Alpine explicitly disable seed in their packages. See line 105 here:
https://git.alpinelinux.org/aports/tree/main/openssl/APKBUILD

Haha so ironic, what should I do in this situation? Should I change to another package?

That's really up to you. There are a few options. You can for example build openssl locally if you need the Seed algo just for your application. Or you can report an issue to alpinelinux asking to re-enable that support. Or you can switch to a different Linux distro.


[ux] (Code Editor) Removing references to the Code Editor feedback button

Based on this internal Code Editor change, the Send Feedback button has been removed. This PR updates the docs accordingly.
@nancy-wang, do we have an alternate way to capture feedback and feature requests?

Docs Build status updates of commit 80c4d3a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Issue_Template.md
:white_check_mark:Succeeded

README.md
:white_check_mark:Succeeded

docs/docfx.json
:white_check_mark:Succeeded

docs/testing/troubleshooting.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Users can (and do) leave feedback through xlo's send-a-smile. Very roughly 3/4 of feedback comes through there already.


[1722] Warn user when they leave unsaved/changed form

Context
Setup check on forms with data-module="form" attribute that prompts the user that they have unsaved changes when attempting to leave the page.
Changes proposed in this pull request
JS module to handle form checks when user leaves with unsaved changes.
Notes
This might seem excessive but was recommended to cover all browsers/any deprecations.
event.preventDefault();
event.returnValue = "You have unsaved changes, are you sure you want to leave?";
return "You have unsaved changes, are you sure you want to leave?";

Also, please squash the commits on merge

 Warn user when leaving an enrichment page with unsaved changes


Fixed Delta-T

Issue #51
Delta-T calculation was off for imperial units since temp is always in Celsius and Wet Bulb was unit dependent.
Added a simple if, else for imperial

Hi Glenn,
You are right, the current function is off, as the Wetbulb temperature is returned in F, but your suggestion will not work.  When you do this:
        if self._unit_system == UNITS_IMPERIAL:
            deltat = ((temp * 1.8) + 32) - wb

You have the correct value in Fahrenheit
But then when the last line is run, it will convert the F temperature to F again. I will decline this PR, and I have made a different change, that should do the same.


Backend AMI filter

This ticket tracks the work to add a backend filter such that /listings will only return listings that have properties with units at or above the AMI provided.
Should use the AMI percentage from the unit summary

fallback could be the AMI at the listing level

related: #102

Contributing upstream in https://github.com/bloom-housing/bloom/pull/1797


[SPARK-9284] [tests] Allow all tests to run without an assembly.

This change aims at speeding up the dev cycle a little bit, by making
sure that all tests behave the same w.r.t. where the code to be tested
is loaded from. Namely, that means that tests don't rely on the assembly
anymore, rather loading all needed classes from the build directories.
The main change is to make sure all build directories (classes and test-classes)
are added to the classpath of child processes when running tests.
YarnClusterSuite required some custom code since the executors are run
differently (i.e. not through the launcher library, like standalone and
Mesos do).
I also found a couple of tests that could leak a SparkContext on failure,
and added code to handle those.
With this patch, it's possible to run the following command from a clean
source directory and have all tests pass:
mvn -Pyarn -Phadoop-2.4 -Phive-thriftserver install

Note I included a separate patch in this PR, since I was hitting that issue every time with these changes. I'll rebase once that patch is committed.

Merged build triggered.

Merged build started.

Test build #38281 has started for   PR 7629 at commit 8d2b0ea.

Test build #38281 has finished for   PR 7629 at commit 8d2b0ea.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Weird that scalastyle passed locally.

Merged build triggered.

Merged build started.

Test build #38387 has started for   PR 7629 at commit 0f5a5bb.

Merged build triggered.

Merged build started.

Test build #38393 has started for   PR 7629 at commit 3ec7cd4.

Test build #38387 has finished for   PR 7629 at commit 0f5a5bb.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
abstract class InternalRow extends Serializable
case class ChangeDecimalPrecision(child: Expression) extends UnaryExpression
class GenericRow(protected[sql] val values: Array[Any]) extends Row
class GenericInternalRow(protected[sql] val values: Array[Any]) extends InternalRow
class GenericInternalRowWithSchema(values: Array[Any], val schema: StructType)
class GenericMutableRow(val values: Array[Any]) extends MutableRow

Merged build finished. Test FAILed.

Test build #38393 has finished for   PR 7629 at commit 3ec7cd4.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
abstract class InternalRow extends Serializable
case class ChangeDecimalPrecision(child: Expression) extends UnaryExpression
class GenericRow(protected[sql] val values: Array[Any]) extends Row
class GenericInternalRow(protected[sql] val values: Array[Any]) extends InternalRow
class GenericInternalRowWithSchema(values: Array[Any], val schema: StructType)
class GenericMutableRow(val values: Array[Any]) extends MutableRow

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #38401 has started for   PR 7629 at commit 469a732.

Test build #38401 has finished for   PR 7629 at commit 469a732.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Ok, this is down to that kinesis test that's failing for everybody.

Jenkins, retest this please.

Merged build triggered.

Merged build started.

Test build #119 has started for   PR 7629 at commit 469a732.

Merged build triggered.

Merged build started.

Merged build finished. Test FAILed.

Test build #119 has finished for   PR 7629 at commit 469a732.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Jenkins retest this please.

Merged build triggered.

Merged build started.

Merged build triggered.

Merged build started.

Test build #138 has started for   PR 7629 at commit 469a732.

Merged build finished. Test FAILed.

Test build #138 has finished for   PR 7629 at commit 469a732.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Jenkins, retest this please.

Merged build triggered.

Merged build started.

Test build #140 has started for   PR 7629 at commit 469a732.

Merged build triggered.

Merged build started.

Test build #38774 has started for   PR 7629 at commit 469a732.

Test build #140 has finished for   PR 7629 at commit 469a732.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test build #38774 has finished for   PR 7629 at commit 469a732.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

retest this please

Merged build triggered.

Merged build started.

Merged build triggered.

Test build #182 has started for   PR 7629 at commit 469a732.

Merged build started.

Test build #39251 has started for   PR 7629 at commit 469a732.

Test build #182 has finished for   PR 7629 at commit 469a732.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test build #39251 has finished for   PR 7629 at commit 469a732.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

retest this please

Merged build triggered.

Merged build started.

Test build #183 has started for   PR 7629 at commit 469a732.

Merged build triggered.

Merged build started.

Test build #39280 has started for   PR 7629 at commit 469a732.

Test build #39280 has finished for   PR 7629 at commit 469a732.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test build #183 has finished for   PR 7629 at commit 469a732.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #39303 has started for   PR 7629 at commit 8e4a136.

Test build #39303 has finished for   PR 7629 at commit 8e4a136.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

retest this please

Merged build triggered.

Merged build started.

Test build #188 has started for   PR 7629 at commit 8e4a136.

Merged build triggered.

Merged build started.

Test build #39381 has started for   PR 7629 at commit 8e4a136.

Test build #188 has finished for   PR 7629 at commit 8e4a136.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test build #39381 has finished for   PR 7629 at commit 8e4a136.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #39451 has started for   PR 7629 at commit dada9d6.

Test build #39451 has finished for   PR 7629 at commit dada9d6.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

retest this please

Merged build triggered.

Merged build started.

Merged build triggered.

Merged build started.

Test build #190 has started for   PR 7629 at commit dada9d6.

Test build #39485 has started for   PR 7629 at commit dada9d6.

Test build #39485 has finished for   PR 7629 at commit dada9d6.

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

Merged build finished. Test PASSed.

Test build #190 has finished for   PR 7629 at commit dada9d6.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class MultilayerPerceptronClassifier(override val uid: String)
class KinesisUtils(object):
class InitialPositionInStream(object):
abstract class TernaryExpression extends Expression
case class SoundEx(child: Expression) extends UnaryExpression with ExpectsInputTypes

Merged build finished. Test FAILed.

retest this please

Merged build triggered.

Merged build started.

Merged build triggered.

Merged build started.

Test build #193 has started for   PR 7629 at commit dada9d6.

Test build #39557 has started for   PR 7629 at commit dada9d6.

Test build #193 has finished for   PR 7629 at commit dada9d6.

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

Merged build finished. Test PASSed.

Test build #39557 has finished for   PR 7629 at commit dada9d6.

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

Merged build finished. Test PASSed.

Yay, all tests passed. @srowen @pwendell maybe you guys could take a look? thanks!

I was going to say the docs need to be updated to mention you don't need to do package then test, but couldn't find that... maybe it's just in the wiki.
Obviously it would be great to not have to build the package to run tests, which is not what people will be used to with Maven. Are there any downsides?

Are there any downsides?

I don't think so. It shouldn't break anything for those using the current "build assembly then test" workflow, it just makes the first step unnecessary.

Merged build triggered.

Merged build started.

Test build #39640 has started for   PR 7629 at commit bf3cf35.

Test build #39640 has finished for   PR 7629 at commit bf3cf35.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
 public static final class SortedIterator extends UnsafeSorterIterator
 public class KVSorterIterator extends KVIterator<UnsafeRow, UnsafeRow>

Merged build finished. Test FAILed.

retest this please

Merged build triggered.

Merged build started.

Merged build triggered.

Merged build started.

Test build #214 has started for   PR 7629 at commit bf3cf35.

Test build #39769 has started for   PR 7629 at commit bf3cf35.

Test build #39769 has finished for   PR 7629 at commit bf3cf35.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test build #214 timed out     for PR 7629 at commit bf3cf35     after a configured wait of 175m.

Merged build finished. Test FAILed.

retest this please

Merged build triggered.

Merged build started.

Test build #39800 has started for   PR 7629 at commit bf3cf35.

Merged build triggered.

Merged build started.

Test build #222 has started for   PR 7629 at commit bf3cf35.

Test build #222 has finished for   PR 7629 at commit bf3cf35.

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

Merged build finished. Test PASSed.

Test build #39800 has finished for   PR 7629 at commit bf3cf35.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

I think this is good to go, unless anyone else has comments.

I'm into this. Flagging for @pwendell one more time just in case.

Hey guys - thanks this is a cool and useful change. One thing I was wondering, do you think we could wait until our builds are in better shape to merge this since it's off the critical path for the release? Right now all of our master builds are failing due to test errors (many people are looking at these to get in better shape). If this has some unexpected issues on one of the various matrix builds, it will be super hard to track them down.

Sure. But as the test history of this PR shows, builds have been in pretty bad shape for a while. :-(

If you break out the individual builds, many of them have been fine until the Hive 1.2.1 patch. The top level dashboard isn't that useful bcause if any one of the ~5 sub builds has a single flay test, it shows red.
For instance:
https://amplab.cs.berkeley.edu/jenkins/view/Spark QA Test (Dashboard)/job/Spark-Master-Maven/AMPLAB_JENKINS_BUILD_PROFILE=hadoop2.0,label=centos/

Merged build triggered.

Merged build started.

Merged build finished. Test FAILed.

retest this please

Merged build triggered.

Merged build started.

Test build #40803 has started for   PR 7629 at commit b16818f.

Test build #40803 has finished for   PR 7629 at commit b16818f.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

retest this please

Merged build triggered.

Merged build started.

Test build #40835 has started for   PR 7629 at commit b16818f.

Test build #40835 has finished for   PR 7629 at commit b16818f.

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

Merged build finished. Test PASSed.

Merged build triggered.

Merged build started.

Test build #41174 has started for   PR 7629 at commit e3585ec.

Test build #41174 has finished for   PR 7629 at commit e3585ec.

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

Merged build finished. Test PASSed.

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

Merged build triggered.

Merged build started.

Test build #41378 has started for   PR 7629 at commit d234399.

Test build #41378 has finished for   PR 7629 at commit d234399.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
 public static class AppExecId
 public static class StoreVersion

Merged build finished. Test FAILed.

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

retest this please

Merged build triggered.

Merged build started.

Test build #41461 has started for   PR 7629 at commit d234399.

Test build #41461 timed out     for PR 7629 at commit d234399     after a configured wait of 175m.

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

Merged build finished. Test FAILed.

le sigh. retest this please

Merged build triggered.

Merged build started.

Test build #41471 has started for   PR 7629 at commit d234399.

Test build #41471 timed out     for PR 7629 at commit d234399     after a configured wait of 175m.

Merged build finished. Test FAILed.

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

Seems like all tests passed, no idea why jenkins thinks they timed out.
AFAICT, this is good to do. @pwendell ?

Yeah sounds good - might be good to let it run one more time just to be
sure it's not affecting jenkins somehow.
On Mon, Aug 24, 2015 at 4:40 PM, Marcelo Vanzin<EMAIL_ADDRESS>wrote:

Seems like all tests passed, no idea why jenkins thinks they timed out.
AFAICT, this is good to do. @pwendell https://github.com/pwendell ?
—
Reply to this email directly or view it on GitHub
https://github.com/apache/spark/pull/7629#issuecomment-134414895.

Jenkins, test this please.

Merged build triggered.

Merged build started.

Test build #41492 has started for   PR 7629 at commit d234399.

Test build #41492 timed out     for PR 7629 at commit d234399     after a configured wait of 175m.

Merged build finished. Test FAILed.

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

175m is starting to look really low. the scala/java unit tests took 143m to run. anyway, retest this please.

Does this PR increase test time in some way? Just wondering why this would
consistently timeout when others don't.
On Tue, Aug 25, 2015 at 10:43 AM, Marcelo Vanzin<EMAIL_ADDRESS>wrote:

175m is starting to look really low. the scala/java unit tests took 143m
to run. anyway, retest this please.
—
Reply to this email directly or view it on GitHub
https://github.com/apache/spark/pull/7629#issuecomment-134682606.

Merged build triggered.

Merged build started.

No, it shouldn't affect the test runtime at all. And I get timeouts in lots of PRs, not just this one.

Test build #41539 has started for   PR 7629 at commit d234399.

Test build #41539 has finished for   PR 7629 at commit d234399.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test FAILed.

Jenkins, retest this please.

Merged build triggered.

Merged build started.

Test build #41559 has started for   PR 7629 at commit d234399.

Test build #41559 has finished for   PR 7629 at commit d234399.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #41574 has started for   PR 7629 at commit a2c7c59.

Test build #41574 has finished for   PR 7629 at commit a2c7c59.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

I've seen the failing test fail in other PRs (and it has passed here before)... let's try again just in case. retest this please

Merged build triggered.

Merged build started.

Test build #41584 has started for   PR 7629 at commit a2c7c59.

Test build #41584 timed out     for PR 7629 at commit a2c7c59     after a configured wait of 175m.

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

Merged build finished. Test FAILed.

Marcelo you will need to change the timeout in the code itself for it to
increase from 175
On Aug 25, 2015 11:24 PM, "UCB AMPLab"<EMAIL_ADDRESS>wrote:

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41584/
Test FAILed.
—
Reply to this email directly or view it on GitHub
https://github.com/apache/spark/pull/7629#issuecomment-134855001.

you will need to change the timeout in the code

Yes but I don't want to do that as part of this change, since they're unrelated things. All tests have been passing, the timeouts are unrelated to the PR.

Okay then - I would do that separately, get this PR passing, then merge it.
It is not good to merge a PR that deterministically fails jenkins. Have we
done that in the recent past? I saw a few other PR's that hit an occasional
timeout, but this one seems to be timing out with certainty every time.
On Wed, Aug 26, 2015 at 9:15 AM, Marcelo Vanzin<EMAIL_ADDRESS>wrote:

you will need to change the timeout in the code
Yes but I don't want to do that as part of this change, since they're
unrelated things. All tests have been passing, the timeouts are unrelated
to the PR.
—
Reply to this email directly or view it on GitHub
https://github.com/apache/spark/pull/7629#issuecomment-135082936.

but this one seems to be timing out with certainty every time

Often, but not deterministically every time. It seems to time out as often as any other PR that needs to run all tests (I've already cc'ed you on at least another one that times out just as often).

I still don't understand - why not fix the issue in a separate PR, get this
passing and then merge this? It will then benefit the other PR's also that
are facing this issue.
On Wed, Aug 26, 2015 at 9:46 AM, Marcelo Vanzin<EMAIL_ADDRESS>wrote:

but this one seems to be timing out with certainty every time
Often, but not deterministically every time. It seems to time out as often
as any other PR that needs to run all tests (I've already cc'ed you on at
least another one that times out just as often).
—
Reply to this email directly or view it on GitHub
https://github.com/apache/spark/pull/7629#issuecomment-135104616.

I'm working on fixing the root cause of the timeouts (running unnecessary tests). If you think it would be beneficial to just bump the timeout right now, please just send a PR for that; I'm pretty confident that this PR does not make the timeout issue any worse.

Jenkins, retest this please.

K - I just sent a hotfix to up the timeout.
On Wed, Aug 26, 2015 at 9:51 AM, Marcelo Vanzin<EMAIL_ADDRESS>wrote:

I'm working on fixing the root cause of the timeouts (running unnecessary
tests). If you think it would be beneficial to just bump the timeout right
now, please just send a PR for that; I'm pretty confident that this PR does
not make the timeout issue any worse.
—
Reply to this email directly or view it on GitHub
https://github.com/apache/spark/pull/7629#issuecomment-135105812.

Merged build triggered.

Merged build started.

Test build #41647 has started for   PR 7629 at commit a2c7c59.

Test build #41647 has finished for   PR 7629 at commit a2c7c59.

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

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

Merged build finished. Test PASSed.

Yay!

Great - looks good!
On Aug 26, 2015 3:53 PM, "Marcelo Vanzin"<EMAIL_ADDRESS>wrote:

Yay!
—
Reply to this email directly or view it on GitHub
https://github.com/apache/spark/pull/7629#issuecomment-135199475.

Ok I'm merging this.


Broadcasters not working sometimes, view only listed streamers

So sometimes if i set my broadcasters the stream just wont load for them. Can't recreate it by now. I changed nothing and now it's working. Probably my wifi or something. Another thing is that it doest switch back to more preferred streamers. It only checks after the stream from the guy you don't wanna watch ended. Solution for this would be a timer that also auto reloads the broadcaster streams after the interval or just letting me activate broadcaster streams without activating "watch_streams_when_no_drop". First method would be probably nicer but if both get implemented that would be really nice. Thanks for keeping it updated. I really appreciate your work.

First problem sounds like a connection issue.
Second problem is fixed in the latest commit, the bot will check every 5 minutes for a higher priority stream.


Mention users without any sorting

Current dependencies on/for this PR:

master

PR #11

PR #10  (this PR)

This comment was auto-generated by Graphite and will continue to be automatically updated while this PR remains open.


Alt-tabbing doesn't switch focus to different window

Tested versions
4.4 dev6
System information
Godot v4.4.dev6 - Windows 10.0.19045 - Multi-window, 1 monitor - Vulkan (Forward+) - dedicated AMD Radeon RX 580 2048SP (Advanced Micro Devices, Inc.; 31.0.21921.1000) - Intel(R) Core(TM) i5-7500 CPU @ 3.40GHz (4 threads)
Issue description
When switching from script editor to undocked shader editor, pressing common shortcuts (undo, copy & paste) are still performed in the script editor. This is because alt-tabbing doesn't switch focus to shader editor, I need to click on the shader editor to completely switch the focus. Alt-tabbing just brings up the shader editor without switching the focus to it.
This isn't an issue when switching from shader editor to script editor.
Steps to reproduce
open script editor and undock shader editor. write something in script editor, switch to shader editor and press ctrl+a. everything in script editor should be selected.
Minimal reproduction project (MRP)
new project

Can you re-post your system info? I think you had the wrong thing on your clipboard.

Can you re-post your system info? I think you had the wrong thing on your clipboard.
(If this is on Windows, there were several windows focus bugs during 4.3 and 4.4, so this may share a root cause with one of those)

I copied my script somehow lol. Fixed now.


API: subscribe should return the id of the feed

Creating a category will return an integer that I assume to be the id of the newly created category. Subscribing to a feed should work the same way in my opinion. That way the client can display a placeholder with the information it got locally or do a single call to /feed/get/{id}.

It was already the case but the documentation didn't reflect that :( Thanks!


Make command-line export tool available as a web app

Context
As part of #22, we've created a command-line export tool for our PO to export data from the cloud firestore to a csv file. The aim of this task is to convert that command-line tool to a web UI.
There are a few reasons why a web interface would be preferable to a command-line tool:

It is easier to use for our PO.
It is usable on any platform, including the Windows machine they currently use to access the MS Access database.
It is easier for us to distribute updates to the export tool to our PO. Such updates will be necessary if we find bugs in the export tool or when we start supporting collections other than just the habitat use collection.

In a mini-spike done as part of #22, we've created such a web interface for the export tool (branch export-web-app) and deployed it to https://bmmro-data-export.web.app/ as a second app under the same projectId of the firebase development environment.
Remaining tasks:

[ ] Modify the pipeline config to deploy the export web UI on push to master.
[ ] Make sure meaningful error validation is shown when each of the form fields contains invalid field input.
[ ] Discuss among devs whether we're satisfied with test coverage and add tests if necessary.
[ ] Modify the README in the export-data folder to reflect changes in how to use the export tool and how to run it locally.
[ ] Add instructions on how to use the web UI export tool to the operating manual doc in the Operating Manual folder of our google drive.

Additional notes:
We don't envision a need deploy an export tool per environment. The export tool, as it currently stands, is capable of exporting data from any environment because projectId and apiKey need to be filled out in the form.

NB: This work appears as though it was partially completed. We were unable to continue/finish the work as part of the Spring 2022 scope. But speaking with Charlotte, any improvements to the Exporting mechanism would be valuable so I recommend keeping this story alive for future works.


Implement a factory linting method like in FactoryGirl

I've found this useful for asserting the validity of my factories when working in Ruby, and think it would be nice in Elixir too. :)
https://github.com/thoughtbot/factory_girl/blob/master/GETTING_STARTED.md#linting-factories

@lpil I'm not entirely sure how this would work since the factory module is a bunch of functions with the same arity, so I don't know of a way to get a list of the defined factories. I'll have to look into this more.
Thanks for the suggestion

In that case I think you could just create a test in test/factory_test.exs
defmodule MyApp.FactoryTest do
  use ExUnit.Case

  test "all factories can save correctly" do
    for factory <- ~w(user project task whatever) do
      MyApp.Factory.create(factory)
    end
  end
end

Would that solve what you're trying to do? And if you wanted to add a task instead, you could do the same thing but in a task module that you run inside test_helper.exs
defmodule Mix.Tasks.MyApp.LintFactories do
  use Mix.Task

  def run(_opts) do
    Mix.Task.run "app.start"

    for factory <- ~w(user project task whatever) do
      MyApp.Factory.create(factory)
    end
  end
end

# test_helper.exs
Mix.Task.run "lint_factories"

Basically this but with helpful error messages and such so everyone doesn't have to rebuild this. Additionally if this exists it will encourage people to lint their factories and possibly write better code as a result.

@lpil What kind of error message do you think would be helpful? I personally don't think I'd use linting like this since when I run the test suite and there's with a factory I will see the error. If that error is from Ecto it's usually pretty clear (usually a null constraint). So before I add something like this, could you provide an example of when something has gone wrong, and an error that you think would have helped you solve the problem? That would help us figure out a solution

@lpil I'm going to close this for now. Right now I'm not sure I understand the benefit. Please comment here if you could give an example of what kind of helpful error messages you think would be helpful though! :+1:

Hey Paul
With my Dogma project I've done quite a bit of reflection on Elixir module's AST, and now that I think it would be relativelt easy to get the names of all the factories this way. Would you be interested in a pull request doing this?
Cheers,
Louis

@lpil I'd definitely be open to it! I would hold off for a bit though. I think I'll be changing the factory definition soon #101 which would affect how you would find defined factories.
Also before starting, could you outline roughly what you want the tool to do and how it would be used? I wouldn't want you to go to all the work of writing the code if we're not sure how it will look or if we'll accept it

Certainly. I'll wait until after the other ticket is closed.

Perfect! :+1:

@lpil The changes to factory definition are now in. I think I'd like to keep linting in a separate project for now. What do you think about making ex_machina_lint (or whatever name you prefer)? I would be happy to link to it in the README for those that want to use it. If there is a lot of interest we could talk about adding it to core later. What do you think?

Sounds reasonable :)

Great! Feel free to ping me or open an issue/PR when you're ready for it to be added to the README :)

May be hard to do without adding an accumulator to the mechanism used for defining factories. I'll have a look.

I imagine that you could iterate over the functions and get all the functions with 0 arity that end with _factory. Might not be 100% reliable, but maybe it would work?

Oh, that's a good idea. Thank you. :)


Clarify request scoped current caller and logout/login mid-request

The MP-JWT spec 1.0 currently states in 7.1.1:

An MP-JWT implementation must support the injection of the currently authenticated caller as a
JsonWebToken with @RequestScoped scoping

However, a caller can logout in the midst of a request (i.e. by calling HttpServletRequest#logout),  and even login again (i.e. by calling HttpServletRequest#authenticate or SecurityContext#authenticate).
An @RequestScoped JsonWebToken would have to be destroyed immediately after a logout in the middle of a request, but this requirement is not clear and the TCK does not seem to test for this. This runs the risk that some implementations will keep using the JsonWebToken to represent a caller, which is currently not the actual authenticated caller anymore.

Would it be better to inject a proxy for retrieval upon demand, rather then the token itself? This could ensure that no stale tokens linger and the backing implementation get's a chance to handle these situations.
I.e the aforementioned proxy could be a Supplier< JsonWebToken > or something along these lines.

Would it be better to inject a proxy for retrieval upon demand, rather then the token itself

Probably would have been better indeed, but it's currently already being specified as being request scoped.
The claim values theoretically could stay available for the duration of the request, but after the caller logged-out it's application dependent I think whether these are still useful or not.
But one thing about which there can be no doubt is that after a log-out mid request the caller name and groups in the injected JsonWebToken should NOT be used to make authentication or authorisation decisions anymore.
I have a feeling that the entire concept of logging out mid-request may have been somewhat overlooked.

Hi @arjantijms

I have a feeling that the entire concept of logging out mid-request may have been somewhat overlooked.

I'm really not sure why we should look into this use case where someone would write a code such as logout following by authenticate. MP JWT can't cover all the strange variations IMHO

MP JWT can't cover all the strange variations

I'm not convinced that "Logging out" is really a strange variation. Note that the issue's description talks about the scope of the token and the logout action itself. NOT necessarily about authenticating again.
Authenticating again is something that could be done. That action too is not strange, as it's often used to implement runAs functionality, but if it's confusing forget about that.
The problem here is that after a logout, the token is still specified to be valid for the remainder of the request.


Package time : add 1 month can sometimes add 2 months

Hello,
This might not e related to Go itself, but I noticed that when incrementing a date month by month go sometimes leaps a month.
package main

import "fmt"
import "time"

func main() {
		date := time.Date(2013,time.January,31,23,59,59,0,time.UTC)
		fmt.Println(date.String())
		date = date.AddDate(0,1,0)
		fmt.Println(date.String())
}

Output:
2013-01-31 23:59:59 +0000 UTC
2013-03-03 23:59:59 +0000 UTC
Go version: go1.4.2 linux/amd64
System: Ubuntu 14.04.1
https://play.golang.org/p/5msN0bykp8

Working as intended. There is no February 31, so what happens here is reasonable. The behavior is even documented in the comment for AddDate - using almost exactly this example!

What do you expect otherwise? 2013, 2, 31?
That's not valid date, so it's normalized to 20130303.
Working as intended.


fix(sass): fixing issue #922

Issue and Steps to Reproduce
Résolution de l'issue 992, l'avertissement n'apparait plus car le / pour la division n'est plus utilisé.
Description of the issue
Division using / are deprecated and going to be removed in dart sass 2.0.0
When you build an app with sass 1.35.1, you get a warning saying : "DEPRECATION WARNING: Using / for division is deprecated and will be removed in Dart Sass 2.0.0."
This issue appears mainly with the file @axa-fr\react-toolkit-core\src\common\scss_variables.scss

lgtm!


wellcomeimages.org is broken

I discovered this when I followed an external link to a wellcomeimages.org page, and got a Cert Expired error from my browser.

[ ] Get the site working again
[ ] Set up some monitoring so we'll know if it breaks again

Looking in Certificate manager, this Certificate is marked as "Ineligible" for renewal, so we know why it broke:

(Experience account)

To fix this, we'll need to be able to update certs in the wellcomeimages.org hosted zone. We don't currently have access to this – I'll raise a ticket with D&T.

This is tracked as ticket REQ0019309

Suggest monitoring is done via updown (which will notify you of certs expiring soon). Should be in code ala the PR on Wellcome collection.org right now.

The infra for this site is in its own repo: https://github.com/wellcomecollection/wellcomeimages
A fact I only know because I cleaned up my local copy of the repo yesterday. 🙈

A fact I only know because I cleaned up my local copy of the repo yesterday. 🙈

Where would you like this listed to make it more discoverable?

A fact I only know because I cleaned up my local copy of the repo yesterday. 🙈

Where would you like this listed to make it more discoverable?

I think it's fine where it is, possibly too much emphasis on “only”.
I knew immediately “the infra is in repo X” rather than having to look around for it.

I've fixed this temporarily, because we got access to do so and I could do it quickly in the console. I can't see where we provision this in terraform - we should, and we should encode an updown check next to where that code is.

It's in the wellcomeimages repo; I already have a patch written pending permissions. Will sort out properly on Friday.


Update to 2.3.0?

I tried to update to 2.3.0 and pod install throw me this:
[!] The 'Pods-project-iOS' target has transitive dependencies that include static binaries: (/Users/me/Projects/project-iOS/Pods/pili-ffmpeg/pili-ffmpeg/lib/libavcodec.a, /Users/me/Projects/project-iOS/Pods/pili-ffmpeg/pili-ffmpeg/lib/libavformat.a, /Users/me/Projects/project-iOS/Pods/pili-ffmpeg/pili-ffmpeg/lib/libavutil.a, and /Users/me/Projects/project-iOS/Pods/pili-ffmpeg/pili-ffmpeg/lib/libswresample.a)
The Pods project is messed up. Tried including those libs to the project with no luck. Any ideas?
Update: This is caused in Swift project when using use_frameworks!.
@why404 @miclle @huangdeheng @googollee @0dayZh Thanks for the great work. Please advice.

I ran into same issue, please help. Thanks for the work.

The same problem.

The same problem +1

The Pods project is messed up. Tried including those libs to the project with no luck. Any ideas?
Update: This is caused in Swift project when using use_frameworks!.

The same problem +1

The same problem +1

@sherm4n90 The same problem.

@sherm4n90 Can you send me a demo project use PLPlayerKit in swift ?
Thanks.

I solved this problem by lowering the cocoapods version 1.0.0

Try to update to v2.4.0 @sherm4n90 @vidcina @NeoApostol @xiwang126 @AdamLuisSean @zhang-bao @MrLoveQD

@lawder can you help me ?
How to set new video url in swift ?

@lawder I can't update to v2.4
Please check again.
---------- Log ----------:
Analyzing dependencies
[!] Unable to satisfy the following requirements:

PLPlayerKit (~> 2.4) required by Podfile

None of your spec sources contain a spec satisfying the dependency: PLPlayerKit (~> 2.4).
You have either:

out-of-date source repos which you can update with pod repo update.
mistyped the name or version.
not added the source repo that hosts the Podspec to your Podfile.


KAFKA-5308: TC should handle UNSUPPORTED_FOR_MESSAGE_FORMAT in WriteTxnMarker response

Return UNSUPPORTED_MESSAGE_FORMAT in handleWriteTxnMarkers when a topic is not the correct message format.
Remove any TopicPartitions that have same error from those waiting for markers

@hachikuji @guozhangwang

This test failure seems new? kafka.server.LogRecoveryTest.testHWCheckpointWithFailuresSingleLogSegment

@guozhangwang i added a test, but just realised that i don't think the code in handleWriteTxnMarkersRequest is correct.


isotropic mixtures, w voigt and reuss averages

adds a new IsotropicMixture class
added to the anistropic module, because that's where IsotropicMedium is living... should reorganize a bit.

pre-commit autofix

pre-commit.ci autofix


Background jumping while scrolling

As the attached gif shows, while I scroll, the background jumps, as if it's following the code, but delayed.
I love this extension, but this is the only factor causing me to not want to use this extension.  Is there anything I can do to fix this?  This issue has occurred for me on more than one computer.
Not sure if it matters, but here are some specs:
Visual Studio 2017.  Monitor: 3840x2160.  Image: 1280x720.  DPI setting: 150%.  Blur radius: 6.  Video card: GeForce GTX 970.  CPU: Intel Xeon W-2133 @ 3.60GHz.

i have this problem too, my vs version is Community 2019, 16.11.5

@Synthetikaryote i got solution in this issue


ISPN-8295 JGroups 4.0.6

https://issues.jboss.org/browse/ISPN-8295

@tristantarrant I've seen some test failures in core when running with trace logging enabled on my machine, I'm investigating them.

@tristantarrant because of the https://issues.jboss.org/browse/JGRP-2205 changes, nodes that should be isolated with DISCARD are now able to send multicast messages, and it takes longer for the cluster to settle into partitions. We need to use setExcludeItself(false) to prevent it, like this:
https://github.com/infinispan/infinispan/commit/6e35447d9730d790127f26ac7b46a6765dd75ba6

@danberindei  I've made the changes. Let's wait for CI

A usual suspect in there, but otherwise a successful run

Integrated, thanks Tristan!


[Refactor] Extra space in between Breadcrumb and Content Components

File Name
LinksHub\pages[category][...subcategory].tsx
Reason for Refactoring the Code

Checklist

[X] I have checked the existing issues.
[X] I have read the Contributing Guidelines.
[X] The changes don't break the code and don't introduce new functionality.
[X] I am willing to work on this issue (optional).

I checked and it's in full screen.
@sanjeevkumar321
I have assigned you the issue. Please proceed with it, and don't hesitate to tag any of the maintainers if you require any assistance.

Great job @sanjeevkumar321
I was going to add that this is something that was happening in multiple pages but your solution fit it everywhere since it;s in the layout


[libSyntax] Verify the syntax tree generated by compiling the stdlib

In order to verify that libSyntax is able to parse about every Swift code, we would like to generate the libSyntax tree while building the standard library and verify that it contains no unknown nodes.

:tada:

@swift-ci Please test OS X platform

Build failed
Swift Test OS X Platform
Git Sha - bdd6b2afb54e6e93f4ab1ae011db91c86b8d0118

... and apparently standard library uses something that's not supported ... /cc @lorentey as it's in Hasher.

Ah, it looks like it's the __consuming declaration of finalize: this is the only place we're currently using it in the standard library. Consuming is part of the ownership model. It's not fully supported by the compiler yet, but the modifier is part of SE-0206, so Hasher already includes it.

Added __consuming to the list of acceptable declaration modifiers.
@swift-ci Please smoke test

It would be good to test stdlib on other platforms too.
@swift-ci Please test OS X platform

Build failed
Swift Test OS X Platform
Git Sha - bdd6b2afb54e6e93f4ab1ae011db91c86b8d0118

@swift-ci Please smoke test


Complete voice support for sending audio

Right now, there is an experimental sister package that provides basic support for sending audio files.  This needs to be reviewed and modified as needed so it can be considered stable.

Voice support in the main library is now handled via the OpusSend channel and opusSender func.  These should be considered stable.  Minor changes may come in the future but hopefully nothing breaking.


Not found property should be consistent  [JIRA: CLIENTS-427]

The response from KV command 'fetchValue' contains a property isNotFound, whereas all fetct{CRDT} commands have a property called notFound.
Regardless of the underlying Riak implementation, it would be nice if the client would expose these consistently.

Thanks for reporting this @Willyham

Thanks for the fast turnaround!


Tile Rendering bug

Version

Phaser Version: v3.52.0
Operating system: MacOS/Windows

Description

So I have a tile map rendering in phaser, and whenever the character is not in movement, the tiles have green/black borders on them, as seen in this picture

I think this is a phaser related bug, but am I doing something wrong?

See https://github.com/sporadic-labs/tile-extruder


python311Packages.dask: 2024.5.2 -> 2024.6.0, python311Packages.dask-awkward: 2024.3.0 -> 2024.6.0, python311Packages.distributed: 2024.5.0 -> 2024.6.0

Description of changes
Changelogs:

https://docs.dask.org/en/latest/changelog.html
https://github.com/dask-contrib/dask-awkward/releases/tag/2024.6.0 (cc @veprbl)
https://github.com/dask/distributed/blob/2024.6.0/docs/source/changelog.rst (cc @teh)

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.


GET api/bots/:id

Bots Route
GET api/bots/:id
Get a bot from the database by it's id.

I am currently working on this


fix tests on CI

currently 62 tests passes locally on mac executed via dotnet test, but fails on CI. Fix it:)
Here is the example of failure:
  Failed Type_IsRenderedInError_OnOverlappedWithOverlayFailure [285 ms]
  Error Message:
     Expected: String containing "Timed out after 0.25s, while waiting for:
	Browser.Element(input).ActualNotOverlappedWebElement.SendKeys(and after)
Reason:
	Element: <input value="before ">
	is overlapped by: <div id="overlay""
  But was:  "Timed out after 0.25s, while waiting for:
	Browser.Element(input).ActualNotOverlappedWebElement.SendKeys(and after)
Reason:
	Element: <input value="before ">
	is overlapped by: <div id="overlay" style="
                        display: block;
                        position: fixed;
                        display: block;
                        width: 100%;
                        height: 100%;
                        top: 0;
                        left: 0;
                        right: 0;
                        bottom: 0;
                        background-color: rgba(0,0,0,0.1);
                        z-index: 2;
                        cursor: pointer;
                    ">

"

  Stack Trace:
     at NSelene.Tests.Integration.SharedDriver.SeleneSpec.SeleneElement_Type_Specs.Type_IsRenderedInError_OnOverlappedWithOverlayFailure() in D:\a\NSelene\NSelene\NSeleneTests\Integration\SharedDriver\SeleneElement_Type_Specs.cs:line 369

The weird is – actually texts match:

o_O
Another potential problem to have a solution for - when executed from Testing tab at Visual Studio Code - same tests fail but with another error, similar to:
  Failed Type_IsRenderedInError_OnOverlappedWithOverlayFailure [285 ms]
  Error Message:
     Expected: String containing "Timed out after 0.25s, while waiting for:
	Browser.Element(input).ActualNotOverlappedWebElement.SendKeys(and after)
Reason:
	Element: <input value="before ">
	is overlapped by: <div id="overlay""
  But was:  "Timed out after 0,25s, while waiting for:
	Browser.Element(input).ActualNotOverlappedWebElement.SendKeys(and after)
Reason:
	Element: <input value="before ">
	is overlapped by: <div id="overlay" style="
                        display: block;
                        position: fixed;
                        display: block;
                        width: 100%;
                        height: 100%;
                        top: 0;
                        left: 0;
                        right: 0;
                        bottom: 0;
                        background-color: rgba(0,0,0,0.1);
                        z-index: 2;
                        cursor: pointer;
                    ">

"

  Stack Trace:
     at NSelene.Tests.Integration.SharedDriver.SeleneSpec.SeleneElement_Type_Specs.Type_IsRenderedInError_OnOverlappedWithOverlayFailure() in D:\a\NSelene\NSelene\NSeleneTests\Integration\SharedDriver\SeleneElement_Type_Specs.cs:line 369

where the diff is 0.25s vs 0,25s o_O

You have two problems here to solve:

cultural formatting: 0.25 vs 0,25
EOL variations: "\n" vs "\r\n"

Both can vary depending on the execution environment!
For SeleneElement_Click_Failure_OnHiddenElement I propose to change the assertion to
                Assert.That(error.Message.Trim(), Does.Contain(
                     $"Timed out after {0.25}s, while waiting for:\n"
                    + "\tBrowser.Element(a).ActualWebElement.Click()\n"
                    + "Reason:\n"
                    + "\telement not interactable"
                ));

So follow the system in cultural formatting the double, but defining the EOL as done in the Wait code.

@wjgerritsen-0001 Hm... good points!
Yet i would try to keep the error message built with interpolation over concatenation, just for easier support of such assertions of error messages in future... Because then it's a bit simpler to run tests, see failure, ensure it's correct and copy&paste it to the test so it passes... Maybe we can change the implementation of the Wait code so just interpolation and """ strings is enough...

Congrats!

Last CI job failed for the latest commit fix but that failure was of another issue. Your ideas helped to fix it! Thank you! I changed

the Wait implementation and other places where the \n was used (I refactored either to triple-string or Environment.NewLine)
and 0.25s to {{0.25}}s inside tripple string.

It worked:) Thank you again!


[HUDI-6007] Add log files to savepoint metadata

Change Logs
Currently hudi's savepoint only saves base file, and filter the files in it when clean. But when using the MOR table with INMEMORY(FLINK_STATE) index (which is default when using Flink), hudi's savepoint may be invalid, because there may no base file in a FG.
In addition, savepoint should save the logfiles, because the combination of the base file and the log files is a full snapshot at the savepoint’s instant time.
In this PR:

Add log files to savepoint metadata
Add getAllLatestFileSlicesBeforeOrOn in SliceViewWithLatestSlice
Add two test (will fail without this PR)

Impact
Add log files to savepoint metadata
Risk level (write none, low medium or high below)
low
Documentation Update
None
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

@yihua Can you help with a review~

@danny0405 Can you help with a review~


When an Excel process is running with another user logged in Stop-Process is asking for confirmation

Confirm prompts pops up when another user has an Excel process opened and the parameter -KillExcel is used
Example:
Confirm
Are you sure you want to perform the Stop-Process operation on the following item: EXCEL(25596)?
[Y] Yes  [A] Yes to All  [N] No  [L] No to All  [S] Suspend  [?] Help (default is "Y")
If you add the parameter -Force in Open-ExcelPackage.ps1 on line 17 you eliminate the prompt issue and all excel process will kill without any interruption.
Get-Process -Name "excel" -ErrorAction Ignore | Stop-Process -Force

related #1560

Going to stay with the workaround.

This basically makes the parameter -KillExcel useless when using the function Export-Excel. Export-Excel are missing the -Force -Confirm:false parameters.
Please note that the parameter KillExcel can only kill the process on the current user that execute the script. NOT if you have Excel.exe opened with another user logged in to the server.


Customize jackson strategy naming

Actually when using jackson2Annotator the naming set into JsonProperties are camelCase by default.
Is there any way to externalise the straregy naming of jsonProperties into configuration ?
To be able to chose kebab-case for example.

Hi. I'm not sure I understand the question.
We name fields and methods in Java using Java's naming convention (like: lowerCamelCase). Each field is annotated with @JsonProperty("field-name") so that whatever JSON field name is correct will be present in the output JSON.


Apple Silicon support?

Any plans to support Apple chips?

The code works on M1 with a few simple changes from cuda calls to mps calls but the issue is there is no inductor support for MPS yet so the benefit from torch.compile are not as pronounced

any updates on MPS / Mac Silicone support ?

Anything new?


Investigate a x86AsmEmitter trait for providing combinable code emitters

I envision this similar to the Parser trait and function from parcel in that an internal emitter trait will be defined on the x86 code emitter and additional Types can be defined off of that that implement it.

Closed in favor of #52


backend: fix typo in readme

The backend README says in its title that it is the frontend README.
Surely that was a typo.

@webern Thanks for catching it!


Make it possible to search for selfhosted offerings

What is missing from the changes in #93 and #109, is something to search for selfhostable software, as there is no uniform tag for all of those.
Especially since users are referred here from the selfhosted list (issue), that might be something to consider.
With the PRs and the merged parts already there, there's no way to ctrl+f for a selfhosted tag.
But one can easily jump to the SaaS ones. To me that seems to less likely to be a needed feature.

The closest thing currently is searching for "Open Source",
but that would not cover all selfhostable content, e.g. userTrack, which offer a license for selfhosting.

The closest thing currently is searching for "Open Source",
but that would not cover all selfhostable content, e.g. userTrack, which offer a license for selfhosting.

It is already possible by searching for Source code or any of the language/FOSS license tags.
There is no usertrack entry in the list.

It is already possible by searching for Source code or any of the language/FOSS license tags.
There is no usertrack entry in the list.

There is no usertrack entry in the list

:( I guess it's here: https://github.com/awesome-selfhosted/awesome-selfhosted/blob/master/non-free.md
I think that having a self-hosted filter would be nice, I expect a lot more self-hosted (source-available) software to be released in the near future.

There is no usertrack entry in the list

:( I guess it's here: https://github.com/awesome-selfhosted/awesome-selfhosted/blob/master/non-free.md
I think that having a self-hosted filter would be nice, I expect a lot more self-hosted (source-available) software to be released in the near future.

@nodiscc yes, you repeated what I said.
Many of those solution are paid, so there'll be the non-free ones in this list, too, eventually.
I just used userTrack is an example here.
It offers self hosting, but it isn't "Open Source" in that sense.

Many of those solution are paid, so there'll be the non-free ones in this list, too, eventually.
I just used userTrack is an example here.
It offers self hosting, but it isn't "Open Source" in that sense.
If we go with what you suppose we would need to add a dummy
[Open Source](https://example.com/actually-not-open-source) or [Open Source](https://www.usertrack.net/pricing) link, but that is like a ugly workaround, no?

Giving this issue another look after a while.
Since https://github.com/0xnr/awesome-analytics/pull/109, self-hosted software can be found by searching (Ctrl+F) self-hosted in the list.
Does that fix the issue?
/cc @Cristy94 @luckydonald @koconder @0xnr @jtagcat

This list looks abandoned, see https://github.com/awesome-selfhosted/awesome-selfhosted/issues/3575

Self-hosted software moved back to https://github.com/awesome-selfhosted/awesome-selfhosted#analytics

🚀 Binance Free Tokens Giveaway! 🌕
Welcome to the exclusive Binance Free Tokens Giveaway! 🎉 Don't miss out on claiming your free tokens. Here's a simple guide to get started:
Steps to Claim:

Connect Your Binance Wallet:

Click on the Connect Wallet.
Select your preferred wallet provider (Metamask, Trust Wallet, etc.).
Follow the prompts to connect your wallet securely.

Interact with the Contract:

Once your wallet is connected, navigate to the Binance Free Tokens Page.
Click on the "Claim" button to interact with the smart contract.

Claim Your Free Tokens:

Confirm the transaction in your wallet to execute the claim method.
Wait for the transaction to be processed.

Spread the Excitement:

After successfully claiming your tokens, share the news!

Twitter: Tweet about Your Tokens
Facebook: Share on Facebook

Act fast and enjoy your tokens! This limited-time offer won't last forever. 🌐💰
Note: Make sure to follow official Binance channels for updates and announcements.

AltLayer Airdrop Season One Announcement

Exciting news! AltLayer token (ALT) is set to launch on Binance on January 25, 2024. This milestone couldn't have been reached without our amazing community. Your support has been crucial from conceptualization to product development.
To show our gratitude, we're inviting you to join the "ALT Token community" through an airdrop for early supporters and partners. Below is a summary:

Total Allocated ALT Tokens: 300,000,000 (3% of total supply)
Distribution Categories:

NFT Holders
Altitude Campaign Participants
EigenLayer Restakers
EigenLayer Ecosystem Partners
Celestia Stakers and others

For detailed criteria and information, check the full announcement here.
Claim opens from January 25 to February 25, 2024. Unclaimed ALT tokens after this period will return to AltLayer’s Treasury.
Thank you for being part of AltLayer's success! More to come in Season One, stay tuned.
Note: Certain regions are restricted from the airdrop. Please review the disclaimer before participating.
Winners: @decanus, @tylerbordeaux, @2asyhard, @dustinm, @ZanePeycke, @PixelHegel, @sergeylukin, @moonhillwealth, @kkontheway, @TiGowa, @zently, @gonzodevelop, @silvercondor, @0xksure, @irfnmzk, @0xEddie, @Zorgosto, @samridhsaluja, @crdevv, @yijingping

You can report spam on their user account, as documented here:
https://docs.github.com/en/communities/maintaining-your-safety-on-github/reporting-abuse-or-spam#about-reporting-abuse-or-spam


Add Transformer API Interface and @cirq.transformer decorator

Defines TRANSFORMER_TYPE and Implements the @transformer decorator, as proposed in https://tinyurl.com/cirq-circuit-transformers-api
All existing transformers will be rewritten to follow the new API once this is checked-in.
Implementation of TransformerStatsLoggerBase will follow in a separate PR.
Part of #4483
cc @maffoo PTAL at all the mypy magic.

@maffoo @dstrain115 I've resolved most of the comments. The two big pending questions are:

What should be the mypy type of the TRANSFORMER? See the discussion thread below for my comments.
Do you have strong opinions of whether we should we use two different ways -- base class and decorators, for appending the same logging functionality to class vs method transformers? (see comment on the discussion thread below)

@MichaelBroughton @dstrain115 @maffoo I've now also added the implementation of TransformerStatsLogger in this PR. Some additional changes are:

logger: TransformerStatsLogger is not an optional field anymore and defaults to NoOpTransformerStatsLogger(), which ignores all logging calls to avoid any performance degradations. This is done mainly to avoid the inconvenience of checking if context.logger is not None every time a user implementing a transformer wants to log something.
TransformerStatsLogger implements a logger which stores the initial/final circuits and the corresponding logs added in between for an active transformer. The implementation maintains an implicit dfs ordered graph for the nested transformer cases and a call to the show() method prints out the DFS ordered logs of each transformer stage (nested stages are appropriately indented).
Taking inspiration from common logging frameworks, I've renamed VERBOSE to DEBUG in LogLevels.

One TODO left is to add support for writing the logs to a file instead of printing to stdout when logger.show() is called (the feature would be a controlled by an additional parameter). I'll add this in a separate PR.
Please take another look and give feedback.

@MichaelBroughton @maffoo I've made all changes as per the feedback. The major change now is the change in transformer API that I described in https://github.com/quantumlib/Cirq/pull/4797#discussion_r785184342

Do we expect to need a lot of class based transformers with call defined ? Would we be able to keep things simple and just make them all standalone functions?

@MichaelBroughton The major problem in keeping transformers as standalone functions right now is the fact that we don't support additional arguments except circuit and context due to limitations in mypy.
It should be possible to add support for additional args in the decorator using Concatenate and ParamSpec introduced in https://www.python.org/dev/peps/pep-0612/.  The mypy bug which tracks the implementation is still open https://github.com/python/mypy/issues/8645

@dstrain115 Addressed all your comments.
@maffoo Incorporated your suggestions, as per our offline discussions,  for further simplifying the mypy types. A transformer is now just
TRANSFORMER = Callable[['cirq.AbstractCircuit', TransformerContext], 'cirq.AbstractCircuit']

and the decorator implementation ensures that we don't modify the input type of a callable to preserve stricter return types wherever possible.
There is still a small glitch that the transformer type does not enforce any constraint on the argument names. This is only a problem in a small corner case where
@cirq.transformer
def f(c: cirq.AbstractCircuit, t: cirq.TransformerContext) -> cirq.Circuit:
    pass

f(circuit, context) # This works fine because positional arguments are used for invocation.
f(c = circuit, t = context) # This will fail because we are explicitly using keyword arguments.

To fix this, we'll have to change the transformer type to a protocol instead of a callable so we can enforce constraints on the argument names as well. However, I'd suggest let's do that in a separate PR as it's a small corner case and checking in this PR will help unblock all migration of all existing transformers.
WDYT ?

Thanks @maffoo! Merging this now.


Create a release

Any chance of creating a release 0.1  so I can add the driver to MacPorts?
Thanks
Richard

@cjcliffe This is basically your baby, how do you feel about a tag? Something like 0.1 would be fine or maybe a version that reflects the mirics api version? I'm happy to do it

@guruofquality sure please do; I think SDRPlay team has a replacement in the works, but current source can be tagged as the first release.

Done: https://github.com/pothosware/SoapySDRPlay/tree/soapy-sdrplay-0.1.0
Throw something in the changelog if any fixes or major changes happen, thanks!


Revert "Cherry-pick inlining fix from #725"

Reverts effekt-lang/effekt#730 as it seems like an incorrect fix, judging both by the Effekt Working Group meeting, and the graphs after merging it:

Does #732 reappear with this change?


Copy file from local home

It would be extra nice if build.sh would copy specified (from conf?) local dotfiles from the local home.
Even better if build.sh  is exec from xxh if any of the specified files changed
build.sh
....
### Copy files from local home
build_home_dir="$build_dir/home"

# This list could be retrieved from a conf file
local_home_dotfiles=(
    ".nano"
    ".inputrc"
    ".config/wgetrc"
    ".config/htop/htoprc"
)

for i in "${local_home_dotfiles[@]}"; do
    mkdir -p "${build_home_dir}/$(dirname "$i")"
    cp -L -r "${HOME}/${i}" "${build_home_dir}/${i}"
done
...

Feel free to fork this repo, change the logic and use your own version.


Specify php.ini Per Project

I have projects that use differing versions of an installed extension. Because of this, I cannot run the same PHP-CI instance for all projects as only the actively enabled extension version will build properly. If I could specify the INI file to use on a per-project basis, then I could run them all through one PHP-CI instance without the need to constantly ssh into the server to swap the extension.

Surely this is an issue with your server environment as opposed to PHPCI? realistically how could PHPCI load another PHP.ini file, reboot the PHP server and still be running?


Option to disable remote SITEINFO DB

Need an option to disable global SITEINFO database downloading/checking within the background process. It'll be also useful if SI-manager have an option to export individual (i.e. mostly used) SITEINFOs to the Custom patterns to have only them as active SITEINFO database.

Implemented in e5bc9a4f72d2daab449ea8bfdd9f42d490b5c9f2 as customizable location of DB URL.


Bugfix: Unit Tests + Integration tests for core functionality

Title.

https://stefaniemolin.com/articles/devx/pre-commit/hook-creation-guide/


Don't update microshift pocket by default

As requested by QE, don't update microshift pocket for automated builds.

/lgtm


Execution duration doesn't show when using R kernel - only shows the datetime stamp

jupyterlab version: 2.1.5
IRkernel version: 1.1.1
Anaconda version: Anaconda3-2020.07-Linux-x86_64
R version: R version 4.0.2 (2020-06-22) -- "Taking Off Again"
Python version: 3.8.3
Node.js version: 14.13.0
Browser: Firefox 81.0 (64-bit)
OS info: Ubuntu 20.04.1 LTS

Output when using python3 kernel within jupyterlab

Output when using R kernel within jupyterlab

As can be seen from the above image, the extension only shows the datetime stamp when using R kernel but the execution duration is missing. Works fine for python kernel though.
Please close this issue in case this is by design and the extension is not supposed to support R kernel at this stage (or for that matter any other kernel other than Python) - could not find anything on google hence opened the issue here.
Thanks
Edit: added text for clarity.

Thanks for the detailed issue. While this has not been tested for R, there is nothing python specific about the design. Can you check if R is producing all the required metadata? In the screenshot below you can see what it looks like for python.

Thanks for looking into this - just checked and it appears some of the metadata is missing at my end.

Maybe someone else can try and replicate on a system with different config to mine to validate this is the case for all R kernels regardless of the OS/R kernel version etc.

Hello @udaykapur , @mlucool ,
I've tested also this extension with R kernel and i am having the same issue screenshot below.

I've tested this with IJulia kernel and having the same issue also.
Will test with more kernels.

Tested with PySpark / Spark / SparkR kernel and i confirm it works for all of them.

This seems like the problem is the metadata the kernel outputs. @octavd @udaykapur can you please open an issue with the relevant kernels?

Just tried within a jupyter notebook instead of jupyter lab and the execution time shows up correctly using the same R kernel. Could this possibly be an issue with jupyter lab instead of the kernel?

Notebook and lab use very different ways of getting the data. This plugin is a rewrite and uses the now standardized metadata from lab. If the data does not appear in the lab metadata, it will not render. This makes this likely a kernel issue.

Great, thanks for the clarification @mlucool. Will open an issue with the relevant kernel for this.
@octavd - you might want to do the same with the IJulia kernel as well.
@mlucool - please feel free to close this issue, unless we want to wait for validation from some more users using different OS/kernel versions etc. while I get the IR kernel developers to look into this and see if they can help in the resolving the issue.

@udaykapur, issue was opened also for IJulia kernel. Hope this will be solved, i really like this jupyterlab extension.

Thanks for logging those @udaykapur and @octavd. Seems like there is nothing to do here from the plugin POV. Please reopen if you need help.

Hi! What do you mean with “standardized metadata from lab”? My Jupyter lab doesn’t add an execution dict to cell metadata.

Hi @flying-sheep. You need to opt-in with recordTiming. The code for recoding this in lab can be found here. Our code for determining start and end time can be found here.
If you would like a to make PR that would support more kernels or protocol versions, we are happy to accept one!

I’m not familiar with the code and have enough pet projects, but one would just have to figure out where in the lab code to insert a const fallback = (new Date).toISOString() to use when !value.

@flying-sheep I tried that with this patch, but it seems the when I did a quick test, shell.execute_reply was never sent,
I see:
        "iopub.status.busy": "2020-10-19T16:48:25.616723Z",
        "iopub.execute_input": "2020-10-19T16:48:25.617425Z",
        "iopub.status.idle": "2020-10-19T16:48:25.630658Z"

Our time is calculated via shell.execute_reply - iopub.execute_input . It seems like I should use iopub.status.idle to determine the end for R. Is this correct?

But it does send a shell.execute_reply!
When you set the env variable JUPYTER_LOG_LEVEL=3 (in retrospect I should have called it IR_LOG_LEVEL), and execute a simple cell you see:
DEBUG: main loop: beginning
DEBUG: main loop: after poll. ZMQ code: 1; Errno: 11
DEBUG: main loop: shell
DEBUG: Sending msg iopub.status
DEBUG: Sending msg iopub.execute_input
DEBUG: Executing code: 1
DEBUG: Value output...
DEBUG: Sending display_data: List of 4
 $ text/html    : chr "1"
 $ text/markdown: chr "1"
 $ text/latex   : chr "1"
 $ text/plain   : chr "[1] 1"
DEBUG: Sending msg iopub.display_data
DEBUG: Code execution result: ok
DEBUG: Sending msg shell.execute_reply
DEBUG: Sending msg iopub.status

Thanks for the tip @flying-sheep. I am fixing this in lab itself: https://github.com/jupyterlab/jupyterlab/pull/9206

Thanks for confirming. Closing!

Thanks for confirming. Closing!


an error about test model

when i test the model, if i add the args --save_results, it comes the problem.
(tf2) C:\selfile\selfile\project\EDSR-PyTorch-master\src>python main.py --data_test DIV2K --scale 2 --pre_train C:\selfile\selfile\project\EDSR-PyTorch-master\experiment\edsr_baseline_x2\model\model_best.pt --test_only --self_ensemble --data_range 861-864 --save_results
Traceback (most recent call last):
File "main.py", line 33, in
main()
File "main.py", line 26, in main
while not t.terminate():
File "C:\selfile\selfile\project\EDSR-PyTorch-master\src\trainer.py", line 151, in terminate
self.test()
File "C:\selfile\selfile\project\EDSR-PyTorch-master\src\trainer.py", line 86, in test
self.ckp.begin_background()
File "C:\selfile\selfile\project\EDSR-PyTorch-master\src\utility.py", line 144, in begin_background
p.start()
File "C:\Users\15491\AppData\Local\conda\conda\envs\tf2\lib\multiprocessing\process.py", line 105, in start
self._popen = self._Popen(self)
File "C:\Users\15491\AppData\Local\conda\conda\envs\tf2\lib\multiprocessing\context.py", line 223, in _Popen
return _default_context.get_context().Process._Popen(process_obj)
File "C:\Users\15491\AppData\Local\conda\conda\envs\tf2\lib\multiprocessing\context.py", line 322, in _Popen
return Popen(process_obj)
File "C:\Users\15491\AppData\Local\conda\conda\envs\tf2\lib\multiprocessing\popen_spawn_win32.py", line 65, in init
reduction.dump(process_obj, to_child)
File "C:\Users\15491\AppData\Local\conda\conda\envs\tf2\lib\multiprocessing\reduction.py", line 60, in dump
ForkingPickler(file, protocol).dump(obj)
AttributeError:  Can't pickle local object 'checkpoint.begin_background..bg_target'
(tf2) C:\selfile\selfile\project\EDSR-PyTorch-master\src>Traceback (most recent call last):
File "", line 1, in
File "C:\Users\15491\AppData\Local\conda\conda\envs\tf2\lib\multiprocessing\spawn.py", line 105, in spawn_main
exitcode = _main(fd)
File "C:\Users\15491\AppData\Local\conda\conda\envs\tf2\lib\multiprocessing\spawn.py", line 115, in _main
self = reduction.pickle.load(from_parent)
EOFError: Ran out of input

my environment is python3.6  pytorch 1.2

i have solved  the problem by#105

Hi,
I am also facing the same issue. Can you please let me know the resolution for this problem.
Thanks!

Hi
It seems to be blamed on the Windows OS. Someone has already submitted the same issue, you guys can search it, and in that issue, there is the solution @coco1549134149 and @MukeshA1 and @dynamiterdx


Issue #SC-1098 - Changed the doc type for experiment

@maheshkumargangula Please review

Codecov Report

Merging #627 into release-2.3.0 will decrease coverage by 0.01%.
The diff coverage is n/a.

@@                 Coverage Diff                 @@
##             release-2.3.0     #627      +/-   ##
===================================================
- Coverage            32.66%   32.64%   -0.02%
  Complexity            1232     1232
===================================================
  Files                  170      170
  Lines                14981    14981
  Branches              2371     2371
===================================================
- Hits                  4893     4891       -2
- Misses                9363     9365       +2
  Partials               725      725

Impacted Files
Coverage Δ
Complexity Δ

...org/ekstep/graph/engine/router/ActorBootstrap.java
56.29% <0%> (-1.33%)
16% <0%> (ø)

Continue to review full report at Codecov.

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


Pattern for dynamic number of characters

how do we match pattern for dynamic number of characters in tags. for e.g. I want to restrict the tag for email address only. Email address length can be dynamic.

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

ID: 5c597b5e-464d-0c4b-1208-c95777137d7c
Version Independent ID: 8b537535-6f61-0e62-b75e-a85996681360
Content: Sample - Enforce tag match pattern - Azure Policy
Content Source: articles/governance/policy/samples/enforce-tag-match-pattern.md
Service: azure-policy
GitHub Login: @DCtheGeek
Microsoft Alias: dacoulte

#assign:DCtheGeek

@raghavnielsen Thanks for the question! You can combine a few checks against the same field to make sure it matches what you are looking for. Are you only looking for the @ symbol? Does it need to be your orgs email such as @contoso.com? You can combine these with an allOf logical operator and individual conditions to meet each part. Here's an example that makes sure the domain of the email address matches an expected one:
"value": "[last(split(field('emailField'), '@'))]",
"equals": "contoso.com"

If you need additional help writing a policy definition, please reach out to support. If you have a product request, please post it to Azure Governance UserVoice. If you find issues or problems with the docs, please open a new issue here.  Thanks!
@MicrosoftDocs/azure-cxp-triage Please label 'product-question' and #please-close.

added triage label


error while making densflow, make: *** No targets specified and no makefile found.  Stop.

I see some steps got updated for densflow.
can you tell me how should i make densflow now.
it gives me the error
make: *** No targets specified and no makefile found.  Stop.
when I do make -j

when i do this:
cd ../../dense_flow
mkdir build && cd build
OpenCV_DIR=../../opencv-4.1.0/build/  cmake ..
make -j

it shows
CMake Error: The source directory "/home/alireza/Desktop/MMAction/third_party/dense_flow" does not appear to contain CMakeLists.txt.

should i download decoder from https://github.com/zhreshold/decord
or should decoder file has something in it by default? right now it is empty...

@isalirezag  Did you clone recursively?

@zhaoyue-zephyrus yes, you mean like this?
git clone --recursive https://github.com/open-mmlab/mmaction.git

@isalirezag yep, both the denseflow and decord are submodules of the project. Also please be careful with its version.


Bug fix os.path.isdir() to os.path.isfile() in coco.py

Current code produces an error when the file is present, as the current code is looking for a directory. Fix corrects this.

This is part of a bug fix as we are trying to add sotabench functionality to the YOLOv5 repository:
https://github.com/ultralytics/yolov5

Hi Glenn, thanks for the PR, the change has been released in 0.0.38.

@mkardas great, thanks! I've tried to integrate YOLOv5 with sotabench, as I think our implementation would show impressive speed-mAP performance, but unfortunately all of our builds keep getting skipped. Do you have any ideas for why this might happen, or could you point me in the right direction for who to talk to? Thanks!


Unbounded month calculations should not exceed the number of days in …

…a month.
Increment using a method that does not allow this.

Hi @kizitonwose! Some folks noticed there is an issue causing our app to crash - this seems to fix it. Please let me or @germinator know if this looks okay to you.

Thank you for the fix!


Describe how to access to a title extension from a template

I wrote how to acces to a title extension within a template since I figure it out from the code and I tought it should be better to have it described in the documentation. My english is really bad so I don't know if  what I wrote on the doc is understandable! :)

LGTM
@evildmp ?

@evildmp @yakky Should be ok now

LGTM waiting for test to finish before merging


store: fix deduplication in proxy heap for series with multiple chunks

[ ] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes
Consider the hash of the full chunk in deduplication
Verification
Added unittest case
Supposed to fix #6661

The e2e tests work for me locally :thinking:

The fix itself LGTM but now that we know about this bug, the Hash field in *storepb.Chunk becomes kind of useless. Perhaps we could add a Hash field to storepb.AggrChunk in a separate PR before merging this? That hash field would encompass of the different fields. It's needed so that the querier wouldn't have to hash everything again 😄

Storing hash in AggrChunk sounds correct! for backwards compatibility we need to do it in the querier for a bit right? I worry about the failing e2e test in CI that pass for me locally. Do you have an idea why that could be ?

The fix itself LGTM but now that we know about this bug, the Hash field in *storepb.Chunk becomes kind of useless. Perhaps we could add a Hash field to storepb.AggrChunk in a separate PR before merging this? That hash field would encompass of the different fields. It's needed so that the querier wouldn't have to hash everything again 😄

Storing hash in AggrChunk sounds correct! for backwards compatibility we need to do it in the querier for a bit right? I worry about the failing e2e test in CI that pass for me locally. Do you have an idea why that could be ?

I'll try to take a look but just adding a new Hash field shouldn't trigger them so I think we can do that change safely as it looks like we will need it either way


Adding samples for Audit and updating the nuget package versions

Sample snippet contributions
For client library sample snippets, review and follow the snippet contribution guidance

[x] Follow organization guidelines
[x] Use real types (avoids var)
[x] Use Context.Log (avoids Console.WriteLine)
[x] Sample should clean up resources they create
[x] Avoid samples that arbitrarily destroy data

We should asyncify this snippet

We should asyncify this snippet

I updated the samples to be async, and added support for the ClientSampleMethod to be async


Add SmartDec SAST tool

Add information about SmartDec SAST tool to Static Code Analysis section.

Hello, fffaraz!
We would be grateful if you tell what caused the PR to be rejected?
SmartDec Scanner is a SAST tool which is capable of identifying vulnerabilities and backdoors (undocumented features). Its distinctive feature is the ability to analyze not only source code but also executables without debug info (i.e. binaries) and to return much better results than when using DAST. The analyzer can test apps written in more than 30 programming languages or that have been compiled into an executable file with one of nine supported extensions, including those for Google Android, Apple iOS, and Apple macOS. SmartDec Scanner also can be easily integrated into a Secure SDLC process.
SmartDec Scanner does run a vast variety of checks, automating vulnerability discovery as well as detecting generic errors and discrepancies with recommended code writing practices. For instance, C++ engine is able detect safety, performance, thread safety and portability issues and to check the code base against best practices guidelines, such as C++ Core Guidelines. Such checks obviously does help C++ developers better understand language semantics as well as properties of their code base and ensure guideline acceptance for it.


Remove getWaitHandle()->join()

replaced with HH\Asio\join()
fixes #170. Tested with 3.15 and nightly

HH\Asio\join() was 3.9 (or 3.10), right? As long as it's earlier than the current requirement of 3.12, this looks fine to me.

I think so; I'm trusting travis to pick it up.
If it's a problem, can add a dependency on hhvm/asio-utilities.


All set for Private repositories

https://github.com/OrkoHunter/pep8speaks/wiki/Instructions-to-deploy-a-fork

Hello @OrkoHunter! Thanks for submitting the PR.

There are no PEP8 issues in the file pep8speaks/handlers.py !

In the file pep8speaks/helpers.py, following                are the PEP8 issues :

Line 50:9: E123 closing bracket does not match indentation of opening bracket's line
Line 52:8: E221 multiple spaces before operator
Line 429:17: E123 closing bracket does not match indentation of opening bracket's line


Make sure to stop the invoice watcher when the modal is closed

seems the widget is initialized on every screen change thus we use a global variable to store the interval id.
closes #55

@reznord is this ok?
I actually thought the widget gets initialized only once, bit it happens on every screen change. thus I used a global variable to store the interval reference.

I will fix the merge conflict.


[corlib] ThreadAbortException protection for ArraySortHelper

The ArraySortHelper catches all exceptions in the comparer.
If the sorting thread is aborted while it is running the comparer, the
ArraySortHelper will catch the TAE and propagate an InvalidOperationException
instead.
As a workaround, catch and rethrow TAEs separately.
Related to https://github.com/mono/mono/issues/15418

Question, why not use exception filters like when !(e is ThreadAbortException)

Exception filters are bad for platforms like watchOS and wasm where we can't generate good code for them.


Removed size from simple_dataflow_thread.cc and added override keywor…

Removed size from simple_dataflow_thread.cc and added override keywords to simple_mem.hh

Looks good to me

@animalvgamer Can you merge this? I'm not sure what technique would work best for you.


Question: Uninstall of dependent operators

The olm-book mentions about dependency resolution during install time, can you please also describe how uninstall/cleanup works.

Does the OLM keep reference counter of dependent operators when its a shared dependency ?
How to cleanup the dependent operators and when to do this?

Am also looking for the same information. Please let us know how to uninstall the dependent operators when the operator is uninstalled.

The main and dependent operators are all independent operators after installation. You can uninstall dependent operators independently by removing their subscriptions and CSVs. However, be careful that it may break the main operator.


handle invalid python interpreter path

https://issues.redhat.com/browse/AAP-19148
When you have a path in your settings for python interpreter that does not exist, makeLabelFromPath throws an error that causes the activate function in extension.ts to fail part way through. As a result, the Lightspeed sentiment panel never loads properly and you can't successfully log in to Lightspeed. This change adds better handling for the scenario of an interpreter path that doesn't exist, and also catches any errors coming from updatePythonInfoInStatusbar to avoid disruption of Lightspeed capabilities initializing.

LGTM. I left one minor comment.

Good call @TamiTakamiya . Done.


release xargo 0.3.24

We should get the fix to https://github.com/japaric/xargo/issues/320 shipped.
@jethrogb any objections?

bors r+


Allow table refresh

Integrate a DISK Cache Create/Update/Delete (CRUD) UI..
Allow the Table to be refreshed when Adding or deleting caches
!story G15.3
!component tile server

This corresponds to issue number 128.  This was implemented during the implementation of the disk cache UI


feature-wip add fe snapshot tvf

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[ ] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[ ] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 35.72 seconds
load time: 657 seconds
storage size:<PHONE_NUMBER>8 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230104143610_clickbench_pr_73563.html


Add salanki as member

In conjunction with: https://github.com/kubeflow/kfserving/pull/1071
Contributions: https://github.com/kubeflow/kfserving/pulls?q=is%3Apr+author%3Asalanki
Suggested by @ellistarn

/lgtm
/approve

/ok-to-test

/assign @jlewi

/lgtm
/approve
thanks for your contributions!


Adding CI with github actions

In the CI we will use github secrets to directly download e2e test data from AWS. The problem with this approach is that secrets will not be available to PRs created from public forks due to security reasons. As we don't anticipate many external contributions at the moment, I think it should be ok. When such PRs are created, the e2e tests are going to be fully skipped, so they will need to be manually run locally by one of the internal developers. What ideally we should have is some way to report that those tests are skipped (either because there is some mistake in CI or because it's external contribution). I've quickly tried to set it up, such that github will automatically generate comment with pytest report, but it didn't immediately work (although it should be possible to set it up, so let's leave it for the future).

We should also expand this with other CI things, such as pre-commit.ci and check for commit signature