Fix ApiAi Middleware

Why do we need the version parameter?

The mandatory version must be submitted in the request, according to the documentation
Also, if you do not send the version in the request, you do not send the data about the completion of the action

https://docs.api.ai/docs/query#post-query

@mpociot If we briefly explain, then without passing a version parameter, the action Incomplete parameter is not passed in response to the query

Codecov Report

Merging #285 into master will not change coverage.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master     #285   +/-   ##
=========================================
  Coverage     83.94%   83.94%
  Complexity      636      636
=========================================
  Files            53       53
  Lines          1912     1912
=========================================
  Hits           1605     1605
  Misses          307      307

Impacted Files
Coverage Δ
Complexity Δ

src/Mpociot/BotMan/Middleware/ApiAi.php
97.61% <100%> (ø)
13 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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

@mpociot You can take pr?


[Updates] Automated vendor dotnet-install script

[Updates] Automated vendor dotnet-install script

I think this is pretty cool 😎


upload distribution directory directly to bucket

Hi, I have a React.JS project I'm trying to deploy to a domain-named bucket.
the example code has no 'dist' directory, but 'dist/' is where the post-compilation code is dumped, i tried using 'destinationNameFn' as mentioned below to change the output directory path (as below) but that doesn't work.
it just creates a symlink to $BUCKET/dist/..  and then  leaves $BUCKET/ empty
:|
        new WebpackGoogleCloudStoragePlugin({
              directory: './dist',
              // NOTE: Array of filenames to include in the uploading process
              include: ['.js', '.html', '.png', '.eot', '.ttf', '.woff', '.woff2', '.svg'],
              // NOTE: Array of filenames to exclude in the uploading process
              exclude: ['.js.map'],
              // NOTE: Options passed directly to
              // Google cloud Node Storage client.
              // This is mostly authentication-wise.
              // For more information:
              // https://github.com/GoogleCloudPlatform/google-cloud-node/tree/master/packages/storage#authentication
              storageOptions: {
                  projectId: 'my-project-id-12345',
                  keyFilename: './config/keyfile.json',
              },
              // NOTE: Options used by
              uploadOptions: {
                  // Where to upload files
                  bucketName: 'example.domain.com',
                  // NOTE: Currently not tested and most likely not working.
                  forceCreateBucket: false,
                  destinationNameFn: file => path.join('../', file.path),
              },
        })

this worked:   destinationNameFn: file => file.path.replace('dist/', ''),


[IOTDB-6096] Make M4 Function do the nullable judgement

Can use the data.zip attanchment in this issue to replay.
The root cause is that we don't consider the nullable value in RowWindow.

coverage: 48.008% (-0.02%) from 48.025% when pulling bb20b02d348ab33d4b5feae60c0c033a969ba2a4 on IOTDB-6096 into e9b9b63a95719c14c674258d92eabf36a0bb69d1 on master.


Version 1.0.0

Graylog2 Version 1.0.0 doesn't appear to recognize this plugin.  I placed it in the plugin directory, restarted the server, and it doesn't detect it.  Other plugins in the same directory work.  Do you plan on making this 1.0.0 compatible?
Thanks in advance :)

This was actually one of my first plugins for Graylog (then in v0.92). I will check to have it updated to the new plugin API. Ronald

Works just fine :) Thanks.
On Wed, Feb 25, 2015 at 7:49 AM, d-fens GmbH<EMAIL_ADDRESS>wrote:

Closed #2
https://github.com/dfch/biz.dfch.j.graylog2.plugin.output.execscript/issues/2
via b490b2e
https://github.com/dfch/biz.dfch.j.graylog2.plugin.output.execscript/commit/b490b2eaa61f53fbf2f7f3e7097537d72bbca3e0
.
—
Reply to this email directly or view it on GitHub
https://github.com/dfch/biz.dfch.j.graylog2.plugin.output.execscript/issues/2#event-241697873
.


Typo fixes

Also in doc/basic.md:

Line 254: should it be "16/31 CAN 4"?
Line 263: missing word, stray space, or stray comma in interface ,?


[CLI] wrong version published to npm

Describe the bug
the version on npm is listed as 0.6.0, but it's actually 0.5.0

To Reproduce
Steps to reproduce the behavior:

npm i -g<EMAIL_ADDRESS>squoosh-cli --version

Expected behavior
outputs 0.6.0
Version:

OS w/ version: Ubuntu 16.04.7 LTS
Node version: 15.4.0
npm version: 7.0.15

Published v0.6.1 :)

Published v0.6.1 :)


feat(frontend): Set up Btc Transactions page

Motivation
The transactions page for Bitcoin was not showing balance, nor nice background.
The error was that it was using the ETH transactions page and that the hero didn't have an assigned background for btc.
Changes

Create new component BtcTransactions.
Use new component in TransactionsSignedIn.
Add a background when the hero is for "btc".

Tests
Tested locally.

UI:

New UI:


Added the support for the replace_string function

Replaces all string matches with a specified string.
Changelog category (leave one):

New Feature

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Changes to support the KQL replace_string function.
Documentation entry for user-facing changes

[ ] Documentation is written (mandatory for new features)

Information about CI checks: https://clickhouse.com/docs/en/development/continuous-integration/

wrong commits. Closing this pull request.


Edge raw image cannot boot after upgrade on RHEL9.2

Issue description:
Within an update snapshot PR https://github.com/osbuild/osbuild-composer/pull/3215, ostree-raw-image.sh failed at step upgrade, vm cannot boot after upgrade, details can be found at https://gitlab.com/redhat/services/products/image-builder/ci/osbuild-composer/-/jobs/3609173220

Setup a local test env with same setting and tried to debug it:

tried to manually do upgrade, vm can boot after upgrade.
tried to run ostree-raw-image.sh, no errors found.

CI job passed without any change, closing this bug.


Wrap exprs in statements so they won't run together

Resolves https://github.com/krzkaczor/babel-plugin-tailcall-optimization/issues/7

Thanks for contribution!

Will you be publishing a new release soon?

It was already published :) (I just forgot push tags to this repo)


Upgrade smooth-scroll to latest version and switch to jsDelivr CDN.

The version of smooth-scroll we were using was quite old (v7.1.1) compared to what is now available (v12).  By upgrading to the latest version, and slightly tweaking the way we instantiate it to support the new construct, this resolves the issue with anchor tags with periods being incorrectly escaped.
Fixes https://github.com/meteor/hexo-theme-meteor/issues/57.

It appears to be fixed based on the Netlify previews which clearly reproduced the problem thanks to the additional test content I added to the test theme in https://github.com/meteor/theme-example/pull/8.
At some point, I'd love to write tests for stuff like this using something like Puppeteer and this guide.
Thanks to @glasser for reporting this originally.


creating new nodes in grid

Hi Deitch and thanks for a great plugin!
This is a change request and not an issue. Hope this is the right place to put it.
When using the jstree contextmenu plugin, can you also add the possibility to enter data in the grid?
Currently, you can only edit the jstree section and it would be fantastic to be able to edit e.g. new records directly in the tree and submit through ajax.
Here is the example screenshot:

I believe there is also a discussion here  about the topic. I'm afraid I cannot help with the implementation.
Thanks!

Hi! Does this trigger any event? How does one know to save the changed data?

Hmm, no, it does not. It just sets the value here https://github.com/deitch/jstree-grid/blob/master/jstreegrid.js#L491
It probably should trigger an event. The closest that appears to exist in the jstree event system is set_text.jstree, but I don't think that fits, as it is only the text of the first column. Perhaps update_cell.jstree?

Based on the documentation I'm not sure if the changed node is passed to set_text (it's referring to 'object' whereas elsewhere 'node' is mentioned), but if it gets the node, it would also be fine since one could implement a function to save all the node data whenever the name or other data is changed..

Let me chime in - I believe update_cell.jstree would be better, as set_text.jstree is a separate event and firing that may cause confusion (especially since it is "internal" so to speak, so listening for it is not a good idea).

All the better if information of the changed data is passed to the event listener (update_cell)
-> would be very easy to just send the changed information..


Scrollbar not auto adjusting in quick conversation

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open any old/new chat
Send a few messages with a time lag between each
Now send a bunch of more messages with reduced time lag(send messages quickly)

Expected Result:
Scrollbar should stick to bottom and its size should shrink as the chat grows
Actual Result:
Scrollbar sticks to bottom but its size increases abruptly in the middle and user fails to scroll properly from that moment
(NOTE: I can reproduce the scrollbar size increase but no issue in scrolling)
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: v1.2.98-2
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/43996225/231290257-e30245b5-9cba-43b6-8b4e-fca704336e92.mp4
https://user-images.githubusercontent.com/43996225/231290260-781df0f0-93a6-49ea-992d-f5c2e3e3ab7f.mp4
https://user-images.githubusercontent.com/43996225/231290282-9a798dcb-4d37-4c61-b1fa-2c954352ee7d.mp4
Expensify/Expensify Issue URL:
Issue reported by: @chiragxarora
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1681216474300829
View all open jobs on GitHub

Triggered auto assignment to @tjferriss (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

The scroll bar doesn't automatically appear for me like in your examples. However, I am not able to scroll back up reliably. I basically get stuck and can't scroll up after a certain point. @kavimuru do you know why I don't get the scrollbar automatically?
https://user-images.githubusercontent.com/8292562/231352415-99cb5f39-f42d-45cf-9572-c885a834175b.mov

Did you try reproducing this in production? Trying on the right version?

📣 @chiragxarora! 📣
Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:

Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.

Format:
Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~014a1b566a7d8ee0c5

✅ Contributor details stored successfully. Thank you for contributing to Expensify!

@tjferriss Eep! 4 days overdue now. Issues have feelings too...

Still discussing...

Hi @tjferriss
It's present in windows OS, still reproducible, in production
Also, your media isn't visible
https://user-images.githubusercontent.com/54641656/233168397-64294072-3cfb-4c60-8d10-4a96738df829.mp4

Job added to Upwork: https://www.upwork.com/jobs/~017736baa7a774ed05

Current assignee @tjferriss is eligible for the External assigner, not assigning anyone new.

Triggered auto assignment to Contributor-plus team member for initial proposal review - @mollfpr (External)

Triggered auto assignment to @hayata-suenaga (External), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

@tjferriss is this issue supposed to be open? were you able to reproduce the issue?

I can reproduce it in Mac Chrome. The scrollbar is jumping and then stuck.
https://user-images.githubusercontent.com/25520267/235035889-45760674-346d-4ef4-bfb0-1f0f767f917a.mov

This is supposed to be open @hayata-suenaga

@hayata-suenaga yes, I jumped the gun on the external tag. I'm not sure if you can reproduce this one. I can't, but I also don't have access to a Windows OS. I asked about this in the #bug-zero room, but never got a response. Are you able to reproduce?

I'll try reproducing this today

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

I got the same issue @tjferriss faced here. Cannot scroll up to the stop.

@tjferriss @mollfpr @hayata-suenaga this issue is now 3 weeks old. There is one more week left before this issue breaks WAQ and will need to go internal. What needs to happen to get a PR in review this week? Please create a thread in #expensify-open-source to discuss. Thanks!

@tjferriss @mollfpr @hayata-suenaga this issue is now 3 weeks old. There is one more week left before this issue breaks WAQ and will need to go internal. What needs to happen to get a PR in review this week? Please create a thread in #expensify-open-source to discuss. Thanks!

@hayata-suenaga and @mollfpr so I think we all agree there is some type of problem with the scrollbar feature in the chat. Do we have enough information to look for proposals?

The issue in OP and the issue I and @tjferriss observed are probably related. @mollfpr could you update the OP to include the new issue, too?
we can let contributors figure out the root cause in their propposals.

@mollfpr could you update the OP to include the new issue, too?

@hayata-suenaga I can't 😅

@hayata-suenaga and @mollfpr so I think we all agree there is some type of problem with the scrollbar feature in the chat. Do we have enough information to look for proposals?

@tjferriss  Yes! As suggested @hayata-suenaga we should update the OP.

@mollfpr updated the OP

asked help in #expensify-open-source channel

This is a dupe of this issue https://github.com/Expensify/App/issues/15116, which is closed in favor of https://github.com/Expensify/App/issues/12054#issuecomment-1505811916


fix: ADDON-66599 deprecate splunk-hec-token default value and break test execution on exception

Changes:

deprecate default value of splunk-hec-token - give a warning when it's used
add console handler to logger - warnings and errors beside being logged to pytest_splunk_addon.log are also logged to console
other minor refactors from PR-803

Already released on develop and tested on couple of repos.
https://github.com/splunk/splunk-add-on-for-google-workspace/pull/515
References:
https://splunk.atlassian.net/browse/ADDON-66599
https://github.com/splunk/pytest-splunk-addon/pull/803/files

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


glitch demo is not working

Page Affected: https://developers.google.com/web/tools/chrome-devtools/console/log
What needs to be done?
Glitch demo is not working.

Fixed


Closing the response's connection does not prevent the body from being downloaded

Hello,
I wanted to see whether it is possible to cancel a client-side connection before the body is downloaded (i.e to prevent big downloads based on the Content-Type header). I found in the aiohttp docs' comments this gist which claims to be able to prevent the download of the body by calling .close() on the response.
I tested the code below that is similar to the gist (mine just closes the connection) and fired up Wireshark to inspect the packets. I saw that even though the response was closed the body has been downloaded.
Is this the correct behaviour? Otherwise is there a way to prevent the download of the body ?
Many thanks,
JB
import asyncio
import aiohttp

async def get_file(session, url):
    async with session.get(url) as response:
        response.close()

loop = asyncio.get_event_loop()
with aiohttp.ClientSession() as session:
        loop.run_until_complete(get_file(session, 'http://www.thinkbroadband.com/download.html'))

server sends payload unconditionally, also your incoming kernel buffer probably large enough to receive whole payload.
you should test it on large file

That makes sense, thank you!


Reallocate interoperability functionality

Reallocating interoperability functionality to TaijaInteroperability.jl package.
Closes #329.

Pipeline for Julia 1.8 and nightly is failing due to unrelated reasons (it also fails on the main branch) but @pat-alt do you know what might be causing the CI / Documentation (pull request) pipeline's fail?

Pipeline for Julia 1.8 and nightly is failing due to unrelated reasons (it also fails on the main branch) but @pat-alt do you know what might be causing the CI / Documentation (pull request) pipeline's fail?

This is weird, it doesn't give me any information other than 'job failed'

I've just triggered a rerun now @kmariuszk, because tests for docs are passing in other places (e.g. https://github.com/JuliaTrustworthyAI/CounterfactualExplanations.jl/pull/357)

Hey @pat-alt, I updated the documentation accordingly and wanted to double-check if everything works by re-rendering it (even though the "Documentation" pipeline is passing). Could you please help me with it? It could be either locally or in the repo.

Happy to merge, just restating my stupid questions from teams here: do you guys know if you can safely merge PR that have unmerged commits from main? Otherwise we'd have to now update each of the PRs after we've merged another PR

If I understand you correctly, it's completely fine to do so if there's no conflicting files. If there are, git is going to complain about this and find the merge conflicts. So, in that sense, it's always safe to merge such PR.


i18n not working?

I follow the README steps to create a new site, and changed default language to "zh-cn":
defaultContentLanguage = "zh-cn"           # Default language to use
[languages.en]
  languageCode = "en"
[languages.zh-cn]
  languageCode = "zh-cn"

Hugo version

Hugo Static Site Generator v0.54.0/extended linux/amd64 BuildDate: unknown

You can check the https://github.com/xianmin/hugo-theme-jane/blob/master/dev-config.toml . This is demo site config.
Maybe you miss the showMenuLanguageChooser ?
https://github.com/xianmin/hugo-theme-jane/blob/d8cd67c83dfe038b4041e9a0ce9db8f338994279/dev-config.toml#L133-L134

Thanks, dev-config.toml works fine, but the exampleSite/config.toml not works.


Vehicles() call should return info on all vehicles

Consider adding vehicle(index) and vehicles().
NB: API breaking change

Perhaps rather than a breaking change we could create a new API along the lines of allVehicles() and allVehiclesAsync() which returned an array of the vehicles for the user.

Or perhaps in 2.x we add vehicle() which is the same as today's vehicles() and allVehicles().  And then for 3.0 we simply rename allVehicles() to vehicles() as a single breaking change.

OK, for 2.1.0 added vehicle() and allVehicles() to allow enumerating multiple vehicles.  For 3.x will plan to break the API and rationalize.

In V3 branch

V3 got too stale and had to be deleted.  Re-opening this issue so we can start from the current main since we have a version breaking bug that needs to be fixed.

rename vehicles() to vehicle() and allVehicles() to vehicles() to address


Error when running on octave

Hi,
I was able to compile and run the code under octave. Nevertheless, I have an exception raising for each model and crashing the execution with the following trace:
Treating Raw_Data as an Adj Matrix
h = -6.3352
W_Distr set to Normal (default)
E_Distr set to Bernoulli (default)
error: Raw Data is not an m by 3 Edge List\n
error: called from
    wsbm_driver>setup_Data at line 470 column 13
    wsbm_driver>format_input at line 163 column 9
    wsbm_driver at line 29 column 1
    wsbm at line 123 column 11
    WSBMDemo at line 24 column 8

Before the crash, suspicious values are the m_i_k_dot_dot(:,i) that has zero everywhere and  especially p_ik_prime_one_minus that has NaN everywhere.
Do you have any idea of what is going on how we can solve this ?

Hi, sorry that the code was only tested under Matlab and not guaranteed to run under octave. There might be compatibility issue due to the use of mex c files to speed things up.


Fixed issue with master when parsing neutron data

Fixed https://github.com/mantidproject/fitbenchmarking/blob/ee3307bd8f3e9210074b55765b44f45dab591f42/fitbenchmarking/input_parsing.py#L249
This line was causing an error: global name 'k' is not defined
Change k to sep_idx

just tested it and it seemed to run fine, I did get an error about the summary tables but we are aware of that bug.


large data to river have some problem

I having large data to river, but this plugin just store data in one index.
How can i dynamic  create index like logstah(a day as a index)?
think very much for my question.

You could achieve this with your scripting:
echo '{
"type" : "jdbc",
"jdbc" : {
"url" : "jdbc:user",
"user" : "user",
"password" : "pass",
"sql" : "select , id as _id from table",
"elasticsearch" : {
"cluster" : "offline_payments",
"host" : "localhost",
"port" : 9300
},
"index" : "index.DATE",
"index_settings" : {
"index" : {
"number_of_shards" : 1
}
},
"max_retries": 3,
"max_retries_wait" : "10s"
}
}' | java
-cp "lib/"
org.xbib.tools.Runner
org.xbib.tools.JDBCImporter
On your .sh; you could play with the DATE variable.


Update m365bp-collaborate-share-securely.md

typo

Docs Build status updates of commit 3b04df7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/business-premium/m365bp-collaborate-share-securely.md
:white_check_mark:Succeeded

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


Youtube playlist in the autoplaylist.txt

Hi,
first of all great work with the bot, works perfect on our servers.
Now to my question is it possible to insert in the autoplaylist.txt just a single youtube link to a playlist where others can contribute and attach to the playlist, and let the bot just restart at the end the playlist again?
If not how could be an easy workaround look like to achieve that, letting others add songs to the playlist?
Thx for the help already, and Merry Christmas.
Our setup: the bot is running on a centos azur server.

No, you can only use single song links right now. Use !pldump  to get a file with every YouTube link in a playlist to put in autoplaylist.txt

Ok thanks for the fast respond and help.
Maybe as a coming feature make the !pldump command write directly in the autoplaylist.txt and let it add the songs to the playlist if they aren't already inside.


🚀 Feature: pagerduty - ability to configure a homepage plugin

🔖 Feature description
It would be great if there was an ability to configure the pagerduty plugin on the backstage homepage. Instead of taking the integration key from an entity's annotation it could be passed in via a component prop.
🎤 Context
In some cases, companies have 1 on-call rota across the whole organization, so it makes more sense for the plugin to appear on the homepage rather than on the component page.
✌ Possible Implementation
No response
👀 Have you spent some time to check if this feature request has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
None

@punke this can be closed right?


feat(host): Initialize Binary Crate

Description
Simple ticket to setup the basic host binary.
Should just be able to run cargo init --bin . inside the bin/host/ directory.

Closed by #143


Files open slowly in server mode

Mathlib contains around 3k files.  I have made a small stress test to see how long it takes to open a file in the editor with that project size (I was generous, and only generated 1.6k files):
$ time lake print-paths Inundation
{"srcPath":["./."],"oleanPath":["./build/lib"],"loadDynlibPaths":[]}
8.58user 0.89system 0:08.09elapsed 117%CPU (0avgtext+0avgdata 388316maxresident)k
0inputs+25616outputs (0major+24645minor)pagefaults 0swaps

That is, Lake by itself takes more than 8 seconds to find the dependencies and check that they're up to date.  This is before Lean can even start to import the oleans and initialize the environment.  So a user would need to wait around 10 seconds after opening a file before Lean starts processing the file.
It takes around 8 milliseconds to compute SHA512 hashes for all of the files in that repo.  I think that time is a good target for lake print-paths as well.

More comparison (not meant to be a competition, or only a friendly one :) ):
# run Nix' fake lake wrapper after full build
$ nix shell .#lean-dev -c time lake print-paths Inundation
Building dependencies...
{"loadDynlibPaths":[],"oleanPath":["/nix/store/ah8wdmw66asczlcs7avj8wvr3571pw61-print-paths-depRoot"],"srcPath":[".","/nix/store/j37b48sd21xnp9zcv0lhrcmjz0kmsxs1-src","/nix/store/j37b48sd21xnp9zcv0lhrcmjz0kmsxs1-src"]}
0.03user 0.02system 0:00.59elapsed 10%CPU (0avgtext+0avgdata 27332maxresident)k
0inputs+0outputs (0major+3302minor)pagefaults 0swaps

# change unrelated file, forcing Nix' eval cache to miss
$ echo >> README.md; nix shell .#lean-dev -c time lake print-paths Inundation
warning: Git tree '/home/sebastian/Downloads/inundation' is dirty
Building dependencies...
building '/nix/store/47fhdv0igi0l5cqh1z5jms00x3s1ijxb-Inundation-deps.json.drv'...
{"loadDynlibPaths":[],"oleanPath":["/nix/store/8b2ywafx5q4mqc2f6mm6s3hapcl2q2hh-print-paths-depRoot"],"srcPath":[".","/nix/store/j37b48sd21xnp9zcv0lhrcmjz0kmsxs1-src","/nix/store/j37b48sd21xnp9zcv0lhrcmjz0kmsxs1-src"]}
2.41user 0.83system 0:06.67elapsed 48%CPU (0avgtext+0avgdata 254512maxresident)k
0inputs+624outputs (0major+63634minor)pagefaults 0swaps

# invalidate one file, which makes Nix recompute all dependencies in the package
$ echo >> Inundation.lean; nix shell .#lean-dev -c time lake print-paths Inundation
Building dependencies...
building '/nix/store/mwjqg5ng34qdng6kwg31shf6wfqbam8x-Inundation-deps.json.drv'...
these 3 derivations will be built:
  /nix/store/jqy3xj3nf22hxi9mwwqqxznsb6rp4z99-Inundation.drv
  /nix/store/irisrcgkcykglqbwy1fmym3chl19chj7-print-paths-depRoot.drv
  /nix/store/a370xhfswiayk659jbpjz702ffqgg933-print-paths.drv
building '/nix/store/jqy3xj3nf22hxi9mwwqqxznsb6rp4z99-Inundation.drv'...
building '/nix/store/irisrcgkcykglqbwy1fmym3chl19chj7-print-paths-depRoot.drv'...
building '/nix/store/a370xhfswiayk659jbpjz702ffqgg933-print-paths.drv'...
{"loadDynlibPaths":[],"oleanPath":["/nix/store/8b2ywafx5q4mqc2f6mm6s3hapcl2q2hh-print-paths-depRoot"],"srcPath":[".","/nix/store/j37b48sd21xnp9zcv0lhrcmjz0kmsxs1-src","/nix/store/j37b48sd21xnp9zcv0lhrcmjz0kmsxs1-src"]}
2.07user 0.68system 0:07.33elapsed 37%CPU (0avgtext+0avgdata 254876maxresident)k
0inputs+592outputs (0major+63646minor)pagefaults 0swaps

So not exactly better except in the ideal case :thinking: ...

Thanks! This is very interesting.  So Nix's advantage is really mostly due to the flake eval cache!
For comparison, could you also post how long lake print-paths Inundation takes on your machine?

@Kha So, according to @gebner's results, this implies that Lake has a 3-4x overhead over Nix (w/o the cache). How much of that do you think is likely just do to the differences in language and codegen of Lean?

this implies that Lake has a 3-4x overhead over Nix (w/o the cache). How much of that do you think is likely just due to the differences in language and codegen of Lean?

I'm not sure where you see the 3-4x overhead here, both Lake and Nix (without eval cache) take around 8 wallclock seconds for a build.  (Note that Nix has a daemon, so don't read too much into the user time.)
For Lake, I think the (lack of) performance has nothing to do with the Lean compiler or using Lean as the programming language.  Most of the issues seem to be architectural or algorithmic.  See for example the accidentally quadratic computation of imports fixed by #132.

@gebner

I'm not sure where you see the 3-4x overhead here, both Lake and Nix (without eval cache) take around 8 wallclock seconds for a build. (Note that Nix has a daemon, so don't read too much into the user time.)

Most of the issues seem to be architectural or algorithmic.

Yeah, I definitely know there are no doubt problems with that. My originally question (which is now essentially moot due to actual difference being smaller than I though) was simply what percent of overhead, if any, should one expect from code written in Lean versus something written in a language like Nix (or vice versa).

My originally question (which is now essentially moot due to actual difference being smaller than I though) was simply what percent of overhead, if any, should one expect from code written in Lean versus something written in a language like Nix (or vice versa).

Your question is moot, but for very different reasons.  Namely that Nix comes built-in with a very different architecture.  At a very high level, you send shell scripts together with a list of dependencies to the Nix daemon.  That daemon will run the shell script in a sandbox with only the dependencies visible, but only the first time.  The result of the shell script is identified by a hash of the shell scripts and the list of dependencies.  Sebastian's Nix integration essentially produces one shell script per olean/o/etc. file.  From what I can tell, a lot of the overhead comes from producing the shell scripts, spawning a sandbox for every program call, copying the source code into the store, and general bookkeeping (Nix uses an actual SQL database to keep track of the build results).  None of which has anything to do with Nix the language.
On a language level, Lean is lightyears ahead of Nix.  (Unless you really want to do lazily evaluated dictionaries that cyclically depend on themselves.)

After #132, the biggest cost centers are Lean.Elab.parseImports (22%), computeFileHash (17%), and loading the lakefile (14%).

Oooh, there are some low-hanging fruit in Lean.Elab.parseImports.  We actually initialize a new empty environment every single time we parse the imports....

@gebner

After #132, the biggest cost centers are Lean.Elab.parseImports (22%)

Oh I didn't realize parseImports was that slow. If that remains the case, making it into an asynchronous job may help save time in Lake (as we can parse the imports of multiple files simultanously).

@Kha I believe you were the one who setup benchmarking on the Lean 4 repository. Any chance you can advise me how I could do the same here on Lake? It would be able to use automagically use performance tests like @gebner's to benchmark PRs, commits, and the like.

For comparison, could you also post how long lake print-paths Inundation takes on your machine?

Right, it's 10s for Lake (without your change). It's a laptop.

@Kha I believe you were the one who setup benchmarking on the Lean 4 repository. Any chance you can advise me how I could do the same here on Lake?

Unfortunately the setup is a bit bespoke, I think we need a separate benchmarking machine per repo... it's also not clear where the infrastructure will go after I finish my PhD.

@Kha ah 😞 -- do you have any advice on alternative solutions for GitHub benchmarking then? -- a quick search turned up the Continuous Benchmark action, but I, unfortunately, do not have much experience on what to look for in this area.

As far as watching the benchmark results of examples in this repository, the amplitude of the benchmarks is about +- 10~20%.

That does not bode well for benchmarking on GitHub Actions

After #132, the biggest cost centers are Lean.Elab.parseImports (22%), computeFileHash (17%), and loading the lakefile (14%).

With the old addition of parseImports', and the new file hash caching (lean4#2444) and precompiled oleans (lean4#2480), all these major cost centers should be addressed. The Inundation benchmark will also be in lean4#2457. Thus, I am closing this issue.

What are the performance numbers and breakdown now?

@digama0 As the benchmark machine is not Gabriel's or Sebastian's, we do not have exactly comparable benchmarks. However, this analysis compares the benchmarks before and after the new changes (where lake build dummy is the Inundation build). I currently do not use Inundation for the no-op build, because it is not as useful as a no-op Lean build (because the modules in Inundation do not have real code and thus things like the hashing optimization do not produce significant results). I also suspect that the crlf2lf change already substantially reduced the hashing time.

@digama0 It is also worth noting that hashing optimization only significantly impacted the task-clock, not the wall-clock time. Lake is already very efficient at hashing files with its massive parallelism as Sebastian's Hotspot analysis showed. However, on machines with few threads/cores it is a major benefit. It may also help downstream project's like cache use the hashes rather than hashing themselves.


[Feature Request] Add LICENSE.md file

I know in your package.json you have the license listed as MIT, I was wondering if you could add an actual LICENSE.md file with the license?

Yeah. I can certainly add this. I want to make a CHANGELOG.md as well.

Dine


Collapse is inaccessible by keyboard if all panels are closed by default

There's a flaw in the logic of the collapse extension: it assumes that one of the collapsible panels is open by default (the .in class), and sets the tabindex of that panel's header to 0 so that it can be reached via keyboard. The tabindex for each of the remaining panel headers is then set to -1:
if(collpanel.hasClass('in')){
    colltab.attr({ 'aria-controls': collpanel.attr('id'), 'aria-selected':'true', 'aria-expanded':'true', 'tabindex':'0' })
    collpanel.attr({ 'role':'tabpanel', 'tabindex':'0', 'aria-labelledby':collid, 'aria-hidden':'false' })
}else{
    colltab.attr({'aria-controls' : collpanel.attr('id'), 'tabindex':'-1' })
    collpanel.attr({ 'role':'tabpanel', 'tabindex':'-1', 'aria-labelledby':collid, 'aria-hidden':'true' })
}

This doesn't work if all of the panels are collapsed by default, because then the tabindex of all the panel headers is set to -1, which makes the entire panel group inaccessible via keyboard.
To fix this, I simply set the tabindex on the headers of the collapsed panels to 0 instead of -1:
colltab.attr({'aria-controls' : collpanel.attr('id'), 'tabindex':'0' })

There don't appear to be any side effects of doing this, other than that I can now hit Tab to move between different panel headers in addition to Left/Right or Up/Down. That's fine with me, and certainly better than the entire panel group being inaccessible!

Even if all panels are collapsed, user can still tab to first tab header and navigate with arrow keys.

No, they can't - that's the bug. They should be able to tab to the first header, whether or not the panel is collapsed. But it doesn't work because when all of the panels are collapsed, none of them have the .in class, so the tabindex for each one gets set to -1.

