Restructure and reformat codebase and add unit test files

Restructure the codebase while taking insights from golang-standards/project-layout
Add unit test file, test the project using go test -v ./..

I think you have to resolve some conflicts because of the changes merged by a previous PR.

@rehrumesh @rhperera Please review.


Fix bug when database ref 'pages' is empty

The first time I browsed to the 'pages' page in the admin, I was greeted with the error "TypeError: Cannot read property 'name' of null"

Thank you for contribution.


expand-all-hidden-comments doesn't work on super-long issues

I'm attempting to expand all 671 comments found in this issue: https://github.com/Koenkk/zigbee2mqtt/issues/1429
I have the Refined Github extension v20.3.16 added to Google Chrome v80.0.3987.149 on MacOS 10.15.3, and I have added a personal token within Options.
When I Option+Click the "Load More..." link, it very quickly disappears and just shows the first of the latest set of comments from 8 days ago. e.g.

After trying Opt+Click:

This is the related pull request: https://github.com/sindresorhus/refined-github/pull/2754
TIA

Did some debugging. Looks like if you do more than 200, it loads 60 and hides the button

I didn't check the limit, but yes, alt-clicking that "Load 672 more..." button returns a 404 instantly.
To be fair, this feature is rather hostile to GitHub because it forces them to render huge pages.
What we can do, if easy, is to limit it to ~200 items, so loading the whole conversation would require 4 clicks instead of the regular 12

Anybody can write user.js to auto click in link under "X hidden items" in  light loop for browser?


streamline onboarding experience

:books: Linear Ticket :books:
#LIQ-768

What is the correct behaviour of the close (x) button?

It seems like it brings you to next step.

What is the correct behaviour of the close (x) button?

It seems like it brings you to next step.

Fixed now.

@skeremidchiev @bradleySuira, please review


How ignore some directories

How could I ignore some directories which do not need test when using busted?

Use tags!
Also, you can use --filter / --filter-out to do this. try running busted --help for more info.

thank you


Activated CUDA?

hello,
I build BART successfully with CUDA:
CUDA?= 1
CUDA_BASE ?= /usr/local/cuda/

Then I run bart ecalib. It works, however, I cann't see the process via nvidia-smi.
So I am wandering if bart use CUDA or not.
thx.

You need to use the '-g' option to use the GPU.
'bart ecalib -g'
For ecalib, only the some part of the processing then uses the GPU. It may not always lead
to a speed up. It makes more sense for 'pics', which can be a lot faster on the GPU.
'bart pics -g'
Please direct future questions to the mailing list. Github issues are not the right place for this.


On spec files changes unit test are not running/reloading

Continues code developing process is not effective. On spec changes need to stop server and start over. No feat back from unit test on refactoring/changing code.
Issue Overview
On code/template changes or spec changes unit test are not "reloader"
Issue Description

On spec.js changes unit test are not reloading.
On .html changes unit test are not reloading.
On code .ts changes unit test are not reloading.
Reproducables
gulp
Information

Operating System
Windows/OSX/Ubuntu/orYourOSHere

Node version
4.x/5.x/orYourNodeVersionHere

NPM Version
2.x/3.x/orYourNPMVersionHere

Environment
Browser/Mobile/WebWorker

try
gulp test -w

Is it included in documentation?

https://github.com/antonybudianto/angular2-starter/wiki/Gulp-Tasks#test
also in
https://github.com/antonybudianto/angular2-starter/releases

Browser sync is not active after this command.

Server is not started.

what are the error traces?

after gulp test -w
http://localhost:3000/
http://prntscr.com/bfquzn

maybe it's because there's already a watch in test -w, but you run serve-dev again which contains watches (sass, ts)

looks like you're not supposed to run gulp test -w and gulp serve-dev together

So in case of gulp test -w and gulp serve-dev running in the same time http://screencast.com/t/UClqJWTsVD
http://screencast.com/t/M8NJZDuFju

How I can run server and unit test together?

Or single is it gulp task for this case?

I don't think it's possible now

Thus it remains a proposal for the future.

Of course, any help will be nice

Thank You.

actually you can run gulp serve-dev / gulp first, then do gulp test -w and no errors raised. doing it vice versa will cause error because gulp serve-dev will cleanup all things and the test watch will be triggered because of that


TGP-1430: Longer Comcode Bugfixes

Bugs were: Longer comcode not populating after coming from CYA, CYA incorrectly populating longer comcode even if it hadn't been answered. Please let me know if you spot any bugs :)
What I did to manually test:
Create record  (Angola for country, 17490 for commode)
“Categorise goods now”
Complete ALL assessments
CYA page shows with no longer commodity code, as expected.
Click change on 3rd assessment
Select no or none of the above
Longer comcode page shows
Enter a longer one (9900 worked for me)
Continue journey
CYA page shows longer comcode, as expected.
Click back until getting to longer comcode page
Longer comcode page now shows previous input, as expected.
Click continue, and redirects to CYA, as expected.
Click change for longer comcode
Longer comcode page still shows previous input, as expected.
I think the bugs have been fixed...

Can we add a test. I'm just worried that we accidentally break this again because so much is changing in this area constantly!

Can we add a test. I'm just worried that we accidentally break this again because so much is changing in this area constantly!

I'll take a look now! I'm hoping it's not too much trouble


