Plugin Activation Failure [Bug]

Describe the bug
When I tried to activate the Theme Sniffer plugin on wpsandbox.net, it threw a fatal error and interrupted the plugin activation. I have attached the screenshot for your reference.

Did you  just clone this or did you try to install the .zip file (either from the release or from the wordpress.org site?)
It could be due to missing extensions on the server. Theme Sniffer runs PHP_CodeSniffer, and PHPCS requires some extensions and a way to execute it on the server.


chore: Goroutine leak detector to help avoid Goroutine leaks.

See https://github.com/uber-go/goleak
Signed-off-by: Bo-Yi Wu<EMAIL_ADDRESS>

Codecov Report

Merging #37 (8135fe1) into master (c37246f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #37   +/-   ##
=======================================
  Coverage   88.47%   88.47%
=======================================
  Files           9        9
  Lines         347      347
=======================================
  Hits          307      307
  Misses         32       32
  Partials        8        8

Flag
Coverage Δ

go-
?

go-1.13
?

go-1.14
?

go-1.15
88.47% <ø> (+0.86%)
:arrow_up:

go-1.16
87.60% <ø> (ø)

go-1.17
?

macos-latest
?

ubuntu-latest
88.47% <ø> (ø)

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

Continue to review full report at Codecov.

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


Get TiledImage from Multi-Image collection

Hi
I have a multi-image in the collection mode. I would like to get the TiledImage the mouse pointer is hovering over. The only idea I have is to loop thru all TiledImages calling tiledImage.viewportToImageCoordinates(x,y) and hoping that if I miss the TiledImage I'm looking for the function will return null. Is there a better approach?
Or maybe I create an overlay for each tiledImage.getBounds() and try to detect which overlay the mouse is hovering over?
D

Looping through the images is what I recommend. Here's an example: https://codepen.io/iangilman/pen/PqxvgN

Thanks, very helpful as usual  :)
Just a little observation
 viewer.viewport.fitBounds(viewer.world.getItemAt(index).getBounds());
Does this look a bit off or this is the expected behaviour of .fitBounds?

@darwinjob Good catch! I've now fixed it, with gestureSettingsMouse: { clickToZoom: false }.


Guest middleware still requiring login

This is  a **bug **.

Prerequisites

