rbs files with untyped block type/method type parameters causes error in bin/test

reproduction:

run bin/init_new_gem foobar (version: 0.0)
edit gems/foobar/0.0/foobar.rbs as below

module Foobar
  def foo: (?) -> untyped
end

running bin/test gems/foobar/0.0, raises an error as below:

Testing gems/foobar/0.0...
Fetching gem metadata from https://rubygems.org/.
Resolving dependencies...
Writing lockfile to ~/path/to/gem_rbs_collection/gems/foobar/0.0/_test/Gemfile.lock
Installing foobar:0.0 (~/path/to/gem_rbs_collection/gems/foobar/0.0)
It's done! 1 gems' RBSs now installed.
bundler: failed to load command: rbs (~/.rbenv/versions/3.3.3/bin/rbs)
~/.rbenv/versions/3.3.3/lib/ruby/gems/3.3.0/gems/rbs-3.5.1/lib/rbs/method_type.rb:130:in `with_nonreturn_void?': undefined method `with_nonreturn_void?' for an instance of RBS::Types::UntypedFunction (NoMethodError)

      if type.with_nonreturn_void?
             ^^^^^^^^^^^^^^^^^^^^^
Did you mean?  with_nonreturn_void
        from ~/.rbenv/versions/3.3.3/lib/ruby/gems/3.3.0/gems/rbs-3.5.1/lib/rbs/cli/validate.rb:274:in `void_type_context_validator'
        from ~/.rbenv/versions/3.3.3/lib/ruby/gems/3.3.0/gems/rbs-3.5.1/lib/rbs/cli/validate.rb:155:in `block (4 levels) in validate_class_module_definition'
        from ~/.rbenv/versions/3.3.3/lib/ruby/gems/3.3.0/gems/rbs-3.5.1/lib/rbs/cli/validate.rb:154:in `each'
        from ~/.rbenv/versions/3.3.3/lib/ruby/gems/3.3.0/gems/rbs-3.5.1/lib/rbs/cli/validate.rb:154:in `block (3 levels) in validate_class_module_definition'
        from ~/.rbenv/versions/3.3.3/lib/ruby/gems/3.3.0/gems/rbs-3.5.1/lib/rbs/ast/declarations.rb:14:in `block in each_member'
        from ~/.rbenv/versions/3.3.3/lib/ruby/gems/3.3.0/gems/rbs-3.5.1/lib/rbs/ast/declarations.rb:12:in `each'
        from ~/.rbenv/versions/3.3.3/lib/ruby/gems/3.3.0/gems/rbs-3.5.1/lib/rbs/ast/declarations.rb:12:in `each_member'
        from ~/.rbenv/versions/3.3.3/lib/ruby/gems/3.3.0/gems/rbs-3.5.1/lib/rbs/cli/validate.rb:150:in `block (2 levels) in validate_class_module_definition'
        from ~/.rbenv/versions/3.3.3/lib/ruby/gems/3.3.0/gems/rbs-3.5.1/lib/rbs/cli/validate.rb:149:in `each'
        from ~/.rbenv/versions/3.3.3/lib/ruby/gems/3.3.0/gems/rbs-3.5.1/lib/rbs/cli/validate.rb:149:in `block in validate_class_module_definition'
        from ~/.rbenv/versions/3.3.3/lib/ruby/gems/3.3.0/gems/rbs-3.5.1/lib/rbs/cli/validate.rb:97:in `each'
        from ~/.rbenv/versions/3.3.3/lib/ruby/gems/3.3.0/gems/rbs-3.5.1/lib/rbs/cli/validate.rb:97:in `validate_class_module_definition'
        from ~/.rbenv/versions/3.3.3/lib/ruby/gems/3.3.0/gems/rbs-3.5.1/lib/rbs/cli/validate.rb:84:in `run'
        from ~/.rbenv/versions/3.3.3/lib/ruby/gems/3.3.0/gems/rbs-3.5.1/lib/rbs/cli.rb:447:in `run_validate'
        from ~/.rbenv/versions/3.3.3/lib/ruby/gems/3.3.0/gems/rbs-3.5.1/lib/rbs/cli.rb:140:in `run'
        from ~/.rbenv/versions/3.3.3/lib/ruby/gems/3.3.0/gems/rbs-3.5.1/exe/rbs:7:in `<top (required)>'
        from ~/.rbenv/versions/3.3.3/bin/rbs:25:in `load'
        from ~/.rbenv/versions/3.3.3/bin/rbs:25:in `<top (required)>'
        from ~/.rbenv/versions/3.3.3/lib/ruby/gems/3.3.0/gems/bundler-2.4.13/lib/bundler/cli/exec.rb:58:in `load'
        from ~/.rbenv/versions/3.3.3/lib/ruby/gems/3.3.0/gems/bundler-2.4.13/lib/bundler/cli/exec.rb:58:in `kernel_load'
        from ~/.rbenv/versions/3.3.3/lib/ruby/gems/3.3.0/gems/bundler-2.4.13/lib/bundler/cli/exec.rb:23:in `run'
        from ~/.rbenv/versions/3.3.3/lib/ruby/gems/3.3.0/gems/bundler-2.4.13/lib/bundler/cli.rb:492:in `exec'
        from ~/.rbenv/versions/3.3.3/lib/ruby/gems/3.3.0/gems/bundler-2.4.13/lib/bundler/vendor/thor/lib/thor/command.rb:27:in `run'
        from ~/.rbenv/versions/3.3.3/lib/ruby/gems/3.3.0/gems/bundler-2.4.13/lib/bundler/vendor/thor/lib/thor/invocation.rb:127:in `invoke_command'
        from ~/.rbenv/versions/3.3.3/lib/ruby/gems/3.3.0/gems/bundler-2.4.13/lib/bundler/vendor/thor/lib/thor.rb:392:in `dispatch'
        from ~/.rbenv/versions/3.3.3/lib/ruby/gems/3.3.0/gems/bundler-2.4.13/lib/bundler/cli.rb:34:in `dispatch'
        from ~/.rbenv/versions/3.3.3/lib/ruby/gems/3.3.0/gems/bundler-2.4.13/lib/bundler/vendor/thor/lib/thor/base.rb:485:in `start'
        from ~/.rbenv/versions/3.3.3/lib/ruby/gems/3.3.0/gems/bundler-2.4.13/lib/bundler/cli.rb:28:in `start'
        from ~/.rbenv/versions/3.3.3/lib/ruby/gems/3.3.0/gems/bundler-2.4.13/exe/bundle:45:in `block in <top (required)>'
        from ~/.rbenv/versions/3.3.3/lib/ruby/gems/3.3.0/gems/bundler-2.4.13/lib/bundler/friendly_errors.rb:117:in `with_friendly_errors'
        from ~/.rbenv/versions/3.3.3/lib/ruby/gems/3.3.0/gems/bundler-2.4.13/exe/bundle:33:in `<top (required)>'
        from ~/.rbenv/versions/3.3.3/bin/bundle:25:in `load'
        from ~/.rbenv/versions/3.3.3/bin/bundle:25:in `<main>'
# Type checking files:

.....................................................................................

No type error detected. 🫖
No untyped calls detected. 🐾
 => Failure 🚨

# Failed gems
gems/foobar/0.0

This is a bug of rbs-3.5.1, and it was fixed at 3.5.2 (refs: https://github.com/ruby/rbs/pull/1906)
Please update your packages.


Blazor (ASP.NET Core Hosted) template fails to load when running in VS with IIS Express

Repro steps:

Create a new Blazor (ASP.NET Core Hosted) template in VS
Ctrl-F5 the .Server project with IIS Express

Expect result:

App runs

Actual result:

Static "Loading..." text appears but the app never loads

Workaround:

Run with Kestrel instead of IIS Express

I was able to repro this and investigated. I believe what happens is that, when you have loaded the Blazor extension (VSIX) into VS, our auto-rebuild mechanism is now causing massive problems. Something in VS/IISExpress must have changed such that our previous logic is now wrong.
Our VSIX now wrecks IISExpress so that it can only handle one or two requests before hanging for 5+ seconds, then each subsequent request makes it hang even longer. Since the Blazor app needs to do about 30 HTTP requests to start up, this would take almost forever.
Plus, this doesn’t just affect Blazor – it affects all ASP.NET Core apps once you’ve loaded the Blazor extension into VS.
We should try to fix it, but if this is deeply nontrivial, we could drop the “Blazor Hosted” auto-rebuild feature for 0.9.0. It didn’t work in 0.8.0 either (though caused less disruption).

Closing as this seems to be resolved already.


add new attribute to the resource to handle shell_out timeouts, obvious fix

@arielrolfo this has been resolved by https://github.com/chef-cookbooks/delivery-sugar/commit/f7f33c751401be51316a51fef2db31b6a0907cbb


⚠ ClamAV has degraded performance

In fe82007, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 533 ms

Resolved: ClamAV performance has improved in a04f419 after 11 minutes.


Adding single contributor

Trying this again.  Hopefully it works better than last time!

@cluhmann
High FIVE!  It worked!!! Yay!


New field type - ranked set + time

Description
Hi Everyone!
Wonder if field type that combines functionality of ranked set field and time field could be implemented?
So that it would become possible to do TopN query but filter it based on time windows? E.g. in query language example in addition to “TopN(stargazer, Row(language=1), n=2)” that can tell me top 2 users that have most stars in language 1 from beginning of time - It could give answer to similar question, what are top 2 users that put most start in language 1 during the last month?
This would be really useful for us since our data always has time dimension attached to it and the main goal of the whole system we are designing is to be able to provide answers to historical questions about what happened with this binary matrix over time, compare state 2 months ago to 1 month ago and so on - and right now we are writing POC that would use just time type fields for everything and we would do queries like  Count(Intersect(Range(<field A row, type=time>, <last month timestamps>), Range(<field B row, type=time>, <last month timestamps>))) for every single combination of field A and field B values we have and then stitch results together in application code, i.e. seems like we would kind of simulate functionality very similar to TopN in application code - but probably it would be miles less performant than what TopN could potentially show, wouldn't it?
Success criteria (What criteria will consider this ticket closeable?)
When we would be able to do TopN for particular time range fast

Hi @dmibor thanks for submitting an issue.
Time fields are implementing by maintaining a separate "view" for every quantum of time that exists. So if the field is configured to "YM", then it would contain views for 2016, 2017... 2016-01, 2016-02... etc. In addition to these views, a "standard" view is also maintained which contains any bit which has been set regardless of the associated timestamp.
Based on some internal discussion, it should be fairly straightforward to enable a ranked cache on the standard view which would then support TopN queries with no change to the query language. Because TopN queries can take an arbitrary bitmap as a filter, it would be possible to do something like TopN(sometimefield, Range(sometimefield=5, 2016-02-05, 2016-04-05)) This would give you the top rows in sometimefield sorted by number of bits set in columns where row 5 has a set bit between February 5th and April 5th in 2016.
If you wanted to do TopN for a specific time range without a filter, you'd either have to (a) create a filter which is a union of all rows in sometimefield during that time range, or (b) maintain a separate row which gets set whenever any row gets a set bit in sometimefield - this would basically mean always having the Union from option (a) precomputed.
So in your example, with the proposed change, it would be possible to TopN(stargazer, Range(language=1, 2016-02-01, 2016-04-01), n=2) assuming that stargazer and language are time fields. Keep in mind that the counts returned are the number of columns that have a set bit anywhere in that time range for that row. If a bit is set in the same row/col multiple times within the time range that will not be reflected in the count.
Does that sound like it would be helpful?

What I meant by (b) was that you would set an additional bit for every bit set in the time field. E.G. if you are doing Set(timefield=2, 2016-04-05) you would also do Set(trackfield=0, 2016-04-05). So regardless of what row is being set in timefield, you set trackfield=0 with the same timestamp. A Range query on trackfield=0 then returns exactly the same as Unioning all rows of timefield in the same time range.
Interestingly, if we made the existence field a time field I think that could the same purpose, though that would add a significant amount of overhead.

Got it. That makes more sense regarding (b). So the suggestion is to maintain an existence row in each time-quantum view of a time field. (I don't think it would be wise to use the index's existence field because that wouldn't be field specific and multiple time fields in the same index would cause problems).

Hey, @jaffee, @travisturner ! Thank you for your replies, this discussion is definitely giving us way to move forward!

Does that sound like it would be helpful?

Idea definitely is! Reading your reply it occurred to me that I can achieve the same thing as enabling ranked cache to standard view even now by using ranked set and time field together in sync (setting bit in both fields at the same time same way as you propose in (b) - when doing Set(timefield=2, 2016-04-05) I would also do Set(rankedfield=2) ). Which would give the the opportunity to do the same query like that - TopN(rankedfield, Range(timefield=5, 2016-02-05, 2016-04-05)). So enabling ranked cache on standard view would not give additional functionality as it seems to me, looks more like a hack that can help save space of one standard view, not sure it's worth adding additional complexity...

If you wanted to do TopN for a specific time range without a filter, you'd either have to (a) create a filter which is a union of all rows in sometimefield during that time range, or (b) maintain a separate row which gets set whenever any row gets a set bit in sometimefield - this would basically mean always having the Union from option (a) precomputed.

Yeah that is exactly what I wanted, thank you for this idea @jaffee ! It's not too useful to intersect with specific row in this query TopN(sometimefield, Range(sometimefield=5, 2016-02-05, 2016-04-05)) but (b) will definitely do the trick
On the other hand this approach has quite important limitations in case if my cacheSize is significantly less than number of rows in the ranked field - in that case I can not trust this query to catch and return bursts, anomalies, since they most likely would not be present in ranked cache. E.g. in stargazer example if user behavior is such that it never present in ranked cache - is generally on average not active and stars almost no repos - but suddenly in particular time window starts to be very active, I might or might not see this depending on how active he is and on how big of a time window im looking at. And it so happens that this info might be extremely valuable, at least in our case.
Initially I was thinking about something like @travisturner mentioned TopN(sometimefield, 2016-02-05, 2016-04-05, n=5)) but I imagine that would require to have ranked cache for each view of time field that would solve above limitation but probably take a lot of space and add additional complexity  and need to merge results from multiple caches?

Based on some internal discussion, it should be fairly straightforward to enable a ranked cache on the standard view which would then support TopN queries with no change to the query language. Because TopN queries can take an arbitrary bitmap as a filter, it would be possible to do something like TopN(sometimefield, Range(sometimefield=5, 2016-02-05, 2016-04-05)) This would give you the top rows in sometimefield sorted by number of bits set in columns where row 5 has a set bit between February 5th and April 5th in 2016.

Btw, wanted to clarify if I'am correct to say that it would give me top rows in sometimefield sorted by number of bits set in standard view but with counts equal to number of bits set in row 5 between February 5th and April 5th in 2016? I.e. sort order will be according to standard view , not according to row=5?

Adding the ability to have a ranked cache on the standard view of a time field should be a pretty easy change, so I think that at least is worth the complexity. The follow-on question is whether we want to explicitly support TopN over a time range without having to pass a filter query - more on this below.
A cache size of ~200k is totally feasible, though it might slow down queries which are forced to traverse the long tail of the cache - this is something you'll have to experiment with on your workload and see if it's ok.
Adding a ranked cache on each view of the time field is an interesting idea - I'd have to think about the consequences of that a bit more. Many smaller caches would probably be more performant than one large one, and merging the results together doesn't seem like it would be too expensive - we already have code that does essentially this for merging the results of TopN from different shards. IMO it's mostly a question of added complexity both on the API side and in the code. I'll be interested to hear others' thoughts on this.
Summary:

definitely add the ability to have a ranked cache on the standard view of a time field.
consider supporting the passing of a time range to TopN directly - I see two options for doing this:
(a) maintain or calculate a single-row time range field which is the union of all rows in the target field and pass it as a filter to the TopN query.
(b) add a ranked cache to every view of the time field - execute the TopN on the views which are in the time range and merge the results.

I need to think more about 2a vs 2b to convince myself that they are fundamentally equivalent and quantify the tradeoffs a bit.

I think if you assume that each row/column combination has only one timestamp associated with it, then 2a and 2b are equivalent as long as the cache sizes of the standard view (for 2a) and the time views (for 2b) can accommodate enough of the row set to support the query.
2b is a bit more complex due to the re-queries it has to do (similar to the way we currently re-query shards for TopN), but I think—depending on the timeQuantum for the field—it might allow for smaller caches.
With that said, it's not obvious to me which would be more performant. I think it would depend on the data and query patterns.

@dmibor I think you're right, although your notation is unclear to me.
The result of the TopN query you posted would be a list of each row id in sometimefield along with a count which is the intersection of all the columns set in that row (regardless of time) with all the columns set in any row within the time range. So if, for example, row 7 had every column set during the time range, then the query would act like a normal TopN query which is not what we're looking for.
Having a ranked cache on the standard view still might be useful, but not for doing TopN queries on a time range - for that I think we will need to add a ranked cache on every view.

@dmibor I think you're right, although your notation is unclear to me.

Sorry about that, diverged from Pilosa notations. Was not too successful attempt to shorten and be more visually clear:
a*b => Intersect(a,b)
a+b => Union(a,b)

And also did this transformation:
Intersect(a,Union(b,c)) = Union( Intersect(a,b), Intersect(a,c) )


Remove unused startPos argument from Parser, coding style improvements, new tests

I looked at the test suite's code coverage and found it's quite good, around 95% depending on the metric. However, I found that the startPos argument to Parser is unused. I also stumbled across weird conditionals: two where this.unexpected() was called in a ternery and one with a break after an this.raise() call. And I added test cases for non-covered code paths.

What do you mean by 'unused'? Not used by the test suite? Parser is exported, so client code could be using that, right?

You're right, src/index.js re-exports Parser, of course.

All right. I've merged the remaining patches. Thanks!

Thanks. That's why I open pull requests for about everything.


Fix issue with confidence rating not showing correctly

When you added the "Confidence" label in the tooltip, you missed one, and you also need to append "settings." to correctly grab the string token. I fixed this so that the label appears correctly in select and non-select modes.

Thanks! Looks good to me :)


Rename Color_4ub

Currently the name is pseudo-hungarian notation. It would be better to represent it as the order of bytes, specifically when dealing with low-level interfaces like DirectX where order matters. Another class could be created to convert to/from these required formats.
Suggest renaming the current implementation to Rgba. Other classes for specific formats (e.g. Brga) can be created along with an explicit conversion constructors if needed.

For some reason, I was under the impression there was both a uint8_t variant of this, and a float variant of this. That doesn't seem to be the case.
I'd be tempted to just rename this to Color and be done with it. The code involved here is not low level enough to really care about color component ordering at the hardware level. We can leave that distinction to low level code where it really matters. From the high level interface to the library, one consistent format would likely make for an easier to use API.
My vote is to just rename to Color.

I'm okay with renaming this to simply Color. At the time I named this I was working with OpenGL and working with the function glColor4ub()... hence the 4ub.
Anyway, I'm in favor of making it simply Color and dropping the post-fixed implementation.


Added Bitfinex, Winkdex and OKCoin API

Hi,
I like Bitfinex the most, therefore I added an API handler for it to the application.
Hope, I did everything as you like it!
BTW: I really like the app! :)
Edit: Arg!
Annndd also OKCoin...

Sorry for the late response - I get thousands of notifications on GitHub and they've become totally useless now. Thanks for the contribution! I'll get a build out asap.

Thanks :)

This issue will go into #8 since I have a couple other tweaks to make before I build.


Make kernel procedure table inclusions part of virtual table builder

Describe your changes
Minor refactoring to stay consistent with the docs. Fixes some minor typos as well.
Checklist before requesting a review

Repo forked and branch created from next according to naming convention.
Commit messages and codestyle follow conventions.
Relevant issues are linked in the PR description.
Tests added for new functionality.
Documentation/comments updated according to changes.

@Al-Kindi-0 what is the status of this? Should we merge?

If you could review it that would be great. Nothing big in this PR except for some re-shuffling and fixing some consistency issues, should be mergeable as is.


chore: fix some function names in comment

Description
fix some function names in comment
Type of change
Please delete options that are not relevant.

[ ] Docs

How has this been tested?
Please describe the tests that you ran or implemented to verify your changes. Provide instructions so we can reproduce.

[ ] Test A
[ ] Test B

How has this been benchmarked?
Please describe the benchmarks that you ran to verify your changes.

[ ] Benchmark A, on Macbook pro M1, 32GB RAM
[ ] Benchmark B, on x86 Intel xxx, 16GB RAM

Checklist:

[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I did not modify files generated from templates
[ ] golangci-lint does not output errors locally
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for the contribution - looks good to me! I'll wait for the CI and then merge to fix/typos-v0.15 branch. We collect smaller documentation fixes into a single branch which we merge before next release. This avoids possible merge conflicts when someone else is working on the branches.


Pipelines continue to be seen in OSIO - even after they have been deleted in OpenShift

See the attached screenshots - the pipelines in question were deleted through the OpenShift console, but they continued to be seen in the OSIO UI - even after that UI was refreshed:

@jstrachan did you say you already had an issue for this?

fixed in the latest tenant. Now deleting a Pipeline removes the project name from the Jenkins Github Org job


Explicitly include episode to avoid hitting ActiveRecord's cache so much

Since @episode is loaded by cancan you can't eager load its comments up-front. You do this in the partial, but for some reasone whenever you're accessing comment.episode it's hitting ActiveRecord's cache. This fixes it.

+1 Goncalo! I was in the audience today when you submitted this pull request during your talk. Well done.


Add FR district scraper

since FR is only providing the last three weeks, add "incremental" data support to run_district_scraper.sh  (not perfect, but it should work). Also note that this will change the order of the file contents between the first and every upcoming run, which shouldn't be a big issue I suppose.

Please wait with merging. The appending of data doesn't work, when data is changed (date/week identical, but for example deceased changes).

7d8d082 should fix this issue (hopefully :sweat_smile: ).

@maekke maybe we should use the same mechanism for appending and updating data like the other scraper (using sqlite as an intermediate file format). WDYT?

@metaodi this would probably be way better and more robust...

okay, I'll work on that.


Loot Table Collision

When you equip the Backpack into the "accessories" slot, it causes all mobs to stop dropping items and XP. When the Backpack is removed from the "accessories" slot, items and XP begins to drop once again.

Hey sry for the late response, i couldn't find any reason for this issue. Resource Backpacks isn't adding any loottable only item and entity Tags so that Accessories/Trinkets/Curios API can add them to the correct Slots.

with version 0.9.0-BETA all compatibility with any Trinket API is removed, so there should be no longer an issue.


How to use auto router?

I have page project in pages like this :
home--index.vue
--banner.vue
--about-me.vue
news--index.vue
--news-detail.vue
so, how to config when i run 'npm run dev' ,i can look my home--index.vue page first? Thanks

Rename home--index.vue to index.vue


Outdated extension notice even with auto update enabled

Issue Type: Bug
Auto Update Extensions is set to All Extensions. I have an "outdated extension" marker on my extensions icon but have not seen any extension in the list that looks special or highlighted in any way. This persisted even with reloading and restarting VS Code. I remember @sana-ajani sharing a similar issue before, being very confused by the sudden ⓵ on the extension icon and not being able to figure out what it points to.

10min later … after some more digging I found clicking the menu item Check for Extension Updates, does trigger a search for @outdated and finally pointed me to Pylance:

VS Code version: Code - Insiders 1.65.0-insider (0283cec6ec2c2d5ec39006a1048b8dc3ea8bc955, 2022-02-08T05:22:23.206Z)
OS version: Darwin x64 21.3.0
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz (16 x 2400)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
3, 3, 4

Memory (System)
64.00GB (1.96GB free)

Process Argv
--enable-proposed-api donjayamanne.kusto --log trace --crash-reporter-id 4cf0a6f0-aaf8-4da3-9b2b-81f6c639c12b

Screen Reader
no

VM
0%

Extensions (52)

Extension
Author (truncated)
Version

Bookmarks
ale
13.2.2

tsl-problem-matcher
amo
0.5.0

emojisense
bie
0.9.0

github-markdown-preview
bie
0.1.0

markdown-checkbox
bie
0.3.2

markdown-emoji
bie
0.2.1

markdown-preview-github-styles
bie
1.0.1

markdown-yaml-preamble
bie
0.1.0

vscode-tailwindcss
bra
0.7.6

npm-intellisense
chr
1.4.0

vscode-github-actions
csc
0.22.0

postcss
css
1.0.9

vscode-eslint
dba
2.2.2

kusto
don
0.3.8

typescript-notebook
don
2.0.6

EditorConfig
Edi
0.16.4

vscode-npm-script
eg2
0.3.24

prettier-vscode
esb
9.2.0

auto-close-tag
for
0.5.14

auto-rename-tag
for
0.1.10

codespaces
Git
1.5.2

copilot
Git
1.7.4924

remotehub
Git
0.25.2022020815

vscode-pull-request-github
Git
0.37.2022020809

restore-git-branch-tabs
gko
0.4.1

headwind
hey
1.7.0

gzipdecompressor
Hyu
1.2.0

theme-monokai-pro-vscode
mon
1.1.20

vscode-docker
ms-
1.19.0

vscode-edge-devtools
ms-
1.4.3

python
ms-
2022.1.1779589815-dev

vscode-pylance
ms-
2022.1.6-pre.1

jupyter
ms-
2022.2.1001811271

jupyter-keymap
ms-
1.0.0

jupyter-renderers
ms-
1.0.6

remote-containers
ms-
0.219.0

remote-ssh
ms-
0.73.2022020715

remote-ssh-edit
ms-
0.73.2022020715

azure-account
ms-
0.9.11

cpptools
ms-
1.8.4

live-server
ms-
0.2.12

vsliveshare
ms-
1.0.5330

debugger-for-edge
msj
1.0.15

java
red
1.3.0

vscode-commons
red
0.0.6

vscode-xml
red
0.18.3

smart-css-autocomplete
sai
1.5.3

mdx
sil
0.1.0

msft-todo-unofficial
tyl
0.0.17

glean
wix
5.2.2

markdown-all-in-one
yzh
3.4.0

vscode-proto3
zxh
0.5.5

A/B Experiments
vsliv695:30137379
vsins829:30139715
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyl392:30422396
pythontb:30258533
pythonvspyt551cf:30291413
pythonptprofiler:30281269
vshan820:30294714
pythondataviewer:30285072
vscod805:30301674
pythonvspyt200:30323110
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
pythonvs932:30404738
wslgetstarted:30421357
vs360:30404995
vsclayoutctrt:30405800
pythonvsnew555:30426298
vscscmwlcmc:30428972
azactmsal:30432848
vscgsvidc:30433759

I have the same issue and this is frustrating me a lot.

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


Some doubts about the absolute value of ViCLIP similarity

Thanks for such beautiful work！
In the past, the similarity between video and text was usually calculating the similarity between each frame and text using text-image CLIP, and then take the average. If the text and video are aligned, the value calculated in this way is usually above 0.9.
However, The value calculated using ViCLIP is only about 0.3.
Could you explain the reason?

Yeah I want to ask this too, not sure how should we threshold out good matches given that cosine similarities are almost always lie between 0.2 -0.4.  I can see that softmax(100* score) is used to get relative closeness among a set of candidates but this doesn't help to exclude unmatched candidates .

Hi folks, I'm encountering the same issue and am just wondering did you guys figure out this or found a way to threshold out irrelevant candidates?
Apologize for the direct mentions @shepnerd  @leexinhao but I think once this is clear, the InternVideo models would definitely get far more adaptions from the community so wondering can you guys can help us on this, thanks in advance!


Don't set up lspconfig clients if conflicting plugins are installed

Is your feature request related to a problem? Please describe.
Hey 👋
I sometimes have users opening issues/discussions because mason-lspconfig sets up lspconfig configurations that conflict with my plugins (haskell-tools.nvim, rustaceanvim), when they use mason to install a language server.
Often (usually when using a distro), they aren't aware that mason-lspconfig is installed.
Describe the solution you'd like
Would it be feasible for mason-lspconfig.nvim to check for the presence of certain plugins (e.g. by searching the runtimepath or with a pcall('require', ...)), and basing the default behaviour on the presence of those plugins?
Describe potential alternatives you've considered
N/A
Additional context
Example: https://github.com/mrcjkb/rustaceanvim/discussions/94

After looking at the codebase, it looks to me like mason-lspconfig doesn't actually call the lspconfig configurations' setup functions automatically (@williamboman can you confirm this?).
I believe it's distros like LazyVim and AstroNVim that are adding this behaviour.


[BUG] Maximum call stack size exceeded while removing node

Describe the bug

Fail to remove node (both worker and witness) due to the warning message Maximum call stack size exceeded shown in the attached snapshot. Rebooting the management node doesn't work. Also tried to do the same node removal from different client (different machine of different IP) and different browsers (Safari and Google Chrome), doesn't work, either.
To Reproduce
Steps to reproduce the behavior:

Install harvester on a machine to play as either a worker or witness node (with the same static IP and management url)
Go to Host page of Harvester UI and then remove the node which is already detected and healthy check OK.
Repeat step 1 with the same network setting (or the same harvester config)
Remove the same node as step 2 until the warning message shows up. (~5 times on my testbed)

Expected behavior
Node removal should work w/o VM running and not been used by other resource
Environment

Harvester ISO version: 1.3.0

I'm unable to reproduce this issue within 3 times.
Steps

setup 1 management node + 1 worker node using ipxe environment.
delete worker node (harvester-node-1) on Host page
add worker node back
delete worker node on UI again

Repeat 2 ~ 4 step for 3 times are all successfully.
I may need to try more times (5+ times) to see if it's reproducable.

But for error Maxmium call stack size exceed, we can find any function recursively call itself and didn't reach the termination condition.   cc @torchiaf

@torchiaf We can close the issue if it can't reproduce.

I'm experincing exactly this problem now, is there a work around? @bk201 I'm now stuck at etcd removed but node not ready not removed state.

Now the situation becomes:


theme builder theme

Credit BYU for suggesting the idea last year
A theme that forces HAXeditor to have definitions it otherwise doesn't. These definitions are for the building blocks of theme construction itself.
site-rss #22
as 1 example, an rss button which would have the correct visualization and link to the correct type of source. This throttles the type (via HAX schema) and then you get the button anywhere you want in your design.  Let's silly though...
site-view #115
Borrowing from the Drupal views concept this would allow you to manipulate the manifest JSON Outline Schema object and bend it to your will. Passing an element into it would allow you to wire up how to visualize it. We'd need a bit expanded logic in HAXeditor to allow for this level of re-wiring abstraction but it's possible. Writing a JSON blob via a json-editor could be done for the input of what to select and how to select it. Then items could be rendered in any fasion in any piece of the UI
site-menu 👍
A menu / list of everything associated in the JSON Outline Schema / manifest data object. Contains all state associated with it and so the inputs could involve just simple slices of the data-set (like start-level="2" depth="1" to only show items that are 2 deeper children of the active item, but only 1 level down from there)
site-query
Allot for remixing and passing data through a templated area
<site-query sort='{"order": "ASC"}' conditions='{"id": "$activeId"}'>
        <template>

[[item.order]]. [[item.title]]

        </template>
      </site-query>

This works.... 😲
Bring it.

it's about as far a long as HAXeditor can handle currently. Until we fix some more state and grid plate issues in the editor it's not worth fleshing this out further.


Populate the sidebar with related videos

On YouTube videos that are blocked, the sidebar for suggested videos is blank. The software should populate this sidebar with videos that are related to the blocked video.

F*ck this. We don't care about this anymore.


feat: Update nri-bundle to version 5.0.66

Update nri-bundle to version 5.0.66

@Pjv93 @mikemcd3912 Please review and merge this


China: optimizations

Coverage remained the same at 100.0% when pulling b1a223ac7022a02ead4ed75bfc82e4fd7183b79c on KJhellico:optim-china into a772f79327a64639ec1fe89c79165d585c3a9b3d on dr-prodigy:beta.


"docs" command not recognized

Describe the bug
When I run dbt docs in terminal, that's what I get:
~/Documents/Code/growth-data/dbt $ dbt docs
usage: dbt [-h] [--version] [-r RECORD_TIMING_INFO] [-d]
           [--log-format {text,json,default}] [--no-write-json] [-S]
           [--warn-error] [--partial-parse | --no-partial-parse]
           {docs,source,init,clean,debug,deps,list,ls,snapshot,rpc,run,compile,test,seed,run-operation}
           ...

An ELT tool for managing your SQL transformations and data models. For more
documentation on these commands, visit: docs.getdbt.com

optional arguments:
  -h, --help            show this help message and exit
  --version             Show version information
  -r RECORD_TIMING_INFO, --record-timing-info RECORD_TIMING_INFO
                        When this option is passed, dbt will output low-level
                        timing stats to the specified file. Example:
                        `--record-timing-info output.profile`
  -d, --debug           Display debug logging during dbt execution. Useful for
                        debugging and making bug reports.
  --log-format {text,json,default}
                        Specify the log format, overriding the command's
                        default.
  --no-write-json       If set, skip writing the manifest and run_results.json
                        files to disk
  -S, --strict          Run schema validations at runtime. This will surface
                        bugs in dbt, but may incur a performance penalty.
  --warn-error          If dbt would normally warn, instead raise an
                        exception. Examples include --models that selects
                        nothing, deprecations, configurations with no
                        associated models, invalid test configurations, and
                        missing sources/refs in tests.
  --partial-parse       Allow for partial parsing by looking for and writing
                        to a pickle file in the target directory. This
                        overrides the user configuration file. WARNING: This
                        can result in unexpected behavior if you use
                        env_var()!
  --no-partial-parse    Disallow partial parsing. This overrides the user
                        configuration file.

Available sub-commands:
  {docs,source,init,clean,debug,deps,list,ls,snapshot,rpc,run,compile,test,seed,run-operation}
    docs                Generate or serve the documentation website for your
                        project.
    source              Manage your project's sources
    init                Initialize a new DBT project.
    clean               Delete all folders in the clean-targets list (usually
                        the dbt_modules and target directories.)
    debug               Show some helpful information about dbt for debugging.
                        Not to be confused with the --debug option which
                        increases verbosity.
    deps                Pull the most recent version of the dependencies
                        listed in packages.yml
    list (ls)           List the resources in your project
    snapshot            Execute snapshots defined in your project
    rpc                 Start a json-rpc server
    run                 Compile SQL and execute against the current target
                        database.
    compile             Generates executable SQL from source model, test, and
                        analysis files. Compiled SQL files are written to the
                        target/ directory.
    test                Runs tests on data in deployed models. Run this after
                        `dbt run`
    seed                Load data from csv files into your data warehouse.
    run-operation       Run the named macro with any supplied arguments.

Specify one of these sub-commands and you can find more help from there.

Everything else is working well (models, tests etc.)
Steps To Reproduce
Run dbt docs command
Expected behavior
Documentation should be generated and served
System information
Which database are you using dbt with?

[x] postgres

The output of dbt --version:
installed version: 0.16.1
   latest version: 0.16.1

Up to date!

The operating system you're using: OSX 10.15.4
The output of python --version: Python 3.7.7

Hi @patrykkalinowski, dbt docs requires a sub-command, generate or serve:
$ dbt docs --help
usage: dbt docs [-h] {generate,serve} ...

optional arguments:
  -h, --help        show this help message and exit

Available sub-commands:
  {generate,serve}

Are you seeing an error when you run dbt docs generate or dbt docs serve?

Thanks @jtcohen6 , it worked.


/root/komodoGUI/src/build/build.xml:136: JAVA_HOME has to be set!

Hi,
i try to build ant that this is the error i'm getting:
/root/komodoGUI/src/build/build.xml:136: JAVA_HOME has to be set!

like stated in the error ouput you have to set your java home path:
export JAVA_HOME=YOUR_JAVA_HOME_PATH


Update Onedrive to version 17.3.6916

After making all changes to the cask:

[x] brew cask audit onedrive --download is error-free.
[x] brew cask style --fix Casks/onedrive.rb reports no offenses.
[x] The commit message includes the cask’s name and version.

Additionally, if updating a cask:

[ ] [If the sha256 changed but the version didn’t][version-checksum],
provide public confirmation by the developer: {{link}}

Thank you for the contribution, @brianveltman.


is cordova truly integrated with react?

I'm trying out to integrate cordova and react, and the issue i'm running into is that, this react app is compiled beforehand and then handed to cordova. Which means using plugins like the device plugin is not possible? device.platform in any react component gives me undefined and breaks the build.
How am I supposed to use Cordova's power to talk to the device from my react components?

It's not a question about react and cordova integration, it's more about cordova + web development.
When you run the app as a website (using npm start), it's much faster and easier to develop than building the application each time to make a change.
A way to solve this is to integrate Live reloading / Hot model replacement  to work also on the app (which is a nice feature I started working on).
Another way to solve this (and it's the way I'm doing right now on my own app) is to mock cordova's behavior.
What does it mean? let me explain with an example.
On cordova you have device.uuid, on the web we will get an exception (because device is undefined).
To mock that behavior I will create another uuid on the web and use global __CORDOVA__:
function guid() {
  function s4() {
    return Math.floor((1 + Math.random()) * 0x10000)
      .toString(16)
      .substring(1);
  }
  return s4() + s4() + '-' + s4() + '-' + s4() + '-' +
    s4() + '-' + s4() + s4() + s4();
}
var uuid = __CORDOVA__ ? device.uuid : guid();
// And continue doing whatever we intended to do with uuid...
$.ajax('users', {id: uuid});
...

As for performance and junk code, minification solves that for you.
In cordova's build __CORDOVA__ is replaced with true and will change this row var uuid = true ? device.uuid : guid(); to this var uuid = device.uuid and will also remove the function guid because it's no longer used anywhere.
This way to can maintain your web app working for easy development and showing/ testing features on different browsers on the computer while making sure it does what you intended.

Thanks this approach is quite reasonable indeed!

Sure :+1:


Rename string format functions

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (9c72995) 100.00% compared to head (3370d86) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #277   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          319       319
  Lines         9897      9897
  Branches       952       952
=========================================
  Hits          9897      9897

Impacted Files
Coverage Δ

src/gravitorch/creators/dataloader/pytorch.py
100.00% <100.00%> (ø)

src/gravitorch/creators/datapipe/sequential.py
100.00% <100.00%> (ø)

src/gravitorch/creators/model/compiled.py
100.00% <100.00%> (ø)

src/gravitorch/creators/model/ddp.py
100.00% <100.00%> (ø)

src/gravitorch/creators/optimizer/zero.py
100.00% <100.00%> (ø)

src/gravitorch/datasources/datapipe.py
100.00% <100.00%> (ø)

src/gravitorch/datasources/dataset.py
100.00% <100.00%> (ø)

src/gravitorch/distributed/utils.py
100.00% <100.00%> (ø)

src/gravitorch/events/manager.py
100.00% <100.00%> (ø)

src/gravitorch/loops/observers/sequential.py
100.00% <100.00%> (ø)

... and 15 more

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Include 3D PCA of genotypes as QC result

Hi,
Do you think it makes sense to include a 3D PCA of genotype results in the final QC output of each bcbio analysis?
For example the code below could be used. But also Plink or another piece of code more efficient than R could be used to craete the PCA matrix.
The plotly 3D pca plot would like this, and would show sample sames up on hovering above each dot.
https://plot.ly/~ptocca/19/_3d-pca/
This would for example be useful to detect sample swaps.  The 3D PCA could maybe go into the multiqc report. Or be a separate QC html output.
Thank you.
#Create a minimal VCF file, faster to parse.
bcftools annotate -x  FMT input.vcf.gz  -O v -o input_minimal.vcf.gz -za

# R code
library(gdsfmt)
library(SNPRelate)
library(plotly)

# Convert VCF to GDS
snpgdsVCF2GDS("input_minimal.vcf.gz", "input_minimal.gds")

# print summary of the creatd GDS file
snpgdsSummary("input_minimal.gds")

#open the GDS file
genofile <- openfn.gds("input_minimal.gds")

# Create the PCA table
sample.id <- read.gdsn(index.gdsn(genofile, "sample.id"))
pop_code <- read.gdsn(index.gdsn(genofile, "sample.id"))
RV <- snpgdsPCA(genofile, autosome.only=FALSE, num.thread=4)
pca<- data.frame(sample.id = sample.id, pop = factor(pop_code),EV1 = RV$eigenvect[,1],  EV2 = RV$eigenvect[,2],  EV3 = RV$eigenvect[,3],   stringsAsFactors = FALSE)

# Plot the 3D PCA
labels <- paste0(pca$sample.id)

p <- plot_ly(x = pca$EV1, y = pca$EV2, z = pca$EV3, text = labels)
p

Neill;
Thanks for the suggestion, this is a great idea and it would be really useful to include. My preferred way to include this would be directly within MultiQC which would involve generating outputs that we could push into a MultiQC plot, rather than including another output file/format. Is that something that seems do-able from your perspective?
A similar idea we've been discussing is predicting ethnicity of human sample inputs using reference panels, which would help flag swaps if you know ethnicities or flag outliers as due to being under-represented in the standard reference.
Thanks again for starting this discussion.

Hi Brad. I would also prefer to have the 3D PCA in the MultiQC report if possible.
I think the PCA matrix can be created with both Plink of SNPRelate in R.  I don't know if the plotyly output display of the 3D PCA matrix can easilly be included in MultiQC.  Maybe that would be a question for the MultiQC developers.
I am btw fine with any tool to create the PCA matrix and to  visualize the 3D PCA. Which ever is easier to get the data into the mutliQC report. It would be extra nice if there was an option do dynamic zoom in and out in PCA display, to be able to zoom in to dense clustered together samples.
Predicting the species and the strain would be very useful for us.
For example Rattus norvegicus +  ACI .  Or Rattus rattus + M520.
Sometimes very closely related individuals are sequences, so I am not sure this always helps.
i.e. all samples are  Rattus rattus + M520, or  Rattus rattus + M520 +  Rattus rattus + M521 where there is almost no genotypic difference plus M520 and M521.  In the last case it would also be difficult to predict a distinct M520  or M521 strain label based on the genotypes.
I am also wondering how to computationally predict the species + strain, and how computational difficult this is.
For sure in some cases this would be very helpful.


remove duplicate test

This is basically the same as tests/pass/imported_main.rs.

@bors r+

:pushpin: Commit 7cd87299f2ec58a11cf013a6e6fce05509e8f09b has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit 7cd87299f2ec58a11cf013a6e6fce05509e8f09b with merge 5162995c73e3732d011429914ecfebce27d301b4...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing 5162995c73e3732d011429914ecfebce27d301b4 to master...


Add connector to Vertex AI API models

Please can you provide few documentation regarding the same and the gemini url that is being used.

Hi @patilpriyadarshini, thank you for raising this issue! We are working on this right now and will keep you updated. Let's use this ticket as the source of truth and I will close this other issue as it is a duplicate

@patilpriyadarshini you can find the instructions to connect to Google Gemini using in this notebook. We will deploy the updated docs by the end of this week, please let us know if this resolves your issue!
https://github.com/BCG-X-Official/artkit/blob/1.0.x/sphinx/source/user_guide/introduction_to_artkit/connecting_to_genai_models.ipynb

Can we connect to GeminiChat using Vertex AI API?

@patilpriyadarshini unfortunately we don't have a direct integration with the Vertex AI API yet, but could you please elaborate what functionality you want to use in GeminiChat that is not supported using the ARTKIT wrapper? Thank you!

In current case GeminiChat uses GenerativeModel from google.generativeai which is general model for all generative AI tasks so there is huge difference between the response of LLM from google.generativeai as compared with vertexai.generative_models. So wanted to check if artkit provides any direct integration with Vertex AI API.

I see, thank you for the details! @patilpriyadarshini at this moment ARTKIT doesn't have integration with the Vertex AI API, so here's what I recommend:
let's rename this issue to be an enhancement "Add connector to Vertex AI API models".I will discuss with the team and update this issue if/when we create this connector. We also highly encourage contributors to contribute in our open source repository!

@patilpriyadarshini I am actively working on a vertex API connector now, can you please confirm if this vertex AI generative_models is the most helpful for you? Please let me know if there are other packages such as language_models that would be helpful. Thank you! https://cloud.google.com/vertex-ai/generative-ai/docs/reference/python/latest/vertexai.generative_models

@seanggani Thank you for taking up this as an enhancement. Yes the vertex AI connection would be really helpful because the response from these models is more promising when tested.

@patilpriyadarshini Can you please review this PR and check if this works for your use case? Thank you!

The latest Vertex AI integration has been released with ARTKIT 1.0.7, so I'm closing this issue


Suggestion for issue template

This is more of a suggestion than an issue. While browsing the issues  I realised a lot of users submit the default content of the issue  template without cleaning up the post.
This could be avoided by using comment tags in the default template  which hides text that is only useful to the user submitting the issue.
A minor adjustment that will drastically improve readability for the collaborators.
As an example I include my suggestion in a code block and as a regular post so you can see the difference:

[x] Are you using the latest driver?

[x] Are you using the latest EVDI version?

[x] If you are using a DisplayLink device, have you checked 'troubleshooting'
on DisplayLink's website?

[x] Is this issue related to evdi/kernel?

Linux distribution and its version :
Linux kernel version:
Xorg version (if used):
Desktop environment in use:

<!-- Initially please make sure the problem hasn't been already addressed: -->
  * [ ] Are you using the latest driver?
<!--  (https://www.displaylink.com/downloads/ubuntu) -->
  * [ ] Are you using the latest EVDI version?
<!--  (https://github.com/DisplayLink/evdi/releases) -->
  * [ ] If you are using a DisplayLink device, have you checked 'troubleshooting'
    on DisplayLink's website?
<!--     (https://support.displaylink.com/knowledgebase/topics/103927-troubleshooting-ubuntu) -->
  * [ ] Is this issue related to evdi/kernel?
 <!--    (if it is rather connected to DisplayLinkManager please take a look at support
    https://support.displaylink.com or forum https://www.displaylink.org/forum/) -->

<!--
Some information for baseline would help a lot: -->
  * Linux distribution and its version :
  * Linux kernel version:
  * Xorg version (if used):
  * Desktop environment in use:

<!-- Please provide good title for the issue and description.
If you can give us logs they will be welcome but please don't paste long listings
as they will make it hard to read the thread, instead you can use other services
for sharing raw text like pastebin or gist.
Also anything that may help to understand the problem will be appreciated
(configuration files, screenshots, photos, video capturing the problem).

Thanks!
-->

Hi @Barabazs,
thanks for the suggestion. I've been thinking about improving the template.
It's great to see that others also think so ;)

Very constructive comment indeed @Barabazs!  Thank you!

Hi @Barabazs,
after a while the updated version is finally live. Thanks for great suggestion :+1:
Closing :smile:


Allow user to specify vagrantfile and included files.

Added ability to process :vagrantfile and :include_files defined in definition.rb
For example:
# work/definitions/boxname/definition.rb
Veewee::Session.declare({
  :vagrantfile => File.expand_path('../Vagrantfile.pkg', __FILE__),
  :include_files => 'file1' # or ['file1', 'file2']
})

where Vagrantfile.pkg is in work/definitions/boxname/ and file1 is in work/.

Closing this, since it's for 0.2.x.  Will open new pr with changes applied to new master.


DSA Guide Book

Practice Questions 🎯
Covered Concepts 💯
Videos and Notes ✅
Solutions 🔥
and much more...

I don't see any reference to page, which you have raised this Issue?
Can you elaborate on the same?

I have a DSA Guide book .can I upload that?

You can upload the book if it follows the guidelines.
If you're familiar with Git, please submit a PR. If you're not familiar with Git, provide the links you're specifying, and make sure to read docs/CONTRIBUTING.md before contributing.


3.0.2 - Unexpected Results in Camera transforms not present in 3.0.1 (C#)

Godot version:
3.0.2 mono
OS/device including version:
GTX 1050 (tested also on GTX 950M)
Issue description:
The Camera/PlayerMovement scripts I wrote in 3.0.1 moves in an unexpected manner in 3.0.2. The exact same project produces different results. I tested in 3.0 (Windows) also and it works as it should, so either this is a regression or a change that I didn't find in documentation. I have tested this on Windows, MacOS High Sierra, and Linux (Ubuntu 17.10) builds (of 3.0.1 and 3.0.2, respectively) before submitting the bug and the problem persists across platforms.
The camera rotates oddly, and the orientation of the player movement becomes unpredictable as movement is based on camera.transform.basis
The reproduction project is 25MB (as low as I could get it to trim without rewriting it) so I will provide a GoogleDrive link of the whole project and the ZIP attached just includes the CS scripts.
The scene is setup like this:

Steps to reproduce:
Attach PlayerMovement.cs to a Kinematic Body. Add a camera as a child of KinematicBody and attach the FollowCamera.cs script
3.0.1:

3.0.2:

Minimal reproduction project:
https://drive.google.com/file/d/16L-DgzgNm6x0laPMDS_HNvpNyshT56OX/

Maybe the same issue as #16937?

I think so. I'll build the latest master and give it a go. Thank you!

@nazgulsenpai Just to make it clear, if you want to confirm whether or not it's caused by the same issue as mine, you'll probably need to try either the official 3.0.2 build or build dad47d8876de45132226985dbbd079e060f6facc rather than the latest master branch.

The initial set of patches for those functions were removed from 3.0.2 because they caused unrelated problems. If it behaves differently from 3.0 then it appears that I've not reverted all relevant parts of these fixes. I'll have a look.

The bug report says that it behaves differently from 3.0.1, which is the version where the behaviour was deemed unsatisfactory and thus reverted back to the 3.0 state in 3.0.2.
So this change is expected, and the 3.0.2 behaviour is (currently) the expected one.

Except the initial report here states 3.0.0 also had the behaviour that they expected.

Yeah, as we currently understand it, if something works 'correctly' in 3.0.1 but not in 3.0.2, then it shouldn't work as expected in 3.0 either.
So, if we can reproduce the issue with 3.0.2 but not with dad47d8 or 3.0, then probably we better check if we have rolled back all the releant changes correctly as @hpvb mentioned.
But if we find that the example project only works correctly in 3.0.1, then probably the supposed 'correct' behavior is in fact the opposite, so we can safely close this issue as a duplicate of #16937.

I'm sorry - the bug DOES exist in 3.0-stable_mono_win64. I don't know how I managed to confuse myself but I have.
So to clear it up, the problem existed in 3.0.0 and 3.0.2.
I just tested with a fresh pull/build from master branch (about 5 hours ago) and it appears the transform issue is fixed, but IDK where to even start here.

Ok, great. Thanks for the heads up and I'll test as soon as I get to the office.
Thank you guys so much for all of your assistance


Disabling SSO manually

Hello, I would like to recommend adding a manual PowerShell method of disabling SSO on a tenant on this article. Thank you.

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

ID: f4fed1fc-4d87-410a-9d19-91af4f1174e3
Version Independent ID: 7731db50-d667-6062-e29b-520430194204
Content: Azure Active Directory Connect: Troubleshoot Seamless Single Sign-On
Content Source: articles/active-directory/connect/active-directory-aadconnect-troubleshoot-sso.md
Service: active-directory
GitHub Login: @swkrish
Microsoft Alias: billmath

@josuvil Please post this as a feedback @ https://feedback.azure.com/forums/169401-azure-active-directory This will allow the community to upvote and for the product team to include into their plans. We also have the "Give Product Feedback" button in our documents now to take you directly to the appropriate feedback page.

@josuvil We will now proceed to close this thread. If there are further questions regarding this matter, please open a new issue and we will gladly continue the discussion.


Symfony process 4 compatibility

This has been implemented in the 2.0 branch of assetic-php/assetic. Until we tag 2.0, you can pull it in by using "assetic/framework": "dev-2.0/dev" in your composer.json files.

@LukeTowers that dev tag doesn't exist

Probably because 2.0 and 3.0 have been tagged since then 😜

Probably because 2.0 and 3.0 have been tagged since then 😜

where?

@exts you're looking at the wrong repo.
https://github.com/assetic-php/assetic/releases

@LukeTowers interesting, that repo isn't on packagist. So I guess I'll force use that repo instead.

@exts I'm not exactly sure where the confusion is coming from, it is available on packagist.
See https://packagist.org/packages/assetic/framework
Does that help?
You should just be able to run composer require assetic/framework

Yeah that's less confusing, that one would not show up when i searched.


⚠ GLDDNS Server (ns2.glddns.com) has degraded performance

In 6c30944, GLDDNS Server (ns2.glddns.com) (ns2.glddns.com) experienced degraded performance:

HTTP code: 200
Response time: 671 ms

Resolved: GLDDNS Server (ns2.glddns.com) performance has improved in 9975994 after 6 minutes.


Hints do not work

Oh I didn't notice, looks like I forgot to add a hook for the help admin txt
thanks for reporting this

this commit fixes the issue https://github.com/SychO9/smf-fa-board-icons/commit/bf461597979c247cca49e940ca4e3dbee4b97137


Register Visual Studio DIA SDK shared libraries

OpenJ9 uses the ddrgen tool from OMR; that tool depends on msdia100.dll which must be registered. This change arranges that both the 32-bit and 64-bit variants are registered as part of installing Visual Studio 2010.

Is there something I can do to help get this merged?

Sorry @keithc-ca a number of folks are promoting adoptopenjdk at a conference this week.  Just looking now.

Figured we can marge this version and continue the discussion in parallel.


fga doesn't select support castoria

Preparation

[X] I tested the latest release
[X] I looked at other issues (even the closed ones)
[X] I read the Troubleshooting Guide

FGO server
NA
FGA build number
1577
Describe the bug
the support picker continuously scrolls past numerous castoria's on support, doesn't choose any other support either, just keeps scrolling
Video
https://youtu.be/Dp4UOFfUAvc
file size too big, here's the youtube link
Device model
Galaxy A21s
Android version
12
Screen size
6.5 inches, 720x1600
RAM
6gb

Got the same issue, even used the support maker to try to get another castoria image, also didn't work

Turn off max ascended in preferred support

Same issue. Turning off max ascend dosent fix it

Same issue. Also tried using the support maker to try to get another castoria image, didn't work.

With max ascended off don't add a ce to preferred ce selection and use the ingame filters to filter out non event ce

With max ascended off don't add a ce to preferred ce selection and use the ingame filters to filter out non event ce

I tried this but it didn't work. OPPO A73

You're 200 versions behind the current one. Update FGA.


[Bug] label disappear in pie chart

Version
latest
Link to Minimal Reproduction
https://codesandbox.io/s/label-disappear-v253w6?file=/src/index.ts
Steps to Reproduce
none
Current Behavior
bottom label disappear

add config coverEnable: true, label  appear

Expected Behavior
label  appear when coverEnable: false,
Environment
- OS:
- Browser:
- Framework:

Any additional comments?
No response

默认的布局逻辑是这样的，默认是会计算两个半圆弧来布局标签，cover enable 开启之后就全都按照整圆弧来布局，所以是这样的效果.
The default layout logic is like this. By default, two semi-arcs are calculated to lay out the labels. After the cover enable is turned on, they are all laid out according to the full arc, so this is the effect.

It will be fixed in 1.3.0, @pairone please add this case to bugserver


fix(typescript): resolve js to ts

:tada: This PR is included in version 2.1.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add DynamoDB to Maven Central

I'm excited about using TitanDB backed by Dynamo, however I'm unable to get the project up and running. When running maven install, DynamoDB local is unable to be found.
No versions available for com.amazonaws:DynamoDBLocal:jar:[1.10, 2.0.0) within specified range
Can we expect DynamoDB local to be added to maven central anytime soon?

We are unable to add DynamoDB Local to Maven Central as it is a Binary-only release. Instead, we host our own Maven repository where we serve out DynamoDB Local. The relevant lines in the pom.xml are here.
Please make sure that you run mvn install before trying to start DynamoDB Local when you follow the instructions of the README.md.

OK to close?

Closing.


lnrpc: Add payment hash to SendResponse

When SendPayment grpc call fails, there is currently no way of associating the failure with a payment.
Payment hash is already extracted by extractPaymentIntent, so it can just be added to SendResponse.
Did not add hash to successful sends, since they have the preimage anyway.

ah, thanks @ccdle12 🙈
any idea what I can do about the timeout in TestLightningWallet/btcwallet/neutrino:reorg_wallet_balance (14.68s)` ? Can't see my change affecting the test.

