Error: using nextjs-with-flask-server app is showing error

I have installed the Server as per the readme file and run a client on the terminal.
the local host appeared and uploaded a doc, when trying to get an answer its displaying the below error.
ready - started server on <IP_ADDRESS>:3000, url: http://localhost:3000
event - compiled client and server successfully in 1182 ms (166 modules)
wait  - compiling...
event - compiled successfully in 100 ms (133 modules)
wait  - compiling / (client and server)...
event - compiled client and server successfully in 521 ms (757 modules)

INFO:werkzeug: * Debugger PIN: 105-174-659
INFO:werkzeug:<IP_ADDRESS> - - [25/Feb/2023 11:56:49] "POST /process_file HTTP/1.1" 200 -
INFO:werkzeug:<IP_ADDRESS> - - [25/Feb/2023 11:56:51] "OPTIONS /answer_question HTTP/1.1" 200 -
INFO:werkzeug:<IP_ADDRESS> - - [25/Feb/2023 11:56:54] "POST /answer_question HTTP/1.1" 200 -
INFO:werkzeug:<IP_ADDRESS> - - [25/Feb/2023 11:57:14] "POST /process_file HTTP/1.1" 200 -
INFO:werkzeug:<IP_ADDRESS> - - [25/Feb/2023 11:57:24] "OPTIONS /answer_question HTTP/1.1" 200 -
INFO:werkzeug:<IP_ADDRESS> - - [25/Feb/2023 11:57:27] "POST /answer_question HTTP/1.1" 200 -

I get a similar error when trying to get an answer from the successfully uploaded document.

got this aswell;
I can silence the error and get the system running by checking whether the parameters of the function existed; It will work with one file but as soon as i add a second it no longer returns anything;
spent a short amount of time modifying the src/components/ files and made some changes which seemed to make an improvement but was unstable;

How did you get it to upload. Mine doesn't upload, it gives the error  POST http://localhost:8080/process_file net::ERR_CONNECTION_REFUSED

Mine doesn't upload either!

Hey @Ayo-faks and @troublesprouter are you running the server in a separate terminal tab, and did you add your own details to server/config.py? I also just pushed a change to the server initializing script that could help.

same problem here. it just does not connect but the code seems correct. i even disabled my firewall but same problem. I wonder what it could be.

Hey @Ayo-faks and @troublesprouter are you running the server in a separate terminal tab, and did you add your own details to server/config.py? I also just pushed a change to the server initializing script that could help.

Hello, I still get the same error. Wonder why it's different for me. And yes, they are opened on different terminals.
Can you test it out on codespaces or gitpod, maybe there is a port forwarding issue.
thanks

Hey @Ayo-faks and @troublesprouter are you running the server in a separate terminal tab, and did you add your own details to server/config.py? I also just pushed a change to the server initializing script that could help.

I tried Linux. It now attempts uploading but fails with "error processing file: AxiosError: Request aborted" in the console.
Any ideas how to make it work?

Update: it worked on Linux. I don't know why.

Update: it worked on Linux. I don't know why.

Are you running it locally on your system?

Update: it worked on Linux. I don't know why.

Are you running it locally on your system? I have still not got it to work. any help, anyone!

Yes, it ran locally. Try to find all places to add the API keys and use 3 terminal windows just to be safe.
Any advice on how to host it publicly with Vercel or something else? What's the simplest way?
I tried running vercel from the root directory but got a 404 error. Not sure what changes have to be made.

I ran into this same problem, but I think it had to do with being rate limited. I was on the free trial and running too many requests per minute. I started waiting two-three minutes between questions and was able to get around this issue.
Related documentation: https://github.com/openai/openai-cookbook/blob/main/examples/How_to_handle_rate_limits.ipynb

Update: it worked on Linux. I don't know why.

Are you running it locally on your system? I have still not got it to work. any help, anyone!

Yes, it ran locally. Try to find all places to add the API keys and use 3 terminal windows just to be safe.
Any advice on how to host it publicly with Vercel or something else? What's the simplest way?
I tried running vercel from the root directory but got a 404 error. Not sure what changes have to be made.

My plan was to deploy the flask app as a container and then host the Nextjs app on vercel.
Do you remember the steps you took to make it work, what changed , where were you running it before? We have exactly the same problem.

I tried mac without success. Don’t know the cause.
I set an openai api key inside the yaml config file too. Not sure that was
there before.
Changed the start script command to python3 from python.
Hope that helps.
Is Vercel ok with both including flask or only with the nextjs part?
On Tue, 21 Mar 2023 at 16:28, faksonbaba @.***> wrote:

Update: it worked on Linux. I don't know why.
Are you running it locally on your system? I have still not got it to
work. any help, anyone!
Yes, it ran locally. Try to find all places to add the API keys and use 3
terminal windows just to be safe.
Any advice on how to host it publicly with Vercel or something else?
What's the simplest way?
I tried running vercel from the root directory but got a 404 error. Not
sure what changes have to be made.
My plan was to deploy the flask app as a container and then host the
Nextjs app on vercel.
Do you remember the steps you took to make it work, what changed , where
were you running it before? We have exactly the same problem.
—
Reply to this email directly, view it on GitHub
https://github.com/openai/openai-cookbook/issues/159#issuecomment-1478172286,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADGDEVWUB5K7YQV7HXKTXO3W5HJJLANCNFSM6AAAAAAVHVPTQI
.
You are receiving this because you were mentioned.Message ID:
@.***>

I tried mac without success. Don’t know the cause. I set an openai api key inside the yaml config file too. Not sure that was there before. Changed the start script command to python3 from python. Hope that helps. Is Vercel ok with both including flask or only with the nextjs part?
…
On Tue, 21 Mar 2023 at 16:28, faksonbaba @.> wrote: Update: it worked on Linux. I don't know why. Are you running it locally on your system? I have still not got it to work. any help, anyone! Yes, it ran locally. Try to find all places to add the API keys and use 3 terminal windows just to be safe. Any advice on how to host it publicly with Vercel or something else? What's the simplest way? I tried running vercel from the root directory but got a 404 error. Not sure what changes have to be made. My plan was to deploy the flask app as a container and then host the Nextjs app on vercel. Do you remember the steps you took to make it work, what changed , where were you running it before? We have exactly the same problem. — Reply to this email directly, view it on GitHub <#159 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADGDEVWUB5K7YQV7HXKTXO3W5HJJLANCNFSM6AAAAAAVHVPTQI . You are receiving this because you were mentioned.Message ID: @.>

For me it is easier to wrap my head around deploying it as a container.
I am almost giving up on this repo and @isafulf does not seem intrested

If the maintainer sees this please add a quick guide to deploy with Vercel.
On Tue, 21 Mar 2023 at 18:15, faksonbaba @.***> wrote:

I tried mac without success. Don’t know the cause. I set an openai api key
inside the yaml config file too. Not sure that was there before. Changed
the start script command to python3 from python. Hope that helps. Is Vercel
ok with both including flask or only with the nextjs part?
… <#m_-1108222687762179029_>
On Tue, 21 Mar 2023 at 16:28, faksonbaba @.> wrote: Update: it worked
on Linux. I don't know why. Are you running it locally on your system? I
have still not got it to work. any help, anyone! Yes, it ran locally. Try
to find all places to add the API keys and use 3 terminal windows just to
be safe. Any advice on how to host it publicly with Vercel or something
else? What's the simplest way? I tried running vercel from the root
directory but got a 404 error. Not sure what changes have to be made. My
plan was to deploy the flask app as a container and then host the Nextjs
app on vercel. Do you remember the steps you took to make it work, what
changed , where were you running it before? We have exactly the same
problem. — Reply to this email directly, view it on GitHub <#159 (comment)
https://github.com/openai/openai-cookbook/issues/159#issuecomment-1478172286>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADGDEVWUB5K7YQV7HXKTXO3W5HJJLANCNFSM6AAAAAAVHVPTQI
https://github.com/notifications/unsubscribe-auth/ADGDEVWUB5K7YQV7HXKTXO3W5HJJLANCNFSM6AAAAAAVHVPTQI
. You are receiving this because you were mentioned.Message ID: @.>
For me it is easier to wrap my head around deploying it as a container.
I am almost giving up on this repo and @isafulf
https://github.com/isafulf does not seem intrested
—
Reply to this email directly, view it on GitHub
https://github.com/openai/openai-cookbook/issues/159#issuecomment-1478377997,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADGDEVWREZQZHUMTH2GND23W5HV5FANCNFSM6AAAAAAVHVPTQI
.
You are receiving this because you were mentioned.Message ID:
@.***>

I get a similar error when trying to get an answer from the successfully uploaded document.

Im curious if anyone has gotten around this? When uploading larger document or more than one document it breaks here.

Has anybody gotten this to work?  I am trying to run this on Linux, but am getting an Axios ERR_CONNECTION_REFUSED when trying to upload a document.  How do I get around this error? Also, @isafulf mentioned editing the server/config.py file with our own information.  What needs to be edited in this file?


Full crash with 2.6 and Blender 2.92

I'm not sure about what's happening the bug, but it's doing a full crash on the scene, before simplifying it I had a different error, but after simplifying it it gives a full blender crash.
I'll send you @Dade916 the file through a PM in the forum since it's part of a project.
To reproduce, get the fil I will send you, open it and press F12, @Theverat thinks is a LuxCore bug more than a BlendLuxCore bug, that's why I'm sending it to you directly

Update: I tested with another complex scene and I got a total crash too, this is under Linux Mint 19.3
LuxCore 2.5 works perfectly fine

I get a lot of warnings for a missing p4_base.blend but I'm assume it still triggers the problem.
It is a crash inside OpenImageIO while reading the first TIFF image. Actual, it is a crash replicable with any of the TIFF in this .blend.
What is really strange is that using the same TIFF in a simple scene (i.e. hand written .cfg/.scn) works fine. This puzzle me.
Looking at the debug trace of the crash, it looks similar to another problem I have patched in our version of OpenImageIO for Linux.
Can you try if the same scene works fine on Windows with v2.6 ? It should if it is the same problem.

Ok, I’ll test it and come back 👍

It does not crash on windows, it works.

This confirm my hypothesis, I will look for a solution but it is actually more a OpenImageIO bug than LuxCore one.

