including commonjs module with binary required files

Hey,
What would be the solution to include commonjs modules that comes with binaries included?
Is that the issue on this plugin side or I am doing it wrong in configuration somehow?
I got i.e. SyntaxError: Unexpected character '�' (1:0) in ../node_modules/child_pty/build/Release/pty.node.. which is reasonable that rollup can't roll the binaries in the code. Searching for this but I did non found any solution.

Can you supply a minimal reproduction so that we can start to investigate? Thanks 👍

Hey @Rich-Harris,
I prepared repository that reproduces that at https://github.com/jkuri/rollup-repro-binary-commonjs. I encountered that issue when bundling a module for electron app and solved it with manually copying child_pty binaries in dist/ directory in a npm postinstall script with a rollup configuration setted i.e.:
paths: {
  'child_pty': './vendor/child_pty'
}

so require finds it there.
Is this the best solution for now? It would be nice if that plugin would automatically manage binaries that comes from 3rd party packages. Otherwise, thanks for this plugin and checking out on this!

I think the best solution might eventually be to create rollup-plugin-node-addons, which would replace addons with placeholders and copy them to a destination of your configuration:
import nodeAddons from 'rollup-plugin-node-addons';
import nodeResolve from 'rollup-plugin-node-resolve';
import commonjs from 'rollup-plugin-commonjs';

export default {
  entry: 'src/index.js',
  dest: 'dist/bundle.js',
  format: 'cjs',
  plugins: [
    nodeAddons({ dest: 'dist/addons' }),
    nodeResolve(),
    commonjs()
  ]
};

Any .node files would be intercepted by the plugin and copied to the dest option, and they'd be imported via relative paths from dist/bundle.js.
For now though, your workaround (basically doing the same thing manually) is probably the best option. Thanks for the repro and explanation of the problem.

Additional plugin sounds fair enough.
I just checked and node-sass for example provides .gyp extension naming for their binaries, so it is neccesarily a .node, might also be without extension, I think. The plugin should detect if a imported file is instance of blob or something and copy it if so. Thank you for checking. Also, let me know if I can help to work on this plugin.

Is this issue going anywhere?  I was trying to use Rollup to bundle an electron app with the keytar package, which has native bindings so it can interact with OS keychains.  I've used this plugin to include node_modules like keytar, and by default it "supports" .node extensions (which I've derived while looking at the source code), though it really doesn't actually do something with it--it really tries to pull it in and parse it even though it is binary, just as mentioned above.
I had been following the idea of creating a package to deal with it, but since this conversation is a nearly a couple of years old (at least from the last message), I want to make sure that there isn't a solution out there already to see if it's worth investing time into this or not.

I started into that example and found that super simple.  I'll have to test it a bit and then maybe I could release that on npm or something.

@cwadrupldijjit do you got any existing code to check and improve?

Apologies for not seeing this sooner--I largely have ignored the notifications from GitHub because I haven't gone through to unwatch repos I don't need to watch.
I'll have to look into this again.  I think I was able to get something that sorta worked for this, though I haven't worked on that project in quite a while since it was a personal projects and work got in the way.  It had something to do with the this.emitAsset() method in the documentation, which moved here:
https://rollupjs.org/guide/en#this-emitasset-assetname-string-source-string-string
I'll have to paste code snippets of my solution, though.  I'll have to also see what I should do about making the plugin into an NPM package and making it easier for you to use it, too.

Hey folks (this is a canned reply, but we mean it!). Thanks to everyone who participated in this issue. We're getting ready to move this plugin to a new home at https://github.com/rollup/plugins, and we have to do some spring cleaning of the issues to make that happen. We're going to close this one, but it doesn't mean that it's not still valid. We've got some time yet before the move while we resolve pending Pull Requests, so if this issue is still relevant, please @ me and I'll make sure it gets transferred to the new repo. :beer:


genesyscloud_tf_export

i am trying to use export without specifying the resource type  the plugin was crashed
i am getting below error.
due to this the demo on Cx as Code was failed today for us.

Hi, sorry you experienced that crash during a demo.
We accessed the rrule property of an architect schedule without checking if it was nil.
We assumed it was set because the swagger documentation has it marked as required, some times the swagger documentation is not consistent with the API responses.
I've created a PR to fix this.


batch does not detect crashed docker and says the jobs are running, indefinitely

new operational issue -- docker container and docker daemon crashed on the instance running the batch job;   batch does not detect a failure at all, says the job is happily running;   idseq thinks the job is running in host filtering;  the rapsearch and gsnap tiers are kept scaled up expecting the job to enter alignment anytime
this situation persists >6 hours
cc @yunfangjuan
https://us-west-2.console.aws.amazon.com/cloudwatch/home?region=us-west-2#logEventViewer:group=/aws/batch/job;stream=aegea_batch/default/dcd2dd24-fcd9-4158-88b7-26e6856166fa;start=P1W

In this case I also have to manually terminate the instance kept alive by batch, just to be sure.

What's kind of cool is that when I terminated the botched instance, idseq detected this as "job killed by batch" and immediately resubmitted the job, nice.    This particular job is only 2 x 65 million reads so not a terribly big job.  This is some pure operational issue.


Check latest release

As a developer, I need to make the app aware of new releases.
Implement a method to check the latest release in GitHub and save it locally through shared_preferences.
Acceptance criteria:

The latest release version is stored in shared_preferences.
It is possible to check if the latest version is newer than the currently installed version.


arch: Remove the duplicated up_tls_info implementation

Summary
Define up_tls_info in arch/arch.h directly if the general one isn't suitable
Impact
Code refactor, no real change
Testing
Pass CI and ostest

Need PR https://github.com/apache/incubator-nuttx-apps/pull/912 to pass the CI, @gustavonihei .


Migrate to Wrangler

Update the terraform operator to use wrangler instead of norman.

Completed: https://github.com/rancher/terraform-controller/pull/27


Button background not tinted in re-generated fragment in ViewPager

[x] I have verified there are no duplicate active or recent bugs, questions, or requests
[x] I have verified that I am using the latest version of Aesthetic (see the README)
[x] I have given my issue description title
[x] I have read over the documentation (before asking questions on how to do something).

Include the following:

Aesthetic version: 0.x.x.x
Device OS version: 0.4.7
Device Manufacturer: Google
Device Name: Nexus 5X

Reproduction Steps

In the sample app, add one more Fragment
Swipe to the third page, then back to the first page
Background of buttons in the first page were not tinted

I have checked that when MainPagerAdapter::getItem(int position) is recalled, buttons in the re-generated view will not be tinted.

I have the same issue with the AboutLibraries plugin.. Buttons aren't colored, but tapping them causes them to be colored

It seems that only when the state was changed did the background change.
Add the following piece of code at the end of AestheticButton::invalidateColors works:
  private void invalidateColors(ColorIsDarkState state) {
    ...
    setEnabled(!isEnabled());
    setEnabled(!isEnabled());
  }

REF: https://stackoverflow.com/questions/36114388/setbackgroundtintlist-does-not-apply-directly#comment59871614_36114388.

I am using your library to theme a project I am working on.  I am loving how easy it is to use.  Previously, on a former project, I created my own theme engine and it was a nightmare to get it to work correctly.  So I thoroughly appreciate the hard work that has been put into this.
I am having this same issue with button color not showing after fragment recreation.  I am not using a ViewPager, however.  I am using only fragments and navigation view.
I am also having an issue with progress bar horizontals.  The library seems to be overriding and making the progress bar be the default circular pattern.
I suppose that until I can figure out how to fix these two issues, I will release the app without this great library.  I will continue to play around with this library.  I do really like it.  Keep up the great work!
Thanks again!


Allow specifying the delimiter for pandas' CSV loader.

Close #48

Checklist:

Does this pull request close an issue? We encourage you to open an issue first if this pull request (PR) is not a
minor change.

[ ] No

[ ] The change in this pull request is minor.

[x] Yes

Close #48

For the following questions, only check the boxes that are applicable.

[x] Ensure that the pull request text above the horizontal line is descriptive.

[x] Add/update relevant tests.

[x] Add/update relevant documents.

Do you have triage permission of this repository?

[ ] No

You are good.

[x] Yes

Does this pull request close an issue?

[ ] No, this pull request also acts as an issue by itself.

[ ] Assign yourself.
[ ] Label this pull request.
[ ] Put a milestone.
[ ] Put this pull request under "In Progress" or "Under Review" pipeline on ZenHub.
[ ] Put this pull request under the appropriate epic on ZenHub.
[ ] Put an estimate on ZenHub.
[ ] Add dependency relationship with other issues/pull requests on ZenHub.

[x] Yes, this pull request addresses an issue and does not act as an issue.

[x] Connect this pull request to the underlying issue on ZenHub.
[x] DON'T:

DON'T assign yourself or anyone else.
DON'T label.
DON'T put a milestone.
DON'T put this pull request under any epic on ZenHub.
DON'T put an estimate on ZenHub.

[x] When the pull request is ready, request review.

FYI, https://github.com/pandas-dev/pandas/issues/38489 is fixed on master. We shouldn't need the workaround once pandas makes a new release.

Yup, I looked through the Pandas issue. Is a release the same as a tag? Seems like they make a vx.x.x tag about once a month, so maybe in January it will be available to us?

Yeah, looks like we would expect it to release in one month.


Proposal: Allow overriding/disabling default TabView close tab hotkey

Proposal:  Allow overriding/disabling default TabView close tab hotkey

Summary

Not all apps benefit from using Ctrl+F4, some also register a second hotkey for closing a tab, e.g. Ctrl+W as this is also commonly used. Devs should be able to disable or replace the hotkey in case it is conflicting or there is a better suited alternative.
Rationale

Ctrl+F4 might already be in use inside the app
Other hotkeys might be more familar for users or might make more sense

Scope

Capability
Priority

Allow disabling the Ctrl+F4 hotkey
Must

Let devs provide an alternative hotkey easily
Should

Open Questions

@stmoy FYI
Also possibly @yaichenbaum FYI
What about the TabVIiew tooltip, should that be changed easily too then?

Seems like a great idea.  @chigy and @beervoley FYI

I would just like to turn off the shortcuts altogether.

Further our app allows the user to specify their own language, which may be different from the Windows language.  If we can't override the ToolTip here (and on the scroll button) ourselves then it's possible the ToolTip "hardcoded" in the control will be in a different language than specified by our app, causing a UX issue.

How do you change the language of the app? If you change the PrimaryLanguageOverride, the ToolTip texts will also change the language accordingly (at least they should).

Yes we use PrimaryLanguageOverride.  You're right; that will change the tooltip language, my bad.  Scratch that part of the issue.
So it's just a matter of being able to override/turn off the tooltip itself.


2024.04.23 build failure

👋 trying to build the latest release, but run into some build issue. The error log is as below:

error build log
  CMake Error at CMakeLists.txt:196 (cmrc_add_resource_library):
    Unknown CMake command "cmrc_add_resource_library".

full build log, https://github.com/Homebrew/homebrew-core/actions/runs/8900317936/job/24441728512
relates to Homebrew/homebrew-core#170471

