Taking screenshot in afterscenario gets attached to next scenario

Environment (please complete the following information):

Node.js version:v10.13.0
NPM version: 6.4.1
webdriver.io version: 4.14.0
wdio-cucumber-framework version: ^2.2.8
wdio-multiple-cucumber-html-reporter version: ^0.3.2

Config of webdriver.io and the reporter
An example of how you configured the reporter in your webdriver.io config
afterScenario: function(scenario) {
    browser.deleteCookie();
    browser.saveScreenshot().then(function(stream) {
      var decodedImage = new Buffer(stream, 'base64').toString('binary');
      scenario.attach(decodedImage, 'image/png');
    });

    console.log('------------------------------------------------------');
    console.log('FINISHED SCENARIO NAME:' + scenario.name);
  },

Describe the bug
On failures or at the end of completion of scenario, we would like to take screenshot. when we do this, the screenshot doesn't get attached to the scenario that is completed, but instead to first step in next scenario
To Reproduce
Add the above code in afterscenario hook and have two tests. Make first test fail. On failure, the screenshot is getting attached to first step next scenario
Expected behavior
It is expected to attach to current scenario on the step where it failed.

Hi @spujar
Please use the the code as explained in the docs https://github.com/wswebcreation/wdio-multiple-cucumber-html-reporter/blob/master/README.md#how-do-i-add-screenshots-to-the-report
Going to close this one because it's working based on this code


Add exposure time tests

This PR adds tests that the exposure time is correctly set for event sequences, e.g. #622. Requires AcqEngJ 0.28.2

@henrypinkard heads up, the error in this run points to a problem in the bridge cleanup, I think: https://github.com/micro-manager/pycro-manager/actions/runs/5393365390/jobs/9792999973?pr=623. I know that the assertion error needs the AcqEngJ updates to pass, I just want to point out that there may be another cleanup issue also.

Ok I'll merge for now and then look into that

I wasn't able to reproduce this cleanup issue. That suggests it is some kind of race condition in the bridge. Something to keep an eye out for...


[BUG] base16 colorscheme not working despite successful build

Field
Description

Plugin
colorschemes.base16

Nixpkgs
nixos-unstable

Home Manager
follows nixpkgs

[x] I have read the FAQ and my bug is not listed there.

Description
I have enabled the apathy colorscheme with the base16 colorscheme plugin. My nixvim configuration builds without errors, however when i start neovim I get following error:
Error detected while processing /home/xqtc/.config/nvim/init.lua:
E5113: Error while calling lua chunk: /home/xqtc/.config/nvim/init.lua:66: module 'base16-colorscheme' not found:
        no field package.preload['base16-colorscheme']
        no file '/nix/store/5d2mg8la2b9haxk007nknv54l5qb97xf-luajit-2.1.1713773202-env/share/lua/5.1/base16-colorscheme.lua'
        no file '/nix/store/5d2mg8la2b9haxk007nknv54l5qb97xf-luajit-2.1.1713773202-env/share/lua/5.1/base16-colorscheme/init.lua'
        no file '/nix/store/5d2mg8la2b9haxk007nknv54l5qb97xf-luajit-2.1.1713773202-env/lib/lua/5.1/base16-colorscheme.so'
stack traceback:
        [C]: in function 'require'
        /home/xqtc/.config/nvim/init.lua:66: in main chunk

I have checked the luajit env path listed in the thrown error in the nix-store and there is indeed nothing related to the base16 plugin.

Minimal, Reproducible Example (MRE)

programs.nixvim = {
  enable = true;
  colorschemes.base16 = {
    enable = true;
    colorscheme = "apathy";
    settings.telescope_borders = true;
  };
}

My full system configuration can be found here

This seems to go deeper. Might not even be nixvim-related. Closing this.


Support JMX monitoring for worker pool status

Hello,
I implemented a feature to support JMX monitoring about following worker pool status;

Number of total workers
Number of active (processing) workers
Number of idle workers

How about this?

@moznion, I like the idea of being able to monitor the WorkerPool but I'm not a fan of introducing a dependency on M(X)Beans in the implementation. I've just checked in a commit that adds methods to see how many workers are working and idle; I'll let your application expose that info as an MBean.


打包成xcframework报错

描述

异常/需求具体描述
Undefined symbols for architecture arm64:
"OBJC_CLASS$_BUAdSDKConfiguration", referenced from:
in flutter_unionad22
"OBJC_CLASS$_BUAdSDKManager", referenced from:
in flutter_unionad21
"OBJC_CLASS$_BUAdSlot", referenced from:
in flutter_unionad8
"OBJC_CLASS$_BUDislikeWords", referenced from:
in flutter_unionad6
"OBJC_CLASS$_BUNativeExpressAdManager", referenced from:
in flutter_unionad8
"OBJC_CLASS$_BUNativeExpressAdView", referenced from:
in flutter_unionad8
"OBJC_CLASS$_BUNativeExpressBannerView", referenced from:
in flutter_unionad6
"OBJC_CLASS$_BUNativeExpressFullscreenVideoAd", referenced from:
in flutter_unionad13
"OBJC_CLASS$_BUNativeExpressRewardedVideoAd", referenced from:
in flutter_unionad18
"OBJC_CLASS$_BURewardedVideoModel", referenced from:
in flutter_unionad18
"OBJC_CLASS$_BUSize", referenced from:
in flutter_unionad16
"OBJC_CLASS$_BUSplashAd", referenced from:
in flutter_unionad19
"OBJC_CLASS$_MobClick", referenced from:
in fl_umeng4
"OBJC_CLASS$_UMConfigure", referenced from:
in fl_umeng4
"_swift_FORCE_LOAD$_swiftCompatibility50", referenced from:
_swift_FORCE_LOAD$swiftCompatibility50$_flutter_unionad in flutter_unionad5
"_swift_FORCE_LOAD$_swiftCompatibility51", referenced from:
_swift_FORCE_LOAD$swiftCompatibility51$_flutter_unionad in flutter_unionad5
"_swift_FORCE_LOAD$_swiftCompatibility56", referenced from:
_swift_FORCE_LOAD$swiftCompatibility56$_flutter_unionad in flutter_unionad5
"_swift_FORCE_LOAD$_swiftCompatibilityConcurrency", referenced from:
_swift_FORCE_LOAD$swiftCompatibilityConcurrency$_flutter_unionad in flutter_unionad5
"_swift_FORCE_LOAD$_swiftCompatibilityDynamicReplacements", referenced from:
_swift_FORCE_LOAD$swiftCompatibilityDynamicReplacements$_flutter_unionad in flutter_unionad5
ld: symbol(s) not found for architecture arm64

设备版本

本地flutter版本、设备型号版本

插件版本

插件版本号

日志

详细的log日志或者截图录屏

尝试本地导入sdk吧，没有尝试过再次打包xcframework


Automatically create toolchain update PRs

Description of changes:
This workflow will attempt to move the toolchain nightly pointer by one day every four hours. In doing so it will create a pull request for just this change. Once that PR is merged, further runs of this workflow will then try to move the pointer even further. (And, thus, running the workflow multiple times a day gives us a chance to catch up.)
Resolved issues:
n/a
Related RFC:
n/a
Call-outs:
n/a
Testing:

How is this change tested? Not at all - changes need to be merged to try this out.

Is this a refactor change? No.

Checklist

[x] Each commit message has a non-empty body, explaining why the change was made
n/a Methods or procedures are documented
[ ] Regression or unit tests are included, or existing tests cover the modified code
[x] My PR is restricted to a single feature or bugfix

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

I'm afraid I have to close this one and need to ask people to look at #2319 instead: I created the branch behind this PR in the main repo instead of my fork, and branch protections now keep me from pushing updates to this branch.


stickler on windows issues

I'm not sure if stickler is supposed to run on windows or not. It comes with .cmd wrappers for the stickler and stickler-server commands, but that could just be auto-gen code.
Anyway, after patching some binmodes into local.rb
tempfile = Tempfile.new(  "uploaded-gem.", temp_dir )
tempfile.binmode
tempfile.write( io.read )
and
def install_gem( spec, io )
  File.open( full_path_to_gem( spec ) , "w+" ) do |of|
    of.binmode
    io.each do |str|
      of.write( str )
    end
  end
end

and "deactivating" syslog in the logable.rb it seems to work.
But the question is, should it...?

@xrmb Yes, the .cmd wrappers are autogen by rubygems on installation. And those are totally reasonable suggestions. Stickler is long overdue for an update especially since it doesn't support the bundler api. so that may be a project coming up.


XDC constraints attributes on cells/ports missing from the netlists

While dealing with https://github.com/SymbiFlow/python-fpga-interchange/pull/108 I have bumped into one issue which has many ways of being solved IMO.
The problem is that the XDC constraints that add attributes do not reflect outside of nextpnr, as the XDC parser is embedded within. This means that the logical netlist won't have information, for instance, on the IOSTANDARD attribute of a pad, unless this is specified in the design.
The attributes are quite important when, for instance, writing the FASM features.
Some of the possible solutions:

enhance the physical netlist schema to include information on additional attributes assigned to the cells during p&r (e.g. when reading the XDCs)
read the XDC file once again alongside with logical and physical netlists
add the attributes when generating the logical netlist and read the XDC at that step

Another option would be to write out a new logical netlist from nextpnr.
In the long-long term, this would be needed if a place and route tool wanted to perform optimisations that manipulate the netlist, such as retiming, resynthesis, or control set remapping.


libfuzzer.h should not pull in fuzzer libraries

Regular dev build that does not need /Fuzzer should succeed without needing additional VS components such as address sanitizer. Since libfuzzer.h does #pragma comment(lib, "libsancov.lib") - that pulls in new dependencies that are not present unless "C++ AddressSanitizer" component is installed in VS.
This #prgama should be removed and all binaries that require the fuzzer libraries should include $(SanitizerLibs) in the <Link> section of vcxproj files.

This goes in pair with #1815

The docs now underscore to install the ASAN component (installed as default). This can be considered as the workaround in the meanwhile.


Add Cygwin setup to known location (usable by later GHA steps)

As discussed in #38 .

Oh, I now understand what you meant. This looks good.

Squashed and rebased to 'master'.

Thanks @rivy!


Retrieve logs from artifact storage if configured

Logs are from K8s pods. If pods are deleted, the logs are gone. We can potentially retrieve it from artifact storage if that’s configured.

Actually this is already supported.


Implement support for stratum 2.0

Description
As an operator, I want Stratum to support Stratum 2.0.
Acceptance Criteria

Explore the Stratum 2.0 specification and present how it can be best implemented with Besu.
Implement support for Stratum 2.0.

PoW is now deprecated so we won't be adding features to it


New Config Menu Clipping

Addon Version
v0.3.16-4
World of Warcraft Version
Retail
Describe the bug
My UI scale is 90% and the new config menu clips at the bottom
https://imgur.com/a/4n87Tmi
I only have one plugin installed: BetterBags - Openable Items
Steps to reproduce
It is this way every time I open the config menu