Please send a PR or please use the latest BS4 https://getbootstrap.com/docs/4.1/components/collapse/ .


Have names of class attributes created during end with a "_"?

Is your feature request related to a problem? Please describe.
I'm always frustrated when I run a sinkhorn solver ot = sinkhorn.solve(geometry) and the attributes of ot created during the solve do not end with a "_", like the sklearn convention (maybe a bad idea for this package), for instance ot.f_ and ot.g_.
Describe the solution you'd like
Have all "fitted" attributes end with "_"

I'm against this change, simply because I'm not aware of any other jax-based library that does this and don't think it really fits that well.

I think the logic here is that jax objects are immutable, so strictly speaking we don't really "update" the values of f and g of an existing SinkhornOutput object, but rather instantiate it from scratch once the solver has finished


Support VK_EXT_rasterization_order_attachment_access

This change adds support for the VK_EXT_rasterization_order_attachment_access extension. This extension basically adds just a few flags to obviate the need for a subpass self-dependency, and is used to implement behavior similar to coherent framebuffer fetch in GLES.

This implementation is incomplete, you are not correctly handling the features struct and without that this cannot be sufficient to validly capture any program using the extension.

Ugh sorry, totally forgot. Done.

This should also have the feature checks added into the others in Serialise_vkCreateDevice (CHECK_PHYS_EXT_FEATURE etc) to make sure the features used in the capture are available on replay and gracefully fail to open the capture. If you look at previous commits that add extension support you'll see the same pattern.
Has this change been tested on an application using the extension, or is this implemented purely against the spec? The original version shouldn't have worked at runtime and some of the force-pushed updates don't seem like they would have compiled since they had duplicate case statements.

Yes actually, I did try the original version and I could capture/replay just fine. The symptoms of the replay being done wrong is synchronization errors, but syncval doesn't support this ext yet anyway (and generates the same synchronization errors either way), which is probably why I didn't notice the ext is not really working in replay.
The intermediate force pushes not building was just myself being hasty with a push :-S

I was following VK_EXT_primitives_generated_query, but looks like I missed the changes in Serialise_vkCreateDevice for that extension too, I'll fix both.

I think I got them all now


Fix the Edit and Reply cancel methods on Comments component in example-forum package

I noticed that the "Edit" and "Reply" cancel methods in CommentsItem.jsx file are not working. For example, when I edit a comment, the "Cancel" option is not working. Likewise with the Reply form.
I confirmed with @justinr1234 for this fix and he said that this is okay because "... SmartForms code already prevents default before calling this callback."
For more details, please see issue reference: https://github.com/VulcanJS/Vulcan/issues/1870
Thanks!

Great! But can you make a PR with just the CommentsItem.jsx changes (and not versions)?
Also, just fyi I will move all example code (including the Forum example) to the starter repo soon (https://github.com/VulcanJS/Vulcan-Starter/)


superfluous xtime

Thank you very much for pointing out this issue.
Implementing this change will actually increase code size with my compilers, for both ARM and x86_64.
Removing the last call to xtime() seem to spoil some vectorization-optimization that the compiler performs.

I'd say this would be worthy of a comment in the code, that the last line is redundant and there for compiler optimization.

Nice! Thanks.

This can be closed, can it not @kokke?

Hi @Legorooj
Yes this issue can be closed.
There is a comment in the code, pointing to this issue - thanks for pointing it out :)

@kokke you need to do it :-) only repo owners and people you've given permission to can close PRs/Issues


Prop E artifact annotations questions

Annotations
These annotations would be added for artifacts:

org.opencontainers.artifact.type: type of artifact (sig, sbom, etc)
org.opencontainers.artifact.description: human readable description for the artifact
org.opencontainers.artifact.created: creation time for a manifest

Are these annotations required for the new artifact type?
Are these annotations valid to use on an image manifest?

The spec would be updated to define the annotations. Individual artifacts may use them, and I think we would push for an artifact should specify a type, but I stopped short of saying an artifact must use any of these annotations.

Closing as PRs are open in upstream repos


[SPARK-32364][SQL][FOLLOWUP] Fix CaseInsensitiveMap usage for Scala 2.13

What changes were proposed in this pull request?
This is a follow-up of #29160. This allows Spark SQL project to compile for Scala 2.13.
Why are the changes needed?
It's needed for #28545
Does this PR introduce any user-facing change?
No
How was this patch tested?
I compiled with Scala 2.13. It fails in Spark REPL project, which will be fixed by #28545

Can one of the admins verify this patch?

Can one of the admins verify this patch?

ok to test

How about filing a new jira ticket as a sub-ticket of https://issues.apache.org/jira/browse/SPARK-25075 ?

Test build #128046 has started for PR 29584 at commit 02e9e2b.

Test build #128046 has finished for PR 29584 at commit 02e9e2b.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Merged build finished. Test PASSed.

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

+1 for @maropu 's advice.

How about filing a new jira ticket as a sub-ticket of https://issues.apache.org/jira/browse/SPARK-25075 ?

Ah, one more comment; please fill the PR description about what kind of compilers errors happens. I think that the information is useful to developers when writing code with scala v2.13.

Test build #128107 has started for PR 29584 at commit c5228e1.

Test build #128107 has finished for PR 29584 at commit c5228e1.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Merged build finished. Test PASSed.

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

How about filing a new jira ticket as a sub-ticket of https://issues.apache.org/jira/browse/SPARK-25075 ?

Done: https://issues.apache.org/jira/browse/SPARK-32756

Thanks, @karolchmist .

Ah, it's just complaining about whitespace:
[error] /home/jenkins/workspace/SparkPullRequestBuilder@3/sql/catalyst/src/main/scala-2.12/org/apache/spark/sql/catalyst/util/CaseInsensitiveMap.scala:45:0: Whitespace at end of line
[error] /home/jenkins/workspace/SparkPullRequestBuilder@3/sql/catalyst/src/main/scala-2.12/org/apache/spark/sql/catalyst/util/CaseInsensitiveMap.scala:45:0: Whitespace at end of line

Test build #128161 has started for PR 29584 at commit a9561ea.

Merged build finished. Test PASSed.

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

Test build #128161 has finished for PR 29584 at commit a9561ea.

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

Merged build finished. Test PASSed.

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

Merged to master


Test upload form fixes

Currently have fixed properly storing the script name for a test file.

@david-yz-liu
Having an issue right now with supporting "Editing" a file. I looked at the original implementation and am unsure how it was supposed to work. It seems they retrieve the test script file but don't ever actually apply anything to it. In my current implementation it seems to try to create a new one without properly providing the rest of the parameters. Tried various solutions without much luck, any advice?
Another issue is that the test support files don't seem to be passed into the form parameters properly.

Still working on fixing editing the file. Similar issues as before as it's trying to create a new file.

@WilsonChiang good work!


[CH-364] Make sure global_context is initialized once at the first invoking of JNI_Onload and finalized once at exit

Changelog category (leave one):

Bug Fix (user-visible misbehavior in official stable or prestable release)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

Make sure global_context is initialized once at the first invoking  of JNI_Onload and finalized once at exit(current not).
Make sure every correspoinding settings and configs are applied at JNI_Onload (current not).
Some refactor works.

Can one of the admins verify this patch?

test this please with 1143

test this please with 1143

test this please with 1143

test this please with 1143

test this please with 1143

test this please with 1143

test this please with 1143

test this please with 1143

test this please with 1143

test this please with 1143

test this please with 1143

test this please with 1143

test this please with 1143

test this please with 1143


Blog Mode: Disable or modify the date prefixes /blog/2020/08/01/title-slug.

🚀 Feature
For blog mode in Docusaurus V2, there should be a clean way to turn off the calendar date section within the URL path for each blog post, enabling more creative documentation use cases. In our own case we've built a "help center" (knowledge base) that would benefit from disabling the path prefix.
example.com/blog/2020/08/18/title-slug    ==>    example.com/blog/title-slug
Have you read the [Contributing Guidelines on issues]
Yes
Motivation
See help.getwisdom.io for a live example.
example.com/2020/08/18/title-slug    ==>    example.com/title-slug
Pitch
Docusaurus has a great blog framework with a tagging system and related tag list pages (/tags/) that in some ways make it a more powerful documentation approach to using docs with a sidebar. We are using the "blog mode" for Help Center like documentation, however each blog entry has a calendar entry time that doesn't provide benefit:

Aesthetics are poor in this case
Semantics are not relevant, given that for us, these posts should be frequently updated (evergreen).

Currently, saving a markdown file without the filename date prefix appears to just default to UTC date instead of no date within the URL path.
Candidate Interfaces:
Possible Interfaces:

string path_prefix: string prefix.

