Branch test tag command

Codecov Report

Merging #323 (c8f6304) into master (0991d40) will decrease coverage by 0.58%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #323      +/-   ##
============================================
- Coverage     62.58%   62.00%   -0.59%
- Complexity      451      455       +4
============================================
  Files            90       90
  Lines          1748     1779      +31
  Branches        184      189       +5
============================================
+ Hits           1094     1103       +9
- Misses          599      625      +26
+ Partials         55       51       -4

Impacted Files
Coverage Δ

...c/main/java/seedu/unify/commons/core/Messages.java
0.00% <ø> (ø)

...in/java/seedu/unify/logic/commands/TagCommand.java
86.36% <ø> (+9.09%)
:arrow_up:

src/main/java/seedu/unify/model/task/Task.java
82.53% <0.00%> (-13.76%)
:arrow_down:

src/main/java/seedu/unify/ui/MainWindow.java
0.00% <0.00%> (ø)

...rc/main/java/seedu/unify/logic/CommandHistory.java
0.00% <0.00%> (ø)

...n/java/seedu/unify/logic/commands/ShowCommand.java
0.00% <0.00%> (ø)

...va/seedu/unify/logic/parser/ShowCommandParser.java
0.00% <0.00%> (ø)

...va/seedu/unify/logic/parser/SortCommandParser.java
0.00% <0.00%> (ø)

...u/unify/model/task/TaskContainsStatePredicate.java

...va/seedu/unify/model/task/TaskIsDonePredicate.java
50.00% <0.00%> (ø)

... and 4 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 0991d40...c8f6304. Read the comment docs.


Decouple Distinct Products from OpenSearch Versions

Is your feature request related to a problem? Please describe.
Some projects release completely independent of OpenSearch. Both Data Prepper and Logstash OSS with OpenSearch Output Plugin are on distinct releases. I believe the OpenSearch CLI is as well.
The way the download page works for these projects lumps them in with a given version of OpenSearch. This can be confusing because it gives a subtle indication that they are released together.
It is also not immediately apparent how to get an older version. Do you as a user go to the previous version link? Or the artifacts archive. (You must go to the artifacts archive)
Describe the solution you'd like
I'd like these products to be decoupled from the current version of OpenSearch.
I'd like to see the Downloads page treat each project equally.
This probably means that there is no longer a concept of a "Previous Version" link at all. Instead, the page would list OpenSearch, Logstash, Data Prepper, OpenSearch CLI as products. You could download the latest version from that page. But, each project would have an option to view and download any old version.
Below is a rough sketch of how it might look:
OpenSearch
OpenSearch is a distributed search and analytics engine based on Apache Lucene. After adding your data to OpenSearch, you can perform full-text searches on it with all of the features you might expect: search by field, search multiple indices, boost fields, rank results by score, sort results by field, and aggregate results.
Platform : Linux (existing drop-down)
Package : x64 / .tar.gz  (existing drop-down)
Download (existing button)
All Versions (new link to all versions)
OpenSearch Dashboards
OpenSearch Dashboards is the default visualization tool for data in OpenSearch. It also serves as a user interface for many of the OpenSearch plugins, including security, alerting, Index State Management, SQL, and more.
Platform : Linux (existing drop-down)
Package : x64 / .tar.gz  (existing drop-down)
Download (existing button)
All Versions (new link to all versions)
Data Prepper
Data Prepper is a component of the OpenSearch project that accepts, filters, transforms, enriches, and routes data at scale. Distributions without a bundled JDK are also available in the artifacts directory.
Platform : Docker (existing drop-down)
Package : x64 / Docker  (existing drop-down)
Download (existing button)
All Versions (new link to all versions)

@dlvenable I see. Yeah, what you are saying makes sense.
This functionality exists in a very raw form currently (https://opensearch.org/artifacts#data-prepper) but what were you looking for would be more styled - something like a table? Something like:

Package
Artifacts

Data Prepper 1.0.0
[x64 Docker]

Data Prepper 1.1.0
[x64 Linux Tar] [x64 Docker]

This would have it's own page/url and would be linked from every version page (download) under each component block

Yes, I think a table like you have will work.

would be linked from every version page (download) under each component block

If I follow you yes. I think the current Data Prepper download section (block?) in the current downloads page would have this link.


Make widget headlines clearer

Kareem thinks "Average Commercial Permit Lifespan in the Last 30 Days" is a little unclear to folks who have no context of seeing this dashboard. How can we simplify this copy?

I don't know how much we can clear it up with headline language, but we could examine adding a "what's this?" to the widgets.

So talking with Donna, we learned that "Lifespan" isn't as important to folks as the time it takes from Permit Application to Permit Issuance. That's because once it's up to the owner, county cares less whether the owner/contractor takes 2 days or 2 years.
Because of that we may re-emphasize the lifespan widget. I'll close this BUT open up a slightly different ticket that clarifies that.


Schedule isn't being rememebred for refuel station

Description
The schedule for station with the refuel icon in it isn't being applied to other trains in that same group. I am having to manually go into every train and tell them all to wait 5 seconds of inactivity before leaving the refuel station.
Reproduction
No response

This actually isn't a bug. Train Schedule Signals uses the refuel icon from TCS, so if you name a station using that icon, those wait conditions are controlled by train schedule signals. Train groups intentionally doesn't sync stations controlled by TSS because TSS will do it for you.

I saw that the station listed in the train was disabled for some so it would jump over it, but without a wait condition I observe the train just flying pass the station and not stopping to get fuel. Is it meant to apply its own wait condition as well? It is the wait conditions that aren’t being synced while the schedule to go to that station does indeed appear in all trains. Talking to some other people, I think I may have communicated that badly in the original post.

The wait conditions are controlled by train schedule signals. If you want to use the refuel station normally, then you need to remove that mod.

I recommend reading the tips & tricks entries for trains schedule signals. Honestly, I don't think that that mod is really necessary with train groups, and it just makes stations more difficult to configure.

I wasn't using the icons from Train Schedule Signals when I created this issue. I was only using the refuel icon from Train Control Signals. I understand that the Train Schedule Signals in a train name will cause it to control the wait conditions of that station, but besides that mod being installed, it played no relevance in the issue, so from my understanding TrainGroup should have been syncing my wait condition.


$(Spotify:currentAlbumArt) appears to not show correctly on button

it appears that instead of showing the actual album art it shows the link to the album art, is this a bug or am i doing something wrong?

I believe it's intended to work like that since it's really hard to and or not possible to draw an image to the button with companion.

It is possible and not that hard to draw an image (there is a png64 property in feedbacks), but this module hasnt implemented it yet

https://github.com/bitfocus/companion/blob/7bf2464b6d46901cac12c91b093aa943dec50d7a/webui/src/Buttons/EditButton/ButtonStyleConfig.jsx This? im assuming, is it possible to add it to my streamdeck now, and if so, how?

Yeah that, but not easily. It will require some code changes in this module to define and implement a new feedback type to do this.
Some work will also need to be done in the module to download the image, maybe resize it and then to encode it correctly and output it from the feedback.
Its fairly straightforward, but not that simple

Take a look at the openweather-rest module.  I built that one to improve my skills with a REST API.
There is a feedback for 'Current Condition Icon' The module loads these from the online API at start.

Hi there. Any chances of implementing the dynamic background image?

Unsure if this is just because I'm using a 3.0 beta of companion but my current work around for this is to use the "Generic: HTTP Requests" Module.
Using the "http: Image from URL" feedback on a button and putting the "$(internal:custom_SpotifyAlbumArt)" variable as the url.
Hope this helps.

Ah damn, nice one! The image from url-feedback is indeed not a feature in 2.4.2. Nice to see it being possible in 3.0. Thanks a lot.

Oh oops, I implemented that in part to support this and forgot to say here...
But yes, this is possible in 3.0

Yeah haha, I've had it set that way for a while but was going through the Issues to look for another issue and thought I would just post in case anyone was looking for a solution!

Is it a working feature already in 3.0.0? I've downloaded it but I can't seem to find it anywhere :/

Okay, so just by using the http requests module, I thought there would be something added next to upload of the png file, thanks a lot ;)


Initialize logger

Context
Create a utility package for our logger code.
What has been done

Initialize logger code
Write tests
Add coverage
Set up npm scripts for running tests, including coverage
Write README

How to test

Run npm test
Test coverage at 100%

Notes
node-logger is taken on npm. Therefore the package name is enrise-logger. What do you think we should do here?

Changes Unknown when pulling 7016f954066618747c923905ae39b90cb3299001 on feature/initial-commit into ** on master**.

🛎

Changes Unknown when pulling f3958baf1f86ccc8f2873b3f421bde4d9ec58616 on feature/initial-commit into ** on master**.


Improve the runtime frames yield depth size to increase dynamically

Description:
ATM max frames stack size is 256 but we can start with a small number and increase it dynamically on demand.

Since we copy stack to heap only on yield lazily, is should be possible to start with no frames array at all.

Following code will break due to maximum frame yeild depth size
`ballerina
public function main() {
bar(0);
}
function bar(int x) {
if (x == 200) {
io:print(x);
runtime:sleep(1);
} else {
bar(x + 1);
}
}
`

Fixed with https://github.com/ballerina-platform/ballerina-lang/pull/35034.


Add some xhyve hardware requirements

