Investigate references returned in an if statement in new mem2reg pass

Problem
When tracking aliases, one case to consider is when a reference is returned from an if statement. In this case, the block parameter becomes a possible alias for each reference returned from each branch of the if. This case is under tested however since returning references from an if with a witness as its condition is currently not allowed (on master, unchanged with the new mem2reg pass).
Happy Case
We should test this code more and ensure aliases are still tracked correctly in this situation. Ideally we should add a test which conclusively shows this.
Alternatives Considered
No response
Additional Context
No response
Would you like to submit a PR for this Issue?
No
Support Needs
No response

@jfecher what is the status of this issue?


Cannot get BotState services because by default BostState are tranformed into CachedBotState

Hi,
At runtime when the key requested doesn't exists in AzureBlobStorage a new CachedBotState is added in the turn services collection. Then I cannot access the original BotState object to create the assessor
var userState = dc.Services.Get<UserState>();
or
var userState = dc.Services[nameof(UserState)] as UserState;
userState is null because of the impossible cast to BotState.CachedBotState
var property = userState.CreateProperty<T>(typeof(T).Name, () => new T());
Do you have a best way to retrieve the service and access the properties or did I miss something for instance initialization?
Thanks

@drub0y for me it isn't a question but more an issue.
Using the underling private object CachedBotState directly in the services collection prevent any use of the BotState and the state management methods.

@ukphillips in the master branch version this extension doesn't exists.
I created my own extensions but in the implementation I have faced the issues mentioned in this thread.
I managed to tackle this by turning the CachedBotState to public.

Good point. I was just looking and it appears a lot around state has changed quite a bit. Looking at the latest code and the tests, it's definitely a little confusing how to use it. Loading the state sets the service (CachedBotState as you mentioned which is private) this doesn't quite feel right, but it would mean that you need that as a public type as you has solved it.
It seems like there should be a more natural way to access the objects outside of the services collection - Like returning the IDictionary<string, object> from the LoadAsync method but reserving the CachedBotState internally for later hash comparing to avoid unnecessary storage round trips.

Sadly I cannot access UserState, ConversationState or a custom BotState actually because of the problem I mentioned, CachedBotState is inaccessible.
Anyway GetPropertyValueAsync is internal so I would have to do some plumbing with a custom BotState manager.

Of course I know that the implementation is not final but I need SSO features...So for now this workaround is good enough for my needs.
Thanks for pointing me to this PR I'll take a look.

I also ran into the DI and middleware issue (or atleast a dependency of the middleware). As a short term fix, I explicitly resolved it when I created the middleware in configure services:
services.AddTransient<ITranscriptLogger, CosmosTranscriptLogger>();
var sp = services.BuildServiceProvider();
var service = sp.GetService<ITranscriptLogger>();
...
options.Middleware.Add(new TranscriptLoggerMiddleware(sp.GetService<ITranscriptLogger>()));
not ideal but it let me side step the issue for now.

Having pulled down the daily builds I am starting to see with some of the issues caused by the latest changes. Based on a few samples I have seen posted, there is an accessor being registered as a singleton after the bot that wraps the variaous IStatePropertyAccessor properties. This works fine to get the injection to work into the Bot main class, but I think I'm running into the same issue as @GhostAtom where I want to inject that into a piece of middleware. So I end up with something looking like this:
services.AddBot<FlexAssistantBot>(options =>
            {
                options.CredentialProvider = new ConfigurationCredentialProvider(Configuration);

                IStorage dataStore = new MemoryStorage();

                options.State.Add(new ConversationState(dataStore));
                options.State.Add(new UserState(dataStore));

                // Forces storage to auto-load on a new message, and auto-save when complete.
                // Put at the beginning of the pipeline.
                options.Middleware.Add(new BotStateSet(options.State.ToArray()));
                options.Middleware.Add(new CustomMiddleware());
            });

            services.AddSingleton<StateAccessors>(sp =>
            {
                var options = sp.GetRequiredService<IOptions<BotFrameworkOptions>>().Value;
                if (options == null)
                {
                    throw new InvalidOperationException("BotFrameworkOptions must be configured prior to setting up the State Accessors");
                }

                var userState = options.State.OfType<UserState>().FirstOrDefault();
                if (userState == null)
                {
                    throw new InvalidOperationException("UserState must be defined and added before adding user-scoped state accessors.");
                }

                var conversationState = options.State.OfType<ConversationState>().FirstOrDefault();
                if (conversationState == null)
                {
                    throw new InvalidOperationException("ConversationState must be defined and added before adding conversation-scoped state accessors.");
                }

                var accessors = new StateAccessors
                {
                    CustomUserState= userState.CreateProperty<CustomUserState>(nameof(CustomUserState)),
                    CustomConversationState= conversationState.CreateProperty<CustomConversationState>(nameof(CustomConversationState))
                };

                return accessors;
            });

So at this point with the same examples I can get my StateAccessors into the bot constructor, but how can I get it into my CustomMiddleware. It has to be registered with a new instance, and I can't register my StateAccessors type until after I have already created the bot (and middleware).
Specifically using samples from this PR https://github.com/Microsoft/botbuilder-dotnet/pull/809

So @ukphillips I'm definitely not a C# expert and this part of the code I'm a little less familiar with but let me take a stab at answering your question...
The moment you create ConversationState or UserState you can call CreateProperty() to define a property accessor. So you should be able to do something like this:
services.AddBot<FlexAssistantBot>(options =>
            {
                options.CredentialProvider = new ConfigurationCredentialProvider(Configuration);

                IStorage dataStore = new MemoryStorage();
                var convoState = new ConversationState(dataStore);
                var userState = new UserState(dataStore);

                options.State.Add(convoState);
                options.State.Add(userState);

                var customProp = convoState.CreateProperty("custom");

                // Forces storage to auto-load on a new message, and auto-save when complete.
                // Put at the beginning of the pipeline.
                options.Middleware.Add(new BotStateSet(options.State.ToArray()));
                options.Middleware.Add(new CustomMiddleware(customProp));
            });

If you then needed your bot to have access to the property as well, you would need to use dependency injection to get the accessor into the bot.

@GhostAtom @ukphillips Just a few comments to close out on this topic.
The starting point and very much the primary abstraction you should be looking at is the accessor. The higher level components, for example, the Dialogs, are coupled only to the accessor. The accessor pattern provides a level of decoupling and significantly some level of name collision avoidance.
As @drub0y points out much of that is internal and you should be focused on the accessor interfaces.
The BotState and CachedBotState classes are ultimately component parts of an implementation of the accessor. There is also plugable model here for different types of physical storage (for example, blob and CosmosDB) but whatever you chose at that level you still get the semantics of BotState.
Incidentally you don't really use BotState directly - instead you subclass it and use that. In the framework you will see subclasses called UserState and ConversationState. In the unit tests you will see more.
The BotState semantics are essentially "last write wins." That might work just fine for you. In a scaled out scenario (in fact, even with multiple concurrent requests) you might observe the implications of the "last write wins" semantics: concurrent state updates can be overwritten. This might not be an issue for you, but if it is, the solution is to implement your own accessor.
If you were to do that you would probably be also implementing some specific logic in your IBot OnTurn, for example, if in your implementation you were to fail a Save because of eTag precondition checks, then you might like to re-fetch the state and retry the logic. In such a scenario you might have also chosen to buffer outbound responses until you see a successful save. There are various alternatives here. However, it is important to understand that this isn't the behavior you get when you use BotState. I'm describing the kind of thing an advanced implementation might decide to do. But given the accessor abstraction such highly involved and complex state solutions can be built. The important point is that they will work with the Dialogs and Middleware.
Another piece that is tied into the BotState world is the AutoSaveStateMiddleware. The idea here is that the BotState accessors support a somewhat hidden lazy load with the middleware supporting a somewhat hidden and optimized save. Just add the middleware and you get the save happening, hence the name "auto."  The save is optimized in a couple of ways: firstly it attempts to only save when things have changed and secondly it attempts to perform multiple BotState saves in parallel.
The AutoSaveStateMiddleware makes a lot of sense when you chose to play with state inside of your own Middleware components. If you don't have any such components in play then there is no difference between using this and adding the SaveChanges calls to the end of your OnTurn logic.
Hopefully this explanation of some of the moving parts was helpful.


Missing $GXX definition when building tutorials with nocuda

Tutorial examples build fine with make (which uses nvcc), but building the nocuda targets yields an error:
[eric@alfred 1_basic_concepts]$ make nocuda
I/home/eric/CARL/include -L/home/eric/CARL/lib -pthread -D __NO_CUDA__  src/main_basic_concepts.cpp -o basic_concepts -lcarlsim
make: I/home/eric/CARL/include: No such file or directory
make: [Makefile:92: nocuda] Error 127 (ignored)

It seems the $GXX variable in the Makefile is not defined:
nocuda: $(main_src_file) $(inc_files) $(cpp_files) $(obj_files)
	$(eval CARLSIM5_FLG += -D __NO_CUDA__)
	$(GXX) $(CARLSIM5_FLG) $(obj_files) $< -o $(proj_target).out $(CARLSIM5_LIB)

Just a typo.  Replacing GXX with CXX fixes Tutorials 1 and 8.


Comment out iso_to_av96 and update cython version

Two build issues

The Kivy of today depends on a newer version of cython. Update it.
The chdkptp.py upstream HEAD has a different API than the one included in the Rpi image

The first exception is the method name iso_to_sv96 has been changed to iso_to_av96. I updated the method call and expected it to just work. Then I got a new exception when calling the updated method.
>>> chdkptp.util.iso_to_av96(100)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python2.7/dist-packages/chdkptp/util.py", line 7, in iso_to_av96
    return global_lua.globals.exposure.iso_to_av96(iso)
TypeError: 'NoneType' object is not callable

I searched through the pi-scan codebase for the string svm and it doesn't appear anywhere except in this option dict, which I think is trying to set the ISO to a value of 100. I commented this out and things are working as expected.
I would like to bring the API up to date but it's not directly related to my project. If it's cool with you the comment seems to fix the functionality on my end.