[X] Are you running the latest version?
[X] Are you reporting to the correct repository?
(enso is made of many specialized packages: https://github.com/laravel-enso)
[X] Did you check the documentation?

Description
Using the below route, with a new controller directory and Index controller and copy and pasted modified Vue route and Vue page file with just content it makes me login before being able to view my about page. The only thing I've changed is adding AppFooter to every page, but that wouldn't break anything would it?
     Route::middleware(['guest'])->group(
         function() {

        Route::namespace('About')
            ->prefix('about')
            ->as('about.')
            ->group(function () {
                Route::get('about', 'Index')->name('index');
    });
});

Steps to Reproduce

Expected behavior

Actual behavior

Tested by removing my appfooter patch and same behaviour.

@curtisdelicata we're releasing today 4.0.1, please try again.
If the problem persists please create an enso fork where you can add the minimum files to reproduce your problem and we'll help you solve it asap.

Thank you. I will update you once done @aocneanu

https://github.com/modularsoftware/enso is the test repository. I'm just committing the basic files now.

I spotted this in one of the core routes js meta...
    guestGuard: true,

I am building now to see if it fixes it.

That fixed it. Sorry for the bother.


path lookups failing

Date you used Cookiecutter PyPackage: 9/15/2017
Cookiecutter version used, if any: 1.5.1
Python version, if any:  3.6.2
Operating System: OSX

Description
If this is the most popular module I would expect things to work out of the gate.. is this not the case?
I created a new project called "experiments" to play around and learn a bit of python.   I thought of using cookiecutter to get a project set up with best practices to learn a bit faster.
However, I'm unable to even resolve packages.  It seems like I'm doing something fundamentally wrong, or python's path lookup is just a bit obtuse.
What I Did
Project structure in ~/src/experiments:

./experiments/

|__./cli.py
|__ ./main.py

pyenv/ (virutalenv)

cli.py code:
import click

from experiments.ResourceType import resource_type
from experiments.main import TestClass

@click.command()
def main(args=None):
    """Console script for experiments."""
    test = TestClass()

if __name__ == "__main__":
    main()

Error:
python ./experiments/cli.py
Traceback (most recent call last):
  File "./experiments/cli.py", line 8, in <module>
    from experiments.main import TestClass
ModuleNotFoundError: No module named 'experiments.main'

python --version
Python 3.6.2

What's going on here?   This seems like a fundamental error for a boilerplate project that has a lot of people on it.   Am I missing some python thing that is not documented?

with your virtualenv active, install the project on editable mode
inside de project folder
pip install -e .

after:
experiments

thanks again.    I didn't know about the -e flag.


feat: add imap quota support and related metrics

This PR adds six additional metrics for accounts with quota support. You may now can alert on running out of quota using this metrics.


[WIP][ja]Fix invalid Japanese words

This is work-in-progress for reviews by Japanese developers.
This PR fixes invalid Japanese words "パラメータ" to "パラメーター".
According to docs, written as 「r」「re」「y」などで終わる単語については、原則付ける, so パラメータ must be unified to パラメーター.
This PR fixes these pages:

https://kubernetes.io/ja/docs/concepts/configuration/organize-cluster-access-kubeconfig
https://kubernetes.io/ja/docs/concepts/overview/working-with-objects/labels
https://kubernetes.io/ja/docs/concepts/security/pod-security-standards
https://kubernetes.io/ja/docs/concepts/storage/dynamic-provisioning
https://kubernetes.io/ja/docs/concepts/storage/volume-snapshot-classes
https://kubernetes.io/ja/docs/concepts/workloads/controllers/deployment
https://kubernetes.io/ja/docs/setup/production-environment/container-runtimes
https://kubernetes.io/ja/docs/setup/production-environment/tools/kubeadm/kubelet-integration
https://kubernetes.io/ja/docs/setup/production-environment/windows/intro-windows-in-kubernetes

/retitle [WIP][ja]Fix invalid Japanese words

/cc @oomichi

/cc @t-inu

/cc @atoato88

/retitle [ja]Fix invalid Japanese words

I additionally found the following lines should be replaced.
https://github.com/kubernetes/website/blob/3e01c055949a8bba741a904f45b5f497e75d4ef4/content/ja/docs/concepts/workloads/controllers/deployment.md?plain=1#L902
https://github.com/kubernetes/website/blob/32d4d697ccdb1ee8184acbbcc05b1ad53ea12db4/content/ja/docs/concepts/storage/storage-capacity.md?plain=1#L56

I additionally found the following lines should be replaced.
https://github.com/kubernetes/website/blob/3e01c055949a8bba741a904f45b5f497e75d4ef4/content/ja/docs/concepts/workloads/controllers/deployment.md?plain=1#L902
https://github.com/kubernetes/website/blob/32d4d697ccdb1ee8184acbbcc05b1ad53ea12db4/content/ja/docs/concepts/storage/storage-capacity.md?plain=1#L56

nice catch! Thanks.

/assign

Please add the following pages to the target list in this PR description.
https://kubernetes.io/ja/docs/concepts/storage/storage-capacity/
https://kubernetes.io/ja/docs/contribute/style/hugo-shortcodes/

LGTM, but need to be rebased onto master

/retest

/check-cla

@YuikoTakada
This PR needs rebase with upstream main branch, that's why the error on netlify occured.
Could you rebase this PR?

Umm, it finished to catch up with upstream.
But then we faced an error on netlify build.
Could you create empty comit for kick the rebuild process on netlify?

Umm, it finished catching up with upstream. But then we faced an error on netlify build. Could you create empty comit for kick the rebuild process on netlify?

@atoato88 Thank you for your advice!
empty commit means  git push -f command, right?

Umm, I didn't commit "empty commit" myself.
So I don't have any sure ideas for that.
I think we need the changing a commit hash. :thinking:

Could you try followings to do empty commit? :
$ git commit --amend
$ git push -f <remote-name> <branch-name>

Please see also:
https://github.com/kubernetes/website/pull/39479#issuecomment-1446408971
It said same thing I commented above.

@t-inu thank you for your comments. I've fixed.

/retitle [ja] Unify Japanese wording
invalid...ではないとおもうので


Add tabindex attr (for navigation to search field by keyboard)

For navigation by keyboard ( https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/tabindex )

I'll let @nlhkabu review, but I should point out that we explicitly removed tabindex attributes from the login form in #6040 due to accessibility issues.

I think, previous issue with login form breaks using tab-autocompletion, right?
In this case autocomplete in search field already off'ed

@nlhkabu, @di any info?..

@di, @nlhkabu, @ewdurbin any info?..

@di, @nlhkabu, @ewdurbin, any info?..
This is useful for navigating through the page by pressing the tab key. Now to get to the search bar - you need to press several times. Although on the main page of the main focus and idea in the search box - it comes first. And you do not want to use the mouse to click on it.
This will not break the availability, because all the next clicks on the "tab" will go like this - from the beginning.
I think this is a very useful suggestion that will shorten a lot of time for everyone who uses the keyboard navigation.
Translated with www.DeepL.com/Translator (free version)

I appreciate your contribution here @saippuakauppias, unfortunately both changing the tab index and auto-focusing the search field actually make PyPI less accessible overall based on WCAG and ARIA guidelines. There's lots more details in #6088 if you're interested.
Also, I should mention that PyPI implements the OpenSearch description format, so if you're accessing PyPI from a browser that supports this standard (like Chrome or Firefox) you can search PyPI without even visiting pypi.org first: https://twitter.com/di_codes/status/1058101995918319617


Multiple seats

Added the ability to have multiple seats, this allows f.e. weston screenshare and a local touch input to work at the same time.

Currently there are issues with displaying the cursor correctly when vnc connections are added and removed.

Thank you for sending this PR.
Could you please update AUTHORS too?
Also, it's unclear whether we can contribute this software to flutter/engine, but do you agree to delegate all rights related to this PR to Sony?

Also, could you please add a signoff (DCO) to the last commit you made?

I have modified the AUTHORS file and added the sign-off. I am fine with delegating all rights related to this PR.

This PR should fix #409 and #341


Package self import not detected by compiler

Description:
The compiler doesn't detect a self import package. That is importing the same package inside a .bal in which it self is residing in. The following sample causes an internal error
Steps to reproduce:
package test.wso2.sample;

import ballerina.io;
import test.wso2.sample;
public function main (string[] args) {
    io:println("Hello, World!");
}

Affected Versions:
0.964.0
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

This is the same issue as https://github.com/ballerina-platform/ballerina-lang/issues/4878. Hence closing.


crashes

it crashes on loading.

I'm going to need more information than that. What is it saying?

not much just says error code :1 if I try with client side
even tried on server to see the logs on console
it just crashes
fabric loader 0.14.7 with nothing but villager covert mod

oh and the minecraft version 1.19 of course with 1.0.0+1.19

latest.log
I'm also having the issue that it just crashes. At a certain moment it gives [main/ERROR]: Mixin apply for mod villagerconverting failed followed by an error I don't understand.  After that it just crashes. I didn't do more in depth testing, but I removed the mod, and now it does run.

like i mentioned there is no crash report at all. it fails to load during preparing session and does not create crash report

Okay I've pushed v1.0.1. Hopefully it should resolve the issue. Please let me know how it works!
https://modrinth.com/mod/roundaround-villager-converting/version/1.0.1

Hey, I don't know if what the etiquette is replying to closed issues, but this fixed it, so thank you so much!

you are awesome. I was not sure how to support you to fix the issue~
thank you very much!

보낸 사람: loadingspace @.>
보낸 날짜: 2022년 6월 11일 토요일 오후 1:35
받는 사람: Roundaround/mc-fabric-villager-converting @.>
참조: FudgeDumbo @.>; Author @.>
제목: Re: [Roundaround/mc-fabric-villager-converting] crashes (Issue #2)
Hey, I don't know if what the etiquette is replying to closed issues, but this fixed it, so thank you so much!
—
Reply to this email directly, view it on GitHubhttps://github.com/Roundaround/mc-fabric-villager-converting/issues/2#issuecomment-1152970288, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AZSBZOM6N3WBFBZBOCZ7UKTVOTE65ANCNFSM5YLTXJBQ.
You are receiving this because you authored the thread.Message ID: @.***>


ambient: filter cross-namespace authz policies

fixes #51556

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
4927a89728060ee0453018a356557edfa6368554
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

In response to a cherrypick label: #51557 failed to apply on top of branch "release-1.22":
Applying: ambient: filter cross-namespace authz policies
Using index info to reconstruct a base tree...
M	istioctl/pkg/writer/ztunnel/configdump/api.go
M	pilot/pkg/model/service.go
M	pilot/pkg/serviceregistry/kube/controller/ambient/policies.go
M	pilot/pkg/serviceregistry/kube/controller/ambient/workloads.go
M	pilot/pkg/serviceregistry/kube/controller/ambient/workloads_test.go
Falling back to patching base and 3-way merge...
Auto-merging pilot/pkg/serviceregistry/kube/controller/ambient/workloads_test.go
Auto-merging pilot/pkg/serviceregistry/kube/controller/ambient/workloads.go
CONFLICT (content): Merge conflict in pilot/pkg/serviceregistry/kube/controller/ambient/workloads.go
Auto-merging pilot/pkg/serviceregistry/kube/controller/ambient/policies.go
CONFLICT (content): Merge conflict in pilot/pkg/serviceregistry/kube/controller/ambient/policies.go
Auto-merging pilot/pkg/model/service.go
Auto-merging istioctl/pkg/writer/ztunnel/configdump/api.go
CONFLICT (content): Merge conflict in istioctl/pkg/writer/ztunnel/configdump/api.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 ambient: filter cross-namespace authz policies
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #51572


Allow all()'s return type to be an object

Sometimes the return type of all() is an object, not an array

Excellent. Can here this morning with the express intention of submitting an identical PR.


The shortcut key Ctrl+C cannot be detected when copying images.

The platform I am using is Win11, and the go version is 1.20. When I use the shortcut keys Ctrl+C to copy images to the clipboard, the "golang. design/x/clipboard" library cannot detect clipboard changes.
package main

import (
	"context"
	"encoding/base64"
	"golang.design/x/clipboard"
	"log"
)

func main() {
	watchLocalClipboard(context.Background())
}

func watchLocalClipboard(ctx context.Context) {
	err := clipboard.Init()
	if err != nil {
		panic(err)
	}
	textCh := clipboard.Watch(ctx, clipboard.FmtText)
	imagCh := clipboard.Watch(ctx, clipboard.FmtImage)
	for {
		select {
		case <-ctx.Done():
			return
		case textData, ok := <-textCh:
			if !ok {
				return
			}
			log.Println(string(textData))
			log.Println("------------>text")
		case imageData, ok := <-imagCh:
			if !ok {
				return
			}
			//log.Println(imageData)
			// 将图片数据编码为 Base64 字符串
			base64String := base64.StdEncoding.EncodeToString(imageData)
			log.Println(base64String)
			//clipboard.Write(clipboard.FmtText, []byte(base64String))
			log.Println("------------>img")
		}
	}
}

On Windows, use Shift+Win+s for clipping images.
I think the image path is just copied if you use Ctrl+C.

Okay, thank you.

As discussed above, this seems not a bug, closing.

在 Windows 上，用于Shift+Win+s剪切图像。
我认为如果您使用Ctrl+C.

On Windows, use Shift+Win+s for clipping images.
I think the image path is just copied if you use Ctrl+C.

In the actual system when I use ctrl+c, it only gets the filename of the image, not the full path. If it was the full path, it would also be beneficial to read this file and then process the decoding.


crystfel: fix x86_64 hash

Description of changes
I used fodwatch and checked the CrystFEL package I'm maintaining. One of the dependencies has a different hash now.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

@ofborg build crystfel


Add REST Scala 3.1 tutorial

Motivation:
Added Scala 3.1 tutorial for Armeria
Modifications:

Added /site/src/pages/tutorials/rest-scala/README.mdx

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.kate seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks a lot for your contribution, @korcena 🙇
It's likely that we'll work with our internal technical writer to evolve this into a more well-formed tutorial. Meanwhile, could you please sign the ICLA following this link?


[ALLUXIO-2163] 2163Awesome Feature

https://alluxio.atlassian.net/browse/ALLUXIO-2163
It's a hot Summer, right?

Can one of the admins verify this patch?

ok to test

Jenkins, test this please

Jenkins, add to whitelist

@HongweiQin thanks for the patch. would you mind to edit the title to be a bit more informative?
thanks

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/Alluxio-Pull-Request-Builder/11155/
Test PASSed.

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/Alluxio-Pull-Request-Builder/11156/
Test PASSed.

@apc999 Sure!

LGTM


update Chinese version for the training kit

Add Chinese github intermediate workbook.
Revise Chinese git cheat sheet. Fix some translation problems.

:+1: thanks @sdq. Before we merge this in, could you add a link to the translations around line 24 in the index (https://github.com/github/training-kit/blob/master/index.html#L24).

@brntbeer  OK, the link has been added.


Keep expanding the applied subscription about 1-2 mins

OS: Windows
Build Version: 20190513.1
Repro Steps:

Install the Azure Function extension --> Make sure there is 1 applied subscription at least.
Try to expand the applied subscription.

Expect:
Expand it immediately.
Actual:
Keep expanding the applied subscription about 1-2 mins. Meanwhile there are not many azure function apps under the subscription.

Does this consistently repro? There could always be internet or Azure issues

This issue consistently reproduces on 3 test machines on our side in this week. And I cannot make sure whether it is an internet issue or not. Due to we are not allowed to install VS Code on our local machine.

Hmm - can you try either the portal or Cloud Explorer and see if they're just as slow at displaying function apps on that VM?

Tried to expand the applied subscription with below:

More
State

Cloud Explorer
Not repro

Storage Explorer
Not repro

Portal
Not repro

App Service   Extension
Not repro

Azure Storage   Extension
Not repro

Cosmos DB   Extension
Not repro

Docker Extension
Not repro

Azure Function   Extension
Repro

Does this issue reproduce for you?

No it doesn't repro for me. Can you either do a screen recording or give me access to the machine?

Ok, the machine info has sent to you.

Okay I think I know what's going on here - will investigate a fix

👍So cool! 🤔Could you help to share any info about this issue?


The "model" could be made more useful

Description
The "model" i.e. "MacBookPro14,3" could be more helpful as "MacBook Pro (15-inch, 2017)".
I added this to my config file to print that out.
Neofetch version
Neofetch 5.0.1
I would have sent a pull request, but it's not "pure bash" has pipes and otherwise doesn't fit your style.
I hope the idea is useful, and someone with more coding skills can add it.
    # modified from https://apple.stackexchange.com/questions/98080/can-a-macs-model-year-be-determined-with-a-terminal-command
    # Setup variables for getting marketing model info from web
    serialNumberKey="$(system_profiler SPHardwareDataType | awk '/Serial/ {print substr($4,9)}')"
    onlineModel="$(curl -s https://support-sp.apple.com/sp/product?cc=$serialNumberKey |
        sed 's|.*<configCode>\(.*\)</configCode>.*|\1|')"
    #
    # Setup variables for getting marketing model info from local plist
    marketingModelFile=/System/Library/PrivateFrameworks/ServerInformation.framework/Versions/A
    marketingModelFile+=/Resources/English.lproj/SIMachineAttributes
    if ! [ -f "${marketingModelFile}.plist" ]; then
        # 10.7 or 10.6
        marketingModelFile=/System/Library/PrivateFrameworks/ServerInformation.framework/Versions/A
        marketingModelFile+=/Resources/English.lproj/XSMachineAttributes
    fi
    if [ -f "${marketingModelFile}.plist" ]; then
        model_ID="$(sysctl -n hw.model)"
        marketingModel="$(defaults read "${marketingModelFile}" "${model_ID}" |
            sed -n -e 's/\\//g' -e 's/.*marketingModel = "\(.*\)";/\1/p')"
    else
        marketingModel="${onlineModel}"
    fi

    prin "Online Model Type" "${onlineModel}"
    prin "Local Model Type" "${marketingModel}"

Here's a test using Mac models I've owned over the years
#!/usr/bin/env bash

echo "Marketing model from online Apple serial number database:"
j=0
for i in HTD5 FY11 DNMP DKQ1 DMW8 642 U36 NRZ; do
    onlineModel=$(curl -s https://support-sp.apple.com/sp/product?cc=${i} |
        sed 's|.*<configCode>\(.*\)</configCode>.*|\1|')
    j=$((j+1))
    echo "    ${j} - ${onlineModel}"
done

echo ""
echo "Marketing model from local plist:"

marketingModelFile=/System/Library/PrivateFrameworks/ServerInformation.framework/Versions/A
marketingModelFile+=/Resources/English.lproj/SIMachineAttributes
j=0
for i in MacBookPro14,3 iMac15,1 iMac13,2 MacBookPro10,1 iMac12,2 MacBookPro5,3 \
    Macmini1,1 PowerBook5,2; do
    model_ID=${i}
    marketingModel="$(defaults read "${marketingModelFile}" "${model_ID}" |
        sed -n -e 's/\\//g' -e 's/.*marketingModel = "\(.*\)";/\1/p')"
    j=$((j+1))
    echo "    ${j} - ${marketingModel}"
done

echo ""
echo "Description from local plist:"

j=0
for i in MacBookPro14,3 iMac15,1 iMac13,2 MacBookPro10,1 iMac12,2 MacBookPro5,3 \
    Macmini1,1 PowerBook5,2; do
    model_ID=${i}
    marketingModel="$(defaults read "${marketingModelFile}" "${model_ID}" |
        sed -n -e 's/\\//g' -e 's/.*description = "\(.*\)";/\1/p')"
    j=$((j+1))
    echo "    ${j} - ${marketingModel}"
done

FWIW, the macOS detailed system report uses the MacBookNN,N notation, so it's not like this isn't something Apple already exposes to users:

The current information output is fine. If the code to display the English model names wasn’t so very complex I’d consider it.
Thanks for opening this issue.

If there was a simpler method of getting that information I’d add it but that doesn’t seem to be the case.

Sorry about the unneeded complexity. I started looking at the neofetch code and realized you’d prefer pure bash.
serialNumberKey="$(system_profiler SPHardwareDataType | awk '/Serial/ {print substr($4,9)}')"
onlineModel="$(curl -s https://support-sp.apple.com/sp/product?cc=${serialNumberKey})"
onlineModel="${onlineModel##*<configCode>}"
onlineModel="${onlineModel%%<*}"

The above seems similar in form to that used in the Mac GPU code.
gpu="$(system_profiler SPDisplaysDataType |
awk -F': ' '/^\ Chipset Model:/ {printf $2 ", "}')"
gpu="${gpu//'/ $'}"
gpu="${gpu%,}”
BTW,  I can understand most of the code, but I was wondering what “gpu" string this code fixes?
${gpu//'/ $’}
This is even shorter, but less immune to changes in the XML
marketingModel="$(curl -s https://support-sp.apple.com/sp/product?cc=${serialNumberKey} |
awk -F '[<>]' '{printf $11}')"

On Jun 21, 2018, at 7:17 PM, Dylan Araps<EMAIL_ADDRESS>wrote:
The current information output is fine. If the code to display the English model names wasn’t so very complex I’d consider it.
Thanks for opening this issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/dylanaraps/neofetch/issues/1038#issuecomment-399299961, or mute the thread https://github.com/notifications/unsubscribe-auth/AAAyo6WiYe0TnXzusqsHft7eUCj21UUtks5t_FPOgaJpZM4UyjS_.

I'd rather not make network requests or add the dependency of an internet connection to Neofetch. If it's stored on the system itself in a file that would help.

It is stored in a local plist at
/System/Library/PrivateFrameworks/ServerInformation.framework/Versions/A/Resources/English.lproj/SIMachineAttributes (or XSMachineAttributes)
but using
defaults read /System/Library/PrivateFrameworks/ServerInformation.framework/Versions/A/Resources/English.lproj/SIMachineAttributes ${modelid}
and processing the result seems too complex and prone to breakage.
I’m perfectly happy leaving the model code in my config file.
Thanks for neofetch! And the pure-bash-bible. It’s a great resource.
Marketing model from local plist:
1 - 15" MacBook Pro with Thunderbolt 3 and Touch ID (Mid 2017)
2 - 27" iMac (Late 2014)
3 - 27" iMac (Late 2012)
4 - MacBook Pro with Retina display, Intel Core i7, 15" (Mid 2012)
5 - 27" iMac (Early 2011)
6 - MacBook Pro Intel Core 2 Duo (aluminum unibody) (Mid 2009)
7 - Mac mini Intel Core (Early 2006)

On Jun 22, 2018, at 7:02 PM, Dylan Araps<EMAIL_ADDRESS>wrote:
I'd rather not make network requests or add the dependency of an internet connection to Neofetch. If it's stored on the system itself in a file that would help.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/dylanaraps/neofetch/issues/1038#issuecomment-399622518, or mute the thread https://github.com/notifications/unsubscribe-auth/AAAyo6YjUXwWgDC6Q9Dm1heioCqoqqutks5t_aHPgaJpZM4UyjS_.


name conventions: rename getExtensiveSubject to getExtensiveSubjects

In interface org.apache.ibatis.autoconstructor.AutoConstructorMapper: method getExtensiveSubject should be renamed to getExtensiveSubjects

Fix via #1643.


[preset] Use the system debugserver for buildbot_incremental,lldb_asan_ubsan

This addresses "this is a non-interactive debug session, could not get
permissions to debug" errors on the oss-lldb-asan-osx bot.

@swift-ci smoke test and merge


data format command broken for HadCRUT4: too many months?

Describe the bug
The esmvaltool data format (or prepare) command fails for HadCRUT4:
$ esmvaltool data prepare HadCRUT4
<snip>
2024-06-07 12:43:03,660 UTC [23736] INFO    [NCL] (0)	INFO    fatal: in format_time (esmvaltool/cmorizers/data/formatters/utilities.ncl), incorrect number of timesteps in input data: 2028 expected, 2062 found
2024-06-07 12:43:03,660 UTC [23736] INFO    [NCL]
2024-06-07 12:43:03,660 UTC [23736] INFO    Formatting successful for dataset HadCRUT4

(Note also the erroneous reporting of success.)
The error message suggests that the code is hard-wired for 169 years of monthly data, but that additional months have been added in an update to the source data.
Attachment: main_log_debug.txt

well, the "success" comes from NCL returning code 0 which is, usually, actual huzzah :grin:


Deduce Domain Name

As I understand, you'd like to know how to fetch the domain name of your API Gateway API created by your CloudFormation stack and reference it as an output. Also, I can see that you're using AWS SAM to build your application.
In the processed template of your CloudFormation stack 'arn:aws:cloudformation:ap-southeast-1:407461997746:stack/days/f37dc1c0-27e2-11ec-ab94-0a12a8352912' there's an AWS::ApiGatewayV2::DomainName resource [1] with logical ID 'ApiGatewayDomainNameV23694ba20e6' which was generated by your 'Gateway' AWS::Serverless::HttpApi resource in the original template. You can reference this resource in your template's outputs by using its logical ID as a static value in a Ref function [2].
For example:
Outputs:
MyDomain:
Value: !Ref ApiGatewayDomainNameV23694ba20e6
Please note that unfortunately, the logical ID of the AWS::ApiGatewayV2::DomainName is not as predictable as other resources generated by AWS::Serverless::HttpApi such as:

'Gateway' for the AWS::ApiGatewayV2::Api resource.
'GatewayApiMapping' for the AWS::ApiGatewayV2::ApiMapping resource.
'GatewayApiGatewayDefaultStage' for the AWS::ApiGatewayV2::Stage resource.

Since the logical ID of AWS::ApiGatewayV2::DomainName is randomly generated when using AWS::Serverless::HttpApi, this logical ID only becomes available after processing the original template for the first time (which has already occurred in the provided stack). Also, If you update the domain name for this API, it will trigger a replacement update [3] and may cause the logical ID of this resource to change and require you to update the logical ID in any Ref functions in your template. My apologies if this behavior causes any frustrations on your end.
I hope I've been able to help, and as always, please don't hesitate to reach out if you have any further questions or concerns.
Links:
[1] https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-apigatewayv2-domainname.html
[2] https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-apigatewayv2-domainname.html#aws-resource-apigatewayv2-domainname-return-values
[3] https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-apigatewayv2-domainname.html#aws-resource-apigatewayv2-domainname-properties

Solved by:
  CNAME:
    Description: The regional domain name for CNAME value
    Value: !GetAtt [Gateway.DomainName, RegionalDomainName]


Rename the prefix of C API from ML to WNN, namespace ml to wnn

The "ML" prefix conflict WebNN IDL definition, so rename it to "WNN" (WebNN).
fix https://github.com/webmachinelearning/webnn-native/issues/208

Thanks, merge it.


Can't include more than one code block from the same file

