[IHP sg13g2] IO pin access problem on the chip right side only

Describe the bug
Global routing failed reporting congestion at the right side IO cells only. However there is no real congestion.
With some experimentation I could say that removing OBS from IO cells in LEF results in successful routing, but those OBS are actually on opposite side of IO cell and are not obstacles for routing. And actually other sides of the chip have same IO and routed without issue. Looks like some issue related to the cells orientation in the OpenROAD with determining pin access.
Expected Behavior
All sides of the chip routed.
Environment
kernel: Linux 6.10.11+bpo-amd64
os: Debian GNU/Linux 12 (bookworm)
cmake version 3.30.5
-- The CXX compiler identification is GNU 12.2.0
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- OpenROAD version: v2.0-16972-g32d202458
-- System name: Linux
-- Compiler: GNU 12.2.0
-- Build type: RELEASE
-- Install prefix: /usr/local
-- C++ Standard: 17
-- C++ Standard Required: ON
-- C++ Extensions: OFF
-- The C compiler identification is GNU 12.2.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /usr/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Performing Test C_COMPILER_SUPPORTS__-Wall
-- Performing Test C_COMPILER_SUPPORTS__-Wall - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wall
-- Performing Test CXX_COMPILER_SUPPORTS__-Wall - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-array-bounds
-- Performing Test C_COMPILER_SUPPORTS__-Wno-array-bounds - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-array-bounds
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-array-bounds - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-nonnull
-- Performing Test C_COMPILER_SUPPORTS__-Wno-nonnull - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-nonnull
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-nonnull - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-maybe-uninitialized
-- Performing Test C_COMPILER_SUPPORTS__-Wno-maybe-uninitialized - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-maybe-uninitialized
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-maybe-uninitialized - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-format-overflow
-- Performing Test C_COMPILER_SUPPORTS__-Wno-format-overflow - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-format-overflow
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-format-overflow - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-unused-variable
-- Performing Test C_COMPILER_SUPPORTS__-Wno-unused-variable - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-unused-variable
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-unused-variable - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-unused-function
-- Performing Test C_COMPILER_SUPPORTS__-Wno-unused-function - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-unused-function
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-unused-function - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-write-strings
-- Performing Test C_COMPILER_SUPPORTS__-Wno-write-strings - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-write-strings
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-write-strings - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-sign-compare
-- Performing Test C_COMPILER_SUPPORTS__-Wno-sign-compare - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-sign-compare
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-sign-compare - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-deprecated
-- Performing Test C_COMPILER_SUPPORTS__-Wno-deprecated - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-deprecated
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-deprecated - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-c++11-narrowing
-- Performing Test C_COMPILER_SUPPORTS__-Wno-c++11-narrowing - Failed
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-c++11-narrowing
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-c++11-narrowing - Failed
-- Performing Test C_COMPILER_SUPPORTS__-Wno-register
-- Performing Test C_COMPILER_SUPPORTS__-Wno-register - Failed
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-register
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-register - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-format
-- Performing Test C_COMPILER_SUPPORTS__-Wno-format - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-format
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-format - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-reserved-user-defined-literal
-- Performing Test C_COMPILER_SUPPORTS__-Wno-reserved-user-defined-literal - Failed
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-reserved-user-defined-literal
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-reserved-user-defined-literal - Failed
-- Performing Test C_COMPILER_SUPPORTS__-fpermissive
-- Performing Test C_COMPILER_SUPPORTS__-fpermissive - Failed
-- Performing Test CXX_COMPILER_SUPPORTS__-fpermissive
-- Performing Test CXX_COMPILER_SUPPORTS__-fpermissive - Success
-- Performing Test C_COMPILER_SUPPORTS__-x
-- Performing Test C_COMPILER_SUPPORTS__-x - Failed
-- Performing Test CXX_COMPILER_SUPPORTS__-x
-- Performing Test CXX_COMPILER_SUPPORTS__-x - Failed
-- Performing Test C_COMPILER_SUPPORTS__c++
-- Performing Test C_COMPILER_SUPPORTS__c++ - Failed
-- Performing Test CXX_COMPILER_SUPPORTS__c++
-- Performing Test CXX_COMPILER_SUPPORTS__c++ - Failed
-- Performing Test C_COMPILER_SUPPORTS__-std=c++17
-- Performing Test C_COMPILER_SUPPORTS__-std=c++17 - Failed
-- Performing Test CXX_COMPILER_SUPPORTS__-std=c++17
-- Performing Test CXX_COMPILER_SUPPORTS__-std=c++17 - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-unused-but-set-variable
-- Performing Test C_COMPILER_SUPPORTS__-Wno-unused-but-set-variable - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-unused-but-set-variable
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-unused-but-set-variable - Success
-- TCL library: /usr/lib/x86_64-linux-gnu/libtcl.so
-- TCL header: /usr/include/tcl/tcl.h
-- TCL readline library: /usr/lib/x86_64-linux-gnu/libtclreadline.so
-- TCL readline header: /usr/include/x86_64-linux-gnu
-- Found SWIG: /usr/bin/swig4.0 (found suitable version "4.1.0", minimum required is "4.0")
-- Using SWIG >= 4.1.0 -flatstaticmethod flag for python
-- Found Boost: /usr/lib/x86_64-linux-gnu/cmake/Boost-1.81.0/BoostConfig.cmake (found version "1.81.0")
-- boost: 1.81.0
-- Found GTest: /usr/lib/x86_64-linux-gnu/cmake/GTest/GTestConfig.cmake (found version "1.12.1")
-- GTest: 1.12.1
-- Found Python3: /usr/include/python3.11 (found version "3.11.2") found components: Development Development.Module Development.Embed
-- Found ZLIB: /usr/lib/x86_64-linux-gnu/libz.so (found version "1.2.13")
-- Found Threads: TRUE
-- spdlog: 1.10.0
-- Found BISON: /usr/bin/bison (found version "3.8.2")
-- Found Doxygen: /usr/bin/doxygen (found version "1.9.4") found components: doxygen missing components: dot
-- STA version: 2.6.0
-- STA git sha: aa598a2f14c5c142e90391a69988523505e7db3d
-- System name: Linux
-- Compiler: GNU 12.2.0
-- Build type: RELEASE
-- Build CXX_FLAGS: -O3 -DNDEBUG
-- Install prefix: /usr/local
-- Found FLEX: /usr/bin/flex (found version "2.6.4")
-- TCL library: /usr/lib/x86_64-linux-gnu/libtcl.so
-- TCL header: /usr/include/tcl/tcl.h
-- TCL readline library: /usr/lib/x86_64-linux-gnu/libtclreadline.so
-- TCL readline header: /usr/include/x86_64-linux-gnu/tclreadline.h
-- CUDD library: not found
-- SSTA: 0
-- Found SWIG: /usr/bin/swig4.0 (found suitable version "4.1.0", minimum required is "3.0")
-- STA executable: /home/serge/projects/chipflow-backend-tools/tools/src/openroad/src/sta/app/sta
-- Found re2: /home/serge/projects/chipflow-backend-tools/tools/or-tools/lib/cmake/re2/re2Config.cmake (found version "11.0.0")
-- Found Clp: /home/serge/projects/chipflow-backend-tools/tools/or-tools/lib/cmake/Clp/ClpConfig.cmake (found version "1.17.7")
-- Found Cbc: /home/serge/projects/chipflow-backend-tools/tools/or-tools/lib/cmake/Cbc/CbcConfig.cmake (found version "2.10.7")
-- Found SCIP: /home/serge/projects/chipflow-backend-tools/tools/or-tools/lib/cmake/scip/scip-config.cmake (found version "9.0.0")
-- Found OpenMP_CXX: -fopenmp (found version "4.5")
-- Found OpenMP: TRUE (found version "4.5")
-- Found OR-Tools: /home/serge/projects/chipflow-backend-tools/tools/or-tools/lib/cmake/ortools (version: 9.11.4210)
-- TCL library: /usr/lib/x86_64-linux-gnu/libtcl.so
-- TCL header: /usr/include/tcl/tcl.h
-- Found OpenMP_C: -fopenmp (found version "4.5")
-- Found OpenMP: TRUE (found version "4.5")
-- Found OpenMP: TRUE (found version "4.5")
-- GUI is enabled
-- Number of processor cores: 4
-- Found Boost: /usr/lib/x86_64-linux-gnu/cmake/Boost-1.81.0/BoostConfig.cmake (found version "1.81.0") found components: serialization
-- Could NOT find VTune (missing: VTune_LIBRARIES VTune_INCLUDE_DIRS)
-- Found Boost: /usr/lib/x86_64-linux-gnu/cmake/Boost-1.81.0/BoostConfig.cmake (found suitable version "1.81.0", minimum required is "1.78")
-- TCL library: /usr/lib/x86_64-linux-gnu/libtcl.so
-- TCL header: /usr/include/tcl/tcl.h
-- Found Boost: /usr/lib/x86_64-linux-gnu/cmake/Boost-1.81.0/BoostConfig.cmake (found version "1.81.0") found components: serialization system thread
-- Found Boost: /usr/lib/x86_64-linux-gnu/cmake/Boost-1.81.0/BoostConfig.cmake (found version "1.81.0")
-- Found Eigen3: /home/serge/projects/chipflow-backend-tools/tools/or-tools/share/eigen3/cmake/Eigen3Config.cmake (found version "3.4.0")
-- TCL readline enabled
-- Tcl Extended disabled
-- Python3 enabled
-- Configuring done (5.8s)
-- Generating done (0.9s)

To Reproduce
5_1_grt.zip
Unzip attached archive (it contains all the dependencies), from inside 5_1_grt folder run make.
The global routing will fail, produced files for analysis will be 5_1_grt/results/5_1_grt-failed.odb and 5_1_grt/reports/congestion.rpt
Relevant log output
No response
Screenshots

Additional Context
No response

May be this will be of help: it some kind of related to Metal2 layer, as removing all the OBS only on this layer from IO LEF results in success. The questions are still the same: why chip right side only and why this cause congestion as those Metal2 OBS rectangles are on opposite side of pin routing.


Show tail of the execution log if run fails

If there is an error encountered during execution the last 30 lines (or so) of stderr/stdout should be displayed on the output to make it easier to debug the problem. This should be enabled by default for the debug level. In verbose and info level it would be nice to always give a reasonable message about the failed command. Or, perhaps, the tail output should be enabled always? @thrix, @pvalena, what do you think?

This has been fixed by #218. Example output:
> tmt run -a plan --name smoke prepare -h shell -s 'sudo dnf install -y tmttt'
/var/tmp/tmt/run-413

/plans/smoke
    discover
        how: shell
        directory: /home/psss/git/tmt
        summary: 1 test selected
    provision
        how: local
        distro: Fedora release 31 (Thirty One)
        kernel: 5.5.11-200.fc31.x86_64
        summary: 1 guest provisioned
    prepare
        how: shell
        overview: 1 script found
    finish

stdout (2/2 lines)
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Last metadata expiration check: 0:13:15 ago on St 29. dubna 2020, 14:37:52 CEST.
No match for argument: tmttt

stderr (1/1 lines)
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Error: Unable to find a match: tmttt

 ERROR  Failed to run command 'sudo dnf install -y tmttt'.


Columnwidth of 0 zero when setting numFmt

First thanks for the great work.
I'm new to this project so perhaps I miss some experience but..
When setting numFmt of a column: sheet.getColumn(3).numFmt = '0.00', the formatting works as expected but the column gets invisible. (using MS Office Excel 2010).
Only if also the width is explicitely set: sheet.getColumn(3).width = 10 the column gets visible again.
Is this by design? Wouldn't it be good to warn for this issue in the docs?

@Sjonnie2nd i'm having the same issue. The problem also occurs when applying styles on columns.

@Siemienik I have researched the code and found that this width attribute is missing from the col tag in sheet1.xml. I think we should give a default value when then user does not set the width value. If you agree with this proposal, I would be happy to submit a PR.
sheet1.xml
<cols>
    <col min="3" max="3" style="1" customWidth="1" />
</cols>

Source Code
  static toModel(columns) {
    // Convert array of Column into compressed list cols
    const cols = [];
    let col = null;
    if (columns) {
      columns.forEach((column, index) => {
        if (column.isDefault) {
          if (col) {
            col = null;
          }
        } else if (!col || !column.equivalentTo(col)) {
          col = {
            min: index + 1,
            max: index + 1,
            width: column.width || 8,   // it's here
            style: column.style,
            isCustomWidth: column.isCustomWidth,
            hidden: column.hidden,
            outlineLevel: column.outlineLevel,
            collapsed: column.collapsed,
          };
          cols.push(col);
        } else {
          col.max = index + 1;
        }
      });
    }
    return cols.length ? cols : undefined;
  }

Is the default value set for worksheet/workbook?


fix(back-to-top): remove border-color rule

Related Ticket(s)
Closes #10721
Description
This PR fixes the border width for the back to top component
Changelog
Changed

remove unneeded border-color rule

@emyarod once you resolve the merge conflicts again just manually merge... the label won't work


Fixed corner case during search

Motivation and context
Resolved #2973
How has this been tested?

Checklist

[x] I submit my changes into the develop branch
[x] I have added a description of my changes into CHANGELOG file
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[x] I have linked related issues (read github docs)
[x] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[x] I have updated the license header for each file (see an example below)

# Copyright (C) 2022 Intel Corporation
#
# SPDX-License-Identifier: MIT

@klakhov
Could you please review? Generally I removed complex code (I implemented it before for optimization), but it ends up with the fact that this optimizations brings the issue, so, I just fall the code back.


something wrong here

following is parts of exercises/images/solution/en.md

Now you learned how to create a link in Markdown. This is a common style for links, but some parsers provide additional tools for alignment, adding classes and other styles.

The challenage is about image, so the finished message maybe about image too.

Yes, seems like this. Could you send a PR?

I am sorry, my english is poor. 😭

@wangding hmm, strange. How did you translate the workshopper then?

I offten reading IT articals in Internet, so I can reading something. But have little to write or speak in English. Please trust me.🙏

That's okay. Nevermind 😉

On Jan 27, 2017, at 02:15, 王顶<EMAIL_ADDRESS>wrote:
I offten reading IT articals in Internet, so I can reading something. But have little to write or speak in English. Please trust me.🙏
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


[@open-wc/rollup-plugin-html] breaks if there is no module in the html file

Given the following html and rollup config I do get the error

hey

const html = require('@open-wc/rollup-plugin-html');

module.exports = {
  output: {
    input: 'input.html',
    dir: './demo/dist',
  },
  plugins: [
    html(),
  ],
};

rollup v2.8.0
bundles index.html → demo/dist...
[!] Error: You must supply options.input to rollup
Error: You must supply options.input to rollup
   at Graph.parseModules (open-wc/node_modules/rollup/dist/shared/rollup.js:18426:19)
   at process._tickCallback (internal/process/next_tick.js:68:7)

What would the expected behavior here be? If there is no module in the index.html, there's no JS to pass to rollup

it should still optimize the html and create the file in the dist folder...


fix(interaction): 修复禁用多选后, 未对行/列头生效

👀 PR includes

🐛 Bugfix

[x] Solve the issue and close #0

📝 Description

修复关闭多选配置后, 行列点击时未读取该配置, 导致依然可以进行多选的问题 (尤其对于趋势分析表, 多选没有任何意义)

const s2Options = {
  interaction: {
    multiSelection: false
  }
}

对 facet.updateScrollOffset 的内部两个方法进行兜底, scrollWithAnimation 和 scrollImmediately

避免只传递 offsetX: {} 和 offsetY: {} 报错的情况

🖼 Screenshot

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.21.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-react-v1.19.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 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Lua 5.1 marked as supported in the rockspec.

Considering it passes the busted tests with Lua 5.1, I’d say there’s not reason to prevent installation for 5.1 through luarocks.

@jonstoler, any updates on this?

Hi @Lukc, sorry for not getting back quicker. This looks good, I'll merge the request.

No problem, and thanks for merging. :)
You should probably reupload the rockspec as well (luarocks upload). ^^

Ah, thanks for the reminder! I don't actually use luarocks so I don't really know how it works.
I'm migrating across two computers right now so I don't have my entire dev environment set up yet. I'll try to get that going soon and upload to LuaRocks within the next day or two.

@Lukc Okay, lua-toml has been marked as Lua 5.1-compatible on LuaRocks. (This required a patch version bump to 1.0.1, but I should be doing that anyway.)


Add more entries of advice for each adjective on Parse

[ ] Add more advice for each adjective
[ ] Add more adjectives on top of the current 6 emotions


[SPARK-48787][BUILD] Upgrade Kafka to 3.7.1

What changes were proposed in this pull request?
The pr aims to upgrade kafka from 3.7.0 to 3.7.1.
Why are the changes needed?
https://downloads.apache.org/kafka/3.7.1/RELEASE_NOTES.html
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass GA.
Was this patch authored or co-authored using generative AI tooling?
No.

cc @dongjoon-hyun

cc @HeartSaVioR

Upgrading bugfix version sounds fine and less risky unless any system test captures some issue. Thanks for the fix.


Move from sprintf to sprintf-js

As per #24, sprintf is deprecated, these changes swap it out for sprintf-js, and should resolve the deprecation warning.
I tested these changes on a number of tests in one of my own projects, and behaviour looks to be as normal.

Why isn't this PR already merged? Is there still support on this library?

Duplicate of #32. Fixed in 1671ad5.


more vignette, updated printall.mustache

Vignette is mostly complete, probably needs some polish/review though.  Still GA todo also.  The mustache update is necessary to make the new text section in the example viz work.

Coverage remained the same at 66.294% when pulling 971c8748e3bdade163853940b649a12bc3c82950 on wdwatkins:master into 765b5b59ebd5f42b3f1b9b1bad90f3bc47069318 on USGS-VIZLAB:master.


Principal Component Analysis - Documentation

Task: Update the Readme.md to explain the process behind Principal Component Analysis  as well as you can
Level-1 : 10 Points

@Simar05519 Would you like to work on this?

OK, I can work on it.
On Mon, Mar 15, 2021 at 9:59 AM Ashwani Rathee @.***>
wrote:

@Simar05519 https://github.com/Simar05519 Would you like to work on
this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Algo-Phantoms/Algo-ScriptML/issues/91#issuecomment-799081328,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQBFEQ7K5V7W2FX56YCWG4TTDWEMNANCNFSM4ZFZA4OA
.

Sure @Simar05519  ,You have been assigned the task to create detailed documentation of Principal Component Analysis .
Please see the documentation guidelines first.
Try to submit a PR in 2 days.

Sure, will do that.
On Mon, Mar 15, 2021 at 10:05 AM Ashwani Rathee @.***>
wrote:

Sure @Simar05519 https://github.com/Simar05519 ,You have been assigned
the task to create detailed documentation of Principal Component Analysis .
Please see the documentation guidelines first.
Try to submit a PR in 2 days.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Algo-Phantoms/Algo-ScriptML/issues/91#issuecomment-799083165,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQBFEQZBRVJL5QT37H2GSBLTDWFBRANCNFSM4ZFZA4OA
.

Hello.
I'm a GSSOC'21 participant and I want to work on this issue.

I'm a Gssoc'21 participant.
Can i work on this issue?

@guptaa98 Would you like to take this up?

@ashwani-rathee yes I would like to work on this issue.

@guptaa98 Sure,go ahead

Merged


Download MNIST data automatically in MNIST demo

MNIST data should be automatically downloaded if not already present.

fixed with commit #4


kube deployment kustomize seem not working properly

Describe the bug
kube deployment kustomize seem not working properly. with the current kustomize
To Reproduce
Steps to reproduce the behavior:

Install using the kustomize
Follow the step by step initiation
In the step of airbyte - github sources the save changes and test will always error, ref: https://community.faros.ai/docs/quickstart-step-2

I'm using GKE 1.27
Expected behavior
the step should have no error
Screenshots and logs
When I retry multiple times below is the result :

After retry multiple time seem success but I got stuck at triggering the github sync now

Any Idea on whats going on?
Deployment

If cloud, GKE 1.27 and latest version

I am confused what S3 has to do with anything here.
@saviogl any idea?


write tf.json and tf.state with pretty json to make them more user-friendly

What type of PR is this?
/kind feature

What this PR does / why we need it:
write tf.json and tf.state with pretty json to make them more user-friendly
Which issue(s) this PR fixes:

Fixes #282
Special notes for your reviewer:
None
Does this PR introduce a user-facing change?

wirite `main.tf.json` and `terraform.tfstate` with pretty json to make them user friendly

Additional documentation e.g., design docs, usage docs, etc.:

none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

6 of 7 (85.71%)  changed or added relevant lines in 2 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.02%) to 72.242%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/util/json/json.go
1
2
50.0%

Files with Coverage Reduction
New Missed Lines
%

pkg/engine/operation/parser/parser.go
3
75.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
4682

Relevant Lines:
6481

💛  - Coveralls


How to load tensorflow's new v2 format checkpoint?

Hi, the author's test_net code maybe just applicable for v1 format checkpoint, but after tensorflow r1.2, the checkpoint will be saved as v2 format by default. How to save into v2 format and restore v2 format? not v1, thanks!

aver.restore(sess, args.model)
Use args.model = '.ckpt', instead of '.ckpt.meta'


[Databox] Adding more fields for stage status

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[x] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[x] SDK of .NET (need service team to ensure code readiness)
[x] SDK of Python
[x] SDK of Java
[x] SDK of Js
[x] SDK of Go
[x] PowerShell
[x] CLI
[x] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
[ ] Ensure to copy the existing version into new directory structure for first commit (including refactoring) and then push new changes including version updates in separate commits. This is required to review the changes efficiently.
Adding a new service

[x] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

@jianyexi Hi Jianye, do you know about the "modelAsExtensible" under "x-ms-enum"?

modelAsExtensible

I don't know too, may be changing to modelAsString is better

Hi All, can you please confirm that changing modelAsExtensible to ModelAsString is fine for the sdk clients?

Hi All, can you please confirm that changing modelAsExtensible to ModelAsString is fine for the sdk clients?

if this change need to be taken, This change will happen along with new api version as well

@ArcturusZhang for Go, @msyyc for Python. Do you support "modelAsExtensible" in "x-ms-enum"?

Hi All, can you please confirm that changing modelAsExtensible to ModelAsString is fine for the sdk clients?

@ArcturusZhang for Go, @msyyc for Python. Do you support "modelAsExtensible" in "x-ms-enum"?

Hi All, can you please confirm that changing modelAsExtensible to ModelAsString is fine for the sdk clients?

No for track 1. @jhendrixMSFT do we support this in track 2?

AFAIK this is not applicable to Go as there are no enumerated types, so by definition our enums (constants) are extensible by nature.

@jianyexi
I've checked with m4, "modelAsExtensible" is not recognized. What will be required for service to change it to "modelAsString". Does it count as breaking change? Need a new api-version?

@jianyexi Jianye Xi FTE
I've checked with m4, "modelAsExtensible" is not recognized. Do you know what will be required for service to change it to "modelAsString"? Does it count as breaking change? Need a new api-version?

I think it's a breaking change for SDK , right ? @ArcturusZhang @msyyc
if everything works well now, it's not required to change to use 'modelAsString' , as this action will introduce breaking change

@jianyexi
Current problem is that adding a single value to enum would require a new api-version (as reported by this PR). So this is the trade-off.
If service rarely add new value to enum, maybe this is fine.

BTW, Go seems fine with it. It does not have a enum, so this will always be extensible.
Java is also fine as current is preview.

Go SDK will always model the enums as strings - therefore this is not a brekaing change for go SDK

Created a new #14909


Same file written simultaneously by different rules

Many thanks for this awesome tool. It makes a lot of things so much easier. However, I think the following is a bug and not a feature.
Snakemake version
7.20.0
Describe the bug
If ruleorder is used the same file (b.txt) can be written by different rules simultaneously.
Minimal example
rule all:
    input:
        "a.txt",
        "b.txt",
        "c.txt"

ruleorder: a_b > b_c
rule a_b:
    output:
        "a.txt",
        "b.txt"

rule b_c:
    output:
        "b.txt",
        "c.txt"

Additional context
Without the ruleorder snakemake raises an AmbiguousRuleException.

Without the ruleorder snakemake raises an AmbiguousRuleException.

This is correct and intentional. Otherwise, Snakemake won't know which rule to choose.
I don't think I understand the behaviour you're looking for... perhaps you have a more complete example?

Just to be clear, you're saying that given the Snakefile you have there, both rule a_b and rule b_c will run, causing b.txt to be written twice?
Are you hoping to get this minimal example to work in some sensible way? Or were you expecting this to error out?

Are you hoping to get this minimal example to work in some sensible way? Or were you expecting this to error out?

My 2p: I think the OP's example should throw an error. Rules a_b and b_c can run in parallel which means you don't know which rule produced b.txt. In fact, you cannot even know whether b.txt is sane since it could be a mixture of the two rules writing to the same file

Sorry for not being clear. Just as @dariober pointed out, the status of b.txt will be unclear. Therefore, I would have expected Snakemake to error out with an AmbiguousRuleException, just as in the case when no rule order is given.

Gotcha, I agree that should be the behaviour

I don't know if this was intended at all but if rule order was specified, doesn't that mean user is aware of and has addressed this conflict?


Fix json warnings

More info here
https://gcc.gnu.org/onlinedocs/gcc/Designated-Inits.html
./json/js0n.c:26:29: warning: initializer overrides prior initialization of this
      subobject [-Winitializer-overrides]
                ['t'] = &&l_bare, ['f'] = &&l_bare, ['n'] = &&l_bare // true...
                                          ^~~~~~~~
./json/js0n.c:15:17: note: previous initialization is here
                [0 ... 255] = &&l_bad,
                              ^~~~~~~
./json/js0n.c:26:47: warning: initializer overrides prior initialization of this
      subobject [-Winitializer-overrides]
  ...['t'] = &&l_bare, ['f'] = &&l_bare, ['n'] = &&l_bare // true, false, null
                                                 ^~~~~~~~
./json/js0n.c:15:17: note: previous initialization is here
                [0 ... 255] = &&l_bad,
                              ^~~~~~~
./json/js0n.c:32:12: warning: initializer overrides prior initialization of this
      subobject [-Winitializer-overrides]
                ['\t'] = &&l_unbare, [' '] = &&l_unbare, ['\r'] = &&l_unbare...
                         ^~~~~~~~~~
./json/js0n.c:30:16: note: previous initialization is here
                [0 ... 31] = &&l_bad,
                             ^~~~~~~
./json/js0n.c:32:32: warning: initializer overrides prior initialization of this
      subobject [-Winitializer-overrides]
                ['\t'] = &&l_unbare, [' '] = &&l_unbare, ['\r'] = &&l_unbare...
                                             ^~~~~~~~~~
./json/js0n.c:31:18: note: previous initialization is here
                [32 ... 126] = &&l_loop, // could be more pedantic/validatio...
                               ^~~~~~~~
./json/js0n.c:32:53: warning: initializer overrides prior initialization of this
      subobject [-Winitializer-overrides]
  ...['\t'] = &&l_unbare, [' '] = &&l_unbare, ['\r'] = &&l_unbare, ['\n'] = &&l_u...
                                                       ^~~~~~~~~~
./json/js0n.c:30:16: note: previous initialization is here
                [0 ... 31] = &&l_bad,
                             ^~~~~~~
./json/js0n.c:32:74: warning: initializer overrides prior initialization of this
      subobject [-Winitializer-overrides]
  ...= &&l_unbare, [' '] = &&l_unbare, ['\r'] = &&l_unbare, ['\n'] = &&l_unbare,
                                                                     ^~~~~~~~~~
./json/js0n.c:30:16: note: previous initialization is here
                [0 ... 31] = &&l_bad,
                             ^~~~~~~
./json/js0n.c:33:11: warning: initializer overrides prior initialization of this
      subobject [-Winitializer-overrides]
                [','] = &&l_unbare, [']'] = &&l_unbare, ['}'] = &&l_unbare,
                        ^~~~~~~~~~
./json/js0n.c:31:18: note: previous initialization is here
                [32 ... 126] = &&l_loop, // could be more pedantic/validatio...
                               ^~~~~~~~
./json/js0n.c:33:31: warning: initializer overrides prior initialization of this
      subobject [-Winitializer-overrides]
                [','] = &&l_unbare, [']'] = &&l_unbare, ['}'] = &&l_unbare,
                                            ^~~~~~~~~~
./json/js0n.c:31:18: note: previous initialization is here
                [32 ... 126] = &&l_loop, // could be more pedantic/validatio...
                               ^~~~~~~~
./json/js0n.c:33:51: warning: initializer overrides prior initialization of this
...

There's an ifdef to build the same thing for outdated C compilers and C++ ones. so there's no need for this anymore


Added logos to new document menu

The new document menu now has logos added to them:

We can't consistently use svg as a background image in Firefox, so the image has to be used with an image tag

Looks good, thanks.


[Profile] Add memory statiscs & annotation

It add two functions:

provide PROFILE_MEM_ANNOTATE macro
print average and maximum usage of memory.

Signed-off-by: Jiho Chu<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #1987. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

:octocat: cibot: @jihochu, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1987-202208251445210.83200907707214-0e2e32a599cefd567a35bd0355f64505965b1e9b/.

:octocat: cibot: @jihochu, test/unittest/unittest_nntrainer_profiler.cpp does not include Doxygen tags such as @file @brief @author @bug. You must include the Doxygen tags in the source code. Please refer to a Doxygen manual at http://github.com/nnstreamer/TAOS-CI/blob/main/ci/doc/doxygen-documentation.md


break

Codecov Report
Attention: 44 lines in your changes are missing coverage. Please review.

Comparison is base (a5274c1) 93.54% compared to head (0e418d6) 54.71%.

Files
Patch %
Lines

swarm/breakci.go
0.00%
22 Missing :warning:

swarm/bvraek3.go
0.00%
22 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##             main       #6       +/-   ##
===========================================
- Coverage   93.54%   54.71%   -38.84%
===========================================
  Files           6        8        +2
  Lines          62      106       +44
===========================================
  Hits           58       58
- Misses          3       47       +44
  Partials        1        1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[confidential ledger] generate with cred scopes to remove written auth policy code

If we generate with the cred scopes, there will be no need to have BearerTokenCredentialPolicy in the wrapped client

Closing, confidential ledge requires a special auth mechanis,, that is more complicated than just credential scope


timestep limit for a single simulation