Fixed yii\db\QueryBuilder to inherit ‘INSERT INTO ... SELECT’ SELECT’s params (fixes #13807)

Q
A

Is bugfix?
yes

New feature?
no

Breaks BC?
yes

Tests pass?
yes

Fixed issues
#13807

This PR fixes QueryBuilder did not account params in a subquery when building INSERT INTO … SELECT introduced in #9420. I didn’t mention a small BC break in UPGRADE.md yet.

Merged. Thank you!


Fix wrong repository to push binaries by asset-publisher

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

/kapetanios reload


Is there a missing sqrt in the adadelta calculations?

I was wondering if there is a missing sqrt call on this line https://github.com/karpathy/convnetjs/blob/master/src/convnet_trainers.js#L117
According to this article (links to adadelta paper)http://sebastianruder.com/optimizing-gradient-descent/index.html#adadelta it would suggest that:
var dx = - Math.sqrt((xsumi[j] + this.eps)/(gsumi[j] + this.eps)) * gij;

should be:
var dx = - Math.sqrt((xsumi[j] + this.eps)/Math.sqrt((gsumi[j] + this.eps))) * gij;

Would be great if someone could correct my understanding if not.

Ah looks like I miss read the parenthesis! Looks fine.


0.6.0 release

Codecov Report

Merging #1486 (9c116c0) into develop (999afa0) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #1486   +/-   ##
========================================
  Coverage    58.89%   58.89%
========================================
  Files           91       91
  Lines         4968     4968
  Branches       837      837
========================================
  Hits          2926     2926
  Misses        1991     1991
  Partials        51       51

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Make --python a global argument

Summary
This PR makes --python and --system global arguments, so that they can be passed anywhere.
Closes https://github.com/astral-sh/uv/issues/3704.

Ok, this leads to a breaking change because pip compile uses -p for --python-version.

Isn't that okay because --python-version is basically a subset of --python? I guess the discovered interpreter could have different markers?

--python-version works even if we can't discover an interpreter with the version, I think?

Example:
❯ echo "flask" | uv pip compile -p 3.8 -
warning: The requested Python version 3.8 is not available; 3.11.8 will be used to build dependencies instead.
Resolved 9 packages in 501ms
# This file was autogenerated by uv via the following command:
#    uv pip compile -p 3.8 -
blinker==1.8.2
    # via flask
click==8.1.7
    # via flask
flask==3.0.3
importlib-metadata==7.1.0
    # via flask
itsdangerous==2.2.0
    # via flask
jinja2==3.1.4
    # via flask
markupsafe==2.1.5
    # via
    #   jinja2
    #   werkzeug
werkzeug==3.0.3
    # via flask
zipp==3.19.0
    # via importlib-metadata

I think we should change it to require --python-version instead of -p but it is breaking.

@zanieb -- Am I right that we no longer want to do this, because we want separate documentation for --python for each command?

I think so? Though we can always just shadow it in a specific command to overwrite the docs right?


Bump pyyaml from 5.3.1 to 5.4

Bumps pyyaml from 5.3.1 to 5.4.

Release notes
Changelog
Commits

Signed-off-by: dependabot[bot]<EMAIL_ADDRESS>
https://github.com/Shivansh-007/gurkbot/security/dependabot/Pipfile/PyYAML/open

Have you tested this locally?

Yes, this has been tested locally @gustavwilliam

Alright, perfect. I'd love to have a second person test this before merging. Currently, I'm not able to do that.


Fetch certs not working

I have deployed kubeseal-webgui with helm chart with autofetch to true in the values.yaml.
But I got this error.
  Warning  Failed     16s (x3 over 36s)  kubelet            Error: failed to create containerd task: failed to create shim: OCI runtime create failed: container_linux.go:380: starting container process caused: exec: "/kubeseal-webgui/bin/kubeseal-fetch.sh": permission denied: unknown

I will try to recreate the issue and try to fix it.

@littlechicks Bug will be available in the next Release.


Opaque device Near clip plane setting defaults to 0.1 at run time

Overview
Opaque device Near clip plane setting defaults to 0.1 at inspector and run time.
Expected Behavior
Default value should be 0.3 in inspector and run time.
Actual Behavior
Steps to reproduce
Select MixedRealityCameraParent --> MixedRealityCamera, the value is 0.1.
Open script in VS and the value is 0.3.
At run time print CameraCache.Main.nearClipPlane and still says 0.1.
Unity Editor Version
MRTP3 & MRTP4
Mixed Reality Toolkit Release Version
842c4a1

0.1 should be fine for immersive devices. Is it not?
If anything, the bug here is that the script needs to be updated to default to 0.1 instead of 0.3. The prefabs have already been updated.

Yeah Kev definitely a bug where it's probably not updating the CameraCache value for some reason.

I think there might be some confusion here based on prefab values vs script defaults.
Previously, the default on both was set to 0.3. The prefabs were updated recently, but the scripts weren't.
Changes made in code, but overwritten in the Inspector on a prefab, do not change the behavior. This would only change the default if the script was added to a new GameObject.
I just validated that the CameraCache is correctly getting updated to whichever value is in the Inspector.

Could you also include the changes from #1309?
I'll open a PR on your branch


Feature allow alpha, beta, & RC

PR with tests for #599

Why delete .pharcc.yml?

Please reset phpbrew phar file from this PR since it will be compiled when rolling out new release

If you want to remove pharcc, please open another PR to remove. PRs should be simple and contain only one subject

I deleted .pharcc.yml because CLIFramework 2.8 (which fixes the bug with building the phar) and pharcc aren't compatible with each other because they require conflicting versions of symfony.yaml.  I will open a new PR to fix #561 which deletes that old stuff.

Thanks!


Madbernardpatch 2

Why: Explanations with Graphical view or gif can be more effective.
Closes [issue link]

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[x] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[x] This pull request impacts the contribution experience

[x] I have added the 'writer impact' label
[x] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

Please add a graphical way to initialize the code and proper use of docs


[🐛 Bug]: Failed to generate worker when using Route Handlers with App Router

next-on-pages environment related information
System:
        Platform: linux
        Arch: x64
        Version: #1 SMP PREEMPT_DYNAMIC Wed, 26 Apr 2023 20:50:14 +0000
        CPU: (4) x64 AMD Ryzen 5 PRO 3400GE w/ Radeon Vega Graphics
        Memory: 8 GB
        Shell: /usr/bin/fish
Binaries:
        Node: 19.9.0
        Yarn: N/A
        npm: 9.6.6
        pnpm: N/A
Package Manager Used: npm
Relevant Packages:
        @cloudflare/next-on-pages: 0.10.1
        vercel: 28.20.0
        next: 13.4.0

Description
I tried to use Next.js App Router with Cloudflare Pages Functions and the build failed when using Route Handlers.
No errors occurred on normal pages only.
 ❯ npx @cloudflare/next-on-pages
⚡ @cloudflare/next-on-pages CLI v.0.10.1
⚡ Detected Package Manager: npm
⚡ Preparing project...
⚡ Project is ready
⚡ Building project...
▲  Vercel CLI 28.20.0
▲  WARNING: You should not upload the `.next` directory.
▲  Installing dependencies...
▲  up to date in 2s
▲
▲  275 packages are looking for funding
▲  run `npm fund` for details
▲  Detected Next.js version: 13.4.0
▲  Detected `package-lock.json` generated by npm 7+...
▲  Running "npm run build"
▲  > build
▲  > next build
▲  warn  - You have enabled experimental feature (typedRoutes) in next.config.js.
▲  warn  - Experimental features are not covered by semver, and may cause unexpected or broken application behavior. Use at your own risk.
▲
▲  info  - Creating an optimized production build...
▲  info  - Compiled successfully
▲  info  - Linting and checking validity of types...
▲  info  - Collecting page data...
▲  info  - Generating static pages (0/3)
▲  info  - Generating static pages (3/3)
▲  info  - Finalizing page optimization...
▲
▲  Route (app)                                Size     First Load JS
▲  ┌ ○ /                                      141 B          76.9 kB
▲  └ ℇ /api                                   0 B                0 B
▲  + First Load JS shared by all              76.8 kB
▲  ├ chunks/139-ab66f6d0e92fcf6c.js         24.4 kB
▲  ├ chunks/2443530c-9d09dde9f906fe30.js    50.5 kB
▲  ├ chunks/main-app-a5aa3221263f4429.js    215 B
▲  └ chunks/webpack-8074fabf81ca3fbd.js     1.61 kB
▲
▲  Route (pages)                              Size     First Load JS
▲  ─ ○ /404                                   178 B            83 kB
▲  + First Load JS shared by all              82.8 kB
▲  ├ chunks/main-b1e0eba8da4ad049.js        81 kB
▲  ├ chunks/pages/_app-c544d6df833bfd4a.js  192 B
▲  └ chunks/webpack-8074fabf81ca3fbd.js     1.61 kB
▲  ℇ  (Streaming)  server-side renders with streaming (uses React 18 SSR streaming or Server Components)
▲  ○  (Static)     automatically rendered as static HTML (uses no initial props)
▲  Traced Next.js server files in: 3.157s
▲  Created all serverless functions in: 2.294s
▲  Collected static files (public/, static/, .next/static): 5.275ms
▲  Build Completed in .vercel/output [18s]
⚡ Completed `npx vercel build`.
✘ [ERROR] Could not resolve "/wasm/wasm_a7e702bc5ba9227243abda7977c8096f59478d3f.wasm"

    ../../../../../../tmp/xjnkms1k1i/api.func.js:2:64:
      2 │   const wasm_a7e702bc5ba9227243abda7977c8096f59478d3f = require("/wasm/wasm_a7e702bc5ba9227243abda7977c8096f59478d3f.wasm");
        ╵                                                                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

✘ [ERROR] Could not resolve "/wasm/wasm_ef4866ecae192fd87727067cf2c0c0cf9fb8b020.wasm"

    ../../../../../../tmp/xjnkms1k1i/api.func.js:3:62:
      3 │ const wasm_ef4866ecae192fd87727067cf2c0c0cf9fb8b020 = require("/wasm/wasm_ef4866ecae192fd87727067cf2c0c0cf9fb8b020.wasm");
        ╵                                                               ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

/home/rokoucha/Projects/github.com/rokoucha/cloudflare-pages-function-next-js/node_modules/esbuild/lib/main.js:1575
  let error = new Error(`${text}${summary}`);
              ^

Error: Build failed with 2 errors:
../../../../../../tmp/xjnkms1k1i/api.func.js:2:64: ERROR: Could not resolve "/wasm/wasm_a7e702bc5ba9227243abda7977c8096f59478d3f.wasm"
../../../../../../tmp/xjnkms1k1i/api.func.js:3:62: ERROR: Could not resolve "/wasm/wasm_ef4866ecae192fd87727067cf2c0c0cf9fb8b020.wasm"
    at failureErrorWithLog (/home/rokoucha/Projects/github.com/rokoucha/cloudflare-pages-function-next-js/node_modules/esbuild/lib/main.js:1575:15)
    at /home/rokoucha/Projects/github.com/rokoucha/cloudflare-pages-function-next-js/node_modules/esbuild/lib/main.js:1033:28
    at runOnEndCallbacks (/home/rokoucha/Projects/github.com/rokoucha/cloudflare-pages-function-next-js/node_modules/esbuild/lib/main.js:1447:61)
    at buildResponseToResult (/home/rokoucha/Projects/github.com/rokoucha/cloudflare-pages-function-next-js/node_modules/esbuild/lib/main.js:1031:7)
    at /home/rokoucha/Projects/github.com/rokoucha/cloudflare-pages-function-next-js/node_modules/esbuild/lib/main.js:1143:14
    at responseCallbacks.<computed> (/home/rokoucha/Projects/github.com/rokoucha/cloudflare-pages-function-next-js/node_modules/esbuild/lib/main.js:680:9)
    at handleIncomingPacket (/home/rokoucha/Projects/github.com/rokoucha/cloudflare-pages-function-next-js/node_modules/esbuild/lib/main.js:735:9)
    at Socket.readFromStdout (/home/rokoucha/Projects/github.com/rokoucha/cloudflare-pages-function-next-js/node_modules/esbuild/lib/main.js:656:7)
    at Socket.emit (node:events:513:28)
    at addChunk (node:internal/streams/readable:324:12) {
  errors: [
    {
      detail: undefined,
      id: '',
      location: {
        column: 64,
        file: '../../../../../../tmp/xjnkms1k1i/api.func.js',
        length: 58,
        line: 2,
        lineText: '  const wasm_a7e702bc5ba9227243abda7977c8096f59478d3f = require("/wasm/wasm_a7e702bc5ba9227243abda7977c8096f59478d3f.wasm");',
        namespace: '',
        suggestion: ''
      },
      notes: [],
      pluginName: '',
      text: 'Could not resolve "/wasm/wasm_a7e702bc5ba9227243abda7977c8096f59478d3f.wasm"'
    },
    {
      detail: undefined,
      id: '',
      location: {
        column: 62,
        file: '../../../../../../tmp/xjnkms1k1i/api.func.js',
        length: 58,
        line: 3,
        lineText: 'const wasm_ef4866ecae192fd87727067cf2c0c0cf9fb8b020 = require("/wasm/wasm_ef4866ecae192fd87727067cf2c0c0cf9fb8b020.wasm");',
        namespace: '',
        suggestion: ''
      },
      notes: [],
      pluginName: '',
      text: 'Could not resolve "/wasm/wasm_ef4866ecae192fd87727067cf2c0c0cf9fb8b020.wasm"'
    }
  ],
  warnings: []
}

Node.js v19.9.0

Reproduction
https://github.com/rokoucha/cloudflare-pages-function-next-js
Pages Deployment Method
None
Pages Deployment ID
No response
Additional Information
No response
Would you like to help?

[ ] Would you like to help fixing this bug?

Hi @rokoucha thanks a lot for the issue (and especially the reproduction, really appreciated! 😄)
Seems like the problem here is that your app is trying to import wasm files which is a known issue: https://github.com/cloudflare/next-on-pages/issues/202
Anyways I pulled down your repo and tried running the app locally, it seems like it worked without issues for me:

However the app there doesn't contain the /api directory which seems likely to be what causes the issue (and I can see the dir in your logs), did you forget to push that?

@dario-piotrowicz I am so sorry, I just pushed it!

no problem it happens 😄👍

Seems like importing from node/server.js is bringing more things in than necessary (specifically next image code that requires wasm), you can fix this by changing node/server.js to node/server, and to make typescript understand the import you need to update moduleResolution to just node:

I think that moduleResolution set to node is the default? did you tweak that?

yeah I looked at that too but that imports looked very ugly to me 😅
(out of curiosity, why do you need node16 there? 🤔)
Anyways if you're ok with it I think we can close the issue since the extra importing is unrelated to next-on-pages and the wasm importing bug is already covered by other issues, would that be ok? 🙂

(out of curiosity, why do you need node16 there? 🤔)
That's my hobby 😁

Anyways if you're ok with it I think we can close the issue since the extra importing is unrelated to next-on-pages and the wasm importing bug is already covered by other issues, would that be ok? 🙂

Yes, it is not next-on-pages bug, I will close this issue.
Thank you for your help! and Thanks for the great software!

Awesome! my pleasure 🙂
thank you for using next-on-pages! ❤


Exception occurs when inserting nested Maps into document

When attempting to insert a more complex nested map into a document the following exception occurs:
System.Runtime.InteropServices.SEHException : External component has thrown an exception.
   at YDotNet.Native.Types.Maps.MapChannel.Insert(IntPtr map, IntPtr transaction, IntPtr key, IntPtr value)
   at YDotNet.Document.Types.Maps.Map.Insert(Transaction transaction, String key, Input input) in X:\ydotnet\YDotNet\Document\Types\Maps\Map.cs:line 51
   at YDotNet.Tests.Unit.Maps.InsertTests.InsertNestedMap() in X:\ydotnet\Tests\YDotNet.Tests.Unit\Maps\InsertTests.cs:line 403
   at System.RuntimeMethodHandle.InvokeMethod(Object target, Void** arguments, Signature sig, Boolean isConstructor)
   at System.Reflection.MethodInvoker.Invoke(Object obj, IntPtr* args, BindingFlags invokeAttr)

The following test case can be used to reproduce the issue:
[Test]
public void InsertNestedMap()
{
    // Arrange
    var doc = new Doc();
    var map = doc.Map("map");

    // Act
    var transaction = doc.WriteTransaction();
    var innerMap = Input.Map(new Dictionary<string, Input>
    {
        { "text", Input.String("Nested data") }
    });
    var outerMap = Input.Map(new Dictionary<string, Input>
    {
        { "innerMap", innerMap }
    });
    map.Insert(transaction, "outerMap", outerMap);
    var length = map.Length(transaction);
    transaction.Commit();

    // Assert
    Assert.That(length, Is.EqualTo(expected: 1));
}

In case it makes a difference I'm using the YDotNet.Native.Win32 native library.
Please do let me know if this is not the best approach to creating nested data structures as I'm relatively new to this library.
Thanks

Thanks a lot. This does not seem to be a problem with YDotNet, but with y-crdt, the rustl library. If you have experience with rust, it would be great if you could reproduce it there.

You probably won't reproduce it directly in rust (we have a lot of tests to confirm that working on nested collections work). We could try to check it out at the yffi level (C FFI bindings to Rust). Sometimes issues happen on that level.

Thanks for feedback @Horusiath . I would wait for this PR, because it changes a lot anyway: https://github.com/y-crdt/ydotnet/pull/94

Sorry for the slow update, I'm new to Rust and it's been quite a few years since I've done any c++ so there may be some mistakes in the code snippets below.

I've tested the original C# test case on the #94 branch and that fails there too.
I've created the following test case in y-crdt which passes

  #[test]
  fn map_nested() {
      let d1 = Doc::with_client_id(1);
      let m1 = d1.get_or_insert_map("map");
      let mut t1 = d1.transact_mut();

      let mut innerMap: HashMap<String, _> = HashMap::new();
      innerMap.insert("text".to_owned(), "Nested data");

      let mut outerMap: HashMap<String, _> = HashMap::new();
      outerMap.insert("innerMap".to_owned(), MapPrelim::from(innerMap));

      m1.insert(&mut t1, "outerMap".to_owned(), MapPrelim::from(outerMap));

      assert_eq!(m1.len(&t1), 1);
  }

The following test case in yffi fails

TEST_CASE("YMap multiple nested maps") {
    YDoc* doc = ydoc_new_with_id(1);
    Branch* map = ymap(doc, "map");
    YTransaction* txn = ydoc_write_transaction(doc, 0, NULL);

    char* key = (char*)"text";
    YInput value = yinput_string("Nested data");
    YInput innerMap = yinput_ymap(&key, &value, 1);

    char* key2 = (char*)"innerMap";
    YInput outerMap = yinput_ymap(&key2, &innerMap, 1);

    ymap_insert(map, txn, "outerMap", &outerMap);
    int length = ymap_len(map, txn);
    ytransaction_commit(txn);
    REQUIRE(length == 1);

    ydoc_destroy(doc);
}

thread '<unnamed>' panicked at yffi\src\lib.rs:2488:17:
Unrecognized YVal value tag.

Simplifying the nesting in yffi will allow the test to pass

TEST_CASE("YMap nested map") {
    YDoc* doc = ydoc_new_with_id(1);
    Branch* map = ymap(doc, "map");
    YTransaction* txn = ydoc_write_transaction(doc, 0, NULL);

    char* key = (char*)"text";
    YInput value = yinput_string("Nested data");
    YInput innerMap = yinput_ymap(&key, &value, 1);

    ymap_insert(map, txn, "innerMap", &innerMap);
    int length = ymap_len(map, txn);
    ytransaction_commit(txn);
    REQUIRE(length == 1);

    ydoc_destroy(doc);
}

I hope that is useful let me know if there is any more I can do to help.

@LSViana It seems that it has been solved. I can work on that.


Weapon Section Rework

Submission Checklist
Required

[ ] The pull request title clearly contains the name of the sheet I am editing.
[ ] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[ ] The pull request makes changes to files in only one sub-folder.
[ ] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: <   >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: <   > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <   > (i.e. Dungeons & Dragons, FATE)

[ ] I have followed the Character Sheets Standards when building this sheet.

[ ] This sheet has been made on behalf of, or by, the game's publisher.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description
The Weapons section was reworked to accommodate a nicer display and the recent changes.

Character Sheet Info Roll20 Internal Use only.


Move Non-Instance-Dependent Logic into Editor API

This should be an easy win, and then allows us a better look at how to port the remaining instance-dependent code to the API later.

Resolved in build 100.


Download or extract video only

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2016.05.16. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2016.05.16

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[x] Question
[ ] Other

Description of your issue, suggested solution and other information
Hi,
I would like to know if it is possible to only get the video (without sound) of any type of video? (Not just DASH Youtube)
Or must I use ffmpeg in addition to youtube-dl to extract the video?
Thanks!

When no bestvideo format present, yes you should use ffmpeg.


[CP] Do not assume that pub is the first command run by "flutter create" #114621

issue_link
https://github.com/flutter/flutter/pull/114621
Commit Hash
https://github.com/flutter/flutter/commit/a440c4689fd600a450f0fca0b42c4b76fcfff4ff
Target
stable
pr_link
https://github.com/flutter/flutter/pull/114945
Impacted Users
Mac tool tests failing
Impact Description
Impacts ability to verify Mac tool tests has no regressions
Workaround
None
Risk
low
Test Coverage
yes
Validation Steps
This is a test only change.

This is a test only change, and should be approved to unblock the latest release.

Released in 3.3.8.


Update Rubocop to be an executable

What does this implement/fix? Explain your changes.

Updates Rubocop to be a formal Atom Beautify executable
Only passes cwd if a .rubocop.yml file is present
...

Does this close any currently open issues?
Closes #2066
...
Any other comments?
...
Checklist
Check all those that are applicable and complete.

[X] Merged with latest master branch
[X] Regenerate documentation with npm run docs
[ ] Add change details to CHANGELOG.md under "Next" section
[X] Added examples for testing to examples/ directory
[X] Travis CI passes (Mac support)
[X] AppVeyor passes (Windows support)

@bbugh @escobera I appreciate your review of this PR since it is applicable to you.

Awesome, thank you for testing @escobera !

@Glavin001 covered my comments already. Looks great @szeck87, glad that #2066 was an easy fix for @escobera!

@escobera @bbugh could you please test once more (even if you have already)?  I made another change and want to make sure it still works for both of you.

Seems to be working. I tested it

with rubocop path specified in options
without rubocop path in options (empty, default)
no .rubocop.yml in tree structure with Indent Size specified
no .rubocop.yml in tree structure without Indent Size specified
.rubocop.yml in user home folder and current project folder
.rubocop.yml in user home folder but not current project folder

All seem to be working just fine. 👍

Thank you all for your input! I will post back here once the fix has been published.

Thanks for the thorough testing @bbugh.

Published to v0.32.2


dbfixup: per project instead of globally

Changes:

dbfixup: split into more functions, add comments
dbfixup: allow taking an external zero bit definition. External masks are not supported at this time
dbfixup: add --db_root arg
fuzzers: use new dbfixup flags to fixup local files more precisely instead of trying to apply globally
fuzzers: consolidate some build scripts, move products to build dir

Context:
dbfixup aggregates multi bit values after segmatch solves for them. Right now this is done globally after data is pushed into the global DB directory. This means that all of the fixups are smashed together and is particularly problematic as currently written because the same fixups are applied to all included tile types. This was fine when all the types needed were CLB like, but this is now conflicting with BRAM.
Rather than expand the current scheme, this restructures the code to work per fuzzer before pushing into the DB. Old interconnect behavior is preserved right now with --clb-int and called in all projects that relied on it

The following fuzzers were checked and seem to still run.
Checked:

001-part-yaml
005-tilegrid
010-lutinit
011-ffconfig
012-clbn5ffmux
013-clbncy0
014-ffsrcemux
015-clbnffmux
016-clbnoutmux
017-clbprecyinit
018-clbram
019-ndi1mux
025-bram-config
026-bram-data

We need to do a full DB run to verify this is fully working, but the tests I was able to run seems okay. I'm merging in the interest in moving things with BRAM forward


Code cleanup

There are a lot of modules used for experimentation that are no longer used.
Also dead code / templates / views / etc.
Needs a good ol' pruning

Fixed some dead scripts in https://github.com/inventree/InvenTree/pull/130

Ongoing, but closed for now


Delete the minSdk from AndroidManifest.xml.

Adapt Android P  API - 28
delete the minSdk from AndroidManifest.xml.

Fixed in another PR.


Adapters should insert timestamp in the case it is missing

FEATURE REQUEST

Is there an open issue addressing this request? If it does, please add a "+1" reaction to the
existing issue, otherwise proceed to step 2.

Describe the feature you are requesting, as well as the possible use case(s) for it.
SenML message can be sent without timestamp, because:

Low-power devices might not have RTC
Clocks are not synchronized via NTP
Some other reason

Protocol adapters - who are the first recepients of the message - should insert a timestamp in the case it is missing.
Even more - we should consider of an another fiels in Mainflux Message{} like received.
Writers will then use this timestamp, and this will help with implementing queue and workers in https://github.com/mainflux/mainflux/issues/1068 and assuring that messages are written in order, although processed by several parallel writers.

Indicate the importance of this feature to you (must-have, should-have, nice-to-have).
Must-have

Having small battery operated sensors without RTC is one of the reasons why I use custom adapters with which I enhance the received message or even repackage it as SenML.
Where are we going to get the time for the timestamps from? Time synchronization between all Mainflux services would be nice to have feature.
I guess simply reading the time from the adapter's container would be a good start. We can add a time provider service and time synchronization to all services later.
@mainflux/contributors wdyt?

@chombium I think as you mentioned - from an underlying Linux system. Containers should be deployed on a NTP synced machines (if even higher precision is needed PTP can be used with manchines that support it).
When you mentioned time provider service, you meant one API that will provide you a current timestamp? I think this will introduce unnecessary delay and incorrectness.

@drasko yes, I thought of one API with different backends for getting the correct time and provide it to the MF services, but I agree that that would introduce delays.
Time syncing on the OS level for containers and time syncing between pods in Kubernetes would be the most reliable thing to do. I guess this is another topic for the deployment guide ;)