@gitlikeagirl that is a known issue with our integration tests, I wouldn't worry about it for now.

There is one more stream.send. The success case. It does contain the preimage, so the hash can be computed. For consistency I'd think it is better to always supply the hash, as it is the main key to identify a payment.

Needs a rebase!

LGTM after rebase!

Ready to land after a rebase, will have to synchronize with any other PRs that also modify the protos.

Space added and rebase done. Just not sure how to address the merge conflict in rpc.pb.go, is it ok to leave as is?


Replace the design resources section of the readme with information about recreating native components

Given that the design resources officially provided by Apple, as well as the components in the Figma kit, do not represent native components in true accuracy (meaning that details about their behavior are missing, as well as some details of their design), it is therefore misleading to provide these resources. Instead, we should provide information about how to create native components for comparison with their Flutter widget counterparts and truly achieve parity.
This would be a good place to link the native-components project repo.

Update: Apple has provided a Sonoma Figma kit, so we should include it on the readme.


WIP: Deprecation Framework

@jacek-lewandowski this is ready for another look whenever you have a sec

Waiting on your approval @jacek-lewandowski  :) I'll get the master 2.0 patch ready

Build against Scala 2.11 finished with success
1230 tests run, 0 skipped, 0 failed.
Build against Scala 2.11 finished with success

