Regex version is incompatible with spaCy

I'm interested in using wordfreq in a project I'm doing with spaCy; unfortunately, both wordfreq and spaCy require different versions of regex at an exact version number. Would it be possible to loosen the requirements on the regex version needed?

Makes sense to me.
I wish that regex could provide more meaningful version numbers about what breaks compatibility. Anyway, from looking at the changelog, I believe that any version from 2017-07-11 to 2018-02-20 should have the same behavior in wordfreq.
A problem still remains when I want to upgrade past the breaking changes in segmentation on 2018-06-05 and 2018-06-08.

Fixed in version 2.2.1, which is out now.


feat: add isFireTV() method for Android devices

Description
Added isFireTV() that allows to check whether an Android device is a Fire TV.
The implementation is based on the Amazon Dev docs: https://developer.amazon.com/docs/fire-tv/identify-amazon-fire-tv-devices.html
Compatibility

OS
Implemented

iOS
❌

Android
✅

Windows
❌

Checklist

[x] I have tested this on a device/simulator for each compatible OS
[x] I added the documentation in README.md
[ ] I mentioned this change in CHANGELOG.md
[x] I updated the typings files (deviceinfo.d.ts, deviceinfo.js.flow)
[ ] I updated the web polyfill (web/index.js)

Interesting...I can understand the need definitely.
I feel like we will have a large and growing set of APIs similar to this though, if we put this in.
How about we go larger, and instead expose the basic API? https://developer.android.com/reference/android/content/pm/PackageManager.html#getSystemAvailableFeatures()
You could return the whole array of available features - which could be very generally useful to the community - and easily pick out if the system had the feature you needed (fire tv)
What do you think? I'd love to merge a change that exposed all available features...

Good idea, initially that what I was looking for. I'm happy to do another PR tomorrow instead of this.

Updated and squashed into 1 commit. Please review and let me know what you think.

This certainly looks more generally applicable - your use case is nailed. My use case is actually to get the list of features though, rather than querying one by one, I'd really like to see the whole list emitted and have the consuming library check if the key they want is there.
I can see your API as being nice sugar on top of that but for the truly general case, could you add a parallel API that just emits the whole set of strings using https://developer.android.com/reference/android/content/pm/PackageManager.html#getSystemAvailableFeatures() ?
I'll trade you that change request (since I can tell your use case is settled already) for me doing the web polyfill + changelog + example app update :smile: maybe we have a deal?

That's fine too. Are you happy then to keep 2 methods within the repo? One to look for a specific feature (as it is my current PR), and the other one to return an array of features? I'll start working on today and update this thread once I finished.

@mikehardy Please see the updated code, getSystemAvailableFeatures() method has been added as well. Let me know if you're happy with the implementation.

Yeah I'm fine with 2 APIs in general - thank you! The code looks good on re-read just now, I'll pull it, and run it today and hopefully merge it + release it. 💪

Strange - when I run this on my emulator I see a 'null' at the end of the feature list:

         'android.hardware.telephony',
         null ],

but when I log out the native code it has an entry, just one that looks a touch different:

2019-04-29 13:23:11.136 7343-7532/com.example I/System.out: Feature info is: FeatureInfo{4978608 android.hardware.telephony v=0 fl=0x0}
2019-04-29 13:23:11.136 7343-7532/com.example I/System.out: Feature info is: FeatureInfo{18ab1a1 glEsVers=2.0 fl=0x0}
2019-04-29 13:23:11.137 7343-7531/com.example I/ReactNativeJS: loaded info

So I'm looking at that really quickly, but - it runs! it looks good in my testing except that, and I'm looking at it now...

Ah okay. Maybe the f.name needs to be wrapped in an if statement. I havent seen null in my case but it can vary based on the device.
Sent from my iPhone

On 29 Apr 2019, at 19:25, Mike Hardy<EMAIL_ADDRESS>wrote:
Strange - when I run this on my emulator I see a 'null' at the end of the feature list:
     'android.hardware.telephony',
     null ],

but when I log out the native code it has an entry, just one that looks a touch different:
2019-04-29 13:23:11.136 7343-7532/com.example I/System.out: Feature info is: FeatureInfo{4978608 android.hardware.telephony v=0 fl=0x0}
2019-04-29 13:23:11.136 7343-7532/com.example I/System.out: Feature info is: FeatureInfo{18ab1a1 glEsVers=2.0 fl=0x0}
2019-04-29 13:23:11.137 7343-7531/com.example I/ReactNativeJS: loaded info
So I'm looking at that really quickly, but - it runs! it looks good in my testing except that, and I'm looking at it now...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Oh seriously !?! Look at the FeatureInfo.toString implementation, they have a specific case for "glEsVers", and it was the one that spits out null:
    @Override
    public String toString() {
        if (name != null) {
            return "FeatureInfo{"
                    + Integer.toHexString(System.identityHashCode(this))
                    + " " + name + " v=" + version + " fl=0x" + Integer.toHexString(flags) + "}";
        } else {
            return "FeatureInfo{"
                    + Integer.toHexString(System.identityHashCode(this))
                    + " glEsVers=" + getGlEsVersion()
                    + " fl=0x" + Integer.toHexString(flags) + "}";
        }
    }

So, yeah, I'm inclined to just do a null check on name but not try to replicate their case-specific logic, as checking OpenGL information is a separate, subtle topic that would need to be it's own API https://developer.android.com/guide/topics/graphics/opengl#version-check
Just doing a null check on f.name with a continue-if-null workaround would be perfect

react-native-device-info 1.6.0 published 🎉
@markhomoki  flawless victory :-), thank you

That's great. Thanks for the update. I found some inconsistency in the ordering in the README.md. I might create another PR and fix it.


!(HTTP 500 because of Recursions of Redirections)

Editied .htaccess to Provide RewriteBase

I have a problem with the rewrite
http://paste.anzahcloud.de/
it does not run

I can open it, It Runs

i dont mean the page, i mean the navigation, they does not run

It' looks like an Issue with the Page Settings in the Configs, can you send me your Siteconfig via PM, then I can look whats wrong.

Ahh, you need This:
RewriteEngine On
RewriteBase /
Redirect Trailing Slashes...
RewriteRule ^(.*)/$ /$1 [L,R=301]
Handle Front Controller...
RewriteCond %{REQUEST_FILENAME} !-d
RewriteCond %{REQUEST_FILENAME} !-f
RewriteRule ^ index.php [L]

I have followed the tutorial installation but the project does not run

Do you use My Version or the Original from sayakb ?
In some Cases you have to modify .htaccess or not.

404 With this simple change
Without I get:
Error in exception handler: View [common/error] not found. in /bootstrap/compiled.php:9429


Adding form type to the generated user id

Changes
Adding form type to the list of values used to generate the user id. This is needed so that the user can complete multiple surveys with the same id but different form types.
How to test

Check out this branch
Turn on server side storage

export EQ_SERVER_SIDE_STORAGE=True
export EQ_SERVER_SIDE_STORAGE_ENCRYPTION=True

Select the MCI survey with form type 0205
Partially complete the survey, save it and then check you can return to it later
With partially completed data select the MCI survey 0203 and check that you are taken to the landing page.

Who can test
Anyone apart form @warrenbailey

LGTM :+1:


Feature Request: ctrl+enter to submit

Hello. Thank you for this plugin. It make my workflow more convenient!
So. I wish to submit modal form with my keyboard. I can use Tab to switch to submit button, but it would be great to do it with one keystroke. I guess it's not hard to implement.

Hello @theotheo , thank you for the kind words.
I could take a look at adding support for that


start commands from readline (interactive)

what are the possibilities to build an interactive console application with commanddotnet? instead of arguments provided with app startup, the same arguments can be given by using a readline. when the readline is ready there should be some way to start a command (CommandRunner.RunCommand?).
can something like this be done with commanddotnet?

Its not possible right now. But it seems that this should not be complicated to achieve.

Is it possible to create a new instance of AppRunner within command like the following?   I haven't tested this.   If this does work, I was wondering as a next step if Echo could be extracted into another class that CliApplication inhertis so the inner AppRunner can use the base class without including the Cli command.

    [ApplicationMetadata(Name = "cli", Description = "Sample interactive console")]
    public class CliApplication
    {
        [InjectProperty]
        public AppSettings AppSettings { get; set; }

        [ApplicationMetadata(Description = "Enter interactive mode")]
        public void Cli()
        {
            Console.Out.WriteLine("Interactive mode");
            while (true)
            {
                //TODO: do not split quoted spaces
                var input = Console.ReadLine().Split(new[] {' '}, StringSplitOptions.RemoveEmptyEntries);
                if (input.Length == 1 && (input[0] == "exit" || input[0] == "quit"))
                {
                    break;
                }

                AppRunner<CliApp2> appRunner = new AppRunner<CliApp2>(AppSettings);
                appRunner.Run(input);
            }
        }

        [ApplicationMetadata(Description = "Echo console")]
        public void Echo(string input)
        {
            Console.Out.WriteLine(input);
        }
    }

got something working by using another lib
https://github.com/commandlineparser/commandline


Azure cosmos - end-of-September release

API change check
API changes are not detected in this pull request.

Latency and throughput looks good, signing off.


Why intervals changed?

For metric.impl.ExplicitMetricSpace, do I need to call finalizeStream after I call api.Plex4.createVietorisRipsStream? My code is as follows:
M = csvread('150pcs1.csv'); // Distance Matrix
m_space = metric.impl.ExplicitMetricSpace(M);
stream = api.Plex4.createVietorisRipsStream(m_space, 2, 7.5);
persistence = api.Plex4.getModularSimplicialAlgorithm(2, 2);
intervals = persistence.computeAnnotatedIntervals(stream);
Also, after I changed max_filtration, my bar codes graph and my intervals changed. Not only the end points of the intervals changed, but also the data points which form these intervals. For example, when I set max_filtration to 7.5, I got an interval
[4.5, 5.625): [9,10] + [0,18] + [45,137] + [36,45] + [1,6] + [3,137] + [3,6] + [7,36] + [7,9] + [10,29] + [1,18] + [0,29],
but after I changed max_filtration to 7, this interval changed to [4.55, 5.6): -[7,49] + [3,6] + [10,29] + [3,31] + [7,31] + [10,49] + [1,6] + [0,18] + [0,29] + [1,18]. I wonder why this happened.
Hongyuan