I think I figured out the issue. It stemmed from the plugin I had not being updated to work with the new config menu; once that plugin updated (Better Bags: Gear First), then the config menu stopped clipping


[Source Request]

Best Indonesian anime website
Source website

Name: Gomunime
Link: https://<IP_ADDRESS>/
Language(s): Indonesia

Other details

Duplicate of #187


⚠ Bofrost.de has degraded performance

In 8258bcc, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 968 ms

Resolved: Bofrost.de performance has improved in 4e11342 after 6 minutes.


Exclude vague locations from Mappable::CollapsibleCollectionOfObjects

Joe demonstrated in discussion #2088 that a large vague location like "USA" will completely obscure more specific observation data on a map, when the vague location gets grouped with more specific locations.
It's really hard to control the order of grouping (it would require calculating each location's surface area from lat/lng — possible but currently confusing) and ordering by this derived value.
@pellaea proposed just tossing the big locations from mapsets, because they will always obscure better data. I believe this is the right idea. It immediately improved the test map I used for debugging, "observations of name 5066":

localhost:3000/names/5066/map

mushroomobserver.org/names/5066/map

The "1º rule" does however obscure some locations closer to the poles that may be considered marginally useful.
Note here that there are no observations shown in Denali National Park or Peace River Area, on this branch:

but these do currently appear on the live site:

coverage: 94.405% (+0.002%) from 94.403%
when pulling 89d296ccf3867e25dbc8751c8ea829629459004c on nimmo-exclude-vague-locations-from-maps
into 1902cc4ad06d0faeaa1831f948d98b032c4cc49e on main.

Ok - I've updated the PR to calculate a consistent area when trying to determine if a location is small enough to be considered "not vague". It uses the proper formula for the area of a spherical patch, converting degrees to radians.
My arbitrary area is 24,000 sq. km. This includes "Peace River" but excludes "Denali".
Doubling this to 48,000 sq. km will include "Denali", but also "Massachusetts".
Feel free to help me decide what the area should be!

Thanks for thinking this through and doing the PR.

Added the following message to show obs, under the location, if the location is vague:

This observation's location will not appear on maps of this taxon because it is too vague.

If the current user is the observation owner, it also says this:

If you would like it to be included in maps, choose a more precise location.

Also added a test that these messages appear if appropriate.


feat: improve detecting missing translations

PR Type

[x] Feature
What Is the Current Behavior?

Missing translations are silently replaced with the ones from the ngx-translate default lang or just the translation key is inserted
The default lang is always pulled for client applications

What Is the New Behavior?

Missing translations trigger an error in the console, so that they can be detected (SSR logs, Sentry, Browser console)
Pulling default fallbacks can be customized
final SSR locale is transferred to the client to be used for initialization

Does this PR Introduce a Breaking Change?

[ ] Yes
[x] No

wait for integration of #655