Build against Scala 2.11 finished with success

Build against Scala 2.10 finished with success
1230 tests run, 0 skipped, 0 failed.
Build against Scala 2.10 finished with success

Build against Scala 2.10 finished with success

Build against Scala 2.10 finished with success
1230 tests run, 0 skipped, 0 failed.
Build against Scala 2.10 finished with success

Build against Scala 2.10 finished with success

Build against Scala 2.11 finished with success
1230 tests run, 0 skipped, 0 failed.
Build against Scala 2.11 finished with success

Build against Scala 2.11 finished with success


svelte-package --preserve-output

Describe the problem
Running svelte-package will rimraf(output) (https://github.com/sveltejs/kit/blob/a3ef25f0cf7d5c5896581fb8701fe0daf8109e6a/packages/package/src/index.js#L44) which means that all operations must be run after running svelte-package
Describe the proposed solution
Add a --preserve-output flag (or similarly named) that skips removing the output directory
This is especially useful in --watch mode when other operations can happen concurrently
Alternatives considered
No response
Importance
would make my life easier
Additional Information
No response

If the idea is accepted, I would love to contribute the feature


Reload redis scripts on reconnect

this is basically the same as #143, but for WorkerPoolImpl

Coverage decreased (-0.5%) to 70.22% when pulling c3927ea59858e633907f54f48ff3394d077b79ea on uberall:reload-scripts-on-reconnect into c0e56a10e7cd5724574adb6c0728d3098b6e8d2e on gresrun:master.

@gresrun any idea when this fix will be available in maven.


Allow marking multiple tasks as completed by holding SHIFT #14

Added Feature "Allow marking multiple tasks as completed by holding SHIFT".

Hi @jatin00000 !
It's not working for me for some reason.

In the description, what is "last" means, like it should be the current last task or any between first and the last?

What sequence of action should code follow
Like shift Hold -> mouse click on first -> mouse click on last?
my current code work on following above course of action.


Update content.json

Added link to johnyorke.me

Fixed up the Feed URL and merged. If there's a better feed URL then please submit another PR with an update to it.
Thanks!


Wording: "All extensions will be enabled" is not accurate

Testing #122252
nitpick, not all extensions will be enabled, globally disabled ones won't.

"Extensions will work without limited functionality"?


Translation

/gpt-translate content/en/home.md content/ja/home.md Japanese

/gpt-translate content/en/.md content/ja/.md Japanese

/gpt-translate content/en/courses//*.md content/ja/courses//*.md Japanese


Runfile() changes the current tab in the editor

Issue Report Checklist

[x] Searched the issues page for similar reports
[x] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
First of all, I guess this is more of an inconvenience than a real problem. Since Spyder 4.0.0b7, calling runfile("someFile.py") in the iPython console produces the following behavior :

If someFile.py is not open in the editor, just execute runfile
If someFile.py is open in the editor, execute runfile and switch to the corresponding tab

Sometimes you need to run / re-run some main file many times while working on another, and having to switch back from the former to the latter each time you execute runfile (or press F6) is quite annoying.
It seems that this behavior was not present in Spyder 4.0.0b6.
What steps reproduce the problem?

Open two files a.py and b.py in the editor
Go to tab of b.py and execute runfile("a.py") (or press F6 after having run a.py beforehand)

Versions

Spyder version: Spyder 4.0.0rc1
Python version: Python 3.6.9 64-bit
Qt version: Qt 5.9.6
PyQt version: PyQt5 5.9.2
Operating System name/version:  Ubuntu 18.04  /  Linux 4.15.0-66-generic

@impact27, could you take a look at this one?

How about runcell? Should runcell switch to the tab? I think it makes sense for runcell at least.

How about runcell? Should runcell switch to the tab? I think it makes sense for runcell at least.

I could see that someone might make a cell in a different file that processes or displays data from the current file kind of like a using a function. If people are running things from the console it might be because they don't want the editor to change.

If people are running things from the console it might be because they don't want the editor to change.

Agreed, I updated the PR.


Document difference to Zappa

Just found this in my newsfeed. Is there a reason I would use this over e.g. https://github.com/Miserlou/Zappa (or at least over the parts of Zappa that deal with translation to WSGI)?

zappa appears to be an entire suite of deployment tools and other things and comes with a hefty list of dependencies (this library also has zero dependencies) -- it also appears to be django specific and won't work with other web frameworks.
here's zappa's ~equivalent module to what this library does: https://github.com/Miserlou/Zappa/blob/master/zappa/wsgi.py
it also appears they don't support api gateway v2 (only v1) (https://github.com/Miserlou/Zappa/issues/2099)


Two Charts, same Page Not working.

Expected Behavior
Both Charts Should be visible.
Current Behavior
Only the second chart is visible.
Possible Solution
n/a
Steps to Reproduce
Example here: https://jsfiddle.net/qz9kfmLt/6/
Context
I'm trying to draw multiple charts on page.
Environment
Chrome/Firefox latest

Looks like the problem seems to be with the dynamic inserting of: document.body.innerHTML += '
Chart1
<canvas id="myChart"></canvas>'.

You are right, the problem you have has nothing to do with Chart.js.
https://jsfiddle.net/ywmd0b6o/
You should be using https://developer.mozilla.org/en-US/docs/Web/API/Document/createElement when appending elements.

Seems like a race codition of some sort is ahpening, if you make the canvases first it will work https://jsfiddle.net/Leelenaleee/sdrjanLv/2/

Yes, thanks!
Inserting the canvas elements first in a loop, then drawing the charts works.


Generation of definitions (response classes) of API operations marked a shidden

[ ] What version of the library are you using?

Version 2.9.2
What kind of issue is this?

[ ] Bug report.

I am supposed to create a limited swagger output, to keep some API undocumented. I have marked the RestController methods with @ApiOperation(hidden=true) but their response object classes are still returned in the list of definitions. I expect that only such definitions would be listed that are used in any public request path.

I had a similar situation with a parameter that I wanted to hide and I could solve it by using @ApiIgnore


Support selectors dict concatenation

Allow concatenating dictionaries with select of dictionaries. Currently, there is no way to do this. Note: This supports <dict> + <select> but does not support <dict>.update(<select>)

Abandoning this, re-though this and there is a workaround for the specific use case we are trying to address


select & exclude configs are not working as expected

I am using select property in the DbtDag. I was assuming models tagged with daily should only be rendered in the task group.
However the DAG does not render anything i.e. empty DAG with no tasks.  Similarly with exclude tag, all the models are appearing in the DAG including the tagged model with exclusion value.
from cosmos.providers.dbt import DbtDag

    my_dag = DbtDag(
        # ...
        select={"configs": ['tag:daily']},
    )

config file: schema.yml
version: 2
models:
  - name: table_1
    description: "A starter dbt model"
    config:
      tags: ['daily']

Thanks @jlaneve  for your response.
I did a quick test and it works like a charm :)