I ran into this issue this evening trying to get dlite running on my older Mac Pro and my MacBook Air.  It looks like xhyve requires hypervisor support and some older hardware does not support this even if you are running OSX 10.10.5: http://apple.stackexchange.com/questions/220323/what-are-the-hardware-requirements-for-hypervisor-framework
So you can save yourself some time and run this command and verify that you can run hypervisor and thus dlite.
sysctl kern.hv_support
kern.hv_support: 0 == sadness
kern.hv_support: 1 == joy
I am in double sadness land because my late-2010 MacBook Air and my 2009 MacPro do not support hypervisor.  :(  Maybe the README.md file could be updated to reflect this requirement.

good find, i'll add that note to the readme


fix: Changed the hostname to :: to enable ipv6 compatibility

:tada: This PR is included in version 2.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


ContextMenu in Editor/Title only available with Tabs active

VSCode Version:1.3.0
OS Version: Windows 10

Using the newly available API
Steps to Reproduce:

Add a menu - editor/title entry to package.json

    "menus": {
        "editor/title": [
            {
                "command": "bookmarks.list",
                "group": "bookmarks"
            }
        ]
     }

Turn off showTabs ("workbench.editor.showTabs": false)

The new context menu command is not available. But if you turn on showTabs ("workbench.editor.showTabs": true), the command appears.
Thanks in advance

The actions then show in the context menu of the tab... Not nice and an active UX discussion. @bpasero for finding the duplicate of this

@jrieken @alefragnani I am not sure this is the same issue, maybe can you clarify? you say that the actions are not showing up in the context menu if you disable tabs, so this would indicate that the issue is not related to us not showing actions when tabs are enabled.
Today:

tabs disabled: all actions show up in the right hand corner
tabs enabled: all actions show up on the tab itself

We plan to change this though to always show all actions int he right hand corner.

Well, I see the commands in both cases. Unsure what the issue then is...

There is an edge case where we show a context menu when tabs are disabled and you right click on the title:

This might cause some confusion because that menu does not contain the actions contributed.
All this will get much better with: https://github.com/Microsoft/vscode/issues/7666

The issue was related to this edge case. In fact I didn't notice that button in the right hand corner, sorry.
Thank you both for clarifying :+1:


Create CODE_OF_CONDUCT.md

updated code of of conduct
fixes issues #95

@prajwaldp223 Closing this PR. Submit each separate PR one for CODE OF CONDUCT and other for Pull Request template

can u mark this under gssoc?


Triple slash comments in prisma schema file not working

Bug description
Problem adding comments to the prisma schema file so that it will show in GraphQL playground docs
As per the docs:
https://www.prisma.io/docs/reference/tools-and-interfaces/prisma-schema/prisma-schema-file
Triple slash comments are the way to show comments inside GraphQL playground docs however, inside my VS Code editor this is not working.
On save, the triple slashes are being automatically spaced
from /// to // / instead
How to reproduce
Steps to reproduce the behavior:

Try to add a triple slash comment in the prisma schema file

Expected behavior
Triple slash comments show on the GraphQL playground docs.
Prisma information

Environment & setup

OS: MacOS
Database: PostgreSQL
Prisma version: 2.0.0-beta.2
Node.js version: 12.16.1

I can confirm the problem. This is actually known problem as we have removed the concept of documentation comments /// in the parser so that we can enable automatic conversion of implcity relations to explict relations.

@pantharshit00 Thanks for the quick reply!
I see. Is there any scope to support documentation comments?
If not, perhaps update the Prisma documentation so that the triple slash comments are removed?

This is fixed in latest master.


Build process stops at rendering chunks

Describe the bug
I'm not really sure if that's Vite or Sveltekit issue, so i've decided to post here.
My project works fine in dev mode, but i cannot build it. I've tried multiple adapters, node versions, sveltekit versions and few other things. Every time build halts at 'rendering chunks'.

I've created really minimal repo to reproduce the bug. Probably it's something that more experienced developer will solve in a blink of an eye (and here i am 4th hour trying to fix it xD).
Let me know if I could provide any more details. I couldn't find a way to get build logs (i dont even know if it's saved anywhere, since build process never really finishes, it just halts xD)
Reproduction
https://github.com/nxy7/test
I've created repo reproducing the issue. Clone it, install dependencies and try to build it.
Logs
No response
System Info
win11

Severity
blocking all usage of SvelteKit
Additional Information
No response

May be related to https://github.com/vitejs/vite/issues/5597 (thought the issue lacks a repro). It's likely a vite issue, though I haven't took a deeper look at it yet.

I can confirm that

Solves the problem, although it's not the best long term solution. Thanks for linking the other issue @bluwy , I should be able to at least upload it now :-)

vite looks like a horrible absurd bundler considering the number of issues and how childish and absurd those issues are. This vite nonsense is ruining the reputation of sveltekit. It's the only thing preventing the release of sveltekit. What a shame!

vite looks like a horrible absurd bundler considering the number of issues and how childish and absurd those issues are. This vite nonsense is ruining the reputation of sveltekit. It's the only thing preventing the release of sveltekit. What a shame!

To me vite seems great (even if there are still plenty of bugs). Give it more time to mature, both svelte and vite are amazing pieces of software :P

If vite.build.minify: false fixes it, then it's almost certainly a Vite issue, so I will go ahead and close this issue since there's nothing that can be done about it in this repo. I recommend following up in the Vite repo. It could help them to have a reproducible test case

1 ) delete node_modules ( rm -rf node_modules ).
2 ) delete package-lock.json ( rm -rf package-lock.json ).
3 ) Removed unused packages in package.json file (Check all the packages to see if there is any where used. If not, remove it in package.json)
4 ) npm install and npm run build.

I resolved this by cloning again the repository. Im using Linux Mint 64 bits.
System Info
System:
    OS: Linux 5.15 Linux Mint 21.1 (Vera)
    CPU: (8) x64 Intel(R) Core(TM) i5-9300H CPU @ 2.40GHz
    Memory: 10.17 GB / 15.51 GB
    Container: Yes
    Shell: 5.8.1 - /usr/bin/zsh
  Binaries:
    Node: 18.14.0 - ~/.nvm/versions/node/v18.14.0/bin/node
    npm: 9.3.1 - ~/.nvm/versions/node/v18.14.0/bin/npm
  Browsers:
    Chrome: 113.0.5668.0
    Firefox: 108.0.1
  npmPackages:
    @sveltejs/adapter-vercel: workspace:^ => 2.4.1
    @sveltejs/amp: workspace:^ => 1.0.2
    @sveltejs/kit: workspace:^ => 1.14.0
    @sveltejs/site-kit: ^3.1.0 => 3.1.0
    svelte: ^3.56.0 => 3.56.0
    vite: ^4.2.0 => 4.2.0


Issue 14 fix non determinism bb

I made three changes to fix non-deterministic behavior.

When finding the midpoint, check if one of the values is null. If so, take the other as the midpoint. This avoids having more than one NaN in the split list.
Partition by featureIdx when finding initialThresholds. This avoids the need to manage frequencies across boundaries.
In ManyValuesFinder removed the call to coalesce. Changing the partitioning within the while loop was causing different splits to be generated. If it is necessary for some reason, an alternative will need to be found.

I ran the tests a couple of dozen times both from within the IDE and on the cmd line on two different computers. They seem to always return consistent results now.

First of all, nice work Barry! Thanks again.
Now I understand why range partitioner causes non-determinism when generating boundary points. If this partitioner was determinist, I would choose the original solution despite it includes some non-boundary points. Your solution is far more exact and accurate than mine. The problem here is that your new partitioner conveys a new shuffling phase and much more information need to be sent across the network (as you include a long index for each distinct value). It can deteriorate the performance of MDLP when facing big datasets. Have you tried some tests with big datasets?
About the coalesce call, it is performed to allow the maximum parallelism when processing "big" features with too much boundary points. Why different partitioning affects entropy computations? The only requirement in this function is that the elements are ordered.

I have not done much performance testing with big data, but plan to soon. I don't have baseline numbers from before these changes, but will have some numbers with these changes soon. It's too bad about the additional shuffling, but I don't know of an alternative. I don't really know why the coalese call was influencing entropy calculations.
for the Run MDLPD on all columns in srvRequest40000 data (label = churned, maxBins = 100, maxByPart = 10000) test

Ok, we wait for the performance results. I'll try to figure out what is happening with coalesce. I still think that a non-determinism MDLP could fit as an approximate big data solution. Original design will be much faster than your solution for sure, and may be more attractive to practitioners because of this reason . We can manage to have two versions of DMDLP, but maybe it could be problematic.

Perhaps we can have a "approximate" option that allows it to run with the faster but less accurate aproach. Spark does that for other things.
I created some unit tests in a branch for a 100k and 10m row dataset. The 10m one is probably to big to add to github since its about 1G on disk.
When I was doing some profiling, I noticed that we can get some performance improvement from using !isEmpty instead of calling count in a couple of places.
Here are some results on my 4 core laptop
10m rows, 4 continuous columns discretized together.

maxBins 50 maxByPart 50000 default par defualt - 25 min
maxBins 50 maxByPart 5000  default par defualt - 53 min

maxBins 50 maxByPart 100000 default par 4 - 19 min
maxBins 50 maxByPart 10000  default par 4 - 39 min

after replacing coutn with !isEmpty  (not quite as much speedup as I had hoped for)

maxBins 50 maxByPart 100000 default par 4 - 17 min
maxBins 50 maxByPart 10000  default par 4 - 35 min

100k rows

maxBins = 50 maxByPart = 1000  default  parallelism=4  57s
maxBins = 50 maxByPart = 10000  default parallelism=4  32s (29 after using !isEmpty instead of count)
maxBins = 50 maxByPart = 20000 default  parallelism=4  10s
maxBins = 50 maxByPart = 100000 default parallelism=4  10s

