Update Capybara inclusions in test_helper.rb

Very simple PR to require/include everything suggested by Capybara for integration with Minitest. I'm not aware of any current problems with our Capybara-based integration tests, but... there are very few of those as yet.
I'm thinking of refactoring two integration tests, StudentTest and LurkerTest, to use Capybara methods instead of our SessionExtensions, so I want to be sure it's set up right.
(The reason for refactoring these tests now, is that they fail with the refactored NamesController/NameDescriptionsControllers. Tests for those refactored routes will require integration tests to click on a link by ID attribute, not href, and that means rewriting some SessionExtensions methods... but Capybara's click methods already have that ability. Moving to Capybara in integration tests, and away from our own test methods, seems desirable for long term maintainability, too.)
Double-checking our setup vs. the Capybara manual, I notice we haven't required/included everything recommended. The manual may be much revised since we (@JoeCohen?) added Capybara, or these may have been conscious omissions.
So this PR will offer the opportunity to discuss the setup, if anybody knows anything about it, and CI will tell if the recommended inclusions and requirements cause any problems.

All the failures result from a conflict between identically-named assert_select methods. It might be helpful to rename ours, as we did for click_mo_link.
Capybara's assert_select method asserts the existence of a non-disabled select box element on the page.
MO's assert_select seems to look for elements on the page matching any kind of selector, although I cannot find where the method itself is defined, only this from SessionFormExtensions.
#  context::          Instance of the session that this form came from.

    # Allow user to make further HTML assertions on the form.
    def assert_select(*args, &block)
      context.assert_select(form, *args, &block)
    end

Question: Is MO's assert_select more like Capybara's assert_selector method?

Aha. Safely ignore this discussion (monologue) if you prefer.
Capybara has anticipated this conflict — in fact the name conflict arises from the many method aliases they provide for some perverse kind of convenience, and we don't even have to use those. They also enable you to not include the aliases, called Capybara::Minitest::Assertions. I reckon Joe already experienced this and made the wise choice.

Coverage remained the same at 92.633% when pulling b76da760ed6fa031c303e87e710954ef8c2b9a6f on nimmo-update-capybara into f80915209ced972848b107323ead4c5d2700573c on master.

Re Capybara:
My 2 cents: Do it by the book.
I plead guilty to installing Capybara. Omissions from what's recommended by
the Capybara manual are likely due to my not understanding the
recommendations, or inability to get things installed properly.
On Fri, Sep 2, 2022 at 12:36 AM andrew nimmo @.***>
wrote:

Very simple PR to require/include everything suggested by Capybara for
integration with Minitest. I'm not aware of any current problems with our
Capybara-based integration tests, but... there are very few of those as yet.
I'm thinking of refactoring two integration tests, StudentTest and
LurkerTest, to use Capybara methods instead of our SessionExtensions, so I
want to be sure it's set up right.
(The reason for refactoring these tests now, is that they fail with the
refactored NamesController/NameDescriptionsControllers. Tests for those
refactored routes will require integration tests to click on a link by ID
attribute, not href, and that means rewriting some SessionExtensions
methods... but Capybara's click methods already have that ability. Moving
to Capybara in integration tests, and away from our own test methods, seems
desirable for long term maintainability, too.)
Double-checking our setup vs. the Capybara manual, I notice we haven't
required/included everything recommended. The manual may be much revised
since we @.*** https://github.com/JoeCohen?) added Capybara, or
these may have been conscious omissions.
So this PR will offer the opportunity to discuss the setup, if anybody
knows anything about it, and CI will tell if the recommended inclusions and
requirements cause any problems.
You can view, comment on, or merge this pull request online at:
https://github.com/MushroomObserver/mushroom-observer/pull/1140
Commit Summary

5be780c
https://github.com/MushroomObserver/mushroom-observer/pull/1140/commits/5be780cdf7d1be2555055ac73ce6e9870ab2f56f
Update test_helper.rb

File Changes
(1 file
https://github.com/MushroomObserver/mushroom-observer/pull/1140/files)

M test/test_helper.rb
https://github.com/MushroomObserver/mushroom-observer/pull/1140/files#diff-ba37813ca277c227a74a372479b7b05b7f3ff085d890ab708f80d62573efdb7a
(11)

Patch Links:

https://github.com/MushroomObserver/mushroom-observer/pull/1140.patch
https://github.com/MushroomObserver/mushroom-observer/pull/1140.diff

—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/1140, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAALDFFJB57Z4VXTSFLNPE3V4GU77ANCNFSM6AAAAAAQDASQO4
.
You are receiving this because you were mentioned.Message ID:
@.***>

Re assert_select.
I am (as usual) so confused.
I use assert_select a lot. I think I'm using Rails' assert_select.
https://guides.rubyonrails.org/v6.1/testing.html#testing-views,
https://github.com/rails/rails-dom-testing/blob/master/lib/rails/dom/testing/assertions/selector_assertions.rb
I didn't know that there's a Capybara assert_select.
I didn't know that there's an MO assert_select.
I'm guessing that the MO assert_select is an extension of Rails
assert_select. But MO assert_select is never called. (I put a breakpoint in
the MO method and ran the test suite (in master). The breakpoint was never
hit.)
So maybe try deleting the MO method to see if that resolves the problem?
On Fri, Sep 2, 2022 at 1:02 AM andrew nimmo @.***>
wrote:

All the failures result from a conflict between identically-named
assert_select methods. It might be helpful to rename ours, as we did for
click_mo_link.
Capybara's assert_select method asserts the existence of a non-disabled
select box element on the page.
MO's assert_select seems to look for elements on the page matching any
kind of selector, although I cannot find where the method itself is
defined, only this from SessionFormExtensions.
context::          Instance of the session that this form came from.
# Allow user to make further HTML assertions on the form.
def assert_select(*args, &block)
  context.assert_select(form, *args, &block)
end

Question: Is MO's assert_select more like Capybara's assert_selector
method
https://rubydoc.info/github/teamcapybara/capybara/master/Capybara/Node/Matchers#assert_selector-instance_method
?
—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/1140#issuecomment-1235193848,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAALDFDFMX7BNYK5KC33OCTV4GYALANCNFSM6AAAAAAQDASQO4
.
You are receiving this because you were mentioned.Message ID:
@.***>

Re assert_select. I am (as usual) so confused. I use assert_select a lot. I think I'm using Rails' assert_select. https://guides.rubyonrails.org/v6.1/testing.html#testing-views, https://github.com/rails/rails-dom-testing/blob/master/lib/rails/dom/testing/assertions/selector_assertions.rb I didn't know that there's a Capybara assert_select. I didn't know that there's an MO assert_select. I'm guessing that the MO assert_select is an extension of Rails assert_select. But MO assert_select is never called. (I put a breakpoint in the MO method and ran the test suite (in master). The breakpoint was never hit.) So maybe try deleting the MO method to see if that resolves the problem?
…
On Fri, Sep 2, 2022 at 1:02 AM andrew nimmo @.**> wrote: All the failures result from a conflict between identically-named assert_select methods. It might be helpful to rename ours, as we did for click_mo_link. Capybara's assert_select method asserts the existence of a non-disabled select box element on the page. MO's assert_select seems to look for elements on the page matching any kind of selector, although I cannot find where the method itself is defined, only this from SessionFormExtensions. # context:: Instance of the session that this form came from. # Allow user to make further HTML assertions on the form. def assert_select(args, &block) context.assert_select(form, args, &block) end Question: Is MO's assert_select more like Capybara's assert_selector method https://rubydoc.info/github/teamcapybara/capybara/master/Capybara/Node/Matchers#assert_selector-instance_method ? — Reply to this email directly, view it on GitHub <#1140 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAALDFDFMX7BNYK5KC33OCTV4GYALANCNFSM6AAAAAAQDASQO4 . You are receiving this because you were mentioned.Message ID: @.>

Looks like you've already figured out how to deal with this.
But maybe still get rid of the MO method.

Superseded by #1146


Can connected with multi database?

Hi, notero is a great zotero plugin, i wonder if it can connected with multi database. usually i need to tracking multi research feild, i wan't mixed those info together.

Hi @ccwanggl! I do hope to add support for multiple databases in the future. Please follow along with #119 for progress updates and a possible temporary workaround.

Duplicate of #119


d3 and mixed content

We're running luigi scheduler behind nginx proxy which serves everything over https. Current index.d3.html links to http://d3js.org/d3.v3.js which doesn't work well with browsers causing 'mixed content' errors. Unfortunately d3js.org doesn't have https version.
The easiest solution would be to bundle d3 with luigi and serve it from /static.

I'm pretty sure they bundled it with luigi? There was at least some huge file that was committed. Find and ping the author in this ticket I would say. :)

the only huge file I see in /static/visualizer/js is dagre-d3.js which doesn't seem to have d3 built in.
https://github.com/spotify/luigi/blob/master/luigi/static/visualiser/index.d3.html loads both d3.v3.js and dagre-d3.js.

Ok, ping the approriate commiters then. :)

d3js is available over TLS un cloudflare
https://cdnjs.cloudflare.com/ajax/libs/d3/3.5.5/d3.min.js . However,
bundling it might be a good idea.
2015-07-08 11:14 GMT-04:00 Arash Rouhani<EMAIL_ADDRESS>
Ok, ping the approriate commiters then. :)
—
Reply to this email directly or view it on GitHub
https://github.com/spotify/luigi/issues/1052#issuecomment-119620372.

Hi @skazi0 ! ... I that library its not available on https!
Either try to change the url, or bundle it!
@Tarrasch, someone said in the past we should use a package manager to manage all the front dependencies... I'll investigate into it, I dont want to keep bundling JS files with some kLOCs.
The thing is, if we do go for the package manager, its likely to make the installation more complicated.

Hi @hadesbox, currently I worked this around locally by downloading the library and serving it from my host. This is OK for me but I thought that you might want some more generic solution that would work out of the box for everyone.

Given my zero knowledge of working with front end. I can't contribute with a technical solution. :)

sounds like we can just serve it from cloudflare using <script src="://cdnjs.cloudflare.com/ajax/libs/d3/3.5.5/d3.min.js"> – this will make sure to use whatever protocol is used

Apparently @erikbern proposal is now an anti-pattern https://twitter.com/paul_irish/status/588502455530311680 - Should we close this @Tarrasch ? or can we explicitly use https for this?

Interesting observation with the anti-pattern. I didn't know that. I haven't seen any contributions to d3 lately so this will probably never be picked up. Closing.


add .bazel extension support for Bazel

e.g. googletest uses the .bazel extension and many other projects seem to do so as well.
TODO:

[x] fix filename
[x] add starlark language support

Changes proposed:

[x] Add

Not sure my changes would achieve this:

... anyone having the bazel extension installed would get the icons for the file extensions defined by the bazel extension.

@robertohuertasm thanks for your support


df operations fail when spaces in column names

Hello @akharche! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file sdc/datatypes/hpat_pandas_dataframe_functions.py:

Line 924:121: E501 line too long (121 > 120 characters)


Treat locally-edited features differently

Check for user=POSM to identify features that have local edits.
Depends on https://github.com/AmericanRedCross/posm/issues/129

Note, I don't know if this is the best long term solution. Finding a way to let editors use their real OSM user name in POSM would be really good. We just haven't thought of the solution to this yet...

What's our current thinking on this issue, @mojodna & @hallahan?

Uhhh...
I defer to @dalekunce.
With a recent extract, the username associated with "local" edits is dkunce_import (and can be pulled in via osm2pgsql w/ the --extra-attributes option (in addition to --hstore-all)).

I redefer back to @mojodna
using my import account is fine as an extra attribute I guess for now.
Maybe we could just watch for all updates after the extract happens?

We are including metadata in OpenMapKit Server showing that the edits were done with OpenMapKit, as well as ODK data endpoints. You can use this changeset info to determine edits were made with OMK, and therefore POSM.


[CardMedia] Allow styling without breaking image

[x] PR has tests / docs demo, and is linted.
[x] Commit and PR titles begin with [ComponentName], and are in imperative form: "[Component] Fix leaky abstraction".
[x] Description explains the issue / use-case resolved, and auto-closes the related issue(s) (http://tr.im/vFqem).

As explained in Docs, MUI should allow overwriting of styles via inline-styles.
Therefore I would expect the following snippet to have a height of 200px and foo.jpg as a background image:
<CardMedia image="foo.jpg" style={{ height: 200 }} />

Currently the custom inline style attribute overwrites the internal styles for background image.
This PR allows users to set custom inline styles without overwriting the internal behaviour directly. Though it still enables users to consciously overwrite backgroundImage in order to stick to the following statement:

You don't have to worry about CSS specificity as the inline-style takes precedence over the regular CSS.

@pex Could you rebase on v1-beta? codecov and argos aren't able to find base line info.

Thanks!


Fix hanging crash when passing null to page#evaluate

Proposed changes in this pull request
phantom-node currently hangs indefinitely if you try to pass an argument of null or undefined to the page#evaluate method. For example:
var Phantom = require('phantom');

var sitepage = null;
var phInstance = null;
Phantom.create()
  .then(instance => {
    phInstance = instance;
    return instance.createPage();
  }).then(page => {
    sitepage = page;
    return page.open('http://httpbin.org/get');
  }).then(status => {
    sitepage.evaluate(function(val) {
      return "Hello world! Parameter value = "+val;
    }, null).then(function(result){
      console.log(result);
    });
  })
  .then(() => {
    sitepage.close();
    phInstance.exit();
  })
  .catch(error => {
    console.log(error);
    phInstance.exit();
  });;

This PR updates the transform method to properly handle both null and undefined.
Checklist

[x] New tests have been added
[x] npm test passes successfully
[x] Documentation has been updated (I figured no documentation updates were necessary, but I'm happy to add documentation for passing parameters into page#evaluate if desired)

@amir20 to review

Nice find. Seems simple. Merging.


Implement originProv

As discussed in the meeting.

Fixed in https://github.com/hkust-taco/mlscript/pull/72.


Query on zip file migration

I perfromed migration succesfully for couple of project however for some of my project from RTC to Bitbucket, we observed some descripencies.
1 .zip file migrated from RTC to BB but no RTC history migrated to Bitbucket
2, .xlsx file migrated but has more history in BB than RTC
Is above are limitations of this tool ?

Hi @smartarix
Is this related to your in the meantime created issue #66 ?
The tool should migrate all files (except if you exclude some file extensions).  So it should migrate all files with all the history present in rtc (except cvs/svn history migrated already to rtc).

Yes, I'm closing this issue.


deploy empty redash + connect to anyway db

@atalyaalon done: https://redash.dataforchange.org.il/

Wonderful thank you!


Dashboards take a long time to load

Looking at the console it seems like we make a request to get all of the variables

you were totally right about the variables btw.. just... for the record


Add a function for rotating a vector by a quaterion

Resolves #141. The implementation should be simpler than #142
I'm not sure if HMM_RotateQV3() follows the proper naming convention, suggestions welcome

I think operations to rotate a vector directly would probably be useful, but I don't want to merge this on its own - if we do, we should also have other functions for rotating a vector directly.
I think we would probably want the following. The naming overlap with HMM_Rotate_[LH|RH] is unfortunate, but probably not actually a problem in practice.

HMM_RotateV2(HMM_Vec2 Vec, float Angle)
HMM_RotateV3Q(HMM_Vec3 Vec, HMM_Quaternion Quat) (this PR, note different name)
HMM_RotateV3AxisAngle_[LH|RH](HMM_Vec3 Vec, float Axis, float Angle) - this is probably not worth including unless we can in some way simplify the math to make it more efficient than HMM_RotateV3Q(vec, HMM_QFromAxisAngle_[LH|RH](axis, angle)).

I don't think we would need or want any C generics or C++ overloads. Thoughts @StrangeZak @dev-dwarf?

I think it is worth having HMM_RotateV3AxisAngle as a function even if its not a custom code path, so I went ahead and added that. I also changed the RotateV3Q as you said. My changes are on #168 since I cant add commits to this PR, but feel free to merge them in here @occivink!
I'm not sure about RotateV2, because it also needs a LH/RH variant really (more often called clockwise or counter-clockwise in 2d). But it would be easy to add that as well.

@dev-dwarf That code looks good, and seeing it now I guess I agree that it's fine to have HMM_RotateV3AxisAngle even if it's not super optimized. Convenience doesn't really cost us anything here, and we could always optimize it later.
I don't think 2D rotation needs LH/RH variants. We use typical trig functions; rotation is unambiguously counter-clockwise about the desired axis. The catch is that in 3D the arrangement of the axes can produce opposite-looking results. But no such problem exists in 2D. Unless you're talking about y-down vs. y-up, which...just no. That would be really weird. Right?

What I am talking could also be thought of as y up vs down, and I do in fact think it's an important distinction and that the orientation is not unambiguous. From my point of view most math books use y up, but plenty of game engines and os apis use y down for screen coordinates.
That said you're the one with merge permissions haha.

Yeah, I guess so. Still, I'm not sure there need to be two separate flavors. When people are working with y-down, wouldn't they expect positive rotation to be clockwise? Certainly any sin/cos math they do would give them that result.
The whole thing is weird, because in 3D we think of rotation about an axis, while in 2D we think of rotation in a plane. In 3D that means you have to worry about LH/RH because the orientation of the plane is ambiguous. In 2D, though, we generally just think of rotation as proceeding from X toward Y. Right? You can conceptualize it as rotation about some pseudo-axis, but...is that how people actually think of it?
I guess our options are probably:

HMM_RotateV2 only, rotates from X toward Y (counterclockwise in Y-up, clockwise in y-down)
HMM_RotateV2_[LH|RH], implicitly defining rotation about a pseudo Z-axis
HMM_RotateV2_[CW|CCW]_[YU|YD], totally unambiguously defining both clockwise/counterclockwise and y-up/y-down.

I personally prefer option 1, making Handmade Math neutral to "screen" or "world" concepts, and keeping its behavior in line with sin/cos and complex numbers.

I think of rotation in 2d and 3d as happening in planes, but in either case you still have to decide which rotation is mapped to positive angles. Since this is all that's happening anyway, it's not such a big deal to just negate the angles if you don't like the default convention.
That being said I think option 1 would be fine : )

Handmade Math now includes the three functions I listed in my previous comment.


Do not block on telemetry

This makes the emitOnce function something that will fire-and-forget the telemetry requests so that a long write to the event logs table or a network failure does not blow up the user experience.

@Strum355 Makes sense since it is a network request and that's ... generally what you do here.


lxd/storage: Use correct operation type

Signed-off-by: Stéphane Graber<EMAIL_ADDRESS>

@tomponline figure it'd be easier for me to merge your branch and send that fix separately


Uses babel.config.js when babelrc is false

It has taken me a long while to figure out that the problems I was having with rollup-plugin-babel were because there is a conflict between the settings passed with babelrc: false and the actual babel.config.js.
Is it possible to disable using babel.config.js when bablerc is set to false?
Even if I set runtimeHelpers: true this causes code to compile weirdly. I am using preserveModules with Rollup and when passing runtimeHelpers: true it causes node_modules to be placed in the output directory with @babel/runtime as the only dependency in that folder.
The only way around this is to set plugins only meant for babel.config.js and not Rollup within an env block.
This problem will likely be alleviated when https://github.com/rollup/rollup-plugin-babel/pull/325 lands, but will still be hard to debug / figure out the proper way of configuring when someone has this problem.
Example
rollup.config.js
...
babel({
  extensions: ['.js', '.ts'],
  exclude: 'node_modules/**',
  babelrc: false,
  // This fixes the problem:
  //
  // configFile: false,
  sourceMaps: true,
  presets: [
    [
    [
      '@babel/preset-env',
      {
        targets: {
          node: 'current',
        },
      },
    ],
    '@babel/preset-typescript',
  ],
})
...

babel.config.js
module.exports = {
  presets: [
    [
      '@babel/preset-env',
      {
        targets: {
          node: 'current',
        },
      },
    ],
    '@babel/preset-typescript',
  ],
  plugins: [
    '@babel/plugin-proposal-class-properties',
    '@babel/plugin-transform-runtime', // Makes `rollup-plugin-babel` complain
  ],

  // Workaround:
  //
  // env: {
  //   test: {
  //     plugins: ['@babel/plugin-transform-runtime'],
  //   },
  // },
};

Seems like there's another option configFile which must be passed in as false along with babelrc to completely disable loading configurations outside of the Rollup config. It might be a good idea to document this and maybe even throw some errors.

Feel free to prepare a documentation PR, as to throwing errors - that's not a good idea, because rollup-plugin-babel should reflect closely Babel's options and if Babel has 2 separate flags for this, then the same should be handled by this plugin.

Sure, can do. Sorry, errors might be too harsh, more like warnings, but I get your point 👍

Going to close this now, because I'm cleaning the issues to migrate this plugin to Rollup plugins monorepo ( https://github.com/rollup/plugins ). Once that happens - this can be raised once again there. Hope you understand.


Changed the version of pytorch to >=1.0.0

Updated the setup.py file to accept for pytorch version >= 1.0.0
(Issue reference : Open Issue #77

@kajal-puri thanks for the contribution ! LoL https://github.com/arraiyopensource/torchgeometry/pull/91/files


Adjustable doc/Jamfile setting

Allow easily switching the documentation between a standalone library and an integrated boost library. Just change a variable at the top of the file standalonelibrary = "true" ;

If switching to antora docs, this PR is obsolete.


Add transport-commons dependency and update others

Closes https://github.com/feathersjs/feathers/issues/2287

This is now fixed, the latest tag on npm was wrong.


Please upgrade axios in the @sendgrid/client since it has security issue

Issue Summary
@sendgrid/client has dependecy to<EMAIL_ADDRESS>and that version of axios has a high severity security issue (as per whitesource) Please see below image, and seems issue is resolved on any version higher than 0.21.4.

Technical details:

sendgrid-client version: 7.6.0
node version: 16

This issue has been added to our internal backlog to be prioritized. Pull requests and +1s on the issue summary will help it move up the backlog.

Hi @eshanholtz any update on upgrading<EMAIL_ADDRESS>to the latest to fix security vulnerability in @sendgrid/client ?

Hi @eshanholtz  any update on this, If need any detail or help let me know? Due to this issue our production release on hold.
This has been since November.

@eshanholtz it's. high severity issue. Should help with prioritization.

if you are using npm >8.3.0, you can add "overrides": {} section to your package.json and force another version of axios, until this is fixed, please make sure you will do regression in case there was a breaking change
https://docs.npmjs.com/cli/v8/configuring-npm/package-json#overrides

@eshanholtz @thinkingserious looks like similar issue was last time fixed within days: https://github.com/sendgrid/sendgrid-nodejs/issues/1237 why now months of silence?

@RezaRahmati tried to use overrides property of package.json but couldn't succees.
Could you please share code change for the same?
Node : 14.19.0
npm : 6.14.16

@RezaRahmati tried to use overrides property of package.json to override axios version to 0.25.0 but couldn't succeed. sample 1 :
{
    "dependencies": {
        "@sendgrid/mail": "^7.6.0",
        "bluebird": "^3.5.1",
    }
    "overrides": {
         "axios": "0.25.0"
    }
}

sample 2 :
{
    "dependencies": {
        "@sendgrid/mail": "^7.6.0",
        "bluebird": "^3.5.1",
    }
    "overrides": {
         "@sendgrid/mail": {
             "@sendgrid/client": {
                  "axios": "0.25.0"
              }
         }
    }
}

Could you please share code change for the same?
Node : 14.19.0 npm : 6.14.16

please read my answer again, it needs node >= 8.3.0


Not existing files cause uncaught exception from CFE.

For example:
import 'does_not_exist.dart';

Exception should not leave CFE, and should be reported as compilation errors.
  Crash when compiling package:test/foo.dart,
  at character offset null:
  FileSystemException(path=package:test/foo.dart; message=Not found)
  #0      _FileSystemEntityAdaptor.readAsBytes (package:analyzer/src/dart/analysis/frontend_resolution.dart:487:7)
  <asynchronous suspension>
  #1      SourceLoader.tokenize (package:front_end/src/fasta/source/source_loader.dart:154:65)
  <asynchronous suspension>
  #2      SourceLoader.buildOutline (package:front_end/src/fasta/source/source_loader.dart:192:26)
  <asynchronous suspension>
  #3      Loader.buildOutlines (package:front_end/src/fasta/loader.dart:176:13)
  <asynchronous suspension>
  #4      KernelTarget.buildOutlines.<anonymous closure> (package:front_end/src/fasta/kernel/kernel_target.dart:235:24)
  <asynchronous suspension>
  #5      withCrashReporting (package:front_end/src/fasta/deprecated_problems.dart:67:24)
  <asynchronous suspension>
  #6      KernelTarget.buildOutlines (package:front_end/src/fasta/kernel/kernel_target.dart:232:12)
  <asynchronous suspension>
  #7      FrontEndCompiler.compile.<anonymous closure>.<anonymous closure> (package:analyzer/src/dart/analysis/frontend_resolution.dart:227:30)

I guess this one was fixed by someone at some point.
To verify this I've written the following program in /tmp/asdf.dart:
import 'lkajsdfjalfjalkjdf.dart';

main() {}

It gives the following output when being compiled:
file:///tmp/lkajsdfjalfjalkjdf.dart:1: Error: No such file or directory
library;
import self as self;

static method #main() → dynamic {
  throw "file:///tmp/lkajsdfjalfjalkjdf.dart:1: Error: No such file or directory";
}

@scheglov Could you verify that the issue is resolved?

I'm updating the implementation of FileSystemEntity in Analyzer, to throw FileSystemException when the file does not exist. The exception does not leave CFE now.
But as you can see, https://github.com/dart-lang/sdk/issues/33686 still happens - we throw from main. I don't know if we want to keep all the similar problem be tracked there, or each instance should be fixed individually, so requires a separate issue, like this one.

Ok, I think I need a clarification here: are we talking about an exception thrown at compile time or at run time?
In my previous message I assume that it's an exception thrown by the compiler during compile time.  At least, that's how I read the title of this GitHub issue.  That's why I've invoked just the compiler and saw now exceptions thrown from it.  There was, however, an exception in the output program that would be thrown if the program was run.
So, I think if this GitHub issue is about an exception thrown during compile time, it should be closed.  If it's about the exception in the output program that would manifest itself during run time, it should be renamed and the description should be changed.  And if that's the case, it's clearly a sub-issue of https://github.com/dart-lang/sdk/issues/33686, as suggested.

It was an exception during compile time.
So, yes, formally this issue, with its initial title and message is fixed.
Feel free to close it.
I'm mentioning #33686 because it seems that there might be different paths leading toward deprecated_InputError, and requiring different solution, so it might be useful to track them separately. If this is not something useful and all of them will be fixed at once, that is fine with me too :-)

Got it.  I think those two, the original exception at compile time and an execution path leading to deprecated_InputError should be tracked separately.  I'm closing this one and will create a specific sub-issue for https://github.com/dart-lang/sdk/issues/33686.

I've created a list of causes for deprecated_InputError in https://github.com/dart-lang/sdk/issues/33686.  This issue is references there, so I think there's no need to open another issue for that.


storePageIn() fails to find container in new doc if nested

Nesting the container even one level under the body tag prevents storePageIn() from find it.

TEST CONTENT

The filter() function won't find the container id (Chrome v43):
storePageIn: function (object, url, doc, id) {
    /../
    object[url] = { // Content is indexed by the url
        /../
        html: $newDoc.filter('#' + id), // Stores the contents of the page
    };
}

Consider using .find() instead?

Would be great to update the docs to say where location of the container is supported for now.
Thanks for a really cool lib!

I've got a pending PR that should take care of this that I just sent in for review. It was basically as you suggested, @ffdead. The only trick was that the $(doc) coming in needed to be collected under a parent node so .find() could be used effectively.

That's great. Big thanks!

@ffdead Can you check to see if this issue has been resolved with @DigitalMachinist's changes?

@ffdead Sorry to pester, but it would be really helpful if you could let us know if this issue has been resolved for you so we can close it. Thanks!

Sorry for the delay guys,
I just tried the Master branch on my current project and was successfully able to nest the smoothstate container inside other wrapping elements. I tried 1 and 2 levels down from the body tag.
Verified!

This gave me a real headache today. Tried to make smoothState working with a nested container and it kept failing (used version 0.7.2 via npm). With the master branch version it works like a charm.
@miguel-perez would you mind making a new release?

Also ran into this same issue again today on a new project using v7.2 via npm.


PowerToys Run opens up but doesn't detect text when Windows Start is open

ℹ Computer information

PowerToys version: 0.29.3
PowerToy Utility: Run
Running PowerToys as Admin: Yes
Windows build number: 10.0.18363.1256

📝 Provide detailed reproduction steps (if any)

open windows start menu
launch powertoys run with shortcut
text is detected by start menu instead of powertoys run

✔ Expected result
text detected by powertoys run
❌ Actual result
text detected by start menu instead and start menu is shown above powertoys run
📷 Screenshots

Duplicate of https://github.com/microsoft/PowerToys/issues/3469

Duplicate of https://github.com/microsoft/PowerToys/issues/3469


Instance weights for learning to rank

I am training a learning to rank model with label, group id, query id and features
my objective is to maximize rank:ndcg@10.
I want to give weights for each label while training in dataframe instead of Libsvvm format.
How can I give it ?

You can specify a weight column using setWeightCol (): https://github.com/dmlc/xgboost/blob/1c08b3b2eaf356a448c156271c394692dc698065/jvm-packages/xgboost4j-spark/src/test/scala/ml/dmlc/xgboost4j/scala/spark/XGBoostRegressorSuite.scala#L118


chore: instantiate replace bcoin with LiquidJS

I think more bitcoin-ops actually, which would be good to have it liquid-ops since we never did.


[jaeger-client-cpp] New port

This is a port of the jaeger c++ client.
In combination with the opentracing port it gives application the ability to produce tracedata.
This pull request contains an version upgrade to the opentracing port, as the previous version produced a release build as the debug version. This makes any code using the debug version to crash.

/azp run

Hi @xgcssch thanks for this PR.
I noticed that this port failed on arm64-windows triplet. Is this expected? If this port doesn't support this triplet, please add vcpkg_fail_port_install(ON_ARCH "arm64") to the top of portfie.cmake.
At the same time, update jaeger-client-cpp:arm64-windows=fail to ci.baseline.txt.
If not, please try to fix it.

Hi @xgcssch thanks for this PR.
I noticed that this port failed on arm64-windows triplet. Is this expected? If this port doesn't support this triplet, please add vcpkg_fail_port_install(ON_ARCH "arm64") to the top of portfie.cmake.
At the same time, update jaeger-client-cpp:arm64-windows=fail to ci.baseline.txt.
If not, please try to fix it.

Hello!
As i've no access to an arm machine, i added the vcpkg_fail_port_install() call.
For the change in ci.baseline.txt: in my fork this file does not exist.
Do you have any idea how to incorporate this change in the current pull request?
I haven't had such a situation before.
Best regards Soenke

As for the issue on arm64-windows, I will have a further investigate.
For the change in ci.baseline.txt, you need to pull from vcpkg master branch to your branch. Since this file is added recently. If you can confirm this port currently doesn't support arm64, you can update it.

As stated in my initial comment to this pull request:

This pull request contains an version upgrade to the opentracing port, as the previous version produced a release build as the debug version. This makes any code using the debug version to crash.

Thanks again.
Could you update the two lines as vcpkg_fail_port_install(ON_ARCH "arm64" ON_TARGET "UWP")?
vcpkg_fail_port_install(ON_TARGET "UWP")
vcpkg_fail_port_install(ON_ARCH "arm64")

Sorry, I didn't see this before.
Besides this, it looks good to me now.

/azp run

/azp run

Hi @xgcssch
Could you please resolve the conflicts?

Hello @NancyLi1013
I'm sorry, but it is not possible to resolve the conflict anymore.
Very annoying: due to the long delay, the time window for integration has passed:
the upstream project https://github.com/jaegertracing/jaeger-client-cpp is still using the Version 1.5 of opentracing. They have to migrate to Version 1.6 until we can give this new port another try.
Best regards
Sönke

Hi @xgcssch
Sorry for the late reply.
I try to update opentracing to 1.6.0 and then build the port with the following errors:
F:\9126\vcpkg\buildtrees\jaeger-client-cpp\src\cc1c5a2d08-596f1d4c7b\src\jaegertracing/Span.h(78): error C2259: 'jaegertracing::Span': cannot instantiate abstract class
F:\9126\vcpkg\buildtrees\jaeger-client-cpp\src\cc1c5a2d08-596f1d4c7b\src\jaegertracing/Span.h(36): note: see declaration of 'jaegertracing::Span'
F:\9126\vcpkg\buildtrees\jaeger-client-cpp\src\cc1c5a2d08-596f1d4c7b\src\jaegertracing/Span.h(78): note: due to following members:
F:\9126\vcpkg\buildtrees\jaeger-client-cpp\src\cc1c5a2d08-596f1d4c7b\src\jaegertracing/Span.h(78): note: 'void opentracing::v3::Span::Log(opentracing::v3::SystemTime,const std::vector<std::pair<opentracing::v3::string_view,opentracing::v3::Value>,std::allocator<std::pair<opentracing::v3::string_view,opentracing::v3::Value>>> &) noexcept': is abstract
F:\9126\vcpkg\installed\x86-windows\include\opentracing/span.h(181): note: see declaration of 'opentracing::v3::Span::Log'
F:\9126\vcpkg\buildtrees\jaeger-client-cpp\src\cc1c5a2d08-596f1d4c7b\src\jaegertracing/Span.h(78): note: 'void opentracing::v3::Span::Log(opentracing::v3::SystemTime,std::initializer_list<std::pair<opentracing::v3::string_view,opentracing::v3::Value>>) noexcept': is abstract
F:\9126\vcpkg\installed\x86-windows\include\opentracing/span.h(177): note: see declaration of 'opentracing::v3::Span::Log'
[7/185] C:\PROGRA~2\MICROS~3\2019\COMMUN~1\VC\Tools\MSVC\1425~1.286\bin\Hostx64\x86\cl.exe   /TP -DBOOST_CONFIG_SUPPRESS_OUTDATED_MESSAGE -DYAML_CPP_DLL -D_SCL_SECURE_NO_WARNINGS -D_SILENCE_CXX17_ITERATOR_BASE_CLASS_DEPRECATION_WARNING -Djaegertracing_EXPORTS -IF:\9126\vcpkg\buildtrees\jaeger-client-cpp\src\cc1c5a2d08-596f1d4c7b\src -Isrc -IF:\9126\vcpkg\installed\x86-windows\include /DWIN32 /D_WINDOWS /W3 /utf-8 /GR /EHsc /MP  /D_DEBUG /MDd /Z7 /Ob0 /Od /RTC1 /showIncludes /FoCMakeFiles\jaegertracing.dir\src\jaegertracing\Tag.cpp.obj /FdCMakeFiles\jaegertracing.dir\ /FS -c F:\9126\vcpkg\buildtrees\jaeger-client-cpp\src\cc1c5a2d08-596f1d4c7b\src\jaegertracing\Tag.cpp
Microsoft (R) C/C++ Optimizing Compiler Version 19.25.28610.4 for x86

Since opentracing has been update to the latest version 1.6.0 in vcpkg.
Currently, we should support the new version.
So could you please post this issue to jaeger-client-cpp and let them look into it?

Rebased with head and removed the change for opentracing.
This should build now

It seems that it doesn't work in current state.
Could you please take a look?

Sorry for the extra iteration. My change didn't made it into the repo.
Cheers

Thanks for your continuous efforts on this PR. All pipelines build successfully now except for osx.
As for the regression on osx pipeline, we are trying to fix this.
Once passed, I will re run this PR.

/azp run

Cool, this looks great! Thanks @xgcssch :)


Create a search based document for weak-to-strong

This document is generated using: https://github.com/james4ever0/prometheous
You can view the website at: https://james4ever0.github.io/weak-to-strong/

that's cool but i don't think it makes sense to include in the main repo sorry


Compiler fails under Windows

When the output.filename contains directory and file (for example build\assets\js\app.js), the compilation under Windows will fail with "Path does not exist" error. That happens because the compiler expects UNIX slash (/) in the path, and gets Windows slash (\) instead.
The test, added in this PR, will fail (under Windows only) when run without the fix.

output.path must be an absolute path

No matter what output.path is. The compiler is broken because of output.filename

yes, it's a bug... I'll review the PR, as workaround you can use / in output.filename...

Thank you for quick reply!
Looking forward for your merge.
Unfortunately, it is not easy for me to use the suggested workaround, because in my case paths are valid and thay are generated by 3-party components.

Thanks


README: Fix typo

Simple typo fix.

Checklist

[ ] Language changes are updated in man/adoc/bpftrace.adoc
[ ] User-visible and non-trivial changes updated in CHANGELOG.md
[ ] The new behaviour is covered by tests

My bad. Thanks for the fix!


[Enhancement] Introduced brpc-java as brpc client

What type of PR is this：

[ ] bug
[ ] feature
[x] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

intruduced brpc-java as brpc client, so that callback can be used.

I have a question, if you already changed to brpc-java, why not remove jprotobuf?

@imay we still use jprotobuf compile .proto to .java, because if we use protoc, the interface will be very different (such as internal class, member public or private). we use brcp-java instead of jprotobuf-rpc.

[FE PR Coverage Check]
:disappointed: fail : 42 / 105 (40.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/rpc/LakeServiceClient.java
0
6
00.00%
[33, 35, 36, 51, 53, 54]

:large_blue_circle:
com/starrocks/common/proc/CurrentQueryInfoProvider.java
0
2
00.00%
[229, 230]

:large_blue_circle:
com/starrocks/qe/ResultReceiver.java
0
15
00.00%
[75, 76, 114, 115, 116, 117, 118, 119, 120, 121, 122, 123, 124, 125, 126]

:large_blue_circle:
com/starrocks/rpc/RpcCallBackImpl.java
2
6
33.33%
[15, 16, 20, 21]

:large_blue_circle:
com/starrocks/rpc/BackendServiceClient.java
16
38
42.11%
[76, 85, 86, 114, 155, 164, 165, 179, 180, 181, 183, 184, 194, 195, 196, 198, 199, 208, 209, 210, 212, 213]

:large_blue_circle:
com/starrocks/rpc/NewBrpcProxy.java
21
35
60.00%
[53, 57, 58, 59, 60, 61, 62, 63, 64, 65, 66, 67, 68, 69]

:large_blue_circle:
com/starrocks/lake/ShardDeleter.java
3
3
100.00%
[]

run starrocks_fe_unittest

run starrocks_fe_unittest

[FE PR Coverage Check]
:disappointed: fail : 34 / 95 (35.79%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/rpc/LakeServiceClient.java
0
6
00.00%
[33, 35, 36, 51, 53, 54]

:large_blue_circle:
com/starrocks/common/proc/CurrentQueryInfoProvider.java
0
2
00.00%
[229, 230]

:large_blue_circle:
com/starrocks/qe/ResultReceiver.java
0
16
00.00%
[76, 77, 115, 116, 117, 118, 119, 120, 121, 122, 123, 124, 125, 126, 127, 128]

:large_blue_circle:
com/starrocks/rpc/BackendServiceClient.java
16
38
42.11%
[76, 85, 86, 114, 155, 164, 165, 179, 180, 181, 183, 184, 194, 195, 196, 198, 199, 208, 209, 210, 212, 213]

:large_blue_circle:
com/starrocks/rpc/BrpcProxy.java
13
26
50.00%
[51, 55, 56, 57, 58, 59, 60, 61, 62, 63, 64, 65, 66]

:large_blue_circle:
com/starrocks/rpc/EmptyRpcCallback.java
2
4
50.00%
[15, 19]

:large_blue_circle:
com/starrocks/lake/ShardDeleter.java
3
3
100.00%
[]

run starrocks_admit_test

LGTM, waiting for the regression tests passed.

run starrocks_admit_test

run starrock_admit_test

run starrocks_fe_unittest

[FE PR Coverage Check]
:disappointed: fail : 31 / 94 (32.98%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/common/proc/CurrentQueryInfoProvider.java
0
2
00.00%
[229, 230]

:large_blue_circle:
com/starrocks/qe/ResultReceiver.java
0
16
00.00%
[76, 77, 115, 116, 117, 118, 119, 120, 121, 122, 123, 124, 125, 126, 127, 128]

:large_blue_circle:
com/starrocks/rpc/LakeServiceClient.java
2
10
20.00%
[41, 42, 46, 47, 51, 52, 53, 54]

:large_blue_circle:
com/starrocks/rpc/EmptyRpcCallback.java
1
3
33.33%
[11, 15]

:large_blue_circle:
com/starrocks/rpc/BackendServiceClient.java
16
38
42.11%
[77, 86, 87, 115, 157, 166, 167, 181, 182, 183, 185, 186, 196, 197, 198, 200, 201, 210, 211, 212, 214, 215]

:large_blue_circle:
com/starrocks/rpc/BrpcProxy.java
12
25
48.00%
[47, 51, 52, 53, 54, 55, 56, 57, 58, 59, 60, 61, 62]

run starrocks_admit_test

I checked the starrocks_admit_test output, there are 15 fails, all the load with brocker (13) are failed, and all the export with brocker (2) are failed, and the errmsg is "timeout", but from the output each test only took a few seconds.


Role manager which can take care of JWT claims

We need a role manager, which can extract role from JWT claims

Evaluating other options


Double work when passing a fastparquet.ParquetFile to read_parquet()

When we pass a fastparquet.ParquetFile to read_parquet() I found that the metadata will be read again in _read_fastparquet, because in read_parquet() , when we pass a fastparquet instance, dask is getting the path from fastparquet metadata and passing it to _read_fastparquet. I think if we are passing a fastparquet instance, we should make leverage of that and avoid to read it again to avoid some unnecessary computations. This can be a burden if metadata is stupidly large.
I'm probably missing something here and there is a reason for the metadada to be read again.
Anyway I'm making comments in the following snippets to mark what I'm saying
def read_parquet(path, columns=None, filters=None, categories=None, index=None,
                 storage_options=None, engine='auto', infer_divisions=None):
    is_ParquetFile = False
    try:
        import fastparquet
        if isinstance(path, fastparquet.api.ParquetFile):
            if path.open != fastparquet.util.default_open:
                assert (re.match('.*://', path.fn)), \
                       ("ParquetFile: Path must contain protocol" +
                        " (e.g., s3://...) when using other than the default" +
                        " LocalFileSystem. Path given: " + path.fn)

            assert (engine in ['auto', 'fastparquet']), \
                   ("'engine' should be set to 'auto' or 'fastparquet' " +
                    'when reading from fastparquet.ParquetFile')
            is_ParquetFile = True
    except ImportError:
            pass

    if is_ParquetFile:
        read = get_engine('fastparquet')['read']
        if path.fn.endswith('_metadata'):
            # remove '_metadata' from path
            urlpath = path.fn[:-len('_metadata')]
        else:
            urlpath = path.fn

        fs, fs_token, paths = get_fs_token_paths(
            urlpath,
            mode='rb',
            storage_options=storage_options
        )
    else:
        read = get_engine(engine)['read']
        fs, fs_token, paths = get_fs_token_paths(
            path, mode='rb',
            storage_options=storage_options
        )

        if isinstance(path, string_types) and len(paths) > 1:
            # Sort paths naturally if multiple paths resulted from a single
            # specification (by '*' globbing)
            paths = sorted(paths, key=natural_sort_key)

   #paths will always be a path and never a fastparquet instance <---- HERE!
    return read(fs, fs_token, paths, columns=columns, filters=filters,
                categories=categories, index=index, infer_divisions=infer_divisions)

def _read_fastparquet(fs, fs_token, paths, columns=None, filters=None,
                      categories=None, index=None, infer_divisions=None):
    import fastparquet

    #this line is useless because paths will never be an instance of fastparquet <---- HERE!
    if isinstance(paths, fastparquet.api.ParquetFile):
        pf = paths
    elif len(paths) > 1:
        if infer_divisions is not False:
            # this scans all the files, allowing index/divisions and filtering
            pf = fastparquet.ParquetFile(paths, open_with=fs.open, sep=fs.sep)
        else:
            return _read_fp_multifile(fs, fs_token, paths, columns=columns,
                                      categories=categories, index=index)
    else:

Can you check by adding the following line at the end of the if is_ParquetFile: block:
paths = path

Agree with that, but unfortunately in _read_fastparquet() function than len(paths) would never be > 1 and it can happen that you are passing a ParquetFile from a list of files. So we probably need to think a smarter way to do it. Don't you think?

it can happen that you are passing a ParquetFile from a list of files

Sorry, didn't quite follow that - in this case, we have already used fastparquet to scan the set of files beforehand, no?

This is really a corner case, but imagine that I read a list of files with fastparquet and now I use that object to make a read_parquet(parquetObject, infer_divisions=False) then we would never return _read_fp_multifile() because of len(parquetObject) = 1.
elif len(paths) > 1:
        if infer_divisions is not False:
            # this scans all the files, allowing index/divisions and filtering
            pf = fastparquet.ParquetFile(paths, open_with=fs.open, sep=fs.sep)
        else:
            return _read_fp_multifile(fs, fs_token, paths, columns=columns,
                                      categories=categories, index=index)

You do not want to reach _read_fp_multifile, that function is specialised for the case where you are reading from a list of files, but do not want to read their metadata beforehand and construct a ParquetFile object. That applies specifically to read_parquet('path/*/*.parquet)` without metadata files, where you can guarantee the schema is the same everywhere, as is the case for pyarrow data-sets and some spark/hive ones.


ASBPlayer will not play audio

I am on Chrome, have ASBPlayer installed, and every feature seems to work except for audio playing. Subtitles, Mining, Anki Integration... everything except for being able to hear the video.

on Chrome, I can't hear the video, too.
Please help.

The audio codec is probably not supported by your browser. You could try a different one as explained here: https://github.com/killergerbah/asbplayer/tree/main?tab=readme-ov-file#browser-compatibility

Getting the same, tried watching Alice in Borderland but for some reason I couldn't hear the voicelines but just the SFX


[Feature request] Allow the ability for the pinned tabs section to show as many pinned tabs have been configured

I found this solution in the reddit forums.
I like the ability for the pinned tabs to show all the tabs in that section without being limited to 6 in number.
The relevant css in the userChrome.css is:
/* Elongate the pinned tabs section */
#vertical-pinned-tabs-container {
  max-height: none !important;
}

Can you please incorporate this into natsumi-browser? BTW, this collection of tweaks is GREAT!!!!! It enhances Zen even further! I am using this for the past few days and I really appreciate your efforts in bringing us this collection in a easily followed installation procedure!

I will follow this up with a PR if you are ok with me contributing. I can put the above code into a conditional such that its controllable from the about:config section as well for those users who would like to opt-in.


Errors in console when interacting with demo for typeahead

With the latest master code, the last demo on the typeahead causes errors in the console:
ui-bootstrap.js:6267 Uncaught TypeError: Cannot read property 'scrollIntoView' of undefined
(anonymous function)	@	ui-bootstrap.js:6267
eventHandler

https://plnkr.co/edit/myFVFqnO2fHNo0aEednm?p=preview
Type some text in the last example, and then hit the arrow down key to scroll through results. This causes the error.
Might just be an issue with the demo and not the code itself, didn't have the time to look into it yet.

Thanks for this - I see what is happening, the children need to select specifically for the li elements. I'll have a fix for this shortly.


is_blacklisted was never awaited

I got some error when i add my own blacklist class
RuntimeWarning: coroutine 'TokenBlacklist.is_blacklisted' was never awaited
Anyone can help?
thanks.

Hello!
Basically, it is just asyncio warning so that never effect on your program.
But if you give me a source code fragment or repo link, I think I can help you a little

Traceback also helpful

this is my class,
`class TokenBlacklist(BlacklistABC):
async def register(self, token: Token):
blacklist = AuthBlacklist(access_token=token.raw_data['jti'])
await blacklist.commit()
async def is_blacklisted(self, token: Token):
    blacklist = await AuthBlacklist.find_one({'_id': token.raw_data['jti']})
    return blacklist != None`

when i access this method with valid token, i got that error and then got "Token has been revoked" message.
@app.route('/users/<id>', methods=['GET']) @jwt_required async def get_user(req, id, token):         ...

this is my class
class TokenBlacklist(BlacklistABC):
    async def register(self, token: Token):
        blacklist = AuthBlacklist(access_token=token.raw_data['jti'])
        await blacklist.commit()

    async def is_blacklisted(self, token: Token):
        blacklist = await AuthBlacklist.find_one({'_id': token.raw_data['jti']})
        return blacklist != None

when i access this method with valid token, i got that error and then got "Token has been revoked" message.
@app.route('/users/<id>', methods=['GET'])
@jwt_required
async def get_user(req, id, token):
     ...

this is the error, in terminal
python3.6/site-packages/sanic_jwt_extended/decorators.py:165:
RuntimeWarning: coroutine 'TokenBlacklist.is_blacklisted' was never awaited
  if JWT.config.use_blacklist and JWT.blacklist.is_blacklisted(token_obj):

OK I found why. is_blacklisted is async method.
but sanic_jwt_extended/decorators.py:165 is never await it.
if JWT.config.use_blacklist and JWT.blacklist.is_blacklisted(token_obj):

and not awaited coro object will always be considered as true. so even token is valid you got revoked warning.
This is my mistake. I will fix it now.
Thank you.

@archrinto patched 1.0.dev10 version released. plz update :>

Thanks @NovemberOscar


Add SAP Generative AI Hub as Model Provider

This PR is to add the SAP Generative AI Hub as a model provider.
The SAP Generative AI Hub provides a proxy for commercial models such as OpenAI and the ability to deploy your own models. OAuth is always used for authentication, i.e. if you use OpenAI models via the Generative AI Hub, you do not need an API key, but the OAuth token.
In this PR, I have slightly modified the OpenAI class and introduced protected methods for determining the URLs and for generating the request headers. Based on the modified class, I have added a SAPGenAI class that adapts the two methods for the Gen AI Hub.

This looks really good! Thanks for being making the PR so comprehensive. I'd prefer to leave SAP GenAI Hub out from modelData because we want to keep the UI uncluttered—we already leave out a few lesser-used providers, otherwise it might be confusing to new users.
I'm guessing this is just a detail of GenAI Hub, but could you explain briefly what the vllm type (vs. azure or openai) is used for?
Other than these two things, I'll run some tests on my own, and if it seems to be working well on your side also, then this is a PR that I am happy to move forward with.

I will remove the  GenAI Hub from model data:
I also got to reopen a new PR, because I did some commits using the wrong account...
Sorry for that

No worries


Does this code with the HT1632 chips

Hi there, great library!
I've tried running your code with the Sure 3208 and it works nicely, this board uses the HT1632C chip.
I've also got a few Sure 0832 boards which don't seem to work.  These use the HT1632. Your code mentions working with the HT1632 in places, but doesn't seem to work out of the box.
Is this possible?

I'm having the same problem with HT1632 ( without C) richardhawthorn . My display works fine with  https://github.com/devdsp/HT1632-AVR this library, but it´s hard to edit. In this library the clock pin is used and works very well. U must to insert a delay to read the words.

I've used this one in the past, and it works, but the library from gauravmm is so simple, it would be great to use it for both displays.


"name" isn't a valid "source" key

Currently lxc and lxd pass around "name" at container creation in the source map.
"name" isn't a valid field there, please use "fingerprint" or "alias".

I think the right fix here is to use "name" as in the original spec. The client doesn't know whether we're creating from an image or an alias, it just passes a string to the server which then creates things.

well, the problem is that the server doesn't know either. You could set a sha256 as an alias for another image.

So I tend to prefer the API being explicit and any random guessing happening on the client tool rather than have the server make a guess behind the scene.

Sure, except didn't we decide yesterday that delete of a container would delete all its snapshots too? I don't mind having it one way or the other (the server doing guessing or the client), but we should figure out which :)

I'm not sure how the two relate. Snapshots from an API point of view are under /1.0/containers/<name>/snapshots so it seems reasonable that a DELETE of /1.0/containers/<name> also removes any snapshot as they'd otherwise not be accessible through the API.
Aliases and fingerprints from an API point of view are two completely different things, the first lies in /1.0/images/aliases/ and the later directly in /1.0/images/
The fact that either can be passed for the same argument of the command line (instead of requiring a prefix as I was proposing back in CPT) is a pure client decision which shouldn't be reflected in the API.

On Thu, Feb 26, 2015 at 08:47:38AM -0800, St�phane Graber wrote:

I'm not sure how the two relate. Snapshots from an API point of view are under /1.0/containers/<name>/snapshots so it seems reasonable that a DELETE of /1.0/containers/<name> also removes any snapshot as they'd otherwise not be accessible through the API.

Seems to me like this is a distinction without a difference. Whatever,
I'll make the change after I'm done with this patch.

Aliases and fingerprints from an API point of view are two completely different things, the first lies in /1.0/images/aliases/ and the later directly in /1.0/images/
The fact that either can be passed for the same argument of the command line (instead of requiring a prefix as I was proposing back in CPT) is a pure client decision which shouldn't be reflected in the API.

Reply to this email directly or view it on GitHub:
https://github.com/lxc/lxd/issues/269#issuecomment-76214506


⚠ Tukarcoid has degraded performance

In 0cda4aa, Tukarcoid (https://tukar.co.id) experienced degraded performance:

HTTP code: 200
Response time: 2654 ms

Resolved: Tukarcoid performance has improved in ff0a336 after 29 minutes.


FileNotFoundError: [Errno 2] No such file or directory: 'BFM/01_MorphableModel.mat'

When trying to run the examples
! python test.py --name=my_model--epoch=20 --img_folder=./datasets/examples

Full stack trace
Traceback (most recent call last):
  File "test.py", line 72, in <module>
    main(0, opt,opt.img_folder)
  File "test.py", line 41, in main
    model = create_model(opt)
  File "/content/Deep3DFaceRecon_pytorch/models/__init__.py", line 65, in create_model
    instance = model(opt)
  File "/content/Deep3DFaceRecon_pytorch/models/facerecon_model.py", line 95, in __init__
    is_train=self.isTrain, default_name=opt.bfm_model
  File "/content/Deep3DFaceRecon_pytorch/models/bfm.py", line 40, in __init__
    transferBFM09(bfm_folder)
  File "/content/Deep3DFaceRecon_pytorch/util/load_mats.py", line 34, in transferBFM09
    original_BFM = loadmat(osp.join(bfm_folder, '01_MorphableModel.mat'))
  File "/usr/local/lib/python3.7/dist-packages/scipy/io/matlab/mio.py", line 216, in loadmat
    with _open_file_context(file_name, appendmat) as f:
  File "/usr/lib/python3.7/contextlib.py", line 112, in __enter__
    return next(self.gen)
  File "/usr/local/lib/python3.7/dist-packages/scipy/io/matlab/mio.py", line 19, in _open_file_context
    f, opened = _open_file(file_like, appendmat, mode)
  File "/usr/local/lib/python3.7/dist-packages/scipy/io/matlab/mio.py", line 45, in _open_file
    return open(file_like, mode), True
FileNotFoundError: [Errno 2] No such file or directory: 'BFM/01_MorphableModel.mat'

I have copied files in BFM folder and model to checkpoints folders:

Solved downloading the file from [here](https://faces.dmi.unibas.ch/bfm/main.php?nav=1-2&id=downloads
).


Documentation site

Before we do the 1.0.0 we need to implement a documentation site. Something like mkdocs which is being used by lacuna.js.

Off the top of my head, here are a few options to explore: mkdocs, metalsmith, gollum, jekyll (would integrate nicely with Github), harp, assemble, docpad

After 40 minutes of looking I've realized something: I don't want to design my own site.
So, we're going with mkdocs.

mkdocs is working pretty well. I'm having an issue wit using it's deploying to Github Pages feature. Some issue with Python versions and Unicode. I'm looking to sort through this ASAP.


Enable Skia port on non-Apple platforms.

Currently, building Skia port is restricted to Apple platform. This patch enables Skia on non-Apple platforms. The location and library file format are platform dependent, I introduced 2 option variables to be set by users.

SKIA_LIBRARY_PATH
full-path specification of the skia library to be linked. if the user want to link skia.dll, SKIA_LIBRARY_PATH should point skia.dll.lib.
WIN32_OBJECT_TYPE
the option to be passed to MSVC or Clang-CL, to control the object type as sharable or non-sharable ("/MD" or "/MT"). This has no effect on non-Windows platforms.

The behaviour without setting them is same with non-patched CMakeLists.txt.
Testing of patched source tree is being done at CircleCI.
If you think such detailed CMake scripting should be out of main CMakeLists.txt and placed in cmake/ folder (as FindCairo.cmake module), please let me know.

Oops, checking of the version of cmake does not work well. I must update.

If you think such detailed CMake scripting should be out of main CMakeLists.txt and placed in cmake/ folder (as FindCairo.cmake module), please let me know.

Maybe we should, yes.

I moved the part for Skia library pathname & compiler flag to cmake/FindSkia.cmake. But I could not move the part to changing the compiler flag (for MSVC) to separated file (my attempt is like this 9ed60cc8a27786573ef3c4f945a49d0b93843f88). It seems that MSVC runtime library hook does not work well within the user-defined functions.

For Win32 testing of Skia, I use following items.

My Skia m70 headers & binaries (shared/static) at github.com/mpsuzuki/skia-binaries
Boost headers required for svg-native-viewer at my office. By using this, the installation process of Boost would be minimized, the consumed time is effectively reduced from 6-8min to 10-30sec. The developers using "free plan" of CircleCI, it might useful.

For Linux, I was using headers & binaries at:

My Skia m70 headers & binaries built on Ubuntu at my office
But, the different dependencies between Ubuntu and Debian is not easy for most developers (to use this binary, many library-development files should be installed). I should compile another set with smaller dependencies.

Umm, installation of genuine & fullsize Boost exceeded 10 min, and CI process is aborted. I would retry with rebased version, after PR#105 is merged.

By merged PR #105, the CI time for Windows is significantly reduced! Please let me merge.


Inconsistent behavior of batch mode and interactive mode

On this code:
module Test

let x : Type * int = (unit, 5)
unfold let first (#a:Type) (#b:Type) (x: (a * b)) : Tot a =
  match x with
  | (a, _) -> a
let z : first x = ()

The behavior of batch mode and interactive mode differs:
$ fstar.exe Test.fst
Unexpected error; please file a bug report, ideally with a minimized version of the source program that triggered the error.
Failure("(.\\Test.fst(3,8-3,12)) Impossible: Forced tk not present on Type")

vs
$ fstar.exe Test.fst --in
#push 1 0
module Test

let x : Type * int = (unit, 5)
unfold let first (#a:Type) (#b:Type) (x: (a * b)) : Tot a =
  match x with
  | (a, _) -> a
let z : first x = ()
#end #done-ok #done-nok

#done-ok

The blowup of the batch mode invocation is also an issue.

Removing the unfold keyword makes it go through even in the batch mode (on the c_relational-ci_r3 branch with the fix for #850).

Hey Aseem. This seems to work now in master after the removal of the tk field.
Want to check if some variation still blows up, or should we close this?


Bulk fix adding --scopes parameter

This is a bulk fix adding the --scopes parameter to az ad sp create-for-rbac commands.

Docs Build status updates of commit 01d2054:
:warning: Validation status: warnings

File
Status
Preview URL
Details

academic-services/knowledge-exploration-service/reference-makes-command-line-tool.md
:warning:Warning
View (>=makes-1.0)
Details

academic-services/knowledge-exploration-service/reference-makes-command-line-tool.md

Line 10, Column 1: [Warning: h1-not-first - See documentation] Markdown content is not allowed before H1 'MAKES command line tool (KESM)'.
Line 85, Column 52: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'MakesPackage' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 89, Column 53: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'MakesPackage' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 101, Column 44: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/azure/virtual-machines/windows/sizes' will be broken in isolated environments. Replace with a relative link.
Line 112, Column 75: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ApplicationSettingJsonPath' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 162, Column 47: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'MakesPackage' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 166, Column 42: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'MakesPackage' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 170, Column 45: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'MakesPackage' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 231, Column 40: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/azure/virtual-machines/windows/sizes' will be broken in isolated environments. Replace with a relative link.
Line 506, Column 68: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/azure/cloud-shell/quickstart' will be broken in isolated environments. Replace with a relative link.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

#sign-off
This is a bulk update that is being implemented cross-repo as az ad sp create-for-rbac no longer defaults to the current / active subscription.  You must provide a --scope parameter value.

#sign-off with sufficient time given for author review.
The Azure subscription no longer defaults to the current active subscription in the az ad sp create-for-rbac reference command.  You must now specify a subscription ID.


error An unhandled error occurred. Please check the console for more information.

error code
An unhandled error occurred. Please check
the console for more information.

error console
plugin:consistent-attachments-and-links:331 SystemError [ERR_FS_EISDIR]: Path is a directory: rm returned EISDIR (is a directory) /Users/XXX/Downloads/XXX/XXX/_attachments

@LOVECHEN need more details what exactly did you do before the issue occurred

@LOVECHEN Can you please provide a sample vault and video reproducing the issue?

When I delete the markdown file, the error message mentioned in this issue also appears. The error stack is as follows:
No active editor.
plugin:obsidian-attachment-name-formatting:3552 Formatting attachments...
plugin:obsidian-attachment-name-formatting:3554 Getting attachments list...
plugin:obsidian-attachment-name-formatting:3685 No attachments found...
plugin:consistent-attachments-and-links:579 RangeError: Maximum call stack size exceeded
    at new t (app.js:1:724942)
    at createTFolderInstance (plugin:consistent-attachments-and-links:464:16)
    at createTFolderInstance (plugin:consistent-attachments-and-links:465:23)
    at createTFolderInstance (plugin:consistent-attachments-and-links:465:23)
    at createTFolderInstance (plugin:consistent-attachments-and-links:465:23)
    at createTFolderInstance (plugin:consistent-attachments-and-links:465:23)
    at createTFolderInstance (plugin:consistent-attachments-and-links:465:23)
    at createTFolderInstance (plugin:consistent-attachments-and-links:465:23)
    at createTFolderInstance (plugin:consistent-attachments-and-links:465:23)
    at createTFolderInstance (plugin:consistent-attachments-and-links:465:23)

When I delete the markdown file, the error message mentioned in this issue also appears. The error stack is as follows:
No active editor.
plugin:obsidian-attachment-name-formatting:3552 Formatting attachments...
plugin:obsidian-attachment-name-formatting:3554 Getting attachments list...
plugin:obsidian-attachment-name-formatting:3685 No attachments found...
plugin:consistent-attachments-and-links:579 RangeError: Maximum call stack size exceeded
    at new t (app.js:1:724942)
    at createTFolderInstance (plugin:consistent-attachments-and-links:464:16)
    at createTFolderInstance (plugin:consistent-attachments-and-links:465:23)
    at createTFolderInstance (plugin:consistent-attachments-and-links:465:23)
    at createTFolderInstance (plugin:consistent-attachments-and-links:465:23)
    at createTFolderInstance (plugin:consistent-attachments-and-links:465:23)
    at createTFolderInstance (plugin:consistent-attachments-and-links:465:23)
    at createTFolderInstance (plugin:consistent-attachments-and-links:465:23)
    at createTFolderInstance (plugin:consistent-attachments-and-links:465:23)
    at createTFolderInstance (plugin:consistent-attachments-and-links:465:23)

My obsidian version is 1.6.7 (Installer 1.6.5).

I found that renaming file can also trigger this problem.

sample.zip
The above sample vault can reproduce this problem. You can trigger by rename the Hello folder.

Fixed in https://github.com/dy-sh/obsidian-consistent-attachments-and-links/releases/tag/3.9.0


MAINT, DOC: bump spyder-kernels

Once Spyder 4.1.4 hits the anaconda defaults channel, we'll want to update the install instructions to state that the base env should have spyder>=4.1.4, and update our environment.yml to require spyder-kernels>=1.9.2. This is necessary because on OSX Catalina (and presumably also on Big Sur) there is an issue with spyder running under py3.7 in base being unable to set the console to use a py3.8 executable (which currently happens by default when installing with our env file).
Check here periodically to see if the relevant versions are out yet:

https://anaconda.org/anaconda/spyder
https://anaconda.org/anaconda/spyder-kernels  (can use pip to get 1.9.2 if the conda release lags spyder 4.1.4)

Crossref spyder-ide/spyder#13252

One of the Spyder devs (@ccordoba12) just mentioned that they expect to release platform binaries of Spyder for Windows and macOS in September (it is already available in Linux via apt, though a very old version; currently 3.3.6 via apt, latest is 4.1.4). TBD whether this news actually makes our installation guidance for MNE-Python easier, depending on how tightly the platform binary integrates with the Anaconda Navigator GUI.  Here is what I think would be necessary to go back to installing MNE-Python into base env:

Anaconda stops installing Spyder into base by default once the platform binaries are available; OR
Anaconda Navigator detects if Spyder had been installed as a platform binary, and (in Navigator GUI) masks the env-specific Spyder launcher with a launcher for the binary; OR
We recommend that all users uninstall Spyder from base before installing MNE-Python & its dependencies, and recommend that afterward they install the Spyder platform binary if they want it; OR
We recommend new users install miniconda + Spyder platform binary (if they want it) so that Spyder isn't in base by default

Option 5 is to ignore the platform binaries, keep using the "modular approach" where Spyder is in base env and MNE-Python is in mne env, and tell users to set the executable in the Spyder GUI to use the version of Python in the mne env.
Opinions welcome.

Hopefully they go with (1). Seems like the easiest approach

(1) would definitely be the best approach. Also, both packages have now been bumped to the versions stated at the top.

Hey guys, I also think (1) would be the easiest option not just for your project but for novice users in general. But there's one important point you should be aware of: our installers will be provided by us and (most probably) they'll use pip packages. That's because the Python installer creation tools (like py2app, py2exe and Pyinstaller) work much better with pip packages.
However, I'll talk to the Anaconda guys to see if they can stop installing Spyder into base.


Having problems with Cors using the api in Ionic framework

Access to fetch at 'psnURL' from origin 'http://localhost:8100' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.
Have to install any plugins?

Sony themselves are responsible for this CORS policy, which Ionic is going to honor. For calling the endpoints client-side, you'll need to set up a Node.js API which uses psn-api that your Ionic app calls, rather than using psn-api directly within your Ionic app.


BUG fix my buggy run export suggestion

This PR fixes my buggy run export suggestion.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to re-render for you, but it looks like there was nothing to do.

This PR fixes my buggy run export suggestion.

@beckermr out of curiosity, was the old meta exporting the package as version(x.x.x)=buildNumber ? And now it'll just be version(x.x.x) ??

Yup! This is my hope!


Tests

Write tests for all the algorithms / computations.

Big +1 on this.  FWIW I find that I'm much more productive when writing tests.  Also the modular design of this library so far has the advantage that it should make this easier to test.  For example it will be convenient to establish confidence over the line search on single-machine operation before switching over to a distributed setting (where any bugs will be much more expensive to find)


adding ScaleFT (Okta ASA) and Caffeine

Hi folks,
We recently rolled out ScaleFT (Okta ASA), so I figured I would share the "recipe" I added to Installomator for it. Tested and confirmed that it works.
Also adding Caffeine, another tool we use. Also tested and confirmed that it works.
Cheers,
Greg

Closing this to separate out the PRs for individual apps. Sorry for the noise!


moveit.py - check for leading slashes

The new version will have slashes at the start of path, which is probably wrong as that signifies an absolute path. Anyhow prexisting manifests will fail cos the files are not identical: one has leadings slashes, the older ones do not. Either commit to one or the other (the root transfers will have a /) and add a check.

Remove the leading slashes. Hashcheck can't read them.


Application Insights for QnA Maker - Extracting confidence score

Hi,
I am currently using the application insights feature of the QnA maker resource in Azure and i am struggling to find a way to pull out the confidence score generated in the API return
I am trying to adjust the below query which is provided by microsoft in the link below but can not manage to figure it out.
https://docs.microsoft.com/en-us/azure/cognitive-services/qnamaker/how-to/get-analytics-knowledge-base
requests
| where url endswith "generateAnswer"
| project timestamp, id, name, resultCode, duration, performanceBucket
| parse kind = regex name with *"(?i)knowledgebases/"KbId"/generateAnswer"
| join kind= inner (
traces | extend id = operation_ParentId
) on id
| extend question = tostring(customDimensions['Question'])
| extend answer = tostring(customDimensions['Answer'])
| project KbId, timestamp, resultCode, duration, question, answer, id, performanceBucket

Thanks,

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

ID: 82bd07c0-ce21-9630-4aea-c93190c227fa
Version Independent ID: f1912f4f-df33-edf2-9efe-3b7ae9e77876
Content: Analytics on knowledgebase - Azure Cognitive Services
Content Source: articles/cognitive-services/QnAMaker/How-To/get-analytics-knowledge-base.md
Service: cognitive-services
Sub-service: qna-maker
GitHub Login: @diberry
Microsoft Alias: diberry

@rohandholakia1 Thank you for your query. We will investigate and update this thread further.

@diberry  Hi Dina, do you know if there any way to get all run and confidence score for it? Thank you.

This query worked for me. The score is inside the customDimensions column which is JSON so it has to be extracted.
 requests     | where url endswith "generateAnswer"     | project timestamp, id, name, resultCode, duration     | parse kind = regex name with *"(?i)knowledgebases/"KbId"/generateAnswer"     | join kind= inner (     traces | extend id = operation_ParentId     ) on id     | extend question = tostring(customDimensions['Question'])     | extend answer = tostring(customDimensions['Answer'])     | extend score = tostring(customDimensions['Score'])     | project KbId, timestamp, resultCode, duration, question, answer, score

Thank you very much for that @diberry, works well

resolved.
#please-close


[Feature Suggestion] Allow Drag and Drop from Sound Search to Playlist

Similar to the way Playlist Enhancer allows sounds to be dragged from one playlist to another, I would like to be able to drag sounds from the search directly into a playlist of my choice.

Thank you for you suggestion. This is now supported by moulinette-sounds (version 2.7.0)


SQL Results Panel is rendering very slow

Issue Type: Bug
Run a select statement to query a table with a few hundred or a few thousand records:
SELECT * FROM
Azure Data Studio version: azuredatastudio 1.9.0 (78a42e1d112ae3231777722b51eaf44f83ddbe55, 2019-07-10T04:31:36.998Z)
OS version: Windows_NT x64 10.0.18362

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-6260U CPU @ 1.80GHz (4 x 1800)

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

Load (avg)
undefined

Memory (System)
15.89GB (5.11GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (9)

Extension
Author (truncated)
Version

admin-pack
Mic
0.0.2

admin-tool-ext-win
Mic
0.0.1

agent
Mic
0.41.0

cms
Mic
0.3.0

dacpac
Mic
0.4.0

import
Mic
0.9.0

profiler
Mic
0.9.0

schema-compare
Mic
0.4.0

poor-sql-formatter
WSR
0.1.0

(1 theme extensions excluded)

Can you try to disable the user options resultsGrid.autoSizeColumns and see if that makes a noticeable difference for you

It turned out to be an issue with Windows desktop with dual monitors. I just reconfigured the dual monitor and the issue was gone.

I tried to disable resultsGrid.autoSizeColumns but it's still rendering very slow with just a thousand records. You can watch the attached recording below. (I zipped the mp4 screen recording because github doesn't allow attaching videos).
screen recording.zip

How many columns are in your result set?

I have like 68 of them. I guess that's making the results grid to render slow. But I also have another table with just 15 columns and 700 records in total and still takes time to render the results. Although it's just a couple of seconds delay and not really a big problem but I think its performance can still be improved.

We have known perf issues when it comes to large number of columns #4875 #5477 . We have some work streams on the back burner to increase the perf of the table overall.

Closing older issues.  This is a duplicate of several other active issues we're currently tracking for upcoming investigation.


runtime delegation to JFR streaming APIs via reflection

Adds support for accessing JFR streaming APIs at runtime via reflection, while still supporting language levels below Java 14, in a manner similar to the VirtualThreads class from https://github.com/palantir/nylon-threads.
This allows consumers to access JFR streaming APIs when running on Java 14 or above, while still targeting language levels below Java 14. The EventStreams and RecordingStreams classes provide factory methods for creating an object similar in shape to jdk.jfr.consumer.EventStream or jdk.jfr.consumer.RecordingStream respectively, which delegates to an underlying instance at runtime. If JFR streaming is not supported by the runtime, an Optional.empty() is returned instead.
Note that EventStreams does not provide support for the onMetadata method of jdk.jfr.consumer.EventStream, as it depends upon a class added in Java 16. Similarly, RecordingStreams does not provide support for jdk.jfr.consumer.RecordingStream#stop() as it was added in Java 20 and I'm making the assumption that most consumers are still targeting Java 17 or below.

Released 0.4.0


Fix loop overruns due to blocking Talon call

The periodic calls to resynchronize the encoders are currently blocking (up to 250ms), which can cause severe loop overruns depending on CAN traffic. Using a timeout of 0 disables blocking. This shouldn't cause any issues since the synchronize method is called periodically anyway.

Sorry, was a little confused about the three repos 🙃


Support loading instantiated data structures into the MerkleStore

As it stands there is not way to load an instantiated data structure (Mmr, Smt, MerkleTree) into the MerkleStore.  The MerkleStore API expects iterators of a specific format depending on the structure being loaded.  We should implement methods on Mmr, Smt, MerkleTree that produce the required iterators that will allow the instantiated data structures to be loaded into the MerkleStore.

My opinion is that we should make a trait for inner_nodes iterator, and implement a generic From trait for that.

Agreed - we can improve the API quite a bit. I like @hackaugusto's suggestion of having an iterator over inner nodes - though, I think we should have a dedicated method instead of (or maybe in addition to) implementing From. This method could look something like:
pub fn append<R, I>(&mut self, nodes: R) -> &mut Self
where
    R: IntoIterator<IntoIter = I>,
    I: Iterator<Item = InnerNodeInfo> + ExactSizeIterator,
{
    for node in nodes.into_iter() {
        debug_assert_eq!(
            Rpo256::merge(&[node.left.into(), node.right.into()]),
            node.value.into()
        );

        self.nodes.insert(
            node.value.into(),
            Node {
                left: node.left.into(),
                right: node.right.into(),
            },
        );
    }

    self
}

If we do this, we can probably get rid of most with_* and add_* methods.

Closed by #139.


Testing

Reviews in this chain:
└https://github.com/Skydio/revup/pull/161 Testing

#
head
base
diff
date
summary

0
9c2ed0d8
df295937
diff
Mar 12 16:32 PM
1 file changed, 1 insertion(+)


how to update 2.3 series from 2.2.1

Upgrade guide not include Upgrade to 2.3.0

For Updating From an older version to a newer version always check for the documentation and then follow the steps given there, this works most of the time. If not try searching on stackoverflow or google it. Further you can check Upgrade guide provided by developers.

Hello @lovejj1994 , the migration steps from 2.2.1 to 2.3 can be found here:
https://github.com/Kong/kong/blob/master/UPGRADE.md#upgrade-to-230
Please reopen if you have specific questions/problems when following these instructions.


Inline variable declarations

Inlined variable declarations across the tree, dogfooding the new fixer/analysis resulted in the following bugs:
https://github.com/dotnet/roslyn/issues/15994
https://github.com/dotnet/roslyn/issues/15993
https://github.com/dotnet/roslyn/issues/15992

tag @dotnet/project-system

Will, as soon as I can figure out how Editor Config support works.

I would say just add an entry in the .ruleset file in the build sub-tree, it will apply for our entire repo.

https://github.com/dotnet/roslyn-project-system/blob/master/build/Rulesets/Roslyn.ruleset


fix path to ida when load ida native version

as IDA pro 7.4 has both  python2 and python3 dir, it will get an error when use idapkg in IDA pro 7.4.

It seems good to me. Thank you!


[100 SigmaUSD] Add ERG/XAU pair datapoint source

Coingecko API should be able to show ERG in XAU prices. See discord - https://discord.com/channels/668903786361651200/669989266478202917/986627954568204320
See the existing NanoErgUsd datapoint source as an example - https://github.com/ergoplatform/oracle-core/blob/fcffd8a1c22900f99511d25f22e1b34ef0fdaf80/core/src/datapoint_source/erg_usd.rs#L8-L32
Add new datapoint source handling in the oracle config load - https://github.com/ergoplatform/oracle-core/blob/fcffd8a1c22900f99511d25f22e1b34ef0fdaf80/core/src/oracle_config.rs#L52-L56

I'll do this.


[ListView] Some props not having any effect

Here's my render method, a simple ListView (contained within a NavigatorIOS and TabBarIOS):
  render() {
    return (
      <ListView
        ref='listView'
        contentInset={{top: 20, bottom: 49}}
        contentOffset={{y: 0}}
        scrollIndicatorInsets={{top: 64, bottom: 49}}
        style={styles.listView}
        dataSource={this.state.dataSource}
        renderHeader={::this.renderHeader}
        renderRow={::this.renderRow}
        automaticallyAdjustContentInsets={false}
        showsVerticalScrollIndicator={true}
        onScrollEndDrag={::this.onScrollEndDrag}
        onScroll={(e) => console.log(e.nativeEvent.contentOffset.y)}
      />
    );
  }

The particular props I'm having issues with are contentOffset and scrollIndicatorInsets. Whatever values I pass these props, I see no changes upon reloading.
My other issue - for some reason the onScrollEndDrag prop function is not called on the iPhone 4S simulator. It is called on all the other sims I've tried...very odd.
Side note: I'm set up using webpack for hot reloads...and I can change the prop values on the fly and watch them work after live reloads. So I can load the view, notice the scroll indicators have not been adjusted...change "top" to 1000 (and watch it become wildly incorrect), then change it back to 64 and watch it become the way it should have been when it first loaded. Of course, if you don't have webpack set up you won't be able to reproduce this - but you should be able to reproduce the fact that the props are seemingly ignored.

Closing this - weird behavior from my react fork


fix: fixed incorrect arguments in rclcpp_service_callback_added

Description
fixed incorrect arguments in rclcpp_service_callback_added
Related links
https://tier4.atlassian.net/browse/RT2-1614
Notes for reviewers

Pre-review checklist for the PR author

[x] I've confirmed the contribution guidelines.
[x] The PR is ready for review.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR has been properly tested.
[ ] The PR has been reviewed.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 55.05%. Comparing base (75e1c3f) to head (5bdcb3f).
Report is 18 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##             main     #289       +/-   ##
===========================================
+ Coverage   25.02%   55.05%   +30.03%
===========================================
  Files          58       28       -30
  Lines        3321     2185     -1136
  Branches     1085     1212      +127
===========================================
+ Hits          831     1203      +372
+ Misses       1756      702     -1054
+ Partials      734      280      -454

Flag
Coverage Δ

differential
55.05% <100.00%> (?)

total
?

Flags with carried forward coverage won't be shown. Click here to find out more.

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

LGTM


Fixate readthedocs link for the point release

Signed-off-by: Vasilij Litvinov<EMAIL_ADDRESS>
What do these changes do?
This change is for 0.13.2 release branch, and its point is to make the link to documentation to always point to its documentation (not to the latest or stable or anything), so it keeps the same regardless of our further actions.

[ ] commit message follows format outlined here
[ ] passes flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
[ ] passes black --check modin/ asv_bench/benchmarks scripts/doc_checker.py
[ ] signed commit with git commit -s
[ ] Resolves #?
[ ] tests added and passing
[ ] module layout described at docs/development/architecture.rst is up-to-date
[ ] added (Issue Number: PR title (PR Number)) and github username to release notes for next major release

Do we need to do this? How many people do we expect will go to the readme for release branches and click on the badge links?

I have no idea TBH 😄 but this is what our release procedure says, which I'm just "following suit"...
I mean, we need a commit to put a tag on anyway, and having a tag in GitHub with pinned links to PyPI and docs is just right.

FYI: the real tests for the release are running off release-0.13.2 branch and are here: https://github.com/modin-project/modin/runs/5143499130?check_suite_focus=true

Should the link to Modin docs work after the changes are merged into release branch? At the moment I see docs unknown link on readme.


Versiones de android

Hi @robingenz  !
In advance I want to thank you for the plugin, it saved my app, I have a question regarding the Android versions.
I created an app and installed the plugin and created an APK that is not signed to be used as a third party, but it only works for versions 13 and 14, for versions 11 or lower it gives me an error and tells me that the module is not installed. the version I installed is the
"@capacitor-mlkit/barcode-scanning": "^5.3.0".
What version can I use so that it runs on devices with Android 11 or lower?
In those versions the rest of my application behaves as expected and only the scanner gives me errors.

Please take a look at this comment: https://github.com/capawesome-team/capacitor-mlkit/issues/38#issuecomment-1626891989
I will close this issue since it is not related to this demo app.


Implement schema command

We should expose the schema allowed for the porter manifest section of the helm mixins. IDEs can use this to provide autocomplete.
$ helm schema
{ json schema dump of the porter yaml section}

This is part of https://github.com/deislabs/porter/issues/76

Implemented in #7


Code from section "Reading a file" warns "deprecation"

The example uses import { copy } from "https://deno.land/std@0.167.0/streams/conversion.ts";, reporting the following warning:
@deprecated
(will be removed after 0.169.0) Import from std/streams/copy.ts instead.

It would be nice to upgrade it to use something like import { copy } from "https://deno.land/std@0.167.0/streams/copy.ts";.

This is removed, but still appears in the docs - need to cull

is this still appears in somewhere? I would be happy to help if you let me know the location.


Update cats-effect-testing-specs2 to 0.5.2

Updates com.codecommit:cats-effect-testing-specs2 from 0.4.2 to 0.5.2.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.codecommit", artifactId = "cats-effect-testing-specs2" } ]

labels: test-library-update, semver-minor

Superseded by #260.


Metric value

Codecov Report

Merging #16 (2a4fbf5) into main (3c4e65f) will increase coverage by 3.41%.
The diff coverage is 28.57%.

@@            Coverage Diff             @@
##             main      #16      +/-   ##
==========================================
+ Coverage   68.75%   72.16%   +3.41%
==========================================
  Files          14       15       +1
  Lines          96       97       +1
==========================================
+ Hits           66       70       +4
+ Misses         30       27       -3

Impacted Files
Coverage Δ

lib/src/models/metric_value.dart
0.00% <0.00%> (ø)

lib/src/models/metric_value_level.dart
30.76% <100.00%> (+30.76%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #16 (2a4fbf5) into main (3c4e65f) will increase coverage by 3.41%.
The diff coverage is 28.57%.

@@            Coverage Diff             @@
##             main      #16      +/-   ##
==========================================
+ Coverage   68.75%   72.16%   +3.41%
==========================================
  Files          14       15       +1
  Lines          96       97       +1
==========================================
+ Hits           66       70       +4
+ Misses         30       27       -3

Impacted Files
Coverage Δ

lib/src/models/metric_value.dart
0.00% <0.00%> (ø)

lib/src/models/metric_value_level.dart
30.76% <100.00%> (+30.76%)
:arrow_up:

Continue to review full report at Codecov.

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


fix: Update kubernetes-sigs nfs-provisioner

Changes
Update the client chart version for the nfs-subdir-external-provisioner from 4.0.8 to 4.0.18 so that we will pull the provisioner image from the registry.k8s.io container registry instead of the deprecated k8s.gcr.io container registry.
Testing

Scenario
DAC deployment tasks
Notes

Prior to chart version change
"baseline, cluster-logging, viya, install"
Filtered list indicates the only image pulled from k8s.gcr.io as: k8s.gcr.io/sig-storage/nfs-subdir-external-provisioner:v4.0.2

After chart version change
"baseline, cluster-logging, cluster-monitoring, viya, install"
All pods stabilized, sas and pg-storage related PVCs are bound with no errors. Filtered list shows the new CR location for the image above as: registry.k8s.io/sig-storage/nfs-subdir-external-provisioner:v4.0.2

The following command was used to create the list of image locations for containers present in the deployment:
kubectl get pods --all-namespaces \
-o jsonpath="{.items[*].spec.containers[*].image}" |\
tr -s '[[:space:]]' '\n' |\
sort |\
uniq -c

Information from Stephen:
Due to the pending depreciation of k8s.gcr.io and redirect to registry.k8s.io, we need to move to a newer version of kubernetes-sigs for the nfs-subdir-external-provisioner to get the image registry update. This is affecting some of our environments where we have stricter controls in place and the redirect is not working. Eventually the redirect will be removed as well, so it is probably a good time to do this.
More information:
https://kubernetes.io/blog/2023/03/10/image-registry-redirect/
https://www.reddit.com/r/devops/comments/11sm10g/deprecation_alert_mar_20_traffic_from_the_old/

Thanks @jarpat, good catch. Pushed a new commit for your review.


Unhandled Exception: Object has been disposed

my pubspec.yaml config

Hey, please be sure to always add a description on your issue, only the source code may not be enough to realise what is happening.
But on a quick look, I noticed that you are doing the sprite load on the same method that you are rendering it. Be sure to load it before hand, like on the game onLoad method for example and then on the render method, you just render it.

I solved it by changing it to this

~Thank u


Improve Code Structure

make a JSON file for the open source organization & use the es6+ method (like a map for displaying all open source organization details in index.js).
add a logo as well for particular open-source organizations
use grid & pagination for awesome user interface
improve read.md file related to task 1

so, the user can easily put data in json file instead of doing this hard coding.

Thanks 🤝 @harshal255 . I was thinking about the same for points 2 and 3. I am definitely going to work on all the points you have mentioned above.

Yeahh great🫡

Thanks 🤝 @harshal255 . I was thinking about the same for points 2 and 3. I am definitely going to work on all the points you have mentioned above.

Do you want to add a good read.md file??

Thanks 🤝 @harshal255 . I was thinking about the same for points 2 and 3. I am definitely going to work on all the points you have mentioned above.

Do you want to add a good read.md file??

Yes, You can go ahead 👍

okay brother 🤝💯

#9 Merged