feat: dynamic epoch+committee

Why
Our Consensus is designed to work within epochs. After a set amount of time an epoch should end, a new committee should be decided on randomly from all the nodes in the network and restart itself with the new epoch parameters and committee members. Before we were just running a fixed version of narwhal from the config.
This PR adds a wrapper around our Narwhal(consensus.rs). This wrapper will pull epoch/committee information from application layer before starting narwhal and start narwhal with the right committee and parameters. This wrapper will also send a request when the epoch end time has passed to signal the node is ready to move epochs.
The epoch change works like this: Every node in the committee keeps a timer to epoch end, when that timer passes it sends a certificate to be ordered that says it is ready to change but stays in the current epoch. When 2/3rds of the nodes in the committee have sent a certificate saying they are ready to move epochs, that last certificate will trigger the epoch change for all nodes. This way we ensure all nodes move to the next epoch at the same exact point.
Fixes # (issue) - No issue was created for this but I will be better with this going forward
What

Renamed service.rs to narwhal.rs. Added a wrapper around it in consensus.rs that is in charge of starting the entire consensus service.
If a node is not on the committee it will no longer run narwhal at all.
We no longer need a committee.json for narwhal. Everything should be in the genesis.toml in ursa-application.
Some utils like like load_json where moved to ursa_utils crate from consensus
Introduced a forge solidity repo into the mono repo in the contracts folder. Has the contracts we are loading in the genesis.toml.
Since epoch starts based on the params of the genesis file i made a simple genesis script that will update epoch 0 to start at the time the script was ran, the script also takes an optional parameter of epoch length in milliseconds(defaults to 5 minutes) cargo run --bin genesis 600000 would update the genesis to start now with an epoch time of 10 min. This script also sets genesis committee to the nodes from https://github.com/qti3e/ursa-nodes for easy local testing and running of the committee.

Demo

Notes

Checklist

[x] I have made corresponding changes to the tests
[x] I have made corresponding changes to the documentation
[x] I have run the app using my feature and ensured that no functionality is broken

The code doesn't look bad on a line-by-line base; most of the lines are doing what is intended to do.
---- Side note:
However, for history-keeping reasons, I also have to say here, probably one last time, that the idea of having EVM as our application layer stinks; we do not run arbitrary ad-hoc logic, which would require redeploying smart contracts (we can always upgrade the node), and even if the choice of a VM would make sense due to deploy on-demand benefit over pure Rust, EVM would still not be the best option to go.
The code already has introduced a ton of inefficiency and reduction in the performance of the reads from the EVM (and its blocking nature), is not fit for a highly available product such as a CDN, and would demand us to have a secondary lock-free in-memory cache of the entire balance map (just an example) in the code base, introducing yet another layer of indirection to the architecture.
The idea that we have to go through a local connection to ourselves to get to the data that is already supposed to be in our memory is just... for lack of a better word bad design.
But apart from the general talk about the entire idea, which I only mentioned, there is a record of me somewhere that I have said it; the implementation of the idea is good, and kudos to @daltoncoder.
P.S The other reason I chose this PR to put up is that this is the first PR trying to merge Solidity source code into the main. However, I do not expect any changes here because, as far as i am concerned, this is not my place to make this decision. And this has been discussed before I was part of the team anyway. I do not expect any changes. I just wanted to have a publicly available note about my opinion about this

Thanks for the review and using my PR to state your sidepoints. I wont get into what i agree or disagree with here, its not really the place. We have multiple calls a week to discuss protocol design, which is how we ended up with these decisions.


Cleanup CONTRIBUTING.md

Pre-Submission Checklist

[x] Your pull request targets the staging branch of FreeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm run test-challenges. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[ ] Closes currently open issue (replace XXXX with an issue no): Closes #XXXX

Description

Lint with Remark: correct numbered lists numbers, add spaces to lists,
use headings in single decreasing increments, reformat table for
readability, correct spelling error, correct emphasis syntax, add
clarification for npm requirement, and reflow line length for 80
column limit.

Thanks @carlosvigil


Added wait-up setup-plausible to fix cryptic error

Fixes

Fixes #2489  by @YashGaur000
Description
Added a "wait up" recipe that waits for Plausible to be available before running set_plausible.sh. Now plausible service is ready for requests and won't end up with cryptic error.

Testing Instructions

Checklist

[ ] My pull request has a descriptive title (not a vague title likeUpdate index.md).
[ ] My pull request targets the default branch of the repository (main) or a parent feature branch.
[ ] My commit messages follow best practices.
[ ] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[ ] I tried running the project locally and verified that there are no visible errors.
[ ] I ran the DAG documentation generator (if applicable).

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

I'm converting this to a draft to avoid successive pings. @YashGaur000 If you're still planning to work on this PR, please let us know by commenting here. Otherwise, I will close this PR by the end of the week.

@YashGaur000 You're welcome to open a new PR if you want to continue this effort. I'm closing this PR.


How to get UpdatedAt timestamp for a contact?

Hello. I'm trying to sync contacts with local storage and remote database. And I'm thinking updatedAt field would be the best to determine if there are any latest changes that should be synced. But I could not find such field in the Contact.
On Android, I found ContactsContract.Contacts.CONTACT_LAST_UPDATED_TIMESTAMP to do something similar. But there are also some issues with this field highlighted here.

@agent515 This field is part of the API, but as far as I know it's usually empty, so this is unfortunately not available. (This is why I created the app Quis 🙂)


Possible flaky create partitions test

If this is a bug report, please fill out the following:

Version of Ruby: circleci/ruby:2.5.1-node
Version of Kafka: 2.1.1
Version of ruby-kafka: master

Please verify that the problem you're seeing hasn't been fixed by the current master of ruby-kafka.
Steps to reproduce
I've been getting failing tests for the create partitions test in CI but only on a single version of Kafka, which feels to me like a flaky test due to async operations or something like that
@dasch how would you like to see this fixed?
Ideas:

Add a static sleep(1) between the create_partitions_for and partitions_for check
Add a retry so we check several times (e.g. by adding https://github.com/NoRedInk/rspec-retry or https://github.com/rstacruz/rspec-repeat)
Other?

Failing test: https://circleci.com/gh/Shopify/ruby-kafka/295?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link

I think the retry approach could work well. I'd love to see a PR.


Release Camel K 1.1.0

Let's collect the good things we're doing to do a release early next week

New Features

Upgraded Camel K Runtime to 1.4.1 (Camel 3.4.0, Camel-Quarkus 1.0.0-CR3, Quarkus 1.6.0.Final) (#1620, @nicolaferraro)
When subscribing to a Knative channel or broker, replies are ignored unless explicitly enabled (#1619, @nicolaferraro)

Other Notable Changes

fix(api): Declare integration Flow as json.RawMessage (#1591, @astefanutti)
Added flag to the install command to allow setting the operator image pull policy (#1612, @johnpoth)
Fixed issue missing help details as reported on #1585 (#1596, @orpiske)
feat: Structured traits configuration serialization (#1600, @astefanutti)
feat: Integration scale sub-resource works with HPA (#1587, @astefanutti)
feat(crd): Generate CRDs from APIs (#1582, @astefanutti)
doc: Add monitoring documentation (#1569, @astefanutti)

Is 1.1.0 officially released? Just making sure before merging Homebrew/homebrew-core#58346

@SeekingMeaning, vote is still ongoing. It will close tomorrow at 16:00 CET and then released if everything is ok (so far, it is).

Camel K 1.1.0 is now released.


resholve: work around nixpkgs aarch64-darwin issues

These are hopefully both just short-term workarounds.

add an oil-dev patch to work around build failure documented in NixOS#154203
override configargparse to disable tests and work around failure reported in NixOS#156807

The OfBorg results should clarify whether this works.

Result of nixpkgs-review pr 155251 run on aarch64-darwin 1

3 packages failed to build:

bashup-events32
bashup-events44
resholve

Reopening for review since ofborg aarch64-darwin ran clean.

the fix makes sense, and ofborg pass.
happy to merge at this point. Leaving this open in case you are looking for more feedback.

I am a bit unsure about the override approach, but if it passes your gut check it is probably fine. :)

Like you said, it's a short term fix :-)


Goto homepage button redirects on login page

Description
When a user is not logged in and user clicks on the getting started button then the user is redirected to the 404 page where a goto homepage button is visible when the user clicks on that button user is redirected to the login screen which is inappropriate for the user.
Screenshots

Additional information
No response
What browser are you seeing the problem on?
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I want to work on this issue. (optional)
[ ] I am a GSSoC'23 contributor

I would like to work on this issue

Hey @Sunny-unik Thank you for your contribution ✨


WIP: Add support for sending event requests via proxy

Do not merge...just yet.

test script:
var Raven = require('raven');
<EMAIL_ADDRESS>{
    transport: new Raven.transports.HTTPSProxyTransport({
        proxyHost: 'localhost',
        proxyPort: 8080,
    })
}).install();

var x = {};
x.something();

http in URL -> HTTPProxyTransport
https in URL -> HTTPSProxyTransport

https://gist.github.com/ndmanvar/7ae7b4db7bf1cda30545c1480fde77fe

Will open if revisited/needed.
Gist is the current working solution.

Closed this, as I am working off of new branch: https://github.com/getsentry/raven-node/pull/397


enable proxy.config.http.negative_revalidating_enabled by default

I think we should enable proxy.config.http.negative_revalidating_enabled by default in ATS 8. This config allows ATS to serve stale when it cannot reach the origin. With the current settings the client receives 502s.  The config is overridable.

The documentation is not completely clear. If this is enabled, then on subsequent requests to the failed object, ATS will revalidate the negatively cached item against the origin? And in that case, if the revalidation fails, it will serve stale content? Although that seems wrong because the cached object would be the negative reply.

How about if I add:
That is, when enabled, |TS| will serve stale when it can not reach the origin server (similar to the stale-if-error Cache-Control directive).

What about this -
.. ts:cv:: CONFIG proxy.config.http.negative_revalidating_enabled INT 0
:reloadable:
:overridable:
Negative revalidating allows |TS| to return stale content if revalidation to the origin fails due
to network or HTTP errors. If it is enabled, than caching the negative response, the current
(possibly stale) content is preserved and served. Note this is considered only on a revalidation
of already cached content and failure means unable to communicate via HTTP to the origin. If the
origin provides a valid response that is not considered a failure in this context.
A value of 0 disables serving stale content and a value of 1 enables keeping and serving stale content if revalidation fails.
.. ts:cv:: CONFIG proxy.config.http.negative_revalidating_lifetime INT 1800
How long, in seconds, to consider a stale cached document valid if If
:ts:cv:proxy.config.http.negative_revalidating_enabled is enabled and |TS| receives a negative
(5xx only) response from the origin server during revalidation.

Don't get this sentence:

If it is enabled, than caching the negative response, the current (possibly stale) content is preserved and served.

I think this should only be in play for stale content -- wouldn't be revalidating fresh content, right? Perhaps
If it is enabled, rather than caching the negative response, the current stale content is preserved and served.

If the origin provides a valid response that is not considered a failure in this context.

Perhaps missing a comma after response?
Would a 5xx error be considered a valid response? We use cache hierarchies extensively, and this setting kicks in when our parents return a 5xx error to the children. Perhaps 'a valid response specific to the object'?
proxy.config.http.negative_revalidating_lifetime
appears to be both reloadable and overridable.

"If enabled, rather than caching the negative response on a revalidate, the ..."

Hmmm. I need to look at the code.

Docs updated, ship this in ATS 8.

Still think proxy.config.http.negative_revalidating_enabled  needs to be enabled by default in 8.

Looks like this is done.


use_frameworks! causing missing module

First off: Thanks for a wonderful library.
Secondly: I'm trying to get a swift pod working in my objective-c project. To do that I have to specify use_framworks! in the podfile. When I do this my #import <DateTools/DateTools.h> isn't found anymore. I've seen the previous issue regarding this, which is resolved but for me it still doesn't work. I've also tried @import DateTools but it doesn't work either.
The version of DateTools I run is 2.0.0 according to the Podfile.lock. My podfile:
platform :ios, '9.0'

source 'https://github.com/CocoaPods/Specs.git'
use_frameworks!
workspace 'MyProject'

target 'MyProject' do
    pod 'AFNetworkActivityLogger'
    pod 'FMDB'
    pod 'FCModel'
    pod 'DateTools'
end

The issue is resolved. Was something messed up with my project settings. Don't know why, but it works now.


Implement Binary Search in C++

Issue Description

Need the C++ implementation of Binary Search algorithm.
Corresponding post on website
Permitted languages - C++

Guidelines

Your code will be live on a tutorial website, so code readability and understandability will be given paramount importance.
Please follow coding conventions listed in CONVENTIONS.md
The directory structure of the project is pretty obvious, but take a moment to go through STRUCTURE.md if you are confused where to place your code.
Please add a few sample input output test case files too. Minimum 1. The more the merrier!

Can I take up this Issue?

Sure go ahead :smile:


KVStore: Expand the usage of thread-wise alloc/dealloc trace

What problem does this PR solve?
Issue Number: close #8835
Problem Summary:

Make it possible that we use thread-wise alloc/dealloc trace in modules other than KVStore
Record both alloc and dealloc, rather than alloc-dealloc

We put this utility in KVStore/FFI, because it bond to FFI interface which is between C++ and Rust. That is also why we call is a joint.
What is changed and how it works?
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/pull-integration-test

/pull-unit-test

/pull-unit-test

/retest

/retest

/test pull-unit-test

/retest

/retest


failed to fetch stdout, stderr, and "full log" in job page after the job completes

"Failed to fetch"
But the "go to tracking page" actually has the log.

cannot reproduce now.


Ubuntu 16: VM doesn't start, process stuck on network configuration.

Since few days I have the issue that every time I do vagrant up the process stuck and doesn't continue. And the only way to make it works is destroying all the VMs and start from scratch.
$ vagrant up m1 a1 a2 p1 boot
Bringing machine 'm1' up with 'virtualbox' provider...
Bringing machine 'a1' up with 'virtualbox' provider...
Bringing machine 'a2' up with 'virtualbox' provider...
Bringing machine 'p1' up with 'virtualbox' provider...
Bringing machine 'boot' up with 'virtualbox' provider...
==> m1: Checking if box 'mesosphere/dcos-centos-virtualbox' is up to date...
==> m1: Clearing any previously set forwarded ports...
==> m1: Clearing any previously set network interfaces...
==> m1: Preparing network interfaces based on configuration...
    m1: Adapter 1: nat
    m1: Adapter 2: hostonly
==> m1: Forwarding ports...
    m1: 22 (guest) => 2222 (host) (adapter 1)
==> m1: Running 'pre-boot' VM customizations...
==> m1: Booting VM...
==> m1: Waiting for machine to boot. This may take a few minutes...
    m1: SSH address: <IP_ADDRESS>:2222
    m1: SSH username: vagrant
    m1: SSH auth method: private key
    m1: Warning: Remote connection disconnect. Retrying...
    m1: Warning: Remote connection disconnect. Retrying...
    m1: Warning: Remote connection disconnect. Retrying...
==> m1: Machine booted and ready!
==> m1: Checking for guest additions in VM...
==> m1: Setting hostname...
==> m1: Configuring and enabling network interfaces...

Versions:

VirtualBox: 5.0.20
Vagrant box: 0.6.3
Vagrant: 1.8.1
DC/OS: 1.7
DC/O vagrant: v0.7.0
Ubuntu: 16.04

Is anyone with the same issue?

I haven't had confirmed success on Ubuntu 16, but haven't had it triaged yet either. Ubuntu 14 and 15 should work as the host OS.

Same problem here, Have anyone solved this?

I have the same problem.override.vm.network :private_network, ip: machine_type['ip'] stuck the process.

After first time run the vagrant up m1 boot a1 p1, I change the 213 line in Vagrantfile:
override.vm.network :private_network, ip: machine_type['ip'] to override.vm.network :private_network, ip: machine_type['ip'], auto_config: false
It is solved! You can try it.
@estebanlll @karlkfi @juanjoserivera

Nice, thanks @xyangwang, now I can continue from where I left.
Btw, can it be related to the issue of the virtualbox modules aren't signed for Ubuntu 16.04 and they are not properly integrated on the kernel and later on it generates this behavior on vagrant?

Thanks @xyangwang, it worked!, looks like this problem is not only related to ubuntu hosts, I am running this on Windows 7 and I had same issue.

Not sure why that would fix things...
The Configuring and enabling network interfaces... message is printed here:
https://github.com/mitchellh/vagrant/blob/master/plugins/providers/virtualbox/action/network.rb#L130
The CentOS network is configured here:
https://github.com/mitchellh/vagrant/blob/master/plugins/guests/debian/cap/configure_networks.rb#L12
I see an Ubuntu 16.04 comment in there:
https://github.com/mitchellh/vagrant/blob/master/plugins/guests/debian/cap/configure_networks.rb#L46
Looks like it was added March 20 (very recently):
https://github.com/mitchellh/vagrant/commit/0505771481222b4893daa5454ff6c9c16f471fa4
So it's possible that an upgrade of Vagrant (or building from master) might also fix your problem.

Vagrant v1.8.1 was released on Dec 24, 2015. So it wont contain the fix listed above. You'll need to build Vagrant from master in order to use it on Ubuntu 16.04 with Debian boxes.

on second thought… that’s for ubuntu 16 guests, not hosts... I doubt CentOS would error without that change,

Here's another bug that affects Ubuntu 16 that is only fixed on master: https://github.com/mitchellh/vagrant/issues/7073
I've added a note to build from source: https://github.com/dcos/dcos-vagrant#operating-system

Ok, I've now tested Ubuntu 16 myself and verified that even building from source does not work because plugins cannot be installed.
Fix: https://github.com/mitchellh/vagrant/pull/7376
Unfortunately, it looks like releases are pretty far apart. So I'm marking Ubuntu 16 as not supported until this is resolved.

FWIW, the problem is with Vagrant 1.8.1 being incompatible with Ruby 2.3.
Ubuntu 16 comes with Ruby 2.3 "manually installed" in a way that can't easily be downgraded or reinstalled via apt-get.

I've clarified that Ubuntu 16 is currently unsupported: https://github.com/dcos/dcos-vagrant#operating-system
Sorry guys, but I don't want to have to manually manage /etc/hosts because Vagrant won't let you install a plugin.


It would be nice with counts for annotations in overall stats

Description of feature
As it is now, we only list the number of ORFs annotated. But this is difficult to interpret as there's nothing to relate to. It would hence be nice with counts also for this. This of course requires that these scripts read also the counts table, but this should not be a major issue, IMO.

solved on PR #209


[SPARK-29439][SQL] DDL commands should not use DataSourceV2Relation

What changes were proposed in this pull request?

Create a new node ResolvedV2Table, and resolve UnresolvedV2Relation to this new node instead of DataSourceV2Relation.
Why are the changes needed?

DataSourceV2Relation is a scan node. It should be used when we want to scan a v2 table. However, the DDL commands do not need to scan a v2 table, they just need a node to hold the v2 table. It's possible that there are rules trying to match DataSourceV2Relation and convert it to something else (e.g. a new DataSourceV2ScanRelation), for better data scan. Unfortunately doing this will break these DDL commands. It's better to have a separated node to hold the v2 table for DDL commands.
Does this PR introduce any user-facing change?

no
How was this patch tested?

existing tests

@brkyvz @rdblue

DataSourceV2Relation is a scan node.

I disagree. It is a relation. It is converted to a scan node when we convert to a physical plan.
I don't think there is much utility to making a change like this, so I'm -1.

I think the solution to the problem is to change the way AlterTable and DeleteFrom wrap a relation.
The same problem exists for write plans: the table that will be written to is resolved to a relation. That relation should not be replaced or modified by a rule like early push-down, so the relation is not added as a child of the plan. If it is not a child, then rules are not automatically run on it.
We can fix this problem the same way. AlterTable should not list the NamedRelation as a child so that rules do not automatically run on it. In addition, ResolveTables should be updated to resolve the relation when it is UnresolvedV2Relation and replace it with DataSourceV2Relation. Last, AlterTable should only return resolved if the relation is resolved.
Since these problems would be introduced by the DataSourceV2ScanRelation work, I'll add this solution to that PR.
@cloud-fan, does that sound like a good solution?

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/16913/
Test PASSed.

Merged build finished. Test PASSed.

I like that @rdblue. Thank you both!
On Fri, Oct 11, 2019, 2:55 PM UCB AMPLab<EMAIL_ADDRESS>wrote:

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/16913/
Test PASSed.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/spark/pull/26091?email_source=notifications&email_token=ABIAE635IDIJ2CJQB76DOJTQODY5PA5CNFSM4I7ZZGFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBBJ3DY#issuecomment-541236623,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABIAE67ZJZ6Q4AICRCKKKKLQODY5PANCNFSM4I7ZZGFA
.

@rdblue yea that works too. I'm closing it and will wait for your early pushdown PR.


Extremely poor performance, especially closing tabs

I really love the functionality of this extension.  Unfortunately, the performance is just so poor that it makes it basically difficult to use for daily work.  The biggest issue is closing tabs.
With 346 tabs open, after closing a tab, it takes 21 seconds before the Tab Manager UI becomes responsive again.  First, the removed tab changes to a blank space in the menu and the UI freezes for 10 seconds, then that gap disappears and it freezes for an additional 11 seconds.  This is on a 2.6 GHz i7-9750H.  It is obviously doing a lot of calculation to re-render of all of my tabs in the list, but it shouldn't take this long.  My guess is that Material-UI is just too heavy for this type of extension.
Edit: I just tried this opening the extension in a new tab instead of the drop-down window, and did not experience the issue.  Rerendering the list took less than a second.
You might argue that it's crazy to have that many tabs open, but honestly, if I wasn't a tab hoarder, I would have no use for your amazing extension!
The second issue is the open time.  When I first click the icon, a giant white box appears on screen for approximately one second.  This is quite jarring, particularly since I'm using a dark theme.  Then the UI appears and the individual tabs load progressively over ~10 seconds.  I could understand if this happened the very first time it was opened after starting the browser, but after that this should be cached so that the display is instant.
One idea for making it seem a bit faster would be to first load the current tab and 6 tabs before/after it to fill up the display, then progressively backfill the tabs before and after that initial screen.  That would cover 90% of my tab management tasks.  I'm rarely hunting through the entire list, and if I am the search functionality works great.
The third performance issue is related to the animations.  The tooltips have some type of zoom animation applied that just makes the UI feel extremely sluggish.  Even worse, they often do a double-zoom, I'm not sure if that is intentional or the animation is running twice for some reason.  I realize this is just a matter of opinion, but personally I would prioritize performance over looking pretty.  I would either eliminate the animation or replace it with a much faster fade-in, and slightly decrease the time delay before the tooltips appear.
Desktop:

OS: Ubuntu
Browser: Firefox
Version 74

Hi @hackel
Thanks for your feedback.

The biggest issue is closing tabs.
The second issue is the open time.

For these two issues, they are eventually the same issue of rendering the tabs.
I usually have 80-150 tabs open on Chrome with MacBook Pro, haven't faced the same issue. Sometimes the UI performance is bad because of the laptop itself is busy to compile code.
I will try this on Firefox with 300 tabs to test the performance.
I spent a ton of effort to optimize the performance, that's why there is progressively rendering and other options like Show URL, Show Tab Tooltip, Highlight Duplicated Tabs. You can try to disable those option to see whether it improve the performance.

Edit: I just tried this opening the extension in a new tab instead of the drop-down window, and did not experience the issue. Rerendering the list took less than a second.

You can use Ctrl-T (macOS) or Alt+T (linux, linux, chromeOS) to open the extension in a standalone mode to see whether it works.

The third performance issue is related to the animations.

I assume you are talking about the tab tooltip, not the tooltip for buttons.
I didn't find this happen on the Chrome browser since Firefox is not my primary browser. I'll work on this to figure out what's the root cause.
In the meantime, if you don't need the tooltip, you can disable the feature as a workaround.

The biggest issue is closing tabs.
This issue happens on Firefox but not Chrome because of the block GC.

https://github.com/xcv58/Tab-Manager-v2/blob/master/src/js/stores/HoverStore.tsx#L35

@hackel just keep you update, I find the root cause of the problem. It's mainly caused by Material UI. And I would gradually replace the Material UI component for the individual tab by #305
On my development, it shows significant performance gain.

@hackel I release v1.5.0 contains several performance improvements. It works smoothly for me on Firefox with 500+ tabs.

Could you please give it a try?

@xcv58 That's an incredible improvement!  With ~400 tabs open, closing a tab happens in less than 1 second.  Opening now takes ~5 seconds, so that's an improvement as well.  Thanks for solving this!
I've noticed some inconsistency when opening the dropdown.  Most of the time, it still opens to a completely blank white square while loading, but occasionally the main (dark-themed) UI will open with a spinner instead of tabs.  Just now it also opened with a white background with spinner, and then the background changed to dark with the spinner still showing until the full UI rendered.  There must be some obscure race condition happening.

Thanks for your feedback. I'm glad it helps.

I've noticed some inconsistency when opening the dropdown. Most of the time, it still opens to a completely blank white square while loading, but occasionally the main (dark-themed) UI will open with a spinner instead of tabs. Just now it also opened with a white background with spinner, and then the background changed to dark with the spinner still showing until the full UI rendered. There must be some obscure race condition happening.

For this one, I believe it's caused by the JS query for dark theme. Since I decided to gradually deprecate the Material UI in this repo.
After I finish the deprecation, we can use the css ONLY approach for different themes. It wouldn't be a problem anymore. But the process may be slow and you can see small improvements over time.

Close since I didn't experience similar performance anymore.


add list package back to immut/list

This package (immut/list) is essentially the same as the previously removed package moonbitlang/core/list. The only difference is that the List type is now defined within the package itself; it's no longer a special built-in type.
The built-in List type in the compiler is scheduled to be removed in the release next week.

Pull Request Test Coverage Report for Build 1634
Details

181 of 194 (93.3%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.3%) to 88.734%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

immut/list/list.mbt
181
194
93.3%

Totals

Change from base Build 1630:
0.3%

Covered Lines:
2615

Relevant Lines:
2947

💛  - Coveralls


Image display

Thought it would be nice adding this feature.
Use a free API as https://www.pexels.com/api/

Not a bad idea at all, I will think how to implement it to at streamlit app with the suggestions generated.

Option added.


bam_vs_bed with unsorted bed files

bam_vs_bed assumes that bed files are sorted. If they arn't then an error is caused when intersectBed is run. However, this error isn't caught, and bam_vs_bed returns without errror. This means that when run in a pipeline, the task can fail without anyone being the wiser.

Thanks, I have made --sort-bed the default and added an --assume-sorted option to restore the previous default behaviour.
Best wishes,
Andreas


Swap from saving balance to spending balance

Once the swap has been completed the funds are not shown in the spending balance after the confirmations

Hey, sorry about that. Can you please navigate to Settings > Advanced > Lightning Connections and tap on the first item, it will show the status of your channel and at the bottom you can find a 'Support' button. If the status is either 'Awaiting Payment' or 'Order Expired' please use this button to send us an email and you will be issued a refund.

Hey,
Thanks for the reply and the app is great I thought to better bring this to your attention.
In regards to the mentioned: Yes, appears a awaiting payment but the support button does not work when I press it

Thanks yeah looks like we need to fix the button as well here. Do you have an email app installed on your phone? Because this button will look for open your email app and add a message for you.
In any case if it doesn't work please just send this screenshot to<EMAIL_ADDRESS>and that will have everything the team needs.

Great! I have sent the email already thank you.
Just in case yes I do have an apps emails in my mobile phone but the support button didn't do anything tho
Thanks once again for having a look.


Fix bug #13792. Allow use of custom className under PaperProps.

This is a different approach than PR https://github.com/mui-org/material-ui/pull/13797.
This allows for actually setting the className, without overriding the default classNames on the Paper component.
Allowing to add className is a legit basic behavior for overriding css classes.

[x] I have followed (at least) the PR section of the contributing guide.

I agree with this. While this does indeed add multiple ways to do the same thing I would expect PaperProps.className to just work. It's IMO not very handy to have to know in that case that classes also has a paper class and that this gets appended too. It requires knowledge of our styling solution and the used css classes  whereas this approach only requires knowledge of PaperProps.

It's fair enough 👍. I have updated the proposed change to minimize the implication of supporting this use case (we will have to do it with many more components).

@eladhayun Your GitHub email configuration doesn't match your commit, you might want to update it to get proper attribution.

@eladhayun It's a great first pull request on Material-UI 👌🏻. Thank you for working on it!


small change for upcoming CommonDataModel 0.3

This PR avoids some ambiguity issues for the upcoming CommonDataModel 0.3.
(In the new version of CommonDataModel 0.3, one can access variables also by standard name, e.g.
ds[CF"longitude"] will be a variable whose standard name is "longitude". CF"longitude" will have the type CommonDataModel.CFStdName and we need to dispatch on that)

Thanks :-)


active markers on hover and click

add active icon with blue border
change event marker to active when hovering over an event block,
click event title or location zooms to marker and changes marker to active
clicking marker scrolls to event description on left

for https://github.com/DevProgress/hrc-events/issues/39

@kielni This is cool, with a few fixes needed:

[ ] Add pointer-events: none; to the children of .list-event (except the rsvp button), because they're blocking the click right now
[ ] Dismiss the popup on click of any other .list-event
[ ] Surface the active map marker (might require some z-index fiddling, or assignment to its own pane)

I can't reproduce the RSVP button clicks being blocked.  It shows the active icon on hover over the RSVP button, and clicking opens the URL (should it be in a new tab?)  I tried on all of my Mac browsers (Chrome, Firefox, and Safari).  Do you have any more details that might help?  Maybe clicking too close to the left edge of the RSVP button goes to the event title instead?  I could try adding some more left margin to the button.

@kielni I totally described it wrong. I mean that the text of each card was blocking the click to center the map - the RSVP being the only part of each card that shouldn't result in a map centering. I'll take a shot at it after the merge. Thanks for the great update!


One pontential issue in win_primal.m

https://github.com/pettni/abstr-refinement/blob/685cf194a2b7d794fe09ba577fb87d04cd7ef8d8/abstr-ref/%40TransSyst/win_primal.m#L72-L73
Hi Peter (@pettni) ,
I am confused about the two lines above. Why don't we need to save the controllers returned by Pre and PGPre here? There is usually a gap between Z and V. I thought that the controller for Pre(V) might be contained by the controller of win_intermediate in the next few lines. But today I encounter some issues when executing the controller returned by the win_primal, which I think might be relevant to my question here. Could you please help me find the answer?
Thank you,
Zexiang

I think you are right; these controllers should be saved and included in the overall controller. Could you provide a small example where the controller fails?

I synthesized a controller for the type of spec [ ] A && <>[ ] B. Let's say the initial state is the state 62. (62doesn't satisfy B.) cont(62)then returns a list of actions18,19,20,21. But 62--18-->62is one transition in the abstraction, which means that the system could be stuck at state62if we keep using the action18`, which is suggested by the synthesized controller.

What would be most useful is if you could extract a minimal example that exhibits this problem that we can include as a test, similar to what is here: https://github.com/pettni/abstr-refinement/blob/master/tests/test_controller.m
In general the controller code has very few tests and there are probably more issues like the one you encountered, so more tests are always welcome.

Thanks for the testcase. I added it as a test and pushed a fix. Let me know if it solves your original problem.

The updated code solved my original problem.
Thanks for the fix!


[KBFS-1645] Block Retrieval Worker

@akalin-keybase, @strib: ready for another look sans reflection in block assignment (necessary reflection in the megacontext though)

@strib: final PR feedback delta: https://github.com/keybase/kbfs/pull/478/files/cb39ba11ffe97f8c68419ee5c4299ca9b5f86988..20c11cd3dfeb2ebcd99a764dac46dd9d5996e8b7


Stackdriver integration removal step 2

Fixes #2672
/cc @piosz
Should be merged after https://github.com/kubernetes/contrib/pull/2673

This change is

please rebase

/lgtm
/approve
FYI @Q-Lee @igorpeshansky


Digital Font enhancement is missing

We need font type option for displaying digits in Digital fonts
Ref: https://github.com/glomadrian/velocimeter-view

this option is already to the next version 1.1.3 4765a03

Also we need an option to set Unit  app:units="mph" or kph etc

you can use app:unit="mph" attribute.
please, look at style page to see all options.

download version 1.1.3 and set your font in assets folder, then use speedTextTypeface attribute like so:
app:speedTextTypeface="fonts/font_name.otf"
or in your code:
speedometer.setSpeedTextTypeface(Typeface.createFromAsset(getAssets(), "fonts/font_name.otf"));

fonts/font_name.otf: is the path to your font in assets folder.
also textTypeface attribute for other text in speedometer.


Create Rakudo Star Template

Are you sure about using rakudo star?

It seemed like a straightforward starting point, and I'll likely create a separate PR for another non-Star template once I've finished work for today.

It seemed like a straightforward starting point, and I'll likely create a separate PR for another non-Star template once I've finished work for today.

It might be better to use others that are "barer". IIRC, rakudo-star included a series of modules besides Rakudo and zef.


Admin: Show the number of previous grants submitted for a user

The admin could see how many grants the user has submitted. On hovering, Ideally will show some stats: 1 approved, 1 rejected, 1 completed, etc.


Horizontal event alignment

Do you want to request a feature or report a bug?
bug
What's the current behavior?
See screenshot: appointment indented further after the middle, 15 min appointment.

Expected behavior:
The two events in the middle should be aligned horizontally to start at the same x position.
Suggested fix:
DayEventLayout.js
function onSameRow(a,b) {
  return (
    // Occupies the same start slot.
    (Math.abs(b.start - a.start) < 15) ||
    // A's start slot overlaps with b's end slot.
    (b.start > a.start && b.start < a.end)
  )
}

At the moment a,b events in the second part of the condition are swapped, and the first part, the difference is set to 30 mins. Should I create a PR?

Fixed in #910


Friend Profile Integration

Overview
Part 1 of 2 of the Image Integration into Feisty. This Request deals with integration logic of the profile images for friends.
This is done by adding a new repository that handles calling and decoding of profile images. The View Model and Models have also been updated to handle this chain of events.
Notes
Removed certain retain cycles to do with the MVVM architecture.
To fix this each point that had an observer had that observer set to be a weak reference. The observer system was also changed from a dictionary of observers to only a single observer as it is unlikely that a model will have multiple view-models observing it, for example.

Try to keep the PRs a bit smaller in future, will be a very necessary skill to have at clients


Define custom media (CSS/JS) per ModelView

Django offers a neat way to define JS/CSS dependencies that get loaded automatically when rendering the form: https://docs.djangoproject.com/en/1.10/topics/forms/media/
If I wanted to have something similar in flask-admin, I'd have to extend 4 templates:
      # Templates
    list_template = 'admin/model/list.html'
    """Default list view template"""

    edit_template = 'admin/model/edit.html'
    """Default edit template"""

    create_template = 'admin/model/create.html'
    """Default create template"""

    details_template = 'admin/model/details.html'
    """Default details view template"""

Or is there an easier way to achieve this? If not, is there general demand for supporting this?

There's no easy way of doing it right now. But pull requests are welcome!

It's easy to extend:
just drop this template to /templates/admin/master.html
{% extends 'admin/base.html' %}
{% block tail_js %}
  {{ super() }}
  {% if admin_view.extra_js %}
    {% for js in admin_view.extra_js %}
      <script src="{{js}}"></script>
    {% endfor %}
  {% endif %}
{% endblock %}

And add this to your ModelView:
class MyModelView(ModelView):
    extra_js = ['//cdn.ckeditor.com/4.6.0/standard/ckeditor.js']

Anyway, I agree this would be better to have in flask-admin base templates.

Fixed in #1400

@vgavro I know this is old but I'm getting an error when trying to load a local script from my static dir:
In ModelView:
extra_js = [url_for('static', filename='admin/admin.js')]
Attempted to generate a URL without the application context being pushed. This has to be executed when application context is available.

It's possible to add extra_js in API doc ?

@alxvallejo if you haven't found a solution yet, checkout this answer.


Recall clients that have previously connected

The client will save the Chat ID to local storage every time it receives a chat:new socket message.

[ ] Convert Client object away from using Socket ID as its unique ID
[ ] Convert Chat object away from using Socket ID as its unique ID

Resolved in #32


The GPU memory requirement for model training

Hello, sir. This is an excellent work. I have much interest in the use of Mamba in point cloud detection, but I only have a single GPU with 12GB memory, much smaller than the memory you state in this paper. Can the model be trained on a single 12GB GPU?

Sorry, we haven't specifically limited the GPU memory to 12 GB. However, you can try reducing the number of layers, halving the output resolution of BEV features, or increasing the size of input voxels.


Modal: Scroll bar is "invisible" when using inverted dimmer

Bug Report
Steps
Open a modal with the prop 'dimmer' set to 'inverted'. Make the screen small enough so that the modal is scrollable.
Expected Result
To be able to see the scrollbar on the dimmer. (e.g. a black/grey scrollbar)
Actual Result
The scrollbar is white on a white dimmer, so it's invisible.
Version
0.82.2
Testcase
https://codesandbox.io/s/k2r1oqq4p7

It seems like it's only a google-chrome issue; I've tested it on firefox and the scrollbars appeared fine.

@TheBurnerGuy all of the css and images for flags is coming from the semantic-ui repository. You'll need to open your issue there. This project only contains the React components that render SUI markup. That can technically be targeted with CSS, but we don't handle any of that here.


Error 0xc00ce558 - GPO permanently breaks

Hi,
This tool looks promising but during testing I encountered an error that simply breaks the targeted GPO rendering it unusable for me as a pentester and for my client. The broken GPO cannot even be deleted. This is as you understand really bad and as long as this is not fixed I cannot use your tool. SharpGPOAbuse suffers from the same issue so to me it seems something has changed on the Windows Server side making both tools incompatible.
I whish I could write the specific scenario that triggers this but I have yet to find a pattern. It seems to happen more frequently when things go "wrong". For example when I forget to add the parameter "-f" or when I enter a GPO that does not exist or I do not have write access on. Just keep trying to execute various sheduled tasks and you will eventually get an error in your shell. Once you get that , access your DC and open  Group Policy Management. In that, rightclick your targeted GPO and select "Edit...". Then expand "Preferences" under either "Computer Configuration" or "User configuration" depending on what object you are targeting. Finally leftclick "Control Panel Settings" and you will get the below error popup.

I have seen this as well, I would say the tool is pretty dangerous without figuring this out, no one likes broken GPOs :)

I've tried playing around, I couldn't reproduce... If this happens again, I'd love to have the XML file associated with the GPO \\<DOMAIN>\SYSVOL\travers.ic\Policies\{<ID>}\Machine\Preferences\ScheduledTasks\ScheduledTasks.xml

I tried to reproduce this but I couldn't. I have rebuilt my AD lab which includes a primary and a secondary DC since reporting this so maybe there was some GPO sync issues triggering this?


limit regions/callback on region update/create

Regions count limit patch
Use option 'count: integer'
Regions callback
Use option regionsCallback
Call wavesurfer.regionsCallback
Example
//Save time to input and localStorage.
var saveRegion = function() {
    var regionHolder = document.getElementById('region-data'),
        regions = wavesurfer.regions.list,
        regionsObj = Object.keys(regions).map(function (id) {
            var region = regions[id];
            return {
                start: region.start,
                end: region.end
            };
        });

    for (var i = 0; i < regionsObj.length; i++) {
        var audioRegion = regionsObj[i],
            start = Math.round(audioRegion.start),
            end = Math.round(audioRegion.end);
        startInput.value = start;
        endInput.value = end;
    }

    localStorage.regions = JSON.stringify(regionsObj);
};

if (wavesurfer.enableDragSelection) {
    wavesurfer.enableDragSelection({
        color: 'rgba(0, 255, 0, 0.1)',
        count: 1
    });
    wavesurfer.regionsCallback = saveRegion;
}

No, sorry.

Counting can be easily done in the application code, so I don't see any reasons to include it into the plugin.
We don't provide callbacks in the API. We emit events.

Just a try without a chance to be approved.
Thanks for a great tool!

Thank you too! If you are solving a specific problem with the callback, let me know and I'm sure we can find a way to do the same using events.
Cheers!


chore: add response.Ok method

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 3 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 64.19%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

response.go
0
3
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.4%

Covered Lines:
337

Relevant Lines:
525

💛  - Coveralls


Configure how the locale name will be displayed

Feature proposal
Depending on the purpose of the field, it might make sense to display the locale names in the UI locale. Google Translate follows that approach:

In other scenarios, it makes sense to display the locale name in the locale itself (so that users can select their locale, which will be written in their own language). Facebook follows that approach:

Describe solution expectations
No response
Additional information
No response

In my opinion, Facebook's approach is more user-friendly than Google's. If a user accidentally selects an unfamiliar locale or language, it can be challenging to find the intended locale afterwards.

Let's go ahead with the Facebook approach, and also let's aim to improve the demo, it seems to be pretty empty right now. Some other ideas:

It should be possible to configure if I want to display, the locale in english, the locale in the locale it self, or both
It should be possible if I want to select the locale, or convert it to select only the country (like a country selector)
It would be great to be able to only allow the selection for certain locales / countries
Based on that we could make the demo more interesting, right now it seems pretty empty. (BTW make the combo box wider in the demo)


Rename llvm_stable feature and remove references from docs.

Fixes: https://github.com/servo/rust-bindgen/issues/409

Awesome! Taking a look now :)

Ping me after removing leading underscores and I'll r+
Thanks again!

Removed the leading underscores, now the build will tell.

Thanks!
@bors-servo r+

:pushpin: Commit 0db962b has been approved by fitzgen

:hourglass: Testing commit 0db962b with merge 0ae42f2...

:sunny: Test successful - status-travis
Approved by: fitzgen
Pushing 0ae42f2c55305aab12b2f55346001b7d5539cb6f to master...


Conflict with other library

I can't use with https://github.com/bravoborja/ReadMoreTextView library. Your library was removing click function from more text

I've commented one line from your code and it works.
override fun colorize() {         val spannable = view.text         check(spannable is Spannable, { "Attached text is not a Spannable, add TextView.BufferType.SPANNABLE when setting text to this TextView." })         spannable as Spannable //        spannable.removeSpans(CharacterStyle::class.java)         if (isHashtagEnabled) {             spannable.putSpansAll(SocialView.HASHTAG_PATTERN, {                 mHashtagListener?.newClickableSpan(spannable, mHashtagColor) ?: ForegroundColorSpan(mHashtagColor.defaultColor)             })         }         if (isMentionEnabled) {             spannable.putSpansAll(SocialView.MENTION_PATTERN, {                 mMentionListener?.newClickableSpan(spannable, mMentionColor) ?: ForegroundColorSpan(mMentionColor.defaultColor)             })         }         if (isHyperlinkEnabled) {             spannable.putSpansAll(SocialView.HYPERLINK_PATTERN, {                 mHyperlinkListener?.newClickableSpan(spannable, mHyperlinkColor, true) ?: object : ForegroundColorSpan(mMentionColor.defaultColor) {                     override fun updateDrawState(ds: TextPaint) {                         super.updateDrawState(ds)                         ds.isUnderlineText = true                     }                 }             })         }     }
Maybe it's wrong. Why you removing clickable spans?

Pardon my slow learning, I fail to understand what you're trying to achieve here. That removeSpans is necessary to clear up old spans before adding new ones.


Call socket.send instead of in the onopen handler.

The websocket is already open by the time the onopen handler was getting
defined.  Instead, just call socket.send() directly.  That causes the alert
message to show up immediately, as intended.

Are you sure? It can take websockets a while to handshake and open; there's no guarantee it's opened by that point. Do you have an API spec source?

Maybe it's just because I was typing it in manually and not copy/pasting it into my javascript console.  When I did that, using the onopen method generated no alert.

Right, as when you're typing there's enough delay between the two lines that it can negotiate; as a script it runs very fast, and doing what you suggest won't work.
Maybe change the PR to add a comment saying that if you're typing it in, not pasting, that you should do it a different way? Or can we find a way that works both with typing and straight execution?

You could do something like this:
    var func = function() {
        if (socket.readyState == 1) {
            socket.send('hello world');
        } else {
            setTimeout(func, 250);
        }
    };
    setTimeout(func, 250);

but I'm afraid that's detracting from the simplicity of the example.

Actually, the last line could just be  func().

That does seem slightly complicated; I think it might be nicer to just have a short callout explaining what onopen means and how you might have to run the send by itself.

Agreed.

I made a documentation change regarding this, but it seems to have evaporated.
When you're following the documentation, testing the initial connection in the JS console, the onopen() never gets called so nothing happens.
Manually calling socket.send('msg') makes the echo happen.
Obviously, in a script, this is going to be different, but in the 'getting started' docs, this should definitely be changed as the naive user will be left scratching their head not knowing to call socket.send to start the process once stalled.

Closing this as I'm going to write a better solution that checks socket state.


Regions no longer work since Update 11

Hello,
Unfortunately, I have had the following problem since the first update to version 11. As soon as I add new code, comments or anything else, the regions stop working. You can still fold these but not where the arrow is. The screen area also looks as if everything is marked.
If I remove the adjustment again, or save everything and restart Visual Studio, everything works again. Until I add or delete some lines again.
So I have to work on the latest version 10 all the time.
I've already tried this in a completely new file with 5 lines of code. Always the same.
Version 10:
https://github.com/microsoft/AL/assets/53939875/d54b3358-3fec-4a0a-a559-f1b79768f15b
Version 12 (As in version 11):
https://github.com/microsoft/AL/assets/53939875/e50efb75-19dd-48ca-a5df-e8048c7785a1
I was able to replicate the problem with other colleagues. However, they don't work with regions and so they hadn't noticed this before.

Can you share an example project where you see this issue so that it can be easily repro-ed?

I tested a little further.
The problem doesn't seem to occur in smaller files. I wanted to create an example for you with just a few lines. The problem didn't exist there. I looked into smaller "old" files and the problem wasn't present there either.
However, the problem arises in every larger file with 1000 or more lines, and correspondingly several regions. Unfortunately I cannot send you any of these projects. The German GDPR prohibits me from doing this.
I'm trying to create a sample project large enough that this problem occurs.

The German GDPR prohibits me from doing this.

The GDPR just protects personal data of natural persons, so you can just remove or anonymize the personal information from the source code. (I would even consider this a bad practice to store personal information inside source code.)
Or maybe you just meant that your company doesn't allow code sharing.

I think I found the problem. A region within a region is probably not possible. Because that creates the problem.
The code below causes the problem as soon as you add more lines. If you remove region 2.2 the problem doesn't occur.
`codeunit 99555 Test
{
#region Test 1
procedure Test1()
var
begin
end;
#endregion Test 1

#region Test2
procedure Test2()
var
begin
    #region Test2.2

    #endregion Test2.2
end;
#endregion Test2

#region Test 3
procedure Test3()
var
begin

end;
#endregion Test 3

#region Test4
procedure Test4()
var
begin

end;
#endregion Test4

#region Test5
procedure Test5()
var
begin
    //Test
end;
#endregion Test5

#region Test6
procedure Test6()
var
begin

end;
#endregion Test6

}`

Thanks for the repro steps!


New Code Example: Fine-tuning Wav2Vec 2.0 using CTC for ASR using Hugging Face Transformers

This code example shows how to fine-tune Wav2Vec 2.0 using CTC for ASR using Hugging Face Transformers. Additionally, it also shows best practices for Hugging Face Datasets.
This is part of my GSoC 2022 contribution. More information about my project can be found on this link and this link
Thank You!

Hi @Sreyan88, thanks for the PR. Are you planning to make changes in response to the feedback? Let us know if you're still working on this. Otherwise we'll close the request. Thanks!

Hi @pcoet , My latest PR resolves all issues addressed by @fchollet . This can be merged. Thank You!


Use Heap properties to optimize .contains

I noticed you use pretty much .indexOf to get if an element is available within a heap.
Why don't you leverage the property of the heap's array representation to move from a O(N) to a O(lgN) complexity?

 Binary heap Is not searchable in  logaritmic time


Way to turn off storage by default

I was using little-state-machine with react-hook-form and had some conditional state that I didn't want lsm to remember on load.
The only way I found to implement this was something like this:
const initialState = {/* whatever you want */}
createStore(
  { ...initialState },
  {
    middleWares: [],
    syncStores: {
      // you can sync with external store and transform the data
      externalStoreName: 'initial',
      transform: ({ externalStoreData, currentStoreData }) => {
        return {
          ...initialState,
         // here add any other state from external or current
        };
      }
    }
  }
);

I actually really like this package a lot and would be happy submitting a PR for what I'm thinking, unless I'm not on the right track.

hey @devoNOTbevo thanks for the support :) keep it simple. Maybe we should create a folder for examples and doing it just like RHF (code + CSB)? your thoughts?

@bluebill1049  yes, I think that could be helpful.  I liked that about RHF actually - great job on that site.  One question - is it possible to prevent LSM from storing at all?  This is developer's responsibility, but I am concerned about instances where user has not given consent to storage, when it's necessary.
In my example, LSM is still storing the data, I'm just not using all of it.

oh you probably can use setStorageType and give it an object with session storage alike API.
call setStorageType({ setItme: () => {}, ...}) before createStore

@bluebill1049  I will give that a try on a current project and report back.  These might can be examples to give also, like you suggested.

if i have some extra time, i will do a codesandbox for you @devoNOTbevo :)

I was able to by setting both getItem and setItem:
setStorageType({
    setItem: () => {},
    getItem: () => {
        return createInitState();
    },
    ...localStorage
});
createStore({ ...createInitState() });


feat(sensors_plus): add magnetometer support

Description

Adds necessary Android, iOS, and web API for magnetometer sensor.

Adds new MagnetometerEvent class

Secondary commit suggests small typo fix in constructors of other FooEvent classes

Adds new Stream<MagnetometerEvent> called magnetometerEvents.

Related Issues
#370 [sensors_plus]: Addition of Magnetometer
Checklist

[X] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[X] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See [Contributor Guide]).
[X] All existing and new tests are passing.
[X] I updated/added relevant documentation (doc comments with ///).
[X] The analyzer (flutter analyze) does not report any problems on my PR.
[X] I read and followed the [Flutter Style Guide].
[X] I am willing to follow-up on review comments in a timely manner.
[X] Testing has been performed to verify Android support.
[ ] Testing has been performed to verify iOS support.
[ ] Testing has been performed to verify web support.

Any browser I tried testing triggered

. . .
else if (error.toString().contains('ReferenceError')) {
  print('$apiName is not supported by the User Agent.');
}

where $apiName is the newly added Magnetometer().

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change.
[X] No, this is not a breaking change.

Thanks for the PR, it looks good and is a nice feature to be added to the package.

Glad to hear that!
I see there were some snags in the tests. Ths Swift formatting... I'm not sure. The portions I added were duplicated from the other identical bits. I do not have a formatter for Swift.
The "missing" tests, as far as I can tell, are in place.
Would love to get whatever is needed done to push this forward! 👍

Please let podspec be as is.
btw, it seems you can get the formatting, did you run melos run format? do you have clang-format installed on your machine?

Please let podspec be as is.

Done.

btw, it seems you can get the formatting, did you run melos run format? do you have clang-format installed on your machine?

I have troubles with melos. I got it to bootstrap thankfully, but any other commands like clean or run format give me a
"No active package melos" error. I thought it might be clang, so it assuring to hear you suggest that. I'll keep looking.

Just a friendly reminder that this PR is ready for review.
I would love to have this functionality added to sensors_plus!

Great, I'll review it this weekend and will release new version.

Interestingly enough after I ran melos run format and pushed, I saw a change waiting for a new push. Likely something with my auto-format when saving in VS Code.
I will try that again.

Interesting, I saw you fixed that previously, maybe you should try once more.

Easy fix 🤞 ready to go.

codecov is showing a change in a file for connectivity_plus that I did not make intentionally but may be a result of some automatic formatting.
Actually I manually checked the alleged change in my repository and found the "missing" line still present. 🤔

What is my next step?

@Zabadam it's great, I am going to merge this actually, I thought I wrote to you but it seems I forgot, Can you just once more merge with main and make sure everything is up to date and also double-check the version number. we had a few releases for these packages.
that's it, I will merge it right away afterward.


cause status code 418 was added in 1.1.1

Found within my testsuite: https://travis-ci.org/chubbyphp/petstore/jobs/567561612
Fatal error: Uncaught Error: Undefined class constant 'Fig\Http\Message\StatusCodeInterface::STATUS_IM_A_TEAPOT'
=> 1.1.1
Fatal error: Uncaught Error: Undefined class constant 'Fig\Http\Message\StatusCodeInterface::STATUS_MISDIRECTED_REQUEST'
=> 1.1.2

Coverage remained the same at 99.731% when pulling e4b05423f13b7aa79eda683b6e757f800d6144bf on dominikzogg:master into 7936bf90368a6aed9b26400d09d63a5aac47efb4 on slimphp:master.

@l0gicgate could you release that as 0.4.1?

@dominikzogg we need to merge #113 and #120 first.

get that, all vendor related


[Multi-Actions] Figure out which model to use and how to use it for the scheduler

See discussion here: https://www.notion.so/dust-tt/Multi-Actions-Technical-Proposal-662aa8afbc584f4285d89becc0a6417b?pvs=4#c774c4d171454e49a9936bbaa8be0815
OpenAI does not support picking functions but prevent generation.
Alternatives:

Prompting the model to not generate ever (to be tested)
Having 2 step. One tool picker and input generation (slow)
Using another model

As part of this task we likely want to add function calling to Anthropic and Cohere to evaluate them as potential scheduler models.
Principle here is that we'll test all of that a lot and we should design with the ability to switch from one approach to another in mind.

@spolu WDYT if closing this ? I think we are converging towars using whatever the user picked (and only letting them pick models that support function calling, which atm means every assistant builder model)

Agreed!


Expose a CLI option that can expose env variables

🚀 Feature Proposal and Motivation
With webpack CLI we are running jest through a test suite that does it's own regex to grep different test cases. It is hard to singly run one case, as it does a glob against a folder. We have set up a global env variable that we want to allow the user to customize through the CLI in Jest instead of having to revise the globals option in Jest at package.json.
This allows users to test their fixes/features more fine-grained instead of having to run the entire suite (which takes about 80s). By exposing this in the CLI, it is more easy to test quick changes on async code which isn't singly using a Jest regex option ( handles it on its own ).
Example
jest --globals GREP_THIS=/ok
instead of
GREP_THIS=/ok jest ..
 "globals": {
      "GREP_THIS": "/ok"
    }

Pitch
Why does this feature belong in the Jest core platform?
It makes life easier for developers! I can work on this if this feature is wanted.

With webpack CLI we are running jest through a test suite that does it's own regex to grep different test cases.

Could it be done without its own regex? Setting up a custom filter feels like it's addressing a symptom rather than the real issue.

Yes we are doing it now. I think that this can be used to seperate env.test or similar to run different environments.
It allows people to spread jest commands into multiple scripts like one for testing in test and one for prod if I wrote this concise enough to be clear.

My point wasn't to shut down your suggestion, it was to try to figure out how we can solve your need in a way that allows you to tap into Jest's capabilities for filtering without needing to implement your own filter. Filtering in Jest is really powerful, both on a file basis and on a test basic. So if we can figure out how you can utilise that, that'd be awesome.

I'd love to work with you on figuring out how to best run the tests. I've been thinking about looking at Webpack and see if we can simplify it; and if the CLI's test suite is like a miniature version of Webpack's test suite, it'd be a great starting point.
If you wanna explore that, it'd be awesome if you could summarise what limitations there are in Jest that forces you to have a single file with tests and importing test cases rather than having multiple test files each importing their own test case. Speed? API? Ease of use? Closer to real-world code?

@SimenB thanks for an elaborative feedback. The fix was to prefix the env before invoking jest as I might have forgot to mention earlier.
On the CLI test side of things, I'd really love to work that out. To emphasize on our test suite: they're horrible.
I'd love to find out a way to have more efficient tests, and the questions you lined up at the end of the previous comment is rather not the case. We ported the old folder from webpack and linked them up in the repository when we were migrating the code. I don't think anyone has the knowledge to fix the tests without breaking anything and I'd love to work that out.
If you have suggestions of how that should be done, I'd be very happy to help out, implement it or to help you do that. That would be great! We have two open issues, one on smoke tests and one on better test infrastructure that I'd appreciate if you had the time to look at:
https://github.com/webpack/webpack-cli/issues/588
Seems like we don't have an issue up for the refactoring of the tests, but I have it on my agenda ( https://github.com/webpack/webpack-cli/pull/527 ). This should be PR'd against the master branch though.
As mentioned, would really mean a lot if you had the time to look at it 💙

@ev1stensberg opened up https://github.com/webpack/webpack-cli/pull/621 with a suggested start

Thank you so much, Simen. ♥

I'll close this then. I think that sort of approach is the way to go.
It would be cool to have a guide in the docs for how to test CLIs with Jest, though. Can circle back to it once we've tried it out.
Thanks for the kick in the butt! I've been meaning to take a look at Webpack's tests since May. A small version of Webpack's test base to start on is perfect.
(Not saying that allowing you to pass env variables through jest is impossible (although I'd probably just use cross-spawn), but I think the use case pitched in this PR is better served by refactoring the tests)


Middleware causing Eloquent problems

Stack trace:
#0 {main}```

If I comment out the middleware in Kernel.php it works fine. Any clue what might cause it?

There is a semicolon missing in the Eloquent class


Issuer validation

Hello,
I have some questions about the validation of the issuer.
From the documentation the issuer identifier is a case sensitive URL using the https scheme that contains scheme, host, and optionally, port number and path components and no query or fragment components.
So http://openid.net/specs/connect/1.0/issuer could be a valid issuer, but it isn't.
Also, when doing some tests locally, I use some local issuer on http://localhost:45543 for instance, but it is not a valid uri, so I had to comment locally the check.
https://github.com/PlusAuth/plusauth-oidc-client-js/blob/af8a49d4b36e916b03ab7eec1efcc6c5064ea157/src/utils/url.ts#L3-L6
https://github.com/PlusAuth/plusauth-oidc-client-js/blob/af8a49d4b36e916b03ab7eec1efcc6c5064ea157/src/client.ts#L83-L85
Best regards

Thanks for the detailed issue. Fix landed in v0.10.2


Chrome Defect: Aardvark Applications Type Selector

When using Chrome 40.0.x (Mac Yosemite), on the arango web interface, applications tab:
Clicking the settings icon opens up the checkboxes for displaying the various types of applications available to install.  However, clicking any of the options sends the user back to the main page.

fixed


[mac] Terminal command option is not working

Changing "Terminal command" option in settings.
But app still using default terminal application.

Will remove that option next release :)


Fix dl modal

I updated protractor because I couldn't get iit to work ... now the tests run a bit faster, too.

LGTM


Initial zero

Ok, i'm created a pull request for my calculator app, main branch.
Now I can close my own issue

Ok, i am merging branches and purge initial-zero branch as unnecessary


txn_types: Add assertion field to Mutation type

What is changed and how it works?
Ref pingcap/tidb#26833
What's Changed:
This PR adds the Assertion field to Mutation type, provides helper function to create instances without assertions, and update all usages of Mutation to use the helpers.
This PR is part of https://github.com/tikv/tikv/pull/10977 and split out since that PR is too large.
Related changes
-
Check List
Tests

Unit test

Side effects

Performance regression

Consumes more CPU
Consumes more MEM

Breaking backward compatibility

Release note
None

/merge

/merge

/merge

/merge

/test

/merge

/merge

/merge

/merge

?????


What's an issue?

What's an issue?
What can I use them for?
What can I not use them for?
When should I use them?
When should I not use them?
How do I format text in issues like bold and italics?
How do I write emojis on issues? :fire: :bug: :pizza: :ship:

It is like a forum that everyone can post a thread and discuss about related topics

A place to discuss problems or ask questions about the project.  Sometimes used to let developers about bugs that might appear.

It can also be used to raise problems in the code so that people can work on fixing them.

A note detailing group discussion about a project - the problems, issues, errors, things that need to be fixed. Issues can be open/private.

Great comments, team! As far as I can tell, GitHub doesn't allow for private issues, although repos can be public or private.

It can be used to discuss projects, ask and answer questions.

It's a good way to track concrete TODOs for a project:

One issue = one task to be completed
Can track who is working on it ("assignees" in the top right)
"Close" the issue when it's done

Other useful stuff:

Flag a problem with some code or results (e.g. "Hmm I think the graph in file X is wrong...")
Use "milestones" and "labels" to note which issues you want to get done by when (Is this urgent? A nice-to-have? Does it need to be done by July 30 for a presentation?)


API refinement

change the plan to set the default value to alpha properties in the fill/stroke apis.

For the sake of SVG, the opacity for stroke/fill is useful. Let's discard this plan.

@hermet I'm thinking about  #1472
Would this be similar to the alpha OK? But I don't know what to do in capi.
      * The join style is used for joining the two line segment while stroking the path.
      *
      * @param[in] join The join style value. The default value is @c StrokeJoin::Bevel.
+     * @param[in] miterlimit Optional Miterlimit of the stroke.
      *
      * @return Result::Success when succeed, Result::FailedAllocation otherwise.
      */
-    Result stroke(StrokeJoin join) noexcept;
+    Result stroke(StrokeJoin join, float miterlimit = 4.0f) noexcept;

+   /**
+     * @brief Gets the stroke miterlimit.
+     *
+     * @return The stroke miterlimit value when succeed, -1 if no stroke was set.
+     */
+    float strokeMiterlimit() const noexcept;

@capnm hello, that change will lead the abi break. So we should introduce with a separate api.
like:
Result strokeMiterlimit(float v);
float strokeMiterlimit();


Offline Support

Sure this must already be requested somewhere, but would be great if the computer is offline, then OmniSharp, NET Core Debugger, Razor Language Server (and anything else), use a cached version you already have on the computer.
I often am offline, and therefore can't really code unless I boot up the program when online before hand. There are steps to work around this but I worry they are quite techy, and may not sort all the different packages that update / download.
Really love the product, this would make it more accessible in my opinion, but know you have a lot on :) Thanks for reading (you guys rock).

It sounds like you should open this issue with the C# extension as they would need to support what you are asking for, as those services are loaded by that extension.


Lotus Release Handoff: Carbonado.1  (r10.0)

Request
Please deploy the Wallaby testnet with the following Lotus release.
Git reference
Branch: https://github.com/filecoin-project/lotus/tree/ntwk/wallaby
Commit: a971868be49e549ee4435a9865402f8a7c35a0f3
Scope
This release contains the release of the Carbonado.1 milestone.
Headline items include:

Implementation of Ethereum Account Abstraction: Users can now send to a new Ethereum addresses (creating an embryo), and then send from the address (transforming the embryo into an Ethereum Account)
Refactored the gas working to be more granular (more here)
Many bugfixes

Images are updated! It's working for this notebook here: https://observablehq.com/@jimpick/fvm-actor-code-playground-erc20-sans-events?collection=@jimpick/filecoin-virtual-machine


[material-ui][Avatar] Simplify valid children assertion

A simplification on #40766 to save bundle size.

https://mui-dashboard.netlify.app/size-comparison?circleCIBuildNumber=643588&baseRef=master&baseCommit=4f793a13d5105373a3eb3765a60bfd17d1d8960c&prNumber=40834

sacrificing readability for such a marginal reduction in bundle size isn't justified.

@ZeeshanTamboli I don't see the balance this way. I don't think this is more complex, it seems simpler to me (truthy values + 0, maybe because I was already exposed to it in #34853.
The primary target is to have a great experience for end-users and developers, the experience maintainers have is secondary but should be aligned to support the former.


[SPARK-22653] executorAddress registered in CoarseGrainedSchedulerBac…

https://issues.apache.org/jira/browse/SPARK-22653
executorRef.address can be null, pass the executorAddress which accounts for it being null a few lines above the fix.
Manually tested this patch. You can reproduce the issue by running a simple spark-shell in yarn client mode with dynamic allocation and request some executors up front. Let those executors idle timeout. Get a heap dump. Without this fix, you will see that addressToExecutorId still contains the ids, with the fix addressToExecutorId is properly cleaned up.

Test build #84310 has started for PR 19850 at commit 2703214.

Test build #84310 has finished for PR 19850 at commit 2703214.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/84310/
Test PASSed.

good catch! merging to master!

thanks @cloud-fan


Failure merging a referenced entity

This test case seems to fail in ORM.
Am I doing something wrong?
Thanks
Note that the join column references a column that's not the identifier.

Superseded by https://github.com/hibernate/hibernate-orm/pull/6495


Cross-compiling redis with armv7l-unknown-linux-gnueabi fails

Issue description
sean@kodenine ~/d/nixbmc> cat default.nix
with import <nixpkgs> {
  crossSystem.config = "armv7l-unknown-linux-gnueabi";
};

callPackage <nixpkgs/nixos/lib/make-squashfs.nix> {
  storeContents = [ redis busybox ];
}

sean@kodenine ~/d/nixbmc> nix build
builder for '/nix/store/zhc824aq2hvgqa54l7mzirv9n1zqdlc5-redis-5.0.1-armv7l-unknown-linux-gnueabi.drv' failed with exit code 2; last 10 log lines:
  make[1]: [Makefile:184: persist-settings] Error 2 (ignored)
      CC adlist.o
  In file included from adlist.c:34:0:
  zmalloc.h:50:10: fatal error: jemalloc/jemalloc.h: No such file or directory
   #include <jemalloc/jemalloc.h>
            ^~~~~~~~~~~~~~~~~~~~~
  compilation terminated.
  make[1]: *** [Makefile:232: adlist.o] Error 1
  make[1]: Leaving directory '/build/redis-5.0.1/src'
  make: *** [Makefile:6: all] Error 2
cannot build derivation '/nix/store/prsxb48b7lk947cnrg95fldxpd97m84h-closure-info-armv7l-unknown-linux-gnueabi.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/simacjs7x1iw02dy3508386vv0wq2hgn-squashfs.img-armv7l-unknown-linux-gnueabi.drv': 1 dependencies couldn't be built
[0 built (1 failed)]
error: build of '/nix/store/simacjs7x1iw02dy3508386vv0wq2hgn-squashfs.img-armv7l-unknown-linux-gnueabi.drv' failed

Steps to reproduce
See above default.nix.
Technical details

system: "x86_64-linux"
host os: Linux 4.14.79, NixOS, 18.09.1381.c1427bf45ff (Jellyfish)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.1.3
channels(root): "nixos-18.09.1381.c1427bf45ff"
channels(sean): "nixpkgs-19.03pre159550.5ba0796bf2a, vertiv"
nixpkgs: /home/sean/.nix-defexpr/channels/nixpkgs

There are several problems, one hiredis it does not find the ar:
https://github.com/redis/hiredis/pull/632
Then redis compiles the Makefile without passing hostcc to the configure flag:
MAKE jemalloc
cd jemalloc && ./configure --with-version=5.1.0-0-g0 --with-lg-quantum=3 --with-jemalloc-prefix=je_ --enable-cc-silence CFLAGS="-std=gnu99 -Wall -pipe -g3 -O3 -funroll-loops " LDFLAGS=""
configure: WARNING: unrecognized options: --enable-cc-silence
checking for xsltproc... false
checking for gcc... aarch64-unknown-linux-gnu-gcc
checking whether the C compiler works... yes
checking for C compiler default output file name... a.out
checking for suffix of executables...
checking whether we are cross compiling... configure: error: in `/tmp/nix-build-redis-5.0.1-aarch64-unknown-linux-gnu.drv-0/redis-5.0.1/deps/jemalloc':
configure: error: cannot run C compiled programs.
If you meant to cross compile, use `--host'.
See `config.log' for more details
make: *** [Makefile:80: jemalloc] Error 1

The AR problem is fixed upstream now, but I have no time too look at the rest.

Should be fixed by #67694, which was written by me and just merged by @matthewbauer :smiley_cat:


TypeError: Class extends value undefined is not a constructor or null

Describe the bug
I am using houdini-plugin-svelte-global-stores , and when I test this example project, getting the following error
TypeError: Class extends value undefined is not a constructor or null
    at /$houdini/plugins/houdini-svelte/stores/listPokemon.js:4:39
    at async instantiateModule (file:///Users/schintha/Developer/Work/SPA/bbb/houdini-issues/node_modules/.pnpm/vite@4.0.4/node_modules/vite/dist/node/chunks/dep-5e7f419b.js:52224:9)

Severity
annoyance
Steps to Reproduce the Bug
https://github.com/xmlking/houdini-issues
pnpm i
pnpm dev
open http://localhost:5173/test1
Reproduction
https://github.com/xmlking/houdini-issues

sorry , now the repo change to public.

any thoughts on why I am getting
ReferenceError: Cannot access 'QueryStoreOffset' before initialization

I am testing simple API https://github.com/xmlking/svelte-starter-kit/tree/main/src/routes/(play)/play/fetching
page show data first and within sec display this error message...

This is with the @next version, right?

I was able to reproduce and will have a fix published very soon

This should be fixed in v1.0.0-next.7


error in running demo.py

F0113 17:47:26.589298  7077 cudnn_conv_layer.cpp:53] Check failed: status == CUDNN_STATUS_SUCCESS (4 vs. 0)  CUDNN_STATUS_INTERNAL_ERROR
*** Check failure stack trace: ***
Aborted (core dumped)
using GTX1060 6G

Same here, using GTX1060 6G

same with me，my GPU is：1080ti

How do you solve this problem at last?

the same problem, my GPU is: Tesla P100

uncomment engine:CAFFE in demo.py，the probloem is solved

uncomment engine:CAFFE in demo.py，the probloem is solved

can you tell us which line it is to uncomment?

uncomment engine:CAFFE in demo.py，the probloem is solved

can you tell us which line it is to uncomment?

uncommnet engine:CAFFE in deploy.prototxt at each layers.

may that be your cudnn level too high for ssd?


Expose "hints" to the planner

#37 adds "subscription hints", which are passed to the control plane via templates. This is useful for things like kafka.numPartitions which are specific to the control plane.
For more advanced use-cases, we need the planner (i.e. PipelineRel etc) to have access to "hints" as well. This would enable users to influence the output (Flink) SQL generated for a pipeline. For example, hints could be used to dynamically enable/disable specific planner rules.
Calcite has a concept of "contexts" which might be useful here. See: https://calcite.apache.org/javadocAggregate/org/apache/calcite/plan/Context.html

In the DDL, we should support Calcite hints (e.g. /*+OPTIONS(foo=bar)*/). This seems to require forking Calcite, or at least copy-pasting a substantial chunk of it, because Calcite's JDBC driver does not provide a mechanism for registering custom hints (or really any hints). Alternatively, we could wrap our planner in a Calcite Framework, which does support custom hints.
Flink SQL also supports Calcite hints, tho I don't think we need to leverage them here. Our hints should get passed through as connector configuration via WITH (foo = bar).


spoke vcn limit for hub_spoke_architecture

Hi,
In the current oci landing zone Terraform code, there is spoke vcn limit mentioned that we can have maximum nine spoke vcn connected to hub for hub_spoke_architecture.  While this conditionally be changed in variable section , is this hard limitation in OCI cloud ? Will changing this value e.g. to 30 cause any implications  or this was placed to avoid slow Terraform refresh etc.?
As per oracle doc "https://docs.oracle.com/en/solutions/hub-spoke-network-drg/index.html#GUID-FB5990CB-A23B-482D-80F4-AE4A548DEC9C"  maximum 300 vnc can be connected via DRG.
Whats the maximum recommended spoke vcn we can have ? and whats the hard limit ?
Thanks

Hello mhca99,
I think that limit in the TF code is a reminiscent when we had LPG connecting the VCNs. You can safely disable that, as we now use the DRG to connect. And you're spot on the soft limit, 300. Not aware of the hard limit. If you need to go beyond 300, please open an SR justifying. I don't think there's a recommendation on the maximum number, it all depends on what you're trying to achieve. Hope this helps.

Thanks Andre for the clarification.


Throw error when rules validations fail

Add a new pipe handler to fail the gulp task if the linter detects any error.
For example:
https://github.com/sasstools/gulp-sass-lint#sasslintfailonerror
https://github.com/adametry/gulp-eslint#eslintfailonerror

@jzsfkzm I can submit a pull request with an implementation if you think this feature is needed.

Awesome!
I want to use linter in git pre-commit hook, but now gulp always returns 0 exit code, even when there are lint errors.

#10


PullToReload example only works on first pull

Reproduced with Uno 1.1.3 (build 5510, installed with Fuse 1.1.1) on macOS 10.12, in dotnet export.
The PullToReload example in the docs (which is sourced from PullToReload.uno) is supposed to flash various colours while PullToReload is in its various states. That only works the first time you pull, the next time it only flashes green.
I'm not sure if there's a bug in the example or in PullToReload itself.
Gif:

I added some DebugActions to the example, otherwise the code is the same as in
PullToReload.uno.

Note that when I do this in preview, I can do "Preview->Refresh" to get it to work one more time. (Then I have to refresh again to make it work one more time etc).


[Loc] Field tooltips in CMakeLists.txt file not localized

More info:
1. This issue is reproduced on both release and pre-release versions, so it's not a regression issue
2. Repro Language: All loc languages
3. Repro platform: Windows, Linux, Mac
Repro steps:
1. Download and unzip CMakeProject1 (2).zip
2. Open it with VSCode and change the display language to CHS
3. Open above project with VS Code.
4. Open CMakeLists.txt file
5. Hover the mouse over the field
Expected result:
Tooltip messages for all fields are localized (Tooltip info localized in VS and screenshot as below)

Actual result:
Tooltip messages for all fields are not localized

Verified on CMake Tools v1.20.25(pre-release), this bug is still reproducing, so reopen it.

This is expected, as we currently still depend on a 3rd party extension. Closing. This will plan to be fixed as part of #3560


fix(events/api/views.py): Let schedule API can return i18n str for keynote data

Types of changes
Thanks for sending a pull request! Please fill in the following content to let us know better about this change.
Please put an x in the box that applies

[x] Bugfix
[ ] New feature
[ ] Refactoring
[ ] Breaking change (any change that would cause existing functionality to not work as expected)
[ ] Documentation Update
[ ] Other (please describe)

Description
Let schedule API to return the keynote speaker's name and keynote title with i18n.
Steps to Test This Pull Request
Steps to reproduce the behavior:

Go to http://localhost:8000/api/events/schedule/

Expected behavior
Will see the keynote speaker's name, and the title is a dict with two keys: zh_hant, en_us.
Related Issue
If applicable, refernce to the issue related to this pull request.
More Information
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here. You may also want to refer
to how to wirte the perfect pull request

Codecov Report

Merging #1043 (bb2b02e) into master (8c719b8) will decrease coverage by 0.05%.
The diff coverage is 60.00%.

@@            Coverage Diff             @@
##           master    #1043      +/-   ##
==========================================
- Coverage   69.09%   69.04%   -0.06%
==========================================
  Files          82       82
  Lines        3291     3295       +4
==========================================
+ Hits         2274     2275       +1
- Misses       1017     1020       +3

Impacted Files
Coverage Δ

src/events/api/views.py
44.04% <60.00%> (+0.33%)
:arrow_up:

src/attendee/api/views.py
42.85% <0.00%> (-5.15%)
: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 8c719b8...bb2b02e. Read the comment docs.


toast text alignment

I want the toast text to be left aligned instead of center, this is what I'm trying to do:
toast('My toast.', {
              duration: 2000,
              position: 'bottom-center',
              style: {
                background: '#ccc',
                width: '300px',
                textAlign: 'left',
              },
              icon: '😀',
            });

What this ends up creating is this:

What I instead want is:

Is there some way to make this work

You can find two possible solutions here: https://github.com/timolins/react-hot-toast/issues/88#issuecomment-863131815
If this is something more people want, we can think about exposing an API for that.


Multilanguage support

I'm building multilingual site where the language preference is part of the URL, e.g.
http://example.com/<somepage>      (Russian, default)
http://example.com/en/<somepage>   (English)
http://example.com/jp/<somepage>   (Japanese)
http://example.com/../             (etc)

Everything is ok, when I use prefix for all languages:
path: '/:lang/<somepage>'

But for default language, I don't need to include language in url, as shown in example. Any ideas how to handle this use case?

Seems like that would just be another route that is before the one's that specify the lang. Then your action can specify a default language for that route.
module.exports = {
    page: {
        method: 'GET',
        path: '/:page',
        handler: require('../components/Page.jsx'),
        // Executed on route match
        action: require('../actions/loadPage')
    },
    pageLang: {
        method: 'GET',
        path: '/:lang/:page',
        handler: require('../components/Page.jsx'),
        action: require('../actions/loadPage')
    }
};

Then in your loadPage action, you can specify a default lang if one is not part of the route.
Alternatively, you could just have one route that uses a regex to check for lang:
{
    page: {
        method: 'GET',
        path: '(/:lang)?/:page',
        handler: require('../components/Page.jsx'),
        action: require('../actions/loadPage')
    }
}

Thanks for reply! This seems work fine for me:
path: '/:lang([a-z]{2})?/<somepage>'

I'm trying also to get some help here:
https://github.com/gpbl/isomorphic500/issues/87
I have a sligthly different problem. I want to change the url name also!
http://example.com/<somepage url in russian>      (Russian, default)
http://example.com/en/<somepage url in english>   (English)
http://example.com/jp/<somepage url in japanese>   (Japanese)
http://example.com/../             (etc)

We can eventually create 4 different routes that point to the same handler, but I would like to use also google hreflang metatag to link the pages in different languages.


backend: Bulk interface for assign conferences

#1768

So finally we always pass recids to the task (even if all the records were selected)?

Yes.


npm package page SEO

I noticed that if you search lax.js npm the npm package page doesn't show up, instead another random package appears. This confused me for a while trying to install it through npm because I didn't look at the readme.
I suggest adding some keywords, specially: laxxx, lax, lax.js, laxjs, laxxxjs.
And also adding the npm package url somewhere in the readme (for example in the NPM Setup section).
Search result:

btw, i love this package. Thanks for creating it :)

Good idea, I'll add it to the next release :)

Done


"GHEORGHENI", "Covacipeter", "Lacu Roșu", "Vargatac" and "Visafolio" are listed in the drop-down menu

Description:
"GHEORGHENI", "Covacipeter", "Lacu Roșu", "Vargatac" and "Visafolio" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "GHEORGHENI" in the search bar.

Expected results
The location is listed in the drop-down menu as "Municipiul GHEORGHENI, HARGHITA" and "Oraș GHEORGHENI, HARGHITA (GHEORGHENI)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Type "Covacipeter" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Covacipeter, HARGHITA (GHEORGHENI)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Type "Vargatac" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Vargatac, HARGHITA (GHEORGHENI)".

Step 8
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 9
Type "Atid" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Atid, HARGHITA (Atid)".