Session 1 rails

@harishA3
You have successfully completed the task, but our tests expected a set of different results, but that doesn't matter since you have understood on how to display data using HTML tags.

@harishA3
You have successfully completed the task, but our tests expected a set of different results, but that doesn't matter since you have understood on how to display data using HTML tags.


[ Question ] need we install impatient.nvim ?

it seems lazy.nvim is already integrated the impatient.nvim do we need to install it still?

Impatient isn't necessary when using lazy from my understanding. If you do want to use it, you should also probably disable the caching in Lazy.


Redefining methods do not work if untyped

Using bsh-2.0b4.jar
This case works
bsh % foo() { print("hi");}
bsh % foo() { print("bye"); }
bsh % foo();
bye
This case fails
bsh % foo( elem ) { print("a");}
bsh % foo( elem ) { print("b"); }
bsh % foo(1);
a                ----> SHOULD BE  "b"
This works
bsh % foo( int elem ) { print("a");}
bsh % foo(int elem ) { print("b"); }
bsh % foo(1);
b
bsh %
Thus if you redefine a method which takes untyped
arguments the redefinition doesnt succeed.
Testing in 2.0b2 this works therefore this is a
regression in 2.0b(3 or 4).
Reported by: *anonymous
Original Ticket: "beanshell/bugs/270":https://sourceforge.net/p/beanshell/bugs/270

This appears to be fixed.  Should be closed.

@opeongo tyvm
Closed: working


Bump blade-web3 to v1.3.

Description:
Bump blade-web3 to v1.3. Fix transactions populated by BladeWallet on mainnet. Fix appMetadata modifying.
Related issue(s):
Fix transactions populated by BladeWallet on mainnet.
Fix appMetadata modifying.
Checklist

[x] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

Hello @dimasmoliar ,
We are testing your branch : feature/bump_blade_web3.

We run "npm install" from root : installation successful
We create .env file in /web as such:

# Log level for the application. The default value is ERROR.
# Acceptable values are ERROR, WARN, INFO, HTTP, VERBOSE, DEBUG and SILLY in order of priority (highest to lowest).
REACT_APP_LOG_LEVEL=ERROR
# JSON array with a factory id in Hedera format `0.0.XXXXX` per environment.
REACT_APP_FACTORIES='[{"Environment":"mainnet","STABLE_COIN_FACTORY_ADDRESS":"0.0.2167166"}, {"Environment":"testnet","STABLE_COIN_FACTORY_ADDRESS":"0.0.3705516"}]'
# The var must be a unique mirror node service for Hedera network, and this is the service which would be used when the UI starts. If the service doesn't require an API key to authorize requests, the API_KEY and HEADER properties must remain empty.
REACT_APP_MIRROR_NODE = '[{"Environment":"mainnet","BASE_URL":"https://mainnet.mirrornode.hedera.com/api/v1/", "API_KEY": "", "HEADER": ""},{"Environment":"testnet","BASE_URL":"https://testnet.mirrornode.hedera.com/api/v1/", "API_KEY": "", "HEADER": ""}]'
REACT_APP_RPC_NODE = '[{"Environment":"mainnet","BASE_URL":"https://mainnet.hashio.io/api/", "API_KEY": "", "HEADER": ""},{"Environment":"testnet","BASE_URL":"http://<IP_ADDRESS>:7546/api/", "API_KEY": "", "HEADER": ""}]'
# This is a proprietary Create React App configuration. You can read more information in its documentation.[Create React App documentation](https://create-react-app.dev/docs/advanced-configuration/)
GENERATE_SOURCEMAP=false
REACT_APP_SHOW_CONFIG=true

We connect using Blade : successful
We try to deploy a stablecoin (default values everywhere, no reserve), after signing the transaction with Blade we get the following errors:

There was an issue with the icon displayed by Blade when connecting, I fixed it and commited the new changes. Blade v1.6.1 works fine however there is an issue with Hashpack, when connecting we get this error


feat: rename devicename and profilename

Signed-off-by: Valina Li<EMAIL_ADDRESS>

@lenny-intel  this is the first draft of rename devicename and profile name, I will make other files change if new names are good


Feature Request: literature notes template

Feature Requested
(I see that the TODO is embedded in the workflow as is, if this isn't too rude: I hope to give the template function a bump!)
Relevant Screenshot
No response
Checklist

[X] The feature would be useful to more users than just me.

might consult https://github.com/PKM-er/obsidian-zotlit?tab=readme-ov-file for its function on generating literature note!

yeah, sorry, I personally do not have the need for it, and in more than half a year, there is also not a single other person that has expressed their interest in that feature. Considering that my own time is somewhat limited, I'm gonna have to pass on this feature due to insufficient demand, sorry!


memory leak while inserting data to database?

I use this library for storing data into a sqlite database and faced some memory leak issues.
After I wrapped the following example code in a while loop, I can reproduce the same memory leak issues
https://github.com/SRombauts/SQLiteCpp/blob/adb7e7c489b0574a6c2f354ab14213bfb9dd4e49/examples/example2/src/main.cpp#L52-L59
Valgrind can also see the memory leak:
==24672== Memcheck, a memory error detector
==24672== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al.
==24672== Using Valgrind-3.13.0 and LibVEX; rerun with -h for copyright info
==24672== Command: ./SQLiteCpp_Example
==24672==
==24672==
==24672== Process terminating with default action of signal 2 (SIGINT)
==24672==    at 0x55F2AF1: fsync (fsync.c:27)
==24672==    by 0x130B86: full_fsync (sqlite3.c:37187)
==24672==    by 0x130D1C: unixSync (sqlite3.c:37280)
==24672==    by 0x125D12: sqlite3OsSync (sqlite3.c:23067)
==24672==    by 0x13ADA6: syncJournal (sqlite3.c:55971)
==24672==    by 0x13D3CA: sqlite3PagerCommitPhaseOne (sqlite3.c:58217)
==24672==    by 0x14878A: sqlite3BtreeCommitPhaseOne (sqlite3.c:68496)
==24672==    by 0x15A0CD: vdbeCommit (sqlite3.c:80458)
==24672==    by 0x15AA70: sqlite3VdbeHalt (sqlite3.c:80860)
==24672==    by 0x1616BF: sqlite3VdbeExec (sqlite3.c:86445)
==24672==    by 0x15E72A: sqlite3Step (sqlite3.c:83691)
==24672==    by 0x15E985: sqlite3_step (sqlite3.c:83748)
==24672==
==24672== HEAP SUMMARY:
==24672==     in use at exit: 153,872 bytes in 90 blocks
==24672==   total heap usage: 675 allocs, 585 frees, 514,656 bytes allocated
==24672==
==24672== 4,104 bytes in 1 blocks are possibly lost in loss record 40 of 43
==24672==    at 0x4C2FB0F: malloc (vg_replace_malloc.c:299)
==24672==    by 0x126655: sqlite3MemMalloc (sqlite3.c:23694)
==24672==    by 0x127020: mallocWithAlarm (sqlite3.c:27576)
==24672==    by 0x1270B9: sqlite3Malloc (sqlite3.c:27606)
==24672==    by 0x135C30: pcache1Alloc (sqlite3.c:50117)
==24672==    by 0x135F0B: sqlite3PageMalloc (sqlite3.c:50260)
==24672==    by 0x14699B: allocateTempSpace (sqlite3.c:67154)
==24672==    by 0x148D3F: btreeCursor (sqlite3.c:68886)
==24672==    by 0x148F25: sqlite3BtreeCursor (sqlite3.c:68943)
==24672==    by 0x166055: sqlite3VdbeExec (sqlite3.c:89176)
==24672==    by 0x15E72A: sqlite3Step (sqlite3.c:83691)
==24672==    by 0x15E985: sqlite3_step (sqlite3.c:83748)
==24672==
==24672== LEAK SUMMARY:
==24672==    definitely lost: 0 bytes in 0 blocks
==24672==    indirectly lost: 0 bytes in 0 blocks
==24672==      possibly lost: 4,104 bytes in 1 blocks
==24672==    still reachable: 149,768 bytes in 89 blocks
==24672==                       of which reachable via heuristic:
==24672==                         length64           : 149,768 bytes in 89 blocks
==24672==         suppressed: 0 bytes in 0 blocks
==24672== Reachable blocks (those to which a pointer was found) are not shown.
==24672== To see them, rerun with: --leak-check=full --show-leak-kinds=all
==24672==
==24672== For counts of detected and suppressed errors, rerun with: -v
==24672== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 0 from 0)

The C interface of SQLite library provide the function int sqlite3_finalize(sqlite3_stmt *pStmt);  for cleaning up the allocated memory, but I can not find any wrapper in your c++ wrapper interface.

Sorry that I didn't try to understand, reproduce, or even answer your question at that time.
What Valgrind output is showing is not memory leak; there is nothing definitely lost

I added some do in the README on how to run valgrind memcheck on the provided "example1" little program https://github.com/SRombauts/SQLiteCpp#valgrind-memcheck
valgrind --leak-check=full --show-leak-kinds=all --track-origins=yes --verbose build/SQLiteCpp_example1

==5339== Memcheck, a memory error detector
==5339== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al.
==5339== Using Valgrind-3.18.1-42b08ed5bd-20211015 and LibVEX; rerun with -h for copyright info
==5339== Command: build/SQLiteCpp_example1
==5339==

[...]

everything ok, quitting
--5339-- REDIR: 0x4c49db0 (libc.so.6:__strcat_avx2) redirected to 0x484ea20 (strcat)
==5339==
==5339== HEAP SUMMARY:
==5339==     in use at exit: 0 bytes in 0 blocks
==5339==   total heap usage: 717 allocs, 717 frees, 1,095,491 bytes allocated
==5339==
==5339== All heap blocks were freed -- no leaks are possible
==5339==
==5339== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)
sebastien@DESKTOP-UUDF0FC:~/SQLiteCpp$ ^Cthub.com/SRombauts/SQLiteCpp.git/'
sebastien@DESKTOP-UUDF0FC:~/SQLiteCpp$


Cortx Request Rate Limit

Is there a default limit to Cortx's request rate?
I am testing out the H2O integration and I'm hitting the below error, which seems to indicate the request rate is too high. Is there any way to configure the request rate limit?
Reduce your request rate (Service: Amazon S3; Status Code: 503; Error Code: ServiceUnavailable;
Request ID: 00000000-0000-0000-0000-000000000000; S3 Extended Request ID: null; Proxy: null)