Should we add tags for expected effort/time for completion?

Like:

short time(minutes)
medium time(single digit hours)
long time(everything above)

might make it easier to find a task, that you can do with the time you have at that moment. We could also abuse milestones for that, it's just grouping after all.

I second the idea, sounds like a good idea, especially for the people still learning, as it allows to quickly sort off tickets of greater difficulty.

I'm not sure it's a good idea. Have to major points for that, here you go:

Everyone has it's own pace. I could say it's an easy task that could be done in one an hour, but for a newbie or programmer with less experience, it would take day or more. Would be better to add tags with level of complexity (easy, normal, hard or any other), that would give a better picture over tasks in general for those who is just start digging in and for all other.
Second thing, we're trying to make this script faster, better, stronger, and with unicorns (just joking)!!! Jokes aside, there should be a better code, better UI and etc, not number of closed issues ;) We're not in a rush - you know. Only critical or blocker tasks could be you number one priority and time consuming :)

regarding 1:
Obviously the pace differs, similar to difficulties this would be a hint to look closer if you are guessing at a completely different amount of effort needed. I don't really care if we label them one or the other, the effect should be similar.
regarding 2:
I was thinking more about being able to finish something, before there is an expected lack of time - vacation for example. I couldn't care less how fast tickets are solved(if they aren't bugs).
What it was meant to prevent is having a prerequisite just waiting for a month or two because of something forseeable ;)

