Not compatible with React 15.0.1

Any plans on fixing this in the near future? Had to rollback in order to accommodate this module.

Yes, I certainly still consider the module under active development. You'll have to give me a few days though. I haven't quite had the time to keep up with the React 15 developments these past weeks, but it seems it should be a priority now.

@ajzozakiewicz I've updated to React 15.0.1 and I didn't find any problems. I've also looked through the change log and couldn't really find anything that would indicate incompatibility. Would you mind giving me some additional info?
(Or were you just referring to the peer dependency config?)

I have that peer dependency issue indeed:
npm ERR! node v4.4.2
npm ERR! npm  v2.15.0
npm ERR! code EPEERINVALID
npm ERR! peerinvalid The package<EMAIL_ADDRESS>does not satisfy its siblings'
peerDependencies requirements!
npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>|| ^15.0.
0-0
npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>|| ^15.0.
0
npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants react-dom@^0.
14.0
How do I resolve this?

For the time being I have released v1.2.0-beta which allows<EMAIL_ADDRESS>as a peer dependency.
I noticed that when content is added/removed completely in rapid succession weird things happen. I imagine this is due to React using fewer spans in the markup, but I have yet to investigate this. I will resolve this issue before releasing a new stable version.

Upgrading to npm 3.8 + now makes peer dependencies warnings so I am now able to build.
Peer dependency was my only issue. Sorry I was not aware I was not on the latest version of npm.
Thanks
AJ
Sent from my iPhone

On Apr 18, 2016, at 1:42 PM, liquidox<EMAIL_ADDRESS>wrote:
I have that peer dependency issue indeed:
npm ERR! node v4.4.2
npm ERR! npm v2.15.0
npm ERR! code EPEERINVALID
npm ERR! peerinvalid The package<EMAIL_ADDRESS>does not satisfy its siblings'
peerDependencies requirements!
npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>|| ^15.0.
0-0
npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>|| ^15.0.
0
npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants react-dom@^0.
14.0
How do I resolve this?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub


move map helper function prototypes to EC

This partially addresses #291 .
The net ebpf extensions used to have the map helper function prototypes in both the xdp and bind program info. This PR moves those to the global helper program info in ebpf_core.
get_helper_prototype_windows callback from verifier uses global_program_info.type.platform_specific_data stored in TLS to get the program type guid, which is supplied during program verification call. This program type stored in TLS is used to obtain the program info from EC by invoking an IOCTL. The IOCTL handler invokes ebpf_program_get_program_info_data which calls program initialize, which in turn loads the extension corresponding to the input program type, and returns the program info provider data (which includes helper function prototypes).
The ebpf_program_get_program_info_data function is now modified to get the program info from both the program type extension and global helper NPI provider implemented by the EC itself. The latter contains the prototypes for global map helper functions. The helper prototypes from both the global and program type specific functions are combined to create a program_info_t object which is later serialized and returned as IOCTL output.

#291 looks like a laundry list of separate issues and this PR only addresses some of them.   Could it be split into two issues, one that is resolved with this PR and one that is not?


Add HTTP dumps for request and response

Hi,
thanks for providing this useful package. I want to add couple of PRs to improve it. The first one, is to add HTTP request and response dumps. Here is how it will look on a client side:
   var s string
    err := requests.
        URL(url).
        DumpRequest().
        DumpResponse().
        ToString(&s).
        Fetch(ctx)

Basically, DumpRequest and DumpResponse provides hooks to dump corresponding HTTP requests and response on stdout. Here is how it may look like on stdout:
# HTTP request dump
GET /some-api HTTP/1.1
Host: host.com
User-Agent: Go-http-client/1.1
Accept-Encoding: gzip
....

# HTTP response dump
\
HTTP/1.1 200 OK
Content-Length: 89
Cms-Server-Time: D=40530 t=1639863873694276
Content-Type: text/plain; charset=utf-8
Date: Sat, 18 Dec 2021 21:44:33 GMT
...

In case of error, the user may see actual HTTP request and response structs too. Feel free to make comments about syntax of the printouts and I may adjust it further.

Thanks for the PR. I’m not going to add this, but if you want to dump requests/responses yourself, you can write a custom transport with requests.RoundTripperFunc and use that in your own packages.


Broken link in testing mozilla.org

https://www-dev.allizom.org/styleguide/docs/send-to-device
In general, it would be a good idea to take a look and update this doc.

This doc contains information of obsolete pages too. The dynamic part needs some major rewrite. I will check with devs to see what pages to put in the 2nd half of the doc.


set up netlify

https://www.netlify.com/

netlify has been added, we can now preview branches being merged into master using http://deploy-preview-<insert branch number>--tc39-web-draft.netlify.com/


[lcms] rename library to lcms2

fixes #1519

If I understand correctly this will make libraw depend on lcms, right? We should probably revise libraw port as well, to ensure it won't link to undeclared dependencies.

Thanks!


Add API development guidelines.

Description
Adds guidelines to help people developing APIs.
Checklist:
Not relevant

Looks good thanks!


Don't display search results by default

As soon as the search field is focused, it already displays a number of search results (which I guess are the most downloaded ones?). That may be a bit confusing - instead it should simply display nothing

Addressed by e5f023cb3e27e3e163d130ee30499f368738d4a1


Multiple Issues with this extension in M 2.3.3

This extension has many issues preventing it from working correctly in the latest Magento 2.3.3

As soon as you click submit to make the order and pay via coinbase commerce, an order is created immediately, even before doing anything else. If you click Cancel and return to store the item is still in the cart but the order is already made.

If you pay the order properly and click the link to return to store, or let it timeout to go back automatically: you go to empty cart page instead of checkout/success (thank you page) and it says:
with this URL:
https://test.performance-pcs.com/coinbasecommerce/webhook/redirect/
"Sorry something went wrong, you will be redirected to checkout in 5 seconds."

The link for the link "Learn More" adds our website to the URL and so it goes to invalid page. Instead of https://commerce.coinbase.com it goes to: https://xxxxxxxxx.com/checkout/“https://commerce.coinbase.com/”

How can anyone even be using this extension with so many bugs?? I mean it looks nice and works well overall but there is no way to use this on a live site just for the first 2 bugs alone. Can anyone help here?? I'd even pay a small fee to fix the code for these few bugs..I'm sure the 3rd one is easy.

So no one replies here or wants to fix this mess of an extension??

Why is this group so useless?? 5v Contributors to the extension and not one person can reply to anyone and fix the problems. In latest M 2.3.4 and php 7.3 add to the list of problems (and this is worst problems). "Unable to find Coinbase order with store id". As posted here by at least 5-6 others.
Why is there no support from Coinbase Commerce??  In it's current state the extension is useless since you cannot view any order made paid with Coinbase Commerce, Extension just breaks backend. Is anyone able to fix this issue, even for pay?????

So with the help with one of our developers we fixed the main problem of not being able to view Coinbase orders in the Admin order grid.
1 exception(s):
Exception #0 (Magento\Framework\Exception\NoSuchEntityException): Unable to find coinbase order with store id "ORD-200325-4-009"
Exception #0 (Magento\Framework\Exception\NoSuchEntityException): Unable to find coinbase order with store id "ORD-200325-4-009"
#1 CoinbaseCommerce\PaymentGateway\Block\Adminhtml\Order\View\Details->getCoinbaseRecord() called at [vendor/coinbase/coinbase-commerce-magento/Block/Adminhtml/Order/View/Details.php:46]
#2 CoinbaseCommerce\PaymentGateway\Block\Adminhtml\Order\View\Details->getCoinsDetail() called at [vendor/coinbase/coinbase-commerce-magento/view/adminhtml/templates/details.phtml:4]
#3 include() called at [vendor/magento/framework/View/TemplateEngine/Php.php:59]

Solution:

/vendor/coinbase/coinbase-commerce-magento/Block/Adminhtml/Order/View/Details.php