The reason why the inteval changed from [4,5, 5.625] to [4.55, 5.6) is explained in the second paragraph of Section 5 of the Javaplex tutorial:
http://www.math.colostate.edu/~adams/research/javaplex_tutorial.pdf
Indeed, Javaplex discretizes the scale parameters from 0 to max_filtration into some number of bins (which you can specify depending on the level of precision you want), and if you change max_filtration, then you change the width of each bin, and hence you change the discretized start and end time of each interval.
The representative cycle changing from [9,10] + [0,18] + [45,137] + [36,45] + [1,6] + [3,137] + [3,6] + [7,36] + [7,9] + [10,29] + [1,18] + [0,29] to -[7,49] + [3,6] + [10,29] + [3,31] + [7,31] + [10,49] + [1,6] + [0,18] + [0,29] + [1,18] is due to the fact that there are many different possible representative cycles for any single homology class! Javaplex always returns one such representative cycle --- there's no notion of a "canonical" representative cycle to return, and that's why Javaplex returns an essentially arbitrary one. I could point you to more references here if you like --- this is the representative cycle that is output by the standard persistence algorithm, and there are references I could point you to here.


Enable scalafmt on library/.

Review by @gzm0

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/3580/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/4723/
Test PASSed.

undesirable (but tolerated) change.

OK, I see. In that case, can you add this to the commit description so it is clear why formatting changes in two different commits. Also, it might be a good idea to link the issues in the commit.

And for now, I'm fine with leaving the AdHoc soft-line length settings.

Closing as this would probably have to be redone from scratch at this point.


Fix JSON-LD schema params mapping

I noticed that the JSON-LD metadata was still showing the default image for image/logo after setting the value under [schema] in params.toml.
The if statement for the logo/image uses schemaType. All other references are .Site.Params.schema.type.
After changing to ...schema.type to match the others, the logo/image is correctly being updated with the value from params.

Thanks, @timcappalli, for spotting this and for the PR!


Replace usage of realpath for maintaining osx compatibility

Bug Report
What happened:
In OSX, realpath is not part of the standard installation. Due to this we are not able to create a brew formula for osx. https://github.com/Homebrew/homebrew-core/pull/29646
What you expected to happen:
Replace realpath with bash string manipulation or something similar

In the linked PR, not sure why you want to run make .generated_files at all. Everything generated should be checked in.
/assign @carolynvs

@MHBauer Thanks for looping me in! I've worked with homebrew a bunch before on similar stuff so I can help shepherd getting their PR merged over there.
As you pointed out, our generated files are checked in so this issue can be closed. @ageekymonk I'll work with you on the homebrew PR to unblock you on whatever you need in order to get your homebrew PR merged.

Is the brew formula something we (service-catalog project) should own?

Homebrew encourages the community to support and maintain their formulas. Having the formula come from the community (and I checked it out, it follows their template for Go pretty much to a T), is a good thing. 💖


index_copy_: incorrect with CUDA inputs

Likely the cause of failures in https://github.com/Lightning-AI/lightning-thunder/pull/788
Take a look at the following repro:
In [1]: import torch, thunder

In [2]: def f(x, idx, src):
   ...:     x.index_copy_(0, idx, src)
   ...:     return x
   ...:

In [3]: jf = thunder.jit(f)

In [4]: x = torch.rand(3, 3)

In [5]: idx = torch.arange(3)

In [6]: src = torch.ones(3, 3)

In [7]: xx = x.clone()

In [8]: f(x, idx, src)
Out[8]:
tensor([[1., 1., 1.],
        [1., 1., 1.],
        [1., 1., 1.]])

In [9]: x
Out[9]:
tensor([[1., 1., 1.],
        [1., 1., 1.],
        [1., 1., 1.]])

In [10]: jf(xx, idx, src)
Out[10]:
tensor([[1., 1., 1.],
        [1., 1., 1.],
        [1., 1., 1.]])

In [11]: xx
Out[11]:
tensor([[1., 1., 1.],
        [1., 1., 1.],
        [1., 1., 1.]])

In [12]: x = torch.rand(3, 3).cuda()

In [13]: idx = idx.cuda()

In [14]: src = src.cuda()

In [15]: xx = x.clone()

In [16]: f(x, idx, src)
Out[16]:
tensor([[1., 1., 1.],
        [1., 1., 1.],
        [1., 1., 1.]], device='cuda:0')

In [17]: x
Out[17]:
tensor([[1., 1., 1.],
        [1., 1., 1.],
        [1., 1., 1.]], device='cuda:0')

In [18]: jf(xx, idx, src)
Out[18]:
tensor([[0.1944, 0.2947, 0.2669],
        [0.3471, 0.5339, 0.4950],
        [0.9053, 0.2321, 0.0383]], device='cuda:0')

In [19]: xx
Out[19]:
tensor([[0.1944, 0.2947, 0.2669],
        [0.3471, 0.5339, 0.4950],
        [0.9053, 0.2321, 0.0383]], device='cuda:0')

This is the trace:
# Constructed by Delete Last Used (took 0 milliseconds)
import torch
from thunder.executors.torchex import no_autocast

@torch.no_grad()
@no_autocast
def computation(x, idx, src):
  # x: "cuda:0 f32[3, 3]"
  # idx: "cuda:0 i64[3]"
  # src: "cuda:0 f32[3, 3]"
  t0 = torch.index_copy(x, 0, idx, src)  # t0: "cuda:0 f32[3, 3]"
    # t0 = ltorch.index_copy(x, 0, idx, src)  # t0: "cuda:0 f32[3, 3]"
      # t0 = prims.index_copy(x, idx, src, 0)  # t0: "cuda:0 f32[3, 3]"
  del idx, src
  nvFusion0(t0, x)
    # prims.copy_(t0, x)
  del t0
  return x

I suspect there is something going on on the boundary between the op and the fusion region. A quick look at torch/__init__.py shows that all in-place ops that are present there have their representative in the nvfuserex, and so they are being put inside a fusion region along with the copy. Copy alone works, but a non-nvfuser prim + copy does not. cc @crcrpar

We should probably also have a look at how copy_ behaves when it is the only op in the fusion. Looks like no kernels are being launched, judging from https://github.com/Lightning-AI/lightning-thunder/blob/29379ec825c6101888c4c3d5af4e4a7cbf475b24/thunder/executors/nvfuserex_impl.py#L2036-L2046

We should probably also have a look at how copy_ behaves when it is the only op in the fusion. Looks like no kernels are being launched, judging from
https://github.com/Lightning-AI/lightning-thunder/blob/29379ec825c6101888c4c3d5af4e4a7cbf475b24/thunder/executors/nvfuserex_impl.py#L2036-L2046
Is my assessment correct? (No CUDA env available to test it at the moment...)

sorry for the late response.
You were right about that. But that's not the right behavior. codegen should be smart enough to figure out when to launch a kernel to realize the inplace update.
I'm patching it and we should be able to revert the WAR in thunder afterwards.


symbols

It would be nice to include the contract tuple values in symbols table and provide apis to query symbolid for specified contract tuple and internal single word symbol.

While QTPyLib doesn't have this functionality, it does offer a few methods that can streamline the process of obtaining the relevant tuple for the instruments you want to trade.
Obviously, US stocks doesn't need any shortcut or api... simply use AAPL, MSFT, etc...
For any of CME Group's Futures (inc. CME, GLOBEX, CBOT, NYMEX and COMEX) you can use either the futures.get_active_contract(...) or FUT.CL methods - which will automatically trade most active futures contract month, based on the previous day’s volume and open interest.
You can also get contract details for any Futures contract traded on IB using futures.get_ib_futures(...) which will give you this data:
{
  'class': 'CL',
  'currency': 'USD',
  'description': 'Light Sweet Crude Oil',
  'exchange': 'NYMEX',
  'has_options': True,
  'intraday_initial': 2175.0,
  'intraday_maintenance': 1740.0,
  'overnight_initial': 4350.0,
  'overnight_maintenance': 3480.0,
  'symbol': 'CL'
}

You can then construct your
from qtpylib import futures

contract = futures.get_ib_futures("CL")
tuple = (contract['symbol'], "FUT", contract['exchange'], contract['currency'], 201612, 0.0, "")

Hope that helps :)

This functionality was just added to version 1.4.9 via the futures.make_tuple(...) method.
Simply pass the symbol, expiry (and optionally, exchange) and you're good :)
from qtpylib import futures

strategy = MyStrategy(
    instruments = [
        futures.make_tuple("CL", 201612),
        futures.make_tuple("GBL", 201612, exchange="DTB")
    ],
    ...
)

More info can be found in the docs:
http://qtpylib.io/docs/latest/futures.html#auto-tuple-generation
Enjoy!

Marking this issue as closed for now


Do we need source dependend features in the analysis.

Or can we achieve the same result (significance) with an energy-dependent theta cut?
This is something which we should find out at some point.
It could reduce the complexity a lot.

So the ICRC stuff by @MaxNoe seems to indicate no. If so, I would like to remove that feature completely.

It is still 7 sigma difference in detection capability currently.

with energy dependent theta cuts?

Do you think it would be good for later  reference, to actually link the plots and results you refer to here? In case somebody wants to review this decision in the years to come?


PCA

Hello, this is a very worthwhile paper, but I have a question for you now. Can you provide the code for the "PCA Visualization Results" part of your paper?Thank you so much！

Hello, this is a very worthwhile paper, but I have a question for you now. Can you provide the code for the "PCA Visualization Results" part of your paper?Thank you so much！

see generate_latentZ_visualize(model, opt) function in https://github.com/mengchuangji/Seis-INN/blob/main/codes/test_generate_noisy_segy.py