null defaulting to current date /blog/2020/08/01/title-slug
'' empty string produces no prefix (special case to not produce /blog//title-slug)
string1/string2 produces /blog/string1/string2/title-slug

boolean path_prefix: Config to disable URL path prefix (currently calendar date)
function in docusaurus.config.js that sets the string prefix for each blog article (blog) => 'level1/level2';.
Stop defaulting to UTC time if the file name is not prefixed with date.

Yes you are right we need a way to have better control of blog slugs.
I tried this:

And the URL was this: http://localhost:3000/blog/2020/08/11/heyyy. In such case we'd rather simply not adding the date to the URL at all.
Using markdown files without dates in filenames should be supported and not produce weird issues likes this.
We need a retrocompatible solution, and I'd be fine with 2 options:

frontmatter slug
boolean option to remove the date from the URL, and opt-out of this legacy behavior (not useful to provide custom path prefix, because there's already a routeBasePath option anyway...)

Note: when using slugs, we should not prefix the slug of the user with a date. If they wants a date, they'll add it to the slug.
In the long term we should encourage users to only use slugs and move out of this legacy url scheme
Someone wants to work on this?

I'll be working on this 🙂

Hello @slorber
So after delving into the code, turns out this is already supported.

In this block of code in docusaurus-plugin-content-blog/blogUtils.ts,
if frontmatter.id is defined, it will use it as the only slug and not append the date to the url path.
I tried:
This blog structure

and this test.md

And the url was available at http://localhost:3000/blog/test

This feature in not documented though, and no mention of id frontmatter is present in the blog docs.
Do you think I should go forward with:

Renaming id to slug
Updating the documentation to include this new frontmatter key

Yes we should:

rename id to slug (add a warning for users using frontmatter.id)
document slug frontmatter
fix the date parsing logic that use current date as a default pathname prefix when filename is test.mdx: it should just lead to /blog/test unless user is using slug frontmatter

Thanks

Fixed by #3284


Can not run examples

Hi,
I tried all 4 methods to run this example and it is ending with errors.
Do I have to load a specif branch?

Hi, please be more specific about the error that you found so we can help you.
The main branch should work as-is.

Lets start with Gitpod

Building wheels for collected packages: dj-database-url, aws-requests-auth, uwsgi, psycopg2
  Building wheel for dj-database-url (setup.py) ... done
  Created wheel for dj-database-url: filename=dj_database_url-0.4.1-py2.py3-none-any.whl size=3779 sha256=160a33aa01b9cebd46109e3d375869971d7112f2a1a4d362bd4f9c52b7ec3004
  Stored in directory: /home/gitpod/.cache/pip/wheels/dd/53/6f/312266de55e8e67f2898e4c314dad7c02a700d12c485081463
  Building wheel for aws-requests-auth (setup.py) ... done
  Created wheel for aws-requests-auth: filename=aws_requests_auth-0.4.0-py3-none-any.whl size=6694 sha256=7be73aebc5284b427d980fcf15a4d6a3cebfa52afc87c74c6c290e06f7137cf3
  Stored in directory: /home/gitpod/.cache/pip/wheels/53/4b/7f/1169efe754e41b099c16cd61705d67df8c5d9c5893f6afdeb1
  Building wheel for uwsgi (setup.py) ... done
  Created wheel for uwsgi: filename=uWSGI-2.0.21-cp38-cp38-linux_x86_64.whl size=7099073 sha256=1c5ef9053dc45a186ea0bb53d5f55ba5ccc02e7c359f7c9e609534ecfc784ab6
  Stored in directory: /home/gitpod/.cache/pip/wheels/ff/1f/b4/8bd0ceee591f1fdbbff1b106e279b74db7f8df386291b2e79b
  Building wheel for psycopg2 (setup.py) ... done
  Created wheel for psycopg2: filename=psycopg2-2.9.5-cp38-cp38-linux_x86_64.whl size=592547 sha256=5f485cefdc1b4d492fdc25ae2ea50d35df3fe405cf71c80e3eb6d62db5715cbe
  Stored in directory: /home/gitpod/.cache/pip/wheels/ae/51/cd/0096c52b03b32a5ffea19b70f2c56c905b5d30774229808041
Successfully built dj-database-url aws-requests-auth uwsgi psycopg2
Installing collected packages: uwsgi, pytz, pyasn1, draftjs-exporter, django-dotenv, dj-database-url, Willow, whitenoise, wagtail-font-awesome-svg, urllib3, telepath, sqlparse, soupsieve, rsa, pyasn1-modules, psycopg2, protobuf, Pillow, l18n, jmespath, html5lib, google-resumable-media, et-xmlfile, docutils, cachetools, backports.zoneinfo, async-timeout, asgiref, anyascii, redis, openpyxl, googleapis-common-protos, google-auth, elasticsearch, Django, botocore, beautifulsoup4, s3transfer, google-api-core, djangorestframework, django-treebeard, django-taggit, django-storages, django-redis, django-permissionedforms, django-modelcluster, django-filter, django-debug-toolbar, django-basic-auth-ip-whitelist, aws-requests-auth, wagtail, google-cloud-core, boto3, google-cloud-storage
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
twine 3.8.0 requires urllib3>=1.26.0, but you have urllib3 1.25.11 which is incompatible.
Successfully installed Django-4.1.7 Pillow-9.4.0 Willow-1.4.1 anyascii-0.3.1 asgiref-3.6.0 async-timeout-4.0.2 aws-requests-auth-0.4.0 backports.zoneinfo-0.2.1 beautifulsoup4-4.11.2 boto3-1.9.189 botocore-1.12.253 cachetools-4.2.4 dj-database-url-0.4.1 django-basic-auth-ip-whitelist-0.5 django-debug-toolbar-3.8.1 django-dotenv-1.4.1 django-filter-22.1 django-modelcluster-6.0 django-permissionedforms-0.1 django-redis-5.2.0 django-storages-1.13.1 django-taggit-3.1.0 django-treebeard-4.6.1 djangorestframework-3.14.0 docutils-0.15.2 draftjs-exporter-2.1.7 elasticsearch-5.5.3 et-xmlfile-1.1.0 google-api-core-2.10.2 google-auth-1.35.0 google-cloud-core-1.7.3 google-cloud-storage-1.20.0 google-resumable-media-0.4.1 googleapis-common-protos-1.58.0 html5lib-1.1 jmespath-0.10.0 l18n-2021.3 openpyxl-3.1.1 protobuf-4.22.0 psycopg2-2.9.5 pyasn1-0.4.8 pyasn1-modules-0.2.8 pytz-2022.7.1 redis-4.5.1 rsa-4.9 s3transfer-0.2.1 soupsieve-2.4 sqlparse-0.4.3 telepath-0.3 urllib3-1.25.11 uwsgi-2.0.21 wagtail-4.2 wagtail-font-awesome-svg-0.0.3 whitenoise-5.0.1
WARNING: You are using pip version 22.0.3; however, version 23.0 is available.
You should consider upgrading via the '/home/gitpod/.pyenv/versions/3.8.12/bin/python -m pip install --upgrade pip' command.
No changes detected
Traceback (most recent call last):
  File "manage.py", line 14, in <module>
    execute_from_command_line(sys.argv)
  File "/workspace/.pip-modules/lib/python3.8/site-packages/django/core/management/__init__.py", line 446, in execute_from_command_line
    utility.execute()
  File "/workspace/.pip-modules/lib/python3.8/site-packages/django/core/management/__init__.py", line 440, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/workspace/.pip-modules/lib/python3.8/site-packages/django/core/management/base.py", line 402, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/workspace/.pip-modules/lib/python3.8/site-packages/django/core/management/base.py", line 448, in execute
    output = self.handle(*args, **options)
  File "/workspace/.pip-modules/lib/python3.8/site-packages/django/core/management/base.py", line 96, in wrapped
    res = handle_func(*args, **kwargs)
  File "/workspace/.pip-modules/lib/python3.8/site-packages/django/core/management/commands/migrate.py", line 114, in handle
    executor = MigrationExecutor(connection, self.migration_progress_callback)
  File "/workspace/.pip-modules/lib/python3.8/site-packages/django/db/migrations/executor.py", line 18, in __init__
    self.loader = MigrationLoader(self.connection)
  File "/workspace/.pip-modules/lib/python3.8/site-packages/django/db/migrations/loader.py", line 58, in __init__
    self.build_graph()
  File "/workspace/.pip-modules/lib/python3.8/site-packages/django/db/migrations/loader.py", line 235, in build_graph
    self.applied_migrations = recorder.applied_migrations()
  File "/workspace/.pip-modules/lib/python3.8/site-packages/django/db/migrations/recorder.py", line 81, in applied_migrations
    if self.has_table():
  File "/workspace/.pip-modules/lib/python3.8/site-packages/django/db/migrations/recorder.py", line 57, in has_table
    with self.connection.cursor() as cursor:
  File "/workspace/.pip-modules/lib/python3.8/site-packages/django/utils/asyncio.py", line 26, in inner
    return func(*args, **kwargs)
  File "/workspace/.pip-modules/lib/python3.8/site-packages/django/db/backends/base/base.py", line 323, in cursor
    return self._cursor()
  File "/workspace/.pip-modules/lib/python3.8/site-packages/django/db/backends/dummy/base.py", line 20, in complain
    raise ImproperlyConfigured(
django.core.exceptions.ImproperlyConfigured: settings.DATABASES is improperly configured. Please supply the ENGINE value. Check settings documentation for more details.
Unknown command: 'load_initial_data'. Did you mean loaddata?
Type 'manage.py help' for usage.
CommandError: You must set settings.ALLOWED_HOSTS if DEBUG is False.
gitpod /workspace/bakerydemo (main) $

Thanks, we made some changes to the setup in #393 which was merged yesterday. Looks like the Gitpod config hasn't been updated and the README was incorrect.
With a local setup, after the following step:
https://github.com/wagtail/bakerydemo/blob/66aa55551b007a1310d8f75f6fd34ac8d42e1aa4/readme.md?plain=1#L153-L157
Updating the .env file with the following content should fix it:
DJANGO_SETTINGS_MODULE=bakerydemo.settings.dev

Thank you :)
Once it is published. Create a release so we know that this version should work for sure :)
Thank you for the fast reply


Support deploying K8S OVA in vSphere using Content Library

Feature Request
Today, the workflow for a vSphere-based deployment still has many moving parts. Users must download the set of K8s OVA and import them vCenter Server. After that, they must convert the image to a vSphere Template before TCE can detect and deploy Management and/or Workload Clusters from.
It would really simplify the user experience if the K8s OVA were available as an externally hosted Content Library like S3 or any HTTP(s) endpoint and then have TCE be able to also search for OVA within Content Library and simply deploy from it. In addition to automatically sync'ing all current and future K8s OVA, there's no additional step to convert image to vSphere Template
Describe alternatives you've considered
N/A
Additional context
For Air-gapped or offline env, customers can also create their own Content Library either backed by a local vSphere Datastore or setup a similar externally hosted Content Library that resides within a user env

Raised this internally on several occasions. Going to put into icebox for now as I don't want to keep giving bad expectations to the community.


Value Cannot be Null. (Parameter 'Text')

Greetings,
Problem: Value Cannot be Null. (Parameter 'Text')
I'm sure I've got all the requirements filled out correctly and that everything is up to date.
No idea what else to do :(
Thanks

Nevermind, i resolved the issue.
Turns out I had the incorrect personal character.ai token and had to delete the AI character restart the program, and then rebuild the AI character along with the correct personal character.ai token.


Reduce PDF file size by completely discarding the clipped out part of an image

Description
Imagine you have an image that you want to have only a part of. You would use something like block(clip: true, image("...") and make sure that some part of the image you don't want isn't visible (https://github.com/typst/typst/issues/2709#issuecomment-2411709983). But now, even if you only see a part of the image, you probably would like the image to be actually cropped, meaning that the invisible part should be removed permanently (from the PDF file).
I think this can either be a field like block(clip: true, crop: true) which is turned on by default, or a dedicated function like crop().
Tagged PDF
There were also some concerns about that, but I have no idea what this means exactly, but still want to mention that.

A small conversation started here.
Use Case
The smaller the PDF file size, the better!

Unfortunately, none of the supported image formats implement ImageDecoderRect. I think this would not make the compiler faster and probably even slower.
If this gets implemented, the compiler should check if the cropped images filesize is actually smaller before including the cropped version. Otherwise stay with the default method.
If I remember correctly, JPG or a version of JPG could get embedded in the PDF without reencoding, so basically just copying the file into the binary without opening it. If this is true, the compiler will definetly become slower when including a jpg image and cropping it.


Vue icon

I'd rather say the original Vue files icon (that is also the original vue logo) is far better looking. Is there any copyright reason to not use it here too ?

This is the concept of these icons. What exactly do you mean?

Don't get me wrong, the icon as a whole is ok and everything, but because of the big area of flat bright color it looks visually bigger than the others. Reducing it by a pixel in each direction would look more balanced with the others, I think. Like this :

Look at the "build-config.js" icon - i modified it to be a bit smaller (actually it is just as wide as the "package.json" icon) and it looks more balanced and not so ... errm ... imposing to me.

Well ... what it is - left, what I think it should be - right :

Depends on what it is in the original file-icons package. If it’s like this, so it should be.
--
Elior
On 15 Dec 2019, 11:33 +0200, Georgi Georgiev<EMAIL_ADDRESS>wrote:

Well ... what it is - left, what I think it should be - right :
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

So you don't modify the icons by yourself .. I see ... you can close this, then, since it is not in your hands to change it.

No you don’t understand, I can change them, but the source of the icons is from the package. If they are like that there, so they should be like that here. It’s for the sake of consistency.
If you think that the color could be changed, tell me.
--
Elior
On 15 Dec 2019, 11:38 +0200, Georgi Georgiev<EMAIL_ADDRESS>wrote:

So you don't modify the icons by yourself .. I see ... you can close this, then, since it is not in your hands to change it.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Yes, I think it should be changed as this is the original Vue.js logo. You can check for yourself here : https://vuejs.org/


Filter by sc vs sn

Need
It would be incredibly useful to be able to sort also by cells vs nuclei in the data portal. I've been looking for lung data to use as example query data to map onto the HLCA, and the only human, 10X data from the lung that only consists of 1 dataset is from lungMap, and that's all single-nucleus data, which might not be the best test case.
Just to add a second feature that would be useful (but harder to do I'm sure), would be the ability to load only the lung data from datasets with multiple tissues.
Approach
Describe the solution you'd like
An additional filter option for cell vs nucleus
Describe alternatives you've considered
The alternative would be go to through all dataset descriptions and then come back to download the data that fits the bill.
Definition of Done
Some automatic filter for sc vs sn to immediately gauge relevant datasets.

First step to a filter is a required metadata field...
https://app.zenhub.com/workspaces/single-cell-5e2a191dad828d52cc78b028/issues/chanzuckerberg/single-cell-curation/227

I'm slowly learning how zenhub works ^^.


CarbonPlan's downscaled climate datasets seems empty.

I am learning to access CarbonPlan's downscaled climate datasets.
But when I ran the example named "Accessing CarbonPlan CMIP6 downscaled climate datasets"  in notebooks, I found that the datasets I got seemed empty.
Why is this? Thank you in advance!

Hey there @YueZhang99, sorry you are having trouble accessing our downscaled datasets.
Just to confirm, is this the notebook you are trying to run? https://github.com/carbonplan/cmip6-downscaling/blob/main/notebooks/accessing_data_example.ipynb
Does this code run successfully for you?
!pip install intake
!pip install intake-esm

import intake

cat = intake.open_esm_datastore(
    "https://cpdataeuwest.blob.core.windows.net/cp-cmip/version1/catalogs/global-downscaled-cmip6.json"
)

cat.df

Thanks for reporting an issue!

Thank you very much for your speedy attention on this matter @norlandrhagen . And I'm sorry that I didn't attach the link to the corresponding notebook.
The notebook you mentioned above is exactly what I am trying to run. The whole code runs successfully without no errors or warnings.
Here are my results for a clearer picture of this issue. https://github.com/YueZhang99/CarbonPlan-issues/blob/main/Accessing%20CarbonPlan%20CMIP6%20downscaled%20climate%20datasets%EF%BC%88Example%EF%BC%89.ipynb

Thank you very much for your speedy attention on this matter @norlandrhagen . And I'm sorry that I didn't attach the link to the corresponding notebook.
The notebook you mentioned above is exactly what I am trying to run. The whole code runs successfully without no errors or warnings.
Here are my results for a clearer picture of this issue.https://github.com/YueZhang99/CarbonPlan-issues/blob/main/Accessing CarbonPlan CMIP6 downscaled climate datasets（Example）.ipynb


Doc update: "TemplatesPlugin" to "ViewTemplates", added imports.

Apparently after breaking-changes in ViewTemplates API doc stayed the same.
I also added imports, as it would took ages for people without IDE to find those.

fwiw this still seems broken. i'm trying to implement my own custom views according to the docs here but it still references
TemplatesPlugin

@mattfeury yes, the docs are not up to date yet.


Update gaze-and-commit.md

Fixing table formatting

Docs Build status updates of commit bc9380d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/gaze-and-commit.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


show shortlink column

see #25
I am also working on a Gutenberg sidebar integration.

@willnorris what do you think?

☺


MEN-3221 06.Server-installation/02.Demo-installation rewrite

ChangeLog:none
Signed-off-by: Peter Grzybowski<EMAIL_ADDRESS>

this one is going to be closed. sorry for the confusion. it is borken (see at the top "unknown repository")
new changes and all your comments addressed here: https://github.com/mendersoftware/mender-docs/pull/992


fix: remove unused config of removed CIS Kubernetes Benchmark feature

Description
Just a cleanup of dead code/configuration/docs of a feature that seems to be removed from trivy-operator.
Checklist

[x] I've read the guidelines for contributing to this repository.
[ ] I've added tests that prove my fix is effective or that my feature works.
[x] I've updated the documentation with the relevant information (if needed).
[ ] I've added usage information (if the PR introduces new options)
[ ] I've included a "before" and "after" example to the description (if the PR is a user interface change).

@erikgb we will add it back soon. This is related to what I'm adding on trivy currently. It won't be kube-bench, bus cisbench directly from trivy.

Closing, as this feature will be re-added.


Invalid this in jQuery iterator arrow function

Before changing the iterator function to an arrow function, the iterator of the jQuery.fn.each in getTabbableNodes() could access the current iterator element via this. Now, with the arrow function the node should instead be accessed as the second argument of the each iterator function.

Fixed on master (v2.0.0).


chore: rearrange/edit README

Did and edit pass on the README.  Specifically:

Updated the features list at the top
Moved Quick Start to the top, since that is what most people are going to be interested in.
Move release notes to the bottom.  Kept a "Upgrading to uuid@3?" line at the top directing people to them, however.
Renamed all "v3.x"-style uuid version references to "uuid@3" to minimize confusion between RFC UUID "version" and uuid module version.  I.e Don't use "version" to refer to anything but RFC UUID version.
Unicode warning symbols as html entity char codes (prefer not having actual unicode chars in source).  Also, the "⚠ ⚠ ⚠ ... WARNING HERE ...⚠ ⚠ ⚠" felt a bit excessive. 😄

Also, I just merged #373 and a rebase is now necessary, sorry!

Failing test is unrelated.  Merging.


Remote file is now before notification friendly

Fixes #33

@axelrtgs sorry for the very long review, thanks a lot for your time investigating this bug and fixing it :)


ability to specify background color as option

Now you can specify background color as an option in constructor or data attribute. Fixes #10 .

any update on this?

@judesfernando You should strongly consider merging this.

@judesfernando I agree with @ola-rende

Updated PR with newest master branch commits to resolve the merging conflicts.

Thank you!


terraform module feature: pre-processing options

I'm thinking of contributing some pre-processing options for the terraform executor. Basically, when using a monorepo, working directory and context become critical, and plan/apply times can skyrocket due to node_modules.  Even ignoring these files doesn't stop them from being evaluated locally before the plan/apply executes.
a "preProcessing" options config would allow you to specify a directory to move the tf files to prior to running tf commands. This would allow you to use relative directories with modules so that they work in editors for syntax highlighting/completion/linting, but execute quickly for local and ci plan/apply because the context directories would be smaller.
Would you support a contribution for this type of feature?

Yes! 🔥

hopefully have something for you soon. Still kinda working out what this would look like, and how to make it easy to use without having to be repeatable for all the targets.  (might be a necesarry evil though).
one additional thing I will add to this, or maybe as a separate thing on its own PR, is the ability to generate lockfiles for providers on other platforms (to support cross-platform dev teams, and combined CI/local workflows)


Fix Ivits counter of power inside Federation

Fix Ivits counter of power inside/outside the Federation is not updated when colonizing from a gaiaformer (but the newly built mine seems to be correctly added to the Federation)

As it turns out, when placing a Space Stations I was connecting to the existing federation also the adjacent gaiaformers; once the corresponding hex was inside the federation, it was disregarded when colonized with an actual mine and the federation's power value was not increased by 1.


Problem in installing dependencies

I tried to install the dependencies on various python 3.7 and above versions. but getting the following error each time:
INFO: pip is looking at multiple versions of lightning to determine which version is compatible with other requirements. This could take a while.
ERROR: Package 'lightning' requires a different Python: 3.7.9 not in '>=3.8'
I am beginner. pl suggest  resolving these problem.

Hi. Unfortunately, Python 3.7 is not supported as the message indicates. You'll have to upgrade to 3.8, 3.9, or 3.10.
See https://pytorch.org/blog/deprecation-cuda-python-support and https://endoflife.date/python

Hi,
I have tried on 3.8, 3.9, 3.10, and 3.11 also but it's not working. Kindly
suggest a solution if possible.
Thanking You
On Mon, 19 Jun 2023 at 15:19, Carlos Mocholí @.***>
wrote:

Hi. Unfortunately, Python 3.7 is not supported as the message indicates.
You'll have to upgrade to 3.8, 3.9, or 3.10.
See https://pytorch.org/blog/deprecation-cuda-python-support and
https://endoflife.date/python
—
Reply to this email directly, view it on GitHub
https://github.com/Lightning-AI/lit-parrot/issues/167#issuecomment-1596867734,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APNIVY5YN6IRQWAJGULF7OTXMAODHANCNFSM6AAAAAAZK6XCJE
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Jeetendra Kumar
Assistant Professor
Department of Computer Science & Applications
Atal Bihari Vajpayee Vishwavidyalaya,Bilaspur(C.G.) INDIA-495009
Website- bilaspuruniversity.ac.in http://bilaspuruniversity.ac.in
E-mail- @.** @.**>
Mob.- +91-8109194501

What error are you getting with the upgraded Python?

Same Error in all versions of Python.
On Tue, Jun 20, 2023, 12:02 AM Carlos Mocholí @.***>
wrote:

What error are you getting with the upgraded Python?
—
Reply to this email directly, view it on GitHub
https://github.com/Lightning-AI/lit-parrot/issues/167#issuecomment-1597601334,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APNIVY4MNUTLJZXZZEDJ2ILXMCLMLANCNFSM6AAAAAAZK6XCJE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Can you verify that you are using the correct Python executable? You can do so by running python -V. It should print a version higher than 3.7. If not, then there's probably still an issue with your environment

Screenshot attached for reference to resolve the issue
On Tue, 20 Jun 2023 at 04:06, Carlos Mocholí @.***>
wrote:

Can you verify that you are using the correct Python executable? You can
do so by running python -V. It should print a version higher than 3.7. If
not, then there's probably still an issue with your environment
—
Reply to this email directly, view it on GitHub
https://github.com/Lightning-AI/lit-parrot/issues/167#issuecomment-1597842681,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APNIVY5PXZBROY4N6ZYKLGLXMDH7RANCNFSM6AAAAAAZK6XCJE
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Jeetendra Kumar
Assistant Professor
Department of Computer Science & Applications
Atal Bihari Vajpayee Vishwavidyalaya,Bilaspur(C.G.) INDIA-495009
Website- bilaspuruniversity.ac.in http://bilaspuruniversity.ac.in
E-mail- @.** @.**>
Mob.- +91-8109194501

It seems like your pip is linked to a different python. Can you try python -m pip install ...?

Great!


Improve token expiered error

When running something like:
make prepare ENV=prod DIR=governance
I got the following error.
2021/03/11 09:25:19 azure.go:49: tf-prepare "error"="InnerError: {\u003cnil\u003e 0xc0003361d0 \u003cnil\u003e 0xc0003361e0 \u003cnil\u003e}\n" "msg"="client.CreateOrUpdate"
2021/03/11 09:25:19 main.go:40: tf-prepare "error"="InnerError: {\u003cnil\u003e 0xc0003361d0 \u003cnil\u003e 0xc0003361e0 \u003cnil\u003e}\n" "msg"="CLI execution failed"
make: *** [Makefile:27: prepare] Error 1

This is due to your az token have expiered. We should have a better error be printed here so that becomes obvious.

This is probably not a current issue.
Recreate if it's still a problem.


How to get OAuth 2.0 access token for Firebase Cloud Messaging HTTP v1 API

The FCM HTTP V1 API requires the server to get a "short-lived OAuth 2.0 access token" to use as the Authorization: Bearer=TOKEN request header. To do so, the docs simply point in the direction of this API Client Library, and they show some examples for the Node and other non-PHP languages:

After exploring the present google-api-php-client library, I can't find any possible way to get this token. There are mentions to OAuth 2.0 and tokens in many places, but there isn't a simple example or explanation on how to get an OAuth 2.0 access token for use with FCM HTTP V1 requests.. That is all I need to get to continue with my own PHP-based curl() implementation of the FCM API.
If google wants developers to migrate to the HTTP v1 API, this process should be clearly explained in a simple example in the very home page of the PHP client library.
Does anyone know how to get such token with this library? Is it even possible? Thanks in advance.

You may use either of the following methods to obtain an access token:
putenv('GOOGLE_APPLICATION_CREDENTIALS=/path/to/keyfile.json');
$scope = 'https://www.googleapis.com/auth/analytics.readonly';
$client = new Google_Client();
$client->setScopes($scope);
$client->useApplicationDefaultCredentials();
print_r($client->fetchAccessTokenWithAssertion());

use Google\Auth\CredentialsLoader;

$scope = 'https://www.googleapis.com/auth/analytics.readonly';
$credentials = CredentialsLoader::makeCredentials($scope, json_decode(file_get_contents(
    '/path/to/keyfile.json'
), true));

print_r($credentials->fetchAuthToken());

If you are not using the API Client for anything besides obtaining an access token, I suggest the latter, as it only requires installation of google/auth, allowing you to drop the much larger dependency on google/apiclient.
If you wish to call Firebase Cloud Messaging via the PHP API Client, you can allow the client to authenticate on your behalf similarly to the first example:
putenv('GOOGLE_APPLICATION_CREDENTIALS=/path/to/keyfile.json');
$scope = 'https://www.googleapis.com/auth/analytics.readonly';
$client = new Google_Client();
$client->setScopes($scope);
$client->useApplicationDefaultCredentials();

$fcm = new Google_Service_FirebaseCloudMessaging($client);
// construct and execute API requests, will be authenticated using the service account.

I'm going to close this issue out. If you're still having trouble, please feel free to re-open!

HI @andreszs is your issue resolved using the solution provided by @jdpedrie ?

hello,
I have the following problem, I put your code above but it doesn't find one of the classes to follow the authentication
Fatal error: Uncaught Error: Class "Google\Auth\CredentialsLoader" not found in /var/www/html/google-api/teste.php:13 Stack trace: #0 {main} thrown in /var/www/html/google-api/teste.php on line 13
fetchAuthToken());

HI @andreszs is your issue resolved using the solution provided by @jdpedrie ?

No, I've solved with this rudimentary function written by GPT instead:
/**
     * Genera nuevo ouath2 access token para FCM.
     * @return string
     */
    private static function get_fcm_oauth2_access_token() {
        // Lee las credenciales desde el archivo JSON
        $serviceAccount = __DIR__.'/keys/'.self::SERVICE_ACCOUNT_KEY;
        $credentials = json_decode(file_get_contents($serviceAccount), true);
        if (empty($credentials)) {
            error_log('Error fetching access token: Failed to parse service account credentials');
            return null;
        }

        // URL del token de Google OAuth2
        $url = 'https://oauth2.googleapis.com/token';

        // Crea el JWT (assertion)
        $header = base64_encode(json_encode(array('alg' => 'RS256', 'typ' => 'JWT')));
        $now = time();
        $expiry = $now + 600; // 10' this stores the expiration time in the token file

        $claimSet = base64_encode(json_encode(array(
            'iss' => $credentials['client_email'],
            'scope' => 'https://www.googleapis.com/auth/cloud-platform',
            'aud' => $url,
            'iat' => $now,
            'exp' => $expiry
        )));

        $signatureInput = $header.'.'.$claimSet;

        // Firma el JWT con la clave privada
        $signature = '';
        if (!openssl_sign($signatureInput, $signature, $credentials['private_key'], 'SHA256')) {
            error_log('Error signing the JWT');
            return null;
        }
        $signedJwt = $signatureInput.'.'.base64_encode($signature);

        // Envía la solicitud para obtener el token
        $postFields = http_build_query(array(
            'grant_type' => 'urn:ietf:params:oauth:grant-type:jwt-bearer',
            'assertion' => $signedJwt
        ));

        $ch = curl_init();
        curl_setopt($ch, CURLOPT_URL, $url);
        curl_setopt($ch, CURLOPT_POST, true);
        curl_setopt($ch, CURLOPT_POSTFIELDS, $postFields);
        curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);
        curl_setopt($ch, CURLOPT_SSL_VERIFYPEER, false);
        curl_setopt($ch, CURLOPT_TIMEOUT, 10);

        $response = curl_exec($ch);
        curl_close($ch);

        $data = json_decode($response, true);

        // Return generated token if available
        if (isset($data['access_token'])) {
            return $data['access_token'];
        } else {
            error_log('Error fetching access token: '.$data['error_description']);
            return null;
        }
    }

Where the SERVICE_ACCOUNT_KEY constant points to the JSON service account file downloaded from the Firebase console.
You can store the retrieved token in a tokens table, alongside the user ID and expiry time, in order to easily retrieve it for reuse while it's still valid. Firebase does not disclose the access token vailidity time, so I've set it to 10 minutes because it's a nice round number, but its real duration could be an hour, a day, a week... nobody knows.

hello, I have the following problem, I put your code above but it doesn't find one of the classes to follow the authentication
Fatal error: Uncaught Error: Class "Google\Auth\CredentialsLoader" not found in /var/www/html/google-api/teste.php:13 Stack trace: #0 {main} thrown in /var/www/html/google-api/teste.php on line 13
fetchAuthToken());

You either failed to download/unzip the classes in the correct path, or your autoloader is not working. Try to install it with Composer whenever possible which takes care of the autoload scripts for you, or figure out why your autoload is failing.


[BUG] Missing crafting recipe for many items, Both in JEI and in Engineer's Manual.

Description:
Some items like bronze and nikolite don't show up on JEI menu and don't have any explanation on how to craft them in the Engineer's Manual. For example I want to craft second tier solar generator, and recipe shows I need two enriched nikolite ingots and two electrum plates. JEI has no recipe for those, Engineer's manual doesn't mention how to craft any of those, it only has the recipe for the machine, and it seems it expects me to figure out what electrum is and how to craft enriched nikolite ingot.
Steps to Reproduce:

Create a Fabric 1.20.1 world
Install Industrial Revolution (optionally JEI too)
Figure out where the recipe is for : Bronze dust/ingot, enriched nikolite/ingot, electrum.

Technical Information:

Minecraft: 1.20.1
Fabric API: 0.14.24
Industrial Revolution: indrev-1.16.5-BETA
Modpack: none

Logs:

No need.

Details:
Description is clear enough. But here's a proof that nikolite ingot not showing up on JEI menu :

use REI not JEI, JEI is not supported

use REI not JEI, JEI is not supported

That solved it, thanks!


Follow up on Exposure Notification API testing by Fraunhofer IIS

Your Question

Documentation File: 2020_06_24_Corona_API_measurements.pdf

Thanks for uploading API testing documentation! If I understand it correctly, the tests were performed using 55dB and 60dB attenuation buckets with weights [1.0, 0.5, 0.0]. This resulted in quite low recall of 47%, meaning that only half of relevant contacts with infected user were properly registered. Assuming current adoption rate of 20% this is further reduced to only 10% chance of being notified about the contact with infected person, assuming everyone is using the app properly all the time and always uploads keys.
After the results of this test RKI made a change of increasing bucket threshold from 60dB to 63dB, but this seems like a small modification. Were there any further tests to confirm how this change affected recall value? Perhaps third bucket could be used to register contacts also in the situations with heavy shadowing? This is related to "yellow card" proposal (https://github.com/corona-warn-app/cwa-app-android/issues/899#issuecomment-663475550)

Sorry for not replying earlier - we just reached out again to Fraunhofer IIS for additional information, as also mentioned in https://github.com/corona-warn-app/cwa-documentation/issues/312#issuecomment-669904815.
Mit freundlichen Grüßen/Best regards,
SW
Corona Warn-App Open Source Team

Hello @kbobrowski,
thanks for starting the discussion. I know it has been a while since you have asked, and therefore I am happy to get back to you today with an answer from the RKI.
Regarding your question, for additional tests I can provide you with the following:
We have planed additional test series in September in cooperation with Frauenhofer IIS. In this series we mainly will focus on Bluetooth signal reflection in metal tubes (bus, train and aeroplane simulations), as well as changes due to the Google and Apple ENF API versions (Version 1.5 and 1.6). Still, it will be an aspect that we examine the effects and impacts of the bucket threshold increment from 60dB to 63dB.
Thank you,
LMM
Corona-Warn-App Open Source Team

@SebastianWolf-SAP @GPclips thanks for keeping us in the loop. I just noticed (via community Slack) a blog post comparing Fraunhofer IIS study and study done by prof. Leith team:

In the last test series, using different mobile devices, about 80 percent of the encounters were recorded correctly. As such, we do not understand the results presented in the Irish study. As far as the Federal Ministry of Health is aware, the Irish study hasn’t been subject to a scientific independent peer review process.

Using accuracy value here (which was 79% in Fraunhofer IIS study) may be misleading, as this was a study with highly uneven class distribution (ground truth was 103 contacts with no exposure and 40 contacts with exposure).
Even if CWA did not work at all (no contacts registered), the accuracy value would be 103 / (103 + 40) = 72%. Recall or F1 score provide less-misleading information about true performance of the system, and these values were lower (47% and 56%).
Would be great to have some comparison between the next Fraunhofer IIS study and prof. Leith study, using comparable statistical values. I would also suggest to refrain in public communication from quoting accuracy values without mentioning uneven class distribution, as it may easily mislead the reader. In the previous study the minimum possible accuracy was 72% for non-functioning app, 79% figure does not look so good in this context, and does not really negate Irish study results.

cc @doug-leith

(@kbobrowski SwissCovid seems to have suffered from the same problems in the public reporting of accuracy/recall https://github.com/DP-3T/bt-measurements/issues/4 )

@pdehaye I think the problem with reporting accuracy is that many understand "accuracy" in this context as "how well the app detects exposures", not as "how well the app stays silent if there is no exposure + how well the app detects exposures" which is the correct interpretation.
But the main difference between prof. Leith study and Fraunhofer IIS study is that the former was in enclosed metal space (tram / bus), and the latter in the large open space - not really comparable.

Hello @kbobrowski and community,
I just reached out to the RKI asking for the September test results. We will inform you as soon as we can and keep you in the loop.
Thanks,
LMM
Corona-Warn-App Open Source Team

@GPclips are there any news wrt the Fraunhofer followup study you can share with us? 🙂

This issue is probably no longer relevant after ENFv2 was integrated into the app. I don't know if it would make sense to also discuss more recent tests on the ENA here and it would be best to create a new issue for that.

Corona-Warn-App Open Source Team

This issue is probably no longer relevant after ENFv2 was integrated into the app. I don't know if it would make sense to also discuss more recent tests on the ENA here and it would be best to create a new issue for that.

Corona-Warn-App Open Source Team

@heinezen I am still interested to learn the results from the September Fraunhofer tests as the underlying technology on the fundamental level (BLE attenuation as distance proxy) didn't change between ENFv1 and ENFv2 and those test should at least provide some insight into that.
Additionally if there are any new test they would also be interesting of course 🙂.

@heinezen I am still interested to learn the results from the September Fraunhofer tests as the underlying technology on the fundamental level (BLE attenuation as distance proxy) didn't change between ENFv1 and ENFv2 and those test should at least provide some insight into that.
Additionally if there are any new test they would also be interesting of course 🙂.

@daimpi I'll try to get a hold of the results :)

Corona-Warn-App Open Source Team

@daimpi I'll try to get a hold of the results :)

Corona-Warn-App Open Source Team

@heinezen (I know that you left the Open-Source-Team a while ago) were you Abel to get the results? If they are not available to the public / it is not planned to make them public, I suggest to close this issue.
cc @dsarkar


escalate to human

Is there an option to escalate to a human username (in slack) if bot fails to answer adequately?

You can send messages to anyone if you know their ID on the channel.  This would allow your code to forward a message it does not understand to a human.  Are you using the node or C# version of the builder?

@chrimc62 I m using c#

@chrimc62 I m using c#. Let me know what is code to use.

You can instantiate an IConnectorClient, make an Message addressed to that user, and send it through the IConnectorClient implementation.

I'm going to close this item for now, but feel free to reopen a new item with more questions.

What would be the node equivalent of IConnectorClient? Thank you.


escaped params silently removing files

I observed, when using Sinatra (this may be found with other things - I don't know), that the escaped params handler is removing (setting to nil) any parameters that are not any of String, Array or Hash.
This is an issue when POSTing multipart forms containing files, which are added to the params as a hash, with one of the items within that hash being a TempFile, pointing to the temporary file containing the uploaded file's contents.
This results in the application being provided with a hash containing what should be the tempfile as nil, and so the uploaded content not being available to the application.
The change https://github.com/rkh/rack-protection/commit/ee53d6ae644411636a91fdd96e96bb722997d753 appears to have broken this functionality, but I am not convinced that the desirable solution would be to revert that commit.
The important change (in lib/rack/protection/escaped_params.rb) appears to be the change from the default behaviour of raising an ArgumentError when the type is unknown to just returning nil (below):
       def escape(object)
         case object
         when Hash   then escape_hash(object)
         when Array  then object.map { |o| escape(o) }
         when String then escape_string(object)
-        else raise ArgumentError, "cannot escape #{object.inspect}"
+        else nil
         end
       end

I believe that the correct behaviour desired for the use case that commit refers to should actually be to return nil if (and only if) object is nil, leaving the default case as it was before raising the ArgumentError.
That behaviour would, if I am reading the code correctly, have the negative effect of causing the escaping to fail in the case that any parameter is not a Hash, Array, String or nil, resulting in the whole of the params being fed eventually in to the application unescaped.
In my use case, it would be preferable to pass this unknown item (the tempfile) through, but any other parameters should be escaped. I could add an explicit handler for TempFile to support this specific case, but I fear that the behaviour of silently dropping parameters is (the line else nil) is very painful to debug.
The best solution I could come up with is:
      def escape(object)
        case object
        when Hash   then escape_hash(object)
        when Array  then object.map { |o| escape(o) }
        when String then escape_string(object)
+       when TempFile then object
+       when nil then nil
+       else raise ArgumentError, "cannot escape #{object.inspect}"
-       else nil
        end
      end

In this case the behaviour is to handle TempFiles explicitly (and not escape them), to handle nil values explicitly (again, not escape them) but in the case of something unhandled to raise an Error.
I suspect that this may well be undesirable for many people as it is a change in behaviour.
An alternative may be to support passing in a block that could be executed in the case of an unknown value / type to allow the developer to handle the cases explicitly for their app. I'm not sure what that would look like, though.
Am I missing something?

Is there any update on this?  Right now it seems like it isn't possible to accept file uploads if you use Rack::Protection, which seems like a kinda big issue, at least coming from the view that the module shouldn't break valid use-cases.
The PR that fixes this has a failing Travis test viewable here which had an error for the latest version of JRuby.  Any insight on how to handle this for the Average Joe, outside of making a fork and maintaining it?


Data protection for sensitive files

Do not merge yet, this is incomplete. Only submitting to get early feedback for the code.

What remains to be done:

I'll add a bunch of tests to make sure the implementation works. This will also undoubtedly reveal a bunch of bugs in there.
Add support for processing the legacy way of storing the data.  Simple enough.
Then, hook up the master lock code and the salt data to use the new class.

Will continue working on this tonight and hopefully will finish tomorrow.

The approach seems good to me, but I don't know much about this stuff, so if other people could comment that would be appreciated.  In any case, this form of protection is definitely better than what we have now for those two files.

Pushed what I have now. Almost there!
Done:

Fixed review findings
Added unit tests
Fixed issues found by unit tests
Hooked up to salt data

Not yet:

Need to run all tests on device to make sure the daemon behaves sanely
Need to hook up the masterlock as well

Ok, now refactored a bit and hooked up the lock code data as well. Now ready for review.

I don't have my dev device at hand right now, but will run all tests on it once I get home before merging all this.

@dcaliste I couldn't add you as a reviewer but of course your opinion is welcome here too.

Please rebase

Rebased on top of latest master. Fixed review findings.
At the end we agreed with @rainemak to just get rid of the "legacy" data handling, as it makes very little sense to support something that was never actually released.
I'll run tests on my device and see how it goes from there.


Default ExecutionContext in blaze-client

Following up from #4782, we can't do exactly the same on BlazeClientBuilder because the ExecutionContext is exposed as a val.  I don't know if this would work, and it's an odious hack, but:

Make the val a def.  (Is this bincompat?  I'm not sure about private synthetic methods.)
If the context is explicit, return it.  If it's the "default" context, return a dummy value that doesn't need to be shut down.
Deprecate it

This should be safe for existing users, because all uses are explicit.  And anybody using the new constructor will get a deprecation method about why it's a lie.
If that's too distasteful or not binary compatible, we can just do it in main.
/cc @kubukoz @bplommer

Dummy value can probably be IORuntime.global.compute, since we have a dependency on CE core.


Remove onboarding's first page

Description ✏
Note The current PR is incomplete. Anyone can complete this. With the current changes, it goes directly to the configuration page but also constantly switches between the chat and the configuration page, need fix.
When the app first starts, it shows this:

Only after clicking PearAI Server, it goes to the below image:

However, we want it to go to the 2nd image right away.
Solution

Change onboarding so that it directly shows to login into PearAI
Change wording: remove "1. Signup at  trypear.ai". It should directly be

Login with PearAI
After login, the website should redirect you back here, if it didn't, click again.
If you haven't signed up, do that first and use the "Open PearAI" button on your dashboard after signing in to come back to the app.
[BUTTON HERE THAT SAYS "Login"]
Click to complete
[BUTTON THAT SAYS "Use PearAI Server"]

Done


AppBar accessibility traversal order

Hi @darshankawar , i want to re-open this issue : https://github.com/flutter/flutter/issues/98570
Thx

Closing this. Let's track in linked issue.


just test

PR description
Fixed Issue(s)

hi @mumianhua - welcome to Besu! What are you interested in working on? Hit us up in discord if you have questions https://discord.gg/hyperledger