Allow zero length variables

Allows zero-length cvxpy.Variables.
This is to increase consistency with other standard libraries that deal with matrix concepts (e.g. numpy, scipy and pandas).

Does this have any practical use, besides consistency?

My use case is to increase generality. Sometimes I have optimisation problems that I solve with a function:
def solve_with_cvxpy(number_of_nodes):
   ...

that when e.g. number_of_nodes=1 a zero-length variable is created somewhere in the process. Currently, I resort to writing two different methods, i.e.
def solve_with_cvxpy_for_one_node():
   ...

def solve_with_cvxpy_for_many_nodes(number_of_nodes):
   ...

and using if statements to switch between the two.
See also https://github.com/pytorch/pytorch/issues/5014 for a related discussion.

Interesting, thanks for sharing context.
@SteveDiamond , what do you think? Would zero-length variables break any assumptions in, e.g., cvxcore, or matrix stuffing?

Chiming in. I don't think this should be merged in without some tests showing how zero-dimensional variables behave "correctly" in a given model. Also, we need to keep in mind that allowing Variables of dimension zero means we're allowing Expressions of dimension zero, so it's not enough to only test against the Variable class.

Yes, I wouldn't be surprised if our codebase implicitly assumes nonzero
size in many places.
On Thu, Nov 12, 2020 at 10:30 AM rileyjmurray<EMAIL_ADDRESS>wrote:

Chiming in. I don't think this should be merged in without some tests
showing how zero-dimensional variables behave "correctly" in a given model.
Also, we need to keep in mind that allowing Variables of dimension zero
means we're allowing Expressions of dimension zero, so it's not enough to
only test against the Variable class.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cvxgrp/cvxpy/pull/1174#issuecomment-726258651, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPG4RDQSBXOWDGF3V3UUQTSPQSUPANCNFSM4TTDF2RA
.

@rileyjmurray I think that zero length expressions are already supported:
>>> cvxpy.Variable((2, 1))[:, []]
Expression(CONSTANT, UNKNOWN, (2, 0))

Zero length expressions are not officially supported. Getting support for them would be quite nice actually, and increase consistency with numpy, but I suspect we would need changes to cvxcore. @nrontsis can you try solving problems involving zero length variables and expressions? This is a change that will require a lot of testing and investigation.


Add Transaction Invariant Checks

Add support for implementing invariant checks that are performed on the ApplyContext for transactions. Invariants are expected to never fail, thus they result in a tec if a failure is detected. If claiming a fee causes an invariant failure, then we tef instead.
Since there is some additional work for each transaction when this enabled, I compared execution time for our full unit-test suite. The times below are total time or running --unittest ten times on this branch and current develop. The conclusion is that this adds about 1 second to total --unittest execution time. I think that indicates fairly minimal impact, but certainly I'd be happy to do more testing.
INVARIANTS:
real    12m10.897s
user    8m58.236s
sys     0m17.008s

DEVELOP :
real    11m59.416s
user    8m49.016s
sys     0m16.576s

This change enables the amendment for all jtx::Env by default. This is a departure from our standard so far of keeping the Env free of all amendments by default - so this particular change needs some specific discussion.

Codecov Report

Merging #2054 into develop will increase coverage by 0.08%.
The diff coverage is 94.17%.

@@             Coverage Diff             @@
##           develop    #2054      +/-   ##
===========================================
+ Coverage    67.71%   67.79%   +0.08%
===========================================
  Files          680      682       +2
  Lines        49219    49308      +89
===========================================
+ Hits         33327    33429     +102
+ Misses       15892    15879      -13

Impacted Files
Coverage Δ

src/ripple/core/ConfigSections.h
100% <ø> (ø)
:white_check_mark:

src/ripple/app/main/Amendments.cpp
100% <ø> (ø)
:white_check_mark:

src/ripple/app/tx/impl/Transactor.h
100% <ø> (ø)
:white_check_mark:

src/ripple/app/tx/impl/ApplyContext.h
100% <ø> (ø)
:white_check_mark:

src/ripple/core/Config.h
100% <ø> (ø)
:white_check_mark:

src/ripple/protocol/TER.h
100% <ø> (ø)
:white_check_mark:

src/ripple/protocol/impl/TER.cpp
93.75% <ø> (ø)
:white_check_mark:

src/ripple/app/tx/impl/ApplyContext.cpp
100% <100%> (+20%)
:white_check_mark:

src/ripple/app/tx/impl/InvariantCheck.cpp
100% <100%> (ø)

src/ripple/protocol/impl/Feature.cpp
100% <100%> (ø)
:white_check_mark:

... and 12 more

Continue to review full report at Codecov.

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

test

Wadda concept

I pushed a change to the way these checks are documented, but I'm happy to roll that back if folks don't like it.

current timing results for unittests (10 iterations)
`
INVARIANTS:
real    12m11.484s
user    9m6.316s
sys     0m16.468s
DEVELOP :
real    12m7.864s
user    8m59.568s
sys     0m16.320s
`

@ximinez yep - I'm looking into it. Some (questionable) code that I added works fine on linux and silently fails at runtime on windows. The questionable code is https://github.com/ripple/rippled/pull/2054/commits/f9ff3da5ee0a4d6d26f6a06ab9bc1835c9dbcf65#diff-a9a3e5216fb43a4420aca7e52f8a7167R32 which allows me to modify a private member under test. I might have to page @HowardHinnant on this one. It's dodgy anyhow, so might just remove it and leave that one check untested.

@mellery451 I had a bad feeling about that one.

That's some pretty crazy code.  If Windows doesn't like it, my vote is dump it.  Another possibility (besides forgoing the test), might be to declare the test struct a friend (requiring a forward declaration of the test).  That might be more trouble than it's worth.

Testing the functionality would be worth it to me, and I wouldn't object to friending the test struct.

I'm game - I agree the test coverage is preferable. I'll submit that change.

@ximinez came up with a solution that uses modifying ctor args only, submitting that.

Merged as 026a2491735f45d54b0bfeec6a06fe4c6420f981


Sort the repositories

hi, how is the possibles keys of "sortBy":"?". or not implemented yet other options?.

It's any of the keys from the main repo object returned by the Github API — things like name, created_at, or watchers.
These are the possible keys (and values) for this repo. Some would make more sense to sort by than others!
"id": 3706336,
"node_id": "MDEwOlJlcG9zaXRvcnkzNzA2MzM2",
"name": "github-widget",
"full_name": "jawj/github-widget",
"private": false,
"html_url": "https://github.com/jawj/github-widget",
"description": "Simple script to display own GitHub projects on a webpage, ordered by number of watchers",
"fork": false,
"url": "https://api.github.com/repos/jawj/github-widget",
"forks_url": "https://api.github.com/repos/jawj/github-widget/forks",
"keys_url": "https://api.github.com/repos/jawj/github-widget/keys{/key_id}",
"collaborators_url": "https://api.github.com/repos/jawj/github-widget/collaborators{/collaborator}",
"teams_url": "https://api.github.com/repos/jawj/github-widget/teams",
"hooks_url": "https://api.github.com/repos/jawj/github-widget/hooks",
"issue_events_url": "https://api.github.com/repos/jawj/github-widget/issues/events{/number}",
"events_url": "https://api.github.com/repos/jawj/github-widget/events",
"assignees_url": "https://api.github.com/repos/jawj/github-widget/assignees{/user}",
"branches_url": "https://api.github.com/repos/jawj/github-widget/branches{/branch}",
"tags_url": "https://api.github.com/repos/jawj/github-widget/tags",
"blobs_url": "https://api.github.com/repos/jawj/github-widget/git/blobs{/sha}",
"git_tags_url": "https://api.github.com/repos/jawj/github-widget/git/tags{/sha}",
"git_refs_url": "https://api.github.com/repos/jawj/github-widget/git/refs{/sha}",
"trees_url": "https://api.github.com/repos/jawj/github-widget/git/trees{/sha}",
"statuses_url": "https://api.github.com/repos/jawj/github-widget/statuses/{sha}",
"languages_url": "https://api.github.com/repos/jawj/github-widget/languages",
"stargazers_url": "https://api.github.com/repos/jawj/github-widget/stargazers",
"contributors_url": "https://api.github.com/repos/jawj/github-widget/contributors",
"subscribers_url": "https://api.github.com/repos/jawj/github-widget/subscribers",
"subscription_url": "https://api.github.com/repos/jawj/github-widget/subscription",
"commits_url": "https://api.github.com/repos/jawj/github-widget/commits{/sha}",
"git_commits_url": "https://api.github.com/repos/jawj/github-widget/git/commits{/sha}",
"comments_url": "https://api.github.com/repos/jawj/github-widget/comments{/number}",
"issue_comment_url": "https://api.github.com/repos/jawj/github-widget/issues/comments{/number}",
"contents_url": "https://api.github.com/repos/jawj/github-widget/contents/{+path}",
"compare_url": "https://api.github.com/repos/jawj/github-widget/compare/{base}...{head}",
"merges_url": "https://api.github.com/repos/jawj/github-widget/merges",
"archive_url": "https://api.github.com/repos/jawj/github-widget/{archive_format}{/ref}",
"downloads_url": "https://api.github.com/repos/jawj/github-widget/downloads",
"issues_url": "https://api.github.com/repos/jawj/github-widget/issues{/number}",
"pulls_url": "https://api.github.com/repos/jawj/github-widget/pulls{/number}",
"milestones_url": "https://api.github.com/repos/jawj/github-widget/milestones{/number}",
"notifications_url": "https://api.github.com/repos/jawj/github-widget/notifications{?since,all,participating}",
"labels_url": "https://api.github.com/repos/jawj/github-widget/labels{/name}",
"releases_url": "https://api.github.com/repos/jawj/github-widget/releases{/id}",
"deployments_url": "https://api.github.com/repos/jawj/github-widget/deployments",
"created_at": "2012-03-13T12:40:15Z",
"updated_at": "2018-05-30T23:26:28Z",
"pushed_at": "2018-04-25T12:54:13Z",
"git_url": "git://github.com/jawj/github-widget.git",
"ssh_url": "git@github.com:jawj/github-widget.git",
"clone_url": "https://github.com/jawj/github-widget.git",
"svn_url": "https://github.com/jawj/github-widget",
"homepage": "http://mackerron.com/webdev/",
"size": 37,
"stargazers_count": 28,
"watchers_count": 28,
"language": "CoffeeScript",
"has_issues": true,
"has_projects": true,
"has_downloads": true,
"has_wiki": true,
"has_pages": false,
"forks_count": 6,
"mirror_url": null,
"archived": false,
"open_issues_count": 3,
"license": null,
"forks": 6,
"open_issues": 3,
"watchers": 28,
"default_branch": "master"


fix(browser): relax vitest peer dependency

vitest v0.29.4 was just released, but trying to install @vitest/browser currently fails because its peer-dependency is set to vitest >= 0.30.0
Maybe this is on purpose, but this PR would avoid passing --force or --legacy-peer-deps when installing the browser package:
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/@vitest/browser
npm ERR!   dev<EMAIL_ADDRESS>from the root project
npm ERR!   peerOptional @vitest/browser@"*" from<EMAIL_ADDRESS>npm ERR!   node_modules/vitest
npm ERR!     dev<EMAIL_ADDRESS>from the root project

Missed that part, thank you 👀


windows上可以运行吗？

