aws-sdk signature issues

I found the following inconsistencies in the aws SDK signatures, which are done via a generator:
message/body/message_body parameters in S3/SQS clients are incorrectly typed to "only IO"
Those parameters can also be a string (it seems that Aws::S3::Types::body and the same variant in SQS should have been handling that instead, when defined as. ::IO | String).
SQS client "send_message" incorrectly typed as well
send_message receives two hashes as parameters, "params" and "options". The RBS signature uses kwargs and mixes parameters from each. Ideally it should be using RBS records for each (although it's probably not possible because of this).
Aws::SQS::Errors::InvalidParameterValue error is missing
Potentially more are?

@HoneyryderChuck Thank you for raising the issue.

message/body/message_body parameters in S3/SQS clients are incorrectly typed to "only IO"

I fixed this issue. Could you confirm this?
https://github.com/ruby/gem_rbs_collection/pull/354

SQS client "send_message" incorrectly typed as well

You are right.
Aws::SQS::Client#send_message and all client methods parameters has "params" and "options" argument.
https://github.com/aws/aws-sdk-ruby/blob/bbe581191d8d0dc9be0f63be5b3656d655c5b8e0/gems/aws-sdk-sqs/lib/aws-sdk-sqs/client.rb#L2233C3-L2233C3
The current implementation is intentionally conscious of being able to check required keyword arguments.
There is a possible way to set the Record type as you mentioned, but it has the disadvantage of not being able to check for required keywords.
I believe this is happening because aws-sdk-ruby implements such dynamic required keyword arguments.
I have not found the best solution to this problem. I am looking for ideas.

Aws::SQS::Errors::InvalidParameterValue error is missing

Aws::SQS::Errors::InvalidParameterValue is a dynamically generated class. Probably the name is obtained by HTTP response and the class is generated under Errors.
https://github.com/aws/aws-sdk-ruby/blob/bbe581191d8d0dc9be0f63be5b3656d655c5b8e0/gems/aws-sdk-core/lib/aws-sdk-core/errors.rb#L393
I have not been able to find a list of names that could occur as this error class and have been unable to define it.
Ideally, the error class should be generated automatically from the list of error definitions maintained by Aws.
There is some documentation currently found, but we do not know if it is the best method.
https://docs.aws.amazon.com/AWSSimpleQueueService/latest/APIReference/CommonErrors.html


readonly don't work

readonly() method seems to do nothing.

Hi! Thanks for reporting this. Fixed in version 2.0.6. :)

Hi! Thanks for reporting this. Fixed in version 2.0.6. :)


Return ptr of new Object out of Slang DLL

export __extern_cpp Ptr<PointLight<float>> __create() {
    Ptr<PointLight<float>> light = new PointLight<float>();
    return &light;
}

When I do this I get explicit conversion from 'Ptr<Ptr<PointLight<float>>>' to 'Ptr<PointLight<float>>' is possible
export __extern_cpp Ptr<PointLight<float>> __create() {
    Ptr<PointLight<float>> light = new PointLight<float>();
    return light;
}

When I do it the correct way I get expected an expression of type 'Ptr<PointLight<float>>', got 'PointLight<float>'
For some reason the Ptr type gets lost when doing the correct thing
So the question is, is this an error or am I doing something wrong. How can I allocate a heap object in a Slang DLL and return it to the caller?

The type of new PointLight<float> is PointLight<float>, and is valid only when PointLight is a class.
A value of class type is always by-reference, so there is no need to wrap it with Ptr<PointLight<float>>.

Note: all of these C++/COM features are highly experimental and incomplete. At this time we don't have the capacity to provide support around these functionalities. The only intended way to return a heap allocated object from a Slang DLL is to expose a COM interface, define a class that implements the interface, and define a function that returns the COM interface, where inside the function you return an instance of the class. See tests\cpu-program\class-com.slang for more details.

What I am doing right now is that I create a byte array outside slang and pass it in via ptr and then fill it with the data but the com interface way sounds way better. Do you know why the macOS bins are missing the slang-rt lib?
Is it possible to have a templated COM interface. When I do that I get
No type conformances are found for interface 'IDifferentiable'. Code generation for current target requires at least one implementation type present in the linkage. Does this work when I provide type conformances?

There is another problem
I can write return new PointLight();  and it works but when I save it to a var and return that it does no longer work, like
export __extern_cpp Light __create() {
    PointLight light = new PointLight();
    return light;
}

I got everything working with the __requirePrelude and __intrinsic_asm feature


PROPOSAL: refactor mysql storage code to injectable interfaces

Proposal
In general Druid architecture is really good, I learnt a lot reading druid code, but there are 2 places where I feel that could be improved:

DbConnectorModule should be extracted as something like CoordinationStorage interface, which wraps all the current DB operations as methods(currently there are sql queries in many places), like listSegments, dropSegment, etc.

io.druid.initialization.Initialization should only load interfaces or register configs. Stuff like DbConnectorModule should be injectable