Take a look at my chdkptp.py fork here:
https://github.com/duerig/chdkptp.py/commits/master
This (and the other commit) are essential bugfixes. I meant to make a PR for Johannes, but I see that I have forgotten. Now that is rectified here: https://github.com/jbaiter/chdkptp.py/pull/5
The iso_to_av96 thing isn't an interface change. It was just a bug in the original code. If we don't set ISO at all, then the quality of the resulting image will be reduced. When I was playing around with this, fixing the bug made a significant difference to apparent graininess.
I'm also working on integrating the build script into the new pi-gen script that generates Raspberry Pi images. I should have that done this week. I'll pull in your build change to that.

Johannes as now accepted my pull request fixing the chdkptp.py bug. So pull the latest version from him, revert that line, and things should work.
Regarding falling back to the local filesystem: most Pi Scan installations are running on read-only filesystems. An overlay makes them look read-writable but any changes will disappear on reboot and writing to them consumes RAM. So for your fallback to work consistently on other installations, it needs to check for this case and only fallback if the local filesystem is truly read-writable and not just a copy-on-write ramdisk.


Proper use of polymorphism / inheritance in Open API 3

I just want to make a correct specification with basic inheritance anywhere in the schemas, be able to show it properly in the swagger editor and generate some code with it, but I can't. I am clearly missing something in Open API 3.
In the specification document (3.0.2), an example uses oneOf, and optionally the Discriminator block :
    MyResponseType:
     oneOf:
      - $ref: '#/components/schemas/Cat'
      - $ref: '#/components/schemas/Dog'
      - $ref: '#/components/schemas/Lizard'
     discriminator:
        propertyName: petType

In this scenario, the only constraint that Cat, Dog and Lizard must satisfy is the mandatory field petType, and a predetermined value.
Then, one can read :

To avoid redundancy, the discriminator MAY be added to a parent schema
definition, and all schemas comprising the parent schema in an allOf
construct may be used as an alternate schema.

I find it nice since it offers a proper use of the natural inheritance of the 3 entities :
components:
  schemas:
    Pet:
      type: object
      required:
      - petType
      properties:
        petType:
          type: string
      discriminator:
        propertyName: petType
        mapping:
          dog: Dog
    Cat:
      allOf:
      - $ref: '#/components/schemas/Pet'
      - type: object
        # all other properties specific to a `Cat`
        properties:
          name:
            type: string
etc

By the way, the spec is a bit ambiguous with

The discriminator object is legal only when using one of the composite
keywords oneOf, anyOf, allOf

Pet does not contains any of these keywords !
However, this second example does not show the consequences on the MyResponseType block.

Does it stay the same ? If so, what should be generated, say, in Java ? A MyResponseType and a Pet classes ? What should the MyResponseType class look like ?
Should MyResponseType be removed from the file, and Pet used instead ? If so, the swagger editor is not capable of tracing the subtypes anymore...
Then, should we add the oneOf to the Pet schema, so that the swagger editor can display the subtypes ? I've tried, it works, and validators say it's Open API 3 compliant, but warn that a cycle gets in the way. It seems to me that this would be formally wrong : an http body with a Cat (petType Cat) with all the Cat and Dog properties would validate because of the oneOf Cat/Dog inherited from Pet. Am I right to interprete it this way ?

