Add privacy policy

Related Issue: closes #242

Summary:
The final step for our analytics, this diff:

Creates a Privacy Policy README
Links to it from within the app, through the ApplicationMenu

Children’s information

We might have to follow COPPA which requires any data from users discovered to be < 13 to be removed if we find out that they’re under 13.

I'd say we also need to link to it on the first screen of the onboarding to warn users that we track some stuff (and possibly add a way to opt-out)

Good idea, I'll add a link to the "intro" screen.
For opt-out, I agree it'd be a good thing to add, but I don't think it's a very high priority. I don't think the kind of tracking we do would bother most folks, since it's so minimal and non-invasive. And I'm wary of adding additional complexity unless we hear from users that the tracking bothers them / stops them from using it.

But I think we should add some stuff from a pre-defined policy to have more legal aspects e.g. from Policy generator.

I'm not too concerned about the legal aspect (at least, not at our current size), given that it's a product that collects zero personally-identifiable information and has zero revenue. But I do think the points you raise are worth including regardless. I'll update

We might have to follow COPPA which requires any data from users discovered to be < 13 to be removed if we find out that they’re under 13.

I don't think this applies for non-personally-identifiable-information. At Khan Academy we take great efforts to be COPPA-compliant for registered users under 13, since we actually store some PII about them (username, parent's email, etc). But unregistered users can still use the product, and we still have our standard analytics event. I'll add instructions for how to submit requests to delete our data though

@AWolf81 I did most of the requested edits, except:

I feel funny explicitly saying something like "By running Guppy, you consent to agree to these terms". Feels so sketchy, and I'm not really worried about our legal culpability at this point (maybe when we're bigger, I'll take it more seriously, but I can't imagine getting a serious fine for such a small, profit-free project)

I didn't link to Mixpanel's PP since their privacy policy explicitly says it doesn't pertain for our users:

This privacy statement is not intended to discuss the collection or use of personal information by a Customer’s use of the Services and does not apply to third-party websites, applications, or services that display or link to different privacy statements.  Because a Customer’s collection or use of personal information is specific to their use and implementation of the Services, please consult a Customer’s privacy statement or the written contract between a Customer and Mixpanel for the Services for more information.  For additional detail, please see the Analytics Platform Activities section below.

Source: https://mixpanel.com/legal/privacy-policy/


plugins/otter: init

https://github.com/jmbuhr/otter.nvim
Unfortunately, cannot run tests for this. After 33 minutes, it exits with SIGSTOP with no error message (both times I tried)

If you're able to rebase, we can do a fast-forward merge with your commit rather than our rebase.

It doesn't really matter

Just noticed that I forgot to add otter as cmp source, will this work?
config = lib.mkIf cfg.enable {
  plugins.cmp = {
    settings.sources = [ { name = "otter"; } ];
  };
}

Just noticed that I forgot to add otter as cmp source, will this work?

We normally do it the other way around, by listing the source in this file. This makes the plugins.cmp.autoEnableSources aware of the plugin.
Is otter.nvim more than just a cmp source? Otherwise we'd probably define it in https://github.com/nix-community/nixvim/tree/main/plugins/completion/cmp/sources instead of directly in plugins.
I'm not 100% familiar with how we do nvim-cmp, so probably best if @GaetanLepage chimes in here.

We normally do it the other way around, by listing the source in this file. This makes the plugins.cmp.autoEnableSources aware of the plugin.
Is otter.nvim more than just a cmp source? Otherwise we'd probably define it in https://github.com/nix-community/nixvim/tree/main/plugins/completion/cmp/sources instead of directly in plugins.
I'm not 100% familiar with how we do nvim-cmp, so probably best if @GaetanLepage chimes in here.

Yes, no matter what, just add it to the source-plugin attrs.

