Warning on Critical Dependency

I built brand new project with a starter kit after adding EmailSignupForm (or any other) i am getting this error
material-ui-theme.js?0449:1 Uncaught TypeError: Cannot read property 'getIn' of undefined
And on my console
WARNING in ./~/encoding/lib/encoding.js
Critical dependencies:
9:12-34 the request of a dependency is an expression
@ ./~/encoding/lib/encoding.js 9:12-34

Hi @tarkanlar - thanks for taking the time to report this.
Two questions:

Is this just a warning, or is something noticeably breaking?
Does this happen with the other themes as well, or just material-ui?

Hi @lynndylanhurley  thanks for reply,
1- I tried with https://github.com/erikras/react-redux-universal-hot-example and  https://github.com/davezuko/react-redux-starter-kit
2- It was just warning
3- I tried with all themes and got the same error.

Thanks @tarkanlar, I'll investigate ASAP!

@lynndylanhurley if you replace import fetch from "node-fetch"; with import fetch from "isomorphic-fetch"; in https://github.com/lynndylanhurley/redux-auth/blob/master/src/utils/verify-auth.js#L1, it will work.
We get that warning when it tries to load node-fetch who depends on encoding for client code.

Thanks @baopham - I'll add that fix to the next release.

Hi @lynndylanhurley, I'm facing the same issue. Any update on the fix ?

+1 same error here, the node-fetch fix doesn't do the trick for me.

strange, it works for me. Did you compile the code after changing the node-fetch line?

Reloading npm start or...?

no compile redux-auth. Or manually replace node-fetch with isomorphic-fetch  in: https://raw.githubusercontent.com/lynndylanhurley/redux-auth/master/index.js

Could you give me a gist of your fixed and compiled index.js? I can't seem to be able to compile... Thanks a lot for your help!

Can you try to replace node_modules/redux-auth/index.js with this gist: https://gist.github.com/baopham/6f6ec3ac707f0838dfa3
All I did was just find and replace node-fetch in the file.

Yeah, just tried replacing it manually too in node_modules/redux-auth/index.js, but I'm still prompted with :
Uncaught TypeError: Cannot read property 'getIn' of undefined

Any idea?

That is a different error than the original poster's error though. Or do you mean you only get that error after replacing the file? Btw, did you setup the reducer:
import {authStateReducer} from "redux-auth";
// create your main reducer
const reducer = combineReducers({
  auth: authStateReducer,
  // ... add your own reducers here
});

Hey @baopham thanks it worked!
Regarding the error, it is the same one that stated by the original poster:

I built brand new project with a starter kit after adding EmailSignupForm (or any other) i am getting this error
material-ui-theme.js?0449:1 Uncaught TypeError: Cannot read property 'getIn' of undefined

So switching to isomorphic-fetch was also needed!
Thanks for your help, hoping for a quick fixed release,
Cheers

@JonathanWi oh right, i only looked at the error on the console. Thanks for confirming.

I am sting seeing this problem with the latest code. Tried replacing "node-fetch" but no luck. Any suggestion? Thanks.

I am still seeing this problem with the latest code.

For future people... I had this error and fixed it by defining the reducer key, as pointed out by @baopham. Use auth: authStateReducer

The latest version of the demo is based on https://github.com/erikras/react-redux-universal-hot-example.
Can you guys please try again with the demo?
https://github.com/lynndylanhurley/redux-auth-demo

Closing this as I cannot reproduce. Please re-open if you have more information.


Added my name to .txt file

Please I am sorry for all the trouble we have caused you but count it all joy. Your PR is about to be merged and you will be able to see your name on this url.
To enable me to do this, READ the README again carefully, update your code with the latest and push the update. I am here for you for the next 6 hours before I close the PR so you can initiate a fresh PR which will be easier to merge.

Sorry I have to close this PR, kindly update your code and initiate a fresh PR
Cheers 🥂


DCES-77: Moving cron settings to a configuration variable

What
Link to story
This is a small change to move the scheduler cron settings to a configuration variable.
Note: It doesn't seem to be working on local docker, but will hopefully test in the dev container
Checklist
Before you ask people to review this PR:

[x] Tests should be passing: ./gradlew test
[x] Github should not be reporting conflicts; you should have recently run git rebase main.
[x] Avoid mixing whitespace changes with code changes in the same commit. These make diffs harder to read and conflicts more likely.
[x] You should have looked at the diff against main and ensured that nothing unexpected is included in your changes.
[x] You should have checked that the commit messages say why the change was made.

Just a suggestion on the naming... "CRON_CONTRIB..." rather than "DCES_CRON_CRONTRIB..." maybe as the DCES prefix is arguably redundant.

Yeah I agree. the only reason I used the prefix is because this variables are used inside the spring configuration tree and wanted to make sure they are optionals for ours. But I will take that into consideration is we end up using the variables in the end

Can you also add configs for the other environments?

Can you also add configs for the other environments?

will do that when we setup the other environments. Currently there are no values at all yet


Saving a file gets walkback (and doesn't remember its name)

If you open a server file (using the Console File men item), trying to save it using Save on Server or Save On Server As... (which are apparently identical anyway) gets a walkback.
Also, Save On Server should save the file with the same name and path, rather than doing a Save On Server As:, and offering to save it as workspace.txt in $ROWAN_PROJECTS_HOME. Both Save on Server or Save On Server As...  should start the file selection dialog in the location of the original file opened.

Think saving a file should be a little better now.


Update poll.js (display started by who and when?)

   You've been reported to github.


Overlay

Implement overlays

This pr is on top of #40 and should look smaller once that's merged.


Use :ident matcher instead of :ty matcher

When a :ty matcher is used in a macro_rules! macro, the captured
argument can no longer be used to match a literal in a new
macro_rules! invocation. For example, the following code:
macro_rules! inner {
    (my_val) => {}
}

macro_rules! outer {
    ($val:ty) => {
        inner!($val);
    }
}

outer!(my_val);

produces the following error:
error: no rules expected the token `my_val`
  --> src/lib.rs:7:16
   |
1  | macro_rules! inner {
   | ------------------ when calling this macro
...
7  |         inner!($val);
   |                ^^^^ no rules expected this token in macro call
...
11 | outer!(my_val);
   | --------------- in this macro invocation
   |
   = note: this error originates in a macro (in Nightly builds, run with -Z macro-backtrace for more info)

That is, once my_val is captured as a :ty, it will not match a my_val literal
in the inner! invocation.
Due to a bug in rustc, code like this was accepted by the compiler
when a proc-macro attribute was applied to the macro invocation.
This affects fourier-algorithms due to the following code:
#[target_cfg(target = "[x86|x86_64]+avx")]
crate::avx_vector! { $type };

The avx_vector! macro is expecting either f32 or f64
as an ordinary token. However, the $type metavariable uses a
:ty matcher, which will prevent it from matching against f32
or f64 (even though the captured type is always f32 or f64).
Due to the target_cfg attribute proc-macro, this code was incorrectly
allowed to compile. When the underlying bug in rustc is fixed
in https://github.com/rust-lang/rust/pull/73084, fourier-algorithms
will stop compiling.
This commit changes two :ty matchers to :ident matchers, which can
be used as arguments to a macro_rules! macro expecting a plain f32
or f64 token. This will still compile on existing compile versions,
and will allow fourier-algorithms to continue to compile once
https://github.com/rust-lang/rust/pull/73084 is merged into rustc.

Done!


fix(ibis): implement workaround for empty json result

Close #909
Description
This PR implements a workaround for this issue to avoid query failure by the special cases.
If the type mapping fails, using the native BigQuery client to get the schema and create an empty pandas DF instead.

Summary by CodeRabbit

New Features

Introduced a new BigQuery connector that enhances query handling, especially for empty results.

Bug Fixes

Improved error handling for schema issues in SQL queries.

Tests

Added a new asynchronous test to validate the response of the BigQuery connector for empty JSON queries.

Try if only json column is empty

Try if only json column is empty

Sounds great.
Added tests to assert this case in af5c332

There are some issues about BigQuery CI. I tested it locally instead. All passed.
 ~/git/wren-engine/ibis-server ▓▒░ just test bigquery                                                                                ░▒▓ ✔ │ 31s │ ibis-server Py │ 18:07:15
poetry run pytest -m 'bigquery'
================================================================================================================================================================================================================== test session starts ==================================================================================================================================================================================================================
platform darwin -- Python 3.11.11, pytest-8.3.4, pluggy-1.5.0
rootdir: /Users/jax/git/wren-engine/ibis-server
configfile: pyproject.toml
plugins: anyio-4.7.0
collected 221 items / 189 deselected / 32 selected

tests/routers/v2/connector/test_bigquery.py ...................                                                                                [ 59%]
tests/routers/v3/connector/bigquery/test_functions.py ....                                                                                [ 71%]
tests/routers/v3/connector/bigquery/test_query.py .........


References need enhancement

At the moment references are listed in an array, children of lines, notes, etc.
For example, in belsunu there is this line in a translation:

Year 63, Ṭebetu (Month X), night of day 2:^1^

@note ^1^ A note to the translation.
The way it's being stored is in an array of references in the line object, and duplicated in an array of references in the note object.
We should come up with a way of:
a) not duplicating
b) linking the reference to both the note and the line
c) linking the object with the specific place it has been written. It could be an individual word, another part of the line (some random lemma), etc.
d) do a, b and c in a way that doesn't overcomplicate the serializer

Eleanor said references should be linked to words. The lexer and serialisation should be changed accordingly.


Fix ref cursor stmt res leak

@c-bik in my local tests the reference doesn't change after doing an execute with the same bind variables so the reuse is working as expected and not creating new references:
mec_oc:fetch_stmt(RefCursor).
{true,[1.0,<<"a">>,null,null,null]}
(cpro1@<IP_ADDRESS>)15> mec_oc:fetch_stmt(RefCursor).
{true,[2.0,<<"b">>,null,null,null]}
(cpro1@<IP_ADDRESS>)16> dpi:safe(fun() -> dpi:stmt_execute(Stmt, []) end).    0                                                                          (cpro1@<IP_ADDRESS>)17> NewRefCursor = dpi:safe(fun() -> dpi:data_get(Data) end).
#Ref<42688.719746273.3376021508.226633>
(cpro1@<IP_ADDRESS>)18> RefCursor.
#Ref<42688.719746273.3376021508.226633>
(cpro1@<IP_ADDRESS>)19> RefCursor = NewRefCursor
(cpro1@<IP_ADDRESS>)19> .
#Ref<42688.719746273.3376021508.226633>
(cpro1@<IP_ADDRESS>)20> RefCursor =:= NewRefCursor.
true
(cpro1@<IP_ADDRESS>)21> mec_oc:fetch_stmt(NewRefCursor).
{true,[3.0,<<"something2">>,<<"something3">>,
       <<"something4">>,<<"something5">>]}