The timestep limit for a particular simulation should be 500 (https://github.com/stanfordnmbl/osim-rl/blob/master/osim/env/osim.py#L70).
But while analysing the training logs to see the variation in path-lengths, I noticed there were quite a few simulations which run for more than 500 steps.
The reason being, I use the done response as the feedback for ending a simulation, and would expect it to end a simulation after 500 steps.
Is this an expected functionality ? or has it sneaked in by mistake ?
I think, the is_done (https://github.com/stanfordnmbl/osim-rl/blob/master/osim/env/run.py#L75) function should handle the upper bound on the max steps, at least for the grader.

With reference to the grader, I get this error if I try to reset the env after 500 steps (and the pelvis is still above the threshold)
[2017-06-23 02:38:52,828] Exception on /v1/envs/4f2b61e1aaf03d3283f135febbe225a4___1c549709df/reset/ [POST]
Traceback (most recent call last):
  File "/home/ubuntu/anaconda2/envs/opensim-rl/lib/python2.7/site-packages/flask/app.py", line 1988, in wsgi_app
    response = self.full_dispatch_request()
  File "/home/ubuntu/anaconda2/envs/opensim-rl/lib/python2.7/site-packages/flask/app.py", line 1641, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/home/ubuntu/anaconda2/envs/opensim-rl/lib/python2.7/site-packages/flask/app.py", line 1544, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/home/ubuntu/anaconda2/envs/opensim-rl/lib/python2.7/site-packages/flask/app.py", line 1639, in full_dispatch_request
    rv = self.dispatch_request()
  File "/home/ubuntu/anaconda2/envs/opensim-rl/lib/python2.7/site-packages/flask/app.py", line 1625, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "gym_http_server.py", line 367, in env_reset
    observation = envs.reset(instance_id)
  File "gym_http_server.py", line 138, in reset
    obs = env._reset(difficulty=2, seed=SEED_MAP[env.trial-1])
  File "gym_http_server.py", line 60, in _reset
    self._before_reset()
  File "/home/ubuntu/anaconda2/envs/opensim-rl/lib/python2.7/site-packages/gym/wrappers/monitoring.py", line 190, in _before_reset
    self.stats_recorder.before_reset()
  File "/home/ubuntu/anaconda2/envs/opensim-rl/lib/python2.7/site-packages/gym/monitoring/stats_recorder.py", line 68, in before_reset
    raise error.Error("Tried to reset environment which is not done. While the monitor is active for {}, you cannot call reset() unless the episode is over.".format(self.env_id))
Error: Tried to reset environment which is not done. While the monitor is active for 0, you cannot call reset() unless the episode is over.


Module not found: Can't resolve 'fs'

./node_modules/next-i18next/dist/commonjs/serverSideTranslations.js:34:0
Module not found: Can't resolve 'fs'

Please check the other issues and provide a minimal reproducible example.
https://github.com/i18next/next-i18next/discussions/1810
https://github.com/i18next/next-i18next/issues/935
https://github.com/i18next/next-i18next/issues/1401
https://github.com/i18next/next-i18next/issues/980
https://github.com/i18next/next-i18next/issues/1010

I think it occurs when you try to use something like this:
import { serverSideTranslations } from 'next-i18next/serverSideTranslations';

export async function getServerSideProps({ locale = 'en' }) {
  return {
    props: {
      ...(await serverSideTranslations(locale, [
        'common',
      ])),
    },
  };
}

inside a file which is not in the src/pages/ path.
You will need to add this structure to your page-level component.


Style/FormatStringToken mark for links

Steps to reproduce the problem

Create test.rb file with content

p 'https://ru.wikipedia.org/wiki/%D0%90_(%D0%BA%D0%B8%D1%80%D0%B8%D0%BB%D0%BB%D0%B8%D1%86%D0%B0)'

Run rubocop test.rb

Result:
Several messages like
tmp.rb:1:45: C: Style/FormatStringToken: Prefer annotated tokens (like %<foo>s) over unannotated tokens (like %s).
p 'https://ru.wikipedia.org/wiki/%D0%90_(%D0%BA%D0%B8%D1%80%D0%B8%D0%BB%D0%BB%D0%B8%D1%86%D0%B0)'

This string have nothing to do with strings tokens. Just a link
RuboCop version
Include the output of rubocop -V. Here's an example:
$ rubocop -V
0.52.0 (using Parser <IP_ADDRESS>, running on ruby 2.4.2 x86_64-linux)

The false positive occurs on %B which is a token for a binary number in a Ruby format string. It might be possible to improve the regular expression of this cop to check whether the string is a URL, which may contain URL encoded characters.

I'm seeing something that might be a similar issue within date-time formats.
C: Style/FormatStringToken: Prefer annotated tokens (like %<foo>s) over unannotated tokens (like %s).   LOGGING_DATETIME_FORMAT = '%m-%d-%Y %H:%M:%S.%N'

@matthewswann see #5223 and #5230

it also happens if you create a HAML-string as we mock in one of our specs:
  let(:template) do
    "render_widget :widget_1\n\
     render_widget :widget_2, position: 'sonstwo'\n\
     fasdfdsfsa fdjksdljfasdf\n\
     %div= render_widget :widget_1,\n :show, position: :testposition\n\
     render_widget :widget_3\n
     render_widget :widget_4,
       position: 'umbruch'\n"
  end


is this project still actively maintained?

Hi,
i'm wondering if this project is still actively maintained or not. I see some really critical bug reports but no activity from existing maintainers.
As it's nature a database needs a reliable environment and consistency in its development.
I'm a happy user of KeyDB but concerned about the existing issues and uncertainty.
If it was just a wrong impression feel free to delete this issue.
michael

there was a recent issue (duplicate of this and closed by @JohnSully ) #788
don't know if it answers your question :)

Hi,
thanks! I saw this issue as well, but created a new one because since then there was no activity in this project at all. No reaction to critical bug reports etc. I think the situation is a bit different now.
Given the time of  almost 2 weeks without a response from a maintainer confirms what was expected i think...

Yeah, I'm probably just going to use Valkey or even something more "cutting-edge" like Garnet since KeyDB seems to be mostly abandoned.

This project is dead now and have some really critical bugs, don't use it in production

does anyone know why it's dead? I mean it's still Snap behind who might have an interest to keep it running!?

Im still using in production systems.

This project is dead now and have some really critical bugs, don't use it in production

See comments from the lead developer last week to understand their development and maintenance priorities: https://github.com/Snapchat/KeyDB/issues/798#issuecomment-2214123117

I just looked at the activity of the 2 main contributors. It appears one is now contributing to Valkey and the other hasn't been online much since June. So yeah, it seems development has definitely stalled. It's a real shame - the EXPIREMEMBERS command is a gamechanger for tagged caching. But I won't be using it in production going forward.


Error with Codeium Auth : not an editor command

I'm new to Neovim, and I'm currently using a configuration called lazyvim.I done manual installation of codeium  However, when I attempted to use the command "Codeium Auth," It say not an editor command
E492: Not an editor command: Codeium Auth
what to do?

Looks like your PlugInstall didn't succeed.
Have you tried restarting neovim, then running :PlugInstall, then run, :Codeium Auth?
If PlugInstall worked you should see codeium listed as one of your installed packages if you restart and run :PlugInstall a 2nd time:

I got same error for PlugInstall and I installed vim-plug (Done Manual Installation , If i use their command flatpak and sh command i end up in can't connect , this is also happend in lunar vim , I downloaded their code and installed manually)

Looks like your PlugInstall didn't succeed.
Have you tried restarting neovim, then running :PlugInstall, then run, :Codeium Auth?
If PlugInstall worked you should see codeium listed as one of your installed packages if you restart and run :PlugInstall a 2nd time:

I got same error for PlugInstall and I installed vim-plug (Done Manual Installation , If i use their command flatpak and sh command i end up in can't connect , this is also happend in lunar vim , I downloaded their code and installed manually)

Manual install fixed the issue for me.

Bumping this. Still does not install with PlugInstall.


Add PixelCNN++ example

EDIT: This should be ready to merge, as you can see I haven't done everything on my original TODO list but what we do have now should be enough to land and iterate on.
TODO:

[x] Finish README
[x] Train to convergence
[x] Maybe simplify train.py
[x] Fix failing tests
[ ] Test a training iteration
[ ] Test receptive field
[x] Add checkpointing
[ ] Add naïve sampling

This LGTM (I mostly skimmed it).
It'd be also great to include generated samples in the README. I know you haven't yet committed your sampling code but could you keep a note to add that once sampling is ready?


Fix plugin test run.sh script parameter error

Please answer these questions before submitting pull request

Why submit this pull request?

[x] Bug fix

[ ] New feature provided

[ ] Improve performance

Related issues

Bug fix

Bug description.

How to fix?

New feature or improvement

Describe the details and related test reports.

@dmsolr Your review

LGTM

/run agent-plugin-test

/run agent-plugin-test


Accessing Related Model Operations

I have a Building model that:

morphOne Address
morphMany Document

In my routes/buildings.js file, I need to save all of this data at once so, wrapped in a transaction, I'm saving the building, then saving the address, then saving all attached documents.
I thought I might be able to explicitly require() only my Building model in the route and access the other models via some kind of chaining syntax. So far, I've been able to do this with the Address relationship:
listing.related('address').save(...).then(...)

I can't seem to make it happen with the morphMany relationship. Guessing because it's a collection?
listing.related('documents').save(...).then(...)

I've tried a lot of variations using at, get, relatedData, etc, but, if it's possible to do this, I just haven't found the right combination of methods.
What's the right approach for something like this?

Currently Bookshelf has no support for batch inserts. I'd recommend dropping to Knex to insert all the records at once.

In this case, I have a single object I need to save. I'm not so much interested in the save operation itself as much as a way to reference the model for saving. What I have now is a Listings route that explicitly requires the Document model and performs a Document.forge(...).save(...) operation.
What I'd like to do is drop the explicit var Document = require('./document') and access the Documentmodel through theListing model the way I can in CakePHP (Listing->Document->save()`) or perhaps in Rails as well. Maybe I'd need to create a separate association to do something like that.
Does that make a little more sense? My goal/preference was to only require the Listing model in the Listing route and use associations to access any other models I may need. Thought that might be possible, but hadn't managed to get it right.

The project leadership of Bookshelf recently changed. In an effort to advance the project we close all issues older than one year.
If you think this issue needs to be re-evaluated please post a comment on why this is still important and we will re-open it.
We also started an open discussion about the future of Bookshelf.js here https://github.com/bookshelf/bookshelf/issues/1600. Feel free to drop by and give us your opinion.
Let's make Bookshelf great again


Use "inert" for Modal and Drawer to dissallow focus on anything behind the backdrop

Describe the feature in detail (code, mocks, or screenshots encouraged)
By using inert you can make a DOM element and it's descendants un-interactive-able which can come in handy when implementing overlays like Modals and Drawers. By putting inert on the %sveltekibody% surrounding div and placing the Modal and Drawer element besides that div we can disable interactions for the entire app on demand whilst having a overlay active on top.
Here is a small demo.
As shown in the project we can utilize the portal action to rerender any element in a different place.
Because I am working on my own headless component library I went ahead and implemented it aswell, it can be found here if you want to see it in action.
This is what the structure of the  will look like when implemented:
Modal/Drawer Hidden:

%sveltekit.body%

Modal/Drawer Visible:

%sveltekit.body%

...

This would also indirectly fix: #1613
What type of pull request would this be?
New Feature
Provide relevant links or additional information.
This feature was discussed prior to me making this issue, the full conversation can be read about here: https://discord.com/channels/1003691521280856084/1003699523522142399/1116800227148566558

Adding a reference to the prior PR which this will replace:
https://github.com/skeletonlabs/skeleton/pull/1614

What's the plan for this PR, I understood once this gets the green light I could implement this?

@Hugos68 we'll circle back. Right now my priorities are elsewhere. But we don't keep tickets that we don't plan to move forward on.
Last we left off I believe the plan was to create a standalone proof of concept of the portal feature. Then determine if that's something we want to implement as a dedicated feature. Once that's in place, we can look to update our modal system to use this + the inert body style.
I'd welcome help with either part of those.

Sounds good, not sure if you noticed but there is a proof of concept repo that showcases the portal + inert feature's to create a accessible modal. Once you have time to look over that I'd love to hear feedback 👍

Great, send me a link here please

This is a standalone minimal repo: https://github.com/Hugos68/modal-drawer-inert-sample-project
This is an implementation of it: https://ghostui-rho.vercel.app/docs/factories/dialog

@endigo9740 What should we do with this? I am scanning through the issues FYI.

@Hugos68 this will be bumped to v3 milestone later this week ;)

In an effort to prepare for Skeleton v3, we're consolidate some related issues down to a single ticket. This will ensure that we can see the full context of requests when the time comes to refactor and update this feature going forward. If you wish to add additional feedback or suggestions, please so here:

https://github.com/skeletonlabs/skeleton/issues/2357


Type for Arrays of Enums not set during migration

With an enum type mapped like so:
protected override void OnModelCreating(ModelBuilder builder) {     builder.ForNpgsqlHasEnum("Mood", new[] { "happy", "sad" }); }
static MyDbContext() {     NpgsqlConnection.GlobalTypeMapper.MapEnum<Mood>("Mood"); }
And an array defined like so:
public List<Mood> Moods{ get; set; }
Throws :
42704: type "Mood[]" does not exist
I think the issue is the generated SQL does not type the array specifically, like so:
ARRAY['happy','sad']::Mood[]

Issue was caused by the casing of the enum.
When mapping used mood instead of Mood the migration went smoothly.
I'll update the docs to point out this caveat.

This looks like a duplicate of #440.
Can you post some version information?

With an enum type mapped like so:
protected override void OnModelCreating(ModelBuilder builder)
{
    builder.ForNpgsqlHasEnum("Mood", new[] { "happy", "sad" });
}

Throws :
42704: type "Mood[]" does not exist

That configuration is correct for the PostgreSQL quoted identifier:
CREATE TYPE "Mood" AS ENUM ('sad', 'ok', 'happy');

Yep, agree with @austindrenski that this is a dup of #440 - I intend to release 2.1.1 this weekend so this should be gone. @JNBarnes can you please test the latest nupkg from the stable feed and report if it resolves the issue for you?

Closing as no additional info was provided within 7 days. But please feel free to post back here with the relevant information and I'll reopen if necessary.


CM function?

Which classical mechanics function have you implemented?

@tirthbha we are still deciding. What has your group chosen?

@reshnashrestha we are planning to choose projectile motion.

We have chosen projectile motion with air resistance.


Standardise building function names in CDK

Background
Split from:

https://github.com/gchq/sleeper/issues/933

In the linked issue we added a way to build a function name without explicitly concatenating it with the instance ID.
Description
We'd like to apply the utility for building a function name across the CDK stacks. We could look at how other resource names are built.

Changed my mind, this would make it harder to change the resource names.


Bug or wrong behaviour in the PieCharts

When the users click in one of the tissues in the piechart the search only shows one experiment:
http://localhost:8000/app/browse.html#/search?q=undefined even when the tissue in the pie charts says that the resource contains 25 experiments.

done


Handle the case when losing connection with the server

Currently when there is network connection issue we fail silently -- in the case of dashboards we keep trying to reload, but in other cases the request just fails.
The desired behaviour should be:

Retry to reload for X times.
In case retries failed, show "Reconnect" message like HackPad has.

This is mainly needed when running queries, because otherwise the user will keep seeing the counter running up without anything really happening.
This should probably be handled by some global $http filter (interceptor?) of some sort.

Offline.js proved to be causing issues on Chrome/Windows. Reverted, and will reimplement with our own simplified version.


Unable to Add Conditions on Data Breakpoints

Does this issue occur when all extensions are disabled?: Yes/No
No: requires debugger extension to reproduce.

VS Code Version: 2b5bf2176ed9acf83442a4914ae01dde65d2b91b
OS Version: RHEL 7

Steps to Reproduce:

Using a debugger that supports data breakpoints. I was working on supporting data breakpoints in the CDT-GDB adapter.
Add a breakpoint to some variable.
The breakpoint will appear in the breakpoints view with text like Break when <variable> is read. The actual text depends on the debug adapter.
Right click on the breakpoint in the breakpoint view and select Edit Condition....
Nothing happens.

The handler for that action is here:
https://github.com/microsoft/vscode/blob/67e47344d235cffdfd7d0f860fb6e5b2a520c385/src/vs/workbench/contrib/debug/browser/breakpointsView.ts#L1392-L1440
Since data breakpoints don't satisfy either of the first two conditions, they're handled in the last block:
https://github.com/microsoft/vscode/blob/67e47344d235cffdfd7d0f860fb6e5b2a520c385/src/vs/workbench/contrib/debug/browser/breakpointsView.ts#L229-L233
I'm not quite sure what's supposed to be happening once the inputBoxData is set, but it doesn't appear to be taking effect.

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.80.1. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Always a pleasure to hear from you, @VSCodeTriageBot. As you'll note, the commit I tested on is in fact newer than the 1.80.1 release.

Repro steps on #195695

I am preparing a PR


Pre coin list blog (#32)

begins urgent revisions

begins urgent revisions

next iteration for updates

proof round for those 2 pages

adds todo

adds tokens hopefully truthful

@afalaleev Please run Update QA stand to test, and if all good, accept/merge, and update production


Raspberry Pi Pico 2040

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
I would like to use PicoVoice with RP2040. KWS is possible with Edge impulse. However, it is hard to find a nice speech-to-intent provider. Plus I would love to have an integrated solution, where KWS and S2I is on the same platform.
Describe alternatives you've considered
Edge Impulse
Additional context
I think edge ASR could get big and I want to use PicoVoice to prototype use cases.

let's keep this open and see how much traction it gets. then we can prioritize long-term. what's the board you are using?

+1 for this, Porcupine and Cheetah support for the RP2040 would be awesome.

I don't think we will be able to prioritize this in the coming months. hence closing for now. If things changes on our end we will re-open to track progress.


Example or documentation on --index-url

Describe the issue briefly here.
I didn't see any examples or documentation on using --index-url for a private PyPi. Experimenting, I found that adding --index-url  in the requirements.in file did the correct thing and mirrored that in the requirements.txt.
Is there additional docs needed here for different pip options? Or you can still do anything in a requirements.txt file in the .in file and have it be preserved?
Environment Versions

OS Type
Python version: $ python -V
pip version:  $ pip --version
pip-tools version: $ pip-compile --version

Steps to replicate

...
...
...

Expected result
...
Actual result
...

Currently, the following options in requirements.in are preserved in requirements.txt:

--index-url
--no-index
--extra-index-url
--only-binary/--no-binary
--trusted-host
--find-links

And yes, it's a lack of documentation for this topic. PRs are welcomed :)

Note, that below options in requirements.txt are not supported in pip-sync. Supported only cli options. There is a PR #824 wich fixes that.

I'll close this based on the above, but please let us know if it doesn't resolve your issue. Thanks!


Error with MIDI

Exception in callback _ProactorBasePipeTransport._call_connection_lost(None)
handle: <Handle _ProactorBasePipeTransport._call_connection_lost(None)>
Traceback (most recent call last):
File "C:\Python3101\lib\asyncio\events.py", line 80, in _run
self._context.run(self._callback, *self._args)
File "C:\Python3101\lib\asyncio\proactor_events.py", line 165, in _call_connection_lost
self._sock.shutdown(socket.SHUT_RDWR)
ConnectionResetError: [WinError 10054] An existing connection was forcibly closed by the remote host
got prompt
Failed to validate prompt for output 180:

MIDILoadAndExtract 190:

Exception when validating inner node: module 'folder_paths' has no attribute 'exists_and_is_file'
Output will be ignored
Loading model from: C:\comf\ComfyUI\models\openunmix\umxl.pth
Using cache found in C:\Users\Kuzma/.cache\torch\hub\sigsep_open-unmix-pytorch_master
Exception in callback _ProactorBasePipeTransport._call_connection_lost(None)
handle: <Handle _ProactorBasePipeTransport._call_connection_lost(None)>
Traceback (most recent call last):
File "C:\Python3101\lib\asyncio\events.py", line 80, in _run
self._context.run(self._callback, *self._args)
File "C:\Python3101\lib\asyncio\proactor_events.py", line 165, in _call_connection_lost
self._sock.shutdown(socket.SHUT_RDWR)
ConnectionResetError: [WinError 10054] An existing connection was forcibly closed by the remote host
Prompt executed in 5.85 seconds

kuzman! Thank you SO much for your interest in my nodes, especially in the MIDI functionality!!!! I have been waiting for someone specifically interested in the MIDI keyboard.
That being said, I have become extremely busy over the last week so I can't fix this at the moment but I hope to be able to in the coming week. Hopefully you'll be willing to give it another shot then! The same applies to the other issue you opened - I really appreciate the feedback but I'll need to come back to it.
Thanks again!!

Hey Ryan, no prob, i will be here hopefully. I'm really interested to make that feature working, cause that will open Pandora's box

KUZMAN
should be fixed lmk!


Clean up 3D Tiles Next documentation

This is another PR that targets the 3d-tiles-next branch. It is to clean up the branch before we open a PR to merge 3d-tiles-next into master
This PR does the following:

Any 3D Tiles Next features are marked as @private (so it doesn't show up in the JSDoc) and @experimental (so even those looking at the code know these features are subject to change)
A few enums were missing documentation of individual members. I added these.
Updated the changelog to mark these features as part of a future release in June

The @private/@experimental changes are perhaps a bit overkill, but we want to be extra careful that we are not introducing public API changes at this time.
@lilleyse could you review? Note that there were many, many files to update, so here's a full list I used as I went: changed-files.txt

Thanks for the pull request @ptrgags!

:heavy_check_mark: Signed CLA found.
:grey_question: Unit tests were not updated.

Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.
[ ] Works (or fails gracefully) in IE11.

@ptrgags I removed some experimental tags. My rule of thumb was:

General purpose files like MortonOrder don't need the tag.

I put glTF loaders in that category since they aren't strictly 3D Tiles Next

Helper files like has3DTilesExtension that aren't specific to 3D Tiles Next don't need the tag
Existing files that have new members related to 3D Tiles Next have the tag on those members
New files related to 3D Tiles Next have the tag on the constructor only

@lilleyse looked through your changes and summary, that makes sense. Thanks for sorting that out. Looks like I went overboard out of caution.
Is there anything else you need to look over before we merge this?

Nope, good to merge.


Introduce MATH_EXPRESSION data constructor

This PR introduces math expression constructor

Pushed PR without syncing with recent changes, creating new PR.


1.1.17 throws AttributeError

Our build system runs pex and throws the following stack trace. I've confirmed I don't see this with 1.1.16.
Traceback (most recent call last):
  File ".tox/package/bin/pex", line 11, in <module>
    sys.exit(main())
  File ".../lib/python3.4/site-packages/pex/bin/pex.py", line 533, in main
    options.cache_dir = make_relative_to_root(options.cache_dir)
  File ".../.tox/package/lib/python3.4/site-packages/pex/bin/pex.py", line 514, in make_relative_to_root
    return os.path.normpath(path.format(pex_root=ENV.PEX_ROOT))
AttributeError: 'list' object has no attribute 'format'

With the additional message
ERROR: InvocationError: 'pex . -o dist/component.pex -c component --disable-cache -v -i <url>'

@wayne-luminal thanks for the bug report and apologies for the breakage. we'll get this fixed up asap.
@digwanderlust looks like this was related to your change that just went out in 1.1.7. mind taking a look?

fwiw, it seems the use of --disable-cache is the bit that causes this.

For some readon disable cache sets the cache dir to [], I'll add a test and fix this.

Fix is up at: https://github.com/pantsbuild/pex/pull/261

this should be fixed in 1.1.8 - now available on pypi: https://pypi.python.org/pypi/pex/1.1.8
verified I can no longer reproduce the breakage using this release. closing.

👍  Thank you!


Going up past the start of a commit message goes into history and sometimes loses the commit message you were typing

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.85.2
OS Version: Windows 11

Steps to Reproduce:

click on source control on the bar on the left side that causes it to scroll
start typing a long commit message, then hit up arrow to go to the top, but overshoot and now it is going back in history
going back down sometimes it brings back what you were typing, but often it does not and you have to start over :(

I really want to disable this behavior so that VSCode will NOT scroll back to previous commit messages just by hitting the up arrow

Similar to #222005

going back down sometimes it brings back what you were typing, but often it does not and you have to start over :(

Could you please elaborate on when does pressing the down-arrow not bring back the changes?
Pressing the keybinding for Undo should also bring back the changes. Have you tried that?

@lszomoru, No I cannot elaborate on when it does not bring back the changes. It seems that if you press up and go past then down then up again eventually it will lose the changes. I did this with a very long commit message and was attempting to go back to the top and it would overshoot often. Even if it does not lose the entry, it is very disconcerting and irritating to have up arrow all of a sudden change the message from the one you are entering to one from the history when you are attempting to go to the top of the message.
What is really needed is some way to turn this behavior off so that up arrow will not go back into history and will instead stop at the top of the message. Even better would be to use page up/down or some other key to go back into history.

Adding @joaomoreno who worked on the initial implementation of the commit message history.

going back down sometimes it brings back what you were typing, but often it does not and you have to start over :(

Could you please elaborate on when does pressing the down-arrow not bring back the changes? Pressing the keybinding for Undo should also bring back the changes. Have you tried that?

Thanks for this tip to hit Undo...when it disappears like that so easily and simply, I felt it was gone forever. It is really easy to occur and hurts the developers that like to write verbose commits...we go back to the top to write more and then it skips to the previous commit, all seems lost, at least we become throughly confused. It is the pits! Or if I hold down the up bottom, it scrolls through like 5 commits before I can figure out what is happening.
If there is a use case where the previous commit info is valuable inside of a text box for a new commit, there must be a different way to do it.


Item Permissions

I have reached out to others to further test that this is not working, and I believe my setup is set correctly per the documentation...
Can you please confirm this for us so we can check items on a person... Thank you... Here is the code I am using directly to try and use the permission item check
user:hasPermission("!item.mafiakey.>0") as per the example....
I am removing the Item and returning it as a alternative...

Just tested it, working fine with !item.edible|tacos.>0.

Changed a few things around and now its working.
Thank-you!


Pino for all workers

adds pino logging to

'geoserver-init'
'rollback-handler'
'error-handler'
'klips-api'

connected to https://github.com/klips-project/rabbitmq-worker/pull/140

@weskamm I agree that there is a duplication of the logger.js. But I also did not find a good solution for this problem.
I also see this as an follow up PR


Sending Email with SES Only Works with Xcode

Describe the bug
I have some simple code to send an email using SES. Sending the email works fine with Xcode using a simulator (iPhone, iPad, Mac), and sending the email also works fine with Xcode using a physical device (iPhone, iPad).
However, when I try to send the email using only a physical device, without Xcode, it fails. It seems like sending email with SES only works when running in Xcode.
Expected Behavior
I expect the email to be sent using SES.
Current Behavior
If the app is run on a device, and not with Xcode, sending the email fails. I get this error.
The operation couldn't be completed.
(AwsCommonRuntimeKit.CommonRunTimeError error:0.)
Reproduction Steps
I created a simple SwiftUI test app and the complete code is below.  You just need to put in valid SES from and to email addresses.
struct ContentView: View {
    @State var isButtonDisabled = false
    @State var showAlert = false
    @State var alertMessage = ""

    var body: some View {
        VStack {
           Button("Send Mail") {
               isButtonDisabled = true
               Task {
                    do {
                         try await sendEmail(fromEmailAddress: fromEMailAddress, toEmailAddress: toEmailAddress, subject: "SendMail Test Mail", message: "This is a test email from the SendMail app")
                        }
                        catch (let error) {
                             print("Error: \(error.localizedDescription)")
                             alertMessage = error.localizedDescription
                             showAlert = true
                        }
                        isButtonDisabled = false
                    }
                }
            .disabled(isButtonDisabled)
        }
        .padding()
        .alert(isPresented: $showAlert) {
               Alert(title: Text("SendMail Error"), message: Text(alertMessage), dismissButton: .default(Text("OK")))
        }
    }

     func sendEmail(fromEmailAddress:String, toEmailAddress:String, subject:String? = nil, message:String? = nil) async throws {
          let sesClient = try SESClient(region: "us-east-1")
          var sendEmailInput = SendEmailInput()
          sendEmailInput.source = fromEmailAddress
          sendEmailInput.destination = SESClientTypes.Destination(toAddresses: [toEmailAddress])
          var body = SESClientTypes.Body()
          let bodyContent = SESClientTypes.Content(charset: "UTF-8", data: message ?? "")
          body.text = bodyContent
          let subject = SESClientTypes.Content(charset: "UTF-8", data: subject ?? "")
          sendEmailInput.message = SESClientTypes.Message(body: body, subject: subject)
          let _ = try await sesClient.sendEmail(input: sendEmailInput)
    }
}

Possible Solution
No response
Additional Information/Context
No response
AWS SWIFT SDK version used
0.20.0
Compiler and Version used
Xcode 14.3.1, Swift 5.8.1
Operating System and version
iOS 16.5.1, macOS 13.4.1

@minimalistcode How are you providing credentials to the SDK when you are running on device?
Certain means of providing credentials (i.e. from your ~/.aws/credentials file) that work when running in Xcode don't work when running on device.

Thanks, that must the problem. The credentials are in ~/.aws/credentials and also in the the Xcode scheme environment variables. I was trying an old app that doesn't use Amplify.  It used to use credentials with Cognito and a single identity pool so I guess I can figure out how to do that with this new SDK.

That was the issue. Thanks. I will close this since it's not a bug.


Error adding products.

When I attempt to add a new product I get the following error:
Error in Saving Product: SQLSTATE[42S22]: Column not found: 1054 Unknown column '0' in 'field list' (SQL: update products set 0 = {"id":3,"type":"PRODUCT","name":"AWG","identifier":"awg","use_as":"VARIATION","field_type":"SELECT","sort_order":12,"created_at":"2017-11-15 22:56:34","updated_at":"2017-11-15 22:56:34","pivot":{"product_id":25,"attribute_id":3}}, updated_at = 2017-12-18 16:28:29 where id = 25)
Error occurs when I create a new product and then attempt to add the product detail information.

Here is the Post Data.
_token | "4osajlu9czouN3pPttBiVJFuIymLrC7jE93NLHGw"
_method | "put"
name | "Honeywell Thermostat Wire"
category_id | array:1 [▼   0 => "4" ]
slug | "honeywell-thermostat-wire"
sku | "HTW-1000-10"
description | "Honeywell OEM 10 gauge thermostat wire."
price | "32.09"
status | "1"
qty | "500"
in_stock | "1"
track_stock | "1"
is_taxable | "1"
page_title | ""
page_description | ""
attributes_specification | array:1 [▼   3 => "" ]
attribute_id | array:1 [▼   0 => "3" ]

To me it seems like Error is belongs to Product Attribute Save/Update bug. Ideally it should ignore if Attribute Value is not specified. I will take a look at the bug and fixed it soon Thanks for raising.


Mis-registration between outputs and  input images

Hi,
Thank you for getting back on the previous enquiry!
I have run the code on my DWI datasets without any issues, and the outputs looked great and sensible. However, the outputs do not seem to be in the same voxel space as the inputs (i.e, the original dwi.nii.gz or b0.nii.gz). Is it possible to tweak the code so that I can overlay the free water corrected FA on FA maps that were previously generated using FSL dtifit?
Many thanks for your help indeed.

Hi, I think issue #2 has some information on how to hack around this. It should be possible to tweak the code as well. Sorry I haven't looked at it in a long time.


Expand-Archive performance

Allow me to copy the contents of an issue posted under PowerShell/PowerShell.
Expand-Archive performance is 6X away from 7zip. In it's current state I would say it barely passes automation perf level. In an interactive shell, one would rarely resort to it if there is an alternative that is 6X faster.
Steps to reproduce
Download some large archive (qt-everywhere-opensource-src-5.9.0-alpha.zip was my use case) and unzip.
Expected behavior
Something like this:
PS C:\Users\Matty\Downloads> Measure-Command \
>> { C:\Kellekek\7-Zip\7z.exe x .\qt-everywhere-opensource-src-5.9.0-alpha.zip }

Days              : 0
Hours             : 0
Minutes           : 37
Seconds           : 57
Milliseconds      : 938
Ticks             :<PHONE_NUMBER>8
TotalDays         : 0,0263650331342593
TotalHours        : 0,632760795222222
TotalMinutes      : 37,9656477133333
TotalSeconds      : 2277,9388628
TotalMilliseconds : 2277938,8628

Actual behavior
PS C:\Users\Matty\Downloads> Measure-Command \
>> { Expand-Archive -Path .\qt-everywhere-opensource-src-5.9.0-alpha.zip -DestinationPath C:\Kellekek\Qt }

Days              : 0
Hours             : 3
Minutes           : 44
Seconds           : 34
Milliseconds      : 218
Ticks             :<PHONE_NUMBER>46
TotalDays         : 0,155951597506944
TotalHours        : 3,74283834016667
TotalMinutes      : 224,57030041
TotalSeconds      : 13474,2180246
TotalMilliseconds : 13474218,0246

Environment data

PS C:\Users\Matty\Downloads> $PSVersionTable

Name                           Value
----                           -----
CLRVersion
WSManStackVersion              3.0
PSVersion                      6.0.0-alpha
PSEdition                      Core
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
GitCommitId                    v6.0.0-alpha.17
BuildVersion                   <IP_ADDRESS>

PS C:\Users\Matty\Downloads> C:\Kellekek\7-Zip\7z.exe

7-Zip [64] 15.14 : Copyright (c) 1999-2015 Igor Pavlov : 2015-12-31
...

PS C:\Users\Matty\Downloads> (Get-ComputerInfo).CsProcessors

Name                      : AMD FX-7500 Radeon R7, 10 Compute Cores 4C+6G
Manufacturer              : AuthenticAMD
Description               : AMD64 Family 21 Model 48 Stepping 1
Architecture              : x64
AddressWidth              : 64
DataWidth                 : 64
MaxClockSpeed             : 2100
CurrentClockSpeed         : 2100
NumberOfCores             : 4
NumberOfLogicalProcessors : 4
ProcessorID               : 178BFBFF00630F01
SocketDesignation         : Socket FS1r2
ProcessorType             : CentralProcessor
Role                      : CPU
Status                    : OK
CpuStatus                 : Enabled
Availability              : RunningOrFullPower

I have the same problem. When it will be fixed?

will this ever get fixed?

Also interested by the resolution of this issue

It is very slow. It needs 10 minutes to extract 200MB. Any workaround? Maybe an option that could improve the extraction performance?

My intent is after PS7 GA to have resources applied to this module and convert it to C# using some OSS libs that will solve a number of outstanding issues.

By the way, if anybody has ideas on a good archive engine that potentially can be used - feel free to post your suggestions here...

Also have the same issue. It's unbelievably slow.

You can get some performance if you do not show progress when Expanding.
Setting $progressPreference = 'SilentlyContinue' before the expand

I found 2 ways to improve unzip speeds.
First, exclude the directory from Windows Defender. It makes things really slow. I'm extracting a 50mb file and doing this changed it from taking about 4 minutes to taking 26 seconds. You must decide if you're OK with the security risk.
Add-MpPreference -ExclusionPath C:\installers

Second, use this .NET class to extract the zip. This changed it from taking 26 seconds down to 12 seconds.
Add-Type -Assembly "System.IO.Compression.Filesystem"
[System.IO.Compression.ZipFile]::ExtractToDirectory("C:\installers\app.zip", "C:\installers\app")

I am having the same issues, same zip file, used to take 3-4 minutes. it is now taking over like 2 hours, randomly, after yesterday 07 Feb 2022, azure outage issues. Was working perfectly before no issues.

I actually cannot understand why Microsoft is not fixing this?

@mkarg This is on Microsoft's radar.

Love to see this issue being addressed

Blog post on Archime module 2.0 and the Github project tracking progress.

Add-Type -Assembly "System.IO.Compression.Filesystem"
[System.IO.Compression.ZipFile]::ExtractToDirectory("C:\installers\app.zip", "C:\installers\app")

@raymondbutcher This! For a 120MB file this improves extraction times from 1min. 20sec. to just 9 sec.

That example by @s-geiger-si  is sooo much faster it is just crazy...

@mkarg This is on Microsoft's radar.

Nice. And when will Microsoft fix it? Two years are gone.

Agree with @mkarg.
@SydneyhSmith, @ayousuf23  any clarification on when to expext this?


Move hex cache

Offline HEX cache is now under built/, and gets populated by pxt serve

I can still repro the issue:

link pxt in pxt-microbit
delete built folder
add a dummy function in basic.cpp to trigger a C++ compilation
run "pxt buildtarget"

----- Offline HEX cache
Extracting target's default project
Installing project's packages (pxt install)
Building the project (pxt build --cloud)
INTERNAL ERROR: Error: dal.d.ts not found in core
    at Object.assert (C:\gh\pxt\built\pxt.js:54542:27)
    at Object.getExtensionInfo (C:\gh\pxt\built\pxt.js:55643:30)
    at C:\gh\pxt\built\pxt.js:57448:35
    at tryCatcher (C:\gh\pxt\node_modules\bluebird\js\release\util.js:16:23)
    at Promise._settlePromiseFromHandler (C:\gh\pxt\node_modules\bluebird\js\release\promise.js:509:31)
    at Promise._settlePromise (C:\gh\pxt\node_modules\bluebird\js\release\promise.js:566:18)
    at Promise._settlePromiseCtx (C:\gh\pxt\node_modules\bluebird\js\release\promise.js:603:10)
    at Async._drainQueue (C:\gh\pxt\node_modules\bluebird\js\release\async.js:143:12)
    at Async._drainQueues (C:\gh\pxt\node_modules\bluebird\js\release\async.js:148:10)
    at Immediate.Async.drainQueues (C:\gh\pxt\node_modules\bluebird\js\release\async.js:17:14)
    at runCallback (timers.js:574:20)
    at tryOnImmediate (timers.js:554:5)
    at processImmediate [as _immediateCallback] (timers.js:533:5)

Building packages under libs/ now also build templates and populate the hexcache; hexcache generation is now completely transparent and is part of buildtarget

Taking a look at the build failure

This now depends on https://github.com/Microsoft/pxt-microbit/pull/303


Moving up/down follows neither visual or actual column now

Issue: https://github.com/Microsoft/vscode/issues/22717
PR: https://github.com/Microsoft/vscode/pull/44259
The PR does not fix the issue and introduced more issues.
Repro
Have the following text:
一二三四五六七八九十
	一二12345678901234567890 // tab in beginning
一二三四五六七八九十
1234567890123456789

Set font to Inconsolata
Set tabWidth to 4
Put cursor in last line after 1234. Now status bar col shows 5`
Up arrow
Now cursor is after 一二三四 and status bar col shows 9
🐛 The cursor is NOT visually aligned with previous col position

If it works correctly, cursor should be after 一二

Regardless, you go up again
Now cursor is after 	一二1 and status bar col shows 10
🐛 The cursor is NOT visually aligned and NOT column-wise aligned with previous col position.

If it's visually aligned moving, you should end up after 	一二 since 4x Chinese char are 8 spaces, and 1x tab + 2x Chinese char are also 8 spaces
If it's column aligned moving, you should end up after the initial tab since you have 4x Chinese char and tabWidth is 4.

@octref Thank you for verifying, but I believe you have attempted to verify in a build that did not contain the fix (i.e. the behaviour you describe sounds a lot like what is on stable):

@alexandrudima I verified by building on commit ba38492bfd5483375dddcce289fffd6060536e69 which has your fix 13 hours ago.

@octref Hmm...
What am I doing differently? Can you reproduce in today's insiders ?

And on the freshly built insiders too with d7dca142ad6051e6b1c7397ea73221feb0160f21

We chatted and the issue turned out to be with the vim extension.

@rebornix Think we should let VSCodeVim know so the new release wouldn't break for Chinese/Japanese Vim users.

This is working correctly in VS Code. Vim issue is tracked in #44958.


Added the doctest library

Saw your C++Now talk and notices that you support both google test and Catch but not doctest. So I added it.

Ey! Sorry it took so long to get an answer :)
This looks good, so I'll go ahead and merge it. We'll try to look after this lib as much as possible, but we'd be hyper grateful if you if you kept an eye on it and let us know when a worthwhile update is released, so it can be added to CE
Again, thanks :)

No problem :)
I'll definitely try to keep an eye on the lib. Especially since I basically use it for all my tests, that I have in my projects.


UploadPresenter: failed to upload: Attempt to invoke interface method 'java.lang.Object[] java.util.Collection.toArray()' on a null object reference

On the latest structured-data branch, this happens when uploading any picture, all the time:
04-07 23:07:21.955  5632  5835 D ConnectivityService: mDiagnosticHisotry added: 20-04-07_23:07:21 fr.free.nrw.commons 'Upload started!', 0
04-07 23:07:21.959  5632 13944 D [a11y]  : getUserAccounts0
04-07 23:07:21.963 21305 21305 E ViewRootImpl: sendUserActionEvent() returned.
04-07 23:07:21.963  5632  5632 D Telecom:SamsungPrebindingServiceImpl: onTopActivityChanged ComponentInfo{com.sec.android.gallery3d/com.samsung.android.gallery.app.activity.GalleryActivity}
04-07 23:07:21.964 21305 21305 D ViewRootImpl@d461a8d[UploadActivity]: MSG_WINDOW_FOCUS_CHANGED 1 1
04-07 23:07:21.964 21305 21305 D InputMethodManager: prepareNavigationBarInfo() DecorView@fee4ba9[UploadActivity]
04-07 23:07:21.964 21305 21305 D InputMethodManager: getNavigationBarColor() -855310
04-07 23:07:21.968  5632  5632 D CS_HdrController: updateMultiWindowState : 0
04-07 23:07:21.968  5632  5632 D MdnieScenarioControlService: MultiWindowState : false
04-07 23:07:21.969 21305 27757 E UploadPresenter: failed to upload: Attempt to invoke interface method 'java.lang.Object[] java.util.Collection.toArray()' on a null object reference

The picture is not uploaded and a toast saying File not found. Please try another file. appears.
Either sharing from Galaxy S10's default gallery app, or choosing from within the app.
Sounds similar to #1976.

try on the other branch if you could as that branch is all about fixing the uploads


<EMAIL_ADDRESS>hash check failed

Download: Status Legend:
Download: (OK):download completed.
Checking hash of FreeClipViewer.zip ... ERROR Hash check failed!
App:         extras/freeclipboardviewer
URL:         http://www.freeclipboardviewer.com/FreeClipViewer.zip
First bytes: 3C 21 44 4F 43 54 59 50
Expected:    2e56d300c131f082275eadac01aaa535216fdfa965eedf2c64dfc63eff5bbfeb
Actual:      795c6733b912e7c750746433de7b4fa35c82ce821308ca653ba9fbc3a04102f7

now i checked it. my isp's secure internet filter cause the issue.

@sethazazel How did you determine that this is what was happening, and how did you fix it?

i opened 2 issue (freeclipboardviewer and cpu-z) and i saw that actual hash codes were same for both.i opened softwares website there was no problem but my browser was firefox quantum and it has a feature called dns over https when i disabled it i couldn't access website, it was blocked by isp for security issues.


Whether the complete 'results.json' of the all dataset & all Methods (NaiveRAG, RQ-RAG, HyDE, GraphRAG, LightRAG) can be provided for reproduction?

I am in the process of reproducing the experiments from the paper and have successfully completed Step 1, 2, and 3, obtaining the query_result data pairs.
However, I am currently stuck as I do not have the answers generated by other methods, which are required for the batch_eval (it needs two sets of answers).
{
    "query": "What role does community-supported agriculture play in supporting small farms?",
    "result": "### Community-Supported Agriculture (CSA) and Its Role in Supporting Small Farms\n\nCommunity-Supported Agriculture (CSA) is a  that directly connects consumers with local farmers by enabling them to purchase shares of a farm’s harvest at the beginning of the growing season. This arrangement provides farmers with upfront capital, which can be crucial for purchasing seeds, fertilizers, and other necessary supplies. In turn, subscribers receive regular deliveries of fresh produce throughout the growing season.\n\n#### Financial Support\n\nOne of the primary benefits of CSA programs for small farms is financial stability. By securing a portion of their income before the growing season begins, farmers can better manage their finances and reduce the risk associated with farming. This is particularly important for small-scale operations, which often operate on tight margins and may struggle with unpredictable weather conditions or market fluctuations.\n\n#### Direct Market Access\n\nCSA also offers small farms direct access to consumers, bypassing intermediaries such as wholesalers and retailers. This direct connection allows farmers to maintain higher profit margins and ensures that they receive a fair price for their products. Additionally, it enables farmers to build personal relationships with their customers, fostering a sense of community and mutual support.\n\n#### Educational Opportunities\n\nThrough CSA programs, farmers have the opportunity to educate their members about sustainable agricultural practices, seasonal eating, and the importance of supporting local food systems. These educational efforts can lead to increased consumer awareness and appreciation for locally grown produce, further encouraging community involvement and support.\n\n#### Challenges and Limitations\n\nWhile CSA can provide significant benefits, it is not without its challenges. For instance, managing a CSA program requires additional time and resources from farmers, including administrative tasks such as marketing, record-keeping, and customer service. Moreover, there can be logistical issues related to distribution and storage of perishable goods.\n\n### Conclusion\n\nIn summary, Community-Supported Agriculture plays a vital role in supporting small farms by providing financial stability, direct market access, and opportunities for education and community engagement. However, it is essential to acknowledge the challenges involved in managing such programs to ensure their long-term sustainability and success.\n\nGiven the input data tables provided do not contain specific details about CSAs or small farms, this overview is based on general knowledge about the topic rather than specific data points."
}

This has led me to wonder if it is possible to use the {cls}_result.json files from Google Drive or Baidu Cloud, which contain results for the methods mentioned in the paper (NaiveRAG, RQ-RAG, HyDE, GraphRAG, LightRAG) across the all datasets (Agriculture, CS, Legal, Mix).
This would significantly enhance the reliability of the experiments and allow anyone to reproduce all the experimental data in Performance Table using the batch_eval.

Agree, providing the experiment scripts and files can help reproduce.


bug: helix themes are broken missing attribute declarations

Did you check docs and existing issues?

[X] I have read all the tokyonight.nvim docs
[X] I have searched the existing issues of tokyonight.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.10.0-dev-2066+g1813661a6-Homebrew
Operating system/version
macOS 14.2.1
Describe the bug
This is specific to helix. Themes that are completely missing attribute definitions as of https://github.com/folke/tokyonight.nvim/commit/9c5ef1e69ad2bd73b4ebe945e7e02284f2658743#diff-7d0281c0678c4322e968c0aaba8a155200ffce96cc30be6005b66e6f536707ccR25
Steps To Reproduce
Copy any of the generated themes for helix and attempt to set them. They do nothing. Go to the very first commit and use the files that were generated there and notice that they work.
Expected Behavior
The themes are applied.


Issue #SB-23367 course filter issue

Merging this PR is a hotfix, and CircleCI is failing because of resource crunch.


Identity - set up persistent store for grants

The current set-up for Identity Server is using an in-memory grant store as per the output below:
info: IdentityServer4.Startup[0]
      You are using the in-memory version of the persisted grant store. This will store consent decisions, authorization codes, refresh and reference tokens in memory only. If you are using any of those features in production, you want to switch to a different store implementation.

We should set up a persistent store for this!

To elaborate on this, we should extend the IdentityContext with the required entities for persisting grant information, and then create an EntityFrameworkGrantStore or similar to plug in to Identity Server


About batch size

Thank you for your excellent research!
I tried your fine-tuning code using A-100 4GPU. However, the maximum batch size I can use is 8, as using a larger batch size results in a CUDA memory error.
According to the paper, you can increase the batch size to 24 with V100 4GPU.
I didn’t change your code, but I just want to confirm the batch size you used for fine-tuning.

I noticed it's written as parser.add_argument('--batch-size', type=int, default=2, help='Batch Size for the model'), but looking at the code, it seems to specify the batch size per GPU.
Sorry for asking before taking a closer look at the code.


Markdown preview loses scrollbar on edit

Steps to Reproduce:

Open a Markdown file, long enough to have a scrollbar in its preview
Open preview to the side
Edit the file

The preview loses the scrollbar once it refreshes due to the file change event.

I believe that this was fixed by #19394, which worked around an upstream chrome issue
Please try VSCode 1.9 and let me know if you still see this
Thanks


Adding webpack for bundling code

This branch is used to add webpack to the system, used for e.g. compiling sass and js files with babel.

[x] Dev command #60
[x] Compile and load js on home page #61
[x] Webpack entry point per page #62
[x] Webpack require css #63
[x] Output compiled css in separate public folder #64
[x] Move from public to static folder #68
[x] Add twitter bootstrap using webpack #66
[x] Add font awesome using webpack #67
[x] Selective inclusion font awesome
[x] Selective inclusion font bootstrap
[x] Investigate removal jquery (bootstrap dependency?)
[x] Investigate selective embedding of bootstrap (who needs a carousel?)
[ ] Move to webpack dev server #65
[ ] Deploy webpack version #69

There remains one final glitch with the font-awesome setup. Setting a custom path allows for using a separate font folder, but does not result in extracting the actual fonts from font-awesome. Therefore on a system without a font folder containing the font-awesome fonts, first a build has to be done without the customised config.


Move CSS string from  to

There was a related question on the sphinx-users mailing list, but no solution:
https://groups.google.com/forum/?nomobile=true#!searchin/sphinx-users/css/sphinx-users/iG0fB9-JdNM/Le-rz_zbRHUJ
Also, the CSS string should only be included only in the pages which actually need it.

This can probably done with the extrahead block in layout.html, but it would be nice to be able to do that programmatically (without adding a separate template file).

Since cdfd4f789b63c2817ba627eb42d26ac658f288ec, the CSS is only included for HTML files generated from notebooks (except if Sphinx version < 1.3.6 is used).
The CSS could also be added to context['metatags'] instead of context['body'], but then it would end up very close to the top of the HTML file, which isn't too great either.
And I guess it is better to define the nbsphinx CSS after other CSS files are included ...

I just rebased the nbinclude stuff to master and cdfd4f7 kind of breaks it, in that the CSS doesn't get loaded so the included notebook looks bad. The rst.rst file in general doesn't work, as you can't manually use the nbinput or nboutput directives anymore without the CSS.

Oh, sorry, I completely missed that!
I created a different solution in #37, can you please try if that works for you?
I'm not sure if that's a proper way to do that (see my doubts in #37), probably you know some improvements?


Support for Base16-gruvbox

Base16 has gruvbox as one of its colorschemes: base16-gruvbox-dark-hard, for example
http://chriskempson.com/projects/base16/

if anybody creates a PR, I'll happily include it. Closing.


Unable to set price for 'Treasure' Items

Since updating to 4.10.0, all 'Treasure' type Items cannot hold a price value. All other Item types are able to be given a price. Whenever I give a 'Treasure' Item a price, the box goes blank. This applies to new Items and old ones from before the update. I have tried diasbling all mods and the issue persists.

I can reproduce. Additionally, editing an existing treasure item with a value resets it to 0.

Try updating a bit more 🙂 It was fixed in an follow-up release.


Fails to return database tables with Realm 2.0 when using named databases

PR to fix issue #37 (Fails to return database tables with Realm 2.0)

Replaced DatabaseId's getPath with getName - as Realm throws a RealmFileError if you try to open the file path.
Added printStackTrace to exceptions (for better debugging by users)

Even if you make 2 fields point to same value, you really shouldn't change other instances pointing to that field, like you did in lines 164-169, Other than that, this seems to be like a reasonable partial fix, even if it imposes a limitation of single unnamed table.

@Dragas Sorry, I think there was a misunderstanding in my explanation.
If you change NO code, then you can get the current plugin to work by just using default named Realm instances (e.g. default.realm). That works today, with no code changes to the plugin.
If you want to support custom named Realms and/or multiple Realm instances (like I have in my applications) - then you can use my PR to modify the plugin to support that use case. Please see the attached image and note that I have TWO uniquely named Realm databases, each with multiple tables.

Also, as for the code changes to line 164-169 - all I did was re-factor the existing code to support the new databaseId. Before, it took a path, created a File from the path, and then used the File to get the name of the Realm database. Now, we're always just using the name of the database, so the additional code was not needed.

+1

@sureshjoshi Thanks you for sending a PR.
I'll review this.

Sorry to ask but when can we expect this to merge ya?

@sureshjoshi i've tried to build an aar base on your changes but I'm getting "Detached from the target" from the developers tools and in the Android Studio log I saw W/stetho: I/O error: java.io.IOException: Broken pipe. Hope you can help on this.

@bryanleesh That usually means an exception was thrown inside Stetho. I currently use this everyday, but haven't seen this issue.
One potential theory. I know there is/was an issue with instant-run and using Stetho in certain cases (https://github.com/facebook/stetho/issues/447) - could it be this?
This is only when you build from source. If you have a jar or aar, then you're good to go. I know you mentioned an aar, but I wasn't sure how you were doing it.

@zaki50 Any progress on this?

Is this going to be merged anytime soon?

Thank you so much to make a PR to fix the bug.
I fixed the bug a bit simpler way, but this PR helps a lot to locate the cause.
I've release 2.1.0 that contains the fix.