Well, a general complexity tag, considering as well time and difficulty ( read into code, change code level), should do the job. Though combining it with a rough time tag cold be nice, too. Yes, people are on different levels in different areas, but I'd say one can roughly estimate the time cost. Combined with some kind of difficulty or complexity information it should be appropriate and informative.

so what about:
Low Effort expected
Medium Effort expected
High Effort expected
would that work for everyone?

Would be fine for me. In addition to the current category tags it shouold offer a decent overview to the required effort.

Sounds good!
I'm not a big fan of Agile methodology, you might get it already :)

There's a huge difference between what Agile was meant to be and what it is used as - but well, I'm not going to introduce code sprints in any case :P

WaterAgile - is one of the best practises, in my previous company :D Best from waterfall and agile => wateragile monster was born :) Using his super powers he destroyed my brain cells!

I think we're done with this one :)


Finished basic pretty-printer. Need to fix indentation

Check it out. I think there's an easy way to add indentation without too many changes so I'm gonna spend a half hour on it real quick.

Ok, I'll let you do indentation and then lets merge it. Might be worth while to take a look at https://github.com/cucapra/dahlia/blob/master/src/main/scala/backends/Document.scala.


Pull request app firewall timedate

This series of commits adds the UCI time and date restriction for firewall rules and redirects to LuCI.

