Threshold to say that it's the same person

@michuanhaohao Can you tell me the threshold value i.e aligned distance  to set,  to say thats its same person or different person

I always choose 0.7. If can statistics all distance of the positive and negative pairs.

but setting 0.7 bring many false positives.
setting 0.7, in 50% of cases, its says that its the same person. But actually they are different. I use aligned distance.
Is there any metric or advice you would give to reduce false postives, when setting the aligned distance threshold to 0.7

Mark. Have the same question.

is there a function in the code that calculate the average of the local distances? I'm thinking of taking the average of the diagonal and take 0.5 as threshold...any suggestion?

@BabuTango Which value are using now ?

@gregorito  Did you find the solution?


Multicoretests-CI bot for ocaml/ocaml developers

A possible end-goal of ocaml-multicore/multicoretests is for it to become a test-suite used by OCaml compiler developers for checking PRs. One way to do this would be through a bot attached to the ocaml/ocaml repo that can be invoked to run the expensive test-suite when it is deemed appropriate, in the same way that the Coq bot works (coq/bot).
This could use parts of Multicoretests-CI for the infrastructure that runs the tests on multiple platforms.

Cc to @shakthimaan as he is preparing work on a bot for https://github.com/ocaml-bench/sandmark
Ideally, we should

offer the ocaml compiler devs just one bot with a generic name, say ocamlbot, rather than have them memorize 3 different ones, and
make it as convenient as possible for them to use.