Question: don't use leading k in Go names. Why?

I can't find any reference indicating why Golint warns when doing this. Some explanation would be greatly appreciated!

Idiomatic Go doesn't use Hungarian notation and variable names shouldn't be prefixed with their intended use. For example, a constant should be named foo or Foo, not kFoo or KFoo.

Thank you for the answer. I have a KBToMB function and it drives me crazy to see the lint error not knowing why.

@DavidGamba  Are you using a recent version of golint? Looking at the code, it only matches on lowercase k, not uppercase K. It also doesn't match if the letter following the k isn't uppercase. So neither KBToMB nor kbToMB should get flagged.

@dominikh, I apologize, the actual name I have is kBToMB because it is unexported. I should be using kbToMB to avoid mixing case in initialisms.

Hi,
I am bit by this rule. I have a kafka consumer and a partition consumer in my struct. I am naming them as kConsumer and pConsumer. But the kConsumer is throwing lint error. I would really want to avoid naming it consumer just to make it clear that its a kafka consumer.
What is the recommendation here ? Do I have to name it kafkaConsumer ?

I am also having issues with this where I am parsing an array of strings of the form []string{"key1", "value1","key2","value2"} for example:
var arr = []string{"key1", "value1","key2","value2"}
for i, kOrV := range arr {
}

I am clearly NOT using Hungarian notation here - the kOrV means keyOrValue so why am I being falsely penalized by golint here?
This is clearly a bug in golint in this case, or at best an annoying false positive.  Is there any way we can fix this?
Thanks!

I'm writing a web server for a site which serves information about a phonetic orthography I've made for English for fun. In one template processing file I have these vars:
	ɑIPA  = "IPA: ɑ/ɔ/ɒ"
	æIPA  = "IPA: æ"
	ɛIPA  = "IPA: ɛ/e"
...
	kIPA  = "IPA: k"
...
	dʒIPA = "IPA: d͡ʒ"
	tʃIPA = "IPA: t͡ʃ"

(Most of the phonemes in the above code were omitted here to conserve space.)
I am told, 'don't use leading k in Go names; var kIPA should be iPA'.
If you look at the pattern here, I am clearly not using Hungarian notation.
I think this shouldn't be marked as a problem.

How about using k short for key? Such as looping a map:
for k, v := range params {
	kTrim := strings.TrimSpace(k)
	vTrim := strings.TrimSpace(v)
	delete(params, k)
	params[kTrim] = vTrim
}

Just adding one more corner case in which this rule bit us, we wrote a function that implements a k shortest path routing algorithm and obviously we called it kShortestPath.
Unfortunately that's also mistakenly identified as Hungarian notation.


get_leading_stride() is too restrictive

I tried to do the following:
xtensor<double, 1> A{1, 2, 3, 4, 5},
                   B{0.25, 0.5, 0.25},
                   C(B.shape(), 0);

// create a view to A looking like
//    1, 2, 3,
//    2, 3, 4,
//    3, 4, 5
static_shape<std::size_t, 2> shape{3, 3}, strides{1, 1};
auto A_as_matrix = adapt(&A(0), 9, no_ownership(), shape, strides);

// call matrix multiplication
C = linalg::dot(A_as_matrix, B);

but got the error message "No valid layout chosen." from get_leading_stride() (xblas_utils.hpp, line 118). However, when I hard-wired get_leading_stride() to return 1, I got the desired result, so my general idea seems to be valid. How can this be solved?

Hi Ullrich!
Hmm, this is a good question. Maybe, if no static layout is chosen, we should look for the smallest stride and use that as leading stride ..?
Do you think that is appropriate?
The other way to go might be to use the xarray/xtensor_adaptor directly, and choose a static layout. (However, your strides do not suggest a valid static layout).

If you agree with the mentioned PR, we can merge and release a new version by tomorrow.

Thanks for the PR. It appears to solve my problem, but I don't know if it has any undesired side effects, as I haven't checked what exactly get_leading_stride() is used for.

get leading stride is a rather simple function: for row-major, it selects the last stride, for col-major it selects the first stride.
Now, without a layout (e.g. custom strides), it becomes a little trickier. Either we throw an error (like before this PR), or we try to guess the best stride (with the new PR).
Other option would be to call "raw" BLAS functions ...

Ok, if you don't see a failure case the solution is good.

Ah, actually, I need to review what I did tomorrow morning. Not sure anymore if we're looking for the smallest or the largest stride :)
But we'll fix it first thing tomorrow

Great.

Hi Ullrich,
The problem is that adapt choses Dynamic Layout and in xtensor blas we require a static layout (row or col major).
If you would use the

Sry

Your last comment gut truncated.
I also tried to pass row_major as a template   parameter, but then the code refused to apply my custom strides. I didn't find another possibility to force my matrix view to be considered row_major.

Just to clarify: The view should preferrably be interpreted as row_major.

I think I've commented on this somewhere else before but the biggest issue with this is that BLAS is not guaranteed / does not work with such a striding scheme as the (e.g. intel) docs state:
lda : Specifies the leading dimension of a as declared in the calling (sub)program.
    For Layout = CblasColMajor, the value of lda must be at least max(1, m).
    For Layout = CblasRowMajor, the value of lda must be at least max(1, n).

so a stride smaller than the shape is not allowed, and the BLAS subroutine returns a negative info value.
It appears to be working in FLENS though -- however, that is a simpler and slower BLAS implementation.
I am going to close this issue for now.


Integrate LLVM at llvm/llvm-project@131dda9

Updates LLVM usage to match llvm/llvm-project@131dda9.
Further updates the MLIR-HLO submodule to tensorflow/mlir-hlo@3c6e1c8.

Had to address that clang-format failed...

Blocked by https://github.com/tensorflow/tensorflow/pull/57100.

I bumped the the mlir-hlo submodule to 817776b as in incorporates the fix.


Bug: Tabs in between the first and last one are blank on startup

Description

Same issue as #13780 but for me when using the open a specific page or pages
Steps To Reproduce

Enable Files settings > Genral > Startup settings > Open a specific page or pages
Add at least three tabs to open
Restart Files
See middle tabs are blank

Requirements

Show all tabs icon and folder names

Files Version
<IP_ADDRESS>
Windows Version
10.0.22621.2715
Log File
debug.log

Thanks for the report


build.sh created with Windows line endings

I'm on macOS Sierra using Forge through Ionide. Just generated a new project and the build.sh has Windows line endings.
Expected Behavior
Running build.sh at the command line should not produce errors.
Current Behavior
Running build.sh fails with "build.sh: line 34: syntax error: unexpected end of file"
Possible Solution
Diffing the build.sh against another (working) project I generated just a few days ago shows that the only difference in build.sh is that the non-working one has CRLF line endings.
Steps to Reproduce (for bugs)

In VS Code 1.10.1 with Ionide 2.23.9, Ionide-FAKE 1.2.3 and Ionide-Paket 1.6.3 open the directory that will be the root folder of the project.
In the command palette, use the "F#: New Project" command, select a classlib, the directory src, and a name for the project.
Wait a bit for the build files to be generated.
Open build.sh in the editor. The editor mode will be CRLF. (This can also be verified using other tools.)
Try using the command "FAKE: Build Default"; the command will fail with a syntax error.

Context
Can't run builds without changing the line endings of the file.
Your Environment

