Forward/back buttons too close together for replaying game

Hi, first off, awesome job with this project, I love this app and the fact that it's OSS.
Often times after finishing a game against the AI I like to go back and replay/analyze it move by move. The issue is these buttons are so small and close together, I'm often halfway through the game and accidentally tap on the >| button which means I have to go tap all the way back to where I was.

At the very least, I was thinking of spacing them out more, or perhaps even adding a scrubber / slider control too (although I recognize that having the individual forward/back buttons is useful).
Any thoughts on this? I'm happy to work on this once we agree on a solution. Also let me know if there's a way you prefer to track feature requests.

Did you realize you can tap on the small blue board position view in the lower-right corner?
Nevertheless, I agree that spacing the buttons out more would be a nice improvement, and it is certainly still within the scope of the next milestone (1.6.0). If you want to try your hand at this it would be an excellent introduction to the project.
I would really love to work with other people on the project, so your collaboration would be very welcome! From looking at your GitHub profile I don't have to tell you anything about pull requests 🙂, so go ahead and fork this repo. The quick-start instructions in README.Developer should hopefully get you started, if not then we can work the issues out via email or some other comms channel. A final note: The develop branch (which you should base your work on) is currently set to build with Xcode 12.3 and iOS SDK 14.3 - this is not yet reflected in README.Developer.
Scrubber/slider: Very good idea! However I see this as a new feature that should be described in a separate issue - go ahead and create a new issue if you like, or I'll do it myself when I get around to it in the next few days. When the issue is there I'll tag it with the "Feature" label, other than that I have no formal routine. I don't think the scrubber/slider feature will fit into 1.6.0 anymore, but let's discuss this in the new issue once it's there.

Did you realize you can tap on the small blue board position view in the lower-right corner?
Nevertheless, I agree that spacing the buttons out more would be a nice improvement, and it is certainly still within the scope of the next milestone (1.6.0). If you want to try your hand at this it would be an excellent introduction to the project.
I would really love to work with other people on the project, so your collaboration would be very welcome! From looking at your GitHub profile I don't have to tell you anything about pull requests 🙂, so go ahead and fork this repo. The quick-start instructions in README.Developer should hopefully get you started, if not then we can work the issues out via email or some other comms channel. A final note: The develop branch (which you should base your work on) is currently set to build with Xcode 12.3 and iOS SDK 14.3 - this is not yet reflected in README.Developer.
Scrubber/slider: Very good idea! However I see this as a new feature that should be described in a separate issue - go ahead and create a new issue if you like, or I'll do it myself when I get around to it in the next few days. When the issue is there I'll tag it with the "Feature" label, other than that I have no formal routine. I don't think the scrubber/slider feature will fit into 1.6.0 anymore, but let's discuss this in the new issue once it's there.

Did you realize you can tap on the small blue board position view in the lower-right corner?

Interesting! I never knew about that, I was always curious about it. Tapping on that button doesn't seem to do anything on my device (iPhone 8 on iOS 14.4). But I just got the project working on the simulator and I can see that board position UI now, which looks great. Seems like a bug then. I can open an issue and look into it given that I should be able to reliably reproduce it not triggering (assuming you can't).

I would really love to work with other people on the project, so your collaboration would be very welcome!

Great! Also, I'm turning on GitHub notifications - been some time since I am back to working on open source and care about them :)

A final note: The develop branch (which you should base your work on) is currently set to build with Xcode 12.3 and iOS SDK 14.3 - this is not yet reflected in README.Developer.

Yep, so building 3rd party tools failed because of this. Turns out I had Xcode 12.4 & the 14.4 SDK installed, and didn't want to downgrade Xcode/do some hacky things. Luckily the fix was pretty small, and I opened a PR for it, assuming you didn't want to lock it for a particular reason (can discuss there): https://github.com/herzbube/littlego/pull/351
Great documentation, by the way.

However I see this as a new feature that should be described in a separate issue

Agreed. I agree that it's not urgent also, especially if we can get this issue ironed out. I can add a new issue for it soon unless you beat me to it.

Tapping on that button doesn't seem to do anything on my device (iPhone 8 on iOS 14.4).

Right. I forgot that I found that bug and fixed it already. Apologies. So no need to open an issue, see #344 and the associated commit.

Great documentation, by the way.

Thanks. I put a lot of effort into it, mostly out of self-interest because I tend to forget things, but of course I always hope that someone else finds it useful, too. It's nice to hear that it does the job.

I can add a new issue for it soon unless you beat me to it.

Done, it's #353.
BTW: I enabled "Discussions" on this repo, so maybe we can take some of the talk there. I started this topic to discuss things related to the upcoming 1.6.0 release, such as whether a slider is in or out of scope.


jsoneditor widget

I am trying to develop a jsoneditor widget in order to have a json text input field powered by jsoneditor
I think that this widget may be of general interest in the surveyjs ecosystem. However I need some help to get it work
Here's the plunker I am working on. To start with the render has some problems... Can anyone help? Thank you

Managed to do that. It was a css conflict. I just wonder if that's the only way to make surveyJS and json editor work together. Have a look to the above plunk!

@masciugo Thank you for the valueble contribution! I'm impressed by the plunk with fixed CSS! Could you please specify - what do you mean under "to make surveyJS and json editor work together"? You've achieved this goal in your plunk, haven't you?

I always try to add less code as possible to make third-party libs work together.
I am not a very css expert but maybe there's not other way than unset outer rules for inner component. I don't know if exist an other smart way to write css rules (for surveyjs in this case) that is than easy to skip by inner components (jsoneditor)


Upgrade a bunch of ESLint packages

These are all an entangled mess so it's not worth trying to do them individually. Also I still need to look into the changelogs of all these deps to make sure they haven't dropped a version of Node we still care about.
If we're gonna drop Node 8 too (which I think we should) we should upgrade to eslint@7 instead of eslint@6. I tried it locally and it works on Node 14.
Fixes #404

I pushed the upgrade to eslint@7. We'd have to require at least Node 8.10 anyway and that's just silly.


Windows comfyui install bug (path cutnpaste)

There are multiple problems with installing on windows, with an existing comfy install.
I eventually gave up and just told it to install a fresh copy, so I dont know ALL the bugs. but one I do know about:
When it asks you for the location of the existing comfyui, a normal thing to do in windows is go to file explorer, click on the thing, and select (copy as path)
but the installer fails on that and complains about "does not end in py".
Yall need to automatically strip off  ""  enclosing windows paths.

Eh, sure, why not -- it now automatically trims stray quotes


cxx11 ABI compatibility

When trying to incorporate the library with other libraries such as spdlog and yaml-ccp I ran into issues with how string was treated due to ABI compatibility
If you update line 13 in the main CmakeLists.txt to:
add_compile_options("-Wall" "-Werror" "-Wextra" "-Wshadow" "-Wno-unused-result" "-Wabi-tag")
(including "-Wabi-tag") you can see these issues as well
In file included from /home/adam/dev/modern-cpp-kafka/include/kafka/RdKafkaHelper.h:5,
                 from /home/adam/dev/modern-cpp-kafka/include/kafka/Error.h:5,
                 from /home/adam/dev/modern-cpp-kafka/include/kafka/ConsumerCommon.h:5,
                 from /home/adam/dev/modern-cpp-kafka/include/kafka/KafkaProducer.h:5,
                 from /home/adam/dev/modern-cpp-kafka/examples/kafka_async_producer_not_copy_payload.cc:1:
/home/adam/dev/modern-cpp-kafka/include/kafka/Types.h: In member function ‘std::string kafka::ConstBuffer::toString() const’:
/home/adam/dev/modern-cpp-kafka/include/kafka/Types.h:46:31: warning: ‘kafka::ConstBuffer::toString() const::<lambda(unsigned char)>’ does not have the "cxx11" ABI tag that ‘std::ostringstream’ {aka ‘std::__cxx11::basic_ostringstream<char>’} (used in the type of ‘kafka::ConstBuffer::toString() const::<lambda(unsigned char)>::<oss capture>’) has [-Wabi-tag]
   46 |         auto printChar = [&oss](const unsigned char c) {
      |                               ^
/home/adam/dev/modern-cpp-kafka/include/kafka/Types.h:46:28: note: ‘kafka::ConstBuffer::toString() const::<lambda(unsigned char)>::<oss capture>’ declared here
   46 |         auto printChar = [&oss](const unsigned char c) {
      |                            ^~~

https://gcc.gnu.org/onlinedocs/libstdc++/manual/using_dual_abi.html
https://gcc.gnu.org/wiki/Cxx11AbiCompatibility
Is there a recommended approach to link with libraries that are using Cxx11Abi?
Currently I cannot build the a producer that uses spdlog nor yaml library

found work around nvm


stackoverflow in regexp_nfa.c addstate()

How to reproduce
vim -i NONE -u NONE -Z -e -c 'echo search("\\v((){328}){389}")' -c 'qa!'

with latest vim (v8.1.0897)
Program received signal SIGSEGV, Segmentation fault.
0x00005555556f537d in addstate (l=0x7fffffffb000, state=0x7fffee089840, subs_arg=0x7fffffffb160, pim=0x0, off_arg=<error reading variable: Cannot access memory at address 0x7fffff7feffc>) at regexp_nfa.c:4295
(gdb) bt 10
#0  0x00005555556f537d in addstate (l=0x7fffffffb000, state=0x7fffee089840, subs_arg=0x7fffffffb160, pim=0x0, off_arg=Python Exception <class 'gdb.MemoryError'> Cannot access memory at address 0x7fffff7feffc:
#1  0x00005555556f57b7 in addstate (l=0x7fffffffb000, state=0x7fffee0897f0, subs_arg=0x7fffffffb160, pim=0x0, off_arg=0) at regexp_nfa.c:4497
#2  0x00005555556f5ae3 in addstate (l=0x7fffffffb000, state=0x7fffee089818, subs_arg=0x7fffffffb160, pim=0x0, off_arg=0) at regexp_nfa.c:4597
#3  0x00005555556f5e14 in addstate (l=0x7fffffffb000, state=0x7fffee0897c8, subs_arg=0x7fffffffb160, pim=0x0, off_arg=0) at regexp_nfa.c:4697
#4  0x00005555556f57b7 in addstate (l=0x7fffffffb000, state=0x7fffee089778, subs_arg=0x7fffffffb160, pim=0x0, off_arg=0) at regexp_nfa.c:4497
#5  0x00005555556f5ae3 in addstate (l=0x7fffffffb000, state=0x7fffee0897a0, subs_arg=0x7fffffffb160, pim=0x0, off_arg=0) at regexp_nfa.c:4597
#6  0x00005555556f5e14 in addstate (l=0x7fffffffb000, state=0x7fffee089750, subs_arg=0x7fffffffb160, pim=0x0, off_arg=0) at regexp_nfa.c:4697
#7  0x00005555556f57b7 in addstate (l=0x7fffffffb000, state=0x7fffee089700, subs_arg=0x7fffffffb160, pim=0x0, off_arg=0) at regexp_nfa.c:4497
#8  0x00005555556f5ae3 in addstate (l=0x7fffffffb000, state=0x7fffee089728, subs_arg=0x7fffffffb160, pim=0x0, off_arg=0) at regexp_nfa.c:4597
#9  0x00005555556f5e14 in addstate (l=0x7fffffffb000, state=0x7fffee0896d8, subs_arg=0x7fffffffb160, pim=0x0, off_arg=0) at regexp_nfa.c:4697
#10 0x00005555556f57b7 in addstate (l=0x7fffffffb000, state=0x7fffee089688, subs_arg=0x7fffffffb160, pim=0x0, off_arg=0) at regexp_nfa.c:4497

This is found by fuzzing (libfuzzer)

I suppose that something like \(\)\{328} or \(\)* could be simplified to \(\) since it's a 0-length match.

Op di 12 feb. 2019 22:15 schreef Bram Moolenaar<EMAIL_ADDRESS>
Closed #3941 https://github.com/vim/vim/issues/3941 via 57ee2b6
https://github.com/vim/vim/commit/57ee2b6e0b5b730d12ee9db00a8e2a577df9e374
.

That commit looks totally unrelated to regular expressions...
Christ van Willegen


Resizing logo to fit better and adding SVG for download use

Current coverage is 100% (diff: 100%)

Merging #94 into master will not change coverage

@@           master    #94   diff @@
====================================
  Files          63     63
  Lines        1844   1844
  Methods         0      0
  Messages        0      0
  Branches        0      0
====================================
  Hits         1844   1844
  Misses          0      0
  Partials        0      0

Powered by Codecov. Last update 28fe182...363e3b5


Prevent crashes for opencv exception and std::exception

Description
fixes #8663 , fixes #9475, fixes #10389, fixes #13217
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)

Changes

[x] Added support for std::exception handling to prevent crash
[x] Added test

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

@mxnet-label-bot add [Bug, OpenCV, Exception Handling, pr-awaiting-review]

This is dependent on waitall PR: https://github.com/apache/incubator-mxnet/pull/14397. Please review that one first.

@anirudh2290 Can you rebase with the master branch and resolve merge conflicts ?

@piyushghai done ! this is ready for review.

@apeforest can you take a look too.

@larroy and @yuxihu . Thanks a lot for your review. I have addressed/responded to your comments.

I approved already.

@mxnet-label-bot update [pr-awaiting-merge]

Merged. Thank you!


[wast mode] Support some Wasm SIMD instructions

These keywords can be found at
https://github.com/WebAssembly/simd/blob/master/proposals/simd/BinarySIMD.md.

Ping for review. Does this change look okay? Is the regex change clear? Thanks!

I put this aside because of the initial "This is not complete yet, more to come" comment.

Yea that was misleading, because I really meant "there are a lot of SIMD instructions, so let me implement them over multiple PRs. This is not the full set, and hopefully small enough to review easily".
Sorry for the confusion, and thank you for merging!


Field updated_at is not updated when an embedded document is changed

Hi guys, I'm here again :P
I have two models in my application:
class Parent
  include Mongoid::Document
  include Mongoid::Timestamps

  field :name, :type => String
  embeds_many :children
end

class Child
  include Mongoid::Document
  field :name, :type => String
end

Then I have created an instance for Parent and an instance for Child, like this:
p = Parent.create :name => "Lucas"
c = Child.new :name => "Clara"

The next step was include c into p, but before I got updated_at from p:
updated_at = p.updated_at
p.children << c

Then I had updated child's name and saved it:
p.children.first.name = "Clarinha"
p.children.first.save!
p.save!

Now the problem happens.  And because of this, Mongoid does not generate a new updated_at value.
puts p.updated_at == updated_at # print true, but should be false.

Is it possible to solve this problem? If not, I'll have to do a uggly workaround here.
Thanks guys.

What version? Can you run this against master? (I thought I had a temp fix in for this.)

I'm using version 2.2.2. I'll try to run using master version

Yup, that's a bug, mate. Also, the parent's updated_at field isn't updated after relating it with the child.

(I confirmed this in 2.3.4 . I'll check against master in a sec.)

I run with 2.3.4 and the bug happened

This problem's happening on master, as of 8e5784881349.


[Accessibility][Name, Role, Value - Flow Chart]: Name is not defined for 'More Option' button.

User Experience
Users who rely on AT will face difficulty to interact with the webpage if the name of the controls are not defined properly. As well as if the arrow's name's are not defined how the AT dependent user will know the flow.
Test Environment
Browser: Microsoft Edge Canary Version: 82.0.425.0 (Official build)
OS build: 2004 (19564.1005)
Screen Reader: Narrator
URL: Bot Framework Composer
Prerequisite Here
Actual Result
Name of the "More Option" is not defined. As well as the name is not defined for the the arrow's and the end node.
Expected:
The name should be defined for all the controls and the arrows so that the user will know the flow of the arrow.
Note: Same issue is repro within 'Bot responses' and 'User Input. Refer attachmentMAS4.1.2_Bot responses_K4W (1).zip
Repro Steps:

Open the above mentioned URL in EDGE.
Select 'Design Flow' from left navigation.
Now, navigate to '+' button with tab in the flow chart.
Observe the issue.

MAS Impacted: MAS4.1.2
Attachment:

@cwhitten Issue still repro, cross checked with 'Master' botframework composer.

@ashish315 hi, I checked with master branch, there are name property in the more options , arrow and end node.

I am able to reproduce this issue. These nodes/button do not have aria-label assigned. This means that the AT does not have information to announce. data-name and name are not announced by AT.
As another note, please do not add aria-labels to the SVG. The SVG should be hidden from the AT (aria-hidden), and a parent div should have the label.

@Padmini171991 could you clarify the need for the end arrow + node to have a name defined? Is this because the non-visual user must know when reaching the end of the step editor? Thank you.

arrow +  node is a svg node, as the notice says: do not have label in svg node, so we can not add labels. @Padmini171991  @corinagum

@alanlong9278 So, is it by design???

@ashish315 I believe this is by design. With mouse (and keyboard) the arrow+ does not receive focus, so I think aria-label is unnecessary. @alanlong9278 please correct me if I am wrong. :)


ARM 1.13: readunalign redeclared, duplicate symbols

Two glitches installing 1.13/arm on Raspbian/arm7 (PI-4):

unaligned2.go:12:6: readUnaligned32 redeclared: "solved" by commenting out duplicate

Duplicate symbol (Acos): unable to debug/solve.  (As a workaround, Is it possible (or safe) to set the loader flag (StrictDupsWarnFlag) to warn instead of die?)

Details below.  Apologies if this is misconfiguration.  I wonder if the '+build' flags are working right, if the build/source assembly steps are broken, or if there were commits merged over.
What version of Go are you using (go version)?

$ go version
go version go1.13 linux/arm

Does this issue reproduce with the latest release?
Yes, using go1.13.linux-armv6l.tar.gz.
Rough install steps before getting to the duplicate symbols (in case they are in error):

tar xf to /usr/local as usual
Define temp dir: go env -w GOTMPDIR=$HOME/tempgo
go build reported buildutil missing, so go get golang.org/x/tools/go/buildutil, got readUnaligned error (see below)
Comment out redeclared readUnaligned[32|64], so go-get and go build produces the duplicate-symbol error.

What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="arm"
GOBIN=""
GOCACHE="${HOME}/.cache/go-build"
GOENV="${HOME}/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="arm"
GOHOSTOS="linux"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="${HOME}/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR="${HOME}/tempgo"
GOTOOLDIR="/usr/local/go/pkg/tool/linux_arm"
GCCGO="gccgo"
GOARM="6" # also tried 7
AR="ar"
CC="gcc"
CXX="c++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -marm -pthread -fmessage-length=0 -fdebug-prefix-map=${HOME}/tempgo/go-build270506164=/tmp/go-build -gno-record-gcc-switches"
GOROOT/bin/go version: go version go1.13 linux/arm
GOROOT/bin/go tool compile -V: compile version go1.13
uname -v: #1245 SMP Fri Jul 12 17:31:45 BST 2019
uname -a: Linux [machine] 4.19.58-v7l+ #1245 SMP Fri Jul 12 17:31:45 BST 2019 armv7l GNU/Linux

What did you do?
Same result compiling my code or yours, or this program:
package main

import ( "math")

func main() {
    if 0 < math.Acos(.5) {
        panic("correct!")
    }
}

What did you expect to see?
panic: correct!
What did you see instead?
2019/09/03 23:51:15 duplicate symbol math.Acos (types 1 and 1) in math and ${HOME}/.cache/go-build/12/12922fad69ac95432e97258bda1b1585274fb2aa40b439b006399ab19f1a293b-d(stubs_arm.o)

Note: same result after removing ${HOME}/.cache/go-build/ and ${HOME}/tempgo/* to force rebuild.
Message duplicate symbol... (types ...) is from objfile.go:
$ cd /usr/local/go/src
$ grep 'duplicate symbol' $(find . -type f -name \*.go) | sed -n /types/p
./cmd/link/internal/objfile/objfile.go:         log.Fatalf("duplicate symbol %s (types %d and %d) in %s and %s", s.Name, s.Type, t, s.File, r.pn)

Using go build -x -v hello.go, it appears this happens after building math, when loading another package.
## create b015/ space (line:87), build math there (line:105)
  87 mkdir -p $WORK/b015/

  94 cat >$WORK/b015/go_asm.h << 'EOF' # internal
  95 EOF
  96 cd /usr/local/go/src/math
  97 /usr/local/go/pkg/tool/linux_arm/asm -trimpath "$WORK/b015=>" -I $WORK/b015/ -I /usr/local/go/pkg/include -D GOOS_linux -D GOARCH_arm -gensymabis -o $WORK/b015/symabis ./asinh_stub.s ./cbrt_stub.s ./erf_stub.s ./pow_stub.s ./sinh_stub.s ./sqrt_arm.s ./stubs_arm.s ./tan_arm.s

 100 cat >$WORK/b015/importcfg << 'EOF' # internal
 101 # import config
 102 packagefile math/bits=$WORK/b016/_pkg_.a
 103 EOF
 104 cd /usr/local/go/src/math
 105 /usr/local/go/pkg/tool/linux_arm/compile -o $WORK/b015/_pkg_.a -trimpath "$WORK/b015=>" -p math -std -buildid wu3qd3hKoUFoFoXYmAc5/wu3qd3hKoUFoFoXYmAc5 -goversion go1.13 -symabis $WORK/b015/symabis -D "" -importcfg $WORK/b015/importcfg -pack -asmhdr $WORK/b015/go_asm.h -c=4 ./abs.go ./acosh.go ./asin.go ./asinh.go ./atan.go ./atan2.go ./atanh.go ./bits.go ./cbrt.go ./const.go ./copysign.go ./dim.go ./erf.go ./erfinv.go ./exp.go ./expm1.go ./floor.go ./frexp.go ./gamma.go ./hypot.go ./j0.go ./j1.go ./jn.go ./ldexp.go ./lgamma.go ./log.go ./log10.go ./log1p.go ./logb.go ./mod.go ./modf.go ./nextafter.go ./pow.go ./pow10.go ./remainder.go ./signbit.go ./sin.go ./sincos.go ./sinh.go ./sqrt.go ./tan.go ./tanh.go ./trig_reduce.go ./unsafe.go

## + stubs
 241 cd /usr/local/go/src/math
 248 /usr/local/go/pkg/tool/linux_arm/asm -trimpath "$WORK/b015=>" -I $WORK/b015/ -I /usr/local/go/pkg/include -D GOOS_linux -D GOARCH_arm -o $WORK/b015/stubs_arm.o ./stubs_arm.s

 252 /usr/local/go/pkg/tool/linux_arm/pack r $WORK/b015/_pkg_.a $WORK/b015/asinh_stub.o $WORK/b015/cbrt_stub.o $WORK/b015/erf_stub.o $WORK/b015/pow_stub.o $WORK/b015/sinh_stub.o $WORK/b015/sqrt_arm.o $WORK/b0
 253 /usr/local/go/pkg/tool/linux_arm/buildid -w $WORK/b015/_pkg_.a # internal

## detect duplicates
 978 cd /usr/local/go/src/go/scanner
 979 /usr/local/go/pkg/tool/linux_arm/compile -o $WORK/b061/_pkg_.a -trimpath "$WORK/b061=>" -p go/scanner -std -complete -buildid cJNf_nnBE7XiKYX5kpa2/cJNf_nnBE7XiKYX5kpa2 -goversion go1.13 -D "" -importcfg
 980 /usr/local/go/pkg/tool/linux_arm/buildid -w $WORK/b048/_pkg_.a # internal

 982 2019/09/04 01:27:49 duplicate symbol math.Acosh (types 1 and 1) in math and $WORK/b015/_pkg_.a(stubs_arm.o)

Possible cause: files not removed after consolidating stubs?

commit to remove dups
Apr 23 math stubs consolidated

These commits appear to remove the .s files (including the asin_arm.s with the Acos symbol), but they still exist in the arm distribution.  However, this may be a red herring since removing them does not solve the problem.
Initial issue go get for buildutil reporting readUnaligned[32|64] redeclared
tools $ go get golang.org/x/tools/go/buildutil
# runtime
/usr/local/go/src/runtime/unaligned2.go:12:6: readUnaligned32 redeclared in this block
    previous declaration at /usr/local/go/src/runtime/alg.go:321:40
/usr/local/go/src/runtime/unaligned2.go:17:6: readUnaligned64 redeclared in this block
    previous declaration at /usr/local/go/src/runtime/alg.go:329:40

Solution: commented out declaration in alg.go in favor of arm-specific code in unaligned2.go.
Variants tried for duplicate symbols issue; OS detail

With and without GO111MODULE
Using 6 or 7 as GOARM value (go distribution indicates 6, but uname reports arm7l).
fyi, although Raspberry PI-4 is a 64-bit processor (BCM2711 is an Arm A72), Raspbian is a 32-bit OS.
I have tried wiping $GOROOT and re-installing - same result.

These commits appear to remove the .s files (including the asin_arm.s with the Acos symbol), but they still exist in the arm distribution.

Where are they? I unpacked go1.13.linux-armv6l.tar.gz from https://golang.org/dl/ on my pc, there is not asin_arm.s file in the archive. If you see a asin_arm.s in your /url/local/go/src/math/ dir, you borked the installation in some way and you now have a weird hybrid of 1.12 and 1.13.

Sorry, not a bug - confirmed pilot error.


Realtime Hotfixes

The default config for auth scope was wrong, changed to 'profile' and 'email'
CORS config for socket.io was not being passed down properly, updated accordingly

Might need to use als.runWith, or requestContext. With requestContext, maybe something like this would work?
declare module 'fastify-request-context' {
  interface RequestContextData {
    USED_BY_EZB: boolean
    IGNORE_RULES: boolean
  }
}

import { requestContext } from 'fastify-request-context'

export function setContext<T extends REALTIME>(key: T, value: AllowedValues[T]) {
    requestContext.set(key, value)
}

the runWith context is provided by either socket.io or fastify request, so there is no issue there, otherwise the tests would be failing
Also nested runWith will result in the outer context being lost. However, if any socket.io emits occur during a fastify request, it is to a different user, so its fine that the previous context is no longer used.


Is deleteStorage no problem to use?

Hi. TypeScript SDK calls deleteStorage to delete the attachment file from Hexabase.
https://github.com/hexabase/hexabase-js/blob/3c8176e4a3dae79c2c6e14edc13a372d03d2a1b6/src/lib/graphql/storage.ts#L13
I've called it, their show attachments count on the admin page even though there is no attachment file.

When I click the row, they show only the attachment icon. But there is no attachment file.

Normally, they show a "no attachment" label, if they have no attachment files like below.

So I think deleteStorage function makes the wrong data.

When I called datastoreDeleteItemFileAttachment function, I got same issue.

I resolved this issue. I updated the item data removing the file id that was deleted.


Added links for all social media

Related Issue

Issue No. #246

Changes you made :

Provided the social media links as you wanted

Describe the changes :

Links to the social media platforms
Check list :

[x] I have followed code style of this project.
[x] I have read the CONTRIBUTING document.
[x] All the aspects mentioned in the issue are covered.
[x] The website is properly working after my changes.
[x] 🌟 ed the repo

Type of change :

[x] New feature (non-breaking change which adds functionality)

Output Screenshots :

For Twitter

@SaiSantosh2209 It's good
But give some proper title to the PR.
There are total 4 cards and you have changed only card's social media links.
Plz change remaining 3 cards links.

@Arun9739 I have added all the social media links for the remaining cards as you mentioned. Please review my PR.

@SaiSantosh2209 Good to go
LGTM 🌟


Optimization

Coverage decreased (-7.3%) to 60.528% when pulling 7bd1eb92afb9659b9f69d8f751f98561819ba535 on optimization into 7e22f052e23fe346013fc25b1ac32cc7931be459 on master.

Coverage decreased (-7.3%) to 60.528% when pulling 9a0ed4f9405bbdb9230336cd289afacd88349a07 on optimization into 7e22f052e23fe346013fc25b1ac32cc7931be459 on master.

Coverage decreased (-7.0%) to 60.845% when pulling 410745461c60a7ebf8a478a3081e1288a1c049ad on optimization into 7e22f052e23fe346013fc25b1ac32cc7931be459 on master.

Coverage decreased (-7.2%) to 60.638% when pulling a4bcf7d7a6e645abd456220a1d7928b031fb1d89 on optimization into 7e22f052e23fe346013fc25b1ac32cc7931be459 on master.

Coverage decreased (-7.0%) to 60.857% when pulling 9fe1099cb1680dc67e063ed29d36009bf8b44c0d on optimization into 7e22f052e23fe346013fc25b1ac32cc7931be459 on master.

Coverage decreased (-3.4%) to 64.388% when pulling 6619cd978ee69169bce0fe5405c530853cd1113a on optimization into 7e22f052e23fe346013fc25b1ac32cc7931be459 on master.


Create placeholder to supply external type definitions

Create IResourceTypeProvider as an extension point for types

public class ResourceTypeReference

We will need to support the optional provider prefix at some point when parsing type refs. Not needed now since there's only one provider, though.

Refers to: src/Bicep.Core/Resources/ResourceTypeReference.cs:12 in cb8668f. [](commit_id = cb8668f75724d23e104187def8666aeadde73fa9, deletion_comment = False)

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@8cb8f24). Click here to learn what that means.
The diff coverage is 84.07%.

@@            Coverage Diff            @@
##             master     #553   +/-   ##
=========================================
  Coverage          ?   93.69%
=========================================
  Files             ?      230
  Lines             ?     8242
  Branches          ?        0
=========================================
  Hits              ?     7722
  Misses            ?      520
  Partials          ?        0

Flag
Coverage Δ

#dotnet
93.69% <84.07%> (?)

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

Impacted Files
Coverage Δ

src/Bicep.Core/Extensions/EnumerableExtensions.cs
75.00% <0.00%> (ø)

...Bicep.Core/TypeSystem/Az/AzResourceTypeProvider.cs
37.50% <37.50%> (ø)

...p.Core.UnitTests/Utils/TestResourceTypeProvider.cs
66.66% <66.66%> (ø)

src/Bicep.Core/TypeSystem/ResourceTypeRegistrar.cs
80.00% <80.00%> (ø)

src/Bicep.Cli.IntegrationTests/ProgramTests.cs
100.00% <100.00%> (ø)

src/Bicep.Cli/Program.cs
97.29% <100.00%> (ø)

...Core.IntegrationTests/Emit/TemplateEmitterTests.cs
100.00% <100.00%> (ø)

...IntegrationTests/SemanticModel/CompilationTests.cs
100.00% <100.00%> (ø)

...tegrationTests/SemanticModel/SemanticModelTests.cs
96.47% <100.00%> (ø)

src/Bicep.Core.Samples/ExamplesTests.cs
100.00% <100.00%> (ø)

... and 252 more


fix PHPPhotoLibrary error didFinishSavingWithError will never be triggered

I had a bunch of fixes in a larger PR, but I'll bring them in one by one.
In this case, the UIAlertController is only triggered if error == nil, so it will never fire.

@LeviDahl Thank you!


Filter for instructors that taught remotely

I think that we already have a small number of workshop that at least one instructor taught remotely. Can we have a filter for the instructors that taught remotely?

As far as I remember, we don't mark instructor tasks as remote.
There are two ways to solve this:

add role "RemoteInstructor" (we already have "Instructor", "Helper", etc.)
add "remote" boolean field to the Task / Role object

Let's defer this for now.


import vmq_server record to elixir plugin

I want to use vmq_reg:publish into my elixir plugin. But it need #vmq_msg record on the fourth argument (look at https://github.com/vernemq/vernemq/blob/1.7.1/apps/vmq_server/src/vmq_reg.erl#L302).
Below is my plugin code.
defmodule ChatAcl do
  @moduledoc false
  require Logger
  require Record

  Record.defrecord(:vmq_msg, Record.extract(:vmq_msg, from_lib: "vmq_server/src/vmq_server.hrl"))

...

and it throw error.
(ArgumentError) lib file vmq_server/src/vmq_server.hrl could not be found
    (elixir) lib/record/extractor.ex:41: Record.Extractor.from_lib_file/1
    (elixir) lib/record/extractor.ex:18: Record.Extractor.from_or_from_lib_file/1
    (elixir) lib/record/extractor.ex:5: Record.Extractor.extract/2

I also copy this vmq_server.hrl file manually and update path to /Users/name/vmq/ext/chat_acl/lib/vmq_server.hrl and it doesn't work. Any idea how to use vmq_reg:publish properly?
I'm avoiding to use {RegisterFun, PublishFun, SubscribeFun} and prefer vmq_reg:publish as I need to publish to other ClientId.

@pewh thanks... not sure I understand the issue
pinging @dergraf: what do you reckon?

@pewh vmq_reg:publish is internal and shouldn't be used. Use the mentioned {RegisterFun, PublishFun, SubscribeFun} instead. Moreover the used vmq_msg record is also internal and shouldn't be used. Also, the vmq_server.hrl is internal and shouldn't be used.
Can you elaborate more about what vmq_reg:publish provides that you can't do with the PublishFun ?

hello @dergraf
my use case is if user triggered on_registered, vernemq will automatically publish to other users (some client ids). If using Publish, the code in elixir should be:
PublishFun("topic/xyz", "the payload", %{ qos: 1 }) # where is parameter for client id?

on https://github.com/vernemq/vernemq/blob/1.7.1/apps/vmq_server/src/vmq_reg.erl#L715, I cannot setup client id if using PublishFun.

You don't need the client id. The PublishFun will publish a message to every subscriber that matches the provided Topic.

Thanks for the response. PublishFun is working as expected, except it doesn't trigger with auth_on_publish and on_publish hook. Is there a way to trigger those after PublishFun?

no. that's currently not possible out of the box. You'd need to call the vmq_plugin:all_till_ok(auth_on_publish, ListOfHookArgs) inside your plugin but this stuff is again very much internal.


Project removed from entry when syncing

to reproduce:

start a time entry / edit a recently created one
assign a project to it
wait for the app to sync
the project will be removed, time entry is synced to sever without project

happens on windows, mac and linux as well

Some related cases #212199, #212213

Seems like a lib issue. The data coming back from time entry creation is overriding the new update data.

Same thing happening to me on a Mac. I keep updating the project for a task and in a few seconds it gets cleared.
I wonder if the fix will be able to put back the projects for all tasks without project or we have to update them again manually.

This started to happen yesterday. (I can confirm that this is a problem)

Same here (OSX) 7.2.150
http://take.ms/fhx4w

My previous toggles are reset of their project randomly, it's really annoying.

Had to switch to web interface, can't use desktop app at all :-1:

Maybe this is API related? I have OSX version 7.2.302,@dandaka has 7.2.150 so it's not about the client.
We need a fix soon, if @Marwzoor is right we may loose past toggls as well and this would be really ugly.

Small update: We are busy looking into this problem (and it indeed looks backend/api related).
There is no indication that the issue will affect time entries in any way other than messing up projects right now, so hopefully the damage will be limited.
We hope we can fix this soon and are very sorry for the inconvenience.
Thanks for your patience!

websocket return was fubar, fixed and should be syncing fine now.

@refiito indeed it seems to be working fine now. I'll get back with feedback in a few hours if anything changes


Checks for local Username when returning ContactUsername

Fixes #388.

Here ya go @asutula!

oops!

oh Carson, you should bump version in core/main.go, package.json, and mobile/package.json

So this contact username will find it's way into all the different types we return from the api that include a username property?

Yes, this is the only place that (for example) ThreadFilesInfo, ThreadCommentInfo, and ThreadLikesInfo ever get this information...

Peeeeerfect. Thanks!


Add new func for plotting fmask data with automatic colours

Proposed changes
Add new func for plotting fmask data with automatic colours. This matches the existing plot_wo function by @MatthewJA

Forgot to include an example:

Shenanigans for dealing with xarray...


"Received invalid first login packet!"

Basically I use a fork of bungeecord, aegis to be more specific(It worked perfectly), today I updated the fork and I started getting this error message spammed in my console. I thought it was an error from the fork but after I searched a lot and tried many thing I reached to the conclusion that is an luckperms error, because the error disapears when I remove that plugin. I tried putting the plugin in a clean server, but I got the same thing again, I dont know what does it implies or what should I do, it makes my players get the "disconected" when they are joinin thru the bungeecord. Any Idea?

I suggest to use either the original BungeeCord or Waterfall, as there is no need to use yet another (rather unknown) fork of BungeeCord, that might change stuff in bad ways.

I actually use it because I need it, my server is a cracked one(No premium server), Aegis fork allow us to make a auth in a virtual server created in the bungeecord, so we dont need to buy other server just for an auth, also gives an antiBot and a AntiVPN/Proxy, so is very useful for my network.

Might be a similar issue as #2143

I checked more deeper and it was a problem with the configuration of the fork. It didnt allow that much packets, already fixed, not a LuckPerms issue.


Halide install puts some source code in tools directory

Describe the bug
vcpkg halide install puts additional source files in the directories/folders:

vcpkg-root/installed/x64-windows/tools
vcpkg-root/installed/x64-windows/debug/tools
files are as follows:

GenGen.cpp
halide_benchmark.h
halide_image.h
halide_image_info.h
halide_image_io.h
halide_malloc_trace.h
halide_trace_config.h
RunGen.h
RunGenMain.cpp

Environment

OS: [Microsoft Windows [Version 10.0.18363.753]]
Compiler:

Visual Studio 2017 Version: 15.9.21  VS Tools Version: 14.16.27023
Visual Studio 2019 Version: 16.5.2  VS Tools Version: 14.25.28610
WinSDK 10.0.18362.0
cmake version 3.17.0
git version 2.26.0.windows.1
Vcpkg package management program version 2020.02.04-nohash-external

To Reproduce
Steps to reproduce the behavior:

./vcpkg install halide[core,opencl,opengl,utils]:x64-windows
No errors created

Expected behavior
I do not think these files should not be here:

either not at all left in buildtrees
or in another source directory/folder

Failure logs
None
Additional context
None

Hi @timautry
Thanks for posting this issue.
These files are installed by upstream source.
https://github.com/halide/Halide/blob/master/CMakeLists.txt#L461-L472.
why do you think these files should not be installed to tools folder?

Creates confusion as to what port they belong

vcpkg does not build nor use the "RunGen" tool which these headers/sources/scripts belong

They should at least be grouped into a tools/halide sub-folder with the halide "RunGen" tool binary

No other tools built by other vcpkg ports have their headers/sources/scripts stored in the tools directory/folder except boost-build which has its own tools/boost-build sub-folder, and the boost-build tool b2.exe is built and used by vcpkg as vcpkg needs b2.exe to build the boost libraries

Headers and source should not be stored in the same directory/folder designed for binaries

Final thought: If vcpkg is not going to build the "RunGen" tool the files serve no purpose and should have the portfile run a patch against the CMakelists.txt as has been done in countless other ports to either:

eliminate copying these files into the root tools folder -or-
as a minimum put them in a tools/halide sub-folder -or-
build the RunGen tool and copy only the binary to the tools folder -or-
a combination of the above

See the reference on the RunGen tool and its use: https://github.com/halide/Halide/blob/master/README_rungen.md

Thanks for your details and suggestions about this issue.
Since these files are not used in vcpkg, it might be better to remove these.
I will submitted a PR to handle this.

I close, no longer use product.


Wireframe turn based game

[ ] two players on board move independently
[ ] Each movement sends position updates to server
[ ] Server sends game state to clients
[ ] Client is Input only (player movement)

Basic Server Model

I'll do the front-end pygame window

Completed by #26 and #27


Cow: Watch itrade-earners.net

Cow requests the watch of the watch_keyword itrade-earners\.net. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

itrade-earners\.net has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


[ci] Dask tests on Linux failing

Description
Created from https://github.com/microsoft/LightGBM/pull/4283#issuecomment-840735651.
Python CI jobs in many Linux environments are failing with the following error for all Dask tests.
>       from distributed.protocol.core import dumps_msgpack
E       ImportError: cannot import name 'dumps_msgpack' from 'distributed.protocol.core' (/root/miniconda/envs/test-env/lib/python3.7/site-packages/distributed/protocol/core.py)

examples:

GitHub Actions (CUDA): https://github.com/microsoft/LightGBM/pull/4283/checks?check_run_id=2577306748
Azure: https://dev.azure.com/lightgbm-ci/lightgbm-ci/_build/results?buildId=10070&view=logs&j=fb919173-48b9-522d-5342-9a59a13eb10b&t=7fcbeee5-d4c1-571c-8072-1d5874591254

Additional Comments
It seems dumps_msgpack was removed in distributed version 2021.04.1, in https://github.com/dask/distributed/pull/4677.
https://github.com/dask/distributed/blob/4637099e2548a963197fdcc04e563401f77adef5/docs/source/changelog.rst#2021041
So I think the root issue is that dask 2021.4.0 is not compatible with distributed 2021.4.1.
This looks very similar to the situation https://github.com/microsoft/LightGBM/pull/4054, where incompatible versions of dask and distributed are uploaded to the official anaconda channel several days apart.
I can see the following versions getting installed in the failing builds:
dask               pkgs/main/noarch::dask-2021.4.0-pyhd3eb1b0_0
dask-core          pkgs/main/noarch::dask-core-2021.4.0-pyhd3eb1b0_0
distributed        pkgs/main/linux-64::distributed-2021.4.1-py39h06a4308_0
...
dask-2021.4.0              |     pyhd3eb1b0_0           5 KB
dask-core-2021.4.0         |     pyhd3eb1b0_0         670 KB
distributed-2021.4.1       |   py37h06a4308_0         1.0 MB

I'm not sure how we're ending up with distributed 2021.4.1 though, or why this just started breaking. It seems dask, dask-core, and distributed haven't been updated in a few weeks, and the latest version for all of them is 2021.4.0.
https://anaconda.org/anaconda/dask

https://anaconda.org/anaconda/dask-core

https://anaconda.org/anaconda/distributed

I do see that distributed was updated to 2021.4.1 on conda-forge about 9 hours ago. Are this project's CI jobs somehow getting that version from conda-forge?
https://anaconda.org/conda-forge/distributed

I've just restarted the failing Linux_Latest builds on https://dev.azure.com/lightgbm-ci/lightgbm-ci/_build/results?buildId=10070&view=results (from #4283 ).
If they fail with the same error as documented above, that would rule out the possibility that distributed 2021.4.1 was uploaded to the main Anaconda channel and then taken down. I think that's very unlikely but just trying to narrow things down.

These jobs failed with the same error: https://dev.azure.com/lightgbm-ci/lightgbm-ci/_build/results?buildId=10070&view=results

I tried to replicate this with docker tonight, running the following command from the root of the repo.
docker run \
  -v $(pwd):/opt/LightGBM \
  -w /opt/LightGBM \
  --env COMPILER=clang \
  --env BUILD_DIRECTORY=/opt/LightGBM \
  --env DEBIAN_FRONTEND='noninteractive' \
  --env IN_UBUNTU_LATEST_CONTAINER='true' \
  --env OS_NAME='linux' \
  --env SETUP_CONDA='true' \
  --env PYTHON_VERSION=3.6 \
  --env TASK=regular \
  -it ubuntu:latest \
  /bin/bash

apt update -y
apt install -y sudo

export CONDA=${HOME}/miniconda
export LGB_VER=$(head -n 1 VERSION.txt)
export PATH=${CONDA}/bin:${PATH}

Here is some of the output conda config --show. I don't see conda-forge listed.
channels:
  - defaults
custom_channels:
  pkgs/main: https://repo.anaconda.com
  pkgs/r: https://repo.anaconda.com
  pkgs/pro: https://repo.anaconda.com
custom_multichannels:
  defaults:
    - https://repo.anaconda.com/pkgs/main
    - https://repo.anaconda.com/pkgs/r
  local:
default_channels:
  - https://repo.anaconda.com/pkgs/main
  - https://repo.anaconda.com/pkgs/r

full output of 'conda config'
add_anaconda_token: True
add_pip_as_python_dependency: True
aggressive_update_packages:
  - ca-certificates
  - certifi
  - openssl
allow_conda_downgrades: False
allow_cycles: True
allow_non_channel_urls: False
allow_softlinks: False
always_copy: False
always_softlink: False
always_yes: True
anaconda_upload: None
auto_activate_base: True
auto_stack: 0
auto_update_conda: True
bld_path:
changeps1: False
channel_alias: https://conda.anaconda.org
channel_priority: flexible
channels:
  - defaults
client_ssl_cert: None
client_ssl_cert_key: None
clobber: False
conda_build: {}
create_default_packages: []
croot: /root/miniconda/conda-bld
custom_channels:
  pkgs/main: https://repo.anaconda.com
  pkgs/r: https://repo.anaconda.com
  pkgs/pro: https://repo.anaconda.com
custom_multichannels:
  defaults:
    - https://repo.anaconda.com/pkgs/main
    - https://repo.anaconda.com/pkgs/r
  local:
debug: False
default_channels:
  - https://repo.anaconda.com/pkgs/main
  - https://repo.anaconda.com/pkgs/r
default_python: 3.8
default_threads: None
deps_modifier: not_set
dev: False
disallowed_packages: []
download_only: False
dry_run: False
enable_private_envs: False
env_prompt: ({default_env})
envs_dirs:
  - /root/miniconda/envs
  - /root/.conda/envs
error_upload_url: https://conda.io/conda-post/unexpected-error
execute_threads: 1
extra_safety_checks: False
force: False
force_32bit: False
force_reinstall: False
force_remove: False
ignore_pinned: False
json: False
local_repodata_ttl: 1
migrated_channel_aliases: []
migrated_custom_channels: {}
non_admin_enabled: True
notify_outdated_conda: True
offline: False
override_channels_enabled: True
path_conflict: clobber
pinned_packages: []
pip_interop_enabled: False
pkgs_dirs:
  - /root/miniconda/pkgs
  - /root/.conda/pkgs
proxy_servers: {}
quiet: False
remote_backoff_factor: 1
remote_connect_timeout_secs: 9.15
remote_max_retries: 3
remote_read_timeout_secs: 60.0
repodata_fns:
  - current_repodata.json
  - repodata.json
repodata_threads: None
report_errors: None
restore_free_channel: False
rollback_enabled: True
root_prefix: /root/miniconda
safety_checks: warn
sat_solver: pycosat
separate_format_cache: False
shortcuts: True
show_channel_urls: None
signing_metadata_url_base: https://repo.anaconda.com/pkgs/main
solver_ignore_timestamps: False
ssl_verify: True
subdir: linux-64
subdirs:
  - linux-64
  - noarch
target_prefix_override:
track_features: []
unsatisfiable_hints: True
unsatisfiable_hints_check_depth: 2
update_modifier: update_specs
use_index_cache: False
use_local: False
use_only_tar_bz2: False
verbosity: 0
verify_threads: 1
whitelist_channels: []

Aha! Looking at repo.anaconda.com, I do see see distributed 2021.04.1 was uploaded earlier today.
https://repo.anaconda.com/pkgs/main/linux-64/

So I guess I misunderstood the relationship between those web pages like https://anaconda.org/anaconda/distributed or https://anaconda.org/main/distributed and what the "default" channels are.

Given that we have agreed in #4054 not to use conda-forge, I think the options to fix these CI jobs are:

Just wait until dask 2021.4.1  is released to the default conda channels

NOTE: it was released to PyPI and conda-forge April 23, not sure how long it will take to get to the default conda channels

pin distributed and dask to 2021.4.0 in CI

@StrikerRUS which option do you prefer? Or is there some other option you can think of that I haven't?

Given that dask and distributed are tight to each other and in fact should be installed with the same version number, I think we should pin their version at our CI and bump it from time to time.
Another option will be switching to get them from PyPI directly. But this option should be investigated for time consumption and possible dependency conflicts. And of course this is suitable only in case these packages are updated simultaneously at PyPI, otherwise we gain nothing from switching to PyPI.

alright. I'll pin them for now to unblock CI but we can leave this repo open to track the work of maybe switching to PyPI.
dask and distributed are released very frequently, as you mentioned, so I'd like to avoid the maintenance work of updating pins if possible.
I've also opened https://github.com/dask/distributed/issues/4819 asking the distributed maintainers if they can influence when these packages are published on the Anaconda main channels.

I'll pin them for now to unblock CI but we can leave this repo open to track the work of maybe switching to PyPI.

Agreed.

so I'd like to avoid the maintenance work of updating pins if possible.

We can create requirements_dask.txt and setup dependabot to update this file if we end up using pip later.

ha sorry I meant "leave this issue" open, not "leave this repo". Typing too fast this morning.

We can create requirements_dask.txt and setup dependabot to update this file if we end up using pip later.

Oh, great idea! I've definitely seen a setup like that be successful in other projects.


.Net: Support function-calling content classes for Gemini non-streaming API

Two new types - FunctionCallRequestContent and FunctionCallResultContent have been added to support connector-agnostic function calling model for non-streaming API: Requirements: https://github.com/microsoft/semantic-kernel/issues/5153. PR: https://github.com/microsoft/semantic-kernel/pull/5800
This item tracks the work required to add a connector-agnostic, function-calling model for the non-streaming API to the Gemini AI connector.

Adopt M.E.AI


Configuration endpoint (read-only)

Description
An endpoint such as /v1/config can help a developer inspect the current active configuration. This can be read-only in a first iteration, but might be extended to support runtime configuration changes in the future.
Motivation
Enhanced DX
Resources
No response
To-do List

[ ] Serve current application configuration via /v1/config

 Configuration endpoint (read-only)


Uploaded calculator_256.png to images/app folder

@sabeehmurtaza9 it would be preferable to have a website so that users could learn a bit more about your app before downloading. Pointing directly to a download with very little other information, I suspect, may seem suspicious to some, which in the end may make getting users for your app more difficult.
You can create a free website for you app using GitHub Pages which can still point to your downloads hosted elsewhere.
I'm going to close this for now, please re-open if you create a website, and I will revise the documentation on this 👍 Thanks!


typos

I found some other typos as well, but I can update with new PRs for those changes.

@ralphtheninja, Thx.

@ralphtheninja, It's better to use back tics, I think. I will update the docs later:)


Port Mopidy-PiDi to Python 3

Same as https://github.com/pimoroni/mopidy-raspberry-gpio/issues/1

All of the wisdom linked in the porting guide should be applied to the plugins, too, which can be found here: https://github.com/pimoroni/pidi-plugins


Extension not working correctly

If you are receiving only "Errors" when blocking on Chrome you will need to update to Chrome 67.

I checked and I'm using Chrome 67, I'm getting the impression that twitter is stopping me from going over 5K for some reason.

There are definitely API limits in place by twitter to prevent people from infinitely blocking. These rate limits are implemented by twitter and undocumented... some people have reported 8k, 10k, before the extension stops. there's not much we can do about it... a rate limit is a rate limit.


RuntimeError: Expected to mark a variable ready only once.

Please check that this issue hasn't been reported before.

[X] I searched previous Bug Reports didn't find any similar reports.

Expected Behavior
not sure if this a bug or a exception in my setup environment (docker and 1 gpu)
Current behaviour
this the error I got:
RuntimeError: Expected to mark a variable ready only once. This error is caused by one of the following reasons: 1) Use of a module parameter outside the forward function. Please make sure model parameters are not shared across multiple concurrent forward-backward passes. or try to use _set_static_graph() as a workaround if this module graph does not change during training loop.2) Reused parameters in multiple reentrant backward passes. For example, if you use multiple checkpoint functions to wrap the same part of your model, it would result in the same set of parameters been used by different reentrant backward passes multiple times, and hence marking a variable ready multiple times. DDP does not support such use cases in default. You can try to use _set_static_graph() as a workaround if your module graph does not change over iterations.
Parameter at index 447 has been marked as ready twice. This means that multiple autograd engine  hooks have fired for this particular parameter during this iteration. You can set the environment variable TORCH_DISTRIBUTED_DEBUG to either INFO or DETAIL to print parameter names for further debugging.
Steps to reproduce
accelerate launch -m axolotl.cli.train qlora.yml
Config yaml
No response
Possible solution
I did a search regarding the problem and figure the ddp_find_unused_parameters should be false. so I had to put ddp: true in my config file that makes the ddp_find_unused_parameters false.
see here:
https://github.com/OpenAccess-AI-Collective/axolotl/blob/06ae39200b60d3482af8bd8bfc1a823c1b24e1da/src/axolotl/core/trainer_builder.py#L644
it's working now.
Which Operating Systems are you using?

[X] Linux
[ ] macOS
[ ] Windows

Python Version
3.10
axolotl branch-commit
0.3.0
Acknowledgements

[X] My issue title is concise, descriptive, and in title casing.
[X] I have searched the existing issues to make sure this bug has not been reported yet.
[X] I am using the latest version of axolotl.
[X] I have provided enough information for the maintainers to reproduce and diagnose the issue.

Could you please provide your config to reproduce?
Iirc, setting that parameter would slow multigpu training a bit.

Will close due to inactivity. Please leave a comment if the issue reoccurs.


Question about CP decomposition in "OpSplittingNonConst" function in the code "PlotHelmholtzUnknown.m"

Dear authors,
I have a question regarding the implementation of your paper's algorithm for solving the Helmholtz equation using tensor decomposition. In section 5.2.4, Remarks, it is mentioned that for n1=n2=n3=30, the algorithm does not solve CP decomposition for (31x3) x (31x3) x (31x3), but instead approximates the operators ∆ and κ(x, y, z)I (which have size of 31x31x31) separately. However, I could not find this in the code "PlotHelmholtzUnknown.m". Can you please confirm whether the code implements the approximation mentioned in the paper?
Thank you for your time and assistance.

My apologies, I figured out what I need to do. Thank you


Locking Tokens when using Ctrl + Click

Core Foundry still has functionality to Ctrl+Click (which draws path) then issuing command "Move along with Ruler" e.g. I have it mapped to ENTER key makes the token to follow pre-drawn path to target.
After I installed Elevation Ruler module, pressing the Enter, triggers Error from Foundry "Token is Locked" and then only GM can move that token.

Ok, I managed to fix it in code. ClientKeybindings didn't properly unlock the token.

To be closed.


Use cross validation to for choosing threshold value

Our current approach uses the predicted probability of matching to estimate a good threshold. If our predictions are not good then everything suffers.
We should use a cross-validation approach, using the training data. We should block and cluster all the data and then use the labels from the training pair (and only those labels) to evaluate a threshold.
If we depend on training data, then we can't use the threshold method for static classes. This actually makes sense since given trained model, the output of the threshold method should be deterministic.

One challenge here is that the training data does not have unique ids.

/sub


SQL Editor Longitudinal Copy Support

SQL Editor Longitudinal Copy Support

Alt + MouseLeft to select text.

Alt + MouseLeft to select text.

It turns out this function already exists (If I understand you correctly). Make triple click on a line in SQL Editor to highlight it.
https://user-images.githubusercontent.com/51405061/106032298-5979e180-60e1-11eb-9617-0385e0cf9177.mov

It turns out this function already exists (If I understand you correctly). Make triple click on a line in SQL Editor to highlight it.
https://user-images.githubusercontent.com/51405061/106032298-5979e180-60e1-11eb-9617-0385e0cf9177.mov

There is no update on the issue for a long time.
If the issue is still actual for you - feel free to reopen it.


Failed to submit a form with file attachment

I've tried to build a contact form with file attachment, after selecting a file and submitting the form, I get this error: "in_array() expects parameter 2 to be array, integer given". Here is the stackstrace
Whoops\Exception\ErrorException thrown with message "in_array() expects parameter 2 to be array, integer given"

Stacktrace:
#20 Whoops\Exception\ErrorException in system/src/Grav/Common/Data/Validation.php:42
#19 in_array in system/src/Grav/Common/Data/Validation.php:42
#18 Grav\Common\Data\Validation:validate in system/src/Grav/Common/Data/BlueprintSchema.php:77
#17 Grav\Common\Data\BlueprintSchema:validateArray in system/src/Grav/Common/Data/BlueprintSchema.php:38
#16 Grav\Common\Data\BlueprintSchema:validate in system/src/Grav/Common/Data/Blueprint.php:97
#15 Grav\Common\Data\Blueprint:validate in system/src/Grav/Common/Data/Data.php:191
#14 Grav\Common\Data\Data:validate in user/plugins/form/classes/form.php:273
#13 Grav\Plugin\Form:post in user/plugins/form/form.php:78
#12 Grav\Plugin\FormPlugin:onPageInitialized in vendor/symfony/event-dispatcher/EventDispatcher.php:184
#11 call_user_func in vendor/symfony/event-dispatcher/EventDispatcher.php:184
#10 Symfony\Component\EventDispatcher\EventDispatcher:doDispatch in vendor/symfony/event-dispatcher/EventDispatcher.php:46
#9 Symfony\Component\EventDispatcher\EventDispatcher:dispatch in vendor/rockettheme/toolbox/Event/src/EventDispatcher.php:23
#8 RocketTheme\Toolbox\Event\EventDispatcher:dispatch in system/src/Grav/Common/Grav.php:297
#7 Grav\Common\Grav:fireEvent in system/src/Grav/Common/Processors/PagesProcessor.php:19
#6 Grav\Common\Processors\PagesProcessor:process in system/src/Grav/Common/Grav.php:121
#5 Grav\Common\Grav:Grav\Common\{closure} in system/src/Grav/Common/Grav.php:383
#4 Grav\Common\Grav:Grav\Common\{closure} in system/src/Grav/Common/Grav.php:359
#3 call_user_func_array in system/src/Grav/Common/Grav.php:359
#2 Grav\Common\Grav:__call in system/src/Grav/Common/Grav.php:122
#1 Grav\Common\Grav:measureTime in system/src/Grav/Common/Grav.php:122
#0 Grav\Common\Grav:process in index.php:45

This is where the error occurs:
        // Special case for files, value is never empty and errors with code 4 instead.
        if (empty($validate['required']) && $field['type'] == 'file' && isset($value['error'])
                && ($value['error'] == UPLOAD_ERR_NO_FILE || in_array(UPLOAD_ERR_NO_FILE, $value['error']))) {
            return $messages;
        }

Here is the content of $value:
Array
(
    [name] => lorem
    [type] => application/octet-stream
    [tmp_name] => /tmp/phprxnZgg
    [error] => 0
    [size] => 6043
)

As the error message already explains, the expected value of $value['error'] is an array, but in my case it is an integer which is equal to UPLOAD_ERR_OK.
I have tried to change the line 42 and 106
from
&& ($value['error'] == UPLOAD_ERR_NO_FILE || in_array(UPLOAD_ERR_NO_FILE, $value['error']))) {

to
&& ($value['error'] == UPLOAD_ERR_NO_FILE
|| (is_array($value['error']) && in_array(UPLOAD_ERR_NO_FILE, $value['error'])))) {

The I've got new error "Illegal string offset 'file'" for line 71 of user/plugins/email/email.php
Here is what from line 67 to 74:
                    $filesToAttach = (array)$params['attachments'];
                    if ($filesToAttach) foreach ($filesToAttach as $fileToAttach) {
                        $filesValues = $form->value($fileToAttach);
                        if ($filesValues) foreach($filesValues as $fileValues) {
                            $filename = $fileValues['file'];
                            $message->attach(\Swift_Attachment::fromPath($filename));
                        }
                    }

It seems the plugin expects $filesValues to be an array of files but in my case it is a single file?
Array
(
    [name] => lorem
    [type] => application/octet-stream
    [tmp_name] => /tmp/phpq3Unf7
    [error] => 0
    [size] => 6043
)

Just in case you need, here is my file upload field:
        -
            name: attachment
            label: 'Attachment'
            type: file
            multiple: false
            destination: 'user/data/attachments'
            accept:
                - application/pdf
                - application/x-pdf
                - application/zip

And here is the "process" section of the form:
        -
            email:
                from: '{{ config.plugins.email.from }}'
                to: ['{{ config.plugins.email.from }}']
                subject: '[Contact] {{ form.value.name|e }}'
                body: '{% include ''forms/data.html.twig'' %}'
                attachments:
                  - 'attachment'

Was there anyone having this problem before? Is this really a bug or something wrong from my side? Thanks!

Can you paste the whole form definition, for quicker testing?

Here is the MD file:
---
title: Contact
form:
    action: /contact
    name: contact-form
    fields:
        -
            name: name
            label: Name
            placeholder: 'Name *'
            type: text
            classes: pure-u-1-1
            validate:
                required: true
        -
            name: email
            label: Email
            placeholder: 'Email *'
            type: email
            classes: pure-u-1-1
            validate:
                required: true
        -
            name: phone
            label: Phone
            placeholder: 'Phone *'
            type: text
            classes: pure-u-1-1
            validate:
                required: true
        -
            name: message
            label: Message
            placeholder: 'Message *'
            type: textarea
            classes: pure-u-1-1
            validate:
                required: true
        -
            name: attachment
            label: 'Attachment'
            type: file
            multiple: false
            destination: 'user/data/attachments'
            accept:
                - application/pdf
                - application/x-pdf
                - application/zip
    buttons:
        -
            type: submit
            value: Send
            classes: 'pure-button pure-button-primary button-large '
    process:
        -
            email:
                from: '{{ config.plugins.email.from }}'
                to: ['{{ config.plugins.email.from }}']
                subject: '[Contact] {{ form.value.name|e }}'
                body: '{% include ''forms/data.html.twig'' %}'
                attachments:
                  - 'attachment'
        -
            save:
                fileprefix: contact-
                dateformat: Ymd-His-u
                extension: txt
                body: '{% include ''forms/data.txt.twig'' %}'
        -
            display: thank-you
---

I tested it, I have no issues.
Make sure you have the latest Grav 1.1 and Form plugin installed, is this the case?

Yes, everything is in the latest version. Let me try again or try on a different server/host. If the problem still exists maybe I will upload the site or user folder to Github to make it easier to test. Thanks!

Sure, or PM it to me on https://gitter.im/flaviocopes

Thank you for your time helping me! I figured out the problem, the form twig file of my theme didn't have the line
{% set scope = scope ?: 'data.' %}

So the fields were not in "data" array and the attachment couldn't be found by Swift.

@tranduyhung Could you please clarify where did you put that "set scope"?
I'm experience the same problem - attachments don't get saved or sent, nothing happens.

@zanami That line is in the default form.html.twig of Form plugin, I overrided the file to build my own form and accidentally deleted that line.

Thanks @tuphamnguyen
I don't have form.html.twig so I guess the default is used
Form version: 2.5.0 Still doesn't work.
I guess it's irrelevant to your issue

@zanami just fixed and issued a new release. Was a problem introduced by 2.5.0


rename websocket commands

api/menu
api/reopenScreen
dyn:relaunch
dyn:reloadCss
dyn:previewScreen

dyn:previewScreen is optional for reactUI.
It opens a screen if it's not already open. If it's not available, the menu will be reloaded and tried to open the screen afterwards.


Readme contains broken link to Spotter App

The Readme file contains a link to the Spotter App. Clicking the link brings a 404 for me - the Spotter App is probably not public (yet)?. Also, the link is missing a ) in order to fix the Markdown syntax.

The spotter_app is public now. Thanks!


Smartbar addition: keep moving cursor when input arrows are held pressed down

I really like this ROM. I would appreciate if you could merge this new commit from DU.
https://github.com/DirtyUnicorns/android_packages_apps_DUI/commit/c196b8cc14852ccc1629b7f9b88730947204135c

Thank you. I really appreciate it.


GO:0016104	 triterpenoid biosynthetic process

InterPro ID / label

GO:0016104 | triterpenoid biosynthetic process | IEA with IPR018333

Example sequences with problematic annotation (ID + gene/protein name):

Description of issue

is a  triterpenoid derivative, rather than a triterpenoid.
you could  probably use  sterol biosynthetic process ?

This highlights a potential ontology issue. The " isoprenoid family" classification is orthogonal to "end product" and it might complicate the curation of pathways.
For instance, in this case, lanosterol is_a triterpenoid, but is also past the commitment point of ergosterol biosynthesis (which isn't classed as isoprenoid biosynthesis because ergosterol is an isoprenoid-derivative rather than an iso[prenoid)
Flagging as an example.

Hi Val,
Thank you very much. I have replaced GO:0016104 with GO:0006694, which will be available in the next InterPro release (102.).
Best wishes
Sara

Hi Sara,
see follow up comments, I changed my mind, I think this is an ontology issue, but I haven't figured out a fix yet.

Ok, thanks, I will revert this then.


Customizable timestamps

Closes https://github.com/knurling-rs/defmt/issues/106
Based on https://github.com/knurling-rs/defmt/pull/342, since that was needed to fix the test in here.
This changes the defmt::timestamp proc macro to work just like the logging macros, but in item scope. This allows full customization of the timestamp, as well as "real" omission if no timestamp is used (currently we just always have a timestamp of 0).
Summary of changes

Add a dummy section to the linker script, whose purpose is just to hold symbols we want to retain but don't otherwise use. This is used to make the linker retain the timestamp format string, since we wouldn't otherwise use its address.
#[defmt::timestamp] is now a regular proc macro instead of an attribute.
A new timestamp tag was added to identify the format string used in defmt::timestamp!.
The default timestamp is not present instead of being 0. elf2table just won't find a format string with timestamp tag and omit it accordingly.
A new µs display hint was added to retain the previous time formatting. It interprets an integer as a time in microseconds and formats it like X.XXXXXX. We could potentially be more fancy here and allow arbitrary scaling of fixed-point decimal and binary values, but that seemed fairly complex, so I did this instead.

Limitations

If you want to use the old timestamp format, you'd use {=u64:µs}. However that is no longer LEB-encoded, so it takes up more space if it's a small number (eg. a monotonic counter). We could fix this by encoding integers with LEB128, or by adding a new leb128 format type.

I probably still need to update the book and other docs. Done!

regarding the book update, whst's left to do there? would it make sense to open an issue about that so it's transparent and we don't forget?

regarding the book update, whst's left to do there? would it make sense to open an issue about that so it's transparent and we don't forget?

I think that's all done now

I think that's all done now

bors r=japaric

bors r=japaric


[Guide] [HOWTO] [Windows] start Restic-browser with selected repository

Create batch file (*.bat)
Change path to yours restic-browser.exe
@echo off
set RESTIC_REPOSITORY=rest:http://example.repository/repo
set RESTIC_PASSWORD=PasswordToRespoitory
start "" d:\Restic-Browser.exe (Path to Restic-Browser.exe)

This batch file will start Restic Browser and connect to selected restic repository.

Thanks for that hint.
Would it be also possible to set the path to restic.exe, because on my system it is not part of $PATH.

You could either change/update the PATH in the bat file, so Restic Browser can find it or use the new app argument:
start "" PATH_TO\Restic-Browser.exe --restic PATH_TO\restic.exe

Thanks for the tip. For Backblaze:
@echo off set RESTIC_REPOSITORY=b2:<YourRepository> set B2_ACCOUNT_ID=<YourAccountID> set B2_ACCOUNT_KEY=<YourAccountKey> set RESTIC_PASSWORD=<YourPassword> start "" .\Restic-Browser.exe
No quotes on the account ID, account key, or password (don't know if that's a problem if there's a space in your password). And obviously change the start path if your .bat isn't in the same folder as the exe.


Revise the recovery mechanism

Hi,
Thanks for the great work!
I've found that the recovery mechanism was to call gameplay() recursively when a player is dead.
But this approach will consume a lot of stack memory when a player keeps playing the game.
So I've modified some codes to restart a game more robustly.

Hi
Thanks a lot for your interest in this project. Apparently, I am no longer maintaining this code, hence won't be able to accept any new features or pull requests.
You are, however, free to fork this project and maintain it as your own.


Add help text to explain what dump does

The built in Nunjucks filter dump is used in a couple of places…

Object filters
Array filters

As someone new to filters I didn't realise what dump did.
Perhaps some help text would be good to explain what it's for (i.e. debugging) and that it's safe to remove if you don't need it.

@benjystanton Ah, that’s a useful insight. Have updated the docs where dump is used, lemme know if this works.

Yeah that's great. Thanks!


Documentation: Element's dependencies order

See original issue on GitLab
In GitLab by [Gitlab user @esodan] on Apr 13, 2018, 16:47
There is no documentation on how dependencies should be ordered.
If the element depends on a set of elements each depending each other, documentation should state if the order of each dependency declaration matters or not.
From this comment: https://gitlab.gnome.org/GNOME/gnome-build-meta/merge_requests/22#note_96808
When it explain, put "Element1" before "Element2", can mean put Element1 up of Element2, so Element1 is build first then Element2.

In GitLab by [Gitlab user @jjardon] on Apr 13, 2018, 18:55
mentioned in merge request !390

In GitLab by [Gitlab user @jjardon] on Apr 13, 2018, 18:56
Thanks for your report [Gitlab user @esodan] ; !390 opened to address this

In GitLab by [Gitlab user @tristanvb] on Apr 16, 2018, 07:03
mentioned in commit fda04f4254e71a94aa8c062511a8289f55176130

In GitLab by [Gitlab user @tristanvb] on Apr 16, 2018, 07:04
mentioned in merge request !398

In GitLab by [Gitlab user @tristanvb] on Apr 16, 2018, 07:15
mentioned in commit 938ec1f438c3613c7f06fa6df956efdc655df880

In GitLab by [Gitlab user @tristanvb] on Apr 16, 2018, 07:34
closed via merge request !398

In GitLab by [Gitlab user @jjardon] on Apr 24, 2018, 02:44
mentioned in commit 67cdf3dce26861047c2e8dedf276b300224820e8

In GitLab by [Gitlab user @jjardon] on Apr 24, 2018, 02:44
mentioned in commit dabb2bb464f3bde7afcb66e396e1ff0b877af00b

In GitLab by [Gitlab user @jjardon] on Apr 24, 2018, 02:45
mentioned in commit 4be5f1f69165b0da9f7df6222ec95b6dd536ece5

In GitLab by [Gitlab user @jjardon] on Apr 24, 2018, 02:45
mentioned in commit 07ac1bfa580ee08e529d1ab7b3e529caf13c1f9c

In GitLab by [Gitlab user @bochecha] on Sep 17, 2018, 11:15
mentioned in commit c0a2a70fbecfa066bb1d74294e282ad7737fab6b


Convert most AMQP types to read-only; Significant API surface cleanup.

Significant cleanup of AMQP internals.
Fixes #4504 #4505.
Pull Request Checklist
Please leverage this checklist as a reminder to address commonly occurring feedback when submitting a pull request to make sure your PR can be reviewed quickly:
See the detailed list in the contributing guide.

[ ] C++ Guidelines
[ ] Doxygen docs
[ ] Unit tests
[ ] No unwanted commits/changes
[ ] Descriptive title/description

[ ] PR is single purpose
[ ] Related issue listed

[ ] Comments in source
[ ] No typos
[ ] Update changelog
[ ] Not work-in-progress
[ ] External references or docs updated
[ ] Self review of PR done
[ ] Any breaking changes?

Ping :).

Ping :).

Ping 2 :)

@LarryOsterman, BTW, if samples do show the usage for the Azure SDK developers, then they can use Core::_internal::Environment, the same way they also do use Amqp::_internal, i.e. once you are user of _internal, the entire _internal surface is available to you. Other thing is if getenv() in samples plays purely a helper role and not the intended use case that we would like to emphasize. Then both are fine.

@LarryOsterman, BTW, if samples do show the usage for the Azure SDK developers, then they can use Core::_internal::Environment, the same way they also do use Amqp::_internal, i.e. once you are user of _internal, the entire _internal surface is available to you. Other thing is if getenv() in samples plays purely a helper role and not the intended use case that we would like to emphasize. Then both are fine.

getenv() in these samples plays the exact same role as it does in all the other samples: It allows external configuration of the samples to work with the service.


Tailwind requires CSS rebuild after every HTML modification. (bin/dev, hotreload gems, etc., not effective)

Hi all,
This one has stumped both myself and (more importantly) those with far more experience and knowledge of the Rails framework and CSS engines alike. I've attached a brief 2.5 minute video below demonstrating the issue. In effect, I can change each class on an element one time, but if I try to change two or more times, I must manually rebuild the CSS.
For example, I can change a class to text-4xl from its current state of text-2xl, and it will work. But if I go to then change it to text-sm or anything other than text-2xl or text-4xl, it will effectively remove the entire class and will default to black. Same with mt classes. A perfect example is changing the default mt-28 property in application.html.erb. If I set it to, say, mt-10, that and mt-28 are the only two classes that will apply. If I tried to set any other mt level, it would fail and effectively be mt-0.
I'm currently running WSL2 Ubuntu 20.04, Rails v7.0.3, Ruby v3.1.2 -- all stable releases. I use importmaps, so no package.json to speak of. I'm now a novice+ programmer, but know my way around a troubleshoot as I was tuned-in about a decade ago. I'm open to trying any and everything in order to solve this and document the solution -- it's the first thing I've seen stump everyone in the GoRails community's CSS channel.
Thanks in advance, everyone!
RM
Link: https://streamable.com/kx0zae

Your server might be serving your manually built css instead of the rebuilt bundle created by tailwindcss cli.
Try running rake assets:clobber and see if that solves the issue.

@robertemaxwell There's a PR submitted to make the "watch" mechanism support WSL.
https://github.com/rails/tailwindcss-rails/pull/168
Does that sound like it might address this issue?

See #199 for a PR which I think will resolve your issue.


Privacy - Avatar and shown name - no options

Hi, as it seems like there is only support for one user picture (which can everybody see) - I thought to ask if that is gonna change at some point.
A user might want to use a personal picture that personal friends/family can only see, while public/all others see another - without face e.g..
Is there any support in Matrix for this or if not, is there any way this privacy option may get added? This is currently a public leaking bit of information.

Matrix supports it, Riot does not.
I recommend opening an issue on Riot-Web as well, as that leads the design and the features. The mobile clients follow behind.

I dont want to change room picture to be my picture. than when I open chat all I see is myself avatar, and not the one of the chat partner I do chat with (in 1:1 direct chats)

I don't think you understand. Have a look at this: https://vault.webdevguru.co.uk/Matrix/per-room-nickname/

No, this is about picture not realname. Too the main expectation is not room based, but global.
Real example from myself:
Have a matrix account with public known username, and association with my realname.
Now I want to chat with my father and others from family over Matrix. Non-Techies - this means they expect to see just my name and a real face icon.
For this father has a public unknown username, nothing that would identify a relation with myself, or give details about him.
In any situation its the goal that I can see his face picture (where no avatar at all should be public) and his real displayname. The same goes the other way round, and with other family members.
This is currently not possible, and thats why I reported this. What I call Privacy in most important level - to use Matrix without leaving public identifiable information.
Other protocols/messengers usually have an Option to set visibility for:

Nobody/ Everybody
Specific persons (users)
Groups (Rooms)

And thats what I expect from Matrix protocol and clients to support too. Maybe not all cases, but the example with father has the main goals. This is nothing I see as optional feature, but a basic need.
I don't know what is currently possible or planned but I do hope, Matrix as speaking for open, decentralization and privacy does get an solution for this.

The per-room avatar and nickname are definitely critical features. Adding an option to edit your nickname per-room without the devtools would be an excellent and relatively simple start for this, then get started on the avatar options.


bsd: plug uv_fs_event_start() error path fd leak

Close the watched file's file descriptor when initialization fails due
to out-of-memory conditions.
CI: https://ci.nodejs.org/job/libuv-test-commit/1239/

Landed in 09ba4778, thanks for the reviews!


[Possibly a tiny bug or a minor issue] Dusk has two staffs/spears?

I am doing a new test-run with the newly released mod here. I'll give more feedback lateron, in particular about Fall. But I ran into a strange thing with Dusk just now.
So Dusk now has ... two staffs. First, the melee-centric close variant which is also known from the older mod. This is the staff that can lateron dispel. A two-handed weapon.
But we also now seem to get a new spear-staff? One that can be wielded in one hand, and also be thrown. So this adds versatility.
However had, both have the word "shilmastan" or whatever it is. Which makes me wonder ... they are two objects. But aren't they the same staff? Or does Dusk now have two staffs?
It's a bit confusing to me; from the description it almost seems as if it should be a single item and we can decide which variant to use. But clearly Dusk has two items in inventory, which is a bit weird to me.
Actually I prefer Dusk wielding two bastard swords, so I almost never use the staff unless I need its dispel ability. But now I see to have two different staffs, which confuses me. If it is a bug or a minor issue then it could perhaps be addressed, but IF it is not a bug then perhaps the dialogue options can be made more clear about it, in particular since I don't need it (but it is nice that it can be thrown, that adds option for Dusk e. g. if he is too injured he can retreat to the backrow and still throw the staff to do damage).
Or perhaps it is another weapon - perhaps the name could then be changed a bit, so as to not confuse it with the staff. (In older gameplays I also managed to clone one of the two bastardswords; I haven't managed to do so again, but if I run into it I'll make sure to report it again.)

It's a bug. You should only have 1 of the 2 staves. Something must have happened with the inventory during the conversation. The thing is, the old staff should have been removed and the new spear-staff left. I've already put out a patch to prevent that from happening. Thanks for reporting it.

Fix loaded for the next release.


Strip merged nodes from AST in compiler

This patch removes AST nodes that are merged in compiler from the tree. Currently it is not the case, and so AST can be messed up during the process.
To illustrate the problem (see also https://github.com/wooorm/remark/pull/162#discussion_r52991739):
ast:
{
  "type": "paragraph",
  "children": [
    {
      "type": "text",
      "value": "foo"
    },
    {
      "type": "text",
      "value": "bar"
    },
    {
      "type": "text",
      "value": "baz"
    }
  ]
}

remark.stringify(ast);

{
  "type": "paragraph",
  "children": [
    {
      "type": "text",
      "value": "foobarbaz"
    },
    {
      "type": "text",
      "value": "bar"
    },
    {
      "type": "text",
      "value": "baz"
    }
  ]
}

Wait, is foobarbaz, bar, baz the real output?
Great otherwise :)

Are you saying that happens now? Hmm, darn.

No, the output is fine (just foobarbaz), but ast contains these extra nodes after remark.stringify call.
Succeeding remark.stringify calls would return garbage (foobarbazbarbaz, then foobarbazbarbazbarbaz &c) :smile:

Oh right, I see, awesome!

Thanks a lot, @eush77, I won’t release it yet though, maybe there are other things coming up this week too!


Cache panel content (This makes edit mode feel reeeeeelly fast)

This adds two options to the panel constructor:

options.cache: Should the HTML be cached on first open
options.primeCache: Should we try to prime the cache on load

Pull this down and build the JS. This really makes edit mode feel much better.

Edit mode is pretty fast now, try not to get motion sickness...

So right now this breaks copying a block to clipboard, thoughts on the best fix for that?

Looks promising. Let's reopen and worry about this on the release/8.0.0 branch.

Did we still want this?

I liked it way back and like it now. Can you get round the clipboard update by caching the panel in parts rather than the whole thing? For example, cache blocks and load clipboard as needed. Or cache both, but invalidate & reload only the clipboard part if there has been a copy.

Did we still want this?

Yes!

BTW, caching all panels is really dangerous, so I'd cache only the blocks and stacks list (so excluding the clipboard) because:

it's the only panel that usually gets opened/closed multiple times (I know that we can keep it open, but that happens anyway)
its content isn't sensitive to its previous state
changes to the blocks list are really rare during a page editing session (for instance: in one window we're editing a page and in another window we install a new package or add a new stack). In those sporadic cases it's enough to refresh the page.

What about adding a whitelist of panel URLs that can be cached? For instance (in form of regular expression), this could be /\/ccm\/system\/panels\/add\?cID=\d+(&tab=blocks)?$/


Documentation anime.running count sometimes wrong

If you open this example: https://animejs.com/documentation/#running, it always says there are no instances running.
LE: It seems to happen if you switch to a new tab and come back, can't reproduce it exactly for now, it also sometimes shows 4 or 5 and I don't understand why, it seems broken.

Fixed in v3.1.0!


Is EventBus actually used?

I received a note from Google Play on my latest app release as follows:

I traced it to here:
https://github.com/birkir/react-native-carplay/blob/109c0a6257854016d51ab16a9dad6110340c29a3/packages/react-native-carplay/android/build.gradle#L107
Is EventBus actually used in the Android Auto implementation? I couldn't find it anywhere.

You are right, it is being used anymore.


Tracking: Stabilize BACKGROUND_DDL

Currently background ddl has been a beta feature for a long time. It's time to make its implementation more polished. This can also simplify the code in the meta backend, so we don't have to maintain compatibility between in-memory and persisted jobs.

[ ] https://github.com/risingwavelabs/risingwave/issues/15603
[ ] Refactor sink: Instead of marking it as "finished", it should also follow the same background creation path as MV.
[ ] https://github.com/risingwavelabs/risingwave/issues/16532

Not urgent, wait for meta sql backend stabilized.

Can be worked on now. Since meta sql backend has been stabilized.


[Question] Split => correct => re-join

Hi there,
I was wondering if it is feasible to split the long read file into smaller chunks. Then in parallel load each chunk with the index. Finally merge them back later to speed up flmrc2 ?
Many thanks,
Tuan

Thanks ! It's good to have some confirmation that this would work in real scenario.


Fileformat not show up

[x] I'm using the latest lualine.
[ ] I didn't find the issue in exsisting issues or prs.
everything is OK but no fileformat

How to reproduce the problem
use packer install lualine
Expected behaviour
should be a fileformat icon?
Actual behaviour

Minimal config to reproduce the issue
require('lualine').setup {
  options = {
    icons_enabled = true,
    theme = 'onelight',
    component_separators = { left = '', right = ''},
    section_separators = { left = '', right = ''},
    disabled_filetypes = {},
    always_divide_middle = true,
  },
  sections = {
    lualine_a = {'mode'},
    lualine_b = {'branch', 'diff', 'diagnostics'},
    lualine_c = {'filename', 'lsp_progress'},
    lualine_x = {'encoding', 'fileformat', 'filetype'},
    lualine_y = {'progress'},
    lualine_z = {'location'}
  },
  inactive_sections = {
    lualine_a = {},
    lualine_b = {},
    lualine_c = {'filename'},
    lualine_x = {'location'},
    lualine_y = {},
    lualine_z = {}
  },
  tabline = {},
  extensions = {}
}

I think the font you're using doesn't have those icons . Can you see if icons here https://github.com/nvim-lualine/lualine.nvim/blob/70691ae350fdbe1f15758e3b8e2973742a7967a9/lua/lualine/components/fileformat.lua#L7-L9 get rendered ?
You should probably try with a different font + Also see if the component works with icons_enabled = false
lualine_x = {'encoding', {'fileformat', icons_enabled = false}, 'filetype'},

Yes, it's because of font, config icons_enabled = false then shows fileformat


django-stubs: Fix HttpResponse missing attribute error.

This is a part of #18777.
As Django monkey patch attributes on the HttpResponse the test client returns. This does not really make the patched attributes available on the response object of the test client, but instead avoids accessing these attributes by taking a detour.
It refactors away response.json() with response.assert_json_success and a new helper response.assert_json_error_data manually and with regexes.
A flag use_zulip_api_format is added for response.assert_json_error_data to properly test test_scim.py.

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

The use_zulip_error_format flag is not worth its complexity. The five tests that set it to False would be simpler and more readable without it.
-        response_dict = self.assert_json_error_data(result, use_zulip_error_format=False)
+        self.assertEqual(result.status_code, 400)
         self.assertEqual(
-            response_dict,
+            orjson.loads(result.content),
             {
                 "schemas": ["urn:ietf:params:scim:api:messages:2.0:Error"],
                 "detail": "Must specify name.formatted, name.givenName or name.familyName when creating a new user",
                 "status": 400,
             },
         )

I noticed that django-stubs actually knows about response.json(). We’re just not declaring our ZulipTestCase.client_get et al. wrappers with the type that knows about them, _MonkeyPatchedWSGIResponse. (It’s not present on all HttpResponse objects, only the ones returned by the test client.)

I noticed that django-stubs actually knows about response.json(). We’re just not declaring our ZulipTestCase.client_get et al. wrappers with the type that knows about it, _MonkeyPatchedWSGIResponse. (It’s not present on all HttpResponse objects, only the ones returned by the test client.)

Yes. We might need to change the return type of the wrapped test client to _MonkeyPatchedWSGIResponse so that it's properly typed. This requires us to import it with django-stubs installed. This causes type errors wherever we have HttpResponse in our test code, and it will be a larger migration than this one.

You can actually write this today, without django-stubs installed (in which case mypy will treat it as Any, like it treatss HttpResponse now).
from typing import TYPE_CHECKING

if TYPE_CHECKING:
    from django.test.client import _MonkeyPatchedWSGIResponse

def client_get(…) → "_MonkeyPatchedWSGIResponse"
    …

Even after we do install django-stubs, we’ll still need that TYPE_CHECKING conditional, since _MonkeyPatchedWSGIResponse doesn’t exist in the real django at runtime.

Makes sense. Though I think it's probably a cleanup to the tests to not have tons of them doing the .json() access directly, so we probably still want to do a version of this PR? Or are you proposing we don't want to do that?

This is great, merged, thanks @PIG208!


chore: Fix typos in "argocd" mentions in usage guide

What type of PR is this?

Uncomment only one  /kind line, and delete the rest.
For example, > /kind bug would simply become: /kind bug

/kind bug
/kind cleanup
/kind failing-test
/kind enhancement
/kind documentation
/kind code-refactoring

What does this PR do / why we need it:
fixes typos in technical text containing the term argocd
Have you updated the necessary documentation?

[x] Documentation update is required by this PR.
[x] Documentation has been updated.

Which issue(s) this PR fixes:
Fixes #?
Test acceptance criteria:

[ ] Unit Test
[ ] E2E Test

How to test changes / Special notes to the reviewer:

/lgtm

@jannfis could you approve this? Thanks

/approve


⚠ Jenkins Plugins Artifactory has degraded performance

In 488744b, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1837 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 785067d after 12 minutes.