One possibility is to trigger actions by tagging the bot (and accepting sub-commands): @ocamlbot sandmark (or benchmark), @ocamlbot multicoretests (or stresstest`).
Another possibility is to trigger actions by recognizing labels (with similar names to the above).


Chromium based browser can't register the gsm

Chromium based browser(such as Samsung Internet Browser, Chrome Beta, Chrome, etc.) can't register the gsm, So It can't send push notification from sites(such as twitter, facebook, etc.)
plus, Samsung Internet Browser is too slow after installing microg..

#737


Allow use of venv rather than virtualenv

How would this feature be useful?
Some projects (for example, pipx) have test suites that cannot run under virtualenv, but require venv. Being able to choose whether to use virtualenv or venv would allow using nox to run the test suite.
Describe the solution you'd like
The simplest solution I can think of is to use a parameter to nox.session, something like @nox.session(stdlib_venv=True). But honestly, it doesn't matter much to me how it gets specified.
Describe alternatives you've considered
I can't really think of a good alternative, other than "use something else" :-( Ideally, virtualenv and venv would be sufficiently compatible that it doesn't matter which you use, but that simply doesn't seem to be the case.

nox is python3 only, so I guess we can just replace virtualenv?

No, unfortunately we still need it to create virtual environments for 2.7
sessions.
On Sun, May 26, 2019, 1:22 PM Santos Gallegos<EMAIL_ADDRESS>wrote:

nox is python3 only, so I guess we can just replace virtualenv?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/theacodes/nox/issues/199?email_source=notifications&email_token=AAB5I46H25V7ILOSQD2HAP3PXLWQLA5CNFSM4HPXJLZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWIMYOA#issuecomment-496028728,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAB5I44CA4YHTFHG5S4VEYLPXLWQLANCNFSM4HPXJLZQ
.

... and there are things that only work in virtualenv (there are some nasty interactions if, for example, you create a venv from within a virtualenv). So I definitely think it's something where the user needs to be able to choose. Of course, as someone who wants to ditch Python 2.7 ASAP, my personal preference would be to go venv by default with an option to use virtualenv, but that would be backward incompatible, so I'm fine with venv being the one that needs to be opt-in.

Thinking of future-proofing this for conda, pyenv, or pipenv support as well (see #183 #4 #159) we should probably just extend the pattern we have with ProcessEnv and VirtualEnv to include Venv, Conda and PyEnvbackends as well. We could abstract a bit more common code of the currentVirtualEnv` class to make it the base for these new backends.
Session decorator wise, I think it makes the most sense to have something like:
@nox.session(py="3.7", venv_backend="venv")

Thoughts?

I like the idea of having different backends, shouldn't be this a global option that could be overridden by each session?

I think per session alone makes the most sense, but happy to hear reasoning
for another global opt.
On Mon, May 27, 2019, 5:22 PM Santos Gallegos<EMAIL_ADDRESS>wrote:

I like the idea of having different backends, shouldn't be this a global
option that could be overridden by each session?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/theacodes/nox/issues/199?email_source=notifications&email_token=AAB5I4ZJIG3BHYEUQUAVHB3PXR3MPA5CNFSM4HPXJLZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWKVFLY#issuecomment-496325295,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAB5I46AHOQDKS5JUV4JUOTPXR3MPANCNFSM4HPXJLZQ
.

I'm trying to migrate pipx's test suite to nox right now and am running into this issue.
The problem is the pip executable is pointing to nox's rather than one that pipx created with venv:
>> ll /tmp/pipx_tests__fc881ve/subdir/pipxhome/.cache/3d7e26ae000b6c1/bin/python
lrwxrwxrwx 1 csmith csmith 47 Jul 28 08:42 /tmp/pipx_tests__fc881ve/subdir/pipxhome/.cache/3d7e26ae000b6c1/bin/python -> /home/csmith/git/pipx/.nox/unittests/bin/python

Is there a workaround available in nox?

To my surprise, I changed
        cmd = [sys.executable, "-m", "virtualenv", self.location]

to
        cmd = [sys.executable, "-m", "venv", self.location]

and nox still worked. The pipx tests then passed with nox. I'll see if I can put together a PR.

I hacked something together, but I'll wait for https://github.com/theacodes/nox/pull/217/ to land before adding the venv backend.

Just merged #217, would love to see your changes! Let me know if you need help.

I've assigned this issue to you, @cs01, but definitely holler if you're not okay with that.

No that's perfect. I think I am in a good spot to be assigned this since I got it working with pipx.


Implement conditional foreign key generation

For now, support is restricted to MySQL and PostgreSQL, while all other
DBMS default to the old behavior.
Ref #831
Signed-off-by: Stephan Peijnik<EMAIL_ADDRESS>

Lets think about this in the v1.0 release.
If we are going to support this, it has to been supported by all dialects, or will be thought as a bug by people.


Accept single exception class as retry_on_exception value

It's slight and minor improvement to appscale.common.retrying module.

https://ocd.appscale.com:8080/job/Daily Build/5659/

Demo:
New test unit test is passed in build:
test_exception_custom_filter (test.unit.test_retrying.TestRetry) ... ok
test_exception_filter (test.unit.test_retrying.TestRetry) ... ok
test_exception_list_filter (test.unit.test_retrying.TestRetry) ... ok
test_no_errors (test.unit.test_retrying.TestRetry) ... ok


[Feature request] Build operator as a test behavior: TestBehavior.BUILD

In Cosmos 1.4, the "build" operator was added.
This is not currently being used in the DbtTaskGroup / DbtDag API. But there is one use case that can be supported: tests! You can run tests and a model in a single dbt invocation using dbt build.
Why would someone want this? A big reason is to reduce the time of DAG runs. Splitting out tests into two Airflow tasks-- one for the node and one for the test-- can slow things down for users who are facing huge bottlenecks in spinning up nodes, e.g. they are not using partial parsing.
This is probably not a simple change because it most likely requires changing some internal assumptions regarding how Cosmos parses DAGs.
I'm very open to someone else taking on this issue, too.

Hi @dwreeves , I like your proposal.
What do you think we introduced this as TestBehaviour.BUILD? I believe this is a relatively small change.


[Update Checkout] Print hashes at the end of the successful clone/update.

Support dump-hashes with skipped repos
Remove duplicated code from dump hashes

@swift-ci smoke test and merge

cc/ @benlangmuir

@shahmishal why did you remove the code for dumping this in json form? That allows one to dump the current state into a .json file and feed it into update-checkout on a different machine and reproduce the exact set of hashes! If you want to change the meaning of dump-hashes, I am fine with that, but please add the json functionality back under a different option. This is another reason why we need update-checkout tests.

Nm. I read the commit wrong. Sorry for the noise!


The Webhook example does not work as shown for the Test Pane

The JSON string used isn't valid JSON and would fail during serialization.  Additionally, the workbook script would take in the JSON as a string object instead of a PSObject.  The solution would be to:

Cast $Webhookdata as a PSObject
Remove the single quotes around the array

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

ID: 7a6394c7-9bef-b8f8-ffd6-9d9d8e2daa07
Version Independent ID: 5ffa20a2-436c-2726-dc57-9d3b49f9ca39
Content: Starting an Azure Automation runbook with a webhook
Content Source: articles/automation/automation-webhooks.md
Service: automation
Product: unspecified
GitHub Login: @georgewallace
Microsoft Alias: gwallace

@Gemakk Thanks for bringing this to our attention. Your comment has been shared with the content owner for further review.

@Gemakk I am working on changes to the article to address this issue and others.. It will be live later today.. I will reply when I have the updates live.

@Gemakk, the updates are now live.  Thank you for the feedback. The examples were updated to make the test pane work but while perserving the formatting that truly comes from a webhook.  #please-close

Thanks!


StanDroid tries to steal clipboard

Right after starting the app, standroid has pasted my clipboard (Luckily, it was just a random sentence I've copied from wikipedia).
I've already uninstalled it because why not, but I'd like to know the reason behind this.

This is probably due to the function that automatically retrieves the video ID from the clipboard and pastes it into the search field.
https://github.com/kusamaru/StanDroid/blob/5c58707eecc7a994fbd4bc900b22106697bf3806/app/src/main/java/com/kusamaru/standroid/MainActivity.kt#L346C1-L369C6
Since this feature may not be recommended for security reasons at this time, I will add a warning and change the specification to make it an opt-in method.


frontend-search: add Angular 2 context

Problem
As described in #7802, the angularjs URL for the frontend-search zsh plugin was outdated. An update was needed to point to the new URL when the user runs "frontend angularjs ...".
What I did
I updated the README.md of the frontend-search plugin to include the new URL for angularjs, and made the appropriate change in frontend-search.plugin.zsh to point to the new URL 'https://angular.io/?search='. I also updated the code that adds a + between the search terms when appending them to the URL to be encoded (so + now becomes %20). I did this because the new search URL for angular drops the search terms in the angular search bar with the + included, instead of a space. I verified that this does not break any of the existing searches we have.
What this fixes
Fixes #7802

I'd keep the old one and add a new one called angular. Angular 1.x is still supported albeit in LTS, so there may be users that still use it. This requires adding angular to the list of available context in the both help messages, and also modify the _frontend-search.sh completion and the README.
I can help out if you don't know how to do some of these things.
+1 to the plus to %20 change.

@mcornella Thank you for the feedback. I have made the changes you suggested. Do you have any additional suggestions?

That was perfect, thanks for this!

That was perfect, thanks for this!


Consider reexporting esp-idf-sys in HAL and SVC as a sys module (ditto for a hal module in SVC)

We need to re-export EspError from esp-idf-sys so library crates that depend on esp-idf-hal don't have tp specify a version of esp-idf-sys. (This is an unfortunately Cargo limitation.)
See https://github.com/esp-rs/esp-idf-sys/issues/225

But would it be just EspError? I recall TickType_t as well? And then there is a lot of pub stuff from esp-idf-sys used in esp-idf-hal in the form of From trait implementations. While - for the latter - I don't expect users to reach out to the esp-idf-sys types, who knows?
Maybe we should just export all of esp-idf-sys as esp_idf_hal::sys?
But then, what about esp-idf-svc which depends on both esp-idf-sys and esp-idf-hal?
Should it re-export esp-idf-sys as esp_idf_svc::sys and esp-idf-hal as esp_idf_svc::hal?

While I'm somehow OK with ^^^ (re-exporting the crates completely in sys / hal modules), let's not forget that the original root cause of the poster's problems is that he simply cannot build esp-idf-sys on his machine. And that problem won't magically disappear by doing re-exports.

Agreed that it won't fix all of the poster's original problems. I'm just recalling similar issues I had over with Http, Hyper, and Warp, where I was constantly getting warnings from rustc that a http::Request in my crate is not the same as the http::Request that Hyper wanted, and Warp wanted yet another http::Request.

Thinking about this some more... for now I want to be conservative, and just expose EspError (at the root) because it is part of the public interface of esp-idf-hal. We have a lot of code that returns Result<T, EspError>, and it'd be nice to not have to grab that second type from esp-idf-sys.
If there are other types similarly situated, we could use those, too, but I can't think of any that are exposed via the API. The only one that has user implications (not necessarily needed in APIs) that I can think of is CONFIG_FREERTOS_HZ (which I would rename to SYSTEM_TICKS_PER_SECOND).

Thinking about this some more... for now I want to be conservative, and just expose EspError (at the root) because it is part of the public interface of esp-idf-hal. We have a lot of code that returns Result<T, EspError>, and it'd be nice to not have to grab that second type from esp-idf-sys.
If there are other types similarly situated, we could use those, too, but I can't think of any that are exposed via the API. The only one that has user implications (not necessarily needed in APIs) that I can think of is CONFIG_FREERTOS_HZ (which I would rename to SYSTEM_TICKS_PER_SECOND).

But what I'm saying is that a lot of esp-idf-sys types are  already part of the public interface of esp-idf-hal and esp-idf-svc. Case in point from just 3 minutes ago. And there are more - as in all peripherals also expose the original esp-idf-sys types.
So really - it should be all or nothing - either we expose all in a sys submodule (BTW - this is what Embassy HALs seem to do for their PAC crates which are roughly equivalent to our sys crate), or we do nothing.

I'd feel a bit better if we could only do this for types and not functions, but I imagine that's not terribly feasible.
At least all of the functions are marked unsafe, though.

Raw pointers (because for the most part this is what the esp-idf-svc handles are and also what we expose) are just as safe/unsafe as unsafe functions.
Both are completely safe until you call them (unsafe functions) or try to dereference them (raw pointers).
So yeah, user would still be protected, as dealing with these types and functions would require an unsafe {} block in both cases.


Images Inside Internal Storage Picture folder - Android

Issue Description
During camera capture images are saved in the cache folder but there are also images saved inside the Picture folder inside internal storage of an Android mobile which are Bin files once capture is done. But those files are save just inside Picture folder and populating the picture folder. How do I save it inside it's own folder on the picture folder similar to the folders below it once I click camera capture?
Image sample below:

Environment

react-native-camera-kit": "^11.2.1"
react-native": "0.63.4"
Platform: Android
Device info (Simulator/Device? OS version? Debug/Release?): Simulator/Device, OS ver 9/10

Found a solution by installing the latest version 12.0.0 and using React native fs readDir.


Mystery Text Discussion: mb.txt

Post your screenshots and discuss your findings about mb.txt here!

While working with AntConc, I came across many references to whales and eventually the name Ishmael. After that, I knew exactly what text I was working with. I had a sneaking suspicion with all the whale talk. Whale is the first "non-filler" word at 22nd most popular word with 1228 results. Ironically, Moby Dick only appears 83 times, being the 102nd most popular 2gram. Granted, most of the ones before it aren't really much of anything. But "the whale" does show up 440 times so maybe they're not all "nothing" 2grams.

N-gram sizes 1,2,3,4 and 5 give me frequencys above 5. N-gram sizes 1,2,3,4 and 5 gives me frequency counts in the double digits. “the sperm whale” gets repeated 116 times and “of the whale” gets repeated 110 times.
Phrases that occur 5-10 times are “out of site of land” and “the bottom of the sea. The higher the n-gram size, the lower the frequency. Basic words like “the” and “of” are the most frequent so they show up larger on voyant.


Updated logic in isPrime to stay consistent

Comment says "from 2 to value / 2" however the code uses a loop that goes all of the way up to value.  I updated the logic to be more consistent with the comment above it.

It is possible to further improve the prime test from 2 to value**0.5 (square root of value). See this example on wikipedia.
There are a lot of other options, but the simplest would be to test first by 2, then test for i in range(3,int(value**0.5)+1,2), i.e. for 3, 5, 6, 7 up to ceil(sqrt(value)).

Regarding your points about square roots, thats true.  In cases where I made something to calculate primes (usually sequentially), I would actually check if its multiple by only the prime numbers calculated before it.  Good note about using floor division too.

@martinbra @alecgirman
I tried both of your ideas, and they didn't work. I did understand the concept behind them though. Do you think for i in range(2, int((value / 2))+1): is what you guys were talking about?

Sorry, i'm on my phone, so the formatting will be a little odd.
@Simple2006 about your commit, yes. You did the right thing. Code and comments should be equivalent. But there's a catch with the range(start,stop,step) function. It only accepts integer numbers, and doesn't evaluate the last one, i.e. range(3,9,2) will evaluate 3, 5 and 7.
If we use the square root upper limit that's what happens for 49...
for i in range(3, 49**0.5, 2)
for i in range(3, 7.0, 2)
So that's why we need the int so that:
for i in range(3, int(49**0.5), 2) evaluates to
for i in range(3, 7 , 2)
But then it is only evaluate to 5 (since range skips the last int). So that's why we need the +1 here:
for i in range(3, int(value**0.5)+1, 2)
So the range is evaluated to range(3,8,2)... => 3,5,7 and than the 7 is present inside the loop.
Got it?

@alecgirman and that's why @ghost commit wont work. There's a +1 missing.
That's is also why we write some test functions with known results to see if our functions are behaving as expected after some changes. There's a lot to talk about it, like, how to be sure our tests cover every possible scenario?

@alecgirman Thanks. I think I've understood what you said and fixed it. I will be closing this. If I understood you wrong and the issue persists, please let me know.

Instead of (value / 2), I think @martinbra is referring to using (value / 2) ** 0.5 instead (the square root of value / 2).  If this causes tests to fail, try adding a +1 to the end of it.

Instead of (value / 2), I think @martinbra is referring to using (value / 2) ** 0.5 instead (the square root of value / 2).  If this causes tests to fail, try adding a +1 to the end of it.

It was ( int(value** 0.5) )+1


Tweak heuristics to find the project root and "build root"

The project root means "this is a folder containing node_modules/bs-platform/{platform}/bsc.exe".
This path needs to be correct because it drives the formatter.
The "build root" represents the nearest directory containing a "bsconfig.json" file. "bsconfig.json" can be used to locate build artefacts and diagnostics.
The previous heuristics seemed to fail in the case users where using yarn workspaces.
Yarn workspaces seem to have the following layout:
/root
  /node_modules
  - package.json
  - yarn.lock
  /folder1
    /src
       - MyApp.res
    /node_modules
    - package.json
    - bsconfig.json

The compiler seems to be located in the node_modules under the root and not in the node_modules of folder1.
By searching for bscPartialPath instead of the nearest bsconfig.json, we can correctly determine the root.
However the bsconfig.json indicates the "build root" for the compiler artefacts and diagnostics in folder1.
Fixes https://github.com/rescript-lang/rescript-vscode/issues/10

I think that the logic you implemented might be a bit too naive, yarn will not hoist dependencies in a couple of cases:

You marked a dep as nohoist
There are different versions of dependencies.

I think a safer approach might be to:
find the project root by actually copying what yarn is doing, this looks like it:
https://github.com/square/find-yarn-workspace-root
but, we also need to check each folder on the way to see if bsc exists in its node_modules.
It's an expensive check, but we can do it only once when the extension starts.

Actually, now looking at the code. my mistake, this seems to behave exactly as expected!

I worked around this for the time being by adding this package.json config to the sub-project where I'm going to be using ReScript:
"workspaces": {
  "nohoist": ["bs-platform"]
},

and then running:
yarn --force

Alternatively you can do this globally by modifying the workspaces definition in your root package.json from something like this:
"workspaces": ["libs/*"],

to this:
"workspaces": {
  "packages": ["libs/*"],
  "nohoist": ["**/bs-platform"]
},

I think this PR is obsolete due to #59

I think this PR is obsolete due to #59

Superseded by https://github.com/rescript-lang/rescript-vscode/pull/59 and https://github.com/rescript-lang/rescript-vscode/commit/d75c6642f270a13981a4e833a3cb3c1ada7f698c

Superseded by https://github.com/rescript-lang/rescript-vscode/pull/59 and https://github.com/rescript-lang/rescript-vscode/commit/d75c6642f270a13981a4e833a3cb3c1ada7f698c


Bug Report: Null Object Reference at start

I build the 1.7 Kade Engine and all nice and dandy until the actual game loads to 21 and 'Null Object Reference' Appears, please help i've been dealing with this the last two days.

open powershell and type in lime test windows -debug
Hope this helps

follow the steps on this page follow the steps on this github page larsiusprime/polymod#39
till this is merged into source code, new users building their game will be met with a crash because of a new polymod thing.

massive lifesaver, thanks!

follow the steps on this page follow the steps on this github page larsiusprime/polymod#39
till this is merged into source code, new users building their game will be met with a crash because of a new polymod thing.


error when calling bind on gpu(0) or gpu(1)

I am using mxnet 0.9.4 python.
When I call bind on a mx.mod.Module object, I got an error.
This only happened when I set context to mx.gpu(1).
Tried mx.cpu(), or [mx.gpu(0), mx.gpu(1)], no error was raised.
Tried building mxnet from source with the latest version on Github, didn't fix this.
Tried rebooting the computer, didn't work.
Tried upgrade cuda, didn't work.
Tried mx.gpu(0), failed too.
I also tried uninstall mxnet 0.9.4 and install 0.9.3a via pip (mxnet-cu80), it turned out that the program stuck every time.
[19:05:45] /home/x/Github/mxnet/dmlc-core/include/dmlc/./logging.h:300: [19:05:45] /home/x/Github/mxnet/mshadow/mshadow/./stream_gpu-inl.h:98: Check failed: err == CUBLAS_STATUS_SUCCESS (1 vs. 0) Create cublas handle failed

Stack trace returned 5 entries:
[bt] (0) /home/x/anaconda2/lib/python2.7/site-packages/mxnet-0.9.4-py2.7.egg/mxnet/libmxnet.so(_ZN7mshadow9NewStreamINS_3gpuEEEPNS_6StreamIT_EEbb+0x1a72) [0x7f7f837bbe52]
[bt] (1) /home/x/anaconda2/lib/python2.7/site-packages/mxnet-0.9.4-py2.7.egg/mxnet/libmxnet.so(_ZNSt17_Function_handlerIFvvEZZN5mxnet6engine23ThreadedEnginePerDevice13PushToExecuteEPNS2_8OprBlockEbENKUlvE1_clEvEUlvE_E9_M_invokeERKSt9_Any_data+0xcf) [0x7f7f837e2d2f]
[bt] (2) /home/x/anaconda2/bin/../lib/libstdc++.so.6(+0xb6560) [0x7f7f73cad560]
[bt] (3) /lib/x86_64-linux-gnu/libpthread.so.0(+0x76ba) [0x7f7f9b4cf6ba]
[bt] (4) /lib/x86_64-linux-gnu/libc.so.6(clone+0x6d) [0x7f7f9aaf582d]

[19:05:45] /home/x/Github/mxnet/dmlc-core/include/dmlc/./logging.h:300: [19:05:45] /home/x/Github/mxnet/mshadow/mshadow/./stream_gpu-inl.h:98: Check failed: err == CUBLAS_STATUS_SUCCESS (1 vs. 0) Create cublas handle failed

Stack trace returned 5 entries:
[bt] (0) /home/x/anaconda2/lib/python2.7/site-packages/mxnet-0.9.4-py2.7.egg/mxnet/libmxnet.so(_ZN7mshadow9NewStreamINS_3gpuEEEPNS_6StreamIT_EEbb+0x1a72) [0x7f7f837bbe52]
[bt] (1) /home/x/anaconda2/lib/python2.7/site-packages/mxnet-0.9.4-py2.7.egg/mxnet/libmxnet.so(_ZNSt17_Function_handlerIFvvEZZN5mxnet6engine23ThreadedEnginePerDevice13PushToExecuteEPNS2_8OprBlockEbENKUlvE1_clEvEUlvE_E9_M_invokeERKSt9_Any_data+0xcf) [0x7f7f837e2d2f]
[bt] (2) /home/x/anaconda2/bin/../lib/libstdc++.so.6(+0xb6560) [0x7f7f73cad560]
[bt] (3) /lib/x86_64-linux-gnu/libpthread.so.0(+0x76ba) [0x7f7f9b4cf6ba]
[bt] (4) /lib/x86_64-linux-gnu/libc.so.6(clone+0x6d) [0x7f7f9aaf582d]

terminate called recursively
terminate called after throwing an instance of 'dmlc::Error'
  what():  [19:05:45] /home/x/Github/mxnet/mshadow/mshadow/./stream_gpu-inl.h:98: Check failed: err == CUBLAS_STATUS_SUCCESS (1 vs. 0) Create cublas handle failed

Stack trace returned 5 entries:
[bt] (0) /home/x/anaconda2/lib/python2.7/site-packages/mxnet-0.9.4-py2.7.egg/mxnet/libmxnet.so(_ZN7mshadow9NewStreamINS_3gpuEEEPNS_6StreamIT_EEbb+0x1a72) [0x7f7f837bbe52]
[bt] (1) /home/x/anaconda2/lib/python2.7/site-packages/mxnet-0.9.4-py2.7.egg/mxnet/libmxnet.so(_ZNSt17_Function_handlerIFvvEZZN5mxnet6engine23ThreadedEnginePerDevice13PushToExecuteEPNS2_8OprBlockEbENKUlvE1_clEvEUlvE_E9_M_invokeERKSt9_Any_data+0xcf) [0x7f7f837e2d2f]
[bt] (2) /home/x/anaconda2/bin/../lib/libstdc++.so.6(+0xb6560) [0x7f7f73cad560]
[bt] (3) /lib/x86_64-linux-gnu/libpthread.so.0(+0x76ba) [0x7f7f9b4cf6ba]
[bt] (4) /lib/x86_64-linux-gnu/libc.so.6(clone+0x6d) [0x7f7f9aaf582d]

somtime the error became Process finished with exit code 139 (interrupted by signal 11: SIGSEGV)

Hi, have  you solved your problem? I met the same error.How do you solve?
Thank you!

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


Support Python3

I've done several modifications to support Python 3. Most changes are minimal, the main difference is that now "expand_wildcard_actions" first checks for a list and then assumes it's a string without checking the type (basestring no longer exists in python3).

I just added one more commit: include the master_permissions file in the package when running "python setup.py install"

Oh shoot - didn't realize this project was getting PR's.  I'll start looking at these ASAP.

Manually ran the one test and it passed with python2/3. Merging.


Break up utils into subpackages with better names

Something called utils will mostly end up as a catch it all module of unrelated stuff.

current contents (which you can see are already pretty unrelated):

astro: star catalog access
datasets: access to external (supporting) data like ctapipe-extra or files in the CTAPIPE_SVC_PATH
CutFlow: class to apply and track cuts applied to events
fitshistogram: class to create and manage N-D histograms, with read/write to FITS
json2fits: convert JSON docs to FITS headers (not used?)
linalg: linear algebra helper functions
dynamic_class:  construct a class by name (I think). Doesn't seem to be used anymore...
table_interpolator, table_interpolator, template_network_interpolator, unstructured_interpolator: general ND interpolators used by ImPACT?
rgbtohex: color conversion for e.g. web guis using Bokeh

I think it was for serializing config files to FITS)

God help us

Had a look at linalg:

rotation_matrix_2d: should go into the coordinate trafos
length: not used
normalize and angle: verbatim copies of the same functions in  HillasReconstructor and not used anywhere else

I think this module should be removed and the function defined where they are used.


BUG: Example broken on main

@wmvanvliet I think #11080 broke tutorials/clinical/30_ecog.py:
https://app.circleci.com/pipelines/github/mne-tools/mne-python/16047/workflows/321560de-8d55-44f2-be22-e68382086eff/jobs/48440
Can you take a look?

Narrowed it down to MNE-BIDS trying to apply the montage, even though no fiducials are there yet. Before, the mri->head transform silently failed and channel positions were applied in MRI space (a situation we don't want). Then later in the example, this is fixed. I'll see if I can make MNE-BIDS be more clever about applying montages.


ListView SetNeedsDisplay won't update in terminal for windows

I want to update the contents of a Listview during runtime but the list is only updated in very unregular intervalls and not on every call to SetNeedsDisplay
          class Program {

		static bool Quit ()
		{
			var n = MessageBox.Query (50, 7, "Quit Demo", "Are you sure you want to quit this demo?", "Yes", "No");
			return n == 0;
		}
		static void Main (string [] args)
		{
			Console.OutputEncoding = System.Text.Encoding.Default;

			Application.UseSystemConsole = true;
			Application.Init ();
			Application.HeightAsBuffer = true;
			var top = Application.Top;

			MenuBar menu = new MenuBar ();
			var statusBar = new StatusBar (new StatusItem [] {
				new StatusItem(Key.CtrlMask | Key.Q, "~^Q~ Quit", () => {
					if(Quit()) Application.RequestStop();
				})
			});

			List<string> ls = new List<string> ();
			ls.Add ("Hello");
			ls.Add ("World");

			var _classListView = new ListView (ls) {
				X = 0,
				Y = 0,
				Width = Dim.Fill (0),
				Height = Dim.Fill (0),
				AllowsMarking = false,
				ColorScheme = Colors.TopLevel,
			};

			var _leftPane = new Window ("Messages") {
				X = 0,
				Y = 0,
				Width = Dim.Percent(10),
				Height = Dim.Fill (1), // for status bar
				CanFocus = false,
				ColorScheme = Colors.TopLevel,
			};

			_leftPane.Add (_classListView);

			var _dataView = new Window ("Message Detail") {
				X = Pos.Right(_leftPane),
				Y = 0,
				Width = Dim.Fill(1),
				Height = Dim.Percent (50),
				CanFocus = false,
				ColorScheme = Colors.TopLevel
			};

			var _msgView = new Window ("Send") {
				X = Pos.Right (_leftPane),
				Y = Pos.Bottom (_dataView),
				Width = Dim.Fill (1),
				Height = Dim.Fill (1),
				CanFocus = false,
				ColorScheme = Colors.TopLevel
			};

			top.Add (_leftPane, _dataView, _msgView);
			top.Add (menu, statusBar);

			int counter = 0;

			Timer tim = new Timer ();
			tim.Elapsed += (sender, eventargs) => {
				ls.Add (counter.ToString ());
				counter++;
				_classListView.SetNeedsDisplay ();
			};
			tim.Interval = 1000;
			tim.Start ();

			Application.Run (top);
			Application.Shutdown ();
		}
	}

This behaviour is unique to the command prompt and powershell inside the "Terminal for Windows" Application.
When i am starting the programm in the regular powershell or command prompt the list is updated but the height of the window seems to be wrong, so i can't see the bottom of the window. It looks like this

Sometimes when i start the programm the display is normal, but the list isn't updated as in the "Terminal for Windows" application.

Use the follow instead the System.Timers.Timer.
bool timer (MainLoop _)
{
	ls.Add (counter.ToString ());
	counter++;
	_classListView.SetNeedsDisplay ();
	return true;
}

Application.MainLoop.AddTimeout (TimeSpan.FromMilliseconds (1000), timer);

The Timer needs to invoke the Application.MainLoopSyncContext . So, with a single change, you also can use the follows two examples:
System.Timers.Timer tim = new System.Timers.Timer ();
tim.Elapsed += (sender, eventargs) => {
	ls.Add (counter.ToString ());
	counter++;
	Application.MainLoop.Invoke (() => _classListView.SetNeedsDisplay ());
};
tim.Interval = 1000;
tim.Start ();

System.Threading.Timer tim = new System.Threading.Timer (new TimerCallback ((_) => {
	ls.Add (counter.ToString ());
	counter++;
	Application.MainLoop.Invoke (() => _classListView.SetNeedsDisplay ());
}), null, 0, 1000);

When i am starting the programm in the regular powershell or command prompt the list is updated but the height of the window seems to be wrong, so i can't see the bottom of the window. It looks like this

Comment the Application.HeightAsBuffer = true; or sets to false and try, please. Sadly, with Windows Terminal the vertical scrollbar is always showed.

You don't need to use the _classListView.SetNeedsDisplay () inside the invoke. It could be an empty action. The important is notify the driver that there are data to being read. So, you may use the following examples:
	ls.Add (counter.ToString ());
	counter++;
	_classListView.SetNeedsDisplay ();
	Application.MainLoop.Invoke (new Action (() => { }));

or:
	ls.Add (counter.ToString ());
	counter++;
	_classListView.SetNeedsDisplay ();
	Application.MainLoop.Invoke (() => { });

or:
	ls.Add (counter.ToString ());
	counter++;
	_classListView.SetNeedsDisplay ();
	Application.MainLoop.Driver.Wakeup ();


Carthage compatibility

This issue is for tracking compatibility with Carthage.
OS X version is already compatible, but we need to make Bolts for iOS/tvOS also compile and work great with Carthage.

:+1:

+1

Bolts for iOS is currently not building with Carthage, generating the following error:
A shell task failed with exit code 1:
error: cannot parse the debug map for "<directory to project>/project/Carthage/Build/iOS/Bolts.framework/Bolts": The file was not recognized as a valid object file

I have not seen other frameworks generate a binary file (in this case the "Bolts" file that is the failure point) when building with Carthage. MacBolts also has this file but it appears to generate without problem while the file for iOS does not. I am not sure how to proceed. Any insight into this file's purpose would be appreciated..

I was incorrect in my previous comment regarding the binary file; some frameworks do appear to generate them as well when built with Carthage.

any news, guys?

Looks like this project is also blocking from using FBSDK

After some investigation, it comes down to the fact that the iOS target for Bolts is a static library. Carthage only supports dynamic libraries so I've submitted my pull request with what I believe are the appropriate changes.

👍🏻

This should now be possible in iOS as of 1ee6d30. I think everything else has to have a framework anyways so I want to say the issue is now closed but I'll test some of it out.

Using Bolts as a dynamic library, I had to set the build setting DYLIB_INSTALL_NAME_BASE to @rpath on the iOS-Dynamic target.   Otherwise everything would compile and link but there would be a runtime error.
With commit 1ee6d30 Carthage still tries to build and fail the static lib.  The only workaround I found so a far is to unshare the iOS target so Carthage builds only the dynamic library.

FWIW, I had success integrating into an iOS app with Carthage without doing anything special. I ran carthage update, linked against the Bolts.framework dynamic framework, and everything worked as expected.
Using:
Xcode 7.1.1
Carthage 0.11.0
Cartfile:
github "BoltsFramework/Bolts-iOS"

Cartfile.resolved:
github "BoltsFramework/Bolts-iOS" "1.5.0"

Note: This project had been previously configured to use other libs via Carthage, so I may have changed a build setting or something that allowed it to work without any further config.

Hey guys, just wanted to let you know that this is coming very very soon.
To test this right now - specify the following in your Cartfile:
github "BoltsFramework/Bolts-iOS" "nlutsenko.dynamic"

If at some later point it is going to complain that the branch no longer exists - then it probably was merged in master.

It's fully supported now, starting from 1.5.1 release.

I just arrived here looking for iOS support due to this warning from Carthage.
Dependency "Bolts-iOS" has no shared framework schemes for any of the platforms: iOS
I ran the following command to have it build for iOS.
carthage build --platform iOS
I will try out the 1.5.1 release.

Try 1.5.1 - I tested with 'master' before releasing - it built for all platforms.

Next I am going to try getting Parse to build. I see the following message from Carthage.
Building scheme "OCMock iOS" in Parse.xcworkspace

Be aware that Parse doesn't yet support Carthage, only Bolts does.
I am working on bringing it to Parse right now.

btw, I do see Bolts.framework and Bolts.framework.dSYM in Carthage/Build/iOS/ now. Thanks @nlutsenko!

@nlutsenko How long will the update for Parse take? I may just fork and make a change to get it done for my immediate needs and could send a PR if you want. I'll review what you did for Bolts and replicate the change.


Use innerText, not innerHTML, for select reading

Using innerHTML meant e.g. & was being shown as & in selected items. It also meant you couldn't search for e.g. & then space, you had to search for &. Switch to textContent || innerText (though I only put innerText in the commit message, who knows why).
Fixes #151.

Wasn't sure where best to add a test (or if just changing one of the country names in the wrapper.js test would have done :) ).

Tests added (one for hint, one for input after selection) :)

Great work! 👍

Published in v1.4.1!


feat(nodes): polymorphic field UI

What type of PR is this? (check all applicable)

[ ] Refactor
[x] Feature
[ ] Bug Fix
[ ] Optimization
[ ] Documentation Update
[ ] Community Node Submission

Description
feat(nodes): remove ui_type overrides for polymorphic fields
feat(ui): render input components for polymorphic fields
Polymorphic fields now render the appropriate input component for their base type.
For example, float polymorphics will render the number input box.
You no longer need to specify ui_type to force it to display.
TODO: The UI may break if a list is provided as the default value for a polymorphic field.
Related Tickets & Documents

Related Issue https://discord.com/channels/1020123559063990373/1130288930319761428/1151913978050641961

Should note that this means now that you can connect a collection to eg cfg_scale on denoise latents. Still works fine when setting the value directly.


Blocking exhausts Native threads

We have ran into an issue while load testing an implementation of our system that uses ZIO 1.0-RC4 where at the end of a long run we start failing because of OutOfMemoryErrors, which seem to report happening when trying to create a thread in the blocking thread pool.
Analysis seems to point to the blocking pool not reusing threads and us getting into a situation where the observed behavior of the Blocking pool seems to be equivalent to calling NewThread while under load.
I suggest 2 courses of action:

Provide a built-in way for users to create a Blocking instance that is just like the Blocking.Live instance, but with the ability to adjust the keepAliveTime, maximumPoolSize, and corePoolSize these are all options a user might want to drive from configuration for example.
Look into the default configuration of Blocking. Consider whether it actually does make sense to set the maxPoolSize to Int.MaxValue

We would like to provide a pleasant, safe out of the box experience that voids these types of issues. It seems like tweaking defaults to the blocking thread pool is warranted... /cc @kaishh

Note, default Blocking.Live does reuse threads – corePoolSize is the parameter that regulates thread reuse, not maxPoolSize – a finite max parameter with SynchronousQueue would just block the caller until a thread is available which is a bit different, there is a test for working caching too (although currently disabled, I just it ran it locally to confirm – https://github.com/scalaz/scalaz-zio/blob/6c7506d380a01a2a2ef6ff8227f4d891a504a917/core/jvm/src/test/scala/scalaz/zio/RTSSpec.scala#L1290)
I think just bumping keepAliveTime from 1 sec to 60 sec, like monix does, may ensure more frequent thread caching.
If you could try to make an example that results in the same behavior that would greatly help pinpoint what exactly is happening. It may be that somehow the threads are not returned back to the pool after the call finishes (or your parallelism level may be so high as to immediately exhaust OS limit, you've mentioned using a Queue to limit workers, but maybe you could try a Semaphore?)

So, the real cause of this error was generation of a new scheduled thread pool on new Clock.Live that was mistakenly used instead of a global Clock.Live:
Damian Reeves @DamianReeves 21:43
OMG!
This is stupid user error

Damian Reeves @DamianReeves 21:43
Yes
I have a call that did had this in it:
.provide(new Clock.Live())
That was directly in my processing flow

Damian Reeves @DamianReeves 21:44
I feel so silly

What we could do to prevent the same mistake is to make the Live default mixins all use globals instead of being generative. Don't know how I feel about that, that may make sense, but may be a step back. I remember Blocking.Live redirected to a global some time ago and that annoyed me – but currently it creates a new pool every time, although I may be misremembering completely.

Actually, the problem is more plainly stated as: Clock.Live, Scheduler.Live and Blocking.Live constructors are all not pure and this is the main cause of trip-up, had they been wrapped in IO, that would make it obvious that a new thread pool allocation happens. We can wrap all these constructors, although to allow cake pattern to work they'd probably need to be in CPS form...

Oops, I forgot Blocking.Live. Re-opening.

Fixed for Blocking.Live as well. Now repeated construction of modules does not allocate new pools.


Support Influxdb v0.9+

There are bunch of new features and API changes.

And apparently solves some data corruption issues related to clustering. Is there a time frame for this issue?

Hello,
Is there anyone working on this one ? Influxdb 0.9.3 is coming and 0.8 branch will not live for a long time.

@mvdan: Did you make any progress on this?

I'll try to have a PR by the end of the day.

Any news?


[DAPHNE-#768] isNan: elementwise check for NaN elements in a matrix - Draft PR

This Draft PR introduces the isNan function to the Daphne project, enabling element-wise NaN checks on matrices (see issue #768) . The function has been implemented and successfully tested in C++ unit tests.
However, there is an issue with invoking the isNan function from a Daphne script, which needs to be resolved.
Implemented Features

[X] Created an IsNanOp operation in DaphneIR as a subclass of EwUnaryOp (elementwise unary operation) (see src/ir/daphneir/DaphneOps.td).
[X] Created a DaphneDSL built-in function isNan and documented it (see src/parser/daphnedsl/DaphneDSLBuiltins.cpp and doc/DaphneDSL/Builtins.md).
[X] Created a DaphneLib function isNan for matrices and scalars and documented it (see src/api/python/daphne/operator/nodes/matrix.py, src/api/python/daphne/operator/nodes/scalar.py, and doc/DaphneLib/APIRef.md).
[X] Created a runtime kernel for isNan by extending the existing ewUnaryMat and ewUnarySca kernel with a new op code (see src/runtime/local/kernels/EwUnaryMat.h).
[X] Added unit test cases for the kernel and script-level test cases for DaphneDSL and DaphneLib.

Issues
The current implementation works in C++ unit tests but fails when invoked from a Daphne script with the following error:

bin/daphne test/api/cli/operations/isNan_1.daphne

[error]: Lowering pipeline error.RewriteToCallKernelOpPass failed with the following message [ no kernels registered for operation `ewIsNan` ]
    | Source file -> "/home/domin/daphne/test/api/cli/operations/isNan_1.daphne":1:6
    |
  1 | print(isNan(3));
    |       ^~~

PassManager failed module lowering, responsible IR written to module_fail.log.

This is the module_fail.log:
module {
  func.func @main() {
    %0 = "daphne.constant"() {value = 3 : si64} : () -> si64
    %1 = "daphne.constant"() {value = true} : () -> i1
    %2 = "daphne.constant"() {value = false} : () -> i1
    %3 = "daphne.constant"() {value = 0x7FF8000000000000 : f64} : () -> f64
    %4 = "daphne.constant"() {value =<PHONE_NUMBER>0080 : ui64} : () -> ui64
    %5 = "daphne.constant"() {value = 1 : index} : () -> index
    %6 = "daphne.constant"() {value = 6 : index} : () -> index
    %7 = "daphne.constant"() {value =<PHONE_NUMBER>1520 : ui64} : () -> ui64
    %8 = "daphne.constant"() {value =<PHONE_NUMBER>1392 : ui64} : () -> ui64
    %9 = "daphne.constant"() {value =<PHONE_NUMBER>46728 : ui64} : () -> ui64
    %c0_i32 = arith.constant 0 : i32
    %10 = "daphne.call_kernel"(%9, %8, %7, %c0_i32) {callee = "_createDaphneContext__DaphneContext__uint64_t__uint64_t__uint64_t"} : (ui64, ui64, ui64, i32) -> !daphne.DaphneContext
    %11 = "daphne.createDaphneContext"(%9, %8, %7) : (ui64, ui64, ui64) -> !daphne.DaphneContext
    %12 = "daphne.ewIsNan"(%0) : (si64) -> si64
    "daphne.print"(%12, %1, %2) : (si64, i1, i1) -> ()
    %13 = "daphne.ewIsNan"(%3) : (f64) -> f64
    "daphne.print"(%13, %1, %2) : (f64, i1, i1) -> ()
    %14 = "daphne.matrixConstant"(%4) : (ui64) -> !daphne.Matrix<6x1xf64>
    %15 = "daphne.reshape"(%14, %6, %5) : (!daphne.Matrix<6x1xf64>, index, index) -> !daphne.Matrix<6x1xf64>
    "daphne.decRef"(%14) : (!daphne.Matrix<6x1xf64>) -> ()
    %16 = "daphne.transpose"(%15) : (!daphne.Matrix<6x1xf64>) -> !daphne.Matrix<1x6xf64>
    "daphne.decRef"(%15) : (!daphne.Matrix<6x1xf64>) -> ()
    %17 = "daphne.ewIsNan"(%16) : (!daphne.Matrix<1x6xf64>) -> !daphne.Matrix<1x6xf64>
    "daphne.decRef"(%16) : (!daphne.Matrix<1x6xf64>) -> ()
    "daphne.print"(%17, %1, %2) : (!daphne.Matrix<1x6xf64>, i1, i1) -> ()
    "daphne.decRef"(%17) : (!daphne.Matrix<1x6xf64>) -> ()
    "daphne.destroyDaphneContext"() : () -> ()
    "daphne.return"() : () -> ()
  }
}

Next Steps and Feedback
I am currently investigating the kernel registration issue and ensuring proper integration. Any insights or suggestions on this matter would be greatly appreciated. Thank you for your time and assistance in reviewing this PR.

Hi @StoeckOverflow, thanks for this pull request!
The error message [ no kernels registered for operation `ewIsNan` ] indicates that DAPHNE looks for a kernel that is registered for the operation ewIsNan (case-sensitive). However, the kernel catalog (materialized in lib/catalog.json) contains kernels for the operation ewIsnan. The simplest solution is to change the mnemonic of the DaphneIR operation to ewIsnan in src/ir/daphneir/DaphneOps.td.

For some background: The catalog file is generated automatically by src/runtime/local/kernels/genKernelInst.py from the information in src/runtime/local/kernels.json. In the latter file, we specify the op codes of the elementwise unary operations to pre-compile like they are spelled in the respective C++ enum, i.e., in upper case letters. When deriving the DaphneIR operation names, genKernelInst.py (line 153) takes the first letter in upper case and all remaining letter in lower case, which makes sense for virtually all op codes.

Hi @pdamme, Thank you for your detailed explanation!
I have implemented the suggested change by modifying the mnemonic of the DaphneIR operation to ewIsnan in src/ir/daphneir/DaphneOps.td. I am pleased to report that this has resolved the issue. All tests have passed successfully, and the pull request is now ready for review.


Add imagepreview extension

Description

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

I suggest that you use an argument instead of using the searchbar which seems to cause some issues, also removing the back action which goes back to a loading list 🙂 Check this video
https://github.com/raycast/extensions/assets/38033107/3d2e909b-6187-4d2e-af5b-9438d34ccdd5


Script to scroll to the latest message.

Script that will scroll to the latest message of your transcript!

I feel like this is somewhat unnecessary and/or inconvenient for most purposes.
Most transcripts, afaik, are used when a channel is going to be deleted (like a ticket close event)
and when that happens, and you are looking back, usually it's for something in the middle or near the beginning, not the end.
if enough people think differently I'll merge it.
I might also refactor part of the code to use a templating engine like ejs or handlebars, and then I might make it an option to enable/disable automatic scrolling.

Open an issue if this is still wanted and I'll add it as an option.


Rework context

A minor breaking change in the way context objects are defined.
The file is now named _.context.ts instead of $.context.ts.
This change was made because putting a $ in a file name can be confusing in Mac/Linux/Unix systems.
Instead of exporting a default value, it the file should now export a class named Context
Old
class Context {
  // ...
}

export default new Context();

New
export class Context {
  // ...
}

In the future, the context class will be able to have a constructor that takes one argument, an instance of the parent context.
// paths/accounts/_context.ts
import { CounterfactContext } from "../../types.ts";

export class Context extends CounterfactContext {
  constructor(registry) {
    this.mainContext = registry.find("/");
    this.productsContext = registry.find("/products");
  }
  // ...
}

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.3%) to 86.296%

Totals

Change from base Build<PHONE_NUMBER>:
0.3%

Covered Lines:
2778

Relevant Lines:
3270

💛  - Coveralls

Thanks @dethell! So nice to have another pair of eyes on this stuff.


Fixing Contract.properties to handle empty options

Previous to this commit, calling properties without providing
an options Hash would result in an attempt to dup nil.
This fixes the issues and adds basic specs for Contract.properties.
Relates to #371.

:heart_eyes:


fix Issue 13337 - Invalid extern C++ namespace resolution

Issue 13337.
Store must occur after prefixing parent so outer namespace appears first in the component list and gets a chance to be substituted first.

Auto-merge toggled on


Remove unused ajax_validate_billing

1. Why is this change necessary?
Removes code that is unused.
The feature ajax_validate_billing got removed from SW4 to SW5 and only caused no errors because the naming of the field changed from 'register_billing_ustid' to 'register_billing_vatid' in https://github.com/shopware/shopware/blob/5.3/themes/Frontend/Bare/frontend/register/billing_fieldset.tpl#L46
2. What does this change do, exactly?
3. Describe each step to reproduce the issue or behaviour.
none
4. Please link to the relevant issues (if any).
none
5. Which documentation changes (if any) need to be made because of this PR?
none
6. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hi @nlubisch, thank you for your contribution! 💙
Sorry it took us so long to merge it. :(


Error on reload or direct enter url to the browser on Production

I'm trying out nextjs for a couple of weeks, and I have build a small web using Nextjs
Everything works fine on Dev environment,
However when it comes to Prod, I'm facing a lot of troubles
I read Nextjs docs and know that I have to run "next build" so I did and then I deploy my app to my droplet (512 MB, 1 core)
Live demo is here: http://tangerine.vn
On Dev env when I open "http://localhost:8080/collections/skincare" (by typing url to browser or navigate on main menu), the page works as I expected, it shows some products
But on Prod env, It's working only if I open the page by navigate on main menu, and when I open devtool (Chrome) to debug I got some weird message what I have no idea where is the root of the problem

Here is my server.js
// LIBRARIES
const express = require('express')
const next = require('next')

const dev = process.env.NODE_ENV !== 'production'
const app = next({ dev })
const handle = app.getRequestHandler()

// CONFIG
app.prepare().then(() => {
  // Start Express
  const server = express()
  /*
   * Search
   */
  server.get('/search/:code', (req, res) => {
    const actual = '/search'
    const params = {
      code: req.params.code
    }
    app.render(req, res, actual, params)
  })

  /*
   * Products by category
   */
  server.get('/collections/:category$', (req, res) => {
    const actual = '/collections'
    const params = {
      category: req.params.category
    }
    app.render(req, res, actual, params)
  })

  /*
   * Product Detail
   */
  server.get('/collections/:category/:productCode$', (req, res) => {
    const actual = '/detail'
    const params = {
      category: req.params.category,
      productCode: req.params.productCode,
    }
    app.render(req, res, actual, params)
  })

  /*
   * Home
   */
  server.get('/', (req, res) => {
    var ip = req.headers['x-real-ip'] || req.connection.remoteAddress;
    res.redirect('/home')
  })
  server.get('*', (req, res) => {
    return handle(req, res)
  })

  /*
   * Run app
   */
  server.listen(3000, (err) => {
    if (err) throw err
    console.log('>> Ready on http://localhost:3000')
  })

})
.catch((ex) => {
  console.error(">> EXCEPTION", ex.stack)
  process.exit(1)
})

Environment

Tech
Version

"express": "^4.16.0",

"next": "^3.2.2",

"react": "^15.6.2",

"webpack": "^3.8.1"

Can someone help me out. Thank you

I have the same problem

I hope the official gives a solution

app.js:36 I.default.find is not a function
TypeError: I.default.find is not a function
    at e.value (http://tangerine.vn/_next/3061f79a-93de-4935-b858-5805099db6f8/page/collections:9:597076)
    at E (http://tangerine.vn/_next/bf6ccc76a8084294eedca63be540d34c/app.js:2:23619)
    at e.r (http://tangerine.vn/_next/bf6ccc76a8084294eedca63be540d34c/app.js:2:26528)
    at e.value (http://tangerine.vn/_next/3061f79a-93de-4935-b858-5805099db6f8/page/collections:9:597407)
    at E (http://tangerine.vn/_next/bf6ccc76a8084294eedca63be540d34c/app.js:2:23619)
    at e.r (http://tangerine.vn/_next/bf6ccc76a8084294eedca63be540d34c/app.js:2:26528)
    at e.value (http://tangerine.vn/_next/3061f79a-93de-4935-b858-5805099db6f8/page/collections:9:597359)
    at E (http://tangerine.vn/_next/bf6ccc76a8084294eedca63be540d34c/app.js:2:23619)
    at e.r (http://tangerine.vn/_next/bf6ccc76a8084294eedca63be540d34c/app.js:2:26528)
    at t.value (http://tangerine.vn/_next/3061f79a-93de-4935-b858-5805099db6f8/page/collections:9:590672)

I think it should be an error in your application code. Could you share your code so we can help investigation? If for confidential reason you cannot share your code, could you make a sample repo that can reproduce your error?

I works perfectly fine if I use the dev config for it on production, but all pages will be loaded really slow due to browser has to download large file (~14M) :(


keystone_manage module fails in liberty

@retr0h, looks like the keystone library removed some modules in liberty, so the keystone_manage module is failing:
NOTIFIED: [keystone | Sync Keystone database] *********************************
failed: [dev-os-control-01] => {"failed": true, "parsed": false}
Traceback (most recent call last):
  File "<stdin>", line 32, in <module>
ImportError: No module named openstack.common

Also see: https://github.com/openstack-ansible-galaxy/openstack-keystone/blob/master/library/keystone_manage#L32
Looking for a fix and will hopefully have something soon.

Related to https://github.com/openstack-ansible/openstack-ansible-modules/pull/56


Shield of Fortification

https://www.d20pfsrd.com/magic/all-spells/s/shield-of-fortification/

Already in Expanded Content!


rc_event_queue

RENDERED
This is rather true RFC (request for comments), than specification, at least for now.
It is somewhat close to https://github.com/bevyengine/rfcs/pull/17 ... but "implementation" is different.

Thanks for creating this RFC - how events interact with 'asynchronous' systems is a real problem area and I'm glad to have more suggests in this space.
However there is a licensing issue which needs to be resolved before we can proceed - rc_event_queue currently does not have a license available.
Note for the author: Please provide a license for rc_event_queue.
In general, I want us to avoid introducing new dependencies not available under MIT/Apache2.0, so I would recommend you use this combination of licenses - this is what bevy itself is licensed under.
The easiest guide to doing so is C-permissive from the Rust API guidelines.
Please note that bevy chooses not to include a copyright notice line in our MIT license following the lead of the Rust project, although I have no opinion for or against it for rc_event_queue (although if you made me choose I'd err on the side of not including one).
For other readers: I should strongly advise against reading the code of rc_event_queue until this is resolved out of an abundance of caution. This also applies to the docs linked in the RFC, which from my cursory reading (of the RFC) seem central to explaining how the proposed change would work.

In general, I want us to avoid introducing new dependencies not available under MIT/Apache2.0, so I would recommend you use this combination of licenses - this is what bevy itself is licensed under.

What's a point of dual licensing? Isn't MIT more permissive than Apache2.0? Initially I was going to license just under MIT.

Please note that bevy chooses not to include a copyright notice line in our MIT license...

What is that mean?

What's a point of dual licensing? Isn't MIT more permissive than Apache2.0? Initially I was going to license just under MIT.

Apache-2.0 has extra rules regarding patents. IANAL, but I think it roughly prevents contributors from contributing code to which a patent they own applies without giving a patent grant to all users of the code. It is incompatible with GPL-2.0 according to some lawyers though, hence the dual license with MIT which is clearly compatible.

License added. If everything goes well - it should become cargo crate. So it can be just another dependency for project, like every else.

By the way, some cargo crates with MIT/APACHE2 have dependencies on MIT only crates (including bevy and rc_event_queue)... Is this ok?

https://law.stackexchange.com/questions/6081/can-i-bundle-mit-licensed-components-in-a-apache-2-0-licensed-project
tl;dr: yes

@alice-i-cecile You said something about that rc_event_queue could be useful for something UI specific. Did you mean Entity Events?
Cuurently, EventQueue must be Pinned, because Readers need to talk back when passing inter-chunk boundary, so its stored in heap, to have stable address.
But I think I can make it non-pinned, you'll just have to provide Reader &EventQueue on each read session. Also, I think, it is possible to built-in one-two small chunks into EventQueue, thus having some form of small_vec....

@alice-i-cecile You said something about that rc_event_queue could be useful for something UI specific. Did you mean Entity Events?

Ah right! Yes, per-entity events were particularly blocked on solving the questions around event lifespan, as I found they tended to have more unusual access patterns. And then in turn, I felt that storing events on entities might be quite useful for handling UI event dispatching.

Cuurently, EventQueue must be Pinned, because Readers need to talk back when passing inter-chunk boundary, so its stored in heap, to have stable address.

Interesting, that makes sense. FWIW, NonSend components should be quite feasible; we've just never had a pressing need for them before. The overhead of just doing that may be better than your non-pinned variant, but I'll leave that judgement call to you.

Main overhead with Arc is construction time, everything else is mostly non-observable, at least for current Bevy Events use-case.
But if you really plan to put EventQueue into Component and add/remove components dynamically, that overhead is a thing.
Considering small buffer optimization implementation, memory-wise it could be approx. 64byte + chunk size, non-heap (until fits small chunk), and you can move it around. As a price for that, you'll need to pass &EventQueue to get Reader::iter().
So I would say, realistically it would be 128-256 bytes on stack. Could that work for your cause?

So .... I thought, what about having some new EventsV2 alongside with the current one? This way - new events will not be a braking change....
Is that acceptable?

I would be hesitant to do that; this should work fine as an external crate if we want early usage feedback.
Keeping both in the engine leads to the sort of caveats and confusion you tend to see in e.g. Unity, where you have two very similar but subtly different tools for the job, whose usage is mixed throughout the code base.

FYI @tower120, we're mostly just blocked on "decision-making attention": there are a large number of other pressing ECS needs that have been getting more attention right now (https://github.com/bevyengine/bevy/discussions/2801) and Cart is focusing on rendering to get 0.6 released <3

I see... And API changes, like making EventReader::iter return Iterator, instead of DoubleEndedIterator, is out of the question. Right?


Store bitmap sizes along with bitmaps in the segment format

Absence of precomputed bitmap sizes makes implementation of features like #3878 and #3848 much less effective than it could be.

One thing to be careful of here is that 4 bytes per bitmap can add up quickly for high cardinality dimensions. Still a useful idea though.

@gianm we can use condensed integer encodings, where the highest bit of each byte is a continuation flag. Number 0-127 take just 1 byte.

yes, for sure that is a valid approach. Just wanting to make sure that this issue is considered. It may also be worth while to skip the size field for small bitmaps where computing size is fast. Odds are, high cardinality dimensions will generally have small bitmaps. I'm not sure though since I'm not sure of the speed difference between reading a vsize int and computing size for a small bitmap.


Add Handling of Schedule Zip Data

Add new ConfigType for zipped up schedule data
Create convert_gtfs dot py that will take a gtfs zip file, pull out
date time information, and generate pyarrow tables for each of the
different csv's inside.
add new s3 util that will allow us to read contents of dot zip files
from s3 using python's ZipFile module without downloading them.
update ingest to have a track for handling dot zip files and a track
for handling dot json dot gz files.
rename convert dot py to convert_gtfs_rt dot py
Add tests and test file for schedule gtfs work

Asana Task: https://app.asana.com/0/1202413225616901/1202183746758634/f

I think this all looks good for the approach we discussed last week. But I have been thinking about edge cases where this could break.
The main one would be if two schedule zip files happen to be issued on the same day. We are basically using they feed_start_date as a primary key for these data tables. It might be a case of over engineering, but I'm thinking we might want to generate and GUID for each zip file we process, add that as a column to each table we're writing and then just join everything together at a later date how we see fit.

It might be a case of over engineering, but I'm thinking we might want to generate and GUID for each zip file we process, add that as a column to each table we're writing and then just join everything together at a later date how we see fit.

This has happened before. Instead of a GUID, another option (which is also sortable) would be the Last-Modified date of the GTFS file.


Feature request: Arcana

I would like to be able to see the full information(arcana in the protocol) of the AST instead of just the simplified ones.

You could either make it configurable or make it such that pressing K on a node shows the arcana. I'm leaning towards the latter but it is more involved.


Create OIPC Redline for Ministry Sign Off.

• As an IAO analyst
• I want to be able to save an OIPC redline PDF with all redactions
• So the ministry coordinator or LSB can download the redline for Ministry Sign Off
Assumptions & Scope
What are the assumptions for this story?
• For release 9.0, the ministry will not have access to the Records Review app.
• The request must be in the Records Review state.
• Every page in the request package must have at least one of these flags applied: Partial Disclosure, Full Disclosure, Withheld in Full, Duplicate, Not Responsive, Duplicate or Non Responsive
• Packages with pages with no flag or the 'Consult' or 'In Progress' flag cannot be saved as a redline
• All redactions will be red boxes with the content visible.
• All records will be saved in one PDF document in Object Storage.
• And all Duplicate and Not Responsive flagged pages will be removed from the redline PDF
• The PDF document will include all redactions, applied with a red box around the visible content, with the section stamp.
• The PDF document will include NO comments, highlights or annotations.
• The PDF document will be viewable in the redaction app.
What is IN scope?
Saving the redline as PDF in Object Storage.
Create Records Package dropdown
What is NOT in scope?
Creating redaction layers and toggling between redaction layers is covered in #3704; Creating the Redline layer is done by default when Records are uploaded into the Records Log.
State changes.
Saved phased releases of the redline will be in future story.
Creating a new records package.
Acceptance Criteria
Scenario 1: Active "Create Response PDF" Dropdown
• GIVEN an analyst is on a request
• AND they have applied the OIPC Layer
• WHEN IAO analyst is in the redaction app viewing the records
• THEN there will be an active "Create Response PDF" dropdown
Scenario 2: Click on "Create Response PDF"  Dropdown
• GIVEN an IAO analyst is in the redaction app viewing the records
• AND they have applied the OIPC Layer
• WHEN IAO analyst clicks on the "Create Response PDF" Dropdown
• THEN the dropdown will open
• AND there will 3 options: 'Redline for Sign Off,' Redline for OIPC,  and 'Response Package for Applicant.'
Scenario 3: Inactive "Redline for Sign Off"
• GIVEN a records package is not in Records Review or Ministry Sign Off
• AND they have applied the OIPC Review Layer
• AND every page in the Records Package has one of these page flags applied: Partial Disclosure, Full Disclosure, Withheld in Full, Duplicate, Not Responsive
• WHEN the IAO analyst is clicks on the "Create Response PDF" Dropdown
• THEN there will be inactive and greyed out "Redline for Sign Off" option in the dropdown
Scenario 4: Active 'Redline for Sign Off'
• GIVEN a records package is in Records Review or Ministry Sign Off
• AND they have applied the OIPC Layer
• AND every page in the Records Package has one of these page flags applied: Partial Disclosure, Full Disclosure, Withheld in Full, Duplicate, Not Responsive
• WHEN the IAO analyst clicks on the "Create Response PDF" dropdown
• THEN the "Redline for Sign Off" will be clickable
• AND the 'Response for OIPC" and "Response Package for Applicant" will not be clickable
Scenario 5: Click "Redline for Sign Off" - Create and Cancel Options
• GIVEN the "Create Response PDF"  dropdown is open
• AND the request is in Records Review
• AND they have applied the OIPC Layer
• WHEN the IAO analyst clicks the active "Redline for Sign Off"
• THEN a modal will open with the message, "Are you sure want to create the redline PDF for ministry sign off.
• AND a "Create Redline PDF" and "Cancel" button
Scenario 7: Click "Cancel" button
• GIVEN an analyst is on the "Redline for Sign Off" modal
• AND they have applied the OIPC Layer
• WHEN the IAO analyst clicks the "Cancel" button
• THEN a modal will close
• AND the analyst will remain in the redaction app
• AND no redline PDF will be saved
Scenario 8: Click "Redline for Sign Off" - Create option
• GIVEN an analyst is on the "Redline for Sign Off" modal
• AND they have applied the OIPC Layer
• WHEN the IAO analyst clicks the "Create Redline PDF" button
• THEN a modal will close
• AND a green success toast will appear when the save is complete
• AND the analyst will remain in the redaction app
• AND the analyst will see the OIPC redline layer in the redaction app so they can review if needed
• AND one redline PDF with all records will be saved in Object Storage
• AND all pages flagged as Duplicate or Non Responsive pages will be removed from the redline
• AND the redline PDF will contain all redactions (red boxes around visible text with the section of the ACT stamped in the box)
• AND the redline PDF will contain no comments or annotations of any kind
Dependencies? What is the impact of this dependency? (If so, link dependency in the ticket, make it visible in a team´s backlog)
Validation Rules? (If yes, list here)
Design
@xxx - please link the Design here
Definition of Ready

[ ] Is there a well articulated User Story?
[ ] Is there Acceptance Criteria that covers all scenarios (happy/sad paths)?
[ ] If there is a user interface, is there a design?
[ ] Does the user story need user research/validation?
[ ] Does this User Story needs stakeholder approval?
[ ] Design / Solution accepted by Product Owner
[ ] Is this user story small enough to be completed in a Sprint? Should it be split?
[ ] Are the dependencies known/ understood? (technical, business, regulatory/policy)
[ ] Has the story been estimated?

Definition of Done

[ ] Passes developer unit tests
[ ] Passes peer code review
[ ] If there's a user interface, passes UX assurance
[ ] Passes QA of Acceptance Criteria with verification in Dev and Test
[ ] Confirm Test cases built and succeeding
[ ] No regression test failures
[ ] Test coverage acceptable by Product Owner
[ ] Ticket ready to be merged to master or story branch
[ ] Developer to list Config changes/ Update documents and designs
[ ] Can be demoed in Sprint Review
[ ] Tagged as part of a Release
[ ] Feature flagged if required
[ ] Change Management activities done?

@m-prodan @KyEggleston  Let me know if there are needed changes. This one is based on what is covered in #3437 Save redline PDF for Ministry Sign Off. I am unsure about the language about saving copies in object storage, etc., but I presume it would be the same...?

We should ask the devs if we can just adjust this story to just have the create redline for Ministry sign-off function available, ideally this should be very little effort and just leverage our existing Ministry sign-off redline creation.
cc @lmullane

Follow up notes:

Business to revise and refine ticket and ACS...
Workflow: OIPC selected as layer (shows all annotations/redactions and s.14) -> A new option under Create Response PDF is added to download this layer (without redactions to s.14) -> Once option selected, Download for OIPC Layer For Ministry Signoff is available for users in records tab. See link below for output:
https://bcgov-my.sharepoint.com/personal/lise_mino_gov_bc_ca/_layouts/15/onedrive.aspx?id=%2Fpersonal%2Flise_mino_gov_bc_ca%2FDocuments%2FMicrosoft Teams Chat Files%2FCurrent.pdf&parent=%2Fpersonal%2Flise_mino_gov_bc_ca%2FDocuments%2FMicrosoft Teams Chat Files&ga=1
Essentially this package is just a redline for signoff package. Potential to reuse code / current logic in place to create the package?

@JHarrietha-AOT Hey! Can you please see if this aligns with your designs?

Refinement Comments:
who can download this package? - Same as normal Redline for Signoff. Both IAO and Ministry can download

Should we call it "OIPC Redline for Signoff" to differentiate between an OIPC redline and the original redline?
@m-prodan, @liseandtea , @KyEggleston , @JHarrietha-AOT

@m-prodan @lmullane @KyEggleston @JHarrietha-AOT As it is the same process as sending a redline to the ministry for sign off and the review is happening on the OIPC layer, I don't think we need to have another option. Too many choices will be confusing I think.

Talked with @liseandtea to confirm functionality.
The "Redline for OIPC" button will create a redline from the OIPC layer with the s14 redactions applied.
The "Redline for Sign Off" button will create a redline from EITHER the OIPC layer or original Redline layer, whichever is currently selected. This confirmation needs to be added to the modal (eg. "Are you sure you want to create a redline from the OIPC (or Redline) layer?"

Download OIPC is the prefix for downloads produced from OIPC layer.


Order of variables causing undefined in IE11 scripts

Order of some of the variables is causing undefined to be created by IE11 script when used as described here:
https://github.com/patternfly/patternfly-next/wiki/IE11-Support
The variables that are being created as undefined are the following:
.pf-c-form__actions {
display: flex;
flex-wrap: wrap;
margin-top: undefined;
margin-right: undefined;
margin-bottom: undefined;
margin-left: undefined;
overflow: hidden; }
.pf-c-login__main-footer-links {
display: flex;
flex-wrap: wrap;
justify-content: center;
padding: 0.5rem 4rem 2rem 4rem;
margin-bottom: undefined; }
.pf-c-wizard__toggle {
position: relative;
z-index: 200;
display: flex;
justify-content: space-between;
width: 100%;
padding: 1.5rem 1rem 1.5rem calc(1rem + undefined + 0.5rem);
.pf-c-wizard__toggle-list {
position: relative;
display: flex;
flex-wrap: wrap;
align-items: baseline;
margin-right: 0.5rem;
margin-bottom: undefined;
list-style: none; }

fixed in https://github.com/patternfly/patternfly-next/pull/1871


https://webhint.io/scanner/ -> internal server error

I do not know, if this has anything to do with the code, or rather with the server configuration. But the online scanner says:" The page cannot be displayed because an internal server error has occurred."

Looks like the website is missing js-yaml and thus the nodejs server side can't run. Moving this issue to the  website repo (https://github.com/webhintio/webhint.io/issues/498)

Thanks a lot for reporting this @utrenkner!


build_stubbed broke when i upgrade to Rails 7.1.3

Description
When i try to upgrade to Rails 7.1 from Rails 6.0, i get some errors in my specs, i found for some reason when i use build_stubbed in validations, it don't work properly.
Reproduction Steps
I just runned my specs and for some reason this broke:
  context "validations" do
    it { is_expected.to(validate_presence_of(:product_id)) }

    it "validate uniqueness of product_id" do
      valid_seasonality_item = create(:seasonality_item)
      invalid_seasonality_item = build_stubbed(:seasonality_item,
        product: valid_seasonality_item.product,
        seasonality: valid_seasonality_item.seasonality,
      )
      invalid_seasonality_item.valid?
      expect(invalid_seasonality_item.errors&.first&.attribute).to(eq(:product_id))
    end
  end

idk, for some reason this dont work, is expected to the invalid_seasonality_item to be invalid, but, its valid in this tests, so, i just changed the build_stubbed for build and thats works
Expected behavior
I want some response about what happened with build_stubbed and a solution for that
Actual behavior
At the moment, my specs with build_stubbed are broke.
System configuration
factory_bot version:  "factory_bot_rails", "~> 6.4.0"
rails version:  "rails", "~> 7.1.3"
ruby version: ruby 3.3.0

We experience the same. After updating Rails from 7.0 to 7.1 it seems that uniqueness validations of models instantiated with build_stubbed are passing even when they should not.
Example:
Consider the following spec, asserting that users with non-unique usernames are invalid:
it 'prevents the creation of multiple users with the same username' do
  create(:user, username: 'someone')

  duplicate = build_stubbed(:user, username: 'someone')
  expect(duplicate).not_to be_valid
end

The spec used to pass in Rails 7.0 (assuming that User validates uniqueness of username), but is breaking with Rails 7.1, because duplicate.valid? mistakenly returns true even if it should return false due to non-unique username. This only happens when using build_stubbed, and instead works correctly when using build.
As far as I can see, build_stubbed should work just like build when it comes to uniqueness validations.

Thx for the reply @lucaong, we ended up just replacing it with the build

Reasoning on what FactoryBot could possibly do to make this issue better, my view is that there is the need for a different strategy than build and build_stubbed.
My (erroneous) mental model of build_stubbed was that it works the same as build (instantiating a new model without persisting it), just stubbing all associations to avoid triggering a lot of database inserts. That's useful for many tests where persisting the model or creating real associations is not needed. Instead, build_stubbed is much more similar to create: it does not actually save the object to the database, but it does redefine methods like persisted? and new_record? to make it look like the model was saved in the database. In a way, I feel that a more appropriate name for it would have been create_stubbed, as the resulting object looks like a persisted one.
There is the need for a currently missing method that builds a new model stubbing all the associations, but without persisting not pretending to persist. Basically, a method doing the same as build, but stubbing the associations to make the test faster (which is what I initially thought build_stubbed was doing). The tests mentioned in this thread are one example where such method would be useful.
In my ideal scenario, FactoryBot would provide these methods:

build - instantiates a model using a factory, without persisting it
build_stubbed - instantiates a model using a factory, and stubbing associations for performance, without persisting it and without faking persistence like the current implementation does
create - creates and persists a model using a factory
create_stubbed - instantiates a model using a factory, stubbing associations, and stubbing persisted? and other similar methods to make it look like the model was persisted (what build_stubbed does currently)

This would involve changing the current documented behavior of build_stubbed, which introduces a breaking change. The alternative would be to avoid a breaking change, keep the current default behavior of build_stubbed (accepting it's slightly misleading name), and add a new option to skip stubbing persisted?, new_record?, and other persistence methods. Something like build_stubbed(:user, stub_persistence: false).
What do the maintainers think about this? If you agree this would be a good feature, and a general direction is agreed (breaking change with cleaner naming vs. non-breaking change with slightly misleading naming), I could hopefully devote some time to send a pull request.


DateTime.AddMonths calculation is incorrect

DateTime.AddMonths performs the calculations by using the local time. However, this does not takes into an account daily saving time transition.
For example, the following code:
var time = DateTime.FromUtc(2013, 6, 1);
var time2 = time.AddMonths(6);
System.Console.WriteLine(time.ToString() + " | " + time.ToUtcString());
System.Console.WriteLine(time2.ToString() + " | " + time2.ToUtcString());

Outputs
Sat Jun 01 2013 03:00:00 GMT+0300 (FLE Daylight Time) | Sat, 01 Jun 2013 00:00:00 GMT
Sun Dec 01 2013 03:00:00 GMT+0200 (FLE Standard Time) | Sun, 01 Dec 2013 01:00:00 GMT

while the correct result (second line) should be:
Sun Dec 01 2013 02:00:00 GMT+0200 (FLE Standard Time) | Sun, 01 Dec 2013 00:00:00 GMT

The reason for this behavior is the DateTime.AddMonths uses local time components instead of UTC ones. More specifically the code is the following:
[InlineCode("new Date({this}.getFullYear(), {this}.getMonth() + {months}, {this}.getDate(), {this}.getHours(), {this}.getMinutes(), {this}.getSeconds(), {this}.getMilliseconds())")]
public DateTime AddMonths(int months)
{
    return default(DateTime);
}

The correct code is:
[InlineCode("new Date(Date.UTC({this}.getUTCFullYear(), {this}.getUTCMonth() + {months}, {this}.getUTCDate(), {this}.getUTCHours(), {this}.getUTCMinutes(), {this}.getUTCSeconds(), {this}.getUTCMilliseconds()))")]
public DateTime AddMonths(int months)
{
    return default(DateTime);
}

Since the month during which the transition to/from DST is performed may change over the years or DST transition may not be performed at all during some years (in some parts of the world), the DateTime.AddYears should be corrected as well:
[InlineCode("new Date(Date.UTC({this}.getUTCFullYear() + {years}, {this}.getUTCMonth(), {this}.getUTCDate(), {this}.getUTCHours(), {this}.getUTCMinutes(), {this}.getUTCSeconds(), {this}.getUTCMilliseconds()))")]
public DateTime AddYears(int years)
{
    return default(DateTime);
}

Regards,
Jecho

Merge request created.

Can you delete commit "1eda354f3a84a6e92856444e8060e29355b0948c"? The code is not correct.

I have developed a class called "DateTimeExt" whose behavior is closer to DateTime in .NET Framework. It uses "DateTimeKind" similar to .NET in order to provide better support for UTC time.
It is implemented in C#, however, if there is a consensus to include it in the runtime library I can translate it to JS.

Kind of depends on how it looks, IMO. Is it online?

Posted here: http://collabedit.com/su22b
The basic idea is that Date in JS is already in UTC time so depending on the DateTimeKind value the property should return either local or UTC value.

Perhaps we could do something quirky like adding an optional property 'kind' to normal DateTimes (eg. [InlineCode("{this}.kind || Unspecified")] public DateTimeKind Kind { get; set; })

Yes - that would be the best approach. However, the properties "Year", "Month", "Hour", etc. should be changed to return correct values. Basically there are two views of a "DateTime" instance - local and UTC depend on the value of the kind field.

I agree. DateTime should work as the .net type; people who want JS behavior can use JsDate.

OK. Then I'll prepare pull requests with changes to DateTime and the runtime library. Although removing methods not available in .NET would result in better design keeping them would provide compatibility with previous versions.

I'd personally would vote to remove methods not available in .NET, but obviously it is your call.

Which methods are you talking about?
Perhaps we should do this in v3 only.

I'm referring to "GetXXX", "GetUtcXXX", "FromUtc", "ParseExactUtc" and any other methods that are not available in .NET Framework.

Yes, we are probably best off removing those if we are introducing the Kind property.

OK. Then shall I create a pull request with changes to DateTime and the runtime library? Or perhaps introduce it as a separate class until v3 is ready?

Can you create a PR to the Roslyn branch?

OK


Publish a wasm-bindgen-test for use as an internal detail within wasm-pack test

As discussed in today's WG meeting, we are aiming for publishing an initial release of the wasm-bindgen-test crate to be used as an unstable implementation detail within a wasm-pack test command for the Rust 2018 Release Candidate milestone. That's 6 weeks from now: 2018-09-13.
What needs to be done before we publish wasm-bindgen-test for use by wasm-pack test?
+cc @alexcrichton @ashleygwilliams

And here is the wasm-pack half: https://github.com/rustwasm/wasm-pack/issues/248

So I followed the steps here https://github.com/rustwasm/wasm-bindgen/blob/master/crates/test/README.md
And got the following error: https://gist.github.com/gnzlbg/bce2a1020dde236e28d466311e4697c8
System is MacOSX 10.13.6
I installed node via homebrew: brew install node, which installed the version 10.8.0
My Cargo.toml looks like:
[package]
name = "wasm-test"
version = "0.1.0"
authors = ["..."]

[target.'cfg(target_arch = "wasm32")'.dev-dependencies]
wasm-bindgen-test = { git = 'https://github.com/rustwasm/wasm-bindgen' }

and in my wasm-test binary crate, I have a .cargo/config that looks like:
[target.wasm32-unknown-unknown]
runner = 'wasm-bindgen-test-runner'

My src/main.rs is just an fn main() {}, and the tests/test.rs is the one in the Readme.

Looks like @gnzlbg's issue was https://github.com/rustwasm/wasm-bindgen/pull/667, thanks for the PR!

Yes, @fitzgen was very helpful on IRC, thanks!

could we do a wasm-bindgen minor release to avoid using this from git ?

Yes, hopefully today or tomorrow.

0.2.16 is published!

I think we can close this issue -- I didn't realize that we had already published wasm-bindgen-test when I filed this issue. I think all the remaining work is needed in wasm-pack and in documentation.


Partial NavigationPage implemented

#65
This is only a partial fix to navigation.

This only works with non-modal navigation. i.e. a NavigationPage
This only works with one NavigationPage at a time as it will take over the browser's history.
This has not been tested with a shared page....probably doesn't work.

But, PushAsync, PopAsync, and PopToRootAsync work.  Clicking on the forward and back buttons on the browser works.  And typing in the url with a hash path also works...
I had to make a pretty big change to the javascript file in order to drill down properties and get access to methods like window.history.pushState.  Now the Call method will parse the first parameter (methodName) and I can write this and have it work:
this.NativeView.Document.Window.Call("history.back")

damn, I just realized I left in a change that wasn't really needed... and was a test.  I created a Location element in the main Ooui project.  That can just go away... also the properties added to Window.  :(

Going to redo pull request with repo that hasn't touched any of the vs specific files.


Fix Dance command R15 compatibility

Adds R15 compatibility for the dance command. #165

now broken in R6 lol

might be some other pr breaking it idk

might be some other pr breaking it idk

My merge only changes which animation is called based on rig type. New issue must be further up the chain when the animation is ran.


ramm

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az configure
Errors:

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az configure

Expected Behavior
Environment Summary
Windows-10-10.0.19044-SP0
Python 3.10.4
Installer: MSI

azure-cli 2.36.0

Extensions:
azure-devops 0.25.0
init 0.1.0
interactive 0.4.5
subscription 0.1.4

Dependencies:
msal 1.17.0
azure-mgmt-resource 20.0.0

Additional Context

@jiasli for awareness

Please provide more information for us to investigate. Thanks.


tsig: Fix compilation without HAVE_SSL

CRYPTO_memcmp is an OpenSSL function that is used outside of the HAVE_SSL path.
This fixes compilation when HAVE_SSL is false.

Thanks for the patch.  Implemented a fallback routine, because without the fixedtime compare there is a vulnerability.  But no SSL for the hash, anyway.  Also fixed compilation for two other cases where compilation without SSL failed, with ratelimits and the newer DNSoverTLS code.