It may be correlated with [#403]

Could you take a swing at a PR to visualize what you think is correct? Then we can shape that PR into a useful wording improvement.

@philsturgeon I am a bit confused by your suggestion, as I still don't know what is both correct towards Open Api3 and functional in the swagger editor and generators. It's seems to me that this very simple situation is a dead-end. Let met put it another way :
Let's consider the general case where MyResponseType is used as the type of a field X in another schema.

if MyResponseType is used without Pet, the description of the json is correct, but generators are lost when generating the field X that could contain one of the three possibilities.
if MyResponseType and Pet both exist, common Java generators are lost too (they should generate a field X of type Pet, but in reality, it is not what happen)
if Pet is used all alone, then the explicit knowledge of its subclasses is lost, therefore the swagger editor will only show the field X with the type Pet
if this knowledge is added in Pet (oneOf...), then the constraints are wrong towards the desired json (and it does not make any sense that a Cat inherits "one of" the other animals properties)

I would eliminate 1 (impossible to generate accurately), 3 (useful information lost for editors) and 4 (incorrect constraints).
So... maybe the correct approach is the second one, as it is the neatest description possible. Maybe it's only a concern for generators. They should implement a bit of logic : get rid of MyResponseType and choose Pet as the type of the field X.

What I mean is that from the words being used I am not sure I'm following, but a PR could make the discussion more complete in my head.
That said, with a discussion about potentially deprecating discriminator, I wonder if if the examples and discussion in here can take care of your use case? #2143

@PierrePrimot I I landed here after reaching the exact same conclusions you did, did you ever find a decent way out of this? I'm considering making the Pet contain the oneOf for Cat & Dog, then creating a fake PetInternal  that Cat and Dog can allOf. But assuming it is even legal OAS, I have no idea if generators are going to like it.

Consider in the case of "allOf" , i don't want to use one of the properties from my reference object , how should I achieve that?
example :
recipientAddress:
allOf:
- $ref: '#/components/schemas/Addresses'
Addresses:
properties:
addressCategory:
type: string
maxLength: 10
description: Type of Address
enum:
- PERMANENT
- COMMUNICATON
example: PERMANENT
addressLine1:
type: string
maxLength: 40
description: First line of the Address
example: host tower
here i dont want property "addressCategory" in my object recipientAddress.

i don't want to use one of the properties from my reference object

allOf:
  - $ref: '#/components/schemas/Addresses'
  - not:
    required:
    - addressCategory

i don't want to use one of the properties from my reference object

allOf:
  - $ref: '#/components/schemas/Addresses'
  - not:
    required:
    - addressCategory

Hello! Is it possible to have a structure like the following, or any other structure that could allow us to have "required" or "not required" structures per $ref within anyOf?
anyOf:
  - $ref: '#/components/schemas/Addresses_1'
	not:
		required:
			- addressCategory_1
  - $ref: '#/components/schemas/Addresses_2'
	not:
		required:
			- addressCategory_2

Any news here?

oneOf confuses me. Lets say the above was a property in an object rather than the return value from a web API call. How would a code generator know what base class to use for the type of the property? It would have to check the base classes for each of the listed subclasses and find the most top-level one? Personally, I don't think oneOf makes any sense for this. Why not, rather than enumerating the subclasses, just list the base class name and the discriminator field?

@jemiller0 because all of this is trying to leverage JSON Schema, which was not built with code generation in mind at all.  It's a constraint system, and things like oneOf and anyOf make a lot of sense as dynamic constraints.  But not as a static type generation system.
OAS 3.1 allows using JSON Schema extension vocabularies, and the hope was for improved code generation support through that mechanism.  So far that has not happened, but some major tooling only recently caught up to OAS 3.1 so despite being several years since its release, it's still early in terms of people working with it.  We'll see what develops - if nothing does, OAS 4 (Moonwalk) might have to take a different approach (IMHO).

This issue kind-of turned into a grab-bag of questions, most of which aren't addressable by spec changes so I am going to close this.  Here are answers to the questions that don't already seem to be answered here:

@PierrePrimot I would assume that your Pet schema replaces MyResponseType.  If the question is more generally about allOf and discriminator, that's being tracked as #2520, #3424 and #3591
@hirleydayan if you are using 3.1 that should work, if 3.0 you need to use allOf to combine the not keywords with the $ref as keywords adjacent to $ref are ignored in 3.0
@jemiller0 I think you just want to use the allOf case (see bullet point above for various relevant issues about clarifying that)


I am trying to run a stager but it is failing

On the target device i put in the stager command to connect it with the koadic CC. However when it is ran it pulls up a blank white window and does nothing afterwards any help would be appreciated.
-sid

Hi,
Are you fixed that issue?


Does MA supports memory mapping?

I have many samples to run, and the reference database is very large. So loading the database from disk to memory takes much more time than alignment. So I was wondering wether MA supports memory mapping to void loading database into memory every time?

Thank you very much for the solution. Sounds practical. I will close this issue.


ci: Add circle ci

Pull Request Test Coverage Report for Build 107

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 92:
0.0%

Covered Lines:
159

Relevant Lines:
159

💛  - Coveralls

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[3.2.0] Inconsistent behaviour when an Internal/publisher user try to edit the Runtime Configurations of an API Product

Description:
After creating an API Product, if a user with the Internal/publisher role goes to check the Runtime Configurations of that API Product he/she will be shown an Unauthenticated request error in the UI as shown below.

Even though that error is displayed, the user can edit the API Product and click on Save without a problem. This behaviour is confusing and need to be fixed in a consistent manner.
Environment details (with versions):

OS: Ubuntu 20.04.2 LTS

This is expected when using the same browser due to cookies/headers which include partial tokens of REST API. When you logout from the previous user the token will be revoked. But if the browser tries to send the token which is already revoked and this situation happens.
Related issue [1]
[1] https://github.com/wso2/product-apim/issues/10601


Implement interface adds an extra | undefined to parameter

TypeScript Version:  4.0.0-dev20200702

Search Terms:

quick fix
implement interface

Code
With strict mode enabled. For the TypeScript:
interface IFoo {
    bar(x?: number | string): void;
}

class Foo implements IFoo {
}

Run implement interface on Foo
Expected behavior:
bar is marked with a ?:
class Foo implements IFoo {
    bar(x?: string | number): void {
        throw new Error('Method not implemented.');
    }
}

Actual behavior:
bar is marked with both ? and its type also now has  | undefined
class Foo implements IFoo {
    bar(x?: string | number | undefined): void {
        throw new Error('Method not implemented.');
    }
}

Playground Link:
Related Issues:

Pretty sure we have logic in the type printer to automatically display these, but check with @weswigham on the matter.


WIP – Update and enforce assessment validations

What problem does this PR fix?
Unclear and unenforced validations around assessment attributes in code.
What does this PR do?
Enforces validations for MCAS, STAR, and DIBELS rows based on analysis of what the raw data coming in from Somerville's Aspen/X2 looks like.

TODO: Branch Somerville/New Bedford.

Closing this and going with a different approach: adding in validations incrementally.


TFTP Sequence Wraps Not Working

TFTP sequence wraps aren't working. After we hit block 65535, we wrap around to sequence number 0 again, and start logging "Ignoring duplicated ACK received for block 65535" until the client gives up. See https://www.cloudshark.org/captures/07ebe14c792b for a packet capture. I've trimmed the packets between 414 and 415.
Fixing this may be as simple as taking the sequence numbers modulo 65536 when checking if we have a duplicate (or out of sequence) ack.

The proposed fix seems sensible, the relevant sections being in Client.handle()

@icb- if you want to make a branch based off of development and submit a PR to development once you have a fix that'd be awesome. Thanks!


image: Add framebuffer support

Tested and confirmed to work with the caca, chafa, jp2a,  pixterm, and w3m backends, although w3m is the only one that actually uses the framebuffer as far as I know. termpix failed to compile with my Rust version, and the rest are only for certain terminal emulators.

How does one test whether this works or not? Just using a framebuffer terminal (like yaft) enough?

I've only tested it on the Linux console, but that should probably work, too.


forObject is expecting an Object not its Class

Closes: https://github.com/quarkiverse/quarkus-mybatis/issues/10

Coverage increased (+0.009%) to 87.258% when pulling 9466ddf19f861654fcea0ed9872be67d1df187e7 on zakkak:fix-null-meta-object into 193e6fa7ef3c7eb07d319adafa3f8822cb1af4a5 on mybatis:master.

Thank you, @zakkak and @zhfeng !

@harawata yeah, I'd verified it with the lastest mybatis in quarkus-mybatis and it works. Thanks !


Fixup zeroex OrderRecord

In order to implement a settlement encoding strategy for an upcoming ZeroExInteraction we basically have to pass an OrderRecord to a smart contract function.
This showed that some fields didn't have the correct type. I also added 2 other things which makes working with the type more ergonomic.
Changes:

remaining_fillable_taker_amount, maker_amount, taker_amount, taker_token_fee_amount: U256 => u128
pool: Bytes => H256
salt: String => U256
expiry: DateTime<Utc> => u64 (retain_valid_order() now does the conversion to check validity)
ZeroExSignature fields are now public
added helper function remaining_maker_amount which gives the scaled amount for partially filled orders

Test Plan
added an unit test for deserializing an OrdersResponse

Codecov Report

Merging #1706 (b8fbf4c) into main (4b5d81a) will increase coverage by 0.04%.
The diff coverage is 79.45%.

@@            Coverage Diff             @@
##             main    #1706      +/-   ##
==========================================
+ Coverage   65.51%   65.56%   +0.04%
==========================================
  Files         179      179
  Lines       35861    35924      +63
==========================================
+ Hits        23496    23553      +57
- Misses      12365    12371       +6


CLOUDP-75269: Simplify PreRunE and PreRunEOrg

Proposed changes

Jira ticket: CLOUDP-75269

Checklist

[x] I have signed the MongoDB CLA
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added any necessary documentation (if appropriate)
[x] I have run make fmt and formatted my code

Further comments

@gssbzn ❓ ❓ ❓ I would like to know what you think about this solution ❓ ❓ ❓


W/AuthSignInClient: Could not launch sign in Intent. Google Play Service is probably being updated

got this error from android emulator
here is the log from logcat
09-23 10:16:49.526 3098-9186/com.go2 V/FA: Recording user engagement, ms: 173834 09-23 10:16:49.528 3098-9186/com.go2 V/FA: Using local app measurement service 09-23 10:16:49.538 3098-9186/com.go2 V/FA: Activity paused, time: 848943 09-23 10:16:49.538 3098-3098/com.go2 V/FA: onActivityCreated 09-23 10:16:49.542 3098-9186/com.go2 D/FA: Logging event (FE): _e, Bundle[{_o=auto, _et=173834, _sc=MainActivity, _si=-1396380087099398574}] 09-23 10:16:49.551 3098-3098/com.go2 W/AuthSignInClient: Could not launch sign in Intent. Google Play Service is probably being updated... 09-23 10:16:49.561 3098-3098/com.go2 V/FA: Local AppMeasurementService is starting up 09-23 10:16:49.565 3098-9186/com.go2 V/FA: Using local app measurement service 09-23 10:16:49.565 3098-9186/com.go2 V/FA: Connection attempt already in progress 09-23 10:16:49.579 3098-3098/com.go2 V/FA: Bound to IMeasurementService interface 09-23 10:16:49.582 3098-9186/com.go2 V/FA: Connected to service 09-23 10:16:49.582 3098-9186/com.go2 V/FA: Processing queued up service tasks: 2 09-23 10:16:49.609 3098-9186/com.go2 V/FA: Activity resumed, time: 849004 09-23 10:16:49.613 3098-9186/com.go2 V/FA: Logging event: origin=auto,name=_e,params=Bundle[mParcelledData.dataSize=132] 09-23 10:16:49.617 3098-9186/com.go2 D/FA: Unable to get advertising id: java.lang.NullPointerException: Attempt to invoke virtual method 'java.lang.String com.google.android.gms.ads.identifier.AdvertisingIdClient$Info.getId()' on a null object reference: com.google.android.gms.internal.zzaua.zzfG(Unknown Source) 09-23 10:16:49.641 3098-9186/com.go2 V/FA: Saving event, name, data size: _e, 69 09-23 10:16:49.642 3098-9186/com.go2 V/FA: Event recorded: Event{appId='com.go2', name='_e', params=Bundle[{_o=auto, _et=173834, _sc=MainActivity, _si=-1396380087099398574}]} 09-23 10:16:49.645 3098-9186/com.go2 V/FA: Upload scheduled in approximately ms: 1742346 09-23 10:16:49.651 3098-9186/com.go2 V/FA: Background event processing time, ms: 39

We have an example app for you to check out. Example.
Make sure your emulator has Play store available else this library won't work on the emulator.
You can verify this in android studio AVD.
If this issue still persists please open a new issue with a reproducible example so we can debug it for you. Thank you.


updated to the 2.1.0 release of fabric8

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


helm not using the updated values.yaml

Hi,
I am facing an issue with helm upgrade where it is not picking up the latest values in values.yaml unless i pass it via --values flag.

I deployed my app using helm upgrade --install baster helm/ --values helm/values.yaml initially
The values.yaml basically has many variable details along with the value for image to be used in our application
The image is build and stored in image registry for every commit i make and so the image value changes in values.yaml.
Since I felt helm upgrade --install baster helm/ --values helm/values.yaml is lengthy and anyway helm pics up the values.yaml by default, i changed it to  helm upgrade --install baster helm/
What surprised me was it did not pick the new image that the values.yaml had, instead it kept using the old values that was used previously when i manually fed it with command line.

The values.yaml should be picked by helm everytime right or does it saves some state at the backend ?
Any help would be appreciated. Thanks!
Output of helm version:
version.BuildInfo{Version:"v3.0.2", GitCommit:"19e47ee3283ae98139d98460de796c1be1e3975f", GitTreeState:"clean", GoVersion:"go1.13.5"}
Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"16", GitVersion:"v1.16.9", GitCommit:"a17149e1a189050796ced469dbd78d380f2ed5ef", GitTreeState:"clean", BuildDate:"2020-04-16T11:44:51Z", GoVersion:"go1.13.9", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"16", GitVersion:"v1.16.9", GitCommit:"a17149e1a189050796ced469dbd78d380f2ed5ef", GitTreeState:"clean", BuildDate:"2020-04-16T11:36:15Z", GoVersion:"go1.13.9", Compiler:"gc", Platform:"linux/amd64"}
Cloud Provider/Platform (AKS, GKE, Minikube etc.):
We deployed k8s using kubespray over openstack VMs.

The dir structure:
helm:

Chart.yaml
templates/

....

values.yaml

grep -vi image helm/values.yaml (built a new image and generated values.yaml with latest image tag)

image: containers.cisco.com/ngcs/baster_api:836ddb4d
image: containers.cisco.com/ngcs/baster_collector:836ddb4d

helm upgrade --install baster helm/ --dry-run --debug | grep -vi image (this just pics the old image)

image: containers.cisco.com/ngcs/baster_api:39b0a00f
image: containers.cisco.com/ngcs/baster_collector:39b0a00f

helm upgrade --install baster helm/ --values values.yaml --dry-run --debug | grep -vi image (only if i feed in manually i get the latest image)

image: containers.cisco.com/ngcs/baster_api:836ddb4d
image: containers.cisco.com/ngcs/baster_collector:836ddb4d

@Arjunkrisha If you want to reuse values already set when first run and then set a particular value then use --set --reuse-values flags.  Some details in the docs: https://helm.sh/docs/intro/using_helm/#the-format-and-limitations-of---set and https://helm.sh/docs/chart_best_practices/values/#consider-how-users-will-use-your-values

@hickeyma Thanks for the quick response.

My question is why helm upgrade is not using the values.yaml.
From my experience i am understanding that helm is saving the values only during installation.
Next time when i do a upgrade it is not picking values from the values.yaml present in the chart directory, instead it is just uses the values that it saved initially.
So the users are forced to feed the values.yaml using --values or -f flag.

Personally i believe helm should be dynamically able to pic up the values present in values.yaml during each upgrade. Do you think this could be a new feature helm team can work on ?
Or probably there is a reason behind this behaviour which i am not aware of. I

@Arjunkrisha Values are saved during an upgrade. You however need the --reuse-values flag in conjunction with --set or --values flag to use the previous values. Otherwise, it uses the default values from the default values file in <chart>/values.yaml.

@hickeyma  I think then what i understand is, i should be running upgrade as helm upgrade  --install --values values.yaml baster helm/ everytime.
Because in my case values.yaml is generated dynamically.
So if i just run helm upgrade  --install baster helm/ then helm will not pick up my dynamically generated values.yaml. Because it looks like, it just takes the saved values from the previous run.

Yes, sounds right. Give it a go and see if that works for you.