lines: 44 ~ 63

    public function getCoinsDetail()
    {
        try {
            $record = $this->getCoinbaseRecord();
            $data['expectedCoins'] = $record->getCoinsExpected() . ' ' . $record->getReceivedCurrency();
            $data['amount'] = $record->getCoinsReceived() . ' ' . $record->getReceivedCurrency();
            $data['status'] = $record->getCoinbaseStatus();
            $data['code'] = $record->getCoinbaseChargeCode();
            $data['transactionId'] = $record->getTransactionId();
            $data['totalPaid'] = $record->getTotalPaid();
            $data['orderPlacedCurrency'] = $this->getOrder()->getOrderCurrencyCode();
        } catch (\Exception $e) {
            $data['expectedCoins'] = '';
            $data['amount'] = '';
            $data['status'] = '';
            $data['code'] = '';
            $data['transactionId'] = '';
            $data['totalPaid'] = '';
            $data['orderPlacedCurrency'] = '';
        }
There is still numerous issues with this extension listed above but no one here or from Coinbase support helps or replies in any manner nor do they seem interested to fix this extension.

Hi @perfpcs unfortunately we are moving towards deprecating support for the Magento plugin as it hasn't captured enough audience for us to maintain it at this time. However, we may come back to it in the future. The php SDK however will be kept alive. Feel free to clone this repository to make modifications for your use, which it seems by now you may have, or post any PR's you would like to fix for the php SDK to share with the community. Sorry we couldn't be more helpful at this time.


Children is  a JSX expression

When I use a JSX expression as children, I got the warning..
...
const record = {
      title: 'foo',
      status: 'bar',
};
return (
  <Record>{record}</Record>
);

warning is like this:
Warning: Failed prop type: Invalid prop `children` of type `object` supplied to `Record`, expected a single ReactElement.

React version is 16.2.0
can you help me??

It works, but has warning..

Do you have Record.propTypes defined and what's inside. If not I guess you have to try:
Record.propTypes = {
  children: PropTypes.object
}


Failed to connect to Jupyter notebook.

Hi, I try to connect to a server running on port 8888 (can be a local server or a forwarded ssh port).
I tried to specify multiple URI, like http://localhost:8888/ and http://<IP_ADDRESS>:8888/, but I always get the same error Failed to connect to Jupyter notebook..

Hi,
I guess I got the same issue here, when running a cell within vscode.

Running this Code works fine using the "Jupyter: Run selection" command...
vscode: 1.30.1
Python 3.5.2 :: Anaconda 4.2.0 (64-bit)
Win10 64 bit

Can both of you turn go to Help | Toggle Developer Tools and paste the Console output here?
You might also try updating to our latest build. It may resolve the issue and it will also output more error information.

Looks like this could be an incompatibility with Don's Jupyter extension (which is now obsolete)

@louisabraham and @EED85. If either of you are still having issues here I can help debug. Just as a good first step (as rchiodo mentioned) make sure that you are updated to the latest version of our extension and paste in the console output when you try to connect.

In my case upgrading with pip2 resolved the problem:
pip2 install --upgrade jupyter notebook

Not having a token is the likely culprit for why the Python extension can't connect. It needs one.
Not sure why Jupyter would run that way unless you explicitly disabled it in the Jupyter config. But we should be forcing that off when we start it (see issue #3475 that should be fixed in the build you have).
From an ananconda prompt, if you run 'jupyter notebook' do you get a user token there?

Then, the python extension will not work for remote Jupyter server if the token is disabled?
BTW, I disabled the token and use ssh forward to connect remote Jupyter server. IMO, ssh will secure the connectivity and there is no need to have token set.
Currently, I have met the same issue to connect remote Jupyter server via vs-code.

@JunyongYao I believe you're correct here. We work with tokens as that is the Jupyter default and we don't currently support passwords. But I think that our URL code always expects the token and you can turn it off as you have done. I've filed a new issue to track this here as it's not really related to this closed issue:
https://github.com/Microsoft/vscode-python/issues/4455

@JunyongYao I took a look at this. Actually you can connect to a server if you turn off both token and password protection. I tested this out myself with the following changes to a default Jupyter config:
c.NotebookApp.token = ''
c.NotebookApp.open_browser = False
c.NotebookApp.disable_check_xsrf = True
The last bit with the disable_checkxsrf is important. We connect using the webservices, so if that setting is still on you will not be able to connect (you'll see 403 errors on the server side).
With those settings I was able to start up a server and then connect to it with just the localhost url and without a token from the VSCode Interactive Windows.


Adding solvers

We'll want to register POMDPs.jl as an official package on METADATA.jl, but we probably don't want to litter METADATA with a dozen or so solvers. What if we just register POMDPs.jl and provide a function like POMDPs.add("MCVI"), kind of like Pkg.add, and it will do the necessary cloning?

Good idea.
On 1/31/2016 8:43 PM, Mykel Kochenderfer wrote:

We'll want to register POMDPs.jl as an official package on
METADATA.jl, but we probably don't want to litter METADATA with a
dozen or so solvers. What if we just register POMDPs.jl and provide a
function like POMDPs.add("MCVI"), kind of like Pkg.add, and it will do
the necessary cloning?
—
Reply to this email directly or view it on GitHub
https://github.com/sisl/POMDPs.jl/issues/57.

I like that as well. This will also allow us to do the builds for solver like SARSOP without the user have to run Pkg.build.

This has been added. You can now run:
using POMDPs
POMDPs.add("SARSOP")

The function will clone and build the solver. The docs have been updated to reflect this.


Adding a language

Hello, I would like to provide translation in my native language Tamil, it has 90M native speakers. I could not see my language in Crowdin. How do I add my language for translation?

I have added Tamil to Crowdin, looking forward to your contribution!
https://crowdin.com/project/easter-eggs/ta

Done. Hope the translations get approved.


Link to blog post is not working

The link to the AWS blog post describing the step-by-step process to use the solution does not work and in order to support customer adoption it's important for us to fix it. Can you please update the link referenced in this section:

Deployment
Please refer to this blog post for the detailed instructions on how to use the solution.

Done! Link is working now.


feat: add support for to_route

This adds support for to_route() this is the same as doing redirect(route()) or redirect()->route(), the current code when trying to use cmp would remove to_ so you'd only have route('something') instead of to_route('something')
I also updated the errors to use int since it was complaining and updated cmp to use the utils helper to reduce code duplication.

Good catch @ALameLlama


Matched properties data with length above 128 bytes aren't output if -i mode selected

In case of using -i mode and pattern that has named properties, if property value above 128 bytes it won't be output in results

fixed in 0.2.16


Add IN and NOT IN in WHERE clause

But on right side use only RAW.

There is it
https://github.com/catfan/Medoo/blob/master/src/Medoo.php#L903
https://github.com/catfan/Medoo/blob/master/src/Medoo.php#L989

But you solution don't solve like this
select * from table1 where id not in (select id2 from table2)

My solution can also solve your problem, and it is more
https://github.com/catfan/Medoo/pull/1003

$this->database->select(
"account",
"user_name",
[
Medoo::raw(" not in (select id2 from table2) "),
'LIMIT' => [20, 100],
"ORDER" => "user_id",
]
);

My solution can also solve your problem, and it is more

[
Medoo::raw(" not in (select id2 from table2) "),

need key on first row of this array:
PHP Fatal error:  Uncaught TypeError: preg_match() expects parameter 2 to be string, object given in Medoo.php:859
You tested this ?
Please correct me, but on this :
$a = $database->select("company", '*', [
        'ID'=> Medoo::raw(" not in (select order_id from fix_orders)"),
    ]);

I recive this SQL:
SELECT * FROM "company" WHERE "ID" =  not in (select order_id from fix_orders)

My solution can also solve your problem, and it is more

[
Medoo::raw(" not in (select id2 from table2) "),

need key on first row of this array:
PHP Fatal error: Uncaught TypeError: preg_match() expects parameter 2 to be string, object given in Medoo.php:859
You tested this ?
Please correct me, but on this :
$a = $database->select("company", '*', [
        'ID'=> Medoo::raw(" not in (select company_id from table2)"),
    ]);

I recive this SQL:
SELECT * FROM "company" WHERE "ID" = not in (select company_id from table2)

Use the master branch in my repository to modify, and you have to write this

$a = $database->select("company", '*', [
Medoo::raw(" not in (select company_id from table2)"),
]);

Use the master branch in my repository to modify, and you have to write this

thank you, I already used my solution in small project and try to share to another people to help


GarageDoor.Report not supported...

From log...
MQTT: GarageDoor.Report for garage door not supported.
Please report all bugs at https://github.com/dkerr64/homebridge-yolink/issues
{"event":"GarageDoor.Report","time":1673032650586,"msgid":"1673032650584","data":{"version":"060a","time":null,"loraInfo":{"signal":-70,"gatewayId":"d88b4c1603008c02","gateways":1}},"deviceId":"d88b4c010003f905"}

we now handle this (as a no-op)


README: syntax of JSON response is inconsistent

Compare the example JSON response in the markdown and the response in the screen captures. They are different.

Good find. Thanks Patrick!

Look at the first and the second markdown excample


Camry TSS2: use torque controller

Worked great! Definite improvement over the current PID tune. Not sure if TSS-P Camrys can also use this tune
Route (only difference is I merged in toyota-steer-faults which has no effect except in high curvature turns at low speed): f15e3c37c118e841|2022-04-20--00-46-27

On second try with the new values, it seems like steering is a bit jerkier in turns and hugs more (with laneless, so could be placebo, on both counts actually). Will look at the data, let me know if anyone tries this

What is The EOD Number for this? I need to Track Down Before it goes into
manufacturing. Thank You
On Wed, Apr 20, 2022 at 4:20 AM Shane Smiskol @.***>
wrote:

Not sure if TSS-P Camrys can also use this tune
You can view, comment on, or merge this pull request online at:
https://github.com/commaai/openpilot/pull/24268
Commit Summary

7a707cd
https://github.com/commaai/openpilot/pull/24268/commits/7a707cdbff8b4e20dc4a052b0614b33d7c4dbd76
use measurements from harald

File Changes
(1 file https://github.com/commaai/openpilot/pull/24268/files)

M selfdrive/car/toyota/interface.py
https://github.com/commaai/openpilot/pull/24268/files#diff-c933b99dbed0471164bc0132e6c5dcb8d32ce3e32051cf1309d60c6474c1c332
(5)

Patch Links:

https://github.com/commaai/openpilot/pull/24268.patch
https://github.com/commaai/openpilot/pull/24268.diff

—
Reply to this email directly, view it on GitHub
https://github.com/commaai/openpilot/pull/24268, or unsubscribe
https://github.com/notifications/unsubscribe-auth/APKZWGXROESNDHEHJRYHRCTVF6443ANCNFSM5T3G6WSQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Appstudio update test-component-pac-bfmk

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-bfmk-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
19 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
50 seconds
inspect-image

✅ Succeeded
2 minutes
clair-scan

✅ Succeeded
51 seconds
deprecated-base-image-check

✅ Succeeded
33 seconds
clamav-scan

✅ Succeeded
8 seconds
sbom-json-check

✅ Succeeded
17 seconds
optional-label-check

✅ Succeeded
17 seconds
label-check

✅ Succeeded
6 seconds
show-summary

Pipelines as Code CI/test-component-pac-bfmk-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
19 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
50 seconds
inspect-image

✅ Succeeded
2 minutes
clair-scan

✅ Succeeded
51 seconds
deprecated-base-image-check

✅ Succeeded
33 seconds
clamav-scan

✅ Succeeded
8 seconds
sbom-json-check

✅ Succeeded
17 seconds
optional-label-check

✅ Succeeded
17 seconds
label-check

✅ Succeeded
6 seconds
show-summary

Pipelines as Code CI/test-component-pac-bfmk-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
13 seconds
clone-repository

✅ Succeeded
48 seconds
build-container

✅ Succeeded
21 seconds
inspect-image

✅ Succeeded
19 seconds
deprecated-base-image-check

✅ Succeeded
11 seconds
clair-scan

✅ Succeeded
43 seconds
clamav-scan

✅ Succeeded
7 seconds
sbom-json-check

✅ Succeeded
16 seconds
label-check

✅ Succeeded
15 seconds
optional-label-check

✅ Succeeded
8 seconds
show-summary


Add youless power meter template

Discussed in https://github.com/evcc-io/evcc/discussions/8919

Originally posted by matrover July 11, 2023
Hi,
is it possible to add the youless P1 meter to EVCC? i couldn't find how to.
The youless has an API which replies JSON and is fairly popular in NL.
see page 23 in: https://www.youless.nl/downloads.html?file=tl_files/downloads/youless-ls120-handleiding-1.50.pdf (in dutch sorry)
it boils down to doing a get command http://YOULESS-IP/e
which replies: [{"tm":1689080858,"net": 124.518,"pwr":-490,"ts0":1689079123,"cs0": 0.339,"ps0": 0,"p1": 99.587,"p2": 95.229,"n1": 37.680,"n2": 32.618,"gas": 6141.894,"gts":2307111505,"wtr": 0.000,"wts":0}]
im able to give remote access for testing purposes.
Thanks in advance

- name: grid
   type: custom
   power:
     source: http
     uri: http://YOURYOULESSIP/e
     headers:
       - content-type: application/json
     jq: .[0]|.pwr # parse response json


Monitoring feature

Monitoring work, buttons to each aspect

@nekrut @bgruening automated runs are also producing consensus FASTA sequences, which may be worth mentioning.

Loading this entire page is resource-consuming. @dannon can we somehow indicate that parts of the page are still loading?

@nekrut @dannon and, yes, I'm seeing a similar issue like @bgruening when trying to view the page. It makes the fan of my laptop spin like crazy and freezes the browser for quite some time though eventually it finished loading for me.

Splitting it up in multiple pages might help?

I'll take a look at the performance of the page now.

@bgruening @wm75 I built a quick component to defer loading until the frame is scrolled to -- this seems to help dramatically on my end but can you test in your browser as well?  If this is a viable approach I'll polish it up.

Yes, way better. Is it possible to show a spinner as long as it loading? And can we get rid of the scrollbars? Many of them are not scrollable, on my screen at least.

Thanks @dannon!


Avoid alignment warnings on some CPUs

Describe the contribution
Fix #437 and partially address #313 (overlapping issue)
On CPUs with strict alignment requirements, some CFE code that uses a char-type pointer (e.g. uint8*) to compute memory addresses triggers an alignment warning when it gets cast back to the actual data type.
In the mempool implementation, the pointer should be sufficiently aligned already, because the address computation already takes CPU alignment requirements into account when calculating the addresses/offsets.

For the CFE_SB pool buffers, using the cpuaddr type, which is integer in nature, avoids the warning.
For the CFE_TBL internal table pointer, use a void* internally to store the buffer pointer, rather than a uint8_t*.  This changes the casting needs elsewhere.

Testing performed
Build CFE with ENABLE_UNIT_TESTS=TRUE
Confirm all unit tests passing
Perform sanity test on CFE (normal startup, send commands from console)
Build for MIPS64 and ensure that (some) alignment warnings are fixed
Expected behavior changes
No change to behavior.  Fixes build warnings only.
System(s) tested on:
Ubuntu 18.04 LTS  64-bit
Additional context
There are still some remaining alignment cast warnings regarding the message types, where a local message buffer is cast to a CFE_SB_Msg_t* which has a higher alignment requirement.  This is a little harder to fix as it requires changing the local buffer definition.
Contributor Info
Joseph Hickey, Vantage Systems, Inc.
Community contributors
You must attach a signed CLA (required for acceptance) or reference one already submitted

Update: changed the description to reflect that this will fix the char* casts cited in #437 but the older issue #313 also mentions issues related to the message types (CFE_SB_Msg_t* casts) that this does not fully address.
There will need to be an additional pull request to fix those ones.

does this mean however that the pointer may not be aligned? I don't know if there are any CPU's that cannot interact with unaligned data but it will affect performance on many platforms. Should we add code to align?

SPARC and MIPS processors (at least) will throw an exception when attempting to access a data structure that is not properly aligned.  I believe ARM and x86 will auto-correct but there is a performance penalty to pay.  Not sure about PowerPC.
For the items specifically in this pull request (the pool address calculation) the buffer descriptor is already a union including large types, such that sizeof(BD_t) will be a multiple of the system alignment required for the large.  In turn all pool buffers are also already allocated to be a multiple of this size.
So all the pointers are already aligned sufficiently, the problem is merely that it was using uint8* intermediate pointer to perform the address calculations, and then casting a uint8* back to the intended type.  This final step is what triggered a warning, just because the final type has a stricter alignment requirement than uint8* does.
By doing the intermediate address calculation as an integer (cpuaddr) the result is exactly the same but gcc doesn't warn about it.

CCB 20190108 - Reviewed and approved for integration candidate


I feex Steppies

This won't actually fix it, no of merging yet. Calculated events need to be typed and added to fflogs defs.

Fixed by ack


Enable use of IUAInboxDelegate protocol

Changes the type of UAInbox delegates to the "IUAInboxDelegate" protocol, so classes that implement that protocol can be UAInboxDelegates. Doesn't require any source changes.

👍

🖼


Filter datasets

Add the ability to filter datasets by a range of properties
Currently all datasets that are visible to a user are displayed. This will become a very large number so we must add the ability to filter these by a range or properties. These filters will appear above the table that displays the datasets.
The current text filter box will remain and will filter only those datasets that are displayed in the same way that if currently does. It should be distinct from the filter elements as this behaviour is different.
Also related to this is the ability to paginate the results, but this is not yet supported by the API (1)
Initial properties to filter
Filtering by these properties is already supported by the API.

Owner (select one user from the list of users) (2)
Mime-type (Note: we should use human friendly names here not terms like chemical/x-mdl-sdfile). This will be a drop down list with values coming from the /type endpoint
Label name (list of comma separated labels). It would be nice to have some sort of autocomplete here, but the full list of labels is not currently available (3)

Filter properties that will be added later
These filters should soon become available

Editor (similar to how the owner filter works)
Label name=value

A note about labels
Labels are simple name=value pairs with the value being optional. Hence these are valid examples:
important
pdb
pdb=1HC3
A dataset can have any number of labels.
Currently only filtering by the label name is supported by the API, but we will soon be adding the ability to specify a name and a value. e.g. if you specify pdb as a label name filter you would get the last two items, but if you specified pdb=1HC3 then you would only get the last item.
IM tasks

@duncanpeacock update this (maybe create new issue?) once pagination is supported
@OliverDudgeon describe how to get the current list of users
@duncanpeacock examine if a list of label names can be made accessible

This react-table doc has an example for how to add filtering to the current table.
https://react-table.tanstack.com/docs/examples/filtering

Available users is currently accessed with this hook:
import { useGetUsers } from '@squonk/data-manager-client/user';

// In a react component
const { data, error, isError, isLoading } = useGetUsers();

This relates to a task of #15

Filtering by labels should allow to filter by multiple labels. The obvious way to handle this is to use a space separated list of terms in the text field. The complication is that label values can contain spaces. @OliverDudgeon suggested that this could be handled using "chips", and happy to consider suggestions for how best to handle this from the UX aspect.
The API already supports this. All terms are ANDed. There is no need to complex AND/OR logic.


Update zhCN.lua

Hi @T3rdW
Thanks again for your work. I merged this and also brought it back into the classic version.
Cheers,
RagedUnicorn

Cheers! :D


RFP: Substrate Identity Directory

Just an RFP.

Thanks @Swader and @mmagician

Thanks @Swader and @mmagician

Thanks @Swader and @mmagician

Thanks @Swader and @mmagician


Tomcat

Hello,
Does anybody have problem with using WebSocketClient with Tomcat-7.0.61.
Client Code.
ExampleClient c = new ExampleClient( new URI( "ws://localhost:8080/WebSocketProxy/websocket/echoAnnotation"), new Draft_17() );
c.connect();
c.send("hhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhh");
onOpen is called and replies.
onMessage is never called whatever i do.
//Server code on Tomcat
@ServerEndpoint("/websocket/echoAnnotation")
public class EchoAnnotation {
@OnMessage
public void onMessage(Session session, String message ) {
System.out.println( "server received: " + message );
}
@OnMessage
public void onMessage(Session session, ByteBuffer message ) {
System.out.println( "server onMessage bytebuf: ");
    try {
        session.getBasicRemote().sendBinary(message);
    } catch (IOException e) {
        // TODO Auto-generated catch block
        e.printStackTrace();
    }
}
@OnOpen
public void open(Session session) {
    System.out.println("onopen");
    try {
        session.getBasicRemote().sendText("hello from server");
    } catch (IOException e) {
        // TODO Auto-generated catch block
        e.printStackTrace();
    }
}

}
Thanks,
Hugo

Sounds like the socket is being closed right after opening, could be that the client is not compatible with Tomcat... Have you tried using a JS client?

Closing old issue without any clear exception.
Greetings
marci4


Use question side as stat site for Auth_user

@nherter what was this about again?

@SantaClaas no idea haha

Ok I'll close this then


Gitorious Merge Request Formatter

Would be a nice integration to have.

Certainly would be. As I don't use Gitorious myself, this is not a priority for me. But I'd gladly merge it in if someone else wrote it :smile:

I'd say that this can be closed since Gitorious is getting shut down by the end of May: https://about.gitlab.com/2015/03/03/gitlab-acquires-gitorious/.

@jeroenj :+1:


The safe-paste plugin does not make pasting safe and breaks other features.

The safe-paste plugin was recently introduced to provide protection from the copy-paste shell command attack as demonstrated at https://thejh.net/misc/website-terminal-copy-paste but fails to do so.
It is even stated on the page that the exact plugin does not work.

Please note that Bracketed Paste Mode DOES NOT fix this because the end sequence can be inside the text you paste! For those of you who have installed the oh-my-zsh stuff, sorry, but the following variant (which includes an escape sequence) still works against you

Enabling this plugin also breaks the escaping of special characters that are pasted including those of URLs.
The safe-paste plugin should be ether removed or fixed immediately because it is giving users a false sense of security and breaks other important features.

I use PuTTY which does not support bracketed paste. Is there a capability of telling zsh to escape newlines (not immediately execute them) if they arrive within a very small amount of time?


[Fusion] Failed to get Convolution Forward solution with filter 1x1

Error while creating Fusion Plan with Convolution+Bias+Activation and filter size 1x1.
 ) AND (data_type = ? ) AND (direction = ? ) AND (spatial_dim = ? ) AND (in_channels = ? ) AND (in_h = ? ) AND (in_w = ? ) AND (in_d = ? ) AND (fil_h = ? ) AND (fil_w = ? ) AND (fil_d = ? ) AND (out_channels = ? ) AND (batchsize = ? ) AND (pad_h = ? ) AND (pad_w = ? ) AND (pad_d = ? ) AND (conv_stride_h = ? ) AND (conv_stride_w = ? ) AND (conv_stride_d = ? ) AND (dilation_h = ? ) AND (dilation_w = ? ) AND (dilation_d = ? ) AND (bias = ? ) AND (group_count = ? ) )AND (arch = 'gfx906' ) AND (num_cu = '60');
MIOpen(OpenCL): Info2 [impl] [NCHW,FP32,F,2,64,56,56,1,1,1,1,64,1,0,0,0,1,1,0,1,1,0,0,1]
MIOpen(OpenCL): Info2 [Prepare] SELECT solver, params FROM perf_db INNER JOIN config ON perf_db.config = config.id WHERE ( (layout = ? ) AND (data_type = ? ) AND (direction = ? ) AND (spatial_dim = ? ) AND (in_channels = ? ) AND (in_h = ? ) AND (in_w = ? ) AND (in_d = ? ) AND (fil_h = ? ) AND (fil_w = ? ) AND (fil_d = ? ) AND (out_channels = ? ) AND (batchsize = ? ) AND (pad_h = ? ) AND (pad_w = ? ) AND (pad_d = ? ) AND (conv_stride_h = ? ) AND (conv_stride_w = ? ) AND (conv_stride_d = ? ) AND (dilation_h = ? ) AND (dilation_w = ? ) AND (dilation_d = ? ) AND (bias = ? ) AND (group_count = ? ) )AND (arch = 'gfx906' ) AND (num_cu = '60');
MIOpen(OpenCL): Info2 [impl] [NCHW,FP32,F,2,64,56,56,1,1,1,1,64,1,0,0,0,1,1,0,1,1,0,0,1]
MIOpen(OpenCL): Info2 [SetValues] , content inserted: ConvAsm1x1U:1,8,1,64,1,4,1,1
MIOpen(OpenCL): Info2 [SetValues] , content inserted: ConvActivAsm1x1U:2,16,2,64,1,3
MIOpen(OpenCL): Info2 [SetValues] , content inserted: ConvOclDirectFwd1x1:1,64,1,1,1,0,16,256,0
MIOpen(OpenCL): Info2 [SetValues] , content inserted: ConvBiasActivAsm1x1U:1,8,1,64,1,4,1,1
MIOpen(OpenCL): Info2 [SetValues] , content inserted: ConvHipImplicitGemmV4R4Fwd:64,32,32,8,2,2
MIOpen(OpenCL): Info2 [SetValues] , content inserted: ConvBinWinogradRxSf2x3:50
MIOpen(OpenCL): Info [GetValues] =ConvOclDirectFwdFused:<values not found>
MIOpen(OpenCL): Info2 [Measure] Db::Load time: 184.414 ms
MIOpen(OpenCL): Info [FindSolutionImpl] Perf Db: record not found for: ConvOclDirectFwdFused
MIOpen(OpenCL): Info2 [GetPerformanceConfig] Returns: 16,16,32,32,1,0,32,128,1
MIOpen(OpenCL): Error [BaseGetSolution] result.out_pix_tile1 == 0 || result.out_pix_tile0 == 0
MIOpen(OpenCL): Info [Compile] No viable kernel found to execute the fusion plan
Convolution+Bias+Activation plan not supported.

The error is thrown in the source:
https://github.com/ROCmSoftwarePlatform/MIOpen/blob/3e45c258b812bf7e3974635c109f92b00ff6db7a/src/solver/conv_ocl_dir2Dfwd.cpp#L305-L309
However, result.out_pix_tile1 or result.out_pix_tile0 are always 0 if the filter size is 1x1:
https://github.com/ROCmSoftwarePlatform/MIOpen/blob/3e45c258b812bf7e3974635c109f92b00ff6db7a/src/solver/conv_ocl_dir2Dfwd_exhaustive_search.cpp#L92-L93
https://github.com/ROCmSoftwarePlatform/MIOpen/blob/3e45c258b812bf7e3974635c109f92b00ff6db7a/src/solver/conv_ocl_dir2Dfwd_exhaustive_search.cpp#L119-L120

ConvOclDirectFwdFused should not be applicable for 1x1 fusions, like its ancestor, ConvOclDirectFwd (which is applicable only for 3x3, 5x5, 7x7, 11x11). It seems like this should be fixed: https://github.com/AMDComputeLibraries/MLOpen/pull/1113/files#r667220325.
@JehandadKhan Do you agree?

@kahmed10 (originated from https://github.com/ROCmSoftwarePlatform/MIOpen/issues/706#issuecomment-877269104):

...I am seeing the following MIOpen errors (Navi21)
MIOpen(HIP): Error [BaseGetSolution] result.out_pix_tile1 == 0 || result.out_pix_tile0 == 0
MIOpen(HIP): Error [BaseGetSolution] result.out_pix_tile1 == 0 || result.out_pix_tile0 == 0
MIOpen(HIP): Error [BaseGetSolution] result.out_pix_tile1 == 0 || result.out_pix_tile0 == 0
MIOpen(HIP): Error [BaseGetSolution] result.out_pix_tile1 == 0 || result.out_pix_tile0 == 0
MIOpen(HIP): Error [BaseGetSolution] result.out_pix_tile1 == 0 || result.out_pix_tile0 == 0
MIOpen(HIP): Error [BaseGetSolution] result.out_pix_tile1 == 0 || result.out_pix_tile0 == 0
MIOpen(HIP): Error [BaseGetSolution] result.out_pix_tile1 == 0 || result.out_pix_tile0 == 0
MIOpen(HIP): Error [BaseGetSolution] result.out_pix_tile1 == 0 || result.out_pix_tile0 == 0
MIOpen(HIP): Error [BaseGetSolution] result.out_pix_tile1 == 0 || result.out_pix_tile0 == 0

This was not present before. I am attaching an updated log, but I'm not sure how to debug. This could also be an artifiact from the gfx1030 ROCm builds I am testing.
resnet50_migraphx_log2.txt

ConvOclDirectFwdFused should not be applicable for 1x1 fusions, like its ancestor, ConvOclDirectFwd (which is applicable only for 3x3, 5x5, 7x7, 11x11). It seems like this should be fixed: https://github.com/AMDComputeLibraries/MLOpen/pull/1113/files#r667220325.
@JehandadKhan Do you agree?

I agree, that is a bug. Opened a #1032 for fix


Project paths for beaengine are absolute

The projectpaths for beaengine are absolute
(C:\Users\Owner...)
They should be relative to the project
(..\beaengine...)

both libraries are precompiled and included in the git, i will change these for people who need to recompile them but its a non-issue atm

How is this a bug? Most projects don't even include the library dependencies and expect the dev to know what they're doing. It's not a bug, or an issue,

i thought i had removed the bug label, my bad


*: clarify backup/restore operator usage

add doc and warning to clarify that  backup/restore operator are example implementation and not for production usage.

cc/ @hongchaodeng @hasbro17

all fixed cc/ @hasbro17

lgtm after minor nit

lgtm


Improve Landing Page

The current landing page looks alright, but I would ideally like to change it into something better
https://cuvette.tech/
The animation on the landing page is not hard to do if you find the lottie animation data for the same.
https://polenhq.com/

@Rishabh-malhotraa I would like to work on this! I was confused with the layout too as in where to put animation, CTA , content?

@Rishabh-malhotraa I would like to work on this! I was confused with the layout too as in where to put animation, CTA , content?

The animation cuvette is using is called lottie animations. Basically they have done animations in after effects, export the lottie data, and just load that data to the browser using a lottie component.
So if you somehow manage to get the lottie data, you install react-lottie library and  pass that data to the Lottie component as props, and animations would work.
The main animations work is done in Adobe aftereffects.

I was asking about how to place the current landing's page content in that? And do you want the exact animation as cuvette's?

Right @Aniket-508 , so yes an exact animation of what cuvette uses would be great!
As far as how to place content w.r.t. to our current landing page.
We would most probably shift the search field to the left side below the "Seach, Schedule, Meet" text and have animations on the right!
Also if we have a cool animation like cuvette, I would remove the particle effect in the background that we currently use.

Hello I am a GSSOC participant, If the issue is still not fixed then I would love to work on it

This is a low priority issue, meaning it's good to have but not really required, but still, if you want to work on it go ahead.
Can you let me know what designs do you have in mind? You can search on dribble and create a figma mockup of the same!

Can I work on this under GSSoC 22

Can I work on this under GSSoC 22

@sagnik2001 can you create a Figma mockup before trying to work on this change, so we know how you plan to improve the landing page!

so @sagnik2001 are you working on this, if yes would you like to collaborate and if no @Rishabh-malhotraa Can I give it a shot ?

@RaviAnand111 do you still want to work on it?


WIKI Updates

Presentations
Add project presentations to: learn --> Media Library --> (new) Project Presentations
Ensuresec generic presentation
CxC generic presentation
Secant generic presentation
Bug statement
Add a statement to this page https://wiki.iota.org/integration-services/welcome under the "caution" box at the bottom. Please also place this in a yellow warning box with a wee bug image and "bug reporting" as the heading
"Please report any bugs found whilst engaging with any part of the integration services to<EMAIL_ADDRESS>

@DuncanMain, do we by any chance have videos for these presentations? I think they'd probably reach a broader audience that way.

@lucas-tortora No we don't, but.... I think we can export them as vids. I will try.


Add integration for Quire - https://quire.io

Please follow this guide when creating a new issue. This speeds up the process of replicating the issue and finding a solution.
Environment details

Operation system:
Extension version:

Expected behavior
Actual behavior
Steps to reproduce the behavior
Additional details

screenshots or screencapture

I've created a pull request #1077 to add this integration

I've created a pull request #1077 to add this integration

Request have some conflicts.

Is there any service like toggl but support quire？


Add LUA parser

Add the ability to load LUA files from Pinocchio. This method is more flexible than URDF format allowing the definition of new joint types.
This code is largely inspired from RBDL (http://rbdl.bitbucket.org). Thanks to Martin Felis to provide this code.

Looks interesting and I'm very happy to see LuaModels getting adopted in other tools.
In RBDL I am using the string "JointTypeEulerZYX" to denote the Euler ZYX joint. Similarly "JointTypeXYZ", "JointTypeYXZ", "JointTypeTranslationXYZ", and "JointTypeSpherical" to denote the other 3-DoF joints. It would be great if you would use the same strings to ensure compatibility. Personally I would also prefer "JointTypeFloatbase" instead of "JointTypeFreeFlyer".
Fixed joints simply are specified as an empty table.
On a minor note: "Lua" is not an acronym, therefore the spelling "LUA" does not make sense. The Lua community will thank you for spelling it correctly.

Thanks Martin for your remarks and useful propositions.
In order to be compatible with RBDL, I will choose the same convention as you did in RBDL. Concerning the fixed joint, I would prefer to mention it explicitly in the model. What do you think about this approach ?
For Lua, I will change the commit message asap. Sorry for my misreading.
Best,

I would assume that the concept of "no joint specification, no movement" is easy to understand and also easy to implement using LuaTables++.
Another reason is the other use of LuaModels in MeshUp: if one were to add additional frames for visualization meshes only one would have to add joint specifications for every added visualization frame to stay compatible with RBDL.
What do you think?

I do agree with your proposition. I did some minor modifications in the Lua parser (fb899b4) to fit your suggestions.
Maybe, in order to be compatible in both directions, could you add "JointTypeRX", JointTypePX", etc in RBDL ?
Thank you again Martin for helping us!

I would rather have the special cases of 1-DoF joints detected from the spatial vector itself. However if you do insist I could imagine adding "JointTypeRevoluteX", "JointTypeRevoluteY", and "JointTypeRevoluteZ". Prismatic joints seldomly are used apart from the first movable body. For now I would leave it out but if there is demand for it I would add it.


Emacs: add "unfill" command

(defun unfill-paragraph ()
  (interactive)
  (let ((fill-column (point-max)))
    (fill-paragraph nil)))

(defun unfill-region ()
  (interactive)
  (let ((fill-column (point-max)))
    (fill-region (region-beginning) (region-end) nil)))

Consider binding unfill-paragraph to M-Q, similar to how fill-paragraph is M-q
See #18, #21, #27

Also consider adding docstrings so that the usual Emacs help commands still work


Auto update locale files

I think for the most part, the makemessages command for locale files doesn't create many conflicts, so having some PR made every time a new commit is pushed would be great!

@Andrew-Chen-Wang would you still like to see this implemented? if so, please assign me this issue.

would appreciate thank you. The script to do this already exists in the repo; we just need a way to auto update in the CI on merge with master

@Andrew-Chen-Wang After looking at what's already in the repo, I'm not clear on what a PR that would resolve this issue needs to implement. The issue seems to describe what the i18n.yaml already does. Currently that workflow is disabled but it works as expected.
From the below list of workflow descriptions, what are you looking for from a PR that closes this issue:

on merge to master, run workflow and open another PR to update locales

Appears to be what's described in this issue and what's already in the disabled i18n.yaml workflow

On PR, test if locales need to be updated, fail if they need to be updated

Easy and would prevent out-of-date locales being merged to master before a release.

On PR, test if locales need to be updated, update the PR locale files if needed

A little more difficult

Something else entirely? please let me know.

This was the last attempt: https://github.com/jazzband/djangorestframework-simplejwt/pull/540#issuecomment-1054683409


feat(multi-chain): Add support for multiple chains

This PR adds support for multiple chains, allowing a client to specify which chain they would like an inclusion proof for. This PR also includes various architecture which result in faster syncing, proof generation and more efficient tree versioning. Further testing and error handling in progress.

This looks killer Daniel! Will probably take me a while to get though all of these changes but I'll let you know once I'm through it!


Implement a .deployignore file to override / add to the excluded paths

So I made a tool to automatically deploy files for my company last weekend and I created a .deployignore file to define what's excluded from that deployment.
But since that idea of an .deployignore file has been a thing to me for a while now, I just wanted to ask if it were possible to include functionality to this plugin to read such a file and use it to add to the excluded paths feature of the deployment section of the JetBrains IDEs?
Another way would be to have an option to import those excludes from that file (just like you'd open the npm tasks panel)

I think it would make teamwork much easier in some places as only the person to create the file would have to set up what's excluded and others then wouldn't have to set it all up again 😊

Another way would be to have an option to import those excludes from that file (just like you'd open the npm tasks panel)

Any input on this? 🤔

@metaa I have reviewed the overall idea of this feature request. To make it possible, I'll have to inject the proper values into the IDE configuration.
Deployment section in IDE settings is not available in the community version, so it's not possible to review the code and put those values with IDEA SDK.

Oh yeah, I didn't think of that 🤔

@metaa Just did additional research - settings values related to this specific option is not stored externally in IDE configuration files, so it's not even possible to bypass the SDK and set it via file system.
No can do, sorry.

For what I know there's keepass files that store deployment info but ...ehh ... yeah, you'd have to ask the user for the password and stuff, so maybe not.


[ios]device lab perf test should support running multiple times and aggregate the result for non-A/B test too

Use case
For device lab tests, if we run A/B test, we can pass in --ab=10, which runs the test 10X2 times and accumulates the results.
Sometimes I just want to compare the performance of several local engine branches (e.g. local main, branch1, branch2, branch3), I don't want to know how they perform compared to non-local engine (which is the A/B test)
Proposal
Option 1:
Upgrade the A/B tests to allow a flag that takes a list of local branch names. e.g. instead of doing --ab=10, we do --count=10 branches=main,branch1,branch2,branch3...
Option 2:
Just allow non-A/B tests to run multiple times and accumulate the result. (e.g. --count=10). It should accumulate the result, and I will compare different branches manually.

Option 1 will be very tricky since you'd have to teach the framework test runner how to check out different engine branches (it doesn't know where the engine repo is) and rebuild the engine between checkouts.  However altering A/B to skip the A (default engine) and just summarize B (local-engine) 10x times sounds simple enough.

Note from our discussion:
Can just comment out https://github.com/flutter/flutter/blob/9d5380889d39e2ce4549b6957d44464087280ee2/dev/devicelab/bin/run.dart#L158-L173
i will continue with this. still helpful to officially support this via a flag tho


Updated README.md for efficiency

Fixes Issue
Removed redundant, dead block-chain development link.
Replaced broken links with working links in:

Android Development Roadmaps
DevOps Roadmaps

Note to reviewers

I saw these broken links, so I fixed them.
If the links were broken on purpose, then ignore this Pull Request.
I'm generally new to contributing to open source and would love feedbacks :)

@kaiwalyakoparkar review the change whenever you can :)


feat: Eliminate entity grouping

Summary
See discussion in https://github.com/kaskada-ai/kaskada/issues/268

This is likely a good idea, but some things to note:

We likely want to think about a way to make the implicit join more explicit. Otherwise this will increase the rate of "confusing implicit joins".
I think I took a look at this at one point and there was some reason why it wouldn't be trivial to do this. It may just be that there are a lot of places that are affected (since the "entity grouping" is used to determine whether things need to be repartitioned), but it may be that there are deeper issues to iron out. This may be easier to address once we have a logical plan instead of the existing DFG.

A quick and dirty way of accomplishing this would be to use the key's type (i.e. string) as the grouping ID.

I support this direction. Any downvotes?

@bjchambers any objection to using the "quick and dirty" solution for now?

My concern here is that we have multiple times had issues related to the implicit join that can occur. This change will increase the occurrences of implicit joins, and likely increase the occurrences of that problem. I think the current compilation will also cause us to join everything with the same grouping. This could actually create problems.
Consider:

Two different entity types (user and store), but both keyed by i64. With this change, they will now be the same entity type.
A query which does a lookup between the two. Before, these were seen as two different entities, so we'd run two separate operations. After this change, there is a chance that we will treat them as a single entity type, thus merging them.
Normally, this wouldn't be a problem due to the way "newness" is handled, but there are aggregations that would allow an aggregation over the "user" entity to count the occurrences of the store as well.

One place that we've seen this occur is something like ticks, which enumerate the entities. With this change, it will enumerate all users and store IDs rather than just all user IDs.

seems like we should NOT do the "quick and dirty" solution then.

The thing is that grouping doesn't prevent, or in my experience meaningfully reduce, the incidence of implicit joins, however it does increase the complexity of the 'user model'.
Adjacent query languages (SQL, CEP, Cascalog, etc) don't provide typed groupings, so users are unlikely to expect the language to provide this type of "safety", and if they experience case (1) I think it's unlikely they'll perceive the resulting empty-join as a failure of Kaskada or the query language. OTOH, if they are trying to work across tables with the "same entity" but without the appropriate configuration I think they're likely to perceive grouping as a confusing feature of Kaskada's design, as a result of its novelty within data query languages.
I think the possibility of implicit joins is orthogonal to grouping - we should solve this on its own terms, not with grouping, since grouping doesn't actually prevent implicit joins.
I also feel that "typed" entities is a layer of complexity in the user model that doesn't pay for itself.
If there are errors in the implementation of joins across tables with the same grouping or in how ticks operate that should be addressed separately, since we currently allow configuration of the table's grouping - the proposal here is simply to use the existing system mechanisms - nothing proposed here changes the current system's behavior.


18487 - Upgrade Vite/Vitest

Issue #: /bcgov/entity/18487
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the bcrs-entities-create-ui license (Apache 2.0).

Does this work as a library inside of auth-web as expected?

Does this remove the current service worker?

Does this work as a library inside of auth-web as expected? (possible to point locally to this PR, then run auth-web)

good catch, just tested, it has some issue

updated service worker should working now:


Merge development to master

What this pull request does
This pull request merges the application to the master branch

Changes Unknown when pulling 407b09a1d0796e29766f8a888e4533759a28bbc2 on development into ** on master**.


Clarification Needed on Proper Use of program.setTrace for Accurate Evaluation and Success Count Increment

I've encountered some confusion while trying to run the optimized version using a different training set. The success count only increments when program.setTrace is called, and I'm unsure of the correct implementation.

taking a look

found some issues in ax will fix

several fixes related to this in the latest release checkout the src/examples/dope-or-nope.ts thats committed.

Awesome. Closing this PR then.


feat(gateway-cli): show balance for each federation

This is to resolve #2508.
I've added a total_msat field to the FederationInfo struct, which is the same name used in the output of fedimint-cli ng info.
It was suggested in https://github.com/fedimint/fedimint/issues/2508#issuecomment-1546471869 to remove the balance command, which I could do here or separately as required.

This is still in progress. I need to investigate the failing test.


Fix compilation of functions that return an array

Given this ElixirScript code...
defmodule Foo do
  def foo do
    [1,2,3]
  end
end

defmodule Main do
  def start(_, _) do
    JS.console.log Foo.foo # Returns `3`
  end
end

It would compile code like....
if (...) {
    const [_0] = __arg_matches__;

    1

    2

    return 3;
}

When it should compile like...
if (...) {
    const [_0] = __arg_matches__;

    return [1, 2, 3];
}

This fixes that by wrapping the body of compile_clause in J.array_expression.

Thanks for catching that!


Color hex should allow you to type

As the title says. The color panel should allow you to type in a custom hex so you don't have to try and get the right colour on the chart

Thx for your advice : )


"Commit all files" does not stage unstaged files anymore

When using the "commit all files" shortcut, unstaged files appear to no longer be staged automatically - in the commit window, no diffs are displayed for the unstaged files at all.
I presume this might be a consequence of #917 .

@nmorenor can you maybe take a look? this looks related to https://github.com/zaggino/brackets-git/pull/959

Sure I will take a look

@joepie91 Hi I was trying to reproduce what you see when trying the "commit all files" shortcut, but I was not able to reproduce?

Could you please provide more details on what you see on your side, as I understand from your previous message you do not see the changes on the commit dialog, is that correct ? I tried the following cases and it worked for me.

Change a file & save it.

use "Commit all files" shortcut

Change a file & save it.

Stage the file by selecting the combo box inside the brackets git Panel

Change a different file & save it

use the "Commit all files" shortcut

In these two cases I see the dialog as posted above, perhaps there is another case I'm not trying.

Could it be that the files you do not see are new files on the repository?

@nmorenor Yes, this is specifically about new files that hadn't been committed before. Not sure why I didn't mention that, sorry for that!
Previously, those files would be added automatically. Now, they are not staged and do not appear in the diff either.


Implement OpenApi schema changes to support validation reports

Describe the problem
As outlined in #223, implement the OpenAPI schema changes to facilitate the integration of validation reports.
Proposed solution
Add the validation report summary to the gtfs datasets on the endpoints /v1/gtfs_feeds/{id}/datasets and /v1/datasets/gtfs/{id} as described in #223.
Alternatives you've considered
No response
Additional context
No response

Schema changes in #224 can be seen here https://github.com/MobilityData/mobility-feed-api/pull/224/files

Todo:

[ ] design SQL Schema for validation report data
[ ] python code for returning validation report data via API

@jujiang526 Examples of validation report JSON from https://gtfs-validator.mobilitydata.org/
MBTA feed validation report
SEPTA feed validation report
Let me know if you're looking for specific use cases.

Completed in #244


[Docs] exhaustively document setup parameters

Summary
As long as setuptools has wrapped the setup function it has only documented keywords that it added or edited the meaning of [1]. The implication is that others are inherited from distutils' setup, but that is not stated anywhere, nor is distutils even mentioned to exist as a module. Cleaver users that go and lookup distutils to find these missing parameter definitions are greeted with a page redirecting them back to setuptools [2] but if they dig, historical API docs can be found [3]. This is a long walk to take to get to the only official place basic parameters such as name or version are specified as part of setup's API.
Now that setuptools has taken official stewardship of distutils, it would be great if the API docs for objects shared by both could be unified.

https://setuptools.pypa.io/en/latest/userguide/keywords.html
https://docs.python.org/3.10/library/distutils.html
https://docs.python.org/3/distutils/setupscript.html

OS / Environment
No response
Additional Information
After this documentation is written, developers will only need to find one page to know everything setup is capable of accepting.
Code of Conduct

[X] I agree to follow the PSF Code of Conduct

Hi @ucodery, I think this is the complete list of keywords:
https://setuptools.pypa.io/en/latest/references/keywords.html
Please note that the list also points out which keywords are deprecated and should not be used.

I see, thank you.
If I diff those different keyword.rst files the only meaningful difference, besides the preamble, is the lack of distutils' keywords on the userguide page. Would there be objection to a PR that drops the userguide page in favor of the reference one? ISTM that one source of this information will make for easier to maintain and easier to find documentation. Already there are small changes in wording  between shared keyword descriptions, presumably as these files are updates independently and are drifting apart.

I think we can merge the descriptions in the userguide into the reference (if relevant), and just keep in the userguide the list of the names of keywords (no description) + a link to the references page at the end for more information.
What do you think? Does it make sense?

I think that makes sense. Linking from the semi- to the fully-complete keywords guide seems helpful to users poking around, as I was.
I am tempted to remove explicit mentions of distutils, as that is no longer a separate package. Both pages mention that some keywords are setuptools' and some aren't, but from now forward they are all managed by the setuptools project. WDYT?

That should be fine overall.
But is there a way we don't loose the information about which keywords were added by setuptools?
That probably will not matter for end users but may help developers and contributors submitting PRs... Maybe moving to the development section of the website?


Update billing info rules

There's a new page in settings for Clips privacy, so the billing info has been bumped down to new positions.
Ideally I would like to find a way to make more rules like these specific with content selectors, but I can't figure that out with these, for now at least.
There's very little to pinpoint them with, especially because their aria labels change with the language the user has set on their client.

Yep, same issues has I had, I currently i also have no other idea than this


Bug: MySQLi Config "encrypt" values not being respected

PHP Version
8.2
CodeIgniter4 Version
4.3.2
CodeIgniter4 Installation Method
Composer (using codeigniter4/appstarter)
Which operating systems have you tested for this bug?
Windows
Which server did you use?
apache
Database
No response
What happened?
While following the instructions for configuring a database connection, I have configured my env file to contain the following:
database.default.DBDriver = MySQLi
database.default.encrypt.ssl_ca = C:\xampp\perl\vendor\lib\Mozilla\CA\cacert.pem
database.default.encrypt.ssl_verify = true

However, it appears these are not being respected by the actual SSL connection, as seen in Database.php. Thus, checking in vendor/codeigniter4/framework/system/Database/MySQLi/Connection.php. it appears that the entire section for checking if is_array($this->encrypt) is skipped over due to not being an array.
I have "fixed" this by adding a simple line beforehand that manually sets the encrypt values:
$this->encrypt = [
    'ssl_key'    => '',
    'ssl_cert'   => '',
    'ssl_ca'     => 'C:\xampp\perl\vendor\lib\Mozilla\CA\cacert.pem',
    'ssl_capath' => '',
    'ssl_cipher' => '',
    'ssl_verify' => 'true'
];

Thus continuing the checks and ensuring I can make an encrypted connection.
Steps to Reproduce

Set any of the encryption subvalues in your .env file, or in Database.php
Go to vendor/codeigniter4/framework/system/Database/MySQLi/Connection.php
Within the connection function, add a simple print_r($this->encrypt); and observe that it appears to be being interpreted as a boolean rather than a array.

Expected Output
The correct values to be passed to Database.php / Connection.php to allow for connecting to databases that enforce SSL connections.
Anything else?
No response

This is not a bug.
See #7311

This is documented, but the current description is not sufficient?
https://codeigniter4.github.io/CodeIgniter4/general/configuration.html#environment-variables-as-replacements-for-data

Hi @kenjis,
Thanks for that, I understand that the env files cannot create new entries - my main issue is thus with the documentation that specifically states what entries can be put into the env file:
https://codeigniter.com/user_guide/database/configuration.html#configuring-with-env-file
I'm happy to find another workaround by forcing the code elsewhere, but if these values are not respected, then should they be removed from the documentation?
Thanks!

Environment variables can only be strings. And CI4's configuration class can only change scalar values by the environment variable.
If a property of a configuration class has a scalar value, you cannot change it to an array. If a property has an array value, you can change the value (scalar) of its elements, but you cannot add an element to the array.
This is not properly documented and is also difficult to understand.
I think we need to improve the documentation.
If you really want to set the values only in the .env, describe the property values in JSON in the .env and add code to convert the JSON string to an array in the constructor of the Config class.

Thanks @kenjis,
Much appreciated on this one - I definitely think the documentation needs some work here.
In the meantime, I'm still not sure what's the best option for me to ensure that I enable SSL when talking to my web database, as short of modifying the /vendor/ directory, I don't appear to be able to enable ssl_verify as seen in Connection.php
Any ideas?
Thanks!

Certainly, this SSL setting will vary depending on the environment, so it is impossible to set the Cert file location in the Config file.
database.default.encrypt = {"ssl_verify":true,"ssl_ca":"C:\\xampp\\perl\\vendor\\lib\\Mozilla\\CA\\cacert.pem"}

--- a/app/Config/Database.php
+++ b/app/Config/Database.php
@@ -80,5 +80,10 @@ class Database extends Config
         if (ENVIRONMENT === 'testing') {
             $this->defaultGroup = 'tests';
         }
+
+        $array = json_decode($this->default['encrypt'], true);
+        if (is_array($array)) {
+            $this->default['encrypt'] = $array;
+        }
     }
 }

I sent a PR to improve the docs: #7454

Thank you so much on both counts for this, very much appreciated!


Cognito mail invitation message template is not supported from cli-inputs.json ??

Before opening, please confirm:

[X] I have searched for duplicate or closed issues and discussions.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

JavaScript Framework
Angular
Amplify APIs
GraphQL API
Amplify Categories
auth
Environment information

# Put output below this line

Describe the bug
On file cli-inputs.json I have
emailVerificationSubject
emailVerificationMessage
But cognito has another template "Invitation message" , can configure that message like Verification message on cli-inputs.json ?
O the only way is add template on cognito web console ?

Expected behavior
I expected on the cli-inputs.json file the same config for Verification Message, some like  :
{
  "version": "1",
  "cognitoConfig": {
          ....
         "emailInvitationSubject": "Your temporary password",
         "emailInvitationMessage": "Your username is {username} and temporary password is {####}."
          ....
   }
 }

Reproduction steps
I expected that amplify-client create fields for email verification messages
Code Snippet
// Put your code below this line.

Log output

// Put your logs below this line

aws-exports.js
No response
Manual configuration
No response
Additional configuration
No response
Mobile Device
No response
Mobile Operating System
No response
Mobile Browser
No response
Mobile Browser Version
No response
Additional information and screenshots
No response

Hey @togro :wave: we can use overrides to set this property! Using this doc for reference we can run amplify override auth and use the following to customize the invitation message template:
import * as cognito from '@aws-cdk/aws-cognito'
import { AmplifyAuthCognitoStackTemplate } from '@aws-amplify/cli-extensibility-helper'

export function override(resources: AmplifyAuthCognitoStackTemplate) {
  const inviteMessageTemplate: cognito.CfnUserPool.InviteMessageTemplateProperty =
    {
      emailMessage:
        'Hello {username}, you have been invited to join our awesome app! Your temporary password is {####}',
      emailSubject: "You're Invited!",
    }

  resources.userPool.adminCreateUserConfig = {
    inviteMessageTemplate,
  }
}

By doing so, this allows us to control the invitation message template from within the overrides.ts file. If we wish to keep the messaging together in cli-inputs.json we can also create new inputs that will be used as parameters to our CloudFormation template:
cli-inputs.json
{
  // ...
  "smsAuthenticationMessage": "Your authentication code is {####}",
  "smsVerificationMessage": "Your verification code is {####}",
  "emailVerificationSubject": "Your verification code",
  "emailVerificationMessage": "Your verification code is {####}",
+ "invitationEmailMessage": "Hello {username}, your temporary password is {####}",
+ "invitationEmailSubject": "You're invited -- from the cli-inputs.json!",
  // ...
}

And modify our overrides.ts to simply reference the new parameters:
import { AmplifyAuthCognitoStackTemplate } from '@aws-amplify/cli-extensibility-helper'

export function override(resources: AmplifyAuthCognitoStackTemplate) {
  resources.userPool.addPropertyOverride('AdminCreateUserConfig', {
    InviteMessageTemplate: {
      EmailMessage: {
        Ref: 'invitationEmailMessage',
      },
      EmailSubject: {
        Ref: 'invitationEmailSubject',
      },
    },
  })
}

Now after we push our resources with amplify push we can see the changes are reflected in the console 🙂

I'll mark this as a feature request to automatically generate those parameters and CFN references for auth resources 🙂

Thanks !
works fine !
👍


handle error in run_cv()

handle the case where an Exception is raised in cv_results = cross_validate(), which enters the except block, but we still returns cv_results which does not exist at that point.
def run_cv(X, y, cv, model, metrics, n_jobs, logger):
    model_name = get_model_name(model)

    # The metrics we want to use for cross-validation
    scorers = {metric.__name__: make_scorer(metric) for metric in metrics}

    logger.info(f"Cross-validating {model_name}...")
    logger.info(f"Parameters: {model.named_steps['model'].get_params()}")

    try:
        cv_results = cross_validate(
            model,
            X,
            y,
            cv=cv,
            scoring=scorers,
            n_jobs=n_jobs,
            return_estimator=True,
            return_indices=True,
        )

    except Exception as e:
        logger.error(f"Failed to cross-validate {model_name}")
        logger.error(e)

    return cv_results

Hey @bryanlimy -- do you have a status update on this one?

I add cv_results=None in https://github.com/alan-turing-institute/autoemulate/blob/5c743a4b34bd4197da8424419b7ec42a7fddfa6a/autoemulate/cross_validate.py#L20 https://github.com/alan-turing-institute/autoemulate/pull/180 so that it won't through an error when cv_results is not initialized, we will have to improve error message in general. I don't think there is a "fix" for this issue.

Note: decided to close this issue and open a separate one about the improvement of the error message.


done username fetching when auth with github

Looks good. It might be good to file the support request in the Meteor issue queue, if one does not already exist.


Build improvements for 1.7 changes

This PR will not work until https://github.com/OSC/ondemand/pull/87 is merged and tagged into latest or ci repos.

This PR should be good to go.

I have a rebase of this ready.  I'm happy to push it and carry this forward. More than the dependency updates (that have conflicts), I think the rpm updates a super valuable.
Let me know.

Can rebase this branch if you want, or new PR , up to you.

Thanks! I just thought I'd take care of it, one less thing for you to do. I've pushed and am pulling this in now.


Add Elixir 1.4 to the text matrix.

Now that @tchoutri has fixed the existing 1.4 warnings (thank you!), let's keep it going.

Coverage remained the same at 83.38% when pulling 05d650dc582cde37f75b025f314b2c62e25bdae9 on elixir-1.4 into de09ba2ccc74e7f89c66679edb2f821576fa13e7 on master.

Coverage remained the same at 83.38% when pulling 05d650dc582cde37f75b025f314b2c62e25bdae9 on elixir-1.4 into de09ba2ccc74e7f89c66679edb2f821576fa13e7 on master.

Coverage remained the same at 83.38% when pulling 05d650dc582cde37f75b025f314b2c62e25bdae9 on elixir-1.4 into de09ba2ccc74e7f89c66679edb2f821576fa13e7 on master.

Coverage remained the same at 83.38% when pulling 875386392ff1c7d14adee192559f7a220eb6d957 on elixir-1.4 into de09ba2ccc74e7f89c66679edb2f821576fa13e7 on master.

Coverage remained the same at 83.38% when pulling 875386392ff1c7d14adee192559f7a220eb6d957 on elixir-1.4 into de09ba2ccc74e7f89c66679edb2f821576fa13e7 on master.

Coverage remained the same at 83.38% when pulling 875386392ff1c7d14adee192559f7a220eb6d957 on elixir-1.4 into de09ba2ccc74e7f89c66679edb2f821576fa13e7 on master.

Coverage remained the same at 83.38% when pulling 875386392ff1c7d14adee192559f7a220eb6d957 on elixir-1.4 into de09ba2ccc74e7f89c66679edb2f821576fa13e7 on master.

Coverage remained the same at 83.38% when pulling 875386392ff1c7d14adee192559f7a220eb6d957 on elixir-1.4 into de09ba2ccc74e7f89c66679edb2f821576fa13e7 on master.

Shoot. Compiler warnings from 1.4 in Ecto code are causing the build to fail. Can the warning be disabled selectively?

I don't know if it's possible, better ask @josevalim ?

On Elixir v1.4 we don't compile dependency with warnings I believe?

@josevalim yes, that's been my experience. The situation here is a little more complicated because we're directly including some of Ecto's files as though they were part of this library (integration tests, notably) and it is those files that are getting caught up in --warnings-as-errors. As of now, this library is still using an old version of Ecto, so it's not really feasible to go fix the warnings in Ecto.

Coverage remained the same at 83.287% when pulling b5ebbeba903f534614049898913190628816df7e on elixir-1.4 into 37c7957cd92642afb7f8047f7e50bd148754b026 on master.

Coverage remained the same at 83.287% when pulling d229795454c94ce75881a552e32e197bc9af38f2 on elixir-1.4 into 37c7957cd92642afb7f8047f7e50bd148754b026 on master.

Coverage remained the same at 83.287% when pulling 53666cb3576802407eced637124da2df7c2b0226 on elixir-1.4 into 37c7957cd92642afb7f8047f7e50bd148754b026 on master.

@tchoutri @josevalim thx for your interest. I wound up finding a minimal solution to the problem, which was to tweak the compiler settings just before calling Code.require on the relevant Ecto code.


webgui, option to disable dragging of rules

Why is this needed?

Disables dragging rows to allow selecting and copying row contents and avoid accidental changes.

Please resolve the conflicts.

@sbeaver-netgate , conflict resolved


Responsive sticky top

This is solution for creating responsive sticky top class which closes #28311.
cc: @mdo

Hi @jahanzaibsuleman07

Could you move @supports (position: sticky) around the loop, this prevents the supports query from being generated multiple times
This needs to be documented in our docs
We also need documentation for if you want to use the mixin below a specific size. I guess we'll need to work with the .position-static utility class then?

Hi @jahanzaibsuleman07

Could you move @supports (position: sticky) around the loop, this prevents the supports query from being generated multiple times
This needs to be documented in our docs
We also need documentation for if you want to use the mixin below a specific size. I guess we'll need to work with the .position-static utility class then?

Hi @MartijnCuppens:
I have addressed the first two points. Please review.
For third point, do you want something in the documentation like below?
**Mixins**

Sass mixin can be used to create a class for a specific viewport and below:

 @include media-breakpoint-down(sm) {
      $infix: breakpoint-infix(sm, $grid-breakpoints);

      .position#{$infix}-static {
        position: static !important;
      }
    }

Hi @MartijnCuppens and @mdo, do we need any further changes here?
Also please see my previous comment. Thanks!


Gas limit seems to not be considered

Using v3.0.0, started with a gas limit set at 5800000:
testrpc -d -t 3 --account="0x558ad1........94101a5926c9, 100..." --account="0x558ad19776d....26c8, 100..." -l 5800000
Expected Behavior
Provided limit should be applied.
Current Behavior
Limit seems to be ignored and use the new default: 90000
Available Accounts
==================
(0) 0x139f459f164a2c96114f2d08638c12b2c4a74037 🔓  - 100 ETH
(1) 0xe18593cba7a64aca8532bf71b6f3c13681774d86 🔓  - 100 ETH

Private Keys
==================
(0) 558ad19776d1eb5289118bf70cc2b6c3e8abd925b1e3cf64fbc894101a5926c9
(1) 558ad19776d1eb5289118bf70cc2b6c3e8abd925b1e3cfd4fbc894101a5926c9

Gas Limit
==================
5800000                                                   <== OK, that´s what I requested

Listening on localhost:8545
<...>
eth_accounts
eth_sendTransaction

  Transaction: 0x3b26242e3e7e3283cf8270b6318f96b89eee83e1bd96459fa23fcdcbb8bc16ab
  Contract created: 0x6893cb1d8a837d01677871a0699b65dc45acc424
  Gas usage: 0x015f90                                     <== Not OK, that´s 90000
  Block Number: 0x01
  Block Time: Fri Oct 07 2016 21:16:23 GMT+0200 (CEST)
  Runtime Error: out of gas

Hi @chevdor. That flag sets the block gas limit and not the transaction gas
limit. If you want your transactions to have a higher gas limit, you should
specify that in the transaction request.

Yep, I just saw that in https://github.com/ethereumjs/testrpc/blob/master/lib/blockchain_double.js#L45
I opened an issue in the webpack solidity-loader. That´s where the the gas limit for the tx should be defined, does not it?
I can run the migrations using truffle migrate but webpack fails.

I did not submit a PR for that but I made a change to customize the tx gas limit:
https://github.com/chevdor/testrpc/commit/0b53b178f948e0a6c90b0cdaa24f914d4800d1ef
That is a working workaround for me atm.
I think we can close this issue. Let me know if the change can be interesting and I can put together a clean PR for it.

@chevdor I do not think it is a good idea to override the default tx gas limit. Dapps should specify the gas limit for each of the calls they make. (This is how it works with real clients too, such as geth/parity.)

I agree with you @axic, this is just a workaround.
I currently have an issue with the solidity-loader for webpack (see link above) that probably does not pass the gas as it should. So for now, this fix is the only option I have. Now, I agree that we may not want to add this feature, this is why I did not make a PR.
Hopefully the solidity-loader issue can be fixed soon, making this obsolete. In the meantime, I have at least this solution and I wanted to document a bit for the people who may run into the same issue.


startup profiling/performance tuning

Has anyone got any tips to share regarding the following?

measuring which parts of sails are taking the longest during startup,
specific issues/patterns to look out for which will slow startup,
getting sails to start faster?

For fastest startup and potential issues, it's worth verifying/optimizing/cleaning-up the bootstrap and database configuration.  Upgrading database adapters or Sails itself when feasible may can potentially improve startup performance.
@alxndrsn  Here's a Gitter post by the founder covering further performance optimization philosophy for Sails - also related to database management.  Scalability https://sailsjs.com/documentation/concepts/deployment/scaling and reliability of the system / architecture hosting the app also play a role in startup performance.
Also found some additional Sails startup advice on Stackoverflow.
As for specifically measuring startup performance, a sails lift currently displays the timestamp of when it finished lifting (limited to whole seconds).   I think there might be a way to tweak this further to display milliseconds with a console timelog script as one startup profiling  technique.  As for 3rd party tools, I'll look around myself for them and report back any significant ones compatible with Sails.
Hope this helps!

Upgrading database adapters or Sails itself when feasible may potentially improve startup performance as well.

Interesting point.  Are you accepting patches for sails-postgresql/machinepack-postgresql?  They currently depend on node-postgresql version 6.1.6, which is more than two years old.

@alxndrsn Sure, we can take a look at them - are they specifically to update node-postgresql?

@johnabrams7 yes, but I'm curious the difference it makes because:

I have open PRs on both repos which haven't received any review, or any other response.  One of them is over 6 months old at this point.

My PR that was merged to sails-postgresql 7 weeks ago has still not been officially released.

I appreciate the burden of maintaining open source software, plus all the hard work that's gone into the Sails framework and supporting library, and I get the impression that support for these two packages is effectively on hold and they are already deprecated in favour of the impending sails-sql adapter (see various comments including https://github.com/balderdashy/sails/issues/6817#issuecomment-513333171, https://github.com/balderdashy/sails/issues/6820#issuecomment-515169117, https://github.com/balderdashy/sails/issues/6880#issuecomment-555219021, https://github.com/balderdashy/sails/issues/6888#issuecomment-555215340).
However, sails-sql still hasn't reached 1.0,  hasn't had a public release for 9 months, and hasn't had a public commit since April 2019.
Is there anything I can do to help expedite PR review/merge or otherwise help with PosgreSQL support?  If I were to submit a PR for updating the node-postgresql dependency for sails-postgresql, what is the chance of getting that released?  Would it be more productive, and helpful to the community, if I were to actively support a PostgreSQL-focussed fork of sails-postgresql?

@alxndrsn Wanted to follow-up: How are you doing on the startup profiling/performance tuning for this one?

Some ideas for profiling SailsJS startup can be found at https://kevin.burke.dev/kevin/node-require-is-dog-slow/, but note that this blogpost is from 2015.


Support controller inheritance without duplicating translations

I'm using responders 2.1
I'm using 2 controllers. One is the main one filters_controller, and I have a second controller with the same name but within a namespace admin that inherits from this one.
# controllers/filters_controller.rb

class FiltersController < ApplicationController
  def update
    respond_with @filter
  end
end

# controllers/admin/filters_controller.rb

module Admin
  class FiltersController < ::FiltersController
    ...
  end
end

Now, when I add the translations for the responder, the only way for the namespaced controller to recognize the texts is by duplicating the translations.
My translations file now looks like this:
# en.yml
en:
  flash:
    admin:
      filters:
        update:
          notice: Your filter was successfully updated.
          alert: Your filter could not be updated.
    filters:
      update:
        notice: Your filter was successfully updated.
        alert: Your filter could not be updated.

Is there a way to avoid duplication?
Otherwise, it would be nice if the reponder gem handles the fallback of these messages to the parent controller, following the behaviour from rails itself that handles all these type of cases for you.

There is no way right now but a PR is welcome. I'm closing this since it is a feature request.


New Script - Alteryx Server

Description
Add a PowerShell script to reconfigure an installed instance of Alteryx Server with a certificate, automating a majority of the steps outlined in the official instructions here https://help.alteryx.com/20221/server/configure-server-ssltls
Script is a modified version of ImportWindowsAdminCenter.ps1
Type of change

[x]  New feature (non-breaking change which adds functionality)

How Has This Been Tested?

[x]  New Certificate Request

Performed through CLI
PS C:\bin\win-acme.v<IP_ADDRESS>0.x64.trimmed> .\wacs.exe --emailaddress<EMAIL_ADDRESS>--accepttos --source manual --host server.example.com --installation script --script "Scripts\ImportAlteryxServer.ps1" --scriptparameters '{CertThumbprint}'

Test Configuration:
Windows Server 2019 on EC2
WACS v<IP_ADDRESS>0.x64.trimmed

Checklist:

[x]  I have performed a self-review of my own code
[x]  I have commented my code, particularly in hard-to-understand areas

@WouterTinus  just seeing this - I can modify the script, do you have a specific example you would want me to base this on?

Nothing specific, but for example this one is pretty clean: https://github.com/win-acme/win-acme/blob/master/dist/Scripts/ImportExchange.v2.ps1
But it's really just that section which moves the cert to another store that's not supposed to be handled at the installation step.


Add 7zip SDK plugin

http://www.7-zip.org/sdk.html

Reopening—this should be doable now using the threaded streaming interface.

Git mirror of the SDK: https://github.com/jljusten/LZMA-SDK


feat: Install Appcues

Asana Task: Install Appcues

Checklist

Tests:

(x) Has tests
( ) Doesn't need tests
( ) Tests deferred (with justification)

Product/Design sign off:

(x) Okayed the plan for the feature (e.g. the design files, or the Asana task)
( ) Reviewed the feature as implemented (e.g. on dev-green, or saw screenshots)
( ) No review needed

Waiting for https://github.com/mbta/terraform_modules/pull/49 merge before proceeding here


VxWorks: use wr-cc for C file, use wr-c++ for CPP file

@alexcrichton Please help to review the code.

r? @alexcrichton


VS Code 'Deploy Bicep File' does not detect change in file used in loadTextContent()

Is your feature request related to a problem? Please describe.
Using VS Code to deploy Bicep file.  Files referenced in loadTextContent() that are changed don't show up in the deployment.
It appears that Bicep in VS Code does not completely build the template if files have not changed.  A file used in loadTextContent() having a change does not cause the template to be rebuilt so the new value in that file does not get deployed.
To Reproduce
main.bicep file references a bicep module (testModule.bicep) which has this line
var textFromFile = loadTextContent('ScriptTestInsert.sql')
that variable is used to set a property in a data factory and is also used as output from the module for verification.

Run Deploy Bicep on main.bicep
Change text in ScriptTextInsert.sql file
Run Deploy Bicep on main.bicep
Look at deployment in Azure portal and the original text from ScriptTestInsert.sql file appears in the module Template and Output

Either of these two steps will cause the changed value to be incorporated in the template

Changing testModule.bicep (even if just adding a space and saving) will trigger the template to be rebuilt and the new value being used in the deployment.
Closing VS Code and reopening does cause the change to be detected. I suspect this is because the caching of the 'build' result is kept in memory and close/open cycle starts fresh.

Describe the solution you'd like
Detect changes in files referenced in loadTextContent() as being changed and do the rebuild before deployment so Azure reflects the recent change.

@bhsubra -- can you take a look and see if this is specific to bicep deploy?

we believe this is the result of caching in the Compilation Manager

@bhsubra - we just got another report of this one. Do we have a workaround for clearing the cache? If not, should we revisit the priority of this one?

@bhsubra / @puicchan - we just got another report of this one. Do we have a workaround for clearing the cache? If not, should we revisit the priority of this one?

@alex-frankel, I am investigating this. Updated the milestone to v0.12. Thanks!


some issues with Promise

I think promise is a pretty big section that should probably be split into its own lesson
Some issues I'm seeing:

we should encourage reject(new Error('str')) rather than rejecting with strings
we should encourage capitalizing Promise constructor
Promise.reduce() and catching SyntaxError is not part of the Promise spec; I guess we should decide whether the lesson uses simple ES6 promises, or Bluebird, or a bit of both
.then(onResolve, onReject) should be described a bit
memoization seems like a bit of an advanced pattern, I think it's generally not needed and might just confuse students as they are trying to learn the basics
Promise.resolve, Promise.reject and Promise.all should be described somewhere
instead of using reduce the last examples should be using map (reduce is used when you want to resolve to a single value, e.g. getting the max width of all loaded images)

for e.g.
var images = Promise.map(imageArray, loadImage);
images.then(renderImages);

Or with ES6:
var images = Promise.all(imageArray.map(loadImage));

Or with error handling:
var notFoundImage = loadImage('notfound.png')

var images = Promise.map(imageArray, function(file) {
  return loadImage(file)
    .then(null, function(err) {
      return notFoundImage
    })
})

Not sure if these and the other PRs flew under the radar. Should I merge?

Yeah for some reason I hadn't even seen em. I merged.


TypeError: llmx.generators.text.hf_textgen.HFTextGenerator() got multiple values for keyword argument 'provider'

Thanks for creating this Library but I am sure "Testing" is not been the priority before releasing this. I just installed and followed the documentation and boom "The Error" while using the HF model.
Here is the complete error:
[Traceback (most recent call last):   File "C:\Users\aiany\OneDrive\Desktop\lida demo\test.py", line 8, in <module>     text_gen = llm(provider="hf", model="uukuguy/speechless-llama2-hermes-orca-platypus-13b", device_map="auto")   File "C:\Users\aiany\OneDrive\Desktop\lida demo\.venv\lib\site-packages\llmx\generators\text\textgen.py", line 75, in llm     return HFTextGenerator(provider=provider, models=models, **kwargs) TypeError: llmx.generators.text.hf_textgen.HFTextGenerator() got multiple values for keyword argument 'provider'](url)
Just these 2 lines of code I have used:
`[from lida import llm
print("Import Successful!")
#text_gen = llm("openai")
text_gen = llm(provider="hf", model="uukuguy/speechless-llama2-hermes-orca-platypus-13b", device_map="auto")](url)`

Hi thanks for flagging this.
I just pushed a fix to llmx release (v 0.0.17a) that addresses this.
pip install llmx

Also ..
A Note on Using Local HuggingFace Models
While llmx can use the huggingface transformers library to run inference with local models, you might get more mileage from using a well-optimized server endpoint like vllm, or FastChat. The general idea is that these tools let you provide an openai-compatible endpoint but also implement optimizations such as dynamic batching, quantization etc to improve throughput. The general steps are:

install vllm, setup endpoint e.g., on port 8000
use openai as your provider to access that endpoint.

from llmx import  llm
hfgen_gen = llm(
    provider="openai",
    api_base="http://localhost:8000",
    api_key="EMPTY,
)
...

!pip3 install --upgrade llmx==0.0.17a0

# Restart the colab session

from lida import Manager
from llmx import  llm
text_gen = llm(provider="hf", model="uukuguy/speechless-llama2-hermes-orca-platypus-13b", device_map="auto")
lida = Manager(text_gen=text_gen)


Fix and update travis stuff

Fix https://github.com/RJacksonm1/node-dota2/issues/268

Does this match all that you dreamed and hoped for @Crazy-Duck ?

You fulfill my wildest dreams and imagination :heart:
However two questions:

Why nuke requestMatchMinimalDetails.js?
Why are there .proto.txt files ?

Oops. Fixed.
No one needs it, and it only exposes my ugly and quick hack (the method needs to be fixed to accept multiple ids, unless someone else did that already).
We can re-add it if anyone misses it.


Custom training data has poor quality on hand

I started from  images using the enerf-outdoor dataset ，used colmap to generate camera parameters and init point clouds. The background quality in the result is very good, but once the characters move there is a noticeable loss of quality.
look at face and hand

maybe you can enlarge the batch size and make the training process longer?
now I also face the same problem and I'm working hard to solve it QwQ


[C++][Parquet][Python] New API to 'zip' or (vertically) 'attach' parquet metadata

Describe the enhancement requested
Hi,
One of the design principles of parquet from their Github page is 'Separating metadata and column data':

Separating metadata and column data.
The format is explicitly designed to separate the metadata from the data. This allows splitting columns into multiple files, as well as having a single metadata file reference multiple parquet files.

In order to achieve the 'columns in different files', we need to

Ensure each file has the same number of row-groups
Ensure each corresponding row-group of each file have the same rows
Grab the 'metadata' from each file, 'zip/attach them vertically', and write out the new metadata file
Feed this metadata while reading the table

It looks like Arrow APIs provide nearly everything to achieve this, except for the bolded portion in point 3 above.
This ticket is requesting the addition of a new API to be able to 'zip'/'join'/'attach' metadata from 2 files.
For e.g.
import pyarrow.parquet as pq
m1 = pq.read_metadata('file1.parquet')  # say this has columns: col1, col2, col3
m1.set_file_path('file1.parquet')

m2 = pq.read_metadata('file2.parquet')  # say this has columns: col4, col5
m2.set_file_path('file2.parquet')

# requesting this new 'zip' API
m = m1.zip(m2)  # needs to ensure same number of row groups, and same number of rows within each row group

# m will now have metadata for col1, col2, col3, col4, col5 each pointing to appropriate data file

m.write_metadata('_metadata')

One this is done, a combined data can be created using:
m = pq.read_metadata('_metadata')
data = pq.ParquetFile('file1.parquet', 'file2.parquet', metadata=m)

# data should now be able to show all columns

Component(s)
C++, Python, Parquet

@wgtmac does spec allowing this currently?

I think this is the parquet summary metadata file. See parquet.summary.metadata.level from  https://github.com/apache/parquet-mr/blob/master/parquet-hadoop/README.md#class-parquetoutputformat. But I don't know whether it is widely used.

@wgtmac, no I don't think _metadata file would be widely used in the bigdata systems like Hadoop/Spark etc. However, with Apache Arrow it does seem to have the required API (in ParquetFile) to read metadata separately from the data.
Of course, I'm also not sure if Apache Arrow will also specifically support the below section from my request (because we currently have no way to stitch 2 metadata files):

One this is done, a combined data can be created using:
m = pq.read_metadata('_metadata')
data = pq.ParquetFile('file1.parquet', 'file2.parquet', metadata=m)

# data should now be able to show all columns

where file1.parquet contains col1, col2, co3 and file2.parquet contains col4 and co5 (different set of columns). Here only the the _metadata file has the overarching information about the 'table' definition.
I'm only guessing that it would be supported, since it has the API to do so. However, it would be nice to confirm that as well.

Before talking about the zip API, it still need some refactoring work to support the metadata file. What is your use case then? The possible use case for metadata file is to combine parquet files of different columns into a larger logical parquet file. From my perspective, if metadata file is not widely used, it seems not worth the effort to implement it.

@wgtmac, sorry I didn't quite understand

it still need some refactoring work to support the metadata file

pyarrow already supports metadata file, right?
https://arrow.apache.org/docs/python/parquet.html#writing-metadata-and-common-metadata-files

Awkward. I even didn't notice that this is already supported. Thanks for pointing it out!

@wgtmac, any further insights into this.
I think the only place that would need to change is https://github.com/apache/arrow/blob/main/cpp/src/parquet/metadata.cc

@mrbrahman This feature is controversial and recently there is a related discussion: https://github.com/apache/parquet-format/pull/242#discussion_r1603234502.


refactor(projectrc): schema changes

Current dependencies on/for this PR:

main

PR #27   👈

This stack of pull requests is managed by Graphite.

Merge activity

Dec 7, 11:36 PM: @@luxass started a stack merge that includes this pull request via Graphite.


Remove deprecated taint

This taint should match the new role name "control-plane"
Kubeadm will remove this with 1.25. It's a bit of an awkward timing, because that looks like it will come out at the end of this month

Good catch!