Did you find a way to reintroduce coalesce into ManyValuesThresholdFinder without bringing back the non-determinism?
When running MDLP on the 1G serrverX dataset on machine with 24 cores. Below is a summary of the timings.
Since maxByPart here is 10,000, and it should really be 100,000 for better performance, these numbers are only useful for understanding where bottlenecks might be.
6s for collect at MDLPDiscretizer: 47
1s for isEmpty at MDLPDiscretizer: 76
33s for sortByKey at MDLPDiscretizer: 153  (160/160 tasks)
This can take a very long time and even run out of memory if the target has a huge disctinct values. I think this is a spark bug see
https://issues.apache.org/jira/browse/SPARK-18181
Here it has 5.
9s for countByValue at InitialThreshodsFinder: 118
2s for zipWithIndex at InitialThreshodsFinder: 61
13s for countByKey at MDLPDiscretizer: 181
There are a whole lot of jobs for count, min, and runJob in ManyValuesThreshold finder
Most of these entries have 1/1 (5 skipped) in the stages column and
1045/1045 (1285 skipped) in the Tasks (for all stages) column
All of the count jobs seem to take 1, 2 or 3 seconds - and there are a lot of them. The total is
120 seconds for count at ManyValuesThreshold: 54 and 125  -
This is a significant bottleneck and can be sped up with using !isEmpty instead of count.
All of the min jobs take 2 or 3 seconds. Total is
28 seconds for min at ManyValuesThreshold: 125
All of the runJob jobs take 1 or 2 seconds. The total is
51 seconds for runJob at ManyValuesThreshold: 83
1s for collect at MDLPDiscretizer.scala:191

I can not reproduce the coalesce problem yet in my ScalaIDE. I've tried several times but the result is always the same. The coalesce generates the same partitions. I think coalesce can generate different partitions as the parent partitions for cands is RangePartitioner. However I cannot understand why it affects the final results. Does it output a different threshold in each trial or it just generate different partitions? Could you add some prints near coalesce in order to know where is the problem
Yes, I remember that I add some isEmpty to avoid these empty and useless counts.

I'm surprised that you cannot reproduce the coalesce problem by just running the MDLPDiscretizerBigSuite. I get different throeshold results depending on whether or not I do the coalesce. I don't know why the thresholds in the results are different. There is a small chance it might be related to https://issues.apache.org/jira/browse/SPARK-14393. The other odd thing I noticed, is that using the coalesce actually makes things run slower (in the cases I looked at). In one case, it went from 29 seconds to 52 seconds. I don't fully understand it, but it seems odd to be doing all those coalesces inside the loop. I added a print statement in the loop and see many lines like this:
coalescing to 2 from 27 nCands = 10549
coalescing to 1 from 27 nCands = 124
coalescing to 1 from 27 nCands = 2932
:
Why not just keep the data on the original partitions? Did you do any tests to see if the coalescing gives a performance advantage? My esperiments so far indicates that it does not.

I did some rough profiling of Spark-MDLP performance using a 9.8 million row dataset running on my laptop. The dataset is too large to checkin, but I put a version at https://app.box.com/s/mrepegkdjv7im0iq1f97slp8c88h6rwj and will add a commented out test for it.
It takes about 3.9 minutes to bin 4 continuous columns if the target has 5 values.
But if I bin only 1 column with that label, it takes 2.3 minutes.
It takes only 2 minutes to to bin 4 continuous columns with a different label which has only 3 values.
I think this was faster because there was more correlation with the other columns. Also this case did not print nearly managed memory leak lines.
Below is a rough profiling of what is happening during the test, and how much time is spent in each part.
reading and cleaning - 8 seconds
fit. about to call train
get the labeled input data (1 seconds)
its cached, now calling train (training is running MDLP on all continuous cols)
in MDLPDiscretizer.runAll
num labels = 5          (10 seconds )
labels = 0.0 -> 0, 1.0 -> 1, 2.0 -> 2, 3.0 -> 4, 4.0 -> 3
Compute classDistrib =  (5 seconds)
classDistrib = 0 -> 1953125, 1 -> 1953125, 2 -> 1953125, 3 -> 1953125, 4 -> 1953125
now sorting distinct values. partitions = 15  {
start sortByKey. numpartitions = 30
16/11/16 10:40:18 ERROR Executor: Managed memory leak detected; size = 78727902 bytes, TID = 152
This looks like a spark memory leak bug. See https://issues.apache.org/jira/browse/SPARK-18181
This shows the distribution of label values for each split.
((3,64.65005),5, 0, 0)
((2,351.14984),9, 0, 0)
((3,55.925964),0, 1, 0)
((3,88.87153),1, 0, 0)
((3,51.176464),0, 1, 0)
((3,81.87806),2, 0, 0)
((3,60.07228),0, 2, 0)
((2,325.05286),0, 8, 0)
((3,62.296955),0, 1, 0)
((3,87.25891),1, 0, 0) .......
    repartition.  numparts = 30
    done sortByKey     (~50 seconds for just for sortByKey)
    16/11/16 10:40:38 ERROR Executor: Managed memory leak detected; size = 38036528 bytes, TID = 213

} done sorting distinct values in          (63 seconds - expensive)
about to find initial thresholds
about to create featureInfo list
featureInfo = (0,5,0,5,1,0), (1,5,5,5,1,1), (2,1632699,10,96042,17,2), (3,5249987,1632709,99057,53,19)
totalParts = 72
points parts = 30
partitionPts parts = 72 count = 6,882,696
done computing initial thresholds        (26 seconds)
about to find all thresholds
the first value of part 0 is 30.0
the first value of part 3 is 320.82028
:
the first value of part 71 is 94.310616
16/11/16 10:41:55 ERROR Executor: Managed memory leak detected; size = 5251642 bytes, TID = 2865
This line is repeated about 60 times
done finding MDLP thresholds      (8 - 117 seconds depending on target - number of values and how it correlates)
Now returning model
To summarize. Finding the unique values and sorting can take more than half the time. Finding the initial thresholds cold take up to about 1/4 of the time. Finally, finding the thresholds using MDLP can take anywhere from 5% to 60% depending on a lot of data dependent factors.

Coalesce operation is done to homogenize the number of points in each partition. In the worst case the previous filter may only select points in a single partition. Coalesce is needed to take advantage of the maximum level of parallelism in clusters. I think what you show here is that the time spent on coalescing is much longer than the time gained using the maximum level of parallelism vs. the previous partitioning scheme. For a small number of elements (27 ncands), coalescing is heavier than maintaining the original scheme, however, I remember that the threshold for "big" features was set to higher value (10,000?).
About the determinism, maybe we can add up a new parameter that allows us to switch between versions: the original one and your version (non-deterministic). Exact version can be useful if we want accurate results, whereas my version can be used when the performance acquires a higher relevance.
What do you think? This solution can make the merge possible.

Yes, I think a new param to allow switching would be good, but I don't know when I will have a chance to add it. The non-deterministic version is very hard to have test cases for.

Yes, it's true. But I don't want to loose the non-deterministic version because it's much faster I think. Maybe we can offer the non-deterministic version without test cases, or to consider good a test if there only exist a small differences between the output and the deterministic results. In other words, just to relax the conditions to pass a test.
What do you think?

Barry, Spark release 2.1.0 is out. Coudl you try your testings in order to know if our problem has been fixed or not, please? As you know, I'm not been able to reproduce the problem in my computer.
Here you can see the notes:
https://spark.apache.org/releases/spark-release-2-1-0.html

I am running on spark 2.1.0 right now using the mdlp/soark 2.x branch that I created. Yes, it works and is deterministic. It was not any changes in 2.x that fixed the non-determinism problem though.

Do you mean that you try out non-deterministic version and it isn't working?

I have not tried the non-deterministic version on 2.1.


Map and Fold/Reduce should each have their own sections

They are very common FP jargon, worth their own sections I think. Also, the "Foldable" can just be part of the Fold/Reduce.
I'll add a PR to address this soon.

Haven't seen a pr. Closing since it's been years.


Align on exact PLG price and limits for Cody GA

Reviewing with leadership today on costs

Done. https://docs.google.com/presentation/d/1E38rmMfGr9BOis-pDjaRhIbzjzZQ4_7cejwxkBMBy2s/edit#slide=id.g2951b123226_0_206


C# reference: Consolidated jump statements