Thanks @hickeyma
It does work when i send the values using --values..
I was actually thinking this as a bug from helm for not picking the values.yaml dynamically rather than us sending it in command line..
After you comments in sounds like helm is designed in such a way. But still i am not sure why it was designed to store and reuse values even though a values.yaml is dynamically created in the correct directory path.

Looking at it again, I have noticed that you are specifying using the --values flag for the default values file: --values helm/values.yaml. I initially thought this was in a different directory to the default chart root.
I am thinking because you are specifying it using the --values flag that it is treating as a non default values file. Can you try it again without the --values flag ?

Correct, values.yaml is on the right folder. If i don't give the --values flag, even if i make any changes to the values.yaml file it is no reflected. That is the reason i created this issue.

I am having the same issues, do we have any updates here?

facing the same issue.

same

me too, helm doesn't take into account the values i've just changed

Try incrementing the version of the chart in the Chart.yaml file.


chore: migrate to projen

Notable changes

Tests moved to lib/__tests__
change-control-lambda moved inside lib instead of a separate package + use NodeJSFunction for bundling (avoiding the need for the build-change-control-lambda script)
custom-resource-handlers moved inside lib (but are still being built with a dedicated script).

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

AWS CodeBuild CI Report

CodeBuild project: GitHubPipelineAutoBuildProj-XK8ItpxnSow5
Commit ID: 9637683b38751834e4d4f55cdf9ebe75d36e3262
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubPipelineAutoBuildProj-XK8ItpxnSow5
Commit ID: 77a25dc197817b6c467166d90d72bb6c0f956ad9
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubPipelineAutoBuildProj-XK8ItpxnSow5
Commit ID: 02b0de662ebce2ed3778098f78f53b8a0aac72a8
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubPipelineAutoBuildProj-XK8ItpxnSow5
Commit ID: df3399bab70af38c203716b6a174f1e2d43989af
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubPipelineAutoBuildProj-XK8ItpxnSow5
Commit ID: 9210ffbc491dd1b66ecacc9dc94244aeb18f0f4b
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubPipelineAutoBuildProj-XK8ItpxnSow5
Commit ID: 0f5c0f05a3e2777a3002dcd4dac56203f916f82f
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubPipelineAutoBuildProj-XK8ItpxnSow5
Commit ID: a4b73b97e1ccfd05458c4d4a59ab14183b9b429a
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubPipelineAutoBuildProj-XK8ItpxnSow5
Commit ID: 7a1242f4643b55078ab26499e1aea2d4a562c5d6
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@eladb It just dawned on me that the switch to NodejsFunction in the change-controller-lambda is a breaking change, since it requires either esbuild or docker to bundle.
I'm going to mark this breaking and release a new major version. Can you acknowledge?


FileNotFoundError: [Errno 2] No such file or directory:

hi , I'm new in AI
I'm  training the model from scratch i download the dataset and the package
I run in google colab:
!cuda=GPU-f491dbb6-2dc7-a094-5417-a57de28df194  config=p3d_car.yml tag=run_tag  /content/unicorn-main\scripts\pipeline.sh
this appears to me:
Traceback (most recent call last):
File "/usr/local/lib/python3.7/dist-packages/scipy/io/matlab/mio.py", line 39, in _open_file
return open(file_like, mode), True
FileNotFoundError: [Errno 2] No such file or directory: '/content/unicorn-main/datasets/pascal_3d/ucmr_anno/data/car_train.mat'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/content/unicorn-main/src/trainer.py", line 292, in
trainer = Trainer(cfg, run_dir, seed=seed)
File "/content/unicorn-main/src/utils/init.py", line 104, in wrapper
return f(*args, **kw)
File "/content/unicorn-main/src/trainer.py", line 41, in init
self.train_loader, self.val_loader, self.test_loader = create_train_val_test_loader(cfg, rank, world_size)
File "/content/unicorn-main/src/dataset/init.py", line 16, in create_train_val_test_loader
train = get_dataset(name)(split="train", **kwargs)
File "/content/unicorn-main/src/dataset/p3d_car.py", line 38, in init
self.data = scio.loadmat(str(path), struct_as_record=False, squeeze_me=True)['images']
File "/usr/local/lib/python3.7/dist-packages/scipy/io/matlab/mio.py", line 224, in loadmat
with _open_file_context(file_name, appendmat) as f:
File "/usr/lib/python3.7/contextlib.py", line 112, in enter
return next(self.gen)
File "/usr/local/lib/python3.7/dist-packages/scipy/io/matlab/mio.py", line 17, in _open_file_context
f, opened = _open_file(file_like, appendmat, mode)
File "/usr/local/lib/python3.7/dist-packages/scipy/io/matlab/mio.py", line 45, in _open_file
return open(file_like, mode), True
FileNotFoundError: [Errno 2] No such file or directory: '/content/unicorn-main/datasets/pascal_3d/ucmr_anno/data/car_train.mat'
i just want to know where the location of ucmr_anno file install so i can fix it and know the path
i spend so much time searching  ucmr_anno file in colab files
help me please and thank for the work

thanks found it


Add data about Gunnar Kudrjavets

Be a good citizen and the data about myself to a new website.

Many thanks @gunnarku !


⚠ PrivateHD has degraded performance

In e2d53c0, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 1926 ms

Resolved: PrivateHD performance has improved in 44ef67e.


Version number has not updated to 0.0.4

Version is hard-defined in pkg/transporter/pipeline.go and is still 0.0.3.

fixed via https://github.com/compose/transporter/commit/5f4ec5206b3f91632094da4de620b2a75cd516ae


Return scope array that was given to us so that account icon number goes away

This PR fixes #150060

I think that this API should never have given the Auth Provider the power to return the scopes array because we use the scopes array as the "id" of the Activity Bar item that needs to be cleared... but we're beyond that now and this PR fixes the only known issue in this space for now.


Enhance management of DHCPv6 state

This commit adds IPv6 DHCP support.
Tested by:
Enable/Disable DHCPv6
configure SLAAC.

CI failed for D-bus Enum, seems latest Dbus is picked by CI build
‘sdbusplus::xyz::openbmc_project::Network::server::EthernetInterface::DHCPConf’
  80 |         {EthernetInterface::DHCPConf::v4v6stateless, {"true", "true", "false"}},
     |                                       ^~~~~~~~~~~~~
../src/ethernet_interface.cpp:83:39: error: ‘v6stateless’ is not a member of ‘sdbusplus::xyz::openbmc_project::Network::server::EthernetInterface::DHCPConf’
  83 |         {EthernetInterface::DHCPConf::v6stateless, {"false", "true", "false"}},
     |                                       ^~~~~~~~~~~

jenkins run tests please


Discuss Strategy: Access and Functional Needs

Problem
Most websites are built without considering the visually impaired, and websites really stink for them. Just as colorblindness has become an issue with Crisis Cleanup, accessibility will become an issue in the future.
Solution
We need to discuss how to best implement these things, and test for them:

[x] Screen-reader friendly
[x] Add descriptive titles to all links
[x] Add descriptive alt text to all images
[ ] Make all website actions easily accessible via a keyboard
[ ] Descriptive error messages
[x] Properly formed HTML and CSS
[x] Good color contrast
[ ] Skip Links option for repetitive content
[ ] HTML "lang" tag in header
[ ] No frames
[ ] No tables for layout purposes
[ ] No fixed font size. Use % or ems.
[ ] Fluid layout; use % or ems for width.
[ ] Close Caption all video, and provide a transcript.
[ ] Tables with  elements
[ ] Every form element has a label
[x] Ensure colors are distinguishable for people with color blindness

Additional context and instructions
Some of these things are easy, and we've already done them. The majority just require a very close attention to detail.
Please sound off in the comments about how to go about this. For example, should @tabiodun just enter a bunch of localized text strings wherever there is supposed to be alt text or a link title, then let @aarontitus fill in the details?

Complete enough for now.


Add draft with social cards

Description
I push this PR so we can take a look at the default social cards. I still need to discover how to add a custom description to all the cards. For the docs we have created in /docs folder we can write it directly in the .md file, but still need to know how to do it for the reference we have from the docstrings.
Would close #183

I have added a sentence/small description for the main pages in our docs, for example:

The remaining pages will render with the default content:

Let me know what you think of those and we can iterate.


Protocol 16 support

This issue lists the changes to the Horizon API introduced by CAP-35. This CAP comprises the public-facing changes in Stellar Protocol 16. The first Horizon version with the updated API is Horizon 2.1.0.
This protocol upgrade is purely additive. We expect a protocol 16 compliant SDK to be able to run successfully against a protocol 16 network.
We are aiming for the following tentative timeline:
March 30: Horizon released with protocol 16 support (done)
Begining of April: Core release with protocol 16 support
May: Protocol 16 deployed to Testnet
June: Protocol 16 Prodnet-deployment vote

New objects
None
Modified objects

Claimable Balance