The step that failed is under section 7
# Load data into H2O from Seagate CORTX Server
df = h2o.import_file(path = "s3://testbucket/h2o.csv")

For the convenience of the Seagate development team, this issue has been mirrored in a private Seagate Jira Server: https://jts.seagate.com/browse/CORTX-29714. Note that community members will not be able to access that Jira server but that is not a problem since all activity in that Jira mirror will be copied into this GitHub issue.

Hi @hseow-seagate thank you for raising the issue, I haven't faced rate limit on CORTX before,  I'll also try the H20 integration and reproduce the same.

Hi @hseow-seagate I have tested the integration on my end using k8s OVA, I'm getting the same error. However when you  reduce the h2o.csv file from ~10k rows to 50 rows the error is not raised.
Parse progress: |████████████████████████████████████████████████████████████████| (done) 100%
@johnbent you are one of the contributors of the notebook, could you please let us know if you tested the integration successfully in the past. Is there an explanation as to why when the file size is reduced the h20 import file function works.
@shailesh-vaidya Is there a default limit to Cortx's request rate?

Hi @r-wambui , I merged the pull request and now github mistakenly tags me as a contributor. I'm afraid that I did not test this myself.

Hi @hseow-seagate I have tested the integration on my end using k8s OVA, I'm getting the same error. However when you reduce the h2o.csv file from ~10k rows to 50 rows the error is not raised.
Parse progress: |████████████████████████████████████████████████████████████████| (done) 100%
@johnbent you are one of the contributors of the notebook, could you please let us know if you tested the integration successfully in the past. Is there an explanation as to why when the file size is reduced the h20 import file function works.
@shailesh-vaidya Is there a default limit to Cortx's request rate?

Hi @r-wambui   - Sorry for the delayed reply. I am not aware of any request rate limit on the CORTX side.  @bkirunge7 Can you please help on this.

Hi @r-wambui, I just tested the H2O integration with the original h2o.cvs (9.5k rows), and did not encounter any error.
I'm using the latest OVA-2.0.0-713 (PI-6).

Suggest you try the latest OVA too.

Closing as an obsolete


agent need to acquire in their team

related to #8074 I do not understand the reason for having team="" go to any team. This is highly un-desirable IMO and WQE should just get in nowhere land. Better the request should not be able to assign if Team="" is the one that is decided.
I don't know why this is happening more frequently now, but this is happening and impacting urgent requests

We have to get rid of Teams and just keep Team if workflow is only assigned in one team. If not we have to keep Teams not Team. Anyway, for now I added the temporary fix which we can deploy next cmsweb. #8098

Appears we had another workflow get sucked down by the hepcloud agent like this:
vlimant_task_HIG-PhaseIITDRSpring17DR-00034__v1_T_170826_004446_7421
At the moment looks like the majority of it got pulled into production agents, but a good fraction is lost in the active hepcloud agent.

@amaltaro @ticoann

temp fix is created #8106 and updated to hepcloud and relval agent (cmssrv237, vocms053)
So those agent shouldn't pull down empty team. The real fix should be removing the teams and make sure team is not empty. (That will be updated next deployment)

thanks. can we please also patch the relval agent so that it does not pick up production work ?

I did both hepcloud and relval. :smile:

great. need to read more :-(

So have been waiting for cmssrv237 to make jobs out of wmagent_dmason_NERSCTEST_170829_201133_755 for about an hour and a half -- see it pulled down the work, but nothing beyond that for the last hour and a half or so.  Impatient, or patch broke something?

Sorry patch broke something. New patch is updated (relval agent as well). Let me know if you still have problem.

I see jobs now -- so far so good!  Will pay attention to next workflow we throw in (likely tomorrow)


Update Azure Functions Core Tool link version

Update link to point to 3.15.1 (3.0.4868) version of tools

@ecfan,
Can you review the proposed changes?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Learn Build status updates of commit 066109d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/logic-apps/create-single-tenant-workflows-visual-studio-code.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@jborsecnik & @joerob-msft: I've asked Joe to submit this PR to the private repo. Closing this PR in the public repo. Thanks!
#please-close


Created example application with @ngrx/store

Hi,
I have recently built an example application using @ngrx/store: https://github.com/sapientglobalmarkets/staffer/tree/master/staffer-ng2-ngrxstore
This uses some idiomatic redux techniques and ideas that I have learnt from this repo. I would be happy if you can do a quick review and also include this in the examples section. I can submit a PR for that.
Great job with the project so far!
Thanks!
Pavan

@pavanpodila Could you submit a PR to include this with repos in the resources section of upcoming docs? Great work! https://github.com/ngrx/ngrx.github.io/blob/master/store/help/resources.md

Thanks! here is the PR

I have closed the PR and will instead include this in my personal examples repo, thanks!

Ok. Thanks!


Connection pooling

@tannernelson mentioned a while ago that this package is only meant to be a simple Swift wrapper, but I'm wondering if you've had a change of heart.
It looks like thread safety has been implemented, so I think the next step would be to add connection pooling. We can reuse a single connection now but the server will usually drop it after a few hours, so we'll probably want to maintain a small pool of connections to recycle as they expire.
What do you guys think?

I wouldn't mind seeing this added. It could go on the Database class. The execution code moved to the Connection class which I think is a lot cleaner now. So users have the option to execute from Database in which case a new connection is created every time, or to create one connection and re-use that.
So if Database was able to automatically re-use a finite amount of connections to improve performance that would be a win win.

That sounds like a good idea. I'm no connection-pooling expert, but we'll need a way to catch dropped connections and recycle them somehow. There's a variety of reasons a request could fail, so we'll probably need to create a new connection and retry the request on the new one.

@collinhundley Why was this issue closed if there seems to be a desire for the feature?
Should a new issue be created proposing the feature before implementing?


Adds support to unpack 64bit floats

Permits to parse 64 bit floats received from Ardour DAW

thanks!

@morbith-dqtz this change is now released as 0.1.6 and should be available via pypi.

thanks for your work, time and diligence !


It is not a bug though a feature

what i am looking is that can't we decode only layout xml and encode it back

apk/res/layout

or any kind of feature which allows us to decode one xml and encode it back as well ?
the reason why i am saying is that because encoding resources back always have problems ,so having option to decode one xml file or files in specific directory and encoding it back ?

How do you decode one file? Any layout file probably has tons of references to styles, strings and more and you can't decode those styles, strings and more without resources.arsc decoding and at that point you've already decoded most of the application. The only remaining resource files would be the remaining layouts and images.
Thats the real reason, so not on my radar to support this enhancement.

this library works
https://github.com/wtchoi/axml
i can decode and encode on android itself with application
maybe you can try better if you wanna take a look

Thanks, but apktool is not engineered in a way to support this. Feel free to use that library though.


add license title

It's not strictly required, but it's useful metadata, and part of the recommended license template text:

http://choosealicense.com/licenses/isc/
https://opensource.org/licenses/isc-license
http://spdx.org/licenses/ISC.html#licenseText

@bryanhelmig I am ok to merge this in but would prefer Zapier weigh in on this text change. A thumbs up or down is sufficient.

👍 We should probably update that license to say 2016 and Zapier Inc. too!

Thanks for the quick review and merge :)


InitializeComponent is undifined/identifier not found

Hi all,
I'm developing a C++ application with WinUI 3/Project Reunion 0.5 . I cleaned and rebuilt my solution as an attempt to solve my an error (#5181) and now my program won't build at all stating InitializeComponent is undifined InitializeComponent identifier not found
What do I do now?

I had MainWindow.xaml and .h and .cpp and .idl in a separate Views folder to keep my code clean. Moving it to the root directory solved this issue. Weird 🤷♂


Redefinition of module 'CSr25519'

I am getting this error when i added the package

Hi. Cab you, please, provide more details (SPM or CococaPods, platform, etc.)

I don't know how many ways are there to install the sdk but when i tried for cocoa pods it showed me

And then i tried for swift package manager ....and now its showing the above error....

For CocoaPods use 0.0.4 as the version for now. I'm working on the library, will release 0.1 soon.

Is there any way ...i can use this library by doing changes to my project or something else?

@ypopovych Thanks So much i resolve it.
Just removed the version from Pod file and then pod install..Now working fine


Curve calibration: support for overriding the tenor associated with a node

By default the instrument implies the node date passed to the analytics during calibration (and the subsequent x value of that node on the calibrated curve), but it needs to be possible to override this regardless of the date implied by the instrument.
This is primarily to support node dates that align with ECB meeting dates. In this case, the node date would be defined as an index (after the current date) into a list of meeting dates.
We may also want to support overriding the node date to an alternative absolute or relative date as part of this.

The date can be overridden from code.


Updated content with reference content

Added content from tutorial steps.

Merged real content for testing updated home page


Add IBC module level errors

Description

part of: #2896
This PR duplicates the errors found here and uses the ibc as the module name. They will be used in a follow up PR.
Commit Message / Changelog Entry
N/A
see the guidelines for commit messages. (view raw markdown for examples)

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md).
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/).
[ ] Added relevant godoc comments.
[ ] Provide a commit message to be used for the changelog entry in the PR description for review.
[ ] Re-reviewed Files changed in the Github PR explorer.
[ ] Review Codecov Report in the comment section below once CI passes.

What do you think about assigning the ibc module name to a codespace constant and using that in the calls to Register?
const codespace = exported.ModuleName

What do you think about assigning the ibc module name to a codespace constant and using that in the calls to Register?
const codespace = exported.ModuleName

don't feel strongly either way! Happy to make this change if you think it makes things clearer


Makefile.pl

Could You prepare makefile.pl or some installation instructions?

Whilst waiting for instructions, if you are struggling to get it going...try extracting the zip to a folder, and then create a file like below and see if it runs......note, it's assuming there is a lib folder alongside it, which has the Binance folder inside that. Naturally you can move it where you want, but make sure the top line references any extra lib files.
You don't need a key to test the bit below.
#!/usr/bin/perl -I lib
use strict;
use warnings;
use Binance::API;
use Data::Dumper;
my $api = Binance::API->new(
apiKey    => 'my_api_key',
secretKey => 'my_secret_key',
);
my $ticker = $api->ticker( symbol => 'ETHBTC' );
warn Dumper $ticker;

