[Bug]: Class "Tempest\EventBus\EventBus" does not exist when running commands

Tempest Version
1.0-alpha1
PHP Version
8.3
Operating System
Windows
Description
Hi.
Currently I have a symfony Console to manage my application (that was developed using a custom framework). But after watching some videos of Brent I thought that maybe using tempest console would be nice. I've followed the commands here: https://tempestphp.com/console/.
First, --stability alpha didn't work.  Composer doesn't seem to support that. At least on windows. But I was able to workaround it setting it in the composer.json. But when running:
./vendor/bin/tempest hello:world Brent
I've got this error:
Fatal error: Uncaught ReflectionException: Class "Tempest\EventBus\EventBus" does not exist in www\tempest-console-demo\vendor\tempest\reflection\src\ClassReflector.php:28
Stack trace:
#0 www\tempest-console-demo\vendor\tempest\reflection\src\ClassReflector.php(28): ReflectionClass->__construct('Tempest\\EventBu...')

Steps to Reproduce

Install tempest/console in a empty project
Create a command class
Run the command

When I perform these steps, I get no error, but I'm running dev-main. Are you running alpha.2 or dev-main?
Alpha 2 does have some issues with installing individual components, which will be fixed in alpha 3. Could you try requiring dev-main instead and retry?
Also, for the --stability alpha flag: that'll be removed from the docs soon :)

Gonna close this one, but feel free to reopen with followup if needed.

Sorry it took me so long. But using dev-main worked. And it worked great.
I think if it can work without the discovery part and slim down the dependencies a bit, it can take the crown from symfony/console as the solution for command apps.  @brendt and the team accomplished something really great here. Congratulations.

Discovery is totally optional, btw, but it's nowhere documented yet how to do it, and it will probably take a couple of steps, but it's definitely possible.
Is there a reason you don't want discovery?

For this project, I would be more confortable with a little bit less magic. And I guess there is a penalty (performance wise) attached to it. The latter probably is not a problem because it is cached anyway.
But in the future I would like to control which commands could be added to the app based on the user or a feature flag, and discovery would be in the way.

Fair enough, it's not super high on the priority list to make it as smooth as possible without discovery, but it's definitely possible and you can already hack it together today. The only thing you really need to be aware of is how the kernel is bootstrappen (see \Tempest\Core\Kernel). You can create your own version without loading discovery, and it should work fine.


[Feature] Allow a transparency toggle/slider for pop-out video windows

Is your feature request related to a problem? Please describe.
While not inherently a problem, my primary use for discord is video calling; as such I use the pop-out window for calls and window sharing, because of that the popout window in my use case tends to be alot larger so I can actually see what's being shared, but I don't need to view the content in 100% opacity since it blocks a large portion of the screen.
Describe the feature you'd like
The ability to introduce transparency to the discord pop-out window.

This can be done with CSS and shouldn't be integrated into the client.

Give this a try.
.pictureInPictureWindow-1B5qSe {
    opacity:.5;
}

Yes more things should be addons rather than client integrations.

Yes more things should be addons rather than client integrations.


Unexpected Token ) but passes test

Challenge Waypoint: Target HTML Elements with Selectors Using jQuery has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/45.0.2454.99 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

jQuery Playground

#left-well

        <button class="btn btn-default target" id="target1">#target1</button>
        <button class="btn btn-default target" id="target2">#target2</button>
        <button class="btn btn-default target" id="target3">#target3</button>

#right-well

        <button class="btn btn-default target" id="target4">#target4</button>
        <button class="btn btn-default target" id="target5">#target5</button>
        <button class="btn btn-default target" id="target6">#target6</button>

As you can see, the tests are passed and in this picture I copy pasted to make sure there wasn't an error from me writing the code myself. I checked similar problems and tried putting in ; where it might be appropriate according to javascript but the same error keeps occuring. Thanks guys!

The $(button)code should be within the body of the function, not the parameters. Please ask for help in the Gitter chat before opening an issue!

@KingDad The test are not passing in your photo.

@KingDad thanks for posting this issue!
Your code in the script tag is incorrect:
$(document).ready(function($("button").addClass("animated bounce")) {

});

it should be:
$(document).ready(function() {
    $("button").addClass("animated bounce");
});

If you have problems with your code, please, post a message in our Gitter chat room first, thanks.
Closing this issue.

Please review the Guidelines for Contributing, thank you!.


Call out that transition styles must be nested in floating

Otherwise the rendering won't work correctly.

Thank you; can you use the Notice component to place this in? And maybe wording (with an example) similar to here: https://floating-ui.com/docs/useFloating#transform

0475c02d8129d364e0073b19e66f27d50e8631ad


⚠ Bofrost.at has degraded performance

In 21545e0, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1433 ms

Resolved: Bofrost.at performance has improved in 3ad0b23.


Update macos-12 to macos-13 (latest macos to still support intel).

GitHub runner deprecated macos-12 (see https://github.com/actions/runner-images/issues/10721). macos-13 is the latest image to still support x86.

Release commit using the update to mac-os13 passed in https://github.com/NomicFoundation/edr/pull/729 (see run https://github.com/NomicFoundation/edr/actions/runs/11896189854/), so I am comfortable merging this now.

my azure release was working fine till yesterday, I run now I am getting
##[Error 1]
The macOS-12 environment is deprecated, consider switching to macOS-13, macOS-14 (macos-latest) or macOS-15. For more details, see https://github.com/actions/runner-images/issues/10721
it seems it's relevant to this issue
is this a bug or I need to add a capability or something to my Agent?

@MHatemAbdelhamid Please check https://github.com/actions/runner-images/issues/10721, which includes the ways of mitigation. This is the repository for the Ethereum Development Runtime.
If you're curious to see how I tackled the changes needed for the EDR repo, you can check the files changed. Good luck!


remove makefile in favor of package.json scripts; CSR -> CS

:confetti_ball: - LGTM!

Simplified the build task

:+1: - down with the Makefile!


how to install? newbies questions

HI Tom,
I download file as zip file, and installed the note.js. But, still have zero clue about the Usage section you mentioned.
with node, how to install? If you don't mind, could you please provide step by step instructions?
Really Appreciated
with node/browserify
npm install --save tokml
otherwise:
wget https://raw.github.com/mapbox/tokml/master/tokml.js
as a binary:
npm install -g tokml
tokml file.geojson > file.kml
tokml < file.geojson > file.kml

See https://docs.npmjs.com/getting-started/installing-node & https://docs.npmjs.com/getting-started/installing-npm-packages-globally


Fix #2 esm build and update deps for npm audit

Current esm build is broken.  Was not able to resolve the component files and .ts files due to rollup config being broken.  I have also updated dependencies marked as a security issue with npm audit.
This should fix issue #2

@obj63mc Have you checked it thoroughly. I'm currently not able to check the PR. If you can confirm that it has been checked thoroughly at your end (manually), then I can merge the PR for the benefit of users facing Issue #2 at my own risk based on your confirmation (trust on you).

I have been able to build it, and use it on a project currently pulling from my fork and simply using the following -
    import { AnimateInViewDirective } from "@neeravp/vue-3-animate-in-view";
    ...
    app.directive("animate-inview", AnimateInViewDirective);

    ... in template/html -
     v-animate-inview="'fade'"

and all works fine.
If you look at the esm file you will see the one distributed with your package basically has none of the code other than the exports.  Where as with mine built you can see it actually includes the code needed.


chore: centralize statuswin icon in tests

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


Improve convergence time of GJK

Improve tetrahedral simplex's origin test.

https://github.com/Godlike/Arion/projects/1 Migration.


[SPARK-31842][TEST] DAGSchedulerSuite: Extract method for pattern of runevent + assert

What changes were proposed in this pull request?
Refactor pattern of runEvent and assert into runEventAndCheck
Why are the changes needed?
Clean repeated code pattern
Does this PR introduce any user-facing change?
No
How was this patch tested?
Changes in UT.

Can one of the admins verify this patch?

Can one of the admins verify this patch?


Run SQL occur error

OrientDB Version, operating system, or hardware.

v2.2 enterprise

Operating System

[x ] Windows

Expected behavior and actual behavior
there is a V named TelephoneNumber and a E named Call
SQL:
TRAVERSE both("Call") FROM (SELECT FROM TelephoneNumber where TeleNo = '12345678') WHILE $depth <= 2
IN CMD WINDOWS, SCREEN ROLLING LIKE THIS
2016-12-22 23:08:57:720 WARNI {db=TeleNet} Record (null) is null [OrientEdgeIterator]
2016-12-22 23:08:57:720 WARNI {db=TeleNet} Record (null) is null [OrientEdgeIterator]
2016-12-22 23:08:57:721 WARNI {db=TeleNet} Record (null) is null [OrientEdgeIterator]
2016-12-22 23:08:57:721 WARNI {db=TeleNet} Record (null) is null [OrientEdgeIterator]
2016-12-22 23:08:57:721 WARNI {db=TeleNet} Record (null) is null [OrientEdgeIterator]
2016-12-22 23:08:57:722 WARNI {db=TeleNet} Record (null) is null [OrientEdgeIterator]
2016-12-22 23:08:57:722 WARNI {db=TeleNet} Record (null) is null [OrientEdgeIterator]
2016-12-22 23:08:57:722 WARNI {db=TeleNet} Record (null) is null [OrientEdgeIterator]
2016-12-22 23:08:57:723 WARNI {db=TeleNet} Record (null) is null [OrientEdgeIterator]
2016-12-22 23:08:57:723 WARNI {db=TeleNet} Record (null) is null [OrientEdgeIterator]
2016-12-22 23:08:57:723 WARNI {db=TeleNet} Record (null) is null [OrientEdgeIterator]
2016-12-22 23:08:57:723 WARNI {db=TeleNet} Record (null) is null [OrientEdgeIterator]
2016-12-22 23:08:57:724 WARNI {db=TeleNet} Record (null) is null [OrientEdgeIterator]
2016-12-22 23:08:57:724 WARNI {db=TeleNet} Record (null) is null [OrientEdgeIterator]
2016-12-22 23:08:57:725 WARNI {db=TeleNet} Record (null) is null [OrientEdgeIterator]
2016-12-22 23:08:57:726 WARNI {db=TeleNet} Record (null) is null [OrientEdgeIterator]
2016-12-22 23:08:57:726 WARNI {db=TeleNet} Record (null) is null [OrientEdgeIterator]
....................
...................
AND WILL NOT STOP

Hi @cuijinyi
do you have a sample dataset to reproduce the problem
Thanks
Luigi

Hi @luigidellaquila
the test database is very huge and don't know why has this error

Hi @cuijinyi
Two options here:

you have record-level security in place and something is not correctly managed (are you using record-level security? http://orientdb.com/docs/2.2.x/Database-Security.html#record-level-security)
you have some "broken" edges in your DB, eg. a link that is pointing to a non existing vertex/edge. Did you do any DELETE UNSAFE or things like that? Are you working in transaction?

Thanks
Luigi

you are right, I'd ever used DELETE UNSAFE command sometime before, maybe it has broken edges


note lists indention are disordered after moving notes from one folder to another

to reproduce:
before a note is moved:

move note 1 from folder A to folder B:

issue description
after note 1 in folder A is moved to folder B, under this theme, note 1 is furtively renamed to 'note 1 1', adding an extra '1' to the note's name.

This is not a bug. Obsidian adds a "1" after any duplicated file name in the same folder, so as to avoid confusion for wiki links, quick switcher, etc.
(Please to think before your post: have you tried this without the theme on?)

@Ali-C-Ila Sorry for the bothering, what you are telling is true.
I was distracted by the above action by Obsidian.
To correct this, I repost the issue.


Digitialocen droplet rise an error of installing dlib

Expected Behavior
pip3 install dlib should work successfully
Current Behavior
( I'm  running on a web console sorry I cannot copy text )

Steps to Reproduce
$ sudo apt-get update
$ sudo apt-get install build-essential cmake
$ sudo apt-get install libopenblas-dev liblapack-dev
$ sudo apt-get install libx11-dev libgtk-3-dev
$ sudo apt-get install python3 python3-dev python3-pip
$ pip install numpy
$ pip install dilb

Version: not specific versiton
Where did you get dlib: pip install dlib
Platform: Digitalocen  Ubuntu 20.04 (LTS) x64  droplet  1 vCPUs 1GB / 25GB Disk
Compiler: beyond my knowledge

Increasing droplet size resolve this issue

Increasing droplet size resolve this issue

can you specify the droplet size needed for successful installation?


Pierre Salvy, greatest coder in the world

@P0N3Y: I know you have an elegant and beautiful function for extracting a COBRA model from the OME database. Do you think you could submit a PR for that function? We could run it by @jlerman44 too...
Thanks! I hope you guys are doing well up there.
-Zak

@zakandrewking I really hope we are the only ones on this thread ahaha. As I told you, I'll submit this to the IP guys so that they give me the thumbs up to give it to you.
I hope you guys are doing well down there :)
-Pierre

OK. Thanks Pierre. sounds good.

Hi Zak,
Got the thumbs up for sharing this code. I will work a little on it so I can ensure its compatibility with the master branch. Will submit a PR soon, hopefully before I leave :)

Excellent! Can you submit the PR against the branch called 'bigg'? Don't worry about compatibility too much—we're still changing a lot of things on that branch.
Best,
Zak
—
Sent from Mailbox
On Mon, Feb 2, 2015 at 5:49 PM, Pierre SALVY<EMAIL_ADDRESS>wrote:

Hi Zak,
Got the thumbs up for sharing this code. I will work a little on it so I can ensure its compatibility with the master branch. Will submit a PR soon, hopefully before I leave :)
Reply to this email directly or view it on GitHub:
https://github.com/SBRG/ome/issues/14#issuecomment-72577156