error: The option plugins.otter.enable' in plugins/languages/otter.nix' is already declared in `plugins/completion/cmp/sources'.

I guess that explains why the other cmp sources in https://github.com/nix-community/nixvim/tree/main/plugins/completion/cmp/sources?rgh-link-date=2024-06-27T14%3A57%3A02Z don't directly use mkNeovimPlugin and instead manually use helpers.mkSettingsOption.
I'm afraid you'll have to take a look over those files and re-work your plugin using the same approach. Also move from plugins/languages/ to plugins/completion/cmp/sources and check how we normally handle tests for cmp sources.

Had to rename the plugin to otter-nvim, because plugins/completion/cmp/sources.nix uses later that name for pkgs.vimPlugin.otter-nvim.
Here is the proper link for the force push: https://github.com/nix-community/nixvim/compare/7dfadb6421cbce87c6ecd3196947192efa888e70..b059fa8d6eebbc12273e23451448610b4d695a4d
I also tested the plugin on my nixos install and it works

Had to rename the plugin to otter-nvim, because plugins/completion/cmp/sources.nix uses later that name for pkgs.vimPlugin.otter-nvim.

The documentation might be a bit misleading (referring to plugins), it actually means the plugin's nixvim module name:
https://github.com/nix-community/nixvim/blob/35e837c10b767c3dbf580f3cade5084f376f7c70/plugins/completion/cmp/cmp-helpers.nix#L48-L56

It's ok if autoEnableSources doesn't work for otter in this PR, so I think we should just merge 7dfadb6421cbce87c6ecd3196947192efa888e70 as is rather than attempting to integrate with the cmp systems.
Users can still manually enable plugins.otter.enable and add "otter" to nvim-cmp sources.
I'll look into what refactoring can be done to make our cmp implementation more flexible, but that shouldn't block this PR.

Apologies again for suggesting you rewrite your perfectly good PR as a cmp-plugin.

It's okay, I was happy to work on this code.
I also added an option (plugins.otter.addCmpSources) to automatically do
plugins.cmp = {
  settings.sources = [ { name = "otter"; } ];
};

and it is enabled by default.

I also noticed that if you define a plugin using plugins/completion/cmp/sources.nix, it doesn't define package option
Compare https://nix-community.github.io/nixvim/plugins/cmp-async-path.html and https://nix-community.github.io/nixvim/plugins/easyescape.html

Just again noticed that I overlooked that otter has dependencies. How are they should be specified in NixVim?
https://github.com/jmbuhr/otter.nvim?tab=readme-ov-file#dependencies

Just again noticed that I overlooked that otter has dependencies. How are they should be specified in NixVim?

They should be dealt with in nixpkgs.

https://github.com/NixOS/nixpkgs/pull/323107

Thanks!

I trust you on this one, merge it if you think that it's ready :)


bug: prisma type error

Provide environment information
System:
OS: macOS 13.3
CPU: (8) arm64 Apple M2
Memory: 55.59 MB / 8.00 GB
Shell: 5.9 - /bin/zsh
Binaries:
Node: 18.15.0 - ~/Library/Caches/fnm_multishells/99917_1680623854545/bin/node
npm: 9.5.0 - ~/Library/Caches/fnm_multishells/99917_1680623854545/bin/npm
Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
Describe the bug
'prisma' implicitly has type 'any' because it does not have a type annotation and is referenced directly or indirectly in its own initializer.ts(7022)
Reproduction repo
https://github.com/t3-oss/create-t3-app
To reproduce
I dunno just trying to get started here
Additional information
No response

Have you generated the types?
pnpm prisma generate
Or just restart the TypeScript server


[WIP] Survival improvments

New version off #3

should we merge this @IceTank ?

should we merge this @IceTank ?

There is still a lot off testing code in this I think. I definitely want to work on this some more in the future tho. I will open it when I feel like it is more polished.


[REQUEST] Emphasize the compatible Python versions - Good First Issue

How would it improve Rich?

Update the demo pictures to the latest Python version terminal.
Emphasize more in the 'Compatibility' section that the range of compatible python versions is 'Python 3.6.1 and above'.

What problem does it solve for you?
It clarifies and makes it easy to see what compatible python versions are sufficient.
This simple update could have helped me, might help others, and might not - but why not try :)
Happy Coding 👍

How would you emphasize it?

Didn't see that, hope that others will see it atleast...
Have a good day 👍 :)

@willmcgugan just added it upon your request, no wonder you didn't see it prior to opening an issue. Anyway, thanks Will, keep it up :)


Script for cleaning checkout_tokens of cancelled orders.

I do not want to merge this change.
Review appreciated but it's a single script we need to run once on particular instance, will be closed after it's used.
Script filters all orders with status canceled and set checkout_token to "" (empty string).

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

closing, script was run.


Fixes a bug when failing assertions on DirectoryStream types

The DirectoryStream class is somewhat special because the API allows it to only support .iterator() being called at most once on any implementations. Due to the nature of how the StandardRepresentation currently works for iterable types, any assertion error that is thrown as a result of assertions failing on a DirectoryStream object can result in IllegalStateExceptions being propagated to the caller if .iterator() has already been called on the class. This can break further if using soft assertions, as the state of the class could change unexpectedly resulting in different exceptions being raised depending on the execution order.
I've added a mechanism to deal with allowing the definition of so-called 'blacklisted' iterable types, just in case other cases like this appear in the JRE standard library anywhere in the future. This will prevent unexpected errors occuring when dealing with the NIO FileSystem APIs.
This was discovered as a result of https://github.com/ascopes/java-compiler-testing/pull/450, where @marschall pointed out that the #iterator() on a DirectoryStream should only be called once in some implementations.

Thanks for the analysis and the PR @ascopes

Integrated thanks @ascopes !


SBG 3D printed mount

Hello, please refer to my fantastic drawing concept. The drawings of the SBG can be found here: https://www.sbg-systems.com/setup/, we are using the Ellipse-N.

Here please

It's upside down.

This was sent off to printing on the weekend


guard random functions for half

#606

It's a little nicer if you keep the order as is (more ifdef guards) imo; this makes it more likely that someone adding a new random function would miss adding the half functions.


chore: Refactored a few workspace related methods for reusability

Description
Only a refactoring task, does not require testing.
Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/ok-to-test

/ok-to-test

Added test plan approved label since the change is only refactoring related.

/ci-merge-check

/ci-merge-check


Chore/cleanup feature flags

drop unused feature flags and be more consistent about the other ones. we should probably also consider:

dropping the cut and copy feature flags
moving extended html vs  html vs text paste to a config of the editor and not a feature flag

In tedi, cut, copy and paste can be implemented as plugins, which can then be disabled by not adding them
we could then export 2 components, one "fully featured" which includes all the "default" and one barebones, where every plugin has to be specified explicitly

while we're at it, could you elaborate a bit what the "extended" means in "extended-html-paste"? Even I don't know that :stuck_out_tongue:


Insert error when field name was a keyword

Insert error when field name was a keyword.
Which version of ShardingSphere did you use?
4.0.0-RC1
Which project did you use? Sharding-JDBC or Sharding-Proxy?
Sharding-JDBC
Expected behavior
insert success.
Actual behavior
[main] INFO ShardingSphere-SQL - Logic SQL: INSERT INTO `ts_order` (`id`, `desc`) VALUES ('1', 'abc')
[main] INFO ShardingSphere-SQL - SQLStatement: InsertStatement(super=DMLStatement(super=AbstractSQLStatement(type=DML, tables=Tables(tables=[Table(name=ts_order, alias=Optional.absent())]), routeConditions=Conditions(orCondition=OrCondition(andConditions=[AndCondition(conditions=[Condition(column=Column(name=id, tableName=ts_order), operator=EQUAL, compareOperator=null, positionValueMap={0=1}, positionIndexMap={})])])), encryptConditions=Conditions(orCondition=OrCondition(andConditions=[])), sqlTokens=[TableToken(tableName=ts_order, quoteCharacter=BACK_QUOTE, schemaNameLength=0), SQLToken(startIndex=23)], parametersIndex=0, logicSQL=INSERT INTO `ts_order` (`id`, `desc`) VALUES ('1', 'abc')), deleteStatement=false, updateTableAlias={}, updateColumnValues={}, whereStartIndex=0, whereStopIndex=0, whereParameterStartIndex=0, whereParameterEndIndex=0), columnNames=[id, desc], values=[InsertValue(columnValues=[org.apache.shardingsphere.core.parse.old.parser.expression.SQLTextExpression@5a62b2a4, org.apache.shardingsphere.core.parse.old.parser.expression.SQLTextExpression@1051817b])])
[main] INFO ShardingSphere-SQL - Actual SQL: ds_0 ::: INSERT INTO `ts_order_0001`  (id, desc) VALUES ('1', 'abc')
Exception in thread "main" com.mysql.jdbc.exceptions.jdbc4.MySQLSyntaxErrorException: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'desc) VALUES ('1', 'abc')' at line 1

Reason analyze (If you can)
Steps to reproduce the behavior, such as: SQL to execute, sharding rule configuration, when exception occur etc.
Example codes for reproduce this issue (such as a github link).

DESC is Reserved Words which cannot use to be a column name.
FYI: https://dev.mysql.com/doc/refman/8.0/en/keywords.html

But you use back quote to mark desc, that is correct.
Let us investigate it.

When I use select statement, the result is correct.
[main] INFO ShardingSphere-SQL - Logic SQL: select id, `desc` from ts_order
[main] INFO ShardingSphere-SQL - SQLStatement: SelectStatement(super=DQLStatement(super=AbstractSQLStatement(type=DQL, tables=Tables(tables=[Table(name=ts_order, alias=Optional.absent())]), routeConditions=Conditions(orCondition=OrCondition(andConditions=[])), encryptConditions=Conditions(orCondition=OrCondition(andConditions=[])), sqlTokens=[TableToken(tableName=ts_order, quoteCharacter=NONE, schemaNameLength=0)], parametersIndex=0, logicSQL=select id, `desc` from ts_order)), containStar=false, firstSelectItemStartIndex=7, selectListStopIndex=16, groupByLastIndex=0, items=[CommonSelectItem(expression=id, alias=Optional.absent()), CommonSelectItem(expression=desc, alias=Optional.absent())], groupByItems=[], orderByItems=[], limit=null, subqueryStatement=null, subqueryStatements=[], subqueryConditions=[])
[main] INFO ShardingSphere-SQL - Actual SQL: ds_0 ::: select id, `desc` from ts_order_0000
[main] INFO ShardingSphere-SQL - Actual SQL: ds_0 ::: select id, `desc` from ts_order_0001

the insert column name was wrong, the correct column name is desc not "desc"

Hi, this bug has been fixed #2387, thanks again.


python312Packages.pysrt: drop nose dependency

Description of changes

#326513

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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/)
24.11 Release Notes (or backporting 23.11 and 24.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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Please migrate to pyproject = true;, and it looks like the licence should be gpl3Only. LGTM otherwise.

Hopefully it's good now

Almost forget the license change

Result of nixpkgs-review pr 330441 run on x86_64-linux 1

4 packages marked as broken and skipped:

python311Packages.subliminal
python311Packages.subliminal.dist
python312Packages.subliminal
python312Packages.subliminal.dist

4 packages built:

python311Packages.pysrt
python311Packages.pysrt.dist
python312Packages.pysrt
python312Packages.pysrt.dist


fix: tag extracting

:tada: This PR is included in version 1.0.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Integrated package

Hi,
Your work is excellent. I am seeking acceleration pipeline of minimap2 by GPU. I noticed your code was not integrated into minimap2 software. Could u pls provide an integrated GPU-minimap2 version?
Best,
Yuanda

Thanks for your interest in our work. However, we don't have integration at this point and we don't have plan to implement it for now. You are welcome to develop it and send us a pull request :-).


[0009] Does strict math affect the accuracy of operations?

Which proposal does this relate to?
Math Modes
Describe the issue or outstanding question.
Do the accuracy of operations differ between strict and fast modes? For example does strict::fma have lower error (in terms of ULPs) than fast::fma?
Additional context
My understanding of the previous precise keyword was that (at least in some cases) higher accuracy operations would be used. Further, I thought that the floating point rules only applied when precise was specified. Some of the builtins linked in other documentation describe an error bound, but I don't think they apply without the precise modifier.
CC @gnl21, @s-perron

Transforming a * b + c into fma(a, b, c) should only be valid if the error on fma is at least as good as the individual operations. Different APIs approach this differently. For example, Vulkan states they have the same accuracy so you might accumulate rounding inaccuracies, but OpenCL requires a correctly rounded result for fma. So layering OpenCL over Vulkan leads to a needing an fma poyfill on many devices. I can't find where HLSL states the error bounds for operations, so that would separately be a great addition to the documentation. Maybe the standard optimizations only move towards at least as accurate implementations but it is difficult to evaluate without knowing what the accuracies are.

It is extremely unfortunate that the existing keyword is precise because it has nothing at all to do with precision, and is not strictly connected to any precision guarantees.

Transforming a * b + c into fma(a, b, c) should only be valid if the error on fma is at least as good as the individual operations.

I don’t think that’s relevant in the context of the existing precise keyword and proposed strict mode. The current precise keyword never allows promotion of a * b + c to fma(a, b, c) regardless of precision guarantees. On most architectures FMA is more precise than separate operations, and the promotion is still illegal under precise.
The goal of the precise keyword is to ensure different shaders compiled from the same code with different options (usually different render stages), generate the same result, and it accomplishes this by making optimization of the contributing computations illegal. The idea being “you get the math as you wrote it, not as the compiler decided to optimize it”. The strict math mode from the proposal will have that same impact by preventing optimizations that impact precision regardless of whether they increase or decrease precision. I chose the word strict specifically to start correcting the conversation because it isn’t really about precision.

Different APIs approach this differently. For example, Vulkan states they have the same accuracy so you might accumulate rounding inaccuracies, but OpenCL requires a correctly rounded result for fma. So layering OpenCL over Vulkan leads to a needing an fma poyfill on many devices.

I don’t think what Vulkan or OpenCL do here is really relevant. There’s some similarity between the HLSL precise keyword and the GLSL precise keyword, but they’re not completely identical in their usage.

I can't find where HLSL states the error bounds for operations, so that would separately be a great addition to the documentation.

HLSL is unfortunately under-specified. Part of why we created this process is to help fix that over time. DXIL (which HLSL lowers to) has some operation precision specifications here, and when lowering to SPIR-V the SPIR-V specification applies.

My concern with implementing this features is the "corner" case like, can I replace c*0 with 0. In Vulkan, Nan preservation is a module level option. Implementing with Vulkan as is, could cause us to miss some optimizations. That is for the SPIR WG to fix.
Based on the conversation above, I think the proposal should be clarified. If precise has nothing to do with precision, and strict is supposed to be the same thing, then we should not define strict in terms of precision. The proposal currently gives the definition strict as

The strict math mode prohibits optimizations which may impact the precision of results.

This definition could be understood to mean that a given operation has a range of acceptable results, and optimizations that might give a result outside of that range are prohibited. This is why Alan is very concerned about what the precision specifications are.
A better definition might be

The strict math mode prohibits optimizations which may change the result of the operation.

Some commentary on what happens with NANs, infs, and denormals would be important as well.
Does that seem reasonable?

The strict math mode prohibits optimizations which may change the result of the operation.

As a bystander, my usage of the existing HLSL precise construct is to ensure that the result of the operation is invariant under the presence of other operations (which may interact with the optimizer) as well as invariant to the shader stage and other compiler flags. This ensures that you can, for example, write out depth information in a compute shader and reasonable expect that it will match the depth computed in a vertex shader and vice versa. If I had to criticize the proposed wording, determinism is already expected in the sense that running the same code with or without precise used should produce identical results. Furthermore, it's also already expected that the codegen for the same input, compiler, and compiler options should be deterministic also.
I'm not really sure what the best wording is, but perhaps working backwards from the intent as described above may help.

My concern with implementing this features is the "corner" case like, can I replace c*0 with 0. In Vulkan, Nan preservation is a module level option. Implementing with Vulkan as is, could cause us to miss some optimizations. That is for the SPIR WG to fix.

This is good to know, but I'm unsure it factors in here. I haven't spent a lot of time thinking through this, but my intent wasn't that strict and fast have different NaN/INF behavior.

Based on the conversation above, I think the proposal should be clarified. If precise has nothing to do with precision, and strict is supposed to be the same thing, then we should not define strict in terms of precision. The proposal currently gives the definition strict as

I agree that I need to clarify the wording, but strict isn't currently defined in terms of granting specific precision. The draft spec says:

The strict math mode prohibits optimizations which may impact the precision of results. This prevents optimizations like fusing multiply and add instructions.

This does not say that strict is more or less precise. It disallows all optimizations that impact floating-point precision regardless of whether they increase or decrease precision. That is consistent with how the precise attribute works (when it does). The available (but old) docs for precise note that it disables arithmetic refactoring. The most important excerpt from the doc page there is this bit:

If components of a mad instruction are tagged as precise, the hardware must execute a mad instruction or the exact equivalent, and it cannot split it into a multiply followed by an add. Conversely, a multiply followed by an add, where either or both are flagged as precise, cannot be merged into a fused mad.

Regardless of whether you assume fused-multiply-add to increase or decrease precision, precise always disallows fusing or de-fusing.

This definition could be understood to mean that a given operation has a range of acceptable results, and optimizations that might give a result outside of that range are prohibited.

Is this not already true? The DIXL spec does define the precision of each operation (sans possible omissions which we should address). I'm pretty sure SPIR-V does the same. I don't know how things work on the Vulkan side, but for DirectX we absolutely have tests that verify floating point computations produce results within given tolerance ranges defined by units of least precision (ULPs). It is exceedingly rare for two different hardware vendors to compute floating point operations bit-for-bit identical.
precise does in effect give a user a way to guarantee that a specific chain of computations will fall within a certain range of possible values by ensuring that the operation precision is predictable (or as @jeremyong put it invariant).

This is why Alan is very concerned about what the precision specifications are.

None of this alters the existing per-operation precision guarantees. Hopefully we can put that concern to rest.

A better definition might be

The strict math mode prohibits optimizations which may change the result of the operation.

This doesn't say the same thing. How do you define the "result"? The exact bit pattern? That's not the guarantee. The guarantee is that we don't perform optimizations that could alter (positively or negatively) the precision of the final result as defined by ULPs.
That means we can't combine or separate instructions, we can't refactor or re-order operations. Each optimization fusing, de-fusing or re-ordering instructions potentially changes the computed precision of a result either by computing it at a different precision or adding/removing/shifting where floating point rounding occurs.
I realize that with my tirade about precise being a bad name, using the word "precision" to define an alternate solution may seem like I'm undercutting myself. The point is that neither the existing precise feature nor this feature actually guarantee precision or any specific operation precision, but they do relate to operation precision. The fact that we used the keyword precise makes this whole conversation so much more confusing because precise does not do what people intuitively expect: it neither increases nor decreases precision in all cases, it can do either depending on context.
The point of choosing the word strict is to say that the compiler is strict about preserving the code as it was written. Thus being predictable and invariant (@jeremyong I really like that wording 😀).

Some commentary on what happens with NANs, infs, and denormals would be important as well.
Does that seem reasonable?

This does really help me a lot in understanding where the spec is confusing, insufficient and where I need to explore some more.
Thank you!


Fixed segfault on loading text editor theme

Fixes the crash seen in #11282

Works for me. Thanks!

works for me too.

It works, still dont know why the EditorNode::getSingleton->get_theme_base()->get_theme() was not defined at _load_theme_settings.


Fix send payment test

fix the send payment test

Seeing if this works when merged in with #8088. If it does, we can merge all the work there to speed up the process of getting all these changes approved.


chore: add Spanner module

@imrannayer - can you give blueprint-solutions admin access to repo?

@imrannayer - can you give blueprint-solutions admin access to repo?

@anaik91 can you plz take care of this?

@imrannayer - can you give blueprint-solutions admin access to repo?

@g-awmalik done


error: member function expected 0 argument(s) when building

I clone the project on thet latest version on master branch (84cfa53c219dc7119b1716368cd6d8ed6dabc867)
zig version : 0.10.1
I'm on archlinux x86_64 Linux 6.2.12-arch1-1
The error :
/home/david/src/gmi2html/src/main.zig:208:16: error: member function expected 0 argument(s), found 1
    while (args.next(allocator)) |maybeArg| {
           ~~~~^~~~~
/usr/lib/zig/std/process.zig:777:9: note: function declared here
    pub fn next(self: *ArgIterator) ?([:0]const u8) {
    ~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
referenced by:
    callMain: /usr/lib/zig/std/start.zig:614:32
    initEventLoopAndCallMain: /usr/lib/zig/std/start.zig:548:51
    remaining reference traces hidden; use '-freference-trace' to see all reference traces

error: gmi2html...
error: The following command exited with error code 1:
/usr/bin/zig build-exe /home/david/src/gmi2html/src/main.zig -fno-strip --cache-dir /home/david/src/gmi2html/zig-cache --global-cache-dir /home/david/.cache/zig --name gmi2html -fsingle-threaded --enable-cache
error: the following build command failed with exit code 1:

I had a quick look and the type of ArgIterator.next changed with zig 0.10.0.
I don't have time to fix right now but will push an updated version in the next few days.
If anyone else wants to make whatever small changes are needed to bring it up to date, send a pull request and I'll merge it.
Until then, are you able to compile it with 0.9.x?

Should be fixed now, I've released version 0.5.0 which is compatible with zig 0.10.0 and 0.10.1.
AUR has been updated too.


'Val cannot be reassigned' when attempting to set container creationTime

Environment:

Jib version: 3.3.1
Build tool: Gradle (Kotlin) v7.6
OS: Windows 11

Description of the issue:
Using Gradle Kotlin, I cannot set the value of container.creationTime because I get the error "Val cannot be reassigned". Gradle build also fails with the same error. Also, setting "USE_CURRENT_TIMESTAMP" as the value fails with a "Type Mismatch" error. See the two screenshots below

Expected behavior:
Successfully set creationTime and Gradle build should be successful
jib-gradle-plugin Configuration:
plugins {
    java
    id("org.springframework.boot") version "2.7.6"
    id("com.google.cloud.tools.jib") version "3.3.1"
}

jib {
    from {
        image = "eclipse-temurin:17-jre"
        platforms {
            platform {
                architecture = "amd64"
                os = "linux"
            }
            platform {
                architecture = "arm64"
                os = "linux"
            }
        }
    }
    to {
        image = "xxxx"
        tags = setOf(System.getenv("SHORT_SHA")?.toString() ?:"")
    }
    container {
        creationTime = "USE_CURRENT_TIMESTAMP"
        mainClass = "xxx"
        ports = listOf("8081")
        jvmFlags = listOf("-Xms512m", "-Xmx1024m")
    }
}

@lewimuchiri  You may find this suggestion to be helpful since the creationTime has been updated to be a Gradle Property.

@mpeddada1 thank you. The suggestion works.
creationTime.set("USE_CURRENT_TIMESTAMP")


[WIP] Use different base image containing MCR

I am in the process of cleaning up the base MCR image a little bit, so you can directly base this image on it. Or is there a reason why you based it on bids_validator except that the MCR image is out of date?

Thanks @octomike, I hadn't realised that the base_validator image was out of date and unmaintained. I don't think the base_MCR image existed at the time the SPM BIDS App was created and it might actually be inspired from it.
An application compiled with the MATLAB Compiler is tied to a specific version of the MATLAB Runtime so it is not necessarily changing that often. Let's continue the discussion on your pull request in base_MCR.


Prototype of stopping after N rows loaded from taps

This is a sketch for how i introduced ability to limit rows from taps for the purposes of testing.
Warning: This code has not been executed. I have working code based on an older revision that i tested. So, i know the approach works with code very similar to this. I cleaned up the code to fit into latest meltano main, so that it would be easier to see the essence of the changes.
If this overall path is of interest, i could do more work-- i really would like thoughts about the overall approach before I do that. I dont know the meltano code well at all, so this might be a totally wrong way to do it.
Other notes that might be helpful:

i first tried a similar approach in the singer runner code, until i realized that was not the code used for meltano run
This uses an environment var to set the row limit. A much better solution of course would be to allow a property on the project and/or taps, so the row limit can be different for each tap. But (a) I dont know enough to approach that, and (b) for us this is sufficient
I am worried about whether this code would work in environments with concurrency more than just one. I did not do any testing along those lines.

@edgarrmondragon - I know it has been a while - but this is still of interest for us. What would be the next step? Resolving the failing checks?

@dcowden ^ fyi

@edgarrmondragon - I know it has been a while - but this is still of interest for us. What would be the next step? Resolving the failing checks?

Yeah, although the current implementation aborts the sync after any N lines have been emitted by the tap. Other than actual records, that count would include schema and state messages too. I'm fine with that, but just wanna make sure that is a good solution for other users' and yall's needs.
That said, some requirements to get this across are probable:

Make this controllable via a CLI option instead of the current env var implementation. I don't think we'd need a new setting for this.
Add a new test that validates this behavior. Without looking at what tests we have that may be a good starting point, this can range from easy to very hard so let me know if you need help with that.

The current pre-commit and coverage failures are ok at the moment, since I expect the refactoring and testing required by the above two points would address them.


No such file or directory error

When the command "ctra new react-component " is executed and there is no "packages" folder in the project, the fallowing error occurs:
[
[Error: ENOENT: no such file or directory, mkdir '/path/to/project/packages/component'] {
errno: -2,
code: 'ENOENT',
syscall: 'mkdir',
path: '/path/to/project/packages/component'
}
]
Infos:
OS: Linux, Ubuntu 19.04

Should be fixed in v0.4.1

push an existing repository from the command line, you have to make branch
git remote add origin https://github.com/Krishna9100/food-website.git
git branch -M main
git push -u origin main


iq roster result from server is not captured by debugStanza flag

debug code is missing in XmppCore.HandleIqResponse()
i fixed it with putting
if (debugStanzas) System.Diagnostics.Debug.WriteLine(iq.ToString());
at the end.

Thanks for this. Please open a pull request against the main branch. This way your contribution will be logged in git.


Hey I don't know how to debug js on ssr project

we can set break point in the nodejs code on server side for debug
we set break point in vue js on browser side for debug
but now how to debug in vue ssr project
@SinanMtl

Can you please brief what you want to debug?

Unfortunately, you can't debug for now. I will working on it. 👍


Client-side Manifest

Compile and copy client side files to www

Last things to test before closing:

[ ] When a client side source file changes, remove old manifest directories.
[ ] Multiple manifest directories can be used (having more than one HEAD throughout the app)
[ ] Switching to another manifest does not invalidate the others


Remove duplicate IDs in mempool requests and responses

Motivation
Currently, Zebra allows duplicate transaction IDs in some mempool service and crawler requests.
But we know those IDs should be unique.
Allowing duplicates is a minor denial of service risk.
Solution

Guarantee unique IDs in mempool service responses
Guarantee unique IDs in crawler task mempool Queue requests

Review
I think @upbqdn designed some of these requests?
Reviewer Checklist

[ ] Code implements Specs and Designs
[ ] Tests for Expected Behaviour
[ ] Tests for Errors

This is medium priority, because it's blocking a medium priority task.

This PR failed with an unrelated macOS acceptance test timeout, PR #2896 should help diagnose.
(This failure does not block merging this PR.)


Make zig.path and zig.zls.path default to "" (look up in PATH).

These will be overwritten by the initial setup process anyway.
This at least ensures that the default value doesn't cause a warning when editing settings.json.
Looking these programs up in PATH seems like a sensible default and is consistent with many other extensions.

Closes #157.

I agree with all points, thanks!


User configurable DNS servers

Often users need to configure specific DNS for their devices. We should enable this feature by allowing them to add this to the devices config.json.

@shaunmulligan do you have a feel for how this should work i.e. is the configured DNS server the only one that is ever queried? does it prefer, fallback to or ignore DHCP provided DNS?

I've implemented this feature using additional dnsmasq server lines which means that DHCP discovered DNS would be tried after configured DNS. Let me know if that sounds ok.
The PR is currently blocked on the NTP PR which modifies some of the same files.

I'll let @shaunmulligan to weight in here but for me this is what I thought as being the solution too.

@willnewton yeah, this sounds good to me 👍

Closed by https://github.com/resin-os/meta-resin/pull/750


Reference files within a module

Eliminate relative path references

+1
I need to import a file in self-installed package, as if the package were trully installed. currently there is only package.json under self-installed package directory.
Assuming I'm working on a npm project named "foo", I need to require('foo/dist/foo-bar-plugin'); or import 'foo/dist/foo.css';. This is a common scenario when writing example/test code, especially in web projects.
I have an idea to implement such a feature:

figure out files to be packed, by checking .gitignore, .npmignore and package.json#files, or just extrat output section Tarball Contents of npm pack --dry-run
add files(except package.json) to node_modules/foo, by copying files listed in Tarball Contents,  or by creating symlinks targeting Tarball Contents

Interesting ideas - I'll have a little ponder over this one :)


Implement robust auth

https://stevepolito.design/blog/rails-authentication-from-scratch
https://rubyonrails.org/2023/10/5/Rails-7-1-0-has-been-released
https://api.rubyonrails.org/classes/ActiveSupport/CurrentAttributes.html

Also https://github.com/rails/rails/issues/50446

Ah yes, thanks. I know they’re planning to ship a auth generator in rails 8. I just bought a copy of the Campfire source code so I can model our auth off that since I suspect this will be the internal reference they’ll use as they’re creating this generator.


Adjust to router changes

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
❌

Fixed issues
-

Tests should pass before merge.


Is it possible for this project to support tpc-bih?

As in the title, is it possible for this "tpch" to support "tpc-bih" in TPC-BiH: A Benchmark for Bitemporal
Databases? Bih benchmark adds columns related to temporal feature in SQL:2011 to origin tpch tables.

No, we are not planning to add support for it anytime soon


No PIN code required after first access in case of version upgrade

Describe the bug
No PIN code required after first access in case of version upgrade
To reproduce the behavior
Steps:

Installed v0.2.6 (131) and use PIN code to open the app
Upgrade version to v1.0.1 (134)
After upgrading successfully, open the app
Observe the screen displayed
Do not enter PIN code and close the app
Re-open the app and observe the screen displayed

Actual: https://share.icloud.com/photos/060Y1D5b0CyUMSf9q40UPVr3g

In Step 4: The enter pin code screen and the create master password screen are overlapped
In Step 6: Do not require enter PIN code to open app.

Expected:

In Step 4: Show the Enter pin code, after un-lock the app then navigate user to the create master password screen
If the user does not enter the PIN code yet, still require PIN code

Version test: v1.0.1(139)
Devices: IP 11
@Quangdm-cdm Please check: The enter pin code screen and the create master password are overlapped
https://user-images.githubusercontent.com/117999489/236418301-8740f935-1b02-4bf9-972e-1ff7e0f177c5.mp4

Version test: v1.0.1(141)
Devices test: IP 11, MI 8 Lite
Validation passed.


Update import/parsers notes in README

Fixes #1273

Coverage decreased (-0.5%) to 93.651% when pulling fa9510d3e9ef4333c84c85a280f32332b324e1d3 on danielmoi:update-import-parsers-readme into bdc05aa1d029b70125ae415e5ca5dca22250858b on benmosher:master.

Great that someone is updating the documentation on this! It helped me solve my case, but while I was trying to figure this out, I found out that all you have to do is this:

Run: npm install --save-dev eslint-import-resolver-typescript
Change my eslintrc.js to include:

settings: {
  'import/resolver': {
    typescript: {}
  }
}

I don't think this is very obvious from the documentation in this PR right now.

@johman10 it'd be great if you reviewed the PR and made a suggestion

The config mentioned in the typescript part of the README doesn't work for me. I made a base project with just 2 typescript files and exactly these extends values in my eslintrc but that doesn't work (see code below).
I think this points to either a problem somewhere else in the project, or a documentation issue. If this project doesn't really support typescript I would suggest pointing to eslint-import-resolver-typescript to let them explain the typescript support instead and remove all the typescript related code in this project.
If this project is trying to support typescript there should probably be a closer look at the support you offer and whether that all still works. I have been unable to figure out where this issue is coming from for now.
@ljharb what is your view on this? Are you trying to support typescript from this plugin?
Output

Command used
eslint --ext js,ts .
Dependencies
@typescript-eslint/eslint-plugin: 1.4.2
@typescript-eslint/parser: 1.4.2
eslint-config-airbnb-base: 13.1.0
eslint-plugin-import: 2.16.0
eslint: 5.15.1

Code
index.ts
import module from './module';

module();

module.ts
export default function () {
  console.log('Hello from module.ts');
}

eslintrc.js
module.exports = {
  env: {
    browser: true,
    es6: true,
  },
  extends: [
    'eslint:recommended',
    'plugin:import/errors',
    'plugin:import/warnings',
    'plugin:import/typescript'
  ],
  parserOptions: {
    ecmaVersion: 2018,
    sourceType: 'module',
  },
};

Yes, this plugin does and will support typescript. That's not the same thing as us having the expertise to fix the docs, or the implementation, or to figure out where the problem lies.

I'm getting the same errors as @johman10, tried adding resolvers as well and no dice.

When using '@typescript-eslint/parser' I get tons of import/named errors while 'typescript-eslint-parser' worked totally fine. So I can't upgrade to the new name / version until this is solved here.

@johman10 / @gretzky I think the TS config is updated to reference the modern TS parser in the latest version.
Closing because I think we got these updates in a later PR, thanks though everyone!


Link for downloading SBU dataset has changed. Enable the test again

[x] Closes #6390

SBU seems to be working again. Therefore, removing it from expected failure test cases

@ambujpawar you don't need to regularly update your PR unless GH is showing you merge conflicts. We will update it before merge. Reviews might take some time since we experience some heavy internal workloads.


not work with big file

i open  a 200M file into two split view
and the sync scroll not work
but a 10M file is ok

Thanks for your feedback, @tim0991, I guess one possible reason is the lines of the big file as the denominator is too big to make the fraction zero.


Adding balena into buildroot

Description
Hello
I am trying to add balena into buildroot. I consider balena_git.bb file from meta-resin as cross-compile procedure. I sent a patch in order to add balena I took some feed back from buildroot.  I have some question and I need your help in some point.
Questions

Why do we need some docker parameters as DOCKER_BUILDTAGS, DOCKER_GITCOMMIT, or directory as .gopath/src/github.com/docker/docker?
build creates .gopath/src/github.com/docker/docker/cmd/mobynit/mobynit binary but we copy bundles/17.06.0-dev/dynbinary-balena/balena. what is the differences?

You can flow the e-mail from https://patchwork.ozlabs.org/patch/925039
if I am wrong at some point please warn

Hi @refiktuzakli ,
DOCKER_GITCOMMIT provides information about the sources at runtime. See:
https://github.com/resin-os/balena/blob/b54f148fe1334735c1dc046643133c4708c501b0/hack/make.sh#L74
https://github.com/resin-os/balena/blob/df5d0f6fe7410472a0f8a4e1f45f991cf71d4def/hack/make/.go-autogen#L16
https://github.com/resin-os/balena/blob/df5d0f6fe7410472a0f8a4e1f45f991cf71d4def/dockerversion/useragent.go#L22
DOCKER_BUILDTAGS is used to strip the binary of things that were not tested/used. Basically, we deactivate the inclusion of some storage drivers.
I'm not sure what your question is about directory as .gopath/src/github.com/docker/docker.
We copy/install the mobynit  tool too. https://github.com/resin-os/meta-resin/blob/master/meta-resin-common/recipes-containers/balena/balena_git.bb#L115 This tool is meant to be used when the root partition is a docker storage itself. So you can have a root file system as a docker image and use mobynit to mount that image and start an init system from it (for example systemd).

I mean we create a link with .gopath/src/github.com/docker/docker, why  this path related with docker why not .gopath/src/github.com/resin-os/balena because we download this from github/resion-os/balena!
Another question is that What if we build without "-ldflags '-extldflags "-static"'", which functionality do we loos? size of balena binary remain same but size of mobynit binary goes down about 1M. How can we use mobynit binary when it build without ldflags?
thanks

linking to .gopath/src/github.com/docker/docker is done since internally the code refers to all subpackages with the github.com/docker/docker prefix, not doing so would break the build

those flags tell the Go compiler to build a static executable (which is actually trickier than just that: https://github.com/golang/go/issues/26492)

The Go package infra was updated to use Go modules and now the .gopath tree is no longer necessary / created.


New Manual Instrumentation Implementation

Reimplement manual instrumentation using a side channel to send data
directly from the target process to OrbitService. The old uprobe-based
implementation is still supported but will be deprecated. The new approach
yields massive performance gains (tested with >300K events/second).
Here is an overview of the new architecture:

This is a big PR, but a lot of code is boiler plate. Here are the components in order of importance:
// Add LockFreeApiEventProducer
src/Api/Orbit.cpp
src/Api/include/Api/LockFreeApiEventProducer.h
// Add ApiEvent proto
src/ClientProtos/capture_data.proto
src/GrpcProtos/capture.proto
src/LinuxTracing/LinuxTracingIntegrationTest.cpp
src/Service/ProducerEventProcessor.cpp
// Add ApiEventProcessor
src/OrbitCaptureClient/ApiEventProcessor.cpp
src/OrbitCaptureClient/CMakeLists.txt
src/OrbitCaptureClient/CaptureEventProcessor.cpp
src/OrbitCaptureClient/include/OrbitCaptureClient/ApiEventProcessor.h
src/OrbitCaptureClient/include/OrbitCaptureClient/CaptureEventProcessor.h
// UI
src/OrbitGl/ThreadTrack.cpp
src/OrbitGl/TimeGraph.cpp
src/OrbitGl/TimeGraph.h
// Add ApiInterface target for introspection
src/Api/CMakeLists.txt
src/OrbitBase/CMakeLists.txt
// Build
CMakeLists.txt
kokoro/builds/build.sh
// Adapt OrbitTest to use the new manual instrumentation API
src/OrbitTest/CMakeLists.txt
src/OrbitTest/OrbitTest.cpp
// Use new C API
src/Api/include/Api/Orbit.h
src/LinuxTracing/TracerThread.cpp
src/OrbitBase/OrbitApiTest.cpp
src/OrbitBase/Tracing.cpp
src/OrbitBase/include/OrbitBase/Tracing.h
// Maintain support for previous manual instrumentation (uprobe based)
src/OrbitGl/ManualInstrumentationManager.cpp
src/OrbitGl/ManualInstrumentationManager.h
// Remove previous version of Orbit.h
src/Orbit.h
Note: The use of the new ScopeTree data structure to merge manual and
dynamic instrumentation data will come in a subsequent PR.

@dpallotti , @florian-kuebler , @dimitry- , thanks a lot for your comments, and thanks for bearing with the big PR. I addressed most of the issues, but I'm still writing the tests, will have those in tomorrow.

@dpallotti , @florian-kuebler , @dimitry- , again thanks for your help. I think I have addressed all the comments, please let me know if I missed anything. I added ApiEventProcessorTest.cpp that tests the ApiEventProcessor both in isolation and as part of a CaptureEventProcessor. Please take another look, thanks!


[RESEARCH]: Using Vega/Vegalite to create charts as outlined in the design sketches

We want to reproduce the follow charts in Vega/Vega-lite grammar:
Line chart for showing simulated trajectories across multiple sampled runs

Show all trajectories
Show statistical aggregate as a separate time series (e.g. mean of the sample-trajectories)

Line chart showing calibration or optimization losses

Show trajectory
May want to consider capabilities to do different scales (linear, log)

Area chart + xy-scatter for showing error distributions and individual error across samples against some ground truth.

Show an area chart (estimate)
Show the "randrop" plot of individual samples

Examples:
Technical example showing raincloud + boxplot of a dataset:

https://gist.github.com/mwdchang/405758adac2e05fc285e50585518fe18

Overlapped by other issues.


KeyError: 'optimizer'

@PeizeSun hi， I met the same problem when funetuning the t2i model:
bash scripts/tokenizer/train_vq_finetune_continue.sh --cloud-save-path /data/vjuicefs_ai_camera_llm/11170092/00proj/LlamaGen/cloud_save --data-path /data/vjuicefs_ai_camera_llm/public_data/vivo_internal_data/AIPortrait/crop_imgs_ffhqcrop_png/20230703_fourth_fix-1_625 --image-size 256 --vq-model VQ-16 --dataset  coco --global-batch-size 32"

Does vq_ds16_t2i.pt also need to be updated?
Traceback (most recent call last):
  File "/data/vjuicefs_ai_camera_llm/11170092/00proj/LlamaGen/tokenizer/tokenizer_image/vq_train.py", line 320, in <module>
    main(args)
  File "/data/vjuicefs_ai_camera_llm/11170092/00proj/LlamaGen/tokenizer/tokenizer_image/vq_train.py", line 150, in main
    optimizer.load_state_dict(checkpoint["optimizer"])
KeyError: 'optimizer'
Traceback (most recent call last):
  File "/data/vjuicefs_ai_camera_llm/11170092/00proj/LlamaGen/tokenizer/tokenizer_image/vq_train.py", line 320, in <module>
    main(args)
  File "/data/vjuicefs_ai_camera_llm/11170092/00proj/LlamaGen/tokenizer/tokenizer_image/vq_train.py", line 150, in main
    optimizer.load_state_dict(checkpoint["optimizer"])
KeyError: 'optimizer'

Or did I make a mistake somewhere? I would be very grateful if you could give me some help！

Hi~
This is not the mistake in your side. It is because our released vq_ds16_t2i.pt doesn’t include optimizer parameters.

Hi~ This is not the mistake in your side. It is because our released vq_ds16_t2i.pt doesn’t include optimizer parameters.

Thank you for response！ I was wondering if there are any plans to release an updated version with the necessary optimizer parameters？ Alternatively, can I just skip training this tokenizer and directly train the t2i model?
I appreciate your guidance and look forward to your response.

I get the same problem when finetuning gptxl,
optimizer.load_state_dict(checkpoint["optimizer"])
KeyError: 'optimizer'
t2i_XL_stage1_256.pt   I wonder if this .pt file has the same problem? @PeizeSun thanks for your attention!


codegen-templates-scripts init doesn't work

I ran codegen-templates-scripts init and got:
error: codegen-templates-scripts-init(1) does not exist, try --help

But I cloned the repo and ran:
./packages/scripts/codegen-templates-scripts/cli/codegen-templates-scripts.js  init

and it worked fine.

Creating a new GraphQL Code Generator template project under: /home/niko/WebstormProjects/deps/graphql-code-generator```