window上调试半天运行不了，报
def test(args):  # Load a trained model that you have fine-tuned (we assume evaluate on cpu)
E       fixture 'args' not found

  available fixtures: cache, capfd, capfdbinary, caplog, capsys, capsysbinary, doctest_namespace, monkeypatch, pytestconfig, record_property, record_xml_attribute, recwarn, tmp_path, tmp_path_factory, tmpdir, tmpdir_factory
  use 'pytest --fixtures [testpath]' for help on them.

我的猜测是args = parser.parse_args()中根本没有加载进去/或者这句话有问题，
这该如何解决？
此外，大佬的各种run**.py文件中的
from pytorch_pretrained_bert.modeling import BertModel, PreTrainedBertModel,
BertPreTrainingHeads
报错。PreTrainedBertModel应改为BertPreTrainedModel
还有，大佬的absa_data_utils.py中没有import csv，但是直接用了csv.reader

Thanks for reminding me as I forgot to put Ubuntu in instructions. I didn't try windows but I think it's hard. For example, bash, compiling for apex, etc.
pytorch-pretrained-bert should be 0.4. I guess you used a newer version.
I'm not sure which version of python picked the unused csv part (from original bert code) out but you can definitely remove that function.

Thanks for reminding me as I forgot to put Ubuntu in instructions. I didn't try windows but I think it's hard. For example, bash, compiling for apex, etc.
pytorch-pretrained-bert should be 0.4. I guess you used a newer version.
I'm not sure which version of python picked the unused csv part (from original bert code) out but you can definitely remove that function.

Thanks for reminding me as I forgot to put Ubuntu in instructions. I didn't try windows but I think it's hard. For example, bash, compiling for apex, etc.
pytorch-pretrained-bert should be 0.4. I guess you used a newer version.
I'm not sure which version of python picked the unused csv part (from original bert code) out but you can definitely remove that function.

thanks for your reply very much，and I‘m very sorry for the bother brought to you by my ignorance.I am a new fish of computer science，and know little  about these konwledge  ,but I should think of these issues were caused by the difference of  library version and operating system。it’s so kind of you for your reminder.wish you have nice life.


Abnormal: Fixed Pkg

Abnormal: Fixed Pkg

Azure Sentinel to Microsoft Sentinel
Required items, please complete
Change(s):

Azure Sentinel to Microsoft Sentinel

Reason for Change(s):

Azure Sentinel to Microsoft Sentinel

Version Updated:

N/A

Testing Completed:

N/A

Checked that the validations are passing and have addressed any issues that are present:

N/A

@v-dvedak
Changed "Azure Sentinel" to "Microsoft Sentinel".

@v-dvedak
Fixed Abnormal Data Connector from "Azure Sentinel" to "Microsoft Sentinel".


Entities disappearing

Sometimes, generally after teleporting, entities are removed from the entity list when they shouldn't be.
@roblabla

Not noticed this myself, more testing needs to be done because this is a severe bug imo.

How can we test this ?

I don't know how to reproduce this. Maybe it should be closed until we can reproduce it ?
Anyway, I haven't noticed this when testing bots, and this is not a big problem, so removing it from the milestone.

Can't reproduce.


Support for React 16

Hey there,
are you planning to officially support React 16?
I have tested this library with 16.0.0-rc.2 and so far, so good.
I'd appreciate feedback from other users how well react-highcharts behaves on their React 16-based environments.
If the feedback will be positive, updated react-highcharts stating React 16 compatibility would be great.

I'm getting this in RC3 / 12.0.0
Warning: Setting defaultProps as an instance property on HighchartsChart is not supported and will be ignored. Instead, define defaultProps as a static property on HighchartsChart.

From what I can tell, I'm not passing anything weird in to the react-highcharts plugin.

I suppose the solution is to add static before defaultProps in this file:
https://github.com/kirjs/react-highcharts/blob/master/src/chartsFactory.jsx

Still have this warning with React 16.0.0 release version.

Any update on this?

Lets wait for
https://github.com/kirjs/react-highcharts/issues/333

any update on this?

I have a bad feeling that this package is abandoned and we would need to contact npm to further work on this package.

Any update on this? The package works fine but I also get the warning after updating to React 16.

There is a PR opened with a fix but still not merged.


Some avatar aren't showing up

For some reason some avatar aren't showing up, while the link works fine https://snapshot.org/#/orangeprotocol.eth https://snapshot.org/#/orangeprotocol.eth/settings maybe something wrong with file format or the Cloudflare worker. File works here: https://ipfs.fleek.co/ipfs/QmeZAvVPKZogrHBFc9SAfUizDbAcvxkSBnqkFixEYpZhjR Maybe Cloudflare worker failed to resolve the image first and cached the response?

Seems like its because it's an SVG


fix(platform-exceptions): remove unnecessary logs

Information
Clean logs

:tada: This PR is included in version 7.36.8 :tada:
The release is available on:

v7.36.8
GitHub release

Your semantic-release bot :package::rocket:


[Kraken] Placing order timeout

I've been having issues of its placing orders takes super long time to return and often result in a timeout. I don't have this problem with other exchanges. Is this a known issue on XChange or Kraken's end?

You can increase the timeout param: https://github.com/timmolter/XChange/wiki/Frequently-Asked-Questions#where-if-anywhere-are-the-http-read-timeouts-configured

I've increased it to 30s, which is way longer than reasonable. In the event of timeout, what's your best practice?

I would catch the exception and do whatever makes the most sense. I've heard Kraken's API is really bad under load.

Yea what i'm doing in case of timeout exception. I'd check the open orders and filled trade to see if that order has been placed. Have to code much more defensively for Kraken. Wanna see if other folks are also experiencing this issue. If yes, then it's mostly their server side's.

I think I can conclude that it's just their server's performance is bad because this also happens on the web UI. I'm going to close this issue.

Kraken is a big mess currently. I've set my timeout to several minutes. You need a time out from their side.  Almost all times a time out from the Kraken side the order isn't placed. (or a service unavailable error). A 502 error is a big guess what happened (trade done or not)

@Achterhoeker Thanks for confirming this. I had to write a lot of code to verify if an order has been placed or not. Even with that, it became unusable because their server's availability and performance are too bad to use for my trading strategy. It'd be great if you can let me know when they fix this. Their customer services are pretty good on the other hand.

I stopped using Kraken over two months ago because of the large number of timeouts and general unreliability. I was thinking of seeing whether things have improved but it seems not!


[BUG] [1.16.5] Server does not shut down properly, leading to world corruption

Version of Advanced Rocketry

1.16.5-2.0.0-16

Have you verified this is an issue in the latest unstable build

[x] Y

Version of LibVulpes

1.16.5-1.0.2-27

Version of Minecraft

1.16.5

Does this occur without other mods installed

[x] Y

Just LibVulpes and AR

Crash report or log or visualVM (if applicable)

https://gist.github.com/SamboyCoding/e6b4e10c5fc9bd83006732665a31b097

Description of the problem

When saving and quitting in singleplayer, or shutting down a dedicated server, the exception above occurs, and the server doesn't actually stop. This inevitably leads to world corruption as well as not being able to load back into the world in singleplayer.

From discussions in the forge discord, FMLJavaModLoadingContext.get() is null outside of the initial game loading process. On top of that, the line in question, https://github.com/Advanced-Rocketry/AdvancedRocketry/blob/1.16.5/src/main/java/zmaster587/advancedRocketry/atmosphere/AtmosphereHandler.java#L81 is not even required as the handler is never registered to that bus, only to MinecraftForge.EVENT_BUS.
Finally, the forge developers suggest against creating multiple handler instances for the same event, and suggest instead

Just have a global event handler that delegates if necessary

Happens to me too xD


fix: dfdaemon unix socket config of downloadGRPC

Signed-off-by: Gaius<EMAIL_ADDRESS>
Description

Fix dfdaemon unix socket config of downloadGRPC.

Related Issue

Motivation and Context

Screenshots (if appropriate)
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation Update (if none of the other choices apply)

Checklist

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.

Codecov Report

Merging #1487 (2a88dd2) into main (0c90975) will increase coverage by 0.17%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #1487      +/-   ##
==========================================
+ Coverage   43.40%   43.58%   +0.17%
==========================================
  Files         120      120
  Lines       10893    10890       -3
==========================================
+ Hits         4728     4746      +18
+ Misses       5657     5642      -15
+ Partials      508      502       -6

Flag
Coverage Δ

Object-compatibility-e2etests
?

e2etests
?

unittests
43.58% <0.00%> (+0.17%)
:arrow_up:

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

Impacted Files
Coverage Δ

client/config/peerhost_linux.go
0.00% <0.00%> (ø)

client/daemon/daemon.go
5.11% <0.00%> (+0.01%)
:arrow_up:

client/daemon/peer/peertask_conductor.go
57.11% <0.00%> (+0.98%)
:arrow_up:

...ent/daemon/peer/peertask_piecetask_synchronizer.go
48.91% <0.00%> (+3.46%)
:arrow_up:


Plugin breaking change to ChangeBlockEvent.Break!

I am currently running

SpongeVanilla version: stable-7 latest setupDecompWorkspace
Java version: jdk1.8.0_241
Operating System: Windows 10

Plugins:

Issue Description
When listening to a ChangeBlockEvent.Break get the transaction BlockSnapshot and get the TileEntity via the getLocation method.
This did return the TileEntity a few months ago. When I updated today it now returns null.
I think this is a change in where the event is fired?

Show some code.

Oh, oops.
    public void onBlockBreak(ChangeBlockEvent.Break event, @Root Player player) {

        for (Transaction<BlockSnapshot> transaction : event.getTransactions()) {
            BlockSnapshot blockSnapshot = transaction.getOriginal();

            if (blockSnapshot.getState().getType() != BlockTypes.STANDING_SIGN && blockSnapshot.getState().getType() != BlockTypes.WALL_SIGN)
                continue;

            if (!blockSnapshot.getLocation().isPresent() || !blockSnapshot.getLocation().get().getTileEntity().isPresent())
                continue;

            ...
        }
    }

Have we managed to find the issue? Nobody can use my plugin on the newer versions.

Have we managed to find the issue?

Obviously not.

Nobody can use my plugin on the newer versions.

It would be super helpful to us if you can find out which version it changed on. If we have a narrower commit range to test, we might be able to fix it quickly.

Show some code.

I'm not sure what your code is meant to do. When asked to show code, give us code that clearly gives us some indication as to what you expect to happen, vs what does happen. I have no idea what your code is meant to do above, which bit fails, etc.
All you have given us is a snippet that doesn't tell us very much, because that is going to cycle the for loop regardless. We want something we can paste into our debuggers and see a tangible outcome.

Still no clue which version, but here is a test plugin jar as a zip TestPlugin-0.1.zip.
Once you've installed it load a world and follow the steps below:

Place a sign.
Break the sign.
Read printout in chat or console.

Previously it would report that the BlockSnapshot had a Location and that there was a TileEntity at the Location.
Now it reports that the BlockSnapshot has a Location but that there is no TileEntity at the Location.

You'll need to rename the .zip back to a .jar

Wait, I forgot to register the Listener. Here's the correct one:
TestPlugin-0.1.zip

Okay, so I tried stable releases all the way back to SV 7.1.9.
SF 7.2.1 RC4011 works fine however. I am beginning to suspect that this might be a long term variation between SV and SF.

Okay, so I tried stable releases all the way back to SV 7.1.9.
SF 7.2.1 RC4011 works fine however. I am beginning to suspect that this might be a long term variation between SV and SF.