Is this related to TIFF images only or it’s a general thing?
I’m asking this to find a workaround if it’s possible :)

The problem you reported seems related only to TIFF files for the moment.

If you convert the 4 TIFF files in your scene to PNG, it works fine here.

I’ll test again, I just deleted the tiff and it crashed, but may be some kind of cache I suspect, I’ll test it and report back, if that’s the workaround then we have a workaround :)

I can confirm it, it's a TIFF problem, I repeated the test and made sure that I had no tiff files around, and bingo, it worked fine, so that's the problem for sure, I can also confirm it from my end :)

Might be related to #345

Totally

I should have fixed this general problem with OpenImageIO and Linux GCC regular expressions (by switching to Boost regex). It was affecting many part of the OIIO code.

I’ll confirm this on my side, but it will probably work since you tested it also with our file :)
Awesome!


WIF SSO (Single SIgn ON) support integration and TAG/Category Search Support

We have added support for Windows Identity Foundation authentication via Federated Authentication and Passive Redirect.
Included is an example Security Token Service WebSite Issuer.  There is a rough draft setup document in the solution folder "Dev Documentation".  Getting it running should be as simple as creating some certificates and setting appropriate web.config settings on MVCForums web.config, all in the documentation.
Contact me with any concerns/questions.

Thanks for the request. I think we'll leave this one for the time being.


Fix for Drop Down List Dialog does not keep default value for Integer type

For static drop down lists, when setting the default value, it was being saved in the database, but when comparing to the values, the values were not being cast to an integer and thus it was comparing strings against integers and thinking the default value did not match. This PR will ensure values are cast to the proper data_type during the default_value comparison.
https://bugzilla.redhat.com/show_bug.cgi?id=1463827
@miq-bot add_label bug, fine/yes
@miq-bot assign @gmcculloug
@gmcculloug This bug was reported on 5.8 and I don't see any clones, I'm adding the fine/yes label but I don't know if we want to add the euwe/yes label as well? Please advise.

@eclarizio Please review failing test

Fine backport details:
$ git log -1
commit c85ed6ce9cc962a109538a42daff3f5dc4eaebd5
Author: Greg McCullough<EMAIL_ADDRESS>Date:   Tue Jun 27 14:21:22 2017 -0400

    Merge pull request #15455 from eclarizio/BZ1463827

    Fix for Drop Down List Dialog does not keep default value for Integer type
    (cherry picked from commit 65ae59ebd2bb9396f1b5e14f0755350f1b9b2776)

    https://bugzilla.redhat.com/show_bug.cgi?id=1468370

@eclarizio Missed you initiall question in the description.  Looks like we need this PR to support back-porting to https://github.com/ManageIQ/manageiq/pull/15456 to Euwe.  Unless you know of a reason this should not be back ported please add the euwe/yes label.

@miq-bot add_label euwe/yes

Euwe backport details:
$ git log -1
commit 9dcd82a08f8b7d4eb861f53034f9afd3802da338
Author: Greg McCullough<EMAIL_ADDRESS>Date:   Tue Jun 27 14:21:22 2017 -0400

    Merge pull request #15455 from eclarizio/BZ1463827

    Fix for Drop Down List Dialog does not keep default value for Integer type
    (cherry picked from commit 65ae59ebd2bb9396f1b5e14f0755350f1b9b2776)

    https://bugzilla.redhat.com/show_bug.cgi?id=1475020


An alert show "Asset load error: "N.Wire" unavailable after asset load."

Asset load error: "N.Wire" unavailable after asset load.
cant load any fonts in your website

Dupe of #664. Please continue there if still a problem.


Feature Request: Report of reviews needed

As a Cub Scout leader, I find it very frustrating that my cubs don't review their leads and assists or the finished milestones at home. I'd really like a report that shows leads, assists and milestones that are missing the review so that I can easily work out who still needs to do them. Currently I go in to each Cub's terrain and check to see if they have and leads/assists needing reviews and then I write them down. This is time consuming with a large Unit.

Nice one, this happens to me a lot too. I will add it in for the next versions new features.

Might see if we can also add a feature to be able to fill out and submit their review on the same screen in case people want to do the reviews in bulk on a section night.


Invalid smithy-build.json causes red-herring error about model files missing

This probably doesn't happen in all cases, but:

git clone https://github.com/awslabs/smithy-rs
Edit build.gradle.kts in aws/sdk-adhoc-test/` to put invalid JSON in smithy-build.json
./gradlew :aws:sdk-adhoc-test:assemble

2: Task failed with an exception.
-----------
* What went wrong:
software/amazon/smithy/model/shapes/ShapeId
> software.amazon.smithy.model.shapes.ShapeId

The error message returned in newer versions of the plugin (0.8.0+).
When I try the following invalid smithy-build.json (which caused the error in the smithy-rs example you provided):
{
    "version": "1.0",
    "projections": {
        "apigateway": {
            "imports": [
                "models/apigateway-rules.smithy"
            ],
            "plugins": {
                "rust-client-codegen": {
                    "runtimeConfig": {
                        "relativePath": "/Volumes/workplace/smithy-rs/rust-runtime"
                    },
                    "codegen": {
                        "includeFluentClient": false,
                        "nullabilityCheckMode": "CLIENT_CAREFUL
                    },
                    "service": "com.amazonaws.apigateway#BackplaneControlService",
                    "module": "apigateway",
                    "moduleVersion": "0.0.1",
                    "moduleDescription": "test",
                    "moduleAuthors": [
<EMAIL_ADDRESS>                    ],
                    "customizationConfig": {
                        "awsSdk": {
                            "generateReadme": false,
                            "requireEndpointResolver": false
                        }
                    }
                }
            }
        }
    }
}

With the new version of the plugin I get the following error message with specific info on the invalid syntax:
* What went wrong:
Execution failed for task ':smithyBuild'.
> Failed to calculate the value of task ':smithyBuild' property 'outputDir'.
   > Failed to query the value of extension 'smithy' property 'outputDirectory'.
      > Error parsing JSON: Expected valid string character ([15, 64])

This is because the newer version of the plugin executes a Node parser on the smithy build files before executing the build task see: here


"Send to a hoster" button

Hi,
Could you add a button to transfer to an hoster like mega or uptobox etc.. ?
That could be nice ! I hope so, now do whatever you want :D
Thanks !

PR welcome.

Added Heroku.


bulk update copyright - issue #3930

using vs code add msft copyright header everywhere the language server suggested across the whole solution.
closes #3930

@LittleLittleCloud
I noticed that the remediation proposed by vs code changed the headers on the files that already had some and remove the lines that had the file name in comments. I'm not sure why those there there or if some tooling needs it. Also unclear on importance of difference between "Microsoft Corporation" and "Microsoft" - I just accepted whatever VS Code proposed. Let me know if it needs to be more tuned.

@rysweet
You might need to also update this .editorconfig?
https://github.com/microsoft/autogen/blob/525546f45dd613eaedf39f2b08bf384865f19e83/dotnet/.editorconfig#L197

@rysweet Are you still going to merge this?

@rysweet Are you still going to merge this?

@LittleLittleCloud - do you know if there is official devdiv or other guidance on what we should use here?

Perhaps we should also update to include the license then?

Yes that make sense

can reopen as a new PR later after we agree on the header


America/La_Paz timezone comes back as BOT (Bogota)

Moment().tz('America/La_Paz').format('z') => 'BOT'
Bogota is -5, while La Paz is -4 GMT.
This happens both in nodeJS as well as in the browser.

Please file this in the moment-timezone repository


Command S doesn't work in a python file

Issue Type: Bug
Even with extensions disabled, command s to save an individual file doesn't work in a python file. Seems to work in everything else.
VS Code version: Code 1.34.0 (a622c65b2c713c890fcf4fbf07cf34049d5fe758, 2019-05-15T21:59:22.738Z)
OS version: Darwin x64 18.6.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7920HQ CPU @ 3.10GHz (8 x 3100)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
7, 16, 11

Memory (System)
16.00GB (2.61GB free)

Process Argv
--disable-extensions .

Screen Reader
no

VM
0%

Extensions disabled

Does it work if you disable the python extension?

Hi,
Even if I disable all extensions and/or run with —disable-extensions options, it doesn’t work for python files.

On May 23, 2019, at 1:49 PM, Alex Ross<EMAIL_ADDRESS>wrote:
Does it work if you disable the python extension?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/microsoft/vscode/issues/74165?email_source=notifications&email_token=AA3WQOVNPLMXRA6ZZIWH7V3PW3KJDA5CNFSM4HOYGILKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWC7IMY#issuecomment-495318067, or mute the thread https://github.com/notifications/unsubscribe-auth/AA3WQOTZ3KTZ2NPBX6Q4C5DPW3KJDANCNFSM4HOYGILA.

Can you try to reproduce with our nightly insider builds? You can give our preview releases a try from: https://code.visualstudio.com/insiders/

Hi Alex,
The problem doesn’t occur with that version, even with the four extensions that I have installed on my regular vscode.
I’m curious, did you have other reports of this? And when do you think it will make it into the official release?
Thanks so much for looking into this!
Stacey

On May 24, 2019, at 5:22 AM, Benjamin Pasero<EMAIL_ADDRESS>wrote:
Can you try to reproduce with our nightly insider builds? You can give our preview releases a try from: https://code.visualstudio.com/insiders/ https://code.visualstudio.com/insiders/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/microsoft/vscode/issues/74165?email_source=notifications&email_token=AA3WQORUUBXQZ64SOEIO2JDPW6XU5A5CNFSM4HOYGILKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWEU6IA#issuecomment-495537952, or mute the thread https://github.com/notifications/unsubscribe-auth/AA3WQOVCJXKO7XMT7CTUDU3PW6XU5ANCNFSM4HOYGILA.

Ok you can continue using insiders until it becomes stable in around 2 weeks.

Hi Alex,
I’ve been using the regular version of vscode, not the insider version. It seemed like the problem went away for a while but I’m seeing it again as of today.
Stacey

On May 24, 2019, at 6:29 AM, Stacey Sern<EMAIL_ADDRESS>wrote:
Hi Alex,
The problem doesn’t occur with that version, even with the four extensions that I have installed on my regular vscode.
I’m curious, did you have other reports of this? And when do you think it will make it into the official release?
Thanks so much for looking into this!
Stacey

On May 24, 2019, at 5:22 AM, Benjamin Pasero<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:
Can you try to reproduce with our nightly insider builds? You can give our preview releases a try from: https://code.visualstudio.com/insiders/ https://code.visualstudio.com/insiders/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/microsoft/vscode/issues/74165?email_source=notifications&email_token=AA3WQORUUBXQZ64SOEIO2JDPW6XU5A5CNFSM4HOYGILKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWEU6IA#issuecomment-495537952, or mute the thread https://github.com/notifications/unsubscribe-auth/AA3WQOVCJXKO7XMT7CTUDU3PW6XU5ANCNFSM4HOYGILA.


feat(avm): add temporary sha256 execution

Please read contributing guidelines and remove this line.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#6604  👈
#6596
#6504
#6452
master

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

Join @IlyasRidhuan and the rest of your teammates on  Graphite

Benchmark results
Metrics with a significant change:

app_circuit_proving_time_in_ms (Test:emit_nullifier): 1,651 (-24%)
app_circuit_proving_time_in_ms (FPC:fee_entrypoint_public): 9,596 (-35%)
protocol_circuit_witness_generation_time_in_ms (private-kernel-tail): 3,680 (+36%)
protocol_circuit_witness_generation_time_in_ms (base-parity): 1,777 (+93%)
protocol_circuit_witness_generation_time_in_ms (public-kernel-app-logic): 253 (+76%)
protocol_circuit_proving_time_in_ms (public-kernel-tail): 862 (-21%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Proof generation
Each column represents the number of threads used in proof generation.

Metric
1 threads
4 threads
16 threads
32 threads
64 threads

proof_construction_time_sha256
5,602 (-2%)
1,522 (-2%)
707 (-1%)
767 (-3%)
771 (+1%)

L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
64 txs

l1_rollup_calldata_size_in_bytes
772
772
772

l1_rollup_calldata_gas
6,844
6,868
6,880

l1_rollup_execution_gas
587,384
587,408
587,420

l2_block_processing_time_in_ms
1,329 (-2%)
4,911 (-4%)
9,801 (-3%)

l2_block_building_time_in_ms
31,794 (-1%)
124,384 (-2%)
248,731 (-2%)

l2_block_rollup_simulation_time_in_ms
31,623 (-1%)
123,766 (-2%)
247,507 (-2%)

l2_block_public_tx_process_time_in_ms
16,508 (-1%)
68,532 (-2%)
138,743 (-2%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
14,757 (-3%)
27,463 (-3%)

node_database_size_in_bytes
21,192,784
37,883,984

pxe_database_size_in_bytes
29,868
12,535

Circuits stats
Stats on running time and I/O sizes collected for every kernel circuit run across all benchmarks.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
512 leaves
1024 leaves
2048 leaves
4096 leaves
32 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.8 (-2%)
17.6 (-2%)
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.7
31.8
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.629 (-2%)
0.540 (-2%)
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
50.0 (-3%)
79.6 (-2%)
261
487 (-4%)
959 (-3%)
1,901 (-3%)
N/A

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
95.9
159
543
1,055
2,079
4,127
N/A

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.512 (-3%)
0.491 (-2%)
0.472
0.455 (-4%)
0.454 (-3%)
0.454 (-3%)
N/A

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
60.0 (-2%)
116 (-3%)
367 (-3%)
715 (-3%)
1,423 (-3%)
2,838 (-3%)
N/A

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
106
208
692
1,363
2,707
5,395
N/A

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.523 (-2%)
0.520 (-3%)
0.499 (-2%)
0.492 (-3%)
0.493 (-3%)
0.493 (-3%)
N/A

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
65.0 (-3%)

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
108

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
0.570 (-3%)

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes
1 registered classes

tx_size_in_bytes
84,613
664,966

Transaction size based on fee payment method
| Metric |  |
| - |  |
Transaction processing duration by data writes.

Metric
0 new note hashes
1 new note hashes
2 new note hashes

tx_pxe_processing_time_ms
29,117 (-2%)
4,115 (-3%)
135,481 (+2%)

Metric
0 public data writes
1 public data writes
2 public data writes
3 public data writes
4 public data writes
8 public data writes

tx_sequencer_processing_time_ms
1,334
2,149 (-3%)
1,486
3,881 (-1%)
1,647
1,929


Fix DOMMatrix.transformPoint()

Hi Jarek!
The specification does not name a copy-constructor; DOMPoint doesn't implement one, leading to a broken DOMMatrix.transformPoint().
Anyway, the intended local clone of the DOMPoint is never written to - we can just remove it.

Thanks, this should be fixed in version 1.0.5.


Cant retrieve first referring campaign through Branch.getFirstReferringBranchUniversalObject();

Hey guys,
somehow there is a problem in our app retrieving the first referring universal object. It always seems to be empty. The dashboard looks like this:

As you can see, it resets to another campaign i previously used (i dont even know the time i used it the last time). But this shouldnt be the proper behaviour of the first referring link, right?
We are using the latest sdk from here, v0.3.3.

Hey @Acimaz - Your dashboard looks like it's properly registering a referred session from that link in the far right hand column. However, you're saying that in the app, you're receiving no data when you opened up the app at 15:29:10?
Are you sure that's you and not someone else testing with that link at the time?

Can you also send some client side logs using setDebug before you call initSession?

Hey @aaustin, yes im sure about it because noone else knows about that marketing link at the moment.
Because our app is really big it will take some time, but when im ready ill provide you with some logs!

Hi @Acimaz , you should to update Unity SDK.
Important notes:

you need to add into manifest into tag "application" that:
android:name="io.branch.referral.BranchApp"
If you use your own Application class - ping me.

we have extended UnityPlayerActivity, and you need to replace "activity" head with that:

if you use your custom extended activity - ping me.

After your changes, try please your application.
Also, you can find in our repo in folder Examples "native" apps for iOS and Android.
You can try that application and test them.
It will be faster and easier for all, because we all need test app that we can try then change etc :)
Anyway, you know better what you will do :)

@antonargunov Hey anton, thanks for the info! I was on holiday, so i couldnt write :)
As soon as we are going to make an update for our app i will update the SDK as well. At the moment everything is working smooth, thats why i wont update it for a while ^^

I'm going to close this out assuming all is well!


issue with Fioti toolbar on m-mode

Hi,
When using firori tool bar component on m-mode buttons does not reneder properly (see attached picture)

Fixed and available for download.


Protect against multiple AVR resources referencing the same user PlacementRule

It is possible that a user may create more than one AVR resource that points to a single user PlacementRule. This would cause reconciliation of the 2 instances to incorrectly manage the user PlacementRule.
Although this is a user error, the AVR reconciler needs to protect against such misconfiguration.
One scheme could be to add an annotation on ownership to the user PlacementRule with a unique value (UID of AVR resource that firsts owns the PlacementRule), and check if this is the same across reconciliations and error out for duplicate AVR resources that point to the same PlacementRule.

Veera and I thought about this couple of weeks ago.  We decided at that time that would be a user error and we didn’t want to solve it at that point.  But annotation is definitely one way of doing it.

On Jun 7, 2021, at 7:59 AM, ShyamsundarR @.***> wrote:
It is possible that a user may create more than one AVR resource that points to a single user PlacementRule. This would cause reconciliation of the 2 instances to incorrectly manage the user PlacementRule.
Although this is a user error, the AVR reconciler needs to protect against such misconfiguration.
One scheme could be to add an annotation on ownership to the user PlacementRule with a unique value (UID of AVR resource that firsts owns the PlacementRule), and check if this is the same across reconciliations and error out for duplicate AVR resources that point to the same PlacementRule.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/RamenDR/ramen/issues/62, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJED3ECTHZ37THW2RHP4VKTTRSYD3ANCNFSM46HQKJNA.

This is fixed


Unable to build cli/v0.8.1

Hello, I attempted to install the Plandex CLI by following the provided steps. However, I encountered an error during the build process:
2024-04-03 22:21:31 egasdad@DESKTOP-QD0B380:~/git/plandex/app/cli$ go build term/select.go:8:2<EMAIL_ADDRESS>replacement directory ../../../survey does not exist
Additionally, here is the version of Go I am using:
2024-04-03 22:21:34 egasdad@DESKTOP-QD0B380:~/git/plandex/app/cli$ go version go version go1.21.3 linux/amd64
commit:
2024-04-03 22:21:44 egasdad@DESKTOP-QD0B380:~/git/plandex/app/cli$ git show commit 1d0d8e0d495ec454a5170cf9d2a7157f76ee793a (HEAD -> main, tag: cli/v0.8.1, origin/main, origin/HEAD) Author: Dane Schneider<EMAIL_ADDRESS>Date:   Wed Apr 3 07:52:20 2024 -0700 Bump version to 0.8.1
Could anyone provide assistance or guidance on resolving this issue?

Ah, that's because a required dependency (https://github.com/plandex-ai/survey) is missing from the build instructions. It should be cloned into the same directory as this repo, so that plandex and survey are sibling directories:
git clone https://github.com/plandex-ai/plandex.git
git clone https://github.com/plandex-ai/survey.git
cd plandex/app/cli
go build -ldflags "-X plandex/version.Version=$(cat version.txt)"
mv plandex /usr/local/bin # adapt as needed for your system

I'll update the README.md

@danenania thank you!


move: versioned compilation 1/n

Description
This PR adds the first set of code changes to compile Move package dependencies against previous compiler versions (by inspecting if Move.lock files specify that such packages should be compiled with a previous compiler). AKA: toolchain versioning. Important points for this PR:

All behavioral changes are guarded by an environment variable VERIFIED_SOURCE_BUILD. This is so that there can be incremental progress / changes without batching a whole lot of unguarded logic. The changes to our build model changes significantly and we need a toggle to test existing behavior against new planned behavior.

This PR does not contain tests because it does not enable the main behavioral changes yet (and also guarded, by env variable as above). I ran some locally sanity checks, but need to include more changes to test various parts. Currently, it's possible to build move packages but not test / publish with this flow yet. It's unclear how we'll test end-to-end behavior (we either need to avoid downloading binaries for testing, because of flakiness, so far checking in a binary for testing may be best, but it is a large binary, which is not ideal. This to be decided).

Next up:

Add changes allowing sui client publish to work with toolchain versioning

More comments inline
Test Plan
Local testing with VERIFIED_SOURCE_BUILD=1 sui move build. Here's an example compiling kiosk with manually changing a dependency to target the 1.15.1 compiler:

Known limitations: VERIFIED_SOURCE_BUILD=1sui client publish does not work yet.

If your changes are not user-facing and do not break anything, you can skip the following section. Otherwise, please briefly describe what has changed under the Release Notes section.
Type of Change (Check all that apply)

[ ] protocol change
[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

Converting to draft--going to put up a different PR that moves this logic to the sui-source-validation crate


Bot reply

Hi,
as mentioned in #91, I implemented a reply of the bot on incoming media. There is a new config file parameter botReply for it. ( :+1: :movie_camera: for videos, :+1: :camera_flash: for images)
Since the bot.js changed pretty much in order to handle documents as gifs, I created this PR to this feature branch #85 . It is a very minimalistic implementation, which can simply be merged into that branch.
@gegu @LoiX07, I'd need your help for the vienna slang and french strings :)

BTW, how are missing strings in the language files are handled? No fallback to English for now, right?

Nice idea :+1
But I think we should differentiate between interface and bot language. The sender does not necessarily have to use the same language as configured for TeleFrame.
In my opinion, it would be better to define the language files for bot communication separately.
I'll take a look at this.

I'll have a look for the French translation !

Done

Why did you close this PR?

I am sorry. The PR was closed when I merged feature/document-assets to develop.
But I'm looking at it right now. I'll get the changes from your repo.

Okay :smile: Don't use my fork's develop branch, but the bot-reply branch


Opening bloxstrap for the first time just opens it then closes it

Seen it opening on task manager then closing a second later, I checked logs and the only thing i see is this

fyi, successful winget install and opening bloxstrap still does nothing, this log is coming from my retry with the github installer (.NET 6 is installed)

Looks like it's definitely related to the user interface in some way. Can I quickly check which version of the runtime you have installed?

dang, there are multiple old ones that i think came from me installing other stuff requiring .NET, do i need to uninstall the old ones?

I don't think they should affect anything, but if you want to try and uninstall just the first one, go ahead.

done, unless i need to restart bloxstrap still crashes

Let me try and obtain the exit code that Bloxstrap returns when it crashes. Open the command prompt and run these two commands:
start /wait %localappdata%\Bloxstrap\Bloxstrap.exe -menu
echo %errorlevel%

output was -532462766

That's 0xe0434352 in hex, which is the exact same error code as other people were having, which looks like another runtime problem.
Can you open the Windows Event Viewer, go to Windows Logs -> Application, and see if there's any recent error logs?

aha! i found the error log:
Application: Bloxstrap.exe CoreCLR Version: 6.0.1623.17311 .NET Version: 6.0.16 Description: The process was terminated due to an unhandled exception. Exception Info: System.IO.FileNotFoundException: The system cannot find the file specified. (0x80070002)    at Wpf.Ui.Interop.Dwmapi.DwmGetColorizationParameters(DWMCOLORIZATIONPARAMS& dwParameters)    at Wpf.Ui.Interop.UnsafeNativeMethods.GetDwmColor()    at Wpf.Ui.Appearance.Accent.GetColorizationColor()    at Wpf.Ui.Appearance.Accent.ApplySystemAccent()    at Wpf.Ui.Mvvm.Services.ThemeService.SetSystemAccent()    at Bloxstrap.Dialogs.FluentDialog..ctor()    at Bloxstrap.Enums.BootstrapperStyleEx.GetNew(BootstrapperStyle bootstrapperStyle)    at Bloxstrap.App.GlobalExceptionHandler(Object sender, DispatcherUnhandledExceptionEventArgs e)    at System.Windows.Threading.Dispatcher.CatchException(Exception e)    at System.Windows.Threading.Dispatcher.CatchExceptionStatic(Object source, Exception e)    at System.Windows.Threading.ExceptionWrapper.CatchException(Object source, Exception e, Delegate catchHandler)    at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)    at System.Windows.Threading.Dispatcher.LegacyInvokeImpl(DispatcherPriority priority, TimeSpan timeout, Delegate method, Object args, Int32 numArgs)    at MS.Win32.HwndSubclass.SubclassWndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam)    at MS.Win32.UnsafeNativeMethods.DispatchMessage(MSG& msg)    at System.Windows.Threading.Dispatcher.PushFrameImpl(DispatcherFrame frame)    at System.Windows.Threading.Dispatcher.PushFrame(DispatcherFrame frame)    at System.Windows.Threading.Dispatcher.Run()    at System.Windows.Application.RunDispatcher(Object ignore)    at System.Windows.Application.RunInternal(Window window)    at System.Windows.Application.Run()    at Bloxstrap.App.Main()

Well, I am at a loss for words right now. Somehow Windows is just not able to find dwmapi.dll. What does doing where dwmapi.dll in the Command Prompt show?

it seems to be fine

Looks like this has been a problem for this library going as far back as July 2022 (lepoco/wpfui#275). I can try and make a workaround for this in the upcoming update, but for now, I can suggest a fairly hacky solution. You won't be able to open the menu, but you should at the very least be able to launch Roblox.
Edit Settings.json in the folder, and change
"BootstrapperStyle": 4

to
"BootstrapperStyle": 3

If you need to configure Bloxstrap, looking through the JSON file should hopefully be clear enough. Ask me if you need help with anything.

Alright, I'll do that when free and will inform you if it works before closing. Thanks for your help and quick response! Much appreciated.

Don't close this, I'll keep this open until the fix for this has actually been released. That's usually what I do.

The workaround worked, thanks!

If you're still there, I'd like to see if this build works for you. Just try and run it, see if anything happens.
https://pizzaboxer.xyz/files/bloxstrap-debug/170/Bloxstrap.exe

I just verified that this should be fixed as #173 had this same problem, but just want to check here to be sure.

Lemme check all this real quick

I think this is because bootstrapper style being valued 3, I'll try with four real quick

That worked!

Glad to say that your fix works now, I read the issue thread that you referred and replaced the bloxstrap exe

Awesome, I'll close this now since I have this marked in the 2.3.0 todo.


⚠ Secret Website 3 has degraded performance

In a66c461, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 2431 ms

Resolved: Secret Website 3 performance has improved in a534903.


python310Packages.einops: remove seemingly unused test dependencies

The amount of successful run and skipped tests stays the same and one warning is removed.
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 259433 run on x86_64-linux 1

12 packages marked as broken and skipped:

python310Packages.dalle-mini
python310Packages.dalle-mini.dist
python310Packages.elegy
python310Packages.elegy.dist
python310Packages.treex
python310Packages.treex.dist
python311Packages.dalle-mini
python311Packages.dalle-mini.dist
python311Packages.elegy
python311Packages.elegy.dist
python311Packages.treex
python311Packages.treex.dist

4 packages failed to build:

python310Packages.bambi
python310Packages.bambi.dist
python310Packages.pymc
python310Packages.pymc.dist

26 packages built:

python310Packages.arviz
python310Packages.arviz.dist
python310Packages.augmax
python310Packages.augmax.dist
python310Packages.einops
python310Packages.einops.dist
python310Packages.encodec
python310Packages.encodec.dist
python310Packages.k-diffusion
python310Packages.k-diffusion.dist
python310Packages.rotary-embedding-torch
python310Packages.rotary-embedding-torch.dist
python310Packages.xarray-einstats
python310Packages.xarray-einstats.dist
python311Packages.augmax
python311Packages.augmax.dist
python311Packages.einops
python311Packages.einops.dist
python311Packages.encodec
python311Packages.encodec.dist
python311Packages.k-diffusion
python311Packages.k-diffusion.dist
python311Packages.rotary-embedding-torch
python311Packages.rotary-embedding-torch.dist
tts
tts.dist

Hi, during conflict resolution for staging-next I added back jupyter due to test usage. Could be related to other changes in that branch.
https://github.com/NixOS/nixpkgs/pull/257792#issuecomment-1751933671


Hard to use keyboard for direct input

When providing both a datetime and an onChange as props, it is just about impossible to use the keyboard to change the value.
Say for instance that the field has a value 1979-11-18 9:17:25 AM, and you just want to change the month from 11 to 3. I would place the cursor behind 11, press backspace, and then immediately it would validate, re-bind the validated value, and the cursor jumps to the end...
I see that internally you set value={..} on the input field.. Have you thought about using defaultValue instead?

Hey !
I just fixed that with 0.0.13


Allow authentication token to be passed to React components

For our use case, we would like to generate an authentication token on our Node backend, then pass it to our React frontend to be used with the AdvancedImage react component.
It's a similar as this one from the legacy React package:
https://github.com/cloudinary/cloudinary-react/issues/242
Ideally it would look something like this:
  <AdvancedImage cldImg={img} plugins={[responsive(), accessibility()]} token={token} />

Hi Chuck,
Thanks for contacting us.
We've created a feature request for our SDK team to consider when planning the future roadmap for the SDK.
If there are any updates or if we have an ETA by which this could be added, we'll add a note here with the details.
In the meantime, alternatives for using Access Control are to enable delivery via your own hostname, so the token can be sent in a cookie, or create the delivery URLs using another method (such as the js-url-gen project) and then adding the URLs via another method.
It may also be possible for you to fork the SDK and add support for this yourself, though we generally recommend using the published SDK versions, because if a bug are found in an SDK version, we'll need to make a new release to resolve ot and your own changes could be incompatible with the new version, preventing you from upgrading to resolve the bug.
Regards,
Stephen

Hi Chuck,
Following up on this, the @cloudinary/url-gen library was recently updated to allow passing query string parameters to the URLs it creates, which will allow you to pass tokens when loading assets via our frontend frameworks.
If you update your dependencies to use 1.10.0 of that library, you can pass queryParams as an option when instantiating CloudinaryImage, and the parameters will be added to the URL used by the component.
There's a simple usage example available here, though to actually use token authentication you'll need to retrieve the token from your server-side code before this: https://codesandbox.io/s/morning-voice-vnt3in?file=/src/App.js
Regards,
Stephen

Thanks @igy
I am no longer on that team to verify this improvement, but I will make sure they're aware of this.

Hi Chuck,
Thank you for doing that. If there are any further questions or comments we can answer them here!
Regards,
John


scholarly-tauthesis:0.8.0

Adds typst version 0.12.0 support.

Also improves a few vector- and matrix-related typesetting commands and provides more example commands.

I am submitting

[ ] a new package
[x] an update for a package

A template for writing Tampere University theses.

I have read and followed the submission guidelines and, in particular, I

[x] selected a name that isn't the most obvious or canonical name for what the package does
[x] added a typst.toml file with all required keys
[x] added a README.md with documentation for my package
[x] have chosen a license and added a LICENSE file or linked one in my README.md
[x] tested my package locally on my system and it worked
[x] excluded PDFs or README images, if any, but not the LICENSE

[x] ensured that my package is licensed such that users can use and distribute the contents of its template directory without restriction, after modifying them through normal use.

I guess the checks failed, because I copied over the .gitignore and .gitattributes files into the package version subfolder?

Ok, those files were not the issue. Any idea how I could avoid doing "too many things" in this update? 🥲

I looked at why the bot did that, and it is definitely a false positive. I will try to fix the bug at some point, but for now I can just merge this PR. Thanks for the package update :)


New html tags

Hi !
I like your editor because his simlicity and "cleanness".
If you have time can you add some new buttons ? I would like sup /sup and sub /sub.
Thanks !

sub and sup was added to the core on develop branch.
Soon in a release.


Refactor api loader to decompose and optimize

Refs: https://github.com/metarhia/impress/issues/1628

[x] tests and linter show no problems (npm t)
[x] code is properly formatted (npm run fmt)

Landed in https://github.com/metarhia/impress/commit/7a70f4e4791fbb43e1c64c99417129f786fb3783


About the comparative methods

Thank you for your insightful work! In Table3, I want to know that how to perform PGD or DIM on CUB with source models pretrained on ImageNet. Thank you~

Hi @lwmming,
In this practical black-box scenario, the attacker cannot craft adversarial examples on a substitute model which is trained in the targeted domain. Therefore, we feed the images to the accessible ImageNet model and update the adversarial examples by the loss of that model.

Hi @lwmming, In this practical black-box scenario, the attacker cannot craft adversarial examples on a substitute model which is trained in the targeted domain. Therefore, we feed the images to the accessible ImageNet model and update the adversarial examples by the loss of that model.

Hi @qilong-zhang ,
Thank you so much for your immediate reply. What kind of loss do you use to update the adversarial examples when performing PGD? Is it Eq. (7) in the paper?

Hi @lwmming, In this practical black-box scenario, the attacker cannot craft adversarial examples on a substitute model which is trained in the targeted domain. Therefore, we feed the images to the accessible ImageNet model and update the adversarial examples by the loss of that model.

Hi @qilong-zhang , Thank you so much for your immediate reply. What kind of loss do you use to update the adversarial examples when performing PGD? Is it Eq. (7) in the paper?

No, Eq. (7) is only for our BIA. For PGD and DIM, we adopt the default loss, i.e., cross-entropy loss.

Hi @lwmming, In this practical black-box scenario, the attacker cannot craft adversarial examples on a substitute model which is trained in the targeted domain. Therefore, we feed the images to the accessible ImageNet model and update the adversarial examples by the loss of that model.

Hi @qilong-zhang , Thank you so much for your immediate reply. What kind of loss do you use to update the adversarial examples when performing PGD? Is it Eq. (7) in the paper?

No, Eq. (7) is only for our BIA. For PGD and DIM, we adopt the default loss, i.e., cross-entropy loss.

OK, but how to assign appropriate labels for images from CUB when adopting the cross-entropy loss to generate adversarial examples?

Hi @lwmming, In this practical black-box scenario, the attacker cannot craft adversarial examples on a substitute model which is trained in the targeted domain. Therefore, we feed the images to the accessible ImageNet model and update the adversarial examples by the loss of that model.

Hi @qilong-zhang , Thank you so much for your immediate reply. What kind of loss do you use to update the adversarial examples when performing PGD? Is it Eq. (7) in the paper?

No, Eq. (7) is only for our BIA. For PGD and DIM, we adopt the default loss, i.e., cross-entropy loss.

OK, but how to assign appropriate labels for images from CUB when adopting the cross-entropy loss to generate adversarial examples?

Using the label of clean input as the true label.

I understand. Thank you very much for your patient QA. Looking forward to your more excellent works~


fix: do not print undefined inside checkbox when list item is not a task

Problem: When the list item does not have a checkbox (no [x] nor [ ]), moving it in the timeline causes [undefined] to be printed instead of the checkbox.
Solution: When rewriting the first line of the list item, only print the "checkbox" part when there used to be a checkbox before.
Reproduction

Use the following Markdown document:
# Day planner

- 09:00 - 09:30 Sleep

Move the task in the timeline (e.g. so it starts at 9:30)

Before (on main)
There is an extra [undefined] entered where the checkbox field would normally be. This causes the list item to no longer be picked up by the plugin, and it disappears from the timeline.
# Day planner

- [undefined] 09:30 - 10:00 Sleep

https://github.com/ivan-lednev/obsidian-day-planner/assets/889383/cd750d4d-f0af-4410-84c1-de582c3ac540
After
# Day planner

- 09:30 - 10:00 Sleep

https://github.com/ivan-lednev/obsidian-day-planner/assets/889383/31910067-4a2e-4853-b676-aaf7ddfcc48e

I think this fixes https://github.com/ivan-lednev/obsidian-day-planner/issues/368? Not super familiar with github... but hopefully mentioning it here will link it.

Yes, good point! I didn't know there is an issue reported for this already.
I'll add a "closes" paragraph in the PR description so GitHub picks it up

Hello, @Gelio, thanks for the PR!


feat: #2 implement news visualization

Implement news visualization.
This should be enough at the beginning.
Close #2

Sorry I coded in the wrong branch. Not willing to rename it :)


fixes-absolute import

Resolves #272

fixes for absolute import in vs code

@arkid15r pushed the suggested changes !..lets wait fir checks to pass


update types

Summary of changes
Changes introduced in this pull request:
Reference issue to close (if applicable)

Closes

CC @1xstj you need also to delete the files after adding them in the gitignore using the git rm command.


Chapter Is Empty

Steps to reproduce
Step 1:Open LNReader App.
Step 2:Open Any Novel From Noveltop1.
Step 3:Open Any Chapter.
Expected behavior
Should Display the chapter when open instead of showing empty chapter
Actual behavior
Is not showing chapter displaying chapter is empty when in webview we can easily read any chapter
LNReader version
1.1.19
Android version
13
Device
POCO F3
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with a source, I should be opening an issue in the sources repository.
[X] I have updated the app to version 1.1.19.
[X] I will fill out all of the requested information in this form.

Wrong Place


DR - Spike - Investigate a separate terraform workspace / workspaces in another region.

User Story
As an MP DevOps Engineer
I need want to be able to have the platform & environments deployable in a different region (eu-west-1 currently)
So that if eu-west-2 goes down, we have a running environment / environments.
Value / Purpose
Required for disaster recovery in another country.
Useful Contacts
Edd Proctor
Additional Information
No response
Proposal / Unknowns
Hypothesis If we... [do a thing] Then... [this will happ]
Proposal A proposal that is something testable, don't worry whether it works or not, it's a place for ideas.
Unknowns Potential pitfalls that could cause the story to expand beyond its original scope. Ideally this section will remain blank.
Unknowns are plenty, but after a meeting and discussion we decided to investigate this rather than an entire copy of the platform.
Definition of Done
Example - [ ] Documentation has been written / updated

[ ] Example is deployed in another workspace in eu-west-1
[ ] Networking has been investigated and documented on what would need to be copied / changed
[ ] Deployment pipeline has been discussed but full changes will be under another card.
[ ] Team member has reviewed.

Will need a little range for cooker in eu-west-1. Have to think of VPCs and other things.

Created tickets off this ticket.


Functionality to split a WDS

Hi and first of all thanks for this amazing work!
In my scenario I have downloaded a small (500K) text-image pair dataset in form of a WDS stored it on an external storage container that I can mount in my system, i.e., I use a local path to instantiate the WDS.
My question now is, if there is a way to split the WDS into train, test, and validation without "manual extract/move/compress" work?
Thanks in advance!

You can either split before downloading
filter at training time
or use the tar api from python ; or the webdataset helpers to extract the required splits


winver

ℹ Computer information

PowerToys version:
PowerToy Utility:
Running PowerToys as Admin:
Windows build number: [run "winver"]

📝 Provide detailed reproduction steps (if any)

…
…
…

✔ Expected result
What is the expected result of the above steps?
❌ Actual result
What is the actual result of the above steps?
📷 Screenshots
Are there any useful screenshots? WinKey+Shift+S and then just paste them directly into the form

ℹ Computer information

PowerToys version:
PowerToy Utility:
Running PowerToys as Admin:
Windows build number: [run "winver"]

📝 Provide detailed reproduction steps (if any)

…
…
…

✔ Expected result
What is the expected result of the above steps?
❌ Actual result
What is the actual result of the above steps?
📷 Screenshots
Are there any useful screenshots? WinKey+Shift+S and then just paste them directly into the form


Do not throw button render errors when ref is invalid

This PR attempts to limit rendering errors related to the Buttons container being removed before async rendering completes. It's inspired by this PR for the zoid react driver that ships with the JS SDK: https://github.com/krakenjs/zoid/pull/335.
Related: #85

This empty container check was also needed: https://github.com/paypal/react-paypal-js/commit/b85932bd1efda453c3b3b952554558d288b67857


Error: on FVTT 11.306

When I tried to update modules today, I got an error below.

I believe this is a network error, and nothing to do with CGMP.


Improve logging

Currently we are using console.log and console.err for debugging. We want to create a debug mode where we can check logs, otherwise they shouldn't be displayed.
Research and implement the best approach you can find for this

I'm on it.

done at #102 , needs further investigation on how to:

properly log nested objects without showing too much info
implement generic logging for types that throw errors

i will like to work on it
Tell us about your background and how it can be leveraged:
I have a strong background in TypeScript, specializing in creating efficient logging utilities. My experience will help implement a robust solution that enhances maintainability and debugging in our projects.
How would you approach the problem?
Define Debug Mode: Use an environment variable to toggle debug mode.
Create a Logger: Develop a TypeScript logger that wraps console.log and checks debug mode.
Integrate the Logger: Replace existing logging calls for consistency.
Testing: Ensure logs behave as expected in both modes.

Go ahead! Assigned

Any further attempts, please check previous PRs for extra comments. One extra thing that was not mentioned in the last round of comments:

We want to be able to setup different log levels and configs to each package. The log configs should be passed in the constructor of each package

for tracking #172

hey
consider a logger like winston or pino with browser-specific handling plugins for more control over object logging and nested data.

hey @gregemax, are you going to continue this task or can i drop the assignment and close the pr?

@d-roak
should i use  winston or pino  for the logger

winston

assigning it internally


[HUDI-2994] Add judgement to existed partitionPath in the catch code block for HU…

…DI-2743
Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contribute/how-to-contribute before opening a pull request.

What is the purpose of the pull request
(For example: This pull request adds quick-start document.)
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Fix looks good to me.


Fail to download an existing blob

Storage Explorer Version: 1.13.0
Build: 20200403.17
Branch: rel/1.13.0
Platform/OS: Windows 10
Architecture: ia32
Regression From: Previous release(1.12.0)
Steps to reproduce:

Expand one storage account -> Blob Containers.
Create one new blob container -> Upload one blob to it -> Download this blob to specify folder.
Download this blob to same folder again -> Select 'Overwrite' in the conflict dialog -> Apply.
Check the result.

Expect Experience:
Successfully overwrite the blob.
Actual Experience:
Fail to download the blob

More Info:

This issue doesn't reproduce on Linux and Mac.
AzCopy Log File:
2020/04/07 09:07:35 ERR: [P#0-T#0] DOWNLOADFAILED: https://arm0402.blob.core.windows.net/test/New%20Text%20Document.txt?se=2020-04-08t07%3A04%3A13z&sig=-REDACTED-&sp=rwdlacup&srt=sco&ss=btqf&st=2020-04-07t07%3A04%3A13z&sv=2019-02-02 : 000 : File Creation Error destination file has no known flags that could cause issue (current set: 20) and azcopy was unable to clear the flag(s), so access will be denied: The process cannot access the file because it is being used by another process. Dst: %5C%5C?\C:\Users\vlimu\Downloads\donwload\DownloadBlob\New Text Document.txt

Verified this issue on build 20200408.10. Fixed.


Rich command: NewTab + DuplicateTab

Resolved / Related Issues
Here are 2 new commands. For other navigation controls, I'm going to need a context to get the current IMultitaskingControl.
There were 2 keyboard shortcuts for NewTab (Ctrl+T and Ctrl+Shift+K). The second has been deleted but can be put back if necessary.
Validation
How did you test these changes?

[x] Built and ran the app
[x] Tested the changes for accessibility

We should support both shortcuts.

Ok. I will implement the custom commands. This will allow to easily have several Hotkeys for the same action. I will update this pr after committing the custom commands. This pr is therefore pending.

There was no need for a second hotkey because I confused 2 commands. Ctrl+Shift+K is for DuplicateTab and not NewTab. Of course, the multiple hotkeys will have other uses. I updated this pr which can be reviewed.


Mapbox maps are not working

API in use have been deprecated:
https://blog.mapbox.com/deprecating-studio-classic-styles-d8892ac38cb4

https://docs.mapbox.com/help/troubleshooting/migrate-legacy-static-tiles-api/#how-can-i-tell-if-my-style-is-a-classic-style-or-a-modern-mapbox-style


Fix create physical bone when up vector is collinear to child origin

Fix issue #39757 but for Godot 4.0
When generating physical bone, set_look_at is used to change the bone direction.
body_transform.set_look_at(Vector3(0, 0, 0), child_rest.origin);
However, if the vector child_rest.origin is collinear with the default up vector =Vector3(0, 1, 0), the cross product cannot generate a third vector (with length > 0) and set_look_at fails.
In order to avoid this issue, set_look_at is now called twice, using 2 different up vectors.
body_transform.set_look_at(Vector3(0, 0, 0), child_rest.origin, Vector3(0, 1, 0));
body_transform.set_look_at(Vector3(0, 0, 0), child_rest.origin, Vector3(0, 0, 1));

Please find a reproduction project for Godot 4.0 at https://github.com/godotengine/godot/issues/39757#issuecomment-829595289

What happens if the first one succeeds and the second test fails?

Good question, I was asking myself this question too yesterday night before sleeping...
In Transform::set_look_at...
	v_z = p_eye - p_target;
	v_z.normalize();
	v_y = p_up;

	v_x = v_y.cross(v_z);
#ifdef MATH_CHECKS
	ERR_FAIL_COND(v_x.length() == 0);
#endif

... the check is done only if MATH_CHECKS is defined. Then, in case of failure, the error is thrown and the basis and origin change, which happen in the next lines, won't happen.
So if the first set_look_at succeed and the second one failed, only the first will change the direction.
However, I am wondering if MATH_CHECKS is always defined in the editor. Currently it is defined in math_defs.h:
#ifdef DEBUG_ENABLED
#define MATH_CHECKS
#endif

But if DEBUG_ENABLED is not enabled in the editor, then the vectors check must be done before using set_look_at...
After looking  it seems it is enabled only if target is release_debug or debug.
Thus, I need to change the PR...

I have changed the solution: up is now checked before being used in set_look_at. The first top comment is also changed to show exactly that.

Can you rebase? I'll try get some review on this pr.

Thank you! Just done rebasing this morning

@pouleyKetchoupp Is this something in your area?

I have created the specific issue for the DAE bones #56242 as it is a different issue compared to this one.
This PR still looks good to me, as it respect the bones after import (good bones for GLTF, wrong for DAE)


[TB] Integrate with FINOS CICD

Problem/Concern
One of the later steps of the transition would be to ensure we are making use of the appropriate CICD pipeline provided to us. More information would be required at this time.
Proposed Solution

From Mao:
I used to be a big fan of Travis, but after Idera's acquisition, all our contacts left the company and a series of security issues pushed us to switch to GitHub Actions, which is now our preferred platform; that being said, we do have projects using other tools, like CircleCI and CodeFresh (though they're planning to move to GitHub Actions).

review deploy to Netlify, scans and SDK pushing to NPM.
In previous careers, used Prod branch commits to trigger actions.  Any opinions on how to do it now?
Proposal:

keep main as prod branch
dev to be current working branch for day to day commits
merge dev into main to "promote"

trigger Netlify deploy in TB repo

we should probably publish separately to Netlify for a dev commit

trigger NPM publish in SDK repo
trigger docker image build and push to DockerHub when this is supported

tag Docker image to easily identify what came from prod and what came from dev if we decide to publish images for both

Vote: approved by @smithbk, @brycecurtis, @evangk6, @aaronreed708

@TheJuanAndOnly99 or @maoo, could one of you please update the Netlify for a11y-theme-builder to deploy for both main and dev branches now?  Maybe the dev one could be "dev.a11y-theme-builder.finos.org"?

Done! https://dev.a11y-theme-builder.finos.org/

Looks like NPM publishing is complete, the switch to dev branch development is complete and the dev deployment is complete.  Just need to finish up #481 and then we can close this issue


Add ref normalizing method to Destination

NormalizeDeprecatedObjectReference updates the Destination to the desired end state post-deprecation. Using this method allows code to ignore the deprecated fields and just operate on Destination's Ref field.
We can use this in sources to first set the Ref, then update its namespace to the current source's namespace as required by URIResolver.
I left the resolver's deprecated fields handling code in place so other projects can continue to use the deprecated fields without normalizing first.
/cc @n3wscott @Harwayne @vaikas-google

/lgtm
/approve
/hold
hold for if you agree with the name change.

/hold cancel

/close
This doesn't seem relevant anymore.


Preview is overlaid by mat-sidenav

When I use the MatSidenavModule from @angular/material2 the fullscreen mode is overlaid by the mat-sidenav components and make navigation impossible.
See https://ngx-gallery-sidenav.stackblitz.io for example.
A workaround would be to set the z-index of mat-sidenav to -1 but I don't know if this is the cleanest way to handle this issue.

mat-sidenav has z-index: 2, mat-sidenav-content which contains ngx-gallery has z-index: 1, because of that preview is under sidenav.
Imo you should listen to previewOpen event and set z-index: 1 on mat-sidenav, and of course set previous value on previewClose

Okay, seems reasonable. We don't know where the ngx-gallery is used, so it makes no sense to handle this inside the library.
But with the outputs previewOpen and previewClose this should work.
Thanks for your input!


Dynamically discover release image OS metadata

Before this commit, CoreOS metadata was hard-coded in this Git repo and required
manual maintenance to map OCP release versions to compatible AMIs. Now, the
metadata is discovered automatically as part of the general release image
metadata lookup, as the CoreOS metadata is part of the payload as of
https://github.com/openshift/installer/pull/4760.
After this commit, HyperShift will only be compatible with OCP release images
which embed the CoreOS metadata.

/test e2e-aws

/test e2e-aws

/test e2e-aws

/test e2e-aws

/lgtm


[$500] Android - Scan IOU - A broken image is downloaded to the device when receipt cannot be downloaded

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:

Launch New Expensify app.
Create a Scan IOU.
Open the IOU details view.
Tap on the receipt > overflow menu > Download.
Close the error message.
Go to the device gallery.
Note that a broken image is downloaded despite the error message.

Expected Result:
Nothing will be downloaded to the device since it shows 'Attachment cannot be downloaded' message.
Actual Result:
A broken image is downloaded to the device despite the 'Attachment cannot be downloaded' message.
Workaround:
Unknown
Platforms:

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

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

Version Number: v1.3.72-8
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://github.com/Expensify/App/assets/115492554/24dfe60d-68a1-407a-b558-1cf646334626
Expensify/Expensify Issue URL:
Issue reported by: Applause-Internal Team
Slack conversation: @
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0126f987f66d4dbcf0
Upwork Job ID:<PHONE_NUMBER>802134528
Last Price Increase: 2023-09-22

Yep, that's a bug. @izarutskaya do you know what caused it to throw the error?

Proposal
Please re-state the problem that we are trying to solve in this issue.
As soon as user try downloading file then error is thrown along with junk file stored.
What is the root cause of that problem?
Root cause exists when we try to copy file in media store which does not exist. https://github.com/Expensify/App/blob/171259c4b47eb1788570db07aa9fbcdcc37ac650/src/libs/fileDownload/index.android.js#L75-L84
This results in junk file saved.
What changes do you think we should make in order to solve the problem?
Use RNFetchBlob.fs.exists(attachmentPath) to check if file exists before trying to save the file.
What alternative solutions did you explore? (Optional)
RNFetchBlob has bugs ,we can tailor each issue like this one and fix them all.

Hi @mananjadhav,
I hope you're well. I would like to request your approval for my proposal. If you can provide the approval, I will be able to create the pull request (PR) within the next hour. Your prompt response would be greatly appreciated.
Thank you for your consideration.
Best regards,
Ayaz Alavi

I think considering the broken behavior @bernhardoj's proposal makes more sense. Tagging @shawnborton for more info.
🎀 👀 🎀 C+ reivewed.

Hmm my take is, is it possible to fix the fact that the image can't be downloaded? If not, then I agree, we should not download a broken image after seeing that error message.

@shawnborton @mananjadhav  it is android only bug. same code is getting file downloaded just fine in other platforms so basically file name convention is not a bug here. As I mentioned in my proposal it is android file system bug and needs to solved only in index.android.js. Quickest solution is to check whether file exists i.e. android file system can read the given file path. If not then dont save the file otherwise file stored will be empty. It is bug in the code it is not checking whether file exists prior to saving it in media store.

Ah then if that is the case, let's try to fix the file not getting downloaded.

@shawnborton yes I have proposed that already.

@marcaaron @sakluger please provide your input about above comments.

Well, wow, I actually don't super understand the root cause here. @bernhardoj's explanation is pretty unclear.

I don't know the reason why a broken file is saved even though the download fails.

Can we not figure this out? Also why did the download fail? What do we need to understand this problem better?

The issue is, that if we try to download the receipt image and log the image URI, it contains an auth token, so the full URI would be like file:///.....?encryptedAuthToken=asdWJWAPdad...
So, it makes sense the lib can't find the file. (the receipt image will initially be a local file and later will be replaced with an image from the server, at least on the web)

I don't really understand what you are trying to say here. Where are we logging an image URI? And why does it "make sense" that the lib (which lib exactly?) can't find the file? 😕
I'm going to kick this one back to @mananjadhav to manage for now to try again. @mananjadhav Please work with the contributors here to put forward a complete proposal that clearly describes the problem and offers a solution.

@marcaaron look at the video I attached with my proposal. Additionally copytomediastore copies existing file to media directory but if file does not exists then it will save empty file there. Android filesystem is not able to understand path provided, it is android only bug and exists due to OS incompatibility with the plugin . Simply using RNFetchBlob.fs.exists(attachmentPath) can avoid saving empty file which will sort out the issue.

@mananjadhav can you please take a look at Marc and Shawn's concerns and advise how you think we should proceed here?

@sakluger I am still catching up on my issues. This looks a bit critical, then can you please reassign someone else?

Thanks for letting me know @mananjadhav, I've reassigned to another C+.

Issue not reproducible during KI retests. (First week)

@mvtglobally can you look at my proposal, it will still prevent empty file to be saved in camera roll. Do you think we can implement that here?

@mvtglobally what happens now when you retest? Do we still show an error but no longer download the file? Or do we no longer show the error?

Can someone hold the reins here and summarize where we are at with this one? Maybe @ArekChr?

Issue not reproducible during KI retests. (Second week)

@sakluger have you looked at my proposal? It is addressing problem we are facing and providing a fix. let me know your thoughts. thanks

@bernhardoj You rightly pointed out the issue. I now understand the problem. When we operate in offline mode, a local file is stored. @marcaaron, the underlying issue is that we're concatenating the auth token to the local file's URI. This works when rendering the image because the Image component can handle the query parameter like ?authToken=bearer.token.etc. However, when the local URL includes the token, RNFetchBlob treats it as part of the file path. As a result, we get a "file not found" error. We should strip the token from the file URL during local file downloading. @bernhardoj's solution addresses this problem.

@ayazalavi I don't review any proposals as I am not an engineer - I'm just helping to manage the GH issue and keep it moving forward.
@marcaaron do you agree with @ArekChr's recommendation to go with @bernhardoj's solution?

Sounds good. Yes I agree after reading the updated proposal.

Looks like this would also fix https://github.com/Expensify/App/issues/29749, could you confirm @ArekChr or @bernhardoj?

[x] Link to the PR: No bug identified.
[x] Link to comment: n/a
[x] Link to discussion: n/a
[x] Determine if we should create a regression test for this bug: We can create regression test.

Regresion test proposal:

Open app and enable offline mode.
Create a Scan IOU.
Open the IOU details view.
Tap on the receipt > overflow menu > Download.
Ensure that the image downloaded succesfully and verify if image is valid.

Do we agree :+1: or :-1:

Proposed regression test steps look good. 👍
I've paid out @bernhardoj, so we're good to close this one. Thanks everyone! 🚀


TypeError: Cannot set property 'DeepDiff' of undefined

Using version 1.0.1, I'm getting that error thrown here.
What's weird is that the code works fine in isolation, but when dropped into my company's monolith, I get that error when running tests. Given that, my guess is that it's related to our using babel-jest, which does some pre-processing on our code.
It looks like #129 addressed this, but there might still be something up with it.

Got the same issue here on 1.0.1 in a react-native project.

Heya! Got the same issue on libraries which have this as a dependency. What workarounds/fixes did you guys settle on? @xanderberkein @xavdid

Check out my PR! https://github.com/flitbit/diff/pull/134

Heya! Quite new in js so i don't understand much how it got fixed, but I suppose the PR fixes web issues as well?
Hopefully this one gets merged and released soon! In the meantime might have to use another library unfortunately :/  (will use it at a different project hopefully!)
But thanks for replying @xanderberkein !

The PR works for any environment where root isn't defined.
You can try testing it out by importing my branch in your "main" project. All dependencies should use that version as well then:
yarn add github:xanderberkein/diff#master

@xanderberkein any chance of that getting merged here and released as part fo the original diff?

Yes, my PR has been merged! Should be part of diff since 1.0.2

Ah, perfect! I've confirmed this is resolved. Thanks so much.


Is there any schedule to provide macOS version?

Often use macOS for development, but service center only provide Windows and Linux version.

ServiceCenter 1.0.0 MacOS version is voting now.  Please take a look at this mail thread.
You can find the release candidate here

Hi @hjianhao  , if you had any question, please contact us again, thanks!


Background image poor quality

When I run the example, the background image is actually being zoomed in and the quality of image is getting poor.
Is there any way to improve it ?

@leeys888

I didn't understand the question.
Can you show me an example image and explain?


IBM-Swift/Kitura-Credentials#46 Add API to set and get session["returnTo"].

IBM-Swift/Kitura-Credentials#46
Motivation and Context
See  IBM-Swift/Kitura-Credentials#46
How Has This Been Tested?
On macOS and Linux.

Codecov Report

Merging #48 into master will increase coverage by 0.23%.
The diff coverage is 75%.

@@            Coverage Diff             @@
##           master      #48      +/-   ##
==========================================
+ Coverage   60.08%   60.31%   +0.23%
==========================================
  Files           5        5
  Lines         243      252       +9
==========================================
+ Hits          146      152       +6
- Misses         97      100       +3

Flag
Coverage Δ

#Credentials
60.31% <75%> (+0.23%)
:arrow_up:

Impacted Files
Coverage Δ

Sources/Credentials/Credentials.swift
59.03% <75%> (+0.31%)
:arrow_up:

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 9f086d7...a7940ef. Read the comment docs.


Changed date input to 'Date', added end-point for create event, creat…

Server now has endpoint '/api/create-event'

Create event now sends event creation to server

Changed create event page to better represent date and time

Date and time need to sill be fetched from creation page. Possibly add react-datetime

Great work, but I'd really appreciate it if you would go back through and configure your tab size to 4 spaces instead of 2 as the rest of the project uses 4 spaces.

Updated with 4 spaces


SO , how to get the auth object

in the Request recognizer class

I guess there is no possibility, only request are available. You can take auth token from headers for checking is request are unique

emmm.. finally resolve this. I try to put the auth into req,  in another middleware


IP Address Range from different to be supported in single IPPool

As per current CRD Spec, InClusterIPPool or GlobalInClusterIPPool supports addresses from 1 subnet only.
For eg,
Single InClusterIPPool and GlobalInClusterIPPool cannot support below two subnets,

<IP_ADDRESS>- <IP_ADDRESS> (Prefix 23)
<IP_ADDRESS> - <IP_ADDRESS> (Prefix 22)

Request you to support both subnets under single IPPool. We were able to use this feature using capm3 ippool.
apiVersion: ipam.metal3.io/v1alpha1
kind: IPPool
metadata:
name: vlan
spec:
pools:

end: <IP_ADDRESS>
prefix: 22
start: <IP_ADDRESS>
end: <IP_ADDRESS>
prefix: 23
start: <IP_ADDRESS>

The spec does support multiple ranges. The examples in the usage examples just show variants, you can provide multiple ranges or subnets.
With the current resources, you'd need to set the prefix to 14, but I guess that's not what you want.
apiVersion: ipam.cluster.x-k8s.io/v1alpha2
kind: InClusterIPPool
metadata:
  name: inclusterippool-sample
spec:
  addresses:
    - <IP_ADDRESS>-<IP_ADDRESS>
    - <IP_ADDRESS>-<IP_ADDRESS>
  prefix: 14
  gateway: <IP_ADDRESS>

I think the issue here is that we do not support different prefix lengths (and therefore also gateways) for different subnets or ranges. With the current CRD it would also be difficult to implement in a nice way. Maybe we could consider adding a list of networks as an alternative to addresses.
apiVersion: ipam.cluster.x-k8s.io/v1alpha2
kind: InClusterIPPool
metadata:
  name: inclusterippool-sample
spec:
  networks:
  - addresses:
    - <IP_ADDRESS>-<IP_ADDRESS>
    prefix: 22
    gateway: <IP_ADDRESS>
  - addresses:
    - <IP_ADDRESS>-<IP_ADDRESS>
    prefix: 23
    gateway: <IP_ADDRESS>

I'm not sure about adding this complexity here. For complicated setups like this, it might be better using an external IPAM system like Netbox or Infoblox. There is no provider for Netbox yet.
My condolences on that network setup, I thought our telco environment was bad, but this seems worse.


ECCV 2020 tutorial: Navigation Basics

Motivation and Context
This is to finalize the tutorial.

How Has This Been Tested

Types of changes

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

Checklist

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have completed my CLA (see CONTRIBUTING)
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

 It should have tried to generate the python script automatically with pre-commit @bigbike . If it didn't, you can run the script in tools to run precommit on the specified file.

The colab will appear at this link until the PR is committed btw: https://colab.research.google.com/github/facebookresearch/habitat-sim/blob/colabs-navigation/examples/tutorials/colabs/ECCV_2020_Navigation.ipynb

So I tested this at the link I posted above and it works! Feel free to give it a try.

@aclegg3, currently Habitat-API is installed after habitat-sim is in the .circleci build. That's why the test is failing. We need to either handle that import failing or install Habitat-API earlier

@aclegg3 Still two minor issues whenever you get the chance, fix the unnecessary import shim and redo to use the new make_video function. :)


Update README.md

Removed extraneous period

This is more like a short sentence than a document title, the period is not a concern. Thanks for the suggestion though.


future cross-site warnings from Firefox

FEATURE DESCRIPTION
Feature Inspiration
There are a bunch of warnings in the firefox console about the advanced search
Feature Description
Avoid the future cross-site designation in the warnings.
Alternatives Considered
none
Comment
Cookie “_gat_gtag_UA_36516510_3” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/” because the scheme does not match. tracebase-dev.princeton.edu
Cookie “_ga” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/” because the scheme does not match. tracebase-dev.princeton.edu
Cookie “_gid” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/” because the scheme does not match. tracebase-dev.princeton.edu
Cookie “_gat” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/” because the scheme does not match. tracebase-dev.princeton.edu
Cookie “_gat_gtag_UA_36516510_3” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/” because the scheme does not match. tracebase-dev.princeton.edu
Cookie “_ga” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/” because the scheme does not match. tracebase-dev.princeton.edu
Cookie “_gid” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/” because the scheme does not match. tracebase-dev.princeton.edu
Cookie “_gat” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/” because the scheme does not match. tracebase-dev.princeton.edu
Cookie “_gat_gtag_UA_36516510_3” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/static/css/sidebar.css” because the scheme does not match. sidebar.css
Cookie “_ga” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/static/css/sidebar.css” because the scheme does not match. sidebar.css
Cookie “_gid” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/static/css/sidebar.css” because the scheme does not match. sidebar.css
Cookie “_gat” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/static/css/sidebar.css” because the scheme does not match. sidebar.css
Cookie “_gat_gtag_UA_36516510_3” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/static/css/extras.css” because the scheme does not match. extras.css
Cookie “_ga” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/static/css/extras.css” because the scheme does not match. extras.css
Cookie “_gid” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/static/css/extras.css” because the scheme does not match. extras.css
Cookie “_gat” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/static/css/extras.css” because the scheme does not match. extras.css
Cookie “_gat_gtag_UA_36516510_3” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/favicon.ico” because the scheme does not match. favicon.ico
Cookie “_ga” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/favicon.ico” because the scheme does not match. favicon.ico
Cookie “_gid” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/favicon.ico” because the scheme does not match. favicon.ico
Cookie “_gat” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/favicon.ico” because the scheme does not match. favicon.ico
Cookie “_gat_gtag_UA_36516510_3” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/static/js/hierarchical_formsets.js” because the scheme does not match. hierarchical_formsets.js
Cookie “_ga” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/static/js/hierarchical_formsets.js” because the scheme does not match. hierarchical_formsets.js
Cookie “_gid” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/static/js/hierarchical_formsets.js” because the scheme does not match. hierarchical_formsets.js
Cookie “_gat” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/static/js/hierarchical_formsets.js” because the scheme does not match. hierarchical_formsets.js
Cookie “_gat_gtag_UA_36516510_3” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/favicon.ico” because the scheme does not match. favicon.ico
Cookie “_ga” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/favicon.ico” because the scheme does not match. favicon.ico
Cookie “_gid” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/favicon.ico” because the scheme does not match. favicon.ico
Cookie “_gat” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/favicon.ico” because the scheme does not match. favicon.ico
Cookie “_gat_gtag_UA_36516510_3” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/DataRepo/search_basic/Study/id/exact/2/peakgroups/” because the scheme does not match. peakgroups
Cookie “_ga” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/DataRepo/search_basic/Study/id/exact/2/peakgroups/” because the scheme does not match. peakgroups
Cookie “_gid” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/DataRepo/search_basic/Study/id/exact/2/peakgroups/” because the scheme does not match. peakgroups
Cookie “_gat” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/DataRepo/search_basic/Study/id/exact/2/peakgroups/” because the scheme does not match. peakgroups
Cookie “_gat_gtag_UA_36516510_3” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/DataRepo/search_basic/Study/id/exact/2/peakgroups/” because the scheme does not match. peakgroups
Cookie “_ga” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/DataRepo/search_basic/Study/id/exact/2/peakgroups/” because the scheme does not match. peakgroups
Cookie “_gid” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/DataRepo/search_basic/Study/id/exact/2/peakgroups/” because the scheme does not match. peakgroups
Cookie “_gat” will be soon treated as cross-site cookie against “http://tracebase-dev.princeton.edu/DataRepo/search_basic/Study/id/exact/2/peakgroups/” because the scheme does not match. peakgroups

ISSUE OWNER SECTION
Assumptions

List of assumptions made WRT the code
E.g. We will assume input is correct (explaining why there is no validation)

Requirements

List of conditions to be met for the feature
E.g. Every column/row must display a value, i.e. cannot be empty

Limitations

A list of things this work will specifically not do
E.g. This feature will only handle the most frequent use case X

Affected/Changed Components

Files
Environment variables
External executables
Database tables
Cron job settings
Etc.

DESIGN
GUI Change description
Describe changes the user will see.
Code Change Description (Pseudocode optional)
Describe code changes planned for the feature.
Tests
A test should be planned for each requirement (above), where possible.

These look like Google Analytics cookies to me. I didn't think we had any analytics code in this project. I do seem some warnings about using the "SameSite" attribute on cookies, but they don't appear to be problematic afaik. Can this ticket be closed?

If that's all this is, then yeah.


Fix :var messages

Fixes #96. @albertpod, @bartvanerp, @ismailsenoz please double check.

Codecov Report

Merging #97 (2cce59f) into master (4e2da80) will decrease coverage by 0.45%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #97      +/-   ##
==========================================
- Coverage   75.00%   74.54%   -0.46%
==========================================
  Files         172      170       -2
  Lines        5452     5379      -73
==========================================
- Hits         4089     4010      -79
- Misses       1363     1369       +6

Impacted Files
Coverage Δ

src/rules/normal_mean_variance/var.jl
100.00% <100.00%> (ø)

src/macrohelpers.jl
58.69% <0.00%> (-19.31%)
:arrow_down:

src/ReactiveMP.jl
50.00% <0.00%> (-16.67%)
:arrow_down:

src/rule.jl
54.62% <0.00%> (-8.79%)
:arrow_down:

src/score.jl
88.88% <0.00%> (-6.77%)
:arrow_down:

src/rocket.jl
77.04% <0.00%> (-1.08%)
:arrow_down:

src/rules/gcv/w.jl

src/rules/gcv/k.jl

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 4e2da80...2cce59f. Read the comment docs.


[Bug] Empty text nodes are not allowed

To Reproduce
Steps to reproduce the behavior:

set value

ctx.set( defaultValueCtx, '```js\n ```' );

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Fixed by https://github.com/Saul-Mirone/milkdown/commit/924e12ddd642af099cc4d8e422fffef3090c410f
Will be released in this week.


An operation without a return schema should default to a Stream/byte[] response type.

openapi: 3.0.3
info:
  title: app that calls graph and gets the me profile
  version: 1.0.0
paths:
  /me:
    get:
      responses:
        200:
          description: Success!

Currently kiota returns this error:
could not find operation return type Get \me

AB#10904

related #221 #134

@darrelmiller I'm guessing that an operation described with only a 204 response should return void in that case, not an empty stream, correct?

related to #302 where anything that's not a known mime type will be treated as stream. Planning to tackle both issues together.

fixed via #1471


[hmac,dv] Verify HMAC idle status

Description
The idle status of HMAC is now exposed via the STATUS CSR in bit 0 (https://github.com/lowRISC/opentitan/issues/22916). This needs to be modeled and predicted in the scoreboard and verified against the STATUS CSR whenever read. For now, I read it from the CSR and update the prediction to match it, but we do not verify it yet.

@andreaskurth I've labelled this as M5, since the driver implementation and ongoing DV work is already uncovering issues for now, so would guarantee sufficient assurance for now.

@martin-velay can provide more details, but when I look at the current hmac_scoreboard I think the idle bit of the STATUS register gets checked -- correct?

@andreaskurth, I don't think this feature has been verified as of now unfortunately.
I don't know why, but I think it was not visible on the main board, probably a tag was missing.
The PR #23749 is only checking the pin in test mode.
I'll add this check in the SCB.

It's not yet verified, we read it from the status register for now.
Issue is linked there - https://github.com/lowRISC/opentitan/blob/master/hw/ip/hmac/dv/env/hmac_scoreboard.sv#L444-L445

Closed as https://github.com/lowRISC/opentitan/pull/24037 has been merged


show all callout

i cannot show all callout of each marker. current only show callout of last marker. any suggest ? thank!

You can show only one callout at a time. This is by design. You can not show many callouts at once.