Version used: Unclear how to determine what version is in Ionide 2.23.9
Environment name and version (F#, .Net): mono 4.4.0, F# 4.0
Operating System and version: macOS Sierra

I think we've recently updated Forge templates and fixed issue with line endings. Try running F#: Refresh Project Templates and check if problem is still reproducible

That worked. However, two things:

Last line of console output is:

Unhandled error:
Could not find file "/Users/[user]/Documents/[path_to_project]/src/build.sh".

Not sure why since the file is there.

The last line of the output when refreshing the project templates is the git clone command.  As far as I can tell there's no output from git clone and also no output indicating that the command to refresh templates is finished.

The

Should be fixed as part of #71


chore(main): release 2.0.1

An automated release has been created for you.
2.0.1 (2023-10-03)
Bug Fixes

Update callback to match spec (#21) (5902653)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/stordco/buffy/releases/tag/v2.0.1 :sunflower:


Nil assertion failed in codegen_fun when using closured object inside captured block inside CONST initializer

I was writing my first Crystal code for a while and I managed to hit this in 5 minutes! I think I have previous maintainer's bad luck.
Here's a minimal test case, which reproduces with --prelude=empty:
class Foo
  def bar
  end
end

def func(&block)
  block.call
end

CONST = begin
  foo = Foo.new
  func do
    foo.bar
  end
end

CONST

Backtrace:
Nil assertion failed (NilAssertionError)
  from /data/programming/crystal-lang/crystal/master/src/nil.cr:106:5 in 'not_nil!'
  from /data/programming/crystal-lang/crystal/master/src/compiler/crystal/codegen/fun.cr:103:47 in 'codegen_fun'
  from /data/programming/crystal-lang/crystal/master/src/compiler/crystal/codegen/fun.cr:51:3 in 'codegen_fun:fun_module_info:is_fun_literal:is_closure'
  from /data/programming/crystal-lang/crystal/master/src/compiler/crystal/codegen/codegen.cr:549:17 in 'visit'
  from /data/programming/crystal-lang/crystal/master/src/compiler/crystal/syntax/visitor.cr:27:12 in 'accept'
  from /data/programming/crystal-lang/crystal/master/src/compiler/crystal/codegen/codegen.cr:2184:7 in 'accept'
  from /data/programming/crystal-lang/crystal/master/src/compiler/crystal/codegen/call.cr:318:5 in 'codegen_call_with_block_as_fun_literal'
  from /data/programming/crystal-lang/crystal/master/src/compiler/crystal/codegen/call.cr:30:9 in 'visit'
  from /data/programming/crystal-lang/crystal/master/src/compiler/crystal/syntax/visitor.cr:27:12 in 'accept'
  from /data/programming/crystal-lang/crystal/master/src/compiler/crystal/codegen/codegen.cr:2184:7 in 'accept'
  from /data/programming/crystal-lang/crystal/master/src/compiler/crystal/codegen/codegen.cr:619:9 in 'visit'
  from /data/programming/crystal-lang/crystal/master/src/compiler/crystal/syntax/visitor.cr:27:12 in 'accept'
  from /data/programming/crystal-lang/crystal/master/src/compiler/crystal/codegen/codegen.cr:2184:7 in 'accept'
  from /data/programming/crystal-lang/crystal/master/src/compiler/crystal/codegen/const.cr:159:13 in 'create_initialize_const_function'
  from /data/programming/crystal-lang/crystal/master/src/compiler/crystal/codegen/const.cr:131:56 in 'initialize_const'
  from /data/programming/crystal-lang/crystal/master/src/compiler/crystal/codegen/codegen.cr:930:9 in 'codegen_assign'
... boring codegen visitor doing it's visiting, clipped for length

Hi!
I'd love to help but I can't compile the compiler anymore :-(
Undefined symbols for architecture x86_64:
  "_roundeven", referenced from:
      _*Float64#round_even:Float64 in F-loat64.o

This is on Mac OSX, LLVM 11.1.0
@straight-shoota Any ideas?

By the way, the issue seems to be that closures don't work right now inside const initializations. It should be easy to fix... once I can get things compiling again. Maybe I'll stick with LLVM 10 for now.

Same error... :-(

@asterite This is weird, and definitely a different issue. You can just revert the last commit (or just checkout master^).

I'm looking into a patch for this, it's interesting poking around in the compiler anyway :)


sic-git:  style guide violation + package naming convention concern

Per KISS Style Guide 1403, package should refer to the tarball and not the git repository (where possible).
There are no KISS rules on package naming conventions (as far as i can tell), but the "*-git" in the package name is confusing.  Considering kiss search only handles exact package names at the moment, I would advise renaming the package from "sic-git" to "sic".

reference: https://k1ss.org/style

Per KISS Style Guide 1403, package should refer to the tarball and not the git repository (where possible).

Some packages may be better served as -git ones and their sources will reflect this. It's all up to the maintainer of the package. Nothing prevents someone from submitting a sic package which tracks releases. There is typically a rationale attached to the package submission (for sic-git, see: https://github.com/kisslinux/community/pull/233).

There are no KISS rules on package naming conventions (as far as i can tell), but the "*-git" in the package name is confusing. Considering kiss search only handles exact package names at the moment, I would advise renaming the package from "sic-git" to "sic".

The -git in the name follows -bin (for binary packages). It's intentional and within reason. Also, kiss search supports globbing (kiss s 'pkg*'). I agree that the documentation here is lacking regarding this information. I'll get onto fixing this.

I've updated the style guide to contain this information. :+1:


JwsBuilder creates Jws payload with empty scope claim

The following JwsBuilder:
String jws = JwsBuilder.withAlgorithm(JwsAlgorithms.RS256)
		.claim("claim1", "value1")
		.sign("kid-1234", signerKey)
		.build();

builds a Jws payload with an empty scope claim as follows:
{
  "scope": "",
  "claim1": "value1"
}

the resulting Jws payload should be:
{
  "claim1": "value1"
}

Good catch, @jgrandja.


Proposal to separate the javascript into two parts; core and rails specific.

Feature Request
Right now the stimulus reflex javascript library is in a single file molded to work perfectly with actioncable. I would like to see a separation into two javascript packages; stimulusreflex-core and stimulus_reflex.
stimulus_reflex would depend on stimulusreflex-core and this package would be tailored to work with rails and thus have a dependency on actioncable.
stimulusreflex-core would contain all the logic for making stimulusreflex work the way it does today except for the part where it assumes that actioncable is the websocket connection.
This feature request can only be dealt with after we've done satisfactory work on #182. #345 Does some excellent groundwork as well, and if that gets merged that would also set us up to split the library into a core and 'rails' specific implementation.
Is your feature request related to a problem?
As a django user I'm able to use the javascript version of stimulus reflex. Right now it depends on the javascript actioncable library, as a django user I wouldn't want that to be included.

Hey @jonathan-s, as you know @julianrubisch has been doing some high-quality thinking about how to make SR more modular with an adapter approach, especially around transport mechanisms.
It seems like in order to support other transports, we'd have to compartmentalize ActionCable anyhow. This would seem to (somewhat) implicitly address the spirit of this issue, because ActionCable would become just another adapter.
This would all very definitely require a major SemVer bump, so we do have some latitude to make big changes. However, the other side of the coin here is that this isn't a trivial adaptation and we are currently chasing other priorities so I want to be realistic about the degree to which this is on an unknown timeline. Hopefully Q2 2021? I'm spitballing.
One thing that could be really helpful is a triage of the API surface to identify which chunks of code would ultimately be moved into an adapter. I don't expect you to own this on the server, but on the client, there's some good conversation to be had around, for example, what I call "library-level" events for connectivity. Eg. does an adapter have to support these events? What does it mean to be connected or disconnected when you're using Ajax? Is there a heartbeat involved?

Thanks @leastbad, I definitely understand that this isn't a trivial update and needs some thinking around it. I added this issue so we can start some thinking around it for this very reason.
You also pose some great questions around what an adapter should support. I'll have a think about this and provide some input here!

As I already noted elsewhere a couple of times, IMHO the hard part is to abstract away the CableReady dependency on the client side, since it we make transport adapters a reality, we are going to need a morphdom wrapper that is not bound to ActionCable.
So my €0.02 are: we should separate these two discussions. I think splitting CR into „core“ and „ws“ should be fairly straightforward

Some thoughts on how I would structure things.
controller.StimulusReflex would have a consumer that you can access, instead of accessing the consumer via the subscription.  The consumer should have a method isOpen. I would also rename isActionCableConnectionOpen to isWebsocketConnectionOpen
That would help here.
The register method would also take a channel argument and a create subscription function. The register method would be part of stimulusreflex-core and from stimulus_reflex you'd export a register method that doesn't require a channel argument or a create subscription function.
This would make it possible for stimulusreflex-core to not depend on getConsumer which is actioncable specific.
The object that is added in subscription here should be a constant that is available in stimulusreflex-core. You'd use that constant as a part to make your own create subscription method.
actionCableConsumer would also be renamed to websocketConsumer
The initialize function would follow a similar pattern as the register function.
I believe that these changes would largely be backwards compatible.

This is exactly what I had hoped for.
My eyes are very close to the ground, so the best help comes from folks looking from above.

Closing because going forward TurboBoost will take the role of the transport layer


Extend driver dispatch tests for MAC multipart

Description

Split mac_sign test case into single and multi part test case,
Split mac_verify test case into single and multi part test case,
Extend multipart test cases to test different number of parts.

For 0 parts I generated result using online generator. Not sure if this is acceptable approach:
https://www.liavaag.org/English/SHA-Generator/HMAC/
All cases are run at least by the component_test_psa_crypto_drivers.
Status
READY
Requires Backporting
Yes
2.x: TBD
Migrations
NO

CI pr-merge seems like hanged (pending) for a very long time. Also all is green. Not sure if this result can be considered as passed.

Rebased on development branch due to some problems with CI.

The command "choco install python --version=3.5.4" failed and exited with 1 during .
I got constantly the following CI error for windows build.

I've seen choco install fail randomly before. It recently failed on https://app.travis-ci.com/github/ARMmbed/mbedtls/jobs/554887762 as well but passed on https://app.travis-ci.com/github/ARMmbed/mbedtls/builds/244535790. The package still exists. It should work if you update the branch.
I clicked the rerun button on this PR but it doesn't seem to have any effect. It used to be a known problem that the rerun button had no effect, but that was when we were on a legacy service plan, and now we're on a normal plan and I think it should work. I clicked the restart button on the job itself and that is rerunning.

@AndrzejKurek @minosgalanakis Sorry guys I had to rebase this PR on development to fix CI issue with choco install.


Define defaults for spec_source_id in PackageConfig

'spec_source_id', when not defined, will default to "Source0" or
"Source" whichever is found first in the spec file.
Make this double-default more obvious by defining it in PackageConfig.
Signed-off-by: Hunor Csomortáni<EMAIL_ADDRESS>

Actually, while you're at it, would it make sense to add fail when no source has been found/updated? maybe with an additional parameter enforce=bool or sth like that.

There is a PackitException raised in _fix_spec_source, if the source tag to be updated is not found. Is this what you meant?

Actually, while you're at it, would it make sense to add fail when no source has been found/updated? maybe with an additional parameter enforce=bool or sth like that.

There is a PackitException raised in _fix_spec_source, if the source tag to be updated is not found. Is this what you meant?

Oh nvm, you're right... this would be relevant in the previous state of implementation :D with this it will traceback nicely.


[SPARK-26348][SQL][TEST] make sure expression is resolved during test

What changes were proposed in this pull request?
cleanup some tests to make sure expression is resolved during test.
How was this patch tested?
test-only PR

cc @gatorsmile @dongjoon-hyun @HyukjinKwon

Test build #100030 has started for PR 23297 at commit 9b0f88d.

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/testing-k8s-prb-make-spark-distribution-unified/6021/
Test PASSed.

Test build #100030 has finished for PR 23297 at commit 9b0f88d.

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/100030/
Test FAILed.

Retest this please.

Test build #100039 has finished for PR 23297 at commit 9b0f88d.

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/100039/
Test FAILed.

retest this please

Test build #100053 has started for PR 23297 at commit 9b0f88d.

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/testing-k8s-prb-make-spark-distribution-unified/6042/
Test PASSed.

Looks good to me

Test build #100053 has finished for PR 23297 at commit 9b0f88d.

This patch fails SparkR 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/100053/
Test FAILed.

This is a test-only PR and doesn't touch Spark R, the R failure is a known issue. I'm merging it to master, thanks!


Don't cache JRE

The JRE layer needs to be present at runtime but does not need to be present at build time. If required a JDK layer should be present at build time instead.
Signed-off-by: Emily Casey<EMAIL_ADDRESS>
Summary

Use Cases

Checklist

[x] I have viewed, signed, and submitted the Contributor License Agreement.
[x] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[x] I have added an integration test, if necessary.
[x] I have reviewed the styleguide for guidance on my code quality.
[x] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

The Liberty buildpack needs a JRE to do its job in this case. It is running a script that's part of the Open Liberty server to generate a new server instance. It could potentially use a JDK, but it doesn't need the build tools of a JDK. I think the ideal case is what it's doing now, at least from the standpoint of installing only the minimal tools required to do the job.
If we switch to using a JDK, I believe that would trigger a JDK to be installed when it's not needed (extra 100M of downloads). For example, if you build your WAR locally and pack build. A JDK would not be installed in this case, but if Liberty requires a JDK then it'll trigger the JDK to be installed unnecessarily.
On the flip side, if I understand the present situation correctly, what happens by having the jre layer as cache + build in Liberty is that even in cases where Liberty is not used, we can still see the jre layer flagged as build + cache = true. This doesn't impact apps where build is from pre-compiled assets, but when building from source Liberty passes detection always so it contributes to the build plan. The way that libpak merges the build plan entries (since multiple buildpacks can request the jre to be added), it ends up essentially taking the sum of everything, which includes the build + cache flags from Liberty (regardless of if Liberty is actually contributing). Thus even in a situation where Liberty is not used, the jre layer is flagged as build + cache + launch = true. This has the downside of creating some additional work.
I'd like to revisit this after https://github.com/paketo-buildpacks/libpak/issues/138 is addressed. It's a common complaint that the buildpacks install software like the JDK too often. I'd like to avoid forcing another JDK download & install if possible to avoid furthering this perception.
I think more investigation would need to be done to gauge impact and investigate other possible solutions. For example, adjusting the merging algorithm used, changing the build plan, or possibly changing Liberty to not call this Java command thus not requiring Java at build time.

If we switch to using a JDK, I believe that would trigger a JDK to be installed when it's not needed (extra 100M of downloads). For example, if you build your WAR locally and pack build. A JDK would not be installed in this case, but if Liberty requires a JDK then it'll trigger the JDK to be installed unnecessarily.

I looked into this more and for some cases, it does require a JDK to be installed where otherwise a JRE would be sufficient, however, this is partly negated by building from source and already having the requirement of a JDK. This case is also isolated to a subset of users, those pushing precompiled WAR files or packaged servers.
This is compared to the present state where you can have issues with the JRE layer flags across all applications. This can cause build failures in some cases and it causes additional work for making the JRE available during build and caching those layers when they are not needed.
As such, we're going to move ahead with a change to use the JDK at build time instead. See #155. Closing this issue as it's not quite the solution we need.


Segmentation Fault in VirtualBox

I tried to check for variants between two datasets in BioLinux 8 in VirtualBox. The command I entered is freebayes -f reference.fasta test_aligned_sorted_marked.bam > var.vcf.
I received the output zsh: segmentation fault  freebayes -f reference.fasta  > var.vcf.
Is this due simply to virtualization? Has anyone else had this problem? I will try running the program directly on the computer as well.

Generally if you get a segmentation error it is due to a bug in the code; however, that bug may not be triggered on all platforms due to differences in how memory is managed. To find the bug I'd suggest having a look a the core file with gdb or running the process within gdb. To do this you probably want to recompile using 'make debug', after first doing 'make clean'. This will let you know where the program is running into problems and will make it easier to work out what is going on.
Having said that, it is possible to get segmentation errors as a result of running out of heap memory; especially with code the has deep recursion.
To use gdb:
http://www.unknownroad.com/rtfm/gdbtut/gdbadvanced.html#CORE
cheers,
Martin

Thank you for your suggestion. I resolved the problem with the unlimit command prior to running.

What did you do with unlimit?
On Wed, Jul 20, 2016 at 2:18 PM Blipblopbloop<EMAIL_ADDRESS>wrote:

Closed #306 https://github.com/ekg/freebayes/issues/306.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ekg/freebayes/issues/306#event-729374533, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAI4EZ1mP1CtUydmH5JyNhRc3hWA89fQks5qXmZwgaJpZM4JQCzj
.

are you sure you don't mean: ulimit
you can change the maximum size of the stack with ulimit, though I don't remember the details. If you know a particular program is likely to require a bigger stack, you can also call use ulimit from within the program to set user limits (man ulimit). However, the man page tells me that this is now obsolete and that you should instead use setrlimit.

I am sure that I mean unlimit. I just typed unlimit into the terminal and hit enter before I ran FreeBayes. To the best of my knowledge, this removes the limit on use of system resources for the next program that is run.

That's a new one to me then. Useful information. I've used ulimit
previously for the same purpose, so thought it might be a type.
thanks,
Martin
--
Martin Jakt
Senior Researcher
Marine Genomics Research Group
Faculty of Biosciences and Aquaculture
University of Nordland
P.O. Box 1490, Bodø, Norway
On 21 July 2016 at 15:28, Blipblopbloop<EMAIL_ADDRESS>wrote:

I am sure that I mean unlimit. I just typed unlimit into the terminal and
hit enter before I ran FreeBayes. To the best of my knowledge, this removes
the limit on use of system resources for the next program that is run.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ekg/freebayes/issues/306#issuecomment-234252719, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJlGdukpnDATboHNKXPQdtmLfen81VsUks5qX3P7gaJpZM4JQCzj
.


Pallet updates according to latest discussions

Additional validations when creating withdrawal proposal.
Added missing docstrings.
Updated tests.

👏


color changes for charts

I've modified todaychart.py to change the chart colors (title, water level plot line, gridlines, x and y-axis labels, x-axis values.  I have inches for the y-axis values and have tried various places in todaychart.py code but cannot find where to change the y-axis value color (4, 6, 8, 10, 12, 14, 16 inches).  Any help where to change color for the y-axis values? (I found where to change colors for everything else).
I'm a complete novice on GitHub but can below is my modified todaychart.py code if that is helpful.  I've changed the line color to green, background to black, gridlines, title, axis labels, x-axis values to white.  I also enlarged the plot area.  I added comments to the end of each line I modifed.
todaychart.txt

Hi, ya that would be a matplotlib feature.  I used that library to create the charts.  Matplotlib is a bit of a bear to work with, if you are like me, only needing it for this one thing.
Your best bet is to reference the matplotlib documentation at https://matplotlib.org/contents.html
I haven't touched matplotlib in a couple of years.  After I got the graph the way I liked it I really haven't needed it.
If you don't have any luck let me know and I can take a look, but I am not expert with it by any stretch.  Matplotlib has  github repo, if you provide your code there someone should be able to answer it fairly easily.
https://github.com/matplotlib/matplotlib

Thank you, will try the documentation and/or the repo group and report back with results.

This would probably be buried in matplotlib’s y-axis spine minor and major ticks.

alaudet and ba05, thank you for your suggestions.  I tried researching the links you provided vs the existing todaychart.py code but was having trouble correlating the existing code to the documentation.  Then I had an idea....add a line of code to specify (overwrite the default, customize) the y-axis values similar to what I thought was happening for the x-axis values.  It worked!
I added this new line which changed the y-axis values to white.
 plt.yticks(rotation=0, color='#FFFFFF') # inch or centimeter (y-axis) values color
My next question is proper coding, would it be preferred to have colors stored in the configuration file and have todaychart read those values, assign to variables in todaychart.py?  If so I can attempt to work on that next.  Attached below is my modified todaychart.py which has a larger chart area, black background, green plot line (water depth vs time) and white for chart title, chart last update timestamp, x-y axis labels, x-y axis values.
todaychart2.txt

Good work, glad to see you got it going to your liking.
If customizing colors is a feature that you want to add in a program then using the config file makes sense.  I just never had the need to do it so I coded it statically.  It's something I would consider if someone forked the code and submitted a working pull request.
I only have one requirement on raspisump version 1 changes to the config file though.  Any change to the config file cannot break the app if someone doesn't add them to their config file.  So that would require checking for the value in the config file and if it is not there using defaults in the code.  If version 2 was ever released I would refactor the code as there would be a requirement to redo your config file.
If someone updates to the latest version1 I don't want to overwrite existing config files.  But for a version 2 I think it would be fair game for users to expect major changes and start with a fresh config file.
The limited times I did that I called the new value in the config file and when it wasn't there it caught an error and then issues the default.
For example, check the file alert.py.  I wanted to add a time interval in the config file.  So try to fetch the alert_interval from raspisump.conf.  Woops there is none, throw an exception error and just assign it a 5...so every 5 minutes.  That way someone upgrading to the latest version would  not have to touch their config file, the old one would work.
# If item in raspisump.conf add to configs dict above.  If not then provide
# a default value
try:
    configs['alert_interval'] = config.getint('email', 'alert_interval')
except configparser.NoOptionError:
    configs['alert_interval'] = 5

Hope that makes sense.

Yes that makes perfect sense.  Thank you.


adds script to import cctbx data to rs

Codecov Report
Attention: Patch coverage is 0% with 140 lines in your changes missing coverage. Please review.

Project coverage is 85.49%. Comparing base (b78e571) to head (36b5873).
Report is 39 commits behind head on main.

Files
Patch %
Lines

reciprocalspaceship/utils/cctbx_to_rs.py
0.00%
140 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #264      +/-   ##
==========================================
- Coverage   91.22%   85.49%   -5.73%
==========================================
  Files          37       38       +1
  Lines        2531     2786     +255
==========================================
+ Hits         2309     2382      +73
- Misses        222      404     +182

Flag
Coverage Δ

unittests
85.49% <0.00%> (-5.73%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

After thinking about it, the cctbx dependencies could be ignored, I think the refl tables can completely be accessed using msgpack

In [92]: import msgpack

In [93]: d = msgpack.load(open("idx-0040_integrated.refl", "rb"), strict_map_key=False)

In [94]: vals = d[2]['data']['intensity.sum.value'][1]

In [95]: np.frombuffer(vals[1])
Out[95]:
array([ 80.41394043,<PHONE_NUMBER>3,  21.55432129, ...,  49.80627441,
       -22.07312012,<PHONE_NUMBER>8])

So a lightweight version of this code could be written and not need any fancy dials conda env

Removed the CCTBX deps , slightly less functionality, but better performance
conda create -n cctbx2rs python=3.12
conda activate cctbx2rs
git clone https://github.com/dermen/reciprocalspaceship.git
cd reciprocalspaceship
pip install -e .

# new deps:
pip install ray msgpack

# test
wget https://smb.slac.stanford.edu/~dermen/rs_ints.tar
tar -xvf ts_ints.tar
rs.cctbx2rs  lys_si_8/ lys_si_8.mtz --ucell 78.3 78.3 36.9 90 90 90 --symbol P4 --nj 10
rs.mtzdump lys_si_8.mtz

this is some inspiring stuff, @dermen. as we discussed on slack, i think rs-booster is the place for command line apps like this. however, i'd like to see this refactored into an io module so we can just run something like
data_set = rs.read_cctbx( ... )

Thanks for putting this together! I agree with Kevin here -- I think it would make sense to include an rs.read_cctbx() (or should it be called rs.read_refl()?). I think generic format conversion scripts are better suited for rs-booster, although such a commandline tool could then just call to the rs.read....() method.

Sounds good, ill rejig it. And yea, read_refl or even read_dials might be more suitable.. technically the refl files are produced by the program dials in most cases..

yes -- but we probably want different methods for stills and rotation method

https://github.com/rs-station/rs-booster/pull/50

@kmdalton , just saw your message.. I think I committed too quickly :)

no problem, we can always refactor the ray stuff after this pr. i think we still haven't converged on a general policy for methods that rely on / benefit from optional dependencies.

@dermen, I like the logging changes you made. I think the code doesn't suffer at all in readability, and it might be a good practice to deploy more widely in the rs codebase.
there are a few more things i'd like to request with this pr.

We typically make io functions available at the rs.io.{function_name level using the module's __init__.py. It'd be great if rs.io.read_dials_stills was populated in there. because you have optional dependencies, i think it'd be best if you
implement a wrapper function with the signature rs.io.read_dials_stills(fnames, unitcell, spacegroup, numjobs=10, backend="...") where backend takes a string like "mpi" or"ray". then make the default backend be "numpy" which doesn't require any additional libraries and runs in serial.
Finally, it'd be good if ray and mpi4py were imported lazily and if the module could check whether they're available and fallback to numpy if not. i did something like this in the crystfel io code here.

let me know if you need a hand with any of this. i can probably help out this week. thanks!

Cool - think its mostly there ! feel free to change things as you see fit! There is no real urgency for me with this.. I am actually anticipating we need to add more options for msgpack data columns, and try/excepts where a column name doesnt exist ...

last commit was necessary to decorate read_dials_stills

Tests are in

i did some refactoring to de-emphasize the mpi idioms and make it possible to test the mpi code serially. the branch lives here.

I guess we still infer the stddev dtype ? Is that ok , and should we mention in docstring ?

Ultimately that one is up to you and what you want the API to do. I would probably defer that, because it will downcast that column to float32. If we are going to infer the stddev dtype, we might as well infer things like HKL dtypes and BATCH dtype. Let me know what you think.

I added an option to specify whether to use mtz dtype inference.

Ok, fixed one syntax error and added a unit cell / space group estimator to  to rs-booster. And tested, seems to work ok:
$ rs.print_refl ml_proc.lysozyme_ML_mode_9/idx-0024_integrated.refl

Found 8 experiment identifiers in ml_proc.lysozyme_ML_mode_9/idx-0024_integrated.refl:
	0: 17da648c-b938-5107-f582-7f7da8b803e8
	1: 1dccff2f-c147-29c6-55e3-163752dfb8fc
	2: 831a07f3-1af2-c739-ef9a-3c249f58e653
	3: 65853d5f-8458-d5e7-43a7-5101c5c3ee09
	4: 229d0132-1cb9-00ff-536a-6f872fede4dc
	5: f619a467-0abc-a18b-cbdf-75eef2df1f3e
	6: cde7c534-73ee-0d7b-5a39-77db7e62ee03
	7: 5b700c75-fa32-a892-0fdb-e53294402c3c

Reflection contents:
             names                           dtypes
        background.dispersion                        double
              background.mean                        double
               background.mse                        double
         background.sum.value                        double
      background.sum.variance                        double
                         bbox                          int6
                            d                        double
                delpsical.rad                        double
                     entering                          bool
                        flags                   std::size_t
                           id                           int
          intensity.sum.value                        double
       intensity.sum.variance                        double
                 miller_index        cctbx::miller::index<>
        num_pixels.background                           int
   num_pixels.background_used                           int
        num_pixels.foreground                           int
             num_pixels.valid                           int
                        panel                   std::size_t
                   partial_id                   std::size_t
                   partiality                        double
                           s1                  vec3<double>
                      shoebox                     Shoebox<>
                    xyzcal.mm                  vec3<double>
                    xyzcal.px                  vec3<double>
              xyzobs.mm.value                  vec3<double>
           xyzobs.mm.variance                  vec3<double>
              xyzobs.px.value                  vec3<double>
           xyzobs.px.variance                  vec3<double>

Number of reflections: 10555

$ rs.from_dials ml_proc.lysozyme_ML_mode_9/ test.mtz --extra-cols num_pixels.foreground background.mean --verbose
Found 63 reflection files.
Found 63 corresponding exptlist files. Will use 5 of them to estimate ucell and space group.
Consensus unit cell from 35 expts: <gemmi.UnitCell(78.3001, 78.3001, 36.8998, 90, 90, 90)>
Consensus space group from 35 expts: <gemmi.SpaceGroup("P 4/m")>
2024-09-20 09:36:25,931	INFO worker.py:1781 -- Started a local Ray instance.
(_get_refl_data pid=2408434) Loading ml_proc.lysozyme_ML_mode_9/idx-0021_integrated.refl
(_get_refl_data pid=2408434) ... Read in data for miller_index
(_get_refl_data pid=2408434) ... Read in data for intensity.sum.value
(_get_refl_data pid=2408434) ... Read in data for intensity.sum.variance
(_get_refl_data pid=2408434) ... Read in data for xyzcal.px
(_get_refl_data pid=2408434) ... Read in data for s1
(_get_refl_data pid=2408434) ... Read in data for delpsical.rad
(_get_refl_data pid=2408434) ... Read in data for id
(_get_refl_data pid=2408434) ... Read in data for num_pixels.foreground
(_get_refl_data pid=2408434) ... Read in data for background.mean
(_get_refl_data pid=2408430) Loading ml_proc.lysozyme_ML_mode_9/idx-0051_integrated.refl [repeated 62x across cluster] (Ray deduplicates logs by default. Set RAY_DEDUP_LOGS=0 to disable log deduplication, or see https://docs.ray.io/en/master/ray-observability/user-guides/configure-logging.html#log-deduplication for more options.)
(_get_refl_data pid=2408444) ... Read in data for miller_index [repeated 62x across cluster]
(_get_refl_data pid=2408444) ... Read in data for intensity.sum.value [repeated 62x across cluster]
(_get_refl_data pid=2408444) ... Read in data for intensity.sum.variance [repeated 62x across cluster]
(_get_refl_data pid=2408444) ... Read in data for xyzcal.px [repeated 62x across cluster]
(_get_refl_data pid=2408444) ... Read in data for s1 [repeated 62x across cluster]
(_get_refl_data pid=2408444) ... Read in data for delpsical.rad [repeated 62x across cluster]
(_get_refl_data pid=2408444) ... Read in data for id [repeated 62x across cluster]
(_get_refl_data pid=2408444) ... Read in data for num_pixels.foreground [repeated 62x across cluster]
(_get_refl_data pid=2408444) ... Read in data for background.mean [repeated 62x across cluster]
Combining and formatting tables!
Found 592100 refls from 460 expts.
Mapping batch column.
Finished combining tables!
Converted column intensity.sum.variance to MTZ-Type Q, took sqrt of the values, and renamed to intensity.sum.sigma.
Writing MTZ test.mtz ...
Done writing MTZ.

$ rs.mtzdump  test.mtz
Spacegroup: P4/m
Extended Hermann-Mauguin name: P 4/m
Unit cell dimensions: 78.300 78.300 36.900 90.000 90.000 90.000

mtz.head():

           background.mean  num_pixels.foreground  delpsical.rad  ...  intensity.sum.value  BATCH  PARTIAL
H   K  L                                                          ...
-36 -6 21            2.146                    843         -0.002  ...               30.927     26     True
    -5 21            2.112                    842         -0.001  ...              -17.545     26     True
    -4 21            2.078                    837          0.001  ...               23.911     26     True
    -3 21            2.065                    838          0.002  ...                55.26     26     True
    -2 21            2.002                    827          0.002  ...              151.213     26     True

[5 rows x 13 columns]

mtz.describe():

       background.mean  num_pixels.foreground  delpsical.rad  ...  intensity.sum.sigma  intensity.sum.value     BATCH
count         592100.0               592100.0       592100.0  ...             592100.0             592100.0  592100.0
mean             10.05                404.062            0.0  ...               72.887             3568.129   230.097
std              7.908                143.466          0.003  ...               43.961            14724.827   133.145
min              0.017                  176.0         -0.046  ...                5.319            -7621.242       0.0
25%              3.714                  287.0         -0.002  ...               47.517               72.518     114.0
50%              6.843                  377.0            0.0  ...               59.611              293.854     231.0
75%             14.711                  494.0          0.002  ...               81.446             1530.024     345.0
max             50.248                 1106.0          0.043  ...              910.503           819365.438     459.0

[8 rows x 12 columns]

mtz.dtypes:

background.mean            MTZReal
num_pixels.foreground       MTZInt
delpsical.rad              MTZReal
s1.0                       MTZReal
s1.1                       MTZReal
s1.2                       MTZReal
xyzcal.px.0                MTZReal
xyzcal.px.1                MTZReal
xyzcal.px.2                MTZReal
intensity.sum.sigma         Stddev
intensity.sum.value      Intensity
BATCH                        Batch
PARTIAL                       bool
dtype: object

$ careless mono --kl-weight=0.001  --mc-samples=20  --mlp-layers=20  --image-layers=1 --dmin=2  --intensity-key=intensity.sum.value  --uncertainty-key=intensity.sum.sigma  dHKL,xyzcal.px.0,xyzcal.px.1,delpsical.rad,BATCH,background.mean,num_pixels.foreground   test.mtz  CL/test
Careless version 0.4.4
Training:   0%|                                                                                | 0/10000 [00:00<?, ?it/s]WARNING: All log messages before absl::InitializeLog() is called are written to STDERR
I0000 00:00:1726850289.048002  571008 service.cc:146] XLA service 0x7fd217b60350 initialized for platform CUDA (this does not guarantee that XLA will be used). Devices:
I0000 00:00:1726850289.048049  571008 service.cc:154]   StreamExecutor device (0): NVIDIA A100-SXM4-40GB, Compute Capability 8.0
I0000 00:00:1726850289.182732  571008 device_compiler.h:188] Compiled cluster using XLA!  This line is logged at most once for the lifetime of the process.
Training:  13%|▎ | 1338/10000 [00:32<02:30, 57.64it/s, Grad Norm=1.50e+03, loss=1.27e+03, F KLDiv=1.13e+00, NLL=1.27e+03]


Checkpoints continue to grow after the first restore

System information

No

OS Platform and Distribution: Mac OS X, v 10.13.2

TensorFlow installed from: binary

TensorFlow version: v1.3.0-rc1-5211-gab0fcac 1.5.0-dev20171126

Python version: Python 3.5.0

Bazel version: N/A

GCC/Compiler version: N/A

CUDA/cuDNN version: N/A

GPU model and memory: Intel Iris Pro 1536 MB

Exact command to reproduce: N/A

Have I written custom code: N/A

Describe the problem
The parameter max_to_keep of the Saver class does not seem to have effect once a model and its training variables are restored. In other words, the first time I train my model, the saver is keeping only max_to_keep checkpoints. Then I interrupt the training. Later, when I resume it, the number of checkpoints keeps going without any apparent limit.
Related issues

https://github.com/tensorflow/tensorflow/issues/5929
https://github.com/tensorflow/tensorflow/issues/6326

Nagging Awaiting TensorFlower: It has been 14 days with no activity and the awaiting tensorflower label was assigned. Please update the label and/or status accordingly.

Nagging Awaiting TensorFlower: It has been 14 days with no activity and the awaiting tensorflower label was assigned. Please update the label and/or status accordingly.

Nagging Awaiting TensorFlower: It has been 14 days with no activity and the awaiting tensorflower label was assigned. Please update the label and/or status accordingly.

Nagging Assignee @bignamehyp: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @bignamehyp: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @bignamehyp: It has been 100 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @bignamehyp: It has been 116 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @bignamehyp: It has been 131 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Is it still an issue for the latest TF?

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

It has been 29 days with no activity and the awaiting response label was assigned. Is this still an issue?

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!

@bignamehyp

Please update the issue when new information becomes available, and we will reopen the issue

Update the issue? Why should I need to update the issue? TF developers have to solve this issue. There's nothing to update.


Exoplayer2/ AndroidMediaPlayerFacade tests

Problem
The underlying AndroidMediaPlayerFacade is untested 😱
Solution
Add some tests so that we can begin tidying up the AndroidMediaPlayer flow.
Test(s) added
errrr, yes 👅
Screenshots
No UI changes.
Paired with
Nobody but tea.

@Dorvaryn Can't respond to your comment 😢  but I am tracking with a TODO and with an issue now fd4dbce


image encryption

I have a question regarding to the security issue. Can cached images be encrypted ?

Well, the entire in-memory caching system is built around the UIImage and NSImage APIs. If your encrypted data can be converted into either of those types, then you can certainly store them in the in-memory cache.
The data will automatically be stored in its original encrypted form in the URLCache. However, there's no decryption logic built into any of the Alamofire image response serializers, so you'd need to implement this on your own.
Hopefully that answers your question.
Cheers. 🍻


minor bug on the extension on iphone

the extension, opened on iPhone, shows the background color of the note as white, even if the dark theme of the app is selected

I put in a quick fix, and it works for Android. @rie39414 Let me know if iPhone is working as well. Thanks.

it worked perfectly on iPhone, thank you very much!


Support night mode?

Hello,
Does this library support night mode?
IF yes how can we implement it and if no what should we do?
Because in dark mode  we have as shown  in the image below

Thanks for rising this up. I guess when designing the library I didn't pay attention to dark themes, but it should be quite easy to add. Perhaps changing some color codes to theme attributes would be sufficient.
I don't have the time to look into it currently, but feel free to try it and open a PR!

Thanks for opening the issue. I guess when designing the library I didn't pay attention to dark themes, but it should be quite easy to add. Perhaps changing some color codes to theme attributes would be sufficient.
I don't have the time to look into it currently, but feel free to try it and open a PR!
Hello,
Why the textColor( in xml )not supported in this library?

Why the textColor( in xml )not supported in this library?

Most probably I didn't consider the use-case of changing the text color. Feel free to correct this and submit an PR!

Hello,
Does this library support night mode?
IF yes how can we implement it and if no what should we do?
Because in dark mode we have as shown in the image below

The Text color not changed in all themes mode, As fixed on the black color

Hello, the issue is resolved
cannot add textColor and/or change the color of number phone in xml inner  ialokim phonefield, but you can change the color from java side when set the color of edit text value for phone edit text.
The solution using java code:
PhoneEditText etPhoneNumber;
etPhoneNumber.getEditText().setTextColor(color);
solved!
Regards.

That's correct thanks @hbo003

Hello, the issue is resolved
cannot add textColor and/or change the color of number phone in xml inner ialokim phonefield, but you can change the color from java side when set the color of edit text value for phone edit text.
The solution using java code:
PhoneEditText etPhoneNumber;
etPhoneNumber.getEditText().setTextColor(color);
solved!
Regards.

That's correct thanks @hbo003

I'm glad you've found a solution, but I'll keep this open as it should be possible to change the textcolor in the xml layout file eventually, too.


Error found!

File uploading issue!!!

Perhaps the problem with Internet access.
Check the firewall settings.
Try to open https://api-content.dropbox.com in your browser.
If the problem persists, please show your code.

Ok sir... Thank you!

Sir, I am uploading file from winform app, upto 1 gb file. it didn't uploaing, Its showing this error

https://github.com/alekseynemiro/nemiro.oauth.dll/issues/12

hi,

when I test instagram I got this error {"code": 403, "error_type": "OAuthForbiddenException", "error_message": "You are not a sandbox user of this client"}

http://stackoverflow.com/questions/34765854/how-to-be-an-instagram-sandbox-user-for-any-client-in-instagram
https://www.instagram.com/developer/sandbox/


Problem on the clustering module

Hi~  I find some problem in the clustering module. It seems like the number of clusters are fixed and can not be changed. Also, it would be nice to make it more flexible.

Maybe you can use the variable of n_clusters to make the number of clusters more flexible, don't use the default number in the sklearn.cluster.KMeans .

I know k-means clustering as well. In cluster analysis, the k-means algorithm can be used to partition the input data set into k clusters. Hope this could help you.

Thx! look more into k-means help solve my problem. Thx guys!


Correct reported ALSA sample format

Are you sure that's correct? Did you try the "music" example?

Doesn't line 28 say alsa::SND_PCM_FORMAT_S16_LE? Am I misunderstanding the implications of that call?
Anyway, I needed this change to make the output of play -e signed sound right in my program with an i16 buffer.  It's possible the bug is somewhere else, though.

Did you try the "music" example?

No, I tried at one point but couldn't build/install vorbis.

The music example is totally buggy and the beep example doesn't work.

Closing in favor of #51
Turns out the music example was itself buggy.


Update README

The README could be much better, and should be updated to be easier to use and more intuitive.
The "Get started" section should have the included bash scripts updated as well to be more accurate and easier to understand.

This is now finished with the 1.0 release.


benchmark suite: first pass at adding a golang dataset

@umpox I took a shot at adding a simple Go test case here, I'll explain some of the pieces I learned inline.
Test plan
This adds a test.

@umpox I wasn't actually able to get this to pass, interesting the test.sh script exits with the success exit code 0 and passes.. but the test suite calls it a failure for some reason? Am I missing something? When I run the test.sh script in this PR inside the tmp directory, it passes as I'd expect
Exit code:   0
Done

🔴 - /var/folders/j4/fs11plcs72s2kkrtnyf7mgy80000gq/T/cody-benchmark-d6Ht8l
@@ -6 +6 @@ func main() {
-    f, err := os.◆
+    f, err := os.Open("data.txt")

┌──────────────────────────────────────────────┬───────────────────────────────────────────────────────────────────────────────┐
│                   (index)                    │                              sourcegraph.cody-ai                              │
├──────────────────────────────────────────────┼───────────────────────────────────────────────────────────────────────────────┤
│ ./test/benchmark/datasets/golang - open-file │ '🔴 - /var/folders/j4/fs11plcs72s2kkrtnyf7mgy80000gq/T/cody-benchmark-d6Ht8l' │
└──────────────────────────────────────────────┴───────────────────────────────────────────────────────────────────────────────┘

sourcegraph.cody-ai:
Pass: 0
Fail: 1
No Change: 0
Pass rate: 0.00%
Pass rate (excluding No Change): 0.00%

@slimsag Strange that this didn't pass, an exit code of 0 should definitely be a success. I'll make the test command spit out logs so we can debug this easier

@slimsag I've merged in some main changes from the branch and simplified the config of this test following your suggestion. Re-run and LMK what you think! Works nicely now:

... interesting the test.sh script exits with the success exit code 0 and passes.. but the test suite calls it a failure for some reason? Am I missing something?

@slimsag, I've been getting a similar error. For me, the reason was not running a Docker service.
  error: Error: Command failed: docker run --mount src="/var/folders/jp/bw890l7d4b549xjmgrb7c67h0000gn/T/cody-benchmark-eZBjKD",target=/app,type=bind cody-benchmark-harness ts-node test.ts
  docker: Cannot connect to the Docker daemon at unix:///var/run/docker.sock. Is the docker daemon running?.
  See 'docker run --help'.

      at ChildProcess.exithandler (node:child_process:419:12)
      at ChildProcess.emit (node:events:512:28)
      at maybeClose (node:internal/child_process:1098:16)
      at Socket.<anonymous> (node:internal/child_process:456:11)
      at Socket.emit (node:events:512:28)
      at Pipe.<anonymous> (node:net:334:12) {
    code: 125,
    killed: false,
    signal: null,
    cmd: 'docker run --mount src="/var/folders/jp/bw890l7d4b549xjmgrb7c67h0000gn/T/cody-benchmark-eZBjKD",target=/app,type=bind cody-benchmark-harness ts-node test.ts',
    stdout: '',
    stderr: 'docker: Cannot connect to the Docker daemon at unix:///var/run/docker.sock. Is the docker daemon running?.\n' +
      "See 'docker run --help'.\n"
  }

I could catch it by adding additional logging to https://github.com/sourcegraph/cody/blob/b77915e7181b8ad8fc53f26c0dc0e8ab166ebd60/vscode/test/benchmark/evaluate-test-case.ts#L20

@taras-yemets Yep definitely, I've added additional logs here for when the test errors. This should hopefully be easier to debug now!

@umpox Updated and added a few more tests, PTAL.
Currently the pass rate for these is not great but I think these are practical tests that reflect meaningful real-world use cases.. and won't rely on e.g. trained knowledge of existing APIs.
Pass: 1
Fail: 6
No Completion: 1
Average Time to Completion*: 1322ms
Median Time to Completion*: 1372ms
Pass Rate: 12.50%
Adjusted Pass Rate**: 14.29%

@slimsag This is interesting! This is the first dataset I've seen where there is a significant difference between us and Copilot
Cody default (Claude instant):

However, Cody with starcoder-16b...
  "cody.autocomplete.advanced.provider": "unstable-fireworks",
  "cody.autocomplete.advanced.model": "starcoder-16b",


Negative  targets

Hi, can I give SegGPT negative image targets to avoid confusion among similiar objects?
Thanks.

I have tried to add 2 images without targets and their 2 correspondent black images, but the model seems have not been improved. Even the same image in the test set was misrecognized by the model.


Training with GPUs

Hi, how can I use GPUs to train the model ? I noticed that TF_NEED_CUDA=0 in TF's configure, can I simply replace it with TF_NEED_CUDA=1 ? Why not use the official TF ?

I assume you are referring to the TF_NEED_CUDA in the Dockerfile. Yes, you may. We have only tested the evaluation externally on CPU, but it should work with GPU as well -- it possible some other Dockerfile changes need to be made for Docker image to see the GPU. To keep it broadly usable, we don't enable it by default. If you try it, we would happy to hear about your attempt.

@kbansal , I kind of figured out how to use multiple GPUs for training. The most important changes are (1) add a .bazelrc to deepmath root directory with content of "import %workspace%/tensorflow/.bazelrc", (2) change the version of tensorflow to r1.12 (because there is no compatible tensorflow-estimator version for your current TF version, when using tf.contrib.distribute.MirroredStrategy).
Currently, I am not using Docker. When I have time, I may give you a corresponding DockerFile.
I have some new questions: (1) I am running deepmath/deephol/train/ experiments.py. Am i right? (2) What is the batch size you actually use in the HOList paper?

1 - yes, that is the training code. We had not ourselves tried the training code outside of Google and provided it as reference only, so glad to hear you were able to use it.
2- I asked my colleagues, and their recollection is 16 or 32. But try as large a batch_size as you are able to according to memory available.


Remove the context menu from the iOS folder list

Generally iOS and iPadOS don't seem to use the context menu for sidebar management. We should remove the context menu and rely on the edit menu and swipe actions.

See #493.


Accessing the bitcoin wallet

Just wondering if the native bitcoin wallet is installed and accessible along with the RTL wallet?
If so, how is it accessed from the command line?
Both bitcoind and bitcoin-qt do not work.
Thanks for the assistance!

in the bitcoin config file /mnt/hdd/bitcoin/bitcoin.conf you need to remove or comment out this line:
disablewallet=1

Thank you very much, such a simple fix.
I must have missed that.


Psalm strategy with Voter::voteOnAttribute, NormalizerInterface::normalize, and similar methods.

From the Voter class
    /**
     * Perform a single access check operation on a given attribute, subject and token.
     * It is safe to assume that $attribute and $subject already passed the "supports()" method check.
     *
     * @param string $attribute
     * @param mixed  $subject
     *
     * @return bool
     */
    abstract protected function voteOnAttribute($attribute, $subject, TokenInterface $token);

Subject has a mixed type, but since there is a supports method, it's safe to restrict the $subject type.
With the following method supports
    public function supports($attribute, $subject)
    {
        if (self::CONTRACT_SHOW !== $attribute) {
            return false;
        }

        if (!$subject instanceof Contract) {
            return false;
        }

        return true;
    }

It should be ok to write
    /**
     * @param string         $attribute
     * @param Contract       $subject
     * @param TokenInterface $token
     *
     * @return bool
     */
    public function voteOnAttribute($attribute, $subject, TokenInterface $token)

Without getting a MoreSpecificImplementedParamType error.
Same for others classes with a supports method.
Is there a way to avoid the error ? Does a stub or something like this could be added to this plugin ? Or the only solution is to use psalm-suppress ?

Hi @seferov ! Any advice/idea about this situation ?
What would be the possible PR to make in this repository ?

@VincentLanglet thanks for reporting. I am not sure it is doable with stubs but I think it is possible handle with plugin hooks

I'm new to this @seferov, could you help me ?
I assume that the rule would be "If the class extends Symfony\Component\Security\Core\Authorization\Voter\Voter, it automatically add a psalm-suppress MoreSpecificImplementedParamType to the voteOnAttribute".
Then the same could be done if the class implements NormalizerInterface...

@VincentLanglet exactly. It would be hard to infer type of subject from supports method and assign type to the subject on voteOnAttribute. Just suppressing MoreSpecificImplementedParamType on voteOnAttribute is ok I think. To do so I added voter stub https://github.com/psalm/psalm-plugin-symfony/pull/127

@VincentLanglet exactly. It would be hard to infer type of subject from supports method and assign type to the subject on voteOnAttribute. Just suppressing MoreSpecificImplementedParamType on voteOnAttribute is ok I think.

I agree

To do so I added voter stub #127

Oh didn't know a stub could have a psalm suppress.
Thanks for your PR, it gives me a great example of how to contribute.
There is also:

NormalizerInterface::normalize (since there is a supportsNormalization method).
DenormalizerInterface::denormalize (since there is a supportsDenormalization method).

But I'm not sure if I should do the contribution.
When I look at the Normalizer provided by Symfony they are checking again the data even if it's done in the supportNormalization method.
class JsonSerializableNormalizer extends AbstractNormalizer
{
    /**
     * {@inheritdoc}
     */
    public function normalize($object, $format = null, array $context = [])
    {
        if ($this->isCircularReference($object, $context)) {
            return $this->handleCircularReference($object);
        }

        if (!$object instanceof \JsonSerializable) {
            throw new InvalidArgumentException(sprintf('The object must implement "%s".', \JsonSerializable::class));
        }

        if (!$this->serializer instanceof NormalizerInterface) {
            throw new LogicException('Cannot normalize object because injected serializer is not a normalizer.');
        }

        return $this->serializer->normalize($object->jsonSerialize(), $format, $context);
    }

    /**
     * {@inheritdoc}
     */
    public function supportsNormalization($data, $format = null)
    {
        return $data instanceof \JsonSerializable;
    }

Maybe it's because the method is public. (Voter::voteOnAttribute was protected instead)
So I think your PR may be enough to close the issue.


Update README with latest upstream status

The README currently says:

The plan is to backport the module and all the necessary patches into the next 2.4.x release of Apache httpd. That will be at least revision 2.4.30. And I will push for it to come out this year. Time will tell.

Is this still accurate? There are some hints it might not be.

Thanks!

Thanks for noticing! Information rots so fast... ;-)


Ensure StreamField panel validation errors are visible

...by rendering panels as expanded, regardless of the collapse setting specified by the Wagtail implementor.
Fixes #8990 partially (for StreamField panels only).

[x] Do the tests still pass?
[x] Does the code comply with the style guide?
[x] For front-end changes: Did you test on all of Wagtail’s supported environments?

Developed and tested on Chrome v114 for macOS but should be browser independent.
Open question: does this need tests? If so, could you please point be to the right place to add a test.
Testing instructions:
This change is a relatively small fix and can be tested by making the following change to blocks.py in bakerydemo to mark a StreamField as collapsed by default, creating the exact situation this PR addresses.
- RecipeStreamBlock()
+ RecipeStreamBlock(collapsed=True)

After making the change, the StreamBlock should be collapsed by default when visiting a recipe page in the bakerydemo Wagtail admin.

Go to a recipe page
Add new heading block to the page's body StreamField. Do not fill the required heading text!
Save the page, this should cause a validation error

With this fix applied
The heading block we just added should be expanded because it has an error. All other StreamBlocks in the body should be collapsed per the collapsed=True default set on the RecipeStreamBlock. See screenshot attached below.

Original situation (without this fix)
The heading block we just added appears collapsed and it is not clear where the validation error is on the page.

Merged in fef8c4ac3c40f918758b6a08bf5523cf85293443