Well, I think I've managed to figure out why things don't work but should, and where they are working but shouldn't.
blockSnapshot.getLocation().get().getTileEntity().get() is not the correct way of doing it according to this forum thread but does work in SF so long as the sign is broken directly and not because it was attached to the broken block. However, the correct way as shown in the thread also no longer works.

So this https://github.com/SpongePowered/SpongeCommon/blob/1552585367d12ec04c60bf6e5501a69c0eca36aa/src/main/java/org/spongepowered/common/event/tracking/TrackingUtil.java#L876 should be tile.getContainers() instead? @gabizou
Currently only the nbt data and custom manipulators from plugins are copied over to the BlockSnapshot

@Faithcaio @gabizou Have just tested the above on local copy. Changing that one line does resolve the issue.

but it causes a performance overhead.
which is the reason why its not there

But this was clearly the intended way of doing it back in 2015. So is there a pull request, issue or something where a decision was made to change this? Because presumably, that would explain what the intended replacement is.

I have found a partial solution BlockSnapshot.createArchetype().get().getTileData().get(DataQuery.of("Text1")).get()).split("\"")[3];
It is not a good solution though, so I think this discussion should remain open in some form so that a better solution can be implemented.

It is not a good solution though, so I think this discussion should remain open in some form so that a better solution can be implemented.

There is already an issue opened for https://github.com/SpongePowered/SpongeCommon/issues/950 which is effectively an NBT processor.


Laravel mix's webpack.config.js pulls in a load of shite which makes it impossible to copy into home directory, and laravel-mix doesn't seem to support removing the default rules

<EMAIL_ADDRESS>node: v8.10.0
npm: 3.5.2
With the default laravel project (made with laravel new), the laravel mix webpack.config.js file actually just imports a bunch of other stuff, rather than being a normal config file - this makes it really difficult to copy out into your project directory to customise. You have to dig inside and change the require paths to point into your node_modules, and even after that I'm getting this:
Invalid configuration object. Webpack has been initialised using a configuration object that does not match the API schema.
 - configuration.module has an unknown property 'loaders'. These properties are valid:
   object { defaultRules?, exprContextCritical?, exprContextRecursive?, exprContextRegExp?, exprContextRequest?, noParse?, rules?, strictExportPresence?, strictThisContextOnImports?, unknownContextCritical?, unknownContextRecursive?, unknownContextRegExp?, unknownContextRequest?, unsafeCache?, wrappedContextCritical?, wrappedContextRecursive?, wrappedContextRegExp? }
   -> Options affecting the normal modules (`NormalModuleFactory`).

On top of this, laravel mix seem to be unable to remove the default rules? The above issue wouldn't be so bad if laravel mix WAS an '80%' solution, but trying to add an SVG loader is impossible because the default svg rule can't be removed ???

Even copying the entire laravel-mix directory into the project root results in this
TypeError: Cannot redefine property: tap
    at Function.defineProperty (<anonymous>)
    at Object.<anonymous> (/var/www/node_modules/laravel-mix/src/helpers.js:21:8)

I have absolutely no idea how people are using this today, what's everyone else's setup??

I have no idea what you're trying to do, but you shouldn't be copying the Mix directory in node_modules to your project root. Of course that's not going to work.
You can override any of the default rules with mix.override(() => {});


onmessage handler to clear all caches

Forcibly delete all caches for the given origin, invoked via a postMessage() with a specific payload from a controlled page. Useful for sites that need to implement a "kill switch".

Closed in #5


[Feature] List of keys returned from onWillAccept and onAccept

Now onWillAccept and onAccept return a list of keys of the stored widget, so you can know the order of the dragged tiles.

please merge

please merge