Will do.

Concerning compatibility, I'll do minor modifcations to my code just to remove some useless stuff for you then, and I can help you integrate it if you want.
Are you also interested in the GPR -> reverse Polish notation -> Tree algorithm ? (note: this needs a supplementary table).

Definitely! We'd love to have that.
—
Sent from Mailbox
On Tue, Feb 3, 2015 at 10:02 AM, Pierre SALVY<EMAIL_ADDRESS>wrote:

Concerning compatibility, I'll do minor modifcations to my code just to remove some useless stuff for you then, and I can help you integrate it if you want.
Are you also interested in the GPR -> reverse Polish notation -> Tree algorithm ? (note: this needs a supplementary table).
Reply to this email directly or view it on GitHub:
https://github.com/SBRG/ome/issues/14#issuecomment-72701390

Okay, I'll join you the code for it.

Also what about domain handling and group filtering ? my actual version allows you to filter in and out reaction groups. Do you want that ? (this also needs you to have reaction groups tables, but that's pretty straightforward)

No, we won't be using those features. So don't worry about them.
—
Sent from Mailbox
On Tue, Feb 3, 2015 at 10:07 AM, Pierre SALVY<EMAIL_ADDRESS>wrote:

Also what about domain handling and group filtering ? my actual version allows you to filter in and out reaction groups. Do you want that ? (this also needs you to have reaction groups tables, but that's pretty straightforward)
Reply to this email directly or view it on GitHub:
https://github.com/SBRG/ome/issues/14#issuecomment-72702240

I will post 3 files:

The pull_model script that handles the heavy lifting
The gpr_rpn_tree file that does the eval/compilation of GPRs
browsing_tools which is a bunch of functions for interacting with the database
Since you are rewriting a lot of stuff, as agreed, you might have to go through the main script (pull_model) to glue everything together.
I will submit the PR as @Amyris.


PMG3000-D20B ONT-ONU(ONT) Link Monitoring

hello world
i am looking for the command to display the ONU-OLT link information
upstream speed and downstream speed. packets tx/rx. packetloss tx/rx and so on.
the tx/rx power i have found.

I am closing the issue as it is not a issue.


Fix development shell for non POSIX-Compliant default shells

This PR will patch the recently made changes to tmuxinator.sh for use with non-POSIX-compliant shells.
Tmuxinator will use the SHELL variable by default which breaks source compatibility inside tmux

Codecov Report

Merging #520 (754fcc5) into master (ef77e77) will decrease coverage by 0.22%.
The diff coverage is n/a.

:exclamation: Current head 754fcc5 differs from pull request most recent head bffa807. Consider uploading reports for the commit bffa807 to get more accurate results

@@            Coverage Diff             @@
##           master     #520      +/-   ##
==========================================
- Coverage   72.94%   72.71%   -0.23%
==========================================
  Files          83       83
  Lines       10933    10974      +41
==========================================
+ Hits         7975     7980       +5
- Misses       2958     2994      +36

Impacted Files
Coverage Δ

client/clientd/src/main.rs
0.84% <0.00%> (-0.31%)
:arrow_down:

client/clientd/src/bin/clientd-cli.rs
2.32% <0.00%> (-0.31%)
:arrow_down:

client/clientd/src/lib.rs
1.38% <0.00%> (-0.09%)
:arrow_down:

fedimint/src/net/connect.rs
95.20% <0.00%> (+0.25%)
:arrow_up:

fedimint/src/net/peers.rs
93.73% <0.00%> (+1.04%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Setup eventloggerprovider in azure monitor distro

Fixes https://github.com/Azure/azure-sdk-for-python/issues/38412

API change check
API changes are not detected in this pull request.


getchar() return <80>d when move cursor

Describe the bug
after calling popup_create() function, echom getchar() return <80><fd>d when move mouse
To Reproduce
Detailed steps to reproduce the behavior:

Run vim --clean (or gvim --clean, etc.)
open popup
echo getchar()
move mouse

Expected behavior
nothing huppend
Screenshots
If applicable, copy/paste the text or add screenshots to help explain your problem.
Environment (please complete the following information):

OS: windows
gvim version: 8.1.2120

Additional context
vim print <80><fd>d

8.1.2120 is 9 months old.  Since then, a lot of commits have been pushed including:

8.2.1015 popup filter gets key with modifier prepended
8.2.1019 mapping  does not work in the GUI

Can you reproduce on master?

FWIW, I can reproduce with 8.2.1114 using:
gvim --clean --cmd ":set encoding=utf-8" -c "call popup_create('foobar', {})" -c "redraw!" -c "echo getchar()"

Ah true, I can reproduce on 8.2.1164 too with your command.

Well, we can specifically ignore the MOUSEMOVE key by using this patch:
diff --git a/src/getchar.c b/src/getchar.c
index 2beffa578..c656ec371 100644
--- a/src/getchar.c
+++ b/src/getchar.c
@@ -1928,7 +1928,10 @@ plain_vgetc(void)

     do
        c = safe_vgetc();
-    while (c == K_IGNORE || c == K_VER_SCROLLBAR || c == K_HOR_SCROLLBAR);
+    while (c == K_IGNORE ||
+           c == K_VER_SCROLLBAR ||
+           c == K_HOR_SCROLLBAR) ||
+           c == K_MOUSEMOVE);

     if (c == K_PS)
        // Only handle the first pasted character.  Drop the rest, since we

But I doubt it is correct. However I am not sure where to handle the Mouseevent specifically in the various getchar functions.

It doesn't appear useful to get mouse-move events through getchar().  Ignoring them in plain_vgetc() is probably not the right spot though.

Thanks for your work.

@brammool thanks, But when we remove this event, how do we handle mouse-move event? This is useful to redraw popup without click mouse!

Why would you need to redraw on mouse move event?

Am 30.07.2020 um 04:37 schrieb Wang Shidong<EMAIL_ADDRESS>
@brammool thanks, But when we remove this event, how do we handle mouse-move event? This is useful to redraw popup without click mouse!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

show doc for a word wothout move cursor。show info in popup based on mouse position。

You can use ballonexpr for mouse-move events, see :h ballonexpr and :h popup_beval-example.


How to prevent creating new tab groups after ?

Hi! Always when I open new tab in new window, when I close it, it is left as new unnamed tab group. How can I prevent this behavior?

Yes, seems that problem was in New Windows setting. Thank you

And can you answer on this quastion. I am very interested in it too — https://github.com/Morikko/sync-tab-groups/issues/162


Channel value prefix and suffix

Describe the desired behavior
At the moment it is not possible to have something like $5.00/minute since you can have only one units.
Describe the current behavior
You can only have $5.00 and then you must append and style /minute separately and manually.
Is this request related to a current issue? No
Additional Context
Here is an example of what I currently do on my own:

It is not a big deal but it would be nice to have an option to do this with the channel value component.

We recognize this is an inconvenience but at the moment we do not plan to include this in our design system.


update de.json

Update german translation

@Staubgeborener thanks!
Could you check one more thing for me here:
https://github.com/homebridge/homebridge-config-ui-x/pull/1626/commits/5e5d2a6b655d3ab0c317ea7bbb00a38f8c5af8d0
Where I have edited de.json does this make sense:
Vielen Dank für die Installation der {{ targetVersion }} Version von {{ pluginName }}

{{ targetVersion }} can be latest or beta

Where I have edited de.json does this make sense:

Vielen Dank für die Installation der {{ targetVersion }} Version von {{ pluginName }}

{{ targetVersion }} can be latest or beta

German is both extremely flexible but also a very strict language at the same time. Very thankless, especially as we have to adopt some words directly from the english language, translate others and reinvent others.
For example
Vielen Dank für die Installation der beta Version von {{ pluginName }}
Is working. More or less, I think the most of us prefer Beta-Version (so we just "create a new word").
But this one
Vielen Dank für die Installation der latest Version von {{ pluginName }}
definitely don't work. No doubt. latest has to be translated to something like aktuellsten.

“latest” and “beta” are taken directly from the npm tags that plugins are published under. So not easy to translate these.
Perhaps I could add some formatting (would apply to all languages eg
Vielen Dank für die Installation der latest Version von {{ pluginName }}
? thanks for the info by the way.

Maybe by doing this in two steps, so putting the beta/latest version naming into braces after the "thanks for installation" sentence (which should work for all translations), and check the stuff before with an if-statement like
if (plugin.latestVersion?.includes('beta')) { ... }

        Thanks for installing  {{ pluginName }}

 ((Beta-)|(Aktuellste )Version)

This would require two new translations for the i18n files: latest and beta.
But in the end probably a bit much work, I think probably everyone of the german Homebridge users/community knows the meaning of "latest version".


Level 3 Running Error

Hello! I was trying run and test my level 3 using Main 3 but I got this error. Does anyone know the likely reason for it happening and where I can look to debug it. Thank you!

maybe try to not type in the "^D" into the input? use ctrl + d instead.

hello managed to run it thank you!


Update Dockerfile

Image built based on Dockerfile in repo would not bring up properly. Bringing inline with documentation.
I suppose that the project could also use an update on the wiki to include using mooncord in a container. I may push in something for that later.

Hey there, thanks for the PR.
i oversaw the PR, sorry about that. Looks good to me. Also about the wiki, i will move to gitbook soon, because the github wiki is kinda limited...


pugixml build failure

Host Environment

OS: Windows 10 Enterprise Version 1809 Build 17763.805

To Reproduce
Steps to reproduce the behavior:
./vcpkg install pugixml
Failure logs
The following packages will be built and installed:
pugixml[core]:x86-windows
Starting package 1/1: pugixml:x86-windows
Building package pugixml[core]:x86-windows...
-- Downloading https://github.com/zeux/pugixml/archive/v1.10.tar.gz...
-- Downloading https://github.com/zeux/pugixml/archive/v1.10.tar.gz... Failed. Status: 7;"Couldn't connect to server"
CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:175 (message):
  Failed to download file.
  If you use a proxy, please set the HTTPS_PROXY and HTTP_PROXY environment
  variables to "https://user:password@your-proxy-ip-address:port/".
  Otherwise, please submit an issue at https://github.com/Microsoft/vcpkg/issues

Call Stack (most recent call first):
scripts/cmake/vcpkg_from_github.cmake:108 (vcpkg_download_distfile)
ports/pugixml/portfile.cmake:3 (vcpkg_from_github)
scripts/ports.cmake:94 (include)
Error: Building package pugixml:x86-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: pugixml:x86-windows
Vcpkg version: 2019.09.12-nohash
Additional context
Tried all the following:

\.vcpkg update
setting HTTP/HTTPS_PROXY env

Directly downloading the tar from a browser works just fine.
https://github.com/zeux/pugixml/archive/v1.10.tar.gz

@syeap I tried to install pugixml and it successfully installed. Make sure that you don't use any proxy and then try to reinstall it. Also, make sure that you have the latest version of vcpkg.

Thanks @ehsan-mohammadi ! Your reply prompted me to hunt down the proxy issue.
Tried a couple of things. And it finally worked.
What finally worked for me was:

Launch a cmd as Admin
set HTTP/HTTPS_PROXY and NO_PROXY env

I was use PowerShell previously. And did not realize that I couldn't do a "set" env in there.


Remove CONFIG_BCH

Summary
drivers/bch/Kconfig: Remove the separete CONFIG_BCH config
"CONFIG_BCH is not set" is confusing as the bch driver itself
is always there unless CONFIG_DISABLE_MOUNTPOINT=y.
Impact
Testing
locally build-tested a few combinations.

@yamt can we close this PR now?


Add pic method to override default emission of -fPIC

Adds a pic method to Config that allows overriding the default emission of -fPIC.
Closes #64

Thanks!


KeyError: 'samples'

Could you please help me with this:
│   190 │   │                                                                                │
│   191 │   │   print(f"self.dataset: {self.dataset}")                                             │
│   192 │   │   samples_batches = self.meta_chain.initial_chain.batch_invoke(batch_inputs, self.   │
│ ❱ 193 │   │   samples_list = [element for sublist in samples_batches for element in sublist['s   │
│   194 │   │   samples_list = self.dataset.remove_duplicates(samples_list)                        │
│   195 │   │   self.dataset.add(samples_list, 0)                                                  │
│   196                                                                                │
╰──────────────────────────────────────────────────────────────────────────────────────────────────╯
KeyError: 'samples'

It seems that there isn't 'samples' in samples_llist

Hi,
In order to provide better help, please provide all the config file changes and the running command.
This error usually happens due to one of the following reasons

Using too weak LLM model (if you modify the default LLM and using GPT-3.5 as the meta-prompt LLM instead of GPT-4)
Your connection permission issues (key/organization, etc')

Thanks for your response. I'm sure that the connection goes on well. It might be the problem of the modified config_default.yml
use_wandb: False
dataset:
    name: 'dataset'
    records_path: null
    initial_dataset: ''
    label_schema: ["Yes", "No"]
    max_samples: 50
    semantic_sampling: False # Change to True in case you don't have M1. Currently there is an issue with faiss and M1

annotator:
    # method : 'llm'
    # config:
    #     api_url: ''
    #     api_key: 'admin.apikey'
    #     workspace: 'admin'
    #     time_interval: 5
    method : 'llm'
    config:
        llm:
            type: 'azure'
            name: 'gpt-4-0125-preview'
#            async_params:
#                retry_interval: 10
#                max_retries: 2
            model_kwargs: {"seed": 220}
        num_workers: 5
        prompt: 'prompts/predictor_completion/prediction.prompt'
        mini_batch_size: 1  #change to >1 if you want to include multiple samples in the one prompt
        mode: 'annotation'

predictor:
    method : 'llm'
    config:
        llm:
            type: 'azure'
            name: 'gpt-4-0125-preview'
#            async_params:
#                retry_interval: 10
#                max_retries: 2
            model_kwargs: {"seed": 220}
        num_workers: 5
        prompt: 'prompts/predictor_completion/prediction.prompt'
        mini_batch_size: 1  #change to >1 if you want to include multiple samples in the one prompt
        mode: 'prediction'

meta_prompts:
    folder: 'prompts/meta_prompts_classification'
    num_err_prompt: 1  # Number of error examples per sample in the prompt generation
    num_err_samples: 2 # Number of error examples per sample in the sample generation
    history_length: 4 # Number of sample in the meta-prompt history
    num_generated_samples: 10 # Number of generated samples at each iteration
    num_initialize_samples: 10 # Number of generated samples at iteration 0, in zero-shot case
    samples_generation_batch: 10 # Number of samples generated in one call to the LLM
    num_workers: 5 #Number of parallel workers
    warmup: 4 # Number of warmup steps

eval:
    function_name: 'accuracy'
    num_large_errors: 4
    num_boundary_predictions : 0
    error_threshold: 0.5

llm:
    type: 'azure'
    name: 'gpt-4-1106-preview'
    temperature: 0.8

stop_criteria:
    max_usage: 2 #In $ in case of OpenAI models, otherwise number of tokens
    patience: 10 # Number of patience steps
    min_delta: 0.01 # Delta for the improvement definition

Thanks for your help!

When using an LLM as the annotator, you should also explicitly define the instructions for the annotator in the configuration file. See the following example:
https://github.com/Eladlev/AutoPrompt/blob/main/docs/installation.md#configure-llm-annotator

Thanks for you quick response! I've changed config_default.yml to
use_wandb: False
dataset:
    name: 'dataset'
    records_path: null
    initial_dataset: ''
    label_schema: ["Yes", "No"]
    max_samples: 50
    semantic_sampling: False # Change to True in case you don't have M1. Currently there is an issue with faiss and M1

annotator:
    method : 'llm'
    config:
        llm:
            type: 'azure'
            name: 'gpt-4-1106-preview'
        instruction:
            'Does this movie review contain a spoiler? answer Yes or No.'
        num_workers: 5
        prompt: 'prompts/predictor_completion/prediction.prompt'
        mini_batch_size: 1
        mode: 'annotation'

predictor:
    method : 'llm'
    config:
        llm:
            type: 'azure'
            name: 'gpt-4-1106-preview'
#            async_params:
#                retry_interval: 10
#                max_retries: 2
            model_kwargs: {"seed": 220}
        num_workers: 5
        prompt: 'prompts/predictor_completion/prediction.prompt'
        mini_batch_size: 1  #change to >1 if you want to include multiple samples in the one prompt
        mode: 'prediction'

meta_prompts:
    folder: 'prompts/meta_prompts_classification'
    num_err_prompt: 1  # Number of error examples per sample in the prompt generation
    num_err_samples: 2 # Number of error examples per sample in the sample generation
    history_length: 4 # Number of sample in the meta-prompt history
    num_generated_samples: 10 # Number of generated samples at each iteration
    num_initialize_samples: 10 # Number of generated samples at iteration 0, in zero-shot case
    samples_generation_batch: 10 # Number of samples generated in one call to the LLM
    num_workers: 5 #Number of parallel workers
    warmup: 4 # Number of warmup steps

eval:
    function_name: 'accuracy'
    num_large_errors: 4
    num_boundary_predictions : 0
    error_threshold: 0.5

llm:
    type: 'azure'
    name: 'gpt-4-1106-preview'
    temperature: 0.8

stop_criteria:
    max_usage: 2 #In $ in case of OpenAI models, otherwise number of tokens
    patience: 10 # Number of patience steps
    min_delta: 0.01 # Delta for the improvement definition

And the running command is
python3 run_pipeline.py     --prompt "Does this movie review contain a spoiler? answer Yes or No"     --task_description "Assistant is an expert classifier that will classify a movie review, and let the user know if it contains a spoiler for the reviewed movie or not."     --num_steps 30

But I still get the error: KeyError: 'samples'. I further printed the value of samples_batches, and find that there isn't 'samples' in samples_batches. I really appreciate your help.
sample batches: [{'num_samples': 10, 'task_description': 'Assistant is an expert classifier that will classify a movie review, and let the user know if it contains a spoiler for the reviewed movie or not.', 'instruction': 'Does this movie review contain a spoiler? answer Yes or No', 'text': 'Below are 10 challenging samples designed to test the capability of the expert classifier in identifying spoilers within movie reviews.\n\n### Sample 1:\nReview: "I must say, the character development in the second act was unexpected. Watching John Doe evolve from a small-time crook to something much greater was a journey I didn't see coming. The climax of the movie had a surprising twist that left me in awe."\n- Spoiler: No\n\n### Sample 2:\nReview: "The film's homage to the noir genre was clever, especially when you realize the protagonist was the villain all along. The subtle hints throughout were a nice touch, and the ending was a fitting tribute to the classics."\n- Spoiler: Yes\n\n### Sample 3:\nReview: "I was particularly impressed by how the movie represented historical events without giving away the key turning point that anyone familiar with the period would recognize. The costumes and setting felt authentic and immersive."\n- Spoiler: No\n\n### Sample 4:\nReview: "The director's choice to revisit the moment from the first movie, where it all began, tied the series together beautifully. It gave a deeper meaning to the protagonist's journey and certainly set up an epic finale."\n- Spoiler: Yes\n\n### Sample 5:\nReview: "The film was a rollercoaster of emotions, and the performances were top-notch. When the credits rolled, I was still reeling from the implications of the scientist's discovery on the narrative as a whole."\n- Spoiler: No\n\n### Sample 6:\nReview: "The movie's take on the multiverse concept was fascinating, especially the scene where the hero meets his demise, which I think is a bold move that will be remembered for years to come."\n- Spoiler: Yes\n\n### Sample 7:\nReview: "Without revealing too much, the film's twist on the traditional 'chosen one' trope was refreshing. It makes you question the nature of destiny and the paths we choose."\n- Spoiler: No\n\n### Sample 8:\nReview: "Interestingly, the final confrontation wasn't between who you'd expect, and the villain's demise was both poetic and satisfying. It's rare to see such a perfect encapsulation of justice in film."\n- Spoiler: Yes\n\n### Sample 9:\nReview: "The interplay between the main characters was the highlight of the film for me. Their final decisions felt earned and resonated with the overarching theme without disclosing the fate of the kingdom they fought to protect."\n- Spoiler: No\n\n### Sample 10:\nReview: "By the end, when the true identity of the puppet master was revealed, it was both a shock and a testament to the intricate storytelling woven throughout the series. The implications for the universe are profound."\n- Spoiler: Yes\n\nThese samples are challenging because they include nuanced language that may give the impression of revealing a plot without actually doing so, or they include actual spoilers cloaked in more subtle language that might not be immediately recognized as a spoiler.'}]

The issue was with the llm 'type', you wrote 'azure' in lowercase and the system expected 'Azure'.
I modified the code to be invariant to lower/upper case

The program is running smoothly now, thank you very much！


Any way to create wire?

Is there any way to create wire or something similar to that?

Hi,
If you are looking for something to short-circuit two nodes together, the usually recommended way would be to use a 0V voltage source. If the wire is ohmic, you can just use the Resistor-component.
If not: a wire is used in schematics to denote everything that is at the same potential, i.e. a single node in Spice#. Spice# does not have any concept of a (graphical) wire and neither does any other Spice simulator. It is only something you will see/use in schematic editors, which will translate everything connected by wires to nodes before sending it to the simulator.
Yours sincerely,
Sven

Thank you that Worked....


Error: spawn ENOENT when running imagemin task

I'm getting the following error when running imagemin. I'm not sure why, but it looks like it's adding a tmp folder inside the 'grunt-jekyll' task? I've tried running it independent of any other task and the same error occurs. After crashing, it leaves some of the images in the output folder with '.tmp' extensions (ex: image.png.tmp). Any ideas would be greatly appreciated, thanks!
Running "imagemin:images" (imagemin) task

D:\Dropbox\Projects\automata\node_modules\grunt-jekyll\node_modules\tmp\lib\tmp.js:261
  throw err;
        ^
Error: spawn ENOENT
  at errnoException (child_process.js:980:11)
  at Process.ChildProcess._handle.onexit (child_process.js:771:34)

This is latest Mac Yosemite. Installed imagemin as well as pngquant latest version. Not working, failing with error message provided by the thread creator.

Installing earlier version of contrib-imagemin fixed it for me. I picked one at random. used 0.4.0, grunt build worked

@SamJacobs Worked for me. I was using 0.8.0 and it started working when I installed 0.7.0

Okay, I found out when this is happening: when the prebuild-test failed and thus is compiling from the source, see extract below:
  ⚠ Couldn't connect to https://raw.github.com/imagemin/optipng-bin/v2.0.4/vendor/linux/x64/optipng (503)
  ⚠ optipng pre-build test failed
  ℹ compiling from source
  ✖ Error: ./configure --with-system-zlib --prefix="/var/www/signdna-video/live/source/node_modules/grunt-contrib-imagemin/node_modules/imagemin/node_modules/imagemin-optipng/node_modules/optipng-bin/vendor" --bindir="/var/www/signdna-video/live/source/node_modules/grunt-contrib-imagemin/node_modules/imagemin/node_modules/imagemin-optipng/node_modules/optipng-bin/vendor" && make install
Command failed:
    at ChildProcess.exithandler (child_process.js:648:15)
    at ChildProcess.emit (events.js:98:17)
    at maybeClose (child_process.js:756:16)
    at Socket.<anonymous> (child_process.js:969:11)
    at Socket.emit (events.js:95:17)
    at Pipe.close (net.js:465:12)

This happened during instable network conditions which is not the real issue here. The problem is that compiling from source results in a different state than when doing the pre-build. Both ways should make the code behave the same.

I'm having a similar issue with jpegtran failing to build, likely due to Github's network problems...
The real underlying issue seems to be that optipng and jpegtran's build scripts can fail "silently," without cascading upwards to imagemin/grunt-imagemin.

I've opened a PR in jpegtran-bin that will hopefully cause compilation errors to propogate upward, so that the NPM module doesn't trick any dependent packages into believing that it was installed successfully.

@schmod, that's exactly what we want. They are optional dependencies.

I left a comment in the other thread.
Essentially, I don't think that's actually a desirable behavior, as it bypasses npm's built-in infrastructure for handling optional dependencies, and effectively makes imagemin and grunt-contrib-imagemin unsafe to use in a scripted environment, as it doesn't provide any reliable mechanism to determine whether or not everything installed correctly.
imagemin already has logic to gracefully treat jpegtran-bin as an optional dependency, but that logic is worthless if jpegtran-bin's build script doesn't report failures.

What about the people that's trying to install something that has imagemin as a dependency but can't get a module working? They will never be able to install their stuff.

Ran into this issue with "grunt-contrib-imagemin": "^0.8.1".  Updated to "grunt-contrib-imagemin": "^0.9.4" fixed my issue.

Updating "grunt-contrib-imagemin" from "^0.9.2" to "^0.9.4" fixed my issue.


Hexagon compilation on MacOS system

Short desc
This changes allow my to compile and tune models for hexagon directly from my macOS laptop without full switching to linux environment.
List of changes

Replace local linker call with call from docker container with Hexagon SDK. Yes, that is the only SDK tool used by TVM during compilation.
Enhanced search of ADB. Not only in PATH, but also in ANDROID_HOME, ANDROID_SDK_ROOT and default sdk installation directory. Mac OS doesn't allow to easily change default PATH env var for UI application launched from dock bar. So adb is not available for IDE by default.

Motivation
Some engineers would like to continue work with comfortable macOS environment even if they have to play with hexagon devices. At this moment there is no official Hexagon SDK for macOS system. Alternatives are next: fully switch to remote linux, use local linux virtual machine or try to introduce required hexagon SDK functionality for macOS. The last option is more preferable to me.

Short desc This changes allow my to compile and tune models for hexagon directly from my macOS laptop without full switching to linux environment.
List of changes

Replace local linker call with call from docker container with Hexagon SDK. Yes, that is the only SDK tool used by TVM during compilation.
Enhanced search of ADB. Not only in PATH, but also in ANDROID_HOME, ANDROID_SDK_ROOT and default sdk installation directory. Mac OS doesn't allow to easily change default PATH env var for UI application launched from dock bar. So adb is not available for IDE by default.

Motivation Some engineers would like to continue work with comfortable macOS environment even if they have to play with hexagon devices. At this moment there is no official Hexagon SDK for macOS system. Alternatives are next: fully switch to remote linux, use local linux virtual machine or try to introduce required hexagon SDK functionality for macOS. The last option is more preferable to me.

Is there any ReadME.md to follow？I am a newbee in Hexagon, and i don't know how to use them. thanks a lot.


No content returned by cat command

While executing a container with command sh -c 'ls -al /all && cat /all/podname && cat /all/secret-data && cat /all/configmap-data', ls shows file size is not zero, but there is no content returned by following cat commands:
total 4
drwxrwxrwt    3 root     root           140 Apr  1 10:18 .
drwxr-xr-x    3 root     root          4096 Apr  1 10:18 ..
drwxr-xr-x    2 root     root           100 Apr  1 10:18 ..4984_01_04_10_18_38.047673749
lrwxrwxrwx    1 root     root            31 Apr  1 10:18 ..data -> ..4984_01_04_10_18_38.047673749
lrwxrwxrwx    1 root     root            21 Apr  1 10:18 configmap-data -> ..data/configmap-data
lrwxrwxrwx    1 root     root            14 Apr  1 10:18 podname -> ..data/podname
lrwxrwxrwx    1 root     root            18 Apr  1 10:18 secret-data -> ..data/secret-data

Releated hyperd log: hyperd.log.txt

could have a try with ; instead of &&? with &&, if any of the cat return non-zero, the rest command won't be executed.

@gnawux The command executes success, so there is no non-zero return code.

@feiskyer all the non zero size files are symlink, and can't tell if they are empty at that time.

any update on this issue?

@laijs @gao-feng Could you help to fix the problem?

ls didn't show the file size zero or not,  could you use ls -Lal instead please?

# hyperctl logs dce0c7c9bd61fc7b53fa0936a0442e0b4c8b93e5000dfd06dd73014380db1938
total 16
drwxrwxrwt    3 root     root           140 Apr 13 03:31 .
drwxr-xr-x    3 root     root          4096 Apr 13 03:31 ..
drwxr-xr-x    2 root     root           100 Apr 13 03:31 ..4984_13_04_03_31_10.967674860
drwxr-xr-x    2 root     root           100 Apr 13 03:31 ..data
-rw-r--r--    1 root     root            17 Apr 13 03:31 configmap-data
-rw-r--r--    1 root     root             4 Apr 13 03:31 podname
-rw-r--r--    1 root     root             5 Apr 13 03:31 secret-data

If the pod is started with sh and exec the same command after it has been started, the output is (which is expected above):
total 16
drwxrwxrwt    3 root     root           140 Apr 13 04:10 .
drwxr-xr-x    3 root     root          4096 Apr 13 04:10 ..
drwxr-xr-x    2 root     root           100 Apr 13 04:10 ..4984_13_04_04_10_49.294962174
drwxr-xr-x    2 root     root           100 Apr 13 04:10 ..data
-rw-r--r--    1 root     root            17 Apr 13 04:10 configmap-data
-rw-r--r--    1 root     root             5 Apr 13 04:10 podname
-rw-r--r--    1 root     root             5 Apr 13 04:10 secret-data
test2adminconfigmap-value-1

If we add a newline character at the end of https://github.com/feiskyer/ops/blob/master/kubernetes/examples/projected-volume.yaml#L28, e.g., - ls -laR /all && cat /all/podname && cat /all/secret-data && echo, then the file contents will be shown.
[hypervsock@kubelet]$kubectl logs test
/all:
total 0
drwxrwxrwt    3 root     root           120 May  5 06:32 .
drwxr-xr-x    3 root     root            82 May  5 06:32 ..
drwxr-xr-x    2 root     root            80 May  5 06:32 ..5985_05_05_14_32_45.093720545
lrwxrwxrwx    1 root     root            31 May  5 06:32 ..data -> ..5985_05_05_14_32_45.093720545
lrwxrwxrwx    1 root     root            14 May  5 06:32 podname -> ..data/podname
lrwxrwxrwx    1 root     root            18 May  5 06:32 secret-data -> ..data/secret-data

/all/..5985_05_05_14_32_45.093720545:
total 8
drwxr-xr-x    2 root     root            80 May  5 06:32 .
drwxrwxrwt    3 root     root           120 May  5 06:32 ..
-rw-r--r--    1 root     root             4 May  5 06:32 podname
-rw-r--r--    1 root     root             5 May  5 06:32 secret-data
testadmin

A simple reproducer:
[hypervsock@~]$sudo hyperctl run --rm busybox echo -n foobar
foobar[hypervsock@~]$
[hypervsock@~]$sudo hyperctl run -d busybox echo -n foobar
POD id is busybox-9702672853
Time to run a POD is 6017 ms
[hypervsock@~]$sudo hyperctl logs busybox-9702672853
[hypervsock@~]$sudo cat /var/run/hyper/Pods/busybox-9702672853/8dff3adc177161430cf1a103eb8ce21cfea813724da48b647b94b50d2c4cd16c-json.log
[hypervsock@~]$

It seems the json file logger is missing container output when there is no terminating newline character.

@bergwolf Could we add a fix for this?

I'm still looking for the root cause but yes it needs to be fixed.


PT-1347 | Fix enrolment form group size validation bug

Description :sparkles:
Persisting min and maxGroupSize values that were only used for validation was causing problems because previous values would be taken from local storage. To fix this min and MaxGroupSize values are now passed ar parameters to getValidationSchema to avoid persisting them to local storage.
Issues :bug:
Closes :no_good_woman:
PT-1347:
Related :handshake:
Testing :alembic:
Automated tests :gear:
Manual testing :construction_worker_man:
Screenshots :camera_flash:
Additional notes :spiral_notepad:

Codecov Report

Merging #191 (0830cca) into master (c209942) will increase coverage by 0.02%.
The diff coverage is 97.87%.

@@            Coverage Diff             @@
##           master     #191      +/-   ##
==========================================
+ Coverage   83.76%   83.79%   +0.02%
==========================================
  Files         113      113
  Lines        3388     3400      +12
  Branches      772      734      -38
==========================================
+ Hits         2838     2849      +11
- Misses        542      543       +1
  Partials        8        8

Impacted Files
Coverage Δ

src/domain/event/occurrences/OccurrenceInfo.tsx
90.10% <75.00%> (-0.59%)
:arrow_down:

...c/domain/enrolment/enrolmentForm/EnrolmentForm.tsx
97.08% <100.00%> (+0.05%)
:arrow_up:

...omain/enrolment/enrolmentForm/ValidationSchema.tsx
100.00% <100.00%> (ø)

.../domain/event/occurrences/EnrolmentFormSection.tsx
89.79% <100.00%> (+0.43%)
:arrow_up:

src/domain/errorPage/ErrorPage.tsx
93.33% <0.00%> (-1.12%)
:arrow_down:

src/domain/event/eventCard/EventCard.tsx
98.57% <0.00%> (ø)

src/domain/notFoundPage/NotFoundPage.tsx
100.00% <0.00%> (ø)

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 c209942...0830cca. Read the comment docs.


Bump version to v1.33.0.

Drop the "pre" in version.

known issue: b/174126929

The title of this PR is misleading. Should be changed to say v1.34.0

Thanks Sanjay for pointing this out, I've already updated the title.


Add integration test pinned to tendermint-go v0.33.5

Closes #304
With this change, CI runs two integration tests:

A stable test to protect against regressions, run against a pinned
version of the tendermint/tendermint docker image.
A latest test to track whether we're maintaining parity with the
latest development version of tendermint-go.

IIUC, we'd ideally like (1) to be a required test, and to get visibility on (2) but not block the PR if it fails. I can configure (2) so that it the job succeeds even if the step fails, but then we lose clear visibility on the failure.
Please see individual commits for more detail.

[X] Referenced an issue explaining the need for the change
[x] Updated all relevant documentation in docs
[x] Updated all code comments where relevant
[x] Wrote tests
[x] Updated CHANGES.md

Codecov Report

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

@@          Coverage Diff           @@
##           master    #324   +/-   ##
======================================
  Coverage    30.5%   30.5%
======================================
  Files         127     127
  Lines        4581    4581
  Branches     1416    1416
======================================
  Hits         1398    1398
  Misses       2216    2216
  Partials      967     967

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 740cb66...8aebdcf. Read the comment docs.

This change should make it possible to make the test-integration-stable test required (if that's desirable).

This change should make it possible to make the test-integration-stable test required (if that's desirable).

Yeah I think we want to fail the build if the integration tests do not pass on stable.
Great job by the way, thanks a lot!

I'm surprised the integration tests still fail despite fixing the version to 0.33.5 🤔 maybe there were some changes we missed in the meantime.

as you looked a lot more into GA, do you have a suggestion how to parameterize this?

I have not. Other than env variables I can't think of anything right now but will think about it for a bit and geet back to you.

I'm surprised the integration tests still fail despite fixing the version to 0.33.5 🤔 maybe there were some changes we missed in the meantime.

This may have something to do with our new lib for encoding json to replace amino.

Merging as is. I'll investigate why the tests are failing in a separate pr later.

This may have something to do with our new lib for encoding json to replace amino.

This is actually because the tests against :latest are still running:
From the introductory comment:

I can configure (2) so that it the job succeeds even if the step fails, but then we lose clear visibility on the failure.

Yes, I think that would be OK for now! Please do (or anyone else with necessary perms)


Add support for 'fade' transition

I need the ability to fade from one item to the next if a user clicks on a pagination link, can this be added easily?

No, this is out of the scope of jCarousel. For slider with fade transitions, irecommend something like Cycle2.


ififuncs.py - updated user list

Updated user list of ififuncs.py to include more staff during cross-training.

Hi @ifi-library, could you test it with any script using this user_list and see if the running and output are ok? Thanks!

Looks good to me btw, but seeing as the numbers are arbitrary and only needed for easy input on the keyboard, it might be worth changing this function so there’s just a list of users, and the function just loops through to make the same output as now but uses something like “for user in range(list)” and then formulate the display string by just doing a plus one for each user? Not sure if that makes any sense, but basically making it so the numbers are not hard coded anymore and instead just are created by a loop. So the only thing you have to update is the list of names when staff change?

Sorry I closed this pr too quick. That sounds good and more convenient in the future. Will work on this.


Search Box not showing the Suggestion in react Search refiners Sample

We were using SearchBox and Search result Result webpart from this sample https://github.com/SharePoint/sp-dev-fx-webparts/tree/dev/samples/react-search-refiners
We have build and Installed the webpart in App catalog Succesfully.
Added the Search Box webpart to the page and Enable the Query Suggestion.
when ever i typed any keyword in Searchbox it does not result anything as suggestion though Search results Webpart displayed the results.
On further investigation using Developer Tools i found that below Rest query called successfully but no result returned.
https://mysite.sharepoint.com/_api/search/suggest?querytext='Shar'&inumberofquerysuggestions=10&fprequerysuggestions=true&fhithighlighting=true&culture=1033&fprefixmatchallterms=true
Based on further search on Internet i found a link stating that _api/search/suggest is not supported in SharePoint Online.
https://sharepoint.stackexchange.com/questions/249176/api-search-suggestquerytext-returns-empty-result
could you please confirm the same or any configuration could be missing for me.
We are using Sharepoint online for your information.
Category
[X ] Question
[X ] Bug
[] Enhancement
Authors
@FranckyC @wobba
Expected or Desired Behavior

Haven't tested in a while, but suggestions also are dependent on results being clicked, which are also site collection scoped. Which means the feature is hard to get working (unless you manually add global suggestions). And we have not implemented click recording in this webpart (yet).

@FranckyC the docs says online is not supported for suggestions.

@wobba At the time when I've implemented it, the suggestions were available in SPO (the static keyword list). Don't know if MS shut down this feature since.

Works fine in my tenant.

Which means docs are wrong 😄

Does it work in your tenant when you simply add some phrases?

0
Iam calling the search api "_api/search/suggest?querytext='*" using httpclient in azure functions. Authentication is done via azure ad application. Application permissions sites.fullcontrol is provided. But when ever i tried to call this suggest api, it returns empty results.
i can able to retreive the results of suggest api using username and password but using the azure ad app auth it gives the empty results.
whether we can able to call this suggest api using application permissions set in azure ad ? kinldy let me know if there are any alternatives


Issue While connecting Frontend

Issue Description
I followed all the steps mentioned in the documentation and watched your YouTube video to create a "FullStack Solana development with React and Anchor" project. Everything worked smoothly until I reached the frontend part.
Problem 1: IDL File
In the documentation, it instructs to create a new folder named anchor in front-end/src and copy over target/types/counter.ts (from the Anchor program) to a new file called idl.ts. The purpose of this step is to use the IDL to create TypeScript objects for interacting with the program.
However, when I opened target/types/counter.ts, I found that it's different from what was expected. It contains additional content, and the line export const IDL: Counter = { ... } is missing.
My counter.ts was like:

```typescript
/**
 * Program IDL in camelCase format in order to be used in JS/TS.
 *
 * Note that this is only a type helper and is not the actual IDL. The original
 * IDL can be found at `target/idl/counter.json`.
 */
export type Counter = {
    "address": "2FcNSYHzEP7gXYZA4YMfiznRVAj7JpZF8Z4fWL7xw9jK",
    "metadata": {
        "name": "counter",
        "version": "0.1.0",
        "spec": "0.1.0",
        "description": "Created with Anchor"
    },
    "instructions": [
        {
            "name": "increment",
            "discriminator": [
                11,
                18,
                104,
                9,
                104,
                174,
                59,
                33
            ],
            "accounts": [
                {
                    "name": "counter",
                    "writable": true,
                    "pda": {
                        "seeds": [
                            {
                                "kind": "const",
                                "value": [
                                    99,
                                    111,
                                    117,
                                    110,
                                    116,
                                    101,
                                    114
                                ]
                            }
                        ]
                    }
                }
            ],
            "args": []
        },
        {
            "name": "initialize",
            "discriminator": [
                175,
                175,
                109,
                31,
                13,
                152,
                155,
                237
            ],
            "accounts": [
                {
                    "name": "user",
                    "writable": true,
                    "signer": true
                },
                {
                    "name": "counter",
                    "writable": true,
                    "pda": {
                        "seeds": [
                            {
                                "kind": "const",
                                "value": [
                                    99,
                                    111,
                                    117,
                                    110,
                                    116,
                                    101,
                                    114
                                ]
                            }
                        ]
                    }
                },
                {
                    "name": "systemProgram",
                    "address": "11111111111111111111111111111111"
                }
            ],
            "args": []
        }
    ],
    "accounts": [
        {
            "name": "counter",
            "discriminator": [
                255,
                176,
                4,
                245,
                188,
                253,
                124,
                25
            ]
        }
    ],
    "types": [
        {
            "name": "counter",
            "type": {
                "kind": "struct",
                "fields": [
                    {
                        "name": "count",
                        "type": "u64"
                    },
                    {
                        "name": "bump",
                        "type": "u8"
                    }
                ]
            }
        }
    ]
};

Problem 2: Error in setup.js
After adding the setup.js file, which looks like this:
import { IdlAccounts, Program } from "@coral-xyz/anchor";
import { IDL, Counter } from "./idl";
import { clusterApiUrl, Connection, PublicKey } from<EMAIL_ADDRESS>
const programId = new PublicKey("2FcNSYHzEP7gXYZA4YMfiznRVAj7JpZF8Z4fWL7xw9jK");
const connection = new Connection(clusterApiUrl("devnet"), "confirmed");

export const program = new Program<Counter>(IDL, programId, {
  connection,
});

export const [mintPDA] = PublicKey.findProgramAddressSync(
  [Buffer.from("mint")],
  program.programId,
);

export const [counterPDA] = PublicKey.findProgramAddressSync(
  [Buffer.from("counter")],
  program.programId,
);

export type CounterData = IdlAccounts<Counter>["counter"];

I encountered an error at the line export const program = new Program<Counter>(IDL, programId, { connection, });, which says:
Argument of type 'PublicKey' is not assignable to parameter of type 'Provider'.
Property 'connection' is missing in type 'PublicKey' but required in type 'Provider'.

Problem 3: Error in Frontend
Following the subsequent steps and adding <CounterState/> in App.tsx, I encountered an error in the frontend console:
Uncaught TypeError: Cannot read properties of undefined (reading 'size')
    at new AccountClient (@coral-xyz_anchor.js?v=7e705056:10404:39)
    at @coral-xyz_anchor.js?v=7e705056:10369:30
    at Array.reduce (<anonymous>)
    at AccountFactory.build (@coral-xyz_anchor.js?v=7e705056:10368:70)
    at NamespaceFactory.build (@coral-xyz_anchor.js?v=7e705056:11480:51)
    at new _Program (@coral-xyz_anchor.js?v=7e705056:11560:98)
    at setup.ts:10:24

Steps Taken

Followed all steps mentioned in the documentation and video.
Checked for typos and ensured correct file placements.
Reviewed code for any missing or incorrect configurations.

Additional Information

Operating System: Windows(using WSL)
solana --version; node -v; yarn --version; anchor --version
solana-cli 1.18.9 (src:9a7dd9ca; feat:3469865029, client:SolanaLabs)
v20.8.0
1.22.22
anchor-cli 0.30.0

Your first issue may be due to a breaking change in Anchor IDLs. Read more here. I was able to fix a similar issue by ensuring the version of Anchor in anchor.toml on the backend matched that in package.json for the frontend.

Same issue here, I tried switching back to Anchor 0.29.0 and that brought up different issues since the generated IDL was different

Same issue here.
https://github.com/coral-xyz/anchor/issues/2914#issuecomment-2084711323

All your problems are related to using Anchor 0.30, which had a bunch of breaking changes. I suggest completing the project with 0.29 instead.
Use avm use 0.29 and yarn add<EMAIL_ADDRESS>Make sure Anchor is on v0.29 in your CLI, counter, AND front-end. You'll have to start over with the contract but you can reuse the front-end.

Problem 1: IDL File
0.30 separates the types and the IDL. Instead of just making a types file, you'd copy over:

/target/idl/counter.json
/target/types/counter.ts

Put these in front-end/src/anchor and use them like this:
import * as idl from "./counter.json";
import type { Counter } from "./counter";

Problem 2: Error in setup.js
When creating a program interface in Anchor 0.30, you can't add programId anymore

You'd just do sth like
export const program = new Program(idl, provider);

But I haven't gotten this to work yet cuz provider is breaking.

Problem 3: Error in Frontend
Likely related to the IDL changes in v0.30 but I can't confirm.

Same issue here, I tried switching back to Anchor 0.29.0 and that brought up different issues since the generated IDL was different

@adlonymous see above. You need to have 0.29 in all three places.

Pretty sure, I'm using 0.29.0, but I get in the browser console:
setup.ts:15 Uncaught ReferenceError: Buffer is not defined
    at setup.ts:15:4

Added this into setup.ts and it appears to work now:
import { Buffer } from "buffer"
import * as process from "process"

globalThis.Buffer = Buffer as unknown as BufferConstructor;
globalThis.process = process as unknown as NodeJS.Process;

You'd just do sth like
export const program = new Program(idl, provider);

But I haven't gotten this to work yet cuz provider is breaking.

Problem 3: Error in Frontend

Likely related to the IDL changes in v0.30 but I can't confirm.

Change Provider To AnchorProvider to fix Provider breaking:
https://coral-xyz.github.io/anchor/ts/classes/AnchorProvider.html
Let me know if that fixes it for you cause I'm still working on figuring out the idl and typescript changes in Anchor 0.30.0. Why do they do stuff like rename functions and classes and make breaking idl changes? I hate it. I've been trying to get my frontend to work for 3 weeks now, meanwhile the on chain Rust code has been really smooth for me when it comes to cross compiling and deploying into bytecode.

How to build a Provider using only the public key?


Clarification on Deeplab Model Configuration and Pre-training

Hi,
Thank you for sharing your research and codebase with the community. I'm currently working on replicating the results and have a query regarding the Deeplab model's setup.
In your repository, it's mentioned that your Deeplab training code is adapted from deeplab-pytorch. I wanted to clarify if you strictly adhered to the default Pascal VOC configuration settings provided in that original repository. Additionally, did you utilize the pre-trained model (pretrained on COCO) that the repo offers, or did you employ a different configuration or pre-trained models for your experiments?
Thank you for your time and assistance!

Hi, thanks for your interest,
We provide the detailed setting in our paper (Appendix D). You can also find more details in this issue.


[FLINK-20059][table-common] Update Javadoc for "merge" method of AggregateFunction

What is the purpose of the change
In the java docs as well as the user docs, the merge method of an aggregation UDF is described as optional, e.g.
Merges a group of accumulator instances into one accumulator instance. This function must be implemented for data stream session window grouping aggregates and data set grouping aggregates.

However, it seems that nowadays this method is required in more cases (e.g. HOP window in streaming).
Brief change log

Update the javadoc of "merge" method.
Update documentations.

Verifying this change
This change is a trivial rework without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Build is passed: https://dev.azure.com/apache-flink/apache-flink/_build/results?buildId=9852&view=results


feat: Product page with the status bar from the POC

Hi everyone!
One more feature from the POC: the status bar containing the compatibility score.
🎥 Here's a demo: https://github.com/user-attachments/assets/572374c0-4524-4cdb-ad01-70d9cbea7fe9
FYI, this implementation is temporary to let us revamp the onboarding (by removing the food prefs).
Normally, it requires the Sliver mechanism, hence, the non-blocking AppBar on scroll (you have to test to understand this)
Also, I wonder if we shouldn't change a bit the compatibility colors… otherwise, there is never some red.
For instance, does 50% deserve a green?

Codecov Report
Attention: Patch coverage is 3.04569% with 191 lines in your changes missing coverage. Please review.

Project coverage is 7.07%. Comparing base (4d9c7fc) to head (acd6ba7).
Report is 439 commits behind head on develop.

Files with missing lines
Patch %
Lines

...pages/product/product_page/new_product_header.dart
3.03%
96 Missing :warning:

...b/pages/product/product_page/new_product_page.dart
0.00%
67 Missing :warning:

..._app/lib/helpers/product_compatibility_helper.dart
0.00%
9 Missing :warning:

...pages/product/product_page/new_product_footer.dart
0.00%
6 Missing :warning:

...ges/smooth_app/lib/pages/product/summary_card.dart
0.00%
5 Missing :warning:

...ges/smooth_app/lib/themes/smooth_theme_colors.dart
0.00%
4 Missing :warning:

...cards/product_cards/smooth_product_card_found.dart
0.00%
1 Missing :warning:

...mooth_app/lib/pages/personalized_ranking_page.dart
0.00%
1 Missing :warning:

..._app/lib/pages/product/compare_products3_page.dart
0.00%
1 Missing :warning:

...ib/pages/product/product_compatibility_header.dart
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           develop   #5808      +/-   ##
==========================================
- Coverage     9.54%   7.07%   -2.48%
==========================================
  Files          325     412      +87
  Lines        16411   22340    +5929
==========================================
+ Hits          1567    1581      +14
- Misses       14844   20759    +5915

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

I was thinking of turning the circular numerical indicator into a "Road to Scores" shortcut when it is the case @g123k Otherwise, LGTM

That's a nice idea.
I create an issue for that


High Gas Use- FP

Attacker: 0x5c0e2ea17832a735c71731ef35d3b348fad78f97
AD Alert: https://app.forta.network/alerts/0x6e650884aa0ced3344a12c2f47e798cbec40aa2e46a5c2adc15a8fd68399c864
Base bots:  High Gas Use Bot
Potential cause:  base alerts https://app.forta.network/alerts/0x4acd990b1deab6ec11feee9ee0b028a4cde112257858765afd8a4b32c0bf40a0https://app.forta.network/alerts/0x372ea1e53c1ed3ad11e6a55ee0cde44e59df6a03b596ce64d1ea7514efc0cd0d exclude airdrop/batchsending txn.

Other alerts

AD: https://app.forta.network/alerts/0x2acea6645547c4cb68c59fc19bc53e3a85a7374a64fc3508951394f56119e7e0
base alert: https://app.forta.network/alerts/0xf07190fdeede3011a7960f2352d9c81f62eee855aa1ce88366b94971b2b966e8  exclude  gas token minting txn.

The first two weren't caught by the separate "Airdrop High Gas" alert we've got, because it expects all tokens to be transferred by one address, while this wasn't the case in this tx (the first token transfer in each was to Disperse and the rest were from Disperse). I updated the bot to fix that in this PR.
Regarding gas tokens, this has been added to the bot since the previous update.


How can I utilize this ??

A nice README guide on how to use this will be awesome, a getting started guide !!

Sure! I will be improve the readme, thanks!
Quick advance:

Run the app (npm start)
Log in with account (A) you want (web whatsapp is loaded)
Send a message from another account (B) to this account (A)
Account (A) reply a dummy message
Your work, include a library to use NLP o something bot messaging system.


Deal with multiple sessions.

Make tmux-easy-motion work in more than one session by:

specifying the exact target (session, window, pane) of commands
using a separate pipe for each session

Need to merge https://github.com/IngoMeyer441/tmux-easy-motion/pull/3 first.

Thanks for your second PR. I need some more explanation: Did tmux-easy-motion not work for you in different sessions without targeting window and pane explicitly? When programming tmux-easy-motion, I thought it would be sufficient to target the active pane.

Thanks for your second PR. I need some more explanation: Did tmux-easy-motion not work for you in different sessions without targeting window and pane explicitly? When programming tmux-easy-motion, I thought it would be sufficient to target the active pane.

Thanks for looking.  Yeah, the PR tries to resolve some bugs I've seen while using tmux-easy-motion.  Here's an easy way to see one of these bugs on master:

launch tmux with a minimal conf file tmux -L test -f minimal.conf such as

run-shell "/path/to/tmux-easy-motion/easy_motion.tmux"

cat a large file
create a new session
cat a large file in that new session
in a second terminal window (or via ssh) attach to the server again and switch to the second session
in the first terminal window, switch to the first session
in the first terminal window, start tmux-easymotion, for example with prefix space s s
in the second terminal window, start tmux-easymotion, for example with prefix space s a
in the second terminal window, press a key to choose a target
at this point, the first session is broken as shown here:

I thought it would be sufficient to target the active pane.

Not having the session, pane and window ids which the client had active when launching easy_motion.sh available to work with results in having to do unnecessary work.  For example, using awk when you could just ask tmux for exactly what you want: https://github.com/IngoMeyer441/tmux-easy-motion/pull/6/files#diff-e015cc3ac522b202bb23852488a7ac2c019e2c82568ecb387b97b60ebbecdf4bL61 .  Another example, asking tmux for information that could have just been passed directly to the script: https://github.com/IngoMeyer441/tmux-easy-motion/pull/6/files#diff-e015cc3ac522b202bb23852488a7ac2c019e2c82568ecb387b97b60ebbecdf4bL52 .  Launching each of these subprocesses is slow, and launching many adds up.
I'll split the commit up into two parts, one which just changes the pipe, and the other which specifies tmux command targets, if the patch works without the second part (which I haven't tried).

Thanks for looking.  Yeah, the PR tries to resolve some bugs I've seen while using tmux-easy-motion.  Here's an easy way to see one of these bugs on master:

launch tmux with a minimal conf file tmux -L test -f minimal.conf such as

run-shell "/path/to/tmux-easy-motion/easy_motion.tmux"

cat a large file
create a new session
cat a large file in that new session
in a second terminal window (or via ssh) attach to the server again and switch to the second session
in the first terminal window, switch to the first session
in the first terminal window, start tmux-easymotion, for example with prefix space s s
in the second terminal window, start tmux-easymotion, for example with prefix space s a
in the second terminal window, press a key to choose a target
at this point, the first session is broken as shown here:

I thought it would be sufficient to target the active pane.

Not having the session, pane and window ids which the client had active when launching easy_motion.sh available to work with results in having to do unnecessary work.  For example, using awk when you could just ask tmux for exactly what you want: https://github.com/IngoMeyer441/tmux-easy-motion/pull/6/files#diff-e015cc3ac522b202bb23852488a7ac2c019e2c82568ecb387b97b60ebbecdf4bL61 .  Another example, asking tmux for information that could have just been passed directly to the script: https://github.com/IngoMeyer441/tmux-easy-motion/pull/6/files#diff-e015cc3ac522b202bb23852488a7ac2c019e2c82568ecb387b97b60ebbecdf4bL52 .  Launching each of these subprocesses is slow, and launching many adds up.
I'll split the commit up into two parts, one which just changes the pipe, and the other which specifies tmux command targets, if the patch works without the second part (which I haven't tried).

Did you really experience performance impacts? When I wrote the first version of tmux-easy-motion I did not really think about performance since I didn't see this plugin as a performance critical program. But it would of course be nice to minimize the delays between a motion command and the display of targets on screen.

Did you really experience performance impacts? When I wrote the first version of tmux-easy-motion I did not really think about performance since I didn't see this plugin as a performance critical program. But it would of course be nice to minimize the delays between a motion command and the display of targets on screen.

What I tried to say: If performance is not relevant, I think it is more important to have readable code and I first thought that passing many arguments would harm readability. But on a second look, the readability is not worse than my version and your changes save unneeded work, so great :+1:

What I tried to say: If performance is not relevant, I think it is more important to have readable code and I first thought that passing many arguments would harm readability. But on a second look, the readability is not worse than my version and your changes save unneeded work, so great :+1:

Okay!  The display-message -p does make for readable code.
What the previous shell scripting for tmux I've done has indicated is that getting rid of subprocesses (cut, awk, sed, and friends) and subshell $() function calling can make a big difference.  And that's very noticeable when tmux isn't responsive as the script executes.  Here's a post chatting about how expensive subshelling is, for example: http://rus.har.mn/blog/2010-07-05/subshells/ .  And there are better ways of writing functions to avoid subshelling than the technique in that post.  You can use indirect references https://tldp.org/LDP/abs/html/ivr.html and take as an argument the name of the variable to assign.

Okay!  The display-message -p does make for readable code.
What the previous shell scripting for tmux I've done has indicated is that getting rid of subprocesses (cut, awk, sed, and friends) and subshell $() function calling can make a big difference.  And that's very noticeable when tmux isn't responsive as the script executes.  Here's a post chatting about how expensive subshelling is, for example: http://rus.har.mn/blog/2010-07-05/subshells/ .  And there are better ways of writing functions to avoid subshelling than the technique in that post.  You can use indirect references https://tldp.org/LDP/abs/html/ivr.html and take as an argument the name of the variable to assign.

Indirect references in functions work with eval if I understand correctly. eval is also quite slow, isn't it?

Indirect references in functions work with eval if I understand correctly. eval is also quite slow, isn't it?

The great thing about this PR is that the session-closed hook can now be used to clean up the temporary session directories 👍.

The great thing about this PR is that the session-closed hook can now be used to clean up the temporary session directories 👍.

Nice idea with session-closed!  I'll do that.
I used superfine to measure the performance of calling a string concatenation function 10000 times written 1. using subshelling 2. using an agreed variable name (retval in the post) and 3. using an indirect reference.  Subshelling was 8.619 s ± 0.153 s.  Agreed variable name was 198.7 ms ± 14.7 ms.  Indirect was 285.6 ms ± 5.9 ms.  So, an indirect reference is a little slower than a named variable, and both are much faster than subshelling.

Nice idea with session-closed!  I'll do that.
I used superfine to measure the performance of calling a string concatenation function 10000 times written 1. using subshelling 2. using an agreed variable name (retval in the post) and 3. using an indirect reference.  Subshelling was 8.619 s ± 0.153 s.  Agreed variable name was 198.7 ms ± 14.7 ms.  Indirect was 285.6 ms ± 5.9 ms.  So, an indirect reference is a little slower than a named variable, and both are much faster than subshelling.

I used superfine to measure the performance of calling a string concatenation function 10000 times written 1. using subshelling 2. using an agreed variable name (retval in the post) and 3. using an indirect reference. Subshelling was 8.619 s ± 0.153 s. Agreed variable name was 198.7 ms ± 14.7 ms. Indirect was 285.6 ms ± 5.9 ms. So, an indirect reference is a little slower than a named variable, and both are much faster than subshelling.

Ok, I learned something new, thanks :smile:
I have only two minor change requests to the current code, the rest looks good to me!

I used superfine to measure the performance of calling a string concatenation function 10000 times written 1. using subshelling 2. using an agreed variable name (retval in the post) and 3. using an indirect reference. Subshelling was 8.619 s ± 0.153 s. Agreed variable name was 198.7 ms ± 14.7 ms. Indirect was 285.6 ms ± 5.9 ms. So, an indirect reference is a little slower than a named variable, and both are much faster than subshelling.

Ok, I learned something new, thanks :smile:
I have only two minor change requests to the current code, the rest looks good to me!

Okay!  I made those changes and one more to take the $ prefix from the session_id only when creating the path.  I think this is ready, and we can add the cleanup on session close in another commit.

Okay!  I made those changes and one more to take the $ prefix from the session_id only when creating the path.  I think this is ready, and we can add the cleanup on session close in another commit.

Thanks again for your time and work! :+1:

Thanks again for your time and work! :+1:

Commit 524b40566888bdecd85834279f7c073aced9886e adds a session-closed hook to clean up the target key pipe temporary directories and it worked so far in my tests.

Commit 524b40566888bdecd85834279f7c073aced9886e adds a session-closed hook to clean up the target key pipe temporary directories and it worked so far in my tests.


snake_case to camelCase ignoring whitespace

Describe the bug
When attempting to convert a whole file with snake case string to camel case string, it convert the whole line in camel case instead of the word in snake case :
Example :

String some_snake_case
int another_snake_case

go to

StringSomeSnakeCase
intAnotherSnakeCase

instead of

String someSnakeCase
int anotherSnakeCase

To Reproduce
Having a file full of snake_case string
use the option from menu :
Edit -> String Manipulation -> switch case -> snake_case / camelCase
Environment :
Tests have been made in intelliJ ultimate and rubyMine (both last version)

Yes, it converts everything selected. Autodetection of suitable substrings would be problematic.

I would recommend using Main Menu | Edit | Find | Select All Occurrences.

This would help you:
Action 'Select All Occurrences For All Carets' was added, bind at Keymap
see https://youtrack.jetbrains.com/issue/IDEA-283466
Implementing what you want is not feasible.


[FLINK-9473][Table API & SQL] Added new methods into ExternalCatalogSchema based on interface Schema changes in Calcite

What is the purpose of the change
Make Flink compilable after Calcite version bump to 1.17.0
Brief change log

new abstract methods dummy implementation
bump calcite version to 1.17.0-SNAPSHOT

Verifying this change
to verify need to check if it or compilable or not e.g.
mvn clean package -DskipTests
This change added tests and can be verified as follows:
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no )
The runtime per-record code paths (performance sensitive): (no )
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no )
The S3 file system connector: (no )

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable )

Thanks for the PR @snuyanzin!
We cannot update the Calcite version to a SNAPSHOT version and have to wait until Calcite 1.17 is released before continuing with this PR.
Best, Fabian

thank you for your comment I will close  this PR as Calcite 1.17.0 released and I found more things to be fixed especially relating to the test. As soon as I finished I will create a new PR with mentioning of Calcite's tickets because of which some changes in  Flink's code are required

We can close this PR as this is covered by #6484.


Respeaker commands use linear volume scale

I can't maintain this module any more.
Have you interest to take new ownership of this module?
https://forum.magicmirror.builders/topic/12779/wanted-new-ownership-of-modules?page=1

Yes

Ok. tell me your Forum ID(if exists) and remove your current your fork of this module on your account to take over this. then tell me. I'll transfer

I don't have forum id. Fork is removed.

done. Allow transfer, now it's yours.


com.orientechnologies.orient.enterprise.channel.binary.OTokenSecurityException: The token provided is expired

Our tests initiated using Jmeter are failing after 4th iteration.
More details are as below :-
2016-07-26 15:06:29:993 INFO  OrientDB auto-config DISKCACHE=6,372MB (heap=6,372MB direct=6,372MB os=28,671MB), assuming maximum direct memory size equals to maximum JVM heap size [orientechnologies]
2016-07-26 15:06:30:047 WARNI MaxDirectMemorySize JVM option is not set or has invalid value, that may cause out of memory errors. Please set the -XX:MaxDirectMemorySize=28671m option when you start the JVM. [Class]
2016-07-26 15:06:30:145 WARNI MaxDirectMemorySize JVM option is not set or has invalid value, that may cause out of memory errors. Please set the -XX:MaxDirectMemorySize=28671m option when you start the JVM. [Class]
2016-07-26 15:06:30:181 INFO  Loading configuration from: C:/IntelliSURF/orientdb-community-2.2.0-GA/config/orientdb-server-config.xml... [OServerConfigurationLoaderXml]
2016-07-26 15:06:30:698 INFO  OrientDB Server v2.2.0 (build @BUILD@) is starting up... [OServer]
2016-07-26 15:06:30:709 INFO  Databases directory: C:\IntelliSURF\orientdb-community-2.2.0-GA\databases [OServer]
2016-07-26 15:06:31:107 INFO  Listening binary connections on <IP_ADDRESS>:3424 (protocol v.36, socket=default) [OServerNetworkListener]
2016-07-26 15:06:31:110 INFO  Listening http connections on <IP_ADDRESS>:2480 (protocol v.10, socket=default) [OServerNetworkListener]
2016-07-26 15:06:31:120 WARNI Found ORIENTDB_ROOT_PASSWORD variable, using this value as root's password [OServer]
2016-07-26 15:06:31:605 INFO  Installing dynamic plugin 'orientdb-spatial-2.2.0-dist.jar'... [OServerPluginManager]
2016-07-26 15:06:31:616 INFO  Spatial index plugin startup [OLuceneSpatialIndexPlugin]
2016-07-26 15:06:31:718 INFO  Spatial index plugin installed and active. Lucene version: 5.3.2 [OLuceneSpatialIndexPlugin]
2016-07-26 15:06:31:719 INFO  Installing dynamic plugin 'studio-2.2.zip'... [OServerPluginManager]
2016-07-26 15:06:31:761 INFO  Installed GREMLIN language v.2.6.0 - graph.pool.max=50 [OGraphServerHandler]
2016-07-26 15:06:31:761 INFO  [OVariableParser.resolveVariables] Error on resolving property: distributed [orientechnologies]
2016-07-26 15:06:31:761 WARNI Authenticated clients can execute any kind of code into the server by using the following allowed languages: [sql] [OServerSideScriptInterpreter]
2016-07-26 15:06:31:770 INFO  ODefaultPasswordAuthenticator is active [ODefaultPasswordAuthenticator]
2016-07-26 15:06:31:770 INFO  OServerConfigAuthenticator is active [OServerConfigAuthenticator]
2016-07-26 15:06:31:770 INFO  OSystemUserAuthenticator is active [OSystemUserAuthenticator]
2016-07-26 15:06:31:770 INFO  OrientDB Server is active v2.2.0 (build @BUILD@). [OServer]
2016-07-26 15:07:12:402 WARNI {db=isc} Storage 'isc' was not closed properly. Will try to recover from write ahead log [OLocalPaginatedStorage]
2016-07-26 15:07:12:403 INFO  {db=isc} Looking for last checkpoint... [OLocalPaginatedStorage]
2016-07-26 15:07:13:556 INFO  {db=isc} Found FUZZY checkpoint. [OLocalPaginatedStorage]
2016-07-26 15:07:13:557 INFO  {db=isc} Data restore procedure from FUZZY checkpoint is started. [OLocalPaginatedStorage]
2016-07-26 15:07:13:559 WARNI {db=isc} Record com.orientechnologies.orient.core.storage.impl.local.paginated.wal.OFuzzyCheckpointEndRecord{lsn=OLogSequenceNumber{segment=0, position=30710138}} will be skipped during data restore [OLocalPaginatedStorage]
2016-07-26 15:07:13:567 WARNI {db=isc} Record OFuzzyCheckpointStartRecord{lsn=OLogSequenceNumber{segment=0, position=30710145}} com.orientechnologies.orient.core.storage.impl.local.paginated.wal.OFuzzyCheckpointStartRecord{lsn=null, previousCheckpoint=OLogSequenceNumber{segment=0, position=30710098}} will be skipped during data restore [OLocalPaginatedStorage]
2016-07-26 15:07:13:567 WARNI {db=isc} Record com.orientechnologies.orient.core.storage.impl.local.paginated.wal.OFuzzyCheckpointEndRecord{lsn=OLogSequenceNumber{segment=0, position=30710185}} will be skipped during data restore [OLocalPaginatedStorage]
2016-07-26 15:07:13:567 INFO  {db=isc} Storage data recover was completed [OLocalPaginatedStorage]Error executing request
com.orientechnologies.orient.enterprise.channel.binary.OTokenSecurityException: The token provided is expired
	at com.orientechnologies.orient.server.OClientConnection.validateSession(OClientConnection.java:167)
	at com.orientechnologies.orient.server.network.protocol.binary.ONetworkProtocolBinary.solveSession(ONetworkProtocolBinary.java:274)
	at com.orientechnologies.orient.server.network.protocol.binary.ONetworkProtocolBinary.onBeforeRequest(ONetworkProtocolBinary.java:215)
	at com.orientechnologies.orient.server.network.protocol.binary.ONetworkProtocolBinary.execute(ONetworkProtocolBinary.java:173)
	at com.orientechnologies.common.thread.OSoftThread.run(OSoftThread.java:77)
Error executing request

Hi @kumarkshiva
The exception you see in the stack trace was already fixed in v. 2.2.5 (latest stable), do you have a chance to try it?
Thanks
Luigi

after upgrade to 2.2.5 , we are seeing the issues :-
2016-08-05 17:31:57:530 INFO  Loading configuration from: C:/IntelliSURF/orientdb-community-2.2.5/config/orientdb-server-config.xml... [OServerConfigurationLoaderXml]
2016-08-05 17:31:57:809 INFO  OrientDB Server v2.2.5 (build @BUILD@) is starting up... [OServer]
2016-08-05 17:31:57:815 INFO  Databases directory: C:\IntelliSURF\orientdb-community-2.2.5\databases [OServer]
2016-08-05 17:31:57:876 INFO  OrientDB auto-config DISKCACHE=3,600MB (heap=3,600MB direct=3,600MB os=16,197MB), assuming maximum direct memory size equals to maximum JVM heap size [orientechnologies]
2016-08-05 17:31:57:877 WARNI MaxDirectMemorySize JVM option is not set or has invalid value, that may cause out of memory errors. Please set the -XX:MaxDirectMemorySize=16197m option when you start the JVM. [Class]
2016-08-05 17:31:58:769 INFO  Listening binary connections on <IP_ADDRESS>:3424 (protocol v.36, socket=default) [OServerNetworkListener]
2016-08-05 17:31:58:776 INFO  Listening http connections on <IP_ADDRESS>:3480 (protocol v.10, socket=default) [OServerNetworkListener]
2016-08-05 17:31:58:788 INFO  Installing dynamic plugin 'studio-2.2.zip'... [OServerPluginManager]
2016-08-05 17:31:58:829 INFO  Installed GREMLIN language v.2.6.0 - graph.pool.max=50 [OGraphServerHandler]
2016-08-05 17:31:58:830 INFO  [OVariableParser.resolveVariables] Error on resolving property: distributed [orientechnologies]
2016-08-05 17:31:58:835 WARNI Authenticated clients can execute any kind of code into the server by using the following allowed languages: [sql] [OServerSideScriptInterpreter]
2016-08-05 17:31:58:839 INFO  ODefaultPasswordAuthenticator is active [ODefaultPasswordAuthenticator]
2016-08-05 17:31:58:841 INFO  OServerConfigAuthenticator is active [OServerConfigAuthenticator]
2016-08-05 17:31:58:842 INFO  OSystemUserAuthenticator is active [OSystemUserAuthenticator]
2016-08-05 17:31:58:844 INFO  OrientDB Server is active v2.2.5 (build @BUILD@). [OServer]
2016-08-05 17:31:59:086 WARNI Token signature failure: AAAAAANpc2MA/////////////wAAAVZP2KtSAQAEcm9vdAAkABdPUmVjb3JkU2VyaWFsaXplckJpbmFyeQANT3JpZW50REIgSmF2YQAFMi4yLjADEseWC0dOHN2KMXqF2cvh0/cNudXkzN/0fZ9yA88s9w== [OTokenHandlerImpl]Error executing request
com.orientechnologies.orient.enterprise.channel.binary.OTokenSecurityException: The token provided is not a valid token, signature does not match
at com.orientechnologies.orient.server.OClientConnection.validateSession(OClientConnection.java:174)
at com.orientechnologies.orient.server.network.protocol.binary.ONetworkProtocolBinary.solveSession(ONetworkProtocolBinary.java:289)
at com.orientechnologies.orient.server.network.protocol.binary.ONetworkProtocolBinary.onBeforeRequest(ONetworkProtocolBinary.java:226)
at com.orientechnologies.orient.server.network.protocol.binary.ONetworkProtocolBinary.execute(ONetworkProtocolBinary.java:184)
at com.orientechnologies.common.thread.OSoftThread.run(OSoftThread.java:77)

I am experiencing the same problem with v 2.2.6.
I have a java application (spring-boot) connected to an OrientDB Server through the java client (I use the connections pool). When the application starts I don't see any problem. After a while every request throws the below errors on the ODB server. It's worth noticing that the request return the correct result but the errors slow down the query drastically.
Error executing request
orientdb_1  | com.orientechnologies.orient.enterprise.channel.binary.OTokenSecurityException: The token provided is expired
orientdb_1  | 	DB name="my-db"
orientdb_1  | 	at com.orientechnologies.orient.server.OClientConnection.validateSession(OClientConnection.java:178)
orientdb_1  | 	at com.orientechnologies.orient.server.network.protocol.binary.ONetworkProtocolBinary.solveSession(ONetworkProtocolBinary.java:294)
orientdb_1  | 	at com.orientechnologies.orient.server.network.protocol.binary.ONetworkProtocolBinary.onBeforeRequest(ONetworkProtocolBinary.java:231)
orientdb_1  | 	at com.orientechnologies.orient.server.network.protocol.binary.ONetworkProtocolBinary.execute(ONetworkProtocolBinary.java:188)
orientdb_1  | 	at com.orientechnologies.common.thread.OSoftThread.run(OSoftThread.java:77)
orientdb_1  | Error executing request
orientdb_1  | com.orientechnologies.orient.enterprise.channel.binary.OTokenSecurityException: missing session and token
orientdb_1  | 	DB name="my-db"
orientdb_1  | 	at com.orientechnologies.orient.server.network.protocol.binary.ONetworkProtocolBinary.solveSession(ONetworkProtocolBinary.java:290)
orientdb_1  | 	at com.orientechnologies.orient.server.network.protocol.binary.ONetworkProtocolBinary.onBeforeRequest(ONetworkProtocolBinary.java:231)
orientdb_1  | 	at com.orientechnologies.orient.server.network.protocol.binary.ONetworkProtocolBinary.execute(ONetworkProtocolBinary.java:188)
orientdb_1  | 	at com.orientechnologies.common.thread.OSoftThread.run(OSoftThread.java:77)

I have upgraded client and server to v 2.2.7 and tested it for several hours.
I can't see the server side errors anymore. When the client makes some request after a period of inactivity I see the following logs
2016-08-18 10:17:54.876  INFO 4563 --- [nio-8090-exec-9] c.o.orient.client.remote.OStorageRemote  : Caught Network I/O errors on <IP_ADDRESS>:2424/my-db, trying an automatic reconnection... (error: Channel is closed)
After a few requests these logs disappear and the queries response time improves (back to the original value I'd say). I think this is the correct behaviour: the 'bad' connections in the pool get replaced with the 'good' ones.
The issue has been fixed. Thanks!

Cool, thanks for the report.


Dummy PR for testing

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/5481
Tests

QA Steps

Tested On

[ ] Web
[ ] Mobile Web
[ ] Desktop
[ ] iOS
[ ] Android

Screenshots

Web

Mobile Web

Desktop

iOS

Android

I have read the CLA Document and I hereby sign the CLA

InternalQA for https://github.com/Expensify/Web-Expensify/pull/32025! Please ignore :)


Code should use threads to run in parallel

You should be able to run in parallel by running multiple processes right now, but it might be faster/more accurate to run multiple threads in one process.
This necessitates some way of having each thread write to a different output file though, so it's not totally trivial.

This is already done.