@c-bik in my local tests the reference doesn't change after doing an execute with the same bind variables so the reuse is working as expected:
mec_oc:fetch_stmt(RefCursor).
{true,[1.0,<<"a">>,null,null,null]}
(cpro1@<IP_ADDRESS>)15> mec_oc:fetch_stmt(RefCursor).
{true,[2.0,<<"b">>,null,null,null]}
(cpro1@<IP_ADDRESS>)16> dpi:safe(fun() -> dpi:stmt_execute(Stmt, []) end).    0                                                                          (cpro1@<IP_ADDRESS>)17> NewRefCursor = dpi:safe(fun() -> dpi:data_get(Data) end).
#Ref<42688.719746273.3376021508.226633>
(cpro1@<IP_ADDRESS>)18> RefCursor.
#Ref<42688.719746273.3376021508.226633>
(cpro1@<IP_ADDRESS>)19> RefCursor = NewRefCursor
(cpro1@<IP_ADDRESS>)19> .
#Ref<42688.719746273.3376021508.226633>
(cpro1@<IP_ADDRESS>)20> RefCursor =:= NewRefCursor.
true
(cpro1@<IP_ADDRESS>)21> mec_oc:fetch_stmt(NewRefCursor).
{true,[3.0,<<"something2">>,<<"something3">>,
       <<"something4">>,<<"something5">>]}

@acautin I am wondering if this means, the lifetime of the Stmt from RefCursor is independent of the lifetime / state of the Stmt which returned the RefCursor Stmt!
In other words the second execute probably has no effect!

@c-bik a test case with a re execution of the query to get a new ref cursor is missing.

Done!


MRG: Correct documented default number of CV splits in cross_val_multiscore()

We're actually using 5 splits and have been for quite a while now, but this bit of the docs hasn't been updated.

Thanks @hoechenberger


Develop

It is definitely not right to impose an upper limit to the number of threads in ThreadPool.h. That is:
if (n_threads == 0 || n_threads > std::thread::hardware_concurrency())...
Processes with more IO would benefit having more threads than std::thread::hardware_concurrency(). The test makes sense only if computations are 100% CPU bound. That is not always the case in real life.
See for example David Schmitt's answer from http://stackoverflow.com/questions/510441/how-many-threads-to-create-and-when ("For all other situations XU / %IO gives an approximation of how many threads are needed to fully use the available XUs").
I would remove the test as it makes using the thread pool quite inefficient for quite a range of applications.

Hey - Sorry for the huge accidental pull request, was merging a change into my own master.
Very good point, you're right, I was only considering 100% CPU bound operations. Removed the test.
I'll leave the pull request closed, I can open it again if you'd like to merge it?

Hi Will, no worry, I am not the owner of the project so cannot approve pull requests, I just set watching it a while back. I was commenting because I saw activity and was curious what new things would happen.


Not discharging!

Lenovo Legion 5 pro 16ACH
Manjaro - Cinnamon
Kernel - 5.15.19
If I have a full charge (100%) and turning on the lenopow won't drain to 50% or 65% (but it should drain), it instead stays at 100% charge and stays that way as long as I'm connected to electricity. If I manually drain my laptop to 70% or whatever percentage and turn on lenopow, the charge won't go above that percentage. I think there is something wrong with these scripts.
There was a script from schinfo that worked great, no matter how much % was in my laptop, it kept at 55-65%.

When I left my laptop for 12 hours it calibrated and discharged to 60% automatically. I think it needs some time to drain. Thanks


net: sockets: Fix error reporting in case of TCP errors

The socket implementation did not report TCP errors correctly, i. e. events like transmission timeout or receiving RST packet were reported as graceful shutdown (i. e. recv() returned 0). Fix this, by making use of the status field in the recv_cb() to forward errors from TCP to socket layer.
Tested with https://github.com/zephyrproject-rtos/zephyr/pull/44547, which introduces a test to verify TCP transmission timeout.

FYI @ssharks

Nice fix, I added a few comments on some details

Nice fix, I added a few comments on some details

Did you? I don't see any.

I attempted to start a review, can you see it? I'm considering testing this in the field as today I already saw the original problem happening.

I attempted to start a review, can you see it? I'm considering testing this in the field as today I already saw the original problem happening.

I can't see any comments in this PR.

Had to rebase due to conflict, please re-review.


Allow admin to upload company logos and display them on listings

Data:

[ ] Store image ID on company entity

Core:

[ ] Update company service to handle CRUD on company entity
[ ] Update company service to have methods to handle saving/deleting/load images

UI:

[ ] Create controller to manage company
[ ] display image on single listing
[ ] display images on listingslist
[ ] Create view to manages images

 US3001: Admin - Lägga till företagslogga


Incorrect filename (uses the package name instead of the folder name)

Hi! I'm trying to use this plugin in a Kotlin sample project before deploying it to my real projects. It generates the file correctly, but the filename is incorrect.
Here is the sample code: examples/kotlin/core/src/commonMain/kotlin/HelloWorld.kt:
package example

const val HELLO_WORLD = "Hello world!"

/**
 * Empty class with documentation.
 */
class HelloWorld

fun foo() {
	println("TMP DO NOT MERGE THIS")
	println(HELLO_WORLD)
}

The file is directly in the kotlin directory (no subdirectories for the example package), as per the Kotlin code conventions.
However, the coverage report generated by this plugin looks like:
<?xml version="1.0" encoding= "UTF-8" ?>
<coverage line-rate="1.0" branch-rate="0.0" complexity="0.0" timestamp="0">
   <sources>
      <source>.</source>
   </sources>
   <packages>
      <package line-rate="1.0" branch-rate="0.0" complexity="0.0" name="example">
         <classes>
            <class line-rate="1.0" branch-rate="0.0" complexity="0.0" name="example/HelloWorld" filename="example/HelloWorld.kt">
               <methods>
                  <method line-rate="1.0" branch-rate="0.0" complexity="0.0" name="<init>" signature="()V">
                     <lines>
                        <line number="8" hits="1" branch="false"/>
                        <line number="11" hits="1" branch="false"/>
                     </lines>
                  </method>
               </methods>
            </class>
            <class line-rate="1.0" branch-rate="0.0" complexity="0.0" name="example/HelloWorldKt" filename="example/HelloWorld.kt">
               <methods>
                  <method line-rate="1.0" branch-rate="0.0" complexity="0.0" name="foo" signature="()V">
                     <lines>
                        <line number="8" hits="1" branch="false"/>
                        <line number="11" hits="1" branch="false"/>
                     </lines>
                  </method>
               </methods>
            </class>
         </classes>
      </package>
   </packages>
</coverage>

The file name is reported as filename="example/HelloWorld.kt", which is wrong.

Reproduction:

git clone https://gitlab.com/opensavvy/automation/gradle-conventions.git
git switch 30-cobertura-xml-report
./gradlew :jacocoToCobertura -p examples/kotlin --include-build ../..

Files of interest:

Plugin configuration: root/src/main/kotlin/root.gradle.kts (online)
Code being tested: examples/kotlin/core/src/commonMain/kotlin/HelloWorld.kt (online)
Tests: examples/kotlin/core/src/commonTest/kotlin/HelloWorldTest.kt (online)

Versions:

Kover 0.7.6
Jacoco to Cobertura 1.2.0

For context, I want to display coverage reports generated by Kover into the GitLab Merge Request UI, but my attempt currently doesn't work. I believe the file name being incorrect is the cause.

Well, is supposed to do that... at least cover2cover does that... can't know if is a kotlin or java project and if the package is virtual and not in a directory as should be in a java project. Nevertheless to fix that I added in 1.3.0 release (to come soon) an new option rootPackageToRemove that will strip the packages in the start of the filename.
So if you set this value to com.exemple and your file is with package com.example and named HelloWorld.kt then. the result filename will be HelloWord.kt
I tried to see the result of the python script on your exemple but it does not run at all as kover Jacoco report does not seem standard.
I hope it fix your problem.


Merge AWS Terraform scripts to master

This PR consists in merging all work that was done towards AWS (no impact on actual codebase) so we avoid having to rebase all the time with the master branch.
AWS work is on hold right now as our main focus is the Concierge testing.

Okay, so I am okay with merging this but I want to get to the root of the difference between our setups.
When I have my VS Code Terraform plugin disabled, I don't see any issues (or syntax highlighting).

When it's enabled, I do see issues (I think about the quotes issue).

There's also something telling me I should download "experimental language server". Should I try turning that on? Did you do it?

Okay, so I am okay with merging this but I want to get to the root of the difference between our setups.
When I have my VS Code Terraform plugin disabled, I don't see any issues (or syntax highlighting).

When it's enabled, I do see issues (I think about the quotes issue).

There's also something telling me I should download "experimental language server". Should I try turning that on? Did you do it?

I believe we should focus on the official Terraform validation command: terraform validate. This command will essentially lint and confirm that the script is syntactically correct. Reason why I am saying this is because the VScode extension is not an official one, but an extension that was created by a single person. I would take the red squiggly with a grain of salt. That's a conversation I had with Max Neuvians as well in the early development of the AWS pivot, and he seemed to be confused by the IDE behavior as well.

Assume that if terraform validate passes you are good to go


Code Review For and By Scientists

Link: http://arxiv.org/pdf/1407.5648v2.pdf
Author: Marian Petre & Greg Wilson
Video: N/A
Description: A summary of two studies examining the opportunities and challenges of implementing a system of code review in the lab.

as added bonus material, don't miss this blog post by @fperez on conducting code reviews as part of live lab meetings.


[$500] Chat - Entering @ twice in compose and then select user suggestion, one @ disappears

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

Version Number: <IP_ADDRESS>
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
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Go to https://staging.new.expensify.com/home
Tap on a report
Enter @ twice in compose
From user suggestion list, select a contact
Note in compose only one @ is shown

Expected Result:
Entering @ twice in compose and then select user suggestion, one @ must not disappear.
Actual Result:
Entering @ twice in compose and then select user suggestion, one @ disappears.
Workaround:
Unknown
Platforms:

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