```[niko@arch-desktop graphql-code-generator]$ codegen-templates-scripts init
Creating a new GraphQL Code Generator template project under: /home/niko/WebstormProjects/deps/graphql-code-generator```

I can't reproduce it, can you please add a repro?

Downloaded the latest version and tried again. Worked just fine


Add polyfill for get_SetupInformation()

Is it possible to add code to handle this Error Message: Method not found: 'System.AppDomainSetup System.AppDomain.get_SetupInformation()'.

What throws the error?

It was an issue with Spire docx generator - Spire does not support .NET Core, so you can ignore my question. Thanks.


Can start same task twice when adding task to tasks.json in between

VSCode Version: Code - Insiders 1.11.0-insider (cc46bbf5b0412e464f89280e828d602fbc06a0f4, 2017-03-31T13:43:36.065Z)
OS Version: Darwin x64 15.6.0
Extensions: none

While verifying #19241
Steps to Reproduce:

Start with:

{
    "version": "0.1.0",
    "_runner": "terminal",
    "tasks": [
        {
            "taskName": "tsc",
            "command": "tsc",
            "args": ["-w"],
            "isShellCommand": true,
            "isBackground": true,
            "problemMatcher": "$tsc-watch"
        // },
        // {
        //     "taskName": "tsc2",
        //     "command": "tsc",
        //     "args": ["-w"],
        //     "isShellCommand": true,
        //     "isBackground": true,
        //     "problemMatcher": "$tsc-watch"
        }
    ]
}