This command should be included and used by find_package(CMakeRC REQUIRED).
Could you please try clearing the cmake cache and try again?
https://github.com/microsoft/vcpkg-tool/blob/d6945642ee5c3076addd1a42c331bbf4cfc97457/CMakeLists.txt#L191
cc @data-queue


feat: Link Tube to Base

Please read contributing guidelines and remove this line.

Incorporated into https://github.com/AztecProtocol/aztec-packages/pull/6971


added method to remove inventory dates closes #153 in getoutfitted

Clever implementation using splice and the first index. Avoids any date comparison and any potential time zone pitfalls. 👍


Support other languages

Request from lovenother on Steam

Feature Description
Support other languages & display text with the same language as the in-game setting

I currently have no knowledge about how it could be done and I think the mod does not have a lot of words at the moment. I'd be glad if anyone would contribute though.

Hello.
I don't know if this will be helpful, but I think the translation framework used by algernon's mod is easy to handle.
It has a generic structure and would be easy to port.
sway's mod also uses the same framework.
And it would be helpful if you could handle the translation in Crowdin.
Many other mods are also able to be translated by Crowdin.
I personally have no problem with PR on Github, but there is a bit of work involved.
If you want to use it as a reference, sway's mod would be a good choice.
algernon's mod is in the process of transitioning to a shared framework for translation.
There are two types of mods that have not been migrated: CSV and XML.
If you want to refer to them, please check with the mod using CSV.

Thanks for the info, but I currently do not have time to implement this personally. I welcome anyone to contribute though.


404 (not found) for a custom service

angular-cli: 1.0.0-beta.10
node: 6.2.1
os: darwin x64
OSX El Capitan
Fighting with the code for 5 hours now... couldn't figure out what I am doing wrong. Please help.
My service (demo/demo.ts):

demo/index.ts:

main.ts:

index.html:

result:

You should try https://gitter.im/angular/angular instead of raising an issue here. Anyways you should check system-config.ts. For further queries on this, I suggest gitter or stackoverflow


dockerTools: prefer local builds

Description of changes

This adds the preferLocalBuild = true; attribute to 4 derivations in pkgs/build-support/docker/default.nix and 1 in pkgs/build-support/references-by-popularity/default.nix.
All these derivations are quick to finish but take as inputs all the runtime closure of the container image we're building.
Setting preferLocalBuild = true; prevents dependencies being copied between remote builders, which is in the majority of cases much more expensive than building the slim derivations locally.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

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

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

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

[x] Fits CONTRIBUTING.md.

All these derivations are quick to finish but take as inputs all the runtime closure of the container image we're building.

This means we copy them to the local machine anyway.

IMO I would love the builder protocol to be smart about this and just build thin wrappers where most things are already cached or in the end going to be copied.


Improve error message for invalid_subdomain case in Google auth

We have all the plumbing logic to customize the confirm_continue_registration error messages for the case of the user having an account with that email in another organization/realm/subdomain (via invalid_subdomain being passed into login_or_register_remote_user).  Ideally, we'd customize the error message in that case.
This is a follow-up to c88163eea8aeeb29da1aa67a29cb925b96e89134.

@zulipbot claim

@zulipbot claim


Added smoke_tcp_ok

Added smoke_tcp_ok function, to test a connection using tcp.

Can you revert the t and squash your commits. I like the addition though! 👍

Reverted!

@lpicanco You need to rebase onto master as some other features were merged recently and it now has conflicts.

Rebased!


Remove unused parameter from private method in widget_inspector

Description
Remove unused parameter from private method in widget_inspector. This is required to enable checking for unused optional parameters of private
functions. dart-lang/sdk#29731
Related Issues
Tests
I added no following tests.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: https://flutter.dev/go/breaking-changes-template Replace this with a link to a pull request that adds the migration guide to https://flutter.dev/docs/release/breaking-changes

🥂


KAFKA-12648: extend IQ APIs to work with named topologies

In the new NamedTopology API being worked on, state store names and their uniqueness requirement is going to be scoped only to the owning topology, rather than to the entire app. In other words, two different named topologies can have different state stores with the same name.
This is going to cause problems for some of the existing IQ APIs which require only a name to resolve the underlying state store. We're now going to need to take in the topology name in addition to the state store name to be able to locate the specific store a user wants to query
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Hey @tolgadur can we also add one more thing to this -- see https://github.com/tolgadur/kafka/pull/1


Truth Maintenance?

Just curious...
Have you considered supporting Truth Maintenance? Why?/Why not?

I thought about it but i need a really specific use case to implement anything, and so far i haven't needed it. I am also really hoping to keep the codebase small. I find that a lot of uses of truth maintenance in clara are really just to get around its inability to directly update facts. There are certainly uses beyond that though...

Yeah if i can think of a good way of explaining it i'll add it. My last sentence was just a hedge because i am not certain what those use cases would be -- but i don't want to assume they don't exist. Let's take the example on clara's website: https://www.clara-rules.org/docs/truthmaint/
Their rule makes you a "PremierCustomer" if you have more than 2000 purchases. But, if after your 2000th order you return an item, it will automatically retract that fact so you won't be a premier customer anymore.
In o'doyle you could do it naively like this:
::premier-customer
[:what
 [person-id ::total-purchases amount]
 :then
 (o/insert! person-id ::premier-customer? (> amount 2000))]

This will always update the ::premier-customer? boolean when the ::total-purchases fact is updated. However, it will cause unnecessary work since you're constantly updating the fact when it's value hasn't changed. You could avoid that by checking the existing value like this:
::premier-customer
[:what
 [person-id ::total-purchases amount]
 [person-id ::premier-customer? premier? {:then false}]
 :then
 (let [new-premier? (> amount 2000)]
   (when (not= premier? new-premier?)
     (o/insert! person-id ::premier-customer? new-premier?)))]

Thanks for the explanation, @oakes !

Hello @oakes , very nice library and presentation as usual. I have been watching other presentations about your text editors and very amazed by all the ideas :)
I implemented a more complete example from Clara Rules to talk about Truth Maintenance (TM) and I would like some code review https://github.com/wandersoncferreira/odoyle-example-truth-maintainance/blob/main/src/odoyle/truth.clj this is my first contact with odoyle and I am still wrapping my head around it.
The comparison basically shows what TM guarantees to users:

a new fact (F1) triggers rule A
rule A inserts new fact (F2) in the database
fact F1 is retracted
4. O'Doyle does not retract F2, Clara Rules does.

Maybe I am missing something.
I relied on this feature some years ago while modeling some financial contracts that had additional entities created depending on contract state e.g.

At 1PM the contract owner sent a partial payment, then I created two new entities RemainderValue, AdditionalFees
At 2PM the contract owner sent the rest of the money, then the two previous entities got removed and a new ContractPaid created

The second step happened automatically because the status of the contract that triggered the creation of those two previous entities now has changed.
Is that a way to replicate this behavior?
Thanks!

I took a look at your code. Firstly, instead of making the ::insert-cold-temperature rule insert new facts, why not just use it as a query directly? In other words, instead of (o/query-all with-retracted-session ::cold) you can just do (o/query-all with-retracted-session ::insert-cold-temperature) and it will return what you want. You can rename it to something like ::cold-temperatures and get rid of the :then block.
It looks like (o/query-all with-retracted-session ::records-facts) works fine, which makes sense, because ultimately that data is coming from a :then-finally which does run after retractions. So any derived facts that are created that way should update correctly after a fact is retracted.

why not just use it as a query directly?

Nice! That was it! I updated other parts of the example to ask queries directly and it produces the same output as Clara :)

I have some experience with Clara as well and the Truth Maintenance was useful, but I'm looking at O'Doyle because for my use case I think TM will actually get in the way.
I'm planning on triggering threads to fetch facts (in RHS) and then update the session with facts like in "agent style" (serialize the responses to the session). I'm guessing this won't be a problem (@oakes please let me know otherwise) with O'Doyle, but doing this in Clara would be problematic. I think the core issue is that there isn't a way to insert facts into Clara and say "oh yeah, btw this is a result of a rule that fired earlier". So, if TM was added to O'Doyle it would be nice if the API could expect "out-of-band" RHSs.


One branch fmx fixes 2022 03 22

fixes required for LSDC server and GUI to start up, and also do governor transitions by using the user screen in the GUI.

I would say that getting things working is the first priority - optimizing without lots of testing when there are explanations for the sleeps is the road to frustration.


[health checker] Validate that required fields are greater than 0

Description:
While protobuf validation exists for things like interval and timeout in cluster health checking configuration, there is no check
for whether values passed in make sense. Setting the interval to something nonsensical like 0 makes the process spin in an error loop so would be a lot more straightforward to have envoy fail fast in cases like these.

@brirams do you think adding, e.g. for interval,
  // The interval between health checks.
  google.protobuf.Duration interval = 2
      [(validate.rules).duration = {required: true, gt: {seconds: 0}}];

is enough? Or do you have a sensible min value in mind?

@dio agree with that fix.

coming back to this a bit late but that definitely seems reasonable. Thanks!


Add Estimation of Distribution Algorithms (EDAs)

Estimation of Distribution Algorithms are stochastic optimisation techniques that take a model-based approach in contract to a population-based approach of genetic algorithms. This is done by building probabilistic models of the most promising candidate solutions. This model is then sampled for more candidate solution and the model is updated/refined iteratively. The model represents the probability distribution of candidate solutions where the probabilities are proportional to the quality of the solution. For a great survey of these algorithms, see this paper.
I would like to merge the following two algorithms into CIlib:

Univariate Marginal Distribution Algorithm (UMDA)
Stochastic Hill-Climbing with Learning by Vectors of Normal Distributions (SHCLVND)

UMDA uses a univariate model and discrete variables, specifically binary strings. SHCLVND is uses real-valued vectors and all variables are assumed to be independent.
Future algorithms to add:

Population-Based Incremental Learning (PBIL)
Extended Compact Genetic Algorithm (ECGA)
Mixed Iterated Density Estimation Evolutionary Algorithm (mIDEA)

I have some questions:

Should the EDAs go in their own package, such as cilib.eda? They are relatively similar to GAs so I'm not sure whether to put them in the cilib.ga package.
How should discrete problems be handled? The position of an Entity is declared as the Spire Numeric type so I am wondering how to support bit vectors, especially the scodec-bits library that @gpampara suggested I use

Thank you!

I'm closing this for now - we can revisit as soon as there is interest in this set of algorithms.


Upgrade to v3004.1