Fixes #10746
✔ Internal preview: Jump statements (C# reference)

@BillWagner thank you. I think I've addressed all of your feedback, please review the new commits.
@IEvangelist thank you for your review too.


Gift Entry: Use lightning-lookup

Critical Changes
Changes

Lookup fields on Gift Entry have been enhanced to share more platform features. When searching for results in a lookup field you will now be presented with the option to "Show all results for [search term]". Clicking this link will show all matching results in table format with additional fields visible. The fields visible are controlled by the user's currently assigned Search Layout for the given SObject.

Issues Closed
Community Ideas Delivered
New Metadata
Deleted Metadata

@bdvorachek This is very cool! Do we need to do any cleanup of the old lookups here, or is that framework still in use for RecordTypes?

@bdvorachek This is very cool! Do we need to do any cleanup of the old lookups here, or is that framework still in use for RecordTypes?

Yep, this is correct. I think a move to a dropdown/picklist for recordtypes would make sense in the future.

@bdvorachek looks good from my perspective!


Java Exception Handling(Try Catch) [hackerrank]

Issue Id you have worked upon -
#123
Briefly explain your program logic -
Here we take two variables as input then we give different inputs like number and then when they create exception we catch them through catch block.
Screenshots(Attach 2 screenshots of your own input and output) -

Checklist:
Eg - If your code follow the below guidelines. Kindly change [] to [x]
All the conditions should be fulfilled for considering your code for merging -
[] I have mentioned the question as comment in my solution file.
[] My code follows the guidelines of this project.
[] I have performed a self-review of my own code.
[] I have commented my code.
[] My code gives the correct output.
[] I confirm that I have not copied the code from anywhere. In case its found that I have copied I can be banned from the repository.
[] I affirm that I strictly follow contributing guidelines and code of conduct.

@neelshah2409,
Kindly fill the checklist by agreeing to it.
Comment your code as well as add question as first comment to the solution file

@neelshah2409, Kindly fill the checklist by agreeing to it. Comment your code as well as add question as first comment to the solution file

@SarthakKeshari Sir I had made the changes ,Please see it .


Remove duplicated attribute

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 77.46%. Comparing base (a5669e4) to head (a1b372e).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #147   +/-   ##
=======================================
  Coverage   77.46%   77.46%
=======================================
  Files          11       11
  Lines        2232     2232
=======================================
  Hits         1729     1729
  Misses        503      503

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Blog App: Add authorization rules

In this PR, I added authorization rules.
Changes

Installed the CanCanCan gem.
Added a role column to the users table.
Implemented Post deletion by an authorized user and admin.
Implemented Comment deletion by an authorized user and admin.

NOTE: My coding partner @serengia has left the program. That is why there is no contribution from him in this PR.

Thank you for the feedback and for approving my PR, @Yerimah.


1.20.2 support

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

self explanatory gui grashes when trying to open in 1.20.2

Maybe wait until I've ported it? Might help...

Maybe wait until I've ported it? Might help...

still waiting

Just gunna add another week onto the timeline seeing as you seem to think that I don't have a life and I have nothing better to do that port a small Minecraft mod...

Just gunna add another week onto the timeline seeing as you seem to think that I don't have a life and I have nothing better to do that port a small Minecraft mod...

alright can I just port it


[Ember-Migration] Join section component created

Issue
Example : Closes #383
What is the change?
Created join-section component  just below the What is Real Dev Squad section, wrote tests for the same and added necessary media queries for the component.
Is Development Tested?

[x] Yes
[ ] No

Before / After Change Screenshots
Before :

After :

@satyam73 I have made the changes you suggested please review them. Thanks.

@rohan09-raj I have made all the changes your suggested please review them.


How I can install it?

Hey!
How I can install this bot?
Im not a programmer...
This may work with a diffrent game? like battlerite or cs ?

I haven't updated this bot in ever so Im not sure how well it works with the latest DiscordPy updates. Some of the stuff is hardcoded but they can be easily changed for any other game/discord server's settings. Email me on my personal email if you want to set it up somewhere?


Knex Migrations for Users, Roles and User_Roles

Create the Knex Migrations in order to create the DB schema for the following tables:
Table roles
Table users
Table user_roles
Check the data model and build the tables and relations here https://github.com/HackYourFuture-CPH/fp-class12#-database-model-diagram

Please check the new DB Diagram on the README file (changes made on PR https://github.com/HackYourFuture-CPH/fp-class12/pull/72)

Ok


refactor(vehicle_velocity_converter)!: prefix package and namespace with autoware

Description
This PR will add the autoware prefix to vehicle_velocity_converter package.
Related links
Related PRs

aip_launch: https://github.com/tier4/aip_launcher/pull/319
autoware-documentation: https://github.com/autowarefoundation/autoware-documentation/pull/618
autoware_tools: https://github.com/autowarefoundation/autoware_tools/pull/124
sample_sensor_kit_launch: https://github.com/autowarefoundation/sample_sensor_kit_launch/pull/107

Parent Issue:

https://github.com/autowarefoundation/autoware/issues/4569

How was this PR tested?
Notes for reviewers
None.
Interface changes
None.

Effects on system behavior
None.

@KeisukeShima @taikitanaka3 @TakaHoribe @takayuki5168 @tkimura4
Please review this pr.

@a-maumau Ready to merge?!

@shmpwk all related PR is approved so if there is no further suggestion, it should be OK.

@shmpwk sorry, could you wait a moment? seems some of repo ( like https://github.com/autowarefoundation/single_lidar_sensor_kit_launch) also needs to be fixed.

I created a PR to single_lidar_sensor_kit_launch.
After the review of the PR in single_lidar_sensor_kit_launch, it should be OK this time.

@a-maumau Thank you so much! Let's merge all PRs.

@a-maumau
Srr, could you resolve conflicts?

@shmpwk I have fixed the conflict!

Thank you so much!


Server not responding/blank screen

Hi,
I have installed the latest 4.5.0 habridge, i can access the server but nothing works. I also do not get the update security setting dialog. Nothing happens when i click HAbridge or login/logout.

The server version is also not shown which tells something is wrong in the compiled java files.

The previous version 4.3.1 works.
Could not locate http://xxxxxxxxxxx/js/angular.js specified in source map http://xxxxxxxxxxxx/js/angular.min.js.map.
Error pops up when developer tools enabled in IE and chrome

Could not locate http://xxxxxxxxxxx/js/angular.js specified in source map http://xxxxxxxxxxxx/js/angular.min.js.map.
Error pops up when developer tools enabled in IE and chrome

I get the same problem  - works fine in Chrome

I get the error in ie and chrome as well.
On Apr 14, 2017 10:07 PM, "declanhadley"<EMAIL_ADDRESS>wrote:
I get the same problem - works fine in Chrome
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bwssytems/ha-bridge/issues/600#issuecomment-294143232,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARiQRP0xNEA_mn88hOJLcGJdI9hO4cn3ks5rv2FvgaJpZM4M8I51
.

Never could get the bridge to Load completely in IE
Chrome did work  but now doesn't with the new 4.5.0 version.
As well Windows Edge use to work but now doesn't
I use Avast SafeZone browser for all bridge views on my PC it worked fine prior to 4.5.0 and still works.
The issue with  angular.jar and IE is a well documented issue however never seen anything with it and Chrome or Edge

That is an unusual error. Will need to see why the javascript map file is pointing that way.

Hi I've just updated to windows creator version (latest w10) and can now see the bridge fine in Edge

I noticed there was another update for Windows 10 however this still didn't fix the issue with the Edge
opening the developer tools I get 4 file not found errors.
Name	Protocol	Method	Result	Content type	Received	Time	Initiator
http://localhost/js/angular-resource.js	HTTP	GET	404	text/html	337 B	13.97 ms
http://localhost/js/angular.js	HTTP	GET	404	text/html	328 B	1.25 ms
http://localhost/js/angular-route.js	HTTP	GET	404	text/html	334 B	3.08 ms
http://localhost/js/angular-sanitize.js	HTTP	GET	404	text/html	337 B	3.99 ms

Windows Creator edition of 10  fixes the issues with the edge browser as declan2511 stated.
However it doesn't fix the issue with IE

I think this is due to the use of "async function"[1] in app.js which requires a very modern browser[2]:
[1] https://github.com/bwssytems/ha-bridge/blob/master/src/main/resources/public/scripts/app.js#L80
[2] http://caniuse.com/#feat=async-functions

I am having the same issue on Ubuntu using Firefox using 4.5.0.  It still works fine with 4.3.1.

Same problem here. Older versions of Firefox and IE that work with 4.3.1 show only the top bar with 4.5.0.  Latest version of Firefox Nightly works fine.

So, it seems it is the async functionality that is used for authentication check.


Per sebb - fix anomolous backslash

How is this possibly working, today?

It is looking for an exact match with "." - it is essentially an extension stripper. The single escape works, but per sebb that's anomalous.

Oh. I realized what is going on. It's just how Python parses string constants. \. is not a recognized escape sequence, so Python keeps it as two characters. For clarity, the \\. form is better.


Firewall Issues and Suggestions

While customizing the role for our own use we found few things which I think will enhance this fantastic ansible role.

3.6.2 should be applied at the end i.e. after 3.6.5. Setting default policy to DROP all unmatched traffic will close all incoming ports before we have a chance to open required ports like SSH.

3.6.2 uses ansible iptables 'JUMP' parameter to create rule to DROP traffic, but should actually use 'POLICY' parameter to set the default policy. With 'JUMP' the default policy remains ACCEPT.

3.6.5 should have a rule to accept incoming new SSH connections. The benchmark suggest that we find all currently open ports and create rule to allow them, so allowing incoming SSH connection should be okay and is required for ansible to work. In strict scenario, we may want to use a variable to provide the ansible task with an allowed IP range(s).

Please let me know your view/comments.

Implemented by PR #39


Introduce simple GraphQL schema testing

This PR introduces:

Jest as a test framework.
A few simple sample tests against Infura and public blockchain data.
Separates schema initialisation from HTTP server init.

[cb55]


Is there a way to disable double CMD shortcut?

I'm really enjoying the action bar but is there a way to disable the CMD+CMD shortcut?
When I hit CMD+something twice, in the second time the action bar appears. This could be because I use Keyboard Maestro to simulate/remap  actions, for example Save All when I hit CMD+S. (Yes I could change the shortcut in Xcode but with Keyboard Maestro all my changes are kept in sync between the Macs I use).
(As a power user of App Code this plugin is one of the few that makes Xcode nice to use again)

Hi there - glad to know you're enjoying the plugin!
As for your question: currently there's no settings UI, but there is a non-documented way of doing what you're asking:
Open the plugin's resources folder (normally under ~/Library/Application Support/Developer/Shared/Xcode/Plug-ins/XCActionBar.xcplugin/Contents/Resources) and find the XCActionBarConfiguration.plist file. Open it and look for the following directives:

Shortcuts/XCActionBarHotKey
Shortcuts/XCActionBarRepeatLastActionHotKey

The first one configures the hot key for presenting/dismissing the action bar. The other one configures the hot key to repeat the last executed action (by default it's mapped to OPTION+OPTION).
To disable them, just change the XCHotKeyListenerRepeatCount value to 0 and restart Xcode.
If you're only interested in disabling the CMD+CMD shortcut, then you only need to change the value for Shortcuts/XCActionBarHotKey.
Let me know if this solved it for you.
Cheers.

That worked. Thanks!


Delete Keylogger keylogs

Hi Would really like to have a feature to delete logs after I'm done reading them.
Thanks

You mean deleting the logs on the remote computer, right? As a small work-around you can use the file manager and navigate to C:\Users\<name>\AppData\Roaming and delete the directory called Logs.

Yes. I was doing that previously however it would be pretty convenient if we had a button under "Refresh logs" so we don't have to jump into the file manager every time the logs get crazy.


Runeense

Adds Lovense product integration for xp drops and levels

Don't close PRs and open new ones, just push to the same branch.
Also, why? (In an existential sense)

Don't close PRs and open new ones, just push to the same branch.

Understood, my apologies. Will keep that in mind for future updates.

Also, why? (In an existential sense)

The cosmos told me it was necessary.


about volume readonly state

Describe the bug
When a volume is marked by readonly, either by volume becomes full, or by user using shell command, or API mark volume readonly. Then if seaweed restarted, the volume may become writable again.
System Setup

List the command line to start "weed master", "weed volume", "weed filer", "weed s3", "weed mount".
OS version
output of weed version
version 30GB 3.72  linux amd64
if using filer, show the content of filer.toml

Expected behavior
Once the volume is marked readonly, it will stay readonly.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
I think there are many possible scenarios to lervage the readonly state of volume, but the primary scenario we want to use readonly is for the remote storage tier.
Say we have a s3 backend storage, in first phase we want to copy the volume dat to s3 storage (for example, the s3 archival tier), to make sure data has better redundency, while we can keep the data warm in local, so any read will hit the local volume dat.
When the data becomes colder (no read any more), or the local storage becomes full, we will upload the volume dat file, and seaweed will remove the volume dat from local, to release the space here. This is the second phase.
When we do the copy in first phase, we want to make sure the data in volume dat does not change anymore, that why we mark the volume readonly before the copy. However, since seaweed does not persist the readonly state, if seaweed restarts (by upgrade), or system reboots, the volume will become writable again, and new file may write to the volume again, which makes the local copy and remote copy is not consistent.
So here I want to discuss the option to make seaweed persist the readonly state, I can think of 2 approaches:

Change the volume dat file to readonly, and then seaweed has to see that state, and update the in memory state accordingly.
Remember the readonly state in vif file, and handle the in memory state accordingly.

added a pr https://github.com/seaweedfs/seaweedfs/pull/5977


Automate the CHANGELOG.md via GH automatically generated release notes.

I'd like to improve the way we are maintaining the CHANGELOG.md in Volto.
At kitconcept, we've been trying lately this approach, with good results:
https://docs.github.com/en/repositories/releasing-projects-on-github/automatically-generated-release-notes
Goals:

the contributor only have to take care of the name of the PR and the categorisation via labels
the maintainer does not have to take care of merge conflicts

Also, the new branching policy, and subsequent PRs backporting should benefit from it.

I'm +1, we use something similar at EEA. It's a positive change. I think sometimes there's more context in the PR (description, comments, etc), so it would be nice if we also get links to the PR (I think we'll do, right)?

Yes, the release notes generated by github include a link to the PR and the author of the PR.

@tiberiuichim it does it, indeed.
It's something like:
* Title (#ISSUE_ID) by @AUTHOR in https://github.com/URL/ISSUE

@jensens asking you as label keeper :)
We have labels marking the PRs as "bug" and "feature".

'01 type: bug'
'03 type: feature (plip)'
'04 type: enhancement'

We'll need a couple more if we want this:

Breaking
Documentation
and an explicit "Internal" would be great too.

What do you think?

I do have a concern about adding another way to handle changelogs. It makes it harder for Plone contributors to do the right thing if they work in multiple repositories. Is there a way to use towncrier with a node project?

One can always execute something in a shell. We'll have to install it locally first in a venv... then a convenience Makefile command, maybe...

I like the way of prefixing the commits with fix, feat, etc. and render the changelog using auto-changelog red-turtle folks are also using it in some addons like volto-form-block may be @cekk or @pnicolli can also comment.

So, during the Volto Team meeting we discussed this. We heard feedback from 'commitlint' + release-it from @pnicolli and from kitconcept using GHA automated release notes. We shared opinions from Towncrier too.
The general opinion was that Towncrier could be the best bet on this since

uses a detached from the flow (in a file) way of define the final message in the changelog
when backporting this could be a very interesting feature
is used already in the Plone ecosystem, so contributors do not have to learn another flow
the downside of creating an issue first to assign it to the news item might be overcame easily, by defining the PR itself or other unique number

as downside, the maintainer(s) need to install Towncrier before the release in an Python env. An automatic way of executing and extracting it when using release-it might be handy... maybe using pipx is a good fit for this...
Since we are already been biten by this when back porting,


python2: fix build with clang 16 on x86_64-darwin

Description of changes
Fixes Python 2 and dependent packages (found running nixpkgs-review) failing to build on staging-next #263535.

Python 2: https://hydra.nixos.org/build/239425695

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 266358 run on x86_64-darwin 1

20 packages marked as broken and skipped:

cassandra_3_0
cassandra_3_11
manta
neard
neard.dev
oni2
platypus
python2
python27
python27Full
python2Full
resholve
resholve.dist
rubyPackages.libv8
rubyPackages_2_7.libv8
rubyPackages_3_0.libv8
rubyPackages_3_1.libv8
rubyPackages_3_2.libv8
rubyPackages_3_3.libv8
storm

14 packages built:

bash-preexec
bashup-events32
bashup-events44
bats
git-ftp
ix
lftp
locate-dominating-file
mons
msmtp
packcc
pdf2odt
shunit2
yadm


update from googleapis

Specifically to get the updated dlp documentation, but several other things came along too.

I'm unable to merge this change (not a project owner). Could I be added?

Done (I think).


Prompt user to select default kernel

Should solve #497, though I don't have a repro unfortunately
TODO: tests

Can't repro either, which might be a good thing 🤷 ?
However, I'm seeing a lot of these:
ERR Cannot get cell UUID for non-code cell!

Ah!

Which also explains why the "Stop Task" button disappeared from the status bar.

ERR Cannot get cell UUID for non-code cell!

This I think is a separate issue. I pushed a fix anyway though

Can't repro either, which might be a good thing 🤷 ?

I didn't eliminate the issue, only made it like before, where you would get prompted to select a default kernel.
I don't know where vscode stores the "default kernel" setting, but we would need to reset that to test it properly.

Thanks for fixing the unrelated bug.
There's a chance that the kernel selection logic is something the VS Code platform improved. I remember researching it and people saying that this "select kernel first" issue couldn't be avoided in "VS Code API user-land".


Add RHEL support

Change:

Piggyback off of CentOS support and add RHEL support.
Note that tailscale provides a repo for RHEL 8, but not RHEL 7. Thus,
we use the RHEL 8 repo if we're on RHEL 8, otherwise we default to
the CentOS one.

Signed-off-by: Rick Elrod<EMAIL_ADDRESS>

Thanks for this PR! I've manually verified with redhat/ubi8 that this role was indeed failing on RedHat distros... I hadn't checked before!
This PR does make the role work for RHEL8 but I cannot verify it for RHEL7 (via a container, at least) because I don't see that redhat publishes one.
@relrod , if you can follow https://github.com/artis3n/ansible-role-tailscale#development-and-contributing and add a TAILSCALE_CI_KEY to your fork, that should be sufficient to get the CI tests passing and we can merge this. I created  https://github.com/artis3n/ansible-role-tailscale/issues/159 for a longer-term followup to ensure RHEL is tested separately from CentOS in the CI suite.

Oh, and rebase from main as I made improvements to the CI testing to speed it up this weekend.

Definitely want to get this support merged in. If I don't hear back by Jan 14, I'll fork this back into a branch in this repo (preserving your contributions) and ensure CI passes.

@artis3n I just tried adding the key but it didn't seem to get picked up. I just force pushed to try and retrigger CI, but if it fails again it might be easiest if you fork it and use the existing key. I rebased against main.

:/ Sorry for the hassle. Not sure how else to offer CI for contributors at this time. Let me fork

Thanks, it's probably something I missed, but I did add it, I was surprised it didn't get picked up.
Let me know if there's anything else you need from me. This role is awesome, I appreciate you keeping it going!
I'll close this out. Thanks again!

No problem! If the secret is set up the same as this, then I'm lost


Javadoc style guide: deprecations

There is a @Deprecated annotation, and there is also a @deprecated javadoc tag (that predates the annotation).
There should be a guideline on which one to use.
  /** Sets the foo.
   * @param foo the foo to be set.
   * @deprecated Use {@code setFooBar} instead.*/
  @Deprecated(forRemoval = true)
  void setFoo(Object foo) {
    // ...
  }

IMHO both must be required.

The annotation provides an idiomatic way of indicating deprecations. It has runtime retention and optional since/forRemoval attributes.
The javadoc tag allows an optional message providing additional information on the deprecation (e.g. whether the method was replaced, or whether a feature does not work as intended).

I agree that both should be present. Here's some additional information to the reasons Javier commented before.


Make fill order notification wording less confusing issue #246

potentially WIP

https://github.com/AugurProject/augur/issues/246


redcarpet Dependency Error

When first trying to use this theme locally, I recieved the following error:
/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/bundler-1.10.6/lib/bundler/resolver.rb:347:in `block in verify_gemfile_dependencies_are_found!': Could not find gem 'jekyll (~> 2.4.0) ruby' in any of the gem sources listed in your Gemfile or available on this machine. (Bundler::GemNotFound)

I used gem install jekyll -v 2.4.0 to downgrade my current version of Jekyll.
In spite of having installed redcarpet-3.3.2, I recieve the following error when trying to serve:
  Dependency Error: Yikes! It looks like you don't have redcarpet or one of its dependencies installed. In order to use Jekyll as currently configured, you'll need to install this gem. The full error message from Ruby is: 'cannot load such file -- redcarpet' If you run into trouble, you can find helpful resources at http://jekyllrb.com/help/!
             ERROR: YOUR SITE COULD NOT BE BUILT:
                    ------------------------------------
                    redcarpet

Completed uninstalled both versions of Jekyll and reinstalled version 2.4.0 and now works as intended.


Move to using OpenSCMDataFrame

Is your feature request related to a problem? Please describe.
At the moment our interface is a bit clunky to use. We could make it much easier by moving to using OpenSCMDataFrame's in the back end instead.
Describe the solution you'd like
Move to using OpenSCMDataFrame throughout pymagicc.
Describe alternatives you've considered
Create our own implementation.
Additional context
Add any other context or screenshots about the feature request here.

superceded by #295


having a problem with=>  rails g rspec:install

What Ruby, Rails and RSpec versions are you using?
Ruby version:
Rails version:
Rspec version:

Observed behaviour

Expected behaviour

Can you provide an example app?

livechat.zip
this the project i am working on Please

This isn't an RSpec issue, but a problem in your C:/Sites/livechat/config/application.rb, comment out the line with load_defaults on it, (line 95). It seems that something is wrong with your Rails install.


Missing ruby 2.5.0p0 on rbenv install --list / ubuntu

I was able to install ruby 2.5.0p0 on my development machine (MacOS) but on the server (Ubuntu) rbenv install --list doesn't show the option 2.5.0p0. It shows 2.5.0-dev instead. Now I normally try to keep my dev/test and prod environments exactly the same so I was wondering if reporting an issue here will help?
Feel free to close this if it has been reported earlier.

If you don't see a new version of Ruby that you expect, it's because your ruby-build install is too old. Check ruby-build --version. Upgrade ruby-build and you'll get the new Ruby versions!


Possible unreported error from GetArithmeticTwap

System information
Osmosis version: v12.0.0
OS & Version: OSX
Commit hash: aa727b5bf25019d561da5d6f82dd9a3892fda2be
Expected behaviour
Assume t0 < t1 < t2, where these are time values
Say the store has the following two TWAP records:

{time: t0, lastErrorTime: t0} (erroneous)
{time: t2, lastErrorTime: t0}

And say we want to GetArithmeticTwap with startTime=t1 and endTime=t2. When GetArithmeticTwap is called, the getInterpolatedRecord function will:

Obtain the record at or before t1, which is {time: t0, lastErrorTime: t0}
Interpolate this record to {time: t1, lastErrorTime: t0} (time updated)
Obtain the record at or before t2, which is {time: t2, lastErrorTime: t0}
Interpolate this record to {time: t2, lastErrorTime: t0}

My understanding here is that the first record should still be considered erroneous even after it was interpolated, since the interpolation is based on erroneous values. The interpolated records are fed into computeArithmeticTwap, which I expect to return an error due to the use of interpolated records based on erroneous values.
Actual behaviour
The computeArithmeticTwap function errors only if endRecord.LastErrorTime (i.e. t0) is after or equal to startRecord.Time (i.e. t1). In our case, t0 < t1 and so the function does not error.
Steps to reproduce the behaviour
Add this failing test to x/twap/api_test.go:
func (s *TestSuite) TestGetArithmeticTwap_ExpectError() {
	t0 := baseTime
	t1 := tPlusOne
	t2 := tPlusOne.Add(time.Second)

	recordsToSet := []types.TwapRecord{withLastErrTime(baseRecord, t0)}
	// note: baseRecord.Time == baseRecord.LastErrTime, so baseRecord is erroneous

	ctxTime := tPlusOneMin
	input := makeSimpleTwapInput(t1, t2, baseQuoteAB)

	s.SetupTest()
	s.preSetRecords(recordsToSet)
	s.Ctx = s.Ctx.WithBlockTime(ctxTime)

	_, err := s.twapkeeper.GetArithmeticTwap(s.Ctx, input.poolId,
		input.quoteAssetDenom, input.baseAssetDenom,
		input.startTime, input.endTime)

	s.Require().Error(err)
}

Oh wow, your definitely right. Great catch!!
We should add a catch for this situation here: https://github.com/osmosis-labs/osmosis/blob/3c7eab90146396d7cb5ee7f4580244be78bf44f6/x/twap/logic.go#L196-L210
	record, err := k.getRecordAtOrBeforeTime(ctx, poolId, t, assetA, assetB)
	if err != nil {
		return types.TwapRecord{}, err
	}
	if record.Time.Equal(record.LastErrTime) {
		record.LastErrTime = t
	}

Lets get a fix for this into v13.
Would you be interested in PR'ing a fix?

Yep I can PR a fix. However, just out of curiosity, would it make more sense to modify the recordWithUpdatedAccumulators function instead, given that it is responsible for updating the record's time. It might as well update the error time as well?
https://github.com/osmosis-labs/osmosis/blob/3c7eab90146396d7cb5ee7f4580244be78bf44f6/x/twap/logic.go#L170-L194
i.e.
newRecord.Time = newTime // this line already exists
if record.Time.Equal(record.LastErrTime) {
	newRecord.LastErrTime = newRecord.Time
}

That SGTM!


Connection issues

I'm using the library in an Elixir project and received the following error after trying to connect... Another project using an earlier version of the library uses the same statement without issue.
iex(1)> {:ok, client} = :cqerl.get_client()
** (ArgumentError) argument error
(stdlib) :ets.lookup(:cqerl_clusters, :"$primary_cluster")
src/cqerl_cluster.erl:52: :cqerl_cluster.get_any_client/1

Is the app running?

Yes the app is running...

I ask because getting that error implies that the ets table doesn't exist. And that table is set up at app startup. So either the call is being made before cqerl is started, or somehow the cqerl_clusters table is being deleted.

At least, those are the only two options that seem likely (and really only the first one seems especially likely). Can you double check by running :application.which_applications() from the shell?

Admittedly I've created an umbrella project but at the same time specified cqerl in the main mix.exs and the configuration options to go along with it.
def application do
[applications: [:vex, :cqerl, :logger]]
end

I cannot see the cqerl library appearing in the list, only the sub modules I created.
iex(1)> :application.which_applications()
[{:dao, 'dao', '0.1.0'}, {:unicorn, 'unicorn', '0.1.0'},
{:search, 'search', '0.1.0'}, {:horse, 'horse', '0.1.0'},
{:donkey, 'donkey', '0.1.0'}, {:insight, 'insight', '0.1.0'},
{:zebra, 'zebra', '0.1.0'}, {:logger, 'logger', '1.3.2'},
{:hex, 'hex', '0.13.0'}, {:inets, 'INETS  CXC 138 49', '6.3.1'},
{:ssl, 'Erlang/OTP SSL application', '8.0'},
{:public_key, 'Public key infrastructure', '1.2'},
{:asn1, 'The Erlang ASN1 compiler version 4.0.3', '4.0.3'},
{:crypto, 'CRYPTO', '3.7'}, {:mix, 'mix', '1.3.2'}, {:iex, 'iex', '1.3.2'},
{:elixir, 'elixir', '1.3.2'}, {:compiler, 'ERTS  CXC 138 10', '7.0.1'},
{:stdlib, 'ERTS  CXC 138 10', '3.0.1'}, {:kernel, 'ERTS  CXC 138 10', '5.0'}]

So the app's not in fact running, and neither is vex by the look of it. Which means your problem is not with cqerl, but with your use of mix.

I think i agree, cqerl is listed among the apps when running :application.which_applications() in my more standard project. I'll backtrack and see what I can come up with.
iex(1)> :application.which_applications()
[{:hippo, 'hippo', '0.1.0'}, {:logger, 'logger', '1.3.2'},
{:cqerl, 'CQErl client for Cassandra', '1.0.1'},
{:uuid, 'Native UUID Generation', '1.4.1'},
{:quickrand, 'Quick Random Number Generation', '1.4.0'}, {:lz4, [], '0.2.2'},
{:snappy, 'snappy compressor/decompressor Erlang NIF wrapper', '1.1.1'},
{:semver, 'Semantic Versioning Support Library', '0.0.1'},
{:re2, 'Erlang NIF bindings for the re2 regex library', '1.4.0'},
{:pooler, [], '1.5.0'}, {:vex, 'vex', '0.5.5'}, {:hex, 'hex', '0.13.0'},
{:inets, 'INETS  CXC 138 49', '6.3.1'},
{:ssl, 'Erlang/OTP SSL application', '8.0'},
{:public_key, 'Public key infrastructure', '1.2'},
{:asn1, 'The Erlang ASN1 compiler version 4.0.3', '4.0.3'},
{:crypto, 'CRYPTO', '3.7'}, {:mix, 'mix', '1.3.2'}, {:iex, 'iex', '1.3.2'},
{:elixir, 'elixir', '1.3.2'}, {:compiler, 'ERTS  CXC 138 10', '7.0.1'},
{:stdlib, 'ERTS  CXC 138 10', '3.0.1'}, {:kernel, 'ERTS  CXC 138 10', '5.0'}]
Many thanks for your help.

No worries - good luck.


[Feature] Flink on K8s

Description
Supporting flink on k8s is a good idea, but there are many problems that need to be solved, such as how to mirror, how to load class classes into flink mirror, which classes are loaded into flink mirror, how to support the monitoring of task status in k8s, and track task status and many more. This is a good pr among the little dolphins
Use case
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[ ] I agree to follow this project's Code of Conduct

Do you have any suggestions?


Prioritize ${OPENSSL_ROOT_DIR} over system paths

This commit reorders the locations where CMake will look for openssl, starting with ${OPENSSL_ROOT_DIR} and then proceeding to system install locations.
This allows the user to provide a different Openssl version with -DOPENSSL_ROOT_DIR=<path/to/openssl>

Greetings @diku89 and thanks for the contribution! Apologies for taking so long to get back to you.
Have you signed the Contributor License Agreement for contributions to DataStax open source projects? If not you can find it at https://cla.datastax.com/. Thanks!

Hello. Thanks for the heads up. I just signed it.

Hey @diku89 , just so I have this right in my own head... the goal here is to make sure that an OpenSSL install supplied via the build param is used in preference over an existing install, correct?  That makes sense to me, so I think we're largely good if that's the case.  I need to test a few things locally but this seems pretty sane to me.
One thing: the syntax ENV OPENSSL_ROOT_DIR on line 87 seems to refer to specifying an OpenSSL install via an environment variable.  If we're preferring an OpenSSL install specified via a build param to a hard-coded value seems like we should prefer one specified via an environment variable as well.  So with that in mind it seems reasonable to make the hard-coded value(s) the last thing in this list.

After some testing I can confirm that this is definitely not behaving as expected.  Here's what we have from the current behaviour in base, when passing props via the command-line and via env vars:

Base using existing config:

-- Found OpenSSL: /usr/local/opt/openssl/lib/libssl.so (found version "1.1.1q")

LIBUV_ROOT_DIR=/work/local/libuv-1.44.2/ cmake -DOPENSSL_ROOT_DIR=/work/local/openssl ..

-- Found OpenSSL: /usr/local/opt/openssl/lib/libssl.so (found version "1.1.1q")

LIBUV_ROOT_DIR=/work/local/libuv-1.44.2/ OPENSSL_ROOT_DIR=/work/local/openssl cmake ..

-- Found OpenSSL: /usr/local/opt/openssl/lib/libssl.so (found version "1.1.1q")

Just moving the default /usr/local/opt/openssl config down seems to leave us in a much better spot:

Base using existing config:

-- Found OpenSSL: /usr/local/opt/openssl/lib/libssl.so (found version "1.1.1q")

LIBUV_ROOT_DIR=/work/local/libuv-1.44.2/ cmake -DOPENSSL_ROOT_DIR=/work/local/openssl ..

-- Found OpenSSL: /work/local/openssl/lib/libssl.so (found version "1.1.1q")

LIBUV_ROOT_DIR=/work/local/libuv-1.44.2/ OPENSSL_ROOT_DIR=/work/local/openssl cmake ..

-- Found OpenSSL: /work/local/openssl/lib/libssl.so (found version "1.1.1q")

I'm going to merge this PR as it stands and tweak it slightly after the fact so that both the CLI prop and the env var can override the default.
Huge thanks to @diku89 for the PR... and for your patience!

Quick note for the record: follow up commit mentioned above is here.


Enforce 512KB event size limit for PagerDuty events

PagerDuty will soon start enforcing a 512 KB event size limit on both the V1 and V2 api (https://v2.developer.pagerduty.com/docs/events-api-v2). This limit has been mentioned in the documentation for a while but was not enforced.
We regularly receive events that breach this limit. After investigating we found that these large events are formed by giant text blobs from the firing and resolving fields (https://github.com/prometheus/alertmanager/blob/master/config/notifiers.go#L49-L50) in details. It appears that these fields can have the descriptions of an unbounded list of alerts. These lists can be viewed in the PagerDuty UI but it's not clear how valuable they are to the users when the list becomes very large (over a hundred).
We have a few proposals for fixing this issue:

The simplest option is to truncate the firing and resolving fields to 250KB each which will leave 12KB for the rest of the message.

Truncate firing to 500KB. If it needs less than 500KB, the rest is used for resolving (or reverse the order if resolving is deemed more important than firing). Basically we ensure that the space taken up by firing plus resolving doesn't exceed 500KB but without a hard boundary on each field.

The above two options have the possibility that the truncation can happen inside an alert description. If it's important to make sure that alert descriptions are always displayed in full, we can apply a limit on the number of alerts used to render firing and resolving. However, unless this limit is quite small, it won't do a good job of ensuring that the full message size is below 512KB because the size of alert descriptions can vary.

We prefer options one or two. Regardless of what option is chosen, there should also be a size check after the message is encoded into JSON. If other text fields are added to the message in the future, this check will make sure that overall message size is under 512KB.

We already do this kind of limit checks at other places but AFAICT it's never been the case for the posted payload itself. We can't rely on the firing and resolving keys being present in the custom details as the user is able to change this from the configuration.
I'd rather check the size of the message to be posted. If it is above the 512KB limit, we replace the Custom Details field (which is likely the offender) with something like {"warning": "custom details not included because otherwise the message would exceed the maximum size of 512KB."}.

That sounds good to me!
I will get started on a PR for this soon.
Do you know when the next alertmanager release is planned to come out?

There is no fixed planning at this stage as the repo is not hyper active.

There's no established planning but I try to release a new version every 3 to 4 months even if there's not a lot of changes.

Closed by #2225


An error occurs when the generation command is executed

[x] This issue isn't duplicated - you can check if it is by using the search bar located at the top left hand corner and select "Issues" on the left.
[x] This issue includes appropriate markdown for sections - e.g. code blocks for crash dumps.
[x] This issue is understandable - feel free to use your native language to write issues if you are not comfortable with English.

Issue description
K2...Level name
When /p generate K2 is executed, an error is output to the console after "Spawn terrain for level" K2 "is being generated in the background" is output.
/p generate If you try to enter World K2 after executing K2, "Building terrain" will be displayed on the client screen and will not proceed.
By the way, a crash dump about this problem was not generated.

Reproducing the issue
Execute /p generate K2 using 5c009a2

Client information
ClientVersion:0.16.1
ServerOS:Ubuntu14.04 LTS
ServerEngine:Genisys
PHPVersion:7

Optional information
http://writening.net/page?zWJWZA
This is the full text of the error

This is not an error. being generated in the background just indicates that it it working. The world will be properly generated after the server is restarted.

@jasonwynn10 Check the "Optional Information" part.

I found that when I edit Config.yml, I get an error.
Config.yml after editing...https://www.evernote.com/shard/s465/sh/24760e0f-755d-4438-bbb4-9050124eef7c/53640baf8317f41a3d668198646d5e4e
Is there a wrong in this Config.yml?

what was changed @FUGAMARU ?

I'm sorry, but even if I run the /p generate K2 command, it will only output "Usage: / p generate " and the world will not be generated so it can not be verified.
I am using the latest commit.

But what was changed in the config?

Changed Line 48, line 75, line 77, line 79, line 82, line 84, line 86.
Please check Config.yml after change ...Link

Easier to read:
[47] # When a player enters a plot, a popup with basic info will be shown
[48] ShowPlotPopup: false
[...]
[74] # The size of a plot
[75]  PlotSize: 35
[76] # The height of a plot
[77]  GroundHeight: 128
[78]  # The width of the road
[79]  RoadWidth: 6
[...]
[81]  # The block that the road is made of (Default is Oak Planks)
[82]  RoadBlock: '45:0'
[83]  # The block that plot walls are made of (Default is Stone Slabs)
[84]  WallBlock: '43:0'
[85]  # The block used as the plot floor (Default is Grass)
[86]  PlotFloorBlock: '1:0'

@Aericio Thank you for your support.

is this issue using the latest version?

Yes of course.


Issue from website page Flutter文档

From URL: https://www.flutterchina.club/docs.md

您好, 我按照官方集成到原生 iOS 项目中, 可以用真机正常跑起来, 但是 archive 确报错? 在文档中并没有看到相关集成到原生 iOS 项目的打包文档. 希望能指点一下! 项目急着上线!

Hello, I am officially integrated into the native iOS project, you can run it with the real machine, but the archive does report an error? The package does not see the packaged documentation integrated into the native iOS project. Hope to give pointers! On the line!

Hello, I integrate into the native iOS project according to the official document, and can run normally with the real machine, but the archive does report an error? I don't see the package document related to the native iOS project in the document. I hope you can help me ! Thanks!

Hi @DaquanZhou
could you please provide your updated  flutter doctor -v?
Also, to better address the issue, would be helpful
if you could post a minimal code sample to reproduce the problem
Thank you

Hi @DaquanZhou
I'm not sure to understand your question,
are you looking for this documentation?
thank you

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact descriptions what happens, logs and the output of 'flutter doctor -v'  please.
All system setups can be slightly different so its always better to open new issues and reference related issues.


No JaCoCo redbar.gif, greenbar.gif images in Coverage Reports on visualstudio.com

Hello,
I am using visualstudio.com.
I added "Publish Code Coverage Results" to the pipeline and chose Jacoco as coverage tool. And when you open build's "Code coverage" tab - there is no redbar.gif, greenbar.gif images.
Also, there are two different links to those images:
.../_build/jacoco-resources/redbar.gif - which returns 404 error
.../Code Coverage Report_1/jacoco-resources/greenbar.gif - which returns an error that image contains errors.

Is it possible to fix this from my side? I have no idea how it was integrated. Any place where I could start?
Thank you!

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: c6ac4ff4-9839-6a6d-8d20-a25084d221ef
Version Independent ID: bd09f139-8e69-8ea1-6671-b7bba8e17c23
Content: Publish Code Coverage Results from a VSTS or TFS build
Content Source: docs/pipelines/tasks/test/publish-code-coverage-results.md
Product: devops
GitHub Login: @erickson-doug
Microsoft Alias: dastahel

I can confirm this issue. The reason is that the image files for some reason get HTML-escaped, e.g. �PNGIHDR&#1....

Apologies for taking so long to reply, @NormanFoxLee.  The code coverage displayed in-product is currently escaped out of concern for the possibliity of running malicious JavaScript.  We're addressing this, but in the meantime, please consider these extensions.  Please reopen this issue if we can provide additional help.


Not able use with SQL Client like SQLWorkbench - No support for catalog / schema / set Max Rows

Hi,
I was excited to see a new JDBC Driver for Athena and thought of using it for our interactive use.
the key issues are

Doesn't support Catalog / Schema
Doesn't support VIRTUAL/ EXTERNAL TABLE ( All tables in Athena are defined as External Tables, It only looks for TABLE and VIEW)
Doesn't support VIRTUAL VIEW ( Any views defined in Athena is VIRTUAL VIEW)
Most Importantly - i Couldn't any query from the SQLWOrkbench as it always returns the following exception in the log  (getWarnings not supported)
No Proxy Support ( we sit behind proxy server and there is no way to pass in PROXY config to client) - ( after patching getWarning to return empty string).
2019-09-02 14:12:41 INFO  Loaded global SSH host configurations from C:\Users\STHSH4.sqlworkbench\wbssh.settings
2019-09-02 14:13:33 INFO  Global SSH host configuration file C:\Users\STHSH4.sqlworkbench\wbssh.settings removed.
2019-09-02 14:13:36 WARN  The JDBC driver does not support the setMaxRows() function! (Not implemented)
2019-09-02 14:13:37 WARN  Error retrieving warnings Not implemented
java.lang.UnsupportedOperationException: Not implemented
at io.burt.athena.AthenaStatement.getWarnings(AthenaStatement.java:281)
at workbench.util.SqlUtil.getWarnings(SqlUtil.java:1651)
at workbench.sql.SqlCommand.appendWarnings(SqlCommand.java:400)
at workbench.sql.commands.SelectCommand.execute(SelectCommand.java:172)
at workbench.sql.StatementRunner.runStatement(StatementRunner.java:552)
at workbench.gui.sql.SqlPanel.displayResult(SqlPanel.java:3468)
at workbench.gui.sql.SqlPanel.runStatement(SqlPanel.java:2199)
at workbench.gui.sql.SqlPanel$16.run(SqlPanel.java:2137)

SELECT count(*) from cmdm_sf_rec_results.account
java.util.concurrent.ExecutionException: software.amazon.awssdk.core.exception.SdkClientException
2019-09-02 14:12:41 INFO  Loaded global SSH host configurations from C:\Users\STHSH4.sqlworkbench\wbssh.settings
2019-09-02 14:13:33 INFO  Global SSH host configuration file C:\Users\STHSH4.sqlworkbench\wbssh.settings removed.
2019-09-02 14:13:36 WARN  The JDBC driver does not support the setMaxRows() function! (Not implemented)
2019-09-02 14:13:37 WARN  Error retrieving warnings Not implemented
java.lang.UnsupportedOperationException: Not implemented
at io.burt.athena.AthenaStatement.getWarnings(AthenaStatement.java:281)
at workbench.util.SqlUtil.getWarnings(SqlUtil.java:1651)
at workbench.sql.SqlCommand.appendWarnings(SqlCommand.java:400)
at workbench.sql.commands.SelectCommand.execute(SelectCommand.java:172)
at workbench.sql.StatementRunner.runStatement(StatementRunner.java:552)
at workbench.gui.sql.SqlPanel.displayResult(SqlPanel.java:3468)
at workbench.gui.sql.SqlPanel.runStatement(SqlPanel.java:2199)
at workbench.gui.sql.SqlPanel$16.run(SqlPanel.java:2137)
2019-09-02 14:13:37 ERROR Error executing:
SELECT count(*) from
java.util.concurrent.ExecutionException: software.amazon.awssdk.core.exception.SdkClientException
2019-09-02 14:13:59 WARN  The JDBC driver does not support the setMaxRows() function! (Not implemented)
2019-09-02 14:13:59 WARN  Error retrieving warnings Not implemented
java.lang.UnsupportedOperationException: Not implemented
at io.burt.athena.AthenaStatement.getWarnings(AthenaStatement.java:281)
at workbench.util.SqlUtil.getWarnings(SqlUtil.java:1651)
at workbench.sql.SqlCommand.appendWarnings(SqlCommand.java:400)
at workbench.sql.commands.SelectCommand.execute(SelectCommand.java:172)
at workbench.sql.StatementRunner.runStatement(StatementRunner.java:552)
at workbench.gui.sql.SqlPanel.displayResult(SqlPanel.java:3468)
at workbench.gui.sql.SqlPanel.runStatement(SqlPanel.java:2199)
at workbench.gui.sql.SqlPanel$16.run(SqlPanel.java:2137)
If the intention is purely for using in java code ( not interactive use) , i understand. I assumed any JDBC (standard) drivers could be used with Interactive tool like SQLWorkbench.  Today we use SQLWorkbench for Athena for Adhoc / interactive/ discovery type which is well suited for athena

Hi and thank you for trying it out. The primary target for us has been non-interactive use, but there's no reason it couldn't work for interactive use eventually. JDBC is huge, and contains a lot of details that are not important for an application, and covering every single one of these has not been a priority. SQLWorkbench will use a couple of features, other tools other features and unless you cover them all there's always something that will break unfortunately.
In this case you seem to have run into that setMaxRows is not implemented. Please read the Known issues & limitations section of the readme for the difference between UnsupportedOperationException/"not implemented" and SQLFeatureNotSupportedException.
Could you clarify what you mean by the list of features? I don't understand what for example "Doesn't support Catalog / Schema" means.

I belive what ssenthi is referring to is full support for the various metadata APIs in Connection and ResultSet (and more?)
DB GUIs typically make heavy use of these in order to present a tree of database objects - schema/database/tables   and usually try to look for things like stored procedures, views, triggers etc.   Possibly this can be hacked by creative use of the result values instead of throwing exceptions ?   I doubt it --
This is the kind of thing that even if your 100% JDBC Compliant theres enough optional and vendor differences that its unlikely to work without huge effort. (or a kind of hybrid with the official JDBC under the hood providing the metadata - not fun )

I think this is an area where we will have to rely on contributions. We will probably not close the holes that appear when using the driver with different tools, and we need help to get them fixed. Some of these will be very easy to fix, but some will be much more complicated.
To take one example, DatabaseMetadata#getProcedures and a similar methods return a ResultSet. The ResultSet interface is enormous, and the implementation we use for actual queries can't be reused in this context. Many of these methods only need to return a ResultSet that is empty, but even an implementation of an empty result is a couple of hundred of lines of code, unfortunately.

For what it is worth, I set up an interactive session in DBeaver, and I was able to query just fine using this driver. I deselected all "optional" features in the driver-manager configuration, and if I open e.g. the tables view, it complains about it not being supported, but opening a SQL editor and running statements work just fine.

To take one example, DatabaseMetadata#getProcedures and a similar methods return a ResultSet. The ResultSet interface is enormous, and the implementation we use for actual queries can't be reused in this context. Many of these methods only need to return a ResultSet that is empty, but even an implementation of an empty result is a couple of hundred of lines of code, unfortunately. I wasn't running queries through a proxy, though.

While I don't know for sure, I would venture a guess that the reason the methods return a ResultSet is that the expectation is that you would effectively use the SQL standard INFORMATION_SCHEMA or an equivalent to issue real database queries to implement the methods underlying the DatabaseMetadata class. That being said, the information-schema implementation in Athena seems a bit lacking (e.g. it doesn't seem to be possible to distinguish a table and a view), and as far as I know, the DDL queries (like SHOW TABLES) does not seem to be composable, so some of the information would require custom ResultSet implementations.
However, contrary to your claim, I think the Result abstraction readily available internally in the driver would make it relatively easy implement the custom ResultSets. An empty ResultSet would likely not be more than 50 lines of boilerplate, which is still a lot, but far from hundreds of lines.


Make newly added products go to the top of the list

"Make newly added items go to the top so users are able to see what has just been added. If 3 or more things are added the new things are placed at the bottom of the list so you have to scroll to check you have added the right thing."

This is now resolved as of commit 51dfc698edb77a60a64f388f060be692c2f49ccb. Newly added products go to the top of the list.