Start first task
Comment in the second task
Start first task again
-> Starts first task a second time instead of showing message about that task already running

Good catch.
Reason being is that we reparse the file and assign a different unique ID. Need to think about how to avoid this.

Moving to May due to priority changes. Terminal will still not be the default.


Change testEnvironment on a per test basis

Do you want to request a feature or report a bug?
Feature
What is the current behavior?
At the moment, I can set the testEnvironment to either node or jsdom in package.json.
My exact problem is that I have a few tests that use Knex, which in turn uses node-uuid, which outputs a console.warn if window exists, but window.crypto is not available.
See also this issue on jsdom: https://github.com/tmpvar/jsdom/issues/1612
The console.warns are not that bad (just annoying), but I can imagine that other modules may have wildly different behaviours for running in node or jsdom, which may lead to obscure bugs.
console.warn node_modules/node-uuid/uuid.js:48
      [SECURITY] node-uuid: crypto not usable, falling back to insecure Math.random()

What is the expected behavior?
Allow every test file to overwrite the setting in package.json. If I understand correctly, every test is run separately, anyways, so this should help?
Please provide your exact Jest configuration and mention your Jest, node, yarn/npm version and operating system.
yarn 0.17.10,
macOS 10.12.2
From  package.json:
"jest": {
  "testEnvironment": "jsdom"
},
...
"babel-jest": "^18.0.0",
"jest": "^18.1.0",