Yay, had this on my to-do list for ages, good to see that you beat me to it.
A couple of things:

translate() should only be called with string literals, otherwise it is impossible to extract original strings from the source code, also it imho makes no sense to translate 1..31
We need a JavaScript counterpart for the new validation types (see cbi.js)
The time stuff adds a lot of new options to the rule details so maybe we should start using tabs, e.g.

General: name, enabled, protocol, action
ICMP: icmp types [will be hidden if proto!=icmp]
Source: src, src_ip, src_port, src_mac
Destination: dest, dest_ip, dest_port
Timing: all the date and time options

What do you think? I can also merge the PR without the tabbing (but with JS validation added) and then take a stab at the tabs myself.

I will try to get to it this weekend but I've got a major cold that is making me miserable since Tuesday.

I haven't finished but I did notice that the commit seems to have missed the actual UI page.

nvm, it was when I went to fix it that I missed the second commit

Ok, I've update based on your comments;
I've also fixed a couple of bugs with the implementation and added the option of using and option using self.oneline = true with a multivalue checkbox which makes a checkbox be space-separated instead of each checkbox item being on a separate line.  This makes the days of week and days of month selection much more compact (and iMO usable).
I'm not sure the tabbing is actually necessay and might actually be more of a pain when it comes to actually creating firewall rules from an data-entry standpoint.