Claimable balance flags have been incorporated (by bumping it's XDR extension version to v1). For now there's only a possible flag (CLAIMABLE_BALANCE_ENABLED_FLAG)

Trustline

A new flag as been added (AUTH_CLAWBACK_ENABLED_FLAG)

New endpoints
None
Modified endpoints
None
New Operations

clawback with the following fields:

Asset fields (identical to the asset fields in the Change Trust, Allow Trust and Set Trustline Flags operations)

asset_type
asset_code
asset_issuer

from - account from which the asset is clawed back
amount - asset amount clawed back

clawback_claimable_balance, with the following fields:

balance_id - claimable balance identifer of the claimable balance to be clawed back

set_trust_line_flags, with the following fields:

Asset fields (identical to the asset fields in the Change Trust, Allow Trust and Set Trustline Flags operations)

asset_type
asset_code
asset_issuer

trustor - account whose trustline is affected by this operation
flag fields, with the same scheme as the Set Options operation (note that CAP-35 introduces the new AUTH_CLAWBACK_ENABLED_FLAG flag)

set_flags - array containing the integer (XDR) representation of the flags to enable
set_flags_s - array containing the textual representation of the flags in set_flags (possible values are: authorized, authorized_to_maintain_liabilites and clawback_enabled)
clear_flags - array containing the integer (XDR) representation of the flags to disable
clear_flags_s - array containing the textual representation of the flags in clear_flags (possible values are: authorized, authorized_to_maintain_liabilites and clawback_enable)

New effects

trustline_flags_updated, with the following fields:

Asset fields (like explained in the operations above):

asset_type
asset_code
asset_issuer

trustor - account whose trustline the effect refers to
authorized_flag - true to indicate the flag is set, field ommited if not set
authorized_to_maintain_liabilites - true to indicate the flag is set, field ommited if not set
clawback_enabled_flag - true to indicate that the flag is set, field ommitted if not set

claimable_balance_clawed_back, with the following fields:

balance_id - claimable balance identifer of the claimable balance clawed back

Deprecations

Operation allow_trust is deprecated in favor of set_trust_line_flags (although it will still be supported by the network)

Effects trustline_authorized, trustline_authorized_to_maintain_liabilities and trustline_deauthorized are deprecated
in favor of trustline_flags_updated. Note how we intentionally didn't add a new trustline_authorized_clawback_enabled effect.
For uniformity, the allow_trust operation will start producing trustline_flags_updated from this release.
For now trustline_authorized, trustline_authorized_to_maintain_liabilities and trustline_deauthorized will continue to be emitted as a result of the allow_trust operation
but in the future we may stop doing so.

Update: CAP35 will now be included in Protocol 17\n\nVersion 16 of the protocol was used to resolve an issue in the network.\n(If interested, there is a blogpost explaining the details).\n\nAs a result, CAP35 will be included in Protocol 17. In orhter words, all the references to Protocol 16 above, now apply to Protocol 17.

Sorry for the formatting issues in the comment above. I hope it's clear enough though.

Duplicate of #3521 (it seems like the SDK issue tool doesn't deal well with closed issues)


feat(config): add config overriding capability with pflags

Providing capability to replace config via flags
    flags := cmd.Flags()
    loader := config.NewLoader(
        config.WithName("config"),
        config.WithType("yaml"),
        config.WithPFlags(flags),
        config.SetPFlagsKeyDelimiter("-"), // must be called after config.WithPFlags
    )

Since we don't expect SetPFlagsKeyDelimiter without the use of WithPFlags, does it makes sense to combine them into one option that would remove the need for having them called in order?

Need to update the readme to add usage and remove todo point.

does it makes sense to combine them into one option that would remove the need for having them called in order?

@rohilsurana  Yes it does. Would it be better if we use other func name? Or using this func name WithPFlags(flags, "-") is sufficient? Let me know if you have other suggestion

Need to update the readme to add usage and remove todo point.

Will do that

That name looks good.

@rohilsurana yes, closing this for now.


flutter.bat crashes when launched (Windows)

## Steps to Reproduce

Download flutter_windows_v0.3.1-betal.zip
Extract
Add the flutter\bin to the User Variables "Path"
Go to \bin and double click on the flutter.bat file
Crashes

## Logs
Flutter crash report; please file at https://github.com/flutter/flutter/issues.
command
flutter
exception
VersionCheckError: VersionCheckError: Command exited with code 128: git log -n 1 --pretty=format:%ad --date=iso
Standard error: fatal: your current branch 'beta' does not have any commits yet
#0      _runSync (package:flutter_tools/src/version.dart:439)
#1      FlutterVersion._latestGitCommitDate (package:flutter_tools/src/version.dart:129)
#2      FlutterVersion.frameworkCommitDate (package:flutter_tools/src/version.dart:120)
#3      FlutterVersion.checkFlutterVersionFreshness (package:flutter_tools/src/version.dart:235)
<asynchronous suspension>
#4      FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:284)
<asynchronous suspension>
#5      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:142)
<asynchronous suspension>
#6      _rootRun (dart:async/zone.dart:1126)
#7      _CustomZone.run (dart:async/zone.dart:1023)
#8      runZoned (dart:async/zone.dart:1501)
#9      AppContext.run (package:flutter_tools/src/base/context.dart:141)
<asynchronous suspension>
#10     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:265)
<asynchronous suspension>
#11     CommandRunner.run.<anonymous closure> (package:args/command_runner.dart:109)
#12     new Future.sync (dart:async/future.dart:224)
#13     CommandRunner.run (package:args/command_runner.dart:109)
#14     FlutterCommandRunner.run (package:flutter_tools/src/runner/flutter_command_runner.dart:174)
#15     run.<anonymous closure> (package:flutter_tools/runner.dart:54)
<asynchronous suspension>
#16     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:142)
<asynchronous suspension>
#17     _rootRun (dart:async/zone.dart:1126)
#18     _CustomZone.run (dart:async/zone.dart:1023)
#19     runZoned (dart:async/zone.dart:1501)
#20     AppContext.run (package:flutter_tools/src/base/context.dart:141)
<asynchronous suspension>
#21     runInContext (package:flutter_tools/src/context_runner.dart:41)
<asynchronous suspension>
#22     run (package:flutter_tools/runner.dart:49)
#23     main (package:flutter_tools/executable.dart:48)
<asynchronous suspension>
#24     main (file:///C:/Flutterspace/flutter/packages/flutter_tools/bin/flutter_tools.dart:8)
#25     _startIsolate.<anonymous closure> (dart:isolate-patch/dart:isolate/isolate_patch.dart:277)
#26     _RawReceivePortImpl._handleMessage (dart:isolate-patch/dart:isolate/isolate_patch.dart:165)

flutter doctor
encountered exception: VersionCheckError: Command exited with code 128: git log -n 1 --pretty=format:%ad --date=iso
Standard error: fatal: your current branch 'beta' does not have any commits yet

#0      _runSync (package:flutter_tools/src/version.dart:439)
#1      FlutterVersion._latestGitCommitDate (package:flutter_tools/src/version.dart:129)
#2      FlutterVersion.frameworkCommitDate (package:flutter_tools/src/version.dart:120)
#3      FlutterVersion.frameworkDate (package:flutter_tools/src/version.dart:83)
#4      _FlutterValidator.validate (package:flutter_tools/src/doctor.dart:256)
<asynchronous suspension>
#5      Doctor.startValidatorTasks (package:flutter_tools/src/doctor.dart:69)
#6      Doctor.diagnose (package:flutter_tools/src/doctor.dart:126)
<asynchronous suspension>
#7      _doctorText.<anonymous closure> (package:flutter_tools/runner.dart:189)
#8      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:142)
<asynchronous suspension>
#9      _rootRun (dart:async/zone.dart:1126)
#10     _CustomZone.run (dart:async/zone.dart:1023)
#11     runZoned (dart:async/zone.dart:1501)
#12     AppContext.run (package:flutter_tools/src/base/context.dart:141)
<asynchronous suspension>
#13     _doctorText (package:flutter_tools/runner.dart:188)
<asynchronous suspension>
#14     _createLocalCrashReport (package:flutter_tools/runner.dart:166)
<asynchronous suspension>
#15     _handleToolError (package:flutter_tools/runner.dart:121)
<asynchronous suspension>
#16     run.<anonymous closure> (package:flutter_tools/runner.dart:58)
<asynchronous suspension>
#17     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:142)
<asynchronous suspension>
#18     _rootRun (dart:async/zone.dart:1126)
#19     _CustomZone.run (dart:async/zone.dart:1023)
#20     runZoned (dart:async/zone.dart:1501)
#21     AppContext.run (package:flutter_tools/src/base/context.dart:141)
<asynchronous suspension>
#22     runInContext (package:flutter_tools/src/context_runner.dart:41)
<asynchronous suspension>
#23     run (package:flutter_tools/runner.dart:49)
#24     main (package:flutter_tools/executable.dart:48)
<asynchronous suspension>
#25     main (file:///C:/Flutterspace/flutter/packages/flutter_tools/bin/flutter_tools.dart:8)
#26     _startIsolate.<anonymous closure> (dart:isolate-patch/dart:isolate/isolate_patch.dart:277)
#27     _RawReceivePortImpl._handleMessage (dart:isolate-patch/dart:isolate/isolate_patch.dart:165)

**## Flutter Doctor **
C:\Users\prasa>flutter doctor
fatal: ambiguous argument 'HEAD': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git  [...] -- [...]'
Oops; flutter has exited unexpectedly.
Sending crash report to Google.
Failed to send crash report. Server responded with HTTP status code 400
Crash report written to C:\Users\prasa\flutter_01.log;
please let us know at https://github.com/flutter/flutter/issues.

@prasannajeet can you try to double-click flutter_console.bat and in there run the command flutter doctor -v and share the output of that?

cc @gspencergoog

@mit-mit I could not find the file flutter_console.bat anywhere. Which is why I went to \bin\flutter.bat as I thought it might have been renamed. What would be the exact location of the file inside the "flutter" folder?

It should be at the root of the flutter folder. Do you not see this?

cc @goderbauer

I don't. This is what I see :( Even when I double click and open the zip file I don't see the bat file
But I see it when i open the same zip file on my Android phone :(

@prasannajeet What app did you use to unzip?

The Windows default option. Right Click and "Extract"

I just double-checked: When I use Windows' build-in unzipper on my system flutter_console.bat gets extracted correctly. It's definitely in the ZIP.
Do you have any kind of virus scanner active that might interrupt the extracting process? You can also try a different unzipper, e.g. 7zip.


Fix flaky payment test

Fix test_payment_gateway_refund_payment_no_payment

Impact

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

Pull Request Checklist

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

Task linked: SALEOR-2789 Flaky payment test


Add error checking in Kimera

https://github.com/ILLIXR/ILLIXR/issues/225

Part of https://github.com/ILLIXR/ILLIXR/pull/227

Update the ILILXR PR with the new commit-hash once this gets merged into master.


Emoji has transparency on Android

A similar issue is here: https://github.com/EricPKerr/react-native-emoji/issues/32 and the solution appears to work. Can we add the default style prop here as well?
https://github.com/TheWidlarzGroup/rn-emoji-keyboard/blob/6735d6570624078381353f825697c7d834f4726f/src/components/SingleEmoji.tsx#L18
Proposed Change:
<Text style={{ fontSize: emojiSize, color: '#000' }}>{item.emoji}</Text>

Hi @mgauthier,
Thank you for reporting this issue  🚀
We've created new PR with that fix #50 and it will be merged and released soon 😄

Hi @mgauthier
I'm sorry that you had to wait so long. It should be fixed in v0.7.1
Please let me know if that works for you 😄


Make sure policies.is_allow_auto_update_schema not null

Motivation
Follow code works fine on 2.8.x but get NPE on 2.9.x
Policies policies = pulsarAdmin.namespaces().getPolicies("namespacePath");
boolean allowedAutoupdateSchema = policies.is_allow_auto_update_schema;

Becasue Policies.is_allow_auto_update_schema changed from boolean to Boolean by #12786 and default value is null.
This may cause user's service fail when upgrade pulsar from 2.8.x to 2.9.x
Modifications
Make sure Policies.is_allow_auto_update_schema not null when call getPolicies
Documentation

[x] no-need-doc
Just keep behaviour same between 2.8.x and 2.9.x

/pulsarbot run-failure-checks

We can add a new interface Policies getPolicies(String namespace, boolean applied) in org.apache.pulsar.client.admin.Namespaces. And set applied as true by default in getPolicies(String namespace).

the applied option should be false by default?  to keep consistent with topic policy does. It's better to start with a PIP, as far as I understand, not all the namespace policies follow the same way, some return null if the namespace doesn't have the policy, some return value from broker level, we need to improve this situation even if the inevitable break change may be introduced, otherwise, we will always be confused by different behaviors.
We should add a release notice for https://github.com/apache/pulsar/pull/12786

the applied option should be false by default?

This default value is just for backward compatible. And we can even deprecate method getPolicies(String namespace).
For user with new version of client and broker, the applied parameter should always be provided by user.
Just one option to keep compatibility.

This default value is just for backward compatible. And we can even deprecate method getPolicies(String namespace).
For user with new version of client and broker, the applied parameter should always be provided by user.
Just one option to keep compatibility.

I see, we have to make tradeoffs here, maybe we should make API follow the same pattern in 3.0

@gaoran10 @michaeljmarshall Please help check this PR, which is fixing a breaking change in the ongoing release 2.8.3, 2.9.2, and 2.10.0. And @wangjialing218 also shared the information under the 2.9.2 VOTE3 email thread, we need to decide as soon as possible whether to start a new round RC.

@codelipenghui - thanks for tagging me. Looking now.

I believe this solution is incomplete. I think we also need to update this method defined on line 517 in the same class:
protected Policies getNamespacePolicies(String tenant, String cluster, String namespace)

After looking at the code a bit more, it appears that updating the second getNamespacePolicies method is unnecessary. However, it would probably be appropriate to keep things consistent.

the applied option should be false by default?

This default value is just for backward compatible. And we can even deprecate method getPolicies(String namespace). For user with new version of client and broker, the applied parameter should always be provided by user. Just one option to keep compatibility.

This makes sense to me. Do you think we need to add the applied logic to this PR @Jason918?

I believe this solution is incomplete. I think we also need to update this method defined on line 517 in the same class:
protected Policies getNamespacePolicies(String tenant, String cluster, String namespace)

After looking at the code a bit more, it appears that updating the second getNamespacePolicies method is unnecessary. However, it would probably be appropriate to keep things consistent.

@michaeljmarshall Done, the second method was duplicated code from first one.

@wangjialing218 - great, I thought those methods looked similar, but I didn't look closely enough to see the one was a duplicate. Thanks for fixing that.

We can go with the current form from my opinion.
I would like to see the 'applied' version of the function some day.
Let's unblock the release.

/pulsarbot run-failure-checks

We can go with the current form from my opinion.
I would like to see the 'applied' version of the function some day.

Agree, maybe we can plan to 2.11


Error volume mounting custom schema

Hello, the code below is my volume mount the for my custom schema(s)
    volumes:
      - ../openldap/custom:/container/service/slapd/assets/config/bootstrap/ldif/custom
      - ../openldap/schema:/container/service/slapd/assets/config/bootstrap/schema

But I keep getting this error message,
-bash-4.2$ docker logs open-ldap
*** CONTAINER_LOG_LEVEL = 3 (info)
*** Copy /container/service to /container/run/service
*** Search service in CONTAINER_SERVICE_DIR = /container/run/service :
*** link /container/run/service/:ssl-tools/startup.sh to /container/run/startup/:ssl-tools
*** link /container/run/service/slapd/startup.sh to /container/run/startup/slapd
*** link /container/run/service/slapd/process.sh to /container/run/process/slapd/run
*** Set environment for startup files
*** Environment files will be proccessed in this order :
Caution: previously defined variables will not be overriden.
/container/environment/99-default/default.startup.yaml
/container/environment/99-default/default.yaml

To see how this files are processed and environment variables values,
run this container with '--loglevel debug'
*** Running /container/run/startup/:ssl-tools...
*** Running /container/run/startup/slapd...
Database and config directory are empty...
Init new ldap server...
invoke-rc.d: could not determine current runlevel
invoke-rc.d: policy-rc.d denied execution of stop.
  Backing up /etc/ldap/slapd.d in /var/backups/slapd-2.4.44+dfsg-5+deb9u2... done.
  Creating initial configuration... done.
  Creating LDAP directory... done.
invoke-rc.d: could not determine current runlevel
invoke-rc.d: policy-rc.d denied execution of start.
rm: cannot remove '/container/run/service/slapd/assets/config/bootstrap/schema/rfc2307bis.*': No such file or directory
*** /container/run/startup/slapd failed with status 1

*** Killing all processes...

Any idea why it fails? I'm thinking this is because the script is looking for some files in https://github.com/osixia/docker-openldap/tree/stable/image/service/slapd/assets/config/bootstrap/schema, but would this be fixed if i basically have the same volume mount code BUT I will include these files in that directory?

@cjbuckethead may be you are confusing a few things here.
Seems to me that all files in image/service/slapd/assets/config/bootstrap are used during image creation (docker build) as opposed that you are trying to use them during image instantiation (docker run).
If you get passed the rm error (changing rm to rm -f for instance) you will still endup with another error like
rm: cannot remove '/container/service/slapd/assets/config/bootstrap/schema': Device or resource busy
rm: cannot remove '/container/service/slapd/assets/config/bootstrap/ldif/custom': Device or resource busy
HTH

@cjbuckethead did you get a chance to look at my previous comment above ?

@obourdon hey, sorry for not responding earlier.
We just opted for the COPY command in the Dockerfile.
The initial use case we wanted was to volume mount a directory of our own schema/s to the bootstrap directory, that I was volume mounting two directories [1] /openldap/custom which contains all our ldif data and [2] /openldap/schema which contains our own schemas
    volumes:
      - ../openldap/custom:/container/service/slapd/assets/config/bootstrap/ldif/custom
      - ../openldap/schema:/container/service/slapd/assets/config/bootstrap/schema

But this approach is not possible as there are other schemas used by the osixia image in the same directory as the /openldap/schema so when I volume mount this, it would overwrite the whole directory which would cause for the error message above to show.
We went with the approach below
COPY /openldap/schema/cities.schema /container/service/slapd/assets/config/bootstrap/schema/cities.schema
...

while this might be tedious if we have numerous schemas, at the very least we wont be deleting some of the schemas the image uses during startup.

@cjbuckethead no pb AT ALL and many thanks for your answer.
I understand your point for many schemas and may be an alternate solution would be to create a (tar for instance) archive of your schemas and unextract this as a single operation in replacement of the numerous COPY operations
In any case, whatever the solution you choose to use, may be we can agree to consider this case resolved and close this issue.
Many thanks again

@cjbuckethead try mounting your custom schema in a subdir of
/container/service/slapd/assets/config/bootstrap/schema
for example :
/container/service/slapd/assets/config/bootstrap/schema/custom

@BertrandGouny I did, for some reason, it's not working recursively.

Run the container with --copy-service. Please read the document here: https://github.com/osixia/docker-openldap#seed-ldap-database-with-ldif
Without --copy-service, the container tries to delete /container/service/slapd/assets/config/:
https://github.com/osixia/docker-openldap/blob/7ce084bab1f9016830548558241fdefb60862482/image/service/slapd/startup.sh#L539
This causes
rm: cannot remove '/container/service/slapd/assets/config/bootstrap/schema': Device or resource busy


remove early-stage from Gallery's About screen

After 1.0 we are probably no longer early stage.

We can make other changes here, like removing all the "preview" stuff and the preview banner.


fix(scheduler): prevent array out-of-bounds when GPU containers are placed between non-GPU containers

What type of PR is this?
/kind bug
What this PR does / why we need it:
This PR fixes a bug in the scheduler where an array out-of-bounds error occurs when a GPU container is placed between non-GPU containers in a multi-container Pod. The bug manifests when the GPU container must be positioned at the beginning or end of the container list, otherwise, it leads to a panic with an array index error. Ensuring that the length of score.Devices[idx] is sufficient before appending resolves this issue. The scheduler now handles this scenario by extending the score.Devices array correctly.
Which issue(s) this PR fixes:
Fixes #571
Special notes for your reviewer:
None
Does this PR introduce a user-facing change?:
None

A very god PR, please sign your commit:)

@archlitchi Thanks for the reminder! I've signed the commit now. Please check and let me know if everything looks good. :)

/approve


clean residual state in binder and emitter, clean test data in versio…

…n cache test

@DanielRosenwasser any other comments?

:+1:


Test French sources

Ordered from easy to difficult:

[x] http://www.1jour1actu.com/
[x] http://www.okapi.fr/
[x] http://www.phosphore.com/
[x] https://www.marianne.net/
[x] http://www.parismatch.com/
[x] http://www.liberation.fr/ :x: - Does not extract content very well.
[x] http://www.leparisien.fr/
[x] http://www.lefigaro.fr/
[x] http://www.lemonde.fr/

whoa! you can click in there! i've just tested 1jour1actu and it works.

Tested the other sources, only one that does not work perfectly (I labeled it).


Quad tilt vtol config

Added a new vehicle configuration for a Quad Tilt Aircraft

Vehicle ID
Main Mixer
Aux Mixer

@samay Thanks for the PR! If you add the missing new lines and rebase on master then I'll get this in.

@tumbili thank you , i added the new lines but not sure why the sitl build failed ?


Maxwell solver example

The Maxwell solver demos:

TM mode 2D time-domain Maxwell's equation with homogeneous material on triangles.
Full 3D time-domain Maxwell's equations with homogeneous material on tetrahedra.
It does not currently include quads and hexes due to their additional requirements for spurious mode free simulations.
The notation within the surface and volume kernels directly aligns with notation from the H&W book.
The maxwell solver includes an L2 error estimate if activated, with consistent mass matrix based integration of the error at W&B nodes.

Looks good. Made a couple tiny changes, and theres only a couple other things I noticed.

profilingTips.text - Do we need this?
The default problem size in setupTet3D.rc is real chunky, even for my workstation GPU. Maybe we can reduce this so it runs reasonably quickly?

I removed the profiling tips file and reduced the size of the box mesh in setupTet3D.rc
Are the testMaxwell.py meshes small enough ?


after fine-tune on my own dataset and try to test, got unexpected error

Hi, @LiheYoung ,thank you for sharing this wonderful work, when test on my fine-tuned model, I got missing keys error like below :
"conv2.weight','conv2.bias','seed_bin_regerssor._net.0.weight'....
I just load my own data and use the command to fine-tune:
python train_mono.py -m zoedepth -d my_data --pretrained_resource="./depth_anything_vitl14.pth" ,
and at test phase, i use:
 python run.py --encoder=vitl / load_from='my fine_tune model' / --localhub.
Since this missing keys are from zoedepth,  it seems train phase use zoedepth model but test use depth_anything, so I write a test file and get the model use build_model based on train_mono.py and load my fine-tune model instead of using DPT_DINOv2 in run.py, everything is normal except the test results are not good.
Hope you can give some advice ,thanks~

@visionpon ，Hello, may I ask how you achieved fine-tuning on your own dataset? Can you provide some detailed guidance?
I am also trying to make slight adjustments on the custom dataset, but I have no idea how to modify the configuration file...

it's easy to  write your own data load file based on the scripts in metric_depth/zoedpth/data/ @swing2331

@visonpon hi i am new to depth estimation. Do you need gt depths for fine tuning or you only use the unlabeled raw img data? Thanks


fix(scheduler): use app name when creating services

When we eventually allow applications to be grouped together in the same namespace, we'll want to
have the service names discoverable to other instances in the same namespace. Having a name like
"app-1234" was created so we could get over the limitation of 24 characters for service names. Now
that we're in v2, we can just restrict app names to 24 characters.
The app name validation regex was not changed because the docker repository name regex is a subset of the kubernetes service name regex, so we still need to keep ourselves restricted to that name regex so we can still push those images to the internal registry.
closes #17
closes #32

Code LGTM.
(changing the schema)
@bacongobbler thanks for this change.

done, now with schema migrations!

along with restricting App name to 24 character also App name should be compatible with C variable standards.
Does the regex checks whether App names follow C variable format ?

No, but it is compatible. We still use the same regex as the docker registry. Since the docker registry regex (only lowercase, alphanumeric, dashes allowed) is a subset of kubernetes's regex, we must primarily comply with that standard, or else we will not be able to push images to the internal registry. Since it is a subset of the standard, we are automatically compatible with it.

@bacongobbler this has a needs manual testing label on it. does that mean I should do so?

Anything scheduler-related can't be covered by unit tests until we add some mock tests to a kubernetes endpoint, so for now we have to test these manually, though since myself and @mboersma have I guess this can be omitted.

ok. looks like I'll be testing this functionality a bit this week anyway. LGTMe-ed

lol, github labels are playing with us :P

haha. race condition :allthethings:


New developer

Closes: #1549
Account Number: f6f38190713621648c7a1d8e52cb300e9da5cad47aa190004dcc22a17c1963d2

@bojan-ilijevski there seems to be no changes in this PR. Can you also make sure your branch is updated based on the latest development so there's no unnecessary commits?


Multiple Kamilia phase 2 sub-phases triggered at the same time

Not sure how this might happen, but it happens:
https://clips.twitch.tv/PlumpColdbloodedSparrowAMPEnergyCherry
It messes with the numbers and clear percentages for the phases.

Older version maybe? Issue was supposedly fixed at some point after release.

Yeah, he indeed must have been using something ancient. Readme said version 0.0.0.
Re-Downloading and auto-updating to 1.1 now, seems like it's fixed.
What's the official download source for 1.1 btw.? Just get 1.02 from delicious-fruit and auto-update it?

Download 1.02 and auto update, yes. Delicious fruit doesn't offer a quick and automatic way to update the link, unfortunately.


[Improvement] Introduce a new class ShuffleTaskInfo

What changes were proposed in this pull request?

There is no need to cache four Map. Introduce a new object, ShuffleTaskInfo, to replace currentTimes, commitCounts, commitLocks and cachedBlockIds.
Why are the changes needed?

This will be clear and save space.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

Passed the ut.

Codecov Report

Merging #158 (a2e80f3) into master (bac99af) will increase coverage by 0.88%.
The diff coverage is 84.84%.

@@             Coverage Diff              @@
##             master     #158      +/-   ##
============================================
+ Coverage     57.36%   58.24%   +0.88%
+ Complexity     1205     1087     -118
============================================
  Files           150      138      -12
  Lines          8198     6965    -1233
  Branches        771      653     -118
============================================
- Hits           4703     4057     -646
+ Misses         3254     2698     -556
+ Partials        241      210      -31

Impacted Files
Coverage Δ

...pache/uniffle/server/ShuffleServerGrpcService.java
1.00% <0.00%> (ø)

.../org/apache/uniffle/server/ShuffleTaskManager.java
64.12% <85.00%> (+0.02%)
:arrow_up:

...ava/org/apache/uniffle/server/ShuffleTaskInfo.java
91.66% <91.66%> (ø)

...storage/handler/impl/DataSkippableReadHandler.java
81.25% <0.00%> (-3.13%)
:arrow_down:

.../apache/uniffle/coordinator/ClientConfManager.java
91.54% <0.00%> (-1.41%)
:arrow_down:

.../java/org/apache/hadoop/mapreduce/RssMRConfig.java

...n/java/org/apache/hadoop/mapreduce/MRIdHelper.java

.../hadoop/mapreduce/task/reduce/RssBypassWriter.java

...preduce/task/reduce/RssRemoteMergeManagerImpl.java

...pache/hadoop/mapreduce/task/reduce/RssFetcher.java

... and 10 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@smallzhongfeng I'm ok with the refactor to clear code clearly, please fix the problem with CI

Could you hava a review for this pr? @colinmjj I will appreciate about it.

If you have any other suggestions, I would be very grateful.@jerqi @colinmjj

+1, LGTM

Thanks for your review.@jerqi @colinmjj


Support an option for interrupted audio playback to continue after interrupting media has paused/muted

I looked into this and it's pretty difficult to pull off with the current architecture, and I don't think it's going to be a highly requested feature anyway - if it is needed, maybe folks can implement it themselves.


feat(ObserverLocator) support value binding select elements

Fixes aurelia/templating-binding#21

If this looks good let me know and I'll back-fill the unit tests.
Integration tests are here:  http://jdanyow.github.io/skeleton-navigation/
Looking at the radio scenario next.

unit tests committed, this is ready.

added more assertions to the test suite


Icon request: Solid Modus Sun and Moon icons

Use case
These icons are used for the dark mode and light mode toggle.
It looks like the solid sun and moon icons are not included in the package.

https://modus-icons.trimble.com/modus-solid/sun/
https://modus-icons.trimble.com/modus-solid/moon

Example image
No response
Icon request checklist

[X] The title starts with "Icon request: " and is followed by the requested icon name
[X] This icon does not represent a brand
[X] This is a single icon or matched pair (Ex: lock / unlock)
[X] I have searched for existing issues and to the best of my knowledge this is not a duplicate
[ ] I have included at least one example image if it could be helpful (optional)

heya! The Sun and Moon icons are included in the modus-icons package within the modus-solid set. If you are using the outlined icon set as a webfont then you can still use the solid icons as individual SVGs, inline SVG or SVG sprite.

I'll mark this as closed but please reach out if you need any help or have any other icon requests.


sign-up fails

Describe the bug
I followed the CompreFace/docs/How-to-Use-CompreFace.md and deployed with docker successfully. But when I begin to sign up an account, the page return

Desktop (please complete the following information):

Linux
chrome
CompreFace_0.5.1

Additional context
docker ps

docker-compose up -d
https://drive.google.com/file/d/1_8kNKcWVWXXIlNQRLG44-OKd34EvQ2S4/view?usp=sharing

CompreFace/java/admin/README.md describes "To open swagger api view follow http://localhost:8080/swagger-ui.html" But I cannot get the API specification on port 8080.

It looks like compreface-admin node hasn't started properly.
Could you run docker logs compreface-admin and send the logs?
Also, you can try these steps:
https://github.com/exadel-inc/CompreFace/issues/539#issuecomment-846960264
We use Swagger internally, this is why this documentation is for those who run the admin module without everything else. If you run CompreFace with docker-compose, swagger will be available here:
http://localhost:8000/swagger/swagger-ui.html - for API endpoints
http://localhost:8000/admin/swagger-ui.html - for Admin endpoints
But as I mentioned - this is the internal features, so I'm not sure if it contains everything.
The latest endpoint documentation is described here:
https://github.com/exadel-inc/CompreFace/blob/master/docs/Rest-API-description.md


Repeating notifications

Describe the feature request
Ability to send repeated notifications every x min/hr if the endpoint is still offline/dead.
Why do you personally want this feature to be implemented?
We now had over 300 nodes monitoring via Gatus and we received the notifications via Slack and Telegram. One scenario is when some nodes are offline during the weekend. We always forget to fix them in the next workday.
I've tried some other services and seems like it's a common feature to alert repeatedly if something still goes wrong.
How long have you been using this project?
No response
Additional information
No response

Sounds like a good idea!
Something like this:
    alerts:
      - type: slack
        enabled: true
        description: "healthcheck failed 3 times in a row"
        send-on-resolved: true
        repeating: true

That said, not all alerting providers should allow this.
For instance, pagerduty incidents need an acknowledgement on resolve, so we probably don't want to trigger multiple alerts simultaneously for pagerduty. For Slack, Discord, Teams, Telegram, etc. it shouldn't be an issue.

Bumping this because we'd also be interested in having this feature, particularly for the MS Teams alerting hooks. We already use send-on-resolved, but if a service was down for 6+ days and the small team missed the initial message sent to a noisy MS Teams channel, it's easy to miss or forget unless it periodically reminds you it is still failing.

From what I can see, #614 is implementing this functionality ready for review

I want this!


Options on presets do not work

Options on presets do not work.
The preset runs fine, its just the output still has !important blocks.
https://gist.github.com/timgrafton/91acca4605e81a3bc209b1ab87fd7192
Regards,
Tim

My bad, I didn't download the latest version of cssnano.


Sort context menu variable list

Resolves
Fixes #1879.
Proposed Changes
Sorts the variable list in the variable context menu:

Sorts list context menus, too:

Reason for Changes
To make the context menu of variable/list menus more useful, and more consistent with other lists of variables in the UI.
Test Coverage
No new unit tests. Tested manually.

LGTM. Thanks @towerofnix!


Register v1.0.0

@JuliaRegistrator register
Release notes:
What's Changed

Move plotting methods to an extension package by @ConnectedSystems in https://github.com/open-AIMS/CoralBlox.jl/pull/2
Allow plotting whe extension is loaded. by @DanTanAtAims in https://github.com/open-AIMS/CoralBlox.jl/pull/3
Consider values close to 0.0 as 0.0 by @ConnectedSystems in https://github.com/open-AIMS/CoralBlox.jl/pull/4
Changes to handle small size classes by @ConnectedSystems in https://github.com/open-AIMS/CoralBlox.jl/pull/5
Implements more accurate growth when cover block goes to new size class by @Zapiano in https://github.com/open-AIMS/CoralBlox.jl/pull/10
Implement changes to provide compatibility with ADRIA.jl by @Zapiano in https://github.com/open-AIMS/CoralBlox.jl/pull/6
Rename package to CoralBlox by @Zapiano in https://github.com/open-AIMS/CoralBlox.jl/pull/14
Update/rename CoralBlox extensions by @Zapiano in https://github.com/open-AIMS/CoralBlox.jl/pull/15
rename apply_survival to apply_mortality by @DanTanAtAims in https://github.com/open-AIMS/CoralBlox.jl/pull/17
Implement a linear extension scale factor model to prevent coral out-growing habitable area by @Zapiano in https://github.com/open-AIMS/CoralBlox.jl/pull/18
Circular Buffer Implementation by @DanTanAtAims in https://github.com/open-AIMS/CoralBlox.jl/pull/11
Add MIT LICENSE by @Zapiano in https://github.com/open-AIMS/CoralBlox.jl/pull/19
Update README by @Zapiano in https://github.com/open-AIMS/CoralBlox.jl/pull/20

New Contributors

@ConnectedSystems made their first contribution in https://github.com/open-AIMS/CoralBlox.jl/pull/2
@DanTanAtAims made their first contribution in https://github.com/open-AIMS/CoralBlox.jl/pull/3
@Zapiano made their first contribution in https://github.com/open-AIMS/CoralBlox.jl/pull/10

Full Changelog: https://github.com/open-AIMS/CoralBlox.jl/commits/v1.0.0

I tried to install @JuliaRegistrator to this repo but apparently it didn't work so I tried registering this manually. This is the correspondent PR: https://github.com/JuliaRegistries/General/pull/112631

Error while trying to register: Action not recognized: to

Ok, now it seems to be here. That's good.

I waiting for this PR to be merged to try again: https://github.com/open-AIMS/CoralBlox.jl/pull/24

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/112634
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v1.0.1 -m "<description of version>" 093195344f2dc47a10fee6250e01849e09c9b3fe
git push origin v1.0.1

Also, note the warning: This looks like a new registration that registers version 1.0.1.
Ideally, you should register an initial release with 0.0.1, 0.1.0 or 1.0.0 version numbers
This can be safely ignored. However, if you want to fix this you can do so. Call register() again after making the fix. This will update the Pull request.

@JuliaRegistrator register

Error while trying to register: "Tag with name v1.0.1 already exists and points to a different commit"

@JuliaRegistrator register

Error while trying to register: "Tag with name v1.0.1 already exists and points to a different commit"

@JuliaRegistrator register

Error while trying to register: "Tag with name v1.0.1 already exists and points to a different commit"

@JuliaRegistrator register

Error while trying to register: "Tag with name v1.0.1 already exists and points to a different commit"


Implement app init

The App commands are not currently super useful, since their main behavior is taken care of by init, but once we start implementing deploy and show, it'll be useful to already have the namespace defined.
This task is for creating an app init command that:

Checks to see if we're in a WS
Prompts the user for the application type they want to build
Prompts the user for the application Dockerfile (right now - in the future we'll let them select creating a directory with one of our templated directories)
Writes the manifest to the workspace root
Writes the application into SSM

This task can be broken down into 2 parts:

Adding the Application to SSM (we currently only store projects and envs)
Wiring up the init command based on the current ecs init command

The work until step 4) is done, step 5) can be picked up after https://github.com/aws/amazon-ecs-cli-v2/pull/225 is merged

Looks like #225 is now merged and writing applications to SSM will be required for #230


work server load division by number of CPUs

http://blog.scoutapp.com/articles/2009/07/31/understanding-load-averages

On multi-processor system, the load is relative to the number of processor cores available. The "100% utilization" mark is 1.00 on a single-core system, 2.00, on a dual-core, 4.00 on a quad-core, etc.
Same with CPUs: a load of 1.00 is 100% CPU utilization on single-core box. On a dual-core box, a load of 2.00 is 100% CPU utilization.

this needs a bit of a practical investigation


New version: QuantumClifford v0.2.2-DEV

Registering package: QuantumClifford
Repository: https://github.com/Krastanov/QuantumClifford.jl
Created by: @Krastanov
Version: v0.2.2-DEV
Commit: 9de29173fe88a9a67e4f2ec0a7466dfd3a47ddc9
Git reference: master
Release notes:

Detailed bibliography added to docs.

This should be closed. The reported mistakes were fixed in https://github.com/JuliaRegistries/General/pull/26901


Provide JSON schema for moddle (including XML extension elements)

Is your feature request related to a problem? Please describe.
As a developer, I want to make sure that my moddle schema instance is checked against a moddle schema definition and any violations are reported by a standard schema validation tool.
Describe the solution you'd like
Provide a complete JSON schema. The parts references in docs folder are helpful, but incomplete.
Describe alternatives you've considered
None
Additional context
I developed a JSON schema and would like you to double check it and include into this repository.

I developed a JSON schema and would like you to double check it and include into this repository.

Sounds like you want to contribute it (as a PR)? We'd highly apprechiate that. We discussed this topic in the past, but were not able to follow through yet.


Create FUNDING.yml

To signoff the previous commit, use amend option:
git commit --amend --signoff

sign them off the prior then push/fetch the state of the old commits, no new commits needed.

its done

signd sealed and delivered. Always signed have you set that  up to do that?
I just wonder Why it's not wasn't signed from the start.


Add options to ignore directories

I ran pyq3 and got:
pyq3 "#init_cmd" .
src/foo/bar.py:111 def init_cmd(args):
.tox/py35/lib/python3.5/site-packages/foo/bar.py:89 def init_cmd(args):
.tox/flake8/lib/python3.5/site-packages/foo/bar.py:89 def init_cmd(args):
.tox/coverage/lib/python3.5/site-packages/foo/bar.py:89 def init_cmd(args):
.tox/doc/lib/python3.5/site-packages/foo/bar:89 def init_cmd(args):

I don't want to include .tox, it's just noise.
Something like:
pyq3 "#init_cmd" . -e ".tox,build,dist,*eggs*" # exclude directories matching these patterns
src/foo/bar.py:111 def init_cmd(args):

It would make sense have a sane default value for -e, containing directories any average Python project will have but probably not want to search (although you can if you override pass -e manually).  Such dirs are:

.tox : environnements tox cache for testing.
.cache: name for a lot of cache dir (such as pip's)
build et dist: setuptools build and distribution dirs
.git/.svn/ etc : most version system dirs.

I agree with you. Some points:
Currently, pyq command do not consider wildcards for path and it should:
You could avoid hidden files by using: pyq <selector> * instead of pyq <selector> .
We should add a --ignore-dir option, just like ack:
ack --py something --ignore-dir=migrations
This is extremely useful. I find myself doing this every time with ack.
But well, sometimes I also want to find something in the migrations directory.
Yes, we should provide sane defaults. I think ack defaults is a good start:
https://github.com/petdance/ack2/blob/44ca211/ConfigDefault.pm#L21

For now you may be able to ignore hidden files by using: pyq <selector> * , just like grep.

:)

Fixed in https://github.com/caioariede/pyq/commit/b719ed8c1c2c73165db0335b53b88f60e2c5bb54

Sweet


mage ui not working on Windows OS

Describe the bug
Currently, mage ui command is not working on Windows OS, and giving the following error

Files Or Markdown that can reproduce the issue

git clone the armada project
install necessary dependencies
run mage ui

Expected behavior
The command was expected to install yarn packages for the internal/lookout/ directory

Actual behavior
It is giving an error that the executable is not found in environment variables, though there exists a yarn bin inside the system environment variables.

Why is the error occurring?
We are using the binaryWithExt function to get the .exe version of yarn, i.e., yarn.exe, which is not the case for Windows OS, since yarn is a binary which can be directly executed with yarn command, without the .exe extension.

I would like to work on this issue