I'll take a look at this one.

@chombium great, thank you! But it would be easier for you to wait for this one: https://github.com/mainflux/mainflux/pull/1080

@manuio there are some things to add to our docker compose files and I guess to the Helm chart to ensure the time sync between the services. The other part about the adapters I will try to solve it on a central place as some kind of "middleware" which will get the decoded message payload, check it and adjust it.

@chombium this is fantastic, but take care that change would have to be made practically in every adapter, as we need to change Message{} struct to add a timestamp field which adapters will set.
We do not have to do it at the same time in all adapters, but that is the final destination.

@drasko I completely agree. I will do it once and have to find the appropriate place to attach the coode

@drasko if it's done in the new generic  publisher no need to duplicate code in adapters

@manuio this is true. @chombium please follow the work here: https://github.com/mainflux/mainflux/pull/1080 on unification of adapter's publishing part. Should be merged soon.

@chombium The Publisher is ready here: https://github.com/mainflux/mainflux/blob/master/broker/nats.go

@chombium I added received timestamp in adapters.
You can continue with the synchronization here: https://github.com/mainflux/mainflux/issues/1107


After uploading to version 1.14.1 I can no longer used command prompt from the editor

VSCode Version: Code 1.14.1 (2648980a697a4c8fb5777dcfb2ab110cec8a2f58, 2017-07-13T19:18:47.188Z)
OS Version: Windows_NT ia32 10.0.15063
Extensions:

Extension
Author (truncated)
Version

html-snippets
abu
0.1.0

vscode-css-formatter
aes
0.0.2

ionic3-vs-ionView-snippets
dan
1.0.1

python
don
0.6.7

tslint
eg2
0.16.0

ionic2-vscode
jgw
1.0.0

docthis
joe
0.4.7

Angular2
joh
2.3.3

csharp
ms-
1.11.0

debugger-for-chrome
msj
3.1.6

debugger-for-edge
msj
0.4.1

ionic2-snippets
oud
1.0.0

vscode-docker
Pet
0.0.16

sass-indented
rob
1.4.1

code-spell-checker
str
1.2.1

nativescript
Tel
0.7.2

cordova-tools
vsm
1.2.7

Steps to Reproduce:

updated from the editor the Version 1.14.1 of VS code
Tried to use the terminal (PowerShell) from vscode with no luck. I restarted the computer, restarted the editor with no luck.
Forgot to add what happens is that it doesn't see any of the node related commands. Ex. node, gulp, ionic etc...
When starting powershell from the windows 10 start menu it works as it should.