You should be able to mock console.warn and window.crypto for every test

If I mock console.warn, I lose every warning, not just that one warning that I don't want.
If I mock window.crypto, I have to match the part of the implementation that node-uuid uses, which seems very brittle to me. Any suggestions how to do this?

for the this particular test suite or even test case you can do something like:
const warn = console.warn;
console.warn = jest.fn(); // mock console
// tests...
console.warn = warn // restore console

You should be able to mock node-uuid too:
jest.mock('node-uuid', () => {
  return {
    v4: jest.fn(() => 1)
  };
});

Oh, thanks, that did it! (For this case)
Should I close this, or could the feature request in general be considered for jest?

If you're interested in creating something like this, feel free to send a PR.
But it won't be high on our priorities list I guess.
@cpojer what do you think?

But it won't be high on our priorities list I guess.

I mean implementing such feature by us, not your PR – pull requests are always welcome, but it's often good to discuss the topic before diving in.

I would also welcome a feature like this. With the jsdom versus node envs, is it just the bootstrapping of the former that takes longer or is it a bit slower generally speaking?
As a side question, how would one go about testing code that's meant to run both in the browser and in node, but that behaves slightly differently depending on where it runs?

We could add:
/**
 * @testEnvironment jsdom
 */

and toggle based on the specified value. I'd be supportive of such a change to Jest.

I'd like to look into this, but I'm unsure where to start.
It looks like the environment is bootstrapped before tests are ready (which is fine for the default, I suppose) here: https://github.com/facebook/jest/blob/7ca7dac05797a6ba2beac4e423504be6ea858c44/packages/jest-runtime/src/cli/index.js#L79-L90
But where would I hook in to check for a @testEnvironment and then potentially create a new env and run that test on there?
A small push in the right direction, and hopefully I can implement this. 😄

@SimenB it's right here: https://github.com/cpojer/jest/blob/master/packages/jest-cli/src/runTest.js#L32 you have the path to the test file and this is where the env is created.