[x] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/b119fee7-5c2b-4518-9b08-8cccdcbe7839
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~010f233b37d70d3b89
Upwork Job ID:<PHONE_NUMBER>868328448
Last Price Increase: 2024-02-08

We think that this bug might be related #vip-vsb
CC @quinthar

I've had a look at this and the input and the Suggestions component are both quite reliant on one another particularly the value property that they share

Proposal
Please re-state the problem that we are trying to solve in this issue.
Entering @ twice in compose and then select user suggestion, one @ must not disappear.
What is the root cause of that problem?
SuggestionMention is finding mentions based on the first index of @ in the search string string. This successfully finds the suggestion e.g. @@here finds @here and then overwrites it losing the first @
What changes do you think we should make in order to solve the problem?
There's a function inside SuggestionMentions that checks the string for "@" at the start of the current word. I propose we add another condition in the if statement. Adding above the other blocks seems suitable.
if (lastWord.slice(0, 2) === '@@') {
    atSignIndex = leftString.lastIndexOf(lastWord) + 1;
    suggestionWord = lastWord;
    prefix = suggestionWord.substring(1);
}

What alternative solutions did you explore? (Optional)
None
Reminder: Please use plain English, be brief and avoid jargon. Feel free to use images, charts or pseudo-code if necessary. Do not post large multi-line diffs or write walls of text. Do not create PRs unless you have been hired for this job.

Proposal
Updated
Updated "Root cause" And made changes a more complete answer

Why was this job not sent to me?

Is this a bug? The first @ is needed to open the suggestion menu And the second one comes as part of the search query, like any other symbol

Both @ symbols are included in the suggestion word so I think it is a bug.

Sorry for my delay - I agree - this is not a bug.  You type @@ and select from the list, that replaces your search and you need to start again - just as if you type @ and any letters.


Fixed javadoc issue

Actually I just realized that this issue will be fixed in your PR 1373, so either remove this PR or remove it from the other PR, otherwise there will be a conflict.


Why does Types.rs exist?

Hello,
As I was working on this repo to try and get the loginlink working, I realized that every time I ran the generation code on master the code generation breaks because of updates.  I didn't want to push my link to a previous tagged commit (i.e 13-rc3), so I started to look into why code generation was breaking.
The first issue was relatively easy to fix.  Quote is a dependency of a type that it implicitly generates and therefore falls into a corner case that incorrectly adds itself as a resource use to its own generated file.
But the second issue is a little bit more intense.  Some dependent types that have more than one dependency are skipped to generate.  This makes a little bit of sense as they are found inside of a "generated" file so to speak and a naive solution would allow you to have two of the same "type" published from two or more different source directories.
The current solution, seems to be to hand code these types into resources Types.rs.  This seems like a non-tenable solution that will cause the generated code to continue to break into the future.  I don't like out of date API's especially when this particular version of the OpenAPI is the "stable one" according to git.
So, what if we modified the generated main file to either a) add on these extra files onto the end of the objects metadata field (this would generate a unique file for each, but would require some refactoring because the objects are borrowed immutable in a number of places) or b) create a new category in metadata that runs after the objects and essentially does the same thing as a) but sidesteps the problem of having to add on to a container that is running inside an iterator loop.
Then you would need to remove some of the hand written types that no longer need to exist.
Thoughts?

