Use with base jekyll?

What do I need to do to use the docs.css as the "one css to rule them all" in a jekyll site?
is there a way to take a sass file and make it part of the jekyll asset generation process?
(I actually expect this to be something like, uh duh, copy foo.sass to _sass and that's-that)

Jekyll includes a Sass converter so there's nothing special in compiling it if you've got Jekyll already up and running for yourself. See http://markdotto.com/2014/09/25/sass-and-jekyll/ for more info.

Just want to mention this here again: only using the built-in Sass compiler won't run Autoprefixer which results in situations like #23.


txnbuid: allow for custom sequence numbers

The Transaction.Build() function uses and consumes the current sequence number of supplied Account. This makes it tricky to build multiple transactions with the same seq number, or future seq numbers. These are crucial for writing smart contract functionality.
In addition, this build step has a side effect of mutating the seq number of the given account which is not obvious and conflates account state with the construction of a transaction.

One possible solution: replace Account in the Transaction with a simple account strkey and seqnum. Transaction.Build() can use these without mutating any state.
In addition, there can be a BuildTx() function on Account which takes in a transaction and builds with the account strkey, seqnum, and consumes that.

Also, I know that current functionality mimics that of other sdks (js, for example) but I don't think we should allow that baggage set the tone.

In the new Go SDK, I implemented it this way because

I assume most users are following a workflow where they need to submit a transaction now, not writing smart contracts for the future. Auto-incrementing + passing an Account provides the simplest API for this main use case.
This is the behaviour of the JS SDK build().
This is the behaviour of the Java SDK build().

This is also the same behaviour as the Python SDK, the .NET SDK, the Scala SDK, and so on.
Universally the approach taken by all the SDKs I am aware of has been to allow power users to modify the sequence number manually, while defaulting to incrementing. This approach is summed up well in the Scala docs:

The sequence number for a new account is the id of the ledger in which is was created. The id increments by one with every submitted transaction. In simple architectures, it is possible to keep track of the next sequence number without querying the network. However, if this is not possible, or the number is unknown, you can directly substitute the response from the @ref:account query into the Transaction constructor.

Changing the design now to something like this proposal has benefits:

An arguably more elegant design (no side-effects) that encourages explicitness.
Future/parallel transactions are simpler to write.
Adding a second build path allows the best of both worlds.

It also has downsides:

It breaks with the convention established by every other SDK in the ecosystem.
It is a major breaking change to this API. No existing code will function after the change.
It adds complexity to the majority use case, since users have to understand the second build path and recognise that it applies to them.
We would need to consider making the same change to the other official SDKs, since presumably the same logic applies. Similarly, we might want to agitate for the same change across community SDKs. This would be a big job.

Note that the current build method does not disallow these future transactions - it just makes them more fiddly to write. A caller simply sets a custom sequence number in their Account struct before calling build.
My suggestion: we modify Transaction.Build() to take an optional argument indicating the sequence number not be incremented. We expand the Account interface with a third method, GetSequenceNumber(), and modify the existing implementations in protocols/horizon and friendbot. This preserves the current default behaviour, while allowing more convenient usage for smart contracts.

@ire-and-curses thanks for that great breakdown of pros and cons. I think your suggestion does a good job in addressing the issue while keeping the ecosystem in sync.

This will bring a breaking change. To be implemented in v2.0

This is very urgent and required since ideally the flow should be like below
				tx := txnbuild.Transaction{ 					SourceAccount: &hAccount0, 					Operations:    []txnbuild.Operation{&op}, 					Timebounds:    timebounds, 					Network:       phrase,                                        Sequence: sequenceNo 				}
The current code for txnbuild creates tx with fixed sequence number for a new transaction (https://github.com/stellar/go/blob/8cd5bd9555680f008b9ce6f6e21715fe155c0c74/txnbuild/transaction.go#L92)
Currently are using the build method to supply the sequence number.

@aanupam23  SequenceNumber is a feature of an account, not the transaction so it is set on the SourceAccount. Typically when you get the details of an account from horizon, it already contains the sequence number so you do not need to specify this.
However, if you need to have more control over what sequence number is used, you can create a txnbuild.SimpleAccount and use that as the source account of the transaction. It has the following shape.
type SimpleAccount struct {
	AccountID string
	Sequence  int64
}

@poliha : Yes I knew about sequence number is feature of account. SimpleAccount works very well, yay :+1:
Here is the code to control sequence number if anyone needs this
`
accountRequest := horizonclient.AccountRequest{AccountID: Address}
Account, err := client.AccountDetail(accountRequest)
seqno, _ := Account.GetSequenceNumber()
// Do anything with sequence number

tx := txnbuild.Transaction{
	SourceAccount: &txnbuild.SimpleAccount{AccountID: Address, Sequence: int64(seqno)},
	Operations:    []txnbuild.Operation{&op},
	Timebounds:    txnbuild.NewInfiniteTimeout(),
	Network: phrase,
}

`
NOTE Whatever sequence number you will provide will get increment automatically so make sure to consider it in your calculations (Sequence number of transaction envelope should always be greater than account sequence number)
https://github.com/stellar/go/blob/d39a0aac6abc94f92cb50d3bc430c65c7be8dbf1/txnbuild/transaction.go#L94

💯 The auto-incrementing of sequence numbers in our SDKs is an unexpected side-effect which requires a developer to track in their head how the internals of the SDK works, can cause bugs, and can be a burden.
I think in principle our SDKs favor mutating objects which cause side-effects in a few ways, some which could be dangerous side-effects, others are inconvenient, and if we're thinking about a v2 of our SDKs we should consider how we can remove them. The cases I know about:

Calling Build increments sequence number.
You cannot call Hash without calling Build first.
If a transaction is changed after calling Build then call Hash, the hash returned is wrong. ❗
You cannot call Sign or Encode without calling Build first, but the API doesn't enforce it.
You cannot call Build again if you've called Sign.

Some ways we could change the SDK APIs to reduce unexpected side-effects and to strongly correlate cause and effect:

Encourage setting the sequence number manually. As a developer who used the Go SDK before I joined SDF, I don't think we get enough win from this auto-incrementing behavior.
Specifically in the Go SDK, remove the Hash, Sign, and Base64/encode operations from the builder class. Have Build return an immutable value that has the Sign and Encode functions on it (similar to the JS SDK).

Presumably fixed by https://github.com/stellar/go/pull/2495 (used fix tag with this issue ID). Can you confirm? If not, please reopen.


Passing props to the ReactFilestack element does not work as expected

We're using rmwc components with material UI. This doesn't work as expected because of your tagMap:
import { Button } from 'rmwc'
<ReactFilestack
  componentDisplayMode={{
    type: Button
  }}
/>

As the ReactFilestack component only renders the button, optimally I would expect the API to be something like this..
import { Button } from 'rmwc'
<ReactFilestack
  component={Button}
  label='Upload a file'
  className='myclass'
/>

So, in your constructor, something like this:
    const {
      apikey,
      clientOptions,
      actionOptions,
      action,
      componentDisplayMode,
      // All other named props that you need
      ...rest
    } = this.props;

    this.componentProps = {...rest};

And in your render(), something like..
  render () {
    const Component = this.componentProps.component;
    return <Component name="filestack" onClick={this.onClickPick} {...this.componentProps} />
  }

We don’t need add extra params, we can do that with customRender:
import "./App.css";

import ReactFilestack from "filestack-react";
import { Button } from "rmwc";
import<EMAIL_ADDRESS>
function App() {
  return (

      <ReactFilestack
        customRender={({ onPick }) => (
          <Button label="Open picker" uneleted onClick={onPick} />
        )}
        apikey="your_api_key"
        actionOptions={{
          maxFiles: 10,
          imageMax: [3600, 3600],
          accept: ["image/jpeg"],
        }}
        onSuccess={(res) => console.log(res)}
      />

  );
}

export default App;

Is it still happening in 4.0 version?


Run pre-commit against entire repo

Now that we have pre-commit, we should be able to cut down on noise generated by some of the github actions. This commit runs pre-commit against the entire repo to whip everything into shape.

@CannonLock I requested a review from you because I noticed that pre-commit modified a few svg's. in origin_ui/src/public. I'm guessing it was just dealing with some whitespace/new lines, but can you double check that it didn't do anything wonky?


Downloading the dataset

Hello there, I have installed the repository like 'pip install lsat'. Then I have tried running python PytorchDataset.py but I get the error ModuleNotFoundError: No module named 'lsat.typing'
Is there any instructions we can follow to download the dataset? Thank you very much~

Hola! ¿hay alguna posibilidad de descarga del conjunto de datos?
el mismo no esta disponible, error 502.

Sorry for the late reply, I've updated the dataset and its links but still haven't updated the python package. Meanwhile you can download it directly from the web page: https://midusi.github.io/LSA-T/
The links should be live now.


how to choose the right model when running on multiple-nodes.

There is every model in every node when running distributed programe.
How to choose the model after finishing tranning.Average model from every node?
And why the model is different in different node even in dist-sync mode?Why it need to generate its model in every node.Xgboost just only has its model in master node.
Could anyone help to explain it?

You can use any one. They shouldn't differ by a lot.


Feature Request: Ability to specify format options with "-o fstype"

docker volume create -d vmdk --name=test -o fstype=
Error response from daemon: create test: VolumeDriver.Create: Not found mkfs for
Supported filesystems found: cramfs, ext2, ext3, ext4, ext4dev, minix, bfs

It would be nice to have an advanced option to specify additional arguments like blocksize etc with "-o fstype" during volume create.
CC/ @cdickmann

One thing is, Get() on a volume can fail and volume meta-data may not be returned to the plugin. In which case we keep falling back to defaults and may not necessarily be able to use a volume if meta-data isn't available (like block size) for any reason.


Change or add -in-place so the output is formatted

Right now if I run jq -i '.some = "attribute"' some.json. The resulting file has no format. Personally I think it should format by default.

I agree with you. I implemented this in 645bd092470efbbd7d9cd7a31c375a44c70052fa. Thanks!


How can I make a "result image" from trained model ?

Hi, @IanTaehoonYoo
How can I make a "result image" from trained model ?
I can't found that code...
Thanks in advance.
Best,
@bemoregt.

Hi, @bemoregt
You can find code in the Trainer class, and the line is 160.
Also, the result images will be saved in the runs folder if you run this project.
If you have any questions, please let me know.
Thank you,
Best Regards,
@IanTaehoonYoo


Fix python maker for Vim: use \r instead of \n

When using Vim instead of Neovim, the python maker failed like this:

Neomake: python stdout:   File "", line 2
Neomake: python stdout:     from sys import argv, exit
Neomake: python stdout:        ^
Neomake: python stdout: SyntaxError: invalid syntax
Neomake: python stdout:
Neomake: python stdout done.
Neomake: python processing 5 lines of output
Neomake: File-level errors cleaned in buffer 1

It seems like we have to use "\r" for newlines, instead of "\n".

Huh, good catch. I thought this had been working, but I tested it out and you're totally right.


bug(input): readonly input trying to open the keyboard

Ionic version:  (check one with "x")
[x] 2.x
I'm submitting a ...  (check one with "x")
[x] bug report
Current behavior:
I have a screen which is meant to be read only. I use ion-inputs to display the data, but I have set the readonly attribute on them. When pressing the ion-input, the keyboard tries to open and the screen moves around.
Also not that having an ion-icon inside an ion-item with an ion-input, the icon click events do not fire.
Happens on both Android and iOS using the latest nightly.

Related code:
 <ion-item *ngIf='issue.Status === 2' class='service-provider'>
                        <ion-icon name='hammer' item-left></ion-icon>
                        <ion-label stacked>Assigned Service Provider</ion-label>
                        <ion-input class='first-input' type="text" readonly [value]='issue.ServiceProviderCompany'></ion-input>
                        <ion-input class='second-input' type="text" readonly [value]='issue.ServiceProviderContactPerson'></ion-input>
                        <ion-icon class="action-icon" name="mail" item-right (click)="sendMail(issue.ServiceProviderEmailAddress)"></ion-icon>
                        <ion-icon class="action-icon" name="call" item-right (click)="callNumber(issue.ServiceProviderContactNumber)"></ion-icon>
                    </ion-item>

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
Cordova CLI: 6.4.0
Ionic Framework Version: 2.0.0-rc.3-201612121605
Ionic CLI Version: 2.1.17
Ionic App Lib Version: 2.1.7
Ionic App Scripts Version: 0.0.47
ios-deploy version: 1.9.0
ios-sim version: 5.0.10
OS: macOS Sierra
Node Version: v6.9.1
Xcode version: Xcode 8.1 Build version 8B62

Hello, thanks for opening an issue with us! I am going to close this issue as a duplicate of https://github.com/driftyco/ionic/issues/9070.


wsl not umount the old root value in config file.

Windows Build Number
Microsoft Windows [Version 10.0.19043.1110]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
Linux version <IP_ADDRESS>-microsoft-standard-WSL2
Distro Version
Ubuntu 20.04
Other Software
No response
Repro Steps

I created wsl.conf file using sudo nano /etc/wsl.conf inside the ubuntu distro, with the following content:

[automount]
enabled=true
root=/mnt/

then shutdown it from windows wsl --shutdown Ubuntu then wait 8 seconds, and run it again
the /mnt/ directory contains the mounted directories and files as expected.
update the wsl.conf file again with the following content:

[automount]
enabled=true
root=/mnt2/

shutdown the distro as before and rerun it again.

Expected Behavior
it should mount the directories to /mnt2/ and umount the old directory which is /mnt/.
Actual Behavior
What happend instated is the two directories are mounted, /mnt/ and /mnt2/.
Diagnostic Logs
I'm not sure if that's right or not but for me it make sense to mount the new configured directory and unmount the old ones.

Are you looking at empty directories in /mnt?
https://github.com/microsoft/WSL/issues/3155#issuecomment-388119693

@0xbadfca11 okay I see, I just look into the folders and they were empty, sorry I didn't check if they contains files or not.


Some help for the beginner ?

Hello,
I wrote so few C++ in my life... I am a beginner here and I would like to get a "from scratch" compilation procedure for this project.
Someone has it ?
Here is my procedure (Linux Mint 19.3), but the compilation crashes :
sudo apt install gcc
# Mint 19.3 embeds gcc-7, which is fully C++ 17 compatible, in theory.
# tried the same with gcc-10, but still not working. I may miss some library.
mkdir web-server
cd web-server
git clone https://github.com/uNetworking/uWebSockets
git submodule init
git submodule update
make

This results in this log, full of errors :
make -C uSockets; \
for FILE in HelloWorld EchoServer BroadcastingEchoServer UpgradeSync UpgradeAsync; do g++ -flto -O3 -lpthread -Wconversion -std=c++17 -Isrc -IuSockets/src examples/$FILE.cpp -o $FILE uSockets/*.o -lz & done; \
for FILE in HelloWorldThreaded EchoServerThreaded; do g++ -pthread -flto -O3 -lpthread -Wconversion -std=c++17 -Isrc -IuSockets/src examples/$FILE.cpp -o $FILE uSockets/*.o -lz & done; \
wait
make[1] : on entre dans le répertoire « /home/baptiste/gcc/web-server/uWebSockets/uSockets »
rm -f *.o
cc -DLIBUS_NO_SSL -std=c11 -Isrc -flto -O3 -c src/*.c src/eventing/*.c src/crypto/*.c
ar rvs uSockets.a *.o
r - context.o
r - epoll_kqueue.o
r - gcd.o
r - libuv.o
r - loop.o
r - openssl.o
r - socket.o
r - wolfssl.o
make[1] : on quitte le répertoire « /home/baptiste/gcc/web-server/uWebSockets/uSockets »
examples/EchoServerThreaded.cpp: In lambda function:
examples/EchoServerThreaded.cpp:43:14: error: no matching function for call to ‘uWS::TemplatedApp<false>::ws<main()::PerSocketData>(const char [3], <brace-enclosed initializer list>)’
             }).listen(9001, [](auto *token) {
              ^
In file included from examples/EchoServerThreaded.cpp:1:0:
src/App.h:99:20: note: candidate: template<class UserData> uWS::TemplatedApp<SSL>&& uWS::TemplatedApp<SSL>::ws(std::__cxx11::string, uWS::TemplatedApp<SSL>::WebSocketBehavior&&) [with UserData = UserData; bool SSL = false]
     TemplatedApp &&ws(std::string pattern, WebSocketBehavior &&behavior) {
                    ^~
src/App.h:99:20: note:   template argument deduction/substitution failed:
examples/EchoServerThreaded.cpp:43:14: note:   cannot convert ‘{SHARED_COMPRESSOR, (16 * 1024), 10, ((1 * 1024) * 1024), <lambda closure object>main()::<lambda(std::thread*)>::<lambda()>::<lambda(auto:11*)>{}, <lambda closure object>main()::<lambda(std::thread*)>::<lambda()>::<lambda(auto:12*, std::string_view, uWS::OpCode)>{}, <lambda closure object>main()::<lambda(std::thread*)>::<lambda()>::<lambda(auto:13*)>{}, <lambda closure object>main()::<lambda(std::thread*)>::<lambda()>::<lambda(auto:14*)>{}, <lambda closure object>main()::<lambda(std::thread*)>::<lambda()>::<lambda(auto:15*)>{}, <lambda closure object>main()::<lambda(std::thread*)>::<lambda()>::<lambda(auto:16*, int, std::string_view)>{}}’ (type ‘<brace-enclosed initializer list>’) to type ‘uWS::TemplatedApp<false>::WebSocketBehavior&&’
             }).listen(9001, [](auto *token) {
              ^
examples/BroadcastingEchoServer.cpp: In function ‘int main()’:
examples/BroadcastingEchoServer.cpp:46:6: error: no matching function for call to ‘uWS::TemplatedApp<false>::ws<main()::PerSocketData>(const char [3], <brace-enclosed initializer list>)’
     }).listen(9001, [](auto *token) {
      ^
In file included from examples/BroadcastingEchoServer.cpp:1:0:
src/App.h:99:20: note: candidate: template<class UserData> uWS::TemplatedApp<SSL>&& uWS::TemplatedApp<SSL>::ws(std::__cxx11::string, uWS::TemplatedApp<SSL>::WebSocketBehavior&&) [with UserData = UserData; bool SSL = false]
     TemplatedApp &&ws(std::string pattern, WebSocketBehavior &&behavior) {
                    ^~
src/App.h:99:20: note:   template argument deduction/substitution failed:
examples/BroadcastingEchoServer.cpp:46:6: note:   cannot convert ‘{SHARED_COMPRESSOR, ((16 * 1024) * 1024), 10, ((1 * 1024) * 1024), <lambda closure object>main()::<lambda(auto:11*)>{}, <lambda closure object>main()::<lambda(auto:12*, std::string_view, uWS::OpCode)>{}, <lambda closure object>main()::<lambda(auto:13*)>{}, <lambda closure object>main()::<lambda(auto:14*)>{}, <lambda closure object>main()::<lambda(auto:15*)>{}, <lambda closure object>main()::<lambda(auto:16*, int, std::string_view)>{}}’ (type ‘<brace-enclosed initializer list>’) to type ‘uWS::TemplatedApp<false>::WebSocketBehavior&&’
     }).listen(9001, [](auto *token) {
      ^
examples/EchoServer.cpp: In function ‘int main()’:
examples/EchoServer.cpp:45:6: error: no matching function for call to ‘uWS::TemplatedApp<true>::ws<main()::PerSocketData>(const char [3], <brace-enclosed initializer list>)’
     }).listen(9001, [](auto *token) {
      ^
In file included from examples/EchoServer.cpp:2:0:
src/App.h:99:20: note: candidate: template<class UserData> uWS::TemplatedApp<SSL>&& uWS::TemplatedApp<SSL>::ws(std::__cxx11::string, uWS::TemplatedApp<SSL>::WebSocketBehavior&&) [with UserData = UserData; bool SSL = true]
     TemplatedApp &&ws(std::string pattern, WebSocketBehavior &&behavior) {
                    ^~
src/App.h:99:20: note:   template argument deduction/substitution failed:
examples/EchoServer.cpp:45:6: note:   cannot convert ‘{SHARED_COMPRESSOR, (16 * 1024), 10, ((1 * 1024) * 1024), <lambda closure object>main()::<lambda(auto:11*)>{}, <lambda closure object>main()::<lambda(auto:12*, std::string_view, uWS::OpCode)>{}, <lambda closure object>main()::<lambda(auto:13*)>{}, <lambda closure object>main()::<lambda(auto:14*)>{}, <lambda closure object>main()::<lambda(auto:15*)>{}, <lambda closure object>main()::<lambda(auto:16*, int, std::string_view)>{}}’ (type ‘<brace-enclosed initializer list>’) to type ‘uWS::TemplatedApp<true>::WebSocketBehavior&&’
     }).listen(9001, [](auto *token) {
      ^

Sorry for this, this is I am sure a really noob question. Everyone has to start one day. (btw I am an experienced web developer, just a noob in building c++ programming infrastructure).

Well, only the compilation of the Echo application crash : I got a working HelloWorld and HelloWorldThreaded apps (and Update...), so I suppose it works well, only the "echo" demo apps miss something.

Hello, I think the examples require C++20 but the library itself does not. So you probably need GCC9 or Clang10. You can always write the examples in C++17 but then they look less "declarative".

I tried with GCC10, which fully supports C++20. Only the "ECHO" examples does not work, so I do not matter. The other examples work well, and, YES!, I developed my first c++ web server in 1 hour.

Ok : Just installed G++-10 (https://askubuntu.com/questions/1192955/how-to-install-g-10-on-ubuntu-18-04) and all examples work well !
Thanks a lot.

Here is an "hello world" application example, with all the instructions to make it run from Debian / Ubuntu systems. I will check it on a newly installed debian / ubuntu / mint distro, this could help noobs like me to know how to use it starting from nothing.


updated support for the OS X platform

copying all the files to osx
making sure it works with the latest osx platform

This pr is hopelessly out of date. If it is still relevant, please rebase it, otherwise I will be closing it shortly.

please close. we got osx support in the meantime added.


docs: 404 on picker docs

Type: docs

Ionic Version: 2.x
Platform: desktop browser
Wrong URL reference of date picker docs

This issue was moved to driftyco/ionic-site#1070


[ZEPPELIN-4645]. Only list the interpreters that the user has permission in creating note and interpreter binding

What is this PR for?
It doesn't make sense to list the interpreters that user don't have permission. This PR just filter the interpreters that user don't have permission in the creating note page and interpreter binding section.
What type of PR is it?
[ Improvement]
Todos

[ ] - Task

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-4645

How should this be tested?

CI pass and tested manually

Screenshots (if appropriate)
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

@zjffdu
According to the PR description it's not clear whats happens to the currently selected default interpreter in interpreter binding section, in case the user has no permission for the default interpreter.
From my point of view, the default interpreter should not be changed automatically due to missing permission of any user trying to run the note. Instead the default interpreter should remain visible and remain the default. In case the user has no access, the current error message should be presented, when the user will try to run the note.
What do you think?

Thanks for the comment @weand Actually I would always add default interpreter in the interpreter binding without checking its permission. Sorry for the unclear PR description, I will update it.

ping @weand

yeah thats fine.

Thanks @weand will merge if no more comment


Add and delete sub-forms - frontend

Add queries and mutation for adding and deleting sub forms

Implemented.


CAD-3257 generator service:  absorb defaultGeneratorScriptFn

This moves the (configuration -> generator run script) translation function into tx-generator-service.nix from the workbench into the NixOS service definition.

bors r+


vmtests: deduplicating code using LVH library for arm64 support

Deduplicate some codes shared between vmtests and LVH:

inject LVH runner.RunConf into vmtests RunConf and reuse existing LVH configuration fields.
use LVH runner.BuildQemuArgs along with custom vmtests logic.
removes the support for HVF, we would need to put it in LVH to get this again but I don't think this is used nowadays.

This is mainly so that we benefit from the arm64 support as LVH was recently updated to support it.

I wouldn't mind a review @kkourt so that I can rebase afterward and make sure I don't rebase twice :)


Update development dependencies for pulumi/pulumi-provider-build-environment

Hello!

Vote on this issue by adding a 👍 reaction
If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details

Should we update the development tools in  pulumi/pulumi-provider-build-environment? specifically this section: Dockerfile
golangci-lint to v1.60.2
pulumictl to v0.0.46
The only consideration is that Goreleaser has been updated to v2 (the current version is v2.2.0).

Yes, we should update these dependencies (and keep them up-to-date). Thanks for opening the issue. There is some concern over breaking folks (especially with the Goreleaser bump), so we'll need to message this somehow to give folks a heads-up.


Small adjustment for Mac

Hi,
I just checked if it would run on a Mac after LDAP was not working in the past and it is working. I have some remarks.
If you could change the 'sed -r' to 'sed -E' in the 'setup.sh' script it would both work on linux and mac. I did not tested it yet but for GNU sed -E should be an alias for -r. Also the script is depending on 'jq'. Maybe making a remark about this would be nice because not all machine have this installed. For a Mac a user can install it using brew install jq.
The remark about the Mac and LDAP in the README.md can be deleted because you are now using a data volume instead of a bind.
Keep up the good work.

Hi @r00tc0d3,
thanks for your feedback.  A pr to address the points you've mentioned is more than welcome.

Also the script is depending on 'jq'.

That is not really true. jq is purely optional

jq is optional you are correct, but it will leave are remark.

For ref: https://github.com/zokradonh/kopano-docker/pull/104 (pr about sed)
I have adapted the setup script to also redirect errors. now no message will be printed when jq is not found.

Great!


linuxPackages: various changes

Motivation for this change

Setting features was broken. And nobody noticed for years.
There were not enough of hardened kernels.

All the old hashes stay the same (not a mass rebuild, despite how it looks).
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] Linux

[X] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[X] Some kernels do boot and work.
[X] Fits CONTRIBUTING.md.

I'd merge the hardned kernel stuff without much hesitation. Note that initially it was only meant as a sandbox for testing potentially controversial features, so I'm kind of meh on adding more hardned variants apart from latest, but if somebody wants it then I'm not against it.
The features stuff looks okay to me but it's outside my purview.

Would you mind if I cherry-picked only the commits affecting hardened-config?

Note that initially it was only meant as a sandbox for testing
potentially controversial features, so I'm kind of meh on adding more
hardned variants apart from latest, but if somebody wants it then
I'm not against it.

I didn't notice any slowdowns after booting into it and dmesg now
clearly shows that I have some security problems, which has a nice
motivating effect :)
TBH, I think we shouldn't just enable most (except PANIC_* and
DEBUG_*) of those options by default. Users that need to run legacy
software can use an "insecure" kernel.

Would you mind if I cherry-picked only the commits affecting hardened-config?

Nah, feel free to pick whatever you like. But the last commit (xen)
depends on the first, so they are not exactly independent.
Also, the first commit, IMHO, needs to be merged to 17.09 too. I was
really surprised features didn't actually do anything (they have no
effect on master, _xen_dom0 kernel has the same hash as the
default).

TBH, I think we shouldn't just enable most (except PANIC_* and

Oops, I meant "I think we should".

I agree some of the configs are "obviously good" and should be moved. I just find it tedious to do so (as it arguably should be, given the number of users affected compared to just the hardened-config).

I've picked the hardened-config stuff. Will pick the features cleanup as well unless somebody objects.

@GrahamcOfBorg eval

@GrahamcOfBorg eval

undefined variable ‘linuxPackages_hardened_copperhead’ at
/home/grahamc/.nix-test-rs/repo/38dca4e3aa6bca43ea96d2fcc04e8229/mr-est/grahamc-laptop/pkgs/top-level/all-packages.nix:12829:31

Should I add an alias for that? Btw, not all the packages in those
linuxPackages evaluate (same as before this patchset).

[X] Automated checks pass.
[X] All kernels I touched compiled fine locally.

Its perfect.

ping @grahamc and @joachifm

ping!

@GrahamcOfBorg eval


Huggingface API: Llama 3.1 8b

implement Llama 3.1 8b for llama 3.1 8b
here the place where we should work on it
API file
ps: we need to implement it for cpu ,GPU is optional for now

we removed the support of hugginface transformer and stick with ollama usage only


fix(learn): add test to return early pattern

Currently no test for a = 0 or b = 0. This results in if(a<=0 || b<= 0) also being a valid solution
Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #38750

Hi @ERElli
Thanks for the quick PR.
We would need a third party confirmation on the issue thread (or anywhere else, like forum) before we can land this. Keeping this blocked for the moment.

Thanks @moT01!
I realized that I hadn't named my branch properly after I had made the PR and there were already comments. I wasn't sure if closing the PR and renaming the branch was the right thing to do.
I'll make sure to name my branch properly next time!


[Bug] Invalid legend (type: scroll) horizontal paddings when data names are too short

Version
5.5.1
Link to Minimal Reproduction
https://echarts.apache.org/examples/en/editor.html?code=PYBwLglsB2AEC8sDeAoWtJgDYFMBcya6GOAHmAQOQBKOAZjgE5OzB2wCGsA6jgEYBnCGByUANEXQCArnxHkqAMQ4BrHLAAiHMB3GTYuOhViUAxjmgjGlIgF8J6MMGBZIIAqmIZGEAOa-mKmEcAFsbdHsiXADoABMPfTAATxB8EwFTRhcsPS9gHwtjSgA3JkhTDhyHYkNjAAZqx1AqEIhY2Nxc4j582KZuNrAACwIARkbYEA52iGhfAgBtBtg6gF0J2O0ORcoAZUpVu2qBJggcAUX9Ty9YaA4QtMoAQVNzAQFYRSywiccUx5AZy6XkY0wg0guJgArHUAKTA4ibHSXG7EJCwYqVaRpUZ1AAsAA4xLd7o99rBIqj0OjMVhsQQAOwAZihxLuDyoGggzFMYEoFN-aIxWLSUIJy3ZjwAoiEOBAcgL9F4aSKCIS8WzSVQAKrQKBwJ6xAT8ylUlV0tJMuoSrUmABqbRwwFghuNFKVsHWHtCICGHCEkOuqOCIV2yVwCSpxAEftiwAA7gAhOmMMYND1eGPTBMAeTodBOYAAGgR01GpLGEwBhFz5KiMXx8DgACmWbeJdQAdFCAJThKm2D2DrzDz0oWwAbiAA
Steps to Reproduce
Nothing, just open the demo.
I set paddings in 0, but rendered horizontal paddings are not 0.
Current Behavior
On scrollable legend horizontal paddings are invalid when data names are too short.
If name is long enough, paddings become correct.
In plain legend paddings are corrent too.
Tested in chrome and firefox, behavior is the same.
Expected Behavior
Horizontal paddings must be equal to the value specified in the settings
Environment
- OS: windows 11
- Browser: chrome/ff
- Framework:

Any additional comments?
No response

type:'scroll' adds some horizontal space because it has pageIcons. Their size could be adjusted with pageIconSize.
Otherwise padding works correctly both horiz/vertical, regardless of name lengths - Demo

📌 please close issue if problem solved.

With long enough names paddings are OK without setting any pageIconSize properties. demo

Setting pageIconSize solves the problem, but with short and long names behaviour is not the same.


 For example, I use sudo !! all day long when on I'm linux systems:
$ apt-get install something
E: Could not open lock file /var/lib/dpkg/lock - open (13: Permission denied)
E: Unable to lock the administration directory (/var/lib/dpkg/), are you root?
$ sudo !!
sudo apt-get install something
[sudo] password for test:

Cool, thank you!


Arrow keys not always working Windows 10 cmd

When I use yo from Windows 10 cmd, the arrow keys are not always working. It then sticks into some kind of readline history list. This is not always happening. Unfortunately I can't pinpoint what is causing it. Using yo version 1.4.8.

This is reported on the Node repo https://github.com/nodejs/node/issues/2996

Don't forget that you can select menu items by their index (one based).

From the node bug, it looks like no keypress events at all are working on windows.

That is not true as most of the time I am perfectly able to walk through
the menus with the arrow keys. Somehow it gets stuck sometimes in some
floating input line itself and then all keypresses are flowing into this
floating input line instead of the registered event handlers.
2015-09-24 19:41 GMT+02:00 Simon Boudrias<EMAIL_ADDRESS>
From the node bug, it looks like no keypress events at all are working on
windows.
—
Reply to this email directly or view it on GitHub
https://github.com/yeoman/yo/issues/360#issuecomment-143000197.

I meant, sometime windows just don't register them, and when that happen you don't receive any keypress event. The issue though is not always happening as sometime the keypress events are correctly fired.

I using "git bash" and i have same problem. But Windows Command Prompt works wery well.

That have been fixed and shipped on Node.js side. https://github.com/nodejs/node/issues/2996

@SBoudrias Still appears to be broken.
Will not work for me on Node 5.8 and Yeoman 1.7
Can we reopen please? https://github.com/nodejs/node/issues/5384

@Schandlich see the issue I just posted please.

@SBoudrias Thank you. That is where I found the link to nodejs/node#5384. :D

still broken here. :(

It's broken for me in Git Bash, but seems to work in cmd.exe
$ npm -v
3.3.12
$ node -v
v5.5.0

yo --version
1.7.0

I'll just put a screenshot to express my feels:

The issue has something to do with a recent windows update I think. It wasn't there few days ago.

I have the same issue, there is someone who already solved it?

@nyxjoel access the generator through the yo menu. The arrow bug is a regression in Node.js, they need to fix it on their side :(

and they did. get the latest node, npm, and yo. at least in my case, it's been fixed.

Note: It is fixed in version >= 6.2.2.
Not for 4.4.5, which they recommend for most users.

Still having this issue in Git Bash on Windows 10

Yep, issue still exists in Git Bash on Win10
I was able to bypass it one day, by setting some xterm function, sadfully I've lost it and now there's no way to make it work on Git Bash.

www.Windows10KeySale.com
You can purchase from here: www.Windows10KeySale.com, 100% working!?

Reported as spam.

Has there been any fix for this?? Still happening for me on git bash windows 10

Git Bash is not a TTY command line tool. As such, it doesn't support any interactive features.
You can use cmd, powershell, and a lot of other command line tool on Windows. Just make sure it is a TTY.

Ok Thanks, that's frustrating cause it works with git version 1.9.5.msysgit.0.

Can this issue be reopened? I'm unable to use Yeoman in Windows 10.
$ yo --version
2.0.0
$ node -v
v8.1.0
Both Powershell and Command Prompt don't let me get past choosing options in the first menu. I can actually switch between them, but once I hit enter, it asks for a name for my project and then, when i type, nothing shows up, it's like it's not captuing my keyboard input...
I read some of the messages above and I decided to try git bash, which showed a slightly different behavior, but ultimately also didn't get me through the first stages much less create the source code!

Exactly the same issue as @prubini87:
windows 10
npm -v         => 5.0.3
node -v        => v.8.1.0
yo --version => 2.0.0
I can select the first option using arrow keys, then 100% of the input is ignored.
How can this be treated as solved if people is still getting this issue?

@prubini87 @CamiloTerevinto looks like Node 8 broke the readline module again. You should report your issues there.

@SBoudrias Yep, you may be right sir, I've rolled back to Node 6 and it's working just fine. It seems to be related to Node 8 indeed. Could you provide @CamiloTerevinto and I a link where we should report this, please?

Bug was reported the Node https://github.com/nodejs/node/issues/13557 you can follow that issue to be notified once resolved.


Excessive locking

I looked at your locking mechanism out of interest because I wrote a locking library myself. I noticed that you may be locking quite excessively and I'm not sure if it's done purposely. For example, if you want to lock on a specific file or on AllStreams, you would not be allowing concurrency for different files/streams. If you don't want this behaviour, you need to consider using a keyed locking library instead, such as the one I made at https://github.com/MarkCiliaVincenti/AsyncKeyedLock which also allows pooling helping to reduce allocations.

I came across this on some ocassions I believe, it's possible to deadlock in some cases if you try to run some of the API synchronously.

But it's not exactly related to synchonously/asynchronously but rather with regards to concurrent asynchronous tasks.

For example, if you want to lock on a specific file or on AllStreams, you would not be allowing concurrency for different files/streams.

The current implementation should allow concurrent usage of multiple filestreams for a single file, and concurrent usage of filestreams from different files. The intent of the implementation is:

The library is configured to allow up to MaxOpenFiles filestreams to be open. This limit is enforced by ensuring only that many readers, or writers, can be active at a specific point in time: https://github.com/alanmcgovern/monotorrent/blob/86fa0a246860185087adfe74dcd140d13a7bbff6/src/MonoTorrent.PieceWriter/MonoTorrent.PieceWriter/DiskWriter.cs#L169-L175 . This async semaphore is entered before a filestream is claimed, and is released when the read/write operation is complete.

Assuming MaxOpenFiles is a large number and there are multiple readers/writers and there's only 1 file [0] in the torrent. Each concurrent reader/writer will need to claim a filestream. In order to do this, it a temporary lock is taken on the AllStreams locker to synchronize access to the underlying collection of available streams. With this lock is held the list of already-open streams is scanned for an available one, and if one is found the stream is locked and returned. At this point the lock on AllStreams is released. Any other threads which need a stream for this file are unblocked now.

The actual read/write operation is executed with just two locks held: the ratelimiting semaphore from point 1, and the lock on the specific filestream from point 2. Once the read/write operation completes, the lock on the filestream will be released and then the ratelimiter will be released too and things are done.

[0] if there are multiple files in the torrent, then they'll all lock on a different AllStreams object, which means multiple threads could claim streams at the same time.
There is also an implicit assumption that access to the Streams dictionary ( https://github.com/alanmcgovern/monotorrent/blob/86fa0a246860185087adfe74dcd140d13a7bbff6/src/MonoTorrent.PieceWriter/MonoTorrent.PieceWriter/DiskWriter.cs#L80 ) is not multi-threaded, and that's guaranteed by virtue of the code invoking the DiskWriter methods being 'correct'.


Adding SnapStart.NET8 and update dependencies

Description of changes:
Adding a new project folder for .NET 8 snapstart with class library deployment. This includes handler warming methods. Also updated all dependencies (was required for Amazon.Lambda.Core to get new Snapstart hooks)
Testing done:
Deployed manually and verified no exceptions in BeforeCheckpoint. Also hit URL to test.
sam build --parameter-overrides LambdaArchitecture=arm64 && sam deploy
Merge Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.
General

[x] I have read the CONTRIBUTING doc
[x] I used the commit message format described in CONTRIBUTING
[x] I have updated any necessary documentation, including READMEs and comments (where appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@jeastham1993 as FYI


add generic vm_device to device.map in f_load instead of assuming .img

This is related to https://github.com/michaeldexter/vmrc/pull/1. It may not be as robust as the other PR, but it does work for me with both img and zvol linux VMs. Basically it does not assume that a grub-style VM uses an .img disk.

Added and testing.
Thanks!


Make notebooks iframeable

Currently they are not by default. Turn on allowing iframes.

Will the default be allow_iframes = False? It would be good to have the person deploying make a conscious decision to enable.

Yeah, I agree! That makes sense.

this is working now yeah?

Yeah, but hackily. Need to document and set it up properly still...
@willingc agreed, default should be false, and even after you should be
able to specify list of domains you can iframe it on.
On Tue, May 16, 2017 at 6:24 PM, Chris Holdgraf<EMAIL_ADDRESS>wrote:

this is working now yeah?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/yuvipanda/builderhub/issues/11#issuecomment-301958904,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAB23jS7GwM1YL9gaCF9CXcxykYIfDJWks5r6kw0gaJpZM4NXCOz
.

--
Yuvi Panda T
http://yuvi.in/blog

@minrk is this done now?

No, notebooks are not launched with iframe permissions at the moment

This still doesn't seem to work - are we planning on making notebooks iframeable? If not we should close this.

No one answered the above so I'll close this, feel free to reopen.


[WIP] Move flux out of the switch domain

Description: As requested by @pvizeli, moves the flux component out of switch domain.
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
Example entry for configuration.yaml (if applicable):

Checklist:
If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

@balloob  @pvizeli  I'm finding it hard to get a good solution for the "turn on/off" of individual flux groups, since the component can have different flux devices.
Previously, each list of lights would be a flux switch, wich could be turned on and off normally. Now, since this is not a switch, that doesn't work implicitly.
Example:
flux:
  office:
    lights:
      - light.office_desk
      - light.office_floor
  livingroom:
    lights:
      - light.tv_lamp

The problem is turning on/off only "office", for example. I was considering adding "turn on" and "turn off" services, and passing the id of the "group" to turn on/off, but that is a weird API IMO.
Any tips?

I think that it's out of scope of this component to offer turning groups on/off. If a user wants that, they can use the group component.

@balloob the problem os not having a group of lights. The thing is that each group can have different settings (different hours or color temps). Currently, I can just define multiple Flux switches, one for each group. Moving this out of the switch platform, how would I do that (and still allow turning off the component for a single group)?

Update the config to support multiple flux groups

@balloob supporting multiple groups is not an issue, I have that almost done. But with the current switch.flux, I can turn each flux switch on/off. How do you suggest turning individual groups on/off?

Ah, my bad for being short in my previous answer.
If you would support multiple Flux groups, each flux group would get their own switch.
Passing no entity id into turn_on/off will just trigger all, as is with the light component etc

This PR seems to have gone stale. Closing it. You can reopen it when you're ready to finish it.


Dependabot alerts

Is anyone else getting this alert?

Yes @mikeysan, and it seems it has a high severity:

Running npm update and commiting the new package.lock.json should fix this right?

Yes, that's it @BOLT04 and the suggested fix is correct too.

That is a great suggestion for people wanting to contribute to open source. Updating dependencies 😍

Can I contribute to this, since no one has yet been assigned?

Sure 👍

I fixed a few outdated(high-risk) dependencies in #305
However, there are some node_module issues that are popping up in the build...

I did not update these as I was not sure about weather updating them might create issues...


Feature request: allow custom templates

First off thanks for providing such a great library! 😸 It works great and does the job pretty solid.
One thing which would be great is to enable an option to provide custom HBS templates (at least for the CSS part).
There's a simple use case for SASS users, if we want to create additional SCSS variables for the icons for eg. using it in a custom ::after selector we need to copy the content: '' attribute manually but a variable (eg. $icon-<dashed-name>) would be way easier to use.
Eventually providing more details about the icons in the callback results would also provide a great option to add more custom scripts without too much regex action.
Let me know what you think about these ideas :)

Hi @janbiasi - I'm both planning to add off-the-shelf SCSS/SASS templates (they would be optional and require the user specifying asset types manually) - and add the ability of using custom generators (maybe both from npm modules and local scripts) although both features depend on available time for this project, and can't promise a time frame (SCSS/SASS should be really simple so hopefully sooner).
Regarding codepoints, I've just published version 1.0.13 which will return the codepoints used in the generator output, so you can in the meantime render your own SCSS template by just using the JS API.
Hope that helps - I'll keep the issue open as a reminder.
Also, if you'd like to contribute, you could open a PR for SASS/SCSS (I'd say let's have both syntaxes). You can find the generators logic in src/generators/asset-types and their templates under templates.
Obviously not expected but absolutely welcome :)
Hope this help

@tancredi thanks a lot for the positive feedback, I'm really looking forward to try 1.0.13 asap. and if I have some spare time this week or weekend I'd like to start working on the said PR for sure 😸!

@janbiasi @tancredi  I addressed this issue in #18 and also provide an extensive default scss template.
see #18 which builds on top of #17 so think away all but the last commit

OK folks firstly thank you so much for the contributions :)
So, this got a bit messy as I've addressed PRs individually and there seemed to be a bit of duplication between the two.
@casaper your PR is still outstanding and after merging most of the changes from @janbiasi I've sorta created a bunch of merge conflicts.
Meanwhile, @casaper , I've merged your changes for better typing of asset type options.
I've included a bunch of other minor changes and fixes to both PRs - and as mentioned, did not add the custom template options for the command line utility as I wanted to keep granular options limited to config file / JS API - which is still open for discussion, I was just trying to get the basic features in first.
@casaper, is there anything from the PR that is missing in the currently merged implementation?
Apologies if I'm making a bit of confusion, limited time and lots to do on this :)

Oh - find all new things on version 1.0.16

@tancredi

Apologies if I'm making a bit of confusion, limited time and lots to do on this :)

No Worries. I was fully aware that there will be incompatibilities with what @janbiasi did in #20.
But that's fine. Now that it is merged I will see If I still have anything to add, to what @janbiasi  did.
I basically need to make it compatible with what I already started using in a project using my dirty temporary package fantasticon-cas.
But I think that this won't be a big deal... Expect another PR in some Time and you will see. I'll close #18 now, since it is anyway not mergeable as is.

@casaper really awesome changes, thanks a lot! 🥳 One thing I noticed lately was that if a dev uses JS-in-TS (// @ts-check directive) it will complain in the fontTypes and assetTypes options as enums aren't valid with the TS directive.
Should we just ignore that as "known limitation", because making these types compatible would mean that we'd need to change the enum to types and consts to keep the current type logic, what do you think @casaper & @tancredi ?

@janbiasi

Should we just ignore that as "known limitation", because making these types compatible would mean that we'd need to change the enum to types and consts to keep the current type logic, what do you think

Yeah, you're right, I also noticed it not being fully as it should be there.
I did it like this because otherwise It would have required quite some refactoring in the default options thing.
I think in general I agree with you, probably the Enums are not the best to go with any more. Rather literal string types in order to improve usability and just plain old constants for looping with the available options.

@casaper literal string types + constants for looping would also be the way I'd go, maybe we should open another feature request for that case, I will close this issue as the requested feature is now available :)

@janbiasi perhaps tomorrow I might have some spare time to make a little pr regarding this

@janbiasi  @tancredi
I actually have an idea to structure the whole config file simpler, and I think it will be easier to use.
Instead of long explanations here, just wait for my PR. It will be like a proposal for a change in config object structure... Allot easier explained in code than in text here :wink:

@janbiasi || @tancredi
I Think it would speed up things if you guys reviewed these PR's @ DefinitelyTyped:

https://github.com/DefinitelyTyped/DefinitelyTyped/pull/49721
https://github.com/DefinitelyTyped/DefinitelyTyped/pull/49722


HGNC parser update

Description
Update to the HGNC parser during the first stage of the xref sprint. SeeENSCORESW-2896.
Parser now using Text::CSV.
CCDS database replaced with an ensembl core database.
Testing
no unit tests.
Manually tested with human.
Results seem consistent.

Coverage remained the same at 81.414% when pulling 1a68211db6774d99ee0bf4ed0e240e325234b8cc on xref_HGNCParser into 1eac5974de1e3739f0380b4eba23eaeaff348111 on feature/xref_sprint.


Global restructuring:  Principles and ideas -- Overview

In order to improve the usability of HED, I think we need to look at each of the top level subtrees and refine the upper levels.  If there are too many items in the tag tree at the top of the tree it is hard for users to find things and to understand the structure for tagging.  Usability guidelines suggest no more than 10 items, but ideally 5 to 7 items at a level.
The top level subtrees are:
Event
Item
Attribute
Action
Participant
Sensory presentation
Questions

What order should the top level subtrees appear in the hierarchy?
Are these the top level subtrees that we need?

We have also not resolved the issue of how context tags apply across multiple events.  For example, we have a series of trials under condition A and another under condition B.  How can we convey that?
There is also Paradigm, which I propose we ignore for the time being.  I would like to take Paradigm out and put in a separate task description hierarchy where we also tag control variables and other things.  The current Paradigm section is somewhat similar to CogPO and to Cognitive Atlas --- but not the same.  It is confusing and I think makes it difficult to tag because it doubles the size of the top level hierarchy.

This has been accomplished by the HED-3G draft specification.

This has been accomplished by the HED-3G draft specification.


selfhost/parser: Support comma in import list

Quick change to make parser recognize a comma after an import list name.

Was already implemented :ninyak:


Make Projected Grid render better in dark areas and lit scenes

Made the color channel use o.Emission instead of o.Albedo to avoid rendering with scene lighting, helping with grid visibility in dark areas, or on dark materials.

Thanks @Kerfuffles this is much better!


[BUG]

Release version
V4.1.2
Describe the bug
While pacifying different names for Dev and QA environment for SHG, system is not able to pick the gateways on QA side, instead it is adding the Dev gateways to the QA gateways list. Reason being there is no configuration mentioned for mapping for SHG in Configuration.prod.yaml.
Please provide examples and what are different configurations with examples based on use cases.
Expected behavior
Is should not copy the gateways to the new region instead it should use the gateway of the target regions for tagging to respective Api's published for sync during the running of the pipeline.
Actual behavior
Process is copying the Dev Self hosted gateways to the QA region and not using the QA gateways.
Reproduction Steps
Use different names as per environment for Dev and Stage and run the APIOps pipeline with no specific configuration motioned for this condition. System is no intelligent enough to recognize that it has to ignore the dev gateways and use QA gateways.

@kumviv23 - We don't support using different resource names across environments. For example, the following is unsupported:

DEV

shg-dev

QA

shg-qa

Instead, what we recommend is using the same resource name and overriding values in configuration

DEV

shg (override DEV-specific values in configuration)

QA

shg (override QA-specific values in configuration)


Fix minor typos

After a quick review, fixes some minor typos in the documentation.  Didn't see a CONTRIBUTING.md, so hope proposing for master branch is ok.  Thanks for contributing to the community! 🎉

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Pending assignment with my open source review board internally. I'll follow up.

I am closing this due to staleness as the spelling mistakes have been resolved in #635 and elsewhere, and the other changes that have not been resolved seem more opinionated.


minimum python version?

I am trying to make fuzzy matches on Python 3.8 (Windows 7 64 bit), with polars 0.20.13 already installed
Any way of installing this package?  Looks like I need a higher Python version but that's the highest one I can use with this OS.
I wonder if there is any workaround
Thanks
@abubelinha

I don't think this package uses any feature that's specific to higher python version. Only work around off the top of my head is to lower the python version requirement. Let me attempt it and see if anything breaks

Can you try out 0.1.3, I've lowered the python version requirement to 3.8, it should work

Off-topic:
In README you describe Fzf-style search syntax "is taken almost verbatim from the Fzf README".
Are you meaning this project? https://github.com/junegunn/fzf?#search-syntax

Hold on.
Still on Python 3.9 machine, where I could successfully install but I couldn't run usage.py example. This line raises error:
from polars_fuzzy_match import fuzzy_match_score
E:\Dropbox\python>c:\python39\python polarex.py
Traceback (most recent call last):
  File "E:\Dropbox\python\polarex.py", line 78, in <module>
    polarsfm_usage(); sys.exit()
  File "E:\Dropbox\python\polarex.py", line 11, in polarsfm_usage
    from polars_fuzzy_match import fuzzy_match_score
  File "c:\python39\lib\site-packages\polars_fuzzy_match\__init__.py", line 4, in <module>
    from polars.type_aliases import IntoExpr
ModuleNotFoundError: No module named 'polars.type_aliases'

The code:
def polarsfm_usage():
	""" Taken from https://github.com/bnm3k/polars-fuzzy-match/blob/master/examples/usage.py """
	import polars as pl
	from polars_fuzzy_match import fuzzy_match_score

	df = pl.DataFrame(
		{
			'strs': ['foo', 'foo quz BAR', 'baaarfoo', 'quz'],
		}
	)
	pattern = 'bar'
	out = (
		df.with_columns(
			score=fuzzy_match_score(
				pl.col('strs'),
				pattern,
			)
		)
		.filter(pl.col('score').is_not_null())
		.sort(by='score', descending=True)
	)
	print(out)

OK, I can confirm successful installation on Python 3.8 (Windows 7 64 bits).
Thanks a lot for your help


[BUG] Indices deletion in leader cluster do not get replicated in follower cluster

What is the bug?
Deleting an index on the cluster leader does not get replicated in cluster follower.
How can one reproduce the bug?
Steps to reproduce the behavior:

Create an index on the cluster leader and add some documents
Enable the cluster replication and wait until the index gets replicated to the follower.
Delete index on the cluster leader
Wait and check on cluster follower nothing gets replicated the same index still exist

What is the expected behavior?
The deleted index on cluster leader should get delete in cluster follower.
What is your host/environment?

Distributor ID:	Ubuntu
Description:	Ubuntu 20.04.6 LTS
Release:	20.04
Codename:	focal

Do you have any screenshots?
N/A
Do you have any additional context?
Using Opensearch:1.3.9

[Catch All Triage - 1, 2, 3, 4, 5]

@vivekkumar1008 Thanks for creating the issue.
This is expected behavior, this is intentional to stop replication on follower cluster when we delete index on leader index. We did this behaviour by keeping few things in mind like if some one wants to do cross cluster search on the follower cluster.
Cleanup of follower cluster indices needs to be done separately.

@nisgoel-amazon should we look into the path of giving this as an option ?

We can take it as a feature request and happy to have contributions.
But this is not an issue, so we can close this one after creating feature request.

Hey thank you so much to give insight info and also taking this request as future feature development.


Update index/generator version for registry-library

Please specify the area for this PR
What does does this PR do / why we need it:
This PR accomplishes the following:

index/generator reference updated to https://github.com/devfile/registry-support/commit/cf509ccd1a6b7a5a2d1c39563c95935175268c25 as it pulls in security fix
go mod vendor ran

Which issue(s) this PR fixes:
Part of https://github.com/devfile/api/issues/1620
PR acceptance criteria:

[ ] Test Coverage

Are your changes sufficiently tested, and are any applicable test cases added or updated to cover your changes?

Documentation (WIP)

[ ] Does the REST API doc need to be updated with your changes?
[ ] Does the registry library doc need to be updated with your changes?

How to test changes / Special notes to the reviewer:

/lgtm after all checks are passing.
I'm curious, I see it also updates the indirect dependencies of k8s.io to v0.29.2 (comes after the general bump we did for all modules). I don't expect any breaking changes as they would have been seen in the direct dependencies already. Just thinking in the future we might want to include this in a list of items when we do similar bumps across all modules/projects

Yeah I initially saw that too, I think it should be okay since the most breaking change of that bump was the Go version I believe? And this already uses 1.21.
+1 on taking note for when we bump other projects though.


Update event check for deleted at column to use getDeletedAtColumn

As per laravel's docs, the deleted_at column name can be changed by the user if he decided to define a constant in the model DELETED_AT.
This PR uses getDeletedAtColumn method provided by SoftDeletes instead of using the column name statically.
Ref: https://github.com/laravel/framework/blob/8.x/src/Illuminate/Database/Eloquent/SoftDeletes.php#L200

https://dev.to/patinthehat/authoring-great-pull-requests-13de

@Gummibeer Check now

https://github.com/spatie/laravel-activitylog/releases/tag/4.3.1


KAFKA-4565: Separation of Internal and External traffic (KIP-103)

To be done:

Add more tests.
Fix one test failure
Namespacing of security configs (in a separate PR)

Read through the whole thing and didn't have many comments - I'm impressed at how clean this fairly large change ended up. You touch a lot of places, but most of it fairly mechanical replacement since the APIs match well. Nice :)
Since this is large, let me know if there are areas you want me to give a bit more attention.

Thanks for the review @gwenshap! I'm particularly interested in feedback regarding the user-visible docs. I tried to address your suggestion and made a few other minor tweaks in the docs for listeners, advertised.listeners and server.properties. Let me know if you have suggestions for improvement.

Started a system tests run:
https://jenkins.confluent.io/job/system-test-kafka-branch-builder/665/
The only remaining thing on my list is an integration test containing two listeners with the same security protocol. Working on that at the moment.

System tests passed apart from the Streams system tests (both are failing in trunk as well).

I pushed an integration test and I am now happy with this PR. Please take a look @hachikuji and we can merge it once you're happy.


Update CI URLs to point to new caching infrastructure

Hello there!
This is an automated pull request submitted by @staticfloat to help package authors transition their Julia installation CI setups to the new binary provider URL.  Please take a look at this PR, and if there is a problem or it doesn't download correctly when CI runs, feel free to ping @staticfloat

Coverage remained the same at 100.0% when pulling 7ca4c2ca88e83eb321e8214cd901d68dff408620 on staticfloat:updated_ci_url into 80e2eb13dc146ae283153deaec2e6e87d43f65a6 on tlycken:master.

Coverage remained the same at 100.0% when pulling 7ca4c2ca88e83eb321e8214cd901d68dff408620 on staticfloat:updated_ci_url into 80e2eb13dc146ae283153deaec2e6e87d43f65a6 on tlycken:master.

Codecov Report

Merging #1 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master     #1   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines          35     35
=====================================
  Hits           35     35

Continue to review full report at Codecov.

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


Showing total count of user registered in homepage

Is your feature request related to a problem? Please describe.
Adding some stats in the homepage will be cool. The numbers maybe not need to be real-time.
Describe the solution you'd like
Some articles said we should use

Search API to get the total: https://developers.notion.com/reference/post-search
Or Page Property to get the unique values https://developers.notion.com/reference/retrieve-a-page-property

Additional context

Do not exposing any data except the count itself.
Implement cache?

I would like to work on this issue.
By homepage do you mean this page?

Where on this page would you like the user count displayed?
Also, can you give more information about the implement cache part?

No, homepage is https://www.tanyaaja.in/.
The landing page.

Ok I understand. Where in the landing page do you want it, and are there some wordings you want to provide me?

The section's title is: Total pengguna terdaftar


initializing time is too long and it seem that never stop

I refer your source ，and do some modify，but it struck in the step of initializing，I try so hard but can't find the reason ，I really hope you can help me to fix this problem，I will do whatever I got to assist to fix this Issues

Sorry it was the camera, the seller sell the bad on to me, i hope that i am not  bother you


TAB event doesn't trigger typeahead:selecetd

How to repeat :
 var myCustomCallback = function(eventObject, suggestionObject, datasetName) {
    alert('hello');
 }

myTypeAhead.on('typeahead:selected', myCustomCallback);

Then navigate on your autocomplete field, type few letter so you get a list of items, and then press TAB. It will select the first item, but it won't trigger the :selected event.

@jharding it works great with typeahead:autocompleted  thanks alot.


hover effect to social media icons

issue title: adding transition & animation to social media icons to adventure awaits - frontend project #3377
Close #3377
Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[] I have created a helpful and easy to understand README.md

@MohitGupta121 @pranjay-poddar @TusharKesarwani kindly review this PR and merge

thank you @TusharKesarwani


Getting "must NOT have additional properties" while generating sbom

I am trying to generate a SBOM for a Java project and I get the following error:
Executing 'mvn org.cyclonedx:cyclonedx-maven-plugin:2.7.9:makeAggregateBom -DoutputName=bom -DincludeTestScope=true -q' in /Users/****
Fallback to executing mvn dependency:tree -DoutputFile=/var/folders/ch/c299n4ss7cgcgt8fkdvnpvgh0000gn/T/cdxmvn-Kr5T5D/mvn-tree.txt
[
  {
    instancePath: '/metadata/component',
    schemaPath: '#/additionalProperties',
    keyword: 'additionalProperties',
    params: { additionalProperty: 'qualifiers' },
    message: 'must NOT have additional properties'
  }
]

I have executed the following command
cdxgen -t java
This looks very similar to the #443 issue
NodeJS version: 20.5.1
cdxgen version: 9.5.0

@mikamp116 just pushed a fix. Could you test with the branch fix/fineract-subcomp?
git clone https://github.com/CycloneDX/cdxgen
git checkout fix/fineract-subcomp
npm install
node bin/cdxgen.js -t java -o bom.json <path>

Hi @prabhu ,
Thank you for the quick answer, I just did and I get the following error:
node:internal/modules/esm/resolve:188
  const resolvedOption = FSLegacyMainResolve(packageJsonUrlString, packageConfig.main, baseStringified);
                         ^

Error: Cannot find package '/Users/{user}/repos/cdxgen/node_modules/connect/package.json' imported from /Users/{user}/repos/cdxgen/server.js
    at legacyMainResolve (node:internal/modules/esm/resolve:188:26)
    at packageResolve (node:internal/modules/esm/resolve:769:14)
    at moduleResolve (node:internal/modules/esm/resolve:831:20)
    at defaultResolve (node:internal/modules/esm/resolve:1036:11)
    at DefaultModuleLoader.resolve (node:internal/modules/esm/loader:251:12)
    at DefaultModuleLoader.getModuleJob (node:internal/modules/esm/loader:140:32)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:76:33)
    at link (node:internal/modules/esm/module_job:75:36) {
  code: 'ERR_MODULE_NOT_FOUND'
}

Node.js v20.5.1

I don't know if it is a problem in the app or in my current setup.
Thanks

@mikamp116 Not seen this error before. What if we try installing globally?
git clone https://github.com/CycloneDX/cdxgen
git checkout fix/fineract-subcomp
npm install
sudo npm uninstall -g @cyclonedx/cdxgen
sudo npm install -g .
cdxgen -t java -o bom.json <path>

Awesome @prabhu that worked
I got rid of that error, but the execution keeps falling back from mvn org.cyclonedx:cyclonedx-maven-plugin:2.7.9:makeAggregateBom to mvn dependency:tree, and this makes a mess in my Dependency Track instance.
Here is the output:
Executing 'mvn org.cyclonedx:cyclonedx-maven-plugin:2.7.9:makeAggregateBom -DoutputName=bom -DincludeTestScope=true -q' in /Users/*****
Fallback to executing mvn dependency:tree -DoutputFile=/var/folders/ch/c299n4ss7cgcgt8fkdvnpvgh0000gn/T/cdxmvn-nrC912/mvn-tree.txt

Do you know why does the fallback happen and how to prevent it?
Thanks in advance

@mikamp116 Can you invoke that mvn command and see if any errors are reported?
mvn org.cyclonedx:cyclonedx-maven-plugin:2.7.9:makeAggregateBom -DoutputName=bom -DincludeTestScope=true -q

Successfully running this command must produce bom.json files in the target directory.

Of course, I already tried it this morning and suprisingly it is not reporting any error, just a warning, here is the output:
[INFO] -------------< ***-service-poc-app >-------------
[INFO] Building ***-service-poc-app 1.1.x.0-SNAPSHOT
[INFO]   from pom.xml
[INFO] --------------------------------[ jar ]---------------------------------
[INFO]
[INFO] --- cyclonedx:2.7.9:makeAggregateBom (default-cli) @ ***-service-poc-app ---
[INFO] CycloneDX: Parameters
[INFO] ------------------------------------------------------------------------
[INFO] schemaVersion          : 1.4
[INFO] includeBomSerialNumber : true
[INFO] includeCompileScope    : true
[INFO] includeProvidedScope   : true
[INFO] includeRuntimeScope    : true
[INFO] includeTestScope       : true
[INFO] includeSystemScope     : true
[INFO] includeLicenseText     : false
[INFO] outputFormat           : all
[INFO] outputName             : bom
[INFO] outputReactorProjects  : true
[INFO] ------------------------------------------------------------------------
[INFO] CycloneDX: Resolving Dependencies
[INFO] CycloneDX: Creating BOM version 1.4 with 223 component(s)
[INFO] CycloneDX: Writing and validating BOM (XML): /{path}/***-service-poc-app/bom.xml
[INFO]            attaching as ***-service-poc-app-1.1.x.0-SNAPSHOT-cyclonedx.xml
[INFO] CycloneDX: Writing and validating BOM (JSON): /{path}/***-service-poc-app/bom.json
[WARNING] Unknown keyword additionalItems - you should define your own Meta Schema. If the keyword is irrelevant for validation, just use a NonValidationKeyword
[INFO]            attaching as ***-service-poc-app-1.1.x.0-SNAPSHOT-cyclonedx.json
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  3.209 s
[INFO] ------------------------------------------------------------------------

Note: I get this output when I run the command without the -q flag. If I attach that flag, the output is empty

Hi @prabhu ,
Sorry, I didn't understand your last message. Any clue on why the execution is falling back from mvn org.cyclonedx:cyclonedx-maven-plugin:2.7.9:makeAggregateBom to mvn dependency:tree ?

@mikamp116 Suspecting that makeAggregateBom is creating the bom json files in a non-standard directory. Can you share the full path to the generated files?

Ofc
[INFO] ------------------------------------------------------------------------
[INFO] CycloneDX: Resolving Dependencies
[INFO] CycloneDX: Creating BOM version 1.4 with 181 component(s)
[INFO] CycloneDX: Writing and validating BOM (XML): /Users/mikamp116/sca/java-service-poc/java-service-poc-app/bom.xml
[INFO]            attaching as java-service-poc-app-1.1.x.0-SNAPSHOT-cyclonedx.xml
[INFO] CycloneDX: Writing and validating BOM (JSON): /Users/mikamp116/sca/java-service-poc/java-service-poc-app/bom.json
[WARNING] Unknown keyword additionalItems - you should define your own Meta Schema. If the keyword is irrelevant for validation, just use a NonValidationKeyword
[INFO]            attaching as java-service-poc-app-1.1.x.0-SNAPSHOT-cyclonedx.json

Thanks @mikamp116 for confirming. Let me create a new ticket and share a PR to fix this.


Double-width characters in bspwm module

Double-width glyphs in bspwm workspaces get truncated when status is 'focused'. I tried to tweak many module parameters but couln't get rid of it. Any Ideas on how to fix this?
Was it working before?
First time I try it.
To Reproduce
[bar/lxpertoldi_hor]
override-redirect = true
width = 100%
height = 27
offset-x = 0
offset-y = 0
radius = 0.0
fixed-center = false
background = ${colors.background}
foreground = ${colors.foreground}
line-size = 3
line-color = #f00
border-size = 4
border-color = ${colors.background}
padding-left = 0
padding-right = 2
module-margin-left = 1
module-margin-right = 2
font-0 = Fira Code:style=Regular:size=10;1
font-1 = 3270Medium Nerd Font:size=11;1
modules-left = bspwm-lxpertoldi
modules-center =
modules-right =
wm-restack = bspwm
cursor-click = pointer

[module/bspwm]
type = internal/bspwm
label-focused = %icon%
label-focused-background = ${colors.background-alt}
label-focused-underline= ${colors.ylw}
label-focused-padding = 1
label-occupied = %icon%
label-occupied-padding = 1
label-urgent = %icon%!
label-urgent-background = ${colors.red}
label-urgent-padding = 1
label-empty = %icon%
label-empty-foreground = ${colors.foreground-alt}
label-empty-padding = 1

[module/bspwm-lxpertoldi]
inherit = module/bspwm
ws-icon-0 = "I;I"
ws-icon-1 = "II;II"
ws-icon-2 = "III;III"
ws-icon-3 = "IV;IV"
ws-icon-4 = "V;V"
ws-icon-5 = "VI;VI"
ws-icon-6 = "VII;VII"
ws-icon-7 = "VIII;VIII"
ws-icon-8 = "IX;IX"
ws-icon-9 = "X;X"
ws-icon-10 = "XI;XI"
ws-icon-11 = "XII;XII"
ws-icon-12 = "XIII;XIII"
ws-icon-13 = "XIV;XIV"
ws-icon-14 = "XV;XV"
ws-icon-15 = "XVI;XVI"
ws-icon-16 = "XVII;XVII"
ws-icon-17 = "XVIII; "
ws-icon-18 = "XIX; "
ws-icon-19 = "XX; "

Polybar Log
Nothing unusual here
Screenshots

Environment:

WM: bspwm
Distro: Arch Linux
Output of polybar -vvv:

polybar 3.4.2

Features: +alsa +curl +i3 +mpd +network(libnl) +pulseaudio +xkeyboard

X extensions: +randr (+monitors) +composite +xkb +xrm +xcursor

Build type: Release
Compiler: /usr/bin/c++
Compiler flags: -march=x86-64 -mtune=generic -O2 -pipe -fno-plt -Wall -Wextra -Wpedantic -O3 -DNDEBUG
Linker flags: -Wl,-O1,--sort-common,--as-needed,-z,relro,-z,now

Additional context
I use Fira Code as main font and 3270-Medium Nerd Font Patched for the extra glyphs.

Duplicate of #991
This seems to be a NerdFonts issue: https://github.com/ryanoasis/nerd-fonts/issues/442


⚠ Bofrost.at has degraded performance

In 4eabe3a, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1150 ms

Resolved: Bofrost.at performance has improved in 446ebc4.


Replace ingres with kong

Optional upgrade. I'll see how nginx ingress goes.
Thought kong has some additional features aside nginx.
https://charts.konghq.com/

/label later

for now ingress is enough plus I might try traefik.


[DOCFIX] Update chinese doc

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2506/
Test PASSed.


Resource base

Base classes for resource management

[Current coverage][cc-pull] is 33.52%

Merging [#4][cc-pull] into [master][cc-base-branch] will increase coverage by 19.92%

4 files in Source/Core/Source were modified. more

Misses -3
Hits +12

2 files (not in diff) in ...urce/Core/Source/Gfx were modified. more

Misses +2

@@           master         #4   diff @@
========================================
  Files           9         13     +4
  Lines         125        182    +57
  Methods         0          0
  Branches        0          0
========================================
+ Hits           17         61    +44
- Misses        108        121    +13
  Partials        0          0

[][cc-pull]

Powered by Codecov. Last updated by [9f07085...38a265b][cc-compare]
[cc-base-branch]: https://codecov.io/gh/MrJaqbq/Volkhvy/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/MrJaqbq/Volkhvy/compare/9f0708542b25136e0ce38be29e2d199dfbd01266...38a265bbe7f5c203bd7258a920666a149d0ee6fa
[cc-pull]: https://codecov.io/gh/MrJaqbq/Volkhvy/pull/4?src=pr


Fixed error "Index out of range" for VirtualInputs and VirtualOutputs

Originally, virtualInput and virtualsOutputs where MidiPortRef with a value of 0
In the previous commit, I replaced MidiportRef() by [MidiPortRef]() wich create an empty array of MidiPortRef.
Replacing [MidiPortRef]() by [MidiPortRef()] should solve the issue by creating an array of MidiPortRef whose first value is 0

Thanks!


Text input not following the keyboard

What is the best was to debug this behaviour for that I can send you a better issue description? I am asking because this should be fixed with Version 9.4.

Thank you for your kindness. But I can't reproduce. For me it seems that in my implementation the textinput does not recognize that it becomes firstResponder.
Maybe I miss something new in the initialization stage?
The only code I use here is
    inverted = false
    bounces = true
    shakeToClearEnabled = true
    keyboardPanningEnabled = true
    shouldScrollToBottomAfterKeyboardShows = true
    let attachmentIcon = UIImage(named: "icon_Attachment") as UIImage?
    leftButton.setImage(attachmentIcon, forState: .Normal)

    // TODO: Find better solution that does not break autoconstraints
    // ...
    leftButton.translatesAutoresizingMaskIntoConstraints = true // this breaks autolayout with constraints

    leftButton.frame = CGRectMake(8, 8, 25, 30)
    leftButton.contentMode = .ScaleAspectFill
    leftButton.imageView?.contentMode = .ScaleAspectFill
    leftButton.imageView?.clipsToBounds = false
    leftButton.tintColor = MyTheme.sbColorPaletteDarkGrey

    rightButton.setTitle("Send", forState: .Normal)

    textInputbar.autoHideRightButton = true
    textInputbar.maxCharCount = 4000;
    textInputbar.counterStyle = SLKCounterStyle.Split
    textInputbar.counterPosition = SLKCounterPosition.Top
    textInputbar.editorTitle.textColor = MyTheme.sbColorPaletteDarkGrey
    textInputbar.editorLeftButton.tintColor = MyTheme.sbColorPaletteDarkGrey
    textInputbar.editorRightButton.tintColor = MyTheme.sbColorPaletteDarkGrey

    textView.layer.borderWidth = 0
    textView.keyboardType = UIKeyboardType.Default
    textView.placeholder = "Type new message here..."
    textView.placeholderColor = MyTheme.sbColorPaletteMediumGrey

My class includes
class MessageStreamViewController: SLKTextViewController, MyCallbackDelegate, MyProtocol, UIImagePickerControllerDelegate, UINavigationControllerDelegate {

Neither the textInputbarDidMove is triggered nor the overridden didChangeKeyboardStatus does fire.

@headkit Do you by any chance override the method func target(forAction action: Selector,   withSender sender: Any?) -> Any? of SLKTextView? I modified the method to prevent "select/copy/paste menu" and screwed something up, when I removed the override it worked again. Spend my entire afternoon figuring it out...

@jpodcedensek hey, thanx for your thoughts on that - unfortunately I don't overwrite any SLKTextView methods at all. I am still diggin'...

strange - gone it is, working fine!

@headkit How to solve this problem? Could you let me know?

For anyone coming in after me: I had to disabled IQKeyboardManagerSwift using viewDidAppear and viewWillDisappear methods.

The above comment helped me.
I did the following:
    override func viewWillAppear(_ animated: Bool) {
        super.viewWillAppear(animated)

        IQKeyboardManager.sharedManager().enable = false
    }

    override func viewWillDisappear(_ animated: Bool) {
        super.viewWillDisappear(animated)

        IQKeyboardManager.sharedManager().enable = true
    }


Setting hot: false still includes /sockjs-node

After ejecting and setting hot: false in /scripts/start.js it stills include the sockjs-node/ websocket.
This should not happen, right?

You probably also want to take this line out too.

Yes, that fixed it! Thanks 👍


Pickle fails with models using FieldTracker

PicklingError: Can't pickle <function save at 0x3e10320>: it's not found as model_utils.tracker.save

This happens with custom User model in Django 1.5 (not sure if it matters). Haven't had a chance to try it with earlier versions or 1.6.

Just stumbled on this issue... would love to see a solution. ;)

Same here as @bforchhammer

I do not plan to work on this issue as my development priorities have shifted away from field tracker.
For anyone who wants to try fixing this: pickling needs to be fixed while avoiding the problem addressed in #143 (see discussion of regression tests for that in #147).

Seems like custom User models are ubiquitous enough that this issue is blocking for many django projects. Would love to know if anyone has found a workaround.

As a workaround, I used dill (https://github.com/uqfoundation/dill) instead of pickle in my code (for storing a queryset into cache).

Unable pickle model with FieldTracker:
tests/market/test_models.py:209: in test_market_offer_model_can_pickle
    pickle.dumps(self.market_offer)
/usr/lib/python2.7/pickle.py:1374: in dumps
    Pickler(file, protocol).dump(obj)
/usr/lib/python2.7/pickle.py:224: in dump
    self.save(obj)
/usr/lib/python2.7/pickle.py:331: in save
    self.save_reduce(obj=obj, *rv)
/usr/lib/python2.7/pickle.py:419: in save_reduce
    save(state)
/usr/lib/python2.7/pickle.py:286: in save
    f(self, obj) # Call unbound method with explicit self
/usr/lib/python2.7/pickle.py:649: in save_dict
    self._batch_setitems(obj.iteritems())
/usr/lib/python2.7/pickle.py:663: in _batch_setitems
    save(v)
/usr/lib/python2.7/pickle.py:286: in save
    f(self, obj) # Call unbound method with explicit self
/usr/lib/python2.7/pickle.py:748: in save_global
    (obj, module, name))
E   PicklingError: Can't pickle <function save at 0x7f9a0d358230>: it's not found as model_utils.tracker.save

I have following which allows me to pickle while preserving post-save signal functionality.
`    def finalize_class(self, sender, **kwargs):
    super(PickleFieldTracker, self).finalize_class(sender, **kwargs)

    # Instead of patching the save method, we define the method
    # This will be called manually, since we want to control order of execution
    def handler(sender, instance, **kwargs):

        if not isinstance(instance, self.model_class):
            return

        update_fields = kwargs.get('update_fields')

        if not update_fields and update_fields is not None:  # () or []
            fields = update_fields
        elif update_fields is None:
            fields = None
        else:
            fields = (
                field for field in update_fields if
                field in self.fields
            )

        getattr(instance, self.attname).set_saved_fields(
            fields=fields
        )

    # We want to send our signal last, else post save signals would not work
    # We do not want to patch save, since that would not allow us to pickle the model.
    original_send = post_save.send

    def send(**kwargs):

        # All original Post Save Signals are received
        ret = original_send(**kwargs)

        # Call our own function
        resp = handler(**kwargs)
        ret.append((handler, resp))
        return ret

    post_save.send = send

def initialize_tracker(self, sender, instance, **kwargs):
    if not isinstance(instance, self.model_class):
        return  # Only init instances of given model (including children)
    tracker = self.tracker_class(instance, self.fields, self.field_map)
    setattr(instance, self.attname, tracker)
    tracker.set_saved_fields()`

I just feel it is hackish to play around with signals execution order, so have not created PR it. If this seems fine, I will create PR for it.

@csimple: In your solution, behavior of the field tracker in a save method is a little different than before. Currently, if you write:
class Foo(models.Model):
    bar = models.CharField(_('bar'), max_length=64)
    field_tracker = FieldTracker(fields=['bar'])

    def save(self, *args, **kwargs):
        print(1, self.field_tracker.has_changed('bar'))
        super().save(*args, **kwargs)
        print(2, self.field_tracker.has_changed('bar'))

obj = Foo.objects.create(bar='bar')
obj.bar = 'new_bar'
obj.save()

You will get:
1 True
2 True

But if you run it using PickleFieldTracker proposed by @csimple, you will get:
1 True
2 False

I am wondering which behavior is the correct one.

@nomarek Since post save signals also return True (it returns True for both original as well as my solution), I think original behaviour was correct.
Which means my earlier assumption that this could be fixed by patching signal is not correct, and we are back to square one.

Guys, I was so happy when found this repo, apparently, as much as I got disappointed when I discovered this bug.
It makes this library absolurely useless, because we use caching extensively. I very much appreciate if someone could fix this issue, because I spent a couple of hours but didn't figure it out myself :(

I also would very much benefit from being able to add an object which has a field_tracker field to my redis cache! Can't do it without being able to pickle it

Since 3.2.0 FieldTracker patches Model.save instead of instance.save, this should not be an issue anymore.


Revert "Fix case when scrollToPosition will not update internal state…

This reverts commit 32855b8abfc0980237970e148fc625edc78f7e01.
fixes #1378
Thanks for contributing to react-virtualized!
Before submitting a pull request, please complete the following checklist:

[ ] The existing test suites (npm test) all pass
[ ] For any new features or bug fixes, both positive and negative test cases have been added
[ ] For any new features, documentation has been added
[ ] For any documentation changes, the text has been proofread and is clear to both experienced users and beginners.
[ ] Format your code with prettier (npm run prettier).
[ ] Run the Flow typechecks (npm run typecheck).

Here is a short checklist of additional things to keep in mind before submitting:

Please make sure your pull request description makes it very clear what you're trying to accomplish. If it's a bug fix, please also provide a failing test case (if possible). In either case, please add additional unit test coverage for your changes. :)
Be sure you have notifications setup so that you'll see my code review responses. (I may ask you to make some adjustments before merging.)

Codecov Report

Merging #1446 into master will decrease coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1446      +/-   ##
==========================================
- Coverage   90.36%   90.35%   -0.02%
==========================================
  Files          59       59
  Lines        1744     1742       -2
==========================================
- Hits         1576     1574       -2
  Misses        168      168

Impacted Files
Coverage Δ

source/Grid/Grid.js
94.44% <100%> (-0.04%)
:arrow_down:

Continue to review full report at Codecov.

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


SQLType cannot be assumed to be static.

This is a core assumption that we make which is inherently false. If statements in the select clause can dynamically change this. I want to know what preparing that query in PG and asking for the return type of that query gives back in this case.

PG does not allow queries which conditionally change their return type.


Add support for X-Loadbalance-To header

We can't use LoadBalanceTo header when passing requests from vulcanproxy to mantrid because at some point vulcanproxy changes (decamelize) LoadBalanceTo into Loadbalanceto (don't know why, maybe to be compliant with some RFC specification?). So we need to introduce another form and we chose X-Loadbalance-To.

I should probably clear that header on nginx so that it's not possible to set it from external world

Looks good. What if both headers are provided?

~$ curl -w "%{http_code}\n" -sL -H 'X-Loadbalance-To: larva-elb.fs.sandbox' -H 'LoadBalanceTo: larva-elb.fs.sandbox' localhost:8080/http_status_responses/204
204

~$ curl -w "%{http_code}\n" -sL -H 'X-Loadbalance-To: larva-elb.fs.sandbox' -H 'LoadBalanceTo: WRONG-elb.fs.sandbox' localhost:8080/http_status_responses/204
204

~$ curl -w "%{http_code}\n" -sLI -H 'X-Loadbalance-To: WRONG-elb.fs.sandbox' -H 'LoadBalanceTo: larva-elb.fs.sandbox' localhost:8080/http_status_responses/204
HTTP/1.0 503 Service Unavailable
Cache-Control: no-cache
Connection: close
Content-Type: text/html

503

If both are set, X-Loadbalance-To takes precedence over LoadBalanceTo (intentionally).

already working on sandbox and staging


Support fetching tags of a resource

Required for https://github.com/BlueBrain/nexus/issues/2709 where resource revisions also show any associated tags

Codecov Report

Merging #366 (14e61a9) into main (82e0d2b) will decrease coverage by 0.09%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##             main     #366      +/-   ##
==========================================
- Coverage   88.66%   88.56%   -0.10%
==========================================
  Files          23       23
  Lines         547      551       +4
  Branches      130      132       +2
==========================================
+ Hits          485      488       +3
- Misses         60       61       +1
  Partials        2        2

Impacted Files
Coverage Δ

packages/nexus-sdk/src/Resource/index.ts
95.12% <66.66%> (-2.25%)
:arrow_down:

packages/nexus-sdk/src/Project/index.ts
80.00% <100.00%> (+1.05%)
:arrow_up:

Continue to review full report at Codecov.

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


Handle scaledown of queues and topics at deploy time

Instead of scaling down queues and durable subscriptions in shutdown hooks, schedule and/or perform the scaledown from the standard-controller. This allows the scaledown to be performed reliably and removes the need for shutdown hooks.

Already done.


DocumentUpdateError / Method not supported on wrong data

I was having an issue with updating a document in arango when I got the following error:
arango.exceptions.DocumentUpdateError: [HTTP 500][ERR 405] method not supported
I tried several things and in the end the problem was actually the data I was sending in the vertex_collection.update method, not the method itself (the document key was missing).
I'm raising the issue just to say that the error raised is a bit misleading and maybe should be fixed.
Other than that, great library, really easy and intuitive to work with. Hat off! 👏 🎩

Hi @Sushi2all,
Thanks for the kind words! I'm glad you find the library useful.
The message is actually coming directly from ArangoDB. Python-arango simply passes along all server error details and tries not to touch them as much as possible. So this actually looks like a bug on ArangoDB's part. The error code and message seem incorrect.


Per game LED presets

I don't know, maybe I'm missing something, but I cannot find in the docs any guide of how to control leds from a pc. You know like all the other led controllers allow you to illuminate only buttons that are used by the currently launched MAME game.

If you have examples of other projects that do that, it would help. Right now there's no communication from the PC to GP2040 at all other than acting as a standard HID device.


Setup application insights authentication in asp.net framework

in my current code
I do
TelemetryConfiguration.Active.ConnectionString = ...
And below is the code to ask for ad authentication found at
https://learn.microsoft.com/en-us/azure/azure-monitor/app/azure-ad-authentication?tabs=net
var config = new TelemetryConfiguration
{
ConnectionString = "InstrumentationKey=00000000-0000-0000-0000-000000000000;IngestionEndpoint=https://xxxx.applicationinsights.azure.com/"
}
var credential = new DefaultAzureCredential();
config.SetAzureTokenCredential(credential);
It's not clear to me how to mix the code to setup ad authentication for ai with the way i am using to provide cn string
Thanks
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 20c3df07-5950-bd23-4bfc-47fe636a5f44
Version Independent ID: c12e72fa-3996-0246-a5cd-e2f9856e04a3
Content: Azure AD authentication for Application Insights - Azure Monitor
Content Source: articles/azure-monitor/app/azure-ad-authentication.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @AaronMaxwell
Microsoft Alias: aaronmax

@sabbadino
Thanks for your feedback! We will investigate and update as appropriate.

@sabbadino, We will now proceed to close this thread as we have not heard back from you. If there are further questions regarding this matter, please tag me in your reply and reopen the issue. We will gladly continue the discussion.

Sorry I did not reply you back soon .
Its not clear how to apply the code mentioned in that article for asp.net not core apps.
In such apps the ai cn string is in the appinsights file .. one does not need to write any code to provide ai cn string ..
In other docs or is shown now to setup ai cn string read from app settings
TelemetryConfiguration.Active.ConnectionString = ...
It's not clear at all how one would wire the code above with
var config = new TelemetryConfiguration
{
ConnectionString = "InstrumentationKey=00000000-0000-0000-0000-000000000000;IngestionEndpoint=https://xxxx.applicationinsights.azure.com/"
}
var credential = new DefaultAzureCredential();
config.SetAzureTokenCredential(credential);
Where should I place the piece of code above ? Global.asax app start ?
I think you should provide a full project sample on GitHub for an aspnet  not core app.
Enrico

@sabbadino, thank you for the clarification.
Yes, you could do this in Application_Start() method of Global.asax file. The following sample snippet should help understand it better.
protected void Application_Start()
        {

            ...
            ...

            var credential = new DefaultAzureCredential();
            TelemetryConfiguration.Active.SetAzureTokenCredential(credential);

        }

The TelemetryConfiguration.Active.SetAzureTokenCredential() method is used here to update the credential on the active ApplicationInsights configuration - read from config file.
Hope this helps. Please let me know if you have any questions. I will also share this feedback with our team to have the example/code snippet in doc clearer.

Yes, now it makes to me ..
Pls update documentation accordingly... Thanks


Windows SubSystem for Linux fails

Just saw your library mentioned in Hacker News and it's something I've been looking for a while now. I'm not a Haskell or WSL expert so hoping someone can help me out.
Repro instructions in WSL
sudo apt-get install haskell-stack
stack upgrade
cd reanimate
stack build
Up until this point everything works fine
Then it fails here
stack ./examples/doc_drawCircle.hs
stack: WARNING! Expecting "stack" at line 2, column 4
stack: WARNING! Missing or unusable stack options specification
stack: WARNING! Using runghc without any additional stack options
2020-10-29 17:16:55.48: Listening...
2020-10-29 17:16:57.48: Opening browser...
Start : This command cannot be run due to the error: The system cannot find the file specified.
At line:1 char:1
+ Start "/mnt/c/Users/Mark/Dev/reanimate/.stack-work/install/x86_64-lin ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [Start-Process], InvalidOperationException
    + FullyQualifiedErrorId : InvalidOperationException,Microsoft.PowerShell.Commands.StartProcessCommand

2020-10-29 17:16:59.44: Browser opened.

Hi @msaroufim. By default reanimate tries to use a browser window to preview your animation. I don't have any experience with WSL but I'm guessing it is not possible to run firefox or chrome from there.
You can either render directly to a video file (stack ./examples/doc_drawCircle.hs render) or you can use reanimate directly from Windows.


Adding New Styles/Updating Style to 2015 BJCP

I would like to see the option to add new brew styles, and also update the style.xml file to the 2015 BJCP Style guide

PR #74 merged, this will be in the next version!
It will require an uninstall / reinstall to pick up the new data.