With these, one can easily replace mysql module to S3 or any online storage that has SLA guaranteed.
Background
Our team started exploring Druid in an internal system, after reading the design documents of Druid, I found mysql is the only thing that worries me, there are plenty of storage options that is more stable and instantly available both internally and externally.
I wrote my main function and copy over all the functions in io.druid.cli.Main by only replacing CliCoordinator and CliRealtime, where we register DatabaseSegmentManager and DatabaseRuleManager to our own class using internal storage. In the runtime.properties for database configs I put a random config that points to localhost(which does not exist).
This works fine for a few days because we replaced DatabaseSegmentManager and DatabaseRuleManager, no real DB connection was made.
It fails today when I tried to upgrade to 0.6.73(was on <IP_ADDRESS>), where druid tries to connect to mysql database to detect whether it is postgresql or not(https://github.com/metamx/druid/commit/d089f05127899c76ece7599adc4dfaacc4842d04). To fix it I either have to fake a sql server or make custom druid build. I feel there is a better way to solve this so writing this proposal.
--
Frank

I am closing this as the metadata store is now pluggable.


Participation in the translation

Hi, @webknjaz!
My email<EMAIL_ADDRESS>

Action items:

[x] invite to @open-source-guild/translators
[ ] invite to slack

[ ] remove email from public access (to protect from spam-bots)

[ ] assign article/issue for translation

@numrut tell about yourself in #general channel once you join LambdaBooks Slack.

Hey @webknjaz, it seems I still haven't received an invitation.

@denysdovhan help

@numrut anyway you may pick any unassigned issue, create a branch (name it translation/##-article-name) and start working on it.


tag popup

The behaviour should be more or less identical to standard mobile features that allow tags to be linked to data.  The main difference is that we require a definition to be provided, so the user must match a pattern (see below).  Wireframes are at the end of our InVision project https://projects.invisionapp.com/share/2G5K1Z39M#/screens/209978896.
Controller code goes in the tags and keywords controller, almost certainly some tweaks to both tag and keyword model are required, and perhaps the tags concern (to handle global_id as an attribute, if it's not already there).
if I use tags_helper#new_tag_tag(object)
   then a link/icon appears in the view
      that link is
         wrapped in a div
           that has a unique UUID for the dive (js timestamp perhaps), *think* this is necessary for downstream interactions, might be wrong
           that references the object by object#global_id
         optionally styled with a class that indicates that this object already has tags

when I click that link
   then I get a popup text box,
     with a prompted instruction inside
     or a list of existing tags on that object
     with a undo icon to the right, that aborts *all* actions

when I type some letters
  then those letters get a background color bubble
  then I search through Keywords#name and #definition for a match, via a keywords_controller method
     when I find matches then I display them below the colored bubble
        when I click on a match
            then I display that match in the bubble
               with an (x) at the end of it for deletion of that keyword
            then I move the cursor to the right of the bubble
            then I inject the hidden fields into object#tags_attributes{} necessary to create a new tag
     when I can not find a match
       then I keep typing
          if I type a phrase like /(keyword\s*)+\s*:\s*some definition for foo\s*,/ (term : definition ,)
             then I create a new bubble, hiding the definition, shown only when I mouse back over (tooltip style behaviour)
             then I append hidden fields to create a new keyword AND tag (tags_attributes: [{keywords_attributes: {name: value, definition: value}}, ... ]
             and I move the cursor for the next keyword

when I find or "create" a new tag
  then I see a "create" button to the right
    when I click this button
      then an call gets made to save the data
         if it succeeds
            then a message is pulsed inside the text area
               and the popup is collapsed
         if it fails
            then a message is pulsed inside the popup
               and the popup remains open

Some of this is now handled with the behavior in tags#new.  The popup/modal version remains to be done.

This issue was subsumed into changes made in overall user interface paradigm by @jlpereira


Metrics Explorer: implement grid layout functionality

This is probably done, but we should verify that it is completed once we have a layout established for the explore view.


Investigate layering Jabbr architecture more

Previously there was no JabbR.Services, JabbR.Library JabbR.Data etc
Do we want to re-architect this now?
Discuss with David Fowler - get his thoughts @davidfowl

Any updates on this that matter right now?

Implemented JabbR.Data in #57
Not sure if we want to continue to create more sub-projects or close this issue.

@adamtuliper what's new with this? I know we have JabbR-Core, JabbR-Core.Data, JabbR-Core.Tests, and we will likely have JabbR-Core.Cordova (or JabbR-Core.Xamarin?) in the future depending on mobile client choice, but what did you have in mind for things like JabbR-Core.Services or JabbR-Core.Library? What would constitute as making this issue able to be closed?

Cordova? nah just do xamarin...


Support querying for L2A products

Currently the function OgcRequest.get_dates queries dates for which L1C products are available. It does so regardless of whether source in Sentinel Hub Configuration Utility is set to L1C or  something else (such as L2A).
Although there is a plan to have an L2A for each L1C product, we currently provide few L2A products. Consequently, when one has source set to L2A  in the Configuration Utility, some queries may result in empty images (see issue #20 ).
We should fix this in the upcoming releases.

This has already been identified and added to list of "improvements", see https://git.sinergise.com/eo/research/wikis/Application-Ideas under "Support all S-H datasources"

Has this been fixed with v.1.1 ?

Yes, since version 1.1.0 the package supports Senintel-2 L2A, Sentinel-1, Landast 8, MODIS and DEM data sources. Therefore I am closing this issue.


Fix facet distribution error

This PR fixes two invalid behaviors and fixes #253:

We were ignoring the list of fields for which the user wanted a facet distribution.
We were not raising any error for when a non-filterable field was requested a facet distribution.

~For the latter behavior I need the help of @curquiza to help me choose the right error type.~

bors merge


move semantics inside of opacity animation

Fixes https://github.com/flutter/flutter/issues/18538
Placing the semantics in the child instead of in the animated builder changes the timing of when it is created to match that of it's children, so talkback is able to find a node to move focus to

test?

Hi @jonahwilliams is this also happening to dropdowns fields? When opening a dropdown the focus goes to 'dismiss' instead of the first element of the dropdown. The user needs to swipe left to go the elements and it is not 'logical order'. Can you give me a hint of how to resolve that? Thanks


Dockerfile fix

Fixes Dockerfile using deprecated args in favour of ENV vars.

LGTM if ARG line is not used. Some information on ENV vars can be found here: https://github.com/SyntropyNet/swapscope/blob/14-update-projects-readme-files/publisher/README.md


Revert Maven publish version

Upgrading the maven publish version to 0.17.0 introduced a breaking change

What went wrong:
An exception occurred applying plugin request [id: 'com.vanniktech.maven.publish']
Failed to apply plugin 'com.vanniktech.maven.publish'.
The env var SONATYPE_NEXUS_USERNAME was removed. Use the Gradle property mavenCentralUsername or the env > >.  var ORG_GRADLE_PROJECT_mavenCentralUsername instead.

I'll create a PR to properly support the latest plugin.

Superseded by #33.


ConflictsWithTypeVariable is not reported for setter

This code gives an error in Analyzer.
class C<T> {
  set T(_) {}
}

But there are no error in CFE.

/cc @askeksa-google


fix(apiconvert): toapi improper omitting

About this change—what it does

fixes improper omitting in ToAPI method of apiconvert

Why this way

it doesn't works properly

acceptance tests: https://github.com/aiven/terraform-provider-aiven/actions/runs/6023419423


Cow: Blacklist 0933-5291-479

Cow requests the blacklist of the number 0933-5291-479. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by double-beep in Charcoal HQ


Support 1.11

Should satisfy #116 =)

So, I was hoping to also add support for the new CBVs in django.contrib.auth, I tried updating the settings.CBV_SOURCES, but wasn't able to get any success with that =/

This is looking great! I'll go take a look and see if I can work out what's going on with the auth views.

So the problem seems to be that some of those views have success_url = reverse_lazy('<something>') as the class default (so we get a NoReverseMatch), whereas I think everything else we've encountered has defaulted to None. I'm not sure yet what I think the best approach to solving this is, but I thought I'd post back now so others can start thinking about it too.

If we're just introspecting, perhaps we can just mock the url pattern it is expecting?

Ha good timing: I was just this minute coming back to update after some tinkering.
That would definitely make the error go away, but I feel like we should strive to document the reverse_lazy call. In that regard, the uses of gettext_lazy will present the same problem (but without the error, as the translate will work).
For both cases, I think we may have to either use the proxy's __reduce__ method, or maybe mock the proxy or some method on it. Although I'm not sure either of those would readily return us something like reverse_lazy, rather it might only get us as far as the wrapped reverse.
In which case maybe we have to inspect the line of code (if we can) and grab from there.

Can't get mocking or anything working myself yet.
The best I can do so far is detecting when it's lazy evaluation and then saying something like "if it's calling reverse then it was lazy_reverse" (plus we have the args that were passed through). This would also work for the ugettext_lazy/gettext_lazy ones. It feels super lame but it might be our best bet unless someone can get the mocking/monkeypatching thing going in a decent way.

OK, I think I've licked this. No need for fancy Mock()'ing at all, just plain-ole duck-typing =)

I've opened #119 based on this because I wanted to try taking it in a slightly different direction. It feels neater to me, but let me know your thoughts.

@mkoistinen Alright if I close this one if we're going with #119 instead?


Include column index information in the SQL Editor

Hi,
In the SQL Editor in Auto complete, include information about the index linked to the column. True when it exists, False otherwise.
Include information about the index on the existing card, where it shows information on the data type, precision, scale, etc.
Example:

This would be a handy feature. But it will require to read table indexes/constraints information (which can take time on systems with bad network throughput).
Maybe we could make it configurable..


Health Check Successful.

-This screen is only in the security question section.
-Did not get this message when we confirmed the pdf share after scanning first QR.
-This screen should appear every time a shard is successfully backed up.

We need to use a toast message here like in other parts of the app. Also, the second line is not needed.

@antuz123 I think this is not required anymore.


Expose mode does not work in an empty space

Shell command to enter in an empty space:
yabai --message window --toggle expose

Native expose mode (exposed on Control + Down by default) works in an empty space.

Not sure why that is; we are calling some macOS API to enter expose. I don't think this is something worth spending a lot of time trying to figure out either, as there is likely nothing we can do to fix it.


Better Container termination status when failing TaskRun for Image pull errors

From #4921 , we introduced the mechanism for failing the TaskRun when any of the step fails with the image pull error: ImagePullBackOff. It also then marks all other steps in the Task as failed with the same reason "TaskRunImagePullFailed". This issue proposes the improvements of the error handling:

Mark other steps terminated as a new pod status reason, instead of marking them all by a Taskrun reason, to clarify that it has not been executed: something like CanceledPreviousStepImagePullError
We could add specific step, image being pulled to the &corev1.ContainerStateTerminated at failTaskRun.
Cleanups on the existing error messages and controller logs.

We shall also explore the extensibility of step/container to examine how we could distinguish TaskRun reason and Step reason more precisely for a better user debug experience. For example, better clarifications of step failures for image pull errors.
/kind bug
/kind feature

/assign


Many Auto Complete Snippets

I have very annoying bug in my Atom - there are many of the same snippets in the Auto Complete function. This is happening only for HTML attributes.

I was trying to disable all others packages, but that dosen't fixed anything.
Do you have some ideas about that?

Not really. What version of atom and language-babel?

Atom 1.21.1 x64
language-babel 2.75.1

I've found where was the problem - in the autocomplete-plus core package. I have to set Similar Suggestion Removal

Actually there was an issue that should be fixed in 2.75.2


Remove leading slash to match RF3986 spec.

I'm not sure about this. Please can you add ltrim($uri, '/') here https://github.com/tgallice/fb-messenger-sdk/blob/master/src/Client.php#L119 ?
To avoid BC break


Add cookie middleware and default providers to webpack plugin generated files

This pr is a second take of https://github.com/Shopify/quilt/pull/935 taking the third-party approach  and works #934 into the solution.

@TheMallen makes sense, I'm working this issue https://github.com/Shopify/quilt/issues/934 into this PR.

superseding by https://github.com/Shopify/quilt/pull/1002 and https://github.com/Shopify/quilt/pull/985


Adding Nominatim support for geocoder

I added support for Nominatim servers to the geocoder component. By default it does a fallback to Google if no type specified, so it is backwards compatible.
Why add Nominatim?
Google is not only restricted licensed but also requires a key and even a paid user to be able to do geocoding.
The Nominatim option allows you to use your own hosted FOSS geocoder server (https://github.com/osm-search/Nominatim) with your custom geospatial objects (by default, based on OpenStreetMap).
For low traffic sparse geocoding, you can also use the hosted nominating as reference, which is the one used on tests.
Not sure if tests should be disabled by default, as they query a remote server we don't have control over.

The failure is not on the code I pushed. Is it related anyway? Maybe a pull rebase could fix it?

I will need some pointers on the karaf stuff :)

This has been merged and camel-karaf aligned. Thanks for this.

Thanks!


How to motivate a border-top of an element with class="table"?

It seems like doksofts ckeditor plugin for bootstrap adds the class "table" to tables.
In the ckeditor css, there is some code that adds a top border to elements of that class:
.table>thead>tr>th,
.table>tbody>tr>th,
.table>tfoot>tr>th,
.table>thead>tr>td,
.table>tbody>tr>td,
.table>tfoot>tr>td
{
padding: 8px;
line-height: 1.42857143;
vertical-align: top;
border-top: 1px solid #ddd;
}
What is motivating this border-top? Wouldn´t it be easier for sites that wants to have such a top border to include that instead of every other site to exclude it?
Did doksoft do it right when they added the table class to tables?

Firstly, regarding the class of .table - yes, as per the documentation http://getbootstrap.com/css/#tables tables should be given an explicit class of .table to be given the bootstrap look and feel.
Secondly: the look and feel of bootstrap includes the use of top border. If you (or any other site) don't like it...don't add the class of .table, or override it. Unless I'm missing your question here...

Thanks for the quick reply! Seems like I have to add yet another !important css rule to all sites. No worries!

@patrickhlauke I did not mean to hurt anyones feelings when I asked how to "motivate" something. It is just that I appreciate the core functionality of bootstrap of dividing a page in columns, rearranging columns when the browser width changes without appreciating all of the look-and-feel decisions. It would be great if the css was divided into core and look-and-feel. Thats all. Plus that I was seriously surprised of a top border this day.

sorry, i probably misread the tone of your original question (and i probably hadn't had enough coffee in me this morning).

It would be great if the css was divided into core and look-and-feel

Ah, now I see what you mean. I think there will be a lot of cases where it's not easy to make a clear split between these two (to many, some of the basic styling parts that define the look itself are considered core as well, as they normalise the appearance of certain structures across browsers, for instance). I'll pass this suggestion on, though :)

:+1:


Show user status in UserScreen, PeopleAutocomplete and right drawer.

Use presence to get status.

Looks good.


KAFKA-16047: Leverage the fenceProducers timeout in the InitProducerId

KAFKA-16047: Leverage the fenceProducers timeout in the InitProducerId
This is expected to respect the timeout that fenceProducers have, in the InitProducerId request.
Committer Checklist (excluded from commit message)

[x] Verify design and implementation
[x] Verify test coverage and CI build status
[x] Verify documentation (including upgrade notes)

cc. @gharris1727

This PR seems stale, please see this alternative https://github.com/apache/kafka/pull/16151


Enhancement : Dark Mode as a config choice

The Mode follows the theme or allows you to chose it on the page.
I was wondering if it was possible to have a config setting such as mode=Light or Dark that you can define as an environment variable in Docker.

But should we force color schemes on users? it already follows the user preference in theory. I'm not opposed to the idea either in general, but would not enable by default.

Sorry.  I don't understand.  I was not suggesting we force anything.    You currently have a default and I'm not suggesting we change that just that we have the setting easily changed by an environment variable to make things easier.
I like being able to config everything via environment variables rather than a simple control panel that will take a lot more coding and isn't really necessary.


wrong etextIsVolume for Tengyur

bdr:UT23703_1321_0000 has bdo:eTextIsVolume  1321 ; which is obviously wrong

also the title "[5]" is just useless


Portrate mode infinite loop on android

When i run in portrate mode, I have infinite loop..
Landscape mode is ok.

similar to #3937 ?

I think we'd need a some kind of logcat output and some minimal example code.

Abandoned and possible duplicate.


Quest: field validation occurs before user enters any data

Describe the bug
When a form is first opened, all the required fields are validated and highlighted in red before the user enters any data.
To Reproduce
Steps to reproduce the behavior:

Open the quest app
Click the register patient button
When the form opens, there are validation errors for all required fields

Expected behavior
No validation should take place when the form is first opened. Validation of each field should only take place after a user enters some data into that field and moves to the next field.
Screenshots

Smartphone (please complete the following information):

Device: Google Pixel XL Emulator
Android version: 10
Build number: sdk_gphone_x86-userdebug 10 QSR1.210802.0017603624 dev-keys
Version: 29

Additional context
This wasn't happening in Quest v0.0.1

This is coming from SDK as it get validated by questionnaire-viewmodel from data-capture

https://github.com/google/android-fhir/issues/1001 SDK issue

This has been resolved


How to Generate Number of Effective Sequences (neff) of an MSA in Mmseqs

Please how do I generate neff of an MSA using Mmseqs? I saw something about that in page 57 of the user guide but not much was discussed on how to get it. Can I get a help on this? Thanks.

Currently you cannot do that easily with MMseqs2. The easiest option would be to pass the MSA to hhmake from HHsuite. It will add the MSA Neff to the HMM output.

Also, (if I could ask a slightly different question on this ticket) please I would to confirm if the MSA generated from the results of an mmseqs search using mmseqs result2msa is sufficient or that using a standalone MSA generation tool like ClustalOmega is more ideal. Thanks.

For a research project, I also need to calculate Neff scores. Since I wanted to use mmseqs2 for this task, I took a look at the code. To my understanding, the per-residue Neff scores are extracted in the sequences when calling the mapSequence method on a Sequence object. I did not find an endpoint that is capable of writing the Neff scores directly to a file or stdout. (I am looking for something like the profile2pssm command.)
Because I did not find the intended functionality, I implemented an additional command called profile2neff that writes a list of Neff scores to a file. The command is written similar to profile2pssm. The code can be found in https://github.com/neftlon/MMseqs2/commit/a601a968e750122dfdc12f8293be44fba4c5b037.
The command iterates over all columns in the query sequence and writes the column's neffM score to a DBWriter.
@milot-mirdita, is this a sensible way of extracting the scores from a profile? Can/should I create a PR containing my changes?

Very cool, thanks a lot @neftlon! Please make a PR and ill take a closer look. I think it's not quite right yet. We encode the Neff float value in a char. There are these two functions in MathUtil.h:
    static char convertNeffToChar(const float neff) {
        float retVal = std::min(255.0f, 1.0f+64.0f*flog2(neff) );
        return std::max(static_cast<unsigned char>(1), static_cast<unsigned char>(retVal + 0.5) );
    }

    static float convertNeffToFloat(unsigned char neffToScale) {
        float retNeff = fpow2((static_cast<float>(neffToScale)-1.0f)/64.0f);;
        return retNeff;
    }

I think you have to use them to get back a real float value. But I haven't looked at this code in quite some time so I am not 100% sure.

Thanks for the hint, I think you are right that the convertNeffToChar function should be used for conversion back from float.
I am going to create a PR, than we can discuss that there!


Add -x option allowing files to be excluded from shared.html

Add -x option
Fix bug that /tmp folder is created in the wrong place when root option is specified
Clean /tmp folder or working folder when process is completed. Without the fix, if the process end in error before the /tmp folder will exist and user has to delete manually before next run.

I signed it!

exclude could be very useful option. It would allow us to generate root app-shell bundle containing some deps, and to exclude those deps from shared.html used by page bundles.
I suggest including this to v2 and publishing final release even if w-c-s is unlikely to be maintained. In our team, we really need this stuff, while CLI does not yet fit our needs.
@garlicnation @ankon WDYT?

@web-padawan That is the exact use case we have. We load some dependencies from app-shell and then each page load shared.html + page level dependencies.

Sounds quite reasonable to me ... and, I think I stumbled over such a case in our app as well.

Mostly LGTM! Nice work!
I'd like to keep the default behavior around the workdir the same for now, then warn about it changing, then change it. If you put overwriting the workdir behind a flag, this looks good to merge.

I can add a flag to blow away the workdir.

@clementchi would you implement that so the PR could be merged?
I wonder if it can be then also merged by @ankon to provide a corresponding gulp plugin support.

@web-padawan that would be the idea, indeed :)

@clementchi what do you think aut adding regex support to exclude option?
E. g. exclude all iron- elements, etc.
And would it be possible to run w-c-s multiple times in that case to create multiple shared bundles like I mentioned in #13 ?

@web-padawan, using regex is a good idea, that is matching the convention. I think the underlying Vulcanizer does not support it, so that we need to generate a list of files base on the regex and send it to Vulcan. I think that can be another PR.

I added -f for blowing away working dir automatically and also aded -o for striping excludes. Please review. Thanks.


:max parameter dosn't work when its value is 1

Version
2.6.11
Reproduction link
https://codepen.io/thomaslwq/pen/abdoZXM
Steps to reproduce
        1. use the component keep-alive
        2. set the max(1) component for cache
        3. the max parameter dosen't work

What is expected?
when the max parameter is 1,it works
What is actually happening?
it doesn't work

Duplicate of https://github.com/vuejs/vue/issues/9972
Please, search for existing issues first 🙂


CVE-2019-20372 causing PCI compliance issues

The current version of nginx being used by nginx-proxy (1.17.5) has a known security issue, as shown here: http://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2019-20372 .  Unfortunately, this issue is causing us to fail our PCI compliance scans.
Any thoughts on when nginx-proxy will be updated to use nginx 1.17.7 or newer?  Or any thoughts on how to manually update the nginx version used so that we can pass our PCI compliance scans?

You can simply modify the version numbers in the Dockerfile and build a new image.

@mxdmedia This issue has been resolved by https://github.com/nginx-proxy/nginx-proxy/pull/1402.

Fixed via #1402

Is it possible to get this pushed to docker hub? I'm using https://github.com/zodern/meteor-up which uses the docker hub image so unfortunately I can't use an image that I build myself. Thanks in advance!


mdx.validate.validateFileLinks does not show diagnostics

Initial checklist

[X] I read the support docs
[X] I read the contributing guide
[X] I agree to follow the code of conduct
[X] I searched issues and couldn’t find anything (or linked relevant results below)

Affected packages and versions
MDX VS Code Extension v1.8.9
Link to runnable example
No response
Steps to reproduce

Open app/message.mdx in the reproduction repo (code below)
Observe that none of the 4 "incorrect path" permutations for the non-existent file mdx2.png report any diagnostics (both images and links, relative paths starting with ./ and "bare" paths without any ./)

## Links

[link relative](./mdx.png) and [link bare](mdx.png)

[link relative - incorrect path](./mdx2.png) and [link bare - incorrect path](mdx2.png)

## Images

![image relative](./mdx.png) and ![image bare](mdx.png)

![image relative - incorrect path](./mdx2.png) and ![image bare - incorrect path](mdx2.png)

Expected behavior
Diagnostics and problems should be reported for non-existent files when mdx.validate.validateFileLinks is set to hint, warning or error
Actual behavior
Diagnostics and problems are not reported for non-existent files when mdx.validate.validateFileLinks is set to hint, warning or error
Runtime
No response
Package manager
No response
OS
macOS
Build and bundle tools
No response

I can observe the same behavior.
In my case, from all available validations, only one works, and that is the link.no-such-reference.
The link.no-such-header-in-file validation also reports, although incorrectly—it reports an error when the header exists.
mdx.mdx file content:
[no-definition] {/* No link definition found: 'no-definition'(link.no-such-reference) */}
[empty-definition]()

[non-existing-file-relative](./not-existing-file)
[non-existing-file-absolute](not-existing-file)

[existing-file-relative](./existing-file.md)
[existing-file-absolute](/existing-file.md)

[existing-file-existing-header](./existing-file.md#existing-header) {/* Header does not exist in file: existing-header(link.no-such-header-in-file) */}
[existing-file-non-existing-header](./existing-file.md#non-existing-header) {/* Header does not exist in file: non-existing-header(link.no-such-header-in-file) */}

[non-existing-file-non-existing-header](./not-existing-file#non-existing-header)
[non-existing-file-non-existing-header](not-existing-file#non-existing-header)
- Header does not exist in file: my_file(link.no-such-header-in-file)

I used the exact same content in an md.md file as control, and all Markdown validations work with plain VS Code Markdown (except for [empty-definition]() not being flagged as no-definition, which I personally think should be flagged as such).
I did this test in a blank VS Code profile that has only MDX extension installed to exclude possible conflicts.


ISPN-11578 Publisher commands can store RequestID as String

https://issues.redhat.com/browse/ISPN-11578

Integrated into master, thanks @ryanemerson !


[BUG]: Dispatches message content occasionally returns null

Link to faulty endpoint
api/v1/dispatches/
The name of your client/application
GWW Bot
Request information
As discussed on Discord this endpoint ocassionally returns null for the message content.
Here is an example
Hard to reproduce
Steps to reproduce
I honestly have no idea, possible ideas that I totally came up with are that an API call to the AH api failing will silently fail 😉
What is expected?
Not null
What is actually happening?
Null

System Info
Any additional comments?
This version of the API is self-hosted. My code has no issue when retrying shortly after (likely after another sync).
Appreciate the work!

as discussed in Discord, this is most likely the ArrowHead API occasionally returning null for messages.
While there's a retry mechanism for HTTP errors, it should also retry if the received messages contain null.


New version: TrajectoryOptimization v0.1.1

Registering package: TrajectoryOptimization
Repository: https://github.com/RoboticExplorationLab/TrajectoryOptimization.jl
Created by: @bjack205
Version: v0.1.1
Commit: 2aeb40e3e91a84e05472910d982ebfe5639e8351
Reviewed by: @bjack205
Reference: https://github.com/RoboticExplorationLab/TrajectoryOptimization.jl/issues/7

Your new version pull request does not meet all of the guidelines for automatic merging.
Specifically, your pull request does not meet the following guidelines:

The following dependencies do not have a compat entry that has an upper bound: BenchmarkTools, BlockArrays, Combinatorics, CoordinateTransformations, DocStringExtensions, Documenter, FFMPEG, Formatting, ForwardDiff, Interpolations, Ipopt, IterTools, MathOptInterface, MeshCatMechanisms, Parameters, PartedArrays, Plots, RecipesBase, RigidBodyDynamics, Rotations, StaticArrays, TimerOutputs, julia

It is important to note that if your pull request does not meet the guidelines for automatic merging, this does not mean that your pull request will never be merged. It just means that your pull request will require manual review by a human.

These guidelines are intended not as requirements for packages but as very conservative guidelines, which, if your new package or new version of a package meets them, it may be automatically merged.

On a separate note, I see that you are registering a release with a version number of the form v0.X.Y.
Does your package have a stable public API? If so, then it's time for you to register version v1.0.0 of your package. (This is not a requirement. It's just a recommendation.)
If your package does not yet have a stable public API, then of course you are not yet ready to release version v1.0.0.

[noblock]

Please fix the compatibility bounds for the next release such that it will be automatically merged.


Installation of the correct requirements

Hi all,
Thanks for this incredible project.
I am able to make the model run using the FSL command line, and now I wanted to try to make also the github source code work on my laptop. I am facing issues in the moment when I try to pip install the requirements, and I get this huge error :
ERROR: Command errored out with exit status 1:
command: /home/carlo/miniconda3/bin/python /home/carlo/miniconda3/lib/python3.9/site-packages/pip/_vendor/pep517/in_process/_in_process.py prepare_metadata_for_build_wheel /tmp/tmp5hj9q35a
cwd: /tmp/pip-install-kg4x1st3/numpy_56d0f5d74adf427d9bacc56e1a100c9f
Complete output (61 lines):
Running from numpy source directory.
:461: UserWarning: Unrecognized setuptools command, proceeding with generating Cython sources and expanding templates
/tmp/pip-install-kg4x1st3/numpy_56d0f5d74adf427d9bacc56e1a100c9f/tools/cythonize.py:75: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
required_version = LooseVersion('0.29.14')
/tmp/pip-install-kg4x1st3/numpy_56d0f5d74adf427d9bacc56e1a100c9f/tools/cythonize.py:77: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
if LooseVersion(cython_version) < required_version:
warning: /tmp/pip-install-kg4x1st3/numpy_56d0f5d74adf427d9bacc56e1a100c9f/numpy/init.pxd:17:0: The 'DEF' statement is deprecated and will be removed in a future Cython version. Consider using global variables, constants, and in-place literals instead. See https://github.com/cython/cython/issues/4310
warning: /tmp/pip-install-kg4x1st3/numpy_56d0f5d74adf427d9bacc56e1a100c9f/numpy/init.pxd:17:0: The 'DEF' statement is deprecated and will be removed in a future Cython version. Consider using global variables, constants, and in-place literals instead. See https://github.com/cython/cython/issues/4310
warning: /tmp/pip-install-kg4x1st3/numpy_56d0f5d74adf427d9bacc56e1a100c9f/numpy/init.pxd:17:0: The 'DEF' statement is deprecated and will be removed in a future Cython version. Consider using global variables, constants, and in-place literals instead. See https://github.com/cython/cython/issues/4310
warning: /tmp/pip-install-kg4x1st3/numpy_56d0f5d74adf427d9bacc56e1a100c9f/numpy/init.pxd:17:0: The 'DEF' statement is deprecated and will be removed in a future Cython version. Consider using global variables, constants, and in-place literals instead. See https://github.com/cython/cython/issues/4310
Error compiling Cython file:
------------------------------------------------------------
...
        for i in range(1, RK_STATE_LEN):
            self.rng_state.key[i] = val[i]
        self.rng_state.pos = i

        self._bitgen.state = &self.rng_state
        self._bitgen.next_uint64 = &mt19937_uint64
                                   ^
------------------------------------------------------------

_mt19937.pyx:138:35: Cannot assign type 'uint64_t (*)(void *) except? -1 nogil' to 'uint64_t (*)(void *) noexcept nogil'
Processing numpy/random/_bounded_integers.pxd.in
Processing numpy/random/mtrand.pyx
Processing numpy/random/_common.pyx
Processing numpy/random/_bit_generator.pyx
Processing numpy/random/_mt19937.pyx
Traceback (most recent call last):
  File "/tmp/pip-install-kg4x1st3/numpy_56d0f5d74adf427d9bacc56e1a100c9f/tools/cythonize.py", line 238, in <module>
    main()
  File "/tmp/pip-install-kg4x1st3/numpy_56d0f5d74adf427d9bacc56e1a100c9f/tools/cythonize.py", line 234, in main
    find_process_files(root_dir)
  File "/tmp/pip-install-kg4x1st3/numpy_56d0f5d74adf427d9bacc56e1a100c9f/tools/cythonize.py", line 225, in find_process_files
    process(root_dir, fromfile, tofile, function, hash_db)
  File "/tmp/pip-install-kg4x1st3/numpy_56d0f5d74adf427d9bacc56e1a100c9f/tools/cythonize.py", line 191, in process
    processor_function(fromfile, tofile)
  File "/tmp/pip-install-kg4x1st3/numpy_56d0f5d74adf427d9bacc56e1a100c9f/tools/cythonize.py", line 80, in process_pyx
    subprocess.check_call(
  File "/home/carlo/miniconda3/lib/python3.9/subprocess.py", line 373, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['/home/carlo/miniconda3/bin/python', '-m', 'cython', '-3', '--fast-fail', '-o', '_mt19937.c', '_mt19937.pyx']' returned non-zero exit status 1.
Cythonizing sources
Traceback (most recent call last):
  File "/home/carlo/miniconda3/lib/python3.9/site-packages/pip/_vendor/pep517/in_process/_in_process.py", line 349, in <module>
    main()
  File "/home/carlo/miniconda3/lib/python3.9/site-packages/pip/_vendor/pep517/in_process/_in_process.py", line 331, in main
    json_out['return_val'] = hook(**hook_input['kwargs'])
  File "/home/carlo/miniconda3/lib/python3.9/site-packages/pip/_vendor/pep517/in_process/_in_process.py", line 151, in prepare_metadata_for_build_wheel
    return hook(metadata_directory, config_settings)
  File "/tmp/pip-build-env-h0s7068_/overlay/lib/python3.9/site-packages/setuptools/build_meta.py", line 380, in prepare_metadata_for_build_wheel
    self.run_setup()
  File "/tmp/pip-build-env-h0s7068_/overlay/lib/python3.9/site-packages/setuptools/build_meta.py", line 487, in run_setup
    super(_BuildMetaLegacyBackend,
  File "/tmp/pip-build-env-h0s7068_/overlay/lib/python3.9/site-packages/setuptools/build_meta.py", line 338, in run_setup
    exec(code, locals())
  File "<string>", line 488, in <module>
  File "<string>", line 469, in setup_package
  File "<string>", line 275, in generate_cython
RuntimeError: Running cythonize failed!
----------------------------------------

WARNING: Discarding https://files.pythonhosted.org/packages/84/1e/ff467ac56bfeaea51d4a2e72d315c1fe440b20192fea7e460f0f248acac8/numpy-1.18.2.zip#sha256=e7894793e6e8540dbeac77c87b489e331947813511108ae097f1715c018b8f3d (from https://pypi.org/simple/numpy/) (requires-python:>=3.5). Command errored out with exit status 1: /home/carlo/miniconda3/bin/python /home/carlo/miniconda3/lib/python3.9/site-packages/pip/_vendor/pep517/in_process/_in_process.py prepare_metadata_for_build_wheel /tmp/tmp5hj9q35a Check the logs for full command output.
ERROR: Could not find a version that satisfies the requirement numpy==1.18.2 (from versions: 1.3.0, 1.4.1, 1.5.0, 1.5.1, 1.6.0, 1.6.1, 1.6.2, 1.7.0, 1.7.1, 1.7.2, 1.8.0, 1.8.1, 1.8.2, 1.9.0, 1.9.1, 1.9.2, 1.9.3, 1.10.0.post2, 1.10.1, 1.10.2, 1.10.4, 1.11.0, 1.11.1, 1.11.2, 1.11.3, 1.12.0, 1.12.1, 1.13.0, 1.13.1, 1.13.3, 1.14.0, 1.14.1, 1.14.2, 1.14.3, 1.14.4, 1.14.5, 1.14.6, 1.15.0, 1.15.1, 1.15.2, 1.15.3, 1.15.4, 1.16.0, 1.16.1, 1.16.2, 1.16.3, 1.16.4, 1.16.5, 1.16.6, 1.17.0, 1.17.1, 1.17.2, 1.17.3, 1.17.4, 1.17.5, 1.18.0, 1.18.1, 1.18.2, 1.18.3, 1.18.4, 1.18.5, 1.19.0, 1.19.1, 1.19.2, 1.19.3, 1.19.4, 1.19.5, 1.20.0, 1.20.1, 1.20.2, 1.20.3, 1.21.0, 1.21.1, 1.21.2, 1.21.3, 1.21.4, 1.21.5, 1.21.6, 1.22.0, 1.22.1, 1.22.2, 1.22.3, 1.22.4, 1.23.0rc1, 1.23.0rc2, 1.23.0rc3, 1.23.0, 1.23.1, 1.23.2, 1.23.3, 1.23.4, 1.23.5, 1.24.0rc1, 1.24.0rc2, 1.24.0, 1.24.1, 1.24.2, 1.24.3, 1.24.4, 1.25.0rc1, 1.25.0, 1.25.1)
ERROR: No matching distribution found for numpy==1.18.2
I remember I faced something like this when trying to install synthseg as well, however the command line you provided over there has been very useful to proceed:
conda create -n synthseg_38 python=3.8 tensorflow-gpu=2.2.0 keras=2.3.1 nibabel matplotlib -c anaconda -c conda-forge
could you provide something like that for synthsr as well? thatd be very useful. thanks!

Hi,
by FSL you mean FreeSurfer right ? ;)
Here it doesn't work because you have python 3.9 and the requirements listed here are for python 3.6. But you can actually use the same virtual environment as for SynthSeg, because both tools use the same libraries/requirements.
Hope this helps
Benjamin


[openssl] Revise vcpkg cmake wrapper

Cherry-picked from #22258.

What does your PR fix?
Fixes usage with CMake 3.4 (tested in #22258).
Sets the input variables used by CMake's Find module to the known locations instead of setting the ignore list.

Which triplets are supported/not supported?

Hmmmmm

Error: vcpkg has crashed; no additional details are available.
The source line is D:\a_work\1\s\src\vcpkg\binarycaching.cpp(586)

https://github.com/microsoft/vcpkg-tool/blob/457b0309cefb58fae0dcb2e4c06de9bd7582967f/src/vcpkg/binarycaching.cpp#L587
Whatever the cause is it certainly isn't this PR :)

Awesome, thanks for addressing my CR and as always thanks for the PR!


plasmid maintenance assay evidence, under ECO:0000006

1. succinct term name: plasmid maintenance assay evidence

2. definition: Evidence from an assay in which a mitotically stable plasmid (or
minichromosome) is introduced into cells, the cells are grown without selection
for several generations, and the proportion of cells that retain the plasmid is
then determined (by comparing colony numbers on selective versus non-selective
media) and the loss per generation calculated.

3. n/a

4. Suggested parent term: I'm confident that it belongs somewhere under
ECO:0000006 'experimental evidence', but beyond that I'm not sure where it
might fit more specifically.

5. Reference for the definition: PMID:6323245

Additional information: synonym: "minichromosome maintenance assay evidence"
EXACT [PMID:6323245]

Transferred and updated from request on the old SourceForge tracker
(https://sourceforge.net/tracker/?func=detail&atid=1127740&aid=
3500434&group_id=36855).

Thanks,
Midori

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 10 Aug 2012 at 10:20

Reopen. This should not be restricted to mitotic cells because the class of assay evidence is relevant to bacteria. For example:
https://www.ncbi.nlm.nih.gov/pubmed/4576229
It might be appropriate (or not) to have parent-child terms to distinguish between eukaryotic and bacterial/archael systems but I'm not sure. I suspect that additional intermediate parents might also be useful in the long run.

We can reopen this issue to discuss.

Hi @jimhu-tamu
We've come up with two options for how to handle this:

We could just remove the phrase "mitotically stable" from the definition. This will make the term more broad and focus the evidence on the plasmid maintenance part.
We could rephrase so that it works for any kind of cell division. For example: change "mitotically stable" to "...in which a plasmid that is capable of being replicated during cell division is introduced..."

We'd rather not make euk/prok parent terms at this point. Just doesn't seem to be a need and the basic assay is the same in both.
Do one of these options look good to you?

Hi @jimhu-tamu
Just checking if you are ok with one of the above options.
Michelle

Hi @jimhu-tamu ,
Please let us know if one of the above options will work.
Thanks
Michelle

We've decided to go with option 2.
Proposed new def:
"A type of experimental evidence in which a plasmid that is capable of being replicated during cell division is introduced into cells, cells are grown without selection for several generations, and retention of the plasmid is determined."
@jimhu-tamu Let us know if you have any objections. If we don't hear in a week, we will push this change.

sorry for not replying to the earlier ping. My only question is whether "during cell division" is too specific and users will interpret that as replication happening during M phase vs S phase of a eukaryotic cell cycle, or in sync with chromosomal replication.
Why not drop "during cell division" and let 'capable of being replicated" be sufficient?

Sounds good to me.
Final new def:
"A type of experimental evidence in which a plasmid that is capable of being replicated is introduced into cells, cells are grown without selection for several generations, and retention of the plasmid is determined."

The new definition has been added and pushed.
Closing this ticket.
James


State drop-down list appending * ∞

The state drop-down list is appending all of the states into the selections every time the element is clicked.
Click a drop down 3x and you'll end up with 3 sets of states in the selections.

I don't see this issue in effect.


Request to Add Product in Start Flutter

Hello,
I am Maheshwari from team GeekyAnts. On behalf of Start Flutter, we add open source products which we find helpful to the community & also we provide credits to author itself.
Let me know if you are interested showcase your product in our open source website.
Looking forward to hear from you.

Hi Maheshwari,
That's sounds great as the product is just to serve other developers :)
Feel free to publish it on your website, a mention about author sounds nice.
Let me know if you need anything adjusted before publishing ;)
Cheers!
On Fri, 16 Aug 2019 at 11:36, maheshwari-1153<EMAIL_ADDRESS>wrote:

Hello,
I am Maheshwari from team GeekyAnts https://geekyants.com/. On behalf
of Start Flutter https://startflutter.com/, we add open source products
which we find helpful to the community & also we provide credits to author
itself.
Let me know if you are interested showcase your product in our open source
website.
Looking forward to hear from you.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/adamstyrc/calendarro/issues/9?email_source=notifications&email_token=AAO4B2EZPSW6XDNCHHHSKBDQEZYJHA5CNFSM4IMFZY7KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HFTQVVQ,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAO4B2BGVV7HA5TJZUQANOLQEZYJHANCNFSM4IMFZY7A
.


Hotfix/Issue-442

This change is

Current coverage is 58.44% (diff: 77.41%)

Merging #458 into master will increase coverage by 0.14%

@@             master       #458   diff @@
==========================================
  Files          1945       1945
  Lines         99966     100725   +759
  Methods           0          0
  Messages          0          0
  Branches      12411      12665   +254
==========================================
+ Hits          58272      58865   +593
- Misses        38586      38731   +145
- Partials       3108       3129    +21

Powered by Codecov. Last update d759fd2...4d921b2


The given key 'error' was not present in the dictionary.

Some events are missing from ElasticSearch and I'm investigating what is happening. Looking into the selflog I get this error every few minutes:
2019-08-23T17:40:38.4905634Z Exception while emitting periodic batch from Serilog.Sinks.Elasticsearch.ElasticsearchSink: System.Collections.Generic.KeyNotFoundException: The given key 'error' was not present in the dictionary.
   at System.Collections.Generic.Dictionary`2.get_Item(TKey key)
   at Serilog.Sinks.Elasticsearch.ElasticsearchSink.EmitBatch(IEnumerable`1 events)
   at Serilog.Sinks.PeriodicBatching.PeriodicBatchingSink.EmitBatchAsync(IEnumerable`1 events)
   at Serilog.Sinks.PeriodicBatching.PeriodicBatchingSink.OnTick()

Is this a bug or I might be causing it from my code?

Can you give it a go with the latest alpha nuget package? There was some fix in the dynamic error message that is returned from the batch operation.

Op 23 aug. 2019 om 20:21 heeft Natan Vivo<EMAIL_ADDRESS>het volgende geschreven:
Some events are missing from ElasticSearch and I'm investigating what is happening. Looking into the selflog I get this error every few minutes:
2019-08-23T17:40:38.4905634Z Exception while emitting periodic batch from Serilog.Sinks.Elasticsearch.ElasticsearchSink: System.Collections.Generic.KeyNotFoundException: The given key 'error' was not present in the dictionary.
at System.Collections.Generic.Dictionary2.get_Item(TKey key)    at Serilog.Sinks.Elasticsearch.ElasticsearchSink.EmitBatch(IEnumerable1 events)
at Serilog.Sinks.PeriodicBatching.PeriodicBatchingSink.EmitBatchAsync(IEnumerable`1 events)
at Serilog.Sinks.PeriodicBatching.PeriodicBatchingSink.OnTick()
Is this a bug or I might be causing it from my code?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I just got the latest dev branch and ran with debug symbols.
The issue is here: https://github.com/serilog/serilog-sinks-elasticsearch/blob/9d9930e2e5e960dc8c1b230fc7eaaac5c84e3dc9/src/Serilog.Sinks.Elasticsearch/Sinks/ElasticSearch/ElasticSearchSink.cs#L75
Looks like the ["error"] is not present. Should it always be there?

Ok, I got what is happening.
The response from ElasticSearch contains all items, not only the ones with errors. This is part of the response:

While DynamicResponse uses TryGetValue for the "index" part, the value itself is another dictionary and that one complains about trying to get "error" without a check.

Hmm, that is unfortunate, I recently merged this one (https://github.com/serilog/serilog-sinks-elasticsearch/pull/266/files) into dev. So if you tried the dev branch, then it still has issues.
Would you care for a PR?

See #271

Is this still an issue with the latest nuget package?

I was having the same issue with 8.0.0.  Updated to alpha0002 and no longer have the problem.


swiping the app away on recent apps kills background

The persistent notification disappears whenever the application is closed via the recent app menu on android (did not test ios). Does this mean that the background process is dead or is it still running?

@clement360 Yes, swiping the app away destroys the background process. Without the webview, the service would have nowhere to send the location updates.

@pmwisdom can this plugin work and track locations even when the app is closed? Just like https://github.com/transistorsoft/cordova-background-geolocation-lt this plugin?


Pot update in SynthVoice example

Looks like the old pot values are set to the current pot values inside the UpdateLED function (oldk2 = k2;)
The old values should only update if the pot has moved past the hysteresis value. Since you use ConditionalParameter on multiple possible targets (probably should be abstracted out) you may want to just set a 'bPotMoved' flag and use that to determine if the old value should be updated
Probably best to move that inside the UpdateKnobs function as well for clarity

Hi Chect!
I'm really sorry for the delay in response.
Yes, this is a great suggestion! You're more than welcome to make those changes and submit a PR.
We would greatly appreciate it. Thank you so much!


1.8.4 flagged as malware

Sparrow 1.8.4. on Windows 11
When a wallet is opened is created an ".tmp" file at "...\AppData\Roaming\Sparrow\hwi..."
With 1.8.3 and 1.8.4 versions Bitdefender flags that file as "Gen:Variant.Lazy.456623" (a trojan.lazy/infostealer).
Uploading the file to virustotal.com it gives 28 detections of that variant, so it is not only a Bitdefender false positive. I send a pdf of that detections.
sparrow.pdf

It is (as you say) a false positive. Antivirus programs use heuristics (non exact matching) to try identify files, which means false positives happen. The file the AV is flagging is HWI, a well known program written by @achow101 (a Bitcoin Core maintainer) which has recently has a new release, v2.4.0.
Note the HWI file the AV is flagging can be deterministically built from source, so we can be sure of what it does - the binary file matches exactly.
You can do two things - configure your antivirus to stop monitoring the ...\AppData\Roaming\Sparrow\hwi folder (which doesn’t change) and report this false positive to your AV vendor. With enough reports, hopefully the we can get the AV algorithms to change.
Technical note:
If you wish to confirm that this is not a virus, here are the steps to do so:

Download the windows specific .zip from https://github.com/bitcoin-core/HWI/releases/tag/2.4.0.
Extract hwi.exe from the zip file and compare it against the .tmp file in ...\AppData\Roaming\Sparrow\hwi. You can use a tool like diff to ensure that it is an exact match, byte for byte.
If you wish to go further, you can build hwi.exe yourself deterministically, so you can verify that the release binary is actually compiled from the source. There are instructions to do so here.

Sparrow 1.8.4. on Windows 11
Same issue with McAfee but there is no way to exclude the folder as the AppData\Roaming\ folder is hidden so you can't exclude it. Also noted it only creates file when connecting to sparrow with a Trezor, not Ledger or Passport wallet.

For McAfee users, you can also do the following:

Copy the file McAfee is flagging (hwi2.4.0...tmp) to some other location.
Exclude that file from McAfee scan
Close Sparrow
Open the file AppData\Roaming\Sparrow\config in Notepad
Change the value of the "hwi" entry to point to the file McAfee is now excluding
Save the config file and exit Notepad
Restart Sparrow

This is a great solution until I dump McAfee… Thank you Craig!
On Fri, Mar 15, 2024 at 2:56 AM, craigraw @.***(mailto:On Fri, Mar 15, 2024 at 2:56 AM, craigraw < wrote:

For McAfee users, you can also do the following:

Copy the file McAfee is flagging (hwi2.4.0...tmp) to some other location.
Exclude that file from McAfee scan
Close Sparrow
Open the file AppData\Roaming\Sparrow\config in Notepad
Change the value of the "hwi" entry to point to the file McAfee is now excluding
Save the config file and exit Notepad
Restart Sparrow

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>


Broken links in readme

Hi @mclin
This project sounds promising!
But when I try to find out more, getting dead links...
Tried clicking on the links in the README ... got 404 for all of them.

Same story on the http://twitter.github.io/plumage.js/ page...
try clicking on the form fields link in the Batteries Included section:

I would submit a PR if I could find them in the project...

Hey thanks for the interest. I fixed the links here.
https://github.com/twitter/plumage.js/commit/9463bae66f7b048161f0ceb32d889c7178eb593c
will do a gh-pages deploy shortly.


Consistent Canvas State after render()

Save and Restore canvas state so transform, style changes etc aren't propagated outside of render()

Couple of issues there linting wise (npm run lint... pixi is very strict with it's linting rules... can be a pain but it does maintain consistent standards!)
/home/travis/build/pixijs/pixi.js/src/core/renderers/canvas/CanvasRenderer.js
203:9  warning  'setTransform' is not defined  no-undef
231:1  warning  Trailing spaces not allowed    no-trailing-spaces
Also, wouldn't the ctx.restore() go AFTER the displayObject.renderCanvas(this); call? @ivanpopelyshev ?

@themoonrat I agree with the changes, if he makes it through lint :)
@cawoodm run "npm run build" locally to see if lint breaks, or run "npm run test" to ensure that its at least working.

All I did was add 2 lines here on github directly. I did not clone the repo nor do I know how to build it.

@cawoodm Could you fix this PR based on the notes above or close it and ask @ivanpopelyshev to do it?

Is it OK now?

Never mind, made separate PR with "context.setTransform". Thank you, you'll name will appear in next pixi release :)

you removed "context" from setTransform and "npm run test" was failing because of that.

Oh my!


dns/unbound-plus: add private-domain feature

Introduces Miscellaneous submenu, adding CSVListField for private-domain to clear user questions regarding private IPs in unbound.

Closes https://github.com/opnsense/core/issues/3692

Indeed, looks a bit weird. I also changed it for dnsbl.volt. Thanks for reviewing :)

I'm doing some more testing, I think there's an issue where when DNSBL is disabled the dnsbl.conf is not cleared or deleted although the dnsbl.inc is. This results in the blocks still being in place as unbound.conf still has the line include: "/var/unbound/etc/*.conf" in it; I assume unbound is told to re-read the conf file.

Nice catch. Do you want to PR to my branch or share your snippet or shall I add something similar (next week)?

I'll try a PR to your branch, if that goes pear shaped I'll send you the snippet.

Here's the snippet for dnsbl.sh
# Merge resulting files (/dev/null in case there are none)         if [ -s "/var/unbound/etc/dnsbl.inc" ]; then         cat $(find ${WORKDIR} -type f -name "*.inc") /dev/null > ${DESTDIR}/dnsbl.conf         chown unbound:unbound ${DESTDIR}/dnsbl.conf         else         rm -rf ${DESTDIR}/dnsbl.conf         fi         rm -rf ${WORKDIR}         pluginctl -s unbound restart

Merged, thanks!


Avoid recommending TLS (going to be obsolete)

As I understood it, TLS authentication is going to be deprecated sooner or later.
Therefore, recommending TLS only for this API sounds confusing.

Thanks for this pull request!


Dirty diff: changes no longer removed when staging or committing

We used to hide dirty changes once staged to the index which was a very cool feature. Looks like we lost it in our current insider builds.

Actually it also does not seem to get removed after committing.

Good catch!


fix: import correct module for notarizing

Recently updated electron-notarize module to recommended @electron/notarize, but forgot to update the module name where it's used.
Also fixed one of the functions that has been moved to a different path.

Check failed due to https://github.com/elastic/synthetics-recorder/issues/338

/test


Changed the Slack Notifier Schedule Hour

Questions
Answers

Description?
Changed the Slack Notifier Schedule Hour

Type?
improvement

BC breaks?
no

Deprecations?
no

Fixed ticket?
N/A

How to test?
N/A

Possible impacts?
N/A

Thank you @Progi1984


config, balance: support label-based balance

What problem does this PR solve?

Issue Number: close #538
Problem Summary:
Support label-based balance. TiProxy should rebalance the connections to the local backends.
What is changed and how it works:

Add config balance.label.label-name which indicates which label name should be used
Add a new factor FactorLabel
Add field Labels to BackendInfo so that FactorLabel can know the labels of backends
Fix the bug of FactorCPU that it may not update the scores when metrics are unavailable

Check List
Tests

[x] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Notable changes

[x] Has configuration change
[ ] Has HTTP API interfaces change
[ ] Has tiproxyctl change
[ ] Other user behavior changes

Release note

Please refer to Release Notes Language Style Guide to write a quality release note.
- Support label-based balance

Codecov Report
Attention: Patch coverage is 77.77778% with 8 lines in your changes are missing coverage. Please review.

:exclamation: No coverage uploaded for pull request base (main@0493bc8). Click here to learn what that means.

Files
Patch %
Lines

pkg/balance/factor/factor_label.go
77.77%
6 Missing :warning:

pkg/balance/factor/factor_balance.go
0.00%
1 Missing :warning:

pkg/balance/factor/factor_cpu.go
66.66%
1 Missing :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #539   +/-   ##
=======================================
  Coverage        ?   65.83%
=======================================
  Files           ?       72
  Lines           ?     6497
  Branches        ?        0
=======================================
  Hits            ?     4277
  Misses          ?     1901
  Partials        ?      319

Flag
Coverage Δ

unit
65.83% <77.77%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Added BigDecimal capabilities to SpEl numeric operators

Hi.
Some time ago, we found that the BigDecimals was not considered in the supported operations by  Expression Language. I also opened an issue: https://jira.springsource.org/browse/SPR-8716
If you are interested, we'd like to share our enhancement.
The updated operators are:

gt
ge
lt
le
eq
ne
plus
minus
divide
multiply
power
modulus

@aclement, please take a look.
On Mar 7, 2012, at 5:09 PM<EMAIL_ADDRESS>wrote:

Hi.
Some time ago, we found that the BigDecimals was not considered in the supported operations by  Expression Language. I also opened an issue: https://jira.springsource.org/browse/SPR-8716
If you are interested, we'd like to share our enhancement.
The updated operators are:

gt
ge
lt
le
eq
ne
plus
minus
divide
multiply
power
modulus

You can merge this Pull Request by running:
git pull https://github.com/giovannidalloglio/spring-framework master
Or you can view, comment on it, or merge it online at:
https://github.com/SpringSource/spring-framework/pull/54
-- Commit Summary --

Added BigDecimal capabilities to SpEl numeric operators

-- File Changes --
M spring-expression/src/main/java/org/springframework/expression/spel/ast/OpDivide.java (13)
M spring-expression/src/main/java/org/springframework/expression/spel/ast/OpEQ.java (15)
M spring-expression/src/main/java/org/springframework/expression/spel/ast/OpGE.java (14)
M spring-expression/src/main/java/org/springframework/expression/spel/ast/OpGT.java (14)
M spring-expression/src/main/java/org/springframework/expression/spel/ast/OpLE.java (14)
M spring-expression/src/main/java/org/springframework/expression/spel/ast/OpLT.java (14)
M spring-expression/src/main/java/org/springframework/expression/spel/ast/OpMinus.java (14)
M spring-expression/src/main/java/org/springframework/expression/spel/ast/OpModulus.java (9)
M spring-expression/src/main/java/org/springframework/expression/spel/ast/OpMultiply.java (9)
M spring-expression/src/main/java/org/springframework/expression/spel/ast/OpNE.java (14)
M spring-expression/src/main/java/org/springframework/expression/spel/ast/OpPlus.java (12)
M spring-expression/src/main/java/org/springframework/expression/spel/ast/OperatorPower.java (10)
M spring-expression/src/test/java/org/springframework/expression/spel/OperatorTests.java (183)
-- Patch Links --
https://github.com/SpringSource/spring-framework/pull/54.patch
https://github.com/SpringSource/spring-framework/pull/54.diff

Reply to this email directly or view it on GitHub:
https://github.com/SpringSource/spring-framework/pull/54


Upgrade openssl-1.0.1k

It would be nice to make a release after this. Thx!

I’ll cut a release now.

:+1:


[TEST] Fix backup_labels_test failure

What's the test to develop? Please describe
After https://github.com/longhorn/longhorn-manager/commit/e2de327809ceaab667d00c882673496b70ff2cd5 merged
Longhorn will auto create new label "longhorn.io/volume-access-mode":"<rwo/rwx>" in backup
Some test case assert the backup label counts and that count did not include new label LonghornLabelVolumeAccessMode
Need to modify test script to match newest product behavior
Describe the items of the test development (DoD, definition of done) you'd like

[x] Modify script
[ ] Verified on pipeline

Additional context
https://ci.longhorn.io/job/public/job/master/job/sles/job/amd64/job/longhorn-tests-sles-amd64/242/

Most failed case related to backup label were fixed by https://github.com/longhorn/longhorn-tests/pull/1098
Somehow tests.test_backing_image.test_recurring_job_labels_with_backing_image[s3]
[tests.test_recurring_job.test_recurring_job_labels[s3] and (https://ci.longhorn.io/job/public/job/master/job/sles/job/amd64/job/longhorn-tests-sles-amd64/248/testReport/junit/tests/test_recurring_job/test_recurring_job_labels_s3_/) were not use function backup_labels_test to assert label counts, need do solve them one by one

Backup label count assert error not happened after code merged, close this ticket


Use resize instead of reserve on String to avoid index out of range e…

…xception when terminating string.

Thanks, this has been merged with b2eddbed901eed7a7f14bb30c7c20b9eec87df10. I've also removed buf[32] = '\0' line, as it was writing out of range, and also was redundant.


Elements with different root tag names are treated as equal

>>> import lxml.etree
>>> a = lxml.etree.fromstring("<a/>")
>>> b = lxml.etree.fromstring("<b/>")
>>> xtdiff.diff(a, b)
OrderedSet()

Additionally, it seems to not check the tag's contents when the tag name is different:
>>> a = lxml.etree.fromstring("<parent> <child/> </parent>")
>>> b = lxml.etree.fromstring("<parent> </parent>")
>>> xtdiff.diff(a, b)
OrderedSet([DELETE(path='/parent/child')])
>>> a = lxml.etree.fromstring("<parentt> <child/> </parentt>")
>>> xtdiff.diff(a, b)
OrderedSet()

@vozhyk- I think this library is dead, so if you are hoping for someone to help resolve it I don't think you will find help here. Better to just try and dig into it yourself at this point. :(

The library was an experiment that we did not end up using. It can be considered unsupported unless there's someone who wants to fork and adopt it.
That being said, I am trying to dig into some of these issues and resolve them — but it might take a bit of time.


Use app label to as app change label for long app names

What this PR does / why we need it:
Use app label to as app change label for long app names
If app names have more than 63 character, use the app label as app change label instead of the app name as there is a hard limit on the number of characters that can be present in a label value
This is the first step towards fixing #646
Does this PR introduce a user-facing change?

Use app label as app-change label if app name has more than 63 characters

Additional Notes for your reviewer:
Review Checklist:

[ ] Follows the developer guidelines
[ ] Relevant tests are added or updated
[ ] Relevant docs in this repo added or updated
[ ] Relevant carvel.dev docs added or updated in a separate PR and there's
a link to that PR
[ ] Code is at least as readable and maintainable as it was before this
change

Additional documentation e.g., Proposal, usage docs, etc.:

LGTM

Linked the related issue ^


Tame standard logging a bit

With the exception of the closing summary line, most informational
logging messages are pushed to --verbose output.

@johnbender Many of the logging messages produced by grunticon feel like the fall into the verbose or, possibly, the debug category.  This PR moves them to verbose, but in many cases I could really see debug.  Let me know your thoughts.

Definitely verbose, thanks!

:+1:


podman: 3.0.0 -> 3.0.1

https://github.com/containers/podman/releases/tag/v3.0.1

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 113662 at 92f9b27c run on aarch64-linux 1

2 packages built:

podman
podman-unwrapped

1 suggestion:

warning: missing-patch-comment
Please add a comment on the line above, explaining the purpose of this patch.
Near pkgs/applications/virtualization/podman/default.nix:29:5:
   |
29 |     ./remove-unconfigured-runtime-warn.patch
   |     ^


Extension issue

Issue Type: Bug
Extension Name: emmet
Extension Version: 1.0.0
OS Version: Darwin x64 20.6.0
VS Code version: 1.69.1

:warning: We have written the needed data into your clipboard. Please paste! :warning:
! autocompletion feature not working. So far, I've not had much success fixing this, and it's beyond frustrating.
I was able to fix this by going to Code > Preferences > Settings > Emmet > Emmet: Trigger Extensions On Tab | Activate Checkbox

Should be fixed in v1.69.2.  In the meantime, try Cmd+Space.

Thanks Arturo, this actually fixed it here: I was able to fix this by going to Code > Preferences > Settings > Emmet > Emmet: Trigger Extensions On Tab | Activate Checkbox

/duplicate

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


SCT-1147 Implement get_users()

Pull Request Test Coverage Report for Build 51

13 of 15 (86.67%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.6%) to 93.434%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/jgikbase/idmapping/storage/mongo/id_mapping_mongo_storage.py
5
7
71.43%

Totals

Change from base Build 50:
-0.6%

Covered Lines:
185

Relevant Lines:
198

💛  - Coveralls

Pull Request Test Coverage Report for Build 51

13 of 15 (86.67%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.6%) to 93.434%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/jgikbase/idmapping/storage/mongo/id_mapping_mongo_storage.py
5
7
71.43%

Totals

Change from base Build 50:
-0.6%

Covered Lines:
185

Relevant Lines:
198

💛  - Coveralls

This looks good to me. I like the usage of Set.


Blazor debugger unable to find .net assemblies

When I try to open sources in .dll it shows:
// Unable to find document file:///C:/Users/Павлюк Е/Desktop/C# hello world\WebApplication1\Startup.cs

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

ID: ec2f61c7-9f51-6735-e328-3c7b495d70c3
Version Independent ID: 4d06e565-282d-6387-aa90-cb2c2986d35a
Content: Debug ASP.NET Core Blazor
Content Source: aspnetcore/blazor/debug.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

OOF for the :gift: holidays :snowman: and will return on January 6.
If you need support, we recommend contacting devs on support channels and chat:

Stack Overflow
ASP.NET Core Slack
ASP.NET Gitter

Potential bugs in the ASP.NET Core framework and product feedback can be reported at aspnet/AspNetCore issues (include full repro steps and cross-link to a repro project in GH when appropriate). For Visual Studio, use the Report a Problem gesture or Suggest a Feature gesture within VS, which open internal issues for the VS team. For Visual Studio Code, you can open an issue at microsoft/vscode issues.
:dancer::man_dancing: Happy New Year! See you in 2020! :balloon::champagne:

I copied the project folder to C:/, and it works properly now.

I don't know what the underlying issue is there. We haven't received feedback yet on this sort of issue. I just checked engineering issues, and I couldn't find anything related to this error.
If you continue to have trouble, open an issue with engineering. Explain your exact repro steps so they can duplicate your experience.
https://github.com/aspnet/AspNetCore/issues


Cold / Flu: We need a data dictionary for the Influenza database

User Story
As both a front-end and back-end developer I need a guide to the fields available from retrieving and pushing data to and from the data warehouse so that I can build an effective data model and ensure that the field lengths and types are correct in type, name, and length. This will eliminate errors due to misunderstanding the nature of the fields in the database.
Tasks

[x]  Create a spreadsheet (workbook) of all of the tables in the current influenza database with one sheet per table.
[x]  List the fields in each table
[x] the field type
[x] create a description (what it is, and purpose)
[ ] Form team vets content (specify members in the comments below)
[ ] Vet this dictionary with the data currently in the database.

Acceptance Criteria

[ ]  (Who? Eduardo?) A non-specialist in this business problem should understand the data dictionary.
[ ] Caleb vets content
[ ] Nass vets content.

The checklist the product owner uses to decide whether a story is done. Objective and discrete answers (such as a yes/no) that can be tested.

I created the first draft of the data dictionary and sent it to Archer, Stella Anne PSA:EX<EMAIL_ADDRESS>Noorizadeh-Kollou, Nassreen PSA:EX<EMAIL_ADDRESS>Liu, Caleb PSA:EX<EMAIL_ADDRESS><EMAIL_ADDRESS>Ramirez, Eduardo PSA:EX<EMAIL_ADDRESS>Caldwell, Chris PSA:EX<EMAIL_ADDRESS>It is over to your side to review this and discuss next steps with me. I suggest creating a data model from this to show the relationships between the tables and fields.

Bhumin to reach out Patrick/Don about the end points.

@mikeshi-bcgov  here is the ERD (V6)

@ca-liu Thanks

We have finalized table and field names but are still adjusting some field lengths, e.g. first_name, last_name to bring them into line with PeopleSoft.

Don is waiting on @mikeshi-bcgov for word that he has completed changes to the tables/fields. Don will complete the final version of the data dictionary by making updates.

Thanks @mikeshi-bcgov: are the fields stable now in terms of type? i.e. is it safe for me to update the documentation?

@DonMoffatt
Thanks, I would say yes it is safe for now-:)) unless it is asked to adjust from Form developer side.

Almost done: @ca-liu would you please look at the worksites and registration tables and put in short definitions for the few fields there where I have question marks? Also @EduardoRamirezBC  would you like to have a look at https://bcgov.sharepoint.com/:x:/r/teams/03991/Shared Documents/Cold and Flu/3_Design/DD_Influenza_db.xlsx?d=w6314fb159a6b4b2290da8b34af8e6820&csf=1&web=1&e=tOsFbpit? It is located here:

Hey @DonMoffatt  - thanks for the update! I've added my comments to the dictionary.
I am not sure about application_id, I think this perhaps is a duplication of registrations_id

Hello @mikeshi-bcgov do we need the application_id in the registrations table? Did add that? We don't have a definition of what it does.

@DonMoffatt
To add application_id to 2 tables was requested by Bhumin, please refer to task #1000

Hey @mikeshi-bcgov
I believe the application_id reference din ticket #1000 by Bhuman is the same as registration_id. It is the primary key for the registrations table.
I think Bhuman got it from your POST call example, 🙂

Hey @iman-jamali-fw for consistent variable naming and to ensure that the id is captured in the backend, could we please update the variable on the FE from application_id to registration_id?
Here is a copy of our ERD for your reference (final copy - naming convention changes - to be produced)
Thank you both!

Hi @ca-liu - to be clear, can I drop application_id from the data dictionary?

Hi @ca-liu - to be clear, can I drop application_id from the data dictionary?

Hey @DonMoffatt,
Yes, I think we can drop application_id from the data dictionary.
Just to double check, I'll bring this up at stand-up on Monday for Iman to take a look at the Front End code to make sure that it is a duplicate then we can drop it.
Thanks!

@ca-liu
the request for application_id is actually before I have shared the post example #1000
Possibly Bhumin was referencing the same thing, sure need to confirm with them

@ca-liu
the request for application_id is actually before I have shared the post example #1000
Possibly Bhumin was referencing the same thing, sure need to confirm with them

Thank you for the clarification, @mikeshi-bcgov!
Good morning @iman-jamali-fw,
When you get a chance, could you please confirm that application_id and registration_id are the same and that we can change it to registration_id for consistent naming?
Thank you for your help!

@ca-liu
Sure I'll look into it today. I know that application_id is the id of submission, not sure if we can rename it.

@ca-liu
I don't think we can merge application_id and registration_id. They're the primary keys of two different tables one in DGJ and another in the warehouse. I think we should keep both on the registration table.

@ca-liu
I don't think we can merge application_id and registration_id. They're the primary keys of two different tables one in DGJ and another in the warehouse. I think we should keep both on the registration table.

Thanks @iman-jamali-fw! That makes sense to me.
Hey @DonMoffatt, @mikeshi-bcgov
We can keep application_id in the registrations table. This is the form Id generated by FormsFlow and we'll need it to retrieve people's submitted forms (for editting).
Thanks everyone!

Hello @mikeshi-bcgov @iman-jamali-fw I am finalizing the data dictionary and noticed that in the db on dev we have a mismatch between the variable definitions for application_id and what we have in the data dictionary. In the data dictionary we have
application_id	string		not null	This is the primary key for FormsFlow from each form.
and in the db we have:
application_id int null
Which should it be?

@DonMoffatt
It's converted from int (in formio) to string in Camunda, so string will be sent to the warehouse in post/put requests. Please convert it to string if that's not too hard. Thank you

@AbhimanyuRaut There is no string type available. I made it varchar(50) with nulls allowed. Is that ok?

Hello @ca-liu @mikeshi-bcgov and @iman-jamali-fw : I have updated the data dictionary on team share and brought it into alignment with what is in the data warehouse in DEV. Iman and Caleb, would you please have a look and if you are satisfied would you tick the boxes above under "Acceptance Criteria" for your respective confirmations and let me know? Caleb, please compare it to the naming in your ERD.  As noted in my previous comment I changed application_id to int (it was already int in one table).

I've also taken Eduardo and Nass off the approval list for the data dictionary: Nass has not been involved with this part of the project and having Eduardo on as an approver was a mistake on my part: I just wanted him to be aware of it for informational purposes for future, similar projects.

Hey @iman-jamali-fw  (for @bhumin-fw), @mikeshi-bcgov  (cc: @DonMoffatt)
I noticed that there is application_id included in the worksite_ministries table. I am going to assume that this was added in error - there is no relationship between the individual registrations and the bridge table.
the registrations will house the wm_id, the worksite_ministries table does not / cannot store application_id as a record or else a many-to-many scenario is created.
Could you please confirm this? Once confirmed, I will send out a final ERD to the team.
Thanks all!

@ca-liu
I think that's an error. application_id should only be added to the registration table if I understand the schema correctly. Maybe @bhumin-fw can confirm.

@DonMoffatt
For sure.
application_id should be there in worksite and registration table only.
As we are having 2 work flow, add worksite-registration and add registration to existing worksite.

Hi @bhumin-fw - I think your note is opposite to what @ca-liu and @iman-jamali-fw said. They said it should only be in the registration but you said it should be in worksite and registration. Just to confirm, are they wrong? It needs to be in both? Can you explain why it needs to be in both?

@DonMoffatt
For sure.
application_id should be there in worksite and registration table only.
As we are having 2 work flow, add worksite-registration and add registration to existing worksite.

Hey @bhumin-fw , can you help me understand why we are adding application_id to worksites? There can be multiple applications / registrations in one worksite address.

@DonMoffatt @ca-liu
We have 2 table worksite and registration.
Case 1: add/update worksite and registration
On the digital journey side we have form with worksite and registration details.
When we submit form, we will have unique ID (Application_id), and data includes of both table worksite and registrations.
Let's say I did click on edit form on Digital Journey, It will show me full form with same data that I did submit (worksite + registration). So having application_id in worksite and registration table help on ODS endpoint to update both table respectively with updated details.
Case 2: Add/update only registration in existing worksite.
Now let's take other path, we have new form where we want to add only registration on existing worksite.
In this case digital journey form only contain registration related fields and worksite_id. So in this case new registration saved with application_id.
In case of edit same submission on digital journey. It will only allow you to update registration related fields.
We do not need application_id in worksite_ministries table(one of Caleb's message about asking for it).
or any other table except worksite and registration.

@DonMoffatt @ca-liu
We have 2 table worksite and registration.
Case 1: add/update worksite and registration
On the digital journey side we have form with worksite and registration details.
When we submit form, we will have unique ID (Application_id), and data includes of both table worksite and registrations.
Let's say I did click on edit form on Digital Journey, It will show me full form with same data that I did submit (worksite + registration). So having application_id in worksite and registration table help on ODS endpoint to update both table respectively with updated details.
Case 2: Add/update only registration in existing worksite.
Now let's take other path, we have new form where we want to add only registration on existing worksite.
In this case digital journey form only contain registration related fields and worksite_id. So in this case new registration saved with application_id.
In case of edit same submission on digital journey. It will only allow you to update registration related fields.
We do not need application_id in worksite_ministries table(one of Caleb's message about asking for it).
or any other table except worksite and registration.

Hey @bhumin-fw - do you have 10 - 15 mins for a quick chat? I don't think I follow the scenarios described above.
Thanks!

@ca-liu I have one bank appointment at 3:30pm, so need to leave for that. mostly I will be available by 4:30pm onwords.
Does that works for you?
If yes please share meeting invite with me so we can connect at 4:30pm or anytime after that for a quick call.

@ca-liu I have one bank appointment at 3:30pm, so need to leave for that. mostly I will be available by 4:30pm onwords.
Does that works for you?
If yes please share meeting invite with me so we can connect at 4:30pm or anytime after that for a quick call.

Thanks @bhumin-fw. I'll book a quick meeting with you at 4:30 - 4:45PM.

@ca-liu Sure. Please share meeting invite on teams for the same. find my ms teams email below.
<EMAIL_ADDRESS>

sorry @bhumin-fw - I selected the wrong date.
If you are free

Microsoft Teams meeting

Join on your computer, mobile app or room device

Click
 here to join the meeting

Meeting ID:
284 525 300 856

Passcode:
nKi6Nu

Download Teams |

Join on the web

Or call in (audio only)

+1<PHONE_NUMBER>,,223267143#
  Canada, Victoria

Phone Conference ID:
223 267 143#

Find a local number
 |
Reset PIN

Toll-free (audio only): 1-888-600-3289

Learn More |

Meeting options

@ca-liu If you feel that you have the answer from bhumin please feel free to finish documenting that field and close this ticket. Thanks.

@ca-liu
Per discussion, to support soft deletion, a new field "deleted_ind"( tinynt(1) ) has been added to table registrations, you may want to update the ERD

@mikeshi-bcgov : wow! you're too fast!!
Thank you so much :)

For: @mikeshi-bcgov  @bhumin-fw  @DonMoffatt @ccoldwell
cc:  @EduardoRamirezBC  @NassNK  @Karla
Hi everyone,
To ensure consistent naming of the rows across all tables in the RDBMS, please note below name changes the database:

application_id is removed from the worksite_ministries table
is_admin is removed from the contacts table
date_updated is now consistent in all tables (including registrations table)
deleted_ind is added to the registrations table for soft delete

Kindly double check your input and output params to match the new names.
Platforms to check for check:

MySQL
MS SQL Server
API endpoints
Front end

Please let me know once your concerning platform has been updated with the new variable names and I will close out this ticket
Thanks everyone for your patience while we finalize this document.
Attachments:
ERD:
https://app.zenhub.com/files/439477060/e7d81952-db87-48b1-be5f-747cf148e42f/download
Data Dictionary
https://app.zenhub.com/files/439477060/20ae0280-e916-497e-8083-79e37024012f/download

Hi @ca-liu and @iman-jamali-fw : when you are satisfied with data dictionary would you tick your respective boxes in the checklist on this ticket and let me know so I can close it?

Hey @DonMoffatt , thanks for checking in. I am good with the final data dictionary and ERD, I just want to make sure that the applications have been updated to reflect the final ERD.
Should I tick off my box?

@ca-liu
Looks good to me.

@ca-liu
Looks good to me.

Thanks for confirming, @iman-jamali-fw . I'm assuming that the variables in the FE / FormsFlow are in alignment with the table rows on the ERD?

As far as I know, they are.

Perfect!


Adversarial competition - preparing directory structure for release of evaluation code

Three subdirectories created in competition directory:

dataset - subdirectory for dataset
dev_toolkit - development toolkit moved to this subdirectory
eval_infra - placeholder directory for evaluation infrastructure

I think build is broken for some reason. This code does not touch anything in core cleverhans, so it should not affect core cleverhans tests.
I also noticed that tests are passing for TF version 1.4.1, but failing for TF version 1.1.0

Yes, that's right TF 1.1 build is failing, see here: https://github.com/tensorflow/cleverhans/pull/370

@goodfeli build is fixed and all tests are passing now, please take a look at this PR


Support baggage

https://github.com/ndrwrbgs/EventHookTracer/blob/011f08162acfdb1c0da7de818f73bd3cb7d48cae/src/Library/EventHookSpan.cs#L114
Already the known dependee needed baggage - so we will need to support it

Also here
https://github.com/ndrwrbgs/EventHookTracer/blob/011f08162acfdb1c0da7de818f73bd3cb7d48cae/src/Library/EventHookSpan.cs#L148

https://github.com/ndrwrbgs/EventHookTracer/blob/011f08162acfdb1c0da7de818f73bd3cb7d48cae/src/Library/EventHookTracer.cs#L22


Update README.md bug

Description
In the table of contents in readme file, the onclick functionality is not added to the Contributors title.
Even after clicking, the page does not route to contributors section.
Also on clicking the license option, no functionality is added. The license section needs to be added.
Screenshots

Additional information
Please assign me the task under GSSoC'23.

completed via #156


Play'A VR Player works.

Play'A VR Player (from Steam) works per:
PimaxXR: Yes
OpenXR Toolkit: Yes
Need OpenComposite: No
Need Parallel Projection: No
It's probably the only VR video player that works without using SteamVR.

Thank you, added to compatibility page!


@fetch-messages fired only when room is opened

Hi, @fetch-messages event is fired only once when I open the room. Also the chat window is not scrolled to the bottom of the page when first set of messages is loaded. I have tried to change the message amount being loaded but that does not help. Here is my component:
      <chat-window
          height="93.5vh"
          theme="light"
          :styles="{
              icons: {
                search: '#ff2f00'
              },
              message: {
                backgroundMe: '#f1e7c0',
              }
            }
            "
          :current-user-id="authUser.chat_user.id"
          :rooms="rooms"
          :room-id="chat_room_id"
          :load-first-room="false"
          :single-room="true"
          :messages="messages"
          :rooms-loaded="roomsLoaded"
          :loading-rooms="loadingRooms"
          :show-audio="false"
          :messages-loaded="messagesLoaded"
          :room-info-enabled="true"
          @room-info="roomInfo($event)"
          @fetch-more-rooms="fetchRooms"
          @fetch-messages="fetchMessages($event)"
          @send-message="sendMessage($event)"
          :text-messages="{
            ROOMS_EMPTY: 'Ei keskusteluja',
            ROOM_EMPTY: 'Keskustelussa ei ole viestejä',
            NEW_MESSAGES: 'Uusia viestejä',
            MESSAGE_DELETED: 'Viesti poistettu',
            MESSAGES_EMPTY: 'Viestit tyhjiä',
            CONVERSATION_STARTED: 'Keskustelu alkoi:',
            TYPE_MESSAGE: 'Kirjoita viesti',
            SEARCH: 'Etsi',
            IS_ONLINE: 'Online-tilassa',
            LAST_SEEN: 'viimeksi nähty ',
            IS_TYPING: 'kirjoittaa viestiä...',
            CANCEL_SELECT_MESSAGE: 'Peruuta viesti'
          }"
      >
      </chat-window>

and here is the fetch:
    fetchMessages({ room, options }){

      console.log('fetchMessages options: ', options)

      if(options && options.reset){
        this.messages = [] //reset messages on load of new chat page
        this.currentRoomId = room.roomId //set as selected room
        this.messagesLoaded = false
      }

      this.axios
          .get('/chat/messages/app', {params: {
              chat_room_id: room.roomId,
              skip: this.messages.length,
            }
          })
          .then(response => {

            var messages = []
            messages = this.messages

            for(var i = 0;i < response.data.length;i++)
                messages.unshift(response.data[i]); //push messages to the bottom of the array

            this.messages = messages //set as message array

            //if less than 20 messages in response, mark chatroom as fully loaded
            if(response.data.length < 20) this.messagesLoaded = true

          })

    },

Using Vue 3 installed
Expected behavior
The chat screen should be scrolled to the bottom and when scrolling to the top @fetch-messages should be fired again

Please use v2.0.1, previous versions are not supported anymore. Let me know if the issue still exists then.
Migration guide: https://github.com/antoine92190/vue-advanced-chat/releases/tag/2.0.0

Please try this.messages = [...messages] instead of this.messages = messages
Otherwise, can you share your repo or a minimal example so I can reproduce on my side?

Actually there was indeed a bug, fixed in https://github.com/antoine92190/vue-advanced-chat/commit/8d87952205aef700ce18d1696f28e6b38b311c8e
Please install version 2.0.2 and let me know if it works.

It works now! Thank you for the fix and thanks again for a great repo!


mwc-tab-bar not work fine in firefox

I use mwc-tab-bar like the following code
  <div style="display: inline-block; max-width: 100%">
    <mwc-tab-bar activeindex="${this._cateid}">${this.renderMenu()}</mwc-tab-bar>
  </div>

on firefox  68.0.1 It show like the following image
firefox 68.0.1 result

Is the problem that the horizontal scrollbar is visible when not needed, or something else?
I don't get the same result in Firefox 68.0 or 70.0.1 using https://codepen.io/asyncliz/pen/gOOBwrw?editors=1000#0.
Does the issue occur if you update to the latest <mwc-tab-bar> and Firefox version? If so, can you provide a full example?

Closing due to inactivity, let us know if you're still having trouble with the latest version of Firefox


Displacement Control - Application of equivalent forces for the LinearAnalyzer

The Linear analyzer needs to be modified so that before the call to solver.Solve(), so that the equivalent forces (or, equivalently,  internal forces) are calculated and transferred to the right hand side.

Added by #123


Fixes Network address parameter #24

@SvenAelterman does it make sense to add the vnetAddressSpace to the -sample.bicepparam file?

@SvenAelterman does it make sense to add the vnetAddressSpace to the -sample.bicepparam file?

Duh... Yes. Doing that.


second/outer skin layer for hd skin not showing up

Read other answers that told me to just add a second layer through tools> Add 2nd Layer. I've done so but I can't see how to actually add the textures for that layer. Also am I supposed to be sizing up and offsetting the second layer (hats/jackets/sleeves) if I want it to show up? My lower layer shows up just fine but I can't even figure out how to edit the 2nd layer I've added, let alone how to give it the correct textures.

You have to edit the newly created cubes to match your texture sheet.
Please join the discord for support questions: https://discord.gg/mKyXdEsMZD
This site is for crash/bug reports and suggestions.


Rewrite readme

As I refactoring the readme, I think the "why should I care" part is what's currently missing.

ping @hongchaodeng :-)

@hongchaodeng Fixed, thanks for review!

LGTM
It's a lot prettier and cleaner.