PR opened #2859

@SimenB have you had a chance to look into https://github.com/facebook/jest/issues/2587#issuecomment-279097635 ? We'd like to always run our tests with both jsdom and node testEnvironments, since we server-render everything - is there an easy way to do that in the same jest run?

You should be able to use Jest’s multi project runner to duplicate your config and change the test environment. That would run all tests twice in a single run.

Thanks! Is there documentation on that you could link to?

See https://facebook.github.io/jest/docs/en/configuration.html#projects-array-string-projectconfig

@cpojer thanks - all the documentation I found talks about how to use "projects", but not how, in a JSON config, to share configuration values across projects. Specifically, I tried to put things top-level, and only put per-project overrides in "projects", but that didn't work - it seemed to ignore all the top-level keys when "projects" was present, so I had to manually duplicate them.

Jest also supports JS configs nowadays, so I probably recommend using a top level JSON config that includes two JS configs that are proxies to the config object that's in a third file and only toggle the env. It's a bit complex but in the end it's a pretty powerful setup. For fewer files you can also only have two files: one that exports the default setup and one that exports that with the env overwritten.

Is there any way that top-level values could get inherited down into projects when both are specified?

Can you open up a separate issue for that? It kinda goes back to the discussion here: https://github.com/facebook/jest/pull/5176#issuecomment-353873177
The difference between GlobalConfig and ProjectConfig is not easy to understand, and things like the inheritance you mention is not really possible (and it's non-obvious).


Content-Length:0 on GET for client.HttpClient with Message to send

I test my server with Rest POST/GET/PUT/DELETE method.
For every request I provide a payload.
For PUT it works very well.
The code looks like this:
public void putUser() {
    variable("uid", 12345);
    variable("name", "Luigi");
    echo("uid is: ${uid}");
    echo("name is: ${name}");

    http().client(basicRestfulClient)
          .put("/v1/user/")
          .contentType("application/json")
          .payload("{\"uid\":${uid}, \"name\":\"${name}\"}");

    http().client(basicRestfulClient)
          .response(HttpStatus.OK)
          .messageType(MessageType.JSON)
          .payload("{ \"URI\": \"/v1/user/\", \"action\": \"modifyUser\", \"method\": \"PUT\", \"name\": \"${name}\", \"uid\": ${uid} }");
}

For GET I have the following:
public void getUser() {
variable("uid", 12345);
echo("uid is: ${uid}");
    http().client(basicRestfulClient)
          .get("/v1/user/")
          .contentType("application/json")
          .payload("{\"uid\":${uid}}");

    http().client(basicRestfulClient)
          .response(HttpStatus.OK)
          .messageType(MessageType.JSON)
          .payload("{ \"URI\": \"/v1/user/\", \"action\": \"getUser\", \"method\": \"GET\", \"uid\": ${uid} }");
}

When I run GET test I'm getting the error from the server, since my server is expecting the body, and for some reason the body is stripped
The debug for GET looks like the following :
07:11:07,365 DEBUG    client.HttpClient| Sending HTTP message to: 'http://localhost:8080/v1/user/'
07:11:07,365 DEBUG    client.HttpClient| Message to send:
{"uid":12345}
07:11:07,366 DEBUG ingClientInterceptor| Sending Http request message
07:11:07,369 DEBUG   Logger.Message_OUT| GET http://localhost:8080/v1/user/
Accept:text/plain, application/json, application/*+json, /
Content-Type:application/json
Content-Length:0
Is it a bug or apache.httpclient doesn't allow any body on GET requests?
Thanks in advance fore your help

The body content is automatically removed for GET requests. I think this is particular because Spring framework's RestTemplate or underlying HttpClient is removing the body for GET requests

Thanks a lot for the super quick response. That's what I thought. By the way, you have a nice test framework.


Try to fix spurious "X commits behind" messages #gitfail

Thank you for your pull request and welcome to our community! We require contributors to sign a Contributor License Agreement and we don't seem to have CLAs on file for these contributors to the Pull Request: (@pmonks). In order for your PR to be reviewed and merged, please follow the directions at the link above.
Project team: please do not merge this Pull Request until Foundation staff have confirmed that a CLA is in place for the new contributor(s) listed above.
If there are any questions, please don't hesitate to get in touch with our Infrastructure Support team. /CC @finos-admin


Support \tcolorbox?

For equations in colored boxes (and colored boxes in $$equations$$)

I don't think tcolorbox works in math mode, which is the primary goal of KaTeX.  Or are you referring to a different package?

My mistake! Looking into this some more, getting a colored box around part of an $$equation$$ seems quite involved, and probably out of scope for katex. That's a shame.

Something like this?

It involves post-processing.
(1) The magenta border on the numerator of the fraction is rather ugly - I tried to style it better but it's not surprising it failed given the space constraints of the square root.
This was a matter of identifying the element and adding a class to that element.
(2) The green border around the "1 +" involved adding a delimiter to the equation (I used [[ ]] double brackets) to identify the bit I wanted a border around. I then scraped the elements between those delimiters, put them in a new span with the required class, and deleted the original spans.
The second approach may be feasible for you, depending on the complexity of the expression.
Here's the fiddle: https://jsfiddle.net/mbourne/pq3awgfe/
Regards
Murray


previewer: support for zip files

Introduce zip file previewer so that users can see and inspect the content of the file before downloading it.
Ideally, for big files, the users would be allowed to mark one (nested) file from the zip and download only that one.  This could be done as a future improvement.
The previewer could use new data demo record introduced in #1927 for tests.

Check https://github.com/inveniosoftware/invenio-previewer/blob/master/invenio_previewer/extensions/zip.py


Bug after window name change from song

Describe your bug here.
I guess im the bug spotter in this engine. Jokes aside, you should know by the title, screen shot is below.

Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
Nope.
If you use 1.12.0 or earlier, did you have Optimized Chart Loading turned on?


automaticAlias doesn't work well with instructions

Hi, I encountered some issues with automaticAlias. Following is my config:
const fuse = FuseBox.init({
  homeDir: "./src",
  output: "./build/$name.js",
  useTypescriptCompiler: true,
  allowSyntheticDefaultImports : true,
  compilerOptions: {
    baseUrl: "./src"
  },
});

const vendor = fuse.bundle("vendor").instructions("~ index.js");
const app = fuse.bundle("app").instructions("> [index.js]");

This results in the code imported from alias (components, utils, etc.) isn't included in either of the bundles. I believe in both cases FuseBox confused the components with packages in node_modules. Note that I do have correct alias mapping logged to console:
  → Applying automatic alias based on baseUrl in tsconfig.json
  →
 	components => "~/components"
	libs => "~/libs"
	scss => "~/scss"
	utils => "~/utils"

If I create multiple bundles but don't use instructions, it works.
Interestingly, replacing compilerOptions with alias will make it work with instructions.

After some debugging turns out WorkflowContext.addAlias only get called once if using compilerOptions, so the contexts of the bundles have empty aliasCollection. When using alias it gets called three times, once for FuseBox.init and two more times for each of the call to FuseBox.bundle.

Thanks for the issue! We will look into it

FuseBox didn't have compilerOptions option on init last time I checked :P
Try setting on FuseBox.init:
tsConfig: [{ baseUrl: './src' }]

which over-writes tsconfig.json file compiler options (merges).

that's correct. Closing this for now

Oh I see, thanks 👍


[fix](index tool)Fix index_tool build error

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Better documentation

The current documentation is on the "quick and dirty" side. Enough to start using the library in a few minutes, but could have better explanations.
Also documenting the internals would probably be beneficial for contributors, though the code size is short enough for peope to get into fast.

I'm not sure we need an entire Github-pages-based website, as the library is fairly easy to use (IMO, but maybe i'm biased?). That said, having a few examples in a more structured manner would be better.
I'm open to ideas, i'd prefer that we establish what is needed and useful before starting.
On the top of my head, having examples of library use for all PB types would be nice (string, byte, submessage, ints...), one example / type to make things clear. Adding all the details for ManagedString usages would be nice.
I'll do a review of issues and subjects this week end i think, i should have a bit of time for that.
Thanks a lot for your interest.

I'm open to ideas, i'd prefer that we establish what is needed and useful before starting.
On the top of my head, having examples of library use for all PB types would be nice (string, byte, submessage, ints...), one example / type to make things clear. Adding all the details for ManagedString usages would be nice.

+1, esepcially on the ManagedString 😄
happy to come up with some examples and more changes to the README and propose a PR once we laid out what should be spelled out first.


When creating a user with an encrypted password with a startup-config, Truncation occurs

Steps to reproduce:
Created a topology that included some Junos vSwitch images -> Added a block config in the topology file -> the resultant encrypted password (if if contained . or / characters) would truncate.
startup-config: |
  system {
    login {
      user someuser {
        class super-user;
        authentication {
          encrypted-password "$6$8gNjhXy6$Vj69F.QdM.Nvz1fDDYhnwb27cMSXJmXxS23ti50z/liD3obqPRYs5UKHVbKFRtqZO.AoXJIVR6Um17RFfSwWm1"
        }
      }
    }
  }

What winds up happening is the encrypted password appears as ".QdM.Nvz1fDDYhnwb27cMSXJmXxS23ti50z/liD3obqPRYs5UKHVbKFRtqZO.AoXJIVR6Um17RFfSwWm1"
ContainerLab Version 0.56.0

Hi @lostdave
containerlab/vrnetlab do not modify the contents of the provided startup config.
The way it works with junos is an disk with the startup-config is mounted to the VM, and junos should read the config from it
https://github.com/hellt/vrnetlab/blob/master/vjunosswitch/docker/make-config.sh
I suppose you should check the contents of that disk to verify that the encrypted string is there as it was in the yaml file

Sorry ....slow to get back to this one..travel and training stuff....
i will have a look in the next couple of weeks and see what is going on

Yes, the issue stems from the fact that the https://github.com/a8m/envsubst package that we use to replace env vards in the topology file replaces the env vars that are not set or empty.
This leads to $foo to be replaced with "" if the $foo is not even set.
Unfortunately, the original package maintainer stopped maintaining the package and this particular feature request left dangling - https://github.com/a8m/envsubst/issues/56
I made a fork of a fork by @dnitsch that added some parts of this functionality, but it doesn't work when the env var is set with the default in the template. To demonstrate this use case I added a test case that currently fails:
https://github.com/hellt/envsubst/blob/50934c197f4028463435a6aa0e8556ca60b442e5/parse/parse_test.go#L179
Looking for brave contributors to see if they can crack it


Fix a ReST escape

BTW, both :class:`Variable`\s (escaping s) and :class:`Variable`\ s (removing a space) are OK.  Its style is mixed in the document.

Jenkins, test this please.

Jenkins CI test (for commit 5fd6e4689e65f4f8f2f5e7b40efccfd1df3a7229, target branch master) succeeded!

LGTM!


fix(falco): update README.md

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

If this PR will release a new chart version please make sure to also uncomment the following line:

/kind chart-release

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area falco-chart

/area falco-exporter-chart

/area falcosidekick-chart

/area event-generator-chart

/area k8s-metacollector

What this PR does / why we need it:
Update README.md.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Checklist

[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] CHANGELOG.md updated

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: alacuku
Once this PR has been reviewed and has the lgtm label, please assign issif for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: alacuku, Issif
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Issif]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 76ff2e819abd9da295bef2c64afaf4f473668b1e


Error running improvecentrality.py - TypeError: '<' not supported between instances of 'DegreeView' and 'int'

Running this under Python 3.8.10 on a Windows Subsystem for Linux (WSL)  Ubuntu 20.04 distro.  The node is on my local network and other scripts like nodeinterface.py work fine.
Command 'python3 ./improvecentrality.py'.

It looks like your node was not found in the graph. Please double-check the key in improvecentrality,conf
I will add a error for this.

I did not see a step in the improvecentrality.py file to add my nodes public key in the improvecentrality.conf file. After adding it the program completed successfully.

The script should have said "Please complete the config and rerun" upon first run, is there a way to improve this message?
I have also added a proper explanation if the pubkey was not found in the graph.

I would make the message say something more specific like: "The nodes public key was either not found or was invalid.  Please verify that the pub_key setting in the improvecentrality.conf has the correct public key for your node and re-run the program".

That's similar to the error message I just added. Upon first run, the program cannot check the key for validity before you have a chance to enter it, that's why the initial message has to be so simple.

This looks fine: https://github.com/Gridflare/lndpytools/commit/c2520290e8cc64fbd9c9891c3df36a259caefcce


model clone

The model's clone is created with reproducing the links between PetriSim objects. Only links by combine places are considered in method clone() are developed and tested. The links by initializing action given by  method doT() should be tested afterwards.
The array links and  method linkObjectsCombiningPlaces() is added. Every call of method linkObjectsCombiningPlaces() is accompanied by adding to array links the object of private type LinkByPlaces. This type save information about places of PetriSim objects which are linked (in detail, Petrisim object plus number of place in it's list of places).

New  version contains the possibility of clone.
You can use clone() if you construct model using linkObjectsCombiningPlaces() . See getModel() in TestPetriObjPaint class. If you create combine places this way the you can create model's clone:
model.clone().
If don't use linkObjectsCombiningPlaces() for combining places model's clone will have clones of PetriSim objects but without links between them.


Add documentation about how logging currently works.

I added information to the README about how logging currently is working. If we want to make zap work everywhere, we'll need to do that work in the future.
Pivotal: https://www.pivotaltracker.com/story/show/155044294

This looks good. I've been thinking a little about how we want to wire up dependencies and this is definitely a consequence of us not having done that yet. If our handlers obviously depended on a configured logger it would have been clear why zap wasn't working from our tests.

Thanks for the feedback!


Add support for ConstantBlock

Fix #323

BTW my patch seems useless.
CONSTANTS are already handled in a really bad manner

may you point me to where they are handled?

may you point me to where they are handled?

I don't really know but in the outside cpp we got:
    struct cadad_Store {
        int ca_type;
        double cai0;
        int reset;
        int mech_type;
        **double FARADAY;**
        int* slist1;
        int* dlist1;
        ThreadDatum* ext_call_thread;
    };

I have found the problem I had with units. in cagk.mod in hippocampus there is this block:
UNITS {
	FARADAY = (faraday)  (kilocoulombs)
	R = 8.313424 (joule/degC)
}

I wonder:
is it possible to have the same thing with CONSTANTS? I mean, is it still a valid mod file? If yes, how do we deal with ispc? Because I fear these constants get declared twice

I have found the problem I had with units. in cagk.mod in hippocampus there is this block:
UNITS {
	FARADAY = (faraday)  (kilocoulombs)
	R = 8.313424 (joule/degC)
}

I wonder:
is it possible to have the same thing with CONSTANTS? I mean, is it still a valid mod file? If yes, how do we deal with ispc? Because I fear these constants get declared twice

I agree that this is an issue and something that NEURON for example doesn't take care of.
I think we could print some error in this case from NMODL though :+1:

I did not investigate thoroughly, I just put here what could be a corner case: what happens if we re-define a variable like FARADAY in CONSTANTS?

I wouldn't about this in code generation pass. This is todo in semantic analysis pass where such error should happen (there is no such pass but that is for future task).

I did not investigate thoroughly, I just put here what could be a corner case: what happens if we re-define a variable like FARADAY in CONSTANTS?

I wouldn't about this in code generation pass. This is todo in semantic analysis pass where such error should happen (there is no such pass but that is for future task).

in units it IS handled. We need to remember to remove/modify the code when we do the semantic visitor


CPU monitoring in EC2 (CloudWatch) intermittent failures

Author Name: OpenNebula Systems Support Team (OpenNebula Systems Support Team)
Original Redmine Issue: 5554, https://dev.opennebula.org/issues/5554
Original Date: 2017-11-15

Due to no datapoint set returned on some API calls.

Original Redmine Comment
Author Name: OpenNebula Systems Support Team (OpenNebula Systems Support Team)
Original Date: 2017-11-15T15:22:07Z

The attached driver does not clear the CPU value if the API call does not return datapoints

fixed in one-5.4 and master


Custom videos not playing

General troubleshooting tips
Required information
In order to help us sort your issue, we ask that you provide the following information:

iMac 2017
Big Sur 11.6
Dual Monitors

AerialLog.txt
Custom screensavers will not play. They are located in Shared under Users but still they don't play.

Apologies as it looks like I completely missed your issue, really sorry about that.
You are running Aerial 1.6.0 and that version is not compatible with Big Sur, please update to latest, you can get it here : https://github.com/glouel/AerialCompanion/releases/latest/download/AerialCompanion.dmg
Again, sorry I missed your issue


[Bug] [dinky-admin] Task permission aspect do not work when enable 'OWNER' startegy

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
When enable strategy not equals 'ALL', although task operation buttons is disabled, tasks can still be operated by restful interfaces as follows:
For example:
here is a task belongs to admin

other users can still operate task by interfaces

What you expected to happen
Fix this bug
How to reproduce
call interface
Anything else
The reason of this bug is this:

Version
dev
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I have submitted a PR to fix this bug: https://github.com/DataLinkDC/dinky/pull/4103


ADTypes v0.2 compatibility for test restructuring

Merge of #2252 into #2237. I want to see the CI pass here before merging #2252  (I'm unable to check this locally for silly reasons).

Sorry, meant to request review on the other PR you just reviewed, not this one.


Release 5.0.1 is broken

See comments in
https://github.com/chef-cookbooks/ohai/commit/d6bfb34a7907295f2960e1f9d6f9afa1d10ce71d

same

5.0.2 has been released, but I'm curious what versions of Chef everyone here is on

12.19.36 is what I'm on.

chef -v
Chef Development Kit Version: 1.2.22
chef-client version: 12.19.36
delivery version: master (0b746cafed65a9ea1a79de3cc546e7922de9187c)
berks version: 5.6.4
kitchen version: 1.16.0

Chef 12.19.36

We ran into this problem as well and 5.0.2 fixed it:
wsoula@aus2-orca-d1:~$ chef-client -v
Chef: 12.15.19

We figured out the root cause is actually defining the action class before the actions. This causes the class_eval to fail. I've opened a PR to move the action_class to the end of each resource which fixes the issue. We also have a PR in the works that resolves the bug in Chef itself so that Chef 13 will just do the right thing. Using action_class.class_eval again will give us back compatibility with Chef 12.5 and 12.6, which are both heavily used.


feat(examples): first implementation of exist-exec

Supports reading queries from the command line or from a file (with -f).
Variables can be bound using the -b parameter.
The the path to the query file, the bound variables and even the query string can be read from standard input by using -.
closes #206
Usage
exist-exec

Execute queries in an exist-db

Usage:
exist-exec [options] [query]

Options:
      --version  Show version number                                   [boolean]
  -f, --file     Read query File (pass - to read from standard input)
  -b, --bind     Bind variables (either as JSON string or - to read from
                 standard input)                            [default: (default)]
  -h, --help     Show help                                             [boolean]

Examples
With query string
spec/examples/exist-exec '1 to 9'

Output
1,2,3,4,5,6,7,8,9

With bound variables
spec/examples/exist-exec \
  -b '{"from": 1, "to": 50}' \
  'sum($from to $to)'

Output
1275

Reading from file and piping output to jq
spec/examples/exist-exec \
  -f xquery/list-resources.xq \
  -b '{"collection":"/db", "glob":"*", "extended":false}' | jq -M

Output
[
  {
    "type": "collection",
    "name": "system",
  },
  {
    "type": "collection",
    "name": "apps",
  }
]

Reading bound variables from standard input
echo '{"a":16}' | spec/examples/exist-exec -b - '$a * $a'

Output
256

Reading query string from standard input
echo '$a * $a' | spec/examples/exist-exec -b '{"a":16}' -

Output
256

@line-o The description sounds great!
I'm curious about the serialization details. Does it use adaptive by default?

I have to admit I don't know the default setting in xmlrpc. I guess   it's xml.

Or string

But from my file example you can see that any serialization can be set in the query.

Well, the other CLI tools also need s running exist DB somewhere. So I think you are right and I will add a note to the  examples readme if that is not clear.

@duncdrum could you have look at https://github.com/eXist-db/node-exist/pull/219/commits/03d03c07db071bdfaab9c3b155f56d2d3d17078f if that makes it clear enough?

Of course I made a typo there (enabaled) sigh

yup that does the trick


Fix pydocstyle D401: first line should be in imperative mood

Description of the changes being introduced by the pull request:
Make sure the codebase is more compliant with Python's docstring convention.

See https://github.com/theupdateframework/python-tuf/pull/2231#pullrequestreview-1222490755


Kotlin-Reflect dependency

Is there a way to drop the kotlin reflection dependency? a 5 MB library for getting the name of the class from companion sounds too much :)

any idea or direction how to do that?
On Sep 16, 2016 18:37, "Yoav Sternberg"<EMAIL_ADDRESS>wrote:

Is there a way to drop the kotlin reflection dependency? a 5 MB library
for getting the name of the class from companion sounds too much :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/MicroUtils/kotlin-logging/issues/8, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ACj0QUoTW70OngockHdFZnR0adgxDtkBks5qqre3gaJpZM4J_EdF
.

one way I think of is checking the name if it ends with "$Companion" and remove it. what do you think?

Since you can name companion this is not 100% working solution, but I believe it is acceptable limitation.

A solution for that is to look for the last $ sign. do you think it is better?

The removal of kotlin-reflect dependency will be in 1.3.3


Where is "tensorflow/lite/delegates/gpu/gl/compiled_model_generated.h"?

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): 16.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: n/a
TensorFlow installed from (source or binary): source
TensorFlow version: master
Python version: n/a
Installed using virtualenv? pip? conda?: n/a
Bazel version (if compiling from source): n/a
GCC/Compiler version (if compiling from source): n/a
CUDA/cuDNN version: n/a
GPU model and memory: n/a

Describe the problem
I'm looking at serialization.h#L28 where it includes tensorflow/lite/delegates/gpu/gl/compiled_model_generated.h, but where's the file located? Is it meant to be generated at compile time? How is it generated?
Thanks

This is not Build/Installation or Bug/Performance issue. Please post this kind of support questions at StackOverflow. There is a big community to support and learn from your questions. GitHub is mainly for addressing bugs in installation and performance. Thanks!

Hi @hoonkai,
I'm facing the same issue while compiling too.
How have you solved this?

me too

@makihiro and @mlinxiang, Please post a new issue and provide all the information asked by the template. We ask this because it's more efficient to have one thread dedicated to one issue. Thanks!

@makihiro and @mlinxiang, Please post a new issue and provide all the information asked by the template. We ask this because it's more efficient to have one thread dedicated to one issue. Thanks!

i fixed this, because i compile tflite use tools/make  tool, first i run bazel build --cxxopt=--std=c++11 -c opt --config android_arm64 tensorflow/lite/delegates/gpu:gl_delegate which will convert tensorflow/lite/delegates/gpu/gl/*.fbs file to *.h in the response path of bazel-genfiles, which contain compiled_model_generated.h, then i copy these head files to the original path where *.fbs, and my build task success.

@mlinxiang Thank you. That works fine for me.


Reset progressbar without 'return' animation?

Hello!
I would like to ask if there is any possibility to reset the progressbar without the returning animation?
My desire solution is:
-----------xxxxxxxx (60%) -> xxxxxxxxxxxxxxxxxx ( 0% ) (no animation, pure reset)
Current default solution using the lib is:
-----------xxxxxxxx (60%) -> -------xxxxxxxxxxxx (40% ) -> -----xxxxxxxxxxxxxx (20% )
etc... (it goes back quickly... but it's annoying unfortunately...)

For your case it's annoying, for others it might be desired :)
You could create your own animation component that wraps CircularProgress based on https://github.com/bgryszko/react-native-circular-progress/blob/master/src/AnimatedCircularProgress.js & make use of componentDidUpdate (https://github.com/bgryszko/react-native-circular-progress/blob/master/src/AnimatedCircularProgress.js#L20) where you would decide if animate the component or set a fix value.

Thanks! Sorry mate, did not want to sound rude of course! :)

@mkpietrzyk how do you solved that? I need the same. Reset without animation

@mkpietrzyk how do you solved that? I need the same. Reset without animation

HI! I am sorry I left the team where the solution was implemented last year so I don't remember. My teammate made a proper fix for it next week this question was asked. I think we used componentDidUpdate with proper animation / re-rendering asset, can't tell you the details because I just don't remember :(

Did you guys solve this yet? (For react, not react-native)


Dev mode bug - adding a feature manager block

Adding a feature manager block to the server.xml after dev mode has started results in an error, when dev mode should trigger installing any newly specified features.
Steps to Reproduce

Start dev mode with a server.xml that does not have a featureManager block.
Add a feature manager block, with a feature specified, ie.

    <featureManager>
        <feature>jakartaee-9.0</feature>
    </featureManager>

Notice the error in the dev mode console:

[INFO] Copied file: /Users/kathrynkodama/devex/sampleProjects/vaccination-demo-app/src/main/liberty/config/server.xml to: /private/var/folders/gr/3p093xzj0f9fj9psx80z_zw00000gn/T/tempConfig18101587657383742248/server.xml
[INFO] Parsing the server file /private/var/folders/gr/3p093xzj0f9fj9psx80z_zw00000gn/T/tempConfig18101587657383742248/server.xml for features and includes.
[INFO] Parsing the server file /private/var/folders/gr/3p093xzj0f9fj9psx80z_zw00000gn/T/tempConfig18101587657383742248/configDropins/overrides/liberty-plugin-variable-config.xml for features and includes.
[ERROR] Could not file process changes for /Users/kathrynkodama/devex/sampleProjects/vaccination-demo-app/src/main/liberty/config/server.xml: null

It seems the server.xml is not successfully copied over to the usr/servers directory. The error message is coming from DevUtil and the getServerXmlFeatures method in ServerFeatureUtil
Expected Behaviour
After completing step 2 above, the expected behaviour would be that Liberty installs the corresponding feature.  This would match the behaviour of starting Liberty with liberty:run and adding the featureManager block in the usr/server/serverName/server.xml file.

Closing as this is not reproducible on latest version of the plugins.


Last Low equal to 0 in brazilian tickers (sometimes)

Frequently, the last Low of data is equal to 0 when downloading Brazilian stocks. As it happened again today,  I tested on Nasdaq and it was fine.

partial code
period = '60wk'
interval = '1wk'
ticker_data = {}
for ticker in tickers:
try:
data = yf.download(ticker, period=period, interval=interval, progress=False, auto_adjust=True, timeout=60)
#data = yf.download(ticker, start='2022-01-02', end='2023-05-01', interval=interval, progress=False, show_errors=False)
ticker_data[ticker] = data
except Exception as e:
print(f"An error occurred while processing {ticker}: {e}")

Try setting repair=True in download()

It works! Thank you very much!


What is the Exodus output?

What is the Exodus output?
I know the p2pkh、p2sh address output, but I want to know what is the mean of the  Exodus Output.

Hi @JiangXiaoZhe, initially the Exodus output was used as marker for transactions, so Omni Layer transactions are easily to find, when scanning through a block.


[tv.bell.ca]

Checklist

[x] I'm reporting a new site support request
[x] I've verified that I'm running youtube-dl version 2020.12.26
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that none of provided URLs violate any copyrights
[x] I've searched the bugtracker for similar site support requests including closed ones

Example URLs

Single video: https://tv.bell.ca/watch/vodAsset-0YMOR

Description

The following URLs are likely relevant -
https://tv.bell.ca/api/vod/v3/assets/0YMOR?platforms=tv,nScreen&tvServices=dth&vodProviderMaps=MAP_BELL_NONIPVOD
https://tv.bell.ca/api/vod/v3/assets?max=10&newOnly=true&platforms=tv,nScreen&providers=tva.ca&showTypes[]=Movie&sortBy=dateAdded&tvServices=dth&vodProviderMaps=MAP_BELL_NONIPVOD
https://tv.bell.ca/api/vod/v3/series/tva.ca/17633194-fr?platforms=tv,nScreen&tvServices=dth&vodProviderMaps=MAP_BELL_NONIPVOD
https://vod1.nscreen.iptv.bell.ca/sdash/2B23J01/index.mpd/Manifest?device=OPPE-WV-VOD2
https://vod1.nscreen.iptv.bell.ca/sdash/2B23J01/index.mpd/S!dxIEVP7...8BNvGkW2cNT1BQRS1XSURFVklORXgg4wEVARYBrw__/QualityLevels(617443)/Fragments(video=Init)
https://vod1.nscreen.iptv.bell.ca/sdash/2B23J01/index.mpd/S!dxIEVP7...8BNvGkW2cNT1BQRS1XSURFVklORXgg4wEVARYBrw__/QualityLevels(192000)/Fragments(audio_483_und=Init)
https://vod1.nscreen.iptv.bell.ca/sdash/2B23J01/index.mpd/S!dxIEVP7...8BNvGkW2cNT1BQRS1XSURFVklORXgg4wEVARYBrw__/QualityLevels(192000)/Fragments(audio_483_und=446)
https://vod1.nscreen.iptv.bell.ca/sdash/2B23J01/index.mpd/S!dxIEVP7...8BNvGkW2cNT1BQRS1XSURFVklORXgg4wEVARYBrw__/QualityLevels(617443)/Fragments(video=0)
https://vod1.nscreen.iptv.bell.ca/sdash/2B23J01/index.mpd/S!dxIEVP7...8BNvGkW2cNT1BQRS1XSURFVklORXgg4wEVARYBrw__/QualityLevels(617443)/Fragments(video=30030000)
https://vod1.nscreen.iptv.bell.ca/sdash/2B23J01/index.mpd/S!dxIEVP7...8BNvGkW2cNT1BQRS1XSURFVklORXgg4wEVARYBrw__/QualityLevels(192000)/Fragments(audio_483_und=144830)
https://vod1.nscreen.iptv.bell.ca/sdash/2B23J01/index.mpd/S!dxIEVP7...8BNvGkW2cNT1BQRS1XSURFVklORXgg4wEVARYBrw__/QualityLevels(617443)/Fragments(video=60060000)
I can provide account credentials if needed - is there an e-mail address that I should send them to?

did you try using the cookies option ??


1.8.0

Coverage remained the same at 100.0% when pulling 069e6ec91346f610631530a363ab55a642413bba on 1.8.0 into 8281af05c80e1a154c04fc47738b7be6726f3b3e on master.


go test fails

--- FAIL: TestImageCompositeModeLighter (0.00s)
panic: opengl: glReadPixels: 1282

91491e58f45a2c0500a35404c63163a11e3ee7e5 was the culprit.

Fixed.