Generate script for create object doesn't always work

Issue Type: Bug
very often, when I go to script create an object from my saved connections, it won't respond. No indictation that it's waiting, no idication that it's still attempting anything. disconnecting and reconnecting sometimes resolves it, but it might take 2-3 reconnections. This is on multple client databases across different azure subscriptions, always azure SQL db.
Azure Data Studio version: azuredatastudio 1.25.1 (e7eacc32c0d20bd4da843320c3bfac231b308c98, 2020-12-11T01:59:24.271Z)
OS version: Windows_NT x64 10.0.19042

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8550U CPU @ 1.80GHz (8 x 1992)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: enabled_onprotected_video_decode: enabledrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
7.81GB (2.80GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (14)

Extension
Author (truncated)
Version

firstresponderkit
dre
0.5.0

mssql-db-insights
mat
0.2.2

admin-pack
Mic
0.0.2

admin-tool-ext-win
Mic
0.0.2

agent
Mic
0.48.0

datavirtualization
Mic
1.10.0

import
Mic
1.1.0

query-history
Mic
0.1.0

schema-compare
Mic
1.8.0

server-report
Mic
0.2.2

azdata-sanddance
msr
3.1.0

sql-search
Red
0.4.0

simple-data-scripter
sea
0.1.0

vscodeintellicode
Vis
1.2.1

@ClutteredSchema Thanks for opening the issue. To help us investigate, could you give some more details on the type of server and the type of object you're trying to script? You can also go to Help -> Toggle Developer Tools -> Console and see if there are any errors there. If there are, please copy paste them here so we can see what the error is.

@ClutteredSchema Thanks for opening the issue. To help us investigate, could you give some more details on the type of server and the type of object you're trying to script? You can also go to Help -> Toggle Developer Tools -> Console and see if there are any errors there. If there are, please copy paste them here so we can see what the error is.


tests: kernel: pipe: Added new test cases for pipe.

The following are the test cases added for pipe.

Test pipe APIs on single data element.
Test pipe APIs on multiple data elements.
Test pipe APIs with forever wait.
Test pipe APIs with timeout.
Test pipe APIs on an empty pipe.
Test pipe get API with forever timeout.
Test pipe get API with limited timeout.

Signed-off-by: Adithya Baglody<EMAIL_ADDRESS>

Codecov Report

Merging #6516 into master will increase coverage by 0.18%.
The diff coverage is 89.84%.

@@            Coverage Diff             @@
##           master    #6516      +/-   ##
==========================================
+ Coverage   53.36%   53.55%   +0.18%
==========================================
  Files         435      437       +2
  Lines       41308    41505     +197
  Branches     7922     7942      +20
==========================================
+ Hits        22046    22227     +181
- Misses      16044    16059      +15
- Partials     3218     3219       +1

Impacted Files
Coverage Δ

tests/kernel/pipe/pipe/src/main.c
66.66% <66.66%> (ø)

tests/kernel/pipe/pipe/src/test_pipe.c
90.2% <90.2%> (ø)

kernel/pipes.c
91.16% <0%> (+1.86%)
:arrow_up:

Continue to review full report at Codecov.

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

recheck

rebased

@andrewboie now running these tests in user mode.


Encoder Does Not Support Multi-Byte Codepoints.

#10 Added support for decoding based on utf-8 codepoints. The Encoder needs to be updated in a similar manner.

For background on this issue see #12


bug: Release 0.5.0 artifacts (like the SDK) do not seem to be available

Issue description
The 0.5.0 version of the plugin does not seem to be actually released.  There is no 0.5.0 tag, there are no 0.5.0 Maven artifacts, but the documentation suggests that there is a 0.5.0 version available
Environment
everywhere
Minimal test code / Steps to reproduce the issue

add plugin SDK dependency as 0.5.0

alternatively,
1a. navigate to the 0.5.0 tag
What's the actual result? (including assertion message & call stack if applicable)

Maven cannot resolve a version 0.5.0 of the SDK
No 0.5.0 tag in Github

What's the expected result?
The 0.5.0 tag and 0.5.0 artifacts should exist

has same error

#309


Limiting "attempts" value in "fastMutex.lock" function

https://github.com/ProtonMail/mutex-browser/blob/cf7746e9dcb13d3837c3eaca86f7b6e713928164/src/fastMutex.ts#L62
The module falls back to cookieMutex for me due to the #3 and then fastMutex.lock call never gets resolved for me.
So maybe it worth liming the attempts count and when the algorithm reaches the limit it throws the error. That would allow earlier issues detection. You know, silent issues are normally hard to notice and debug.

Yes definitely. I added a max limit in https://github.com/ProtonMail/mutex-browser/commit/48dfddfdf9d55efc8b22116799017d6174ed4d14.
However the problem of it never resolving sounds like it could be caused by something else. Is the cookie setting mechanism not working well in the environment it is being run in?

sounds like it could be caused by something else

Of course. I didn't point to the issue cause here but suggested making the potential infinity looping issue sound louder by introducing the iteration limit, so it's easier to notice the problematic looping. Then when the issue gets noticed it needs to be debugged which I didn't do (not planning yet) since I just ended up using a fixed indexedDB support test. The indexedDB-based mutex seems to work well for me, at least promises don't get into the continuously unresolved state.

Is the cookie setting mechanism not working well in the environment it is being run in?

It's the @electron environment, recent stable version.


Unknown error when attempting to stop a deployment

When I start a deployment, then try to stop it using reco deploy stop <ID> I get the error message:
histogram-array $ reco deploy stop fc27a0c5-a106-414f-8cd0-f13038fd3ee0

Error: Unknown error occurred

But if I look at the deploy list I see it's trying to terminate:
histogram-array $ reco deploy list

             DEPLOYMENT ID                                BUILD ID                           COMMAND               STATUS           STARTED         DURATION
  fc27a0c5-a106-414f-8cd0-f13038fd3ee0      b5e66fb4-4b0e-4f8c-97d6-32248f02f320      test-histogram             terminating      1 minute ago      -
  7766670e-de85-467e-8469-bf94bd255ebc      b5e66fb4-4b0e-4f8c-97d6-32248f02f320      test-histogran             errored          1 week ago        2m15s
  448ea0d3-d66f-4d14-940f-14524c8d1464      33a606f2-7f5a-4748-bb5f-221473011e93      bench-histogram-FPGA       completed        2 months ago      3m7s

Just to update everyone on the status, this is an API-side error where if you create a deployment and terminate it rapidly we haven't gotten around to creating your deployment by the time the termination comes in. This causes an internal error which reco reports as unknown error occurred.
The fixes for this are two-fold:

Fix the API, that work is in progress
Have reco report the error provided by the API instead of a generic message.


Workflow graph - ELK configuration

Description
Deep dive into ELK configurations available and determine best fit for OS web app graph

Please add your planning poker estimate with ZenHub @Prasy12


[File Browser] Allow TAB in File Editor

The behavior of the TAB key in Hue default when editing file (meaning moving to the next HTML anchor). When editing file TAB should be consider as a character as it might be the value separator. Maybe a check box to enable or disable the tab behavior can be set + some javascript to handle tab when focus is in the textbox

How about http://superuser.com/questions/67934/typing-the-tab-character-in-browser-text-boxes ?


3.0.0-preview5 Nested Relationship should be Insert but treated as Update

Nice to talk to you over here @roji :clap:  I've been using PGSQL lately on a project :smile:    Not sure if this is a problem in the provider or upstream, but I figured I'd start here...
Models:
using System.ComponentModel.DataAnnotations;
using System.Security.Cryptography;
using Microsoft.EntityFrameworkCore;

namespace Test
{
    public class Blog
    {
        public static void OnModelCreating(ModelBuilder modelBuilder)
        {
            modelBuilder.Entity<Blog>(entity =>
            {
                entity.HasMany(m => m.Posts)
                    .WithOne(m => m.Blog)
                    .HasForeignKey(m => m.BlogId)
                    .HasPrincipalKey(m => m.Id);
                entity.HasIndex(m => m.Name);
                entity.Property(m => m.Created)
                    .HasColumnType("TIMESTAMP WITH TIME ZONE")
                    .HasDefaultValueSql("CURRENT_TIMESTAMP");
            });
        }

        [Key] public Guid Id { get; set; } = SequentialGuidGenerator.Next();

        public DateTime Created { get; set; }

        [Required] public string Name { get; set; }

        public virtual ICollection<Post> Posts { get; set; } = new List<Post>();
    }

    public class Post
    {
        public static void OnModelCreating(ModelBuilder modelBuilder)
        {
            modelBuilder.Entity<Post>(entity =>
            {
                entity.HasOne(m => m.Blog)
                    .WithMany(m => m.Posts)
                    .HasForeignKey(m => m.BlogId)
                    .HasPrincipalKey(m => m.Id);
                entity.HasIndex(m => m.Name);
                entity.Property(m => m.Created)
                    .HasColumnType("TIMESTAMP WITH TIME ZONE")
                    .HasDefaultValueSql("CURRENT_TIMESTAMP");
            });
        }

        [Key] public Guid Id { get; set; } = SequentialGuidGenerator.Next();

        public DateTime Created { get; set; }

        [Required] public string Name { get; set; }

        [Required] public string Content { get; set; }

        [Required] public Guid BlogId { get; set; }

        public virtual Blog Blog { get; set; }
    }

    public static class SequentialGuidGenerator
    {
        private static readonly RandomNumberGenerator Rng = RandomNumberGenerator.Create();

        public static Guid Next()
        {
            var randomBytes = new byte[8];
            Rng.GetBytes(randomBytes);
            var ticks = (ulong) DateTime.UtcNow.Ticks;

            var guid = new Guid((uint) (ticks >> 32), (ushort) (ticks << 32 >> 48), (ushort) (ticks << 48 >> 48),
                randomBytes[0],
                randomBytes[1],
                randomBytes[2],
                randomBytes[3],
                randomBytes[4],
                randomBytes[5],
                randomBytes[6],
                randomBytes[7]);

            return guid;
        }
    }
}

Insert a Blog and a Post:
            var blog = new Blog
            {
                Name = "Blog"
            };
            _db.Blog.Add(blog);
            var post = new Post
            {
                Name = "Post",
                Content = "Test"
            };
            blog.Posts.Add(post);
            await _db.SaveChangesAsync();

Get an exception, appears that it get's the Insert correct for the Blog, but it is treating the Post as an Update for some reason:
dbug: Microsoft.EntityFrameworkCore.Database.Command[20100]
      Executing DbCommand [Parameters=[@p0='?' (DbType = Guid), @p1='?'], CommandType='Text', CommandTimeout='30']
      INSERT INTO "Blog" ("Id", "Name")
      VALUES (@p0, @p1)
      RETURNING "Created";
dbug: Microsoft.EntityFrameworkCore.Database.Command[20101]
      Executed DbCommand (21ms) [Parameters=[@p0='?' (DbType = Guid), @p1='?'], CommandType='Text', CommandTimeout='30']
      INSERT INTO "Blog" ("Id", "Name")
      VALUES (@p0, @p1)
      RETURNING "Created";
dbug: Microsoft.EntityFrameworkCore.Database.Command[20300]
      A data reader was disposed.
dbug: Microsoft.EntityFrameworkCore.Database.Command[20100]
      Executing DbCommand [Parameters=[@p6='?' (DbType = Guid), @p2='?' (DbType = Guid), @p3='?', @p4='?' (DbType = DateTimeOffset), @p5='?'], CommandType='Text', CommandTimeout='30']
      UPDATE "Post" SET "BlogId" = @p2, "Content" = @p3, "Created" = @p4, "Name" = @p5
      WHERE "Id" = @p6;
dbug: Microsoft.EntityFrameworkCore.Database.Command[20101]
      Executed DbCommand (5ms) [Parameters=[@p6='?' (DbType = Guid), @p2='?' (DbType = Guid), @p3='?', @p4='?' (DbType = DateTimeOffset), @p5='?'], CommandType='Text', CommandTimeout='30']
      UPDATE "Post" SET "BlogId" = @p2, "Content" = @p3, "Created" = @p4, "Name" = @p5
      WHERE "Id" = @p6;
dbug: Microsoft.EntityFrameworkCore.Database.Command[20300]
      A data reader was disposed.
Database operation expected to affect 1 row(s) but actually affected 0 row(s). Data may have been modified or deleted since entities were loaded. See http://go.microsoft.com/fwlink/?LinkId=527962 for information on understanding and handling optimistic concurrency exceptions.

Unhandled Exception: Microsoft.EntityFrameworkCore.DbUpdateConcurrencyException: Database operation expected to affect 1 row(s) but actually affected 0 row(s). Data may have been modified or deleted since entities were loaded. See http://go.microsoft.com/fwlink/?LinkId=527962 for information on understanding and handling optimistic concurrency exceptions.
   at Npgsql.EntityFrameworkCore.PostgreSQL.Update.Internal.NpgsqlModificationCommandBatch.ConsumeAsync(RelationalDataReader reader, CancellationToken cancellationToken)
   at Microsoft.EntityFrameworkCore.Update.ReaderModificationCommandBatch.ExecuteAsync(IRelationalConnection connection, CancellationToken cancellationToken)
   at Microsoft.EntityFrameworkCore.Update.Internal.BatchExecutor.ExecuteAsync(DbContext _, ValueTuple`2 parameters, CancellationToken cancellationToken)
   at Npgsql.EntityFrameworkCore.PostgreSQL.Storage.Internal.NpgsqlExecutionStrategy.ExecuteAsync[TState,TResult](TState state, Func`4 operation, Func`4 verifySucceeded, CancellationToken cancellationToken)
   at Microsoft.EntityFrameworkCore.ChangeTracking.Internal.StateManager.SaveChangesAsync(IList`1 entriesToSave, CancellationToken cancellationToken)
   at Microsoft.EntityFrameworkCore.ChangeTracking.Internal.StateManager.SaveChangesAsync(Boolean acceptAllChangesOnSuccess, CancellationToken cancellationToken)
   at Microsoft.EntityFrameworkCore.DbContext.SaveChangesAsync(Boolean acceptAllChangesOnSuccess, CancellationToken cancellationToken)

Env Info:
dotnet: 3.0.100-preview5-011568
Npgsql.EntityFrameworkCore.PostgreSQL Version: 3.0.0-preview5
Microsoft.EntityFrameworkCore.Design Version: 3.0.0-preview5.19227.1
postgres: 10.4

Looks like it was a problem with the ordering of _db.Blog.Add(blog), that needs to go after the related Post has been added to the collection.
            var blog = new Blog
            {
                Name = "Blog"
            };
            var post = new Post
            {
                Name = "Post",
                Content = "Test"
            };
            blog.Posts.Add(post);
            _db.Blog.Add(blog);
            await _db.SaveChangesAsync();

Makes sense, since the documentation for DbContext.Add states:

Begins tracking the given entity, and any other reachable entities that are not already being tracked, in the Added state such that they will be inserted into the database when SaveChanges() is called.


website responsive ness is lacking i would like to improve it kindly assign it to me i am a ssoc participant

@vaibhavsharma1234 please provide the description and attach some screenshot for the same.

please refer to this sir


[Bug][FLINK-31363]KafkaSink failed to commit transactions under EXACTLY_ONCE semantics.

What is the purpose of the change
FLINK-31363
When KafkaSink starts Exactly once and no data is written to the topic during a checkpoint, the transaction commit exception is triggered, with the following exception.

If data is written during the checkpoint, this exception will not occur.
After the transaction attribute transactionStarted is changed to false, whether there is data writing or not will not affect the transaction commit.
Brief change log
(for example:)

The TaskInfo is stored in the blob store on job creation time as a persistent artifact
Deployments RPC transmits only the blob storage reference
TaskManagers retrieve the TaskInfo from the blob cache

Verifying this change
Please make sure both new and modified tests in this PR follows the conventions defined in our code quality guide: https://flink.apache.org/contributing/code-style-and-quality-common.html#testing
(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (100MB)
Extended integration test for recovery after master (JobManager) failure
Added test that validates that TaskInfo is transferred only once across recoveries
Manually verified the change by running a 4 node cluster with 2 JobManagers and 4 TaskManagers, a stateful streaming program, and killing one JobManager and two TaskManagers during the execution, verifying that recovery happens correctly.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@fapaul @AHeise PTAL.


Store.updateData should handle a flat list of updates/inserts

Would like to enhance this new API for the common use case where you have a flat record set and a set of updates or inserts, and just want the store to do the right thing by using the ids.

Subsumed in the following:
Store enhancements for partial loading and client-side editing support #1495


Merged gplayers to tmdbplayers

Hey there, a couple of weeks ago, GPlayers experienced some downtime. Consequently, I made the decision to integrate its players with TMDBPlayers. I successfully incorporated all the new players and also performed updates on several existing ones. While adding new players was straightforward, updating proved to be a bit challenging, mainly due to the comparison process. Nevertheless, I believe everything should now be accurate. If you like, feel free to merge this PR. If you notice anything wrong, please don't hesitate to message, and I'll do my best to fix it.
Thank you
✨ - Added 9lives Players
✨ - Added Absolution Players
✨ - Added Adina Players
✨ - Added Affenity Players
✨ - Added Amazon Players
✨ - Added Base19 Players
✨ - Added Buds Players
✨ - Added Coalition. Players
✨ - Added Dramacool Players
✨ - Added Dradis Players
✨ - Added Dynasty Players
✨ - Added Fen Players
✨ - Added Fenlight Players
✨ - Added Gaia Players
✨ - Added Hulu Players
✨ - Added Imdbtrailers Players
✨ - Added Kingpin Players
✨ - Added Kit Players
✨ - Added Luffy Players
✨ - Added Metv Players
✨ - Added Moria Players
✨ - Added Netflix Players
✨ - Added Pov Players
✨ - Added Quicksilver Players
✨ - Added Shazam Players
✨ - Added Thechains Players
✨ - Added Thelab Players
✨ - Added Thelabjr Players
✨ - Added Thepromise Players
✨ - Added Twilight Players
✨ - Added Unleashed Players
✨ - Added Wizz Players
✨ - Added Wutu Players
✨ - Added Youtube Players
✨ - Added Zoro Players
⚙ - Updated Alvin Players
⚙ - Updated Gaia Players
⚙ - Updated Genocide Players
⚙ - Updated Homelander Players
⚙ - Updated Nightwing Players
⚙ - Updated Scrubsv2 Players
⚙ - Updated Thecrew Players

Your right, for now, I'm going to work on this more and see what needs to be added and what needs to be modified and everything. Thank you for reviewing these.

Hey so im actually going to close this for now and work on this for a later date, reason why is cause i worked on this pr today on my laptop and i should have done this on my desktop. Will have this reopened when finished. Thank you

No need to close this one; you can just make the modifications as a new commit and they'll show up here.


doc: llama-stack build --config help text references old directory

What does this PR do?

llama-stack build --config help text references example_configs which no longer exists
Update to refer new directory format to avoid confusion

Before submitting

[x] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).

Thanks!


[xcode-archive] fails at fetching devices if some are in the state "Ineligible for x days"

Some of my iOS devices are in the state "Ineligible for x days" at:
https://developer.apple.com/account/resources/devices/list

Fetching Apple Developer Portal devices
Failed to execute Step main logic:
  failed to manage code signing: failed to ensure test devices: failed to fetch devices: failed to query Developer Portal: invalid device status p, stacktrace: /private/var/folders/dw/lz_9l61x0y9089f673sqt55h0000gn/T/2230338121/devices.rb:63:in `map_device_status_to_api_status'
  /private/var/folders/dw/lz_9l61x0y9089f673sqt55h0000gn/T/2230338121/devices.rb:15:in `block in list_devices'
  /private/var/folders/dw/lz_9l61x0y9089f673sqt55h0000gn/T/2230338121/devices.rb:8:in `each'
  /private/var/folders/dw/lz_9l61x0y9089f673sqt55h0000gn/T/2230338121/devices.rb:8:in `list_devices'
  main.rb:63:in `<main>'

It seems like it's only a mapping issue: map_device_status_to_api_status
Locally I can create an archive with Xcode (14.2) therefore I think it should be possible on bitrise as well.

We are having this same issue when adding new devices that are in the "Processing" status.


Why the api for headers does not work?

headers.set(name, value) will make this mistake,have a demo for this?

I made the mistake,It's needs backend to cooperate


scripts: Fix risky uses of non-raw regex strings in Python scripts

Fixes pylint warnings like this one:
doc/conf.py:325:0: W1401: Anomalous backslash in string: '\s'.
String constant might be missing an r prefix.
(anomalous-backslash-in-string)

The reason for this warning is that backslash escapes are interpreted in
non-raw (non-r-prefixed) strings. For example, '\a' and r'\a' are not
the same string (first one has a single ASCII bell character, second one
has two characters).
It just happens that there's no \s (or \., or \/) escape for example,
and '\s' turns into two characters (as needed for a regex). It's risky
to rely on stuff like that regexes though. Best to make them raw strings
unless they're super trivial.
Also note that '\s' and '\\s' turn into the same string.
Another tip: A literal ' can be put into a string with "blah'blah"
instead of 'blah\'blah'.

Codecov Report

Merging #14803 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #14803   +/-   ##
=======================================
  Coverage   52.51%   52.51%
=======================================
  Files         309      309
  Lines       45048    45048
  Branches    10419    10419
=======================================
  Hits        23656    23656
  Misses      16584    16584
  Partials     4808     4808

Continue to review full report at Codecov.

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


Unable to mock methods on the view config

It appears the view config is different:
   34:     before do
   35:       byebug
=> 36:       allow(controller.view_context).to receive_messages(facet_group_names: [nil], facet_field_names: [:facet_field_1], facet_limit_for: 10)
   37:     end
(byebug) controller.view_context.object_id
70206343712180
(byebug) c

[1, 10] in /Users/jcoyne85/workspace/projectblacklight/blacklight/app/views/catalog/_facet_group.html.erb
    1: <% # main container for facets/limits menu -%>
    2: <% byebug %>
=>  3: <% if has_facet_values? facet_field_names(groupname), response %>
(byebug) self.object_id
70206343979280

So the mocks are never called.
This may be because my component calls a regular view render.
e.g.:
app/components/list.html.erb:
<% # container for facet groups -%>
<% facet_group_names.each do |groupname| %>
  <%= render 'catalog/facet_group', groupname: groupname, response: response %>
<% end %>

@jcoyne I believe you are correct in that it is due to the use of a regular view render, as we are passing those calls to the enclosing view context.

So, even without the nested render, the view context in the test is not the view context in the component:
[19, 28] in /Users/jcoyne85/workspace/projectblacklight/blacklight/spec/components/blacklight/facet/list_spec.rb
   19:
   20:   before do
   21:     blacklight_config.facet_fields['facet_field_1'] = facet_field
   22:     byebug
   23:
=> 24:     allow(controller.view_context).to receive(:render)
   25:   end
   26:
   27:   it "Calls facet_group for each name" do
   28:     rendered
(byebug) controller.view_context.object_id
70178698582340
(byebug) c

[12, 21] in /Users/jcoyne85/workspace/projectblacklight/blacklight/app/components/blacklight/facet/list.rb
   12:         blacklight_config.facet_fields.map { |_facet, opts| opts[:group] }.uniq
   13:       end
   14:
   15:       def render(*)
   16:         byebug
=> 17:         super
   18:       end
   19:
   20:       attr_reader :response
   21:
(byebug) s

[72, 81] in /Users/jcoyne85/.rbenv/versions/2.6.4/lib/ruby/gems/2.6.0/gems/actionview-component-1.3.3/lib/action_view/component/base.rb
   72:       end
   73:
   74:       def initialize(*); end
   75:
   76:       def render(options = {}, args = {}, &block)
=> 77:         if options.is_a?(String) || (options.is_a?(Hash) && options.has_key?(:partial))
   78:           view_context.render(options, args, &block)
   79:         else
   80:           super
   81:         end
(byebug) view_context.object_id
70178698798700

Ah, I think my problem is that view_context is not stable:
(byebug) controller.view_context.object_id
70140428391180
(byebug) controller.view_context.object_id
70140428405480
(byebug) controller.view_context.object_id
70140428443240


Fixed fabric8mq integration test

Default controller name is fabric8mq-controller, not fabric8MQ

Can one of the admins verify this patch?

ok to test

Thanks @davidatkins!

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/fabric8-quickstarts-master-pullreq/257/
Test PASSed.

Merged as 5d7a9f88b995e6fe8691d180307d7a9c87ee02f9.


Adding some informative comments for Label creation functions

@intrigued-rishi This is not to discourage anyone from adding comments. However, I don't see any of these comments as really being necessary. It is already pretty obvious what those functions do, just by their names. Meaning, this code is self documenting.
Can you justify the need for these comments?


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_71fuj1jwz7/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_71fuj1jwz7/clone/recipe/meta.yaml:28: missing_pip_check: For pypi packages, pip check should be present in the test commands
ERROR: /tmp/abs_71fuj1jwz7/clone/recipe/meta.yaml:36: missing_documentation: The recipe is missing a doc_url or doc_source_url
WARNING: /tmp/abs_71fuj1jwz7/clone/recipe/meta.yaml:36: missing_description: The recipe is missing a description
Errors were found


nltk in Python 3.4 pos_tag issue?

I'm trying to brush up on NLTK because I'm going to need some of its functions when I'm working on my senior thesis this fall. I figured that starting with a POS tagger would be fine, but whenever I try to tag something I get this error:
Traceback (most recent call last):
File "<pyshell#22>", line 1, in
pos_tag_sents(sents)
File "C:\Python34\lib\site-packages\nltk\tag_init_.py", line 108, in pos_tag_sents
tagger = load(_POS_TAGGER)
File "C:\Python34\lib\site-packages\nltk\data.py", line 779, in load
resource_val = pickle.load(opened_resource)
UnicodeDecodeError: 'ascii' codec can't decode byte 0xcb in position 0: ordinal not in range(128)
I tried Googling it, and went to the files that it's talking about to see if anything is wonky, but I couldn't find anything. I was doing exactly what the NLTK book (NLP With Python) said, but it wouldn't work. Whether I accidentally messed something up or it's a problem with the NLTK stuff in Python 3.4, I would really appreciate it if someone could help me out.

where can i find data.py file?
i also getting same error as above


add instructions about milestone and change agreed label, following i…

See issue #371 for discussion of these changes.
Release checklist

[NA] Authors updated in cf-conventions.adoc?
[NA] Next version in cf-conventions.adoc up to date? Versioning inspired by SemVer.
[NA] history.adoc up to date?
[NA] Conformance document up-to-date?

Only change I made was to remove the empty lines between the bullet points. When there's an empty line, Markdown renders that as a new bullet list all on its own, so we had multiple bullet lists with only 1 bullet point each. This way we have 1 list with multiple bullets.

Merged into #372


Proper EDNS0 (non)support

Coverage increased (+0.1%) to 93.927% when pulling 20db2136aeed35ac6667008e7cde317396e12a66 on edns0 into 5b1e51534f5c6738742e6b56af887e3280501168 on master.


Removing a "TODO" that is not necessary any more

Cleaning up the comments.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: mbanikazemi
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: diemtvu
If they are not already assigned, you can assign the PR to them by writing /assign @diemtvu in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS

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

@mbanikazemi: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
6a2bf4cd554b7a458e3284e24d4e134439f21ea5
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@mbanikazemi: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
6a2bf4cd554b7a458e3284e24d4e134439f21ea5
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-k8s-tests.sh
6a2bf4cd554b7a458e3284e24d4e134439f21ea5
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/lgtm

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

Needs approval from an approver in each of these files:

pilot/OWNERS [hzxuzhonghu]

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

@mbanikazemi: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
6a2bf4cd554b7a458e3284e24d4e134439f21ea5
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
6a2bf4cd554b7a458e3284e24d4e134439f21ea5
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Application Emails

🛠 Changelog

Refactor email building (a lot)
Add support for Application email templates

Add new Application email template types
New email types are sent in correct circumstances
Testing of Application email templates in Django Admin

🧪 Test plan

Automated tests
Manually test the Django Admin, including sending test emails (no need to actually send any emails, but check the console that they are printed there correctly)

🎫 Tickets

TILA-3225

Related to https://github.com/City-of-Helsinki/tilavarauspalvelu-ui/pull/1073

Looks good, settings.SEND_RESERVATION_NOTIFICATION_EMAILS could be renamed SEND_NOTIFICATION_EMAILS now that there are also application emails


build: package material api docs into npm package

@jelbourn This would be ready for review. I have some plans to reduce the Bazel arguments overhead, but I want to address this in a follow-up if that's okay.


Added JWT authentication.

I've included examples of use in the examples folder.
A possible enhancement is that the jwt token is considered for renewal while it is being verified.

Coverage decreased (-1.7%) to 98.299% when pulling 5d16cfb4ee790c8670798dfcb5ba6bbffb5de1ff on dsmurrell:develop into 78a6be2c2a8edfc7b0c4f6c259fe5e39ae9cfddb on timothycrosley:develop.

Coverage decreased (-1.5%) to 98.48% when pulling a5cd7f52d3213438c75df122110c10489df6d100 on dsmurrell:develop into 78a6be2c2a8edfc7b0c4f6c259fe5e39ae9cfddb on timothycrosley:develop.

I've modified the code to catch jwt.InvalidTokenError exceptions.
This is the base class which includes exceptions such as jwt.ExpiredSignatureError and jwt.DecodeError. jwt.ExpiredSignatureError will be far more common so it should definitely catch these silently. I'm less convinced about catching jwt.DecodeErrors silently as this is relatively unexpected.
It's possible I should only be catching jwt.ExpiredSignatureError exceptions?
For someone trying to get this working from the client side, it would be quite useful to know which exception is being caught. Another question is, how would someone who is trying to get this working for the first time get feedback about what exception is being thrown?

Hi @dsmurrell,
In general, the design currently is such that if there is an error you want to customize output for the correct way to do that is simply to allow the exception to bubble up, so incorrect credentials should be handled as False, and for others simply raise. In the case that you want to provide additional information beyond this for handling the exception, you could add contextual information to the request.context or in the exception that you raise for things higher up to introspect.
Thanks!
~Timothy

Hey guys, how is this going? JWT is a must-have auth backend for every serious backend microservice so what's left? can we help?
Cheers,
Dgz

We have already implemented the JWT authentication using the provided hug.hug.authentication.token that supports treating Authorization header in a raw way, I can't see the point to implement full fledge JWT authenticator, microframeworks are meant to not add unnecessary stuff, if you @timothycrosley wants us to do a PR with a fully functional example in the examples folder we can do it, I paste here a working summary of it:
import hug
import jwt  # pip install pyjwt
from datetime import datetime, timedelta
from falcon import HTTPUnauthorized

# These can be in a constants/settings file
JWT_OPTIONS = {
        'verify_signature': True,
        'verify_exp': True,
        'verify_nbf': False,
        'verify_iat': False,
        'verify_aud': True,
        'verify_iss': True,
        'require_exp': True,
        'require_iat': False,
        'require_nbf': False
    }
SECRET_KEY = 'some-super-secret'
JWT_ISSUER = 'some-issuer'
JWT_AUDIENCE = 'some-audience'
JWT_OPTIONS_ALGORITHM = 'HS256'

def decode_token(token, options=JWT_OPTIONS):
    """
    Decodes a JWT token and returns payload info
    :return:
    """
    return jwt.decode(
        token,
        SECRET_KEY,
        issuer=JWT_ISSUER,
        audience=JWT_AUDIENCE,
        options=options,
        algorithms=(JWT_OPTIONS_ALGORITHM,)
    )

def create_token(payload, exp=None) -> object:
    """
    Creates JWT Token
    :return:
    """
    if exp is None:
        exp = datetime.utcnow() + timedelta(seconds=3600)
    _token = {
        'aud': JWT_AUDIENCE,
        'exp': exp,
        'iss': JWT_ISSUER,
    }
    _token.update(payload)
    return jwt.encode(
        _token,
        SECRET_KEY,
        algorithm=JWT_OPTIONS_ALGORITHM
    ).decode('utf-8')

def parse_header(authorization_header, exception_class=HTTPUnauthorized):
    """
    Parses an authorization header. Accepts a custom Exception if any failure
    :param authorization_header:
    :param exception_class:
    :return:
    """
    if authorization_header:
        parts = authorization_header.split()

        if parts[0].lower() != 'bearer' or len(parts) == 1 or len(
                parts) > 2:
            raise exception_class("invalid_header")

        jwt_token = parts[1]
        try:
            # Decode token
            payload = decode_token(jwt_token)
            return jwt_token, payload
        except jwt.ExpiredSignature:  # pragma: no cover
            raise exception_class("token is expired")

        except jwt.InvalidAudienceError:  # pragma: no cover
            raise exception_class("incorrect audience")

        except jwt.DecodeError:  # pragma: no cover
            raise exception_class("token signature is invalid")

        except jwt.InvalidIssuerError:  # pragma: no cover
            raise exception_class("token issuer is invalid")

        except Exception as exc:  # pragma: no cover
            raise exception_class(exc)
    else:  # pragma: no cover
        raise exception_class("Authorization header not present")

def jwt_token_verify(auth_header):
    """
    Verifier function for hug token authenticator
    :param auth_header:
    :return:
    """
    # Hug do not extract Bearer prefix
    auth_token, payload = parse_header(auth_header)
    return payload

token_key_authentication = hug.authentication.token(jwt_token_verify)

# Then the protected API
@hug.get('/jwt', requires=token_key_authentication)
def some_jwt_endpoint():
      return "I'm JWT!"

# Then the JWT Token Generator
@hug.get('/token')
def some_jwt_endpoint():
      return create_token({'some': 'payload'})

Hello @timothycrosley, @dsmurrell and @OGKevin.
What's the status of this PR? It seemed to be almost mergeable some months ago, but nothing new since. Is there something we can do to help?
I've seen with interest @danigosa 's solution, but I'm not sure whether it's unit-tested somewhere, respecting the standard, and it misses the token refreshing capabilities that I rely on.

@briceparent im not an official maintainer. It is up to @timothycrosley to merge.

@dsmurrell are you planning on finishing this?

@tadeoiiit, I did want to finish this at some point. I ended up moving to work on another project that had flask serving it's backend so I never got around to finishing this.
@timothycrosley, has much changed on the authentication side? Is this PR still something that could benefit people if completed? Is so, I can put in some work this weekend to finish it.

@dsmurrell authentication is mostly the same, I think finishing it would be useful! We may end up moving it into a plugin - but either way it's important to support :) Thank you for your work on it!


Fully supervised baseline

I don't know much about tensorflow, so I'm a little confused about CIFAR10 fully supervised trainings.
Just to be sure, are you training for 2^(10 + 16) / 50000 = over 1000 epochs (lr 0.002) for the fully supervised baseline?
As I understand that wrn-28-2 is originally trained for 200 epochs (lr 0.1) and can only get to 95.9%-96.0% accuracy on average with autoaugment and cutout. While in your paper it is a little higher than that (96.38%). So is your training schedule better than other people?
Also, do you have the fully-supervised numbers for other labeled ratios (e.g. 4000 labels), it would feel more official and convincing if the numbers are reported by you.

Yeah, roughly 1000 epochs. It seems like semi-supervised learning requires more time to train than fully-supervised training. We tried hard to optimize all of our baselines, to make sure that we were comparing against the best of what could be done. For example, we substantially improved the accuracy of VAT in the low-label regime when doing the MixMatch experiments.
I don't think we have any fully-supervised numbers for 4000 labeled examples.

Thanks for your reply! I'm trying to re-implement this work or FixMatch in PyTorch, maybe I could find a faster training setup in the process.

Negative, it would seem impossible to train faster without degrading performance. But I do have a fully supervised number on 4000 labeled (CIFAR 10), I can get 86.0% accuracy if I use RA (14 basic op + cutout), N=1, random M.


ImportError: No module named django.core

When I am trying to start the graphite server using the following command
PYTHONPATH=pwd/storage/whisper ./bin/run-graphite-devel-server.py --port=80 --libs=pwd/webapp /data/graphite 1>/data/graphite/storage/log/webapp/process.log 2>&1 &
I am getting the following error
File "./bin/run-graphite-devel-server.py", line 6, in
from django.core import management
ImportError: No module named django.core.
Previously this used to work when I had installed carbon, whisper and graphite-web using  python setup.py.
On attempting a reinstall I started getting error "Whisper module could not be loaded". As a result I installed all these 3 using pip and I have started getting the error.

This is related to issue #2427

make sure you have django installed, it is a supported version and that it is in your pythonpath.

Installing Django again versio 1.11.20 ran the graphite server but the error log file still has
WARNING: whisper module could not be loaded, whisper support disabled
and no metrics is getting displayed in graphite-web.

Did you try installing whisper?
On Thu, 28 Feb 2019, 19:34 Shirish Kale<EMAIL_ADDRESS>wrote:

Installing Django again versio 1.11.20 ran the graphite server but the
error log file still has
WARNING: whisper module could not be loaded, whisper support disabled
and no metrics is getting displayed in graphite-web.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/graphite-project/graphite-web/issues/2428#issuecomment-468385501,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACxrWvfoD0AbLEcYlGqHQKVgcgr2ffAJks5vSCEngaJpZM4bXD6E
.

Yes. No use. Still the same error message

@shirishkale : something is really wrong with your PYTHONPATH. I'm not sure how properly detect that problem or fix it. It's far beyond Graphite per se, it's a generic python problem.
How do you trying to install Graphite? From source or packages? Do you use virtualenv?

All components of graphite were installed 3 times.
First Round

All Carbon, Whisper and Graphite were installed using the downloaded zip files version 1.1.5.
Using python setup.py install --prefix=  --install-lib=
It installed fine except django-admin.py was not under /data/graphite directory where the install were done but in /usr/lib/python2.7/site-packages/Django-1.11.16-py2.7.egg/django/bin/
Neo4j metrics was captured and graphs were correctly displayed.

I uninstalled everything and followed the same procedure to have a repeatable process
Second Round

Install went fine. All components loaded correctly
The error for Not able load Whisper started and no metrics or graphs were displayed
I went ahead and uninstalled anything

To fix the issue of not having django-admin.py under PYTHONPATH all the three components were reinstalled using a different method.
Round Three

Install Carbon, Whisper and Graphite-web using "pip install https://github.com/graphite-project/graphite-web/tarball/master" --prefix=  --install-lib=
Install went ok but ran into issues with Django
REinstalled Django. Noe django -admin.py is under "/data/graphite" where graphite is installed
Still the same error

Hence I am not sure why it worked the first time and will not work the last two times.
All this while my python binaries are under "/usr/bin/python".
Hence can Graphite be installed under any custom directory or has to be under "/usr/bin/python"
path somewhere?
If the install worked the first time then what could create issues for Python suddenly?

The following line complains about not able to load whisper module
django-admin.py runserver --pythonpath /data/graphite/webapp --settings graphite.settings <IP_ADDRESS>:80
What could be wrong with pythonpath when this command explicitly mentions the PYTHONPATH?

is your whisper module installed in /data/graphite/webapp?

No. Whisper module is installed in "/data/graphite" libraries in "/data/graphite/lib".
Graphite web is installed in "/data/graphite" libraries in "/data/graphite/webapp"
When I try to start the graphite-web, then the runserver commands of graphite web complains that it can not load whisper module.

If your modules are installed in /data/graphite/lib why are you specifying /data/graphite/webapp as your pythonpath?

Only whisper and carbon are installed in /data/graphite/lib. Graphite-Web is in webapp? Should I change the install location for graphite-web also to "/data/graphite/lib"?

Well, you installed your modules in a location which is not known by Python. Now Python cannot find them.

Does that mean graphite-web will not work correctly if I install it into a non-default location? Because even if I install everything in /data/graphite even then graphite-web complains that Whisper module can not be loaded. Which means there is no option but to install it in /opt/graphite. Is that so?

you'll need to tell Python where to find its modules. hint

Any Path I use its the same message can not load whisper module? Where exactly it is looking for Whisper module?

Python searches exactly the paths specified in its PYTHONPATH for modules. You can run python -c 'import sys; print(sys.path)' to find out the specified path. If using a custom path it would look something like PYTHONPATH=/tmp/foo/bar python -c 'import sys; print(sys.path)'. see https://docs.python.org/3.7/library/sys.html#sys.path and https://docs.python.org/3.7/using/cmdline.html#envvar-PYTHONPATH for more info.
Anyway, I've closed this as this is not an issue. I think I gave you enough pointers to figure it out yourself. I can't help you any further.


validation_epoch_end only gets the outputs from one process

Hi,
I need the whole validation set to get the validation result.
Current validation_epoch_end only gets the outputs from current process.
Can I collect the gather the outputs from different gpus, and then run validation_epoch_end. And also I don't necessary need it to run on all processes, I only need it to run once.
How can I achieve that?

I use detectron2.utils.comm.all_gather and gather to achieve what I want.
But it would be better if anything builtin in lightning can work for this purpose.

See also: https://github.com/PyTorchLightning/pytorch-lightning/issues/2413

I have something like this in my validation loop:
Sorry to comment on this closed ticket but is this now implemented for validation loop? I am currently doing something like:

def validation_epoch_end(self, outputs):
loss_val = torch.stack([x['val_loss'] for x in outputs]).mean()
log_dict = {'validation_loss': loss_val, 'step': self.current_epoch}
return {'log': log_dict, 'val_loss': log_dict['validation_loss'], 'progress_bar': log_dict}

but this takes only one process into account. Is there a way to aggregate all the batches when using `ddp`? Is using `diet.all_gather`the way to do it?

dist all gather only support torch tensor I believe. detectron2 all_gather can gather anything. It works well for me now.

I see a few other posts that have been closed, so I am assuming this has somehow been sorted and is possible. I also opened another issue about this: https://github.com/PyTorchLightning/pytorch-lightning/issues/2435
Hope someone can help :)

currently I use this to gather other processes' outputs:
    def test_epoch_end(self, outputs):
        avg_loss = torch.stack([x['test_loss'] for x in outputs]).mean()
        avg_acc = torch.stack([x['test_acc'] for x in outputs]).mean()

        dist.all_reduce(avg_acc, op=dist.ReduceOp.SUM)
        avg_acc = avg_acc / self.trainer.world_size

Now, new TensorMetric class can do dist_all_reduce automatically, but it only support SUM, you have to divide by self.trainer.world_size if you want MEAN.  If you have dict outputs, you can iterate all values and use dist_all_reduce

this is fixed on master

@williamFalcon
I don't think it is fixing my original quesiton. I asked if there is a way that the validation_epoch_end can gather the outputs from all the gpus, and do evaluation.
The reason is, in my case, the output of the validation set has to be evaluated as a whole. So I can't simply calculate the score in each process and all reduce them.

this is fixed on master
@williamFalcon
When you say fixed on master, what do you mean? Will this do it automatically or do we need to do something about it? Can you point us to the relevant bit in the lightning source code?

My guess is this
https://github.com/PyTorchLightning/pytorch-lightning/commit/309ed75c5d6740538fca6d9a571d85606ac6d48b

@dagap @ruotianluo install the latest version!
pip install pytorch-lightning==0.8.4 --upgrade

@williamFalcon Thanks for that. So, we do not need to do anything from the caller end? it will do this automagically in validation/test modes?

yup!
Magic is here:
https://github.com/PyTorchLightning/pytorch-lightning/blob/325852c6df93f749bb843bff1a3cdba41698722c/pytorch_lightning/trainer/evaluation_loop.py#L366

@williamFalcon I am still confused.
What is the expected behavior? Is there any doc related to this?
I guess a concrete example would be helpful. If I have 2gpus with ddp, and the validation set has 500batches, so each gpu gets 250 batches.
What is the size of the input argument outputs for validaiton_epoch_end?

the behavior is transparent to you. meaning, if you calculate accuracy on 2 gpus, then we average the accuracy across gpus, but it's transparent to you.
Gpu 0
def validation_step(...)
   return {'acc': 10}

Gpu 1
def validation_step(...)
   return {'acc': 20}

What is logged, printed, etc... is 15.0

Thanks. It's the same as I understand.
The reason why I said it doesn't solve my question, because in my case the validation_step does not return an accuracy, not even a number, but a set of captions.
What I want to achieve, is to collect all the captions of the validation images. So the input of validation_epoch_end is all the captions, and validaiton_epoch_end can call an external library to evaluate the results.
I think it's fine to evaluate separately, but it is just safer to evaluate all as a whole. That's why I asked the question at the first place.
For now, I think my workaround is fine.

@williamFalcon
This does seem to be there on the latest master. Has this been removed again?

it’s disabled on master at the moment because

it will blow up your ram
it may only work with certain tensors

so, we are cleaning it up and adding back in

Now, my tensor not all the cross GPU syncing automagically


[Program]: Write a PHP program to find the transpose of a matrix

Description
Write a PHP program to find the transpose of a matrix
The transpose of a matrix is an operator which flips a matrix over its diagonal, that is it switches the row and column indices of the matrix by producing another matrix denoted as AT.
Input  :
1 2 3
4 5 6
7 8 9

Output :
1 4 7
2 5 8
3 6 9

How to contribute

Save the solution in program/program/find-the-transpose-of-a-matrix/program/find_the_transpose_of_a_matrix.php file

!assign


[QUESTION]  Allowed memory size of 134217728 bytes exhausted (tried to allocate 98304 bytes)

Before you submit this issue, you has been search all existed issues and search the documentation

[] I've been search all existed issues
[] I've been read all documentation

Describe your question
从1.1升级到2.0之后. 启动就报了内存超标.  错误提示信息为:
Fatal error: Allowed memory size of 134217728 bytes exhausted (tried to allocate 98304 bytes) in /var/www/jingling/vendor/roave/better-reflection/src/SourceLocator/Ast/Parser/MemoizingParser.php on line 53
以下是我composer.json的关键信息
"require": {
"php": ">=7.4",
"ext-swoole": ">=4.5",
"hyperf/cache": "~2.0.0",
"hyperf/command": "~2.0.0",
"hyperf/config": "~2.0.0",
"hyperf/contract": "~2.0.0",
"hyperf/database": "~2.0.0",
"hyperf/db-connection": "^2.0",
"hyperf/devtool": "~2.0.0",
"hyperf/di": "~2.0.0",
"hyperf/dispatcher": "~2.0.0",
"hyperf/event": "~2.0.0",
"hyperf/exception-handler": "~2.0.0",
"hyperf/framework": "~2.0.0",
"hyperf/guzzle": "~2.0.0",
"hyperf/http-server": "~2.0.0",
"hyperf/logger": "~2.0.0",
"hyperf/memory": "~2.0.0",
"hyperf/paginator": "~2.0.0",
"hyperf/pool": "~2.0.0",
"hyperf/process": "~2.0.0",
"hyperf/redis": "~2.0.0",
"hyperf/utils": "~2.0.0",
"hyperf/validation": "^2.0",
"hyperf/testing": "^2.0",
"hyperf/constants": "^2.0",
"hyperf/crontab": "^2.0",
"hyperf/amqp": "^2.0",
"hyperf/view": "^2.0",
"hyperf/task": "^2.0",
"hyperf/session": "^2.0",
"hyperf/filesystem": "^2.0",
"xxtime/flysystem-aliyun-oss": "^1.5",
"duncan3dc/blade": "^4.8",
"hyperf/model-cache": "^2.0",
"phper666/jwt-auth": "~3.0.0"
},
"require-dev": {
"swoft/swoole-ide-helper": "^4.2",
"phpmd/phpmd": "^2.6",
"friendsofphp/php-cs-fixer": "^2.14",
"mockery/mockery": "^1.0",
"doctrine/common": "^2.9",
"phpstan/phpstan": "^0.12"
},

内存溢出了  有没有相关代码

内存溢出了 有没有相关代码

大概需要怎样的代码. 我是直接start服务就报错了

打断点查一下走到哪里爆炸的.
还有就是看一下有没有循环引用的地方   a 引用 b    b 引用 c     c  引用 a    这类的情况

php.ini 里的 内存设置小了.  我是默认的128M. 不够用.  设置为512可以跑了


Add support for new Firebase 3.x.x. authentication methods

There are a bunch of new authentication methods in the Firebase 3.x.x SDK which are not found in AngularFire 2.x.x. These include the following:

firebase.auth.Auth

applyActionCode()
checkActionCode()
confirmPasswordReset()
fetchProvidersForEmail()
getRedirectResult()
verifyPasswordReset()

firebase.User

getToken()
link()
linkWithPopup()
linkWithRedirect()
reauthenticate()
reload()
sendEmailVerification()
unlink()
updatedProfile()

The only reason we include the existing set of authentication methods is because those are the ones that were around in the Firebase 2.x.x SDK. Now that we have a bunch of new methods, we should probably include them in the $firebaseAuth service or else users will have to drop down to the vanilla Firebase SDK half of the time.
The counterpoint to adding them is that we end up just re-implementing a bunch of existing functionality in the vanilla SDK. Is there a different approach we can take to handle the $digest loop issues that doesn't involve use wrapping every method in the underlying SDK?
cc/ @katowulf, @abehaskins, @davideast - Thoughts?

We should also consider deprecating $getAuth() in favor of $getCurrentUser().

Is there a different approach we can take to handle the $digest loop issues that doesn't involve use wrapping every method in the underlying SDK?

Unfortunately not, any async event will need to be wrapped.
I think it's important to provide these methods to keep continuity for the developer, so I'm on board 💯.

Well, technically we have lots of terrible choices available!
We could monitor web sockets and trigger a compile when an incoming event is received (Angular 2 does some things like this internally; this one is actually not "terrible"). We could $interval( $timeout.bind(null, function(){}), 500 ) to trigger an automatic compile every half of a second. We could hack into the Firebase notification process and make it trigger $timeout.
And also not a completely terrible idea, although not a perfect one:  We could just provide a general wrapper that allows you to call pretty much anything that returns a promise and converts it to an Angular promise which trigger a compile. Thus you could do something like: angularfire.when( firebase.auth().signInWithPassword() ).then(...);
For that matter, we could just provide some instructions in the readme for "calling other SDK methods" which uses $q.when( ... ) to achieve the same result.

I'm not actually sure that promise idea would work Kato.
I'm pretty sure (based on my experience upgrading the tests) that Promises resolving don't trigger digestion, in fact it's the opposite - .then() calls wont be called until a digestion happens, so wrapping with $q.when would result in weird / inconsistent response times since the promises wouldn't resolve until something else triggered a digestion.

I'm fairly sure that's only true of the mock promises, or maybe has to do with the fact that we have multiple layers of $timeout and promises, which requires multiple flushes of the mock $q library. I'm not sure why they behave so strangely. But $q does trigger an eval on root scope.

the thing with $q is that when your promise is resolved (and also presumably on reject or notify), it invokes your handlers within $rootScope.evalAsync which ensures that after invocation, it'll trigger a digest and thus the rest of your application can have a chance to update or otherwise respond to the changes, just the way we like it.

This is also in the docs but too dense to link to and summarize quickly, so this non-authoritative answer is a better tl;dr.
But even if they don't trigger a compile (it's not necessarily a digest loop, btw) then a simple wrapper achieves the same end.

Having issues using vanilla 3.0 authorization in AngularJS, any eta on having 3.0 auth support?

@CTucker1327  this issue is about peripheral methods supported in the new auth SDK and not about vanilla auth. Auth is supported.
Please submit a new issue rather than responding to a tangentially related thread. Be sure to include an mcve and any useful debug output from firebase.database.enableLogging(true); and error logs.

Is someone working on implementing the methods @jwngr mentioned (didn't see any PRs)?
Do you need some help on those (maybe add label 'help-needed')?
Can you update on the progress?
Anyway, would love to help, just don't want to do work that someone is already doing

Nope, no one is working on this. But we would gladly enjoy some help and will take some PRs if you are willing to put them together! I think the first step would be putting together a proposal of all the methods to include. Once we have the API ironed out in this thread, the implementation can happen. Do you have a proposal for how we should organize all the new auth API methods?

I haven't looked at the code yet, but I believe the $firebaseAuth service is indeed the right place and it sould have a wrapping class around firebase's User and implement all methods by wrapping them with a promise generate by $q like $q.resolve() or $q.when().
Makes sence?

Maybe. The thing is that AngularFire currently has all user auth methods off of the $firebaseAuth service. However, the new 3.x.x Firebase SDKs have different methods off of firebase.auth.Auth and others off of `firebase.User. We should probably move in that direction as well with this library.
On a similar note, wrapping all 30 or so methods in a promise that works with the $digest loop seems like a lot of work for little benefit. I wonder if we should instead just document how users can do this themselves or provide a helper method that takes a Firebase promise and returns a new one that is properly wrapped so it works with the $digest loop. I haven't thought through all the details myself, but that would be a first step to do in order to update the lib.

So to take the new v3.x.x approach IMO the current $firebaseAuth service would be the firebase.User and we'll expose a new service to handle the firebase.auth.Auth service's methods.
About the wrapping thing, I don't think we have a choice. Writing an AngularJS module without handling change detection ($digest in this case) means ignoring the framework we're writing for. We could add a config (that should be on by default) opting whether to $digest but that would do the opposite of saving work that needs to be done :smile:

Let's not wrap the SDK if we have an alternative

About the wrapping thing, I don't think we have a choice.

A recipe or example of how to accomplish auth effectively would be a decent compromise on cost vs. value and take us most of the way to an answer here, while avoiding the excessive wrappage.
A generic service that accepts an auth promise?
Perhaps another alternative would be to provide an auth service that accepts an auth handler and handles the digest portion, without having to wrap all the methods. Usage might look something like this:
app.service('authenticateUsingEmail', function($firebaseAuth) {
    return function(email, password) {
         // handleV3Login represents our service that helps with $digest et al
         // it receives a promise representing one of the firebase.auth methods
         // and (maybe?) it returns a promise that resolves to authUser
         $firebaseAuth.handleV3Login(
               firebase.auth().signInWithEmailAndPassword(email, password);
         );
    };
});

app.controller('LoginController', function(authenticateUsingEmail, $scope) {
    $scope.signIn = authenticateUsingEmail;
}

Maybe we don't need either?
One other thought occurs to me here. If we simply monitor firebase.auth().onAuthStateChanged() and trigger $digest any time it is triggered, then maybe we don't need either of the above. One could simply call firebase.auth().<auth method>() and $digest could still be triggered appropriately since onAuthStateChanged() will be triggered accordingly.
I think I like this answer the best.

I looked at it the wrong way. @katowulf, I totally agree with your last point. Since there's already that beautiful listener for auth changes, we can depend on that for change detection.
The documentation should say implicitly that the framework assumes that this is the way consumers use. Maybe add some code samples.
The generic service seems redundant since consumer can use $q the exact same way.
So are you suggesting leaving the service as it is (keeping the User instance as an internal state that is exposed using $getAuth() or getCurrentUser()) and implement by simply calling the SDK's methods?

how to applyActionCode() in angularfire ?

Great, we just spent about one day running in circles while trying to figure out why getRedirectResult wasn't working / wasn't even there. Assumed it was an issue with require-js but as you understand that was a red herring.
Ok.

Note that even though AngularFire doesn't have these methods implemented just yet, you can always drop down to the vanilla JavaScript library while using AngularFire. You don't have to choose between them and can instead use both in conjunction.

forgot password reset link , see the video link completely solved
https://youtu.be/daGZO6YAkRg

Closing as stale. At this point we consider this library complete and are advising remaining developers move to Angular and @angular/fire.


Isotope mass is replaced by default mass when creating supercells

Hello there,
I am creating supercells for a fully deuterated material in Quantum ESPRESSO. The scf.in input file to create the supercells contains the following ATOMIC_SPECIES. Note that H has been provided with the deuterium mass (2 amu) in order to perform a deuterium calculation:
ATOMIC_SPECIES
   H    2.01410   H.upf
   C   12.01060   C.upf
   I  126.90400   I.upf
   N   14.00650   N.upf
  Pb  207.20000  Pb.upf

The bug occurs when creating supercells with
phonopy --qe -d --dim="2 2 2" -c scf.in

This creates supercells with the default mass of hydrogen instead of deuterium.
See the ATOMIC_SPECIES in all the new supercell files:
ATOMIC_SPECIES
 Pb  207.20000   Pb.upf
  I  126.90447   I.upf
  N   14.00670   N.upf
  C   12.01070   C.upf
  H    1.00794   H.upf

Of course, the expected behaviour would be to preserve the atomic mass introduced in the input, instead of the default one.
This behaviour was not observed when the atomic species symbol was unknown to Phonopy and was replaced by a random one, as mentioned in issue #412, where the atomic mass remained as originally introduced, although the atomic symbol was changed.

Phonopy doesn't read masses from scf.in. You have to specify them with mass tag, https://phonopy.github.io/phonopy/setting-tags.html#mass. If you want to support reading masses from scf.in, you have to modify https://github.com/phonopy/phonopy/blob/develop/phonopy/interface/qe.py. However, this is also related to crystal symmetry. When your input is not a primitive cell, then you have to carefully check the relationship between crystal symmetry and a list of masses of your input file. To avoid this complexity, phonopy accepts masses of primitive cell.

Sorry for the confusion, and thanks for the clarification. Moved the relevant discussion to #412.


feat: short circuit conditional

Short circuit conditional Pill 💊

please @all-contributors add @neodmy for code, content


bug: ion-item border color css variable does not work in production android apk

Bug Report
Ionic version:
4.11.5
Current behavior:
In the documents, ion-item border color can be styled by setting --border-color css variable. It works in case of debug android apk but when we run play store release apk, the border color shows as black instead of the custom color.
Expected behavior:
the play store release apk should show the custom border color.
Steps to reproduce:
Start with tabs starter and build for debug and production. In case of debug the border-color styled via css variable works but in case of production apk, the css variable does not work.
Related code:
.form ion-item, ion-toolbar ion-item,.searchbar-input.sc-ion-searchbar-md, form ion-item {
	border-radius: 4px 4px 4px 4px;
	border-color: #dde5ed; /* works in production apk */
	--border-color: var(--ion-color-border,#dde5ed) !important; /* does not work in production apk */
	margin-top:15px;
}

Other information:
if the border-color is set via standard css property, it works in production.
Ionic info:
Ionic:

   Ionic CLI                     : 5.4.8 (C:\Users\Gaurav's PC\AppData\Roaming\npm\node_modules\ionic)
   Ionic Framework               : @ionic/angular 4.11.5
   @angular-devkit/build-angular : 0.801.3
   @angular-devkit/schematics    : 8.1.3
   @angular/cli                  : 8.1.3
   @ionic/angular-toolkit        : 2.0.0

Cordova:

   Cordova CLI       : 9.0.0<EMAIL_ADDRESS>   Cordova Platforms : android 8.1.0
   Cordova Plugins   : cordova-plugin-ionic-keyboard 2.2.0, cordova-plugin-ionic-webview 4.1.2, (and 19 other plugins)

Utility:

   cordova-res : 0.3.0 (update available: 0.8.1)
   native-run  : 0.2.2 (update available: 0.2.9)

System:

   Android SDK Tools : 26.1.1 (C:\AndroidSDK)
   NodeJS            : v10.15.3 (C:\Program Files\nodejs\node.exe)
   npm               : 6.12.0
   OS                : Windows 10

Thanks for the issue. I am going to close this as a duplicate of https://github.com/ionic-team/ionic/issues/17781.
Please see https://github.com/ionic-team/ionic/issues/17781#issuecomment-636126472 for an explanation of why this issue is happening and what you can do to work around it. Thanks!


Cow: Watch softmetrica.ro

Cow requests the watch of the watch_keyword softmetrica\.ro. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

softmetrica\.ro has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by VLAZ in Charcoal HQ


More CLI flexibility

The CLI should also accept the config file via an environment variable LURK_CONFIG.
It should also have --no-env and --no-config flags to ensure that the behavior is 100% determined by command line arguments.

Related: https://github.com/lurk-lab/lurk-rs/pull/581#discussion_r1284748827

We should have a default config file location, eventually one for global and one for local settings.
It may be more user-friendly to not allow customization of config file locations, and instead document where they can be found.


Add option to specify selector in which Leaflet is loaded.

I am using a custom _table.html template. I want to surround the table with an element which has overflow-x: scroll so that my table scrolls within its container rather than filling the width of the page. But in this case the leaflet map is put inside the scrollable container too because the code is doing this:
let table = document.querySelector('table.rows-and-columns');    table.parentNode.insertBefore(el, table);

I've put a PR here: https://github.com/simonw/datasette-cluster-map/pull/7


Add favicon to NC Open Pass site

It would be great to have s small hexagon that shows up in the browser tab

Done


Revert "Disable asserts for non-empty nulls (#8183)" [databricks]

This reverts commit 6f3818939a44c087122658e2860c2bf4cd7e4792 and enables asserts for non-empty nulls
fixes #8184
fixes #8308

build

build

Looks like there are memory leaks seen in the CI. I didn't see them in my smoke test. I will try to recreate them and update this PR

This PR depends on https://github.com/rapidsai/cudf/pull/13597

build

Databricks is unreachable in the premerge

build

build

The DB is crashing because there is a double-free corruption happening on DB. I am currently working on root causing it

This is caused by a cudf::split returning a ColumnView with non-empty nulls. I have created an issue for ithttps://github.com/rapidsai/cudf/issues/13638
One thing to note is that DB CI is running cache_tests with PCBS enabled, while on other env we don't run PCBS tests

I have verified that rapidsai/cudf#13647 fixes the issue on DB and on spark-311 when tested locally. I will kickoff premerge for this once cudf nightly has been published

build

[2023-06-30T13:15:22.758Z] [gw0] [  0%] PASSED ../../src/main/python/arithmetic_ops_test.py::test_addition_subtraction_mixed[+-Decimal(15,3)-Decimal(4,2)][INJECT_OOM]
[2023-06-30T13:15:22.758Z] ../../src/main/python/arithmetic_ops_test.py::test_addition_subtraction_mixed[+-Decimal(15,3)-Decimal(3,-2)][INJECT_OOM] 23/06/30 13:15:22 ERROR Executor: Exception in task 1.0 in stage 24.0 (TID 97)
[2023-06-30T13:15:22.758Z] java.lang.ArithmeticException: Decimal(expanded,-7.6299917396276706084744903254669041E+39,36,-5}) cannot be represented as Decimal(38, 0).
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.sql.types.Decimal.toPrecision(Decimal.scala:369)
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.sql.types.Decimal.floor(Decimal.scala:522)
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.sql.catalyst.expressions.GeneratedClass$GeneratedIteratorForCodegenStage1.processNext(Unknown Source)
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.sql.execution.BufferedRowIterator.hasNext(BufferedRowIterator.java:43)
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.sql.execution.WholeStageCodegenExec$$anon$1.hasNext(WholeStageCodegenExec.scala:755)
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.sql.execution.SparkPlan.$anonfun$getByteArrayRdd$1(SparkPlan.scala:345)
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.rdd.RDD.$anonfun$mapPartitionsInternal$2(RDD.scala:898)
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.rdd.RDD.$anonfun$mapPartitionsInternal$2$adapted(RDD.scala:898)
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.rdd.MapPartitionsRDD.compute(MapPartitionsRDD.scala:52)
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.rdd.RDD.computeOrReadCheckpoint(RDD.scala:373)
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.rdd.RDD.iterator(RDD.scala:337)
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.scheduler.ResultTask.runTask(ResultTask.scala:90)
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.scheduler.Task.run(Task.scala:131)
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.executor.Executor$TaskRunner.$anonfun$run$3(Executor.scala:497)
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.util.Utils$.tryWithSafeFinally(Utils.scala:1439)
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.executor.Executor$TaskRunner.run(Executor.scala:500)
[2023-06-30T13:15:22.758Z] 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
[2023-06-30T13:15:22.758Z] 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
[2023-06-30T13:15:22.758Z] 	at java.lang.Thread.run(Thread.java:750)
[2023-06-30T13:15:22.758Z] 23/06/30 13:15:22 WARN TaskSetManager: Lost task 1.0 in stage 24.0 (TID 97) (premerge-ci-2-jenkins-rapids-premerge-github-7424-4gbrb-qvkh8 executor driver): java.lang.ArithmeticException: Decimal(expanded,-7.6299917396276706084744903254669041E+39,36,-5}) cannot be represented as Decimal(38, 0).
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.sql.types.Decimal.toPrecision(Decimal.scala:369)
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.sql.types.Decimal.floor(Decimal.scala:522)
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.sql.catalyst.expressions.GeneratedClass$GeneratedIteratorForCodegenStage1.processNext(Unknown Source)
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.sql.execution.BufferedRowIterator.hasNext(BufferedRowIterator.java:43)
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.sql.execution.WholeStageCodegenExec$$anon$1.hasNext(WholeStageCodegenExec.scala:755)
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.sql.execution.SparkPlan.$anonfun$getByteArrayRdd$1(SparkPlan.scala:345)
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.rdd.RDD.$anonfun$mapPartitionsInternal$2(RDD.scala:898)
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.rdd.RDD.$anonfun$mapPartitionsInternal$2$adapted(RDD.scala:898)
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.rdd.MapPartitionsRDD.compute(MapPartitionsRDD.scala:52)
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.rdd.RDD.computeOrReadCheckpoint(RDD.scala:373)
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.rdd.RDD.iterator(RDD.scala:337)
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.scheduler.ResultTask.runTask(ResultTask.scala:90)
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.scheduler.Task.run(Task.scala:131)
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.executor.Executor$TaskRunner.$anonfun$run$3(Executor.scala:497)
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.util.Utils$.tryWithSafeFinally(Utils.scala:1439)
[2023-06-30T13:15:22.758Z] 	at org.apache.spark.executor.Executor$TaskRunner.run(Executor.scala:500)
[2023-06-30T13:15:22.758Z] 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
[2023-06-30T13:15:22.758Z] 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
[2023-06-30T13:15:22.759Z] 	at java.lang.Thread.run(Thread.java:750)

Seems like INJECT_OOM is causing ArithmeticException to be thrown. Still looking into this more

So, only 340 still has memory leaks and the pre-merge fails due to JVM crash. Looking into this, will post an update ASAP

I am unable to repro the crash on my workstation after running the crashing test, which seems to be RegularExpressionTranspilerSuite from the log. I also ran OrcScanSuite which shows memory leaks in pre-merge but not on my workstation. Researching further into the version of cudf running on pre-merge as there may be a discrepancy there.

There is a separate bug for tracking this problem. Since it seems to be intermittent and most probably unrelated to this PR I will kick off the pre-merge again

build


impl_reflect! for EulerRot instead of treating it as an opaque value

Objective
Currently, Bevy implements reflection for glam::EulerRot using:
impl_reflect_value!(::glam::EulerRot(Debug, Default, Deserialize, Serialize));

Treating it as an opaque type. However, it's useful to expose the EulerRot enum variants directly, which I make use of from a drop down selection box in bevy_egui. This PR changes this to use impl_reflect!.
Importantly, Bevy currently uses glam 0.28.0, in which EulerRot has just 6 variants. In glam 0.29.0, this is exanded to 24 variants, see https://github.com/bitshifter/glam-rs/commit/bb2ab05613e98e9cbf24693473a0c4e7beb04e87. When Bevy updates to 0.29.0, this reflect impl must also be updated to include the new variants.
Solution
Replaces the impl_reflect_value! with impl_reflect! and a handwritten version of EulerRot with the same variants.
Testing
Added a tests module to glam.rs to ensure that de/serialization works. However, my main concern is making sure that the number of enum variants matches glam's, which I'm not sure how to do using Enum.

FYI @targrub you'll want to change this impl in #15249.


feat(text-field): implement in line version of TextField

📥 Proposed changes
Implement inline (in-text) version of TextField as discussed in #426 . This also adds a screenreader-only version of the label so it can be hidden without creating an empty <label> element.
☑ Submission checklist

[ ] I have read the CONTRIBUTING doc
[ ] yarn build works locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

💬 Further comments
affects: @fremtind/jkl-core, @fremtind/jkl-text-input-react, @fremtind/jkl-text-input,
@fremtind/jkl-typography-react
Implementation details

Repurposes the existing inline prop (which had no real effect) for this, which strictly speaking breaks current functionality; We should give a heads-up to the teams before publishing.
Adds both a manual and an automatic way of controlling input length: The new width prop takes a string, which is passed to the css width property as-is; If no width is passed, the field will use any maxLength value of less than 15 to decide the input length. This system ensures good affordance, and enables fine control.

Further implications
We should probably create similar versions of the Dropdown/Select and DatePicker components.

Still think we should avoid or at lease warn developers about props that override each other

Still think we should avoid or at lease warn developers about props that override each other

Yes, I agree. Will push a commit with a warning in dev now, and make a test branch with the inline version as a separate component!


ERROR in Unexpected value 'CanvasWhiteboardModule.  Please add a @NgModule annotation.

Hi,
I added 'CanvasWhiteboardModule' in app.module.ts. And canvas is used in one of my component. But when use
ng serve --aot
I am getting following error.
ERROR in Unexpected value 'CanvasWhiteboardModule in C:/projectfFolder/node_modules/ng2-canvas-whiteboard/dist/ng2-canvas-whiteboard.d.ts' imported by the module 'AppModule in C:/projectfFolder/src/app/app.module.ts'. Please add a @NgModule annotation.
i domt why i am getting this error. Please suggest

Hi and thank you for the report,
It seems that the library is outdated with Angular's Ahead of time Compilation.
This will be fixed with version 1.3.0.
Also, please update the @angular/compiler-cli dependency to version 1.2.4 to remove another AOT compilation problem (angular issue -> https://github.com/angular/angular-cli/issues/4551)

@sarathscs  please try the building with the 1.3.0 version and tell me if there are any problems again


Reviewed code

I just reviewed the code, specified types, fixed an overflow problem, separated the routes to a single file

Hello @anasfik !
Thank you so much for this contribution ❤
But please check comments and effective dart docs
https://dart.dev/guides/language/effective-dart/design


Failed to compile in iOS

I thought this was resolved by now based on all the other posts from last year.  But it looks like its still an ongoing issue.
The following build commands failed:
	CompileSwift normal armv7 /Users/macknife/dev/myApp/platforms/ios/myApp/Plugins/cordova-plugin-qrscanner/QRScanner.swift
	CompileSwiftSources normal armv7 com.apple.xcode.tools.swift.compiler
	CompileSwiftSources normal arm64 com.apple.xcode.tools.swift.compiler
	CompileSwift normal arm64 /Users/macknife/dev/myApp/platforms/ios/myApp/Plugins/cordova-plugin-qrscanner/QRScanner.swift

I tried adding the <preference name="UseSwiftLanguageVersion" value="5" /> to my config.xml file but the errors are still there.
I am reading that I need change: UIApplication.openSettingsURLString to UIApplicationOpenSettingsURLString - but where do I make this change?

well, turns out the same posts last year are still the fix here:
In the following file: myApp/platforms/ios/myApp/Plugins/cordova-plugin-qrscanner/QRScanner.swift
Update lines 471 and 484 - by replacing UIApplication.openSettingsURLString to UIApplicationOpenSettingsURLString`

Why isn't this update merged into the master?  Until it is, ppl better update the original plugin in the project in case you have to platform remove/add ios otherwise you will have to update the above file manually every time.

Please merge this into master.

Please merge this into master.

@rolinger @98teg @piwpio @iChristosK @Catscratch We've forked the repo and upgraded it to modern versions. Check https://github.com/nosoloforks/cordova-plugin-qrscanner (v4.0.0 is not published on NPM)

Maybe it helps to link @matiu  directly here.
I had the same issue just now

i met this error either. i try to fix it without forking a clone to change the source code.
as @rolinger  said, the error occurred because the Swift API changed.

well, turns out the same posts last year are still the fix here:
In the following file: myApp/platforms/ios/myApp/Plugins/cordova-plugin-qrscanner/QRScanner.swift
Update lines 471 and 484 - by replacing UIApplication.openSettingsURLString to UIApplicationOpenSettingsURLString`

we can change the cordova swift version to < 4.2, errors fixed
<platform name="ios">
    <allow-intent href="itms:*" />
    <allow-intent href="itms-apps:*" />
    <!-- you may update platform ios using `cordova platform rm ios` & `cordova platform add ios` -->
    <preference name="UseSwiftLanguageVersion" value="4" />
</platform>


Show trip duration

Packing is often directly depending on the number of days/night spend out of the house. I suggest showing this information somewhere. In further iterations, you could even have suggested items+amount based on templates (defined by the user, learned based on previous trip, etc.)

Yes I agree with you these would be nice features.
Adding trip duration in list or summary is quiet easy, and will be done.
Then we could mark items like "quantity equals nbre of days" like socks for example is something I also thought about.
Suggestions of items based on previous trips is already done and will be in next release. But suggested items are ordered on number of trips they appear in, a better sorting would be to order it based on the fact they appear on trips of similar durations.


sending "wrong password" to nickserv

I noticed that default settings on UnrealIRCD, the 'PASS' command sends a password to NickServ. So I'm temporarily using this method in settings to ident to NickServ on UnrealIRCD.


Add support for different architecture

Fixes #142
Added map for linux arch = binary name and auto selection of go arch binary (go_arch)

Could we use this architecture variable for downloading the right binary releases as well?
https://github.com/hyphacoop/cosmos-ansible/blob/7f28a4d6fcd1e0bdbf97ce99cfed01bbd852b525/roles/gaia/defaults/main.yml#L10

@YurkoWasHere could we use this architecture variable for downloading the right binary releases as well?

Yep,
Problem with atomic PRs, dependencies :) Would have been better if 143 would be merged before 138 :P
https://github.com/hyphacoop/cosmos-ansible/issues/122#issuecomment-119099a4489

Merged main into this branch
https://github.com/hyphacoop/cosmos-ansible/pull/143/commits/a1535021fddd7951d7f5864fa8940f8cc7a81963
Then added support for binary file arch
https://github.com/hyphacoop/cosmos-ansible/pull/143/commits/a3e0119f75677ae46ee46085c71a5e9efdbac793


[CARBONDATA-3118] Parallelize block pruning of default datamap in driver for filter query processing

TODO: After testing, change CARBON_DRIVER_PRUNING_MULTI_THREAD_ENABLE_FILES_COUNT = 100000
Parallelize block pruning of default datamap in driver for filter query processing
Background:
We do block pruning for the filter queries at the driver side.
In real time big data scenario, we can have millions of carbon files for
one carbon table.
It is currently observed that for 1 million carbon files it takes around 5
seconds for block pruning. As each carbon file takes around 0.005ms for
pruning
(with only one filter columns set in 'column_meta_cache' tblproperty).
If the files are more, we might take more time for block pruning.
Also, spark Job will not be launched until block pruning is completed. so,
the user will not know what is happening at that time and why spark job is
not launching.
currently, block pruning is taking time as each segment processing is
happening sequentially. we can reduce the time by parallelizing it.
solution:
Keep default number of threads for block pruning as 4.
User can reduce this number by a carbon property
"carbon.max.driver.threads.for.pruning" to set between -> 1 to 4.
In TableDataMap.prune(),
group the segments as per the threads by distributing equal carbon files to
each thread.
Launch the threads for a group of segments to handle block pruning.
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[ ] Any interfaces changed? NA

[ ] Any backward compatibility impacted? NA

[ ] Document update required? updated

[ ] Testing done
done manual testing with huge data.
Also ran all the UT testcases by making
CARBON_DRIVER_PRUNING_MULTI_THREAD_ENABLE_FILES_COUNT = 1

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1470/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9728/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1680/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1473/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1683/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9731/

please describe this PR

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9735/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1484/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1694/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9742/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1488/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1698/

Build Success with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9746/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1489/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1699/

Build Success with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9747/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1497/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9754/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1706/

@manishgupta88 , @ravipesala : please review

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1498/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1707/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9755/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1500/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1709/

Build Success with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9762/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1505/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1714/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1509/

Build Success with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9767/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1719/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1515/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1725/

Build Success with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9773/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1517/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1727/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9775/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1519/

Build Success with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9777/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1729/

LGTM

did a conflict resolve with documentation file. Let the build run again

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1528/

Build Success with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9786/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1738/

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1529/

Build Success with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9787/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1739/

retest this please

LGTM

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1543/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1754/

Build Success with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9802/


ABC has moved.

The ABC project has moved from bitbucket to github.  This breaks the master yosys makefile, which references a hard URL for ABC.  Further, ABC is also no longer maintained under mercurial.  It is now under git source code control.  A quick examination suggests that only the main Makefile needs to be patched, although the patch required is a touch more intrusive than just changing URL to the new location.

Thanks! I'll fix this right away.

OMG! This is awesome!

Fixed in 6f681c4.

So maybe we can have a new release with this fix in it soon?  It's been a while since 0.7, and now it doesn't even build anymore...  :wink:

@zeldin Uh! Yeah. That's a really good point.. I'll make a new release some time in May.


Handle tcp test ping in gedis server at the start at threebot start

at threebot start we do port check to make sure the server is up.
but this exception should be handled in gedis server with better log

I got this error while trying to start threebot.

This is due to a to a change in the threebot server. Need to delete old threebot isntance j.servers.threebot.delete('default')

verified.


How to send raiblocks without syncing

How can I export the private keys and send raiblocks (that was deposited in a wallet) without syncing that wallet ?
I ask because it is definitely not going to sync after so many hours (still havent sync 5%). I think there are some serious scalability issues with raiblock since it takes days to sync and its still very new (imagine if we have 100x more accounts), that would become impossible.
Now my raiblocks are stuck in a unsyncable wallet and I would like to know how to send that (just like it is possible to do with ether, bitcoin, litecoin. They can be sent by importing the private key into a light wallet then sending it. But i dont find how to do that for raiblocks. Thanks.

@marc477 I had the same problem. The workaround is take your local wallet seed and import it to their online wallet. Your balance would be available right away. Yes I agree they need to fix the local wallet sync problems ASAP.


code blocks contrast is very low

Hello,
I find that the code blocks are very hard to read, having a very low contrast. It can be quite disturbing especially for documentation with a lot of code examples.
inline code from the Bootstrap example

bigger code block, from a project of mine, even harder to read IMO

I know we can provide our own CSS rules on a per project basis, but I think default style should promote readability and comfort.
How do you feel about it ?

Now that you bring it up, I agree—the default styles should be more accessible. We’re using one of the default highlight.js themes I believe, so it’s just a matter of us replacing that with a more accessible one.
Would be nice to make that theme configurable via config too.
Thanks for raising this issue!


Support next 5.1 phases

Next 5.1 introduced phases in the next.config.js file so specific plugins can only get applied in a specific phase. We should also add support for it.
I'm not sure about the api yet, maybe something like this:
// next.config.js

module.exports = withPlugins([

  // only add next-offline on production build & production server
  [offline, {
    /* offline config */
  }, [PHASE_PRODUCTION_BUILD, PHASE_PRODUCTION_SERVER]],

  // add next-typescript to all phases (default if none is specified)
  [typescript, {
    /* typescript config */
  }],

  // add next-sass in every phase except on the development server (when first argument is '!' -> blacklisting phases)
  [sass, {
    /* sass config */
  }, ['!', PHASE_DEVELOPMENT_SERVER]],

], nextConfiguration);

Please comment if you have inputs or another idea :)

Good idea, I'll make this optionally available for plugin developers.
But I think I'll still also make this available in the next.config.js file if a plugin doesn't add support for it so users can still decide (or overwrite) the behavior. It's a nice-to-have feature as Next.js officially added support for phases.

yeah, especially given that there are no other ways to separate dev plugins from prod plugins causing even dev plugins to included in main dependencies of package.json.  At the run-time, composition should be intelligent enough to ignore missing dev plugins deps, which may be used with conditional import of such plugins so as to not throw any js error.

Hi @prateekrastogi, version 2 of next-compose-plugins just got released 🎉 It includes many improvements like phase specific plugins and/or configuration, additional information/possibilities for plugin developers and optional plugins (e.g. when they are only in the devDependencies).
I would suggest you can check out the updated readme.
For the optional (dev) dependencies, you can take a look at the optional plugins section. Would this resolve the problems you had described? Or do you still have a use-case which would not be solved now?
Thanks again for your inputs and feel free to create new issues when you have more ideas or a problem with the newly introduced functionalities :)

@cyrilwanner Tried using the updated plugin in my projects and its working perfectly. Just a little clarification regarding optional plugin that some optional plugins may be dev dependencies, but do not necessarily need to be run in PHASE_DEVELOPMENT_SERVER . They may run in some other phases. For example, next-bundle-analyzer needs to be run in PHASE_PRODUCTION_BUILD, but will generally be preferred to included as dev dependency. So, may be removing  "and so you only want to apply it during PHASE_DEVELOPMENT_SERVER"  line from readme will help avoid confusion.

@prateekrastogi thank you for testing the updated plugin! You are right, that makes total sense, I'll remove this line from the readme.


Disable PKCS5 module

@zvolin and I have been discussing the PKCS5 implementation in MbedTLS and it seems pretty broken and untested. e.g. https://github.com/Mbed-TLS/mbedtls/issues/7038#issuecomment-1549965210. We should disable it.

From https://github.com/fortanix/rust-mbedtls/pull/213, you can see that although I removed the support of pkcs12 support in rust side, I still turn on the C define of it because some other code in C side is actually using pkcs12 (mainly in TLS for parse certificates). And this also applies to pkcs5.
And also the related function  pbkdf2_hmac which used pkcs5_pbkdf2_hmac_ext is  currently used by our internal projects.
So I think this issue need to be put off or declined.


InteractionCreate Event - "sendEmbed" entfernen

[ ] removing the function and adding simple reply to the catch.