Thanks for the plugin. It's very useful. Not sure if you're still maintaining it, but when I try to import two or more sections of code from the same file I get an error. This is from two different code blocks (and different marker names). For example:
```typescript file=./packages/login/src/types.ts start=start_mdx_hook end=end_mdx_hook
```
```typescript file=./packages/login/src/types.ts start=start_mdx_hook2 end=end_mdx_hook2
```
results in the error:
Error: MDX compilation failed for file "/home/jg/www/npm-library/shared/docs/docs/components/login.mdx"
Cause: Ambiguous code block start marker. Found more than once in ./shared/common/src/types.ts, at lines 27,50
Details:
Error: Ambiguous code block start marker. Found more than once in ./shared/common/src/types.ts, at lines 27,50
at getSnippet (/home/jg/www/npm-library/node_modules/.pnpm/remark-code-snippets@1.1.0/node_modules/remark-code-snippets/dist/index.js:81:19)
at transformer (/home/jg/www/npm-library/node_modules/.pnpm/remark-code-snippets@1.1.0/node_modules/remark-code-snippets/dist/index.js:62:30)
at wrapped (file:///home/jg/www/npm-library/node_modules/.pnpm/trough@2.1.0/node_modules/trough/index.js:115:27)
at next (file:///home/jg/www/npm-library/node_modules/.pnpm/trough@2.1.0/node_modules/trough/index.js:65:23)
at done (file:///home/jg/www/npm-library/node_modules/.pnpm/trough@2.1.0/node_modules/trough/index.js:148:7)
at then (file:///home/jg/www/npm-library/node_modules/.pnpm/trough@2.1.0/node_modules/trough/index.js:158:5)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at Object.mdxLoader (/home/jg/www/npm-library/node_modules/.pnpm/@docusaurus+mdx-loader@3.1.0_@docusaurus+types@3.1.0_esbuild@0.19.11_react-dom@18.2.0_react@18.2.0/node_modules/@docusaurus/mdx-loader/lib/loader.js:130:25)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

False alarm, got it to work. Must have been caching issues either from the browser or webpack, or perhaps the regex in the plugin doesn't like numbers. In any case, after changing the hook names and a manual build it's all working as expected.


Streamlit JSONDecodeError: Expecting value: line 1 column 1 (char 0)

Dear @DmitryKey , it is a pleasure to follow your guides and go through the examples. Thank you for providing such interesting tools.
Concerning "Neural Search with BERT and Solr" everything seems to go alright including indexing and searching on the solr server, however, once I start streamlit the following errors occur when I search. Do you have any idea about what it may be?
JSONDecodeError: Expecting value: line 1 column 1 (char 0)
Traceback:

File "/home/n/documents/my_jina_app/baas/env/lib/python3.6/site-packages/streamlit/script_runner.py", line 337, in _run_script
    exec(code, module.__dict__)
File "/home/n/documents/my_jina_app/baas/bert-solr-search/src/search_demo_solr.py", line 122, in <module>
    docs, query_time, numfound = sc.query("vector", query)
File "src/client/solr_client.py", line 102, in query
    resp = resp.json()
File "/home/n/documents/my_jina_app/baas/env/lib/python3.6/site-packages/requests/models.py", line 898, in json
    return complexjson.loads(self.text, **kwargs)
File "/home/n/anaconda3/lib/python3.6/json/__init__.py", line 354, in loads
    return _default_decoder.decode(s)
File "/home/n/anaconda3/lib/python3.6/json/decoder.py", line 339, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "/home/n/anaconda3/lib/python3.6/json/decoder.py", line 357, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None

Thank you for your consideration.

Hi @xeisberg
Thank you for kind words and glad to hear that the tools are interesting to you.
For the error above -- do you see any errors in Solr's logs?
Does the index exist and it contains the vector field?
Can you please paste the screenshot of the streamlit UI, since it has few params -- so it will be easier to reproduce on my side.

Dear @DmitryKey . Thank you for taking the time to consider my issue. I should also mention that I am using solr 8.5.2 with this plugin version https://github.com/markhng525/solr-vector-scoring
I did try solr 6.6.0 and 8.8.0 following your instructions and ended up with the same error while using streamlit although indexing worked fine. I have not saved the error logs but can quickly do so as I have kept the folders intact and easy to run.
solr logging looks like this:

I think the indexing is working fine as can be seen in this screenshot with the vector:

However, if I run "defType=vp" for the vector function I get the following error:

{   "responseHeader":{     "status":500,     "QTime":0,     "params":{       "q":"*:*",       "defType":"vp",       "wt":"json",       "debugQuery":"on",       "_":"1620351183292"}},   "error":{     "trace":"java.lang.NullPointerException\n\tat com.github.saaay71.solr.query.VectorQParserPlugin$1.parse(VectorQParserPlugin.java:34)\n\tat org.apache.solr.search.QParser.getQuery(QParser.java:174)\n\tat org.apache.solr.handler.component.QueryComponent.prepare(QueryComponent.java:161)\n\tat org.apache.solr.handler.component.SearchHandler.handleRequestBody(SearchHandler.java:309)\n\tat org.apache.solr.handler.RequestHandlerBase.handleRequest(RequestHandlerBase.java:211)\n\tat org.apache.solr.core.SolrCore.execute(SolrCore.java:2596)\n\tat org.apache.solr.servlet.HttpSolrCall.execute(HttpSolrCall.java:802)\n\tat org.apache.solr.servlet.HttpSolrCall.call(HttpSolrCall.java:579)\n\tat org.apache.solr.servlet.SolrDispatchFilter.doFilter(SolrDispatchFilter.java:420)\n\tat org.apache.solr.servlet.SolrDispatchFilter.doFilter(SolrDispatchFilter.java:352)\n\tat org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1596)\n\tat org.eclipse.jetty.servlet.ServletHandler.doHandle(ServletHandler.java:545)\n\tat org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:143)\n\tat org.eclipse.jetty.security.SecurityHandler.handle(SecurityHandler.java:590)\n\tat org.eclipse.jetty.server.handler.HandlerWrapper.handle(HandlerWrapper.java:127)\n\tat org.eclipse.jetty.server.handler.ScopedHandler.nextHandle(ScopedHandler.java:235)\n\tat org.eclipse.jetty.server.session.SessionHandler.doHandle(SessionHandler.java:1607)\n\tat org.eclipse.jetty.server.handler.ScopedHandler.nextHandle(ScopedHandler.java:233)\n\tat org.eclipse.jetty.server.handler.ContextHandler.doHandle(ContextHandler.java:1297)\n\tat org.eclipse.jetty.server.handler.ScopedHandler.nextScope(ScopedHandler.java:188)\n\tat org.eclipse.jetty.servlet.ServletHandler.doScope(ServletHandler.java:485)\n\tat org.eclipse.jetty.server.session.SessionHandler.doScope(SessionHandler.java:1577)\n\tat org.eclipse.jetty.server.handler.ScopedHandler.nextScope(ScopedHandler.java:186)\n\tat org.eclipse.jetty.server.handler.ContextHandler.doScope(ContextHandler.java:1212)\n\tat org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:141)\n\tat org.eclipse.jetty.server.handler.ContextHandlerCollection.handle(ContextHandlerCollection.java:221)\n\tat org.eclipse.jetty.server.handler.InetAccessHandler.handle(InetAccessHandler.java:177)\n\tat org.eclipse.jetty.server.handler.HandlerCollection.handle(HandlerCollection.java:146)\n\tat org.eclipse.jetty.server.handler.HandlerWrapper.handle(HandlerWrapper.java:127)\n\tat org.eclipse.jetty.rewrite.handler.RewriteHandler.handle(RewriteHandler.java:322)\n\tat org.eclipse.jetty.server.handler.HandlerWrapper.handle(HandlerWrapper.java:127)\n\tat org.eclipse.jetty.server.Server.handle(Server.java:500)\n\tat org.eclipse.jetty.server.HttpChannel.lambda$handle$1(HttpChannel.java:383)\n\tat org.eclipse.jetty.server.HttpChannel.dispatch(HttpChannel.java:547)\n\tat org.eclipse.jetty.server.HttpChannel.handle(HttpChannel.java:375)\n\tat org.eclipse.jetty.server.HttpConnection.onFillable(HttpConnection.java:270)\n\tat org.eclipse.jetty.io.AbstractConnection$ReadCallback.succeeded(AbstractConnection.java:311)\n\tat org.eclipse.jetty.io.FillInterest.fillable(FillInterest.java:103)\n\tat org.eclipse.jetty.io.ChannelEndPoint$2.run(ChannelEndPoint.java:117)\n\tat org.eclipse.jetty.util.thread.strategy.EatWhatYouKill.runTask(EatWhatYouKill.java:336)\n\tat org.eclipse.jetty.util.thread.strategy.EatWhatYouKill.doProduce(EatWhatYouKill.java:313)\n\tat org.eclipse.jetty.util.thread.strategy.EatWhatYouKill.tryProduce(EatWhatYouKill.java:171)\n\tat org.eclipse.jetty.util.thread.strategy.EatWhatYouKill.run(EatWhatYouKill.java:129)\n\tat org.eclipse.jetty.util.thread.ReservedThreadExecutor$ReservedThread.run(ReservedThreadExecutor.java:388)\n\tat org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:806)\n\tat org.eclipse.jetty.util.thread.QueuedThreadPool$Runner.run(QueuedThreadPool.java:938)\n\tat java.lang.Thread.run(Thread.java:748)\n",     "code":500}}
Here are some screenshots of the streamlit UI. It does not work with any of the parameter settings that I have tried.

@DmitryKey Thank you. I beg your pardon for not being clear in the last message. I tried with the Solr 6.6.0, 8.0.0, and 8.5.2 with their respective plugins hoping to solve the problem.
Anyway, I tried again now creating a new jar plug using mvn package from your link and used it with Solr 8.0.0 and the same error occurs as before. The beforementioned queries functions well in the browser so I wonder what the problem could be.

hi @xeisberg let's compare the python dependencies. Can you please run
pip freeze > requirements_freeze.txt
and then compare this file with the same file in git? https://github.com/DmitryKey/bert-solr-search/blob/feature/odfe-1M-indexing-optimization/requirements_freeze.txt
And the other thing -- can you compare your managed_schema and solrconfig.xml to these: https://github.com/DmitryKey/bert-solr-search/tree/feature/odfe-1M-indexing-optimization/solr_conf/8.0.0

Hello @DmitryKey .
In the virtual environment I installed your requirements_freeze.txt so the only package not the same when comparing was :
53 | ipython | 7.17.0 | 7.16.1, could this be a problem?
The configs are 100% match.
Given that the query is working with http://localhost:8983/solr/vector_10/query?fl=name,score,vector&q={!vp f=vector [..], is it possible to query is some other way without going through streamlit? Of course this not fully related to this issue.

thanks for checking that!
How many dimensions do your vectors have?
In my case I use 768 dim vectors, so the only way to query them is to use HTTP POST, because the character length of the query exceeds HTTP GET.
Btw, would you be open for a quick zoom call to show your vector search setup? It might be a little faster to figure things out.
Ok, if not: we can also continue discussing here!

@DmitryKey I see. I am not sure how to check the dimensions of the vectors. I am using BERT uncased_L-4_H-128_A-2.
For sure. If you send me a like or another way of contacting you I could join you. When would it suit you?

@xeisberg I've sent you a zoom link for tomorrow, hope it works out for you.

Looking at your model name, I believe it is generating 128 dimensional vectors. You can double check that by looking inside the unziped directory with the model, look for config.json and parameter name is hidden_size.

@DmitryKey Thank you. Speak to you later today. Indeed, it has 128 dimensional vectors as specified in the config. Thank you for clarifying.

Thank you for the assistance @DmitryKey.
So in the end the problem was the src/search_demo_solr.py file
on line 122 :      docs,query_time, numfound = sc.query("vector", query) where one needs to replace "vector" with the core name in your solr index which may be determined by the amount of files that one indexes(it my case it became vector_10 for 10 files.


Failure in transact with Sqlite.jsm

Reported by @chrmod:
NS_ERROR_UNEXPECTED: Component returned failure code: 0x8000ffff (NS_ERROR_UNEXPECTED) [mozIStorageAsyncStatement.bindByIndex]  Sqlite.jsm:700

Krzysztof, what's the call to transact that's raising this error?

      db.transact([
        {
          "db/id": datomish.tempid(),
          "page/url": url,
        }
      ]);

with scheme:
    const schema = {
      "name": "pages",
      "attributes": [
        {
          "name":         "page/url",
          "type":         "string",
          "cardinality":  "one",
          "unique":       "identity",
          "doc": "A page's URL."
        },
      ]
    };

waves goodbye to Datomish


Fix Nix Build

I replaced the nix flake by one only providing a dev shell with elan.
While having a proper nix build is always nice, it seems like we are trying to replicate parts of existing tools like lake, which does not seem feasible to me. I think it is easiest to have a unified buildchain here using elan as the Lean version manager and lake as the build tool. This should give us enough reproducibility already.


amdvlk 32bit

Project description
#82305 introduced amdvlk to nixos already, but afaict this only works with 64bit application, but since a lot of especially games are still 32bit it really would be nice to also be able to run those with amdvlk.
I currently have this:
{
environment.variables.VK_ICD_FILENAMES = "${pkgs.amdvlk}/share/vulkan/icd.d/amd_icd64.json:/run/opengl-driver-32/share/vulkan/icd.d/radeon_icd.i686.json";
}

In my nixos configuration to make sure that if possible amdvlk is used but also to make sure 32bit applications can still run
Metadata
should be the same

What would be the right way to do this with nix?
The possibilities I see are:

Build with -DCMAKE_C_FLAGS=-m32 -DCMAKE_CXX_FLAGS=-m32
Use the nix i686 toolchain (working amdvlk here: https://github.com/Flakebi/nixpkgs/commit/71f08eb979ca2de060e2717c0a1c83e5703b8ef1, can be used as pkgsi686Linux.amdvlk)

A disadvantage of 2. is that it does not support lto.


fix: revert Split_Learning_for_bank_marketing.ipynb.

Fix wrong modification from #663

#669 will fix the doc. Close this pr.


build(devs-dep): upgrade jest deps and prettier

Summary

Upgrade jest to next-11 and prettier 2.3.0
Test plan

Green CI
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information
N.A.

Pull Request Test Coverage Report for Build 870811392

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 93.936%

Totals

Change from base Build 870538333:
0.0%

Covered Lines:
1004

Relevant Lines:
1034

💛  - Coveralls


FillFoward price scale factor update

Expected Behavior

For adjusted data, price scaling is handled correctly for filled forward bars on a factor file change date with no data

Actual Behavior

After https://github.com/QuantConnect/Lean/pull/4650 price scaling was pushed to the top of the subscription stack after fill forwarding. In the case that a price scale factor change occurs in a date for which we don't have any data, the system will emit fill forward bars and (bug) was updating the scale but the fill forward bars should use prev scale!

Potential Solution

Review, refactor, fix & test

Reproducing the Problem

class OptimizedVentralProcessor(QCAlgorithm):
    def Initialize(self):
        self.SetStartDate(2011, 11, 1)  # Set Start Date
        self.SetEndDate(2011, 12, 5)  # Set Start Date
        self.SetCash(100000)  # Set Strategy Cash
        self.symbol = self.AddEquity("UST", Resolution.Minute).Symbol # reproduces issue
        #self.symbol = self.AddEquity("UST", Resolution.Minute, Market.USA, False).Symbol # does not reproduce issue
        self.Schedule.On(
            self.DateRules.EveryDay(self.symbol),
            self.TimeRules.At(9,59),
            self.PlotPrice)
    def PlotPrice(self):
        self.Plot(f'{self.symbol}', 'Ask', self.Securities[self.symbol].AskPrice)
        self.Plot(f'{self.symbol}', 'Bid', self.Securities[self.symbol].BidPrice)
        self.Plot(f'{self.symbol}', 'Price', self.Securities[self.symbol].Price)

System Information

N/A
Checklist

[x] I have completely filled out this template
[x] I have confirmed that this issue exists on the current master branch
[x] I have confirmed that this is not a duplicate issue by searching issues

[x] I have provided detailed steps to reproduce the issue

While digging for this issue and adding tests, detected that after https://github.com/QuantConnect/Lean/pull/4650 the clone operation performed before adjusting the data caused us to lose the IsFillForward flag for QuoteBars/Ticks, TradeBars are using memberwise clone which covered it bug

Closed by https://github.com/QuantConnect/Lean/pull/4836


Float literals - fix main primitives to use real_t casting

Uses (real_t) casting to ensure appropriate calculations are done in 32 bit where real_t is compiled as 32 bit.
2nd part of #57703
Notes

There are also a couple of places I've added extra brackets where my compiler has indicated the order of precedence is ambiguous.

This might have been discussed already but wouldn't it be simpler to put the (real_t) cast in the math defines directly? Or do we want them to stay double by default and just cast to real_t in critical paths where it makes sense?

This might have been discussed already but wouldn't it be simpler to put the (real_t) cast in the math defines directly? Or do we want them to stay double by default and just cast to real_t in critical paths where it makes sense?

I did mention this in the first PR. One disadvantage of putting the cast in the math defines is that sometimes when compiling real_t as float we actually may want to use double precision. So we probably either end up needing two sets of defines, or we cast when using.
Either is okay .. changing the define means it will be used in more places automatically, whereas casting in place makes it more explicit what is going on from the calling code. Pick your poison! :grinning:
(I'm happy to go with either approach BTW).

Thanks!


Getting contents of the resulting html file

In my bundle I create an iframe dynamically and set it's contents to the source of the html file(which has links to the resulting css and js files from another entry file.
To do this I replace an identifier with the source of html file
{
  entries: {
    bundle: {
    },
    iframe: {
    }
  },
  plugins: [
    new HtmlPlugin({
      template: join(PATHS.iframe, 'views/iframe.ejs'),
      filename: join(PATHS.dist, 'iframe.html'),
      inject: false,
      chunkSortMode: 'dependency',
      chunks: ['iframe'],
    }),
  ]

In the bundle:
// create iframe
const doc = iframe.contentDocument;
doc.open();
doc.write(process.env.IFRAME_HTML);

then i replace this string with the source of iframe.html.
Right now I do it with this plugin:
    {
      /**
       * @param {Compiler} compiler
       */
      apply(compiler) {
        compiler.plugin('done', () => {
          const iframeHtml = fs.readFileSync(join(PATHS.dist, 'iframe.html')).toString('utf-8');

          replaceInFile.sync({
            files: [join(PATHS.dist, '/**/*.js')],
            replace: /process\.env\.IFRAME_HTML/g,
            with() {
              return `"${iframeHtml}"`;
            },
          });
        });
      },
    },

It works for production build, but doesn't work with webpack-dev-server, obviously. I think I should perform the replacement on an earlier stage of the compilation, but it's not clear on which one, because html-webpack-plugin uses a child compiler.
At what point can I do this? When does html-webpack-plugin emit the html file?
Or is there another way to achieve it, besides creating a separate build step for the iframe?
Thank you!

You can use a html-webpack-plugin event

This is exactly what I needed. Completely forgot about this section in the README. Thank you!

I wrote a plugin based on html-webpack-plugin-after-emit event, but this event isn't emitted, when i use it with webpack-dev-server, because there is this check https://github.com/ampedandwired/html-webpack-plugin/blob/8b79742f6f944720cb63822551508bf48173d971/index.js#L84.

You would have to disable the cache

Thank you, again. And sorry for the trouble. Again. :)


Run sugardough tests in docker.

Running the tests only on travis is not enough because we don't test the docker image build procedure.
I temporarily created a job here https://ci.masterfirefoxos.com/job/sugardough/ until we move this CI to a proper place.

@jgmize r?

Simple change and it works so i'm merging this.


Point map on user location after accept permission

This PR fixes issue #24 opened by @rocboronat.
I've added an option to get LastKnowLocationonce user accepts the permission. I'm not sure if it should get last location, or ask for location update

Thanks for that quick fix!

Done with that @rocboronat

Thanks @alorma ! You have my lovely finger 💋  👍

👍


API for programmatically embedding Tablesaw Plot graphs?

I want to embed custom graphs, and I'm not sure where to start. I'm a Java expert, but I'm new to Jupyter Notebook and IJava, so maybe you can point me to some documentation.
First of all I installed Anaconda and then installed the IJava kernel in Jupyter Notebook with no problem. So far it's working without a hitch using OpenJDK 11 on Windows 10!
Next I tried to use a Java-based dataframe package Tablesaw. I was able to add its Maven dependencies, load a CSV file, and create a plot. Very nice!
However to produce a graph Tablesaw generates a temporary HTML file using Plotly, and invokes the browser to show the interactive plot. In other words, the graph does not appear inside Jupyter Notebook.
Tablesaw has an example at https://nbviewer.jupyter.org/github/benmccann/nbviewer-test/blob/master/Tablesaw.ipynb using the BeakerX kernel (not IJava), and as you can see (scroll down to "Play (Money)ball with Linear Regression"), they are embedding a Tablesaw Plot directly within Jupyter Notebook.
I just assumed that IJava was better and more up-to-date with the latest Java versions (because it's built directly on JShell), so before I switch to BeakerX I wanted to find out if I could do the same thing in IJava. Where do I start to figure out how to embed a Tablesaw Plot object as an interactive graph, the way they are doing that in BeakerX?

I added the same question on Stack Overflow, and even put a bounty on it, but got no responses. I can put a higher bounty on it, but the complete lack of response makes me wonder if nobody is interested. Any tips or just an indication of where to look would be helpful.

Hi @garretwilson, I'm glad you found the kernel and had no issues installing it! That is something I've been focusing on.
As far as a comparison to BeakerX, I would say that IJava is simply more focused on Java? A main use of the kernel is in teaching which I think it has been successful with because the backend is jshell which stays very true to the language specification. BeakerX being a more mature tool has a lot more "extras" that IJava doesn't yet (one being widgets but I have something in the works) and also has many more kernels than just java.
I can only work on this in my spare time which I haven't had a lot of lately which is why progress is slow at the moment. What I can point you to is the display and render functions (https://github.com/SpencerPark/IJava/blob/master/docs/display.md) that are "builtins" in the IJava kernel. Writing a custom renderer is possibly but not completely necessary. Check out RenderTest.java in the base kernel for a few examples of ways to setup a custom renderer both in the case where you are writing the class as well as the case where you are using a library and can't change the class definition.
Here is some progress towards that inline plot but there is an issue currently in that we don't get a global when importing the umd plot.ly module because jupyter is using require.js so the awful hack is to stick it in a iframe. If you can find a way to get the Plotly global we can render it inline. Anyways here is that to hopefully give an idea of how to approach this:
import tech.tablesaw.plotly.api.*;
import tech.tablesaw.plotly.components.*;

String[] animals = {"bear", "cat", "giraffe"};
double[] cuteness = {90.1, 84.3, 99.7};

Table cuteAnimals = Table.create("Cute Animals")
    .addColumns(
        StringColumn.create("Animal types", animals),
        DoubleColumn.create("rating", cuteness)
    );

Figure figure = AreaPlot.create("Cute animals", cuteAnimals, "Animal types", "rating");

String raw = Page.pageBuilder(figure, "plot_div_id").build().toString();
raw = "data:text/html;charset=utf-8;base64," + Base64.getEncoder().encodeToString(raw.getBytes("ascii"));
render("", "text/html");

Alright I am not sure what was going on the other day but the following should be working fine:
import io.github.spencerpark.ijava.IJava;

IJava.getKernelInstance().getRenderer()
    .createRegistration(tech.tablesaw.plotly.components.Figure.class)
    .preferring(io.github.spencerpark.jupyter.kernel.display.mime.MIMEType.TEXT_HTML)
    .register((figure, ctx) -> {
        ctx.renderIfRequested(io.github.spencerpark.jupyter.kernel.display.mime.MIMEType.TEXT_HTML, () -> {
            String id = UUID.randomUUID().toString().replace("-", "");

            figure.asJavascript(id);
            Map<String, Object> context = figure.getContext();

            StringBuilder html = new StringBuilder();
            html.append("

\n");
            html.append("\n");
            return html.toString();
        });
    });

The above code registers a renderer for tablesaw Figures. Then the figures can be displayed via the display, render, or last expression in cell.
import tech.tablesaw.api.*;
import tech.tablesaw.plotly.api.*;
import tech.tablesaw.plotly.components.*;

String[] animals = {"bear", "cat", "giraffe"};
double[] cuteness = {90.1, 84.3, 99.7};

Table cuteAnimals = Table.create("Cute Animals")
    .addColumns(
        StringColumn.create("Animal types", animals),
        DoubleColumn.create("rating", cuteness)
    );

AreaPlot.create("Cute animals", cuteAnimals, "Animal types", "rating");

Note: the above snippet works in the jupyter notebook. Jupyterlab has disabled javascript output but if you are interested in that, there is a jupyterlab-renderers renderer for plotly that we can look into.

@SpencerPark, thanks immensely for this feedback. I'm excited to try it out, and the Tablesaw team is anxious to hear back as well. I just have a pile of work to get through before I can get back to this. In the meantime, if you want to put any of this as an answer on my Stack Overflow question, you can get the bounty I put up before it expires. Cheers!

OMG, @SpencerPark , I'm speechless. This is incredible! It appears you went and did some in-depth research on Tablesaw, and got down into the nitty gritty enough to tell me exactly what I need. I couldn't have asked for more; my sincere thanks. And the output is just incredible! This opens up so many possibilities.
I'm going to talk to the people at https://github.com/jtablesaw/tablesaw/ and see what we can do about getting some sort of IJava enabler built-in.
Then I have so many things I want to try out with this. Thanks again!

I have a hard time to do that with IJava. I expand this script and make it even plot with an image. However, I cannot store the state of the image if I try to save the notebook. Only JavaScript saved and image are not rendered if you open the notebook again. Is there any viable path for me to execute the html and probably display the base64 encoding image directly from the cell (so I can save and share with others).
// For rendering plots in Jupyter Notebook
import io.github.spencerpark.ijava.IJava;

IJava.getKernelInstance().getRenderer()
    .createRegistration(tech.tablesaw.plotly.components.Figure.class)
    .preferring(io.github.spencerpark.jupyter.kernel.display.mime.MIMEType.TEXT_HTML)
    .register((figure, ctx) -> {
        ctx.renderIfRequested(io.github.spencerpark.jupyter.kernel.display.mime.MIMEType.TEXT_HTML, () -> {
            String id = UUID.randomUUID().toString().replace("-", "");
            boolean plotImage = System.getenv().containsKey("D2L_PLOT_IMAGE");

            figure.asJavascript(id);
            Map<String, Object> context = figure.getContext();
            StringBuilder html = new StringBuilder();
            html.append("\n");
            html.append("

\n");
            html.append("\n");
            return html.toString();
    });
});


bump github.com/Microsoft/go-winio

The minor fixes contained finally get us to a point where we can run things like go build ./... or go test ./... in the repo, which is nice!

[merge]


[Added] OFL License

OFL License

OFL is used by almost all libre fonts. Selling fonts by themselves is not allowed, but bundling with software or selling font design services is allowed. Font names must usually be changed before making any modifications. Document embedding is allowed.

https://opensource.org/licenses/OFL-1.1

@forelabs Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@forelabs Thank you for signing the Contributor License Agreement!


Star

Good afternoon, sorry to bother you. For some reason, many people are giving star to my repo:
https://github.com/89z/googleplay
I saw you gave my repo a star, thank you. However I don't know how its happening. Can you say how you found my project? Is it linked somewhere?

Hi, I don't remember exactly but there is a Explore repositories section at GitHub feed page which recommends some repos.
Also, I recommend to take a look at Insights page (Traffic section) to see websites that refer to your repo.


Cannot resolve ./style in react-toolbox lib/components

After installing react-toolbox over react-starter-kit i came across an error below:
ERROR in ./~/react-toolbox/lib/list/ListCheckbox.js     Module not found: Error: Cannot resolve 'file' or 'directory' ./style in /Users/Username/WebstormProjects/projectName/node_modules/react-toolbox/lib/list      @ ./~/react-toolbox/lib/list/ListCheckbox.js 23:13-31
And this error iterate for all react-toolBox components used in my codes
I have tried to change webpack config but wasn't helpful

This is probably happening because the starter kit does not have the correct css-loaders for webpack. Look at React Toolbox's webpack file and under the loaders you will see one for css/scss. Copy that line into your webpack file and NPM install css-loader, sass-loader, and auto prefixer. Then run webpack and all should be well. You may need to also npm install the extract text plugin for webpack too if you want to output a css file.

I had the same issue. See what I've changed in my PR here: https://github.com/coryhouse/react-slingshot/pull/55/files

You have to add node_modules/react-toolbox to be processed by webpack too

Thank you for your replies. @javivelasco Could you please provide a quick tuts for your solution.
How to add node/modules/react-toolbox to webpack?

I have tried all suggested solution, but still its not working
You can check my webpack.config.js in the link below
https://github.com/kriasoft/react-starter-kit/blob/master/tools/webpack.config.js

You are not telling webpack to resolve css extension nor explicitly including node_modules!

I have wasted my entire day on this issue.  More specific help would be greatly appreciated!
Things I have already tried:

I have npm installed css-loader, sass-loader and auto-prefixer as well as the extract-text-webpack-plugin.

I have added '.css' to the list of "resolve extensions" (even though this is not present in the webpack.config.development.js file)

I have explicitly included "node_modules"

I have explicitly included "node_modules/react-toolbox/lib/button" (as a test)

If anyone is able to resolve this, please provide a comment with the steps!

Share your webpack config

var path = require('path');
var webpack = require('webpack');
const autoprefixer = require('autoprefixer');
const ExtractTextPlugin = require('extract-text-webpack-plugin');
module.exports = {
devtool: 'eval',
entry: [
'webpack-dev-server/client?http://localhost:3000',
'webpack/hot/only-dev-server',
'./src/index'
],
output: {
path: path.join(_dirname, 'dist'),
filename: 'bundle.js',
publicPath: '/static/'
},
resolve: {
extensions: ['', '.jsx', '.css', '.scss', '.js', '.json']
},
module: {
loaders: [
{
test: /.(js|jsx)$/,
loader: 'babel',
exclude: /(node_modules)/
}, {
// "include" is commonly used to match the directories
include: [
path.resolve(dirname, "node_modules"),
path.resolve(dirname, "node_modules/react-toolbox/lib/button")
],
test: /.(scss|css)$/,
loader: ExtractTextPlugin.extract('style', 'css?sourceMap&modules&importLoaders=1&localIdentName=[name][local][hash:base64:5]!postcss!sass?sourceMap')
}
]
},
postcss: [autoprefixer],
plugins: [
new ExtractTextPlugin('spec.css', { allChunks: true }),
new webpack.HotModuleReplacementPlugin(),
new webpack.NoErrorsPlugin()
]
};

Here's an extract of my working config
  module: {
    loaders: [
      {test: /\.js$/, exclude: /node_modules/, loaders: ['react-hot']},
      {test: /\.json$/, loader: 'json'},
      {
        test: /\.js$/,
        exclude: /node_modules/,
        loader: 'babel',
        query: {
          presets: ['es2015', 'react'],
          plugins: [
            'transform-class-properties',
            'transform-object-rest-spread',
            'transform-decorators-legacy',
            'transform-runtime',
            'jsx-control-statements/babel'
          ]
        }
      },
      {
        test: /(\.scss|\.css)$/,
        loader: ExtractTextPlugin.extract('style', 'css?sourceMap&modules&importLoaders=1&localIdentName=[name]__[local]___[hash:base64:5]!postcss!sass?sourceMap!toolbox')
      }
    ]
  },

I was with same issue and as there is not much on traceback... I compared with another project's package.json and webpack config... I got that I just forgot.
npm install --save-dev css-loader toolbox-loader sass-loader style-loader

Just install 'react-style-proptype' into your node_modules


1460 - Make Two Arrays Equal by Reversing Sub-arrays

Description of the Problem
Given two integer arrays of equal length target and arr.
In one step, you can select any non-empty sub-array of arr and reverse it. You are allowed to make any number of steps.
Return True if you can make arr equal to target, or False otherwise.
Example 1:
Input: target = [1,2,3,4], arr = [2,4,1,3]
Output: true
Explanation: You can follow the next steps to convert arr to target:
1- Reverse sub-array [2,4,1], arr becomes [1,4,2,3]
2- Reverse sub-array [4,2], arr becomes [1,2,4,3]
3- Reverse sub-array [4,3], arr becomes [1,2,3,4]
There are multiple ways to convert arr to target, this is not the only way to do so.

Code
class Solution:
    def canBeEqual(self, target: List[int], arr: List[int]) -> bool:

Link To The LeetCode Problem
LeetCode

Kindly Assign issue to me, as there is no issue against this problem

I would be glad to contribute a solution @vJechsmayr

@vJechsmayr  kindly assign this to me i want to give it a try

@joy2306 kindly read Contributing guidelines.


Events in tokens pallet do not completely cover balance changes

Since not every change of token balance is indicated with events, it becomes immensely difficult to correctly track tokens balances in a block indexer.
I suggest to offer a complete set of events that does indicate any changes in token balances like it's done in the balances pallet.

Can you point out what's missing? PR are always welcome.

I think the only event that is missing is Slashed
/// Some amount was removed from the account (e.g. for misbehavior).
Slashed { who: T::AccountId, amount: T::Balance },

I can create the PR. As I see it now the event belongs into orml-currency.
I am just wondering why are some events like
		/// Currency transfer success.
		Transferred {
			currency_id: CurrencyIdOf<T>,
			from: T::AccountId,
			to: T::AccountId,
			amount: BalanceOf<T>,
		},
		/// Update balance success.
		BalanceUpdated {
			currency_id: CurrencyIdOf<T>,
			who: T::AccountId,
			amount: AmountOf<T>,
		},
		/// Deposit success.
		Deposited {
			currency_id: CurrencyIdOf<T>,
			who: T::AccountId,
			amount: BalanceOf<T>,
		},
		/// Withdraw success.
		Withdrawn {
			currency_id: CurrencyIdOf<T>,
			who: T::AccountId,
			amount: BalanceOf<T>,
		},

emitted in orml-currencies? I think it should just do the calls to the respective trait implementations specified by Config types MultiCurrency and NativeCurrency  and the implementations should be responsible for emitting the events. Otherwise multiple events that state the same will be emitted.

The original idea is that events are emitted by the extrinsic pallet (currencies), not the underlying implementation (tokens) so that people can decide what event to emit.
But I does see it could create some inconsistency.

I agree we should make it that only implementations emit events, and to cover all possible balance changes. In this way ORML users don't have to worry about adding events, and are always free to add custom ones if needed while wrapping these impl.
The current balance related events could be updated as:

Add Slashed event to orml-tokens.
In orml-tokens, deposit events in traits/pallet impl, instead of in dispatchable calls.
Remove events in orml-currencies, to get rid of events deposit duplication.

orml tokens now emits events whenever balance change occurs.
events are emitted in implementation level. Duplicate deposits are removed.
orml currencie's events have all been removed


ARROW-1383: [C++] Add vector append variant to primitive array builders that accepts std::vector

Other libraries may have null indicators in the form of bits or bytes. std::vector<bool> is a bit-packed container like Arrow's internal representation.

+1. I'll be happy to make any additional improvements in follow up JIRAs


seeing other users data after login/out with different users on ionic

I've run into what seems to me to be a pretty serious issue. I've got devise as well as device_token_auth setup side-by-side, as well as ng-token-auth for the ionic front-end. When logging in/out with different users everything is working fine as long as I login using regular devise or in my ionic project with ng-token-auth in a regular browser (localStorage is used for storing the tokens).
However, when running the ionic project on an android phone, when I log in through a first user and then out, then login with a second user I still see the first user's data. Looking at the rails logs I see that  for some requests actually the id of the previous user is used when using current_user.id in the query.
First the user logs in and the correct user id is select (id=20), so far so good:
2015-09-11T17:05:00.418741+00:00 app[web.1]: Started POST "/api/v1/auth/sign_in" for <IP> at 2015-09-11 17:05:00 +0000
[...]
2015-09-11T17:05:00.848609+00:00 app[web.1]:   User Load (4.4ms)  SELECT  "users".* FROM "users" WHERE "users"."id" = $1 LIMIT 1 FOR UPDATE  [["id", 20]]

Then immediately after login, ui-router invokes another state and another request is made using this query in the controller with current_user.id:
@votables = Votable.joins("LEFT JOIN votes ON votes.votable_id = votables.id AND user_id = #{current_user.id}").where("votes.id is NULL")

This triggers the following query for getting the current_user details:
2015-09-11T17:05:01.432071+00:00 app[web.1]: Started GET "/api/v1/votes.json" for <IP> at 2015-09-11 17:05:01 +0000
[...]
2015-09-11T17:05:01.448617+00:00 app[web.1]:   User Load (1.9ms)  SELECT  "users".* FROM "users" WHERE "users"."id" = $1  ORDER BY "users"."id" ASC LIMIT 1  [["id", 18]]

As you can see here somehow the results for another user with id=18 are returned instead of the expected details for user with id=20!!
The strange thing is that in parallel to this query I update the account with a token I received from the ionic platform that does use the right user id:
2015-09-11T17:05:03.542369+00:00 app[web.1]: Started PUT "/api/v1/auth" for <IP> at 2015-09-11 17:05:03 +0000
2015-09-11T17:05:04.115428+00:00 app[web.1]:   User Load (14.2ms)  SELECT  "users".* FROM "users" WHERE "users"."id" = $1 LIMIT 1 FOR UPDATE  [["id", 20]]

Even stranger to me is that, when I look at the request headers that the ionic app is sending to devise_token_auth for both these last two queries, that the client and access-token headers are exactly the same. I understand that this is to be expected due to the batch_request_buffer_throttle setting. However, I really don't understand how two queries with the exact same tokens can result in a different user id on the server side. BTW these tokens are the ones that were sent in the response to the sign_in request.
I will continue to try and research this myself but I would love some guidance with what seems to me to be a serious bug.
Thanks!

I see now that even before the sign in a request is being made by the client to validate the token from what I assume is the previous session:
Started GET "/api/v1/auth/validate_token" for <IP> at 2015-09-12 01:29:26 +0300
Processing by DeviseTokenAuth::TokenValidationsController#validate_token as HTML
  User Load (0.5ms)  SELECT  "users".* FROM "users" WHERE "users"."id" = ?  ORDER BY "users"."id" ASC LIMIT 1  [["id", 18]]

So my guess is that the source of this token is not properly updated on cordova/ionic after a sign_in. I'll try to trace it tomorrow.

@abhayastudios I'm having the exact same issue! After a previously logged in User1 logged out, User2 will get authenticated back in with User1's session!!
This is a really serious issue, I'm also searching for ways to solve it...

@ianchen06 does it happen to you also in a cordova/ionic project only? are you testing on android/iphone?

@abhayastudios I have not tested it on mobile yet, but it seems to be related to this pull request?
https://github.com/lynndylanhurley/devise_token_auth/pull/200
@lynndylanhurley Have you seen others reporting this issue? Thanks

When I run 'clear data' on android for the ionic application then the next login will work ok until I login with another user, so it is definitely something with the localStorage.

I'm using cookies and have also this issue (2nd user get previous user data).
From my understanding, there is two issues :

sign_out doesn't delete the cookies related to session handling
sign_out doesn't invalidate theses tokens

I have same issue in my angular web app. I just don't know what to do for my users. Some of my users access the same computer and things have become a real big mess. Manually deleting the _app_session before signing in with another user seems to resolve the issue but, they can't go on this path. Hope we get a solution soon.

I've currently reverted version 0.1.31 which works well.

I also had the same problem with logging user using email and oauth on the same device. In my rails console, I got this warning message, "Can't verify CSRF token authenticity", when trying to sign user out.
Started DELETE "/api/v1/auth/sign_out" for xxx.xxx.x.xxx at 2015-10-02 00:24:42 +0700
Processing by Users::SessionsController#destroy as JSON
Can't verify CSRF token authenticity
To solve my problem, I added :destroy in the exception in my session controller.
####################
#app/controllers/users/sessions_controller.rb
class Users::SessionsController < DeviseTokenAuth::SessionsController
protect_from_forgery except: [:create, :destroy]
def create
super
end
def resource_params
params.permit(devise_parameter_sanitizer.for(:sign_in), :format, :session)
end
end
####################
After that, everything is fine.

+1 to @stephenbaidu. I was experiencing this problem on a mobile project and reverting back 0.1.31 fixed it.

@mikeorr85 I also had to peg devise itself at 3.5.1 in order to resolve the error nameError (uninitialized constant DeviseTokenAuth::Concerns::User::BCrypt) which occurred in production.

Had the same problem. Turns out Rails CSRF protection prevented devise token auth from destroying the session:
Processing by Users::SessionsController#destroy as JSON
Can't verify CSRF token authenticity
Installed the ng-rails-csrf library and problem was resolved.
https://github.com/xrd/ng-rails-csrf/blob/master/vendor/assets/javascripts/ng-rails-csrf.js

Glad I took a second look at this. I was able to get passed my trouble while upgrading back to the latest version (0.1.36) simply by overriding the sessions controller as @makarakao originally posted.
routes.rb
  mount_devise_token_auth_for 'User', at: 'auth', controllers: {
    sessions: 'devise_token_auth_overrides/sessions'
  }

controllers/devise_token_auth_overrides/sessions_controller.rb
module DeviseTokenAuthOverrides
  class SessionsController < DeviseTokenAuth::SessionsController
    protect_from_forgery except: [:create, :destroy]
  end
end

Can you guys please try out version 0.1.37.beta1 and see if this has been resolved?

0.1.37.beta1 resolves the problem for me, and I no longer needed to override the SessionsController.

That's great! Thx @mikeorr85!!

Absolutely. Thank you for maintaining the gem!

@lynndylanhurley
I can confirm that with 0.1.37.beta2 the issue is resolved!! WOOHOO :)
Thanks a lot!

:smiley:

Fast-forward to 2018 and I had the same issue with version 0.1.42 in an Angular app (no Ionic involved though) - thanks to @makarakao and @ryankc33 I noticed that I was also getting the "Can't verify CSRF token authenticity" warning message.
Adding the except: [:create, :destroy] exception in the SessionsController did fix it, but I wanted to fix the actual root cause (being the failed CSRF token check). ng-rails-csrf did not work for me (I could see a token in the request headers but it still failed validation), however angular_rails_csrf did work.
In my case I didn't have to call $auth.invalidateTokens().


tf.keras.utils.plot_model with Sequential API

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 16.04
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 1.7.0
Python version: 2.7
Bazel version (if compiling from source): n/a
GCC/Compiler version (if compiling from source): n/a
CUDA/cuDNN version: n/a
GPU model and memory: n/a
Exact command to reproduce: See code/logs.

Describe the problem
tf.keras.utils.plot_model does not work with a model specified in the Sequential API.
Source code / logs
import tensorflow as tf

model = tf.keras.models.Sequential()
model.add(tf.keras.layers.Dense(10, input_dim=50))

tf.keras.utils.plot_model(model, "sequential.png")

Error tensorflow 1.7.0:
Traceback (most recent call last):
  File "plot_tf.py", line 6, in <module>
    tf.keras.utils.plot_model(model, "sequential.png")
  File "/Users/removed/.local/lib/python2.7/site-packages/tensorflow/python/keras/_impl/keras/utils/vis_utils.py", line 149, in plot_model
    dot = model_to_dot(model, show_shapes, show_layer_names, rankdir)
  File "/Users/removed/.local/lib/python2.7/site-packages/tensorflow/python/keras/_impl/keras/utils/vis_utils.py", line 123, in model_to_dot
    if node_key in model._container_nodes:
AttributeError: 'Model' object has no attribute '_container_nodes'

Error tensorflow 1.8.0.rc1
Traceback (most recent call last):
  File "plot_tf.py", line 6, in <module>
    tf.keras.utils.plot_model(model, "sequential.png")
  File "/Users/removed/.local/lib/python2.7/site-packages/tensorflow/python/keras/_impl/keras/utils/vis_utils.py", line 149, in plot_model
    dot = model_to_dot(model, show_shapes, show_layer_names, rankdir)
  File "/Users/removed/.local/lib/python2.7/site-packages/tensorflow/python/keras/_impl/keras/utils/vis_utils.py", line 80, in model_to_dot
    model = model.model
AttributeError: 'Sequential' object has no attribute 'model'

Note in tensorflow 1.8.0.rc1 the functional API appears to be working:
import tensorflow as tf

inputs = tf.keras.layers.Input(shape=(784,))
outputs = tf.keras.layers.Dense(10)(inputs)
model = tf.keras.models.Model(inputs=inputs, outputs=outputs)

tf.keras.utils.plot_model(model, "functional.png")

in tensorflow 1.7.0 the functional API has the same error as the Sequential API.

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


endWithParent inside a starts block needs it's own mode and starts mode deep cloned (with handlebars converted)

@yyyc514 I have rebased your branch onto the master and converted the function calls in the handlebars-language into variables. Works like a charm.
Replaces #2201

I'll review later when I have more focus, but looks about right!  Thanks!  And thanks for the deregister fix also... did you also want to yank the API from hilightjs code?

@egor-rogov There, now we have the change that this makes possible in the PR also, is that better for you?

@egor-rogov There, now we have the change that this makes possible in the PR also, is that better for you?

Almost (:
We don't need deregisterLanguage() anymore, do we?
I see your point on the test case and agree. The only thing test/bugs doesn't seem a good name to me. This is all about bugs. Maybe test/parser?

yes  It should be safe to remove now. That is a better name for sure

@nknapp Do you want to finish up or you want me to take this to the finish line?

It would be great if you could finish this.

There has got to be some easier way for two people to collaborate on one PR.

Pushing a new PR of my own now that I'm taking this back.

As a collaborator of the repo, you can push on my branch. The other way around is the difficult way

I didn’t see how.

You have to add the remote URL of my fork to your local clone. Then you can pushbto the PR branch. GitHub allows this when a PR author has checked "allow maintainers to push to my branch". But it only works if you have push access to the repo that the PR was created for, so I couldn't have pushed to your branch

At least I believe it works that way

So you're saying I could have pushed to your master? Interesting.  I almost considered just trying it... I will next time. :)

To "nknapp:deep_clone...", yes. I think you still can. But I'm not sure

At the time it said it was from your master IIRC.  But I could be mistaken.


Cleanup

This doesn't change the behaviour in any way, but removes unused data and adds a __main__.py so users can execute the script using python -m ....
Will add more tests before finalizing.

Codecov Report

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

@@            Coverage Diff            @@
##             master       #1   +/-   ##
=========================================
  Coverage          ?   87.37%
=========================================
  Files             ?        6
  Lines             ?      198
  Branches          ?        0
=========================================
  Hits              ?      173
  Misses            ?       25
  Partials          ?        0

Continue to review full report at Codecov.

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

Codecov Report

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

@@            Coverage Diff            @@
##             master       #1   +/-   ##
=========================================
  Coverage          ?   87.37%
=========================================
  Files             ?        6
  Lines             ?      198
  Branches          ?        0
=========================================
  Hits              ?      173
  Misses            ?       25
  Partials          ?        0

Continue to review full report at Codecov.

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


combined format unavailable

I am trying to get Apache to output using the combined format for my access log.
In my httpd.conf file I have this configuration for access logging:
<IfModule log_config_module>
    #
    # The following directives define some format nicknames for use with
    # a CustomLog directive (see below).
    #
    LogFormat "%h %l %u %t \"%r\" %>s %b \"%{Referer}i\" \"%{User-Agent}i\"" combined
    LogFormat "%h %l %u %t \"%r\" %>s %b" common

    <IfModule logio_module>
      # You need to enable mod_logio.c to use %I and %O
      LogFormat "%h %l %u %t \"%r\" %>s %b \"%{Referer}i\" \"%{User-Agent}i\" %I %O" combinedio
    </IfModule>

    #
    # The location and format of the access logfile (Common Logfile Format).
    # If you do not define any access logfiles within a <VirtualHost>
    # container, they will be logged here.  Contrariwise, if you *do*
    # define per-<VirtualHost> access logfiles, transactions will be
    # logged therein and *not* in this file.
    #
    #CustomLog /proc/self/fd/1 common
    #CustomLog /var/log/apache2/access.log common

    #
    # If you prefer a logfile with access, agent, and referer information
    # (Combined Logfile Format) you can use the following directive.
    #
    CustomLog /var/log/apache2/access.log combined
</IfModule>

I believe it's unchanged from the defaults.
However, when I run docker logs <apache-container>  I don't get the referer or user-agent information in the log entries (I also get duplicate lines for each request, but that's probably a separate issue).
What do I need to do to get that information output in the log entries?
Thanks.

When you specify /var/log/apache2/access.log, you end up with your logs going to a file inside the container -- here's the relevant CustomLog line from the image:
$ docker run --rm httpd grep '  CustomLog' conf/httpd.conf
    CustomLog /proc/self/fd/1 common

If you want to access the logs via docker logs, you'll want to replicate this so the logs go to the container's stdout instead of to a file.

Okay, thank you for posting that code. It helped me dig in a little deeper.
I know this might qualify as a separate issue, but my ultimate goal is to have the apache container write logs to files on the host using the combined format.
To this end I cloned the repo and commented out the lines that I thought were sending logs to stdout. And then I specified a bind mount for where I thought the container would be writing the logs, with the intention of being able to access these files on the host:
volumes:
      # log files
      - /path/on/host/apache_logs:/var/log/apache2

However, when I run the code you posted above on my image, I get the following output:
CustomLog "logs/access_log" common
I don't know where this configuration comes from. And when I remote into the container I can't find any directory or file named access_logs.
The only logs directory is at /usr/local/apache2/logs. And it only contains the httpd.pid file.
Thanks for any guidance you can provide.

I would suggest trying a dedicated support forum, such as the Docker Community Forums, the Docker Community Slack, or Stack Overflow for more help debugging the details of what's going wrong here.
If you want to completely disable our logging changes, you'll want to remove all three of these lines:
https://github.com/docker-library/httpd/blob/a7acc047c8970e0502f4c6b577775f840f1f99b6/2.4/Dockerfile#L205-L207
(Or revert them in your own configuration, which would allow you to avoid compiling httpd from source.)


Create eth57

name: 'Pull request'
about: A new pull request
New Pull Request
Checklist

[ ] My code is conform the code style
[ ] I have made corresponding changes to the documentation
[ ] I have updated Typescript definitions when needed

Issue Description

Related issue: #FILL_THIS_OUT
Solution Description

eth57

This PR doesn't provide sufficient details. Declining this.


Update NOTICE.txt

@grundleborg Kind reminder to review this.

/update-branch

Cherry pick is scheduled.

/cherry-pick cloud

Cherry pick is scheduled.


Publishing to App Store

Are there plans to publish this to App Store? Very soon (at least in EU) it will be possible to side load apps, so you'll be able to publish it to a different app store, and won't need to pay for App Store fee.

Yes, as soon as I don't have to pay Apple 99$/year, I'll publish the iOS version of OpenOTP.
But afaiu there will be many restrictions on sideloading, so we will see how smooth the process will be - check out for updates in nearest weeks/months

True, I'm also not sure what to expect yet. I think they will have alternative app stores. Maybe there will be a store for open source apps. Like F-Droid on Android. But sideloading will be allowed very soon like in 2 weeks I think.

I'll close this issue with proper reference as soon as it's implemented on master, so you get the update.

iOS 17.4 is out and it brings support for alternative App Stores! But I'm not sure if there is a nice App Store for open source apps. I'm not even sure which app stores exist.

Hello! Here is a Free (only 1.5 euros/year) and Open Source alternative AppStore where you can publish your app! https://fosstodon.org/@altstore/112287538562764439


fix(curriculum): Incomplete argument in test

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #50135

Thank you @Shenghan0329 for trying to solve the issue, the issue is solved by another PR, I am closing this one.


[0.9.x] [KOGITO-1407] Enable scenario 'Process instance events are stored in Data Index' with native enabled

https://issues.redhat.com/browse/KOGITO-1407
Many thanks for submiting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

[x] You have read the contributors guide
[x] Pull Request title is properly formatted: [KOGITO-XYZ] Subject
[x] Pull Request contains link to the JIRA issue
[x] Pull Request contains description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket
[ ] Your feature/bug fix has a unit test that verifies it
[x] You've tested the new feature/bug fix in an actual OpenShift cluster

works with 0.9.1-rc1 images

looks like circle ci still in pending state even with the job finished, restart it again, let's see.


Build error

Hi
When I try to build the extension I get the error
CSC : error CS2001: Source file '.._Build_\Version.cs' could not be found about six times.
I can see the file is mention in the csproj file, but not in the filesystem
How to fix it?

Hi,
this looks like an issue in our synchronization to GitHub. We will fix this in future updates. As a workaround you can create the Version.cs file yourself with the following content:
internal static class Version
{
    // assembly version number
    internal const string AssemblyVersion = "<IP_ADDRESS>";
}

Fixed


Correct and update Spanish and Catalan translations

This will probably be the last translation update I submit to CAS, for a few reasons:

Converting accented characters to Unicode escape sequences is tedious, cumbersome, and error-prone.
Unlike PO files, the original English string is not displayed next to the translated string.
Strings that have changed in the original English are not marked for review in the translation file.
In a few months I will no longer be using CAS.

Nevertheless, I hope that my efforts to provide accurate and up-to-date translations will help other CAS users in the future.

Thanks for taking time on this. Indeed, using Unicode characters is painful, but it's not mandatory if you don't forget to encode your files in UTF-8.
Be cautious with single quote, generally they don't display well: did you check the messages with single quote?

Do single quotes always have to be escaped, or only in messages that include {0} or {1}?

I think you're right, it must only be escaped in the messages with {x}.

I've converted both files to UTF-8 and escaped the apostrophes in messages that contain {0} or {1}.

+1

@alexhenrie thank you for your contributions. we hope you can keep it up albeit all the unicode complexity.


[Feature] default user role for distribution group

Frequently I create groups for users with higher permissions than "tester".  (such as a group of developers)  So, I want to be able to set the default user role for a distribution group.  When I add new users to a group, they should get the default role set for the group, unless I override it.
Alternatively, I can set all users with a custom role manually, but this gets rather repetitive.

Hi @tele-bird , thanks for your feedback. We will take into account for future prioritization.
Best regards,
Blanca


Update 2015-02-09-search-syntax.md

Have all of these changes been implemented separately? Can we close the ticket? Or do we have to revisit the changes and re-merge?

I added this in


Fix #12642, add line height for axispointer

Brief Information
This pull request is in the type of:

[x] bug fixing
[ ] new feature
[ ] others

What does this PR do?
Pass line height to getBoundingRect
Fixed issues
Close #12642
Details
Before: What was the problem?

After: How is it fixed in this PR?

Usage
Are there any API changes?

[ ] The API has been changed.

Related test cases or examples to use the new APIs
NA.
Others
Merging options

[ ] Please squash the commits into a single one when merge.

Other information

I try to fix the issue like this too, but i  thought the result seem set the height but no lineHeight,and i didn't find a good way to solve it.😁
Maybe the label should be centered vertically?

In fact, I did a test about this a few days ago and the result is that besides lineHeight, there are more properties are invalid for tooltip.axisPointer.label such as width, height,  textShadowColor, textBorderColor, textBorderWidth.
Maybe this PR can fix them in a row.

I try to fix the issue like this too, but i thought the result seem set the height but no lineHeight,and i didn't find a good way to solve it.😁
Maybe the label should be centered vertically?

@wf123537200  Woo, You were right, Didn't notice it,  :sweat_smile:

In fact, I did a test about this a few days ago and the result is that besides lineHeight, there are more properties are invalid for tooltip.axisPointer.label such as width, height, textShadowColor, textBorderColor, textBorderWidth.
Maybe this PR can continue to finish the TODO: rich at line 90 to fix them in a row.

Thanks for heads up, I will try

Hi~ @susiwen8 Thanks your fixing.
I've been inspired by following code lines from @Ovilia ~
Maybe we can make some changes like this
https://github.com/apache/incubator-echarts/blob/5a94212c43f0a4b5a8da7c26cf20192d29fb1d02/src/component/tooltip/TooltipRichContent.js#L139-L147
As mentioned above, there may be no need to solve this issue as present scheme.
var textRect = textContain.getBoundingRect(text, font, null, null, null, labelModel.get('lineHeight'));

Some changes are as follows
style: graphic.setTextStyle({}, labelModel, {
    text: text,
    fill: bgColor,
    stroke: labelModel.get('borderColor') || 'transparent',
    lineWidth: labelModel.get('borderWidth') || 0,
    textFont: font,
    textPosition: 'inside',
    textPadding: paddings,
    textFill: labelModel.getTextColor()
})

This can solve the issue that lineHeight is invalid and in the meanwhile, there are more supported properties of label.(Note that some new supported properties are now missing description in the official document)
label: {
    fontWeight: 'bold',
    fontSize: 20,
    textShadowBlur: 10,
    textShadowOffsetX: 10,
    textShadowOffsetY: 10,
    textShadowColor: 'red',
    backgroundColor: 'yellow',
    textBorderColor: 'blue',
    textBorderWidth: 5,
    borderWidth: 2,
    borderColor: 'cyan',
    padding: [10, 20, 30, 40],
    shadowColor: 'green',
    shadowBlur: 5,
    shadowOffsetX: 10,
    shadowOffsetY: 10,
    borderRadius: 20,
    lineHeight: 40,
    color: 'purple',
    // width: 400, // rich is needed, but rich is TODO
    // height: 200, // rich is needed, but rich is TODO
}

A test screenshot(Just for test, No beauty)

Looks good.
In the last, that's a pity rich label is still a TODO. I have no idea about it by now, but I think we can refer to implements of TooltipRichContent
Let we make it right together~ : )

@plainheart Would you submit a new PR for it?

Sorry, that's only my suggestions and I thought you would finish it. Now you have closed this issue, I have to open a new one for this later. Feel free to give some advice on rich label. Thank you!

Sorry, that's only my suggestion and I thought you would finish it. Now you have closed this issue, I have to open a new one for this later. Feel free to give some advice on rich label. Thank you!

:smiley :smiley :smiley I agree with you. You solution is the right way to fix this issue, no hard feeling. And to be honesty, I have ton of work to do, so I might not have time for it :joy: :joy: :joy: