Start Using SAP HANA 2.0, express edition (Virtual Machine Method)

Hi there,
I have problems with the step 5. I press Y but after a very long time HDB daemon dose not start.
I downloaded the VMware Workstation 12 Player and the hxexsa.ova file.
"Cannot find running HANA server."

HDB start
"FAIL: process hdbdaemon HDB Daemon not running"

Hi, please use the Q&A (https://answers.sap.com/index.html) for this kind of issue as this is for issues related to the tutorials, not technical problems.  This is a common error with many probable root causes, so I suggest you browse the existing posts before you post your own. Lastly, the following information helps troubleshooting: your computer's technical characteristics (remember you need at least 16GB RAM to run the XSA image and you might also want to check free disk space) and the errors in the tracefiles as the error message you are posting does not say much.  Cheers!


Enable-AzureADDirectoryRole missing parameter DirectoryRole

Hi Team!
According to doc https://docs.microsoft.com/en-us/powershell/module/Azuread/Enable-AzureADDirectoryRole?view=azureadps-2.0
Cmdlet Enable-AzureADDirectoryRole missing parameter DirectoryRole. While trying to use RoleTemplateId parameter I'm getting
Enable-AzureADDirectoryRole : Error occurred while executing ActivateDirectoryRole Code: Request_BadRequest Message: Instantiation of a 'DirectoryRole' object rerepresenting the implicit user role is not currently supported. RequestId: 59d8442c-6819-4a58-9989-084a640cc7cf DateTimeStamp: Tue, 30 May 2017 14:49:15 GMT HttpStatusCode: BadRequest HttpStatusDescription: Bad Request HttpResponseStatus: Completed

@msevastian - thank you for reporting the issue.  @RobdeJong - Can you please take a look at this reported issue?  Many thanks!

@VinceSmith - can you please look at the reported issue?
Thanks - Rob

@VinceSmith any update on your progress?  Thanks!

@msevastian - can you verify you are using the latest version of the PowerShell module?

No success too

@RobdeJong as you see I have installed latest available version of Azure AD module


Pokemons are getting vanished. Every time.

Somehow, the clone of master branch isn't able to catch Pokemons, they're getting vanished.
Here's some more info on this:
[#] Scanning area for objects (1 / 25)
map_cells are 21
[#] Walking from (28.6289703, 77.205265) to (28.6289703, 77.205265) for approx. 0.0 seconds
[#] Finished walking to (28.6289703, 77.205265)
(28.6289703, 77.205265, 0)
[#] Scanning area for objects (2 / 25)
map_cells are 21
[#] Walking from (28.6289703, 77.205265) to (28.6314703, 77.205265) for approx. 112.0 seconds
walking hook  02016-07-24 09:16:54,469 [   rpc_api] [WARNING] Unexpected HTTP server response - needs 200 got 502
walking hook  110 [#] Finished walking to (28.6314703, 77.205265)
(28.6314703, 77.205265, 0)
[#] Scanning area for objects (3 / 25)
map_cells are 21
[#] Walking from (28.6314703, 77.205265) to (28.6314703, 77.207765) for approx. 98.0 seconds
walking hook  96 [#] Finished walking to (28.6314703, 77.207765)
(28.6314703, 77.207765, 0)
[#] Scanning area for objects (4 / 25)
map_cells are 21
[#] Walking from (28.6314703, 77.207765) to (28.6289703, 77.207765) for approx. 112.0 seconds
walking hook  110 [#] Finished walking to (28.6289703, 77.207765)
(28.6289703, 77.207765, 0)
[#] Scanning area for objects (5 / 25)
map_cells are 21
[#] Walking from (28.6289703, 77.207765) to (28.626470299999998, 77.207765) for approx. 112.0 seconds
walking hook  110 [#] Finished walking to (28.626470299999998, 77.207765)
(28.626470299999998, 77.207765, 0)
[#] Scanning area for objects (6 / 25)
map_cells are 21
[#] Walking from (28.626470299999998, 77.207765) to (28.626470299999998, 77.205265) for approx. 98.0 seconds
walking hook  96 [#] Finished walking to (28.626470299999998, 77.205265)
(28.626470299999998, 77.205265, 0)
[#] Scanning area for objects (7 / 25)
map_cells are 21
[#] Walking from (28.626470299999998, 77.205265) to (28.626470299999998, 77.202765) for approx. 98.0 seconds
walking hook  96 [#] Finished walking to (28.626470299999998, 77.202765)
(28.626470299999998, 77.202765, 0)
[#] Scanning area for objects (8 / 25)
map_cells are 21
[#] Walking from (28.626470299999998, 77.202765) to (28.6289703, 77.202765) for approx. 112.0 seconds
walking hook  110 [#] Finished walking to (28.6289703, 77.202765)
(28.6289703, 77.202765, 0)
[#] Scanning area for objects (9 / 25)
map_cells are 21
[#] Walking from (28.6289703, 77.202765) to (28.6314703, 77.202765) for approx. 112.0 seconds
walking hook  110 [#] Finished walking to (28.6314703, 77.202765)
(28.6314703, 77.202765, 0)
[#] Scanning area for objects (10 / 25)
map_cells are 21
[#] Something rustles nearby!
[x] Found pokemon at distance<PHONE_NUMBER>12m
[#] Walking from (28.6314703, 77.202765) to (28.631224720434584, 77.20256539448013) for approx. 14.0 seconds
[#] Finished walking to (28.631224720434584, 77.20256539448013)
[x] Walked to Pokemon
[#] A Wild Ekans appeared! [CP31]
[x] Using Pokeball...
[x] Oh no! Ekans vanished! :(
[x] Found pokemon at distance<PHONE_NUMBER>12m
[#] Walking from (28.631224720434584, 77.20256539448013) to (28.631224720434584, 77.20256539448013) for approx. 0.0 seconds
[#] Finished walking to (28.631224720434584, 77.20256539448013)
[x] Walked to Pokemon
[#] Walking from (28.631224720434584, 77.20256539448013) to (28.633970299999998, 77.202765) for approx. 123.0 seconds
walking hook  121 [#] Finished walking to (28.633970299999998, 77.202765)
(28.633970299999998, 77.202765, 0)
[#] Scanning area for objects (11 / 25)
map_cells are 21
[#] Something rustles nearby!
[x] Found pokemon at distance<PHONE_NUMBER>5m
[#] Walking from (28.633970299999998, 77.202765) to (28.633762834946523, 77.20193653631071) for approx. 34.0 seconds
[#] Finished walking to (28.633762834946523, 77.20193653631071)
[x] Walked to Pokemon
[#] A Wild Ekans appeared! [CP73]
[x] Using Pokeball...
[x] Oh no! Ekans vanished! :(
[x] Found pokemon at distance<PHONE_NUMBER>5m
[#] Walking from (28.633762834946523, 77.20193653631071) to (28.633762834946523, 77.20193653631071) for approx. 0.0 seconds
[#] Finished walking to (28.633762834946523, 77.20193653631071)
[x] Walked to Pokemon
[#] Walking from (28.633762834946523, 77.20193653631071) to (28.633970299999998, 77.205265) for approx. 131.0 seconds
walking hook  30 [ USER ABORTED, EXITING.. ]

This has been happening for last 45 mins constantly.
Running on Windows 10 64 bit.
How to fix this?

please use dev branch

Please use the development branch. Using the master branch may of gotten you softbanned.

You might probably be softbanned.

You are softbanned for 1-3h (in most cases is it 1h).

Oh. Okay. Thanks guys. :D :+1:


Nice (stealth) project!

Hey Steven,
I read your associated blog post with interest and found my way here from there. I was surprised to see that despite this being a very nicely crafted piece of software it's missing the basic readme and docs essential to give people the confidence to adopt it. Are you planning to eventually release this project? I'd like to use it!
Best,
Darrick

Hi Darrick,
thanks for reaching out. This repository was actually only intended to support the blog posts. In the process of writing, I discover better and more performant ways to solve the tasks mentioned in the blog posts, which results to a rather volatile API. To comply with Clojure’s philosophy not to break things and because I thought nobody cares anyway, I was too lazy to publish anything publicly in a stable way. But if you have use for this project, I could invest some work to release something that matches Clojure’s standards. What exactly do you want to use? The search abstraction or the matcher build upon this abstraction?
Best,
Steven

The piece that most interests me personally is the =* matcher you built. I pulled it into a project that I'm working on as a test dependency to try it out, and it works great! I'm sure there are plenty of people who'd benefit from using it as well who just don't know it exists yet.
For my own personal use, I'm happy to use the project as it is since I don't mind trying out alpha software, especially on the dev tools side, so no need to do anything to support me.
If it matters, I personally don't think you need to worry about releasing totally stable code. If it doesn't have much adoption it doesn't matter if you change it, or if it does and you want to make major changes, it's easy enough to just rename the project and move on!
Cheers,
Darrick

I moved the =* matcher to lomin/sinho, and uploaded a 1.0.0 release of it to clojars, since the matcher itself is stable. I also added a basic README.
Here in this repository, things will continue to change and break, but will have no impact for the =* matcher.
Thanks for kicking this off, Darrick!


Improve error message due to bad "from" argument

Repro is with Zed commit d5b407d.
Let's start with this working example:
rm -rf $ZED_LAKE_ROOT
zed lake init
zed lake create -p logs -orderby ts:desc
zed lake load -p logs ~/work/zed-sample-data/zeek-default/*
zed lake query -z 'from logs | count()'

If I'd made a typo in the from argument of that last command, here's what I'd see:
$ zed lake query -z 'from logz | count()'
Valid encoded KSUIDs are 27 characters

While experienced users may come to understand the role KSUIDs have to play in Zed Lakes, since in this case the user was typing what they believed to be a Pool name similar to the user-visible one that's shown to them in the app, the error message should probably not exclusively reference KSUIDs.

Verified in Zed commit 96be300.
The error message has been replaced with something more informative.
$ zed -version
Version: v0.29.0-416-g96be3005

$ zed lake query -z 'from logz | count()'
logz: pool not found

Thanks @mattnibs!


Support inline code

I think I will use
`j expression`

syntax similar to Knitr for markdown,

Closed with #80


Crafting update episode 1

About the PR

Added bunch of new craftable stuff, and items to craft with
Why / Balance

milestone
Technical details

7
Media

7
Requirements

[x ] I have read and am following the Pull Request and Changelog Guidelines.
[ x] I have added media to this PR or it does not require an ingame showcase.

Breaking changes

Changelog

Gonna continue tomorrow, and will try to add a craft for it

okay, fixed issue with textures but the game is still broken

You add improvised jetpacks but no construction graphs?

You add improvised jetpacks but no construction graphs?

i didn't make them yet :godmode:
i WILL make them

if the pr isnt finished dont make it open

hasnt been touched in two weeks, ask for it to be reopened when you are working on it again / its done


Use additional npm / Yarn options to speed up installation

The node_modules installation should be as fast and deterministic as possible, as it's taking place internally within a Python package build. Routine developer-facing information should be omitted, as this is expected to be handled as part of the direct Javascript development process.
See:
https://docs.npmjs.com/cli/v7/commands/npm-install#configuration https://yarnpkg.com/cli/install#options

@blink1073 Is this something that you'll be interested in reviewing?


Donor recognition page

WIP
Slim layout came from #15.
Using middleman's data functionality to store the donors as json files.

[x] Responsive
[ ] Datas
[ ] How to link to it

OH! Also need to figure out how we want to link to this on the website.

Fixed some responsive issues and added a logo to the slim layout.
@amandaharlin @the-simian Do we have data for the donors/organizers? Format doesn't really matter (I can transform to the below style).
[{
   "name": "SST Software",
   "image_url": "sst.png",
   "level": "Gold"
}]

bump ~ devin still wants some data (thank you for the reminder!)

I merged these changes to master, and then tweaked things a bit. I thought that GitHub would recognize that all of the commits in this PR are now in master and automatically show the PR as merged. Not sure... Anyway, thanks for the work on this @DevinClark!


Kafka producer should raise an exception when it fails to connect to broker

What this PR does / why we need it:
In an ingestion job from the Python SDK, the producer should throw an exception if it fails to push data to the Kafka broker.
Which issue(s) this PR fixes:

Fixes #626
Does this PR introduce a user-facing change?:

NONE

/retest

/test test-end-to-end-batch

/approve

/lgtm

/lgtm

/test test-end-to-end-batch


Complete GettingStarted ToDo Documentation

The Getting Started Document
https://github.com/hyperledger/aries-cloudagent-python/tree/master/docs/GettingStartedAriesDev
Lists several documents that are To Do items:
Request filling in. Especially the credential attributes to better understand how to use Sovrin ZCP credentials.
To Do: Establishing a connection between Aries Agents
To Do: Issuing an Indy credential: From Issuer to Holder/Prover
To Do: Presenting an Indy credential: From Holder/Prover to Verifier
To Do: Next steps: Creating your own Aries Agent

Revamping the documentation.  This will not be relevant — closing.


Beginning in the FixedTarget and Observer classes

@bmorris3 – When you make pull requests, we get a notification, but we don't know whether you want us to review them at this point or not. In Astropy and generally on Github, people do this differently, you could:

Say whether this is ready for review or not in the description.
If it's work in progress (usually is when the PR is first made) and we shouldn't review yet, add WIP: to the title or add a task list (https://help.github.com/articles/writing-on-github/#task-lists) of things you still want to do in this PR.

Another important thing to know (which you probably do) is that Github doesn't send notifications when you push new commits to a branch. So if there has been review, and then you've addressed the comments, and want something to be merged or a second round of review, you have to leave a comment  saying so.

@bmorris3 - something else is confusing me about this PR: it appears to have some (all?) of the commits from #4.  Did you mean to do that?  I'm guessing not, because it looks like this is only supposed to include some changes in core.py, and you have a commit where you removed the API doc.  This doesn't actually remove it from history, and if you merge this after the API is merged, this will delete that document, which I don't think is what you want.
  I try to avoid ever having an "origin"/"upstream" because I find it confusing, so I always use the same names as github assigns.  But YMMV.
git rebase -i upstream/master

this will bring up an editor showing you all the commits between master and this branch. You'll want to delete the top 3 (which are from the API branch).  Then when you close your editor it will give you an error about The previous cherry-pick is now empty, possibly due to conflict resolution.  Just do git rebase --continue and everything will be fine.   Once you've done that, you'll see that this branch is just the two commits you wanted and nothing else.  So you can do git push origin initclasses -f and all should be good.

Just to leave a note here – @eteq helped me do the rebase, they won't appear in the PR.

Those were not the commits you were looking for.
– Obi Eteq

A quick note: that missing indent @bmorris3 added in the last commit at @adrn's suggestion was not a problem per se with the package template, because there's no core module in the package template.  (no action required, just an FYI about why it happened)

I left a few more comments @bmorris3, but all but the last are phrasing/polish bits on the docs, nothing substantial on the code itself.  The last one is an important subtlety, though, that I think needs to be addressed (fortunately, it's easy to fix).

@eteq : thanks for the comments! They've been addressed in the latest commit.

@eteq @adrn – Does one of you have time for a final review and hitting the merge button for this PR later today?

This looks good to me -- way to go @bmorris3! I'm going to fire away and merge this.

:balloon:
Have a nice weekend, everyone!

:confetti_ball: One of many reasons to celebrate today!

I merged this for :rainbow: !


[alpha-issue] - TidePool - Looks like all boluses are labeled as "automated". This highlighted bolus was manual.

          Looks like all boluses are labeled as "automated". This highlighted bolus was manual.

(This test was done via the alpha branch, not specifically this PR, btw)

Originally posted by @MikePlante1 in https://github.com/nightscout/Open-iAPS/issues/30#issuecomment-2045816305

I think this is resolved, but maybe not by the PR that was referenced above.

This is not resolved. I just tested it. All boluses are labeled automatic:

smb
manual bolus
external bolus

In Loop the distinction is made that manual and external (non-pump) both show up as manual (no label shown and a lighter blue) and automatic shows up as automatic (label says automatic and darker blue).

This issue is still present in main branch.

Addressed for TCD / Trio 1.0.0 dev.


Please expose docChanges() for Firestore SDK

Reference: https://github.com/GitLiveApp/firebase-kotlin-sdk/discussions/115#discussioncomment-219985
Many thanks

@nbransby Wow, thanks for the responsibly!
When you will release this version?

@nbransby Wow, thanks for the responsibly!
When you will release this version?

Next week

Next week

Next week

Still not released? I see 1.2.0 released and I did found doChanges capability
@nbransby

Next week

Still not released? I see 1.2.0 released and I did found doChanges capability
@nbransby

Yes it should be there, called documentChanges

Yes it should be there, called documentChanges


Test Clients should increment generation on calls to replace when the .spec field has been modified.

TestResourceClient.replace currently has a check in place to increment the .metadata.resourceVersion version field but the .metadata.generation field is not incremented when there are changes to the .spec section of a CR. This leads to bad behavior while testing as certain patterns will compare generation and observedGeneration to determine if action should be taken.
According to discussions from the k8s dev team, generation:

should be incremented for all changes made to the .spec field of a CR.
should not be incremented if there are changes only to .metadata or .status

Looking at TestResourceClient.replace, .metadata.resourceVersion is being incremented and an additional check and increment should probably be added to .metadata.generation as well.

Thanks for the report and the detailed information on how it should work!


Usage question: constrained optimization satisfy just one constraint out of several

I'd like to run a constrained optimization such that a point is within the constrained set as long as it satisfies one of the constraints I impose. The default setting for constrained optimization requires a valid candidate point to satisfy all of the constraints.
Currently, I'm identifying the best point from a constrained optimization with each of the constraints separately. I basically create separate Bayesian optimization models with one of the constraints I'm considering. I then select the point with the highest acquisition score out of the best points from each of the separate constrained optimization problems as the next point to evaluate.
Is there a better way of doing this? I would really appreciate any help.

The only alternative I see here is to implement your own optimizer, maybe inheriting from the one in the package, that would provide this kind of customized behavior.

As a followup, do you have any advice for overriding certain functions within the GPyOpt library. I'm realizing that I could effectively solve my problems if I modified the acquisition_function function in this file https://github.com/SheffieldML/GPyOpt/blob/master/GPyOpt/acquisitions/base.py and the indicator_constraints function in this file https://github.com/SheffieldML/GPyOpt/blob/master/GPyOpt/core/task/space.py.
I do not know how to selectively override these functions and advice on how to proceed would be greatly appreciated.

One option would be to use Modular Bayes Opt class that allows you to define your problem in terms of objects. This way you can re-use some components of GPyOpt while providing your own implementations for others. Tutorial notebook where we override the acquisition function: https://github.com/SheffieldML/GPyOpt/blob/master/manual/GPyOpt_modular_bayesian_optimization.ipynb

I assume it's also possible to override the DesignSpace component as well?

Yes, although I would advise to exercise caution while doing that, given how central parameter space object is in the library

def indicator_constraints(self, x):
    x = np.atleast_2d(x)
    I_x = np.ones((x.shape[0],1))

    if self.constraints is not None:
        for d in self.constraints:
            try:
                exec('constraint = lambda x:' + d['constraint'], globals())
                ind_x = (constraint(x)<0)*1
                I_x = I_x + ind_x.reshape(x.shape[0],1)
            except:
                print('Fail to compile the constraint: ' + str(d))
                raise

        I_x = (I_x >= 1).astype(float)
        return I_x

    else:
        return I_x

I've implemented the function above within my own custom Design_space class to override the default indicator constraints function.  Basically, I'm allowing a point to be feasible as long as it satisfies one of the constraints (an "or" instead of the default "and"). I'm then passing this as a class to the modular Bayesian optimization class along with default classes for the other required arguments (as demonstrated in the Modular Bayesian Optimization tutorial).
I've tested a few different constraints. However for this particular set of constraints, when I call run_optimization, the call doesn't terminate.
constraints = [{'name': 'constr_1', 'constraint': '-x[:,1] + 0.5'},
{'name': 'constr_1', 'constraint': 'x[:,1] + 0.5'}
]
I'm wondering if anyone has an idea of what may be causing the non-terminating behavior. Any advice would be greatly appreciated!

Here is the link to the full code.
OverrideSpace.txt

@youngapsmikes @apaleyes  The GPyOpt code re-initializes the design space during the optimization here. Also, as I've pointed out in #94, the anchor point generator calls constraints on both the expanded and non-expanded variables, so be careful.


Fixed #25748 -- Added clarification on projects vs. applications.

A table was proposed in https://github.com/django/django/pull/5788, but I personally found it too simplified. Other opinions welcome.

I like it.

So do I.


Inline Diff Highlight Groups

Is your feature request related to a problem? Please describe.
When looking through diffs that are not full line deletions/additions, I'm a bit frustrated by not having a clear view of what exactly got modified.
Describe the solution you'd like
I'd love to have inline-diff highlight groups akin to those of gitsigns.
A simple example:
neogit:

gitsigns:

I've wanted this too, actually. I'll see what I can do.

Is this something where diffchar.vjm can help ?

@CKolkey Is there any updates on this? I can not waiting to use this feature.


[SPARK-12632][PYSPARK][DOC] PySpark fpm and als parameter desc to consistent format

Part of task for SPARK-11219 to make PySpark MLlib parameter description formatting consistent.  This is for the fpm and recommendation modules.
Closes #10602
Closes #10897

Test build #51195 has started for PR 11186 at commit 744e37d.

Test build #51195 has finished for PR 11186 at commit 744e37d.

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

Merged build finished. Test PASSed.

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

Closes #10602
Closes #10897

@BryanCutler made a quick pass. While we're doing the format change, we may as well make a few little doc clean ups as per my comments.

Thanks for taking a look @MLnick , I agree that we might as well try to clean up the docs as well.  I made the corrections you suggested and some others that I found - also tried to sync up the scala side too.
ping @mengxr to take a look also

Merged build finished. Test FAILed.

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

jenkins retest please

Jenkins retest this please

Test build #51401 has started for PR 11186 at commit 37de948.

Test build #51401 has finished for PR 11186 at commit 37de948.

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

Merged build finished. Test PASSed.

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

Test build #51572 has started for PR 11186 at commit 5d59b4e.

Test build #51572 has finished for PR 11186 at commit 5d59b4e.

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

Merged build finished. Test PASSed.

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

LGTM, merged into master. Thanks @BryanCutler!


Update docs

What Changed

Point out SB 7 "peer" requirement
Attempt to add clarity

Change Type

[ ] maintenance
[x] documentation
[ ] patch
[ ] minor
[ ] major

📦 Published PR as canary version: 0.0.12--canary.4.107d2c1.0

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

:rocket: PR was released in v0.0.18 :rocket:


Node crashes in docker container

I may be overly hopeful for a 6 minutes of cast to a gif, but this is what I'm getting
docker run --rm -v $PWD:/data asciinema/asciicast2gif -s 1.0 -t solarized-dark demo.cast demo.gif
==> Loading demo.cast...
==> Spawning PhantomJS renderer...
==> Generating frame screenshots...

<--- Last few GCs --->

  104433 ms: Mark-sweep 1372.0 (1434.0) -> 1372.0 (1434.0) MB, 678.8 / 0.0 ms [allocation failure] [GC in old space requested].
  105111 ms: Mark-sweep 1372.0 (1434.0) -> 1373.0 (1404.0) MB, 677.6 / 0.0 ms [last resort gc].
  105784 ms: Mark-sweep 1373.0 (1404.0) -> 1374.0 (1404.0) MB, 673.1 / 0.0 ms [last resort gc].

<--- JS stacktrace --->

==== JS stack trace =========================================

Security context: 0x9f9db0cf781 <JS Object>
    1: Dm [/app/main.js:~626] [pc=0x34f3123b23b6] (this=0x6c6287c409 <JS Global Object>,a=0x24fd68bcc6b9 <a sl with map 0x9ecc840fbf9>,b=59)
    2: Em [/app/main.js:~627] [pc=0x34f3123cb54f] (this=0x6c6287c409 <JS Global Object>,a=0x24fd68bcc6b9 <a sl with map 0x9ecc840fbf9>,b=59)
    3: /* anonymous */ [/app/main.js:325] [pc=0x34f31258f74e] (this=0x6c6287c409 <JS Global Object>)
    4: Qf [/app...

FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory
 1: node::Abort() [node]
 2: 0x7d007c [node]
 3: v8::Utils::ReportApiFailure(char const*, char const*) [node]
 4: v8::internal::V8::FatalProcessOutOfMemory(char const*, bool) [node]
 5: v8::internal::Factory::NewFillerObject(int, bool, v8::internal::AllocationSpace) [node]
 6: v8::internal::Runtime_AllocateInTargetSpace(int, v8::internal::Object**, v8::internal::Isolate*) [node]
 7: 0x34f3120079a7
/app/asciicast2gif: line 118:    16 Aborted                 (core dumped) node $NODE_OPTS "${asciicast2gif_dir}/main.js" "${1}" "${2}" "${tmp_dir}" $theme $speed $scale

I was able to escape the heap limit with -e NODE_OPTIONS='--max_old_space_size=4096' in the docker command, that led me step closer
docker run --rm -v $PWD:/data -e NODE_OPTIONS='--max_old_space_size=4096' asciinema/asciicast2gif -s 1.0 -t solarized-dark demo.cast demo.gif
==> Loading demo.cast...
==> Spawning PhantomJS renderer...
==> Generating frame screenshots...
==> Combining 4338 screenshots into GIF file...
/app/main.js:697
Qe)}function ep(a){try{var b=a[0];return b.h?b.h(a):b.call(null,a)}catch(c){if(c instanceof Object)throw b=c,Yo(a[6]),b;throw c;}}function fp(a,b,c){c=Zo(c,dp(function(c){a[2]=c;a[1]=b;return ep(a)}));return r(c)?(a[2]=Nb(c),a[1]=b,dj):null}function gp(a,b){a=a[6];null!=b&&a.Dc(null,b,dp(function(){return function(){return null}}(a)));Yo(a);return a}
                                                                                ^

Error: spawnSync /bin/sh E2BIG
    at exports._errnoException (util.js:1020:11)
    at spawnSync (child_process.js:451:20)
    at Object.execSync (child_process.js:507:13)
    at Dp (/app/main.js:708:246)
    at /app/main.js:713:178
    at Function.b (/app/main.js:709:287)
    at ep (/app/main.js:697:48)
    at /app/main.js:697:193
    at /app/main.js:689:264
    at Immediate.Po (/app/main.js:685:331)

Is it possible that you link to example .cast file that helps to reproduce this?

https://s3.amazonaws.com/hlolli/demo.cast

I have same problem, what happen?
root@goorm:/workspace/whdzera(master)# asciicast2gif https://asciinema.org/a/266267.cast demo.gif
==> Loading https://asciinema.org/a/266267.cast...
==> Spawning PhantomJS renderer...
==> Generating frame screenshots...
==> Combining 42 screenshots into GIF file...

/usr/local/lib/node_modules/asciicast2gif/main.js:697
):b.call(null,a)}catch(c){if(c instanceof Object)throw b=c,Yo(a[6]),b;throw c;
                                                                    ^
TypeError: Object #<Object> has no method 'execSync'
    at Dp (/usr/local/lib/node_modules/asciicast2gif/main.js:708:246)
    at /usr/local/lib/node_modules/asciicast2gif/main.js:713:178
    at Function.b (/usr/local/lib/node_modules/asciicast2gif/main.js:709:287)
    at ep (/usr/local/lib/node_modules/asciicast2gif/main.js:697:48)
    at /usr/local/lib/node_modules/asciicast2gif/main.js:697:193
    at /usr/local/lib/node_modules/asciicast2gif/main.js:689:264
    at Object.Po [as _onImmediate] (/usr/local/lib/node_modules/asciicast2gif/main.js:685:331)
    at processImmediate [as _immediateCallback] (timers.js:330:15)

Wow, this asciicast file is 80 MB, I'd say it's on a heavy side :)
I don't think asciicast2gif will manage. There's just too many frames and the current implementation requires all of it to fit into memory (image concat done by imagemagick's convert).
However I just released asciicast2gif's successor - https://github.com/asciinema/agg . It's meant to handle any asciicast you throw at it very fast with minimal memory usage. You should try it out (asciicast2gif is no longer maintained).
I generated gif from your demo with agg, took 1 minute:
marcin@puter ~/tmp> agg -v --fps-cap=15 --speed=3 issue-53.{cast,gif}
[2022-08-14T14:33:10Z INFO  agg] terminal size: 174x38
[2022-08-14T14:33:10Z INFO  agg] selected font family: JetBrains Mono
[2022-08-14T14:33:10Z INFO  agg] selected theme: dracula
[2022-08-14T14:33:10Z INFO  agg] gif dimensions: 1478x764
1412 / 1412 [=========================================================] 100.00 % 23.61/s
[2022-08-14T14:34:09Z INFO  agg] rendering finished in 59.800545s

The generated gif file was huge - 212 MB. So I ran it through gifsicle:
marcin@puter ~/tmp> gifsicle --lossy=80 -k 64 -O3 -Okeep-empty issue-53.gif -o issue-53-opt.gif
gifsicle: warning: huge GIF, conserving memory (processing may take a while)

This one took 10 minutes or so (didn't count), and produced 117 MB gif.
Either way, this type of animation is an extreme case asciinema will never be a good fit for.


[BUG] The RestController always consumes content when handling real HTTP requests

Describe the bug
The BaseRestHandler includes a check whether content (the REST body) has been consumed by an API, throwing an exception here if it does not do so:
https://github.com/opensearch-project/OpenSearch/blob/c25b00c6b62fd6766999c09fd06dd1f7b281b3eb/server/src/main/java/org/opensearch/rest/BaseRestHandler.java#L125-L127
This exception is tested in OpenSearch in a few places.
Unfortunately, it appears that these unit tests which rely on the behavior of the BaseRestHandler do not take into account the fact that the RestController always consumes content here when a REST request arrives via the HTTP handler:
https://github.com/opensearch-project/OpenSearch/blob/2dc071ff2546b0c6483b9e0a382badf9ac353cdc/server/src/main/java/org/opensearch/rest/RestController.java#L296
It appears that in an actual cluster handling HTTP requests, the above exception is dead code.
However, it is actively tested directly calling the Rest Handler without using the Rest Controller code path which reads the content.
This is confusing to developers, undocumented and inconsistent behavior, and leads to incorrect assumptions, ultimately returning a confusing 500 error code to end users without any hint of why.
Related component
Other
To Reproduce

Run OpenSearch with the Flow Framework plugin installed, which relied on this exception handling
Query a workflow that does not support a body, but include a body in the request, for example

localhost:9200/_plugins/_flow_framework/workflow/any_id_here
{}

Get this response:

{
    "error": {
        "root_cause": [
            {
                "type": "illegal_state_exception",
                "reason": "Channel is already closed"
            }
        ],
        "type": "illegal_state_exception",
        "reason": "Channel is already closed"
    },
    "status": 500
}

Expected behavior
Given the multiple tests of the error message, one would suggest the expected behavior is a 400-series REST response with error message: "request [GET /_plugins/_flow_framework/workflow/{workflow_id}] does not support having a body"
However, testing the other OpenSearch APIs with unit tests for this behavior, it appears they silently ignore the body.  In particular, the commit which added the exception indicated that it should have triggered on deleting an index while including a body.
Given that this behavior has existed for 5 years, and the exception was only put in a year before that, and there are probably many hundreds (thousands? millions?) of scripts which might break if we restore the "tested" expected behavior, it seems that perhaps users should "expect" the presence of a body to be silently ignored.
Additional Details
Plugins
Reproduced with only the Flow Framework plugin installed as reported in https://github.com/opensearch-project/flow-framework/issues/638
Additional context
I'm happy to submit a PR to "fix" this but would like some input from other maintainers on how they'd like it fixed.  I see two options:

In the REST controller content length check, use the overload that does not consume the content, e.g.,

final int contentLength = request.content(false).length();

This will restore the "intended" behavior and probably break scripts worldwide.

Delete the consumed content check entirely and remove any tests that rely on it; explicitly code this behavior if desired; or appropriately document that content is ignored.

Why is the channel already closed? Are there multiple places trying to send a response on the RestChannel? Once a response is sent on the RestChannel (like this) the channel is closed and any attempt to send another response will receive a Channel is already closed error.
Does the check make sense where it is in the first place? I don't believe many actions will read the request content inside of prepareRequest, but instead will read them when the action is accepted here.

Why is the channel already closed?

Null pointer exception in our (my) implementation. I returned null never expecting it to be read because it would have hit that other illegal argument exception.

Does the check make sense where it is in the first place?

No, I never liked it because it allowed you to go through all of the actions you would normally take, and only gave an error at the end after the response, swallowing/hiding any actual response.  I disliked where it was so much I coded up a fail-fast (but didn't realize it didn't actually have any effect other than in tests).

I don't believe many actions will read the request content inside of prepareRequest, but instead will read them when the action is accepted here.

It varies. The content sometimes needs to be parsed on the Rest Action side before being sent over transport.  Many plugins do this.

To be clear, the bug is that there's a block of dead code (in actual operation) that's only triggered on unit tests that is confusing to devs like me. :)

[Triage - attendees 1 2 3 4 5 6 7 8]
@dbwiddis Thanks for filing


[KeyAuth Plugin] Generate key when no key is given on creation

Currently you are required to pass a key, the plugin should generate a key when no key is passed, making it easier to use and configure.
Should a user pass a key, the KeyAuth plugin should favor that key over generating.

Dup of #48

Good catch :+1:


Need a checkbox "Don't ask me for signing for the next n minutes"

I need this feature to manage CRDT apps properly, otherwise users will keep seeing signing dialogs frequently

@tomicvladan can you look into how this could be done, write here and we prioritize afterwards.

It shouldn't be difficult to implement this in code. We could store timestamp when approval expires in the extension storage, and check each time when a new signing request arrives. But could this be a security risk? And I'm not sure how to implement this in UI? We still want to keep the current behaviour as default.


Integrate aeternity blockchain

Name: aeternity

HD Derivation Scheme: 457

Symbol: AE

Documentation:
https://aeternity.com/
https://github.com/aeternity

Reason:
Checklist

[ ] Implement functionality in C++. Put it in a subfolder of src/.

[ ] Address (if necessary)
[ ] Transaction (if necessary)
[ ] Signer

[ ] Write unit tests. Put them in a subfolder of tests/.
[ ] Mnemonic phrase - > Address derivation test. Put this test in the CoinTests.cpp file
[ ] Add relevant constants in TWCoinType, TWP2SHPrefix, TWEthereymChainID, TWHRP, etc., as necessary.
[ ] Return correct curve and purpose in src/Coin.cpp.
[ ] Implement address validation and derivation in src/Coin.cpp.
[ ] Implement coin configuration src/include/TWCoinTypeConfiguration.cpp.
[ ] Write interface header in include/TrustWalletCore and implement the interface in src/interface.

[ ] Address interface (if necessary).
[ ] Signing interface.

[ ] Validate generated code in Android an iOS projects. Write integration tests for each.
[ ] Add a coin icon to "Supported Blockchains" section in README.md

[ ] Upload coin icon to TrustWallet/tokens if necessary

Thank you @vikmeup we're spreading this in our community and we already have candidates which would like to tackle this tasks. They will followup here soon.

Thank you @vikmeup we're spreading this in our community and we already have candidates which would like to tackle this tasks. They will followup here soon.
Edit: btw. its aeternity not Aethernity :)

Hi @eminogrande, any update on this? Is there anyone going to work on it from your community?

I'm on it

Hi @Defuera, please check with the owner of this issue before you start.

@johnnynanjiang hi, it's been discussed with Viktor before.

@Defuera @johnnynanjiang @eminogrande
Just notice @Defuera's comments,  are you working on this issue now?

@zyfrank yeap

@eminogrande seems the right person for this task is @Defuera ,considering he has spent some time on it?

@zyfrank and @Defuera you can also work together, i'm happy to increase the bounty.  I'd like @vikmeup to confirm the final code before the bounty gets paid. Thanks for your efforts!

@zyfrank can you connect to me on telegram @vikmeup? I will connect you together

@eminogrande , @vikmeup : a follow-up (trust-wallet related) bounty could be this one:

Isolate Chain Specific Code Units

The final goal would be to have the complete(!) source-code of a chain (e.g. aeternity) within one(!) folder only (ideally producing a linkable library)

@Defuera Hello from Gitcoin Core - are you still working on this issue? Please submit a WIP PR or comment back within the next 3 days or you will be removed from this ticket and it will be returned to an ‘Open’ status. Please let us know if you have questions!

[x]  reminder (3 days)
[ ]  escalation to mods (6 days)

Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days

Why is this keep showing up? PR is already in progress 🤔

Why is this keep showing up?

An yearl old gitcoin bug.
I've notified gitcoin several times. Sadly, they keep spamming the issues with their bot.

@eminogrande Hi, I have some question that I cannot find answers neither in documentation nor in aeternity sdks, what is the best place for me to inquire?

Ya know what? I snooze it for 100 days for now, so it doesn't pester you as much.

Ya know what? I snooze it for 100 days for now, so it doesn't pester you as much.

That is a workaround. It is time to fix the defect(!) logic of the bot, see #4434

Issue-Processing @ Gitcoin:

You are blocked from the gitcoinco organization and cannot comment because of this content.
https://github.com/gitcoinco/web/issues/4434#issuecomment-493983372

No wonder this issue-count has skyrocketed there...

Lazaridis has been warned repeatedly about our code of conduct after repeatedly violating it and has refused to change his behaviour.
@Defuera i'm still commited to fixing the gitcoinbot issue.  the latest is at https://github.com/gitcoinco/web/issues/4434#issuecomment-493983372

Lazaridis has been warned repeatedly about our code of conduct after repeatedly violating it and has refused to change his behaviour.

 Because Ethereum has excellent liberal qualities on their projects. Even harsh criticism never leads to censorship.
You should move over to TRON or EOS and others: its full of fascists there.
Note to readers:
ConsenSys Bounty: find the violating behavior.
Note to readers 2:
Totally overwhelmed product-managers are play a major role in "The ConsenSys Failure" and the overall "Ethereum 1 Failure" (and possibly the upcoming Ethereum 2 Failure).

@lazaridiscom you were blocked because of repeated violations of our code of conduct and a stubborn refusal to change your behaviour.  as i repeatedly warned you, anyone is welcome to be critical / file bug reports as long as they do not violate our code of conduct.
i am unfollowing this thread now and will not be commenting again.

The CoC protects (freethinking) people like me from (fascist like) abusers or your kind - at least in theory. Dictators always like to twist CoC (laws) as they see fit. They have the guns, or the buttons.
Anyone can see (even without my PERFECTLY VALID issues/comments) that you FAIL to manage the gitcoin.co product.
Blocking people that make your failure transparent does not change anything.
Note to readers:
The ConsenSys bouty is still active: find the violating behavior

looks like my fix for the gitcoinbot issue worked!  closing that issue now

nah, still spams, e.g.: https://github.com/utunga/homesafe/issues/1#issuecomment-495286234

@lazaridiscom that message from gitcoinbot is to spec.  the bounty hunter had not commented within several days.   and it correctly created conversation about the timeline between the hunter and the funder

@lazaridiscom that message from gitcoinbot is to spec. the bounty hunter had not commented within several days, and thus gitcoinbot commented with a reminder. and it helpfully created conversation about the timeline between the hunter and the funder

Yeah, and "helpfully" the bot posts the message twice(!).
Excellent Design!

bless you, i hope you are able to find peace 🙏

The double messaging continues, despite your "fix":
https://github.com/utunga/homesafe/issues/1#issuecomment-495286234
https://github.com/utunga/homesafe/issues/1#issuecomment-495286290
Fix it (or better: kill this amateurish bot) and i may find peace.

FWIW as creator of the issue you reference I didn't find the contributions of the gitcoinbot to be overly 'spammy'. As @owocki said it sparked a useful conversation. Now that I think about it I probably would vote for the gitcoin bot to be a little more polite and a little less inclined to speak up but its only a question of degree. All in all I found  it more useful than not.

Well, you should look at other issues (ssometimes 10 to 20 spam messages in a row). As for here, it is about the double-messaging (spam/bug)..

I probably would vote for the gitcoin bot to be a little more polite and a little less inclined to speak up but its only a question of degree

more or less agree. Reminders should be send via email, and if posted to issues, should be a tiny one-liner, totally neutral in tone.


Fix the CopyPopUpView injection

Creates a new qualifier @DefaultView that solves the CDI ambiguity.

+1 to merge.

I'll try to run it locally and then merge.

+1 to merge.

@csadilek @ederign We really need this merged soon (all workbench webapps are currently broken)

@ederign I've just fixed the alignment issue that you saw.

@manstis  I'm just waiting to "green" from jenkins.

thanks @karreiro

Thanks everyone :-)

I see that this is merged already, but just thought I'd give some feedback in case anyone has time to clean up or enhance this code in the future:

I think @FallbackView would be a better name than @DefaultView (because really it's the implementation you fallback on when there is no @Default).

Similarly defaultCopyPopUpViewProvider would be better named fallbackCopyPopUpViewProvider.

It looks odd now that copyPopUpViewInstance is an Instance while defaultCopyPopUpViewProvider is a ManagedInstance. They should probably both be ManagedInstance.

Max, do we need ManagedInstance? Both instances are created through CDI interfaces so won't the underlying BeanManager handle clean up already? Fallback vs Default is, in my opinion, a matter of personal preference... personally a qualifier to specify an "override" is more clear.. but I'm conscious my opinions are just that: opinions and the way somebody else implements something need not be wrong just different. This sounds like a rant... it's not; so apologies if it reads that way.

Fallback implies, to me, there's an expectation you should provide your own implementation otherwise it'll fallback to an erm.. default ;-) Default suggests to me you could provide your own implementation.. but this should be discussed over a couple if beers :-)

Hey Michael,
Yeah, naming is subjective but I have a less subjective concern I should have been more direct about: both implementations have qualifiers with the word "default" in them (because one is @DefaultView and the other is implicitly @Default). With that in mind, I suggested "fallback" as a remedy but there are any number of other names not involving the word "default" that I would like equally. But if this doesn't bother or confuse others then I can live with that.
Regarding the ManagedInstance it should almost always be preferred over a plain Instance. Using Instance or the bean manager directly provides no automatic clean up; you are taking responsibility for manually managing the life-cycle of any @Dependent beans you instantiate. So Instance is only as good as ManagedInstance if you are certain that every bean you instantiate is @ApplicationScoped/@Singleton.
That being said there is still a potential memory leak (if the beans created by the Instance/ManagedInstance are @Dependent) with the producer in this commit that isn't fixed simply by using ManagedInstance. Assuming the produced beans are dependent, one of these conditions needs to be met for them to be cleaned up:

destroyBean is called on the produced instances.
The bean instance is created by a ManagedInstance which is destroyed.

The reason (2) doesn't happen in this case is because the producer type is @ApplicationScoped, so the field-injected ManagedInstance will be in service as long as the app is running.
Some solutions that fix this:

Use only ManagedInstance and make the producer type @Dependent.
Use only ManagedInstance but instead of injecting them as fields, add them as parameters of the producer method.
Add a @Disposes method in the producer type that calls destroyBean.

The first two solutions are preferable, but the third can be appropriate if there is other clean-up code that should be executed (so probably not in this case).

@mbarkley Thanks for your detail reply :-)
I see your point re: @DefaultView vs @Default (which would be the match for copyPopUpViewInstance). It is a bit confusing.. The same clarification can be obtained if we moved the qualifiers from the "default" to the "custom" i.e. copyPopUpViewInstance is qualified with (for example) @CustomCopyPopup and the implementation in kie-wb-common uses the same qualifier. Use of @DefaultView removed.
@karreiro Could you please consider this discussion; and implement which ever you prefer (as long as we move away from @DefaultView and @Default). Also please consider @mbarkley second option to ensure CopyPopUpView instances are destroyed.

Ok @manstis and thanks @mbarkley :-)


"Remember Upto" field got reset to 100 from 500 and I lost over 300 closed tabs

I'm sorry Tab Wrangler lost your tabs. Can you give any more info about when or how it happened? I want to find the cause so I can ensure it's fixed in the next version.
Which version of TW are you using (see the "About" tab)?
Was the pop up open when this happened?
Any other info you can remember?

v5.0.1
I'm not sure when exactly I lost the tabs. Hadn't worked on  my laptop for a couple of days apart from opening a few tabs maybe.Today, sometime into browsing I saw the 100 and checked the list. I have my recent 100 tabs but lost everything before that. I had around 300-350 tabs wrangled.
Also, I had set a keyboard shortcut which also got reset. Actually this is what made me realise the lost tabs. So the problem could have occured earlier.
Not sure what else I can share. I'll go through my web history once to recover the tabs since most of them were from last week, so its not much of a hassle. Just thought of reporting it, since this cannot happen by user error.
Let me know if you wanna remote.
From: Ross Allen
Sent: Monday, 3 April, 2:17 a.m.
Subject: Re: [jacobSingh/tabwrangler] "Remember Upto" field got reset to 100 from 500 and I lost over 300 closed tabs (#121)
To: jacobSingh/tabwrangler
Cc: Vinit Shah, Author
I'm sorry Tab Wrangler lost your tabs. Can you give any more info about when or how it happened? I want to find the cause so I can ensure it's fixed in the next version.
Which version of TW are you using (see the "About" tab)?
Was the pop up open when this happened?
Any other info you can remember?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks for the details. If the keyboard shortcut was lost, it sounds like your settings were lost and reverted to the defaults. I haven't yet seen this problem, but I will see if I can reproduce it.

I haven't been able to reproduce this while working on Tab Wrangler. If you have more details on the steps to reproduce this, I am happy to take a look. Please reopen if you have more details.


bibtex-parsing edge cases

Some valid BibTeX entries (requires #43/#56):

@foo{} (arbitrary entry type, missing key)
@article(foo=bar) (parenthesis, custom field)

Interesting, both are fatal errors in BibLaTeX while natbib fails gracefully. Is the lack of key in the second example intended?

@article(foo=bar) is actually interpreted with foo=bar as the key based on my tests. The new parser should cover that (as well as custom fields and parentheses). Missing keys as in the first example might be a bit more difficult.


Knex CRUD improvements

Description
This PR brings some general refactoring of the core CRUD model as well as fixes some bugs and adds the ability to order items by a field in ascending or descending order

Added separate input types for create/update and delete mutations
Add order by types and implementation for Postgres
Add some tests for ordering.

Verification
query findNotes {
  findNotes(limit:5, orderBy:{field:"id"}) {
    items {
      id
      title
    }
  }
}

@wtrocki what are we going to do with ?
Is there even any point in having this, since people's REST API will likely not follow this format exactly, and advanced filtering is not possible in this way.

RestDataProvider will be removed. I did not wanted to remove it in my PR to not cause conflicts - I'm avoiding runtime changes now.
Target is to merge both PR's at some point an then start with further refactoring of runtime


connection.tsx - wallet.signTransaction(...) function does not exist.

dapp-scaffold/src/contexts/connection.tsx
At line 257 in connection.tsx there is a function call to wallet.signTransaction(...)
transaction = await wallet.signTransaction(transaction);
VSCode gives me the following tool tip Property 'signTransaction' does not exist on type 'WalletAdapter'. Did you mean 'sendTransaction'?
I go to the definition of wallet which is of type WalletAdapter which is imported from @solana/wallet-adapter-base
The definition there extends WalletAdapaterProps & EventEmitter<WalletAdapterEvents>
WalletAdapterProps  is an interface and is missing the field signTransaction which is being called at the top level await wallet.signTransaction()
I'm new to solana, rust, and react and was wondering if I can get some help here. I was thinking that this was perhaps deprecated but I couldn't find any information.
This is the only issue I'm having after a clean git clone followed by a yarn install.

Closing this issue. This was caused by having an outdated version of @solana/wallet-adapter-base installed


Document how to use the plugin with a compiled-to-js Elm app

Since I'm using Webstorm, i leave it to it to compile the elm code. What I get as a result, obviously, is a js module that i'm including in my webpack config like so:
resolve: {
  extensions: ['.js'],
  alias: {
    Elm: resolve('builds/intellij/scripts/elm-program.js')
  }
}

Right now, a change in elm-program.js causes total reload.

If you don't use elm-webpack-loader to compile Elm, just make sure webpack uses elm-hot to load your compiled js file.
Add this to your webpack config file:
loaders: [
  // ...
  {
      test: /elm-program\.js$/,
      loader: 'elm-hot'
  }
  // ...
]

Yeah.. I already tried that. I still get full reload.

Fluxxu is super-friendly. Thanx to our chat we figured that because the compiled js is an "outside" module we need to pipe it through the elm-hot loader explicitly in the import statement:
import {Main} from "elm-hot!Elm";


Change humanize to use MB vs MiB.

Fixes #1653
Signed-off-by: Jhon Honce<EMAIL_ADDRESS>

The test failure on F29 is because a brand new runc with a bug in it has been used.  I have a PR to fix this and we can rebase when that gets in.

@jwhonce  please rebase and tests should pass

/approve

/lgtm


Reproduce DR learner notebook using chirho.robust

This PR closes #372, #373
Blocked by #459.

 See #474


Update PlentymarketsRestClient.php

Enable return of PDF documents

Thx for your contribution! I'm on vacation at the moment but I'll look into this next week when I'm back.

@ewaldmedia creating a new version now. Thanks!


Export enums from protobuf as values

Currently the following code breaks:
import { BillboardMode } from '@dcl/sdk/ecs'
console.log(BillboardMode.BM_ALL_AXES)

because BillboardMode is re-exported as type only

Only happening when --isolatedModules is provided to typescript


JENKINS-70550 Replace Deprecated IOUtils.LINE_SEPARATOR

Replace deprecated IOUtils.LINE_SEPARATOR with System.lineSeparator()

[x] Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
[x] Ensure that the pull request title represents the desired changelog entry
[x] Please describe what you did
[x] Link to relevant issues in GitHub or Jira
[x] Link to relevant pull requests, esp. upstream and downstream changes
[ ] Ensure you have provided tests - that demonstrates feature works or fixes the issue

This issue has been resolved together with JENKINS-70550: Content Replace Plugin Eats Trailing Newlines


Server crash on summoning Foliot Lumberjack

Describe the bug
On a server pack, any attempt to summon the Foliot Lumberjack results in a java.lang.NullPointerException crash.
To Reproduce
Steps to reproduce the behavior:
Activate the summoning process for the Foliot Lumberjack.
Expected behavior
The server does not crash.
Screenshots
If applicable, add screenshots to help explain your problem.

System (please complete the following information):

Occultism Version: occultism-1.16.5-1.0.10.jar
OS: Bisect Hosting
Minecraft Version: 1.16.4
Modpack Link and Version, or list of mods:
I can share via Discord if requested.

I found a way to fix this, but out of interest, did you do anything special to obtain the book of binding?
Based on the crashlog the book had no foliot bound to it, which should not usually occur.

Nothing special with the book. Though, before I tried using the book of binding with the Foliot name, I tried to use a book of calling with a different Foliot name. Nothing happened, so I switched back to the book of binding. Once all five items were consumed (not including the book), about 6 seconds later, the server crashed.


Lap times deleted

Proposed new feature or change:
I note in the FAQ a note about not being able to identify deleted laptimes. I just spotted amongst the FIA event and timing PDFs a document that reports on the Race control messages issued during each session (example), and these seem to include references to deleted laptimes:

I don't know if there are any FIA document scrapers out there that publish scrapes of these documents via an API. Gen AI tools such as Claude.AI seem to be able to extract the data into a JSON object easily enough; I'm not sure if locally run models can also extract the data reliably and accuratley into a structured form.

Race control messages are available from the API and this is implemented since quite some time. The documentation is out of date here. When I pull the data, I get
LapNumber                LapTime  Deleted                   DeletedReason
153        1.0                    NaT    False
154        2.0 0 days 00:02:22.483000    False
155        3.0 0 days 00:02:18.215000    False
156        4.0 0 days 00:02:22.341000    False
157        5.0 0 days 00:01:32.386000     True  TRACK LIMITS AT TURN 13 LAP 5
158        6.0 0 days 00:01:29.044000    False
159        7.0 0 days 00:01:28.057000    False

I'll update the docs. I had completely forgotten that this is noted there. Thanks for the hint.

For now, I'll just remove this outdated information from the documentation.
https://docs.fastf1.dev/core.html#fastf1.core.Laps mentions that Deleted and DeletedReason are available as columns of the Laps DataFrame. Like many other things, it is not easy to find.
A big overhaul of the documentation is in the works, and I need to find some way to better communicate the capabilities and limitations of FastF1 there.


[Bug]: Dependent package conflict / base images error

Is there an existing issue for the same bug?

[X] I have checked the existing issues.

Branch name
main
Commit ID
None
Other environment information
OS type: Windows / infiniflow/ragflow-base:v2.0

Actual behavior
No matter on windows or Dockerfile with FROM infiniflow/ragflow-base:v2.0 , I always meet the problem when I  pip install the requirements_dev.txt packages,
28.02 The conflict is caused by:
28.02     The user requested pydantic==2.6.2
28.02     openai 1.12.0 depends on pydantic<3 and >=1.9.0
28.02     zhipuai 2.0.1 depends on pydantic>=2.5.2
28.02     mistralai 0.4.2 depends on pydantic<3 and >=2.5.2
28.02     google-generativeai 0.7.2 depends on pydantic
28.02     groq 0.9.0 depends on pydantic<3 and >=1.9.0
28.02     jina 3.27.2 depends on pydantic<2.0.0

Are you sure that the versions of the dependency packages you have given are correct?
In addition, the Dockerfile base images infiniflow/ragflow-base:v2.0 seems do NOT meet the requirements of the latest code. That's why I would like to RUN pip install -r requirements_dev.txt in the Dockerfile.
Expected behavior

Check dependency package requirements.
Confirm whether the base image infiniflow/ragflow-base:v2.0 meets the latest requirements.

Steps to reproduce
pip install -r requirements_dev.txt

Additional information
28.02 The conflict is caused by:
28.02     The user requested pydantic==2.6.2
28.02     openai 1.12.0 depends on pydantic<3 and >=1.9.0
28.02     zhipuai 2.0.1 depends on pydantic>=2.5.2
28.02     mistralai 0.4.2 depends on pydantic<3 and >=2.5.2
28.02     google-generativeai 0.7.2 depends on pydantic
28.02     groq 0.9.0 depends on pydantic<3 and >=1.9.0
28.02     jina 3.27.2 depends on pydantic<2.0.0

23.30 The conflict is caused by:
23.30     The user requested pycryptodome==3.20.0
23.30     minio 7.2.4 depends on pycryptodome
23.30     volcengine 1.0.141 depends on pycryptodome==3.9.9

Finally, I make it to set up from the source code. Here are my setps:

use conda with python 3.10 but NOT 3.11
resolve dependency conflicts, here it is:

pycryptodome==3.9.9
protobuf==3.20.2
huggingface-hub==0.24.0
mysqlclient==2.2.4
jian==3.10.1
~~fasttext==0.9.2~~
fasttext-wheel==0.9.2

ATTENTION:

this works on my windows and the base image FROM infiniflow/ragflow-base:v2.0
maybe other omissions in the requirements I mentioned above
here is my pip list, which is for reference only

https://github.com/yzztin/ragflow/blob/main/requirements-zzy.txt

Perhaps you will encounter other problems, but since even me this novice succeeded, you certainly can as well. :>


pin linter execution in github actions to specific go version

Issue #, if available:
Description of changes:
Checking if we can set goversion this way for the linter
Testing (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/cherry-pick release-0.8
/cherry-pick release-0.7
/cherry-pick release-0.6

/cherry-pick release-0.8


Fix map.view.center.xyz channel descriptions

From Pigelvy to the cmapi mailing list:

The documentation of map.view.center doesn’t look good to me.
For example, on the left side tree one can see map.view.overlay, map.view.feature, map.view.location, map.view.bounds but when one clicks on them, on the right side the title says map.view.center.overlay/map.view.center.feature/map.view.center.location/map.view.center.bounds
Shouldn’t the left menu dealing with map.view.center messages be hierarchically ordered?

Basically, the tree view should be set to map.view.center, and underneath that should be map.view.center.overlay, map.view.center.feature, map.view.center.location, and map.view.center.bounds

Fixed but not yet pushed to the site

Eric -please push to public


Need a distribution for Windows

We have a distribution for Linux/Unix. Need the same for Windows.

Added test runner script but did not add windows distribution task in Gradle.


Js mini Project : "Key-Sounds"

Is your feature request related to a problem? Please describe.
There should be JS codes for the Key-Sounds project.
Describe the solution you'd like
I want to add js mini project. @keshavsingh4522
Please assign this to me.

I am interested in doing this project.
Thank You


allow to remove MergeCell

PR Details
This PR exposes a method to remove a MergeCell.

Description
I needs to flat all mergecells in excel. I've tried to get the area of a mergecell and use SetCellValue to set each cell in this area but it didn't work.
I think a RemoveMergeCell is needed.

Related Issue
#527
Motivation and Context

How Has This Been Tested

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[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 read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Codecov Report

Merging #528 into master will decrease coverage by 0.18%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #528      +/-   ##
==========================================
- Coverage   96.71%   96.52%   -0.19%
==========================================
  Files          27       27
  Lines        5600     5611      +11
==========================================
  Hits         5416     5416
- Misses        101      112      +11
  Partials       83       83

Impacted Files
Coverage Δ

cellmerged.go
65.62% <0%> (-34.38%)
:arrow_down:

Continue to review full report at Codecov.

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


[npm node] support for custom npm registry

there are two types of custom npm registries:

public: due to geographical or maybe political reasons https://registry.npm.taobao.org/—essentially copy of https://registry.npmjs.org/
private: due to legal requirements (my case) or redundancy reasoning—either npm enterprise or sinopia

affected endpoints:

npm node version integration now supports /npm/https/registry.npm.taobao.org/v/express.svg:-/^\/npm\/v\/(?:@([^/]+))?\/?([^/]*)\/?([^/]*)\.(svg|png|gif|jpg|json)$/
+/^\/npm(?:\/(http(?:s)?)\/([^/]+))?\/v\/(?:@([^/]+))?\/?([^/]*)\/?([^/]*)\.(svg|png|gif|jpg|json)$/

npm license integration now supports /npm/https/registry.npm.taobao.org/l/express.svg:-/^\/npm\/l\/(?:@([^/]+)\/)?([^/]+)\.(svg|png|gif|jpg|json)$/
+/^\/npm(?:\/(http(?:s)?)\/([^/]+))?\/l\/(?:@([^/]+)\/)?([^/]+)\.(svg|png|gif|jpg|json)$/

npm node version integration now supports /node/https/registry.npm.taobao.org/v/express.svg:-/^\/npm\/l\/(?:@([^/]+)\/)?([^/]+)\.(svg|png|gif|jpg|json)$/
+/^\/npm(?:\/(http(?:s)?)\/([^/]+))?\/l\/(?:@([^/]+)\/)?([^/]+)\.(svg|png|gif|jpg|json)$/

downloads related endpoints are not affected due to custom https://api.npmjs.org/ server, which complicates stuff. But apart from complexity, there are several reasons not to modify these endpoints:

for public registries its (maybe) okay to use default registry
for private registries this downloads metric is (maybe) unnecessary

P.S. inspiration for /npm(/:protocol/:host)/… is jenkins endpoint

Thanks for this!
I appreciate that you're modeling it the Jenkins badge. Since #1186 it's possible to pass arbitrary parameters through the query string. I'd like to use this for exception / rarely used options, long options, or options which require munging to fit into the URL. This seems like a good place for it. Basically, something like ?registry_uri=.
The benefit is a more consistent and understandable URL pattern. The resultant URL isn't quite as pretty and takes a bit of effort to encode, but it's also not going to be used by many people, and after #701 it should be really easy to encode one of these through the UI.
You can see a few examples in server.js where an array of query string field names is passed in. This is from the dynamic JSON badge:
queryParams: ['uri', 'query', 'prefix', 'suffix'],

no problems. i'll change stuff. just to be on the same page. here this is the list to change/add:

[ ] query params instead of complicating the url pattern
[ ] remove extra examples
[ ] tests

and thanks for the feedback

Yup, that's it. And if you keep the helper function, please move it to a different file.
Thanks!

only tests left

done

because of added queryParams and following added indentation to handler diff looks messy.
to make things simpler to review use diff url without whitespace diff https://github.com/badges/shields/pull/1277/files?w=1

tests are ok, its just ping to chinese registry is long and mocha is timing out.
what do you think?

Excellent! Thanks for the diff; it's easy enough to read. This looks great.
Only one thing I'd ask for, is tests of the license badge. It seems think we have any now. The reason it seems important is that all that code was in flight. You could just pick a package and hard-code the expected value.
Is there another mirror that's faster to reach from the U.S.? Alternatively we could just leave the flaky test. I've solved the timeout issue in IcedFrisby; it's waiting to be shipped in 2.0.

sure, i'll add tests for license service

about registries

its complicated. there used to be some semi-supported mirrors. Nowadays its only registry.npm.taobao.org and r.cnpmjs.org with average ping 275ms and 425ms respectively. So if taobao isnt a good fit, then r.cnpmjs isnt either.

~/projects/oss/shields feat/custom-npm-registry
☯ npm i -g nrm -dddddd
npm info it worked if it ends with ok
npm info using<EMAIL_ADDRESS>npm info using<EMAIL_ADDRESS>npm http fetch GET 304 https://registry.npmjs.org/nrm 938ms (from cache)
npm http fetch GET 304 https://registry.npmjs.org/async 96ms (from cache)
npm http fetch GET 304 https://registry.npmjs.org/ini 125ms (from cache)
npm http fetch GET 304 https://registry.npmjs.org/node-echo 125ms (from cache)
npm http fetch GET 304 https://registry.npmjs.org/extend 129ms (from cache)
npm http fetch GET 304 https://registry.npmjs.org/request 146ms (from cache)
npm http fetch GET 304 https://registry.npmjs.org/npm 228ms (from cache)
npm http fetch GET 304 https://registry.npmjs.org/only 254ms (from cache)
npm http fetch GET 304 https://registry.npmjs.org/open 784ms (from cache)
npm http fetch GET 304 https://registry.npmjs.org/commander 797ms (from cache)
npm http fetch GET 304 https://registry.npmjs.org/mkdirp 60ms (from cache)
npm http fetch GET 304 https://registry.npmjs.org/coffee-script 139ms (from cache)
npm http fetch GET 304 https://registry.npmjs.org/jistype 990ms (from cache)

from this log default registry is registry.npmjs.org. but there is as well registry.npmjs.org which is basically a dns alias to default one:
☯ ping registry.npmjs.org -c 11
PING a.sni.fastly.net (<IP_ADDRESS>): 56 data bytes
avg 48ms

☯ ping registry.npmjs.com -c 11
PING a.sni.fastly.net (<IP_ADDRESS>): 56 data bytes
avg 11 ms

i'll use registry.npmjs.com, because its not default one, but fast

yay, registry.npmjs.com fixed it

done

if anything left to fix, this pull-request should be fine. @paulmelnikow thanks for your input, feedback and patience

Thanks for all your work!

hurray

thanks

will you release it to https://shields.io/ anytime soon?

I can't; I don't have access. Deploys usually happen every 1–3 weeks. Thaddée, who has limited time on this project, is the only sysadmin. He's working on giving me access to deploy and logs, but doing so is complicated because the hosting account (and maybe the servers too) are shared with other services he runs.


Summary Evaluation is not visible on Experiments WebUI

I implemented this following the video, but the f1-score is not displayed on the WebUI.
https://www.youtube.com/watch?v=zMgrHzs_cAg
Looking at the evaluators, the calculations are being done, the functions are executed properly, and the values are calculated correctly. However, it is not being displayed as s chart.

Could you share please share code?

@hinthornw
Sure, I used summary_evaluators.
def precision(runs: list, examples: list):
    true_positives = sum([1 for run, example in zip(runs, examples) if run.outputs["is_target"] == example.outputs["is_target"]])
    false_positives = sum([1 for run, example in zip(runs, examples) if run.outputs["is_target"] != example.outputs["is_target"]])
    return {"score": true_positives / (true_positives + false_positives), "key": "precision"}

def recall(runs: list, examples: list):
    true_positives = sum([1 for run, example in zip(runs, examples) if run.outputs["is_target"] == example.outputs["is_target"]])
    false_negatives = sum([1 for run, example in zip(runs, examples) if run.outputs["is_target"] != example.outputs["is_target"]])
    return {"score": true_positives / (true_positives + false_negatives), "key": "recall"}

def f1_score_summary_evaluator(runs: List[Run], examples: List[Example]) -> dict:
    true_positives = 0
    false_positives = 0
    false_negatives = 0
    for run, example in zip(runs, examples):
        # Matches the output format of your dataset
        reference = example.outputs["is_target"]
        # Matches the output dict in `predict` function below

        prediction = run.outputs.get("is_target",None)
        if prediction == None:
            false_negatives += 1
            continue

        if reference and prediction == reference:
            true_positives += 1
        elif prediction and not reference:
            false_positives += 1
        elif not prediction and reference:
            false_negatives += 1
    if true_positives == 0:
        return {"key": "f1_score", "score": 0.0}

    precision = true_positives / (true_positives + false_positives)
    recall = true_positives / (true_positives + false_negatives)
    f1_score = 2 * (precision * recall) / (precision + recall)
    return {"key": "f1_score", "score": f1_score}

results = evaluate(
    lambda inputs: label_text(inputs["body"]),
    data=client.list_examples(dataset_name=dataset_name, as_of="latest"),
    evaluators=[correct_label],
    summary_evaluators=[f1_score_summary_evaluator,precision, recall],
    experiment_prefix="pickup-gemini-subset",
)

@hinthornw
The code used in the video is same as the current version?
https://www.youtube.com/watch?v=zMgrHzs_cAg

Same issue, the metrics are not being displayed in the UI despite results._summary_results shows the EvaluationResult with the metrics.

We're having the same issue.
Official documentation says the summary evaluation results should be visible in the experiments UI, but they're not
https://docs.smith.langchain.com/how_to_guides/evaluation/evaluate_llm_application#use-a-summary-evaluator

Also the summary evaluation results aren't really exposed in an official manner in the Langsmith SDK. The only way to programmatically access them is to use results._summary_results as said by @whatever-afk, which is not a public API

@0d431 we didn't have any issue returning multiple evaluation results though. Here's what our summary evaluator function looks like:
def evaluate_precision_recall(
    runs: Sequence[Run], examples: Sequence[Example]
) -> EvaluationResults:
    true_positives_count = 0
    detected_count = 0
    reference_count = 0

    for example, run in zip(examples, runs):
        reference = set((example.outputs and example.outputs.get("data")) or [])
        detected = set((run.outputs and run.outputs.get("data")) or [])
        true_positives = detected.intersection(reference)

        true_positives_count += len(true_positives)
        detected_count += len(detected)
        reference_count += len(reference)

    precision = true_positives_count / detected_count
    recall = true_positives_count / reference_count
    f_score = 2 * precision * recall / (precision + recall)

    return EvaluationResults(
        results=[
            EvaluationResult(
                key="precision",
                score=precision,
            ),
            EvaluationResult(
                key="recall",
                score=recall,
            ),
            EvaluationResult(
                key="f_score",
                score=f_score,
            ),
        ]
    )

@0d431 i think this works only for a single run i.e when you are evaluating the output vs the reference for each, the code @mobiware actually output the results as expected
{
  "results": [
    {
      "key": "precision",
      "score": 0.8974358974358975,
      "evaluator_info": {}
    },
    {
      "key": "recall",
      "score": 0.9722222222222222,
      "evaluator_info": {}
    },
    {
      "key": "f_score",
      "score": 0.9333333333333333,
      "evaluator_info": {}
    }
  ]
}

However, the issue is we are still unable to see the results in the UI.

+1. Facing the same issue as well. Summary Metrics can only be viewed by accessing results._summary_results in code, but the metrics don't show up in the web UI.

In fact as of today I can now see the summary evaluation in Langsmith web UI, including for past Experiments !
@hinthornw should we open a separate ticket for the other issue mentioned in this ticket, i.e. the fact that after running evaluate() the summary results are available only on ExperimentResults._summary_results which is a private attribute. It would be nice to have a public accessor to this attribute

@mobiware What did you change that makes the summary evaluation show up in the LangSmith Web UI?

Hi all! There were a few versions of langsmith where the summary metrics were no longer being reported. If you upgrade to any of the past 5 or so versions, this should be fixed.

I believe this was resolved in the aforementioned changes. LMK if you believe otherwise

Having the same issue.
@hinthornw by "upgrade langsmith" are you referring to updating the local version of the langsmith package? That doesn't seem to fix the UI issue, the summary score only appears as a column in the list of experiments for a dataset, not in the detailed view though. In the video shared earlier, the f1 score appears below the list of runs, but that's not the case when running the code locally.


Add vue-cloneya

https://github.com/ridaamirini/vue-cloneya
Category: UI Components/Miscellaneous

close due to merged it

mute the thread
At 2018-05-18 23:11:35, "Rida Amirini"<EMAIL_ADDRESS>wrote:
https://github.com/ridaamirini/vue-cloneya
Category: UI Components/Miscellaneous
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


[BUG]State for powered down node stays in-progress

Describe the bug
State of a powered down node is 'in-progress'. It should be 'down'
To Reproduce
Steps to reproduce the behavior:
Power down an Active node on a multi cluster env
Expected behavior
The State should change from Active to down
Support bundle

Environment:

Harvester ISO version: main
Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630): ipxe vagrant env

Additional context
Add any other context about the problem here.

Is this a UI area issue? When I click View in API, in-progress occurs in metadata.state. However, state field is not in ObjectMeta. Probably, UI should check Ready condition in status. For a shutdown node, Ready condition shows Unknown.
metadata: {
  # ...
  state: {
    "error": false,
    "message": "Kubelet stopped posting node status.",
    "name": "in-progress",
    "transitioning": true
  }
}

nope, actually this is a backend issue as the harvester node API has a different status in contrast with the rancher's v1/nodes, found this is identical to https://github.com/harvester/harvester/issues/1272, close this as a duplicate issue.


Add onScrollBegin and onTouchEnd for Android

Is it a bugfix ?
Yes
https://github.com/leecade/react-native-swiper/issues/489
Is it a new feature ?
No
Describe what you've done:
The swiper is now listening on Android ViewPagerAndroid's onPageScrollStateChanged event.
How to test it ?
Just use onScrollBeginDrag.

Thanks for this PR! I confirm this fixes onScrollBeginDrag not being called in my app. Merged in my own fork until @arribbar wakes up and this repo gets a npm version bump, haven't had one since a year ago now.


[Bug] spark connector, when doris table is non-paritioned.   where clause is useless

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
1.2.9
What's Wrong?
doris表是非分区表的情况下，用spark connector去读取到df以后。无法使用where 算子。写了也不生效，变成了全部读取，然后全表写入。

What You Expected?
组件bug。
How to Reproduce?
1.准备一个非分区的doris表。
2.插入100条数据
3.使用spark doris connector读取到df
4.随便加一个where算子，带上条件
最后会发现条件没用
Anything Else?
No response
Are you willing to submit PR?

[x] Yes I am willing to submit a PR!

Code of Conduct

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

Which Spark connector version are you using?

Which Spark connector version are you using?

1.3.1

Which Spark connector version are you using?

1.3.1
org.apache.doris spark-doris-connector-3.2_${spark.scala.version} 1.3.1

This is due to a query execution error in Doris, not the connector. The problem occurs in versions 1.2.8 and 1.2.9. You can upgrade to version 2.x to solve this problem.

Which Spark connector version are you using?

1.3.1
org.apache.doris spark-doris-connector-3.2_${spark.scala.version} 1.3.1

This is due to a query execution error in Doris, not the connector. The problem occurs in versions 1.2.8 and 1.2.9. You can upgrade to version 2.x to solve this problem.

我也遇到这个问题了，doris.filter.query只对分区列生效，
spark.read.format..xxxx.load后接上.cache()再createOrReplaceTempView，
然后视图上执行where 可以正常过滤掉数据.
如果把cache()去掉，在视图上执行where过滤依然不生效，
猜测应该是把where条件下推到doris引擎中去执行导致

Which Spark connector version are you using?

1.3.1
org.apache.doris spark-doris-connector-3.2_${spark.scala.version} 1.3.1

This is due to a query execution error in Doris, not the connector. The problem occurs in versions 1.2.8 and 1.2.9. You can upgrade to version 2.x to solve this problem.

我也遇到这个问题了，doris.filter.query只对分区列生效， spark.read.format..xxxx.load后接上.cache()再createOrReplaceTempView， 然后视图上执行where 可以正常过滤掉数据. 如果把cache()去掉，在视图上执行where过滤依然不生效， 猜测应该是把where条件下推到doris引擎中去执行导致

Thx, 尝试了一下。加上cache可以暂时避免这个问题。 核心还是得升级doris版本

Which Spark connector version are you using?

1.3.1
org.apache.doris spark-doris-connector-3.2_${spark.scala.version} 1.3.1

This is due to a query execution error in Doris, not the connector. The problem occurs in versions 1.2.8 and 1.2.9. You can upgrade to version 2.x to solve this problem.

我也遇到这个问题了，doris.filter.query只对分区列生效， spark.read.format..xxxx.load后接上.cache()再createOrReplaceTempView， 然后视图上执行where 可以正常过滤掉数据. 如果把cache()去掉，在视图上执行where过滤依然不生效， 猜测应该是把where条件下推到doris引擎中去执行导致

That's right. The connector has correctly pushed the predicate down to the scan operator, and the exception occurred on the Doris side. Caching the dataframe returned by load is a solution. Adding predicates to the cached dataframe will be processed by Spark.


No where it is specified how to query for say bugs found in a given build using the latest link types found in build, integrated in build and integrated in release

[Enter feedback here]

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

ID: 6a015352-1835-70c9-9bff-7f98e5b0bf1b
Version Independent ID: 1183c0d4-1948-ee63-7c3f-b2c7b9bdef3b
Content: Frequently Asked Questions (FAQs) about Queries in Azure Boards and Azure DevOps - Azure Boards
Content Source: docs/boards/queries/query-faqs.yml
Service: azure-devops-boards
GitHub Login: @chcomley
Microsoft Alias: chcomley

@vijay-gowda -- Vijay, thank you for your feedback.  Please consider these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow

@chcomley -- Chrystal, please look into this potential documentation issue.

Unfortunately, we have been unable to address this issue in a timely manner. We sincerely apologize for the delay and are closing this issue. As we phase out GitHub issues for this repository, we encourage you to propose changes using public PRs or submit feedback through our updated control at the top of the page. For more information on our feedback systems, see https://aka.ms/contentuserfeedback.


Updated Lithuanian layout pack, merged it into master

This PR contains the following changes:

Adds the AOSP Lithuanian dictionary
Changes the keyHeight for the default Lithuanian layout from small to normal (I found small quite uncomfortable for fast typing)
Drops the separate Lithuanian branch by merging it into master
Updates all graphic assets

I have released this update in Google Play already (https://play.google.com/store/apps/details?id=org.herrlado.ask.languagepack.lithuanian) and marked it as beta for now.
Regarding dictionaries: I've left both the AOSP and the one we used originally in the source, but with AOSP, our originally produced dictionary might actually be unnecessary (especially since it's isn't really Apache licensed).
The release process currently expects a key named "lithuanian" in the key store. I used Lado's key to sign it (key fingerprint is A0:C8:20:B7:F4:42:8B:0A:E4:C1:B0:15:F5:FF:2F:E9:36:A9:29:0F), which I suspect is probably same as the key used for Czech language pack. @menny I can forward the key to you if you don't have it.

@menny is there anything that's preventing you from reviewing PRs?

@rimas-kudelis  it looks good, and I'll gladly merge this. Just a question about the pack ownership (see inline)

@menny I can transfer ownership of the pack to you or to @friesenkiwi. I'd love to keep access to publish it myself too if possible, but if not, that's fine by me.
Also, I commented about the signing key in my initial comment. Do you have it already or should I send it your way?

@menny ping

So, if you want me to publish, please email me the signing key to<EMAIL_ADDRESS>

I can't give access to publish ATM, but it will be automatically published once hitting master

Also, until I have the keys and ser them up correctly in the secrets mechanism, I can not publish this. Can you switch the publish flag to false?

@menny I've sent the key your way. Assuming that the actual transfer of ownership of the package on Google Play is the only thing that's left now, I guess we could wait with merging until that is done and avoid flipping the flag? Would that work for you?

sure. Follow the guide here: https://support.google.com/googleplay/android-developer/answer/6230247?hl=en
I'll send you my details over email

@menny could you resend me your details (play console email address and order id)? I was unable to transfer the ownership of the package using the details you provided me with previously. :(

@rimas-kudelis I replied to your email about the ownership. Let me know if the new details work for you.

Thanks! The app transfer is now pending.

got it: https://play.google.com/store/apps/details?id=org.herrlado.ask.languagepack.lithuanian

Now all that's left is to delete the Lithuanian branch. :)


Pylance reports problems in interactive window without a way to ignore them

Environment data

Language Server version: 2024.3.1 (pyright version 1.1.351, commit 87171fbf)
OS and version: Windows 10 22H2 build 19045.4170
Python version (& distribution if applicable, e.g. Anaconda): 3.11.8, installed via conda

Basically https://github.com/microsoft/vscode-jupyter/issues/8289.
The last comment there indicated that if the issue still occurs it should be reported under the appropriate repository.

Chatted with Don about this. The code that suppressed diagnostics in https://github.com/microsoft/vscode-jupyter/issues/8289 was in the jupyter-lsp-middleware which we no longer use. Also, he felt that the decision of whether to suppress diagnostics in the IW should be handled per language. So:

We need to decide if we really want to suppress diagnostics in the IW -- not return them at all.
If we return diagnostics, we need to understand why the squiggles are not appearing.
If we return diagnostics, we need to ensure that they can be suppressed by users via the ignore config setting.

I think it's fair to display diagnostics on the input box only, but not on executed blocks/cells as one can't go back to fix issues.
#3 is also a great point, I'm not sure what would be the value we'd support in this case though. I'm also unsure if we should handle IW and the Native REPL differently?
cc @cwebster-99


Maybe can add passes new optition: else on new line

Like phpstorm format

Thanks @awebc I'm not planning introducing new rules at this moment, as maintaining the package as it's is my main goal.


[feature]added sorting algorithm

Added a sorting algorithm.

Description

Resolves #(issue number)
What does your PR belong to?

[ ] Website
[x] Snippets
[ ] General / Things regarding the repository (like CI Integration)

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (non-breaking improvement of a snippet)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have checked that the changes are working properly
[x] I have checked that there isn't any PR doing the same
[x] I have read the CONTRIBUTING document.

@kriadmin Can you help me find why travis ci is not able to build readme.md.

@sachans Due to (board) exams I am busy for this month. I will review it and merge it when I come back.

Thanks for contributing :+1:

But it is not showing that you merged

Oops sorry. I merged it now


Starter-kit: Comments not showing

Hi! I am just starting with the starter-kit for enterprise, and I am a bit confused of how to implement the comments, since I can see that there are files dedicated for comments, but when I am creating an article, it doesn't show the comment section (even though it is not disabled when publishing).
What should I do to enable it?

Hi there!
Thank you for reaching out and using the starter-kit for your enterprise blog!
Let's go through a couple of scenarios to understand why this might be happening:
Scenarios for the Comments Section Not Showing:

Comments Disabled in Settings:

Normally, if comments are disabled during the publishing process, the comments section won't be rendered. Since you've mentioned that comments are not disabled, we can rule this out as a cause.

No Comments on the Article:

The comments section is set to display only if there are existing comments on the article. The condition for rendering the comments section in the enterprise theme is:{!post.preferences.disableComments && post.comments.totalDocuments > 0 && <PostComments />}

This means that if there are no comments on the post yet (post.comments.totalDocuments is 0), the comments section won't appear.

If you have confirmed that the post has comments and you are still not seeing the comments section, we would need to investigate a bit deeper. Please provide the contents of your .env.local file so we can check for any misconfigurations or other issues that might be affecting the comments display.
Best regards,
Harsh

Hi @Witomi,
I hope you're doing well.
I wanted to follow up on the above questions.
Looking forward to your response.


feat(ts): provide type checking through fork-ts-checker-webpack-plugin

Types of changes

[ ] Bug fix (a non-breaking change which fixes an issue)
[x] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description
This PR improves Nuxt TS support by adding a build option useForkTschecker that enable type checking thanks to fork-ts-checker-webpack-plugin.
Type checking makes you able to know if there is TS related errors within your code, which means that without it you may have Nuxt Webpack compilation succeed, but the errors only reported at runtime and not in an elegant way.
The type checker will check your code each time you make changes. In other terms, it's trigerred after each new Webpack compilation.
Usage
// nuxt.config.js
export default {
  build: {
    useForkTsChecker: true
  }
}

useForkTsChecker can either be a boolean or an object. If it's an object, it will override the plugin options
// nuxt.config.js
export default {
  build: {
    useForkTsChecker: {
      silent: true,
      tslint: false
    }
  }
}

Example of an error reported by the type checker

Project requirements to use this feature
The fork-ts-checker-webpack-plugin is used/enabled only if the user project meets these requirements :

useForkTsChecker option is enabled (true) in the Nuxt configuration file
ts-loader is configured with the transpileOnly mode (to not have double type checkers)
fork-ts-checker-webpack-plugin (and also tslint) devDependencies are installed

If the two first rules are fulfilled but not the latter, a warning will let you know that your project is missing the dependencies required :

Documentation
Documentation related to this PR will come later as a whole TS support documentation.
Checklist:

[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly. (PR: #)
[x] I have added tests to cover my changes (if not applicable, please state why)
[x] All new and existing tests are passing.

Codecov Report

Merging #4611 into dev will decrease coverage by 0.13%.
The diff coverage is 63.63%.

@@            Coverage Diff             @@
##              dev    #4611      +/-   ##
==========================================
- Coverage   90.88%   90.75%   -0.14%
==========================================
  Files          61       61
  Lines        2206     2217      +11
  Branches      536      540       +4
==========================================
+ Hits         2005     2012       +7
- Misses        182      185       +3
- Partials       19       20       +1

Impacted Files
Coverage Δ

packages/config/src/config/build.js
85.71% <ø> (ø)
:arrow_up:

packages/common/src/utils.js
97.02% <40%> (-1.24%)
:arrow_down:

packages/webpack/src/config/base.js
94.5% <83.33%> (-0.79%)
:arrow_down:

Continue to review full report at Codecov.

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

@pi0 Thanks for your review ! I'm going to make the needed changes :)

Kind of, and you can use the return value of resolveModule if it exists, then the require won’t resolve all the paths again.

Wait before merging, I need to test something :P

@pi0 @clarkdo Alright, need final review & LGTM :)

@clarkdo @pi0 I moved the plugin in client.js as suggested 🙂
(I forced push to revert a build-failing commit while keeping Git history clean, I tried to remove tslint devDependency but was not a good idea RIP 😅)

🎊

@kevinmarrec one more thing, with eslint it was recommended to also add eslint-loader in nuxt-config.js, is there any value of adding tslint-loader i.e.
      if (isDev && isClient) {
        config.module.rules.push({
          enforce: "pre",
          test: /\.(ts|js|vue)$/,
          loader: "tslint-loader",
          exclude: /(node_modules)/
        })
      }

@husayt This behavior is likely what ForkTsChecker does if you configure the tslint option of it :)
So no tslint-loader needed, ForkTsChecker will check if your files have lint errors whenever you change them 😉


Update to Truth 1.0 and downgrade to non-beta JUnit

@lesv

Failures do not appear related:
------------------------------------------------------------
- testing kms
------------------------------------------------------------
[ERROR] src/main/java/com/example/Asymmetric.java:[120] (naming) AbbreviationAsWordInName: Abbreviation in name 'decryptRSA' must contain no more than '2' consecutive capital letters.
[ERROR] src/main/java/com/example/Asymmetric.java:[138] (naming) AbbreviationAsWordInName: Abbreviation in name 'encryptRSA' must contain no more than '2' consecutive capital letters.
[ERROR] src/main/java/com/example/Asymmetric.java:[200] (naming) AbbreviationAsWordInName: Abbreviation in name 'verifySignatureRSA' must contain no more than '2' consecutive capital letters.
[ERROR] src/test/java/com/example/AsymmetricIT.java:[135] (naming) AbbreviationAsWordInName: Abbreviation in name 'testRSAEncryptDecrypt' must contain no more than '2' consecutive capital letters.
[ERROR] src/test/java/com/example/AsymmetricIT.java:[144] (naming) AbbreviationAsWordInName: Abbreviation in name 'testRSASignVerify' must contain no more than '2' consecutive capital letters.
[ERROR] src/test/java/com/example/AsymmetricIT.java:[156] (naming) AbbreviationAsWordInName: Abbreviation in name 'testECSignVerify' must contain no more than '2' consecutive capital letters.
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:3.0.0:check (default) on project kms-samples: You have 6 Checkstyle violations. -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException

 Testing failed: Maven returned a non-zero exit code.

This is one example of why I don't like checkstyle presubmit checks on Github.

This is one example of why I don't like checkstyle presubmit checks on Github.

The check is there for a reason, the problem is that it was somehow submitted in a broken state to begin with.


performance redis exploratory

not sure these changes are needed here
it doesn't appear that rqworker are used

not sure these changes are needed here
it doesn't appear that rqworker are used

We [should] support both cases in open source label studio.
Users can start it as simple way as possible, without additional services, like redis, but also, users should have the ability to set up connection with redis and improve performance that way.


group x list autocomplete wrong

When you type /lp group  list + TAB it's not correctly autocompleting it. It's becomming  info instead of  listmembers

Duplicate of #419?

Oh, or you just excluded that in your example?
This is likely because "list" is an alias of the info command.
https://github.com/lucko/LuckPerms/blob/master/common/src/main/java/me/lucko/luckperms/common/commands/CommandManager.java#L344

Aha. It would be great if you can remove the alias :)


Added simple Jenkinsfile for presubmits

This change is

For some reason, I cannot get jenkins to run any PR from manager. Looking into it now.

Reviewed 4 of 4 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


feat: add Georgian README translations

Description
Added a Georgian translated version of the README.md file.
Contribution Checklist:

[ ] The pull request only addresses one issue or adds one feature.
[ ] The pull request does not introduce any breaking changes
[ ] I have added screenshots or gifs to help explain the change if applicable.
[ ] I have read the contribution guidelines.
[ ] Create an issue and link to the pull request.

Note: Keeping the PR small and focused helps make it easier to review and merge. If you have multiple changes you want to make, please consider submitting them as separate pull requests.
Publishing to New Package Managers
Please see here for more information.

Merged! Thanks @LeoGrambert !


Remove Selection

Hi, is there a way to unselect node programmaticly?
I was looking at tree model documentation, but I haven't seen one, that would allow me to do that.

Doesn't matter, I made it work using:
   @ViewChild("angularTree") public treeComponent: TreeComponent;

   removeSelection() {
       this.treeComponent.treeModel.getFocusedNode().setIsActive(false);
   }