Hi Danilo, thanks for the PR.
During our localization rework (#695) we adopted some of your SSR-specific code. Thank you very much for your contribution.
The missing translation handling is definitely a nice-to-have but not a priority for us right now. You're very welcome to rebase and integrate your changes with the current internationalization module and I will look at it ASAP! Otherwise, we will close this PR and track the enhancement ourselves.

@MaxKless I went for the rebasing. Please have a look 😄


BUG: Responsive issue for the settings page

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
The settings page is not entirely responsive, please let me work on it as a contributor for gssoc 2024!

Record

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

@vilasini-v fine.

please assign it to me


Adding iterable testing

This would help some IDL testing like FetchHeaders or NodeList

This is still waiting on review.


[Feature] GCS download parallelism support

Describe the bug
Currently GCS data is downloaded in a single thread in Flytekit. gsutil has support for parallel downloads, that should be leveraged.
Expected behavior
Faster downloads
Flyte component

[ ] Overall
[ ] Flyte Setup and Installation scripts
[ ] Flyte Documentation
[ ] Flyte communication (slack/email etc)
[ ] FlytePropeller
[ ] FlyteIDL (Flyte specification language)
[x] Flytekit (Python SDK)
[ ] FlyteAdmin (Control Plane service)
[ ] FlytePlugins
[ ] DataCatalog
[ ] FlyteStdlib (common libraries)
[ ] FlyteConsole (UI)
[ ] Other

To Reproduce
NA

/assign jeevb

Thank you @jeevb for the implementation

Done


Comparisson with other frameworks

I'll like to see a comparison between this library and other approaches followed by some frameworks, like react or solid.js (the latter being a simplified version of the former).
I know that solid for example doesn't have different priorities for its scheduler, I'll like to know how much it benefits to do the distinction between 'user-visible' and 'background' in your approach.

Yeah, I think that a comparison more on technicalities would be good, code examples would be more difficult to do since they are not directly comparable (as you mentioned).
I noted for example that your API is more centered about utility functions for more granular control (isTimeToYield and yieldControl).
Solid's one is more like a queue of task that are resolved ASAP without priority distinction, and you can only give it tasks to enqueue (not in the public API I think).
It would also be cool to know how the Prioritized Task Scheduling API could be used for the same purpose.
I did some digging into this rabbit hole since I wanted to create my own scheduler for a personal project, I took some inspiration from this library (thank you in advance), as well as Solid's one, and tried to mix it with some ideas from Idle Until Urgent strategy.

Great ideas. Thanks! I can work on some of those. Maybe first, I want to improve the library in general. I have ideas for a refactoring and better scheduling techniques in general.
What are you working on? Would you like to share it or is it something for private use?

Sounds interesting, I'll be paying attention on that.
In my case I find it fun to work on complicated, sometimes low-level things to code.
So... why not to make an entire javascript framework as a side project? there are not many of them out there (/s).
I worked on how to schedule the tasks it may perform, In my case I have 4 priorities, from highest to low. I use the highest priority for DOM updates and the other tree depending the case.
My API is something like:
export type Priority = Low | Medium | High | Highest;
export type Process = Generator<VoidFunction>

function batchProcess(process: () => Process, priority: Priority): Promise<void>

A Process is a bunch of tasks that needs to be executed, I went for a Generator since I found it more practical.
The highest priority executes as many tasks as possible using MessageChannel and checking if a yield is necessary.
The other 3 priorities only execute one task at a time and yields, it uses the Prioritized Task  Scheduling API (if available), or uses Promises that are resolved depending on the priority:

Hight: queueMicrotask.
Normal: requestAnimationFrame.
Low: requestIdleCallback (if available) or setTimeout.

Dunno if it's the optimal way to do it, but I'm pretty happy with the result (wouldn't mind some opinions)

Also, please add a link to your repo here in the comment if you decide to open-source your implementation. If it's in a bigger project, point to the part that handles scheduling.


Protobuf patch has trailing whitespace triggering warning.

Applying patch in Windows:
aregm@Barmaley MINGW64 /c/Projects/pysamprof (master)
$ cd 3rd_party/protobuf-c && git apply ../protobuf-c-vs2008-support.patch
../protobuf-c-vs2008-support.patch:36: trailing whitespace.
../protobuf-c-vs2008-support.patch:39: trailing whitespace.
warning: protobuf-c/protobuf-c.h has type 100644, expected 100755
warning: protoc-c/c_message.cc has type 100644, expected 100755
warning: 2 lines add whitespace errors.

Should be fixed with 02f760d6c37ef811f59633bcd67fd6a48886e62a

Should be fixed by 02f760d6c37ef811f59633bcd67fd6a48886e62a.


refactor: control default entities display resolution

Just some minor adjustments to some sensor entities that showed a large number of digits after the decimal in the default display. There may be more sensors to "tame" than the ones I changed (among non active ones possibly).
I notice, as I create this PR, that the VERSION = "1.0.0" has been changed to VERSION = "2.1.26" and this is probably a change that should be thrown away (it likely comes from tests I did on my HA instance before submitting the PR).
Thanks to all contributors for their efforts!

Don't worry about the tests, I need to fix them.

I'm a newbie when it comes to PR management on GitHub. I reverted the version back to its original format; committed and pushed. It seems to be automagically inserted in the PR. Just tell me if I'm doing wrong (my apologies if so).

You're doing fine, that's exactly how you do it.

Thanks!


[Pulsar] Fix available zone not configured and clean up the terraform script

Motivation
Recently when I modified the terraform.tfvars to use AWS CN and find the deployment failed.

Error: Error launching source instance: Unsupported: Your requested instance type (t2.small) is not supported in your requested Availability Zone (cn-north-1d). Please retry your request by not specifying an Availability Zone or choosing cn-north-1a, cn-north-1b.

However, I've configured the az variable with cn-north-1a (not cn-north-1d) in terraform.tfvars. And then I find the az is never used. It should be specified when creating a subnet.
In addition, when I ran terraform init, I found some warnings:
**Warning:** **Interpolation-only expressions are deprecated**

 on provision-pulsar-aws.tf line 2, in provider "aws":
  2:  region = "${var.region}"

Terraform 0.11 and earlier required all non-constant expressions to be
provided via interpolation syntax, but this pattern is now deprecated. To
silence this warning, remove the "${ sequence from the start and the }"
sequence from the end of this expression, leaving just the inner expression.

**Warning:** **Version constraints inside provider configuration blocks are deprecated**

 on provision-pulsar-aws.tf line 3, in provider "aws":
  3:  version = "3.50"

Terraform 0.13 and earlier allowed provider version constraints inside the
provider configuration block, but that is now deprecated and will be removed
in a future version of Terraform. To silence this warning, move the provider
version constraint into the required_providers block.

They are not serious problems but can be improved.
modifications
The most important change is importing the az variable in terraform script and apply it to the availability_zone property of aws_subnet resource.
The cleanup changes include:

Move the provider version constraint into the required_providers block (see aws example usage).
Use HCL2 semantics, i.e. change "${name}" to name.
Add output for public and private IPs of all VMs.

Finally, run terraform fmt to format terraform scripts. It involves many changes but IMO it is worth it.

After applying this patch, we can see the cached available zone was changed.
  # aws_subnet.benchmark_subnet must be replaced
-/+ resource "aws_subnet" "benchmark_subnet" {
      ~ arn                             = "arn:aws-cn:ec2:cn-north-1:146097325273:subnet/subnet-0fe3ea3f9a0829d35" -> (known after apply)
      ~ availability_zone               = "cn-north-1d" -> "cn-north-1a" # forces replacement


Feat: Allow to extend knex query builder

This PR allows to extend query builder with custom functions such as paginate or onDuplicateUpdate that will allow add custom behaviour or specific dialect methods over the basic query builder functions.
knex('products')
    .where('price', '<', 20)
    .paginate(15, 1, true)
    .then(paginator => {
        console.log(paginator.current_page);
        console.log(paginator.data);
    });

knex()
  .insert({ id: 1, name: 'foo' })
  .into(customers)
  .onDuplicateUpdate('name');

// which will generate

Insert  into customers (id, name) values (1, 'foo') on duplicate key update name=Values(name)

Currently there is no way to properly extend the QueryBuilder without monkey-patching the QueryBuilder.prototype.
closes #1158

This feature is wanted and it has been discussed before. I want to research earlier discussions and review the code, since for this I want knex to have carefully designed api and functionality.
This fundamental features should have design discussions before doing PRs or to be prepared to redo the whole implementation in a different manner.

Since my implementation is small one i would happy to redo it.
I'm open to discussion 😊

TBH, this seems a simple and non-intrusive solution, so I would vote for including this implementation.
My only concern is about TypeScript part of the equation - my assumption is that since QueryBuilder is part of Knex namespace, it should be possible to augment its interface locally and have additional methods on it. I wonder if it's possible to add test for this in https://github.com/tgriesser/knex/blob/master/types/test.ts
One additional suggestion: it might make sense to throw a warning when you are overriding existing method using .extend()

Related issues:
https://github.com/tgriesser/knex/issues/2071
https://github.com/tgriesser/knex/issues/1579
https://github.com/tgriesser/knex/issues/1456
https://github.com/tgriesser/knex/issues/1158

There doesn't seem to be many valuable proposals in prior issues, TBH. I'd be more inclined to give the tooling by which a user could implement addOperator as a plugin. from #1456 doesn't seem feasible int he nearby future, and, frankly, sounds like an over-engineering for simple cases where this particular three-lines long solution would do just fine.

TBH, this seems a simple and non-intrusive solution, so I would vote for including this implementation.
My only concern is about TypeScript part of the equation - my assumption is that since QueryBuilder is part of Knex namespace, it should be possible to augment its interface locally and have additional methods on it. I wonder if it's possible to add test for this in https://github.com/tgriesser/knex/blob/master/types/test.ts
One additional suggestion: it might make sense to throw a warning when you are overriding existing method using .extend()

I can check, TS by design allows to extend interfaces, but there some case it won't work.
I'm already using that at https://github.com/felixmosh/knex-on-duplicate-update/blob/master/types.d.ts
I will add a test for this.

There doesn't seem to be many valuable proposals in prior issues, TBH. I'd be more inclined to give the tooling by which a user could implement addOperator as a plugin. from #1456 doesn't seem feasible in he nearby future, and, frankly, sounds like an over-engineering for simple cases where this particular three-lines long solution would do just fine.

Agree, it can be a base for more pluggable solution (if needed).
What do you think regarding exposing QueryBuilder constructor on knex?

@kibertoad I'm not sure how to test the type augmentation of the QueryBuilder extension, can you give me some pointers?

@felixmosh Regarding testing: not sure, never tested for this case myself, but from my understanding it is possible to assert literal types for specific variables
https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/node/v8/node-tests.ts
e. g. // $ExpectType <T extends Function>(fn: T, message: string) => T
Not sure if it's going to work, but worth a try.

Sounds like only the issue #1158 had some really meaningful discussion around the area. Right now I think that the only drawback with this extension method is to be able to nicely override old methods with extended functionality. Being able to call original implementation of rewritten method sounds like something that I would like to have (I've used that quite a lot with objection).
I think that for now we could add this way that just allows to add stuff to prototype and pass it around to other builders as experimental extension support (to be able to deprecate it fast if it is not sufficient). From that experience we can decide later on if we actually want to make extending to work with real inheritance and setting then query builder instances to knex in a bit same way that it was suggested here https://github.com/tgriesser/knex/issues/1158#issuecomment-176093904 and how it was implemented for objection).

Nope:
src/transaction.js
src/query/joinclause.js (TBH, join builder is a farily different beast, I'm not sure querybuilder methods make much sense there. @elhigu, what use-case did you have in mind?)

For transaction all the same methods should be found that are found from main query builder... so all use cases I suppose :)
For join builder and subquery builder probably doesn't need to have all the extensions... I suppose extender method could have opts : { queryBuilder: true, joinBuilder: true, subBuilder: true } object to tell if what we are extending (I don't remember if there really is a separate subquery builder though).

Then another thing which needs testing is to check that instance created with knex.withUserParams({customUserParam: 'table1'}); works fine with extended functionlality

@elhigu Given that joinBuilder has methods along the line of
  – onIn
  – onNotIn
  – onNull
  – onNotNull
  – onExists
  – onNotExists
  – onBetween
  – onNotBetween

, is that really related to extending QueryBuilder per se? Probably there could be separate method exposed for extending just the JoinBuilder, but I would say that would be a separate PR altogether.

Separate PR sounds fine if that even is ever needed. I agree that it doesn't sound really important.

Sorry for the confusion, what are the action items? :)

@felixmosh

Add test for using custom methods from transaction (const trx = await knex.transaction; trx.customMethod())
Add test for using custom methods from knex copy with user params (const knewWithParams = knex.withUserParam({}); knewWithParams .customMethod())
Add documentation in https://github.com/knex/documentation
Try adding a test in https://github.com/tgriesser/knex/blob/master/types/test.ts that would assert interface augmentation to work.
This one was suggested by @elhigu -> possibility to override existing methods while preserving reference to original methods somehow (I guess those could be reattached to original object using some prefix or container). Not sure if this works correctly for custom methods attached to QueryBuilder, needs a try. I would imagine having something like

function where() {
console.log('log stuff')
this.original.where(...args) // or this.originalWhere(...args)
}

knex.QueryBuilder.extend('where', where)

@felixmosh

Add test for using custom methods from transaction (const trx = await knex.transaction; trx.customMethod())
Add test for using custom methods from knex copy with user params (const knewWithParams = knex.withUserParam({}); knewWithParams .customMethod())
Add documentation in https://github.com/knex/documentation
Try adding a test in https://github.com/tgriesser/knex/blob/master/types/test.ts that would assert interface augmentation to work.
This one was suggested by @elhigu -> possibility to override existing methods while preserving reference to original methods somehow (I guess those could be reattached to original object using some prefix or container). Not sure if this works correctly for custom methods attached to QueryBuilder, needs a try. I would imagine having something like

function where() {
console.log('log stuff')
this.original.where(...args) // or this.originalWhere(...args)
}

knex.QueryBuilder.extend('where', where)

Thanx, will work on it at the evening.
Regarding point #5, I think that this should be a separate feature, currently it is not allowed to override existing methods.

@felixmosh Sounds good, as long as you think that could be introduced later in a non-breaking way.

Lets mark this support experimental, so it will be ok to break when we have more user experience about how it should work.

Agree, how do i mark it as experimental?

Just by mentioning it in documentation

Unfortunately, this needs to be updated to accomodate for /src -> /lib movement of knex files. Should be fairly easy fix, though.

No problem, i will fix that at the weekend.

LGTM!
@elhigu @lorefnon What do you think?

A documentation PR https://github.com/knex/documentation/pull/218

@lorefnon What do you think?

I am personally not a huge fan of multiple libraries extending shared context, and in general, one import potentially changing the outcome of another import (across files).
I think this will also cause jump to definition to not work (even in a pure typescript codebase - it will just jump to the declaration which would have to be separately defined).
But given that people see value in this, and also because the current knex codebase is already not very statically explorable (because of not using ES6 classes in some places, extensive aliasing etc. features like jump-to-definition, ide outline etc. often don't work), hence I don't have a very strong opinion here.

Released in 0.19.1


No ability to define specific tester groups for TestFairy action

Issue Description
There is no way to define the list of tester groups for an upload of a TestFairy build to notify too.  This is useful so that the correct people are notified instead of none. Specifically we have multiple apps that we build and push to TestFairy, this allows us to only send updates to the correct testers for only the apps they should get notifications from.
Currently the 'testers_groups' option does not exist for the TestFairy action used below.
Configuration Files
Fastfile:
lane :testfairy do
    ENV["MATCH_FORCE_ENTERPRISE"] = "1"
    match(type: "enterprise")

    gym(scheme: "MyScheme")

    testfairy(api_key: ".......", comment: "I like turtles", testers_groups: "people-i-like-to-spam")
  end

Environment
fastlane version (run fastlane -v): fastlane 1.81.0
Do you use bundler to execute fastlane (i.e. bundle exec fastlane)? no
Do you use a Ruby environment manager (e.g. chruby, rbenv, rvm)? no
#4210

Thanks for opening up this proposal for a pull request. I'm going to mark this as 'help wanted' so people can take a look. I'm also going to add this to our internal review so we can let you know what we think.
Thanks! 👍


Support for rendering secondaryCta in variant

Note that this is based on #66, so the base branch should be updated once that PR is merged.
Adds support for a secondary button.
Storybook preview:

Hey @andre1050 thanks for fixing that conflict. I rebased on master to fix another one which crept in after the article view history changes were merged. I also made primary the default priority like you suggested - good idea.

Hi! 👋
I've noticed that the button styles you're using are not featured in the design system, which we are currently addressing. We think something like the pairing you're using features quite commonly around our products and deserves to be added.
In the meantime, rather than maintain a copy of design system buttons, would it be useful if you could use one of the buttons in the design system and override the background colour, hover colour and text colour? Do you think something like this would get you what you need, until we get around to supporting buttons of this colour?
Let me know if you need a hand or if you have any questions 🙂

👋 Hey @SiAdcock! Thanks for the suggestion, that sounds great. I'll give it a go on this PR and shout if I have any issues.

Closing as #77 also implements secondary CTA functionality.


Flags on Native wallet causing it to crash

Overview
When selecting some of the feature flags on native wallet it causes a black screen to appear, alternatively it just loads the dashboard for ever.
Flags that are causing a black screen: ThorChain and Osmosis.
Flags causing the dashboard to load indefinitely: Multi-Account, LP and Fox Farming. (Still is also happening on MM for these flags)
Noticed that if you turn the flag on in KK that you can then switch to native and it will stay on and not cause the screen to crash.
References and additional details

Log into release.shapeshift and connect to Native wallet
Turn on the feature flag of either Omsosis or ThorChain and notice the screen goes black
Refresh and turn on mutli account, LP or Fox Farming flag and notice after applying the flag the dashboard just keeps loading.
Switch to KK and turn on any of the flags
Switch back to native and notice behavior is as expected.

Acceptance Criteria
Turning flags on should work as expected and not cause a black screen or have the dashboard load continuously.
Need By Date
No response
Screenshots/Mockups
No response
Ownership

[X] If my bounty needs engineering or needs product I have added the respective labels on the right
[X] As the sponsor of this bounty I will review the changes in a preview environment (ops/product) or review the PR (engineering)

Estimated effort
No response
Sponsor / Stakeholder
No response
Bounty Hunters

Join our discord
Include an expected timeline for you to complete work in the work plan when you apply for this bounty!
Please refer to this link for some basic info
Please do not start work on this issue until you are approved in Gitcoin.

closing as we shouldn't be toggling flags at runtime, especially chain related flags, plz test environments per the built version


Theme classes broken

Hi, I've been trying to run the "Quickstart for dynamic execution and server-driven UI" from the /readme.md.
I get the following error when trying to build the app:
../../../../../AppData/Local/Pub/Cache/hosted/pub.dev/flutter_eval-0.7.5/lib/src/material/text_theme.dart(108,7): error GC6690633: No named parameter with the name 'headline1'. [C:\flutter\build\windows\x64\flutter\flutter_assemble.vcxproj]
../../../../../AppData/Local/Pub/Cache/hosted/pub.dev/flutter_eval-0.7.5/lib/src/material/text_theme.dart(195,34): error G4127D1E8: The getter 'headline1' isn't defined for the class 'TextTheme'. [C:\flutter\build\windows\x64\flutter\flutter_assemble.vcxproj]
../../../../../AppData/Local/Pub/Cache/hosted/pub.dev/flutter_eval-0.7.5/lib/src/material/text_theme.dart(198,34): error G4127D1E8: The getter 'headline2' isn't defined for the class 'TextTheme'. [C:\flutter\build\windows\x64\flutter\flutter_assemble.vcxproj]
../../../../../AppData/Local/Pub/Cache/hosted/pub.dev/flutter_eval-0.7.5/lib/src/material/text_theme.dart(201,34): error G4127D1E8: The getter 'headline3' isn't defined for the class 'TextTheme'. [C:\flutter\build\windows\x64\flutter\flutter_assemble.vcxproj]
../../../../../AppData/Local/Pub/Cache/hosted/pub.dev/flutter_eval-0.7.5/lib/src/material/text_theme.dart(204,34): error G4127D1E8: The getter 'headline4' isn't defined for the class 'TextTheme'. [C:\flutter\build\windows\x64\flutter\flutter_assemble.vcxproj]
../../../../../AppData/Local/Pub/Cache/hosted/pub.dev/flutter_eval-0.7.5/lib/src/material/text_theme.dart(207,34): error G4127D1E8: The getter 'headline5' isn't defined for the class 'TextTheme'. [C:\flutter\build\windows\x64\flutter\flutter_assemble.vcxproj]
../../../../../AppData/Local/Pub/Cache/hosted/pub.dev/flutter_eval-0.7.5/lib/src/material/text_theme.dart(210,34): error G4127D1E8: The getter 'headline6' isn't defined for the class 'TextTheme'. [C:\flutter\build\windows\x64\flutter\flutter_assemble.vcxproj]
../../../../../AppData/Local/Pub/Cache/hosted/pub.dev/flutter_eval-0.7.5/lib/src/material/text_theme.dart(213,34): error G4127D1E8: The getter 'subtitle1' isn't defined for the class 'TextTheme'. [C:\flutter\build\windows\x64\flutter\flutter_assemble.vcxproj]
../../../../../AppData/Local/Pub/Cache/hosted/pub.dev/flutter_eval-0.7.5/lib/src/material/text_theme.dart(216,34): error G4127D1E8: The getter 'subtitle2' isn't defined for the class 'TextTheme'. [C:\flutter\build\windows\x64\flutter\flutter_assemble.vcxproj]
../../../../../AppData/Local/Pub/Cache/hosted/pub.dev/flutter_eval-0.7.5/lib/src/material/text_theme.dart(219,34): error G4127D1E8: The getter 'bodyText1' isn't defined for the class 'TextTheme'. [C:\flutter\build\windows\x64\flutter\flutter_assemble.vcxproj]
../../../../../AppData/Local/Pub/Cache/hosted/pub.dev/flutter_eval-0.7.5/lib/src/material/text_theme.dart(222,34): error G4127D1E8: The getter 'bodyText2' isn't defined for the class 'TextTheme'. [C:\flutter\build\windows\x64\flutter\flutter_assemble.vcxproj]
../../../../../AppData/Local/Pub/Cache/hosted/pub.dev/flutter_eval-0.7.5/lib/src/material/theme_data.dart(122,7): error GC6690633: No named parameter with the name 'bottomAppBarColor'. [C:\flutter\build\windows\x64\flutter\flutter_assemble.vcxproj]
../../../../../AppData/Local/Pub/Cache/hosted/pub.dev/flutter_eval-0.7.5/lib/src/material/theme_data.dart(158,35): error G4127D1E8: The getter 'bottomAppBarColor' isn't defined for the class 'ThemeData'. [C:\flutter\build\windows\x64\flutter\flutter_assemble.vcxproj]

I believe it's related to this: https://docs.flutter.dev/release/breaking-changes/3-19-deprecations
I'm using Flutter v3.22

Fix is on it's way

Fixed in v0.7.6


Re-add get-codeowners.ps1

Re-Adding get-codeowners.ps1 to unblock cpp.

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

+1 assuming this is just a re-add as you say, but as a reviewer with less context it would be helpful to have a description of and/or links to the blocking issue, etc.

+1 assuming this is just a re-add as you say, but as a reviewer with less context it would be helpful to have a description of and/or links to the blocking issue, etc.
Its needed at https://github.com/Azure/azure-sdk-for-cpp/blob/main/eng/pipelines/templates/stages/archetype-cpp-release.yml#L197

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow


after call AfterInsert

defer handleAfterInsertProcessorFunc(bean)

Codecov Report

Merging #750 into master will increase coverage by 0.01%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master     #750      +/-   ##
==========================================
+ Coverage    51.3%   51.31%   +0.01%
==========================================
  Files          38       38
  Lines        8526     8526
==========================================
+ Hits         4374     4375       +1
  Misses       3669     3669
+ Partials      483      482       -1

Impacted Files
Coverage Δ

session_insert.go
57.77% <50%> (ø)
:arrow_up:

xorm.go
67.6% <0%> (+1.4%)
: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 109cb1a...399f0e0. Read the comment docs.

Could you add some tests?


Version 1.6 Future Milestone - More Lists Included

I'm testing an update internally that adds more lists that is slated release as version 1.6 in around 2 weeks time.
Update: Currently in testing the list now combines 41 lists.
Stay tuned!

Testing 1.6 is coming along nicely. It may be publicly released earlier than I expected in around 1 week from now barring any bad bugs.

completed major update.

Disabled one of the lists due to issues on the YouTube Channel page.

For 1.6.8 I've disabled the list "AFNIL - Anti Fake News International List" and enabled the "Web Annoyances Ultralist". AFNIL is currently causing a weird "new tab does not work" issue so until I resolve it properly it'll be not included.


5.0.0-M3: HTML report can't be disabled with formats option in Maven plugin

Describe the bug
With the new option of defining multiple report formats there is no way to disable the default HTML report set for format.
Version of dependency-check used
5.0.0-M3
To Reproduce
		<plugin>
		  <groupId>org.owasp</groupId>
		  <artifactId>dependency-check-maven</artifactId>
		  <version>5.0.0-M3</version>
		  <configuration>
			<formats>
				<format>JSON</format>
			</formats>
		  </configuration>
		  <executions>
			  <execution>
				  <goals>
					  <goal>check</goal>
				  </goals>
			  </execution>
		  </executions>
		</plugin>

Expected behavior
Only JSON report is generated.
Actual behaviour
JSON and HTML report are generated.
Aditional context
There is an awkward workaround by setting one of the formats from formats as the value for format. But I think this is not intuitive and cumbersome.
 <configuration>
       <format>JSON</format>
	<formats>
		<format>JSON</format>
                 <format>JUNIT</format>
	</formats>
</configuration>

Thanks for the PR!


Add more details for Interlock 3 upgrade

Proposed changes

Add additional details for Interlock 3
Fix formatting issue in 3.2.0 section

Related issues (optional)
https://docker.atlassian.net/browse/ENGDOCS-208

Deploy preview for docsdocker ready!
Built with commit b3b5243458728f53e59ca47d2417be32a3c39776
https://deploy-preview-9604--docsdocker.netlify.com


Removing input type from schema does not remove generated type

My Netflix DGS project is using com.netflix.graphql.dgs.codegen:graphql-dgs-codegen-shared-core=5.4.0
On my local machine, I was developing a new schema with a new input type. Netflix DGS generated the input type, I wrote code against it, everything was good.
Then, I decided to rename the input type to something else. I regenerated the schema using the generateJava task, everything was good. But I forgot to go into my application code and use the new input type name. Everything compiled so I thought it was good, so I pushed up to CI. It failed on the CI server because the old type name didn't exist anymore.
I would expect Netflix DGS to regenerate a clean set of classes each time by wiping out ALL of the existing generated code and generating all new files every time. Or at the very least, detecting code files that don't belong anymore and deleting them.

Can you please try with the latest version 6.0.2?Sent from my iPhoneOn Aug 24, 2023, at 1:54 PM, David Butler @.***> wrote:
My Netflix DGS project is using com.netflix.graphql.dgs.codegen:graphql-dgs-codegen-shared-core=5.4.0
On my local machine, I was developing a new schema with a new input type. Netflix DGS generated the input type, I wrote code against it, everything was good.
Then, I decided to rename the input type to something else. I regenerated the schema using the generateJava task, everything was good. But I forgot to go into my application code and use the new input type name. Everything compiled so I thought it was good, so I pushed up to CI. It failed on the CI server because the old type name didn't exist anymore.
I would expect Netflix DGS to regenerate a clean set of classes each time by wiping out ALL of the existing generated code and generating all new files every time. Or at the very least, detecting code files that don't belong anymore and deleting them.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

Hello, I can confirm that this is still happening on v6.0.2


Websocket Success & Failure Handler Framework

This PR refactors the event handler life cycle on websocket.

Group discussion 1/11: we'll close it after #1387 is merged because some of the changes in this PR are migrated to #1387

closed in favor of #1387.


[magma] Fix build for Windows with CUDA 12.6

Fixes #40941

[x] Changes comply with the maintainer guide.
[ ] SHA512s are updated for each updated download.
[ ] The "supports" clause reflects platforms that may be fixed by this new version.
[ ] Any fixed CI baseline entries are removed from that file.
[ ] Any patches that are no longer applied are deleted from the port's directory.
[ ] When updating the upstream version, the "port-version" is reset (removed from vcpkg.json).
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

@microsoft-github-policy-service agree company="Texel"

Can you try submitting this change upstream (to magma)?

@BillyONeal Looks like they've already added a fix, but c/d/sgbtf2_kernels.cu are missing from the repo. https://github.com/icl-utk-edu/magma/blob/4cb0501d76e79b8529162559588876455fef778a/magmablas/zgbtf2_kernels.cu#L15


This closes #664, support get embedded cell images

PR Details
Support user feedback read wps cell can't embed image
Description
Related Issue
Motivation and Context
Types of changes

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

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

Comparison is base (dfaf418) 99.05% compared to head (f8815a2) 99.05%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1759   +/-   ##
=======================================
  Coverage   99.05%   99.05%
=======================================
  Files          32       32
  Lines       28159    28212   +53
=======================================
+ Hits        27892    27945   +53
  Misses        177      177
  Partials       90       90

Flag
Coverage Δ

unittests
99.05% <100.00%> (+<0.01%)
:arrow_up:

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.


Current version is incompatible with Boost 1.70.0

Websocket_Base: no known conversion for argument 1 from boost::asio::execution_context to boost::asio::io_context

Thanks for the heads up! I'm developing on Arch Linux where the latest version of Boost available is 1.69.0. It looks like Boost 1.70.0 has been out since April 12th, 2019. I'll take a look at building Boost 1.70.0 from source and try to figure it out.

Already done some part of job (server without SSL) I will check other cmakes and align

ok, my conclusion is that currently there is no point of merging, 1.70.0 contains bugs that are resolved in future versions, in my humble opinion waiting for stable build is optimal :v


Update README.md

This branch has conflicts that must be resolved
Use the web editor or the  to resolve conflicts.
Conflicting files
README.md


Resize active window to be the largest

I feel like this has been asked before, but I have not found any results. Is there any way to make chunkwm behave like the golden ratio plugin for Vim? I have set the bsp_split_ratio to 0.618, but how do I make chunkwm resize the active window to be the largest, while staying in the golden ratio balance?

There is currently no way to automate the behavior of the golden ratio vim plugin.
Setting the bsp_split_ratio will only affect the initial split of newly spawned windows.
I'm not particularly interested in adding a config-option in the tiling plugin that adds this specific behavior.
There is a notion of the biggest window in the desktop, that can be used to achieve this if you have 2 windows only; you can use the command chunkc tiling::window --swap  biggest, to make the focused window swap place with the currently biggest window.
However, I suppose you would like this to happen automatically when a window-focus event occur or something though.
I wonder if it might be worth looking into adding the ability to hook shellscripts into the chunkwm-core's event dispatch system. So that you could basically do chunkc core::subscribe --event <event> --action <shell-script> and whenever said event triggers, they are first dispatched to whatever plugins, and at the end they can also trigger registered shell-scripts. This way you can react to events through shell-scripts, which could potentially call chunkc to signal / query a plugin to do stuff.
As the system is right now, any time you want to do something "custom" that relies on something like a focus change, you'd have to write a plugin to be able to react to that, which seems like overkill.
Say we had such a system, we could expand the query system and this feature could be implemented at a user level roughly as follows:
# entry in .chunkwmrc
chunkc core::subscribe --event window_focused --action my_script.sh

# my_script.sh
$new_focus=..(arg1 passed from event)
$child=`chunkc tiling::query --window-child-position $new_focus`
$ratio=`chunkc tiling::query --window-ratio $new_focus`

# if left-child has smaller region than right-child
# swap left-child with right-child to make the window fill the bigger region
chunkc tiling::window --swap sibling

I'm not sure if that would work with the current threading-model though.


Is it Linux compatible ?

Hi Jan Skalny,
I am facing so many issues while compiling the source code on Ubuntu 19.10. I could see that **#ifdef** is existing to differentiate the code between freeBSD and other operating systems. Can you please clarify the following queries ?
 1. Is the ERPS repo successfully compiles on LINUX ? if yes, can you please share the systems requirements on which system it runs, compiler version and also please share if any additional changes need to be done.
 2. If not compatible with linux, please share your your view.

I wrote Makefiles for erps-ctl app and kernel module. Was able to compile erps-ctl app but facing so many compilation issues related to kernel module.

Hi,
is this project still alive? I would be interested to have ERPS available on linux / for simulation with mininet.
Kind regards

Hi @d-jsb
Based on my experience this project was done based on freeBSD. It is not greatly compatible with Linux, we have to make changes accrodingly. Also the kernel module misses the send or recive(one of them) erps pkts functionality.


Replace existing links to the GitHub repository and files

Links exist in Github documentation and in eleventy processed pages

Have done first pass. Still to-do
in root, all checked except

[ ] README.md (as contains links that will change to engineering.homeoffice.gov.uk)

[ ] CONTRIBUTING.md (as contains links that will change to engineering.homeoffice.gov.uk and technical info that needs updating)

[ ] in /.github - need to check all as there are absolute links to standards for writing principles and patterns etc (will change to engineering.homeoffice.gov.uk)

[ ] /docs/patterns - todo

[ ] /docs/principles - todo

[ ] /docs/standards - todo

[ ] in /docs root - all checked

[ ] /docs/assets/logos - all checked

[ ] /docs/styles - all checked

[ ] in /technical-docs - all checked

[ ] in /nginx - all checked

[ ] in lib/markdown - all checked

[ ] in /helm - all checked

[ ] in /data - all checked

[ ] in /includes - all checked

[ ] in /cypress - all checked


support version macros in Component Registry components (IDFGH-11972)

Is your feature request related to a problem?
I don't see a way to specify a component's version just once and have it propagate to other files.
In a component (not mine) I'm using, I find 6 distinct places (not counting the changelog) where the number has been individually maintained :
victus: {106} find managed_components/bblanchon__arduinojson -type f -exec grep 7.0.2 {} ; -print
version=7.0.2
managed_components/bblanchon__arduinojson/library.properties
version: 7.0.2
managed_components/bblanchon__arduinojson/idf_component.yml
#define ARDUINOJSON_VERSION "7.0.2"
managed_components/bblanchon__arduinojson/src/ArduinoJson/version.hpp
version: 7.0.2.{build}
managed_components/bblanchon__arduinojson/appveyor.yml
v7.0.2 (2024-01-19)
managed_components/bblanchon__arduinojson/CHANGELOG.md
"version": "7.0.2",
managed_components/bblanchon__arduinojson/library.json
project(ArduinoJson VERSION 7.0.2)
managed_components/bblanchon__arduinojson/CMakeLists.txt
victus: {107}
Describe the solution you'd like.
I'm sure we can do better than this.
Describe alternatives you've considered.
No response
Additional context.
No response

For the managed components, there is already
idf_component_get_property(ver namespace__componentname COMPONENT_VERSION)

and you can then propagate the version into C source code using target_compile_definitions. So I think this is already supported from IDF side.
That said, with tools like https://commitizen-tools.github.io/commitizen/ it's not really a problem to automatically update the version number in multiple files when making a release. So it's up to the maintainer of the particular component to decide which approach to use.

Thanks, but I can't find anything about that in the docs. How do I call or use this ?
Looking for COMPONENT_VERSION on https://docs.espressif.com returns no results.
Same thing for
find $IDF_PATH -type f -exec grep COMPONENT_VERSION {} ; -print

Sorry about that, we forgot to mention this in the docs. It's only mentioned in the changelog of the component manager: https://github.com/espressif/idf-component-manager/blob/287b9e4351afd363a534aee9a7f2e8ea5fe1162e/CHANGELOG.md?plain=1#L178.
You can add this line to the CMakeLists.txt file where you need to access the version. The first argument is the variable which will receive the version value. The second argument is the component name. If the component is obtained from the registry, use the namespace and the component name, separated by double underscore. Here's an example:
https://github.com/espressif/esp-bsp/blob/f0c6bc2befa1daec8bcd86c931e36bb98e7caa9b/components/esp_lvgl_port/CMakeLists.txt#L3
We'll add this to the docs.

Took a bit of experimenting to get this to work. I assume this is what you mean ? (Lines 4 and 36)
`

Grab the Acmeclient version number

idf_component_get_property(acmeclient_version dannybackx__acmeclient COMPONENT_VERSION)

message(STATUS "acmeclient version: ${acmeclient_version}")

set(SOURCES
App.cpp Ota.cpp WebServer.cpp Lcd.cpp Config.cpp
Sensors.cpp Alarm.cpp Siren.cpp Peers.cpp
Logger.cpp
App.h Ota.h WebServer.h Lcd.h Config.h
Sensors.h Alarm.h Siren.h Peers.h
esp32s3-zx3d95ce01s.h screen.c
Logger.h
Network.cpp Network.h
)
idf_component_register(
SRC_DIRS ${DIRS} "."
INCLUDE_DIRS "." ${DIRS} ${BUILD_DIR}
EMBED_TXTFILES icon.png
REQUIRES esp_lcd driver lvgl esp_timer bus touch_panel
esp_event nvs_flash esp_wifi esp_timer esp_netif
app_update
esp_http_server esp_https_server
arduinojson acmeclient
WhatsApp rcswitch ftpserver
)
target_compile_options(${COMPONENT_LIB} PRIVATE "-Wno-format" "-Wno-deprecated-enum-enum-conversion")

Insert Acmeclient version number into build

target_compile_options(${COMPONENT_LIB} PRIVATE "-DACMECLIENT_VERSION="${acmeclient_version}"")
add_custom_command(OUTPUT ${BUILD_DIR}/build.h
COMMAND ${COMPONENT_DIR}/generate_build_date ${BUILD_DIR}/build.h
DEPENDS ${SOURCES} CMakeLists.txt ${BUILD_DIR}/config/sdkconfig.h
VERBATIM)
add_custom_target(buildh DEPENDS ${BUILD_DIR}/build.h)
add_dependencies(${COMPONENT_LIB} buildh)
`

Looks like I can, see https://sourceforge.net/p/esp32-acme-client/code/HEAD/tree/trunk/CMakeLists.txt .


Allow verbs to be defined in subpackages (similar to the type change)

Currently restricted to the root of the module which is very annoying.

Chatted with @worstell and the new schema extractor should account for this once it lands (hopefully later today). After that, I can retest and fix any remaining issues here.

completed in #1711


Add TxParams to build_transfer_tx

Wallet::transfer doesn't support customization of fee parameters. This is needed to be able to use the SDK against production environments.

Created https://github.com/FuelLabs/fuels-ts/issues/255 to track this on the ts side


Unable to pull/push via rsync if git-fat submodule has path with spaces in it

Say a main repo. A refers to a submodule B, but the path to submodule B includes spaces:
A/Data Sets/B
then git fat pull (with an rsync remote) fails, as the path is not correctly quoted for the call to rsync.
Problem relevant for current release 0.3.4.

Fixed?

The issue is when splitting cmd_tmpl by spaces: https://github.com/cyaninc/git-fat/blob/develop/git_fat/git_fat.py#L369
As src/dst may contain spaces it will split these paths in half which causes issue. The solution would be to split cmd_tmpl and only then append src/dst to the cmd list.

I thought I had fixed this in develop but it's apparent that's not the case.
I just got back from vacation so I should have more time to catch up with the changes and do a release.

Just an update on this:  I got back from vacation 2 weeks ago, but haven't had a lot of time because I was looking for a new place.  I just finished moving into the new place this weekend so I should be able to get through the backlog.

No worries at all.  I know what that's like, recently having moved myself. :wink:


Failed loading 'libopencv_videoio.so'

GET /mjpeg.php - Uncaught FFI\Exception: Failed loading 'libopencv_videoio.so' in C:\Users**\Downloads\PhpWebcamMjpeg-master\vendor\vdechenaux\webcam\src\Webcam.php:36

Hey,
The project requires OpenCV as mentioned here https://github.com/vdechenaux/PhpWebcam#requirements
I don't know how to install it on Windows but you'll probably found your answer on the Internet ;)


"help" More details need in Usage Instructions

Hi
Thank you for your work, I follow the usage instructions but I can't using it correctly. because I'm luck the corresponding data set. I have try to understand your code but I found that those dataset has some unfamiliar suffix. Those suffix hinder me to find correct data with data preprocessing.
Thank you for your kindness assistance.

Hi @PlayDeep ,
Thanks for taking a look at this repository. Can you provide a bit more detail what you exactly did and where you are stuck/what you are trying to do next?
In general you should be able to take a look at the resulting files using less or vim.
All files are text files in utf-8 encoding, here's a short summary of the meaning of the file endings:

src: text in the source language
tgt: text in the target language
talp: alignment points
bpe: text splitted into sentence piece/bpe units
lc: lowercase text

Let me know if this already helps or if you have more questions.

@thomasZen Thank you for your notes and your scripts, I have cloesed this issue after fixing the problem. The problem is caused by runing scripts with sh.
In the end, thank you for your kindness reply.


Prevent UnsupportedOperationException on some openjdks

@aloubyansky noticed that following exception happens when he is building locally with openjdk 11:
Caused by: java.lang.UnsupportedOperationException
    at jdk.nio.zipfs.ZipFileSystemProvider.newFileSystem (ZipFileSystemProvider.java:128)
    at io.quarkus.fs.util.ZipUtils.newFileSystem (ZipUtils.java:214)
    at io.quarkus.maven.ExtensionDescriptorMojo$1.visitEnter (ExtensionDescriptorMojo.java:545)

(https://quarkusio.zulipchat.com/#narrow/stream/187038-dev/topic/main.20is.20broken/near/265959481)
This exception does not happen on temurin jdk-11, because the whole path.getFileSystem() != FileSystems.getDefault() check is not present in ZipFileSystemProvider.
CI runs temurin.
With e.g. Oracle openjdk 11.0.2, I was able to reproduce this exception.
Starting with jdk12, the check is also removed from the main openjdk source.
temurin:

oracle openjdk:

0.0.7 was released including this change.

We could also add multiple JDK vendors to CI. If you guys think it's worthwhile I can have a look.


Add polymorph check

@CompRhys
Closes #71
New functionality
Check whether datasets contain polymorphs using MatbenchTask.has_polymorphs
dielectric = MatbenchTask("matbench_dielectric")
dielectric.load()
dielectric.has_polymorphs

>>> True

steels = MatbenchTask("matbench_steels")
steels.load()
steels.has_polymorphs

>>> False

Note the composition tasks in the original benchmarks should not contain any polymorphs. The structure tasks (I believe) should mostly contain polymorphs except for the jdft2d one.
Caveats
composition tasks
Note the way I have "polymorphs" here is not strictly polymorphs for composition tasks, since the structure is not given. For experimental data, it may be possible that identical materials (same comp, same structure) were synthesized in two different experiments and resulted in two different property recordings. These samples would flag the set as having a polymorph. has_polymorphs for composition tasks basically just sees if there are any repeated compositions.
structure tasks just check if there is >1 of the same reduced composition
has_polymorphs is currently true if there is more than 1 of the same reduced composition, without consideration that there may be multiple entries of the same exact cell. This is because strictly checking for polymorphism (i.e., structure similarity matching) seems kind of expensive? For the datasets in matbench currently, I don't think this should be a problem because MP/JDFT don't have different calculated values for identical cells. There is 1 unique polymorph per calculated value

@CompRhys let me know if this is sort of what you were looking for and ill merge it

Looks great!


Remove includes for full configuration

current ConfigurationBuilder includes option don't allow full redis configuration with micronaut application.yml, and just allow properties in includes option.

don't allow full redis configuration with micronaut application.yml

Can you provide non working config example?


How to preserve state in between tab changes

I have a tab that is supposed to load images from the network. How do I prevent the images from reloading when I switch tabs?

i have this problem ,any one can help me ?


Integration test for direct archiving is failing

Integration test is failing as archiving response with an internal server error.
0:02:18   _envelope_is_in_state returned True
0:02:18  envelope is in Archiving.
/builds/hca/ingest-integration-tests/tests/runners/dataset_runner.py:73: ResourceWarning: unclosed file <_io.BufferedReader name='/builds/hca/ingest-integration-tests/tests/fixtures/datasets/SS2/SS2.xlsx'>
  project_uuid=project_uuid)
E
======================================================================
ERROR: test_direct_archiving (tests.test_ingest.TestRun)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/builds/hca/ingest-integration-tests/tests/test_ingest.py", line 102, in test_direct_archiving
    self.ingest_to_direct_archives(TestRun.DATASET_NAME)
  File "/builds/hca/ingest-integration-tests/tests/test_ingest.py", line 62, in ingest_to_direct_archives
    self.runner.direct_archived_run(dataset_fixture, self.deployment)
  File "/builds/hca/ingest-integration-tests/tests/runners/dataset_runner.py", line 42, in direct_archived_run
    self.ingest_archiver.archive_submission(payload)
  File "/builds/hca/ingest-integration-tests/tests/ingest_agents.py", line 263, in archive_submission
    r.raise_for_status()
  File "/usr/local/lib/python3.6/site-packages/requests/models.py", line 943, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 500 Server Error: INTERNAL SERVER ERROR for url: https://archiver.ingest.dev.archive.data.humancellatlas.org/archiveSubmissions
----------------------------------------------------------------------
Ran 1 test in 143.657s
FAILED (errors=1)
Cleaning up file based variables
00:01
ERROR: Job failed: exit code 1

After checking the cause of this error it turned out that the username/password is not defined in kube deployment configuration.

@ke4 working on this. Biostudies test environment is exposed now so it is available for us to use. Karoly to work on this bug today

Fixes:

https://github.com/ebi-ait/ingest-kube-deployment/pull/37
https://github.com/ebi-ait/ingest-kube-deployment/pull/38


[release-4.13] e2e: Cleanup shared OIDC provider on SIGTERM

cherry-pick of #2435

/area cli testing


Arisilon/issue 15 - [merged]

In GitLab by @arisilon on Oct 29, 2019, 16:43
Merges arisilon/issue-15 -> master

In GitLab by @arisilon on Oct 29, 2019, 16:46
merged

In GitLab by @arisilon on Oct 29, 2019, 16:44
enabled an automatic merge when the pipeline for c3855d1c5bd9ad6e669e8b92575136c32ffd2e09 succeeds

In GitLab by @arisilon on Oct 29, 2019, 16:46
mentioned in commit e6df7794be6643d7f70db2b801a15fbadd048a47


permessage-deflate extension support

RFC

Hi!  I'm in need of a high performance websocket impl and have been kicking the tires on ws-tool, but just hit a websocket api that requires deflate. I'd contribute to this but I'm a late term newbie to rust and I'm afraid I'd only hinder your mojo.

I am implementing this feature recently, but since the deflate algorithm implemented in pure rust does not have the required parameters for permessage-deflate extension, it can only be bound with libz, which will cause ws-tool fail to compile without system dependencies, so I am still considering whether to re-implement the deflate algorithm; in addition, the implementation of permessage-delate in different language is slightly different, and I prefer to follow browser implementation. Anyway, feel free to fork this repo and make PR

An experimental per message deflate implementation has been implemented in #24


Performance enhancements of conditional logit

In the past, I've used Pylogit (specifically the MNL) on a large dataset of 200mln rows. I have noticed two bottlenecks:

The sparse matrix structure for weights_per_obs is not always kept, causing a 200 mln x 200 mln dense numpy array to be created, see also issue #79.
The derivatives dh_dv for a conditional logit represent an identity matrix but are coded as a csr_matrix. This causes the calculation dh_dv.dot(design) to be relatively slow even though its result is trivially design.

To remedy the first bottleneck, I used the same solution proposed in issue #79.
For the second bottleneck, I made an efficient identity_matrix class (derived from scipy's spmatrix). When such an identity matrix I is multiplied with A using I.dot(A) we get A again.
I've run a benchmark by making a script that estimates an MNL on the usual Swiss-Metro dataset. I ran the line-profiler on some of the critical functions, namely calc_gradient and calc_fisher_info_matrix. In summary, this change reduced the computation time of calc_gradient by 26% (from 0.080697 to 0.059372), and that of calc_fisher_info_matrix by 99% (!) (from 0.906896s to 0.0062323s).
Profiling results are attached.
profile_before.txt
profile_after.txt

Hi @mathijsvdv thanks so much for this, the speedups are fantastic.
Thanks also for your patience as I've been much delayed in responding to this PR and to the issue that spawned it.
I should be able to take a look at this within a week or and update the package.
Thanks again for your help!

Glad you like it @timothyb0912 ! I really appreciate all the work you've done to make a flexible logit estimation suite so I'm happy to help!


Uncaught Error: Option 'schema' can not be changed

If I create a react-prosemirror component instance with a custom schema, then any time the parent component' render method runs, I get the above error.
Just a side note, inside componentDidUpdate, when previous and current options are compared, the non-primitive values (schema, place, commands and doc [ if it is in some fancy format]) will be never  equal, due to the 'shallow' equality checking, therefore pm.setOption is called unnecessary. I have not dig into this deeper, so not sure how does this affect performance, however it might worth to keep it in mind.
So back to the real issue, since I am not changing the schema in the parent component, I just made a dirty hack in componentDidUpdate, and filtered out schema, like this:
Object.keys(current).filter( k => k!=='schema').forEach(...);

However if someone wants pass in different schema objects to react-prosemirror, based on some state or props of the parent component, then the inability of changing schema of an existing ProseMirror instance, could be problematic. What do you think?

Strange. I don't see a good way to check if an option can be updated from the outside. I think we're just going to need to depend on ProseMirror's error message (which is pretty reasonable).
We actually are doing a strict equality check.
if (current[k] !== previous[k]) {
	this.pm.setOption(k, current[k])
}

Are you creating the schema inside the render function of the component?
const MyComponent = React.createClass({
	render() {
		const options = {schema: new SchemaSpec({foo: Foo})}
		return <ProseMirror options={options} />
	}
})

If you do that, the schema will be recreated every time and always fail the strict equality check. Instead, maybe try creating it outside of the component (or once, in componentWillMount)?
const MyComponent = React.createClass({
	render() {
		const options = {schema: mySchema}
		return <ProseMirror options={options} />
	}
})

I think any scenarios that require changing the schema dynamically are too tricky to try to handle automatically. I suppose you could pull the content out and recreate the ProseMirror instance, but that doesn't sound like something that's safe enough to do at the level of this library.

I played with this a moment and I confirmed that at least the scenarios I described are correct. I don't know about your specific case.
Allowing the exception to bubble up was scrambling the editor unpleasantly, so I just pushed a change that catches it and logs the original error along with a hopefully helpful warning. https://github.com/tgecho/react-prosemirror/commit/54657745e06253e1b436bb5f6c6178e702e0486a

Good call! At the moment I'm defining my custom schema inside the render function, since I do not need dynamic schema, am just going to move it outside of the component, as you suggested.
Thank you!


Receiving multiple SMS at once

Hi,
thanks for great library. I'm using it for receiving sms. My usecase if I can receive at same time like 70 sms at once. Does library will handle it asynchronously (have some internal buffer or how it is done please)? Also do I need to delete received SMS (I juts know if from older phones that they refuse to receive is some number of SMS is received). Thanks.

I'm could swear I replied to this already, but apparently not...
This library is purely about encoding and decoding SMS PDUs.  It does not interact with the modem that sends or receives the PDUs - that is outside the scope of this library.
The issue with "older phones" you mention is probably related to a setup where the modem is configured to store received SMS PDUs to the SIM card rather than passing them directly to the application, and the application then needs to retrieve them from the SIM.  Again that is outside the scope of this library.


How to add About me

I tried to add About mein widgets like Category,but it doesn't work. Any help?

+1

I'm not sure for About me since I don't know how spaces are handled, but here is how I made an About page:

new folder /source/about/
new file /source/about/index.md
add to index.md:

title: About me
date: 2016-01-03 17:12:40
description: "All about meeeeee."
type: "about"

---
(text here)

In /themes/landscape-plus/_config.yml add a line under menu: to read:   About: /about

Should work.  Yes Hexo's documentation sometimes is hard to find what you are thinking.

just add about.md in your blog source folder
modify landscape-plus _config.yml,  add about: /about.html in menu settings

那么, 如何定制About.html的样式
并且, 我希望能够有"商品介绍", "明天天气", "交通情况"等多个可单独定制样式的页面

请问, 我该怎么做, 谢谢

补上图


Automating the generation of publishable bits

With this change, the "all-publish" target will produce local dev, beta-publishable and release-publishable bits (without any localization), and the just-beta-package and just-release-package targets will package (potentially localized) already built bits for the appropriate kind of publishing.

@nisheetjain it'd be good if you could check over the modifications I made to your code

Closing this PR in favour of one @nisheetjain has been working more on


"ERROR: canvas_item_add_triangle_array: Condition ' vertex_count == 0 ' is true." when opening editor

Godot version:
Master since 8346d90c5, still reproducible in 6475c990.
OS/device including version:
Linux, but not OS dependent.
Issue description:
Opening any project in the editor triggers an error spam:
ERROR: canvas_item_add_triangle_array: Condition ' vertex_count == 0 ' is true.
   At: servers/visual/visual_server_canvas.cpp:762.
ERROR: canvas_item_add_triangle_array: Condition ' vertex_count == 0 ' is true.
   At: servers/visual/visual_server_canvas.cpp:762.

It's a regression from 8346d90c5 / #26609, CC @pouleyKetchoupp @toger5.
Bisected:
8346d90c5e659bac086ac2a3426f13efcd094840 is the first bad commit
commit 8346d90c5e659bac086ac2a3426f13efcd094840
Author: PouleyKetchoupp<EMAIL_ADDRESS>Date:   Tue Mar 19 10:32:13 2019 +0100

    Properly handle different border widths in rounded corner aa

    Note: removed (unused and not implemented) support for multiple border colors

:040000 040000 f4f64f4a61a36b1e09ad337702491aa9660607f6 b2915845b4369d34d9e9bd0ea064f3d538b15dc7 M      editor
:040000 040000 b2e8929e8c80591e2623645e8813f66a7236af5b b6497e1f4335f48c1cb528968dbb39fcdd40075e M      modules
:040000 040000 72bf08a01ff60121e66985729630556cf86c161d c967fa0eaab3df89f549cea0840999cbf79f943f M      scene

Steps to reproduce:
Open any project, check terminal.

Ok, I'm going to check that.

thank you for the quick fix!


Polars support

polars is growing fast as the new standard for dataframe but it doesn't provide a way to plot the data for Rust user.
I think providing a polars integration under a feature flag, like you did for the ndarray, could be a great I idea? What do you think of it?

I agree in principle and would be happy to accept a pull request.

Seconding this, is there a good recommended way to work with polars series using plotly rs right now?

+1


Pretrained model in windows

I am trying to use the pretrained model (Demo-Europarl-EN.pcl) in Windows to convert a file "raw.txt" into "test.txt" with raw.txt has missing punctuations while test is the output file with punctuations
Sorry, i am a little confused on how to do that. Please correct me if I am wrong. Do I do this?
cat raw.txt | python punctuator.py Demo-Europarl-EN.pcl test.txt
cat is a linux command, how can i run this in windows command? Appreciate advice. thanks.

I don't have a windows machine at hand right now, but in PowerShell you can maybe try:
Get-Content ".\raw.txt" | python punctuator.py Demo-Europarl-EN.pcl test.txt
If that does not work, then you can try modifying the line: https://github.com/ottokart/punctuator2/blob/5161946e0fdc144a607db4eaa4ef968e8f6e3d77/punctuator.py#L163
to read an input file instead of stdin. For example:
input_text = codecs.open(sys.argv[4], 'r', 'utf-8').read()
and run the script with:
python punctuator.py Demo-Europarl-EN.pcl test.txt 0 raw.txt

Thanks. Got the answer from a friend to use the below instead. and working.
type raw.txt | python punctuator.py Demo-Europarl-EN.pcl test.txt
Also, found that I could not use python 3 2to3 script to run it in Python 3.  Had to use Python 2 directly only.

@Nicbecoz how did you make it work on Windows ?
I am also trying to make it work on Windows 10
could you explain the steps?


Is this project active?

Hi,
Not many queries are been answered or it looks any active development is happening.
Is this project active on GitHub or any where else?

alot of people wanne use this but it seems they dont maintain it anymore. i suggest if they dont maintain it anymore then fork it and lets continue on another repo.

+1 on this.
With kafka 0.10.2, a lot of new admin feature will be available like real api with policy or new jmx metrics by partitions.
I think a fork can be very helpful. I actually use the fork of @Innometrics

+1 to support if we have active members to continue this project, either this or any other forked version.

Would be interested to hear what people think about moving this to an Apache project.

@patelh yes please. I think there's a lot of goodwill to improve this project

@patelh @simplesteph Agree. so far there is not a realy good/handy GUI tool to manage/monitor kafka would be great to improve this project

What is pre-requisite to move this project to Apache project, do we have PMC from Apache projects here or I can contact and get the update.

@BDeus I can't find this project's fork in @Innometrics 's list, are you still using their fork?

No i use the master now
the pull request has been merged since february https://github.com/yahoo/kafka-manager/pull/282

I would love to get this into Apache.


cannot DROP table due to dirty dependency status

Describe the bug
https://risingwave-labs.slack.com/archives/C06K0R9UTQR/p1730782295995279?thread_ts=1730737567.989079&cid=C06K0R9UTQR
SELECT id, name, schema_id FROM rw_catalog.rw_tables WHERE name = 'macros';
  id  |  name  | schema_id
------+--------+-----------
 4815 | macros |      4028

select * from rw_depend where refobjid = 4815;
 objid | refobjid
-------+----------
(0 rows)

drop table macros;
ERROR:  Failed to run the query

Caused by:
  Permission denied: PermissionDenied: Fail to delete table `macros` because 2356 other relation(s) depend on it

Error message/log
No response
To Reproduce
No response
Expected behavior
No response
How did you deploy RisingWave?
Cloud
The version of RisingWave
v1.10.2-patch-us-east-2-672-agg-state-cache-metric
Additional context
No response

Some updates:

id
name
fragment_ids
actor_ids
downstream_fragment_ids

4815
macros
[4517, 4518]
[6033, 6034, ...]
[4523 .. 5618, 6005]

rw_depend only contains records for relations that have already been successfully created, and the data comes from dependent_relations in the proto.
The dependency check uses the in-memory relation_ref_count, but the error log shows that the reference count of macros is at 2356, which is clearly dirty. But why is it so large? 🤔 There are indeed many downstream fragments.

This indicates that table macros indeed has some creating downstream streaming jobs, after the user executes drop cascade, both the macros and the downstream relations' catalog are deleted. The table fragments of macros have also been deleted, but the actors of the downstream jobs were not clear during recovery, resulting in a constant recovery loop error reporting "actor xxxx not found in info table". This is likely related to the cache of in_progress_creation_streaming_job not being updated. The successful recovery after restarting meta can corroborate this, as actors downstream should have been cleaned after the restart.


Missing data about operation_parentID

Hi Team
We are using this SDK to get telemetry to an application Insights resource, however due to there is not details about how to configure the operation_parentID value we are using the same id in the dependencies tracking, and we are getting an Application Map iterative showing only one instance instead of show the rest of the components interacting with the project. Could you please update this documentation in order to provide more details about the configuration of this vale.
Thanks, have an excellent day.

Operation ID/Parent Operation ID are left to the user in this SDK.  (The idea behind this is that some autocollection library could be built on top of this to provide those values.)  This can be set on the TelemetryContext object associated with the TelemetryClient or on individual Telemetry instances via:
telemetry.ContextTags()[contracts.OperationParentId] = "parent id"
// or
client.Context().Tags.Operation().SetParentId("parent id")

But I guess what you're asking is what these things ought to look like.  I'd point you to this document to start.  To be fair it's light in details, but should provide a reasonable starting point.

This is a bit outdated article, but it gives an idea what values should be set where to enable Applicaiton Map: http://apmtips.com/blog/2017/10/18/two-types-of-correlation/

Please re-open if you have more questions


TabRepo 2.0 Feature Tracker

For TabRepo 2.0, several quality of life changes should be made for ease of use. This list will evolve over time.

Hyperparameter dictionary should have keys matching the names elsewhere. Currently there is an inconsistency, sometimes a model is called "LightGBM_c1", another time it is called "LightGBM_c1_BAG_L1". The name should be consistent in all usages and contexts.

Another thing that has been my radar for some time is to have tabrepo on huggingface. It will speedup the download time by ~8x (download is very slow from outside) and would make the dataset more visible.

Having an example or an API that allows to "join" two repository would be also be quite useful. One could do:
from tabrepo import load_repository
from tabrepo.utils import merge_repositories
repo = load_repository("D244_F3_C1530_30")
repo_with_new_method = load_repository("D244_F3_C1530_30")
# builds a repository from the two, filter models that appear only in all task, underneath, just calls the repo that contains a given model
repo_union = merge_repositories([repo, repo_with_new_method], force_dense=True)

@geoalgo Yes, repo joining is something I plan to implement. I added a tracking GitHub issue: #65


Adding  branch-name copy  to clipboard and button in branches screen.

Adding  branch-name copy  to clipboard and button in branches screen
Replaces https://github.com/go-gitea/gitea/pull/25569
Issue #https://github.com/go-gitea/gitea/issues/25120
New mocks:

@silverwind @lunny @wxiaoguang

Now I guess the classes are fine.

Wow, this table looks so bad in those screenshots, but probably not this PRs fault (alone). Please provide comparison before/after screenshots here to ensure this didn't regress it further.

Wow, this table looks so bad in those screenshots, but probably not this PRs fault (alone). Please provide comparison before/after screenshots here to ensure this didn't regress it further.

💯 👍 😄
Honestly, I am very happy that we do care about UI/UX a lot. But existing UI of gitea need a lot of improvements. IMO a dedicated milestone is required.

Wow, this table looks so bad in those screenshots, but probably not this PRs fault (alone). Please provide comparison before/after screenshots here to ensure this didn't regress it further.

My soul is also hearting when I see

 , in the world of flex and grid and with specially in the mobile layout.
Here is the before and after comparison.
Before

Mobile view:

After  applying copy button:

Mobile view:

IMO we should change
 layout in next followup PR. Let this PR merge if this address the required ISSUE fix.
Otherwise we will lose the track.


updated go text pkg to v0.3.7 (trivy vulnerability)

Updated go text package (golang.org/x/text) to v0.3.7 as trivy reported vulnerability in v0.3.6

@sethiyash I think you might have missed running go mod vendor or building the package once.

Looks like there are no changes in the vendor files is that expected?
But I believe the verify-no-dirty-files script would have caught that.

Looks like there are no changes in the vendor files is that expected?

I assumed that that's because it's part of Go itself. We can confirm to be sure.


TODAY: MACBOOK not working Everywhere demo about audio???

hi @muaz-khan .
TODAY: MACBOOK not working Everywhere demo about audio ???

Please recheck the demos. Seems fixed.

Every day I have to test it . But today everything except audio still works . I have test with two machines macbook , and win , it's still working .
link demo : https://rtcmulticonnection.herokuapp.com/demos/Scalable-Broadcast.html

Please make sure following file is up-to-dated:

https://cdn.webrtc-experiment.com/rmc3.min.js

// Last time updated: 2016-06-18 7:54:10 AM UTC

I checked on your demo . I checked my mic on skype and it 's working well .

Yeah!.


.indexof() is not working for an array of int64

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
[int64[]]$testarray=(10,20,40,50)
$testarray.indexof(20)
Expected behavior
Return position of indexed value. (1)

Actual behavior
The returned position returned is always -1

Error details
No response
Environment data
Name                           Value
----                           -----
PSVersion                      7.1.3
PSEdition                      Core
GitCommitId                    7.1.3
OS                             Microsoft Windows 10.0.22000
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

This works:  $testarray.indexof([int64]20)
You have to cast as int64, otherwise it's interpreted as int32.
 (20).Gettype()

You can also use the L suffix to write int64 literals; $testArray.IndexOf(20L)
IndexOf() I believe has object as the parameter type, so it won't auto-cast to long.

$testarray.indexof

OverloadDefinitions
-------------------
int IList.IndexOf(System.Object value)


Using with Automation? Fullscreen + Sound by movement?

Is there any way to activate fullscreen mode and unmute mode by automation?

Duplicate #25


CUDA out of memory during eval for neus

Hello, I'm trying to train the neus model. It reports CUDA out of memory error during evaluation. I have a 24GB GPU. Is that expected? How can I solve it? When I add more GPU, it doesn't seem to distribute the work. Each GPU goes out of capacity.

You could try reducing the number of training and evaluation rays per batch, 24GB should definitely be enough for most models.

@iszihan What is the resolution of your image? Could you try to add something like
if torch.is_tensor(output):
    output = output.detach()

before this line https://github.com/autonomousvision/sdfstudio/blob/master/nerfstudio/models/base_model.py#L182?

I added that -- it still goes out of memory for resolution of 1024. :/

The OOM does happen during the loop linked here https://github.com/autonomousvision/sdfstudio/blob/master/nerfstudio/models/base_model.py#L182. In my script, I had 9871788 num_rays to evaluate.

@iszihan Maybe some computational graph is not automatically released. Could you add torch.cuda.empty_cache() around there to see if it helps. Otherwise we need to figure out which outputs are still on pytorch's computational graph.

I tried adding that it doesn't go out of memory as fast but still does. Also curious why would neus model have 9871788 rays to evaluate. I used nerfacto before, the same function return only 1440000 rays to evaluate.

I met the same question. Have you solved it ?


Added review of the event

Adding the review of the dat event by the dat organizers.

Thank you for reviewing. Merging now.

great work! this is amazing


Project does not work with Python 3.10

Seems like this project only works with python 3.7 would it be possible to update this to the current version of python?

Hey! so I tried a bit but I was having trouble balancing all of the
dependencies so I let it go... Sorry!
On Sat, Dec 17, 2022 at 10:40 PM Ezra Mendelson @.***>
wrote:

Seems like this project only works with python 3.7 would it be possible to
update this to the current version of python?
—
Reply to this email directly, view it on GitHub
https://github.com/noamephrat94/flask-highlight-maker/issues/1, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ASSOI5PFYC5ZN54OMFOF663WNYQL7ANCNFSM6AAAAAATCEAMZ4
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

No problem, thanks for the quick response!

I was able to get it to work on Python 3.10 by removing the unused basketball-reference-scraper module.
Now that I have the website working I am trying to use the create highlight feature. Do I need to pass in mp4 of the full game in the GAME URL field?


Handle levain files from command line

Allow commands (shell, install) to receive levain files directly from the command line.
Examples:
levain install C:\Users\userid\myPackages.levain.yaml eclipse S:\Company\company.levain.yaml vscode

It could be handled as a FileSystemRepository with a single file/package.
OPTIONAL: We should monitor updates on these files as well.

Also #47

Also #47


chore: release

🤖 New release

borsh: 1.1.0 -> 1.1.1 (✓ API compatible changes)
borsh-derive: 1.1.0 -> 1.1.1

Changelog
borsh

1.1.1 - 2023-10-25
Other

add test_schema_conflicts (#254)
addressed warnings in docs generation (#252)
simplify discriminants handling in schema derives (#241)

This PR was generated with release-plz.

Since the breaking change is under unstable__schema feature, it should not be 2.0 release, so we will need to update this release manually before merging this PR.

Since the breaking change is under unstable__schema feature, it should not be 2.0 release, so we will need to update this release manually before merging this PR.

there was no breaking change, this was due to exclamation mark chore! pop!ping up in master commit message somewhat unexpectedly.


feat(core-api): hasTransactionFinality() on connector API

Adding function hasTransactionFinality() on connector API, to let ledger query whether the configured consensus has transaction finality at run time.
Fixes #354

@jscode017 That test failure is shown by the CI logs is not related to the changes you made, if you rebase your branch onto #810 (and mark it as a dependency of this PR) then the tests should start passing. Alternatively just wait a little until #810 gets merged and then rebase onto main to achieve the same outcome.

LGTM


Update all run_exports pins to pin to 'x'

This PR updates all run_exports pins (those things that enforce compatibility version pins from this package on its dependents) to just pin the major GDS version, now that the GDS project is using semantic package versioning to indicate library compatibility.
@emaros, please confirm (and merge).

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

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.


Fix build workflow

This PR does the following:

makes sure package data in grizli/data is included in both sdist and wheel
fixes the cython compile under setuptools to work with modern python -m build . and by extension pip install
uses setuptools-scm for versioning based on git tags

Still to do in this PR:

change the github actions workflow to test this all properly
try it all out in a fresh env and see what breaks

@BSGalvan feel free to add a review this PR. This is the next step in getting it to install.

Ok, CI passes!
I've updated the description at the top.  I think this is ready for final review @ivastar and @BSGalvan.  Thanks!


Customized Pipeline "spacy_sklearn" + "ner_duckling"

I am facing Java memory error while running with customized pipeline by adding "ner_duckling".
If I remove "ner_duckling" and just use "spacy_sklearn" it is working fine.
rasa-nlu==0.11.3
spacy==2.0.9
duckling==1.7.3
INFO:rasa_nlu.utils.spacy_utils:Trying to load spacy model with name 'en_core_web_lg'
INFO:rasa_nlu.components:Added 'nlp_spacy' to component cache. Key 'nlp_spacy-en_core_web_lg'.
Java HotSpot(TM) 64-Bit Server VM warning: INFO: os::commit_memory(0x0000000089a00000, 32505856, 0) failed; error='The paging file is too small for this operation to complete' (DOS error/errno=1455)
#
# There is insufficient memory for the Java Runtime Environment to continue.
# Native memory allocation (mmap) failed to map 32505856 bytes for Failed to commit area from 0x0000000089a00000 to 0x000000008b900000 of length 32505856.
# An error report file with more information is saved as:
# S:\Learnings\Python\RASA_NLU\test\hs_err_pid16272.log

{
    "pipeline": ["nlp_spacy", "tokenizer_spacy", "intent_entity_featurizer_regex", "intent_featurizer_spacy", "ner_crf","ner_duckling", "ner_synonyms",  "intent_classifier_sklearn"],
    "path" : "./projects",
    "data" : "./data/my_intents",
    "spacy_model_name": "en_core_web_lg"
}

I need duckling since I deal with lot of date and time stamp related messages.
How to add "ner_duckling" at the end of "spacy_sklearn" pipeline?

I suspect your machine is running out of memory from loading a spaCy model (can be a couple Gb) plus booting a jvm at the same time.
You might want to think about running the new haskell duckling in a docker container and using the ner_duckling_http component instead

@amn41 Thanks it worked. I just replaced "ner_duckling" with "ner_duckling_http" (haskell duckling).
When I saw the "D:\Program Files\Python36\Lib\site-packages\duckling\wrapper.py"
def __init__(self,
                 jvm_started=False,
                 parse_datetime=False,
                 language=Language.ENGLISH,
                 minimum_heap_size='128m',
                 maximum_heap_size='2048m'):

I changed the maximum head size to 6GB
def __init__(self,
                 jvm_started=False,
                 parse_datetime=False,
                 language=Language.ENGLISH,
                 minimum_heap_size='128m',
                 maximum_heap_size='6144m'):

Now I can use "ner_duckling" component without any issue.
My File pipeline is
"pipeline": ["nlp_spacy", "tokenizer_spacy", "intent_entity_featurizer_regex", "intent_featurizer_spacy","ner_crf", "ner_synonyms" , "ner_duckling", "intent_classifier_sklearn" ]

http://nlu.rasa.ai/faq.html?highlight=warning#why-am-i-getting-an-undefinedmetricwarning

@amn41 Thanks a Lot.


Fade to color does not reach final color

When I fade to color white (#FFFFFFFF) the last frame that is generated is not completely white/empty, the previous frame is still visible (although very faint).
The last frame should be the final color ;-)

Indeed.

Fixed. Thanks.