Uppercase a single letter

Welp, I was bored, so here's a tiny little issue I found in your template.
Hopefully this does not annoy you tooooo much. ;)

Thanks!


[Suggestion] ci: only enable one OS by default, document windows platform

Reasoning: A vast majority of Squeak/Smalltalk applications are platform-independent anyway, causing a considerable overhead in tedious build times for all projects. By documenting but disabling other platforms by default, we leave relevant teams the possibility to easily opt-in to multi-platform builds but reduce build times and also energy usage by default. At the same time, we encourage them to discuss cross-platform requirements and implications with their customer.
In addition to the pros above, there would also be some cons:

Teams might be unaware of cross-platform concerns (either a blessing or a curse?)
Redundant builds may reveal flaky/nondeterministic tests (but in such cases, the association with a certain OS will often be misleading)

@fniephaus What do you think? :) /cc @kdauer

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 15.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
12

Relevant Lines:
80

💛  - Coveralls


make basic things work on Android

Fixes https://github.com/rust-lang/miri/issues/3608

@bors r+

:pushpin: Commit 0f617aeb1d6c1e55e58543de427f7ba6e64bb550 has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit 0f617aeb1d6c1e55e58543de427f7ba6e64bb550 with merge 7bfc04914d78d3810a65a0e7ae8e9b43d20a8787...

:broken_heart: Test failed - checks-actions

@bors r+

:pushpin: Commit 2349d03bb5bec487588ecb09555f03b0f0d46ab8 has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit 2349d03bb5bec487588ecb09555f03b0f0d46ab8 with merge 97cd966092fa6b073f4a910c5d0642aa9724c789...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing 97cd966092fa6b073f4a910c5d0642aa9724c789 to master...


Fix BasketLiquidation objective

There are certain issues with limits in the BasketLiquidation objective. This PR aims to fix it

Here is a MWE to reproduce the issue
#=
# Liquidating a basket of tokens
This example illustrates how to use CFMMRouter.jl to liquidate a basket of tokens.
=#
using CFMMRouter
using LinearAlgebra

## Create CFMMs
cfmms = [
    ProductTwoCoin([14e3, 15e6], 0.99, [1, 2]),
    ProductTwoCoin([19e3, 11e6], 0.99, [1, 2]),
]

Δin = [0, 3000]

## Build a routing problem with liquidation objective
router = Router(
    BasketLiquidation(1, Δin),
    cfmms,
    maximum([maximum(cfmm.Ai) for cfmm in cfmms]),
)

## Optimize!
route!(router)

## Print results
Ψ = round.(Int, netflows(router))
println("Input Basket: $(round.(Int, Δin))")
println("Net trade: $Ψ")
println("Amount recieved: $(Ψ[1])")

#=
We can also see the list of individual trades with each CFMM:
=#
## Print individual trades
for (i, (Δ, Λ)) in enumerate(zip(router.Δs, router.Λs))
    tokens = router.cfmms[i].Ai
    println("CFMM $i:")
    println("\tTendered basket:")
    for (ind, δ) in enumerate(Δ)
        if δ > eps()
            print("\t  $(tokens[ind]): $(round(Int, δ)), ")
        end
    end
    println("\n\tRecieved basket:")
    for (ind, λ) in enumerate(Λ)
        if λ > eps()
            print("\t  $(tokens[ind]): $(round(Int, λ)), ")
        end
    end
    print("\n")
end

Without the fix, this computes:
Input Basket: [0, 3000]
Net trade: [32981, -45187377944]
Amount recieved: 32981
CFMM 1:
	Tendered basket:
	  2:<PHONE_NUMBER>1,
	Recieved basket:
	  1: 13991,
CFMM 2:
	Tendered basket:
	  2:<PHONE_NUMBER>3,
	Recieved basket:
	  1: 18991,


Feature/debug mode

Hi @php-rock, this is a great idea. But we can do the same at the middleware level.
Maybe it's worth making an event?

ProcedureAccepted - The event occurs on any correct/incorrect request before it is processed.
ProcedureSucceeded - The event occurs after a successful request
ProcedureFailed - The event occurs after a failed request

As far as developer mode is concerned, I would like to see slightly different data.
For example trace errors in the returned JSON.


Publish-gh-pages script fails -- error: git clone failed

🐛 Bug Report
When trying to publish changes to our github pages hosted site, the npm run publish-gh-pages step fails with the following error code (full output from powershell attached)
generate.js triggered...
feed.js triggered...
feed.js triggered...
sitemap.js triggered...
Site built successfully. Generated files in 'build' folder.
7ce717117293691c87678f7b3cc7fbe015e946de
fatal: Too many arguments.
Error: git clone failed
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! @ publish-gh-pages: docusaurus-publish
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the @ publish-gh-pages script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\artus\AppData\Roaming\npm-cache_logs\2020-01-30T23_49_07_563Z-debug.log
Have you read the Contributing Guidelines on issues?
Yes
To Reproduce
Windows 10
node (12.14.1)
npm (6.13.4)
yarn (1.21.1)

clone documentation repo locally
make edits in visual studio code
run command: cmd /C "set GIT_USER=[myuser] && set CURRENT_BRANCH=master && npm run publish-gh-pages"
fails :(

Note -- also fails in the same way when using yarn run publish-gh-pages
Expected behavior
My 1st time trying to push changes to our docusaurus site. I expected it to work without any errors.
Actual Behavior
npm run publish-gh-pages failed
Reproducible Demo
Repo that I'm trying to push changes for
https://github.com/OpenBCI/Documentation/tree/master
docusaurus-errortxt.txt

same issue with identical environment

Temporary solution!!
file website/node_modules/docusaurus/lib/publish-gh-pages.js
line 20: const GIT_USER = process.env.GIT_USER.trim();
just trim git user

This is what I have been using in azure and works just fine
git config --global user.email<EMAIL_ADDRESS>git config --global user.name "$(GH_USER)"
GIT_USER=$(GH_USER) npm run publish-gh-pages

Running it directly from my machine (windows 10) also worked without errors.
I hope it helps :)

The problem is caused by a whiteline that is appended to the GIT_USER env variable on windows 10 when you run the cmd /C "set GIT_USER=<GITHUB_USERNAME> && yarn deploy" command. It can be fixed by running cmd /C "set GIT_USER=<GITHUB_USERNAME>&& yarn deploy"  instead (note the missing space between <GITHUB_USERNAME> and &&).
It's quite unfortunate that the command is included in the official docs so probably a lot of windows users will encounter this problem.

Could you help to submit a PR for Windows? That would be much appreciated!

The problem is caused by a whiteline that is appended to the GIT_USER env variable on windows 10 when you run the cmd /C "set GIT_USER=<GITHUB_USERNAME> && yarn deploy" command. It can be fixed by running cmd /C "set GIT_USER=<GITHUB_USERNAME>&& yarn deploy" instead (note the missing space between <GITHUB_USERNAME> and &&).
It's quite unfortunate that the command is included in the official docs so probably a lot of windows users will encounter this problem.

Nice, it work ! Thank you sir.

I'm having this problem and i'm not even on Windows

Cloning into 'D:\Practice\React\Restaurant App\restaurantapp\node_modules.cache\gh-pages\https!akshaymalik75.github.io!React_forms'...
fatal: repository 'https://akshaymalik75.github.io/React_forms/' not found
I am getting this error even I have changed the repo name please help.


feat: 实现树状模式下可复制

👀 PR includes

✨ Feature

[ ] New feature

🎨 Enhance

[ ] Code style optimization
[ ] Refactoring
[ ] Change the UI
[ ] Improve the performance
[ ] Type optimization

🐛 Bugfix

[X] Solve the issue and close #1941

🔧 Chore

[X] Test case
[ ] Docs / demos update
[ ] CI / workflow
[ ] Release version
[ ] Other ()

📝 Description

🖼 Screenshot

feat: 实现树状模式下可复制
fix: 修复存在空数据时复制错误的问题

Before
After

❌
✅

🔗 Related issue link

🔍 Self-Check before the merge

[ ] Add or update relevant docs.
[ ] Add or update relevant demos.
[ ] Add or update test case.
[ ] Add or update relevant TypeScript definitions.

:tada: This PR is included in version @antv/s2-v1.37.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-v1.37.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-v2.0.0-next.4 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Container Mythbusters curated content

Content for curated article on video "Container Mythbusters"
satisfies issue #401

Pinging @bartlettroscoe on this. Since this is a busy time, let us know if you need help with this

@pagrubel, why was PR #401 closed and this new one opened?

@bartlettroscoe That was what I thought #5 of
Site/CuratedContentEditorialWorkflow.md instructed the author to do
Create a pull request that references the issue and close the issue.

why was PR #401 closed and this new one opened?

Sorry, I got confused.  I could have swan that I reviewed a PR for this content.
Please let me know when this is ready to review.

@bartlettroscoe I think its ready for review and can be published if you think all is well.

This is now published: https://github.com/betterscientificsoftware/betterscientificsoftware.github.io/blob/master/CuratedContent/ContainerMythbusters.md


PowerToys error

Microsoft PowerToys version
.074.1.0
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
General
Steps to reproduce
2023-10-18.txt
✔ Expected Behavior
nothing
❌ Actual Behavior
Error message when starting the PC and loading the application.
Other Software
No response

The problem is solved. It was created by uninstalling Autodesk. Just uninstall PowerToys and reinstall it and everything works again. Thank you


Avoid early exit when last step not converging

Solves #21 and in the process found a bug that if the last step didn't converge when attempting the last time step, the time loop would still exit even if not converged. This is now fixed and tested for.

Codecov Report

Merging #23 (89c6c6b) into main (233f247) will increase coverage by 1.82%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main      #23      +/-   ##
==========================================
+ Coverage   93.29%   95.12%   +1.82%
==========================================
  Files           7        7
  Lines         164      164
==========================================
+ Hits          153      156       +3
+ Misses         11        8       -3

Impacted Files
Coverage Δ

src/QuasiStaticSolver.jl
100.00% <100.00%> (+4.16%)
:arrow_up:

... and 2 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Investigate unification of standard and semi-virtual TIN classes

Currently, Tinfour includes two packages that use different versions of the IQuadEdge interface. The standard package uses QuadEdge, while the semivirtual package uses SemiVirtualEdge. The QuadEdge implementation is faster and simpler, but uses about twice as much memory as the SemiVirtualEdge implementation.   Both implementations have their places depending on the needs of an application.
The problem with the current implementation is that the classes in the two packages are substantially duplicates of each other.  Therefore, this Issue entry proposes an investigation into unifying the two sets of classes as much as possible.
Both QuadEdge and SemiVirtualEdge implement IQuadEdge, so the obvious approach for the unification would be to refactor the standard and semi-virtual classes so that that operate on the generic interface rather than on the specific classes that they use. Ordinarily, this approach would be "good coding practice" (or just plain common sense).  But, Tinfour has unique performance issues due to the large number of samples in some of the data sets it processes.
So far, efforts to change the code to use the generic-interface approach have encountered a performance penalty.  When linking together edges using the setForward() and setReverse() methods, there comes a point where it becomes necessary to specifically cast the IQuadEdge references to either SemiVirtualEdge or QuadEdge (depending on the implementation).  This is a classic case of Java "downcasting".  Because Java must specifically check object references at run time to verify that they represent instances of the relevant class, downcasting slows down program execution (this is in contrast to the case of "upcasting" where a specific instance is converted to a generic interface... since upcasting references can be verified and resolved at compile time, they carry no performance penalty).   In tests so far, the "generic" implementations have incurred a 20 to 30 percent performance hit due to downcasting overhead.  So avoiding downcasting is not just a fatuous preference.
Therefore I am currently investigating ways to organize the classes so the impact of downcasting is minimized. Doing so will involve some reorganization of the class-to-package assignments and general clean up of interfaces. More details will follow depending on the success or failure of the effort.

Shelving this issue for now.  The potential reduction in code size was not worth the extra cost in terms of processing overhead.  However, I was also discouraged by the complexity of the migration effort, so I'm not convinced this is impossible.  I may revisit this in the future


Empty space at the bottom of grid

I'm submitting a ...  (check one with "x")
[x] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, instead see https://github.com/ceolter/ag-grid/blob/master/CONTRIBUTING.md#question

Current behavior

Currently you can see extra empty space after last row. Reproduced when floating bottom row enabled.

Expected behavior

I guess this shouldn't be here.
Minimal reproduction of the problem with instructions

https://www.ag-grid.com/react-examples/examples/?fromDocs&example=floating-row
https://plnkr.co/edit/Of88H3?p=preview

ag-Grid version: 12.0.2

Browser: Chrome, Safari

Language: All

Hi,
This is expected behaviour - if you have a grid of a certain height where the rows + header heights dont exactly fit there will be an amount of space left over. You can try this by setting the grid height to 700px for example

Empty space always there, even grid height equals rows height.

Any news?

I have also encountered this issue. Is it possible to get it fixed?

That's the fix for it: https://github.com/ag-grid/ag-grid/commit/f037ba13ee728e2e0fc84208cb16ffd1029dcd7e#diff-374544943f4ef689f74fd75c395193d0R576

@petyosi yup, but this breaks pinned columns. Also, check RTL mode.


beta.18 undocumented breaking theme change?

Just updated from beta.17 to beta.18 and noticed the following behavior change:
When importing material-ui components from another module, theme overrides are no longer applied.
My scenario is the following:
I have two node modules: "ui" and "ui-common"
The "ui-common" module builds some wrapper components which include material-ui components (ie. specific groupings of text fields).
Normally when I include these "ui-common" components in my "ui" project, the styling overrides applied to the main project (via ) also apply to the composite components loaded in from the "ui-common" module. This is as desired.
However, after updating to beta.18, all composite components in my "ui-common" project are now falling back to default material-ui theming (notably, the primary color is the default blue). All composite components within the main "ui" project are still getting the correct theme applied.
Has anyone else noticed this behavior? It's unfortunately kind of a complex setup to reproduce and I didn't see anything the the changelog for beta.18 that would have affected this use case.

Whoops, nevermind. Looks like it was some weird build issue on my end. Everything is working again. Please ignore this issue.


Refactor PolarsBackend.Series to improve ops using scalar values

This is a refactor in the PolarsBackend.Series to reduce memory usage by avoiding the need to create series for scalar values. Instead it is using the lazy backend and expressions (lazy series) to pass down the literal value to Rust.
Benchmark (using main)
small = Explorer.Series.from_list(for _i <- 0..100, do: :random.uniform(1_000))
medium = Explorer.Series.from_list(for _i <- 0..1_000, do: :random.uniform(1_000))
big = Explorer.Series.from_list(for _i <- 0..10_000, do: :random.uniform(1_000))
biggest = Explorer.Series.from_list(for _i <- 0..100_000, do: :random.uniform(1_000))

Benchee.run(
  %{
    "add w/ list duplicated" => fn input -> Explorer.Series.add(input, 42) end,
    "add using expressions" => fn input ->
      df = Explorer.DataFrame.mutate_with(Explorer.DataFrame.new(left: input), fn ldf ->
        [result: Explorer.Series.add(ldf["left"], 42)]
      end)

      df["result"]
    end
  },
  memory_time: 2,
  inputs: %{
    "small" => small,
    "medium" => medium,
    "big" => big,
    "biggest" => biggest
  }
)

Benchmark results
Operating System: Linux
CPU Information: AMD Ryzen 9 5950X 16-Core Processor
Number of Available Cores: 32
Available memory: 15.52 GB
Elixir 1.14.0
Erlang 25.1

Benchmark suite executing with the following configuration:
warmup: 2 s
time: 5 s
memory time: 2 s
reduction time: 0 ns
parallel: 1
inputs: big, biggest, medium, small
Estimated total run time: 1.20 min

Benchmarking add using expressions with input big ...
Benchmarking add using expressions with input biggest ...
Benchmarking add using expressions with input medium ...
Benchmarking add using expressions with input small ...
Benchmarking add w/ list duplicated with input big ...
Benchmarking add w/ list duplicated with input biggest ...
Benchmarking add w/ list duplicated with input medium ...
Benchmarking add w/ list duplicated with input small ...

##### With input big #####
Name                             ips        average  deviation         median         99th %
add using expressions         2.41 K        0.41 ms     ±4.53%        0.41 ms        0.46 ms
add w/ list duplicated        0.62 K        1.60 ms     ±0.64%        1.60 ms        1.63 ms

Comparison:
add using expressions         2.41 K
add w/ list duplicated        0.62 K - 3.87x slower +1.19 ms

Memory usage statistics:

Name                      Memory usage
add using expressions          5.04 KB
add w/ list duplicated       157.30 KB - 31.22x memory usage +152.26 KB

**All measurements for memory usage were the same**

##### With input biggest #####
Name                             ips        average  deviation         median         99th %
add using expressions         430.01        2.33 ms     ±1.08%        2.32 ms        2.40 ms
add w/ list duplicated         64.69       15.46 ms     ±1.53%       15.48 ms       15.76 ms

Comparison:
add using expressions         430.01
add w/ list duplicated         64.69 - 6.65x slower +13.13 ms

Memory usage statistics:

Name                      Memory usage
add using expressions       0.00492 MB
add w/ list duplicated         1.53 MB - 310.28x memory usage +1.52 MB

**All measurements for memory usage were the same**

##### With input medium #####
Name                             ips        average  deviation         median         99th %
add w/ list duplicated        6.14 K      162.83 μs     ±5.05%      163.79 μs      180.28 μs
add using expressions         4.75 K      210.32 μs     ±8.18%      206.90 μs      259.20 μs

Comparison:
add w/ list duplicated        6.14 K
add using expressions         4.75 K - 1.29x slower +47.49 μs

Memory usage statistics:

Name                      Memory usage
add w/ list duplicated        16.67 KB
add using expressions          5.04 KB - 0.30x memory usage -11.63281 KB

**All measurements for memory usage were the same**

##### With input small #####
Name                             ips        average  deviation         median         99th %
add w/ list duplicated       30.52 K       32.77 μs    ±68.29%       29.75 μs       98.42 μs
add using expressions         5.07 K      197.28 μs     ±9.70%      193.42 μs      261.05 μs

Comparison:
add w/ list duplicated       30.52 K
add using expressions         5.07 K - 6.02x slower +164.51 μs

Memory usage statistics:

Name                      Memory usage
add w/ list duplicated         2.61 KB
add using expressions          5.04 KB - 1.93x memory usage +2.43 KB

**All measurements for memory usage were the same**

Closes #368

@nallwhy @josevalim please take a look :)

There is something more we can optimize, but we can do this later.
https://github.com/nallwhy/explorer/commit/9b77387ed41d133348a10f8417f00f35ad4baff9
Operating System: macOS
CPU Information: Apple M1 Pro
Number of Available Cores: 8
Available memory: 32 GB
Elixir 1.14.0
Erlang 25.1.1

Benchmark suite executing with the following configuration:
warmup: 2 s
time: 5 s
memory time: 2 s
reduction time: 0 ns
parallel: 1
inputs: big, biggest, medium, small
Estimated total run time: 1.80 min

Benchmarking add using broadcasting with input big ...
Benchmarking add using broadcasting with input biggest ...
Benchmarking add using broadcasting with input medium ...
Benchmarking add using broadcasting with input small ...
Benchmarking add using expressions with input big ...
Benchmarking add using expressions with input biggest ...
Benchmarking add using expressions with input medium ...
Benchmarking add using expressions with input small ...
Benchmarking add w/ list duplicated with input big ...
Benchmarking add w/ list duplicated with input biggest ...
Benchmarking add w/ list duplicated with input medium ...
Benchmarking add w/ list duplicated with input small ...

##### With input big #####
Name                             ips        average  deviation         median         99th %
add using broadcasting        1.53 K      655.48 μs   ±207.79%      344.96 μs     7877.22 μs
add using expressions         1.12 K      893.28 μs   ±173.96%      501.37 μs     7753.85 μs
add w/ list duplicated        1.00 K     1002.88 μs   ±178.99%      533.17 μs     8928.72 μs

Comparison:
add using broadcasting        1.53 K
add using expressions         1.12 K - 1.36x slower +237.81 μs
add w/ list duplicated        1.00 K - 1.53x slower +347.40 μs

Memory usage statistics:

Name                      Memory usage
add using broadcasting         1.29 KB
add using expressions         11.56 KB - 8.97x memory usage +10.27 KB
add w/ list duplicated         5.73 KB - 4.45x memory usage +4.45 KB

**All measurements for memory usage were the same**

##### With input biggest #####
Name                             ips        average  deviation         median         99th %
add w/ list duplicated        233.55        4.28 ms    ±53.21%        3.03 ms       11.04 ms
add using broadcasting        233.05        4.29 ms    ±55.15%        3.04 ms       11.67 ms
add using expressions         226.12        4.42 ms    ±57.30%        3.03 ms       11.32 ms

Comparison:
add w/ list duplicated        233.55
add using broadcasting        233.05 - 1.00x slower +0.00906 ms
add using expressions         226.12 - 1.03x slower +0.141 ms

Memory usage statistics:

Name                      Memory usage
add w/ list duplicated         5.73 KB
add using broadcasting         1.29 KB - 0.22x memory usage -4.44531 KB
add using expressions         11.56 KB - 2.02x memory usage +5.83 KB

**All measurements for memory usage were the same**

##### With input medium #####
Name                             ips        average  deviation         median         99th %
add using broadcasting        7.12 K      140.38 μs   ±437.13%       81.83 μs     1022.77 μs
add using expressions         1.97 K      507.46 μs   ±237.73%      262.67 μs     7374.36 μs
add w/ list duplicated        1.80 K      556.72 μs   ±226.41%      306.83 μs     7218.52 μs

Comparison:
add using broadcasting        7.12 K
add using expressions         1.97 K - 3.61x slower +367.08 μs
add w/ list duplicated        1.80 K - 3.97x slower +416.34 μs

Memory usage statistics:

Name                      Memory usage
add using broadcasting         1.29 KB
add using expressions         11.56 KB - 8.97x memory usage +10.27 KB
add w/ list duplicated         5.73 KB - 4.45x memory usage +4.45 KB

**All measurements for memory usage were the same**

##### With input small #####
Name                             ips        average  deviation         median         99th %
add using broadcasting       16.56 K       60.39 μs   ±670.69%       21.83 μs      179.03 μs
add using expressions         2.21 K      452.54 μs   ±251.95%      241.62 μs     7165.44 μs
add w/ list duplicated        1.89 K      530.40 μs   ±231.28%      292.85 μs     7205.33 μs

Comparison:
add using broadcasting       16.56 K
add using expressions         2.21 K - 7.49x slower +392.14 μs
add w/ list duplicated        1.89 K - 8.78x slower +470.01 μs

Memory usage statistics:

Name                      Memory usage
add using broadcasting         1.29 KB
add using expressions         11.56 KB - 8.97x memory usage +10.27 KB
add w/ list duplicated         5.73 KB - 4.45x memory usage +4.45 KB

**All measurements for memory usage were the same**

I think there is something we can optimize more, but that's for next time.

For sure! I think that way going to be faster, but it's hard to maintain because some functions may have 3 possible branches: (series, series), (series, scalar), (scalar, series). We could use macros to make the work less painful, but I agree that this could be for the next time :)

@nallwhy oh, I see now. Yeah, it's a big difference for medium to small series.
I'm going to merge this, but if you want to implement this optimization, go ahead :D
I think it may worth for the long term :)


the graphics memory（16 GB）, the max batch size I used was 2

the graphics memory（16 GB）, the max batch size I used was 2
2 is too small,  16G???????????

You can try different batch size of course as long as it doesn't crash. I guess my implementation is not efficient enough and/or the HR-Net itself is too complicated.

the graphics memory（16 GB）, the max batch size I used was 2
2 is too small, 16G???????????

Hi henbucuoshanghai,
You can try different batch size of course as long as it doesn't crash. I guess my implementation is not efficient enough and/or the HR-Net itself is too complicated.


Add range input for minZoom/maxZoom

As described in #432 I've added <input type="range/> as new interface for entering minZoom/maxZoom
Demo: https://1314-84182601-gh.circle-artifacts.com/0/artifacts/build/index.html
Fixes #432

Thanks @orangemug , with Chrome it works nicely, with Firefox it's not possible to slide the handles (they get resetted immediately).

@orangemug maybe the Firefox issue is related to a race condition:
https://stackoverflow.com/questions/30550314/react-input-range-slider-not-propagating-last-value-to-its-owner-in-chrome (in this post Chrome isn't working)

@orangemug maybe the Firefox issue is related to a race condition:
https://stackoverflow.com/questions/30550314/react-input-range-slider-not-propagating-last-value-to-its-owner-in-chrome (in this post Chrome isn't working)

@pathmapper I think you're partially correct, however I fixed the race condition and it was still failing. The only thing I can think is that the 'UI pause' (because of the slow performance) when the DOM updates causes the range slider in FF to 'halt'.
I've changed the code to only update the value when the pointerup event fires. I think this is the best we can do at the moment, what do you think?
Demo https://1336-84182601-gh.circle-artifacts.com/0/artifacts/build/index.html#0.34/0/0

@orangemug I think it's fine to only update the value on pointerup.
However, there is still something wrong (FF 67.0 on Linux):
Most of the times, if the mouse is moved after pointerup, the slider gets resetted:

Hmmm, well this is a pain, I don't get that issue on FF on OSX. I'll try to sort out a VM for testing see if I can get to the bottom of it.

Here is the FF on Linux issue with open Components tab of the latest CI profiling build. Including PointerUp everything works as expected. The issue appears if the mouse if moved after PointerUp: The value resets and the state switches to editing: true.

Nice try, but the issue is still there.
Maybe this helps (we are using display: flex):
https://stackoverflow.com/questions/57225124/react-vertical-range-slider-always-resets-to-default-ignoring-new-state
Or use something like this:
https://www.npmjs.com/package/react-rangeslider

Here a gif with the logging which was added in the last commit:

Thanks @pathmapper, that's helpful. I've added an additional attempt at a fix and some more logging.

We're getting close!
The previous issue is fixed (the value is not resetted anymore if the mouse is moved after PointerUp):

Now the value is resetted only if there is no mousemove after PointerUp before dragging the slider again:

The issue is still there, the logs for getDerivedStateFromProps are undefined

maxZoom logs itself as getDerivedStateFromProps[1] I've move some logging about so it actually logs that value.

New logs:


Update shift and rotate Simplicity implementions in Coq.

This update matches the definitions from Haskell that are used in jet specifications.
In particular, the Simplicity SHA256 compression function implementation now matches the Simplicity specification defined in Haskell.

Fixes #261

@apoelstra does your CI run the Coq build?


Didn't work, no wireless connection profile found

This examples doesn't work, I cannot find any wireless connection profiles. I found this examples that does work: http://studio-geek.com/archives/1031.
var tetheringManager = NetworkOperatorTetheringManager.CreateFromConnectionProfile(NetworkInformation.GetInternetConnectionProfile());
string AP_SSID = "network";
string AP_PASS = "fubarfubar";
var accessPointConfig = new NetworkOperatorTetheringAccessPointConfiguration()
{
Ssid = AP_SSID,
Passphrase = AP_PASS
};
await tetheringManager.ConfigureAccessPointAsync(accessPointConfig);
await tetheringManager.StartTetheringAsync();

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 4b9db9be-5d4c-611a-674e-c463859a03b5
Version Independent ID: a3729731-8407-e6b4-891f-46235f26c299
Content: Internet Connection Sharing - Windows IoT
Content Source: windows-iotcore/connect-your-device/InternetConnectionSharing.md
Product: windows-iot
GitHub Login: @saraclay
Microsoft Alias: saclayt

Hi @FinitelyFailed, we were able to solve this and it looks like 1) you may have created a WiFi Profile at first and 2) you will need to add this to your manifest:

Hope this works for you.


Unable to reach the sensor value

Do I need to put
require 'sysfs_one_wire'
into my rb code for OneWire to work ?
I'm sorry, I have no idea about how to use this gem... please gives me some examples, thanks!!

Hi,
It works as any gem. You 'require' it in your code or you use bundler for
it.
On Mar 13, 2016 8:20 AM, "Damao Tim Lin"<EMAIL_ADDRESS>wrote:

Do I need to put
require 'sysfs_one_wire'
into my rb code for OneWire to work ?
I'm sorry, I have no idea about how to use this gem... please gives me
some examples, thanks!!
—
Reply to this email directly or view it on GitHub
https://github.com/asapp/one_wire/issues/1.

Hi @elmatou ,
I couldn't get the value,
OneWire.load('/sys/bus/w1/devices/28-031581efbaff/w1_slave')
soil_temp = OneWire::Thermometer.value()

home/pi/site_server/clone_from_mac/dashing_test/jobs/convergence.rb:7:in `<top (required)>':

undefined method `value' for OneWire::Thermometer:Class (NoMethodError)

I don't how to reach the value, please advise, thanks you!

Hi @elmatou
I couldn't get the value from Thermometer
Below is my code, can you help me to take a look ?
OneWire.load('/sys/bus/w1/devices/28-031581efbaff/w1_slave')
current_soil_temp = OneWire::Thermometer.new.last_value

There are no data been called. Please advise how to reach the value, thanks!!!

Ok, with OneWire.devices we can see your sensor is well recognized.
You have to get the Thermometer object, with :
soil_sensor = OneWire.devices.first
# or
soil_sensor = OneWire.load('/sys/bus/w1/devices/28-031581efbaff')

then you have access  to some methods such as :
current_soil_temp = soil_sensor.value                  # current temperature in °C
soil_sensor.last_value          # last checked temperature, return nil if you didn't called value before
soil_sensor.id                       # 1wire device uniqe id
soil_sensor.name                 # device uniqe name
soil_sensor.w1_slave           # return the w1_slave file, as in 'cat'
soil_sensor.dump                 # return an array which can be stored in database for later loading

I'll add it to the readme for easier usage.
clear enough ?

It works very very good! Thanks!!!!! @elmatou

Nice !
Of course you can contribute.
As I only have a DS18B20 in my project I cannot test with other devices/sensor.
It would be very helpful to have the "dump" of cat w1_slave of other sensors you have by you.
Or some intel on any other device (eprom, ibutton, switch) you can have a hand on.

@elmatou I trying to test soil moisture, by using capacitor not resistance. Do you have any recommendation sensor or maybe diy one ?
And also I looking forward to add some water flow sensors,
http://www.aliexpress.com/item/New-Water-Flow-Sensor-Switch-Meter-Counter-Hall-Sensor-Flowmeter-0-3-6L-min-Free-shippingFree/32271887755.html?spm=2114.01010208.3.76.YomE2I&ws_ab_test=searchweb201556_2,searchweb201602_4_10034_507_10020_10001_10002_10017_10010_10005_10011_10006_10021_10003_10004_10022_10009_10008_10018_10019,searchweb201603_1&btsid=d4ec5b5d-5567-4bb9-a3e6-e248c4949536
http://www.aliexpress.com/item/G1-2-Water-heater-flow-sensor-water-flow-meter-YF-S201-10-PCS-LOT/1188662717.html
Not sure how to read the data, do you have any thoughts ?

I'm not familiar with the device... maybe you can get the pulse with PiPiper. maybe you should optoisolate the pulse signal to protect the raspi...


feat(table): MatCellDefWhen

Feature Description
Like matRowDefWhen, it was interesting to have this capability for MatCellDef as in the example below.
Use Case

  <ng-container cdkColumnDef="name">
     Name
     {{element.name}}
    {{element.firstname}} {{element.lastname}}
  </ng-container>

For your scenario you could do something like
 {{ isFullname ? element.firstname + ' ' + element.laastname : element.name }}

Yeah, like @mrmokwa points out, you'd be better off doing something within an expression for your example.
I don't see us adding this feature since a table cell is a much more tightly scoped than a row; the row has configuration for all of the cells, so there's no other way to swap row templates than with the when predicate, but for a cell you can just use a standard ngIf, etc.


[BUG] New-PnPAzureCertificate: Decoded string is not a valid IDN name. (Parameter 'unicode')

Reporting an Issue or Missing Feature
An Issue
Expected behavior
Command creates a certificate and outputs it
Actual behavior
The following error is thrown:

Steps to reproduce behavior
Just run the following with no parameters:
New-PnPAzureCertificate
What is the version of the Cmdlet module you are running?
2.1.1
Which operating system/environment are you running PnP PowerShell on?
Windows Server 2019, latest updates
PowerShell 7.3.4
However, if I do the same thing on a Windows 11 Beta channel running PowerShell 7.3.4 I get no error.
I'm not sure what New-PnPAzureCertificate is angry about on my server.

@New-Authentication - have you tried adding some name to the certificate ? Maybe that can help ?
I can't repro on Win 11 and neither can you.
So, if it works on Win 11, but not on server , not sure if we can do anything here.

Closing due to lack of response.
Also , if it works in win11 but not in win server 2019 , not sure we can fix it either 🤷♂

Hi @gautamdsheth
Sorry for my lack of response. Something is broken with how PowerShell 7 works on Windows Server. Get-ADUser (and other standard ADDS commands) produces the same error. I don't know how to describe it any better. It is run of Azure with a standard Azure Windows 2019 image.
I know it can be hard to troubleshoot but to say that because it works on Windows 11 then there is nothing to do is not really a valid reason for not doing anything.

Same issue on 7.3.5

Oops - thought I was in PowerShell GitHub - of cause you probably couldn't fix this if it is a generic PowerShell 7 problem.
Sorry for my bitterness!


Fix #1572 router-link in nested component behaves differently than one in a page-route

Fix #1572
The reason for bug is that the state of the parent component is modified

Thanks for the PR, at then end we are going with #2029


Update to React 15.1.0

I also switched to the development (unminified) builds which give useful warnings which you typically want when building examples in a bin.

Just saw this was merged – thank you.


Roots mailing list

just set this up: http://groups.google.com/group/roots-theme
feel free to post any questions or feature requests up on there
/cc @mwrather @r3df @dwightgunning

re-opening for now to bring some more attention to the google group - http://groups.google.com/group/roots-theme


Update package_service_mapping.json with service names from REST API docs

Syncing the ref content ToC names with the ones used for the REST APIs (taken from the Swagger sources) here: https://docs.microsoft.com/en-us/rest/api/

@erickson-doug,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

Codecov Report

Merging #1217 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1217   +/-   ##
=======================================
  Coverage   55.96%   55.96%
=======================================
  Files        2426     2426
  Lines       65118    65118
=======================================
  Hits        36444    36444
  Misses      28674    28674

Continue to review full report at Codecov.

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


University Nangui Abrogoua

add una

@Assamoi-Intelligence I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host univ-una.ci not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for understanding!

Hi @philipto  this is the university official website URL https://www.univ-na.ci/

@Assamoi-Intelligence I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host univ-una.ci not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for understanding!


"ClientAuthError: Token renewal operation failed due to timeout" for authProvider.getAccessToken()

Please follow the issue template below. Failure to do so will result in a delay in answering your question.

Library

[X]<EMAIL_ADDRESS>or<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>
Important: Please fill in your exact version number above, e.g<EMAIL_ADDRESS><EMAIL_ADDRESS>Framework
ReactJS using react-aad
Description
authProvider.getIdToken() is working as expected. authProvider.GetAccessToken() is not and is producing ClientAuthError: Token renewal operation failed due to timeout. errors. Keeps looping through this before ultimately timing out.
Error Message

Security

[ ] Is this issue security related?

Regression

[ ] Did this behavior work before?
Version:

MSAL Configuration
// ../utils/authProvider.js
import { MsalAuthProvider, LoginType } from 'react-aad-msal';
// import { Logger, LogLevel } from "msal";

// The auth provider should be a singleton. Best practice is to only have it ever instantiated once.
// Avoid creating an instance inside the component it will be recreated on each render.
// If two providers are created on the same page it will cause authentication errors.
export const authProvider = new MsalAuthProvider(
  {
    auth: {
      authority:
        'https://login.microsoftonline.com/tenant_id/',
      clientId: '<client_id>',
      postLogoutRedirectUri: 'https://' + window.location.origin + '/admin/',
      redirectUri: 'https://' + window.location.host + '/admin/',
      validateAuthority: true,
    },
    cache: {
      cacheLocation: 'sessionStorage',
      storeAuthStateInCookie: true,
    },
  },
  {
    scopes: ['openid', "profile", "user.read"],
  },
  {
    loginType: LoginType.Redirect,
    // When a token is refreshed it will be done by loading a page in an iframe.
    // Rather than reloading the same page, we can point to an empty html file which will prevent
    // site resources from being loaded twice.
    tokenRefreshUri: window.location.origin + "/auth.html"
  }
);

Reproduction steps
// App.js
import React from 'react';
import { BrowserRouter as Router, Route, Switch } from 'react-router-dom';
import { AzureAD, AuthenticationState } from 'react-aad-msal';

// Component Imports
import Authentication from './authentication';
import Dashboard from './dashboard';
import Navbar from './navbar';

// Utilities Imports
import { authProvider } from '../utils/authProvider';

const App = () => {
  return (
    <AzureAD provider={authProvider} forceLogin={true}>
      {({ logout, authenticationState }) => {
        switch (authenticationState) {
          case AuthenticationState.Authenticated:
            return (
              <Router basename={process.env.PUBLIC_URL}>
                <Navbar logout={logout} />
                <Switch>
                  <Route path='/dashboard/' component={Dashboard} />
                  <Route path='/' component={Authentication} />
                </Switch>
              </Router>
            );
          default:
            return
Authenticating...
;
        }
      }}
    </AzureAD>
  );
};

export default App;

// ./authentication/Authentication.js
import React, { useState, useEffect } from 'react';
import { authProvider } from '../../utils/authProvider';

const Authentication = () => {
  const [idToken, setIdToken] = useState('');
  const [accessToken, setAccessToken] = useState('');

  useEffect(() => {
    if (!idToken) {
      getIdToken();
    }

    if (!accessToken) {
      getAccessToken();
    }
  }, [idToken, accessToken]);

  let getIdToken = async () => {
    let token = await authProvider.getIdToken();
    setIdToken(token);
  };

  let getAccessToken = async () => {
    let token = await authProvider.getAccessToken();
    setAccessToken(token);
  };

  if (idToken.idToken) {
    return
Have idToken
;
  } else {
    return
Fetching token...
;
  }
};

export default Authentication;

Expected behavior
That it should retrieve the accessToken for validation.
Browsers/Environment

[X] Chrome
[ ] Firefox
[ ] Edge
[ ] Safari
[ ] IE
[ ] Other (Please add browser name here)

In the above image, can you try opening the login.microsoftonline.com url in the "The current window does not have permission to navigate the target from to..." error message in its own tab? What happens when you do that?

One thing to note is that your redirect uri must either be http://localhost:<port> or an https url. I am guessing you will see an error message regarding your redirect uri, which must be registered in teh Azure Portal and needs to be https or localhost.

Ok looks, like I might not have the app configured properly on my end, so I'll read up on that to see where I need to enable this in Azure.
As for the redirect, I do have my local dev environment https://<IP_ADDRESS>/admin/ in the Azure App Registration redirect. I do get the "Unsafe" error because I'm not using a certificate locally, but seems to work.


Fatal error: Uncaught Error: Class 'FFMpeg\FFMpeg' not found

It seems like there were many questions similar to what i am suffering from.
I took a look at them, and I still can’t figure out how to do it. I would like to test how to capture a thumbnail image from the existing video file(later, my goal is to capture a thumbnail from live streaming video).
Below is current set up in my server

Q
A

Linux Ubuntu
16.04

PHP version
7.0

Nginx Version
1.10.3

FFmpeg Version
FFmpeg 2.8.11 (installed by 'apt install ffmpeg)

I checked the location of ffmpeg(/usr/bin/ffmpeg) and ffprobe(/usr/bin/ffprobe)
echo "extension=ffmpeg.so" >> /etc/php/7.0/fpm/php.ini
Following the instructions, I installed PHP-FFMpeg through composer.
$composer require php-ffmpeg/php-ffmpeg
Then, I created .php file and copied and pasted the below codes from "Basic Usage" example.
<?php
header("Content-Type: text/html;charset=UTF-8");

  $ffmpeg = FFMpeg\FFMpeg::create([
                'ffmpeg.binaries'  => '/usr/bin/ffmpeg',
                'ffprobe.binaries' => '/usr/bin/ffmprobe',
                'timeout'          => 3600, // the timeout for the underlying process
                'ffmpeg.threads'   => 1,   // the number of threads that FFMpeg should use
            ], $logger);

  // Open your video file
    $video = $ffmpeg->open('./test.mp4');

    $video
        ->filters()
        ->resize(new FFMpeg\Coordinate\Dimension(320, 240))
        ->synchronize();

    $video
        ->frame(FFMpeg\Coordinate\TimeCode::fromSeconds(10))
        ->save('test_frame.jpg');
  }
?>

When i run the php file, it returns the below error
Fatal error: Uncaught Error: Class 'FFMpeg\FFMpeg' not found
As I take a look at the similar problems, I saw some people mention about composer.json file.
As I do not know what to do with that file, I simply copied and pasted it under the same directory.
Since I am new to using PHP-FFMpeg, I know there is something i am missing. I will be very appreciated if you guide me through step by step. If there is anything that i need to do on my end, I will respond as quick as possible.

You need to put this before the first line
require_once 'vendor/autoload.php';

because you need to include the composer libs that you provided. So this is basic for composer, because you need to load them when you want to use them.

I put require_once 'vendor/autoload.php'; right after the php tag.
This gives me the below error.
Warning: require_once(vendor/autoload.php): failed to open stream: No such file or directory

How did you required the lib? Global or not? If global, try to require it just in the directory.

My guess is that you're missing a use statement:
use FFMpeg;

by using this code
use FFMpeg;
i m getting error

Parse error: syntax error, unexpected 'use' (T_USE)

Did you load the composer autoloader?

Yeah i used composer autoloader.
then the new error comes.
Fatal error: Uncaught Alchemy\BinaryDriver\Exception\ExecutionFailureException: ffmpeg failed to execute command '/usr/bin/ffmpeg' '-y' '-i' 'tmp/how_to_repair_water_damage_mobile_display_fault_solution.mp4' '-async' '1' '-metadata:s:v:0' 'start_time=0' '-vcodec' 'libx264' '-acodec' 'libfaac' '-b:v' '1000k' '-refs' '6' '-coder' '1' '-sc_threshold' '40' '-flags' '+loop' '-me_range' '16' '-subq' '7' '-i_qfactor' '0.71' '-qcomp' '0.6' '-qdiff' '4' '-trellis' '1' '-b:a' '128k' '-vf' '[in]scale=320:240 [out]' '-pass' '1' '-passlogfile' '/tmp/ffmpeg-passes599179429625aan82w/pass-5991794296323' 'export-x264.mp4' in /var/www/OnlineClasses/core/plugins/FFMpeg/vendor/alchemy/binary-driver/src/Alchemy/BinaryDriver/ProcessRunner.php:100 Stack trace: #0 /var/www/OnlineClasses/core/plugins/FFMpeg/vendor/alchemy/binary-driver/src/Alchemy/BinaryDriver/ProcessRunner.php(72): Alchemy\BinaryDriver\ProcessRunner->doExecutionFailure(''/usr/bin/ffmpe...') #1 /var/www/OnlineClasses/core/plugins/FFMpeg/vendor/alchemy/binary-driver/src/Alchemy/Bi in
I dont know why this error accurs, i have searched on Google and i dont find any relevant answers.

I'll take a look as soon as I've time to investigate..


clicking on pod delete button can cause command to be executed in minisplit

we should always prefer main-Terminal execution for these commands
k get pods
click on a pod
split the layout so that the k get pods table is in a minisplit
click the delete button
the command should be executed in a main terminal, but seems to be executed in the k get pods minisplit

i think the problem is in BaseSidecar
it has a state variable that is the tab, and it updates that tab onResponse. which doesn’t seem like it even is consistent. e.g. if in one Tab, which say has a 2-way split; if i first click to open the sidecar in split1, then click to open the sidecar in split2… now the sidecar will “point” to split2…

i can’t remember why tab needs to be a state variable. but robably because historically we needed a tab for its REPL controller.
i think the right way to do this is to use the new pexecInCurrentTab which takes care of all of this for us, rather than using tab.REPL.pexec


minor, refine exception handling.

Can one of the admins verify this patch?

Pull Request Test Coverage Report for Build 3045

0 of 1 (0.0%) changed or added relevant line in 1 file are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.005%) to 23.894%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

server-base/src/main/java/org/apache/kylin/rest/service/AdminService.java
0
1
0.0%

Files with Coverage Reduction
New Missed Lines
%

core-cube/src/main/java/org/apache/kylin/cube/inmemcubing/MemDiskStore.java
3
78.42%

Totals

Change from base Build 3044:
0.005%

Covered Lines:
14336

Relevant Lines:
59999

💛  - Coveralls


[Bug] LexicalClearEditorPlugin not working

as title

Please try the latest version and feel free to open this issue again. Thanks!


Add file streaming examples

Closes #133
Changes:

Add a section about files and streaming to the Fetch Data example.
Rewrote the File Server example to explains how people can write their own file server from scratch.

I tried to keep the file server example as simple as possible, but it may be missing some exception handling. Should I add it?

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

I have modified my PR with your recommendations.
I restored most of the original text for the std/http file server.

Also, small detail, but Deno.open doesn't work on Deno Deploy. It makes sense since GitHub isn't appropriate for storing large files, but it does make Deno Deploy less compatible with non-Deno Deploy code 😕

This is expected - not all Deno namespace APIs are available on Deploy

Also, small detail, but Deno.open doesn't work on Deno Deploy. It makes sense since GitHub isn't appropriate for storing large files, but it does make Deno Deploy less compatible with non-Deno Deploy code confused

We are working on it!


Use pre-commit.ci for automatic formatting

Most useful would be a rst formatter, because the docs are often a point of "conflict" -re-wrapping them in different editors can be the source of conflicts and huge diffs.
I didn't find an automated formatter yet; linters exist, though:

https://github.com/myint/rstcheck
https://github.com/Lucas-C/pre-commit-hooks-markup

Other possible uses:

flake8 + autopep8 or similar
maybe even black to completely "outsource" Python formatting
if we increase use of types (#506), we could run mypy, too

Maybe https://github.com/dzhu/rstfmt


Convert to latest swift syntax

I just updated to the compiler recommended settings to silence all warnings.

Thanks!


Add a login page for Google login

Write a page where users who login are redirected to choose their roles

Delayed this to iteration 3 since need to learn how to use JS to send information to the REST API

Once again delayed to iteration4


individual line length modification

so i'm pretty happy how this flowchart looks:

(click to see in original size)
but it's really itching me to modify the following:

first line segment after / 250 Ok. / should be shorter
first line segment after / 250 Done. / should be longer
line after < ^DATA > no should be shorter

i naively thought that if we could just inject modification values from the flowchart syntax - cause "fixing" the whole auto layout algorithm probably would never work for every chart -, we then would only need to add this modification value to the line length in code... well, i got the first part done:
diff --git a/release/flowchart-1.3.4.js b/release/flowchart-1.3.4.js
index fe6051a..b76c816 100644
--- a/release/flowchart-1.3.4.js
+++ b/release/flowchart-1.3.4.js
@@ -446,6 +446,7 @@
     this.symbol = symbol;
     this.connectedTo = [];
     this.symbolType = options.symbolType;
+    this.lineLengthMod = options.lineLengthMod;
     this.flowstate = (options.flowstate || 'future');

     this.next_direction = options.next && options['direction_next'] ? options['direction_next'] : undefined;
@@ -1381,14 +1382,17 @@

         chart.symbols[symbol.key] = symbol;

-      } else if (line.indexOf('->') >= 0) {
+      } else if (/-[\{-\d\}]*>/.test(line)) {
         // flow
-        var flowSymbols = line.split('->');
-        for (var i = 0, lenS = flowSymbols.length; i < lenS; i++) {
-          var flowSymb = flowSymbols[i];
+        var flowSymbols = line.split(/-[\{]*([-\d]*)[\}]*>/);
+
+        for (var j = 0, lenS = flowSymbols.length; j < lenS; j+=2) {
+          var i = j/2;
+          var flowSymb = flowSymbols[j];

           var realSymb = getSymbol(flowSymb);
           var next = getNextPath(flowSymb);
+          realSymb.lineLengthMod = parseInt(flowSymbols[j+1], 10) || 0;

           var direction = null;
           if (next.indexOf(',') >= 0) {
@@ -1401,8 +1405,8 @@
             chart.start = realSymb;
           }

-          if (i + 1 < lenS) {
-            var nextSymb = flowSymbols[i + 1];
+          if (j + 2 < lenS) {
+            var nextSymb = flowSymbols[j + 2];
             realSymb[next] = getSymbol(nextSymb);
             realSymb['direction_' + next] = direction;
             direction = null;

that allows to do:
foo->bar-{10}>zap-{-20}>fop

and in (javascript) code we have for the bar object lineLengthMod:10 and for zap object lineLengthMod:-20 attributes.
for the second part i just could not findout where to modify the (first) line (segment) lengths in the spots i wanted to, so i'm giving up for now and hope someone else can figure it out. :heart_eyes:
ps. source for the image can be found at http://ix.io/edM

sorry for late reply!
closing cause i-forgot-and-now-doing-other-things :sweat_smile:

Can the latest version of flowchart.js modify the individual line length?

No

My English is poor.  I can't show you my disappointment to know this message.


Sending giant emoji

Now that it's possible to change the thread emoji, is there some way that the giant one can be sent (the one you get when you press and hold the emoji)?

Yea, it's possible. When I got some more time, I will implement to sendMessage a this feature!

Going to close this because we can track it in #234


[core] Handle --load-info-json format selection errors

Closes #9388
Template

Before submitting a pull request make sure you have:

[x] At least skimmed through contributing guidelines including yt-dlp coding conventions
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8 and ran relevant tests

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check all of the following options that apply:

[x] I am the original author of this code and I am willing to release it under Unlicense

What is the purpose of your pull request?

[x] Core bug fix/improvement

ugh.. the top level error handling is an absolute mess. This still does not bring download_with_info_file on parity with download, especially with regards to --ignore-errors.
Good enough for now, but we should consider refactoring


(CSCI 310, Group #16) Create a window for transferring files to the server, connect window to sftp.py

Resolves #21, resolves #22
Enables the "transfer data" button on the main window. When clicked, a new window appears, allowing the user to input their login information and select a source and destination folder. The send button will not work until all fields are filled, and a popup will show after a successful and unsuccessful transfer.
Screenshot of file transfer GUI:
https://drive.google.com/file/d/1w-jbgjfCv6OvQug9V6v4aUyvif-q_YxC/view?usp=share_link
Group 16 team members:

Hamaad Zahid
Jarrod Pirino
Kris Heck

File transfer GUI added to new tab interface, closing request


Directory/file name parsing and auto mp3 conversion hacks

If you're interesting, I added some directory and filename parsing if metadata isn't found.   I also added auto-conversion to mp3 since html audio tags only support a few types... and it's browser specific.   It's quite a mess, so converting everything to mp3 is safer and allows all of my music to work.   Of course ffmpeg is required for this functionality.

Hey @pennyfx , this is some pretty cool stuff.
It's a little bit of a mess, but hey, so is all my code ;)
If you add a simple check for ffmpeg support to that transcoding function (see the youtube download function for how to do this) then I'll merge.

Thanks, I'll clean it up a little.  Just wanted to see if you were interested in this functionality or not.

Yeah I am.
There was an issue where someone was requesting this (#14), but I closed it and just added it to the trello board as a feature request.
It's good to see it getting implemented though! :+1:

Closing this PR due to inactivity


Tom components 20221114

I think this approach is superseded by the one I'm about to merge from #56, so closing for now.


fix: check if enough balance to pay the fee

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


Shorthand CRUD methods.

When calling a query shorthand, infer the CRUD method from the start of the query name:

getById = retrieve
createNewUser = create
etc.

Longhand example:
$resultSet = $databaseClient["collectionName"]->query("getById", ["id" => 1]);

Shorthand example:
$resultSet = $databaseClient["collectionName"]->getById(["id" => 1]);

Method starting with "create" or "insert" will return a string ID.
Method starting with "get" or "retrieve" will return a ResultSet.
Method starting with "set" or "update" will return an integer number of affected rows.
Method starting with "remove" or "delete" will return an integer number of affected rows.

Closed in preference of #51


BASIC program execution errors

When I executed the following code:
[CLASSIC]
Lbl A
Input Str0
If Str0="..":Goto Q
Goto A
Lbl Q
[MATHPRINT]

The program hangs, without switching to MATHPRINT.  A few seconds later, when the Run indicator stops spinning, I pressed [ON], which caused the RAM to reset.  Another thing is that when I ran a test version of BonziCE, the input field looked weird: (the picture was taken w/ TI-Connect CE, when the cursor was "off".  The plot mark wasn't there before.  Also, the homescreen is a bit messed up when I ran BonziCE (input Field is lower, the $ is the start of the field).  The homescreen was cleared before the Input.
Error picture:

Fixed with 84097eb


Add [Coinbase Withdrawals coinbase.com] #exchange GDTYD6…FHHVYN

Address: GDTYD6BWWZIOHD2PPLTFQYW3KVTT5YEF2EVWMTUJTVGQ4U2UBKFHHVYN
Tags: exchange
Title: Coinbase Withdrawals  Domain: coinbase.com
Requested by @renea4343

Can you please clarify you reason you believe this is a Coinbase Withdrawals address?
Can you help with transactions numbers?

@Renea4343 Please feel free to give updates and reopen this issue if you have more information.


Disable Jormungandr tests in CI

Issue Number

Overview

88bf743cbb47c8f0406df73ae5a179abc75e11be
Disable Jormungandr tests in CI

Comments

hydra failure:
https://hydra.iohk.io/build/4641890/nixlog/1
Failures:

  test/unit/Cardano/Wallet/Primitive/FeeSpec.hs:379:9:
  1) Cardano.Wallet.Primitive.Fee.prop_rebalanceSelection The fee balancing algorithm converges for any coin selection.
       Falsifiable (after 2281 tests):
         SaveMoney
         Coin {getCoin = 31946}
         selection (before):
         inputs:
           - 1st 00f9f187 (~ 163271 @ 61646472...64722d33)
           - 3rd ab297d60 (~ 164465 @ 61646472...64722d33)
           - 2nd e973e54b (~ 165206 @ 61646472...64722d31)
           - 2nd e1a6aa15 (~ 178461 @ 61646472...64722d31)
         withdrawal: 876
         reclaim: 570642
         outputs:
           - 95750 @ 61646472...64722d32
           - 170567 @ 61646472...64722d33
           - 125186 @ 61646472...64722d31
           - 132069 @ 61646472...64722d30
           - 65925 @ 61646472...64722d32
         change: [82782]
         deposit: 381078

         selection (after):
         inputs:
           - 1st 00f9f187 (~ 163271 @ 61646472...64722d33)
           - 3rd ab297d60 (~ 164465 @ 61646472...64722d33)
           - 2nd e973e54b (~ 165206 @ 61646472...64722d31)
           - 2nd e1a6aa15 (~ 178461 @ 61646472...64722d31)
         withdrawal: 876
         reclaim: 570642
         outputs:
           - 95750 @ 61646472...64722d32
           - 170567 @ 61646472...64722d33
           - 125186 @ 61646472...64722d31
           - 132069 @ 61646472...64722d30
           - 65925 @ 61646472...64722d32
         change: [31946]
         deposit: 381078

         delta (before): 189564
         delta (after):  240400
         total fee:      240400
         remaining fee:  0

  To rerun use: --match "/Cardano.Wallet.Primitive.Fee/prop_rebalanceSelection/The fee balancing algorithm converges for any coin selection./"

Randomized with seed<PHONE_NUMBER>

Finished in 752.0834 seconds
2315 examples, 1 failure, 6 pending

#2215

bors r+

bors r+

bors r+


minor grammar fix

"Adam's" as in "Adam has"

@trommel ahh, wasn't noticeable from my first read .. sorry bout that, :+1:

No worries!


Fix unquoted params value in curl example

In the curl example for using parameters with SQL, fix the JSON body that is provided in the example so that it quotes the params identifier.

Thanks!


chore: bump metapackage dependencies

Holding https://github.com/open-telemetry/opentelemetry-js/pull/1382 release for this.

@open-telemetry/javascript-approvers need 2 more approvals here so I can merge it before the 0.10.2 release. Was hoping to get that today.


[Doc] Fix doc for the wrong default value of maxPendingChunkedMessage

Motivation
There are inconsistencies between the code and the documentation regarding the default value of maxPendingChunkedMessage.
Here is the discussion thread in the mailing list: https://lists.apache.org/thread/w4vfb5jd5xo9dndhrt7655pbhxn5xdvk
Modifications

Fix the wrong default value of maxPendingChunkedMessage. Use 10 as the default value.

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Documentation

[x] doc

/pulsarbot run-failure-checks


IBInjection error

I followed your instructions as well as many other tutorials, but I keep getting:
IDEBundleInjection.c: Error 3587 loading bundle '/Users/fbartolom/Library/Developer/Xcode/DerivedData/inArrivoHD-ebjdiuuwdpdvchgmpsyqkpvvvyhw/Build/Products/Debug-iphonesimulator/inArrivoHD Tests.xctest': The bundle “inArrivoHD Tests” couldn’t be loaded because it is damaged or missing necessary resources.
And no solution in view.

This looks like a project setup issue rather than a restkit issue.


No rules were found matching this request

Description
After typing "verba start" in the console. The browser shows this error message.
I'm using a virtual environment, and I installed it with pip.
Is this a bug or a feature?

[ ]  Bug

Steps to Reproduce
verba start

Additional context
✔ Connected to Weaviate
ℹ Setting up components
ℹ Retrieve Config From Weaviate
✔ Config Saved in Weaviate
ℹ Setting READER to BasicReader
ℹ Setting CHUNKER to TokenChunker
ℹ Setting EMBEDDER to ADAEmbedder
ℹ Setting RETRIEVER to WindowRetriever
ℹ Setting GENERATOR to GPT3
INFO:     Started server process [44741]
INFO:     Waiting for application startup.
INFO:     Application startup complete.

Apologies, it was a port issue. I changed the port from 8000 to another, and it’s working fine now.


Code insufficient

When will you upload the complete code?

https://github.com/htzheng/CM-GAN-Inpainting/issues/1#issuecomment-1410393041
Likely will not be released due to the linked statement.


Upgrade ScanCode to version 3.0.2

See https://github.com/nexB/scancode-toolkit/releases/tag/v3.0.2
The ScanCode result JSON format was changed a bit: a json array
named headers has been added and the files_count node has been
moved underneath it. Adding a fallback to compute that value based
on the distinct paths keeps backwards compatibility.
Signed-off-by: Frank Viernau<EMAIL_ADDRESS>

This change is

Also, please refer to issue #1381 in the commit message to auto-close it upon merge.

/rebase

As appveyor is green the failure in travis cannot be important, it seems indeed to be unrelated, thus merging this PR finally.


C++ API -> "Pick and Place" tutorial is not ROS2-ified

https://github.com/ros-planning/moveit2_tutorials/issues/15

I am closing this since, pick and place will be replaced by MTC

The "pick and place" under "Tutorials" section indeed uses MTC. But there's another one under "Examples -> C++ API", which doesn't seem to be ROS2-ified.

We need to remove that tutorial is the functionality is deprecated


adds a French Name to the desktop file

I have changed a lot things to get my change upstream. Therefore I have to decline your pull request. I will also remove my fork since the things I wanted has been upstreamed.


[reactive-element] Error: "The following properties on element ... will not trigger updates as expected because they are set using class fields"

Description
Until today everything was working with my projects as expected. However, today I've update the lit library and I started seeing a bunch of errors like the following:
Uncaught (in promise) Error: The following properties on element [...] will not trigger updates as expected because they are set using class fields: [...]. Native class fields and some compiled output will overwrite accessors used for detecting changes. See https://lit.dev/msg/class-field-shadowing for more information.

Two independent projects started having the same problem after updating lit. Not really sure what version I was using before but I was working on these projects for the past few weeks and I was keeping them up-to-date.
I read the related article. I am using TS to write my components. I am little bit lost here. It looks like my code is good to work with decorators. It was working well before. Now everything is broken. I am not even sure how to reproduce this. One of the elements that triggers these errors has the following properties declaration:
export default class UserAvatarElement extends LitElement {
  static get styles(): CSSResult {
    return css`...`;
  }

  private _user?: IUser;

  /**
   * Whether has user information
   */
  @state() protected hasUser = false;

  /**
   * Set with the user. The computed user initials.
   */
  @state() protected userInitials?: string;

  /**
   * Whether the main dropdown is opened
   */
  @property({ type: Boolean }) opened = false;

  /**
   * @attribute
   */
  @property({ type: Object })
  get user(): IUser | undefined {
    return this._user;
  }

  set user(value: IUser | undefined) {
    const old = this._user;
    if (old === value) {
      return;
    }
    if (old && value && old.key === value.key) {
      return;
    }
    this._user = value;
    this.requestUpdate('user', old);
    this._processUser(value);
  }
  ...
}

This element produces the following error:
The following properties on element user-avatar will not trigger updates as expected because they are set using class fields: hasUser, userInitials, opened. Native class fields and some compiled output will overwrite accessors used for detecting changes. See https://lit.dev/msg/class-field-shadowing for more information.

I tried to move all initializers to the constructor but nothing changed. Any help is be appreciated.

The walk-around for now is to change exportConditions in the web-dev-server.config.mjs file to contain production. This way I can continue working on my projects.

After spending even more time I pinpointed the problem to the latest release of @web/dev-server-esbuild version 0.3.0. In one of my projects this was the only change in the dependencies. After downgrading it to 0.2.16 all started working again.
I will cross reference this issue there.

"useDefineForClassFields": false is the right setting to apply here:  https://lit.dev/docs/components/decorators/#enabling-decorators
Though how to apply that setting can be complicated with tools that wrap tsc or don't use tsc at all like `esbuild.
It seems like from https://github.com/evanw/esbuild/issues/2220 and https://github.com/modernweb-dev/web/issues/1921 that the problem here is that @web/dev-server-esbuild isn't reading tsconfig.json and so useDefineForClassFields can't be set to false. I'll close this as a bug in @web/dev-server-esbuild.
cc @LarsDenBakker and @daKmoR if they know if that bug has a solution.

Solution to the problem:

update to "@web/dev-server-esbuild": "^0.3.1"
set useDefineForClassFields to false in tsconfig
load tsconfig.json file with

        esbuildPlugin({
            ts: true,
            target: "auto",
            tsconfig: fileURLToPath(new URL('./tsconfig.json', import.meta.url)),
        }),

(according to https://github.com/modernweb-dev/web/blob/master/packages/dev-server-esbuild/CHANGELOG.md#031 there is a possible setting to keep the tsconfig.json file in sync)

i use rollup
"experimentalDecorators": true, "useDefineForClassFields": false,
the config is not helpful for me


Reset Elasticsearch 7.13 links in scripts

Description
In scripts bin/opensearch-env and config/jvm.options, the Elasticsearch 7.13 (current) version links have now been reset to Elasticsearch 7.12 version links.
Issues Resolved
#949
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

start gradle check


separation of the subscription and the download function wanted?

Is a separation of the subscription and the download function wanted? If no separation - we can simply use Tom's solution and delete other code

Tom's solution allows a user to subscribe and to optionally add the argument --download
# connect, and simply echo messages
pywis-pubsub subscribe --config local.yml

# connect, and download messages
pywis-pubsub subscribe --config local.yml --download

This solution looked OK for me.

Yes, download is done via on_message_handler --> get_data.
So the download is executed directly when a message is received. This could lead to problems with large downloads. In docker it was passed to Aria2 service and the download was taken over by the service so that the sub program can go directly to the next message again. In the split solution this is implemented via a shared directory.
Possibly such problems do not occur when using in the wis2box, I have no problem to take first the solution of Tom and wait and see if it comes to problems. We can always change. Therefore I have now deleted the two folders for the separate scripts and the docker solution parts


kubectl-convert: add a plugin for kubectl.

Description of changes
Add the kubectl plugin: kubectl convert (cf: kubectl-convert: add a plugin for kubectl.  )
A plugin for Kubernetes command-line tool kubectl, which allows you to convert manifests between different API versions.
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Clone of https://github.com/NixOS/nixpkgs/pull/177498

@ofborg eval


{% highlight html %} are using the wrong font

Some pages like this have some code samples in Times, not Courier. Seems like the class and data-lang attributes are going wrong somewhere.

It looks like it's a difference between the snippet embed types.
{% highlight html %} is using Times while {% include_code ... is using the correct font.

I literally see no difference between this bug and https://github.com/google/WebFundamentals/issues/1861
Has anyone touched anything to do with the code styling?
1.) I don't see the times samples as Sam has screenshotted
2.) I've moved everything to Roboto Mono
If someone can confirm that is the case for https://material-dot-web-central.appspot.com/web/fundamentals/design-and-ui/media/video/add-a-video and close this bug and the other bug (https://github.com/google/WebFundamentals/issues/1861) I'd appreciate it.

Fixed! Whee!


Alert or confirm in render path hangs the dev server and waits for user input in the console

I encountered the issue when playing around with a timer with an alert and setting minutes to -1 running locally (I have the code in one of my repos)

the minimal case is the example app from the fresh init script with the following code in index.tsx:

I guess that it shouldn't wait for the user's input in the console.
Not sure if it's the right repo for the issue though.
deno 1.23.1
fresh 1.0

This is working as designed. Deno implements alert and confirm. The page is server side rendered, and as such all of the component code for a given page is executed on the server when a user makes a request. If you want some code to only execute on the client (if it is in an island), wrap it in an if (IS_BROWSER).

Oh, thank you for clarifying that. The timer code I wrote is actually on an island.
I'm closing the issue then.
Cheers.


updating mincer version to fix sass issue

Looks like there are a few test failures.

It seems the jump between 1.2.4 and 1.2.5 causes some of the cryptic ENOENT errors, apart from the almost-routine hash differences.

Oh never mind, it seems to be fixed by changing the hash (see https://github.com/netsgnut/connect-assets/commit/a3986b4c28361d4e06c6ff360dd8afb9b09e2578).
FYI I have actually have a branch that bumps mincer, uglify-js and csswring that could supersede this one. However I was getting lazy and tested upon the unreleased mincer 1.3.0 instead (see https://github.com/netsgnut/mincer/tree/postcss4-compat and nodeca/mincer#200 for potentially how it could go).
I will do a PR once mincer 1.3.0 is released officially.

well i've updated the hashes

@dmtaub Agreed. That said I am still not sure why the hashes are changed every time mincer is updated.

FWIW, I think having hashes in the tests at all is a disincentive to trust the tests...

Can you explain? My motivation for having them in the tests is that you should be explicit about changing assets — if you make a JavaScript-specific update and it changes a CSS file, I’m glad the tests fail. I’m open to change though if presented a solid argument! :)

Merged #313 instead, since it included all of this and more!
Thanks @dmtaub for the work, sorry these commits didn’t make it in, but at least your problem is fixed! :100:


Release/3.7.0 prepare

Checkout the 3.7.0 release prepare changes. Once we agree on this content, we can raise the PR to merge Dev to master and release 3.7.0

This LGTM if we stage some of the other PRs into dev as well before main!

Yes. this PR is too large at the moment. so lets merge this and start focused branches that we


Update model files and requirements to MuJoCo 2.3.7

All compiled model files and the requirements.txt need to be updated to mujoco 2.3.7.

fixed 56ff3cd0f19cbfd3ba31682e2f5547711c5a8b80


Fix keyboard opening problem on click items on mobile

This pull request solves the problem mentioned in the issues: #1803 #1511.
Problem explanation:

Enable live-search and multiple
Open the page in a mobile device
Click an item from the list
Keyboard shows up, this is annoying and bad UX.

I've made a tiny addition to the condition for focusing to searchBox inside onClick event handler of list items. It now also checks the mobile option before focusing the input. If mobile option is present, it won't focus, thus preventing the keyboard opening issue.
I couldn't reproduce the issue in an isolated environment, but I've added a related test-case into test.html. You can copy the test-case I added to test.html into current test.html to see the problem occurring, and then check the example in my branch to see it's fixed.

Closing all old PR against master.


Update integration test config to stack training data

The previous PR https://github.com/ai2cm/fv3net/pull/1655 changed the RF training to take in stacked data. The integration test config needs to be updated to stack the training data. It also needs to use a cache directory when training to avoid errors that occur when trying to stack data that is already stacked with a multi index. The latter issue will be resolved in the future when tf datasets are used in training.
Breaking change:
Since our current train-diags-prog argo workflow currently uses the same configuration file for validation during training and offline diags, there'd have to be a new distinction between the validation-config.yaml passed to the training workflow vs the test data config passed to the offline diags.
This isn't that big of a deal as the using validation data in training can be optional, but it'd be a breaking change in the sense the parameter formerly known as validation-data-config would now be optional and only be used for training time validation, and a new parameter test-data-config would have to be provided for the offline diags.
If this is approved, I'll make those changes to vcm-workflow-control as well.

Is there a way we can reduce the copy-and-paste between the three configs? In particular, do we expect the mapper_config part to ever change?
Yes, sometimes- if the preprocessing and saving is done beforehand, and the training data points to some preexisting subsampled netcdfs, then the train config will use batches_from_netcdf while the test data config will probably still use batches_from_mapper (since the offline diags wants unstacked data).


[FEATURE]

Is this a unique feature?

[X] I have checked "open" AND "closed" issues and this is not a duplicate

Is your feature request related to a problem/unavailable functionality? Please describe.

Proposed Solution
Create the hover CSS functionality over all the buttons to make them look more enhanced and better style appealing!
Screenshots
No response
Do you want to work on this issue?
Yes
If "yes" to above, please explain how you would technically implement this
No response

@Simer13  could you specify which buttons you are mentioning here? I believe all buttons on the landing page have some sort of a hover effect.

@ChinmayMhatre When we hover over it, only the color changes we can have the border hover as well with scale and shadow effect too, giving it more 3d effect

Thank you for raising.  As we are currently looking at a redesign, we can approach this additional hover effect at that time - so I shall close this issue for now.


added experiment chat mode for experimenting in the same session

feature overview:

After the agent is created, these things happen:

The temp directory is created that is stored in .tmp/openagi, this is where the local chromadb db is stored until the program dies.
Since the session id does not change in experiment mode, the subsequent runs can access the temp db.
The ltm also provides context for subsequent sessions, so improvement can be seen real time

The description is fixed for all sessions but the query can be varied.


Fixed hoist attribute in both role builder and role update builder

The isHoisted in RoleBuilder and RoleUpdateBuilder wasn't working because the key was named 'hoisted but it should be 'hoist'. I've fixed it and tested it as well and now it's working properly.

Thanks!


Migrate to MAUI in the future

I adopted WPF because of my fast development speed, but I should migrate in the future

I don't think I need it because I have the Blazor version.

https://devblogs.microsoft.com/visualstudio/visual-studio-for-mac-retirement-announcement/
bye bye🙏
I thought MAUI will retirement too?
uum, I don't feel like trying to do it so Close issue~~ 🐈⬛


Sanitize submodule not here yet?

I just pulled the latetst version, but it look like the sanitize submodule which is now used in all the code isn't here yet?

Yeah, my mistake. I didn't notice that I forgot to git add it.


why add detach() ?

why do you detach()
https://github.com/rishikksh20/Avocodo-pytorch/blob/b298072e82a78b478697b30033ce4bfd20507634/train.py#L130
and not at
https://github.com/rishikksh20/Avocodo-pytorch/blob/b298072e82a78b478697b30033ce4bfd20507634/train.py#L148
@rishikksh20

The Collaborative Multi-Band Discriminator in the line 130 is for updating the discriminator only, we take the detach to do copy to new tensor so that the generator don't get gradient.
In the line 148, we will update gradient for the generator


v7.0.0-rc.1 IconLayer loadimage missing.

Description
It seems going from deck.gl v7.0.0-beta.3 to rc1 introduced a bug for us when loading the IconLayer. I suspect this is due to deck.gl doing something different wrt transpiling as the error seems related to loaders.gl not being available, my error stems from es5 version of this:
https://github.com/uber/deck.gl/blob/master/modules/layers/src/icon-layer/icon-manager.js#L4
Repro Steps
We haven been using ClojureScript with a tool named shadow-cljs deck.gl for a while and have usually been able to require from the deck and luma without issues.
If this issue doesn't immediately give a hint to what the issue might be then I can supply a minimal-repro-repo or steps. I realize these JS dependency/module systems are ridiculously complicated and maybe this is a broader issue with JS dependency loading that is not in deck.gl/loaders.gl scope.
Environment

Framework Version: deck.gl v7.0.0-rc.1
Browser Version: Chrome 72.0
OS: OSX latest

Logs
react-dom.development.js:17963 Uncaught TypeError: (0 , _core2.loadImage) is not a function
    at IconManager.value (icon-manager.js:268)
    at IconManager.value (icon-manager.js:249)
    at IconLayer.value (icon-layer.js:186)
    at IconLayer.value (layer.js:703)
    at IconLayer.value (layer.js:694)

@xintongxia This function moved to @loaders.gl/images in<EMAIL_ADDRESS>This was to:

keep @loaders.gl/core small
enable deck.gl to avoid importing @loaders.gl/core in the modules (ok to import in examples of course)

Great, 7.0.0 solved this for us, thanks!


Can you add support to SVG file?

convert from SVG to ICO

Sure, right now I'm starting to work on it, I'll try to do it quickly. The feature will be available in version 2.1.0 :)

Version 2.1.1 has already been released, which includes the improvement you requested. ✅


Proposal for modular refactoring strategy

Files in the codebase are interdependent on each other in a way that is almost wholly implicit: nearly everything lives in (and hence references) the global scope. It is therefore easy to further pollute the global scope when adding features, and to completely break functionality in seemingly unrelated areas because of unclear scope-access.
This proposal aims to make the codebase more modularised without committing to a build system, whilst also being achievable in small, incremental and minimally disruptive steps that can continuously be deployed to production.
Phase 1: explicitly import dependencies: explictly export values
Take src/help.js as an example. It depends on $ (jquery), $Window (defined in src/$Window.js) and E (defined in src/helpers.js. It exposes the method show_help. It can be refactored on its own as follows, without breaking existing functionality:
// src/help.js
;(function({
    // lib/jquery.min.js
    $,
    // src/$Window.js
    $Window,
    // src/helpers.js
    E
}, exports) {

// ...rest of the existing file, then...

exports.show_help = show_help;

})(window, window);

Here I've used object destructuring to make the dependency injection look a bit nicer. Everything is still pushed into the global scope, but it is clear at the top of the file what its dependencies are, and it is clear at the bottom of the file what its exports are. Furthermore, the IIFE shields any declared properties from being exported unless it is explicitly done. This procedure can be done for effectively all of the existing files; the work is isolated to one file, and can be done concurrently for multiple files across different PRs.
Note that even at this early stage, said modularisation makes unit-testing more plausible, because we know exactly what properties/objects need to be mocked for the window object in each file.
Phase 2: group dependencies under a namespace
Once every file has made its dependencies explicit, it is then possible to group a file's exports under a namespace. For now, it would be fine to put things under something like jspaint.file_name. So looking back at src/help.js, we can export the module to jspaint.help:
// src/help.js
window.jspaint = window.jspaint || {};
;(function({
    // lib/jquery.min.js
    $,
    // src/$Window.js
    $Window,
    // src/helpers.js
    E
}, exports) {

// ...rest of the existing file, then...

exports.show_help = show_help;

})(window, window.jspaint.help = {});

Then in the modules where show_help is used (e.g. src/menu.js), you import jspaint.help and destructure it to get show_menu:
// src/menu.js
;(function({
    // dependencies that haven't been namespaced yet
}, {
    // jspaint namespace passed here as an argument, and the help module is extracted
    help
}, exports) {

// Extract the required dependency from the help module
const { show_menu } = help;

// ...rest of the file as normal, then...

exports.menus = menus;

})(window, window.jspaint, window);

This would need to be done at the level of a whole module at a time, so it would be best to do this for the smaller modules first (i.e. the ones which have the fewest exports and are used in the fewest files). This would iteratively be done until eventually no module exports directly onto window, and the only explicit imports directly from window are for libraries (e.g. jquery).
Phase 3: using an established module format
Once the modules have all been isolated appropriately by the above steps, translating them into a particular module format (i.e. ES6, CommonJS, AMD) is simple because it is just a case of translating the syntax (i.e. turning the dependency injection into the respective require/import structure, and the exports into the corresponding exporting structure). This would allow the project to be loaded through one root file, and let the module loader handle the ordering of the dependencies, rather than manually ordering the <script> tags. This doesn't mean that the project will need a build step: using SystemJS we can use any of the popular module formats and load them in the browser asynchronously. Because the order of the loading no longer matters, this wouldn't cause much of an issue (except perhaps with the initial loading of the theme, but again there are workarounds for what to show the user on the initial draw of the page).

This is just a proposal draft, and I'm making it because this project is awesome and I definitely want to make more contributions to it. But I also want it to be easier in the future for others to make contributions, and it is worth opening a dialog about how to break the codebase down into smaller, isolated, more manageable chunks without doing for a massive revamp or re-write.

Some notes on possible pain points with this strategy:
Statefull variables
At first, some modules won't be easily refactored because they depend on the current state of the app (e.g. selected_tool). Variables like this may need to be left as global until something else can be done about them (e.g. encapsulating the state of the app into its own module, and passing that to these methods?).

Hey, I really appreciate your taking the time to write this.
Back in 2022, I wrapped a lot of JS files in IIFEs, in a0e119ba and subsequent commits, following your Phase 1 advice, although only tracking exports and not imports.
I've now begun converting scripts to ES Modules, using the following pattern:
// @ts-check
/* global $canvas, $Component, $left, $right, $status_text, get_direction, localize, main_canvas, return_to_tools, select_tool, select_tools, selected_tool, selected_tools */
import { $G, E, make_css_cursor } from "./helpers.js";
import { get_theme } from "./theme.js";

...

export { $ToolBox };
// Temporary globals until all dependent code is converted to ES Modules
window.$ToolBox = $ToolBox;

I had to first mark all (or most) of my scripts with defer so that the order is preserved as I start to convert scripts to type="module" since module scripts are implicitly deferred.
Then, by keeping global exports around alongside ESM exports, I can progressively transition scripts to ESM.
I have ESLint configured to ignore unknown globals generally, but I'm re-enabling reporting of global access for the set of files converted to ESM so that I'm aware of any globals I'm using. (In particular, attempts to write to global variables without a window. prefix would fail, so it's important to catch.)
	"overrides": [
		{
			files: [
				"src/theme.js",
				"src/helpers.js",
				"src/$ToolBox.js",
				"src/$FontBox.js",
				"src/$ColorBox.js",
				"src/OnCanvasObject.js",
				"src/OnCanvasHelperLayer.js",
				"src/OnCanvasSelection.js",
				"src/OnCanvasTextBox.js",
				"src/Handles.js",
				"cypress/**/*.js",
			],
			parserOptions: { sourceType: "module" },
			rules: {
				"no-undef": "warn",
				"no-unused-vars": "error",
			},
		}
	]

I'm also adding JSDoc for typechecking as I go. This lets me keep the code as JS while adding type safety. I went with this approach because, while I'm familiar with TypeScript, I don't want to convert the code to TypeScript because 1. I've mostly managed to avoid adding any compile steps to the project, and 2. "JS" is in the name of the project! It's not "TS Paint", after all!
It's gonna be a bit messy until everything is ESM and then it should be able to be cleaned up pretty swiftly, I think.
At any rate, I've just pushed to master with a smattering of scripts converted to ESM.

Correction: Apparently ES Modules can write to globals defined externally. demo fiddle
<body>
	<script>
		window.foo = "";
		let i = 0;
		setTimeout(()=> { console.log({foo, i}); }, 500);
	</script>
	<script type="module">
		foo = "bar";
		for (i = 0; i < 10; i++) {
			console.log(i);
		}
	</script>
</body>

This actually makes things easier, as I don't need to figure out an alternative way to set/store global app state until I convert the file that owns the state to ESM.


resolved_ts: remove hash set to save memory

What is changed and how it works?

Issue Number: Close #15553
What's Changed:

The Resolver uses a hash set to keep track of locks associated with
the same timestamp. When the length of the hash set reaches zero,
it indicates that the transaction has been fully committed. To save
memory, we can replace the hash set with an integer.

Related changes
Check List
Tests

Unit test
Integration test

sysbench --db-driver=mysql --mysql-host=<HOST> --mysql-port=<PORT> --mysql-user=root \
        --mysql-db=test --tables=2 --table-size=100000000 --create_secondary=false \
        ./oltp_update_index prepare

-- warm up cache and others.
update sbtest1 set c='a';

-- sbtest2 leak.
update sbtest2 set c='b';

Left master branch, righ this PR.

Release note
None

/merge

/hold

/merge

/test

/merge

/merge

/merge


Password image strength

Title and Issue number
Title: UI design extension to Password Strength checkers.
Description: Added a project in basic Javascript-UI interaction that would be a good start for new learners in Web Development to try their hands on. The Project also consists of a challenge for new students that could help them understand better JS-UI coexistence.
Close #5056
Checklist:

[X] I have mentioned the issue number in my Pull Request.
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have created a helpful and easy to understand README.md

@MohitGupta121 kindly check the PR whenever you find time. Thanks.

@MohitGupta121 @TusharKesarwani really sorry for the repetition, please have a look at the PR whenever possible.

@MohitGupta121 @TusharKesarwani any updates?


[Bugfix] truncate filenames in invokeai batch that exceed max filename length

This prevents invokeai-batch from trying to create image files whose names would exceed PC_NAME_MAX.
Closes #3115

Thanks for the approvals!


v2.1.5 no longer works with node 0.10

postcss-svgo requires svgo v0.7. This uses ES2015 syntax, which makes it incompatible with node 0.10.
It surprises me that a patch update to postcss-svgo and minor update to svgo would drop support for node 0.10.
Anyway, I solved my issue by pinning the version of postcss-svgo, but just wanted to bring this to your attention.
Thanks!

Oh, sorry. Didn't realise there was anyone else still using this version (end-of-life next month!), have stopped running tests on 0.10 for a few months now so any support has been a happy coincidence. But officially I've stopped supporting this platform, for numerous reasons (0.12 runs the test suites much faster, among others).
Thanks for the issue! You should definitely look into upgrading to at least version 4, as 0.12 will be on its way out at the beginning of next year. 😄

Yep, I don't use it locally, but one of our customer's servers is a CentOS box and apparently that's the version they got through yum.

It'd be nice if you could specify which versions of Node you support in your package.json as described here: https://docs.npmjs.com/files/package.json#engines . I'm not sure exactly what the effect of that would be, but it may help others catch this error sooner.

Sure, would be happy to accept pull requests for any of my packages that are missing this (which is probably a majority of them). 😄

After further research, the engines are only enforced if engine-strict is set to true in the npm configs, and after a half hour of fiddling, I haven't been able to successfully make npm reject a build due to this. It's probably not worth the trouble to change this now since so few people will be using v0.10.0 and have engine-strict already set to true.

Okay, no problem. I'll make a mental note to add this in once I drop support for 0.12 (beginning next year). 👍


add support of bulk operations for ORM in ORACLE and SQLite backends

Hello everyone,
I am excited to submit this pull request, which (re)introduces support for bulk operations in the SOCI library's ORM (Object-Relational Mapping) functionality. This work is based on the initial groundwork laid by @abc100m, and I have built upon their contributions to enhance the capabilities of the library.
I would appreciate it if the community could review and provide feedback on the changes. Your insights and suggestions are valuable in ensuring the quality and effectiveness of this new feature.
Thank you for your attention and consideration. I look forward to collaborating with you all to further enhance the SOCI library.
Best regards,
Aleksey

1. I have reverted all the whitespace-only changes.

Thanks!

Regarding the replacement of at(pos) with [pos], these changes were already present in the original PR, so I simply included them as-is.

Sorry for not realizing this.

However, I have now reverted these changes for the time being. It's worth mentioning that the usage of at(pos) and [pos] in this file is inconsistent, which could be addressed in a separate PR.

I agree that inconsistencies should be fixed but it's definitely better to do them in a separate PR. Thanks for fixing this up too!

What is the status of this PR, I think the bulk adding is a very important feature for soci.
I have a similar discussion issue discussion here: ORM: What is the correct way to bulk add a lot of rows? · Issue #1066 · SOCI/soci, in the issue, I have to use a for loop to add many rows, but I do need a simple way to do that.
Thanks.

I apologize for abandoning this PR. I would like to complete the work on it; however, I currently have more urgent tasks to attend to.
Furthermore, I do not have a readily available development environment on Windows and FreeBSD. Any assistance would be greatly appreciated.
Thank you.

It doesn't look like anything is going to happen here, so I have no choice but to close it, sorry.