Thanks for the interest. I've had this project on hold for a couple of months now. I'll provide installation instructions.
As you may have noticed, not all Binance API actions are implemented. Feel free to provide extensions to our current implementation.

I've already learned how to write Makefile.PL myself. I will immediately put a pull request with installation instructions.
@taskula are You sure that in query signature for "query" is generating fine? I tried add allOrders query but have problems:
1526651669821SUB: Binance::API::Request::_execSUB: Binance::API::Request::_exec{"code":-1102,"msg":"Mandatory parameter 'signature' was not sent, was empty/null, or malformed."}

I added to API.pm these lines:
sub allorders {
    my ($self, %params) = @_;
    unless ($params{'symbol'}) {
        $self->log->error('Parameter "symbol" required');
        Binance::Exception::Parameter::Required->throw(
            error => 'Parameter "symbol" required',
            parameters => ['symbol']
        );
    }
    my $query = {
        symbol    => $params{'symbol'},
        orderId => $params{'orderId'},
        limit => $params{'limit'},
    };
    return $self->ua->get('/api/v3/allOrders', { signed => 1, query => $query } );
}

I would love to help with the development of this project, but this problem limits me. Would you be so kind and directed me what should I do?
For GET server dont allow the use requestbody.

There seems to be a bug. Thanks for noticing, I'll fix it!

@xFFFFF
As of now I found at least two issues (fixed in 98e3a68401dfbf09388729cf4918fc3d0ec1e277):

I was using SHA512 instead of SHA256
Signature was not included in the GET request with only query parameters

I now get another error:
{"code":-1022,"msg":"Signature for this request is not valid."}
If you want to figure out what's wrong with the signature calculation, feel free :)


Plugin doesn't appear to work with Vue

Issue: This plugin doesn't appear to work when used alongside the Vue JavaScript library.
Here's a demo illustrating the problem: https://codepen.io/d-at-pm/pen/bGNqZzb
In this scenario the Bootstrap dependencies and the plugin are initialized before Vue is mounted into the DOM. Once a new file is selected the input label does not update with the new filename.
It is worth mentioning that if the plugin itself is initialized after Vue is mounted to the DOM then it does work.
This might not actually be an issue with this plugin itself and probably due to the way that Vue handles DOM nodes when initialized but I am curious if anyone else is aware of or has experienced this issue.

Hi @do-adams,
This plugin is written in VanillaJS so it should work everywhere.
Like you said:

It is worth mentioning that if the plugin itself is initialized after Vue is mounted to the DOM then it does work.

I think it's something related to how VueJS works but not related on how this plugin works.
I close this issue but feel free to discuss here that subject.

Thanks for taking the time to address the issue @Johann-S


import { Fragment, jsx, jsxs } from 'react/jsx-runtime' (all imports are undefined in Next.js app)?

How do I get this working (found this straight in the starry-night README):
import { Fragment, jsx, jsxs } from 'react/jsx-runtime'

For this line:
const reactNode = toJsxRuntime(tree, { Fragment, jsx, jsxs })

Getting this error for each of the imports:
Module '"react/jsx-runtime"' has no exported member 'Fragment'.

This is in a Next.js app, do I need to add something to my webpack config or next.config.js?
I am using React 18.2.0 on Next.js 14.1.2-canary.1 with these:
+ @wooorm/starry-night 3.2.0
+ hast-util-to-jsx-runtime 2.3.0

My full code is currently this:
import { common, createStarryNight } from '@wooorm/starry-night'
import { toJsxRuntime } from 'hast-util-to-jsx-runtime'
import { useEffect, useState } from 'react'
import { Fragment, jsx, jsxs } from 'react/jsx-runtime'

let starryNight: any

export default function Code({
  scope = 'source.ts',
  children,
}: {
  scope: string
  children: string
}) {
  const [sn, setStarryNight] = useState<any>(starryNight)
  const [code, setCode] = useState<React.ReactNode>(children)

  useEffect(() => {
    createStarryNight(common).then(sn => {
      starryNight = sn
      setStarryNight(sn)
    })
  }, [])

  useEffect(() => {
    if (!sn) {
      return
    }

    const tree = sn.highlight(children, scope)
    const reactNode = toJsxRuntime(tree, { Fragment, jsx, jsxs })
    setCode(reactNode)
  }, [sn, scope, children])

  //

  //
  return code
}

