reserved identifier violation

I would like to point out that identifiers like "_E_AFALG_H_" and "__PPC_ARCH_H__" do eventually not fit to the expected naming convention of the C++ language standard.
Would you like to adjust your selection for unique names?

Can you provide some specifics?  Our public files are fine, and a quick find/xargs/grep didn't see anything amiss.

How do you think about to make the mentioned include guards also standard complaint?

shrug.  not a big deal since oyou can't compile openssl with C++ now anyway, but sure.  make a PR.

@mattcaswell might as well:  see gitlab :)

The same rules apply in C as well.  The C++ standard is based on the C standard in this regard.  The guideline is simple, anything starting with an underscore is reserved for the compiler suite implementation, as well as anything that includes a double underscore somewhere.
The risk we take everytime we define anything starting with an underscore is that it might clash with something the compiler defines, now or in the future.
That being said, I would be strongly surprised if any compiler would define _E_AFALG_H_ or __PPC_ARCH_H__ unless it was out of pure spite ;-)  ...  and then, it's in non-public header files.  That makes this issue very low priority, to be revisited at a later time if someone reminds us.

How do you think about to avoid that this software depends on undefined behaviour?

fixed in<PHONE_NUMBER>5e93c8453f6c0b55a5dd0b93f668fa  thank you!

Thanks for the renaming of two include guards.
Are there a few update candidates left over?

Can we skip the guessing games? ☺


Dead debug state

run VS Code
have a crash (sorry, fuzzy...)
kill debuggee from dock
disconnect/stop from viewlet
the UI still things it is debugging

I've seen this kind of thing once in awhile but haven't figured out what can cause it exactly.

I think this can be reliably reproduced if the extension host crashes (or is killed)


FAB-18210 etcdraft.Chain private halt() respect exported Halt()

Type of change

Bug fix

Description
After etcdraft.Chain Halt() was called, e.g. from RemoveChannel, it might happen
that the internal halt() is still called by one of the chain's
go-routine. In this case, halt() should not call the haltCallback.

Related issues
Issue: FAB-18210

Looks good, but no test?

Voila


function as default value

Hi there,
I am just coming from SimpleSchema and want to translate my schemas to astronomy.
I didn't find the right place in your documentation.. I want to add the current users id to the owner field of my schema which worked like this in SimpleSchema:
owner: {
    type: String,
    default:function(){ return this.userId }
}

Is that possible with astronomy? Do I have to register an event for that?
Cheers
Marc

Hi, yes right now you have to use the beforeInsert event. However, it's good idea to implement such feature. I will add it in the next release.

I see you are not lazy Mr.Myjagi! :+1:

Feature has been implemented and will appear in the next release. It's getting closer to this moment. So stay tuned :). For now I'm closing this issue.