Reproduces without extensions: Yes/No
Yes

Did you see any errors in the integrated terminal?

Only that it couldn't recognize node.
After deleting and reinstalling everything work as expected.
The issue was only when updating to that version of  code.

Sounds like a problem with the update. /cc @Tyriar

I'm glad the issue got resolved after reinstallation. Sorry about that.


KeyError when pyproject.toml dependencies are dynamically read from requirements.txt

Describe the bug
I usually have the following structure in my projects:
# pyproject.toml
[build-system]
requires = ["setuptools>=67.6.0"]
build-backend = "setuptools.build_meta"
[project]
dynamic = ["dependencies"]
# no `dependencies` key
[tool.setuptools.dynamic]
dependencies = { file = ["requirements.txt"] }

And then the projects requirements are listed in the requirements.txt file. This is useful because IDEs will usually syntax-highlight PEP 440 syntax in requirements files, but not in strings inside TOML tables.
When I run deptry . with such a project, it crashes with the following traceback:
Traceback (most recent call last):
  File ".../bin/deptry", line 8, in <module>
    sys.exit(deptry())
  File ".../python3.10/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
[...]
  File ".../python3.10/site-packages/deptry/core.py", line 63, in run
    dependencies_extract = self._get_dependencies(dependency_management_format)
  File ".../python3.10/site-packages/deptry/core.py", line 142, in _get_dependencies
    return PEP621DependencyGetter(self.config, self.package_module_name_map).get()
  File ".../python3.10/site-packages/deptry/dependency_getter/pep_621.py", line 41, in get
    dependencies = [*self._get_dependencies(), *itertools.chain(*self._get_optional_dependencies().values())]
  File ".../python3.10/site-packages/deptry/dependency_getter/pep_621.py", line 48, in _get_dependencies
    dependency_strings: list[str] = pyproject_data["project"]["dependencies"]
KeyError: 'dependencies'

To Reproduce
Using a pyproject.toml file as described above (with no dependencies key inside the top-level project table, but mentioning "dependencies" as a dynamic metadata), inside a Python 3.10 virtual environment, run:
pip install -e .
pip install deptry
deptry .

Expected behavior
Either:

Support for the the tool.setuptools.dynamic.dependencies.file idiom (see https://setuptools.pypa.io/en/latest/userguide/pyproject_config.html#dynamic-metadata); or
A clear error message saying that this idiom is not supported.

System:

OS: Ubuntu 20.04 inside WSL (Windows 10)
Language Version: Python 3.10.12
Poetry version: [not using poetry or any other packaging framework]

Interesting, didn't know that it was possible to do that with setuptools. This is possibly something we could implement.
Unfortunately, until this is implemented, there is no escape hatch to force deptry to use requirements.txt, even when using --requirements-txt requirements.txt, since it will still use PEP 621 metadata from pyproject.toml if a project key is found (see here and here). It might also be something we want to improve in the future.

Same issue here: pyproject.toml without project.dependencies key and with a requirements.txt leads to the stack trace posted above by @fofoni.
I'd say this still fits the use case stated as supported ("Projects that use a requirements.txt file according to the pip standards").

Hello, I am interested in working on this issue. Since I am new to project, I will appreciate if I can get a bit more guidance on what changes are supposed to be made and what is the expected behaviour?

We have the same issue for all our Python projects in our organisation.

There is ongoing work to support this in https://github.com/fpgmaas/deptry/pull/881, if anyone wants to try it out and report back on whether this works.


Add missing cfg for adc module

Hmm I assume this is out of scope for this crate, all other modules assume the commonly used components are enabled.


Copy edited error message

grammar fix: “unversioned” instead of “unversion”
edited for clarity

Thanks for the fix!

@swift-ci please smoke test

@swift-ci please smoke test

Looks like a test mentions the error message I changed? Want me to add that to the PR? (I confess I just made the edit straight off of GitHub & didn't even clone locally.)

Yes, the unit test would need to updated as part of the PR.  I didn't realize that this message was being mentioned in a test, so this is good catch by the smoke tests that every PR requires.
As for testing locally, the Documentation/Development.md document inside the repository contains information about how to run the tests locally.  In particular, I always do a Utilities/bootstrap test before putting up a PR (step 5).

@swift-ci please smoke test

@pcantrell Thanks again for fixing the diagnostic message here.  If the wording in the unit tests can be updated so that the tests pass, we can merge this.  Thanks!

Hi Mishal,
I noticed you closed this PR, could you provide background on why it was closed?
— Tom

On Oct 6, 2020, at 11:41 AM, Mishal Shah<EMAIL_ADDRESS>wrote:
Closed #2673 https://github.com/apple/swift-package-manager/pull/2673.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/apple/swift-package-manager/pull/2673#event-3847018011, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABD6L7QN7DPTXBWTADCTFTSJNQHDANCNFSM4MENWV7Q.

We moved our default branch to main and deleted master branch, GitHub automatically closed the PR. Please update the base branch to main.  More detail about the branch update - https://forums.swift.org/t/updating-branch-names/40412

Hi @pcantrell, the Swift project moved the default branch to main and deleted master branch, so GitHub automatically closed the PR. Please rebase on top of the main and we can reopen.
More detail about the branch update - https://forums.swift.org/t/updating-branch-names/40412


How about putting clone() into GeneratedMessage?

Protobuf library: Every generate class that extends GeneratedMessage has a clone method, how about putting clone() into GeneratedMessage?
////////////////////////////////////////////////////////////////////////////////////
Editor: 1.5.0.dev_04_02 (2014-06-06)
OS: Windows 8 - amd64 (6.2)
JVM: 1.7.0_51
projects: 5
open dart files: 2
auto-run pub: true
localhost resolves to: <IP_ADDRESS>
mem max/total/free: 1778 / 1209 / 552 MB
thread count: 36
index: 1063521 relationships in 180908 keys in 1616 sources
SDK installed: true
Dartium installed: true

Fixed a while ago.


Update free-programming-cheatsheets.md

What does this PR do?
Add resource(s)
For resources
Description
Added a vim sheet that have more than 15 language translations. Also backed by duckDuckGo and should be a alternative in your list.
Why is this valuable (or not)?
Easy to understand, and available in MANY languages.
How do we know it's really free?
You can read about how to contribute (and help improve) by viewing our README . There you can see how to set up this project, or how to contribute a new language. Here is a big thank you to our contributors!
This project is licensed under The MIT License (MIT) .
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

I have made the echanges and evyerhing looks OK.

Conflict fixed.

Conflict solved.

no changes


Investigate tooManyRequests Rollbar item

Started reporting a lot last night after we rolled out the alert change:
https://rollbar.com/finestructure/SPI-server/items/15/

Not good, we are in fact at 0 at the moment:
Thu 11 Jun 2020 07:31:30 CEST
rate.remaining: 0

Well, of course we're burning through our request budget at twice the rate...
🤦♂
The Linode box is running with the same API key.

I've stopped services there now. Our budget has reset 20 mins ago and I had stopped requests on the mini at that time. On the next deploy we'll go up again with just the mini, a pretty intect budget and a 10 min cool-off.
That should keep everything in the clear and we can roll back to 5 min cool-off later on if we want.

I'm not sure if notifications are working this morning, but if you get this email do you wanna have a quick chat about this?


[FIX] Alternate fix for autosizing issue for notifications.

@RocketChat/android

Closes #2220
Changes:
Changed dp sizing to sp for notifications TextView.
Screenshots:

The issue has already been addressed by me in PR #2289. (https://github.com/RocketChat/Rocket.Chat.Android/pull/2289)
@filipedelimabrito I'm curious which of my solutions is more suitable for the task(and why!).

@NBAMIR I think that this explanation made from @HusseinElFeky will help you understand why.


Error reporting

Theoretically, this is awaiting RedPRL/asai, but we can speed things up either way.


Exception in parse_guild_member_add

Summary
The library seems to run into an issue (randomly) where the guild's member count is None.
Reproduction Steps
NA
Minimal Reproducible Code
NA
Expected Results
NA
Actual Results
Traceback (most recent call last):
  File "bot.py", line 112, in <module>
    libertas_support.run()
  File "bot.py", line 91, in run
    super().run(config.TOKEN)
  File "/usr/local/lib/python3.8/dist-packages/disnake/client.py", line 834, in run
    return future.result()
  File "/usr/local/lib/python3.8/dist-packages/disnake/client.py", line 813, in runner
    await self.start(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/disnake/client.py", line 777, in start
    await self.connect(reconnect=reconnect)
  File "/usr/local/lib/python3.8/dist-packages/disnake/client.py", line 674, in connect
    await self.ws.poll_event()
  File "/usr/local/lib/python3.8/dist-packages/disnake/gateway.py", line 588, in poll_event
    await self.received_message(msg.data)
  File "/usr/local/lib/python3.8/dist-packages/disnake/gateway.py", line 538, in received_message
    func(data)
  File "/usr/local/lib/python3.8/dist-packages/disnake/state.py", line 1167, in parse_guild_member_add
    guild._member_count += 1
TypeError: unsupported operand type(s) for +=: 'NoneType' and 'int'

Intents
NA
System Information

Python v3.8.10-final
disnake v2.2.0-beta

disnake pkg_resources: v2.2.0

aiohttp v3.7.4.post0
system info: Linux 5.4.0-73-generic #82-Ubuntu SMP Wed Apr 14 17:39:42 UTC 2021

Checklist

[X] I have searched the open issues for duplicates.
[X] I have shown the entire traceback, if possible.
[X] I have removed my token from display, if visible.

Additional Context
No response

This was a known issue in the (short-lived) version 2.2.0, it should be fixed in later versions; does this still occur on the latest version?

I'm not sure. I'll update my bot instance.

@Kyzegs Any news on this issue?

As mentioned by @shiftinv, the issue got fixed in 2.2.0.


septum assembly primary name

In GO we call septation "septum assembly" but for  FYPO purposes can we use
"septation" in the term labels as that is the common usage?

Well, OK, although it will diverge from GO  ("septation" is a BROAD synonym for GO:0000917), which risks making the FYPO names a little less suited to the species-neutral cell-level phenotype ontology vision. And it's a PITA to do text edits in Protege. But yeah, I can do it.

No worries, maybe it is better as it is. People say "septation" but  thinking about it now this probably  usually  means "ring contraction+ septum formation" which occur at the same time. We tend to equate the use of in text  'septation' with 'septum assembly' but really it's the whole caboodle.
in 'cut' for example:
An inviable phenotype observed in the vegetative growth phase of the life cycle in which a cell undergoes septation despite abnormal mitotic nuclear division, such that the septum physically divides the nucleus into two parts, giving rise to inviable daughter cells.
But it's really ring constriction+septum assembly here.
I have no idea how to resolve this, so maybe we can quietly ignore until some logical issues arise...


Create a Makefile for the project.

Support make, and make install.
Don't forget that the project requires libmath!
Also, add the build instructions to the readme file.

Hi, if this issue is still under consideration, I'd like to take care of it.
Let me know as soon as possible.
Thanks


[FEATURE] Add support for sending additional metadata in the MessagingInterface

Feature Description
In the current implementation of message interface I can only send a Message & a timestamp associated with the message,
There is no accomodation to send additional metadata about the message which isn't part of the data but could be propogated to the downstream implementations,
These could be used downstream as kafka headers or redis hashes or partition keys.
Possible Implementation
metadata can be a simplistic HashMap<String, String> and can be either accomodated as a new param in the MessagingInterface or a new method on the Message Interface.
The implementers of MessageInterface can handle it as follows

Kafka
Send the metadata as headers
EventLogger
Log the metadata as a separate key along with the raw data

Have you spent some time checking if this feature request has been raised before?

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

Have you read the Contributing Guidelines?

[X] I have read the Contributing Guidelines

Are you willing to submit a PR?
None

I am interested in this

Hey @tanbirali thanks for your interest.
I'll assign this issue, you can ask for clarifications here or reach out on our slack/discord.

whenever I am running the docker-compose up -d all the services are working but the hyperswitch-hyperswitch-server-1 is exiting everytime starting the container, how can I fix this

Hey @tanbirali ,
Can you share the logs that are emitted on docker failure ?
you can get them via docker compose logs hyperswitch-server..
Also these are the steps for running hyperswitch locally via docker compose you can walk through them again if anything is amiss...
There were some similar issues reported and solved in the discord can check if your issue is present there

Hey @tanbirali ,
Are you still interested/working on this,
Is there any clarification/help you need about this issue.

I am working on it, will try to create a PR soon

Sorry I am not able to run the project locally, I tried having a convo in the discord community, something or the other is not working for me, sorry for delaying it, I am trying in the mean time if anyone is willing to take this issue, please assign it to the individual

Hey @tanbirali no worries, I'll keep the issue unassigned for now.


⚠ API (search) has degraded performance

In 58f8766, API (search) (https://api.dev.package-search.services.jetbrains.com/search-packages) experienced degraded performance:

HTTP code: 200
Response time: 8046 ms

Resolved: API (search) performance has improved in cbd74b8 after 44 minutes.


Audio not recorded on videos longer than 10 seconds.

Applying http://stackoverflow.com/questions/26735627/ios-8-ipad-avcapturemoviefileoutput-drops-loses-never-gets-audio-track-after fixes #49.

Thanks for this too!


General build issues on windows

I wanted to build the project on windows machine and faced a few issues:

when executing win_make.bat I got the issue related to the missing libintl3.dll and libiconv2.dll libraries
after fixing the issue with missing libraries I got further weird issues like:

process_begin: CreateProcess(NULL, rm -f firmware.bin firmware.packed.bin firmware start.o init.o, ...) failed.
make (e=2): The system cannot find the file specified.
make: *** [clean] Błąd 2

or
process_begin: CreateProcess(NULL, arm-none-eabi-gcc -c -mcpu=cortex-m0 start.S -o start.o, ...) failed.
make (e=2): The system cannot find the file specified.

or
c:\prj\dev-test\sources\various\uv-k5-firmware-custom>arm-none-eabi-gcc -c -mcpu=cortex-m0 start.S -o start.o
arm-none-eabi-gcc: fatal error: cannot execute 'cc1': CreateProcess: No such file or directory
compilation terminated.

The first issue I fixed with downloading the Dependency.zip libraries of make from https://gnuwin32.sourceforge.net/packages/make.htm web page, extracting it somewhere and adding its bin directory to the classpath. I didn't install the gnu_make-3.81.exe as its written in the main README.md; I just downloaded the zipped file and extracted it somewhere.
The second issue in my opinion is because of the quotation marks in the win_make.bat around the PATH variable. I removed them and the compilation works perfect now:
@set PATH=c:\prj\dev-test\tools\gcc-arm-none-eabi-10.3-2021.10\bin\;%PATH%
@set PATH=c:\prj\dev-test\tools\gcc-arm-none-eabi-10.3-2021.10\arm-none-eabi\bin\;%PATH%
@set PATH=c:\prj\dev-test\tools\make-3.81-bin\bin\;%PATH%
@set PATH=c:\prj\dev-test\tools\make-3.81-dep\bin\;%PATH%

oops
I've updated win_make.bat as per your message.

I put the quotation marks around the paths because often windows directories have spaces in them, mine do, and bat files and command lines don't like directories with spaces without quotation marks.

Thanks a lot for clarification. Indeed, since long time I avoid to put spaces in the directories paths, especially on windows (but also on linux).
Let's keep this issue as a quick help for a beginners in this project (like me :)), so they don't spend lots of time on finding out why the compilation with win_make.bat doesn't work.

Not putting spaces in your directory names is no solution at all, the solution is to fix whatever can't cope with spaces (it's bad programming/coding/practices not too allow for them) because spaces in names is fully part of the OS.


Improve Node.js index.js comments

Codecov Report

Merging #1614 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #1614      +/-   ##
============================================
+ Coverage     86.77%   86.77%   +<.01%
- Complexity     4168     4169       +1
============================================
  Files           408      408
  Lines         16711    16713       +2
  Branches       1797     1797
============================================
+ Hits          14501    14503       +2
  Misses         1715     1715
  Partials        495      495

Impacted Files
Coverage Δ
Complexity Δ

...transformer/nodejs/NodeJSPackageMetadataNamer.java
83.33% <100%> (+1.51%)
5 <1> (+1)
:arrow_up:

...nsformer/nodejs/NodeJSGapicSurfaceTransformer.java
99.51% <100%> (ø)
31 <8> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


Devspace 6 resolves dependency variables regardless of v1beta11 disabled flag

What happened?
Given a v1beta11 schema that utilizes dependency.disabled to enable or disable a dependency, the upgraded v2beta1 schema no longer has the disabled flag, and devspace will download all dependencies in order to resolve variables. This can cause issues in CI when the environment does not have credentials to perform git clone
What did you expect to happen instead?
Downloading the dependencies should be skipped
How can we reproduce the bug?
My devspace.yaml:
version: v1beta11
vars:
- name: DEP1_DISABLED
  default: false
deployments:
- name: nginx1
  helm:
    componentChart: true
    values:
      containers:
      - image: nginx
dependencies:
  - name: dep1
    disabled: ${DEP1_DISABLED}
    source:
      git: https://github.com/doesntexist/youcantcloneme

> devspace deploy --var DEP1_DISABLED=true
...
warn Error cloning or pulling git repository https://github.com/doesntexist/youcantcloneme: Error running 'git clone https://github.com/doesntexist/youcantcloneme /Users/russellcentanni/.devspace/dependencies/https-github-com-doesntexist-youcantcloneme --depth 1': error executing 'git clone https://github.com/doesntexist/youcantcloneme /Users/russellcentanni/.devspace/dependencies/https-github-com-doesntexist-youcantcloneme --depth 1':  -> Cloning into '/Users/russellcentanni/.devspace/dependencies/https-github-com-doesntexist-youcantcloneme'...
remote: Repository not found.
fatal: repository 'https://github.com/doesntexist/youcantcloneme/' not found

fatal deploy dependencies: resolve dependencies: loading config for dependency dep1: Couldn't load '/Users/russellcentanni/.devspace/dependencies/https-github-com-doesntexist-youcantcloneme/devspace.yaml': stat /Users/russellcentanni/.devspace/dependencies/https-github-com-doesntexist-youcantcloneme/devspace.yaml: no such file or directory

Local Environment:

DevSpace Version: 6.1.1
Operating System: mac
ARCH of the OS: ARM64
Kubernetes Cluster:
Cloud Provider: N/A
Kubernetes Version: v1.25.1

Anything else we need to know?
Using --skip-dependency dep1 is a workaround.
Users encountered a different error than the above example, but the root cause of resolving dependency variables when they should be disabled is the same.

/kind bug

@lizardruss thanks for the issue! Mhhh I guess we need to bring the flag back as its also useful for other use cases


Server side datasource doesn't work with get request

Describe the bug
Server side datasource doesn't work with get request.
To Reproduce
Steps to reproduce the behavior:

Make a get request by server side datasource table.
Data table doesn't load the requested data.

Expected behavior
Requested data to be loaded into the table.
Desktop (please complete the following information):

OS: Win10
Browser: Chrome

@RudolfDaniel: Server side datasource is designed for server-side paging scenarios (we'll need to update the documentation). A GET request can be good if it sends paging parameters in some way (query params, headers). If the response contains only a list of entities, client-side pagination handling would be more appropriate.


The "servers:edit" command cannot be found

I'm running the Docker inside Kubernetes and attaching to the console.  I was able to add configs and export and import based on the README but when i just run console or any help i see this error in the help screen and i am unable to actually see any help for commands:
 The "servers:edit" command cannot be found because it is registered under multiple names. Make sure you don't set a different name via constructor or "setName()".
bash-5.1$ console
WatchState ed5d09c

Usage:
  command [options] [arguments]

Options:
  -h, --help                  Display help for the given command. When no command is given display help for the list command
  -q, --quiet                 Do not output any message
  -V, --version               Display this application version
      --ansi|--no-ansi        Force (or disable --no-ansi) ANSI output
  -n, --no-interaction        Do not ask any interactive question
      --context|--no-context  Add context to output messages. Not all commands support this option.
      --trace                 Enable tracing mode. Not all commands support this option.
  -o, --output=OUTPUT         Change output display mode. Can be [table, json, yaml]. Not all commands support this option. [default: "table"]
  -v|vv|vvv, --verbose        Increase the verbosity of messages: 1 for normal output, 2 for more verbose output and 3 for debug

  The "servers:edit" command cannot be found because it is registered under multiple names. Make sure you don't set a different name via constructor or "setName()".

list [--raw] [--format FORMAT] [--short] [--] [<namespace>]

@eanzie Hi, Thank you for your report, i think i fixed it in the dev branch could you please switch to this tag and see if it's works for you ghcr.io/arabcoders/watchstate:dev

That fixed the issue, thanks!


[BUG] Input data type for Matalb

I tried to run the example code in matlab with with src = rand(3,5). Same for dst.
src = rand(3,5);;
dst = rand(3,5);
cbar2 = 1;
noise_bound = 0.01;
estimate_scaling = false; % we know there's no scale difference
rot_alg = 0; % use GNC-TLS, set to 1 for FGR
rot_gnc_factor = 1.4;
rot_max_iters = 100;
rot_cost_threshold = 1e-12;
% The MEX function version
[s, R, t, time_taken] = teaser_solve_mex(src, dst, cbar2, ...
noise_bound, estimate_scaling, rot_alg, rot_gnc_factor,    ...
rot_max_iters, rot_cost_threshold);
% The MATLAB wrapper version
[s, R, t, time_taken] = teaser_solve(src, dst, 'Cbar2', cbar2,    'NoiseBound', noise_bound, ...
'EstimateScaling',    estimate_scaling, 'RotationEstimationAlgorithm', rot_alg, ...
'RotationGNCFactor', rot_gnc_factor, 'RotationMaxIterations', 100, ...
'RotationCostThreshold', rot_cost_threshold);
The following error appeared:
Undefined function 'teaser_solve' for input arguments of type 'double'.
I also had the same problem for 5x3 matrices, type single and PointCloud; and with teaser_solve_mex function.
Could you help me?

I want to know How did you compile successfully in Matlab？

Hi @WJinhai @jbalado thanks for your interest. I will take a look at the MATLAB binding.

I want to know How did you compile successfully in Matlab？

Sorry, I didn't say it before, but it's running on linux, a clean install with just the mapping toolbox (and the necessary ones).


Remove space from distro

Pull Request Test Coverage Report for Build 990828549

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

Totals

Change from base Build 990816539:
0.0%

Covered Lines:
26167

Relevant Lines:
55572

💛  - Coveralls


Migrate 'components/sidebar_right_menu' module to TypeScript

Summary
Migrate 'components/sidebar_right_menu' module to TypeScript
Ticket Link
https://github.com/mattermost/mattermost-server/issues/16163
Related Pull Requests
Nil
Screenshots
Nil

Hello @KevinMarioGerard,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

/update-branch

/update_branch


[Question] How to migrate ignore

The new version uses the simple format over the advanced format.
This means this configuration:
"lint-staged": {
  "linters": {
    "*.js": [
      "prettier-standard",
      "git add"
    ],
    "package.json": [
      "finepack",
      "git add"
    ]
  },
  "ignore": [
    "static/*.{js, css}"
  ]
}

Should be written like:
"lint-staged": {
  "*.js": [
    "prettier-standard",
    "git add"
  ],
  "package.json": [
    "finepack",
    "git add"
  ]
}

The thing that's no clear to me is, then, how I can ignore files there?

https://github.com/okonet/lint-staged#example-ignore-files-from-match
That said, why do even need to ignore files? Are you editing them and committing them to the repository?

because "*.js" is considering static bundled files that I need to include as part of the git repo as valid, so I need to ignore them since they are the output of a process, not the original source files.
so I need to create a specific file and use a specific dependency just for doing the same? :(

You probably want to use .prettierignore anyway, so that prettier itself ignores the build folder. This issue is related to https://github.com/okonet/lint-staged/issues/651, where you can find an ignore example

not sure why you are assuming I'm using prettier; for me, they are two unrelated tools

Oh sorry, I read prettier-standard and interpreted it as prettier.

In any case, prettier-standard should also support the .prettierignore file, according to https://github.com/sheerun/prettier-standard/issues/50.
I think it's more logical for the actual linter task to contain its own ignore configuration, since this enables the same ignore rules for editors etc, outside of lint-staged.
Maybe we should update the README on this "philosophy". Thoughts, @okonet?

Yes! It’s definitely doesn’t belong to this package IMO.


Really, really disable pulseaudio when pulseaudio=false.

Without this my app would write an error to stdout. Somethin like "Pulseaudio: Cannot connect."

That does not make sense, all files in this folder are guarded with PULSEAUDIO_ENABLED, which is only defined if env['pulseaudio'] is True in platform/x11/detect.py.
Moreover, this doesn't build on non-X11 since env['pulseaudio'] exists only when you run scons p=x11.

Not very familiar with scons so I did what made my executable exclude pulseaudio. It's easy enough to test and see that without the change pulseaudio is included even when pulseaudio is disabled with pulseaudio=no.
As for the check for env['pulseaudio']  should that be changed to
if 'pulseaudio' in env:
Also now I wonder why https://github.com/godotengine/godot/blob/master/drivers/SCsub#L40
does not cause a problem on x11. Maybe linux scons is more forgiving???

Also now I wonder why https://github.com/godotengine/godot/blob/master/drivers/SCsub#L40 does not cause a problem on x11. Maybe linux scons is more forgiving???

That's because vsproj is defined as an option for all platforms in SConstruct, while pulseaudio is only defined as an option in platform/x11/detect.py.

As for the check for env['pulseaudio']  should that be changed to
if 'pulseaudio' in env:

It should be if 'pulseaudio' in env and env['pulseaudio']:
But I'm still confused about the issue you're having in the first place, I'll see if I can reproduce it.

I've built with scons p=x11 pulseaudio=no, and I can't reproduce the issue.
$ ldd bin/godot.x11.tools.64
        linux-vdso.so.1 (0x00007ffdd3b79000)
        libXcursor.so.1 => /lib64/libXcursor.so.1 (0x00007f472d545000)
        libXinerama.so.1 => /lib64/libXinerama.so.1 (0x00007f472d342000)
        libXrandr.so.2 => /lib64/libXrandr.so.2 (0x00007f472d137000)
        libXrender.so.1 => /lib64/libXrender.so.1 (0x00007f472cf2d000)
        libX11.so.6 => /lib64/libX11.so.6 (0x00007f472cbf0000)
        libXi.so.6 => /lib64/libXi.so.6 (0x00007f472c9e0000)
        libfreetype.so.6 => /lib64/libfreetype.so.6 (0x00007f472c742000)
        libpng16.so.16 => /lib64/libpng16.so.16 (0x00007f472c4e4000)
        libasound.so.2 => /lib64/libasound.so.2 (0x00007f472c1d6000)
        libz.so.1 => /lib64/libz.so.1 (0x00007f472bfba000)
        libGL.so.1 => /usr/lib64/nvidia-current/libGL.so.1 (0x00007f472bc7d000)
        libpthread.so.0 => /lib64/libpthread.so.0 (0x00007f472ba60000)
        libdl.so.2 => /lib64/libdl.so.2 (0x00007f472b85c000)
        libstdc++.so.6 => /lib64/libstdc++.so.6 (0x00007f472b4de000)
        libm.so.6 => /lib64/libm.so.6 (0x00007f472b1de000)
        libgcc_s.so.1 => /lib64/libgcc_s.so.1 (0x00007f472afc7000)
        libc.so.6 => /lib64/libc.so.6 (0x00007f472ac14000)
        libXfixes.so.3 => /lib64/libXfixes.so.3 (0x00007f472aa0e000)
        libXext.so.6 => /lib64/libXext.so.6 (0x00007f472a7fc000)
        libxcb.so.1 => /lib64/libxcb.so.1 (0x00007f472a5d6000)
        libbz2.so.1 => /lib64/libbz2.so.1 (0x00007f472a3c6000)
        librt.so.1 => /lib64/librt.so.1 (0x00007f472a1be000)
        /lib64/ld-linux-x86-64.so.2 (0x00007f472d750000)
        libnvidia-tls.so.390.59 => /usr/lib64/nvidia-current/tls/libnvidia-tls.so.390.59 (0x00007f4729fba000)
        libnvidia-glcore.so.390.59 => /usr/lib64/nvidia-current/libnvidia-glcore.so.390.59 (0x00007f47281b9000)
        libXau.so.6 => /lib64/libXau.so.6 (0x00007f4727fb5000)
        libXdmcp.so.6 => /lib64/libXdmcp.so.6 (0x00007f4727daf000)
        libbsd.so.0 => /lib64/libbsd.so.0 (0x00007f4727b9a000)

--> pulseaudio isn't linked
Running a game:
$ godot-git --main-pack daisy_dangerous_linux.pck
ALSA lib pcm.c:8323:(snd_pcm_recover) underrun occurred
ALSA lib pcm.c:8323:(snd_pcm_recover) underrun occurred
OpenGL ES 3.0 Renderer: GeForce GTX 670MX/PCIe/SSE2
Loading level: res://intro_screen.tscn   State: 0

ALSA warnings, but no Pulseaudio message. Sounds works fine.

I ended up just deleting all *.o files and cleaning up old directories that had been removed (like godot/tools) and rebuilding and disabling pulseaudio works as it should. I really, really do not know how the env got into that state. I use arch=x64 and use_static_cpp=yes and gdnative_wrapper=yes bits=64.
The only thing that I know is that the local git clone was old so maybe some old stuff triggered this. I have had issues with some things related to how the repo has changed over time.


Remove Laravel parts

Description
I've extracted the Laravel parts (config, service provider and facades) into swisnl/json-api-client-laravel.
Motivation and context
Closes #80
How has this been tested?
Tested with existing unit tests.
Types of changes
What types of changes does your code introduce? Put an x in all the boxes that apply:

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

Checklist:
Go over all the following points, and put an x in all the boxes that apply.

[x] I have read the CONTRIBUTING document.
[x] My pull request addresses exactly one patch/feature.
[x] I have created a branch for this patch/feature.
[x] Each individual commit in the pull request is meaningful.
[x] I have added tests to cover my changes.
[x] If my change requires a change to the documentation, I have updated it accordingly.

This is looking good, clean split.


Current Behavior
response = openai.ChatCompletion.create(model="gpt-3.5-turbo",temperature = 1.0,  # Change temperature here

File "/root/anaconda3/envs/openai/lib/python3.9/site-packages/gptcache/adapter/openai.py", line 100, in create
return adapt(
File "/root/anaconda3/envs/openai/lib/python3.9/site-packages/gptcache/adapter/adapter.py", line 172, in adapt
return_message = time_cal(
File "/root/anaconda3/envs/openai/lib/python3.9/site-packages/gptcache/utils/time.py", line 9, in inner
res = func(*args, **kwargs)
File "/root/anaconda3/envs/openai/lib/python3.9/site-packages/gptcache/adapter/adapter.py", line 161, in post_process
return_message = chat_cache.post_process_messages_func(
File "/root/anaconda3/envs/openai/lib/python3.9/site-packages/gptcache/processor/post.py", line 84, in temperature_softmax
scores = softmax([x / temperature for x in scores])
File "/root/anaconda3/envs/openai/lib/python3.9/site-packages/gptcache/utils/softmax.py", line 5, in softmax
x = np.array(x)
ValueError: setting an array element with a sequence. The requested array has an inhomogeneous shape after 1 dimensions. The detected shape was (2,) + inhomogeneous part.
Expected Behavior
No response
Steps To Reproduce
No response
Environment
GPTCache==0.1.32
Anything else?
No response

Is it convenient to provide the completed test code?

i will post this issue in another open bug ... where source code is mentioned ...is this good practice or not ?

Is it convenient to provide the completed test code?

In fact, what I mean is that you can write a simple demo to reproduce the error you encountered.

ok
import os
import time

#import openai as origOpenai
from gptcache import cache, Config
from gptcache.core import Cache
from gptcache.manager import manager_factory
from gptcache.embedding import Onnx
from gptcache.processor.post import temperature_softmax
from gptcache.similarity_evaluation.distance import SearchDistanceEvaluation
from gptcache.similarity_evaluation.onnx import OnnxModelEvaluation
from gptcache.adapter import openai
from gptcache.adapter.api import init_similar_cache
from gptcache.processor.pre import get_openai_moderation_input

from flask import Flask, redirect, render_template, request, url_for

cacheTemperature = 1.0
sysPrompt = "I want you to act as a Teacher.Do not act like chatbot. Reply concisely."
openaiKey = ''

app = Flask(__name__)

os.environ['OPENAI_API_KEY'] = openaiKey
#origOpenai.api_key = openai.api_key = os.getenv("OPENAI_API_KEY")
cache.set_openai_key()

llm_cache = Cache()
onnx = Onnx()
data_manager = manager_factory("sqlite,faiss", data_dir="llm_cache", vector_params={"dimension": onnx.dimension})
llm_cache.init(
    embedding_func=onnx.to_embeddings,
    data_manager=data_manager,
    similarity_evaluation=OnnxModelEvaluation(),
    post_process_messages_func=temperature_softmax
)

moderation_cache = Cache()
moderation_data_manager = manager_factory(data_dir="moderation_cache")
moderation_cache.init(
    data_manager=moderation_data_manager,
    pre_embedding_func=get_openai_moderation_input
)

@app.route("/chat", methods=["POST"])
def process():
    question = request.form.get('input', '')
    answer = getOepnAiResponse(question)
    return {'asnwer': answer}, 200

def getOepnAiResponse(question):
    start = time.time()
    if not question.strip():
        print("empty question" + " ,,, " + "empty answer" + " ,,, " + str(round(time.time() - start, 2)) + " ,,, " + "0", flush=True)
        answer = "No input received. Please enter a valid input and try again."
        return answer

    try:
        modRes = openai.Moderation.create(input=question, cache_obj=moderation_cache)
        print(str(modRes), flush=True)
        modOutput = modRes["results"][0]["flagged"]
    except Exception as e:
        print("got error once - " + str(e), flush=True)
        modRes = openai.Moderation.create(input=question, cache_obj=moderation_cache)
        modOutput = modRes["results"][0]["flagged"]
        print("after error modOutput = " + str(modOutput), flush=True)

    if not modOutput:
        response = openai.ChatCompletion.create(model="gpt-3.5-turbo",temperature = cacheTemperature,  # Change temperature here
            messages=[{"role": "system", "content":sysPrompt},
            {"role": "user", "content": question}],cache_obj=llm_cache)
        answer = response['choices'][0]['message']['content']
        token = str(response['usage']['total_tokens'])
        print(question + " ,,, " + answer + " ,,, " + str(round(time.time() - start, 2)) + " ,,, " + token, flush=True)
        return answer
    else:
        print(question + " ,,, " + "moderation failed" + " ,,, " + str(round(time.time() - start, 2)) + " ,,, " + "0", flush=True)
        answer = "Your question violates the policy. Please ask only relevant and appropriate questions."
        return answer

if __name__ == '__main__':
    app.run(host='<IP_ADDRESS>', port=5000, threaded=False)

i am getting some other error too ...mentioning below :
response = openai.ChatCompletion.create(model="gpt-3.5-turbo",temperature = 1.0,  # Change temperature here

File "/root/anaconda3/envs/openai/lib/python3.9/site-packages/gptcache/adapter/openai.py", line 100, in create
return adapt(
File "/root/anaconda3/envs/openai/lib/python3.9/site-packages/gptcache/adapter/adapter.py", line 172, in adapt
return_message = time_cal(
File "/root/anaconda3/envs/openai/lib/python3.9/site-packages/gptcache/utils/time.py", line 9, in inner
res = func(*args, **kwargs)
File "/root/anaconda3/envs/openai/lib/python3.9/site-packages/gptcache/adapter/adapter.py", line 161, in post_process
return_message = chat_cache.post_process_messages_func(
File "/root/anaconda3/envs/openai/lib/python3.9/site-packages/gptcache/processor/post.py", line 84, in temperature_softmax
scores = softmax([x / temperature for x in scores])
File "/root/anaconda3/envs/openai/lib/python3.9/site-packages/gptcache/utils/softmax.py", line 5, in softmax
x = np.array(x)
ValueError: setting an array element with a sequence. The requested array has an inhomogeneous shape after 1 dimensions. The detected shape was (2,) + inhomogeneous part.
response = openai.ChatCompletion.create(model="gpt-3.5-turbo",temperature = 1.0,  # Change temperature here

File "/root/anaconda3/envs/openai/lib/python3.9/site-packages/gptcache/adapter/openai.py", line 100, in create
return adapt(
File "/root/anaconda3/envs/openai/lib/python3.9/site-packages/gptcache/adapter/adapter.py", line 172, in adapt
return_message = time_cal(
File "/root/anaconda3/envs/openai/lib/python3.9/site-packages/gptcache/utils/time.py", line 9, in inner
res = func(*args, **kwargs)
File "/root/anaconda3/envs/openai/lib/python3.9/site-packages/gptcache/adapter/adapter.py", line 161, in post_process
return_message = chat_cache.post_process_messages_func(
File "/root/anaconda3/envs/openai/lib/python3.9/site-packages/gptcache/processor/post.py", line 84, in temperature_softmax
scores = softmax([x / temperature for x in scores])
File "/root/anaconda3/envs/openai/lib/python3.9/site-packages/gptcache/utils/softmax.py", line 6, in softmax
assert len(x.shape) == 1, f"Expect to get a shape of (len,) but got {x.shape}."
AssertionError: Expect to get a shape of (len,) but got (1, 1).

@kishanios123 i will fix the error in a new pull request. This is due to onnx similarity evaluation

assert len(x.shape) == 1, f"Expect to get a shape of (len,) but got {x.shape}." AssertionError: Expect to get a shape of (len,) but got (2, 1).

Ok, Thanks... the other two issues will also be fixed?
assert len(x.shape) == 1, f"Expect to get a shape of (len,) but got {x.shape}."
AssertionError: Expect to get a shape of (len,) but got (2, 1).
assert len(x.shape) == 1, f"Expect to get a shape of (len,) but got {x.shape}."
AssertionError: Expect to get a shape of (len,) but got (1, 1).

yes

I have thoroughly run your test code and, after repairing it, I found no issues. You may also want to try installing the dev version to confirm.
Thank you once again for your inquiries and support !!!

@kishanios123 the issue should be fixed in the latest version

ok, I have updated the package... and will update you if found any bugs...


[Enhancement]: Remove a lot of providers from the Manifest.

You have a lot of 'providertags inManifest` as:
    <provider
        android:name="dev.dworks.apps.anexplorer.provider.NonMediaDocumentsProvider"
        android:authorities="${applicationId}.nonmedia.documents"
        android:exported="true"
        android:grantUriPermissions="true"
        android:permission="android.permission.MANAGE_DOCUMENT">
        <intent-filter>
            <action android:name="dev.dworks.apps.anexplorer.action.DOCUMENTS_PROVIDER" />
        </intent-filter>
    </provider>

with the only difference in android:authorities="${applicationId}.nonmedia.documents". I suppose you can combine most of them under one provider tag with
    <meta-data
         android:name="android.support.FILE_PROVIDER_PATHS"
         android:resource="@xml/file_provider_paths"/>

specifying all needed path in file_provider_paths.

@isabsent  hello, sorry for commenting here.
I went through your answer at 'https://github.com/google/ExoPlayer/issues/4212#issuecomment-387071600' for feeding byte array to Exoplayer.
But as a new to android, I'm unable to understand and implement it. Can you help me with it?

Hi!
The answer you link is concerned with how to use ExoPlayer when you haven't a source like byte[], File or ByteBuffer where random access to the source is available. You have an InputStream (not FileInputStream or ByteArrayInputStream) and want to play media from it. If you have a byte[] as a source, the problem is disappear. What is a task you a trying to solve?

Thanks for your reply.
I'm getting data dynamically as array of bytes from an event. So how would
you proceed with it?
On Sun, 9 Feb, 2020, 8:39 AM isabsent<EMAIL_ADDRESS>wrote:

Hi!
The answer you link is concerned with how to use ExoPlayer when you
haven't a source like byte[], File or ByteBuffer where random access to
the source is available. You have an InputStream (not FileInputStream or
ByteArrayInputStream) and want to play media from it. If you have a byte[]
as a source, the problem is disappear. What is a task you a trying to solve?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/1hakr/AnExplorer/issues/123?email_source=notifications&email_token=AFVFLPWGXVRPTUJP7AXJZJLRB5XU7A5CNFSM4KLBVPG2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELGBIFI#issuecomment-583799829,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFVFLPST5ENRVPAXFRPEAO3RB5XU7ANCNFSM4KLBVPGQ
.

If you already have your data as a byte[] to the moment you are going to send it to ExoPlayer, you can envelop your byte[] in the ByteArrayDataSource and create a MediaSource from it:
public MediaSource createMediaSourceFromByteArray(byte[] bytes) {
    ByteArrayDataSource byteArrayDataSource = new ByteArrayDataSource(bytes);
    DataSource.Factory factory = new DataSource.Factory() {

        @Override
        public DataSource createDataSource() {
            return byteArrayDataSource;
        }
    };
    MediaSource mediaSource = new ExtractorMediaSource.Factory(factory)
            .setExtractorsFactory(new DefaultExtractorsFactory())
            .createMediaSource(Uri.EMPTY);
    return Objects.requireNonNull(mediaSource, "MediaSource cannot be null");
}

Then,  you can set above MediaSource to ExopLayer:
MediaSource byteArrayMediaSource = createMediaSourceFromByteArray(bytes);
exoPlayer.prepare(byteArrayMediaSource);

Thanks, I will try this.
On Sun, 9 Feb, 2020, 2:18 PM isabsent<EMAIL_ADDRESS>wrote:

If you already have your data as a byte[] to the moment you are going to
send it to ExoPlayer, you can envelop your byte[] in the
ByteArrayDataSource and create a MediaSource from it:
public MediaSource createMediaSourceFromByteArray(byte[] bytes) {
ByteArrayDataSource byteArrayDataSource = new ByteArrayDataSource(bytes);
DataSource.Factory factory = new DataSource.Factory() {
    @Override
    public DataSource createDataSource() {
        return byteArrayDataSource;
    }
};
MediaSource mediaSource = new ExtractorMediaSource.Factory(factory)
        .setExtractorsFactory(new DefaultExtractorsFactory())
        .createMediaSource(Uri.EMPTY);
return Objects.requireNonNull(mediaSource, "MediaSource cannot be null");

}
Then, you can set above MediaSource to ExopLayer:
MediaSource byteArrayMediaSource = createMediaSourceFromByteArray(bytes);
exoPlayer.prepare(byteArrayMediaSource);
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/1hakr/AnExplorer/issues/123?email_source=notifications&email_token=AFVFLPTHD2IWL3VKBCHY3WDRB67N3A5CNFSM4KLBVPG2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELGGLGQ#issuecomment-583820698,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFVFLPTH5DEI3VPZFW43YU3RB67N3ANCNFSM4KLBVPGQ
.


Problems with trying to setup Crostini to build Breath

Following the instructions for setting up Crostini to do this via the instructions here returns a few errors.
The first 3 steps gives me this:
crosh> vmc stop termina
crosh> vmc start termina
(termina) chronos@localhost ~ $ vmc container termina penguin --privileged true
-bash: vmc: command not found
(termina) chronos@localhost ~ $

Exiting that takes be back to crosh, where I tried:
crosh> vmc stop termina
crosh> vmc start termina && vmc container termina penguin --privileged true
Error: UnrecognizedOption("privileged")
[ERROR:src/main.rs:183] ERROR: command failed
crosh>

I've been trying several other variations of performing the instructions to no avail. I feel the solution is likely super simple, and I simply lack enough knowledge of Crostini. I would greatly appreciate any guidance, thanks!

I think you have to run: vmc container termina penguin --privileged true from Crosh? Neither work for me either though on ChromeOS 101.
When I try to run bash setup.sh cli arch I receive:
utils/bootstrap.sh: line 9: There are files in /mnt! Please clear this directory of any valuable information!:
No such file or directory

When I try to run these commands as described in the guide I receive the following errors:
From Crosh:
crosh> vmc container termina penguin --privileged true

Error: operation `container_start` failed: failed to start container: `FAILED`: failed to update privilege level: Invalid config: LXD was configured to only allow unprivileged containers
[ERROR:src/main.rs:183] ERROR: command failed

From Termina:
(termina) chronos@localhost ~ $ vmc container termina penguin --privileged true
-bash: vmc: command not found

I remember running those as well early on in troubleshooting the problem, and got the same results. This may be some change in ChromeOS itself, removing the ability to make containers privileged? That would be my guess from LXD was configured to only allow unprivileged containers

A comment from the thread (linked above) that I started:
It is actually possible if your host device has a newer kernel, although we haven't documented it yet.  what does `uname -a` show in crosh (*not* inside the VM) ?

Could you give me the output of uname -a from crosh? I'm guessing that priveledged containers are only allowed on certain, newer (maybe 5.4+) kernel versions.

uname -a returns Linux localhost 4.14.272-18805-g12d6850827f5 #1 SMP PREEMPT Wed May 4 19:55:34 PDT 2022 x86_64 Intel(R) Celeron(R) N4020 CPU @ 1.10GHz GenuineIntel GNU/Linux.
That explains why it's not working, I'm on a fairly outdated kernel. I suppose I'll need to find another way to setup Breath then?

I guess you could use Crouton or another computer with a bootable USB of Ubuntu. ChromeOS is really frustrating for those of us who like to tinker with the OS - they still haven't gave raw USB access.

Alright, thanks for the help, I think I can setup a crouton environment, not like ChromeOS will see much use after I get this going.


Start implementing standard NN modules in managed code.

This is an attempt at refactoring some of the built-in modules to rely on functional APIs rather than C++ modules. It was started last year, but never merged. It was brought to the forefront by the identified need to make more module attributes available, which requires a lot of work, some of which would be duplicative of this.
It should have no impact on performance or functionality, except that more module attributes will be available.
Note that this is not a PR into main, but rather a creating of a new branch where we will collaborate on this.

@NiklasGustafsson
Curious..why it is stopped and closed?

Because it is superceded by #1214


Add txt.fabio.com.ar

This PR is:

[x] Adding a new domain
[ ] Updating existing domain size
[ ] Changing domain name
[ ] Removing existing domain from list
[ ] Website code changes (512kb.club site)
[ ] Other not listed

[x] I used the uncompressed size of the site
[x] I have included a link to the GTMetrix report
[x] The domain is in the correct alphabetical order
[ ] This site is not a ultra lightweight site
[x] The following information is filled identical to the data file

I confirm that I have read the FAQ section, particularly the two red items around minimal pages and inappropriate content and I attest that this site is neither of these things.

[x] Check to confirm

- domain: txt.fabio.com.ar
  url: https://txt.fabio.com.ar/
  size: 312
  last_checked: 2023-04-19

GTMetrix Report: https://gtmetrix.com/reports/txt.fabio.com.ar/MblJss5T/

Thanks Fabio!


[Rendering] Content Security Policy violations in adaptive-expressions npm library

@BenBullock1992 commented on Wed Jun 09 2021
Target Platform
Please list the platform(s) that your application is targeting and remove platforms you are not targeting:

NodeJS

Target Application
Angular single-page web application
Problem Description
When serving our Angular application through our Node.js/Express.js web server, we are seeing Content Security Policy violations coming from the adaptive-expressions npm library when we try to render an Adaptive Card.
The violation is being raised as this library calls functions like "eval()" and "new Function()" which are considered dangerous since they are susceptible to XSS attacks.
Please could you advise on possible workarounds which will not compromise our security?
Version of SDK
"adaptive-expressions": "^4.12.0",
"adaptivecards": "^2.9.0",
"adaptivecards-templating": "^2.1.0"

A quick grep through the codebase shows no obvious direct use of eval or new Function() - can you share a minimal index.html file illustrating the security violations?

I've done a little digging and can see that the violation is coming from browser.js inside the adaptive-expressions NPM package.
Path: node_modules/adaptive-expressions/lib/browser.js
I'm unable to share an index.html file with you but here is a screenshot of the error:

It looks like the return Function piece of code is due to browserify using node-util when bundling. I suspect the core.js use of eval is for a similar reason. How are you consuming the adaptive-expressions package?

No, we're not bundling it ourselves as Angular manages this for us.
We render Adaptive Cards in our Angular application so have the following three NPM dependencies installed:
"adaptive-expressions": "^4.14.1"
"adaptivecards": "^2.9.0"
"adaptivecards-templating": "^2.1.0"
We use the adaptivecards-templating library for expanding our data into our template. This library has a peer dependency on adaptive-expressions. We're not specifically using anything from adaptive-expressions ourselves.

At the moment, there isn't much we can do without a larger overhaul of our browser bundling tools and possibly the existing package dependencies.
Our use of browserify introduces a transitive dependency on is-generator-function which uses return Function("...") internally. While we could potentially remove the direct use of require("util") within adaptive-expressions, it's quite difficult to do the same through all the transitive dependencies as well (i.e., dependencies of the dependencies listed in libraries/adaptive-expressions/package.json).
Unfortunately, unblocking your scenario (without changing your CSP settings) is quite difficult and not something we have prioritized at the moment.

What version of browserify are you using? After this change, Function constructor wouldn't be called unless isGeneratorFunction is actually called.


11/16/2022 PM Publish

Prior to this change, distinguishing between issue templates and links could be a little difficult.
This change:

Reorders the issue templates so filing a bug is the first option, then suggesting an improvement, committing to a quality contribution, and finally reporting a product change
Updates the issue template titles to use imperative language with a call to action and links in the issue picker to be questions
Updates the Quality Contributions project links to point to the updated templates

PR Summary

PR Checklist

[ ] Descriptive Title: This PR's title is a synopsis of the changes it proposes.
[ ] Summary: This PR's summary describes the scope and intent of the change.
[ ] Contributor's Guide: I have read the contributors guide.
[ ] Style: This PR adheres to the style guide.

Learn Build status updates of commit 3eae646:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.github/ISSUE_TEMPLATE/00-bug.yml
:white_check_mark:Succeeded

.github/ISSUE_TEMPLATE/00-quality.yml
:white_check_mark:Succeeded
n/a (file deleted or renamed)

.github/ISSUE_TEMPLATE/01-article.yml
:white_check_mark:Succeeded

.github/ISSUE_TEMPLATE/02-bug.yml
:white_check_mark:Succeeded
n/a (file deleted or renamed)

.github/ISSUE_TEMPLATE/02-quality.yml
:white_check_mark:Succeeded

.github/ISSUE_TEMPLATE/config.yml
:white_check_mark:Succeeded

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


ISPN-6869 Avoid deadlock when stopping DefaultCacheManager

https://issues.jboss.org/browse/ISPN-6869

I'm not sure I understand how it can help here? The issue is not about synchronizing on one particular lock (see jira description for more details)

Frankly, I think the synchronization policy in DefaultCacheManager is slightly wrong and adding another synchronization object is not the right approach here. Let me explain this a more...
When starting a new cache (inside wireAndStartCache method) we synchronize on caches instance (which is a concurrent map so it's wrong IMO). The shutdown procedure (stop method) synchronizes on this and a flag - stopping (!?). The terminate method (the one that actually kills the caches) uses no synchronization - just takes caches one by one (with a not-null check) and kills them.
As you can see we use different objects for synchronizing starting caches (caches field) and stopping them (this and stopping flag). In my opinion the best way to solve the deadlock described in JIRA is to refactor startup/shutdown procedure (and not just add new lock). I totally agree with @vjuranek  approach of using a Lock for this (shutdown and creating new cache should probably own a write lock whereas returning new cache should operate on read lock). There might be even better approach for this - StampedLock which supports upgrading from read lock to write lock (could be useful in getCache method - return if exists with read lock and if it doesn't - upgrade to write lock and wire a new cache).
cc @danberindei @rvansa

I don't think you need a read lock for getting the cache, accessing CHM is fine. And I would refrain StampedLock unless you need it for perf purposes - the risk of misuse is too high.
Nevertheless, I agree that all the synchronization should be executed on single lock (be it intrinsic lock or explicit one) and I would rather use ComponentStatus on DCM than another custom flag.

@slaskawi @rvansa can you elaborate why do you think one lock can solve the problem? This was the first solution I tried and it resulted into several other deadlock on different places. I didn't investigate much further, but it seems to me that that it would require changes in several other places (maybe substantial changes) and the risk to introduce new deadlock(s) would be IMHO quite high. Also synchronizing everything on one single lock (even if it's deadlock free) would very likely result into much worse performance.
@slaskawi  as for sync on caches - I believe the sync there is not because of caches object itself, but to create and store CacheWrapper atomically.
@rvansa ComponentStatus would be better choice if it's properly setup during whole lifecycle of the component (otherwise it can be quite confusing), requiring obtain write lock several times, while it's actually not needed anywhere besides stopping, so not completely sure if it worth it.

@vjuranek - Could you please tell us more how you implemented this (ideally share a branch, PR or a diff)? I believe you can not deadlock when using a single re-entrant lock, so there must be something else going on there...
As I suggested - IMO one lock should be ok - a read lock for obtaining a cache and a write lock for wiring it up and killing it (@rvansa prefers component status whereas I StampedLock with an upgrade but frankly - it doesn't matter it's only a detail).
BTW - I believe we should also have @danberindei opinion on that. I wouldn't like to move this forward without his comment...

@slaskawi sorry for late reply, I was on PTO on Thu and Fri. Unfortunately I haven't my old implementation as it didn't work (probably not very helpful, but I have some issue with starting a cache from cache starting listener listener -  i.e. with StartCacheFromListenerTest and core testsuite execution was considerably slower). I was thinking about it little bit and agree it would be probably doable to have only one lock (and adjusting other part to avoid e.g. StartCacheFromListenerTest failures), but I still think such implementation wouldn't be very efficient.
Anyway, completely agree we should wait for insight from @danberindei

rebased
@danberindei could you plase take a look once you have some time? Thanks

The failure after rebase is fixed now

Unfortunately I was looking at the wrong place... Of course LGTM!

@vjuranek Maybe if you added a test case, alternative solutions would prove as wrong immediately.

@rvansa IIRC I tried, but wasn't able to figure out any reliable reproducer .... will look on it again

@vjuranek @rvansa  use  byteman for the reproducer?  [1]
[1] http://www.jboss.org/dms/judcon/2013unitedstates/presentations/judcon2013_day3track1session1.pdf

Sorry it took so long to look at this @vjuranek!
The deadlock seems to come from the fact that InternalCacheRegistryImpl.stop uses DCM.getCache(name, false) -- something I should have removed with ISPN-6277. DCM.getCache(name, false) blocks when another thread is in the process of starting the cache. But global components, including InternalCacheRegistryImpl, are stopped while holding the GlobalComponentRegisty monitor, blocking any threads using the global components. If the cache starting thread needs to inject a global component into a cache component, it blocks too, causing the deadlock. The simple fix here is to remove InternalCacheRegistryImpl.stop().
DefaultCacheManager.stopCaches() also has a problem: it ignores caches that are not fully started yet. But instead of adding a new lock, I'd rather reuse the CacheWrapper latch. DCM.wireAndStartCache can also use ConcurrentMap methods instead of synchronized (caches).
I thought this would be a good time to change getCache(name) so it reports any exception during cache start to all callers of getCache(name), not just the first one, so I replaced CacheWrapper with a CompletableFuture and I implemented my proposed changes here: https://github.com/danberindei/infinispan/commits/ISPN-6869_dan
The branch also includes a reproducer. It doesn't rely on InternalCacheRegistryImpl.stop, it just checks that manager.getCache(name) blocks manager.stop(), and manager.stop() fails all manager.getCache(name) calls that start after it.
WDYT?

Thanks for review @danberindei !
Your solution is definitely more general and nicer (I even didn't noticed that caches are stopped on two different places and InternalCacheRegistryImpl.stop() can be completely removed) - closing this PR. Btw: also neat reproducer! Thanks

@danberindei which PR is replacing this one? AFAICT, the problem is still happening on master