No clue. This project is tested and it works.
Your issue seems to be about Next.js, React’s types, or hast-util-to-jsx-runtime. Have you tried reducing your problem to an MVP, using non-alpha releases, checked without Next.js? Checked if it’s just a TS error (`@ts-expect-error) or an actual error? There’s nothing that needs to change here, so I’ll close it.


Rename variable/class names to improve readability

⚠  This branch includes features that will be introduced with #1549, don't merge before the respective branch landed.

Change all DSL variable and class names from attrib to attribute to improve readability.
See diff here
/cc @wavesoft

@orlandohohmeier could you rebase please?


Getting feature extent in Openlayers.

I believe this is possible with the mm boxselector.
But would it be possible to retrieve the clicked feature's shape or even better its extent?
The ideas is to enable a click/zoom to extent functionality in my service.
Thanks in advanced.

Okay, closing here; open a new ticket for specific concerns or open a ticket on the specific project.


VMDK plugin client is using wrong config file path

vmdk_plugin/utils/config/config.go is missing a dir in the config file path.
Fixing as below:
@@ -26,7 +26,7 @@ const (
// Default paths - used in log init in main() and test:
    // DefaultConfigPath is the default location of Log configuration file

  DefaultConfigPath = "/etc/docker-volume-vsphere.conf"

  DefaultConfigPath = "/etc/init/docker-volume-vsphere.conf"
  // DefaultLogPath is the default location of log (trace) file
  DefaultLogPath = "/var/log/docker-volume-vsphere.log"

Closing this, the DVV plugin config file used to manage the service has the same name as the config file (json formatted) used by the plugin for run time config. And that doesn't exist! Closing.


log: fix obscure crashes due to size_t promotion

It dawns on me now that I should've been hinting or forcing the type at
all of the LOGGER_LOG endpoints to ensure they match the va_arg side on
logger.c. This patch initially swaps size_t's for int's so at least it's
using less complicated promotion rules.
Fixes a crash on centos 7 when logging sets via watch mutations
TODO

[ ] audit all LOGGER_LOG callers and ensure the passed arguments match the va_arg side.

took a quickish look at all the va_arg calls and logger calls. they seem good enough, though if I ever make a serious pass at it I should line them up more carefully.


[ISSUE #1727] Fix sentinel plugin's fallbackUri in rule.

// Describe your PR here; eg. Fixes #issueNo

Make sure that:

[x] You have read the contribution guidelines.
[x] You submit test cases (unit or integration tests) that back your changes.
[x] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #1733 (edacfb3) into master (5fcf2b8) will decrease coverage by 0.40%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1733      +/-   ##
============================================
- Coverage     62.79%   62.39%   -0.41%
+ Complexity     2381     2372       -9
============================================
  Files           478      478
  Lines         10238    10238
  Branches       1036     1036
============================================
- Hits           6429     6388      -41
- Misses         3295     3337      +42
+ Partials        514      513       -1

Impacted Files
Coverage Δ

...enyu/client/tars/TarsServiceBeanPostProcessor.java
32.43% <0.00%> (-25.68%)
:arrow_down:

...henyu/plugin/grpc/resolver/ShenyuNameResolver.java
45.26% <0.00%> (-17.90%)
:arrow_down:

...henyu/plugin/httpclient/NettyHttpClientPlugin.java
57.14% <0.00%> (-2.05%)
:arrow_down:

...che/shenyu/sync/data/http/HttpSyncDataService.java
63.10% <0.00%> (-1.95%)
:arrow_down:

...elimiter/handler/RateLimiterPluginDataHandler.java
69.69% <0.00%> (-1.52%)
:arrow_down:

.../plugin/grpc/loadbalance/AbstractLoadBalancer.java
68.47% <0.00%> (-1.09%)
:arrow_down:

Continue to review full report at Codecov.

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

check the ci please

@SaberSola The ci failure is not related.


Save State Client Response

When running the save-state client command, the client always receives back an error, even when the snapshot is created successfully. Fix this so that the save-state command reports its success as well as a summary of its result

The problem is that the mina-indexer client expects a serialized version of the response but it's returning a string. I will close this issue when the following PR gets merged: https://github.com/Granola-Team/mina-indexer/pull/218

This issue disappear when we removed the save snapshot feature


XSS vulnerability in Clearing

Clearing uses jQuery's .html() method to display the caption, which introduces XSS vulnerability.
even my safely encoded data-caption="Cross Site Scripting Test"><script type="text/javascript" src="http://www.naked-design.at/js/xss-test.js"></script>" got executed.
i fixed this locally and i'll submit a pull request in a minute.

hey, this was my first fix on github! :-)
somehow the commit ended up in issue #6640, sorry!

Awesome! Thanks Keep em' coming :)


[TEVA 1135] Education phase mappings

This PR adds a new field to the organisations model that maps school phase to a readable school phase.
Jira ticket URL
https://dfedigital.atlassian.net/browse/TEVA-1135
Changes in this PR

Add mapped school phase
Update school update job
Index mapped school phase and update filters builder

Next steps

Maybe an array_enum would be useful here?

Maybe an array_enum would be useful here?

If you think a school can have more than one phase? I don't think that's possible.


Refactor additonal user details form

✨ Codu Pull Request 💻
Pull Request details
I am splitting the additional features form into multiple steps/separate forms.
In this PR I add logic for the first form  + server action for handling prisma update
https://youtu.be/EgmpBidVXb0

Summary by CodeRabbit

Refactor

Renamed and restructured form handling entities for clarity and efficiency.
Updated form submission logic for enhanced user interaction.

New Features

Introduced distinct validation rules for different sections of the user input form.

Bug Fixes

Improved error messaging for form submissions to enhance user experience.

I dont understand why the zod schema file (additionalUserDetails.ts) was included as a diff. I had pulled down the changes.
Should I create a new branch each time?


Add round, ceil, floor, truncate to the In-Database Column type

Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[x] The documentation has been updated, if necessary.
[x] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
[x] All code follows the
Scala,
Java,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
All code has been tested:

[x] Unit tests have been written where possible.
[x] If GUI codebase was changed, the GUI was tested when built using ./run ide build.

This implementation of round is a generic, "by hand" version using column arithmetic. This approach will only be used in a few cases; most of the time we can use the built-in ROUND() function in both backends.

@jdunkerley Works the same as the in-memory column in what way?

If I understand your question and James's intent correctly, I think it was a question/remark that technically we could implement the round operation in the in-memory backend analogously - using "primitive" column operations (which are themselves vectorized in Java). This may likely be much faster than the Enso-Java roundtrips of mapping the Enso-implemented round. (I guess we could even try to abstract this away into common logic, agnostic of DB vs in-memory backend - just a function that takes a column and transforms it to get the rounded value)
I like this suggestion because then we rely on a single implementation for Column.round (we still have separate one for primitive Number.round but that is rather hard to avoid). And likely it can give us better performance. I imagine a round implemented purely in Java may still be a bit faster, because using the primitive column ops will cause quite a bit of allocations of intermediate columns whose cost is probably not insignificant. Still it can be faster than current in memory solution - but we'd probably have to measure that to be sure - it can be an interesting experiment.

Implemented in-memory round with column arithmetic instead of Column_Ops.map_over_storage, results are good:
Before:
.round floats
Column.round floats/iteration:0: 451.38ms
Column.round floats/iteration:1: 412.05ms
Column.round floats/iteration:2: 410.55ms
Column.round floats/iteration:3: 413.93ms
Column.round floats/iteration:4: 411.51ms
Column.round floats/iteration:5: 413.17ms
Column.round floats/iteration:6: 279.48ms
Column.round floats/iteration:7: 203.94ms
Column.round floats/iteration:8: 201.2ms
Column.round floats/iteration:9: 204.96ms
Column.round floats average: 340.22ms
.round ints
Column.round ints/iteration:0: 1.31ms
Column.round ints/iteration:1: 0.49ms
Column.round ints/iteration:2: 0.25ms
Column.round ints/iteration:3: 0.03ms
Column.round ints/iteration:4: 0.04ms
Column.round ints/iteration:5: 0.03ms
Column.round ints/iteration:6: 0.04ms
Column.round ints/iteration:7: 0.03ms
Column.round ints/iteration:8: 0.01ms
Column.round ints/iteration:9: 0.01ms
Column.round ints average: 0.22ms

After:
.round floats
Column.round floats/iteration:0: 199.4ms
Column.round floats/iteration:1: 138.21ms
Column.round floats/iteration:2: 149.31ms
Column.round floats/iteration:3: 142.21ms
Column.round floats/iteration:4: 121.46ms
Column.round floats/iteration:5: 121.41ms
Column.round floats/iteration:6: 143.53ms
Column.round floats/iteration:7: 164.51ms
Column.round floats/iteration:8: 166.33ms
Column.round floats/iteration:9: 121.97ms
Column.round floats average: 146.83ms
.round ints
Column.round ints/iteration:0: 0.38ms
Column.round ints/iteration:1: 0.08ms
Column.round ints/iteration:2: 0.15ms
Column.round ints/iteration:3: 0.03ms
Column.round ints/iteration:4: 0.04ms
Column.round ints/iteration:5: 0.03ms
Column.round ints/iteration:6: 0.13ms
Column.round ints/iteration:7: 0.13ms
Column.round ints/iteration:8: 0.07ms
Column.round ints/iteration:9: 0.09ms
Column.round ints average: 0.11ms

@GregoryTravis what is the holdup here?

@xvcgreg Many many special cases and inconsistencies between different database backends.


[WIP] Attributed String

For a more centralized discussion
Todo:

[x] implement libui api
[x] update example

[ ] screenshot

[x] the forEach methods in OpenType and AttributedString
[x] overloading UiAttributedString.appendAttributed[1/2] - using nbind and js
[x] overload UiFontAttribute.newUnderlineColor
[ ] is the API naming okay? (Ui... - Text... - UiText...)
[ ] memory leaks (no new ones introduced?)
[ ] documentation
[ ] (implement test ?)

Implementation questions:

[ ] https://github.com/parro-it/libui-node/blob/c20e9c45c9520e5b5850c765a0f062d3a6043d56/src/ui-node.h#L668-L670
[ ] https://github.com/parro-it/libui-node/blob/c20e9c45c9520e5b5850c765a0f062d3a6043d56/src/Font/UiAttributedString.cc#L68-L69

Closes #70

Why doesn't this work?
C++:
static UiFontAttribute *UiFontAttribute::newColor(Color c);

JS:
UiFontAttribute.newColor(new libui.Color(0.75, 0.25, 0.5, 0.75))
                ^

Error: Type mismatch

🎉 🎉 🎉 🎉 🎉

There is a function to iterate over all attribues in a string with a callback. Is this correct?
// signature: for cb: func(const uiAttributedString *s, const uiAttribute *a, size_t start, size_t end, void *data)
void UiAttributedString::forEachAttribute(nbind::cbFunction& cb, void *data) {
	uiAttributedStringForEachAttribute(s, cb, data);
}

There is a function to iterate over all attribues in a string with a callback. Is this correct?
No, I can't recall where I already found this pattern, but what we need is declare a struct to hold Js custom data and Js callback:

struct cb_data {
     nbind::cbFunction& cb;     // this is the Js land callback fn
     void *data;                          // a pointer to data coming from Js land (probably unused, but just for completess)
}

Then, a function with the libui expected signature. This one receive as
data argument an instance of struct cb_data:
unsigned int c_cb(const uiAttributedString *str, const uiAttribute *attr, unsigned long a, unsigned long b, void *data) {
   struct cb_data *d = (struct cb_data *) data;
   d->cb(str, attr, a, b, d->data);   // I don't know if I'm calling it correctly ...
   delete d;
}

Then, in forEachAttribute, you create an instance of cb_data struct, fill it, and call uiAttributedStringForEachAttribute with that:
void UiAttributedString::forEachAttribute(nbind::cbFunction& cb, void *data) {
   struct cb_data *d = new struct cb_data {cb, data};

   uiAttributedStringForEachAttribute(s, c_cb, d);
}

I hope this make sense to you... I didn't try to compile this....

Yes, for sure. Sorry

So that works, but calling in JS:
otf.forEach((feat, str, val)=>{
	console.log(feat, str, val);
});
> Error: Wrong number of arguments, expected 2

otf.forEach((feat, str, val)=>{
	console.log(feat, str, val);
}, null);
> Error: TypeError: Type mismatch

If you please commit you last changes I'll pull them and check the error

I left the failing lines in the text examples.
If you want to run it, you need to use the current library and header file.

If you want to run it, you need to use the current library and header file.

Meanwhile, I'm preparing a new release in my fork with precompiled binaries for that

Meanwhile, I'm preparing a new release in my fork with precompiled binaries for that.

We might want to wait for https://github.com/andlabs/libui/issues/320.

We might want to wait for andlabs/libui#320.

We can choose to use the new published libui release of libui only in this branch by changing libui property inapckage json only here.

I left the failing lines in the text examples.

Could you point exactly where? I can't find it ☺

It's working 🎉

BTW, it could be useful to override toString method in attribute classes, in order to print the attr name instead of that UiAttributedString {} stuff... do you think it could be useful?

overloading UiAttributedString.appendAttributed[2] - using js or nbind ?

I would go with nbind - it's a new feature I never used and could be useful to check it.

BTW, it could be useful to override toString method in attribute classes, in order to print the attr name instead of that UiAttributedString {} stuff... do you think it could be useful?

It would be nice, but I currently don't even see a usecase for the forEach method.

Me neither, but it's not specific to the forEach method. Anyway, I see you already override it, but seems to have no effect on the console log output

At runtime, I got segfault and errors regarding the number of arguments.

I saw the commented lines. Maybe we found a nbind bug

Me neither, but it's not specific to the forEach method. Anyway, I see you already override it, but seems to have no effect on the console log output

Uhm, no actually you implement it on UiAttributedString, not on UiFontAttribute

BTW, it could be useful to override toString method in attribute classes

Uhm, on a second thought, I think it could be not so easy, we could eventually add this later  on...

Linux and macOS versions are building well on CI (there's only XO complaining on the example file).
But it seems Windows has some problem compiling: https://ci.appveyor.com/project/parro-it/libui-node/build/1.0.193/job/0df2c23q4ypo6q20
uiDrawTextLayoutParams params = {
		.String = str->getHandle(),
		.DefaultFont = defaultFont->getHandle(),
		.Width = width,
		.Align = (uiDrawTextAlign) align
	};

It doesn't like the named struct constructor. Do you know what standard that syntax belong to?

"Designated Initializers"
But this should work as well:
uiDrawTextLayoutParams params = {
		str->getHandle(),
		defaultFont->getHandle(),
		width,
		(uiDrawTextAlign) align
	};

But this should work as well:

Thank you, changed.

memory leaks

We'll have to check for this on the whole implemenation. If you try the check-memory.js example, you'll see that every time you open that big window (and that's strangely slow BTW) some memory is leaked (around 3mb on my machine). At the time I was thinking it depends by libui not correctly freeing child controls on window closing, but I was not sure.
Moreover, I fear some memory is leaked every time we return a char * from a libui method. libui philosophy is that we are responsible to free the string, but how I don't think nbind free it (how it can take this responsibility?). And once we return it, we are gone with that.

(and that's strangely slow BTW)

Well, with 300 elements...

Moreover, I fear some memory is leaked every time we return a char * from a libui method.

Not sure where to put this, but this doesn't look like it's getting freed:
https://github.com/parro-it/libui-node/blob/36c278f9453d9ec9750bb91cc1b5fb1d400877ff/src/UiArea/UiAreaKeyEvent.cc#L13-L18

Yes that's exactly the kind of situation I was speaking about: we cannot obviously free the string before return it, and once returned, we lost control.
I thought of some possible solutions:

Nbind could take care of freeing the string, just after creating the JS String.
We can keep a global linked list of such pointers, and free them all later, maybe in next tick.

Same situation arise practically everywhere we have a method returning a string, only it's libui that is doing allocation.
See e.g. uiEntryText method here: https://github.com/andlabs/libui/blob/master/unix/entry.c#L28
(by the way, I don't understand why andlabs is doing a copy of the string there, seems an expensive operation, especially when entries contains lot of text)

Nbind could take care of freeing the string, just after creating the JS String.

How?
A workaround would be writing JS wrappers for these problematic functions and freeing the C++ string there.

How?

By signaling with a flag or similar that returned pointer should be freed, so that it can take the responsibility...

A workaround would be writing JS wrappers for these problematic functions and freeing the C++ string there.

Sure, that will solve it, but I think there are many of those... lot of work!

Can we return a std::string directly (not a pointer?). With C++, this would enable Return value optimization (basically returning an object directly without a pointer).

Yes, I think this is a great idea, we should really give it a try!

The creator functions in UiFontAttribute are leaking memory.
Tested with:
const interval = setInterval(() => {
	for (let i = 0; i < 1000; i++) {
		const attr = libui.UiFontAttribute.newStretch(2);
		attr.free();
	}
	procStats.stats((err, result) => {
		if (err) {
			// label.text = err.message;
		} else {
			const text = `
Memory: ${humanize.filesize(result.memory)}
Heap: ${humanize.filesize(result.memoryInfo.heapUsed)}`;
			console.log(text);
		}
	});
	global.gc();
}, 10);

Somehow, attr.free(); doesn't always actually invoke that function (usually only every other attribute).
Tested with:
UiFontAttribute::~UiFontAttribute(){
	if(a != nullptr){
		printf("NOT FREED!\n");
	}
}

void UiFontAttribute::free(){
	if(!appended){
		uiFreeAttribute(a);
		a = nullptr;
		printf("freed! %p\n", a);
	}
}

It seems the destructor is called before the free call?

As you say, sometime the free method is not called at all!
But either when it's called and when it's not, the method leak anyway

Guess what? I renamed the method to freeIt, and now it's called all times!
Even more strange, the attr object now have both a free method and a freeIt one!
I don't know from where this additional free method can come from.

Very strange.
Methods of UiArea:
[ 'getKey',
  'getExtKey',
  'getModifier',
  'getModifiers',
  'getUp',
  'free' ]

Methods of UiKeyEvent:
[ 'destroy',
  'setParent',
  'toplevel',
  'getVisible',
  'setVisible',
  'getEnabled',
  'setEnabled',
  'setSize',
  'queueRedrawAll',
  'scrollTo',
  'free' ]

I didn't check, but I guess they have the same strange behavior. Maybe a bug of nbind?

Well, yes!
Add free() method to call destructor manually from JavaScript. Fixes #9 (nbind commit f15ef)

Good catch, I opened an issue here: https://github.com/parro-it/libui-node/issues/79

I wrote this little C test:
#include <stdio.h>
#include "../../ui.h"

int main(void)
{
	uiInitOptions o;
	const char *err;

	memset(&o, 0, sizeof (uiInitOptions));
	err = uiInit(&o);
	if (err != NULL) {
		fprintf(stderr, "error initializing ui: %s\n", err);
		uiFreeInitError(err);
		return 1;
	}

	while(1){
		uiMultilineEntry *e = uiNewMultilineEntry();
		uiControlDestroy(uiControl(e));
	}

	uiUninit();
	return 0;
}

It lacks memory too, so seems better to investigate on libui first.

But this one works fine:
int main(void)
{
	uiInitOptions o;
	const char *err;

	memset(&o, 0, sizeof (uiInitOptions));
	err = uiInit(&o);
	if (err != NULL) {
		fprintf(stderr, "error initializing ui: %s\n", err);
		uiFreeInitError(err);
		return 1;
	}

	uiMainSteps();
	unsigned int i = 0;
	while(1){
		uiButton *e = uiNewButton("");
		uiControlDestroy(uiControl(e));
		uiMainStep(0);
		usleep(1000 * 50);
		printf("%d\n", i++);
	}

	uiUninit();
	return 0;
}

It happens that cocoa free controls memory only if the GUI event loop is running.

Interesting, same goes for attributes:

(implement test ?)

Actually, the tests just call class property setters with some value and check that getters return the same value. Not really useful tests, but better than nothing... you could the same with these new classes.

 // TODO needs to actually be of that type - maybe implement check and return -1 if wrong (or throw js error)
 // It is an error to call this on a uiAttribute that does not hold a ...
 const char *getFamily();

Why it can't be changed to a std::string as we do with other char*?

// TODO how this (and strlen) work with unicode?
 size_t end = start + strlen(str);

It depend on what this->setAttribute(attr, start, end); expect as argument: byte offset or unicode codepoint offset. Anyway, this seem to conflict with how you calculate start. I thin you start holding codepoint position , and end adding that to a byte length.

Why it can't be changed to a std::string as we do with other char*?

We can, but I don't think it would make a difference: the string gets freed by uiFreeAttribute and isn't duplicated when calling getFamily():
// uiAttributeFamily() returns the font family stored in a. The
// returned string is owned by a.

I tried changing free to the destructor in FontAttribute.
 :

That make sense, JS garbage collector cannot know we are keeping a reference to the libui fontattribute pointer on C side, so it free the result of ontAttribute.newFamily('Courier New').
This should work:
const attr = FontAttribute.newFamily('Courier New');
str.appendAttributed('font family', attr);

I don't know how can we solve this problem...

So it's correct that that object is garbage collected before the function returns?

So it's correct that that object is garbage collected before the function returns?

Uhm, no, actually it's not. Maybe nbind does not increase ref count on argument. I'm a bit confused at the moment , I should rest..

It depend on what this->setAttribute(attr, start, end); expect as argument: byte offset or unicode codepoint offset. Anyway, this seem to conflict with how you calculate start. I thin you start holding codepoint position , and end adding that to a byte length.

// uiAttributedStringLength() returns the number of UTF-8 bytes in
// the textual content

So both start and end are in actual bytes.

But we should really check for this... MAybe add some nice emoji to the example?

👏🏼👏🏼 Awesome!

The issue with the FontAttribute destructor was that the object was copied in FontAttribute.new.... and multiple wrapper destructors ran on the same underlying attribute. Move constructor solved it.

Good work! Once discovered, it seems clear...
I noticed you return FontAttribute and not a pointer to it.
I don't understand how nbind could handle that. It's a new feature in nbind?

I noticed you return FontAttribute and not a pointer to it.
I don't understand how nbind could handle that. It's a new feature in nbind?

I'm not sure I understand what you mean.
This is basically the same technique as returning std::string.

This is basically the same technique as returning std::string.

Yes, on C++ side it's clear how that work, but I think nbind know specifically how to handle a std::string (it convert it to a JS String).
But it cannot know about FontAttribute I guess.
Anyway, it works, so I'm obviously wrong...

If you look at Color class, Size and others, they have a method specifically used by nbind to convert an instance of the C++ class into a JS object.

I don't know why it works. 😕 😆

I don't know why it works.

I think I've discovered why. From nbind readme:

Note: currently passing objects by pointer on Node.js requires the class to have a "copy constructor" initializing itself from a pointer. This will probably be fixed later.

That was fixed eventually (because all UiControl descendants does not have a copy constructor), but the old behaviour is still working and it happens on FontAttribute you has a copy constructor.
@jjrv, if I'm right, I could write a PR to fix the documentation.

passing objects by pointer

I'm returning objects, not pointers:
https://github.com/parro-it/libui-node/blob/e7ec52e7cf93e74beba1ffd279555b82a1010038/src/Font/FontAttribute.cc#L94-L96

it happens on FontAttribute you has a copy constructor.

This is a move constructor: (a copy constructor has one ampersand)
https://github.com/parro-it/libui-node/blob/e7ec52e7cf93e74beba1ffd279555b82a1010038/src/Font/FontAttribute.cc#L6

I'm returning objects, not pointers:

Ah, yes you're right 😕

This is a move constructor: (a copy constructor has one ampersand)

Ah, yes you're right again  😕

Please don't ask me what the difference between a move and copy constructor is 😕

😆  we both should return to school 😆😆
Maybe I can ask an expert: /cc @roncapat

Did you add

implement libui api

?

From the list of changes, it seems... but I never updated that comment.

I saw you back-merged 0_2_0 changes, is this ready to be merged, or you plan to add some tests for the new feature?

I won't have much time in the near future and most of the functions are "tested" in some way in the example (e.g. getter and setter). So I guess it's fine for now....

(I have pushed it; you'll either need to cherrypick it out of common-namespace-cleanup or preferably just wait for me to merge it probably later tonight =P )

Also wait, did you figure out how to use the foreach function pointer stuff? Or do you need more explanations? Because I can explain it.

Thank you, it's working now:
https://github.com/parro-it/libui-node/blob/cfec5e9e2c4836d95f13bd0c4a6fabc5a43921e7/src/Font/AttributedString.cc#L47-L68

Yep, you see what the void * stuff is for now. In that case, two things: 1) make sure FontAttribute((uiAttribute *)a) doesn't destroy a itself, since you don't own it, and 2) the return value from the C callback is really something of the uiForEach type, so make sure your code is following that, even if you are implying a return uiForEachContinue from the JavaScript code if that callback doesn't explicitly return a value.

make sure FontAttribute((uiAttribute *)a) doesn't destroy a itself when it gets destroyed, since you don't own a and it's not a copy

:+1: That's what setAppended does

the return value from the C callback is really something of the uiForEach type,

I'll do that

Well you aren't using f in the actual call to the JavaScript code, but instead are making an all new FontAttribute there, so you might want to fix that ;)

I'm trying to publish an updated release of libui, but found some problem compiling on Windows: details here: https://github.com/andlabs/libui/pull/157

@mischnic I published a new libui release, and bumped version here.
I'm going to merge this, do you feels there's some other thing to be done before?

Go ahead!


[SuperEditor][SuperReader] - Lock scrolling when content fits within available space (Resolves #1409)

[SuperEditor][SuperReader] - Lock scrolling when content fits within available space (Resolves #1409)
Currently in the SuperEditor or SuperReader, the document is scrollable even if the content fits within the available space as seen in the below demo.
This demo wraps the SuperEditor within an IntrinsicHeight widget to demonstrate the issue,
https://github.com/superlistapp/super_editor/assets/7259036/7965cec1-8b61-4af9-a7c4-13b3f2442ef6
This PR tries to address this issue by locking the scrolling for the document in such cases. We lock the scrolling when the content of the document fits within the available viewport space and there isn't any unconsumed content within the Scrollable.
Scroll lock is introduced within both DocumentScroller and AutoScrollController.  When the scroll lock is enabled, any calls/actions related to scrolling are ignored to prevent the document from scrolling.
After this latest updates, the scrolling is locked whenever content fits within available space as seen in the following demo,
https://github.com/superlistapp/super_editor/assets/65209850/a5c83f3b-a2db-4c0e-b96f-0d5288ca27a6

@matthew-carroll Could you please take a look at the approach to fix this issue? It's still WIP but wanted your opinion on the approach and see if whether this is the direction that we want to move in with this issue.

@rutvik110 before evaluating your choice of solution, can you describe what you investigated? For example, why is the scrollable letting us scroll when there's no content to scroll?

@rutvik110 before evaluating your choice of solution, can you describe what you investigated? For example, why is the scrollable letting us scroll when there's no content to scroll?

There seems to be two reasons here that are causing the overscroll.
The first reason is related to our handling of the scrolling within Scrollable. In case of auto-scrolling in response to mouse events, currently we are always trying to scroll irrespective of whether there's any content left to scroll.
This calls to scroll is leading to the overscroll/scroll-bounce that we are noticing in demo you shared.
Second reason is bit more ambiguous and it may be totally different issue that I may've confused with this one but here it's. It seems to be particularly related to usage of IntrinisicHeight. I noticed that when we have large chunks of paragraphs within SuperEditor, the final height calculated by the IntrinisicHeight to layout the SuperEditor isn't correct and is smaller by some magnitude. So the SuperEditor is laid out with the height smaller than it's contents actual height which results in introducing some content within SuperEditor beyond its scrollable viewport.
https://github.com/superlistapp/super_editor/assets/65209850/630ebbd6-bb09-4fb5-a776-a3ae818fa49c

Also, what about when the user is using the scroll wheel? I was able to overscroll using the scroll wheel which I believe is unrelated to going ballistic.

ah yah, had to add the same max extent check to jumpBy which resolved this.  @matthew-carroll Could you please re-check this on your end?
Also, what are your thoughts on the previous comment I made regarding extent check in the reverse direction?

I think the current changes are probably good as-is. Can you confirm that these changes work for SuperReader as well as SuperEditor? Also, can you write a test for each of these scenarios in the appropriate test file?
And please don't forget to revert the code that I mentioned in an earlier comment.

Great! Will check on SuperReader once which I believe should work fine as well and get onto writing tests for this.


Consider checking for and using system getent before using custom getent

Hi, I was testing this script in a shared server setting where we use freeipa for logins and encountered the error:
x11docker ERROR: Could not find user '...' in /etc/passwd.
Since we use freeipa, our users aren't stored in the /etc/passwd file. The system getent command is capable of working with this, but the custom getent command used isn't. I realize this was implemented to address #477 where getent wasn't implemented for some systems/shells.
I think some way of testing for and using the system getent first would suffice. I haven't tested thoroughly, but I made the following quick change below to get it working for me.
mygetent() {
  command -v getent >/dev/null 2>&1 && getent $1 $2 || \
  grep "^${2:-NOENTRY}:" < "/etc/${1:-NOFILESPECIFIED}"
}

Thank you very much!
You suggested code is implemented in the latest master version.


[Program]: Write a Dart program to print inverted pyramid pattern

Description
Write a Dart program to print inverted pyramid pattern
Input   : 5

Output :
1 2 3 4 5
1 2 3 4
1 2 3
1 2
1

Contributed by - @PraaneshSelvaraj

How to contribute

Save the solution in program/program/print-inverted-pyramid-pattern/print_inverted_pyramid_pattern.dart file

!assign


bttester: add another characteristic requiring authentication

This characteristic is required by GATT/SR/GAR/BI-21-C
(it requires two characteristics requiring authentication for
reading)

As of PTS 8.0.2 this is not required


Drag handle won't work on appended blocks?

Hi,
Following the drag handle example http://vnjs.net/www/project/freewall/example/draggable.html
If a block is appended using the appendBlock method and the added html has a data-handle attribute the entire block will listen for the drag event and act accordingly, as though no handle exists.
Any ideas?
Thank you!

Hi,
when you append new blocks, you also set this property:
data-draggable=true
so the new blocks can be draggable
Best

Thank you for the quick reply!
That did not appear to work...but I don't think I explained the issue correctly =).  The new blocks were always draggable.  I want to limit drag listeners to the handle section for appended blocks.  Currently, appended blocks will drag anywhere when clicked, even within the designated handle area.
ex: If I pre-populate the wall as per http://vnjs.net/www/project/freewall/example/draggable.html and then instantiate the plugin, the handle area works and limits the drag event listeners to that section of the block div.handle, just like in your example.  However, appended blocks thereafter seem to ignore the drag item handle and behaves as though no handle exists.  Non handle sections are able to drag the block.
Thanks for your time!

Hi again,
Discovered the problem.  It's on my side.  The object was being rendered too late and your awesome plugin couldn't identify the drag element correctly.
Thanks again!


Any updates to WIP branch?

Any news about the WIP or any other issues?
Can we merge WIP into main?
Also are you planning to send a PR to barnabycolby ?

@E3V3A thanks for the nudge. No recent changes to the WIP branch.  I've sent a message to @barnabycolby before but did not receive a response.  I'll go ahead and send a PR.


Activation of Snaps causes unsmoothed visual jitter

With the new approach, not using TopLevel to solve issues in Ysort, I noticed that smoothing doesn't seem to work when snap options are active, such as Snap vertices and transformations. Previously, with Top Level, the jitter was resolved. Now, it seems that it's no longer rounding the sprite's position and causing visual jitter. I tested this on version 4.x. ~It seems that without top level, there might be some limitation, perhaps?

I will detail below the project settings and provide a demonstration video:

https://github.com/lawnjelly/smoothing-addon/assets/81387545/91f635f9-bb6b-454c-a376-73c6c8756612

Ah yes this is interesting. I'm not super familiar with the snapping in 4.x, but it could be the snapping is having trouble coping with the math going on when you have a smoothing node as a child of the node you are moving.
There are still some issues afaik with the core snapping in 4.x but it will be good if you can work around these.
So two options:

I can easily add a toggle to the smoothing node where it can do the old behaviour (of toplevel). The only downside is that in this mode you will have the problems with y sort and visibility back.
I suspect that the snapping might work better if you don't make the smoothing node a direct child of the target, i.e. put it on a separate branch, as was recommended when we first introduced the addon. (This might be worth trying to see if it cures the problem, even if less convenient).

If you can create a small minimum reproduction project I might be able to pin down better what is going on (maybe we won't need these workarounds).

Ah yes this is interesting. I'm not super familiar with the snapping in 4.x, but it could be the snapping is having trouble coping with the math going on when you have a smoothing node as a child of the node you are moving.
There are still some issues afaik with the core snapping in 4.x but it will be good if you can work around these.
So two options:

I can easily add a toggle to the smoothing node where it can do the old behaviour (of toplevel). The only downside is that in this mode you will have the problems with y sort and visibility back.
I suspect that the snapping might work better if you don't make the smoothing node a direct child of the target, i.e. put it on a separate branch, as was recommended when we first introduced the addon. (This might be worth trying to see if it cures the problem, even if less convenient).

If you can create a small minimum reproduction project I might be able to pin down better what is going on (maybe we won't need these workarounds).

Here's a little demo! You can run the main scene "mapManager", if you want to change the player's node hierarchy it is in "visual.tscn"
Project.zip
I tested it on a different branch and I didn't notice any difference

And it can come with a small problem, when you change the hierarchy you need to indicate the animation node again to the code. Just access "heroi.tscn" and indicate animationtree to it. a short video to help. Any feedback will be welcome. Thanks 🙂😂
https://github.com/lawnjelly/smoothing-addon/assets/81387545/24ac7ca1-03f2-4fd6-9b22-94b88e441398

Testing the MRP with and without toplevel set, it seems there is a subtle difference, the parts of the model seem to be snapped as one with toplevel set, and without it they are snapped individually, which leads to a strange blancmange effect where the body parts jitter amongst themselves.
The problematic setting is snap_2d_vertices_to_pixel, which is GPU snapping. This is something I'll have to deal with eventually with 4.x doing core physics interpolation. Luckily core interpolation is able to do a much more sensible path.
The issue in the scene tree without using toplevel, is that the global transforms of the parent node are inherited by the child, and thus the global transform of the smoothing node must be done in relation to the parent, rather than in global space (which is more numerically stable). Core interpolation introduces a new mode for canvas items where they can be used in global space.
However for the smoothing addon, which will be used prior to this new mode, global transform of smoothing nodes (without toplevel) is done by first undoing the global transform of the parent. This means the local transform of the smoothing node is quite variable, and also subject to float error. This may not be seen with regular use, but can upset the snapping as snapping has distinct floating point borders.
So for now I'll probably either recommend not to use GPU snapping in this scenario, or to set a toplevel bool on the smoothing node. I'll add this now, it should be pretty simple, and upload a new version to the repository.

And it can come with a small problem, when you change the hierarchy you need to indicate the animation node again to the code. Just access "heroi.tscn" and indicate animationtree to it. a short video to help. Any feedback will be welcome.

Does this occur without the smoothing addon? Maybe this is a general engine bug?

Let me know if the new toplevel mode in 1.2.1 fixes this for you.

And it can come with a small problem, when you change the hierarchy you need to indicate the animation node again to the code. Just access "heroi.tscn" and indicate animationtree to it. a short video to help. Any feedback will be welcome.

Does this occur without the smoothing addon? Maybe this is a general engine bug?

This is expected behavior. I indicate where the animation nodes are when the hierarchy is changed the position changes a little. I think it's more the code. It's not a problem;
Yes, with Toplevel it now solves it completely... but then I return to the scenario of being left without y-sort. Maybe I should use a custom Ysort? like using Z index for now or something like that. , if I deactivate "snap vertices" and transform there is this problem of jitter in the fitting of the body part. So, the ideal would be to use the addon with TopLevel

Yes y-sort with toplevel is kind of tricky, because toplevel basically breaks the scene tree in the VisualServer, and attaches the node to the root instead of the parent. It seems designed for modal dialog boxes and tooltips, rather than this scenario.
What we really want, is a way of parenting a CanvasItem such that it ignores the parent transform, and applies the local transform in global space. That way, y-sorting and visibility would "just work" :tm: .
This is actually what I had to implement in core for core 2D interpolation. So it is chicken egg problem, the problem will be solved, when 4.x has core interpolation. But by that time, you will no longer need the addon. :slightly_smiling_face:
This does suggest maybe I'm better off spending the time implementing it properly by pushing ahead with the core physics interpolation for 4.x, rather than spending too long hacking the addon.

Yes y-sort with toplevel is kind of tricky, because toplevel basically breaks the scene tree in the VisualServer, and attaches the node to the root instead of the parent. It seems designed for modal dialog boxes and tooltips, rather than this scenario.
What we really want, is a way of parenting a CanvasItem such that it ignores the parent transform, and applies the local transform in global space. That way, y-sorting and visibility would "just work" ™ .
This is actually what I had to implement in core for core 2D interpolation. So it is chicken egg problem, the problem will be solved, when 4.x has core interpolation. But by that time, you will no longer need the addon. 🙂
This does suggest maybe I'm better off spending the time implementing it properly by pushing ahead with the core physics interpolation for 4.x, rather than spending too long hacking the addon.

if I deactivate "snap vertices" and transform there is this problem of jitter in the fitting of the body part.

Note that snapping vertices and transform is completely different in 4.x. Transform is done on CPU, vertices is done on GPU. When I tested it, snapping vertices caused problems, but not snapping transform.

Yes. In fact, with the toplevel approach, I don't even need to enable "snap vertices", toplevel already seems to resolve the displaced pixels. Just like you mentioned, treating them as "one", not separately. I'm already happy with at least understanding the root of the problem. Now it's time to wait until this is supported correctly as you mentioned. Thanks for all the support! I'll keep an eye on this in the future ^^
Vertices off
Toplevel on:

Vertices off
Toplevel off:


NukeOps start with either station codes or their codes

About the PR

the paper that nukies get at the start now contains the codes for ALL nukes, not just their own.
have fun raiding the vault at your leisure.
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

add: Nuclear Operatives now can start with either the codes to their own nuke or the codes to the station's nuke.

i'm not really a fan of this one, part of the challenge of nuclear operatives imo is also guarding your own nuclear bomb
however if this is still a popular option, maybe an alternate thing could occur where you sacrifice more telecrystals to get all nuke codes, or at least the nuke codes on the target station?

This should be for a separate mode or alternative nuke ops mode. Regular nuke ops or war ops should be about protecting the nuke and getting out. Lone op should be more focused on getting the station nuke set off. Maybe the alternative mode could be something like clown ops where it's a rare spawn, but it gives them all codes?

make game randomly decide which code nukies spawn with

with this new behavior, it picks between giving the ops the station codes or the mobile nuke codes.

ok merge now

ok merge now

please no its the only open pr i have left with this gone i'll be nothing

please merge it i need it for my loneops pr

@EmoGarbage404 merge master, mostly so tests work.

done


sava qrcode

hello dear, how to save the qrcode as a image

this module is only used to display a qrocde.


add swift maker

There is also https://github.com/neomake/neomake/pull/392.
Does it make sense to enable both at once?

I bet it does. Linters and compilers like peanut butter and chocolate... I'll try it out and see if it rocks.

It maybe nice to use SwiftPM if the directory contains a Package.swift file.

There we go, thanks!

I've created a PR for SwiftPM support https://github.com/neomake/neomake/pull/1890


Re-deploy the foundation's DNS seeders for NU5 testnet

See #1841 for details.
Tasks
After we merge #2796 and #2808, we need to:

[x] Deploy a new DNS seeder instance for NU5 testnet (170014), but don't switch the Foundation's DNS records yet

failed deployment https://github.com/ZcashFoundation/coredns-zcash/actions/runs/1267893660
fix PRs:

https://github.com/ZcashFoundation/coredns-zcash/pull/9
https://github.com/ZcashFoundation/coredns-zcash/pull/10

successful deployment: https://github.com/ZcashFoundation/coredns-zcash/runs/3703970205

[x] Deploy a new DNS seeder instance for NU5 testnet (170015), but don't switch the Foundation's DNS records yet

https://github.com/ZcashFoundation/zebra/issues/2808
https://github.com/ZcashFoundation/dnsseeder/pull/13
https://github.com/ZcashFoundation/coredns-zcash/actions/runs/1288695743

[x] Check that the testnet seeder is returning new peers
[x] Check that the mainnet seeder continues to return current peers
[x] Switch the Foundation's DNS records to point to the new instance

Deployed new seeder:
https://github.com/ZcashFoundation/coredns-zcash/actions/runs/1271133091

Here are the default mainnet and testnet DNS seeders:
https://github.com/ZcashFoundation/zebra/blob/b42ab67a4bad6383c5236aac9d116c3f6a65b569/zebra-network/src/config.rs#L161-L176

Currently the testnet seeders are returning:

ECC - 3 nodes
ZFND - 5 nodes
YOLO - 9 nodes

There are some common node addresses across the seeders.

Currently the mainnet seeders are returning:

ECC - 25 nodes
ZFND - 21 nodes
new ZFND - 16 nodes
YOLO - 25 nodes

Given the number of nodes on mainnet, we don't expect common node addresses across the seeders.

The ZFND mainnet seeder is now returning up-to-date peers, except for a few outdated peers, due to a known bug:

https://github.com/ZcashFoundation/dnsseeder/issues/10

The ZFND testnet seeder is now returning up-to-date peers:


Should mask zero inundation depths in ensemble generation

Right now the user needs to know to do this. Treating this as a bug since it will likely end up as one for everyone but me. I'll do this as soon as I can

Default behavior should be to mask and send a Warning to the user so they know to change the argument if that is undesired

This is only the case if you don't use the get_inundations function. The best way to address this is to make that function more generalizable. It's too specific to scenarios where your depth grids are based on design events


feat: adds guards for when there is no schools within a trust or LA

Context
AB#239062 AB#239063
Change proposed in this pull request
Adds guards to prevent render of front-end component when there is no schools within an LA or Trust.
Front-end is currently erroring (silently, but can be seen in browser console).
Back-end is logging errors due to URN collection being empty.
Guidance to review
N/A
Checklist (add/remove as appropriate)

[x] Work items have been linked (use AB#)
[x] Your code builds clean without any errors or warnings
[x] You have run all unit/integration tests and they pass
[x] Your branch has been rebased onto main
[x] You have tested by running locally

✅ Validated fix locally with LA 928 and Trust 05051218, which both currently fail in prod.


jfrog-cli: 2.50.4 -> 2.52.8

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/jfrog-cli/versions.
meta.description for jfrog-cli is: Client for accessing to JFrog's Artifactory and Mission Control through their respective REST APIs
meta.homepage for jfrog-cli is: https://github.com/jfrog/jfrog-cli
meta.changelog for jfrog-cli is: https://github.com/jfrog/jfrog-cli/releases/tag/v2.52.2
Updates performed

Golang update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.52.2 with grep in /nix/store/2qp8z5v1qvffnzprsgynhihmfm9avqib-jfrog-cli-2.52.2
found 2.52.2 in filename of file in /nix/store/2qp8z5v1qvffnzprsgynhihmfm9avqib-jfrog-cli-2.52.2

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
jfrog-cli

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/2qp8z5v1qvffnzprsgynhihmfm9avqib-jfrog-cli-2.52.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A jfrog-cli https://github.com/r-ryantm/nixpkgs/archive/0fd41af20c0cbb66044001d158c6f0e513aa553b.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/0fd41af20c0cbb66044001d158c6f0e513aa553b#jfrog-cli

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/2qp8z5v1qvffnzprsgynhihmfm9avqib-jfrog-cli-2.52.2
ls -la /nix/store/2qp8z5v1qvffnzprsgynhihmfm9avqib-jfrog-cli-2.52.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

jfrog-cli

Maintainer pings
cc @Detegr for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/jfrog-cli/versions.
meta.description for jfrog-cli is: Client for accessing to JFrog's Artifactory and Mission Control through their respective REST APIs
meta.homepage for jfrog-cli is: https://github.com/jfrog/jfrog-cli
meta.changelog for jfrog-cli is: https://github.com/jfrog/jfrog-cli/releases/tag/v2.52.6
Updates performed

Golang update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.52.6 with grep in /nix/store/l76x0zkznczp91ff2nfs3zlw70pqajmx-jfrog-cli-2.52.6
found 2.52.6 in filename of file in /nix/store/l76x0zkznczp91ff2nfs3zlw70pqajmx-jfrog-cli-2.52.6

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
jfrog-cli

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/l76x0zkznczp91ff2nfs3zlw70pqajmx-jfrog-cli-2.52.6 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A jfrog-cli https://github.com/r-ryantm/nixpkgs/archive/0c758f6e2a16d1fbcdae5d659ffa318393e10302.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/0c758f6e2a16d1fbcdae5d659ffa318393e10302#jfrog-cli

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/l76x0zkznczp91ff2nfs3zlw70pqajmx-jfrog-cli-2.52.6
ls -la /nix/store/l76x0zkznczp91ff2nfs3zlw70pqajmx-jfrog-cli-2.52.6/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

jfrog-cli

Maintainer pings
cc @Detegr for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/jfrog-cli/versions.
meta.description for jfrog-cli is: Client for accessing to JFrog's Artifactory and Mission Control through their respective REST APIs
meta.homepage for jfrog-cli is: https://github.com/jfrog/jfrog-cli
meta.changelog for jfrog-cli is: https://github.com/jfrog/jfrog-cli/releases/tag/v2.52.8
Updates performed

Golang update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.52.8 with grep in /nix/store/9mvy5kkzfnrnj2n3xgjv3a9hpsx274pr-jfrog-cli-2.52.8
found 2.52.8 in filename of file in /nix/store/9mvy5kkzfnrnj2n3xgjv3a9hpsx274pr-jfrog-cli-2.52.8

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
jfrog-cli

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/9mvy5kkzfnrnj2n3xgjv3a9hpsx274pr-jfrog-cli-2.52.8 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A jfrog-cli https://github.com/r-ryantm/nixpkgs/archive/b5732252190967e26f9e1e35fd24252686e8c4d1.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/b5732252190967e26f9e1e35fd24252686e8c4d1#jfrog-cli

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/9mvy5kkzfnrnj2n3xgjv3a9hpsx274pr-jfrog-cli-2.52.8
ls -la /nix/store/9mvy5kkzfnrnj2n3xgjv3a9hpsx274pr-jfrog-cli-2.52.8/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

jfrog-cli

Maintainer pings
cc @Detegr for testing.


Custom handlebars are not working

Describe the bug
I noticed that when I try to use one of the default custom handlebars from custom_handlebar.json or a self-written one I get an empty response body.
Handlebar I'm trying to use:
  {
    "name": "name",
    "logic": "(()=>{const name = request.query.name; return name;})()"
  }

Mock definition:
HTTP/1.1 200 OK
Content-Type: text/plain

{{#name}}{{/name}}

Logs:
2023-07-29 03:08:19 debug: Using new line as \n
2023-07-29 03:08:19 debug: Response Status set to 200
2023-07-29 03:08:19 debug: Headers Set Content-Type:  text/plain
2023-07-29 03:08:19 warn: Unexpected end of JSON input
2023-07-29 03:08:19 debug: Generated Response
2023-07-29 03:08:19 debug: HTTP PUT /nc/uploadVerificationDocument :: Query Parameters: {"name":"test"} | Request Headers {"user-agent":"vscode-restclient","accept-encoding":"gzip, deflate","content-length":"0","host":"localhost:8080","connection":"close"} | Request Body ""

Response:
HTTP/1.1 200 OK
X-Powered-By: Express
Vary: Origin, Accept-Encoding
Content-Type: text/plain; charset=utf-8
Content-Length: 0
ETag: W/"0-2jmj7l5rSw0yVb/vlWAYkK/YBwk"
Date: Fri, 28 Jul 2023 23:08:19 GMT
Connection: close

I have injection enabled in my config:
injection:
  enable: true

Maybe there is anything I'm missing or not doing properly?

Using custom handlebars also requires another config ext_helper. Could you check if you have specified the path to the custom_handlebar.json? In the default config.yml, it would be a disabled property, you would need to uncomment it before you can use them.
ext_helpers: "./custom_handlebar.json"

Can't believe I missed that one! Thanks @shubhendumadhukar, everything is working now


vscode reports a need to upgrade Gradle to an earlier version for JDK-21

I am using JDK-21 and Gradle 8.4, but when I open the project it tells me that the current project is not supported by Gradle 8.4 and I should install 8.3-rc-1.
It seems to work of with 8.4.

I get the same, but I am not sure this is related to to this Java VSCode plugin - I get the same  stupid warning with the latest IntelliJ for my project that uses Gradle 8.4/Java 21.  I posted some to reproduce unrelated issue here, but you should see the above warning as well when you open the code.
Seems like the warning is harmless though and quickly disappers?

It seems you get it once for every sub project.

This is a  Gradle 8.4 issue, to be fixed in 8.5 apparently:
https://github.com/gradle/gradle/issues/25574

FYI: Fix available in NB21: https://github.com/apache/netbeans/pull/6807

Merged https://github.com/oracle/javavscode/pull/128


Exclude communication

Sorry, I didn't mean to create a PR in your repo. Just wanted to check it in my fork that I created from this repo.


Elixir - tcp transport ping pong test fails.

@aniruddhasd & @palaniravi reported failure of this test
https://github.com/ockam-network/ockam/blob/develop/implementations/elixir/ockam/ockam/test/ockam/router_test.exs#L291

do you have a diagram that shows the supervisory tree to get some basic idea of over all flow?

I usually use Erlang's Observer application to get a visual of the supervision tree. Would that work for what you need?
https://erlang.org/doc/apps/observer/observer_ug.html

sure I will fire the observer and take a look. I use it also! thanks.

were you able to run observer on the docker? i am getting the following error. tried running in erlang shell also, I wanted to makes sure the docker's erlang footprint is complete and has everything installed.
13:24:26.991 [error] WX Failed loading<EMAIL_ADDRESS>{:error,
{{:load_driver,
'libwx_gtk2u_stc-3.0.so.0: cannot open shared object file: No such file or directory'},
[
{:wxe_server, :start, 1, [file: 'wxe_server.erl', line: 65]},
{:wx, :new, 1, [file: 'wx.erl', line: 115]},
{:observer_wx, :init, 1, [file: 'observer_wx.erl', line: 107]},
{:wx_object, :init_it, 6, [file: 'wx_object.erl', line: 404]},
{:proc_lib, :init_p_do_apply, 3, [file: 'proc_lib.erl', line: 226]}
]}}

Closing due to inactivity. The test failure was not detected in a while also.


naming the master locally has no effect for the cloud

Confusing:
if you name the master locally, it will not re-use the old name (id) in the Kontena Cloud (or, prompt for it, or fail).
kontena master join --name devmaster http://localhost:9292 invit3c0d3

This will work, but the master name will be devmaster-2 in the cloud and devmaster locally, which is bad since the user might not see what automagically just happened.
Feel free to close this if this is "works as intentended", just saying that there is too much magic for my taste (and I'm a Ruby on Rails magic believer (!))

There are two hard things in computer science: naming things, cache invalidation and off-by-one errors.

Does this fit here?
There's also the problem of the master hostname being autogenerated instead of using the one you entered during master deploy from cli.
Master can be named in a variety of locations and all of them could be different. At least:

Master hostname
Master config server.name
Cli config name:
Cloud master name


Remove the use of hard-coded .exe suffix

Additionally, clarify that lookup(executable:) expects the argument without executable suffix.
Fixes #1186

@swift-ci please test

Please provide the results from a local test run as the CI won't run the test suite on Windows.  We should ensure that the test suite isn't regressing on Windows due to this change prior to merging.

Please provide the results from a local test run as the CI won't run the test suite on Windows. We should ensure that the test suite isn't regressing on Windows due to this change prior to merging.

Well, that’s something beyond my scope — as I mentioned before, Swift Driver test suite crashes silently on my machines. If you know how to deal with such problem, I’m glad that you can help.

0xc00000fd means STATUS_STACK_OVERFLOW.  There is likely an infinite recursion going on but can't tell from just the screenshot what is going on.  I would say step through the assembly and find where the recursion starts and with what parameters, perhaps we can isolate that.

@compnerd I’m willing to further investigate the problem in a new issue in apple/swift. So for this PR could you test on your machine? I believe this is a pure NFC and thus should introduce no public behavioral change.

Fixed a silly bug — now the test suite can run and all failures are irrelevant (around several parsing tests).

@swift-ci please test

@swift-ci please test Windows platform


feat: client/worker parallel build

#124  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @hi-ogawa and the rest of your teammates on  Graphite


"Copy Link Location" is broken

When you start Binary Ninja, it will show you links for "Recently opened files".  If you right click and select "Copy Link Location", and then paste you get this: 0

Fixed by removing the option.


If I load without the backround=XXX in URL then no background

Description
When I load without the background=XXX in the URL then I get no background. No default as I expected it would just pick one. But then I was also unable to pick one from the menu as well.
Version
2.00-alpha.3.3
What browser are you seeing the problem on? What version are you running?
Chrome v109.0
Steps to reproduce
go to
https://mapwith.ai/rapid-v2-alpha3#datasets=fbRoads,msBuildings&disable_features=boundaries&map=20.06/44.08708/-103.20483
Then go to the menu and choose a background. Nothing happens.
The browser URL at the time you encountered the bug
https://mapwith.ai/rapid-v2-alpha3#datasets=fbRoads,msBuildings&disable_features=boundaries&map=18.68/42.78848/-83.92366

May be a dupe of https://github.com/facebook/RapiD/issues/767

yup - dupe - I will close.


⚠ Quirk Club Argentina (API endpoint) has degraded performance

In db0cf6e, Quirk Club Argentina (API endpoint) (https://us-central1-quirkclub-dev.cloudfunctions.net/api/check/api) experienced degraded performance:

HTTP code: 200
Response time: 8668 ms

Resolved: Quirk Club Argentina (API endpoint) performance has improved in 6b33976.


I am new to flutter and still stuck at flutter installation

i am watching a video tutorial on flutter. and I am stuck at the downloading the flutter packages on the flutter console.
i have download and extract the flutter in a folder,
opened the Flutter console,
entered "flutter doctor " in the console, now nothing is happening, and just stuck at this point, please i need help!.
i have tried reinstalling and disabling my anti virus.

yes, it show the version...
I have resolved the issue, thank you Maheshmnj for the quick response.
i kill all the task with this  "taskkill /F /IM dart.exe", and re-enter the "flutter doctor" and it worked.

Thanks for the update @iYAXES, Glad you were able to resolve the issue :)


Unify durations

It doesn't entirely "unify" them in the code, but they now behave identically.
You can now say both from 'now - 10h' and from '-10h' (they are synonyms).
The units supported are s, m, h, d, w (week = 7 days), M (month = 30 days), y (year = 365 days).

Why require 'now - XX'? This is meant to be syntactic sugar, otherwise it just adds more friction when you're pulling metrics from the UI or otherwise building queries.

TODO: allow '-10h' as a valid relative timestamp
(originally left out due to (erroneous) concerns regarding conflicts between parseDuration and parseDate, now resolved)

Now short relative timestamps ('-10h') are supported again.

If we support both now - 10h and -10h, what are the benefits of having the former?

There isn't much advantage. I'll pull the longer way out, and unify with duration parsing a bit more.

LGTM on my side - @achow ?


CIF internal usability testing with Ops Team

Describe the task
Complete first round of usability testing with at least one user to better understand if functionality to date meets user needs.
Acceptance Criteria

[x] make sure the testing environment has data
[x] print script
[x] prepare data collection tool (spreadsheet)
[x] disseminate data collection tool to attendees

Additional context

Ideally test once app deployed into TEST environment

Hi @elliottSlee - what are your thoughts/capacity on pulling this ticket into this upcoming sprint (March 7 - 18)? Ideally, I'd like to see this started :)

@elliottSlee - thank you and kudos for all the awesome work and analysis that went into this task. I really appreciate the insights gleamed and the relationship building that resulted from this work. A huge thank you to @suhafa for using this opportunity to get feedback on initial designs related to reports.