Upgrade salt-master to version 3004.1 Silicon
Upgrade salt-bootstrap to version 2022.03.15
Upgrade pygit2 to version 1.9.1
Upgrade libgit2 to version 1.4.2
Fix Jinja2 version to avoid Markup import error (https://github.com/saltstack/salt/issues/61848)
Change Docker base image to ubuntu:hirsute-20220113

More info on Salt 3004.1 is available in the official Release Notes.

Can you approve this please?!

Can you approve this please?!

Hi!
There is an issue with the jinja2 package version. I'm working to fix it asap.

I saw the issue while testing your build.  I will pull the latest version and test it to.
Thanks!

Surprised you're not getting this error.
Step 15/25 : COPY assets/sbin/* /usr/local/sbin
1 error occurred:
* Status: When using COPY with more than one source file, the destination must be a directory and end with a /, Code: 1

Jinja2 error is fixed now. But libgit2 is not finding libssh2...
I'm fixing it right now, and also I'm trying to improve compilation times by disabling tests compilation...

Just traditional dependency hell :dagger:

Testing!  Thanks a bunch!!

Thanks @visbits for using my image! Just let me know if you find bugs!


Tensorflow1.3 dependencies

Motivation for this change
This branch prepares for tensorflow 1.3 by adding the necessary dependencies.
Unfortunately, this branch does not add tensorflow 1.3 itself because of a circular dependency between tensorflow and tensorflow_tensorboard.
Reviewers: please advise as how to deal with such a circular dependency between python packages. The manual https://nixos.org/nixpkgs/manual/#how-to-solve-circular-dependencies prescribe to override the packages to remove the offending dependency however it is silent on how this should be done.
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[x] Linux

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Unfortunately, this branch does not add tensorflow 1.3 itself because of a circular dependency between tensorflow and tensorflow_tensorboard.

Reviewers: please advise as how to deal with such a circular dependency between python packages. The manual https://nixos.org/nixpkgs/manual/#how-to-solve-circular-dependencies prescribe to override the packages to remove the offending dependency however it is silent on how this should be done.

Are they build from source or are you installing binary wheels?

In case of the former, it typically means overriding the install_requires that is passed to setup() in setup.py. In case of the latter there is no solution, as far as I know.

Regarding the circular dependency: tensorflow and tensorboard are provided as wheels. I found a workaround though: one can supply --no-deps to pip. Then it won't check the dependencies at install time.
But then again, I cannot install both tensorflow and tensorboard because of the following collision:
 /nix/store/wbw4k3ih95qix8fqrwv35c32s54laj51-python3.5-tensorflow-1.3.0/lib/python3.5/site-packages/external/__pycache__/__init__.cpython-35.pyc' and
/nix/store/glc905np5xqpj5pfy0mph12ihxmvrwiv-python3.5-tensorflow_tensorboard-0.1.4/lib/python3.5/site-packages/external/__pycache__/__init__.cpython-35.pyc

Tensorflow is still usable without tensorboard, so I would propose to not install tensorboard for the time being.

@FRidh
I have apply the style changes.  bleach 1.5 may be dropped in case we go with my proposal (above comment).

@jyp if it checks for dependencies then its likely that it is also needed during runtime. Have you checked upstream what they say about requiring bleach?

@FRidh I don't think I made myself so clear :)
I'm thinking to remove tensorboard  from the dependencies of tensorflow (core). Tensorboard is the visualization engine of tensorflow. According to my tests it isn't required to run tensorflow models --- only to visualize the results. So as far as I understand tensorboard is not a necessary component and should be dropped from the dependencies of tensorflow --- even upstream.
Now, we have then the opportunity to remove tensorboard entirely from nixpkgs; and in turn its dependencies, including bleach 1.5.

@FRidh Do you think that this PR could be accepted in the current state? I've dropped the extra bleach version. What remains is I believe rather uncontroversial:

update version of markdown
add newest protobuf version
add newest cudnn version
add weakref backport

I've added these commits to #29009.


what hardware configuration is needed to run your work?

Dear authors, thank you for your excellent work. I want to know what hardware configuration is needed to run your work.

An RTX 3090 with 24G memory is enough.


Did anybody know how to add a nova-component inside the collapsible resource manager?

sorry, all good!

Can you share please ?


Fixing Spanish with Paulina

Working with @montanez at hacknight to translate some remaining sneaky English and improve the Spanish we had to be more consistent. (Pledge is Promesa!)
It relates to the following issue #s:

Bumps #1633

lgtm. I think it needs an i18n-tasks normalize, after that let's merge


The Travis CI access fail due of permission

@sole I can't access to settings. Or i should leave the app? If yes, i will leave :(

Sorry @sole. It's maybe mistake of npmjs or [travis], i not sure

Sorry @dalisoft it is my fault! I revoked a token and forgot to update it on this repository. I will do it and update you here

Is it working now? it looks like it should be?

@sole My tokens will been not a one, i am curius because nobody can't access to my profile and can't revoke my token without my permission, right? There it's not your fault. My tokens was used for 2 repos, not a one. Anyway i am already released release with my fork. I test it later

@sole I am still can't access to Travis CI

I don't understand what you're trying to do - can you explain exactly what you're trying to do and where?

I cant set my token for es6 tween on travis to release new version

Right, I don't really know what is actually happening. Looking at the travis output I can only see
><EMAIL_ADDRESS>semantic-release /home/travis/build/tweenjs/es6-tween
> semantic-release pre && npm publish && semantic-release post --owner=sole --user=dalisoft
[Travis Deploy Once]: Success at attempt 1. All 1 jobs passed.
semantic-release ERR! pre Failed to determine new version.
semantic-release ERR! pre ENOCHANGE There are no relevant changes, so no new version is released.

and I don't know why is it failing to determine a new version.
I'll tell you what I've looked at, and let's see if together we can figure out what's wrong:

In travis settings, under the tween.js/es6-tweenjs settings there are GH and NPM tokens

My first suspect is here: the NPM token belongs to the user tweenjsadmin that I created so that publishes will not be done by me, but by that account.
But your semantic release configuration tries to publish as owner = sole using a token for tweenjsadmin, so that's not going to work. Maybe if you change the config to use the tweenjsadmin it will work?

We also had a similar issue a while ago with semantic-release which couldn't determine the current version

Yes, only one account can be publish with Travis + semantic-release

semantic-release ERR! pre Failed to determine new version.
semantic-release ERR! pre ENOCHANGE There are no relevant changes, so no new version is released.
```.

Try trigger the version by comment on the commit

fix(travis): trigger travis, may fix the issue

@sole semantic-release fixes in v2.2.1, i did some config changes and improvements for performance and repo. I think you like it


Create new icons for Clarity iconography: vol 7

Select one ...  (check one with "x")
[ ] bug
[ ] feature request
[ X ] enhancement

Create a new set of 10 (or more) icons to add to Clarity Iconography: volume 7

Set 1 of 2:
vol7_ions_1of2.zip

See Icon categories here:
https://confluence.eng.vmware.com/pages/editpage.action?pageId=196082902

One SVG in the file is using strokes and I can't use Sketch to convert it properly. Sketch messes with the sizing too much and moves the image to the left. I've contacted @lil-kim to get a non-stroked version of this SVG.
Completion of this issue is blocked until then. This is a priority for today because today is the only day I have to get a PR in for this issue.


check child is meaningless

Check child is meaning. when code run here, child must be null!

Coverage remained the same at 100.0% when pulling 20a257e497ab8610eabc10da6e17162060abebb3 on MrErHu:master into f7834ec9fb7848581c6fbc7833bcbe2f2db9fe61 on developit:master.

I don't think this is true for children with mixed keyed/unkeyed.

@developit We reassign child to null just a few lines above this check https://github.com/developit/preact/blob/master/src/vdom/diff.js#L195 . Seems like this can be indeed removed.


Setup Linters & React App

Hi 👋 Code Reviewer:
In this project:

build a react app using Create React App .
Set up Linters.
used correctly Gitflow.
Follow JavaScript best practices.

Hi @Strangeal, Thanks a lot for your efforts and advice 😄.

Hi, @frank1738 Thanks a lot for your efforts in reviewing my code 😃 .


about thread safe

hello, I have a problem.
pseudocode:
conn = clusterClient.connect() for(i : list){   executor.submit(()->{     futures.add(conn.sync.evalsha(sha, [i], args...))   }) }
assume list.size is 2,executor's thread greater than 2,args is not same each time.
when the both 'i' are same,redis server executes script once.
when the both 'i' are not same,redis server executes script twice.
I see "A scalable thread-safe Redis cluster client. Multiple threads may share one connection. " in doc.
I know this usage is more terrible than use async().

so the RedisClusterClient is still thread-safe?

my version is 4.3.0 final

sorry . It's my problem.


Use dataclasses

Thanks to @MASaraji for introducing such a useful thing to me.
You can use this to reduce lines of your code and making it more readable as well.
Here's a link:
https://docs.python.org/3/library/dataclasses.html

can  you help me with something
i think something not right about when i send my mail to GitHub its information needed
to protect community  of devs and people and knowledge of ai
i cant trust my computer nor my internet provider ;'( nor people around me

can you help me with something
i think something not right about when i send my mail to GitHub its information needed
to protect community of devs and people and knowledge of ai
i cant trust my computer nor my internet provider ;'( nor people around me

You sure you're in the right place?!

sorry i go find someone else help me

can you give me github support mail

sorry man
i got panic nothing more sorry for trouble

I don't know the email address but here's a link which might help you: https://support.github.com/request

i just didn't understand what happened here
sorry my action was bad
i just got panic attack
i hope you the best man
;( sorry
thanks for helping me

You are welcome!

there is module in python caller attr it will help you write less code

https://github.com/python-attrs/attrs page
they have home page too
its like dataclasses may help you in your project


Allow user to not require you download all docs from external CDN source

I am adding an option that allows you to download an entire site (even from external CDN sources) to make site completely offline capable.
But sometimes I do not want this, so make option that allows user not to do this (maybe a '-a' flag or something)

Do this after I add my new "wget" flags currently only my dev machine.


Include in toml-lang list of implementations

You should submit a PR to toml-lang/toml in order to be included in their README as a TOML implementation.
See here.

I plan on doing so, but I want to implement a few more features and have more extensive testing first.


API ldap/users/import

When we try import a non existent user on ldap a error 500 is returned by endpoint without any kind of information.
Would be nice if the API returns a 404 with a array with users not found on LDAP.

Fixed it in  #4674


Feature request: Bridge Mode?

I'm a noob when it comes to this, but I am curious if through a similar way that you disabled wi-fi if this unit could be put into bridge mode? The lack of this feature from T-mobile is insane. Bummed to have to send it back.

It's entirely possible. TBH - I am not an expert on this unit. I just wrote a script to automate the process detailed in this video:
https://www.youtube.com/watch?v=FTI9AOaYedc
If you can find docs detailing this feature and the appropriate endpoint, I could add it.


The list-item-indent rule is buggy when it comes to checkboxes

╭─soda  ~/Tests/remark-lint
╰─$ npm list -g --depth=0 remark remark-lint
/Users/soda/.nvm/versions/node/v4.2.6/lib
+--<EMAIL_ADDRESS>`--<EMAIL_ADDRESS>
╭─soda  ~/Tests/remark-lint
╰─$ cat test.md
- [ ] Item 1
- [x] Item 2
- [ ] Item 3
╭─soda  ~/Tests/remark-lint
╰─$ cat .remarkrc
{
    "plugins": {
        "lint": {
            "reset": true,
            "list-item-indent": 'space'
        }
    },
    "settings": {
        "gfm": true
    }
}
╭─soda  ~/Tests/remark-lint
╰─$ remark test.md -u remark-lint -c .remarkrc
-   [ ] Item 1
-   [x] Item 2
-   [ ] Item 3
test.md
        1:7  warning  Incorrect list-item indent: add 2 spaces  list-item-indent
        2:7  warning  Incorrect list-item indent: add 2 spaces  list-item-indent
        3:7  warning  Incorrect list-item indent: add 2 spaces  list-item-indent

⚠ 3 warnings

Actually it is broken for any lists……
Maybe I misunderstood this rule's meaning?

Well, first of all, remark detects the .remarkrc file and automatically loads the used plug-ins, namely, remark-lint, so you don’t need to specify it on the CLI too (that’s why you get double warnings).
The default style suggested by list-item-indent is to "tab-size" indentation, which is why you see the warnings (as it detects an indentation of 2, not 4). See the docs on how to change that behviour.

I have reset this rule to 'space' (there was a typo in the original post, but I've fixed that and the error remains)
I typed only one space, but remark detected 2, this is confusing

Oh I see. It's the -u remark-lint part in the command that causes this weird behavior.

See the first paragraph of my previous post, you must remove -u remark-lint.

Yup, it causes duplicate remark-lint, the first with options the second without

But it's copied from README.md

Use remark-lint together with remark:
npm install --global remark remark-lint

Let’s say example.md looks as follows:
* Hello

[World][]

Then, to run remark-lint on example.md:
remark example.md -u remark-lint

The documentation needs to be updated.

No, the usage section does not contain a remarkrc file.
The remarkrc example later is a different example, showing different settings and different results.


Josh dev

Finally got this working... I think something was wrong with the documents that I already had in my local DB for some reason.
Note: The error messages for the endpoints changed for some reason so that on the frontend you get the message in error.error instead of error.message. I am not at all sure why this happened because I didn't change the responses in the /join or /register routes. I went through and changed the frontend to accommodate this change.

So this is ready to go? 😄

@torch2424 Yeah, should be good now. Just added one more to check to make sure email isn't already taken.

Forsure, Do you mind if I pull this in after the demo? Just to be safe? :)

@torch2424 I kind of wanted it for the demo, that is why I was rushing to finish it yesterday haha but if you aren't feeling it, no worries 👍


Added watchdog protection

Increases robustness when interfacing with USB devices. Protects against processes that exit before whole FW is sent to device at boot stage and removes the need to power cycle the device in that case.
Includes https://github.com/luxonis/XLink/pull/39 changes which address similar case but at MXID retrieval step.
Fixes: #291
CC: @diablodale for giving it a try on how it pairs with #291
EDIT: And on #300

Fixes: #300

Sure, I can get you feedback Thursday evening.

To test it, I setup my Windows VS2019 x64 shared_lib, debug ...

https://github.com/luxonis/depthai-core/tree/watchdog_protection
git submodule update --init --recursive
https://github.com/luxonis/XLink/tree/watchdog_protection

The test+example has many failures. The first three...
test 1 failed with
[ctest] 1: Stack trace (most recent call last):
[ctest] 1: #16   Object "", at 00007FF9C9717034, in BaseThreadInitThunk
[ctest] 1: #15   Source "d:\a01\_work\12\s\src\vctools\crt\vcstartup\src\startup\exe_main.cpp", line 17, in mainCRTStartup [00007FF69136D3FE]
[ctest] 1: #14   Source "d:\a01\_work\12\s\src\vctools\crt\vcstartup\src\startup\exe_common.inl", line 331, in __scrt_common_main [00007FF69136D0CE]
[ctest] 1: #13   Source "d:\a01\_work\12\s\src\vctools\crt\vcstartup\src\startup\exe_common.inl", line 288, in __scrt_common_main_seh [00007FF69136D20E]
[ctest] 1: #12   Source "d:\a01\_work\12\s\src\vctools\crt\vcstartup\src\startup\exe_common.inl", line 79, in invoke_main [00007FF69136D369]
[ctest] 1: #11   Source "C:\njs\depthai-core\tests\src\color_camera_node_test.cpp", line 41, in main [00007FF6913627CC]
[ctest] 1:          38:         std::tie(found, deviceInfo) = dai::Device::getFirstAvailableDevice();
[ctest] 1:          39:
[ctest] 1:          40:         if(found) {
[ctest] 1:       >  41:             dai::Device d(p, deviceInfo);
[ctest] 1:          42:
[ctest] 1:          43:             auto previewQueue = d.getOutputQueue("preview", 8, true);
[ctest] 1: #10   Source "C:\njs\depthai-core\src\device\Device.cpp", line 46, in dai::Device::Device [00007FF94B9B91D2]
[ctest] 1:          43:     tryStartPipeline(pipeline);
[ctest] 1:          44: }
[ctest] 1:          45:
[ctest] 1:       >  46: Device::Device(const Pipeline& pipeline, const DeviceInfo& devInfo) : DeviceBase(pipeline.getOpenVINOVersion(), devInfo, false) {
[ctest] 1:          47:     tryStartPipeline(pipeline);
[ctest] 1:          48: }
[ctest] 1: #9    Source "C:\njs\depthai-core\src\device\DeviceBase.cpp", line 233, in dai::DeviceBase::DeviceBase [00007FF94B9D56B2]
[ctest] 1:         230: DeviceBase::DeviceBase(OpenVINO::Version version, const DeviceInfo& devInfo) : DeviceBase(version, devInfo, false) {}
[ctest] 1:         231:
[ctest] 1:         232: DeviceBase::DeviceBase(OpenVINO::Version version, const DeviceInfo& devInfo, bool usb2Mode) : deviceInfo(devInfo) {
[ctest] 1:       > 233:     init(version, usb2Mode, "");
[ctest] 1:         234: }
[ctest] 1:         235:
[ctest] 1:         236: DeviceBase::DeviceBase(OpenVINO::Version version, const DeviceInfo& devInfo, UsbSpeed maxUsbSpeed) : deviceInfo(devInfo) {
[ctest] 1: #8    Source "C:\njs\depthai-core\src\device\DeviceBase.cpp", line 425, in dai::DeviceBase::init [00007FF94B9D8471]
[ctest] 1:         422:     // Specify the OpenVINO version
[ctest] 1:         423:     cfg.version = version;
[ctest] 1:         424:     init2(cfg, pathToMvcmd, {});
[ctest] 1:       > 425: }
[ctest] 1:         426: void DeviceBase::init(const Pipeline& pipeline, bool usb2Mode, const std::string& pathToMvcmd) {
[ctest] 1:         427:     Config cfg = pipeline.getDeviceConfig();
[ctest] 1:         428:     // Modify usb speed
[ctest] 1: #7    Source "C:\njs\depthai-core\src\device\DeviceBase.cpp", line 500, in dai::DeviceBase::init2 [00007FF94B9D8BC6]
[ctest] 1:         497:     // Init device (if bootloader, handle correctly - issue USB boot command)
[ctest] 1:         498:     if(deviceInfo.state == X_LINK_UNBOOTED) {
[ctest] 1:         499:         // Unbooted device found, boot and connect with XLinkConnection constructor
[ctest] 1:       > 500:         connection = std::make_shared<XLinkConnection>(deviceInfo, fwWithConfig);
[ctest] 1:         501:     } else if(deviceInfo.state == X_LINK_BOOTLOADER || deviceInfo.state == X_LINK_FLASH_BOOTED) {
[ctest] 1:         502:         // If device is in flash booted state, reset to bootloader and then continue by booting appropriate FW
[ctest] 1:         503:         if(deviceInfo.state == X_LINK_FLASH_BOOTED) {
[ctest] 1: #6    Source "C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.29.30133\include\memory", line 2748, in std::make_shared<dai::XLinkConnection,dai::DeviceInfo &,std::vector<unsigned char,std::allocator<unsigned char> > &> [00007FF94BA320C3]
[ctest] 1:        2745:     shared_ptr<_Ty>
[ctest] 1:        2746: #endif // _HAS_CXX20
[ctest] 1:        2747:     make_shared(_Types&&... _Args) { // make a shared_ptr to non-array object
[ctest] 1:       >2748:     const auto _Rx = new _Ref_count_obj2<_Ty>(_STD forward<_Types>(_Args)...);
[ctest] 1:        2749:     shared_ptr<_Ty> _Ret;
[ctest] 1:        2750:     _Ret._Set_ptr_rep_and_enable_shared(_STD addressof(_Rx->_Storage._Value), _Rx);
[ctest] 1:        2751:     return _Ret;
[ctest] 1: #5    Source "C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.29.30133\include\memory", line 2029, in std::_Ref_count_obj2<dai::XLinkConnection>::_Ref_count_obj2<dai::XLinkConnection><dai::DeviceInfo &,std::vector<unsigned char,std::allocator<unsigned char> > &> [00007FF94B9DCF1E]
[ctest] 1:        2026:         } else
[ctest] 1:        2027: #endif // _HAS_CXX20
[ctest] 1:        2028:         {
[ctest] 1:       >2029:             _Construct_in_place(_Storage._Value, _STD forward<_Types>(_Args)...);
[ctest] 1:        2030:         }
[ctest] 1:        2031:     }
[ctest] 1: #4    Source "C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.29.30133\include\xutility", line 160, in std::_Construct_in_place<dai::XLinkConnection,dai::DeviceInfo &,std::vector<unsigned char,std::allocator<unsigned char> > &> [00007FF94B9F89C3]
[ctest] 1:         157:     {
[ctest] 1:         158:         ::new (_Voidify_iter(_STD addressof(_Obj))) _Ty(_STD forward<_Types>(_Args)...);
[ctest] 1:         159:     }
[ctest] 1:       > 160: }
[ctest] 1:         161:
[ctest] 1:         162: // FUNCTION TEMPLATE _Default_construct_in_place
[ctest] 1:         163: template <class _Ty>
[ctest] 1: #3    Source "C:\njs\depthai-core\src\xlink\XLinkConnection.cpp", line 150, in dai::XLinkConnection::XLinkConnection [00007FF94BC5E679]
[ctest] 1:         147:     bootDevice = true;
[ctest] 1:         148:     bootWithPath = false;
[ctest] 1:         149:     this->mvcmd = std::move(mvcmdBinary);
[ctest] 1:       > 150:     initDevice(deviceDesc, expectedState);
[ctest] 1:         151: }
[ctest] 1:         152:
[ctest] 1:         153: XLinkConnection::XLinkConnection(const DeviceInfo& deviceDesc, std::string pathToMvcmd, XLinkDeviceState_t expectedState) {
[ctest] 1: #2    Source "C:\njs\depthai-core\src\xlink\XLinkConnection.cpp", line 305, in dai::XLinkConnection::initDevice [00007FF94BC5F6CE]
[ctest] 1:         302:     // Search for booted device
[ctest] 1:         303:     {
[ctest] 1:         304:         // Create description of device to look for
[ctest] 1:       > 305:         DeviceInfo bootedDeviceInfo = deviceToInit;
[ctest] 1:         306:         if(deviceToInit.desc.protocol != X_LINK_TCP_IP) {
[ctest] 1:         307:             bootedDeviceInfo = deviceInfoFix(deviceToInit, expectedState);
[ctest] 1:         308:         }
[ctest] 1: #1    Object "", at 00007FF99D0DB460, in CxxThrowException
[ctest] 1: #0    Object "", at 00007FF9C8434F69, in RaiseException
[ctest]  1/65 Test  #1: color_camera_node_test ..............***Failed    1.72 sec

test 2 failed
[ctest] 2: Stack trace (most recent call last):
[ctest] 2: #16   Object "", at 00007FF9C9717034, in BaseThreadInitThunk
[ctest] 2: #15   Source "d:\a01\_work\12\s\src\vctools\crt\vcstartup\src\startup\exe_main.cpp", line 17, in mainCRTStartup [00007FF63917F49E]
[ctest] 2: #14   Source "d:\a01\_work\12\s\src\vctools\crt\vcstartup\src\startup\exe_common.inl", line 331, in __scrt_common_main [00007FF63917F16E]
[ctest] 2: #13   Source "d:\a01\_work\12\s\src\vctools\crt\vcstartup\src\startup\exe_common.inl", line 288, in __scrt_common_main_seh [00007FF63917F2AE]
[ctest] 2: #12   Source "d:\a01\_work\12\s\src\vctools\crt\vcstartup\src\startup\exe_common.inl", line 79, in invoke_main [00007FF63917F409]
[ctest] 2: #11   Source "C:\njs\depthai-core\tests\src\image_manip_node_test.cpp", line 43, in main [00007FF639172B44]
[ctest] 2:          40:         std::tie(found, deviceInfo) = dai::Device::getFirstAvailableDevice();
[ctest] 2:          41:
[ctest] 2:          42:         if(found) {
[ctest] 2:       >  43:             dai::Device d(p, deviceInfo);
[ctest] 2:          44:
[ctest] 2:          45:             auto in = d.getInputQueue("in");
[ctest] 2:          46:             auto out = d.getOutputQueue("out");
[ctest] 2: #10   Source "C:\njs\depthai-core\src\device\Device.cpp", line 46, in dai::Device::Device [00007FF94B9B91D2]
[ctest] 2:          43:     tryStartPipeline(pipeline);
[ctest] 2:          44: }
[ctest] 2:          45:
[ctest] 2:       >  46: Device::Device(const Pipeline& pipeline, const DeviceInfo& devInfo) : DeviceBase(pipeline.getOpenVINOVersion(), devInfo, false) {
[ctest] 2:          47:     tryStartPipeline(pipeline);
[ctest] 2:          48: }
[ctest] 2: #9    Source "C:\njs\depthai-core\src\device\DeviceBase.cpp", line 233, in dai::DeviceBase::DeviceBase [00007FF94B9D56B2]
[ctest] 2:         230: DeviceBase::DeviceBase(OpenVINO::Version version, const DeviceInfo& devInfo) : DeviceBase(version, devInfo, false) {}
[ctest] 2:         231:
[ctest] 2:         232: DeviceBase::DeviceBase(OpenVINO::Version version, const DeviceInfo& devInfo, bool usb2Mode) : deviceInfo(devInfo) {
[ctest] 2:       > 233:     init(version, usb2Mode, "");
[ctest] 2:         234: }
[ctest] 2:         235:
[ctest] 2:         236: DeviceBase::DeviceBase(OpenVINO::Version version, const DeviceInfo& devInfo, UsbSpeed maxUsbSpeed) : deviceInfo(devInfo) {
[ctest] 2: #8    Source "C:\njs\depthai-core\src\device\DeviceBase.cpp", line 425, in dai::DeviceBase::init [00007FF94B9D8471]
[ctest] 2:         422:     // Specify the OpenVINO version
[ctest] 2:         423:     cfg.version = version;
[ctest] 2:         424:     init2(cfg, pathToMvcmd, {});
[ctest] 2:       > 425: }
[ctest] 2:         426: void DeviceBase::init(const Pipeline& pipeline, bool usb2Mode, const std::string& pathToMvcmd) {
[ctest] 2:         427:     Config cfg = pipeline.getDeviceConfig();
[ctest] 2:         428:     // Modify usb speed
[ctest] 2: #7    Source "C:\njs\depthai-core\src\device\DeviceBase.cpp", line 500, in dai::DeviceBase::init2 [00007FF94B9D8BC6]
[ctest] 2:         497:     // Init device (if bootloader, handle correctly - issue USB boot command)
[ctest] 2:         498:     if(deviceInfo.state == X_LINK_UNBOOTED) {
[ctest] 2:         499:         // Unbooted device found, boot and connect with XLinkConnection constructor
[ctest] 2:       > 500:         connection = std::make_shared<XLinkConnection>(deviceInfo, fwWithConfig);
[ctest] 2:         501:     } else if(deviceInfo.state == X_LINK_BOOTLOADER || deviceInfo.state == X_LINK_FLASH_BOOTED) {
[ctest] 2:         502:         // If device is in flash booted state, reset to bootloader and then continue by booting appropriate FW
[ctest] 2:         503:         if(deviceInfo.state == X_LINK_FLASH_BOOTED) {
[ctest] 2: #6    Source "C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.29.30133\include\memory", line 2748, in std::make_shared<dai::XLinkConnection,dai::DeviceInfo &,std::vector<unsigned char,std::allocator<unsigned char> > &> [00007FF94BA320C3]
[ctest] 2:        2745:     shared_ptr<_Ty>
[ctest] 2:        2746: #endif // _HAS_CXX20
[ctest] 2:        2747:     make_shared(_Types&&... _Args) { // make a shared_ptr to non-array object
[ctest] 2:       >2748:     const auto _Rx = new _Ref_count_obj2<_Ty>(_STD forward<_Types>(_Args)...);
[ctest] 2:        2749:     shared_ptr<_Ty> _Ret;
[ctest] 2:        2750:     _Ret._Set_ptr_rep_and_enable_shared(_STD addressof(_Rx->_Storage._Value), _Rx);
[ctest] 2:        2751:     return _Ret;
[ctest] 2: #5    Source "C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.29.30133\include\memory", line 2029, in std::_Ref_count_obj2<dai::XLinkConnection>::_Ref_count_obj2<dai::XLinkConnection><dai::DeviceInfo &,std::vector<unsigned char,std::allocator<unsigned char> > &> [00007FF94B9DCF1E]
[ctest] 2:        2026:         } else
[ctest] 2:        2027: #endif // _HAS_CXX20
[ctest] 2:        2028:         {
[ctest] 2:       >2029:             _Construct_in_place(_Storage._Value, _STD forward<_Types>(_Args)...);
[ctest] 2:        2030:         }
[ctest] 2:        2031:     }
[ctest] 2: #4    Source "C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.29.30133\include\xutility", line 160, in std::_Construct_in_place<dai::XLinkConnection,dai::DeviceInfo &,std::vector<unsigned char,std::allocator<unsigned char> > &> [00007FF94B9F89C3]
[ctest] 2:         157:     {
[ctest] 2:         158:         ::new (_Voidify_iter(_STD addressof(_Obj))) _Ty(_STD forward<_Types>(_Args)...);
[ctest] 2:         159:     }
[ctest] 2:       > 160: }
[ctest] 2:         161:
[ctest] 2:         162: // FUNCTION TEMPLATE _Default_construct_in_place
[ctest] 2:         163: template <class _Ty>
[ctest] 2: #3    Source "C:\njs\depthai-core\src\xlink\XLinkConnection.cpp", line 150, in dai::XLinkConnection::XLinkConnection [00007FF94BC5E679]
[ctest] 2:         147:     bootDevice = true;
[ctest] 2:         148:     bootWithPath = false;
[ctest] 2:         149:     this->mvcmd = std::move(mvcmdBinary);
[ctest] 2:       > 150:     initDevice(deviceDesc, expectedState);
[ctest] 2:         151: }
[ctest] 2:         152:
[ctest] 2:         153: XLinkConnection::XLinkConnection(const DeviceInfo& deviceDesc, std::string pathToMvcmd, XLinkDeviceState_t expectedState) {
[ctest] 2: #2    Source "C:\njs\depthai-core\src\xlink\XLinkConnection.cpp", line 288, in dai::XLinkConnection::initDevice [00007FF94BC5F5FF]
[ctest] 2:         286:         // If device not found
[ctest] 2:         287:         if(rc != X_LINK_SUCCESS) {
[ctest] 2:       > 288:             throw std::runtime_error("Failed to find device (" + std::string(deviceToBoot.desc.name) + "), error message: " + convertErrorCodeToString(rc));
[ctest] 2:         289:         }
[ctest] 2:         290:
[ctest] 2:         291:         bool bootStatus;
[ctest] 2: #1    Object "", at 00007FF99D0DB460, in CxxThrowException
[ctest] 2: #0    Object "", at 00007FF9C8434F69, in RaiseException
[ctest]  2/65 Test  #2: image_manip_node_test ...............***Failed   16.13 sec

test 13 failed
[ctest] 13: Stack trace (most recent call last):
[ctest] 13: #13   Object "", at 00007FF9C9717034, in BaseThreadInitThunk
[ctest] 13: #12   Source "d:\a01\_work\12\s\src\vctools\crt\vcstartup\src\startup\exe_main.cpp", line 17, in mainCRTStartup [00007FF7291C1F3E]
[ctest] 13: #11   Source "d:\a01\_work\12\s\src\vctools\crt\vcstartup\src\startup\exe_common.inl", line 331, in __scrt_common_main [00007FF7291C1C0E]
[ctest] 13: #10   Source "d:\a01\_work\12\s\src\vctools\crt\vcstartup\src\startup\exe_common.inl", line 288, in __scrt_common_main_seh [00007FF7291C1D4E]
[ctest] 13: #9    Source "d:\a01\_work\12\s\src\vctools\crt\vcstartup\src\startup\exe_common.inl", line 79, in invoke_main [00007FF7291C1EA9]
[ctest] 13: #8    Source "C:\njs\depthai-core\examples\ColorCamera\rgb_preview.cpp", line 28, in main [00007FF728976110]
[ctest] 13:          25:     camRgb->preview.link(xoutRgb->input);
[ctest] 13:          26:
[ctest] 13:          27:     // Connect to device and start pipeline
[ctest] 13:       >  28:     dai::Device device(pipeline, dai::UsbSpeed::SUPER);
[ctest] 13:          29:
[ctest] 13:          30:     cout << "Connected cameras: ";
[ctest] 13:          31:     for(const auto& cam : device.getConnectedCameras()) {
[ctest] 13: #7    Source "C:\njs\depthai-core\src\device\Device.cpp", line 34, in dai::Device::Device [00007FF94B9B8F2F]
[ctest] 13:          31:     tryStartPipeline(pipeline);
[ctest] 13:          32: }
[ctest] 13:          33:
[ctest] 13:       >  34: Device::Device(const Pipeline& pipeline, UsbSpeed maxUsbSpeed) : DeviceBase(pipeline.getOpenVINOVersion(), maxUsbSpeed) {
[ctest] 13:          35:     tryStartPipeline(pipeline);
[ctest] 13:          36: }
[ctest] 13: #6    Source "C:\njs\depthai-core\src\device\DeviceBase.cpp", line 298, in dai::DeviceBase::DeviceBase [00007FF94B9D4EED]
[ctest] 13:         296:     // If no device found, throw
[ctest] 13:         297:     if(!found) throw std::runtime_error("No available devices");
[ctest] 13:       > 298:     init(version, maxUsbSpeed, "");
[ctest] 13:         299: }
[ctest] 13:         300:
[ctest] 13:         301: DeviceBase::DeviceBase(const Pipeline& pipeline) : DeviceBase(pipeline.getOpenVINOVersion()) {
[ctest] 13: #5    Source "C:\njs\depthai-core\src\device\DeviceBase.cpp", line 439, in dai::DeviceBase::init [00007FF94B9D861D]
[ctest] 13:         436:     // Specify the OpenVINO version
[ctest] 13:         437:     cfg.version = version;
[ctest] 13:         438:     init2(cfg, pathToMvcmd, {});
[ctest] 13:       > 439: }
[ctest] 13:         440: void DeviceBase::init(const Pipeline& pipeline, UsbSpeed maxUsbSpeed, const std::string& pathToMvcmd) {
[ctest] 13:         441:     Config cfg = pipeline.getDeviceConfig();
[ctest] 13:         442:     // Modify usb speed
[ctest] 13: #4    Source "C:\njs\depthai-core\src\device\DeviceBase.cpp", line 670, in dai::DeviceBase::init2 [00007FF94B9D9179]
[ctest] 13:         667:             setLogOutputLevel(level);
[ctest] 13:         668:         } else {
[ctest] 13:         669:             setLogLevel(LogLevel::WARN);
[ctest] 13:       > 670:             setLogOutputLevel(LogLevel::WARN);
[ctest] 13:         671:         }
[ctest] 13:         672:
[ctest] 13:         673:         // Sets system inforation logging rate. By default 1s
[ctest] 13: #3    Object "", at 00007FF9CAB51456, in RtlCaptureContext2
[ctest] 13: #2    Object "", at 00007FF99F783A12, in _NLG_Return2
[ctest] 13: #1    Object "", at 00007FF99F784833, in _NLG_Return2
[ctest] 13: #0    Object "", at 00007FF9C8434F69, in RaiseException
[ctest] 13: -- After process executed,  produced the following exit code: -529697949
[ctest] 13: CMake Error at C:/njs/depthai-core/examples/cmake/ExecuteTestTimeout.cmake:42 (message):
[ctest] 13:    produced an error (-529697949) while running
[ctest] 13:
[ctest] 13:
[ctest] 13/65 Test #13: rgb_preview .........................***Failed    7.03 sec

Please wait. I deleted my whole build directory and powercycled everything. Then rebuild all.
Now I'm getting better results... tests are running..

@diablodale thanks for the feedback
It does indeed not address #300 yet - it can still return <error> device. I'll add that check as well and not return a device at all if its in that state.
Regarding the above test - does it then come back online after ~5s where its again available to be used and isn't in <error> state? This is the main intention of this changes, for the device not to get stuck there and require a powercycle.
Regarding #291 - we could increase the default search time, will circle back on this... Otherwise it should still resolve the device not found IF the wait is there for the devices to come back online. Main idea is that they now should always come back online eventually, no matter the bad state they were left in.
Note: we've seen one other issue become more apparent in this branch (OsDrvCpr critical error might sometimes pop up) - we are investigating, might be a existing error and recent FW changes just made it surface more often..

Regarding the above test - does it then come back online after ~5s where its again available to be used and isn't in <error> state? This is the main intention of this changes, for the device not to get stuck there and require a powercycle.

Yes, I think so. Same setup and repo steps as recently above.
Got "" output, pause, crash. After the crash I ran bootloader_version.exe and it successfully returned good output
...
        290:
        291:         bool bootStatus;
#1    Object "", at 00007FF9B55AB460, in CxxThrowException
#0    Object "", at 00007FF9C8434F69, in RaiseException

C:\njs\depthai-core>build\examples\bootloader_version.exe
Found device with name: 14442C10C1A1C6D200-ma2480
Version: 0.0.15

C:\njs\depthai-core>build\examples\bootloader_version.exe
Found device with name: 14442C10C1A1C6D200-ma2480
Version: 0.0.15

Regarding #291 - we could increase the default search time, will circle back on this... Otherwise it should still resolve the device not found IF the wait is there for the devices to come back online. Main idea is that they now should always come back online eventually, no matter the bad state they were left in.

I would think the behavior of the ecosystem to be the default search time is greater than the time needed for the device to respond/recover. Today's code handles respond reasonably, but does not (yet) handle recover. I can imagine several approaches

default search time in code is set to always be > device recover time
search APIs like getFirstAvailableDevice(), getAllAvailableDevices(), etc. have a search time paramstatic std::tuple<bool, DeviceInfo> getFirstAvailableDevice(chrono::milliseconds searchTimeout = WAIT_FOR_SEARCH_TIMEOUT);

update docs to caution of this search time<->recover time issue so that app dev's know to always catch and loop/retry. I hesitate slightly on this because dev's often won't read it and won't code it.

default search time in code is set to always be > device recover time

That'd be nice, however, our worst case can be a tad longer (PoE devices specifically).
I still think we should bump it, but I'm wondering if that can be too high - eg. for example if we need 15s worst case, would that be okay, or too much (examples would take 15s to tell you that no device is present). Is there an upper limit for okay or would anything work...etc..
It would be nice however as then this cases would be covered by this mechanism.

That'd be nice, however, our worst case can be a tad longer (PoE devices specifically). I still think we should bump it, but I'm wondering if that can be too high - eg. for example if we need 15s worst case, would that be okay, or too much (examples would take 15s to tell you that no device is present). Is there an upper limit for okay or would anything work...etc..

I like my option 2. Have a default that handles all reasonable scenarios and then let the app developer change it with a param.
What is reasonable time? The non-errant workflow is the case to optimize. If not already known, I recommend running lots of tests and collect exact timing with PoE and all the XLinkProtocol_t to determine the connection time curve and choose the search time based on that value.
Next, overload or specialize search/connection apis to take XLinkProtocol_t and a custom search time. It will cascade a small amount of work to apis like getAllConnectedDevices(). APIs that are specific to protocols use search-specific protocol search times -- unless overridden by custom param. With this, it allows USBIP-only searches to timeout much quicker.
// code uses hardcoded search time specific to protocol; from the test curves
getFirstAvailableDevice(XLinkProtocol_t proto = X_LINK_ANY_PROTOCOL);

// custom protocol, custom timeout
getFirstAvailableDevice(XLinkProtocol_t proto = X_LINK_ANY_PROTOCOL, chrono::milliseconds searchTimeout = WAIT_FOR_SEARCH_TIMEOUT);

getAllAvailableDevices(XLinkProtocol_t proto = X_LINK_ANY_PROTOCOL);
getAllAvailableDevices(XLinkProtocol_t proto = X_LINK_ANY_PROTOCOL, chrono::milliseconds searchTimeout = WAIT_FOR_SEARCH_TIMEOUT);

Makes sense on this approach - we'd cover all the cases with it.
We'll brainstorm a bit about it internally as well, and address.

Hi @themarpe, saw your recent commits like https://github.com/luxonis/depthai-core/blob/bd11775f2c747766900ed244f85f0ac295b807c9/src/device/DeviceBase.cpp
I don't see api/programmatic control; instead env vars. Are APIs forthcoming?
I ask because my app is distributed to customers such that it is not practical or reliable for them to dig into bat files or system control panels; some customers are in very tightly controlled computers with no admin access.
I sometimes can from the app set a envvar such that the app's process can see it, but it is sketchy on Windows to do that as there are different env var spaces (OS, c/c++ runtime, etc.) and the space in which depthai-core reads an envvar has to be in the same way I would write to that envvar...and...it has to read it after my app would write it. Meaning one-time reads by depthai-core have to be very late in the process so that my app has time to load DLLs, globals startup, etc...and then my app could (maybe) write to an envvar...and then depthai-core later reads it.

@diablodale
Those are mostly intended to cover existing apps that use broad API like getting any available device, etc... without having to recompile/change the code for it to target different set of devices or a specific device in mind. Debugging oriented mostly.
I'll add corresponding API to set those as well, specifically for DEPTHAI_SEARCH_TIME and DEPTHAI_PROTOCOL on various search functions, to have an easier way accessing those


Tensorflow Probability is breaking with Keras 3

Issue Type
Bug
Source
source
Keras Version
Keras 3.5.0
Custom Code
No
OS Platform and Distribution
Mac OS Sonoma 14.6.1
Python version
3.10.9
GPU model and memory
No response
Current Behavior?
vq_vae.py is getting failed while running with Keras 3 (in both cases - without any changes and with keras 3 backend agnostic changes) because of tensorflow_probability. It seems tensorflow_probability is not compatible with Keras 3.
AttributeError: module 'tensorflow._api.v2.compat.v2.__internal__' has no attribute 'register_load_context_function'. Did you mean: 'register_call_context_function'?
Standalone code to reproduce the issue or tutorial link
Simply running vq_vae.py with Keras 3 will produce this error.

Relevant log output
No response

Vector-Quantized Variational Autoencoders is not migrated to Keras 3 yet, if you're interested in contributing to the migration, please feel free to create a PR. Thanks!

Vector-Quantized Variational Autoencoders is not migrated to Keras 3 yet, if you're interested in contributing to the migration, please feel free to create a PR. Thanks!

Yes, I was working on that only but since it is throwing error for Tensorflow-probability, I raised this issue. I think Tensorflow-probability is not compatible with Keras 3 and it is required for this example. Please let me know if you have any workaround for this.


Not overwriting top-level child i.e. --append

Can we have an option to --append that doesn't overwrite the top-level child?

There's a --merge already...  Does that do what you need? :-)

Hi there Mike. No the --merge option overwrites the top-level child. Here is my firebase:

I would like to be able to import and append records under the top-level children i.e. events,geo,log,users

Can you give an example of the json you're trying to import and the command-line you're using?

{
"events": {
"-KKQQIUR7FAVxBOPOFhr": {
"dateAdded":<PHONE_NUMBER>926,
"owner": "62e6aaa0-a50c-4448-a381-f02efde2316d",
"type": "boycott"
}
},
"geo": {
"-KKQQIUR7FAVxBOPOFhr": {
".priority": "qw3yttz1k9",
"g": "qw3yttz1k9",
"l": [
41.5825024,
-73.7622713
]
}
},
"log": "null",
"users": {
"62e6aaa0-a50c-4448-a381-f02efde2316d" : {
"events" : {
"-KKQQIUR7FAVxBOPOFhr" : {
"type" : "boycott"
}
}
}
}
}

firebase-import --service_account my-service.json --database_url https://my.firebaseio.com --path / --json import.json --merge

I see...   Looks like you're trying to append to 3 different paths (/events, /geo, and /users) at once (not sure what the expected behavior of "log" is since it is null instead of having children to append).
Unfortunately that's not supported, sorry.  firebase-import lets you completely overwrite a location or with --merge, merge the top-level children in at that location.  You're trying to do a "merge" of the children of the top-level-children. :-)
You could do this with 3 separate firebase-import calls (one to append to "events", one to append to "geo", one to append to "users"), which should be pretty straightforward if this is just a one-time import (the primary use case for firebase-import).
If you're trying to use firebase-import for some sort of recurring data importing, you're probably better off using the REST API or one of the client SDKs.  For info on doing atomic writes to multiple locations across your Firebase, see here.
FWIW, I don't think an --append option would be the right approach.  --merge already dose a merge one-level deep (i.e. it would overwrite "/events", "/geo", and "/users" but wouldn't overwrite any other existing top-level children, e.g. "/users").  It looks like you want to do a merge 2 levels deep.  So we'd need an arbitrary "--merge " option or something.

Grandkids eh? What can you do.... :/
I tried 3 separate firebase-import calls but the result was the same, which wasnt surprising, because I just split the file into 3 parts.
This would be a one-time import (mostly) use case. I will have a look at trying to merge the exported json file with another that has the data to be imported and then import the bigger file but json is a bugger for that kind of thing.
Can you please add export to, and import from, csv to your already huge workload? That would be a hugely helpful feature.
Thanks a lot for the fast help. You can close this issue :+1:

To do it with 3 separate imports you'd need to e.g. create an events.json with the events moved to the top-level, e.g. { "-KKQQIUR7FAVXBOPOFhr": { ... }, ... }
And then do an import with path /events, e.g.: firebase-import --service_account my-service.json --database_url https://my.firebaseio.com/ --path /events --json events.json --merge
I'll keep CSV in mind.  Pull Requests are always welcome as well. :-)

oic. This now works just fine. Thanks again Mike.


User name and URL

The user name is in the URL (on the official server at least). It create a privacy flaw. Remember that HTTP referer inform where you come from (if you let it enable). Users can be identified because of it.
In all cases, we have to specify our user name in the login form. So the question is, is it needed to have the user name in the URL ?

Are you talking about the url of the hosted account like this one: https://myusername.miniflux.net/?
Yes It's required to have the username in the url to map your account to the right data folder and database. Your login doesn't need to match your account name.

Since the hosted version is using HTTPS, the referrer is not sent to other websites
There is also a CSP rule to disable the referrer
There is a meta tag to disable the referrer (only supported by some browsers)
External links contains the attribute rel="noreferrer" (only supported by some browsers)
Images can be proxified by Miniflux itself

Great! I wasn't aware of this possibilities.


Change output file format to allow easy modification

Make format produced by LogFilters::_filters_to_string easy to read and alter within preferred editor.

Initially resolved within #f5d2f79


Remote bookmarks on mobile version of Lagrange

Is "1.4.2 Remote bookmarks" function is working in mobile version of Lagrange (iOS)? I've seen "Use as Bookmark Source" option at "Edit bookmark" dialog. But I don't know where is "Refresh Remote Sources" menu item in mobile version. New items seem to be unsynchronised.

Yeah it should be working, although you need to trigger refreshes manually. The menu item can be found in the bookmarks list context menu (long-press on any bookmark).


Marker Cluster

Any chance for an implementation of markercluster?

I would need it too! Maybe we could just build a new component of svelte-leaflet based on leaflet.markercluster ?
Actually I'm using leaflet + markercluster as action in svelte-kit. The only issue is that I can't make the marker reactive for now…

@Hello yoh-extradat @eur2 ,
Did you successfully integrate leaflet.markercluster? Would you be willing to share a piece of code? From what I understand, to make one MarkerCluster the same way as Marker, we would need the export L (to retrieve the map context) and the EventBridge but nont of those are exported at the moment.
Am I correct? Is this possible to update the library to expose those?


Bundle Install Error on Windows

Error Report
Questions
Please fill out answers to these questions, it'll help us figure out
why things are going wrong.

What did you do?
I ran the command C:/tools/ruby23/bin/bundle update

What did you expect to happen?
I expected Bundler to update my project gems

What happened instead?
Instead, what happened was this error

Have you tried any solutions posted on similar issues in our issue tracker, stack overflow, or google?
I tried updating bundler, and deleting the files in my temp directory. Did not work.

Have you read our issues document, https://github.com/bundler/bundler/blob/master/doc/contributing/ISSUES.md?
...

Backtrace
Errno::ENOTEMPTY: Directory not empty @ dir_s_rmdir - C:/Users/ANKIT/AppData/Local/Temp/bundler-compact-index-20170309-6212-1ury8z2
  C:/tools/ruby23/lib/ruby/2.3.0/fileutils.rb:1445:in `rmdir'
  C:/tools/ruby23/lib/ruby/2.3.0/fileutils.rb:1445:in `block in remove_dir1'
  C:/tools/ruby23/lib/ruby/2.3.0/fileutils.rb:1459:in `platform_support'
  C:/tools/ruby23/lib/ruby/2.3.0/fileutils.rb:1444:in `remove_dir1'
  C:/tools/ruby23/lib/ruby/2.3.0/fileutils.rb:1437:in `remove'
  C:/tools/ruby23/lib/ruby/2.3.0/fileutils.rb:779:in `block in remove_entry'
  C:/tools/ruby23/lib/ruby/2.3.0/fileutils.rb:1494:in `ensure in postorder_traverse'
  C:/tools/ruby23/lib/ruby/2.3.0/fileutils.rb:1494:in `postorder_traverse'
  C:/tools/ruby23/lib/ruby/2.3.0/fileutils.rb:777:in `remove_entry'
  C:/tools/ruby23/lib/ruby/2.3.0/tmpdir.rb:95:in `ensure in mktmpdir'
  C:/tools/ruby23/lib/ruby/2.3.0/tmpdir.rb:95:in `mktmpdir'
  C:/tools/ruby23/lib/ruby/gems/2.3.0/gems/bundler-1.14.6/lib/bundler/compact_index_client/updater.rb:30:in `update'
  C:/tools/ruby23/lib/ruby/gems/2.3.0/gems/bundler-1.14.6/lib/bundler/compact_index_client.rb:81:in `update'
  C:/tools/ruby23/lib/ruby/gems/2.3.0/gems/bundler-1.14.6/lib/bundler/compact_index_client.rb:97:in `update_info'
  C:/tools/ruby23/lib/ruby/gems/2.3.0/gems/bundler-1.14.6/lib/bundler/compact_index_client.rb:54:in `block in dependencies'
  C:/tools/ruby23/lib/ruby/gems/2.3.0/gems/bundler-1.14.6/lib/bundler/fetcher/compact_index.rb:87:in `block (3 levels) in compact_index_client'
  C:/tools/ruby23/lib/ruby/gems/2.3.0/gems/bundler-1.14.6/lib/bundler/worker.rb:63:in `apply_func'
  C:/tools/ruby23/lib/ruby/gems/2.3.0/gems/bundler-1.14.6/lib/bundler/worker.rb:58:in `block in process_queue'
  C:/tools/ruby23/lib/ruby/gems/2.3.0/gems/bundler-1.14.6/lib/bundler/worker.rb:55:in `loop'
  C:/tools/ruby23/lib/ruby/gems/2.3.0/gems/bundler-1.14.6/lib/bundler/worker.rb:55:in `process_queue'
  C:/tools/ruby23/lib/ruby/gems/2.3.0/gems/bundler-1.14.6/lib/bundler/worker.rb:88:in `block (2 levels) in create_threads'

Environment
Bundler   1.14.6
Rubygems  2.6.10
Ruby      2.3.1p112 (2016-04-26 revision 54768) [x64-mingw32]
GEM_HOME  C:/tools/ruby23/lib/ruby/gems/2.3.0
GEM_PATH  C:/tools/ruby23/lib/ruby/gems/2.3.0;C:/Users/ANKIT/.gem/ruby/2.3.0
Git       2.11.0.windows.1
Platform  x64-mingw32
OpenSSL   OpenSSL 1.0.1l 15 Jan 2015

Thanks for the bug report, If you manually remove the folder at C:/Users/ANKIT/AppData/Local/Temp/bundler-compact-index-20170309-6212-1ury8z2 that should fix your problem.

Hi Colby,
I tried that a few times but it still does not work for me. The error is see when I run bundle install is pasted below.
Fetching gem metadata from https://rubygems.org/....
Retrying fetcher due to error (2/4): Errno::ENOTEMPTY Directory not empty @ dir_s_rmdir - C:/Users/ANKIT/AppData/Local/Temp/bundler-compact-index-20170309-14540-1vtukfv........
Retrying fetcher due to error (3/4): Errno::ENOTEMPTY Directory not empty @ dir_s_rmdir - C:/Users/ANKIT/AppData/Local/Temp/bundler-compact-index-20170309-14540-1kw6msa..
Fetching version metadata from https://rubygems.org/.
Fetching version metadata from https://rubygems.org/.
Could not find gem 'minima (~> 2.0) x64-mingw32' in any of the gem sources
listed in your Gemfile.

Hi Colby,
After restarting my computer, and cleaning up my temp directory., it seems to have worked. Thanks for the response and the help.


Technoly/Skills/Tools

Icon Request

Icon name: html, css, javascript, github, php, react, etc.

Tags of this icon:  tools/technologies/skills

Use case:

Screenshots of similar icons:

There is need for technology tools icons to make the icons set complete.


Next release?

Is there any new release planned? Now it has gone 28 days since last release. I'm waiting for a fix that was merged 16 days ago.

Hi @LennartHaggkvistHM, there will be a new release this week. Thank you.

It was released today, so closing this.


RSpec/SortMetadata autocorrect broke code for it_behaves_like

Create sample rspec file like this:

# frozen_string_literal: true

RSpec.shared_examples 'some example' do |parameter1, parameter2|
  it 'uses the given parameter' do
    p "par1: #{parameter1}"
    p "par2: #{parameter2}"
    expect('a').to eq('a')
  end
end

describe 'enterprise' do
  it_behaves_like 'some example', 'b', 'a' do
    let(:obj) { 'b' }
  end
end

run it and get output:

"par1: b"
"par2: a"
.

Finished in 0.00213 seconds (files took 0.08339 seconds to load)
1 example, 0 failures

Run autocorrect for this: rubocop -A spec/test_spec.rb
Result will be:

# frozen_string_literal: true

RSpec.shared_examples 'some example' do |parameter1, parameter2|
  it 'uses the given parameter' do
    p "par1: #{parameter1}"
    p "par2: #{parameter2}"
    expect('a').to eq('a')
  end
end

describe 'enterprise' do
  it_behaves_like 'some example', 'a', 'b' do
    let(:obj) { 'b' }
  end
end

Run it again:

"par1: a"
"par2: b"
.

Finished in 0.00207 seconds (files took 0.08415 seconds to load)
1 example, 0 failures

So order of arguments is change and this may broke codebase
Env:
1.37.0 (using Parser <IP_ADDRESS>, rubocop-ast 1.23.0, running on ruby 3.1.2) [x86_64-linux]
  - rubocop-performance 1.15.0
  - rubocop-rake 0.6.0
  - rubocop-rspec 2.14.1

Thanks for reporting!
I believe the fix should be:
- #rspec? {#Examples.all #ExampleGroups.all #SharedGroups.all #Hooks.all #Includes.all}
+ #rspec? {#Examples.all #ExampleGroups.all #SharedGroups.all #Hooks.all}

and a spec making sure we don't mess with positional arguments order of include_examples/include_context/it_behaves_like with and without blocks.
A short reference of how RSpec deals with metadata.
Would you like to send a PR, @ShockwaveNN ?

Would you like to send a PR, @ShockwaveNN ?

Sorry, I think I'll pass. I'm happy to report issues, but not sure that I've enough free time to dig in the code right now

No worries, I'll take care of it.
Meanwhile, stuck with a peculiar bug where the removal of #Includes.all is causing RSpec DSL config not to load in specs 🤯


OAuth with outlook failed with "server_error"

Checklist

[X] I have used the search function to see if someone else has already submitted the same bug report.
[X] I will describe the problem with as much detail as possible.

App version
6.201
Where did you get the app from?
F-Droid
Android version
12, Graphene OS
Device model
Pixel 5
Steps to reproduce

Try login with any @outlook.com account
Redirect to the webpage in an external browser (Bromite), enter login credentials
Being redirected to the app

Expected behavior
Logged into the account
Actual behavior
"Authorization failed with the following error: server_error"
Logs
37453F24055879E10A73E9E65D33BDC4AC61EDDD324508902C7F37725B2F1AC6.txt

This doesn't sound like a bug in K-9 Mail. The server is returning an error. Please contact the outlook.com support to find out why that is.


Azure Semantic Search query_type='semantic' not found

Package Name:  azure-search-documents
Package Version:  11.3.0
Operating System: Windows
Python Version: 3.9.13

Describe the bug
I am trying to use the 'semantic' capabilities of azure cognitive search through the Python SDK. I have followed all the file changes made here #17690 and tested the code exactly as mentioned in the sample_semantic_search.py file. But the following is the error I receive
Message: The request is invalid. Details: Requested value 'semantic' was not found

As per the documentation I executed the code client.search(search_text="luxury", query_type="semantic", query_language="en-us")). For some reason it is not able to identify the keyword 'semantic'. I looked into the python source code files and found out that under the Enum Class QueryType there are only two values: 'simple' and 'full' but there is not 'semantic'
To Reproduce
Steps to reproduce the behavior:

Follow the sample_semantic_search.py file as shown here sdk/search/azure-search-documents/samples/sample_semantic_search.py in the github repository
Run the code to reproduce the same issue

Expected behavior
I expected it to fetch a list descriptions that were semantically similar to my query.
Additional context
I have already gone through all the documentation and finished the necessary setup in my azure portal. On another note I faced this similar issue with the semanticConfiguration parameter as well

Hi @kashifkhan I ran the code you have provided and saw this error
cannot import name 'SemanticConfiguration' from 'azure.search.documents.indexes.models'
So I checked this file which has the SemanticConfiguration class imported
https://github.com/Azure/azure-sdk-for-python/blob/main/sdk/search/azure-search-documents/azure/search/documents/indexes/models/__init__.py
However in the version I have 11.3.0 I do not have this class in the same file.
I did do pip install azure-search-documents --upgrade and seems like 11.3.0 is the latest version however seems like the main branch is different from what is being fetched through pip
I could potentially copy all the code from the latest branch and do it but this would be a workaround. Please let me know the next steps. Thanks for the help in advance!

Sorry @Anirudh42 . This feature is not GAed on the service side and we cannot make a stable library with this feature ready.
For now, the only way to use it is to install a preview version of azure-search-documents.
You can install it by pip install --pre.
Thanks.

Ok understood @xiangyan99 . I will install the preview version and use it. I will close this issue for now. Thanks for the assistance. May I know what is the timeline for the release for this new feature?

We know it has been in preview for a while. Unfortunately, there is no precise ETA yet. But the service team is currently working on it.

Ok thanks for letting me know 🙂

I installed the preview version
Name: azure-search-documents
Version: 11.4.0b3
Still getting the same error:
ImportError: cannot import name 'SemanticConfiguration' from 'azure.search.documents.indexes.models' (/anaconda/envs/azureml_py38/lib/python3.8/site-packages/azure/search/documents/indexes/models/init.py)

/unresolve


update contributing guide date format

The format requested was used for 2016 and 2017 data, but from then on the
format changed. Probably easier to re-standardise on recent data than to
coerce recent data to fit.
ref https://github.com/Readify/DevEvents/issues/69

@swalladge oops yes good catch


add some more info about signing commits and verification and the setup

Describe your changes
Add some more info to the template about how to set up the commit signing for successful verification.

PTAL @jihoon-seo :)


Unable to Sign in to Author App

After installing the Author App into a Team I am unable to sign-in. App acts like its loading then shows a Sign In button. Clicking the button opens up a separate window to complete the sign-in process but then just pops back to the sign in button. I checked to make sure my UPN was in the SenderUPNList and can confirm thats correct. I can see my sign-ins to the app in Azure AD were successful.

@cueball59 Hi, do you have an alias? this might be different from your email address, can you try adding that in the UPN list & try accessing the author application?


enable DPLPMTUD on macOS dual-stack sockets

My new draft in the TSV WG (https://datatracker.ietf.org/doc/draft-seemann-tsvwg-udp-fragmentation/) contains details on how IP fragmentation is handled on different platforms.

I updated the PR to test if a socket is a dual-stack socket on older (before Sequoia) macOS versions. If it's a dual-stack socket, DPLPMTUD will be disabled.


Make vector template-polymorphic again

After coq/coq#9918, vector was no-longer template polymorphic.  This
backwards-compatible commit fixes that.
Note that this is required for mirror-core to build with Coq v8.10 without universe errors.

Looks good to me. Questions that seem to stem from this are:
1/ does it need to be done anywhere else?
2/ there was talk at some point to eliminate template universe polymorphism. should we just convert this into a true universe polymorphic definition?

Not for mirror core to build, but if you have an template-polymorphic inductive in a section anywhere, then this needs to be done for it to retain its template polymorphic status.
I think ideally all inductives will eventually be universe polymorphic and cumulative, unless you want them not to be for performance reasons.  But not all the compatibility is there yet.  I didn't bother to check whether or not making this universe-polymorphic fixed the issue in mirror-core (it might), but, yeah, making is universe polymorphic is probably better in the long run.

It seems like this might also be an issue with OptionMonad, see https://sympa.inria.fr/sympa/arc/coq-club/2019-12/msg00011.html

I'm going to go ahead and merge this.


Adding in translation for Work show page for identifiers

refs #1176
Original view:

New view:

Coverage decreased (-55.05%) to 0.0% when pulling 4bf7be90457e48f083b339a2efdee9192abd5852 on i#1176 into 86ecdd21885000166b2088ad01864f42ff44d119 on develop.

@awead I updated the comments.


Create module to make for easier use of PSC

This adds a LICENSE along with a short example
I tested this in our internal test environment and it successfully created the PSC
Test Plan
Ensure that domain validation works
TODO

[ ] Make proxy network configurable

another note:
let's remove all the 22 and 80 ports from firewall rules. they were there only for testing purposes, and they shouldn't be used in prod.

@michaellzc What about ssh cloning? We might want to leave port 22 open for that

@michaellzc What about ssh cloning? We might want to leave port 22 open for that

I doubt anyone with GitHub will want to use ssh key for git clone.
Additionally, Cloud does not support configuring customer's SSH key today (it needs to be mounted from a CM or Secret to gitserver, and you can't do so from siteconfig)


Pixelbook screen refresh changes when it gets overheated

After spending a lot of time trying to figure out what is happening with this Pixelbook, it seems that the culprit is this; after a couple hours of use, the screen refresh rate gets wonky and looks like a set of lines.
I just confirmed that this is the case by holding the laptop over an air conditioner, which fixed the problem. I'm going to check on Pixelbook overheating problems.

So, this seems to be ameliorated by switching to Chrome OS dev channel (version 70). This has been running with no visual failures since yesterday. I think I'll be able to close this soon.


add: tests for firestore adapter

Ported the Prisma tests.
Relates to #18

@AlexMeuer thanks a lot for getting this done! One problem though, did you have ts-mock-firebase installed locally? I wasn't included in the package.json, but is required by some of your tests.
I tried to install myself, but unfortunately it won't compile on my linux install, its trying to compile from source this grpc library from python I think, so theres all sorts of node-gyp errors. There hasn't been a release in 2 years for ts-mock-firebase unfortunately so I assume some versions its requiring are too old / incompatible with current python / node-gyp / whatever

Hmm, I must have forgotten to save the dependency. I'll have a look for an alternative when I've time.

Spent some time trying to replace ts-mock-firebase with firebase-mock but it lacks support for serverTimestamp as well as screwing with async/await. Might just have to manually mock the used methods, but it doesn't look like I'll have more time this week.

Hey so first of all, thanks for giving it a shot and reporting back on any and all stumbling blocks!
I'll take a shot at specifically mocking the used methods then. Dom have you pushed your latest state to this branch?

Thank you for this, but I think https://github.com/nextauthjs/adapters/pull/90 will address most of this, and also some of the stuff here has been made redundant by improving our core package. It takes care of error handling instead of the adapters.


update resin-settings-storage and add the browser tests

Looks great! Do you need me to merge here as well? Looks like there are tons of modules that weren't moved to resin-io-modules at all :/

Yeah, I can't merge on resin-io repos that I'm not added to directly.
Please merge and publish or move and give me the permissions (I could build and publish myself given the needed permissions and npm creds).  I've added you to all repositories, so you can merge and publish yourself and don't have to depend on me. Let me know if you encounter any repo where you still don't have permissions in (I've added you to the cli/sdk team, which should be including all repos).

Thanks! I can merge here now.
Would you mind sharing the required npm creds in passpack so that I can publish as well?

I sadly don't seem to have enough permissions to share the credentials
(just view them), so I guess reaching out to Apostolis/Aggelos would be
the way to go. Alternative, I can publish now so you don't get blocked
by it.
Let me know!
On Fri, Sep 09, 2016 at 02:08:32PM -0700, Eugene Mirotin wrote:

Thanks! I can merge here now.
Would you mind sharing the required npm creds in passpack so that I can publish as well?
--
You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
https://github.com/resin-io/resin-token/pull/25#issuecomment-246040411

--
Juan Cruz Viotti
Software Engineer

OK please publish now then.

All good @jviotti I happened to have access, published v2.4.3 to npm.
Hope all looks good


Emojis are slow and work not well

The emoji selection (and scrolling in the area, where you select your emojis)  works not well and is slow.

IMHO only a problem on Retina Screens


Fixed link to PasswordHasher because it was missing Core.

Hi @whytheday, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, DNFBOT;

Link should point to master, not dev.

https://github.com/aspnet/Identity/blob/master/src/Microsoft.AspNet.Identity/PasswordHasher.cs

:shipit:

Thanks! 70b2a362d9abbf0f3405ce94d0f859dd23fa7e16


sol-connman: Add sol-connman basic API

so-connman is for implementing connection manager features.
The APIs should be implemented in Zephyr, connman and so on.
The patches are for implementing sol-connman basic APIs to connman.
the connman D-Bus APIs are wrapped in the so-connman APIs.
The user can use the so-connman APIs to control connman.
Signed-off-by: Wu Zheng<EMAIL_ADDRESS>

According to the comments of https://github.com/solettaproject/soletta/pull/1848,
the related issues have been fixed.
It is the version 2 of sol-connman.

The two new monitors have been added to monitor error and the system state.
Remove sol_connman_service_get_call_result.
The functions of get services vector and network link have been modified.
the vector of service has been changed to ptr vector.
streq and the other soletta functions have been used to replace strcmp and so on.
no use malloc  for network address.
add new monitor to test cases.
others.

Please review it again. Thanks.

we're almost there! just very minor issues.
Also, remember to rename sol-connman.h to sol-netctl.h and all namespace to sol_netctl_.

The patches have been closed.
The new version sol-connman patches have been submitted.
The link is #1905