Sorry I'm review what actually got in the commit and it looks like I lost cbi.js.
Will fix.

Ok, the cbi.js is there but the datatypes.lua isn't but I have to play around with github to make sure that's not just an me misreading github's presentation of the branch.

Ok, it's actually correct, I was just misreading github.

I've reviewed this changeset because of the number dumb mistakes with changing the upload dir pull request - this changeset is correct.

Apparently I was wrong - I had managed to pull in my version of ipaddrport which isn't needed for this pull request and was better implemented in the l2tp pull request already merged.
That is fixed now.

I had forgotten to push a fix for a javascript syntax error that wasn't immediately obvious until I noticed that dynamic lists weren't working and pulled out the the debugger.  Apparently if there is a syntax error in cbi.js it's not as obvious as lua errors.

Merged, thanks!


getDoc and getSummary don't work on servers

The getDoc and getSummary helpers don't work on a server object. If I want to get the description for a server, I have to call server.description. I'm curious why the handling for a Type vs a Server is different, and I'd prefer to have them united. Thanks

@iscai-msft Can you provide a playground that shows what you are trying to get the doc from?

https://microsoft.github.io/typespec/playground?c=aW1wb3J0ICJAdHlwZXNwZWMvaHR0cCI7Cgp1c2luZyBUeXBlU3BlYy5IdHRwOwoKQHNlcnZlcigKICAie2VuZHBvaW50fSIsxBBUZXN0xh8gyBrFGXsKICAgIC8qKsUIICogTmVlZCB0byBiZSBzZXQgYXMgJ8R2Oi8vbG9jYWxob3N0OjMwMDAnIGluIGNsaWVudC7HPC%2FFCMhfOiB1cmzEY30KKQpuYW1lc3BhY2UgU%2BUAhS5WZXJzaW9ucy5Ob3THDGVk5ADEaGVhZCBvcCBnZXQoKTogdm9pZDs%3D&e=%40typespec%2Fopenapi3&options={}

@iscai-msft It looks like this is picked up by the swagger emitter:

Sorry I misspoke, I meant the openapi emitter for typespec gen in the example I shared. i'll keep looking into my issue

@iscai-msft It actually does if you include the @service decorator, which is required for emitting anything from http.


Feature/fieldlist netcdf

Adds to_points and to_proj methods to the NetCDFField class.

Hi James, I noticed you have ifas.nc in docs, it is a huge file (22MB) and should not be stored in the repo. I created an emohawk space on get.ecmwf.int  https://get.ecmwf.int/#browse/browse:test-data:emohawk and put all the larger files there, and they can be downloaded on demand.

Hi @sandorkertesz, thanks for the good point! I have uploaded the offending netCDF to the nexus repo and removed it from the GitHub repo - I've also added a method for retrieving remote data for testing purposes.


re-evalulate Illinous rapidvent

They may have made progress.

https://rapidvent.grainger.illinois.edu/


npm owner ls return 'admin party!'

I'm trying to set an owner to our instance of Sinopia (taken from this repo). and every time I try npm owner ls it returns admin party!. Any ideas on how to add an owner or list owners of specific packages?

Related https://github.com/rlidwka/sinopia/issues/386

According this https://github.com/npm/npm/blob/1284a4f335e585f07889b01d33f980b12b4ae461/lib/owner.js#L125
The origin of that messages comes from npm 3.10.10
 var owners = data.maintainers
      if (!owners || !owners.length) {
        msg = 'admin party!'
      } else {
        msg = owners.map(function (o) {
          return o.name + ' <' + o.email + '>'
        }).join('\n')
      }
      output(msg)
      cb(er, owners)
    })

I found out, the data received from Verdaccio never delivers data.maintainers and that's the origin of the bug.

npm doesn't include the list of maintainers if the auth is token based when you publish a package as you can see here
https://github.com/npm/npm-registry-client/blob/master/lib/publish.js#L76
  if (!auth.token) {
    root.maintainers = [{ name: auth.username, email: auth.email }]
    data.maintainers = JSON.parse(JSON.stringify(root.maintainers))
  }

That information never reaches verdaccio thus by default if the maintainers field is missing npm returns admin party!.  I've noticed npm never sends that information not even on the main npm registry if the auth is token based. It seems that data is handled internally, so, this issue rather sounds as a enhancement than a bug.


How can one define a new layer that have multiple inputs?

I want to define a new layer that have multiple inputs.
Each input has a different meaning and shape. How can I implement this layer using Keras?

Use the functional API.

Define a custom layer where the call method accepts a list of tensors (and may return a list of tensors, or just a single tensor).
Use it like:

c, d = layer([a, b])

Note that the get_output_shape_for method should also accept a list of shape tuples.

Thank you. Can you give an example for a layer with a call method that accepts a list of tensors?

@benjaminklein Look at the source. Here for example.

@carlthome Thank you.

I have the same problem.
I defined everything you described, but i get an error using it.
for example (very simple):
lets say that the input is just one layer, and the "call" function returns "input". it works
say we accept multiple inputs, and we set the "call" function to return "input[0]". NOT WORKING
(get_output_shape_for is set accordingly)
Any ideas?

@benjaminklein, if your function doesn't contain any parameter to be learned from data. Here is an example, just put all the inputs into a list.
def concat_test(input):
a = input[0]
b = input[1]
return K.concatenate([a, b], axis=1)
var_1 = Input(shape=(1,))
var_2 = Input(shape=(2,))
var = Lambda(concat_test, name='concat_test')([var_1, var_2])
But if your function has some parameters or weights needed to be trained, you'd better write your own layer. Since I am not very familiar with how to writing own layer, so I just give you the advice. Hope it helpful.

define a self layer which doesn't include trainable parameters
class MyLayer(Layer):
# initialize the layer, and set an extra parameter axis. No need to include inputs parameter!
def __init__(self, axis, **kwargs):
    self.axis = axis
    self.result = None
    super(MyLayer, self).__init__(**kwargs)

# first use build function to define parameters, Creates the layer weights.
# input_shape will automatic collect input shapes to build layer
def build(self, input_shape):
    print(input_shape)
    super(MyLayer, self).build(input_shape)

# This is where the layer's logic lives. In this example, I just concat two tensors.
def call(self, inputs, **kwargs):
    a = inputs[0]
    b = inputs[1]
    self.result = K.concatenate([a, b], axis=self.axis)
    return self.result

# return output shape
def compute_output_shape(self, input_shape):
    return K.int_shape(self.result)

var_1 = Input(shape=(1,))
var_2 = Input(shape=(2,))
var_test = MyLayer(axis=1)([var_1, var_2])
model = Model(inputs=[var_1, var_2], outputs=[var_test])
model.compile(optimizer='adam', loss='mse')
print(model.summary())


Dirty Property Is ALWAYS Based on Original InitialValues

I've created a really basic form and set the initialVales in the reduxForm config.  The dirty flag works fine (and can be checked by clicking the PROPS button.  I can edit the fields and if the lastName matches the initialValues the dirty flag is false.  However, after clicking the REINIT button the form is dirty.  Clicking reset, sets the last name back to Dave, however clicking props shows the initialValues is still the original LLCoolJones value.  It appears that the dirty flag is ALWAYS calculated against the initialValue set in the reduxForm config.  If I manually enter LLCoolJones in the lastName field after a Reinit, the dirty flag becomes false.  I took my approach directly from the "Initialize From State" example.  I'm using method #2 and using props.initialize to update the initialValues, and while the form values do change the dirty/pristine flag is not correct.  I would assume that dirty would ALWAYS be false after a reset (unless setting keepDirtyOnReinitialize to true).
If I remove the initialValues from the reduxForm config.  Then both REINIT and REINIT2 buttons work fine and the dirty prop is ALWAYS correct.  Which is consistent with what appears to be the initialValues prop used for determining dirty/pristine but if it's undefined then it seems to properly check against the reinitialized values.
To make matter stranger, if I uncomment the enableReinitialize config parameter, then dirty is ALWAYS false no matter how I change the form fields (using initialize or manually entering values).
Any ideas / help GREATLY appreciated.
class LoginForm extends Component {

  render() {
    return (
      <form name="loginForm" className="form">

          <Field name="firstName" type="text" component={RenderField} label="First Name"/>
          <Field name="lastName" type="text" component={RenderField} label="Last Name"/>

          <button type="button" onClick={() => console.log("FormProps", this.props) } className="btn form__submit-btn">PROPS</button>
          <button type="button" onClick={() => { console.log("ReInitializing"); this.props.initialize({ lastName: "Dave"} ) }} className="btn form__submit-btn">REINIT</button>
          <button type="button" onClick={() => { console.log("ReInitializing"); this.props.initialize({ lastName: "Yowza"} ) }} className="btn form__submit-btn">REINIT2</button>
          <button type="button" onClick={() => { console.log("ReSetting"); this.props.reset(); }} className="btn form__submit-btn">RESET</button>

      </form>
    );
  }
}