You can temporarily use my fork to achieve this. It's updated til the latest push on the original repo.
Sadly, i think the owner doesn't mantain anymore this repo.
And btw, i saw your other opened issues. For vertical and horizonal spacing, you can resolve using the padding function. For observer changes, it actually works (i've tested it with changeNotifier and riverpod), so if it doesn't work for you, probably you're doing something wrong.


[19.03] environment.noXlibs: disable gnome3 for pinentry

Backport of #59051 to release-19.03
cc @delroth @timokau

This is a backport of f8eec8dc34733db11a0619809c86af6903e267f6 right? Should probably use cherry-pick -x for that.
Other than that, I have no opinion on this. I don't know why pinentry_curses would depend on gnome3. Why did you CC me?


Dhruvr4

The time range option is added for the time profile and usage profile for sum and sumd file

I've split the Time Profile and Usage Profile parts of this PR into #127 and #128. Take a look at those to review them and ensure that they've captured the main thrust of your changes here.


Removed internal variable pass_context_arg_name

Fixes #1532 .
Changes proposed in this pull request:

I removed the parameter pass_context_arg_name passed internally as asked from the various constructors, and left in the SecurityHandlerFactory a constant called context_kw, following the same pattern of required_scopes_kw.

I don't know if it is better to put in uppercase both values (context_kw and required_scopes_kw), now they are constants in the end.
I left as the other one to have consistency, but if you prefer I can change them both.

Thanks @leonardofesta!
Could you update the failing tests to make them pass?

Hi @leonardofesta, did you have a chance to have another look at this? Let me know if you need support.

Sorry @RobbeSneyders I was on vacation and I didn't saw the results. Now it should pass all tests.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

4 of 6 (66.67%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.01%) to 94.645%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

connexion/security/security_handler_factory.py
2
4
50.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
2704

Relevant Lines:
2857

💛  - Coveralls

Any update @leonardofesta?

Any update @leonardofesta?
I just pushed the update.
I'm using context_ as suggested as value, in both cases used as constant.

I basically restored the decorator functionality by using a boolean with default == False to maintain the same logic
In the security handler part, please recheck  but now it should be correct

Thanks @leonardofesta, looks good to me.
I pushed a change to the test, as it wasn't actually testing the new behavior.


Maven support to generate java classes

I'm trying to use apollo-android for my java(maven) project, i don't see any plugin for maven to generate java classes using schema.json and query.

Maven is not currently supported, only gradle. The only way right now I think of is to run manually compiler via command line.

Duplicate with https://github.com/apollographql/apollo-android/issues/1178

can you give me the steps to generate maven artifact from the github fork?


fix(GetConstraintsPertab): Also work with non-lowercase column names

Possibly closes #6
(Sorry about the linebreak thing at the end of the file, GH UI doesn't let me not have one when I edit files there.)

@janpio will it be possible to merge this fix on your branch (https://github.com/janpio/mock-data/pull/1), once done I will have this pull request squashed onto master branch for a new release

Done.
Still a bit confused how the problem I tried to solve is solved via the other one, but - if it works it works 🤷♀

The issue is the columns case sensitivity is kept till it reached the copy statement, during the execution postgres replaces the uppercase with lower case, so added double quotes at copy statement for column to keep the case sensitivity.


Serenity/airbrake

I'm not convinced that catching the raise from unpermitted_parameters!(params)  is the way to go here.
Try overriding this method directly instead.


master dissertation : ELK stack+elastalert+X-Pack (elastalert doesn't send email)

Hello community, I'm actually working on my final project before graduating and the term is 'SIEM = ELKStack with elastalert for email alerting and X-Pack .
I've followed instruction given on a documentation, i can get match while testing rules. elastalert start normally but no email is received via gmail. Y
I've configured SSMTP and can reciev emailvia command line.
Honestly a big help is warmly welcomed.
@Qmando
OS : CenTos 7
python v 2.7
### 1. rules
### 1.1 Example_rules.yaml
name: Example frequency rule
type: frequency
index: filebeat-*
num_events: 2
timeframe:
hours: 4
filter:

query:
query_string:
query: "message: authentication failure OR failure password"
timestamp_field: "@timestamp"
alert:
"email"
email<EMAIL_ADDRESS>smtp_host: "smtp.gmail.com" #for google email addresses
smtp_port: 587 #for google email addresses
smtp_ssl: true
from_addr: "elastalert@"
smtp_auth_file: "smtp_auth_file.yaml"

1.2 : smtp_auth_file.yaml
user<EMAIL_ADDRESS>password: "xxxxxx"
1.3 : testing rules
Would have written the following documents to writeback index (default is elastalert_status):
silence - {'rule_name': 'Example frequency rule', '@timestamp': datetime.datetime(2019, 10, 3, 12, 59, 34, 674200, tzinfo=tzutc()), 'exponent': 0, 'until': datetime.datetime(2019, 10, 3, 13, 0, 34, 674193, tzinfo=tzutc())}
elastalert_status - {'hits': 1232, 'matches': 616, '@timestamp': datetime.datetime(2019, 10, 3, 12, 59, 34, 724094, tzinfo=tzutc()), 'rule_name': 'Example frequency rule', 'starttime': datetime.datetime(2019, 10, 3, 8, 57, 9, 275361, tzinfo=tzutc()), 'endtime': datetime.datetime(2019, 10, 3, 12, 59, 33, 275361, tzinfo=tzutc()), 'time_taken': 1.4450719356536865}
1.4 : starting elastalert
INFO:elastalert:Starting up
INFO:elastalert:Sleeping for 59.994702 seconds
2. ssmtp
mailhub:587=smtp.gmail.com   # SMTP server for Gmail
Hostname=soc.kytl.tech
UseTLS=YES
UseSTARTTLS=Yes
FromLineOverride=YES #TO CHANGE FROM EMAIL
<EMAIL_ADDRESS># Redirect root email
<EMAIL_ADDRESS>AuthPass=xxxxxxx
AuthMethod=LOGIN
RewriteDomain=gmail.com
#file  : revaliases#
root:xxxxxxxx@gmail.com:smtp.gmail.com:587
### testing email email received **
mail -s "adasdas"<EMAIL_ADDRESS>root<EMAIL_ADDRESS>| root<EMAIL_ADDRESS>| 3:04 PM (1 minute ago)
hello elastalert world

@Qmando I wish you can help on this or just  a guidelines.

Maybe try setting the from_addr to your full email address ? Google is probably rejecting the email because of that

@Qmando       from_addr<EMAIL_ADDRESS>as shown above, have changed it still email not received. Is there any other way of testing for email alerting, may be yahoo etc which is sure to receive email alerting

The error that occurred when specifying the Gmail address with from_addr was not a program problem but a setting problem.
Launch a web browser on the terminal running ElastAlert, access the Google settings screen, go to the Google account → App that can access the account → Allow less secure apps: Disable → Change to enable. When I specified the Gmail address with from_addr and confirmed the operation, it worked normally.

smtp_port: 587
smtp_host: "smtp.gmail.com"
smtp_port: 587
smtp_ssl: false
smtp_auth_file: '/opt/elastalert/smtp/smtp_auth_user.yaml"

smtp_auth_user.yaml
user: xxx@gmail
password: xxx

smtp_port: 465
smtp_host: "smtp.gmail.com"
smtp_port: 465
smtp_ssl: true
smtp_auth_file: '/opt/elastalert/smtp/smtp_auth_user.yaml"

smtp_auth_user.yaml
user: xxx@gmail
password: xxx


Select a User shows empty while creating an event

Hi,
While creating an event, Select a Channel and Select a user shows empty. How to make it work. Am I missing some integration?

Please help. Thanks in Advance.

Hi. Try typing something in this field; if it doesn't help, you can try accessing Mattermost through a browser, open the console, and check what the requests return and if there are any errors in the logs.


Fix spacing in Footnotes

Not jumpy anymore after #834 but now 'overlapping'

It's not there already and I hope (very much, yeah) that it will not be
there after our big css refactor
On Fri, Sep 14, 2018 at 11:12 AM Oliver Buchtala<EMAIL_ADDRESS>wrote:

Assigned #832 https://github.com/substance/texture/issues/832 to
@Integral https://github.com/Integral.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/substance/texture/issues/832#event-1845590456, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AALG-ucxufmEgZJInTZtexcOPeqbILzOks5ua3LngaJpZM4Wo5zy
.

@oliver---- that's ok, we know about this and trying hard to improve :)

Fixed via #847


On Scroll jumps to end of file

Issue Type: Bug
I have my cursor set on first line, but when I scroll the page using mouse wheel it jumps at the end of the file.
This happens when I switch app or after some idle time.
After the behvior it starts to work fine untill I stop using the editor and retry after a while to scroll.
I'm working on vue file atm,  haven't tested on other files.
VS Code version: Code 1.36.1 (2213894ea0415ee8c85c5eea0d0ff81ecc191529, 2019-07-08T22:55:08.091Z)
OS version: Linux x64 4.18.0-25-generic snap

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-6200U CPU @ 2.30GHz (4 x 2720)

GPU Status
2d_canvas: unavailable_softwareflash_3d: disabled_softwareflash_stage3d: disabled_softwareflash_stage3d_baseline: disabled_softwaregpu_compositing: disabled_softwaremultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwareoop_rasterization: disabled_offprotected_video_decode: disabled_offrasterization: disabled_softwareskia_deferred_display_list: disabled_offskia_renderer: disabled_offsurface_synchronization: enabled_onvideo_decode: disabled_softwareviz_display_compositor: disabled_offwebgl: unavailable_softwarewebgl2: unavailable_software

Load (avg)
1, 1, 2

Memory (System)
7.50GB (0.63GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (45)

Extension
Author (truncated)
Version

html-snippets
abu
0.2.1

project-manager
ale
10.5.2

copy-relative-path
ale
0.0.2

alignment
ann
0.3.0

path-intellisense
chr
1.4.2

bracket-pair-colorizer-2
Coe
0.0.28

vscode-markdownlint
Dav
0.28.0

vscode-eslint
dba
1.9.0

githistory
don
0.4.6

jquerysnippets
don
0.0.1

es7-react-js-snippets
dsz
2.3.0

gitlens
eam
9.8.5

vscode-html-css
ecm
0.2.0

EditorConfig
Edi
0.13.0

vscode-npm-script
eg2
0.3.8

vsc-material-theme
Equ
29.2.0

prettier-vscode
esb
1.9.0

auto-close-tag
for
0.5.6

code-runner
for
0.9.11

gc-excelviewer
Gra
2.1.32

beautify
Hoo
1.5.0

sftp
lix
1.12.7

Angular-BeastCode
Mik
8.0.10

HTMLHint
mka
0.6.0

vscode-docker
ms-
0.7.0

mssql
ms-
1.6.0

python
ms-
2019.6.24221

azure-account
ms-
0.8.4

Go
ms-
0.11.4

vscode-typescript-tslint-plugin
ms-
1.2.2

debugger-for-chrome
msj
4.11.6

vetur
oct
0.21.1

vscode-coldbox
ort
0.1.0

material-icon-theme
PKi
3.8.1

vue-vscode-snippets
sdr
1.7.1

code-settings-sync
Sha
3.4.0

stylelint
shi
0.51.0

code-spell-checker
str
1.7.17

bootstrap4-vscode
the
6.1.0

vscodeintellicode
Vis
1.1.7

vscode-icons
vsc
8.8.0

gitblame
wad
2.8.1

JavaScriptSnippets
xab
1.7.2

markdown-all-in-one
yzh
2.4.0

html-css-class-completion
Zig
1.19.0

Does this reproduce in the latest VS Code insiders build with all extensions disabled?

@mjbvz  Okay, so I just installed everything on my home PC and so far I haven't seen the issue again.
The bug is on my office laptop. As its weekend I'm unable to test it as I don't have the laptop with me.
Will check on Monday and let you know. Thanks for reply.

Same problem.
Ubuntu 18.04
Version: 1.36.1
Commit: 2213894ea0415ee8c85c5eea0d0ff81ecc191529

Sorry for late reply @mjbvz
*So I Installed insider build and do not see the issue. No plugins just plain install.
How do I reset vscode to uninstall all plugins and settings? So I can do the test run on it as well.
Maybe I'll uninstall vscode that would reset it, and try.

@bsienn a screen recording can definitely help. What happens when you run code-insiders --disable-extensions?

@rebornix Sorry to interrupt your conversation. I does not have the Linux machine at hand now, but on my end, the problem is quite easy to describe.

Open a file in VS Code.
Use Alt+Tab to switch to any other application and then scroll mouse.
Use Alt+Tab to switch back to VS Code.
Slightly scroll mouse wheel (this step is necessary), then all the scrolls I did in another application will get accumulated here to VS Code window. As a result, it suddenly jumps to another line.

PS: I installed two plugins: Bracket Pair Colorizer and Remote VSCode. Both my MacBook and Linux have the exactly same VSCode and plugins, but it works fine on my MacBook.
PPS: For the Linux machine, I have dual monitors. In step 2, scrolls in another monitor does not accumulate. Only scrolls in the same monitor of VSCode count.

As the above comment says, its exactly as it says.
An other note is, I do have dual setup monitor, and this does happen on my 2nd screen on Ubuntu.
@rebornix I installed insider with no new plugins, but I didn't did disable-extensions. Will try and let you know.
I have PC at home with single monitor, with exactly same plugins, but the issue doesn't reproduce on PC. Only happens in my laptop with dual LCD.

I am using Ubuntu 18.04.2 LTS, Linux kernel x64 4.18.0-25, single monitor. Using code 1.36.1 2213894ea0415ee8c85c5eea0d0ff81ecc191529 x64.
The issue happens fairly easy on all my computers (pc on Arch Linux, new notebook on Ubuntu and old notebook on Arch Linux): change focus to another app, scroll for some time (mouse or touchpad), go back to code. This problem bothers me for some time, I was really surprised that the issue was only opened 10 days ago.
I made a simple vídeo to Youtube using code-insiders on the same computer (OS and kernel specified above) on the version 1.37.0-insider 3802a986823977ec472fc993854e1a753037fc8f x64 and command-line code-insiders --disable-extensions: https://youtu.be/5hd1k_oTa_M

Hi @bsienn, I found this bug also appears when switching between some other apps like Chrome and Files app. Maybe a Ubuntu bug? How about your computer?

Oh wow, that looks crazy. On our side, we react to mousewheel events and to the scroll values they bring in. We have to use the values given by the events themselves to cover ballistic scrolling via the touchpad...
I think this is an issue in Electron/Chromium. Does the same happen when alt-tabbing from Chromium or another Electron based app like Atom/Slack?

Let's track in #28795


”Frecăței”, ”Cataloi”, ”Poșta”, ”Telița” are listed in the drop-down menu

Description:
When user writes ”Frecăței”, ”Cataloi”, ”Poșta”, ”Telița” the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Frecăței"

Expected results
The location appears in the drop-down menu as "Sat Frecăței, TULCEA (Frecăței)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Cataloi"

Expected results
The location appears in the drop-down menu as "Sat Cataloi, TULCEA (Frecăței)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Poșta"

Expected results
The location appears in the drop-down menu as "Sat Poșta, TULCEA (Frecăței)".

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Write in the search bar "Telița"

Expected results
The location appears in the drop-down menu as "Sat Telița, TULCEA (Frecăței)".


[Codebundle] Postgres Blocked Query and Blocked Backends Inspection

Scope
Runs a taskset inspection on a postgresql database, checking for blocked queries  and table locks on Postgres and Report Remediations.
Requested By (Optional)

@harsh098

Related Integrations

Kubernetes
Postgresql

Definition of Done

[ ] Attach a screenshot of the SLI or Taskset running in a workspace.

[ ] Codebundle Implementation accomplishes scope (or provide a justification for why it may differ)

[ ] Attach a screenshot of the codebundle pushing a metric locally (if an SLI) or showing a report (if a taskset)

Optional Comments
This could potentially be split into 2 variants: one for public but authenticated db instances, and one for kubernetes workloads connected to via the kube api

Codebundle Repo
https://github.com/harsh098/postgres-codecollection
Screenshots
Taskset
Checks for Blocked Queries on the Cluster

Bonus : Added Mitigations

SLI
Reports no of blcoking queries on the Postgres Cluster

This codebundle request and detail history is being migrated to an internal tracking tool for authors. Please reach out to the RunWhen team if you have any questions.


Inline mode doesn't call on_chosen_inline_result()

Hi,
I run the example from docs on inline bot (see below).
And never see on_chosen_inline_result() is called.  Instead, when I click to choose result in Telegram I see chat() is called.
Not sure it's a bug, probably I do something wrong.
Can somebody help with that?
====== inlinebot.py =====
import sys
import telepot
from telepot.namedtuple import InlineQueryResultArticle, InputTextMessageContent
def on_inline_query(msg):
def compute():
query_id, from_id, query_string = telepot.glance(msg, flavor='inline_query')
print ('Inline Query:', query_id, from_id, query_string)
    articles = [InlineQueryResultArticle(
        id='abc',
        title=query_string,
        input_message_content=InputTextMessageContent(
            message_text=query_string
        )
    )]

    return articles

answerer.answer(msg, compute)

def on_chosen_inline_result(msg):
result_id, from_id, query_string = telepot.glance(msg, flavor='chosen_inline_result')
print ('Chosen Inline Result:', result_id, from_id, query_string)
def chat(msg):
print ('Get msg:', msg)
TOKEN = sys.argv[1]  # get token from command-line
bot = telepot.Bot(TOKEN)
answerer = telepot.helper.Answerer(bot)

I got it.  /setinlinefeedback should be used to get a on_chosen_inline_result() called.


failed to use scrcpy wirelessly

➜  atrade git:(master) adb connect <IP_ADDRESS>:5555
connected to <IP_ADDRESS>:5555
➜  atrade git:(master) scrcpy --always-on-top --turn-screen-off
adb: error: failed to get feature set: more than one device/emulator
2019-06-26 12:38:17.613 scrcpy[6988:3034997] ERROR: "adb push" returned with value 1
➜  atrade git:(master) adb devices
List of devices attached
xxxxxxxxxxxxx	device
<IP_ADDRESS>:5555	device

➜  atrade git:(master) scrcpy --always-on-top --turn-screen-off
adb: error: failed to get feature set: more than one device/emulator
2019-06-26 12:38:33.450 scrcpy[7188:3036929] ERROR: "adb push" returned with value 1
➜  atrade git:(master) scrcpy
adb: error: failed to get feature set: more than one device/emulator
2019-06-26 12:38:38.503 scrcpy[7204:3037078] ERROR: "adb push" returned with value 1

it seems I have multiple devices, I have to specify one of them, this issue can be closed

scrcpy -s <IP_ADDRESS>:5555

(or unplug USB to get only 1 device)

scrcpy -s <IP_ADDRESS>:5555

(or unplug USB to get only 1 device)

I'm sure I've already unplugged usb, no idea why it list two devices


Remove unused keys from crosschain's keys.go

Many of the keys are not used in code:
https://github.com/zeta-chain/zeta-node/blob/3783141a9dc4b35b3dcc64f958bc56f3beff24d6/x/crosschain/types/keys.go#L37-L59
I think here are the ones that are actually used:
const (
	SendKey                = "Send-value-"
	LastBlockHeightKey     = "LastBlockHeight-value-"
	ChainNoncesKey         = "ChainNonces-value-"
	GasPriceKey            = "GasPrice-value-"
	TSSKey                 = "TSS-value-"
	KeygenKey              = "Keygen-value-"
	OutTxTrackerKeyPrefix  = "OutTxTracker-value-"
	NonceToCctxKeyPrefix   = "NonceToCctx-value-"
	PendingNoncesKeyPrefix = "PendingNonces-value-"
)

I'm going to create a more general task about removing all unused variables


Objective-C compatibility

Some quick changes I made to make this project Objective-C compatible
@delannoyk This definitely needs review and I'm very open to doing things differently with TimeRange or .failed error if you disagree with my fixes

Addresses #67 if merged

Codecov Report

Merging #105 into master will decrease coverage by 0.35%.
The diff coverage is 66.66%.

@@            Coverage Diff            @@
##           master    #105      +/-   ##
=========================================
- Coverage   73.85%   73.5%   -0.36%
=========================================
  Files          39      39
  Lines        2735    2721      -14
=========================================
- Hits         2020    2000      -20
- Misses        715     721       +6

Impacted Files
Coverage Δ

AudioPlayer/AudioPlayer/player/AudioPlayer.swift
29.95% <ø> (ø)
:arrow_up:

...layer/AudioPlayer/player/AudioPlayerDelegate.swift
0% <ø> (ø)
:arrow_up:

...er/player/extensions/AudioPlayer+CurrentItem.swift
10% <0%> (-1.54%)
:arrow_down:

...er/player/extensions/AudioPlayer+PlayerEvent.swift
24.56% <0%> (-0.22%)
:arrow_down:

...yer/player/extensions/AudioPlayer+RetryEvent.swift
0% <0%> (ø)
:arrow_up:

...ayer/AudioPlayerTests/AudioPlayerState_Tests.swift
100% <100%> (ø)
:arrow_up:

...ioPlayer/AudioPlayer/player/AudioPlayerState.swift
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Wow that is some cool work @ddfreiling
I'll try to review asap!

Hey man, Sorry 3 years later I'm waking up and I think the community has mostly moved away from Obj-C.
I'm gonna close this PR.


Use a common BLAKE2 definition for both b and s, thus also bringing XOF support to BLAKE2s. Document how BLAKE2 XOFs work

The first three patches are functional no-ops, and just move the BLAKE2B stuff from being declared directly into a macro.
The fourth patch replaces the manual implementation of BLAKE2s with a call to the macro, thus matching them both in funxionality. This effectively completes 786b9a8d3f8e203c5536e36b9a9bab83bde0311a.
The fifth patch explicitly documents what I discovered the hard way over three hours yesterday:

openssl has EVP_DigestFinalXOF() which is a shorthand for setting "xoflen" to the argument and then running EVP_DigestFinal()
this matches how SHAKE works (output length has no bearing on the hashing, so you can set "xoflen" at any time)
BLAKE2 puts the output length into the parameter block, which is part of the seed
naturally, you need to pass the output length to the init, so you set "xoflen" in the params block for EVP_DigestInit_ex2()
setting it after init and before the first byte does nothing, but you can still do it
EVP_DigestFinalXOF() has no bearing on it at all, but you can still pass it there. or you can just use EVP_DigestFinal()
if you didn't set it in EVP_DigestInit_ex2() but did pass it to EVP_DigestFinalXOF(), you just get wrong data (full-width hash, but truncated)
support for "xoflen" in BLAKE2b appeared in 786b9a8d3f8e203c5536e36b9a9bab83bde0311a in 2021, and is only in openssl-3.2.0-alpha[12] tags
EVP_DigestInit_ex2() doesn't error if it gets a parameter the hash function doesn't understand
I know this because on Bookworm openssl 3.0.11, code with full error-checking and EVP_DigestFinal() produces a truncated hash, and on upstream openssl HEAD the same code produces the correct short hash
you only get an error if you use EVP_DigestFinalXOF() which you shouldn't be using since it does nothing but implies it does

This all could've been avoided with a startlingly small patch that documents it (+20-4, and most of that +20 is in EVP_MD-BLAKE2.pod which is mostly copied from -SHAKE). The first four patches are really in service of not having to differentiate between b2s and b2b in the manual.
This also means that EVP_DigestInit_ex2() blindly accepting whatever it doesn't understand is an oddly dangerous design?

@mattcaswell I actually think this should go in 3.2.0
The refactoring is fairly straightforward and the xoflen support in 2b only is quite surprising.
The documentation is clearly needed as the xoflen handling in Blake is very much different from SHAKE.

@slontis, does this impact your proposal for the digest squeeze function?

OTC: should we classify these are XOF and use xoflen for the digest's output size?  The alternative is that they are not XOF and use size.

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

This PR conflicts with 6a0ae393dd554eb718e5148696e8f437d4faae5b
So needs rebasing/fixing.

rebased

@paulidale AFAIK the OTC hold question is not relevant anymore. Can you please remove the label if you agree?

@nabijaczleweli I think you'll also need to modify EVP_MD-Blake2 manpage and I'd kindly ask you to add a CHANGES.md entry about adding the variable size support to Blake2s.

Well, if we're going that far, I should probably document that BLAKE2b also has it? 786b9a8 doesn't have any docs in it, incl. CHANGES.md.

Hold removed, we should have removed it after the discussion.
For the record, the OTC decided that this wasn't an XOF digest and that the digest size parameter should be used to specify the length.  This change to be made to the pre-existing digest before 3.2 beta series releases (because the variable length change hadn't been in a release, it could be updated so long as it was before the API freeze at the first 3.2 beta release).

EVP_MD-BLAKE2(7) was already modified with text much like was in this PR in https://github.com/openssl/openssl/commit/6a0ae393dd554eb718e5148696e8f437d4faae5b#diff-a94d99d3c13153981b6da8a09b5d5fefcb3f8d6016c8cf0752f44fb30addc919.
CHANGES.md entry added.

This pull request is ready to merge

Merged to the master branch with partially squashing the commits. Thank you for your contribution.


FORMAT ERROR  : Your output was not formatted correctly. You must always include one discussion and one command as part of your response.

Describe the bug
I am trying to run the swe agent on local llm without Open ai key, For this I have modified the OpenAI Class to access the gradio url. In Gradio I have hosted a model "Code Qwen 0.5B".
Below is the modifed Open AI Class to integrate the gradio.
`
class OpenAIModel(BaseModel):
MODELS = {
"gpt-3.5-turbo-0125": {
"max_context": 16_385,
"cost_per_input_token": 5e-07,
"cost_per_output_token": 1.5e-06,
},
"gpt-3.5-turbo-1106": {
"max_context": 16_385,
"cost_per_input_token": 1.5e-06,
"cost_per_output_token": 2e-06,
},
"gpt-3.5-turbo-16k-0613": {
"max_context": 16_385,
"cost_per_input_token": 1.5e-06,
"cost_per_output_token": 2e-06,
},
"gpt-4-32k-0613": {
"max_context": 32_768,
"cost_per_input_token": 6e-05,
"cost_per_output_token": 0.00012,
},
"gpt-4-0613": {
"max_context": 8_192,
"cost_per_input_token": 3e-05,
"cost_per_output_token": 6e-05,
},
"gpt-4-1106-preview": {
"max_context": 128_000,
"cost_per_input_token": 1e-05,
"cost_per_output_token": 3e-05,
},
"gpt-4-0125-preview": {
"max_context": 128_000,
"cost_per_input_token": 1e-05,
"cost_per_output_token": 3e-05,
},
"gpt-4-turbo-2024-04-09": {
"max_context": 128_000,
"cost_per_input_token": 1e-05,
"cost_per_output_token": 3e-05,
},
}
SHORTCUTS = {
    "gpt3": "gpt-3.5-turbo-1106",
    "gpt3-legacy": "gpt-3.5-turbo-16k-0613",
    "gpt4": "gpt-4-1106-preview",
    "gpt4-legacy": "gpt-4-0613",
    "gpt4-0125": "gpt-4-0125-preview",
    "gpt3-0125": "gpt-3.5-turbo-0125",
    "gpt4-turbo": "gpt-4-turbo-2024-04-09",
}

def __init__(self, args: ModelArguments, commands: list[Command]):
    super().__init__(args, commands)

    # Set OpenAI key
    cfg = config.Config(os.path.join(os.getcwd(), "keys.cfg"))
    api_key = cfg.get("OPENAI_API_KEY", None)
    if api_key:
        self.client = OpenAI(api_key=api_key, base_url="https://api.openai.com")
    else:
        self.api_url = "https://451edefe4e25c41ffb.gradio.live/api/predict/"

def history_to_messages(
        self,
        history: list[dict[str, str]],
        is_demonstration: bool = False,
) -> str | list[dict[str, str]]:
    """
    Create `messages` by filtering out all keys except for role/content per `history` turn
    """
    # Remove system messages if it is a demonstration
    if is_demonstration:
        history = [entry for entry in history if entry["role"] != "system"]
        return "\n".join([entry["content"] for entry in history])
    # Return history components with just role, content fields
    return [{k: v for k, v in entry.items() if k in ["role", "content"]} for entry in history]

def query(self, history: list[dict[str, str]], is_demonstration: bool = False) -> str:
    """Query the custom Gradio API with the given prompt and return the response."""
    # Format the history into messages
    messages = self.history_to_messages(history, is_demonstration)

    # For demonstration, messages is a single string
    if is_demonstration:
        prompt = messages
    else:
        prompt = json.dumps(messages)

    try:
        response = requests.post(self.api_url, json={"prompt": prompt})
        if response.status_code == 200:
            return response.json()  # Adjust depending on the actual response structure
        else:
            return "Failed to get response: " + response.text
    except Exception as e:
        return f"An error occurred: {str(e)}"

`
And Below is the Hosted Model Gradio Code.
`from transformers import AutoModelForCausalLM, AutoTokenizer
device = "cuda" # the device to load the model onto
model = AutoModelForCausalLM.from_pretrained(
"Qwen/Qwen1.5-0.5B-Chat",
torch_dtype="auto",
device_map="auto"
)
tokenizer = AutoTokenizer.from_pretrained("Qwen/Qwen1.5-0.5B-Chat")
prompt = "Give me a short introduction to large language model."
messages = [
{"role": "system", "content": "You are a helpful assistant."},
{"role": "user", "content": prompt}
]
text = tokenizer.apply_chat_template(
messages,
tokenize=False,
add_generation_prompt=True
)
model_inputs = tokenizer([text], return_tensors="pt").to(device)
generated_ids = model.generate(
model_inputs.input_ids,
max_new_tokens=512
)
generated_ids = [
output_ids[len(input_ids):] for input_ids, output_ids in zip(model_inputs.input_ids, generated_ids)
]
response = tokenizer.batch_decode(generated_ids, skip_special_tokens=True)[0]
from transformers import AutoModelForCausalLM, AutoTokenizer
from gradio import Interface
import torch
Load the Qwen model (assuming you have transformers installed)
device = "cuda" if torch.cuda.is_available() else "cpu"  # Use GPU if available
model = AutoModelForCausalLM.from_pretrained(
"Qwen/Qwen1.5-0.5B-Chat",
torch_dtype="auto",
device_map="auto"
)
tokenizer = AutoTokenizer.from_pretrained("Qwen/Qwen1.5-0.5B-Chat")
def generate_introduction(prompt):
"""
Generates an introduction to large language models using the Qwen model, formatted for a specific system.
Args:
    prompt: A string prompt for the model.

Returns:
    A string containing the formatted response as required by the system.
"""
# Generate the model response
messages = [
    {"role": "system", "content": "You are a helpful assistant."},
    {"role": "user", "content": prompt}
]
text = tokenizer.apply_chat_template(
    messages,
    tokenize=False,
    add_generation_prompt=True
)
model_inputs = tokenizer([text], return_tensors="pt").to(device)

generated_ids = model.generate(
    model_inputs.input_ids,
    max_new_tokens=512
)
generated_ids = [
    output_ids[len(input_ids):] for input_ids, output_ids in zip(model_inputs.input_ids, generated_ids)
]

response = tokenizer.batch_decode(generated_ids, skip_special_tokens=True)[0]

return response.strip()

Define the Gradio interface
interface = Interface(
fn=generate_introduction,
inputs="text",
outputs="text",
title="Introduction to Large Language Models",
description="Enter a prompt to get a short introduction to large language models generated by the Qwen model."
)
Launch the Gradio interface
interface.launch()
`
After pasting gradio url into models.py openAI class I run the project.
But When I run below command to start the project.
python3.9 run.py  --model_name gpt4  --data_path  /home/uzair/project/test-repo/problem_statements/1.md --repo_path /home/uzair/project/test-repo --config_file config/default_from_url.yaml --apply_patch_locally
It starts and agent copies the local issue from local repo, after copying it shows some format error given below:
`
WARNING  FORMAT ERROR
Your output was not formatted correctly. You must always include one discussion and one command as part of your response. Make sure you
do not have multiple discussion/command tags.
Please make sure your output precisely matches the following format:
DISCUSSION
Discuss here with yourself about what your planning and what you're going to do in this step.
     ```
     command(s) that you're going to run
     ```

WARNING  Malformat limit reached:
Failed to get response: {"detail":[{"type":"missing","loc":["body","data"],"msg":"Field required","input":{"prompt":"[{"role":
"system", "content": "SETTING: You are an autonomous programmer,
`
Questions

Can I run swe agent with local llm?
Can you help in resolving error?
Is it possible to acheive what I am trying to do?

Steps/commands/code to Reproduce
python3.9 run.py  --model_name gpt4  --data_path  /home/uzair/project/test-repo/problem_statements/1.md --repo_path /home/uzair/project/test-repo --config_file config/default_from_url.yaml --apply_patch_locally
Error message/results
WARNING  FORMAT ERROR
Your output was not formatted correctly. You must always include one discussion and one command as part of your response. Make sure you
do not have multiple discussion/command tags.
Please make sure your output precisely matches the following format:
DISCUSSION
Discuss here with yourself about what your planning and what you're going to do in this step.
     ```
     command(s) that you're going to run
     ```

WARNING  Malformat limit reached:
Failed to get response: {"detail":[{"type":"missing","loc":["body","data"],"msg":"Field required","input":{"prompt":"[{"role":
"system", "content": "SETTING: You are an autonomous programmer,
System Information
Windows Core i7 8 gen
Checklist

[X] I'm running with the latest docker container/on the latest development version
[X] I've searched the other issues for a duplicate
[X] I have copied the full command/code that I ran (as text, not as screenshot!)
[X] If applicable: I have copied the full log file/error message that was the result (as text, not as screenshot!)
[X] I have enclosed code/log messages in triple backticks (docs) and clicked "Preview" to make sure it's displayed correctly.

Hi!
SWE-agent is complex and will probably not run well on small, local LMs. For now, rather than breaking our heads trying to figure how to make this work, we're going to wait a few months to let local LMs get better before we try to make SWE-agent work with them.
Closing this for now, as we will prioritize other topics, but if this is something you care about then we think it's a cool topic to work on and you should continue. We're sure someone will get this to work at some point.


Add workflow graphic to about page

The about page needs a graphic that describes the flow of batcher in order to break up the text.

Not required


Card

Feature Request
Cards with 0px border Radius.
Problem description
I have an idea related to cards, Cards does not have hover property .
Proposed solution
Cards should have hover property. On hover, we could have change background Color, add shadow property, change border Radius,
change outline color etc.
MVP
Component code:

I am dummy Heading

At vero eos et accusamus et iusto odio dignissimos ducimus qui blanditiis praesentium voluptatum deleniti atque corrupti
            quos dolores et quas molestias excepturi sint occaecat

Styling:
.mainContainer{
    background-color:lightcoral;
    height: 13rem;
    width: 17rem;
    align-self: center;
    margin: 12rem;
    padding: 10px;
    border-width: 1px;
    border-color: lightcoral;
    border-style: solid;
}
.mainContainer:hover{
    background-color:lightpink;
    border-radius: 8px;
    border-color: lightslategray;
    border-width: 1px;
}
.headingContainer{
background-color:transparent;
display: flex;
}
.textContainer{
background-color:transparent;
display: flex;
height: 7.5rem;
}
.textBold{
font-weight: bold;
font-size: 20px;
}

Can i work on this.

Hey @asma-mughal I would like to work on it. Could you assign it to me?

Sure @asma-mughal

Hey @asma-mughal I would like to work on it. Could you assign it to me?

But I don't know how to assign issue to you?


Merge of branch 6-14-24-1500 containing merge of changes from staging

This PR contains the CurrentProd6-14-1500 branch which contains the changes from staging which were merged in via #589
( This process is being tried because after the reverts it is not showing/allowing to create a PR from Staging to Main as it does not see any difference between the 2 branches - whereas there are a lot of actual changes in staging yet to be pushed out)

@petersonjdNIH  @colemanjrNIH  Please take a look at this PR and comment if this will work in merging the changes we need from Staging.
This PR does not directly show all the changes we expect, however the only commit that it shows is the #589 which does show all the staging changes we want.

This PR has zero files changed, so I don't think it'd do anything?


[scan] Fix device parameter not working in fastlane

Fix device parameter not working in fastlane

Self +1, since this is a hotfix release


Find Sum of all Sub Matrices of a given Matrix 3

Is your feature request related to a problem? Please describe.
Finding the sum of all submatrices of a given matrix and returning the total sum.
Describe the solution you'd like
I would like to solve this in C++. I would like to optimize the O(n ^ 4) time and O(n ^ 2) space approach I used here #58

Sure @kiruba-r11 , you can go ahead !


Update Dataflow BATCH_ENVIRONMENT_MAJOR_VERSION to 6

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify.
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @aaltay

LGTM, thank you.

Changes Unknown when pulling 4cee177c8a798015189c1be5061164b22c3ccfe3 on charlesccychen:update-major-version into ** on apache:master**.


playable page is down

The project page https://github.com/ncase/polygons links to the playable page http://ncase.me/polygons, but the latter has been unresponsive for some time now.

The main site appears to be up and working for me.

You are correct; I see it now, but I had to disconnect from my universities VPN to do so.  Thanks.

I guess this issue can be closed then?

I agree it can be closed.  I'll still try to learn why this is happening and post here just for the informational value if I can figure it out.


Re nest parallelize

Re nest the pipelines so things will parallelize

blocked by qiime2/qiime2#790

Looks like this plugin is slower when adding --parallel!
Is this PR related or should I open a new issue?

@ebolyen and @gregcaporaso now that qiime2/qiime2#790 is merged this ought to work as intended

@Oddant1, fyi I rebased to test with the most recent environment files - that's what the force-commits are, in case you're wondering.


Provide import quick-fixes for things from internal modules

Currently the import quick fix suggestions only suggest modules available on Hoogle. This should be expanded to search other modules in the same package/project/folder being worked on.
This may involve having to typecheck other modules in the background so we can get the exported symbols

Could we somehow get the list of exposed-modules and other-modules from the canal file?

I believe this is already working, for example I'm able to complete Haskell.Ide.Engine.* modules when working with the hie codebase. Is there anything missing here?

@Lorenzo sorry should have been more clear, this is for the missing variable/symbol suggestions that suggest modules to import


fix(Tooltip single target): Support for multiple target elements

It fixes #1185
Here is stackblitz demo with fix - https://stackblitz.com/edit/react-k8rkzv

[x] Bug fix
[ ] New feature
[ ] Chore
[ ] Breaking change
[x] There is an open issue which this change addresses
[x] I have read the CONTRIBUTING document.
[x] My commits follow the Git Commit Guidelines
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.

[ ] I have updated the documentation accordingly.

[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Up


Search again buttons should be hidden when search is in progress

Fixes #105

This also sort of fixes #107 since it hides the buttons at the same time the scanning progress circle animation starts.


feat(cli): Add tail flag to the log command

fixes #3004
Motivation
When a pod has a lot of log entries, it can be convenient to be able to get only the last entries of the logs.
Modifications:

Add an int flag --tail to get the specified amount of last lines of the logs. The default value is -1 indicating that all lines of the logs are expected like the previous default behavior

Release Note
feat(cli): Add tail flag to the log command

@squakez yes, the default behavior remains the same

The 3 violations raised are not related to these changes https://github.com/apache/camel-k/pull/3589/files#annotation_4463020346

The only failure is on TestNativeIntegrations/automatic_rollout_deployment_from_fast-jar_to_native_kit which is a known failing test

BTW how do you proceed in camel-k in terms of PR validation? I mean: How many PR approvals are needed? Who merges it?

I believe we are on hold for merging pull reqs which add a feature and are not related to stabilising e2e just because we've been in the process of releasing 1.10.0 and moving to 1.11.0-SNAPSHOT. When everything is set up, we could resume merging them.

I believe we are on hold for merging pull reqs which add a feature and are not related to stabilising e2e just because we've been in the process of releasing 1.10.0 and moving to 1.11.0-SNAPSHOT. When everything is set up, we could resume merging them.

I see it is a specific situation, but what about the process in a normal situation?

I see it is a specific situation, but what about the process in a normal situation?

@essobedo Basically, once a PR has enough approvals and all CI checks have passed anyone with "Member" right to this repo could merge it. Practically, @squakez, I, and @oscerd would take care of them for most cases these days.

By the way since @essobedo is a committer, he could do it by himself.

By the way since @essobedo is a committer, he could do it by himself.

Yes I know but I'm still new to the project and I'm not a Go expert that's why I'm asking

By the way since @essobedo is a committer, he could do it by himself.

Yes I know but I'm still new to the project and I'm not a Go expert that's why I'm asking

No worries. If the PR was approved by some committer and checks passed, then, you can merge yourself. I usually wait 24/48h more to see any other review. Also, I prefer to apply "rebase and merge" strategy in order to have a linear story of the commits on the main branch.

Looks all good. Let's merge.


Clarify the way of classifying metered connection

The original explanation is all about user, but not the API's behaviour, which is uncertain and not helpful.
According to this post on SO, cellular network and hotspot based on cellular network are explicitly metered.

Thanks, @sunnylqm.
According to Android API http://developer.android.com/reference/android/net/ConnectivityManager.html isActiveNetworkMetered has a broader meaning including monetary, performance and battery limitations.
For most cases your change makes sense but I think we should keep the generic description in place.

I'll close for now but feel free to reopen it if you keep the original definition of metered as well


add support signMessage in nightly wallet

Description
Please include a summary of the wallet and how users can install it. Be sure to take a look at our guide on Custom Wallets to understand core concepts when integrating with Wallet Selector.
Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

Hi @Pawel-Szydlo ,
Thank you for raising this PR.
Can you please change the branch from main to dev?
We are getting an error message while testing signMessage.
The message is shown after we approve the Signature Request.
Please take a look at the image below for the information.

Hi @Pawel-Szydlo , Thank you for raising this PR.
Can you please change the branch from main to dev?
We are getting an error message while testing signMessage. The message is shown after we approve the Signature Request.
Please take a look at the image below for the information.

Hi, a new version of the nightly extension came out this morning, 1.0.4 to be exact. This version has support for signMessage, if you could test it again.
Sorry for the confusion.

Hi @Pawel-Szydlo
While testing out the signMessage we have some other requirements for the implementation.
The UI that shows Signature Request for the user to approve the signMessage, should contain
more information about what this action/function does.
I will attach a similar UI ( Meteor Wallet ) below to give you an example:

Also, the dApp developers should be able to call signMessage even if they are not connected or signedIn.

@erditkurteshiSQA

What requirements do you have for the implementation of signMessage?

Nightly has its own simple, clear user interface. It is additionally clear which site the wallet is linked to.

Also, the dApp developers should be able to call signMessage even if they are not connected or signedIn.
please explain how this would work?

Please explain how this would work?
Because if the user hasn't selected the nightly extension from the wallet list, how does the dApp developer know that they can call the signMessage method from the nightly version? So, nightly users first have to connect to the page in question, and then the methods signAndSendTransaction, signMessage, etc., can be called

@Pawel-Szydlo ,
The idea of signMessage is for dApps to be sign messages without having to create Limited Access Keys (function-call access-key).
In a dApp you can programmatically call the nightly signMessage without using the modal:
(async () => {
  const wallet = await selector.wallet("nightly");
  const message = "test message for verification";
  let nonceArray: Uint8Array = new Uint8Array(32);
  nonceArray = crypto.getRandomValues(nonceArray);
  const nonce = Buffer.from(nonceArray);
  const recipient = "myapp.com";

  await wallet.signMessage({ message, recipient, nonce });
})();

Also for more information, I will attach the link below:
https://github.com/near/NEPs/blob/master/neps/nep-0413.md

@erditkurteshiSQA
I understand.
Nightly does not use limited access keys, so users do not need NEAR to sign the message.
The connection to the page is for security purposes, preventing unwanted pages from sending requests to the Nightly wallet. Therefore, Nightly users first connect to the page (browser tab), and then methods can be called from the Nightly interface.

@Pawel-Szydlo
Can you first check if the wallet is connected then sign the message:
if (!_state.wallet.connected) {
 await _state.wallet.connect();
}

and then call signMessage method.

@erditkurteshiSQA
good idea, changes added to pr.


#47 parse format section

This PR adds functionality to parse the new format section in the markdown files. The parsed data will also be written when generating the markdown files but I didn't know where to put the parsed information in the generated model.ttl. With this PR the CI in the spdx-3-model should be fixed.
part of #47

Actually, if both metadata and format sections are ulista, the code may be the same.

@zvr I added another commit and rewrote the code so that the metadata and format section use the same parsing methods. Please have another look, it would be good if we can get this PR merged soon to fix the CI in the spdx-3-model to catch possible other formatting errors there.


Adding colorado.edu

Adding colorado.edu domain (University of Colorado - Boulder)

@marcofanti I am sorry, the domain colorado.edu is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Pack at https://education.github.com/discount_requests/application. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Use relative path for git commands in install cmd

Motivation
Paths to git commands when running forge install and forge init are broken on Windows.
Solution
Make the paths relative to the root of the project where relevant, which fixes the issue - keeping this as a draft until I have tested it on Windows
Remaining tasks

[ ] Test on Windows
[ ] Check exit code of git commands so they don't fail silently

Fixes #621 and fixes #631

Also fixes #631 correct?

Yeah, it would

uh, why this pr fix #631 ?

I linked to the wrong PR


Hide command output?

How do I hide the output of Keybindings shortcut?
ie. ctrl+[ to go back a directory, when I hit ctrl+[ it displays cd .. then runs it, how do I hide this ?

Sorry, that's not possible—or at least, not in a way that doesn't potentially screw up your prompt. Such functionality would be possible to implement as part of your prompt, but that's out of scope for this plugin.

I discovered that there might be a way to do this, after all.

good to here :)

All right, it's there now. 🙂 Please see the Readme for more info.


make image pull policy for server and client to be configurable

currently the imagePullPolicy for opal server and client is hardcoded to IfNotPresent
changed it to be configurable

Notes to reviewers:

did not make the postgresql image pull policy to be configurable as i didn't see a use case for it, for server and client, the use case is if someone is building their own opal server/client images with latest tag, kubernetes wouldn't pull latest image as image pull policy is hard coded to IfNotPresent
don't see anyone needing that for the postgresql image

Hey @thilak009 ,
Looks great, thanks for the contribution 🥇


Infowindow support on markerwith label component

Getting Help
Before doing this, did you:

Read the documentation
Read the source code

You can get someone's help in three ways:

Ask on StackOverflow with a google-maps tag or use react-google-maps as a keyword
Ask in the chat room
Create a Pull Request with your solutions to your problem

Please, be noted, no one, I mean, no one, is obligated to help you in ANY means. Your time is valuable, so does our contributors. Don't waste our time posting questions like “how do I do X with React-Google-Maps” and “my code doesn't work”. This is not the primary purpose of the issue tracker. Don't abuse.

You don't help.


build error

lutter/.pub-cache/hosted/pub.flutter-io.cn/nima-1.0.3/lib/nima.dart:119:18: Error:
The argument type 'Int32List' can't be assigned to the parameter type 'Uint16List'.
 - 'Int32List' is from 'dart:typed_data'.
 - 'Uint16List' is from 'dart:typed_data'.
Try changing the type of the parameter, or casting the argument to 'Uint16List'.
        indices: _indices, textureCoordinates: _uvBuffer);
                 ^
file:///Users/jintian/development/flutter/.pub-cache/hosted/pub.flutter-io.cn/palette_generator-0.1.1/lib/palette_gene
rator.dart:188:29: Error: The argument type 'void Function(ImageInfo, bool)' can't be assigned to the parameter type
'ImageStreamListener'.
 - 'ImageInfo' is from 'package:flutter/src/painting/image_stream.dart'
 ('file:///Users/jintian/development/flutter/packages/flutter/lib/src/painting/image_stream.dart').
 - 'ImageStreamListener' is from 'package:flutter/src/painting/image_stream.dart'
 ('file:///Users/jintian/development/flutter/packages/flutter/lib/src/painting/image_stream.dart').
Try changing the type of the parameter, or casting the argument to 'ImageStreamListener'.
      stream.removeListener(imageListener);
                            ^
file:///Users/jintian/development/flutter/.pub-cache/hosted/pub.flutter-io.cn/palette_generator-0.1.1/lib/palette_gene
rator.dart:194:31: Error: The argument type 'void Function(ImageInfo, bool)' can't be assigned to the parameter type
'ImageStreamListener'.
 - 'ImageInfo' is from 'package:flutter/src/painting/image_stream.dart'
 ('file:///Users/jintian/development/flutter/packages/flutter/lib/src/painting/image_stream.dart').
 - 'ImageStreamListener' is from 'package:flutter/src/painting/image_stream.dart'
 ('file:///Users/jintian/development/flutter/packages/flutter/lib/src/painting/image_stream.dart').
Try changing the type of the parameter, or casting the argument to 'ImageStreamListener'.
        stream.removeListener(imageListener);
                              ^

Should be fixed in nima 1.0.4!


fix name of index file

Looks like this is named incorrectly. The website linked in the readme gets a 404, presumably because of this.

So this is part of the training. Nevermind then


Need to improve the test coverage on rueidiscompat

Currently, we have poor test coverage on rueidiscompat:

Details of which lines are not covered can be found here: https://app.codecov.io/gh/redis/rueidis/tree/main/rueidiscompat
We hope that we can improve the coverage to be close to 100%.

Hi @mingdaoy, thank you! You can just send PRs whenever you make any small improvements to the coverage.


libkb: strip leading whitespace for two sig ancient payloads before checking for '{'

fixes keybase id nougad
See https://github.com/keybase/keybase-issues/issues/3543

(only need one review whoever grabs it first)

i am not received reward


Vagrant: put the /srv/zulip stuff into the home directory

When I did vagrant ssh I got confused because I couldn't find the Zulip repository/files. I had to cd to /srv/zulip. Tim thinks maybe we should just set up the Vagrant installation so that stuff is in the user's homedir and the user doesn't have to look for it.

Thanks for remembering to put this in the bug tracker!  I definitely want to make this happen :)

We can also just create a symlink in homedir pointing to /srv/zulip. Or we can shift zulip to home directory and make a symlink in /srv/zulip. We have hardcoded /srv/zulip at many places and simply moving it can introduce bugs.

Yeah, that's a great idea.
-Tim Abbott (mobile)
On Jun 7, 2016 10:44 PM, "Eklavya Sharma"<EMAIL_ADDRESS>wrote:

We can also just create a symlink in homedir pointing to /srv/zulip. Or
we can shift zulip to home directory and make a symlink in /srv/zulip. We
have hardcoded /srv/zulip at many places and simply moving it can
introduce bugs.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/zulip/zulip/issues/922#issuecomment-224493865, or mute
the thread
https://github.com/notifications/unsubscribe/ACnm2iAi-ZAonrzd7xz9T7-2jsGpURQ2ks5qJlacgaJpZM4IuQkI
.

I can work on this.

Cool, sounds good!

Possible alternates:

shift zulip to home directory and symlink to /srv/zulip
let the code remain in /srv/zulip and symlink zulip folder in home directory
mode zulip to home directory and replace /srv/zulip everywhere.

What should I opt?

Let's go with option 2: leaving the code as-in and making a zulip -> /srv/zulip symlink in the home directory.  I think you probably want to use something like ln -nsf to create the symlink.


Add option to change several duty slots at once

Hey @indocomsoft! We're about to handle this issue. Can you help clarify a little bit on it? Is it on the "change current duties" functionality or something else?

I can't quite remember exactly, but I think so! Like selecting several slots and changing them for one particular person. Not sure how the UI/UX would look for that though

I added the "select all" option when members try to drop their duties which are longer or equal to 2 hours. They can select all their duty slots of the week by clicking the "select all" checkbox. Thus, they do not need to click the duty slots one by one when dropping duties.


Introduce a reactive rules engine

This issue is to explore the design of a possible reactive rules engine implementation. If you are reading this, please share your thoughts with comments or by upvoting/downvoting the issue.
The current implementations of the RulesEngine interface are proactive: we know facts upfront, and we act on them (by applying some rules). The idea of a reactive rules engine is the following: we don't know facts upfront, so we will react on them (by applying some rules). Based on this idea, a reactive rules engine would be a listener/subscriber to a stream of facts happening in time and could be triggered in two ways:
1. On a time window (ie a time frame like the last hour or the last 30 minutes)

2. On a space window (ie fact frame like for example the last 10 facts, regardless of when they occur)

This design makes it possible to implement use cases like:

If X and Y happened in the last 3 minutes, then execute action Z
if the last 10 known facts contain F1 and F2, then execute action A

A real word example would be: If a customer places more than 3 orders during the same week, then offer her/him a 10% discount on the next order. In this example the stream of facts would be the orders placed customers. This design requires the introduction of two interfaces to correctly represent the new rules engine type hierarchy:

The current DefaultRulesEngine and InferenceRulesEngine would be updated to implement ProactiveRulesEngine. The ReactiveRulesEngine should be designed with a reactive API that is library agnostic (probably by using the java.util.concurrent.Flow APIs) so that it can be implemented with any reactive streams library (like RxJava, Reactor, etc). It could be even implemented for other streaming technologies like Apache Kafka: I'm thinking of a KafkaReactiveRulesEngine that could be implemented as a Kafka consumer of a stream of facts with a configurable max.poll.records to take only x facts from the topic and apply rules on them.
I believe this would be a nice addition to Easy Rules and would open a whole new range of use cases that can be implemented using this new feature.
NB: Please note that the project is in maintenance mode, and I won't be able to implement this feature in the near future or review any contribution related to it. I'm just writing my thoughts down here to gather feedback from the community. So please share your thoughts with comments or by upvoting/downvoting the issue. Thank you upfront!

In the past two years, I had done a lot of experiments around Facts and Rules in the company's projects. Especially about the generation or acquisition of Facts objects.  We all know Fact can be a POJO that represents an event,
We often encounter requirements like this: if event x has occurred twice in the past 5 minutes, then perform action Y.
We used the rule engine based on this principle: Although we don’t know when the event will happen, the event must be defined in advance by us. If we don’t know anything about the event (don’t know what the event holds Data), we have no way to set rules for such events and expect it to run correctly.
Therefore, we need professional data modeling analysts to help us in model design, These models contain a lot of fields. Finally, these models can be generated using batch (Hadoop) or streaming computing framework (Kafka streaming). However, we need to face such a situation, if the batch processing or streaming is not completed within the expected time, these Models will be discarded (not executed by the engine), resulting in our results are not so accurate.
It is a very useful idea if we had ReactiveRulesEngine. as you say, it will open a whole new range of use cases.

And a new feature, run fact2 after fact1 has been executed for 5 minutes. The interval between     fact1 end of run     and  fact2 start run  is 5min.


Does TiCDC support data capture before CDC cluster was started?

Before asking a question, make sure you have

[X] Searched existing Stack Overflow questions
[X] Googled your question
[X] Searched open and closed GitHub issues
[X] Read the documentation

What is your question?
I need clarification with TiCDC, does TiCDC support data capture before CDC cluster was started? If I have a cluster with data in it, is TiCDC able to replicate existing data into the new DB or do I need another tool?

--start-ts: Specifies the starting TSO of the changefeed. From this TSO, the TiCDC cluster starts pulling data. The default value is the current time.

How do I get the TSO for the past? select master status; only returns the TSO of now.


Monolog throws an error on read-only filesystems

We have a webserver where the filesystem is mounted readonly. The log dir is a symlink to a read-write part of the filesystem. Monolog, as of 1.15.0, tries to create the dir on every log action and raises an exception.
Line 51 of StreamHandler.php is the issue: https://github.com/Seldaek/monolog/blob/6dd76b944d42d7b27bb3d09cd0f6fa88a80817f5/src/Monolog/Handler/StreamHandler.php#L51
If I comment that out, it works. Could a check as to whether the folder already exists be added?
Thanks!

This ticket is wrong. On looking into it further, it turns out that https://github.com/Flynsarmy/Slim-Monolog/blob/master/Log/MonologWriter.php#L108 creates an unused monolog instance with an invalid log dir. Since the mkdir seems to be happen on instantiation, it fails.
I think this is probably a monolog bug. I don't expect to have a directory be created when I instantiate a class. I would expect it to occur on first log.

I took a shot at fixing this in https://github.com/Seldaek/monolog/pull/607. Tests all pass locally.

I fixed this in Flynsarmy/Slim-Monolog.
Please do a composer update.
In version 1.0.1 you can call the constructor with an extra parameter, set it false, so it won't create an unused monolog instance.

I'd posit that the strategy here (whether or not my specific patch), is better than adding a merge param to Slim-Monolog. The situation where a developer might want the default $name parameter for MonologWriter but passes in a StreamHandler isn't handled. Also, if you did want to work around this in Slim-Monolog, you could just always merge the $name param since there are no side-effects to creating a string, and only set a default StreamHandler if one isn't passed. The $merge param strategy asks the developer to determine something that should probably be an internal detail.
The fundamental issue here is that instantiating a class shouldn't modify the system environment. This is an unexpected side-effect as demonstrated by the former assumption in Slim-Monolog that it could instantiate a throwaway class without negative repercussions. I think that this assumption is sound. Writing to the log is expected to change the system environment, and creating the directory at that point is expected.
Does this make sense?
Thanks for your work on this!

Will, thank you for your reply and advices!  My fix was a quick, maybe not entirely thought out solution...
I'm looking forward for your fix in Monolog.


Message ID from messages with files.

Hi!
files_upload_v2 function does not return ts and thread_ts arguments. For my program, I need to know parent and child message ID. I tried to use permalink in blocks argument in postMessage function, but I get invalid_blocks error.
And for some reason there are two absolutely identical file objects in my response (I don’t know, maybe that’s how it should be).
Question: How can I get the parent and child message IDs for a message with attached files/images?
Thanks!
  response = slack_client.files_upload_v2(
           file = image_path
       )

{
  "ok": true,
  "files": [
    {
      "id": "F0808JW9XMY",
      "created":<PHONE_NUMBER>,
      "timestamp":<PHONE_NUMBER>,
      "name": "61GsgAsIIKL._AC_SL1400_.jpg",
      "title": "Uploaded file",
      "mimetype": "",
      "filetype": "",
      "pretty_type": "",
      "user": "U07RZCBPMUL",
      "user_team": "T0AJK4YAU",
      "editable": false,
      "size": 55409,
      "mode": "hosted",
      "is_external": false,
      "external_type": "",
      "is_public": false,
      "public_url_shared": false,
      "display_as_bot": false,
      "username": "",
      "url_private": "https://files.slack.com/files-pri/T0AJK4YAU-F0808JW9XMY/61gsgasiikl._ac_sl1400_.jpg",
      "url_private_download": "https://files.slack.com/files-pri/T0AJK4YAU-F0808JW9XMY/download/61gsgasiikl._ac_sl1400_.jpg",
      "media_display_type": "unknown",
      "permalink": "https://kievhacklab.slack.com/files/U07RZCBPMUL/F0808JW9XMY/61gsgasiikl._ac_sl1400_.jpg",
      "permalink_public": "https://slack-files.com/T0AJK4YAU-F0808JW9XMY-8c4adea2dd",
      "comments_count": 0,
      "is_starred": false,
      "shares": {},
      "channels": [],
      "groups": [],
      "ims": [],
      "has_more_shares": false,
      "has_rich_preview": false,
      "file_access": "visible"
    }
  ],
  "file": {
    "id": "F0808JW9XMY",
    "created":<PHONE_NUMBER>,
    "timestamp":<PHONE_NUMBER>,
    "name": "61GsgAsIIKL._AC_SL1400_.jpg",
    "title": "Uploaded file",
    "mimetype": "",
    "filetype": "",
    "pretty_type": "",
    "user": "U07RZCBPMUL",
    "user_team": "T0AJK4YAU",
    "editable": false,
    "size": 55409,
    "mode": "hosted",
    "is_external": false,
    "external_type": "",
    "is_public": false,
    "public_url_shared": false,
    "display_as_bot": false,
    "username": "",
    "url_private": "https://files.slack.com/files-pri/T0AJK4YAU-F0808JW9XMY/61gsgasiikl._ac_sl1400_.jpg",
    "url_private_download": "https://files.slack.com/files-pri/T0AJK4YAU-F0808JW9XMY/download/61gsgasiikl._ac_sl1400_.jpg",
    "media_display_type": "unknown",
    "permalink": "https://kievhacklab.slack.com/files/U07RZCBPMUL/F0808JW9XMY/61gsgasiikl._ac_sl1400_.jpg",
    "permalink_public": "https://slack-files.com/T0AJK4YAU-F0808JW9XMY-8c4adea2dd",
    "comments_count": 0,
    "is_starred": false,
    "shares": {},
    "channels": [],
    "groups": [],
    "ims": [],
    "has_more_shares": false,
    "has_rich_preview": false,
    "file_access": "visible"
  }
}

blocks = [
        {
            "type": "image",
            "title": {
                "type": "plain_text",
                "text": "Please enjoy this photo of a kitten"
            },
            "block_id": "image4",
            "image_url": response['file']['permalink'],
            "alt_text": "An incredibly cute kitten."
        }
    ]

response = slack_client.chat_postMessage(
            channel=channel,
            text="Hello",
            blocks=blocks
        )

Error sending message to Slack: invalid_blocks

Hi @denikryt, thank you for taking the time to write in here. I understand this could be confusing but the v2 method asynchronously completes the uploading process. So, if your code needs to know the message ts, your code needs to poll files.info API method until "shares" property shows up. See also:

https://github.com/slackapi/python-slack-sdk/issues/1329#issuecomment-1430589611
https://github.com/slackapi/python-slack-sdk/issues/1461#issuecomment-1960432170

I hope this clarifies things.

Thank you very much for such a quick response!


Chore: bump vulnerable dependencies

When running in TomEE I found that there was a compatibility issue with Jersey. I fixed that.


Provided screen factories causing unnecessary overhead when opening mod menu screen

I'm currently using the API to provide config screens for other mods. As stated by the docs, I need to return a map of all the mods ids and their associated config screen factory. However ModMenuApi#getProvidedConfigScreenFactories is being called for every mod and library, and this means my provider is running 46 times at minimum (Fabric, it's libraries, and Mod Menu). My suggestion would be to call getProvidedConfigScreenFactories from each provider once, combine them and then request from that map.

Good point, I am trying to figure out why I didn't do it like that to begin with, but can't come up with anything. I'll do what you suggested.

So, it turns out this breaks a ton of mods that use config providers because they relied on the fact that it wasn't accessed until later... I may have to revert this until those mods have fixed their implementations. Affected mods so far seem to be Architectury and owo-lib

I think I can fix it by delaying the accessing of the fields until the first time a config screen is requested, but it may cause a brief lag spike. At least it would only be one time instead of every time though.

Lazy initialize on the first time opening ModsScreen, that should fix compatibility.

yup, that's basically what I did in 6.2.1


Added brand colors and fonts to tailwind config

[ ] No discrepancies across browsers. Eg. chrome vs safari
[ ] Pages pass W3C Validation for HTML (https://validator.w3.org)
[ ] CSS (https://jigsaw.w3.org/css-validator/)
[ ] Google sign-in is operational
[ ] Accessibility functions
[ ] Pages can scale without distorting page
[ ] No dead links
[ ] Navbar is consistent across the site
[ ] Pages are screen-reader accessible
[ ] Contrast meets standards for accessibility
[ ] Pages are keyboard accessible
[ ] Passes WAVE Evaluation Tool chrome extension

Opening a PR so we have something to reference when we talk. The name of the styling classes will be changed, just not sure what to name them. We should go over how we want to enforce these rules.