I don't understand how to use the beforeInsert event to achieve this. If I want to capture the userId into a field before insert, I imagine it would look like this:
...
events: {
    'beforeInsert': function(e) {
      this.set('creatorId', Meteor.userId());
  },
...

But this error appears: Error: Meteor.userId can only be invoked in method calls. Use this.userId in publish functions.
How would I achieve this using the Event system in Astronomy?

You should create a new issue for this question, however I will answer your it here.
If you are using Meteor.userId() you should be aware of the context in which you have to invoke it. As the error states you should call it in the Meteor method. As the beforeInsert event gets called when invoking the save() method, so you should put the code doc.save() in a method.
Meteor.methods({
  '/doc/save': function(doc) {
    doc.save();
  }
});

And call method:
var doc = new Doc();
Meteor.call('/doc/save', doc);

There's not other way around to save it and get userId. You can't just write the doc.save() on the client and use allow/deny rules if you are using Meteor.userId(). It's not something Astronomy specific, it's how it works in Meteor.
I've read somewhere a discussion of MDG with developers about removing allow/deny rules from Meteor and force users to create methods instead. In fact allow/deny rules and Collection.insert() etc. functions are using Meteor methods to perform an action. However, not many developers are aware of that.
You can also read many articles about not using allow/deny rules because they are not reliable and can cause many hidden security holes. I would encourage you to avoid using them.


Special case "Limit already reached" after app reset

Where to find the issue
[Google/Android]: Error when communicating with Google API (39508)
https://www.coronawarn.app/en/faq/#API39508
Also work-in-progress PR #789.
Describe the issue
As described in "Reset App late in day causes "Limit already reached" error message" https://github.com/corona-warn-app/cwa-app-android/issues/2248 the error message "Limit already reached" can be caused by resetting the app late in the day after approximately 20:00 UTC (21:00 CET or 22:00 CEST) when the app has been operating correctly and is permanently connected to WiFi, so that it will have performed a risk evaluation every 4 hours.
According to https://developers.google.com/android/exposure-notifications/exposure-notifications-api#modes provideDiagnosisKeys quota is Six per day (counter resets at 00:00 UTC) in ExposureWindow mode (aka v2). So if the first submission is done at 00:01 UTC, followed by 04:01, 08:01, 12:01, 16:01 at 20:01 the quota of six submissions for the day has already been used up. This should not normally matter, because the next submission will not be due until after 24:00 UTC. However, Reset App, could provoke an additional submission and thus exceed the quota.
This Reset App issue will not be fixed (see https://github.com/corona-warn-app/cwa-app-android/issues/2248#issuecomment-773900831) since it is considered an edge case and would be complex to work around.
The situation is not described in the FAQ https://www.coronawarn.app/en/faq/#API39508.
Suggested change
In the draft PR #789, after the sentence:
"However, should you experience an issue related to this error notification, please submit your report here: associated GitHub issue 1459"
add
EN
If you have just carried out "Reset App" in the app Settings and you then receive the error message "Limit already reached", please follow the instructions in the message and wait until the next day for the limit to be automatically reset.
DE
Wenn Sie gerade "Anwendung zurücksetzen" in den Einstellungen der App durchgeführt haben, und danach die Fehlermeldung "Limit bereits erreicht" bekommen, dann warten Sie bitte bis zum nächsten Tag, dass der Limit automatisch zurück gesetzt wird, wie in der Meldung empfohlen.

Also relevant: https://github.com/corona-warn-app/cwa-website/pull/895

This issue was resolved by PR #895, which was merged on May 3, 2021.


Bruce Williams

{
    "preferred_name": "Bruce Williams",
    "csulb_email"<EMAIL_ADDRESS>    "pronouns": "he/him/his",
    "previous_gis_experience": "Discovered GIS in 2018, after purchasing a drone to capture aerial imagery. After taking a drone mapping class, I discovered the world of GIS and have been enthusiastically studying the subject ever since. To that end, I completed a year of study and earned my GIS Certificate at Rio Hondo College. I have attended several Esri User Conferences and \n an Esri Developers Summit seeking to add to my understanding of the topic. I have a B.S. in Information Systems from California Baptist University and am currently studying for my Master of Science in GIS at California State University Long Beach.",
    "previous_remote_sensing_experience": "One semester of Remote Sensing at CSULB.",
    "previous_programming_experience": "Three semester of Python",
    "github_repo_link": "https://github.com/nullislandlife/GISC606",
    "github_project_link": "https://github.com/users/nullislandlife/projects/4",
    "what_you_hope_to_get_out_of_this_class": "I am looking for a functional knowledge of Remote Sensing"
}

Hey @nullislandlife,
Thanks for sharing about yourself. Could you also share about your larger MS project taking place outside this class?
I'd like to curate the course content to the interests of the students (including you), so anything you can share about your project or interests in remote sensing will help me. It sounds like you're most interested in content related to practical applications of remote sensing and perhaps some automation too?
Along those lines, I'd like you to start thinking about your final project for this class. I'm encouraging all students to pick a topic related to their larger MS project, but you are free to select any apply remote sensing topic of interest.
Look forward to working with you this semester.
Best,
Alex

https://github.com/nullislandlife/GISC606

https://github.com/nullislandlife/GISC606/blob/main/GISC606_L2.ipynb

https://github.com/nullislandlife/GISC606/blob/main/GISC606_L3.ipynb

I thought I had posted my Lab01 assignment, but it looks like I posted it but did not comment as instructed. Please forgive the late assignment.
https://github.com/nullislandlife/GISC606/blob/main/GISC606_Lab1.ipynb

https://github.com/nullislandlife/GISC606/blob/main/GISC606_L4.ipynb

https://nullislandlife.github.io/Lab5/


Questions about the setting

Thank you for your nice work.
I just want to ask about that which the SSL consistency loss used in your AKC/ARC/AKC+ARC in the Papar Table 1.

I guess it is the basic threshold-based Pseudo-labeling without sharpening(MixMatch) or Argmax (Fixmatch).

Hi,
In the table, the method of "AKC", "ARC", and "AKC+ARC" did not use any other SSL consistency loss (only using AKC or/and ARC in our paper).
The method of "MixMatch+AKC+ARC" used the additional SSL consistency loss in MixMatch, and "Fixmatch +AKC+ARC" used the additional SSL consistency loss in Fixmatch paper (similar to threshold-based Pseudo-labeling with strong data augmentation).

Ok, I see. So in other words, AKC+ARC(Labeled Data and Unlabeled Data)+CE(Labeled Data) can achieve almost SOTA performance?  That's cool!!!

Yes,
It is right for CUB-200-2011 dataset, especially for a small amount of unlabeled data.
But for CIFAR-10, also needs to add additional SSL consistency loss.

Yes,
It is right for CUB-200-2011 dataset, especially for a small amount of unlabeled data.
But for CIFAR-10, also needs to add additional SSL consistency loss.

I See. Thanks again for this nice work!


Tool object requires License object

Problem description
When you do a requests.GET call on the api/v1/tool endpoint, you will get a dict like this:
{'author': 'The NLP Sandbox Team',
 'authorEmail'<EMAIL_ADDRESS> 'description': 'An example implementation of the NLP Sandbox Date Annotator',
 'license': 'apache-2.0',
 'name': 'date-annotator-example',
 'repository': 'github:nlpsandbox/date-annotator-example',
 'toolApiVersion': '1.0.0',
 'toolType': 'nlpsandbox:date-annotator',
 'url': 'https://github.com/nlpsandbox/date-annotator-example',
 'version': '1.0.0'}

An easy way to validate this is to simply use the model that is auto generated by openapi-generator but that isn't possible, because you will get an error like this
annotator.models.Tool(**tool_response_json)
...
annotator.exceptions.ApiTypeError: Invalid type for variable 'license'. Required value type is License and passed type was str at ['license']

Steps to reproduce the issue
from annotator.models import Tool

tool_info = {'author': 'The NLP Sandbox Team',
 'author_email'<EMAIL_ADDRESS> 'description': 'An example implementation of the NLP Sandbox Date Annotator',
 'license': 'apache-2.0',
 'name': 'date-annotator-example',
 'repository': 'github:nlpsandbox/date-annotator-example',
 'tool_api_version': '1.0.0',
 'tool_type': 'nlpsandbox:date-annotator',
 'url': 'https://github.com/nlpsandbox/date-annotator-example',
 'version': '1.0.0'}
Tool(**tool_info)
---------------------------------------------------------------------------
ApiTypeError                              Traceback (most recent call last)
<ipython-input-2-21202fccb122> in <module>
     11  'url': 'https://github.com/nlpsandbox/date-annotator-example',
     12  'version': '1.0.0'}
---> 13 Tool(**tool_info)

~/sage/dream/nlp/nlp-sandbox-client/annotator/model_utils.py in wrapped_init(self, *args, **kwargs)
   1562         if spec_property_naming:
   1563             kwargs = change_keys_js_to_python(kwargs, self.__class__)
-> 1564         return fn(self, *args, **kwargs)
   1565     return wrapped_init
   1566

~/sage/dream/nlp/nlp-sandbox-client/annotator/model/tool.py in __init__(self, name, version, license, repository, description, author, author_email, url, tool_type, tool_api_version, *args, **kwargs)
    220         self.name = name
    221         self.version = version
--> 222         self.license = license
    223         self.repository = repository
    224         self.description = description

~/sage/dream/nlp/nlp-sandbox-client/annotator/model_utils.py in __setattr__(self, attr, value)
    166     def __setattr__(self, attr, value):
    167         """set the value of an attribute using dot notation: `instance.attr = val`"""
--> 168         self[attr] = value
    169
    170     def __getattr__(self, attr):

~/sage/dream/nlp/nlp-sandbox-client/annotator/model_utils.py in __setitem__(self, name, value)
    351             return
    352
--> 353         self.set_attribute(name, value)
    354
    355     def get(self, name, default=None):

~/sage/dream/nlp/nlp-sandbox-client/annotator/model_utils.py in set_attribute(self, name, value)
    140             value = validate_and_convert_types(
    141                 value, required_types_mixed, path_to_item, self._spec_property_naming,
--> 142                 self._check_type, configuration=self._configuration)
    143         if (name,) in self.allowed_values:
    144             check_allowed_values(

~/sage/dream/nlp/nlp-sandbox-client/annotator/model_utils.py in validate_and_convert_types(input_value, required_types_mixed, path_to_item, spec_property_naming, _check_type, configuration)
   1406         else:
   1407             raise get_type_error(input_value, path_to_item, valid_classes,
-> 1408                                  key_type=False)
   1409
   1410     # input_value's type is in valid_classes

ApiTypeError: Invalid type for variable 'license'. Required value type is License and passed type was str at ['license']

Expected behavior
Ideally, license doesn't have to be an object, as it is just a string.
Alternative solutions
That being said, this isn't really an issue if you use the SDK client to obtain the tool, but it is an issue when trying to test if the tool redirects / to /tool
import annotator
from annotator.api import tool_api
configuration = annotator.Configuration(host=host)
with annotator.ApiClient(configuration) as api_client:
    tool_instance = tool_api.ToolApi(api_client)
    tool_info = tool_instance.get_tool()

When you do a requests.GET call on the api/v1/tool endpoint, you will get a dict like this:

Do we really get a dictionary or a Tool object?

That being said, this isn't really an issue if you use the SDK client to obtain the tool, but it is an issue when trying to test if the tool redirects / to /tool

What is the issue exactly ? Not getting a Tool object when querying GET /? If we can get a Tool object from GET /api/v1/tool, why can't we get it from '/'?

That's the issue, currently you have to use the sdk to GET /api/v1/tool if you want a tool object.  Unless you do this:
tool_info = requests.get(....)
tool_info['license'] = License(tool_info['license'])
tool_obj = Tool(**tool_info)
It's not 100% an issue because we can just interact with the api with the SDK client function but the SDK client doesn't provide a function to GET / so I am using requests.GET.
Furthermore, this is also a formatting concern.  It is super odd to have objects in place for string values.

Closing as we have a solution for this.  That being said, it is still good to be aware of this issue.


no such container404

When trying to create a container from blalor/centos, I get an error: no such container404No such image: blalor/centos:latest (tag:latest)
Also, some spaces are missing in that message - it should be: no such container 404 No such image: blalor/centos:latest (tag:latest)
or: no such container: 404 No such image: blalor/centos:latest (tag:latest)

When i pick an exact version, not latest, I get: server error500No command specified

@bargulg It appears that image is a base container. Since it doesn't give Kitematic the commands necessary, you'll have to start it via command line.
Try the Docker CLI button on Kitematic (bottom left)

@mchiang0610 that explains error 500, but not error 404

@bargulg 404 means Kitematic had trouble pulling. (We shouldn't display a 404 error in the future. We put most errors into 404 state but doesn't print out the actual error. This will be corrected in the future builds.

merging this with issue https://github.com/kitematic/kitematic/issues/626

When I try to install MySQL I get a 404 in kitematic, and this on terminal:
bash-3.2$ docker pull mysql
latest: Pulling from mysql
a16207605ac6: Download complete
a16207605ac6: Error downloading dependent layers
ea994adb8f5e: Downloading [===========>                                       ] 1.801 MB/8.069 MB
b9a671861279: Download complete
a58de78a6a7f: Download complete
12724ce38086: Download complete
6875e56d8c4c: Layer already being pulled by another client. Waiting.
6bafb24bd6c1: Pulling fs layer
6bafb24bd6c1: Download complete
4d217892bcfe: Download complete
b741ec2037b8: Download complete
96f8e8f069ed: Download complete
9eefddb060bc: Download complete
9eefddb060bc: Error pulling image (latest) from mysql, Driver aufs failed to create image rootfs a16207605ac67b9cf0ff3c9ff208d276a9e2020bfa545a80be9ddb9911104757: open /mnt/sda1/var/lib/docker/aufs/layers/c12da83ee7c184e35558b5cc8953b1f1035c8c232bda7ff0a4ba53ead3dad337: no such file or directory a4ba53ead3dad337: no such file or directory
c12da83ee7c1: Download complete
FATA[0021] Error pulling image (latest) from mysql, Driver aufs failed to create image rootfs a16207605ac67b9cf0ff3c9ff208d276a9e2020bfa545a80be9ddb9911104757: open /mnt/sda1/var/lib/docker/aufs/layers/c12da83ee7c184e35558b5cc8953b1f1035c8c232bda7ff0a4ba53ead3dad337: no such file or directory
bash-3.2$

docker pull redis works, I get "image is up to date", but when I try to install in kitematic I get server 500 as per below:

any ideas?


the use of AES

Hi, @mofanv , it's really great work, and I have a question about the use of AES.
void aes_cbc_TA(char* xcrypt, float* gradient, int org_len)
{
    IMSG("aes_cbc_TA %s ing\n", xcrypt);
    //convert float array to uint_8 one by one
    uint8_t *byte;
    uint8_t array[org_len*4];
    for(int z = 0; z < org_len; z++){
        byte = (uint8_t*)(&gradient[z]);
        for(int y = 0; y < 4; y++){
            array[z*4 + y] = byte[y];
        }
    }

    //set ctx, iv, and key for aes
    int enc_len = (int)(org_len/4);
    struct AES_ctx ctx;
    uint8_t iv[] = { 0x00, 0x01, 0x02, 0x03, 0x04, 0x05, 0x06, 0x07, 0x08, 0x09, 0x0a, 0x0b, 0x0c, 0x0d, 0x0e, 0x0f };
    uint8_t key[16] = { (uint8_t)0x2b, (uint8_t)0x7e, (uint8_t)0x15, (uint8_t)0x16, (uint8_t)0x28, (uint8_t)0xae, (uint8_t)0xd2, (uint8_t)0xa6, (uint8_t)0xab, (uint8_t)0xf7, (uint8_t)0x15, (uint8_t)0x88, (uint8_t)0x09, (uint8_t)0xcf, (uint8_t)0x4f, (uint8_t)0x3c };

    //encryption
    AES_init_ctx_iv(&ctx, key, iv);
    for (int i = 0; i < enc_len; ++i)
    {
        if(strncmp(xcrypt, "encrypt", 2) == 0){
            AES_CBC_encrypt_buffer(&ctx, array + (i * 16), 16);
        }else if(strncmp(xcrypt, "decrypt", 2) == 0){
            AES_CBC_decrypt_buffer(&ctx, array + (i * 16), 16);
        }
    }

    //convert uint8_t to float one by one
    for(int z = 0; z < org_len; z++){
        gradient[z] = *(float*)(&array[z*4]);
    }
}

For the encryption, it seems that we do not output the ctxt? I want to use the workflow of client-server AES, that is, client encrypts the message and sends the ctxt to the server, then decrypted inside the enclave.

Hi @DylanWangWQF , thanks!
Please note that we were using hardcoded AES keys inside both server and client TEEs. So key management is not presented, and this file is actually not necessary for our prototype. I guess maybe a tiny AES c like this https://github.com/kokke/tiny-AES-c is what you need? I remember our AES submodule (during previous tests) is from there.

This tiny AES-c is helpful, thanks for your help!


Add Dynamic List of Byte Lists Support

This PR adds support for
type ETHMergeTransactions struct {
	OpaqueList [][]byte `json:"opaque_list" ssz-size:"?,?" ssz-max:"2048,2048"`
}

Which does not exist before. Being able to specify ssz-max:"2048,2048" is important for the eth1 <> eth2 merge functionality

Forgot about unmarshal, will edit

Please also run "make build-spec-tests" so that we can see the difference in the encodings to see if this changes affects any other struct.

Is this the correct encoding for a dynamic list? At least for other dynamic types without fixed size length you have to write offsets first.

@ferranbt getting this cannot happen when running the build
go run sszgen/*.go --path ./spectests/structs.go --include ./spectests/external,./spectests/external2
panic: it cannot happen

goroutine 1 [running]:
main.(*Value).unmarshalList(0xc000138e80, 0xc000012900, 0xc00030e170)
        /Users/me/Documents/Go/src/github.com/ferranbt/fastssz/sszgen/unmarshal.go:133 +0x6a5
main.(*Value).unmarshal(0xc000138e80, 0x11a46a8, 0x3, 0x2, 0xc000319978)
        /Users/me/Documents/Go/src/github.com/ferranbt/fastssz/sszgen/unmarshal.go:95 +0xbfd
main.(*Value).umarshalContainer(0xc000138e00, 0xc0002c5301, 0xc00030e170, 0xa, 0xc0002c14c8, 0xc0002abb00)
        /Users/me/Documents/Go/src/github.com/ferranbt/fastssz/sszgen/unmarshal.go:301 +0x120a
main.(*env).unmarshal(0xc00016fe38, 0xc0000164a0, 0x7, 0xc000138e00, 0xc000255000, 0x3da)
        /Users/me/Documents/Go/src/github.com/ferranbt/fastssz/sszgen/unmarshal.go:20 +0xd7
main.(*env).print(0xc00016fe38, 0x7ffeefbff501, 0xc00011a600, 0x1e, 0x20, 0xc00001c500, 0x1f, 0xc00010be38, 0x115faa0, 0xc00010be38, ...)
        /Users/me/Documents/Go/src/github.com/ferranbt/fastssz/sszgen/main.go:390 +0x62d
main.(*env).generateEncodings(0xc00016fe38, 0x0, 0x0, 0x1f, 0xc000073d00)
        /Users/me/Documents/Go/src/github.com/ferranbt/fastssz/sszgen/main.go:323 +0x1d4
main.encode(0x7ffeefbff521, 0x16, 0x12e4250, 0x0, 0x0, 0x0, 0x0, 0xc00005e020, 0x2, 0x2, ...)
        /Users/me/Documents/Go/src/github.com/ferranbt/fastssz/sszgen/main.go:99 +0x266
main.main()
        /Users/me/Documents/Go/src/github.com/ferranbt/fastssz/sszgen/main.go:41 +0x3cb
exit status 2

hard to identify what's going on, and yes it seems like we might need to add the offset

Figured it out!

Is there any way to test this with the spectests?

Is it intetional that ssz-size:"?,1024" ssz-max:"?,1024" doesn't work as it can be expected? maybe an error should be raised then.

Is this PR still working? AFAIK this is the only new type required for merge right?

Does this PR need to be merged before Transactions can be merkleized?

Closing the PR since this has already being implemented.


fetchBusinessSearch returns max 50 results

Docs on https://pub.dev/packages/yelp_fusion_client/versions/0.2.0+3 say fetchBusinessSearch() : Get up to 1000 businesses based on the provided search criteria, but setting limit parameter > 50 throws an error.

How do I get more than 50 results per request and how to get more than 1000 businesses?
The API can only return up to 1,000 results at this time. The API doesn't allow more than 50 per request. You need to use the offset parameter to get the next page of results. (For using offset, give it any number. If you have limit=50, that means you're getting results 1-50, so give it offset=51 and you'll get 51-100.)

Yelp Fusion - FAQ


ONNX Runtime GenAI C# API

This pull-request introduces the C# API for ONNX Runtime GenAI. The C# API follows the C API closely and leverages the C API as the interface with the ONNX Runtime GenAI core.
The API 📚
The C# API primarily revolves around three classes:

Model - Entry point for running GenAI with the library. Used for loading the model and the config.
GeneratorParams - Used to set the configurable parameters for search.
Generator - Main computation engine. Used for computing the logits and retrieving the generated tokens.

Sample usage 👨💻
using Model model = new Model(modelPath, DeviceType.CPU)
using GeneratorParams generatorParams = new GeneratorParams(model)

generatorParams.SetMaxLength(maxLength);
generatorParams.SetInputIDs(inputIDs, sequenceLength, batchSize);

using Generator generator = new Generator(model, generatorParams)

while (!generator.IsDone())
{
    generator.ComputeLogits();
    generator.GenerateNextTokenTop();
}

for (ulong i = 0; i < batchSize; i++)
{
    var sequence = generator.GetSequence((int)i);
    ... // use the output tokens
}

What does this Pull-Request Include? 🤔

The C# API for Windows. The build generates a managed dll called Microsoft.ML.OnnxRuntimeGenAI.dll for x64 using net8.0.
Test that leverages the managed dll built.
This pull-request does not include packaging.
Module definition for exporting the native functions in the C library.

What's next? 🔄

Run build and tests in CI pipeline.
C# packaging.
Linux C# support
Build the C# API as part of the cmake build.
Tokenizer functions to be added to C# API to complete the API

The summary example features nested using blocks. It would be better to use using statements that are scope based, so blocks are not required, although they may be useful when you want to dispose of something before the current scope ends.

The summary example features nested using blocks. It would be better to use using statements that are scope based, so blocks are not required, although they may be useful when you want to dispose of something before the current scope ends.

Updated the sample example to avoid using the nested using statements.

Thank you for the review @yuslepukhin @RyanUnderhill :)


docs: Link the new at_mono doc in README.md

- What I did
Removed the old explanation of "why create this repo" and replaced it with a link to the new doc.
- How I did it
- How to verify it
- Description for the changelog
docs: Link the new at_mono doc in README.md

@cpswan Added back the paragraph and updated the contents so that it aligns better with the write up in .github.


Not able to get the documentation using hapi-swagger

When i hit documentation with my API's URL i end up with
500 : {"statusCode":500,"error":"Internal Server Error","message":"An internal server error occurred"} http://krsa15:3000/swagger.json
And in my terminal i see this exception being caught.
Debug: internal, implementation, error
ValidationError: Uncaught error: {
"basePath": "/",
"schemes": [
"http"
],
"swagger": "2.0",
"host": "krsa15:3000",
"version" [1]: "1.0.0"
}
[1] "version" is not allowed

Hi since the plug-in moved to v3.x it has validated the options. Also at that time the options where changed to more closely match Swaggers properties, some of these changes where breaking.  The error you are getting looks like your using the older options structure with a newer version of the plug-in. Please update to the latest version which is currently v4.2.0, then make sure your options match the documentation on the readme. If you are still having problem can you please provide the registration code your using for the plug-in and any options you are setting.
I hope the  above helps, can you let me known if this information revolved the problem or not so I can close this issue

I am using hapi-swagger plugin .. Here is my package.json file data..
"dependencies": {
"boom": "^3.1.2",
"devtool": "^1.7.6",
"hapi": "^12.0.1",
"hapi-mongodb": "^4.1.0",
"hapi-swagger": "^4.0.0",
"inert": "^3.2.0",
"joi": "^7.1.0",
"kafka-node": "^0.3.1",
"kafkaesque": "^0.1.1",
"mongodb": "^2.1.4",
"vision": "^4.0.1"
}

OK your using a new new version of the plug-in so your code should look something like:
const Hapi = require('hapi');
const Inert = require('inert');
const Vision = require('vision');
const HapiSwagger = require('hapi-swagger');

const server = new Hapi.Server();
server.connection({
        host: 'localhost',
        port: 3000
    });

const options = {
    info: {
            'title': 'Test API Documentation',
            'version': '16.5.0',
        }
    };

server.register([
    Inert,
    Vision,
    {
        'register': HapiSwagger,
        'options': options
    }], (err) => {
        server.start( (err) => {
           if (err) {
                console.log(err);
            } else {
                console.log('Server running at:', server.info.uri);
            }
        });
    });

server.route(....);

If you give the above code a go it should work. You will need to add some routes with the tags=['api'] property.
The error suggest that the option object above is in the wrong format. If your still having problems can you  give the code you use for setting up plug-ins and their options like the code above.

I'm triple checking syntax but we are not getting the documentation to show either on brand new dev as well. Also, interestingly when we also run be-more-hapi we're getting the same result... we get the docu page view and have the header, the readme, etc but the core swagger ui isn't rendering. Noticed getting a 404 on /swaggerui/lib/underscore-min.js. Still looking at things but thought worth reporting given the active dialog here...

Hi sorry to hear your having so many problems.
The custom page in be-more-hapi example was out of date and so was the code example for custom pages on the readme in the hapi-swagger repo. I have now updated both. If you get the latest version of be-more-hapi either from the repo or npm it should now work correctly.
If you could tell if the be-more-hapi project is now works for you that would be great. I still believe that the original issue is caused by something else, so eliminating this problem first will be useful.
PS I am sorry about outdate code in the  be-more-hapi repo, hope it did not waste too much of your time.

Hi Glenn... huge thanks. Things are working fine. I agree with you that the original issue is something different but we're sorted on this other aspect now. Again- huge thanks and nice work on this... have a good one!


BackgroundTransfer with FTP source doesn't work on ARM

When running the BackgroundTransfer > CS> Scenario1_Download sample with an external FTP source address, the following error is received when debugging on ARM:
Execution error: Error HRESULT E_FAIL has been returned from a call to a COM component.
The exact same code works fine when compiled for x86 / x64.

Thanks for the report. We'll investigate to determine whether this is a product bug or a platform limitation.


Adding power support to travis.yml

Signed-off-by: ghatwala<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build 1393

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

Totals

Change from base Build 1391:
0.0%

Covered Lines:
12587

Relevant Lines:
17851

💛  - Coveralls

Pull Request Test Coverage Report for Build 1393

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

Totals

Change from base Build 1391:
0.0%

Covered Lines:
12587

Relevant Lines:
17851

💛  - Coveralls

Hi all , any updates here ?


Update docs about toggleElement of Accordion

Description
As a part of improvement of Accordion docs, I added content about using toggleElement of Accordion.
In fact I had to work about basic Accordion, styled Accordion, and some customizations, but @luke-hanwook have worked some already. I would really like to appreciate your hard work, and apologize for my late. I couldn't work for this because I was busy to work on my job, and I was sick suddenly. However, this is all an excuse, and although it is an open source, I want to apologize for being late for what I promised to do together.
Some commits are not meaningful to remain that on commit history, so I think squash this branch or some commits is the best way.
I've only done a little bit of work thanks to @luke-hanwook, so please let me know if there are more tasks I can do. I will do that task in this PR(or in new PR if needed).
Test Plan
None
Related Issues
None
Tests
None
Checklist
Before you create this PR confirms that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] Run yarn test:all and make sure nothing fails. You can run yarn test -u to update snapshots if needed.
[x] I am willing to follow-up on review comments in a timely manner.

Thanks! sorry for the late review.


Use LoopVectorization.@turbo for evaluation loops

This switches from @inbounds @simd to @turbo for most of the inner evaluation loops. There is a 30% speedup which is huge.
However, there's currently a StackOverflowError when running the Float16 tests. I'm also not sure if LoopVectorization will be safe to use, since ideally I want to allow for any user-defined operator - would it mean certain passed operators fail? Also need to test it on a distributed system.

Okay looks like LoopVectorization.jl hasn't even implemented SpecialFunctions, and will raise a StackOverflowError if any of them are used (as an operator): https://github.com/JuliaSIMD/LoopVectorization.jl/issues/233. I'm only seeing it in Float16 since that's the first test (which tries a gamma function as an operator).
So this change seems not possible at the moment, unless @turbo is capable of falling back to non-SIMD operations to avoid StackOverflowErrors.

Raised issue on https://github.com/JuliaSIMD/LoopVectorization.jl/issues/430

Probably also want to switch to using similar throughout, rather than Array{T,1}(undef, n). This is also needed independent of this change for generality to input array type.

With https://github.com/JuliaSIMD/LoopVectorization.jl/pull/431 being merged, this should now work. Let's see how stable things are.


Possible Casting Error?

Hi,
May I ask if there is a possible casting error here at line 251, which happens when command instanceof BatchExecutionCommand = true ?
If that would be an casting error, is the try-catch construct (line 244-line 262) around it meant to handle such errors?
https://github.com/apache/incubator-kie-drools/blob/b00ef1d816c03dd3ff450d507c1ba70008629073/drools-kiesession/src/main/java/org/drools/kiesession/session/StatelessKnowledgeSessionImpl.java#L245-L251
Best,
Yahui

Why are you asking? Can you reproduce any CCE? In reality all ours Command are also ExecutableCommand (and yes, we could probably unify those 2 interfaces into a single one) so this should never happen. That said feel free to reopen this issue in case you can reproduce the CCE.


Install Failed, Fix below.

Install failed, looks like one of the links for chocolatey has gone down.
https://boxstarter.org/bootstrapper.ps1 results in 404.
The following works as an alternate link:
https://raw.githubusercontent.com/chocolatey/boxstarter/master/BuildScripts/bootstrapper.ps1

Hah, I just logged in to GitHub to open the very same issue. I was struggling for a bit as there was not a single error message that would indicate what is actually an issue.

This should be fixed by boxstarter, because the link https://boxstarter.org/bootstrapper.ps1 has been the default for years. Not only FLARE-VM is affected, probably many other software is failing because of this problem. The issue is being tracked here
The link provided above is a nice temporary fix to get the installation script of FLARE-VM to work, but it points to the master branch (which is in active development). A better fix should use the latest release (v2.13.0) to get bootstrapper.ps1
A Better Alternate Link
https://raw.githubusercontent.com/chocolatey/boxstarter/v2.13.0/BuildScripts/bootstrapper.ps1

Thanks for bringing this up.
I would recommend not using a static raw link to an old Boxstarter release, we'll fix the corresponding issue on our site asap.
sorry for the inconvenience.

Please note this issue has been resolved and should not occur anymore.

Thanks for the fix :)

Thanks, just verified good on my end.


fix: retained refresh token when auth token refresh request is executed

Description

Retained refresh token from initial setup as refresh token is not returned usually in OAuth2 refresh token flow.

PR fixes following issue(s)
Fixes #24767
Type of change

Bug fix (non-breaking change which fixes an issue)

Testing

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[x] Manual
[ ] Jest
[ ] Cypress

Test Plan

Create authenticated rest api datasource and let its token expire or edit token expiry time under datasourcConfiguration field somewhere. Then execute an api call and the call should return successfully.

Checklist:
Dev activity

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

QA activity:

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

/ok-to-test

/build-deploy-preview

/ok-to-test

/ok-to-test


Cargo watch terminates when binary crashes

Cargo watch terminates when the binary that it executes crashes, but it would be better if it could keep running..

Ah nvm, it doesn't terminate -.-


Each attribute in metadata needs to exist somewhere else in the OME-Zarr file

This would be the best-case scenario. Let's see whether we can enforce it strictly.

Result from some conversation with people using OME-Zarrs:

We should not be using the OME-XML transitionary metadata. It's not meant for long-term and we don't need to get into that
We should be able to store arbitrary key-value pairs with metadata in the .zattrs. Let's think about where in the .zattrs they should go for each case and, if it's something more general, let's try to contribute it back to the spec

Are we currently using any of those? (or: is there an easily-accessible list?)

No, we aren't using it. It would have been one of the options to store additional metadata, see details here: https://ngff.openmicroscopy.org/latest/#bf2raw
I'm glad we don't need to get into this :)

Current items that are in metadata and that we could think about moving to .zattrs:

Lists of plates/wells/images. These are already present in the .zattrs files, but it is convenient to duplicate them in the metadata to avoid a lot of parsing and string styling here and there.
Number of pyramid levels. This is also already available in the .zattrs files, if we accept that tasks should start by reading this information.
Coarsening factor. This is not directly available, but it can be retrieved with a bit of work from the scale transformations.
Channel list. This currently has an (ordered!) list like ["A01_C01", ...]. Related to #211.

Then there are a few attributes which we use to smoothly propagate some parameters within pairs of associated tasks (server-side ref https://github.com/fractal-analytics-platform/fractal-server/issues/6). This is related to #177, that is, the choice of how to split arguments into two sets: the ones to be filled in automatically by fractal (e.g. component will most likely be here) and the ones to be specified as part of args.
Currently, we use
5. original_paths, to pass the image folders from zarr-creation task to yokogawa_to_zarr
6. replicate_zarr attributes (again, there are some paths of the original file to replicate), which are then used for the MIP task. This one we could in principle refactor, since it is probably a bit redundant.
Unless I've missed something, this is the current list.
Note that some of these parameters already exist in two forms, for (non-)multiplexing cases.

Broadly speaking, I think that adding some key-value pairs to some .zattrs could be extremely helpful, e.g., for #211 (and maybe somehow also for #199 or #200).

Note that once PR #239 is merge we won't have channel_list in metadata any more.

Yesterday we re-discussed this issue, and we identified multiple (current) uses of metadata. Most of them should be deprecated, in favor of more specific sources of information.

Provide read/write access to component list ("read" from fractal-server, "write" from a combination of tasks and fractal-server). We plan to defer this functionality to a new task (ref #521). TBD later if this has to be a standard task or the "init" phase of a new, more complex, task object.
Store dataset information (e.g. coarsening_xy and the number of pyramid levels). This use is now deferred to #351; and then this use should be deprecated, and these parameters should not belong to Dataset.meta.
Store the dataset history -> this should not pass through meta any more - ref https://github.com/fractal-analytics-platform/fractal-server/issues/838.
Exchange information in a pair of tasks - ref #299. This is a way of using meta as a temporary buffer.

Since each use of metadata is related to a specific issue, I'm closing this one.


MangaDex tracker doesn't update chapter count

Steps to reproduce
Try selecting a chapter number.
Then click ok
Go to manga page, then go to tracker page back.
https://user-images.githubusercontent.com/63583961/182122185-73967fc7-970a-4a16-9858-39f5c5cc88b8.mp4
Expected behavior
It should set it as how i set
Actual behavior
It reverts back to 0
Crash logs
No response
Tachiyomi version
1.8.4
Android version
Android 11
Device
Tecno Spark 7T
Other details
No response
Acknowledgements

[X] This is a TachiyomiSY specific issue that does not happen in Tachiyomi Preview.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have tried the troubleshooting guide.
[X] I have updated the app to version 1.8.4.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

Known limitation with the Mangadex api. Can't do anything about it for now


Provide an API to support charging planning done in the EV and provided via the EVSE or online API

Is your feature request related to a problem? Please describe.
Some EVs (Audi Q8 e-tron, Taycan, Mercedes EQE/EQS, VW MEB EVs with software version >= 3.1) support communicating via ISO15118 over AC to the EVSE. These EVs also do support setting up charging plans in the EV itself or the app (which sends the data to the EV).
This includes:

Minimum SoC: charging to that SoC level as quickly as possible
Timers with dates/weekdays/time: Target SoC and/or climating
Direct charging option: works like the "Now" mode in EVCC
Maximum SoC: when no timer is set having an maximum SoC level

All this information can be communicated via compatible EVSEs and therefor be provided to evcc. Some EVSEs that support this are Porsche/Audi Mobile Charger Connect, OpenWB Pro (as far as I know technically, not sure if it provides an API for that yet), more coming.
Important note: the EV will not send SoC values for the targets but energy (Wh) demands.
Describe the solution you'd like
Right now some of this data, e.g. minimum SoC, is for some EVs fetched via the Online API. All the others values can only be set via the evcc API or the UI.
It would be great if there would be an Go API to provide all these either via the Online API or the EVSE. In that case the evcc UI might not allow switching modes or setting/changing the plans but mark the settings being fetched from the EV itself.

The EEbus charger already has access to loadpoint and it's api. Most functionality is available, schedule is not available at all. It's not clear what is missing?

Being able to:

Set minimum charge energy amount (which could be translated to an SoC for displaying purpose)
Set charging target time and energy amount via the charger implementation (which could be translated to an SoC for displaying purpose, but not 100% if the energy is required for climating as well which is unknown)
Get power tariffs for a duration
In these cases evcc should only limit the charging in the case of overload protection scenarios or PV surplus charging (depends on the EVs charging mode and demand)
A way to tell the UI that this data is from the EV and changes need to be done in the EV instead of the UI. To be clarified how this for all the possible cases could be done

So there are interfaces missing to do these and then handling of the different cases need to be defined wether what evcc will do and what not in adjusting the limits as well as how the UI would work and what it would allow.

No, these are not possible. But thanks for looking into it.


KeyError occurs when importing the meta file

Hi,
After I download the pre-trained model "20161030-023650", I want to restore the model by load_model() in "facenet.py".  Unfortunately, a KeyError occurs:
In [15]: facenet.load_model('20161030-023650/', meta_file, ckpt_file)
KeyError                                  Traceback (most recent call last)
 in ()
----> 1 facenet.load_model('20161030-023650/', meta_file, ckpt_file)
facenet.py in load_model(model_dir, meta_file, ckpt_file)
468 def load_model(model_dir, meta_file, ckpt_file):
469   model_dir_exp = os.path.expanduser(model_dir)
--> 470   saver = tf.train.import_meta_graph(os.path.join(model_dir_exp, meta_file))
471   saver.restore(tf.get_default_session(), os.path.join(model_dir_exp,ckpt_file))
472
/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.pyc in import_meta_graph(meta_graph_or_file)
1456     return _import_meta_graph_def(meta_graph_or_file)
1457   else:
-> 1458     return _import_meta_graph_def(read_meta_graph_file(meta_graph_or_file))
1459
1460
/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/saver.pyc in _import_meta_graph_def(meta_graph_def)
1346     producer_op_list = meta_graph_def.meta_info_def.stripped_op_list
1347   importer.import_graph_def(meta_graph_def.graph_def, name="",
-> 1348                             producer_op_list=producer_op_list)
1349
1350   # Restores all the other collections.
/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/importer.pyc in import_graph_def(graph_def, input_map, return_elements, name, op_dict, producer_op_list)
250     for node in graph_def.node:
251       # Set any default attr values that aren't present.
--> 252       op_def = op_dict[node.op]
253       for attr_def in op_def.attr:
254         key = attr_def.name
KeyError: u'RsqrtGrad'
I use Tensorflow 0.10 in Ubuntu, with CUDA 7.5 and CUDNN 4.
How can I solve this problem?

I have exported the model as a graphdef protobuf that can be evaluated with a new module validate_on_lfw_new.py.
It should just be to download the model and point to it when running evaluation, something like
python validate_on_lfw_new.py ~/datasets/lfw/lfw_mtcnnpy_160 ~/models/export/20161030-023650.pb

That seems work for evaluation.
However, if the weights in .pb file is from a freezed graph, can I just import_graph_def it and re-train the model?

If you want to retrain the model another alternative is to create the model programatically, i.e. by calling inference(...) for the model, like it's done in facenet_train_classifier.py.
And then you can load the file containing the parameters, i.e. model-20161030-023650.ckpt-80000 using saver.restore(...) in the same way as it's done in load_model(...).

OK, I get it now. Thanks for your explanation!

Which tensorflow version is the model trained for the create the graph file then?

Hi David,
First of all, thank you for sharing you great work.
I have a problem importing the meta file.
(I use Tensorflow 0.10 in Ubuntu 14.04 LTS, with CUDA 7.5 and CUDNN 5. )

when running validate_on_lfw.py, the below error is thrown.

Model directory: ../models/20161030-023650
Metagraph file: model-20161030-023650.meta
Checkpoint file: model-20161030-023650.ckpt-80000
Traceback (most recent call last):
File "validate_on_lfw.py", line 89, in
main(parse_arguments(sys.argv[1:]))
File "validate_on_lfw.py", line 57, in main
facenet.load_model(args.model_dir, meta_file, ckpt_file)
File "/data/deep_learning/acl_work/facenet/src/facenet.py", line 469, in load_model
saver = tf.train.import_meta_graph(os.path.join(model_dir_exp, meta_file))
File "/home/misunpar/tensorflow/local/lib/python2.7/site-packages/tensorflow/python/training/saver.py", line 1458, in import_meta_graph
return _import_meta_graph_def(read_meta_graph_file(meta_graph_or_file))
File "/home/misunpar/tensorflow/local/lib/python2.7/site-packages/tensorflow/python/training/saver.py", line 1348, in _import_meta_graph_def
producer_op_list=producer_op_list)
File "/home/misunpar/tensorflow/local/lib/python2.7/site-packages/tensorflow/python/framework/importer.py", line 252, in import_graph_def
op_def = op_dict[node.op]
KeyError: u'RsqrtGrad'

when running validate_on_lfw_new.py w/ your exported pb file, it also threw the below error.

E tensorflow/core/common_runtime/executor.cc:334] Executor failed to create kernel. Invalid argument: NodeDef mentions attr 'out_type' not in Op<name=Shape; signature=input:T -> output:int32; attr=T:type>; NodeDef: import/InceptionResnetV1/Conv2d_1a_3x3/BatchNorm/moments/sufficient_statistics/Shape = ShapeT=DT_FLOAT, out_type=DT_INT32, _device="/job:localhost/replica:0/task:0/gpu:0"
[[Node: import/InceptionResnetV1/Conv2d_1a_3x3/BatchNorm/moments/sufficient_statistics/Shape = ShapeT=DT_FLOAT, out_type=DT_INT32, _device="/job:localhost/replica:0/task:0/gpu:0"]]
Traceback (most recent call last):
File "validate_on_lfw_new.py", line 87, in
main(parse_arguments(sys.argv[1:]))
File "validate_on_lfw_new.py", line 61, in main
images_placeholder, phase_train_placeholder, embeddings, nrof_folds=args.lfw_nrof_folds)
File "/data/deep_learning/acl_work/facenet/src/lfw.py", line 51, in validate
emb_array[start_index:end_index,:] = sess.run(embeddings, feed_dict=feed_dict)
File "/home/misunpar/tensorflow/local/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 710, in run
run_metadata_ptr)
File "/home/misunpar/tensorflow/local/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 908, in _run
feed_dict_string, options, run_metadata)
File "/home/misunpar/tensorflow/local/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 958, in _do_run
target_list, options, run_metadata)
File "/home/misunpar/tensorflow/local/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 978, in _do_call
raise type(e)(node_def, op, message)
tensorflow.python.framework.errors.InvalidArgumentError: NodeDef mentions attr 'out_type' not in Op<name=Shape; signature=input:T -> output:int32; attr=T:type>; NodeDef: import/InceptionResnetV1/Conv2d_1a_3x3/BatchNorm/moments/sufficient_statistics/Shape = ShapeT=DT_FLOAT, out_type=DT_INT32, _device="/job:localhost/replica:0/task:0/gpu:0"
[[Node: import/InceptionResnetV1/Conv2d_1a_3x3/BatchNorm/moments/sufficient_statistics/Shape = ShapeT=DT_FLOAT, out_type=DT_INT32, _device="/job:localhost/replica:0/task:0/gpu:0"]]
Caused by op u'import/InceptionResnetV1/Conv2d_1a_3x3/BatchNorm/moments/sufficient_statistics/Shape', defined at:
File "validate_on_lfw_new.py", line 87, in
main(parse_arguments(sys.argv[1:]))
File "validate_on_lfw_new.py", line 50, in main
return_elements=return_elements, name='import')
File "/home/misunpar/tensorflow/local/lib/python2.7/site-packages/tensorflow/python/framework/importer.py", line 279, in import_graph_def
op_def=op_def)
File "/home/misunpar/tensorflow/local/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 2317, in create_op
original_op=self._default_original_op, op_def=op_def)
File "/home/misunpar/tensorflow/local/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 1239, in init
self._traceback = _extract_stack()
Can you please help me on this issue?
Thank you so much.


✨ Migration: Ams Daylight Saving Conflict Resolver (DSCR)

Overview
This component is used to resolve the daylight saving time conflict (only occurring in some specific areas, e.g. US). It should give users two options when the following two requirements are both met:

User is in an area that has the daylight saving time.
The selected date and time is in the potential conflict range (e.g. for US: 1:00 AM on November 7, 2021).

Tasks

[x] Functionality rebuild.
[x] Styling.

Design
We have two different designs, we will use the first one for the first major release:

Current stage of DSCR development:
https://user-images.githubusercontent.com/36816148/171733399-36425022-fce8-45b8-a046-71de33a51133.mp4

Finish development!


about head hitbox

Has this issue already been reported? Did you ask for help before making this report?

[X] I have searched the existing issues
[X] I have asked for help

Current Behavior
When I hit the heads of the entities, sometimes the head hits, sometimes the body counts and sometimes it doesn't count at all. It looks like it needs reconsideration.
Expected Behavior
When I hit the heads of entities, it should count heads directly.
Steps To Reproduce

I point it at the heads of entities.
I shoot last.

Frequency
%100
Version and Log

Anything else?
By the way, the monster I tested is HUSK. Even though I made the HUSK monster's hitbox equal, if I hit the throat side, the head counts.
(Video)
HUSK:
Horizontal_Entity: false
HEAD: 0.25
BODY: 0.25
ARMS: true
LEGS: 0.25
FEET: 0.25

Some entities hitboxes just are like that

https://github.com/WeaponMechanics/MechanicsMain/commit/40d3ec52b1081a2a63c142fc9cf0d0dbcc2a74ba


Expose more of RESTler configuration settings as part of Run Configuration

/// <param name="name">Name of the webhook "tag"</param>

this should be webhookName now

Refers to: src/APIService/ApiService/Controllers/Webhook.fs:217 in f19ed29. [](commit_id = f19ed292daaa51d5e5e7ec597c0876241cba711b, deletion_comment = False)


Missing get categories recursively

I truely love this extension and made my work a lot easier.
However the only thing I am missing is retrieving the full list of categories recursively.. it already has a parent id so it can just as easy be recursive..

You may already call GET /api/rest/categories/parent/:parent recursively to build a tree. Were you expecting a ready-made tree instead?

Very helpful but facing a problem .This is working in localhost but not working in staging server. Firstly i had tested this extension on localhost, it has been working smoothly but its not working on staging server . Even i had followed the same instructions or path as i did on localhost .Please help.


[CHANGE ME] Re-generated to pick up changes from self.

This PR was generated using Autosynth. :rainbow:
Synth log will be available here:
https://source.cloud.google.com/results/invocations/3f35c52e-e8a4-495e-8072-959688955792/targets

[ ] To automatically regenerate this PR, check this box.

@simonz130 The CODEOWNERS file is managed by templates. The intention of that line is to ensure the java-samples readability folks approve sample changes. If we want to include api-logging as an owner of those files, we can add a template exclude in synth.py


refactor separate logic of AccountSettingsDashboard into separate hook

What does this PR do and why?
Provide a description of what this PR does. Link to any relevant GitHub issues, Notion tasks or Discord discussions.
Screenshots or screen recordings
If applicable, provide screenshots or screen recordings to demonstrate the changes.
Acceptance checklist

[ ] I have evaluated the Approval Guidelines for this PR.
[ ] (if relevant) I have tested this PR in all supported browsers.
[ ] (if relevant) I have tested this PR in dark mode and light mode (if applicable).

Current dependencies on/for this PR:

main

PR #3335

PR #3336   👈

This comment was auto-generated by Graphite.

@wraeth-eth queued this pull request to merge with Graphite.


Dataset Viewer issue for LawalAfeez/englishreview-ds-mini

Link
No response
Description
Unable to view the split data
Owner
No response

It's currently working, as far as I understand
https://huggingface.co/datasets/LawalAfeez/englishreview-ds-mini/viewer/LawalAfeez--englishreview-ds-mini/train

What was your issue?


Cleanup MakeResultOr

the function MakeResultOr has been used to extract a result from multiple checks. This worked when results were binary (Fail/Pass) but not anymore. I think sub-checks should return (score, err) or (bool, err); and the caller should decide what to do.
It's hard to have a one-size-fit-all function: errors may be handled differently depending on context: if an err is returned, shall we return immediately or continue and try the next check?
Example of code this will simplify:  https://github.com/ossf/scorecard/blob/main/checks/sast.go#L38-L71

Done with the 2 PRs mentioned in comments. Closing.


Support for configuring multiple NICs for vSphere

What this PR does / why we need it:
This PR adds support for configuring multiple networks for VMs on vSphere. The field vmNetName is considered deprecated and networks should be used instead. In case both network and vmNetName are specified, the validating webhooks will reject the machine deployment.
Which issue(s) this PR fixes:

Fixes #1671
What type of PR is this?

/kind feature
Special notes for your reviewer:
Does this PR introduce a user-facing change? Then add your Release Note here:

* Support for configuring multiple networks for vSphere.
* [Action Required] The field `vmNetName` is considered deprecated for vSphere and `networks` should be used instead.

Documentation:

NONE

/lgtm

/cherrypick release/v1.57


Added array_merge_deep()

This adds what should have been the native array_merge_recursive() behavior.

Coverage increased (+0.03%) to 97.46% when pulling 2e5b5b4a89169da33a07737af5fb81be75de6545 on yoosefi:master into a718b375cc54bdc13412de641e3cc5ce216c27f5 on brandonwamboldt:master.

Coverage increased (+0.03%) to 97.46% when pulling bea796eefba236ef68e9fc92b88e196d9dcbb095 on yoosefi:master into a718b375cc54bdc13412de641e3cc5ce216c27f5 on brandonwamboldt:master.


[cairo / fontconfig] build failure

Host Environment

OS: macOS 10.15.5
Compiler: Apple Clang

To Reproduce
Steps to reproduce the behavior:
./vcpkg install cairo (works)

actually try to use it in a script

Failure logs
Undefined symbols for architecture x86_64:
  "_XML_ErrorString", referenced from:
      _FcConfigParseAndLoadFromMemoryInternal in libfontconfig.a(fcxml.o)
  "_XML_GetBuffer", referenced from:
      _FcConfigParseAndLoadFromMemoryInternal in libfontconfig.a(fcxml.o)
  "_XML_GetCurrentLineNumber", referenced from:
      _FcConfigMessage in libfontconfig.a(fcxml.o)
  "_XML_GetErrorCode", referenced from:
      _FcConfigParseAndLoadFromMemoryInternal in libfontconfig.a(fcxml.o)
  "_XML_ParseBuffer", referenced from:
      _FcConfigParseAndLoadFromMemoryInternal in libfontconfig.a(fcxml.o)
  "_XML_ParserCreate", referenced from:
      _FcConfigParseAndLoadFromMemoryInternal in libfontconfig.a(fcxml.o)
  "_XML_ParserFree", referenced from:
      _FcConfigParseAndLoadFromMemoryInternal in libfontconfig.a(fcxml.o)
  "_XML_SetCharacterDataHandler", referenced from:
      _FcConfigParseAndLoadFromMemoryInternal in libfontconfig.a(fcxml.o)
  "_XML_SetDoctypeDeclHandler", referenced from:
      _FcConfigParseAndLoadFromMemoryInternal in libfontconfig.a(fcxml.o)
  "_XML_SetElementHandler", referenced from:
      _FcConfigParseAndLoadFromMemoryInternal in libfontconfig.a(fcxml.o)
  "_XML_SetUserData", referenced from:
      _FcConfigParseAndLoadFromMemoryInternal in libfontconfig.a(fcxml.o)
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Additional context
Reason: expat is not linked automatically, cairo doesn't link it to the target:
target_link_libraries(mytarget unofficial::cairo::cairo)

Adding expat fixes it, but I am not sure if this should be fixed in fontconfig or in cairo.

This issue becomes even more clear when trying to build for arm64-osx:
==> vcpkg/buildtrees/fontconfig/config-arm64-osx-dbg-err.log <==
rm: conftest.dSYM: is a directory
rm: conftest.dSYM: is a directory
rm: conftest.dSYM: is a directory
Configured with: --prefix=/Applications/Xcode_12.2.app/Contents/Developer/usr --with-gxx-include-dir=/Library/Developer/CommandLineTools/SDKs/MacOSX10.15.sdk/usr/include/c++/4.2.1
configure: error:
*** expat is required. or try to use --enable-libxml2

From your provided failure log, I think it's a port bug.

In fontconfig's pkgconfig:
  1 prefix=${pcfiledir}/../..
  2
  3 exec_prefix=${prefix}
  4 libdir=${exec_prefix}/lib
  5 includedir=${prefix}/include
  6 sysconfdir=${prefix}/etc
  7 localstatedir=${prefix}/var
  8 PACKAGE=fontconfig
  9 confdir=${sysconfdir}/fonts
 10 cachedir=${localstatedir}/cache/${PACKAGE}
 11
 12 Name: Fontconfig
 13 Description: Font configuration and customization library
 14 Version: 2.13.1
 15 Requires:  freetype2 >= 21.0.15  uuid expat
 16 Libs: -L"${libdir}" -lfontconfig
 17 Cflags: -I"${includedir}"

expat does exist in the pkgconfig, can you rebuild fontconfig and try again?

I get the same problem on Apple.
In my code, use ${Fontconfig_LIBRARIES} cause that
find_package(Fontconfig REQUIRED)
target_link_libraries(<my_target>LINK_PRIVATE ${Fontconfig_LIBRARIES})

solve by use
find_package(Fontconfig REQUIRED)
target_link_libraries(<my_target>LINK_PRIVATE Fontconfig::Fontconfig)

@JackBoosY I think this has been fixed since then.

okay, please ping our team member to reopen this PR if it still exist.


No license in repo

There’s currently no license in this repo, which means that there’s no permission to use it in other projects.
Would you like me to submit a pull request that adds the MIT license? If not, is there a different license you’d prefer?
Thanks for the awesome shader! It looks excellent.

Fixed.
Sorry for the wait :)

Thank you! :)


Alow jpa ticket registry to use an extended entity manager

to disallow from closing prematurely when vending STs...

It seems to have broken some test.

Will correct the test case and merge.

@leleuj build is now functional, but it looks like we are having issues with this:
Download https://repo1.maven.org/maven2/com/nimbusds/nimbus-jose-jwt/4.10/nimbus-jose-jwt-4.10.pom
Download https://oss.sonatype.org/content/repositories/snapshots/org/pac4j/pac4j-oauth/1.8.1-SNAPSHOT/pac4j-oauth-1.8.1-20151208.110711-46.jar
Download https://oss.sonatype.org/content/repositories/snapshots/org/pac4j/pac4j-saml/1.8.1-SNAPSHOT/pac4j-saml-1.8.1-20151208.110653-46.jar
Download https://oss.sonatype.org/content/repositories/snapshots/org/pac4j/pac4j-config/1.8.1-SNAPSHOT/pac4j-config-1.8.1-20151208.110722-16.jar
Download https://oss.sonatype.org/content/repositories/snapshots/org/pac4j/pac4j-cas/1.8.1-SNAPSHOT/pac4j-cas-1.8.1-20151208.110642-46.jar
Download https://oss.sonatype.org/content/repositories/snapshots/org/pac4j/pac4j-oidc/1.8.1-SNAPSHOT/pac4j-oidc-1.8.1-20151208.110717-46.jar
Download https://repo1.maven.org/maven2/com/nimbusds/nimbus-jose-jwt/4.10/nimbus-jose-jwt-4.10.jar
[ant:iajc] /home/travis/build/Jasig/cas/cas-server-support-pac4j/src/main/java/org/jasig/cas/support/pac4j/Pac4jConfiguration.java:3 [error] The import org.pac4j.config.client.ConfigPropertiesFactory cannot be resolved
[ant:iajc] import org.pac4j.config.client.ConfigPropertiesFactory;
[ant:iajc]        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[ant:iajc] /home/travis/build/Jasig/cas/cas-server-support-pac4j/src/main/java/org/jasig/cas/support/pac4j/Pac4jConfiguration.java:63 [error] ConfigPropertiesFactory cannot be resolved to a type
[ant:iajc] final ConfigFactory configFactory = new ConfigPropertiesFactory(properties);
[ant:iajc]                                         ^^^^^^^^^^^^^^
[ant:iajc]
[ant:iajc] 2 error

Should be fixed when 1.8.1 comes out. So I am ignoring it for now.


Question/Issue: HEX Color for p9k / pywal support

I am trying to integrate p9k into my pywal theme, however, pywal generates either escape codes or hex codes for color settings.
I tried using POWERLEVEL9K_OS_ICON_FOREGROUND="$($color1)" after generating a wal color scheme, but all I get when sourcing my .zshrc is getColor:12: command not found: #692F8E
Environment Information

zsh 5.5.1 (x86_64-apple-darwin17.5.0)
antigen
p9k is installed from master branch (antigen theme bhilburn/powerlevel9k powerlevel9k)
iterm2

Powerlevel9k makes use of Z Shell's %F{x} and %f for foreground and %K{x} and %k for background colours. We don't use the escape codes that are required for what you are attempting to do.

@bhilburn Can we close this issue as there is no further activity?

Necro-bumping this issue because I have the same wish. I can't, for the life of me, find a theme as cool as this one with pywal support. Because the color options are limited to color codes, colors are always slightly off from my usual colorscheme. Is it possible to add hex color support to p9k? Or would that require a major rewrite? @bhilburn

This would require a major rewrite since the generators use the %F, %f, %K, and %k codes (for 256 colors and hex color support would require using ASCII escape codes.


Prevent showing the UI5 app internal page without successful authentication

OpenUI5 version: 1.86
Browser/version (+device/version): Chrome Dev
Upon the authentication I validate the user session:
if (isUserSessionValid) {
	const oRouter = UIComponent.getRouterFor(this);
	oRouter.navTo("overview");
} else {
	this.getOwnerComponent().openAuthDialog();
}

If isUserSessionValid is true, then I forward an user to the internal page, otherwise I show the login dialog.
The problem is, however, that an user can change the value of isUserSessionValid in DevTools and then getting forwarded to the UI5 app internal page. Of course, due to a lack of a valid session, no piece of the business data will be displayed, just an empty UI5 app template, but I would like to prevent even such screen.
If it would be a classical webapp, I would just send an appropriate server response with a redirect to the login page (e.g. res.redirect(403, "/login");). But, if I understand it correctly, since I'm sending am asynchronous request, a plain res.redirect won't work out and I'm required to implement a redirection logic on the UI5-client, which can be manipulated and bypassed by user.
How to prevent a manipulation of a view navigation in UI5 and ensure that unauthorized user can't get any piece of the UI5-app code?

I think you've already answered the question yourself: You can't fully prevent a user from manipulating code that runs on the client-side.
If you want to prevent an unauthorized user from accessing the client-side code (e.g. view/controller) you need to enforce authorization on the server also for those static files.
When bundling the application code you also need to ensure that those files are separate from the "public" files.
One approach would be to have 2 separate components, one for the public page / auth dialog and one for the actual application.
Hope this helps 👍🏻


[MM-20744] Remove the full screen modal css from Jira and reuse the css in the webapp

Summary

Reduce the custom fullscreen modal scss to the bare minimum we need for Jira.
Changing go.mod for go1.13 (it wouldn't compile for me without it)
@jfrerich I removed the z-index change, but I'm not sure what this will break. Can you let me know what I need to test? If we need to, we can add our own class and create a special jira-plugin-fullscreen-modal, that would fix the problem that this caused for Asaad (see the Jira ticket).
@asaadmahmood what do you think, just make our own fullscreen modal class for this css we need?

Links

Fixes MM-20744

@asaadmahmood Some full screen modal css is not the same as the webapp's. Specifically the positioning of the "back" and "close-x" buttons. These two buttons have been moved closer to top two corners of the modal, to take into account the placement of the form in the modal.
How do you suggest we consolidate these changes while reusing the existing css? One way would be to override only the necessary attributes for the css classes used by the buttons, or use different classes for the two buttons.

@mickmister Another way would be to include that css in the webapp too, that can be used with another modifier class for those cases.

@asaadmahmood In general, we shouldn't include Jira-plugin-specific css in the webapp, right? So what do you think we could do here? Could we add a jira-modal class?

@cpoile We shouldn't use jira specific css anywhere, not even in the plugin, if the full screen modal has a different state with the back button/close button that is a bit compact, then that should just be another variation for the modal that can be used for other plugins or other states for the full screen modal in our app.

@mickmister @jfrerich Since you were the ones working on this, can we remove these changes and just use the standard full screen modal css?

I agree with @asaadmahmood with making the css in the webapp "configurable", in that we can specify different available values based on combinations of classes. For example, we use something like margin-bottom x3 when we want to apply a certain amount of margin.
I think we can use something like close-x modal-corner (not a good name example, however) to specify that we want the styles that place the buttons closer to the upper corners of the modal.

can we remove these changes and just use the standard full screen modal css?

@cpoile Other than the changes to the "back" and "close" buttons to be closer to the upper corners of the modal, none of this css is different from what was taken from the docup plugin, which I believe was copied directly from the webapp. So most of the css can be used straight from the webapp, but we still need to allow the plugin to express the intent to place the buttons close to the upper corners of the modal.

@jfrerich Just a reminder about the z-index change -- I removed it because that's what was causing Asaad's bug. I want to make sure this doesn't break anything.

@asaadmahmood Looking over the changes, I'm not sure if should put this in the webapp. The change seems too few and too specific for this use case. Would you want to add in parameters to the css on the webapp like @mickmister suggested?
Otherwise, we could make a jira-modal class, and use that for these little changes? Your call.

@cpoile, I've been looking for the code change regarding the zIndex, but so far have only found this https://github.com/mattermost/mattermost-plugin-jira/commit/d99e966f65c3b4a2c3b3e450020b55594bbf1abb#diff-9b097f02b4ddd564b16955991958f147R172, which is a change I had made in the confirm_modal.

I would test that the Create Jira Issue and Attach to Jira Issue post commands both show up with this change just to verify nothing breaks. The other area would be the /jira subscribe list command.  The test for the confirm_modal would be to create a subscription and then delete it.  That will open the confirm_modal.

Should this be going into 2.3? Or is it ok to defer until later?

I don't know, I've been out of Jira for so long. @mickmister ?

If the issue found on community is fixed without this PR, then it is not necessary for 2.3 imo. I'm not sure what issues someone may run into other than the exact scenario we experienced on community.
@asaadmahmood I think we need your judgment on this

@mickmister I think Asaad is out for awhile, and I'm really not sure what to do with this. I don't totally agree with moving the changes into webapp, I think we should just merge this for now and then subclass or specialize the webapp css later. It's tech debt.

@cpanato @mickmister Yes, that issue is fixed, we don't need to take this for 2.3. I think we should give this some time before pushing it.

@asaadmahmood why you mention me? any actions do i need to take?

@cpanato No, I think he meant my name. Sorry for the confusion.

Apologies @cpanato

The only gotcha I can think of with merging this is the placement of the close-x and back buttons. Pulling down from master, the positioning is incorrect again. I seem to have broken it in a completely unrelated PR https://github.com/mattermost/mattermost-plugin-jira/commit/ad3cc4c2581e73c91a9fa520a4ec7a8299a4e0b3#diff-0f8116c7c9b5c1bb6abab32cd1b3cd5f . I will create a new PR for the button positioning fix.

@asaadmahmood or @mickmister  I'm a bit confused about how to validate that the issue is resolved without this PR. Is it simply to validate the modal is working correctly where https://github.com/mattermost/mattermost-webapp/pull/4351 has been applied?

@cpoile wonder if you might also be able to answer the question above ☝ about validating without this PR.
I never actually saw the issue occur on community but I could try testing on a different server.

@DHaussermann I'm not sure what you're referring to -- the issue that Asaad was originally having was fixed by https://github.com/mattermost/mattermost-webapp/pull/4351, are you talking about verifying that /that PR/ fixes the problem?

@cpoile yes I want to make sure that  mattermost/mattermost-webapp#4351 fixes the issue without the need for this PR but, I'm not sure what this bug looks like when it's occurring.

@asaadmahmood Can you help @DHaussermann reproduce the original bug you were seeing? Thanks!

@DHaussermann You can view the tooltip on the plugins marketplace modal on https://ci-extensions.azure.k8s.mattermost.com/

@mickmister @jfrerich ping - is this waiting on you?

@mickmister With the changes you made to the css in the other PR, where does that leave us with this? Either we add those to the webapp css (not a big fan of that), or we subclass our own modals (jira-fullscreen-modal or something)?

@cpoile, with the z-index removal, were you able test the Create Jira Issue and Attach to Jira Issue post commands and the confirm modal from trying to delete a subscription the /jira subscribe list command?

Per @asaadmahmood's comment above, I think we are not going to change anything more pertaining to this css for 2.3. We haven't reached an agreement on how to resolve the css duplication.

we subclass our own modals (jira-fullscreen-modal or something)

@cpoile This sounds good, but I don't think I understand it fully. Can you explain what you mean by subclass? We cannot import any webapp components into the plugin fyi, only reuse css. I'm not sure if that's relevant.

@mickmister I guess I mean use our own class, instead of shadowing/redeclaring/?? .FullScreenModal. (got to brush up on my scss terms)

Tested what @jfrerich mentioned, everything works after removing the z-index.
@mickmister Your choice on what we should do, design-wise.

Closing in favor of https://github.com/mattermost/mattermost-plugin-jira/pull/413


switch to github actions

fix #4

switch from azure pipelines to github actions
incorporate updates from aiida-core

Note: Supposedly psql is already present on the ubuntu and macos images but supposedly this is only the client, does not include pg_ctl (not found by PGTest.which) and there is no postgres server running.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@535e2ee). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #5   +/-   ##
=========================================
  Coverage          ?   46.54%
=========================================
  Files             ?        1
  Lines             ?      159
  Branches          ?        0
=========================================
  Hits              ?       74
  Misses            ?       85
  Partials          ?        0

Continue to review full report at Codecov.

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


General Grievance: Watch syscorp.in

General Grievance requests the watch of the watch_keyword syscorp\.in. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by double-beep in Charcoal HQ

Approved by Jeff Schaller in Charcoal HQ


⚡ feat(team generator): fully functional team generator

TEAM GENERATOR IS DONE AND WORKS!

Fixed merge conflict in dashboard.js


MB-10673 Add callout  to est weight display if any shipments are non GHC

Jira ticket for this change
Summary
On MTO page, added a note with a link to the move details page if any shipments are not moved by the GHC prime.
Setup to Run Your Code

💻 You will need to use three separate terminals to test this locally.
Terminal 1
Start the Storybook locally.
make storybook

Terminal 2
Start the UI locally.
make client_run

Terminal 3
Start the Go server locally.
make server_run

Additional steps

Login as a TOO
For local, use NTSSUB move and change the NTS-release shipment to be an external vendor
For staging, create a move that has an HHG and an NTS-release
Approve the move and view the Move Task Order
You should see a note in the Estimated weight (Total) block indicating there are shipments not handled by the prime and a link back to the move details page.

Verification Steps for Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.

Frontend

[ ] User facing changes have been reviewed by design.
[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] There are no new console errors in the browser devtools
[ ] There are no new console errors in the test output
[ ] If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.

Backend

[ ] Code follows the guidelines for Logging
[ ] The requirements listed in Querying the Database Safely have been satisfied.

Database
Any new migrations/schema changes:

[ ] Follows our guidelines for Zero-Downtime Deploys
[ ] Have been communicated to #g-database
[ ] Secure migrations have been tested following the instructions in our docs

Screenshots

Messages

:book:
:link: MB-10673

  Generated by :no_entry_sign: dangerJS against d70b6260d15ab119240f82f48d98abbd42b2bed1

I'm still seeing the 0 show in the ephemeral deploy. 😞

The fix was so simple I couldn't see it. 😆


Crash when adjusting drawers connected to Refined Storage

crash-2022-07-07_13.57.24-server.txt
Drawer Controller is connected to a Refined Storage network with an External Storage attachment. Many items are missing from the drawer network and some of them show double the amount they should. While trying to check my Functional Storage network and using the Linking Tool my game will crash and requires me to restore from a backup to get back into the game.

Do you have multiple FS controllers connected to each other?


Run LLVM tests in Dredd CI

Currently the Dredd CI builds an LLVM library after mutation (I think it's InstCombine). It would be good if the CI could also use llvm-lit to run the InstCombine unit tests.

This no longer appears to be feasible to run in the pipeline. This has been replaced by issue #83.


Require gRPC TLS transport security, server verification, client authentication

Add gRPC client-to-server TLS transport security and server verification (prevent MITM)

A CA certificate (e.g. ca.crt) should be used to sign a server certificate (server.crt for a private key server.key)
gRPC API server requires a server certificate and key to be enabled (passed via -cert-file and -key-file)
gRPC client bootcmd tool must verify the server identity using a known CA (passed to bootcmd via -ca-file)

Example client and server commands/flags with rkt and Docker
Require TLS client auth for gRPC API

gRPC API server requires a CA certificate to authenticate clients (passed via -ca-file)
gRPC clients must authenticate with a client certificate and key (passed via -cert-file and -key-file)

Add examples/etc/bootcfg/cert-gen to create fake (ca, server, client) credentials for running the gRPC API examples locally.

rel #140 cc @gtank @brianredbeard

Updated per feedback and rebased.

Looks good!


Get value of the rule parameter

It would be useful if we can get the value of the parameter of a given rule.
Let say we have something like that :
{
  "_default": "Field {field} is not valid.",
  "min": "Field {field} must contain at least {min} character(s)."
}

get rules () {
  return {
    username: 'required|min:3|max:15|unique:users,username',
    email: 'required|email|max:30|unique:users,email',
    password: 'required|min:6|max:60'
  }
}

async fails (errorMessages) {
    let error = errorMessages[0] // Remove that if validateAll returns true.
    let messageData = {
      field: error.field
    }

    messageData[error.validation] = error.rule // <-- Notice here the new property. It should print of course 3 because "min:3" if there's a min error.

    return this.ctx.response.send({
      field: error.field,
      message: Antl.formatMessage(`validation.${error.validation ? error.validation : '_default'}`, messageData)
    })
  }

What do you think about that?

It's supported, check here https://indicative.adonisjs.com/docs/templating#_available_keywords

Thank you!
But how I can add the 'argument' array here :
return this.ctx.response.badRequest({
  field: error.field,
  message: Antl.formatMessage(`validation.${rule ? rule : '_default'}`, {
    field: error.field
  })
})

please ?

You don't have to pass it. It will be injected automatically by indicative. Your message will look as follows.
{
  "min": "Field {{ field }} must contain at least {{ argument.0 }} character(s)."
}

The double curly brace {{ will not be parsed by antl and hence forwarded to indicative.

Thanks!
But it doesn't seem to work or I did something wrong.
When I try using formatMessage or get :
antl.formatMessage(`validation.${rule ? rule : '_default'}`, {
  field: error.field
}) // Returns empty string

antl.formatMessage(`validation.${rule ? rule : '_default'}`) // Returns empty string

antl.get(`validation.${rule ? rule : '_default'}`) // Returns string not parsed, i.e : Field {{ field }} must contain at least {{ argument.0 }} character(s).

Maybe I don't get the way to achieve it?
(sorry to be hassle)

So the problem is, you are not using the inbuilt messages property of the validator and making messages yourself. In that case, it is expected that no transformation of values will be done.
What you need is simple
get messages () {
  min: this.ctx.antl.formatMessage('validation.min')
}

Thanks, it works now!
(it was not mentioned in the docs :p)
Also, is it possible to do something like that :
get fields () {
  return {
    username: 'pseudo'
  }
}

Nope, that's not possible and also not something I am planning to add

Thanks, it works now!
(it was not mentioned in the docs :p)
Also, is it possible to do something like that :
get fields () {
  return {
    username: 'pseudo'
  }
}

so that indicative will replace {{ field }} by the new field value.

I try here and not working. 😢


Remove holoviews.interface subpackage

Everything in this subpackage is either deprecated or about to be deprecated with more solid support elsewhere in HoloViews.

We could move Collector somewhere now. Did we agree we'd move it to topographica?

Did we agree we'd move it to topographica?

I think so.

Done.


RenderFlex children have non-zero flex but incoming height constraints are unbounded.

I am facing this issue during run the application can any one help me.

Hi @Muhammad-84,
This issue doesn't seem to describe a bug or a feature request. Please see https://flutter.dev/community#community-grid for resources and asking questions like this. You may also get some help if you post it on Stack Overflow with a minimal code sample that clearly describes your problem.
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and we will reopen it.
Thank you


Refactor Services

I can assign you here, @NimperX. 😃

Cool @CrazyMarvin. I'll go with this.

Maybe add an better description and some tags to this issue

@NimperX did some refactoring in #88. Do you think the issue can be closed then?

I've comment my review on #88. This PR only refactors the SayingService and not the YogiBotService

I cannot find your comment, @markuscodes1. 😅
Where do I have to look?

Sorry forgot to submit the review. Take a look again into #88

@CrazyMarvin I think this issue can be closed, cause all the changes metioned inthe issue had been resolved in last commit of #89 (commit b93db4df4d33a62ed64f0c5a5f15f2faf792cfd3 )


Indentation is broken

When I add an if statement, and press <Enter>, the end keyword is added. So far so good. However, if the if statement is indented already (or nested), i.e.: The i of the if and the e of end is not in column 0, and if I indent the selection with = (in normal mode using ==, or in visual mode), then the indentation completely breaks.
The breakage gets worse every time a block of code is indented, as shown in the screenshots.

Vim-fish should not break =.
I don't have endwise installed, but my vim is still unable to indent code properly when using vim-fish.
Endwise is nice but a luxury, indenting code properly is a must.
How can I tell vim-fish to be compatible with =?

Yeah, I'm also seeing broken behaviour when indenting using =. Have anyone come to any conclusions regarding this?

Any updates?

@nhooyr None of them works well on indentation.


Show alarm limits on graph

Description:
Show alert limits in graph for the chosen graph history.
Toggle to show or hide limits from view.
Design to be confirmed with Sini-Maaria

Here is an idea we could try. I will also design a toggle button for the alert view.

@io53 can start implementing, ask Sini-Maaria about the toggle design.

Color codes:
BG: 50%, #ff954d
Line:  #f15a24

Alert limit not show  correctly for pressure

Looks like alert area is not filled correctly

@io53 the issue still persists on Mac Chrome, other browsers handle this correctly. This is visible on dashboard view as well.

Try 80% opacity on graphs.
Use the same alert color on numbers.

@io53 here are the colours for you to check

@io53
Comment from Sini-Maaria:
"This could be tested with the same color as the orange on the graphs."

QA passed for me