LoginForm = reduxForm({
    form: 'loginForm',              // <------ same form name
    initialValues: { 'lastName': 'LLCoolJones' },
    // enableReinitialize: true,
})(LoginForm);

Seems related to #2149

Also #2156

Seems related to #2427 as well.  The common thread is the initalValues on the form are not updated when this.props.initialize is called.

Because this issue saw almost no activity for a few months, I'm closing it.


drop unsupported_reason_add

part of:

https://github.com/ManageIQ/manageiq/pull/22898

Deprecating unsupported_reason_add. Returning the string instead
Note, these are all the same:
unsupported_reason_add(:feature, unsupported_reason(:feature2)) unless supports_feature2?
unsupported_reason_add(:feature, unsupported_reason(:feature2)) unless supports?(:feature2)
unsupported_reason(:feature2) unless supports?(:feature2)
unsupported_reason(:feature2)

update:

added 2 more deletes for unsupported_reason_add
fixed unless ems.supports?(feature) ; _("reason") ; end and replaced w/ standard ems.unsupported_reason(feature) instead


net/http: considering supporting CIDR notation in no_proxy env variable

Feature request - It would be helpful if the net/http client supported CIDR notation in the no_proxy environment variable, or allowed another way to configure the net/http client to ignore the http_proxy for large networks (ie <IP_ADDRESS>/8 etc).
I had no luck finding an official "spec" for the no_proxy environment variable, but I believe supporting this notation would cause no issues and would allow users of net/http to add proxy exclusions for large networks.
Python requests library implements this feature with the no_proxy env variable - https://github.com/kennethreitz/requests/blob/master/requests/utils.py#L569
and most browsers allow some sort of CIDR notation / wildcards for proxy exclusions.
During my searching, I came across an old issue https://github.com/golang/go/issues/2158 where the functionality was discussed but not really addressed (as far as the CIDR notation goes)

What version of Go are you using (go version)?
go version go1.6.3 linux/amd64
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
What did you do?
Tried to use CIDR notation in the no_proxy env variable to exclude net/http client from using web proxy specified in http_proxy env variable for private network <IP_ADDRESS>/8
What did you expect to see?
Expected http client library to ignore http_proxy for addresses in the <IP_ADDRESS>/8 network.
What did you see instead?
net/http client tried to use the http_proxy for get/post requests to addresses in <IP_ADDRESS>/8 network.

Cheers, and thanks for golang!

I'd rather not extend the functionality of ProxyFromEnvironment, especially when there's no spec for it.
You can implement net/http.Transport.Proxy yourself (https://golang.org/pkg/net/http/#Transport) for specific behavior.

@bradfitz  afaics the entire use of NO_PROXY and proxy env vars is effectively convention, and afaics.. the issue with transport construction, is that as go based software becomes more popular its not even an option for the users to set that up themselves, their dependent on std lib behavior. ie. here's a recent debug session i had where curl and python respect it but a go based binary (etcdctl) does not. this makes quite a bit painful for users of go software, and i'm seeing it filed multiple times on software written in go (particularly docker/moby).
$ export NO_PROXY=s3.amazonaws.com,localhost,<IP_ADDRESS>,<IP_ADDRESS>,<IP_ADDRESS>/8"

curl works with it
$ curl -X GET http://<IP_ADDRESS>:6666/v2/members
{"members":[{"id":"ce2a822cea30bfca","name":"calico","peerURLs":["http://localhost:2380","http://localhost:7001"],"clientURLs":["http://<IP_ADDRESS>:6666"]}]}

as does python
$ python
Python 2.7.12 (default, Nov 19 2016, 06:48:10)
[GCC 5.4.0 20160609] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import requests
>>> requests.get('http://<IP_ADDRESS>:6666/v2/members').json()
{u'members': [{u'clientURLs': [u'http://<IP_ADDRESS>:6666'], u'id': u'ce2a822cea30bfca', u'peerURLs': [u'http://localhost:2380', u'http://localhost:7001'], u'name': u'calico'}]}

as does ruby
$ irb
irb(main):001:0> require 'net/http'
=> true
irb(main):002:0> uri = URI('http://<IP_ADDRESS>:6666/v2/members')
=> #<URI::HTTP http://<IP_ADDRESS>:6666/v2/members>
irb(main):003:0> Net::HTTP.get(uri)
=> "{\"members\":[{\"id\":\"ce2a822cea30bfca\",\"name\":\"calico\",\"peerURLs\":[\"http://localhost:2380\",\"http://localhost:7001\"],\"clientURLs\":[\"http://<IP_ADDRESS>:6666\"]}]}\n"
irb(main):004:0>

but go based binaries don't
$ ./etcdctl --peers http://<IP_ADDRESS>:6666/ cluster-health
cluster may be unhealthy: failed to list members
Error:  unexpected status code 407

@kapilt, okay, I'll reopen, but I don't plan to work on this myself.

For this to move forward, sometime should try to first write a "spec" for it, documenting which other languages/libraries already implement said spec. (But if languages Foo and Bar both just use libcurl or whatever, say that.)

Looking at implementations across other languages and browsers, a full implementation of a no_proxy spec can get complicated quite quickly, and there are arguments about what is right and what is wrong.
I vote to keep it simple with domain, ip and CIDR exclusions. This is what python requests library does. My need for no_proxy in corporate and government environments is just excluding domains and local subnets (<IP_ADDRESS>/8 etc).
Ports, URL schemes, IP wildcards (10.*.3.*), and the special keywords like <local> I believe should be avoided.
Following simple rules:

Environment variable must be no_proxy or NO_PROXY
Should contain a comma-separated list of domains, ip addresses, or networks.
Parent domain will exclude parent and all sub domains (ie google.com would exclude google.com and code.google.com)
Domains in the following form: google.com, .google.com and *.google.com are all equivalent
If sub domain is provided, parent domain(s) is not matched (ie code.google.com would not exclude google.com)
IP addresses (ie <IP_ADDRESS>, <IP_ADDRESS>) (IPv6?)
Networks in CIDR notation (ie <IP_ADDRESS>/16) (IPv6?)
Both localhost and <IP_ADDRESS> are excluded by default
All other notations are ignored

References
python requests library (very simple - domain, ip, CIDR check)
https://github.com/requests/requests/blob/master/requests/utils.py#L629
https://github.com/requests/requests/blob/master/requests/utils.py#L583
Chromium (more complex - support for different schemes, ports and parts of domains)
https://cs.chromium.org/chromium/src/net/proxy/proxy_bypass_rules.h?l=142
https://cs.chromium.org/chromium/src/net/proxy/proxy_bypass_rules.h?l=153
https://cs.chromium.org/chromium/src/net/proxy/proxy_bypass_rules.cc?l=255
https://cs.chromium.org/chromium/src/net/proxy/proxy_config_service_linux_unittest.cc?q=no_proxy+package:^chromium$&l=1076
Some more reading
https://developer.mozilla.org/en-US/docs/No_Proxy_For_configuration
https://github.com/curl/curl/issues/1208
https://www.gnu.org/software/wget/manual/html_node/Proxies.html
https://github.com/libproxy/libproxy/blob/master/libproxy/modules/ignore_domain.cpp
https://github.com/libproxy/libproxy/blob/master/libproxy/modules/ignore_hostname.cpp
https://github.com/libproxy/libproxy/blob/master/libproxy/modules/ignore_ip.cpp

I'd like to help out on this so I created the above proposal and hopefully, it get's approved and then I can get started on the coding side with a little nudge in the right direction :grin:  I added @forrejam as a contributor to the proposal doc because he did quite a bit of the researching and researching of stuff.

@bradfitz Is there anything else that needs to be done to consider this for inclusion in a future release? I like many others are hitting issues with Go programs not recognizing CIDR ranges in the NO_PROXY while other applications/languages accept them. One has to become creative when specifying a NO_PROXY to work around the limitations of any Go program.

I'm still on leave. /cc @tombergan

I'm back from leave. I've submitted your proposal doc CL. Visible at https://github.com/golang/proposal/blob/master/design/16704-cidr-notation-no-proxy.md
What's not entirely clear from your doc is who all supports CIDRs in $no_proxy. Just Python? Or others?
This seems fine, though. Want to prepare a CL?
It won't happen for Go 1.10, though, which entered feature freeze today.

What's not entirely clear from your doc is who all supports CIDRs in $no_proxy. Just Python? Or others?

@forrejam's comment has a nice algorithm summary and answers the question of who supports CIDRs in no_proxy. @kapilt's comment has another summary. Looks like chrome, firefox, libproxy, python, ruby, and curl support CIDRs in $no_proxy.

@tombergan, ah, right. I had expected that to be summarized in the doc, though.
In any case, I'm fine with this. Somebody can send a CL.

@bradfitz Given your last comment in my change, should I just abandon this change and create a new one for x/net/http/proxy?
Looks like we could attempt to reuse bits from x/net/proxy or else we have two different ways to parse no_proxy. Although I believe we already have differences between the two. Just a thought.

I've hit this problem too - another +1 for an upstream fix.  (We might work around it ourselves for now.)

Found this thread while investigating Packer issues (compiled in GoLang) in which it is implied a fix may go out in 1.11 due this month?  Hope so.
At any rate let me +1 this as well.  Proxy use in certain verticals is common, and in the age of automation we often find ourselves needing to hit IPs as DNS simply isn't available yet.

From what I can see in https://go-review.googlesource.com/c/net/+/115255 it looks like it is progressing! :)

@jansmets, see https://golang.org/wiki/NoPlusOne


Exclude setGroupingSize test from both Hotspot and OpenJ9 on JDK14+

The change made in https://github.com/AdoptOpenJDK/openjdk-systemtest/pull/329 will exclude DecimalFormat.setGroupingSize from 8 and 11.
However, https://bugs.openjdk.java.net/browse/JDK-8231851 applies to jdk versions 14+
This PR reverts https://github.com/AdoptOpenJDK/openjdk-systemtest/pull/329, and adds excludes the exclude files specific to 14+ runs of this test, and removes the check for openj9 from the mauve stf wrapper code.
Signed-off-by<EMAIL_ADDRESS><EMAIL_ADDRESS>

Test OpenJ9 JDK 14: Test_grinder/job/Grinder/6266/console

@JasonFengJ9 - FYI.

I don't think it's worth putting in code which says 'if > jdkn' as conditionals for running these tests, or creating specific exclude files.  The maintenance overhead is just too great.  The life of the mauve test suite is limited anyway - it has not been maintained for 10 years.  Rather than creating more config files I would advocate removing release (and jvm) specific config files and just run with the lowest common denominator of tests across all releases.

https://github.com/AdoptOpenJDK/openjdk-systemtest/pull/330#issuecomment-591884003 is fine with me.

Sounds Good. I've created an issue to clean up Mauve tests and have a single load across all platforms and sdk versions https://github.com/AdoptOpenJDK/openjdk-systemtest/issues/331.
Closing this PR without merging.