Following up on this, I implemented b.  It required a lot of things to change.  There were several bugs in the code generation portion that were being papered over by hacks in the mapping portion and hand written code.  I fixed them and started removing the handwritten code, particularly in types.rs.
So far, the current API is compiling again.  I've also refactored the main generation code so that it's more readable (more functions broken up into logical groupings rather than one giant function with several loops.)
The code is compiling on all 5 targets and has been tested (though the tests don't have very much coverage).  I can make a pull request if you want, but I'll wait for a response.
The biggest change I had to make was to make "nullable" objects in the API, Box<Option<>> rather than just Option<>.  This is because some of them are circular dependencies (the one I know about is the api_errors and the payment_intent structs).  An Option<> doesn't allow for this circular dependency to happen in Rust for obvious and well documented reasons.
This means that my update would be a breaking change for sure, but master won't compile with the current code anyway, and my code does, so I think it's a fair trade.  Let me know if you want the code or if you'll pass.
All the best, thanks for the work with the async code, it's really great.

Hi there, sorry I have been away recently.
To give some more context, types.rs is around simply because that's how it was handled in previous library pre-fork. I have been meaning to have the generator automatically pick it up but haven't gotten round to it.
I am very much for these changes (thanks!!!!) and very keen to see a PR / get them merged. I have been deliberately holding off of the next release until I could fix this so this is great news.
As for refactoring the generation code: once again, thanks, that has been improved compared to the previous version but still needs some modernising.
Looking forwards to seeing the PR up.
Cheers, Alex

(also, it's been annoying me that the automated generate PR fails every week so it'll be nice to not have to keep closing it :D)

I opened a pull request for this issue.  Since it's larger and #108 depends on it, I'll wait to pull request that feature after this one is accepted.


What is train_mvs_nerf_fusion_finetuning_pl.py used for?

Thanks for your nice work!
What is train_mvs_nerf_fusion_finetuning_pl.py for when there is already a train_mvs_nerf_finetuning_pl.py ?

Oh, I forgot to remove this file, it's used for some fusion experiments, you can just bypass this file.

Thanks for your reply.


Json parse error on consul 1.10.4

Im trying to list all registered services on consul 1.10.4. And its seems that serde_json can't parse response from consul.
ResponseParseError { source: invalid type: map, expected a string at line 1 column 135, content: Some(\"{\\\"test_service\\\":{\\\"ID\\\":\\\"test_service\\\",\\\"Service\\\":\\\"test_service\\\",\\\"Tags\\\":[],\\\"Meta\\\":{},\\\"Port\\\":1234,\\\"Address\\\":\\\"<IP_ADDRESS>\\\",\\\"TaggedAddresses\\\":{\\\"lan_ipv4\\\":{\\\"Address\\\":\\\"<IP_ADDRESS>\\\",\\\"Port\\\":1234},\\\"wan_ipv4\\\":{\\\"Address\\\":\\\"<IP_ADDRESS>\\\",\\\"Port\\\":1234}},\\\"Weights\\\":{\\\"Passing\\\":1,\\\"Warning\\\":1},\\\"EnableTagOverride\\\":false,\\\"Datacenter\\\":\\\"dc1\\\"}}\") } }"

Would you be able to replicate the request and post the actual JSON content for review?

request: curl http://<IP_ADDRESS>:8500/v1/agent/services
Json:
   "my_service":{
      "ID":"my_service",
      "Service":"my_service",
      "Tags":[

      ],
      "Meta":{

      },
      "Port":1234,
      "Address":"<IP_ADDRESS>",
      "TaggedAddresses":{
         "lan_ipv4":{
            "Address":"<IP_ADDRESS>",
            "Port":1234
         },
         "wan_ipv4":{
            "Address":"<IP_ADDRESS>",
            "Port":1234
         }
      },
      "Weights":{
         "Passing":1,
         "Warning":1
      },
      "EnableTagOverride":false,
      "Datacenter":"dc1"
   }
}```

Can you verify that #14 fixes this?

Works for me

It would be nice if this change was pushed to crates.io. For anyone who wants it use before it's pushed, you can specify your dependency like so: consulrs = { git = "https://github.com/jmgilman/consulrs" }


[Tracking Bug] List of disabled workloads.

This issue lists the workloads that we have disabled due to regressions in compilers or frameworks that we do not maintain.

CUDA
XLA Compiler-Level:

BERT_LARGE_FP16_JAX_384XI32_BATCH{1, 16, 24, 32, 48, 64, 512, 1024, 1280}: https://github.com/openxla/openxla-benchmark/issues/117
T5_LARGE_FP32_JAX_512XI32_BATCH512: null.
T5_LARGE_BF16_JAX_512XI32_BATCH512: null.
T5_LARGE_FP16_JAX_512XI32_BATCH512: null.
T5_LARGE_FP32_TF_512XI32_BATCH512: null.

XLA Framework-Level:

T5_LARGE_FP32_TF_512XI32_BATCH512: null.
BERT_LARGE_FP32_JAX_384XI32_BATCH{1024, 1280}: 1 hour timeout.
T5_LARGE_FP32_JAX_512XI32_BATCH{64, 512}: resource exhausted.

Inductor Framework-Level:

RESNET50_FP32_PT_3X224X224XF32_BATCH2048: CUDA out of memory.
BERT_LARGE_FP32_PT_384XI32_BATCH{64, 512}: CUDA out of memory.
BERT_LARGE_FP32_PT_384XI32_BATCH{1024, 1280}: CUDA error: an illegal memory access.
BERT_LARGE_FP16_PT_384XI32_BATCH{16, 24, 32, 48, 64}: https://github.com/openxla/openxla-benchmark/issues/62

CPU
XLA Framework-Level:

EFFICIENTNETB7_FP32_TF_600X600X3XF32_BATCH128: 1 hour timeout.

Inductor Framework-Level:

BERT_LARGE_FP32_PT_384XI32_BATCH{32, 64}: null


Delete the unuse-lib of statements at tools directory

This have some statements which imported corresponding lib but they are not used, so attempt to delete it.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok to test.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok to test.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok to test.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok to test.

For conflicting.


Add basic geolocation API interface.

@brandonedens Ping. Will you be still working on this? (:

Any update on this? :)

@Boscop Doesn't look like it. If you need this functionality then perhaps you would be interested in continuing this PR? (:


How can I do merge without --log param?

How I can disable default --log param?
https://github.com/divmain/GitSavvy/blob/master/core/commands/merge.py#L49
Trying to use this config
{
"global_flags": {
        "merge": ["--no-log"]
    }
}

But seems that --no-log is placed before default --log (according to
https://github.com/divmain/GitSavvy/blob/1394489e1fa042996d9ba20817c0e8e4d45dcbb0/core/git_command.py#L245)

Why this important? Because people who use JIRA smart commits for logging time include time record to commits messages. Merge with --log provoke logging time twice for every commit
Example commit messages:

simple commit message
KATA-964 #time 0h 54m immutable tips

merge commit message

Merge branch 'KATA-958-manage-notices-as-administrator' into dev

* KATA-958-manage-notices-as-administrator: (23 commits)
KATA-964 #time 0h 54m immutable tips
KATA-964 #time 0h 4m Fix some
KATA-964 #time 0h 4m just time log
...

This should be an easy fix - will include in the next release @max-mykhailenko.

Sent pull request for that https://github.com/divmain/GitSavvy/pull/188

I have this error again, probably because commands order in line
https://github.com/divmain/GitSavvy/blob/master/core/git_command.py#L265
@divmain please, check it, we have big problems with that

In my previous pull request it was changed to
if global_flags and git_cmd in global_flags:
-            args = [git_cmd] + global_flags[git_cmd] + addl_args
+            args = [git_cmd] + addl_args + global_flags[git_cmd]

Hey @max-mykhailenko, unfortunately that broke several other cases and we had to revert.  I think the simplest approach here will be to make that --log param optional.  Just pushed a commit, let me know if it works for you.


Switch publishing to s01

Daniel seemed to think org.typelevel may already be setup for s01.oss.sonatype.org. So, let's find out.

Let's see what happens...


RESTHost | POST request's body data is undefined, or an empty object or something that was not expected

This occurs on the post request as a body is sent via that.

The issue has been resolved!
Download: https://github.com/SkylixGH/LuxJS/releases/tag/v1.0.0-dev.3


updateSearchWithDebounce

Signed-off-by: James Talton<EMAIL_ADDRESS>When we have large lists of items, each key press in search was running the expensive search function.
This will add a debounce to search making it more usable for scale.

/lgtm


Update the affine parameter description in several classes

Description
closes #3317
As discussed in #3317, the affine parameter description in some classes describes outdated behavior. This PR is to make it consistently describe that the affine transform will be chained with any scale, translation or shear that is provided.
Type of change

[x] This change requires a documentation update

References
inspection of Layer.__init__ to verify that the transforms are chained.
How has this been tested?
manual inspection that the changed classes pass the affine on to Layer via super.__init__() and Layer does chain the transforms.
This was also verified manually in the GUI canvas when displaying an Image added via Viewer.add_image
Final checklist:

[x] My PR is the minimum possible work for the desired functionality
~- [ ] I have commented my code, particularly in hard-to-understand areas~
[x] I have made corresponding changes to the documentation
~- [ ] I have added tests that prove my fix is effective or that my feature works~
[ ] If I included new strings, I have used trans. to make them localizable.
For more information see our translations guide.

scipy.ndimage has a special decorator that can replace variables with centrally defined parameter text. For example, this example scipy.ndimage docstring and others get the actual text as defined here. The doccer.py utility that enables this is not part of the public API, but we could vendor it if we wanted to use this kind of approach.
This makes sure things stay consistent across functions, but a downside is that it is a little bit annoying when reading the source code directly. You have to go reference the text in that separate file or in the generated documentation rather than seeing it right inline.

That's pretty neat, though I was thinking of something simpler that would just use a reST directive like the following.
For other keyword only arguments see :class:`napari.layers.Layer`.

Sphinx will then generate the link in the built docs, but if you're in the source code, you'll still have to manually navigate to Layer from Points (unless there are some reST/sphinx plugins for some IDEs that make it a clickable link), though it's fairly clear where to find it.
Anyway, this is definitely a future discussion (maybe part of the documentation working group?), and this PR should be submitted as is.


vulcand exits when etcd watch fails

https://github.com/mailgun/vulcand/blob/master/backend/etcdbackend/etcdbackend.go#L399
results in the following on my coreos cluster:
Aug 04 02:17:25 coreos.novalocal bash[6109]: ERROR Aug  4 02:17:25.192: [etcdbackend.go:405] Stop watching: Etcd client error: unexpected end of JSON input
Aug 04 02:17:25 coreos.novalocal bash[6109]: INFO  Aug  4 02:17:25.192: [service.go:121] Stopped watching changes with error: unexpected end of JSON input. Shutting down
with error
Aug 04 02:17:25 coreos.novalocal bash[6109]: INFO  Aug  4 02:17:25.192: [service.go:111] Got request to shutdown with error: unexpected end of JSON input
Aug 04 02:17:25 coreos.novalocal bash[6109]: Service exited with error: Service exited with error: unexpected end of JSON input
i was told in coreos irc that the watch should be retried and not treated as a fatal error.
perhaps vulcand can catch json.SyntaxError (the type of the error passed) and not treat it as fatal, or ignore any error.

Etcd 2.0 should not respond with empty body
https://github.com/coreos/go-etcd/issues/170#issuecomment-68959994


Navigation events?

Does this package expose any navigation events? Here an example use case:
I would like to pause a websocket connection when I navigate to another page, for example from A to B, and then resume when I go back from B to A.
Reading the package description makes me believe, that the page A would be still fully active even when I am on page B.
My current app architecture can get a little bit crazy, where I can have in stack/history page A > B > C1 > C2 > D. Simply, too deep, with same pages in navigation but with different construct params, but unfortunately this is needed.

You can do it with NavigatorObserver or Guards


Please allow Advanced Paste to work with ChatGPT inside Azure OpenAI Services

Description of the new feature / enhancement
In Settings for Advanced Paste, the user should be able to select whether they want to use an OpenAI API key or an Azure OpenAI Services API key.  If Azure OpenAI, then they will also need to enter the URL to the model such as https://[MODEL_URL].openai.azure.com/openai/deployments/[MODEL_NAME]/chat/completions?api-version=2024-08-01-preview
Scenario when this would be used?
This is important for business users.  Our business data is protected inside Azure and all our org's policies are applied.  Since it's so easy to use Advanced Paste with content from emails and business documents, then it would be very easy to make the mistake of sending sensitive business data to OpenAI's public service, which nearly every IT security team prohibits by policy.
Supporting information
https://blogs.microsoft.com/on-the-issues/2024/03/28/data-protection-responsible-ai-azure-copilot/

/dup https://github.com/microsoft/PowerToys/issues/32960


Challenge http://www.freecodecamp.com/challenges/waypoint-use-the-bootstrap-grid-to-put-elements-side-by-side has an issue

Please tell us in detail here how we can make this challenge better.
When using the  the mobile page has space on the right side, you can actually side scroll to the left. In order to fix this, you need to wrap  with a .
Every  must be inside a  in order to fix this issue. But fixing this issues causes other problems in the buttons. The words don't fit in the buttons.

Thanks. This is a known side effect of using the iframe as we are. We'll look at improving mobile view in the future but it's not our priority now on coding challenges.


Consider switching code highlighting

to https://github.com/shikijs/shiki

we now reuse markdown highlighting directly - no need for extra highlighting.
We could use it to power highlighting for markdown (but that's a whole different story)
not a current goal for now


Re-run script on file changes even after finished execution

Consider the following script:
// index.ts
console.log("Finished");

What happens
When running tsnd index.ts it runs the script and exists as the script finishes.
What should happen
When running tsnd index.ts it runs the script and waits for changes. If the file is changed it re-runs it again. Just as e.g. nodemon does it.

there is a --respawn param

Oh, wow. I somehow never noticed it... Thanks @whitecolor.


Support specifying machine type of cloudpods VM

What would you like to be added:
Support specifying mahine type of cloudpods VM. Some new features, such as supporting USB 3.0 mass storage is supported by Q35 only. Current cloudpods vm supports machine type of PC only.

https://github.com/yunionio/dashboard/pull/1468


should the embedded language include fileTypes

running into an interesting bug with my text editor which uses these syntaxes to highlight code
it considers languages based on their root scope name sorted so unfortunately source.cpp.embedded.macro.json has higher priority than source.cpp.json and as such it gets matched for .cpp because the embedded grammar includes fileTypes
I'm not sure whether it should or not, and it fixes my problem if I remove fileTypes from the embedded syntax -- wondering if that would be an acceptable patch here!
thanks again for these syntaxes, they've been really useful!
EDIT: oops forgot to post my issue -- https://github.com/asottile/babi/issues/151

You're right Embdedded grammars shouldn't have filetypes. The main grammar should be the only thing that uses/imports the embedded grammar, so it shouldn't really matter.
I went ahead and made a push that takes it out

Your project looks really interesting! Glad I could help!
I've been wanting a better nano for quite a while.

thanks!  the patch on the main branch looks great!


Picture-in-picture: Play / Pause does not work

In playback mode "picture-in-picture" the button "Play / Pause" does not react. The window must first be maximised to normal size again for the button reacts again.
Samsung Galaxy A13 5G (SM-A136B)
Android 12
ZAPP 7.0.4 (61)

Thank you very much for your feedback!
I cannot reproduce this issue on my devices or emulators. Do you experience similar issues with other apps?

I'll keep an eye on it and get back to you.

The phenomenon probably only occurs when other apps with integrated players are open at the same time (Zattoo, Chrome Browser).

The error presumably only occurs under certain conditions or temporarily. Currently it does not work again.
I keep my eyes open for possible causes.

It seems to be confirmed that the error always occurs when another application has used the PIP mode shortly before.
In the current case, it was the ARD media library app.

I think I have now also observed the problem with picture-in-picture in another app. I think this would indicate an Android problem or a shared buggy software library. I'll keep an eye on this.


Add timing options for generation function

This PR adds timing functionality to the generation utilities for both per-token and e2e. I have a question on how to expose this info back to the caller instead of printing directly in the function. Any suggestions @nairbv @JRosenkranz?

This PR adds timing functionality to the generation utilities for both per-token and e2e. I have a question on how to expose this info back to the caller instead of printing directly in the function. Any suggestions @nairbv @JRosenkranz?

Maybe we could just add the timing to a logger info level? Not explicitly return it back to the caller.

if we're going to print to stdout in this function, I'd suggest using a debugger like https://docs.python.org/3/library/logging.html
then maybe instead of a 'timing' parameter, we just use the log level on the logger. E.g. if logger.getEffectiveLevel() <= logging.DEBUG:

From an internal discussion, we do need to return the whole array of timings in the "per_token" case, as some users need to see differences in token 5 vs token 10 for example.

I also removed the prints for now. and I'll let the caller deal with that however they want


Review: AI Gen Examples for ma.argsort

Link to your fork:

https://github.com/numpy/numpy/compare/main...otieno-juma:numpy:ai-examples-ma-argsort
Checks

spin build && python -m pip install . && spin docs && python tools/refguide_check.py --doctests && spin lint

[ ] All tests have passed
[ ] A visual inspection of the generated docs is correct

Acceptance Criteria:

Close when the following are complete, in order:

[ ] Reviewed by @bmwoodruff
[ ] Any suggestions to fork have been merged.
[ ] Reviewed by Numpy Mainter:

When all tasks are complete:

[ ] Submitted as PR to NumPy.

The example doesn't involve any sorting at all, rather it just defines an array and then shows it.  This example is definitely not worth adding to the codebase.
We'll need to refine our prompt to get better examples, if we hope to add them to the codebase.  Focus on the ones that have no examples, or try creating better prompts.
You now know what it takes to get something to pass the tests, and prepare it for inclusion.  Now we just need better examples, or focus on different aspects of the docstring.


Fixed bug in Logarithm Calculator

Related Issuse
After clicking in logarithm calculator , the nav bar isnt getting closed on its own while it gets closed on its own when we click on other other calculators
Closes: #406
Describe the changes you've made
I added the missing code i.e,
$("#logValues").click(function(){
openit("#log_values");
closenav();
clearall();
});
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.

Well Done :)

Thank you 😊


Add travis configuration file

This adds the Travis CI configuration file as requested in #2

Thanks master i have added .travis.yml file with earlier PR #2


brew cask upgrade java fails

$ brew cask upgrade java
==> Upgrading 1 outdated package, with result:
java 10,46:76eac37278c24557a3c4199677f19b62
==> Satisfying dependencies
==> Downloading http://download.oracle.com/otn-pub/java/jdk/10+46/76eac37278c24557a3c4199677f19b62/jdk-10_osx-x64_bin.dmg
######################################################################## 100.0%Warning: Failed to create the file
Warning: /Users/xxx/Library/Caches/Homebrew/Cask/java--10,46:76eac37278c24
Warning: 557a3c4199677f19b62.dmg.incomplete: No such file or directory

curl: (23) Failed writing body (0 != 5491)
==> Purging files for version 10,46:76eac37278c24557a3c4199677f19b62 of Cask java
Error: Download failed on Cask 'java' with message: Download failed: http://download.oracle.com/otn-pub/java/jdk/10+46/76eac37278c24557a3c4199677f19b62/jdk-10_osx-x64_bin.dmg

When I try manually http://download.oracle.com/errors/download-fail-1505220.html

Please fill out the template and provide all the requested information and the issue may be reopened.


Add files via upload, may have a brief check if you got time

DeviceID needs to match.  Check the slides for label requirements.
`
% X-coord, Y-coord, Polarization, wavelength, type, deviceID, params
-165, 425, TE, 1550, device, erikch, VernierFilterheater
% X-coord, Y-coord, deviceID, padName, params
173, 550, VFheater, G1, comment
173, 425, VFheater, S1, comment
174, 299, VFheater, G2, comment
173, 175, VFheater, S2, comment
% X-coord, Y-coord, recipeID, params
`

Updated the labels, should be consistent with slides now


parser error for json keys with spaces

Hi Team,
In my JSON key contains spaces . When I am sorting it is giving parse error . Please help me on this

Find a simple way to fix this.
If your orderBy key name is first name,
then if you write the code like:
it doesn't work.
but if you add a \u0022 around like:
	, it works


The image changes permissions when mounting the host docker.sock into the container.

First: thanks for making this image; it's been extremely handy to get up and running with self-hosted runners.
I do have one request: would it be possible to remove the permissions change here?
https://github.com/tcardonne/docker-github-runner/blob/861abc39c7e6a56c1211ac744b5e4a09696f13e9/debian-buster/entrypoint.sh#L5
Since docker-in-docker isn't advised, the "best-practice" solution is to mount the host's docker socket into the container. (You note this in the README and provide the solution: -v /var/run/docker.sock:/var/run/docker.sock).
However, doing so will cause permissions on the host docker socket to change, as a result of the linked command above.
This is the only thing preventing us from using your image on a larger-scale, so if you have any questions/clarifications here I'd be glad to answer them!
Thanks again!

i think it is caused by this line in Dockerfile: https://github.com/tcardonne/docker-github-runner/blob/master/debian-buster/Dockerfile#L66
the fix could be remove this line, and add switch user in entrypoint.sh

or, similar as above, remove https://github.com/tcardonne/docker-github-runner/blob/master/debian-buster/Dockerfile#L66
and add flag allows starting the runner with env RUNNER_ALLOW_RUNASROOT=true, see also: https://github.com/actions/runner/pull/227

Hello,
This was a silly workaround, so I changed the image to run the runner as root instead of using a dedicated user (which had sudo privileges anyway).
I've changed that in the master branch (https://github.com/tcardonne/docker-github-runner/commit/90ae74aa0c02aae17d32647637a3add33736ebd1), but I will tag a new release later this week-end.
In the meantime, can you tell me if this solution would fit your needs?

Implemented in v1.2.0 🚀


Allow selection of disabled dates in date inputs

Changes
Enhancement to the date picker components to allow selection of dates that are normally disabled by minDate, maxDate and disabledDates

[delete] branch

Changelog entry

Add option to allow selection of disabled dates in DateInput and DateRangeInput

Additional information

You may refer to this ticket

Approved but to resolve conflicts first


New to tis but have an error stopping me can anyone hhelp?

[Info] Using cuda for inference.
[Step 0] Number of frames available for inference: 822
[Step 1] Using saved landmarks.
[Step 2] 3DMM Extraction In Video:: 100%|████████████████████████████████████████████| 822/822 [00:08<00:00, 97.96it/s]
using expression center
Load checkpoint from: checkpoints/DNet.pt
Load checkpoint from: checkpoints/LNet.pth
Load checkpoint from: checkpoints/ENet.pth
[Step 3] Using saved stablized video.
[Step 4] Load audio; Length of mel chunks: 819
[Step 5] Reference Enhancement:  14%|██████▋                                         | 114/819 [00:37<03:49,  3.07it/s]
Traceback (most recent call last):
File "C:\Users\black\Downloads\video-retalking-main\venv\lib\site-packages\gradio\queueing.py", line 407, in call_prediction
output = await route_utils.call_process_api(
File "C:\Users\black\Downloads\video-retalking-main\venv\lib\site-packages\gradio\route_utils.py", line 226, in call_process_api
output = await app.get_blocks().process_api(
File "C:\Users\black\Downloads\video-retalking-main\venv\lib\site-packages\gradio\blocks.py", line 1550, in process_api
result = await self.call_function(
File "C:\Users\black\Downloads\video-retalking-main\venv\lib\site-packages\gradio\blocks.py", line 1185, in call_function
prediction = await anyio.to_thread.run_sync(
File "C:\Users\black\Downloads\video-retalking-main\venv\lib\site-packages\anyio\to_thread.py", line 56, in run_sync
return await get_async_backend().run_sync_in_worker_thread(
File "C:\Users\black\Downloads\video-retalking-main\venv\lib\site-packages\anyio_backends_asyncio.py", line 2177, in run_sync_in_worker_thread
return await future
File "C:\Users\black\Downloads\video-retalking-main\venv\lib\site-packages\anyio_backends_asyncio.py", line 859, in run
result = context.run(func, *args)
File "C:\Users\black\Downloads\video-retalking-main\venv\lib\site-packages\gradio\utils.py", line 661, in wrapper
response = f(*args, **kwargs)
File "C:\Users\black\Downloads\video-retalking-main\webUI.py", line 383, in inference
pred, _, _ = enhancer.process(img, img, face_enhance=True, possion_blending=False)
File "C:\Users\black\Downloads\video-retalking-main\third_part\GPEN\gpen_face_enhancer.py", line 113, in process
mask_sharp = cv2.GaussianBlur(mask_sharp, (0,0), sigmaX=1, sigmaY=1, borderType = cv2.BORDER_DEFAULT)
UnboundLocalError: local variable 'mask_sharp' referenced before assignment
I have absolutely no idea what i have done wrong, It was supposed to be a standalone download that required nothing but running of a bat file, anyway thanks for any help!

Im on windows 10 let me know if you need any other info.


Create a new sample section in the docs with links to examples

Example apps are available at github.com/grails-spring-security-samples
Create a new sample section towards the end of the documentations near the tutorials section

This is a duplicate of https://github.com/grails-plugins/grails-spring-security-core/issues/458

This is already visible in the snapshot docs. https://grails-plugins.github.io/grails-spring-security-core/snapshot/#examples
thanks @ddelponte


Update directives parser to include missing documents

closes #238

@mico please release this update. It blocks addressing the issue.


How to Load Google API Javascript Library on Server

I'm trying to access the Google Sheets API from an hMailServer script. The script is Javascript and executes on a server, not in a browser. If you're unfamiliar with hMailServer, consider a script periodically running on a server as a chron job.
Google's documentation gives plenty of information on how to load the Google API Javascript client from HTML, but I haven't found any that describes how to do it from Javascript that runs on the server side and not in a browser context.
How do you load the Google API Javascript library in a script running on a server?
Also, what are the differences among:
https://apis.google.com/js/api.js
https://apis.google.com/js/client.js
https://apis.google.com/js/client:platform.js

As shown in our documentation for supported environments, the library only work in HTML pages. It uses HTML functionalities such as iframe, and browser APIs such as XMLHttpRequest.
https://apis.google.com/js/api.js loads the gapi loader. You can then use the loader to load other modules, such as load the client module with gapi.load('client', callback).
https://apis.google.com/js/client.js loads the gapi loader and then asynchronously loads the client module.
Similarly, https://apis.google.com/js/client:platform.js loads the gapi loader and then asynchronously loads the client module and the platform module.

Closing this as the environment is not a supported one.

Thanks for replying, @bochunz. I'll look into using HTTP requests from the Javascript server side to access the Google Sheets API.


Statistics are not working.

version 1.6.6
I launch the game from the launcher. But every stat is at 0:00

Howdy!
Aside from the version, you have not filled this issue with sufficient information to tackle the problem. Can I assume you are on Windows 10/11, the launcher is placed in an unprivileged folder and you are just clicking the launch button?
Still, this may be a hard one to debug. There are no other reports yet and I haven't personally experienced this issue myself. Furthermore the statistics should be getting saved in the AppData folder, which shouldn't be having any problems to be written into.

Sorry, I somehow assumed that was enough information.
On Windows 10. The launcher is in a folder not on drive C.
I play regularly and always from the launcher, but the stats are always zeros.
Also I don't know if it is supposed to work like this, but when I start the game, the launcher exits. I'm not sure how it is supposed to track the time, if it is not running, now that I think about it.
Sorry, I am stupid, probably was because of this.
Maybe add a hint on the settings page that not keeping the launcher running will disable the stats.


Improve performance during login and registration

There are performance issues when running /login or /register.
This is related to password hashing, and it should be fixed.

Fixed in latest commit


arm32 support

Search, emu support
Presets

Added armv7 constants and initial GadgetSearch support.
Need to:

Confirm GadgetSearch works (compare w ropper and ROPgadget)
Add emulation support

ldrdeq breaks emulation

GadgetSearch works /w regex matching for gadget violation
Working on GadgetEmu, requires final step emulation still need to:

Add final instruction to the analysis pane (for risc only?)

Presets
Fix max recursive emulations

Fix double gadgets bug
Find max recursion edge cases
Presets

Default search behavior broken for arm32


esbuild: Unterminated string literal with two different script tags

Describe the bug
I'm trying to build a component library and I use vue-tsc to generate the types for my component and vite to build my components into .umd.jsand .esm.js. I had an issue trying to generate types with vue-tsc with this component.
UiCTA.vue
<script setup lang="ts">
import { reactive } from 'vue';

interface Props {
  primary?: boolean;
}

const props = withDefaults(defineProps<Props>(), {
  primary: false,
});

const { primary } = reactive(props);
</script>

<template>
  <button class="btn" :class="{ primary }">
    <slot />
  </button>
</template>

<style scoped>
.btn {
  padding: 0.5rem 1rem;
}

.btn.primary {
  background: hsl(239, 100%, 27%);
  color: #fff;
}
</style>

I solved the issue by putting the Typescript declaration inside a script without the setup attribute as adviced here.
<script lang="ts">
interface Props {
  primary?: boolean;
}
</script>

<script setup lang="ts">
import { reactive } from 'vue';

const props = withDefaults(defineProps<Props>(), {
  primary: false,
});

However, I now get an issue with vite build as pasted below. It seems like there is a syntax error.
Reproduction
https://stackblitz.com/edit/vitejs-vite-kvjfs2?file=src/UiCTA.vue
System Info
System:
    OS: macOS 12.0.1
    CPU: (12) x64 Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz
    Memory: 176.20 MB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 16.14.2 - ~/.nvm/versions/node/v16.14.2/bin/node
    Yarn: 3.2.0 - /usr/local/bin/yarn
    npm: 8.5.0 - ~/.nvm/versions/node/v16.14.2/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 100.0.4896.127
    Firefox Developer Edition: 99.0
    Safari: 15.1
  npmPackages:
    @vitejs/plugin-vue: 2.3.1 => 2.3.1
    vite: 2.9.5 => 2.9.5

Used Package Manager
yarn
Logs
❯ yarn build
vite v2.9.5 building for production...
✓ 1 modules transformed.
[vite:vue] Transform failed with 1 error:
/Users/theo/projects/ui-library/packages/components/UiCTA/src/UiCTA.vue:17:37: ERROR: Unterminated string literal
file: /Users/theo/projects/ui-library/packages/components/UiCTA/src/UiCTA.vue:17:37

Unterminated string literal
15 |    setup(__props: any) {
16 |
17 |  const props = __props as { primarye';
   |                                       ^
18 |
19 |  cons }

error during build:
Error: Transform failed with 1 error:
/Users/theo/projects/ui-library/packages/components/UiCTA/src/UiCTA.vue:17:37: ERROR: Unterminated string literal
    at failureErrorWithLog (/Users/theo/projects/ui-library/node_modules/esbuild/lib/main.js:1603:15)
    at /Users/theo/projects/ui-library/node_modules/esbuild/lib/main.js:1392:29
    at /Users/theo/projects/ui-library/node_modules/esbuild/lib/main.js:666:9
    at handleIncomingPacket (/Users/theo/projects/ui-library/node_modules/esbuild/lib/main.js:763:9)
    at Socket.readFromStdout (/Users/theo/projects/ui-library/node_modules/esbuild/lib/main.js:632:7)
    at Socket.emit (node:events:526:28)
    at addChunk (node:internal/streams/readable:315:12)
    at readableAddChunk (node:internal/streams/readable:289:9)
    at Socket.Readable.push (node:internal/streams/readable:228:10)
    at Pipe.onStreamRead (node:internal/stream_base_commons:190:23)

Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

It's a bug in upstream. vuejs/core#5830
vue sfc playground

Closing as it's already been tracked in the upstream


IQN

Description

Motivation and Context

[ ] I have raised an issue to propose this change (required for new features and bug fixes)

Types of changes

[ ] 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)
[ ] Documentation (update in the documentation)

Checklist:

[ ] I've read the CONTRIBUTION guide (required)
[ ] I have updated the changelog accordingly (required).
[ ] My change requires a change to the documentation.
[ ] I have updated the tests accordingly (required for a bug fix or a new feature).
[ ] I have updated the documentation accordingly.
[ ] I have reformatted the code using make format (required)
[ ] I have checked the codestyle using make check-codestyle and make lint (required)
[ ] I have ensured make pytest and make type both pass. (required)

Note: we are using a maximum length of 127 characters per line

https://wandb.ai/qgallouedec/sb3-iqn

Honestly this is so cool. There are so many DQN variants and extensions I don't know how you keep up or even choose which to implement!


[Design] Clean up logging.

Removing scopes as we want to limit the number of scopes being created.
Added new log statements using the new log format string.

Please note: I have not yet modified the tests to reflect these changes. Will update them once the design looks ok.
Related PR in routing which has been updated: https://github.com/aspnet/Routing/pull/172
/cc: @rynowak

:watch:

Updated. /cc: @rynowak

:shipit:


phpbrew + blackfire.io?

Hi, I got a little bit lost while trying to install blackfire.io.
They have their own repositories for most common OSes, and also there seems to be a way to install it manually, but:
If you need to install the stack manually because every other installation methods
failed, please contact us and mention your platform and PHP version.

In the meantime, continue the following steps and you will be guided
through manual installation.

You are strongly advised to use the other installation methods available if possible as
they give you seamless upgrade methods.

Is this achievable with phpbrew extension manager?

@reafle to install the extension, you can:

Start from https://blackfire.io/docs/up-and-running/installation#install-probe-.
Download the extension by choosing "Linux Binary".
Rename the downloaded file to blackfire.so and move it to the extension directory.
Enable the extension (make sure you're using PHPBrew 1.23 or newer).

In my case it is:
$ wget https://packages.blackfire.io/binaries/blackfire-php/1.18.0/blackfire-php-linux_amd64-php-71.so
$ mv blackfire-php-linux_amd64-php-71.so ~/.phpbrew/php/php-7.1.10/lib/php/extensions/no-debug-non-zts-20160303/blackfire.so
$ phpbrew ext enable blackfire

Great, thanks! Version 1.23 rocks!
I wonder Is there a non-hacky way to map the version of blackfire-php-linux_amd64-php-71.so to php-7.1.10 so that the correct extension retrieval could be automated.
Would it be plausible to someday have blackfire installation through phpbrew via ext install or maybe even app get ?

I'm afraid not. app get is used to install something written in pure php, and ext install is for extension on pecl.php.net, which requires open-source.


Adding frdm_rw612 BLE support

Adding BLE support to frdm_rw612 board.

CI failure appears unrelated, https://github.com/zephyrproject-rtos/zephyr/pull/73886 may fix it- not sure if the filtering on that PR is correct to disable userspace tests for that board

Rebased, CI is now passing. Thanks @danieldegrasse for fixing the CI issue.


Support incremental load

This implementation allows us to load records incrementally.
This feature is silular to last_path of Embulk FileInput plugin.
I added following options

incremental_field Optional<List<String>>
last_record Optional<Map<String, Object>>

First execution
# config.yml
in:
  type: mongodb
  ...
  query: '{"field1": {"$gte": 3}}'
  incremental_field:
    - field2
    - field3

Plugin will create new query string and sort string.
query '{"field1": {"$gte": 3}}' # No change at the first execution
sort '{"field2":1,"field3":1}' # field2 asc, field3 asc

At the end of the execution, plugin will generate last_record field that contains last loaded values.
If user exec with embulk -c option, these values will be writen at the file.
$ embulk run -c diff.yml config.yml
...
[INFO] (main): Next config diff: {"in":{"last_record":{"field2":54216,"field3":123}},"out":{}}

$ cat diff.yml
in:
  last_record:
    field2: 54216
    field3: 123
out: {}

Second execution
$ embulk run -c diff.yml config.yml

Plugin will create following query and sort string.
These values are based on the values written at diff.yml and these values are different from at the first execution.
So we can achieve incremental load.
query '{"field1": {"$gte": 3}, "field2":{"$gt": 54216}, "field3":{"$gt": 123}}'
sort '{"field2":1,"field3":1}'

hmm. CI faild at only Java7 environments.
I'll fix unit test.

All test passed successfully.
Therere was a difference of element order in ConfigDiff between Java7 and Java8.
So I fixed unittest.
ConfigDiff diff = plugin.transaction(config, new Control());
String expected = "{\"int32_field\":32864,\"double_field\":1.23,\"datetime_field\":{\"$date\":\"2015-01-27T10:23:49.000Z\"},\"boolean_field\":true}";
assertEquals(expected, diff.getNested("last_record").toString());

ConfigDiff diff = plugin.transaction(config, new Control());
ConfigDiff lastRecord = diff.getNested("last_record");

assertEquals("32864", lastRecord.get(String.class, "int32_field"));
assertEquals("1.23", lastRecord.get(String.class, "double_field"));
assertEquals("{$date=2015-01-27T10:23:49.000Z}", lastRecord.get(Map.class, "datetime_field").toString());
assertEquals("true", lastRecord.get(String.class, "boolean_field"));

LGTM!

but conflicted

I fixed!

Thanks


Support NumPy scalars in ChainerX arithmetics

Fixes  #7000

Jenkins, test this please.

Jenkins CI test (for commit cb076452b238e4b0a29bb52c8b1d08ccf4397220, target branch master) succeeded!


Feature | project.yml definition checks and validations

What?

✅ Since our recommended use case for the Multi-Account reference architecture under the Well-Architected framework implies that we have default OUs and SCPs. We will need to add some validation scheme for our project.yml, probably some check before rendering the templates.
✅ It could be a list of rules that the project.yaml has to comply with and reflect this in warnings or errors.
✅ 1st Vaidation: Check that the OUs definition is not empty; later we can keep adding checks and conditions that will rise as necessary. Hence OUs section must exist and not be empty is a good start.

📒 CONSIDERATION: Can OUs and SCPs be left as optional? But with a big WARNING message stating that it is a bad practice. May be the user want to remove them, but it wouldn't be following best practice.
Why?

If we get a malformed project.yaml the rendering of the templates will failt with very little grace.

Concern

We should always have AWS Organizations OUs as best practice.

Would we then be leaving this to user preference?
What if the user want to dismiss account's OUs?

Just because by not creating OUs we would lose the SCPs and their default association to OUs to limit permissions. This should (must) be a must considering security by design. Although it is not (completely) wrong to make it optional, but then the user could disregard best practices if it undercuts OUs and SCPs.

Additional checks to consider:

Make sure the email addresses used to create the project are different from the default/example ones (maybe mark them with a special character?)
Check for other values that may create conflict down the line either from a project rendering standpoint or a architecture deployment standpoint.


GitHub Action stuck on "starting your workflow run"

https://github.com/atc0005/elbow/commit/e244ccce30359de8b7129e39d05ba63fcabef392/checks?check_suite_id=286266038
I had thought this was entirely on GitHub's end (and it still might be), but digging around in the community forums I see several users note that typos in the configuration can lead to the same behavior:

https://github.community/t5/GitHub-Actions/Stuck-on-quot-starting-your-workflow-run-quot/m-p/31702#M873
https://github.com/actions/setup-ruby/issues/26

Others didn't seem to have any such typo:

https://github.community/t5/GitHub-Actions/Action-blocked-on-quot-Starting-a-workflow-run-quot/m-p/33855#M1685
https://github.com/goreleaser/godownloader/pull/142/files#annotation_6656311066563110

See also #103, #106, #104, #105

I suspect I found it:
    runs-on: ${{ matrix.platform }}
    strategy:
      matrix:
        # Supported versions of Go
        go-version: [1.12.x, 1.13.x]

        # Supported LTS versions of Ubuntu Linux
        os: [ubuntu-16.04, ubuntu-18.04]

This line:
    runs-on: ${{ matrix.platform }}

Fixed by work on #108.


Pass through all unhandled responses to ChromeCastSpontaneousEventLis…

…tener
The documentation says that receivers will send spontaneous events to devices w/ the
request id = 0, but in practice this is not true. If I control a chromecast from my
phone I'm receiving updates in this library with requestIds that are absolutely invalid.
For example, this library creates id's that start at 0. On a fresh launch of this app
if I adjust the volume via my phone it is reporting ids in the 200's.

Merged, thanks.
Just a little formatting issue with imports order. I will add a new checkstyle rule so it will match default settings in IntelliJ Idea. I will update formatting everywhere myself.

Maybe you should export the rule you're using and check it in. I'm using a slightly modified version of Square's styles available at https://github.com/square/java-code-styles
Big difference is we use 2 spaces where you use 4.


[ListView] performance with more complicated renderRow

I was trying to implement a real time filtering using textInput. However, the re-rendering of the ListView is incredibly slow (about 1 seconds after finish typing).
This is what the renderRow function returns: (I simplified it a lot from my real renderRow, still slow)
<View>
  <TouchableOpacity onPress={this.props.onSelect}>
    <View style={cellstyle}>
        <Image
          source={require('image!logo')}
          style={[
            styles.cellImage,
            {resizeMode: Image.resizeMode.contain}
          ]}
        />

      <View style={styles.textContainer}>
        <Text numberOfLines={2} lineHeight={1}>
          {this.props.item.name.trim()}
        </Text>
        <Text>
          {digits(this.props.item.price)}
        </Text>
      </View>
    </View>
  </TouchableOpacity>
  <View style={styles.cellBorder} />
</View>

I have tried to replace TouchableOpacity with View, but that still doesn't help improve the performance much.

Is it related to rowHasChanged?

my rowHasChanged was the same as the documentation:
rowHasChanged: (r1, r2) => r1 !== r2

assume each of your item has unique id:
rowHasChanged:(r1, r2)=> r1.item.id !== r2.item.id

I doubt it would make anything faster, since r1 !== r2 does the reference checking, which should be more optimal than r1.item.id !== r2.item.id where you have to reference the object for the id, and compare them.

Any updates on this? Having similar performance issues.

+1

+1 for me as well, ListViews are becoming a bit of an issue in our app. Our renderRows contain the same if not more nodes than the example above.

+1 seeing this issue as well.

+1

Thank you for reporting this issue and appreciate your patience. We've  notified the core team for an update on this issue. We're looking for a response within the next 30 days or the issue may be closed.

This is a big issue for myself as well -- seems to be especially bad on older devices, newer devices don't seem to exhibit much issue with very complex rows.

+1 here. Tried SGListView instead of a standard one. A bit better, but it still feels like a hybrid app...

I tried with a very simple row on android, just (item) => (<Viev><Text>{item.name}</Text></View>) and even then it's slow to unusable.

Any updates on this one so far?

:+1:

+1

+1

Does removal (just asking) of the image make a difference?

The ListView performance is not very good in general. I have also an image in each row and the scrolling and re-rendering is not very smooth. Without the image the performance is much better.

The issue isn't list view then. It's probably the images rendering.  They seem to cause the JS event loop to stutter :(

Just something to try...set shouldRasterizeIOS={true} on your images in renderRow(). I've seen some performance improvements this way.

+1

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with ProductPains! Check out its new home: https://productpains.com/post/react-native/listview-performance-with-more-complicated-renderrow
ProductPains helps the community prioritize the most important issues thanks to its voting feature.
It is easy to use - just login with GitHub.
Also, if this issue is a bug, please consider sending a PR with a fix.
We're a small team and rely on the community for bug fixes of issues that don't affect fb apps.

+1

+1

I really don't think this should be closed. It's a big deal.

Here a gif showing the problem: https://www.dropbox.com/s/60z3hrs76ijs96c/SlowPepperoni.gif?dl=0

Try this:
shouldComponentUpdate(nextProps, nextState) {
return nextState.SOME_DATA !== this.state.SOME_DATA;
}

+1
I have similar this problem.

+1

i am facing same issues how to solve ?

+1


Uncaught TypeError: Cannot read property 'toLowerCase' of undefined

Got one more error like

Uncaught TypeError: Cannot read property 'toLowerCase' of undefined
at Object.convert.DataUnits (convert.js:214)
at :1:9

when I executed
convert.DataUnits(1);
I think we need to have a fallback option when other two params are null or undefined, just like we have it for:
convert.DataUnits();   // false
Thanks!

:+1:


Make WASM Tutorial Easier to follow

Received feedback that current WASM tutorial is hard to follow or distracted

I'm willing to serve as a guinea pig for that 😉

@hobofan great! I would stay on the lookout for updates to truebit-os as that will include wasm support soon and should make the process easier.


[iOS] :recycle: Use the defined Typography on each screen.

Issue

close #1144

Overview (Required)

The previously defined Typography style is now set up like Android.

Movie (Optional)

Before
After

@ry-itto @tkhs0604
For some reason, no reviewer has been assigned again, so please give us a code review. 🙇


Multiproject conversion

To convert several projects, it would be nice to have the ability to select multiple repositories and the ability to track conversion progress.

Thank you for taking the time to add this feature. Unfortunately, due to my inability to test this project anymore, I'm unable to accept large pull requests. However, feel free to promote this on your own fork.
I would also be interested in someone else taking over as maintainer, if they're willing to create an automated regression test. I think that's the only safe way to bring together the various forks. If that happens, we can reopen this PR.


rename package to gplibrary to match repo name

per https://github.com/convexengineering/solar/commit/07d17d3673b2d517ca278de7322cccf1ece73312#commitcomment-28826411

@bqpd I have created the branch libRename and changed all mentions of gpkitmodels to gplibrary, since it was really easy. This might make people upset though when their models start throwing errors. How does one deal with that?

@bqpd are we planning to implement this at any point, perhaps with a deprecation warning? It seems like low hanging fruit.

This also has to do with issue #142

Let's do it! unfortunately there's not quite room for a deprecation warning. I think the following steps should do:

[ ] change each repo's references to gpkitmodels to a reference to gplibrary
[ ] update the Jenkins scripts
[ ] run repo tests / model tests / etc. to make sure they run on jenkins.


Spacing and Footer Information

The sections for "Scholarships," "Jobs," and "Forum" are too closely packed, giving the layout a cramped feel and also The footer text and section feel small and cramped, making the additional content less accessible and inviting for the user to read.

Evenly space the feature boxes horizontally, giving each section enough margin and padding to ensure they don't look crammed together.
Use consistent font sizes and spacing for both the headings and the body text inside the boxes to improve readability.
I saw this website in Iphone 14 pro max mode and these were the errors so i would be happy to make it more responsive..
Please assign me this issue under gssoc'24 EXTENDED and hacktoberfest

@MrityunjaySingh09 this issue has been solved, so we are closing this issue.


Locale Calendar - Today at xxxx

Hi,
Another request in the .calendar() function's locale definition. For "Today at 12:00" you simply want sometime to just write "at 12:00". For example, "We'll open at 12:00".
Can we add another locale calendar definition for just [at], and an option in the .calendar() function to use that definition for today?
Thanks!

Why do you need to do this through the calendar function, rather than through format and your own logic?

tl;dr
Well, basically, since momentjs locale already have all the translation to the "at" syntax.
The longer answer:
I don't really need to go through the format, I just need to configure calendar with a different syntax for "Today". However holding my own translations to that syntax in ALL languages seems like something that should be shared to all, no?

Thanks for opening this request.
For now, we will not take this change.
Please re-open if you still have concerns.


Fix Uncaught TypeError: options.icon.slice is not a function when using L.AwesomeMarkers.icon  on startIcon and endIcon

https://github.com/mpetazzoni/leaflet-gpx/issues/161

Thanks for the PR! This breaks setting icons for categories/waypoint types though, which are expected to be set with:
markers: {
    wptIcons: {
      'Coffee shop': new L.AwesomeMarkers.icon({
        icon: 'coffee',
        prefix: 'fa',
        markerColor: 'blue',
        iconColor: 'white'
      })
    }
}

Can you find a way to tell those situations apart in _prepare_markers() ?

Thanks for your comment.
I am little bit confused, wouldn't that be this line https://github.com/mpetazzoni/leaflet-gpx/blob/main/gpx.js#L292 ?

Ah yes you're right, I already had to handle that case because type(null) === 'object'


How high can  ‘bit_acc’  reach  without BCH code？

Hello, tancik!

Excuse me, if you don't use BCH code, what is the decoding accuracy (bit_acc)?

Every 12 bytes (96 bits), only 5 bits can be corrected, right?

In encode_image.py and decode_image.py,  The BCH parameters you use are:BCH_POLYNOMIAL = 137
BCH_BITS = 5

According to the bchlib source code, my understanding is,
The entire packet can only be corrected by 5 bits, right?
In your case, packet is 12 bytes (96 bits), data is 7 bytes (56 bits, embedded link length)

Table 1 of the paper has a breakdown of bit accuracy for different camera / printer pairs - https://arxiv.org/abs/1904.05343
Yep, you are correct.

Table 1 of the paper has a breakdown of bit accuracy for different camera / printer pairs - https://arxiv.org/abs/1904.05343
Yep, you are correct.

Oh  I see.
Your reply is very helpful.
Thanks！
---Original---
From: "Matthew<EMAIL_ADDRESS>Date: 2021/1/5 06:45:44
To<EMAIL_ADDRESS>Cc: "Author"<author<EMAIL_ADDRESS>Subject: Re: [tancik/StegaStamp] How high can  ‘bit_acc’  reach  without BCH code？ (#31)
Table 1 of the paper has a breakdown of bit accuracy for different camera / printer pairs - https://arxiv.org/abs/1904.05343
Yep, you are correct.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Oh  I see.
Your reply is very helpful.
Thanks！
---Original---
From: "Matthew<EMAIL_ADDRESS>Date: 2021/1/5 06:45:44
To<EMAIL_ADDRESS>Cc: "Author"<author<EMAIL_ADDRESS>Subject: Re: [tancik/StegaStamp] How high can  ‘bit_acc’  reach  without BCH code？ (#31)
Table 1 of the paper has a breakdown of bit accuracy for different camera / printer pairs - https://arxiv.org/abs/1904.05343
Yep, you are correct.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Adds an example market catalogue middleware

Coverage decreased (-0.03%) to 94.259% when pulling 4c8e1beca058c19c2df7b1e2ab1493103d8d1af8 on lunswor:add-example-middleware into 4f08f389a3c5c2722b60252055b143cfdfefb0f7 on liampauling:master.


MinGW warns about 'I64' ms_printf length modifier

This is just a warning.  We could suppress this with -Wno-format.  It may be preferable to find an ISO compatible way to solve this though.
Compiling results in lots of:

heartbeats-simple\src\hb.c:215:5: warning: ISO C does not support the 'I64' ms_printf length modifier [-Wformat=]

I'm not seeing this in AppVeyor builds (anymore?) and can't recall exactly where else we might've seen it before.  This is not an issue anybody has complained about and I can't reproduce it, so I'm closing the report.


Considering add a cached boto storage

From https://stackoverflow.com/a/42912118:
import hashlib

from django.conf import settings
from django.core.cache import cache
from storages.backends.s3boto3 import S3Boto3Storage

class CachedS3Boto3Storage(S3Boto3Storage):
    """ adds caching for temporary urls """

    def url(self, name):
        # Add a prefix to avoid conflicts with any other apps
        key = hashlib.md5("CachedS3Boto3Storage_%s" % name).hexdigest()
        result = cache.get(key)
        if result:
            return result

        # No cached value exists, follow the usual logic
        result = super(CachedS3Boto3Storage, self).url(name)

        # Cache the result for 3/4 of the temp_url's lifetime.
        try:
            timeout = settings.AWS_QUERYSTRING_EXPIRE
        except:
            timeout = 3600
        timeout = int(timeout*.75)
        cache.set(key, result, timeout)

        return result

When viewing galleries of files, every page load uses a newly signed URL so it can't be cached. This makes navigating page to page a bit laggier. This storage would return the same signed url for 3/4 of the lifetime of the signature.

One concern here is if the app has a database cache enabled, would this use a query for every URL rendered on the page?


Does Model Merger work with Flux models (Loras)?

Does Model Merger work with Flux models (Loras)? Can we merge several Flux LoRAs in one?

it surely could work, check https://x.com/jfischoff/status/1824183079549931736 by fal.ai
their flux trainer is using ai-toolkit, but i'm not sure what they use to merge loras yet


[Task]: Fix compatibility issues in GDrive connector with SL update 8 versions

Connector Name
module/gdrive (Google Drive)
Task Description
$subject.

Closing via ballerina-platform/module-ballerinax-googleapis.drive#156


Follow-ups on create-testnet-data v1

Roadmap

[x] https://github.com/input-output-hk/cardano-cli/issues/473#issuecomment-1828297866 (see also https://input-output-rnd.slack.com/archives/C031GFLC3DF/p1701118783237699) → Done in https://github.com/input-output-hk/cardano-cli/pull/512
[ ] https://github.com/input-output-hk/cardano-cli/pull/488#discussion_r1411731739
[ ] https://github.com/input-output-hk/cardano-cli/issues/473#issuecomment-1824238007
[X] Discuss the desired meaning of the --supply and the --supply-delegated flags. Do we want to keep the previous behavior that the total supply is --supply plus --supply-delegated?  Or do we want --supply to be the real total supply and the non delegated supply to be --supply minus --supply-delegated? → Done in https://github.com/IntersectMBO/cardano-cli/pull/581
[x] Use create-testnet-data instead of create-staked in cardano-node here.
[ ] Simplify the computation of the distribution: https://github.com/input-output-hk/cardano-cli/pull/512#discussion_r1423848579

Additional context
The first item above is kind about reintroducing https://github.com/input-output-hk/cardano-node/pull/4021, but behind a conditional flag:

Discussing this with @Jimbo4350 he came up with a glorious idea of having an option to write (or not) the delegated keys to disk. The current behavior is that it does not write them to disk after a request from the benchmarking team. Having this as an option gives us more flexibility for other use cases (i.e. tests) where we do need the stake keys available.

The last item above is about this remark from @carlhammann :

I can't comment directly on the line, because it's not (yet 😛) part of the diff, but the definition of distribution in the next line is an atrocity, IMO. It should be
distribution = concat $ zipWith (\params poolIx -> replicate (delegsForPool poolIx) params) >poolParams [1..]

or something similar, instead of relying on the semantics of list comprehensions with unused parameters.

Closing as remaining improvements and checks have been done by @palas (see https://github.com/IntersectMBO/cardano-cli/pull/646) and others the last weeks.


Add coveralls test reporting.

Automatically reports test coverage to Coveralls on successful Travis builds.

Changes Unknown when pulling bb5cc51d076d9f7a61cc472fd8b61e1c2fc4196d on SOI-32/implement-test-coverage into ** on master**.


Documentation - Accessibility tab cannot be initialized

Description
Accessing to the following URL https://ovh.github.io/ovh-ui-kit/?path=/story/design-system-components-action-menu-webcomponent--compact leds to an infinite initialization on the "Accessibility" tab.
Issue brought by: #656
Steps to reproduce

Go to: https://ovh.github.io/ovh-ui-kit/?path=/story/design-system-components-action-menu-webcomponent--compact
Reach Accessibility tab
See error

Expected behavior:
Accessibility tab should work.
Actual behavior:
Tab cannot initialize.

Frequency:
5 (always)
Additional information
Browsers:

Chromium (Version 84.0.4147.105)
Firefox 79.0

Operating System: GNU/Linux
Distribution: Ubuntu 18.04 (64-bit)

This should be a configuration issue, I'll check if something is missing.
The warning is not related to the accessibility tab. Normally, the change from | to / has already been done.


Sidebar not updating with latest piped values

Steps to reproduce the behaviour

Create Answer with title "Answer One"
Create Answer in a new section and pipe "Answer One"
Delete "Answer One"
Navigate back to the new section

Expected behaviour

Question title piped value is updated with "[Untitled Answer]"
Sidebar piped value is updated with "[Untitled Answer]"

Actual behaviour

Question title piped value is updated with "[Untitled Answer]"
Sidebar piped value is NOT updated with "[Untitled Answer]" and retains the old piped value
Hard reload does not resolve this issue
Editing the title resolves this issue

@adammahmood is this something we're deferring. It sounds similar to what @tgandrews mentioned to me this morning about the pain he's going to be having piping metadata. Shall we discuss in the next tech session?

We should definitely look at this soon-ish.
Currently, the sidebar does not attempt to fetch the latest data for a piped answer, therefore these values end up stale. We need to re-architect how piping values are stored and stop relying on the UI to rebuild pipe values.


Feat: Adding Contributor section

Describe the feature
I propose to add a contributor section involves creating a visually appealing layout showcasing profiles
Add ScreenShots

i propose to add contributor section after FAQ section
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@anuragverma108 kindly please assign this work to me under GSSOC'24

Assigned!
but create it in about us page not in index!
Make sure to attach before & after screenshots in your PR and include the issue number.
Also, please star the repo before working on it. Happy open-source contributing!


Refactor/plans in definition

:warning: WIP :warning:
:warning: DRAFT :warning:
Issue
https://github.com/gravitee-io/issues/issues/7693

🚀 CI was able to deploy the build of this PR, so you can now try it directly here
Notes: The deployed app is linked to the management API of APIM master. (Same login and password as APIM master)

📚  View the storybook of this branch here

Replaced by this PR https://github.com/gravitee-io/gravitee-api-management/pull/1947, with a new branch name to run e2e tests


routeProvider templateUrl accepting custom $http config

As 'templateUrl' may be request for json or html, different http "accept" value will cause for different return results. So it is suggested that custom $http config for $routeProvider

As 'templateUrl' may be request for json or html

How come? How would you write a template in JSON? Are you using special templates format / pre-processor?
In any case, if any configuration is to be added for template request it should be done on the https://github.com/angular/angular.js/blob/master/src/ng/templateRequest.js level. But before thinking about such conf options we need to better understand the use case.
@chiu0602 could you please provide more details on what you are trying to do?

When templateUrl is specified, http get with 'Accept' = 'application/json, text/plain, /' ($httpProvider.headers.common) as request header. But for the page I enquired, different content will be returned when accept header with value "application/json" or "text/plain" comes first
For my case, 'text/plain' should come first, now I have to set it on $httpProvider.headers.common, but this affect global setting of $http. So I fail to achieve my aim with current release of AngularJS.
Thanks.

OK, this sounds reasonable.

We should allow people enhancing httpOptions in a provider.

That has been possible for a while: https://github.com/angular/angular.js/commit/b2fc39d2ddac64249b4f2961ee18b878a1e98251